HNSW memory footprint

I am working with large image descriptors datasets (1e8-1e9 32-d vectors) and, based on initial tests, HNSW has an excellent scaling complexity. However, the problem arises from the memory footprint, and I am not sure whether this is the limitation of my usage, the method or, maybe, some inefficiency in the implementation.
I have read somewhere that a high-recall graph index is possible while taking around 20-40% of the dataset size in memory, but I see numbers around 300% (30gb of ram used after addDataPoint(), ~120gb of ram used after createIndex()).
I use python bindings and the parameters in that particular case were M=15, ef=15, efConstruction=200,
Based on the paper, it should be taking around 15 * 2 * 130M vectors*(4-8) bytes which should traslate to 15-30gb

Thank you for the tests.
At the default settings HNSW takes a lot of memory in case of L2 and cosine distance. This is because HNSW makes a full copy of index+data in the optimized form. This doubles the amount of memory used, but speeds up the search, especially when fast distance functions are used. In future the issue can be solved by building the optimized index from the scratch.
There are several workarounds at the moment.

Do not build the optimized index by setting the "skip_optimized_index" build parameter to 1.
If you have a larger memory server available, you can build the optimized index, save it to disk and reload it later on a smaller memory machine. In this case only the optimized index containing all data will be loaded.

Another option is to reduce the M and maxM0 parameters. The memory consumption is mostly defined by the maxM0, and by default it is set to 2M. Such selection usually results in getting higher performance at high accuracies, however in some cases maxM0 can be decreased to 1.5M or even M without a significant loss in accuracy.
So, you can try to reduce M and/or set maxM0 to 1.5*M or M.

Hi @yurymalkov, Greg is saying that his memory footprint is 10x compared to what's possible. It's not clear how this happens, I am running some tests now.

@yurymalkov thank you for the suggestions, I will run some tests with those amendments!

Dear @funtusov one more note.
I have just finished running a test with 5M sift vectors (0.5KB per vector). After reading the data, I get 3.3 GB virtual and 2.7 GB resident, which seems to be ok, because data would use 2.5 GB. After creating an index with skip_optimized_index=1 and M=15, the memory footprint is: 5.7 GB virtual and 4.7 GB resident.
It's larger than expected, so we will have to investigate the issue more carefully in the future. Perhaps, it can be reduced.
Anyways, please, note that addDataPoint creates **a duplicate ** of the vector in memory. Therefore, one shouldn't keep vectors in the Python data structures, just read them one by one and pass to NMSLIB's addDataPoint. For example, one can do this using the following function:
def read_data(fn):
    with open(fn) as f:
      for line in f:
        yield [float(v) for v in line.strip().split()]

@searchivarius Thanks for the clarifications! Another potential improvement would be to be able to store data points with a user-defined type. For instance, I am using ORB descriptors, which are 256 bits, and OpenCV stores them as 32 8-bit integers, which could give a 4x memory reduction compared to a 32bit float

This is currently possible, but it requires implementing a custom space. It's a bit of a headache, but it's not overly complicated.

@searchivarius Could you please point to the general direction (which custom space/at what step is it used)? Is it possible on the python side?

Unfortunately, this requires dabbling with C++ code first. A basic description is given regarding creating a space in C++ is given in https://github.com/searchivarius/nmslib/blob/master/docs/manual.pdf section 7.2 (creating a space).
At the very least, one has to implement the function that creates an object from the string and the function that computes the distance.
Furthermore, a custom space isn't clearly supported by vector bindings. One has to use generic Python bindings (that pass only strings) or a query server.

It dawned on me that the initial calculation of how much memory would the edges take (maxM0 = 30 * 130M vectors*(4-8) bytes) didn't include into account the enumeration of vertices.
Am I right that for each edge we store the id's of two vertices + the measure of distance between them? So, it would take, say, 2*32-bit unsigned integers + a 32 bit float?
If that is the case, is there any theoretical way to store only some local references to the vertices, not global ones?
For cases where both the vectors and distances are quite small (i.e. 256-bit orb descriptors with hamming distance), storing the edge count dwarfs the datatset itself memory storage-wise.

Hello Greg,
The distance between the vertices is not stored in memory. Only one id per link is stored (links possibly can be non-symmetrical).
We have not yet investigated the reason of the excessive memory usage during the construction. I think this may be an overhead of using c++ structures, but I am not sure.
You can check the minimum memory footprint by looking at the disk size of stored hnsw index. It includes both data and the stored links and should be slightly bigger than you should predict by rough estimates. After saving/loading it should the memory usage should be reduced considerably.

Probably due to memory fragmentation one should multiply the minimum consumption by some coefficient (likely less than 2). A more compact version of the index can be created (or saved to disk and mmaped back), but it would require more work to implement.


[Env] Default to always printing gemfile & gemspecs

This will save a lot of time spent asking for them, and is something I thought we'd already done

This is another nitpick and not necessary to your changes but while you're changing Env i would like to change the way we call the report from Bundler::Env.new.report to just Bundler::Env.report.
Is this a problem because backwards compatibility? Any thoughts?

That should probably be in a separate PR

@bundlerbot r+


Clear timer on activation

This resolves a potential lifecycle edge case where applicationDidBecomeActive handler is called before the scheduled “callClose” happens (this timer is setup on applicationWillResignActive handler and it is scheduled to happen half a second after). In this case, the SDK will initialize and then be de-initialized when callClose triggers, causing any subsequent call to Branch’s methods will make the application to crash with the error: “Branch SDK Error: making request before init succeeded!".
We've been unable to reproduce this but logically, it is possible.
@derrickstaten

Any changes to lifecycle make me nervous. I think this needs to be extensively tested by QA before pushing to master. It seems relatively innocuous but could do great harm if there are cases we aren't anticipating. 👍  pending rigorous QA.

Partner also confirmed this resolved his issue.


az functionapp deployment failures

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az functionapp deployment user show
Errors:
'function' object has no attribute 'metadata'
Traceback (most recent call last):
Temp\pip-install-3pnsb50r\knack\knack\cli.py, ln 206, in invoke
azure\cli\core\commands\__init__.py, ln 608, in execute
azure\cli\core\commands\__init__.py, ln 666, in _run_jobs_serially
azure\cli\core\commands\__init__.py, ln 659, in _run_job
Local\Temp\pip-install-3pnsb50r\six\six.py, ln 703, in reraise
azure\cli\core\commands\__init__.py, ln 636, in _run_job
azure\cli\core\commands\__init__.py, ln 306, in __call__
azure\cli\core\commands\arm.py, ln 749, in handler
azure\cli\core\commands\arm.py, ln 761, in show_exception_handler
azure\cli\core\commands\arm.py, ln 747, in handler
mgmt\web\v2019_08_01\operations\_web_site_management_client_operations.py, ln 40, in get_publishing_user
AttributeError: 'function' object has no attribute 'metadata'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az functionapp deployment user show

Expected Behavior
existing deployment credentials
Environment Summary
Windows-10-10.0.18362-SP0
Python 3.6.6

azure-cli 2.1.0

Additional Context

functionapp

@Selekena Apologies for the late reply. Could you please let me know if you are still facing the same issue with most recent version of Az CLI? Awaiting you reply.


Fixes AttachmentInternal cast exception

Whoops, opened a PR to the wrong repository.
Sorry about the trouble!


Revert "CONFLUENT: kafka-run-class: set default log dir to /var/log/kafka"

This reverts commit bc070fa9d61ae5e9730fc5f56ba012979bf98ed1.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Thanks for the review. Merging to master, 2.4. 2.3


Process / Sample to configure with own provided Certificate ?

AFAIK , there is no documentation on how to do with above ? I think I know the process (see below) but would be good to have confirmed / a sample powershell or similar

Import Cert
Export Cert as .cert
ReadIn as Base64 text
run Powershell command to add base64 to Azure AD Application ID

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

ID: 17f663ca-635a-10ab-d78a-f60424a5484c
Version Independent ID: cd178cad-c4c4-16a0-ca02-ac336165cad7
Content: Configure the Azure MFA NPS extension
Content Source: articles/active-directory/authentication/howto-mfa-nps-extension-advanced.md
Service: active-directory
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

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

Opened backlog item to see what can be created to support this. Working with the PG to get an update.

#please-close


Can't get bindto to bind

Bringing in a number of charts into a SPA, but can't get them to bind to the container.
In an external js data file I specify the binding like this, for chart42Data:
bindto: document.getElementById('chart42Graph')
I import the data, and initialize the data:
this.handler.$emit('init', this.chart42Data)
Hoping it ends up in this container, but it doesn't:
<vue-c3 :handler="handler" id="chart42Graph"></vue-c3>
Instead, the last item I initialize attaches to all vue-c3 containers.
What am I missing?

Also, when trying to build my application, I get this error:
ERROR in build.js from UglifyJs Unexpected token: punc (() [./node_modules/vue-c3/dist/vue-c3.umd.js:26,0][build.js:29893,11]
Any suggestions?

I'll look into it.

So sorry, I forgot to update this, but the auglifyJS error was unrelated.
Also, for various reasons I switched to Billboards so I'm good now.


inherit box-sizing

references:

http://www.paulirish.com/2012/box-sizing-border-box-ftw/
https://css-tricks.com/inheriting-box-sizing-probably-slightly-better-best-practice/

@jonathantneal What do you think? Is it worth to merge?

@jonathantneal Do you mind merging my PR?

It would not be appropriate, as this is not actually a mistake as listed at https://wiki.csswg.org/ideas/mistakes
However, you can always assign { 'box-sizing': false } and use something like sanitize.css


Add Monolog handler

Q
A

Branch?
2.1

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

License
MIT

This PR implements a Monolog handler and closes #790, which is a highly requested feature. Code is a stripped down version of the one submitted by @HazAT in the official Monolog repository. There are a few questions to discuss before merging:

Like in the original PR, the handler needs an hub instance to work, but since the current hub is something that can change during the lifecycle of an application (e.g. when the Hub::setCurrent() method is called) it's not clear how this should affect the handler itself: e.g. if the handler is binded to an old or different hub than the current one it will report errors to the wrong client (if it's different between hubs)
Since Monolog is a logger library a message is likely to always be present along with the context that may include the exception and imho we want to log it regardless. Calling the captureMessage method provides no way to pass the exception while calling the captureException has no way to pass the message. To solve the issue, instead of calling those methods like in the original PR, I'm building the raw payload of data to build the event from and I'm using the captureEvent method directly. Since the two methods mentioned before have a slightly different behavior regarding the stacktrace capture (attach_stacktrace option), the same exact behavior cannot be accomplished with such way

