Variables created using var are not completable.

Actual behavior (Required!)
As shown in THIS EXAMPLE
The variable router does not auto-complete correctly when initiating the omni-complete
Expected behavior (Required!)
That I would be presented the correct options for the Router class type
The steps to reproduce actual behavior (Required!)

Clone THIS REPO
Open the project using vim with this plugin enabled
Go to line 22, type router. and initiate autocompletion

Environment (Required!)

OS: Linux (Xubuntu 21.10)
Vim version:

VIM - Vi IMproved 8.2 (2019 Dec 12, compiled Sep 21 2021 08:39:53)
Included patches: 1-2434, 3402-3403, 3409, 3428

Neovim version:

Q&A

Yes, I tried minimal .vimrc configuraion.
Yes, I have enabled logs with JCdebugEnableLogs and can put here content of JCdebugGetLogContent command, if you need.

271.740240 [context] context: CONTEXT_AFTER_DOT
271.740341 [context] dot expression: router.
271.740430 [complete] complete after dot. expr: router.
271.740618 [collector] get declared class name for: "router"
271.775161 [complete] F3. "router.|"  typename: "var"
271.775191 [collector] get class info. class: var
271.792788 [server] communicate: -E "com.example.matrix_testing.var" [collector#FetchInfoFromServer]
271.900418 [server]
271.900668 [server] communicate: -E "java.lang.var" [collector#FetchInfoFromServer]
272.009019 [server]
272.010143 [server] communicate: -E "io.vertx.core.AbstractVerticle$var" [collector#FetchInfoFromServer]
272.120748 [server]
272.122307 [complete] get member list. outputkind: 0
272.122496 [complete] search member. name: , kind: 0, outputkind: 0
272.140664 [complete] finish completion  0.404853s
290.126112 [server] communicate: -clear-from-cache "com.example.matrix_testing.MainVerticle" [s:RemoveCurrentFromCache]
290.227628 [server]
290.228478 [server] communicate: -async -recompile-class "com.example.matrix_testing.MainVerticle" [s:RemoveCurrentFromCache]
290.330706 [server]
290.331762 [server] communicate: -async -add-source-to-cache "-source /home/dphillips/Documents/Zanclus/Workspace/vertx-matrix-params-example/src/main/java/com/example/matrix_testing/MainVerticle.java -class MainVerticle -package com.example.matrix_testing" [s:RemoveCurrentFromCache]

Even, if you wish, I can set g:JavaComplete_JavaviLogLevel to 'debug', then set g:JavaComplete_JavaviLogDirectory, and put here server logs, too.

Screenshot (Optional)

The output of :redir and :message (Optional)

Actually yes, not yet implemented.

If you give me some hints, I might be able to submit a PR. I'll try to take a look later today.

I guess it requires some hacks in vim java parser class, that I still wish to replace with javaparser library.
But may be there is enough information returned by parser.

Thanks for the hint! I'll try to look into it.
DEVEN PHILLIPS
ARCHITECT, RED HAT CLOUD NATIVE RUNTIMES PRACTICE
[image: https://www.redhat.com/] https://www.redhat.com/
On Mon, Nov 8, 2021 at 8:41 AM Artur Shaik @.***> wrote:

I guess it requires some hacks in vim java parser class, that I still
wish to replace with javaparser library.
But may be there is enough information returned by parser.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/artur-shaik/vim-javacomplete2/issues/463#issuecomment-963161052,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAYK24A2ZWPQZEIAYW5GSTUK7HQZANCNFSM5HQX5MPA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Consider switch to jc.nvim


More helpful error when L2=>L1 message not posted

'"could not find content with matching hash" could include something about 'are you sure the message is posted on L2=>L1'' - mach34

just to be clear, you mean here right - https://github.com/AztecProtocol/aztec-packages/blob/26ef8ff78a6ae5a8ebb1f1a35c87f37c33cf1e44/yarn-project/aztec-nr/aztec/src/messaging.nr#L11


alter the builder script to pull data directly from a database instead of building from JSON

Currently: we use a file in the repository to do this.
Interim: we could reference the regularly updated file in this mirror repository if we for some reason don't want to use our API.
Ideally: we should reference the Data Sources API endpoint from this issue: https://github.com/Police-Data-Accessibility-Project/data-sources-app/issues/1

See issue #5


Replace deprecated version_compare with version

See https://github.com/ome/ansible-role-omero-server/issues/42
This was replaced renamed in Ansible 2.5

Seems trivially :thumbsup:


Update vrs-about.md

Some suggestions for the portions about limitations. Let's talk about them...

Re-opening to try again...


Refine specialization constants API

#2381 makes them just ranges into raw [u8] data. This isn't very convenient.
One way to improve it could be to carry the data inside the specialization constant instead of storing it separately:
pub struct Specialization<V: Borrow<[u8]>> {
    /// Constant identifier in shader source.
    pub id: u32,
    /// Value to override specialization constant.
    pub value: V,
}

I made such an attempt in this branch but it appears to be adding a level of indirection that isn't very convenient for either portability or Vulkan backend.
Another direction of improvement would be adding a trait that allows conversion of primitive types (like f32 or u32) to a specialization constant, because otherwise using them directly is very inconvenient.

I don't have any extra input on it, but given it's a potentially API breaking change, we need to resolve this somehow by 0.2.

Using Borrow trait for owned values isn't really nice for API. For instance, &Vec<u8> doesn't fit here.

@omni-viral we use Borrow everywhere else. Could you clarify on why &Vec<u8> isn't fitting?

Vec<u8> does implement Borrow<[u8]>
&'a Vec<u8> does not implement Borrow<[u8]>

@omni-viral oh, I see now what you are trying to say. Borrow trait for owned values isn't really nice. That's true, and that's why we are using Deref<Borrow<..>> everywhere, not owned borrow.
To be fair, in case of Vec<u8> you can just coerce it to a slice, and it works.

AsRef is implemented transitively for references.
But then AsRef<[u8]> is not implemented for &[u8]
Let me illustrate it with this snippet

Deref<Borrow<..>>

Never saw such bound in gfx-hal API?


Iconv does not work on alpine linux

I'm not familiar with it, but, probably, musl libc does not support //IGNORE suffix.
my patch:
diff --git a/src/iconv.cr b/src/iconv.cr
index d1a824e..acd7bcb 100644
--- a/src/iconv.cr
+++ b/src/iconv.cr
@@ -8,7 +8,7 @@ struct Iconv
     original_from, original_to = from, to

     @skip_invalid = invalid == :skip
-    {% unless flag?(:freebsd) %}
+    {% unless flag?(:freebsd) || flag?(:musl) %}
     if @skip_invalid
       from = "#{from}//IGNORE"
       to = "#{to}//IGNORE"

alpine linux: 3.5
crystal: 0.20.5

I know musl-libc doesn't support //TRANSLIT but there is nothing said about //IGNORE.

Looking at the implementation, there is no support for the //IGNORE nor //TRANSLIT extenstions:
http://git.musl-libc.org/cgit/musl/tree/src/locale/iconv.c

thank you!


Incude used tools in sample table

Description of feature
It can be unclear to some people which tools were used to get the values reported, include toolnames in table

Also include the number of identified contigs for clustering


Enable travis build checking

Enable travis build checking to streamline the process of validating pull requests.

No response from upstream so closing :(


Speed up import of cloudasset data into temporary sql table.

Strip unused data before writing to the database
Switch from SQLAlchemy ORM to Core style bulk CAI inserts
Ensure the maximum number of rows are written on each bulk insert

This is one of a series of changes to reduce the time to complete
a forseti inventory snapshot for large organizations.
This change reduces the time to import data into a cloudsql database
from a test system by about 50%, from about 312 seconds to about 144
seconds for around 350 megabytes of raw data.
Thanks for opening a Pull Request!
Here's a handy checklist to ensure your PR goes smoothly.

[x] I signed Google's Contributor License Agreement
[x] My code conforms to Google's python style.
[x] My PR at a minimum doesn't decrease unit-test coverage (if applicable).
[x] My PR has been functionally tested.
[x] All of the tests pass
[x] I have submitted a corresponding PR in the Forseti Terraform module (if applicable).

These guidelines and more can be found in our contributing guidelines.

Thanks for this! One QQ: where is the switch from SQLAlchemy ORM to Core style bulk CAI insert happening?

It's primarily in the populate_cai_data method in inventory/storage.py


I built a site with Eleventy!

Built https://www.webstoemp.com with Eleventy earlier this year. Have a couple of blogposts up detailing the move from Jekyll and how I use 11ty to build multilingual sites.
Source https://github.com/jeromecoupe/webstoemp

{
"name": "Webstoemp",
"url": "https://www.webstoemp.com/",
"description": "The portfolio and blog of Jérôme Coupé, a teacher, designer and front-end developer from Brussels, Belgium.",
"twitter": "jeromecoupe"
}

please close this issue because the site already exists


Spike: Use Observables instead of Promises

When doing asynchronous operations (http-calls, access database) within our code, we should spike if Promises or Observables work better for us.

Thank you for creating the issue 👍


tusc-sh: init at 1.0.0

Description of changes
simple binary to run the tus protocol for file uploads.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

I took care of both comments.

I've tested more extensively, thank you for suggesting!
I've resolved all the missing dependencies, and I have one error left, but it appears to be a protocol error.
I think this is good to merge right now. I'm working with upstream to try to understand where the issue could be.

After a brief interaction, upstream has made some very minor updates. Updated to 1.0.2
waiting for ofborg to pass before merging.
Let me know if anything though.


OpenJDK Update (July 2024 Patch releases)

Is your feature request related to a problem? Please describe
OpenJDK JDK 22.0.2, 21.0.4, 17.0.12, 11.0.24, and 8u421 have been released [1], [2]
[1] https://blogs.oracle.com/java/post/jdk-2202-2104-17012-11024-and-8u421-have-been-released
[1] https://openjdk.org/groups/vulnerability/advisories/2024-07-16
Describe the solution you'd like
Update JDKs across all maintenance branches.
Related component
Build
Describe alternatives you've considered
N/A
Additional context

https://www.oracle.com/java/technologies/javase/21-0-4-relnotes.html
https://www.oracle.com/java/technologies/javase/17-0-12-relnotes.html
https://www.oracle.com/java/technologies/javase/11-0-24-relnotes.html
https://www.oracle.com/java/technologies/javase/8u421-relnotes.html

[Triage - attendees 1 2 3]
@reta Thanks for creating this issue

Waiting for all jdk artifacts available to update ci runners.


python: expose highest-level API

All strategies enry.language() is now available.
This addresses part of the #154 that should be enough to unblock #246

Thank you for the prompt review, @kuba-- 🙇

this part is ready to be merged but I do not have enough permissions to do that 😢


Replace collection with iteration

I've found a lot of collections, which can be replaced with Iter, save memory and improve performance for first results.
Why collections was used instead of Iter?

Good day,
try suggesting or replacing some. but be careful as your latest patch have broken functionality of "match any". the crate is used in our mission-critical production so I really care about keeping it working properly.


Remove all instances of hackEDU from GitHub repository descriptions

Do you see any offending repos?

Sloth Gifs
On Wed, 25 Nov 2015 3:20 pm Zach Latta<EMAIL_ADDRESS>wrote:

Do you see any offending repos?
—
Reply to this email directly or view it on GitHub
https://github.com/hackclub/meta/issues/325#issuecomment-159488187.

The frontend legacy
On Wed, 25 Nov 2015 3:32 pm Harrison Shoebridge<EMAIL_ADDRESS>wrote:

Sloth Gifs
On Wed, 25 Nov 2015 3:20 pm Zach Latta<EMAIL_ADDRESS>wrote:

Do you see any offending repos?
—
Reply to this email directly or view it on GitHub
https://github.com/hackclub/meta/issues/325#issuecomment-159488187.

I'll compile a list when I get home. Currently on mobile :)
On Wed, 25 Nov 2015 3:33 pm Harrison Shoebridge<EMAIL_ADDRESS>wrote:

The frontend legacy
On Wed, 25 Nov 2015 3:32 pm Harrison Shoebridge <
<EMAIL_ADDRESS>wrote:

Sloth Gifs
On Wed, 25 Nov 2015 3:20 pm Zach Latta<EMAIL_ADDRESS>wrote:

Do you see any offending repos?
—
Reply to this email directly or view it on GitHub
https://github.com/hackclub/meta/issues/325#issuecomment-159488187.

@zachlatta OK. These are the current offenders.

[ ] https://github.com/hackclub/sloth-gifs
[ ] https://github.com/hackclub/website
[ ] https://github.com/hackclub/hack-camp
[ ] https://github.com/hackclub/hackedu-legacy (possibly should be kept for legacy reasons)
[ ] https://github.com/hackclub/frontend-legacy (possibly should be kept for legacy reasons)

We can probably leave hack camp alone because it is a legacy project.

Alright, sloth-gifs and website are both fixed. Let's keep references to hackEDU in hack-camp, hackedu-legacy, and frontend-legacy for archival purposes.


Update silencer-lib, silencer-plugin to 1.7.9

Updates

com.github.ghik:silencer-lib
com.github.ghik:silencer-plugin

from 1.7.7 to 1.7.9.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.ghik" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.github.ghik" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #450.


Update README.md

Thank you @jacksorjacksor !
But because your PR to docs repo already counts as a code PR no need to add a low code/ no code contribution to here.
So closing this PR!

Ah my apologies, thanks for that!


Fixed activateROS for catking build and conda envs. Updated readme with uninstall

See title. See also #7 . Catkin build achieved compatibility by removing the check for CMakeLists.txt. Is there some other test you would want to add @ARoefer (such as src, devel dirs) or is checking for the venv enough?

Thank you for addressing this! I have so far not been able to come up with another reliable check. Maybe the combination of src and pyenv? But maybe the check for pyenv is enough, as it seems to be an atypical name for venvs. Alternative solution would be that we put an empty file at the root of a workspace. Something like .rosvenv_ws. What do you think?

Alternative solution would be that we put an empty file at the root of a workspace. Something like .rosvenv_ws. What do you think?
This feels like unnecessary complexity to me, so I'd rather avoid it.
Maybe the combination of src and pyenv?
Yeah, in my workspace I only have build  condenv.txt  devel  logs  src. But folders like src are so generic that there's probably little value in checking for them. So checking for pyvenv/condaenv.txt is likely enough.

Also, there's little damage done by not checking well enough at this stage. Worst case is that something like ws sourcing will fail. So maybe, we can extend the error message that is generated in such a case to point to the possibility that the ws dir is faulty.

I think you're right, there's such a thing as overthinking it. I'll merge it now and then we can improve in the future. (I wan't to improve the docker tooling anyway some time in July)


Repo does not have "ground_truth_package.h"

#include "ground_truth_package.h" in the main.cpp

Thanks! Addressed with 0281ea9


Adding new account does not work as Amplify bucket is empty

Describe the bug
I am trying to add a new account to import resources from. This is the parent account of the AWS organization and Perspective is deployed in a sub-account. However, i can neither view the template or deploy the template in the step for beginning the Cloud Formation stack deployment into accounts you want to from. This is because the amplify bucket from which these templates are supposed to be deployed is completely empty.
To Reproduce
Steps to reproduce the behavior:

Go to 'Account and Regions' in the left menu.
Click on 'Import' tab.
Enter 'account number' and select 'region' for the region you want to import.
Click on 'Import' button.
Click on 'view template' button.
You get error message

NoSuchKey
The specified key does not exist.
public/cfn/import-templates/accounts/5b3674e3eb1129c175d1b591322c361ad24c856f.template
77DFBD373653757A
Lhrl+tjubVkk2mqZ/iCg/+ZMxsuA5b/Nvpn1IwnDw9oBg/NKUo1ZA+lwib5xFBR0HmT7V7EpGJM=

Expected behavior
I should be able to view the cloud formation template that needs to be deployed into my account that i am trying to add into Perspective.
Screenshots
If applicable, add screenshots to help explain your problem.
Browser (please complete the following information):

Name Chrome
Version 85.0.4183.121

Additional context
Add any other context about the problem here.
The reason we are getting this error because even through the AWS Perspective launch cloudformation stack completed successfully, including all the sub-stacks, the amplify buckets has no contents inside. And referring to the documentation for Automated Deployment, it does not have any additional steps to perform to populate the amplify bucket.

I believe this is related to #8. We have fixed the issue in v1.0.1. If you are happy to delete the v1.0.0 stack and redeploy using the latest from here that will fix your issue. We understand that might be quite painful and are working on instructions to be able to upgrade from v1.0.0. These will be available soon.
As a temporary workaround, the templates are stored in the DiscoveryBucket. You can download these replace the AccountId and AggregationRegion parameters with the account Dd and Region you deployed Perspective in. Then you can deploy this in the account and Region you are importing.

I believe this is related to #8. We have fixed the issue in v1.0.1. If you are happy to delete the v1.0.0 stack and redeploy using the latest from here that will fix your issue. We understand that might be quite painful and are working on instructions to be able to upgrade from v1.0.0. These will be available soon.
As a temporary workaround, the templates are stored in the DiscoveryBucket. You can download these replace the AccountId and AggregationRegion parameters with the account Dd and Region you deployed Perspective in. Then you can deploy this in the account and Region you are importing.

I will undeploy and redeploy. However, in this case the documentation link for the launch template should also be updated constantly to refer to the correct version of the cloudformation template.

I've read trough the issue #8 and this is not the same error. This error occurs because the amplify bucket is empty. If the step to enable "Public Access to Amplify Bucket" is a mandatory step, then we should make sure that it is documented appropriately as not being optional or provide the steps for a manual workaround if the optional step cannot be performed.

+1

Please create a new issue tagged with v1.1.0 if you are still experiencing this issue.


Elemental Barrage might be bugged

Describe the bug
My save game is dependent on Tabletop Tweaks from selecting arcanist exploits specific to this mod (quick study and druid spell as a wizard), so I can't load a game with it disabled.  Whenever I use the Elemental Barrage mythic feat on a melee character with lots of attacks, the game will freeze randomly during combat.  Happens very often in real time, not as often in turn based mode.  I have disabled every other mod, but I can't disable this mod to test it, as my save game wont load.
To Reproduce
Steps to reproduce the behavior:
Participate in combat, especially with multiple mobs in real time with a melee character, hasted, with multiple attacks and elemental barrage.  My specific character with it is a sorceror dragon disciple eldritch knight Lich who uses a flame weapon + acid gloves to set off elemental barrage.  Unfortunately, I can't tell if Tabletop tweaks is part of the problem or not, since I can't disable the mod for testing.
Expected behavior
Game not to freeze during combat.

Sounds like the freeze I get when I have 20 undead champions all getting exploded by Corrupted Blood at the same time, hah.


Update protoc-gen-validate to v0.6.6

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/lgtm

/approve


Frends.AmazonSQS.Send

https://github.com/CommunityHiQ/Frends.Community.AWS.SQS

I can do this.

Hi


build: Update Vulkan-Headers and VUL to 1.3.270

CI Vulkan-ExtensionLayer build queued with queue ID 80897.

CI Vulkan-ExtensionLayer build # 795 running.

CI Vulkan-ExtensionLayer build # 795 passed.


add STARK Curve field emulated parameters

This PR simply adds the parameters of the prime field of the STARK Curve ( https://docs.starkware.co/starkex/crypto/stark-curve.html )  for experimentation.

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

yep +1 nothing prevents defining this in the repo that imports gnark


Adding alignement lines (snapping)

In most drawing softwares, alignment lines are shown when moving an object to place it more easily at specific places (centered with another object, aligned to the left, etc.)
Is there a way to add this? Not necessarily by default but, for example, when pressing a specific key.
Here is a picture of what I mean by alignment lines:

Any feedback?


Events: Affecting other nodes and edges with node events

Hi
I'm building a social connections graph where, while hovering on a person, her closest friends (nodes/edges) are also highlighted.
I made a small demo but there's some glitchy behaviour with ForceLayout.
https://codesandbox.io/s/github/chervontsev/vue-network-graph-test-v1/tree/main/?file=/src/App.vue
Is there a more optimal way to do this?
Test cases:
Case 1:

Initial state, upper left checkbox OFF
Hover mouse on any edge - edge thickens, adjacent nodes thicken
Hover mouse on any node - closest related nodes thicken
Result: in current setup all as expected

Case 2:

Initial state, upper left checkbox ON
Hover mouse on any edge - edge thickens, adjacent nodes thicken
Hover mouse on any node - closest related nodes and edges thicken
Result: in current setup, while hovering on a node, whole structure starts twitching and spinning

Assumptions
While the twitching and spinning version looks nice and smooth, a lot of elements are being recalculated and redrawn at the same time. Displaying 30-50 nodes, cranks up the CPU load drastically.
This effect happens when I mutate the widths that are set in the edge model.
Presumably this change triggers the graph rebuild.
For nodes I used a custom template and access them via $ref.
This way the data model is not affected and the structure stays in place.
Conclusion
Is there maybe a way to put refs on edges?
Or a way to stop/restart forceSimulation outside of ForceLayout?

Fixed it : )
https://codesandbox.io/s/github/chervontsev/vue-network-graph-test-v2-fixed/tree/main/?file=/src/App.vue
The problem seems to have been in changing the model.
But if you put your edge behaviour logic in the config everything works as expected!
The same solution can be propagated to other config parts if the need arises.
Quite an awesome library : )

Hi @chervontsev,
Thank you for your detailed report and sharing a workaround!
You have already reached what I consider to be the optimal solution.
However, I think that the behavior where changing any property of an edge triggers a recalculation of the force simulation is undesirable, so I have fixed the library. (v0.5.14)
In the latest version, the recalculation is not performed only by changing the edge model as in the first code.
Thank you for using v-network-graph!
Best Regards,

Thank you : ) I'll definitely check it out!


Close ZIP file before deleting

This is necessary on Windows (unfortunately), otherwise a "file in use" exception is generated, and this crashes createdb.

Thanks so much for the fix. It's been a long time since I tested it on Windows.
Did everything else work ok? If so I'll merge this in and do a patch release.

I haven't noticed any other problems, but then again I haven't done much more than do the initial install and site creation. I'll certainly keep an eye out, though.

Ok cool. Thanks again for the patch.


if i have older version of game, can i use older versions of REFramework that support my  games build?

i have an older version of the game so theoretically i could use older versions of REFramework for it to work or am i wrong with this assumption?

Don't open issues like this; use the discussions tab. Follow the template given when opening issues as well. You do not give enough information and are leaving large gaps in attempting to solve the issue.


❓ Phone auto-locks while recording is occuring

Question
My iPhone 11 pro is set to 30 seconds auto-lock. When recording, the auto-lock kicks in. I can't find anything in this plugin to help regulate this.
Shouldn't this plugin keep the device awake while video is recording, or at least expose an api to do that?
What I tried
No response
VisionCamera Version
2.14.0
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar questions in the issues page as well as in the discussions page and found none.

I'm also experiencing this issue. I'll note that there are packages to rectify this (ie react-native-keep-awake) so i think there is a way, but those packages arent downloading properly and i believe are deprecated.
need help too!

Closing as this is a stale issue - this might have been fixed with the full rewrite in VisionCamera V3 (🥳) - if not, please create a new issue.

Hey!
Thanks for this awesome package.
I'm not sure it's fixed. I can see my iPhone going to sleep using v3 while recording a video.
I think I can use something like isIdleTimerDisabled equivalent in JS as a workaround.
I use React Native without Expo

Hey - thank you.
I believe all apps with active AVCaptureSession/AVCapturePreviewLayer instances do not go to sleep, this is the first time I heard about this - does this happen in the example app as well?

Yes I can reproduce in the example app using the main branch. For the record I'm using an iPhone 13 (iOS 17.2).
However to reproduce I have to comment this part in CaptureButton to make sure the recording continues without holding down the button (same case in my app)
case State.END:
case State.FAILED:
case State.CANCELLED: {
  // exit "recording mode"
  try {
    if (pressDownDate.current == null) throw new Error('PressDownDate ref .current was null!')
    const now = new Date()
    const diff = now.getTime() - pressDownDate.current.getTime()
    pressDownDate.current = undefined
    if (diff < START_RECORDING_DELAY) {
      // user has released the button within 200ms, so his intention is to take a single picture.
      await takePhoto()
    } else {
      // user has held the button for more than 200ms, so he has been recording this entire time.
      await stopRecording()
    }
  } finally {
    setTimeout(() => {
      isPressingButton.value = false
      setIsPressingButton(false)
    }, 500)
  }
  return
}

Let me know if you want me to dig further

Looks like Amazon recommends setting the idleTimer off on their example app for broadcasting too.
https://docs.aws.amazon.com/fr_fr/ivs/latest/LowLatencyUserGuide/broadcast-ios-getting-started.html#broadcast-ios-disable-idle-timer

Created a PR to fix this: #2421


runtime error in sentiment.py

Hi @lvwerra,
I encounter the following error while running the sentiment.py example.
/anaconda3/envs/py39_rl/lib/python3.9/site-packages/trl/trainer/ppo_trainer.py", line 375, in generate
    input_ids=query_tensor.unsqueeze(dim=0), **generation_kwargs
AttributeError: 'list' object has no attribute 'unsqueeze'

The error is appearing at the following line
for epoch, batch in tqdm(enumerate(ppo_trainer.dataloader)):
    query_tensors = batch["input_ids"]

    # Get response from gpt2
    response_tensors = ppo_trainer.generate(
        query_tensors, return_prompt=False, length_sampler=output_length_sampler, **generation_kwargs
    )

I am using the latest version of trl.
Any suggestions ?

What version of trl are you using?
Try installing the latest version from source using
pip install -e .

Thanks @lvwerra . I guess i figured that out as well. It requires
transformers==4.27.4
torch=2.0.0


Allow users to check out multiple units/plants

Currently, users are able to bulk evaluate and submit for different units and plants if the data is loaded in the workspace. The evaluate and submit process will check out the affected units and plants.
Per 1/22/2024 meeting with EPA, enhance the application to also allow users to check out multiple units and plants prior to evaluate/submit.

Maybe change to plant level locking, instead of unit level.

@annalbrecht we discussed this ticket internally and would like to deprioritize this to Phase 1B. Could you please relabel this? Thank you.
FYA - This ticket was deprioritized since it isn't a blocker for industry submission and is a low priority compared with other development work.

@annalbrecht EPA would like for us to pull this ticket back into Phase 1B with a change in priority from High to Low. If the dev team has any questions, we can discuss them at the next meeting.


Setup docstring linting

Is your feature request related to a use case or problem? Please describe.
Docstrings are a returning issue on PRs, it would be great if we could automate the feedback on them as well for Google style docstrings.
Describe the solution you'd like
Setup the docstyle and docparams plugins.
What is the urgency from your perspective for this issue? Is it blocking important work?
P2 - we should do it in the next couple of quarters

@balopat :  I'd like to help out in this endeavour since I've some experience in RTD & Sphinx.  Would you still be needing help?

Hi @divya-mohan0209 - that would be great! Our .pylintrc lives under dev_tools/conf. We are following Google style.  Please raise a PR and we can discuss the breakages and how to fix them, which flag to turn on or off!

The new PR for this issue is: https://github.com/quantumlib/Cirq/pull/4267

OK, an incremental linter was too intrusive as it requires a significant refactor for even individual files.  I vote for @balopat 's Idea to  use # pylint: disable=missing-raises-doc for all the cases where the refactor is not trivial.

Sounds good to me! :) Thank you for bearing with us through this!

This is follow up from discussion on PR #4345
@mpharrigan @balopat Maybe we should go through the docparams lint rules and decide which ones we actually want. Here they are with notes:
differing-param-doc (W9017) - "%s" differing in parameter documentation Please check parameter names in declarations.
10 occurrences
differing-type-doc (W9018) - "%s" differing in parameter type documentation Please check parameter names in type declarations.
0 occurrences
multiple-constructor-doc (W9005) - "%s" has constructor parameters documented in class and init Please remove parameter declarations in the class or constructor.
0 occurrences
missing-param-doc (W9015) - "%s" missing in parameter documentation Please add parameter declarations for all parameters.
In a current PR, reviewers have said this would be useful
missing-type-doc (W9016) - "%s" missing in parameter type documentation Please add parameter type declarations for all parameters.
38 occurrences
missing-raises-doc (W9006) - "%s" not documented as being raised Please document exceptions for all raised exception types.
Already implemented
useless-param-doc (W9019) - "%s" useless ignored parameter documentation Please remove the ignored parameter documentation.
0 occurrences
useless-type-doc (W9020) - "%s" useless ignored parameter type documentation Please remove the ignored parameter type documentation.
0 occurrences - no cost and some benefit
missing-return-doc (W9011) - Missing return documentation Please add documentation about what this method returns.
218 occurrences
missing-return-type-doc (W9012) - Missing return type documentation Please document the type returned by this method.
105 occurrences
missing-yield-doc (W9013) - Missing yield documentation Please add documentation about what this generator yields.
118 occurrences
missing-yield-type-doc (W9014) - Missing yield type documentation Please document the type yielded by this method.
64 occurrences
redundant-returns-doc (W9008) - Redundant returns documentation Please remove the return/rtype documentation from this method.
12 occurrences
redundant-yields-doc (W9010) - Redundant yields documentation Please remove the yields documentation from this method.
1 occurrence

This has created 13 pages of files with TODOs!  I'm going to start hacking on them, it's a good way for me to read code that has been added since I stopped paying as much attention :)


