Configurable game/category/version/type filtering

For Modrinth and CurseForge, it'd be useful to have more controls for how the install and update commands filter files. This could consist of:

Configuration in pack.toml to specify the filters applied; defaults to allowing mods for the game version and modloader specified

Should it default to allowing mods for any minor game version?

Configuration per-file to specify filters applied - this would only accept filtering options that work per-file not per-addon (so you can't specify that a mod should only be updated if the mod is a resource pack)
A command somewhat like packwiz init to set up these filters

Can set up filters per-file or for the whole pack

The ability to override these options when invoking install/update commands? Could be useful for one-time addition of resource packs, or searching for mods in a category
Possible filter options:

Modloader (Modrinth/CurseForge)
Game version (Modrinth/CurseForge)
Category (Modrinth/CurseForge)
File alpha/beta/release status (Modrinth/CurseForge)
Side (Modrinth)
File type (CurseForge)

For this purpose it would also be beneficial to specify paths for file types, so that resource packs go in the correct folder

Game (CurseForge)

Defaults to Minecraft (of course) but can be overridden to a different game

This would allow non-mod files (e.g. resource packs, data packs, shaders) to be managed and updated properly (#86), as well as allowing more flexibility over how updated files are chosen. These filtering options will likely replace acceptable-game-versions in pack format 1.0.

Would this also include ignoring patch game versions (1.18.x) when installing a mod?

Yeah, I'd probably implement some sort of version range system.

Yeah, I'd probably implement some sort of version range system.

Any ETA? If not it's fine. Just wanna know so I can decide whether or not to wait.

Probably quite a while; my time is limited right now and the packwiz-installer changes are highest priority right now.


ignore compile same files

i have large project . laravel mix in large project not have solution for ignore compile same plugins like bootstrap-scss , in each change bootstrap-scss recompiling and this not good.
how i can set interval for one file for example just compile one time after run npm run watch
thank you

i found simple way .
just create another mix like bellow
let mixFirst = require('laravel-mix');
let mixSecond  = require('laravel-mix');
//First mix
mixFirst.js(moduleFolder + '/Base/src/Asset/js/Base.js',		'public/app/js')
.sass(moduleFolder + '/Base/src/Asset/scss/Base.scss',	'public/app/css');
//second mix
mixSecond.js(moduleFolder + '/Base/src/Asset/js/Base.js',		'public/app/js')
.sass(moduleFolder + '/Base/src/Asset/scss/Base.scss',	'public/app/css');

@vahidalvandi
I didn't know about it.
I will try it tomorrow and if this worked, it should go to docs.
Thanks

yes it worked !


Adding IDE code completion support for custom event emitter functions?

I've been using this package for an event heavy project I'm working on and its been great to work with. Thanks :) - I am struggling to figure out a way to add my own emitter definitions so my IDE understands when I start typing the .emit() code it can help me by showing me what parameters it will accept. Based on what I have seen other people doing with other libraries and what I gathered from looking at your source code this is my best attempt:
"use strict";

import {EventEmitter, EventListener, EventNames} from "eventemitter3";

export interface CEvents {
    'before-request': any[]; // based on your comments in the source code on extending interface EventTypes
}

export declare interface CEventEmitter {
    on<T extends EventNames<CEvents>>(
        event: T,
        fn: EventListener<CEvents, T>,
        context?: any
    ): this;

    // emit
}

export class CEventEmitter extends EventEmitter {
    constructor() {
        super();
    }
}

As you can see I haven't even been able to implement the on* method correctly since I still have the emit* section commented out. This is as close as I can get it and I pretty much hit a wall. My compiler gives me the following complaint with this setup (when hovering CEventEmitter of export declare interface CEventEmitter):
TS2430: Interface 'CEventEmitter' incorrectly extends interface 'EventEmitter<string | symbol, any>'.

Types of property 'on' are incompatible.
    Type '<T extends "before-request">(event: T, fn: (...args: ArgumentMap<EventTypes>[Extract<T, "before-request">]) => void, context?: any) => this' is not assignable to type '<T extends string | symbol>(event: T, fn: (...args: any[]) => void, context?: any) => this'.

    Types of parameters 'event' and 'event' are incompatible.
        Type 'T' is not assignable to type '"before-request"'.
        Type 'string | symbol' is not assignable to type '"before-request"'.
        Type 'string' is not assignable to type '"before-request"'.

I hope someone out there can see what I'm missing and help me out... I'm sorry if this is the wrong place for this post. I'm not sure where else I would put it besides stack overflow. I figured I would try here since this is kind niche. I'm not new to JavaScript but this is my first stab at a real TypeScript project so I'm trying to understand and learn how this works. Thank you in advance.

hello
not a maintainer or anything, just put some thoughts regarding your task and this is what I ended up with:
import { EventEmitter } from 'eventemitter3';

interface Events {
  event1: { payload1: string };
  event2: { payload2: number };
}

class TypedEmitter< Events = Record< string, unknown > > {
  __ee = new EventEmitter();

  emit< K extends keyof Events >(type: K, payload: Events[K]) {
    this.__ee.emit(String(type), payload);
  }

  on< K extends keyof Events >(type: K, handler: (payload: Events[K]) => unknown) {
    this.__ee.on(String(type), handler);
  }
}

const typedEmitterInstance = new TypedEmitter<Events>();

typedEmitterInstance.emit('event1', { payload1: '' });
typedEmitterInstance.on('event2', p => console.log(p.payload2));

IMHO, gives pretty decent autocomplete experience
My setup is

VSCode 1.74.2
<EMAIL_ADDRESS>typescript v 4.9.4

Hi, guys.
I had the same problem with IDE support. So I found another solution - C#-like events.
It greatly improved the navigation and the editor hints.
I use VSCode with @js-doc, no Typescript.
There is an example how I define events:
class Kitty {
  /** @readonly */
  emitter = new EventEmitter();

  /**
   * Event with 2 args: string, integer.
   * @type {TypedEvent<[string, integer]>}
   * @readonly
   */
  eventSay = new TypedEvent(this.emitter, "say");

  /**
   * @param {string} word
   * @param {integer} cnt
   */
  say(word, cnt) {
    this.eventSay.safeEmit(word, cnt);
  }
}

const kitty = new Kitty();

kitty.eventSay.on((w, c) => { throw new Error('unhandled') }); // it will be ignored due to safeEmit
kitty.eventSay.once((w, c) => console.log(`once say ${w}:${typeof(w)} ${c}:${typeof(c)}`));
kitty.eventSay.once((w, c) => console.log(`on say ${w}:${typeof(w)} ${c}:${typeof(c)}`));

kitty.say("meow", 3);

Repo link: https://github.com/dgolovin-dev/eventemitter3-typedevent

This is a bit old, but for others, here's how I solved it with TypeScript.
In my case I wanted a custom class that is itself just an emitter, so I do things like this.emit("...", ...) in methods.
import { EventEmitter } from "eventemitter3";

// The types of the arguments you'll pass
interface argOne {
    foo: string;
}
interface argTwo {
    bar: number;
}

// This will contain all possible events and the args those events will pass
interface MyCustomEvents {
    "my.event": [argOne, argTwo];
}

// A new event emitter that uses the custom events
class TypedEmitter extends EventEmitter<MyCustomEvents> {
    constructor(props) {
        super();
    }

    doThing(text: string) {
        const argA = {foo: text};
        const argB = {bar: 1};

        this.emit("my.event", argA, argB);
    }
}

const test = new TypedEmitter();

test.on("my.event", (argOne, argTwo) => {
    // argOne is of type argOne
    // argTwo is of type argTwo
});

// My IDE will yell at me in these, for the respective reasons:

// no such event
test.on("doesnt.exist", () => {})

// that's not an operation I can perform on the type argOne is
test.on("my.event", (argOne) => {
    argOne++;
});

// you don't receive that many args
test.on("my.event", (argOne, argTwo, argThree) => {

});

// It will also yell at me for the same reasons when using .emit()

WebStorm 2023.3.4
TypeScript 5.2.2
<EMAIL_ADDRESS>


Compatibility with Turnip Features

I have a test suite that is a mixture of spec files in spec/**/*_spec.rb and Turnip features in spec/**/*.feature.  I'm trying to generate the report on TravisCI and with the pattern spec/**/*{_spec.rb,.feature} it correctly runs all of the specs and turnip features; however the generated report doesn't include any of the features.  I've looked through the source of this project and didn't spot anything obvious that I thought could be to blame.  Do you have any thoughts on how I could go about troubleshooting this?
This is the command I'm using to run the tests:
KNAPSACK_TEST_FILE_PATTERN="spec/**/*{_spec.rb,.feature}" KNAPSACK_GENERATE_REPORT=true bundle exec rspec spec

@greggroth Hi, I've just did quick test and pattern works in case when .feature file has rspec tests.
I guess your .feature file has cucumber test inside so in that case you need to generate a separate report for cucumber tests and use cucumber adapter:
https://github.com/ArturT/knapsack#step-for-cucumber

The specs are written using the gherkin syntax, but are ran using rspec, e.g. bin/rspec spec/features/logging_in.feature.  Would that still require a separate report?  I would think it would be included in the rspec report.

Hmm, so it looks like rspec hooks were not fired and that's why report doesn't have .feature files.
https://github.com/ArturT/knapsack/blob/master/lib/knapsack/adapters/rspec_adapter.rb#L9
Maybe you are using a separate spec_helper.rb for .feature and you missed to bind there Knapsack::Adapters::RspecAdapter.
If I will find some time during weekend I can prepare test suite with turnip to debug the problem unless you will figure out it earlier.

Thank you for the hint!  I dug in and found it's an issue with Knapsack::Adapters::RspecAdapter.test_path's ability to parse the filename from example metadata.  This is when running a turnip feature:
pry(#<RSpec::ExampleGroups::REDACTED::REDACTED>)> Knapsack::Adapters::RspecAdapter.test_path(::RSpec.current_example.metadata[:example_group])
=> "/Users/greggory/.gem/ruby/2.2.2/gems/turnip-1.2.4/lib/turnip/rspec.rb"

and with a normal spec file:
[1] pry(#<RSpec::ExampleGroups::REDACTED::REDACTED::REDACTED>)> Knapsack::Adapters::RspecAdapter.test_path(::RSpec.current_example.metadata[:example_group])
=> "./spec/models/user_spec.rb"

I'll dig in further and see if I can't put together a PR to improve this.

Cool. You may want to take a look how I get the test file path in other adapters
https://github.com/ArturT/knapsack/tree/master/lib/knapsack/adapters
Please let me know if you will have something. In case you stuck I will help with that.


Update spring-boot-starter-test, ... to 3.3.2

About this PR
📦 Updates

org.springframework.boot:spring-boot-starter-test
org.springframework.boot:spring-boot-starter-web

from 2.7.18 to 3.3.2 ⚠
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.springframework.boot" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.springframework.boot" }
}]

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

Superseded by #2014.


add token document types with multi spans

This Pr adds the following document types:

TokenDocumentWithLabeledMultiSpans,
TokenDocumentWithLabeledMultiSpansAndLabeledPartitions,
TokenDocumentWithLabeledMultiSpansAndBinaryRelations, and
TokenDocumentWithLabeledMultiSpansBinaryRelationsAndLabeledPartitions.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.83%. Comparing base (dea7466) to head (af5d2b4).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #75      +/-   ##
==========================================
+ Coverage   95.82%   95.83%   +0.01%
==========================================
  Files          52       52
  Lines        3806     3818      +12