We're missing just one small thing! The changelog!

Done, I always forget to update it!

Amazing! Thank you 👍

Trying to update but am being blocked by this PR not being released. What's the ETA? :smile:

No ETA, but this will be released with 2.1. Check out the milestone for progress: https://github.com/getsentry/sentry-php/milestone/2

Alright, thanks for the reply! I'll check back in a while and stay at sentry/sentry-symfony 2.x for now as it seems to not cause issues with Symfony 4.x


Recommended migration of library to PSR-4

Heads up!
Currently, the code is following PSR-0 standards of PHP. It is recommended to follow and migrate the library code to PSR-4 standards.
Thank you!

Thank you for your feature request. This client library is currently in maintenance mode. We are fixing necessary bugs and adding essential features to ensure this library continues to meet your needs for accessing Google APIs. After careful consideration, we have decided not to proceed with this enhancement. If you disagree, please feel free to re-open this issue and explain why you believe it is essential for users of this client library.

@mattwhisenhunt careful consideration? Really? Meaning you considered remaining obsolete. This API is useless in a modern era. Nonetheless, this API needs a full rewrite anyways.

@timothymarois HEY-YO you gotchme. It is a canned response.
I guess another way of putting it would have been: We are in the middle of a full rewrite anyways but we are playing are cards close to the chest until we know exact release dates. We are really excited that this new version is going to address some of the shortcomings of the current client library. Additionally, we want to make sure we are supporting current users of this client library especially with issues that are deal breakers. That is why we really encourage new issues being created and triaged and addressed. We also don't want to fall victims to the CADT model, so I am trying to get the bug count down as close to 0 as possible. That is a lot of work to do so we have employed some techniques, courteous, canned responses in this case, to get things done faster.
Gratifyingly, we have a fantastic community of developers from all over the world that are willing to jump in and test code and answer questions and submit PRs. So if my consideration seems too brief to be careful it is only in an attempt to make it also timely but never final. It's my attempt to avoid the unnecessary effort of others and also give the original feature requester info they need to make decisions as soon as possible.

+1 @mattwisenhunt
Have you seen the new labels with descriptions?
I feel like a maintenance label with this wonderful response of yours as the description would be great.
It would also be wonderful if you googlers played the promise card on EVERYTHING. Lol
I just want to async n' wait on your hand already. :)


Fix: Add a missing Material Widget in the history

Hi everyone,
From the history, opening a product and then going back to the history generates an issue with a missing Material widget.
The fix is not that hard to implement.
Before: Before.webm
After: After.webm

Codecov Report

Merging #4111 (4cf8ff0) into develop (af9f28b) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##           develop    #4111   +/-   ##
========================================
  Coverage    10.84%   10.84%
========================================
  Files          273      273
  Lines        13516    13512    -4
========================================
  Hits          1466     1466
+ Misses       12050    12046    -4

Impacted Files
Coverage Δ

...cards/product_cards/smooth_product_card_found.dart
0.00% <0.00%> (ø)

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

https://github.com/openfoodfacts/smooth-app/assets/1689815/26af2eec-4e25-44af-8098-7ec0c439a376

screen-20230609-211104.mp4

It's another issue:
======== Exception caught by scheduler library =====================================================
The following assertion was thrown during a scheduler callback:
There are multiple heroes that share the same tag within a subtree.

Within each subtree for which heroes are to be animated (i.e. a PageRoute subtree), each Hero must have a unique non-null tag.
In this case, multiple heroes had the following tag:<PHONE_NUMBER>514
Here is the subtree for one of the offending heroes: Hero
  tag:<PHONE_NUMBER>514
  dependencies: [_ModalScopeStatus]
  state: _HeroState#d52c2
When the exception was thrown, this was the stack:
#0      Hero._allHeroesFor.inviteHero.<anonymous closure> (package:flutter/src/widgets/heroes.dart:283:11)
#1      Hero._allHeroesFor.inviteHero (package:flutter/src/widgets/heroes.dart:294:8)
#2      Hero._allHeroesFor.visitor (package:flutter/src/widgets/heroes.dart:322:13)
#3      SingleChildRenderObjectElement.visitChildren (package:flutter/src/widgets/framework.dart:6363:14)
#4      Hero._allHeroesFor.visitor (package:flutter/src/widgets/heroes.dart:328:15)
#5      ComponentElement.visitChildren (package:flutter/src/widgets/framework.dart:5020:14)
#6      Hero._allHeroesFor.visitor (package:flutter/src/widgets/heroes.dart:328:15)
#7      ComponentElement.visitChildren (package:flutter/src/widgets/framework.dart:5020:14)


Bug fix to provide default CNI networking.

Refer CPU-Manager-For-Kubernetes fails to launch when using kubeconfig multus #14 for more info

Tested with CMK. Works fine!


[APK Management Server to Agent Client Communication] Design CRDs

Description:
This is to define CP and DP CRDs.

CRDs Document - https://docs.google.com/document/d/1CL5ug6neMn-caLbjzLOJms8TNuMSw9evSaEHjQRFudQ/edit?usp=sharing
Tasks -

[ ] Runtime API to k8 server - Amila
[ ] Agent to k8 server - Naduni


Breadcrumbs not responsive on mobile

For Bug Reports

BookStack Version: Demo Website
PHP Version: ^
MySQL Version: ^

Expected Behavior
On mobile, the breadcrumbs should either be hidden or collapsed.  (I achieved this with a quick css fix).

Actual Behavior
Breadcrumbs are non-responsive and take up space when reading articles.

@spaceraccoon Thanks for reporting. Will be in the next bugfix release. Made them collapse to just icons on mobile and prevented the 'toolbar' sections vertically collapsing.

Thanks Dan! That's much better than the quick and dirty CSS solition.
On Sat, Jul 22, 2017 at 11:57 PM Dan Brown<EMAIL_ADDRESS>wrote:

@spaceraccoon https://github.com/spaceraccoon Thanks for reporting.
Will be in the next bugfix release. Made them collapse to just icons on
mobile and prevented the 'toolbar' sections vertically collapsing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BookStackApp/BookStack/issues/426#issuecomment-317188920,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARj5GVNzfO0e7CV8HS6dVPe_8GKBLBRxks5sQg3egaJpZM4OODpz
.


Blank Maps when There is No Farm

I think it's better to hide the maps instead of displaying a blank maps.

Let me mention @alcatrazia for this enhancement feedback. We can elaborate more about the UX. Thanks for your feedback.

The enhancement had been implemented in this commit https://github.com/Tanibox/tania/commit/01dced4c71d81ad2608c8990ff27d3fde873df05. It will be released for v1.1.


Request: add some way to apply the "save actions" to all files, everywhere in the project

Describe the solution you'd like
I know it triggers each time you save, but maybe as a start, it could be nice to apply the changes to all files.
If this is already possible, please let me know.
For what product
Android Studio.

Hello @AndroidDeveloperLB, thank you for your continued interest in the plugin. Yes, you can use "batch mode" for that:

Activate batch mode: https://github.com/dubreuia/intellij-plugin-save-actions#activation
Then use the following action: "Code > Save Actions > Execute on multiple files" and following the scope dialog

@dubreuia Oh cool. Thank you!


Warning node message without use http2

I'm not using http2 and by console node show me always this message
(node:4046) ExperimentalWarning: The http2 module is an experimental API.

Yeah if you load up roads-server it pulls in http2. Just pulling in http2 logs that warning.
Your code won't use http2 unless you use roads-server's http2 object.
I can tweak this so the message shows up later, but it's low priority right now.

This is technically a roads server issue, so I'm going to close this


EventHubBufferedProducerClient: Partition Assignment

Summary
Publishing events using the EventHubProducerClient is optimized for high and consistent throughput scenarios where an application would like to assert control over collecting events into batches and the decision of when those batches are sent to Event Hubs. Developers are expected to build and manage batches according to the needs of their application, while taking into account the constraints that Event Hubs places on which events may be batched together. This allows developers to prioritize trade-offs between ensuring batch density, enforcing strict ordering of events, and publishing on a consistent and predictable schedule.
Feedback from developers has indicated that this approach quickly becomes complicated in real-world applications and demands applications bear the burden of that complexity. A frequent ask is for the Event Hubs client library to include functionality to abstract that complexity away from applications and manage batching, concurrency, and sending implicitly. Developers have let us know that they would like a higher-level type similar to the event processor, but for publishing events rather than consuming them.  This is the goal for the EventHubBufferedProducerClient.
Scope of Work

Implement the partition assignment logic, covering:

Events enqueued for automatic assignment are distributed using a round-robin approach to all partitions
Partition keys are hashed using the "legacy" Event Hubs service strategy (details are Microsoft internal)

Out of Scope

Basic functionality;  this is covered under a dedicated issue

Flush and clear functionality;  this is covered under a dedicated issue

Success Criteria

The design has been implemented; developers can use the producer with partition assignment functional.

Events are assigned random partitions, including those with partition keys.

The tests necessary for its validation have been created or adjusted and pass reliably.

The existing test suite continues to produce deterministic results and pass reliably.

Related Issues and References

