Typo in Snippet

@Html.DisplayFor(modelItem => item.CreatedDate)

Should be

@Html.DisplayFor(modelItem => item.Done)

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

ID: 48a90540-55d7-99ae-7a47-6cf8e526d631
Version Independent ID: 190e73c1-7711-05d2-afeb-5565d7a3dd37
Content: Build a .NET Core and SQL Database web app in Azure App Service on Linux
Content Source: articles/app-service/containers/tutorial-dotnetcore-sqldb-app.md
Service: app-service-web
GitHub Login: @cephalin
Microsoft Alias: cephalin

@rgrwatson85 Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@rgrwatson85 Thank you for taking the time to report it to us. I've made the update and it should be published in the next day.

#please-close


Pub.dev documentation at_lookup

Pub.dev content and (source)s - CHECKLIST OF TO DO's BELOW
All the content shown on pub.dev and where it comes from.
Example: Shelf (will point to one of our own once completed)

Package name, version <pubspec.yaml>

Published date (generated)

Platform support (generated based on project structure)

Tabbed navigation

Readme (note: initial content comes from readme.md)
Changelog  example: firebase_auth
Example (github repo)
Installing (generated)
Versions (generated) <pubspec.yaml>
Scores (generated)
**Admin  (generated)
**Activity log (generated)

Right column

Scores  (generated)
Publisher  (generated)
Metadata  <pubspec.yaml description>
Homepage <pubspec.yaml>
Repository (generated)
View/report issues (generated)
Documentation (generated)

Documentation url <pubspec.yaml>
API reference

License (github repo)
Dependencies (generated)
More (generated)

All items to be completed:

[x] Package should be implemented with rSDK
[ ] Include an example (sample) app based on at_app skeleton
[ ] Add example.md to explain how to run the sample app
[ ] Ensure the readme.md follows the best practice template (WIP)
[ ] Make sure and add in terms that will help get this package to the top of the search results
[x] Update changelog.md with clear description of the update
[x] Bump the version as appropriate in pubspec.yaml for the package
[ ] Bump the version as appropriate in pubspec.yaml for the example
[x] Check that the description is accurate in pubspec.yaml for the package
[x] As appropriate, add a documentation link to atsign.dev for the package

In atsign.dev, the link under at_lookup package redirects to at_lookup package in pub.dev. Adding the atsign.dev link below
https://atsign.dev/docs/functional_architecture/libraries/

PR for the changes: https://github.com/atsign-foundation/at_libraries/pull/125

Merged the document changes of at_lookup package to trunk branch.


tests: unit: util: Rework test structure for C++ testing

Rework test structure to handle C++ test in a better way. Similar approach already applied in other tests which are executed for C and C++. Getting rid of test.inc which due to non-standard extension was not covered by complience check.

Example of similar rework: #43791

It seems that checkpatch is expecting that FOR_EACH() macro is a conditional statement like for or switch. @keith-zephyr can you help?

checkpatch is confused because it doesn't recognize that FOR_EACH_NONEMPTY_TERM includes a terminating comma and wants the following lines indented as if was a line continuation.
Adding unnecessary indents stops checkpatch from complaining (but I'm not suggesting you do this)
	uint8_t array[] = {
		FOR_EACH_NONEMPTY_TERM(SQUARE, (,))
			FOR_EACH_NONEMPTY_TERM(SQUARE, (,),)
				255
	};

I think the checkpatch configuration is attempting to detect code blocks defined by some of the single/double link list macros.
The fix probably involves generating an allow  list of FOR_EACH macros that define a code block.  The alternative to create a list of FOR_EACH macros that should be ignored by this check.

@nashif @yperess @tejlmand can you take a look? @keith-zephyr can you confirm if checkpatch patch is ok?

@nordic-krch hey this needs a rebase

@nashif @keith-zephyr I had resolve conflicts can you re-approve?


Update embedded TDS

Task/Issue URL: https://app.asana.com/0/856498667320406/1201837636311069
Tech Design URL:
CC:
Description:
Update embedded TDS file, add tests and fix missing ETag when fetching config for the first time.
Steps to test this PR:
Test embedded list & etag.
Test configuration downloading and ETag usage.
General smoke tests.

Device Testing:

[ ] iPhone
[ ] iPad

OS Testing:

[ ] iOS 13
[ ] iOS 14
[ ] iOS 15

Internal references:
Software Engineering Expectations
Technical Design Template
When ready for review, remember to post the PR in MM

@brindy ready to look at!

I'm struggling to get the app to update the config.  Seems like it's not able to update tracker data or the bloom filter?  I've tried running as a background fetch and I've even done a clean install with the embedded etag and hash modified to make it look different.  Ping me when you're free and let's zoom it. :)

@brindy can this be a problem with debug screen? Maybe it doesn't show the embedded etag, but since it matches remote one nothing is stored?


Try not ignoring integration test packages

Which problem is this PR solving?

Description of the changes

How was this change tested?

Checklist

[ ] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
[ ] I have signed all commits
[ ] I have added unit tests for the new functionality
[ ] I have run lint and test steps successfully

for jaeger: make lint test
for jaeger-ui: npm run lint and npm run test

Bad idea, many code paths in integration tests are not executed unless there are failures. There's little value in writing coverage for those.


Improve CAFOS pairs matching

This improves astrometric calibration of CAFOS polarimetry images using the distance between pairs computed from already calibrated images.
The mean and std was computed by executing the following commands in the IOP4 IPython terminal:
In [1]: qs = ReducedFit.objects.filter(flags__has=ReducedFit.FLAGS.BUILT_REDUCED, instrument="CAFOS2.2").all()
In [2]: np.mean([redf.astrometry_info[-1]['seg_disp_sign'] for redf in qs[len(qs)-300:len(qs)-1]], axis=0)
In [3]: np.std([redf.astrometry_info[-1]['seg_disp_sign'] for redf in qs[len(qs)-300:len(qs)-1]], axis=0)

By using precomputed distances from already calibrated images, it increases the success rate for images with low number of stars where blind pair matching might fail.

The mean and std was computed by executing the following commands in the IOP4 IPython terminal:
In [1]: qs = ReducedFit.objects.filter(flags__has=ReducedFit.FLAGS.BUILT_REDUCED, instrument="CAFOS2.2").all()
In [2]: np.mean([redf.astrometry_info[-1]['seg_disp_sign'] for redf in qs[len(qs)-300:len(qs)-1]], axis=0)
In [3]: np.std([redf.astrometry_info[-1]['seg_disp_sign'] for redf in qs[len(qs)-300:len(qs)-1]], axis=0)

Do you think it is worth to keep this piece of information in the docs?

It will be one of the jupyter notebooks that I'm -slowly- writing for the docs.


Add option to decode CBOR bignum to interface{} as *big.Int

Closes #444
This PR adds decoding option BigIntDecMode which specifies how to decode CBOR bignum to Go interface{}.

BigIntDecodeValue (default) makes CBOR bignum decode to big.Int.
BigIntDecodePointer makes CBOR bignum decode to *big.Int.

Default behavior is unchanged to maintain backward compatibility.  However, using the new option is recommended as it improves speed, reduces memory use, and is more ergonomic.
Benchmark Comparison
                  │ bench_decode_to_bigint.txt │     bench_decode_to_pbigint.txt      │
                  │           sec/op           │    sec/op     vs base                │
DecodeToBigInt-12                  238.9n ± 8%   196.2n ± 10%  -17.87% (p=0.000 n=20)

                  │ bench_decode_to_bigint.txt │    bench_decode_to_pbigint.txt     │
                  │            B/op            │    B/op     vs base                │
DecodeToBigInt-12                   144.0 ± 0%   112.0 ± 0%  -22.22% (p=0.000 n=20)

                  │ bench_decode_to_bigint.txt │    bench_decode_to_pbigint.txt     │
                  │         allocs/op          │ allocs/op   vs base                │
DecodeToBigInt-12                   5.000 ± 0%   4.000 ± 0%  -20.00% (p=0.000 n=20)

Special Thanks
Many thanks to @extemporalgenome for opening issue #444!  :+1:

@x448 Thanks!  I added benchmark comparisons to the PR text.

Thank you!


internal/restorable: forcing restoring images doesn't work on browsers

Ebitengine Version
90562ee84b9bf20df9434887bc29d6a54bd15fde
Operating System

[ ] Windows
[ ] macOS
[ ] Linux
[ ] FreeBSD
[ ] OpenBSD
[ ] Android
[ ] iOS
[ ] Nintendo Switch
[ ] Xbox
[X] Web Browsers

Go Version (go version)
go version go1.20.4 darwin/amd64
What steps will reproduce the problem?

Apply this patch

--- a/internal/restorable/images.go
+++ b/internal/restorable/images.go
@@ -23,7 +23,7 @@ import (
 )

 // forceRestoring reports whether restoring forcely happens or not.
-var forceRestoring = false
+var forceRestoring = true

 var needsRestoringByGraphicsDriver bool

Run the 'block' example with wasmserve

go run github.com/hajimehoshi/wasmserve@lateset ./examples/blocks

What is the expected result?
Rendering succeeds
What happens instead?
Rendering fails
GL_INVALID_FRAMEBUFFER_OPERATION: Draw framebuffer is incomplete

Anything else you feel useful to add?
As long as restoring is not needed on browsers, this is not an actual serious issue. However, this might be a potential issue on Android, which uses restoring.
Other simple examples like rotate didn't cause this issue.