==========================================
+ Hits         3647     3659      +12
  Misses        159      159

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Management server metrics only shows on 1 GUI at a time

ISSUE TYPE

Bug Report

COMPONENT NAME

GUI

CLOUDSTACK VERSION

4.18.1

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

When you go to the infrastructure>management server section and toggle on the metrics switch. We can only see the current metrics of the management vm that you are currently logged into. Both management servers are connected to the same mysql DB.
STEPS TO REPRODUCE

EXPECTED RESULTS

We excepted to see both management server metrics from either managements VMs GUI

ACTUAL RESULTS

The expected result do not happen

@Tbaugus44  In my tests I do see the metrics for both management servers.

my view @Tbaugus44 :

Might there be some connection problem between your MSses?
If you investigate the logs, anything related that you see?

Could you tell me what logs i need to look at. I am very new to Apachecloudstack and especially mysql. thanks

@Tbaugus44  You can look for error messages like "Failed to generate zone metrics response".
You can find the below messages otherwise:
2023-12-07 07:31:54,172 DEBUG [o.a.c.m.MetricsServiceImpl] (qtp1550261631-17:ctx-d36e58f5 ctx-b91e5e56) (logid:13334424) Getting stats for mgmt2.shapeblue.com[c40c8a5b-06a5-45db-aebf-e195f09811b0]
2023-12-07 07:31:54,173 DEBUG [o.a.c.m.MetricsServiceImpl] (qtp1550261631-17:ctx-d36e58f5 ctx-b91e5e56) (logid:13334424) Status object found for MS mgmt2.shapeblue.com  - com.cloud.server.ManagementServerHostStatsEntry@7df6e68c[agentCount=0,availableProcessors=2,collectionTime=Thu Dec 07 07:31:37 UTC 2023,cpuUtilization=0.0,dbLocal=false,freeJvmMemoryBytes=548813448,heapMemoryTotal=1908932608,heapMemoryUsed=216256336,jvmName=16961@mgmt2,jvmStartTime=1701931499082,jvmUptime=2797982,jvmVendor=Ubuntu,jvmVersion=11.0.21+9-post-Ubuntu-0ubuntu122.04,kernelVersion=5.15.0-35-generic,loadAverage=0.0,logInfo=/var/log/cloudstack/management/management-server.log using: 2.2M

This feature is implemented by #5588  for more information.

this is what i see in the /var/log/cloudstack/management/management-server.log
2023-12-19 16:33:05,892 DEBUG [o.a.c.a.StaticRoleBasedAPIAccessChecker] (qtp1550261631-319:ctx-203a91a8 ctx-feb9f116) (logid:54559778) RoleService is enabled. We will use it instead of StaticRoleBasedAPIAccessChecker.
2023-12-19 16:33:05,893 DEBUG [o.a.c.r.ApiRateLimitServiceImpl] (qtp1550261631-319:ctx-203a91a8 ctx-feb9f116) (logid:54559778) API rate limiting is disabled. We will not use ApiRateLimitService.
2023-12-19 16:33:05,895 WARN  [c.c.a.d.ParamGenericValidationWorker] (qtp1550261631-319:ctx-203a91a8 ctx-feb9f116) (logid:54559778) Received unknown parameters for command listManagementServersMetrics. Unknown parameters : listall
2023-12-19 16:33:05,897 DEBUG [o.a.c.h.HAManagerImpl] (BackgroundTaskPollManager-6:ctx-758e48ce) (logid:816b1eb7) HA health check task is running...
2023-12-19 16:33:05,898 DEBUG [o.a.c.m.MetricsServiceImpl] (qtp1550261631-319:ctx-203a91a8 ctx-feb9f116) (logid:54559778) Getting metrics for 2 MS hosts.
2023-12-19 16:33:05,898 DEBUG [o.a.c.m.MetricsServiceImpl] (qtp1550261631-319:ctx-203a91a8 ctx-feb9f116) (logid:54559778) Processing metrics for MS hosts 75ca0d4d-0ff1-4d61-af60-3d556099cc2e.
2023-12-19 16:33:05,898 DEBUG [o.a.c.m.MetricsServiceImpl] (qtp1550261631-319:ctx-203a91a8 ctx-feb9f116) (logid:54559778) Getting stats for erl-cs-mgt-vm-01[75ca0d4d-0ff1-4d61-af60-3d556099cc2e]
2023-12-19 16:33:05,898 DEBUG [o.a.c.m.MetricsServiceImpl] (qtp1550261631-319:ctx-203a91a8 ctx-feb9f116) (logid:54559778) Status object found for MS erl-cs-mgt-vm-01 - com.cloud.server.ManagementServerHostStatsEntry@39c57a65[agentCount=3,availableProcessors=8,collectionTime=Tue Dec 19 16:32:22 EST 2023,cpuUtilization=0.0,dbLocal=false,freeJvmMemoryBytes=225172320,heapMemoryTotal=1908932608,heapMemoryUsed=538380584,jvmName=87196@ERL-CS-MGT-VM-01,jvmStartTime=1703021457674,jvmUptime=84386,jvmVendor=Ubuntu,jvmVersion=11.0.21+9-post-Ubuntu-0ubuntu122.04,kernelVersion=5.15.0-91-generic,loadAverage=0.603515625,logInfo=/var/log/cloudstack/management/management-server.log using: 78M
on disk /dev/mapper/ubuntu--vg-ubuntu--lv mounted on / (8% full),managementServerHostId=1,managementServerHostUuid=75ca0d4d-0ff1-4d61-af60-3d556099cc2e,maxJvmMemoryBytes=1908932607,osDistribution=Ubuntu 22.04.3 LTS,pid=87196,processJvmMemoryBytes=0,sessions=2,systemBootTime=Mon Dec 18 10:40:57 EST 2023,systemCyclesUsage={224316,62928,85523366},systemLoadAverages={0.6,0.32,0.11},systemMemoryFree=13519716352,systemMemoryTotal=16764178432,systemMemoryUsed=1239116,systemMemoryVirtualSize=8340467712,systemTotalCpuCycles=24741.92,threadsBlockedCount=0,threadsDaemonCount=13,threadsRunnableCount=17,threadsTerminatedCount=0,threadsTotalCount=203,threadsWaitingCount=100,totalCommitted=947888128,totalInit=269811712,totalJvmMemoryBytes=762314752,totalUsed=716017328,usageLocal=false].
2023-12-19 16:33:05,898 DEBUG [o.a.c.m.MetricsServiceImpl] (qtp1550261631-319:ctx-203a91a8 ctx-feb9f116) (logid:54559778) Processing metrics for MS hosts 10defb5c-f546-434f-ac9e-f4f69eb334e9.
2023-12-19 16:33:05,898 DEBUG [o.a.c.m.MetricsServiceImpl] (qtp1550261631-319:ctx-203a91a8 ctx-feb9f116) (logid:54559778) Getting stats for ERL-CS-MGT-VM-02[10defb5c-f546-434f-ac9e-f4f69eb334e9]
2023-12-19 16:33:05,899 INFO  [o.a.c.m.MetricsServiceImpl] (qtp1550261631-319:ctx-203a91a8 ctx-feb9f116) (logid:54559778) No status object found for MS ERL-CS-MGT-VM-02 - 10defb5c-f546-434f-ac9e-f4f69eb334e9.
2023-12-19 16:33:05,901 DEBUG [c.c.a.ApiServlet] (qtp1550261631-319:ctx-203a91a8 ctx-feb9f116) (logid:54559778) ===END===  <IP_ADDRESS> -- GET  listall=true&page=1&pagesize=20&command=listManagementServersMetrics&response=json

this indeed means there is no data for "the other" host. One thing springs out: erl-cs-mgt-vm-01 vs. ERL-CS-MGT-VM-02 the two MS names are in different case. This would not be an issue in DNS but might be somewhere in the code.

Can you check the other one to see if you see the same difference?
Also, can you check what is in the mshosts table?

@Tbaugus44  like @DaanHoogland  mentioned there could be some stale records for the same management server, try this:
select id,msid,name,state,last_update from mshost where removed is null;


B4 Submission

rejected:B1
It looks like you are trying to submit B1, however it seems like the following submission criteria are problematic:

Logs are missing
We’ve thus marked B1 as rejected, however if this is a mistake, please upload the log summary and we can re-review.

confirmed:B4
address:3E5gbj4Hr8Es3Z44rMdVqBJKdyR5oktvSf7Xtv17MpHXF6dC2H
baker:434

confirmed:B4
address:3E5gbj4Hr8Es3Z44rMdVqBJKdyR5oktvSf7Xtv17MpHXF6dC2H
baker:434


  I'm trying to replicate this but seems to fix itself when I log out and log back in to the host.
When starting gdu I see a blank screen for a couple of seconds before the UI pops up and it scans the current location. When in gdu, scrolling up and down with the keyboard is extremely delayed.
Another thing I noticed since I changed my primary driver to Windows 11 is that the UI flashes each time I scroll up or down. ncdu does not flash updates when scrolling through the location content. This also did not happen when I used Linux. It must be something that Windows Terminal is doing different.
https://user-images.githubusercontent.com/13314239/189061326-c9003e38-17e1-4222-b33e-731616f6b5cb.mp4
I'm using Windows terminal 1.14.

I'm not able to replicate the slowness in some sessions. Usually after logging out and back in gdu works like normal.

Let's add option to disable the mouse support and compare running gdu with that.

I can't reproduce the slowness, but yes the constant redrawing of the terminal on any action is quite jarring. I suspect it has something to do with the curses Go backend that gdu uses.

can't replicate


Visual Studio 2017 Versioning for Hosted Container Template...

You need to have Visual Studio Version 15.7.2 to use the Hosted Container Template. To Repro create a VM with Visual Studio 2017 and try to follow these instructions. To work around you'll need to update VS in the Windows VM and the Hosted Container Templates will be available.

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

ID: 922b9c51-6a6d-1118-d2a2-6d485b222ce8
Version Independent ID: 6874cc89-a1f8-addd-430f-a4d6b7f89ae8
Content: Create an Azure Service Fabric Windows container application
Content Source: articles/service-fabric/service-fabric-quickstart-containers.md
Service: service-fabric
Product: unspecified
GitHub Login: @rwike77
Microsoft Alias: ryanwi

@timmyreilly Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@timmyreilly  Hi there, I can't repro what you're seeing.  The Hosted Container template has been out for maybe a year now, actually.  There is no requirement to use VS 2017 15.7.2, either.  If you're still experiencing problems, can I ask you to open up an issue in the engineering team repo?  #please-close


Partition DOC, DOCX, PPT, PPTX: No images detected

Hello,
I'am using ustructured io through library to analyze doc, docx, ppt, pptx documento.
When I use the following functions:

partition_doc;
partition_docx;
partition_ppt;
partition_pptx;
to analyze documents containing images,
the object returned doens't contain related images.
Is it the right behavior?
Thanks you for answers.

@liturrig Yes, this is the expected behavior. The unstructured library does not extract images for the DOC, DOCX, PPT, or PPTX formats.
Image extraction will be available for these formats soon on the REST APIs, including the freemium API. For PPT and PPTX that's likely to be within two weeks. For DOC and DOCX somewhat longer.


Support processed res deRedundancy

解压 .ap_  -> 删除重复的文件，再重新定位 arsc 重复的资源 -> 重新打包 .ap_

Could you show me an example of duplicated entries in .ap_ archive file?

@johnsonlee I added the relevant log to report.
it will check  duplicated entries in same parent file.
test
test demo

deme's layout, drawable has duplicated files:

ic_launcher_background.xml duplicated with ic_launcher_background2.xml,  ic_launcher_background3.xml
activity_main2.xml  duplicated with  activity_main3.xml,activity_main5.xml,activity_main7.xml

see its report:
0.85% intermediates/processed_res/debug/out/resources-debug.ap_ 8,167 961,453 /Users/luwenjie/Documents/demo/SlowFunctionCheckDemo/app/build/intermediates/processed_res/debug/out/resources-debug.ap_
---------------------------------------------------------------------
 TOTAL                                                          8,167

removed 1 entry duplicated with Entry:[name:res/color/checkbox_themeable_attribute_color.xml, crc:106581170, size:464]:
1. Entry:[name:res/color/radiobutton_themeable_attribute_color.xml, crc:106581170, size:464]

removed 1 entry duplicated with Entry:[name:res/color/mtrl_bottom_nav_colored_ripple_color.xml, crc:2638324326, size:948]:
1. Entry:[name:res/color/mtrl_tabs_colored_ripple_color.xml, crc:2638324326, size:948]

removed 1 entry duplicated with Entry:[name:res/color/mtrl_bottom_nav_ripple_color.xml, crc:153417060, size:1672]:
1. Entry:[name:res/color/mtrl_tabs_ripple_color.xml, crc:153417060, size:1672]

removed 1 entry duplicated with Entry:[name:res/color/mtrl_navigation_item_icon_tint.xml, crc:945490082, size:624]:
1. Entry:[name:res/color/mtrl_navigation_item_text_color.xml, crc:945490082, size:624]

removed 1 entry duplicated with Entry:[name:res/color-v23/abc_tint_edittext.xml, crc:1954525703, size:668]:
1. Entry:[name:res/color-v23/abc_tint_spinner.xml, crc:1954525703, size:668]

removed 2 entry duplicated with Entry:[name:res/drawable/ic_launcher_background.xml, crc:3985735082, size:5696]:
1. Entry:[name:res/drawable/ic_launcher_background2.xml, crc:3985735082, size:5696]
2. Entry:[name:res/drawable/ic_launcher_background3.xml, crc:3985735082, size:5696]

removed 1 entry duplicated with Entry:[name:res/drawable/mtrl_popupmenu_background.xml, crc:1492599386, size:740]:
1. Entry:[name:res/drawable/mtrl_popupmenu_background_dark.xml, crc:1492599386, size:740]

removed 1 entry duplicated with Entry:[name:res/interpolator/btn_checkbox_checked_mtrl_animation_interpolator_0.xml, crc:1170948913, size:316]:
1. Entry:[name:res/interpolator/btn_checkbox_unchecked_mtrl_animation_interpolator_0.xml, crc:1170948913, size:316]

removed 1 entry duplicated with Entry:[name:res/interpolator/btn_checkbox_checked_mtrl_animation_interpolator_1.xml, crc:3970749230, size:328]:
1. Entry:[name:res/interpolator/btn_checkbox_unchecked_mtrl_animation_interpolator_1.xml, crc:3970749230, size:328]

removed 1 entry duplicated with Entry:[name:res/interpolator/btn_radio_to_off_mtrl_animation_interpolator_0.xml, crc:2960756502, size:320]:
1. Entry:[name:res/interpolator/btn_radio_to_on_mtrl_animation_interpolator_0.xml, crc:2960756502, size:320]

removed 3 entry duplicated with Entry:[name:res/layout/activity_main2.xml, crc:245409587, size:880]:
1. Entry:[name:res/layout/activity_main3.xml, crc:245409587, size:880]
2. Entry:[name:res/layout/activity_main5.xml, crc:245409587, size:880]
3. Entry:[name:res/layout/activity_main7.xml, crc:245409587, size:880]

removed 1 entry duplicated with Entry:[name:res/mipmap-anydpi-v26/ic_launcher.xml, crc:1146363823, size:448]:
1. Entry:[name:res/mipmap-anydpi-v26/ic_launcher_round.xml, crc:1146363823, size:448]

test on my app
report:
0.46% intermediates/processed_res/release/out/resources-release.ap_ 7,990 1,740,548 /Users/luwenjie/Documents/androidStudioProjects/mygithub/xbtc-android/app/build/intermediates/processed_res/release/out/resources-release.ap_
-------------------------------------------------------------------------
 TOTAL                                                              7,990

removed 1 entry duplicated with Entry:[name:res/anim/nav_default_enter_anim.xml, crc:2324485647, size:460]:
1. Entry:[name:res/anim/nav_default_pop_enter_anim.xml, crc:2324485647, size:460]

removed 1 entry duplicated with Entry:[name:res/anim/nav_default_exit_anim.xml, crc:1627603134, size:460]:
1. Entry:[name:res/anim/nav_default_pop_exit_anim.xml, crc:1627603134, size:460]

removed 1 entry duplicated with Entry:[name:res/animator/nav_default_enter_anim.xml, crc:2354174408, size:452]:
1. Entry:[name:res/animator/nav_default_pop_enter_anim.xml, crc:2354174408, size:452]

removed 1 entry duplicated with Entry:[name:res/animator/nav_default_exit_anim.xml, crc:2143089099, size:452]:
1. Entry:[name:res/animator/nav_default_pop_exit_anim.xml, crc:2143089099, size:452]

removed 1 entry duplicated with Entry:[name:res/color/checkbox_themeable_attribute_color.xml, crc:2371085192, size:464]:
1. Entry:[name:res/color/radiobutton_themeable_attribute_color.xml, crc:2371085192, size:464]

removed 1 entry duplicated with Entry:[name:res/color/mtrl_bottom_nav_colored_ripple_color.xml, crc:3344682506, size:948]:
1. Entry:[name:res/color/mtrl_tabs_colored_ripple_color.xml, crc:3344682506, size:948]

removed 1 entry duplicated with Entry:[name:res/color/mtrl_bottom_nav_ripple_color.xml, crc:542667710, size:1672]:
1. Entry:[name:res/color/mtrl_tabs_ripple_color.xml, crc:542667710, size:1672]

removed 1 entry duplicated with Entry:[name:res/color/mtrl_navigation_item_icon_tint.xml, crc:1622613214, size:624]:
1. Entry:[name:res/color/mtrl_navigation_item_text_color.xml, crc:1622613214, size:624]

removed 1 entry duplicated with Entry:[name:res/color-v23/abc_tint_edittext.xml, crc:4249494616, size:668]:
1. Entry:[name:res/color-v23/abc_tint_spinner.xml, crc:4249494616, size:668]

removed 2 entry duplicated with Entry:[name:res/drawable/abc_ratingbar_indicator_material.xml, crc:1548506544, size:124]:
1. Entry:[name:res/drawable/abc_ratingbar_material.xml, crc:1548506544, size:124]
2. Entry:[name:res/drawable/abc_ratingbar_small_material.xml, crc:1548506544, size:124]

removed 1 entry duplicated with Entry:[name:res/drawable/ic_darkmode_24dp.xml, crc:1677281061, size:2028]:
1. Entry:[name:res/drawable/ic_setting_darkmode.xml, crc:1677281061, size:2028]

removed 1 entry duplicated with Entry:[name:res/drawable/mtrl_popupmenu_background.xml, crc:861533045, size:740]:
1. Entry:[name:res/drawable/mtrl_popupmenu_background_dark.xml, crc:861533045, size:740]

removed 1 entry duplicated with Entry:[name:res/interpolator/btn_checkbox_checked_mtrl_animation_interpolator_0.xml, crc:1013122958, size:316]:
1. Entry:[name:res/interpolator/btn_checkbox_unchecked_mtrl_animation_interpolator_0.xml, crc:1013122958, size:316]

removed 1 entry duplicated with Entry:[name:res/interpolator/btn_checkbox_checked_mtrl_animation_interpolator_1.xml, crc:2500106129, size:328]:
1. Entry:[name:res/interpolator/btn_checkbox_unchecked_mtrl_animation_interpolator_1.xml, crc:2500106129, size:328]

removed 1 entry duplicated with Entry:[name:res/interpolator/btn_radio_to_off_mtrl_animation_interpolator_0.xml, crc:3385969577, size:320]:
1. Entry:[name:res/interpolator/btn_radio_to_on_mtrl_animation_interpolator_0.xml, crc:3385969577, size:320]

removed 2 entry duplicated with Entry:[name:res/layout/page_preupload_config.xml, crc:2745024722, size:360]:
1. Entry:[name:res/layout/view_prime.xml, crc:2745024722, size:360]
2. Entry:[name:res/layout/view_pwd_login.xml, crc:2745024722, size:360]

removed 1 entry duplicated with Entry:[name:res/mipmap-anydpi-v26/ic_launcher.xml, crc:800106227, size:448]:
1. Entry:[name:res/mipmap-anydpi-v26/ic_launcher_round.xml, crc:800106227, size:448]

I quite confused, what's the definition of the term duplicated entry?

isn't same crc😂?

What if some resources with same CRC have been used by code?

the value which entry is duplicated  in arsc  has relocation to first resource. Is there a problem with who exit first？
maybe you mean getting resources not through R.X.X. intead of accessing  file directly in some other way？delete this files will cause exception.

Not the case, if no duplicate entries exist, it's a waste of I/O twice

thank you , i will check it


[Feature] Translation Support

Short summary
Translation Support
Detailed description
Any plans to support translation?
Examples 🖼
No response
Additional context 🗒
No response

Hi @Tntdruid , yes there are minor improvements in the pipeline and the addition of localization functions for translation. 😄 I will fix some bugs and prepare for release.

Very nice 👍

@Tntdruid Feature for translations is in new release. PR is needed for further translation 👍


[Home/Artist] Omit already followed artists

E.g. in the list of popular artists is Andy Warhol which I already follow.

This probably needs to be done in metaphysics btw.

@broskoski fixed it for suggested artists in https://github.com/artsy/metaphysics/issues/389

This is done with:

https://github.com/artsy/gravity/pull/10309
https://github.com/artsy/gravity/pull/10316
https://github.com/artsy/metaphysics/pull/393


Separate Hubspot functionality into Core project

Following issue #133 , I've split the HubSpot functionalities into a separate *.Core project.

Makes sense, leaving this as it is.


[EBPF] Fix cleanup jobs running before setup finishes

What does this PR do?
Forces KMT cleanup jobs to wait for the corresponding setup job to finish.
Motivation
For KMT we have the following dependency graph
[other jobs] ---> kmt_run_tests ---> cleanup
              /                  /
setup_env  --/------------------/

Based on the dependencies, we would expect that the cleanup jobs only run when both the kmt_run_tests AND the setup_env jobs finish. However, it seems that Gitlab tries to simplify the dependency graph: as it sees that kmt_run_tests already depend on setup_env, it assumes that dependency implies a dependency of cleanup on setup_env and removes the explicity dependency we define.
However, that causes a problem when a dependency of kmt_run_tests fails before setup_env has had a chance to finish. Gitlab will mark the kmt_run_tests as skipped, and because it removes the explicit dependency of cleanup on setup_env, it will immediately schedule cleanup. This means that the instances that the setup_env job ends up provisioning will not be cleaned up.
With this PR, we add a explicit loop that waits for the corresponding setup job to finish. This way, even if Gitlab schedules the cleanup jobs incorrectly, they will clean up the instances correctly.
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


error: failed to run custom build command for ring v0.17.8

Crate name
webserver-rs
Build failure link
https://docs.rs/crate/webserver-rs/0.5.2/builds/1202362/x86_64-apple-darwin.txt
Additional details
The doc generates failure for Windows and Apple-Darwin platforms.

sorry for the delayed response.
Currently we are using cross compilation to compile all non-default platforms.
In this case, ring can't be compiled like this on non-default platforms, which is probably also why its docs are only available for one platform. We don't see the failing logs because storing them was added later.
#1957 might be a solution, or #93. Until them, many crates choose conditional compilation to make the crate build purely for docs in our build.
for reference:

https://docs.rs/about/builds


enhancement / typo

correcting a typo.

@seedkar1 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Fixes issue #3158


[i18n] update some translations

A few updated strings from TC

my usage of google translated has skyrocketed this month :)