Failed to retrieve IDEs: nodename nor servname provided,or not known

Description
For me is #477 not fixed. This happens with latest main release of Coder v2.18.0 and latest Jetbrains Gateway 2024.3. Coder provider plugin is  at version 2.15.1.

As one could see the --bg host is not found on a Windows machine. Even the entry is in ssh config file. Curious is that the same Workspace (we open IntelliJ via Button and using URI handler) works on a linux system. It seems this has something to do with Windows implementation.
This is a blocker for us as it is unable to start new IntelliJ based workspaces from Windows clients.

I am running into the same problem on a Windows machine.  I've tried the connection from within IntelliJ Ultimate and from within Jetbrains Gateway.
There seems to be a problem in de SSH config.  I cannot even connect with plain ssh.
I run coder config-ssh, I can see the changes in ~/.ssh/config.  When I run ssh coder.a, I get the following output:
/bin/sh: line 0: exec: C:Usersb.derijckereAppDataLocalcoder-gateway82pk376jkrj44.pit-1.try.coder.appcoder-windows-amd64.exe: not found
kex_exchange_identification: Connection closed by remote host

It seems to want to fail to execute the ProxyCommand
If I change the single \ in the ProxyCommand to \, from ProxyCommand C:\Users\b.derijckere\AppData\Local\coder-gateway\82pk376jkrj44.pit-1.try.coder.app\coder-windows-amd64.exe ... to ProxyCommand C:\\Users\\b.derijckere\\AppData\\Local\\coder-gateway\\82pk376jkrj44.pit-1.try.coder.app\\coder-windows-amd64.exe ..., then the SSH connection succeeds.
Unfortunately, I cannot make the same changes in de config for the Host coder-jetbrain--... since that config is overwritten by the coder-gateway plugin each time.


