Waymo scene test

Thanks for your excellent work!
I follow the README to train gaussianpro on the Waymo dataset you provided, and I don't change any parameters, but i can't get same results as your paper showed, especially the rendered normal result, do you know why?
what i run:

what you show:

Thank you for your recognition. Your results do seem similar to the baseline. I suspect there might be an issue with the line breaks in your bash file, i.e. \ in each line, causing the parameters to not be properly inputted.

Hi @kcheng1021,
Many for this very interesting works.
I reached the same issue as @FAM-vive. I run it a second time with the train command on an unique line:
python train.py -s my-data-path -m my-output-path --eval --flatten_loss --position_lr_init 0.000016 --scaling_lr 0.001 --percent_dense 0.0005 --port 1021 --dataset waymo --sky_seg --normal_loss --depth_loss --propagation_interval 30 --depth_error_min_threshold 0.8 --depth_error_max_threshold 1.0 --propagated_iteration_begin 1000 --propagated_iteration_after 12000 --patch_size 20 --lambda_l1_normal 0.001 --lambda_cos_normal 0.001

the metric are lower than what you published:
"SSIM": 0.9529152512550354,
"PSNR": 35.29518127441406,
"LPIPS": 0.226970374584198
For information the metric of baseline: python train.py -s my-data-path  -m my-output-path --eval --position_lr_init 0.000016 --scaling_lr 0.001 --percent_dense 0.0005 --port 1021 --dataset waymo
are very close to ours:
SSIM :    0.9498498
PSNR :   35.0038567
LPIPS:    0.2342816
Clearly the normals rendering is really worst than ours. Do you have any idea of I am doing wrong ?
thanks

Based on the current findings, it seems that the propagation process might not have been properly executed in your results. To better understand the situation and identify the underlying causes, it would greatly benefit the analysis if you could share the scene results from your training.
Additionally, I would like to offer my assistance in other two ways: either by remotely connecting to your machine or by providing my contact information via email. (drop your email or other contact info to me) This will allow us to establish direct communication and solve the issue in the quickest possible manner. @FAM-vive @merriaux

@FAM-vive @merriaux
I have met the similar problem as yours. It turns out that there is a mistake in my bash file just as @kcheng1021 mentioned.
After I fixed it, I got the correct result as showed below:

Hi  @kcheng1021 and @KizzySama
Yes I used waymo sequence from your google drive link. I didn't notice any warning during training.
I will write you an email, and start upload of training results.
@KizzySama, could share with us the train cmd you use ? or have a look to the cmd I used in my first message ?
thanks

Hi @kcheng1021 and @KizzySama Yes I used waymo sequence from your google drive link. I didn't notice any warning during training. I will write you an email, and start upload of training results.
@KizzySama, could share with us the train cmd you use ? or have a look to the cmd I used in my first message ? thanks

Your training command looks correct, maybe you need to check whether your propagation is up and running. My result looks like:

Hi, @merriaux @FAM-vive. Perphas I found the reason. I specified the GPU architecture as sm86 in the CMakeLists.txt, but your GPUs may have a lower compute capability, which is why the propagation was not successful. Please replace it with a version that is suitable for your GPU.

@kcheng1021 , thanks so much! I have solved the problem by what you said, and then the propagation process executed successfully, and finally i get same results as your paper showed.


Unclear how to use set-header with multiple headers

A key use case for set-header is rewriting cookies. In many applications there is more than one cookie being set, so this would involve multiple "Set-Cookie" headers. Here, I attempt to simply replay the outbound cookies in a new header:
<set-header name="Test-Cookie" exists-action="override">
   <value>@{
      return context.Response.Headers.GetValueOrDefault("Set-Cookie", "");
   }</value>
</set-header>

This results in a single header being output, with comma-separated cookies. This approach isn't helpful because cookies set with an expiry date have a comma in the date portion, as noted by these docs. However this statement:

In case of those exceptions, multiple header values will not be concatenated into one string and will be passed as separate headers, for example...

Doesn't appear to be true because the above example XML policy snippet results in all "Set-Cookie" headers being string joined into a comma-separated string, even though some of those cookies contain dates with commas in them. This is a nightmare for parsing.
Finally, simply setting the header value as above doesn't replicate the Set-Cookie behaviour in the Test-Cookie header: instead of multiple instances of "Test-Cookie" header, we have a single instance with all the cookies concatenated together. It's none-obvious that returning a header value would not result in the same header structure.
There's an implication in the docs that:

Exceptions include standardized headers whose values ... may contain date (Cookie, Set-Cookie, Warning)

But it's not clear what the nature of that exception is. Should I not use GetValueOrDefault with Set-Cookie?
Finally, the following guidance could do with an example for how you set these from an expression:
<!-- Copy this snippet into the inbound element to forward some context information, user id and the region the gateway is hosted in, to the backend service for logging or evaluation -->
<set-header name="x-request-context-data" exists-action="override">
  <value>@(context.User.Id)</value>
  <value>@(context.Deployment.Region)</value>
</set-header>

It doesn't appear that you can return a string array from an expression here. Would a yield return work? Remove the "value" field and put the expression returning an array there? To be clear, what I'd like to achieve is:
<!--Fix any cookies by rewriting their path -->
<set-header name="Test-Cookie" exists-action="override">
   <value>@{
      var headerValue = context.Response.Headers.GetValueOrDefault("Set-Cookie", "");
      var inputCookies = headerValue.Split(',');
      var outputCookies = new List<string>();
      foreach (var inputCookie in inputCookies) {
         outputCookies.Add(inputCookie.Replace("path=/;", "path=/api;"));
      }
      return outputCookies;
   }</value>
</set-header>

This fails validation because outputCookies is of the wrong type, and also wouldn't work because the string.Split would break over a cookie's expiry date, but it gives an idea of what I'm trying to accomplish. I also don't think this is a crazy outlandish scenario.
Please could the docs be updated to make it clearer how to deal with multi-instance headers?

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

ID: b57f5625-4eeb-0b5c-24f3-094da2d9414d
Version Independent ID: de82cd56-a100-5c32-a4c6-171454033ad9
Content: Azure API Management policy reference - set-header
Content Source: articles/api-management/set-header-policy.md
Service: api-management
GitHub Login: @dlepow
Microsoft Alias: danlep

@nathankitchen
Thanks for your feedback! We will investigate and update as appropriate.

@nathankitchen Thanks again for sharing your feedback here. I provided an answer and code snippet regarding your questions in https://learn.microsoft.com/en-us/answers/questions/1390333/how-to-use-apim-set-header-policy-to-manage-set-co thread.
To summarize,

GetValueOrDefault returns comma-separated response header values as described in the doc.
Exceptions include standardized headers whose values ... may contain date (Cookie, Set-Cookie, Warning) works as expected and shared the output in the thread. Need to make sure the backend API returns as multiple header values instead of comma separated values.
Set Header doc covers more about the policy, attributes and usage etc. We try to cover use cases, scenario-based policy snippets in https://github.com/Azure/api-management-policy-snippets repo and this will be a good addition there. I see we have an open item regarding this (we can use it for tracking) and will pass the feedback to our product team to add it there.

I hope this helps and if you have any questions, please feel free to reach out or add a comment. Would be happy to answer any. We will proceed to close this thread for now.


#663 Fix missing "done" button in ChangeLogView

Great! Please rebase your change and remove the previous commits.

I will do it 😂

You need to start on main branch and make sure the pr only contains the changes needed.


Add error callback support to require.ensure

What kind of change does this PR introduce?
Fixes https://github.com/webpack/webpack/issues/758 by adding support for an error handling callback to require.ensure

Did you add tests for your changes?
Yes, tests added in test/cases/parsing/issue-758
These were borrowed/adapted from https://github.com/webpack/webpack/pull/3247.

If relevant, link to documentation update:

Summary

System.import already supports error callbacks, but require.ensure is still useful for a couple of reasons:

Supports chunk naming (https://github.com/webpack/webpack/issues/3132)
Supports recursive code splitting (https://github.com/webpack/webpack/issues/3364#issuecomment-273983958)

Does this PR introduce a breaking change?
No. require.ensure still supports chunk naming with the old 3 argument style.

Other information

I was thinking it'd also be good to add a couple of tests to test/cases/chunks possibly under test/cases/chunks/named-chunks but I could use some pointers on how the sync stuff is working in the existing tests.
I'm not quite sure what these lines are doing:
var sync = false;
//...
sync.should.be.ok();

Wow what an incredible first PR to take on btw.

I've been percolating on this issue for quite a while! I had actually forked the awesome work of @richardscarrott (https://github.com/richardscarrott/require-error-handler-webpack-plugin/pull/5) and adapted/rewritten his work to make it work with Webpack 2.1.0-beta.25 a few months ago, and then realized it totally broke with the changes made Webpack internals since then.
This motivated me to make a new PR for require-ensure error callbacks so I don't have to keep maintaining the fork! :-)
If this PR gets approved, I'd also really love to make a PR on the bundle-loader to also add error callback support there. @richardscarrott already reworked the bundle-loader to handle this (https://github.com/richardscarrott/require-error-handler-webpack-plugin/blob/master/src/BundleLoader.js) and I have a corresponding fork created (https://github.com/jharris4/bundle-loader)

Is there anything else I can do to push this along to get it merged?
We're already using the require.ensure error callbacks in production, so to allow us to move from Webpack 1.x to 2.2 I've created this plugin which enables the changes: https://github.com/jharris4/require-ensure-error-webpack-plugin.
It has a standalone example of the changes in action if anyone's curious...

@TheLarkInn @sokra Still curious to know if there's a reason for not merging this... :-)
I'd be happy to take on any related doc updates, or adding tests for other features that depend on require.ensure if that'd help move things along.

Thanks

Awesome! I think the bundle-loader needs a PR to work with this now though.
I'll try to get to that when I have some spare time.


[stable/redmine] Remove distro tag

Signed-off-by: Carlos Rodriguez Hernandez<EMAIL_ADDRESS>As part of the Debian 8 to 9 migration, we started using the "debian-9" tag.
At this moment, we should remove them and use the main version only.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md

/lgtm

/ok-to-test


clangd hangs when building AST of nested macro

Problem: clangs hangs at building AST when expanding nested macro
code: https://gist.github.com/9tin9tin9/8145049d78d8c8dacba88fccc4941bc1
log: https://gist.github.com/9tin9tin9/0099bdb6d2261956bf0a9889e167596e
system information: Macbook Pro 2017; MacOS Big Sur 11.5.2

Hmm, I can't reproduce this hang, though clangd --check does take 40 seconds for "testing features at each token".
Would you be able to check if this still hangs with a recent build e.g. https://github.com/clangd/clangd/releases/tag/snapshot_20211031 ?

It took 6 mins for it to kill itself on the original version 13.0.0
log: https://gist.github.com/9tin9tin9/9be0f22e1dced26a2b92bdec91d2289d
And using the build you gave me
log: https://gist.github.com/9tin9tin9/1166d168372b782810a4bb2334c7a146
took 6 mins for it to kill itself again.

I think the main issue here is the memory usage. It uses about 14GB memory at its peak. Running clang on the same code only consumes about 80MB. Sending out a patch which brings down peak memory usage down to ~60MB.


restrictToOwner 'as' option

In docs, it is said that, for the restrictToOwner hook,

For find method calls and patch, update and remove of many (with id set to null), the queryWithCurrentUser hook will be called to limit the query to the current user.

However, restrictToOwner has a ownerField option, whereas queryWithCurrentUser wants a as option.
When calling queryWithCurrentUser, restrictToOwner shouldn't rename the ownerField options as ?

It does indeed, that's what happens when you run your tests only against the defaults. Thanks for the catch, fixed in v0.1.1

Do we need to update anything in the docs? I think it makes sense now right?

No, no need to update the docs.
Thanks !


Non-updated policies are pushed to devices

When we click the apply policy changes button, policy which does not have any changes are also add to devices. This will cause performance issue when we have many devices.

Closing the issue as per the above comment.


Added deferReply() to indicate the bot is "thinking" on servers

Hey,
just changed a small thing to answers on servers. The bot now shows "ChatGPT is thinking..." while he tries to get an answer. When the response is available, the bot will show it.

Hi can you put "await" front of the .deferReply();

ofc

Thank you!


Mix.Error Unknown dependency :ecto_sql

Environment

Erlang/OTP 23 [erts-11.1.7] [source] [64-bit] [smp:4:4] [ds:4:4:10] [async-threads:1] [hipe]
Elixir 1.11.2 (compiled with Erlang/OTP 23)
VSCode ElixirLS version: 0.6.5
Operating System Version: linux 5.4.0-67-generic

Output
Started ElixirLS v0.6.5
Elixir version: "1.11.2 (compiled with Erlang/OTP 23)"
Erlang version: "23"
ElixirLS compiled with Elixir 1.8.2 and erlang 21
Skipping build because cwd changed from /home/stefan/phoenix/pan-in-development/. to /home/stefan/phoenix/pan-in-development
[Info  - 09:53:17] Dialyzer analysis is up to date
[Warn  - 09:53:30] warning: Unable to get formatter options for /home/stefan/phoenix/pan-in-development/lib/pan_web/frontend_controller/category_frontend_controller.ex: Mix.Error Unknown dependency :ecto_sql given to :import_deps in the formatter configuration. The dependency is not listed in your mix.exs for environment :test
  (language_server 0.6.5) lib/language_server/source_file.ex:331: ElixirLS.LanguageServer.SourceFile.formatter_opts/1
  (language_server 0.6.5) lib/language_server/providers/formatting.ex:7: ElixirLS.LanguageServer.Providers.Formatting.format/3
  (language_server 0.6.5) lib/language_server/server.ex:753: anonymous fn/3 in ElixirLS.LanguageServer.Server.handle_request_async/2

[Error - 09:53:30] Request textDocument/formatting failed.
  Message: Unable to fetch formatter options
  Code: -32603

More Info, Comments and Thanks

The full repo code is available to be browsed over here.
The mix file contains ecto_sql as an explicit dependency
If I'd remove ecto_sql from formatter.exs and just leave phoenix in as a dependency, the error message would be basically the same, but with phoenix
I earlier had an extra formatter.exs with ecto_sql in the migrations dir where it created the same error message.
The mix format task works nicely on the command line, so I have a fallback solution.
I had it working before (with older elixir and phoenix versions) on this machine.
Thank you for all your work! (It was really super nice, as long as it worked, you made me switch to VSCode for elixir-development, I was on Atom before.)
If I can provide any more infos to help sort this out, please let me know.

Cannot reproduce.
The repo you posted does not import any deps in .formatter.exs. Even when I added :phoenix and :ecto_sql I couldn't reproduce this crash.
Please check if your Project Dir config is correct, this line is suspicious
Skipping build because cwd changed from /home/stefan/phoenix/pan-in-development/. to /home/stefan/phoenix/pan-in-development

If you're using standard directory layout then it should be empty.

Thank you for pointing me in the right direction, I had "." as Project Dir setting. Removed that, restarted, compilation succeeded and formatting docs works like a charm again.


Update flutter framework ci to use api 33 devices in either emulators or FTL

Api 33 is used by a many android users and flutter ci has not been updated to run on devices past api 30.
https://docs.flutter.dev/reference/supported-platforms
Turns out emulators use api 31 for some tests.
{"dependency": "android_virtual_device", "version": "31"}
in .ci.yaml

Documenation https://g3doc.corp.google.com/company/teams/flutter/infrastructure/luci/android_emulators.md?cl=head
Emulator code https://cs.opensource.google/flutter/recipes/+/main:recipe_modules/android_virtual_device/api.py

Engine code is here engine/src/flutter/ci/firebase_testlab.py

https://github.com/flutter/flutter/issues/114036

https://flutter-review.googlesource.com/c/recipes/+/45048

https://flutter-review.googlesource.com/c/recipes/+/45049

@reidbaker seems like the infra changes have already landed. I'll remove team: infra and add a: tests. Please feel free to add the infra label again if there are some action items for infra.

@godofredoc yep that is correct

Completed by https://github.com/flutter/engine/pull/42492


Add custom root to analyze annotations from DSA AI adapter

[ ] Prevent the girder from ingesting data for the ML output
[ ] Create a new tag for annotation data from the AI adapter.
[ ] Update the worker is a task is completed.

We really should do this for annotation files in the worker -- not differentiation what cli generates the annotation files.


missing matplotlib import in introduction.py

Hi Joel,
I have created this pull request after including the matplotlib import in introduction.py.
https://github.com/joelgrus/data-science-from-scratch/pull/15
please let me know if you have any questions.

This is already merged by Joel


Python Program for Merge Sort

Definition of done:

Clear code should be written.
Comments will be really appreciable.
Add minimum 2 test cases to test the code.
Add you code under Python/Data Structure and Algorithms/<ProgramName.py>
Make sure to add the Screenshot of Program Testing in the comment while merging the PR

can i work on this @jatinkumar027

Sure

this task is done . please merge request @jatinkumar027


Make cursor a pointer on hovering over footer links

Is your feature request related to a problem? Please describe.
In the web app, the links in the footer should announce that they are interactive by making the cursor a pointer when it hovers over a link.
Additional context

I can't reproduce this, for me the cursor is correctly a pointer.


Uglify errors with jarallax

I am getting the following error when I try to uglify the following.
I don't think the problem is with jarallax as code works fine on its own.
Contents of app.js
import {
    jarallax,
    jarallaxElement,
    jarallaxVideo
} from 'jarallax';

Error
ERROR in /js/app.js from UglifyJs
TypeError: sym.definition is not a function
    at may_modify (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:8931:31)
    at TreeTransformer.abort [as before] (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:8541:74)
    at AST_VarDef.eval [as transform] (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:4570:35)
    at eval (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:4586:25)
    at doit (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:130:23)
    at MAP (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:156:52)
    at do_list (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:4585:16)
    at eval (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:4664:28)
    at AST_Const.eval [as transform] (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:4573:17)
    at collapse (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:8617:39)
    at tighten_body (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:8429:17)
    at eval (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:10529:9)
    at AST_Accessor.eval [as optimize] (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:7682:23)
    at Compressor.before (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:7665:24)
    at AST_Accessor.eval [as transform] (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:4570:35)
    at eval (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:4739:33)
    at Compressor.before (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:7660:9)
    at AST_ConciseMethod.eval [as transform] (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:4570:35)
    at eval (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:4586:25)
    at doit (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:130:23)
    at MAP (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:156:52)
    at do_list (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:4585:16)
    at eval (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:4745:27)
    at Compressor.before (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:7660:9)
    at AST_DefClass.eval [as transform] (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:4570:35)
    at eval (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:4586:25)
    at doit (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:130:23)
    at MAP (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:156:52)
    at do_list (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:4585:16)
    at eval (eval at <anonymous> (/redacted/path/node_modules/uglify-es/tools/node.js:21:1), <anonymous>:4682:25)
error An unexpected error occurred: "Command failed.
Exit code: 2
Command: sh

Resolved by #296

Bug workaround for yarn users:
https://github.com/fabiosantoscode/terser#replacing-uglify-es-with-terser-in-a-project-using-yarn

Close in favor https://github.com/webpack-contrib/uglifyjs-webpack-plugin/issues/264

@evilebottnawi This bug has nothing to do with inline and is not addressed by #264.
It is resolved by replacing uglify-es with terser in #296.

Close in favor https://github.com/webpack-contrib/uglifyjs-webpack-plugin/issues/298


image to byte array - segment too large

This might be more of a question, depending on if this is actually solvable with the current vips-ffm api.
JVips:
	@Override
	public byte[] writeJpegImage(VipsImage image, float quality) {
		try {
			image.applyIccTransform("srgb", new ICCTransformOptions().embedded(true));
		} catch(VipsException ignored) {}

		return image.writeJPEGToArray(toVipsQuality(quality), true);
	}

vips-ffm:
	@Override
	public byte[] writeJpegImage(VImage image, float quality) {
		VBlob blob = image
			.iccTransform("srgb", VipsOption.Boolean("embedded", true))
			.jpegsaveBuffer(VipsOption.Int("Q", toVipsQuality(quality)));

		return blob.getUnsafeAddress().toArray(ValueLayout.JAVA_BYTE);
	}

This works up to a certain image size.
For larger images though, it produces the following:
Caused by: java.lang.IllegalStateException: Segment is too large to wrap as byte[]. Size:<PHONE_NUMBER>854775807
        at jdk.internal.foreign.AbstractMemorySegmentImpl.checkArraySize(AbstractMemorySegmentImpl.java:396) ~[?:?]
        at jdk.internal.foreign.AbstractMemorySegmentImpl.toArray(AbstractMemorySegmentImpl.java:352) ~[?:?]
        at jdk.internal.foreign.AbstractMemorySegmentImpl.toArray(AbstractMemorySegmentImpl.java:318) ~[?:?]
        at some.package.SomeClass.writeJpegImage(SomeClass.java:80) ~[app_B:?]

What would be the correct way to get a byte array from an image, without writing it out to disk and reading it in again?

Actually, looking at the reported size, it looks to me like either a NUL termination byte is missing at the end of the blob, or I need to get the length somewhere to limit the used blob size, because a jpeg image is definitely not around 9 petabyte :)

I think it's reasonable to want a safe (no raw pointers) and efficient (no copying) way to get at raw VBlob (VipsArea) data, so I've added a couple of new methods to VBlob in 0.5.7:

VBlob.asByteBuffer() - recommended way to get at the bytes, should be directly mapped to native memory for efficiency, and you can still access to the byte[] array from there if you need it
VBlob.byteSize() - if you're sure you just want the byte length of the data, grabbed right out of the VipsArea struct

@lopcode this seems to work now, thanks!
I think it would be good to add to docs/readme/javadoc/... that the ByteBuffer returned by these is a DirectByteBuffer pointing to the memory inside the Arena, and thus should not be accessed once the arena goes out of scope (I just had to debug an issue with that, and have now switched back to returning only byte[] from our final write* methods to make sure there'll be no issue with that).

@lopcode this seems to work now (0.5.9), thanks!
I think it would be good to add to docs/readme/javadoc/... that the ByteBuffer returned by these is a DirectByteBuffer pointing to the memory inside the Arena, and thus should not be accessed once the arena goes out of scope (I just had to debug an issue with that, and have now switched back to returning only byte[] from our final write* methods to make sure there'll be no issue with that).

Yep agree about the docs - I made two improvements:

Renamed VipsBlob.asByteBuffer to VipsBlob.asArenaScopedByteBuffer
Added VipsBlob.asClonedByteBuffer
Made docs clearer for both


Secondary sidebar - easier drag and drop

Right now for me to drag and drop a view in the secondary sidebar I have to drop it in the small title area of the secondary sidebar.
If I drop it in the large view area, it basically splits the view container.
I know this logically makes sense, but makes the whole DnD experience unusable.
Could DnD see if I am dragging the only view in the container, and if yes always drop it as a separate container?
I am constantly being bitten by this (even with previous DnD scenarios).

@isidorn but that is not specific to the secondary sidebar, no? that is true for primary and panel as well. Are you suggesting to change this only for secondary sidebar?

Aligning with how it works with editors could work, even though its overall a smaller hit area.

Worth exploring. I do see there being a couple of weird scenarios. Also, I don't think it will fell as natural/intuitive as for the editor grid due to the smaller hit area, but it might still work.

The reason I like this idea is that I wonder if the 80% case is to move a view into the bar instead of splitting. Also for the editor, the hit zone to move is much larger than splitting. I would think for views its even more a case of moving vs. splitting.

Keep in mind that the vertical hit zone in sidebar is same length as in editor area. So imho it might work.

if the 80% case is to move a view into the bar instead of splitting

This is the key point here 💯

I think it makes sense. I think it is more intuitive that moving a view container from one location to another should just change its location instead of merging it into another view container. This should be true for all locations (Primary Sidebar, Secondary Sidebar, Panel). If user moves the view container on top of the another view in the same location, then it should move all views.

Can we borrow the drag and drop behavior from editor where you can hold shift to drop into the editor instead of splitting the editors. Cases would be:

Dragging views within their own container for reorder (no shift behavior, just drops in the view container)
Dragging a view from another container, (debatable what the non-shift behavior should be but I recommend it creates a new custom container by default and holding shift drops inside the container)
Dragging a view container from another panel/composite (no shift: moves container to bar, shift: drops views in split)

Weird case: dragging a view container from the same panel/composite into another view container. Following the rules above, the default action would be a no-op as you would just move the view container in the composite bar. There shouldn't be ambiguity here though, we should just split.


let API names follow Rust idiom

I saw two examples so far:

rep::Event::timeNano should be rep::Event::time_nano
errors::Error:IO(IoError) should be errors::Error:Io(IoError)

Good call.

this was fixed in the long needed migration to serde https://github.com/softprops/shiplift/pull/100


[tizen] black blank screen on app running from Windows

Description

Fix tizen blank screen problem in devices when run it  from windows os

Related issues

https://github.com/flexn-io/renative/issues/1653

Npm releases
n/a

App is installed but fails to launch, maybe tv issue, as when I try to launch it with remote on tv it also fails to launch, tries to launch for some time (spinner) and then stops trying


Additional logging per #926.

What this PR does / why we need it:
This adds additional logging around the rfc2136 provider. There is a commented out block that may be a better version, it should be removed if not.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Someone with a better idea about how the documentation is laid out should review what I've wrote to try and homogenize the style with links and stuff.
Release note:

NONE

/ok-to-test

@munnerz Thanks for your feedback, very helpful! It's a pleasure to give something back to the project.

Reversed the changes back to sequential "fail-fast" checking of servers and updated the docs so users could be more versed in the expected behavior.
Also added some additional explanation and cleaned up some grammar there.
Finally modified the printing routine for TSIG key to mask out the middle of the key as described


Recommend installing numba and tbb for WoQ

Fix https://github.com/intel/neural-compressor/issues/2031#issuecomment-2429698785
render version https://github.com/intel/neural-compressor/blob/numba_doc/docs/source/3x/PT_WeightOnlyQuant.md#get-started

Could we also simplify the log message?

BTW, many colleagues complain with me about the tbb issue.
How about using below logic:
try:
if not check_numba_tbb():
logger.warning("To accelerate packing, please install numba with pip install numba tbb")
except:
logger.warning(“numba and tbb is installed but tbb is not correctly configured")
logger.warning(“please follow xxx to configure it, here we use numpy instead")

BTW, the tbb issue is not friendly. It will break progress during quantization. How about using below logic:
try:
   if not is_numba_tbb_available():               # no package
       logger.warning("To accelerate packing, please install numba with `pip install numba tbb`")
except:                                                         # has package but installed incorrectly
   logger.warning(“numba and tbb is installed but tbb is not correctly configured")
   logger.warning(“please follow xxx to configure it, here we use numpy instead")
...                                                                  # has package and installed correctly

is_numba_tbb_available should both check installation and tbb configuration

Do you mean the program will crash if tbb isn’t configured correctly?
It’s not easy to summarize how to make numba and tbb work as expected in one sentence; maybe I could write a short note for it.

Yes, It will cruch. BTW, I mean use link here.
logger.warning(f“please follow {link} to configure it, here we use numpy instead")

Yes, It will cruch. BTW, I mean to use link here. we already have introduce about this issue in FAQ. logger.warning(f“please follow {link} to configure it, here we use numpy instead")

I see, I will enhance this check.

Enhanced the TBB check on https://github.com/intel/neural-compressor/pull/2068, close it.


Deploy Checklist: New Expensify 2023-05-22

Release Version: 1.3.17-5
Compare Changes: https://github.com/Expensify/App/compare/production...staging
This release contains changes from the following pull requests:

[x] https://github.com/Expensify/App/pull/15051
[x] https://github.com/Expensify/App/pull/17477
[x] https://github.com/Expensify/App/pull/18073
[x] https://github.com/Expensify/App/pull/18214
[x] https://github.com/Expensify/App/pull/18523
[x] https://github.com/Expensify/App/pull/18782
[x] https://github.com/Expensify/App/pull/18809
[x] https://github.com/Expensify/App/pull/18847
[x] https://github.com/Expensify/App/pull/18944
[x] https://github.com/Expensify/App/pull/18958
[x] https://github.com/Expensify/App/pull/18960
[x] https://github.com/Expensify/App/pull/18967
[x] https://github.com/Expensify/App/pull/18987
[x] https://github.com/Expensify/App/pull/18995
[x] https://github.com/Expensify/App/pull/19021
[x] https://github.com/Expensify/App/pull/19030
[x] https://github.com/Expensify/App/pull/19040
[x] https://github.com/Expensify/App/pull/19046
[x] https://github.com/Expensify/App/pull/19064
[x] https://github.com/Expensify/App/pull/19074
[x] https://github.com/Expensify/App/pull/19094
[x] https://github.com/Expensify/App/pull/19097
[x] https://github.com/Expensify/App/pull/19101
[x] https://github.com/Expensify/App/pull/19103
[x] https://github.com/Expensify/App/pull/19107
[x] https://github.com/Expensify/App/pull/19128
[x] https://github.com/Expensify/App/pull/19133
[x] https://github.com/Expensify/App/pull/19139
[x] https://github.com/Expensify/App/pull/19182
[x] https://github.com/Expensify/App/pull/19194
[x] https://github.com/Expensify/App/pull/19197
[x] https://github.com/Expensify/App/pull/19198
[x] https://github.com/Expensify/App/pull/19200
[x] https://github.com/Expensify/App/pull/19219
[x] https://github.com/Expensify/App/pull/19223
[x] https://github.com/Expensify/App/pull/19224
[x] https://github.com/Expensify/App/pull/19225
[x] https://github.com/Expensify/App/pull/19230
[x] https://github.com/Expensify/App/pull/19237
[x] https://github.com/Expensify/App/pull/19244
[x] https://github.com/Expensify/App/pull/19249
[x] https://github.com/Expensify/App/pull/19292
[x] https://github.com/Expensify/App/pull/19294
[x] https://github.com/Expensify/App/pull/19300
[x] https://github.com/Expensify/App/pull/19307
[x] https://github.com/Expensify/App/pull/19315
[x] https://github.com/Expensify/App/pull/19333
[x] https://github.com/Expensify/App/pull/19334
[x] https://github.com/Expensify/App/pull/19335
[x] https://github.com/Expensify/App/pull/19339
[x] https://github.com/Expensify/App/pull/19356
[x] https://github.com/Expensify/App/pull/19378
[x] https://github.com/Expensify/App/pull/19402
[x] https://github.com/Expensify/App/pull/19537
[x] https://github.com/Expensify/App/pull/19546

Deploy Blockers:

[x] https://github.com/Expensify/App/issues/19440
[x] https://github.com/Expensify/App/issues/19482
[x] https://github.com/Expensify/App/issues/19542
[x] https://github.com/Expensify/App/issues/19543

Deployer verifications:

[x] I checked the App Timing Dashboard and verified this release does not cause a noticeable performance regression.
[x] I checked Firebase Crashlytics and verified that this release does not introduce any new crashes. More detailed instructions on this verification can be found here.
[x] I checked GitHub Status and verified there is no reported incident with Actions.

cc @Expensify/applauseleads

:rocket: All staging deploys are complete, @Expensify/applauseleads please begin QA on version https://github.com/Expensify/App/releases/tag/1.3.17-0 :rocket:

@chiragsalian
We still dont have IOS build. Can you pls double check if Apple approved it

:tada: All set?…You bet! @Expensify/applauseleads https://github.com/Expensify/App/releases/tag/1.3.17-1 has been deployed to staging :tada:

@chiragsalian Regression is still in progress. Current summary
PR https://github.com/Expensify/App/pull/19101 is failing https://github.com/Expensify/App/issues/19469. repro in PROD checking off
#19335 is failing https://github.com/Expensify/App/issues/19462. We are not able to validate in PROD as number is not added. pls double check
https://github.com/Expensify/App/pull/19333 is failing  https://github.com/Expensify/App/issues/19493. repro in PROD checking off

:clap: Heads up @Expensify/applauseleads :clap:
A new pull request has been :cherries: cherry-picked :cherries: to staging, and will be deployed to staging in version 1.3.17-2 :rocket:

:tada: All set?…You bet! @Expensify/applauseleads https://github.com/Expensify/App/releases/tag/1.3.17-2 has been deployed to staging :tada:

:clap: Heads up @Expensify/applauseleads :clap:
A new pull request has been :cherries: cherry-picked :cherries: to staging, and will be deployed to staging in version 1.3.17-3 :rocket:

:tada: All set?…You bet! @Expensify/applauseleads https://github.com/Expensify/App/releases/tag/1.3.17-3 has been deployed to staging :tada:

https://github.com/Expensify/App/pull/19230 is a pass. Checking off.

:clap: Heads up @Expensify/applauseleads :clap:
A new pull request has been :cherries: cherry-picked :cherries: to staging, and will be deployed to staging in version 1.3.17-5 :rocket:

:tada: All set?…You bet! @Expensify/applauseleads https://github.com/Expensify/App/releases/tag/1.3.17-5 has been deployed to staging :tada:

https://github.com/Expensify/App/pull/19294 is pass on all platforms but Desktop. We are blocked on Desktop and URL doesnt navigate to the Concierge.

I checked off some items as per the discussions here

:shipit:


🔢  way to set proxy in fresh?

Hi everybody,I'd thanks you for nice fresh framework!
who can I set proxy for other server like proxy in webpack

You can set up an API route (a route with just a handler and no page component) that proxies requests to an upstream server using fetch.
We should add a docs page for this.

@alfuhigi I created a simple and quick server proxy https://github.com/gigitux/corsproxy
The code is pretty straightforward, so you can copy-paste it and add a new API route to your fresh project.


Draft: Codecov code coverage

Signed-off-by: Marcin Trojanowski<EMAIL_ADDRESS>
Summary
Codecov code coverage

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1e9a304). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #89   +/-   ##
=========================================
  Coverage          ?   95.27%
=========================================
  Files             ?       11
  Lines             ?      127
  Branches          ?        0
=========================================
  Hits              ?      121
  Misses            ?        6
  Partials          ?        0

Flag
Coverage Δ

unittests
95.27% <0.00%> (?)

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

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 1e9a304...5d0f88c. Read the comment docs.


How to deploy own custom dataset and pre-trained model?

Hi, Thanks for open-source the code!  Do you have any doc or tutorial for deploying own custom dataset and pre-trained model?

Hi @TmacTmac1992
(1) Deploying own custom dataset:
Please find this help doc for your reference.
(2) Deploying pre-trained model
You can load dataset with pre-labeling json on the platform directly. Does this meet your needs?
Also are you looking for connecting with your model?
May I ask which pre-trained model are you deploying?

Thanks for your quick reply!
Does xtreme1 support all model trained form Openpcdet?

Thanks for your interest. More models from openmmlab will be supported soon. Although the models from openmmlab share similar framework, the code has to be adapted to the Xtreme1. We also welcome more developers participating this project to optimize the interface and integrate more models.

Thanks for your answer!
Do you provide industry solution for xtreme1?

Yes, we do. Let’s discuss it more and please email to<EMAIL_ADDRESS>


Handling source-adjacent packages?

Do you have any advice on how to handle the situation of "a source control repo that contains a library package at the repo root + multiple binary packages in subdirectories", where the binary packages depend on crates defined in the library package, often referenced via relative paths in Cargo.toml?
(Here, I am specifically interested in using crane to write a flake that exposes outputs for the binary packages -- e.g., a web server, a wasm frontend, and some platform-specific GUI clients.)
Thanks! (and thanks also for writing and publishing crane; it looks like a lovely addition to the nix + rust ecosystem!)

Hi @mstone, thank you for the kind words!
Philosophically, Crane aims to be a thin wrapper around cargo to bridge any gaps with Nix. Besides that it should be possible to configure the derivation to invoke whatever cargo commands you'd use to build the project even outside of Nix.
Is the project in question all one big cargo workspace? By default Crane will build all binaries in the workspace, but you can override the cargo build flags to specify only one specific binary like so:
let
  src = ./src;
  # Build workspace artifacts (and cache the evaluation)
  cargoArtifacts = crane.buildDepsOnly {
    inherit src;
  };
in
{
  myBin = crane.buildPackage {
    inherit src cargoArtifacts;
    cargoBuildCommand = "cargo build --release --bin myBin";
  };

  myOtherBin = crane.buildPackage {
    inherit src cargoArtifacts;
    cargoBuildCommand = "cargo build --release --bin myOtherBin";
  };
}

If the project is modeled as multiple, distinct cargo-workspaces, there may need to be more configuration steps (e.g. to point cargo to the right place, or get everything vendored appropriately, etc.), but I can't speak much more about it without looking at a concrete example.
Does this help?

It does help, both to hear this description of the philosophy behind crane and to see from your reply the crucial cargoArtifacts argument that I had been neglecting to set in my calls to crane.buildPackage -- thanks!


Checkboxes in TransferList don't work if the values in the list aren't strings

Here's the code I used to reproduce the issue:
initial_values = [
        [
            {
                "value": 2,
                "label": "React",
            },
            {"value": 2, "label": "Angular"},
            {"value": 3, "label": "Next.js"},
            {"value": 4, "label": "Blitz.js"},
            {"value": 5, "label": "Gatsby.js"},
            {"value": 6, "label": "Vue"},
        ],
        [
            {"value": 7, "label": "Svelte"},
            {"value": 8, "label": "Redwood"},
            {"value": 9, "label": "NumPy"},
            {"value": 10, "label": "Django"},
            {"value": 11, "label": "Flask"},
        ],
    ]

dmc.TransferList(
    id="item-transfer-list",
    value=initial_values,
    searchPlaceholder=[
        "Search for item to add...",
        "Search for item to remove...",
    ],
    nothingFound=[
        "Cannot find for item to add",
        "Cannot find for item to remove",
    ],
    placeholder=[
        "No for item left to add",
        "No for item left to remove",
    ],
    mt=20,
),

The checkboxes will work if you convert the values using the str() function.

TransferList is no longer included in dmc.


Use NewClientWithIPAndWait when it creates metadataClient

Pls see https://github.com/rancher/rancher/issues/8603

@niusmallnan please separate this into two commits: one for the vendor changes and one for the changes to our code

@cjellick  have separate this into two commits.

@niusmallnan is this PR still relevant? if so, could you rebase?

@alena1108
Still in the discussion, no final decision.
This PR has a lower priority for now.

Closing as the pr is no longer relevant for rancher master (2.0)


Alignment of buttons containing an icon

Current Behavior
Experimenting with the buttons from the docs, I noticed that buttons containing an icon have their positioning changed slightly (to become higher).
Default:

Experimenting shows it's the height of the icon that influences this, up to a certain value, beyond which the visual size of the icon no longer changes:

Easiest solution is to wrap all icons in a container and use flex to align items:

But I'm curious if it might be possible to address icon styling, in order to make it easier for other devs out of the box. I don't know a solution offhand to achieve this, so wanted to open this for discussion. Any ideas?
Expected Behavior
I would expect all buttons to have same vertical alignment.
Steps To Reproduce
Just add buttons from the docs:
<script>
	import { Button } from '$lib/components/ui/button';
	import { Mail } from 'lucide-svelte';
	import { ChevronRight } from 'lucide-svelte';
	import { Loader2 } from 'lucide-svelte';
</script>

	<div class="flex items-center">
		<Button>Button</Button>
		<Button variant="secondary">Button</Button>
		<Button variant="outline">Button</Button>
		<Button variant="ghost">Button</Button>
		<Button variant="link">Button</Button>
		<Button variant="destructive">Button</Button>
		<Button>
			<Mail class="mr-2 h-4 w-4" />
			Login with Email
		</Button>

		<Button disabled>
			<Loader2 class="mr-2 h-4 w-4 animate-spin" />
			Please wait
		</Button>

		<Button variant="outline" size="icon">
			<ChevronRight className="h-4 w-4" />
		</Button>
	</div>

Link to Reproduction / Stackblitz
No response
More Information
No response

This is very weird behaviour, can't really find any other topics that discuss it online.
For some reason, the SVG (or its child nodes) changes the offsetTop property from 3px to 0px.
If you use a 4x4 svg then another (temporary) fix is adding relative and top-[3px] to the button class:
<Button class="relative top-[3px]">
	<Mail class="mr-2 h-4 w-4" />
	Login with Email
</Button>


Content Search via Msg IDs

It would be helpful to conduct a Content Search for specific Msg IDs.
Workflow starts with an audit log search. The unique data point provided in an Audit Log is the message ID. To my knowledge, I can't use Message ID directly to search for messages within content search and / or eDiscovery.

@brettsec Please indicate, which document you are referring to. Thanks

@bratsche We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if you see an improvement is required in the specific area of this documentation. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


cookie parser do not work

crow::App<crow::CookieParser, ExampleMiddleware> app;

    app.get_middleware<ExampleMiddleware>().setMessage("hello");

    CROW_ROUTE(app, "/")
        //.name("hello")
    ([&app](const crow::request& req, crow::response& res){
        auto cookie_context = app.get_context<crow::CookieParser>(req);
        cookie_context.set_cookie("123", "456");
        res.write("hello world!");
        res.end();
    });

And who can help me know how to get other middleware's context?

maybe you can use:
app.get_context(req)

I dont want to warm up old issues, but I also have trouble with creating an "working" cookie response. Using the middleware seems easy. Since the doku about this part is not done jet, i would hope that you could point my mistake out:
`int main()
{
//crow::SimpleApp app;
crow::Appcrow::CookieParser app;
CROW_ROUTE(app, "/")
([&app](const crow::request& req) {
auto ctx = app.get_contextcrow::CookieParser(req);
ctx.set_cookie("foo", "baa");
std::string c;
c = ctx.get_cookie("foo");

return "Hello world: " + foo;

});
app
.port(18080)
.multithreaded()
.r`

See this line:
    auto ctx = app.get_context<crow::CookieParser>(req);

By using auto ctx, you copied CookieParser::context and it's internal structures.
If you change it to auto& ctx, it will set cookie after the response.
There's still a problem: With the current implementation, get_cookie can't read set_cookied value. I don't know which one is better (can read / can't read)

Well at least I got this working atm. Thank you!

        std::string get_cookie(const std::string& key) const
        {   return get_cookie(key, true);    }

        std::string get_cookie(const std::string& key, const bool current) const
        {
            auto cookie = jar.find(key);
            if (cookie != jar.end())
                return cookie->second;

            if(current == false) return {};

            cookie = cookies_to_add.find(key);
            if (cookie != cookies_to_add.end())
                return cookie->second;

            return {};
        }

Will returns "unsend" cookies to other context requests. If this is what you had in mind.

im sure that not


demo: update crypto-perf Dockerfile

This change:

adds cloning and building of QAT-enabled DPDK to the Dockerfile
disables unnecessary compilation of DPDK kernel modules which depend on the host kernel version
fixes issue with missing ./resolv.conf file and dpdk directory

Codecov Report

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

@@          Coverage Diff           @@
##           master    #122   +/-   ##
======================================
  Coverage    79.7%   79.7%
======================================
  Files          10      10
  Lines        1212    1212
======================================
  Hits          966     966
  Misses        223     223
  Partials       23      23

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 60c36a7...fe7a4cd. Read the comment docs.

The image builds fine but running the sample application dpdk-test-crypto-perf fails. Do you think it could be because we don't build kernel modules, is there a way of accessing host kernel modules without elevating the pod privileges?

@swatisehgal could you post output with the error message? If the app doesn't start it may be either some configuration issue or missing libraries for specific crypto offloads like AESNI, which is not included in this image at the moment.
Also, not sure what do you mean by "accessing host kernel modules", but kernel and its modules are shared across all containers/pods, you're going to need privileges only for certain resources (like hugepages or uio devices).
General rule of thumb is that Docker images shouldn't contain anything host specific, because this will make them not portable. If we will build Docker image with DPDK modules on 4.4.14-200 kernel, these modules won't work with any other kernel version and your image will by unusable on any other host but the one you created it on. Also, loading kernel modules and binding devices to them should be done in the host context, not inside container, right?

after investigating the issue together with @swatisehgal, we changed the DPDK version - for some reason 18.08 didn't work, now verified commit will be used instead
beyond that removed some unnecessary packages installed with yum

@przemeklal I tested the latest Dockerfile and am able to to run the application now! Good Work! Can you please squash the commits and ensure that the Travis CI build doesn't fail so we can proceed to merge the PR.

Looks like the CI failure is not related to this PR.

sorry, my bad.

squashed and force-pushed as requested

/lgtm

@rojkov Is anyone looking into this CI issue? We would like to proceed to merge this PR

@swatisehgal I've just restarted the build. Let's hope it helps.

didn't help. I'll investigate further.

Should be fixed by PR #123, please review it.
I'll restart CI build for this PR when PR #123 is accepted.

@przemeklal please, rebase this PR on top of the latest master with the fix for CI issue. It should help to get it merged. Thanks.


Add MaxTerraformStateSizeBytes (max_terraform_state_size_bytes) settings support

The following PR adds support for the  MaxTerraformStateSizeBytes (max_terraform_state_size_bytes) setting.
Quote from GitLab Application settings API:

Maximum size in bytes of the Terraform state files. Set this to 0 for unlimited file size.

Cool, thanks a lot for the fast review and merge @svanharmelen


search doesn't match producer names

The search is only matching the beginning of strings right now. If you search for greymass, teamgreymass doesn't appear since it starts with team.
Searching should allow for more fuzzy matches to help find what voters are looking for.

I was unable to reproduce this in Windows. team returned teamgreymass as an option, as expected.

Yeah after further testing, this appears to be a false alarm. Not sure exactly what was happening when the user reported it was searching. There may be some sort of bug here, but not enough information to proceed.


install issues

when installing and using the user "filecoin" the cron job is installed with "filecoin" as user - even if that user has no write access to var/lib/prometheus/node-exporter
debian 10

Could you confirm if you install the prometheus-node-exporter before launching the install script or if you let the install script install it for you ?

i think i can confirm that. i first installed the dependencies. i will give it another shot since the calibration net seems to be up to date now.
but it shouldn't matter. a package installed by apt should'n be able to know who installed it.

that's correct. but the install script set the rights on the /var/lib/prometheus/node-exporter . So I think if it's failing it's because the rights are removed by node-exporter (like when it's running the first time or something)

You can try running the install script a second time, to see if it solves the problem

Not enough information to reproduce the bug. I modified the code. We will see if it corrects it.

Not enough information to reproduce the bug. I modified the code. We will see if it corrects it.


fix: ensure that semantic is passed a valid models.Ecosystem

Resolves #1115

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

Project coverage is 66.87%. Comparing base (9ab83d0) to head (19d6cb8).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1116      +/-   ##
==========================================
+ Coverage   65.99%   66.87%   +0.88%
==========================================
  Files         160      161       +1
  Lines       12910    10198    -2712
==========================================
- Hits         8520     6820    -1700
+ Misses       3917     2892    -1025
- Partials      473      486      +13

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


Change phone validations: Allow international phones, apply them everywhere.

Jira Ticket
These changes include:

Allow international phone numbers for Submit Users.
Only allow UK phone numbers for Search Users.
Bring validations to the models, not only on the forms.
Align validations with GovUK API phone validations.
Extra: Refactor locales & form validation errors.

We want to allow Submit users to use international phone numbers.
As we want to avoid allowing users to set a phone number that will be rejected later by GovUK Notify API when trying to send a 2FA SMS, we decided to align our service phone number validations with GovUK Notify ones.
By duplicating the validations in both the forms and model we avoid situations where an user that got its phone number set outside one of the validated forms, ends with an invalid phone that later is rejected by GovUK Notify API (this is currently happening for users who registered through "Complete Registration" page following an invitation)
Phone validations rules:

Phone is identified as UK phone if either:

Starts with "0" (but not "00").
Ignoring starting "0"s and removing special symbols (+, spaces, -, (, )):

Starts with UK prefix (44).
Starts with UK Mobile prefix(7) and its length is less than 11 digits.

UK Phone numbers must:

Start with UK Mobile prefix ("7")
Be 10 digits long.

Any phone number not identified as UK number is assumed to be an international phone number.

International phone numbers must:

Be between 8 and 15 digits length
Start with a prefix listed in the yaml file with the list of approved prefixes we introduced in the PR.

Extra work
During the development of this ticket noticed that in multiple forms we had to specify the scope/route for the validation errors from the translations in the YAML locales files.
This could be avoided by moving those errors into the correct YAML route/tree so the location can be automatically inferred by Rails and the error declaration becomes more succinct.
As I was already doing it for the forms with the 2FA mobile phone validations, decided to refactor this on the rest of the forms and keep them consistent with the refactor.

It appears that, for international numbers, it is stripping out + and 00 so I think I'm not receiving messages.
Eg. when user enters number 0091 or +91 (india code) then it should retain same format in db and when I click change number and see the pre-populated phone number field as 0091 or +91. This applies to all international numbers.

spotted one more issue, so in search domain registration journey,

enter valid UK number, fill. in other fields and submit form
On 2fa security code, click on code not received and change phone number, enter international number +918374303074
submit form, it returns 422 error
Expected to return validation error  - Enter valid phone number

Thanks for identifying these issues @nasirkhanbeis . Both issues have got addressed and covered with corresponding tests.

Hi @scruti  re-tested above issues, looking good as expected. thanks!
All good


! (not) operator overload for SortOptions

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

SortOptions type can be overloaded for ! operator to take reverse sort.
Describe the solution you'd like

Describe alternatives you've considered

Additional context

label_issue.py automatically added labels {'arrow'} from #3727


Uberbooty

Add Branding

Thank you for this incredible work, unfortunately, we're not ready for this type of rebrand right now.


Trade Station needs a ghost warp point

Description

I cant go check the state of the station without flying around until i find it or a cargo tech already being there, and it's kind of dumb to put a station beacon on the map so it's just more easily solved by adding a warp point kind of like how the nuke and nuke disks work (likely by attaching it to the station as a whole?)

just put a warp point on it thats why we kept the warp points

just put a warp point on it thats why we kept the warp points

oh

I guess i can do it later


postgres container doesn't start

Hi,
when trying a fresh installation (2.2.1) with docker-compose, the psql container fails to start:
$ docker ps
CONTAINER ID   IMAGE                             COMMAND                  CREATED          STATUS                          PORTS                                       NAMES
6cc186aa46bb   openbmp/postgres:2.2.0            "/docker-entrypoint.…"   18 minutes ago   Restarting (1) 51 seconds ago                                               obmp-psql
...

2022-07-12T10:51:27.874811111Z chmod: changing permissions of '/var/lib/postgresql/data': Operation not permitted
2022-07-12T10:51:27.894332284Z The files belonging to this database system will be owned by user "postgres".
2022-07-12T10:51:27.894345207Z This user must also own the server process.
2022-07-12T10:51:27.894348489Z
2022-07-12T10:51:27.894397094Z The database cluster will be initialized with locale "C.UTF-8".
2022-07-12T10:51:27.894411355Z The default database encoding has accordingly been set to "UTF8".
2022-07-12T10:51:27.894415012Z The default text search configuration will be set to "english".
2022-07-12T10:51:27.894417982Z
2022-07-12T10:51:27.894420545Z Data page checksums are disabled.
2022-07-12T10:51:27.894423148Z
2022-07-12T10:51:27.894481259Z initdb: error: could not change permissions of directory "/var/lib/postgresql/data": Operation not permitted
2022-07-12T10:51:27.932717657Z fixing permissions on existing directory /var/lib/postgresql/data ...

The same also happens if I comment out the volumes section in the docker.compose.yml
Any help/hint is appreciated.
Thanks
Stefan

Was this resolved?  Thanks a there a documentation update that would help?

Had to do a  chown 1000 ${OBMP_DATA_ROOT}/postgres/data/  on the docker host.


add note about custom plugins

Thanks for this. If you get some time, I'd welcome an update to the wiki to document how to do this with an example. :-)


Transparency issue with nucleus svg mask

Describe the bug
When using colors with some amount of transparency, the svg mask gives bad results for overlapping particles.
To Reproduce
Steps to reproduce the behavior:

Use transparency on nucleus particle fill and overlap some particles.

Expected behavior
The output should be symmetrical and not have a missing section. The particle border should still be masked, but not the particle fill.
Screenshots

Consumer now has ability to turn the mask off in scenarios like the one pictured above.

In cases where nucleus fill color is partially transparent nucleusMaskOverlap={false} may look better than the default setting:

vs


A deployment guide for Azure Web Apps

As per this discussion, it would be great to have a deployment guide for Azure Web Apps as well.

https://www.prisma.io/docs/guides/deployment/deployment-guides/deploying-to-azure-functions

From what I understand, Azure Web Apps use serverless functions
https://learn.microsoft.com/en-us/azure/app-service/overview

App Service is a higher level service that pulls in different other lower level services of Azure. My comment was regarding the link that @meletj posted to make sure it is clear that App Servicea and Azure Functions are different things.

Currently not planned, we might consider this in the future as we rethink our deployment guides.


Linux server will not start

`Found path: /RustDedicated
Mono path[0] = '/RustDedicated_Data/Managed'
Mono path[1] = '/RustDedicated_Data/Mono'
Mono config path = '/RustDedicated_Data/Mono/etc'
Initialize engine version: 5.4.2f2 (b7e030c65c9b)
Forcing GfxDevice: Null
NullGfxDevice:
Version:  NULL 1.0 [1.0]
Renderer: Null Device
Vendor:   Unity Technologies
AudioManager: Using Unknown: NoSound Driver
Begin MonoManager ReloadAssembly
Loading /RustDedicated_Data/Managed/UnityEngine.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Assembly-CSharp-firstpass.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Assembly-CSharp.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Assembly-UnityScript-firstpass.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/UnityEngine.UI.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/UnityEngine.Networking.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/UnityEngine.Analytics.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/ConvexHull.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/I18N.CJK.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/I18N.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/I18N.MidEast.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/I18N.Other.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/I18N.Rare.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/I18N.West.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Ionic.Zip.Reduced.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/LZ4.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Newtonsoft.Json.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Behave.Unity.Assets.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Behave.Unity.Runtime.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/protobuf-net.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/RustBuild.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/EasyAntiCheat.Client.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/EasyAntiCheat.Server.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Facepunch.Console.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Facepunch.Input.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Facepunch.Network.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Facepunch.Raknet.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Facepunch.Rcon.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Facepunch.System.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Facepunch.UnityEngine.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Rust.Data.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Rust.Global.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Rust.UI.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Rust.Workshop.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/websocket-sharp.dll into Unity Child Domain
Loading /RustDedicated_Data/Managed/Facepunch.Steamworks.dll into Unity Child Domain

Completed reload, in  0.050 seconds
OnLevelWasLoaded was found on LeanTween
This message has been deprecated and will be removed in a later version of Unity.
Add a delegate to SceneManager.sceneLoaded instead to get notifications after scene loading has completed
Shader 'Nature/Terrain/Diffuse': dependency 'AddPassShader' shader 'Hidden/TerrainEngine/Splatmap/Diffuse-AddPass' not found
Shader 'Hidden/TerrainEngine/Splatmap/Specular-AddPass': fallback shader 'Hidden/TerrainEngine/Splatmap/Diffuse-AddPass' not found
Shader 'Hidden/TerrainEngine/Splatmap/Standard-AddPass': fallback shader 'Hidden/TerrainEngine/Splatmap/Diffuse-AddPass' not found
UnloadTime: 0.925000 ms
Bootstart Startup
System
Name: 66
OS:   Linux 3.10 unknown 64bit
CPU
Memory: 152915 MB
GPU
API:    NULL 1.0 [1.0]
Memory: 128 MB
SM:     30
Memory
Private:     0 MB
Working Set: 0 MB
Process
Priority: 0
Threads:  0
Memory:   69 MB
Peak:     70 MB
Mono
Collects: 4
Memory:   0 MB
Command Line: "/RustDedicated" "-batchmode" "-nographics" "+server.ip" "<IP_ADDRESS>" "+server.port" "12024" "+server.tickrate" "30" "+server.maxplayers" "50" "+server.worldsize" "4000" "+rcon.ip" "<IP_ADDRESS>" "+rcon.port" "22025" "+rcon.password" "******" "-logfile"
server.ip: "<IP_ADDRESS>"
server.port: "12024"
server.tickrate: "30"
server.maxplayers: "50"
server.worldsize: "4000"
rcon.ip: "<IP_ADDRESS>"
rcon.port: "22025"
server.ip: "<IP_ADDRESS>"
server.port: "12024"
server.tickrate: "30"
server.maxplayers: "50"
server.worldsize: "4000"
rcon.ip: "<IP_ADDRESS>"
rcon.port: "22025"
UnauthorizedAccessException: Access to the path "/backup/1/my_server_identity/cfg/bans.cfg" is denied.
at System.IO.File.Delete (System.String path) [0x00000] in :0
at System.IO.Directory.RecursiveDelete (System.String path) [0x00000] in :0
at System.IO.Directory.RecursiveDelete (System.String path) [0x00000] in :0
at System.IO.Directory.Delete (System.String path, Boolean recursive) [0x00000] in :0
at System.IO.DirectoryInfo.Delete (Boolean recursive) [0x00000] in :0
at (wrapper remoting-invoke-with-check) System.IO.DirectoryInfo:Delete (bool)
at DirectoryEx.Backup (System.String[VMU] names) [0x00000] in :0
at ConVar.Server.backup () [0x00000] in :0
at ConsoleGen.m__1EE (.Arg arg) [0x00000] in :0
at ConsoleSystem+Run.Internal (.Arg arg, Boolean bWantReply) [0x00000] in :0
UnityEngine.DebugLogHandler:Internal_LogException(Exception, Object)
UnityEngine.DebugLogHandler:LogException(Exception, Object)
UnityEngine.Logger:LogException(Exception, Object)
UnityEngine.Debug:LogException(Exception)
Run:Internal(Arg, Boolean)
SystemRealm:Normal(RunOptions, String, String&, Object[VMU])
SystemRealm:Normal(RunOptions, String, Object[VMU])
SystemRealm:Normal(String, Object[VMU])
Bootstrap:Init_Config()
UnityEngine.SetupCoroutine:InvokeMoveNext(IEnumerator, IntPtr)
Server Config Loaded
Running server/my_server_identity/cfg/bans.cfg
Running server/my_server_identity/cfg/users.cfg
Unloading 4 Unused Serialized files (Serialized files now loaded: 72)
UnloadTime: 6.569000 ms
Floating point textures aren't supported on this device (Main Camera (UnityStandardAssets.CinematicEffects.TonemappingColorGrading))
[AmplifyOcclusion] Null graphics device detected. Skipping effect silently.
Unloading 49 unused Assets to reduce memory usage. Loaded Objects now: 53221.
Total: 28.255999 ms (FindLiveObjects: 3.408000 ms CreateObjectMapping: 2.037000 ms MarkObjects: 18.813999 ms  DeleteObjects: 3.996000 ms)
Loading Oxide Core v2.0.3024...
Loading extensions...
Loaded extension CSharp v1.0.3024 by Oxide Team
Loaded extension MySql v1.0.3024 by Oxide Team
Loaded extension Python v1.0.3024 by Oxide Team
Loaded extension SQLite v1.0.3024 by Oxide Team
Loaded extension Unity v1.0.3024 by Oxide Team
Loaded extension Rust v1.0.3024 by Oxide Team
Loaded extension Rust:IO for Oxide v2.9.0 by playrust.io / dcode
Using Covalence provider for game 'Rust'
Loading plugins...
Loading plugins Unity Core v1.0.3024 by Oxide Team
Loading plugins Rust v1.0.3024 by Oxide Team
Loading plugins Rust:IO for Oxide v2.9.0 by playrust.io / dcode
RCon Started on 22025
[3.7s] Procedural Objects
INFO [0.4s] Clutter Topology
INFO [4.2s] Procedural Generation
Destroyed 19 old entities
Spawning 10651 entities
AngryAnt Behave version 2.5 (d859964) - Copyright (C) Emil Johansen - AngryAnt 2015
Loading Complete
Enforcing SpawnPopMap Spawned Initializing 1065Initializing 10 stabFacepunch.Steamworks UnitFacepunch.SteamwFacepunch.SteamwDllNotFoundException: libat (wrapper managed-to-native) SteamNative.Platform/Linux64/Native:SteamInternal_GameServer_Init (uint,uint16,uint16,uintat SteamNative.Platform+Linux64.SteamApi_SteamInternal_GameServer_Init (UInt32 unIP, UInt16 usPort, UInt16 usGamePort, UInt16 usQueryPort, Int32 eServerMode, System.String pchVersionString) [0x00000] in <filenaat SteamNative.SteamApi.SteamInternal_GameServer_Init (UInt32 unIP, UInt16 usPort, UInt16 usGamePort, UInt16 usQueryPort, Int32 eServerMode, System.String pchVersionString) [0x00000] in <filenaat Facepunch.Steamworks.Interop.NativeInterface.InitServer (Facepunch.Steamworks.BaseSteamworks steamworks, UInt32 IpAddress, UInt16 usPort, UInt16 GamePort, UInt16 QueryPort, Int32 eServerMode, System.String pchVersionString) [0x00000] in <filenaat Facepunch.Steamworks.Server..ctor (UInt32 appId, UInt32 IpAddress, UInt16 SteamPort, UInt16 GamePort, UInt16 QueryPort, Boolean Secure, System.String VersionString) [0x00000] in <filenaat Facepunch.Steamworks.Server..ctor (UInt32 appId, UInt32 IpAddress, UInt16 GamePort, UInt16 QueryPort, Boolean Secure, System.String VersionString) [0x00000] in <filenaat Facepunch.Steamworks.Server..ctor (UInt32 appId, UInt32 IpAddress, UInt16 GamePort, Boolean Secure, System.String VersionString) [0x00000] in <filenaat ServerMgr.StartSteamServer () [0x00000] in <filenaat ServerMgr.OpenConnection () [0x00000] in <filenaat Bootstrap+c__IteratorF.MoveNext () [0x00000] in <filenaat UnityEngine.SetupCoroutine.InvokeMoveNext (IEnumerator enumerator, IntPtr returnValueAddress) [0x00000] in :0 `

Connecting anonymously to Steam Public...Logged in OK Waiting for license info...OK Update state (0x5) validating, progress: 0.07 (4198973 /<PHONE_NUMBER>) Update state (0x5) validating, progress: 4.25 (239079997 /<PHONE_NUMBER>) Update state (0x5) validating, progress: 13.86 (780145213 /<PHONE_NUMBER>) Update state (0x5) validating, progress: 21.53<PHONE_NUMBER> /<PHONE_NUMBER>) Update state (0x5) validating, progress: 30.06<PHONE_NUMBER> /<PHONE_NUMBER>) Update state (0x5) validating, progress: 39.00<PHONE_NUMBER> /<PHONE_NUMBER>) Update state (0x5) validating, progress: 48.69<PHONE_NUMBER> /<PHONE_NUMBER>) Update state (0x5) validating, progress: 58.15<PHONE_NUMBER> /<PHONE_NUMBER>) Update state (0x5) validating, progress: 67.22<PHONE_NUMBER> /<PHONE_NUMBER>) Update state (0x5) validating, progress: 73.41<PHONE_NUMBER> /<PHONE_NUMBER>) Update state (0x5) validating, progress: 79.48<PHONE_NUMBER> /<PHONE_NUMBER>) Update state (0x5) validating, progress: 86.41<PHONE_NUMBER> /<PHONE_NUMBER>) Update state (0x5) validating, progress: 95.94<PHONE_NUMBER> /<PHONE_NUMBER>) Success! App '258550' fully installed.

