 It says:
Executed 497 of 3181 SUCCESS
A throw new Error() in some tests does not do anything. Is there something I need to configure?

This has something to with my branch. It happens here as well: https://github.com/openlayers/openlayers/pull/11628/checks?check_run_id=1214216334
And it also happens if I remove the new tests. This means that the code changes in src are responsible for tests not running. This does not seem to be the desired behaviour of the test runner.

It does not happen in the main branch locally.

I pushed a revert of the test directory to the branch and it still skips most tests, even in the CI: https://github.com/openlayers/openlayers/runs/1214605057?check_suite_focus=true

This means that the code changes in src are responsible for tests not running. This does not seem to be the desired behaviour of the test runner.

Agreed.

The error apparently comes from some code execution after done is called. I can fix this, but it is really odd that the runner just stops execution and tells that he has succeeded. Maybe I should file this as a bug in karma/karma-runner ?

Can you elaborate? What change makes the issue go away? It is not unusual that code is executed after done() is called, since many things happen asynchronously. I wouldn't be surprised if the issue comes from the changes in the xhr loader, which is asynchronous.

This happens because of the following lines: https://github.com/openlayers/openlayers/blob/fed74c6cf71aab497914d46e43166a78d92e1406/src/ol/featureloader.js#L155-L158
addFeatures triggers the 'addfeature' which then calls done. The call of success triggers the 'featureloadend' event - but not much more.

If the lines are reversed or done is called in a setTimeout all tests run.

This is indeed a strange behavior. Not sure if this is a test runner problem. Let's leave this open for now.

Can you return a promise? Som test suites deal with that.

The problem is that the code that is executed internally in openlayers after the success function somehow makes the test runner crush in a way that it does not even report it as an error.

Ping @simonseyock, is this still an issue?


RDKTV-18719 : WpeFramework crash fix

Reason for change:
HdmiCecSink shared_ptr cross reference in DisplaySettings
plugin is removed.
Test Procedure: None
Risks: Low
Change-Id: I6812715c05daa05a52459c2f3b962dc1a080dc70
Signed-off-by:Anooj Cheriyan<EMAIL_ADDRESS>

New one updated: https://github.com/rdkcentral/rdkservices/pull/3103


Bumps version of jackson-databind lib to tackle CVE-2019-14379

Similar to https://github.com/openzipkin/zipkin-dependencies/pull/145
The version of jackson-databind in use, 2.9.9, is said to be vulnerable. The version <IP_ADDRESS> has fixed this issue. I've picked <IP_ADDRESS>, the latest patch in the minor version. Here's the description of the CVE: https://nvd.nist.gov/vuln/detail/CVE-2019-14379
As I'm new to the project, is there a quick way we can assure this doesn't break core? Otherwise, please let me know how else I can manually integration-test this patch.
Cheers,
Greg

thanks! similar comments to zipkin-dependencies.
Here are the three things that use it

kafka has this change, just not released yet. https://github.com/apache/kafka/pull/7125
I presume the next patch of spring boot has this due to a custom jackson version https://github.com/spring-projects/spring-boot/blob/2.1.x/spring-boot-project/spring-boot-dependencies/pom.xml#L84
armeria doesn't seem to quite yet have this https://github.com/line/armeria/blob/master/dependencies.yml#L20

Can you help raise a ticket for armeria? Meanwhile I can also see if we can remove what may be an accidental indirect dep on spring-boot-starter-json
We can override the version anyway, but we should revert this when our indirect deps resolve at or past this.


Fix display and turbolink

Fix display for bootstrap dropdown
Fix turbolink issue with javascript

Problem :

Using actual version of dev-2..0.0 :

It's OKAY ! We need to turbolinks:load ready page:change ! ! !


bgpv2,doc: Update troubleshooting doc with CiliumBGPClusterConfig status conditions

We introduced a few status conditions in the CiliumBGPClusterConfig for error reporting (#35517, #35527, #35593). Update the troubleshooting guide to leverage them.
bgpv2,doc: Update troubleshooting doc with CiliumBGPClusterConfig status conditions

/test


Request: CreateVisibleSignature Example

Is it possible to provide an example of how to do a visual signature(image) into a signature field and then to sign it with a certificate?

If you have a similar example of want you want to do from the desktop version (preferably from 1.8.x) then it shouldn't be too hard to do.


Filter out easy alerts and updated the rules to k8s_containers & post…

Signed-off-by: Ravi Ranjan<EMAIL_ADDRESS>What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area falco-chart
What this PR does / why we need it:
Bases on our investigations and finding , Contact K8S API Server From Container" should be ignored for container images which are known "controllers".
Which issue(s) this PR fixes:
N/A

Fixes #
Special notes for your reviewer:
This issue consists in filtering a few "obvious" false-positives.
Contact K8S API Server From Container" should be ignored for container images which are known "controllers".
Checklist

[X] Chart Version bumped
[ ] Variables are documented in the README.md
[X] CHANGELOG.md updated

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

e30161e Filter out easy alerts and updated the rules to k8s_containers & postgres_running_wal_e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: raviranjanelastisys
To complete the pull request process, please assign nestorsalceda after the PR has been reviewed.
You can assign the PR to them by writing /assign @nestorsalceda in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

falco/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @raviranjanelastisys! It looks like this is your first PR to falcosecurity/charts 🎉

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

e30161e Filter out easy alerts and updated the rules to k8s_containers & postgres_running_wal_e
a0e46b1 Merge branch 'master' of github.com:elastisys/charts-1

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@raviranjanelastisys: Adding label do-not-merge/contains-merge-commits because PR contains merge commits, which are not allowed in this repository.
Use git rebase to reapply your commits on top of the target branch. Detailed instructions for doing so can be found here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/assign @nestorsalceda

@nestorsalceda Hi , Any update on this PR ?

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1f9c933 fix

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: raviranjanelastisys
To complete the pull request process, please ask for approval from nestorsalceda after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

falco/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

afd1f20 Filter out easy alerts and updated the rules to k8s_containers & postgres_running_wal_e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Hey @raviranjanelastisys
Same as the other PR, rules changes must happen directly in the falco repository. :)
So could you open a PR there, please?
Thank you again for your time :pray:
/close

@leogr: Closed this PR.

In response to this:

Hey @raviranjanelastisys
Same as the other PR, rules changes must happen directly in the falco repository. :)
So could you open a PR there, please?
Thank you again for your time :pray:
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@leogr Thanks !! I will open the PR in the repository suggested by you . thanks

Opened the PR against the correct repos as suggested - https://github.com/falcosecurity/falco/pull/1981


Missing Sqlite3 Implementation

I suspect this is similar to the weird dependency chain that happens in go land when trying to use sqlite in go land, but the I try to import sqlite3 I get the error ModuleNotFoundError: No module named '_sqlite3'. Is it just that the process for loading up sqlite3 uses too much weird C code for it to be properly handled in the standard way? Would it be worthwhile to attempt to recompile the python with this compatibility? This is some great work! So excited to try and use this for some of my projects!

Hi @paradise-runner 👋
Glad to hear that you've found this project useful 🙌
Yeah since sqlite3 has a native component (written in C) it's not straight forward to support it in this runtime. Basically the way to go would be to compile the Python interpreter together with the native part of sqlite3 into one .wasm target.
The downside is that for every dependency (that has a native component) that you might need, you'd need to compile it ahead of time into the one big .wasm target. So that's not ideal.
Also you'd need to verify that sqlite3 can be compiled to wasm in the first place as well.
I'll continue to tinker with this and see if I could provide some nice way of getting native dependencies working, even in some hacky way 👍

I'll close this out then and continue to work with it. Thanks again!


Debug breakpoint extensions

Probably ought to test this interface out in an example and make sure it makes sense before merging :)

Done 🫡

Just a warning from experience: if you move to add breakpoints via IDebugControl::AddBreakpoint make sure to wrap it in a ManuallyDrop, or you'll cause a use-after-free... the docs say the lifetime of a breakpoint is managed by IDebugControl::AddBreakpoint and IDebugControl::RemoveBreakpoint. If you let IDebugBreakpoint drop, it will cause a call to IUnknown::Release and cause UAF. You'll have to implement Drop for your breakpoint type that calls IDebugBreakpoint::GetAdder and then IDebugControl::RemoveBreakpoint if you want automatic cleanups (or store the creating client with the breakpoint).
Not a problem yet, but wanted to drop a note here because I have spent many hours debugging that exact issue when dealing with IDebugBreakpoint 😅


Updated the prerequisite version to 0.35

Upon wrestling with issue #68, it turned out that I do need JupyterLab 0.35.1 to get the new jupyterlab-vim to build locally.
Please finish the following when submitting a pull request:
==> I was simply editing the documentation. Will skip it for now.

[x] Skipped Add a line to History.md briefly documenting the change.

If this is a release, additionally do the following:

[x] Skipped Bump the package version in package.json
[x] Skipped Update the dependencies in package.json
[x] Skipped Run jlpm install to update yarn.lock

Thanks!

Thanks for taking the time and sending a PR with the update!


(815) Expose attributes needed to show completed task details to supplier

Submissions:

updated_at / submitted_at
invoice_total_value / order_total_value
invoiced? (true if has associated SubmissionInvoice)

Submission files:

filename

Closing, as I'm reworking with @tekin.


fix/to use pull_request_target

The PR builds are being executed without Docker login, leading to issues with rate limiting. Since build.yml cannot access the repository secrets with the pull_request event, we need to switch to pull_request_target. This will allow forked PRs to access the repository secrets, enabling the PR builds to pull Docker images within the authenticated user rate limit.
However, using pull_request_target has its drawbacks. Any changes to the workflow could potentially expose the secrets.. Therefore, to mitigate the risk of exposing secrets, we will not build PRs that include changes to the workflow files.
The above solution is interim. An ideal solution would be to check if there are any changes in the workflow files. If there are, we should request approval from a list of reviewers. Once approved, we can proceed to build the PR.
I will be testing this approach on sample repositories to verify the GitHub Actions workflow. Once it is confirmed to be working, I will raise a PR for the implementation.

I saw this and was going to review it, and then saw it was draft. I think it makes sense, although, as you wrote, we will need to be careful. The approach looks good. Now that it is a regular PR, all good.

Any idea why the git fetch is failing? Is it something to do with the the context change due to the changed event to pull_request_target?

@yash-zededa Could you please also add this (+ docker secrets) to eden.yml?

I saw this and was going to review it, and then saw it was draft. I think it makes sense, although, as you wrote, we will need to be careful. The approach looks good. Now that it is a regular PR, all good.

@deitch I don't think so this is because of the context changed. We had the same issue with<PHONE_NUMBER>.
I think the issue is because

we are attempting to fetch changes directly into the branch that is currently checked out, which Git does not allow to avoid potential conflicts and overwrites.

Considering we are only modifying the WF we can ignore for now.

@yash-zededa Could you please also add this (+ docker secrets) to eden.yml?

@milan-zededa  I reviewed the changes @uncleDecart made for the Docker secrets and noticed some inconsistencies with the Eden version. I've updated the version to 0.9.9 and added the parameter secrets: inherit, which should ideally pass the secrets from the calling workflow repository to the called workflow (Eden in our case).
That said, we need to test the workflow. Once the PR is approved, we should be able to see the results.