The conflict is cause

system : Mac Mini M1
using : python3 -m pip install deepface
and error message is :
use these package versions have conflicting dependencies.
The conflict is caused by:
deepface 0.0.72 depends on tensorflow>=1.9.0
deepface 0.0.71 depends on tensorflow>=1.9.0
deepface 0.0.70 depends on tensorflow>=1.9.0
deepface 0.0.69 depends on tensorflow>=1.9.0
deepface 0.0.68 depends on tensorflow>=1.9.0
deepface 0.0.67 depends on tensorflow>=1.9.0
deepface 0.0.66 depends on tensorflow>=1.9.0
deepface 0.0.65 depends on tensorflow>=1.9.0
deepface 0.0.64 depends on tensorflow>=1.9.0
deepface 0.0.63 depends on tensorflow>=1.9.0
deepface 0.0.62 depends on tensorflow>=1.9.0
deepface 0.0.61 depends on tensorflow>=1.9.0
deepface 0.0.60 depends on tensorflow>=1.9.0
deepface 0.0.59 depends on tensorflow>=1.9.0
deepface 0.0.58 depends on tensorflow>=1.9.0
deepface 0.0.57 depends on tensorflow>=1.9.0
deepface 0.0.56 depends on tensorflow>=1.9.0
deepface 0.0.55 depends on tensorflow>=1.9.0
deepface 0.0.54 depends on tensorflow>=1.9.0
deepface 0.0.53 depends on tensorflow>=1.9.0
deepface 0.0.52 depends on tensorflow>=1.9.0
deepface 0.0.51 depends on tensorflow>=1.9.0
deepface 0.0.50 depends on tensorflow>=1.9.0
deepface 0.0.49 depends on tensorflow>=1.9.0
deepface 0.0.48 depends on tensorflow>=1.9.0
deepface 0.0.47 depends on tensorflow>=1.9.0
deepface 0.0.46 depends on tensorflow>=1.9.0
deepface 0.0.45 depends on tensorflow>=1.9.0
deepface 0.0.44 depends on tensorflow>=1.9.0
deepface 0.0.43 depends on tensorflow>=1.9.0
deepface 0.0.41 depends on tensorflow>=1.9.0
deepface 0.0.40 depends on tensorflow>=1.9.0
deepface 0.0.39 depends on tensorflow>=1.9.0
deepface 0.0.38 depends on tensorflow>=1.9.0
deepface 0.0.37 depends on tensorflow>=1.9.0
deepface 0.0.36 depends on tensorflow>=1.9.0
deepface 0.0.35 depends on tensorflow>=1.9.0
deepface 0.0.34 depends on tensorflow>=1.9.0
deepface 0.0.33 depends on tensorflow>=1.9.0
deepface 0.0.32 depends on tensorflow>=1.9.0
deepface 0.0.31 depends on tensorflow>=1.9.0
deepface 0.0.30 depends on tensorflow>=1.9.0
deepface 0.0.26 depends on tensorflow>=1.9.0
deepface 0.0.25 depends on tensorflow>=1.9.0
deepface 0.0.24 depends on tensorflow>=1.9.0
deepface 0.0.23 depends on tensorflow>=1.9.0
deepface 0.0.22 depends on tensorflow>=1.9.0
deepface 0.0.21 depends on tensorflow>=1.9.0
deepface 0.0.20 depends on tensorflow>=1.9.0
deepface 0.0.19 depends on tensorflow>=1.9.0
deepface 0.0.18 depends on tensorflow>=1.9.0
deepface 0.0.16 depends on tensorflow>=1.9.0
deepface 0.0.15 depends on tensorflow>=1.9.0
deepface 0.0.14 depends on tensorflow>=1.9.0
deepface 0.0.13 depends on tensorflow>=1.9.0
deepface 0.0.12 depends on tensorflow>=1.9.0
deepface 0.0.11 depends on tensorflow>=1.9.0
deepface 0.0.10 depends on tensorflow>=1.9.0
deepface 0.0.9 depends on tensorflow>=1.9.0
deepface 0.0.7 depends on tensorflow>=1.9.0
deepface 0.0.6 depends on tensorflow>=1.9.0
deepface 0.0.5 depends on tensorflow>=1.9.0
deepface 0.0.4 depends on tensorflow>=1.9.0
deepface 0.0.3 depends on tensorflow>=1.9.0
deepface 0.0.2 depends on tensorflow>=1.9.0
deepface 0.0.1 depends on tensorflow>=1.9.0
To fix this you could try to:

loosen the range of package versions you've specified
remove package versions to allow pip attempt to solve the dependency conflict

+1

@gaplan are you on windows too?

@mikila85 no i'm using mac mini M1  - Bigsur.

Python 3.8.9 does not support tensorflow 1.9.0 :(

I am using python 3.8.12 and tf 2.2.0 in my mac (intel processor)
try to install tensorflow 2.2.0 before installing deepface
pip install tensorflow==2.2.0
pip install deepface

Please look this issue as well: https://github.com/serengil/deepface/issues/391

closed due to duplication


Feedback!  ( not sure where to put/write this )

Hey @morris
First off, great work, I read this last night and really loved your write up and the code.  Constraining yourself like you did really makes you think differently about how to do things, and I think the way that you did it helped shed light onto what is possible.
Pure JS is really powerful.  For me I started on a similar adventure building a website called Stor.Guru.  I previously picked up the book Single Page Web Applications, SPA Book, by @mmikowski, which has a robust format for SPAs similar to what you are doing but not 100% Vanilla JS.  The book uses jQuery and a couple other libraries so it is not as constrained as your experiment but it does capture the essence of what you are trying to accomplish by not using frameworks.

One point that he brings up is that you can effectively develop in this style with only a browser, text editor, and terminal.  No other applications or frameworks are necessary. This keeps development cycles very quick.  You can also do this anywhere, even without an internet connection. Keeping it as easy as JS, CSS, & HTML files that you can open in your browser makes this setup incredibly simple, and that is a huge benefit.
As you continue down this path, like you have started with your application, you can use techniques that make this simple style of development even more powerful.  Your thoughts on event-based data flow and custom events are spot on. Custom events are a core part of the app that is built in the SPA Book.  This setup allows for any new module in the app to just subscribe to messages that are important to the part of the page that it manages. But the internal messages are only the first layer. The SPA Book uses WebSockets with socket.io, for messaging with the server that are intercepted by a "Data Layer" which then dispatches the custom events to rest of the application.  Not advocating for Socket.IO here, just trying to explain how the messaging is done in this instance.  The native WebSocket API I believe is pretty simple and supported everywhere now a days.  Handling new events is as easy as calling the "update" function with the new data. You basically have Socket Msg --> Custom Event --> Module Update()
Having messaging go through a single point, the Data Layer, is key.   With the fake layer you can simulate messaging from the server that looks identical to what the rest of the application expects and without ever even starting a webserver. you now have a backend for the application you are developing with just a browser, a text editor, and some static JS, CSS, & HTML files. The fake layer basically becomes your unit tests for the frontend application and a way to quickly develop new modules.
The final part of the SPA book which you might find interesting is Client-Side Storage and I see you have used localStorage some in vanilla-todo.  In the book they use an in memory database, TaffyDB, but I have since used IndexedDB in place of it, which is also a native API.  Client-Side Storage allows you to store relevant data models from the backend for use and modification by the client. With IndexedDB you have a full object store, now offline, that the client can use to build its page with.  To me IndexedDB is the next level of localStorage. I use the idb library as a wrapper for the native API since it is Promised, also not ES5 but still pretty low level, almost vanilla, js.
When you combine Client-Side Storage with the Fake and Data Layers you can build an application that has a faked backend ( including database ) that communicates and sends messages to your frontend. The frontend handles messages, stores data in Client-Side Storage, and fires custom events listened to by each feature module who go get the new data from client storage and update their little portion of the page.  All of this with just static files, a browser, and text editor.  All of which can be used offline. Pretty cool!
I have implemented this type of faked communication for the demo mode of my app, Stor.Guru Demo, just as an example of it working.
Disclamer: It was not my intent to bore you with this much feedback or to come off like I actually know what I'm doing lol. But I think I was just so excited to see someone else go in the opposite direction of JS frameworks and discover the power in what they can build. Your write up is really well done and my post ( this issue ) is pretty poor and does not due justice to the SPA Book or the topics from it I was explaining. I am not trying to pitch the SPA Book, I just noticed that the topics your post and it brought up are similar! Kudos for the awesome write up of your code and the code itself, I would love to chat more about this topic and your experiences with it!  Thanks!

I appreciate your feedback very much, thank you! I've heard of the book. It certainly tries to tackle similar problems (frameworks do, too). Anyways I believe being active in the field (using vanilla, using frameworks, or reading books) and trying to find good practices builds understanding and experience, and you seem to be doing that :)
For this kind of discussion, I think e-mail is a better fit<EMAIL_ADDRESS>✨
Thanks again, and good luck with your project!

Hi @morris and @zinglax
Author of the book here (along with Josh Powell, of course). I have a pretty comprehensive project used in a number of high-volume commercial products.  The only major sticking point in more general adoption IMO is the lack of generator scripts.  But those are coming.
Updates are continuing as this is the basis for yet another commercial product.  Pending upgrades include messaging usinjg JSON Pure over native WebSockets and a client upgrade to ES6. Socket.IO is no longer needed or beneficial now that implementation has become consistent and reliable :)
Cheers, Mike


[SPARK-12521][SQL][WIP] JDBCRelation does not honor lowerBound/upperBound

JDBCRelation is not bounding the rows when lowerBound/upperBound are given. This change honors the bounds given.

Can one of the admins verify this patch?

Marking it [WIP] to invite discussion here. : ) As I suspect the original code includes infinity on both smaller than side and greater than side for a reason.

@xguo27 see my comment on the JIRA: https://issues.apache.org/jira/browse/SPARK-12521
I think this is intended, and should not be altered.

Thanks @hvanhovell for clarifying it up. I will close this PR.


Fixed the bug for linkcode_resolve function

Description
This change adjusts the linkcode_resolve function in the conf.py file to properly handle the mapping of module names to their corresponding GitHub URLs. Specifically, it addresses the issue where the assumption about the source code location for CondorProvider is incorrect.
Changed Behaviour
After this change, when generating documentation, the links to the source code on GitHub for modules like parsl.provider.CondorProvider will correctly point to the appropriate location, ensuring accurate reference to the source code.
Fixes
Fixes # (issue)
Type of change
Choose which options apply, and delete the ones which do not apply.

Bug fix

Tagging @benclifford because I'm not super familiar with what exactly this bug is fixing.

@christailu do you still want to work on this PR?


Permission Denied while mounting NFS shared folders

Hi,
I'm getting this output when trying to set a boot2docker-vagrant instance. Can you please recommend what can I do in this case?
https://gist.github.com/anonymous/780f7fc28b44d7e393f8
Thanks,
Alfredo.

Hi @Brolag,
Please post here the output from cat /etc/exports and showmount -e

Hi @lmakarov,
Thanks for your quick answer, here is the output:
MacBook-Pro-de-Alfredo:gn2 alfredo$  cat /etc/exports
/Users/alfredo -alldirs -mapall=alfredo -network <IP_ADDRESS> -mask <IP_ADDRESS>
VAGRANT-BEGIN: 501 e3bca53b-d940-4d58-a2c3-cee8f47f289a
"/Users/alfredo/workspace/gn2" <IP_ADDRESS> -alldirs -mapall=501:20
VAGRANT-END: 501 e3bca53b-d940-4d58-a2c3-cee8f47f289a

MacBook-Pro-de-Alfredo:gn2 alfredo$ showmount -e
Exports list on localhost:
/Users/alfredo                      <IP_ADDRESS>

@Brolag

Remove /Users/alfredo -alldirs -mapall=alfredo -network <IP_ADDRESS> -mask <IP_ADDRESS> from your /etc/exports
sudo nfsd restart
vagrant reload

Where did you get that record from? It overlaps/conflicts with the one set by Vagrant.
By the IP address (<IP_ADDRESS>) I can guess you were trying to set NFS mounts up manually with the stock boot2docker? :)

@lmakarov, done, but still getting the permission denied error. https://gist.github.com/Brolag/c535312c56285eb554c2

@Brolag It is pretty unlikely, that the box IP would be different, but can you please check that it is indeed <IP_ADDRESS>?
vagrant ssh -c 'ifconfig | grep <IP_ADDRESS>'

Also try the following (one at a time, vagrant reload in-between attempts):

disable the firewall on your Mac
change 'tcp' to 'udp' in the following line

@lmakarov,
This is the output of the first command:
MacBook-Pro-de-Alfredo:gn2 alfredo$ vagrant ssh -c 'ifconfig | grep <IP_ADDRESS>'
Connection to <IP_ADDRESS> closed.
This is my routing table
https://gist.github.com/Brolag/74eb51584e592177cb18
The other steps didn't work.

