Overflow issue

Describe the bug
🔴When we are on Mobile view, there is overflow which creates a scroll and a unexpected strip is visible
.....
🔴 You can also work on the theme button you can align it properly , coz according to ux rule it's not an appropriate place for this button

@deepakydv9315 is this open can I try?

I tried something is this ok?

That's great brother but GitHub text is not aligned properly. It will be
best to align it
On Thu, 6 Oct, 2022, 3:06 pm Ebenezer, @.***> wrote:

I tried something is this ok?
[image: Screenshot 2022-10-06 150508]
https://user-images.githubusercontent.com/108586141/194279569-3dc763d9-46da-4209-8028-bc33b52d8c20.png
[image: Screenshot 2022-10-06 150446]
https://user-images.githubusercontent.com/108586141/194279570-beb1f06a-e352-4510-9116-ee0c75fdfba9.png
—
Reply to this email directly, view it on GitHub
https://github.com/Vishal-raj-1/Awesome-JavaScript-Projects/issues/1928#issuecomment-1269688594,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASIPOFT6M7IUP457KSKSRVLWB2MRRANCNFSM5TLOW4EA
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

ok np on that

How about this?

That's great ✨✨
On Thu, 6 Oct, 2022, 3:28 pm Ebenezer, @.***> wrote:

[image: image]
https://user-images.githubusercontent.com/108586141/194284514-1966b4cb-0660-441d-b1d0-95bc0a81326c.png
How about this?
—
Reply to this email directly, view it on GitHub
https://github.com/Vishal-raj-1/Awesome-JavaScript-Projects/issues/1928#issuecomment-1269724212,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASIPOFSRP6IPJJ2D2LRHVX3WB2PE3ANCNFSM5TLOW4EA
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

so should i place a pr?

im not able to do a pr
it says Vishal-raj-1:main and ebenezer0311:new are entirely different commit histories.


Upgrade to Souffle v2.3 from v2.1.

External part of #601

Yes, no (additional) harm in becoming further out of sync with internal Souffle. But ideally, we should update internal Souffle to also be version 2.3.


Python < 3.8 support?

I would like to know why the Python Version requirement is 3.8.
I tested it locally (by changing the things to 3.7 or perhaps lower) and it works.
Unless a local runner test for this already done. Thanks!

I may have forgotten the reason why I set the requirement to 3.8 but I think the reason why is that it was the current version of Python I have installed at the time I started this repo. And I was too lazy to check on older Python versions. I can lower the required Python version to 3.7, or lower, if needed

So far as I find out that this code is "partially compatible with 3.7 or lower".
on maiconverter/__init__.py there's an import notation that is can be checked by version difference, the importlib.metadata which is a 3.8 feature, but the backports exists for 3.7 or lower as importlib_metadata. Otherwise I could make use it fully on 3.7 itself.

I see. So to make it compatible with older versions I could change the import in __init__.py to something like this:
try:
    from importlib.metadata import version, PackageNotFoundError
except ImportError:
    from importlib_metadata import version, PackageNotFoundError

change python_requires to ">=3" insetup.py
and add this to setup.py
extras_require={
        ':python_version < "3.8"': [
            "importlib-metadata"
        ]
    }

Let me know if I miss something

sounds fine to me, tested with installation from 3.6 and lower as well...
it sounds like the setuptools-scm (v6) only supports py3.7 and above as it uses __future__.annotation feature.

That means that only python versions 3.7 and above can build this package and doesn't affect older python versions from installing from a wheel file, right?

I just looked at how to make future-annotations work and on second thought I'm not keen on adding support for it. I'd need to add an encoding cookie at the start of every file that uses annotations. Even then the backport only goes until python 3.6.1. Considering that Python 3.6 is already EOL, I'm not motivated to add support for it.
So I'll only add support until Python 3.7 which is still actively supported.

there's no need, unless you are ok with downgrading or limiting the setuptools-scm version. :+1:

version 0.14.6 should add support to python 3.7. If I messed up, kindly reopen this issue. Thanks again for your time


Support for UBWCTP10

Hello, is there way you could add support for files with extension .UBWCTP10 (Universal Bandwidth Compression) sensor raw dump that is used by Qualcomm Snapdragon phones?
It briefly mentioned on qualcomm development forum:

app/sw code cannot access the UBWC format data directly, there is no public information for the UBWC format.
ofcourse you can still use glReadPixels() to read the UBWC format framework, driver will perform the conversion from  UBWC data to non-UBWC data when reutrn to app.

From what i could found about it:

UBWCTP10 is a YUV-format image data format, a block-based compression format, also known as a 10-bit compression format. It is a compression format for image and video data in resource-constrained application scenarios such as mobile devices. UBWCTP10 can compress image and video data into a smaller space, reduce bandwidth and storage requirements, and provide higher image quality and lower power consumption. However, the processing speed of UBWCTP10 may be slower than the uncompressed format because the compressed data needs to be decompressed.

Reference file could be found here https://www.upload.ee/files/17184486/UBWCTP10.zip.html
File contents.

Hello, this seems to be a proprietary format, without any specification or at least some public information about it. Thus without knowing how to decode the file and how the compression is done it would be very difficult to add support for it. Maybe with your work if you have access to Qualcomm developer tools you can find some way to convert it to another format (maybe even open a ticket to Qualcomm directly), but I have never encountered these UBWCTP10 files before so I am afraid I will not be able to help you more than this.


Is it possible to allow the list view and deleting images on all sites

Is it possible to allow the list view and not only the thumbnail grid view on all sites?

I haven't yet found a way to achieve this cleanly. But it would be preferable, yes.

I tried a bit but it dosen’t work. I disabled this function
add_filter( 'wp_prepare_attachment_for_js', function( array $response, \WP_Post $attachment, $meta ) : array {
	if ( is_media_site() ) {
		return $response;
	}

	// Prevent media from being deleted from any site other than the network media library site.
	// This is needed in order to prevent incorrect posts from being deleted on the local site.
	unset( $response['nonces']['delete'] );

	return $response;
}, 0, 3 );

the I tried to delete an attachment. But the delete dosen’t work the attachment is still in the library…


Fix mx25um51345g flash size on RT685 and RT595

Fix the flash size to be in megabits on the RT685 and RT595 EVK, as required by the jedec,jesd216 binding. Introduce an extension to the size functions in kconfigfunctions.py to convert a node size between bits and bytes, so the CONFIG_FLASH_SIZE symbol can be calculated correctly.
Fixes #50394

@DerekSnell FYI


Visualizer: Fix of updating zoomState when input/grammar changes

zoomState needs to be cleared when input/grammar changed

LGTM, but after the demo we should think about whether there is a better (or more general) way to do this.


Update index.md

Remove extra line in Sources

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi @maulikrathod
It seems that similar changes have already been accepted earlier for this article you're editing, sorry about that. 😓
If you feel you have more to add, please feel free to open up a new PR.
Thanks again! 😊

If you have any questions, feel free to reach out through Gitter or by commenting below. 💬


Make electee stopped electee page moar betterz

Namely own boot electees from future sign ups if they just now unsigned up. Also block those who aren't electing from signing up for members only events

Only, not own


Button V2: Remove outline for hover, active, focus

Button V2: Remove outline for hover, active, focus

This update adds CSS to nullify the outline styles which may occur during
integration.
These outline styles are typically defined within reset/normalize
styles. Adding outline: none directly to the Button helps ensure that outline
removal is prioritized.
https://github.com/helpscout/hsds-react/issues/559

Pull Request Test Coverage Report for Build 1932

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

Totals

Change from base Build 1930:
0.0%

Covered Lines:
7391

Relevant Lines:
7391

💛  - Coveralls


Graphic bug with display of blocks

Unexpected canvas size: the canvas is only a quarter of the screen (for screen with dimensions around 560x300).

[x] : Find the bug
[ ] : Find its origin
[ ] : Solve it

It looks like the canvas has its original size (300x150).
These lines don't seem to work:
https://github.com/Wixonic/WebCraft/blob/1a5b73e34af9dcf4963601542e11862324735f93/Game/scripts/display.js#L22-L28

Update: The problem seems to come from the THREEPerspectiveCamera and not from the size of the canvas.

Update: The problem doesn't seem to come from the THREEPerspectiveCamera.


make library more modular and reusable

Currently, most of the methods do their job well, but can't be reused. There is also the issue with overlap in the functionality. For example, both inside the likelihood and the risk methods, it computes the prior over the hidden states, given the spread parameters.
To resolve this, large parts of the code base should be broken apart and refactored into smaller and more general methods. A consequence of this would also be that large methods like the risk and the likelihood are easier to understand on an abstract level and the smaller methods they are made up of can be tested more effectively.

Recently, I needed to write a risk prediction in the LyProX app. There, too, I needed to write a lot of redundant stuff.
For example, the function create_diagnose_probs in the LyProX' new risk predictor should be something I can use this library for. But since it's not well modularized, I needed to write that from scratch.
Similarly - or even more so - for the function create_marginalisation: This is routinely computed in the lymph library, but in a way that makes it somewhat hard to reuse from the outside. And therefore, I had to write that function again as well.


JavaScript

JavaScript
User story
As a bootcamp student
I want the prework notes to be structured on a webpage
So I can easily find and read the information
Acceptance Criteria
GIVEN a prework study guide website
WHEN I view the study guide
THEN I can see the four topics I learned listed along with a suggestion on what I should study first

