Kdtree

Implementations of KDTree for fast search in a point cloud
Demonstration on using the implementation for transferring nodal based grid functions partitioned on different number of processes

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#3325
@bslazarov
@tzanio
@termi-official + @tzanio + @sebastiangrimberg
11/22/22
⌛due 12/6/22
⌛due 12/13/22

PR Checklist

[ ] Code builds.
[ ] Code passes make style.
[ ] Update CHANGELOG:

[ ] Is this a new feature users need to be aware of? New or updated example or miniapp?
[ ] Does it make sense to create a new section in the CHANGELOG to group with other related features?

[ ] Update INSTALL:

[ ] Had a new optional library been added? If so, what range of versions of this library are required? (Make sure the external library is compatible with our BSD license, e.g. it is not licensed under GPL!)
[ ] Have the version ranges for any required or optional libraries changed?
[ ] Does make or cmake have a new target?
[ ] Did the requirements or the installation process change? (rare)

[ ] Update continuous integration server configurations if necessary (e.g. with new version requirements for each of MFEM's dependencies)

[ ] .github
[ ] .appveyor.yml

[ ] Update .gitignore:

[ ] Check if make distclean; git status shows any files that were generated from the source by the project (not an IDE) but we don't want to track in the repository.
[ ] Add new patterns (just for the new files above) and re-run the above test.

[ ] New examples:

[ ] All sample runs at the top of the example source file work.
[ ] Update examples/makefile:

[ ] Add the example code to the appropriate SEQ_EXAMPLES and PAR_EXAMPLES variables.
[ ] Add any files generated by it to the clean target.
[ ] Add the example binary and any files generated by it to the top-level .gitignore file.

[ ] Update examples/CMakeLists.txt:

[ ] Add the example code to the ALL_EXE_SRCS variable.
[ ] Make sure THIS_TEST_OPTIONS is set correctly for the new example.

[ ] List the new example in doc/CodeDocumentation.dox.
[ ] If new examples directory (e.g.examples/pumi), list it in doc/CodeDocumentation.conf.in
[ ] Companion pull request for documentation in mfem/web repo:

[ ] Update or add example-specific documentation, see e.g. the src/examples.md.
[ ] Add the description, labels and screenshots in src/examples.md and src/img.
[ ] In examples.md, list the example under the appropriate categories, add new categories if necessary.
[ ] Add a short description of the example in the "Extensive Examples" section of features.md.

[ ] New miniapps:

[ ] All sample runs at the top of the miniapp source file work.
[ ] Update top-level makefile and makefile in corresponding miniapp directory.
[ ] Add the miniapp binary and any files generated by it to the top-level .gitignore file.
[ ] Update CMake build system:

[ ] Update the CMakeLists.txt file in the miniapps directory, if the new miniapp is in a new directory.
[ ] Add/update the CMakeLists.txt file in the new miniapp directory.
[ ] Consider adding a new test for the new miniapp.

[ ] List the new miniapp in doc/CodeDocumentation.dox
[ ] If new miniapps directory (e.g.miniapps/nurbs), add it to MINIAPP_SUBDIRS in the makefile.
[ ] If new miniapps directory (e.g.miniapps/nurbs), list it in doc/CodeDocumentation.conf.in
[ ] Companion pull request for documentation in mfem/web repo:

[ ] Update or add miniapp-specific documentation, see e.g. the src/meshing.md and src/electromagnetics.md files.
[ ] Add the description, labels and screenshots in src/examples.md and src/img.
[ ] The miniapps go at the end of the page, and are usually listed only under a specific "Application (PDE)" category.
[ ] Add a short description of the miniapp in the "Extensive Examples" section of features.md.

[ ] New capability:

[ ] All new public, protected, and private classes, methods, data members, and functions have full Doxygen-style documentation in source comments. Documentation should include descriptions of member data, function arguments and return values, template parameters, and prerequisites for calling new functions.
[ ] Pointer arguments and return values must specify whether ownership is being transferred or lent with the call.
[ ] Any new functions should include descriptions of their intended use e.g. for internal use only, user-facing, etc., along with references to example code whenever possible/appropriate.
[ ] Consider adding new sample runs in existing examples to highlight the new capability.
[ ] Consider saving cool simulation pictures with the new capability in the Confluence gallery (LLNL only) or submitting them, via pull request, to the gallery section of the mfem/web repo.
[ ] If this is a major new feature, consider mentioning it in the short summary inside README (rare).
[ ] List major new classes in doc/CodeDocumentation.dox (rare).

[ ] Update this checklist, if the new pull request affects it.
[ ] Run make unittest to make sure all unit tests pass.
[ ] Run the tests in tests/scripts.
[ ] (LLNL only) After merging:

[ ] Update internal tests to include the new features.

Let me quickly try to summarize to sanity check if I get the idea correctly: The KDTreeNodalProjection is a nearest neighbor interpolator for grid functions defined on two arbitrary meshes, thus complementing GSLIBPointFinder.  GSLIBPointFinder utilizes an interpolation per element.

KDTreeNodalProjection does not interpolate. It will just find matching nodes and will project the data from the dof/dofs associated with src node to the dof/dofs associated with the target node.

@termi-official, @sebastiangrimberg and @tepperly -- this has been updated and is ready for review again.
Can you please take another look?

I am in Ireland on vacation through the end of April. I'll look after 1 May.
Tom
On Tue, Apr 25, 2023, 7:52 PM Tzanio Kolev @.***> wrote:

@termi-official https://github.com/termi-official, @sebastiangrimberg
https://github.com/sebastiangrimberg and @tepperly
https://github.com/tepperly -- this has been updated and is ready for
review again.
Can you please take another look?
—
Reply to this email directly, view it on GitHub
https://github.com/mfem/mfem/pull/3325#issuecomment-1522262043, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACM3KGHCJCRTWTLXZQ4ND3XDAMNZANCNFSM6AAAAAASAIIXHU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@bslazarov, can you please address the reviewer feedback?

🔔

Hi @sebastiangrimberg and @tepperly , Could you please approve the PR if you do not have any more comments. Best regards, Boyan

@bslazarov, can you merge master here and resolve the conflicts above

@sebastiangrimberg The KDTreeNodalProjection is templated as requested. Please, review the PR and approve it if you do not have anymore comments. I have added a base projection class in order to accommodate dimension independent drivers.

@sebastiangrimberg The KDTreeNodalProjection is templated as requested. Please, review the PR and approve it if you do not have anymore comments. I have added a base projection class in order to accommodate dimension independent drivers.

Nice! Thanks for that, much improved. Any thoughts on resolving the first bullet point in my comment?

@sebastiangrimberg The KDTreeNodalProjection is templated as requested. Please, review the PR and approve it if you do not have anymore comments. I have added a base projection class in order to accommodate dimension independent drivers.

Nice! Thanks for that, much improved. Any thoughts on resolving the first bullet point in my comment?

The location have been discussed during several MFEM meetings and the consensus was that this should be the correct place for the nodal mapper. If necessary - it can be always moved to other location later.

Perhaps you can help me to understand the arguments for this? I'm sorry to keep pushing on this, but I don't see the downsides of further file/class organization by keeping file lengthier shorter and more descriptive. Combining (not directly related) classes into the same header and definition files as done here is also against the style adopted by most of the code in the fem/ directory.
If there's a strong argument for it or I am overruled then that's understandable, but for now I'm still a proponent of separating things until I better understand the motivations.

Perhaps you can help me to understand the arguments for this? I'm sorry to keep pushing on this, but I don't see the downsides of further file/class organization by keeping file lengthier shorter and more descriptive. Combining (not directly related) classes into the same header and definition files as done here is also against the style adopted by most of the code in the fem/ directory.
If there's a strong argument for it or I am overruled then that's understandable, but for now I'm still a proponent of separating things until I better understand the motivations. Also I'm happy to go ahead and make the change I have in mind here on this PR and then approve, but please let me know if you'd like me to do this to help out.

The kdtree class is general and will be used for applications other than the current example. I intend to use it for meshless/meshfree discretizations, i.e., searching points in a point cloud and general mapping between different meshes. All these are unrelated to grid functions and code in the fem directory. On the other hand, the nodal map class works on grid functions and nothing else. So, its place is precisely the directory where grid functions are defined. Of course, we can separate the nodal map from the grid function class (still leaving it in the fem directory); however,  this will create another header/cpp file for a relatively small set of code.

Ah, yes to be clear I was only proposing moving to new fem/kdtree.hpp/.cpp files, not moving this code outside of the fem/ directory. I think that is a useful change for organizations.

@sebastiangrimberg The KDTreeNodalProjection is templated as requested. Please, review the PR and approve it if you do not have anymore comments. I have added a base projection class in order to accommodate dimension independent drivers.

Nice! Thanks for that, much improved. Any thoughts on resolving the first bullet point in my comment?

The location have been discussed during several MFEM meetings and the consensus was that this should be the correct place for the nodal mapper. If necessary - it can be always moved to other location later.

Ah, yes to be clear I was only proposing moving to new fem/kdtree.hpp/.cpp files, not moving this code outside of the fem/ directory. I think that is a useful change for the sake of clarity and organization.

The fem directory contains files for FEM discretizations. The kdtree.hpp is not related directly to any FEM discretization technique or algorithm.   Thus, current kdtree placement is definitely better considering its general applicability.

Hi @bslazarov, please see the branch sjg/kdtree for my proposal along with some other final style consistency changes before merging. If this looks acceptable to you, please forward merge your branch and I will approve.

Hi @bslazarov and @sebastiangrimberg,
Sorry for my delayed comments on this PR. I think it is an important one and I'd like to include it in tomorrow's release.
I looked at the code again and I actually agree with @sebastiangrimberg. His branch above is indeed an improvement, both organizationally and stylistically. I suggest we go with it. @bslazarov, do you agree?
Tzanio

Thanks @tzanio. @bslazarov, if you are busy today I'm happy to take care of the addressing the final comments so we can get this into the release tomorrow. Just let me know, thanks!

Hi @bslazarov and @sebastiangrimberg,
Sorry for my delayed comments on this PR. I think it is an important one and I'd like to include it in tomorrow's release.
I looked at the code again and I actually agree with @sebastiangrimberg. His branch above is indeed an improvement, both organizationally and stylistically. I suggest we go with it. @bslazarov, do you agree?
Tzanio

Sure - I will prepare it in the afternoon.

Thanks @tzanio. @bslazarov, if you are busy today I'm happy to take care of the addressing the final comments so we can get this into the release tomorrow. Just let me know, thanks!

Thank you Sebastian! I will work on it in the afternoon.

Thanks @tzanio. @bslazarov, if you are busy today I'm happy to take care of the addressing the final comments so we can get this into the release tomorrow. Just let me know, thanks!

All changes are updated - please, re-review and approve :-)

