Request credentials for an os-climate bucket

Requesting credentials for github user: anotb
Requesting credentials for the bucket: redhat-osc-physical-landing-647521352890
Email address to send a credential download link<EMAIL_ADDRESS>Request checklist:

[X] User can establish connection with the bucket
[X] User can read files from the bucket
[X] User can write files to the bucket

closing - let us know if you don't have access


how to test your denoising function？

Thank you for your outstanding work!  But I don't know how to test your denoising function. How can I input a wild motion and use your algorithm to remove the noise?

You should use the Encoder, whose code is not published yet. We plan on publishing the encoder code on June 2023 (delayed due to other commitments).
As a temporary solution, you can use our Optimizer code, which is already released. The results are good enough for motions that are not extremely out of the distribution on which the network has been trained.
Let me know if this helps.

Hello, I have tried the inverse optimization using my motion capture data. But the resulting motion is shaky and the feet does not lie on the ground. Is it common to encounter this situation? Is there any advice to make the final motion smooth and stable?

To achieve the best results with the encoder (or the optimizer) the motion capture data needs to be from the same distribution as the training data - e.g. have the same skeleton topology, bone lengths, similar root position values. You may need to align your motion capture data with the training data to achieve this. This should improve the results.
Hope this is helpful, let me know if you have further questions.

Thank you for your quick reply. I have retargeted the motion capture data to mixamo skeleton and run the preprocess_edges script provided in the repo. I think that the resulting data should be consistent with the training data in distribution. But the reconstructed motion is still floating. I am not sure if I am testing an outlier motion, because it is a dance motion that moves quite fast. Could you offer me more advice on this?

Dear cyj907, it seems you are doing the correct preprocessing. To provide further assistance, we would need to review your data. Regrettably, this can be a rather time-consuming task and the capacity of our team is fully utilized for other tasks

Dear cyj907, it seems you are doing the correct preprocessing. To provide further assistance, we would need to review your data. Regrettably, this can be a rather time-consuming task and the capacity of our team is fully utilized for other tasks

Thank you for your reply. Finally I decide to train MoDi using my own data and see if it is a problem of data distribution. But I have a hard time figuring out the meaning of edge_list in Edge class in data.py. Could you tell me what the numbers mean in the tuple? Is it the index of joint? And the tuple means the edge that connects two joints? If so, what is the corresponding joint orders? I print out the joints in names_with_root, but it seems unrelated to edge_list. I'd appreciate it very much if you offer me some explanation on the Edge class. Thank you.

Let's set up a Zoom meeting where I can explain stuff in detail.
Please write to my email<EMAIL_ADDRESS>and let me know your time zone and when you are free.


Cache size of last ReadAt on S3 for performance

Each obj.Stat() call adds another request to the S3 endpoint
for some commands a lot of ReadAt calls are made for the same
object in S3. This patch essentially cuts the number of calls
to S3 in this case in half. Speeding up the progress and lowering
costs to S3.
It is assumed that the object in S3 should not change and therefor
the cached size should be valid. (If it would not be then the entire
restic process would have false information since it does two distinct
reads on a file which has changed in the mean time ?)
Note: in my case the s3-backend.log show a lot of
(almost) consecutive byte ranges like:
Range: bytes=3352327-3356150
Range: bytes=3356151-3357129

So there is a further improvement to be made there I think :)

Unscientific comparison runs of restic prune
[13:58] 1.96%  1 / 51 snapshots
vs
[2:58] 1.92%  1 / 52 snapshots

Codecov Report

Merging #883 into master will increase coverage by 0.1%.
The diff coverage is 71.42%.

@@            Coverage Diff            @@
##           master     #883     +/-   ##
=========================================
+ Coverage   53.66%   53.77%   +0.1%
=========================================
  Files          92       92
  Lines        7625     7632      +7
=========================================
+ Hits         4092     4104     +12
+ Misses       2919     2915      -4
+ Partials      614      613      -1

Impacted Files
Coverage Δ

src/restic/backend/s3/s3.go
79.09% <71.42%> (+3.21%)
:white_check_mark:

src/restic/archiver/archiver.go
64.96% <0%> (+0.22%)
:white_check_mark:

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 bf30b28...b523eef. Read the comment docs.

The original patch only kept stat history for one file. Despite that, it was enough for a dramatic improvement in prune time.
The updated patch does keep the history for longer. It's definitely better, but, for a very large restore task it might cause large memory growth..
I think it's better to refactor each backend.Load to not support negative offset. The feature is only used in one place. Then s3.Load does not need to call stat at all.


Update scala-reflect to 2.13.8

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

Files still referring to the old version number
The following files still refer to the old version number (2.13.6).
You might want to review and update them manually.
modules/install/src/test/scala/coursier/install/InstallTests.scala

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

labels: library-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

Possibly blocked by https://github.com/typelevel/simulacrum/pull/226.

Superseded by https://github.com/coursier/coursier/pull/2631.


Fix types for getWorksheet()

Summary
Fix types for getWorksheet(). It can return undefined
Test plan
N/A
Related to source code (for typings update)
lib/doc/workbook.js#L143

Tests on node v8 failed. I will fix them before the final merge:
https://github.com/exceljs/exceljs/issues/2237


feat: create list page

create list page
improve prettier

Blud, this is hard-coded. I suppose the buttons to add or remove items on the list will be implemented with Angular?

yes :)


Allow using micro-dev programmatically

Hi! I wanted to submit an idea I'm using myself with ts-node-dev in a TypeScript + (Apollo) GraphQL project where I use micro as my server library.
Problem
Using a package.json script such as the following:
"start": "ts-node-dev node_modules/.bin/micro-dev -p 4000 src/server.ts",

causes the process to crash on syntax errors etc. With micro this can be fixed by programmatically using micro:
# package.json
"start": "ts-node-dev src/server.ts"

# server.ts
const server = require('micro')(handler)
server.listen(PORT)

However, as we all know (https://github.com/zeit/micro/issues/337) micro-dev does not support programmatical usage. While many of the features are CLI specific, some are not.
Solution
With this PR in place, one can do something like:
const PORT = process.env.PORT || 4000

if (process.env.NODE_ENV === 'development') {
  const microDev = require('micro-dev')
  microDev({ silent: false, limit: '1mb', host: '::', port: PORT })(handler)
} else {
  const server = require('micro')(handler)
  server.listen(PORT)
}

Comments

you guys might prefer https://github.com/zeit/micro/issues/337#issuecomment-365670943
the message part breaks DRY but we can clean that up if this direction feels correct
we're not really supporting too many micro-dev features
it might make sense to not call .listen within micro-dev when using the library like this. It would make this PR simpler and the usage a bit cleaner:

const server = require('micro')(handler)
if (process.env.NODE_ENV === 'development') {
  server = require('micro-dev')({ silent: false, limit: '1mb'})(handler)
}
server.listen(PORT)

It nothing else, this might act as a good reference for other micro & TypeScript users!
Cheers :)

Would be nice to release this change. Really appreciated.

Please release this change. Much appreciated!

Since this isn't published yet, someone should add another PR to return the server object from the , so that you can programmatically call:
  server.close(function () {
    console.log('Server shutdown.')
    process.exit()
  })

Could we get a release with this? Now that next includes an API system - it would be really nice to include dev output via the next runtime.

please release this change :) thanks


Upgrade cryptography dependency to 44.x

Describe the bug
Python dependency cryptography released yesterday version 44.0.0. Since then, we are having a dependency conflict resolution because pyseto requires a cryptography version lower than 44.0.0. So, is there any plan to upgrade this dependency?
To Reproduce
Steps to reproduce the behavior:
Try to update cryptography package to the latest (44.0.0) while having pyseto also installed on the same virtualenv.
Expected behavior
No dependency conflicts while trying to install latest versions of both cryptography and pyseto.
Thanks,
Jose M. Prieto

@PrieJos Thank you for filing the issue! I've fixed  and released v1.8.1.

@dajiaji Thanks so much for the prompt reaction on this.


Bug: Canary tries to create a domain with retention=0 which is not allowed by the default MinRetention

See https://github.com/uber/cadence/blob/663f74cf81acb69aad1db3451b9201da89a5653a/canary/client.go#L54
And https://github.com/uber/cadence/blob/663f74cf81acb69aad1db3451b9201da89a5653a/common/domain/const.go#L27

Retention = 0 is required for the archival canary test to pass I think.
In the default dynamicconfig for local environment we have set the min retention day to 0. https://github.com/uber/cadence/blob/master/config/dynamicconfig/development.yaml#L4
For other environments the domain should be manually registered for now. This is a known issue: https://github.com/uber/cadence/issues/2001
We can probably add a logic in canary to only register the domain in local/development environment.


Empty space in attribute

I can't seem to purposely leave a blank space in an attribute. Specifically for ng-list where I want to use a space as a delimiter.
Example jade file. Runs as expected running in gulp test server.
input(type="text", name="someField", ng-model="vm.someModel", ng-list=" ", ng-trim="false")
The resulting html after a gulp build is:
<input type="text" name="someField" ng-model="vm.someModel" ng-list="" ng-trim="false">
My gulp build.js includes:
.pipe($.minifyHtml({
      empty: true,
      spare: true,
      quotes: true,
      loose: true
    }))

I have tried alternating using empty or spare individually with the same results. Is there another option or a workaround?

This is probably due to the trim() of attributes, see https://github.com/Swaagie/minimize/blob/master/lib/helpers.js#L158. That said you can use the whitespace option to prevent the trim. Closing this issue.


UUID Cast Error

Took me a while to track down the change in #2056, butUUID.cast/1 fails rather non-obviously if UUIDs don't conform exactly to the UUIDv4 spec. I have some legacy data where integer primary keys had been migrated to UUIDs by padding with zeros (e.g. "00000000-0000-0000-0000-000000000034")
Resulting error:
** (ArgumentError) cannot load `<<0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 34>>` as type :binary_id for :id in schema Repo.User
    (ecto) lib/ecto/schema.ex:1418: Ecto.Schema.load!/5
    (ecto) lib/ecto/schema.ex:1399: Ecto.Schema.__load__/5
    (ecto) lib/ecto/schema.ex:1371: Ecto.Schema.__load__/6
    (ecto) lib/ecto/adapters/sql.ex:582: anonymous fn/4 in Ecto.Adapters.SQL.process_row/4
    (elixir) lib/enum.ex:1357: Enum."-map_reduce/3-lists^mapfoldl/2-0-"/3
    (ecto) lib/ecto/adapters/sql.ex:579: Ecto.Adapters.SQL.process_row/4
    (postgrex) lib/postgrex/query.ex:77: DBConnection.Query.Postgrex.Query.decode_map/3
    (postgrex) lib/postgrex/query.ex:64: DBConnection.Query.Postgrex.Query.decode/3