Please use our forums or Rust's forums for general server support.

@lukespragg http://oxidemod.org/threads/linux-server-will-not-start.23507/
I have already submitted,But still not resolved


Restore pdoc extension

Attempts to replace the propietary pdoc extension for
asciidoc with a standard library with built-in support
for asciidoc have not panned out.
This patch restores the use of the propietary pdoc
extension for generating the python-sdk documentation
Signed-off-by: Ori Liel<EMAIL_ADDRESS>

going with a different solution


rafactor(ci): Only run-tests job for PRs from a fork

We are encountering errors in our CICD when workflows are triggered from forked repositories e.g #438+. These workflows fail because of some of our jobs publish assets and require write permissions. This PR adds conditions to the workflows to only run the run-tests job when a PR is made from a fork.
Another option would be to create a branch specifically for pull requests from forks. If we decide to go that route, we would only need to add another workflow file with the following content:
on:
  pull_request:
    branches:
      - '[NEW_BRANCH_NAME_HERE]'

  run-tests:

    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
      - uses: actions/setup-node@v2
        with:
          node-version: 16.x

      - uses: denoland/setup-deno@v1
        with:
          deno-version: "1.18.2"

      - name: build binary and make executable
        env:
          TAQ_VERSION: ${{ github.ref_name }}
          DENO_DIR: "./deno"
        run: |
          COMMIT=`git rev-parse --short "$GITHUB_SHA"`
          deno compile --output taq --allow-run --allow-write --allow-read --allow-env index.ts --quickstart "`cat quickstart.md`" --setBuild "$TAQ_VERSION/$COMMIT" --setVersion "$TAQ_VERSION"
          [[ $(./taq init -p ./test_project) == "Project taq'ified!" ]]
          chmod +x taq
          echo "$(pwd)" >> $GITHUB_PATH
      ...