Add datalabelsRoundDecimalPlace option

Enable to select the decimal place to which round for
data labels on the charts.

Sample of dashboard when datalabelsRoundDecimalPlace is set as 0.

When datalabelsRoundDecimalPlace is set as 1 (by default),

When datalabelsRoundDecimalPlace is set as 2,


Error: 0xffffffff On windows update

Environment
[Version 10.0.19041.450]
WSL 2
Windows build number: [run `[Environment]::OSVersion` for powershell, or `ver` for cmd]
Your Distribution version: [On Debian or Ubuntu run `lsb_release -r` in WSL]
Whether the issue is on WSL 2 and/or WSL 1: [run `cat /proc/version` in WSL]

Steps to reproduce
After installing windows updates on reboot. I got a Error: 0xffffffff.
After a hour of searching for a solution I Stopped and restarted Hyper-V Virtual Machine Management service and dockerD service.
Then it started working again

Was this Windows 10 Pro or Home?

/template


Map crashes app

Below is my rep.
I followed the tutorial https://github.com/eclipsesource/tabris-maps-example
https://github.com/iteearmah/lokate

What do you mean by "crashes"? Could you provide more details about the platform and versions you are on?

its andriod, tabris verion is nightly build.

Want to know if I'm not doing something right....Can anyone try the map to see the results