Design Proposal for the EventHubBufferedProducerClient
EventHubBufferedProducerClient: Infrastructure and Functionality (#23480)
EventHubBufferedProducerClient: Flush and Clear (#23482)

Further discussion can be found here:  https://gist.github.com/jsquire/d4a42ab3fd898e61a278317557b6dbbb


Fixed sidebar scrolling

Sidebar on main window would not scroll properly, and resizing the window would clip the UI on smaller screen sizes.

Scrolling now functions correctly for me!


fix bug of show_result

Fixed the bug that could not save the result picture when passing in out_file parameter. Relevant issues and pull requests are #613 and #403 .

Hi @zhijl ！
First of all, we want to express our gratitude for your significant PR in the mmdetection project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.
We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/raweFPmdzG
If you have WeChat，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤


Add checking of version on release branches

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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 23.659%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
60

Relevant Lines:
258

💛  - Coveralls


CAMEL-11925: Migrate atmos properties file configuration to component properties

https://issues.apache.org/jira/browse/CAMEL-11925

LGTM

Thanks the PR has been merged. I'm closing this.

@davsclaus I guess it would be useful to backport to 2.20.x too. WDYT?

Yeah sure the old code is clunky

Backported to 2.20.x too


It would be nice to show support for white-space-collapsing css property

It would be nice to show support for white-space-collapsing css property

This issue was imported from Google Moderator
Moderator votes: +33
Issue added by James@Sharewire on 2012-04-04
To vote this issue up or down, simply include +1 or -1 in your comment.

+1

+1,
Anyway it seems that it's fully supported everywhere: https://css-tricks.com/almanac/properties/w/whitespace/

+1

+1 * 1000

+1

+1

+1

+1

+1

There appear to be no plans to support this in any browser, so have added it as a basic search results entry until that changes:
http://caniuse.com/#search=text-space-collapse


Add note for configMap volume.

You must create the ConfigMaps first before you can use it.

/approve
/lgtm


"Bug" fixed that FFmpeg with the CancellationToken doesn't always exit.

If you call Cancel() on the CancellationTokenSource and pass the CancellationToken to a Task like Engine.ExecuteAsync(string arguments, CancellationToken cancellationToken), ffmpeg.exe is not always terminated. Now it is checked whether the process is still running and if so, ffmpeg.exe will be terminated.
I'm new to GitHub. For my use case, this fix works. I only tested it in my scenario and recommend to test it again before merging. Please do not merge this pull request if the current behavior is intended to cancel only the task itself and dont kill the ffmpeg.exe process.

a global fix has been pushed to resolve this issue


stax convention for batch / single-instance?

Just a quick question. I'm writing some extensions for stax and I'm wondering if you have a convention regarding single-instance vs batch input.
For instance, in pytorch and keras all input is expected to be a batch of inputs, with the first axis being the batch axis by convention.
Looking at stax.Flatten, it does look like a similar convention is used in stax. Other transformations are compatible with both single-instance and batch inputs.
The reason I'm asking is that I found myself writing conditionals to deal with both, like
if input.ndim >= 2:
  # something something vmap
else:
  # single-instance behavior

I prefer not to have to do this and assume that all input is batched.
Is the stax convention that all input is batched?
Thanks!
PS I love the jax project, keep up the good work!!

Yes, stax is written to act on a batch of examples at once.
(There has been some interest in writing an “unbatched” neural net library in JAX and relying upon vmap for vectorization, but there are a few weird neural net layers like batch norm that act across a batch, and it isn’t clear yet that we can always match the performance of “pre-batched” neural net layers.)

I’ll mark this as closed since the original question is answered, but feel free to follow-up

Thanks @shoyer !!


Save checkbox state

it looks like changes to the way that histograms are being written/read back are causing the failing tests

@codymccoy I will review and address any other failing tests if necessary after you'd have the chance to work on #707


Plugin can't use when use comments plugins with ajax

Reported by dien | December 12th, 2010 @ 10:36 PM
Plugin can't use when use comments plugin with ajax.

Need more information about problem. What exactly not working?

Had this issue...I recreated recaptcha in my ajax form submit callback:
Recaptcha.create("public_key",
"element_id",
{
theme: "red",
callback: Recaptcha.focus_response_field
}
);
In order for it to work I had to have my recaptcha input filed wrapped in div id="field_id."   Must be an ID, not a class.

You can define the id of the div like this:
$this->Recaptcha->display('div' => array('id' => 'myId'));
Check the docblock of the display method.


Update remote-management.mdx

Updated instructions to create systemd service for linux.

use sensible log location
add [install] section so service can be enabled
add systemctl commands to enable and start service
add system log command to debug any potential issues

Summary

Screenshots (optional)

Documentation checklist

[ ] The documentation style guide has been adhered to.
[ ] If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
[ ] Files which have changed name or location have been allocated redirects.

Hey @adegoodyer, thanks for the PR. The commands shown in the remote tunnel dashboard flow should automatically create and enable cloudflared.service. Therefore, the instructions are oriented towards updating the existing cloudflared.service file. May I ask why you needed to create cloudflared.service from scratch?

Tested out the remote tunnel configuration steps and confirmed that the docs were missing a restart command. Thanks for the suggestions!

Tested out the remote tunnel configuration steps and confirmed that the docs were missing a restart command. Thanks for the suggestions!

@ranbel You're most welcome! In future I'll shape the PR more in line with Cloudflare paradigms and provide more evidence of the testing I did to save a bit of work your end.


refactor!: Simplify over-complicated platform adapter APIs

I remembered a wart in the API of the core (non-subclassing) Windows adapter, which I mentioned in this comment on a winit issue. Basically, handle_wm_getobject doesn't really need to return an intermediate struct that is then converted into an LRESULT in a second step. Any window implementations that need to acquire a mutex, mutable borrow, or the like in order to handle the WM_GETOBJECT message can work around the reentrancy issue by storing the adapter in an Rc, obtaining a reference to the adapter, then calling handle_wm_getobject while the mutex/borrow/whatever isn't held. I decided that this workaround is better than complicating the API for everyone.
Then I realized that, by the same logic, we really don't need to have the update methods return a QueuedEvents struct that the caller then has to call raise on. On Windows, that separation could theoretically have allowed the updating and the raising of events to happen on different threads. But then I ended up making it all single-threaded anyway on macOS. So I decided to require that the top-level Adapter struct on Windows be bound to the thread that owns the window, while keeping the ability for UIA to run most of the provider methods on another thread. In any case, I eliminated QueuedEvents, so the API is simpler.

Let's put this on hold until I figure out what, if anything, I'm doing about #295. I just realized that this PR and some resolutions to that issue may conflict with each other.

I'm abandoning this PR. After thinking about this some more, I believe my earlier instinct was correct; it's a more decoupled design, and it might be necessary for integrating lazy initialization into the core Windows and macOS platform adapters as I'm thinking about doing (see #295). In particular, we shouldn't lightly discard the ability to apply tree updates on a background thread on Windows. If anything, we should figure out a way to do that on macOS as well, since everything else on macOS is bottlenecked on the UI thread already. And the API I proposed in the winit issue comment that I mentioned earlier, that got me started on the path of this PR, probably needs rethinking anyway in light of #295.


expected/actual editor for failing tests has display issues when in inline mode

Does this issue occur when all extensions are disabled?: Yes (except for necessary extensions to run Java tests)

VS Code Version: 1.62.3
OS Version: Ubuntu 18.04

Steps to Reproduce:

With the Java extension pack installed, write a unit test with a failing assertion
Click the Open in Editor button to open the comparison

When the comparison editor is in side-by-side mode, it looks OK

But in inline mode, the "Expected" and "Actual" hints are overlapped with the text, making it hard to read

Maybe a general problem with CodeLens rendering when diff is displaying in inline mode?

Maybe a general problem with CodeLens rendering when diff is displaying in inline mode?

Yes - that may actually be the issue!
Using the GitLens code lens, and setting "diffEditor.codeLens": true,, the code lens looks OK in the side-by-side diff editor

But not in inline mode

Interestingly, the issue seems to only occur if the code lens is immediately above a removed line.  If it is above or within an added line, the code lens displays OK in inline mode


Online casino game interface

Online casino game interface: what matters to you?

If you're looking for an online casino with an excellent user experience, I recommend giving Ricky Casino a try, download ricky casino app for easy access to a variety of gambling games, including slots, table games, and live casinos, all from your mobile device. The app is available for both Android and iOS, providing smooth gameplay with fast loading times, so you can enjoy your favorite games anytime, anywhere, without interruption.
When choosing a platform, ease of use is essential. Ricky Casino’s interface is designed for simplicity, making it easy to navigate through the different games and features. Its high level of security and quick transactions also ensure that your gaming experience is safe and convenient. Whether you're a seasoned player or new to online casinos, Ricky Casino offers a solid platform to explore.


[BUG] Step from guide says model deployed but failed

What is the bug?
In the sphinx docs it says the model deployed successfully but the output says it failed.
https://github.com/opensearch-project/opensearch-py-ml/blob/1237aa61bb40cf6cef06002cffa02093ef4b2023/docs/source/examples/demo_ml_commons_integration.ipynb#L318C1-L318C414)
How can one reproduce the bug?
Steps to reproduce the behavior:

Go to 'https://opensearch-project.github.io/opensearch-py-ml/examples/demo_ml_commons_integration.html#Step-2:-Load-Model
View the result. It says it deployed but the output says it failed.

Thanks for the catch. Fixed in this PR.
Updated doc: https://opensearch-project.github.io/opensearch-py-ml/examples/demo_ml_commons_integration.html


Factor out drawing context from platform window implementation

@x42 There, find something better to complain about :P

Thank you!


Ip address can't be above 255

###My Result:
`Got Some IPv4 addresses:
2.343.543.543
1.031.844.254
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
2.343.516.516
<IP_ADDRESS>
394.394.909.59
<IP_ADDRESS>
<IP_ADDRESS>
3.996.89.89
382.167.814.178
<IP_ADDRESS>
2.343.544.544
<IP_ADDRESS>
`

maximum possible ip is 255 only can't be 343,543

fixed.
Please check and let me know.


Should you update Splat to 2.0.0 ?

Unable to resolve dependencies. 'Splat 2.0.0' is not compatible with 'reactiveui-core 7.3.0 constraint: Splat (>= 1.4.0 && < 2.0.0)

Since Splat 2.0 is on netstandard (1.1), this could be considered for v8 of RxUI.

This is actually already covered under #1332, which is for v8, that  PR is waiting on some more reviewing though.

2.0 is coming with #1332


iOS no notification

Hello, thanks for this awesame script
i have two iphone one iPhone 14 pro with iOS 17 all notifications enabled and i have notifications
but in the second iphone XS with iOS 17 all notifcations enabled and i dont have any notifications.
thanks for the work

The notification should work on either version of you iPhone. Can you double check the profile screen and make sure notifications are swithed on? If it is on maybe switch it off/on to see if there is some sort of glitch there?
In case it wasn't clear, notifications are only sent as a result of a connected contact's actions. Your contact would either need to start a topic, post a message, etc...
Please let me know if the problem persists or is resolved, and thank you for using the project!

Thanks for the fast reply,
iphone X restarted and switched on/off the notification  in phone and in user account and in topic setting.
started a new topic in iphone x with the user 2 host, added user 1 in topic (iphone 14 pro) the user 1 is notified
and when the user 1 write a message in the topic the user 2 is not notified.

I just tested on an iPhone X, and the notifications appear for me. I'll try and get my hands on an iPhone XS, but I don't anticipate there being a difference.
A couple more questions:

Is the app closed when you expect to receive the notification? If you app is running it will update the topic and not use the iOS notification.
Do you have any custom configuration on your phone? Someone using tor said they had issues until they disabled it. I'm not sure what the issue is yet, but I would like to be compatible.
Are the accounts for user 1 & user 2 hosted on the same node? Perhaps there is some server configuration difference blocking the outbound notifications?

Hi Balzac,
I appreciate your time and effort on this project. Unfortunately, I have the same problem as described above.
Notifications are reaching my Android phone but not my iPhone.
I have literally tried everything but no luck so far.

I've been trying to reproduce the issue, but I still cannot. Are you also using an iPhone XS?
If you create an account on the demo site [https://databag.coredb.org] can you receive notification from that server?
If you are able to log into your server and run a sql command:

sqlite3 /var/lib/databag/databag.db
sqlite> select id, push_type, push_enabled, token, platform from sessions;
Can you confirm the push_type for your iphone record says 'fcm' and 'push_enabled' has a value of 1?

Thanks for reporting this, hopefully with the more data points I will be able to reproduce. Once I do that, I can usually resolve the issues quickly.

I am finally able to reproduce the issue. If I login after the first install then no notifications appear, but if I logout and relogin it works. I wasn't able to reproduce this bug, because it does not show up on testflight, which is were I do most of my testing. I hope to push out a fix within the next week or two, sorry.

If you get a chance, can you update your iOs app and see if the issue is fixed? You will need to logout and log back in to register a new push token. Sorry it took so long to (hopefully) fix the issue.


Provide cluster roles to manage CRs

What happens?
It doesn't seem to be sufficient to be an admin of a namespace (at least on OpenShift 3.11) to view the GitOpsConfig CRs.
What were you expecting to happen?
We should provide roles that will allow viewing and managing CRs. Viewing might be more important, because any of the CRs should be managed via GitOps anyway.

Aggregate to admin, edit, and view? How should the roles get applied? Should that be automated? Should that be different if the operator is installed namespaced?

I'm thinking about keeping it simple right now and providing an admin and view role. I created 2 basic ones and will submit them soon with my helm installer PR.

Meant for human use or by the operator?

because any of the CRs should be managed via GitOps anyway.

Do you intend for a given CR the operator is operating on to manage itself? Or is that intended to be managed by an upstream CR? Former there's a bootstrapping problem, latter you've got an location/ownership issue.

We already have a custom role (both cluster and namespace) for the operator itself. So this one is just intended for people access.
It could be owned by an upstream CR, but that adds a whole new set of issues. We need to first get some runtime on it to determine what the best ways of managing it will be. We might have to split this out into two different charts in the future, one that requires cluster-admin and one that doesn't.
I want to keep this simple for now, until we learn more.

@sanbornick I thought of this and think we should separate it right now. It'll be a much more realworld-ish approach. Stay tuned for an updated PR...

@sanbornick I added the role aggregation. It'll be part of the larger PR

Friendly reminder to remove this block when this issue is figured out.
https://github.com/KohlsTechnology/eunomia/pull/67/files#diff-2fce78a7f7833e3b8b8a8087cc964131R69-R83


Header Improvements

Pull Request: Header Improvements
Changes Implemented:

Persistent Header:

The header is now fixed and remains at the top of the page while scrolling.
This ensures easy access to navigation and enhances user experience.

Remove Profile Picture:

The user's profile picture has been removed from the UI.
The interface now focuses more on streamlined functionality.

Dynamic Login/Logout Button:

When the user is logged in, the UI displays a "Logout" button.
When the user is logged out, a "Login" button is shown instead.
This improves user experience by showing relevant actions based on the current login state.

Testing:

Verified header remains on top during scrolling.
Checked that the profile picture is no longer displayed in the UI.
Tested login/logout functionality to ensure the correct button is shown based on the user's status.

Notes:

Ensure to clear cache if UI changes are not reflected immediately.

I tested login/logout functionality . It the correct


emptyRule linter breaks for rules that only contain a mixin or only @media queries

Code such as this
.responsive-img( @imgName ) {
    .set-img( 'normal/@{imgName}' );
}

or this
.responsive-img( @imgName ) {
    @media screen and ( max-width: 480px ){
        .set-img( 'mobile-lowres/@{imgName}' );
    }
}

produces the "There shouldn't be any empty rules present" linting error.

Should be fixed in 0.3.1, let me know if it still persists.


feat(csi-liveness-probe): update chart and YAML

https://github.com/longhorn/longhorn/issues/3907

Depending on https://github.com/longhorn/longhorn/issues/4945.
cc @innobead


fix: skip recipientId in to_bytes for tx type 1 and 4

Proposed changes

In second signature and multi signature transactions the recipient id is not supposed to be included in the signature/id calculation. Add an additional check to to_bytes to explicitely exclude the recipientId. This also allows such transactions to be verified, even if the recipientId is set.
Types of changes

[X] Bugfix (non-breaking change which fixes an issue)

Checklist

[X] Lint and unit tests pass locally with my changes
[X] I have added tests that prove my fix is effective or that my feature works

<!--

Codecov Report

Merging #26 into master will increase coverage by 0.12%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #26      +/-   ##
=========================================
+ Coverage   78.97%   79.1%   +0.12%
=========================================
  Files          35      35
  Lines         490     493       +3
=========================================
+ Hits          387     390       +3
  Misses        103     103

Impacted Files
Coverage Δ

...b/arkecosystem/crypto/transactions/deserializer.ex
87.23% <ø> (ø)
:arrow_up:

...ib/arkecosystem/crypto/transactions/transaction.ex
90.27% <100%> (+0.42%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c707471...7401422. Read the comment docs.


Host and user Commands

Feature Overview & Customer Need
As a  Conjur user
I would like to use the conjur cli
In order to update my password
As a  Conjur user or host
I would like to use the conjur cli
In order to rotate API Key of a user or a host (it can also be me :))
COMMANDS
rotate-api-key   Rotate a user's or host's API key. If no user/host ID is provided the logged in resource's API Key is rotated.
change-password    Update the password of the logged-in user either the command is interactive meaning password is requested from the user or it is provided by -p /--password.
Help
For each command and subcommand a help is needed see here:
Help for host: https://ljfz3b.axshare.com/#id=paicg6&p=conjur_help__user&g=1
Help for user: https://ljfz3b.axshare.com/#id=paicg6&p=conjur_help__user&g=1
Quality
Make sure we have test coverage of the host commands.  Create test plan and execute accordingly.
Process logic and Demo
Conjur use with a machine that the Python CLI already been installed and conjur init & login ran
Now the user run the following commands:
conjur user  rotate-api-key  <user_id>
show the result of rotate-api-key (3v087323qv....ryq38201w9wvx)
conjur user  change-password  -p new_pass
show response
conjur host rotate-api-key  <host_id>
show the result of rotate-api-key
User messages
All user messages regarding host and user should be reviewed again
Especially error messages
if an argument is missing we need to return the help of the command
UX Guidelines
Follow guidelines here: https://github.com/cyberark/conjur-api-python3/blob/master/guidelines/python-cli-ux-guidelines.md
Documentation
Please provide enhance documentation in online help and readme
DOD

[ ] Implement the user command & subcommands Conjur Python CLI in all 3 platforms
[ ] Implement the host command & subcommands Conjur Python CLI in all 3 platforms
[ ] Demo the flow described above
[ ] Automatic integration tests written according to a test plan and passed successfully
[ ] UT written for all classes\functions\major logic flows and passed successfully
[ ] Security action items were taken
[ ] Supportability was taken into account - troubleshooting section in the readme enhanced and reviewed by TW & PO
[ ] Fill in the user messages - link to a page with all user messages for CLI saved and reviewed.
[ ] Fill in the logs and place here a link to the logs files - link to a page with all logs for CLI saved and reviewed.
[ ] Logs were reviewed by TW and PO
[ ] Are there new Audits to this feature? If so please document them and provide here a link
[ ] Configurations were reviewed by PO - link if something changed
[ ] Documentation HO to TW and review docs

@InbalZilberman @eladkug just a note that the secrets need to meet the following Conjur criteria

Choose a password that includes: 12-128 characters, 2 uppercase letters, 2 lowercase letters, 1 digit and 1 special character"

@sharonr78 for conjur user change-password -p new_pass I imagine we will need some sort of Success! Password has been changed response b/c currently in Ruby CLI, there is no such message. Just the following experience:
conjur user update_password sigal@usrsig
Enter the password (it will not be echoed):
Confirm the password:

@sharonr78 for conjur user change-password -p new_pass I imagine we will need some sort of Success! Password has been changed response b/c currently in Ruby CLI, there is no such message. Just the following experience:
conjur user update_password sigal@usrsig
Enter the password (it will not be echoed):
Confirm the password:

I totally agree. We should aim to provide such feedback messages whenever possible.
Let's use this message (according to our microcopy's voice and tone): Success! Password was changed

@InbalZilberman Following the password change feedback message, we should also have it for key rotation, as it's not enough to show the new key. I suggest to show the output message, like so:
conjur user rotate-api-key
Success! API key was rotated. New key: 198qmk51cgsa061ghh80x2s56ct99grch1j066xcgnke5431rj3v
conjur host rotate-api-key <host_id>
Success! API key was rotated. New key: 3k7k0vb2sf2emjjcphtx14jfvrbbfn8vm3t79mnz19tk2252dsz9wf

If we are really going for a consistent experience in the CLI, we need to stick to the same pattern for all commands:
conjur <action> <option1> <option2> ...

I would also discourage use of optional arguments when the parameter is not optional  (e.g. conjur user change-password -p new_pass). Password here is not optional and as such doesn't need a flag and should be a positional arg.
If we do these two things, the more appropriate forms of these commands would look a bit different:
conjur rotate-api-key                             # Rotates own key
conjur rotate-api-key host <host_id>             # Rotates a host's key
conjur rotate-api-key user <host_id>              # Rotates a user's key
conjur update-password                            # Updates own password
conjur update-password user <username> <password> # Update user's password

@sgnn7 I really like this approach and would like to omit the options whenever possible, but there are commands which require options, like list or init, so the pattern is not always consistent.
Can we use the following general usage? conjur [global options] command [subcommand] [options...] [arguments...]
Here are a few possible use-cases:
conjur whoami  -  single command only
conjur --version - single global option only
conjur user rotate-api-key joe - command + subcommand + arg
conjur list --kind user --limit 4 - command + 2 options + 2 args (no subcommand for 'list')
conjur variable get secrets/mysecret - command + subcommand + arg
Actually, we can say that whenever we have a subcommand we can omit the use of options.
WDYT?

I like this approach. In my mind this idea of options (--) is for consolidating information and screening out the information you don't need. I think the best example of this would be with list where we have the option to add restrict or throw out the information we don't need.
For example with list --limit 4, we are screening out all the noise and just want 4 entries to be returned
In short, I think options should be reserved for screening out info and not providing info (like secret for example)

@sgnn7 I do like
conjur rotate-api-key                             # Rotates own key
But i think we can change
conjur rotate-api-key host <host_id>
into one simple
conjur rotate-api-key <resource_id>
WDYT?
Yet for update-password this option makes a lot of sense
conjur update-password                            # Updates own password
but this one
conjur update-password user <username> <password> # Update user's password
not so much :(
The sentence is broken I do think
conjur  update-password -u <user> -p <password> # Update user's password
is clearer. WDYT?

@InbalZilberman The problem with conjur update-password -u <user> -p <password> is that it implies that both values are optional and unconnected to each other to where something like this might be allowed: conjur update-password -u <user>. By using positional arguments (conjur update-password alice "alicepassword"), you ensure that when a user is specified, you must also have a password associated with it.

@InbalZilberman Some links on this:

https://git-scm.com/docs/git-remote - You can see how git here uses the mandatory params as positional args
A stack overflow question I found on this has a good conversation about it: https://softwareengineering.stackexchange.com/questions/366218/positional-arguments-vs-options-in-a-command-line-interface

into one simple
conjur rotate-api-key <resource_id>
WDYT?

I like this idea especially since our REST looks like: /authn/{account}/api_key?role={kind}:{identifier} so we don't absolutely need the user or the host, rather on the full name
If we choose this then we will need to be consistent also on update-password so it will look like:
conjur update-password <resourceid>
Which is much cleaner IMO

conjur update-password user

@InbalZilberman where do you see the ability to update-password for other users? In our REST API for changing passwords I don't see the option of updating any user's but the current logged-in one. See here
From my understanding you can only rotate_api_key on yourself and on someone else but changing the password can only be yours and not someone else's

Changed conjur user change-password to not include -p

@InbalZilberman please see my comments below:

Link to host command help: https://ljfz3b.axshare.com/#id=wsk7ud&p=conjur_help__host&g=1
Link to user command help: https://ljfz3b.axshare.com/#id=mzfbsw&p=conjur_help__user&g=1
I really think we need to go with positional parameters for both host and user, rather than flags, so the commands will look like this:

conjur user rotate-api-key (rotates the API key for the logged-in user)
conjur user rotate-api-key admin (rotates the API key for the admin user)
conjur user change-password Myp@ssw0rd! (changes the password for the logged-in user to Myp@ssw0rd!)
conjur host rotate-api-key <host_id> (rotates the API key for <host_id>)
An example of success messages would be:
conjur user rotate-api-key
Success! API key was rotated. New key: 198qmk51cgsa061ghh80x2s56ct99grch1j066xcgnke5431rj3v
conjur user change-password Myp@ssw0rd!
Success! password was changed.

@InbalZilberman please update the link to the host help command to this: https://ljfz3b.axshare.com/#id=wsk7ud&p=conjur_help__host&g=1


fails to connect on alterate ssh port

My remote host uses an alternate port for ssh. According to the docs, I should be able to supply supplemental configuration environment variable (SSH_ARGS) to specify that port, but it doesn't work, as in:
SSH_ARGS="-p 2222"
In the function "available_devicepair()", there is a call to the ssh program that does NOT supply the ${SSH_ARGS} variable to the ssh program. This causes the connection to hang up, time out and fail when an alternate ssh port is used.

iptables -t nat -I OUTPUT -s sip -d dip -p tcp -m tcp --dport 22 -j DNAT --to-destination dip:2222


Openstack: support for self signed certificates for api endpoint

Having the option to disable ssl verification for the openstack api endpoint would be perfect, currently I have to mutate the machine-controller and machine-controller-webhook deployments with certificate mounts from the OS.

@PaulRitzkat0110 since #895 we have support for custom root certificates. Please point -ca-bundle flag to your own PEM encoded root CA bundle.


Chore: Make Formatting to look like in any Rust project

Changes proposed in this pull request:

fmt changes

Related Issues
Closing #

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 83.13253% with 14 lines in your changes missing coverage. Please review.

Project coverage is 83.13%. Comparing base (79ae9ab) to head (ddcf37d).

Files
Patch %
Lines

src/visitor/visitor.rs
60.00%
8 Missing :warning:

src/ast/common/ast_term.rs
82.35%
3 Missing :warning:

src/ast/cdl/module_term.rs
66.66%
2 Missing :warning:

src/ast/cdl/service_term.rs
85.71%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #28   +/-   ##
=======================================
  Coverage   83.13%   83.13%
=======================================
  Files           8        8
  Lines          83       83
=======================================
  Hits           69       69
  Misses         14       14

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


Const Records: Relax shape for Particles

Description
For MPI-parallel I/O output, we developed a new method in ADIOS2 that does not need an initial metadata gather ("JoinedArrays"). To be able to use this mode, we need to relax the requirements to write a shape for constant records in a species (particle group), because otherwise we still have to do a collective gather.
This adds the need for a slight additional read fallback implementation on the reader side.
Affected Components

base

Logic Changes
The required attribute shape in constant record components is now optional for records in particle groups (species), if there is at least another record to recover the shape from in the same particle species.
Writer Changes
The required attribute shape in constant record components is now optional for records in particle groups (species).
Reader Changes
The required attribute shape in constant record components is now optional for records in particle groups (species).
If the attribute is missing, go through other records and components of the same species and pick the first one that has a shape (e.g., non-constant record component full extent or a constant record component with a shape) and use that information to recover.
What would a reader need to change? Link implementation examples!

[ ]  openPMD-api: https://github.com/openPMD/openPMD-api/issues/1663 https://github.com/openPMD/openPMD-api/pull/1661 @franzpoeschel

[x]  yt: https://github.com/yt-project/yt/... - will be based on openPMD-api soon
[x]  ParaView: based on openPMD-api
[x]  openPMD-viewer: base on openPMD-api for ADIOS2

[ ] openPMD-validator: https://github.com/openPMD/openPMD-validator/... @ax3l
[ ]  VisIt: https://github.com/openPMD/openPMD-visit-plugin/...

Data Converter
No changes needed. Files from 1.X will be forward compatible with regards to this change.

This has practical consequences for parsing a corner case of openPMD data, namely constant scalar particle records.
Take for example this constant scalar particle record:
  int32_t   /data/500/particles/e_all/mass/macroWeighted                        attr   = 0
  uint64_t  /data/500/particles/e_all/mass/shape                                attr   = {1829775}
  double    /data/500/particles/e_all/mass/timeOffset                           attr   = 0
  double    /data/500/particles/e_all/mass/unitDimension                        attr   = {0, 1, 0, 0, 0, 0, 0}
  double    /data/500/particles/e_all/mass/unitSI                               attr   = 8.26366e-28
  double    /data/500/particles/e_all/mass/value                                attr   = 0.00110234
  double    /data/500/particles/e_all/mass/weightingPower                       attr   = 1

How does a parser distinguish /data/500/particles/e_all/mass from e.g. /data/500/particles/e_all/momentum:
  float     /data/500/particles/e_all/momentum/x                          {1829775} = 0 / 0
  float     /data/500/particles/e_all/momentum/y                          {1829775} = 0 / 0
  float     /data/500/particles/e_all/momentum/z                          {1829775} = 0 / 0

Currently, the openPMD-api implements this corner case in ParticleSpecies.cpp by:
            auto value = std::find(att_begin, att_end, "value");
            auto shape = std::find(att_begin, att_end, "shape");
            if (value != att_end && shape != att_end)
            {
                RecordComponent &rc = r;
                IOHandler()->enqueue(IOTask(&rc, pOpen));
                IOHandler()->flush(internal::defaultFlushParams);
                rc.get().m_isConstant = true;
            }
            try
            {
                r.read();
            }

I.e., when parsing a group /data/500/particles/e_all/MYSTERY, it checks if the group contains the attributes value and shape. If yes, it's a constant scalar component, otherwise it's a normal record.
With this standard change, I see two options:

We only check for the attribute value. MIght lead to problems with datasets that for some reason use an attribute named value.
Follow-up question: For legacy (1.0) datasets, do we keep checking for both attributes? If yes, the relaxed standard definition cannot be applied to old data. If no, some old datasets might be broken.
We check if the record contains subgroups or datasets and treat it as a constant component if not. Might lead to problems with partially written data, e.g. after a crash.

I suggest that we decide for one clearly defined scheme to detect constant scalar components and standardize that. 1. is easier to implement.
Also, do I understand it correctly that the relaxed constant component markup is not applicable to Mesh records?


[NVIDIA TF] Enable BF16 TopK

Adds GPU bfloat16 support for TopK.

Hi @rainwoodman Can you please review this PR ? Thank you!

@trevor-m Can you please check @cantonios's comments and keep us posted ? Thank you!

Seems auto-merge is not happening but the changes are merged into master now, so we can close this. Thank you for the PR.


Fix lint error

Somehow a lint error slipped through the cracks; this PR fixes it

I tried merging this branch with youtube-api. It fixed the lint error but can't run the work I've been doing. For example, api/oauth/login/youtube doesn't work. :(


implemented display_prompt action

It can be useful to have the prompt displayed in the floating window as well, e.g. for the Raw input, but also can give a sense of the exact prompt that is used to query the LLM. To this end I'v added a display_prompt action that is a bit more involved than making a custom action - it needs the parsed_prompt as input, and the show_spinner needs to be modified as well.
Here it is in action:

Here's how it looks for code:

For greater flexibility, would it be better if the util.show_spinner method allowed for full customization of all text around the spinner itself? Accepting a table with pre and post keys.
E.g. to achieve current functionality (prior to this PR) you would do something like:
local timer = require("ollama.util").show_spinner(bufnr, { pre = "Generating... " })

Then for an action to display the prompt as its created, you could do something like:
show_spinner(bufnr, { pre = parsed_prompt .. "\nGenerating... " })

post key would be an optional string just like pre in case, for instance, someone wanted an action that displayed the spinner during the entire generation task - instead of just before the first streamed token.

That makes sense - it would be cool to know when the generation process has ended. I was also thinking that after it's done the > Generating ... could turn into > model_name so it visibly splits the prompt from the output. But then we're going towards chat :slightly_smiling_face:
btw I put the > in Generating so it's formatted nicely - it's more visible at least in catpuccin.
Feel free to shape it as you will - I implemented it because I needed  it to be able to copy and save/edit my prompts :wink:

In what other context do you need to know when generation has ended? The ollama API sends a done property in its final response - you can see it being used in other actions.

Thinking on it, it might be better if show_spinner is (optionally) given a line number to display the spinner on, instead of rewriting the entire buffer on each loop. I'll fuss around with it and see what I come up with.

Looks good :love_you_gesture: - maybe have the model name when it's done (even generating?), like: > zephyr in X s

First the important thing - displaying the prompt is a necessary feature that is currently lacking.

It allows users to see the prompt, which is great especially for complex prompts involving selected text/code.
It allows users to copy the interaction and keep it for reference - that's why it's important to have > modelname leading the response.

On increasing the complexity of the code - the actions can really be optimized into one function that displays if needed and includes the prompt in the display if chosen. Because I would have _prompt work with display_replace too.
About the roadmap. To tell you the truth I'm leaning even more away from specialized prompts. Currently I have 18 prompts - 12 for text which I forward to a general language model and 6 for code which I forward to codellama. And at some point it get's hard to remember what exact prompt each of them spawns - any input you have to enter is in a blank input field.
As a user I wouldn't mind having to type a bit of extra words to use less prompts - so in other words fewer generic prompts that you can use for anything you like, and they should be built around the $sel, $ftype and $buf - and one set for text another for code:

just user input
user input + selection
user input + selection + buffer for context

Another thing that's currently a problem is that there is no way to iterate on the response, i.e. you can't modify the prompt if you don't like the output. Without the display_prompt you don't even get to see/copy the prompt you've typed :smile:
And keeping the model name there before the response would make it easy to copy paste the prompt and response for reference. Plus visually it separates the two nicely - if you have a large prompt that includes text/code.
The optimal workflow would be:

to have a pop-up or a split window, or even better a dedicated LLM tab (based on opts)
be able to input/modify the prompt directly in the pop-up (so it should be modifiable) - in these terms the hard coded prompts would be like prompt snippets where the user can <Tab> between different parts and be able to modify them
on a key stroke - the finished prompt is sent to the LLM and there is a > model name leading the response
after this the user can re-edit the prompt in the same window and re-send the modified version to the LLM (with the same key stroke) - this something that can be easily done in the web UIs.

One step ahead of this would be a full on chat where the user can follow a response with another prompt in the same popup window and the previous answers get transferred as context (if needed - currently no context is kept between calls to the api no?)
I ramble. Hope it helps :+1:

Good call on reducing the actions into a single configurable base action. As a side effect of that, creating a sort of action 'factory' to achieve this might make it easier for users to tune things with less effort/custom code required.
I also really appreciate you sharing your experiences and workflows. I'm not yet sure how I would implement certain features of your 'optimal workflow' as I'm still fairly new to the neovim APIs, but I will keep it in mind as I continue to R&D.
On the more generic prompts - this was actually my intent with the default prompts: have a small handful of generalized prompts for most tasks, and let users extend the list with their own specialized ones. I admit there's a lot to be improved there. Feel free to open an issue or PR if you have more ideas on that.
In the meantime, I'll update & finally merge this PR with the model name leading the response as you suggested, and then I'll work on refactoring the actions to be less verbose in code.

Awesome :love_you_gesture: keep up the good work :rocket:


(Bug) local deployment in Windows doesn't work

Local deployment in Windows should be double checked. It doesn't work by following these steps https://github.com/poanetwork/token-wizard/wiki/Token-Wizard-Local-Dev-Version#prerequisites

I got it to build in Ubuntu server 14.x.x with "npm run build", after:
$   git clone https://github.com/poanetwork/token-wizard.git wiz
$   cd wiz
$   git submodule update --init --recursive --remote
$   npm install
$   cp .env.example .envgit clone https://github.com/poanetwork/token-wizard.git wiz
cd wiz
git submodule update --init --recursive --remote
npm install
cp .env.example .env
I then moved the build directory in a Win 7 partition and, in Git Bash on Win 7, and ran the following:
$   npm install -g serve
C:\Users\turtle\AppData\Roaming\npm\serve -> C:\Users\turtle\AppData\Roaming\npm\node_modules\serve\bin\serve.js

<EMAIL_ADDRESS>added 130 packages in 8.082s

$   serve -s build
serve: Running on port 5000
I am now able to change the css, js and html parameters before launching the ICO on the MainNet with the landing page I want to have and all the MetaMask stuff sorted out. Thanks POA Network, you are doing a great job for the community, and the future of humanity.
Mickael LEVY
CEO @ K SYSTEMS LTD
Website: https://ksystems.io
Telegram Channel: https://t.me/ksys1
Twitter: @ksystemsio @mickaelksys
Gitter: https://gitter.im/ksystemsio/Lobby
ERC-20 Token: KSYS


feat: making tasks cancellable

What does this PR do?
This PR is the continuity of https://github.com/containers/podman-desktop-extension-ai-lab/pull/725. It adds the cancellationToken to the Task object.
This allow the frontend to call studioClient.requestCancelToken on user action.
Screenshot / video of UI
https://github.com/containers/podman-desktop-extension-ai-lab/assets/42176370/0b85edea-946e-4b6c-8eed-af2ba7232861
What issues does this PR fix or reference?
Fixes https://github.com/containers/podman-desktop-extension-ai-lab/issues/520
How to test this PR?

[x] unit tests has been provided

If i download a model, cancel it and try to download it again it does nothing

Nice catch ! The issue was a bug apparently in the existing code, a downloader has a completed properties which is true when success or failure has been reached. I fixed it by removing the downloader when we receive a completion event.
https://github.com/containers/podman-desktop-extension-ai-lab/assets/42176370/bafd603b-25b1-46d6-ac7c-667a165aa550

Thanks!! Tested and works fine 🚀

Thanks to you for reviewing and testing !

This partially cover https://github.com/containers/podman-desktop-extension-ai-lab/issues/467


Use JavaScript to make return button in img-big pages go to precedent page

The return button in img-big pages (button-return) currently makes the user go to a specific page in HTML (and the link can be wrong if I messed up with my Ctrl + C / Ctrl + V). It has to be something like that:
onclick="window.history.go(-1); return false;"

In the HTML.

The window.history.go(-1) method worked. To make button styling work, the :link after .button-return was removed.


Wrong error message for null in array + possible memory leak

Minimal test case:
CREATE TABLE arraytest
(
    created_date Date DEFAULT toDate(created_at),
    created_at DateTime DEFAULT now(),
    strings Array(String) DEFAULT emptyArrayString()
)
ENGINE = MergeTree(created_date, cityHash64(created_at), (created_date, cityHash64(created_at)), 8192)

insert into arraytest (created_at, strings) values (now(), ['aaaaa', 'bbbbb', null]);

Instead of some error about wrong type in array got:
Exception on client:
Code: 173. DB::Exception: Allocator: Cannot mmap 8.00 EiB., errno: 12, strerror: Cannot allocate memory

Minimal test case prepared with 1.1.54343 but can be reproduced on recent versions also.

Forgot to mention. If this errors happens for some time(as we had) then you'll get segfaults in threads and after some more time server will silently close. This makes me think that memory leaks are very possible.

Fixed in https://github.com/yandex/ClickHouse/pull/3487.

Awesome! Thanks!


ember-simple-auth-devise doesn't add token header

I really don't know if this is a bug, but it drives me crazy. I'm trying to add ember-simple-auth with ember-simple-auth-devise to my application. All works fine, authentication, login, tokens are written to localStorage, but authorization doesn't work: the token header isn't set.
I'm using Ember 1.8.1, Ember Data 1.0.0-beta.12, Handlebars 1.3.0, jQuery 1.11.1, Ember Simple Auth 0.7.2 and Ember Simple Auth Devise 0.7.2 for my frontend. My backend is backed with Rails, Grape and Devise.
Both ember-simple-auth and ember-simple-auth-devise are added to my package.json/bower.json and installed (otherwise authentication/login wouldn't work :smile_cat:) and the authorizer is set in my environment.js file:
ENV['simple-auth'] = {
  authorizer: 'simple-auth-authorizer:devise'
};

I just played around with some console.log debugging. It seems that the authorize function don't get fired. All other functions could be logged successfully, except authorize and I can't figure out why.
I opened a question on stackoverflow too, if this isn't a bug and just a mistake of mine.

I have had this exactly same problem. Although I configured crossOriginWhitelist and it still does not work. I have tried everything I could but not able to make this work.
Using Ember 1.13, Ember Data 1.13, ember-cli 1.13, ember-cli-simple-auth 0.8, ember-cli-simple-auth-devise 0.8 and bundled versions of ActiveModelAdapter (now separate from ember-data) 1.13

@henriquecf I had the same problem, in my case, the issue was in my SessionsController. Make sure your response after creating a session looks like this.
def create
  super do |user|
    data = {
      token:      user.authentication_token,
      email: user.email
    }
    render json: data, status: 201 and return
  end
end

The issue was that I was returning user_email instead of email.

I have even decided to use Oauth after all the problems I have had, but I will try this @holamendi thanks!!


Hindi letter classification Project

Hi @TAHIR0110 @Atharv714  ,
I hope you're doing well. I've addressed the issue #182 and created a pull request to resolve it.
Could you please review the PR and let me know if any adjustments are needed?
Best regards,
Tanisha Garg

@TAHIR0110 @Atharv714 Please review this PR asap.


VIDEO-7997 Implement support for Adaptive Simulcast. Consume 7.4.0

Description
This PR implements support for Adaptive Simulcast on the Ahoy App by consuming Android SDK 7.4.0.
Contributing to Twilio

All third-party contributors acknowledge that any contributions they provide will be made under the same open-source license that the open-source project is provided under.

[x] I acknowledge that all my contributions will be made under the project's license.

All tests green. https://app.circleci.com/pipelines/github/twilio/twilio-video-app-android/1783/workflows/de31dec4-c212-428a-b008-6857aea5e136
Merging.


Would u add other sky objects like a jupiter or ringed planet to show up once in a while?

Sure its possible, you could create some kind of variant on the star that is larger or has some other features.
I currently have no plans to work further on this project though. Feel free to fork it and try


Infinite scrolling?

Do you have any plans to include infinite scrolling for fixed height tables?

+1

Hello,
Just send a pull request for this implementation.
https://github.com/esvit/ng-table/pull/560


Add edit button in contact diary where user selects entries

Avoid duplicates

[x] This issue has not already been raised before
[x] If you are proposing a new feature, please do so in CWA-Wishlist

Current Implementation
Currently the only way to edit the list of locations and people is in the contact diary's main view via the context menu in the upper right corner.
Suggested Enhancement

Basically all list views in iOS feature an edit button in the top corner so the user might expect one in this view as well, but then needs to go back to the parent view to change or delete entries. Therefore I would suggest adding this button for calling the edit view directly.

Expected Benefits

The user can directly edit all the entries in the same view where he also chooses them, making editing more quickly and bringing it on par with basically all list views in iOS.

Internal Tracking ID: EXPOSUREAPP-3034
Internal Tracking ID: EXPOSUREAPP-5495

Related Issue: https://github.com/corona-warn-app/cwa-wishlist/issues/334

@stefan-schoene Thanks for you suggestion.

Internal Tracking ID: EXPOSUREAPP-3034

We're on it together with @m-seidel

@cornelius-behrend @m-seidel You would like to provide a PR? If so, I will cross-check with developers.
Thanks. Best wishes, DS

Corona-Warn-App Open Source Team

@cornelius-behrend
AFAIK you can just start working (I see you already did).
The contributing document is outdated reg. this, contributors aren't assigned to issues anymore (or @dsarkar?)

@dsarkar yes, we would like to add a PR for that if the issue was not already tackled. We would appreciate if you could check it :)

@cornelius-behrend @m-seidel OK, I will check if they would be interested in such a PR. However, they might not merge it, at the end, and this can have different reasons, e.g. quality of the code, rejected by the stakeholders, or other reasons. The are PRs from the community that have been merged, but not all of them if there was the need of major changes or quality issues or for other reasons. Stand by, I will com back to you. Thanks.

Corona-Warn-App Open Source Team

@dsarkar thank you! We're doing this as part of the Staffbase hackathon. So either way we learned something or we have contributed to the project which both is fine :)

@cornelius-behrend @m-seidel
nice! Did you implement that for Android, or iOS, or even both?

I'm so free and post the link to the branch of @m-seidel & @cornelius-behrend here: https://github.com/m-seidel/cwa-app-ios/tree/feature/2196-add-edit-button-in-contact-diary
Also a huge thanks from my side! Hope that this makes it into the app!

@Ein-Tim Thanks for posting the repo link! :) We're currently focussing on the iOS since we only have 2 days @ndegendogo

@ndegendogo @Ein-Tim @cornelius-behrend @m-seidel
Typically the stakeholders insist on feature parity for both OS. However, we will see, if there might be an exception, or if this PR triggers that the feature will be developed also in Android soon.

Some comments from my side:

A (new) user could think that the edit button is to remove the person/place for the day, and not from the whole contact diary.
Thus, I would suggest that the warning which now pops up when you delete a person or a place should definitely also be there. This is something that should be implemented from the beginning on.

We'll see more similar list views in the upcoming CWA version 2.0. The "My Check-Ins" screen (reachable by tapping on "Check In" on the Tab bar) & the "My QR Codes" screen (reachable by tapping on "Create QR Code" on the start screen).
These views should also have this edit button. This is something that can also be implemented later.

The story is currently described internally the following way:
As a user I would like to use commonly used ways of editing a person or place entry directly.

tap and hold on an entry to open a selection to edit or delete
swipe to delete
add a pencil icon to edit
add a bin icon to delete

@dsarkar

[ ] tap and hold on an entry to open a selection to edit or delete
[x] swipe to delete
[ ] add a pencil icon to edit
[x] add a bin icon to delete

I think 1 and 3 can be easily added as well. If you decide to have this as a PR, we will finish the task by adding this as well as fixing the remaining bugs and add tests.

@dsarkar @heinezen
Could you give us feedback if the PR from @cornelius-behrend & @m-seidel would be accepted? It would be sad if this opportunity is not taken to get this create enhancement into the app.

@Ein-Tim see my latest comment in related issue: https://github.com/corona-warn-app/cwa-wishlist/issues/334. Nothing being worked on currently. As mentioned above, amongst others, stakeholders acceptance, quality of code and maintaining feature parity between Android and iOS are factors in such decisions.

@Ein-Tim @cornelius-behrend @m-seidel
Just to be clear, the dev team would review a community PR for this and will likely merge it if it is accepted and passes the CI checks. Contributions are welcome here. The RKI and other stakeholders might require the same feature for iOS, but that shouldn't stop anyone from contributing for Android.

Corona-Warn-App Open Source Team

@cornelius-behrend @m-seidel
Will you follow up with a PR? It would be a pity if these great changes just get lost!


Applications: background

Need to implement mechanism of setting correct background after loading

This is done as a part of another bug. Please test that all apps have correct gradient background.

checked


Support group bot

Would be cool to have a home-grown bot living among us in the support chat on Telegram.
So let's brainstorm about what it could do for us!
Some ideas:

Allow linking to GitHub issues / PRs for the core library.
gh#123 -> https://github.com/akalongman/php-telegram-bot/(issues|pulls)/123
Send automatic notification on new release (with changelog even?)

Would be cool to have this for Dev-Group too?
Dev-Group:

send new issues with link
send new PR with link

Support-Group:

/ask ->  "Don't ask to ask, just ask"
/wiki -> Link to Wiki
/latest -> Latest Version with Link?
/issue -> "Please open an Issue for this: $link"
/allsupporters -> just for admins, tags all Supporters in Group and request help/info
/link "name" -> add preselected custom links like an link to an wiki page with often asked questions

@lichtscheu Great ideas! 👌
Also, taking #438 into account, we could use inline pagination to display the Wiki entries.
/wiki <search-term> could display only pages that are related to what is being searched.
Maybe have a separate /faq that we manage separately/manually?
I guess we could also include /changelog <version-number>.

How about a faq in Wiki? ;)

Yes, that would be the easiest and makes total sense!
We could make sure it's parseable, to keep the option open of implementing an /faq command if need be.

would be cool if there were numbers so you could /faq 13 and then for special we could set some sort auf names so /faq nomysql relates to /faq 13

And to the /link "name" part, these are custom links we can set, something like /link telegrambotapi gives the link to the telegram bot api page, and if someone writes own commands for public access and someone asks for a command like this we could link it with /link surveycommand

Ah right, so basically shortcuts that print links, yes?
As for the FAQ command, I think it would make sense to have a page in the wiki that gets used by the bot, so that updates are automatically added to the command.
Regarding the numbers, it would make sense to keep them the same, even if the wiki page gets reorganised at some point. So they would be 'hard-coded' to the FAQ entry in the wiki page. What do you think?
e.g.
Q5: Question number 5?
A5: Yes, answer number 5!
(hmm, this doesn't look too pretty, would need to find a nicer way of presenting them)

Yeah, some sort of shortcut!
Do we really need to reorganise? ^^
i think i would go with
5. "Question"
"Answer" and use some sort of markdown to make it parsable
i would parse the wiki on daily cron, and save the info as json object in the options table =) so we have the "automatically" add new faq to command.

👌 Perfecto!
Instead of cron, we could also make use of GitHub webhooks, which will let you know when the wiki has been updated.

or this =)