This patch worked
index 87304c47e..e8b4d09af 100644
--- a/internal/graphicsdriver/opengl/graphics.go
+++ b/internal/graphicsdriver/opengl/graphics.go
@@ -286,7 +286,7 @@ func (g *Graphics) SetVsyncEnabled(enabled bool) {
 func (g *Graphics) NeedsRestoring() bool {
        // Though it is possible to have a logic to restore the graphics data for GPU, do not use it for performance (#1603).
        if runtime.GOOS == "js" {
-               return false
+               return true
        }
        return g.context.ctx.IsES()
 }

So, probably only forcing restoring causes some problematic issues...


[MNT] Cleanup Deep Learning Models

Describe the issue
In many of our deep learning models, we have the keyword argument metrics = None. Then we use the following code:
if self.metrics is None:
            metrics = ["accuracy"]
else:
            metrics = self.metrics

Instead we could directly start with metrics = "accuracy" for brevity and remove the above given block.
Suggest a potential alternative/fix
Change as given above!
Additional context
No response

is anyone working on this?

metrics = "accuracy" seems more concise. But wouldn't it sacrifice the flexibility and customization options that the current implementation provides? @aadya940

Sure, I'd like to know the community's opinion on this. If @aadya940 's approach is more preferrable, can I take up this issue? :)

@MatthewMiddlehurst may I know your opinion on this?

@hadifawaz1999
metrics="accuracy" Seems better. It should be easy to check if the input is a string and turn it into a list. This may require editing the docstring to update the valid inputs (I assume str and list of str?).

@MatthewMiddlehurst as long as its always transformed to list inside the code than it should be good to go, good first issue

Thanks on your opinions. Would you assign this issue to me, I'd like to work in it :)


how to create and delete topic with kafka-rust

Looks like there is no way to do this with this crate.


Naming issue - OnDismissListener in Drawer.kt

https://github.com/microsoft/fluentui-android/blob/2f9491e8d98721a4672bf1e462b78fcbec979ce0/FluentUI/src/main/java/com/microsoft/fluentui/drawer/Drawer.kt#L41
fun onDrawerDismissListener() should be renamed to something like fun onDrawerDismissed()

closing this issue, we now have Drawer v2 available


Sending multiselect form values through to endpoint with Guzzle 7.4

Guzzle version(s) affected: 7.4.0
PHP version: 7.4.22 (hint: php --version)
cURL version: 7.78.0 (hint: php -i | grep cURL)
Description
I am using guzzlehttp/guzzle version 7.4.0 in my Symfony 4.4 project and I am attempting to form data to an end point with the following code:
$response = $this->client->request('POST', $url, [
   'form_params' => $params,
   'http_errors' => false,
   'timeout' => 5,
   'verify' => false,
]);

$params looks like this:
array:16 [▼
  "oid" => "00D751234508hOT"
  "retURL" => "http://crmpicco.co.uk"
  "first_name" => "Stefan"
  "last_name" => "Klos"
  "email" =><EMAIL_ADDRESS>  "phone" => "07720962787"
  "description" => "servicehelp"
  "company" => "vtech"
  "country" => "DE"
  "url" => "https://crmpicco-email-val.s.crmpicco.co.uk"
  "00N4P12345GUEH3" => "Marketing"
  "00N4P12345GUEH4" => "Corporate"
  "00N4P12345FpC5G" => "Less than 500"
  "00N4P12345Gm0Qn" => "1,001-10,000"
  "00N4P12345GUEGL" => "White Label"
  "00N4P12345GhXWd" => array:2 [▼
    0 => "Consulting"
    1 => "Hosting"
  ]
]

ALL of the other fields pull through excluding the "multiple" select (or multiselect) which has two values - Consulting and Hosting.
If I create a simple HTML form, save it as a .html file and submit the form to match my Symfony-built form then the data pulls through to the endpoint fine and when I inspect the data sent through in the browser's console I can see the structure looks like this:
{
...
    "00N4P12345GhXWd": [
        "Consulting",
        "Hosting"
    ],
...
}

Does Guzzle support a "multiselect" and if so how do I pass this through?

I don’t think we do, and i’m not sure if it’s actually supported by the RFC to repeat fields?

It feels like a hacky workaround, but this worked:
'00N4P12345GhXWd' => implode(';', $servicehelp)

That seams about right. Thank you


Improved iOS Xcode install/configuration instructions

For ppl struggling with XCode
"Add node_modules/react-native-material-kit/iOS/RCTMaterialKit.xcodeproj to your xcode project, usually under the Libraries group"
See the yellow folders in your project view to the top left. Find Libraries folder. Drag the mentioned .xcodeproj file there.
"Add libRCTMaterialKit.a (from Products under RCTMaterialKit.xcodeproj) to build target's Linked Frameworks and Libraries list"
Click your project (top left), then select general tab and find section Linked Frameworks and Libraries list at the bottom. Now find the RCTMaterialKit.xcodeproj you added and open it in the project view, then expand Products folder and find the file mentioned. Now drag it over into the list of linked libraries.
For me the file libRCTMaterialKit.a was red before I dragged it over and still red after.
Hope it still works!

As a developer coming from the android environment its still a bit unclear how to proceed.

install with NPM done.
install on the android app done.
install on IOS... its a bit unclear what you refer as the Library. folder.

The android instructions itself, I find are very clear to follow.
Please help me, by saying add this to this file and that to that file underneath this option.

Xcode is a bit different to Android Studio or Eclipse.
Libraries is something called 'Group' (kind of virtual folders), people usually manipulate them by drag & drop (instead of by editing the .xcodeproj file), if your project is created by react-native init, it should already be there:

you can add files or projects to the group by drag & drop or context menu, and then follow the awesome instructions above written by @kristianmandrup :beer:
Good luck!

Ah thanks, I started it with the android approach. That is why my experience with ios is minor. Gonna look up how to init correctly :).

Closing this for now. Would welcome PR for README


implement a checker into the Main.java class that will throw an exception if the default encoding is wrong

@nlchar how about we add a checker into our Main.java class, which will throw an exception if the default encoding is wrong? Something similar to this: https://github.com/yegor256/rultor/blob/master/src/main/java/com/rultor/web/TkApp.java#L86-L92
Originally posted by @yegor256 in https://github.com/cqfn/eo/issues/198#issuecomment-772223701

@IngeniariusSoftware

@nlchar let's not forget to add a unit test to validate that the checking works

@0crat in

@0crat in

Yegor, what does the command mean?
We planned that @ingeniariussoftware will make the task

Are you speaking to me or about me here; you must always start your message with my name if you want to address it to me, see §1

@yegor256/z everybody who has role DEV is banned at #201; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@yegor256/z everybody who has role DEV is banned at #201; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@IngeniariusSoftware leave your comment on the facts you've learned about the issue here, please.

Added to the current sprint. The assignee: @IngeniariusSoftware

@IngeniariusSoftware leave your comment on the facts you've learned about the issue here, please.
AFAIK, you found out that the encoding check in the Main.java is not needed. Can you write down the results of your investigations here, please? Do we really need the Main.java check? How can we solve the actual problem?

@nlchar/z I see that you unassigned this issue; the order is still assigned to @nlchar/z though; to cancel the order use refuse, as in §6

@nlchar/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@nlchar ok


project api test and bugfix

1.Add project API test
2. Bug fix: The type of public in project.go is fixed to int32

Coverage increased (+2.7%) to 29.992% when pulling 02d5281f0c605137e596f69d117cc148f481ca88 on zgdxiaoxiao:new-dev into 76cbd80ca600437ef4740951e0c60225ba93f327 on vmware:dev.


Fractal update to 0.13.* to aviod conflicts with other libraries

You can depend on Fractal 0.13.* yourself as this package should work with 0.12 and 0.13. Just add it to your composer.json.

@jasonlewis Actually it doesn't. I've already tried out myself and it's a strict dependency for fractal 0.12.*

Dev version doesn't have the strict dependency. I used a workaround when using the stable version:
"league/fractal": "0.13.0 as 0.12.0"


Modifying Scroll to Top button

Is your feature request related to a problem? Please describe.
It's not a problem, it's just I would like to improve the "scroll to top" button.  Currently, when someone clicks on it, the page reloads to the home page instead of sliding to the top. This action feels forceful rather than being convenient.
Describe the solution you'd like
My solution would be to work on its Javascript functionality and make it slide to the top instead of reloading the whole website. Also, I can change a bit of CSS to make it look better on hover.
Additional context
@Amit366 I would like to work on this improvisation issue as part of the Hacktoberfest challenge 2021. Any extra ideas on this solution are always welcomed.

@Maaz-Code u can work on it

@Maaz-Code r u working on it??


Q: More Details on 2 Way Audio

Default Home Assistant lovelace cards don't support 2-way audio. You can use 2-way audio from Add-on Web UI. But you need use HTTPS to access the microphone. This is a browser restriction and cannot be avoided.

Can you please provide a configuration example of using 2 way audio in Home Assistant? Also clarification on the statement about needing https to access the microphone please?
I have several cameras that are 2 way audio capable but have not been able to use this feature. Looking forward to experimenting!
Thank you!

There is no advanced config. You just need to meet the conditions:

camera support ONVIF Profile T
you open stream from go2rtc web interface
you use HTTPS to access to go2rtc web interface via Hass add-on or Ngrok or Nginx, etc

PS. You may run go2rtc on your PC and open web interface via http://localhost:1984/. This is also will work

I will give it a try following the comments you made. THANK YOU


Uncaught Error: ENOENT, node_modules/keybinding-resolver/keymaps/keybinding-resolver.json not found in /usr/share/atom/resources/app.asar

[Enter steps to reproduce below:]

...
...

Atom Version: 0.194.0
System: linux 3.11.10-29-desktop
Thrown From: Atom Core
Stack Trace
Uncaught Error: ENOENT, node_modules/keybinding-resolver/keymaps/keybinding-resolver.json not found in /usr/share/atom/resources/app.asar
At /usr/share/atom/resources/app.asar/src/workspace.js:457

Error: ENOENT, node_modules/keybinding-resolver/keymaps/keybinding-resolver.json not found in /usr/share/atom/resources/app.asar
    at createNotFoundError (ATOM_SHELL_ASAR.js:101:13)
    at Object.module.(anonymous function) [as open] (ATOM_SHELL_ASAR.js:156:25)
    at ReadStream.open (fs.js:1668:6)
    at new ReadStream (fs.js:1655:10)
    at Object.fs.createReadStream (fs.js:1610:10)
    at File.module.exports.File.read (/usr/share/atom/resources/app.asar/node_modules/pathwatcher/lib/file.js:286:27)
    at TextBuffer.module.exports.TextBuffer.updateCachedDiskContents (/usr/share/atom/resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:948:61)
    at TextBuffer.module.exports.TextBuffer.load (/usr/share/atom/resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:974:19)
    at Project.module.exports.Project.buildBuffer (/usr/share/atom/resources/app.asar/src/project.js:446:21)
    at Project.module.exports.Project.bufferForPath (/usr/share/atom/resources/app.asar/src/project.js:416:63)

Commands
  3x -1:02.2.0 editor:copy-path (atom-text-editor.editor.is-focused)
  2x -1:00.8.0 editor:toggle-line-comments (atom-text-editor.editor.is-focused)
     -0:09.3.0 key-binding-resolver:toggle (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "themes": [
      "atom-dark-ui",
      "atom-dark-syntax"
    ],
    "autoHideMenuBar": true
  },
  "editor": {
    "invisibles": {
      "eol": "\\n"
    },
    "showInvisibles": true,
    "softWrap": true,
    "fontSize": 13
  }
}

Installed Packages
# User
No installed packages

# Dev
No dev packages

Duplicate of #6511 which will be fixed in the next Atom version, 0.195.0.


Todo:

hi thank you for this awesome project, however there are some bugs:

luci kmod dashboard doesnt match with background, needs fix and scrollbar on left is missing, it would be awesome if you develop a seperate bg manager like luci theme argon:
https://github.com/jerrykuku/luci-theme-argon

because the luci-app-dashboard css is different / has not entered the luci-theme-alpha css, it cannot be set or does not adjust to the theme. later it will be fixed by us. thank you for your support.

If this issue is resolved, it's better to just close it.
CC @derisamedia @AmirulAndalib


Test Quarkus S3 extension with Camel Quarkus AWS 2 S3

Fixes #2590 and replace #3107

I'll rebase this PR while #3136 get merged.


Release GHA is not working…

Checklist

[x] I did check there are no other issues relating to this.
[x] I am running a supported version.
[x] Setupr version given by pynpc --version: NA
[x] Python version given by python --version: NA

What are the steps to reproduce it?

