1.0: Proposal: set up elastic ip for single node clusters

I had previously said something about this in the slack channel and was told it might be a good id to propose it over here on git hub.

Closing this for now as our default installation uses the default AWS public DNS address for nodes. We can re-open or re-visit this if we need to...


Add Shuffle-nbit(32bit/64bit) integers in a across lanes

Add Shuffle-nbit(32bit/64bit) integers in a across lanes using the corresponding 256/512bit mask and store the results in dst.
It's just for 256/512bit(Avx2/Avx512), not for 128bit(neon/sse).

I think we implemented the expected functionality in xsimd::swizzle <https://xsimd.readthedocs.io/en/latest/api/data_transfer.html?highlight=swizzle#_CPPv4I000_Dp2VtE7swizzleNSt9enable_ifINSt13is_arithmeticI1TE5valueE5batchI1T1AEE4typeERK5batchI1T1AE14batch_constantI5batchI2Vt1AEDp6ValuesE>_ and we can close this PR. @guyuqi can you confirm you're fine with it, or propose another plan?

I think we implemented the expected functionality in xsimd::swizzle and we can close this PR. @guyuqi can you confirm you're fine with it, or propose another plan?

It's ok.


adding attributes to the shape model

It would be useful if one could combine some additional attributes, such as the subject's age, weight, height, gender, or even some properties of the shape itself, e.g. length or width of the bone, together with the shape model.
An example use case of such a combined model would be to complete a fractured bone by conditioning the model, not only on the observed healthy part, but additionally on the patient's attributes: age, weight, length of the contralateral bone , etc., which should improve the completion results.
The method presented in this paper https://link.springer.com/chapter/10.1007/978-3-319-64870-5_5  could for example be used in the implementation (generalized to all shapes, not just faces).

I don't know how well the method of Remi Blanc fits into Scalismo. For the copula method, it is definitely possible to squeeze it in. It, however, breaks with some ideas the framework was originally built on - the dependency structure is built on values that have no more a "meaning".
The overhead in Scalismo is more a decision that should be done on a high-level. The alternative would be to provide the implementation as a separate project.
Since the applicability of both methods is quite wide, I would argue to put it as part of the Scalismo framework.

As this issue has been inactive for a long time, I am closing it. In case there is still interest, we can open a new issue once we start with the implementation.


Do we need a gotcha document?

As far as I know, 2.1 introduces some possibly "breaking" changes for the users:

location of the config file inside the official image
shared secret generation
any other?

Should we have a document somewhere to keep track of that for users upgrading? Maybe as part of the changelog?

After some discussion, it seems the release notes are a good place for that.

some comment about the base image being different

cc @bcwalrus @marcusmartins

Might add notes about deprecation of the size field in notifications ( https://github.com/docker/distribution/commit/26b7fe4a9161bc99b43a398f275b1e460449ca34 )

Added to release notes: https://docs.google.com/document/d/1ZrlatwhF8Bcy-1EXJwlYi7SqLKwx-J10wlMiG350hoQ/edit#
ping @dmp42

Is there a better name than 'gotchas' for that section?

Gotchas added to release notes.


fix: uninitialized variables

Description
Of the clang-tidy pointers, the following items should be checked

misc-unused-parameters
cppcoreguidelines-init-variables (uninitialised variables)
cppcoreguidelines-pro-type-member-init (uninitialised member variables)
cppcoreguidelines-narrowing-conversions (precision loss)
Code checks extracted two cases of code to be corrected in relation to cppcoreguidelines-init-variables.

Related links
https://tier4.atlassian.net/browse/RT2-738
Notes for reviewers

Pre-review checklist for the PR author

[x] I've confirmed the contribution guidelines.
[x] The PR is ready for review.

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

[ ] The PR has been properly tested.
[ ] The PR has been reviewed.

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

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

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

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

Project coverage is 55.05%. Comparing base (75e1c3f) to head (8a2d81c).
Report is 18 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #290       +/-   ##
===========================================
+ Coverage   25.02%   55.05%   +30.03%
===========================================
  Files          58       28       -30
  Lines        3321     2185     -1136
  Branches     1085     1210      +125
===========================================
+ Hits          831     1203      +372
+ Misses       1756      702     -1054
+ Partials      734      280      -454

Flag
Coverage Δ

differential
55.05% <ø> (?)

total
?

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

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

This PR is not merged and closed as there are no operational problems without this modification.


Java - Source/target version

What is the intended levels of Java Source and target versions? in the pom.xml, the maven-compiler-plugin specifies a value of 1.5, while the project properties say it's 1.6.
It's better to specify the values inside the project properties.
Also, are there plans to support Java 1.8 yet? That way, we could introduce a lot of lambda sugars to the code base.

pom.xml is now set to hav a source and target of 1.7.
<properties>
    <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
    <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
    <maven.compiler.source>1.7</maven.compiler.source>
    <maven.compiler.target>1.7</maven.compiler.target>
    <encoding>UTF-8</encoding>
</properties>

It is actually redundant for us to specify them in the maven-compiler-plugin but at least they are the same value now.
We have no plans to target Java 8 at this time.


InApp messages not sent (iOS and Android)

Hello, it's me again 😇.
I have the latest Customerio SDK version and the inApp configured to be true.
    "customerio-expo-plugin": "^1.0.0-beta.15",
    "customerio-reactnative": "^3.5.2",

The SDK is configured this way:
  useEffect(() => {
    const data = new CustomerioConfig();
    data.logLevel = CioLogLevel.debug;
    data.enableInApp = true;

    const env = new CustomerIOEnv();
    env.siteId = process.env.EXPO_PUBLIC_CUSTOMERIO_SITE_ID;
    env.apiKey = process.env.EXPO_PUBLIC_CUSTOMERIO_API_KEY;
    env.region = Region.EU;

    CustomerIO.initialize(env, data);
  }, []);

I tried sending a test in app message with CIO ID and email, but nothing appears in the app. Same on iOS and Android.
The push notifications work well.
I don't know how to debug this. I don't see any logs.
Any help appreciated :)

Hey @Samox, thank you for reaching out.
To help make sure we're not missing anything outside of the initialization itself, please send the logs that you captured to<EMAIL_ADDRESS>along with a link to the message in Customer.io that you're using to test. With that information, we'll be able to take a closer look at your setup using the Activity Logs in your workspace.
I'm going to go ahead and close the issue here and look out for your email, but please re-open if you have issues with sending the information to the support email.


Add pack_24_8 helper function for constructing AS Instances

Fixes #474, cc @expenses
Draft because we can possibly improve API design here by making a special type instead of u32 that has this as a constructor? This doesn't include a full builder yet, but if we do anything more fancy than this we should move these types to a new file directly included in ash/vk/ instead of having static text (that doesn't depend on anything from the autogeneration process) hardcoded in the generator.

I think using a custom type is more idiomatic, and conveys to the user that it's not just a single integer. If the inner u32 is kept private, it can also prevent incorrect usage.


Changing the order of numerical columns in side panel does not have any effect in the table view

I put there the human dev. index  first with fertility attribute, selected the countries with the most children per woman and then I wanted to see how they stand when I change the order in the side panel for the sorting to be driven by condom use second after the human dev. index...nothing happened

@mstreit This is should be fixed from recent merge right ?

I did not test the scenario @mijar mentioned. Can you do this please?

Works.


Changed website title

Resolves issue #34
Description -> Changed title and description in app/landing.tsx to match the project

Resolves issue #34