Moving this discussion over to https://github.com/PHP-Telegram-Bot/Support-Bot


Appveyor build (fails on IE 11)

@antonybudianto I wanted to see if your starter project works on IE.
This PR shows an error when running the unit tests on IE
See https://ci.appveyor.com/project/jesperronn/angular2-starter/build/6#L997
I am probably missing an IE specific shim or something. Any suggestions?
PS (please don't merge this – at least not until it works )

Hi, thanks for the PR!
Can you put these references in karma files array?
<!-- 1. Load libraries -->
    <!-- IE required polyfills, in this exact order -->
    <script src="node_modules/es6-shim/es6-shim.min.js"></script>
    <script src="node_modules/systemjs/dist/system-polyfills.js"></script>
    <script src="node_modules/angular2/es6/dev/src/testing/shims_for_IE.js"></script>

If it works, then it just needs a rebase and readme appveyor badge, and I'll merge it :smile:

@antonybudianto will give your suggestions a try. Sure will rebase when ready, but it's nice for myself do document my own learning process about setting this correctly up :+1:
Thanks for input!
Remember, though, that you also have to sign into appveyor on behalf of your project in order to tell them to build it :)

the build succeed!
now all left:
[ ] rebase
[ ] this ref can be removed

@antonybudianto you have to put the badge on the readme yourself. You can find it in appveyor>settings>badges for the project.
The image has a special URL which I cannot guess.