I pushed a commit to fix the failing make style.

Merged in next for testing...


Stats are missing in non-tabular packages

We are missing stats (especially bytes) attributes for packages AND for resources, see http://pkgstore.datahub.io/examples/geojson-tutorial/latest/datapackage.json

Fixed via https://github.com/frictionlessdata/datapackage-pipelines/commit/e5173698ea81383b6eda76f25c1f71586b814caf


Please fixe content-type for .wasm (WebAssembly)

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report  <!-- Please search GitHub for a similar issue or PR before submitting -->
[x] Feature request
[ ] Documentation issue or request
[ ] Other: <!-- Please describe: -->

Current behavior

I'm compiling WebAssembly (*.wasm) files and can't serve these properly to the browser.
Any ".wasm"-file is of content-type: "application/octet-stream"
Expected behavior
The proper content-type should be "application/wasm"
Environment
Browser:

- [ ] Chrome (desktop) version XX
- [x] Firefox version 62.0.2
- [ ] Safari (desktop) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:

- Live Server: 5.1.1
- Platform:  Windows, compiling in WSL-ubuntu
- Visual Studio Code: 1.27.2

Others

This issue exists in Chrome v79 too.

Please tell me when it will be ok :)

I'm still having this issue on chrome 83 and mac os

I also still have this problem. Here is a quick work around though, if you need it:
I only have the MIME-type issue, when I try to load my wasm-module via the WebAssembly.instantiateStreaming(fetch('path/to/my_module.wasm')) -API.
With the slightly more verbose an less efficient API: WebAssembly.instantiate(bytes)  the import works completely fine.
Both in comparison:
// Desired API-usage (Currently apparently not working with Live-Server plugin)
async function getModuleFunctions() {
    const resultObject = await WebAssembly.instantiateStreaming(fetch('path/to/my_module.wasm'));
    return resultObj.instance.exports;
}

// Older API as work-around
async function getModuleFunctions() {
    const fetchResult = await fetch('path/to/my_module.wasm');
    const bytes = await fetchResult.arrayBuffer();
    const resultObj = await WebAssembly.instantiate(bytes);
    // The functions reside in the exports section on the resultObject.instance
    return resultObj.instance.exports;
}

how is this still not fixed...

#697 The issue will be fixed once this is merged.

:( Is there a version update?
Or for plugins is it possible to provide a configuration item that supports file suffix mapping content-type

Same issue here. I'll be using python3 -m http.server until this is fixed.


Fixed - foreground services are killed by the OS but there is a catch

Android 12 and later versions are designed to terminate foreground services automatically after days or weeks.
09-17 08:02:23.129  2670  2818 W ActivityManager: Exception when unbinding service com....otm.v3/com....otm.vpn.service.FilterService
09-17 08:02:23.129  2670  2818 W ActivityManager: android.os.DeadObjectException

If you check the logs and your service was a STICKY one, the ActivityManager is automatically restarting it with the following kill reason:
ActivityManager: Rescheduling restart of crashed service com.cujo.otm.v3/.service.CujoService in 9967ms for mem-pressure-event - mem pressure is the reason in my case. Just follow the LogCat and find yours
This means that the onStartCommand has already been called, but with a null intent.
In this situation, you will need to manually reinitialize everything.

Spend 2 weeks for finding this behaviour

Hello @Duna @cj-marius-duna .. in my opinion this is nothing special in Android 12.. this is how FGS (or in earlier Android simply Services) worked on from the well "beginning". Even when the the service was sticky system may restart the service any time when memory is getting low and you need to cope with that in the code... for instacne expect intent to be null or expect a flow when onCreate is not called in the lifecycle but only onStartCommand...


Somehow that commit https://github.com/owncloud/ocis/commit/26ee19d6fee65f3ddc88484220184c0c29ae27dd got onto master branch without the normal PR and CI process. It needs to be revert to make CI green again.
Related Issue

Fixes https://github.com/owncloud/core/issues/38657

How Has This Been Tested?
CI
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[x] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

It is really bizarre. https://github.com/owncloud/ocis/commit/26ee19d6fee65f3ddc88484220184c0c29ae27dd got pushed to master, not force pushed at all, and I have no clue how. It was definitely my machine 🤔

It is really bizarre. 26ee19d got pushed to master, not force pushed at all, and I have no clue how. It was definitely my machine thinking

You must have some super-admin privs to the repo, that override branch protection.

I am Admin, however I tried pushing something to master and it blocked me 🤷

@phil-davis

The culprit... I'll check that box :)

having a look at my git reflog I might have been drunk:


Can't render tab character with Paragraph.

Description
I'm trying to render text that contains tabs but the tab character is not rendered.
To Reproduce
Here's the code:
use std::io::{self, stdout};

use crossterm::{
	event::{self, Event, KeyCode},
	terminal::{disable_raw_mode, enable_raw_mode, EnterAlternateScreen, LeaveAlternateScreen},
	ExecutableCommand,
};
use ratatui::{prelude::*, widgets::*};

fn main() -> Result<(), std::io::Error> {
	enable_raw_mode()?;
	stdout().execute(EnterAlternateScreen)?;
	let mut terminal = Terminal::new(CrosstermBackend::new(stdout()))?;

	let mut quit = false;
	while !quit {
		terminal.draw(ui)?;
		quit = handle_events()?;
	}

	disable_raw_mode()?;
	stdout().execute(LeaveAlternateScreen)?;
	Ok(())
}

fn handle_events() -> io::Result<bool> {
	if event::poll(std::time::Duration::from_millis(50))? {
		if let Event::Key(key) = event::read()? {
			if key.kind == event::KeyEventKind::Press && key.code == KeyCode::Char('q') {
				return Ok(true);
			}
		}
	}
	Ok(false)
}

fn ui(frame: &mut Frame) {
	frame.render_widget(
		Paragraph::new(Text::raw("Test\ttest")),
		frame.size(),
	);
}

Expected behavior
The tab character between "Test" and "test" should be rendered just like echo 'Test\ttest'
Screenshots

Environment

OS: Linux
Terminal Emulator: GNOME Terminal (version 3.48.3)
Font: Iosevka
Crate version: 0.25.0
Backend: crossterm

I wrote a very simple test:
        #[test]
        fn render_tab() {
            let span = Span::styled("a\tb", Style::default());
            let mut buf = Buffer::empty(Rect::new(0, 0, 15, 1));
            span.render(buf.area, &mut buf);
        }

And I printed out the widths of the symbols being rendered here:
https://github.com/ratatui-org/ratatui/blob/27e9216cea7f25fcf172fe0a8f11e7cca222b055/src/text/span.rs#L364
using this dbg! statement:
dbg!((g.symbol, symbol_width));

Here's what I get:

It seems like g.symbol.width() from https://crates.io/crates/unicode-width assumes \t is 0 wide. I'm not sure what the best way to solve this is other than special casing \t.

More context: https://github.com/unicode-rs/unicode-width/issues/6

I was actually looking at this exact code the other day for another reason. If we change this to correctly identify the length of ANSI sequences here, the we can probably just allow ANSI to be inside strings directly without needing ansi-to-tui. Right now the ANSI takes effect, but the amount of characters skipped is wrong.
For now though, you'll likely need to replace tabs and probably other non-printable characters in your text with reasonable defaults (there isn't one that would make all people happy for tab widths).

Originally posted by @mcclure in https://github.com/ratatui/ratatui/issues/1366

When I put \t into my Paragraph text weird things happen.
Repro
https://github.com/mcclure/rs-bug branch tui-tab-bug.
cargo run this. This is a fork of the "popup" sample. It displays this string which contains a mix of \ts and \ns. Ignore the on screen instructions, the controls you care about are a (which turns trim on and off) and s (which replaces the entire on-screen text with some random keysmash).
You may find it also interesting to change line 44 so the default value of trim is different, I get different results (which I did not screenshot).
Screenshots
Application, initially. It looks normal.

Hit a to enable alternate-text.

It's supposed to look like that. Hit a again to restore the original text. Now hit s. This will have the current paragraph be displayed with trim off.

Now that trim is off, and the tabs are appearing… That looks weird. Hit a again to replace this text with the alt text.

This looks even weirder. Having displayed text with tabs just once broke the "dirty rectangles" of the triangle. Hit a again to restore the original text.

Oh no.
Environment
OS: Ubuntu 24.04 LTS,
Terminal Emulator: gnome-terminal
Font: ? Ubuntu default
Crate version: 0.28.1
Backend: I don't know what that means


[Core][Distributed] add coordinator to reduce code duplication in tp and pp

This PR adds coordinator, the last piece in https://github.com/vllm-project/vllm/issues/3587 . The main benefit is that all distributed stuff (e.g. complicated selection logic in communication ops, prev/next rank logic, initialization&destruction logic) can be shared for all tp/pp code.
After this PR, we explicitly require the code writer specify which group it wants to operate on:

world group, by get_world()
tp group, by get_tp()
pp group, by get_pp()

After that, they can just call all_reduce, prev_rank ... without any further changes.

@youkaichao Great! I love this refactoring PR 👍
My code in #5414 can be simpler now :)


Update system_api.c

out_info->model was zeroed by memset

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Cherry-picked in 9f39411. Thanks again!


Support passing in a date string

Correct me if I am wrong, but it seems like right now it is not possible to pass something like "2016-12-27T16:16:22.585Z" to set the initial value, but passing a Date object works, so if I do Date("2016-12-27T16:16:22.585Z") and then pass that in, it works.
Do you think flatpickr could have an if to check if the passed value is a string, and convert it to a Date object behind the scenes?