@Brolag looks like the box IP address is wrong (not <IP_ADDRESS>).
Please do a full vagrant ssh -c 'ifconfig'.
There should be something similar to this
eth1      Link encap:Ethernet  HWaddr 08:00:27:E0:D0:F2
          inet addr:<IP_ADDRESS>  Bcast:<IP_ADDRESS>  Mask:<IP_ADDRESS>
          inet6 addr: fe80::a00:27ff:fee0:d0f2/64 Scope:Link
          UP BROADCAST RUNNING MULTICAST  MTU:1500  Metric:1
          RX packets:145604 errors:0 dropped:0 overruns:0 frame:0
          TX packets:76548 errors:0 dropped:0 overruns:0 carrier:0
          collisions:0 txqueuelen:1000
          RX bytes:46527209 (44.3 MiB)  TX bytes:13787799 (13.1 MiB)

Have you modified the Vagrantfile?

@lmakarov I haven´t modified it.
´´´
MacBook-Pro-de-Alfredo:gn2 alfredo$ vagrant ssh -c "ifconfig"
docker0   Link encap:Ethernet  HWaddr 7A:DE:10:4A:13:F0
inet addr:<IP_ADDRESS>  Bcast:<IP_ADDRESS>  Mask:<IP_ADDRESS>
inet6 addr: fe80::78de:10ff:fe4a:13f0/64 Scope:Link
UP BROADCAST RUNNING MULTICAST  MTU:1500  Metric:1
RX packets:0 errors:0 dropped:0 overruns:0 frame:0
TX packets:8 errors:0 dropped:0 overruns:0 carrier:0
collisions:0 txqueuelen:0
RX bytes:0 (0.0 B)  TX bytes:648 (648.0 B)
eth0      Link encap:Ethernet  HWaddr 08:00:27:28:0F:77
inet addr:<IP_ADDRESS>  Bcast:<IP_ADDRESS>  Mask:<IP_ADDRESS>
inet6 addr: fe80::a00:27ff:fe28:f77/64 Scope:Link
UP BROADCAST RUNNING MULTICAST  MTU:1500  Metric:1
RX packets:401 errors:0 dropped:0 overruns:0 frame:0
TX packets:289 errors:0 dropped:0 overruns:0 carrier:0
collisions:0 txqueuelen:1000
RX bytes:45973 (44.8 KiB)  TX bytes:38334 (37.4 KiB)
eth1      Link encap:Ethernet  HWaddr 08:00:27:CD:18:E0
inet addr:<IP_ADDRESS>  Bcast:<IP_ADDRESS>  Mask:<IP_ADDRESS>
inet6 addr: fe80::a00:27ff:fecd:18e0/64 Scope:Link
UP BROADCAST RUNNING MULTICAST  MTU:1500  Metric:1
RX packets:6 errors:0 dropped:0 overruns:0 frame:0
TX packets:11 errors:0 dropped:0 overruns:0 carrier:0
collisions:0 txqueuelen:1000
RX bytes:1524 (1.4 KiB)  TX bytes:1427 (1.3 KiB)
lo        Link encap:Local Loopback
inet addr:<IP_ADDRESS>  Mask:<IP_ADDRESS>
inet6 addr: ::1/128 Scope:Host
UP LOOPBACK RUNNING  MTU:65536  Metric:1
RX packets:80 errors:0 dropped:0 overruns:0 frame:0
TX packets:80 errors:0 dropped:0 overruns:0 carrier:0
collisions:0 txqueuelen:0
RX bytes:5760 (5.6 KiB)  TX bytes:5760 (5.6 KiB)
´´´

For whatever reason the box is using the stock boot2docker IP range (192.168.56.x) ...
Open VirtualBox Preferences => Network => Host-only Networks and remove all adapters - http://take.ms/ITjc4. Next do a vagrant reload.

@lmakarov awesome! mounting is working fine now! Thanks a lot!

@Brolag I'm glad this works for you now :)


modified ops.c's wordcount to support chinese wordcount in UTF-8

I'm using spf13-vim configuration and it is using airline-vim, then i found this issues https://github.com/vim-airline/vim-airline/issues/1356
For example, the string "一二三四五,六七八九十" will only be treated as two words. However in the Chinese word statistics, it should be 10.
I am not familiar with the existing code style, but this change can make it work. I found wordcount function relatively in both ops.c:line_count_info and misc2.c:vim_isspace(int x), but i only modified ops.c.
Thank you!

Well, the test did not pass. The meaning of error message is because the other language utf-8 problem, then such a change is not perfect, then I close this issues.


Create a sg721-vending collection that uses computed vaules

Override minting and querying of token_uri to use a computed value of base_uri + token_id.
Updating should only update base_uri.

Should also have the ability to:

update existing tokens
add new tokens


got an unexpected keyword argument 'sample_rate'