@jesperronn , all great! thanks for the PR! I'll merge it and add the badge later


Use 'https' for the github URL

Since github.com automatically redirects HTTP requests to HTTPS, this eliminates unnecessary redirects.

Thank you!


< is not a valid character within attribute names: (error occurred in 'an unknown module' @ line 14 : column 8)

Not sure what's going on here, but tracking progress regardless,
error log
=================================================================================

ENV Summary:

  TIME: Sat Jan 08 2022 10:58:35 GMT-0500 (Eastern Standard Time)
  TITLE: ember
  ARGV:
  - 🏠/.volta/tools/image/node/16.13.1/bin/node
  - <repo>/node_modules/.bin/ember
  - serve
  - -p
  - 4202
  EXEC_PATH: 🏠/.volta/tools/image/node/16.13.1/bin/node
  TMPDIR: /tmp
  SHELL: /bin/bash
  PATH:
  - /tmp/yarn--1641657510431-0.36332227589673827
  - <repo>/node_modules/.bin
  - 🏠/.config/yarn/link/node_modules/.bin
  - 🏠/.yarn/bin
  - 🏠/.volta/tools/image/node/16.13.1/libexec/lib/node_modules/npm/bin/node-gyp-bin
  - 🏠/.volta/tools/image/node/16.13.1/lib/node_modules/npm/bin/node-gyp-bin
  - 🏠/.volta/tools/image/node/16.13.1/bin/node_modules/npm/bin/node-gyp-bin
  - 🏠/.volta/tools/image/npm/8.3.0/bin
  - 🏠/.volta/tools/image/yarn/1.22.17/bin
  - 🏠/.volta/tools/image/node/16.13.1/bin
  - 🏠/.volta/bin
  - 🏠/.pythons/Python-3.6.3/bin
  - 🏠/.cargo/bin
  - 🏠/Applications
  - 🏠/apps/phantomjs/bin
  - 🏠/scripts/system-utils
  - 🏠/scripts/git
  - 🏠/scripts/rails
  - 🏠/scripts
  - 🏠/.volta/bin
  - 🏠/.local/bin
  - /usr/local/sbin
  - /usr/local/bin
  - /usr/sbin
  - /usr/bin
  - /sbin
  - /bin
  - /usr/games
  - /usr/local/games
  - /snap/bin
  - 🏠/.dotnet/tools
  - 🏠/.dotnet/tools
  - 🏠/.fzf/bin
  PLATFORM: linux x64
  FREEMEM: 956055552
  TOTALMEM:<PHONE_NUMBER>0
  UPTIME: 1214137.1
  LOADAVG: 0.19,0.23,0.2
  CPUS:
  - Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz - 3192
  - Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz - 3192
  - Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz - 3192
  - Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz - 3192
  - Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz - 3192
  - Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz - 3192
  ENDIANNESS: LE
  VERSIONS:
  - ares: 1.18.1
  - brotli: 1.0.9
  - cldr: 39.0
  - icu: 69.1
  - llhttp: 6.0.4
  - modules: 93
  - napi: 8
  - nghttp2: 1.45.1
  - nghttp3: 0.1.0-DEV
  - ngtcp2: 0.1.0-DEV
  - node: 16.13.1
  - openssl: 1.1.1l+quic
  - tz: 2021a
  - unicode: 13.0
  - uv: 1.42.0
  - v8: <IP_ADDRESS>-node.14
  - zlib: 1.2.11