I have created a local build and added the maps plugin:
cordova plugin add https://github.com/eclipsesource/tabris-maps --variable API_KEY_FOR_ANDROID="<my-key>"
The was shown as expected. Could you provide some information about your setup or even better a stacktrace of the crash?

Just did a release build of the https://github.com/eclipsesource/tabris-maps-example repo
I got the following error
Starting build
Preparing build environment
Using build template revision 0c1e7ba4b565a9b7d523a4bf5c19ca5c601765a3
Build environment created 57417d436508b0e45b609762
Cloning app from https://github.com/iteearmah/tabris-maps-example.git
Using app revision 3f8d68ec0c5a8aaeb8512050c77ffe6c5b6a9f1f
Processing App Directory
Applying config.xml
WARNING: The Tabris.js version used for the build (1.8.0) does not match the version configured in the package.json of your app (^1.5.x). This may cause runtime errors!
WARNING: A release build must disable the developer console. Change the preference "EnableDeveloperConsole" to "false" in your config.xml.
Writing Android keystore
Encrypting Android Passwords
These things will be built:
  App Id: com.eclipsesource.tabris.maps.example
  Tabris.js Version: 1.8.0
  Debug Mode: false
  Platform: ANDROID
  Plugins:
    https://github.com/eclipsesource/tabris-maps.git
Creating build job 11530d26cc2116b449ddddff7e5f6f85bae2362d
Scheduling job (57417d436508b0e45b609762). This may take a few minutes...
ERROR: Build failed. Please download the detailed log.

This isn't an app crash but is a build failure.
WARNING: The Tabris.js version used for the build (1.8.0) does not match the version configured in the package.json of your app (^1.5.x). This may cause runtime errors!

Update your package.json to set your tabris dependency to ^1.8.0
WARNING: A release build must disable the developer console. Change the preference "EnableDeveloperConsole" to "false" in your config.xml.

I don't know if this will cause the build to fail but it's a good warning to heed--try fixing this as well.
If you continue to see this:
ERROR: Build failed. Please download the detailed log.

Please post the entire detailed log which can be obtained by clicking the "Detailed log" button immediately below this line of text.

Total time: 1 mins 29.563 secs
Built the following apk(s):
	/home/travis/build/eclipsesource/tabris-js-build-template/build/platforms/android/build/outputs/apk/android-release-unsigned.apk
Running "create-artifacts" task
Running "shell:sign_apk" (shell) task
jarsigner: Certificate chain not found for: mystore.  mystore must reference a valid KeyStore key entry containing a private key and corresponding public key certificate chain.
Warning: Command failed: /bin/sh -c jarsigner -verbose -sigalg SHA1withRSA -digestalg SHA1 -tsa https://timestamp.geotrust.com/tsa -keystore signing/android.keystore -storepass $TABRIS_ANDROID_STORE_PASSWORD -keypass $TABRIS_ANDROID_KEY_PASSWORD -signedjar build/platforms/android/build/outputs/apk/com.eclipsesource.tabris.maps.example-unaligned.apk build/platforms/android/build/outputs/apk/android-release-unsigned.apk $TABRIS_ANDROID_ALIAS
 Use --force to continue.
Aborted due to warnings.
travis_time:end:17826d70:start=1463909853711003349,finish=1463909953058978154,duration=99347974805

The command "grunt build deploy" exited with 6.
Done. Your build exited with 1.

Did you upload a valid keystore?  See:
http://docs.build.phonegap.com/en_US/signing_signing-android.md.html#Android Signing

Hi did now; I was able to build but closes and say " Unfortunately,
TabisMapExample  has stopped.
On Mon, May 23, 2016 at 7:04 PM, Tim Bond<EMAIL_ADDRESS>wrote:

Did you upload a valid keystore https://tabrisjs.com/iteearmah/keys?
See:
http://docs.build.phonegap.com/en_US/signing_signing-android.md.html#Android Signing
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/eclipsesource/tabris-js/issues/840#issuecomment-221060256

@iteearmah do you have access to the android crash stacktrace?

Kindly help me with how to get that.
On Wed, May 25, 2016 at 12:19 PM, Moritz Post<EMAIL_ADDRESS>wrote:

@iteearmah https://github.com/iteearmah do you have access to the
android crash stacktrace?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/eclipsesource/tabris-js/issues/840#issuecomment-221556881

--
Samuel Armah
Tel:<PHONE_NUMBER>29
Facebook http://facebook.com/iteearmah | Linkedin
http://www.linkedin.com/profile/view?id=289834392 | Twitter
https://twitter.com/iteearmah

with the android sdk installed you can use the logcat tool or with android studio installed you can use the Android Monitor view.

http://stackoverflow.com/a/29200956
Also I believe apps like Logcat give you the same info without having to set up the Android SDK