I got this when i try to convert an audio file using hybrid[rmvpe+fcpe] and i got a slowed and muffled voice audio, what can it be?
Converting audio 'assets\audios\BeYou.wav'...
Calculating f0 pitch estimations for methods ['rmvpe', 'fcpe']
An error occurred during audio conversion: init() got an unexpected keyword argument 'sample_rate'
Traceback (most recent call last):
File "C:\Users\Rushi\Desktop\Ai\Applio-3.2.2\rvc\infer\infer.py", line 256, in convert_audio
audio_opt = self.vc.pipeline(
File "C:\Users\Rushi\Desktop\Ai\Applio-3.2.2\rvc\infer\pipeline.py", line 585, in pipeline
pitch, pitchf = self.get_f0(
File "C:\Users\Rushi\Desktop\Ai\Applio-3.2.2\rvc\infer\pipeline.py", line 351, in get_f0
f0 = self.get_f0_hybrid(
File "C:\Users\Rushi\Desktop\Ai\Applio-3.2.2\rvc\infer\pipeline.py", line 274, in get_f0_hybrid
self.model_fcpe = FCPEF0Predictor(
TypeError: init() got an unexpected keyword argument 'sample_rate'

duplicated, fixed in the next update


[BUG] Better log signal for host started

Describe the bug
We're working towards releasing the rust host as default but ran into an issue where we don't have a clear log message like with the OTP host to validate whether or not the host started successfully (from logs alone).
Expected behavior
Two parter here. Update the host to give a log we can key off of then update the test then fix this line:
https://github.com/wasmCloud/wash/blob/f9279294ea7602ad6bbc55a5f3dc8940f2d46d71/crates/wash-lib/src/start/wasmcloud.rs#L491
**Environment (please complete the following information) **

OS: macos
Shell zsh
wash Version [e.g. 0.2.0]

Additional context
Add any other context about the problem here.

Think this was fixed by wasmcloud/wasmcloud#619

@brooksmtownsend The host emits a better start message, but the wash tests need to be updated to check for the updated message :)


ncurses: 6.0 -> 6.0-20170729

Ncurses has no stable release since v6.0 which was released two years ago.
Given that even debian started using unstable weekly updates, I see no issue in doing the same.
This has been tested on my package set, but not the whole nixpkgs.
:warning: this is a small mass-rebuild.
Fixes #19785.

Does upstream want the world to use their unreleased version?

@bjornfor I would say that this is the versioning scheme of ncurses. Every distro is using these, from debian to fedora. I can however ask to the main developper, but I am not sure it is worth bothering him with that.

I noticed after the upgrade to 6.0-20170729, tig used in a zsh session with TERM=xterm-256color has display problems.  No crash, just that the graph is visually off.
Downgrading ncurses back 6.0 or upgrading to 20170902 fixed the issue I experienced.
 I assume that you fix was http://invisible-island.net/ncurses/NEWS.html#t20170827.
I hope they will release an official release in the near future, but I thank you for your PR in the meantime.


Unknown option: base.extra with Babel 6

error with babel-loader
how to use babel-plugin-react-transform ??
ERROR in ./src/templates/azul/app.js
Module build failed: ReferenceError: [BABEL] /Users/carlos/www/mall4g-templates/src/templates/azul/app.js: Unknown option: base.extra
    at Logger.error (/Users/carlos/www/mall4g-templates/node_modules/babel-core/lib/transformation/file/logger.js:41:11)
    at OptionManager.mergeOptions (/Users/carlos/www/mall4g-templates/node_modules/babel-core/lib/transformation/file/options/option-manager.js:262:18)
    at OptionManager.init (/Users/carlos/www/mall4g-templates/node_modules/babel-core/lib/transformation/file/options/option-manager.js:416:10)
    at File.initOptions (/Users/carlos/www/mall4g-templates/node_modules/babel-core/lib/transformation/file/index.js:190:75)
    at new File (/Users/carlos/www/mall4g-templates/node_modules/babel-core/lib/transformation/file/index.js:121:22)
    at Pipeline.transform (/Users/carlos/www/mall4g-templates/node_modules/babel-core/lib/transformation/pipeline.js:42:16)
    at transpile (/Users/carlos/www/mall4g-templates/node_modules/babel-loader/index.js:14:22)
    at Object.module.exports (/Users/carlos/www/mall4g-templates/node_modules/babel-loader/index.js:87:14)

Do you use beta version 2.0.0?
npm install --save-dev babel-plugin-react-transform@beta

Current stable 1.1 is babel5 and not babel6 compatible.

I am seeing the same, using
 ├──<EMAIL_ADDRESS>

For anyone who runs upon this in the future:
I had to remove the webpack configuration parameters and move this to the babel config proper. After doing this, everything appears to be working again.
https://github.com/kpdecker/linoleum-webpack/commit/e4f9e31a1ffffd27f5e595485d1ea2e88e91ae5a

Ah I'm using .babelrc so could not reproduce.
Also when you pass params to the loader in webpack it is ok to JSON.stringify it. Try it if you keen, that might work.

yes, i am use beta version 2.0.0

Your .babelrc file has an invalid option extra. This option was taken away in Babel 6 in favor of plugin specific options.

how to config webpack with babel-plugin-react-transform, in app with server side renderin ?

In your webpack config, when you specify the babel-loader object, add a query property, which will contain the plugin configuration.
It should be something like this.

thanks @arhcstolen  (y)


Rewrite to TS/ES6

rewrite the conditional loader to TS/ES6

https://github.com/biotope/rfcs/pull/1

solved by #15


circular import issues

We need to import modules from here to there because the Types are needed.
Here's a good article to solve this problem: Yet another solution to dig you out of a circular import hole in Python

Fixed by the commit.

This few days, I guess I get into circular import problems. With the same code, Only I get this bug, of which I'am confused.
Below this my analyzation.

When I import Contact class in python console. It also occur import problems.

from src.wechaty.user.contact import Contact
Traceback (most recent call last):
  File "<input>", line 1, in <module>
  File "/Applications/PyCharm.app/Contents/plugins/python/helpers/pydev/_pydev_bundle/pydev_import_hook.py", line 21, in do_import
    module = self._system_import(name, *args, **kwargs)
  File "/Users/wujingwujing/PycharmProjects/python-wechaty/src/wechaty/__init__.py", line 7, in <module>
    from .wechaty import Wechaty
  File "/Applications/PyCharm.app/Contents/plugins/python/helpers/pydev/_pydev_bundle/pydev_import_hook.py", line 21, in do_import
    module = self._system_import(name, *args, **kwargs)
  File "/Users/wujingwujing/PycharmProjects/python-wechaty/src/wechaty/wechaty.py", line 37, in <module>
    from .user import (
  File "/Applications/PyCharm.app/Contents/plugins/python/helpers/pydev/_pydev_bundle/pydev_import_hook.py", line 21, in do_import
    module = self._system_import(name, *args, **kwargs)
  File "/Users/wujingwujing/PycharmProjects/python-wechaty/src/wechaty/user/__init__.py", line 8, in <module>
    from .message       import Message
  File "/Applications/PyCharm.app/Contents/plugins/python/helpers/pydev/_pydev_bundle/pydev_import_hook.py", line 21, in do_import
    module = self._system_import(name, *args, **kwargs)
  File "/Users/wujingwujing/PycharmProjects/python-wechaty/src/wechaty/user/message.py", line 41, in <module>
    from .room import Room
  File "/Applications/PyCharm.app/Contents/plugins/python/helpers/pydev/_pydev_bundle/pydev_import_hook.py", line 21, in do_import
    module = self._system_import(name, *args, **kwargs)
  File "/Users/wujingwujing/PycharmProjects/python-wechaty/src/wechaty/user/room.py", line 33, in <module>
    from .message import Message

When I look deep into pytype analyse file, it contains 51 files which need to analyse. Our project only contains 26 files need to be analysed.
When I look into .pytype/pyi directory. It contains many *.pyi-1 files, meaning these file have been analysed more then one time.

└── src
    ├── wechaty
    │   ├── accessory.pyi
    │   ├── accessory.pyi-1
    │   ├── config.pyi
    │   ├── types.pyi
    │   ├── types.pyi-1
    │   ├── user
    │   │   ├── Contact.pyi-1
    │   │   ├── Friendship.pyi-1
    │   │   ├── Image.pyi-1
    │   │   ├── Message.pyi-1
    │   │   ├── Room.pyi-1
    │   │   ├── Tag.pyi-1
    │   │   ├── __init__.pyi-1
    │   │   ├── favorite.pyi-1
    │   │   ├── mini_program.pyi
    │   │   └── url_link.pyi
    │   └── wechaty.pyi-1
    └── wechaty_puppet
        ├── contact.pyi
        ├── file_box.pyi
        ├── friendship.pyi
        ├── message.pyi
        ├── mini_program.pyi
        ├── puppet.pyi
        ├── room.pyi
        └── url_link_payload.pyi

I have change a lot of code, so dare not submit code. I will modify the code from master branch, and add my code to the branch little by little.

Yes, please split the code into different features/branches and then submit into sperate Pull Requests.
That is the best practice of the progress of our developing because we can see what happens via different PRs and make discussion & debugging more affectional and easier.

I met the problem when I did refactoring work on it, it helps much, and as a conclusion, I post the usage guide as below:

If a type is only used for typing hinting but never been referenced in the code, we can feel free to import that classes under a if TYPE_CHECKING clause, like:

from typing import TYPE_CHECKING

if TYPE_CHECKING:
    from wechaty import (
        Message,
    )

# so we can safely use the Message class as a type hinting:
def on_message(msg: Message):
    print(msg)

# !!ATTENTION!! but we CANNOT use it as a reference:
def on_message(msg):
    # DONT: if you use it like this way, you are trying to use it as a reference and will possibly got a `Message is not defined` error
    if isinstance(msg, Message):
        print(msg)
    # DONT: instantiate the class is also not allowed in this way.
    msg = Message('hello')

So, before you decide whether to put the import variable inside or outside the TYPE_CHECKING clause, you should be aware that whether you will use the name as a reference or purely as a type hinting usage.

So the problem is solved now, the issue is ok to be closed.

@fish-ball  Can you show us your solution ?
# !!ATTENTION!! but we CANNOT use it as a reference:
def on_message(msg):
    # DONT: if you use it like this way, you are trying to use it as a reference and will possibly got a `Message is not defined` error
    if isinstance(msg, Message):
        print(msg)
    # DONT: instantiate the class is also not allowed in this way.
    msg = Message('hello')

My solution is to import the target module at local function. So, it doesn't need to import the target module at the top.

@wj-Mcat local importing is ok, but it seems that the pylint check will fail.

@fish-ball  Of course, if the first choice will cause the circular dependency problems which can't be avoided, I will make the second choice with the skiping annotation.


Fix bug type in EnergyMarketCodeType

The input parameter code_type of DeliveryArea was the protobuf type delivery_area_pb2.EnergyMarketCodeType.ValueType instead of the python wrapper type EnergyMarketCodeType.

Moved to #32


[TC-MC-9.2] Simulated test crashing when made a run

Summary:
when trying out simulated device test case for "[TC-MC-9.2]" we are observing a crash showing response as : "Response Failure: IM Error 0x000005C3: General error: 0xc3"
Execution logs :
TC_MC_9_2_app_side.txt
TC_MC_9_2_CHip_tool.txt

This issue appears to be happening, because MatterPostAttributeReadCallback gets called before sending report in BuildAndSendSingleReportData which causes the thread to End Early and terminate the message transmission to the controller. The simulated app will need to wait until after the SenReport is complete and then move on to the next test. This will require SDK changes, assigning to @vivien-apple to implement changes.
Steps to reproduce:

Generate Attached simplified YAML for simulated app
Compile simulated app
Start simulated app
Pair simulated app with chip-tool
Run the following chip-tool command: chip-tool applicationbasic read application-version 1 1

Test_TC_MC_9_2_Simulated.txt

This issue was created by me using a account that i had to delete due to access issues, I will continue to follow these issues with my current id.

This issue needs to be closed.


When SSHing into a KubevirtCI, long terminal commands are not wrapped, therefore looking weird

What happened:
When SSHing into a KubevirtCI node, if the command is long enough, it is not warpped by instead being messy overriding the beginning of the command:
> ./cluster-up/ssh.sh node02
Last login: Sun Sep 15 08:35:14 2024 from <IP_ADDRESS>

# This is short enough, looking good
[vagrant@node02 ~]$ find -regex '.*pod.*' -type d -exec cat {}/memory.max \;
[vagrant@node02 ~]$

# Adding `&& ls {}` at the end of the find exec command
{} \;t@node02 ~]$                                                            s \;                                                                          l
\;agrant@node02 ~]$ find -regex '.*pod.*' -type d -exec cat {}/memory.max &&  \;

This is how it looks like:

What you expected to happen:
The command to be wrapped into a new line in a human-readable way.
How to reproduce it (as minimally and precisely as possible):
As explained above. 100% reproducible.
Environment:

KubeVirt version (use virtctl version): v1.3 (main)

/label good-first-issue

Seems like a PTY issue along with SIGWINCH not being handled properly.
Execing into the Pod and then running ssh.sh seems to work fine.
Also verified this by doing the following:
$ podman exec -it <kubevirtci container> sh
$ trap 'echo test' WINCH

Resizing the terminal doesn't do anything.

TBH this may be a general RFE for virtctl console (virtctl ssh seems to be aware according to dosu)
Back to the subject, for the kubevirtci ssh script, I don't think we're aware of SIGWINCH or try to do any resizing at all.
As a quick W/A one could run stty size on the host machine and apply the same values on the kvci node

@iholder101  - does this belong more to kubevirt/kubevirtci? I can transfer the issue if you want?

@iholder101 - does this belong more to kubevirt/kubevirtci? I can transfer the issue if you want?

Yes, feel free to do so if you think that's more appropriate! Thanks!


Support for ruby 2.5 and 2.6?

I was looking into using the redis-clustering gem (depends on redis-cluster-client) in a project I support that happens to be on ruby 2.6. I'm able to use redis-rb in the project since that supports ruby 2.5+ but we'd like to use redis cluster. I made some changes in my fork of redis-cluster-client that gets the test suite passing on 2.5 and 2.6. If I opened a PR with the changes would that be something you'd be willing to take a look at or do you want to keep redis-cluster-client at ruby 2.7+?
Additionally the changes to support ruby 2.6 were minimal (changing filter_map) compared to ruby 2.5 changes so if you didn't want to support 2.5 but were okay with 2.6 I can make a PR with just the 2.6 changes.
Thanks!

Does the redis gem v4.8.0 may fit your project? We don't plan to support EOL versions of Ruby in this gem so far. Although we would like many people to use this gem, I don't know how many projects use EOL versions of Ruby. So I googled it. It seems that there might be many users keeping to use EOL Ruby version.
https://www.jetbrains.com/lp/devecosystem-2021/ruby/
Hmm... I have ambivalence, both I want to increase users and I don't want to maintain legacy code. Please give me some time to think about it.

May the redis gem v4.8.0 fit your project?

From what I understand of the v4.8.0 cluster code if a pipeline has keys across more than one node you would get a cross slot error which would be problematic for us.

I don't know how many projects use EOL versions of Ruby. So I googled it. It seems that there might be many users keeping to use EOL Ruby version.

That data is interesting, thanks for sharing it!
To be completely transparent, if I had a gem I was working on I wouldn't want to support EOL ruby versions. Maybe the best path to take for the health of redis-cluster-client is for it to not support EOL ruby versions at this time and I keep 2.6 support in my fork. Hopefully that could increase my teams priority in upgrading rails so we can get on ruby 2.7 and then switch away from my fork.
Thank you for taking the time to respond and have a great day!


dockerTools: create /tmp in rootLayer

Description of changes
Create a /tmp directory inside the docker root layer. Closes  #257172
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Hello, dockerTools maintainer here :wave:
Thanks to @ryota-ka's PR #301755, my attention was brought back to this PR, for which I didn't get almost any time to perform a review. I didn't see a reason to make a fuss about it back then, but - unfortunately - now I do.
See my review there, but I'll summarize:

Adding /tmp to all images reduces security by introducing an unnecessary and undeclared capability
dockerTools is first and foremost a tool, a mechanism, and not a complete solution. Opinionated code belongs in separate functions.

Unfortunately, such an interface has yet to be built, whether that's based on existing NixOS modules, or something custom.
Would you be interested in such an interface?
Would you mind for this PR to be reverted?

I don't mind that, feel free to revert this


Update Go dependencies in hack utils

Closes #863

@JeffNeff you reported this, would you mind taking a look?

@JeffNeff you reported this, would you mind taking a look?

sorry I missed this.. i was out last week attending a funeral.


Add a module for Emacs daemon

Motivation for this change

fix #15807
Several users are manually adding a systemd user service to their configuration.nix. A NixOS module avoid copy/pasting half-baked definitions.

Things done

[x] Tested using sandboxing
Built on platform(s)

[x] NixOS

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

/cc @TikhonJelvis @anderspapitto @PierreR @ttuegel @rvl

I was wondering if it would be helpful to add an option to set the EDITOR variable to an emacsclient wrapper script. Something like this: b9cf774.

In the nixos test attached to this PR, the waitForWindow at the end seems ok but the screenshot is always black. Can somebody help me please?

@rvl thank you very much for your proposal, I merged it. I only slightly changed it to take care of https://github.com/rvl/nixpkgs/commit/b9cf774a7f5bf45a927cb61133d21fbb4110075d#commitcomment-17946739. My only problem with this patch is that if defaultEditor is set to false, EDITOR is set to the empty string instead of being untouched.

OK, right on both counts. This should fix the EDITOR variable:
environment.variables = if cfg.defaultEditor then {
  EDITOR = mkOverride 900 "${editorScript}/bin/editor-emacs.sh";
} else {};

$machine->waitForText("GNU Emacs");

where does this text come from. It works, but I don't see it in the screenshot.

Also it might be simpler to run the test as root, like in the firefox test.

I can't make that work

That text comes from the modeline: "For information about GNU Emacs and the GNU system, type C-h C-a."
Running as root worked for me. (I assume it's root, or some default user that's not alice). Try making the test minimalist using icewm just like firefox.nix.

@rvl do you have an idea how to make systemctl --user daemon-reload work for root? The current script fails with this output:
machine: running command: systemctl --no-pager show 'multi-user.target'
machine: exit status 0
machine: must succeed: systemctl --user daemon-reload
machine# Failed to connect to bus: No such file or directory
machine: exit status 1
machine: output:
error: command `systemctl --user daemon-reload' did not succeed (exit code 1)
command `systemctl --user daemon-reload' did not succeed (exit code 1)
cleaning up

Failed to connect to bus: No such file or directory

I think that can be fixed by setting XDG_RUNTIME_DIR=/run/user/$(id -u), assuming that /run/user/0 has been setup by logging in to the icewm session.

@rvl thanks, it works fine now. I have the impression systemctl --user enable emacs is not necessary (and not supported by NixOS). Can I change the comment at https://github.com/NixOS/nixpkgs/pull/16356/commits/99f4e40b83d09eaad07c1979d657916ff2585fb9#diff-754738b9bf80ce26b4b69e5e7216cb28R32 ?
Can I merge the PR?

Thanks. I recommend changing the $machine->sleep(1) into $machine->waitUntilSucceeds("systemctl --user daemon-reload");. This would effectively wait for the user service manager to be started.
I'm not really sure about how is the best way to enable a systemd user service in NixOS. Because configuring systemd.user.services.emacs.wantedBy = [ "default.target" ]; will cause an emacs daemon to be started for any user who logs in (e.g. root as well). There needs to be a symlink in ~/.config/systemd/user so that it automatically starts emacs daemon. But maybe do delete the text about systemctl --user enable emacs because the warning message from systemctl is quite confusing.
It looks merge-worthy to me, but I'm not highly experienced with NixOS.

[...] and the --login part would be optional and configurable with some boolean.

what is exactly the impact of this option and how would users configure environment variables? With what's already in the PR, it is already possible to add variables to environment.variables and they are automatically added to the emacs daemon.

@DamienCassou I was wondering if the enabled option should be split into something like installed and enabled. The first option would just put the emacs service file in place. The second option would imply the first and make the service wantedBy = [ "default.target" ].
@michalrus I think you have a power user case. In your situation, try directly setting systemd.user.services.emacs.serviceConfig.ExecStart = pkgs.lib.mkForce "$SHELL --login -c 'emacs --daemon'".

[...] and the --login part would be optional and configurable with some boolean.

what is exactly the impact of this option and how would users configure environment variables?

@rvl, @DamienCassou, true, it might be a power-user case… but I dunno.
I’m using my dotfiles on several different systems (non-NixOS, but I like NixOS the most, obviously :heart_eyes_cat:). It is a common practice to set your “login” env vars (i.e. ones that every application you’re running as your user sees) in your ~/.profile.
That’s why I think Emacs being one the most important applications used by a user, should also read ~/.profile automatically. And that’s easily achievable with $SHELL --login -c 'exec emacs', if you’re not starting it from a user session that has already read ~/.profile.

and the --login part would be optional and configurable with some boolean

I'm not against adding a boolean for that, it doesn't cost much. @michalrus Could you please send me a patch? (that way you have to explain the purpose of the boolean :-)).

wondering if the enabled option should be split into something like installed and enabled.

I love this solution. Thanks
Someone on IRC mentioned yesterday that systemctl support for user services was far from great and he recommended using XDG autostart instead (https://github.com/ttuegel/nixos-config/blob/master/programs/emacs.nix). Do you have an opinion about that?

@DamienCassou, sure. Something like that:
diff --git a/nixos/modules/services/editors/emacs.nix b/nixos/modules/services/editors/emacs.nix
index 92b9a90..777dd97 100644
--- a/nixos/modules/services/editors/emacs.nix
+++ b/nixos/modules/services/editors/emacs.nix
@@ -51,6 +51,18 @@ in {
         using the EDITOR environment variable.
       '';
     };
+
+    exportProfile = mkOption {
+      type = types.bool;
+      default = false;
+      example = true;
+      description = ''
+        When enabled, just before starting the daemon, will read
+        user’s custom environment variables defined in HOME (e.g.
+        in ~/.profile or similar) using user’s login shell of
+        choice for the task.
+      '';
+    };
   };

   config = mkIf cfg.enable {
@@ -59,7 +71,9 @@ in {

       serviceConfig = {
         Type      = "forking";
-        ExecStart = "${pkgs.bash}/bin/bash -c 'source ${config.system.build.setEnvironment}; exec ${cfg.package}/bin/emacs --daemon'";
+        ExecStart = ''
+          ${pkgs.bash}/bin/bash -c 'source ${config.system.build.setEnvironment}; exec $SHELL ${optionalString cfg.exportProfile "--login"} -c "exec ${cfg.package}/bin/emacs --daemon"'
+          '';
         ExecStop  = "${cfg.package}/bin/emacsclient --eval (kill-emacs)";
         Restart   = "always";
       };

On the other hand this is not just specific to Emacs… And this solution is a hack for Emacs only. I dunno. Reading some (global?) user defined variables might be considered… impure? =) Anyway, it’s for you to decide, thanks! /cc @kmicu

À propos, couldn’t the first ${pkgs.bash}/bin/bash -c be ommited, as there will be a regular file generated from ExecStart and its absolute path will be used?

does xdg autostart work without any Desktop Environment (say, with only i3 or xmonad installed)? Anyway I think the problems that it solves are exactly the same problems that are solved by running from inside a login shell (or is that not true)?
also, doesn't using a login shell remove the need for source ${config.system.build.setEnvironment} ?

XDG autostart won't work when you're sshing into a server without a desktop environment. (Anyway in the long term I imagine gnome-session, etc will be replaced by the systemd user service manager.) An option could be added, but from the user's perspective that would be redundant. If the systemd user service works well enough, it doesn't matter how the emacs daemon is started.
@michalrus, I checked your dotfiles. Which environment variables would you need in emacs which aren't better set through environment.variables?

@michalrus thanks for your patch. Could you please add a unit test just like the existing one does?

@michalrus, I checked your dotfiles. Which environment variables would you need in emacs which aren't better set through environment.variables?

@rvl, all enviroment vars are better set in environment.variables, that’s for sure. =) But, unfortunately, I’m also using these dotfiles on other systems. Hmm. Possibly CHROMIUM_USER_FLAGS, if Chromium was my default browser and I happened to launch it from Emacs. But that’s a silly example. Hmm, maybe DEBFULLNAME, if I happened to be creating a Debian package. Or my custom WINELOADER if I ran some Wine-powered program from Emacs. But these are silly examples, aren’t they? Well, if I set these in environment.variables I’d have to track two places for any changes (to have these vars on other non-NixOS systems), I think that’s what my… intuition was about.
Generally, when launching Emacs in a graphical evironment—emacsclient really, with Emacs daemon running in the background, but that’s just optimization details—I would assume that my custom variables that I carefully, over the years, put into my ~/.profile or similar would be visible there…
But this assumption may be completely wrong. I’m not so sure anymore… and I don’t really like the idea of environment variables, as these are global.

@michalrus thanks for your patch. Could you please add a unit test just like the existing one does?

@DamienCassou, thank you, too, for considering it, but right now I don’t know if we should do this (details above)… Do you want to do this? Is this a valid idea after all? Hmmm.
And if it is, possibly, it shouldn’t be local to just Emacs, right? There are other systemd --user services out there.

More on that matter, courtesy of @kmicu:

GNOME on Wayland currently has a problem with environment variables: there are some traditional ways to set environment variables for X11 sessions or login shells using shell script fragments (/etc/X11/Xsession.d, /etc/X11/xinit/xinitrc.d, /etc/profile.d), but these do not apply to Wayland, or to noninteractive login environments like cron and systemd --user. We are also keen to avoid requiring a Turing-complete shell language during session startup, because it's difficult to reason about and potentially rather inefficient.
― https://www.collabora.com/about-us/blog/2016/06/22/gtk-hackfest-2016/

Ok guys. Here is a new version. The only thing I'm not happy with it the name of the editor editor-emacs.sh. Any suggestion?
If you are all ok with that PR, I will just merge it.

Looks great. Agreed about the name of the editor script. What about calling it ec?

ec looks non explicit enough to me (but that's exactly the name of my bash alias :-)). I propose emacseditor instead (just like emacsclient).
If nobody complains, I will merge the current commits.

Ok, merged. Thank you all.

Thank you!
BTW, #7329 is probably related? (Re: different environments in systemd user units and login sessions).

@michalrus normally, this is taken care of by https://github.com/NixOS/nixpkgs/pull/16356/files#diff-754738b9bf80ce26b4b69e5e7216cb28R74. If you find any problem, please tell me.


Coverage computation does not match the pull request

Summary
Fixes #7142

Occurred changes and/or fixed issues
Technical notes summary

Changed ref on the GH checkout action to Checkout pull request HEAD commit instead of merge commit, avoiding wrong file changes reports and generating false deltas (see first screenshot and here the log).
Restored HTML coverage generation and uploaded as artifacts
Removed deprecated nyc configuration in package.json, reintroduced with a wrong merge confllict
Updated excluded paths such like .nuxt and .nuxt-prod
Corrected included paths, as E2E skipped pkg
Printed out list of changed files on coverage, to ensure correct computation
Updated the Codecov comment below by configuration

Note: Commit info have been restored by toggling the app in the rancher user settings. Full enable triggers checks on coverage delta.
Areas or cases that should be tested
Generated statistic in this page should solve all the mentioned issues. It's also possible to checkout from this branch and try a PR to own fork and see results, without polluting this repository.
Areas which could experience regressions
None.
Screenshot/Video

This PR is currently not really ready, but the status seems to be required to be listed on Codecov.

@cnotv PR looks a bit stale, do we need it in 2.7q3 or can it be moved out?

@cnotv PR looks a bit stale, do we need it in 2.7q3 or can it be moved out?

definitely moved.

@cnotv PR looks a bit stale, do we need it in 2.7q3 or can it be moved out?

As defined in the other comment, this PR is needed to be set as ready since the results is going to be displayed here.

They may have updated something as now it correctly report only the changed files.
It does not compare to master though, which has the same percentage.


Allow to specify --group and --marathon value via ENV

This implements #197
Should be fully backwards compatible.

Looks good. Have you tested it?

I tested that when started with HAPROXY_GROUP=myapplb HAPROXY_MARATHON=https://leader.mesos:8443 it results in
   15 ?        S+     0:00 python3 /marathon-lb/marathon_lb.py --syslog-socket /dev/null --haproxy-config /marathon-lb/haproxy.cfg --ssl-certs /etc/ssl/cert.pem --command sv reload /marathon-lb/service/haproxy --sse --marathon https://leader.mesos:8443 --group myapplb --health-check

and when started without any extra args it runs with the same defaults it's currently using.

Cool, thanks!

This doesn't seem to work. I'm getting the following error:
/marathon-lb/run: line 116: MARATHON_ARGS[@]: unbound variable

I'm looking into it. Did you provide any special arguments?


Questions about evaluation indicators

I downloaded the UAVDT data set and evaluation tool, but I did not find out how to use it to generate test results from the tool. I did a lot of searches, and there are no similar tutorials on the Internet. I searched the original paper and did not find the author. Contact information.
But I really need the same evaluation criteria as the open source papers to evaluate the quality of my network. You published your paper with this data set. I wonder if you can send me the testing tools you use. I want to use this to evaluate your own network.

Hi, after some research, I could find the dataset page. Maybe it changed location since I used it? I don't know. Anyway, it is there now: https://sites.google.com/view/grli-uavdt. Tell me if you have any problems accessing it. If you scroll to the bottom of the page, the evaluation tool is the link titled "[DET/MOT toolkit (Google Drive)]", at this URL: (https://drive.google.com/file/d/19498uJd7T9w4quwnQEy62nibt3uyT9pq/view. After that, you can see that the testing set is defined in the file utils/CalculateDetectionPR_overall.m. Good luck!

I did download the tool in that data set and run the MATLAB file. I don't know how to evaluate my network using the matlab tool. There seems to be no similar tutorial on the Internet. I'm really sorry. I may not have described it clearly before. I'm sorry.

在 2021年12月1日，上午10:06，Hughes Perreault @.***> 写道：

Hi, after some research, I could find the dataset page. Maybe it changed location since I used it? I don't know. Anyway, it is there now: https://sites.google.com/view/grli-uavdt. Tell me if you have any problems accessing it. If you scroll to the bottom of the page, the evaluation tool is the link titled "[DET/MOT toolkit (Google Drive)]", at this URL: (https://drive.google.com/file/d/19498uJd7T9w4quwnQEy62nibt3uyT9pq/view. After that, you can see that the testing set is defined in the file utils/CalculateDetectionPR_overall.m. Good luck!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Do you use read_csv to generate the test text file on the entire data set? I see that such a function is written in your code. So can I copy this part of your code to generate the text file needed for testing. Can you tell me which of these is the script you used to generate the txt file in the end? Also, can I get so many txt files directly by running this script?

Ah okay I see... You have to run the file CalculateDetectionsPR. I suggest you try to run results for Faster R-CNN first, as it is the one that's in the file by default. You can change the name at line 2 in this file, and it will change the folder that is evaluated in "RES_DET". Also, if you have trouble, and you only want the overall results, you can comment the other lines, like this. If I remember correctly, you should be able to reproduce the results in the original UAVDT paper like that:
`%% calculate the AP scores of detectors in terms of different attributes
detection = 'det_FRCNN'; % detector names, i.e., det_FRCNN, det_RFCN, det_RON, det_SSD
CalculateDetectionPR_overall(detection); % calculate the AP scores of the test-set.
% CalculateDetectionPR_seq(detection); % calculate the AP scores in sequence-level attribute
% calculate the AP scores in object-level attribute
% for obj_attr = 1:3
% 1 for Vehicle Category;
% 2 for Vehicle Occlusion;
% 3 for Out-of-view;
% CalculateDetectionPR_obj(detection, obj_attr);
% end`

/run_on_csv_method.py Is it a script that generates test results? It seems that he should not be able to generate all the txt files, how do you operate？

This script generates test results, but in an incorrect format for this tool. I had another very simple script which I no longer have access to. But really if you check RES_DET\det_FRCNN\M0203.txt, you will see that the format is simple, you can write a conversion script very easily yourself. Some values are unused for object detection evaluation, and are set to 1 or -1.

SPLIT = 'uav-test'
if SPLIT == 'test':
    source_lines = open(os.path.join(DATASET_DIR, 'test-tf-all.csv'), 'r').readlines()
    target_file = open(os.path.join(base_dir, exp_id, 'ua-test.csv'), 'a')
elif SPLIT == 'train1on10':
    source_lines = open(os.path.join(DATASET_DIR, 'train-tf.csv'), 'r').readlines() # + open('/store/datasets/UA-Detrac/val-tf-all.csv', 'r').readlines()
    target_file = open(os.path.join(base_dir, exp_id, 'ua-train1on10.csv'), 'w')
elif SPLIT == 'trainval':
    source_lines = open(os.path.join(DATASET_DIR, 'train-tf-all.csv'), 'r').readlines() + open('/store/datasets/UA-Detrac/val-tf-all.csv', 'r').readlines()
    target_file = open(os.path.join(base_dir, exp_id, 'ua-trainval.csv'), 'w')
elif SPLIT == 'uav-test':
    source_lines = open('/store/datasets/UAV/val.csv', 'r').readlines()
    target_file = open(os.path.join(base_dir, exp_id, 'uav-test.csv'), 'w')
elif SPLIT == 'uav-val':
    source_lines = open('/store/datasets/UAV/val-sub.csv', 'r').readlines()
    target_file = open(os.path.join(base_dir, exp_id, 'uav-val2.csv'), 'w')
elif SPLIT == 'changedetection':
    source_lines = open('/store/datasets/changedetection/changedetection.csv', 'r').readlines()
    target_file = open(os.path.join(base_dir, exp_id, 'changedetection.csv'), 'w')
elif SPLIT == 'ped1':
    source_lines = open('/store/datasets/ped1/csv.csv', 'r').readlines()
    target_file = open(os.path.join(base_dir, exp_id, 'results.csv'), 'w')

Have you saved the path of the picture in a csv file?
Read file used to generate txt file?
So how to generate the txt file needed for matlab tool evaluation?
Are all data used for evaluation?
I'm still not sure?
I only need to know how to generate the txt file needed for matlab tool testing to realize the evaluation of my network.
I'm really sorry to disturb you, but it is still difficult for me to understand the generation logic of a series of txt files and what kind of data it uses only from your code.
If it is convenient, can you give me your contact information, wecat or Telegram or other, I only have a little question? But it seems difficult to say clearly in the mail?

I can write my own conversion tool. It is really simple.
But do I need to use all the data in the results I generate? Including the pictures of each frame and even the data used in training. If this is the case, then I think I understand.

For my research I converted all the datasets in the same simple CSV format, which is:
path_to_file, x0, y0, x1, y1, label

and then when I generate the results:
path_to_file, x0, y0, x1, y1, label, score

I later use a custom script to reconvert each dataset into their own format for evaluation. The advantage for me is that all my data is in the same format during the work.

Actually I found the script for UAVDT to CSV, here it is: https://1drv.ms/u/s!Ahf1HD3oV-4xgtwyi3vRFiAE5lwBbA?e=FRiIus. Hope this helps!

First of all, thank you for your help, I have solved the previous problem.
However, new problems have emerged. I used centernet as my baseline to train on this data set, using coco indicators, but I found that the indicators change very strangely during training. Loss decreases with training, but the accuracy of the network on the test set gets worse with training, which is far from the results in your paper. I did not use the cneternet you used, but I am sure that there is no problem with my network, and it can get normal results on the coco data set.
I used the ignore.txt file of the dataset to process the data, because I think that unlabeled instances may cause instability in network training. I don't know what kind of treatment you have done?
I currently have no way to get a relatively normal baseline indicator. It makes it difficult for me to continue my work. Do you have any good suggestions?
For small goals, have you made improvements to the official code (it does not seem to be from the paper)?
What index did you use during training, VOC or COCO?


Revert "Revert "[kmac] Use Empty signal from FIFO""

Reverts lowRISC/opentitan#9302

Revised the Author field and force pushed. Let me wait the CI result and merge.


Not able to preview localized Skillmaps

For translators, there is no easy way to preview their Skillmap translations.
The "?forcelang=ar" flag does not seem to work with Skillmap URL's

Do they have the queryparameter in the correct spot? it looks like it's going through https://arcade.makecode.com/--skillmap?forcelang=ja#racer (that said, there are either very few fully translated elements in skillmaps currently, but if you click the cogwheel it at least shows 'high contrast mode' toggle in the given language).
It's pretty easy to put it after the # and have it become part of the hash part by accident, in which case it won't get read / will mess up the skillmap selection https://arcade.makecode.com/--skillmap#racer?forcelang=ja

Ah!  Good to know - I was putting the flag in the wrong place.  I'll let this translator know.

This does not seem to work, I have tried the above on Chrome/Edge an in Incognito.
Once I enter a Skillmap the language of the editor switches back to English.

@jwunderl ?

I see, it's persisting to the editor correctly for approved langs but for unapproved it's ignoring the 'force' state -- we need to propagate the forcelang flag explicitly. thanks for pointing that out!


Add Wi-Fi example

Initial version of the Wi-Fi exercise. Happy to modify/change anything!

Thanks for working on this!
I guess the example / exercise is fine - it just feels a bit weird going from basic interrupt handling to making an http request via WiFi.
Maybe we need some more introduction like explaining that we have support for WiFi via an additional crate and that there is embbeded-svc to control it and how it relates to smoltcp ... something like that. I don't have an idea how it should really look like - and maybe it's a "me" thing
But yeah, looking at the std training they even start with http after "Hello World" 🤔 so would be definitely good to hear from others what they think about it. Also, this is the initial version - so maybe just something to refine later

I think you are completely right, adding those kinds of introduction should help the reader a lot, specially if it's newbie. I just added a small intro, let me know what do you think about it. Im a happy to rewrite/expand/change it as needed


jetbrains: 2024.1.3 -> 2024.1.4

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 331294 run on x86_64-linux 1

5 packages built:

jetbrains.pycharm-community (jetbrains.pycharm-community-src)
jetbrains.pycharm-community-bin
jetbrains.pycharm-professional
jetbrains.rust-rover
jetbrains.writerside


ParetoNBD Posterior Population Sampling and Misc Changes

This PR addresses https://github.com/pymc-labs/pymc-marketing/issues/319, https://github.com/pymc-labs/pymc-marketing/issues/318, and https://github.com/pymc-labs/pymc-marketing/issues/278 for ParetoNBDModel.
The most significant addition is the distribution_customer_population method, which can be used to plot the distribution of estimated customer purchase frequencies:

I also performed some maintenance like removing the "Experimental Status" user warning, and cleaning up the unit tests and dev notebook. build_model() is now auto-called internally, and ParetoNBDModel.fit() also returns self, which is aligned with the sklearn convention.
I experimented with adding Slice Sampling for this PR. Although it is 3-4x faster than NUTS for this particular model, the performance was not as reliable as I'd hoped. In the future I'll create pytensor PRs enabling support for external samplers.

:books: Documentation preview :books:: https://pymc-marketing--401.org.readthedocs.build/en/401/

ping @ricardoV94