@yash-zededa Seems that build workflows are not running for this PR (and eden tests depend on them).

Let's this...


2 more questions: image and date

Hello,

Is it possible to show posts image at individual post view?

How can manage date format to fit language configuration? ie: Spanish date is different from English and I supposed is different to Chinese too.

Thanks in advance !

This is possible, but I think adding the post image in the individual post page may interfere with the sight, especially when just entering.

Yep, we should handle the date format in different languages. I'm already pushing a commit 5f15a23 to execute it.
But there have some jobs I need your help.
You can click the commit link above, copy the zh.toml content, create a new file named es.toml (Spanish), and paste. Then replace the weekday and month with Spanish.
Last, follow the summary.html's change, add these codes:
{{ if ... }}
{{ else if eq .Site.Language.Lang "es" }}
  (Write Spanish date format here)
{{ else }}

Related doc: https://gohugo.io/content-management/multilingual/#customize-dates

Can you make some highlights to get it working for our case? Maybe we can put an if "show_covers = true" condition?

I created data folder and put es.toml there. Then I put this in summary:

  {{ if eq .Site.Language.Lang "es" }}
    {{ index .Site.Data.es.Weekday (printf "%d" .Date.Weekday) }}，{{ .Date.Day }} de {{ index .Site.Data.es.Month (printf "%d" .Date.Month) }} de {{ .Date.Year }}
  {{ else }}
    {{ .Date.Format "Monday, Jan 2, 2006" }}
  {{ end }}

but got this error:
Rebuild failed:

Failed to render pages: render of "home" failed: execute of template failed: template: index.html:14:9: executing "main" at <.Render>: error calling Render: failed to execute template ["summary"] v: "/usr/share/blog/themes/dream/layouts/_default/summary.html:21:7": execute of template failed: template: _default/summary.html:21:7: executing "_default/summary.html" at <index .Site.Data.es.Weekday (printf "%d" .Date.Weekday)>: error calling index: index of untyped nil

thanks !! :green_heart:

Ok, I can put a config to show the summary image in the single post page.

Is your idea to put the image here?

Seems that you have weekday missing in the es.toml. You can give this repo a PR that I can help u to finish it,

Yes ! then I also can move it .. but not sure how to link there. Also I'm not sure where to put featured images, now I'm putting at dream/static/img but I think should be outside dream folder.

#131 :green_heart:

You can put your own images in your blog/static/. All static resources' path will relative to this folder.

@lupa18 I'm already finished the question 1. Here is the doc: https://g1eny0ung.site/hugo-theme-dream/#/params-configurations?id=showsummarycoverinpost-true

Thanks !! here the result: https://antivirus.tedic.org/ ;)

Looks great!


Don't import backports in Python 3

System information

Arch Linux (up-to-date)
TensorFlow installed from binary (with pacman, Arch package manager)
TensorFlow version: 1.2.0
No GPU or CUDA

Describe the problem
tensorflow/python/util/tf_should_use.py is doing from backports import weakref (introduced by    cf238e1f2f68309822e1adb3f86dd439c0b87441), though I guess this is only useful for Python 2. In Python 3, we could simply import weakref. This bug has been observed by other people and reported on the Arch Linux bug tracker.
Traceback
python -c "import tensorflow"
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/lib/python3.6/site-packages/tensorflow/__init__.py", line 24, in <module>
    from tensorflow.python import *
  File "/usr/lib/python3.6/site-packages/tensorflow/python/__init__.py", line 63, in <module>
    from tensorflow.python.framework.framework_lib import *
  File "/usr/lib/python3.6/site-packages/tensorflow/python/framework/framework_lib.py", line 100, in <module>
    from tensorflow.python.framework.subscribe import subscribe
  File "/usr/lib/python3.6/site-packages/tensorflow/python/framework/subscribe.py", line 26, in <module>
    from tensorflow.python.ops import variables
  File "/usr/lib/python3.6/site-packages/tensorflow/python/ops/variables.py", line 26, in <module>
    from tensorflow.python.ops import control_flow_ops
  File "/usr/lib/python3.6/site-packages/tensorflow/python/ops/control_flow_ops.py", line 70, in <module>
    from tensorflow.python.ops import tensor_array_ops
  File "/usr/lib/python3.6/site-packages/tensorflow/python/ops/tensor_array_ops.py", line 33, in <module>
    from tensorflow.python.util import tf_should_use
  File "/usr/lib/python3.6/site-packages/tensorflow/python/util/tf_should_use.py", line 28, in <module>
    from backports import weakref  # pylint: disable=g-bad-import-order
ModuleNotFoundError: No module named 'backports'

In tensorflow/blob/master/tensorflow/python/util/tf_should_use.py the function weakref.finalize is used. This is new since Python 3.4. Thus, it is not a distinction between Python 2 and Python 3 here.
Possible solution is to check if the finalize function exists in weakref module and import backports.weakref otherwise. Importing weakref should be safe, as it exists since Python 2.1:
import weakref
if not hasattr(weakref, 'finalize'):
    from backports import weakref

Or alternatively
try:
    from weakref import finalize
except ImportError:
    from backports.weakref import finalize

as finalize is (currently) the only used member from weakref module in tf_should_use.
@mdeff Maybe you want to change the title of the issue to something like "Only use weakref.finalize from backports in Python < 3.4"

@ebrevdo What do you think? I could also add a PR for this.

I just opened PR #11174 and preferred @spinnau's second option because it's "easier to ask for forgiveness than permission".


Yarn workspaces support (mono-repo)

I tried the plugin on a mono-repo that uses yarn workspaces but I've been unsuccessful so far with what looks to be file resolution error:
Import trace for requested module:
../stylex-lib/index.js
./app/components/ButtonsDemo.tsx
thread '<unnamed>' panicked at src/shared/utils/common.rs:538:8:
called `Result::unwrap()` on an `Err` value: StripPrefixError(())
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
thread '' panicked at /Users/geist/.cargo/registry/src/index.crates.io-6f17d22bba15001f/swc-0.273.24/src/plugin.rs:166:14:
failed to invoke plugin: failed to invoke plugin on 'Some("/nextjs-app-dir-stylex/packages/stylex-lib/index.js")'

Steps to reproduce
https://github.com/sumanbh/nextjs-stylex-workspaces-issue
Note: I'm also setting nmHoistingLimits otherwise the plugin wouldn't resolve the workspace lib stylex-lib.

Hi @sumanbh,
Thank you for finding the problem.
I've tested your example with the original babel plugin and the code with defining colors also doesn't work, although the following error is more informative: Only static values are allowed inside of a stylex.create() call.. This happens because the constants must be imported from a file with a relative path that ends with ".stylex.js", but in your example with importing colors from another workspace, the path is absolute.
import { colors } from 'stylex-lib/colors.stylex';

// ...

<ThemeableButton
  onClick={onClick}
  style={(styles.bordered, testStyles.base)}
>
  Bordered Button
</ThemeableButton>

Here is a link to the official documentation.
In this project, I follow the API of the original plugin and even after fixing this code, it will throw an error, as mentioned above.
Perhaps you should open an issue in the original StyleX repository, and if a solution is found, or has already been found, then I will happily implement it in this project as well.
P.S.
If you change all absolute paths to relatives, for example ../../../stylex-lib/colors.stylex in the packages/next-app/app/components/ButtonTokens.stylex.ts file, then everything starts working as expected with the babel plugin:
import { colors } from '../../../stylex-lib/colors.stylex';

// ...

export const buttonTokens = stylex.defineVars({
  bgColor: colors.blue12,
  textColor: 'white',
  cornerRadius: '4px',
  paddingBlock: '4px',
  paddingInline: '8px',
});

Unfortunately, there is still a problem with the SWC plugin, which also will be fixed soon in the next version.

Hi @sumanbh,
Thank you for finding the problem.
The reason for the path resolving issues is how the plugin handles importing styles from an external file, which are passed to the style property:
import * as stylex from '@stylexjs/stylex';
import { testStyles } from 'stylex-lib';

// ...

<ThemeableButton
  onClick={onClick}
  style={(styles.bordered, testStyles.base /* <-- this one */)}
>
  Bordered Button
</ThemeableButton>

// ...

If you remove them, all other styles are applied correctly.
import * as stylex from '@stylexjs/stylex';

// ...

<ThemeableButton
  onClick={onClick}
  style={(styles.bordered)}
>
  Bordered Button
</ThemeableButton>

// ...

This issue will be fixed in the next version soon.

Hi @sumanbh,
I have released<EMAIL_ADDRESS>with a fix for this issue and prepared a sandbox with the updated version of the plugin.
Could you check that everything works correctly now?


Remove 'TestCategory!=Live' from build definition

Problem
There are two steps (tasks) in the build definition that run unit tests. One for live tests and the other for non-live tests. The tests are not able on their own to determine if connection strings are present in the environment, so '--filter TestCategory!=Live' is used to filter out live tests in one of the steps.
Solution
The tests should detect the presence or absences of connection strings in the build environment and use that to make the run/no-run decision. That way one step can be used for all unit tests.