The current workflows could then remain as they are.
Testing plan

Only run-tests job for fork PRs
https://github.com/ecadlabs/taqueria/actions/runs/1959955142

@mweichert @alexzbusko could you please provide feedback on the proposed changes in the PR description?


feat(detected_object_validator): parameterize validator threshold for each label

Description
This PR add parameter for min_pointcloud_num in obstacle_pointcloud_based_validator which is hardcoded in validator node file. For small categories this value can be set to a smaller one than the current value of 10.
Tests performed
WIP
Notes for reviewers

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

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

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

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

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

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

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

https://github.com/autowarefoundation/autoware.universe/pull/2894


Use Google Reader API in Miniflux

Hi
Question about… For some time miniflux support Google Reader API as option, is it possible to use it currently with newsboat? for example pretending to be other service?
Google Reader in miniflux is using seperate user/password compared to miniflux own api

Sure it's possible, and your idea is correct — make Newsboat pretend that it's talking to a different service. Google Reader API was never properly documented, so every implementation possibly has some warts, but you could start with FeedHQ. Follow the docs, but in feedhq-url, specify your Miniflux URL.
Please tell me if this works!

Did it work, @Tokariew? Which remote API did you end up using?

@Minoru Sorry for delay… I didn't have time to test it properly…
I will check this later, because it seems that I can't make newsboat authenticate with existing sources in newsboat and url/login/password provided by miniflux

