[chip-test] chip_sw_usb_suspend

Test point name
chip_sw_usb_suspend
Host side component
SystemVerilog
OpenTitanTool infrastructure implemented
Unknown
Contact person
@a-will
Checklist
Please fill out this checklist as items are completed. Link to PRs and issues as appropriate.

[ ] Check if existing test covers most or all of this testpoint (if so, either extend said test to cover all points, or skip the next 3 checkboxes)
[ ] Device-side (C) component developed
[ ] Bazel build rules developed
[ ] Host-side component developed
[ ] HJSON test plan updated with test name (so it shows up in the dashboard)
[ ] Test added to dvsim nightly regression (and passing at time of checking)

Deferring to V3 / M7 as P2 as discussed in today's triage meeting and with @alees24.

We've discussed in the triage meeting that we should prioritize this chip level test as it covers an relevant use case @alees24 . Would you mind taking a look at this please?

This is a very old test point description; a more comprehensive set of 7 tests exists in daft PR #23200 which have been run on all targets including ES SiVal. The main problem with SiVal/ES/FPGA tests is each party having appropriate time out values for the other party (USB host software and device).
The chip sim is an easier, more predictable target not subject to the vagaries of non-real time operating systems and variable workloads. I've added an estimate of 2 days to finish off the tests for ASIC-level chip sim only; they have previously run there a number of times, but may have suffered bit-rot.

Thanks for this updated information, @alees24. If you could indeed prioritize finishing the DV TLTs that cover this testpoint, that would be great! Please feel free to update the test point definition while you work on this.


revise Admin UI tooltips

https://www.figma.com/file/67kIKXv9t1bPDURaQ7ZOYH/obsrv_admin-ui-help-messaging?node-id=4%3A0

Review & correct tooltips (if needed)
Provide "learn more" links to docs

Figma mockups draw directly from https://docs.google.com/spreadsheets/d/1C1J8mIv8sFSJ_SzyQPVDybp-E2EVyNsN9i4l38gmQao/edit#gid=1556052224
Relates to #6227

Closing, as the scope is too broad & this is now an implicit subtask in any new documentation of Admin UI updates.
This has recently been underway in #6777 #6707 and #4736.


Terms & Privacy policy verification

In wallet we have terms and privacy policy:

@Pow1404  for verification:

If those are up-to-date.
Maybe both regulations should be outside wallet (easier to edit) and wallet should only link to them? (can be separate improvement issue)

The terms and conditions are tagged outside the wallet when you are in the process of creating a profile , this issue is tested and can be closed


Improper copyright section of CONTACT page

What is the bug*
Copyright section of CONTACT page is not synchronize.
Expected behavior
All the pages must have same copyright section
Screenshots

Improper copyright section of CONTACT page which is not synchronize with other

Assign me this issue I can improve it and make proper.

@moinmulla Thank you for your interest
But this issue is alreday present and has been assigned to someone else
Show your suport by stargazing the repo 🌟


Can't Mask Bias Weights For Dense Layer

I've noticed that bias weights for a dense layer are not eligible for masking. The reason for this is that in the _prunables method, the boolean returned is (len(weights.shape) > 1 and ....
The bias weights in a dense layer have a shape (<# of nodes>,), which has a length equal to 1. This causes the boolean to return False.
I would have thought that bias weights should also be prunable. Do you purposely want to be certain they're not prunable? If not, are there any "gotcha's" I should be aware of, if I just change the > to a >= ?

AFAIK the papers that this code was inspired from do not talk about pruning bias weights (perhaps because pruning bias weights potentially only saves one add vs pruning other weights potentially saving many multiplies). That said, experimentation is a good thing. :-)
The logic in LotteryTicketPruner._prunable() is:
        return len(weights.shape) > 1 and 'keras.layers' in type(layer).__module__ and type(
            layer).__name__ in self.prunable_layer_names

Changing this to ... len(weights.shape) >= 1 ... should do what you want since iterate_prunables() should then start including those bias weights in the pruning process.


Enable CongestionControl::Block if QoS is reliable.

With DDS the only combination of QoS which makes a Publisher to block the publications in case of network congestion is with Reliability=RELIABLE and History=KEEP_ALL.
In order to closely mimic this behaviour rmw_zenoh is setting Z_CONGESTION_CONTROL_BLOCK only for this combinaison of QoS.
However, in case of a Publisher for a "latched topic", it will use such combination of QoS:
Reliability=RELIABLE, History=KEEP_LAST(1) and Durability=TRANSIENT_LOCAL.
This leads rmw_zenoh to set Z_CONGESTION_CONTROL_DROP for the Publisher. And in case of very congested network the single publication might be dropped. That's what happens in the case discussed here:
https://github.com/ZettaScaleLabs/rmw_zenoh/issues/38#issuecomment-2484892922
This PR solves this problem setting Z_CONGESTION_CONTROL_BLOCK for any Publisher with QoS Reliability=RELIABLE, whatever the other QoS are.

This looks good to me.
@clalancette could you please also review it and advise if we shall merge it to dev/1.0.0 branch now or if we shall wait the merge of dev/1.0.0 to rolling

@JEnoch Thank you for adding comments and a detailed description. I always admire the ability you make things clearer :laughing:
I will also test the PR with nav2 to verify it again.

I will also test the PR with nav2 to verify it again.

Okay, it works properly after the changes.


Direct Style Shift

This is WIP PR where we translate shift, reset, and resume to direct style in JS...
Overall, the performance stays the same; only tree explore looks faster. Strangely, resume non-tail get's MUCH slower (factor 2-3). Maybe the higher-order functions weren't that bad before for inlining by the JIT.

Conclusion: we save some JS stack, but it might not be worth it. (cc @phischu)

We could revisit this, once tuples landed in JS: https://tc39.es/proposal-record-tuple/tutorial/
Maybe this will be more obvious to the JITter

Also, regardless of the result, we should cherry-pick resumptions as statement in CPS.

Resume has been cherry picked in #708


Swipe back gesture does not work if page pushed to stack using getRootNavs()

Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
Please do not submit support requests or "How to" questions here. Instead, please use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

If you have a view that contains tabs and one tab page has a button that pushes a page to the stack using
this.navCtrl.push('apage'); you can use the swipe back gesture to go back to the previous page in the stack
If however, the button uses this, so that the tabs are not visible when the new page is pushed, then the swipe back gesture does not work.
this.app.getRootNavs()[0].push('apage');
Expected behavior:

The swipe back gesture should work
Steps to reproduce:

Related code:

Create a tab view with more than one page, and another page called 'apage'
On a page in the tab view add 2 buttons
One button should run this.app.getRootNavs()[0].push('apage'); and the other button this.navCtrl.push('apage');

Note you need this at the top:
    public navCtrl: NavController,
    public app: App

insert short code snippets here

Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
cli packages: (/Users/tarang/Desktop/mobile-ios/node_modules)

    @ionic/cli-utils  : 1.19.0
    ionic (Ionic CLI) : 3.19.0

global packages:

    cordova (Cordova CLI) : 7.1.0

local packages:

    @ionic/app-scripts : 3.1.0
    Cordova Platforms  : ios 4.5.3
    Ionic Framework    : ionic-angular 3.9.2

System:

    ios-deploy : 1.9.2
    ios-sim    : 6.0.0
    Node       : v8.7.0
    npm        : 5.4.2
    OS         : macOS High Sierra
    Xcode      : Xcode 9.0 Build version 9A235

Environment Variables:

    ANDROID_HOME : not set

Misc:

    backend : legacy

Hello! Thank you for opening an issue with us! Would you be able to provide a sample application via GitHub that demonstrates the issue you are having? Thanks for using Ionic!

Hi Ken I have created a sample Ionic project via GitHub and supporting notes. Please take a look at https://github.com/akhatri/ionic-test-case

+1 have same issue

this same issue +1


Mobile - article - bottom bits

Acceptance criteria

"Updated" with pencil button is stacked vertically with "Did this page help you?"
Within "Did this page help you", "Did this page help you" and button underneath are now stacked vertically

Update: left a request in a review

Update: Melissa review updated, check again
something about a button component


Add support for stream block

Overview
This pull request adds a StreamBlockFactory, which supersedes the existing StreamFieldFactory functionality, adding supported for nested streams. Support for current functionality is maintained, and the new functionality uses the same syntax as StreamFieldFactory for instantiating a stream value's inner blocks (e.g. MyPageFactory(body__0__struct__title="foobar").
With the new functionality, StreamBlockFactory subclasses can be defined and used as follows:
class StreamBlockFactoryA(StreamBlockFactory):
    struct = factory.SubFactory(MyStructBlock)
    text = factory.SubFactory(MyTextBlock)

    class Meta:
        model = StreamBlockA

