SOUNDS: Use 16 bits per sample

Description
Makes sounds sound less bad.
Fixes #1514
Type of change

[x] 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 not work as expected)
[ ] This change requires a documentation update
[ ] Code quality improvements

How to Test
Deploy locally and test following code on playground with sounds library enabled:
const test_sound = sine_sound(400, 0.4);

const soft_sound = make_sound(t => get_wave(test_sound)(t) / 2 / 2 / 2 / 2 / 2 / 2, get_duration(test_sound));
play(soft_sound);

It should not sound terrible (like a sawtooth/square wave) and instead should just sound like a soft sine wave.
Checklist
Please delete options that are not relevant.

[x] I have tested this code

Pull Request Test Coverage Report for Build 7625

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 32.592%

Totals

Change from base Build 7622:
0.0%

Covered Lines:
2270

Relevant Lines:
6424

💛  - Coveralls

Fixed.


Version bump for next release

It was actually #118, but it actually didn't cause mpt_trie itself to require a major bump but trace_decoder I think because we added a new enum variant to an existing enum?
--- failure enum_variant_added: enum variant added on exhaustive enum ---

Description:
A publicly-visible enum without #[non_exhaustive] has a new variant.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#enum-variant-new
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.30.0/src/lints/enum_variant_added.ron

Failed in:
  variant TraceParsingError:TrieOpError in /home/brendan/work/predicate_labs/our_repos/zk_evm/trace_decoder/src/decoding.rs:63
     Summary semver requires new major version: 1 major and 0 minor checks failed

It was actually #118, but it actually didn't cause mpt_trie itself to require a major bump but trace_decoder I think because we added a new enum variant to an existing enum?
--- failure enum_variant_added: enum variant added on exhaustive enum ---

Description:
A publicly-visible enum without #[non_exhaustive] has a new variant.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#enum-variant-new
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.30.0/src/lints/enum_variant_added.ron

Failed in:
  variant TraceParsingError:TrieOpError in /home/brendan/work/predicate_labs/our_repos/zk_evm/trace_decoder/src/decoding.rs:63
     Summary semver requires new major version: 1 major and 0 minor checks failed

Makes sense. I just couldn't place what the change was in my head


€ and other html codes

Would be interesting to know what browser support what.

These should be universally supported, see comment at #5503 for details.


Improve performance

Right the time from original message to box message is very high.

https://martinheinz.dev/blog/64

it looks like I can't really improve performance
as most of the time is not spent on my end of the program (which takes <100ms)
2026b6ac14

  _     ._   __/__   _ _  _  _ _/_   Recorded: 19:30:58  Samples:  29
 /_//_/// /_\ / //_// / //_'/ //     Duration: 0.961     CPU time: 0.028
/   _/                      v4.1.1

Program: sadpandabot.py

0.960 _run_once  asyncio/base_events.py:1815
├─ 0.499 _run  asyncio/events.py:78
│     [23 frames hidden]  asyncio, aiohttp, nio
│        0.367 _drain_helper  aiohttp/base_protocol.py:78
│        └─ 0.367 [await]
│        0.098 _handle_joined_rooms  nio/client/async_client.py:564
│        └─ 0.098 on_message  sadpandabot.py:52
│           └─ 0.098 parse_exlinks  sadpandabot.py:64
│              ├─ 0.072 get_galleries  ehapi.py:14
│              │  └─ 0.072 api_gallery  ehapi.py:51
│              │     └─ 0.072 post  requests/api.py:105
│              │           [61 frames hidden]  requests, urllib3, socket, <built-in>...
│              └─ 0.018 fetch_img  sadpandabot.py:35
│                 └─ 0.018 __aenter__  aiohttp/client.py:1137
│                       [10 frames hidden]  aiohttp, asyncio
└─ 0.461 select  selectors.py:452
      [3 frames hidden]  selectors, <built-in>
         0.461 epoll.poll  <built-in>:0


fix(@angular/build): improve URL rebasing for hyphenated Sass namespaced variables

Sass variable namespaces can contain either a hyphen or underscore in the namespace identifier. The URL rebasing support within the Angular CLI will now account for these type of namespaces and rebase the evaluated URL as expected.

The changes were merged into the following branches: main, 18.2.x, 19.0.x


Dungeon Crawl Loot Replaced

The dungeon crawl mod (https://www.curseforge.com/minecraft/mc-mods/dungeon-crawl) generates chests without custom loot in them

Can you specify what version of Minecraft you're having this issue with, along with what version of Lootr?

forge 1.19.2 43.3.7
lootr <IP_ADDRESS>
dungeon crawl 2.3.12

can confirm this is happening with dungeon crawl on NeoForge as well latest version of both mods on cf as of today

It's taken me a little while to get to this, but from what I can gather looking through the code, the issue seems to be that the customPersistentData is being discarded during conversion. I'm not sure if that will be the entire fix, but I'll include something in the next 1.20 release for Forge simply for testing purposes.
Unfortunately, the NeoForge/1.20.4/1.20.6 versions are quite a bit behind the main branch and will take a bit more effort.

@marcellonovak I believe this should be fixed with the latest release for 1.19.
@Saereth As above, this should be fixed for the 1.18.2, 1.19 and 1.20 releases. Unfortunately, with the 1.20.4 -> 1.20.6 release, I'm not entirely sure what I'm going to do with 1.20.4 releases -- as there's already a huge chunk of code that I still have to port.

No worres, the pack im working on in neoforge 1.20.4 so whichever way you go is understandable, nice if it could get sorted but if not its not the end of the world


[BUG]: how to infer？

🐛 Describe the bug
i have finished the process of train. i just want to ask how to infer?could u please show the code?
Environment
No response

Hi @sanwei111 Thanks for your proposal, this is a similar issue at #2868 . We will fix it soon.


feat: initial implementation of caching feature

This is an initial implementation of caching feature using lru crate.
The load test is needed for production use. Also, this caching feature should be shipped as a separated feature as users can select the compiled features.
This is related to #55 (Ha, I think I should write an article on rpxy after this PR is done.)

This is huge :+1:

I just upgraded caching feature in the hybrid manner of temporary file cache and on-memory cache.

With the current features, I believe rpxy is ready and complete for our usage scenario. As soon as I can, I will proceed with benchmarks to compare our current production scenario (nginx with proxy cache) and rpxy with cache enabled.

Hi Paulo, Thanks! I hope this version works flawlessly and faster than your current setup! Please let me know if you find anything that should be updated or additionally needed.

I will surely do!


init script doesn't have data-dir (0.5.0)

I never used consul 0.4.0, so this may be me missing something, but the current init script bundled here doesn't work.
consul expects a data-dir option to be passed to it, which isn't included:
[root@host tmp]# tail /var/log/consul
==> Must specify data directory using -data-dir
==> Must specify data directory using -data-dir
==> Must specify data directory using -data-dir
==> Must specify data directory using -data-dir

The module should probably include support for setting this up, too

Never mind, bad config


feat: add schema tests and official JSON Schema

includes pass and fail scenarios for Overlay schema tests.
Also includes the full schema per #72

@jeremyfiel Could you please add a description to one of the Action Objects?
That's the only part of the schema that is not yet tested.

should be all covered now

should be all covered now

Sorry, should have been more precise: please add to a "pass" test case, the "fail" ones don't count into the schema coverage.

removed constraints for update/remove, updated tests

@jeremyfiel How can I view the diff between this state and the last state I reviewed?

There is a compare hyperlink just above our comment box.  It's not available on mobile so I don't have it to share. It's available on desktop though


Emit better error message when assignment precedes T.let declaration

Partially addresses an issue reported at #3482. If a let-less assignment to a variable came before it was "declared" with let, info on the declared type would be missing, so we would default to treating the declared type as NilClass even if that was not the eventually declared type, which resulted in confusing error messages. With this change we check if the type info was missing rather than accepting the default, and produce a more reasonable error message.
It would be nice if we could add some signposts to the error message pointing out the let that conflicts with the assignment, but I think that's probably out of reach without some major work.
Motivation

See #3482
Test plan

See included automated tests.

Example:
→ View on sorbet.run
# typed: true
x = "foo"
x = T.let("bar", String)

Behavior before:
editor.rb:2: Incompatible assignment to variable declared via let: String("foo") is not a subtype of NilClass https://srb.help/7001
     2 |x = "foo"
            ^^^^^
Errors: 1

Behavior after:
editor.rb:3: Assignment to a variable declared via let occurs before the let declaration has been reached https://srb.help/7033
     3 |x = "foo"
            ^^^^^
  Autocorrect: Use `-a` to autocorrect
    editor.rb:3: Replace with T.let("foo", String)
     3 |x = "foo"
            ^^^^^
Errors: 1

@jez Copy suggestions always warmly welcomed :)