Did you manage to make it work @Tokariew?

I tried couple of times with all the remotes API on miniflux, but probably missing something :|
Sorry for not posting.

FeedHQ works for me with Newsboat from master and Miniflux 2.0.37. I followed these steps to configure Miniflux (I specified "googletest" both as a login and the password), and then I followed these steps to configure Newsboat like so:
urls-source "feedhq"
feedhq-url "localhost"
feedhq-login "googletest"
feedhq-password "googletest"

Can you please check if this exact configuration works for you? If it doesn't, please tell me what versions of Newsboat and Miniflux you are running, and provide the output from Newsboat.

@Tokariew: Looks similar to https://github.com/newsboat/newsboat/issues/1912#issuecomment-1002745325

@dennisschagt Tried manually downloading http://localhost:8080/reader/atom/feed/1?n=20 but got 404… not sure what url format is needed to download entries from feed on miniflux

Judging by https://github.com/miniflux/v2/blob/938ae6828ca4bcf7f0b427c6f9f2962d20f33613/googlereader/handler.go#L197, Miniflux simply doesn't implement the /reader/atom endpoint. They have /stream/items/contents instead, which returns specific items rather than an entire feed. (This seem to be a legitimate part of Google Reader API, see e.g. BazQux notes on it).
None of our code uses this other API endpoint, so one can't use Newsboat with Miniflux's Google Reader API.
@Tokariew, why not use Miniflux's native API?

