nova writes temp snapshot files to the root partition

What version (commit) do you encounter this issue on?
11.2.9-11-g0504bcc
What type of issue is this (bug, feature, documentation, regression)?
seems like a bug to me
What is the expected behavior?
I'd expect the snapshot temp file to be saved on the larger partition, /var/lib/nova.
What is actually happening?
Nova is writing the snapshot to /var/lib/libvirt/qemu/save/, and filling up the root partition in some cases.
How can we reproduce this issue?
nova image-create <image uuid> <snapshot name>
watch -n2 "ls -alh /var/lib/libvirt/qemu/save/"
How can we verify your issue is addressed?
First bug submission, I am not sure what to put here.
If this is a large issue, what can be done immediately to alleviate the issue while working on a longer-term solution?
Symlink /var/lib/libvirt/qemu/save/ to somewhere under /var/lib/nova
What is your evaluation on the importance of this issue, and why?
Very important.  Customers snapshotting large instances can fill up the root partition on the compute node with a single snapshot.
What branches / milestones do you believe this issue should target?
This specific version is the only place I've seen the behavior.  I don't have an accessible lab to test further.
The list below is what is currently supported for backports
Remove branches (lines) you do not wish to target.

[ ] kilo

This ML thread may relate to this issue: http://lists.openstack.org/pipermail/openstack-operators/2016-April/010238.html

What version of libvirt and qemu are you using?
1.2.2 and 2.0 (libvirt and qemu) are known to cause issues.
see http://lists.openstack.org/pipermail/openstack-operators/2016-April/010251.html
adding the following is a possible workaround if we use a newer libvirt/qemu
[workarounds]
disable_libvirt_livesnapshot=False

Here's the help text for that option, the bad part is that cold snapshots cause an instance outage while the snapshot is running.
Live snapshots allow the snapshot of the disk to happen without an
interruption to the guest, using coordination with a guest agent to
quiesce the filesystem.

When using libvirt 1.2.2 live snapshots fail intermittently under load
(likely related to concurrent libvirt/qemu operations). This config
option provides a mechanism to disable live snapshot, in favor of cold
snapshot, while this is resolved. Cold snapshot causes an instance
outage while the guest is going through the snapshotting process.

For more information, refer to the bug report:

   https://bugs.launchpad.net/nova/+bug/1334398

Possible values:

* True: Live snapshot is disabled when using libvirt
* False: Live snapshots are always used when snapshotting (as long as
  there is a new enough libvirt and the backend storage supports it)

Confirmed that even in mitaka it uses /var/lib/libvirt/qemu/save/  it also cleans up after itself when done, so we should be able to rmdir it and replace with a simlink as long as nothing is snapshotting at the time.

This is something that should likely be in OSA (creating the symlink).  Gonna raise it in irc/upstream bug.

looks like we just can't change the path, libvirt maintainers aren't that nice... http://comments.gmane.org/gmane.comp.emulators.libvirt/53301

I don't think this will hit kilo as it is an upstream OSA bug, I'm working on getting it into mitaka/liberty though.
https://bugs.launchpad.net/openstack-ansible/+bug/1585325

master merged, backporting now
kilo is dead though...

ok, we just need sha bumps now
@git-harry when does that happen next for all maintained branches?

@prometheanfire we have a release scheduled for next week, so I will update the SHAs today/Monday

fix committed


tests: disable UI animation for E2E

Issue number:
Summary
Changes
E2E tests are slowed down by animation. This PR disable animation when the app is running under test
User experience
No effect
Checklist
If your change doesn't seem to apply, please leave them unchecked.

[ ] I have performed a self-review of this change
[ ] Changes have been tested
[ ] Changes are documented
[ ] PR title follows conventional commit semantics

no speed up or something


Fetch entries from within entries

greeting -> Hey I am {npc.bob.name}

Committed https://github.com/tabularelf/lexicon/commit/befe472ffbce70d140871f94a382beaa4b690cbe


New live mainnet eu central 1

Context
Changes:

Cherry-picked commit which configures pg-boss-worker properly
Added new live-mainnet@eu-central-1 deployment  (Not currently live until everything syncs. Current domain is tmp-live-mainnet.lw.iog.io)

@iccicci Are we okay with deploying pg-boss change to all environments?

The pool-delist-schedule also needs the schedules.json file and the path where it can be found through environment variable.
Was these two more changes imported in this branch by some other PR?

Are we okay with deploying pg-boss change to all environments?

I would do that not all at once: I would prefer to deploy it on dev-preview, check if it works as expected, proceed with dev-preprod... and so on.

Sorry, I should have mentioned that the way we sync it will not impact existing environments. What we do is:

Create database stakepoolv2
Spin up separate stakepool projector and pg-boss-worker on the new database
Wait for it to sync
Once it does, we switch providers to use new database
Test it on dev
If it works, push it to prod

@iccicci does this action plan sound good?

To make this plan to work we need two pg-boss-worker and stake-pool-projector instances:

a pg-boss-worker, stake-pool-projector and stake-pool-provider instances with old configuration pointing to the existing database
a pg-boss-worker and stake-pool-projector instances with new configuration pointing to the new database

once synced, point the stake-pool-provider to the new database and dismiss old pg-boss-worker and stake-pool-projector instances and old database.
Do you confirm this is the plan?

Yes, I confirm

Yes, I confirm

Ok! Let's proceed


DF User Logger Version 1.3.2 has a malformed Manifest file

It appears the issue is with the tag not having a correct name df-user-logger_1.3.2 instead of df-logger_1.3.2

It appears the issue is with the tag not having a correct name df-user-logger_1.3.2 instead of df-logger_1.3.2

Tags have been corrected.

Tags have been corrected.


puzzlefs linux kernel module

This issue will track the necessary changes required to get puzzlefs into the linux kernel.

The following dependencies are required for implementing a puzzlefs kernel module:

the serde crate and its dependencies https://github.com/Rust-for-Linux/linux/pull/1007/commits
the filesystem abstractions for rust in the linux kernel: https://github.com/Rust-for-Linux/linux/issues/1004 and https://rust-for-linux.zulipchat.com/#narrow/stream/384413-Filesystems/topic/Filesystem.20abstractions
serde_cbor integration into kernel, potentially with the alloc feature if it can be supported: https://rust-for-linux.zulipchat.com/#narrow/stream/291565-Help/topic/Missing.20parts.20of.20alloc
this_error integration into the kernel

Licensing discussion:
https://rust-for-linux.zulipchat.com/#narrow/stream/291565-Help/topic/Missing.20parts.20of.20alloc
https://docs.kernel.org/process/license-rules.html

List of missing rust features in the kernel, needed by puzzlefs:

a TryClone trait, use case: try_clone() a Vec
a TryFromIterator together with a try_collect method, used extensively in puzzlefs and other crates such as hex.

First RFC patch series<EMAIL_ADDRESS>Github branch: https://github.com/Rust-for-Linux/linux/compare/rust-next...ariel-miculas:puzzlefs_rfc_v1
Article in phoronix: https://www.phoronix.com/news/Experimental-Rust-PuzzleFS
Next steps towards upstreaming will be discussed with the Rust for Linux people on June 21st: https://rust-for-linux.zulipchat.com/#narrow/stream/384413-Filesystems/topic/Filesystem.20abstractions

v2 of the kernel driver<EMAIL_ADDRESS>Github branch: https://github.com/ariel-miculas/linux/tree/puzzlefs_rfc_v2

There's a new version of the puzzlefs driver based on Wedson's read-only filesystem abstractions branch. @tych0 FYI.

Nice, thanks for the update. Is your plan to archive this repo at some point?
I'm still finagling some time to work on PuzzleFS at work. I'm hopeful that after the first of the year I'll be able to lend some hours to the effort!

Fair enough, I didn't know if those were going to live in tools/ or something as well. Thanks!

First we need to convince people it is upstream-worthy.

Of course :)

We now have a dedicated page under Rust for Linux.

The latest PuzzleFS patch series is here. There have been discussions about PuzzleFS in this email thread and #114.


feat(middleware): add 'ignore_in_path' feature to ignore unnecessary automatic installations

This adds the ability to ignore automatic installations if an language server command is already available on the user's PATH. I currently have it in place to only be regarding the automatic installer. If the user manually installs an LSP then it should still install as they are specifically choosing to install it.

If the user provides the LSP in the ensure_installed list it also ignores the check for if the executable is found in the path because it would make sense to respect the user's decision to "ensure the server is installed" with nvim-lsp-installer


warning NU1701: Package 'EnterpriseLibrary.TransientFaultHandling 6.0.1304' was restored using '.NETFramework,Version=v4.6.1' instead of the project target framework '.NETCoreApp,Version=v2.0'. This package may not be fully compatible with your project.

When building the Spanner project.  Far from fatal, but we should be using https://www.nuget.org/packages/TransientFaultHandling.Core/

I've had the same error for BigQuery project. Fixed it there to use Retrier from TestUtils. I'll fix this too.

Submitted PR: https://github.com/GoogleCloudPlatform/dotnet-docs-samples/pull/571

Warning	NU1701	Package 'Antlr <IP_ADDRESS>04' was restored using '.NETFramework,Version=v4.6.1 in MVC5 ,netcore


[PE-D] Missing find command for events in UG

There is a find command for events in the Command Summary of the UG and it works in the application but is not reflected in the UG with description, examples and format.

Labels: severity.Medium type.DocumentationBug
original: nguyiyang/ped#7

Duplicate, #219 contains the same issue along with other missing commands


Camera stuttering with rigidbody 3d

I have a simple script that applies addforce to a rigidbody3d, and my camera seems to be struggling to keep up. I tried this addon but have no had any luck so far.
this is what my code looks like:
func _input(event):
	if event is InputEventMouseMotion:
		rotate_y(-deg_to_rad(event.relative.x * MOUSE_SENS))
		$Smoothing/Head.rotate_x(-deg_to_rad(event.relative.y * MOUSE_SENS))

		var head_rotation = $Smoothing/Head.rotation_degrees;
		head_rotation.x = clamp(head_rotation.x, -90, 90);
		$Smoothing/Head.rotation_degrees = head_rotation;

And here is an image of my hierarchy:

Trying to apply transforms manually on top of smoothed transforms can be quite tricky, there are a number of gotchas, especially with cameras.
This would probably need an MRP to investigate.

Hello, sorry for the delay. Here is an MRP for my project!
eva.tscn is what you can use for testing
Thanks!
MRP.zip

I'm trying the MRP, and reducing tick rate to 10tps (to show interpolation problems).

my camera seems to be struggling to keep up

Can you elaborate on the problem?
Is the problem you are describing involving pressing the thrusters, or just the rotation of the camera with the mouse?

