CDN-in-a-Box now caches Carton dependencies by default

What does this PR (Pull Request) do?

[x] This PR is not related to any Issue

This moves some stuff around in the CDN-in-a-Box Traffic Ops (Perl) Dockerfile. It copies the source traffic_ops/app/cpanfile file into the container before importing the RPM, and will download Traffic Ops's Carton dependencies before installing it. This means that changes to the Traffic Ops RPM won't trigger downloading Carton dependencies, thus significantly reducing the time it takes to rebuild the image after making changes. It also pre-installs the dependencies currently listed in the Traffic Ops spec file to make that installation a little faster.
This does make the cached layers of the build bigger, but the resulting image should not actually be any larger. Hopefully, for frequent re-builders, the cache space can be easily reclaimed with docker prune or similar.
Previously, something similar could be accomplished by keeping a local copy of the dependencies tarballed and gzipped in a special place (infrastructure/cdn-in-a-box/traffic_ops/local/tmp.tgz, I think?) and the build process would load that data if it exists. However, in order to make use of that feature, the user must first have knowledge of the install process of Traffic Ops, as well as requiring them to have both a Perl interpreter and Carton and their associated dependencies. This runs counter to the philosophy of self-containment that Docker promotes, and could be difficult for, say, Windows users who want to run CDN-in-a-Box. It also runs somewhat counter to the philosophy of CDN-in-a-Box, in my opinion, in that a user should be able to just "press a button" and have a Traffic Control ecosystem spring to life. So I do believe that changing the system in this way is a worthwhile improvement.
Which Traffic Control components are affected by this PR?

CDN in a Box

Feature change is an implementation detail - shouldn't be documented. I looked for documentation on the "old" way of using locally-cached dependencies, but I couldn't find anything.
What is the best way to verify this PR?
Build and run CDN-in-a-Box to verify that Traffic Ops is properly built/installed and operates as normal. No tests because it only affects CDN-in-a-Box, which has no testing.
The following criteria are ALL met by this PR

[x] I have explained why tests are unnecessary
[x] I have explained why documentation is unnecessary
[x] An update to CHANGELOG.md is not necessary
[x] This PR includes any and all required license headers
[x] This PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3978/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3987/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3990/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3991/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4030/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4038/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4059/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4104/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4139/
Test PASSed.

I don't know how I missed this when it came in, but I just want to say, this is awesome. Should make builds drastically faster.
We should considering the Carton vendoring in TO's RPM, if it's possible, so production deployments would get it for free too. Maybe even vendoring it in Git, so even the RPM build is that much faster.

From what I understand, including the Perl modules was considered before this project became open source,  but the legal dept nixed it due to various licenses, etc needing to be considered.  This happened before I joined the project, but that's the history....


PDF preview not working

Microsoft PowerToys version
0.73.0
Installation method
Microsoft Store
Running as admin
None
Area(s) with issue?
File Explorer: Preview Pane
Steps to reproduce
PowerToysReport_2023-09-13-19-36-48.zip

System

Provider

[ Name]  Application Error
[ Guid]  {a0e9b465-b939-57d7-b27d-95d8e925ff57}
EventID 1000
Version 0
Level 2
Task 100
Opcode 0
Keywords 0x8000000000000000

TimeCreated

[ SystemTime]  2023-09-13T11:36:22.5386311Z
EventRecordID 93435
Correlation

Execution

[ ProcessID]  48396
[ ThreadID]  48640
Channel Application
Computer GuJialu-ROG

Security

[ UserID]  S-1-5-21-342916336-2928179877-3793502957-1001

EventData
AppName PowerToys.PdfPreviewHandler.exe
AppVersion <IP_ADDRESS>
AppTimeStamp 64b07659
ModuleName coreclr.dll
ModuleVersion 7.0.1023.36312
ModuleTimeStamp 64b06d6c
ExceptionCode c0000005
FaultingOffset<PHONE_NUMBER>1c98a6
ProcessId 0x28c4
ProcessCreationTime 0x1d9e63685025e09
AppPath C:\Program Files\PowerToys\PowerToys.PdfPreviewHandler.exe
ModulePath C:\Program Files\PowerToys\coreclr.dll
IntegratorReportId 319e00c6-224b-41d9-90c6-b4e0327f794d
PackageFullName
PackageRelativeAppId

✔ Expected Behavior
Showing the pdf file preview in the preview panel.
❌ Actual Behavior
Completely blank
Other Software
No response

Experienced the same problem before. You might want to disable "Always run as administrator" in PowerToys. I guess this issue persists cause PowerToys is running an elevated process.

/dup https://github.com/microsoft/PowerToys/issues/29916

https://github.com/microsoft/PowerToys/issues/28572 has a sample PDF

/dup https://github.com/microsoft/PowerToys/issues/29644


CommandToExecute-Must-Use-ProtectedSettings flags nested deployments

The test looking for secrets in the commandToExecute for VM extensions needs to more restrictive query on Find-JsonContent - we need to ensure the property is in a vm extension - otherwise we flag things like nested deployments that pass the cmd through to the extension.