I was hoping too use Google Reader Api to get enclosures from miniflux. Because to add them currently seems to require changes in miniflux and newsboat…

I see you're working on it already though: https://github.com/miniflux/v2/issues/1059 :rocket:


Initial commit of primary beam correction module

Please review the primary beam correction module.

The latest updates look good - there's just one comment that didn't get addressed.
Please also add the new script to the list of scripts in setup.cfg.
I'll let you and Kim decide whether to do unit tests as part of this PR or as a new one. It might be easier to split them if Kim wants to get going on using this code from katsdpcontim.

Thanks for putting up with the long review process Isaac (not helped by the holidays). Hit the merge button! Also it's then good practice to delete the branch (Github will give you a button for this after you've done the merge) so that we don't clutter things with old branches.

Thanks for putting up with the long review process Isaac (not helped by the holidays). Hit the merge button! Also it's then good practice to delete the branch (Github will give you a button for this after you've done the merge) so that we don't clutter things with old branches.


Added thread for message processing

I took the embedded state machine approach and made a state machine based thread that waits roughly 25ms or a message counter to reach a certain batch size for reducing the amount of dispatcher calls. I ran the tests and nothing broke 👍. It is noticeable faster than calling the dispatcher method for each message that is being logged. All suggestions are accepted.

Sure I'll make the adjustments. I maybe broke something because I used the vscode web version