Unsure whether the validation should be reversed or just better noted in the docs.
Environment

Elixir version: 1.5.0
Database and version: Postgres 9.4
Ecto version (mix deps): 2.1.5
Database adapter and version (mix deps): postgrex 0.13.0
Operating system: Debian

Does Postgres accept those as valid UUIDs?

Accepts them just fine. I think the uuid type was meant to be as generic.

We relaxed the validations in the 2.1 branch and released 2.1.6 with the fix.
The validations will be kept on the 2.2 branch and Ecto.UUID will only accept UUIDs conforming to the specification in https://tools.ietf.org/html/rfc4122

@michalmuskala As far as I can tell, RFC 4122 only uses the  version and variant for the creation of Time-Based, Name-Based, and Random Number UUIDs.
In fact, Section 3 states:

Validation mechanism:
Apart from determining whether the timestamp portion of the UUID
is in the future and therefore not yet assignable, there is no
mechanism for determining whether a UUID is 'valid'.

The example implementation at the bottom of the RFC does not validate UUIDs.
ITU-T Rec. X.667 also states in Section 10:

Apart from determining if the variant bits are set correctly, and that the Time value used in a time-based UUID is in the future (and therefore not yet assignable), there is no mechanism for determining if a UUID is valid in any real sense, as all possible values can otherwise occur.

Other standards, like ISO/IEC 9834-8, also focus primarily on the generation of UUIDs.
Neither PostgreSQL, MySQL, MongoDB, Oracle, nor SQL Server validate UUIDs in any way beyond checking that they represent 128-bits, 16-bytes, or are composed of 32-bytes of hexadecimal characters (36 if you include the hyphens).

PostgreSQL
MySQL
MongoDB
Oracle
SQL Server - this one is my favorite: you can even have extra data at the end of the UUID and its still valid :confused:

The current (Ecto 2.2) UUID validation also rejects the Nil UUID as specified in Section 4.1.7 of RFC 4122:
iex> Ecto.UUID.cast(<<0::128>>)
:error

I use ULID for UUIDs that are lexicographically sortable, which is not currently compatible with Ecto 2.2.
So, all of that said: I do not think there is any basis that Ecto should be validating UUIDs in its current form (ie. validating version and variant).

Also, I'd be happy to submit a pull request, but wanted to see if anyone had a differing opinion first 😀

Thank you @potatosalad! I have reverted those changes and pushed to master!

Huzzah!  My weird ids work again!  Thanks, @josevalim.

I guess I can't say anything other than: "I'm convinced" 😃

I wonder, I have something similar with JSON, I have
** (ArgumentError) cannot load `"{}"` as type :map


fix a typo

This is just a typo. But this makes a issue when I tried to train without temporal symmetry.

Thanks for your help. Merged.


add "key:" prefix to Espresso keySchema

add "key:" prefix to Espresso keySchema (e.g. "key:fieldName01")
support input json with empty "documentSchema"

Coverage remained the same at 0.0% when pulling 25af80b7c4e49b623827b7d01c6db52110261334 on ericsun2:linkedin into 0a1fb40ca11e3eb9db92a079d970f8a1287dce36 on linkedin:linkedin.


fix: Blocked Loading Mixed Active Content Issue Prevents Chart Viewing on Flexmeasures App

This update addresses the problem that was preventing the viewing of charts on the Flexmeasures app. By modifying the base.html file, we now perform a check for the flask env variable. If the environment is set to production, the app will utilize the HTTPS protocol; otherwise, it will use HTTP. This adjustment ensures that the loading of mixed active content is unblocked, enabling smooth chart viewing on the app.

Could we access to that PythonAnyWhere hosted app from external internet?
Yes, I have deployed a docker container app and I don't think this issue would only happen on GCP. Maybe we could face it on other cloud provider too.

I do believe this is likely to happen in other production scenarios, but why is PA different? Would be good to understand. We can look at the network tab on the production system and check what URL is called for this API endpoint by the asset page there...


Error in OracleFMWInfrastructure container run: com.oracle.cie.domain.ValidateException: 40318: Invalid or missing JDBC datasource connection parameters.

I am recieving a JDBC URL validation error when running a container with the "oracle/fmw-infrastructure:<IP_ADDRESS>" image:
docker-images/OracleFMWInfrastructure/dockerfiles/<IP_ADDRESS>/Dockerfile
DB Image:
oracle/database-enterprise:<IP_ADDRESS>
The container log:
https://pastebin.com/WmXKTUP9
The RCU log:
https://pastebin.com/ZxQy0rtY
The WLST offline log (ERROR @ LN 445):
https://pastebin.com/h1twW3rR
It seems the RCU schemas properly get loaded into the DB. But during domain creation the createInfraDomain.py script fails validation of JDBC url. However, from what I see in the logs nothing is obviously incorrect.
Any help is appreciated! I can also provide more detail if needed.
Docker info:
Containers: 3
Running: 2
Paused: 0
Stopped: 1
Images: 107
Server Version: 18.03.0-ce
Storage Driver: overlay2
Backing Filesystem: extfs
Supports d_type: true
Native Overlay Diff: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
Volume: local
Network: bridge host ipvlan macvlan null overlay
Log: awslogs fluentd gcplogs gelf journald json-file logentries splunk syslog
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: cfd04396dc68220d1cecbe686a6cc3aa5ce3667c
runc version: 4fc53a81fb7c994640722ac585fa9ca548971871
init version: 949e6fa
Security Options:
seccomp
Profile: default
Kernel Version: 4.9.87-linuxkit-aufs
Operating System: Docker for Windows
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 7.768GiB
Name: linuxkit-00155d630101
ID: TAWZ:HTPZ:GBRW:NONR:QCFN:34KX:KINW:IXJ7:Q45J:4AWS:MAM2:FX7P
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
File Descriptors: 37
Goroutines: 58
System Time: 2019-11-25T19:28:44.5834882Z
EventsListeners: 2
Registry: https://index.docker.io/v1/
Labels:
Experimental: true
Insecure Registries:
<IP_ADDRESS>/8
Live Restore Enabled: false

@anpanigr could you please take a look and see why this is not working?

@mriccell, @anpanigr as an FYI: I ended up switching to a linux host and was having an easier time getting this image working. Now that I understand how to get it working on a linux host I will most likely attempt to get this working in windows again in the next few days.
I will let you know if I can more accurately re-create this issue, and hopefully tell you how.

I will close this issue but please reopen or enter a new one if you still run into problems.


Add LIBSPDM_CERT_PARSE_SUPPORT for device image size reduction.

Fix https://github.com/DMTF/libspdm/issues/1761
Signed-off-by: Jiewen Yao<EMAIL_ADDRESS>

@steven-bellock , any additional feedback?

Either as part of this pull request, or a subsequent pull request, update libspdm_macro_check.h  to check what capabilities are needed for LIBSPDM_CERT_PARSE_SUPPORT to be 1 (or 0).

Either as part of this pull request, or a subsequent pull request, update libspdm_macro_check.h to check what capabilities are needed for LIBSPDM_CERT_PARSE_SUPPORT to be 1 (or 0).

Current design is: responder may set it to 0 (no mutual auth) or 1 (mutual auth). requester should set it 1.


Short title for Newcastle Declaration in header

Using the full title in the header/nav-bar takes up too much space. Optionally we could shorten it further to just Newcastle Declaration.

Done! The RTD preview doesn't work correctly, but I've tested it locally.


Randomise Empty Searches

Randomised emptySearches for all ticket types considering clients issue the ticket at the same time. (they won't match as they move to the add mode at the same time)

@hirokazutomoda
https://github.com/Diarkis/diarkis-server-template/pull/33#pullrequestreview-2362088933
I think so, and could you add comment?

@KohjiNakahata done


add all JSON.stringify args to writeJSON function

Hello,
First of all, thanks for this whole project, I really enjoy it.
However, I found myself wanting to indent the json content of my file but couldn't using the writeJSON from the lib.
I guess this shouldn't add any regressions, please let me know what you think about this 😄

Thank you! This seems useful and your code works well.
My only request is would you be able to take a pass at the README?
We would have to at the least:

update the inputs for writeJSON
ideally, keep the usage example, just add an extra one that shows the additional parameters

Thanks for the review, I had completely forgot the README part ! 😄
I used any to type replacer in the README to simplify as it's an overload with either (this: any, key: string, value: any) => any or (number | string)[] | null

Thanks!


Fetch LuaJIT from GitHub mirror on Travis

I've merged master with luabitop installation fix into luarocks-3 branch so the build should pass now.


Unexpected coloring of the token in the editor

While examining code in the diff editor, I noticed that the color schema appears to be off. Notice how in the following image, the this token on line 322 is colored green while it should actually be a dark blue (like the other this tokens).

I was using the following code:

Code

private _renderFoldingIconForLine(container: HTMLSpanElement, foldingModel: FoldingModel | null | undefined, index: number, line: number): FoldingIcon | undefined {
   const showFoldingControls: 'mouseover' | 'always' | 'never' =
   this._editor.getOption(EditorOption.showFoldingControls);
   if (!foldingModel || showFoldingControls === 'never') {
      return;
   }
   const foldingRegions = foldingModel.regions;
   const indexOfFoldingRegion = foldingRegions.findRange(line);
   const startLineNumber = foldingRegions.getStartLineNumber(indexOfFoldingRegion);
   const isFoldingScope = line === startLineNumber;
   if (!isFoldingScope) {
      return;
   }
   const foldingIconNode = container.appendChild(document.createElement('div'));
   const isCollapsed = foldingRegions.isCollapsed(indexOfFoldingRegion);
   foldingIconNode.className = ThemeIcon.asClassName(isCollapsed ? foldingCollapsedIcon : foldingExpandedIcon);
   const foldingIcon = new FoldingIcon(foldingIconNode, isCollapsed);
   foldingIcon.setVisible(isCollapsed || showFoldingControls === 'always');
   foldingIcon.setTransitionRequired(true);

   this._foldingIconStore.add(dom.addDisposableListener(foldingIconNode, dom.EventType.CLICK, () => {
	toggleCollapseState(foldingModel, Number.MAX_VALUE, [line]);
	foldingIcon.isCollapsed = !isCollapsed;
	const scrollTop = (
		isCollapsed ?
			this._editor.getTopForLineNumber(startLineNumber)
			: this._editor.getTopForLineNumber(foldingRegions.getEndLineNumber(indexOfFoldingRegion)))
		- this._lineHeight * index + 1;
	this._editor.setScrollTop(scrollTop);
   }));
   return foldingIcon;
}

Hey @hediet, not sure if I should assign this to you? Feel free to change assignment.

When I add a whitespace after ? it works:

globalThis.x = (
   globalThis ?
   1 : foobar
);

I see, the color schema is correct now too in the editor. I suppose that it had something to do with saving the document or formatting the document for the token colors to be rendered appropriately. I'll close the issue.

I think this is indeed a bug of the typescript grammar!


[BUG] Need adjust harvester chart version in harvester-installer before release

Describe the bug

After installer by iso, the job of harvester installation always Error
To Reproduce
Steps to reproduce the behavior:

Install harvester by iso

Expected behavior

The job of harvester installation will complete
Log

Error: failed to download "https://<IP_ADDRESS>:443/static/charts/harvester-0.1.0.tgz" (hint: running `helm repo update` may help)

Environment:

Harvester ISO version:
Installation Mode: ISO/app-mode
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630):