We could consider using an conditional fact similar to the .NET team (https://github.com/dotnet/arcade/blob/master/src/Microsoft.DotNet.XUnitExtensions/src/Attributes/ConditionalFactAttribute.cs), at least for the xunit based tests.
What we want to avoid is requiring that people need to always pass TestCategory!=Live explicitly in order to get a passing local test run.

@danieljurek we should get the test guidelines updated with what should happen here generally.  Specifically that live tests check if their settings are set up before running rather than having the engineering system condition their execution.

Silently skipping when there is no connection string in the environment is prone to hiding issues with tests. I would rather have tests fail loudly when things are not set up correctly than silently pass.

@jsquire I'm fine with closing this but I do still think it would be nice to have the default runs when folks run locally to not pick-up live tests. However that for your team to decide.


x/build: 11 of our 20 Macs are offline, VMware problems

Our 10-node MacMini cluster VMware cluster is unhappy.
11 of our VMs are offline.
makemac keeps trying to get it back to 20 but is failing with:
Aug 22 16:22:43 godns makemac[1309]: 2018/08/22 16:22:43 Error creating 10.11: govc vm.create ...: exit status 1, govc: datastore 'BOOT_5' not found

/cc @dmitshur @andybons

vSphere crapped itself. MacStadium restarted something. I'll check on it more tomorrow.

MacStadium support says:

Hey Brad. We identified an issue with the host configurations that was causing the vCenter service to not start properly. We have remediated the issue and are rebooting the vCenter. It should be back up shortly. I will let you know when it is available again. Thanks.

(I can authenticate to the vCenter Flash UI now, but I get API errors and all Hosts & VMs are missing in inventory)

Mostly fixed now (for this round). Few oddities remain.

All good.


CAN PLEASE SOMEONE HELP ME WITH THIS ONE

This this is the router i have i want  to turn it access point but i cant i can enter to the interface but there is not much setting  and also cant download the firmware
company name: Orange FUNBOX
country: Morocco
Device Type ZTEH288A
Device Serial No. ZTEEG9KN7704995
Hardware Version ZT-H288A-1.0.0
Software Version ZT_H288A-ma-<IP_ADDRESS>
Boot Version V1.0.2
default user and password:
Username : user
Password: OrangeDQFT
Thank You

Thanks already find solution

ty

Hey @alicedormlol , can you please help me with the same version.
I'm also from morocco and i cant find how to download the config.bin file.
Thanks.

i also can't download the  config.bin only to upload it sad

How did you do it  , i'm having currently the same issue  , orange d zbb doesn't let me configure mine

How did you do it , i'm having currently the same issue , orange d zbb doesn't let me configure mine

dkhalt b support
username: support
password: OrangeF830
3la hsab akhir ar9am f mac address 3andak

did u found something new about config.bin extraction ?

kifach jbdeti lconfig ?

This is the firmware link https://github.com/Freesat-tr/firmware-zte-zxhn-h288a-Orange-maroc
be carefull


Incorrectly assuming that IP address = localhost

I worry that this may display PHP error information to end-users on some REDCap installations using external modules. There are a decent amount of valid REDCap production installations that use an IP address for their REDCap web address (rather than a domain name). Perhaps there is another way to detect a dev environment, such as looking only for internal IPs, such as 10.X.X.X or 192.168.X.X. Thoughts?
https://github.com/vanderbilt/redcap-external-modules/blob/ae148bba624efd8f986d3a216ed0e4b7a6822b14/classes/ExternalModules.php#L108-L116

Why not put a flag in redcap_config that allows you to declare an 'instance' of redcap to be in development mode?
Related - one other 'flag' I wanted to request would be to 'force localhost over http' - this has come up in the myCap app, but could also come up with EM's if people use API methods to try and do actions on the server.  For us, since https is stripped off at the load balancer, if our server tries to do https://localhost we have issues so we have to switch to http.  This, by the way, also breaks the API playground for us...

@123andy, regarding 'force localhost over http', i've not heard of that being an issue myself, but if you want to put in a pull request in REDCap for it, be my guest.
@mmcev106, right now in REDCap all i have is isDev(), which checks if the server is either  localhost or my specific desktop's IP address. It, like isLocalhost() in ExtMods, merely turns on error reporting.
So regarding a "dev" flag to denote development mode of a REDCap instance, i think that is reasonable. We'll look into adding that soon, and then perhaps that flag can be utilized here in isLocalhost(), if that sounds good. Thanks!

Sounds good to me!  Since you're added the flag, I can make the EM change.

Okay, i went ahead and just put in a pull request (https://github.com/vanderbilt/redcap-external-modules/pull/34) for this since i was implementing it on the REDCap side.

Thanks!


Parabolic orbits are not working

It looks like parabolic orbits (e=1) are not working:
orbit = KerrGeoOrbit[0, 11, 1, 1]

returns a bunch of 1/0 errors.

Is it just Schwarzschild?

Actually, I don't think we ever implemented parabolic orbits, did we?

Apparently, I started doing this at some point. I'll investigate.

After looking at it, I remember why I previously abandoned this. Taking the e->1 limit can be done relatively easily for many parts of the solution. For example, the polar solution reverts to the trivial Schwarzschild solution. The however, the limit of the t solution appears  non-trivial. My first attempt yielded a divergent answer. Probably the best route is to take the limit of the EOMs and then solve the resulting elliptic integrals.


Code samples for blobs

Sample code moved from blob readme PR to its own PR. Includes test recordings and updated example to delete blob snapshots.

Can one of the admins verify this patch?


Adapt plugin to 1.0 (breaking change)

Elysia 1.0 has been released and there's a breaking change with hooks, changing the default scope of .on handlers from global to local. This puts it explicitly to global so the plugin can work on 1.0.
This is a breaking change so I updated to a major version.

Would it be possible to make it configurable?

The only other configuration that would make sense is scoped, but I guess you could add a parameter to choose in options

Thank you!


x-axis tooltip hides the actual x value on hover

I'm using a custom formater and specificly this
tooltip: {
      enabled: true,
      x: {
        formatter: (val) => {
          return 'Total transactions'
        }
      }
    }

It works correctly but when i hover over a day, it hides that day and displays the value that it got. This is normal behavior but how can i fix it?

The problem in the above image is that Wednesday is hidden behind Total transactions in the X axis

It will override but not change dynamically while scoll, it's stick on the same situations

A new property xaxis.tooltip.formatter will be provided - which will override the default tooltip.x formatter function

It will override but not change dynamically while scoll, it's stick on the same situations


Script to download finished artifacts

I wrote a Python program to download all the artifacts from the latest Appveyor build: https://bitbucket.org/ned/coveragepy/src/d14e44a86376ec95645584360501ab5afc70d26f/ci/download_appveyor.py?at=default
Maybe others will find it useful.

A very similar script also at
https://github.com/ionelmc/python-lazy-object-proxy/commit/967e3837500bc382269f4e9e8bca351044712e96

I think it would be useful to submit one of those a as a PR for inclusion to the repo along with a new section to the README.md to explain the use cases for this script.

Why not use twine to push artifacts to PyPI whenever the commit is tagged?

Why not use twine to push artifacts to PyPI whenever the commit is tagged?
Continuing my suggestion above about twine,
this is a sample appveyor.yml:
environment:
  PYPI_USER: "FOO"
  PYPI_PSWD:
    # See https://www.appveyor.com/docs/build-configuration/#secure-variables
    secure: <paste_your-encrypted-value>

#... regular build here

on_success:
  - ps: >-
      $cmtid = git rev-parse HEAD
      if (git describe --exact-match $cmtid) {
        twine upload -u PYPI_USER -p PYPI_PSWD dist/*
      }

Of course the situation becomes a bit more complex if gpg-signing is needed.

Why not use twine to push artifacts to PyPI whenever the commit is tagged?

Good idea.

@ankostis wouldn't it be better to simply check if APPVEYOR_REPO_TAG environment variable is true?
https://www.appveyor.com/docs/environment-variables/
git describe only works with annotated tags, not "lightweight" ones.

also, I wonder if that ps snippet would be better placed under deploy_script: instead of on_success:?

hmm, I noticed that if I use deploy_script to upload to PyPI, then Appveyor will skip the deployment to Github Releases which is also specified under its own deploy section.
So it looks like deploy_script will override any other deploy.

JFYI: twine can now take passwords and usernames from environment variables without the need to use it in a command (and potentially leaking it on some CI services):
https://github.com/pypa/twine/blob/master/docs/changelog.rst

:feature:144 Retrieve configuration from the environment as a default.
Repository URL will default to TWINE_REPOSITORY
Username will default to TWINE_USERNAME
Password will default to TWINE_PASSWORD

I had to use this to make it work with the new twine env variables:
on_success:
  - ps: >-
      if ($env:APPVEYOR_REPO_TAG -eq "true") {
        twine upload dist/*.whl
      }

Without the -eq "true" it would upload on every build :-(

The twine-powershell snippet used to work for me, but now it started failing because it appears that twine is printing to stderr and powershell... understands that as a failure (LOL).
https://ci.appveyor.com/project/anthrotype/zopfli/build/1.0.21/job/4uefbbr3xn8s6r5a#L105
Anybody knows a workaround? Like rewriting that powershell code as normal batch cmd?
Or muting twine so it doesn't print to stderr?
thanks

ok, I made it work by calling twine via cmd /c twine upload dist\*.whl 2>&1 so that cmd redirects stderr to stdout before powershell can even see it. Ugly, but works:
https://github.com/obp/py-zopfli/commit/2d211eb793d37169f8006eaf12af7f6af085058c
https://ci.appveyor.com/project/anthrotype/zopfli/build/job/gv6x233d02o8yuvr


Error when playing music from loaded png

Hi, @ftsf  ! @felipebueno from the PICO-8 BBS here.
I saw the video where you play Celeste on PicoLove at 60fps and it's preatty neat. So I cloned your repo and tried to run it here but it's throwing an error: Error: main.lua:1227: attempt to index local 'm' (a nil value)

Hey, @ftsf you can ignore this issue. I realized the problem... I was trying to run the png file, not the p8.
It's working now :smile:

How do you regulate the framerate at 60fps? It runs to fast hahah

I don't think this issue is resolved, there is a bug with music when loading from PNG =)
I'll update the issue name.

@felipebueno it requires changes to the game to operate correctly at 60fps.
http://static.impbox.net/celeste60fps.p8

I realized the problem... I was trying to run the png file
https://ringtonedownload.in/

here https://mp3tomato.com/


Build failure on several platforms after InputEvent changes

Operating system or device - Godot version:
Windows, iOS, OSX, maybe others - 5b3709d
Issue description:
The massive refactor of InputEvent into a series of objects broke the compilation on at least Windows, iOS and OSX according to Travis: https://travis-ci.org/godotengine/godot/builds/234395343
Most likely all non-X11 platforms need fixing, as @reduz only made sure his development platform was compiling and running fine (and given the amount of code that had to be changed, we can't blame him :)).

In windows at least, travis build shows error in:
line 71 InputModifierState mod_state because reduz deleted the structure InputModifierState
	struct KeyEvent {

		InputModifierState mod_state;
		UINT uMsg;
		WPARAM wParam;
		LPARAM lParam;
	};

The deleted struct is in file input_event.h line 140, and I believe it was replaced with class InputEventWithModifiers : public InputEvent  in line 173. So a quick rename of all occurrences might fix the issue.
I could be wrong though, I can't test right now.

I tried to fix most platforms but didn't test..
On May 21, 2017 5:45 PM, "Thaer Razeq"<EMAIL_ADDRESS>wrote:

In windows at least, travis build shows error in:
line 71 InputModifierState mod_state because reduz deleted the structure
InputModifierState
struct KeyEvent {
  InputModifierState mod_state;
  UINT uMsg;
  WPARAM wParam;
  LPARAM lParam;

};
The deleted struct is in file input_event.h line 140, and I believe it
was replaced with class InputEventWithModifiers : public InputEvent in
line 173. So a quick rename of all occurrences might fix the issue.
I could be wrong though, I can't test right now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/8849#issuecomment-302961981,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z25TQrOcldTPYnFpfYm_8TGigT-ARks5r8KJcgaJpZM4Nhq_E
.

I might can check osx. Ev.get_factor() = something seems to be wrong.
Or i dont know enough about cpp...

likely needs to be replaced by set_factor()
On Sun, May 21, 2017 at 6:46 PM, Timo<EMAIL_ADDRESS>wrote:

I might can check osx. Ev.get_factor() = something seems to be wrong.
Or i dont know enough about cpp...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/8849#issuecomment-302965526,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z279M2RPz12zz4Sw99G1xGuC0MBlQks5r8LDAgaJpZM4Nhq_E
.

OSX fixed by #8853. You can check that PR to see what likely needs to be done in the other platforms.

Windows fixed by #8876 and iOS fixed by #8877.

Left to review:

[ ] Android
[ ] Javascript
[ ] UWP

#9271 fixes the JavaScript platform

All main platforms were fixed, the rest will be fixed as soon as someone has a look at them.


Create tool to output only yugipedia CardTable2 model as json

A tool that just takes the yugipedia API and outputs the CardTable2 model as-is. This can be used to cache the data for projects that use directly yugipedia, mostly for the community. Although at some point it might be useful to split download of data and processing to our models, but I would prefer to only keep our models.
This is considered a good first issue and thus marked as such cause when other parts of the tooling are ready, it is just a matter of putting the pieces together into a simple tool. Nevertheless, that is the reason that it is still blocked.

CardTable2 won't exists anymore after #122. In addition, this tool is the perfect starting point to evaluate quarkus cli-implementation with picocli as all the components are already ready. Setting the priority to high to start with this tool first.


x/tools/gopls: add support for folding comment regions

gopls version
golang.org/x/tools/gopls v0.7.5
What did you do?
Wrote a program with comment regions
package foo

//region foomethods

func Bar() string {
	return "Bar"
}

func Baz() string {
	return Baz""
}

//endregion foomethods

What did you expect to see?
I expected to be able to collapse the regions

What did you see instead?
No folding support by the language server.

Editor and settings
To get the desired effect I was able to switch editor.foldingStrategy = "indentation". This does not seem ideal though as I believe this bypasses any special folding that the language server provides.

Thank you for the issue.
Is 'region' supported by other Go tools? Do you have a reference for how 'region' should behave?
Go source code is consumed by many tools, including other editors and pkg.go.dev. Adding comment annotations to control the behavior of any one tool is generally discouraged.

Is 'region' supported by other Go tools?

No. But I don't think it makes sense in the context of other Go tooling. This is just an IDE code organization feature.

Do you have a reference for how 'region' should behave?

It should behave just like folding an if statement or a function. Here is VS Codes official docs and the syntax other language servers support for this feature: folding

Go source code is consumed by many tools, including other editors and pkg.go.dev.

These comments are not attached to any symbols so they would not show up in godoc.

Adding comment annotations to control the behavior of any one tool is generally discouraged.

I hear you, this is really one an editor feature. But I would say it is a common feature. So it really only makes context in of a language server.

Related, with maybe some extra info: https://github.com/microsoft/vscode/issues/91140

Duplicate of https://github.com/golang/vscode-go/issues/134

Closing in favor of the vscode issue.


[18.09 backport] pkg/archive:CopyTo(): fix for long dest filename

relates to https://github.com/moby/moby/pull/35739
As reported in docker/for-linux/issues/484, since Docker 18.06
docker cp with a destination file name fails with the following error:

archive/tar: cannot encode header: Format specifies USTAR; and USTAR cannot encode Name="a_very_xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx_long_filename_that_is_101_characters"

The problem is caused by changes in Go 1.10 archive/tar, which
mis-guesses the tar stream format as USTAR (rather than PAX),
which, in turn, leads to inability to specify file names
longer than 100 characters.
This tar stream is sent by TarWithOptions() (which, since we switched to
Go 1.10, explicitly sets format=PAX for every file, see FileInfoHeader(),
and before Go 1.10 it was PAX by default). Unfortunately, the receiving
side, RebaseArchiveEntries(), which calls tar.Next(), mistakenly guesses
header format as USTAR, which leads to the above error.
The fix is easy: set the format to PAX in RebaseArchiveEntries()
where we read the tar stream and change the file name.
A unit test is added to prevent future regressions.
NOTE this code is not used by dockerd, but rather but docker cli
(also possibly other clients), so this needs to be re-vendored
to cli in order to take effect.
Signed-off-by: Kir Kolyshkin<EMAIL_ADDRESS>(cherry picked from commit f55a4176febbd0dffd6e5eb65beb70bc32912d0b)
Signed-off-by: Sebastiaan van Stijn<EMAIL_ADDRESS>
- What I did
- How I did it
- How to verify it
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

After this is merged, we need to update the cli vendor as well 😅
ping @kolyshkin @andrewhsu

(for the reference, this is a backport of moby/moby#38634)
LGTM


Updated deploy script to re-generate mypy stubs for --strict support.

Since MyPy cannot understand dynamically generated __all__ in --strict mode, this will ensure users can still use mypy --strict with their applications without us hardcoding around 100 names into each package-level __init__ module.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #10   +/-   ##
=======================================
  Coverage   79.79%   79.79%
=======================================
  Files          78       78
  Lines        9560     9560
  Branches     1218     1218
=======================================
  Hits         7628     7628
  Misses       1876     1876
  Partials       56       56

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 1935de3...957d996. Read the comment docs.


Merging Develop into Master 4.1.2019 - Expert Insights docs and release note, call out links update

Description of the change:
Reason for the change:
Link to original source:

Closes #

Hello @Whatthefoxsays,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


Exception thrown in DXSETUP.EXE

Use rolling release. 7eb86b223f578097ccf8307ad6e0136713040c19
DXSETUP.zip
e:\DXSETUP.EXE: System.AggregateException: One or more errors occurred. (Requested to read 4 bytes from content, 0 returned)
 ---> System.IO.EndOfStreamException: Requested to read 4 bytes from content, 0 returned
   at SabreTools.IO.Extensions.ByteArrayReaderExtensions.ReadToBuffer(Byte[] content, Int32& offset, Int32 length)
   at SabreTools.IO.Extensions.ByteArrayReaderExtensions.ReadUInt32(Byte[] content, Int32& offset)
   at SabreTools.Serialization.Wrappers.PortableExecutable.ParseDebugTable()
   at SabreTools.Serialization.Wrappers.PortableExecutable.get_DebugData()
   at SabreTools.Serialization.Wrappers.PortableExecutable.FindCodeViewDebugTableByPath(String path)
   at BinaryObjectScanner.Protection.Macrovision.SafeDiscCheckPortableExecutable(String file, PortableExecutable pex, Boolean includeDebug) in /home/runner/work/BinaryObjectScanner/BinaryObjectScanner/BinaryObjectScanner/Protection/Macrovision.SafeDisc.cs:line 111
   at BinaryObjectScanner.Protection.Macrovision.CheckPortableExecutable(String file, PortableExecutable pex, Boolean includeDebug) in /home/runner/work/BinaryObjectScanner/BinaryObjectScanner/BinaryObjectScanner/Protection/Macrovision.cs:line 149
   at BinaryObjectScanner.FileType.Executable.<>c__DisplayClass29_0.<RunPortableExecutableChecks>b__0(IPortableExecutableCheck checkClass) in /home/runner/work/BinaryObjectScanner/BinaryObjectScanner/BinaryObjectScanner/FileType/Executable.cs:line 499
   at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`2.<ForWorker>b__1(RangeWorker& currentWorker, Int64 timeout, Boolean& replicationDelegateYieldedBeforeCompletion)
--- End of stack trace from previous location ---
   at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`2.<ForWorker>b__1(RangeWorker& currentWorker, Int64 timeout, Boolean& replicationDelegateYieldedBeforeCompletion)
   at System.Threading.Tasks.TaskReplicator.Replica.Execute()
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.TaskReplicator.Run[TState](ReplicatableUserAction`1 action, ParallelOptions options, Boolean stopOnFirstFailure)
   at System.Threading.Tasks.Parallel.ForWorker[TLocal,TInt](TInt fromInclusive, TInt toExclusive, ParallelOptions parallelOptions, Action`1 body, Action`2 bodyWithState, Func`4 bodyWithLocal, Func`1 localInit, Action`1 localFinally)
--- End of stack trace from previous location ---
   at System.Threading.Tasks.Parallel.ForWorker[TLocal,TInt](TInt fromInclusive, TInt toExclusive, ParallelOptions parallelOptions, Action`1 body, Action`2 bodyWithState, Func`4 bodyWithLocal, Func`1 localInit, Action`1 localFinally)
   at System.Threading.Tasks.Parallel.ForEachWorker[TSource,TLocal](IEnumerable`1 source, ParallelOptions parallelOptions, Action`1 body, Action`2 bodyWithState, Action`3 bodyWithStateAndIndex, Func`4 bodyWithStateAndLocal, Func`5 bodyWithEverything, Func`1 localInit, Action`1 localFinally)
   at System.Threading.Tasks.Parallel.ForEach[TSource](IEnumerable`1 source, Action`1 body)
   at BinaryObjectScanner.FileType.Executable.RunPortableExecutableChecks(String file, Stream stream, PortableExecutable pex, Boolean includeDebug) in /home/runner/work/BinaryObjectScanner/BinaryObjectScanner/BinaryObjectScanner/FileType/Executable.cs:line 495
   at BinaryObjectScanner.Scanner.ProcessExecutable(Executable executable, String fileName, Stream stream) in /home/runner/work/BinaryObjectScanner/BinaryObjectScanner/BinaryObjectScanner/Scanner.cs:line 501
   at BinaryObjectScanner.Scanner.GetInternalProtections(String fileName, Stream stream) in /home/runner/work/BinaryObjectScanner/BinaryObjectScanner/BinaryObjectScanner/Scanner.cs:line 350

This had to be fixed in Serialization. TL;DR is that the debug table is expected to exist if a certain set of things are set in the header and these have it stripped out regardless. Serialization needs to handle this by assuming that nothing will exist, unfortunately.

In the meantime, latest WIP will have a fix that wraps the call above and stops it from throwing an exception all the way up the stack. That one went in as of https://github.com/SabreTools/BinaryObjectScanner/commit/e3eed76826fe8a4254783eb59e91abe69233e935

Looks good :+1:
E:\Test_net8.0_win-x64_Debug\Test\bin\Debug\net8.0\win-x64\publish>test -d e:\DXSETUP.EXE
0.00%:  -
0.00%: e:\DXSETUP.EXE - Checking file
100.00%: e:\DXSETUP.EXE -
Time elapsed: 00:00:00.4630160


Define query parameters JQLike

So with I guess Angular 1.4.2. we have the possibility to change the default paramSerializer like so
.config(['$httpProvider', function ($httpProvider) {
            $httpProvider.defaults.paramSerializer = '$httpParamSerializerJQLike';
        }])

The outcoming http request would look like:
http://api.example.de/v1_1/domains?filter[domain]=somedomain&from=2015-08-21&to=2015-08-21&page=1&page_size=100
Note the filter[domain]=somedomain part.
Now I want to advance our project so that all those parameters are also reflected in the URL as query parameters. That is no problem for params like page, page_size, from, to and so on, I can just define them in the state.
But I have a problem with the filter[domain] part. Our table provider ng-table generates url's like
http://example.de/#/app/domains?page=1&count=100&filter%5Bdomain%5D=somedomain
and I didn't found a way to get ui-router to accept filter[domain] as a query param. Do you have any suggestions how I can work around this problem or is there an option to accept wildcard parameters?

Angular 4.2?
Did you try: :filter%5Bdomain%5D?

Sorry I mean Angular 1.4.2. Yeah I tried that but it does not work.

what configurations have you tried? Can you post a plnkr demonstrating please

Closing due to inactivity


Initial webview replay loading setup

The app will read build-time settings, such as prefix and auth token, and use them for accessing the replay.
URLs entered in location bar will then load from the specified prefix.

[ ] setup build time settings, including WEB_ARCHIVE_PREFIX and AUTH_TOKEN
[ ] have the location bar navigate the webview to $WEB_ARCHIVE_PREFIX/<url>
[ ] configure the session used by the webview to add a custom header, X-Auth-Token: $AUTH_TOKEN for now.
[ ] configure the session used by the webview to block downloads

@ikreymer what is the reason behind the requirement to use <webview> vs BrowserView? Electron docs have this warning:

We currently recommend to not use the webview tag and to consider alternatives, like iframe, Electron's BrowserView, or an architecture that avoids embedded content altogether.

@ikreymer what is the reason behind the requirement to use <webview> vs iframe or BrowserView? Electron docs have this warning:

We currently recommend to not use the webview tag and to consider alternatives, like iframe, Electron's BrowserView, or an architecture that avoids embedded content altogether.

Yeah, this has been the recommendation for a number of years, but it seems webview support is generally pretty good in the latest versions. The reason for not using iframe is to be able to have a separate session for the contents.
The BrowserView is more cumbersome to work with, and would require responding to nav events in the main process and more interprocess communication.
We can definitely revisit if run into any issues

Breaking out the rest of the things into their own issues

This is done!


Add e2e framework tests execution to infra-deployments

Current PR change the openshift ci scripts to include the execution of e2e tests in every PR checks.
The e2e binary was included in previous PRs: https://github.com/redhat-appstudio/infra-deployments/pull/43
Related issue: https://issues.redhat.com/browse/RHDP-416

/test appstudio-e2e-deployment

/test appstudio-e2e-deployment

/test appstudio-e2e-deployment

/test appstudio-e2e-deployment


Switch from pre-commit to Hatch scripts

Description
Pre-commit runs Git hooks. It can run on different Git events like pre-push and can also easily run on CI (continuous integration) platforms like GitHub Actions. These pre-commit hooks are often related to code quality and help ensure code quality checks are continuously enforced.
While it is helpful for continuously running code quality checks, pre-commit has some downsides.

pre-commit requires separate dependency versions and configuration in its own file .pre-commit-config.yaml, even when those dependencies are already configured in a Python requirements file like pyproject.toml (https://github.com/br3ndonland/inboard/commit/84b25bc).
pre-commit requires separate file paths because it identifies files with Python regular expressions ("regexes") instead of globs like many other tools, and disregards file paths present in other configuration files. Translation between globs and regexes can be complicated and files can be unexpectedly excluded as a result.
pre-commit commands sometimes have different behavior than the commands would have normally.

pre-commit/pre-commit#1495
pre-commit/pre-commit#1580
pre-commit/pre-commit#2967
streetsidesoftware/cspell-cli#52
streetsidesoftware/cspell-cli#170

pre-commit issues are often not explained or fixed.

pre-commit/mirrors-mypy#83 (note that issues were disabled on this repo)
pre-commit/mirrors-prettier#29 (note that issues were disabled on this repo)
pre-commit/pre-commit-hooks#810
pre-commit/pre-commit-hooks#835
pre-commit/pre-commit#2578
PyCQA/flake8#234

Hatch scripts can replace pre-commit.
Changes

Add Hatch scripts to replace pre-commit commands

check:

Black
Flake8
isort
Mypy
Prettier
CSpell

format:

Black
isort
Prettier

Replace other pre-commit hooks where possible

Replace mixed-line-ending by adding * text=auto eol=lf to a .gitattributes file

Update GitHub Actions workflows to run Hatch scripts
Update docs for Hatch scripts
Remove pre-commit from pyproject.toml
Remove .pre-commit-config.yaml

Related

Hatch docs: config - environments - scripts
Pre-commit

I find your writeups incredibly informative, thanks again!


No license?

Just noticed, there's no MIT license in the Gridcoin repo. Could we generate an MIT license & properly acknowledge bitcoin/peercoin/blackcoin?
Cheers

licence is an important issue!
At least from point of that Gridcoin is derived from several other coins (1),
and those have already a licence to which Gridcoin MUST adhere.
If Gridcoin code does not list a licence possible risks are sueing, ...
(1)  "It uses parts from source code of Bitcoin, Litecoin, Novacoin, Peercoin and Blackcoin for its Proof-of-Stake." http://wiki.gridcoin.us/Proof-of-Research#Protocol_and_Architecture

There is some info in the COPYING file

The MIT License (MIT)
Copyright (c)
Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:
The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.
 IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

Can't we just add a bit of GRC info into that?

GRCTest, what file do you believe is missing in our repo?  You can take a look at litecoin as an example being branched from Bitcoin, and then post the name of the file you believe is missing in our repo.

I was looking for a license.md or license.txt file that sometimes gets generated with new GitHub repo, but the file 'COPYING' is the license file! https://github.com/gridcoin/Gridcoin-Research/blob/master/COPYING
False alarm, can close this issue!


prepare script to build when npm install from git

When I ran npm install --package-lock git+https://github.com/asteasolutions/zod-to-openapi.git#5c9ae89 to use the latest code changes, it does not install js files, so my code fails with:
Error: Cannot find module<EMAIL_ADDRESS>Please verify that the package.json has a valid "main" entry

This fix adds a prepare script to run npm run build.

Thank you @mikehinchey  I'll cherry pick your commits into a sub branch so that I can release this now instead of waiting for 5.0.0

It should now be available in v4.7.0


What's the difference between "quantization bit in advanced configuration" and "export quantization bit in export tab"?

as title. can you please explain it?  Thanks!

quantization_bit is used for QLoRA training. It utilizes bitsandbytes to quantize model weights on-the-fly
export_quantization_bit is used for model exporting. It utilizes AutoGPTQ to perform post-training quantization


Fix Asynchronous Components docs

Asynchronous Components are out of date!

Fixed in https://github.com/quantmind/pulsar/releases/tag/1.0.1


Module 'LayoutTestBase' not found

I'm getting an error when trying to build for tests, even without #import "LayoutTest.h" anywhere.

It seems like the linker is having a hard time with the @import. I'm only linking these pods with my test target, which has a minimum build of iOS 7.0. I'm using cocoapods 0.38.2. I'm facing this issue in pod 'LayoutTest', '1.1.0' and pod 'LayoutTest', '~> 1.1.1'
Is there anything I can do fix this issue? I would love to be able to integrate these view tests! It seems like a very well done testing framework. Thanks!

me too ~

Don't think iOS 7 support will be possible as Embedded Frameworks aren't supported. Podspec will need updated to reflect this. @plivesey can hopefully confirm though, not sure what minimum target linkedin are supporting.

Hmmmm. I'd like to support iOS 7, because there's nothing in the code which is iOS 8 specific.
I think this was needed for swift support because using #import "LayoutTestBase.h" didn't work with Allow non-modular Includes.
Does changing to #import "LayoutTestBase.h" fix the problem for you guys?
Before abandoning iOS 7, I'd go for:

Adding iOS 8 as the minimum build target for LayoutTest/Swift. This is because Swift isn't supported for cocoapods < iOS 8 (maybe we don't need this because cocoapods will already warn people?)
Finding a work around to make this work on iOS 7. My first idea is to do something like  #if BUILD_SESSINGS.target == 'iOS 7'

I think I'll have some time to look at this today.

Ok. I've investigated, and the problem seems to be around use_frameworks!
I have two projects. One is Objective-C and has this Podfile:
target :'TestLayoutObjcTests' do
pod 'LayoutTest'
end

It has a deployment target of iOS 7 and fails to compile. If I change @import LayoutTestBase to #import "LYTLayoutPropertyTester.h", it compiles. Also, if I add use_frameworks! to the start of the podfile, it compiles. Therefore, I think the bug occurs when not using use_frameworks!.
In my other project, I have a swift project which has the following Podfile:
use_frameworks!

target :'TestLayoutPodTests' do
pod 'LayoutTest/Swift'
end

This will compile normally. If I change @import LayoutTestBase to #import "LYTLayoutPropertyTester.h", it fails to compile. Therefore, this fix will not work because it will break existing projects.
So, iOS 7 is not actually the problem. It's an absence of use_frameworks. However, it also seems like this problem must have been hit before since it will happen whenever you have a pod which depends on another pod. I'll do some research and see what I can find. Anyone else have any ideas on how to fix?

In the meantime, if you need a workaround, you could change your Podfile to use_frameworks!, but completely understand if you don't want to do this or this isn't possible (is this even supported for iOS 7? I know it's not if you use swift...not sure about objective-c).

I've opened a stack overflow question: http://stackoverflow.com/questions/35468498/include-of-non-modular-header-inside-framework-error-how-do-you-make-transitiv
Had no luck investigating this this morning.

Hi @plivesey - I just want to give you some feedback that I've been following your investigation and really appreciate you looking into this! I'm going to try a couple of things tomorrow and I'll report back here.
I originally tried adding use_frameworks! for only my testing target, but I didn't have any luck. Right now, it doesn't look all of our dependencies play nice with use_frameworks (maybe because of iOS 7?), so hopefully we can find a different solution.
Anyway, I'll keep you updated on any progress I make. Thanks again!

Great. Thanks @cconeil.
I have another temporary work around. Change your podfile to:
pod 'LayoutTestBase'

Then, copy and paste LYTLayoutTestCase into your project. Change the top of the file manually to #import "LYTLayoutPropertyTester.h".
Now, you should be able to write layout tests. This solution is clearly not acceptable long term, but a possible workaround for now.

Oh, and also, we may have some progress...someone has been experimenting with custom module maps. Will update the thread later today.

Some progress is being made: https://github.com/goyalankit/LayoutTestSample/commit/f53a5d8bab7e4d6eb1a545caa0a6b7b3b5c2f90c

Also: https://guides.cocoapods.org/syntax/podspec.html#module_map

Note: the parse SDK seems to get around this problem somehow:
https://github.com/ParsePlatform/Parse-SDK-iOS-OSX
See this file which is a .h which imports something from a dependency:
https://github.com/ParsePlatform/Parse-SDK-iOS-OSX/blob/master/Parse/PFCloud.h
I can't work out what they are doing differently though yet.

This looks promising. Something like:
#if __has_include(<RealmMapView/RealmMapView-BridgingHeader.h>)
@import RBQFetchedResultsController;
#else
#import <RBQFetchedResultsController/RBQFetchRequest.h>
#endif

https://github.com/CocoaPods/CocoaPods/issues/4420

According to this thread, this issue may have been fixed in Cocoapods-1.0.0-beta. Looks like it was committed Nov 12th, but 39.0 was released in October.

This seems to work in my local tests:
#if __has_include(<LayoutTestBase/LayoutTestBase-BridgingHeader.h>)
@import LayoutTestBase;
#else
#import <LayoutTestBase/LYTLayoutPropertyTester.h>
#endif

I've pushed some commits to the repo which I think fix the problem.
@cconeil @chentoo could you please try your projects against HEAD to make sure it works for you? I haven't actually released the version, so you'll have to do:
pod 'LayoutTest', :head

I think that's the right syntax?

If it works for your projects, I'll do a release tomorrow.

Hey @plivesey - it looks like it's linking properly now! Thanks so much for your work.
With that being said, I'm still running into some issues. In order to get it work, I took your advice and used pod 'LayoutTest', :head (I also had to add pod 'LayoutTestBase', :head to get it to compile).
Here are my issues:

I'm not able to #import <LayoutTestBase/LayoutTestBase.h>. I was looking at the github project, and it looks like that file exists, but I can't seem to find it after doing a pod install. Maybe there is a problem with your .podspec? I've solved this issue by importing each file from LayoutTestBase on its own.
There might be some sort of infinite loop? Since I've updated to the :head of both of these pods, my one test takes minutes to run, and finally stops with this error:
*** Test operation was canceled. If you believe this error represents a bug, please attach the log file at /var/folders/dh/rm4hrgm50y92q0lr77x_q4cm0000gn/T/com.apple.dt.XCTest-status/Session-2016-02-19_13:43:12-rbFvd9.log.
That didn't look right to me.

Thanks for all of your help! Let me know if you want me to try anything else.

Thanks for checking it out. Yeah, makes sense that you need both LayoutTest and LayoutTestBase. As for the imports, where is this code? Is this in your app code?
I'm not sure if there is a 'real' import all file. I should probably add one...so used to the @import syntax :).
Can you paste the code of your test? I can give some advice on how to debug then.
Thanks for your help getting this working.

Btw, if you checkout the project, in TestProjects/SampleAppiOS7 there's an example of a very simple test which passes. So something works, but potentially there's a bug elsewhere.

@cconeil any update? I'd like to release this soon :), but if it's not working for you...maybe it won't help much?

@plivesey sorry about that - I was able to get it working with :head so you should feel free to go ahead and release.
Thanks!

I'll do it today - thanks.
On Tue, Feb 23, 2016 at 1:00 PM Chris<EMAIL_ADDRESS>wrote:

@plivesey https://github.com/plivesey sorry about that - I was able to
get it working with :head so you should feel free to go ahead and release.
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/linkedin/LayoutTest-iOS/issues/8#issuecomment-187902516
.

Going to close this ticket since we have a working solution. Fix will be in 1.2.0.


Bottom App Bar is not working properly.

I tried Bottom App Bar on a Lenovo a 2010 smartphone. But it isn't working properly. You'll know the issue by watching below image:

A small space is shown below that Bottom Nav Bar. How could I remove it?
When I toggled my web browser (Chrome) to Desktop mod, the UI became more bad:-(
Check below screenshot:

Some codes are shown below:
App.js:
`import React, { Component } from 'react';
import Ui from './Ui';
class App extends Component {
render() {
return (

);
}
}
export default App;`
Ui.js:
`import React, { Fragment } from 'react';
import PropTypes from 'prop-types';
import { withStyles } from '@material-ui/core/styles';
import AppBar from '@material-ui/core/AppBar';
import Toolbar from '@material-ui/core/Toolbar';
import Typography from '@material-ui/core/Typography';
import IconButton from '@material-ui/core/IconButton';
import Paper from '@material-ui/core/Paper';
import Button from '@material-ui/core/Button';
import List from '@material-ui/core/List';
import ListItem from '@material-ui/core/ListItem';
import ListItemText from '@material-ui/core/ListItemText';
import ListSubheader from '@material-ui/core/ListSubheader';
import Avatar from '@material-ui/core/Avatar';
import MenuIcon from '@material-ui/icons/Menu';
import AddIcon from '@material-ui/icons/Add';
import SearchIcon from '@material-ui/icons/Search';
import MoreIcon from '@material-ui/icons/MoreVert';
const styles = theme => ({
container: {
position: 'relative',
maxWidth: 500,
},
text: {
paddingTop: theme.spacing.unit,
paddingLeft: theme.spacing.unit * 2,
paddingRight: theme.spacing.unit * 2,
},
main: {
height: 400,
overflowY: 'auto',
position: 'relative',
},
list: {
marginBottom: theme.spacing.unit * 2,
},
subHeader: {
backgroundColor: '#fff',
},
toolbar: {
alignItems: 'center',
justifyContent: 'space-between',
},
fabButton: {
position: 'absolute',
top: -30,
left: 0,
right: 0,
margin: '0 auto',
},
});
const messages = [
{
id: 1,
primary: 'Brunch this week?',
secondary: "I'll be in the neighbourhood this week. Let's grab a bite to eat",
person: '/static/images/adeel.jpg',
},
{
id: 2,
primary: 'Birthday Gift',
secondary: 'Do you have a suggestion for a good present for John on his work
anniversary. I am really confused & would love your thoughts on it.',
person: '/static/images/uxceo-128.jpg',
},
{
id: 3,
primary: 'Recipe to try',
secondary: 'I am try out this new BBQ recipe, I think this might be amazing',
person: '/static/images/remy.jpg',
},
{
id: 4,
primary: 'Yes!',
secondary: 'I have the tickets to the ReactConf for this year.',
person: '/static/images/uxceo-128.jpg',
},
{
id: 5,
primary: "Doctor's Appointment",
secondary: 'My appointment for the doctor was rescheduled for next Saturday.',
person: '/static/images/adeel.jpg',
},
{
id: 6,
primary: 'Discussion',
secondary: 'Menus that are generated by the bottom app bar (such as a bottom
navigation drawer or overflow menu) open as bottom sheets at a higher elevation
than the bar.',
person: '/static/images/remy.jpg',
},
{
id: 7,
primary: 'Summer BBQ',
secondary: 'Who wants to have a cookout this weekend? I just got some furniture
for my backyard and would love to fire up the grill.',
person: '/static/images/uxceo-128.jpg',
},
];
function BottomAppBar(props) {
const { classes } = props;
return (

Inbox

{messages.map(({ id, primary, secondary, person }) => (

{id === 1 && Today}
{id === 3 && Yesterday}

))}

);
}
BottomAppBar.propTypes = {
classes: PropTypes.object.isRequired,
};
export default withStyles(styles)(BottomAppBar);`
Please help me...
I'm new to ReactJS and Material-Ui

@AJ-Creations Don't miss this section: https://material-ui.com/getting-started/usage/#responsive-meta-tag

@AJ-Creations It's also important to notice that the demo wasn't written to be usable outside of the box.


pkg: kernel && new-kernel: enable RPi4 thermal driver

After kernel version 5.6.y we have separate thermal driver for rpi4 board.
Enable it in kernel and new-kernel packages.
Signed-off-by: Aleksandrov Dmitriy<EMAIL_ADDRESS>

Should we also enable CONFIG_ARM_RASPBERRYPI_CPUFREQ to support frequency adjustment?


Want to add more useful and descriptive C++ codes

Being a C++ developer I would request you to allow me to add some more codes into your repository to make it even more comprehensive.

Go Ahead🎈

Even I want to contribute too.

@LikhithaTadikonda Make a new issue with more specific title on the kind of code you wish to add, and then make a PR referencing that issue.


unhashable type: 'LineString'

While I was trying to repeat this: https://github.com/gboeing/osmnx/blob/master/examples/04-example-simplify-network.ipynb, I have this issue:
import osmnx as ox
G = ox.graph_from_place('caneva, Italy')
ox.plot_graph(G, node_color='b', node_zorder=3)
# simplify the network
G2 = G.copy()
G2 = ox.simplify_graph(G2)

result:
TypeError                                 Traceback (most recent call last)
<ipython-input-61-b9e77563d9b6> in <module>()
      1 # simplify the network
      2 G2 = G.copy()
----> 3 G2 = ox.simplify_graph(G2)

/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/osmnx/osmnx.py in simplify_graph(G_, strict)
   2373         for key in edge_attributes:
   2374             # don't touch the length attribute, we'll sum it at the end
-> 2375             if len(set(edge_attributes[key])) == 1 and not key == 'length':
   2376                 # if there's only 1 unique value in this attribute list, consolidate it to the single value (the zero-th)
   2377                 edge_attributes[key] = edge_attributes[key][0]

TypeError: unhashable type: 'LineString'

Python3, shapely 1.6.

Yes, this is because you are trying to simplify a graph that has already been simplified. By default, graph_from_place performs simplification before returning the graph to you (it has a simplify argument with default value = True). If you want to simplify manually after creating the graph, turn simplify off when you call graph_from_place, like this:
import osmnx as ox
G = ox.graph_from_place('Caneva, Italy', simplify=False)
ox.plot_graph(G)
G2 = ox.simplify_graph(G.copy())
ox.plot_graph(G2)

I will create a clearer exception message when trying to simplify an already simplified graph so that this is unambiguous.

Ok, new error message has been pushed and is available in the latest beta version, 0.1b2. Available via pip install osmnx --upgrade.

awesome! Thanks for the velocity :))


figure out how to upload the image to the server. Post request?

Image payload is now able to be sent to the server, but now need to verify that the image payload is correct somehow. matplotlib visual output serverside? or just run the circle finding algo on the image and then send the annotated image back to the client... i'm feeling the second option.


startEchoTest returns -8 and fails to echo back

I am attempting to implement a startEchoTest, but calling this returns -8 and does not function as expected. I am unable to find what this means (in the AgoraErrorCode documentation the code it skips from 7 to 9)
When I run this test, the application is already connected to a channel. The user is a Host. I create a token with a unique channel Id and a uid of 0xFFFFFFFF. I only wish to test Audio.
After speaking for over 2 seconds I expect to hear my voice repeated back.
What else should I try?
Example Code:
Future<void> _startEchoTest() async {
    Random random = new Random();
    int randomNumber = random.nextInt(100);
    String uniqueChannelId = '${widget.channelId}_$randomNumber';
    TokenResult? agoraTokenUid = await AgoraMicroService.getToken(uniqueChannelId, '0xFFFFFFFF', '1', null);

    widget.rtc.engine?.startEchoTest(
      EchoTestConfiguration(
        token: agoraTokenUid?.token,
        channelId: uniqueChannelId,
        enableVideo: false,
        enableAudio: true,
        intervalInSeconds: 2)
      );
  }

Log:
flutter: XXXXX getToken /api/get/tokens/bigangein4hosts1687132959468_54?role=1&timeout=null&userAccount=0xFFFFFFFF
flutter: response = {"code":200,"rtc_token":"00613629663327f4d27bc13f073dc6f52b9IAC1kHzOGsuBSmWjHxAfpmFm03JZnGPFmUNt2VfuWIC7pUG2aBuHvm31IgAd3PEFdZabZAQAAQAVYZpkAgAVYZpkAwAVYZpkBAAVYZpk","rtm_token":"00613629663327f4d27bc13f073dc6f52b9IAAuxfUZJXAp7hxAd+ubIuJfth/9Hr9xuXiwJTn93AXJSVIs1sgAAAAAEAAKlDoEdZabZAEA6AMVYZpk","uid":1076467332}
flutter: response.statusCode = 200, phrase = OK
[info] [iris_rtc_engine_impl.cc:97] api name RtcEngine_startEchoTest3 params {"config":{"channelId":"bigangein4hosts1687132959468_54","enableAudio":true,"enableVideo":false,"intervalInSeconds":2,"token":"00613629663327f4d27bc13f073dc6f52b9IAC1kHzOGsuBSmWjHxAfpmFm03JZnGPFmUNt2VfuWIC7pUG2aBuHvm31IgAd3PEFdZabZAQAAQAVYZpkAgAVYZpkAwAVYZpkBAAVYZpk"}}
[info] [iris_rtc_engine_impl.cc:113] api name RtcEngine_startEchoTest3 ret 0 result {"result":-8}
[info] [iris_rtc_engine_impl.cc:97] api name RtcEngine_stopEchoTest params {}
[info] [iris_rtc_engine_impl.cc:113] api name RtcEngine_stopEchoTest ret 0 result {"result":-5}
[VERBOSE-2:dart_vm_initializer.cc(41)] Unhandled Exception: AgoraRtcException(-5, null)
#0      RtcEngineImpl.stopEchoTest (package:agora_rtc_engine/src/binding/agora_rtc_engine_impl.dart:414:7)
<asynchronous suspension>

iPhone XS iOS 16.5
Agora Flutter SDK 6.2.0 / iOS 4.2.0

Can you try this example to see if it works or not?
https://github.com/AgoraIO-Extensions/Agora-Flutter-SDK/blob/main/example/lib/examples/advanced/precall_test/precall_test.dart

The PreCallTest isn't enabled for iOS or Android it seems
  if (kIsWeb || !(Platform.isAndroid || Platform.isIOS))
    {'name': 'PreCallTest', 'widget': const PreCallTest()},

Removing that check and running gives:
[info] [iris_rtc_device_manager_impl.cc:48] api name AudioDeviceManager_getRecordingDevice params {}
[error] [iris_rtc_device_manager_impl.cc:55] api name AudioDeviceManager_getRecordingDevice ret -1 result
[VERBOSE-2:dart_vm_initializer.cc(41)] Unhandled Exception: AgoraRtcException(-1, null)
#0      AudioDeviceManagerImpl.getRecordingDevice (package:agora_rtc_engine/src/binding/audio_device_manager_impl.dart:169:7)
<asynchronous suspension>
#1      _State._enumerateVideoDevices (package:agora_rtc_engine_example/examples/advanced/precall_test/precall_test.dart:156:34)
<asynchronous suspension>
#2      _State._initEngine (package:agora_rtc_engine_example/examples/advanced/precall_test/precall_test.dart:114:5)
<asynchronous suspension>
C++ got rtc.enable_xdump_upload notify: dirpath=/uw/config/global/rtc.enable_xdump_upload: val=5

I'll keep having a play, as there are some other examples I am interested in. A few Android only things (perms for instance) in code though.

I'm sorry that the PreCallTest case only works on desktop at this time.
I think you can try to call the startEchoTest  before joinChannel.

I've got it working with a bit of hacking, but as you said it only works when the app has NOT already joined a channel.
I/spdlog  ( 1503): [2023-06-30 14:12:29.972] [1503] [info] [iris_rtc_engine_impl.cc:97] api name RtcEngine_startEchoTest3 params {"config":{"channelId":"testEcho","intervalInSeconds":4,"token":"0****="}}
I/agora.io( 1503): (00000213): RtcEngine::startEchoTest(this:0xcd385c00, {"config":{"view":"0x0", "enableAudio":1, "enableVideo":1, "token":"0****=", "channelId":"testEcho", "intervalInSeconds":4}})
I/agora.io( 1503): (00000214):   RtcEngine::startEchoTest_(this:0xcd385c00, config[a/v: 1/1, cname:testEcho, view:0x0, token:0****=] interval:4)
E/agora.io( 1503): API call to start echo test: Already in channel(audio track published).
I/spdlog  ( 1503): [2023-06-30 14:12:29.974] [1503] [info] [iris_rtc_engine_impl.cc:113] api name RtcEngine_startEchoTest3 ret 0 result {"result":-8}

This seems to contradict the documentation at https://api-ref.agora.io/en/video-sdk/flutter/6.x/API/class_irtcengine.html#api_irtcengine_startechotest3 where it says:

Attention:
You can call this method either before or after joining a channel.

Let me know if you want to see what changes I needed to make to get the precall working on phones

@ChrisMICDUP
The doc is a little confusing for "after joining a channel", you can call it after joining a channel only the audio/video not be published, say that this should work
  await _engine.joinChannel(
    token: '<token>',
    channelId: 'channelId',
    uid: 0,
    options: ChannelMediaOptions(
      ...
      publishCameraTrack: false, // do not publish the video
      publishMicrophoneTrack: false, // do not publish the audio
    ),
  );

  // Then call `startEchoTest `

I had requested the doc team to update the doc.

Thanks @littleGnAl I have it working now. For any others dropping by this issue and for Agora docs, my use case is that I have already joined an Agora channel with video and audio enabled. I now wish to run the echoTest to confirm my audio is clear.
I am required to do the following: 1. Generate a new token with the uid of 0xFFFFFFFF (my token service requires a string), 2. disable video and audio, 3. run echo test, 4. stop echo test, 5. reenable video and audio. Note that others in the channel will hear your test
    TokenResult? agoraTokenUid = await AgoraMicroService.getToken(widget.channelId, 0xFFFFFFFF.toString(), '1', null);
    await widget.rtc.engine?.updateChannelMediaOptions(ChannelMediaOptions(publishMicrophoneTrack: false, publishCameraTrack: false));

    await widget.rtc.engine?.startEchoTest(
      EchoTestConfiguration(
        token: agoraTokenUid?.token,
        channelId: widget.channelId,
        intervalInSeconds: 4)
      );

updateChannelMediaOptions  should work too, just keep in mind that do not publish the audio/video before startEchoTest  should be fine.


List mappings page

Issue: When access a created mappings(columns saved) springboot returns [org.springframework.http.converter.HttpMessageNotWritableException: Could not write JSON: EL1008E: Property or field 'providedBy' cannot be found on object of type 'cat.udl.eps.softarch.demo.domain.Column' - maybe not public or not valid?]
Solution: Change serialization by return resourceAssembler.toFullResource(mapping); to new ResponseEntity<>(m, HttpStatus.OK);


config option to flag a cidrs  peers not to be able to connect peers of the same cidr

I am not sure how to address this, i believe, the behavior is sort of intentional, but it might be also b.a.d...
Problem:
Peers of the same cidr can connect each other without further actions. It makes a lot of sense to allow this configuration on site2site vpns and also on cidrs for servers, as any service will have its own authentication, but not on human users local endpoint systems (classic name: roadwarriors).
I think, this is problematic as most devs see their local machine also as server for testing services locally and these may run without any authentication as they are local to the dev.
These services are exposed to the vpn, if they are not bound to a dedicated interface. Most services are really bound to <IP_ADDRESS> as default, if they are not databases which usually bind to <IP_ADDRESS> as default.
Even if the developer does not portforward her innernet port to her cablemodem/router: It is sufficient, if she initiates a connection to another vpn and keep it running to expose this port as it is udp.
Example:
Alice and Bob are humans and in the cidr humans. Machine-A is a service in the cidr svc. Humans cidr is allowed to connect to cidr svc.
Alice connects to machine-a (vpn peer, not in her cidr, but allowed)
Bob does inn list --tree and nmaps Alices ip address:
$ nmap <IP_ADDRESS>
Starting Nmap 7.80 ( https://nmap.org ) at 2021-04-23 08:56 UTC
Nmap scan report for alice.fcloud.wg (<IP_ADDRESS>)
Host is up (0.024s latency).
Not shown: 996 closed ports
PORT     STATE SERVICE
80/tcp   open  http
3000/tcp open  ppp
8086/tcp open  d-s-n
8888/tcp open  sun-answerbook

From Alices point of view (the nmap only catches tcp):
[0] % sudo netstat -tpaun |awk '{print $1,$4}' | grep -E '<IP_ADDRESS>|:::'
tcp <IP_ADDRESS>:80
tcp6 :::8086
tcp6 :::8888
tcp6 :::3000
tcp6 :::80
udp <IP_ADDRESS>:49029
udp <IP_ADDRESS>:68
udp <IP_ADDRESS>:5353
udp6 :::49029

In a classic (centralized) wireguard vpn network it is easy to prevent this by managing routing on the central server and sending human peer vpn config stencils maintaining the AllowedIPs netmask as /32.
To achive the same with innernet i would need to create a lot of /30 nets with one human user in each. And i would need to enable any single human peer instead of groups of users in the same cidr.
Please keep in mind that it will not help to setup local firewall rules against this on Alices machine as the traffic has already passed the external interface (is authenticated and decrypted) and than checked, if the source ip is allowed.
(https://www.wireguard.com/#cryptokey-routing)
As an admin i feel the need trying to protect my users as most as possible against unforeseen co-drivers on their personal hardware.
Kudos to the attendees in the discord discussion which made it possible to sharpen the request.
I propose to introduce a configuration flag (prevent inter-cidr communication or similar) to prevent that peers of a cidr can connect each other without further configuration.
At least we should add a remark to the security-recommendations chapter, but i feel that most classical users will not read it.

Thanks for the great issue!
The current behavior is very much intentional and is mentioned whenever a CIDR is created, but it was definitely designed with a certain scenario in mind where each CIDR was considered more like a traditional LAN and associations were between CIDRs were more like "site-to-site" connections.

Please keep in mind that it will not help to setup local firewall rules against this on Alices machine as the traffic has already passed the external interface (is authenticated and decrypted) and than checked, if the source ip is allowed.

I'm not sure I understand this part, local firewall rules should work for this use case. You can spin up a VM with an HTTP server listening on http://<IP_ADDRESS>:80, then we can setup a quick-and-easy UFW firewall configuration like so:
# Assuming you have an innernet called "evilcorp", configured to listen on 51820.
sudo ufw allow 51820/udp

Then try to reach that HTTP server from a connected peer, netfilter will deny.
Now add another rule:
sudo ufw allow in on evilcorp to any port 80 proto tcp

And the HTTP server should become reachable.

As an admin i feel the need trying to protect my users as most as possible against unforeseen co-drivers on their personal hardware.

Right - for innernet to run networks that have peers with varying levels of trust, we should reconsider this default. Perhaps it's best to, instead of a specific flag, have all CIDRs default to not exposing its own peers to each other, and then one can simply add an association between the CIDR and itself to have peers within that CIDR be visible to each other...

Sorry for the long time for that answer.
You are totally right with running firewall services on the users machine, my mistake (i made a typo in the input chain drop policy in my testing).
My point: A lot of enduser machines do not run (proper) local firewall services, they think to be hidden behind a NAT gateway with proper firewall configuration and they have not portforwarded their innernet port to that gateway.
Other peers are able to connect these machines.
For my perspective i am fine with not exposing peers of the same cidr to each other, it will fit to my ideas of grouping human and non human peers and connect them with little effort.

@bmullan yeah, the 51820/udp is for WireGuard, exposed to the internet, 80/tcp for the server that you are allowing to be visible only over the WireGuard network.

Wouldn't this be more directly accomplished by not having the members of the VLAN have each other as "peers" inside WireGuard?  (Then there's a natural "firewall" between them because they're not connected to begin with and just happen to share a CIDR in the larger IP space.)


Is it possible to run multiple eval_net.py process with multiple workers?

Hi guys.
When running the eval_net.py on the UCF-101 split_1 with 4 workers, I noticed that all my GPUs are running, but the GPU usage doesn't reach to the maximum. So I tried to run another eval_net.py process to evaluate the UCF-101 split_2, but I encountered lots of error messages like the following:
Setting device 1517
WARNING: Logging before InitGoogleLogging() is written to STDERR
F0908 02:12:50.717952  3673 common.cpp:187] Check failed: error == cudaSuccess (10 vs. 0)  invalid device ordinal
*** Check failure stack trace: ***
Setting device 1518
WARNING: Logging before InitGoogleLogging() is written to STDERR
F0908 02:12:51.023628  3675 common.cpp:187] Check failed: error == cudaSuccess (10 vs. 0)  invalid device ordinal
*** Check failure stack trace: ***

I am not sure these errors are caused by my machine(maybe my command was wrong)
Or the design of TSN doesn't allow doing this?

This error means you might be requesting more workers than the number of available GPU.
Also, you are running 4 workers using eval_net.py. I don't quite understand the title of this issue.

@yjxiong
Oh, I get it.
I have 4 GPU cores and have an eval_net.py process running with 4 workers already.
When I run the second process of eval_net.py with 4 workers, the total 8 workers exceeds the amount of my GPU cores and promtps the error messages.

It is OK to run two eval_net.py with 4 workers each. There might be some other error.

@yjxiong
Thanks a lot! I will try it again.

Hi @GBJim
I used to meet this problem too. Try adding CUDA_VISIBLE_DEVICES=0,1,2,3 before the python command, It works fine for me.

@xlliuplus7
Thanks for the tip!

It is strange. I did not add CUDA_VISIBLE_DEVICES=0,1,2,3 or anything else. I can still run two eval_net with 4 workers perfectly.

@xlliu7  http://stackoverflow.com/questions/34775522/tensorflow-mutiple-sessions-with-mutiple-gpus
you mean this ->$ CUDA_VISIBLE_DEVICES=0 python my_script.py  # Uses GPU 0. RIght?
But I tried, and get error:  error = cudaSuccess: (10 vs 0) invalid device ordinal.
Do you have any suggestions?

@KnightOfTheMoonlight  That's exactly what I mean.
I use a workstation with 4 GPUs + CUDA 7.5 + cuDNN v5. The toolbox was compiled with MPI support.
I found it neccessary to add the "CUDA_VISIBLE_DEVICES" environment to use multiple GPUs. Otherwise I will get the "invalid device ordinal" error. I guess the caffe toolbox fails to detect available GPUs on my workstation. Your problem seems to be opposite to mine.
In fact, you can also mannually specify the dev_id in "eval_net.py". See https://github.com/yjxiong/temporal-segment-networks/blob/master/tools/eval_net.py#L32


How do I use this program?

I'm a little bit confused with the basic usage of this program.
I'm not sure where you add the URL and how you run the program.
Can someone let me know where I insert and the URL(s), adjust output folders, and run the program?

Follow #30839 !
Read https://github.com/ytdl-org/youtube-dl#readme.
After installing the program, try this in your terminal:
youtube-dl --help
If you're not familiar with running command-line programs, use your preferred web search tool to find out more.

I appreciate the response. I've read through most of this information already and still can't find the solution. I'm not an expert, but I do have a fair amount of experience with command line.
I'm just trying to find basic information about how to run the program. This is what I've tried based on what is written in the description. youtube-dl [OPTIONS] URL [URL...]
eg.
youtube-dl [OPTIONS] URL https://www.youtube.com/watch?v=EdmOc6jna88&ab_channel=WhistlinDiesel
The return says "No matches found".
Can you let me know specifically what I'm doing wrong?

Those XXX are supposed to be place-holders that you fill in, optional when enclosed in [].
So here I'm using the OPTIONS -v -F (= --verbose --list-formats), and quoting the URL for POSIX (use "double quotes" for Windows cmd); also I'm running the yt-dl Python module (python -m youtube_dl) from my development path rather than running a youtube-dl command:
$ python -m youtube_dl -v -F 'https://www.youtube.com/watch?v=EdmOc6jna88&ab_channel=WhistlinDiesel'
[debug] System config: [u'--prefer-ffmpeg']
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'-F', u'https://www.youtube.com/watch?v=EdmOc6jna88&ab_channel=WhistlinDiesel']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Git HEAD: 32539c964
[debug] Python 2.7.18 (CPython i686 32bit) - Linux-4.4.0-210-generic-i686-with-Ubuntu-16.04-xenial - OpenSSL 1.1.1t  7 Feb 2023 - glibc 2.15
[debug] exe versions: avconv 4.3, avprobe 4.3, ffmpeg 4.3, ffprobe 4.3
[debug] Proxy map: {}
[youtube] EdmOc6jna88: Downloading webpage
[debug] [youtube] Decrypted nsig Bti3-WvuGAKIHDj => g-ctQGXSeduJ0Q
[debug] [youtube] Decrypted nsig M5c3sPaMjnwesOO => gCvPWJ3x4q0Jag
[info] Available formats for EdmOc6jna88:
format code  extension  resolution note
249          webm       audio only tiny   51k , webm_dash container, opus  (48000Hz), 3.86MiB
250          webm       audio only tiny   67k , webm_dash container, opus  (48000Hz), 5.10MiB
140          m4a        audio only tiny  129k , m4a_dash container, mp4a.40.2 (44100Hz), 9.76MiB
251          webm       audio only tiny  135k , webm_dash container, opus  (48000Hz), 10.18MiB
278          webm       256x144    144p   76k , webm_dash container, vp9, 30fps, video only, 5.78MiB
394          mp4        256x144    144p   76k , mp4_dash container, av01.0.00M.08, 30fps, video only, 5.79MiB
160          mp4        256x144    144p   91k , mp4_dash container, avc1.4d400c, 30fps, video only, 6.86MiB
242          webm       426x240    240p  149k , webm_dash container, vp9, 30fps, video only, 11.26MiB
395          mp4        426x240    240p  151k , mp4_dash container, av01.0.00M.08, 30fps, video only, 11.46MiB
133          mp4        426x240    240p  209k , mp4_dash container, avc1.4d4015, 30fps, video only, 15.81MiB
396          mp4        640x360    360p  329k , mp4_dash container, av01.0.01M.08, 30fps, video only, 24.86MiB
243          webm       640x360    360p  381k , webm_dash container, vp9, 30fps, video only, 28.79MiB
134          mp4        640x360    360p  472k , mp4_dash container, avc1.4d401e, 30fps, video only, 35.63MiB
244          webm       854x480    480p  536k , webm_dash container, vp9, 30fps, video only, 40.45MiB
397          mp4        854x480    480p  622k , mp4_dash container, av01.0.04M.08, 30fps, video only, 46.96MiB
135          mp4        854x480    480p  795k , mp4_dash container, avc1.4d401f, 30fps, video only, 59.95MiB
398          mp4        1280x720   720p 1192k , mp4_dash container, av01.0.05M.08, 30fps, video only, 89.89MiB
247          webm       1280x720   720p 1195k , webm_dash container, vp9, 30fps, video only, 90.10MiB
136          mp4        1280x720   720p 1297k , mp4_dash container, avc1.4d401f, 30fps, video only, 97.84MiB
248          webm       1920x1080  1080p 2190k , webm_dash container, vp9, 30fps, video only, 165.10MiB
399          mp4        1920x1080  1080p 2458k , mp4_dash container, av01.0.08M.08, 30fps, video only, 185.36MiB
137          mp4        1920x1080  1080p 4082k , mp4_dash container, avc1.640028, 30fps, video only, 307.76MiB
400          mp4        2560x1440  1440p 7101k , mp4_dash container, av01.0.12M.08, 30fps, video only, 535.34MiB
271          webm       2560x1440  1440p 8627k , webm_dash container, vp9, 30fps, video only, 650.44MiB
401          mp4        3840x2160  2160p 14519k , mp4_dash container, av01.0.12M.08, 30fps, video only, 1.07GiB
313          webm       3840x2160  2160p 17489k , webm_dash container, vp9, 30fps, video only, 1.29GiB
18           mp4        640x360    360p  734k , avc1.42001E, 30fps, mp4a.40.2 (44100Hz), 55.35MiB
22           mp4        1280x720   720p 1426k , avc1.64001F, 30fps, mp4a.40.2 (44100Hz) (best)
$

To obtain a version that actually works (as above) with YouTube, review #31530.

@Russellva ?

Why would you open an issue for this? Read the docs

@Russellva can this issue be closed?

OP hasn't responded to a ping from many weeks ago.


Add a test to verify that long periods of silence don't break the connection

+1, when do you plan to run it (I see it is ignored for now)?

+1, when do you plan to run it (I see it is ignored for now)?

It is being run on CI but all the integration tests are disabled by default: https://github.com/NordSecurity/boringtun/actions/runs/7407781146/job/20154641562?pr=14#step:4:516


docker files

This lets one get going with a single command:
docker compose  --env-file .env.local up

Only thing that's different is that .env.local needs to point to api via container name instead of localhost: eg DUST_API=http://dust-api:3001

I realized that for some reason I need both the env file and inline env vars for dust to work. Don't gonna merge until I figure out what's going on.
Will investigate tomorrow.

Roger that :+1:


[doc] aufs must not be incompatibile with itself

Select a storage driver guide (https://docs.docker.com/engine/userguide/storagedriver/selectadriver/) has a following table.

Storage driver
Must match backing filesystem
Incompatible with

overlay
No
btrfs aufs zfs

aufs
No
btrfs aufs

btrfs
Yes
N/A

devicemapper
No
N/A

vfs
No
N/A

zfs
Yes
N/A

But aufs must not be incompatible with itself. I suspect the auther intended to:
|aufs         |No                            |btrfs overlay      |
Thought?

No, @kosaki
From https://github.com/docker/docker/blob/master/daemon/graphdriver/aufs/aufs.go#L98
you could check that, aufs can not run on top of aufs.(it means incompatible here)

Oh, I misunderstood what the table mean. Please close this ticket then.

Correct, running layered filesystems on top of layered filesystems (e.g. when running docker-in-docker) is known to cause issues.
Looking at the table, I think overlay should also be added to that table (i.e. don't run overlay on overlay, or aufs on overlay)


rs-viewer bug

When pausing a .bag file on RS Viewer, the 3D vision does pause. But then, when pressing the play button, I have to first press the 3D play button and then the .bag button, because if I just press the general .bag play button the 3D vision won't work. It would be a lot easier if they could be synchronized.

@amaanda Which librealsense version did you get the issue? And could you please provide a snapshot with the mark of the buttons you mentioned so that we can understand the problem more clearly? Thanks!

@RealSenseCustomerSupport I am using 2.19.0.
The red circled button is the general play button, and the blue circled one is the "3D play button".

@amaanda I tried v2.20 and can see that the frame for the pause moment was still there after click general play button. This pause frame can be cleared after click "3D play button". Is that the problem what you see?

@RealSenseCustomerSupport Yes, it is

@amaanda Got your point now. We will look into this and update when get fix. Thanks!

Pending on JIRA DSO-12584. No update yet.

Hi @amaanda
the issue fixed on:
https://github.com/IntelRealSense/librealsense/pull/7700

@amaanda The fix in https://github.com/IntelRealSense/librealsense/pull/7700 now merged to development branch. Could you please have a try? Thanks!

Hi @RealSenseSupport I will try it as soon as possible. Thanks! =)

@amaanda SDK v2.40 includes the fix for the issue and released now. Please try it with RealSenseViewer. Thanks!

@amaanda As the issue resolve, closed the issue accordingly. Thanks!


add problem to find roots of quadratic equation

made the changes, tested with some examples, also resolved the conflict