Actually, defaultDate seems to be what I would want for a predefined date, and it says in the docs anything that works with Date.parse should work, which "2016-12-27T16:16:22.585Z" does work with Date.parse, but setting defaultDate to "2016-12-27T16:16:22.585Z" does not work with flatpickr.

@chmln I think a simple solution here, is to remove the replacing of dashes with slashes you are doing, and only allow strings that work with Date.parse in both Firefox and Chrome

Also see https://github.com/chmln/flatpickr/commit/9b910062513a0d6e78b325f76f2db1f346d2cc00

Release coming soon.

Ah nice, thanks! Would you be interested in some help converting this into an ES2015 class? I'd be glad to help out with that effort, and I think we should write some tests as well.

Would you be interested in some help converting this into an ES2015 class?

I've already tried it actually, didn't see any practical benefits but quite a few regressions.

I think we should write some tests as well

Yes, absolutely. I'll incorporate some by the next release and will accept PRs from there

I just think it would be much easier to follow the code if it were a class with methods, but it's really no benefit, just syntactic sugar.
I think tests first would be a good idea, then we can revisit possible refactors after we have tests to make sure nothing breaks 😄 .
Let me know where I can help out!


could you please reduce the rate to 2400bps ?

2400bps is the standard rate of the narrowband vocoder.

There was a reply about the target bit rate in this issue: #18

Our current main use case is VoIP, that is why we chose 3kbps. But I can see the value of lower bit rate for HF radio use cases, so we will have that in mind.

From my understanding, the target bit rate is set as a "goal" of the training process, so I guess it's not possible to have an adjustable bit rate for the encoder. You might have to wait for new models with different bit rates.


TS-4258: NULL out server_session when it is destroyed.

+1. Should this be a back port to 6.1.2?

Meh. It only happens with certain obscure plugin shenanigans involving transforms. Unfortunately those are permitted (essentially delaying at certain callout points) and done at Y! so the core should handle it correctly.


Sync-DbaSqlLoginPermission > Sync-DbaLoginPermission

the previous command name never sat well with me. I realized today it's because it's too long and contains an unnecessary Sql. I'd like to rename it.

Not that my opinion means that much, but I think that's fine as long as the command covers both DOMAIN\User logins and regular SQL logins

good question, it does do both 👍

I'm good

+1 for me too

sweet thanks yall, gonna do it

I endorse the wisdom of our fearless leader.

The current name makes me think of SQL logins so I'd change it 👍

yall the best, thank you. will do this sometime this week/weekend


Process artifacts : Server failed to authenticate the request

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=184860&view=logs&j=43b54640-f671-5524-8f7b-714d77229de9&s=380ab8c6-4139-55ff-15c4-fb48260f4dca&t=8c631bd7-3091-56a7-0716-a38699ccbe61&l=492

Very strange, never saw this before and seems to have been a temporary Azure China issue, since successive builds have succeeded.


Fixed issue with the name 'Engine' does not exists in the current context

After following the sample code in the readme file, I was unable to compile the code with visual studio community 2015. I read the source code and found out that the Engine class lives in the RazorEngine namespace. I just added it and now works!

I changed the documentation accordingly, thanks


pod install error

My 'Podfile':
source 'https://github.com/CocoaPods/Specs.git'
platform :ios, '8.0'
target 'XXX' do
  use_frameworks!
  pod 'Each', '~> 1.2'
end

And, pod install
the error is
[!] Oh no, an error occurred.

Search for existing GitHub issues similar to yours:
https://github.com/CocoaPods/CocoaPods/search?q=cannot+load+such+file+--+nanaimo&type=Issues

If none exists, create a ticket, with the template displayed above, on:
https://github.com/CocoaPods/CocoaPods/issues/new

Be sure to first read the contributing guide for details on how to properly submit a ticket:
https://github.com/CocoaPods/CocoaPods/blob/master/CONTRIBUTING.md

but, if I delete pod 'Each', '~> 1.2'
the other pods is available

cocoapods version  1.2.0

Thanks for opening this issue.
It seems that the latest version hasn't been pushed correctly on CocoaPods.
I just did it, please run pod repo update && pod install to fix your problem.


Bug - Number for US population is incorrect

The app shows a wrong number for US population (23,780,000).

Closed by https://github.com/archydeberker/corona-calculator/pull/30


Invalid Parameter format for num_feature expect int but value='4294967258'

[18:18:13] start ip-172-31-20-145:1
[18:18:13] Load part of data 1 of 4 parts
[18:18:45] Load part of data 1 of 4 parts
terminate called after throwing an instance of 'dmlc::ParamError'
what():  Invalid Parameter format for num_feature expect int but value='4294967258'
This is with training of Criteo TB dataset for 1 day's worth in libsvm format. I was also using 1 days worth of data for test.
https://www.csie.ntu.edu.tw/~cjlin/libsvmtools/datasets/binary.html#criteo

@newnovice01 This is a limitation of XGBoost: number of features cannot exceed INT_MAX<PHONE_NUMBER>). For now, I will have the program throw exception when number of features is too big. We do want to remove the limitation eventually, however.


Dont treat empty strings as a valid analytics profile

See commits for more detail.
cc @dsingleton

:zap:


Undo Redo Support

Is there a way to do undo/redo using Synced Store?

Yes @websiddu ! You should be able to use the regular Yjs undomanager, e.g.:
const val = getYjsValuestore.value)

const undoManager = new Y.UndoManager(val)

// Do stuff to val (Let undoManager capture the updates)

undoManager.undo() // Revert the captured updates

Does this answer your question?


improve observability for procedure

What problem does the new feature solve?
Currently we are not able to check the state of each procedure, how long it takes, which resources it keeps, or which state it at. It's hard to debug or verify things related to one specific procedure.
This ticket proposes to improve the observability of procedures, by exposing the above things somewhere. Like an HTTP API or a table in information_schema.
Useful links

procedure state
https://github.com/GreptimeTeam/greptimedb/blob/1b5862223c0a8561ef1ce3045128d4c5a27c7d5e/src/common/procedure/src/procedure.rs#L301-L315

acquiring locks
https://github.com/GreptimeTeam/greptimedb/blob/530353785cd632faa6ec71552d25ed9da137cb0b/src/common/procedure/src/local/runner.rs#L115-L125

implement cluster_info table in information_schema https://github.com/GreptimeTeam/greptimedb/pull/3832

What does the feature do?

[ ] Improve ProcedureState::Running to contain a String. This string shows a procedure's current stage.
[ ] Count the procedure's execution time and store it.
[ ] Expose those information to information_schema

Implementation challenges
No response

I would like to work on it.
How can I verify my change locally for this improvement?

Improve ProcedureState::Running to contain a String. This string shows a procedure's current stage.

For this feature, one approach is adding a debug or trace log in the runner to print the current state of the procedure.
https://github.com/GreptimeTeam/greptimedb/blob/1b5862223c0a8561ef1ce3045128d4c5a27c7d5e/src/common/procedure/src/local/runner.rs#L195-L197
@WenyXu Do you have any suggestions?

For the execution time, it should be measuring how long does execute_procedure_in_loop() take right?
What precision (milliseconds, microseconds, nanoseconds?) do you want for the execution_time?

Milliseconds should be enough.


Missing some frames while showing keyboard when there is GoogleMap Widget

There are some frames missing when showing up the keyboard, it may be on the same page or another page that is pushed from a page that contains GoogleMap Widget.
Code

Sample Code
Here is a sample code to reproduce the problem.
import 'dart:async';

import 'package:flutter/material.dart';
import 'package:google_maps_flutter/google_maps_flutter.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Google Maps Demo',
      home: MapSample(),
    );
  }
}

class MapSample extends StatefulWidget {
  @override
  State<MapSample> createState() => MapSampleState();
}

class MapSampleState extends State<MapSample> {
  Completer<GoogleMapController> _controller = Completer();

  static final CameraPosition _kGooglePlex = CameraPosition(
    target: LatLng(37.42796133580664, -122.085749655962),
    zoom: 14.4746,
  );

  static final CameraPosition _kLake = CameraPosition(
      bearing:<PHONE_NUMBER>395799,
      target: LatLng(37.43296265331129, -122.08832357078792),
      tilt: 59.440717697143555,
      zoom: 19.151926040649414);

  @override
  Widget build(BuildContext context) {
    return new Scaffold(
      body: Stack(
        children: [
          GoogleMap(
            mapType: MapType.hybrid,
            initialCameraPosition: _kGooglePlex,
            onMapCreated: (GoogleMapController controller) {
              _controller.complete(controller);
            },
          ),
          Align(
            alignment: Alignment.bottomCenter,
            child: Container(
              color: Colors.white,
              child: TextField(
                decoration: InputDecoration(
                    hintText: "Search Here",
                    contentPadding:
                        EdgeInsets.symmetric(horizontal: 20, vertical: 10)),
              ),
            ),
          ),
        ],
      ),
    );
  }

  Future<void> _goToTheLake() async {
    final GoogleMapController controller = await _controller.future;
    controller.animateCamera(CameraUpdate.newCameraPosition(_kLake));
  }
}

https://user-images.githubusercontent.com/91472810/134912673-2f67e8a2-5f54-42be-882d-0ac812e60987.mp4

**Target Platform: Android
**Target OS version: 11
**Devices: Samsung Galaxy A21s
Logs