Start GHA
It fails.

What did you expect to happen?
The action should have created a (patch) release.
What actually happened?
Logs
Any more details?
Semantic python issue

Fixed.


GEM051 - Presence of geometric context (IVS-43)

I completely agree that the bareminimum should be that there should be:
1. A minimum of at least one contex

@aothms
I was already wondering if there would be a rule that simple in IFC.
Schema-wide applicability are added in https://github.com/buildingSMART/ifc-gherkin-rules/pull/226/commits/d727319a4afad6c630fa9c28625379f2817dc43e

regarding shall be IfcGeometricRepresentationContext it can also be its subtype so inheritance should be consider, just not its supertype IfcRepresentationContext which is the default type of the attribute.

I think this is covered by
Its entity type is 'IfcGeometricRepresentationContext including subtypes'
IfcRepresentationContext here is False, IfcGeometricRepresentationSubContext is True. Added another testfile to check this
https://github.com/buildingSMART/ifc-gherkin-rules/pull/226/commits/aaf71fe7d758b8c1bac93c2889fd9a77d10eb6cd


Fix generic code hit by doctests not correctly remapped

cargo-llvm-cov currently uses --remap-path-prefix internally (not sure for the reason why?).
However that option is not recognized by rustdoc at the moment, which means that generic code that is instantiated in doctests is not counted towards the coverage, as it ends up in a "different" file as far as llvm tools are concerned.
I think in order to fix this, the flag needs to be supported in rustdoc as well, and I am exploring that option right now, so stay tuned.
relates to #2
I opened https://github.com/rust-lang/rust/pull/92648 for the necessary changes in rustdoc, with which I generated the test snapshots in this PR, however I would like to somehow increase the confidence here a bit. Also I have seen that there is some code that does feature detection, and it might be a good idea to do that for this flag as well (and gate the test on the flag being available). Can you maybe point me in the right direction how to best do that?

cargo-llvm-cov currently uses --remap-path-prefix internally (not sure for the reason why?).

IIRC, this was needed because sometimes macros are displayed with absolute path in HTML reports.
https://github.com/taiki-e/cargo-llvm-cov/blob/30efe472911693470cca4a8491ac3abea73ac423/src/main.rs#L205-L206
I guess the underlying bug has not been fixed yet, but currently the ignore_filename_regex excludes a few more directories, so I don't think those will displayed in the report anyway.

Are you suggesting the flag might not be necessary after all?

Are you suggesting the flag might not be necessary after all?

Yes

uff… I tried to do that, but now all the snapshot tests have randomized paths in them. I give up 😓

(I'm starting to think to provide an option to disable --remap-path-prefix: https://github.com/taiki-e/cargo-llvm-cov/issues/140#issuecomment-1055457353)

Closing as https://github.com/rust-lang/rust/pull/92648 has been closed.
Also, --remap-path-prefix is optional since cargo-llvm-cov 0.3 (https://github.com/taiki-e/cargo-llvm-cov/pull/141), so the problem may no longer be reproducible without the --remap-path-prefix option.
Thanks anyway for the PR!


Feature Request:  Support app.bsky.feed.getSuggestedFeeds

Packages/Tools

[ ] at_identifier
[ ] nsid
[ ] at_uri
[ ] xrpc
[ ] multiformats
[ ] atproto_core
[ ] atproto
[x] bluesky
[ ] bluesky_text
[ ] bluesky_cli
[ ] bluesky_post
[ ] others

Proposal

#731

Risks

@all-contributors please add @gildaswise for code, test, idea

@all-contributors please add @gildaswise for code, test, idea, content


Speedtest fails to setup on my system

Home Assistant release with the issue:

0.81.0.dev0
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):

cat /proc/version
Linux version 4.15.0-34-generic (buildd@lgw01-amd64-047) (gcc version 7.3.0 (Ubuntu 7.3.0-16ubuntu3)) #37-Ubuntu SMP Mon Aug 27 15:21:48 UTC 2018

Component/platform:

https://www.home-assistant.io/components/sensor.speedtest/
Description of problem:
The platform can not be setup without the "day" config
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
- platform: speedtest
  monitored_conditions:
    - ping
    - download
    - upload

Traceback (if applicable):
Traceback (most recent call last):
  File "/home/knutas/src/home-assistant/env/lib/python3.6/site-packages/homeassistant-0.81.0.dev0-py3.6.egg/homeassistant/helpers/entity_platform.py", line 128, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT, loop=hass.loop)
  File "/usr/lib/python3.6/asyncio/tasks.py", line 358, in wait_for
    return fut.result()
  File "/usr/lib/python3.6/concurrent/futures/thread.py", line 56, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/home/knutas/src/home-assistant/env/lib/python3.6/site-packages/homeassistant-0.81.0.dev0-py3.6.egg/homeassistant/components/sensor/speedtest.py", line 65, in setup_platform
    data = SpeedtestData(hass, config)
  File "/home/knutas/src/home-assistant/env/lib/python3.6/site-packages/homeassistant-0.81.0.dev0-py3.6.egg/homeassistant/components/sensor/speedtest.py", line 160, in __init__
    day=config.get(CONF_DAY))
  File "/home/knutas/src/home-assistant/env/lib/python3.6/site-packages/homeassistant-0.81.0.dev0-py3.6.egg/homeassistant/helpers/event.py", line 28, in factory
    hass.loop, ft.partial(async_factory, *args, **kwargs)).result()
  File "/usr/lib/python3.6/concurrent/futures/_base.py", line 432, in result
    return self.__get_result()
  File "/usr/lib/python3.6/concurrent/futures/_base.py", line 384, in __get_result
    raise self._exception
  File "/home/knutas/src/home-assistant/env/lib/python3.6/site-packages/homeassistant-0.81.0.dev0-py3.6.egg/homeassistant/util/async_.py", line 195, in run_callback
    future.set_result(callback(*args))
TypeError: async_track_time_change() got an unexpected keyword argument 'day'

Additional information:

Cc @OttoWinter

Oh yes, that's a bug. I was looking for all uses of the base method, but forgot to look for the uses of each base method too 🤔
The problem is that day wil be a bit hard to implement, and would complicate future efforts to introduce day_of_week triggers. It's still possible, and I will evaluate if this day_of_month option will be easy to re-implement as a fix.
On the other hand though, I can't imagine many people are using the day option... Why would you run a speed test only on specific days of the month. I mean only running speed tests on some days of the week would kinda still make sense, but day of month seems a bit odd to me. Seems like an option that was just introduced "because you can" (here)
What would you think about just removing that option as a breaking change? I mean users can still manually trigger the speedtest with automations if they really need to.

@OttoWinter I think that would be fine, it actually seems like the entire timer configuration in the speedtest sensor could just be delegated to automations.

I agree, who would ever want to have a test a specific day every month?


Add Source Filtering to Suggest API

https://www.elastic.co/guide/en/elasticsearch/reference/current/search-suggesters-completion.html#querying

Source Filtering has not been implemented yet on _suggest endpoint: https://github.com/elastic/elasticsearch/issues/20482
For the meantime, this can be worked around by using suggestions on the _search endpoint, which does support source filtering.

Closing this as _suggest endpoint will not have source filtering implemented since there are plans to deprecate it in the near future and remove in a future major release.
The document has been updated as part of https://github.com/elastic/elasticsearch/pull/21268 to reflect this.


No nodes are displayed if any are unreachable

If any of my hosts are unreachable the plugin seems to dislike the non-zero exit code returned by Ansible and won't show any hosts in the nodes section. If I change my hosts file so only the online hosts are in inventory, it works as expected.
PLAY RECAP *********************************************************************
SERVER-01 : ok=3    changed=2    unreachable=0    failed=0
SERVER-02 : ok=2    changed=1    unreachable=0    failed=0
SERVER-03 : ok=0    changed=0    unreachable=1    failed=0
SERVER-04 : ok=0    changed=0    unreachable=1    failed=0
SERVER-05 : ok=0    changed=0    unreachable=1    failed=0
SERVER-06 : ok=2    changed=1    unreachable=0    failed=0
SERVER-07 : ok=0    changed=0    unreachable=1    failed=0
SERVER-08 : ok=2    changed=1    unreachable=0    failed=0
SERVER-09 : ok=2    changed=1    unreachable=0    failed=0
SERVER-10 : ok=0    changed=0    unreachable=1    failed=0
SERVER-11 : ok=0    changed=0    unreachable=1    failed=0
SERVER-12 : ok=0    changed=0    unreachable=1    failed=0
SERVER-13 : ok=0    changed=0    unreachable=1    failed=0
SERVER-14 : ok=0    changed=0    unreachable=1    failed=0
SERVER-15 : ok=0    changed=0    unreachable=1    failed=0
SERVER-16 : ok=0    changed=0    unreachable=1    failed=0
SERVER-17 : ok=2    changed=1    unreachable=0    failed=0
SERVER-18 : ok=2    changed=1    unreachable=0    failed=0
SERVER-19 : ok=2    changed=1    unreachable=0    failed=0
SERVER-20 : ok=2    changed=1    unreachable=0    failed=0
SERVER-21 : ok=2    changed=1    unreachable=0    failed=0
SERVER-22 : ok=0    changed=0    unreachable=1    failed=0
SERVER-23 : ok=0    changed=0    unreachable=1    failed=0
SERVER-24 : ok=0    changed=0    unreachable=1    failed=0
SERVER-25 : ok=2    changed=1    unreachable=0    failed=0
SERVER-26 : ok=0    changed=0    unreachable=1    failed=0
DATABASE-19 : ok=0    changed=0    unreachable=1    failed=0
DATABASE-20 : ok=0    changed=0    unreachable=1    failed=0
DATABASE-21 : ok=0    changed=0    unreachable=1    failed=0
DATABASE-22 : ok=0    changed=0    unreachable=1    failed=0
DATABASE-23 : ok=0    changed=0    unreachable=1    failed=0
DATABASE-24 : ok=0    changed=0    unreachable=1    failed=0
DATABASE-25 : ok=0    changed=0    unreachable=1    failed=0

