Range statement for test ... does not use range value in t.Run linter is not always correct

Following piece of code currently triggers the linter, even though it is a valid code.
func TestFunctionRangeNotUsingRangeValueInTDotRun(t *testing.T) {
	t.Parallel()

	testCases := map[string]struct {
		value string
	}{{value: "foo"}}
	for n, tc := range testCases {
		t.Run(n, func(t *testing.T) {
			t.Parallel()
			fmt.Println(tc.value)
		})
	}
}

I have a similar example, that I consider to be valid.
func TestFunctionRangeNotUsingRangeValueInTDotRun(t *testing.T) {
	t.Parallel()

	testCases := []struct {
		Name     string
		Sequence int
	}{{Name: "foo", Sequence: 1337}}

	for _, tc := range testCases {
		tc := tc
		t.Run(fmt.Sprintf("%v->%d", tc.Name, tc.Sequence), func(t *testing.T) {
			t.Parallel()
			// ...
		})
	}
}

I think, it false positive with nested for range like this:
func TestValidateStateSwitchInconcistency(t *testing.T) {
	t.Parallel()

	transitions := map[models.TaskState][]models.SwitchTaskStateEvent{
		unknownState: {
			models.TodoTaskEvent,
			models.DoneTaskEvent,
			models.CancelTaskEvent,
			models.ToWorkTaskEvent,
			unknownEvent,
		},
		models.TaskStateSimple: {
			models.ToWorkTaskEvent,
			unknownEvent,
		},
		// ...
	}

	for state, tr := range transitions {
		for _, ev := range tr {
			state := state
			ev := ev
			t.Run(string(state)+" -> "+ev.String(), func(t *testing.T) {
				t.Parallel()
				// ...
			})
		}
	}
}

One more false positive example:
func TestNodeType_String(t *testing.T) {
	test := func(nt NodeType, want string) func(*testing.T) {
		return func(t *testing.T) {
			t.Parallel()
			...
		}
	}

	tests := []struct {
		name string
		nt   NodeType
		want string
	}{
		...
	}

	t.Parallel()
	for _, tt := range tests {
		t.Run(tt.name, test(tt.nt, tt.want))
	}
}

The linter tries to determine the name of the variable used in the first Run parameter, and then it checks whether there are any statements initializing that variable from the range's value identifier. It expects the expression in the Run parameter to be of the form var.property, and then it returns the name of var. All the examples in this issue are cases where the expression doesn't match that pattern.
A different technique would be to identify the range variable first, then the variable(s) initialized from that. Then, check whether those variables are used anywhere in the Run parameter's expression. It would provide a pretty good estimate of whether the Run name depends on the test case. It can never be perfect, though.

I've just started to apply this linter, and the pattern in the original post (looping over a map of test cases keyed by the test name) is so common that the number of false positives is unmanageable. Here's a good piece on why this pattern makes sense:
https://dave.cheney.net/2019/05/07/prefer-table-driven-tests
Barring a means to detect this, is there a way to control which of the rules the linter runs, i.e. to include the other checks and just disable Range statement for test ...?

Barring a means to detect this, is there a way to control which of the rules the linter runs, i.e. to include the other checks and just disable Range statement for test ...?

golangci-lint is capable of doing that.

Could you give me a little more detail? I'll check the documentation myself, too. I know I can disable the entire paralleltest linter (in fact, that's been my current strategy), but ideally I'd like to keep the other checks, which have proved helpful.

@jackwhelpton put the following snippet in your .golangci.yml configuration:
issues:
  exclude-rules:
    # False positive: https://github.com/kunwardeep/paralleltest/issues/8.
    - linters:
      - paralleltest
      text: "does not use range value in test Run"

Better to use slices - this is how auto generated tests works and slices are also faster

Better to use slices - this is how auto generated tests works and slices are also faster to iterate

I don't think iteration performance is crucial here, but I'm curious what kind of auto generated tests you talk about. Do you have some examples?

I don't think iteration performance is crucial here

Ofcource it's not :) Just another disadvantage of working with maps in table-driven go tests.

What kind of auto generated tests you talk about

I'm talking about tests generated by VSCode or Goland. I think VSCode uses https://github.com/cweill/gotests but I'm not sure

I'm not sure I see any advantage in slices over maps: iteration performance for a handful of test cases screams premature optimization to me.
By contrast, a map provides you with a structure that can be keyed by test name, which improves readability (the test name and data can be fetched into separate variables as part of the loop), and also offers a slight shuffling of execution order that could be beneficial: Dave's article I linked to above provides a pretty good explanation.
I've not tended to use "auto-generated" code as a guideline (Go's own source code, sometimes - although even that varies quite a lot in style based on the author). In many situations, what you get from looking at auto-generated code is something that's "good enough" in all cases, not ideal in the one you're specifically trying to address.
Until there's a workaround for this, the suppression @invidian suggested works a charm.

iteration performance for a handful of test cases screams premature optimization to me.

To me too

By contrast, a map provides you with a structure that can be keyed by test name, which improves readability

I agree :) That's why I've used to use them until I came into trouble we discuss here

I've not tended to use "auto-generated" code as a guideline

That's not a guideline (yet) but it seems reasonable to try to improve automation instead of rejecting it
The main problem here is parallel testing - other disadvantages like code-generation and performance are not so important but they are still there

Following piece of code currently triggers the linter, even though it is a valid code.

@invidian, FWIW, that's not a valid Go. Here I modified it to make it compile (testCases was defined incorrectly) and to highlight a problem:
func TestFunctionRangeNotUsingRangeValueInTDotRun(t *testing.T) {
    t.Parallel()

    testCases := map[string]struct {
        value string
    }{
        "1": {value: "foo"},
        "2": {value: "bar"},
    }

    for n, tc := range testCases {
        n, tc := n, tc // fixes the problem

        t.Run(n, func(t *testing.T) {
            t.Parallel()
            fmt.Println(tc.value)
        })
    }
}

Without n, tc := n, tc // fixes the problem line, the output is likely to be "bar, bar" instead of "foo, bar" or "bar, foo". That's due to https://golang.org/doc/faq#closures_and_goroutines (t.Parallel + t.Run creates a goroutine).
Some other examples in this issue are similarly incorrect.

That being said, this linter complains about both forms of this code, with and without n, tc := n, tc // fixes the problem line. I hope that could be fixed.

we need linter for this stuff n, tc := n, tc

we need linter for this stuff n, tc := n, tc

This is checked by scopelint, but it's now deprecated in golangci-lint :(

Doesn't exportloopref suffice? I have this in my golangci-lint config:
  # The following linters should not be enabled:
  ...
  # - scopelint         deprecated, supplanted by exportloopref

See https://github.com/kyoh86/looppointer/issues/8

Oh boo. That's good to know... adding those lines has become second nature within my tests, but it's a pain there's now nothing that catches their absence, as I was bitten by that a fair few times in earlier days.

This is embarrassing. My email filter was junking all these emails. The filter has since been fixed, and the issue is now resolved. Thanks for your effort @rkennedy and @rarguelloF. Once again, my apologies


Check if filter value exists

Hi.
I want to know is there any way I can check if the filter() function returns a record or not ?

@adnanafzal565 Try to execute your query and check how many lines are returned:
let result = try db.prepare(table.filter(yourExpression))
if let _ = result.next() {
    // You have at least one record
}

Yes, it was fixed. Thank you.


add warning system for parameters alongside error system

This is being implemented in #404

I got confused, this was done in FactorFence even though it wasn't used there...  closing


Allow calls to type constructors on primitives to appear in constant expressions

What was wrong?
As stated in #809 one very annoying thing that arises in pretty much every attempt to write real world Fe code is that it is currently not possible to use addresses as constants.
How was it fixed?
Hacked until it worked.

Please give me a day to estimate how much this change affects the fe-v2 implementation. I guess this case would be fine.
But I want to share my feeling that I'm afraid fe-v2 implementation will never end if I have to implement all arbitrary changes in a "proper" way.

@Y-Nak Sure, no worries. I don't mean to burden your work but this issue is a real pain when writing real world Fe code so I hope we can resolve it before the next release.

I think that instead of allowing primitive type constructors/casts, we should just interpret the value using the declared type, and error on values that don't fit that type.
Eg const FOO: address = 0xfefefefe would be allowed, but const BAR: u8 = 1000 would be rejected.

@sbillig That's what I initially tried to do but it wasn't obvious to me how that would work.
const BAR: u8 = 1000 is rejected because 1000 is parsed as a Expr::Num(num_str) and so we check the num_strut in against the permitted size directly. But in the case of const FOO: address = 0xfefefef the 0xfefefef is again parsed as an Expr::Num(num_str) and since address isn't a numeric type I'm not sure how exactly we would use it to constrain the size of the numeric expression. We could say that we can coerce a u256 into an address but since address is a smaller type than u256 it doesn't seem right to just truncate left or right.
A cleaner way imho would be to add u160 as a numeric type and make address an alias to it.

const BAR: u8 = 1000 is rejected because 1000 is parsed as a Expr::Num(num_str) and so we check the num_str against the permitted size directly. But in the case of const FOO: address = 0xfefefef the 0xfefefef is again parsed as an Expr::Num(num_str) and since address isn't a numeric type I'm not sure how exactly we would use it to constrain the size of the numeric expression. We could say that we can coerce a u256 into an address but since address is a smaller type than u256 it doesn't seem right to just truncate left or right.
A cleaner way imho would be to add u160 as a numeric type and make address an alias to it.

A think Address should be a newtype rather than an alias, but it could certainly be defined in std in terms of u160.
For now, we can just constrain it to a u160 internally.  I think this is all doable without much trouble, by checking for Type::Address in traversal::expressions::expr_num and Constant::from_num_str, and checking if the string can be interpreted as a u160. This of course, would allow the use of a number literal anywhere an address is expected. I started along this path, but was using H160 which won't work because it expects a full 40-char hex string. I'm happy to continue, but also happy to have you do it instead :)
(Aside: perhaps non-hex address literals should be disallowed?)

@sbillig sounds good to me!

I'm happy to continue

Yes, please take it, I'm happy to review it then.


🌱 add release process

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
TODOs:

[x] squash commits
[ ] include documentation
[ ] add unit tests

Closing in favor of #28


Split out entrypoint resolution and injection, and script conversion

Split out entrypoint resolution and injection, and script conversion