Additional context
Add any other context about the problem here.

Validation Version

master-3d9efd26-head

Steps

Install harvester by iso/app-mode
View helm-install-harvester pod and  harvester pod logs

kubectl logs -f helm-install-harvester-pmj2r -n kube-system
kubectl logs -f harvester-9cffd97b8-hq8ts -n harvester-system

Results

Successful deployment
No error reported in the logs


Implement mask-creation function

This is a feature request to implement mask-creation function to Chainer.
We use masking quite often when we have a variable length sequence, e.g. NLP applications (tokens) and ranking applications (series of items to be compared).
It would be very handy to have mask-creation function built-in to Chainer.
What I mean by mask-creation is like the one that is implemented in Tensorflow:
tf.sequence_mask([1, 3, 2], 5) =
  [[True, False, False, False, False],
   [True, True, True, False, False],
   [True, True, False, False, False]]

(from https://www.tensorflow.org/api_docs/python/tf/sequence_mask)
My suggestion is that we implement sequence_mask exactly as one implemented in Tensorflow.
It has 4 positive sides compared to directly implementing mask function:

You can easily use mask to do masking in values of your choice because you sometimes want to mask values with non-zero values. e.g. ListNet,
You can re-use the mask for performance,
You can use the mask for indexing with functions.get_items, and
Compatibility with Tensorflow

This function is non-differentiable to both the inputs and thus extremely easy to implement.
I would love to contribute by implementing and PR-ing this feature.
What you guys think of this feature request?

@koreyou thank you for this suggestion. Is it the case that the use case if very different from chainer.functions.pad_sequence?

No. chainer.functions.pad_sequence seems to be absolutely fine for this purpose. Sorry that I didn't notice this function.

Great. Again, thank you for the request. Tensorflow seems to have pad_sequences under contrib so I'll keep en eye out.


angle-grinder: 0.16 -> 0.17.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/angle-grinder/versions.
meta.description for angle-grinder is: "Slice and dice logs on the command line"
meta.homepage for angle-grinder is: "https://github.com/rcoh/angle-grinder"
meta.changelog for angle-grinder is: ""
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

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

/nix/store/vnz2dxdd8nmln1ckyd5pbgpfrf867cy5-angle-grinder-0.17.0/bin/agrind passed the binary check.

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

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

found 0.17.0 with grep in /nix/store/vnz2dxdd8nmln1ckyd5pbgpfrf867cy5-angle-grinder-0.17.0

found 0.17.0 in filename of file in /nix/store/vnz2dxdd8nmln1ckyd5pbgpfrf867cy5-angle-grinder-0.17.0

directory tree listing: https://gist.github.com/90f26adc1ba50d3c2c6de501ce2de421

du listing: https://gist.github.com/e692bd42cfc90604fa716dc2cd9eb2ee

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
angle-grinder

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/vnz2dxdd8nmln1ckyd5pbgpfrf867cy5-angle-grinder-0.17.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A angle-grinder https://github.com/r-ryantm/nixpkgs/archive/571a504e01a0ca1816f1d9a6695828a2e479b59f.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/vnz2dxdd8nmln1ckyd5pbgpfrf867cy5-angle-grinder-0.17.0
ls -la /nix/store/vnz2dxdd8nmln1ckyd5pbgpfrf867cy5-angle-grinder-0.17.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

angle-grinder

Maintainer pings
cc @bbigras for testing.

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

1 package built:

angle-grinder

Builds and run fine.

Thanks!


k3s io go powershell

Make embeddable
Remove link local addresses
Use charon from path if available
Preserve PATH for extension backend
Update logging for kube subnet manager
Log with klog instead of glog
Support k8s v1.18
Use pkg/errors instead of juju/errors
Update aws vpc backend for log formatting
Remove glide files
Add go.mod, udpate vendor
[migration k3s-io] update go-powershell dependency

ugh, not sure how this ended up here. thanks gh pr create --fill


How to handle InvalidCharacterError?

I started using this library as an alternative to atob in Safari. It handles more cases than the native implementation, but I still run into this error occasionally. It doesn't tell me which character is causing the problem, and I don't know how to handle it. What can be done to make atob more robust?

Could you provide example input for which atob throws an InvalidCharacterError?

I had an issue with this yesterday, but the user edited the page today and the offending characters are gone, and it started to work again. I'm going to update this issue if I see that error again.

Okay. Sounds good. :)


can't submit csv file to kaggle using kaggle api

In the current Ribonanza competiton user can do offline inference and then submit a csv,
I want to submit csv via kaggle-api to reduce manual labor, however I'm facing the following error
Is there anything I'm doing wrong?

Nevermind, after submitting zipped version submission went through


chore: add repository to npm package.json

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 15.65%. Comparing base (12359b3) to head (22a1792).
Report is 3 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #291       +/-   ##
===========================================
+ Coverage   12.29%   15.65%    +3.36%
===========================================
  Files        2246      187     -2059
  Lines      191038    16757   -174281
  Branches   174951      670   -174281
===========================================
- Hits        23479     2624    -20855
+ Misses     166392    14133   -152259
+ Partials     1167        0     -1167

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


add rbx-2.5.2

Add install support for Rubinius 2.5.2

Thanks Tom.

I’ll push a new release of ruby-build now.

You are very welcome, thank you so much for ruby-build!


Keyboard manager not opening at all

Microsoft PowerToys version
0.79
Installation method
WinGet
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce
Keyboard manager does not work, buttons are clickable, but if you click:
Show remappings:

Remap a key and Remap a shortcut does not open, the cursor does the loading animation for one second, but it will not open any window
Same behaviour for opening directly from the tab of Keyboard Manager:

✔ Expected Behavior
The keyboard manager opens the window where I can do the remapping
❌ Actual Behavior
The keyboard manager buttons do nothing!
Other Software
My windows installation was debloated using: https://github.com/ChrisTitusTech/winutil
I also run https://github.com/valinet/ExplorerPatcher

PowerToysReport_2024-03-29-14-14-11.zip
Here is the report a bug zip

Hi!
The issue you are experiencing is https://github.com/microsoft/PowerToys/issues/31708
Can you try the tests mentioned in https://github.com/microsoft/PowerToys/issues/31708#issuecomment-1986515217 and report the results in the issue?
/dup #31708


[Fix][Connector-V2] Fix known directory create and delete ignore issues

Purpose of this pull request
Fix known directory create and delete ignore issues:
When creating or deleting, should consider the role of ignore key
Does this PR introduce any user-facing change?
How was this patch tested?
Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist
Add ci label in label-scope-conf
Add e2e testcase in seatunnel-e2e
Update connector plugin_config

[ ] Update the release-note.

Could you add some test case? Thanks.

Could you add some test case? Thanks.

get


Status / Queues root queue bandwidth calculation fix. Issue #3381

[X] Redmine Issue: https://redmine.pfsense.org/issues/3381
[X] Ready for review

There is no need to do += for the root queue,
but only for child queues that are directly under the root queue
unused/outdated code is also deleted

Done


Save cache of functions in the cache file

Reading all function files takes up too much time when regenerating the cache file.

 If you were to cache them, then what on earth made you call it "regeneraet"?!


Gather download statistics weekly

Hi @haesleinhuepf,
I tried to eliminate all errors I could find in the script, but I think the API endpoint for my Github Access Token is incorrect. I receive a 404 Not Found error from the GitHub API when trying to create a branch or interact with the repository.
It would be super cool if you could test this PR and I am also happy to make adjustments, I just don't know how to solve this access problem. I have created Token(classic) with repo and workflow enabled.
This PR closes #144.
Best,
Mara

Hi @marabuuu ,
thanks for working on this! I had to fix some issues on the python side. Furthermore, I modified the workflow so that it runs on Tuesdays. Let's see if a PR comes tomorrow.
Thanks again!
Best,
Robert


Support multiple different AssetLoaders with the same extension

Currently, the AssetServer stores its references to handlers as a dictionary of string to box. But the file extension to asset type relationship is not one-to-one -- with GLTF, the file can contain anything from Meshes and Materials to Textures to Skins (aka bones).
In a language with reflection, it would be possible to select handlers depending on the type. While Rust doesn't have true reflection, it does have TypeId which would allow us to match on both AssetLoader type and file extension. This might be a good solution that would also let us preserve the existing API. This could also solve the TODO in the asset server file that mentions that load() needs to be strongly typed.
Another solution that is slightly less ergonomic is to create a load_a method that takes a String specifying the type of the asset we want to load as well as the path.
If we can determine a good path to take, I may be able to implement it, since I am working on something that directly requires this support. (No extra credit for guessing that it's GLTF :D )

What about allowing multiple loaders, where additional loaders for the extension could use set_labeled_asset instead of set_default_asset?
impl AssetLoader for FontLoader {
    fn load<'a>(
        &'a self,
        bytes: &'a [u8],
        load_context: &'a mut LoadContext,
    ) -> BoxedFuture<'a, Result<()>> {
        Box::pin(async move {
            load_context.set_labeled_asset("mesh", LoadedAsset::new(...));
            Ok(())
        })
    }

    fn extensions(&self) -> &[&str] {
        &["ttf"]
    }
}

Although, then the user needs to know the label when loading
let asset = asset_server.load("fonts/FiraMono-Medium.ttf#mesh");


update kubectl wait command to match the correct label