ERROR Summary:

  - broccoliBuilderErrorStack: SyntaxError: < is not a valid character within attribute names: (error occurred in 'an unknown module' @ line 14 : column 8)
    at generateSyntaxError (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:10007:17)
    at TokenizerEventHandlers.reportSyntaxError (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:11690:13)
    at EventedTokenizer.attributeName (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:21177:27)
    at EventedTokenizer.tokenizePart (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:21400:19)
    at TokenizerEventHandlers.ContentStatement (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:11166:22)
    at TokenizerEventHandlers.acceptNode (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:10890:29)
    at TokenizerEventHandlers.Program (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:10969:14)
    at TokenizerEventHandlers.acceptTemplate (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:10886:29)
    at preprocess (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:11808:74)
    at getTemplateLocals (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:13388:15)
  - code: [undefined]
  - codeFrame: < is not a valid character within attribute names: (error occurred in 'an unknown module' @ line 14 : column 8)
  - errorMessage: statechart-actors/components/main-page.gjs: < is not a valid character within attribute names: (error occurred in 'an unknown module' @ line 14 : column 8)
        in /tmp/broccoli-1026938OopdK8hZVxWX/out-289-funnel
        at broccoli-persistent-filter:Mapper
  - errorType: Build Error
  - location:
    - column: [undefined]
    - file: statechart-actors/components/main-page.gjs
    - line: [undefined]
    - treeDir: /tmp/broccoli-1026938OopdK8hZVxWX/out-289-funnel
  - message: statechart-actors/components/main-page.gjs: < is not a valid character within attribute names: (error occurred in 'an unknown module' @ line 14 : column 8)
        in /tmp/broccoli-1026938OopdK8hZVxWX/out-289-funnel
        at broccoli-persistent-filter:Mapper
  - name: Error
  - nodeAnnotation: [undefined]
  - nodeName: broccoli-persistent-filter:Mapper
  - originalErrorMessage: < is not a valid character within attribute names: (error occurred in 'an unknown module' @ line 14 : column 8)
  - stack: SyntaxError: < is not a valid character within attribute names: (error occurred in 'an unknown module' @ line 14 : column 8)
    at generateSyntaxError (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:10007:17)
    at TokenizerEventHandlers.reportSyntaxError (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:11690:13)
    at EventedTokenizer.attributeName (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:21177:27)
    at EventedTokenizer.tokenizePart (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:21400:19)
    at TokenizerEventHandlers.ContentStatement (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:11166:22)
    at TokenizerEventHandlers.acceptNode (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:10890:29)
    at TokenizerEventHandlers.Program (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:10969:14)
    at TokenizerEventHandlers.acceptTemplate (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:10886:29)
    at preprocess (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:11808:74)
    at getTemplateLocals (<repo>/node_modules/ember-source/dist/ember-template-compiler.js:13388:15)