This is the next and largest step of the effort to simplify and separate
MakePod out into digestible chunks (#1605)

Behavioral changes:
- Script->Command conversion now happens before entrypoint rewriting,
  rather than converting the rewritten entrypoint args.
- Image name->digest lookups are cached locally while resolving a single
  TaskRun's steps.
- Entrypoint lookups also update the step's digest. This was a race
  before: if an image was pushed between resolution and pod start, the
  resolved command might be out-of-date.

Some redundant test cases have been removed from taskrun_test.go -- this
file should test only behavior of taskrun.go, which is now smaller.

Unit tests for individual transformation behavior has been moved into
individual test files in pkg/pod, with some integration tests in
pod_test.go -- some of these could likely be removed as well, if we feel
like it.

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

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

See the contribution guide for more details.
/assign @sbwsg

/lgtm


Add the stack auditor, a plugin that helps operators know which stacks are in use on the platform, to the cli plugin

Submitting Plugins
If you haven't yet, please review our contributing guidelines:
https://github.com/cloudfoundry/cli-plugin-repo#submitting-plugins
In particular ensure the following requirements are being met:

[x] The plugin's name field in repo-index.yml matches the Name field in the plugin's plugin.PluginMetadata section
[x] The plugin's url field in repo-index.yml contains the same version from the version field

Description of the Change
Add the stack auditor, a plugin that helps operators know which stacks are in use on the platform, to the cli plugin registry

LGTM


Fixed href to download posts in SERVER mode

When I tried using cms.js in SERVER mode, the URLs for the posts and pages to download were wrong, as they missed the subdirectories ("posts" and "pages"). This is pull request fixes the problem. I'm not a Javascript developer, but it seems to work just fine.

@chrisdiana Just curious whether you need anything else to merge this request? Thank you!

Thanks for submitting the PR @maksyms! Before merging in could you share your environment so I can try to replicate the problem? Any of these might help.

Server type (Apache, Estatic, etc.)
URL or repo where the problem is occurring

@chrisdiana Sure!
The problem is using the latest Apache on the latest Ubuntu. I'm afraid the URL is private, using an internal domain.
The way to reproduce is as follows:

Deploy cms.js on Apache2 in Ubuntu
Configure it to use SERVER mode
publish posts and pages

When you hover over the links to posts and pages or click on them, you will see that "posts" or "pages" parts of the link are missing, hence, the link produces 404.
I hope this helps!

That patch didn't work for me.
I had to first check the defaults.mode and rewrite the like you did:
this.url = defaults.mode === 'SERVER' ? type + '/' + url : url;

Interesting. I'm not sure what defaults.mode is, but perhaps @chrisdiana can comment which one is the right one to test?

@maksyms @paulbruggeman  defaults.mode refers to whether to use SERVER mode that is a self host option for running on your own webserver (i.e. Apache which is what you want) vs GITHUB mode which uses Github pages.
Make sure that directory indexing is enabled on your server. That is what CMS.js uses to know what files are there by pulling all the anchor tags. Take a look here:
https://github.com/chrisdiana/cms.js/wiki/Server-Mode

@maksyms I dug into this a little more and it looks like you're right with this not working on Apache. Apache and Estatic servers seem handle directory listing a little different now:
Apache doesn't prepend the directory to the anchor link like Estatic does.
Apache2 directory listing files

Estatic directory listing files

This fix you submitted will fix Apache but will also break Estatic setups. Also, we should go to the root of the problem which is the FileCollection.getFileUrl() method and keep this fix at the FileCollection level since it's affecting to more than one file:
https://github.com/chrisdiana/cms.js/blob/4440f6f638b5db37e0c5e3809c9de2be8ebdcaec/src/filecollection.js#L54
  getFileUrl(file, mode) {
    return (mode === 'GITHUB') ? file['download_url'] : file.getAttribute('href');
  }

We'll need to add a way to just get the filename from the path and pass in the type but changing this function to this should work.
var getFilenameFromPath = function (str) {
    return str.split('\\').pop().split('/').pop();
}

getFileUrl(file, mode, type) {
   return (mode === 'GITHUB') ? file['download_url'] : `${type}/${getFilenameFromPath(file.getAttribute('href'))}`;
}

Then, update this function:
https://github.com/chrisdiana/cms.js/blob/4440f6f638b5db37e0c5e3809c9de2be8ebdcaec/src/filecollection.js#L92
to
var fileUrl = this.getFileUrl(file, this.config.mode, this.type);

Did you commit this so we can test?
I just noticed this is similar or the same as-
Issues trying to retrieve pages/posts file listing in SERVER mode #103

@chrisdiana Nice one, thank you! Yes, your suggestions make a lot of sense. 👍
Unfortunately, I no longer have the Apache in question, and don't have a cms.js install to test this on. Apologies.

Sounds good @maksyms 👍
@paulbruggeman Ill add the fix later this evening


[Kernel] Validate readSchema is a subset of the table schema in ScanBuilder

Feature request
Which Delta project/connector is this regarding?

[ ] Spark
[ ] Standalone
[ ] Flink
[x] Kernel
[ ] Other (fill in here)

Overview
Validate that the read schema provided in ScanBuilder is a subset of the snapshot schema.
https://github.com/delta-io/delta/blob/master/kernel/kernel-api/src/main/java/io/delta/kernel/internal/ScanBuilderImpl.java#L80
Motivation
Better validation / error messages.

Hi @allisonport-db, I'm interested in working on this issue can I get started with this ?

@kannanjayachandran assigning you!
You just need to implement the check in the function linked. And add tests please.

Hello @allisonport-db , I  want to know if there is a specific fiile where we write our tests and also I want to know if I can build my own exception class to handle error logging

Hi @allisonport-db, I am a beginner to open source project contribution and I wanted to ask if i am allowed to do a pull request on this feature even if someone is already working on it ?


Windows Libraries Folder Focus

This isn't a real "issue" with HashCheck but a request for a suggestion or workaround.
Is there a trick to force Windows Explorer to focus on Library folders when creating a checksum file?
If you highlight files within any Libraries folder/directory, then right click and select "Create Checksum File....", when the Save As dialog appears, the focus is not on the current folder.
The "Save As" focuses correctly when used on folders/directories outside of Libraries.
Any suggestion would be appreciated.

I suspect this is an OS bug, but I'm not sure. I can't reproduce it on Windows 10 - when I select "Create Checksum File..." on a file inside a library (after enabling libraries in Explorer, their display is disabled by default in Windows 10), the correct library folder is highlighted on the left and the destination folder also matches up correctly.
Unless perhaps I'm misunderstanding what you're describing?

Thank you for the information gurnec. I have been using third party file managers instead of Windows Explorer and they all work fine. I have also disabled Windows Libraries and everything works fine as well.
You can probably delete this "issue" since it is an operating system problem. Simplest solution is to disable the libraries function.


Hooks details: first column is hook' step; None value when no hooks

Following up from https://github.com/konveyor/forklift-ui/pull/617#issuecomment-853309195

Hook step is now in first column then hook definition
None is displayed when there is no hook for plan
Also renames React component IDs from review to details

@gildub FYI these review-* ids were added for QE tests to use, it's fine to change them where it makes sense but we need to make sure we let @ibragins know when we do. I'll ping him.

@mturley, yeah I was wondering about that, I checked they weren't involved in unit tests but thanks for following up with QE!

Yeah I'm not seeing them in qe-tests yet either, but he asked me to add them for that reason so 🤷
No problem 😄

Thanks @gildub. Looks good.


config set errors out after prompt for the value if no compose file is found

$ AWS_PROFILE=defang-sandbox defang config set BOARD_PASSWORD
 * Using AWS provider
? Enter value for "BOARD_PASSWORD": [? for help] ****
Error:no Compose file found

[x] Also note a missing space between Error: and no Compose file found

Fixed by #471


(Regulated) organization: Don't allow page to be unpublished if there are active engagements

A user can unpublish their org page by clicking the "unpublish" button: https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=2142%3A60877
Unpublish modal, if there are active projects and engagements: https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=2233%3A60007
Unpublish modal, if there are no active projects and engagements: https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=2233%3A60020
Successfully unpublishing it will take the user back to their org page, now back in draft mode with a link to go back to edit mode: https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=2233%3A60033

Do not allow orgs to publish projects if their page is not published
active projects = any project with an engagement that's invited people

We need to handle not showing and not allowing access to Projects and engagements (that don't have active users) when the publish page is unpublished. At the moment these things are still accessible with a 404 page returned when trying to access the org page from them. Additionally, the org shouldn't be able to create new projects and engagements when they don't have a published public page.


[4.2][CSBindings] Try to infer bindings from related type variables

Attempt to use potential bindings inferred for related types variables
discoverable through 'subtype' constraints, this helps to build a
more precise bindings domain for each type variable.
Resolves: rdar://problem/38159133
(cherry picked from commit 380ea4a671549dbd9ae1bc5b48b33036ed6a1a51)

@swift-ci please smoke test and merge

@swift-ci please test macOS platform


us-ga-cherokee

cherokee county
http://gis.cherokeega.com/

Could probably use
http://gis.cherokeega.com/arcgis/rest/services/Address/MapServer/0
or
http://gis.cherokeega.com/arcgis/rest/services/TaxParcels/MapServer/6


Misuse of word "refine"

Logically the sentence should be "refrain from using wildcards"

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

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

@FlyingBlueMonkey  - thanks for your contribution - merging now

@FlyingBlueMonkey  - thanks for your contribution - merging now


rabbitmq-server: 3.7.18 -> 3.8.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/rabbitmq-server/versions.
meta.description for rabbitmq-server is: '"An implementation of the AMQP messaging protocol"'.
meta.homepage for rabbitmq-server is: '"http://www.rabbitmq.com/"
Release on GitHub
Compare changes on GitHub

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.8.0 with grep in /nix/store/zrcb810wxfk4imzvfwvkc09z30isl523-rabbitmq-server-3.8.0

directory tree listing: https://gist.github.com/1b5f567aa6ac52ae82ec063c4d184a67

du listing: https://gist.github.com/9e05959f52bb95d69293d0d2e6fce838

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
rabbitmq-server

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/zrcb810wxfk4imzvfwvkc09z30isl523-rabbitmq-server-3.8.0 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A rabbitmq-server https://github.com/r-ryantm/nixpkgs/archive/34d6906dad73f2d153d5358ace129cf3855db7e0.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/zrcb810wxfk4imzvfwvkc09z30isl523-rabbitmq-server-3.8.0
ls -la /nix/store/zrcb810wxfk4imzvfwvkc09z30isl523-rabbitmq-server-3.8.0/bin

cc @Profpatsch for testing.

@GrahamcOfBorg test rabbitmq

nix-review happy for me macos 10.13.


Bayfield County, Wisconsin

Currently getting clear terms of use/ copyright information from the county. Spoke to Carmen Novak<PHONE_NUMBER>) today. Data lives in rest services. Need to find addresses, but will clear up with back and forth in formal email process about copyright.
Main rest: https://maps.bayfieldcounty.wi.gov/ArcGIS/rest/services/BayfieldCoBaseMap/MapServer
Addresses:
Buildings: https://maps.bayfieldcounty.wi.gov/ArcGIS/rest/services/BayfieldCoBaseMap/MapServer/0
Parcels: https://maps.bayfieldcounty.wi.gov/ArcGIS/rest/services/BayfieldCoBaseMap/MapServer/14
Copyright:
Will update. Stay tuned!
Carmen Novak
GIS Technician | Bayfield County Land Records
715-373-3426 |<EMAIL_ADDRESS>Email sent:
Carmen,
Thanks for the call today! Brief recap/ overview and a few questions. I am participating in an opensource project Openaddresses. I am one of the many people out there looking for data for addresses, buildings and parcels to add to the dataset/ project. The project heavily relies on open data and clear copyright/ terms of use. We try to find the best official resources that have consistent updates. In the past I've emailed you or Scott about parcels, and more recently your team has put together the rest services (thank you). So we would like to use the data from there for the project: https://maps.bayfieldcounty.wi.gov/arcgis/rest/services/BayfieldCoBaseMap/MapServer
The project is here: https://openaddresses.io/Github here: https://github.com/openaddresses/
Again, we would love to use the counties data in the project. I'd like to confirm/ clarify the terms of use or copyright on the data. One county I always look to for understanding this is Dakota County, Minnesota: https://resources.gisdata.mn.gov/pub/gdrs/data/pub/us_mn_co_dakota/plan_parcels/metadata/metadata.html
but there are so many variations of licenses: https://creativecommons.org/share-your-work/cclicenses/
so the questions:1)does the county have address points in the rest services somewhere?2)can you somehow update your site or within the rest services/ maps to clarify the terms of use/ copyright?
If you have any questions for me, please ask. Again, really appreciate your team putting these layers together and always helping me with data!
Thanks,Justin