ERROR ExceptionCatchingResourceModelSource: [ResourceModelSource: 2.source (Ansible Resource Model Source), project: Ansible-test]
com.dtolabs.rundeck.core.resources.ResourceModelSourceException: Error running playbook.
        at com.batix.rundeck.AnsibleResourceModelSource.getNodes(AnsibleResourceModelSource.java:72)
        at com.dtolabs.rundeck.core.resources.ExceptionCatchingResourceModelSource.getNodes(ExceptionCatchingResourceModelSource.java:57)
        at com.dtolabs.rundeck.core.common.ProjectNodeSupport.getNodeSet(ProjectNodeSupport.java:92)
        at com.dtolabs.rundeck.core.common.ProjectNodeSupport$ProjectNodesSource.getNodes(ProjectNodeSupport.java:292)
        at com.dtolabs.rundeck.core.resources.ExceptionCatchingResourceModelSource.getNodes(ExceptionCatchingResourceModelSource.java:57)
        at com.dtolabs.rundeck.core.resources.ResourceModelSource$getNodes.call(Unknown Source)
        at rundeck.services.nodes.CachedProjectNodes.reloadNodeSet(CachedProjectNodes.groovy:26)
        at rundeck.services.nodes.CachedProjectNodes$reloadNodeSet.call(Unknown Source)
        at rundeck.services.NodeService$_loadNodes_closure3.doCall(NodeService.groovy:232)
        at rundeck.services.NodeService$_loadNodes_closure3.doCall(NodeService.groovy)
        at sun.reflect.GeneratedMethodAccessor820.invoke(Unknown Source)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:90)
        at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:324)
        at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1207)
        at groovy.lang.ExpandoMetaClass.invokeMethod(ExpandoMetaClass.java:1110)
        at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1016)
        at groovy.lang.Closure.call(Closure.java:423)
        at groovy.lang.Closure.call(Closure.java:417)
        at com.codahale.metrics.Timer.time(Timer.java:99)
        at com.codahale.metrics.Timer$time.call(Unknown Source)
        at org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCall(CallSiteArray.java:45)
        at com.codahale.metrics.Timer$time.call(Unknown Source)
        at MetricswebGrailsPlugin$_addDynamicMetricMethods_closure26.doCall(MetricswebGrailsPlugin.groovy:174)
        at sun.reflect.GeneratedMethodAccessor516.invoke(Unknown Source)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:90)
        at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:324)
        at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1207)
        at groovy.lang.ExpandoMetaClass.invokeMethod(ExpandoMetaClass.java:1110)
        at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1016)
        at groovy.lang.Closure.call(Closure.java:423)
        at org.codehaus.groovy.runtime.metaclass.ClosureStaticMetaMethod.invoke(ClosureStaticMetaMethod.java:59)
        at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:324)
        at org.codehaus.groovy.runtime.callsite.PogoMetaMethodSite$PogoMetaMethodSiteNoUnwrap.invoke(PogoMetaMethodSite.java:230)
        at org.codehaus.groovy.runtime.callsite.PogoMetaMethodSite.call(PogoMetaMethodSite.java:68)
        at org.codehaus.groovy.runtime.callsite.AbstractCallSite.call(AbstractCallSite.java:124)
        at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callSafe(AbstractCallSite.java:96)
        at rundeck.services.NodeService.loadNodes(NodeService.groovy:244)
        at sun.reflect.GeneratedMethodAccessor813.invoke(Unknown Source)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:90)
        at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:324)
        at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1207)
        at groovy.lang.ExpandoMetaClass.invokeMethod(ExpandoMetaClass.java:1110)
        at org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnCurrentN(ScriptBytecodeAdapter.java:80)
        at rundeck.services.NodeService.this$dist$invoke$1(NodeService.groovy)
        at rundeck.services.NodeService$2.methodMissing(NodeService.groovy)
        at sun.reflect.GeneratedMethodAccessor741.invoke(Unknown Source)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:90)
        at groovy.lang.MetaClassImpl.invokeMissingMethod(MetaClassImpl.java:933)
        at groovy.lang.MetaClassImpl.invokePropertyOrMissing(MetaClassImpl.java:1256)
        at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1209)
        at groovy.lang.ExpandoMetaClass.invokeMethod(ExpandoMetaClass.java:1110)
        at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1016)
        at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:807)
        at rundeck.services.NodeService$2.invokeMethod(NodeService.groovy)
        at groovy.lang.MetaClassImpl.invokeMethodOnGroovyObject(MetaClassImpl.java:1272)
        at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1166)
        at groovy.lang.ExpandoMetaClass.invokeMethod(ExpandoMetaClass.java:1110)
        at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1016)
        at org.codehaus.groovy.runtime.callsite.PogoMetaClassSite.callCurrent(PogoMetaClassSite.java:66)
        at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callCurrent(AbstractCallSite.java:145)
        at rundeck.services.NodeService$_2_reload_closure1.doCall(NodeService.groovy:96)
        at rundeck.services.NodeService$_2_reload_closure1.doCall(NodeService.groovy)
        at sun.reflect.GeneratedMethodAccessor946.invoke(Unknown Source)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:90)
        at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:324)
        at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1207)
        at groovy.lang.ExpandoMetaClass.invokeMethod(ExpandoMetaClass.java:1110)
        at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1016)
        at groovy.lang.Closure.call(Closure.java:423)
        at groovy.lang.Closure.call(Closure.java:417)
        at java.util.concurrent.FutureTask.run(FutureTask.java:262)
        at org.springframework.core.task.SimpleAsyncTaskExecutor$ConcurrencyThrottlingRunnable.run(SimpleAsyncTaskExecutor.java:251)
        at java.lang.Thread.run(Thread.java:745)
Caused by: com.batix.rundeck.AnsibleStepException: ERROR: Ansible execution returned with non zero code.
        at com.batix.rundeck.AnsibleRunner.run(AnsibleRunner.java:298)
        at com.batix.rundeck.AnsibleResourceModelSource.getNodes(AnsibleResourceModelSource.java:70)
        ... 82 more
WARN  LoggingResourceModelSourceCache: [ResourceModelSource: 2.source (Ansible Resource Model Source), project: Ansible-test] Returning cached model data

and with the edited inventory:
PLAY RECAP *********************************************************************
SERVER-01 : ok=3    changed=2    unreachable=0    failed=0
SERVER-02 : ok=2    changed=1    unreachable=0    failed=0
SERVER-06 : ok=2    changed=1    unreachable=0    failed=0
SERVER-08 : ok=2    changed=1    unreachable=0    failed=0
SERVER-09 : ok=2    changed=1    unreachable=0    failed=0
SERVER-17 : ok=2    changed=1    unreachable=0    failed=0
SERVER-18 : ok=2    changed=1    unreachable=0    failed=0
SERVER-19 : ok=2    changed=1    unreachable=0    failed=0
SERVER-20 : ok=2    changed=1    unreachable=0    failed=0
SERVER-21 : ok=2    changed=1    unreachable=0    failed=0

INFO  ExecutionUtilService: Execution successful: 41466

Hi @cwhits ,
Thanks for reporting this, I agree it is probably better if the host facts gathering omits bad hosts and still display good ones. Although errors due to unreachable hosts should not go silently. So probably it will be best if this still works and report errors at the same time, not sure if this can be done thought. I will investigate a nice way of doing it.

Perhaps something with error handling within the playbook?
http://docs.ansible.com/ansible/playbooks_error_handling.html

@yassineazzouz Is it better if the host facts gathering omits the bad ones? Wouldn't we want all of our inventory to reside under nodes within Rundeck and then let Ansible return the unreachable host error when we try and execute a command against that host? Or, if we're gathering facts, add a fact/tag to that host saying it's offline?
It already appears that if a host is imported (while online), it won't be removed later if it goes offline because of the crash then the non-zero exit code is returned.

This also causes problem in updating the Resource model in Rundeck. If you provision new instances and some of your hosts are stop then it won't update the new instance to Rundeck resource model.

This constant rebuilding of nodes list is really really annoying. It makes it impossible to use ansible plugins for rundeck if I have some jobs thats going on while I'm working with other runbooks, for instance when having servers rebooting.
Perhaps it could be possible to disable this auto generation some how, and only enable import from /etc/ansible/hosts when we explicitly need to rebuild the node list or catch new facts?

+1. It should be possible to add reachable hosts and delete(ignore) unreachable/failed ones.
Regards

Confirmed, I'm having the exact same problem.

Thanks for reporting this, Coming in next release :)

Hi @QUASARFREAK @javierurien @cwhits @rubal033 @telefax ,
This is resolved in release 2.0.0, please refer to the readme file for the full list of new features.
Thanks,

Hi
I have tried release 2.0.0 and its not working fine with "Gather Facts" enabled.
If i enable gather facts checkbox it takes forever to update the nodes and never display it. In service logs i can see all the processing is getting complete but nodes won't get updated and Rundeck keeps showing "Loading matched nodes..." message.

Hi @rubal033 ,
Please note that old configurations does not work on v2.0, so you need to create a new project after setting the new plugin jar. you maybe have some hosts unreachable, are you using "Ignore Host Discovery Errors" ?
Thanks,

Yes i have created a new project after installing the new release and yes "Ignore Host Discovery Errors" is also checked. It works fine if i don't use "Gather Facts" . But if use it then it won't show the nodes and i have to delete the project and create new to get the nodes without "Gather Facts".
Also while saving the project settings with "Gather Facts" enabled it takes lot of time to save , sometimes even i got session out during that time. And after re-login if i try to open the project setting it takes lot of time to open it. In service logs i am not getting any logs related to this.
Thanks

Hi @rubal033 , which rundeck version are you using ? also can you post the exact ansible command that is being executed ?

Hi @yassineazzouz
I am using Rundeck 2.6.8-1 . I am not executing any command, just trying to get Nodes by using "Ansible Resource Model" and stuck there only when the "Gather facts" is checked.
Right now not even able to open the setting for the project i am trying to populate nodes with.
In service logs i got logs as:
PLAY [all] *********************************************************************
TASK [file] ********************************************************************
changed: [tag-population-001 -> localhost] => {"changed": true, "gid": 1007, "group": "rundeck", "mode": "0775", "owner": "rundeck", "path": "/tmp/rundeck/ansible-hosts8305448749762189786/data", "size": 4096, "state": "directory", "uid": 1007}
TASK [template] ****************************************************************
changed: [broadcast-consumer-preemptive-ig-qzat -> localhost] => {"changed": true, "checksum": "c5859cac8a8ffc492edbd5659047768ba7429e23", "dest": "/tmp/rundeck/ansible-hosts8305448749762189786/data/broadcast-consumer-preemptive-ig-qzat", "gid": 1007, "group": "rundeck", "md5sum": "01238ec0c68c19da1844d83f3ab834b6", "mode": "0664", "owner": "rundeck", "size": 58222, "src": "/home/rundeck/.ansible/tmp/ansible-tmp-1476190652.53-239703006392661/source", "state": "file", "uid": 1007}
changed: [tag-population-001 -> localhost] => {"changed": true, "checksum": "5a1c37e556feea42461009f5a85e04be5aa7a691", "dest": "/tmp/rundeck/ansible-hosts8305448749762189786/data/tag-population-001", "gid": 1007, "group": "rundeck", "md5sum": "1d18e3a70046b8f3b990ab69c0757901", "mode": "0664", "owner": "rundeck", "size": 57776, "src": "/home/rundeck/.ansible/tmp/ansible-tmp-1476190652.47-111114993632118/source", "state": "file", "uid": 1007}
I have attached 2 screenshots:

Setting i am using with "Ansible Resource Model".
The status of my Nodes since last 1 hr.

With the 2.0 version it worked here, thanks!

Hi @rubal033
I don't think this is related to the plugin itself or rundeck I tested with the same rundeck version and it works perfectly fine for me. I think the ansible process is hanging, this could be related to multiple things :
ansible could be waiting for an authentication password