@punyakrit Sir, please do label the issue (#34) and this PR according to GSSoC norms


Accessibility: Update alternative text on Manage Households "Add" buttons

We added alternative text to image buttons in the Manage Households page.
Critical Changes
Changes
Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

Tracking #W-7832109

Original PR closed to rename branch.  Reference PR#5815 for original comments/conversation/approvals.

@CrystalPreston-Watson Need to delete original branch due to naming error, so the PR had to be re-created as well.

@rjanjanam , can you please check into robot test failures next week? Thanks.
CC @npsp-reedestockton , @ravensisco

@rjanjanam The two robot test failures on ManageHouseholds are data locator issues directly caused by the changes in this work item.  The "Add" button still begins with "Add", but the name of the record (Contact) begin added is appended to the Add.  Please let me know if you want me to add a locator for this.  Thanks!

@rjanjanam The two robot test failures on ManageHouseholds are data locator issues directly caused by the changes in this work item. The "Add" button still begins with "Add", but the name of the record (Contact) begin added is appended to the Add. Please let me know if you want me to add a locator for this. Thanks!

@npsp-reedestockton  yes they are failing due to locator change. Please add a data-qa-locator and comment out the failing tests with the line below. That way they will not run as part of the build and later I can fix it.
[tags]                                  unstable

@npsp-reedestockton , thanks for making the changes.
@ravensisco : builds passed. This PR can be merged as soon as the merge freeze is lifted, there is no other outstanding task.


Gyro not working and throwing RangeErrors

When having webgyro connected to cemu, and connecting my phone (Xs on 12.2) to the https server, it starts continously throwing out of range errors. Gyroscope data are not transfered over to cemu. Neither restarting my phone or pc worked.

fixed
try gyro1.7_en.zip


Standalone doc is not correct for CLion

The build directory in CLion is 'cmake-build-debug', not 'build'.
Or you can right-click Standalone's CMakeLists.txt file and select 'Load CMake Project'. Then Standalone becomes available to Run or Debug.

Yeah, the actual executable location is dependent on the CMake generator used. I currently assume Makefiles, but perhaps we should specify that better in the docs!


Saved models?

I am training a very simple try (just 1 picture) coco model just to try the system and using the command
python train_coco.py --datadir /root/detr-tensorflow/coco --batch_size 8  --target_batch 32
It seems to work, but it is unclear if it actually saves the model? I'd like to later use it in the webcam demo
Any help appreciated, thanks

Hi, Thanks for asking,
In the train_coco file exmaple, the following code is used to train and evaluate the model
# Run the training for 100 epochs
for epoch_nb in range(100):
    training.eval(detr, valid_dt, config, CLASS_NAME, evaluation_step=200)
    training.fit(detr, train_dt, optimzers, config, epoch_nb, CLASS_NAME)

Therefore, to save the model, you must manually save the model in the for loop
# Run the training for 100 epochs
for epoch_nb in range(100):
    training.eval(detr, valid_dt, config, CLASS_NAME, evaluation_step=200)
    training.fit(detr, train_dt, optimzers, config, epoch_nb, CLASS_NAME)
    # Save the model here
    detr.save("detr-model.ckpt")

Then at inference you can simply load the same model without the pre trained weights and  load you're weights afterwards
detr = get_detr_model(config, include_top=True, weights=None)
detr.load_weights("detr-model.ckpt")

Hi, Thanks for asking,
In the train_coco file exmaple, the following code is used to train and evaluate the model
# Run the training for 100 epochs
for epoch_nb in range(100):
    training.eval(detr, valid_dt, config, CLASS_NAME, evaluation_step=200)
    training.fit(detr, train_dt, optimzers, config, epoch_nb, CLASS_NAME)

Therefore, to save the model, you must manually save the model in the for loop
# Run the training for 100 epochs
for epoch_nb in range(100):
    training.eval(detr, valid_dt, config, CLASS_NAME, evaluation_step=200)
    training.fit(detr, train_dt, optimzers, config, epoch_nb, CLASS_NAME)
    # Save the model here
    detr.save("detr-model.ckpt")

Then at inference you can simply load the same model without the pre trained weights and  load you're weights afterwards
detr = get_detr_model(config, include_top=True, weights=None)
detr.load_weights("detr-model.ckpt")

Thanks, but getting this:
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/saving/saved_model/save_impl.py", line 570, in call_and_return_conditional_losses
call_output = layer_call(inputs, *args, **kwargs)
File "/root/detr-tensorflow/networks/transformer.py", line 219, in call
key_mem = memory + pos_encoding
TypeError: unsupported operand type(s) for +: 'NoneType' and 'NoneType'

Thanks, but getting this:
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/saving/saved_model/save_impl.py", line 570, in call_and_return_conditional_losses
call_output = layer_call(inputs, *args, **kwargs)
File "/root/detr-tensorflow/networks/transformer.py", line 219, in call
key_mem = memory + pos_encoding
TypeError: unsupported operand type(s) for +: 'NoneType' and 'NoneType'

detr.save_weights("detr-model.ckpt")

Instead of
detr.save("detr-model.ckpt")

detr.save_weights("detr-model.ckpt")

Instead of
detr.save("detr-model.ckpt")

Ok, I got it to work and imported it into the webcam, but it seems to pull regular COCO labels instead of the single one that was declared in the original JSON train file

Ok, I got it to work and imported it into the webcam, but it seems to pull regular COCO labels instead of the single one that was declared in the original JSON train file

Hi @Sequential-circuits , I'm working today on a tutorial that should definetely help you out with thoses issues.
Regarding your last question, it might just be a display issue. You must pass the argument class_name  to the function numpy_bbox_to_image where class_name is the list you used to train your model.

Hi @Sequential-circuits , I'm working today on a tutorial that should definetely help you out with thoses issues.
Regarding your last question, it might just be a display issue. You must pass the argument class_name  to the function numpy_bbox_to_image where class_name is the list you used to train your model.

Thanks. My model has only one class, so I really don't understand where it got those other classes it shows up

Thanks. My model has only one class, so I really don't understand where it got those other classes it shows up

Hi the README you'll find links to some tutorial that should help you to train on your data.
Best,
Thibault

Hi the README you'll find links to some tutorial that should help you to train on your data.
Best,
Thibault


color difference between canvas and download/reproduction

So here's a weird one: While playing around with colors and reproducibility, I've noticed that my matplotlib-figures looked brighter than what I've seen on the screen. And apparently that's true, even though the downloaded figure has the right colors again:

funnily enough, the grey node stroke did not change between display in the browser and matplotlib/downloaded png. Ideas what's happening here?

paging @ulfaslak for some time later

oh, and the background color is also plain white in all of them

code for reproduction
import numpy as np
from netwulf import visualize
import networkx as nx

G = nx.barabasi_albert_graph(5,1)

config = {
    # Input/output
    'zoom':4,
    # Physics
    'node_charge': -70,
    'node_gravity': 0.5,
    'link_distance': 15,
    'link_distance_variation': 2,
    'node_collision': True,
    'wiggle_nodes': False,
    'freeze_nodes': False,
    # Nodes
    'node_fill_color': '#79aa00',
    'node_stroke_color': '#898989',
    'node_label_color': '#888',

    'display_node_labels': True,
    'scale_node_size_by_strength': True,
    'node_size': 5,
    'node_stroke_width': 1,
    'node_size_unevenness': 0.5,
    # Links
    'link_color': '#758000',
    'link_width': 11,
    'link_alpha': 0.2,
    'link_width_unevenness': 0.5,
    # Thresholding
    'display_singleton_nodes': True,
    'min_link_weight_percentile': 0,
    'max_link_weight_percentile': 100
}

stylized_network, new_config = visualize(G,config=config)

import pprint
pp = pprint.PrettyPrinter(indent=4)
pp.pprint(stylized_network)

from netwulf import draw_netwulf
import matplotlib.pyplot as pl

fig, ax = draw_netwulf(stylized_network)
pl.show()

Pretty sure I'm not colorblind, but I actually can't tell the difference :O


frontend: Accept given YAML/JSON in EditorDialog

This change aims to allow EditorDialog to accept generic YAML/JSON text rather than explicitly expecting an item that looks like a Kubernetes resource. It also adds placeholder text in the editor for improving accessibility.
Fixes: #1820

Closed in favor of #1996


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>chore(scripts): cover@#overwrite c8 --temp-directory build/coverage ava && c8 report -r lcov --temp-directory build/coverage
chore(package): add xo from template
chore(package): add nyc from template
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/-test.js,tests/-test.mjs' as in template
chore(package): set $.ava.extensions='js,mjs' as in template

Codecov Report

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

@@           Coverage Diff           @@
##           master     #310   +/-   ##
=======================================
  Coverage   84.11%   84.11%
=======================================
  Files           1        1
  Lines         214      214
=======================================
  Hits          180      180
  Misses         34       34

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 4e3c2f9...61c6534. Read the comment docs.

Coverage remained the same at 84.81% when pulling 61c6534c104e4d8144bd7b4563d2b86cfbdf90e5 on template-sync-1 into 4e3c2f9f668f8a0c287285aa495ab0bace9cd1d6 on master.

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


PSR2 Coding standards

Just an idea, I like PSR2 a lot better than the WordPress coding standards, mainly because it's an actual standard that is accepted everywhere instead of only in WordPress. This would only be a start. As PSR2 would require renaming of the functions etc. (Which I also would happily provide. Just let me know).

Yes, PSR* is much better also for me. I'm often in the conflict with the WP Standard and PSR, but public and older projects use more often the WP Standard in my work.


[Plex] Added the ability to specify subPath on each extraData mount

Added the ability to specify subPath on each of the extraData mounts
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Chart Version bumped
[x] Variables are documented in the README.md - added to values.yaml
[x] Title of the PR starts with chart name (e.g. [radarr])

@billimek @bjw-s Back with another subPath PR :) let me know if I missed anything. Thank you!

Actually, I dont know what the difference is between extraMounts and extraData. What I really need is extraMounts but I think extraMounts shouldn't create a new volume, it should be as the name implies extraMounts from the same volumes.
My use case is I need to mount 2 different paths from the same claim, one has the movies and tv etc and the other has my pictures/home videos. With extra data it doesnt work because theres 2 volumes declared with the same claim. With extraMounts the same thing happens.
Would it be ok to change extraMounts to not create a volume? this would be breaking change, but I think is correct. Perhaps then I need to add a third extra* in persistence?

I actually would like to see if the Plex chart can be migrated to the new common library, if there is going to be a new major version. That would provide all the required flexibility, but would also keep all/most of our charts working the same way.

I agree with @bjw-s it would be nice to convert over the common library, but there's some advanced configuration happening in the deployment like the plexPreferences: and certificate: objects.

I'm not sure if I know enough about the common library but I'd be willing to give it a go. Are there some docs and an example of a chart that is using the library?

If you pull the latest master branch and run grep -r "name: common" . you'll see all the charts that use it. The comoon library is at https://github.com/k8s-at-home/charts/tree/master/charts/common

@onedr0p thank you I'll take a look. As far as the change I want to make.
Are you all ok with changing extraMounts to not create volumes but only generate mounts?

This change is alright with me! It shouldn't break any existing setups.


timer: fix timeout race condition with smp

when enabling smp, the timer's timeout function maybe
handled by another cpu B before cpu A starts to set
the waiting thread as pending and add it into the
timer's wait_queue, so it will cause that the thread
is always pending though the timer has been expired.
so use a spinlock to make sure that those threads waiting
on this timer have already been added into the timer's
wait queue before running timer's timeout function.
I want to re-use the spinlock "lock" in kernel/timer.c file, so I modify the name to a not common name.
fix: #37304
Signed-off-by: Chen Peng1<EMAIL_ADDRESS>

hi @dcpleung @andyross Could you help take a look of this? thanks.

Can you explain exactly what the sequencing is here?  Like, "CPU A does this" "CPU B does this", "CPU A then takes this interrupt and ..."
I'm having trouble understanding what the race is.  I believe you, I just don't see it.
Also: can you split the renaming (which is fine) into a separate patch to make the logic changes clear?

k_timer timeout handler (z_timer_experation_handler()) is completely unsynchronized!

@andyross yes, you are right, get the timer lock in this function to synchronize is the correct fix. I tested your change, it solved the issue.
and thanks for your help, I only know that it needs to use timer lock to sync, but I can't decide where it's the best place to put it, thanks your explaination.

hi @andyross, could you help review this again? I updated the code according to your change, thanks.


Error trying to run it in server only mode

Hello, I'm trying to run MagicMirror² in server only mode using the following command line
docker run -d publish 80:8080 restart always volume ~/magic_mirror/config:/opt/magic_mirror/config volume ~/magic_mirror/modules:/opt/magic_mirror/modules volume /etc/localtime:/etc/localtime:ro name magic_mirror  bastilimbach/docker-magicmirror
but it isn't working. This is the error message: "Unable to find image 'publish:latest' locally
docker: Error response from daemon: pull access denied for publish, repository does not exist or may require 'docker login': denied: requested access to the resource is denied.
See 'docker run --help'."
I re-installed Docker and GIT, but the error remains the same.
I'm using a windows 10 machine.