Hi!  What's the status of this?  Really hoping for an update.

@augustoproiete - can you publish an updated nuget package I can use?

Hello, is there any new update on this?

When will this be committed?
@augustoproiete - If you're not able to maintain this project, can you make me an admin?  I have a vested interest in this being advanced.

@TonyValenti did you try the artifact nupkg? how did it work?

@TonyValenti See messages above - This needs manual testing, didn't hear back from you since Oct 18th https://github.com/serilog-contrib/serilog-sinks-richtextbox/pull/14#issuecomment-946244158

Thanks!  I'll grab the source and do a deep test later today/tomorrow.

@TonyValenti How did the test go? I imagine merging this will have an impact on https://github.com/serilog-contrib/serilog-sinks-richtextbox/pull/32 so might be good to get this one merged first and release a new preview

I wasn't able to figure out how to test it without it being merged. Please merge then I'll review and resolve any issues.
My changes on my PR are minimal to existing code and I'd be surprised if there were any issues that took longer than a second to resolve.

@TonyValenti Download the NuGet package is here (package artifact), unzip into a local feed/folder

This works great.

@Skaptor your changes have been merged, thanks for your contribution :+1:

@augustoproiete Happy to help. Thank you for this sink!


Update ns-winuser-pointer_touch_info.md

Simple number update.

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

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