merged javascript notes


Add AspNet component.

Please answer these questions before submitting pull request

Why submit this pull request?

[ ] Bug fix

[x] New feature provided

[ ] Improve performance

Related issues

Bug fix

Bug description.

How to fix?

New feature or improvement

Describe the details and related test reports.

Coverage increased (+0.004%) to 23.947% when pulling c54d75eed2db3b4e61dbfb3a53fa250916a8cd2b on liuhaoyang:feature/component into e36de08ee207b54d2f4ea1c9a7c80fae652c21a3 on apache:master.


Console Application Should Wait for Input

Instead of our existing infinite loop, we should put the IntervalTriggeredMessages on their own thread, leaving the console application available to respond to user input. As part of this, we should set up an "exit" command to turn the bot off instead of requiring Ctrl + c as we do now.

i will try to do this :)


Update to leaflet-routing-machine 2.5.0

Closes #127 (total distance is shown with fixed precision by default), also includes some minor performance tweaks that are noticable for very long routes and/or slow machines.

BTW, I'm not sure if you prefer PRs and commits to include the updated bundle*files - I left them out to make diffs easier to read.


Never assume that the kind of a type variable is ‘type’

I was thinking this could show up some mistakes with kind instantiation.

Seems like I was right! We get the following test failures:
---- function_app stdout ----
	thread 'function_app' panicked at 'assertion failed: `(left == right)` (left: `Ok(App(Builtin(Function), [Generic(Generic { id: 0x103c0c0b0:a0, kind: Hole }), Generic(Generic { id: 0x103c0c088:a1, kind: Hole })]))`, right: `Ok(App(Builtin(Function), [Generic(Generic { id: 0x103c0c0b0:a0, kind: Type }), Generic(Generic { id: 0x103c0c088:a1, kind: Type })]))`)', check/tests/completion.rs:111

---- types_should_be_fully_instantiated_even_on_errors stdout ----
	thread 'types_should_be_fully_instantiated_even_on_errors' panicked at 'assertion failed: `(left == right)` (left: `App(Builtin(Function), [Generic(Generic { id: 0x110c5d108:a0, kind: Hole }), Generic(Generic { id: 0x110c5d108:a0, kind: Hole })])`, right: `App(Builtin(Function), [Generic(Generic { id: 0x110c5d108:a0, kind: Type }), Generic(Generic { id: 0x110c5d108:a0, kind: Type })])`)', check/tests/pass.rs:758

Any ideas?

I switched back to using Kind::typ() in unify_type to see if things passed. It got past the gluon_check tests, but then I get:
   Compiling gluon v0.2.2 (file:///Users/brendan/dev/rust/gluon)
error: environment variable `OUT_DIR` not defined
  --> build.rs:42:39
   |
42 |         let out_file_name = Path::new(env!("OUT_DIR")).join(file);
   |                                       ^^^^^^^^^^^^^^^

Dunno if this is just for the latest nightly...

The OUT_DIR could be related to rust-lang/cargo#3368

Created a PR that fixes the environment variable problem: gluon-lang/gluon#228

Hmm, I'm guessing we would not want to have kind polymorphism, so we would have to default to using Kind::Type?

I'm just wondering - if we don't allow for kind polymorphism, should we event have Kind::Variable?

Created a PR that fixes the environment variable problem: gluon-lang/gluon#228

Ran into that problem as well but should have actually pushed the fix upstream :).

I'm just wondering - if we don't allow for kind polymorphism, should we event have Kind::Variable?

As long as we have kind inference Kind::Variable will be needed and I don't think dropping that would be a good idea since all type variables would need to be explicitly annotated with kinds.

Hmm, I'm guessing we would not want to have kind polymorphism, so we would have to default to using Kind::Type when the kind can't be inferred in the context? For example in the function_app test:

Yeah Kind::Type seems like the sanest default (Haskell does it as well). Wouldn't mind having kind polymorphism though as long as it does not mess up anything else in unification.

Where do you think the defaulting should happen?

I guess I just would prefer not to overwrite concrete kinds once they have been set.

After finishing checking a group of type definitions is probably the sanest way of doing it. For type signatures I don't think defaulting can happen since all type variables will appear in some type which only has concrete kinds.

I guess I just would prefer not to overwrite concrete kinds once they have been set.

Does this happen somewhere today?

I'm guessing we don, because as I mentioned in https://github.com/gluon-lang/gluon/pull/227#discussion_r92938697 the kind vars are created defaulting to Kind::Type...


Fix performance regression of get_json_object due to unnecessary stream synchronization

After a stream sync added in https://github.com/NVIDIA/spark-rapids-jni/pull/2266, the performance of get_json_object surprisingly drops by more than 10%. This PR removes that stream sync by combining the error check values together and only synchronizing stream just once.

build

build

build

build

I just ran performance and correctness tests and it looks good. If anything it is slightly faster than before. I want to rerun with the latest merge changes you put in too. But that should only be a few more mins

build

build


Switching to TensorFlow style guide

We should consider switching to the TensorFlow style guide.  It's essentially the same as Google's style guide for Python, with a few exceptions.  The rest of the TensorFlow ecosystem already follows this guide for their Python code.  Here's an incomplete list of some things that would change:

PEP8 linting with pylint instead of flake8 (they provide a specific pylintrc file to use for checking code quality that we can also use)
Files with unit tests live inside the modules they test (#410)
Switching the interpreted tab length from 4 spaces to 2 spaces
Line length reduced to 79 from 99 (made easier by the reduction in tab length)

Someone should be assigned this and own it if we decide to proceed.  I'd be willing to volunteer if nobody finds fixing linter errors particularly engaging :D

Not immediately a fan of the latter two but think we should do this to get into the habit


<EMAIL_ADDRESS>Fix homepage, add EOL note

Fixes homepage

Closes #14346

[x] I have read the Contributing Guide.

/verify


Question about multiple Lambda code bundles and cold boot times

At the moment this plugin creates a single code bundle with all of the handler functions as separate files within e.g. my-app.zip with homepage.js and product.js in the zip.
However, I thought one of the main reasons that Next.js created a Lambda per function was so that Lambda code bundle size could be kept to a minimum thus improving cold boot times? I.e. might have expected my-app-homepage.zip, my-app-product.zip etc. instead?

The lambda, at runtime only requires a single page bundle, regardless of what's inside the deployment artifact. I think this is what makes most of the difference compared to the more traditional monolith approach which at runtime serves all the pages.
However, you're right there could be separate deployment artifacts per Lambda each with the separate page bundles, which in theory should improve the time involved in downloading the artifact from the Lambda.
I'm going to close this in favour of a feature request
https://github.com/danielcondemarin/serverless-nextjs-plugin/issues/49.
Thanks!

Yes, I saw that only a single require was used per page - I can see how this should have a big impact. Instinctively it feels like the size of each bundle should also affect cold start but interestingly this article suggests that it does not https://read.acloud.guru/does-coding-language-memory-or-package-size-affect-cold-starts-of-aws-lambda-a15e26d12c76
Thanks for raising the feature 👍

Yeah read the same article! I do however feel like the individual artefacts make more sense, especially considering it's easy to implement and shouldn't affect anything else, so will have a look when I get a chance 👍

BTW - just to follow up on this. Bundle size did indeed make a difference to cold start times when I tested this:

How do you know is bundle size and not the overhead added by express?  I expect the bootstrap times for express to be higher.

Yeah that's fair but I think that should be reflected in the difference in hot start time I.e. slower by ms not seconds. I  will try large single page bundle to verify though.

Okay you're right - it does look like Express cold boot is what accounts for most of the slowdown:

Bundle size does still seem to make a small difference - but I was only able to do a quick experiment (5 cold + 5 hot starts in each env).

@danielcondemarin I'm starting to wonder whether it actually might be better to keep a single bundle? Also, now not sure what benefits of Next.js "serverless" approach is i.e. self-contained page-level JS files. I can only imagine is slows down builds (imagine 50x page-level JS files, each of which has same dependencies baked in e.g. React), means more Lambdas to manage (using Terraform / Serverless etc. - and potential CloudFormation resource limit issues from latter) vs. just having single bundle, single Lambda + v. lightweight routing (not Express).

Next serverless approach means horizontally scalable pages. For example, doesn't matter how much your bundle size grows you'll never hit a Lambda size limit. The same applies to the Lamba memory, with one lambda for everything you can only scale that vertically.
The management aspect you could argue that serverless is doing that for you. I would ask what is it difficult to manage?
In terms of the CF limits you're absolutely right, that's an issue. At some point I hope Amazon increases the limit or serverless introduces some workaround for it but seems like that might take a while.

Yeah, I want to believe in the above but I can't imagine practically hitting bundle size limits for lambda with single app (at least for projects I'm thinking of).
I'm terms of scaling only certain pages over others I'm not really sure how this results in better performance tbh (can't imagine it really affects number of cold boots).
This plugin is great for managing deployment (although it may not be an option for us).
I think there are some arguments about isolation and with that better security and perhaps the potential to deploy only changed Lambdas? The speed improvements at the moment may come from having single code by bundle vs. requiring into node_modules.
Anyway, thanks for your time - I should take this question to the Next.js forum.

@willbamford Thought to let you know about the new next component release. I believe it addresses some of your concerns above :)


[stable/fluentd-elasticsearch] change fluentd :scheme from http to https