class MyPageFactory(PageFactory):
    body = StreamFieldFactory(StreamBlockFactoryA)

    class Meta:
        model = MyPage

StreamBlockFactories can be used as sub-factories of StructBlockFactory, ListBlockFactory and StreamBlockFactory, for example:
class StreamBlockFactoryB(StreamBlockFactory):
    inner_stream = factory.SubFactory(StreamBlockFactoryA)

    class Meta:
        model = StreamBlockB

Breaking changes
This PR significantly changes the way that StreamFieldFactory works under the hood. The changes provide a deeper integration with factory_boy's object generation logic, resulting in the requirement that all BlockFactory values passed to an old-style StreamFieldFactory (dictionary-style) declaration must be wrapped in SubFactories (this is covered in more detail in the changes to the readme). I propose a new major version release (3.0)

Thanks for your initial review @bcdickinson ! I've marked the items that are no longer relevant as resolved.

I'm encountering an issue testing this locally when calling factory classes that contain StreamFieldFactory declarations and not passing any kwargs (e.g. just MyPageFactory(parent=some_page) with a body__0="my_struct_block" kwarg or some such). It seems like the step builder code can't handle an absence of block data and I get this (truncated stack trace to only show factory code frames):
  File "/venv/lib/python3.8/site-packages/factory/base.py", line 40, in __call__
    return cls.create(**kwargs)
  File "/venv/lib/python3.8/site-packages/factory/base.py", line 528, in create
    return cls._generate(enums.CREATE_STRATEGY, kwargs)
  File "/venv/lib/python3.8/site-packages/factory/django.py", line 117, in _generate
    return super()._generate(strategy, params)
  File "/venv/lib/python3.8/site-packages/factory/base.py", line 465, in _generate
    return step.build()
  File "/venv/lib/python3.8/site-packages/factory/builder.py", line 258, in build
    step.resolve(pre)
  File "/venv/lib/python3.8/site-packages/factory/builder.py", line 199, in resolve
    self.attributes[field_name] = getattr(self.stub, field_name)
  File "/venv/lib/python3.8/site-packages/factory/builder.py", line 344, in __getattr__
    value = value.evaluate_pre(
  File "/venv/lib/python3.8/site-packages/factory/declarations.py", line 48, in evaluate_pre
    return self.evaluate(instance, step, context)
  File "/venv/lib/python3.8/site-packages/wagtail_factories/blocks.py", line 113, in evaluate
    return self.stream_block_factory(**extra)
  File "/venv/lib/python3.8/site-packages/factory/base.py", line 40, in __call__
    return cls.create(**kwargs)
  File "/venv/lib/python3.8/site-packages/factory/base.py", line 528, in create
    return cls._generate(enums.CREATE_STRATEGY, kwargs)
  File "/venv/lib/python3.8/site-packages/wagtail_factories/blocks.py", line 44, in _generate
    step = cls._builder_class(cls._meta, params, strategy)
  File "/venv/lib/python3.8/site-packages/wagtail_factories/builder.py", line 46, in __init__
    indexed_block_names, extra_declarations = self.get_block_declarations(
  File "/venv/lib/python3.8/site-packages/wagtail_factories/builder.py", line 100, in get_block_declarations
    self.validate_block_indexes(indexed_block_names, factory_meta)
  File "/venv/lib/python3.8/site-packages/wagtail_factories/builder.py", line 108, in validate_block_indexes
    for declared, expected in zip_longest(indexes, range(max(indexes) + 1)):
ValueError: max() arg is an empty sequence```

As discussed on our call just now, I think in this instance it would be sensible to just handle this by leaving the body empty and not throwing an error. Also, as we noted, there's currently no way of defining default data for the `StreamBlockFactory`, so I'll raise a new issue for that.

@bcdickinson good catch, the same issue also existed with requesting the default value for a StreamBlockFactory in a StreamBlockFactory, e.g.
MyPageFactory(body__0="inner_stream")

Fixed in https://github.com/wagtail/wagtail-factories/pull/55/commits/e893a8a47f421437239c288ce1d493b87000d8d8

Nice, I'd say this is there now! Happy for this to be merged and released.


Change copyDirectory to copyDir

This PR is part of the normalization of our API. It normalizes our copyDirectory() call with that of our mixins.

Coverage decreased (-18.85%) to 61.64% when pulling e7c5c0dcae528a1613d33645672fe2c99506e9e6 on change-copyDirectory-to-copyDir into cf23d91713241e43199dd43802b256c771c3a976 on master.


Attack speed "more" qualifier calculated incorrectly

Currently, "more attack speed" quantities are being added together before being combined with "increased attack speed". "more" qualifiers should be multiplied together, not summed.
Source: http://pathofexile.gamepedia.com/Stat#Flat.2C_additive_and_multiplicative_stats
Example:
76% increased attack speed (from tree)
10% more attack speed (from DW)
93% more attack speed (L19 Multistrike)
Current: (1 + 0.76) * (1 + 1.03) = 3.5728
Correct: (1 + 0.76) * (1 + 0.10) * (1 + 0.93) = 3.73648
Relevant source file: Compute.cs, Lines 627-647
I'm not sure if this is globally miscalculated in the app for other types. I have not checked.

I'm working on a PR, but I'm not very familiar with the source code yet and don't primarily code in C#, so it'll probably be very rough around the edges.

Should be a pretty simple change. You add 1 to the value and multiply then
together. Currently they are summed, so you would just need to change the
one line. (Two in this case because less is also calculated wrongly)
On Sep 26, 2016 5:30 PM, "Jason Robinson"<EMAIL_ADDRESS>wrote:

I'm working on a PR, but I'm not very familiar with the source code yet
and don't primarily code in C#, so it'll probably be very rough around the
edges.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/EmmittJ/PoESkillTree/issues/380#issuecomment-249735901,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHIbbUsudXNkqmDw-ieTEZ56m6xjLSshks5quGOrgaJpZM4KHIze
.


General feedback on v0.5

Thank you for the wonderful tool. Its definitely a breath of fresh air for those of us on macos with i3 envy!
I spent 2 days with it (v0.5beta) and figured I detail my experience in case its useful (take it with a pinch of salt given my inexperience with the tool):
Good

docs are really nice and to the point
installation is a breeze  (esp not having to deal with SIP shenanigans)
I love the minimal menu-bar

Needs-work

configurations are hit and miss. Reload would sometime take but say missed half the time. If lucky then there would be a parser error. Otherwise nothing but silent fail.
focus stopped working randomly  (alt-h works but alt-l mysteriously stops working)
Re-mappings did not always work. E.g. I can remap alt-enter. But exact same command with alt-t refuses to work. I double checked there are no duplicates or anything. alt-t keeps taking me to workspace=T. It would be helpful if I could make it dump out current key mappings for debugging.
Relaunching causes quite a bit of mess. I suppose I can use the after-startup command to clean things up a bit
alt-0 sometime refuses to go to workspace 0. I'd say 3 out of 5 times. I have to use the menubar to go there -- sometime even that does not work (then I click on the app that I know is in space 0 on the dock)
docs mention v0.6 (e.g callback), but I could not find that version on the release page.

Once again, thank you for your hard work.

Did some digging. Looks like my default browser wezterm was blocking while spawning a new window. So that was messing up aerospace. So I will retract (2) and (3) 😌

Thanks for the feedback!

Reload would sometime take but say missed half the time. If lucky then there would be a parser error. Otherwise nothing but silent fail.

I'd be happy if you could provide a reproducer. Normally, if something fails during config parsing, a terminal should open up with the error message:

Relaunching causes quite a bit of mess

Well, yes. Right now, AeroSpace doesn't preserve the layout during relaunch. Please open an issue, if you think that's a problem. Normally, you shouldn't be relaunching the AeroSpace

alt-0 sometime refuses to go to workspace 0. I'd say 3 out of 5 times. I have to use the menubar to go there -- sometime even that does not work (then I click on the app that I know is in space 0 on the dock)

That's weird. Is it specifically about workspace 0? Can you please rename the workspace 0 and see if the problem happens with the new lexically smallest workspace?
Are you sure that you don't have any other app that intercepts alt-0 shortcut?

docs mention v0.6 (e.g callback), but I could not find that version on the release page.

0.6.0-Beta version is in development. It's in main branch, it's not yet released.

Otherwise nothing but silent fail.

It would be nice if the major mode switches would flash a quick toast as feedback. Especially in the beginning this is very useful to get to know the thing after wich you should be able to disable it.


[Discussion] What audio parameters can reduce the memory usage of the GPU

My graphics card has only 6g of RAM, and I can recognize up to 8 minutes of audio; I want to recognize audio for a longer period of time. Can I reduce the audio quality for a longer time?
I also want to know what factors are related to the memory occupied by the spleeter.
I use -b param，it cant Extended time。
spleeter separate $1 -o output -b 32k -c mp3

Hi @m986883511,
Reducing the audio quality of your input won't change the memory footprint of spleeter, as anyway signals will be resampled to 44.1kHz (in stereo).
The things that can reduce the memory footprint are the following:

the duration of the audio: You can then chunk your audio file into small pieces so that the memory footprint is under your memory limit. Note that you can still process several files with a single call of spleeter (CLI) without affecting much the memory footprint
the model:

models separating to less stems have a smaller footprint (2stems has a lower footprint than 4stems that has a lower footprint than 5stems)
models that process up to higher frequencies ("stems-16kHz" models) have a larger memory footprint. This is actually handled by the F parameter that set the highest frequency bin up to which the model will process. Note that it has to be a multiple of 64. F is linearly related to the highest frequency in Hz and F=1024 corresponds to 11025Hz. If you're not interested in high frequencies, you may reduce F.

Hi @romi1502 ,
is it possible for me to get the model summary or the number of parameters to calculate the memory footprint?


Add button to copy last month's budgets

Add a new button which copies last month's budgets over to the current month's budgets. If actively selecting January as the month, then copy over budgets from December.

I think there should also be a button to clear the current month's budgets, as well as a button to clear future month's budgets. So to clear all budgets, essentially go to January, clear future month's budgets (February–December), and then clear the current month (January). So when copying over a new year, you could navigate to January, copy over last December's budgets, and then clear all future budgets. If you wish to start all budgets from scratch, then clear future month's budgets and clear January's budgets.


ImportError: cannot import name 'patch_compiled_autograd' from 'unsloth_zoo.patching_utils'

Hello, I have an error while trying to use LLama Vision. Could you please check it?
Reproduction:
!pip install unsloth
!pip uninstall unsloth -y && pip install --upgrade --no-cache-dir --no-deps git+https://github.com/unslothai/unsloth.git
!pip install unsloth-zoo
from unsloth import FastVisionModel
Error:
ImportError: cannot import name 'patch_compiled_autograd' from 'unsloth_zoo.patching_utils'
Version
Python: 3.9.20
unsloth: 2024.12.4
unsloth_zoo: 2024.11.5

I've tried the solution (updating both unsloth-zoo and unsloth) given in #1313 . However, the problem still remains.


Clean up Pass by reference & const

Description
Some functions pass various Godot objects by reference, others not. Here's a sizeof comparison, showing which objects are worth passing by reference. By reference passes a uint64_t 8 byte pointer.
Type, sizeof(type)

Transform3D: 48
AABB: 24
Variant: 24
Color: 16
Vector3: 12
Vector2, Vector2i: 8
Ref, String, Array, Dict, RID: 8

There are also many uses of const Ref<Terrain3DStorage> & in parameters, when the ref is already a reference pointer.
Tasks:

Standardize on everything >4 bytes passing by const & for 32-bit systems
Add const to parameters where missing and appropriate

Started on a PR but then accidentally pushed it into main, so WIP at 24008a6137c38cd00156f3efbda9626413eae5a9

@mihe I noticed all throughout Godot-jolt, you pass const RID&. But RID is already 8 bytes, and it copies over the same RID, so the use of & seems redundant. Is there another reason you use it?

The generated (C++) code for RID scared me enough to stop passing it by-value. I haven't looked at the generated assembly code for it, but seeing as how the copy constructor has to go through the bindings means that it'll never get inlined/optimized away, so struck me as quite expensive. Its move constructor is also not defined in the header file, so you're sort of relying on LTO/LTCG to optimize out that function call.
But for the most part I try to keep my codebase mostly Godot-like (with a lot of arbitrary exceptions) and Godot passes RID by-ref everywhere, presumably because of 32-bit platforms.

Thanks @mihe


MM-20578 Migrate 'components/admin_console/system_user_detail' module…

Summary
Migrate 'components/admin_console/system_user_detail' module and associated tests to TypeScript
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/13494

Hello @pikami,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @jfrerich @emilyacook

@pikami Could you resolve the conflicts on this PR for us? After that I can have another look.

@devinbinnie conflicts resolved.

@pikami Could you please resolve the merge conflict? Then the PR is ready to go.

@hanzei Merge conflict resolved.

Awesome, thanks @pikami :+1:


Configure debug broker support in release mode

We currently support MSAL trusting debug brokers; however, we need to be sure that people don't set this in error for release builds that we'll be shipped to production.

When the build-type is RELEASE and trust debug brokers is enabled throw a runtime exception.

Implementation
We need to check that the application trusting debug brokers is allowed to do so, hence the BrokerValidator hosts a variable that keeps a record of applications with the ability to trust debug brokers.

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@008e815). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             dev    #1651   +/-   ##
======================================
  Coverage       ?   14.57%
  Complexity     ?      321
======================================
  Files          ?      165
  Lines          ?     7436
  Branches       ?      702
======================================
  Hits           ?     1084
  Misses         ?     6181
  Partials       ?      171

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 008e815...0e7188f. Read the comment docs.

So what is the plan here? It seems like the only way this is now actually configurable is to first declare trust via BuildConfig, right? So that means the builds that are in Maven central are not actually ever going to be able to trust debug brokers because they are not in debug mode and also don't have trust declared in Build Config. I think that is fine...
My question is how is this debug broker support useful now? I suspect that the plan would be to publish two versions of common going forward to at least the internal maven? One that has debug support declared via BuildConfig and one that doesn't. Is that the correct understanding? If that's the case, I'm not sure if we even need the setShouldTrustDebugBroker method in BrokerValidator at all. I'm not seeing any utility since the functioning is really controlled by build config.

@shahzaibj  as discussed, we are now checking the application's package name and signature hash before allowing it to trust debug brokers.
For a start we allow only MSAL_TEST_APP and MSAL_AUTOMATION_APP to do so.


Recommend changing README to log in as user vagrant, not p4tc

This line of the README recommends logging in as user p4tc to the VM:
ssh -p 2222<EMAIL_ADDRESS># password: p4tc

However, that user account has no files in its home directory, and the files in /home/vagrant are not readable as user p4tc without using sudo, so it seems kind of useless to log in as that user.  I would recommend changing the above lines of the README to say this instead:
ssh -p 2222<EMAIL_ADDRESS># password: vagrant

Hey @jafingerhut thank you for the catch
We merged a PR (https://github.com/p4tc-dev/p4tc-tutorial-pub/pull/4) implemeting your suggestion

Closing as completed.  Thanks.


APIGOV-16534 - add redaction library

Please make sure I did not miss calling the necessary redaction where ever needed

Maybe we should also ask Xenon team to have a look?

Nice


File not found

I build a new monorepo by @microsoft/rush and add a new vue project that added vite-plugin-vue-devtools. But when I started this project, the plugin didn't work and gave me this exception: "GET http://localhost:5173/@id/virtual:vue-devtools-path:overlay.js net::ERR_ABORTED 404 (Not Found)" .
I have attached the code. Use rushx dev to start the vue project.
common.zip

I will track it.


How to set the font color of the current date to custom color?

That's not supported.
It's a small enough change if you fork the library.

thank you


Connection to server is erroring. Shutting down server

Issue Type: Bug
PHP Fatal error:  Allowed memory size of<PHONE_NUMBER> bytes exhausted (tried to allocate 20480 bytes) in /Users/helloworld/.vscode/extensions/felixfbecker.php-intellisense-2.3.13/vendor/felixfbecker/language-server/src/Index/Index.php on line 379
[Error - 4:19:22 PM] Connection to server is erroring. Shutting down server.
Extension version: 2.3.13
VS Code version: Code 1.38.1 (b37e54c98e1a74ba89e03073e5a3761284e3ffb0, 2019-09-11T13:31:32.854Z)
OS version: Darwin x64 18.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8257U CPU @ 1.40GHz (8 x 1400)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
1, 2, 2

Memory (System)
16.00GB (4.09GB free)

Process Argv

Screen Reader
no

VM
0%

Can you share the file that causes this error when opened? Or is it different files every time?
It appears to be consuming 4 gigs of memory.

Similar error, no file open. Happens when I reopen or I add folders to a fresh window.
In terminal output:
PHP Fatal error:  Allowed memory size of 2097152 bytes exhausted (tried to allocate 69632 bytes) in /Users/camwyn/.vscode/extensions/felixfbecker.php-intellisense-2.3.14/vendor/jetbrains/phpstorm-stubs/PhpStormStubsMap.php on line 5670

Language server exited with exit code 255```

Extension version: 2.3.14
VSCode version 1.41.1
OS: Mac Catalina 10.15.2 (19C57)

PHP Fatal error:  Allowed memory size of 2097152 bytes

Did you try increasing memory to GB-level? Only 2 MBs would not be enought for anything.

For me that appears to have fixed it 🤞
Worth mentioning that the value was set to 0M when I check the setting?


Documentation Setup

This is the epic card for getting our documentation up to snuff. Actionable sub cards are referenced in this description. We can break this down further if any part seems too large or seems like it could be worked on simultaneously.
Operator Guide - for setting up a production instance of VinylDNS #39

[ ] Install Guide
[ ] Configuration Guide https://github.com/vinyldns/vinyldns/issues/27
[ ]  ~Monitoring Guide~

User Guide - for end users of VinylDNS #40

[x] Portal Guide
[x] API Guide (existing API documentation)
[x] Tooling (existing tooling page)
[x] FAQ

Developer Guide - for developers working on VinylDNS #42

[ ] Local install guide

System Design - overview of how VinylDNS is structured. Will be linked to from other guides. #41

[x] Architecture
[x] Components
[x] Integration

RFC Guide  - how to go about proposing new features/significant changes #43

[x]  What is an RFC
[x] VinylDNS RFC process
[x] Example RFC

Contributing - how to contribute to VinylDNS #44
Roadmap: short term and long term plans for VinylDNS #45

Can the documentation include a description of what problem(s) VinylDNS is solving?  How is the current way of doing things so bad?

Completed.


chore: move MySQL datasource validation tests to a separate file

Description

A separate file had been created earlier to segregate all the datasource validation tests for MySQL but only few tests had been moved to it. This PR moves the remaining tests to this file. This should help de-clutter the main test file which has become too lengthy to read.

PR fixes following issue(s)
Fixes #26830
Type of change

Chore (housekeeping or task changes that don't impact user perception)

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[x] JUnit

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test

/ok-to-test


Invalid escape sequence \o (IDFGH-3564)

This issue is related to the last two posts in https://github.com/espressif/esp-idf/issues/4398, but this new issue has not been filed yet... So, I do this now...
Environment

Development Kit:      ESP32-DevKitC
Module or chip used:  ESP32
IDF version: ESP-IDF v4.2-dev-1905-g625bd5eb1 (tested with an older ESP-IDF v4.2-dev-1660 as well)
Build System: CMake|idf.py
Compiler version: xtensa-esp32-elf-gcc (crosstool-NG esp-2020r1) 8.2.0
Operating System:     Windows 10 Pro 64bit ESP Command Prompt
Using an IDE?: VS Code, but build is in ESP command prompt
Power Supply:         USB

Problem Description
Building a trivial project just works fine. Then I tried to add the 3rd party component https://github.com/bbulkow/FastLED-idf as follows
idf_component_register(
    SRCS "main.cc" "functionblocks.cc" "plcmanager.cc"
    REQUIRES "FastLED-idf" <---- I just added this line
    EMBED_FILES "../../devtools/fbexec.data")

Of course, I cloned the FastLED-idf repository into {project_dir}\components
Expected Behavior
Successful build
Actual Behavior
c:\repos\labathome\firmware>idf build
Executing action: all (aliases: build)
Running cmake in directory c:\repos\labathome\firmware\build
Executing "cmake -G Ninja -DPYTHON_DEPS_CHECKED=1 -DESP_PLATFORM=1 --warn-uninitialized -DIDF_TARGET=esp32 -DCCACHE_ENABLE=1 c:\repos\labathome\firmware"...
Warn about uninitialized values.
-- Found Git: C:/Program Files/Git/cmd/git.exe (found version "2.26.0.windows.1")
-- ccache will be used for faster recompilation
-- The C compiler identification is GNU 8.2.0
-- The CXX compiler identification is GNU 8.2.0
-- The ASM compiler identification is GNU
-- Found assembler: C:/opt/espressif/tools/xtensa-esp32-elf/esp-2020r1-8.2.0/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc.exe
-- Check for working C compiler: C:/opt/espressif/tools/xtensa-esp32-elf/esp-2020r1-8.2.0/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc.exe
-- Check for working C compiler: C:/opt/espressif/tools/xtensa-esp32-elf/esp-2020r1-8.2.0/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc.exe -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: C:/opt/espressif/tools/xtensa-esp32-elf/esp-2020r1-8.2.0/xtensa-esp32-elf/bin/xtensa-esp32-elf-g++.exe
-- Check for working CXX compiler: C:/opt/espressif/tools/xtensa-esp32-elf/esp-2020r1-8.2.0/xtensa-esp32-elf/bin/xtensa-esp32-elf-g++.exe -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Building ESP-IDF components for target esp32
-- Found Git: C:/Program Files/Git/cmd/git.exe (found version "2.26.0.windows.1")
CMake Error at C:/opt/esp-idf/tools/cmake/component.cmake:221 (message):
  CMake Warning (dev) at build_properties.temp.cmake:2 (set):

    Syntax error in cmake code at

      C:/repos/labathome/firmware/build/build_properties.temp.cmake:2

    when parsing string

      C:\opt\espressif\python_env\idf4.2_py3.8_env\Scripts\python.exe

    Invalid escape sequence \o

    Policy CMP0010 is not set: Bad variable reference syntax is an error.  Run
    "cmake --help-policy CMP0010" for policy details.  Use the cmake_policy
    command to set the policy and suppress this warning.

  Call Stack (most recent call first):

    C:/opt/esp-idf/tools/cmake/scripts/component_get_requirements.cmake:1 (include)

  This warning is for project developers.  Use -Wno-dev to suppress it.

  CMake Error at
  C:/opt/espressif/tools/cmake/3.16.4/share/cmake-3.16/Modules/ExternalProject.cmake:1029
  (define_property):

    define_property command is not scriptable

  Call Stack (most recent call first):

    C:/opt/esp-idf/tools/cmake/kconfig.cmake:1 (include)
    C:/opt/esp-idf/tools/cmake/idf.cmake:41 (include)
    C:/opt/esp-idf/tools/cmake/project.cmake:7 (include)
    C:/repos/labathome/firmware/components/FastLED-idf/CMakeLists.txt:4 (include)
    C:/opt/esp-idf/tools/cmake/scripts/component_get_requirements.cmake:60 (include)
    C:/opt/esp-idf/tools/cmake/scripts/component_get_requirements.cmake:76 (__component_get_requirements)

Call Stack (most recent call first):
  C:/opt/esp-idf/tools/cmake/build.cmake:405 (__component_get_requirements)
  C:/opt/esp-idf/tools/cmake/project.cmake:395 (idf_build_process)
  CMakeLists.txt:3 (project)

-- Configuring incomplete, errors occurred!
See also "C:/repos/labathome/firmware/build/CMakeFiles/CMakeOutput.log".
cmake failed with exit code 1

Steps to reproduce

clone simple project
REQUIRES 3rd party lib

Thanks for reporting, we will look into.

Same issue on latest master after adding a line to official modbus example: mb_slave\main\CMakeLists.txt
add_definitions(-DINCLUDE_xTaskGetHandle)

I had the same problem, the fix above worked for me.

Same problem, the fix by @keelung-yang worked. Thank you!

Hi, sorry about this. An official fix has been merged internally.

@keelung-yang this workaround worked for me. Thank you!

@keelung-yang Thanks for the workaround, this fixed it for me too :)

If you are using VS Code and this issue persists even after patching the idf.py file, try changing all backslashes \ to forward slashes / in Settings/ESP-IDF/Python Bin Path Win (idf.pythonBinPathWin):

"idf.pythonBinPathWin": "c:/espressif-tools-latest/python_env/idf4.3_py3.8_env/Scripts/python.exe"

This resolved the issue for me entirely.

@mvacha
This resolved the issue for me entirely.
thanks！

If you are using VS Code and this issue persists even after patching the idf.py file, try changing all backslashes \ to forward slashes / in Settings/ESP-IDF/Python Bin Path Win (idf.pythonBinPathWin):

"idf.pythonBinPathWin": "c:/espressif-tools-latest/python_env/idf4.3_py3.8_env/Scripts/python.exe"

This resolved the issue for me entirely.

This resolves mine problem in ESP-IDF extension in vs code. Thanks!

If you are using VS Code and this issue persists even after patching the idf.py file, try changing all backslashes \ to forward slashes / in Settings/ESP-IDF/Python Bin Path Win (idf.pythonBinPathWin):
"idf.pythonBinPathWin": "c:/espressif-tools-latest/python_env/idf4.3_py3.8_env/Scripts/python.exe"

This resolved the issue for me entirely.

Worked for me as well, using Windows and VS code ESP-IDF extension. Thanks!

Thanks for reporting and updates, sorry for slow turnaround, fix is available at https://github.com/espressif/esp-idf/commit/f5b1f43a2b0d88493224ed3587bf67f8c52eaa5b, feel free to reopen.


RVC colab disconnect

am I the only one that get disconnected when I try to run the colab version of RVC ? I try on different pc and same problem :/
It's disconnect me from the google colab during the step2b on harvest mode or during step 3 after <10 min.

same here, I started using RVC on Colab 3 days ago and I could do nice session with 200 epochs. now 20 epochs and cut off. And only 20 voica samples. So colab is not usable anymore :(

help PLZ !!!

There's problem between github and colab i think, 🤔
for solution i don't recommended using github to colab,
for now i use hugingface,
check this
https://colab.research.google.com/drive/1sC3PcoVQCt-LEtptH3v5SX7xXer2YdEO?authuser=1#scrollTo=Cj_6ldpEZ8w2

Same.. when disconnect all runtime reset all file gone. And you must re-install all. Please fix it

I think that it's a not RVC notebook but Google Colab issue.
I tried to use a simplified notebook with the same result (disconnect after about 10min).

it's google colab issue, i don't know the details, but i think they are trying to ban rvc 🤔

Facing the same issue, i think it has something to do with their updated policy on not allowing the following anymore:

Glad I am not the only one with this issue. It seems like there aren't many people complaining about it yet which makes me wonder if I am just too poor to buy a Google collab package.

I tried to run training without GUI e.g.
!mkdir /content/Retrieval-based-Voice-Conversion-WebUI/logs/myvoice
python3 /content/Retrieval-based-Voice-Conversion-WebUI/trainset_preprocess_pipeline_print.py /content/drive/MyDrive/dataset/myvoice 40000 2 /content/Retrieval-based-Voice-Conversion-WebUI/logs/myvoice False

The result is the same - disconnect after 5-10min. So it's a not the policy restriction. It looks like a repository ban.

exactly, i found this since day one,
i tried to upload all repo to my drive including hubert, pre-trained files etc, and extract it to colab (before i do training), and surprisingly it run without any issue, you can try it,

Same problem.
Tried different accounts and different colab notebooks.
Disconnected between 10 and 20 epochs

I copied the repisotory on Google disk, then downloaded assets.
The first attempt was to run from Google disk and it failed with an error like #441.
So I run T4-session and copied RVC folder with assets to /content/RVC and finally the train was completed successfully.
My Colab has only 4 steps
# 1. Attach Google Drive
from google.colab import drive
drive.mount("/content/drive")

# 2. Copy RVC on the session machine
!cp -r /content/drive/MyDrive/RVC /content/```

# 3. Install dependencies
!nvidia-smi

!apt-get -y install build-essential python3-dev ffmpeg
!pip3 install --upgrade setuptools wheel
!pip3 install --upgrade pip
!pip3 install faiss-cpu==1.7.2 fairseq gradio==3.14.0 ffmpeg ffmpeg-python praat-parselmouth pyworld numpy==1.23.5 numba==0.56.4 librosa==0.9.2
%cd /content/RVC
!pip3 install -r requirements.txt

# 4. Run UI
%cd /content/RVC
!python3 infer-web.py --colab --pycmd python3

Worked fine but last 3 weeks it started to disconnect after 10 minutes of work.

At the bottom of the above image: creating deep fakes.

I think that it's a not RVC notebook but Google Colab issue. I tried to use a simplified notebook with the same result (disconnect after about 10min).

Can it be solved by buying the pro version?

Facing the same issue, i think it has something to do with their updated policy on not allowing the following anymore:

What is the point of colab if it is not going to run a program?

I bought a pro subscription, and yet.... I ggt an error. I lost a great deal of tokens to get 0 results.

Will nobody help?! I have the exact same problem! Been a week now and nothing's changed. I don't have enough data to install the local version. PLEASE HELP!!!

I found this notebook on Kaggle, and it doesn't use WebUI, so it doesn't face disconnection issues on Google Colab or Kaggle.

I tried to run training without GUI e.g.
!mkdir /content/Retrieval-based-Voice-Conversion-WebUI/logs/myvoice
python3 /content/Retrieval-based-Voice-Conversion-WebUI/trainset_preprocess_pipeline_print.py /content/drive/MyDrive/dataset/myvoice 40000 2 /content/Retrieval-based-Voice-Conversion-WebUI/logs/myvoice False

The result is the same - disconnect after 5-10min. So it's a not the policy restriction. It looks like a repository ban.

Because the webui code is still present it has to be fully stripped

I found this notebook on Kaggle, and it doesn't use WebUI, so it doesn't face disconnection issues on Google Colab or Kaggle.

Is there an English version? Chrome doesnt translate the full page for some reason


Possibility to put libcoro into a mainloop?

I'd just like to write a libcoro wrapper around some boost::asio stuff, however I couldn't figure out how to use sync_wait into a mainloop like that one of boost asio? Any idea?

I think you'll want to use coro::io_scheduler and use the manual drive functions
/**
     * Given a thread_strategy_t::manual this function should be called at regular intervals to
     * process events that are ready.  If a using thread_strategy_t::spawn this is run continously
     * on a dedicated background thread and does not need to be manually invoked.
     * @param timeout If no events are ready how long should the function wait for events to be ready?
     *                Passing zero (default) for the timeout will check for any events that are
     *                ready now, and then return.  This could be zero events.  Passing -1 means block
     *                indefinitely until an event happens.
     * @param return The number of tasks currently executing or waiting to execute.
     */
    auto process_events(std::chrono::milliseconds timeout = std::chrono::milliseconds{0}) -> std::size_t;

It won't be possible to integrate coro::sync_wait into an event loop since its blocking.

Nope, I can't use io_scheduler here, since it conflicts with some boost stuff.

Define "conflicts"?

Closing since I'm not sure what the problem is


Keep Raw on Camera?

How to keep RAW pictures on my Camera using gphoto2?
Is there a way to keep RAW's on the Camera, like here:
https://github.com/andreknieriem/photobooth/wiki/FAQ#how-to-keep-pictures-on-my-camera-using-gphoto2

Not implemented currently.
Maybe you want to add it?
We use python gphoto2 https://github.com/jim-easterbrook/python-gphoto2/tree/main/examples to connect the camera.
The logic is here: https://github.com/photobooth-app/photobooth-app/blob/main/photobooth/services/backends/gphoto2.py

additional information on this topic:
https://github.com/photobooth-app/photobooth-app/discussions/9#discussioncomment-8367046

Hi,
I improved the gphoto2 backend to

keep raw on camera and
avoid stuck if shot with raw+jpeg.

As a playground I used following script which works fine on my canon 1100D.
To keep raw on camera you need to set the capture target to something permanent. For canon it is memory card.
After capture it will flush the eventqueue now that avoids getting stuck gphoto on second capture.
import gphoto2 as gp
import os

c = gp.Camera()
c.init()
# defined events http://www.gphoto.org/doc/api/gphoto2-camera_8h.html#a438ab2ac60ad5d5ced30e4201476800b and
# https://github.com/gphoto/libgphoto2/blob/846ae9f2199eb8bcbcb6d8bfe47fc961a033519a/gphoto2/gphoto2-camera.h#L103
event_texts = {}
for name in ('GP_EVENT_UNKNOWN', 'GP_EVENT_TIMEOUT', 'GP_EVENT_FILE_ADDED',
                'GP_EVENT_FOLDER_ADDED', 'GP_EVENT_CAPTURE_COMPLETE', 'GP_EVENT_FILE_CHANGED'):
    event_texts[getattr(gp, name)] = name

print(event_texts)

# change capturetarget
cfg = c.get_config()
capturetarget_cfg = cfg.get_child_by_name('capturetarget')
print(capturetarget_cfg)
capturetarget = capturetarget_cfg.get_value()
print(capturetarget)
capturetarget_cfg.set_value('Internal RAM')
#capturetarget_cfg.set_value("Memory card")
capturetarget = capturetarget_cfg.get_value()
print(capturetarget)
gp.check_result(gp.gp_camera_set_config(c, cfg))

for i in range(2):
	captured_files:list[tuple[str:str]]=[]
	print("Capture #%1d" % i)

	data = c.capture(gp.GP_CAPTURE_IMAGE)
	captured_files.append((data.folder,data.name))

	#empty the event queue
	typ,data = c.wait_for_event(200)
	while typ != gp.GP_EVENT_TIMEOUT:

		print(f"Event: {event_texts.get(typ,f'unknown event index: {typ}')}, data: {data}")

		if typ == gp.GP_EVENT_FILE_ADDED:
			captured_files.append((data.folder,data.name))

		#try to grab another event
		typ,data = c.wait_for_event(10) # timeout in ms

	print("thats all for now...")

	print(captured_files)

	file_to_download=None
	for captured_file in captured_files:
		_, file_extension = os.path.splitext(captured_file[1])
		print(file_extension)
		if str(file_extension).lower() in (".jpg", ".jpeg"):
			print("ok, this file to download!")
			file_to_download=captured_file

			break

	if file_to_download is None:
		raise RuntimeError("no capture or no jpeg captured! shooting in raw-only mode?")

	f2 = c.file_get(captured_file[0],captured_file[1],gp.GP_FILE_TYPE_NORMAL)
	dest2 = os.path.join("/tmp",captured_file[1])
	f2.save(dest2)
	print(f"saved {dest2}")

c.exit()


[BEAM-9547] Add indexing tests

We pass close to half of the the Loc tsts, but it's simpler to just skip them all until Loc supports __setitem__.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Python PreCommit


fix some problem in self distillation example in HPO after transition…

… to newAPI
Signed-off-by: Wang, Yi A<EMAIL_ADDRESS>Type of Change
feature or bug fix or documentation or validation or others
API changed or not
Description
detail description
JIRA ticket: xxx
Expected Behavior & Potential Risk
the expected behavior that triggered by this PR
How has this PR been tested?
how to reproduce the test (including hardware information)
Dependency Change?
any library dependency introduced or removed

@xin3he please review


Release Airnode 0.11.1

Patch of Airnode 0.11.1 contains:

Airnode attemps to fail the request when the sponsor wallet is invalid#1749
fix: links to new vitepress docs #1731

Done. Created https://github.com/api3dao/airnode/issues/1773 for testing.


Need help in beginning

How exactly do i try this out/ run this.

Hi,
In order to run archimate-js lib, you need a web app. Try with archimate-js-demo.
Step 1: download archimate-js and open this project in your IDE. Install the project dependencies via npm install.
Step 2: download archimate-js-demo and open this project in your IDE. Edit package.json file and replace "archimate-js": "0.0.3" with your local archimate-js project path (eg. "archimate-js": "file:../archimate-js")
Step 3: install archimate-js-demo project dependencies via npm install.
Step 4: run the demo npm run start
You can now modify archimate-js files and see changes via the demo app.

Thanks (sorry for the late reply :p )


On-screen keyboard characters / and \ are swapped

Fixed with https://github.com/alex0809/netsurf-base-reMarkable/commit/0a44646f4f8fe1a36ebc7167b0f90383853c63df


Choose an initial testing framework (jest?)

Need to choose a testing framework for this and figure out how to get things working.

Not proceeding further with testing for now as Typescript does enough for this.


version 1.1.4 release

We are going to release v1.1.4 within a few days. The operations I did for this are:
git checkout -b v1.1.4-release
./scripts/set_version.sh 1.1.4
git add . -v
git commit -m'version 1.1.4'
git push seratch v1.1.4-release

I will do a releases in these steps tomorrow: https://github.com/slackapi/java-slack-sdk/blob/main/.github/maintainers_guide.md#releasing
Changelog
The following is the release note for this patch release.
Changes

[slack-api-client] #554 #553 Add new admin.conversations.* APIs - Thanks @seratch
[slack-app-backend] #539 Fix #538 by adding api_app_id to SlashCommandPayload - Thanks @seratch
[slack-api-client] #540 Fix a typo in ChatUnfurlRequest docs - Thanks @markrmullan
[slack-api-client] Update data classes/constants for Web APIs and Audit Logs API - Thanks @seratch
[slack-api-client] #556 Add original_connected_channel_id to entry.channel in Audit Logs response - Thanks @seratch

All issues/pull requests: https://github.com/slackapi/java-slack-sdk/milestone/11?closed=1
All changes: https://github.com/slackapi/java-slack-sdk/compare/v1.1.3...v1.1.4

(I'm going to cut off v1.1.4 tag. The diff is https://github.com/slackapi/java-slack-sdk/compare/v1.1.3...main )
Category (place an x in each of the [ ])

[x] bolt (Bolt for Java)
[ ] bolt-{sub modules} (Bolt for Java - optional modules)
[x] slack-api-client (Slack API Clients)
[x] slack-api-model (Slack API Data Models)
[ ] slack-api-*-kotlin-extension (Kotlin Extensions for Slack API Clients)
[x] slack-app-backend (The primitive layer of Bolt for Java)

Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

This is ready for release but I will do it early next week.

merged by https://github.com/slackapi/java-slack-sdk/commit/55c363a10e6f50aed2790bf73bc5042238deead7


Rpath addition and deletion.

[ ] Rpath addition
[x] Rpath deletion

Addition support is blocking on #38. Once we merge that in, I can fill the necessary bits in here :smile:
cc @UniqMartin

Aside from two minor suggestions, I think the “delete RPATH” feature is ready to be shipped.
Which brings me to the following question: Would it be a big bother to split off the “add RPATH” code into a separate branch (and future PR)? This way we could already merge the deletion code and reviewing the addition (once implemented) would become easier.

Would it be a big bother to split off the “add RPATH” code into a separate branch (and future PR)?

Not at all! I'll do this presently.

Not at all! I'll do this presently.

Thanks, much appreciated! :bow:

UniqMartin changed the title from Rpath addition. to Rpath deletion. 5 minutes ago

Oops :sweat_smile:

No worries! I think this is ready to merge. Could you please strip the [WIP] prefixes from your commit messages, push one last time, and then merge this?

Sure thing, I'll do it when I'm back at home.

The funny thing is you also confused what went where with the branch names, so you'll now have to live with the fact that the delete-rpath branch is actually about RPATH addition. 😀
It sounded like you're not opposed to merging this and because this lives in a repository where we both have commit access, I just went ahead, stripped the prefixes for you, and then merged.
Good work! I guess now that all the relevant infrastructure is in place, it's RPATH addition time!

The funny thing is you also confused what went where with the branch names, so you'll now have to live with the fact that the delete-rpath branch is actually about RPATH addition

Yeah, I was pretty tired when I split the branch off :sweat_smile:...

Not at all, thanks!


EFS mount inconsistent with PentestPortal EC2 instances

🐛 Bug Report
When PentestPortal EC2 instances are spun up, the EFS mount sometimes fails.
To Reproduce
Steps to reproduce the behavior:

Spin up several PentestPortal EC2 instances
Check to see if any have an attempted EFS mount that failed

Expected behavior
The EFS mounts should always be available on PentestPortal instances.

This morning, we were informed that a freshly-created environment (env11) was found to have its 4 Kali instances created without successful EFS volume mounts.  Running sudo mount -a resulted in successful mounting of the share, but it's unclear why this problem occurred.

This morning, we were informed that a freshly-created environment (env11) was found to have its 4 Kali instances created without successful EFS volume mounts.  Running sudo mount -a resulted in successful mounting of the share, but it's unclear why this problem occurred.

This morning, we were informed that a freshly-created environment (env11) was found to have its 4 Kali instances created without successful EFS volume mounts. Running sudo mount -a resulted in successful mounting of the share, but it's unclear why this problem occurred.

I think the problem was made less likely by 638f182, but obviously it was not completely eliminated.  I think the issue is that Terraform has no way to distinguish between the "resource created" and "resource created and ready for use" states.  In the case that @dav3r mentioned, I suspect that the share was created but did not become available for attachment until after the Kali instances had already booted up and automounted the mounts listed in /etc/fstab.
A way to remedy this is to add a simple shell to the cloud-init configuration that will attempt to mount the share in a loop until successful.  This way the share will get mounted even if the instance slips through the crack left by the Terraform dependency.
What do you think, @dav3r?

This morning, we were informed that a freshly-created environment (env11) was found to have its 4 Kali instances created without successful EFS volume mounts. Running sudo mount -a resulted in successful mounting of the share, but it's unclear why this problem occurred.

I think the problem was made less likely by 638f182, but obviously it was not completely eliminated.  I think the issue is that Terraform has no way to distinguish between the "resource created" and "resource created and ready for use" states.  In the case that @dav3r mentioned, I suspect that the share was created but did not become available for attachment until after the Kali instances had already booted up and automounted the mounts listed in /etc/fstab.
A way to remedy this is to add a simple shell to the cloud-init configuration that will attempt to mount the share in a loop until successful.  This way the share will get mounted even if the instance slips through the crack left by the Terraform dependency.
What do you think, @dav3r?

@jsf9k That makes sense to me- I'm on board with your plan.

@jsf9k That makes sense to me- I'm on board with your plan.


Demo broken

Demo is not working due:
"Mixed Content: The page at 'https://vinnymoreira.com/stellarnav-js-demo/' was loaded over HTTPS, but requested an insecure script 'http://ajax.googleapis.com/ajax/libs/jquery/1.11.1/jquery.min.js'. This request has been blocked; the content must be served over HTTPS."

Hi there,
Thanks for pointing that out! I have already fixed it and demo is working.


Service 'trafficops-perl' failed to build: [...] dial tcp: lookup geolite.maxmind.com: No address associated with hostname

I'm submitting a ...

[x] bug report
[ ] new feature / enhancement request
[ ] improvement request (usability, performance, tech debt, etc.)
[ ] other

Traffic Control components affected ...

[x] CDN in a Box
[x] Documentation
[ ] Grove
[ ] Traffic Control Client
[ ] Traffic Monitor
[x] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] unknown

Current behavior:

The trafficops-perl Docker image for CDN-in-a-Box fails to build because the geolite.maxmind.com subdomain no longer resolves.
Expected / new behavior:

trafficops-perl should build successfully.
Minimal reproduction of the problem with instructions:

docker rmi cdn-in-a-box_perl;
docker-compose build trafficops-perl;

Failed build log follows:
Building trafficops-perl
Step 1/16 : FROM centos:7
 ---> 5e35e350aded
Step 2/16 : LABEL   name="Traffic Ops Perl"<EMAIL_ADDRESS> ---> Using cache
 ---> cfa2c7d6e139
Step 3/16 : EXPOSE 443
 ---> Using cache
 ---> f20b3afea2b4
Step 4/16 : ENV MOJO_MODE production
 ---> Using cache
 ---> 07c71701c21e
Step 5/16 : RUN rpm --import /etc/pki/rpm-gpg/RPM-GPG-KEY-CentOS-7 &&     rpm --import https://dl.fedoraproject.org/pub/epel/RPM-GPG-KEY-EPEL-7 &&     rpm --import https://download.postgresql.org/pub/repos/yum/RPM-GPG-KEY-PGDG &&     yum -y update ca-certificates &&     yum-config-manager --add-repo https://download.postgresql.org/pub/repos/yum/9.6/redhat/rhel-7-x86_64 &&     yum install -y         epel-release         pgdg-redhat-latest &&     yum install -y         bind-utils         cpanminus         expat-devel         gcc-c++         gettext         git         iproute         jq         libcurl         libcurl-devel         libidn-devel         libpcap-devel         mkisofs         net-tools         nmap-ncat         openssl         openssl-devel         perl         perl-core         perl-Crypt-ScryptKDF         perl-DBD-Pg         perl-DBI         perl-DBIx-Connector         perl-Digest-SHA1         perl-JSON         perl-JSON-PP         perl-libwww-perl         perl-TermReadKey         perl-Test-CPAN-Meta         perl-WWW-Curl         postgresql96         postgresql96-devel         postgresql96-libs         tar &&    yum -y clean all
 ---> Using cache
 ---> 9c6488d6690d
Step 6/16 : ADD https://geolite.maxmind.com/download/geoip/database/GeoLite2-City.tar.gz /
ERROR: Service 'trafficops-perl' failed to build: ADD failed: Get https://geolite.maxmind.com/download/geoip/database/GeoLite2-City.tar.gz: dial tcp: lookup geolite.maxmind.com: No address associated with hostname

Anything else:

MaxMind no longer publicly hosts their downloads, instead requiring you to register (for free). See: https://blog.maxmind.com/2019/12/18/significant-changes-to-accessing-and-using-geolite2-databases/
This also affects the Traffic Ops post-install script.
Docs will need to be updated to remove references to geolite.maxmind.com.

Is the process no longer possible to automate?
Also, I think we added this because TR refused to start without a database. If that's (still?) true, then IMO that requirement should be relaxed, since geolocating clients via IP database lookup is not a requirement for TR to function.

Without at least a GeoIP DB, TR will always default to whatever the default lat/long is for geolocating clients (assuming no CZF either). That makes it pretty useless for testing and demo purposes.
It sounds like it can still be automated as long as you create an account and provide the license key via a qparam. So...not sure if we can just create some "shared" Traffic Control account and use that. There's probably a lot of fine print to read. The website says they reserve the right to limit the number of downloads within a limited amount of time.

BTW looking at the TR code, it looks like if there is no geolocation.polling.url configured, it will fail to process the snapshot. So basically, that parameter is required, meaning a maxmind DB is required. Relaxing that requirement to make a maxmind DB optional will likely require changes to several places in TR, and while TR is mostly useless without some kind of Geo-IP DB for geolocation, I'm leaning more towards the view of keeping it required.

We already track the db ourself in our repo, along with its license. Any reason not to just reuse what we have?

That is a really old version of the DB, but maybe that is enough for CIAB. That is, if the new EULA and downloads don't apply to old downloads from before this new EULA existed.

The latest Internet Archive backup (2019-12-27) has the same license. If we can keep the old DB, we can just as easily use this newer one from before the new EULA.

"maybe enough for CIAB" is what I was going for with my suggestion. You might not be able to get a perfectly efficient TR instance without that database, but it could still function by falling back to the same static coordinates for every client request. It's not enough to demo or test Traffic Router but it would be enough to demo and at least partially test Traffic Control. Besides, it shouldn't preclude you from using a database in such demos or tests.
We should maybe spend some time looking into that shared account - personally I like that better than having to update a 3rd party file in our own version tracking.

I think the shared account is probably a non-starter given the fine print in the new EULA. Our best bet for now is to probably just use the internet archive backup or the "test" DB we already have backed up into our repo.

For users who still want the database, maybe we could make a config var titled "Download Maxmind Database?", use the backup if it's set to "no" (default), then require an account ID and license key if it is set to "yes". As for downloading it, maxmind/geoipupdate lays it out in relatively few lines. Maybe we could do something similar in our postinstall script, replacing the first InstallUtils::execCommand() call in setupMaxMind():
curl \
    --verbose \
    --compressed \
    --user "${MAXMIND_ACCOUNT_ID}:${MAXMIND_LICENSE_KEY}" \
    'https://updates.maxmind.com/geoip/databases/GeoLite2-City/update?db_md5=00000000000000000000000000000000' |
    gzip > /GeoLite2-City.mmdb.gz

But yeah, users would need to supply their own account ID and license key to do that.


Adding tickInterval as in HighCharts

Closes #190
Possibilities:

tickValues: Array of Number [5, 10, 15] Here displays only 3 ticks
tickInterval: Number  5 will display a tick every 5
ticks: Number 5 Will force 5 ticks to be displayed (mode auto)

How are you doing this?

ticks: Number 5 Will force 5 ticks to be displayed (mode auto)

does not look like you are forcing only 5 ticks

ticks is a props that already exists. Have not touched it. Just listing all the possibilities to display ticks

I added on tickInterval

got, it ticks will actually not force, it is just a guidance,
see https://github.com/d3/d3-scale#continuous_ticks
got confused with the comment about force.
This looks good

yes you are right, sometimes it is less, sometimes more.


Wrong link in documentation

What issue does this PR address?
The link in the documentation was redirecting to a config file that did not contain the needed Config.GetIdentityResources().
Does this PR introduce a breaking change?
No
Please check if the PR fulfills these requirements

[x] The commit follows our guidelines
[x] Unit Tests for the changes have been added (for bug fixes / features)

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


HTTP://www.electroncash.org/ does not redirect to HTTPS version

The downloads are then done without encryption.

OT: There's a typo in the footer ("Soure"), and can't find the website sources.

i was going to move this to a new host at some point. im not even sure how its doing https. i didnt buy an SSL cert.  why is this important? possible mitm?

all links are https


[Feature] Add array function of array_repeat #47834

Why I'm doing:
What I'm doing:
Fixes #19658
What type of PR is this:

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[x] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[x] This pr needs user documentation (for new or modified features or behaviors)

[x] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.3
[x] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

Can someone review this RP?

@silverbullet233 @kangkaisen @stdpain @mofeiatwork My unit tests have covered all the code paths. Why are there many code paths not covered in codecov/project/be-total? Is it calculated based on the code paths covered by the sql test?

ignore backport check: 3.3.1

@Mergifyio backport branch-3.3


fix: alt error mix with height/width

More granular detection of alt and resize in image
This was previously discuted into the discord troubleshooting channel.
Normally it shoulds support and detect size and alt text.

can you run npm run format please? should stop the checks from failing

This regex would allow a single match and returns capture groups. The rubular links shows a few test cases we tried on discord.

Doesn't works on my side, because i use + as an alt text. We need to "sanitize"/"escape" the text before, i think?

superseded by https://github.com/jackyzha0/quartz/commit/211f95c5277be05b0f1768d36ad8d9129241e663

Hello, this doesn't fix the current issue and the alt continue to be considered as width…


Error: Buffer.from is not a function

What version of this package are you using?
2.0.2
What operating system, Node.js, and npm version?
OS: KDE neon 5.23 (Ubuntu 20.04)
Node version: v17.6.0
npm version: 8.5.1
What happened?
When running npm start, the following happened.
A JavaScript error occurred in the main process
Uncaught Exception:
TypeError: Buffer.from is not a function
    at Object.<anonymous> (/home/zeankundev/nuTorrent/node_modules/bencode/lib/encode.js:49:22)
    at Module._compile (module.js:428:26)
    at Object.Module._extensions..js (module.js:446:10)
    at Module.load (module.js:353:32)
    at Function.Module._load (module.js:308:12)
    at Module.require (module.js:363:17)
    at require (module.js:382:17)
    at Object.<anonymous> (/home/zeankundev/nuTorrent/node_modules/bencode/lib/index.js:3:18)
    at Module._compile (module.js:428:26)
    at Object.Module._extensions..js (module.js:446:10)

What did you expect to happen?
Launching the application to debug
Are you willing to submit a pull request to fix this bug?
No

Definitely odd, are you trying to load this in a frontend environment? What does your npm start directive do? Can you share a bit more about the setup, e.g.. if you are using webpack or something for a react app?

When I started it with npm start, it fires the following.
electron .

and starting hot server runs as following.
webpack-dev-server --config webpack/webpack-hot-dev-server.config.js --hot --progress --colors --port 2992 --inline

What version of webpack are you on? I know from v5 onwards they removed support for node polyfills, I don't know if that affects electron as well, but it would mean Buffer is unavailable.
Is your current project open source? If not, could you make a small electron project that just uses Buffer.from() and see if it works / share it?


ARROW-6486: [Python] Allow subclassing & monkey-patching of Table

Currently, many classes in pyarrow behave strangely to the Python user: they are neither subclassable not monkey-patchable.
>>> import pyarrow as pa
>>> class MyTable(pa.Table):
... pass
...
>>> table = MyTable.from_arrays([], [])
>>> type(table)
<class 'pyarrow.lib.Table'>

The factory method did not return an instance of our subclass...
Never mind, let's monkey-patch Table:
>>> pa.TableOriginal = pa.Table
>>> pa.Table = MyTable
>>> table = pa.Table.from_arrays([], [])
>>> type(table)
<class 'pyarrow.lib.Table'>

OK, that did not work either.
Let's be sneaky:

>>> table._class_ = MyTable
Traceback (most recent call last):
{{ File "<stdin>", line 1, in <module>}}
TypeError: _class_ assignment only supported for heap types or ModuleType subclasses
>>>

There is currently no way to modify or extend the behaviour of a Table instance. Users can use only what pyarrow provides out of the box. - This is likely to be a source of frustration for many python users.
The attached PR remedies this for the Table class:
>>> import pyarrow as pa
>>> class MyTable(pa.Table):
... pass
...
>>> table = MyTable.from_arrays([], [])
>>> type(table)
<class '_main_.MyTable'>
>>>
>>> pa.TableOriginal = pa.Table
>>> pa.Table = MyTable
>>> table = pa.Table.from_arrays([], [])
>>> type(table)
<class '_main_.MyTable'>
>>>

Ideally, these modifications would be extended to the other cython-defined classes of pyarrow, but given that Table is likely to be the interface that most users begin their interaction with, I thought this would be a good start.
Keeping the changes limited to a single class should also keep merge conflicts manageable.

Codecov Report

Merging #5322 into master will increase coverage by 1.55%.
The diff coverage is 79.54%.

@@            Coverage Diff             @@
##           master    #5322      +/-   ##
==========================================
+ Coverage    87.7%   89.26%   +1.55%
==========================================
  Files        1036      753     -283
  Lines      149741   108819   -40922
  Branches     1437        0    -1437
==========================================
- Hits       131328    97135   -34193
+ Misses      18051    11684    -6367
+ Partials      362        0     -362

Impacted Files
Coverage Δ

cpp/src/arrow/python/pyarrow.cc
25.88% <0%> (-0.31%)
:arrow_down:

python/pyarrow/tests/test_strategies.py
58.82% <0%> (ø)
:arrow_up:

python/pyarrow/tests/test_table.py
99.64% <100%> (+0.02%)
:arrow_up:

python/pyarrow/_orc.pyx
77.77% <100%> (ø)
:arrow_up:

python/pyarrow/ipc.pxi
81.03% <100%> (ø)
:arrow_up:

python/pyarrow/_parquet.pyx
91.33% <100%> (ø)
:arrow_up:

python/pyarrow/_json.pyx
90.47% <100%> (ø)
:arrow_up:

python/pyarrow/tests/test_misc.py
95.45% <100%> (+0.71%)
:arrow_up:

python/pyarrow/_csv.pyx
99.28% <100%> (ø)
:arrow_up:

cpp/src/arrow/python/pyarrow_api.h
81.53% <100%> (ø)
:arrow_up:

... and 289 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0158ae1...a581155. Read the comment docs.

I am not very excited about this. These classes are intended to be developer-facing, not user-facing. If you desire a different user interface, you should use composition rather than inheritance.

I agree with Wes. As I mentioned on the issue: from experience with pandas, properly enabling subclassing of a complex data structure adds quite some complexity to the code with lots of corner cases. So I am not sure this is something we want to do for pyarrow Table. As Wes said, there are alternatives to subclassing if you want to use pyarrow.Table's functionality in a custom interface.

FWIW, the same is also true of NumPy. Subclasses of ndarray have never really worked out the way they were intended, and I think it would have been better to not offer the feature in the first place.

I'm revising my position on this to -1.

I'm closing for maintainability concerns. We can discuss more on dev@


Designing IoT Solutions

Be sure to visit our Resources Page for tools, resources, and example articles to go over.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
We're looking for the first 2-3 paragraphs of the article that appropriately summarize what your article will be about.
NOTE: We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
Proposed title of article
Designing IoT Solutions
Introduction paragraph (2-3 paragraphs):
The emergence of disruptive technologies such as Big data, AI, and Internet of Things (IoT) has brought a rapid growth in the Fourth Industrial revolution. The industrial revolution being data driven, is the primary reason for the rise in development of IoT applications and solutions.
IoT solutions and applications leverage on network of physical objects -''things'' that are embedded with sensors, software and data for the purposes of connecting and exchanging data over a network or the internet. This article will give readers an overview of what is takes to design IoT solutions.
Key takeaways:

What is IoT
IoT architecture
IoT protocols
IoT development platforms and use cases

References:
N/A
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

@OMONDIELLY it seems you have two topic suggestions open. Please pick one to elaborate and proceed with 👍

Hi @hector , I'll go with the recently sent issue (#1825). Thanks for the
heads up
On Mon, Mar 22, 2021 at 10:10 PM hectorkambow @.***>
wrote:

@OMONDIELLY https://github.com/OMONDIELLY it seems you have two topic
suggestions open. Please pick one to elaborate and proceed with 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/issues/1825#issuecomment-804324280,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJSCYTRFDC5ER6KO7DDEPULTE6I2VANCNFSM4ZTHRMLA
.

Hi.. does this mean that I can go ahead with this or wait for Molly to send an approved tag?

@OMONDIELLY seems like a cool topic - approved. 👍