I noticed the kubectl wait command in the docs didn't work based on the label name. PR fixes it so that it works—tested against a cluster:
$ kubectl wait Deployment -n netchecks -l app.kubernetes.io/instance=netchecks-operator --for condition=Available --timeout=90s
deployment.apps/netchecks-operator condition met
deployment.apps/netchecks-operator-policy-reporter condition met

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 89.627%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
216

Relevant Lines:
241

💛  - Coveralls


Download button not entirely visible

Now it is weirdly in the middle:

And, for some zoom levels, still cropped:

This is not happening in the production version. Could you take another look? (it is a minor).


181 further work fhir ts spring config

Changes
*Makes properties Spring configurable for reading results from the external terminology server.
*Refactors to allow using Spring properties read from the application module.
Related issue
Closes https://github.com/ehrbase/project_management/issues/209
Additional information and checks
Includes the implementation to read the response from the FHIR TS into DvCodedText

Hi @stefanspiska , thanks for the comments:-)
1-yes autowired was not in use there. It is deleted now.
2-I do not think that is a problem because lexical, syntactical and semantic analysis occur at different stages. So a client with AQL autocorrect can perform "online" lexical and  syntactic analysis (at client side), while the semantic analysis is later performed on the server side resolving the terminology invocation into the list of codes for the SQL statement. I think that will be more efficient also if we have several MATCHES clauses.
3-Agree, just waiting for the openEHR SEC consensus on this.


Add DNS server option

Hi, I have made a small bash script to set the DNS of a system, this is useful for someone who wants to use their pi-hole or just wants to use Google DNS (<IP_ADDRESS>) for example, heres a link to the repo https://github.com/CleanMachine1/Linux-DNS-Set
I will add it in a pull request if required!

I'm not sure I know what a DNS server is lol. time to research.
is your script tested?

I done some testing earlier, I need to test whether it works on RPi-OS though, but it definitely works on Linux Mint :rofl:

DNS - In simple terms -
Requested Website
DNS
Gives IP

OK, just grabbed my Portable Pi :) and run the script (found a bug with \n and fixed) and it definitely works!

so I'll add it in 1.5.0!
see https://github.com/Itai-Nelken/Pi-Assistant/projects/2

Could you put
echo "Made By CleanMachine1"
at the top of the script? ;)

Could you put
echo "Made By CleanMachine1"
at the top of the script? ;)

I'll credit you somehow, probably like that just in bold/different color.

I have put a echo command at the top of the script followed by 3 seconds to read it
That does but you want to do all the colors and things I wont complain 👀

I prefer to make the credit to not wait, but instead appear in a way that catches the eye.
here are some ideas:

figlet: (install with apt) figlet "This script was made by CleanMachine1"
figlet + lolcat (rainbow colors): (install with apt) figlet "This script was made by CleanMachine1" | lolcat the -a option can be added to lolcat so it prints the figlet from figlet (This script was made by CleanMachine1) line by line as well as in rainbow colors.
simple colored + bold text
simple colored + bold text + ascii graphic, for example:

echo '
        ___________________________________
______/This script was made by CleanMachine1\______
'

(I suck at ascii graphics as you see).

cowsay with the tux character: (install with apt) cowsay -f tux "This script was made by CleanMachine1"(looks really cool in my opinion, can be piped to lolcat for it to be colorful).
output from cowsay with tux character (not actually colored):

< This script was made by CleanMachine1 >
----------
   \
    \
        .--.
       |o_o |
       |:_/ |
      //   \ \
     (|     | )
    /'\_   _/`\
    \___)=(___/

after that a 0.3 second sleep should be enough for it to be eye catching in my opinion.

also I'm probably going to add a 'Community script' category including your script and the one from #21.

Wow, I was very aware of the cowsay, lolcat and figlet but that cowsay tux is very cool!
I'll add another commit to my repo moving the change, and although you may have done it, I want that tux!

I'd say ignore lolcat as its bloated as it needs to install ruby
ignore cowsay as well and just copy the output of cowsay and put it in echo commands

sounds good, but why ignore cowsay? because its a extra dependency?

Yes, why install more when it can be achieved easier

Hmm, actually I can't figure out a way to get the lines into a echo command as it takes all the backticks and other bash peices and takes them as commands

Cowsay is a <100Kb install

Add This into the install script
sudo apt install cowsay -y

I need some help..
as the script needs to be run as root, cowsay isn't under root due to it being a "game"

I am fully done
Done a few commits
You may want to remove the script if you have it installed as it has a lot of small bugs
I have added everything needed
I fixed the cowsay by just directing it to run the binary straight from /usr/bin/games

Hmm, actually I can't figure out a way to get the lines into a echo command as it takes all the backticks and other bash peices and takes them as commands

Use ' instead of "

Its fixed now, considering how light cowsay is, and how fun it is, I am sure the user can spare 100kb of data for a friendly tux, or block of cheese

< I am a block of cheese! >

_____   _________
/     _/         |
|                 ||
|                 ||
|    ###\  /###   | |
|     0  /  0    | |
/|                 | |
/ |        <        |\
| /|                 | | |
| |     _/   |  | |
| |                 | / /
/||                 /|||
----------------|
| |    | |
***    ***
/\  /\

Only complaint I have is that 3 seconds is to long for credits in my opinion, I would use 0.3 seconds because the big cowsay tux catches the eye, so no need to let it stsy therebgor so long.
An alternative would be to find abway to keep the credits always at the top while the script runs.
Once that is done it will be added once I come round to implement all the stuff I want and release v1.5.0.

1 second and it will be 1 second!

Done and done!
The raw github link might take up to 5 mins to refresh

Ok, this will probably be my last commit
I am going to change the scripts name from main.sh to DNSSet.sh otherwise it may conflict with other files

Been a quarter of an earth year, ever plan to add this, if anyone is still using this

TBH I abandoned this project. I think I might archive this repo.