Host checking enabled
Hosts not responding
What I typically do is enable ansible logging in rundeck to look at what ansible is doing in the background, you can do so, by setting (uncommanting):
log_path = /var/log/ansible.log
in /etc/ansible/ansible.cfg, make sure the file is writable by rundeck, you can juste put it to 0666 for testing.
then look at the logs in /var/log/ansible.log to see why the process is hanging. If you could try the command that is being executed and post the full output I can probably tell more what is going on.
A good idea will be also to check the running ansible processes, I have seen some cases in v1.x where rundeck leaves a lot of orphans processes, and that was fixed in v2.0 so I suggest you try :
ps -eaf | grep -i ansible (then kill any running process)
Another good practice is to add : export ANSIBLE_HOST_KEY_CHECKING=False in your /etc/rundeck/profile to avoid problems related to host key checking.

I hope that helps,

Hi
Getting my nodes populated with info. Looks like adding  export ANSIBLE_HOST_KEY_CHECKING=False in  /etc/rundeck/profile worked for me.
It's not showing me all the nodes as while running playbook and i can see in logs those nodes showing they are unreachable and its look like ansible issue.
Plugin is working fine for me.
Thanks You

Thanks @rubal033 , glad it works for you, you can also try to register known hosts fingerprint, using "rundeck" user to avoid such issues, disabling host key checking is not safe. You need just to su - rundeck then do ssh to all hosts to make sure that all hosts fingerprint are registered.
I usually automate this process in ansible, you can just include this script in your playbooks are just execute it whenever you add a new host:

hosts: all
gather_facts: False
serial: 1
tasks:

name: Make sure the known hosts file exists
file: path={{ ssh_known_hosts_file }} state=file
delegate_to: <IP_ADDRESS>
run_once: true
name: Scan the current public key of the server
command: "ssh-keyscan -T 10 {{ inventory_hostname }}"
changed_when: False
register: current_host_key
delegate_to: <IP_ADDRESS>
name: Register the current public key
lineinfile: >
dest="{{ ssh_known_hosts_file }}"
regexp="^(.){{ inventory_hostname }}(.)$"
line="{{current_host_key.stdout}}"
delegate_to: <IP_ADDRESS>
vars:
ssh_known_hosts_file: "~/.ssh/known_hosts"`

I hope that helps,

Hi
One more problem i am facing with using "gather facts" is that  Ansible skip some node to identify saying they are unreachable. But on running ping or setup on those nodes manually from Ansible always give me success and desired response.
Is there any specific Ansible setting need to be done for the plugin ?

Hi @rubal033 ,
If your ansible commands are working fine with "rundeck" user the plugin should be working fine, it's just a wrapper. I suggest you enable file logging on your /etc/ansible/ansible.cfg ; just set : log_path = /var/log/ansible.log.
Then post the command being executed  for host discovery and the logs. then try to run the exact same command (with same options) just replace ansible-playbook with ansible all -m ping and keep all the other options and post the output.
Thanks,

Hi @yassineazzouz
i have enabled the  log_path = /var/log/ansible.log and got the attached output for the gathering hosts. I have then filtered unreachable host from that output and  ran the ping module for those filtered hosts and the output of that also attached with mail. All the running nodes responded with success to ping module.
Thanks
playbook-output.txt
ping_output.txt

Hi
This is resolved after making some changes to the ansible conf. related to ssh_arg.
Thanks


The search queue size default value of the config files is wrong

Describe the bug
The default value of the config file is set to 0 instead of 1000.
Initial discovery was made in https://github.com/meilisearch/meilisearch/issues/4991
Even though the bug is on an experimental feature flag, it actually affects all instances 😬
To Reproduce
Steps to reproduce the behavior:

Starts meilisearch with an empty config file

Expected behavior
A clear and concise description of what you expected to happen.
Meilisearch version:
v1.9 to v1.11

Closed by #4992


[9.4.x] ISPN-9786 Update Prometheus JMX agent configuration pattern

Update Prometheus metrics format for 9.4.x branch

Just waiting for CI

@ryanemerson this one came clean ;-)

@$@#$ it did not make it in.


Checking whether a molecule has a specific ring-size

Configuration:

RDKit Version:
Operating system: 2018.09.3
Python version (if relevant): CentOS Linux 7 (Core)
Are you using condo? 2.7.14
If you are using conda, which channel did you install the rdkit from? conda-forge
If you are not using conda: how did you install the RDKit? -

Description:

Is there any function that allows finding specific ring sizes? Let's say, I want to check if there is a 14-membered ring, and if so, it will print it.

Your code sample here

Assuming that you're interested in any ring and not just members of the smallest set of smallest rings, there's nothing built in. It's easy to do that search though:
In [2]: p = Chem.MolFromSmarts('*~1'+'~*'*12+'~*~1')

In [3]: print(Chem.MolFromSmiles('C1'+'C'*12+'C1').GetSubstructMatches(p))
((0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13),)

In [4]: print(Chem.MolFromSmiles('C1'+'C'*11+'C1').GetSubstructMatches(p))
()


Add Synapse Spark Authentication Option

Add option to authenticate from Synapse Spark using mssparkutils. This is required since the azure-identity library does not work in Azure Synapse. Microsoft Fabric Spark may have a similar limitation.
Reference: https://github.com/Azure/azure-sdk-for-python/issues/26997

Just sharing here some manual tests from Synapse Spark:

@marvinbuss , Have you tried this in both Fabric and Synapse workspace notebooks?

Note that mssparkutils library is not available in pypi to add it as a dependency. The dbt-fabric adapter is missing the packege dependency. Here is an example.

@prdpsvs I have only validated Synapse as for Fabric there is no DW supported yet for the token generation as documented here: https://learn.microsoft.com/en-us/fabric/data-engineering/microsoft-spark-utilities#get-token

Note that mssparkutils library is not available in pypi to add it as a dependency. The dbt-fabric adapter is missing the packege dependency. Here is an example.

There is only a dummy library supported as documented here: https://learn.microsoft.com/en-us/azure/synapse-analytics/spark/microsoft-spark-utilities?pivots=programming-language-python#package-dependencies
Would you like me to add that to the requirements-dev.txt?

@prdpsvs  Please let me know how to proceed here. We can also have a quick sync to discuss how we can integrate the proposed solution. Thanks!

Closing this PR as I raised another one.


plugins.flutter.io/camera

I want to work on Camera Plugin.
Can you advice on current roadblocks, so help me work out the work required.
Saw this but not sure.
https://github.com/go-flutter-desktop/go-flutter/issues/134
Event Channel would be needed?
its getting close: https://github.com/go-flutter-desktop/go-flutter/pull/178

Event Channel is about to be merged, some works still needs to be made on the flutter/engine side.
What's missing is the texture API. Once the texture API is complete, we will begin the development such plugins

Progress on #102 has been made.
The Texture api is in beta release (use hover run -b '@feature/gl_texture' to test it)
The texture_tutorial has been updated with a video example.
For a cross-platform webcam go library, https://github.com/hybridgroup/gocv is your best bet!
@joeblew99 if you want to start to work on the camera plugin, the texture tutorial with the video example is a great start!

thanks @Drakirus  fantastic support.
will try to get something working

Hi, is there any update to this?

@joeblew99 hasn't reported back so I don't think he has made progress on this.
As for go-flutter-desktop maintainers, it's not manageable to implement all plugins by ourselves, we believe the community should help us when it comes to implementing plug-in.
If you are interested in continuing, please check the texture_tutorial (as it is very closed to this demand) our wiki and our plugins repo to found code-samples/documentation/guide on how to implement plugins in general. We are also available on our gitter.im, where you can questions.

the example above is for displaying the content I think, is there any example of getting raw data from devices/hardware?

In my above comment I referenced gocv

Hey all
We are working on it along with @cloudwebrtc
It's working on Mac, iOS and Android.
Getting it working on windows and Linux will require more cgo work.
It can also do s teen capture.
Almost working in flutter web

Also check out the ion project !.this is the golang server side parts to allow video streaming and pub sub.
It's part of the webrtc pion project. Really awesome stuff

Awesome, don't hesitate to submit a PR on https://github.com/go-flutter-desktop/plugins once you have something working!

@joeblew99 can I have a preview? Working on Mac is enough for me. I have a project using tflite_native for object detection, and is demonstrating how flutter and tflite can be truly cross-platform.


fix: user edit

Editing a user without changing their password would fail with the error "Property [password] does not exist on this collection instance". This change uses the $record for the user as provided by filament instead of fetching the user based on form input data, since the user will never be found if you search for it in the DB by the new name which was not yet persisted.
Partially resolves #4

thanks for your support @kolaente


请问为什么仅仅在三个特征图上做检测，而不是原论文中的5个？

RT

A good result is obtained on three feature map.It's also to be consistent with the official version of mxnet.

官方源码发布的resnet50权重也仅仅是3个分支


[Functions] Updating function wipes out userConfig

There's a bug that's existed for a while in Pulsar where a Pulsar Function's userConfig is wiped out if it's not provided when a function is updated.
To reproduce:

Create a Pulsar function with a userConfig. For example:

bin/pulsar-admin functions create --tenant myTenant --namespace myNamespace --name myFunction --inputs persistent://myTenant/myNamespace/input-topic --output persistent://myTenant/myNamespace/output-topic --jar http://repository.xyz.com/content/jar-with-dependencies.jar --classname com.company.pulsar.ExampleFunction --userConfig '{"prop1":"val1","prop2":"val2"}';

Update the function without providing the existing userConfig:

bin/pulsar-admin functions update --tenant myTenant --namespace myNamespace --name myFunction --parallelism 5

Get the function:

bin/pulsar-admin functions get --tenant myTenant --namespace myNamespace --name myFunction
Notice that userConfig is now empty.

Feel free to assign this one to me. I think I know what needs to be done to fix it.

Looks like this was already fixed by https://github.com/apache/pulsar/pull/10732


Integrate tag list

Codecov Report

Merging #166 (259615f) into master (7c5924b) will decrease coverage by 1.59%.
The diff coverage is 2.35%.

 Learn more


Fix warnings and errors with setup

Fix man files not being included.
Fix deprecation warnings related to licenses field and
packages.
Fix git revision not being found correctly because setup.py
is executed in a tmp directory. Instead find the revision
in the makefile and pass it into setup.
Topic: fixes
Reviewers: brian-k, aaron

Reviews in this chain:
└https://github.com/Skydio/revup/pull/49 Fix warnings and errors with setup

#
head
base
diff
date
summary

0
157725ea
07b7b91f
diff
Sep 19 1:31 PM
3 files changed, 16 insertions(+), 30 deletions(-)

GitHub Actions, nice - I guess I'll wait to review until that's passing?

I have no idea why but when I try to make this patch
`diff --git a/setup.cfg b/setup.cfg
index 8e637fd..bbd693f 100644
--- a/setup.cfg
+++ b/setup.cfg
@@ -34,10 +34,11 @@ classifiers =
[options]
package_dir =
= .
packages =
revup