Version of Helm and Kubernetes: v2.11.0 and v1.10.3
Which chart: [stable/fluentd-elasticsearch]
What happened:
I'm using the AWS Elasticsearch service, and I have installed the chart with this command:
helm install --name fluentd-elasticsearch --set elasticsearch.host=es.region.es.amazonaws.com,elasticsearch.port=443 stable/fluentd-elasticsearch

But I'm getting this error message in the logs:
2018-12-17 01:12:14 +0000 [warn]: [elasticsearch] got unrecoverable error in primary and no secondary error_class=Fluent::Plugin::ElasticsearchOutput::ConnectionFailure error="Can not reach Elasticsearch cluster ({:host=>\"es.region.es.amazonaws.com\", :port=>443, :scheme=>\"http\"})!"

Is there a way to change the default :scheme from http to https?
I've also tested the connection between the pods and our Elasticsearch cluster and everything is working good.
Thanks!
@monotek

Update: I've manually edited the Config Map to add the option: :scheme https and it worked!
2018-12-17 01:37:07 +0000 [info]: [elasticsearch] Connection opened to Elasticsearch cluster => {:host=>"es.region.es.amazonaws.com", :port=>443, :scheme=>"https"}
2018-12-17 01:37:07 +0000 [info]: [elasticsearch] Detected ES 6.x: ES 7.x will only accept `_doc` in type_name.
2018-12-17 01:37:07 +0000 [warn]: [elasticsearch] Detected ES 6.x or above and enabled insecure security: You might have to specify `ssl_version TLSv1_2` in configuration.

I didn't figured out yet how to make this changes permanently. 😅
Should I be worried about this warn message: You might have to specify ssl_version TLSv1_2 in configuration?

Hey @rauny-brandao,
to make it a bit easier i've just added the setting to the following pr: https://github.com/helm/charts/pull/9382


refactor: improve mobile responsiveness

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


added UI changes to sidebar items and removed unused image assets

Addressed Issue #18 in this PR
Changes made:

Added necessary UI highlights to indicated the selected sidebar items
Switched from image icons to react-icons
Removed unused icons from asset folder

Great Pr brother .  Looking forward to see more contribution like this . !


Let Project & Settings inherit more shared methods

Expected Behavior
For now, we have to write like this
public class ExamplePlugin : org.gradle.api.Plugin<ExtensionAware> {
    override fun apply(target: ExtensionAware) {
        when (target) {
            is Project -> target.gradle.registerExampleServiceIfAbsent()
            is Settings -> target.gradle.registerExampleServiceIfAbsent()
        }
    }

    private fun Gradle.registerExampleServiceIfAbsent(): Provider<ExampleService> =
        sharedServices.registerIfAbsent(
            name = ExampleService.NAME,
            implementationType = ExampleService::class,
            configureAction = {},
        )
}

Provide an GradleAware interface, something like this
// In org.gradle.api.Project
public interface Project extends Comparable<Project>, ExtensionAware, PluginAware, GradleAware

// In org.gradle.api.initialization.Settings
public interface Settings extends PluginAware, ExtensionAware, GradleAware

public interface GradleAware {
    Gradle getGradle();
}

Then it is more convenient to achieve this goal.
public class ExamplePlugin2 : org.gradle.api.Plugin<ExtensionAware> {
    override fun apply(target: ExtensionAware) {
        when (target) {
            is GradleAware -> target.gradle.registerExampleServiceIfAbsent()
        }
        when (target) {
            is Project -> // Do something for project
            is Settings ->  // Do something for settings
        }
    }
}

If we go further, we can do something like this
// TODO Maybe find a better name
public interface CommonAware extends PluginAware, ExtensionAware, GradleAware {
    BuildLayout getLayout();

    ProviderFactory getProviders();

    File getRootDir();
}

// In org.gradle.api.Project
public interface Project extends Comparable<Project>, CommonAware

// In org.gradle.api.initialization.Settings
public interface Settings extends CommonAware

Current Behavior (optional)
No response
Context
It has been explained above.

Thank you for your interest in Gradle!
This issue needs a decision from the team responsible for that area. They have been informed. Response time may vary.

In most cases, in particular, with Gradle, you can use @Inject. We do need to better document what is available for injection (this is tracked at #27427). For many other use cases, like copy or exec, there are dedicated services which are also injectable. Because of this, we are unlikely to invest more in this area.
Of course, please comment here or file a new issue if a new use case not covered by injection arises.

Hi @mlopatkin, I just facing a new case about reading a file from both Settings & Project from rootDir. I also need ProviderFactory.gradleProperty() to retrieve some value.

}

I also need to call this function not just twice because my project has lots of composite build projects. It would be nice if we have the "CommonAware" like above. So that, I can write something like:
fun CommonAware.resolveXXX(): XXX {
    rootDir.resolve("xxx")
    providers.gradleProperty("xxxxx").get()
    ...
}


[Doc](flink connector)Add new configuration be nodes

Proposed changes
Issue Number: close #xxx

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

run buildall


snapshot engine: fix #416 (remove the hardcoding) and #433 (add url in metadata)

This is addressing the little bit of leftover work from the snapshotEngine merge:
snapshotEngine should be totally separate from xtz-shots since anyone can deploy their own. So, all the xtz-shots specific stuff should be hosted on the xtz-shots-website repo, and the snapshotEngine should grab whatever it needs to build the static page.
This is done in this PR. We no longer have a _config.yml in the tezos-k8s repository: we now grab it from xtz-shots-website and append the remote theme config to it with a heredoc.... not super slick but it does the job.
The advantage is that we no longer have to modify config.yml here whenever there is some change in the website metadata (for example, the list of networks on the top right of the page used to say "hangzhounet" despite us having removed it from xtz-shots-website. no longer).
The only leftover references to xtz-shots website are the commented out values in values.yaml (because of course, anyone who would want to deploy a snapshot website should still look into ours to understand how it works)
The only leftover files are Gemfile and Gemfile.lock which I think is ok.
I am also adding full URL to the metadata, this way operators can simply 1. query the metadata 2. parse the url and and 3. download the snapshot

I'm assuming this is tested works?

Now yes. It's deployed on ithaca-shots. Here is the new metadata:
nochem@fedora ~/workspace/pulumi-polkadot/charts/plenty-bridge () $ curl -Ls https://ithacanet.xtz-shots.io/rolling-tarball-metadata
{
  "block_hash": "BLiT1qiY15DVM3p23KKNDGrQH3aMS5VdDVipY9dDXjk3QUkUz8Z",
  "block_height": "492980",
  "block_timestamp": "2022-05-06T02:29:30Z",
  "filename": "tezos-ithacanet-rolling-tarball-492980.lz4",
  "url": "https://ithacanet.xtz-shots.io/tezos-ithacanet-rolling-tarball-492980.lz4",
  "sha256": "911222c63243baadca77900df1af650c9bde7a25ffdbb86f518de51fdc67ea53",
  "filesize_bytes": "92478089",
  "filesize": "88M",
  "tezos_version": "6e2037c9 (2022-04-07 12:44:15 +0200) (12.3)",
  "chain_name": "ithacanet",
  "history_mode": "rolling",
  "artifact_type": "tarball"
}

Do we want to make deploying a website optional? That might be too much work right now. I'm not sure.
A user may not want a custom domain for their site and might just want to use a raw url to the bucket. Do we want to handle this now?

No, let's defer until we are actually trying to deploy without a website. Thanks for approval.


React Snippets › Settings: Import React On Top not working for ts variants

Similar to #211 but specifically for ts snippets.

@wKovacs64 released in v4.3.0


Can't execue lein on windows

Describe the bug
I put the lein.bat under "C:/Users/david/bin" and add the path in enviroments.
But the plug in can't execute it. And I get
Cannot run program "lein" (in directory "C:\Users\david\source\stock"): CreateProcess error=2

Environment (please complete the following information):
IntelliJ IDEA 2024.1.4 (Ultimate Edition)
Build #IU-241.18034.62, built on June 21, 2024
Licensed to Tianpo Gao
You have a perpetual fallback license for this version.
Subscription is active until June 13, 2025.
Runtime version: 17.0.11+1-b1207.24 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Windows 11.0
GC: G1 Young Generation, G1 Old Generation
Memory: 4096M
Cores: 8
Registry:
ide.experimental.ui=true
Non-Bundled Plugins:
com.github.clojure-repl (1.4.1)
com.github.clojure-lsp (2.6.0)
Kotlin: 241.18034.62-IJ

Hello @DavidAlphaFox,
Can you please add more information about the error? When it happens? Do it have the stacktrace?

hum, maybe for windows we should run it as lein.bat 🤔

@DavidAlphaFox could you confirm that if you run lein.bat repl in your terminal it works?

@DavidAlphaFox I made some adjustments that I know are required for windows, could you install this plugin please?
I think it might fix #84 as well
clojure-repl-1.4.2.zip

I'm pretty sure it fixes windows support, LMK otherwise

@ericdallo
I tried the 1.5.0 on my windows machine. It still gives me the same error.
By the way I use Windows Home version.
IntelliJ IDEA 2024.1.6 (Ultimate Edition)
Build #IU-241.19072.14, built on August 8, 2024
Licensed to Tianpo Gao
You have a perpetual fallback license for this version.
Subscription is active until June 13, 2025.
Runtime version: 17.0.11+1-b1207.30 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Windows 11.0
GC: G1 Young Generation, G1 Old Generation
Memory: 4096M
Cores: 8
Registry:
  ide.experimental.ui=true