Newbie error, guys. I forgot to type "--"

Thanks for coming back and closing the issue! You‘re surely not the first one to forget the double dashes 😂


Limit exceeded.

Expected Behaviour
Crawl blogs.
Actual Behaviour
Received Limit Exceeded error. Suggested to lower Tumblr api number of connections. No change when lowered.
Steps to Reproduce the Problem
Detail the steps below to help us reproduce the problem you encountered.

Add twitter blog to Queue.
Click Crawl.

Specifications
Windows 10 Home. TumblThree <IP_ADDRESS>

Update. I tried crawling only tumblr blogs and those were able to crawl. This error seems to only be affecting Twitter "blogs".

Thanks for reporting a problem.
Yes, a few days ago the Twitter crawler stopped to work. It is still unclear why. Looking on the error message it seems to be a problem on their side, and it wouldn't be the first time. But even if the error message should be another one then, it could be a problem in the app, if maybe they changed something in the structure or identifiers.
Does anyone have more information?

Yes, I know why.
A couple of days this "NFT avatar image" change went live, and there has been changes to their site backend in the same vein, like apparently turning off the old API.
Only the new GraphQL API remaining, it seems...
https://blog.twitter.com/engineering/en_us/topics/infrastructure/2020/rebuild_twitter_public_api_2020
https://gist.github.com/MichaelCurrin/ae75545c451629b909a7dd7580a08add

Thanks. Yes, I already found the first link. The second one is an example, also back from 2020. Do you have another link what they discontinued a few days ago?
But that doesn't explain why the urls are still accessible, some work normally and some return 429 instead of 404 or any other error code.
I have to look further in the next days.

I'd simply assume that the inconsistent return results and error codes stem from a gradual roll-out of the backend changes, that's all

It should work again now.

The issue has been fixed and closed. You can still comment. Feel free to ask for reopening the issue if needed.


Broken links

seekr.org takes me to an 'Internal Server Error' page. Also, the link below takes me to a 'Not Found' page.

Thanks for taking the time to point these out!
seekr.org likely isn't going to be supported moving forward, so I've removed it from the README (at least for now).
But I've updated the second link you pointed out to https://github.com/CalabreseLab/seekr/blob/master/seekr/tests/data/example_2mers.csv


Should be struct Transform instead of struct Position in README.md

https://github.com/stillwwater/twoecs/blob/245f67267ca3f81ac84c51454a2d4edf02d0e65f/README.md#L16
and then all links go to type Transform

Thanks for catching that :)


By vectorizing computations and using Numba JIT, the performance of SRISK calculations has been significantly improved.

Note: The initial installation and compilation of JIT functions take relatively longer.

Sorry I missed this PR. Let me check into this later this month. I was trying to use C++ extension when possible but maybe replacing all C++ extensions with Numba is also a good idea.


Problems installing nanopolish

Hello,
when I try to install nanopolish on my macOS Catalina 10.15.3 with Darwin 19.3.0, python 3.7 and Apple clang version 11.0.0 (clang-1<IP_ADDRESS>), I get, once I type "make" after git cloning and cd-ing to the nanopolish folder, the following Error:

c++ -o src/main/nanopolish.o -c -g -O3 -std=c++11 -fopenmp -fsigned-char -D_FILE_OFFSET_BITS=64  -I./include -I./htslib -I./minimap2 -I./fast5/include -I./src -I./src/hmm -I./src/thirdparty -I./src/thirdparty/scrappie -I./src/common -I./src/alignment -I./src/pore_model -I./src/io -I./eigen/ -fPIC src/main/nanopolish.cpp
clang: error: unsupported option '-fopenmp'
make: *** [src/main/nanopolish.o] Error 1

Any help would be greatly appreciated! Many many thanks in advance!
Cheers

Please try with gcc, clang doesn't support OpenMP which nanopolish relies on.

Thank you! I will try that!


Class participants don't show up.

The dropdown menu shows nothing. The arrow to expand it also doesn't register the first time you click it.

Works as intended. Kudos! Thanks for listening.


Remove AI Grant

To my knowledge, Daniel hasn't run it in a few years. It should either be removed or marked as on indefinite hiatus.

Thanks for suggesting @bzreinhardt, good point! I started a section for "inactive programs" and moved AI Grant over there. I do like the idea of maintaining a canonical list of examples, even if they're no longer active, so I don't forget what I've come across 😊


Clear out unused files

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Clears out a load of unused components that aren't in the UI, trying to make the directories less confusing

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Other information:

please rebase


(Improvement) Implement the overlay to bottom/top view

Allow the user to move the bottom view over the top or vice-versa.

I don't understand the effect :S do you have any schema or drawing to show me this?

@ppamorim bump!

Like AndroidSlidingUpPanel:
https://github.com/umano/AndroidSlidingUpPanel

That's not in the original youtube component. I've added your feature to maximize minimize but this doesn't match with the feature look and feel. Sorry.

Well, I believe that the library name is DraggablePanel and not YoutubePanel...
We can be better than the Youtube itself.
But what's the need for it ? Sometimes it's necessary to perform this resizing in some cases such as when the user will enter text in EditText without the keyboard being on top...

I need to do this. I have a bottom view where user comments are shown and an EditText is provided for the viewing user to add comments. When the EditText gains focus and the keyboard is shown, I want to be able to overlay the bottom view on top of the top view. Right now, users can't even see the EditText box when the keyboard is open.


Brainstorm combined (fallback) network improvements

How can we make it more efficient and preserve bandwidth?
What edge cases haven't we covered ? e.g. node operators only running on the webserver

https://www.notion.so/espressosys/Networking-improvement-candidate-ae01c370a12d4aca8de65c44aa5aa43d?pvs=4


[BACKLOG-23271] Fixing typos

https://jira.pentaho.com/browse/BACKLOG-23271

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'plugins/streaming/impls/mqtt' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'plugins/streaming/impls/mqtt' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  plugins/streaming/impls/mqtt/src/main/resources/org/pentaho/di/trans/step/mqtt/messages/messages_en_US.properties


⚠ Processors has degraded performance

In ef2ea30, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 91 ms

Resolved: Processors performance has improved in 93d10e6.


Mediaplayer clementine remote