revup/man1
python_requires = >=3.8
install_requires =
aiohttp
rich
setup_requires =
`

despite everything working locally, CI fails at the pip install . step with "package directory 'revup/man1' does not exist". I guess i'll put this off since it was only handling a warning anyway


is it possible to set the sound using specified music or mp3

is it possible to set the sound using specified music or mp3

Most phones you can change the default alarm ringtone in the system settings or using the "Clock" app


Get rid of origin if possible - less config, easier data migration, moving server between domains

Atomic Data uses a lot of URLs. The server needs to be aware where it is hosted in order to provide valid JSON-AD.
However, this poses a problem: Changes to port or schema (using HTTPS or HTTP) or domain causes identities to no longer resolve.
This requires users to migrate their data manually (e.g. export JSON-AD, rename all domains).
It would probably better to use a different type of ID internally. For example, instead of storing local identifiers as https://domain/ we do something likelocal:.

Whenever we serialise any resource, we need to convert it using the server_url.
We need to consider all the places where we use these URLs:

Properties
Values

I like the idea of introducing atomic:// URL scheme and use current tauri app to register atomic:// schema locally, with an atomic server internally translating URLs to relevant FQDN + path.
To illustrate our discussion check out how ripgrep handles URLs schemas for different text editors: Printer


Replace sqlalchemy-trino with trino[sqlalchemy]

Affected module
Ingestion: trino client is out of date
** Context **
trino client (https://github.com/trinodb/trino-python-client) is now (and since v0.307.0) releasing with built-in SQLAlchemy driver
See sqlalchemy-trino v0.4.1 release: https://github.com/dungdm93/sqlalchemy-trino/releases/tag/v0.4.1
Describe the bug
sqlalchemy-trino is now locked on an older version (v0.306.0) of trino client (which is missing some needed fields, eg. http_session).
trino[sqlalchemy] should be used instead of sqlalchemy-trino
Expected behavior
replace sqlalchemy-trino with trino[sqlalchemy]: https://github.com/open-metadata/OpenMetadata/blob/3b7b65d46cdeff62c478a9ea4ac3734a8b4da365/ingestion/setup.py#L105
Version:

OpenMetadata version: 0.9.0
OpenMetadata Ingestion package version: [e.g. openmetadata-ingestion[trino]==0.9.0]

May I be assigned to this? Thanks

Closing this issue as PR #3800 is merged.


Clarification: use of Main Axis Alignment inside a Single Child Scroll View

Hello there,
I stumbled upon the following inconsistency: say you have to code a simple Log-In form.
My log-in form is composed of some Images and an Expanded widget whose child is a Form.
I want such form to have several fields, so I'm using another Column with MainAxisAlignment.center inside it, which works perfectly fine (i.e. as intended: everything is centered).
Now, I want such Column to be scrollable, and here's where the problems occur.
Whenever such Column is wrapped inside a SingleChildScrollView, the Column widget is shrinked down to the bare minimum. The Expanded option is not ignored (that's good), but instead the _SingleChildViewport is the Widget actually expanding on the UI (that's bad: I expect the actual column to expand).
Therefore, the MainAxisAlignment.center is now useless, since now all the content is "compressed" up to the top; I tried a workaround with the MainAxisSize.max option, but it is simply left ignored.
On the internet I found some clever workarounds, but it really doesn't feel right or at least consistent to code like that.
Example Code

This should suffice as an example.
Column(
      mainAxisAlignment: MainAxisAlignment.spaceEvenly,
      crossAxisAlignment: CrossAxisAlignment.center,
      children: [
        const SizedBox(height: 50),  // some other widgets, decorations, etc.
        Expanded(  // the Form i need
          child: SingleChildScrollView(
            child: Column(
              crossAxisAlignment: CrossAxisAlignment.center,
              mainAxisAlignment: MainAxisAlignment.center,  // USELESS
              mainAxisSize: MainAxisSize.max,  // IGNORED
              children: [
                const Text("Some label"),
                TextField(...),
                const SizedBox(height: 5),
                const Text("Some other label"),
                TextField(...),
                ElevatedButton(
                  onPressed: doStuff(),
                  child: const Text(
                    "......",
                  ),
                ),
              ],
            ),
          ),
        ),
        const SizedBox(height: 50),  // some more decorations...
      ],
    )

Expected results:
A evenly-distributed form
Actual results:
A compressed-to-the-top form.
Workaround Code
      mainAxisAlignment: MainAxisAlignment.spaceEvenly,
      crossAxisAlignment: CrossAxisAlignment.stretch,
      children: [
        const SizedBox(height: 50),
        Expanded(
          child: Center(   // WORKAROUND
            child: SingleChildScrollView(
              child: Column(
                crossAxisAlignment: CrossAxisAlignment.center,
                mainAxisAlignment: MainAxisAlignment.center,
                children: [
                  const Text("Some label"),
                  TextField(...),
                  const SizedBox(height: 5),
                  const Text("Some other label"),
                  TextField(...),
                  ElevatedButton(
                    onPressed: doStuff(),
                    child: const Text(
                      "......",
                    ),
                  ),
                ],
              ),
            ),
          ),
        ),
        const SizedBox(height: 50),
      ],
    );

While using the Flutter DevTools, You can see that the _SingleChildViewport disappears as it is incorporated in the Center widget. I can't really understand why or how this should work.

@luca-venir
Check this documentation on usage of SingleChildScrollView with a Column and see if it helps to understand the behavior.
Also, your question is better suited for support channels like StackOverflow as you are seeking more of a clarification on the usage of your implementation, and doesn't sound like a bug, so I suggest you to please ask this question there.
If you think there's a gap in the documentation or need more explaination on it's usage, write in comments and I'll reopen it.
Closing for now.


Allow outputing input tank from Vat.

I accidentally clicked the vat with my portable tank. And it filled up the left tank.
An easy way to solve this is a button to allow switching tanks like IC2 does.

That's what those "empty tank" buttons are for.

I think we can all agree that wasting 8 buckets of rocket fuel is not a trivial issue. The issue here is that there is no way to choose what tank a right click goes into.

EnderIO features this nifty Orange and Blue In and Outputs, you could replace the "kill" button with a color switcher :) so when you click the button, the vat changes the in and output tank and visualizes it with the button color

As one tank is an input and one tank is the output, the logic is quite clear. I see no difference here to e.g. throwing your lone Ender Pearl into the SAG Mill instead of the Alloy Smelter.
But if we want to add a solution, a button to drain the input tank into the output tank would be enough. The output tank can be pulled from. It could even replace the left "dump" button, as that functionality would then be possible by "drain"+"dump output".

@HenryLoenwind that would work :smile:. 8B may not be much. But I wasn't really happy when I clicked the clear tank button.

Something I really liked about the IC2 solid/fluid canning machine was the button that swapped the input and output tank contents. Would that feature be applicable for The Vat as well?


Linux. GLFW Crash after terminating the Godot Project

➜  glfw git:(master) ✗ scons platform=linux bits=64
scons: Reading SConscript files ...
scons: done reading SConscript files.
scons: Building targets ...
scons: project/gdnative/linuxbsd/libglfw_godot.so is up to date.
scons: done building targets.

➜  glfw git:(master) ✗ Godot_v3.2.2-stable_x11.64 --path project
Godot Engine v3.2.2.stable.official - https://godotengine.org
OpenGL ES 2.0 Renderer: GeForce GTX 1060/PCIe/SSE2
OpenGL ES 2.0 Batching: ON

GLFW._init()
create window: 400x400
user says: "please close."
glfw_close
glfw_close
glfw_destructor
handle_crash: Program crashed with signal 11
Dumping the backtrace. Please include this when reporting the bug on https://github.com/godotengine/godot/issues
[1] /usr/lib/libc.so.6(+0x3d6a0) [0x7ff6a3d916a0] (??:0)
[2] /usr/lib/libpthread.so.0(pthread_mutex_lock+0x4) [0x7ff6a3a02664] (??:0)
[3] /usr/lib/libX11.so.6(XrmDestroyDatabase+0x2f) [0x7ff6a3c31d0f] (??:0)
[4] /usr/lib/libX11.so.6(_XFreeDisplayStructure+0x3bc) [0x7ff6a3c18a2c] (??:0)
[5] /usr/lib/libX11.so.6(XCloseDisplay+0xd3) [0x7ff6a3c06313] (??:0)
[6] Godot_v3.2.2-stable_x11.64() [0x29746fe] (<artificial>:?)
[7] Godot_v3.2.2-stable_x11.64() [0x2a5a792] (<artificial>:?)
[8] Godot_v3.2.2-stable_x11.64() [0x8582ff] (??:?)
[9] /usr/lib/libc.so.6(__libc_start_main+0xf2) [0x7ff6a3d7c152] (??:0)
[10] Godot_v3.2.2-stable_x11.64() [0x86695e] (??:?)
-- END OF BACKTRACE --
[1]    83286 abort (core dumped)  Godot_v3.2.2-stable_x11.64 --path project

I'm on Arch linux 64.
If I get rid of the GLFW code, I wont get the crash dump like this.

@dave0null Please upload a minimal reproduction project to make this easier to troubleshoot.

It's exactly the GLFW project under the C folder in the gdnative demos folder. You don't need to change anything. Other than running it under Linux

@dave0null Would you be willing to attempt to fix this bug? I'm not familiar with GLFW myself.


SSE does not work again...

What version of Hono are you using?
latest
What runtime/platform is your app running on? (with version if possible)
node
What steps can reproduce the bug?
The sse does not work again in the latest version...
I feel super frustrated after spending 1 hour and not understanding what was happening with my code, when I moved to the version "@hono/node-server": "1.8.2" and  "hono": "4.1.1", it works correctly, I have already fixed this same error in the past here https://github.com/honojs/hono/commit/b827efb3fb3fa988e74a5b96d1cfcd75d415f2af
What is the expected behavior?
SSE should works
What do you see instead?

Additional information

Hi @damianpumar
Can you fix it? I can't understand that behavior only with your information.

Yes, I can fix it again, but it was fixed by me some month ago...
Basically, using the latest version, the SSE server does not work...


I get error that says I need to define the access_token and access_token_secret when using user auth

So I am building  react application that allows users to login using twitter auth from firebase and once they login it displays their user timeline. So I am trying to use this npm package to be able to give me the user's twitter username which I will pass into another component to retrieve the user timeline to be  displayed.
I made sure to install the twitter-v2 package, and I set up my client variable with all the needed keys. Right now I am trying to test it out to see if it works by retrieveing tweets with a tweets id:

But then I get this weird issue:

I also made sure this app is inside a approved project allowed to use the v2 endpoint and I have the correct keys to be able to pass in. I was wondering do you guys know how I would be able to fix this issue? Thank you all for the help I really appreciate it!

This error is shown when the client hasn't been correctly constructed with Twitter's required API keys. Could you share the code where you construct the client so that we can dig deeper?

The image above shows how I imported the twitter v2 package into the React Component. And below shows how I set the client

Then I'd place the appropriate keys where I put those placeholder values for each key. And then I called the get request for the tweets and it led to that error I got. Is the way I am setting things in the client wrong, possibly because of the way I'm importing the package or is it something else?

This is now fixed in version 0.1.4 (see #22)
Please update to the new version and let me know if anything else goes awry.

Thank you so much! That fixed the issue and I have been able to access the API!


Load culled tiles with low priorty

Fixes #4894
What is happening in #4894 is basically a failure of spatial coherence.  We use a tile's bounding volume for hierarchical culling, but the bounding volume is computed from the vertices of the tile.  So there is no guarantee that:

The children of the tile, or
The real tile geometry once it is loaded

are contained within the tile's bounding volume.  In the particular case in #4894, a clear cache resulted in an upsampled tile with vertices (and hence bounding volume) far above the real terrain surface at that location.  The upsampled geometry was deemed to be outside the view frustum and culled, and as a result the real geometry was never loaded.
The solution implemented here is to make sure that culled tiles are still loaded with low priority so that the mistake is eventually rectified.  A much better solution (but orders of magnitude more work) would be to fix the spatial coherence.

I just checked this for our code and it this solved our problem. Thanks.
Hmm, if i understood it correctly we could also solve this in the  converter if we change the quantized mesh and propagate the minimumHeight and maximumHeight from the lowest terrain level to the hightest terrain level.
So that a child tile is always inside the boundingvolume of the parent tile ?
A high level tile would than potentially have a flat terrain at height 0m, but the minHeight and maxHeight in the tile would be something like 0m and 1200m depending on the children.
Would this be correct ?

change the quantized mesh and propagate the minimumHeight and maximumHeight from the lowest terrain level to the hightest terrain level.

You should absolutely do this.  The minimumHeight and maximumHeight properties are meant to capture the "real" (i.e. most detailed) min/max in the area covered by the tile.  STK Terrain Server (and therefore STK World Terrain) already does this.
It won't fix the problem described here, though, because Cesium still computes bounding volumes from the vertices instead of from the min/max heights.

For typical views (top down, horizon, 45 degree), how many more tiles does this load?

because Cesium still computes bounding volumes from the vertices instead of from the min/max heights.

Is it reasonable to use the min/max heights?

A much better solution (but orders of magnitude more work) would be to fix the spatial coherence.

What else is required other than fixing the bounding volumes to always include all the children's geometry?  Isn't this possible when a tile has the min min and max max heights for all of its descendants?

For typical views (top down, horizon, 45 degree), how many more tiles does this load?

I didn't measure, but it doesn't really matter, because:

It's more important that it work than that it be fast, and it currently is pretty broken.
The new tiles are loaded with low priority, which means that under normal circumstances (i.e. when we don't see this bug), the new tiles will be loaded after all the important ones are loaded.  So it won't affect the user's perception of the performance even if it does mean a few more tiles are loaded in the background.

Is it reasonable to use the min/max heights?
What else is required other than fixing the bounding volumes to always include all the children's geometry? Isn't this possible when a tile has the min min and max max heights for all of its descendants?

I'm not sure.  I plan to investigate this more when I get a chance, but it seemed important to fix the bug first.

Given that we've only ran into this problem once in practice, I am hesitant to merge this without knowing the before/after numbers.  We could leave this branch open until we know the impact or have a better fix.  If the impact is small, merging this now would be fine.

I've been running into this issue more lately.  It can most easily be reduced by entering 110 45 in the geocoder, which produces really bad culling issues in master but are fixed in this branch.  I'm all for improving correctness first (since there are no obvious performance issues), but I'll let @pjcozzi decide when to merge since he has concerns.

Quick and dirty test:
Camera sandcastle, terrain on, starting from home and clicking "Fly in a City":
master:

323 .terrain requests
1.4MB

missingTiles:

334 requests
1.5MB

I didn't test frame rates but didn't notice any issues (not that there'd be any).

@denverpierce your initial numbers are promising, thanks for checking.
I would still like someone to test a few common cases: top-down, 45 degree, horizon, flight with throttling on to be sure here.  This has become standard practice for changes with a potential core performance impact.


extending afalg with aes-cbc-192/256

Hi,
Please consider this changeset which is the result of comments from pull request 4448.
These changes extend support for aes-cbc-192 and aes-cbc-256 in afalg.
Thanks
Jitendra

I'm wondering if this couldn't be tidied up by having an array of 3 EVP_CIPHER structures and retrieving all the necessary parameters from them instead of hard coding them in multiple functions.

Please refer the following which takes care of the comments from Stephen.
https://github.com/openssl/openssl/pull/4484


"usesExpansionFile = true" doesn't work on CoronaBuilder

"usesExpansionFile = true" doesn't work on CoronaBuilder.
Simulator build works.

As I understand it, CoronaBuillder and Native builds ignore build.settings, therefore there's no place that we can request usesExpansionFile = true. Would it be possible to add this as an option to the CoronaBuilder params.lua configuration file?
@Shchvova

Corona builder does use build.settings to the full extend. There's a bug somewhere. Also, you can use build.settings in native as well to set up plugins with gradle targets in Corona group.

@Shchvova Thanks for you reply. Another thing is that "neverStripDebugInfo = true" doesn't work on CoronaBuilder either.
I wonder if there is more options in build.settings that don't work on CoronaBuilder.

@Shchvova
We're now using CoronaBuilder to automate builds from the command line, and these two issues are blockers for us. Is there any way we can help fix this?

Thanks @Shchvova, we'll test from our end.


Mesh trait

Turning Mesh into a trait would provide several benefits:

can be implemented automatically for Buffer<T>, where T: VertexFormat, since our buffer info has the size, so we can use it as the number of vertices
allows mesh proxies without re-allocation of attributes on the heap. For example, an instanced mesh can be a proxy of the per-vertex mesh and per-instance mesh data.

Heck yeah. This sounds like a good idea.

Would it be possible to implement it for Vec?

Not sure about Vec, actually. A Mesh will need to return some attributes, which point to buffers. Thus, it will not be able to create/update buffers on the fly with Vec.


Reviewer: Installs / theblockstats.com

What is the problem you are seeing? Please describe.
Apps are rewarded that are not used by many users
How is this problem misaligned with goals of app mining?
This discourage other app publishers to develop high quality apps
What is the explicit recommendation you’re looking to propose?
Add an app reviewer that analyzes and rates the usage of apps.
Additional context
http://theblockstats.com/ shows number of users for multi-player apps
Projects about decentralized analytics were mentioned

Examples:
authDomain | App Mining Rank | number of multi-players
https://publishing.graphitedocs.com | 10 | 19
https://xordrive.io | 12 | 25
Gladys | 4 | I guess very few.

@friedger I think the number of mulit-players you mentioned for Graphite Publishing and Xordrive are incorrect, from what I can see in http://theblockstats.com/

Also, Gladys uses single-player auth, so new sign ins won't show up here. The 4 is probably just from when the developer was testing it.

@hstove @muneebm thank you for the verification.
There is indeed no data for Gladys. However, there is only a potential of 35k users altogether vs 4billion on the web. Furthermore, the advantage of blockstack for Gladys users is not immediate, only in case they want to reset or replace a device. This is very comparable to OI ConvertCSV while OI ConvertCSV has a potential of  100k+ users (depending on what you count..)

I'm not sure how single player apps will perform here. eg SpringRole has atleast 1100 blockstack connects but we will never show up in blockstats because it can only track multi-player ones.

@kar2905 I don't promote using the multi-player data. I am very much for privacy and very much like apps that do not publish in the profile.
This is a place to discuss the aspect of app usage in the app mining algorithms.

@friedger yet your proposal only discusses a stat mechanism that excludes those apps you praise for privacy.  -1 for theblockstats being mentioned anywhere near app-mining algorithms.
It seems like the Awario proposal would be a reasonable stand-in for real-life user tracking.

Just updated ticket title to reflect a proposal about http://theblockstats.com/ specifically. There are a few other potential methods but they will likely require PBC build some software. Would be great to get a discussion going specifically re installs and if we can overcome the single player challenge. cc @GinaAbrams

I don't think number of installs is a good measure similar to number of downloads in an app store. Good apps in the sense of the app mining criteria do not need many users.
The number only tells how many people have seen the app at least once. It does not say much about that app, maybe that the app is already around for a long time, maybe that lots of users tried it only on one specific date.
I would value privacy higher than user tracking/analytics.

Add an app reviewer that analyzes and rates the usage of apps.

Cool, I agree this is a great idea. If not Blockstats, then who/what? I value the discussion, just trying to get to a really specific action we can implement now.

I think we could probably use the blockstats as a stopgap for several months until we find a way to measure ongoing usage of apps (if that makes sense to even do). Using the blockstats doesn’t have to be long term, and doesn’t have to simply count absolute # of installs.
For example you could weight monthly growth in relation to other apps like 70% and weight total installs like 30% or something.
Also you could use the blockstats but weight names registered that cost money some proportion more, so you have a tax to Sybil attacks. Very interested ideas can emerge from this one.
Further the system could also give an outsized reward to apps that create IDs that visit more apps. So if Graphite creates an ID, they get +1 but if that ID then uses a second app +2.

And single user apps are always ranked higher than multi user apps.
On Sat, 23 Mar 2019, 23:48 Patrick Stanley<EMAIL_ADDRESS>wrote:

I think we could probably use the blockstats as a stopgap for several
months until we find a way to measure ongoing usage of apps (if that makes
sense to even do). Using the blockstats doesn’t have to be long term, and
doesn’t have to simply count absolute # of installs.
For example you could weight monthly growth in relation to other apps like
70% and weight total installs like 30% or something.
Also you could use the blockstats but weight names registered that cost
money some proportion more, so you have a tax to Sybil attacks. Very
interested ideas can emerge from this one.
Further the system could also give an outsized reward to apps that create
IDs that visit more apps. So if Graphite creates an ID, they get +1 but
if that ID then uses a second app +2.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/blockstack/app-mining/issues/67#issuecomment-475911342,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABYcWQoUUuTnOc6VHeV-B7pfFpmk_nNCks5vZq8sgaJpZM4b-yom
.

In general, having a "stopgap" reviewer that looks at growth metrics makes a lot of sense.
Tracking growth in a decentralized way is a fairly complicated problem given user privacy but it's likely that Blockstack makes more progress on "privacy-focused growth tracking" in the coming months. As more sophisticated ways to track user engagement becomes available, the growth reviewer can also evolve with time.

Lastly, @dantrevino is it not simple good hygiene to have multiple identities? The onus is on the individual to have a burner ID and that seems perfectly reasonable for this purpose cc @jcnelson who agrees with this approach. I don’t have one bitcoin wallet and I don’t have one identity for the sake of privacy and security.

Its not a matter of using a burner id to not be identified.  How are you going to count non-social apps?
Another issue, how many of my 12 burner ids are going to count towards SocialDapp "+1"s?  i.e. If I use burnerA.id for my meetup group sharing, and burnerB.id for my hackathon planning, and burnerC.id for  posting my Anti-Google manifesto, does SocialDapp now get 3 +1s, even though there is only one "user"?

Sounds like we have some major issues with theblockstats. Blockstack team is going to explore some more anti-fragile solutions this quarter. Iceboxing for now, will come back with new proposals later.


optremove partition & histogram from col stats to reduce memory usage

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Error: Protocol error (Network.getResponseBody): Request content was evicted from inspector cache from getting response body from large XML page (around 13mb)

I am trying to get XML from a large XML page (around 13 mb) using Puppeteer. When I make the request and try to get the response body it returns the error: Error: Protocol error (Network.getResponseBody): Request content was evicted from inspector cache. I also tried the solution from this issue: https://github.com/puppeteer/puppeteer/issues/1599#issuecomment-355473214 but it did not work.
Steps to reproduce
Tell us about your environment:

Puppeteer version: 5.4.1
Platform / OS version: Mac OS Big Sur (11.0.1)
URLs (if applicable): view-source:https://rentals.ca/sitemaps/sitemap-on-toronto.xml
Node.js version: 15.2.0

What steps will reproduce the problem?
const browser =  await puppeteer.launch({ args: ["--proxy-server='direct://'", '--proxy-bypass-list=*'], headless: false })
const page = await browser.newPage()
await page.setUserAgent("Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/86.0.4240.193 Safari/537.36")
const client = await page.target().createCDPSession()
await client.send('Network.enable', {
            maxResourceBufferSize: 1024 * 1204 * 100,
            maxTotalBufferSize: 1024 * 1204 * 200,
            maxPostDataSize: 1024 * 1204 * 200
})
var response = await page.goto("view-source:https://rentals.ca/sitemaps/sitemap-on-toronto.xml", {waitUntil: 'networkidle2'})
var pageXML = ""
if (response !== null) {
    var pageXML = await response.text()
    await page.close()
    await browser.close()
}

What is the expected result?
The expected result is the response XML.
What happens instead?
It throws the error: Error: Protocol error (Network.getResponseBody): Request content was evicted from inspector cache.

https://github.com/puppeteer/puppeteer/issues/1599
My problem is solved.

I have already tried that @hxrain, it did not work, still get the same error.

+1 having the same issue here
also, from #1599 where the accepted solution (probably) is
await page._client.send('Network.enable', {
      maxResourceBufferSize: 1024 * 1204 * 100,
      maxTotalBufferSize: 1024 * 1204 * 200,
    })

I can't get it to work in TypeScript as _client is a private property
For me, this error was thrown when I tried to download image:
var view = await page.goto(url, { timeout: 0, waitUntil: 'networkidle2' });
await fs.writeFileSync(path, await view.buffer())

It's not all the time, but I can't grasp how it got triggered

Hi, I have the same problem with downloading images.
Error: Protocol error (Network.getResponseBody): Request content was evicted from inspector cache
I noticed that problem is because there is no "Content-Length" in the Headers for this image and it is undefined. For all others there is "Content-Length" and code is working. How to bypass this? I see transferred 548kb but I cant save it because buffer is undefined.

Hello,
I'm receiving this error too when downloading large web pages.
To get around this I used the fix above and it worked just fine.
await page._client.send('Network.enable', {
      maxResourceBufferSize: 1024 * 1204 * 100,
      maxTotalBufferSize: 1024 * 1204 * 200,
    })

However with more recent versions of Puppeteer, this is no longer working and I receive the following error:
page._client.send is not a function
Since page._client.send is no longer available I am wondering whether something has replaced it with something I should be using instead?
Thank you for your help with this.

@ArtHoff The property _client seems not to exist anymore. Instead get it from createCDPSession.
const client = await page.target().createCDPSession();

await page._client.send('Network.enable', {
    maxResourceBufferSize: 1024 * 1204 * 100,
    maxTotalBufferSize: 1024 * 1204 * 200,
});

@Orlandster Thank you for your suggestion; however, unfortunately when using that change my original problem comes back, it appears that the maxResourceBufferSize and maxTotalBufferSize are ignored now:
ProtocolError: Protocol error (Network.getResponseBody): Request content was evicted from inspector cache
    at /myProg/node_modules/puppeteer/lib/cjs/puppeteer/common/Connection.js:278:24

Is there anything else I need to do?
This is with puppeteer 15.3.2

+1
  const client = await page.target().createCDPSession();

  await client.send('Network.enable', {
    maxResourceBufferSize: 1024 * 1204 * 100,
    maxTotalBufferSize: 1024 * 1204 * 200,
  });

this seems to not work anymore
any solution/workaround for now?
thanks...

+1
Hi Guys,
Any updated on that or workaround for now?
I was updating to latest puppeteer versions, but I started face same issue.

+1

I have conducted extensive research and finally found a solution. The method is very simple.
_client ---> _client()
await page._client().send("Network.enable", {
  maxResourceBufferSize: 1024 * 1204 * 50,
  maxTotalBufferSize: 1024 * 1204 * 100,
});

The only change you need to make is to add a pair of parentheses after _client. Since version 13, _client in Page has become a function. That's all it takes, and I have verified it successfully in the latest version.
Some individuals previously suggested using page.target().createCDPSession(), which led to significant confusion. Although it doesn't throw an error, it doesn't work as intended.

@eskimo220 you are a legend!
This works :tada:

I have conducted extensive research and finally found a solution. The method is very simple.
_client ---> _client()
await page._client().send("Network.enable", {
  maxResourceBufferSize: 1024 * 1204 * 50,
  maxTotalBufferSize: 1024 * 1204 * 100,
});

The only change you need to make is to add a pair of parentheses after _client. Since version 13, _client in Page has become a function. That's all it takes, and I have verified it successfully in the latest version.
Some individuals previously suggested using page.target().createCDPSession(), which led to significant confusion. Although it doesn't throw an error, it doesn't work as intended.

it works! Amazing!

By disabling the browser cache, the issue was resolved.
await client.send('Network.enable', {       maxResourceBufferSize: 1024 * 1204 * 100,       maxTotalBufferSize: 1024 * 1204 * 1000,     });  // disabled cache await page.setCacheEnabled(false);


google-cloud-spanner: Add alternate r/w txn API that doesn't require use of callback

In the Google Cloud Spanner Java API, the only way to perform read/write transactions is by providing a callback, via the TransactionRunner interface.
I understand that the API is trying to hide the details of the need to automatically retry transactions as a convenience to the programmer, but this limitation is a serious problem, at least for me. I need to be able to manage the transaction lifecycle myself, even if that means I have to perform my own retries (e.g., based on catching some sort of "retryable" exception, in this case AbortedException).
To make this problem more concrete, suppose one wanted to implement Spring's PlatformTransactionManager for Google Cloud Spanner, so as to fit in with one's existing code, and use one's existing retry logic. It would not be possible with the current API.
It would be easy to augment the API in a backward compatible way, by adding a method returning a TransactionContext directly to the user, and let the user handle the retries. Of course this new method could be documented as "more dangerous and less convenient" to scare away noobs, etc.
Can this alternate (more traditional) transaction API style be added to the Java API?
Thanks.

Also please add the ability to get the commit timestamp from the transaction. There is currently no method TransactionContext.getCommitTimestamp(). Thanks.

@saturnism @emmanuelbernard This will be needed to implement JTA as well.

Closing this as this is duplicate of #1810


Add link to Omega CTest in Quick Start

In this PR I just added a link inside of the quick start documentation for the Omega CTest utility. I built the documentation locally and it looks correct to me.

Checklist

[x] Developer's Guide has been updated
[x] Documentation has been built locally and changes look as expected

@xylar I kept this very simple as you can see. Please let me know if you would like me to add any more information, or if just the link is enough!


Keep track of live allocated memory

What does this PR do?

Ensure we instrument frees
Introduce a live allocation object

Motivation
What inspired you to submit this pull request?  In order to motivate your point, it may be valuable to explain why this change is useful or what problem is being solved.
Additional Notes
This is the section to put technical guidance/constraints, call out potential regressions, cite sources, and in general offer some exposition on the how of the PR.  This section doesn't need to be incredibly detailed, but it makes life easier for reviewers!
How to test the change?
Describe here in detail how the change can be validated.  This is a great section to call out specific tests you've added or improved, or to acknowledge code sections which are particularly difficult to test.

To be improved :

Remove hardcode in the worker
Avoid memory growth in the library

Closing in favour of https://github.com/DataDog/ddprof/pull/197


Advice on using lol-html purely as html parser

Would it be possible to somehow disable rewriting functionality and simply use this library as a fast HTML parser/selector?...
I could simply ignore the output but some help from the library for such use case would be nice to have

It already operates as just a parser if you don't modify any tokens passed to the handlers

Would it be possible to improve API a little bit?
maybe another HtmlRewriter::new which doesn't require us to specify an output sink?


gpt4 is not generally available to new users

gpt4 is only available to "all existing API developers with a history of successful payments".  i.e. before the first billing cycle, new users will not be able to use gpt4.
I see that commit cdf8f9a4b2b4a65993227ac5af1eaf3f1b85c9d8 works well with gpt3.5-turbo-16k, but later commits stop working.

Thanks for filing this issue. The latest release v0.8.2 and main branch of github should be working correctly again. There was a period where aider assumed everyone had access to gpt-4, but that was resolved this morning.
See #71 for more details.
Let me know if you're still having any problems after upgrading. Or you can work around the bug with by running aider with --model gpt-3.5-turbo-16k to explicitly select a 3.5 model.

I'm going to close this issue for now, but please feel free to re-open if you have further problems.

latest main does not wokr, no.  --model gpt-3.5-turbo-16k selects the gpt-3.5 turbo model, but the diff detection is broken.

latest main does not work, no. --model gpt-3.5-turbo-16k selects the gpt-3.5 turbo model, but the diff detection is broken.

latest, not working
https://asciinema.org/a/WhosqpVZAZtk9PFFQbW92fw3v
commit https://github.com/paul-gauthier/aider/commit/cdf8f9a4b2b4a65993227ac5af1eaf3f1b85c9d8 works
https://asciinema.org/a/lRM3pT49q99Od6Ft1ZpMunZQ0


Implement use of predifined keyword such as

People should be able to run command as follow ( only if they're logged in )
>>> resp = yql.select('yahoo.social.profile').where(['guid','=','@me'])

Those predefined keyword must be  preceded by a @

Check this doc


Modify UI component - MainWindowTrackermon

Add new class which is a clone of MainWindow that fits Trackermon shows.
Other info

Fix checkstyle issues
Modify JavaDocs and names of methods/variables

Codecov Report

Merging #66 (ce0d652) into master (ae7c41a) will decrease coverage by 2.31%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master      #66      +/-   ##
============================================
- Coverage     53.16%   50.85%   -2.32%
  Complexity      399      399
============================================
  Files           100      102       +2
  Lines          1672     1748      +76
  Branches        177      181       +4
============================================
  Hits            889      889
- Misses          751      827      +76
  Partials         32       32

Impacted Files
Coverage Δ

...in/java/seedu/address/ui/CommandBoxTrackermon.java
0.00% <0.00%> (ø)

...in/java/seedu/address/ui/MainWindowTrackermon.java
0.00% <0.00%> (ø)

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 ae7c41a...ce0d652. Read the comment docs.