For paper trail purposes: we reverted this (#3595) because I forgot to run our standard tests against Stripe's codebase. :flushed: Will likely re-PR this soon.


No warning/error on Array of choices empty. Also gives "Object reference not set to an instance of an object."

Describe the bug

When starting up a new multi-choice prompt, I tried to add my choices to "Bot Asks"'s Response Variations. My bad. The Composer handles Array of choices with simple choices empty list with runtime "Object reference not set to an instance of an object." exception. Meaning the bot's response in a text box is that. I would expect a warning/error for this kind of mistake. If I remove all the items from an already working Array of choices, the bot doesn't throw the exception, and asking the prompt with 0 options.
Version

1.4.0-nightly.231404.da58480
Browser

[ ] Electron distribution
[x] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Add -> Ask a question -> Multi-choice
Fill a prompt Response variation
Start the bot
Trigger the Multi-choice

Expected behavior

A multi-choice prompt with no options and/or edit time warning/error for the multi-choice element.
Screenshots

Additional context

@garypretty  would we look to spot the issue at Design time?

@mrtank we will take an action item and work on this. Most likely lean on Option 2 as Option 1 involves changes downstream in our runtime.

@mewa1024

Yes, I think Option 2, warning message, is a good approach.

@garypretty, @mewa1024, should this be assigned a milestone for tracking?

Good point. Assigning this to R14.


Add Education Section to README

Add a link for Data Science for All
Add a link for ICS 80

Please address the comments.

Addressed.


Exposing RawMutex and RawRwLock in the public API?

I'm writing some unsafe code that manages raw pointers directly for perf and I really need to store a RW lock inside a larger data-structure. The parking_lot_core crate has useful primitives to build such a lock, but all what I really need is the RawRwLock type that's already in parking_lot. There's little point in me implementing it from scratch.
Could RawRwLock (and thus likely RawMutex) be exposed in the public API? This would make writing low-level data-structures with embedded locks much easier.

I personally just use Mutex<()> when I need a raw mutex for low-level code. Is this insufficient in your case?

I did think of that, but was unsure what the following code would then compile down to:
pub struct RwLock<T: ?Sized> {
    raw: RawRwLock,
    data: UnsafeCell<T>,
}

What does rustc do with data? My guess is that it stops existing entirely, but I don't really want to rely on that for raw memory I'll be mmap()ing to disk.
Which leads me to another question: RawRwLock uses an AtomicUsize internally, which means that the size of the struct varies on 32 vs 64 bit OSs. Since I want to use RawRwLock in a B-tree page that is going to be written to disk through mmap(), I need a stable size. I could write some metadata to the file to detect if someone tries to read on a 32 bit OS a file written with on 64 bit OS, but that's a hassle I'd rather avoid.
If this were all in-memory, then Mutex<()> would suffice, but since I'm writing to disk, having a known size makes life much easier.

data does get completely optimized anyways, however RawRwLock uses AtomicUsize internally, so this won't help in your case.
Another issue is that parking_lot mutexes and rwlocks are not consistent across processes, such as if two processes mmap the same file and both try to grab the same lock, you will have issues.
For your use case I would strongly recommend using Linux futexes directly, which are always based on an AtomicU32. If you do not use FUTEX_PRIVATE_FLAG then futexes will work correctly across multiple processes.

Another issue is that parking_lot mutexes and rwlocks are not consistent across processes, such as if two processes mmap the same file and both try to grab the same lock, you will have issues.
For your use case I would strongly recommend using Linux futexes directly, which are always based on an AtomicU32. If you do not use FUTEX_PRIVATE_FLAG then futexes will work correctly across multiple processes.

This I agree is excellent advice, but I don't need the ability to read this on-disk data from multiple processes at the same time (and won't be able to for other architectural reasons). I'm using mmap() for fast access to a multi-GB file that's possibly beyond the size of RAM.
I don't actually need the RawRwLock to be persisted to disk at all. I'm putting it in the B-tree page for the sake of performance (data locality).

In that case you could just reserve space for a 64-bit value in your file format, but only use the low 32 bits if you are running on a 32-bit OS.

And add assert!(mem::size_of::<RwLock<()>>() <= 8) to catch any (unlikely) changes to the size of RwLock.

That's actually a pretty good idea! Didn't think of that at all; seems obvious in retrospect.
8 bytes is can be quite a lot for low-level code; what benefits do you see in keeping it usize vs fixing it to u32?

Mainly because AtomicU32 isn't stable yet, while AtomicUsize is. Also because in theory, the maximum number of concurrent read locks is proportional to the address space size (hence usize).