Logs
flutter analyze
Analyzing google_map_keyboard_sample...                                  No issues found! (ran in 2.2s)
flutter doctor -v
[✓] Flutter (Channel stable, 2.2.3, on macOS 11.5.2 20G95 darwin-x64, locale en-US)
    • Flutter version 2.2.3 at /Users/ethiopianairlines/flutter
    • Framework revision f4abaa0735 (3 months ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/ethiopianairlines/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Ultimate Edition (version 2021.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.60.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.26.0

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 94.0.4606.61
    ! Device <IP_ADDRESS>:5555 is offline.

! Doctor found issues in 1 category.

Hi @TesfalemNigussie
Unfortunately, the Flutter version you're using is outdated and is no longer supported, it's not feasible to provide any bug fixes for such an old version of Flutter.
I kindly advise you to update to the latest Flutter stable, if you encounter the issue again, please comment below with the details from the latest stable, including steps to replicate the issue.
Can you please upgrade to the latest  stable and try again
Execute the following commands to upgrade
flutter channel stable
flutter upgrade --force
flutter doctor -v
If the problem persists, can you please provide your flutter doctor -v with the upgrade, complete reproducible minimal code sample, and complete flutter run --verbose logs running the minimal code sample?
Thank you

@TahaTesser As suggested I have updated my flutter to the current stable version.
[✓] Flutter (Channel stable, 2.5.1, on macOS 11.5.2 20G95 darwin-x64, locale en-US)
    • Flutter version 2.5.1 at /Users/ethiopianairlines/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ffb2ecea52 (11 days ago), 2021-09-17 15:26:33 -0400
    • Engine revision b3af521a05
    • Dart version 2.14.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/ethiopianairlines/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Ultimate Edition (version 2021.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.60.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.26.0

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 94.0.4606.61

• No issues found!

but the issue still persists on the current stable version as well.

Hi @TesfalemNigussie
I just tried your code sample on multiple devices on the latest Flutter stable channel. I notice the keyboard jumps, map resizes when running the app in profile mode. Is this what you describing by "Missing frames".
The framerate is consistently when opening the keyboard tho

https://user-images.githubusercontent.com/48603081/135227735-f7f2e204-ff47-409f-bce4-fe02fac417ad.mp4

flutter doctor -v
[✓] Flutter (Channel stable, 2.5.1, on Microsoft Windows [Version 10.0.22000.194], locale en-US)
    • Flutter version 2.5.1 at C:\Users\Taha\Code\flutter_stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ffb2ecea52 (12 days ago), 2021-09-17 15:26:33 -0400
    • Engine revision b3af521a05
    • Dart version 2.14.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Users\Taha\Code\android-sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_SDK_ROOT = C:\Users\Taha\Code\android-sdk
    • Java binary at: C:\Users\Taha\Code\android-studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.3)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.11.31702.278
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 2020.3)
    • Android Studio at C:\Users\Taha\Code\android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] VS Code (version 1.60.2)
    • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.26.0

[✓] Connected device (4 available)
    • IN2011 (mobile)   • c9d8ee0c • android-arm64  • Android 11 (API 30)
    • Windows (desktop) • windows  • windows-x64    • Microsoft Windows [Version 10.0.22000.194]
    • Chrome (web)      • chrome   • web-javascript • Google Chrome 94.0.4606.61
    • Edge (web)        • edge     • web-javascript • Microsoft Edge 93.0.961.47

• No issues found!

Can you please explain the issue in detail? Perhaps screenshots of the devtools

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


shell: backends: add init log level for dummy shell

Dummy shell was always started with log backend enabled with filter statically set to INFO level.
AFAIK no use of such log backend can be made, thus causing waste of resources.
The new changes keep INFO as the default filter level. In order to disable log backend to be attached to the Dummy shell, CONFIG_SHELL_DUMMY_INIT_LOG_LEVEL_NONE has to be set.

I wonder what the use case is here? As someone uses dummy backend to call commands from code is it not enough to just disable "shell as log backend" option?

I wonder what the use case is here? As someone uses dummy backend to call commands from code is it not enough to just disable "shell as log backend" option?

This PR target 2 potential limitations:

all the shell backends (ie. MQTT, RTT, Telnet, UART) but the Dummy have the log backend optionally enabled based on the selected log level. The Dummy backend is the only one to have the log backend always enabled at init time and no Kconfig symbol for customizing the initial log level
dummy shell can be used to call commands from code, but actually the logs cannot be used/captured. Even mcumgr that makes use of the Dummy shell for its shell exec sub-command, does not support collecting logs through the Dummy shell since Zephyr does not support this feature yet: https://docs.zephyrproject.org/3.1.0/services/device_mgmt/mcumgr.html

Thus the Dummy Log backend is basically useless (as far as I know).
Nevertheless, I reckon some CPU and RAM is allocated to processing logs for the Dummy shell log backend.

@marco85mars20 ok it is clear now.


Polish UI for @trustRequired filter

Testing #122230

open the extensions viewlet and filter by @trustRequired
there are two groups but the headers are rather cryptic
I suggest to insert some kind of message atop of each group that explains what this means

/cc @misolori

/cc @sandy081 as well

I wonder if this should apply to all of our cases where we do this splitting (WSL, Codespaces, Containers, etc.) because they all can fall under that same rule as well:

I wonder if this should apply to all of our cases where we do this splitting

+💯 It is so hard to convey the meaning of these bucket through titles only

Totally agree to have more info for views - Have you thought about providing this info using our custom hovers?


[Multischema changelog]

get changelog once (skip repeat per scheme and skip again when a migration is applied)

Closing this PR, please reopen a new PR if this is still required.


Braces don't always lower indentation level

For example, in match statements where a comma follows, or in a closure where a paren-semicolon follows. Neither of those have correct indentation, as they both report one level more than they should. Moving the following characters onto their own line corrects the issue.
While typing it will work fine most of the time, but when pasting code with "Auto Indent on Paste" set to true it causes quite a few incorrect indents.
Below is an example of what happens: (Note the last brace ends the function, not the match statement.)

Edit: Hmm, it's a bit more complicated than that, it seems that sometimes (like in let bindings) it works fine, even if the ending brace has a semicolon after it. And sometimes it works fine in if statements, and other times it doesn't. The source I'm using to test is https://github.com/gfx-rs/gfx-rs/tree/master/examples/deferred

I think now they are overlowering the indentation level to 0. Here's how it looks in my editor:
fn main() {
    loop {
|                     // the cursor position is on column 0
}                     // autindented  to col 0
//...
}

the same indentation happens on every level:
fn main() {
// ...
    loop {
        match a {
|                     // the cursor position is on col 0
}                     // autindented to col 0
    }
//...
}

I'm not sure if this behavior can be changed by grammar rules (not that I'm aware of). Since the behavior changed over time, I think they changed something in the Atom editor itself. At the current version (0.182.0) this seems to work well