FYI @justinelliotmeyers, Github doesn't send email notifications when a field is edited in the issue. Can you add updates as comments in the future so I get the email? Thank you sir!

Carmen followed up:
Hi Justin,
We are still looking into a copyright addition/clarification on the website, but I wanted to follow up on your second question on the address data published service, which is available here: https://maps.bayfieldcounty.wi.gov/ArcGIS/rest/services/BayfieldCoBaseMap/MapServer/0.
Thanks!
Carmen Novak

Data is public domain!

I updated the buildings. Addresses and parcels were already updated.

Thanks for contacting them, too!


Contribute to adding profiling as OTEL supported event type

As profiling has grown in popularity and more companies have started to build tooling around profiling, there has been a lot of discussion about adding profiling as an official OTEL supported event type.
Logs, metrics, and traces have already gone through a similar process to become more standardized and seeing as we have identified profiling as an "increasingly important" observability signal in our whitepaper  it would be great for the observability tag to help provide input / resources towards this effort.
Several interested groups from a profiling-developer slack channel, the OTEL issue linked above, and some who discussed this live at Kubecon EU this year have started to plan for making progress on this and I've compiled some general thoughts about possible next steps in a doc here.
I'd love to discuss this as a group at the next meeting and also to gather thoughts, ideas, opinions, etc here as well!

Can this issue be considered "done" given profilings status in OTeL?

I think at this point yes :)
I DO think we should ideate (new issues / ideas heartily welcomed!) on how we maintain connection and alignment with the newly formed SIG Profiling, but the projects in-scope for the TAG generally, open-telemetry inclusive.


Separate the commands in the 'New' button

What would you like to be added?
Now in the dropdown of the 'New' button, we have commands to create new projects, new views and new notes/items. Should they be separated and moved to places like project selector and view selector?

Why is this needed?
To integrate commands which are at the same level.

Actually, I had it like that before 1.0.0. I changed it because I felt like the bar became too busy with several buttons that weren't used that often.
If we add more object types, like recurring notes, it would fit well with the New button.
I'm not married to the current design, but I'd like to avoid changing it too often. If we change the top bar, I'd like to do some more design work before changing it.
For example, I can see the "New view" in your screenshot, but how would you add a new project, or note?

Fisrtly congrats! Wish you a wonderful marriage!
For new projects, the new command can be integrated into the project setting menu. An alternative solution is, add an option called "Create new project" in the project selector.

To save space in the toolbar, an iconbutton for new view is enough. The design also simplifies view creation from a two-step operation to one.

As adding note is also a project-level command, maybe where the current new button lies is the best place for the notes.
However, as there are note creating commands/shortcuts in most of the views, we can even cancel the new button. For example, there are buttons for new notes in both Table view and Board view, and we can double click on the date cell in the calendar view to create new note. Then, find a way to add new notes in the Gallery view seems the only thing we need to do. Here's an protype using the blank card to create new notes.

I'd like to avoid changing it too often.

Yeah, maybe we can provide a transition period during when old & new ways all works. Currently, adding support to create new project in project setting menu, new view in the toolbar and new note by clicking a blank card in Gallery view would enhance the UX, I think.

If we add more object types, like recurring notes, it would fit well with the New button.

If the recurring notes are still physically notes, but with several special header fields, we can leave it to note creating modals. However, if they touchs the project configs, then the new button do have advantage in compatibility. I haven't considered it before but it worth. and the changes should be applied in an appropriate time.
I'd also like to help with this when designs are settled. Thanks for your patiently reviewing and glad to know your opinion.

closed by #468


Fix git_commits_{ahead,before} when no upstream branch is defined

If @{u} is not defined, git rev-list will give an error; redirect
to stderr the error and deal with this case in what follows.
This fixes issue #6656

Btw.:
[[ "$commits" != "0" ]] alone just works fine for this case ;)
For some reason it does not work for me; the if clause is executed if $commits
is empty
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/robbyrussell/oh-my-zsh/pull/6658#discussion_r172388652

On Monday, March 5, 2018 8:47:34 PM EST Shea690901 wrote:

Any special reason you don't write:
 if [[ -n "$commits" -a "$commits" != 0 ]] ?

possibly my ignorance; however the above does not work for me. I changed it to
if [[ -n "$commits" && "$commits" != 0 ]]
which seems to do the trick

Concerning

Any special reason you don't write:  if [[ -n "$commits" -a "$commits" != 0 ]] ?