To narrow the search we should look in the vm extensions:
https://github.com/Azure/azure-quickstart-templates/blob/master/intel-lustre-clients-vmss-centos/azuredeploy.json#L338-L345
When we find commandToExecute, there must be a type property with the value of "CustomScript"
This should catch VMs/VMSSs as well as nested vs top-level child resources (since we don't go too far up the tree/path)


Can lead to confusion

In the page it reads:
"Your account must meet one of the following to enable traffic analytics:
Your account must have any one of the following Azure roles at the subscription scope: owner, contributor, or network contributor."
Above that statement it reads:
"Traffic analytics requires the following prerequisites:
[..]
An Azure storage account, to store raw flow logs.
An Azure Log Analytics workspace, with read and write access."
What I believe can bring confusion is that unless another higher role (i.e. Owner, Contributor) has already created the needed storage account and log analytics workspace, a network contributor by itself will not be able to enable the Traffic Analytics.

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

ID: aa32b540-55b2-1691-6f38-e8687565cecc
Version Independent ID: 8ad04d57-6370-c8dd-1049-1826a823bd12
Content: Traffic analytics FAQ - Azure Network Watcher
Content Source: articles/network-watcher/traffic-analytics-faq.yml
Service: network-watcher
GitHub Login: @halkazwini
Microsoft Alias: halkazwini

@SQLtattoo
Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

@SQLtattoo
Thank you for your feedback!
You're absolutely right network contributor role:
I removed the storage account from the prerequisites as it a prerequisite for flow logs not traffic analytics itself.
I added Microsoft.OperationalInsights/workspaces/*.
The updates will be reflected in the public do in a few hours.  I'll close this GitHub issue for now, but feel free to comment here as necessary.
#please-close


"Unexpected error"

Have a good day and a happy new year.
Trying to connect to Greek version of smart 3 always get error "Unexpected error"
Router ip is on 192.168.3.* and the rest of my network setup including home assistant is on 192.168.1.* so I don't know if this could explain the problem.
Here is the log.
`This error originated from a custom integration.
Logger: custom_components.speedport.config_flow
Source: custom_components/speedport/config_flow.py:64
Integration: Speedport (documentation, issues)
First occurred: 00:51:55 (9 occurrences)
Last logged: 09:30:58
Unexpected exception
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/speedport/connection.py", line 21, in decode
ciphertext_tag = bytes.fromhex(data)
^^^^^^^^^^^^^^^^^^^
ValueError: non-hexadecimal number found in fromhex() arg at position 0
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/config/custom_components/speedport/config_flow.py", line 64, in async_step_user
info = await validate_input(self.hass, user_input)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/speedport/config_flow.py", line 31, in validate_input
speedport = await Speedport(
^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/speedport/speedport.py", line 60, in create
await self.update_status()
File "/usr/local/lib/python3.11/site-packages/speedport/speedport.py", line 118, in update_status
self._status = await self.api.get_status()
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/speedport/api.py", line 116, in get_status
return await self.api.get("data/Status.json")
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/speedport/connection.py", line 97, in get
return decode(await response.text(), key=key)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/speedport/connection.py", line 23, in decode
raise exceptions.DecryptionKeyError("Wrong decryption key") from exc
speedport.exceptions.DecryptionKeyError: Wrong decryption key
`
Thank you for your time

I don't think so because remote management need to define a custom port and there is no option to select a port to the intergration.Hopefully Andre will come back to this.
Ps.Same provider and same fw version with you chris

Same problem here.
Provider: Telekom
Router: Speedport Smart 4 Typ B
Firmware-Version: 010<IP_ADDRESS>.0

I'm getting the same error in the logs.
Router: Speedport Smart 3
Firmware-Version: 010<IP_ADDRESS>.0 (no new updates available for me)

Same here
Router: Speedport Smart 3
Firmware-Version: 010<IP_ADDRESS>.0

Hey, please check if you have a different value for keyArrayDefault in the jquery functions encryptccm() or decryptccm() in your http://speedport.ip/js/jquery-addons.js
For me and many other it looks like this:
var keyArrayDefault = "cdc0cac1280b516e674f0057e4929bca84447cca8425007e33a88a5cf598a190";

so I use it as a static value.

@Andre0512
I've checked and the value is exactly the same:
var keyArrayDefault = "cdc0cac1280b516e674f0057e4929bca84447cca8425007e33a88a5cf598a190";

My values are also in both functions the same

Any progress on this matter? I get the same error trying to connect to the Greek version of the Smart 3, fw version 02.10.34.
This error originated from a custom integration.
Logger: custom_components.speedport.config_flow
Source: custom_components/speedport/config_flow.py:65
integration: Speedport (documentation, issues)
First occurred: October 27, 2024 at 11:21:54 PM (10 occurrences)
Last logged: 12:19:32 AM
Unexpected exception
Traceback (most recent call last):
File "/usr/local/lib/python3.12/site-packages/speedport/connection.py", line 21, in decode
ciphertext_tag = bytes.fromhex(data)
^^^^^^^^^^^^^^^^^^^
ValueError: non-hexadecimal number found in fromhex() arg at position 0
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/config/custom_components/speedport/config_flow.py", line 65, in async_step_user
info = await validate_input(self.hass, user_input)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/speedport/config_flow.py", line 32, in validate_input
speedport = await Speedport(
^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/speedport/speedport.py", line 60, in create
await self.update_status()
File "/usr/local/lib/python3.12/site-packages/speedport/speedport.py", line 118, in update_status
self._status = await self.api.get_status()
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/speedport/api.py", line 116, in get_status
return await self.api.get("data/Status.json")
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/speedport/connection.py", line 105, in get
return decode(await response.text(), key=key)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/speedport/connection.py", line 23, in decode
raise exceptions.DecryptionKeyError("Wrong decryption key") from exc
speedport.exceptions.DecryptionKeyError: Wrong decryption key


Teams app uploaded but nothing happening

Hello
I configure everything but after app is deployed on BTP and uploaded to teams nothing happening - no landing page and so on... how I can see any logs to see where is the problem ?
Best
Denkov

Hi Denkov,
First, I want confirm if you used the docker container with automated setup. If that is the case, there is a logs/ folder within the container that could help pinpoint the error.
However, if everything was setup via the automated setup without errors, then this is likely a .env issue or Azure resource issue. Either way (and even if manual setup was used), the logger tied to each application should provide some information on what went wrong. My suspicion would be the backend logs would be more useful. You should be able to see the logs of an application by navigating to the space level. From there, select the backend application, and you should see a logs section on the left side panel - click on that.
Do you receive any errors when trying to open the app in teams after the upload to teams? Please check the network tab from dev tools as well, this may provide useful information for debugging.
Happy to help with any follow-up questions, but please be careful if you are pasting anything in - some of the logs and errors contain sensitive information.
Best,
Alex

Hello
I use automation setup tool and everything is fine, but after deployment on teams I didnt see that page ->

And we don't have S4 so do you have any hint how I can mock it and use the app just for test (demo)?
I want to make simple action by entering any value and show something in output.
Best,
Denkov

Hi Denkov,
Unfortunately, there is no way to mock the app without the connection to S4. We strongly recommend working with an S/4HANA system admin to setup the ABAP job, destination services, and OAuth configuration of event mesh in the S/4HANA system.
Alternatively, Bridge supports connections to C4C and SFSF, we would be happy to help you set up either of those situations, provided you have access to either of them. Either of these cases would allow you to enter values and show outputs.
Best,
Alex


Add gettid

I tested this under linux, and I noticed that this seems to also be built for OSX. It would be appreciated if someone could test this under OSX.
I'm not familiar enough with rust to know if there is a way of integrating this without creating a sub-crate.

Although OSX has gettid, it appears to be not the same thing and I question whether we should expose it for OSX at all: http://elliotth.blogspot.com/2012/04/gettid-on-mac-os.html.
gettid should be exposed from libc.  We should also update unistd to use the libc getpid and getppid.

It should be noted that libc does expose pthread_self(): https://doc.rust-lang.org/libc/x86_64-unknown-linux-gnu/libc/fn.pthread_self.html
Use of pthread_self is probably preferable in many cases, especially where some degree of portability is desired (there are still gotchas).  It might be reasonable for nix to be extended to include a wrapping of pthread that adds the typical nix niceties.
That being said, I can still imagine use cases where getting access to gettid() would be nice.  I think it should be added for Linux in libc -- it should also be a direct syscall as glibc does not provide any wrapper according to the man pages (http://man7.org/linux/man-pages/man2/gettid.2.html).

I think it should be added for Linux in libc -- it should also be a direct syscall as glibc does not provide any wrapper according to the man pages (http://man7.org/linux/man-pages/man2/gettid.2.html).

I am not sure it can be added to libc as a function in that case. The syscall number could and should be added though. We have a similar situation with pivot_root(2), where nix wraps a call to syscall(2).

We should also update unistd to use the libc getpid and getppid.

Ah I might have found a thing to do on this lazy Saturday afternoon. At least until I go ice skating... :-)

It turns out I am blind.  We are already using the libc getpid/getppid.  More time for skating I guess.

I'm happy to only expose gettid for linux and not for OSX, I just don't know how to do that on the rust side.
The SYS_gettid constant is different for different platforms (i.e. arm, 32-bit, and 64-bit x86 all have different constants for SYS_gettid) otherwise I could have done the syscall from rust. Maybe there's a way to create rust constants?

And @kamalmarhubi is correct, glibc doesn't have a gittid function to expose, which is why the syscall needs to be done directly. Personally, I find it exremely useful to have the tid exposed as shows up in the ps listing (i.e. ps -T shows both pids and tids) and pthread_self would not be useful in this situation. Or for examining /proc/PID/task/TID.

@dhylands

The SYS_gettid constant is different for different platforms (i.e. arm, 32-bit, and 64-bit x86 all have different constants for SYS_gettid) otherwise I could have done the syscall from rust. Maybe there's a way to create rust constants?

The right thing to do would be put them in the libc hierarchy, one of the files in here:

https://github.com/rust-lang/libc/tree/master/src/unix/notbsd/linux/other/b32
https://github.com/rust-lang/libc/tree/master/src/unix/notbsd/linux/other/b64

@dhylands

I'm happy to only expose gettid for linux and not for OSX, I just don't know how to do that on the rust side.

The pattern is something like this:
#[cfg(target_os = "linux")]
pub fn gettid() -> pid_t {
   unsafe { syscall(...) }
}

Ahh - ok - I'll have a go at adding SYS_gettid there.

@dhylands :+1:

:umbrella: The latest upstream changes (presumably #294) made this pull request unmergeable. Please resolve the merge conflicts.

I just got SYS_gettid merged into rust-lang/libc (https://github.com/rust-lang/libc/pull/209), so I need to rework this anyways, since I can now do a rust-only solution.

:+1: :-)

Well done :+1:

I've updated the PR, but it can't be merged until lib releases 0.2.8 (and then I'll need to update the Cargo.toml file as well). It also looks like I got the type wrong on SYS_gettid, so I put through a libc PR to fix that: https://github.com/rust-lang/libc/pull/213
I can build nix-rust locally (by redirecting libc to my local copy).
I had a question about the "as i32" and whether I should use as i32 or as pid_t. I'll guess I should change it to pid_t but I thoght I would find out what others thoughts were.

Update PR to use libc 0.2.8 and the one test that doesn't pass seems unrelated (it's for darwin and this PR should only affect linux). The same build on my repository was green:
https://travis-ci.org/dhylands/nix/builds/114422104
This PR adds a gettid by doing  a syscall using libc::SYS_gettid

@dhylands thanks! You're right, test_sigwait is flaky. I'll make a PR in a second to disable it, as it fails a bit too often...

Just a minor change to the conditional compilation, and this is good to go! Thanks!

ok - fixed up the conditionals. I had to make a few other changes as the tests started to fail in other mysterious ways, so I followed a pattern I found elsewhere in the test files.
The only failure is the test_semwait on darwin.

Made the test retry and it works. @homu r+

:pushpin: Commit ca75212 has been approved by kamalmarhubi

Thanks @dhylands!!

:zap: Test exempted - status

@dhylands

I had to make a few other changes as the tests started to fail in other mysterious ways, so I followed a pattern I found elsewhere in the test files.

What do you mean by this? Sounds like something we should document if it wasn't clear.

Of course, I tried to go back and reproduce the problem and can't any more.
I was seeing local failures (when running cargo test locally) related to wait and execvpe. It was probably something stale in my build or tree or something.

The wait and execvpe problems are related to thread parallelism and should not happen with RUST_TEST_THREADS=1.  Unfortunately, I am not aware of a way to set this up so when a developer runs cargo test things are automatically set up this way.

I've got some ideas for fixing the tests (#251)

Excellent, look forward to seeing what you come up with.  As an aside, I put out a feeler in a reddit thread for suggestions on testing out systems code in rust without a good mocking framework: https://www.reddit.com/r/rust/comments/494tkf/looking_for_suggestions_mocking_external_functions/
Calling the actual system calls only lets us test the happy path (and even then not always as consistently as we would like).

Moving conversation to #251


Start with Existing Git Repo

I am having trouble getting started with git-tfs.  I have an existing Git repo and I want to push it to a new TFS Team Project.  How do I do that?  All of the documentation explains how to do just the opposite.

Yes, indeed because that's not possible and git-tfs has not been created
for that.
The way TFVC works make it nearly impossible to do it :-(
Perhaps you could have a look at git-tf and you should be able to push your
main branch (but only one...)

Ok, thank you so much for the information.  I will give git-tf another try.  Receiving an error when I attempt to configure and will post a question over there.

That 's one of the problem of git-tf, it is not maintained anymore, so I think that you are on your own.
But I really don't understand the goal of migrating from git (quite better!) to TFVC (very painful and frustrating!). Why don't migrate to tfs git?
And don't tell me that it's a choice made by the managers :-(

I am also on the same boat here, guessing I need to scrap my git history and start from scratch :( corporate management ftw.

Look at git-tf or at a script I wrote (for fun) to be able to rcheckin an history with git-tfs :
https://gist.github.com/pmiossec/2421ffcc8011e873641b
But in the 2 cases, you will only be able to push your master branch only without branches nor merges!

Thanks, much appriciated!

I didn't remember that I wrote a post about that...
http://stackoverflow.com/questions/15753712/git-tfs-starting-from-git/15769232#15769232


feat: add erg

@mason-org/triage notify(Reason::NewPullRequest)
This comment ensures a notification is delivered to maintainers.
 🤖 This is an automated comment.    📖 Source code


Installation issue

Why I'm getting below error, please kindly help, btw what's the name of the conda new env?
(replica) PS D:\code\MegFaceAnimate> conda env create -f environment.yaml
Collecting package metadata (repodata.json): done
Solving environment: failed
ResolvePackageNotFound:

ld_impl_linux-64==2.38=h1181459_1
tbb==2021.8.0=hdb19cb5_0
libuuid==1.41.5=h5eee18b_0
cudatoolkit-dev==11.7.0=h1de0b5d_6
mpc==1.1.0=h10f8cd9_1
gmp==6.2.1=h295c915_3
jinja2==3.1.3=py310h06a4308_0
readline==8.2=h5eee18b_0
libcufft==<IP_ADDRESS>=h4fbf590_0
xz==5.4.6=h5eee18b_0
pytorch-cuda==11.7=h778d358_5
mpfr==4.0.2=hb69a4c5_1
markupsafe==2.1.3=py310h5eee18b_0

You on windows or linux?

@Inferencer
He is using Windows.
I have the same question, is it going to work on Windows?

@Inferencer He is using Windows.
I have the same question, is it going to work on Windows?

so when working woth windows you will need to remove the extended version so it will be
ld_impl_linux-64==2.38
tbb==2021.8.0
libuuid==1.41.5
cudatoolkit-dev==11.7.0
mpc==1.1.0
gmp==6.2.1
jinja2==3.1.3
readline==8.2
libcufft==<IP_ADDRESS>
xz==5.4.6
pytorch-cuda==11.7
mpfr==4.0.2
markupsafe==2.1.3

this will install most of them but there might be some stuff you need to find a windows version for for instance
      ld_impl_linux-64==2.38
given that this has Linux in the words and i cant find a windows a windows package (only did a quick search) i suggest you use WSL to run a linux env on windows

@Inferencer He is using Windows.
I have the same question, is it going to work on Windows?

Thank you for your work on this. I would like to mention that a lot of users use Windows and they miss out on these tools which are meant primarily for Linux. The main reason being not everyone is Technical.
I would request if there is any possibility to support this on Windows in future will be helpful.

Thx for trying our work!
Sorry for the confusion, we have marked the conda config was intended for a Linux setup.
We plan to work on a Windows setup after releasing data cuartion and training script stuff, may not be very soon as code optimization may take some time.
Afaik the python package part should be the same for a windows setup, so removing some certain Linux libaries should work on Windows, not sure cause we don't have a windows setup at hand right now.

Thanks for your work.
Maby a installer for Pinokio would help all the windows and mac users.
https://github.com/pinokiocomputer/pinokio

https://github.com/megvii-research/megactor/issues/17#issuecomment-2440905459
root@autodl-container-af5f4cadf9-d25db46f:~/autodl-tmp/megactor-main_another/megactor# cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04.5 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.5 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy

it work well


SQLAlchemy upgrade required to fix security vulnerability.

Known moderate severity security vulnerability detected in SQLAlchemy < 1.3.0defined in setup.py.
setup.py update suggested: SQLAlchemy ~> 1.3.0.
NOTE: We don't run raw SQL queries via the IAM explain feature that uses the underlying sqlalchemy library. We feel we are not vulnerable to the injection attack.

Will this change be back ported to previous releases?

Based on release notes I believe you need version 1.3.0b3 - see release notes here: https://docs.sqlalchemy.org/en/13/changelog/changelog_13.html#change-1.3.0b3

Adding CVE details:
CVE-2019-7548 More information
moderate severity
Vulnerable versions: < 1.3.0
Patched version: 1.3.0
SQLAlchemy 1.2.17 has SQL Injection when the group_by parameter can be controlled.
CVE-2019-7164 More information
moderate severity
Vulnerable versions: < 1.3.0
Patched version: 1.3.0
SQLAlchemy through 1.2.17 and 1.3.x through 1.3.0b2 allows SQL Injection via the order_by parameter.

Based on release notes I believe you need version 1.3.0b3 - see release notes here: https://docs.sqlalchemy.org/en/13/changelog/changelog_13.html#change-1.3.0b3

I saw this page but I was hesitant to upgrade to 1.3.0b2 as it is in beta.

Understood. My intent was to point out that 1.3.0b3 is the minimum version where the fix was applied. I would suggest using a stable version either 1.3.1 or taking the latest 1.3.4 version.

We don't run raw SQL queries via the IAM explain feature that uses the underlying sqlalchemy library. We feel we are not vulnerable to the injection attack.

Can we address the issue #2750 at the same time we address the upgrade of sqlalchemy in this issue please.

These are two separate issues and should be addressed separately. #2749 is about SQLAlchemy security vulnerability and #2750 is about CloudSqlAclScanner not working.

Just dismissed GitHub advisory on this topic for our repo (https://github.com/forseti-security/forseti-security/network/alert/setup.py/SQLAlchemy/closed).
We will be updating to 1.3 of this module as part of Python 3 migration.
NOTE: We don't run raw SQL queries via the IAM explain feature that uses the underlying sqlalchemy library. We feel we are not vulnerable to the injection attack.

We don't run raw SQL queries via the IAM explain feature that uses the underlying SQLAlchemy library. The only access to call those SQL queries is via GRPC to the Forseti server, and there is no internet exposure.
We feel we are not vulnerable to the injection attack.


[AVM Module Issue]: Support the MongoDB API and missing outputs AZURE_COSMOS_CONNECTION_STRING,AZURE_COSMOS_DATABASE_NAME

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Feature Request
(Optional) Module Version
No response
(Optional) Correlation Id
No response
Description
Please support the MongoDB API and add the outputs AZURE_COSMOS_CONNECTION_STRING,AZURE_COSMOS_DATABASE_NAME.
For example (azd):

MongoDB API:
https://github.com/Azure/azure-dev/blob/main/templates/common/infra/terraform/core/database/cosmos/cosmos.tf#L28

Outputs:
https://github.com/Azure/azure-dev/blob/main/templates/common/infra/terraform/core/database/cosmos/cosmos_output.tf#L1
https://github.com/Azure/azure-dev/blob/main/templates/common/infra/terraform/core/database/cosmos/cosmos_output.tf#L6

CC: @jongio and @bryansan-msft

@bryansan-msft  The two missing outputs AZURE_COSMOS_CONNECTION_STRING and AZURE_COSMOS_DATABASE_NAME have not been fixed yet. Please review and fix.

@Menghua1 you have those two outputs values already in the existing outputs. There is no reason to duplicate them.

@Menghua1 you have those two outputs values already in the existing outputs. There is no reason to duplicate them.

Do you mean the existing output resource? According to the Terraform AVM guide, specialized exposed properties should be output instead of the entire resource object.

That is weird because when i released the first module version I had that but I was forced by the AVM reviewer to change it to expose the whole objects

@jongio Any ideas on the above comments?

I can change it when i have a free timeslot, in fact i would rather have control of the outputs but I still comment it because I was forced to do it in this current way

@bryansan-msft Additionally, the existing output resource contains the output AZURE_COSMOS_CONNECTION_STRING, and the AZURE_COSMOS_DATABASE_NAME(azurerm_cosmosdb_mongo_database.mongodb.name) has not been output yet.

@bryansan-msft - It is my understanding that the issue hasn't been resolved because of the additional outputs needed.  There could be a misunderstanding on how it was resolved on your end.  Would you be able to point us in the right direction to get those outputs?

@jongio as @Menghua1 mentioned the module is not following the AVM rules regarding outputs and not only that the database name output is missing for mongodb as I forgot to output it
The connection string can be currently retrieved using the "resource" output similar to the sql one if you are already using that API in this module
As soon as I have a free time slot I will have a look into the outputs

I have just merged a PR with the outputs refactor to follow the AVM guidelines + expose the mongodb missing outputs


Update to work with Python 3

HashTag-py.txt
I cleaned up the code and now it works with Python 3 on Windows. This is my fist time doing this, if I screwed up, let me know.

Well,sadly,  this project seems abandoned...


Use edeliver to deploy

Prod is already using this.

 [Change deploy system] Remove gatling from the API

The current commit only took out some unneeded configs, and put some instructions in the README.
Merging!


Typescript support

[X] Test for TypeScript app
[X] Modify build to support TypeScript
[x] Use tsd to automatically pull in references
[x] Test for TypeScript modules, controller and route
[x] Correct implementation of route utils
[ ] Correct TypeScript templates
[x] Support other types

FYI, I've been using TypeScript in ng-poly in production for a while without major problem. Obviously, the module system can be improved but it is not a major problem for us.

That's great to hear. Do you guys have any suggestions for improving the modules?
The Gulpfiles have been updated to build the app and tests in a tmp directory for unit tests. The templates are updated. Unfortunately, there is an issue with angular-protractor and jQuery conflicting (https://github.com/borisyankov/DefinitelyTyped/issues/2734) for e2e tests.

Awesome! Thanks a lot! I will update our build to the latest version.


Rename public_trade_lists to public_trades_list for better clarity

What's needed?
Some properties should be renamed for better clarity.
Proposed solution

// ...

// ListPublicTradesResponse is a message that contains a list of historic public
// trades.
// This dataset is vital for tasks such as training machine learning models,
// backtesting trading strategies, and conducting market analysis.
message ListPublicTradesResponse {
  // List of all public trades that met the specified filtering criteria.
- repeated PublicTrade public_trade_lists = 1;
+ repeated PublicTrade public_trades_list = 1;

  // Metadata for pagination, including token for next page to retrieve.
  frequenz.api.common.v1.pagination.PaginationInfo pagination_info = 2;
}

// ...

Use cases
No response
Alternatives and workarounds
No response
Additional context
No response

This change doesn't pass protolint with the following error message:
Repeated field name "public_trades_list" must be pluralized name "public_trades_lists"

My suggestion would be to remove the lists suffix at all because that fact that this is a list is part of the type and as such visible for the user.

Done in #47


chinese pre-trained model

hi, could you provide a chinese pre-trained model, the training cost is too high...

Thanks for the suggestion. We will add the action item to the road map.


Case sensitivity

This is a fairly small thing, but I ran into this on Linux - and it may impact Mac users as well:
$> fw1 create app fooApp subsystem
ERROR
The app skeleton [/home/chip/.CommandBox/cfml/modules/fw1-commands/resources/skeletons/subsystem.zip] doesn't exist. Valid options are Subsystem, Examples, Skeleton, Basic, Subsystem-Legacy.

It worked when I re-ran the command and updated the case to:
$>  fw1 create app fooApp Subsystem
Easy enough to fix here, but case sensitivity may be an issue elsewhere.

Nice catch. I'll look it over and get that straightened out.

This should be resolved now from this commit https://github.com/framework-one/fw1-commands/commit/129e009a5ddc607ec911518e3845b6730a0ed564


Change build script references from vcruntime140 to vcruntime140_1

Unreal no longer uses the vcruntime140.dll in a server build and therefore this page is out of date. Unreal Server builds will not run in PlayFab containers unless vcruntime140_1.dll is referenced. The fix is to change the Server.build.cs reference to use vcruntime140_1.dll rather than vcruntime140.dll.

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

ID: ba6ecd70-64ec-e730-e601-d9c378821a79
Version Independent ID: e6b81f00-5381-e1a1-dd24-976bc99723c2
Content: Integrating the PlayFab GSDK into Unreal Engine 4 - PlayFab
Content Source: playfab-docs/features/multiplayer/servers/playfabgsdk_guide.md
Product: playfab
GitHub Login: @joannaleecy
Microsoft Alias: joanlee

Sorry for the incredibly long delay in responding. I imagine this was a problem when reported, and thank you for taking the time to do so. The page no longer exists and since I am not finding other similar references in this repo, I will close the issue.


Is there an architecture overview of Noms?

I am trying to find documents  of the core data structure in Noms. And, I found some docs  in the "doc" directory, however, it has no graph illustration.
Could you please provide some graph illustration of Nom's core data structrue. I think it will be better for others to understand how Noms works.
Thanks very much.

The architectural overview is here:
https://github.com/attic-labs/noms/blob/master/doc/intro.md
I agree adding illustrations would be helpful. Patches welcome!


Understanding KLLoss?

Hello,
Can you explain the way you calculated KLLoss in VAE? Isn't this a KL Divergence?
Thanks.

Yes this is just the KL divergence, and Wikipedia shows the general form for multivariate Gaussians, which can be simplified for the prior of N(0, 1). Unfortunately I'm not sure I'm able to give much more of an explanation, but perhaps the Tutorial on Variational Autoencoders will be helpful.

Closing (due to a month of inactivity).


Rpi 3 instructions

Readme update for running on pi 3

Before approve this...i think is better to write few words about how to use MAX3323EEPE+ and how "to plug it"

The UART on the pi has nothing to do with it is the only thing. In order to make any TTL circuit work with rs-232 logic levels you have to have a level converter or you will fry GPIO pins. This is true of a raspberry pi, arduino, your general purpose computer, or a super computer. This is why you needed the usb to serial converter. It was "converting" logic levels for you as your usb input is trying to operate at 5v-0 and the rs-232 output is going to be +-12/10vdc.

So I guess what I'm saying is it is kind of redundant to explain how to connect a logic level converter when that is true of any computer that doesn't operate at rs-232 power levels.  There is also an innumerable amount of rs-232 logic conversion chips so trying to explain how one works is a bit unnecessary when the end user could be using one of thousands of converter chips.


Update examples for phases overhaul

As discussed in #442. Mainly this updates the turn order simulator, but I also fixed the Phases Diagram example.
Some of the turn order implementations are quite different after the overhaul, so you might want to check you’re happy with my interpretation of the originals.
I’ve tweaked the simulator by adding a display for the current stage of each player and made the phase show null when the phase is null (previously it would have been 'default' in those cases).

Looks great!
Couple of minor comments:

Let's not display 'stage undefined' (but just remove the stage message altogether for this case).
Can we get rid of the quotes around the stage and phase names?  It might be helpful when the stage is the empty string, but looks unnecessary otherwise.

@nicolodavis Great!

Fixed this.

Getting rid of the quotes might actually make it less clear (for those who don’t already understand the mechanics). 3 of the 4 examples that use stages only use the empty string stage, so I think it’s helpful to show there is something there somehow. I guess there’d be an empty grey rectangle, but that isn’t obviously an empty string. Happy to change it if you’re sure, but I do think this could be clearer for someone trying to figure out the mechanics.

Oh I meant we can show the quotes if it is any empty string, but remove it when the string is longer.

Ohhhh, right! Sorry for the misunderstanding. Should be fixed now.


Error during setup-wordpress task

I encountered two issues while trying to use the Yootheme Starter Plugin:

The script fails during the setup-wordpress task due to a missing directory.
The expected prompts for plugin metadata (title, description, author name, etc.) were not presented during the create-project step, as stated in the documentation.

Steps to reproduce

Run the following command in the plugins folder of WordPressto to create a new plugin:composer create-project yootheme/starter-plugin myplugin

Issue 1: The creation process completes without asking for any plugin metadata (title, description, author, etc.), even though the documentation states that these fields should be prompted.
Navigate to the plugin directory:cd myplugin

Run the setup task:task setup-wordpress

Issue 2: The following error occurs:Script YOOtheme\Starter\TaskHelper::run handling the run:task event terminated with an exception

In Finder.php line 592:
The "C:/xampp/htdocs/yootheme/wp-content/plugins/myplugin/dist/wordpress/myplugin" directory does not exist.

task: Failed to run task "setup-wordpress": exit status 1

Expected behavior

During the composer create-project command, I expected to be prompted for additional plugin metadata (title, description, author, etc.) as described in the documentation.
The setup-wordpress task should complete successfully and copy the necessary plugin files from the build folder to the plugin root folder.

Screenshots

Environment:

Yootheme Starter Plugin version: 1.0.3
Composer version: 2.7.9
PHP version: 8.2.12
Operating System: Windows 11 (XAMPP)

Hi, thanks for reporting the issue. Unfortunately I don't have access to a Windows system to debug on currently.
We will look into it asap

I can confirm this, Windows 10 is my OS.

We had a look into the issue and unfortunately the way we wanted to run the composer command after creating the project does not work in Windows. There's an open issue in the composer project and from what I read it's not solvable because of a limitation how Symfony/Process executes processes on windows composer/composer#11709
We will remove the create:plugin command from the post-create-project-cmd hook. The step composer create:plugin then needs to be executed manually after the project has been created.

Hey Hendrik, I hope I'm understanding right. Basically the post-create-project-cmd isn't running, so post project creation I run create:plugin and that should do it... It's not working for me.
It asks all the questions, I filled everything in, it still isn't showing up in the plugin section of my admin panel on WP and I'm not seeing the files in the plg root that I would expect for a wordpress plugin. Actually, come to think of it, how does it know it's for Wordpress and not Joomla without me specifying that? With "task" there was differentiation.

@GenoPeppino did you run the setup-wordpress task afterward? The create:plugin command creates the files in the _build folder. To have them in the plugin root folder you need to run task setup-wordpress to make the plugin available in WordPress.
https://github.com/yootheme/starter-plugin?tab=readme-ov-file#set-up-the-plugin

I get the following error when I run task setup-wordpress...
Script YOOtheme\Starter\TaskHelper::run handling the run:task event terminated with an exception
In Finder.php line 592:
The ".../test-three-ytp/wordpress/test_three_ytp" directory does not exist.
task: Failed to run task "setup-wordpress": exit status 1

@GenoPeppino
setup-wordpress has two copy steps.
First it copies all *.php file from build/wordpress to dist/wordpressand replaces placeholders.
Then all files in dist/wordpress are copied to the plugin root folder.
I'm not sure where the path ../test-three-ytp/wordpress/test_three_ytp is could be coming from?
Are you on the latest version?
Please also ensure you are also using the latest starter-utils. The dependencies can be update with the composer update command

@GenoPeppino setup-wordpress has two copy steps. First it copies all *.php file from build/wordpress to dist/wordpressand replaces placeholders. Then all files in dist/wordpress are copied to the plugin root folder.
I'm not sure where the path ../test-three-ytp/wordpress/test_three_ytp is could be coming from? Are you on the latest version? Please also ensure you are also using the latest starter-utils. The dependencies can be update with the composer update command

To the best of my understanding. composer update in the root didn't change anything.
Please note a mistake in the path I shared (accidentally removed dist). Mine is basically the same as the one in the initial question from @Taimurian
".../wp-content/plugins/my-plugin/dist/wordpress/my-plugin" directory does not exist.
Full error message:
task: [utils:copy] composer run:task -q -- copy '*.php' dist/wordpress/test_ninety_ytp
Copied 0 files to 'dist/wordpress/test_ninety_ytp'
task: [utils:placeholder] composer run:task -q -- placeholder dist/wordpress/test_ninety_ytp/test_ninety_ytp.php '{"AUTHOR":"None","AUTHOREMAIL":"none@poorup.com","AUTHORURL":"http://poorup.com","COPYRIGHT":"Copyright (C)","DATE":"2024-11-20","DESCRIPTION":"test ninety plugin description","LICENSE":"GNU General Public License","NAME":"test_ninety_ytp","PHPMINIMUM":"7.4","UPDATEHOST":"","UPDATEURI":"","VERSION":"0.0.1","WORDPRESSMINIMUM":"6.2"}'
task: [utils:copy] composer run:task -q -- copy '**' ./
Script YOOtheme\Starter\TaskHelper::run handling the run:task event terminated with an exception
In Finder.php line 592:
The ".../wp-content/plugins/test-ninety-ytp/dist/wordpress/test_ninety_ytp" directory does not exist.
run:task [--dev] [--no-dev] [--] [...]
task: Failed to run task "setup-wordpress": exit status 1

task: [utils:copy] composer run:task -q -- copy '*.php' dist/wordpress/test_ninety_ytp
Copied 0 files to 'dist/wordpress/test_ninety_ytp'

It looks like the plugin files weren't created in the build folder?
After you run composer create:plugin you should see the file structure like here: https://github.com/yootheme/starter-plugin?tab=readme-ov-file#create-a-new-plugin

task: [utils:copy] composer run:task -q -- copy '*.php' dist/wordpress/test_ninety_ytp
Copied 0 files to 'dist/wordpress/test_ninety_ytp'

It looks like the plugin files weren't created in the build folder? After you run composer create:plugin you should see the file structure like here: https://github.com/yootheme/starter-plugin?tab=readme-ov-file#create-a-new-plugin

That's not what I'm seeing. Here are the differences:
├── build
│   ├── joomla
│       ├── plugin.stub
│       ├── plugin.xml
│   ├── wordpress
│       ├── plugin.stub
│   └── Taskfile.yml

In this order I ran the following:
composer create-project yootheme/starter-plugin test-ninety-ytp (no errors)
navigate to new plugin root directory
composer create:plugin (questions asked, questions answered)
task setup-wordpress (error shown above)
While my error matches the first one reported, NOTE the difference:
'{"AUTHOR":"None","AUTHOREMAIL":"none@poorup.com","AUTHORURL":"http://poorup.com","COPYRIGHT":"Copyright (C)","DATE":"2024-11-20","DESCRIPTION":"test ninety plugin description","LICENSE":"GNU General Public License","NAME":"test_ninety_ytp","PHPMINIMUM":"7.4","UPDATEHOST":"","UPDATEURI":"","VERSION":"0.0.1","WORDPRESSMINIMUM":"6.2"}'
This shows that Author, author URL, description, etc were all filled in during the create:plugin process. So it did run, and I didn't see any errors.

Hey there, any progress?


Win32Metadata support

I opened the issue that requests DirectStorage support on win32metadata project. The contributor said it is needed that DirectStorage team produce its metadata. I think it would be helpful to generate language bindings.

Thanks for the feedback, I'm going to try find an answer to the "can a third party redistribute the dlls in a rust crate" (or other packaging system) question.  This is certainly a scenario we want to enable.
The easiest way to use a nuget package, if you don't want to use nuget or vcpkg directly, is to just unzip the nuget file (these are just zip files with a different extension) and use it from there.

@damyanp are there still plans to get these bindings into the Windows SDK - and as such into win32metadata - or is the .winmd in the NuGet package all that's available for now?

There are currently no plans to add the DirectStorage headers to the Windows SDK.  However, the nuget package does include the windmds, and the header files are now under the MIT license.

@damyanp thanks, but the winmd itself is not code, and is also not in distributable_files.txt, meaning that a user-crate is not allowed to carry this file around?

For all intents and purposes the winmd file is equivalent to the header file.  We can see about finding some way to more explicitly explain this in the license.
I'd imagine that any user crate would need to download the dlls anyway to be useful, so I wonder if the winmd needs to be included in the crate anyway?

@damyanp that'd be appreciated, as the winmd files are definitely binary.

Not exactly sure. For a Rust crate we'd need the .lib at build/compile time (Rust crates are distributed as source code), and the .dll at runtime, and there's not always a consensus whether the API wrapper crate or the crate consumer should provide these.
However, we don't need the winmd in the crate; only in the git repo (for convenience) to generate Rust source code that is checked in to the repo and published as a crate (so that the end-user has minimal dependencies, for what ends up being a 2k-2.5k generated bindings file).
Note that it's not my crate, so any of this is better discussed with the actual author.

Besides, I've filed https://github.com/microsoft/DirectStorage/issues/38 and am inclined to generate a .winmd locally to have improved bindings and dodge the licensing complexity.


Assimp linking issue on Mageia 6 (looking for -lassimpd)

Hi there,
I'm trying to build the current master branch on Mageia 6 x64, using system-installed assimp 3.2.
It fails linking with:
[  1%] Linking CXX executable bin/vulkanscene
/usr/bin/cmake -E cmake_link_script CMakeFiles/vulkanscene.dir/link.txt --verbose=1
/bin/c++    -DVK_USE_PLATFORM_XCB_KHR -DNOMINMAX -D_USE_MATH_DEFINES -O3 -DNDEBUG   CMakeFiles/vulkanscene.dir/vulkanscene/vulkanscene.cpp.o CMakeFiles/vulkanscene.dir/base/vulkandebug.cpp.o CMakeFiles/vulkanscene.dir/base/vulkanandroid.cpp.o CMakeFiles/vulkanscene.dir/base/vulkantools.cpp.o CMakeFiles/vulkanscene.dir/base/vulkanexamplebase.cpp.o  -o bin/vulkanscene -rdynamic -lxcb libs/vulkan/libvulkan.so libs/vulkan/libvulkan.so -lassimpd -lpthread -Wl,-rpath,/home/akien/tmp/Vulkan/libs/vulkan
/bin/ld: cannot find -lassimpd
collect2: error: ld returned 1 exit status
CMakeFiles/vulkanscene.dir/build.make:201: recipe for target 'bin/vulkanscene' failed

-lassimpd would be the debug version of libassimp.so, but such debug versions of libraries are rarely packaged by distro maintainers.
I haven't spotted yet in the buildsystem what triggers the linking of -lassimpd instead of -lassimp, but I'll keep my eyes open..

I'll investigate on my end as this looks like it could be a packaging issue in Mageia.

This does seem to be a packaging issue as the d suffix comes from:
$ grep "ASSIMP_LIBRARIES" /usr/lib64/cmake/assimp-3.2/assimp-config.cmake
set( ASSIMP_LIBRARIES assimp${ASSIMP_LIBRARY_SUFFIX})
set( ASSIMP_LIBRARIES ${ASSIMP_LIBRARIES}d)

Thx for hinting me at this. I'm having the exact same problem on my Ubuntu development machine, so I guess it's time to update to the latest ASSIMP version.

A quick fix to get your current version working would be sed -i /usr/lib/cmake/assimp-3.2/assimp-config.cmake -e 's/\${ASSIMP_LIBRARIES}d/\${ASSIMP_LIBRARIES}/' (escape sequence untested, might need two \, and the path to the file on Ubuntu might differ too.


There isn't any space on sides in attachment preview when you send a long name attachment - reported by @Tushu17

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to Chat
Send a Long name Document(Attachment)

Expected Result:
It should have a proper padding on both sides.
Actual Result:
It's overlapping side of screen
Workaround:
unknown
Platform:

Where is this issue occurring?

Mobile Web

Version Number: 1.1.41-0
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: @Tushu17
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1645293374798619
View all open jobs on GitHub

Triggered auto assignment to @jboniface (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Triggered auto assignment to @thienlnam (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Triggered auto assignment to @kadiealexander (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

Sorry, hold off on posting this - I'm not sure I agree this is a necessary update
The output looks fine to me


Update unit tests for remote.py

Description:
test_remote seems to be one of the culprits for tox sometimes failing
Checklist:
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

Awesome 🐬


Create Box components

See individual issue in other repo: https://github.com/AmpersandTarski/prototype-frontend/issues


ERROR: black 19.10b0 has requirement attrs>=18.1.0, but you'll have attrs 17.2.0 which is incompatible.

the requirements.txt state a strict equality on 17.2.0 , is it necessary ? as it seems to make jinjalint incompatible with other popular tool (I can provide a PR if necessary)

This is a big blocker to jinjalint adoption for me as well. @allan-simon have you had the chance to look into what it would take to update attrs? My understanding is "not much".
Black’s dependency is defined as "attrs>=18.1.0", for reference, and attrs has a strict backwards-compat policy, so we could probably use a similarly-open range.

In the end I’ve fixed this in my fork and decided to use the exact same ranges as black.


errors when installing addon in blender

i select install in blender 2.18a and it comes up with the error pinned

Please read the https://github.com/advancedfx/afx-blender-scripts/blob/master/advancedfx/readme.txt
You probably forgot to install Blender Source Tools 3.0.1 first.

Closing due to lacking feedback.


Add ability to cutomize blackfire based images

Hi,
I started using warden on our own project. I need to use custom php-fpm and php-debug images because we're using additional PHP extensions that are not bundled with Warden (in the .warden/warden-env.yml file).
So far, I wasn't able to find a way to override the php-blackfire image.

Can you confirm how you attempted to override them?
I would think they'd be overrideable by specifying a different image for the php-blackfire service, and the blackfire-agent service if necessary

Can you confirm how you attempted to override them?
I would think they'd be overrideable by specifying a different image for the php-blackfire service, and the blackfire-agent service if necessary

Basically, if I'll add them to the .warden/warden-env.yml file - these services will also be created even when I'll have WARDEN_BLACKFIRE=0 in my .env config

Basically, if I'll add them to the .warden/warden-env.yml file - these services will also be created even when I'll have WARDEN_BLACKFIRE=0 in my .env config

I don't have time to attempt this right now, so I'm going to document my notes here:
This might be possible by updating https://github.com/davidalger/warden/blob/495176185a6cfbdff5f775b3f91ecc84d7dd3959/utils/env.sh#L77 such that it also searches the WARDEN_ENV_PATH for matching files.

I don't have time to attempt this right now, so I'm going to document my notes here:
This might be possible by updating https://github.com/davidalger/warden/blob/495176185a6cfbdff5f775b3f91ecc84d7dd3959/utils/env.sh#L77 such that it also searches the WARDEN_ENV_PATH for matching files.

Apart from changing lines like this:
    image: docker.io/wardenenv/php-fpm:${PHP_VERSION:-7.4}${WARDEN_SVC_PHP_VARIANT:-}-blackfire

to something like:
    image: ${WARDEN_SVC_PHP_BLACKFIRE_IMAGE:-docker.io/wardenenv/php-fpm}:${PHP_VERSION:-7.4}${WARDEN_SVC_PHP_VARIANT:-}-blackfire

so the image name could be overridden (would still require the same tagging format), I think the only way to make it happen is what @navarr suggests adding WARDEN_ENV_PATH to the list of search paths appendEnvPartialIfExists is using.
If someone wants to put together a PR for this, I'd accept it. Curious edge case, but common enough I'm wondering if adding variables to allow easily replacing the image might be a better solve.

Apart from changing lines like this:
    image: docker.io/wardenenv/php-fpm:${PHP_VERSION:-7.4}${WARDEN_SVC_PHP_VARIANT:-}-blackfire

to something like:
    image: ${WARDEN_SVC_PHP_BLACKFIRE_IMAGE:-docker.io/wardenenv/php-fpm}:${PHP_VERSION:-7.4}${WARDEN_SVC_PHP_VARIANT:-}-blackfire

so the image name could be overridden (would still require the same tagging format), I think the only way to make it happen is what @navarr suggests adding WARDEN_ENV_PATH to the list of search paths appendEnvPartialIfExists is using.
If someone wants to put together a PR for this, I'd accept it. Curious edge case, but common enough I'm wondering if adding variables to allow easily replacing the image might be a better solve.

I think it would be good to provide the ability to override not only Blackfire images but all images used in projects

Let's keep this issue open

Still actual

Keep it open

Actual

I believe this is resolved by 5a24ca7d50790e35d7856ef5aaa9002040ff5e46.  Please correct me if I am wrong.


Port more templates

How can you deal with this kind of template?
<%= %Q|"#{@base["01"]["rgb"][0]}"|.rjust(5) %>
source
Nunjucks template support function. Can you implement something like this?

I wrote a quick bash script to convert basic query form erb to nunjucks:
#!/bin/bash

readonly regex='(.*)(<%= @base\[\"0)(\w)(\"\]\[\")(\w+)(\"\] %>)(.*)' # ^$ with m flag dont help

infile=$1

while read -r line ; do
  if [[ $line =~ $regex ]] ; then
    echo "${BASH_REMATCH[1]}{{ base[\"0${BASH_REMATCH[3]}\"][\"${BASH_REMATCH[5]}\"] }}${BASH_REMATCH[9]}"
  else
    echo "${line}"
  fi
done < "$infile"

It is a work in progress, but a start. I loose the indentation (missing whitespace at start).
You can see the regex used here:
(.*)(<%= @base\[\"0)(\w)(\"\]\[\")(\w+)(\"\] %>)(.*)

Debuggex Demo

How can you deal with this kind of template?

<%= %Q|"#{@base["01"]["rgb"][0]}"|.rjust(5) %>

Well, Nunjucks does allow you to evaluate an expression within brackets:
{{ base["hex"]["rgb"] + "foo" }}

I'm not sure what the equivalent to rjust in JavaScript is at this time, but you could do something like:
{{ base["hex"]["rgb"].rjust(5) }}

I guess rjust is like PadLeft in C#?

I wrote a quick bash script to convert basic query form erb to nunjucks:

Awesome! Seriously, that is so cool :smile:
I guess there are two options:

Run the above script on every template and submit a massive pull request
Put the script in a Gist and somewhere in readme.md, tell the user that, if they want to port a template, they can do so using your script, refine it by hand if neeeded, and submit a pull request.

I guess rjust is like PadLeft in C#?

Yes (ctrl+f "rjust").

but I wouldn't want to define such a specific function just to be used for one template

normal.

Awesome! Seriously, that is so cool :smile:

:smile:
I think we should convert most of them ourself and build a specific PR for this with a huge check list to show the progression. I will move the script to gist. A good idea would be to create a wiki with a "how to convert an erb to nunjucks". Also you could move the project to an organization. It will be nice to add external projects, tools, libraries, etc.

I think we should convert most of them ourself and build a specific PR for this with a huge check list to show the progression.

I agree but instead of a massive pull request, let's make an issue with a task list. See #19 for details.

A good idea would be to create a wiki with a "how to convert an erb to nunjucks".

:+1:

Also you could move the project to an organization. It will be nice to add external projects, tools, libraries, etc.

 I actually  made an organization the other day, mostly to reserve the name. If and when tools emerge, I will migrate this repository to the organization.

I am looking to convert the content of i3status.
The template is:
# ~/.i3status.conf
# Base16 {{ scheme }}, by {{ author }}
# template by Matt Parnell, @parnmatt

general {
    colors = true
    color_good = "#{{ base["0B"]["hex"] }}"
    color_bad = "{{ base["08"]["hex"] }}"
    ## remember to add your prefered interval
    # interval = 5
}

## remember to add the rest of your configure

For this case, we dont care at all about dark or light. Only colors are here. I will make a copy of the file under the light name.

What should we do with template's content containing erb comment like zathura?

@aloisdg You can either:

Use HTML comments (if Nunjucks support them)
Remove them

I don't mind.

I dont want to loose information. I mind remove them. We should care about contribution like this even if it come from another repo.

@alexbooker TIL nunjucks supports comments !


revert axom submodule

I did notice this is out of sync with the commit in the Spack package. We should prioritize rebuilding the TPLs soon. I gather this was included to fix the Axom bug that was recently introduced and fix that would have caused issues in the codevelop build.


setSubscription throws getNotificationTypes:mSubscriptionStatus: -15 error

Description:
Our app has settings to toggle notifications, (not related to the OS settings). I would like to use the setSubscription() method when they toggle on or off, but it keeps throwing the error: "getNotificationTypes:mSubscriptionStatus: -15" and resets my app to the router screen.
It does indeed update the subscription status, however, it does so while throwing that error.
The notifications work so far, just hitting this bug on this one function.
Environment
iOS, iPhone XS
OneSignal version: ^3.2.12
I installed it using the instructions, including the cocoa pods method.
Steps to Reproduce Issue:
just using this method:
OneSignal.setSubscription(this.state.overallNotifications);
Anything else:
shows this in the console, or in the alert if I turned that on in the debugging:
getNotificationTypes:mSubscriptionStatus: -15

@Werbelow this would be an issue with whatever @types provider you are using. Please post your package.json. It looks like whoever is providing your types for our SDK hasnt updated to include those new functions.

Closing since this is not an issue with our SDK. But I would be happy to chat with the types provider

thanks @Nightsd01
As far as I know I am not using any typing.
But here is my package.json
  "name": "",
  "version": "0.0.1",
  "private": true,
  "scripts": {
    "start": "node node_modules/react-native/local-cli/cli.js start",
    "test": "jest",
    "build-android": "cd android && ./gradlew assembleRelease",
    "bundle": "react-native run-ios --configuration Release",
    "reset-cache": "watchman watch-del-all && npm start -- --reset-cache"
  },
  "dependencies": {
    "@feathersjs/authentication-client": "^1.0.6",
    "@feathersjs/feathers": "^3.2.3",
    "@feathersjs/rest-client": "^1.4.5",
    "axios": "^0.18.0",
    "core-js": "^2.5.7",
    "crypto-js": "^3.1.9-1",
    "date-fns": "^1.29.0",
    "instabug-reactnative": "^8.0.20",
    "lodash": "^4.17.11",
    "prop-types": "^15.6.2",
    "react": "16.5.0",
    "react-native": "0.57.2",
    "react-native-action-button": "^2.8.5",
    "react-native-autogrow-textinput": "^5.2.0",
    "react-native-check-box": "^2.1.6",
    "react-native-ci-tools": "^1.1.0",
    "react-native-dropdownalert": "^3.9.1",
    "react-native-fast-image": "^5.1.1",
    "react-native-firebase": "^5.2.0",
    "react-native-image-picker": "^0.27.1",
    "react-native-image-view": "^2.0.12",
    "react-native-modal": "^7.0.0",
    "react-native-onesignal": "^3.2.12",
    "react-native-picker-select": "^5.1.0",
    "react-native-router-flux": "^4.0.5",
    "react-native-vector-icons": "^6.0.1",
    "react-native-video": "^4.3.1",
    "react-redux": "^5.0.7",
    "react-timeago": "^4.1.9",
    "redux": "^4.0.1",
    "redux-devtools-extension": "^2.13.5",
    "redux-thunk": "^2.3.0"
  },
  "devDependencies": {
    "babel-jest": "23.6.0",
    "eslint": "^5.13.0",
    "eslint-config-standard": "^12.0.0",
    "eslint-plugin-import": "^2.16.0",
    "eslint-plugin-node": "^8.0.1",
    "eslint-plugin-promise": "^4.0.1",
    "eslint-plugin-standard": "^4.0.0",
    "jest": "23.6.0",
    "metro-react-native-babel-preset": "0.48.0",
    "react-test-renderer": "16.5.0"
  },
  "jest": {
    "preset": "react-native"
  }
}


Add health check trait to allow adding checks to run before starting services

This should be separate from the AppService trait because multiple services may depend on the same resources, so putting the checks in a common place would be helpful.

Do we need to add a new App method for this? Or would it make sense to add it to the ServiceRegistry?


Transform.java:34-37: Please remove this class and also...

The puzzle 998-179b06ba from #998 has to be resolved:
https://github.com/yegor256/takes/blob/47805287af4df195dfb51c8633e232e7725cc442/src/main/java/org/takes/misc/Transform.java#L34-L37
The puzzle was created by @fanifieiev on 29-Nov-19.
Estimate: 30 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@paulodamaso/z please, pay attention to this issue

Job #1014 is now in scope, role is DEV

The job #1014 assigned to @fanifieiev/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

The puzzle 998-179b06ba has disappeared from the source code, that's why I closed this issue.

@sereshqua/z please review this job completed by @fanifieiev/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

@0crat quality good


How to navigate to a normal ContentPage in fullscreen from NavigationPage with TabbedPage?

This Xamarin official sample demonstrates how to use TabbedPage with NavigationPage together. The result looks like this:

As you can see, when it navigates to a sub-page, the sub-page is "nested" in the TabbedPage. I want to know how can I make the sub-page in fullscreen, which means when it's on the sub-page, it takes up the whole screen and the TabbedPage doesn't show. Below is an example:

As Microsoft suggestes

While a NavigationPage can be placed in a TabbedPage, it's not recommended to place a TabbedPage into a NavigationPage. This is because, on iOS, a UITabBarController always acts as a wrapper for the UINavigationController.

I don't know how to implement this.

Hi @yangshuairocks
The easiest way is to push your ContentPage as a modal page using Navigation.PushModalAsync rather than Navigation.PushAsync.


fix: add Kiran1689

closes #1951
Checklist for profile addition

[x] Name the PR fix: add <your-username>.
[x] You've added file named your-username.json in the profiles directory.
[x] You've linked the PR to the correct issue.

Other additions/changes

[x] Changes made in this PR are mentioned below
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] I acknowledge that all my contributions will be made under the project's license.
[x] I've read the contributing guidelines

This PR adds/fixes

fixes #1951
Notes

@schmelto


EXO module "Version 2.0.6 Preview5 or later" not available to install/download

Just over a week ago this page was updated to reference version 2.0.6 preview 5 to support connecting to the Security and Compliance Center using an application/certificate.
However this version doesn't appear to be available; the latest version in the PowerShell gallery is still 2.0.6 preview 4.
We are desperately awaiting this change due to internal policies around using service accounts, so would be really grateful if there was an update on availability. As noted in opened and closed issue #9066 this was mentioned in Message Center post MC316448, and app/cert auth was planned by late Feb/early March.
There was a lot of hope/excitement to see the docs updated but unfortunately they are currently incorrect as 2.0.6 preview 5 isn't available still.
Thanks!
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d2cfafdf-cbe9-2ffe-eabe-becc66d51728
Version Independent ID: 4a46c8a8-dc70-d877-271e-6679c465a6d5
Content: App-only authentication
Content Source: exchange/docs-conceptual/app-only-auth-powershell-v2.md
Service: exchange-online
GitHub Login: @chrisda
Microsoft Alias: chrisda

@stefs987 thanks for carrying the torch 🔥.

It's out now

Just spotted it! Great :)


Generalize the state_data getters

Good job with this, it's a really nice start!
In the end I believe that we could use this library to manage all types of the devices that Electrolux returns with the API.
In my case I own an air-purifier and a portable air conditioner and while it plays nicely with the appliances/{self.id}/info and initial appliances/ calls, the response format seems to differ for the appliances/{self.id}/state calls.
For example, the /appliances responses:
{'applianceId': 'RED', 'applianceName': 'Air purifier', 'applianceType': 'PUREA9', 'created': '2024-01-14T20:21:55.716+00:00'}
{'applianceId': 'RED', 'applianceName': 'Air conditioner', 'applianceType': 'Azul', 'created': '2024-08-02T13:38:28.197+00:00'}

For /info:
{'serialNumber': 'RED', 'pnc': 'RED', 'brand': 'ELECTROLUX', 'deviceType': 'AIR_PURIFIER', 'model': 'A9', 'variant': 'CADR600', 'colour': 'GREY'}
{'serialNumber': 'RED', 'pnc': 'RED', 'brand': 'ELECTROLUX', 'deviceType': 'PORTABLE_AIR_CONDITIONER', 'model': 'COMFORT600', 'variant': 'AZULTM01', 'colour': 'MATTSILVER'}

and finally for the /state (pretty-printed):
{
  "applianceId": "RED",
  "connectionState": "Connected",
  "status": "enabled",
  "properties": {
    "reported": {
      "FilterRFID": "RED",
      "FilterLife": 100,
      "Fanspeed": 2,
      "UILight": true,
      "FilterType": 96,
      "DspIcoPM2_5": true,
      "DspIcoPM1": true,
      "DspIcoPM10": true,
      "DspIcoTVOC": true,
      "logE": 3103,
      "logW": 454,
      "InterfaceVer": 20230110,
      "VmNo_NIU": "RED",
      "TVOCBrand": "AMS",
      "FrmVer_NIU": "3.0.1",
      "capabilities": {
        "tasks": {}
      },
      "SignalStrength": "FAIR",
      "tasks": {},
      "ErrPM2_5": false,
      "ErrTVOC": false,
      "ErrTempHumidity": false,
      "ErrFanMtr": false,
      "ErrCommSensorDisplayBrd": false,
      "DoorOpen": false,
      "Workmode": "Auto",
      "SafetyLock": false,
      "Ionizer": false,
      "$version": 1072,
      "deviceId": "RED",
      "PM1": 6,
      "PM2_5": 7,
      "PM10": 8,
      "CO2": 787,
      "Temp": 24,
      "Humidity": 61,
      "RSSI": -62,
      "TVOC": 404,
      "ECO2": 1145
    }
  }
}
{
  "applianceId": "RED",
  "connectionState": "Disconnected",
  "status": "enabled",
  "properties": {
    "reported": {
      "applianceState": "off",
      "startTime": 0,
      "stopTime": 0,
      "temperatureRepresentation": "celsius",
      "sleepMode": "off",
      "targetTemperatureC": 23,
      "uiLockMode": false,
      "mode": "cool",
      "fanSpeedSetting": "auto",
      "verticalSwing": "off",
      "filterState": "good",
      "VmNo_MCU": "V1.0.2",
      "dataModelVersion": "1.0.1",
      "VmNo_NIU": "RED",
      "capabilities": {},
      "tasks": {},
      "schedulerSession": "off",
      "upgradeState": "idle",
      "logE": 0,
      "logW": 343,
      "schedulerMode": "off",
      "$version": 58,
      "deviceId": "RED",
      "ambientTemperatureC": 22,
      "compressorState": "off",
      "totalRuntime": 3177,
      "compressorCoolingRuntime": 2511,
      "compressorHeatingRuntime": 0,
      "filterRuntime": 3240,
      "networkInterface": {
        "linkQualityIndicator": "GOOD",
        "rssi": -61
      },
      "ambientTemperatureF": None,
      "hepaFilterLifeTime": 3240,
      "fourWayValveState": "off",
      "evapDefrostState": "off"
    }
  }
}

This actually makes sense, because different device types will have different properties to show.
This makes me wonder if we should just set the properties.reported as state_data and return the whole dict for user to manipulate or maybe parse the possible configs with pydantic or something similar. The latter provides some validation on the lib level, but I'm not sure if it's desired
TBH returning the whole dict seems better as it seems that the possible commands that can be sent to the device use the same keys, so an implementing application could simply check if e.g. targetTemperatureC exists in the return dict and assume that it can be sent out as a command if it does.

Thank you @dezeroku for your input. I totally agree with you. It can be up to the user to parse the device specific properties.


Field to get the collection title in templates + option to disable collections folders

{{collectionTitle}} can be used in templates to get the collection title.
I've also added an option to enable/disable the storage of articles in collections folders.
I needed these two features, so maybe others will need them too?
It's my first pull request, I'm not sure how it works, maybe I've not done the things the right way.

Thanks for the contribution.
You can use {{collection.title}} to get the collection title. This is more extensible than the {{collectionTitle}} if there are new fields about the collection that need to include in the template. The document should be enhanced for more details about this, as mentioned in #27.
As for the option of disabling the collections folders, can you open a separate PR for this? Thanks.


Wrote a Vignette explaining the math and a use case

I wrote a vignette which I hope is useful in explaining the math behind each of the metrics as well as a use case example of using IR metrics with a recommendation system (my current use case). Only changes to the general package structure is in the description to support building the vignette.

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

Thanks for this, much appreciated! I think you should get credit, are you happy if we add your name at the top of this?

That's totally fine. I found the package useful and thought this would be a beneficial contribution.


Make kubectl related tests more reliable

These tests did not constrain their queries to a specific resource,
nor did they perform proper cleanup.
Release note:
None

/retest

/cherry-pick release-0.26


Pull image before executing S2I script [1.4]

Backport of openshift/source-to-image#743 for 1.4

@bparees ptal


Fine-tuned BERT SavedModel hard-referencing vocab.txt from TF Hub temporary files.

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 20.04
TensorFlow installed from (source or binary): pip
TensorFlow version (use command below): v2.4.0-rc4-71-g582c8d236cb 2.4.0
Python version: 3.8.5 64-bit
CUDA/cuDNN version: 11.0
GPU model and memory: GeForce GTX 1080Ti, 11175MiB Memory

Describe the current behavior
I was using the Classify text with BERT notebook locally on my machine with electra_small model as a base model for fine-tuning. I've saved fine-tuned model locally, loaded it back and made sure it works.  Turned out it was looking for a temporary tfhub files which were no longer there causing /tmp/tfhub_modules/09bd4e665682e6f03bc72fbcff7a68bf6879910e/assets/vocab.txt; No such file or directory error. I've reloaded these models from TF hub and model started loading and working just fine. I renamed /tmp/tfhub_modules/ directory and model stopped loading again. I've checked the directory I've saved my fine-tuned model into and it had the very same vocab.txt file with the same content as in TF Hub temp file, but refused to use it for some reason.
Describe the expected behavior
I've expected SavedModel to be self-contained and use its own local vocbulary file rather than hard-referencing a vocab file from external temporary dir of TF Hub.
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.
---------------------------------------------------------------------------
NotFoundError                             Traceback (most recent call last)
~/PycharmProjects/DismissabilityTF/venv/lib/python3.8/site-packages/tensorflow/python/saved_model/load.py in load_internal(export_dir, tags, options, loader_cls, filters)
    888       try:
--> 889         loader = loader_cls(object_graph_proto, saved_model_proto, export_dir,
    890                             ckpt_options, filters)

~/PycharmProjects/DismissabilityTF/venv/lib/python3.8/site-packages/tensorflow/python/saved_model/load.py in __init__(self, object_graph_proto, saved_model_proto, export_dir, ckpt_options, filters)
    164       if isinstance(node, tracking.CapturableResource):
--> 165         init_op = node._initialize()  # pylint: disable=protected-access
    166         if not context.executing_eagerly():

~/PycharmProjects/DismissabilityTF/venv/lib/python3.8/site-packages/tensorflow/python/eager/def_function.py in __call__(self, *args, **kwds)
    827     with trace.Trace(self._name) as tm:
--> 828       result = self._call(*args, **kwds)
    829       compiler = "xla" if self._experimental_compile else "nonXla"

~/PycharmProjects/DismissabilityTF/venv/lib/python3.8/site-packages/tensorflow/python/eager/def_function.py in _call(self, *args, **kwds)
    893       # If we did not create any variables the trace we have is good enough.
--> 894       return self._concrete_stateful_fn._call_flat(
    895           filtered_flat_args, self._concrete_stateful_fn.captured_inputs)  # pylint: disable=protected-access

~/PycharmProjects/DismissabilityTF/venv/lib/python3.8/site-packages/tensorflow/python/eager/function.py in _call_flat(self, args, captured_inputs, cancellation_manager)
   1917       # No tape is watching; skip to running the function.
-> 1918       return self._build_call_outputs(self._inference_function.call(
   1919           ctx, args, cancellation_manager=cancellation_manager))

~/PycharmProjects/DismissabilityTF/venv/lib/python3.8/site-packages/tensorflow/python/eager/function.py in call(self, ctx, args, cancellation_manager)
    554         if cancellation_manager is None:
--> 555           outputs = execute.execute(
    556               str(self.signature.name),

~/PycharmProjects/DismissabilityTF/venv/lib/python3.8/site-packages/tensorflow/python/eager/execute.py in quick_execute(op_name, num_outputs, inputs, attrs, ctx, name)
     58     ctx.ensure_initialized()
---> 59     tensors = pywrap_tfe.TFE_Py_Execute(ctx._handle, device_name, op_name,
     60                                         inputs, attrs, num_outputs)

NotFoundError: 2 root error(s) found.
  (0) Not found:  /tmp/tfhub_modules/09bd4e665682e6f03bc72fbcff7a68bf6879910e/assets/vocab.txt; No such file or directory
	 [[{{node StatefulPartitionedCall/text_file_init/InitializeTableFromTextFileV2}}]]
  (1) Not found:  /tmp/tfhub_modules/09bd4e665682e6f03bc72fbcff7a68bf6879910e/assets/vocab.txt; No such file or directory
	 [[{{node StatefulPartitionedCall/text_file_init/InitializeTableFromTextFileV2}}]]
	 [[StatefulPartitionedCall/text_file_init/InitializeTableFromTextFileV2/_3]]
0 successful operations.
0 derived errors ignored. [Op:__inference_restored_function_body_57226]

Function call stack:
restored_function_body -> restored_function_body

During handling of the above exception, another exception occurred:

FileNotFoundError                         Traceback (most recent call last)
<ipython-input-1-910e5da7da87> in <module>
      5 saved_model_path = './imdb_bert'
      6 tf.get_logger().setLevel('ERROR')
----> 7 reloaded_model = tf.saved_model.load(saved_model_path)

~/PycharmProjects/DismissabilityTF/venv/lib/python3.8/site-packages/tensorflow/python/saved_model/load.py in load(export_dir, tags, options)
    857     ValueError: If `tags` don't match a MetaGraph in the SavedModel.
    858   """
--> 859   return load_internal(export_dir, tags, options)["root"]
    860
    861

~/PycharmProjects/DismissabilityTF/venv/lib/python3.8/site-packages/tensorflow/python/saved_model/load.py in load_internal(export_dir, tags, options, loader_cls, filters)
    890                             ckpt_options, filters)
    891       except errors.NotFoundError as err:
--> 892         raise FileNotFoundError(
    893             str(err) + "\n If trying to load on a different device from the "
    894             "computational device, consider using setting the "

FileNotFoundError: 2 root error(s) found.
  (0) Not found:  /tmp/tfhub_modules/09bd4e665682e6f03bc72fbcff7a68bf6879910e/assets/vocab.txt; No such file or directory
	 [[{{node StatefulPartitionedCall/text_file_init/InitializeTableFromTextFileV2}}]]
  (1) Not found:  /tmp/tfhub_modules/09bd4e665682e6f03bc72fbcff7a68bf6879910e/assets/vocab.txt; No such file or directory
	 [[{{node StatefulPartitionedCall/text_file_init/InitializeTableFromTextFileV2}}]]
	 [[StatefulPartitionedCall/text_file_init/InitializeTableFromTextFileV2/_3]]
0 successful operations.
0 derived errors ignored. [Op:__inference_restored_function_body_57226]

Function call stack:
restored_function_body -> restored_function_body

 If trying to load on a different device from the computational device, consider using setting the `experimental_io_device` option on tf.saved_model.LoadOptions to the io_device such as '/job:localhost'.

@vforvoid
Please share the code you have executed on your local,such that we can replicate the issue faced.

@vforvoid
Please share the code you have executed on your local,such that we can replicate the issue faced.

Basically it's a fragment of a Classify text with BERT notebook
import tensorflow_hub as hub
import tensorflow_text
import tensorflow as tf

def build_classifier_model():
    tfhub_handle_encoder = 'https://tfhub.dev/google/electra_small/2'
    tfhub_handle_preprocess = 'https://tfhub.dev/tensorflow/bert_en_uncased_preprocess/2'

    text_input = tf.keras.layers.Input(shape=(), dtype=tf.string, name='text')
    preprocessing_layer = hub.KerasLayer(tfhub_handle_preprocess, name='preprocessing')
    encoder_inputs = preprocessing_layer(text_input)
    encoder = hub.KerasLayer(tfhub_handle_encoder, trainable=True, name='BERT_encoder')
    outputs = encoder(encoder_inputs)
    net = outputs['pooled_output']
    net = tf.keras.layers.Dropout(0.1)(net)
    net = tf.keras.layers.Dense(1, activation=None, name='classifier')(net)
    return tf.keras.Model(text_input, net)

model_save_path = './modelname'

# Run this one first, it will create a saved model in your working dir.

# if __name__ == '__main__':
#     classifier_model = build_classifier_model()
#     bert_raw_result = classifier_model(tf.constant(['some dummy text']))
#     print(bert_raw_result)
#     classifier_model.save(model_save_path, include_optimizer=False)

# Run this one next, this should work.

if __name__ == '__main__':
    classifier_model = tf.saved_model.load(model_save_path)
    bert_raw_result = classifier_model(tf.constant(['some dummy text']))
    print(bert_raw_result)

# Now, try to remove / rename `/tmp/tfhub_modules/` directory and run the code above again. It fails for me with exception I provided in OP.
# So there's a hard reference to `/tmp/tfhub_modules/09bd4e665682e6f03bc72fbcff7a68bf6879910e/assets/vocab.txt` file instead of the one in "modelname" saved model.

Basically it's a fragment of a Classify text with BERT notebook
import tensorflow_hub as hub
import tensorflow_text
import tensorflow as tf

def build_classifier_model():
    tfhub_handle_encoder = 'https://tfhub.dev/google/electra_small/2'
    tfhub_handle_preprocess = 'https://tfhub.dev/tensorflow/bert_en_uncased_preprocess/2'

    text_input = tf.keras.layers.Input(shape=(), dtype=tf.string, name='text')
    preprocessing_layer = hub.KerasLayer(tfhub_handle_preprocess, name='preprocessing')
    encoder_inputs = preprocessing_layer(text_input)
    encoder = hub.KerasLayer(tfhub_handle_encoder, trainable=True, name='BERT_encoder')
    outputs = encoder(encoder_inputs)
    net = outputs['pooled_output']
    net = tf.keras.layers.Dropout(0.1)(net)
    net = tf.keras.layers.Dense(1, activation=None, name='classifier')(net)
    return tf.keras.Model(text_input, net)

model_save_path = './modelname'

# Run this one first, it will create a saved model in your working dir.

# if __name__ == '__main__':
#     classifier_model = build_classifier_model()
#     bert_raw_result = classifier_model(tf.constant(['some dummy text']))
#     print(bert_raw_result)
#     classifier_model.save(model_save_path, include_optimizer=False)

# Run this one next, this should work.

if __name__ == '__main__':
    classifier_model = tf.saved_model.load(model_save_path)
    bert_raw_result = classifier_model(tf.constant(['some dummy text']))
    print(bert_raw_result)

# Now, try to remove / rename `/tmp/tfhub_modules/` directory and run the code above again. It fails for me with exception I provided in OP.
# So there's a hard reference to `/tmp/tfhub_modules/09bd4e665682e6f03bc72fbcff7a68bf6879910e/assets/vocab.txt` file instead of the one in "modelname" saved model.

Side question: is there any way to read BERT EN Uncased Preprocess model's code? I tried to search for its bert_pack_inputs function code, no luck.

Side question: is there any way to read BERT EN Uncased Preprocess model's code? I tried to search for its bert_pack_inputs function code, no luck.

Same as https://github.com/tensorflow/hub/issues/719

Same as https://github.com/tensorflow/hub/issues/719


receive events about printer status

Hey,
Is it possible to get printer status from connected devices?
For example if paper is empty or the connection is lost
Keep up the great work!

@NqCompiler hi! Sure. Please take a look at ‘startMonitorPrinter’. I suppose it returns all data you’ve mentioned.
moreover after every print - promise with corresponding data will be returned as well

I had so many tabs open and was looking at the examples from your fork from react-native-thermal-receipt-printer and somehow ended up creating the issue at the right place.
Thanks for the fast response and help :)

@tr3v3r  @NqCompiler  when device is connected using instantiate method, printing is working fine but this startsMonitorPrinter() is showing
DISCONNECT OFFLINE UNKNOWN for
 EscPosPrinter.addPrinterStatusListener((status) => {

      console.log(status.connection, status.online, status.paper); // will be executed every 5 sec
    });

What is the issue?


The limit of Selection

VSCode Version: Code 1.15.1 (41abd21afdf7424c89319ee7cb0445cc6f376959, 2017-08-16T18:07:25.676Z)
OS Version: Windows_NT x64 10.0.15063
Extensions:

Extension
Author (truncated)
Version

python
don
0.7.0

vsc-material-theme
Equ
1.0.2

Go
luk
0.6.63

vscode-icons
rob
7.12.0

Steps to Reproduce:

a big file (>2000 lines)
select some words
when the selections are larger than 1000, only the first 1000 can be highlight while others stay unchanged.
the scroll bar are also consistent with the selection, cannot highlight more selections.

Reproduces without extensions: Yes/No

Duplicate of #1110


Tooltip is not working in materialize

                  <input name="CC1" type="radio" id="CC1"  value="CS"/>
                  CS

CSS
.material-tooltip {
padding: 5px 20px 20px;
border-radius: 10px;
background: rgba(37, 36, 36, 0.9);
overflow: visible;

}
.material-tooltip .backdrop {
display: none !important;
}
.material-tooltip:after {
position: absolute;
content: " ";
top: 47.5%;
left: 0%;
margin-left: -12%;
width: 0;
height: 0;
border-style: solid;
border-width: 13px 0px 20px 13px;
border-color: rgba(37, 36, 36, 0.9) transparent transparent transparent;
border-top: 12px solid transparent;
border-bottom: 12px solid transparent;
border-right: 12px solid rgba(37, 36, 36, 0.9);

}
SCRIPT

It is working fine in JSFiddle but not in my main code

If it is working in a jsfiddle and not your main code you should look into how you are calling your dependencies. Are you sure you don't have any errors?

In addition to what @acburst has said. Are you using any other UI framework that might be conflicting the tooltip function? like jQueryUI, ...

I can't get tooltips to work neither. It was working for me in version 0.97.7 but when I change the materialize.js to version 0.98.1 tooltips are not working (no error).

@prachi708 and @CorentinMic Please provide a codepen, is the best way to see what is exactly happening and how could be resolved

Here is the codepen with the 0.98.1 version, it's not working: https://codepen.io/anon/pen/xqvJPO
If you change the js source with 0.97.8 version, it's working : https://codepen.io/anon/pen/QpeBmo

After multiples attempts, I figured out that it was my sass command to update the css file in the new version that hadn't worked. So I was using the old version css file. Tooltips are working in 0.98.1. Sorry for the inconvenience.


PERF: Reduce verbosity of itk::GaussianOperator warning

itk::GaussianOperator can be set with a MaximumKernelWidth parameter
such that a kernel that would otherwise overflow this size is cropped to
the maximum width. Cropping to the set maximum width happens frequently
in cases such as Gaussian blurring for multiscale pyramid generation and
is expected behavior, but the previous warning message makes it seem
like a failure case. This commit reduces the verbosity of the message
from a warning to a debug printout so that developers can get insights
into kernel cropping when needed, but are not alarmed by default warning
printouts.
Previous behavior: Warning whenever Gaussian kernel is correctly truncated to user specifications.

WARNING: In C:\P\IPP\ITK-source\ITK\Modules\Core\Common\include\itkGaussianOperator.hxx, line 57
GaussianOperator (000000C7587FC630): Kernel size has exceeded the specified maximum width of 32 and has been truncated to 33 elements.  You can raise the maximum width using the SetMaximumKernelWidth method.

New behavior: No printout unless explicitly enabled.

PR Checklist

[x] No major design changes were made (or the changes have been approved)

Refer to the ITK Software Guide for
further development details if necessary.

In case it is related, ... I am now getting an error message in the master branch:
Building CXX object Modules/Registration/Common/test/CMakeFiles/ITKRegistrationCommonTestDriver.dir/itkMultiResolu
tionImageRegistrationMethodTest_1.cxx.o
FAILED: Modules/Registration/Common/test/CMakeFiles/ITKRegistrationCommonTestDriver.dir/itkMultiResolutionImageRegistration
MethodTest_1.cxx.o
[munch]
In file included from /home/local/KHQ/lee.newberg/git/InsightSoftwareConsortium/ITK-master/Modules/Core/Common/include/itkLightObject.h:21,
                 from /home/local/KHQ/lee.newberg/git/InsightSoftwareConsortium/ITK-master/Modules/Core/Common/include/itkObject.h:31,
                 from /home/local/KHQ/lee.newberg/git/InsightSoftwareConsortium/ITK-master/Modules/Core/Transform/include/itkTransformBase.h:23,
                 from /home/local/KHQ/lee.newberg/git/InsightSoftwareConsortium/ITK-master/Modules/Core/Transform/include/itkTransform.h:22,
                 from /home/local/KHQ/lee.newberg/git/InsightSoftwareConsortium/ITK-master/Modules/Core/Transform/include/itkBSplineBaseTransform.h:22,
                 from /home/local/KHQ/lee.newberg/git/InsightSoftwareConsortium/ITK-master/Modules/Registration/Common/include/itkImageToImageMetric.h:21,
                 from /home/local/KHQ/lee.newberg/git/InsightSoftwareConsortium/ITK-master/Modules/Registration/Common/include/itkMutualInformationImageToImageMetric.h:21,
                 from /home/local/KHQ/lee.newberg/git/InsightSoftwareConsortium/ITK-master/Modules/Registration/Common/test/itkMultiResolutionImageRegistrationMethodTest_1.cxx:19:
/home/local/KHQ/lee.newberg/git/InsightSoftwareConsortium/ITK-master/Modules/Core/Common/include/itkGaussianOperator.hxx: In instantiation of ‘typename itk::GaussianOperator<TPixel, VDimension, TAllocator>::Superclass::CoefficientVector itk::GaussianOperator<TPixel, VDimension, TAllocator>::GenerateCoefficients() [with TPixel = float; unsigned int VDimension = 3; TAllocator = itk::NeighborhoodAllocator<float>; typename itk::GaussianOperator<TPixel, VDimension, TAllocator>::Superclass::CoefficientVector = std::vector<double>]’:
/home/local/KHQ/lee.newberg/git/InsightSoftwareConsortium/ITK-master/Modules/Core/Common/include/itkGaussianOperator.hxx:26:1:   required from here
/home/local/KHQ/lee.newberg/git/InsightSoftwareConsortium/ITK-master/Modules/Core/Common/include/itkMacro.h:489:17: error: ‘class itk::GaussianOperator<float, 3, itk::NeighborhoodAllocator<float> >’ has no member named ‘GetDebug’
  489 |       if (this->GetDebug() && itk::Object::GetGlobalWarningDisplay())          \
      |           ~~~~~~^~~~~~~~
/home/local/KHQ/lee.newberg/git/InsightSoftwareConsortium/ITK-master/Modules/Core/Common/include/itkGaussianOperator.hxx:55:7: note: in expansion of macro ‘itkDebugMacro’
   55 |       itkDebugMacro(<< "Kernel size has exceeded the specified maximum width of " << m_MaximumKernelWidth
      |       ^~~~~~~~~~~~~
/home/local/KHQ/lee.newberg/git/InsightSoftwareConsortium/ITK-master/Modules/Core/Common/include/itkGaussianOperator.hxx: In instantiation of ‘typename itk::GaussianOperator<TPixel, VDimension, TAllocator>::Superclass::CoefficientVector itk::GaussianOperator<TPixel, VDimension, TAllocator>::GenerateCoefficients() [with TPixel = double; unsigned int VDimension = 3; TAllocator = itk::NeighborhoodAllocator<double>; typename itk::GaussianOperator<TPixel, VDimension, TAllocator>::Superclass::CoefficientVector = std::vector<double>]’:
/home/local/KHQ/lee.newberg/git/InsightSoftwareConsortium/ITK-master/Modules/Core/Common/include/itkGaussianOperator.hxx:26:1:   required from here
/home/local/KHQ/lee.newberg/git/InsightSoftwareConsortium/ITK-master/Modules/Core/Common/include/itkMacro.h:489:17: error: ‘class itk::GaussianOperator<double, 3, itk::NeighborhoodAllocator<double> >’ has no member named ‘GetDebug’
  489 |       if (this->GetDebug() && itk::Object::GetGlobalWarningDisplay())          \
      |           ~~~~~~^~~~~~~~
/home/local/KHQ/lee.newberg/git/InsightSoftwareConsortium/ITK-master/Modules/Core/Common/include/itkGaussianOperator.hxx:55:7: note: in expansion of macro ‘itkDebugMacro’

It certainly appears related. Interesting that this did not show up in CI. @Leengit could you please share details on your build platform?

Linux 5.13.0-41-generic #46~20.04.1-Ubuntu SMP Wed Apr 20 13:16:21 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
gcc (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0
ninja 1.10.0

Thank you, on review I am also able to see failures in CDash Expected Nightly Builds. My guess is that this means itkDebugMacro is only valid in objects inheriting from a certain base, perhaps ProcessObject? Maybe I missed something in documentation?
I will open an issue to track. If we can't use debug printouts here I would be in favor of removing the warning message entirely. It could be useful in certain scenarios where the user is unaware of a default truncation parameter, but in general it is simply informing the user that execution is proceeding according to design.

Perhaps a quick, sloppy fix to get the master branch working again.  And an issue that asks that it be fixed more properly.


we note that new queries only show message totals for those who were active in channel within query limit

we are not now showing a square for every member of the channel - maybe we should be ...
note also this does affect the score

we should be, but note that our use case for slack is small teams, so there will not be >8 or so squares per channel.  i don't think slack really works at a larger scale than that.
Armando Fox
Professor, Computer Science Division
Faculty Advisor, UC Berkeley MOOCLab
fox .at. cs .dot. berkeley .dot. edu
581 Soda Hall MC#1776, Berkeley, CA 94720-1776
+1.510.642.6820 / http://www.cs.berkeley.edu/~fox http://www.cs.berkeley.edu/~fox
TAKE MY CLASS FOR FREE VIA edX:  http://saas-class.org
LIKE IT ON FACEBOOK:  http://facebook.com/saas-class http://facebook.com/saas-class
TRY MY BOOK (NOT FREE): http://www.saasbook.info http://beta.saasbook.info/ (goes with the course)

On Sep 7, 2016, at 7:01 AM, Sam Joseph<EMAIL_ADDRESS>wrote:
we are not now showing a square for every member of the channel - maybe we should be ...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/AgileVentures/project_metric_slack/issues/14, or mute the thread https://github.com/notifications/unsubscribe-auth/AAQmTDgAbnGtvdEHwx1BzqF1ax1FNCyUks5qnsO9gaJpZM4J28_e.


Cards to be separated by lines

I saw a nice card-like slide where, instead of playing cards each Heading Level 4 was separated by a vertical line. Could be horizontal line instead - if that is the card orientation.

Coded.

Shipped in v3.6.


Red5 Pro Coturn configuration + ubuntu 22.04 + minor changes

Add Red5 Pro Coturn configuration on the single Red5 Pro server and Stream Manager server
Ubuntu 22.04
Minor changes

Hi @iolesyk, Everything looks good to me. I think we can merge this branch


PipelineEditor should notify user if forked

If an operation is created in the PipelineEditor, it is pretty unlikely that it will fork. However, if it does, the user should be notified

duplicate of #82