Add conventions around PHP code in the project

The addition of test agains documentation gave me more ideas about things I could check with tests

Classes docblocks: every class / interface must have minimal description.
Public methods dockblocks: every public method must have minimal description.
Package dependencies usage and declaration: every package must declare explicitely the dependency they are using.
Source dependencies synchronisation: the main repository must be synchronized with all packages.

Yeah you're right @halundraN !
My point is : most of the time, a comment is very welcome, but in some very rare spots it is 100% useless.
This is why I rather want this kind of test, because I'm able to set my own rules to decide whether a comment is required here.
I've already removed non public methods, accessors, ...
But I was not able to add the exceptions for the places you spotted here.
So again, yeah this might be boring at some very particular classes/methods, but most of the time it is good to add.


Console log

Hi, My console does not show me the errors in my code anymore. can anyone tell me why?

Closing as duplicate
Please see http://forum.freecodecamp.com/t/creating-a-new-github-issue/18392
Happy Coding!


Easy to accidentally restart game on controller

Describe the bug
The restart game button can be accidentally triggered
To Reproduce

Player 1 hovers over the restart button.
Player 2 plays a move
Game is restarted

Expected behavior
Either ignore restart button when move is being played, or at least add a confirmation?
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: all with controller support

Additional context
Add any other context about the problem here.

resolved with f98760c