Strange...
While, for easier reference, I just looked into man test, I also tested this (just to be sure, I didn't make any mistakes).
I'm using zsh 5.3 ...
Ouch, just found my mistake: [[ uses a little different syntax, during my tests I used if [ ... instead of if [[ ......
Concerning

[[ "$commits" != "0" ]] alone just works fine for this case ;)

I typed a little too fast and send before completely thinking through what I had typed, as far as I was concerned from my side I had deleted this comment because of it's error, at least it vanished from my view...

Thanks for the PR!


I can't run the blockchain after updating the newest version.

I am a Mac OS user.
I ran the git pull yesterday, then there's an error when I input this command to run the blockchain system:
./nodeos -e -p eosio --plugin eosio::wallet_api_plugin --plugin eosio::chain_api_plugin --plugin eosio::account_history_api_plugin

I roll-backed my code to the old version, and the command is valid now, so I think the problem coming from the updating.:-)

The newest version of the code attempts to reserve 32 GB of disk space for the shared memory file.  If you don't have sufficient disk space for a file that large and don't have a filesystem capable of creating sparse files, you'll need to add an additional command line parameter to specify a smaller size.  To get the previous behavior of 1 GB, use --shared-memory-size-mb 1024.  This option can be added to the config.ini file (as shared-memory-size-mb = 1024) if you'd like to persist it.


[BPF] proxy - syncer must not count not-ready endpoints

We include terminating eps in the eps list so that we can prevent conntrack clean up, but they are not written into the backend map and so they should not be reflected in the backends count.
Test that Terminating is propagated through proxy frontend. Test in FV that proxy removes the terminating backend from the NAT table.
refs https://github.com/projectcalico/calico/issues/8423
Description

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

TBD

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

The bpf test error is a know affinity flake in ctlb, and is purely test infra related


BD-906:Relicense Thunder to Metrological Apache

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Mallikarjuna Masina seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Copyright scan failure
Commit: 1490eb3ca57fe65f3644b0369014f1772d341ecd
Report detail: https://gist.github.com/c3031bfd4b5163f2598dbf29c87c90ae

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 321 files pending identification.

Protex Server Path: /home/blackduck/github/Thunder/462/rdkcentral/Thunder

Commit: 1490eb3ca57fe65f3644b0369014f1772d341ecd

Report detail: gist

Copyright scan failure
Commit: 153b3ac882bd818c4571320085564885075ddeaf
Report detail: https://gist.github.com/71f7dc5039f38e3d8138a488eec71325

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 386 files pending identification.

Protex Server Path: /home/blackduck/github/Thunder/462/rdkcentral/Thunder

Commit: 153b3ac882bd818c4571320085564885075ddeaf

Report detail: gist

Thanks Mallikarjuna,
A few more RDK Headers I am seeing please:
cmake/modules/FindSlauncher.cmake
Source/processcontainers/implementations/AWCImplementation/AWCImplementation.h
Source/processcontainers/implementations/AWCImplementation/AWCImplementation.cpp

I noticed these missing, please add Metrological headers:
Source/com/ConnectorType.cpp
Source/com/ConnectorType.h
Source/plugins/Types.h
Source/processcontainers/implementations/CRunImplementation/CRunImplementation.cpp
Tests/cryptography/NetflixVaultGenerator/main.cpp
Tests/loader/CMakeLists.txt
Tests/loader/main.cpp
Tests/unit/core/test_doorbell.cpp
Tests/unit/core/test_statetrigger.cpp
Thanks

Copyright scan failure
Commit: 29ecd3399f925f0e5b430a13724deb4c1a8a2826
Report detail: https://gist.github.com/5a2144f5314d6156298a9b1e7f770b22

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 386 files pending identification.

Protex Server Path: /home/blackduck/github/Thunder/462/rdkcentral/Thunder

Commit: 29ecd3399f925f0e5b430a13724deb4c1a8a2826

Report detail: gist

Copyright scan failure
Commit: b5b22c7b494a66ed4a68fec4748a954cb2d915fa
Report detail: https://gist.github.com/196913e32e780b29e3bd4f631a1f50c6

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 397 files pending identification.

Protex Server Path: /home/blackduck/github/Thunder/462/rdkcentral/Thunder

Commit: b5b22c7b494a66ed4a68fec4748a954cb2d915fa

Report detail: gist

Thanks Mallikarjuna - happy from compliance point of view

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: Upvoted the blackduck failure.  I need to be told when this is merged so I can tell Blackduck that Metrological copyrights are OK.

Commit: b5b22c7b494a66ed4a68fec4748a954cb2d915fa

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 397 files pending identification.

Protex Server Path: /home/blackduck/github/Thunder/462/rdkcentral/Thunder

Commit: 09450d4d8b909ec749b6a9de9cd9e11f34f67b12

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 397 files pending identification.

Protex Server Path: /home/blackduck/github/Thunder/462/rdkcentral/Thunder

Commit: 09450d4d8b909ec749b6a9de9cd9e11f34f67b12

Report detail: gist

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: Not clearing in BlackDuck because this needs to be done at merge so existing stuff isn't broken.

Commit: 09450d4d8b909ec749b6a9de9cd9e11f34f67b12

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 394 files pending identification.

Protex Server Path: /home/blackduck/github/Thunder/462/rdkcentral/Thunder

Commit: 4e35cc6f0d43f21d949db11ee9e231883d272484

Report detail: gist

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: Will fix properly when merged

Commit: 4e35cc6f0d43f21d949db11ee9e231883d272484

Sorry, seems there is a merge conflict again. Could you remove the Test/cryptography from the list and I will immediatly merge it before anything else :-) Sorry!

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 384 files pending identification.

Protex Server Path: /home/blackduck/github/Thunder/462/rdkcentral/Thunder

Commit: 1a461cad377dcf91cf7f4d9c3b9dce2bc8b3b43e

Report detail: gist

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: Just Metrological copyrights

Commit: 1a461cad377dcf91cf7f4d9c3b9dce2bc8b3b43e

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 384 files pending identification.

Protex Server Path: /home/blackduck/github/Thunder/462/rdkcentral/Thunder

Commit: a01e51df9abe633a6b5669e42613d9a901f00f34

Report detail: gist

@mhughesacn Oops The Black Duck disagrees again :-)

Hi Pierre.  We all have to jump together.  I upvote here and tweak the grumpy Duck so it likes Metrological headers and dislikes RDK ones.  So: please comment here saying when you are going to merge, and then I do these steps, and you merge and everything is happy.  Or just pick a time.
Thanks,
Martin

@mhughesacn I am now blocking all other work to finally get these new headers in, to avoid new merge conflicts so if you approve them now I merge them immediatly!

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: For merge

Commit: a01e51df9abe633a6b5669e42613d9a901f00f34


Set Captcha Background transparent

Hello,
How i can set the background to transparent?

Thank you

thanks, i give a try


scipy/scipy-dev Docker image appears to have outdated dependencies

Sorry if this is the wrong repo to report this, I don't know where you keep your Dockerfiles.
I was following the instructions here and I had to upgrade the following packages:

meson
cython
pybind11

Steps to reproduce:
# On the host
docker run -it --rm -v ${PWD}/:/home/scipy scipy/scipy-dev

As-is:
# In the container

(scipy-dev) gitpod > ~/scipy $ cd /home/scipy
(scipy-dev) gitpod > /home/scipy $ conda activate scipy-dev
(scipy-dev) gitpod > /home/scipy $ python dev.py build
💻  meson setup /home/scipy/build --prefix /home/scipy/build-install
The Meson build system
Version: 1.0.0
Source dir: /home/scipy
Build dir: /home/scipy/build
Build type: native build

meson.build:1:0: ERROR: Meson version is 1.0.0 but project requires >= 1.1.0

A full log can be found at /home/scipy/build/meson-logs/meson-log.txt
Meson build setup failed!
(scipy-dev) gitpod > /home/scipy $

After pip install --upgrade meson:
# In the container

(scipy-dev) gitpod > /home/scipy $ python dev.py build
💻  meson setup /home/scipy/build --prefix /home/scipy/build-install
The Meson build system
Version: 1.1.1
Source dir: /home/scipy
Build dir: /home/scipy/build
Build type: native build
Project name: SciPy
Project version: 1.12.0.dev0
C compiler for the host machine: /opt/conda/envs/scipy-dev/bin/x86_64-conda-linux-gnu-cc (gcc 11.3.0 "x86_64-conda-linux-gnu-cc (conda-forge gcc 11.3.0-19) 11.3.0")
C linker for the host machine: /opt/conda/envs/scipy-dev/bin/x86_64-conda-linux-gnu-cc ld.bfd 2.39
C++ compiler for the host machine: /opt/conda/envs/scipy-dev/bin/x86_64-conda-linux-gnu-c++ (gcc 11.3.0 "x86_64-conda-linux-gnu-c++ (conda-forge gcc 11.3.0-19) 11.3.0")
C++ linker for the host machine: /opt/conda/envs/scipy-dev/bin/x86_64-conda-linux-gnu-c++ ld.bfd 2.39
Cython compiler for the host machine: cython (cython 0.29.33)
Host machine cpu family: x86_64
Host machine cpu: x86_64
Program python3 found: YES (/opt/conda/envs/scipy-dev/bin/python3.1)
Found pkg-config: /opt/conda/envs/scipy-dev/bin/pkg-config (0.29.2)
Program cython found: YES (/opt/conda/envs/scipy-dev/bin/cython)

meson.build:54:2: ERROR: Problem encountered: SciPy requires Cython >= 0.29.35

After pip install --upgrade cython:
# In the container

(scipy-dev) gitpod > /home/scipy $ python dev.py build

...

scipy/meson.build:84:15: ERROR: Dependency lookup for pybind11 with method 'pkgconfig' failed: Invalid version, need 'pybind11' ['>=2.10.4'] found '2.10.3'.

After pip install --upgrade pybind11, it finally works:
# In the container

(scipy-dev) gitpod > /home/scipy $ python dev.py build

...many lines of output...

Build targets in project: 254

SciPy 1.12.0.dev0

  User defined options
    prefix: /home/scipy/build-install

Found ninja-1.11.1 at /opt/conda/envs/scipy-dev/bin/ninja
Meson build setup OK
💻  ninja -C /home/scipy/build -j6
ninja: Entering directory `/home/scipy/build'
[5/1614] Generating scipy/generate-version with a custom command
fatal: bad revision '^v1.10.0'
[15/1614] Compiling C++ object scipy/_lib/_uarray/_uarray.cpython-310-x86_64-linux-gnu.so.p/vectorcall.

FYI, this is the Dockerfile I now use locally to make my life easier:
FROM scipy/scipy-dev@sha256:0313e34a23b7f5b8c3f549b7d56c318dc19bddd2ce2d30781332fa52aaeecd48

# Fix for other issue: https://github.com/scipy/scipy/issues/18662
WORKDIR /home/scipy

RUN conda activate scipy-dev

RUN pip install meson==1.1.1 Cython==0.29.35 pybind11==2.10.4

Hi @tadamcz, we don't support anymore docker. Please use code space alternatively.
I will close this one for now and the other one too.

OK makes sense, thanks! I was confused why I couldn't find the Dockerfile anywhere in your repos.
I think I will use my own home-cooked Docker images then (taking this as the base Dockerfile in case you remove your images from Docker Hub, feel free to do so)
Codespaces are a cool concept but afaik VSC only, and I prefer JetBrains IDEs.

I did not try it, but you can apparently use PyCharm et cie. Go to your settings and then under Codespace you can select JetBrains Gateway.


Upload to EVK

Hi guys,
Great work! A quick question. Did you succeed in loading this to an EVK device? I am trying to upload the following program using mcuxpresso's GUI Flash tool to an imxrt1052 EVKB, it seems to upload it, but fails to light up the LED.
I would appreciate any feedback :)
//! Prints "Hello, world!" on the host console using semihosting

#![no_main]
#![no_std]

extern crate panic_halt;

use cortex_m_rt::entry;
use cortex_m_semihosting::{debug, hprintln};
use imxrt_ral::{read_reg, write_reg, modify_reg, reset_reg};
use imxrt_ral::{gpio};

fn set_pin_9(gpio: &gpio::RegisterBlock) {
    write_reg!(gpio, gpio, GDIR, 1<<9);
    write_reg!(gpio, gpio, DR, 1<<9);
}

#[entry]
fn main() -> ! {
    // exit QEMU
    // NOTE do not run this on hardware; it can corrupt OpenOCD state
    // debug::exit(debug::EXIT_SUCCESS);

    let gpio1 = gpio::GPIO1::take().unwrap();
    set_pin_9 (&gpio1);
    loop {}
}

memory.x
MEMORY
{
  /* Define each memory region */
  FLASH (rx) : ORIGIN = 0x60000000, LENGTH = 0x4000000 /* 64M bytes (alias Flash) */
  RAM (rwx) : ORIGIN = 0x20000000, LENGTH = 0x20000 /* 128K bytes (alias RAM) */
}

You most likely need to setup the boot-gen stuff as well. I've been working on the GPIO side of this to get it going on my 1062 evk and 1011 evk. I'm hoping to try that stuff out this week and get a sample app going. If you beat me to it, please do share so we can put it up here!

So some updates on this, I did try to build and run a sample on the board in a few different ways, and when it didn't run right away I started digging. It looks like perhaps some linker script issues that I need to sort out. I've made some issues in this repo and the imxrt-rt repo if you'd like to follow along or add to the discussions.

I seems that the elf binary requires a .boot_hdr section (that contains the interrupt vector table) and has the FSCB (FlexSPI Configuration Block). I am still trying to figure out how to set up this. I will keep you posted.

So looking closer at the NXP SDK they enable some clocks prior to things working.
I've managed to get a pretty similar example working on my 1060evk today finally, after fighting some linker issues. imxrt-rt needs to use the gnu linker not the default llvm linker for the provided link.x script it provides. I'll look into maybe fixing that in the future.

So final conclusion here is that you'd likely need to enable some clocks and set the iomux's pin config to be ALT5 for gpio usage.
The hal will take care of all of this as we move forward. Let me know if that helps or if there's more we can do to help in the meantime.

Just an update, the 1060evk now has blinking leds, uarts, timers, and pwm working. I'm going to close this issue unless you intend on working on a 1050evk BSP and feature flag for the hal, which we would gladly accept!

Feel free to reopen or open new issues if you like. Closing for now


Custom field "Datetime" editing: false don't work as expected

Bug Reports:
A)Expected Behavior: Following this example i try to disable editing and expect, that edit template is not shown in edit mode.
B) Actual Behavior: The edit template is still showing up, but the value don't get updated on save.
C) See this fiddle.

Its not bug, it works as is implemented

Hello and thank you for the fast answer.
If it's not a bug, is it possible to make it work as i expect it to do?
I just tested it with an other custom field with jquery autocomplete and the behavior is the same.
I hoped that custom field can behave like the integrated ones.

change this
       insertTemplate: function(value) {
            return this._insertPicker = $("<input>").datepicker({ defaultDate: new Date() });
       },
      editTemplate: function(value) {
         return this._editPicker = $("<input>").datepicker().datepicker("setDate", new Date(value));
      },

to this
  insertTemplate: function(value) {
       if (!this.inserting)
            return "";
    return this._insertPicker = $("<input>").datepicker({ defaultDate: new Date() });
  },

  editTemplate: function(value) {
           if (!this.inserting)
              return this.itemTemplate.apply(this, arguments);
    return this._editPicker = $("<input>").datepicker().datepicker("setDate", new Date(value));
  },

Works like a charm.
Thank you.


Make checkForSubImage null-safe

Make checkForSubImage null-safe for TiffImagingParameters params.
Especially as params gets set as null in TiffImageParser.java:183 getAllBufferedImages()

I ran into this today and was just on he way to create a ticket. Any chance this can be merged?

I ran into this today and was just on he way to create a ticket. Any chance this can be merged?

Hello @mercuryLuz
Do you have an image that reproduces the issue? Would please create a PR with a unit test?

Yeah I can provide that later today. Will open a new PR for that

Yeah I can provide that later today. Will open a new PR for that

Hello @Ditscheridou
Please follow up or close this PR. TY!

Closing: In git master now. You are credited in changes.xml.


Showing tool bar over the screen

Please see the error log
-[UIWindow endDisablingInterfaceAutorotationAnimated:] called on <UITextEffectsWindow: 0x17fe1680; frame = (0 0; 320 480); opaque = NO; autoresize = W+H; layer = <UIWindowLayer: 0x17fe19d0>> without matching -beginDisablingInterfaceAutorotation. Ignoring.

Should be fixed on develop by #1063


FileFlows App

Added the FileFlows application.
Tested by manually importing the docker-compsoe.yml into CassOS.
Not entirely sure if theres a more involved teseting process that tests the taglines/description etc.

Theres possibly four variations of this to add.  Decided to start with the most common/basic one, the combinations are

FileFlows (this one)
FileFlows with NVIDIA support
FileFlows Node
FileFlows Node with NVIDIA

Didn't want to flood the store with 4 similar apps.   Be nice if there was a way to select which option or something.  Even if it was just

FileFlows
FileFlows Node

And the user edits or sets a toggle for NVIDIA support, this is how it works on unRAID.


installation error

Forked the project. And running the setup the way README describes. When running npm install i get the following error. What can i do to proceed with the install?
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| >= 3.2.0-dev || >= 3.3.0-dev || >= 3.4.0-dev || >= 3.5.0-dev || >= 3.6.0-dev || >= 3.6.0-beta || >= 3.7.0-dev || >= 3.7.0-beta but none is installed. You must install peer dependencies yourself.

Ok typescript was not installed. Solved by running npm install typescript and rerunning npm install

Hi @RemcoHalman, thank you for raising this issue 😃.
Your question is completely valid and I think we can improve the documentation on the README to include that TypeScript needs to be installed. If you want you can raise a PR with something like this:
Prerequisites

Typescript
TypeScript is a dependency of <the package name> , so you need to install it first.
npm install typescript

@BOLT04  Thanks for the tip! Added to the README and made a pull request. As a bonus added a little styling

Looks likes typescript is missing from the project, we need to add it to package.json in the devDependencies - I think this project needs a reboot though


1337x and LimeTor not working only PirateBay working !!!

Hi, the overall app is nice, but there are issues -> 1337x and LimeTor not working. Can you please check what's the problem?

Lime Torrent is working fine for me. However, i  1337x the issue with the
web scrapping. Use VPN for Lime Torrent.
On Fri, Jul 8, 2022, 12:34 AM Ishtiaq Ali Chughtai @.***>
wrote:

Hi, the overall app is nice, but there are issues -> 1337x and LimeTor not
working. Can you please check what's the problem?
—
Reply to this email directly, view it on GitHub
https://github.com/priyanktejani/fluid-torrent-search-engine/issues/1,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALGBK2BH2KVIAPTW5O6W2V3VS5SQHANCNFSM527EOGXQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hmm, well with VPN Lime Torrent still not working and please tell me how can I add more torrent websites. And how to contact you can you give me your email. Thanks

?

email<EMAIL_ADDRESS>


Alfred v4 not supported

Please add support for the new v4 of Alfred, old extension does not work anymore

As of Alfred 4, the bundle identifier for current and future versions has been unified to  'com.runningwithcrayons.Alfred' (versus previous version-specific IDs such as 'com.runningwithcrayons.Alfred-2' / '…-3' etc.).
If you feel like updating it yourself for immediate functionality, simply add the following entries to the extensions plist and php files, respectively:
Config.plist

<string>com.runningwithcrayons.Alfred</string>
→ to the <key>Bundle Identifiers</key> array
@ /Users/[username]/Library/Application Support/PopClip/Extensions/Alfred.popclipext/Config.plist

— and —
alfred.php

$alfred3bid='com.runningwithcrayons.Alfred';
→ // select target app
@ /Users/[username]/Library/Application Support/PopClip/Extensions/Alfred.popclipext/alfred.php

*Note: You'll need to nuke the _Signature.plist and change the 'pilotmoon' part of the Extension Identifier while you'll in there as these changes will break the code signing.
→ <string>com.make-this-part-whatever.popclip.extension.append</string>
Or, if you don't feel like doing all that…
→ http://bit.ly/AlfredDasBlatz
: )

Done (d179511)
And published to https://pilotmoon.com/popclip/extensions/page/Alfred
Thanks @icycles


combine PR #109 and PR #110

Changes from @jobol  and @kooltux look good, build results were also okay -> merging.


Polymer 2 Enhancement: Custom Serializers and Deserializers.

Polymer 1 allowed you to override the serialize method (not too useful) but it appears that this functionality has been removed in v2. It would be very useful to be able to provide custom property/attribute serialization/deserialization for object and array properties. e.g. If you have a "margin" property, the value would be:
{
    top: 2,
    right: 4,
    bottom: 6,
    left: 8
}

This gets JSON serialized to the attribute like so (ugly):
margin='{ "top": 2, "right": 4, "bottom": 6, "left": 8 }'

I'd like to be able to provide custom functionality that's similar to the WPF type converters to map it to something like this:
margin="2, 4, 6, 8"

Looking at the Polymer source, there's a couple of ways of doing it. Class level functions that take a property/attribute name seems the simplest but I'm not sure if they'd play nicely with mixins that add properties. Adding the serializer/deserializer functions to the property definition itself seems the nicest to me:
margin: {
    type: Object,
    value: ....,
    reflectToAttribute: true,
    serialize: function () {...},
    deserialize: function () {...}
}

First prize would be for Polymer to properly use property types, then we could register global type converters:
margin: {
    type: Thickness,
    ...
}

class Thickness {
    // Define the left, right, top and bottom properties.
}

class ThicknessTypeConverter {
    serialize (attribute, value) {}
    deserialize (property, value) {}
}

// Register the converter somehow.

Or perhaps add the functionality to the type object itself (maybe have a "Serializable" base class):
class Thickness extends Serializable {
    // Define the left, right, top and bottom properties.
    serialize (attribute, value) {}
    deserialize (property, value) {}
}

Hi Paul,
This have been removed, but it's been moved... Specifically, in keeping
with the new API standards, these are now "protected" methods,
_serializeValue and _deserializeValue.
I'll make sure this info gets added to the upgrade guide.
Thanks,
Arthur
On May 10, 2017 9:42 PM, "Paul Mason"<EMAIL_ADDRESS>wrote:
Polymer 1 allowed you to override the serialize method (not too useful) but
it appears that this functionality has been removed in v2. It would be very
useful to be able to provide custom property/attribute
serialization/deserialization for object and array properties. e.g. If you
have a "margin" property, the value would be:
{
top: 2,
right: 4,
bottom: 6,
left: 8
}
This gets JSON serialized to the attribute like so (ugly):
margin='{ "top": 2, "right": 4, "bottom": 6, "left": 8 }'
I'd like to be able to provide custom functionality that's similar to the
WPF type converters to map it to something like this:
margin="2, 4, 6, 8"
Looking at the Polymer source, there's a couple of ways of doing it. Class
level functions that take a property/attribute name seems the simplest but
I'm not sure if they'd play nicely with mixins that add properties. Adding
the serializer/deserializer functions to the property definition itself
seems the nicest to me:
margin: {
type: Object,
value: ....,
reflectToAttribute: true,
serialize: function () {...},
deserialize: function () {...}
}
First prize would be for Polymer to properly use property types, then we
could register global type converters:
margin: {
type: Thickness,
...
}
class Thickness {
// Define the left, right, top and bottom properties.
}
class ThicknessTypeConverter {
serialize (attribute, value) {}
deserialize (property, value) {}
}
// Register the converter somehow.
Or perhaps add the functionality to the type object itself (maybe have a
"Serializable" base class):
class Thickness extends Serializable {
// Define the left, right, top and bottom properties.
serialize (attribute, value) {}
deserialize (property, value) {}
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Polymer/polymer/issues/4592, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABGJ7qtV82VY25brRYNUooOfLQMQz0oBks5r4pGYgaJpZM4NXgdT
.

I presume that means we shouldn't be overriding _serializeValue and _deserializeValue now? It would be good to be able to do that, and get the attribute/property name passed to the two methods as well.

Though I see your point in more complex datastructures where you want to deserialize in a specific manner, I do think margin is something that should exists in CSS, and be manipulated using CSS custom properties.
Sent on the go - just so so you know!

Den 12. maj 2017 kl. 10.02 skrev Paul Mason<EMAIL_ADDRESS>I presume that means we shouldn't be overriding _serializeValue and _deserializeValue now? It would be good to be able to do that, and get the attribute/property name passed to the two methods as well.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@noahlaux The margin thing was just an example because it illustrates nicely how a data structure can be collapsed into a simpler attribute representation.  Arrays of strings are probably a better example.
something='["one", "two", "three"]'

becomes
something="one, two, three"

I see your point, I just happen to like the array structure better, because
it uses javascript own natural way to describe a list, and as an added
bonus you can directly bind an array to that attribute and get expected
results. If I should end up with "one, two, three" and my starting point is
["one", "two", "three"] I'd have to join prior to binding, I don't see the
benefits, it's really just more work.
Kind Regards
Noah Laux
Senior Software Developer
<EMAIL_ADDRESS>-  (+45) 60 77 74 33
Skype: noahlaux // facebook.com/noahlaux // @noahlaux
http://twitter.com/noahlaux // last.fm/noahlaux
2017-05-12 10:19 GMT+02:00 Paul Mason<EMAIL_ADDRESS>
@noahlaux https://github.com/noahlaux The margin thing was just an
example because it illustrates nicely how a data structure can be collapsed
into a simpler attribute representation. Arrays of strings are probably a
better example.
something='["one", "two", "three"]'
becomes
something="one, two, three"
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Polymer/polymer/issues/4592#issuecomment-301014654,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA1wb_0cm2jQ2jcJjV9WUa0ISNZIH_Ggks5r5BYGgaJpZM4NXgdT
.

+1
I'd specifically like custom deserialization (serialization would be good too).
Sometimes there is a property that I'd like to accept different possible input formats for but would like to have a single normalized value to work with internally.

I can deserialize/normalize via an observer, but then when my observer updates its own property value (with the normalized value) it will trigger the observer again and that is awkward and can cause bugs.
Instead I can have a computed property based off the other property value, but that seems like unnecessary clutter and the components public interface becomes confusing.
Are there better ways of doing this currently?

If I could do the deserialization myself then it would cleanup that logic. Deserialization would need to be invoked when the property is set directly as well as when it's set via its attribute.

Afaik in Polymer 2 any protected methods are meant to be able to be overridden or extended, and the less reliable to override or extend would be the private methods (2 underscores vs 1 for protected).
So it seems to me that you shouldn't have any problem with overriding these methods.

It'd be useful for Date objects. I'm working on a web app using Polymer and every data displayed is fetched from an API which returns data in JSON, thus dates are in text format. But in my components I'd like to have a Date object instead of a String so I can use the same base object for different uses :

Filtering (using string search with long format e.g. Saturday, May 13th, 2017)
Sorting (using string comparison with short format e.g. 2017-05-13)
Displaying to user (using whatever format you want)

This way I wouldn't have to create Date objects from the base String value every time I need to perform formatting to filter/sort nor to have multiple computed properties based on this String value.

Just found the doc about the _deserializeValue() method which solves the problem for me.

Great! I also added this to the guides to make things a little clearer.
https://www.polymer-project.org/2.0/docs/devguide/properties#custom-deserializers
Note that Date is already handled automatically by Polymer—if you have a property specified as type: Date and you set the corresponding attribute to a string representation, Polymer creates a Date object for you, just like it creates an array from a JSON-encoded array.

Note that it'd be easy to implement a scheme like @PaulHMason described like this:
_deserializeValue(value, type) {
  if (type && type.fromString) {
    return type.fromString(value);
  } else {
    return super._deserializeValue(value, type);
  }
}

Custom deserializing for arrays would be a little trickier, as you'd have to set the type to something  other than Array. However, you can always use jsdoc comments to clarify the actual property type.
That doesn't help with serializing, since serializing is always based on the property's current type, not its declared type.

Closing per https://github.com/Polymer/polymer/issues/4592#issuecomment-301910424


New App Registration Procedure in Azure Portal

The instructions for creating an App Registration in the App Center is outdated for the Notifications Hub tutorial.  The azure portal now has an "App registrations" feature.  Please update.

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

ID: 5b48be73-2f29-6b2a-c837-751d3862bcdb
Version Independent ID: efd00d73-1ff5-8689-8415-fe1630795123
Content: Send notifications to Universal Windows Platform apps using Azure Notification Hubs
Content Source: articles/notification-hubs/notification-hubs-windows-store-dotnet-get-started-wns-push-notification.md
Service: notification-hubs
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@stevek180 Thanks for the feedback. I have assigned the issue to the content author to evaluate and update the document as appropriate.

As we are moving to a new feedback system, please reopen this issue or contact Microsoft Support if you still want to provide feedback on the documentation. Thank you! #please-close


Path Planning(M-step) DP search: why static obs cost calculation is based on Frenet coordinate system? moving obs cost calculation is based on Cartesian coordinate system?

I print the static obs bounding box (need convert from Frenet coordinate to Cartesian coordinate); while, for moving obs, just need directly print this min max corner(don't need convert).

@KevinYuk I think you've pointed out why. In the line you marked as above, GetCostBetweenObsBoxes(), the bounding boxes of other obstacles along their trajectory are based on  Cartesian coordinate, provided by Prediction module.


Multimeter Widget does not list all connected devices

Joulescope model
JS220, JS110
UI version
1.0.35
What OS are you seeing the problem on?
Windows 11
What happened?
If you connect multiple Joulescopes to the host computer and then start the Joulescope UI, the Multimeter Widget's Device dropdown only shows a single available device.
What was expected?
The Multimeter Widget's Device dropdown should list all connected devices.
How to reproduce?

Connect two Joulescopes to the host computer
Start the Joulescope UI
Select the Multimeter view, if not already selected
Click on the Device dropdown
Observe that not all devices are listed.  However, they are shown in the Device Control widget.

Extra information
No response

Fixed by 1.0.42


unable to run code coverage as default

Bug Report or Feature Request (mark with an x)
- [x] bug report
- [ ] feature request

Versions.
@angular/cli: 1.0.5
node: 7.7.1
os: darwin x64
@angular/animations: 4.1.3
@angular/common: 4.1.3
@angular/compiler: 4.1.3
@angular/core: 4.1.3
@angular/forms: 4.1.3
@angular/http: 4.1.3
@angular/platform-browser: 4.1.3
@angular/platform-browser-dynamic: 4.1.3
@angular/router: 4.1.3
@angular/cli: 1.0.5
@angular/compiler-cli: 4.1.3

Repro steps.
Edit the karma.conf.js file, changing these lines
    reporters: config.angularCli && config.angularCli.codeCoverage
              ? ['progress', 'coverage-istanbul']
              : ['progress', 'kjhtml'],

to
    reporters:  ['progress', 'coverage-istanbul']

The log given by the failure.
No error logged.
Desired functionality.
My expectation was that by removing the conditional check it would always run coverage, but so far that isn't the case. I believe the codeCoverage default of false set elsewhere (I'm still confirming this is the case) overrides changes made in the karma config.

It would be nice if options specified in angularCli in karma.conf.js would override what's loaded here: https://github.com/angular/angular-cli/blob/master/packages/%40angular/cli/tasks/test.ts#L32

@jodytate In newer versions we don't use the conditional in karma.conf.js anymore, and remove coverage-istanbul if it's there to ensure compatibility with older configs. Using code coverage is not just a matter of using the reporter though, it also requires build changes to instrument code.
So you cannot obtain the behaviour you're looking for just by changing your karma config, you need to use ng test --code-coverage to ensure the build uses the right preprocessors.
@jensbodal doing that would lead to a lot of duplication and difficulty in keeping feature parity. We only do it for extremely common options like --single-run. It's better to use karma.conf.js for most config items instead.

@filipesilva Many thanks for the follow-up and explanation.


How can I change the font family?

How can I change the font family of Label & dropdown values?

                   <Dropdown
                                labelFontSize='12'
                                 style={{fontSize : 14, fontFamily: Fonts.TitilliumWeb}}
                                data={data}
                              />

Also how can I increase the width & height of the dropdown?

itemTextStyle prop can be used for dropdown items, see: https://github.com/n4kz/react-native-material-dropdown/issues/18

Can you please share some example code? @pdsouza

@AgNm To change the fontFamily and fontSize for the dropdown items this works for me:
<Dropdown
    itemTextStyle={{fontFamily:  'Avenir-Light'}}
    fontSize={12}
/>

Thanks for questions!

Could not reply better than @pdsouza just did. Thanks!
Dropdown height depends on fontSize and itemCount and itemPadding. Dropdown width can be adjusted using dropdownMargins.

@pdsouza @n4kz I am able to change the font family of Dropdown list using itemTextStyle but once I select any value, selected value has default font family, how can I change it?

Same question as @AgNm, there does not seem to be an equivalent labelTextStyle prop. How do we achieve this?

+1

The same issue as @AgNm  and @isaachinman, There is no any prop to change selected items, font family.

@n4kz This shouldbe re-opened as there is now way how you can set the fontFamily for the label. Adding a labelTextStyle would be great.


I CAN'T DO ANYTHING!!!!!!!!!!!!!

I have been trying to text people but it will take forever to send. It also didn’t want to let me sign in. I had to keep spamming the login to Instagram button to finally get in. Can you please explain to me what’s going on? And how I can update the app? You guys are literally no help. @ifedapoolarewaju you made this app so why aren't you helping solve the problem. The robot doesn't help us resolve the problem. All it does is tell us our problem is the same as other peoples. Please solve the problem.

This is an issue in IG, you can check that their service is experiencing outages


Revert "Bump semver from 6.3.0 to 6.3.1"

Reverts DARIAEngineering/dcaf_case_management#3084
This somehow breaks the tests 🤷

Looks like that might have been a flake? I see main passing now


Alias swift to Objective-C

I've merged this. If Swift's build logic diverges from that of Objective-C, we can look at separating them.

It's been deployed to production, but the builds will fail until we route the builds to the Mac infrastructure.


Laggy input

I'm not sure if this is a problem with mozc or with fcitx5 but there is noticeable lag when I try to write Japanese with fcitx5-mozc. I think it can be best observed and demonstrated by holding down a key. Here's a comparison between typing with fcitx5-mozc and my normal keyboard layout.
https://user-images.githubusercontent.com/31477160/151264229-a188ce35-f59d-4611-97f6-da160f608776.mp4
I'm seeing the same lag when deleting as well.
I don't just notice it when holding down a key, I notice it when I type normally as well and it messes up my typing a lot.
System information:

Arch Linux, Kernel 5.16.2-arch1-1
KDE Plasma
Installed Arch packages

fcitx5-mozc
fcitx5-configtool
fcitx5-qt
fcitx5-gtk

Turns out it was a fcitx5 issue, not mozc.
Sorry about that. I'll close this issue.


Admonition V-draggable has[vite] Pre-transform error:

Hello,
When I paste the following code into slidev new ppt,
<Admonition title="V-draggable" color='teal-light' width="300px" v-drag="[65,279,287,160,7]">
If you want to `v-drag` an admonition, you should set the width to a fixed value.
</Admonition>

the following error occured.
14:22:12 [vite] Pre-transform error: Attribute name cannot contain U+0022 ("), U+0027 ('), and U+003C (<).
14:22:12 [vite] Pre-transform error: Attribute name cannot contain U+0022 ("), U+0027 ('), and U+003C (<). (x2)
14:22:12 [vite] Internal server error: Attribute name cannot contain U+0022 ("), U+0027 ('), and U+003C (<).
  Plugin: vite:vue

I find that the title of Admonition' prop cannot have v-drag.
<Admonition title="v-drag" color='teal-light' width="300px" v-drag="[65,279,287,160,7]">
If you want to `v-drag` an admonition, you should set the width to a fixed value.
</Admonition>

The error as follows:
|  </AdmonitionType>
  6  |  <Admonition title="v-drag :markdownSource="[4,5,19]""  color='teal-light' width="300px" v-drag="[56,319,300,68]">
     |                                                      ^
  7  |  This is my admonition content.
  8  |  </Admonition>

The page 50 of the example.md caused the above error. The title text cannot starts with 'V-drag'. Anything else is okay. I guess the parser treated the v-drag as a tag.


First version of Spock test runner

Initial implementation of Arquillian Spock Runner.

Works with ARQ 1.0.0.Final-SNAPSHOT
Tested against Weld EE Embedded

rebased and pushed upstream
Also made a "Spock TestRunner" component and a spock_1.0.0.next version in JIRA.
Your now officially Spock Module Lead.  Thanks a lot! I feel honoured.


POI 4.0.0 uptake

POI 4.0.0 is about to be released. I can fix up the build file when the POI jars are fully released.

I have a version of this change published to sonatype snaphots repo - https://github.com/pjfanning/excel-streaming-reader

Hi @monitorjbl,
did you already have a look at this PR?
I'm using excel-streaming-reader in https://github.com/crealytics/spark-excel and would like to build a new version with POI 4.0.

Hi @monitorjbl,
Now that everybody is thinking of moving to JDK 11 we need an updated version of the libraries and I was thinking of moving to POI 4.0 as well. Can you give it a look?

@borjapenman I have a POI 4.0.0 compatible release done at https://github.com/pjfanning/excel-streaming-reader - I've just added openjdk11 testing)

Hey guys, sorry for taking so long to review things. It's been kind of a hectic summer and I haven't had much time to spend on anything in GitHub. Things are (hopefully) winding down now, I'm reviewing the open PRs now.

Would you be able to merge in your forked version of @zwom 's fix in #154 as well?

Thanks a lot!


Path assertion for file contents is unusable if you don't use the default filesystem

It is unusable because of this particular line:
  File actualAsFile = actual.toFile();

This can never work for anything else than the default filesystem.

Hi fge,
what version are you referring to? I can not find the File actualAsFile = actual.toFile(); anywhere in current the codebase.
I guess this is a duplicate of https://github.com/joel-costigliola/assertj-core/issues/439 which was fixed with https://github.com/joel-costigliola/assertj-core/commit/cba1f870e41815fbf85497510d8744c1a3b6ef58 ?

@PascalSchumacher this is mockito-core 1.10.19.

@PascalSchumacher this is mockito-core 1.10.19.

???

Oops, sorry, brain fart...
assertj-core 3.1.0.

Please give assertj-core 3.2.0 a try. The issue should be fixed in 3.2.0.

@fge have you checked 3.2.0 ?

@joel-costigliola I have, and it indeed solves the problem. Thanks!
Closing issue.


Validation is only estimated on part of the images

Validation images undergo FixScaleCrop, that is, only the central rectangular region of the validation image participate in evaluating the model. The boundaries of the image are ignored.
When taking into account the entire validation images the performance of the trained model (resnet backbone) reduces from mIoU=78.43% to 77.89%

Hi @shaibagon, how did you test it on the whole image?
I just converted the batch size to 1 and removed that transformation in dataloader, but I am getting following results:
Acc:0.9413967692026853, Acc_class:0.8627290601827288, mIoU:0.7544719867976595, fwIoU: 0.8950505045612793
Loss: 269.871

Also, I used given pretrained resnet model with default parameters (except batch-size, which was set to 1)


Invalid new account Flow balance

Describe the bug
The balance of newly created emulator accounts in Flowser UI apparently doesn't match the one from Flow CLI.

See Discord for more context: https://discord.com/channels/613813861610684416/1047281367320383558/1047550099359801414
Context

Macbook M1

https://github.com/onflow/fcl-js/issues/1502


aws-region not working

I can only deploy to Singapore (ap-southeast-1) when I used the awscli package and specify my default location as ap-southeast-1. I dont think 'aws-region' within my zapp-settings.json works I am afraid.
Cheers,
Andy.
    "dev": {
        "django_settings": "aci_data_backup.settings",
        "profile_name": "default",
        "s3_bucket": "zappa-aci-data-backup",
        "aws-region": "ap-southeast-1"
    }

Use _, not -.

May I ask, is there an error reported when I've a config setting wrong? Didn't spot one, but sort of thing I wouldn't see!  Thanks, Andy.


fix int8 skip module config

Type of Change
feature or bug fix or documentation or others
API changed or not
Description
detail description
JIRA ticket: xxx
Expected Behavior & Potential Risk
the expected behavior that triggered by this PR
How has this PR been tested?
how to reproduce the test (including hardware information)
Dependency Change?
any library dependency introduced or removed

offline validated chatglm2, dolly v2 3b.

pls update other configs (rtn, gptq and others)
https://github.com/intel/intel-extension-for-transformers/blob/b400cb9d31bd009eb4e54d04941f3586e878079e/intel_extension_for_transformers/transformers/utils/config.py#L834

thanks review, improve.


Reduce context switching during inference

~/code/llama.cpp/main --model ~/code/crabml/testdata/open-llama-3b-q8_0.gguf   13.15s user 0.31s system 190% cpu 7.073 total
./target/release/crabml-cli -m ./testdata/open-llama-3b-q8_0.gguf  --steps 10  13.09s user 0.63s system 190% cpu 7.188 total

now, the performance gap mainly resides in the system's time cost.
additionally, crabml hitted 134,202 context switches after generating 100 tokens, whereas llama.cpp only switched contexts 20,476 times. that's over six times more context switches for crabml.

i once made a benchmark over sgemv with rayon and threadpool, it shows that rayon cost about 1 times larger system times.
i'll try use a vanilla thread pool instead of rayon on the matmul primitive

the x86_64 one is still not optimized yet. but this is the perf stat of llama.cpp and crabml:
 Performance counter stats for './main --model /home/flaneur/code/crabml/testdata/open-llama-3b-q8_0.gguf -p captain america -n 100 -t 16 --temp 0.0':

         96,107.79 msec task-clock                       #   15.209 CPUs utilized
             1,963      context-switches                 #   20.425 /sec
               267      cpu-migrations                   #    2.778 /sec
             6,908      page-faults                      #   71.878 /sec
   175,679,285,515      cpu_atom/cycles/                 #    1.828 GHz                         (51.47%)
   270,608,002,912      cpu_core/cycles/                 #    2.816 GHz                         (90.85%)
   140,636,195,017      cpu_atom/instructions/           #    0.80  insn per cycle              (60.60%)
   283,841,570,592      cpu_core/instructions/           #    1.62  insn per cycle              (90.85%)
    15,189,781,480      cpu_atom/branches/               #  158.049 M/sec                       (60.26%)
    42,033,830,392      cpu_core/branches/               #  437.361 M/sec                       (90.85%)
        54,878,243      cpu_atom/branch-misses/          #    0.36% of all branches             (60.84%)
        22,452,938      cpu_core/branch-misses/          #    0.15% of all branches             (90.85%)
             TopdownL1 (cpu_core)                 #     79.1 %  tma_backend_bound
                                                  #      0.5 %  tma_bad_speculation
                                                  #      1.8 %  tma_frontend_bound
                                                  #     18.7 %  tma_retiring             (90.85%)
             TopdownL1 (cpu_atom)                 #      7.1 %  tma_bad_speculation      (61.55%)
                                                  #      1.4 %  tma_frontend_bound       (61.95%)
                                                  #     70.1 %  tma_backend_bound
                                                  #     70.1 %  tma_backend_bound_aux    (62.08%)
                                                  #     21.6 %  tma_retiring             (61.60%)

       6.318954083 seconds time elapsed

      93.098646000 seconds user
       1.114657000 seconds sys

flaneur@databend-arch ~/c/llama.cpp (master)> perf stat  ~/code/crabml/target/release/crabml-cli -m ~/code/crabml/testdata/open-llama-3b-q8_0.gguf "captain america" --steps 100 -t 0 -T 32
captain america: the winter soldier
Captain America: The Winter Soldier (2014)
Captain America: The Winter Soldier (2014) is a movie starring Chris Evans, Scarlett Johansson, and Sebastian Stan. After the cataclysmic events in New York with The Avengers, Steve Rogers, aka Captain America is living quietly in Washington, D.C. and trying to adjust to the modern world. But...
Action, Adventure, Sci-Fi, Thr
6.0286927 tokens/s, 32 threads

 Performance counter stats for '/home/flaneur/code/crabml/target/release/crabml-cli -m /home/flaneur/code/crabml/testdata/open-llama-3b-q8_0.gguf captain america --steps 100 -t 0 -T 32':

        390,387.81 msec task-clock:u                     #   22.504 CPUs utilized
                 0      context-switches:u               #    0.000 /sec
                 0      cpu-migrations:u                 #    0.000 /sec
            59,760      page-faults:u                    #  153.079 /sec
 1,280,909,384,046      cpu_atom/cycles/u                #    3.281 GHz                         (18.40%)
 1,719,393,263,846      cpu_core/cycles/u                #    4.404 GHz                         (66.26%)
 3,134,046,354,957      cpu_atom/instructions/u          #    2.45  insn per cycle              (21.50%)
 3,356,674,127,857      cpu_core/instructions/u          #    2.62  insn per cycle              (66.26%)
   195,715,139,717      cpu_atom/branches/u              #  501.335 M/sec                       (21.52%)
   211,425,061,602      cpu_core/branches/u              #  541.577 M/sec                       (66.26%)
       570,974,945      cpu_atom/branch-misses/u         #    0.29% of all branches             (21.54%)
       650,523,749      cpu_core/branch-misses/u         #    0.33% of all branches             (66.26%)
             TopdownL1 (cpu_core)                 #     16.2 %  tma_backend_bound
                                                  #      3.7 %  tma_bad_speculation
                                                  #     12.3 %  tma_frontend_bound
                                                  #     67.9 %  tma_retiring             (66.26%)
             TopdownL1 (cpu_atom)                 #      5.5 %  tma_bad_speculation      (21.54%)
                                                  #      2.9 %  tma_frontend_bound       (21.54%)
                                                  #     41.6 %  tma_backend_bound
                                                  #     41.6 %  tma_backend_bound_aux    (21.53%)
                                                  #     49.8 %  tma_retiring             (21.49%)

      17.347741896 seconds time elapsed

<PHONE_NUMBER>00 seconds user
      35.483865000 seconds sys

it shows that llama.cpp has a much fewer page faults amount

imho, the system time cost of 0.2s is not exactly the wall time elapsed.
we have a thread pool that polls tasks from a channel. If a task is not busy, it is considered to be full with futex wait syscalls. however, this does not impact the wall time.
We should focus more on the wall time aspect.


about config

it is can use an config file eg: 1.conf ?  i think this is easy config.
If I want to use multiple data sources at the same time, how do I configure its?
Also I did not understand how to add my own prefixes list in the configuration file?
how use ssh mode for frr?  this is need via ssh login wich port for config?  Some parts of the help file do not understand.

Hello,

I did not understand the question? There is no configuration file, everything is passed via the command line.
You can only use one JSON per RTR server. But you can have as many RTR servers as you want.
GoRTR does not have this feature. Although, it's a request I have had a few times (enabling local configured prefixes). In the meantime: I would suggest to pre-process the JSON that is provided to GoRTR.
I have never configured it with FRR. You will have to follow the guide. To configure GoRTR, the steps are there.

1.I mean it is recommended to increase the way you use the configuration file.
2.Add multiple sources to increase reliability. If a source data has problems or cannot be obtained, using multiple sources will not cause service interruption.
3.RIPE allows you to host management RPKI, How do it add an json file?  this is need creat new any files?

That's more of a preference of mine not to have GoRTR manage configuration files. I do not really see what would require a configuration file.
If there is an issue, it will keep using the cached data. This would add more complexity in case the two data sources are different (eg: different prefixes in the second one). This is why I'd suggest to run multiple GoRTR.
You have to add your prefixes on the RIPE RPKI dashboard here: https://my.ripe.net/#/rpki

i install the gortr_0.11.4_amd64.deb, but i can't find the /etc/gortr folder .
ubuntu 19.0.4   Linux 5.2.9-050209-lowlatency x86_64

The package definition is here:
https://github.com/cloudflare/gortr/blob/master/Makefile#L53-L56
The configuration file is in /etc/default/gortr

i mean is lost a lot of files in /etc/gortr  like Gopkg.toml  etc...
an server not creat /etc/gortr
The configuration file is in /etc/default/gortr  only show
GORTR_ARGS=

Yes, you can populate:
GORTR_ARGS= -cache https://example.com/rpki.json -verify=false in /etc/default/gortr
for instance

@liuxyon started a branch that implements SLURM in order to do overrides and filters (#34):
https://github.com/cloudflare/gortr/tree/feature/filter
Commenting here as this was one of your concerns.


Add feature to stack duplicate flags

I noticed your recent change log enables flag merging for duplicate flags. It would be great if this feature could also stack the effect (or add a function to allow stacking) either using the token stack module or something similar. This would be very helpful for effects like bleeding where the effect (and dae effects) could stack on top of itself over time

That was actually referring to a different kind of flag merging. But I get your idea. I think that kind of falls outside the scope of this module... Could make a good separate module though


Fix return type in AMQPConsumer constructor

Originally only had void as the return type, and the example on the README.md showed a Promise being returned, causing yellow squiggles from Typescript / Sonarlint.

Isn't that because your onMessage callback returns a Promise. In that case I guess we should change to any in the lib since we can't control what users of the lib return in their callbacks. WDYT @carlhoerberg?


Fix check release

That works! The visual regression test failing may be sporadic, let's ignore for now


[Test] Create a proper test for pre-aggregation refreshing

Create a proper test for pre-aggregation refreshing
see https://github.com/Canner/canner-metric-layer/pull/252

legacy issue.


Remove hcc support

Since hcc has been deprecated for a while now, this change removes support in tensile. This change currently leaves options and global params like --cxx-compiler and CxxCompiler in place.

CI is not running on this due to the branch conflict. Not sure if gfx908 shortages are impacting this.

There wasn't any merge conflict when the timeouts took place earlier this morning and yesterday.  I just merged in a guest PR that caused this merge conflict within the past hour or so.

LGTM.
Only thing I would ponder is whether to keep the tensile option for cxx-compiler in case we needed to need to differentiate with a future release again with another supported compiler. Otherwise, nice to clean a bunch of stuff up :)

Thanks for reviewing! Yea, I left the cxx-compiler option in for now, we can review if it should also be removed later.


Allow to bind the webserver to a specific address

Add an option to allow users to choose to what IP address the web server must bind. Default to <IP_ADDRESS> (all IP addresses).

Coverage decreased (-0.08%) to 80.932% when pulling ee00db2086dbf9144ecc0321c36c166da44e4c91 on plumbeo:bind-address into f43e5f18d58c73b10aea3d7ca603e916bed5ed9e on airbnb:master.

What about using -b as gunicorn does?

@xrmx I used -a because caravel already uses the -p flag to select the port while gunicorn uses -b to select both address and port at the same time, so I thought using the same flag with a different meaning would be confusing. I can change it, in case.

@plumbeo yeah, make sense :)

LGTM


Index

Codecov Report

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

@@          Coverage Diff          @@
##           master     #8   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           4      5    +1
  Lines          85     89    +4
  Branches        6      7    +1
=====================================
+ Hits           85     89    +4

Impacted Files
Coverage Δ

src/statistics.js
100% <100%> (ø)
:arrow_up:

src/redisGraph.js
100% <100%> (ø)
:arrow_up:

src/label.js
100% <100%> (ø)

src/resultSet.js
100% <100%> (ø)
:arrow_up:

src/record.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[mimalloc] update to 2.0.5

Library name: mimalloc
New version number: 2.0.5
Other information that may be useful: https://github.com/microsoft/mimalloc/releases/tag/v2.0.5

Fix by #23264


Adds test about bidirectional rules

Ticket
Redmine ticket: https://redmine.openinfosecfoundation.org/issues/5665
#1910 with more tests

negative for frames
positive for tls with ja3 and ja3s (worked out of the box)

Would like to see more test cases, for different protocols, as well as --engine-analysis output tests.

Would like to see more test cases, for different protocols

Which ones besides tls and http ?

as well as --engine-analysis output tests.

Doing that

Next in https://github.com/OISF/suricata-verify/pull/2083


Update securitycontextconstraints.yaml

ordered volume list in falco scc yaml alphabetically

What type of PR is this?
/kind bug
/kind chart-release
/area falco-chart
What this PR does / why we need it:
Making the helm chart more robust by re-ordering configuration in the falco helm chart scc yaml
Which issue(s) this PR fixes:
Sync errors in ArgoCD as a result of the unordered list which is ordered by the kube-api alphabetically. Argo then compares plain yaml and fails

Fixes #693
Special notes for your reviewer:
Checklist

[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] CHANGELOG.md updated

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: toamto94
Once this PR has been reviewed and has the lgtm label, please assign cpanato for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @toamto94! It looks like this is your first PR to falcosecurity/charts 🎉

Thanks @toamto94,
To be valid, your changes must come with a bump of the chart version and an update of changelog.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: toamto94
Once this PR has been reviewed and has the lgtm label, please ask for approval from issif. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

5c3abcf Merge branch 'fb_toamto94-patch-scc-v2' of https://github.com/toamto94/charts into fb_toamto94-patch-scc-v2

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Adding label do-not-merge/contains-merge-commits because PR contains merge commits, which are not allowed in this repository.
Use git rebase to reapply your commits on top of the target branch. Detailed instructions for doing so can be found here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

5c3abcf Merge branch 'fb_toamto94-patch-scc-v2' of https://github.com/toamto94/charts into fb_toamto94-patch-scc-v2

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

c731c20 Revert "ordered volume list in falco scc yaml alphabetically"

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

changelog and chart bump has been added

Hey @toamto94, could you update the docs by running make docs?

Hey @toamto94, could you update the docs by running make docs?

done :)

LGTM label has been added.  Git tree hash: 5fa9ff93775c6499a0c67203122a24f32e78b3d0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: alacuku, toamto94
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [alacuku]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[CI]

Current Behavior
<EMAIL_ADDRESS>Desired Behavior
Alain2200
Implementation

Acceptance Tests

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation
Meshery documentation site and source
🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack
🧪 Meshery Test Plan Spreadsheet

Is there any additional info you want to give around this issue?

@Alainmma can you elaborate more about the issue?

Please create new issue with proper title and description. Thank you!