:(

I'm closing this issue as I have moved on to other things and don't have the time or interest to keep working on Pi-Assistant.


Fix TS must contain a metric errors, and cleanup error aggregation

Fixes https://github.com/GoogleCloudPlatform/opentelemetry-operations-go/issues/490
The reason for the errors was that we were re-setting the capacity of the tss list for each point.  This resulted in empty TimeSeries structs being added to the list of timeseries, and produced errors on export.

The error handling changes aren't part of the bug-fix.  It probably makes more sense to put those in another PR.

Moved error handling changes to https://github.com/GoogleCloudPlatform/opentelemetry-operations-go/pull/492


Fixes to compile once WFCORE-759 has been merged

https://github.com/wildfly/wildfly-core/pull/813 will fail until this is merged

Windows Build 1782 is now running using a merge of f4f33781340f2486d767849451448702f0babb63

Linux with security manager Build 1609 is now running using a merge of f4f33781340f2486d767849451448702f0babb63

Linux Build 6708 is now running using a merge of f4f33781340f2486d767849451448702f0babb63

Linux with security manager Build 1609 outcome was SUCCESS using a merge of f4f33781340f2486d767849451448702f0babb63
Summary: Tests passed: 833, ignored: 322 Build time: 0:06:21

Windows Build 1782 outcome was SUCCESS using a merge of f4f33781340f2486d767849451448702f0babb63
Summary: Tests passed: 2878, ignored: 377 Build time: 0:47:10

Linux Build 6708 outcome was FAILURE using a merge of f4f33781340f2486d767849451448702f0babb63
Summary: Tests failed: 30 (30 new), passed: 2788, ignored: 377 Build time: 1:07:24
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.clustering.cluster.ejb.remote.RemoteFailoverTestCase(SYNC-tcp).testStatefulFailover: <no details avaliable>

org.jboss.as.test.manualmode.ws.WSAttributesChangesTestCase.testWsdlHostChanges: <no details avaliable>

org.jboss.as.test.manualmode.ws.WSAttributesChangesTestCase.testWsdlPathRewriteRuleChanges: <no details avaliable>

org.jboss.as.test.manualmode.ws.WSAttributesChangesTestCase.testWsdlPortChanges: <no details avaliable>

org.jboss.as.test.manualmode.ws.WSAttributesChangesTestCase.testWsdlUriSchemeChanges: <no details avaliable>

org.wildfly.test.integration.security.picketlink.federation.SAMLMetadataTestCase.testFederationWithGlobalLogout: <no details avaliable>

org.wildfly.test.integration.security.picketlink.federation.SAMLMetadataTestCase.testFederationWithLocalLogout: <no details avaliable>

org.wildfly.test.integration.security.picketlink.federation.SAMLPostBindingTestCase.testFederationWithGlobalLogout: <no details avaliable>

org.wildfly.test.integration.security.picketlink.federation.SAMLPostBindingTestCase.testFederationWithLocalLogout: <no details avaliable>

org.wildfly.test.integration.security.picketlink.federation.SAMLRedirectBindingWithSignaturesTestCase.testFederationWithGlobalLogout: <no details avaliable>

org.wildfly.test.integration.security.picketlink.federation.SAMLRedirectBindingWithSignaturesTestCase.testFederationWithLocalLogout: <no details avaliable>

##### there are 20 more failed tests, see build details


Suggest an props like autoFillEmail

I want to auto-fill the email field with given email address.
But current version doesn't have this option.
Though my suggestion is silly, I think it will be good in the future.
Thank you

Isn't this yet possible?  I'm also interested in filling this field with string taken from other component :(.


[SMALL PATCH] Sql sensor

Description:
Just reopening #12142 to address small fix

@dgomes make sure to use new branches for your PRs.


Building from source

Just checked out your v3. Because I can't get 2.9.1 working at all. Never build a Java app before. But am hitting this error:
C:\Users\User\AppData\Local\Temp\js-graphql-intellij\js-graphql-intellij-plugin>gradlew buildPlugin
Starting a Gradle Daemon, 1 incompatible and 1 stopped Daemons could not be reused, use --status for details
Configuration on demand is an incubating feature.

> Task :generateGraphQLEndpointDocParser FAILED
java.lang.UnsupportedClassVersionError: org/intellij/grammar/Main has been compiled by a more recent version of the Java Runtime (class file version 55.0), this version of the Java Runtime only recognizes class file versions up to 52.0

What's a quick walkthrough to building the .jar files that I can then copy into WebStorm?
Many thanks,
Also,  If you could release v3 that would be awesome 👍

@binary64 hi! Sorry for the late answer. It seems that this is caused by some incompatibility of the JDKs used. You could check it in the project structure window. I use Jetbrains runtime 11.0.11, Amazon Corretto should be fine too. You can download and install any JDK from the same dropdown if it's missing using "+ Add SDK". There were also some problems with the build process, which I just fixed.
Besides I'm preparing a release right now so I hope it will resolve your issues.


Client: special streaming camera

Investigate the integration of an obs/other streaming software.
Consider a 3rd-person streaming camera that will follow the player, but smooth out sudden moves and can be set to a custom resolution for more efficient streaming.
Thoughts:

different LOD settings for the streaming camera
settings for the camera position controls ( add a dummy 'camera man' logic to better capture action? )

Found this if you want to go the OBS Plugin route. Then a user could just install the client as usual, and then plugin, and use it as a filter for a capture source in OBS. This is a little old, but I think it lays out the process. May help with adding what is needed in the code?
https://obsproject.com/forum/threads/how-to-create-obs-studio-plugins.53470/


Query error in MySQL when order() and union() is used together

Hello.
I have a proble when I use union().
Example:
var sequel = require('squel').useFlavour('mysql');
var q = squel
.select().from('table1')
.order('date')
.limit(10)
.union(
    squel.select().from('table2')
);
console.log(q.toString())

My intention:
(SELECT * FROM table1 ORDER BY date ASC LIMIT 10) UNION (SELECT * FROM table2)

The result query is
SELECT * FROM table1 ORDER BY date ASC LIMIT 10 UNION (SELECT * FROM table2)

But this query is not valid in MySQL
Kernel error: Error( 1221 ) HY000: "Incorrect usage of UNION and ORDER BY"

Could you fix this issue? or am I missing something?

@peecky have you fond the solution for this?

@ravenjohn I found a solution, wrapping the first select query with a simple select query
    var q = squel
    .select()
    .from(
        squel.select().from('table1')
        .order('date')
        .limit(10),
    't1')
    .union(
        squel.select().from('table2')
    );

The result:
SELECT * FROM (SELECT * FROM table1 ORDER BY date ASC LIMIT 10) `t1` UNION (SELECT * FROM table2)

Awesome! Thanks a lot!


Security update

Codecov Report

Merging #184 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #184   +/-   ##
=======================================
  Coverage   90.91%   90.91%
=======================================
  Files          78       78
  Lines        2916     2916
  Branches      615      536   -79
=======================================
  Hits         2651     2651
  Misses        263      263
  Partials        2        2

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 535ba36...d1d1fd4. Read the comment docs.


Can we have working example with assets in the documentation for the simple_save?

System information

TensorFlow version: 1.12
Doc Link: https://www.tensorflow.org/api_docs/python/tf/saved_model/simple_save

Simple_save seems to be a newer and more standardized saving approach. Since it can also use assets to store vocabularies, I wonder if it is possible to have a working example in document based on keras sentiment analysis system's data? So there is place to look up how to really integrate the entire model (including vocabulary, word embedding, etc) in a single simple_save.

@allenlavoie Allen, what is the current best way to save a model and do we have a documentation page for it?
simple_save seems to be deprecated.

simple_save won't appear in v2. We do have a SavedModel API: https://www.tensorflow.org/alpha/guide/saved_model
I don't have plans to write a sentiment analysis example, but it does sound useful.

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


allow dump model to update given hive table partition

to: @jq @yolken

👍   please also update the generic pipeline hackpad

sure, will update the document


Paper should have and belong to many authors

Scenario
Given a paper
Then it should have an empty list of authors
Hints
A paper can have many authors and an author can write many papers.
This relationship is indicated in Rails by annotating both models (i.e. the Author and Paper) with a has_and_belongs_to_many association.
To store this n:m relation, create a new database table in a migration: guide.
There is a generator which can produce join tables if 'JoinTable' is part of the name:
rails generate migration CreateJoinTableAuthorPaper author paper
To test if such a relation exists, you can test whether a paper has an empty list of authors.
Reminder: Migrations are stored in db/migrate and are applied with the rails db:migrate command.
Error

Got NoMethodError: undefined method `authors' for #Paper:0x0000564085a622c0

37 exercise tests have passed. There are 44 in total. You will solve multiple at once towards the end.

Sorry, it's not working for me yet:
Scenario
Given a paper
Then it should have an empty list of authors
Error

Got NoMethodError: undefined method `authors' for #Paper:0x000055d60f3ca128

If you have problems solving this task, please don't hesitate to contact the teaching team!


Attach README.md and CHANGELOG.md as artifacts

In this PR I'm attaching the README.md and CHANGELOG.md files as artifacts. This broken when I modified the artifact publishing logic because Maven doesn't know anything about these files. If this doesn't work then I'll look at reverting my commit.

/azp run java - appconfiguration

/azp run java - appconfiguration

/azp run java - appconfiguration

/azp run java - appconfiguration

/cc @JimSuplizio @scbedd can I get a review/approval?


Add zsh completion

Is your feature request related to a problem? Please describe.
Zsh is my default working shell environment, and it would be nice to have a zsh completion like bash. I often use snakemake to create and execute complex pipelines of tasks, and I would like to have a better user experience with zsh.
Describe the solution you'd like
I want to provide a similar completion to bash, which can complete the snakemake options, rules, and files in a smart and efficient way. I think this would benefit many users who use both snakemake and zsh, and improve their productivity and satisfaction.
Describe alternatives you've considered
I have considered using the bash completion scripts with bashcompinit, but it does not work at all. The code I tried is below:
autoload -Uz compinit && compinit
autoload -U +X bashcompinit && bashcompinit

complete -C snakemake-bash-completion snakemake%

I was thinking about that yesterday as well, and found that snakemake's help output is structured enough that you can just use:
compdef _gnu_generic snakemake

as described at https://github.com/zsh-users/zsh-completions/blob/master/zsh-completions-howto.org#completing-generic-gnu-commands .

Maybe that could be put somewhere into the documentation, because it really helps you navigate the amazing number of command line args that snakemake offers.

Thank you @coroa! I will give it a try.
Also, I found an amazing repository that solves my problem now - zsh-completions-bio. It is very comprehensive and contains nearly all the zsh completions I need for my bioinformatics work, I would highly recommend it for every bioinformatician who using zsh.


where is the images provided for hw2?

RT

02-feature-inversion/images/face.jpg

that is just one picture(;´༎ຶД༎ຶ`)


Extension fails to load when built with hiredis 0.14 on armv7h

All previous versions worked fine. Arch recently upgraded hiredis to 0.14.0-1. I built it for armv6 against php 7.4.3 and hiredis 0.14.0-1 and I can load and run the extension
phpinfo()
PHP Version => 7.4.2

System => Linux runeaudio 4.19.102-1-ARCH #1 SMP PREEMPT Thu Feb 13 22:13:34 CST 2020 armv6l
Build Date => Jan 21 2020 19:19:29
...
phpiredis

phpiredis => enabled
phpiredis version => 1.0.0
hiredis version => 0.14.0

When I build it for armv7h, I get an error:
PHP Warning:  PHP Startup: Unable to load dynamic library 'phpiredis' (tried: /usr/lib/php/modules/phpiredis (/usr/lib/php/modules/phpiredis: cannot open shared object file: No such file or directory), /usr/lib/php/modules/phpiredis.so (/usr/lib/php/modules/phpiredis.so: undefined symbol: redisReplyReaderGetError)) in Unknown on line 0

Warning: PHP Startup: Unable to load dynamic library 'phpiredis' (tried: /usr/lib/php/modules/phpiredis (/usr/lib/php/modules/phpiredis: cannot open shared object file: No such file or directory), /usr/lib/php/modules/phpiredis.so (/usr/lib/php/modules/phpiredis.so: undefined symbol: redisReplyReaderGetError)) in Unknown on line 0
phpinfo()
PHP Version => 7.4.2

System => Linux livingrune 4.19.102-1-ARCH #1 SMP PREEMPT Sun Feb 9 19:07:08 UTC 2020 armv7l
Build Date => Jan 21 2020 19:19:33

If I revert the hiredis version to 0.13.3-2 (on both the build machine and the running machine) and re-build the extension, I get no error
Both the build machine and the machine I am using it on are 'up to date' with packages. I did a pacman -Syu on both machines and re-built the extension
phpinfo()
PHP Version => 7.4.2

System => Linux livingrune 4.19.102-1-ARCH #1 SMP PREEMPT Sun Feb 9 19:07:08 UTC 2020 armv7l
Build Date => Jan 21 2020 19:19:33
...
phpiredis

phpiredis => enabled
phpiredis version => 1.0.0
hiredis version => 0.13.3

Both the armv6h and armv7h kernel and hiredis extension are built using the same PKGBUILD file.

This package also builds but doe snot work on x86_64. It requires the same work-around: revert hiredis to 0.13.3. When built against this, all works. Curious that it works on armv6h, but not on armv7h or x86_64

Ignore this issue. It is not relevant. Was a different version of hiredis between the builds when I reported it. Has been resolved on the Arch Linux side.

Changes to support Hiredis >= 0.14, >= 1.0 have been merged in the v1.1 branch, the new minor release will be shipped in a few days is everything is fine. Thank you.


Support mattn/sudo to support Windows

Thanks to @mattn
https://vim-jp.slack.com/archives/C03C4RC9F/p1558588750034500

I confirmed that it works.

@mattn, @hokorobi Thanks a lot :tada:


Gazebo Simulation Environment

Hi, I am working on this project and I am trying to change the simulation environment with different one. Unfortunately, I could not change the simulation environment. I created a new world with ".world" extension file but it dis not work. In addition of that I tried to remove current objects but the ground did not remove. Can you help me about where part of the project should I change?

the .world file is in
px4_fast_planner/worlds/
and now the world is written in
px4_fast_planner/launch/mavros.launch line 34 world arg
so you can edit the file 'px4_fast_planner/worlds/outdoor_village.world'
or add new world file and edit the line 34 of 'px4_fast_planner/launch/mavros.launch'

Firstly, thank you for your reply but it was not the case that I faced up. I tried to change the world but the drone did not flt. Then, I edited the file with gedit and it works for me. Thanks!


[Question] Memory footprint and is the project still maintained

Hello,
I have been long time Spotify ANNoy user, and I've recently come across HSNW. I'm a Java guy so I obiously took a look at the full java implementation, and then this project too.
I'm surprised by the performances, but even more, by the memory footprint used by this library and I'm wondering if someone could validate the number I'm seeing.
I've loaded about 2.5 million tensors with 1024 dimensions and from what I can see on JvisualVM, the memory consumption of a Spring Java API in idle (just index loaded), is about 500mb. Is that possible? (see picture below)

Where are tensors data stored? On disk?
I also have another question, is this project still maintained?
Thanks,
Gio

Hi Gio, hope you're well.
Back then when we wrote the binding we were mostly aiming to have the lower query time hnswlib provided. I personally don't remember figures we could use as comparison but I imagine we would have a similar memory footprint using the native library. If not, we could optimize that.
I haven't been working with hnswlib for a while but from what I remember the references are kept in memory. You have the option to write the state of your index in disk if you want (and restore it).
Have you tried using the java implementation? how were your figures?
I don't think this project is being actively maintained anymore but it should be working fine still. Hnswlib released new updates and improvements which weren't tested with this binding. If you want, I could have a look into that.
Wishing you a great weekend.
Best regards,
Hussama

Thanks for the very quick reply,
I have indeed tested the java client, but using double rather than float, and I could not fully build the index with 10GB of heap.
I'm in the middle of rebuilding the full 2.5milion w/ float, but the current projections is 250k seems to use 1.2 gig, so it should all fit in 12GB.
This JNA implementation does everything w/o, apparently, ever exceeding 1GB. I guess the C native code allocate and deallocate memory as it goes. What I don't understand is, when I dump the index on disk it is about 11GB, but in memory is less than 1GB... where are the other 11GB saved?
We need to replace Spotify ANNoy, and after some very quick tests, can appreciate the power of HNSW, in terms of performances and accuracies (results a much better).
I need to pick up the right HNSW library/framework to replace ANNoy and I was wondering if this project is still maitained because seems to be the best java solution.
If you were so kind to test it with the latest hnswlib it would be amazing, and I would be delighted to pair-program/review the code, so that I could start learning and contributing.
I would be using this library in argusnft.com, and AI Powered NFT Fake Detection Platform and our goal is to extract embeddings for all the (picture-based) nft from all the blockchains, and load them in ANN indexes. A pretty huge objective. If you have recommentions or fancy a chat let me know!
You too enjoy the weekend and thanks for the superquick reply.
☮

We also moved from annoy back then because updating the index in runtime was not possible and some other problems.
Hmmmmmmmm, yeah, that sounds suspicious indeed 😛 but I believe if we had an issue we would have spotted already in production. You're right, the memory allocation/free is taking place as it runs (can't guarantee there are no memory leaks) and when dumping the index it is most likely generating and writing down the entire state space and parameters.
Talking about the Java one, I remember it took a while to build the indexes with large datasets (even in parallel) and the query time was so performant either (in comparison to the native) which made us try out other solutions. Building the index natively (in python) and restoring in few secs on Java was also a big plus for us.
That sounds a cool project. I'm not a data science / ml hero; our wizard was @alexcarterkarsus. Alex, would you have any recommendation? Would hnswlib be your go to library for Gio's use case?
--
I had a quick look and since I left STST, I'm afraid I can't push updates to the library. I will fork, upgrade hnswlib and open a pull request once ready.
Which platform would you be using the library on? WIn64, AMD64? ARM64?
Have a nice weekend! Take care!

Thanks again for the detailed response, it's really helping.
Nice to meet you @alexcarterkarsus !
I would have another couple of questions:

What was the type of document you guys were indexing? And the tensor dimension?
Is the index file dump interoperable among hnsw libraries? java/jna/c/python?
Where were you storing the tensors for long time storage? Relational db? S3? Asking because we currently have 2.5m tensor now, and possibly close to 50m in just a couple of months and we were wondering if you guys had any recommendations.

So many questions! But very exciting space and this project is providing me with so much knowledge! Thank you!

1: leave it to alex 🧠 😛
2: yeah; they use the same code underneath so it is indeed interoperable across languages 🙌. I'm not so sure if is across architectures, I'm afraid it isn't.
3: our use case, no, the model had periodically recreated due to the constant new data (it was prepared separately and stored somewhere in the cloud indeed). For yours, I can see it would be more static (keeping track of existing nfts and adding new), right?
Yeah, there are some challenges to keep in mind, the bigger the index, higher is the insertion and query time and maybe the fine tuning of the hnswlib parameters can help. 7M was one our acceptance criteria and it was still working fine for us.
Not sure if you could organize and split it in different models?

hey all,
sorry for the late reply!
regarding question 1: we built our vectors from a word2vec model, and the
dimension is either 50 or 100. The biggest index has around 10M vectors,
and it is still quite performant.
cheers
Alex
On Sat, Sep 17, 2022 at 11:08 AM Hussama Ismail @.***>
wrote:

1: leave it to alex 🧠 😛
2: yeah; they use the same code underneath so it is indeed interoperable
across languages 🙌. I'm not so sure if is across architectures, I'm
afraid it isn't.
3: our use case, no, the model had periodically recreated due to the
constant new data (it was prepared separately and stored somewhere in the
cloud indeed). For yours, I can see it would be more static (keeping track
of existing nfts and adding new), right?
Yeah, there are some challenges to keep in mind, the bigger the index,
higher is the insertion and query time and maybe the fine tuning of the
hnswlib parameters can help. 7M was one our acceptance criteria and it was
still working fine for us.
Not sure if you could organize and split it in different models?
—
Reply to this email directly, view it on GitHub
https://github.com/stepstone-tech/hnswlib-jna/issues/12#issuecomment-1250034197,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKRX3UFEEJEWYGOURX66LHDV6WC7PANCNFSM6AAAAAAQOSCPE4
.
You are receiving this because you were mentioned.Message ID:
@.***>


Integrate cryptoconditions version 02

travis will fail:

[x] Hook up travis with CC 02 branch

Codecov Report

Merging #1562 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #1562      +/-   ##
=========================================
- Coverage    98.4%   98.4%   -0.01%
=========================================
  Files          58      58
  Lines        2821    2820       -1
=========================================
- Hits         2776    2775       -1
  Misses         45      45

@sbellem Can you confirm that this pull request takes care of issue #1516

@sbellem Can you confirm that this pull request takes care of issue #1516

It does not yet. Thanks for the reminder!

Lets do #1516 in a different pr. We can do that during QA week.


Fix maybe-uninitialised sequence number

Hi Peter, yes I'm scratching my head over it as well.  Asked Fraser as well, but seemed best to ask your advice again.  Seems not a trivial bug that I can fix out of the box.


TypeScript examples

rubico is compatible with TypeScript, right now there's not enough examples. All examples should go in the examples directory. All examples should be runnable, with output that makes sense.
Here is one I did: squaredOdds.ts
I plan to document the examples in the near future.
Of course, more JavaScript examples are always welcome.
Also, please include any relevant links (if any) in the example source.

hey @richytong, i wanna help on ts examples.I checked your squaredOdds example and CONTRIBUTING.md file , how should i start ?Can you assign me a basic example to start with ?
Thanks

@toltarisa Welcome! Actually I'm going to revamp the example system so it's more useful. Here's the new issue for examples (/x/)

I've decided against TypeScript examples because I fundamentally do not support that direction. If you're still down to help with JavaScript examples, the issue is open

this is why https://youtu.be/jmPZztKIFf4?t=108

You know what, I don't know why I was so mean. Also the examples are very necessary. I would love help on this, I just need to set up some tooling.


Can one  job of a  project  be update?

I want to update one job of  a project.  but I have to upload all of them。
Is there some way to update one job?

As i know, no or you have to update that job file directly to db

I want update one job of project.  Some times,  add a job to a project  or  del a job from a project .  but I have to upload all of them


Switch to httpclient gem for handling http requests

Because it has better support for configuring an HTTP proxy by setting the environment variable HTTP_PROXY.

Nudge

Closing because it is unlikely this will be merged


Add possibility of adding condition when CUSTOM auth policy has been used

Describe the feature request
When using a CUSTOM authorization policy in a setup with a default empty AuthorizationPolicy (i.e. deny all by default), you must also crease an identical ALLOW policy. If there are any conditions, like methods or hosts, they must be duplicated.
This introduces a possible security issue if the CUSTOM policy is changed to be narrower than the ALLOW policy.
If an CUSTOM policy could leave an "audit trail" on the request, an ALLOW policy could use this instead of attempting to duplicate the policy conditions of the CUSTOM policy. For example, if it is possible to set conditions in an http filter, we could introduce request.ext_authz as an array of all ext authz modules that have been applied.
Describe alternatives you've considered

a CUSTOM_ALLOW policy or additional attributes to customize the behavior, like bypassing other authorization policies when set. I think this would complicate the flow further and be less flexible.
Setting an HTTP header in the external provider, and checking it in the next authoritzation policy. This is not Out Of Band, and is more vulnerable to security issues.

Affected product area (please put an X in all that apply)
[ ] Ambient
[ ] Docs
[ ] Dual Stack
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[X] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Additional context
Example of current situation:
---
apiVersion: security.istio.io/v1
kind: AuthorizationPolicy
metadata:
  name: "productpage-viewer"
  namespace: default
spec:
  selector:
    matchLabels:
      app: productpage
  action: CUSTOM
  provider:
    name: authz-module-1
  rules:
  - to:
    - operation:
        hosts: ["ext-hostname.example.com"]
---
apiVersion: security.istio.io/v1
kind: AuthorizationPolicy
metadata:
  name: "productpage-viewer"
  namespace: default
spec:
  selector:
    matchLabels:
      app: productpage
  action: ALLOW
  rules:
  - to:
    - operation:
        hosts: ["ext-hostname.example.com"]

Proposed solution:
---
apiVersion: security.istio.io/v1
kind: AuthorizationPolicy
metadata:
  name: "productpage-viewer"
  namespace: default
spec:
  selector:
    matchLabels:
      app: productpage
  action: CUSTOM
  provider:
    name: authz-module-1
  rules:
  - to:
    - operation:
        hosts: ["ext-hostname.example.com"]
---
apiVersion: security.istio.io/v1beta1
kind: AuthorizationPolicy
metadata:
  name: "productpage-viewer"
  namespace: default
spec:
  action: ALLOW
  rules:
  - when:
    - key: request.ext_authz
      values: ["authz-module-1"] # Only true when allowed by the CUSTOM provider authz-module-1

CC @aryan16

@aryan16 did you have any thoughts here? I've seen multiple internal developers confused with the current "right way to write this policy" and having to sync the policies.


Adding Glossary to navbar

Also, I don't think it's visually obvious that the footnote links (like "[26]") are links. Can you make it clearer? In my head I'm imagining the code would look something like [26] and visually link part would be "portal link"-colored and the brackets are text-colored.

Flowbite is for the modal functionality. Yes, I can make the link a portal link color.

We already use modals elsewhere in the application. Is there a reason you didn't use the same system here?

Yes, because its just for simple text, responsive design already built in, uses tailwind, already structured, quick and easy to use and not complicated. We've had to make many changes to our modals.


Jcenter is closed

Hi,
Jcenter is closed.
Can you publish it in other place?
implementation "com.cioccarellia:kite:1.2.0"

Could not find com.cioccarellia:kite:1.2.0.

Thanks.

Today im using temporary:
implementation "com.github.cioccarellia:kite:master-SNAPSHOT"

Kite is actually hosted on Maven Central since 4 months, it somehow has not occurred to me to update my publishing id and instead kept the old one. Fixing now

Nope, readme checks out, you somehow messed up the dependency.
What's shown in the readme works.
dependencies {
-    implementation "com.cioccarellia:kite:1.2.0"
+    implementation "com.github.cioccarellia:kite:1.2.0"
}

I just flashed for a moment here because if you were actually right that meant I had spend 5 hours swapping off kite from jcenter to maven and not cared spending 5 seconds adding a .github coordinate suffix to the readme, which sounded extra stupid even for me.

Hi,
No problem man.
Thanks.


Update: Core versions, v1.1 migration guide, adapter guide

There are two kinds of "breaking" changes that are allowed in dbt minor versions:

Python interface for adapter plugin maintainers. This is said implicitly in a couple places, but I think it's worth calling out explicitly in both "About dbt Core versions" and "Building a new adapter" (our dedicated guide). The language here is an imperfect placeholder — @dataders I'd appreciate your help in how we might set/communicate this expectation.
Metadata interfaces, which are versioned independently, and will always bump their versions to reflect an underlying change. This is a good thing—no surprises!—but it does require some coordination during upgrades if using external tools that parse metdata, or an advanced metadata-powered feature within dbt such as --state-powered node selection. A user just ran into this, and perceived it as a breaking change: https://github.com/dbt-labs/dbt-core/issues/5213

@runleonarun Thanks for the review, and the recommendations! I just pushed up a new commit with those changes.
@dataders I see you requested yourself for review (thank you!), so I'll wait to merge until you've had a chance to look.

For the future, I'm wondering if there's a logical way to divide this content into smaller sections. I know we already took out the Testing your adapter, but this is still a lot of content on one page.

Completely agree. I'm very excited to see what @dataders + @dbeatty10 do with the place ;) (and truly very grateful to you both for taking over codeownership in this area)

Some of the content in this PR has been overtaken by events — specifically the addition of "backwards compatible" manifest deserialization, which makes the note about state:modified in the v1.1 guide less relevant.
@dataders I think the content about upgrade expectations for adapter maintainers is still relevant. Let me know if you think it has a place on those pages.

This one had been waiting on me. I just updated and merged. Thanks for the review and revision back in July, Anders!


chore: fix issues prevent compilation with later JDKs

This just fixes an annoying issue that prevents JDK 11+ from working with our build and adds some associated missing javadoc. This is annoying if you happen to be working with other projects that require a later version of the JDK.
Specifically, there was a changes in later versions of javac that applied more rules, mostly around javadoc. These caused errors we could not customize, so I've disabled the lint checks in the maven-javadoc-plugin and I instead enforce them with checkstyle, which is more customizable.

Codecov Report

Merging #178 (120b127) into main (7f06b9a) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 120b127 differs from pull request most recent head 5ef9d03. Consider uploading reports for the commit 5ef9d03 to get more accurate results

@@            Coverage Diff            @@
##               main     #178   +/-   ##
=========================================
  Coverage     91.60%   91.60%
  Complexity      187      187
=========================================
  Files            20       20
  Lines           417      417
  Branches         22       22
=========================================
  Hits            382      382
  Misses           23       23
  Partials         12       12

Flag
Coverage Δ

unittests
91.60% <ø> (ø)

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

Impacted Files
Coverage Δ

src/main/java/dev/openfeature/sdk/BooleanHook.java
0.00% <ø> (ø)

src/main/java/dev/openfeature/sdk/DoubleHook.java
0.00% <ø> (ø)

src/main/java/dev/openfeature/sdk/ErrorCode.java
100.00% <ø> (ø)

...ava/dev/openfeature/sdk/FlagEvaluationDetails.java
100.00% <ø> (ø)

...c/main/java/dev/openfeature/sdk/FlagValueType.java
100.00% <ø> (ø)

src/main/java/dev/openfeature/sdk/HookSupport.java
95.12% <ø> (ø)

src/main/java/dev/openfeature/sdk/IntegerHook.java
0.00% <ø> (ø)

...ain/java/dev/openfeature/sdk/MutableStructure.java
92.85% <ø> (ø)

...in/java/dev/openfeature/sdk/OpenFeatureClient.java
97.91% <ø> (ø)

src/main/java/dev/openfeature/sdk/Reason.java
100.00% <ø> (ø)

... and 3 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Parsing error: Unterminated regular expression literal.

Hey,
first of all thanks for building this. I really appreciate the work you put into this project.
I'm running into a problem with linting MDX files alongside react and typescript. I have a working setup for all of the react parts, but whatever i try, i am not able to get .mdx linting to work.
The linter shows 'Parsing error: Unterminated regular expression literal.' on jsx in MDX files.

Lorem

-----------^----

I made a small test case here:
https://github.com/tizzle/mdx-lint-bug
Do you have any idea what could be going on and can point me towards a solution? That would be great.
Best

https://github.com/tizzle/mdx-lint-bug/blob/master/.eslintrc.json#L9
It seems unnecessary.

If I remove it, it leads to the following error:
Parsing error: "parserOptions.project" has been set for @typescript-eslint/parser.
The file does not match your project config: /Users/tillhinrichs/Downloads/mdx-lint-bug/src/test.mdx.
The extension for the file (.mdx) is non-standard. You should add "parserOptions.extraFileExtensions" to your config.

It seems an incorrect usage to enable parserOptions.project with @typescript-eslint/parser for all files, you should use overrides for .tsx? files since v2.2.0.
 Thank you so much for the quick response and prompt resolution.
Cheers


Style layers in layerlist

A user should be able to control the style of a layer's corresponding geometry within the layerlist.
I'd prefer that they be able to access this control via a menu brought up from a right-click on a layer in the layerlist.  The color of the layer would represented in the layerlist (possibly as a 5px border left of the text). The color could also represent whether the state was visible or not (turn to grayscale or something...).

Updating this to pertain to the entire style of the layer.  This could include putting the icon representing a point feature into the layerlist.
Additionally, representing visibility of the layer seems unnecessary, as the checkbox does that for us.


Can't find utils.default_paths and model_fitting.initialize_fitting. These cannot be installed through pip.”

Hello! Thank you for your outstanding work! During the code processing stage, when I run the Z_score.ipynb file, I cannot download utils.default_paths and model_fitting.initialize_fitting through pip install. After reading the text, I think this should be the code you wrote because it contains some preset information. At the same time, I cannot find this kind of package on the Internet. Is the code incomplete? Or can I download them in some way?

Hi,
You can find the code here: https://github.com/mmhenderson/modfit/blob/master/code/utils/default_paths.py
Apologies, this code release was a copy of my research code base. I intend to clean up the code soon.


⚠ syncer: clean opaqued keys before construct super cluster object

Signed-off-by: zhuangqh<EMAIL_ADDRESS>

What this PR does / why we need it:
prevent tenant from using super cluster annotation/labels
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 #

Most of the code changes are refactoring changes. The added logic makes sense for preventing tenant from abusing certain keys maintained by the super cluster.
/lgtm
/approve


Remove unnecessary save() calls

As per https://vladmihalcea.com/best-spring-data-jparepository/.

Won't do. There was only one save on a managed entity (in com.romanboehm.wichtelnng.usecases.registerparticipant.RegisterParticipantService#register), but that one is actually necessary to persist the event's participants.


Rename the repo topic

Can we rename the topic of the repo to napari not napari-plugin or maybe we can have both?

Done!

also this is the first time I hear of topics. :joy: As I was reading this sleep


Error Deploying to Heroku

My Application running wel in local, but it return Error from LaravelData\Support\DataConfig

Did you update your composer dependencies on Heroku?

Further to what Wulfheart wrote
Please make sure:

config/data.php was created
clear the cache

This looks like the correct solution, I'm gonna close this.
Errors like

Unresolvable dependency resolving [Parameter #0 [ «required> array $config il in class
Spatie\LaravelData\Support\DataConfig

Always indicate the SettingsServiceProvider didn't run.


Added helm reference

Signed-off-by: Susanne Tünker<EMAIL_ADDRESS>

@rachfop I believe we can also remove the "profile" folder, no?


New version: vim.vim.nightly version 9.1.0418

Pull request has been created with WinGet Releaser v2 :rocket:
Microsoft Reviewers: Open in CodeFlow

Hi @chrisbra,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 881.)

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-05-17 16:51:29.191 [FAIL] Installer failed security check. Url: https://github.com/vim/vim-win32-installer/releases/download/v9.1.0418/gvim_9.1.0418_x86.exe Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
(Automated response - build 881.)

@wingetbot run


Error connecting to Netapp cluster

Describe the bug
K8S Deployment connecting to NetApp getting the following:
9:39AM INF command-line-arguments/poller.go:153 > log level used: info Poller=netappdev
9:39AM INF command-line-arguments/poller.go:154 > options config: /opt/harvest.yml Poller=netappdev
9:39AM INF command-line-arguments/poller.go:179 > started in foreground [pid=1] Poller=netappdev
9:39AM WRN command-line-arguments/poller.go:532 > init collector-object (Zapi:Node): connection error => api request rejected => Unable to find API: cluster-identity-get Poller=netappdev
9:39AM WRN command-line-arguments/poller.go:534 > aborting collector (Zapi) Poller=netappdev
9:39AM WRN command-line-arguments/poller.go:532 > init collector-object (ZapiPerf:SystemNode): connection error => api request rejected => Unable to find API: cluster-identity-get Poller=netappdev
9:39AM WRN command-line-arguments/poller.go:534 > aborting collector (ZapiPerf) Poller=netappdev
9:39AM WRN command-line-arguments/poller.go:257 > no collectors initialized, stopping Poller=netappdev
9:39AM INF command-line-arguments/poller.go:421 > cleaning up and stopping [pid=1] Poller=netappdev
Environment

Harvest version: harvest version 2.0-nightly (commit ) (build date 2021-07-22T15:42:55+0000) linux/amd64
Command line arguments used: In deployment
OS: K8S v1.19.12 , Docker version 18.06.1-ce, build e68fc7a
Install method: K8S deployment
ONTAP Version: 9.8-p4
Other:

To Reproduce
Steps to reproduce the behavior:
Expected behavior
It should connect to my Netapp storage
Actual behavior
got errors:
init collector-object (Zapi:Node): connection error => api request rejected => Unable to find API: cluster-identity-get Poller=netappdev
Possible solution, workaround, fix
No workaround or fix that i know
Additional context
deployment.yaml:
```
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: harvest
  namespace: prometheus-exporter
  labels:
    app: harvest
spec:
  replicas: 1
  selector:
    matchLabels:
      app: harvest
  template:
    metadata:
      labels:
        app: harvest
    spec:
      volumes:
        - name: zapi-default-config
          configMap:
            name: zapi-default-config

        - name: harvest-poller-config
          secret:
            secretName: harvest-poller-config

      containers:
      - name: harvest
        image: artifact.local/rahulguptajss/harvest:21.05.4-2
        args:
        - "--config"
        - "/opt/harvest.yml"
        - "--poller"
        - "netappdev"
        resources:
          limits:
            memory: 2G
            cpu: 1000m
          requests:
            memory: 1G
            cpu: 500m
        ports:
        - name: http
          containerPort: 12990
        volumeMounts:
          - name: zapi-default-config
            mountPath: /opt/harvest/conf/zapi/default.yaml
            subPath: default.yaml
          - name: harvest-poller-config
            mountPath: /opt/harvest.yml
            subPath: harvest.yml
---
apiVersion: v1
kind: Service
metadata:
  labels:
    app: harvest
  name: harvest
  namespace: prometheus-exporter
spec:
  ports:
  - port: 12990
    targetPort: 12990
    name: metrics
  selector:
    app: harvest
  type: ClusterIP
---
kind: ServiceMonitor
apiVersion: monitoring.coreos.com/v1
metadata:
  name: harvest
  namespace: prometheus-exporter
  labels:
    prometheus: netapp
    prometheusEnv: prd
spec:
  selector:
    matchLabels:
      app: harvest
  endpoints:
  - port: metrics
    interval: 1m
    path: /metrics
---
apiVersion: v1
kind: Secret
metadata:
  name: harvest-poller-config
  namespace: prometheus-exporter
  labels:
    app: harvest
type: Opaque
stringData:
  harvest.yml: |
    Exporters:
      prometheus:
        exporter: Prometheus
        local_http_addr: <IP_ADDRESS>
        port: 12990
      prometheus1:
        exporter: Prometheus
        port: 12991
    Defaults:
      collectors:
        - Zapi
        - ZapiPerf
      use_insecure_tls: true   # <====== adjust as needed to enable/disable TLS checks
      exporters:
        - prometheus1

    Pollers:
      # this is a special/optional poller that provides stats about harvest itself
      # to remove this poller, simply delete the section "unix"
      unix:
        datacenter: local
        addr: localhost
        collectors:
          - Unix
        exporters:
          - prometheus

      netappdev:                # <====== add your cluster(s) here, they use the exporter defined three lines above
        datacenter: DEV
        addr: <IP_ADDRESS>
        auth_style: basic_auth
        username: "domain\\username"
        password: "password"
        exporters:
          - prometheus1
---
apiVersion: v1
kind: ConfigMap
metadata:
  name: zapi-default-config
  namespace: prometheus-exporter
  labels:
    app: harvest
data:
  default.yaml: |
    collector:          Zapi

    # Order here matters!
    schedule:
      - instance: 300s
      - data: 50s

    client_timeout: 40

    objects:
      Node:             node.yaml
      Aggregate:        aggr.yaml
      Volume:           volume.yaml
      SnapMirror:       snapmirror.yaml
      Disk:             disk.yaml
      Shelf:            shelf.yaml
      Status:           status.yaml
      Subsystem:        subsystem.yaml
      Lun:              lun.yaml

i cant run it on any other machine.
but after i talked to the storage guys and changed the ip i got this:
7:49AM INF command-line-arguments/poller.go:153 > log level used: info Poller=netappdev
7:49AM INF command-line-arguments/poller.go:154 > options config: /opt/harvest.yml Poller=netappdev
7:49AM INF command-line-arguments/poller.go:179 > started in foreground [pid=1] Poller=netappdev
7:49AM WRN command-line-arguments/poller.go:532 > init collector-object (Zapi:Node): connection error => api request rejected => Insufficient privileges: user 'domain1\user1' does not have read access to this resource Poller=netappdev
7:49AM WRN command-line-arguments/poller.go:534 > aborting collector (Zapi) Poller=netappdev
7:49AM WRN command-line-arguments/poller.go:532 > init collector-object (ZapiPerf:SystemNode): connection error => api request rejected => Insufficient privileges: user 'domain1\user1' does not have read access to this resource Poller=netappdev
7:49AM WRN command-line-arguments/poller.go:534 > aborting collector (ZapiPerf) Poller=netappdev
7:49AM WRN command-line-arguments/poller.go:257 > no collectors initialized, stopping Poller=netappdev
7:49AM INF command-line-arguments/poller.go:421 > cleaning up and stopping [pid=1] Poller=netappdev

this are the permissions they gave me:
API	ACCESS LEVEL
/api/cluster	Read-Only
/api/name-services/dns	Read-Only
/api/name-services/ldap	Read-Only
/api/name-services/name-mappings	Read-Only
/api/name-services/nis	Read-Only
/api/name-services/nsswitch	Read-Only
/api/network/fc/interfaces	Read-Only
/api/network/fc/logins	Read-Only
/api/network/fc/wwpn-aliases	Read-Only
/api/network/http-proxy	Read-Only
/api/network/ip/interfaces	Read-Only
/api/network/ip/service-policies	Read-Only
/api/protocols/audit	Read-Only
/api/protocols/cifs/home-directory/search-paths	Read-Only
/api/protocols/cifs/services	Read-Only
/api/protocols/cifs/shares	Read-Only
/api/protocols/cifs/unix-symlink-mapping	Read-Only
/api/protocols/file-security/effective-permissions	Read-Only
/api/protocols/fpolicy	Read-Only
/api/protocols/nfs/connected-clients	Read-Only
/api/protocols/nfs/export-policies	Read-Only
/api/protocols/nfs/kerberos/interfaces	Read-Only
/api/protocols/nfs/kerberos/realms	Read-Only
/api/protocols/nfs/services	Read-Only
/api/protocols/nvme/interfaces	Read-Only
/api/protocols/nvme/services	Read-Only
/api/protocols/nvme/subsystem-controllers	Read-Only
/api/protocols/nvme/subsystem-maps	Read-Only
/api/protocols/nvme/subsystems	Read-Only
/api/protocols/san/fcp/services	Read-Only
/api/protocols/san/igroups	Read-Only
/api/protocols/san/iscsi/credentials	Read-Only
/api/protocols/san/iscsi/services	Read-Only
/api/protocols/san/iscsi/sessions	Read-Only
/api/protocols/san/lun-maps	Read-Only
/api/protocols/vscan	Read-Only
/api/protocols/vscan/server-status	Read-Only
/api/security/accounts	Read-Only
/api/security/authentication/password	Read/Write
/api/security/authentication/publickeys	Read/Write
/api/security/azure-key-vaults	Read-Only
/api/security/certificates	Read-Only
/api/security/gcp-kms	Read-Only
/api/security/ipsec	Read-Only
/api/security/ipsec/policies	Read-Only
/api/security/ipsec/security-associations	Read-Only
/api/security/key-managers	Read-Only
/api/security/login/messages	Read-Only
/api/security/roles	None
/api/storage/file/clone	Read-Only
/api/storage/flexcache/flexcaches	Read-Only
/api/storage/flexcache/origins	Read-Only
/api/storage/luns	Read-Only
/api/storage/namespaces	Read-Only
/api/storage/qtrees	Read-Only
/api/storage/quota/reports	Read-Only
/api/storage/quota/rules	Read-Only
/api/storage/snapshot-policies	Read-Only
/api/storage/volumes	Read-Only
/api/svm/peer-permissions	Read-Only
/api/svm/peers	Read-Only
/api/svm/svms	Read-Only
/api/svms	Read-Only

hi @LH8PPL - those permissions look like REST permissions and may not match the permissions Harvest needs since Harvest is sending ZAPIs. Can you take a look at this page on permissions and check that you have necessary access?

Hi, thanks for the replies.
This is what our storage guys sent me, they say this are the permissions they gave me through the CLI:
DEFAULT                              none
df                                   readonly
lun                                  readonly
network interface                    readonly
network interface failover-groups    none
security certificate                 readonly
security certificate file            none
security ipsec config show           readonly
security ipsec policy                all
security ipsec show-ikesa            readonly
security ipsec show-ipsecsa          readonly
security key-manager external        readonly
security key-manager key query       all
security key-manager show-key-store  readonly
security key-manager volume-encryption-supported  readonly
security login banner                readonly
security login motd                  readonly
security login password              all
security login publickey             all
security login role show-ontapi      all
security login role show-user-capability  all
security ssh                         readonly
security ssl                         readonly
set                                  all
statistics                           readonly
statistics cache                     none
statistics catalog                   readonly
statistics lif                       none
statistics lun                       none
statistics samples                   none
statistics show-periodic             none
statistics start                     none
statistics stop                      none
statistics volume                    none
timezone                             all
version                              all
volume                               readonly
volume check                         none
volume constituent                   none
volume efficiency                    none
volume move                          none
vserver                              readonly
vserver http-proxy                   readonly
vserver nvme                         readonly
vserver services name-service unix-group max-limit  none
vserver services name-service unix-user max-limit  none
vserver services web                 none

All set on this one, closing


feat: remove dependency on request

This is like 99% home :)  I'm having some struggles figuring out the failing kitchen sink test.  Ready for a first look.

@stephenplusplus I have a few questions about the code in here that I think may be easier to discuss over a GVC.  You have any time this afternoon or tomorrow to chat?

@JustinBeckwith I pushed a commit that is probably the source of the issue. I'm not sure what's going on, which you can read from the console.log's :) The behavior of cork- and uncorking seems to not be working as expected. Leaving it here in case you have any ideas, otherwise I'll look into it as time permits!

@stephenplusplus I decided to do this in 2 parts.  This part will just work the tests so that the next part  is easier :)

gentle ping