I still get incorrect indentation when I have multi-line function parameters:
fn foo_multi_line(a: i32
                  b: i32) {
                      | <- cursor here

Same problem still exists as described in this post https://discuss.atom.io/t/auto-indent-slightly-off/18198

I just committed some basic indent pattern settings which I took from the language-c package. Indentation behavior is changed with the increaseIndentPattern and decreaseIndentPattern settings. This current settings should work for most cases, but probably could need some adjustments for Rust specifics.
Unfortunately I don't have much spare time to dig deeper into it, so I'd appreciate any help.
Issue #35 and #44 are also related to this.


Fix order of actions in quick-review-buttons

Fixes #5010
Test URLs

https://togithub.com/refined-github/sandbox/pull/4/files

Screenshots

Open PR

Before
	After

Own open PR

Before
	After

Approve should stay on the right to preserve our original order. Also because the positive action is usually on the right

I kinda changed my mind on this change in the meanwhile, but let's go forward because the hotfix has been there for a week already. We can review (undo) this in a few months and move the "Comment" button to the right.
The issue was specifically changing the right button’s action without changing its style. This PR still helps do this change in two phases.


data-menu-offset percentage offset

currently, data-menu-offset can't handle percentage offsets.
I fixed it by changing line 134 to:
                //Is it a percentage offset?
                if(/p$/.test(menuOffset)) {
                    targetTop += (menuOffset.slice(0, -1) / 100) * document.documentElement.clientHeight;
                } else {
                    targetTop += +menuOffset;
                }

Can you add this to the source?

You have a fluid-height navigation? Or what's your use-case?

the height of the top bar changes due to a skrollr action (i'm not using it
as navigation bar, but as logo bar, but still, idea is the same). See
http://lottehoogers.nl/jadoe/
This site uses the skrollr menu library that I adapted.
BR
Leo
On Fri, Mar 20, 2015 at 10:09 AM, Alexander Prinzhorn <
<EMAIL_ADDRESS>wrote:

You have a fluid-height navigation? Or what's your use-case?
—
Reply to this email directly or view it on GitHub
https://github.com/Prinzhorn/skrollr-menu/issues/71#issuecomment-83960401
.

It is also not possible to give an offset other than px - e.g. em/rem - it would be great if skrollrmenu would support the possible css values here.


Trouble buffer totally empty for all modes

I have just installed this plugin. When I run :Trouble (or :Trouble <mode> for any mode), the buffer that opens at the bottom of the screen is always totally empty.
I have a properly configured LSP and have opened a file I know to have errors. The diagnostics are showing up for that file with LSP saga.
Here is a minimal vim config that reproduces the issue (nvim -u minimal.vim).
minimal.vim
call plug#begin('/home/marcel/.config/nvim/plugged')
	Plug 'neovim/nvim-lspconfig'
	Plug 'folke/trouble.nvim'
call plug#end()

lua << EOF
  require'lspconfig'.rust_analyzer.setup { }

  require("trouble").setup {
    -- your configuration comes here
    -- or leave it empty to use the default settings
    -- refer to the configuration section below
  }
EOF

LspInfo
 Use [q] or [Esc] to quit the window

 Language client log: /home/marcel/.cache/nvim/lsp.log
 Detected filetype:   rust

 1 client(s) attached to this buffer:

 Client: rust_analyzer (id: 1, pid: 521209, bufnr: [1])
 	filetypes:       rust
 	autostart:       true
 	root directory:  /home/marcel/code/rust_motd
 	cmd:             rust-analyzer

 Configured servers list: rust_analyzer

nvim version: v0.8.0-dev
trouble version: 929315ea

I have the same/similar issue. Trouble was previously working for me.
:lua = vim.diagnostic.get() shows the proper diagnostic list. I have no clue how to even start figuring out what's going on.

I have the same issue. Did either of you find a solution?

I have it set up with lsp-zero and have the same issue

I am having the same issue with lazy.nvim :/ Has anyone found a solution?

Actually, I think I might have found the problem. The issue is that trouble will silently save the toggle mode. So if for instance, if you use :TroubleToggle quickfix first (which comes up empty), and then use :TroubleToggle again, it will automatically use :TroubleToggle quickfix. Try :TroubleToggle workspace_diagnostics and see if that works.

Development on the main branch is EOL.
Trouble has been rewritten and will be merged in main soon.
This issue/feature either no longer exists or has been implemented on dev.
For more info, see https://github.com/folke/trouble.nvim/tree/dev


Contact us page form is not responsive for mobile,mac's and ipad users

Me a GSSOC'24 extended member can you assign it to me

sure go ahead
Make sure to join the discord server for further communication
Don't forget to star the repository
Complete the task in next 2 days and kindly mention your issue # in your pr like this:
Closes #IssueNumber
Note: the level of the issue may change based on the quality of the PR code
Also please follow the contributing.md steps properly


systemd target support

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: 7
Ruby: 3
Distribution: Fedora
Module version: 3.8.0

How to reproduce (e.g Puppet code you use)
What are you seeing
Can native support for enabling/disabling start/stop systemd targets be added?
What behaviour did you expect instead
Output log
Any additional information you'd like to impart
In many ways these can be treated like other units, but it is helpful to have explicit control over targets like nfs.target or nfs-server.target.

good call, I was looking in the wrong place.


Adding observer boots model before request is run through middleware

Laravel Version: 5.3.26
PHP Version: 7.0

Description:
Observers make models boot before request runs through middleware.
In my Service Provider's boot method, I am registering an observer like this:
User::observe(UserObserver::class).
This User model also has a trait that registers a global scope for the class, and this global scope is depending on a request variable set by a global middleware. Specifically, this is using the Landlord package for multi-tenancy. The middleware sets the tenant_id, the models should then use this tenant_id in their global scopes. But that's an implementation detail and besides the point.
My problem is that the observer is registered when the service provider boots, and this is before the request is passed through the middleware. When calling User::observe, a new instance of the class is instantiated so we can call $instance->getObservableEvents. Instantiating the class effectively also calls boot on it, before middleware is run.
Not sure if this is intended but I think this behavior is inconsistent with how the other model event listeners work. It is possible to add listeners to all the default model events without booting the model. User::creating for instance.
My workaround currently looks like this:
Event::listen('eloquent.booted: ' . User::class, function () {
    User::observe(UserObserver::class);
});

This ensures that the observer is not registered until after the model has been booted, instead of having the process of registering the observer actually boot the model.
A possible solution would be to make Model->getObservableEvents() static, but that would in turn cause problems for manually setting custom observable events.

Just ran into this last night, same version of Laravel. I was setting up observers in a service provider, and then adding a global scope based on Request::route() values.
I ended up setting up my observers inside of my eloquent models in the boot function, which seemed like an ok workaround at the time (as it becomes very verbose that this model has an observer), but definitely feels like one shouldn't be forced to avoid service providers.

I noticed when hooking into a models saving event with an observer while attaching a relation, the relation won't get attached if it is accessed within the observer's saving method. Might be related to this issue.

This looks like intended behavior to me. If you think of it in reverse, if the model did not boot when it was created, that would create unexpected behaviour as well. So it has to be one or the other.
The workaround listed seems perfectly valid when you need this functionality.
Please let me know if you think there is something I have not considered in relation to this.


TypeError: Cannot read properties of undefined (reading 'Event')

Running into this issue when trying to use the opensea-js library.  The error only shows up on build (svelte-kit).
It looks like it's erroring out when trying to import schemas/main/CryptoKitties/index.ts
It's saying that Web3.abi is undefined:
...
  events: {
    transfer: [{
      type: Web3.AbiType.Event,
      name: 'Transfer',
      target: '0x06012c8cf97bead5deae237070f9587f8e7a266d',
      anonymous: false,
      inputs: [
...

I have already installed web3 separately and made sure it's present in node modules.  I'm not sure why that import would be coming back empty.

If others face this solution.  A manual replace of AbiType.Event with 'event' and AbiType.Function with 'function' resolved these issues.  Looks like the AbiType dict is exported at dist/index.js but for some reason with vite that wasn't working.


Unexpected redirect getting crumb

Bug Report
Describe the bug
Hello,
Everything is ok when I run my project on local in docker.
When I put the same project on a vps, always with docker, but through intermediate nginx server I have the following error when the lib getting crumb:
Unexpected redirect to https://finance.yahoo.com/quote/AAPL?guccounter=1

I don't know why the request doesn't return the url with collectConsent?sessionId=XXX...
Minimal Reproduction
const response = await yahooFinance.quoteSummary(`${ticker}`, {
      modules: ["assetProfile", "price"],
});

Environment
Browser or Node: Node
Node version: 14.17
Npm version: 6.14.15
Library version: 2.7.0
Additional Context
VPS on Hostinger
Docker
Nginx
HTTPS / SSL

Hum ok I see; my VPS was located in France, but after your response I made a transfer to USA, and now its works like a charm !
Thanks !


Future of mobile-toolkit for angular v4+ and cli v1+

I am interested in how all this affects the Mobile-toolkit Project.
Correct me if I'm wrong, but it seems that the mobile-toolkit was stagnated, in part, by dependency on angular-cli (--mobile flag removed) and  angular-universal (used for app-shell)(used for app-shell).
So now that<EMAIL_ADDRESS>merged the angular-universal core.. I am curious about the future of the mobile-toolkit.
I haven't seen any major new commits or updates to the angular-universal repo since the platform-server update. Does anyone here know the angular-universal repo/project discontinued? If so, does that mean the mobile-toolkit is as well?
Thanks

The return of --mobile flag is on hold. It's connected with architecting the more general idea of different platform types support in the CLI. It's under design at the moment. But the absence of this flag is not a blocker for using Mobile Toolkit.
The Mobile Toolkit contains 2 packages at the moment: @angular/service-worker and @angular/app-shell. Let me explain what's happening there.
@angular/app-shell
It called App Shell, but in the context of PWA it's actually "Advanced" App Shell. It's not about just providing a simple offline version of the app UI, but optimizing it for the first load (via server-side rendering) and providing better developer experience (by shellRender and shellNoRender directives). Unfortunately, this functionality is not yet integrated into CLI. But you can make it working by the manual setup. Bottomline: @angular/app-shell is not integrated into CLI yet, but we can build an actual, say, classic App Shell using...
@angular/service-worker
Now the adding of Angular Service Worker (NGSW) to your angular app is done by adding
"serviceWorker": true to .angular-cli.json, "apps[0]"
manually or via ng set apps.0.serviceWorker=true
Please notice, this way the idea was changed: instead of initial scaffolding --mobile app, now it's adding the PWA functionality to your existing (of newly generated) app.
After this setting, you have the App Shell of your app ready. Everything is done automatically (the offline version of your UI).
In addition to Static content cache (the base of App Shell idea), NGSW exposes useful methods for you to cover some other aspects of PWA (Push and Dynamic content caching at the moment). And it's under active development.
I'll present on this topic at ng-conf Automatic Progressive Web Apps using the Angular Mobile Toolkit - there will be a livestream available.
There is my Angular Mobile Toolkit workshop doc available with more details (a bit outdated, though).
Lack of fresh NGSW documentation is the main issue. I'm working on PR with the updated instructions.

Right now is not working. I cant import APP_SHELL_DIRECTIVES or providers.

@lautarobock FYI, this is my example repo of Angular 4 w/ platform-server and app-shell, and it works well.
https://github.com/laco0416/angular-plarform-server-example
Currently, shellRender and shellNoRender are provided by AppShellModule. See https://github.com/laco0416/angular-plarform-server-example/blob/master/src/app/app.module.ts and https://github.com/laco0416/angular-plarform-server-example/blob/master/src/app/app.module.server.ts
import { AppShellModule } from '@angular/app-shell';

thanks @laco0416
Now I'm trying to use workers for caching data retrived from an endpoint.
Is that possible right now?

any news? @webmaxru? someone?

Also curious about the current status here. Is there a roadmap for mobile kit?

waiting anxious for more documentation. I find the ngsw.manifest auto generation with webpack a bit difficult to understand.

@Together-Robots you can follow this Google I/O talk for more information https://events.google.com/io/schedule/?section=may-18&sid=5bd70da9-c3b6-4b39-85c2-a8fbe140b7f2.

Another workshop document covering all the current functionality of Mobile Toolkit (except server-side rendered app shell):
https://bit.ly/pwa-angularsummit-2017
I'm working on converting this 52-pages doc to set of Medium articles + set of PRs to the official documentation.

@webmaxru Just so I won't have to wade through all the 52 pages.. could you comment on what the differences are between this new doc and the one you posted earlier https://github.com/angular/mobile-toolkit/issues/138#issuecomment-290423194
Thank you very much for all this by the way!

@somombo I added:

Ignore option in Static content cache plugin
Updated settings for Routing plugin
External plugin
Dynamic content cache plugin
Workflow completely changes (from 3 to 9 steps)
Fixed inconsistencies, mistakes

Thanks!

I think this project has great implemented starting points to get an angular app to work as pwa with a minimum of configuration. There isn't an equal libary to cover the boilerplate code for creating a pwa with angular.
Really good parts:

integration with angular-cli
could be used as webpack plugin
plugin concept with well written plugins:

dynamic
static auto generated and auto merged ngsw-manifest.json
push
...

companion service
bedirectional messaging channels companion <-> service worker

But I miss a roadmap and some continuous implementation progress in the last year. With the release of angular 2 the angular mobile toolkit was announce as the upcoming pwa toolbox for angular. But currently it is marked as experimental.
What are the goals (architectural and functional) of the mobile toolkit and how can they be achieved?
I tested the mobile toolkit for a messenger app and I missed some features. Some of them are allready present as PR (#122, #137, #104). How can the merge process be accelerated?
Missed features:

VAPID support for push
adding custom plugins by ngsw-manifest.json
adding custom plugins for events like notificationclick
core plugins for notification events (click, actions, close)
update on server triggered push
update through timer

What is the best way to support this project? (Under the condition it gets a roadmap...)

@jo3d3v
Very nice list!
Some comments:

Support for VAPID landed a couple of hours ago https://github.com/angular/mobile-toolkit/commit/950f99fd7de9be7ffa657d862617a94c332b77b1 :)
Adding custom plugins/events is possible (in a bit hackish way though), check the last step of my workshop: https://bit.ly/pwa-angularsummit-2017

I think that the vapid support having been added is good, but there's still a lot of work to do to make this library actually usable for push notifications. One of the major things is to refactor the way that events happen to avoid warnings/issues like I raised in #134.
I know that @alxhub is busy, so I don't blame him for the slow progress, but I agree that for this project to succeed there's got to be a better way to contribute. I'd happily contribute in a large way towards making push notifications useful, but all issues/PRs I've opened have effectively been ignored so I'm not sure what course to take to speed things along. (Sorry @alxhub don't mean to be duplicative, but this is probably a better venue to discuss these issues than where we were last talking)

Hey @jo3d3v, I agree with @webmaxru, that's a very cool feature list.

VAPID support for push

Implemented! @treyrich has a good point with https://github.com/angular/mobile-toolkit/issues/134, I will fix this next.

adding custom plugins by ngsw-manifest.json

This is likely not going to be supported. NGSW tries to draw a line between the SW's code and its configuration. Updates to code actually clear the whole set of caches, to ensure a SW never ends up in a broken state where a cache from an old version is incompatible with the new code.
Adding/removing plugins is a code change, and so I'd rather not hide it behind a change in configuration.
There is an additional complication that importScripts doesn't have the same behavior with regard to code changes & the update workflow as the main SW script does, so it's not always clear when imported scripts will update.

adding custom plugins for events like notificationclick

This is supported today by building your own SW using the process that @webmaxru shows, though I have some work to do to iron out typings in this case.

core plugins for notification events (click, actions, close)

Agreed, definitely needed.
update on server triggered push

I really want to do something like this too.

update through timer

This can be accomplished now with the NgServiceWorker API in the application - .checkForUpdates() and the .updates stream allow the app to both initiate the update check and to choose to update before NGSW would.

provide service worker for dev mode not only for prod for testing purpose

I agree, but we need to explore more what kind of experience a SW in dev mode would create.

Thank you for giving a view into the future of service worker development. Sounds great!!! But there are still 2 questions:

What is planed to support by the mobile toolkit?

Only encapsulate service worker boilerplate code?
Or creating APIs beyond that? (Enhanced but outdated AppShell is a very good idea)

How can the community (including me) support this project to accelerate development speed?

Provide qualitative PRs in a coordinated way (a plan/roadmap is needed for that)

Demo App
Docs

...
Review/Merging process for PRs

@jo3d3v,
The "mobile toolkit" as a container for mobile-related tooling will likely be going away. I'm planning on merging @angular/service-worker into the https://github.com/angular/angular repository. The application shell concept itself needs to be rethought and a proper solution will probably look very different.
As for the SW itself, I agree a proper roadmap, documentation site, PR review process, etc are all crucial missing pieces to the project. One of the main challenges with accepting community contributions currently is the lack of a working CI environment, for example. Moving to angular/angular greatly improves this situation.
I'm not sure what you mean by "only encapsulate service worker boilerplate code". It's possible to build your own version of NGSW today, with custom plugins, by running rollup over an entrypoint such as https://github.com/angular/mobile-toolkit/blob/master/service-worker/worker/src/worker/builds/basic.ts.

TL;DR
For anyone not understanding how to use the mobile toolkit (like me), check:
https://bit.ly/pwa-angularsummit-2017

I'll just put this up here
https://www.youtube.com/watch?v=C8KcW1Nj3Mw
Thanks for presenting this @alxhub at google I/O!
Looks like there are a lot of new tools like ng-pwa-tools that are meant to make the workflow easier, yet to be documented though ;-)

@somombo Some clarifications: CLI-way (serviceWorker: true) will remain the main way to add NGSW to the application. ng-pwa-tools most likely will be baked into CLI as well as remain the separate tools for the cases when CLI is not an option.

@webmaxru look forward to seeing in cli

@alxhub,
Thank you for the clarification. Now, I have an idea for the future of mobile toolkit. Once all preparations have be done for development of @angular/service-worker in https://github.com/angular/angular, please propagate it to the community (maybe this issue thread).
With "only encapsulate service worker boilerplate code" I meant all the code you have to write every time for a new PWA to register listeners, create communication channel between service worker and clients... and so on.

/slightly_offtopic: Everyone is welcome to continue this and many other PWA discussions in dedicated PWA slack team with almost 200 developers from different companies (incl. all major browsers and PWA libs): https://bit.ly/pwa-slack

@jo3d3v, et al, I have a preliminary port of @angular/service-worker into the Angular main repo on this branch: https://github.com/alxhub/angular/tree/service-worker

@alxhub is there an example for server push with express using ng-pwa-tools?

update: looks like we are pretty much waiting on https://github.com/angular/angular/pull/17276 to be merged. I expect/hope that once that's done, the angular-cli project will take a fresh look at implementing basic service-worker/pwa scaffolding/flags in the cli.
Hopefully the PR will make it to Angular v5... i haven't seen anything about "pwa" or "service workers" in the latest Angular 5 beta changelogs. Should we still expect this to land in Angular 5, @alxhub ?
Thanks for all your hard work and patience.

Update: Looks like a preliminary port of @angular/service-worker has been merged into the main angular repo see PR https://github.com/angular/angular/pull/19274. This is available now starting Angular v5.0.0-rc.0
It indroduces/includes:

@angular/service-worker: a library for use within Angular client apps to communicate with the service worker.
@angular/service-worker/gen: a library for generating ngsw.json files from glob-based SW config files.
@angular/service-worker/ngsw-worker.js: the bundled service worker script itself.
@angular/service-worker/ngsw-cli.js: a CLI tool for generating ngsw.json files from glob-based SW config files.

Thank you @alxhub! 🎉
Look forward to seeing some guidance on how we can start playing with these!

Final Angular v5 has officially been released See changelog. And it includes the @angular/service-worker package!
Now we wait patiently for some documentation/guidance from @alxhub
@StephenFluin, I found it odd that the official blog post announcement for the v5 release did not mention service-worker or pwa. Hopefully the team is still focused on making easy pwa dev a reality.

@somombo We are definitely focused on making easy PWA dev a reality. We didn't announce the service worker changes because they have not yet landed with CLI support. Stay tuned!

@StephenFluin we really appreciate it! great work and thanks for your timely and continued feedback.

@somombo you can start exploring the new NGSW right now: I've published 2 articles covering more or less everything in the Angular Service Worker v5 (including how to emulate Angular CLI 1.6 NGSW support):
https://medium.com/@webmaxru/a-new-angular-service-worker-creating-automatic-progressive-web-apps-part-1-theory-37d7d7647cc7
https://medium.com/@webmaxru/a-new-angular-service-worker-creating-automatic-progressive-web-apps-part-2-practice-3221471269a1

That's great! Will be sure to check this out. Thanks for the feedback!

Hi everyone, service worker and app-shell have landed stable support in @angular/cli v1.6+
Here are some useful resources:

Official Service worker docs https://angular.io/guide/service-worker-intro
Tutorial by @webmaxru https://alligator.io/angular/service-workers/
Updated .angular-cli.json schema<EMAIL_ADDRESS>
I will now close this issue out. Feel free to comment on the future of app-shell and service worker in the main angular and angular-cli repos


Executing plan with hidden directory fails

I suppose this may or may not be a bug, depending on your point of view.
Description
Executing terragrunt plan against a configuration that requires a hidden directory fails because the hidden directory is not copied to the working directory.
Steps

Create a configuration that requires a hidden directory. In my case, I am attempting to create a Nomad job for an agent that has HTTPS enabled. (See Nomad's Encryption documentation for details.)
resource "nomad_job" "job" {
  jobspec = "${data.template_file.job.rendered}"
}

Set the NOMAD_CACERT environment variable.
$ export NOMAD_CACERT=.ssl/ca.pem

Run the Terragrunt command.
$ terragrunt plan

Expected result
The plan output. Instead, I receive an error:
Error running plan: 1 error(s) occurred:

* provider.nomad: failed to configure Nomad API: Error loading CA File: open .ssl/ca.pem: no such file or directory

Workaround
Two workarounds that I can see.

Copy the cert to the temporary directory.

$ cp -r .ssl /var/folders/1f/x60208f52bv2y2w53_h_pxth0000gn/T/terragrunt/4m5kqybpDM3aaDXlxG3CfuMrXNY/QltMrwpCerDYHpcQxM9XyMTnkMs/nomad/nomad-jobs/.ssl

Export the full path to NOMAD_CACERT.

$ export NOMAD_CACERT=~/projects/infrastructure-live/src/sandbox/apps/demo-server/.ssl/ca.pem

Ah, that's because the CopyFolderContents  method intentionally skips hidden files and folders: https://github.com/gruntwork-io/terragrunt/blob/1b0cb787c0314681e92f5ff083486eb5a988452f/util/file.go#L132. That's intentional to avoid copying .git, .terraform, and other code that you probably don't want copied.
Out of curiosity, why do you keep that file in a hidden folder? Storing secrets in plaintext in a hidden folder is no more secure than in a normal folder...

I see. Well, tbh I inherited it that way. Now that I know this is intentional, I'll make some changes to get it to work.

Ah, got it. Thanks for following up!


16.3 List sort bug

The List<T> length is wrong during sort resulting in it trying to sort on empty entries.
https://deck.net/e5fa533efeaf5347e36f8639b96b021e
public class Program
{
    public static void Main()
    {
        var list = new List<Foo>();

        list.Add(new Foo() { V = 1 });
        list.Add(new Foo() { V = 2 });

        list.Sort((a,b) =>
        {
            return a.V.CompareTo(b.V);
        });
    }

    class Foo
    {
        public int V;
    }
}

Actual Result
Throws a null reference as it'll try sort 6x null items (pre-allocated array space) rather than just the two entries.
 TypeError: Cannot read property 'V' of null
    at https://deck.net/RunHandler.ashx?h=1614692329:16:45
    at Array.sort (native)
    at Object.sort (https://deck.net/resources/js/bridge/bridge.min.js?16.3.0:7:221597)
    at ctor.sort$2 (https://deck.net/resources/js/bridge/bridge.min.js?16.3.0:7:262950)

@sichbo – Thanks for the report. We will investigate right away.

@sichbo – A pull request with a fix for this defect has been submitted.

Thanks for the quick fix guys!
Just a suggestion if you felt like it you could probably avoid potentially expensive array allocations by pretending to have an IComparer, something along the lines of:
        public void Sort(Comparison<T> comparison)
        {
            if (comparison == null)
            {
                throw new System.ArgumentNullException("comparison");
            }

            if (_size > 1)
            {
                Func<T,T,int> comparer = (T a, T b) => { return comparison(a, b); };
                Array.Sort(_items, 0, _size, new {
                    compare = comparer
                }.As<IComparer<T>>());
            }
        }

Emits
            sort$2: function (comparison) {
                if (Bridge.staticEquals(comparison, null)) {
                    throw new System.ArgumentNullException("comparison");
                }

                if (this._size > 1) {
                    var comparer = function (a, b) {
                        return comparison(a, b);
                    };
                    System.Array.sort(this._items, 0, this._size, new $asm.$AnonymousType$1(comparer));
                }
            },

Rock on.


train realersgan -> torch error

Hello,
while training a real-ersgan model
08-12-2024 12:00 AM | INFO: [ epoch:   0 ] [ iter: 32,600 ] [ performance: 4.022 it/s ] [ lr: 8.00e-04 ] [ eta: 2 days, 18:51:02 ] [ l_g_mssim: 4.7528e-01 ] [ l_g_consistency: 4.9110e-01 ] [ l_g_percep: 3.5055e-01 ] [ l_g_gan: 1.7565e+00 ] [ l_g_total: 3.0734e+00 ] [ l_d_real: 4.9024e-08 ] [ out_d_real: 1.7784e+01 ] [ l_d_fake: 1.5977e-10 ] [ out_d_fake: -1.7565e+01 ] [ l_d_total: 2.4592e-08 ]
08-12-2024 12:00 AM | INFO: [ epoch:   0 ] [ iter: 32,700 ] [ performance: 4.065 it/s ] [ lr: 8.00e-04 ] [ eta: 2 days, 18:50:27 ] [ l_g_mssim: 5.6157e-01 ] [ l_g_consistency: 5.9041e-01 ] [ l_g_percep: 3.6276e-01 ] [ l_g_gan: 1.7891e+00 ] [ l_g_total: 3.3039e+00 ] [ l_d_real: 3.6554e-08 ] [ out_d_real: 1.8129e+01 ] [ l_d_fake: 4.5475e-11 ] [ out_d_fake: -1.7891e+01 ] [ l_d_total: 1.8300e-08 ]
08-12-2024 12:01 AM | INFO: [ epoch:   0 ] [ iter: 32,800 ] [ performance: 4.061 it/s ] [ lr: 8.00e-04 ] [ eta: 2 days, 18:49:56 ] [ l_g_mssim: 5.6816e-01 ] [ l_g_consistency: 6.6751e-01 ] [ l_g_percep: 3.6129e-01 ] [ l_g_gan: 1.8294e+00 ] [ l_g_total: 3.4264e+00 ] [ l_d_real: 2.9898e-08 ] [ out_d_real: 1.8301e+01 ] [ l_d_fake: 5.4570e-12 ] [ out_d_fake: -1.8294e+01 ] [ l_d_total: 1.4952e-08 ]
Traceback (most recent call last):
  File "/home/XXX/neosr/neosr/train.py", line 374, in <module>
    train_pipeline(str(root_path))
  File "/home/XXX/neosr/neosr/train.py", line 280, in train_pipeline
    model.optimize_parameters(current_iter)  # type: ignore[reportFunctionMemberAccess,attr-defined]
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/XXX/neosr/neosr/neosr/models/image.py", line 702, in optimize_parameters
    self.closure(current_iter)
  File "/home/XXX/neosr/neosr/neosr/models/image.py", line 607, in closure
    torch.nn.utils.clip_grad_norm_(
  File "/home/XXX/miniconda3/envs/neosr/lib/python3.12/site-packages/torch/nn/utils/clip_grad.py", line 30, in _no_grad_wrapper
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/home/XXX/miniconda3/envs/neosr/lib/python3.12/site-packages/torch/nn/utils/clip_grad.py", line 99, in clip_grad_norm_
    raise RuntimeError(
RuntimeError: The total norm of order 2.0 for gradients from `parameters` is non-finite, so it cannot be clipped. To disable this error and scale the gradients by the non-finite norm anyway, set `error_if_nonfinite=False`

This came after some time. Call:
python train.py -opt options/train_esrgan_otf_v1.toml
and the .toml file is standard, only image path was adjusted.
[datasets.train]
type = "otf"
dataroot_gt = '/home/XXX/neosr/neosr/GENAI_conv-to-jpg_hardlinks-multiscale-sub_selected'
patch_size = 64
batch_size = 8
#accumulate = 1

No eval, ie. commented out
#[datasets.val]
etc...

network standard:
[network_g]
type = "esrgan"

[network_d]
type = "ea2fpn"

OS: Debian sid, nvidia rtx4090
Nothing else was changed.
How can this be solved?
Thanks!

Hi. Please update neosr, this has been fixed a few days ago. I'd also recommend using the new configuration files provided in options/, they have been changed as well.


Migrate client id

Description
update client id kSecAttrAccessible to kSecAttrAccessibleAfterFirstUnlockThisDeviceOnly, so it can be accessed when app is in background. - potential fix for a crash.
How Has This Been Tested?
manually:

build app with client id kSecAttrAccessibleWhenUnlockedThisDeviceOnly as on production
client Id keychain item is updated after running with the following changes

Due Dilligence

[ ] Breaking change
[ ] Requires a documentation update

Closing for now as this may fix the issue and migration may be not needed.


video and audio not working

My screens working properly as when new user join video calling, new user with default user screen apperars. But video or audio transmission not working.

@bacancy please use and refer to maintained version of opentok gem here: https://github.com/stijnster/opentok


Invalid arguments: PubAdsService.refresh([]).

Version: 0.20.0
When calling DFPManager.refresh(); in a useEffect I get the following error google tag console:

Edit:
When I pass a slot id it correctly refreshes that slot. The docs mention that you do not need to explicitly set the slot to refresh. Is this a bug or maybe I have configured something wrong.

I don't know if this is related, but I found the functionality to refresh ad has changed in recent version. Instead of passing gptSlot as expected by PubAdsService, it is now passing slotId.
https://github.com/jaanauati/react-dfp/blob/e5b46729e1b92178027008c849440f15fb0edde6/js/manager.js#L360-L362
This used to work up to version 0.20.0:
https://github.com/jaanauati/react-dfp/blob/126d4fea858ba34982d76e752b7a374ac58698fb/js/manager.js#L356-L359

@squgeim thanks for diagnosing the issue.
@daraclare can you take a look at this bug. I tried updating to the latest version and am getting this error as well.

@jaanauati
Please inform us on any updates for the deprecated google functions, we are facing the same warnings in our project.


Install, uninstall CLI

With meteor I dearly miss the simplicity of npm install {pkg} and npm uninstall {pkg}
My OCD brain was hating having to look up the latest version of packages and manually add them to my packages.json file, so I made a little CLI so I can ./cli install {pkg} and ./cli uninstall {pkg} but this is not the most elegant of solutions
Any thoughts on a possible better solution that can be worked into this package? My quick and dirty CLI tool below
#!/usr/bin/env coffee

# USAGE:
#   ./cli install chalk
#   ./cli uninstall chalk

exec = require('child_process').exec
fs = require 'fs'

argv = process.argv

handleError = (error) ->
  throw error if error

command = argv[2]

switch command
  when 'install'
    pkg = argv[3]

    exec "npm show #{pkg} version", (error, output) ->
      handleError error
      latestVersion = output.trim()

      fs.readFile 'packages.json', 'utf8', (error, contents) ->
        handleError error

        json = JSON.parse contents
        json[pkg] = latestVersion
        newString = JSON.stringify json, null, 2
        newString += '\n'

        fs.writeFile 'packages.json', newString, (error) ->
          handleError error

          console.info "Latest #{pkg} added to ./packages.json"

  when 'uninstall'
    pkg = argv[3]

    fs.readFile 'packages.json', 'utf8', (error, contents) ->
      handleError error

      json = JSON.parse contents
      if json[pkg]
        delete json[pkg]
      else
        console.warn "Huh? #{pkg} is not in your packages.json"
        return

      newString = JSON.stringify json, null, 2
      newString += '\n'

      fs.writeFile 'packages.json', newString, (error) ->
        handleError error

        console.info "#{pkg} removed from packages.json"

This is probably not possible without this - https://github.com/MeteorCommunity/discussions/issues/9
But who knows if there is a workaround anyone can think of in the interim?


Show dataset contents in reference pages

Please select a category the issue is focused on?
Function Documentation
Let us know where something needs a refresh or put your idea here!
pharmaversesdtm doesn't show column-by-column breakdowns of datasets:

But pharmaverseadam does:

It would be good to add it to the former.

@vbshuliar pls could you work on this instead?


Updated package to newest

Description
Until now we have used an alpha package for handling Integration Events. Now i have upgraded to newest version, which basically removes the ShouldHandle handle
References

#0000

Codecov Report

Merging #415 (dc86e07) into main (21978a1) will decrease coverage by 1.83%.
Report is 115 commits behind head on main.
The diff coverage is 73.19%.

@@            Coverage Diff             @@
##             main     #415      +/-   ##
==========================================
- Coverage   83.07%   81.24%   -1.83%
==========================================
  Files         456      560     +104
  Lines       10418    13076    +2658
==========================================
+ Hits         8655    10624    +1969
- Misses       1438     2051     +613
- Partials      325      401      +76

Files Changed
Coverage Δ

...ivedMessages/ArchivedMessageGetDocumentListener.cs
0.00% <ø> (ø)

.../ArchivedMessages/ArchivedMessageSearchListener.cs
0.00% <0.00%> (ø)

source/Api/Configuration/InboxMessageMetadata.cs
0.00% <0.00%> (ø)

...ation/Middleware/Correlation/CorrelationContext.cs
66.66% <ø> (ø)

...uration/Middleware/UnHandledExceptionMiddleware.cs
0.00% <0.00%> (ø)

source/Api/Configuration/SystemTimer.cs
0.00% <ø> (ø)

source/Api/EventListeners/InboxEventListener.cs
0.00% <0.00%> (ø)

...rce/Api/EventListeners/IntegrationEventListener.cs
0.00% <0.00%> (ø)

...essages/RequestAggregatedMeasureMessageReceiver.cs
0.00% <0.00%> (ø)

...equestAggregatedMeasureTransactionQueueListener.cs
0.00% <0.00%> (ø)

... and 169 more

... and 143 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


How to change the dot on an event date to text or number

How to change the dot on an event date to text or number
My code, i use it to create a dot, it ok:
override fun decorate(view: DayViewFacade) {
            view.setBackgroundDrawable(resources.getDrawable(R.drawable.current_day))
            view.addSpan(DotSpan(5f, color))
        }

But i want to create a text or number instead of dot, i try use:
view.addSpan("text here")
Nothing is added.  Do we have any solution to this problem?

This question has been asked a couple of time. Please check previous responses thank you.

i solved it

But i have a problem. I put lunar date on each solar date, performance is very slow ....

Decorators are definitely not the best in terms of performance atm, we have a couple of issues that might help you, but improving it would requires more effort in general. Feel free to post your solution here on how you solved your first problem. Thanks, closing this now.


movie test 5 strange results on Windows

I've attached a picture of the results on Windows, and the expected results.
Windows results.

Expected results

Discovered this has nothing to do with movies.
Certain image sizes produce the wrong results, in fact produce images with different size than requested: the movie test requests a widthxheight of 76x63 for Window 2 (upper left corner).
Actual image size is  116x63 (for all image formats we support writing).
I tried saving at 100x100 and got 116x100.
There appears to be a minimum width of 116.
Requesting 116x63 yielded the correct image.


(fix) apply .wordwrap class to school URL

before:

after:

 Apply word-wrap to URLs in school details


[bug]: If runtime directory exists, configuration script does not create the initfile

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Linux
GPU
cuda
VRAM
16
What happened?
Until recently, if the ~/.invokeai init file did not exist, the invoke.py script would automatically proceed to create the file (albeit without correct defaults), and continue to load the CLI or web UI.
After a recent change (possibly #1517?), the invoke.py script tries to run configure_invokeai.py to reconfigure the root dir. But if the model config directory already exists (specifically, it checks for the configs/stable-diffusion/v1-inference.yaml file), the logic in the configure script bypasses the code path that creates the initfile altogether.
This means if we're missing the init file, but have an existing runtime dir, the application can never load, because it goes into an infinite loop with emergency_model_reconfigure, looking for the initfile but never creating it.
Potential workaround is to comment out https://github.com/invoke-ai/InvokeAI/blob/development/scripts/configure_invokeai.py#L697, which causes full re-initialization of the runtime dir even if it exists.
Screenshots
No response
Additional context
No response
Contact Details
No response

I started fixing this in #1614, but still not quite working because Args.parse_args() uses different logic for finding the init file. Can aim to finish it later today... :sleeping:

Fixed somewhere along the way


Readmefix

http://www.urbandictionary.com/define.php?term=grip

looked like a typo to me. sorry

No worries. You're not the first, so I probably should change it anyway :)


python311Packages.pandas: 2.1.0 -> 2.1.1

Description of changes
Update pandas.
Changelog: https://github.com/pandas-dev/pandas/releases/tag/v2.1.1
cc @7c6f434c @FRidh @knedlsepp
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Release notes are at https://pandas.pydata.org/pandas-docs/version/2.1.1/whatsnew/v2.1.1.html.


Create readme for Product cards

How to create a Readme for Product cards ?

Copy each component code and paste it in Tailwind Playground
Take a screenshot of that. (Make sure to have a proper aspect ratio. 16:9 will be good) and add it to the readme in the table format
Make sure to enclose the screenshot under anchor tag so that one can just choose the product card and after clicking that image will be redirected to the code of that component.
NOTE:- Make sure each component screenshot is of equal size so that it fits in the layout.

For Reference you can see Button Readme

Please assign me @Rajdip019


Hugo quickstart

Works like Hugo new site, but prepopulates with the Hyde theme and an intro doc as content.
Could also start server mode and open a browser to the site.
Bam! Welcome to Hugo.

I think how DocPad provides a list of pre-built skeletons for the user to choose from when docpad run is first run in an empty directory could be one way to approach this.  I remembered being rather impressed by it when I first played with it some months ago.

Bundling a theme would increase the size of the Hugo file. Probably not much at first, but it's more likely to grow features over time than shed them.
I think that it would be better to put this into a Hugo installer rather than into Hugo itself. That would decouple the theme from the Hugo development. Plus having an installer automatically makes a project look more mature.

A Hugo installer is a good idea. We should make an issue for that.
--
Steve Francia
http://stevefrancia.com
http://spf13.com
http://twitter.com/spf13
On Thu, Apr 2, 2015 at 12:24 PM, Michael Henderson<EMAIL_ADDRESS>
wrote:

Bundling a theme would increase the size of the Hugo file. Probably not
much at first, but it's more likely to grow features over time than shed
them.
I think that it would be better to put this into a Hugo installer rather
than into Hugo itself. That would decouple the theme from the Hugo
development. Plus having an installer automatically makes a project look
more mature.
—
Reply to this email directly or view it on GitHub
https://github.com/spf13/hugo/issues/505#issuecomment-88965503.

@ahsanulhaque what is the status of this?

Bundling a theme would increase the size of the Hugo file. Probably not much at first, but it's more likely to grow features over time than shed them.

We don't have to include the theme directly into the Hugo binary. Instead we could setup a Github repository that contains a small example website. Github allows us to download the latest (stable) source from master as a zip file.
This only needs to be extracted and all file copied to the right places. This way we can update the example website without being dependent on an installer or the Hugo binary.
The repo of the example site could use tags to indicate releases. This way we avoid compatibality issues by fetching the right version based on the tag in the repo and the version of the Hugo binary.

This issue has been automatically marked as stale because it has not been commented on for at least six months.
The resources of the Hugo team are limited, and so we are asking for your help.
If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.
If this is a feature request, and you feel that it is still relevant and valuable, please tell us why.
This issue will automatically be closed in the near future if no further activity occurs. Thank you for all your contributions.


Consider adding default expression for empty expressions

Accumulo supports the functionality of setting a default expression for the empty expression.  Need to determine if this library should offer a similar functionality that would use a configured access expression when it sees an empty expression.
This could look like the following the API, where when the empty expression is encountered the expression C&D is used instead.
        var evaluator = AccessEvaluator.builder()
                                       .authorizations(authsSet)
                                       .emptyExpressionDefault("C&D")
                                       .build();

Could possibly have an option to disallow the empty expression instead of this, where it would throw an exception when seeing the empty expression.  Something like the following.
        var evaluator = AccessEvaluator.builder()
                                       .authorizations(authsSet)
                                       .failOnEmptyExpression(true)
                                       .build();

I don't think we really need the default expression, since the caller can easily swap out the empty expression it sees in the data with one it is configured to use instead, if it really wanted to, before it calls the evaluator. That's not really something the evaluator needs to do. The same is true about the failing on empty expressions. It'd be weird for the caller to intentionally pass an empty expression to this library, and then expect this library to throw the exception, rather than it throw the exception itself prior to passing it to the library in the first place.  I think these features would just bloat the API, without adding much utility to calling code.


titan.properties on classpath results in startup failure.

Seen in 0.9.M1
Having a titan.properties on the classpath causes a failure on startup.
Exception in thread "main" java.lang.ExceptionInInitializerError
	at com.thinkaurelius.titan.graphdb.configuration.GraphDatabaseConfiguration.<init>(GraphDatabaseConfiguration.java:1296)
	at com.thinkaurelius.titan.core.TitanFactory.open(TitanFactory.java:91)
	at com.thinkaurelius.titan.core.TitanFactory.open(TitanFactory.java:71)
	at runnable.TestConfBug.main(TestConfBug.java:21)
Caused by: java.lang.UnsupportedOperationException
	at com.google.common.collect.ImmutableMap.put(ImmutableMap.java:326)
	at com.thinkaurelius.titan.diskstorage.Backend.registerShorthands(Backend.java:726)
	at com.thinkaurelius.titan.diskstorage.Backend.<clinit>(Backend.java:717)
	... 4 more

The problem is that an ImmutableMap is returned and passed on the following line:
registerShorthands(props, "storage.", StandardStoreManager.getAllManagerClasses())

...then registerShorthands calls put on the Immutable Map
shorthands.put(shorthand, clazz);

...which results in the UnsupportedOperationException.
Taking a step back, I wonder if auto-loading titan.properties is actually useful (but I'm not clear on it's original purpose).

Thanks for the report.  I think this involved two separate problems:

When I made the maps of shorthand->classname immutable, I didn't account for the code in the trace, which attempts to put custom entries from titan.properties into those maps

titan.properties is a collision-prone name; it's too likely that a user will name their config file titan.properties to make assumptions about its contents (in this case, that it contains index and/or storage backend shorthand name-to-class mappings)

Defining custom shorthand names for backends is useful, but centralizing this info in a single file is not a great way to go about it.  Also, I'm not sure whether anyone actually cares about the custom shorthand feature (which is probably undocumented).  I went back and forth on this in my commits linked above, but in the end I removed the custom shorthand feature altogether.
If there's any demand for custom shorthand definition, then we might want to look at an alternative that allows us to spread these mappings out across multiple files, so that a preferred shorthand name can ship in the same jar that contains the implementation (maybe but not necessarily ServiceLoader).