Description:
Adds support for Clementine Music Player (https://www.clementine-player.org) as media player. Includes album art, volume management, and playlist selection.
Pull request in home-assistant.github.io with documentation (if applicable):
https://github.com/home-assistant/home-assistant.github.io/pull/2007
Example entry for configuration.yaml:
media_player:
  - platform: clementine
    host: <IP_ADDRESS>

Checklist:
If user exposed functionality or configuration variables are added/changed:

[X] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[X] New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
[x] New dependencies are only imported inside functions that use them ([example][ex-import]).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

script/gen_requirements_all.py is failing for me:
File "script/gen_requirements_all.py", line 39, in explore_module
   module = importlib.import_module(package)
 File "/usr/lib/python3.5/importlib/__init__.py", line 126, in import_module
   return _bootstrap._gcd_import(name[level:], package, level)
 File "<frozen importlib._bootstrap>", line 986, in _gcd_import
 File "<frozen importlib._bootstrap>", line 969, in _find_and_load
 File "<frozen importlib._bootstrap>", line 958, in _find_and_load_unlocked
 File "<frozen importlib._bootstrap>", line 673, in _load_unlocked
 File "<frozen importlib._bootstrap_external>", line 665, in exec_module
 File "<frozen importlib._bootstrap>", line 222, in _call_with_frames_removed
 File "/home/jjmontes/git/hass/env/lib/python3.5/site-packages/homeassistant/components/openalpr.py", line 19, in <module>
   from homeassistant.components.ffmpeg import (
ImportError: cannot import name 'get_binary'

Please read this https://github.com/home-assistant/home-assistant/issues/4210 and can you aprove that all property stuff are cached in your library?

Yes, I'm aware of that. Everything is in variables, in-memory. No I/O of
any kind is performed, and all the accesses are thread safe. In fact, a
future version of this component could be made entirely async, because the
Clementine remote protocol allows for it, but this should be safe for now.
I also have been testing it for several days now.


readme has incorrect port if using docker

theres a minor oversight in your readme. if you pull your project from the docker registry with:

docker run -d -p 49153:5000 sahilsk/j2parser

the port to access it will be 49153, not 5000.

Hello,
Sorry about that.  I suggest a simple 5000 to 5000 mapping then:

docker run -d -p 5000:5000 sahilsk/j2parser

Sounds good to you ?

sure. thanks!
On Monday, February 8, 2016, Antonin Bourguignon<EMAIL_ADDRESS>wrote:

Hello,
Sorry about that. I suggest a simple 5000 to 5000 mapping then:
docker run -d -p 5000:5000 sahilsk/j2parser
Sounds good to you ?
—
Reply to this email directly or view it on GitHub
https://github.com/abourguignon/jinja2-live-parser/issues/6#issuecomment-181387209
.


Contect us form : Phone Number and input box not getting Validated

Describe the bug
In the Contact us form , the Phone number is not getting validated with proper number validation rules and thus we are able to submit any random number in the phone number section
To Reproduce
Steps to reproduce the behavior:

Go to 'Contact Us in Home page'
Click on 'Fill in a random number and click submit'
The form will get submitted

Expected behavior
For a random number in the phone number input tag , it should check for the number validation
Screenshots

Even for random number , the form is getting submitted
Desktop (please complete the following information):

OS: Windows
Browser [chrome]
Version 11

@Priyaaa1 Please assign me this issue under GSSOC'24 tag as i am a GSSOC Contributer


Added Cask for Now Desktop

My team at @zeit and I decided to add the application nevertheless.
When looking over this PR, please keep in mind that the download isn't versioned and we're also not distributing over tools like Github Releases, so I also left the appcast away.
I carefully read the documentation and made sure to align with it.

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[x] Named the cask according to the token reference.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.
[x] Checked there are no open pull requests for the same cask.
[x] Checked that the cask was not already refused in closed issues.

Everything from https://github.com/caskroom/homebrew-cask/pull/27397 still applies. Please make this versioned and add the appcast.

Please understand this: HBC is not your (or any single developer’s) personal distribution channel. We exist for the users, and users overwhelmingly (the ones that made their voices heard, at least) prefer versioned downloads for security reasons.
You cannot bend the rules to fit your case.
I prefer :latest and have made that clear when we were discussing this at the time. But even being the oldest active maintainer in HBC, I surrender to the evidence and let my preferences aside. In the end, HBC is for users.
You’re more than free and welcome to maintain your own tap for this software, and direct your users there. Taps are by far my favourite feature of HB, as they make sure you aren’t confined to our rules for specific repos. You can make your own rules, and taps are incredibly easy to setup!
But if you want your apps in the official main repos, I ask that you follow our rules. They’re what keeps our sanity and project consistency amongst the flood of PRs we get everyday.
If you do change your mind, please do let us know. As it is, I see no difference between this and #27397.

And as I said before, if you as the maintainer are unwilling to add cask-repair -v {new_version} now to your build steps which does everything automatically, including submitting the PR, then I don’t want the submission to come from you as that signifies the cask will be left abandoned.

Lastly, please do not open repeated PRs when one gets rejected. Keep the discussion in the original one so we can solve the problem. Closing does not prevent further posting.

Okay. I'm submitting a versioned one! 😊
Just explain this please. There are hundreds of these in the repo!

Just explain this please. There are hundreds of these in the repo.

That one doesn’t have an auto_updates. As far as we know, it doesn’t auto update. It also may not have an appcast.
Or it may have both of those but we didn’t know it at the time. Maybe https://github.com/caskroom/homebrew-cask/issues/6872 makes it clearer. You see, we do not accept casks that use an installer or pkg and that do not provide an uninstall. We require the latter to be present as well, for the cask to go through.
Now, those casks in that issue did go through at the time, and are indeed present, so what gives? Well, the answer is simple: they were added before the rule that would prevent them from being added was implemented. So that means we need to fix those and we don’t let anymore in to not exacerbate the problem.
This is the same thing.
Casks with such errors are usually fixed when they are discovered (just take a look at the recente excellent work by @colindunn to add missing appcasts).
Every time a rule is implemented, we can’t automatically fix all the present casks that disobeyed it. But we can be vigilant to the new ones, and fix the old ones as time permits. Not letting more errors creep through makes sure the task of fixing the old ones is a finite one, and that is paramount.


Allow to create Ed25519Sign.KeyPair with a given private key.

This change will allow to create Ed25519Sign.KeyPair with a given private key.
The usecase is to create a key pair from bip39 mnemonic:
val mnemonic = MnemonicCode("user input")
val privateKey = mnemonic.toEntropy()
val keyPair = Ed25519Sign.KeyPair.createKeyPair(privateKey)

val publicKey = keyPair.publicKey
val signer = Ed25519Sign(keyPair.privateKey)

@googlebot I signed it!

@tholenst Could you please review it?

@tholenst Could you please review it?

Hi @aartikov, thanks for your PR, sorry for the time it took to get to it.
An equivalent API was added in this commit and thus the API added by this PR would be redundant. I am going to close this PR, please feel free to reopen it if you think there is something missing.


Cassandra Cluster Example Add Check for Null Endpoints (redo)

Fix for Issue: Cassandra Cluster Example Custom Seed Cores on First Node #5737
Also note: for this change to be effective, the following jar must be rebuilt and checked in:
kubernetes/examples/cassandra/image/kubernetes-cassandra.jar

However, this example does not have any visible java build and testing infrastructure.

Signed SLA:
Samsung SDS America
<EMAIL_ADDRESS><EMAIL_ADDRESS>( alternate email<EMAIL_ADDRESS>

Cassandra Cluster Example Add Check for Null Endpoints. #5752
Redo.

Actually.. a question.   Should this pull request actually go against master?   let me know and I'll start it all again.

Hi Mikel,
Thanks for looking through the Cassandra example and catching this bug. And sorry about the CLA Bot -- it doesn't always do well with corporate CLAs...
As to your other question, yes, you should make the pull request against master since we are unlikely to make another cherry pick release from the 0.13 branch before we cut a 0.14 next week.
You should also update https://github.com/GoogleCloudPlatform/kubernetes/blob/master/examples/cassandra/README.md which inlines the source from this file to keep it in sync (as it inlines this source file).

Ok!  Let me close this one, and open another one against master.

moved to Cassandra Cluster Example Add Check for Null Endpoints (redo2) #5808


Docs are not properly displaying the list items with  tag

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://docs.microsoft.com/en-us/dotnet/api/microsoft.bot.builder.botframeworkadapter.continueconversationasync?view=botbuilder-dotnet-stable'
Scroll down to 'Remarks'
See list not displaying its list item it just displays the bullet

Expected behavior
Display the list items

Submitting a PR in the botbuilder-dotnet repo to fix this issue. The doc fix should go live shortly after the 4.3 NuGet packages are released.


Handle error return of listener gracefulStop() method

Purpose

$Subject
Currently error returns from gracefulStop() methods of listeners results in panics. Actual behavior should be returning error. Added a fix with PR.

Fixes #40978
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Closing this, since the commits are added to https://github.com/ballerina-platform/ballerina-lang/pull/41416


Cannot read property '$touch' of undefined for input type="number"

 <input type="number" min="1" v-model.trim="object.nbr" @input="$v.object.nbr.$touch()" />
Givng me Cannot read property '$touch' of undefined . Seems to work fine on an input type of text.  does vuelidate not handle newer input types or am I doing something wrong?

nevermind, my error. feel free to delete my comments...


remove unused / broken code

This one is not really necessary, but it would be nice if you accept - then my fork will stay closer to  yours.
Apart from obviously not used libs (stats, dat gui) and vars, there are 3 major points:

dynamic items purged from the code that supports some cube animations - it is not used at the moment, but if you will start using it, you will run into problems with multiplayer (where one user triggered the animation but all the others did not); another reason is this simplifies the job for my in-game editor 😉
development layout and json loading removed - it did not really work at all (I have replaced it with this in my fork - but that is beyond this PR);
orbit controls - they did not follow the player, and even if that was fixed - I dont think it would be really useful.

Ah this dynamic items system... If you knew how big the project was when I started it ! It was basically Zelda, with moving NPCs and pets, side quests... Then I realised how much work it would be (especially the level design and modelling), so I cut it down a lot.

I'd really like to merge your map editor when you finish it !

That would probably be somewhere in the summer. So far, only scaling-moving and removing the cubes is supported. No tiles, no planes.
But I sort of like where it is going, e g you could move the cube and immediately see how the player interacts with it. Or you could edit glb and add then add the cube to match it - the opposite workflow to what you had before.
A day ago I have added 3 lines of code to keep the guy from falling when gliding, so that I could navigate the map easy but still keep the player bahavior exactly as in the game - I have then spent like half an hour just flying around with no clear purpose. Things like that sort of keep it fun for me.

It's super exciting, especially if you can try the map in one click after editing !

A day ago I have added 3 lines of code to keep the girl from falling when gliding, so that I could navigate the map easy but still keep the player bahavior exactly as in the game - I have then spent like half an hour just flying around with no clear purpose. Things like that sort of keep it fun for me.

I was thinking about it, if you let the player fiddle with the map, it may be better to let them choose their powers at start, and maybe add some simple options like tweaking jump height, or as you said making new fun powers like infinite flying or overpowered dash ?

I still have infinity jump not changed to double jump in my fork, so together with infinite gliding that covers pretty much every navigation need.


Add Population Date and Population Source fields

Added Population Date and Population Source fields that only show up if Population is already tagged on the feature. Unlike the Source field, Population Source completes the most popular values on taginfo instead of a fixed set of known values, since there are so many possible values depending on the region.
Ideally, census:population would be deprecated in favor of a combination of population and population:date: openstreetmap/iD#6678. However, I don’t think the deprecated.json format supports splitting a tag value across two keys (which is one of the problems with census:population in the first place).
Fixes openstreetmap/iD#6672 except for the validation ideas in there. In my opinion, it would be worth visually grouping the fields first to set a user expectation that the fields are related: openstreetmap/iD#8104.

In my opinion, it would be worth visually grouping the fields first to set a user expectation that the fields are related: openstreetmap/iD#8104.

:+1:

Ideally, census:population would be deprecated in favor of a combination of population and population:date: openstreetmap/iD#6678. However, I don’t think the deprecated.json format supports splitting a tag value across two keys (which is one of the problems with census:population in the first place).

After reading the cleanup section of the corresponding wiki page, I think this is a bit more complex than what one could reasonably model in the context of the deprecated.json data. To me it looks like as if such situations would need to get "hard" coded validators for each explicit tag, wouldn't they?

Right, I only meant that census:population would ideally be deprecated, but we’re far from that point and I don’t think iD’s tag deprecation mechanism is the right tool for the job. (It’s probably a job for MapRoulette.) This PR would assist any manual efforts at cleaning up population data by making the approved tags more visible.


Update oc4ids_schema table

@Ravf95 spotted that this needs updating with the latest schema:

I checked the oc4ids_schema table from the oc4ids database and I think it's outdated, some fields are missing.
e.g. forecast and metrics fields are missing

@Ravf95 I've updated the table in the database - let me know if you have any issues.


feat: publish preview to IPFS on PR

move CI from jenkins to circle
use ipfs-dns-deploy to get an IPFS preview status link on PRs

License: MIT
Signed-off-by: Oli Evans<EMAIL_ADDRESS>

@PedroMiguelSS i'm seeing test failures when trying to push...
  Running: service-worker.js...

  1) Service worker activates and then deactivates successfully:
     Uncaught SyntaxError: Unexpected token ...

@alanshaw also mentioned it in #254. I did take a look at this and I found out that cypress has a problem when we use cypress run command. This command by default runs the tests headlessly in the Electron browser. Electron browser does not support es6 syntax.
We are currently using babel to transpile our src/ code but I think we must create a lighter babel config to transpile our node_modules/ code so that Electron does not complain about syntax errors like this one.
Alternatively, we can add --browser chrome to cy:run script. This will open chrome and run the tests automatically. However, this is can only be a temporary solution because when we pass the --browser argument it will attempt to launch the browser if detected on the machine. This is not ideal to run the tests automatically on the CI.
If you want to be sure that the tests are not failing you can run npm run test:e2e and it will open up Chrome and then you can run the tests manually.
I'll fix this, but I need to include this task on my sprint. I'm currently working on IDM project. We can discuss how important comparing to IDM tasks and include it on my sprint.


Add ability to grab pageURL as a state object

Love your work on this, makes it so much easier for me to implement swiftui webview.
I was playing around with it to see if I can save the page's url but because the your webview package is constructed on load i couldnt use the general " webView.url" call.
I added the following code on your WebViewCoordinator extension after calling "   public internal(set) var pageURL: String?
" on your WebViewState constructor
webView.evaluateJavaScript("document.URL.toString()") { (response, error) in
if let url = response as? String {
var newState = self.webView.state
newState.pageURL = url
self.webView.state = newState
}
}
Think itd be a great function to have especially for newbies like me so they dont have to go chasing around the code trying to figure out how to load the url : )

Hi, thanks for the feedback! Version 1.0.7 now contains pageURL state property.


Update issue template

What this PR does / why we need it:
this PR use Github issue better!
Special notes for your reviewer:
You can preview it in my fork: click me
Have a good day!
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

@robertsirc  It's fine. This PR is not meant to fix a bug, instead of to enhance the form of Github issues. You could click my fork and create new issue to try it!

I know what it is for. That is why I asked if you can add a section for helm charts to reproduce the bug.

@robertsirc sorry. I don't know what you mena

Sure you have sections called Helm Version and kubectl Version can we add a new section called helm chart with a description of Link to a helm chart that can reproduce this error.

@robertsirc I get it


Adding source endings to tokens

Currently working on adding source line endings to all tokens as discussed here. However, I wanted to move this to it's own issue instead of hijacking the other.
Going to make it match babylon's loc format which looks like:
var token = {
    loc: {
        start: { line:1, column:1 },
        end: { line:1, column:13 }
    }
};

Note: Babylon's loc property also includes a loc.filename property; however, attaching the filename using the this.filename property from the lexer resulted in the absolute file path and thus produced unit tests that are OS/folder dependent. I'm just leaving it off right now until I get input on this. Using the relative file path or reworking the tests a bit would solve this issue.
Any thoughts?

So, given the following examples:
- var name = "Josh"
^ ^
1 2

p hello #{name}!
        ^ ^
        1 2

p= name
 ^ ^
 1 2

The lexer currently marks the beginning of the code/interpolated-code token as being at position 1 for all examples. However, it then goes on to set val starting at position 2. I would argue that any issues regarding Pug's code syntax will throw an error in the lexer at the place of the error. After lexing, the position of the code syntax is not nearly as important as the code itself.
If we create tokens for JavaScriptExpressions (as @ForbesLindesay mentioned here) and store them inside code/interpolated-code tokens, then we would be storing both positions.
Thoughts?
For now, I'll just use position 2 and will change it later if needed.

For the filename, in the test cases, you can just replace __dirname with '' then replace \\ with /.
Yes, I think we will need a separate JavaScript expression token.  This will be essential when we generate source maps/use babel-code-gen.  Using position 1 sounds fine for now.

Cool, I'll go with that. I suppose we could have the lexer just store the relative file path as well.
Yeah, keep thinking an identifier token for tag/attribute/mixin/class/id/filter/etc names as well as a JavaScript expression token will be in order later down the road. For now just focusing on getting an end location for all current tokens.
For future reference though, consider the following:
a("foo"=bar)
--^^-^^---^
  12 34   5

I would expect the lexer to have an attribute token spanning from 1 to 5, followed by an identifier spanning 2 to 3 (since quoted attribute names do not include the quotes), and then a JavaScript expression to span bar, correct?
So far, things are going pretty well. Had to write a function to console log all result tokens followed by the text they are spanning so I can quickly see if one of the endpoints are off. After that, things are moving along well. Getting a little hung up on the attributes as the code is a little tricky but not bad. Attributes with quoted names currently start at position 2 (using the example above) where I would expect it to start at position 1, correct?

For info if you need it, here is the work-in-progress rewrite of pug-code-gen using babel AST. it works and parses the javascript expressions AST which we can remove when the lexer will 'cache' the parsed AST.
https://github.com/jeromew/pug-code-gen/blob/master/index.js
currently I use https://github.com/jeromew/pug-code-gen/blob/master/index.js#L108 for parsing expressions and https://github.com/jeromew/pug-code-gen/blob/master/index.js#L111 for parsing argument-like structures.
The loc numbering with these hacks has an offset + there are corner cases that could be parsed wrongly with these constructs so a PR is ongoing on babylon on https://github.com/babel/babylon/pull/213 to be ably to parse expressions directly.

The lexer should not produce a separate tag token + identifier token, since the tag token in the lexer ends at the end of the tag name.  It's the parser where we would create most of that nesting.
Yes, I think currently we just produce a single "attributes" token, but I think it would be better if the lexer produced a stream of start-attributes attribute-key attribute-value end-attributes`.

@jeromew: Nice! I'm excited for that babylon pull request. I was doing the hacky stuff you described on a project myself. Thank you for pointing these lines out! I'll keep them in mind as I get closer to that piece.
@ForbesLindesay: That makes sense. I've been reading up on the difference between lexers and parsers and believe I was blurring the two. I'm keeping the single attribute token for now until I have spans on all tokens but it is ready for the setup you described. Definitely cleaned it up IMO.
I did find an inconsistency. One of the tests has the following line:
a(foo, bar, baz)

The expected tokens for these attributes are:
{"type":"attribute","line":3,"col":3,"name":"foo","val":true,"mustEscape":true}
{"type":"attribute","line":3,"col":8,"name":"bar","val":true,"mustEscape":false}
{"type":"attribute","line":3,"col":13,"name":"baz","val":true,"mustEscape":false}

Notice the values of mustEscape for each? The first is true while the others are false. Maybe it doesn't matter because a boolean val doesn't use the mustEscape value? Or is this by design? The value is initialized here and reset here.

@jbsulli, good catch! Practically, I guess it doesn't matter, but I think the preferred behavior should be to reset mustEscape to true (in case some plugin that changed val forgot to set mustEscape to true).

Yes, this definitely looks like a bug.  mustEscape should be set to the same value in all three attributes.  I would tend to agree with @TimothyGu that true is the better default here.

@jbsulli We're moving everything to a single monorepo: https://github.com/pugjs/pug
There a few advantages:

It is easy to run all the tests, rather than only the tests that directly test the lexer.
Jest, the test runner we're now using, has a great watch mode (npm run watch).
Jest has snapshots builtin, and I've set them up so they automatically truncate absolute file paths so that they work fine, providing the filename is a string and is a "filename" property.

I suggest making your changes on the monorepo as this repo will be disappearing soon :)

Done! That was much crazier than I expected but I learned a lot. All unit tests are passing and I have comparable test coverage (I'd be down for getting it to 100%, if you want).
All tokens are the same except the location data and there are no new tokens, yet. Most tokens just needed the this.tokEnd(tok) function I wrote to be called just before saving the token.
Attributes, however, were completely rewritten as it was much too complicated to keep track of actual starting and ending locations. There are now three functions handling attributes: attrs, attributeName, and attributeValue. This was to make it easy to pull them out if we want separate tokens in the future. The mustEscape value is now true for all boolean attributes.
I'll work on building a pull request for the new monorepo so you guys can look at it.

Thanks, this all sounds incredibly awesome.  Getting to 100% test coverage would be great, but don't stress over getting it there as part of this pull request. I'd much rather see this pull request merged with decent test coverage, then see a subsequent pull request that just increases the test coverage.
Can't wait to see the code you've written, especially the new attributes parsing code.

Started working on moving it over to the monorepo. The snapshots are making my eyes cross. Is there a way to break those out so it's not a single 46,000 line file? Haven't messed with Jest so I'm still learning it. So far it seems pretty cool. Their expect function is more inline with what I'm used to (Mocha/Chai) and they have a pretty sweet plugin for VSCode. The custom matchers sound awesome! I've been writing things like that in Mocha for a while.

The only way to split the file up is to run the tests in separate JavaScript files.  I'm starting to move towards this approach with new tests (e.g. https://github.com/pugjs/pug/pull/2685/commits/d91bab775694266f1b33bc343f730b224f306f9c), but I don't think it's worth doing so with existing ones.  I don't think it makes too much difference to how bad it is to review the diffs though.
It might be something we could propose to jest though.

Hey everyone. I've been fighting the flu so I've not been able to get anything done on this lately. But now I'm ready to get this going again.
What are your thoughts for things like tags, mixins, blocks, etc? Do you want the loc value to span just the tag and attributes? Or all child content as well? Ex:
#content
    p.first
        a(href="http://www.google.com") Google
        | is planning on taking over the world.

So do you want the loc to span just the tag name and attributes (ex: #content, p.first, a(href="http://www.google.com) or the children as well (ex: #content would span the entire source above, p.first would span all the way until the period after world, and a(...) would span Google)? We could have it span the definition but have block nodes (that hold the children anyway) span the children. Any thoughts?

Hm, you also have different file sources being included into the main file. The loc can only span lines/columns of a single file as it is currently defined. I don't think this is really an issue but a block node spanning an include would later have content from different source files. Just some thoughts.
So what I am thinking for now is have blocks span all children but the tag would span its name, id, classes, attributes, and &attributes but not the children. When includes/extends happen, the container block's loc value would stay the same but the children merged in would have their own loc values pointing to their source files.

The general rule is: the loc should refer to the entirety of the source code that was part of that token (before linking)  In the lexer, this normally means just the tag.  In the parser, it always means tag + children.  i.e. everything that's within that object.  Don't worry about the multiple files issue, since that doesn't happen until the pug-linker stage.  After linking, we just preserve the values before linking.

@ForbesLindesay: That makes things much easier.
I've gotten pretty far with the parser but not sure I like how I've implemented adding the loc value to the node. The problem is merging all children into the current node. Right now I have wrapper functions that you add around each node you add as a child. This takes care of expanding the parent's loc so it includes the child. However, the wrapper function approach seems fragile and requires a pretty good knowledge of what is actually going on. Doesn't seem like it will be very user friendly for other maintainers going forward.  Right now there are 4 different functions for handling this and they really don't seem intuitive. You can see what is going on by going checking out this commit.
I feel like a better approach would be to track the last legit token's ending location and expand the parent's end to that position when the children are done being parsed. But this is difficult because all the advance logic is handled inside a entirely different module (token-stream). I don't know that I can do this in a friendly way without some pretty big changes. I'm curious if a lot of the parser logic could be cleaned up by treating the nodes more anonymously and including the token-stream handling inside the module (with more pug-specific handling). I'm going to play with it a bit and see if it's a bad road to go down or not. Any thoughts? Are you OK with some pretty big changes? I don't want to hijack this.
If you want to really see what the loc values look like, you should check out my handy CLI tool pug-loc-debugger. It's been a life-saver for this project and is kind of fun to see in action. It should be pretty future-proof too because it's a global package and just takes over the pug modules in the folder it's running from and taps the plugin system. Great example of using the pug plugin system too. Just clone my repo, switch to the source-end-locations branch and run the pug-loc-debugger from that directory.

I don't have any strong opinions on this at the moment.  Would you be able to submit a pull request for adding token endings to the lexer?  I'd much rather review the updates to the lexer, get that landed, then start looking at what changes are needed in the parser.  It's much easier if we do one thing at a time :)

P.S. pug-loc-debugger looks awesome.  I've been meaning to build something like that for a while.

@nojvek, thanks for the bump! I forgot about this project with all the life changes I had (new job/big move). I'm wrapping up the pug-lexer piece now and just got it to pass all the tests. This is just the first phase though. Gotta update the parser etc. to implement endings as well.
Pug source maps are going to be sick. Also looking forward to getting everything moved over to babel and passing the babel AST along so we (hopefully) have a nice speed improvement.

Re: life changes. Totally understand.
Very much looking forward to sourcemaps and speed changes.
You are the change the world needs to be :)
On Sun, Feb 11, 2018 at 12:05 AM Joshua Sullivan<EMAIL_ADDRESS>wrote:

@nojvek https://github.com/nojvek, thanks for the bump! I forgot about
this project with all the life changes I had (new job/big move). I'm
wrapping up the pug-lexer piece now and just got it to pass all the
tests. This is just the first phase though. Gotta update the parser etc. to
implement endings as well.
Pug source maps are going to be sick. Also looking forward to getting
everything moved over to babel and passing the babel AST along so we
(hopefully) have a nice speed improvement.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pugjs/pug-lexer/issues/76#issuecomment-364731725, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA-JVAy9PQEJRURtgkiOlujnydtB8ZQMks5tTp80gaJpZM4LmFUJ
.

https://github.com/pugjs/pug/pull/2957


Graphql Apollo + Websocket

New example showing off Apollo Server with subscriptions.
This could eventually be integrating right into the ApolloApi construct itself.

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


fix(plugins): tostring to handle nil as well

Summary
the error object may be nil, in which case the concatenation fails without an explicit tostring call
Checklist

[ ] The Pull Request has tests
[x] A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Do we know the root cause why e.plugin is there but no e.err?

Do we know the root cause why e.plugin is there but no e.err?

@bungle Good question!
I ran into it while testing; I simply put an error() call in the phase handler, which caused the error object to be nil. So the root-cause was not in the gateway code, it's just that the gateway should be able to handle that gracefully.

Do we know the root cause why e.plugin is there but no e.err?

@bungle Good question!
I ran into it while testing; I simply put an error() call in the phase handler, which caused the error object to be nil. So the root-cause was not in the gateway code, it's just that the gateway should be able to handle that gracefully.

Add a test?


Can we monitor apache tomcat 9 using this exporter ?

Can we monitor apache tomcat 9 using this exporter? If we can monitor can you provide the details of how to monitor?
I tried whatever there in the README.MD file. For httpd it is working fine, but I need to monitor the Tomcat application.
I am using centos 7
webserver httpd
application server Apache Tomcat
In httpd we have http://url/server-status
whereas  in  apache tomcat application we have http://url/manager/status/all  where we get all parameters of server status
I tried
./apache_exporter -scrape_uri="http://tomcat:tomcat@<IP_ADDRESS>:8080/manager/status/all/
The exporter is showing apache_up 1, but I am not able to get the metrics
Thanks in Advance

Tomcat is a different project hosted by the Apache foundation.
You should use its own exporter, such as https://github.com/nlighten/tomcat_exporter

Thank you very much for your reply. I thought that we can monitor the apache tomcat also. Now I got it


Backport to Python 2

any updates on the backport ?

I am currently working on it.

@yonilx, I have drafted some possible solution for this issue on this branch
https://github.com/ntucllab/libact/tree/backport2
Since we might have to take some time to review over the code, you may use the code on this branch first. Also feel free to give us comments.
Thanks.

We've merged the backport branch into master, it should support python2 now.

Thanks !


Option to leave REX prefix unmasked in instruction pattern search

Is your feature request related to a problem? Please describe.
When editing an instruction sequence in the instruction pattern search window, ghidra always(?) masks out the low 3 bits of the REX prefix sometimes leading to false positives in search results.
As an example, consider 41 ff d2 which is call r10. Ghidra will generate the search string [01000...] ff d2, which also matches 40 ff d2 (call rdx). I cannot find a way to disable this behavior; even if I enter 41 ff d2 directly into the bytes editor it will still mask the REX prefix.
This is somewhat surprising since d2 already encodes some register information, and ghidra gives you the option to mask it. I expected leaving r10 unmasked in the UI to generate a search string like [01000..1] ff d2.
Describe the solution you'd like
I would like either of the following:

A way to toggle REX prefix masks for individual instructions
When an operand is unmasked, the corresponding information in the REX prefix should be unmasked

I'm not sure how feasible 2 is since x86 is a bit of a mess.
Describe alternatives you've considered
The alternative is editing the search string manually, but it's error prone and painful for longer instruction sequences since the editor is in another window without accompanying instruction information. You also can't use the instruction pattern search window to do this since it doesn't let you modify the mask.
Additional context
None

After messing with this problem for a couple days I think this might be unsolvable with the current instruction pattern search, so I'm going to close this.
My original example is flawed because call rdx doesn't actually need a REX prefix. A good compiler isn't going to emit 40 ff d2; it will just emit ff d2. In the case where a REX prefix is present, ghidra will always generate a search string that matches 3 bytes, but some calls are 2 bytes. This means that even if you mask out r10 in call r10 and do an instruction search, you'll never find occurrences of the 2-byte call rdx; instead the only results will be other calls that have a REX prefix. There are ways to solve this like regex, but it would be a huge effort to implement and result in slower searches. Instead I will just work search strings out on a case by case basis.


Add check null

Fixes #https://github.com/pion/ion-sfu/issues/499

Add configuration when creating PeerConnection, related to #https://github.com/flutter-webrtc/flutter-webrtc/pull/552

LGTM


compilation.afterOptimizeChunks hook not supported with rspack

System Info
N/A
Details
compilation.afterOptimizeChunks hook not supported with rspack.
we use plugins in our apps which modify the chunks at afterOptimizeChunks hooks in compilation. Request to support these hooks.
Infact none of the chunk related hooks are working, it would be great to prioritize their support.
Reproduce link
No response
Reproduce Steps
N/A

I have the same needs and hope to support them as soon as possible

bump


Improve reproducer role

With this patch, we extend the reproducer role in such a way it will
properly integrate and support dev-scripts provided VMs, and properly
manage the ones we may want to inject using libvirt_manager.
With this change, we ensure that:

devscripts is running first, so that it can create its networks

libvirt_manager comes next, and is able to consume any existing
networks provided by devscripts

the reproducer is able to generate and provide mandatory files related
to the inventory and network, even if we're not running a CI Job
Reproducer

we introduce a "light" EPEL management, in the wait of some formal
inclusion in either ci_setup or repo_setup

we correct a tiny issue in the push_code.yml that prevented correct
matching of the "kind" of repository (remote or local), since match()
is greedy.

This patch is a major refactoring of the reproducer role, allowing to
get extended features.
As a pull request owner and reviewers, I checked that:

[X] It has been iterated over and over during the VA-1 introduction

/unhold
As said in the beginning of the new doc page, it will be updated once we get all the needed pieces in, so that we can make an easy-to-follow (aka easy-to-copy) workflow.

/approve

/approve cancel
I'll request a full review on due time.

Not needed anymore, data are in.


PMID:24268782

abolished protein localization to nucleus during cellular response to hydrogen peroxide
decreased protein oxidation during cellular response to hydrogen peroxide

abolished protein localization to nucleus during cellular response to hydrogen peroxide FYPO:0004835
decreased protein oxidation during cellular response to hydrogen peroxide FYPO:0004836

edit file: da1bd1e5786ffb9e20004d45148111063341b9df
release: 6ed5941ce8ecfabee18f07523157e66f731a322c


Translate all NFVO<->VNFM amqp message to RPC over amqp

It is better to use the convertSendAndReceive method always when from NFVO is sent a message to the VNFM (like the plugins).

I would suggest to discuss this further before taking this direction..

too late


adding unit tests for lib folder

Reason/Context

Lib folder , has 2 files , majorly api.js and staticHelpers.js , these files have utility functions that are used all over the codebase in variety of components .

- Api.js :

This has 3 functions namely getAllPosts() ,  getPostBySlug() ,  getDocBySlug()  .
getAllPosts() : This function returns all the posts from post.json file , ie blogs , docs , etc .
getPostBySlug() : This function checks for the post type and matches it with slug .
getDocBySlug() : This function is used to retrieve a specific document by its slug. It takes two parameters: structuredPosts, which is an array of structured posts, and slug, which is the unique identifier for the document. It searches through the structuredPosts array and returns the first document that matches the given slug and is not a section.

-StaticHelpers
-getEvents : returns events with sorted dates
Description

Write tests using cypress
This is not a breaking change

This is for Gsoc and I shall be working on this


feat: adding vendor module chunks in Correlation plugin

##Added vendorModules chunk

vendorModule ==> /node_modules except sharedModules
This is required to get this stats used in client side preload/prefetch or somewhere to hint/ report vendors

Is this ready to be reviewed

Is this ready to be reviewed

Yes Please check and provide comment so that necessary changes.


fix: context loss [SBK-346]

Context seems to be lost when calling streamFromEventLog (error: this is undefined) in the SDK.

bind 'this' when calling the function
Use arrowfunction to maintain the StreamManager context even when it is being called by other code

console.logs in index.ts are used to debug when using the SDK. They need to be removed in a later release.


Aves wrongly thinks some of my jpeg files are webp

Describe the bug
Aves is detecting some files as WebP instead of their actual file format (for now, I've only seen this happen to files with the .jpg extention)
To Reproduce
Unknown.
Expected behavior
Should recognize the file as a jpg file and allow tagging.
Screenshots

System information and logs:
aves-logs-20240822_202838.txt
Package: deckers.thibault.aves.libre
Installer: com.google.android.packageinstaller
Aves version: 1.11.8-libre, build 12702
Flutter: stable 3.22.3
Android version: 10, API 29
Android build: QP1A.190711.020.G965U1UES9FVD1
Device: samsung SM-G965U1
Support: dynamic colors=false, geocoder=true, HDR=true
Mobile services: not available
Connectivity: wifi
System locales: en_US
Storage volumes: /storage/emulated/0/, /storage/0123-4567/
Storage grants:
Error reporting: true

Could you please attach one of these files here?

This is the one of them. I downloaded the image from reddit of that helps, i think all the affected images are reddit downloaded, but I also have a tone of reddit downloaded images that arnt affected and are treated normally so idk

The image you attached is actually a webp, even though the file name extension is "jpg". It's confirmed by other software like ExifTool and Media Info.


How to make img relative path work?

Hi,
We're using picocms at our office on our intranet. People are using editors such as Retext to write the Markdown and we really need to enable relative paths so that Retext and other editors displays images correctly. We cannot use absolute paths.
I am considering hacking picocms, but since I can see this feature has been requested several times, I think it's much better to enable this using a setting in the configuration.
Before traversing the entire codebase in order to locate the relevant code I want to ask where the code dealing with paths resides and if there is an easy "hack" to make relative paths work when needed?
Kind regards.

Actually there's no code dealing with paths in Markdown files in Pico, all paths (including relative paths) are taken as-is (even %base_url% is just a placeholder that is replaced by Pico's base URL before parsing Markdown). So, if you use a link like ![Cats](assets/cats.jpg), Pico will parse this to <img src="assets/cats.jpg" alt="Cats" />. After adding a appropriate base HTML attribute (e.g. <base href="{{ base_url }}" />) to the <head> of your Twig templates (e.g. themes/my_theme/index.twig), relative paths like the one shown above work just fine.
The problem is that ReText doesn't know anything about this and interprets paths relative to the current file. So, for example, if there's a ![Cats](assets/cats.jpg) in content/sub/page.md, what file is supposed to be included? It should be assets/cats.jpg, however, since ReText treats all paths relative to the current file, it thinks the path is content/sub/assets/cats.jpg. This won't work for obvious reasons, in ReText you'd need a path like ../../assets/cats.jpg, resulting in content/sub/../../assets/cats.jpg (equals assets/cats.jpg).
You could write a simple Parsedown extension (hook into Parsedown::inlineLink()) to change relative URLs like ../../assets/cats.jpg to %base_url%/assets/cats.jpg.
Try something like the following (untested code):
class ParsedownRelativePicoPlugin extends AbstractPicoPlugin
{
    public function onParsedownRegistered(Parsedown &$parsedown)
    {
        $config = $this->getPico()->getConfig();

        $parsedown = new ParsedownRelative();
        $parsedown->setPico($this->getPico());
        $parsedown->setBreaksEnabled((bool) $config['content_config']['breaks']);
        $parsedown->setMarkupEscaped((bool) $config['content_config']['escape']);
        $parsedown->setUrlsLinked((bool) $config['content_config']['auto_urls']);
    }
}

class ParsedownRelative extends ParsedownExtra
{
    protected $pico;

    public function setPico(Pico $pico)
    {
        $this->pico = $pico;
    }

    public function getPico()
    {
        return $this->pico;
    }

    protected function inlineLink($excerpt)
    {
        $link = parent::inlineLink($excerpt);
        if ($link === null) {
            return;
        }

        $href = $link['element']['attributes']['href'];
        if (($href[0] !== '/') && !preg_match('#^[A-Za-z][A-Za-z0-9+\-.]*://#', $href)) {
            $basePath = $this->getPico()->getRequestFile();
            $targetPath = $this->normalizePath(dirname($basePath) . '/' . $href);

            $rootDir = $this->getPico()->getRootDir();
            $rootDirLength = strlen($rootDir);
            if (substr($targetPath, 0, $rootDirLength) === $rootDir) {
                $baseUrl = $this->getPico()->getBaseUrl();
                $link['element']['attributes']['href'] = $baseUrl . substr($targetPath, $rootDirLength);
            }
        }

        return $link;
    }

    protected function normalizePath($path)
    {
        $path = str_replace('\\', '/', $path);
        $pathParts = explode('/', $path);

        $result = array();
        foreach ($pathParts as $pathPart) {
            if (($pathPart === '') || ($pathPart === '.')) {
                continue;
            } elseif ($pathPart === '..') {
                array_pop($result);
                continue;
            }
            $result[] = $pathPart;
        }

        return implode('/', $result);
    }
}

Hello PhrozenByte, i am trying to understand what you wrote there. Basically what you described is exactly what i am looking to do, replace ../../assets/cats.jpg with either %assets_url%/cats.jpg or %base_url%/assets/cats.jpg if that'd make more sense. Unluckily your description of a solution is very unclear to me, as to where and how i would have to play with the piece of code you wrote there.
What i did was "create" a plugin called ParsedownRelativePicoPlugin with the code you wrote. The image URLs in the html still look like ../../assets/cats.jpg. Now i am not sure if it's something to do with the code (i can't really understand it), or if i wasn't meant to make a plugin out of that code but something else. Could you please elaborate a bit more on what i may be doing wrong?


MJX Documentation API Update

It looks like part of the API documentation may either be outdated or contain a typo. The API call mjx.put_data has two required arguments for both model and data as seen here.
Some sections of the documentation don't reflect this and instead suggest one argument. For beginners starting with this API this may cause some initial confusion (although its very minor as its straight forward to check the function signature):

Caveat: I am a beginner in using mujoco-mjx so apologies if I missed something.

Added doc update here as it was quick to fix.


BlockQuote while condition is incorrect, scoping doesn't match preview or GitHub

Copying comment from another repository: https://github.com/microsoft/vscode-textmate/issues/110#issuecomment-524756619
Example:
> one nest.
>> second nest, new paragraph.
> continues paragraph of second nest.
also continues paragraph of second nest.

Actually (via GitHub):

one nest.

second nest, new paragraph.
continues paragraph of second nest.
also continues paragraph of second nest.

VS Code Editor highlight:

From VS Code Preview:

I believe VS Code's Markdown grammar is wrong.  The while rule is expecting to find additional lines starting with > but that is not how the previewer works.  I believe the correct test is, at a minimum, a non-blank line. but possibly also tests for other kinds of markup, or precisely the same as the while clause of list_paragraph.    This is the correct while condition for blockquote: "while": "(^|\\G)([ ]{0,3}(>) ?|(?!\\s*$|#|[ ]{0,3}([-*_>][ ]{2,}){3,}[ \\t]*$\\n?|[ ]{0,3}[*+->]|[ ]{0,3}[0-9]+\\.))".  With this change, I get the correct scoping.

I'll try to post a PR shortly.   As I went through the spec for blockquotes, I noticed where the REGEX still failed some situations, and it became more complicated to correct, but I think I have it.


Fix for horizontal scrollbar

You could see an horizontal scrollbar all the time. That's now fixed
changing the list item width to 90% instead of 100%, otherwise the
margin-left activates the overflow-x.

Hi @bitkris-dev,
Thanks for the contribution.
Could you just do 2 things?

Add a before/after preview of this because i have an OSX and ain't seeing this issue (maybe Win/Linux probably have this).
Add yourself in the .all-contributorsrc :haircut_man:

Thanks and well spotted!

Sure, no problem! That's the screenshot:

@bitkris-dev can you do me a favor?

@jccguimaraes Sure, just ask! :)

can you checkout branch 1.0.0 and see if this issue still occurs?
Keep in mind a lot is changing :).
It won't mess with your current db file but, for the sake of it, just backup it.
Would appreciate it :+1:

Unfortunately, the branch 1.0.0 doesn't work at all. (In fact, I have also noticed that Travis CI wasn't successfull on the build).
Those are the errors appearing:

you need to npm install :)

yeah, but maybe the css files will brake due to being in win :(

Also all can't create anything right now through the plugin... you need to copy the spec/mocks/project-viewer.jsonto wherever the atom.getConfigDirPath() gives you...

Ops, sorry! I'm new to atom package development.
I've correctly installed the new 1.0.0 version, but the first notification said "No local database found". So it isn't able to detect previous database and projects. I've tried to open other settings, clicking on "Import from Gist" or any other menu item, nothing happens. I don't know why it looks like broken on my machine.
Uninstalling it and reinstalling the official apm version restored my previous projects. So I can confirm you it doesn't screw up the database, it's only unable to read the previous one.

yeah, was too slow on giving you instructions :( read my comments above. I will push some changes in a minute for ease

Ok, I've followed your instructions. Now this is what appears to me:

As you can see, no more horizontal scrollbar! For the rest, it's the same, all the menu items don't work. Also, right-clicking the fake projects doesn't do anything, I can't edit them.
Trying to open them gives me this error:

But I suppose it's just a matter of wrong directory.
If you have other requests, feel free to contact me again whenever you want. Glad to help :)

hey @bitkris-dev,
sorry for late replying, was feeling sick.
Well, it is supposed not to work mostly anything :) i am in the process of a major refactor in the editor viewer. You can check the progress by Menu -> Project Viewer -> Open editor. But it is only UI for now. (that error is normal, but fixed also)
Regarding the version, it is intended that way, because when publishing in Atom, it will bump the version up!

It makes sense :) nice!


Animate snowflakes only if tab/browser is active

Solution http://stackoverflow.com/questions/1760250/how-to-tell-if-browser-tab-is-active

http://caniuse.com/#feat=pagevisibility

@liepa great!


Not getting results ordered by Relevance

Hi!
I'm searching for Dealership names where all names begin with the brand name (eg. Mazda -X-).
If I search with -X- I get the results I want, filtered just by that Dealership.

But if I search by "Mazda -X-", of course I get all the results because it matches all the Dealerships with "Mazda" in their name, but I would think that "Mazda -X-" has to be the top result, but as you can see in the screenshot, it's nowhere to be seen in the first page.

This happens also in other kind of searches on other fields like user names and last names, or in joined tables (eg. searching for dealership names from the salesmen table).
I've tried playing with the field weights with no positive results.
Any help would be appreciated! :)

Have you managed to resolve tihs?


Stuck at 225 No transfer to ABOR

Hi,
This extension really helps me in daily-tasks but I'm starting to get an error like this one:
[06-12 12:37:05] [info] [file-save] c:\xxxxx\style-async.scss.tpl [06-12 12:37:05] [trace] handle upload file for c:\xxxxx\style-async.scss.tpl [06-12 12:37:05] [debug] > PASV [06-12 12:37:05] [debug] < '227 Entering Passive Mode (3,229,160,81,4,114).\r\n' [06-12 12:37:05] [debug] > LIST /static [06-12 12:37:05] [debug] < '150 Here comes the directory listing.\r\n' [06-12 12:37:06] [debug] < '226 Directory send OK.\r\n' [06-12 12:37:06] [debug] > PASV [06-12 12:37:06] [debug] < '227 Entering Passive Mode (3,229,160,81,4,69).\r\n' [06-12 12:37:08] [debug] > ABOR [06-12 12:37:08] [debug] < '225 No transfer to ABOR.\r\n' [06-12 12:39:57] [debug] < '421 Timeout.\r\n' [06-12 12:40:54] [debug] < '421 Timeout.\r\n' [06-12 12:42:08] [debug] < '421 Timeout.\r\n'
The problem is that, it stuck in this 225 erro messagem then after some minutes start to receive a 421 but the extension in the console shows like still running.
I need to close the entire VS Code, open again and do a save to upload but it keeps repeating this and nothing happens.
Could you please help me?

Just an update on this, when it get stuck in this ABOR stuff, the timeout won't work anymore and the plugin is stuck until I restart VS code.


[TBDGen] The non-deallocating destructor applies to some @objc classes.

The non-deallocating destructor doesn't exists when dealloc can be overriden,
which means any class that inherits from a class defined in Objective-C. This
isn't necessarily all @objc classes, because of the
-disable-objc-attr-requires-foundation-module flag.
Fixes rdar://problem/40542246.

@swift-ci please smoke test


Remove Session from domain layer

Session is clearly Infrastructure responsibility. Domain doesn't care about it so it shouldn't be there.

Pull Request Test Coverage Report for Build 326

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.009%) to 95.387%

Totals

Change from base Build 323:
0.009%

Covered Lines:
517

Relevant Lines:
542

💛  - Coveralls


Fix clippy warnings

Fix clippy warnings: iter_nth_zero.
Defining next() without nth(0) will be little bit redundant. Therefore, it's better to allow it in this case.

This fix the clippy warnings below:
warning: called `.nth(0)` on a `std::iter::Iterator`
   --> src/tiling/plane_region.rs:526:5
    |
526 |     self.nth(0)
    |     ^^^^^^^^^^^ help: try calling: `self.next()`

warning: called `.nth(0)` on a `std::iter::Iterator`
   --> src/tiling/plane_region.rs:560:5
    |
560 |     self.nth(0)
    |     ^^^^^^^^^^^ help: try calling: `self.next()`


Is it possible to enable ARCore Depth API ?

Is it possible to enable ARCore Depth API in this plugin?  I have enable the ARCore Cloud Anchor API for my app and it work fine. but it would be fantastic if we can use the ARCore Depth API feature. by the way , i hope you can add the delete selected node function in your example just like the onRemoveEverything function.

hey @raysonlim77 ,
unfortunately the version of Sceneform used in this plugin does not support the Depth API feature. I suggest migrating to the community-supported Sceneform at some point (see https://github.com/CariusLars/ar_flutter_plugin/issues/16 ) as this would enable us to use Depth API. Is this something you might want to look into?

Yes,That's exactly what im looking for, but i have no idea how to migrate it, Im looking forward to see this feature available in this plugin. anyway i truly appreciate your kind suggestion.


Estimate the head orientation from my webcam through mediapipe 468 landmarks

Please make sure that this is a solution issue.
System information (Please provide as much relevant information as possible)

Have I written custom code (as opposed to using a stock example script provided in Mediapipe):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04, Android 11, iOS 14.4):
MediaPipe version:
Bazel version:
Solution (e.g. FaceMesh, Pose, Holistic):
Programming Language and version ( e.g. C++, Python, Java):

Describe the expected behavior:
Standalone code you may have used to try to get what you need :
If there is a problem, provide a reproducible test case that is the bare minimum necessary to generate the problem. If possible, please share a link to Colab/repo link /any notebook:
Other info / Complete Logs :
Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached:

Face Geometry API should help you with that however @kostyaby could suggest a better solutions since he is the main author of face estimation ML pipeline.

Hey @showbit01,
As @ablavatski, [Face Geometry API] is indeed what can help for your use-case. MediaPipe JS API now provides this kind of support, please take a look:

https://www.npmjs.com/package/@mediapipe/face_mesh
https://www.jsdelivr.com/package/npm/@mediapipe/face_mesh
index.d.ts; what you are interested is the "FaceGeometry" part of the API


Add state change callback

Callback gets called every time the test changes state
Limit upload measurements count by config value measurementPointsTimespan

Thanks for the pull request!
However, could you please fix:
Commit 17b72de2
The new "onStateChange" is great, however it will be called multiple times for each state change since each "thread" sets the state in its "onStateEnter". Please add a mechanism such that the stateChange event is only fired once for each state transition.
Commit 567d012
I get the need for less measurement points. However, my fear is that measurement points can be lost in certain situations (e.g. the point before the end point of the measurement). Could you please test all edge cases?
Just asking: The maximum of these measurement points is also effectively limited by the chunk size and the RMBTServer code. Did you see a huge reduction with your code?

State change
Isn't _change test specific? As such it would only get called once, when the state actually changes

Measurement points: I investigated this because I was getting a couple thousand points during upload.
You mean the final "TIME x BYTES y"? Well, its not exactly lost, it depends on the resolution. Hmm..how about saving the last received measurement and adding it upon the "TIME x" (if not already added)?

Sorry about the states, you're right, _state already handles this.
Thank you for the observations concerning upload measurement points!


feat: merge solhint master branch

I did it in local and will push the branch with the solved conflicts.


New logo proposal

I thought I would give it a try and make something more into the current hype.
Included SVG derived from the original logo.

Before:

After:

A little too sharp on the edges for my taste. Could you make it a bit rounded and see what that looks like? :)

#812 moved the title to the right. Another tweak you could try is to make the logo as wide as the navigation.

@AndrejMitrovic Erm, let me see another day. Spent hours already. I don't really like rounded edges everywhere like on the current site, but it makes sense to have subtle ones like on the menu. Can't guarantee it will look better.
@CyberShadow Makes sense. Maybe we can display the SVG directly then (the PNG is exactly the SVG render). But going to be now.

I appreciate the work you've put in, but I'd like to keep the current logo:

logo consistency is very important for branding. We've done a good job associating D with the current logo
I like the enameled look of the current one
I like the current rounded edges, the subtle variation in color, the soft edges of the D part, the D fills the logo, etc.

logo consistency is very important for branding

A lot of companies (possibly all major ones) tweak their logo over time while keeping the core theme.
See for yourself: https://www.google.com/search?tbm=isch&q=logo+evolution

I know some do. But not that quickly, and they launch new logos with marketing campaigns.

BTW, please use PNG for screenshots.

BDFL-induced closing.

The comments of this week's This Week in D seem to centralize on the need of a new logo:
https://www.reddit.com/r/programming/comments/2tozgb/this_week_in_d_website_makeover_c_interop_import/
@WalterBright I suggest that you reconsider.

There's no rationale given, either here or on reddit or on the other PR for a logo change. Changing an established logo is not something to be taken lightly. Let's not spend resources on an illusion of progress.

What's the rationale of all the work we've been doing this week on changing the website design?

Wow I didn't want to launch such a heated debate, it was really just a proposal.
I'm fully OK with the logo staying as it is: branding is indeed important and I like BDFL decisions.
I just wanted to try another thing and eventually make more material for D-related stuff like "d-idioms" or "This Week In D" where the current logo looks odd.
I will use this logo for "d-idioms" not because I think it looks better (such a statement I can't make, for example the original logo has lovely reflections) but because my logo will integrate better in the surroundings on this site.

I do appreciate the effort you put into helping out! Thank you

We should make a couple of proposal and the post to reddit and ask on opinions whether the logo should change. Then it's practically automatically accepted as the new logo (just like the recent @changes).

While I don't personally like the redesign suggested above (or maybe it just lacks a few finishing touches and detail here and there), I think it may be a move in the right direction since the current logo is a bit tacky/90's-style and may seem completely oddball when slapped on an otherwise polished website with a design that tries to stay modern. Redesigning it is then a matter of making it more compatible with other content -- that should be the primary reason and not just doing it for the sake of it.

logo consistency is very important for branding

I agree; but while this is true in general, the subtle design details may have to be changed in order to keep in touch with the latest trends. E.g., Microsoft, Google and Apple:

http://gizmodo.com/the-evolution-of-googles-iconic-logo-1582297667
http://blogs.windows.com/bloggingwindows/2012/02/17/redesigning-the-windows-logo/
http://blogs.tribune.com.pk/story/19792/the-evolution-of-the-apple-logo/

the current logo is a bit tacky/90's-style and may seem completely oddball when slapped on an otherwise polished website with a design that tries to stay modern. Redesigning it is then a matter of making it more compatible with other content -- that should be the primary reason and not just doing it for the sake of it.

Exactly my point, that's why new logo proposal keep the "D + planet" shape.


Can you crop animated GIFs using this library?

Can you crop animated GIFs using this library. It seems to convert the GIF image to PNG while cropping. Is there some way or some option to crop an animated GIF while keeping the animation using this library?

No, this is nog possible.
Only jpg and png.

Thank you. When I try to crop an animated gif using this library, I see a "base image". I am wondering what frame of the GIF does the library pick up for the "base image"?