Can I output [(https://github.com/mapsplugin/cordova-plugin-googlemaps)] in a Composite similar to DOM div?

Example
document.addEventListener("deviceready", function() {
  var mapDiv = document.getElementById("map_canvas");

  const GOOGLE = new plugin.google.maps.LatLng(37.422476,-122.08425);
  var map = plugin.google.maps.Map.getMap(mapDiv, {
    'camera': {
      'latLng': GOOGLE,
      'zoom': 17
    }
  });

  map.addEventListener(plugin.google.maps.event.MAP_READY, function() {

    map.addMarker({
      'position': GOOGLE,
      'title': "Hello GoogleMap for Cordova!"
    }, function(marker) {

      marker.showInfoWindow();

    });

  });
});

I can't seem to get the tabris map working on my Jelly Bean 4.1.2

Can I output cordova-plugin-googlemaps in a Composite similar to DOM div?

No, that plugin depends on the DOM and in the docs:

Tabris.js uses a native UI and no HTML5. As a result all Plug-Ins that manipulate the DOM will not work.

I can't seem to get the tabris map working on my Jelly Bean 4.1.2

What error messages are you seeing?

can someone fork and build https://github.com/eclipsesource/tabris-maps-example

Do we need  Google Play Service to Use Google maps in Apps

Hi @iteearmah. I just verified, that tabris-maps-example builds fine with our build service at https://tabrisjs.com The app produced by that build is working as expected.
Please be sure to use a valid Google Maps API key in your build.
You have to associate the API key with either a production or debug key you use when building your app.
When you use a debug key, it has to be a key created by your own and registered with https://tabrisjs.com so the build services does not create a new debug key for each build under the hood (default behavior if you do not specify a key explicitly).
Of course you can use tabris-maps-example as a starting point to make your own experiments.
Please let me know if this helps.

Can I use https://github.com/mapbox/mapbox-sdk-js in tabris

This is my apk http://s000.tinyupload.com/?file_id=02210187360703963462
It says Unfortunately, TabrisMapsExample has stopped.

Hi Guys... I know I have been on this issue for a while now.
If anybody can give me a step by step guide to building and making it work, I will really I appreciate it.
Also it will help others too.
Thanks

@iteearmah What is your current problem state?

Same as stated above. U can download and install my apk link above
On Wed, Jul 6, 2016 at 2:15 PM, Moritz Post<EMAIL_ADDRESS>wrote:

@iteearmah https://github.com/iteearmah What is your current problem
state?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/eclipsesource/tabris-js/issues/840#issuecomment-230784627,
or mute the thread
https://github.com/notifications/unsubscribe/AMcljQcGep4AervvEkpGk0yVmQPGTgy9ks5qS7h1gaJpZM4Id8E0
.

@iteearmah You mean the build problem you mentioned here? https://github.com/eclipsesource/tabris-js/issues/840#issuecomment-220823214

Yes please
On Jul 7, 2016 8:11 AM, "Moritz Post"<EMAIL_ADDRESS>wrote:

@iteearmah https://github.com/iteearmah You mean the build problem you
mentioned here? #840 (comment)
https://github.com/eclipsesource/tabris-js/issues/840#issuecomment-220823214
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/eclipsesource/tabris-js/issues/840#issuecomment-231011756,
or mute the thread
https://github.com/notifications/unsubscribe/AMcljaTYVN0pfxstqpeuPrn3uf846EyMks5qTLSugaJpZM4Id8E0
.

Just tried the maps on an android 5.1 it opens the app. It shows the google app container. and the three buttons.
I'm thinking it add to do with my previous and android version Jelly Bean 4.1.2

@iteearmah Are you able to provide a crash log entry? Use logcat from the android sdk.

Unfortunately, I flashed my android device and replaced the 5.1

@iteearmah If you don't see a map than it is probably related to the API key not beieng properly inserted. Make sure to use a build hook to insert the api key: https://github.com/eclipsesource/tabris-maps-example/blob/master/hooks/api_key_hook.js

With the release of tabris-maps 2.0 this issue became invalid. https://github.com/eclipsesource/tabris-plugin-maps
Closing for now.


Convert Callback to be an enum to support Fn and FnOnce.

Closes #1039

@jstarry Thanks for your advice. I had tripped wrapping the callaback in an Option, but without the RefCell I obviously couldn't get it to work. One resulting issue now is the build is failing due to clippy warning about the complexity of the type:
error: very complex type used. Consider factoring parts into `type` definitions
  --> yew/src/callback.rs:19:18
   |
19 |     CallbackOnce(Rc<RefCell<Option<Box<dyn FnOnce(IN)>>>>),
   |                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |

I thought I'd solicit some input on how you'd break this up, since it's mostly about what to name the type, and how much of the type to extract. I was thinking something like:
type OptionalCallback<IN> = RefCell<Option<Box<dyn FnOnce(IN)>>>

Nice work!
How about this:
type CallbackOnce<IN> = RefCell<Option<Box<dyn FnOnce(IN)>>>;

Also can you please add this function: Scope::callback_once to make it easy for devs to create one time callbacks?
(ComponentLink is aliased to scope btw)

Thanks, I agree Callback::once looks a lot better! Now if only I could remember to run cargo fmt before pushing 🙈


BySearch() documentation

What versions are you running?
$ go list -m github.com/chromedp/chromedp
github.com/chromedp/chromedp v0.8.0
$ google-chrome --version
(using Docker container chromedp/headless-shell:latest)
$ go version
go version go1.16.15 linux/amd64

What did you do? Include clear steps.

chromedp.Click("#login-btn")

Full source code can be viewed here: https://github.com/porjo/ing-au-go/blob/d60003a6df2a098a0bf2874d77e085bc6be1cd1d/main.go#L124
What did you expect to see?
I expected the login button to be clicked
What did you see instead?
Button was not clicked and process timed out after context deadline.
When I changed the Click to be chromedp.Click("//*[@id=\"login-btn\"]") then it work as expected.
The issue is about clarification of the documentation. Doco says that by default, when QueryOption is not provided, the BySearch selector will be used. It says this about it:

The BySearch (default) option enables querying for elements by plain text, CSS selector or XPath query, wrapping DOM.performSearch.

It's not clear when plain text will be used vs CSS selector. In my case, I think what was happening is that when I supplied #login-btn it was performing a plain text search which was matching multiple elements in the HTML source (there is inline CSS that matches and the actual HTML element).
Could the documentation be updated to make this behaviour clear?

It's not clear when plain text will be used vs CSS selector.

It seems that you don't understand the document correctly. There are 3 sets of elements (see the list below). The search result is the union of all the sets (instead of one of the sets):

elements matched by plain text
elements matched by CSS selector
elements matched by XPath

chromedp.Click select the first element in the result. In the case of "#login-btn", the first element is not the button you expected.
If you want it to treat the selector as CSS selector only, it's recommended to specify the chromedp.ByQuery option explicitly.

In that case, could the doco be changed to this?

The BySearch (default) option enables querying for elements by plain text, CSS selector and XPath query, wrapping DOM.performSearch. The search result is the union of all the sets.

Sorry, English is not my native language. But I don't think the change make it more clear. FYI, the current doc for DOM.performSearch is

query
string
Plain text or query selector or XPath search query.

I think the important point to make in the documentation is that the function can return multiple results, which may be from one or more of the search types - whichever finds a match. That is why I suggested the use of 'and' rather than 'or' in my suggested text. I don't think this is made clear in the existing documentation.

As what I said before, English is not my native language. I will leave the decision to others.


[DevDocs] Improve documentation for hwnd_cache

We need to improve the documentation for
https://github.com/microsoft/PowerToys/blob/master/src/common/hwnd_data_cache.h
and
https://github.com/microsoft/PowerToys/blob/master/src/common/hwnd_data_cache.cpp
Also consider renaming some methods:
get_window_and_path -> this function filters out the current window and if not filtered, it returns the ancestor window handle and path
get_window -> this function is identical to get_window_and_path but returns only the ancestor handle

hwnd_data_cahce was removed, so this is obsolete


[Bug]: After log in and click on any link, occur exception

What happened?
After log in and click on any link, an exception pops up and back home page and I see an error reported on the console.
Uncaught (in promise) TypeError: (t || []).forEach is not a function
    at chunk-06619d26.be7ef06b.js:1:16850
    at d (chunk-vendors.c73d2b69.js:47:735548)
    at Generator._invoke (chunk-vendors.c73d2b69.js:47:736874)
    at Generator.next (chunk-vendors.c73d2b69.js:47:735977)
    at o (chunk-vendors.c73d2b69.js:1:339865)
    at s (chunk-vendors.c73d2b69.js:1:340068)

Affects Versions
0.4.0
What engines are you seeing the problem on?
AMS
How to reproduce
I have not found a way to consistently reproduce
Relevant log output
No response
Anything else

Code of Conduct

[X] I agree to follow this project's Code of Conduct

cc @huiyuanz

@huiyuanz This did not happen after I redeployed on k8s.
close.


Initializing Blockchain text....

Hi,
I'm guessing this is placeholder text but i will send it in anyway just in case.
During my first startup i got this:
https://1drv.ms/u/s!Ag9C_L5-r1CxhPN7VewzgNA2-R7lcA?e=8Vq9Wm
Is this a bug or will the funny text be replaced?

You are correct, it is placeholder text.

Ok then my bad sorry.

Sorry, let me clarify - it is placeholder text and it needs to be removed. For reference, this is the text:
Adipiscing Tristique Ullamcorper Ornare Consectetur


Eph ember support operation modes

Description:
Adds support for operation modes in the EphEmber climate device
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
[x] New dependencies are only imported inside functions that use them ([example][ex-import]).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

@frenck Docs updated on PR https://github.com/home-assistant/home-assistant.io/pull/5967

I'm not sure why the latest CI building is failing on pylint. I'm not getting any errors locally (using py3.6) and can't see any reference to errors in my module.

I've restarted that build job. Let's see what happens.


Update help

Description
This PR contains the following changes:

Updated the contextual help popup code to support the backend changes (see ENSWBSITES-930)

related articles are now different for different articles — huge improvement for code readability
video articles are now articles in their own right

Added the back and forward history buttons for the contextual help popup (see ENSWBSITES-1052). They became necessary because now each article has its own set of related articles (or may have none), so the user needs a way to return to the article he was viewing before he clicked on a related article.
Added the About Ensembl page (can be accessed by clicking on the "About Ensembl" image in the top right corner of the screen, see ENSWBSITES-931).

Deployment URL
http://update-help.review.ensembl.org/
Views affected

Contextual help popup
About Ensembl page

Left to be done in a subsequent PR

VideoArticle component to be extracted from the contextual help popup
TextArticle component (currently used on the About Ensembl page) to be re-used in the contextual help popup
Help app to be added, which will re-use both the VideoArticle and the TextArticle components

To code reviewers
If you try to run the code in this branch locally, notice that you will need to update the proxy in the webpack config to route all requests for /api/docs to http://update-help.review.ensembl.org/api/docs/

Wouldn't it be nice to add a similar styling to the menu links to show the active page like it does on the side menu?

Wouldn't it be nice to add a similar styling to the menu links to show the active page like it does on the side menu?

Yes. I was sure that I had checked with the XD and that is did not style the top menu, but I was wrong.

@msmanoj I've updated the top menu, renamed data field of the menu to items, deleted the unused files and fixed the typo in the comment. I'm reluctant to change the article and video type labels for articles to something like text-article and video-article. We might change them to text and video if we like this better than article and video; but since this will also require certain changes on the backend, I propose to leave it for a separate PR.

@msmanoj I've updated the top menu, renamed data field of the menu to items, deleted the unused files and fixed the typo in the comment. I'm reluctant to change the article and video type labels for articles to something like text-article and video-article. We might change them to text and video if we like this better than article and video; but since this will also require certain changes on the backend, I propose to leave it for a separate PR.

I think we should do this in a separate PR as we rely on backend changes.


US635070: Update to latest release of .NET build environment and runtime

A developer build has not yet been created for this branch. Click here to go ahead and create the build...

CI Build Link:
http://sou-jenkins2.swinfra.net:8080/job/FileAnalysisSuite/job/FileAnalysisSuite~adaptersdk-schema~US635070~CI


Adding density parameter to Marker.setBitmap.

This pull requests addresses the following issue: https://github.com/tangrams/tangram-es/issues/1925
Previously, bitmaps were not transferred to the Texture object with a density parameter, which lead to scaled up bitmaps on the map when the device density is higher than 1.

I kind of new that this PR would break the iOS build. I haven't set up a Mac to try it out yet, and I'm not too familiar with the iPhone since a few years ago. It looks like TGMarker.mm needs to add a density parameter to tangramInstance.setMarkerBitmap(...., density);
I am unclear of what density should be. Should it be UIScreen.scale?

I can check out the iOS implementation and see how to make that match.
For the Android changes, I'm now setting up a repro case to verify that the changes fix what we expect. @polar How were you creating the BitmapDrawables that ended up with the wrong size?

@matteblair Yes, I was creating the BitmapDrawables.

I just tried it, and sadly the density is not fixed.

Hey sorry, I dropped the ball on this PR - didn't want to delay the already-late release :\
I'll take another run at this to figure out the root issue and I expect we'll be able to put the change in a patch release - it shouldn't be a breaking change.

Hey thanks again for finding the cause of this issue, the fix should be applied in https://github.com/tangrams/tangram-es/pull/1976, let me know if you see other problems!


Consistent Lambda Layers versions across regions

Is your idea related to a problem? Please describe.
Currently, the lambda layers versions (the latest digit in the layer ARN) are not the same for each region:
arn:aws:lambda:eu-west-1:336392948345:layer:AWSDataWrangler-Python39:5 for eu-west-1
arn:aws:lambda:eu-west-2:336392948345:layer:AWSDataWrangler-Python39:3 for eu-west-2
Additionally, some versions are not available in all the regions:
arn:aws:lambda:ca-central-1:336392948345:layer:AWSDataWrangler-Python39:3 is the latest in ca-central-1
but arn:aws:lambda:us-east-1:336392948345:layer:AWSDataWrangler-Python39:3 is not available in us-east-1.
At the end it's not possible to implement a CDK stack relying on Data Wrangler layer and make it deployable in multiple regions.
Describe the solution you'd like
Consistent version numbers in all the regions where the Lambda Layer is available
P.S. Please do not attach files as it's considered a security risk. Add code snippets directly in the message body as much as possible.

In that case why are there gaps in the versioning? For example in eu-west-1, we have:
arn:aws:lambda:eu-west-1:336392948345:layer:AWSDataWrangler-Python38:7
and `arn:aws:lambda:eu-west-1:336392948345:layer:AWSDataWrangler-Python38:4``
but no arn:aws:lambda:eu-west-1:336392948345:layer:AWSDataWrangler-Python38:6
no arn:aws:lambda:eu-west-1:336392948345:layer:AWSDataWrangler-Python38:5

Hi @vgkowski, unfortunately due to how we manage these layers there was incident where layers had to be recreated (deleting current version and creating a new and incremented version). This should not happen in the future as we continue to increment.
Take a look at our Serverless Applicaction Repository (SAR) apps as well, as they are pinned explicitly to their wrangler semantic version which may be a better experience for you.


Jinja doesn't deal well with generators

Consider this simple code:
In [0]: from jinja2 import Template
    ...: def gen10():
    ...:     for i in range(10):
    ...:         yield i
    ...:
    ...: Template("{% for i in data %} {{i}}{% endfor %}\n" * 2).render(data = gen10())
    ...:
Out[0]: u' 0 1 2 3 4 5 6 7 8 9\n'

I was expecting my template to return<PHONE_NUMBER><PHONE_NUMBER>. The explanation is not difficult, but I think Jinja could detect generators and consume them recursively when the context is built.
This has to be done recursively because I could imagine something like:
def gen():
    for i in range(10):
        yield gen()

I don't think the would be a good idea. If I iterate over something in Jinja I don't expect any automated recursion in case one of the items is also a generator / iterable / list / ...

So could I do something like this instead:
Template("{% for i in data() %} {{i}}{% endfor %}\n" * 2).render(data = gen10)

Have you tried it? That should work.

Yeah it works :) So it could be the answer to my question actually...


Adding the ability to use DependencyInjection to add TelemetryInitializers

Fix Issue # .

[x] I ran Unit Tests locally.

For significant contributions please make sure you have completed the following items:

[ ] Changes in public surface reviewed

[ ] Design discussion issue #

[x] CHANGELOG.md updated with one line description of the fix, and a link to the original issue.

[x] The PR will trigger build, unit tests, and functional tests automatically. If your PR was submitted from fork - mention one of committers to initiate the build for you.
If you want to to re-run the build/tests, the easiest way is to simply Close and Re-Open this same PR. (Just click 'close pull request' followed by 'open pull request' buttons at the bottom of the PR)

Please follow [these] (https://github.com/Microsoft/ApplicationInsights-aspnetcore/blob/develop/Readme.md) instructions to build and test locally.

@cijothomas Please review


tests: pkg.sensors.TestPolicyListingWhileLoadUnload is flaky

Unfortunately we don't have artifacts since the workflow seems broken :(
https://github.com/cilium/tetragon/actions/runs/10416242015/job/28848114625?pr=2804#step:8:181
--- FAIL: TestPolicyListingWhileLoadUnload (10.00s)
    manager_test.go:351: adding policy
    manager_test.go:380: disabling policy
    delayed_sensor_test.go:70: unblocked failed: channel does not seem to be empty
    manager_test.go:382:
        	Error Trace:	/home/runner/actions-runner/_work/tetragon/tetragon/go/src/github.com/cilium/tetragon/pkg/sensors/manager_test.go:382
        	Error:      	Received unexpected error:
        	            	failed to unload tracing policy "test-policy": failed to unload all sensors from collection test-policy: TestDelayedSensor/Unload timeout when waiting for unblocking
        	Test:       	TestPolicyListingWhileLoadUnload
FAIL

oups duplicate of https://github.com/cilium/tetragon/issues/2576.


Scav junkbox craft with elite crafting skill allows for starting second craft without payment

https://streamable.com/62wquc

Originally written by chomp

Looking at the assembly, this seems like it may be a BSG issue.
ScavCaseView.UpdateView() passes base.Producer.ProducingItems.Any<KeyValuePair<string, GClass1921>>() as the isProducing parameter to the requirement view's Show method. This however seems to check whether ANY of the scav case crafts are running, not this particular one.
isProducing is then utilized in the following code, which guarantees that IsFulfilled is set to true:
int num = (isProducing ? requirement.IntCount : Mathf.Min(requirement.UserItemsCount, requirement.IntCount));
this.IsFulfilled = num >= requirement.IntCount && requirement.Error == null;

Originally written by DrakiaXYZ

Flagging as BSG issue and closing. If we ever get someone with elite crafting on live to try it out and confirm its a SPT issue we can reopen.

Originally written by chomp


Testing error

After add Turing command, I tested the installation, for which the following error occurred. Would it be possible to let me know how to fix this issue? Thanks.
WARNING: could not import MCMCChains.AbstractChains into Turing ERROR: LoadError: LoadError: UndefVarError: AbstractChains not defined Stacktrace:  [1] include at ./boot.jl:328 [inlined]  [2] include_relative(::Module, ::String) at ./loading.jl:1105  [3] include at ./Base.jl:31 [inlined]  [4] include(::String) at /home/ssing/.julia/packages/Turing/xFzfF/src/Turing.jl:1  [5] top-level scope at /home/ssing/.julia/packages/Turing/xFzfF/src/Turing.jl:124  [6] include at ./boot.jl:328 [inlined]  [7] include_relative(::Module, ::String) at ./loading.jl:1105  [8] include(::Module, ::String) at ./Base.jl:31  [9] top-level scope at none:2  [10] eval at ./boot.jl:330 [inlined]  [11] eval(::Expr) at ./client.jl:425  [12] top-level scope at ./none:3 in expression starting at /home/ssing/.julia/packages/Turing/xFzfF/src/utilities/Utilities.jl:6 in expression starting at /home/ssing/.julia/packages/Turing/xFzfF/src/Turing.jl:124 ERROR: LoadError: Failed to precompile Turing [fce5fe82-541a-59a6-adf8-730c64b5f9a0] to /home/ssing/.julia/compiled/v1.3/Turing/gm4QC_IaOP0.ji. Stacktrace:  [1] error(::String) at ./error.jl:33  [2] compilecache(::Base.PkgId, ::String) at ./loading.jl:1283  [3] _require(::Base.PkgId) at ./loading.jl:1024  [4] require(::Base.PkgId) at ./loading.jl:922  [5] require(::Module, ::Symbol) at ./loading.jl:917  [6] include at ./boot.jl:328 [inlined]  [7] include_relative(::Module, ::String) at ./loading.jl:1105  [8] include(::Module, ::String) at ./Base.jl:31  [9] include(::String) at ./client.jl:424  [10] top-level scope at none:6 in expression starting at /home/ssing/.julia/packages/Turing/xFzfF/test/runtests.jl:4 ERROR: Package Turing errored during testing

Please see issues #1211, #1195, or this Discourse post.

@cpfiffer Thanks for the links! Installed Turing :+1:


[MINOR] delete useless log object

Change Logs
Describe context and summary for this change. Highlight if any code was copied.
Impact
Describe any public API or user-facing feature change or any performance impact.
Risk level (write none, low medium or high below)
If medium or high, explain what verification was done to mitigate the risks.
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

Please also optimize the imports.

@codope done

Thanks! Can you also please update the PR description?

Thanks! Can you also please update the PR description?

@codope  done


itemsPerRow may result 0 (starting from rev25.2 ) preventing image to show

Describe the bug
Since rev 25.1 in some situations itemsPerRow gives 0 preventing the file to be shown.
That's because itemsPerRow  is now the return value of the function getItemsPerRow = function(horizontalSpace, itemWidth).
If horizontalSpace < itemWidth the return is = 0 and the image is not shown without any warning.
Logs
To Reproduce
In my case the pond element is a fieldset with all elements inside having a margin of 2 pixels each side.
When loading files, I have 1 file for any row.
Until 25.1 itemsPerRow was always = 1.
With 25.2 itemsPerRow is always = 0.
Looking at HTML generated by filepond with a debugger, in my case:
horizontalSpace = 940
itemWidth = 944 (940 + 4 pixels for margins)
And in this situation the function
var getItemsPerRow = function(horizontalSpace, itemWidth) {
// add one pixel leeway, when using percentages for item width total items can be 1.99 per row
return Math.floor((horizontalSpace + 1) / itemWidth);
};
gives zero.
Once understood this, I solved my problem adding a CSS that shorted the file items, ex :
.filepond--item { width: 920px;}
Previously (Until 25.1 ) no filepond--item css was needed.
Expected behavior
If the computation of itemsPerRow is always done with at least one file to show, then getItemsPerRow() would return always a value >= 1.
Or if zero is the correct return value of getItemsPerRow(), perhaps would be better try to draw anyway the image.
In this way the rendering would show space problem and programmer would add/correct the related css.
Stacktraces
Information about your project:

Crome 86.01.1 64 bits
MacOS 10 .14.6
FilePond version: problem started with 52.2

Additional context
greetings
Andrea

Fixing now!

Should be fixed in 4.26.0

Closed because inactive.


Support dynamic registration for native methods using --dynamic-register

Thanks for the pr, I'll look into it this week.

merged. thanks.


dev-mq-infra-instance.yaml requires 'SyncOptions: Replace'

Based on some of the CR fields that will change in the QueueManager's deployment YAML, not all of them are allowed to be changed on a running instance. For example, you cannot switch from a deployed SingleInstance Qmgr to a NativeHA Qmgr by updating/applying/patching the YAML.
By updating the ArgoCD Application to include the following options, it allows ArgoCD to replace the object outright and create it anew in the dev namespace:
  syncPolicy:
    automated:
      prune: true
      selfHeal: true
    syncOptions:
      - Replace=true

We don't need this everywhere, but it allows for cleaner deployment in the dev-related pipelines with the ability to manually navigate some of the setup for the staging and prod namespaces that would have less likelihood to switch some of those forbidden properties.

Resolved as of https://github.com/cloud-native-toolkit-demos/multi-tenancy-gitops-apps/commit/0404b143b34e9d57bf04a5e4af092efe4fbded9e for both dev and staging instances.


Test with Python 3.10

We're trying to use Python 3.10 in AiiDAlab and need to test if all dependencies support it.
optimade-client currently declares support only up to Python 3.9 . Is there any known reason that it doesn't support higher versions?
cc @unkcpz

@unkcpz or @CasperWA can you approve the workflows to run on this PR? Thanks!

Hi @danielhollas, I don't have permission to do that. It has to be @CasperWA. Maybe @CasperWA can give me write permission for this repo?

We're trying to use Python 3.10 in AiiDAlab and need to test if all dependencies support it.
optimade-client currently declares support only up to Python 3.9 . Is there any known reason that it doesn't support higher versions?

It's an older package. Python 3.10 is new. It's just the way of things.
Simply because support for higher/newer versions are not explicitly declared doesn't mean it shouldn't work.
It's easy to test locally by simply installing the package under a newer Python version and testing it out - no need to force the explicit metadata declaration simultaneously - just saying.

@CasperWA thank you for the info.
I am putting this as a Draft until we can manually verify that everything works with Python 3.10.
I don't personally use this package so I am not a good person for that.
@unkcpz I will link this PR in the Python 3.10 issue as a TODO item.

This repository has been extended to support Python 3.9-11, removing support for Python 3.7-8.


Fixed esm module imports not working

Solved issue #35

Bumped version to v1.1.3 as I consider this a rather major bug fix following semver.
Build process now has additional script fixup.js which:

Adds package.json to cjs and esm distributions which specifies the type (module or commonjs)
Recursively adds .js file extension to imports within the package to comply with proper import usage in node.js (this part can be improved, but works fine with current version of package and will support any foreseeable changes)

Thanks for your PR!
I tried to avoid using the manual change of the imports, as it's a bit hacky. Unfortunately, I didn't find any better solution for now. I'll look a bit more into it tomorrow and if I don't find anything, I'll merge this.

Thanks for your PR!
I tried to avoid using the manual change of the imports, as it's a bit hacky. Unfortunately, I didn't find any better solution for now. I'll look a bit more into it tomorrow and if I don't find anything, I'll merge this.

Yep. Only solution I see is if you have the tests run on the distributable instead of the source code, which could probably work, but would require you to build before testing always.

Thanks again! I just converted the fixup script to TS.


Support for LocalStoreServerCertificateSource Find by value/type

Can you please add support for finding the certificate in the local store by value and findtype, and not just by subject name?
It is beneficial to be able to locate the certificate by something other than subject name in most scenarios. Subject name lookup right now does a FindBySubjectName and the recommendation is to use FindBySubjectDistinguishedName because it is a more specific search https://learn.microsoft.com/en-us/dotnet/api/system.security.cryptography.x509certificates.x509findtype?view=net-8.0

I agree.
Would you like to contribute this functionality?

Add 'FindType' to ServerLocalCertificateStoreConfigurationOptions  with default value of FindBySubjectName
Add 'FindType' as another input parameter for  LocalStoreServerCertificateSource  and implement the needed logic
Pass FindType in ConfigureLocalStoreServerCertificateSelection to LocalStoreServerCertificateSource  https://github.com/mataness/Kestrel.HttpsCertificateSelection/blob/main/Kestrel.HttpsCertificateSelection/HttpsConnectionAdapterOptionsExtensions.cs

Yes, if you can grant the permission I can have a PR

Permissions for what? fork the repo and submit a PR

https://github.com/mataness/Kestrel.HttpsCertificateSelection/pull/4

v2 released to public feed


Collision Prevention with distance sensor

Replacement for  #10998
Changes:

map distance_sensor data to obstacle_distance message (Collision Prevention now supports range data from distance sensor only, companion computer only, both companion and distance sensor)
start cm8jl65 distance sensor with an orientation set by a parameter
add failsafe if the range data are stale. Go first into loiter and then switch off the collision prevention such that the user can still fly position control.

FYI @mhkabir  - I already flew it today and I have few things to fix.
When this is done we need to update a couple of docs. Just some notes for me ... off the top of my head:

Distance Sensors (Rangefinders) - top paragraph can now link to collision prevention doc
Collision Prevention > PX4 Distance Sensor

Needs to clarify setup - ie any hardware, software parameters that need to be set, including supported physical orientation of sensor.
Confirm that supporting collision prevention does not disable anything else, unless other use has incompatible sensor orientation.
Confirm how many sensors can be used - ie whether multiple rangefinders are supported on the vehicle and for what purpose, and if multiple are supported, which are used for collision sensors.
Confirm modes collision prevention using distance sensor works in.

@mhkabir Flight with a distance sensor where really god. Log is coming (Flight Review is down).
I still need to test data from companion + distance sensor

Flight test went well!
https://logs.px4.io/plot_app?log=604ebb24-07eb-426f-8a81-3b53c2707fc8

FYI @dagar we got a bunch of [load_mon] wq:lp_default low on stack! (292 bytes left) and [load_mon] wq:I2C1 low on stack! (276 bytes left) messages, and this shouldn't affect those subsystems.

FYI @dagar we got a bunch of [load_mon] wq:lp_default low on stack! (292 bytes left) and [load_mon] wq:I2C1 low on stack! (276 bytes left) messages, and this shouldn't affect those subsystems.

That should already be resolved in master. https://github.com/PX4/Firmware/commit/26e041c43c1cf5345a863089c5e0d9ca58cef9be

@MaEtUgR would you mind reviewing it? We had good flights with it already

@dagar with these changes the mc_pos_control is low on stack. See log https://review.px4.io/plot_app?log=4ad9105e-55f1-4f2e-9253-8ea41d2372de

Out of flash on fmu v2

@dagar Flash is still an issue on fmu v2

@bkueng could you please review again? I addressed the comments. Thank you


Improve pagination response

Description
Need to add more information when we response to a pagination request.
Each pagination response should include a total field and not just the actual dataset:
{
  dataset: Array<T>
  total: number
}

Screenshots
No response
Additional information
No response

Closed https://github.com/origranot/reduced.to/pull/408


Failing Build (just some missing includes)

I went to give a try to build this project (mostly just to poke around with it), and following the standard auto-tools process (./autogen.sh, ./configure, make), I ran into several build errors (fixable just by adding four includes).
I tried to open a PR for the fixes, but github seems to be struggling on my browser, so attached is the appropriate patch for use with git apply!

Managed to get github to wake up from its sleepiness. PR (#5) opened, sorry for the noise!


Viewer should be able to stream in .rrd files whilst they're still being written to

We can log to files in a streaming fashion already today:
import rerun as rr
import time

rr.init("rerun_example_points3d_simple")
rr.save("/tmp/stream.rrd")

for i in range(0, 100000):
    rr.set_time_sequence("frame", i)
    rr.log("points", rr.TextLog(f"frame {i}"))
    time.sleep(0.2)

That's great, but we cannot visualize file recordings in a streaming fashion:
$ rerun /tmp/stream.rrd  # will only show what it is there _right now_

I just tried this with Rerun 0.18.2. The viewer can read from /tmp/stream.rrd while the script logs to it, but it only reads the data that was present when the viewer opened and doesn't continue to read new data as it comes in

I did a quick experiment to see if I could use flock https://linux.die.net/man/2/flock to check if another process has write-access to a file, but at least on my Mac the call to flock(file, libc::LOCK_EX | libc::LOCK_NB) succeeds even if another Rerun process has the file open for writing.

Quick summary of the issue:

message Decoder that is used as an Iterator when reading the .rrd file is unaware of the fact file may still be written to, hence when it sees EoF from the reader, it concludes there's no more data and the Iterator will return None and reading and decoding stops.

Had a quick chat with Emil on the path forward and we discussed following steps:

add retrying for file-based reader (we can either make Decoder aware of the file or we can create a RetryableBuffReader and pass it to the Decoder. Retrying is just dumb loop with sleep and we retry indefinitely.
instead of sleep + retry, we rely on notify, so retrying is slightly smarter
we add a new message type that signifies "rrd file end" and is written by the FileSink and then we make Decoder aware of the message and we retry if we get EoF from the reader, but we still haven't seen the end file message. Once we see end of file message / marker, decoder iterator is done. Retry strategy in this case can be:

stop after first time you see end of file marker message
block and read forever
try to see if there's more data after end of file marker message
(these are useful for cases when the user concats multiple files together manually)

Fixed as part of PR #7475.


Logs have useless fields

This Improvement request (usability, performance, tech debt, etc.) affects these Traffic Control components:

Traffic Ops

Current behavior:
Each changelog entry currently includes the following fields:
export interface Log {
	id: number;
	lastUpdated: Date;
	level: "APICHANGE";
	message: string;
	ticketNum: null;
	user: string;
}

New behavior:
IDs have no purpose for changelog entries because they are immutable by nature. "level" used to have meaning when changes could be made through either the now-removed Traffic Ops UI or the API, where it would show that distinction. But that no distinction no longer exists (and was always of questionable merit at any rate IMO, so I'm not sure historical preservation is a good reason to keep it around). ticketNum has not only no known purpose, but also there's no way to populate it, so it's just always null. There's no reasonable way I can think of to make use of that field. Finally, lastUpdated is not necessarily "useless", it's just a bad name IMO. A changelog entry is never "updated", it is only created.
With all this in mind, I propose a new format for /logs output:
export interface Log {
	/**
	 * This should be an RFC3339 timestamp instead
	 * of the custom format used today for "lastUpdated"
	 */
	date: Date;
	message: string;
	user: string;
}

This isn't a big enough change to merit a blueprint, IMO, but I could do that too.

Also this is somewhat unrelated, but the response from /logs/newcount doesn't use proper camelCase. It returns an object with the property newLogcount, but that should be newLogCount. Or honestly the response could just be a number, don't need any kind of label on it like that.


Observing Compilation Error while compiling netopeer2 inside docker

Hi Team,
We successfully compiled Libyang, Sysrepo, Libnetconf2, and Netopeer2 outside of Docker without any issues. However, when attempting to compile Netopeer2 inside Docker, we encountered an error. Below is the specific error message:
/usr/local/lib/libnetconf2.so: undefined reference to<EMAIL_ADDRESS>/usr/local/lib/libnetconf2.so: undefined reference to<EMAIL_ADDRESS>/usr/local/lib/libnetconf2.so: undefined reference to<EMAIL_ADDRESS>/usr/lib/x86_64-linux-gnu/libssh.so: undefined reference to<EMAIL_ADDRESS>/usr/local/lib/libnetconf2.so: undefined reference to<EMAIL_ADDRESS>/usr/lib/x86_64-linux-gnu/libssh.so: undefined reference to<EMAIL_ADDRESS>/usr/local/lib/libnetconf2.so: undefined reference to<EMAIL_ADDRESS>collect2: error: ld returned 1 exit status
CMakeFiles/netopeer2-server.dir/build.make:135: recipe for target 'netopeer2-server' failed
make[2]: *** [netopeer2-server] Error 1
CMakeFiles/Makefile2:180: recipe for target 'CMakeFiles/netopeer2-server.dir/all' failed
make[1]: *** [CMakeFiles/netopeer2-server.dir/all] Error 2
Makefile:145: recipe for target 'all' failed
make: *** [all] Error 2

Libcurl Version inside docker :-
pkg-config --modversion libcurl
7.38.0

Please share your inputs on this.
Thanks,
Aravind.

Hi Michal,
After successfully updating the OpenSSL version to 3.0.0, I encountered an issue with the authentication method when running sysrepocfg -Evim -m ietf-netconf-server inside Docker. Netopeer2Server and Netopeer2Cli are failing to connect due to authentication errors. Is there a way to update the authentication method to use password authentication within the Docker environment?
sysrepocfg -Evim -m ietf-netconf-server
sysrepocfg error: Exec failed (No such file or directory)
For more details you may try to increase the verbosity up to "-v3".

Netopeer2Server Log :-
[INF]: LN: Accepted a connection on <IP_ADDRESS>:10830 from <IP_ADDRESS>:57476.
[INF]: LN: Received an SSH message "request-service" of subtype "ssh-userauth".
[INF]: LN: Received an SSH message "request-auth" of subtype "none".
[INF]: LN: Received an SSH message "request-auth" of subtype "interactive".
[INF]: LN: User "root" is not allowed to authenticate using a password.
[ERR]: LN: Unable to get root's credentials.
[INF]: LN: Failed user "root" authentication attempt (#1).
[2024/04/24 11:39:43.872709, 1] ssh_packet_disconnect_callback:  Received SSH_MSG_DISCONNECT: 11:Bye Bye
[ERR]: LN: Communication SSH socket unexpectedly closed.

Netopeer2Cli log :-
 connect --host <IP_ADDRESS> --port 10830 --login root
nc ERROR: Unable to authenticate to the remote server (all attempts via supported authentication methods failed).
cmd_connect: Connecting to the <IP_ADDRESS>:10830 as user "root" failed.

Please share your inputs on this.
Thanks.

Understand what the command sysrepocfg -Evim -m ietf-netconf-server does exactly (sysrepoctfg -h) and use a different text editor, one you actually have installed.

0.10.4 version we are using.
pkg-config --modversion libssh
0.10.4

Is the version the same for both the client and the server? It may also be an issue with OpenSSL, perhaps. I cannot really help you, it is a dependency problem.

Great! I've successfully connected to the server using SSH and TLS callhome methods. Is there a way to change the default authentication method to password? After compilation, the authentication method defaults to keyboard-interactive, but I prefer password authentication as the default method.
Please Share your input on this.
Thanks,
Aravind

Look at example_configuration/ssh_listen.xml for examples of configuration each authentication method. Note that some implementations depend on how libnetconf2 was compiled.


master-post-init prevents cluster from entering Running state

I'm trying to use an adapted version of the master-post-init.sh script as an initialization action. When I do so, the part of the code that checks for the cluster to be in RUNNING state appears to prevent the cluster from actually entering that state, leading to a timeout eventually. I have a feeling that this is because all initialization actions must be complete before the status can change, so effectively we get a deadlock. Is there a way to have a script that is guaranteed to execute when the cluster is RUNNING?

No, you will need to use master-post-init.sh for this, or submit some job (Pig shell script, for example) that will run when cluster is in running state.


Two choices for integration testing

This isn't done, but I'm looking for feedback on using either cucumber or rspec for integration tests, which would aid in BDD development and be run before releases.   If  rspec is chosen, we can configure it to skip the 'integration' folder unless requested.
This PR includes both options for comparison.
For #144
One difficulty here is I am getting several different answers for the tests, so what's the acceptance criteria?  Just absence of errors?
5,845 soccer fields
204 soccer fields
4,125 soccer fields

@mattlindsey Isn't RSpec enough for testing library code? Why do we need cucumber it doesn't seem to bring additional value to what we can already do with rspec testing an Agent.

@alchaplinsky Yeah.  I'll redo this proposal to only include the rspec file, exclude the 'integrations' folder when running rspec, and add a short note in the README suggesting contributors utilize the specs directly.

@alchaplinsky.  Is this better?  I think they're useful.

@mattlindsey I think integration tests for concept like Agent is a good idea. But we need to mock requests to external APIs. I don't think that anyone would want to pay for tokens used during test runs.

Personally I wanted these to aid in development, but I can just keep them for myself. Existing tests already do the mocking.

@alchaplinsky Sorry, I meant to tag you on the last comment.
I added a line in the README to clarify the intent of the directory and files.

Ok, got it. So this is a type of test that actually runs against real APIs and makes sure that code works with real integrations. Thus we skip running them every time when we run a test suite and we can enable them by setting the INTEGRATION env variable. Is that right? I'd probably name that variable something more descriptive like:
INTEGRATION_TESTING = true
INTEGRATION_TESTS_ENABLED = true
Otherwise, it is not entirely clear what INTEGRATION stands for.

INTEGRATION_TESTS_ENABLED = true

@alchaplinsky Yeah. That's better. Done

Closing this to avoid too many open PRs but leaving issue #144 open.


Update index.js

two reason:
1、server.timeout after 2 minutes
2、socket will emit 'timeout' event too

Hi! I don't think you actually understand the purpose of the module. This timeout has no relation to the socket timeout. This is a different mechanism.
Once the socket timeout occurs, the request is destroyed and you cannot respond. This module emits a timeout event on the request, which is in addition to a socket timeout and allows you to respond and stop your logic.
I'm sorry, but we wont be accepting this pull request. I can re evaluate id you fully describe what your change is trying to achieve, provide documentation changes, provide test changes, etc.

Thanks for your explanation, I should manage socket time out myself.

Right. It's also important to note that a socket can belong to multiple requests, due to http pipelining. This module is designed to add a timeout that only spans a single request (also, the conditions that actually cause the timeout are different as well, where this module is for time to response headers and the socket is just raw socket idling, below the http level).


grpc-io: upgrade to latest version - v0.13.0.

What is changed and how it works?

Issue Number: Close #xxx

What's Changed:
Upgrade the grpc-io lib to the latest version - v0.13.0.

Meanwhile, the format of `cargo.toml` is re-arranged and become tidier than the older version.

Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

Upgrade the grpc-io lib to the latest version - v0.13.0.

You need to compare the performance between the current and this pr.

Thx for reminding.
Yep, I will add and supply the performance testing results between master and this pr.

/test

/test retry=7

/test threads=4 retry=7

Maybe this pr could be paused, found two problems to this pr:

According to testing records, based on TPCC workloads, the newly upgraded version v0.13.0 caused the raising on CPU.usage, about ~12% to ~15% based on the results.
The uts still have some abnormal cases which cannot be fixed directly, throwing SIGABRT signal and aborting all consequential cases. Basically, I've tried to replace the unsafe tokio::runtime with the safe RuntimeWrapper, which will safely releasing the Runtime object when drop. But some uts still are hung by the abnormal signal SIGABRT.

PS: By testing with tpcc-1000 warehouses (CPU: 16 cores) , the performance results can be reviewed in the following table. And the throughput didn't differ a lot, but the CPU.usage on upgrading v0.13.0 is much higher than the nightly version.

QPS part: (higher is better)

Concurrency
upgrading v0.13.0
Master (nightly version)

50 threads
20841
21272

100 threads
21552
21640

150 threads
21378
21531

200 threads
20586
21104

CPU part: (higher means more cost on CPU.usage)

Concurrency
upgrading v0.13.0
Master (nightly version)

50 threads
1421%
1283%

100 threads
1530%
1347%

150 threads
1610%
1422%

200 threads
1645%
1464%


🎁 Jib CLI > Add the windows install process with chocolatey 🎁

Hi, I'm a Jib fan.
Most of the time, i'm running Jib from maven and gradle , but recently I had to use it from command line. To help windows colleagues adopt it, I created a Chocolatey package to make the install process even easier under windows.
Hopefully you'll like it.
Thanks a lot for Jib, it helps us save so much time !
https://twitter.com/rastadidi/status/1380255793648312321
Kind Regards,
Adrien

CLA ✔

@adriens oh, awesome! Basically I don't really know what chocolatey is or have a way to test this, but sounds like this can be really helpful to the users.
For the CLA check, I also verified you are signed. I think you just need to leave a comment here with @googlebot I signed it! per instructions.

@googlebot I signed it!

@chanseokoh : thanks a lot for your very kind feedback.
Jib is an amazing tool, making us so productive 👏 🚀

Thanks! Merging now.


Wrong loss on discriminator

Hey! I found that when optimizing the discriminator its loss is defined as:
d_loss = 1-real_out+fake_out
which translates into D(G(z))+1-D(x). Shouldn't it be:
d_loss = 1+real_out-fake_out
so it matches the discriminator definition D(x)+1-D(G(z))?

@manurare no, we want real_out to be close 1, and fake_out to close 0


introduce error msg on type mismatch

pxt-blockly has a few mechanisms for types (e.g. the shape of holes are determined based on those types), however, that might not be suitable for what we need. I'm assuming that however, this would be easy to implement.

will be tricky when using variables. I think it should be fine to throw an error when the types don't match in those cases. we want them to learn by doing.
another option is to also check the variable type as the user adds to the code


Setting default workflow permissions

Waiting on it being added to the github provider:
https://github.com/integrations/terraform-provider-github/issues/1266

Wrong repo


Casting to ChromeCast not possible, feature missing.

What version of WebTorrent Desktop? (See the 'About WebTorrent' menu)
Version 0.3.3 (0-a5366c3)
What operating system and version?
OS X El Capitan 10.11.4 (15E65)
What did you do?
Play an item while it was finished/still downloading
What did you expect to happen?
The ChromeCast button to appear
What actually happened?
No ChromeCast button, not able to stream it to my device.

Chromecast devices are supported. This must be a temporary fluke on your network. Try restarting your Chromecast and/or the WebTorrent app.
Does casting to the Chromecast from another device work?

It's working on the very same device: same Mac, same connection, same router, same chrome cast device (NexusTV) not rebooting anything, it works like a charm. Worked before with WebTorrent Desktop (same version), is not working now.

Can you restart WebTorrent Desktop and see if it works?

Just did. Nothing happened, no way to stream it, feature is still missing, as you can see from the screenshot. Also tried with Chrome on my Mac and iPhone, both working perfectly.

@digisign76 The chromecast icon only shows up when a Chromecast is detected on your network. So this means that detection is failing.
What kind of Chromecast do you have? Can you share the software/firmware version information?
Instructions for where to find the version info can be found here under "How do I know my firmware version?"

Thanks for your reply. Point is the ChromeCast device was active, working and discoverable by all the others devices, but not by web torrent Desktop client. The ChromeCast in my case is the Google Nexus TV (Asus made, but Google specs), and works pretty well, better than the ChromeCast dongle I had before. Anyway, later yesterday, with no reason at all, web torrent started to work again properly and I was able to stream on my ChromeCast. My guess is that, while downloading few items, especially if the bandwidth get seriously high, Webtorrent client stops showing you the Cast Button. You can still play your download in progress but not cast them, not even if they are completed.
I might be wrong but everything was back to normal not after many restart, but only when all the downloads were completed.

@digisign76 Thanks for sharing your theory about the cause of the missing Chromecast icon. I think you might be right. See https://github.com/feross/webtorrent-desktop/issues/362
@mafintosh -- does that seem reasonable to you? Could having a saturated network connection cause Chromecasts on the local network to not be detected?

@feross By the way, the network was not saturated the I had that issue: I have tried with other devices and Chromecast was totally discoverable and available... it actually worked.

I had similar issue so I switched chromecast and my computer from 5Ghz to 2.4Ghz restarted webtorrent and saw the icon.


Tryto merge

This is what i'm trying to do.. Merge something from the Baseline with a role.

Sorry for not having this catched up. In the meantime we have changed the project quite a lot and added a lot of new stuff.

Nevermind, i just used it as an example


Is there a way to set the default style for a sheet

For example, if one wants to make a sheet borderless. Currently one may be able to  achieve this by using the  sheet.SetRowStyle with the TotalRows as the end argument. But that is a big loop, theTotalRows is 1048576.

Thanks for your issue. You can use the SetColStyle function to set styles for all cells in a worksheet, for example:
err := f.SetColStyle("Sheet1", "A:XFD", 0)

If you want to hide the gride line for a worksheet, please use the SetSheetView function like this:
disable := false
err := f.SetSheetView("Sheet1", -1, &excelize.ViewOptions{
    ShowGridLines: &disable,
})

very much appreciated!


Error: INSTALLATION FAILED: create: failed to create: Request entity too large: limit is 3145728

Hey, I downloaded the helm package. and tried to install it, but I'm consistently having two issues:

Error: INSTALLATION FAILED: create: failed to create: Secret "sh.helm.release.v1.billing.v1" is invalid: data: Too long: must have at most 1048576 bytes
Error: INSTALLATION FAILED: create: failed to create: Request entity too large: limit is 3145728

Can anyone please elaborate how to fix this problem? In the past, I did helm install Lago he v1.0.0 successfully, but when I just tried to install from this repo I'm having this issue

for some reason this hack is working.
helm package .
helm template billing lago-1.2.1.tgz  --set frontUrl=http://billing.neira.localhost:8080,apiUrl=http://billing-api.neira.localhost:8080 | kubectl apply -f -
I don't understand why regular install command helm install billing . --set frontUrl=<myURL>,apiUrl=<myURL> is causing  "INSTALLATION FAILED: create: failed to create:the Secret "sh.helm.release.v1.billing.v1" is invalid: data: Too long: must have at most 1048576 bytes" error.
If anyone knows, please elaborate on that, I'm not super skilled in Helm, so please be patient if I'm lacking some basic knowledge.

I think it's because dependencies got included, let me check this!

thanks for getting back quickly!
Let me know if you need some more info. I did notice that after either helm install or helm package commands, those two .tar files are being generated. Screenshot:

I did try to remove those .tgz files and then launch again those 2 commands, but then I'm getting error that dependencies were not found or something. So it's quote tricky situation.
For context, I'm using Docker Desktop's Kubernetes.

Any updates?

Is there any updates on this? Thanks!

I just added an .helmignore file, if you're able to pull an try again 🙏

Just checked. Working perfectly! 🤌
Thanks @jdenquin !


Crashes when folder selected

As described above. If the preview window is opened and I navigate through the files as soon as a directory is selected it crashes. Here's a screenshot:

Other than that, the plugin is great!

Oops. I think the issue is that you’re not on neovim nightly. I am using some new APIs.

I will try to make it more backwards compatible tomorrow.

@b0o Yeah no biggie, I can press escape but it just makes for a smoother experience. Happy to wait :-)

Should be fixed, mind giving it a try?

@b0o This works a treat now, great work thanks :-)