@Amanieu You say that parking_lot mutexes and rwlocks cannot be used across process boundaries, is this still true?
( I am not sure why it would have changed, but you did mention using mmap in #305, so it seems, that it could be possible)

No, this is still not possible and will never be possible. You should use an OS-specific mutex that supports this.


bpf,make: add ability to run bpf unit tests in the Cilium builder

This commit adds a top level Makefile target for running eBPF unit tests within the Cilium builder container.
The Cilium builder container ensures all the dependencies necessary to build and run the tests are present.
After this commit, running make run_bpf_tests from the root of the repo will run the tests in a container, while running the same target in the ./test directory will run the tests locally, outside of the container
Add top level `make run_bpf_tests` target to run eBPF unit tests in the Cilium builder container

Don't think we need to run full suite for this, smoke tests cover running the effected eBPF unit tests, and this passes.
Just need reviews.

Can we do something about the Travis issue? Ignoring it won't make it go away.


Glitch art

Added the file for issue #19
Added the file for Glitch Art on the image as well as updated the ReadMe.

Added the file for issue #19
Added the file for Glitch Art on the image as well as updated the ReadMe.

Nice work. I believe we can improve the degree to which it is converted into glitch effect. The current one does not apply much visual changes to large images as the one we have in the repo


(PDB-2632) add inventory endpoint and support for dotted fields

Add an inventory endpoint of fact name=>value pairs, along with capability to
query it and the resources parameters column with "dotted syntax" like
[= parameters.foo bar]. Also add support in PQL. This does not include
projection, regex matching of path elements, or array indexing of path
elements.

I need to give this another once over in the morning for code cleanup, more docs, and better testing of the PQL regex.

@puppetlabs-jenkins test this please

@puppetlabs-jenkins test this please

@puppetlabs-jenkins test this please

@puppetlabs-jenkins test this please

@puppetlabs-jenkins test this please

@puppetlabs-jenkins test this please

@puppetlabs-jenkins test this please

@puppetlabs-jenkins test this please

@ajroetker addressed your comments. Good call on reflection but it looks fine.

Test PASSed

@senior addressed your comments

Test PASSed

Test FAILed

@puppetlabs-jenkins test this please

Test PASSed

Test FAILed

Test PASSed

@wkalt you're my hero :-).


Update and moving Restore.md

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/Quickstart/Restore.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/TOC.md
:warning:Warning

Details

docs/Quickstart/Troubleshooting-Restore.md
:white_check_mark:Succeeded

docs/TOC.md

[Warning] Invalid file link:(~/Quickstart/Restore.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/TOC.md
:warning:Warning

Details

docs/Consume-Packages/Package-restore-troubleshooting.md
:white_check_mark:Succeeded

docs/Quickstart/Troubleshooting-Restore.md
:white_check_mark:Succeeded

docs/TOC.md

[Warning] Invalid file link:(~/Quickstart/Restore.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/TOC.md
:warning:Warning

Details

docs/Consume-Packages/Package-restore-troubleshooting.md
:white_check_mark:Succeeded

docs/Quickstart/Troubleshooting-Restore.md
:white_check_mark:Succeeded

docs/TOC.md

[Warning] Invalid file link:(~/Quickstart/Restore.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/Consume-Packages/Package-restore-troubleshooting.md
:white_check_mark:Succeeded

docs/Quickstart/restore.md
:white_check_mark:Succeeded

docs/TOC.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/Consume-Packages/Package-restore-troubleshooting.md
:white_check_mark:Succeeded

docs/Quickstart/restore.md
:white_check_mark:Succeeded

docs/TOC.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


When selecting a theme during onboarding the header image does not display correctly on your page

We discussed this before.

Currently onboarding only sets the colors of the theme. To set the images, they would need to either be loaded, encoded as base64 and saved, or pre-encoded as base64 and the pre-encoded versions saved.
I think pre-encoding them would be the better solution.


[WIP] Win32 Handle extension

Question: Do I add vkGetMemoryWin32HandleKHR into allocator functions, or do I add it as a function argument?
First approach is clean wrt argument count, but if I don't have Win32 platform enabled this is kinda pointless. Second is less appealing, but safe, because it relies solely on user providing function, being static or dynamic

I would add it to the struct with functions, just like all other Vulkan functions.

Is it safe to use an atomic? What if 2 threads want to fetch the handle at the same time? Shouldn't you use a mutex around the whole fetching logic?

Is it safe to use an atomic? What if 2 threads want to fetch the handle at the same time? Shouldn't you use a mutex around the whole fetching logic?

No, atomics are safe. I also did the relax ordering, because we have no memory to make barrier onto.
If 2 threads are accessing there are 2 situations that may happen:

already assigned -> easy duplicate

not assigned: -> creates handle (maybe twice, but who cares)
then there is compare_exchange, which is atomic, and if the underlying vlue changes it returns false.
One compare_exchange always succeeds, others will return false, and handle will not be reassigned. If there is a driver that shouts error (didn't find any major one) this is also OK, since that means there is handle somewhere already. Any excessive HANDLES are removed. We also provide handles for other processes, that's why there is that AMD fix there.

Actually I could tinker with a lot of atomics in this lib and make it lock free. There are lock free queues there, so maybe one of the next steps :)

Isn't this whole effort about making sure we don't call vkGetMemoryWin32HandleKHR twice for a device memory object? As you posted the validation message in the other ticket:

VUID-VkMemoryGetWin32HandleInfoKHR-handleType-00663
If handleType is defined as an NT handle, vkGetMemoryWin32HandleKHR must be called no more than once for each valid unique combination of memory and handleType

If so, isn't your current code unsafe in this regard when called from multiple threads?
When talking about an UB, we typically understand that anything can happen, including entire app or system crash, so is there really "no harm" when we trigger an an UB?

Hmm, you're right, though there was a limitation for me, hence why I chose to use atomics, and that being absence of mutex on dedicated memory. Should I add one?

Yes please, I think you need a mutex for this to be safe.
Another topic to think about: What if an allocation ends up as a dedicated allocation?

If it has VkDeviceMemory it is not a problem. There is no mutex in this structure, so I think I will add one

Added a mutex. There is a really low chance in current setup that it will be used, so it imposes little overhead

It seems you reformatted the entire file with some automatic formatting tool! Please don't do this. It makes it difficult to see what are the real differences you introduce to the code.
Can you please surround all your code with an #ifdef telling about the availability of the VK_KHR_external_memory_win32 extension, similar to e.g. VMA_EXTERNAL_MEMORY?

crap, reflexes

Done

Hopefully I defined correct one. Also, I have a linker error with static function, seems like it is only possible to get the function dynamically

Yes, functions from extensions can only be fetched dynamically.

do I set it to null, or try to fetch with dynamic code?

Please do like the other extensions do - for Static don't do anything, for Dynamic try to fetch the function pointer, for ValidateVulkanFunctions - assert if not null if the extension is enabled.

Are you willing to develop some simple test for this new functionality?

Sure

ok, then I'll just remove the call

I wonder, why don't we allow for dedicated allocations to have custom pNext?

What do I specify at memory type bits on pool?

To create a custom pool you need to find the right memory type and choose it explicitly:
https://gpuopen-librariesandsdks.github.io/VulkanMemoryAllocator/html/custom_memory_pools.html#custom_memory_pools_MemTypeIndex

I added code for fetching the extension and creating a pool. Can't launch test, however, it throws at some random point

Fixed, working like a charm

Well, I'd take it to the new PR.
I'd also like to do several things:

Slice the library into different files for easier development, and creating a generator to concatenate them to a single file.
Make use of inline and make lib pure header only
Make 2 targets, 1 is static and 2 is header only cmake target
Formatting bot + .clang-format

Tell me if you are interested in some of them :)

Also VmaDeviceMemoryBlock has the same m_pMappedData and handle, should that also be replaced?

OK, I'll analyze your code soon and hopefully I merge it. I agree the optimization to extract new structure VmaAllocationExtraData is better done as a separate task.
I am sorry but I am not interested in any of the big changes you proposed. The library is now mature and used by many developers. I don't want to do a revolution that could break anyone's workflow. If you want to introduce such big changes to the code, please feel free to maintain your own fork. I am happy to link to it from README.
About the Cmake script especially, I don't want to make more major changes, because this is a never-ending story of pull requests from users with various visions about how it should look like. The more such requests I get, the closer I am to removing the Cmake script completely. I regret adding it in the first place. I agree with the arguments of @ocornut about the value of not having it, as mentioned in https://github.com/ocornut/imgui/issues/7892, section "3.6. Lack Of A Build System Is A Feature". Please also note that VMA ships with the Vulkan SDK as "vk_mem_alloc.h" file only, not the entire repository and no Cmake script.

VmaDeviceMemoryBlock doesn't need to have the mapped pointer extracted to a separate structure because I am not concerned about adding a new member to this class. Only the allocation object should stay lightweight.

Ah, ok. I thought about making Wisdom Memory allocator, but that would require me to also get D3D12 MA shredded, which is a ton of work.

I have Wisdom library, which is super thin compared to others, but super demanding in terms of Vulkan (works no less than 1.3 and requires heavy extensions). I stumbled upon both allocators, and now they are part of its standard. If there is a wish, I could do the Wisdom memory allocator and do the extraction for both.

It is up to you. From my perspective, it is good if we just have the change to VMA discussed in this PR.

Thank you very much for this extensive and high quality contribution.
You didn't test dedicated allocations. Because of that, you didn't meet the bug that happens because VmaAllocation objects are created using a custom CPU memory pool and don't get their constructors and destructors called, so the handle of a dedicated allocation could have garbage data. This is another argument to keep the handle, together with the mapped pointer, in a new dynamically allocated structure VmaAllocationExtraData. I added it.
I also made some other fixes and written documentation for the new feature. Please let me know if it looks good to you.

I need to finish one project, then I will continue. Where can I find new documentation?

It is already part of the documentation available online, most notably:
https://gpuopen-librariesandsdks.github.io/VulkanMemoryAllocator/html/vk_khr_external_memory_win32.html
https://gpuopen-librariesandsdks.github.io/VulkanMemoryAllocator/html/group__group__alloc.html#ga8d327b7458d8cf426b84b5efba9bb9bf

Wow, documentation looks awesome!


"onTouchStart" preventDefault is ignored, causes double step to happen

This was tested on Chrome 58 with responsive mode switched on which means it treats clicks as touch events. This results in step being done twice on a single click.

The following message gets logged in the console.
[Intervention] Unable to preventDefault inside passive event listener due to target being treated as passive. See https://www.chromestatus.com/features/5093566007214080

Same issue here

Reproduced in react-numeric-input v2.0.9 and 2.1.0 in desktop Chrome in device mode and latest Chrome browser on Android.

Hi @vlad-ignatov,
Made a quick fix on this issue according to recommendation on developers.google.com:
https://developers.google.com/web/updates/2017/01/scrolling-intervention
If your touchstart listener calls preventDefault(), ensure preventDefault() is also called from associated touchend listeners to continue suppressing the generation of click events and other default tap behavior.

Please review and merge if appropriate:
https://github.com/vlad-ignatov/react-numeric-input/pull/56
That fixes the double click issue in devtools' device mode so should work on mobiles with touch events.

This is now fixed thanks to @lanceon.


When calling a shuttle train any "Interups" added to the train previously are wiped out

In Factorio 2.0 you can add "Interrupts" to the trains, I use one in my trains to trigger refuelling and was trying to use it with the shuttle trains. However when you call a train, that train has it's interrupts wiped out.
It would be awesome if we could use "Interrupts" with the shuttle trains so they can still go off an refuel and various other things. As we can just add refuelling at the shuttle train depots, it's not a desperate thing, but would be a nice to have.

It's worth noting too that shuttle trains that go to stations with circuit networks often end up with a schedule being pushed onto it.
Wonder if we are able to do one of the following to fix it too:

Prevent schedules being added to the trains when called to a nearby station
Add an option to that allows preventing trains being called to the nearest station and instead always uses the player position along the track; so we can disable it when we have smart train stations setup.

When I get chance I'll raise a PR to add in a new option to allow preventing of using nearby stations and instead use nearest track entity instead.
As for the original issue regarding the interrupts, I don't know enough about the modding etc to be able to fix that at this time.

Set the station filter correctly and shuttle trains will refuse to use these stations. Also for pickup.
E.g. in my current game I have set the "Station names to filter" to: Refuel,↦,↤

Regarding interrupts: I am not completely convinced if interrupts are a thing to be implemented. When I call a shuttle train I want it to come now and not to do some sort of interrupt first while I wait. Same when I ride it.

Ah thanks @StefanTT, I was about to code up the extra option as above but there is no point if I can use filters to achieve this, I can see there is a toggle to invert the filter too, which is perfect.
All of my stations have "dynamic" names that use icons, so they weren't exactly useful for choosing a destination. Using the filtering I can limit it to only go to stations starting "Shuttle - "for argument sake and also using such a filter means calling a train wont use stations; based on what you've said.
I will give it a go this evening. Thank you 👍
--
FYI the only interrupt I was going to add was one to go refuel when the train is low, all of my trains have this; I don't see any other use for interrupts at this time; therefore just re-fuelling them at their depo(s) will be fine.

I sometimes filter for "/" when I end the production station names with "/i" for input stations and "/o" for output stations, or even some "/io". Nowadays often the arrows I mentioned before.


Function as listener argument is not suggested by code completion

It seems to depend on function return type: void vs dynamic. In this sample both clickHandler and otherClickHandler work, but completion suggests only otherClickHandler.
Code to copy/paste:
import 'dart:html';

main() {
  var element = querySelector("foo");
  element.onClick.listen(cli);
}

void clickHandler() {}

otherClickHandler() {}

Stream.listen's first parameter is a void Function(T), but your clickHandler is a void Function(). It should not suggest clickHandler.

@srawlins I guess either both clickHandler() and otherClickHandler() should be in the completion list or none.
BTW nothing changes if I add a parameter to both functions (and in this case, both variants should definitely be suggested):
import 'dart:html';

main() {
  var element = querySelector("foo");
  element.onClick.listen(cli<caret>);
}

void clickHandler(MouseEvent a) {}

otherClickHandler(MouseEvent a) {}

I do not have permissions to reopen the issue.

Sorry about that. You're right. void clickHandler(MouseEvent a) {} and otherClickHandler(MouseEvent a) {} should be suggested.

The situation is almost worse at this point. In the following code, placing the cursor after (cli and requesting completions the returned list includes otherClickHandler but not clickHandler.
class C<T> {
  void m(void Function(T) f) {}
}

void f(C<String> c) {
  c.m(cli);
}

void clickHandler(String s) {}
otherClickHandler(String s) {}

The situation is almost worse at this point. In the following code, placing the cursor after (cli and requesting completions the returned list includes otherClickHandler but not clickHandler.
class C<T> {
  void m(void Function(T) f) {}
}

void f(C<String> c) {
  c.m(cli);
}

void clickHandler(String s) {}
otherClickHandler(String s) {}


Search feature does not search on attachments names

Hi,
When we add some attachments, will could not apply a search (CTRL+F) to find them by it attachment name (I'm not asking to find them by filename but attachment name),.
Server Running on Ubuntu 20.04, Client app on Android and Windows 10.
Rocket.Chat (Server): 3.17.1
Rocket.Chat.Electron (Electron/Desktop) 3.2.4
Is there any setting relevant changed. No
To Reproduce
Steps to reproduce the behavior:

Go to any private discussion or on any channel.

Add and Send an attachment like an image as flow:

Add some more attachment with names.

using search button or CTRL+F: Search for an name of attachment previously added.

You can see that the search feature does not look at attachments name.

Expected behavior
Rocket chat search on attachments names

This is a duplicate of https://github.com/RocketChat/Rocket.Chat/issues/23114
And I think, that issue itself not with Electron app, but with RocketChat in general.

Please test if the problem persists with our latest release. I will be closing the issue but if there is any problem yet, we will open it again.


Chrome becomes unresponsive when headless: false

Steps to reproduce
Tell us about your environment:

Puppeteer version: 1.3.0
Platform / OS version: macOS 10.12.6
Node.js version: 8.11.0

What steps will reproduce the problem?
const Puppeteer = require('puppeteer');

async function run() {
  const url = 'https://www.nike.com/launch';

  const browser = await Puppeteer.launch({ headless: false });

  const page = await browser.newPage();

  await page.goto(url);
  await page.screenshot({fullPage: true, path: 'example.png'});

  return browser.close();
}

(async () => {
  await run();
})();

What is the expected result?
In the script above, puppeteer launch in non-headless mode, navigate to a URL, load the page and take a screenshot.
What happens instead?
When puppeteer navigates to a particular URL (https://www.nike.com/launch), it attempts to load the page but Chromium quickly becomes unresponsive. The method will time out and Chromium will exit as well.
See verbose logs.
The strange thing is that the script completes successfully if headless: true is configured instead. I have also tested the script using my local Chrome (66.0.3359.139) and Chrome Canary (68.0.3417.0) and the issue still persists. I have also been able to run the bundled version of Chromium separately and navigate to the page successfully manually.
I have also tried setting the viewport and user agent of the request to mimic manual interact with the page, but this did not help:
await page.setViewport({
  width: 1920,
  height: 1080
})

await page.setUserAgent('Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/66.0.3359.139 Safari/537.36');

I'm at a loss as to what could be happening. The only thing I have considered is that the webpage is loading a large number of assets (images), but the dev tools become unresponsive when this occurs so its difficult to debug.

Really strange, must be something on my end. I'll keep digging, thanks for the sanity check!

This doesn't repro for me as well; speculatively closing.


"Couldn't load Object" when using a src Firebase URL

Description
I'm using Firebase for my app, but setting up model-viewer to reference the storage is proving problematic.
When attempting to launch Scene Viewer on Android I get "Couldn't load Object - Looks like there's something wrong with this object."

The model loads fine when the URL is relative or points to some other external source. When I include an ios-src, iOS correctly handles the Firebase urls and displays the model.
To simplify and to make sure it wasn't my model, I uploaded the Astronaut and replicated a basic setup on glitch. A snippet like this should replicate the issue (spaces added to the URL so you don't have to scroll horizontally):
<model-viewer src="https://firebasestorage.googleapis.com/v0/b
                   /<proj>.appspot.com/o/model%2FAstronaut.glb
                   ?alt=media&token=<token>"
              camera-controls ar
              interaction-prompt="auto">
</model-viewer>

Live Demo
Demo on Glitch
Browser Affected

[x] Chrome (tested on version 76.0.3809.132)

OS

[x] Android (tested on version 9)

Versions

model-viewer: v0.6.0

Hey, thanks for thorough description of the problem @ebrayton . This is a problem with Scene Viewer (the native app that <model-viewer> launches on Android). I'll forward the report to that team and let you know if they think it's a problem on their side.

Thank you. Let me know, too, if there's any other information they need.

Hi everybody:
Same behavior here, when trying to test the web component from my own (secure) server:

Debugging the web page by means of "Remote devices" tool on desktop Chrome I get:
model-viewer.js:57191 Uncaught (in promise) TypeError: fullscreen error
    at HTMLElement.[onARButtonContainerFallbackClick] (model-viewer.js:57191)
    at HTMLDivElement.ARModelViewerElement.<computed> (model-viewer.js:57058)

Any help on this will be welcome!

Same here, I have that problem either with glb/gltf models hosted on external source (github.io) and local

I'm having the same problem when using my Firebase Storage repo. The only way I managed to load the GLB model from there was changing the rules so everyone could access the file; obviously I cannot leave it like that. Any progress on this?

Hey folks, sorry for letting this fall off the radar. We determined that this is actually a <model-viewer> bug. We are accidentally breaking URLs that have sensitive query strings when we pass them into Scene Viewer.

In that case, is there any way to fix this?
Enviado desde mi iPhone

El 24 oct. 2019, a la(s) 11:53, Christopher Joel<EMAIL_ADDRESS>escribió:

Hey folks, sorry for letting this fall off the radar. We determined that this is actually a  bug. We are accidentally breaking URLs that have sensitive query strings when we pass them into Scene Viewer.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

FYI we have opened https://github.com/GoogleWebComponents/model-viewer/pull/855 to resolve this issue. If anyone has time, it would be really helpful if you could test out this PR to make sure it resolves the issue for you.
Try it out by using this build:
<script type="module" src="https://github.pika.dev/googlewebcomponents/model-viewer/pr/855"></script>

BTW once we have sorted this out, we plan to release v0.7.2 with this fix.

A fix for this has been released as v0.7.2. Please ping this bug if you are still having problems and we will re-open to investigate!

Confirmed v0.7.2 working for me. Thanks!

Hi there:
Problems was solved for me days ago but now are here again (tested on Samsung Galaxy Tab S4 and Motorola Moto g6).
FYI I'm loading the lib files from CDN, as suggested:
<!-- Loads <model-viewer> for modern browsers: -->
<script type="module"
<EMAIL_ADDRESS></script>

<!-- Loads <model-viewer> for old browsers like IE11: -->
<script nomodule
<EMAIL_ADDRESS></script>

May be a new release of libs (I can´t fin the revision number in the script) have broke again the previously fixed?

I ran into this issue today.  When using a Firebase  stored GLTF file it would not load.  However, in the corresponding USDZ file worked in Quick Look.  This is only with Firebase.  When using files hosted in S3, both are working.

@theranbrig we fixed this one a while back. If you are experiencing it, you may be using an old version or <model-viewer>, or else there is a new bug. Can you share an example URL that I can try to confirm that your file is failing to load?

@cdata I am on version 0.8.1.  Here is a file I recently uploaded:
https://firebasestorage.googleapis.com/v0/b/yzed-88819.appspot.com/o/images%2FGLTF_DOWNSIZED_JUMPERJACKET_V5.gltf?alt=media&token=a5c77518-0b0d-4980-839c-c951c270f153

Same happens on my website (version 0.8.1):
https://www.vr-innovations.nl/diensten-visualisatie/
File:
https://www.vr-innovations.nl/files/Totempaal.glb

@theranbrig I tried throwing together a simple test case with your model so that I can try this out for myself. Unfortunately, the CORS heads are too restrictive and I cannot load the model (see http://lopsided-motion.glitch.me/fb-eg.html).
Would it be possible for you to put together a test case like that w/ sufficiently permissive CORS so that I can diagnose the issue you are seeing?

We are not using Firebase for hosting and have moved onto S3.  However, the strange thing was with that was that the CORS did not affect the USDZ or the images that I had uploaded.  It was only on the GLB/GLTF file.

For .usdz, the file is ultimately loaded by a native app and may not be subject to CORS restructures. For images, the browser is able to load images (via <img> or CSS styles) without CORS permissions.
One of the cases where CORS permissions apply is when you load a file with fetch or XMLHTTPRequest. This is the case when we load glTF and glB files. We do this because we want to render these 3D models inline in the browser, but the browser doesn't have a "built-in" way to render them, so we need to read the contents of the file in order to render it ourselves. In most cases where JavaScript can read the response to a network request, CORS becomes an issue.
So, sufficiently permissive CORS headers need to be set in the response for <model-viewer> to work as intended.

Hi there,
I'm getting the same issue.
I'm trying to use the Google Scene Viewer as described here:

and using a firebase storage download url as the file.
I know this strictly falls outside the scope of Model Viewer, but I'm hoping someone will be willing to help me out!

@tpsiaki Do you have any guidance you could provide to @ataylor09?

@elalish Out of interest, do you know how Model Viewer handles this? Isn't moddel viewer using Scene Viewer behind the scenes to start AR on an Android device?

@ataylor09 our default is now webXR instead of SceneViewer, so that we don't have to leave the browser or redownload the model. When you specify ar-mode="scene-viewer", we just pass the src URL to their intent, but we get no feedback from then on.

We switched to hosting our GLTF and USDZ files on AWS S3.  No problems at all.  Not sure if that is a viable option for everyone, but we haven't had any issues since switching.

Thanks but in my case I would not like to use S3 since all models and other logic is built on top of Firebase. One would expect Google to support integration between their products but I guess not in this case...

@ataylor09 @jpeltone Do you have examples of the urls to the gltf or glb files you're trying to load from firebase or elsewhere that are failing?

@jpeltone - The link you provided seems to maybe not be including the url get params to the file= parameter sent to scene viewer.  This link is telling Scene Viewer that the gltf can be downloaded at https://firebasestorage.googleapis.com/v0/b/ which seems to be incomplete.

Thanks for replying, I did not want to paste the full link as the access is behind authentication but I created temporary file here: https://firebasestorage.googleapis.com/v0/b/ardom-ee02f.appspot.com/o/models%2Fscene.gltf?alt=media&token=899e893b-8d92-42d1-bfb5-2e2485104784
However, this file is only the gltf file, is it so that also the bin file and textures are also needed? I could not get the model working in the preview tool either even with all the files included so it's probably something with the model itself? I have created sfb file from the model successfully with Sceneform plug-in though so it should be working.

update: I was able to see this model successfully in the preview-tool: https://firebasestorage.googleapis.com/v0/b/ardom-ee02f.appspot.com/o/models%2Fwall-lamp.zip?alt=media&token=14865e06-29e4-4397-93c4-5e4b085b7263 when uploading the zip file but if I upload the files in the zip to the preview tool separately it does not work and it does not work from Android app code either but gives the same error as when accessing the gltf file.

update 2: I packed the file to glb and uploading it to the preview tool works and shows fine but still does not work from Android app.
File: https://arvr.google.com/scene-viewer/1.0?file=https://firebasestorage.googleapis.com/v0/b/ardom-ee02f.appspot.com/o/models%2Fout.glb?alt=media&token=a4839672-3d8b-436b-9c37-ad088d1fe3e5
related code:
Intent sceneViewerIntent = new Intent(Intent.ACTION_VIEW);
sceneViewerIntent.setData(Uri.parse("https://arvr.google.com/scene-viewer/1.0?file=https://firebasestorage.googleapis.com/v0/b/ardom-ee02f.appspot.com/o/models%2Fout.glb?alt=media&token=a4839672-3d8b-436b-9c37-ad088d1fe3e5"));
sceneViewerIntent.setPackage("com.google.android.googlequicksearchbox");
startActivity(sceneViewerIntent);
Another finding: the Help link in the online preview tool does not work at least for me but gives: developers.google.com refused to connect.

This is because the & in your intent is being treated as part of the intent url rather than part of the file url, and the %2F is url decoded when the file parameter is read out the intent.  You need to url escape the & with %26 and the % with %25.
This works for me:
https://arvr.google.com/scene-viewer/1.0?file=https://firebasestorage.googleapis.com/v0/b/ardom-ee02f.appspot.com/o/models%252Fout.glb?alt=media%26token=a4839672-3d8b-436b-9c37-ad088d1fe3e5

Great, thanks, I missed that when just copying the link. I still need to check how getDownloadUrl() in Firebase StorageReference manages url escaping and try it out.

Hi, are we sure this issue hasn't resurfaced? I am getting the same issue, urls with query params (for example: https://firebasestorage.googleapis.com/v0/b/sharpar-io.appspot.com/o/DELETEME%2Fgithubexample.glb?alt=media&token=61e28706-0488-4a24-82d7-9bde2d66f611) are throwing the 'couldn't load object' error in scene viewer.
I understand that it is an issue with the formatting of the url... however I've played around with url-encoding various parts of the 'src' attribute directly and all I get are 403 errors (because I am url-encoding parst of firebase's authentication token query).
This leads me to believe that the problem is in the model-viewer code itself, in how it is formatting the intent sent to scene-viewer.

@Sheldonfrith Can you link to your page where this is not working? That would allow me to debug. Also, if you use webXR mode instead of scene-viewer does it work?

Any updates here? Facing the same issue, I hit CORS when I try to access a .glb/.gltf file on firebase storage.

Any updates here? Facing the same issue, I hit CORS when I try to access a .glb/.gltf file on firebase storage.

@naiveHobo I'm not sure you're seeing the same problem, and this is a pretty long, closed thread. Maybe time to start a new issue with your own link and repro steps. You will need proper CORS headers from wherever you're serving from, so if Firebase doesn't do that or allow it to be configured, you may need to use a different server.

@naiveHobo I'm not sure you're seeing the same problem, and this is a pretty long, closed thread. Maybe time to start a new issue with your own link and repro steps. You will need proper CORS headers from wherever you're serving from, so if Firebase doesn't do that or allow it to be configured, you may need to use a different server.

Hi, I don't have much time left to make major change in DB (I am using Firebase), do you have any recommendations for workaround this issue ?

@holaworlds You can view it by setting the firebase storage bucket for cors. See this post for details:
https://stackoverflow.com/questions/37760695/firebase-storage-and-access-control-allow-origin

THANK YOU @ulbqb for that link. I don't know why it's buried in a closed issue. This solution should be much easier to find. Adding here to draw more attention to it.
If your models are hosted on Firebase Storage
SOLUTION ⤵
cloud.google.com/storage/docs/configuring-cors


Undefined symbols for architecture arm64

Hello, I have the following issue when running after fresh install (already linked and installed cocoapods):
ld: warning: directory not found for option '-F/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS15.4.sdk/System/iOSSupport/System/Library/Frameworks'
Undefined symbols for architecture arm64:
  "_OBJC_METACLASS_$_RCTEventEmitter", referenced from:
      _OBJC_METACLASS_$_RNLocation in RNLocation.o
  "_OBJC_CLASS_$_RCTEventEmitter", referenced from:
      _OBJC_CLASS_$_RNLocation in RNLocation.o
  "_OBJC_CLASS_$_RCTConvert", referenced from:
      objc-class-ref in RNLocation.o
  "_RCTRegisterModule", referenced from:
      +[RNLocation load] in RNLocation.o
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Does somebody have seen anything similar before?

pre_install do |installer|
installer.pod_targets.each do |pod|
if pod.name.eql?('react-native-location')
def pod.build_type
Pod::BuildType.static_library
end
end
end
end


Removed all thresholds except time

Summary

Address Issue #1118. Removes all thresholds except time, still determined by API call.
Changelog

[CATEGORY] [TYPE] - Message
[General][Change] - Removed device type & distance threshold for logging BT devices.
Test Plan

Frequency of BT logs should increase and align with time specification in S3.

@kevdesilva01  How's the device battery life for this PR?

@a6wu I'm not sure at the moment but I can test that out on several devices and find out.

@a6wu no significant effect on battery life.
iPhone 11 - one year old
With app open:
98% at 9:45am
94% at 10:45am
4% in an hour
With app minimized but phone screen on:
91% at 11:00am
90% at 12:00pm
1% in an hour
Motorola g7 Power - 2 months old
With app open:
100% at 1:45pm
99:% at 2:45pm
1% in an hour
With app minimized but phone screen on:
99% at 2:45pm
99% at  3:35pm
0% in an hour


[BUG] DiskService.SetLength causes (random) exception

We're currently using LiteDb to store about 350k small objects in a Collection. This is inside a .NET Core 3.1 ASP.NET Application.
At random times, but pretty consistently in every run, the following Exception is thrown. Even when the dataset is as small as 5000, to stay under the BulkInsert default Bulk-size.
IOException: The requested operation cannot be performed on a file with a user-mapped section open. : 'Database-log.db'
System.IO.FileStream.SetLengthCore(long value)
System.IO.FileStream.SetLengthInternal(long value)
System.IO.FileStream.SetLength(long value)
LiteDB.Engine.DiskService.SetLength(long length, FileOrigin origin)
LiteDB.Engine.WalIndexService.Clear()

Whenever we switch to Shared, the issue obviously goes away because a log-file is no longer used, but the issues reported by others in the last few hours start occurring.
Upon investigating, I tried a bunch of different things, but eventually decided to just "timeout" SetLength and try again using the following code:
            int attempt = 1;

            tryAgain:

            try
            {
                stream.SetLength(length);
            }
            catch
            {
                Debug.WriteLine($"SetLength #{attempt} failed, retrying in 100ms...");

                Task.Delay(100).Wait();

                attempt++;

                goto tryAgain;
            }

I've run the 350k BulkImport with this piece of code in the DiskService.SetLength multiple times, and never seen it reach a second attempt. This leads me to conclude two things:

A race condition is happening inside the code of LiteDb, a Reader stream is being used at a time where the DiskService wants to write.
An external factor like a virusscanner is at play here.

For the first, I cannot say for sure, but I'm not seeing weird things in the code, it all seems properly thread-safe.
For the second, in an attempt to figure out the problem, I made a test-project which does exactly the same things in the same way. However, even after multiple runs, I cannot reproduce the issue at all, which is frustrating. If a virusscanner was too noisy, my test-bed would break. The test-bed I attached is an exact replica of the current code-base where we have this problem, so it's weird it happens in the "real deal", but not the test-version...
I hope you find this useful and can figure out a potential reason why this happens. Attached is my test-bed for this particular issue containing a Console-app and a Web-app mirroring the exact situation where have this issue on. I obviously cannot supply the realData.json, but I can confirm the data itself does not matter for the actual Exception to happen.
LiteDBTester.zip
PS. It looks like that using Insert instead of BulkInsert can trigger the Exception faster.

@LennardF1989 I ran your test project and I couldn't reproduce the issue either. There's probably something different in how the real code is using LiteDB - maybe its datafile is in a place that "activates" your anti-virus, while the test project is not.
If there was any way I could run (or at least take a look at) the real code, that could be helpful.

@lbnascimento I figured it out. Knowing what I know now, it is so obvious...
I noticed that from one commit to another things kept working as it should. So I started going back in time until it broke again. Turns out... sigh... it was my TortoiseGit trying to generate a status icon in Windows Explorer for the file because it got add to the repository.
As soon as I told git to delete the file and stop tracking it, LiteDB worked perfectly, even with 350k Insert-calls in a row. As soon as I added the file back to enable tracking, the issue returned.
In the end, it was not a virusscanner, but my trusty 'ol TortoiseGit, whom I never suspected at all.
I'll close this issue, but let it be a warning for all: Something else can be affecting your files as well! In my case, it was my least suspected and useful tool that stabbed me in the back :(


Update sql-database-managed-instance-configure-p2s.md

the file attachVPNGatewayAz.ps1   was removed from the repo as far as i can see

@t-yoziv : Thanks for your contribution! The author, @srdan-bozovic-msft, has been notified to review your proposed change.

#sign-off


Difficulty getting CSV Output

Such a great parser!  Thank you for writing it.  I'm not having any luck getting output in CSV format.  Both HTML and CSV options return HTML output in version 1.3.0.

Thank you Chad for your feedback! Glad you liked it. We will have a look
into the matter with the .CSV export.
As far as the further fields are concerned, we will need to re-research
this log in order to evaluate them properly which I would love to do but at
the moment I am afraid my schedule would not allow it.
Nonetheless I will try my best to catch up as soon as I can and add a
feature or two. If you have access to paid solutions you can always use USB
Detective. I am sure you know of it. It has incorporated our research to
their latest update.
Kind regards,
Evangelos Dragonas
Στις Κυρ 16 Ιαν 2022 στις 1:14 π.μ., ο/η chadtilbury <
@.***> έγραψε:

Such a great parser! Thank you for writing it. I'm not having any luck
getting output in CSV format. Both HTML and CSV options return HTML output
in version 1.3.0.
—
Reply to this email directly, view it on GitHub
https://github.com/theAtropos4n6/Partition-4DiagnosticParser/issues/1,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ3YSGPXHQEF54VG3RLCQ2DUWH5VPANCNFSM5MBQ7REA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Bump to 0.41.2 to remove solar and storage attributes

Just looked at the changes in 0.41.2 - looks good to me.


Fix bumpversion config and package listing

[Current coverage][cc-pull] is 92.98%

Merging [#21][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master        #21   diff @@
==========================================
  Files             2          2
  Lines           114        114
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            106        106
  Misses            8          8
  Partials          0          0

Powered by Codecov. Last updated by [32aa22b...f76765b][cc-compare]
[cc-base-branch]: https://codecov.io/gh/scrapy-plugins/scrapy-crawlera/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/scrapy-plugins/scrapy-crawlera/compare/32aa22bd4cb667122b2691ab6afae823dd494e2e...f76765b96c763cd69188074c2b28a04f52a73fcb
[cc-pull]: https://codecov.io/gh/scrapy-plugins/scrapy-crawlera/pull/21?src=pr

@eLRuLL , any chance you could review this in the next days?
Thanks.


Browser to view data

How can I view data that was saved? Is there any application that able to do this?

@aznj No, you can't. This is not available by design. Paper has been designed from the ground up as a scheme-less data storage meaning you and only you (who saved the data) can read them back properly. Paper serialize classes data into binary and requires the exact same classes to read data back. This helps Paper to be very fast and support seamless migration to the updated class structures. But as a downside – no data browser.
To look at the data (if you can read them programmatically) I'd sugggest to save it to JSON file once read is done. This is not a part of Paper and should be implemented on the client side if needed.


Editor StringName and Viewport optimizations

Added explicit return type to the SNAME macro, since it was returning StringName by value in some places.
Add some extra SNAME usages.
Change some ClassDB methods to use const StringName & arguments.
Cache the Window parent in Control because it's used in is_layout_rtl(), which is called often.
Only enable internal processing for viewports that need it.
Change CanvasItem::group to be a StringName because it's only used as that.

Thanks!

Cache the Window parent in Control because it's used in is_layout_rtl(), which is called often.

It's probably possible to cache the return value of is_layout_rtl(), and recalculate it only when NOTIFICATION_TRANSLATION_CHANGED or NOTIFICATION_LAYOUT_DIRECTION_CHANGED is received. I'll check it a bit later.


Relationship of [rw|ro] and the propagation options for volume

Can anyone please show me the relationship of [rw|ro] and [[r]shared|[r]slave|[r]private]? I found that I cannot use both options in [rw|ro] and [[r]shared|[r]slave|[r]private] in one command.

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

How did you provide the options? This seems to work fine for me;
docker run -v /foobar:/foobar:ro,private -it --rm ubuntu

thanks, I should use "," instead.


Add promise support for onClose handlers. fixes #69

oops, I forgot the tests.

@delvedor please review the tests. I'm not sure if they are correct this way.

hm... so the tests must be ES5, no async keyword

Do we need to update Typescript definition ?

Yes please
Il giorno ven 3 ago 2018 alle 20:04 HelloEdit<EMAIL_ADDRESS>ha
scritto:

Do we need to update Typescript definition ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mcollina/avvio/pull/70#issuecomment-410332049, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AADL4xf3Z4JxXIBqJHgSf41-ngELXrPuks5uNJCWgaJpZM4VuStU
.


DUOS-1912-draft

DUOS-1912

Have you read Terra's Contributing Guide lately? If not, do that first.

Label PR with a Jira ticket number and include a link to the ticket
Label PR with a security risk modifier [no, low, medium, high]
PR describes scope of changes
Get a minimum of one thumbs worth of review, preferably two if enough team members are available
Get PO sign-off for all non-trivial UI or workflow changes
Verify all tests go green
Test this change deployed correctly and works on dev environment after deployment

No longer necessary due to functionality in https://github.com/DataBiosphere/duos-ui/pull/1699


nextflow clone does not pull the latest github version

It seems to be pulling version 0.1 right now

is this one still an issue?


Remove gcm package from the repository.

I moved it to https://github.com/mpcomplete/flutter_gcm for now.

LGTM


Refactor the middleware related functions in AppRegistry. Their input take is messy.

Proposed solution.

Make new function to make a group of middleware.
Make function such that they accept only specified type of data, and not ...$args.
Make function that give name to the middleware. Using that name to access one middleware or group.
Update middleware function based on that.

More solutions could be in the way.

The following has been marked as complete.
We did update the entire AppRegistry and converted to Registry, and then also made some changes in the Router class to make sure that all of the above mentioned problems are solved.


Need to make port and ICMP on server tests mutually exclusive

Per the docs
if protocol is TCP, defaults to port 80. Protocol and port (shown below) are mutually exclusive - if ICMP is selected and port is provided, the request will error.`

https://github.com/william20111/terraform-provider-thousandeyes/blob/5a94a428ac448f17f2c4d8ae7695543ae2ded5eb/thousandeyes/resource_agent_to_server.go#L67-L82
So we need to handle this in validation

this has been resolved in a previous version.  This issue can be closed as resolved.


I need ideas from your side

请提出你的问题
Hi Sir,
I am using paddlenlp document intelligence model for image vqa. Please give ideas for these below 3 queries?

Model getting two much of memory usage. How to use optimize this?
How to send multiple question or how to process multiple question at a time?, [Ex, I will send 5 questions for a image, i need process parallel and get output with less memory consumption] Its there any way to solve this?
How to load paddlenlp models from my local folder?

Thank you for using it. We no longer maintain this feature.


quantile() does not work with columns of type Decimal

Code Sample, a copy-pastable example if possible
This snippet does not work (returns an empty DataFrame)
pd.DataFrame([
    Decimal(162.7646939720129171), Decimal(162.5470404198062433),
    Decimal(162.9334883683360258), Decimal(161.7975872981700753),
    Decimal(160.9450499461786868), Decimal(163.1648865446716900),
]).quantile(0.8)

Expected Output
This snippet works as expected (returns a Series with one value)
pd.DataFrame([
<PHONE_NUMBER>720129171,<PHONE_NUMBER>198062433,
<PHONE_NUMBER>683360258,<PHONE_NUMBER>981700753,
<PHONE_NUMBER>461786868,<PHONE_NUMBER>446716900,
]).quantile(0.8)

Shouldn't both snippets work the same way?
output of pd.show_versions()
INSTALLED VERSIONS
commit: None
python: 3.4.3.final.0
python-bits: 64
OS: Linux
OS-release: 3.13.0-86-generic
machine: x86_64
processor: x86_64
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8
pandas: 0.18.1
nose: None
pip: 8.1.2
setuptools: 20.9.0
Cython: None
numpy: 1.11.0
scipy: None
statsmodels: None
xarray: None
IPython: 4.2.0
sphinx: None
patsy: None
dateutil: 2.5.2
pytz: 2016.4
blosc: None
bottleneck: None
tables: None
numexpr: None
matplotlib: None
openpyxl: None
xlrd: None
xlwt: None
xlsxwriter: None
lxml: None
bs4: None
html5lib: None
httplib2: None
apiclient: None
sqlalchemy: None
pymysql: 0.7.2.None
psycopg2: 2.6.1 (dt dec pq3 ext lo64)
jinja2: 2.8
boto: None
pandas_datareader: None

I don't think this is supported. numpy.percentile doesn't even work on Decimals, so not much we can do.

its possible (esp in the new way .quantile works), but we don't support Decimal very much.

what we really need is Block support for Decimal. Though I don't see much actual utility for this.
closing as no action, but if you'd like to submit a PR would be ok with it.

Some things do work, eg mean, also in numpy:
In [57]: a = np.array([Decimal(160.9450499461786868), Decimal(163.1648865446716900)])

In [58]: a
Out[58]:
array([Decimal('160.94504994617869897410855628550052642822265625'),
       Decimal('163.164886544671702495179488323628902435302734375')], dtype=object)

In [59]: a.mean()
Out[59]: Decimal('162.0549682454252007346440223')

So I would rather raise this issue on the numpy side


Default to "None" for the limit

On the master branch, if I try to import with no limit set, I get the following error:
Invalid value for integer parameter limit: .

We should remove the datatype restriction and set the default to None. That way, it will work properly.

I think with this change, we have to change line https://github.com/girder/large_image/blame/master/sources/dicom/test_dicom/web_client_specs/dicomWebSpec.js#L115 back to return $('.g-validation-failed-message').html() === 'Invalid limit'; for the tests to pass.

I think with this change, we have to change line https://github.com/girder/large_image/blame/master/sources/dicom/test_dicom/web_client_specs/dicomWebSpec.js#L115 back to return $('.g-validation-failed-message').html() === 'Invalid limit'; for the tests to pass.

Done. Looks like the CI is passing now.


Problem with ModelCheckpoint

I have recently noticed that the keras.callbacks.ModelCheckpoint saves the model after updating its weights but it should save it before updating the weights!

@AminHP
Moving this issue to closed status as there has been no activity, in case you still face the error please create a new issue.


missing jsoniter-scala-macros

The jsoniter example code won't compile because of
object macros is not a member of package com.github.plokhotnyuk.jsoniter_scala

I think the reason is that its scope is Test here

I tried to use sttp in a client project.
It's not enough to add only
"com.softwaremill.sttp.client3"         %% "jsoniter"                        % sttpVersion,

to libraryDependencies.
Did I miss anything from the documentation?
Now I have a workaround. Adding the following made it work.
 "com.github.plokhotnyuk.jsoniter-scala" %% "jsoniter-scala-macros"           % jsoniterVersion % "provided",

Feel free to close this issue if it not a valid one.

Ah, I suppose if you want to use macro-based derivation with jsoniter, you need the additional dependency. That's how jsoniter works - probably adding this to the docs would be a good idea. That's however not something we can fix in code.

I see. thanks for the explanation!


Update phone-number-management-for-australia.md

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/2133
Added note under steps.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@CarolynRowe Changes look correct. Could you please review and merge? Thanks!

@tonysmit, are you OK with this change?


Added OSC commands

Added oscilloscope commands for DAQC2plate.

This looks great.  Sorry, I didn't see the PR when it came in.  I'll test this out and get back to you.  Do you have any notes about using this?  Any example flows that are useful to you that you could include?  Thanks!


Export PPTX as PDF

I am using python-pptx in my workflow as part of my test suite. One option that I would love to see added is the ability to save PowerPoints as PDFs. Currently I have an rough solution where I start a subprocess32 with the command
libreoffice --headless --invisible --convert-to pdf *.pptx
when LibreOffice is available. But from a cursory glance at the source, it appears it may be possible to export the presentation object as a PDF directly. I could use unoconv as well I believe, but it has a much smaller install base than LibreOffice.
I realize this is probably an edge use case, but I figured I would just throw the feature request out there.

+1
On Fri, Mar 24, 2017 at 4:04 PM, Jacob Kenin<EMAIL_ADDRESS>wrote:

I am using python-pptx in my workflow as part of my test suite. One
option that I would love to see added is the ability to save PowerPoints as
PDFs. Currently I have an rough solution where I start a subprocess32
with the command
libreoffice --headless --invisible --convert-to pdf *.pptx
when LibreOffice is available. But from a cursory glance at the source, it
appears it may be possible to export the presentation object as a PDF
directly. I could use unoconv as well I believe, but it has a much
smaller install base than LibreOffice.
I realize this is probably an edge use case, but I figured I would just
throw the feature request out there.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/scanny/python-pptx/issues/271, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFIi5e1FR-Pgk2_bYz0Q0CRYc_Mb2c_gks5ro9tsgaJpZM4MoX1C
.

Printing is a form of rendering (amongst other things I expect) and the scope of python-pptx doesn't include rendering. Extending into the rendering realm is a topic that comes up from time to time, but the short story is I don't see python-pptx ever having a renderer.
But that doesn't mean it can't have a handy interface into some other library that provides aspects like that.
If someone works out a smooth workflow between python-pptx and a stable package, we can definitely talk about how to integrate them better.


2.6.1 created error "cannot load such file addressable"

Yesterday, the tests in my project were passing.  Today, the fail because of

". . . /gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:274:in require': cannot load such file -- addressable (LoadError) 	from /usr/local/rvm/gems/ruby-2.1.3@ion_documentation_engine/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:274:in block in require'"

The only 2 gems that changed in the Gemfile.lock in my project between yesterday and today are

capybara from 2.6.0 to 2.6.1
selenium-webdriver from 2.49.0 to 2.50.0

Forcing capybara to 2.6.0 resolves the problem and the tests run again.
In the Gemfile.lock of the version that fails, it has addressable listed, so I don't think the gem is missing entirely.
capybara (2.6.1)
      addressable
      mime-types (>= 1.16)
      nokogiri (>= 1.3.3)
      rack (>= 1.0.0)
      rack-test (>= 0.5.4)
      xpath (~> 2.0)

Why would changing to capybara 2.6.1 be breaking tests?
Thank you for the help.

Hmmm -- it shouldn't be --- 2.6.1 was released because addressable wasn't being included in the current_path_query file - not sure why it would be failing for you.

What version of addressable was being loaded in your gemfile.lock?

Ok -- found it -- require 'addressable' would require addressable 2.4.0 ---   I will release Capybara 2.6.2 soon to fix that oversight

Awesome was just about to respond with a sample Gemfile.lock. :+1:
Thanks

I just had a same issue, I resolve the issue by requiring addressable/uri in /lib/capybara/queries/current_path_query.rb:1

:speech_balloon: @rezadehganpour

I just had a same issue, I resolved it by requiring addressable/uri in /lib/capybara/queries/current_path_query.rb:1
this is because the addressable is not a file

It is in addressable 2.4.0. The issue is that Capybara 2.6.0 created a dependency on addressable 2.4.0 but its gemspec allows any version of addressable. Changing the require  statement in current_path_query.rb, like you did, seems a better solution than constraining the version of addressable in the Capybara gemspec.
Thanks for reporting this @hb9358, and for starting to work on a fix, @jnicklas!

2.6.2 released - should be fixed

:clap: @twalpole

2.6.2 fixed the issue for me, thanks @twalpole and @hb9358 for reporting.

Yea same here. thanks!


Getting same result when using original 1.5 model and other 1.5 based model

I got same result when i tried to use other model from civitai. i modified the pretrained model path to downloaded .safetensor file. did i miss anything?
Also is there's anything i can help with?

plz give the config about your model path

plz give the config about your model path

pretrained_model_path: "magicanimate/pretrained_models/Realistic_Vision_V6.0_NV_B1.safetensors"
pretrained_vae_path: "magicanimate/pretrained_models/sd-vae-ft-mse"
pretrained_controlnet_path: "magicanimate/pretrained_models/MagicAnimate/densepose_controlnet"
pretrained_appearance_encoder_path: "magicanimate/pretrained_models/MagicAnimate/appearance_encoder"
pretrained_unet_path: ""
motion_module: "magicanimate/pretrained_models/MagicAnimate/temporal_attention/temporal_attention.ckpt"
savename: null
fusion_blocks: "midup"
seed:           [1]
steps:          25
guidance_scale: 7.5
source_image:

"inputs/applications/source_image/monalisa.png"
"inputs/applications/source_image/0002.png"
"inputs/applications/source_image/demo4.png"
"inputs/applications/source_image/dalle2.jpeg"
"inputs/applications/source_image/dalle8.jpeg"
"inputs/applications/source_image/multi1_source.png"
video_path:
"inputs/applications/driving/densepose/running.mp4"
"inputs/applications/driving/densepose/demo4.mp4"
"inputs/applications/driving/densepose/demo4.mp4"
"inputs/applications/driving/densepose/running2.mp4"
"inputs/applications/driving/densepose/dancing2.mp4"
"inputs/applications/driving/densepose/multi_dancing.mp4"

inference_config: "configs/inference/inference.yaml"
size: 512
L:    16
S:    1
I:    0
clip: 0
offset: 0
max_length: null
video_type: "condition"
invert_video: false
save_individual_videos: false

plz give the config about your model path

pretrained_model_path: "magicanimate/pretrained_models/Realistic_Vision_V6.0_NV_B1.safetensors"
pretrained_vae_path: "magicanimate/pretrained_models/sd-vae-ft-mse"
pretrained_controlnet_path: "magicanimate/pretrained_models/MagicAnimate/densepose_controlnet"
pretrained_appearance_encoder_path: "magicanimate/pretrained_models/MagicAnimate/appearance_encoder"
pretrained_unet_path: ""

motion_module: "magicanimate/pretrained_models/MagicAnimate/temporal_attention/temporal_attention.ckpt"

savename: null

fusion_blocks: "midup"

seed:           [1]
steps:          25
guidance_scale: 7.5

source_image:
  - "inputs/applications/source_image/monalisa.png"
  - "inputs/applications/source_image/0002.png"
  - "inputs/applications/source_image/demo4.png"
  - "inputs/applications/source_image/dalle2.jpeg"
  - "inputs/applications/source_image/dalle8.jpeg"
  - "inputs/applications/source_image/multi1_source.png"
video_path:
  - "inputs/applications/driving/densepose/running.mp4"
  - "inputs/applications/driving/densepose/demo4.mp4"
  - "inputs/applications/driving/densepose/demo4.mp4"
  - "inputs/applications/driving/densepose/running2.mp4"
  - "inputs/applications/driving/densepose/dancing2.mp4"
  - "inputs/applications/driving/densepose/multi_dancing.mp4"

inference_config: "configs/inference/inference.yaml"
size: 512
L:    16
S:    1
I:    0
clip: 0
offset: 0
max_length: null
video_type: "condition"
invert_video: false
save_individual_videos: false

unet_additional_kwargs:
  unet_use_cross_frame_attention: false
  unet_use_temporal_attention: false
  use_motion_module: true
  motion_module_resolutions:
  - 1
  - 2
  - 4
  - 8
  motion_module_mid_block: false
  motion_module_decoder_only: false
  motion_module_type: Vanilla
  motion_module_kwargs:
    num_attention_heads: 8
    num_transformer_block: 1
    attention_block_types:
    - Temporal_Self
    - Temporal_Self
    temporal_position_encoding: true
    temporal_position_encoding_max_len: 24
    temporal_attention_dim_div: 1

noise_scheduler_kwargs:
  beta_start: 0.00085
  beta_end: 0.012
  beta_schedule: "linear"

This is the original repository, not my repository. The use of safetensors was added independently of the windows version, the original doesn't support this.

@sdbs i am using yours
i merged from yours to mine
https://raw.githubusercontent.com/sdbds/magic-animate-for-windows/main/configs/prompts/animation.yaml

I think it's likely that the results don't look any different because the original reference was weighted too high.
Actually reading the model itself didn't report any errors or other problems.

I think changing some of the delicate models might make the chipping of the face or hands less...

@sdbds someone on the official thread said that U-NET from
motion_module: "magicanimate/pretrained_models/MagicAnimate/temporal_attention/temporal_attention.ckpt"
overwriting the U-NET we load from custom checkpoint

@sdbds someone on the official thread said that U-NET from
motion_module: "magicanimate/pretrained_models/MagicAnimate/temporal_attention/temporal_attention.ckpt"
overwriting the U-NET we load from custom checkpoint

Actually the custom checkpoint only affects the reference migration of the image, the final generation still relies on the official temporal model, just like animatediff. So it's not a bug, the example I gave is also just able to read anime's images.

@sdbds do you think output can be improved? because it is really bad. i tried 768x768 and it was even worse

@sdbds do you think output can be improved? because it is really bad. i tried 768x768 and it was even worse

it bases SD1.5 so 512X512 is best result with offical setting.
if I can split motion module,we can change the MM unet

@sdbds do you think output can be improved? because it is really bad. i tried 768x768 and it was even worse

it bases SD1.5 so 512X512 is best result with offical setting. if I can split motion module,we can change the MM unet

awesome i hope you can do

I think it's likely that the results don't look any different because the original reference was weighted too high. Actually loading the model itself didn't report any errors or other problems.

Tks for replying.
this sounds resonable for me

not bug


1000G callset?

Hi there,
Has the pipeline been run on the high coverage 1000G data released by NYGC? I'd love to get my hands on the callset if available!
Thanks,
Jared

Hi,  thanks for your interest! We are preparing the manuscript on these samples, where the callset will be delivered together. Paper will come soon !

Thanks, looking forward to it.


⚠ ClamAV has degraded performance

In 17a954a, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 68 ms

Resolved: ClamAV performance has improved in fe09763 after 5 minutes.


[FEATURE] allow seeing .md files in commit records in copado

idea is that we dont just see name, key, type and path but can also have a sneak peek at the content.
maybe adding a markdown rendering plugin for salesforce could also be really beneficial here, if that exists.
this was a request from our pilot client but it would seriously increase UX.
One could even consider showing it on the retrieve page but that would require a lot more time each time we refresh...

maybe sth like this? https://github.com/jsmithdev/lightndown


docs(DI articles - showing providers using object literals)

@wardbell I have extended both the DI cookbook and the dev guide to include the new object literal based syntax for defining providers.
Is this what you had in mind?
The cookbook was definitely easier to update since all the variations on provide(..) are lumped together in the same component. This made it possible to show both versions in all cases with little effort.
In the dev guide the components are much more granular, so I think it would sacrifice code readability if we go through and show all the samples using both flavors. Instead I focused on explaining the new syntax as a yet another alternative (there are three alternatives now).
cc: @PascalPrecht

Made minor tweaks. Good to go!


Fix (firefox:62): LIBDBUSMENU-GLIB-WARNING

Install dbus-x11 to fix the below warning in ECS Task log:
(firefox:62): LIBDBUSMENU-GLIB-WARNING **: 14:57:48.711: Unable to get session bus: Failed to execute child process ?dbus-launch? (No such file or directory)
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This fixes first part of issue #65

This fixes first part of issue #65


How do we achieve "sr.reveal('.box', 50)" with vue-scroll-reveal

Thank you for this vue plugin...
How do we achieve "sr.reveal('.box', 50)" with vue-scroll-reveal
I see in your example you apply it to each elements.
But if I want to specify severals classes or elements at once, like ".box" like the one in scrollrevealjs example, How do I get that?

I just pushed 1.0.7, which exposes the reveal() method from Vue.$sr and this.$sr, which should allow you to do what you want for your use case.

Thank you @tserkov ...
Working Excellently!


Rules silently ignored if one can't be found.

If you have a rule_files listing:
rule_files:
  - "/prometheus_config/bla.rules"
  - "/prometheus_config/bliep.rules

And ff the file bla.rules can't be found all rules are (silently ignored).

What version are you running? In 0.14.0 you should get an error logged at least (reproduced it this way).
ERRO[0000] Error loading rules, previous rule set restored: open test/test.rules: no such file or directory

Do you not see this error?
Previous rule set is the empty one on startup.
In 0.15.0 it should error on reading bad rule files and exit if that happens on startup. Missing files are not errored on (to be a bit liberal as one might add them later and adding whole directories like a/*.rules is possible). Other files are still read.
The new promtool is actually errors during config checking if there is a non-glob file that does not exist. Having more strictness in this direction seems fine.
Is that more like the behavior you would expect?

[ Quoting<EMAIL_ADDRESS>in "Re: [prometheus] Rules silently ign..." ]

What version are you running? In 0.14.0 you should get an error logged at least (reproduced it this way).
ERRO[0000] Error loading rules, previous rule set restored: open test/test.rules: no such file or directory

Do you not see this error?

Yep, I see it.

Previous rule set is the empty one on startup.

Hmmm, yes. Dropping all rules makes it easier to spot a bit one.

In 0.15.0 it should error on reading bad rule files and exit if that happens on startup. Missing files are not errored on (to be a bit liberal as one might add them later and adding whole directories like a/*.rules is possible). Other files are still read.
The new promtool is actually errors during config checking if there is a non-glob file that does not exist. Having more strictness in this direction seems fine.
Is that more like the behavior you would expect?

Jep, what you describe for 0.15.0 is exactly what I am looking for.
Thanks!
/Miek
--
Miek Gieben


Privacy Policy and Terms of Service links don't work

Problem
Privacy Policy and Terms of Service links don't work (user just redirected to home page).
Expected Behavior
Privacy Policy and Terms of Service links open pages with relevant info which matches our actual policy and explains in plain language privacy implications (like all transactions being available publicly on chain).
Steps to reproduce

Go to https://wallet.testnet.near.org
Try pressing Privacy Policy or Terms of Service links

@kcole16 looks like we need this for mainnet launch as well

@kcole16 do we have text for privacy policy / terms of service?

We can link to the near.org privacy policy: https://www.iubenda.com/privacy-policy/44958041?ifr=true&height=775&newmarkup=no
And working with Julie to get a Wallet-specific ToS

We need to make sure update e-mail templates as well https://github.com/near/near-contract-helper/pull/210/files
Also note that near.org Privacy Policy doesn't cover most important issue – that everything you do on NEAR blockchain is completely public info to anyone (like Twitter).

ToS shipped in https://github.com/near/near-wallet/pull/1649


Require ClientHello to have run before creating objects

Moderate refactoring / renaming of methods on Client with the main functional change that we now require the ClientHello RPC to have been run before creating any objects:

I renamed Client._verify to Client._init to make it more clear that it's an initialization step (we plan to use this method to retrieve the image builder version — and perhaps other information — that will be needed later).

I added a Client.authenticated property, which is set when Client._init runs. This is checked by _LocalApp._create_all_objects. We plan to remove _LocalApp; but I assume we'll just port _create_all_objects somewhere else? This may affect the behavior of Client.from_credentials (presumably by improving the error message returned for bad credentials?)

I renamed Client.unauthenticated_client to Client.anonymous, since the client can be in an unauthenticated or authenticated state depending on whether  ._init has been called, and the purpose of this method is to create a client without a token (i.e., not associated with any specific workspace)

I made a few marginal improvements to the code quality e.g. filling in some type annotations and docstrings, making the classmethod actually use cls (no practical effect now since we don't have Client subtypes), etc.

Resolves MOD-2674

BTW I guess Client.unauthenticated_client is technically a public method but we discourage users from touching Client directly; not sure if it needs a deprecation?


Find the actual issue keeping conhosts from exiting when tabs are closed

We have a job object for the conhost in PTY mode that is supporting the connection on each tab. But this was because the conhosts weren't exiting.
However, conhosts should just exit when they have nothing to do.
This represents taking the job object off and figuring out what the actual underlying issue is that is keeping the conhost from realizing it can just exit.

We think this was actually probably resolved by #2525. It's not happening anymore, so we're going to resolve it for now?


I can't get the iOS project to build

It gives me an error like this:
ld: warning: directory not found for option '-F/Users/sakarit/repositories/xxx/ios/Build/Products/Debug-iphonesimulator/GTMOAuth2'
ld: warning: directory not found for option '-F/Users/sakarit/repositories/xxx/ios/Build/Products/Debug-iphonesimulator/GTMSessionFetcher'
ld: warning: directory not found for option '-F/Users/sakarit/repositories/xxx/ios/Build/Products/Debug-iphonesimulator/GoogleToolboxForMac'
ld: framework not found GTMOAuth2
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Do you have any idea what could be wrong?

Okay I found out what was wrong. The thing that I did was that I closed the Xcode, installed the cocoapods and then reopened my project with the original .xcodeproj file instead of the new .xcworkspace file. Of course it didn't not work. 😄
Maybe this could be added to the installing instructions!

@sakarit Added to README.md :)


Specifying a /root Mount that Is not $HOME Does Not Work.

In my first start of K2, I wanted to isolate my machine from any files that might be created by the startup.  So I created a k2wrap directory, and specified that as my /root mount.  This work for some items, but there are many assumptions that  there will be other $HOME directories available there.  e.g. .ssh, .aws.  This was not spelled out in the documentation.
Also, there may also be some other hard coded assumptions that ~/ is the mount.
Please document this better, and if it is not allowable to use a mount other than $HOME say that!
I would advocate allowing the /root mount to be wherever the user wants.

this is a documentation issue.  k2 needs to have ~/.aws and ~/.ssh available in the running container's root user directory.  how the user gets those directories into the running container can take many forms.
docs will be updated for this.

may be done already.  check out hte k2 readme.

this is partially done.  Going to try and knock it out while conformance tests run.

Just to add my $0.02, this is a purely documentation thing. I have tested using a mktemp -d directory with the .aws/credentials file and an id_rsa file (properly referenced in the config.yaml) and it worked fine.


The real bug

I think this is the real bug.
https://github.com/waldo-seg/waldo/blob/master/scripts/waldo/data_manipulation.py#L159
why does i have axis=1 and j have axis=0?  they seem swapped.

I think you are mixing up two different indexing conventions- or maybe you
are assuming that different parts of the code have different indexing
conventions.
We are using the same indexing convention everywhere.  It's (height,
width).  The first index (i or x) is always height, the second (j or y) is
always width.
On Thu, May 24, 2018 at 5:25 PM, Yiwen Shao<EMAIL_ADDRESS>wrote:

since we are comparing x,y with x+i, y+j, we need to firstly roll the
matrix in the horizontal direction, which is axis=1, and then roll in
vertical direction, which is axis=0.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/waldo-seg/waldo/issues/51#issuecomment-391868195, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADJVuxox2WxBtIYKar9ULdlZ2TTo48Mqks5t1yVDgaJpZM4UNA4c
.

I guess it would be clearer to just never use x and y as variables for
indexes, as that would imply a (width, height) indexing which is not what
we are using.
On Thu, May 24, 2018 at 5:28 PM, Daniel Povey<EMAIL_ADDRESS>wrote:

I think you are mixing up two different indexing conventions- or maybe you
are assuming that different parts of the code have different indexing
conventions.
We are using the same indexing convention everywhere.  It's (height,
width).  The first index (i or x) is always height, the second (j or y) is
always width.
On Thu, May 24, 2018 at 5:25 PM, Yiwen Shao<EMAIL_ADDRESS>wrote:

since we are comparing x,y with x+i, y+j, we need to firstly roll the
matrix in the horizontal direction, which is axis=1, and then roll in
vertical direction, which is axis=0.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/waldo-seg/waldo/issues/51#issuecomment-391868195,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADJVuxox2WxBtIYKar9ULdlZ2TTo48Mqks5t1yVDgaJpZM4UNA4c
.

Now I see it. Thanks!

Good news! Now the results are perfect!

Very nice! While you are making a PR, could you please also resize the images in data preparation so that they are never cropped?

I did the padding part, i will add it.

Thanks @aarora8! I will make a PR soon. You can make you modifications on top of it.

OK, thanks!

ok, thanks.


services.mjolnir.pantalaimon.options has no effect

Describe the bug
The services.mjolnir.pantalaimon.options option has no effect.
For example, given the following config:
  services.mjolnir = {
    homeserverUrl = "http://localhost:8008";

    pantalaimon = {
      enable = true;
      username = "...";
      passwordFile = "...";
      options.listenPort = 8100;
    };
  };

running nixos-rebuild switch and then inspecting with nixos-option services.pantalaimon-headless.instances gives:
Value:
{ mjolnir = {
  dataPath = "/var/lib/pantalaimon-mjolnir";
  extraSettings = { };
  homeserver = "http://localhost:8008";
  listenAddress = "localhost";
  listenPort = 8009;
  logLevel = "warning";
  ssl = true;
}; }

Note that the port is wrong.
If I manually add services.pantalaimon-headless.instances.mjolnir.listenPort = 8100; then I can achieve the desired affect, but that is not a great situation.
Steps To Reproduce
Steps to reproduce the behavior:

use the above config
observe that it's listening on the wrong port

Expected behavior
the setting should work
Notify maintainers

@jojosch
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.76, NixOS, 21.11 (Porcupine)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.5pre20211007_844dd90`
 - channels(root): `"nixos-21.11pre331460.931ab058daa"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module: pantalaimon

Hmm, I tried fixing it but I'm not really able to, probably due to a lack of knowledge in NixOS modules. The mjolnir-module was more or less my fist "real" NixOS module.

In the mjolnir module when merging the { homeserver = cfg.homeserverUrl; } and cfg.pantalaimon.options there were brackets missing:
nix-repl> lib.mkIf true { homeserver = "https://matrix.org"; } // { listenPort = 8100; }
{ _type = "if"; condition = true; content = { ... }; listenPort = 8100; }

vs
nix-repl> lib.mkIf true ({ homeserver = "https://matrix.org"; } // { listenPort = 8100; })
{ _type = "if"; condition = true; content = { ... }; }

but when using the fixed version I get the following error (e.g. when trying to run the mjolnir test):
error: The option `services.mjolnir.pantalaimon.options.homeserver' is used but not defined.
(use '--show-trace' to show detailed location information)

When setting a default value for the homeserver option the error goes away but the homeserver value then cannot be changed from the default value defined in pantalaimon-options.nix ... but the listenPort, listenAddress and other options can be changed and appear in the generated config ...