Non-Bundled Plugins:
  com.github.clojure-repl (1.5.0)
  com.github.clojure-lsp (2.6.1)
Kotlin: 241.19072.14-IJ


Complete P2P implementation

Will this project achieve complete tendermint P2P functionality in the future (which is completely corresponding to the tendermint P2P in the Go language version)? Currently, it seems only implements the function of secret connection, but there are still many things that are missing

@brapse @tarcieri @xla @hdevalence hello guys, cc. Can we discuss it?

Unlikely, given it isn't a priority.

@xla. Currently, if Tendermint Rust only implements the type without implementing a node, it will be meaningless. Currently, only the Tendermint P2P in the Go language is fully implemented. To be honest, Tendermint-Rust has a lot of gaps to fill.

It's not meaningless, we built a whole blockchain using it. The crate provides common data types for modeling Tendermint/CometBFT data, and that's very valuable.


More examples of training

Hi. I liked your work. I think it's wise to add, for example, training a small model on Shakespeare's dataset or TinyStories?

@Ttimofeyka yes I agree. I was actually looking for mini-datasets that could even serve as additional unit-tests. I don't know both of these datasets, but can have a look. Do you have any further references of what to train on those?

@Ttimofeyka yes I agree. I was actually looking for mini-datasets that could even serve as additional unit-tests. I don't know both of these datasets, but can have a look. Do you have any further references of what to train on those?

Sure.
https://huggingface.co/datasets/flpelerin/tinystories-100k (as the smallest example)
https://huggingface.co/datasets/JeanKaddour/minipile (good choice for GPT-2 equivalent)
https://huggingface.co/datasets/cchoi1022/openwebtext_20 (as many implementations of GPT-2)

Thanks, I did start developing a NLP model demo using stacked barebone stacked mingrus  here. https://github.com/cheind/mingru/blob/nlp-gpt-like-demo/mini_nlp.py
For tiny Shakespeare, this already produces quite satisfactory text (as far as I can tell) when trained from scratch with tokenizer based on gpt2.

Thanks, I did start developing a NLP model demo using stacked barebone stacked mingrus here: https://github.com/cheind/mingru/blob/nlp-gpt-like-demo/mini_nlp.py For tiny Shakespeare, this starts producing satisfactory texts (as far as I can tell) after 1 epoch, when trained from scratch with tokenizer based on gpt2.

Good! Try to put the most difficult part of the code in a separate file inside a mingru folder, 'cause code seems a bit of crap now (all things messed).
Also try to check https://github.com/KellerJordan/modded-nanogpt - this is a more fast version of NanoGPT project, may be useful.

Thanks your suggestions. Text generation uses a barbone stacked MinGRU using GTP-2 vocab-size, I'm not planning to extend this demo any further.


Can improve the hover effect of the buttons

We can make the hover effect of buttons more attractive.

Can you assign this issue to me,I wanna work on this


[New] all Scenario methods to show figures accept controll arguments of figures

Summary of this new feature
All methods of Scenario class to show figures, including .history() and .simulate(), are expected to accept controll  arguments of cs.line_plot() function. e.g. ylim=(0, None) to set y range.
(Optional) Solution
Add **kwargs to Scenario.history() etc. so that we can controll cs.line_plot() function internally called.

This is related to #503.

This is related to #503.

This is related to #495.

This is related to #495.


Fixes #162

The regex wasn't matching the line. I think Atom use to includes newlines on to the beginning of text buffers, this is no longer the case. This fixes the Wrap Lines command.
Ref #162
cc @MoritzKn
Edit: Actually, looks like due to Atom's reporting system, #162 is actually an accumulation of multiple errors, all with the same error message Uncaught TypeError: Cannot read property '1' of null - this PR will resolve the issues regarding the wrap lines function, but not all reported in that thread. The issue should remain open.

Thanks for your PR! You're right #162 is a real mess. I tried to sort it out and added a comment which should give an overview.
Could you please explain how to reproduce the error you are fixing with this?

@MoritzKn Yeah, sorry for not including the repro.

Add the following code to a JavaScript text editor. (This works in other languages as well)

/**
 * This is a horribly long comment that should wrap because I will have to scroll horizontally to see the entire thing because it is so long
 */

Have your Preferred Line Length in Atom settings be around 80, which is the default. Esentially, enough for the sample text to be longer than the line length.
Select anywhere in the sentence
Run the Docblockr: Wrap Lines command
Will throw Cannot read property '1' of null which can be traced back to the change made

I went through the stack trace, this line causes the error. I noticed that the text variable in that scope didn't have a leading line break, which caused the match on line 583 to fail.
As per the cause of this, I think Atom use to include a newline when getting buffer regions, this is no longer the case.

Also, was just searching for any other lingering issues on this matter, found #28 which actually implemented this same fix, but wasn't merged because it included another odd function - not sure what value it adds over Wrap Lines at face value.

Ok, thank you! Good research, I appreciate that. :smile:

Released with version v0.8.7.

Thanks for the timely merge @MoritzKn!


some Tooltips for elements in HEDA Create Contact function properly, many do not.

Many elements in the HEDA Create Contact screen have tooltip icons. Some of them like Work Phone and Work Email function correctly.  Others like Preferred Phone and Preferred Email do not. (Note that the Tooltip for Work Email says "See Preferred Email", but the Tooltip for Preferred Email does nothing.)
Many examples of both working and non-working Tooltips exist on this page.
seen in Firefox 50.1 on OSX

Here is a correct tool tip:

This is all controlled by the platform and the standard lightning experience. I have searched for a platform bug for this and cannot find one. This is out of our control. This can be closed.


sometimes click event doesnt fire?

Hi
I am using the following bit of code to replicate a "material design" click wave event on a drop down / button.
The problem is that sometimes my nya-bootstrap select does not update the button text - the value in the background is update correctly (ID value) however the actual button text is not. When i remove the code below it works perfectly.
Any idea where the clash could be?
 if ($('body').hasClass('page-md')) {
        // Material design click effect
        // credit where credit's due; http://thecodeplayer.com/walkthrough/ripple-click-effect-google-material-design
        var element, circle, d, x, y;
        $('body').on('click', 'a.btn, button.btn, input.btn, label.btn', function(e) {
            element = $(this);

            if(element.find(".md-click-circle").length == 0) {
                element.prepend("");
            }

            circle = element.find(".md-click-circle");
            circle.removeClass("md-click-animate");

            if(!circle.height() && !circle.width()) {
                d = Math.max(element.outerWidth(), element.outerHeight());
                circle.css({height: d, width: d});
            }

            x = e.pageX - element.offset().left - circle.width()/2;
            y = e.pageY - element.offset().top - circle.height()/2;

            circle.css({top: y+'px', left: x+'px'}).addClass("md-click-animate");

            setTimeout(function() {
                circle.remove();
            }, 1000);
        });
    }

I use dropdownToggle.children().eq(0) to query the element inside button for updating button text. You prepend an element inside the button will make it fail to change the text

Thanks for the update. now i understand... do you have any suggestions?

I just wonder why you want use this bootstrap style components in your 'material design' app. A material design select component should be a preferred.

Its a material design theme for bootstrap so there is a ripple effect.
I find that the material design angular version is too premature and is missing a lot of basic components.

I think rewrite DOM transverse using native querySelector and querySelectorAll may solve this


Add Morgan Lovett to Contributors list

Added my name to the list on Constributers.md

@mrlovett, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added Facebook share and tweet feature to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


Update test Java versions

Is your feature request related to a problem? Please describe.

Java lifecycle updates.
Describe the solution you'd like

We should stop testing 14 which was EOL in Sept and start testing 16 in preparation for the availability of LTS 17 in Sept.
We should also update the prereq statement to be Java 8 or 11 as those are the LTS versions.
Describe alternatives you've considered

Additional context

17 is available as the LTS now, so now this work should change to result in:
Supported versions: 8, 11 (17 when the core supports it)
Travis test versions: 8, 11, (17 if we can resolve powermock issues)

As mentioned in the orignal PR related to this ticket it is possible to workaround the PowerMock issue by using --illegal-access=warn (the default in Java 9-15) since in Java 16 the default changed to --illegal-access=deny. The --illegal-access switch has been removed entirely in Java 17 so that workaround of switching deny back to warn is no longer possible.
The workaround is to instruct the JVM to export (and open) the necessary packages from the java.base module to the unnamed test modules e.g.
<surefire.options>--add-exports java.base/java.lang=ALL-UNNAMED --add-exports java.base/java.util=ALL-UNNAMED --add-opens java.base/java.lang=ALL-UNNAMED --add-opens java.base/java.util=ALL-UNNAMED</surefire.options>

Using that workaround instead should allow the tests to pass using any Java 9+ without needing to globally allow illegal access while we wait for https://github.com/powermock/powermock/issues/1094
For more info on this see JEP 261 breaking encapsulation.

Tried out, this JVM setting indeed worked. So I've updated the content of the PR.

Turned out 14 is not LST version and 17 needs further testing to be fully supported version.
Related documentation change: https://github.com/IBM/cloudant-java-sdk/pull/198


fix title angle

I noticed the title had a slight angle and it should not have, so I fixed it. The exported png is now also 400dpi.

Thank you @pdimens! I really appreciate the attention to detail.