When thrusting around, my game just stutters and i can move my camera around but my body doesnt keep moving. The reason i think the camera is the problem is because if i disable the camera movement code, the physics stuttering disappears. Here is a video of what i am describing:
https://streamable.com/5d3gvt

There are a number of problems I believe.
One initial thing is you shouldn't be changing rotation of physics objects directly (especially outside the physics tick), you should be applying forces, and angular velocity. Applying transform directly will screw up the physics simulation.
For first person cameras often the camera yaw and pitch is handled above the rigid body, rather than on the rigid body.

Here is fixed scene:

Note there is a yaw Node3D which is where I'm now applying yaw (and pitch). Mouse control should be below the smoothing node because it shouldn't be smoothed.
Here the smoothing node is off, because your major bugs are nothing to do with smoothing.
You have some choices to make:
For game such as this, you could use a sphere instead of capsule. This means you don't need to worry about applying mouse input to the collision shape, which is where things get hairy.
If you do want the mouse input to affect the collision shape rotation, you will probably get at best, laggy response, because physics only responds on physics ticks.
If you are using a sphere and handling the yaw and pitch below the rigid body in the tree, then you will need to back calculate the direction for thrust to be applied (or maybe get the direction of the child nodes directly, and use this).
Typically for 1st person games the approach I've mentioned is used, store yaw and pitch yourself, and set the rotation directly. For space games with no up, you do need to use relative rotations. I would still store these in your own gdscript personally, but you might be able to get away with relying on the node transform.
How you compromise the need for responsive mouse movement, and the need to align rigid body rotation is up to you to decide for your game design.
Anyway hope that gets you started, there are no problems here so far related to smoothing node as far as I can see.
(Also don't put collision shapes under smoothing node, they should be child of rigid body).

Thank you so much! I appreciate the quick responses tremendously. Thanks for the addon!!


Missing metrics

Hello, I installed prometheus-pve-exporter in docker and enabled to scrape the PVE metrics. (I'm using version 8.2.4.).
My issue is that I receive just your internal metrics and not the PVE relevant.
Any advice/hint?
Many thanks,
Vaclav
HELP python_gc_objects_collected_total Objects collected during gc
TYPE python_gc_objects_collected_total counter
python_gc_objects_collected_total{generation="0"} 1.103518e+06
python_gc_objects_collected_total{generation="1"} 288297.0
python_gc_objects_collected_total{generation="2"} 20598.0
HELP python_gc_objects_uncollectable_total Uncollectable objects found during GC
TYPE python_gc_objects_uncollectable_total counter
python_gc_objects_uncollectable_total{generation="0"} 0.0
python_gc_objects_uncollectable_total{generation="1"} 0.0
python_gc_objects_uncollectable_total{generation="2"} 0.0
HELP python_gc_collections_total Number of times this generation was collected
TYPE python_gc_collections_total counter
python_gc_collections_total{generation="0"} 1834.0
python_gc_collections_total{generation="1"} 166.0
python_gc_collections_total{generation="2"} 5.0
HELP python_info Python platform information
TYPE python_info gauge
python_info{implementation="CPython",major="3",minor="12",patchlevel="3",version="3.12.3"} 1.0
HELP process_virtual_memory_bytes Virtual memory size in bytes.
TYPE process_virtual_memory_bytes gauge
process_virtual_memory_bytes 4.4347392e+07
HELP process_resident_memory_bytes Resident memory size in bytes.
TYPE process_resident_memory_bytes gauge
process_resident_memory_bytes 3.2395264e+07
HELP process_start_time_seconds Start time of the process since unix epoch in seconds.
TYPE process_start_time_seconds gauge
process_start_time_seconds 1.72052305563e+09
HELP process_cpu_seconds_total Total user and system CPU time spent in seconds.
TYPE process_cpu_seconds_total counter
process_cpu_seconds_total 30.87
HELP process_open_fds Number of open file descriptors.
TYPE process_open_fds gauge
process_open_fds 13.0
HELP process_max_fds Maximum number of open file descriptors.
TYPE process_max_fds gauge
process_max_fds 1.048576e+06
HELP pve_collection_duration_seconds Duration of collections by the PVE exporter
TYPE pve_collection_duration_seconds summary
pve_collection_duration_seconds_count{module="default"} 0.0
pve_collection_duration_seconds_sum{module="default"} 0.0
HELP pve_collection_duration_seconds_created Duration of collections by the PVE exporter
TYPE pve_collection_duration_seconds_created gauge
pve_collection_duration_seconds_created{module="default"} 1.7205230564812522e+09
HELP pve_request_errors_total Errors in requests to PVE exporter
TYPE pve_request_errors_total counter
pve_request_errors_total{module="default"} 91.0
HELP pve_request_errors_created Errors in requests to PVE exporter
TYPE pve_request_errors_created gauge
pve_request_errors_created{module="default"} 1.7205230564812417e+09

It was an issue of misconfiguration. I didn't carefully follow the manual.

How did you solve it i couldn't find anything wrong when looking at my config and the manual?

Hi, my docker-compose was wrong and I didn't call the logs correctly. I had
multiple issues as I run prometheus on the separate device.
út 27. 8. 2024 v 18:26 odesílatel yanne057 @.***>
napsal:

How did you solve it i couldn't find anything wrong when looking at my
config and the manual?
—
Reply to this email directly, view it on GitHub
https://github.com/prometheus-pve/prometheus-pve-exporter/issues/265#issuecomment-2313018255,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMSRS35QKSUSZ2D3AKCR37DZTSSE7AVCNFSM6AAAAABKTBHILKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJTGAYTQMRVGU
.
You are receiving this because you modified the open/close state.Message
ID: @.***
com>


App logo

Create a logo that can be used universally throughout the app and as the Icon.
This can also coincide with issue #72 Create an Icon Pack
Logo Concept
The goal is to create a representative character for the smart mapping app. We want to convey a helpful personality that is playful and adventurous. You can work with the name Map-e, Mappy or propose something new, just keep in mind it should be as short as possible and be reasonably perceived as a character/personality. Think of things like siri, alexa, and the android robot etc for inspiration as to the strategy here. Map-e should be your reliable little pocket size guide to find new things and navigate the way to get you there, Even when signal is spotty.
Requirements – Constraints
• logo must be distinguishable down to 16px x 16px
• Should scale cleanly
• should work both monotone and full color
• should be usable for the complete icon pack in all sizes

I know there was some discussion on the team boards, but has a logo been selected?
@TheDevPath/ui-ux-design-team

It's still open game to anyone feeling artistic!

Yes please help! I’m bad at this :D

So I am going to be working on the Logo & Icon. I was thinking of doing something called NAVI. This gives us the short iconic style name like Siri & Alexa. It also references the sidekick/ companion of Link in the Legend of Zelda. Also Navi is short for Navigation, which is what we are doing for the user!

Looks great so far!

Hi guys, I am working on Issue #128  and for that I was wondering if we could get more sizes for the logo?
The Web Manifest Section in the Intro to PWA course materials recommend the following sizes: 48,96,128,144,192,256,384,512. We have 192,512,182,16,32,150.
Ideally we would also have 48,96,128,144,256,384


[JSONModel toString] returns null

I have a JSONModel child class , which seems like that,
    "attachment" : raw data bytes as NSData
}```

When I am trying to convert it as NSDictionary or to JSONString it is giving me error as

[JSONModel.m:1051] EXCEPTION: *** +[NSJSONSerialization dataWithJSONObject:options:error:]: value parameter is nil

Here are my .h and .m files

__QrosityChatImage.h__
```objective-c
#import "JSONModel.h"

@interface QrosityChatImage : JSONModel
@property (nonatomic, strong) NSData *attachment;
- (instancetype)initWithImageData:(NSData*)data;
@end

QrosityChatImage.m
#import "QrosityChatImage.h"

@implementation QrosityChatImage
- (instancetype)initWithImageData:(NSData*)data{
    if (self = [super init]) {
        _attachment = data;
    }
    return self;
}
@end

I wonder if a null byte is causing this issue?


Feature Request: undo delete feature

If I delete a feature, I have no way to undo it. It would be great if I could undo a delete.

+1, would be cool also for when you accidentally move a feature.


Inconsistent behaviour ios/jvm, write() doesn't update position on iOS

Using version 1.3.9
Having following test
@Test
fun test() {
    val text = "Hello world!"
    val input = PlatformBuffer.wrap(text.encodeToByteArray())
    val output = PlatformBuffer.allocate(text.length)
    output.write(input)
    assertThat(input.position()).isEqualTo(text.length) //<- ios fails
    assertThat(output.position()).isEqualTo(text.length)
    output.position(0)
    assertThat(output.readString(text.length)).isEqualTo(text)
}

JVM: OK
iOS: fails
com.willowtreeapps.opentest4k.AssertionFailedError: expected:<[12]> but was:<[0]>
Expected :[12]
Actual   :[0]

Interesting, that is definitely a bug. I should be able to get a fix up within a day or two. Thanks for filling this issue!

should be fixed in 1.3.10


2-way audio to a Home Assistant media_player

I have a scenario where i'd like to talk back (To my dog in this case), but i'd like to send the audio to a speaker located elsewhere in my house instead of the source camera, which is available in Home Assistant as a media_player
I'm using Unifi cameras and i have this setup like this
streams:
  camera.dog_camera:
    - rtspx://<IP_ADDRESS>:7441/LR42fugdNOfosafG

I've read the docs multiple times and struggle to understand if this is possible or not. E.g. something like this
streams:
  camera.dog_camera:
    - rtspx://<IP_ADDRESS>:7441/LR42fugdNOfosafG
    - hass:media_player.living_room

Would appreciate if someone had a solution or any tips that could point me in the right direction.

There is no working solution for now. But I have an idea how to do it. It's not easy task and very rare case. So it will be low priority.

Current go2rtc version doesn't support forwarding backchannel audio to app's output as REST API. If this feature would be added, the problem would be solved.

This is a great idea to implement something like an intercom system


when run/references/classification/train.py   AttributeError: module 'torchvision.models' has no attribute 'get_model'

🐛 Describe the bug
AttributeError: module 'torchvision.models' has no attribute 'get_model'
Versions
AttributeError: module 'torchvision.models' has no attribute 'get_model'

@cqray1990 you likely have an old-ish version of torchvision. Update to 0.15 (or IIRC get_models() was introduced in 0.13, but I didn't double check).
If you do have 0.15 installed and the issue persists, let us know.


question about post analysis

Hello,
First, thank you for developing this method and building easy to use code. I've ran the code to hla-type 10X scrna-seq samples and so far it's working great at genotyping my individuals.
However, I need to use the program with -- skipPostAnalysis tag, since otherwise the analysis runs for several days, at which point my jobs on the cluster are killed.
I'm working with cancer samples. In some cancers, a fraction of cancer cells in the same individual might express only one hla allele, while other cells might express both (and sometimes cells can express neither). In the long run, I'd like to be able to distinguish those cell groups. First step would be to hla-type each cell. As far as I understand, the part of the post analysis that is the 'single-cell representation' can give per-cell allelic abundance estimate. Is that correct?
If so, is there a way to run post analysis in parallel?
For example, (and I'm totally making this up because I don't know how it's ran), if post analysis is running T1K for each cell independently, then I can split _aligned_1.fa _aligned_2.fa by barcodes stored in _aligned_bc.fa and run multiple jobs at the same time, one per barcode with --stage 2 tag.
Would that be valid?
(and yes, if I run it like that, then I'd end up with bunch of output files which I'd need to deal with, but that's okay as long as the procedure is valid)
Thank you,
Ivan

Thank you for using T1K. The most time-consuming step is inferring the novel variants, which can take a long time in some cases. I will add some parameters to control the time spent on this step so you can use the post-analysis.

I have updated the constraint as an option in the github repo. Could you please pull the github repo and give the new postanalysis stage a try?

Thank you!
Running it now with default --post-varMaxGroup parameter. I'll let you know how long it took to finish.

Thank you for finding this issue. I have fixed this issue in the wrapper to the github. I guess the slowness may not comes from variant calling now, or this parameter did not resolve the bottleneck. I'll look into this a bit more.
For this 22-hours run, is it 22 hours after the analyzer step?

Thank you for fixing the issue!
Yes. Everything else is done within ~40 min. Here are the last couple lines of the log file of the current run that's still running:
[Tue Apr  9 12:26:04 2024] SYSTEM CALL: /mnt/beegfs/projects/human/tools/T1K/analyzer  --varMaxGroup 2 --barcode P0base.v4_aligned_bc.fa -o P0base.v4 -t 24 -f /mnt/beegfs/projects/human/tools/T1K/hlaidx/hlaidx_rna_seq.fa -a P0base.v4_allele.tsv -1 P0base.v4_aligned_1.fa -2 P0base.v4_aligned_2.fa
[Tue Apr  9 12:26:07 2024] Found 1975284 read fragments. Start read assignment.
[Tue Apr  9 12:26:10 2024] Finish read end assignments.
[Tue Apr  9 12:26:11 2024] Finish read fragment assignments. 1960892 read fragments can be assigned (average 1.63 alleles/read).
[Tue Apr  9 12:26:11 2024] Finish allele quantification in 11 EM iterations.

Previously (before the introduction of --post-varMaxGroup), for this particular dataset, I managed to finish the run on the same cluster node I'm running it on now. This is what the ending of that log file looked like:
[Fri Apr  5 20:37:30 2024] Found 1975284 read fragments. Start read assignment.
[Fri Apr  5 20:37:33 2024] Finish read end assignments.
[Fri Apr  5 20:37:34 2024] Finish read fragment assignments. 1960892 read fragments can be assigned (average 1.63 alleles/read).
[Fri Apr  5 20:37:34 2024] Finish allele quantification in 11 EM iterations.
[Sun Apr  7 10:14:30 2024] Post analysis finishes.
[Sun Apr  7 10:14:30 2024] Finish.

Once current run finishes, I can let you know how how much faster it was.

I just pushed an update to github, so you can use "--varMaxGroup 0" to completely skip the variant calling step. Is it possible to share the XX_aligned_1/2.fa, and the XXX_allele.tsv files? I'll check the reason of the slowness. Thank you.

Yes on .tsv file (would sending it via email be okay?), but I need to check and get back to you about .fa files.
Just to confirm, these files only contain only reads that overlap hla loci, correct?

Email is fantastic. My email is<EMAIL_ADDRESS>.

Just to confirm, these files only contain only reads that overlap hla loci, correct?
Right.

I just noticed that you didn't have the "-s 0.97" in the command. Did you run T1K with the option "--preset HLA"?

That was it! I didn't have that part in my call. After adding it, runs finished quickly.
Thank you very much for figuring it!

I'll close this issue for now. If you have found any other issues, please feel free to reopen this one or create another thread.


Dynamic dataset page needs new content/per year content

Dynamic content:

[ ] 2018-lar
[ ] 2018-ts
[ ] 2018-lar-specs
[ ] 2018-ts-spec


while tapping "US" from dialog picker selects Canada as  a selected country.

To Reproduce:
1> open country dialogue
2> Type "us" into search fields
3> Then Select the USA from list..
after this steps you will get "Canada" as a selected country, for the first time after initialising country picker. can you please guide me ?

it's not happening for me
can you check with the latest release?

Thank you @imtoori ..but the latest version is forcing me to upgrade the current dart SDK. which is not feasible for me at present.

I am having the same problem, following thread.

I'd be really happy to solve this issue, but I can't reproduce it
Can you give me more info?


VoicingUtteranceQueue improvements

@jessegreenberg and I saw two spots that could use minor optimization.
when the voicing utteranceQueue is disabled, we want to clear the queue. Otherwise all previous responses will just wait there and be the first to be announced when next enabled, even if not enabled again for 10 minutes. We decided this shouldn't be for the voicing feature in general, just for PhET's implementation (so in joist not scenery).
We can also now no-op for empty queues with the step timer. This minor improvement will help even sims that don't have voicing enabled (since voicingUtteranceQueue is a singleton that is always created.

All set here @jessegreenberg, thanks for the consult.


Kanban Setup

Dear Reviewer,
Please find our kanban setup for the final capstone below
Kanban board
In this Group there are four Members

Abel @abeltsew
Sasan @dev89s
ikoote @ikoote1
Tayyab @tayyabkm

Hi @abeltsew,
Wow, you did it 🎉

Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
To highlight

All cards have descriptions, have been assigned and have pointings.  The pointings are equal according to the requirement.✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


[top] Enable pwrmgr_smoketest in FPGA

Various top level fixes to enable pwrmgr_smoke_test in FPGA.
This should unblock certain FPGA development activities.
As part of this PR, part of the power okay signaling is bypassed, pending more investigation (#9507 )

LGTM. Do we have an issue to track the AST problem (so that we do not forget to change it back)? Maybe it also makes sense to add a TODO(#...) in the code...

yep see #9507
I honestly was going to add it back into the code..but CI just takes so long that I decided to just live with it for now.

Thanks @tjaychen! I tried all three cases that we were interested in with the bitstream from this PR: sw reset request and alert triggered resets work fine. keymgr_functest no longer hangs on wake up but fails (same as what I observed yesterday):
I00004 keymgr_functest.c:317] Starting test keymgr_rom_ext_test...
E00005 keymgr_functest.c:258] Error at line: 258

We will change the keymgr_functest to use sw reset request (assuming the test doesn't actually require a wake up) but wanted to share what I observed, could be relevant to #9507.

LGTM. Do we have an issue to track the AST problem (so that we do not forget to change it back)? Maybe it also makes sense to add a TODO(#...) in the code...

yep see #9507 I honestly was going to add it back into the code..but CI just takes so long that I decided to just live with it for now.

Yeah I know. Providing the link to the issue in this PR is fine.

yeah i feel like this might be something different that needs debugging.
When keymgr advance fails, can you check to see what the error status in keymgr is?
We probably need to modify the test to check that the seeds programmed in flash / otp are actually still correct after a low power exit, otherwise keymgr may refuse to advance if those fields are empty.


(MODULES-2980) Move vendored resources test

This commit moves the vendored resources test to acceptance to ensure that a problem is found sooner

The original PR that introduced this is at #58
I don't see any reason that test needed to be via puppet agent rather than puppet apply. +1 to not involving the master.


Issue reporter forces refresh - Safari - Codespaces Web

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

Accidentally hit refresh
🐛  this dialog cannot be escaped

And after allowing the refresh, the issue reporter is totally blank


Fix permission error when sorting pages having page type restrictions

This is a cherry pick of 128b319b99 but I am adding it via a pull request because I want the tox tests to run. I think the Jinja2 3.1.0 release on March 24th may mean we need to adjust the install / testing requirements to get some of the test combinations to run.

I cherrypicked the Jinja2 fixes from main and now most of the tests run - except when testing against Django's current main branch.


[macos] Fix nullptr issue during compilation.

Added a macos compiler flag to fix a nullptr issue during compilation, full credit to @guillem-sopra
This has already been fixed for the godot2 branch but for some reason was not yet added to the godot3 branch.

Hmm.  That see to indicate that your compiler is too old.  As we discussed, you should be using a newer version of your compiler for Godot 3.x than the one for Godot 2.x.... but I don't know shit about Mac's compilers!


i want visualize attribute!

@Deanplayerljx
i print attribute features use object_vocab.(provided h5 file index and object_vocab.txt)
it looks like label is same as new_tags label.

this is original image

this is attribute image( I received the data you provided)

this is new_tags image( I received the data you provided)
(1) new tag is used attribute feature??(not new_tags.h5, only attribute.h5)
(2) i want visualize 'attribute'! but i can't find index that is matching attribute vocab label .
where is attribute indices? help T^T
Thank you:)

Hi,
Sorry for the late reply. You can find the attribute information here.
For new tag boxes:
train set
val set
For original vcr boxes:
train set
val set
Please use following script to read these files.
For new tag boxes (replace tsv with actual file name):
FIELDNAMES = ['image_id', 'num_boxes', 'boxes','objects', 'attrs']

imageid2det = {}
csv.field_size_limit(sys.maxsize)
with open(tsv, "r") as tsv_in_file:
    reader = csv.DictReader(tsv_in_file, delimiter='\t', fieldnames = FIELDNAMES)
    for item in reader:
        image_id = split + '-' + item['image_id']

        obj_indices = np.frombuffer(base64.b64decode(item['objects']),dtype=np.int)
        obj_names_dict = {}
        obj_names_list = []
        for obj_idx in obj_indices:

            obj_name = classes[obj_idx+1]
            obj_names_list.append(obj_name)
            if obj_name not in obj_names_dict:
                obj_names_dict[obj_name] = 0
            obj_names_dict[obj_name] += 1

        attr_indices = np.frombuffer(base64.b64decode(item['attrs']),dtype=np.int)
        num_boxes = int(item['num_boxes'])
        boxes = np.frombuffer(base64.b64decode(item['boxes']),dtype=np.float32).reshape(num_boxes, -1)
        imageid2det[image_id] = {'obj_names_dict':obj_names_dict, 'obj_names_list':obj_names_list,'objects': obj_indices, 'attrs':attr_indices, 'boxes':boxes}

For original vcr boxes (replace tsv_gt with actual file name):
FIELDNAMES_GT = ['image_id', 'num_boxes', 'boxes','objects', 'attrs']

imageid2det_gt = {}
csv.field_size_limit(sys.maxsize)
with open(tsv_gt, "r") as tsv_in_file:
    reader = csv.DictReader(tsv_in_file, delimiter='\t', fieldnames = FIELDNAMES_GT)
    for item in reader:
        image_id = split + '-' + item['image_id']

        obj_indices = item['objects']
        obj_indices = obj_indices.strip('[]')
        obj_indices = obj_indices.split(' ')
        obj_indices = filter(None,obj_indices)
        obj_indices = [int(x) for x in obj_indices]

        obj_names_dict = {}
        obj_names_list = []
        for obj_idx in obj_indices:
            obj_name = classes[obj_idx+1]
            obj_names_list.append(obj_name)
            if obj_name not in obj_names_dict:
                obj_names_dict[obj_name] = 0
            obj_names_dict[obj_name] += 1

        attr_indices = item['attrs']
        attr_indices = attr_indices.strip('[]')
        attr_indices = attr_indices.split(' ')
        attr_indices = filter(None,attr_indices)
        attr_indices = [int(x) for x in attr_indices]

        num_boxes = int(item['num_boxes'])
        boxes = np.frombuffer(base64.b64decode(item['boxes']),dtype=np.float32).reshape(num_boxes, -1)
        imageid2det_gt[image_id] = {'obj_names_dict':obj_names_dict, 'obj_names_list':obj_names_list,'objects': obj_indices, 'attrs':attr_indices, 'boxes':boxes}


Fix asciidoc warnings by using proper links

Proof of all warnings gone:

Proof of warnings before this PR:

Note: I fixed only main branch because I noticed in other projects that maintainers want to do downmerge manually.

@TheJavaGuy Hello, thank you for pointing this out. I'll take care of the backport to support branch.

@all-contributors please add @TheJavaGuy for doc


Adding compact view option also for filmstrip view

As per issue #684

Thank you @semla -- the code looks great! I'll test it out to see how it looks in the GUI and will try to merge it in soon.
It might be a few weeks as I'm a bit busy with house stuff still.
Cheers 🙌

Glad to hear it is looking good!
Busy at the moment and it has been a while since I saw the code so I think I leave the fixes for you.
Cheers

Thank you for the quick response 🤝 I'll fix up the minor edge case -- I appreciate the addition 🙇
Maybe this weekend 🤞

I'm hoping to have a release at the end of February 😓 -- this year has been busier than I ever anticipated.

I have a guess why there was some need for css improvement 😅  -- in the code (template) you instruct the app to use a class called compact-margins but it isn't part of the PR 👌
I'll merge in and fix in main
Thank you again for the contribution 🙇
This will be a part of the 5th Anniversary Release today 🥳

🙇  thank you @semla for your contribution -- I made a small fix with this commit: https://github.com/whyboris/Video-Hub-App/commit/09c57b8fdf4a577d258376bf17dc64953cda5581
Looks beautiful ❤


Revert "[5.1] Add --prefer-stable and --prefer-lowest build for Travi…

Reverts laravel/framework#8920 because those builds are just going to fail forever until doctrine releases a new version of instantiator.

But isn't that the kind of problem that that PR was supposed to surface?

This PR should ONLY have been merged if it wasn't on allow failures and was passing.

i.e. it should have been merged WITH the relevant fixes, otherwise, it's pointless.

I didn't include a fixed because that would add other thing that taylorotwell might not agree to do straight away.
As an example? Would he be agree to increase the min Carbon deps, or change the plural stuff.

As a valid case, if 5.1 go stable the test would only depends on latest dev, and not check for stable. The plural stuff would still return wrong value.

The plural stuff would still return wrong value.

That's not our fault though. It's not up to us to test other people's libraries.

That's not our fault though. It's not up to us to test other people's libraries.

Why exactly why I don't put any fix in the original PR so that it doesn't include any changes that you or taylor would reject (and surely close).

This should have been rejected anyway because the tests are failing.

Arguments:

Should reject PR because it doesn't include relevant fixes.
Shouldn't change the test because it other people's libraries.
Should reject PR because tests are failing.
Go back to reason 1.

We should delete the criterium test, than only merge the perfer-stable part of the pull and have it not on allow failures.

@GrahamCampbell it seems logic we would want to test on the lowest version of the dependency we are requiring? If they fail, we need to up our dependency? No?

@taylorotwell Yes, but only if a PR is first merged to correct our version constraints so that test can actually pass.

only if a PR is first merged

Why first? We should have the tests in place regardless. If a test fails, we need to up the dependency.

I don't buy really buy that.

It doesn't really matter how we fix this, but can we just fix it please. :)


Small styling issue with horizontal overflow

I saw this today and thought I'd flag a minor UI / styling issue.

❤

Thanks for pointing out!
Fix is on the way: https://github.com/near/near-explorer/pull/1016


Error when running Invoke-PSCodeHealth

Expected Behavior
Invoke-PSCodeHealth should run tests.
Current Behavior
Running Invoke-PSCodeHealth produces the following error:
Invoke-Pester : A parameter cannot be found that matches parameter name 'Show'.
At C:\Program Files\WindowsPowerShell\Modules\PSCodeHealth\0.1.139\Private\Metrics\Get-FunctionTestCoverage.ps1:51 char:127
+ ... ath = $SourcePath; Function = $FunctionName } -PassThru -Show None -V ...
+                                                             ~~~~~
    + CategoryInfo          : InvalidArgument: (:) [Invoke-Pester], ParameterBindingException
    + FullyQualifiedErrorId : NamedParameterNotFound,Invoke-Pester

Possible Solution
Perhaps a different version of Pester was released. Here is what I am currently using on my machine.
ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Script     3.4.0      Pester                              {AfterAll, AfterEach, Assert-MockCalled, Assert-VerifiableMocks...}

Steps to Reproduce (for bugs)

Context

Your Environment
ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Script     0.1.139    PSCodeHealth                        {Get-PSCodeHealthComplianceRule, Invoke-PSCodeHealth, Test-PSCodeHealt...

Thanks @Tiberriver256 for the feedback.
Yes, your deduction was correct : the Show parameter of Invoke-Pester doesn't exist in version 3.4.0 of Pester.
It was introduced in Pester 4.0 and this is why I listed Pester version 4.0.x (or later) in the requirements.
That said, I think it would be fairly simple to handle this better in PSCodeHealth, and make it compatible with Pester version 3.4.x (and later).
This Show parameter is the only Pester feature that PSCodeHealth uses which is different between versions 3.x and 4.0.x.
And these older versions of Pester have an equivalent to -Show None : the Quiet switch parameter.
So I could check for the Pester module version, and if it is older than 4.x, use -Quiet instead of -Show None.
I will test to verify it is that simple. If it is, I will make the change.


[SUGGESTION] Saving TOP layout separately

MC 1.10.2
Forge 2254
TOP 1.4.5
One of the players of my modpack noticed that their configuration of TOP got reset when he updated the pack. It's because it's all saved in the same file as the general config that I have to provide with each update. Can you consider doing it like JEI where your search term and your hide and cheat settings are saved separately? Maybe it also makes sense to save per world which overrides standard settings which you could still define/provide in the normal config? Or maybe you have a better idea how to solve this?

A quick note on this one. Rather than storing it in configs which would be overridden best to store it locally in the player.dat of the world save. However I get the feeling that this would effect locking specific settings in the config on the server side


feat(v1): Running feature bar chart

heatmap requires too many other components (select + tooltip), so will do it another time.


updateNetworkStatus function can't be run on iOS simulator

At the moment our implementation of NetworkReachability uses hardware-specific details.
Also we can not use Bluetooth on simulator via CoreBluetooth/BluetoothManager framework.
So we need to handle this situation when application is running on simulator.

Fix is implemented in private branch.

@dersim-davaod Any reason not to check this in? It will help the node folks when they need to investigate a failing node test on iOS since they don't have iOS devices.

The main reason why I didn't do that is lack of time due to #881 and #1029.
I will do this within one day.


tuned-adm: command not found

Hello @alexandruanghel , I'm facing this issue when trying to setup a static inventory. Any clue about which step I missed ?
TASK [common : Activate the tuned profile] **********************************************************fatal: [comp-prod1]: FAILED! => {"changed": true, "cmd": "tuned-adm profile hortonworks", "delta": "0:00:00.003807", "end": "2019-07-20 23:10:35.137544", "msg": "non-zero return code", "rc": 127, "start": "2019-07-20 23:10:35.133737", "stderr": "/bin/sh: tuned-adm: command not found", "stderr_lines": ["/bin/sh: tuned-adm: command not found"], "stdout": "", "stdout_lines": []}

Hi @belgacea are you running any special RHEL7 images? Normally RHEL7/Centos7 imainges should come with tuned installed (rpm -qa|grep tuned).
But there might be lighter images without that package so if I need to add a check, it would help me if I know the image you're using.

Hi @alexandruanghel , I don't know which package is missing, but we are running under CentOS 7.6 Minimal (64-Bit) from this provider.

Yes, I imagined that's the case; a minimal image would most likely not contain the tuned package, which is part of the default official RHEL distribution.
I'll add a simple check so that ansible only runs that command when the package is installed.


core_AbstractPage_client is not defined in production

Hello, I have been using the package with electron 26.2.1 in the main process (node.js)
it works nicely in the dev mood, but in production it produce the attached error about core_AbstractPage_client being undefined
it's can be produced by just setting up a function with:
  const anthropic = new Anthropic({
    apiKey: "****"
  });

appreciate your help!

Hmm, we don't explicitly support electron. What version of Node is it running under the hood? Or does it use another engine?

@imyosh I've created an example repository where the anthropic ai sdk is used in an electron app https://github.com/linus-amg/electron-anthropic-sdk
I'm currently on MacOS so I did not try in an environment which uses windows (as you seem to do), but I hope it works for you too. Don't hesitate to ping me if you have any additional questions.
Parts of the code which might matter when you try to implement this yourself:

https://github.com/linus-amg/electron-anthropic-sdk/blob/main/src/main.ts#L29 adds an event handler to a node internal event called trigger-anthropic-sdk
which get's exposed to the rendering process via https://github.com/linus-amg/electron-anthropic-sdk/blob/main/src/preload.ts#L6
which in the end gets invoked via frontend code here: https://github.com/linus-amg/electron-anthropic-sdk/blob/main/index.html#L28

hope this helps, good luck!


Back-port "Increase height of importSuccessModal"

Ticket Link
Groupings-1403
Test Checklist

[x] Unit Tests Passed:
[x] Jasmine Tests Passed:
[x] General Visual Inspection:

I'm assuming that you will deal with the reported conflicts.


just to confirmed the behaviour

with this plugin, we can jump from within a vim in a tmux pane to another tmux pane without tmux prefix, but, it can not jump reversely without prefix key, is it right?
now in my tmux,  and  will work without prefix, no matter there is vim launched,  and  does not work without prefix for vim.

Hello @FinallyFinancialFreedom. To confirm, the intended functionality is for _non_prefixed key bindings to work in either vim or tmux. The plugin should know if you're in Vim or tmux, and even if you're at the edge of a Vim window, and do the correct thing. You should not need to use the tmux prefix before any key sequence to move.
Please see the troubleshooting guide which hopefully can help you sort out your issue. I'm going to close this now as it is too generic for us to help with, but please feel free to re-open or comment if you end up trying the troubleshooting tips and are still having trouble.


Tilemap not displaying correctly with vertex shader

Godot version:
3.2 beta
OS/device including version:
mac / android
Issue description:
I have a tilemap

the solid red area is initially off the screen and not visible. By using a vertex shader to apply a barrel distortion of sorts, the solid red area will ordinarily become visible.

this is working ok for me in 3.1.1 on my mac and android devices with the default quadrant size of 16.
in 3.2beta 1 I am seeing this

its always like that on my android devices but sometimes its ok on the mac until you resize the screen and then the left solid portion will flicker on and off. I have tried changing the quadrant size but was unable to get it working for me as it does in 3.1.1
Steps to reproduce:
run the attached project, if it looks okay try resizing the screen as well.
Minimal reproduction project:
BARRELDIST.zip

Might be a side-effect of #30425, as previously the quadrants with tile coordinates between -quadrant_size and 0 were treated as being part of the quadrants on the other side of the axis (with tile coordinates between 0 and +quadrant_size).
Maybe moving the tiles so that they start at tile coordinates (0, 0) instead of (-x, -x), and then moving the tilemap to compensate would help.
Note that culling of out-of-screen quadrants (and other CanvasItems) happens before the vertex shader gets to run, so the fact that worked in the previous version is just a lucky concidence of bugs. It might be better to down-scale the tilemap and use the vertex shader to upscale it, so that it would not show parts that are originally off-screen.

Thanks for the suggestions, moving all the tiles to positive coords and shifting the transform of the tilemap to compensate works, so long as the off screen tiles are not culled in a future bugfix.

Testing on linux mint 18.3, with GeForce GTX 750 Ti, nvidia 384.130
Can reproduce in 3.1.2.stable thru 3.2.3.rc3
Worked fine in 3.0.stable thru 3.1.1.stable, as reported
Initially thought this was resolved as starting from 3.2.1 it looked correct by default, but sure enough if you resize the window the left part flickers on and off.


Issue#283 - User authentication (Sign up) successful

Description
A clear and concise description of what the PR does.

This PR does the following:

Fixes user authentication error during Sign up

Related Issues
Link any related issues using the format Fixes #issue_number.
This helps to automatically close related issues when the PR is merged.

Placeholder: "Fixes #283 "

Changes
List the detailed changes made in this PR.

Fixed a bug in Sign Up page due to multiple sending of tokens after signing up

Testing Instructions
Detailed instructions on how to test the changes. Include any setup needed and specific test cases.

Pull this branch.
Run npm install to install dependencies.
Run npm test to execute the test suite.
Verify that valid MONGO_URL exists

Screenshots (if applicable)

Additional Context
Any additional context or information that reviewers should be aware of.

After signing up, login in using the same email and password.

Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I am working on this issue under GSSOC

Description
A clear and concise description of what the PR does.

This PR does the following:

Fixes user authentication error during Sign up

Related Issues
Link any related issues using the format Fixes #issue_number. This helps to automatically close related issues when the PR is merged.

Placeholder: "Fixes Working on the user authentication in the sign up page #283 "

Changes
List the detailed changes made in this PR.

Fixed a bug in Sign Up page due to multiple sending of tokens after signing up

Testing Instructions
Detailed instructions on how to test the changes. Include any setup needed and specific test cases.

Pull this branch.
Run npm install to install dependencies.
Run npm test to execute the test suite.
Verify that valid MONGO_URL exists

Screenshots (if applicable)

Additional Context
Any additional context or information that reviewers should be aware of.

After signing up, login in using the same email and password.

Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.

[x]  I have performed a self-review of my code
[x]  I have commented my code, particularly in hard-to-understand areas
[x]  My changes generate no new warnings
[x]  I am working on this issue under GSSOC

Kindly resolve these conflicts @Lanaanvar

how do i resolve this issue? Not able to click the resolve conflict button

Is this okay?

Is this okay?

There is conflict in package lock file. You can remove your own package-lock and keep the updated one from the master branch.

I'll explain the issue. When a customer signs up in the registration page, it shows : "An error occurred. Please try again later.". So I checked the backend code and found the problem that was causing this : The original code had multiple calls to sendToken(customer, 201, res). It was also sending multiple responses. So i did the following :

Removed the duplicate call to sendToken(customer, 201, res).
Only a single response is sent with all the required information, including the refreshToken, customer, and optional token.
Changed the status code from 404 to 400 for invalid requests, as it’s more appropriate for bad requests.
Explicitly included only necessary fields from the customer object in the response.

I hope this is understandable.

I'll explain the issue. When a customer signs up in the registration page, it shows : "An error occurred. Please try again later.". So I checked the backend code and found the problem that was causing this : The original code had multiple calls to sendToken(customer, 201, res). It was also sending multiple responses. So i did the following :
1. Removed the duplicate call to sendToken(customer, 201, res).

2. Only a single response is sent with all the required information, including the refreshToken, customer, and optional token.

3. Changed the status code from 404 to 400 for invalid requests, as it’s more appropriate for bad requests.

4. Explicitly included only necessary fields from the customer object in the response.

I hope this is understandable.

Yes, this is comprehensive and understandable. Great work !! 👍
@Trisha-tech You may verify and merge


Plot script

Hi team,
I've been changing the python script as per our action item but I started to feel that id I waited until it was 100% done this PR would be way too hard to understand.
The commits are all very small and descriptive but the gist of the changes are:

Python2/3 compatibility
Start of a HeadroomPlot documentation
First unit tests for existing functionality
Focus on improving readability on readThroughputData

I'm happy to spend time with a pair to explain thought and intention and make sure this is something you're all ok with.

@vsanta Could you please rebase your branch as we have removed some files?
Thanks,
Danny & Iryna

Merged.


PowerMock Agent and TestNG 6.9.+

I'm working on following issue https://github.com/jayway/powermock/issues/640
Everything fine in case if PowerMock and TestNG are sued without attaching javaagent, but in case if a user wants use PowerMock Java Agent  instead PowerMockObjectFactory  then the following exception is thrown

objc[56954]: Class JavaLaunchHelper is implemented in both /Library/Java/JavaVirtualMachines/jdk1.8.0_74.jdk/Contents/Home/bin/java and /Library/Java/JavaVirtualMachines/jdk1.8.0_74.jdk/Contents/Home/jre/lib/libinstrument.dylib. One of the two will be used. Which one is undefined.
java.lang.VerifyError: Expecting a stackmap frame at branch target 11
Exception Details:
Location:
org/testng/internal/ObjectFactoryImpl.newInstance(Ljava/lang/reflect/Constructor;[Ljava/lang/Object;)Ljava/lang/Object; @0: aload_1
Reason:
Expected stackmap frame at this location.
Bytecode:
0x0000000: 2b04 b600 1f2b 2cb6 0022 b04e 2bb6 0026
0x0000010: b800 2cb0 4e2b b600 26b8 002c b04e bb00
0x0000020: 2e59 bb00 3059 b700 3112 33b6 0037 2bc6
0x0000030: 000d 2bb6 0026 b600 3da7 0005 123f b600
0x0000040: 37b6 0042 2db7 0045 bf
Exception Handler Table:
bci [0, 10] => handler: 11
bci [0, 10] => handler: 20
bci [0, 10] => handler: 29
at org.powermock.modules.testng.PowerMockObjectFactory.(PowerMockObjectFactory.java:37)
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
at java.lang.Class.newInstance(Class.java:442)
at org.testng.xml.TestNGContentHandler.xmlSuite(TestNGContentHandler.java:205)
at org.testng.xml.TestNGContentHandler.startElement(TestNGContentHandler.java:519)
at com.sun.org.apache.xerces.internal.parsers.AbstractSAXParser.startElement(AbstractSAXParser.java:509)
at com.sun.org.apache.xerces.internal.impl.dtd.XMLDTDValidator.startElement(XMLDTDValidator.java:745)
at com.sun.org.apache.xerces.internal.impl.XMLNSDocumentScannerImpl.scanStartElement(XMLNSDocumentScannerImpl.java:380)
at com.sun.org.apache.xerces.internal.impl.XMLNSDocumentScannerImpl$NSContentDriver.scanRootElementHook(XMLNSDocumentScannerImpl.java:614)
at com.sun.org.apache.xerces.internal.impl.XMLDocumentFragmentScannerImpl$FragmentContentDriver.next(XMLDocumentFragmentScannerImpl.java:3135)
at com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl$PrologDriver.next(XMLDocumentScannerImpl.java:880)
at com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl.next(XMLDocumentScannerImpl.java:606)
at com.sun.org.apache.xerces.internal.impl.XMLNSDocumentScannerImpl.next(XMLNSDocumentScannerImpl.java:118)
at com.sun.org.apache.xerces.internal.impl.XMLDocumentFragmentScannerImpl.scanDocument(XMLDocumentFragmentScannerImpl.java:510)
at com.sun.org.apache.xerces.internal.parsers.XML11Configuration.parse(XML11Configuration.java:848)
at com.sun.org.apache.xerces.internal.parsers.XML11Configuration.parse(XML11Configuration.java:777)
at com.sun.org.apache.xerces.internal.parsers.XMLParser.parse(XMLParser.java:141)
at com.sun.org.apache.xerces.internal.parsers.AbstractSAXParser.parse(AbstractSAXParser.java:1213)
at com.sun.org.apache.xerces.internal.jaxp.SAXParserImpl$JAXPSAXParser.parse(SAXParserImpl.java:643)
at com.sun.org.apache.xerces.internal.jaxp.SAXParserImpl.parse(SAXParserImpl.java:327)
at javax.xml.parsers.SAXParser.parse(SAXParser.java:195)
at org.testng.xml.XMLParser.parse(XMLParser.java:39)
at org.testng.xml.SuiteXmlParser.parse(SuiteXmlParser.java:16)
at org.testng.xml.SuiteXmlParser.parse(SuiteXmlParser.java:9)
at org.testng.xml.Parser.parse(Parser.java:172)
at org.testng.TestNG.initializeSuitesAndJarFile(TestNG.java:305)
at org.testng.IDEARemoteTestNG.run(IDEARemoteTestNG.java:44)
at org.testng.RemoteTestNGStarter.main(RemoteTestNGStarter.java:122)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at com.intellij.rt.execution.application.AppMain.main(AppMain.java:144)

And it's issue only with TestNG 6.9+. I cannot figure out what changed in TestNG since 6.8.22. I see it Git history that the ObjectFactoryImpl has not been changed  since 6.8.22 release.
By the way, PowerMock javaagent in this case doesn't  modified the ObjectFactoryImpl class.
I create the sample which can help to reproduce the issue
https://github.com/thekingnothing/testng-sample
Could you help me?

Strange behavior and I'm not familiar with agent or bytecode.
But I found some links:
http://stackoverflow.com/questions/7936006/why-am-i-getting-the-following-error-when-running-google-app-from-eclipse
http://stackoverflow.com/questions/15122890/java-lang-verifyerror-expecting-a-stackmap-frame-at-branch-target-jdk-1-7
https://github.com/orfjackal/retrolambda/issues/25
As I understand them, it could work out-of-the-box with java7 specific options.
The next question will be: where exactly and why is stackmap missing? Is there some generated code somewhere?

I've also found that -noverify option could help (I haven't tested it yet), but I wonder what was changed that could break using Java Agent. As I mentioned, there is no class chasing in this case the PowerMockClassTransformer returns null as it's recommended in [documentation](http://docs.oracle.com/javase/8/docs/api/java/lang/instrument/ClassFileTransformer.html#transform-java.lang.ClassLoader-java.lang.String-java.lang.Class-java.security.ProtectionDomain-byte:A- and it means that class isn't transformed.
Has been changed version of JDK which is used to build TestNG? Or any other environment changing since 6.8.22 ?

Has been changed version of JDK which is used to build TestNG? Or any other environment changing since 6.8.22 ?
I don't know what JDK @cbeust used for the releases, but we can imagine he sometimes upgraded its environment.

TestNG 6.9.4 dropped Java6 support: https://github.com/cbeust/testng/blob/master/CHANGES.txt#L83
Don't know if it change something or not.
And I still don't understand why it is working well without agent.

Finally, I've figured out what happen. It's there complex issue and several little mistakes and issue evolve into such stranger defect.

Since,TestNG 6.9.4 the 1.7 is target byte code version.
In JDK 1.7 Stack Map Frames verifying became mandatory.
I've found deeper in javagnet that even if class does not modified, it is gone through ASM ne way or another to remove final modifier. (And I fixed issue here)
ASM had a defect regarding incorrect byte code generation for JDK 7 and higher
javaagent cannot use dependencies, so ASM was repacked into PowerMock JavaAgent. (I've been trying repacked several ASM version, but I've failed, maybe some other classes which use ASM also should be changed, I will do it later)

Result: incorrect byte generation -> java.lang.VerifyError: Expecting a stackmap frame at branch target 11
Why it's working well without agent? Because, without agent we can what we want with class and we use Javassist to remove final modifier and it correctly use latest ASM version without this defect. I guess, or Javassist somehow handle this case.
Thank you! I'm closing the issue.

:+1: Thank for the explaination!
And happy to know it will be fixed :)

Indeed, thanks for the detective work, @thekingnothing, very impressive!


Using values of output displayed

Hi ! When I run ktlint on a file, I see the filename , line, col & the detail on the cmd. Now I wish to use the values of these somewhere else after running this. How can I access these values for each LintError! Any info would be of great help. Thanks a lot.
PS: Could not find any channel, so asking here. Thanks once again !

Try something like ktlint --reporter=plain --reporter=json,output=ktlint.json (this will generate ktlint.json file containing lint errors) or https://github.com/shyiko/ktlint#usage.


Would it be possible for wren to replace a bash language and use as shell in Linux distributions?

Also,
Would it make sense to make a wren shell and suggest it as a replacement for GNU Bash?

I would say no, the usage and aims are completely different.


Add gzip compression via .htaccess

https://varvy.com/pagespeed/enable-compression.html

.htaccess compression is only applicable for us neither the NGINX compression because we dont use linux or apache servers. When the public site is having project file and is included in the current solution, i will enable it thru azure web service
http://www.brandonmartinez.com/2015/08/20/enable-gzip-compression-for-azure-web-apps/


Question/Bug: BUTTON_DATA_INVALID if callback_data is stringified json

Question
Is it possible to send callback_data as stringified json? Do I have to format it in any way? It would be easier to parse afterward on the hook.
const callbackData = {
    action: 'helpRequestInterested',
    userId: user._id,
    helpRequestId: helpRequest._id
};
const stringifiedCBdata = JSON.stringify(callbackData);
const keyboard = {
    reply_markup: {
        inline_keyboard:[[
            { text: "This works", callback_data: `${callbackData.action} ${callbackData.userId}_${callbackData.helpRequestId}` },
            // { text: "This throws the err", callback_data: stringifiedCBdata },
        ]],
        hide_keyboard: false,
        resize_keyboard: true
    }
};

I may found two related issues in other repositories:
Allowed charcters => maybe {} is not allowed?
Size limitation but the stringified json shouldn't be much longer.

why not just go JSON.stringify(your object) and then JSON.parse(callback_data) later in your code? with callback_data obviously being extracted from the appropriate place

@david1602 I don't quite understand the message is stringified in the second (commented) inline keyboard but it throws an err.

https://core.telegram.org/bots/api#inlinekeyboardbutton
callback_data | String | Optional. Data to be sent in a callback query to the bot when button is pressed, 1-64 bytes

https://core.telegram.org/bots/api#inlinekeyboardbutton
callback_data | String | Optional. Data to be sent in a callback query to the bot when button is pressed, 1-64 bytes

Faced this issue while using non ASCII characters, so:
tgbotapi.NewInlineKeyboardRow(tgbotapi.NewInlineKeyboardButtonData(h.Title, "Бот-алертинга утилизирует много ко"))

Is ok, but:
tgbotapi.NewInlineKeyboardRow(tgbotapi.NewInlineKeyboardButtonData(h.Title, "Бот-алертинга утилизирует много кон"))

provokes error:
2023/01/30 16:33:36 Endpoint: sendMessage, response: {"ok":false,"error_code":400,"description":"Bad Request: BUTTON_DATA_INVALID"}
ERRO[0005] Bad Request: BUTTON_DATA_INVALID

But actually, it is problem only on my side. Non ACCII character in Golang not equals 1 byte. Maybe it'll help somebody.


Update Props.md

If you start the tutorial from the "getting-started" section you'll get an error because the AwesomeProject it's not registered:
"Application AwsomeProject has not been registered. This is either due to a require() error during initialization or failure to call AppRegistry.registerComponent.".

I am not sure these are meant to be examples that build upon the previous page. They do not register as "AwesomeProject" because they're standalone examples.

I agree with @hramos, these docs are independent and this does not need to be changed


Adding content for Gateway API Concepts Session

I hate to add more work, but would you mind including a small README with that has a description and links back to the YouTube vid? e.g. https://github.com/google/learn-oss-with-google/blob/main/kubernetes/hpa-introduction/README.md

As the vids have launched, I'm going to go ahead and merge this. I'll follow up with a README in another PR 👍


pose_postprocess for multiple classes

I trained the pose model with 2 classes. Each class has 5 keypoints. I successfully converted the model, but I'm unable to extract the multiple classes from the output. How I update the pose_postprocess for the multiple classes? (python)

I trained the pose model with 2 classes. Each class has 5 keypoints. I successfully converted the model, but I'm unable to extract the multiple classes from the output. How I update the pose_postprocess for the multiple classes? (python)

Not support yet.
Because the coco pose just one class.
Could you please give me a pt model for testing?

I resolved this issue using ultralytics pose post-processing. I can't provide you the current model. However, I can prepare a sample model, if it is still required for testing.

I resolved this issue using ultralytics pose post-processing. I can't provide you the current model. However, I can prepare a sample model, if it is still required for testing.

Sure I will really appreciate.


Lottie animation glitch on android

Description
https://github.com/lottie-react-native/lottie-react-native/assets/10797596/cc55557e-4ff8-41b6-a44b-754586caa92e
As shown in the video above, the animation shows a glitch in all the app animations in android only after react native new architecture upgrade enabled with bridgeless enabled.
Steps to reproduce
use LottieView component as documented.
Snack or a link to a repository
https://snack.expo.io/
Lottie React Native version
6.7.2
React Native version
0.74.3
Platforms
Android
Workflow
React Native
Architecture
Fabric (New Architecture)
Build type
Debug app & dev bundle
Device
Android emulator
Acknowledgements
Yes

@TheRogue76 What information is needed ?

A reproducible (will not be able to do anything without this, and https://snack.expo.io/ is most certainly not a repro)
More info. You mention the animation shows a glitch in all the app animations in android only after react native new architecture upgrade enabled with bridgeless enabled.. Do these animations work fine if you turn off the new arch, or did you experience the issue when you went from V5 to V6? If it is from V5 to V6, did you try the render mode prop, as mentioned in the migration guide?


Fix reboot Xenial

Without this fix, we can't reboot because the networking service which comes from the package ifupdown, closes the network connection during the reboot.

LGTM


Use more than one CPU core in IPA

Overview
Currently IPA is not capable of using more than one core at a time. Futures that we create while running a query are not spawned as executor's task, therefore they are all polled inside one thread. Multi-threaded Tokio runtime that we use just bounce the parent future and all its children between threads/cores but it only polls one future at a time.
All of the above makes IPA quite slow at the moment. One core does not generate enough work for the network layer and our throughput is quite miserable. We need to find a way how to parallelize work and leverage all (ok more than one) cores available on the host.
TODO: add more context and possible solutions

I want to try async_scoped. Yes, it is generally unsafe to have task scopes, but in our case it may be acceptable, because we don't use mem::forget in prod code.

I made the initial prototype that uses async_scoped and it seems to be working fine for a toy protocol and @benjaminsavage's attribution and capping. However there is one major inconvenience - internally scope uses FuturesUnordered which makes the order it resolves the spawned futures unpredictable.
   for i in 0..10 {
        spawner.spawn(async move { let record_id = RecordId::from(i); /* do something */ })
   }

   let results = spawner.collect().await; // results may appear in any order

It is definitely happening in our tests when I plug in async_scoped
// H1
collected results = [Ok((RecordId(1), (20_mod31, 22_mod31))), Ok((RecordId(0), (1_mod31, 20_mod31)))]
// H2
collected results = [Ok((RecordId(0), (20_mod31, 10_mod31))), Ok((RecordId(1), (22_mod31, 21_mod31)))]
// H3
collected results = [Ok((RecordId(0), (10_mod31, 1_mod31))), Ok((RecordId(1), (21_mod31, 20_mod31)))]

There are several ways we can tackle this issue:
Ignore and let caller's to re-order results
  for i in 0..10 {
        spawner.spawn(async move { let record_id = RecordId::from(i); /* do something */  (record_id, meaningful_result) })
   }

   let mut results: Vec<(RecordId, _)> = spawner.collect().await;

   // sort by record_id
   results.sort_by(|a, b| a.record_id.cmp(&b.record_id));

   // drop record_id
   results.map(|a| a.1).collect()

least favorite option, because of performance penalty and mental overhead. In IPA we mostly process things in order, so this code will have to appear everywhere where we process things in parallel
Create a Scope wrapper that tracks the order internally
  for i in 0..10 {
        ipa_spawner.spawn(async move { /* do something */ meaningful_result })
   }

   // collect returns results in the order they were spawned by keeping a buffer large enough
   // to accommodate all results
   let mut results: Vec<_> = ipa_spawner.collect().await;

   // sort by record_id
   results.sort_by(|a, b| a.record_id.cmp(&b.record_id));

   // drop record_id
   results.map(|a| a.1).collect()

This removes the mental and performance overhead, however it is likely not going to play well with seq_try_join_all API for two reasons:

it needs a more granular control than collect, because it removes individual futures as soon as they are resolved from the front of the queue and adds new ones to the back. It can't wait until the whole active set is resolved before spawning new futures
Ordering is important so seq_try_join_all will likely need to manage spare capacity to keep futures that are outside of active window which will complicate the implementation.

(for parallel version of seq_try_join_all does not to manage future polling manually, because all futures are polled by the executor. It can simply traverse the list from the head and remove futures already resolved)
Implement IPA-owned version of async-scoped
While it is possible to work with the authors of async_scoped and bring FuturesOrdered or other implementations that will allow scope stream to yield elements in order they were spawned, it could be possible that async_scoped API is not shaped for our needs
 impl<'a, T: Send + 'static, Sp: Spawner<T> + Blocker> Scope<'a, T, Sp> {
 pub fn spawn<F: Future<Output = T> + Send + 'a>(&mut self, f: F) { ... }

spawn method does not return a handle and Scope object does not provide any control over individual futures being spawned in parallel. We could implement our own Scope object, but the core functionality of async_scoped is inside the Scope object
    let handle = Sp::spawn(unsafe {
            std::mem::transmute::<_, Pin<Box<dyn Future<Output = T> + Send>>>(
                Box::pin(f) as Pin<Box<dyn Future<Output = T>>>
            )
        });

this is what we need to get our hands on. There is a risk that we get it wrong and introduce use-after-free or other meaty UB but this may be the only way to achieve what we want: processing many records in parallel and adding more work as soon as there is capacity available for it.
The shape of this API may look like this (if I can make it work)
  impl <'a, T: Send + 'static> ParallelScope<T> {
     pub <unsafe?> fn spawn<F: Future<Output = T> + Send + 'a>(&mut self, f: F) -> ScopeHandle<'a> { ... }
  }

ScopeHandle<'_> will work similarly to JoinHandle and will be stored inside SequentialFutures. It does not need to be polled in order to activate the future, but it can be checked/removed from the list once resolved. As async_scoped we would need to be careful with drops/panics and make sure if ParallelScope is dropped, it cancels all the futures before returning.
Thoughts @andyleiserson @martinthomson ?

#873 is the first attempt to address this


Add missing shebang

I noticed the missing shebang when I checked it with ShellCheck. I don't have an old installation to test, but the change shouldn't cause any problems.

test failure is unrelated to this PR, will look into this within the next days ...
@MichMich because of your comment in the changed file # This file is still here to keep PM2 working on older installations.
Is the file still needed? What is meant with "older installations"?

I've got no idea. Can't recall it. @sdetweil knows a lot about PM2. Maybe he can shine a light on this file.

pm2 won't launch npm start directly
so mm.sh is used, but directly launched with sh, not using the shebang approach to relaunch
pm2 needs the .sh extension to know how to launch. , maybe that's the missing shebang problem..

so if I understand @sdetweil  correctly the file is still needed and then this should me merged.

yes, correct on both points


Link for MmGetMdlPfnArray is wrong.

MmGetMdlPfnArray takes one to mm-bad-pointer, the correct link is https://docs.microsoft.com/en-us/windows-hardware/drivers/ddi/wdm/nf-wdm-mmgetmdlpfnarray. Looks like some other links might be wrong as well.

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

ID: 8beff88c-c22b-179c-de6c-d0e502279244
Version Independent ID: 58a0ee70-fe5b-398c-5fcf-a9bf795ba056
Content: _MDL (wdm.h) - Windows drivers
Content Source: wdk-ddi-src/content/wdm/ns-wdm-_mdl.md
Product: windows-hardware
Technology: wdk-api-reference
GitHub Login: @barrygolden
Microsoft Alias: bagold

Hi @paul-rosswurm-1
Thank you for pointing this out to us. I've updated the links to point to the correct and current pages. The updates should be published later today.
Thanks for helping us improve the docs!
~ Mark


Unable to edit OneClick Bookworm metadata

STR:

Start at My Bookworms
Select Edit for a successful Bookworm
Change one character in the Source Name
Click Update All
Return to edit page using Edit link or visit front end user GUI
Changes not reflected

I've changed metadata in several fields, and nothing seems to stick. If there are fields that can't be changed, that's fine, but the UX implies that they all can.

Tagging @sa501428 for the OneClick. Could you let us know what version of Bookworm is currently running on the OneClick server?
I don't know what "Source Name" means here--is that an option on the OneClick server?

It's still using an older release. It'll take time to update it.

How old is the release? One possible fix for this were some commits about a year ago, give or take a few months.


Scripting Addition: Connection Issue

After setting up the Scripting Addition using the wiki. Anytime I do yabai --restart-service I am met with this message:

Running sudo yabai --load-sa doesn't show a password prompt and elements of my config file that don't require the Scripting Addition are working.
When trying to run any command that uses scripting addition I get the response cannot [do X] due to an error with the scripting-addition.
csrutil status returns: System Integrity Protection status: unknown (Custom Configuration)
I have a 2020 Macbook Pro with 2 GHz Quad-Core Intel Core i5 (I know, pretty bad) if that's of any help.

It turns out I needed to sudo pkill Dock and retry loading the Scripting Addition. Seems to have fixed the issue.


Design Meeting Notes, 2/22/2023

Restricting --moduleResolution bundler to --module esnext

--moduleResolution bundler
Some confusion due to a footgun.

When we look at package.json's exports field, we always look at the import condition, never the require.

For example, import import "yargs"

--moduleResolution nodenext depends on the package.json type field for a .ts or .js file
--moduleResolution bundler just looks at the syntax, regardless of the file type. require uses the require condition, imports use the import condition.

If you're using this mode, what is your end goal? What is the "correct" output target?

Bundlers usually prefer ES modules format.
But if you use --module commonjs, the output for import "foo" will be require("foo") - and bundlers will switch their resolution strategy compared to what TypeScript actually does.

This is a footgun!

Proposal: limit this mode to --module esnext.

What about a stable ES module target apart from ESNext?

es2015 - basic module syntax
es2020 - import.meta
es2015 - top-level await

Aside: respecting __esModule markers not consistent across bundlers.

It might make sense to have a corresponding --module target for --moduleResolution bundler

But it would be bad to call it that - we have no interest in bundling.

Conclusion: limit --module to esnext for --moduleResolution bundler

Consulting valueOf() in Relational Comparisons
#52807

Idea: consult .valueOf() to see if JS relational comparisons are valid.

Found bugs

VS Code issue with sorting URIs by last-touched time - accidentally comparing the functions that would actually update the time instead of the times themselves.
Comparison on opaque types - fine at runtime, plausible bug long term. If these were branded intead of opaque, it would work better.

How do you deal with this?
function compare<T extends string | number>(x: T, y: T) {
  return x < y;
}

Cast both sides to string or to number? Not correct.
Cast both to any? Ehh

Weird to say we'd just support valueOf but not [Symbol.toPrimitive]

Do people actually define these?
Separately of each other?
Moment doesn't

Sure but other code does, and might?

Admittedly, the interplay between these is not 100% clear. Which one is used by the other?

Still, feels like it's an overall reasonable change.

We would like to investigate bringing it in for TypeScript 5.1.

es2015 - top-level await

Typo?  TLA is es2022.

Yup, nice catch - that was me copy/pasting twice and forgetting to fix the second one.


pandoc-*-arm64.deb breaks Debian auto download

https://github.com/jgm/pandoc/releases/tag/2.13
https://travis-ci.org/github/bebraw/pypandoc/jobs/768518864#L425
$ ~/bin/pandoc -v

/home/travis/.travis/functions: line 109: /home/travis/bin/pandoc: cannot execute binary file: Exec format error

Hi there
this issue has been fixed, and will be out in the next version, 1.6


make expressionMaxLength enabled by default

struts.ognl.expressionMaxLength enabled by default to block unknow 0day

Could you register a ticket in JIRA? This is quite important information for users
https://issues.apache.org/jira/browse/WW

Could you register a ticket in JIRA? This is quite important information for users https://issues.apache.org/jira/browse/WW

@lukaszlenart
sorry because some reason，this url is block , sorry about that
pls consider this pr blow again
https://github.com/apache/struts/pull/537/commits/2e0d1f4de7e19dd7d725f299a22ca6a377740ad6

i chose other way to protect my app


'invalid syntax' reported for Python

Hi there
macOS 10.12.1; Vim 8.0; Syntastic 3.8.0-10 (Vim 800, Darwin); Available checker: Python
Great plugin, thanks.
I'm learning Python and have been using the following way of calling the print() function:
print(
    """
    Hello world
    """
)

This is the form used in my textbook, and Python3 renders the script happily, but Syntastic reports the following error:

invalid syntax

Is this perhaps a Python 2/3 difference? (New to Python so I can't be sure).
Cheers
Ian

Currently enabled checker: python

Currently enabled checker: python

So, what checker reports it?

Sorry, lcd047, you've lost me. I thought that was the checker... How, precisely, do I get the info you're requesting?

That's what syntastic runs by default.  It may or may not be the checker that produced the errors.
To answer your question: :h syntastic-debug.  Or just look at the statusline of the error window.
If it really is the python checker, you should probably look first at line endings of your source (LF vs. CR vs. CRLF).  Syntastic doesn't "invent" error messages, it's always an external program that produces them.

Thanks, lcd. This isn't in my status line (I'm using Airline; might be why) but setting the debug variable did the trick. The checker is confirmed as 'python/python', as far as I can see from the debug message, and Vim (given the command set ff? ffs? returns 'fileformats=unix,dos')---when I force LF endings using setlocal ff=unix and save the file, the python checker still reports the above syntax as in error.

I'm using Airline; might be why

airline does show the name of the checker by default:

Vim (given the command set ff? ffs?) returns 'fileformats=unix,dos'.

set ff? ffs? will tell you how Vim detects line endings and how it writes the file.  It won't tell you anything about the current line endings.  Actually, Vim is not the right tool to find out whether your file contains binary junk, use od instead.

When I force LF endings using setlocal ff=unix and save the file, the python checker still reports the above syntax as in error.

Syntastic checks the file on disk, not the contents of the current buffer.  So changing ff will take effect only after you write the file.  It also won't affect BOMs.
Anyway, like I said, the problem has nothing to do with syntastic.  You'll probably get the same error when you try to run the file.

Morning,
Thanks for getting back to me. I think we’re going around in circles a bit.  Not sure if this email will reach you, but some comments below. Since you’ve closed the issue, you might want to move on with your life! No need to reply if you’d prefer not to.

On 28 Dec 2016, at 08:25, LCD 047<EMAIL_ADDRESS>wrote:
I'm using Airline; might be why
airline does show the name of the checker by default:
https://cloud.githubusercontent.com/assets/1281736/21517345/07857cbe-cce7-11e6-9c54-585b43336887.png

Quite possibly it does, but I’m not using the default settings.

Vim (given the command set ff? ffs?) returns 'fileformats=unix,dos'.
set ff? ffs? will tell you how Vim detects line endings and how it writes the file. It won't tell you anything about the current line endings. Actually, Vim is not the right tool to find out whether your file contains binary junk, use od instead.

Thanks - I didn’t know about ‘od’. I don’t have any reason to think Vim is using non-standard line endings - I’ve never changed that.

When I force LF endings using setlocal ff=unix and save the file, the python checker still reports the above syntax as in error.
Syntastic checks the file on disk, not the contents of the current buffer. So changing ff will take effect only after you write the file. It also won't affect BOMs

Sure - but if Syntastic is checking the written form of the file, saving the file after ff-unix would therefore fix it because this is changing the written form. Am I missing something?

Anyway, like I said, the problem has nothing to do with syntastic. You'll probably get the same error when you try to run the file.

I don’t see any evidence that it’s anything other than Syntastic, and, as I wrote in earlier, my Python interpreter is not reporting a syntax error when running the script—there isn’t one because the syntax is valid (or at least appears to be).
Do you have any ideas what I could do next?
Cheers
Ian

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/vim-syntastic/syntastic/issues/1938#issuecomment-269443478, or mute the thread https://github.com/notifications/unsubscribe-auth/AR4EgxuslzJ1c6k9AofjLm8i4vSghCaEks5rMhz_gaJpZM4LWacE.

Hi Lcd
Further to my last email, I’ve realised that this is indeed a Python version issue (stupidly, I was looking at the wrong line in the file; it’s a Python3-specific version of the ‘print’ function that is causing the error).
This makes it much easier to solve. Thanks again for your help.
Cheers
Ian

On 28 Dec 2016, at 08:25, LCD 047<EMAIL_ADDRESS>wrote:
I'm using Airline; might be why
airline does show the name of the checker by default:
https://cloud.githubusercontent.com/assets/1281736/21517345/07857cbe-cce7-11e6-9c54-585b43336887.png
Vim (given the command set ff? ffs?) returns 'fileformats=unix,dos'.
set ff? ffs? will tell you how Vim detects line endings and how it writes the file. It won't tell you anything about the current line endings. Actually, Vim is not the right tool to find out whether your file contains binary junk, use od instead.
When I force LF endings using setlocal ff=unix and save the file, the python checker still reports the above syntax as in error.
Syntastic checks the file on disk, not the contents of the current buffer. So changing ff will take effect only after you write the file. It also won't affect BOMs.
Anyway, like I said, the problem has nothing to do with syntastic. You'll probably get the same error when you try to run the file.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/vim-syntastic/syntastic/issues/1938#issuecomment-269443478, or mute the thread https://github.com/notifications/unsubscribe-auth/AR4EgxuslzJ1c6k9AofjLm8i4vSghCaEks5rMhz_gaJpZM4LWacE.

Am I missing something?

  There are still plenty of weird corner cases left for you to explore: BOM, the environment in Vim being different from the environment in your terminal, autocmds and / or other plugins interferring, the error list being actually produced by some other plugin, and so on.  Life with Vim is usually infuriating, but never boring. shrug

I don’t see any evidence that it’s anything other than Syntastic

As I said: syntastic doesn't pull error messages out of thin air.  Syntastic doesn't now, nor care, about the contents of your files.  What syntastic does is run some linters against your files, and shows you the results in a window.  As such, the answer to "why do I get message X?" is never ever syntastic.  Some checker produces the error messages, not syntastic itself.  Syntastic could be (and often is) the answer to the question "why I don't get message X?" (because that might be a parsing bug).

Do you have any ideas what I could do next?

Ok.  Please set g:syntastic_debug to 3, run the checker, run :mes and post the output.


Documentation improvement

Issue 01
Before running the "Emulator Setup" the distutils package is required.
Distutils package is available only for python3 and  AOSP requires python 2.7. Still the package is required. Otherwise getting the following error,
no module named distutils.dir_util

Please add the following line in documentaiton,
sudo apt-get install python3-distutils -y

Issue 02
Repo tool is required. Please add the following link in documentation,
https://source.android.com/setup/develop#installing-repo
Additionally,
sudo apt-get install curl -y

Issue 03
sudo apt-get install lzip
Issue 04
sudo apt-get install m4

Issue 05
sudo apt install python-pip -y
pip install zxcvbn
pip install numpy

Issue 06
This repo should be cloned with --recursive flag, otherwise dj-on-github/sp800_22_tests files will not get downloaded.
sudo apt-get install git -y
git clone --recursive https://github.com/lucapiccolboni/crylogger.git

Issue 07
sudo apt-get install cpu-checker
kvm-ok
sudo chown $USER /dev/kvm

Thanks! It is always difficult to keep track of dependencies. This is great. Thanks for your contributions.

I have added a couple of more points. I will keep updating this issue until I am done with my current attempt. Please keep this issue open till then. Thanks!

Here are some other issues. I run crylogger in Ubuntu 20.04.
Adb dependency is needed
sudo apt-get install adb

otherwise run.py line 30
return subprocess.check_output(split, stderr=subprocess.STDOUT).decode("utf-8")
would throw an error that "File or directory not found"
aes example needs to be signed
The orginal compiled apk can not be installed for the below reason
---------------adb: failed to install /home/zhang/documents/code/crylogger/scripts/data/apks/com.example.aes_0.apk: Failure [INSTALL_PARSE_FAILED_NO_CERTIFICATES: Package /data/app/vmdl1148869426.tmp/base.apk has no certificates at entry AndroidManifest.xml]

use android studio and sign the bundled apk will help

Thanks for reporting these issues!
As far as I remember, I didn't have issues in installing the apk of the aes example, but I'm glad you were able to find a solution.
For future reference, here are some additional guidelines on how to create a building environment (it is linked in the README). Most dependencies are listed there. There are additional dependencies for the Python scripts check.py and run.py.
Thanks.

Hello @lucapiccolboni I am done with my current attempt. Everything looks good as expected. I couldn't run a good number apps against the tool, but that is not related to documentation improvement. You can close the issue and update the README.md file. If you like me to send a PR, I would be happy to do that as well. Thanks!


Help in understanding build/quine.js

I am trying to understand this particular file  build/quine.js
Could anyone explain the intent of this ? Is this the helper code to make it run in web workers ?

Yes, it's related to Tangram needing to access the source code of itself in order to instantiate its web workers. It's a somewhat unusual requirement. quine.js hooks into the Browserify build process to add some code to enable this. We may also look at other build strategies in the future.
There is some background here:
https://github.com/tangrams/tangram/issues/252
And here:
https://github.com/tangrams/tangram/pull/499


Empty rules shown on configs page (grid layout)

Rules are shown as empty blocks when the layout grid is enabled if there are any unsupported rules (e.g. added by a plugin)

If you add eslint-plugin-prettier which includes eslint-config-prettier rules related to vue, flowtype & etc.
Func getRuleFromName returns undefined as the payload has no such rules
https://github.com/eslint/config-inspector/blob/27654d5f0f24b491b346fb921de25da9e2c3e956/app/components/RuleList.vue#L15-L19
https://github.com/eslint/config-inspector/blob/27654d5f0f24b491b346fb921de25da9e2c3e956/app/composables/payload.ts#L89-L92
Maybe it's worth checking if the rule exists in the payload?

We temporarily closed this due to the lack of enough information. We could not identify whether it was a bug or a userland misconfiguration with the given info.
Please provide a minimal reproduction to reopen the issue.
Thanks.

Why reproduction is required


Recursive file adding does not work

Python version: 2.7.12
wx.Python version: 3.0.2
Debreate version: 0.7.12
Platform: Ubuntu 14/16 LTS
In debreate 0.7.10 you could add all the files of a folder recursively to your package on the files page and the folder structure was kept. Now it adds all files to the same folder and ignores the structure.
Step to reproduce

Create new project
Go to "files" page
Select a folder with subfolders from your computer
Click the "add" button

Expected behaviour
All files are added with the original folder structure as in the selected folder
Actual behaviour
All files are added to the "Target" folder and the folder structure is ignored

Thank you for reporting this. It has been fixed in commit 9eb138a.
I will leave this report open for the time being in case of errors in the new code.

0.7.13 released.

The folder structure is still ignored with 0.7.13 release.

@brnW Could you please explain the steps you are taking to add directories, & the resulting output. I am unable to reproduce the issue. It works for me as expected.

CPU :   x86_64 (64-bit PAE)
System :   Linux Mint 18.1
Python :  2.7.12
wxPython : 3.0.2
Debreate : 0.7.13

Open Debreate
Go to "File" page
Browse to the main project folder on your computer
Then select the /usr folder (with subfolders)
Select "Custom"
Write /usr on the custom field
Click the "add" button

Expected behaviour (like 0.7.10 release) : all files are added with the original folder structure as in /usr (for example /usr/bin/mycommand --->  mycommand in /usr/bin)
Actual behaviour (0.7.13) : only "usr" folder is added to /usr target (that is /usr/usr) and the structure is ignored.
If you select all the tree, each file or folder is added in /usr target.
Thanks for this work !

Okay, if I understand correctly, this issue is different than the original topic.
When adding the directory "usr" to the file list, the directory structure is preserved. It's just that the top-level directory "usr" is not trimmed from the file path names.
So, let's say the added directory "usr" contains the file path "usr/bin/foo" and "usr/share/bar":

usr -- bin -- foo
   |
    -- share -- bar

Setting the target directory to "/usr" and adding the directory results in the file paths being "/usr/usr/bin/foo" and "/usr/usr/share/bar". Whereas in 0.7.10, if I remember correctly, they would have been "/usr/bin/foo" and "/usr/share/bar".
I understand how this could be frustrating. It sounds to me that the answer would be to have an option to preserve the parent directory from all file path names, like the following, so that both behaviors are supported:

Leaving "Preserve top-level directories" unchecked would have the behavior of 0.7.10.

Option to preserve top-level directories added. See issue #10