=================================================================================}

based on the file it's pointing at in /tmp, it looks like the transformation didn't happen:
/tmp/broccoli-1026938OopdK8hZVxWX/out-289-funnel/statechart-actors/components/main-page.gjs
import Helper from '@ember/component/helper';
import { hash } from '@ember/helper';

import { List } from './list';
import { Header } from './header';
import { PostLoader } from './posts-loader';
import { DetailLoadingState } from './posts';

// const currentPostId = () => getService('router').currentRoute.params.postId;
// const getPost = (posts, id) => posts.find(post => post.id === id);
// class getService extends Helper {
//   compute([name]) {
//     return getOwner(this).lookup(`service:${name}`);
//   }
// }

const Details = <template>

<EMAIL_ADDRESS>

  {{#if (@state.matches 'loaded')}}

Details are loaded

  {{/if}}
</template>

<template>
  <PostsLoader as |posts|>
    <DetailLoadingState as |state send|>

        <List @posts={{posts.records}} />

          {{#let (currentPostId) as |id|}}
            {{ (send 'LOAD_DETAILS' (hash postId=id)) }}
            <Details @post={{getPost posts id}} @state={{get state.context.posts id}}
          {{/let}}

    </DetailLoadingState>
  </PostsLoader>
</template>

What's goofy is that the error line/column is in a commented out part of the file (that I don't want to leave un-commented, but I commented it out to kinda confirm that I think the stack trace is complaining about a different file than what is reported?)
but,
other components are successfully converted

After commenting out different parts of the file, I've made progress

So it seems something is wrong with my default export template 🤔
(uncommenting this comment block causes the error to return)

ah ok, here is the situation causing the error:
const Details = <template>
  content doesn't matter
</template>

<template>

</template>

the presence of a trailing semilocon after the first closing </template> doesn't seem to matter (error is the same)

oh no.
I found the original problem:
             <Details @post={{getPost posts id}} @state={{get state.context.posts id}}

unclosed tag here
but this was way on line 39.
anyway to fix the line/column number reporting?

Should be resolved in v4. Thanks for reporting!
If this is still an issue, lemme know and we can reopen.


(Improvement) Kitkat(4.4, 4.4.1, 4.4.2) Users get alert message that "uploading file" is not available

This is a workaround (due to some Kitkat versions having problem of not supporting "openFileChooser" or such API).
ref : https://code.google.com/p/android/issues/detail?id=62220

547d877 done


using System CA store not working

Type: Bug
This is a fresh install of VSCode using official apt packages on a new system. VS Code is not using my system CA Store. I have to disable my corporate VPN to access extensions. I tried with and without the "experimental" setting that is supposed to enable using the system CA store. All other tools pointed to /etc/ssl/certs/ca-certificates.crt on this machine can get through the proxy, so pretty sure this is a VS Code issue.
 There is another certificate store for applications using Chromium's network stack, see https://chromium.googlesource.com/chromium/src/+/master/docs/linux/cert_management.md#linux-cert-management for how to add your certificates to that. (/cc @deepak1556 in case there have been updates to this.)

The error I see is "Error while fetching extensions. XHR failed". This machine does not have a chromium based browser installed (Jammy comes with firefox) but I installed libnss3-tools and added my organization's root CA following those instructions. That did not fix the problem, however.

VS Code uses Electron which is based on Chromium. I would have expected that to make it work.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Add --fio_blocksize option

Lets users set the block size in the fio benchmark. This also required a
workaround for a bug in Pint; the fix has been submitted to upstream
Pint, but has not been merged yet.

@stfeng @yuyantingzero please take a look.

A few comments, LGTM otherwise

@yuyantingzero I just pushed a commit that I think addresses all of your comments. PTAL.

LGTM.


Image.getSize(uri) not working

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[x] Use the latest React Native version: https://github.com/facebook/react-native/releases
[x] Run react-native info in your terminal and paste its contents under "Environment"
[x] Let us know how to reproduce the issue. Include a code sample, share a project, or
share an app that reproduces the issue using https://snack.expo.io/

Environment

OS: Windows 10
Node: 8.11.2
Yarn: 1.6.0
npm: 6.0.0
Watchman: Not Found
Xcode: N/A
Android Studio: Version  <IP_ADDRESS> AI-173.4720617
Description

Not able to use the static method of Image getSize(uri)
I tried the following syntax but with no success:
Image.getSize("file:///storage/emulated/0/Download/Image.jpg")
Image.getSize("/storage/emulated/0/Download/Image.jpg")
Image.getSize("/sdcard/Download/Image.jpg")
Steps to Reproduce

Image.getSize(uri)
Expected Behavior

Retrieve the width and height (in pixels) of an image prior to displaying it.
Actual Behavior

Throws Warning:
Failed to get size for image: file:///storage/emulated/0/Download/Image.jpg

This seems to be working as intended. According to the docs, this method "Does not work for static image resources."
Your URIs appear to point to static image resources, the size of which you should be able to obtain in other ways. This method is intended for remote resources.

Hey @hramos can you tell me what are the other ways. The image is located in Downloads directory. I tried resolveAssetSource but with no success

check if you have 'READ_EXTERNAL_STORAGE' and 'READ_INTERNAL_STORAGE' permissions in your AndroidManifest.

Ran into same issue and finally found other ways, couldn't get a local image that was being stored in Android storage directory. Came across this from eXist-FraGGer which worked perfectly.
https://github.com/eXist-FraGGer/react-native-image-size