Evcxr freezes with no explanation

I must be missing something really basic, but this doesn't work:
use evcxr::*;
fn main() -> Result<(), evcxr::Error>{
    evcxr::runtime_hook();
    let (mut context,_) = EvalContext::new()?;
    context.eval("let s = String::new();")?;
    context.eval(r#"s.push_str("Hello ");"#)?;
    context.eval(r#"s.push_str("World");"#)?;
    context.eval(r#"println!("{}", s);"#)?;

    Ok(())
}

the app just freezes. This is on Rust 1.51 ubuntu linux
specifically it freezes on  EvalContext::new()

It probably is running, but is just really really slowly. Evcxr uses rust-analyzer, which pretty much needs optimization enabled in order to function.
I have the following in my Cargo.toml so that I get optimization even for non-release builds:
[profile.dev]
opt-level = 2

Otherwise, it's worth mentioning that stdout from the evaluation context won't go to your stdout. It goes to the EvalContextOutputs that is the second return value from EvalContext::new(). The following code runs as expected:
use evcxr::*;

fn main() -> Result<(), evcxr::Error>{
    evcxr::runtime_hook();
    let (mut context, outputs) = EvalContext::new()?;
    context.eval("let mut s = String::new();")?;
    context.eval(r#"s.push_str("Hello ");"#)?;
    context.eval(r#"s.push_str("World");"#)?;
    context.eval(r#"println!("{}", s);"#)?;

    // Dropping `context` shuts it down, otherwise the loop that follows will continue to wait in
    // case the context sends more output.
    drop(context);

    while let Ok(line) = outputs.stdout.recv() {
        println!("{}", line);
    }

    Ok(())
}

oh wow, it was just slow,your code worked!


[TypeScript] Missing method tsConfigPath on runner API

What is your Test Scenario?
I'm using a script to programmatically run TestCafe. I'd like to pass in a custom tsconfig.json to the runner. The method is listed in your documentation under runner but it is not in the type definition.
What is the Current behavior?
Missing tsConfigPath in the Runner interface.
What is the Expected behavior?
It would be defined. Something like:
/**
  * The absolute or relative path to the TypeScript configuration file. Relative paths resolve from the current directory (the directory from which you run TestCafe).
  */
tsConfigPath(path: string): this;

Steps to Reproduce:
You can look at the Runner interface here: https://github.com/DevExpress/testcafe/blob/master/ts-defs-src/runner-api/runner-api.d.ts#L48
Your Environment details:

testcafe version:                   v1.6.0 (latest)

I'm willing to fix this. I would love to contribute to TestCafe.


How to use it

Hello, I have successfully compiled a DLL file in Visual Studio2022, but when I inject it into the game, the injector shows successful injection, but there is no response in the game. How should I use this compiled DLL file？

把编译好的dll发给我我教你

Okay, I already have a compiled DLL file. How can I contact you via email? Telegram? Discord? Or what other contact information?


[Feature Request] change readonly tooltip label "cannot edit read-only editor" when used "createDiffEditor"

Context

[X] This issue is not a bug report. (please use a different template for reporting a bug)
[X] This issue is not a duplicate of an existing issue. (please use the search to find existing issues)

Description
when i use 'createDiffEditor' && 'readonly' to compared two code diff, "cannot edit read-only editor" will show if i input something. I want to change the tip

With https://github.com/microsoft/vscode/commit/052f02175f4752c36024c18cfbca4e13403e10c3, you can just skip the readOnlyMessage contribution and provide your own.
This will be available in the next monaco editor release.


Sort lables by description info if it exists, otherwise sort by name

For example, we can let the description start with a number (1#Java, 2#Docker, 3#K8s, etc.)

Thanks


Failed to connect-PnPOnline with UseWebLogin

Unhandled Exception: System.IO.FileNotFoundException: Could not load file or assembly 'System.Windows.Forms, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089'. The system cannot find the file specified.
at OfficeDevPnP.Core.AuthenticationManager.<>c__DisplayClass22_0.b__0()
at System.Threading.Thread.ThreadMain_ThreadStart()
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
Win 10
PowerShell Core

@officedocsbot assign @yogkumgit

Use Windows PowerShell. PnP modules require .NET Framework components that are not available in .NET Core (hence PoSh Core).

@tseward  Thank you very much for the contribution and sharing this explanation. @azMantas
Hope this comment is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Add Minify CSS task

New minify_css task should be added to ImageTransformationMixin
Filestack docs can be found here
This task takes two arguments: level and gzip.
It should be implemented similarly to resize, crop or rotate.
Expected usage:
filelink.minify_css()
filelink.minify_css(level=1, gzip=False)

Tests are required.
For reference, please see PR https://github.com/filestack/filestack-python/pull/38

Is the filelink a Client object? Can the link to the css file that needs to be minified, be outside of filestack.com, or should it belong to this domain exclusively?

@KIRA009 Filelink and Client objects are not the same.
Link to css file does not have to be a filestack.com url, something like this will work:
https://cdn.filestackcontent.com/API_KEY/minify_css/https://stackpath.bootstrapcdn.com/bootstrap/4.3.1/css/bootstrap.css

Please see how other tasks are implemented.


[Migrate actions] Create view team action

Description
As a maintainer, I need to query a GitHub team, so that I can do things like exclude Core team members from the contributors annotations and determine if a PR is a community PR.
### Acceptance criteria
- [ ] I can call this action to get a list of members on the Core Group GitHub team

Impact to other teams
This impacts Adapters and Core since they are the primary consumers of these actions.
Proposals

named github/team/view
inputs team
outputs members

We have a PR with these contents already.


How to use this parser on other languages

I want to use this Parser with Chinese Dataset, and try to use mbart(https://github.com/pytorch/fairseq/tree/main/examples/mbart). But the difference of data preprocess and encode/decode makes it hard to use this parser with Chinese. How can I do that?

+1

we will release a structured-mbart branch next week, check https://twitter.com/RamonAstudill12/ for updates

@WDZRMPCBIT @xxllp see https://github.com/IBM/transition-amr-parser/tree/structured-mbart

I am using the code from your README to parse EN sentences:
https://github.com/IBM/transition-amr-parser?tab=readme-ov-file#parse-with-a-pretrained-model
Is there a straightforward way to adapt it for parsing the other available languages using one of your pre-trained models?


refactor: use memo for modern utils

I'm not aware that the original pr https://github.com/nuxt/nuxt.js/pull/6953 has been merge.
Address review from @pi0 and @pimlie

Use local variable to implement memo
Avoid calling require for every request

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@b9e5256). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             dev    #6954   +/-   ##
======================================
  Coverage       ?   62.76%
======================================
  Files          ?       82
  Lines          ?     3309
  Branches       ?      900
======================================
  Hits           ?     2077
  Misses         ?      989
  Partials       ?      243

Flag
Coverage Δ

#unittests
62.76% <ø> (?)

Impacted Files
Coverage Δ

packages/utils/src/modern.js
100% <ø> (ø)

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 b9e5256...46d4867. Read the comment docs.

@clarkdo Functionality wise this is working, but all the refactoring doesnt help with the fact that I just dont want any semver code to be included in my lambda build. These changes still includes the semver code in my lambda build so that code still needs to be parsed (ie its just not executed atm). So at the moment I can still only achieve that by stubbing the semver package.

As I mentioned in previous pr, semver will be executed when you’re using modern mode.
In server side, we won’t bundle node modules into server dist, so if you don’t even want include semver package, I think you can add it into exclude in your lambda building config.


Update staramr to version 1.0.0 in Galaxy

Update staramr to version 1.0.0 in Galaxy (when released).

Dear @apetkau ,
When the v1.0.0 will be released, I can help you to update the galaxy wrapper.

Thanks so much @pimarin . There are still a few small issues to address in staramr before updating the Galaxy wrapper, but we are getting close.

Pretty cool ! don't hesitate !


Project Testing

Would be nice to have more info about testing the projects on different browsers/devices. How to use debug on devices, etc.

Yeah! :+1:  This could be a pretty big topic when you get into remote debugging for android etc. We could have it in a different repo or a different markdown file.


Update copyrights to use "Paperclip Contributors"

Addresses issue #390
Updates the name of the copyright holder in each of the licenses to "Paperclip Contributors"
I wasn't sure if we wanted to change the dates to match? Or if there was a reason the dates on the two licenses were different, so I left that the same. Please let me know if there are any other changes that should be made. This is my first time contributing to this project, so please also let me know if there are any conventions I should have followed that I didn't.  :)

Thanks @drueck !
If I had to guess when the apache license was copied Ravi forgot to update it.
Let's update the date as well if you don't mind :)

Thanks @drueck ! If I had to guess when the apache license was copied Ravi forgot to update it. Let's update the date as well if you don't mind :)

Haha, yeah, that makes sense. Updated so they're both 2019.

Thanks @drueck ! If I had to guess when the apache license was copied Ravi forgot to update it. Let's update the date as well if you don't mind :)

Haha, yeah, that makes sense. Updated so they're both 2019.

Sorry I meant to 2022

Sorry I meant to 2022

My bad. I wasn't sure what date was supposed to be used, so I googled it, and I misinterpreted what I read. It said something like that it should be the year the copyright was applied, which I took to mean the first year that the copyright was applied, but after further reading, I discovered that it means the year that it is currently applied, which I guess is the most recent year of changes/releases. 😅  So yeah, 2022 makes sense, and also you asked me to do that, so either way, updated.  Sorry for so much back and forth on a tiny change. But thanks for the opportunity to contribute. I'll be taking a look at other code issues hopefully this weekend to see if there's something more useful I can contribute as well.

Sorry I meant to 2022

My bad. I wasn't sure what date was supposed to be used, so I googled it, and I misinterpreted what I read. It said something like that it should be the year the copyright was applied, which I took to mean the first year that the copyright was applied, but after further reading, I discovered that it means the year that it is currently applied, which I guess is the most recent year of changes/releases. sweat_smile So yeah, 2022 makes sense, and also you asked me to do that, so either way, updated. Sorry for so much back and forth on a tiny change. But thanks for the opportunity to contribute. I'll be taking a look at other code issues hopefully this weekend to see if there's something more useful I can contribute as well.

No worries at all, I also wasn't entirely sure what the correct behaviour should be, though other projects seem to have also understood it like this.
Thank you!


Is it possible to get execution time of each queries

I log queries with setQueryCallback() nut we only have the queries, is it possible to get the query time execution?
perhaps in the $params argument?

Hm no, you can always wrap with your own timing functions to measure execution time of method calls on LessQL objects

To do this, one would:

Add queryEndCallback in a subclass of LessQL\Database.
Find usages of onQuery in LessQL\Database.
Overrule those methods in the subclass.

https://gist.github.com/rubenvincenten/ef8aaa32a0e413592c8df74ba7ae910e
This should really be implemented in the v0 branch however, tagging it like v0.4? I'm assuming the next branch will be 1.x.

Something like the debug panel of Zebra Database would be amazing!


Getalltags

all api endpoints now include all tags with any snippets returned

Great job, Tang! Keep up the amazing work!


2nd row of cards overlays first

The Halvar, God of Battle should be shown.

2nd row offset just needs adjusted

tested on site, all good!  :)


Thought/Guidance for Encapsulating and Exposing Components in NgModules

First of all thank you for exposing this repo for vetting Angular-based library projects. Based on this configuration, I have been able to successfully author, test and publish a few assets for use within my organization. Although you state in the README that this is for simple libraries, what is the evolving thought/guidance for encapsulating and exposing various component types in NgModules, i.e. ChartsModule, MultiMediaModule, etc. Thanks!

Heya, I have to mention that that this quickstart isn't finalized yet. It may not be fully compatible with Angular.
You should be able to export whatever you want though, it's not limited in that sense. Encapsulating and exposing components in NgModules follows the same logic as it would inside your application.
https://angular.io/docs/ts/latest/guide/ngmodule.html would be the current available guidance. Do you feel like it should be expanded with library specific information? Off the top of my head I don't see any specific caveat.

I think adding a few exposed ngModule types could be of benefit:

Service module as Provider and when/when not to use "forRoot()" vs "forChild()" in root application vs. lazy-loading config, respectively
UI module that demos wrapping a few pure components with entryComponents list config

As you know, there are Pipes, Directives, Validators, Animations, etc. that could also be NgModule encapsulated examples but these 2 types examples cover what, IMHO, must library developers seek to get started with.
This is a work-in-progress for me also and I hope to shift my organizations front-end developers to pivot into library contributors rather than non-resuable, one-shot component authors. I'll be more than happy to contribute over the next weeks.

I have a use case where modularizing at multiple levels is throwing the following exception:
`> tsc -p src/demo/
src/demo/app/app.component.ts(2,10): error TS2305: Module '"/Users/x201413/Projects/pru-lib-ng3/src/lib/index"' has no exported member 'TestComponent'.
src/demo/app/app.component.ts(3,10): error TS2305: Module '"/Users/x201413/Projects/pru-lib-ng3/src/lib/index"' has no exported member 'ComponentsModule'.`
Here is my TestComponent.ts
`import { Component } from '@angular/core';
@Component({
moduleId: module.id,
selector: 'test-component',
templateUrl: './test.component.html',
styleUrls: ['./test.component.css']
})
export class TestComponent {
name = 'Angular Library';
}`
The test.component.html

Hello {{name}}

A components.module.ts
`import { NgModule } from '@angular/core';
import { CommonModule } from '@angular/common';
import { TestComponent } from './test/test.component';
@NgModule({
imports: [
CommonModule
],
declarations: [
TestComponent
],
entryComponents: [
TestComponent
]
})
export class ComponentsModule { }`
The lib.module.ts
`import { NgModule } from '@angular/core';
import { CommonModule } from '@angular/common';
import { ComponentsModule } from './components/components.module';
@NgModule({
imports: [
CommonModule,
AnimationsModule,
AttributesModule,
ComponentsModule,
PipesModule,
ServicesModule,
ValidatorsModule
],
exports: [
AnimationsModule,
AttributesModule,
ComponentsModule,
PipesModule,
ServicesModule,
ValidatorsModule
],
declarations: []
})
export class LibModule {}`
And finally, index.ts
export { LibModule } from './src/lib.module';
Did I miss something?

@filipesilva

I have to mention that that this quickstart isn't finalized yet. It may not be fully compatible with Angular.

Since the status of this document says "last update: 2017-04-07, status: implemented" can we assume that the format itself is finalized, but just that this quickstart may not fully implement the final format yet?


CHANGELOG: release RC10

Has the Slicer been checked?

Yup, @smallhive has at least two projects converted now and yeah, there were #481 and #482.

current Slicer  isn't very tasty for the node (previous generic ObjectWriter was more appropriate than current client-like interface), but it works and requires pretty small adoptions


scanNumber: smaller return value + side effects

Make scanNumber return only the SyntaxKind it scanned, and consistently side-effect tokenValue to return the value. Previously, it sometimes set tokenValue, and then returned a two-property object containing both kind and tokenValue.
This might be more efficient for this particular case: returning objects is a pattern I want to prevent spreading in the scanner. So this change is mostly about style.
Notably, scanNumber already side-effects tokenValue in some places, so this change makes the code more consistent.
I noticed this while working on mini-typescript, which is the only time I read the scanner code.

@typescript-bot perf test this


passing props from server to client.

How do i pass my api data from server to client as props and,
is it advisable to pass API fetch data from server to client and re-use it to render component.
Note: not using redux

Yes, it is strongly advised to use prefetched data from the server to "re-use" or "hydrate" the client side application.
Even if you do not use redux, the general concepts from the feature/redux branch still apply.
You want to fetch the critical data to run your app on the server and attach them to the window object. On the client side, you can use this data to populate a component's internal state.
An example could be
<!-- EJS -->
<script>
  window.__TIMER_STATE__ = <%= timerState =>;
</script>

// Timer component
class Timer extends Component {
  constructor() {
    this.state = window.__TIMER_STATE__;
    delete window.__TIMER_STATE__;
  }
}

The timerState would be passed down to the EJS by the server and contains an object with your fetch results.
Let me know, if this helps and/or refer to the redux branch for conceptional insight.

Hey @rherwig thanks, this worked for me, although i have 2 question here.

is it safe to put all data on window object, in some cases this may be really large chunks of data.
is there a way i can hide the data on window object,
what is the use of this delete window.TIMER_STATE;

Glad it worked!

This is considered best practice for hydrating apps with server side data. But you are not wrong: the more data, the longer the page takes to parse. This is why you should only provide the minimum required data for the requested page to work via window.
You could even populate the window object with different data, depending on the page the user requests.

This is done, to avoid cluttering the window object at runtime, since it is generally discouraged to make data globally available via window. This is what we use data stores for. Also: when using hot-reloading, there might be the issue of hydrating your app multiple times with the window data, thus overriding the current app state. Deleting the server-state from the window works around this.

looks good, thanks.


reopen keyboard when tap back on gallery page.

For Android simulator, the file cancel event happens after the visibilitychange and window focus event, but for physical Android, in most cases it happens before them, so the keyboard does not reliably reopen on physical devices.
This compatibility code allows us to reliably open the keyboard always.
Details

Fixed Issues

$ https://github.com/Expensify/App/issues/25872
PROPOSAL: https://github.com/Expensify/App/issues/13922#issuecomment-1693002644
Tests

Open any chat
Focus the composer to open the keyboard
Tap plus icon in the composer, and then click Add Attachment
Select "Files" from Choose an action modal
Tap the back button to exit.

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Open any chat
Focus the composer to open the keyboard
Tap plus icon in the composer, and then click Add Attachment
Select "Files" from Choose an action modal
Tap the back button to exit.

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/8579651/6061687c-11b9-493a-9504-76c869bb9a3a

Mobile Web - Chrome
https://github.com/Expensify/App/assets/8579651/3529f511-16c6-4ccd-ad7a-4439b516f796

Mobile Web - Safari
https://github.com/Expensify/App/assets/8579651/1cdbb56b-e0c2-415c-8151-c0d2f2d81e26

Desktop
https://github.com/Expensify/App/assets/8579651/36618992-c1b8-414e-8de2-111e561e30e3

iOS
https://github.com/Expensify/App/assets/8579651/97de753b-0655-4b2d-a2a7-443a78dd65ec

Android
https://github.com/Expensify/App/assets/8579651/bd802612-d947-4b5b-a83e-07d89676dd8d

I am not sure if we should concern about this inconsistency.
Repro step:

Focus on composer
Tap android device back button
Verify that keyboard is hidden and composer is still highlighted
Tap plus icon in the composer, and then click Add Attachment
Select "Files" from Choose an action modal
Tap the back button to exit
(before fix) keeps Step 3 but (after fix) keyboard shows again

The root cause is clear that Step 3 status is still considered as focused regardless of keyboard visibility.
https://github.com/Expensify/App/assets/108292595/d069e88b-d81c-4ed2-ae4c-ee99779bc7e1

What we're trying to fix in this PR:

Focus on composer
~Tap android device back button~ - this is ignored so still show keyboard
Tap plus icon in the composer, and then click Add Attachment
Select "Files" from Choose an action modal
Tap the back button to exit
(before fix) keyboard doesn't show but (after fix) keyboard shows again. Both cases, composer is still highlighted

So if we don't consider keyboard visibility, this PR causes another inconsistency while fixing one inconsistency.

Hi, @situchan, do you need more inputs? Or can we move forward? 😄

I think we can move forward. No engineer assigned on GH so no way to confirm

@ntdiary please pull main

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/108292595/e97895da-1090-4c3c-ae36-922b023c1366

Mobile Web - Chrome
https://github.com/Expensify/App/assets/108292595/d069e88b-d81c-4ed2-ae4c-ee99779bc7e1

Mobile Web - Safari
https://github.com/Expensify/App/assets/108292595/de9dbf27-622a-47a2-84db-3580b4a9654c

Desktop
https://github.com/Expensify/App/assets/108292595/8dd1b25d-0e4f-4730-9c7e-b2ff35fd032c

iOS
https://github.com/Expensify/App/assets/108292595/87aa6d05-0c01-4fcd-8629-3a6742507587

Android
https://github.com/Expensify/App/assets/108292595/eba70f2e-9460-4df6-be5d-7a83ef10510d

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/aldo-expensify in version: 1.3.69-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 1.3.69-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/aldo-expensify in version: 1.3.70-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


warning: not using recommended Hue bridge api version 1.33.0

Why does this appear every time? Restartet ten times - don't work

Because your Hue bridge is on a different firmware version that what I develop and test against.  I don’t think they’ve posted release notes for 1.34.0 yet, so I don’t know if there’s any breaking changes.
This is just a warning message and doesn’t effect the functioning of homebridge-hue.  What’s not working?

Okay, thanks.
But I can't add my non-homekit plugs to homebridge, which are working on the hue bridge perfectly.
I installed the homebridge-hue plugin and then restartet homebridge, once it told me the first and second part of user (like "001788FFFE123456": "7340HUSBUD878") and then it says that it can't connects to the bridge.
At this part I always get these errors:
bridge request 1...error

Note that these values are case sensitive.  Could you please attach the debug dump file (if any is created) and the debug log file, see https://github.com/ebaauw/homebridge-hue#debug-log-file.

For me the debug log didn't worked, I'll put the Terminal Code instead:
[10/9/2019, 7:02:44 PM] [Hue] Philips hue: Philips BSB002 bridge v1934129020, api v1.34.0
[10/9/2019, 7:02:44 PM] [Hue] Philips hue: warning: not using recommended Hue bridge api version 1.33.0
[10/9/2019, 7:02:44 PM] [Hue] Philips hue: bridge request 2: post / {"devicetype":"homebridge-hue#makesmart-server"}
[10/9/2019, 7:02:44 PM] [Hue] Philips hue: bridge error 101: link button not pressed
[10/9/2019, 7:02:44 PM] [Hue] Philips hue: press link button on the bridge to create a user
[10/9/2019, 7:02:44 PM] [MagicHome-Platform] Unable to connect to bulb at [<IP_ADDRESS>]: [Errno 113] No route to host
[10/9/2019, 7:02:44 PM] [MagicHome-Platform] [ 'Updating Device',
'<IP_ADDRESS>',
{ H: 255, S: 100, L: 50 },
false ]
[10/9/2019, 7:02:49 PM] [Hue] 192...41: bridge request 1: get /config
[10/9/2019, 7:02:49 PM] [Hue] 192...41: bridge communication error ETIMEDOUT on 192...41
[10/9/2019, 7:02:49 PM] [Hue] 192...23: bridge request 1: get /config
[10/9/2019, 7:02:49 PM] [Hue] 192...23: bridge communication error ETIMEDOUT on 192...23
[10/9/2019, 7:02:59 PM] [Hue] Philips hue: bridge request 3: post / {"devicetype":"homebridge-hue#makesmart-server"}
[10/9/2019, 7:02:59 PM] [Hue] Philips hue: bridge error 101: link button not pressed
[10/9/2019, 7:02:59 PM] [Hue] Philips hue: press link button on the bridge to create a user
[10/9/2019, 7:03:09 PM] [Hue] 192...41: bridge request 2: get /config
[10/9/2019, 7:03:09 PM] [Hue] 192...41: bridge communication error ETIMEDOUT on 192...41
[10/9/2019, 7:03:09 PM] [Hue] 192...23: bridge request 2: get /config
[10/9/2019, 7:03:09 PM] [Hue] 192...23: bridge communication error ETIMEDOUT on 192...23
[10/9/2019, 7:03:14 PM] [Hue] Philips hue: created user - please edit config.json and restart homebridge
"platforms": [
{
"platform": "Hue",
"users": {
"ECB5FAFFFE09B82E": "AhiZYuK4gviTIdyKwthjixBs55VNcclaqYYzr9M7"
}
}
]
[10/9/2019, 7:03:14 PM] [Hue] Philips hue: 3 accessories
[10/9/2019, 7:03:29 PM] [Hue] 192...41: bridge request 3: get /config
[10/9/2019, 7:03:29 PM] [Hue] 192...41: bridge communication error ETIMEDOUT on 192...41
[10/9/2019, 7:03:29 PM] [Hue] 192...23: bridge request 3: get /config
[10/9/2019, 7:03:29 PM] [Hue] 192...23: bridge communication error ETIMEDOUT on 192...23
[10/9/2019, 7:03:44 PM] [MagicHome-Platform] [ 'Polling Light', '<IP_ADDRESS>' ]
[10/9/2019, 7:03:44 PM] [MagicHome-Platform] /usr/local/lib/node_modules/homebridge-magichome-platform/src/flux_led.py <IP_ADDRESS> -i
[10/9/2019, 7:03:48 PM] [MagicHome-Platform] Unable to connect to bulb at [<IP_ADDRESS>]: [Errno 113] No route to host
[10/9/2019, 7:03:48 PM] [MagicHome-Platform] [ 'Updating Device',

Homebridge-hue discovered three bridges, but only one of these is reachable.  I think your ISP is sharing “your” public IP address with others, and you’re seeing your neighbours’ bridges.  Best set hosts in config.json to bypass discovery.

Thanks for you response, but if you mean the "bridge request 3" etc error, that also went's till endless (I think, I'm already at 13). And I doubt that there are so many bridges in my neighbourhood. Do you now an alternativ?

Can you please help me? :) I would like to integrate the plugs in Homekit, otherwise I have to send them back :(

Did you set hosts in config.json to the ip address of your Hue bridge?

No I don’t know what you mean, can you please send me the code

You have to add a line to your config.json in the hue plugin containing the host ip. Syntax is explained here: https://github.com/ebaauw/homebridge-hue/wiki/Configuration

Freaking god, thank you so much, with the host it works perfectly, thank you both!


Unset fake PrimaryKey value before (un)hooking relation

Coverage remained the same at 100.0% when pulling a9cf9f516b392363a5b9d34cbcb18286402818db on CyberiaResurrection:UnsetFakePK into 51cb1706de83b5ea46d45efd49c8f4577c470f34 on Algo-Web:master.


document the release process

It looks like we're using an Actions workflow with a handy-dandy tool called goreleaser to facilitate the process of cutting new releases of cog.
Let's update the CONTRIBUTING doc with details about how and when releases are created.

Just walked through this with @bfirsh and cut a new release! 🍐
git fetch --all --tags
git tag v0.0.11
git push --tags

Visit https://github.com/replicate/cog/actions


Increment version for storage releases

Increment package version after release of azure-storage-blob

Will do manually 😄


"Maximum number of submissions" option for exams

Several people have asked for the option to limit the number of times students can submit answers to individual parts.
Each part should show the number of submissions remaining, and be locked after they've all been used.
I'm not personally in favour of this - we'll get lots of emails from students saying they accidentally submitted, for one - but people keep asking for it.

Hi Christian, I am the one who emailed you like 24 hours ago. I am not actually sure how the fact that students often accidentally ending the exam and the idea of making them have only n number of trials conflict with each other though.

My views on this have hardened even further: I don't think it's right to limit the number of submissions, and we shouldn't give this tool to authors who haven't thought through the consequences.


TypeError: Cannot read property 'applications' of undefined

When trying to play a video, I get the following output:
found device "weejewel-chromecast" at <IP_ADDRESS>:8009
found device "weejewel-chromecast" at <IP_ADDRESS>:8009
connected, launching app ...
connected, launching app ...

/Users/weejewel/Desktop/node_modules/castv2-client/lib/controllers/receiver.js:50
    callback(null, response.status.applications || []);
                                  ^
TypeError: Cannot read property 'applications' of undefined
    at /Users/weejewel/Desktop/node_modules/castv2-client/lib/controllers/receiver.js:50:35
    at ReceiverController.onmessage (/Users/weejewel/Desktop/node_modules/castv2-client/lib/controllers/request-response.js:27:7)
    at ReceiverController.emit (events.js:117:20)
    at Channel.onmessage (/Users/weejewel/Desktop/node_modules/castv2-client/lib/controllers/controller.js:16:10)
    at Channel.emit (events.js:98:17)
    at Client.onmessage (/Users/weejewel/Desktop/node_modules/castv2-client/node_modules/castv2/lib/channel.js:23:10)
    at Client.emit (events.js:117:20)
    at PacketStreamWrapper.onpacket (/Users/weejewel/Desktop/node_modules/castv2-client/node_modules/castv2/lib/client.js:79:10)
    at PacketStreamWrapper.emit (events.js:95:17)
    at CleartextStream.<anonymous> (/Users/weejewel/Desktop/node_modules/castv2-client/node_modules/castv2/lib/packet-stream-wrapper.js:28:16)

Do you still get that error? Google has changed some stuff lately.

Yep, still the same error!

I'm wondering why the MDNS scan finds your device twice.

That does not always happen though

Woah, it's working! I guess the chromecast just OTA'd itself! Awesome, thanks.

:D


Lag support draft PR

This is a draft PR for LAG support on MEV-TS from P4-CP module.
This is a WIP PR.
Objective is to support active-backup use case of LAG done via bonding driver. The netlink messages related to LAG are received in the krnlmon switchlink module, which is then processed by swithcsai module and passed on to switchapi module of krnlmon. switchapi module is responsible to making TDI calls to MEV backend and program the relevant P4 tables.

The unit testing for krnlmon LAG changes isn't present in this PR. I am planning to bring-up the CMock infra for krnlmon and write  sample unit tests for the LAG use case. The UT activity will be tracked against this JIRA https://jira.devtools.intel.com/browse/NVFMSA-9329.


No videos shown

No videos shown on http://ip:4444/video/ url
Here is my docker compose:
version: '3'
services:
selenoid:
image: "aerokube/selenoid:1.10.9"
container_name: selenoid
ports:
- "4444:4444"
networks:
- mybridge
volumes:
- ".:/etc/selenoid"
- "./target:/output"
- "/var/run/docker.sock:/var/run/docker.sock"
- "./Results:/opt/selenoid/video"
environment:
- "OVERRIDE_VIDEO_OUTPUT_DIR=./Results"
command: ["-conf", "/etc/selenoid/browsers.json", "-video-output-dir", "/opt/selenoid/video","-container-network", "mybridge", "-limit","2", "-video-recorder-image","selenoid/video-recorder:latest-release"]
selenoid-ui:
image: "aerokube/selenoid-ui:1.10.9"
container_name: selenoid-ui
links:
- selenoid
depends_on:
- selenoid
ports:
- "8083:8080"
networks:
- mybridge
command: ["--selenoid-uri", "http://selenoid:4444"]
Selenoid logs:
[CREATING_CONTAINER] [selenoid/vnc:chrome_112.0]
2023-05-09 16:05:00 2023/05/09 15:05:00 [1] [STARTING_CONTAINER] [selenoid/vnc:chrome_112.0] [5b21e0ead5b1a8437b8aca65c0d30ed7e95a57bc1714665204216631ae09d864]
2023-05-09 16:05:01 2023/05/09 15:05:01 [1] [CONTAINER_STARTED] [selenoid/vnc:chrome_112.0] [5b21e0ead5b1a8437b8aca65c0d30ed7e95a57bc1714665204216631ae09d864] [0.97s]
2023-05-09 16:05:01 2023/05/09 15:05:01 [1] [CREATING_VIDEO_CONTAINER] [selenoid/video-recorder:latest-release]
2023-05-09 16:05:02 2023/05/09 15:05:02 [1] [STARTING_VIDEO_CONTAINER] [selenoid/video-recorder:latest-release]
2023-05-09 16:05:03 2023/05/09 15:05:03 [1] [VIDEO_CONTAINER_STARTED] [selenoid/video-recorder:latest-release]
2023-05-09 16:05:03 2023/05/09 15:05:03 [1] [SERVICE_STARTED] [selenoid/vnc:chrome_112.0] [5b21e0ead5b1a8437b8aca65c0d30ed7e95a57bc1714665204216631ae09d864] [0.02s]
2023-05-09 16:05:03 2023/05/09 15:05:03 [1] [PROXY_TO] [5b21e0ead5b1a8437b8aca65c0d30ed7e95a57bc1714665204216631ae09d864] [http://<IP_ADDRESS>:4444/]
2023-05-09 16:05:03 2023/05/09 15:05:03 [1] [SESSION_ATTEMPTED] [http://<IP_ADDRESS>:4444/] [1]
2023-05-09 16:05:04 2023/05/09 15:05:04 [1] [SESSION_CREATED] [4ca6416c557214e059287eaacc3485e6] [1] [4.43s]
2023-05-09 16:06:06 2023/05/09 15:06:06 [44] [VIDEO_LISTING] [unknown] [<IP_ADDRESS>]
[SHUTTING_DOWN] [5m0s]
2023-05-09 16:06:28 2023/05/09 15:06:28 [1] [STOPPING_VIDEO_CONTAINER] [622e3fdd2c61eff8ce668cabe86336ea7c4ce53921fff19733b99d8f4cb61b63]

@vania-pooh They didn't appear I don't know why

@vania-pooh I tried to use it but still doesn't work and also when I check /opt/selenoid/video in the selenoid container I don't found the video there but in the video container the video is in the /data folder

@Kinzabl then make sure you don't have error messages in Selenoid log related to copying video and no error messages in video recorder container log (e.g. in mp4 format video resolution should be even).

@vania-pooh In the video container I have this message in log : Non-monotonous DTS in output stream 0:1; previous: 230397, current: 204082; changing to 230398. This may result in incorrect timestamps in the output file.

@Ghizlane369 that's is not an error. There should be another message and the end of the output.

@vania-pooh This is the error in selenoid container : [VIDEO_ERROR] [Failed to list directory : open /opt/selenoid/video: no such file or directory]

@Ghizlane369 that is to say volume mount in Docker Compose file is being done incorrectly.

This is my docker compose file:
selenoid:
image: "aerokube/selenoid:1.10.9"
container_name: selenoid
ports:
- "4444:4444"
networks:
- mybridge
volumes:
- ".:/etc/selenoid"
- "./target:/output"
- "/var/run/docker.sock:/var/run/docker.sock"
- "./Results:/opt/selenoid/video"
environment:
- "OVERRIDE_VIDEO_OUTPUT_DIR=./Results"
command: ["-conf", "/etc/selenoid/browsers.json", "-video-output-dir", "/opt/selenoid/video","-container-network", "mybridge", "-limit","2", "-video-recorder-image","selenoid/video-recorder:latest-release"]

@Ghizlane369 try absolute paths, relative paths are not recommended.

@vania-pooh It works thank you but there why this it should for both absolute and relative path

@vania-pooh It works thank you but why it's not recommended to use the relative path?

@Ghizlane369 because you never know what path will be used as base path.

@vania-pooh Thank you for your help.


dynalite + level-sublevel

With level-sublevel 6.6.3 release, tests that use dynalite started failing in our project.
As dynalite version remains unchanged, seems to be level-sublevel issue.
Error text is:
Uncaught TypeError: Cannot convert undefined or null to object
at itemSize (node_modules/dynalite/db/index.js:455:21)
at node_modules/dynalite/db/index.js:833:13
at Lazy.<anonymous> (node_modules/lazy/lazy.js:142:25)
at Lazy.<anonymous> (node_modules/lazy/lazy.js:73:19)
at ReadStream.<anonymous> (node_modules/lazy/lazy.js:50:22)
at emitReadable_ (node_modules/readable-stream/lib/_stream_readable.js:505:10)
at emitReadable (node_modules/readable-stream/lib/_stream_readable.js:499:62)
at onEofChunk (node_modules/readable-stream/lib/_stream_readable.js:487:3)
at readableAddChunk (node_modules/readable-stream/lib/_stream_readable.js:258:5)
at ReadStream.Readable.push (node_modules/readable-stream/lib/_stream_readable.js:245:10)
at node_modules/level-iterator-stream/index.js:29:12
at node_modules/level-sublevel/read-stream.js:14:16
at Immediate.<anonymous> (node_modules/memdown/node_modules/abstract-leveldown/abstract-iterator.js:29:14)
at Immediate.<anonymous> (node_modules/async-listener/glue.js:188:31)

Will add more details as far as I get diff between 6.6.2 / 6.6.3
This the error appeared with this commit: https://github.com/dominictarr/level-sublevel/commit/9d0f164d85b03fcab0f7b3587251d2984a76ce93

We have the same problem.

Unfortunately I don't have enough time to proceed with investigation at the moment. Stream implementation is the cause as far as I understood. No more technical details at the moment

Currently suspect it might be level-iterator-stream v2.x which was added in this patch, which had breaking changes: https://github.com/Level/iterator-stream/blob/master/UPGRADING.md
Still looking into it though

https://github.com/dominictarr/level-sublevel/pull/116#issuecomment-403541860

So dynalite has had a package-lock.json since 1.3.0 – which should have prevented this breakage for most people anyway.
I've gone ahead and locked down the level-sublevel version to 6.6.2 in package.json too, just in case. Will look at moving to subleveldown

I've reverted 6.6.2 in 6.6.5, should be back to normal.