Mention Create React App

Seems like a good idea to me

Why not to mention this boilerplate in react-create-app's README in return? :-)

One of the reasons for Create React App’s existence is that people used to complain “omg I have to learn redux, sagas and ton of other things to create a React hello world”. I don’t think pointing beginners to it is a good idea 😄 . Advanced users will find it anyway I think.

I understand this motivation and I′m actually really glad that you guys created that repo. I wish it existed half a year ago when I was entering this world of lots scary worlds and my head was just exploding :–)  React-create-app is something I now recommend to colleagues to make their first steps in React.
But at the same time, react-create-app is something that the newcomers might want to outgrow at some point. Or at least it would be good for them to know that there exist quite an advanced ways of linking React with lots of other things. This understanding will help them build a better initial model of the ecosystem and increase the chances of preferring React to Angular, for example.
Mentioning the most starred boilerplates like this one in some ‘advanced alternatives’ section won't harm, but instead provide people with richer set of options, which they can switch between at different stages of their journey into React.

I agree with @kachkaev here, that seems fair! ping @gaearon

This might be relevant here too: https://github.com/facebookincubator/create-react-app/issues/580

I'm hesitant partly because of naming. I feel that if we link to something called "react boilerplate" too many beginners will go there because it sounds familiar. It's a model they already understand. But react-boilerplate is not really about React. If anything, this project is react/redux boilerplate, with an opinionated twist (sagas). I recognize these are valuable but it being called react-boilerplate contributes to the confusion about what react is, and to the fear of missing out. I don't have a good solution to this but I hope you can see where I'm coming from.

The only users RB is going to lose by linkjng to CRA is those who didn't need RB anyway. Those who are looking for runtime setup will see that CRA doesn't provide it.
On the other hand, people completely new to CRA might not even understand the difference between runtime and build setup, and can't assess the difference. Those who need runtime setup will google for "react boilerplate" if they are unsatisfied with CRA, and this project will be their first hit anyway.

Closing as there seems to be no interest from the maintainers. I added some more thoughts on how I think RB approach could be improved here: https://github.com/facebookincubator/create-react-app/issues/580#issuecomment-249362743. Cheers!


Add support for ScaledType

Add support for scaledType attribute to expand, crop ... the video.
For example, to play videos with different resolution than the VideoTextureview without the black bars

API16+
mediaPlayer.setVideoScalingMode(MediaPlayer.VIDEO_SCALING_MODE_SCALE_TO_FIT_WITH_CROPPING);

+1 for this

Fixed https://github.com/malmstein/fenster/releases/tag/v.0.0.2

Fixed https://github.com/malmstein/fenster/releases/tag/v.0.0.2

Thanks @malmstein :clap: :dancer: :+1:


BatchNorm is not in 'test mode' inside update_train_data

Hi!
In example mnist_4.2_batchnorm_convolutional.py we have:
    # compute training values for visualisation
    if update_train_data:
        a, c, im, ca, da = sess.run([accuracy, cross_entropy, I, conv_activations, dense_activations], {X: batch_X, Y_: batch_Y, tst: False, pkeep: 1.0, pkeep_conv: 1.0})
        print(str(i) + ": accuracy:" + str(a) + " loss: " + str(c) + " (lr:" + str(learning_rate) + ")")
        datavis.append_training_curves_data(i, a, c)
        datavis.update_image1(im)
datavis.append_data_histograms(i, ca, da)

But shouldn't tst: False be actually True there?

Good question. It will not change anything in the training. The values computed here are for visualisation only. So question is: will this change produce "more relevant" curves on the graphs ?
If this was dropout, then yes, definitely, you need to disable dropout when you compute the test and training curves, otherwise your training curve, computed on a half-brain-dead (dropped out) network will be very bad, sometimes worse than the test curve which should never happen (test loss is normally always higher(=worse) than training loss).
For batch norm, maybe that is the case as well. You have to try. I implemented this in this way because of the way batchnorm works. It uses statistics computed on the current batch of training data. When measuring performance on your test dataset, there is no "current batch of training data" so you have to use some other stats (running average of multiple previous batches). That is what the tst:True flag enables. When measuring your training metrics, the current batch exists and using stats computed on it for batch norm is not a problem so I did not feel I should enable a solution for a problem that I did not have. But maybe I was wrong there.

Thank you very much for the explanation


Request for documentation surrounding animationConfig

Personally not a fan of the bouncy animation and I'm curious about how I can adjust this prop to remove it.

@salieflewis
I created a quick patch that reduces the bounce effect - https://github.com/backpackapp-io/react-native-toast/issues/35#issuecomment-2372734871

Thank you @fredrifoUni
I tried recreating this patch but nothing changed for me. Are there further adjustments I need to make when actually calling the toast for these styles to apply?

Thank you @fredrifoUni
I tried recreating this patch but nothing changed for me. Are there further adjustments I need to make when actually calling the toast for these styles to apply?

No problem! This patch only works when passing position: ToastPosition.BOTTOM in the ToastOptions. If you want to modify it for ToastPosition.TOP, you'll have to patch the other if statement.

Will be fixed once #40 is merged allowing for total control over animationConfig


Add support for DAPRSIDEKICK_XXX variables

Description
It is now possible to define DAPRSIDEKICK_XXX variables to set or override top-level configuration options. Note that this feature is limited to those options defined in the DaprProcessOptions base class which will equalluy apply to either the Sidecar, Placement or Sentry processes depending which is configured.
For example you can now set DAPRSIDEKICK_ENABLED=false to completely disable Dapr Sidekick, which is useful if you are running inside a pod/container where Dapr is injected separately.
Issue reference
Fixes Issue #9
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation where possible

Codecov Report

Merging #14 (92908a4) into main (ee1044c) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #14      +/-   ##
==========================================
+ Coverage   88.88%   88.90%   +0.01%
==========================================
  Files         104      104
  Lines        2231     2235       +4
  Branches      453      446       -7
==========================================
+ Hits         1983     1987       +4
  Misses        159      159
  Partials       89       89

Flag
Coverage Δ

net35
85.43% <ø> (ø)

net48
?

net5.0
88.30% <100.00%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/Man.Dapr.Sidekick/Options/DaprOptions.cs
100.00% <ø> (ø)

...NetCore/DaprSidekickServiceCollectionExtensions.cs
97.14% <100.00%> (+0.36%)
:arrow_up:

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 ee1044c...92908a4. Read the comment docs.


Ensure downloaded images with transparent background looks correct

I recall that the colors looked incorrect previously possibly due to premultiplied alpha settings?

No combinations of alpha rendering and premultiplied alpha settings seem to result in the image being different. Will need to test exporting the images again. It may occur only in saved png files?
Test fiddle
https://jsfiddle.net/cyrf21zm/

Looks like everything lines up with the displayed canvas. Using premultipliedAlpha === true is what should be used for the best result when rendering and saving:

premultipliedAlpha: false
premultipliedAlpha: true


feat: add time and input

This adds top-level time and input namespaces.
time is used to facilitate starting/ending timers including the ability to pass in a sync or async callback which will be timed. This is a pattern we already used in npm/cli and can be replaced with this code from proc-log instead.
input is used to send events indicating user input is being read. There are two ways to do this: start and end functions or a read function which will return a promise and emit the promise's resolvers to the consumer.
This also adds a KEYS to each namespace which is an enum for getting the levels by name. In npm/cli our consumer creates these with log.LEVELS.reduce((a, k) => (a[k] = k, a), {}) so that we don't need to type any string names in our handelrs.

Maybe I'll see it when I review again tomorrow but I don't understand the function of KEYS

This question applies to log.pause/unpause and input.start/end.  Whose concern should it be if multiple starts/pauses are called and then multiple unpause/ends?  How is a consumer to know when ALL interested parties have indicated that logs should unpause or input has truly ended?  Obviously if two asynchronous operations are trying to do input they'll inevitably collide with each other so perhaps that's not a concern to solve.  This is thinking out loud at this point.

Maybe I'll see it when I review again tomorrow but I don't understand the function of KEYS

It is only there to avoid having to manually type strings in consumers. Here's a diff of what you would replace if KEYS were exported.
+ const { output } = require('proc-log')

 process.on('output', (level, ...args) => {
   switch (level) {
-    case 'standard':
+    case output.KEYS.standard:
       console.log(args)
       break
-    case 'error'
+    case output.KEYS.error:
       console.error(args)
       break
   }
 })

It is only there to avoid having to manually type strings in consumers

You still have to know and type the indices to KEYS, I don't get the difference.  I'm willing to accept this is just me though.

How is a consumer to know when ALL interested parties have indicated that logs should unpause or input has truly ended?

This is why I opted for time.start/end(name) to take a timer name. Multiple timers can be run and it's up to the consumer to handle all of them simultaneously. This is how lib/utils/timers.js in npm/cli works currently.
input.start/end/read() don't take a name argument since there is only one interface for user input. If there are multiple producers nested which are emitting input events, there will be other problems. So I didn't think it should be encouraged to try and namespace input events. Should this be added to input as an escape hatch for complex terminal UI scenarios I'm not envisioning?

Should this be added to input as an escape hatch for complex terminal UI scenarios I'm not envisioning?

No, you thought about this more than I did already.  This is find as-is.

My remaining 3 comments (readline, log.pause, KEYS) are just those: comments. None of them are blockers but you can act on them if you choose.

@wraithgar Pushed a commit removing args from log.pause() and `log.resume(). Leaving the rest as-is.


Run embed-docs then strip-docs on CI

We should run python3 generator.py --embed-docs followed by a python3 generator.py --strip-docs and exist with an error if there are any changes. This is mostly to ensure we are currently setting the rustdoc-ignore-next for in-code docs.

Same for gtk3-rs as well btw


feat: changelog action & example

The changelog action uses towncrier and adds fragments (files) to a directory of your choice based on your pyproject.toml settings for towncrier. Each fragment is named in the format pr_number.pr_label.md. The pr_label is determined by the labels in the pull request. For example, if you have the "enhancement" and "maintenance" labels, it will set the pr_label as "added". This means the fragment will be added to the changelog under the "Added" subsection. Side note - it seems like the majority of PRs have the maintenance label, so that's why I chose to categorize the PR based on other labels other than maintenance.
I think it would make the most sense to put this action at the end of the .github/workflows/label.yml file. For this action to work, you have to add the "types" line under "pull_request:" in label.yml file:
on:
  pull_request:
    # opened, reopened, and synchronize are default for pull_request
    # edited - when PR title or body is changed
    # labeled - when labels are added to PR
    types: [opened, reopened, synchronize, edited, labeled]

labeler:
...

changelog-fragment:
  name: "Create changelog fragment"
  needs: [labeler]
  runs-on: ubuntu-latest
  steps:
    - uses: ansys/actions/doc-changelog@{{ version }}
      with:
        token: ${{ '{{ secrets.GITHUB_TOKEN }}' }}

Here is an example of how to set up your repository to use towncrier/this action: https://github.com/ansys/ansys-templates/compare/demo/changelog-action?expand=1
This action will only create the fragments for the changelog. You will have to run towncrier build --yes --version <version> to create the changelog using these fragments. Here is an example: https://towncrier.readthedocs.io/en/stable/markdown.html
This PR is the same as #357 but has less commits so it would be easier to review. I figured I could go over this with everyone at the team meeting to get everyone's opinions :)

This is great @klmcadams! I will try to review it today

Just curious - why did we close #357?

Just curious - why did we close #357? Commits get squashed and merged anyway.

I closed it because there were a ton of comments and commits, so I thought it would be easier to review if I started a new PR


Add debug logs

Add debug logs to debug state changes

Codecov Report
Base: 68.73% // Head: 68.75% // Increases project coverage by +0.01% :tada:

Coverage data is based on head (f5f6343) compared to base (206aef1).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2262      +/-   ##
============================================
+ Coverage     68.73%   68.75%   +0.01%
- Complexity     9811     9814       +3
============================================
  Files           751      751
  Lines         57499    57504       +5
  Branches       7068     7068
============================================
+ Hits          39524    39536      +12
+ Misses        15684    15680       -4
+ Partials       2291     2288       -3

Impacted Files
Coverage Δ

...m/github/ambry/replication/ReplicationManager.java
81.37% <100.00%> (+0.66%)
:arrow_up:

...va/com/github/ambry/router/TtlUpdateOperation.java
82.85% <0.00%> (-0.58%)
:arrow_down:

...va/com/github/ambry/replication/ReplicaThread.java
78.52% <0.00%> (+0.25%)
:arrow_up:

...ain/java/com/github/ambry/router/PutOperation.java
91.00% <0.00%> (+0.43%)
:arrow_up:

.../com/github/ambry/cloud/CloudStorageCompactor.java
75.28% <0.00%> (+2.24%)
:arrow_up:

...java/com/github/ambry/clustermap/CloudReplica.java
76.92% <0.00%> (+3.84%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@justinlin-linkedin can u please take another look and merge if ok ?


Fix theme overriding architecture

Purpose

Fixed theme overriding architecture

Extended Semantic UI Theming structure
Made default theme as the base and made other themes as sub-themes
Made custom composer definitions overridable

@hevayo - Changed branch to release-0.990.0


ITEM_COMMAND not working

What happened?
ITEM_COMMAND("EXIT MENU", exitMenu));  //close menu and off backlight

void exitMenu()
{
  Serial.println("working");
  menu.resetMenu();
  menu.hide();
}

And function not invoking when presing ENTER.
But I can navigate throught menu, I can enter to submenu using ENTER.
Arduino & STM32F4.
Any ideas?
Version
3.3.2
What environment are you seeing the problem in?
Platform IO
Board type
blackpill STM32F4ccu6
Relevant log output
No response

Please make sure that your imports are correctly placed
see https://lcdmenu.forntoh.dev/usage

This could be marked more clearly, I've been debugging for 3 hours before I found this!


Issues with text overlay

Hello,
I am trying to change the text overlay using --text_overlay_fmt but I don't seem to be able to get it to write over multiple lines.
The readme says use the letter n, but I have tried to use every combination and I don't seem to be able to get it to work.
Any help would be much appreciated!

Use \n. For example:
--text_overlay_fmt "City: {event_city}\nDate/Time: {local_timestamp_rolling}"

Legend, worked like a charm.
Thank you!


couldn't process CM file: expected a class

Environment

VS Code version: n/a
Millet version: 71988b9cfea4faed3c0aa77ad5f417895a55f99d
OS version: macOS 10.15.7

Steps to reproduce
git clone https://github.com/jez/multi-grep
cd multi-grep
mv multi-grep.cm sources.cm # see issue #3
~/github/millet/target/debgug/cli

Expected behavior
no error
Actual behavior
❯ ~/github/millet/target/debug/cli
Error: /Users/jez/Dropbox/Documents/Programming/cli/multi-grep/sources.cm: couldn't process CM file: expected a class

Caused by:
    expected a class

It would be nice to at least know which line number of the CM file failed to parse, so I could know what the problem is. The contents of the CM file in question:
Group is
  $/regexp-lib.cm
  vendor/cmlib/cmlib.cm

  src/prelude.sml : succ-ml
  src/util.sml : succ-ml

  src/options.sml : succ-ml

  src/main.sig
  src/main.sml : succ-ml

yeah, i put more effort into having good error locations etc for sml errors, while the cm error reporting is far more rudimentary.
in this case i’m thinking the error is because the class (the thing after the :) isn’t understood by millet. specifically “succ-ml”. millet only knows about a few classes, and actually allows processing even fewer (cm and sml only).

classes millet can parse from a string: https://github.com/azdavis/millet/blob/71988b9cfea4faed3c0aa77ad5f417895a55f99d/crates/cm/src/types.rs#L186
logic that rejects all but sml and cm: https://github.com/azdavis/millet/blob/71988b9cfea4faed3c0aa77ad5f417895a55f99d/crates/cm/src/lower.rs#L24

i think instead of having an exhaustive list of classes (i think it’s possible to define arbitrary cm classes with user code?) i could just keep it a string and then only accept some, like “sml”, “cm”, and “cmfile”. that would improve the error here

i think there are actually two separate issues here:

the error message for an unknown class should be better (this issue, addressable with the above comment)
errors in cm files should have location info (a new issue, more involved)


Rerack knapsack test

I rule and have completed some work on Case Manager that's ready for review!
We've added a bunch of new system tests, so now's a good time to rerack the split so node 0 doesn't take forever in CI.
No issues or prod code changes, so merging after CI completes.

Codecov Report

Merging #1416 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1416   +/-   ##
=======================================
  Coverage   94.51%   94.51%
=======================================
  Files          63       63
  Lines        1440     1440
=======================================
  Hits         1361     1361
  Misses         79       79

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 278a22e...33386f4. Read the comment docs.


Minimum Length Exclusion

Requesting an option to only include edits that are within a minimum duration (exg 0.5s or 1s minimum duration).
This would be to reduce choppiness where edits  are very short in duration.

That's what minclip does. Read the --help docs

Which version is that in? Does not appear to exist in the latest version of auto-editor (23.51.1).
$ ./bin/auto-editor --help |grep -i mini
$ ./bin/auto-editor --help |grep -i clip
$ ./bin/auto-editor --version
23w51a (23.51.1)
$ ./bin/pip install --upgrade auto-editor
Requirement already satisfied: auto-editor in ./lib/python3.11/site-packages (23.51.1)
Requirement already satisfied: numpy>=1.22.0 in ./lib/python3.11/site-packages (from auto-editor) (1.26.2)
Requirement already satisfied: pillow==10.1.0 in ./lib/python3.11/site-packages (from auto-editor) (10.1.0)
Requirement already satisfied: pyav==12.0.0 in ./lib/python3.11/site-packages (from auto-editor) (12.0.0)
Requirement already satisfied: ae-ffmpeg==1.1.* in ./lib/python3.11/site-packages (from auto-editor) (1.1.2)
$

That's what minclip does. Read the --help docs

Which version is that in? Does not appear to exist in the latest version of auto-editor (23.51.1).
$ ./bin/auto-editor --help |grep -i mini
$ ./bin/auto-editor --help |grep -i clip
$ ./bin/auto-editor --version
23w51a (23.51.1)
$ ./bin/pip install --upgrade auto-editor
Requirement already satisfied: auto-editor in ./lib/python3.11/site-packages (23.51.1)
Requirement already satisfied: numpy>=1.22.0 in ./lib/python3.11/site-packages (from auto-editor) (1.26.2)
Requirement already satisfied: pillow==10.1.0 in ./lib/python3.11/site-packages (from auto-editor) (10.1.0)
Requirement already satisfied: pyav==12.0.0 in ./lib/python3.11/site-packages (from auto-editor) (12.0.0)
Requirement already satisfied: ae-ffmpeg==1.1.* in ./lib/python3.11/site-packages (from auto-editor) (1.1.2)
$

Docs are in auto-editor --edit --help

After testing minclip, it appears that increasing that value increases all clips to at least that value.
Increasing the minclip value appears to increase the overall duration of the output file. Instead of removing clips under the minclip value (reducing the total duration).
For example, unless I'm mistaken it appears that minclip=5 will take a clip that is 1ms and extend the duration to make it 5ms.
This was with --edit audio:threshold=2%,minclip=5
The origin of the post is asking about filtering clips that do not meet a minimum duration. Example if minduration=5ms and there is an edited clip in the timeline that is only 1ms, the 1ms clip is not included in the final output.

Pull Request submitted to help clarify options listed in auto-editor --edit --help.


Update process to request a review of a brand new plugin

Summary
A community contributor, @monsdar, brought into our attention that the process for requesting a review of the whole code base of a plugin was unclear. This PR rewrites the process with a different approach, assuming that most plugins will be built from the starter template, and will contain commits from that repository.
See the conversation in Mattermost for more context.
Ticket Link
--

Looping in @hanzei here who's been through this process a few times for an initial review.


Active workflows gets executed even if deleted

Describe the bug
Active workflows gets executed even if deleted
To Reproduce

Create and workflow to modify record with composeRecord Trigger and enable it.
Insert a record to check if workflow is enabled and working.
Delete the workflow.
Insert another record and you will see that the workflow still got executed even if it was deleted.

Expected behavior
Wordflow should not executed if deleted.
Current Workaround
Open deleted workflow from admin app.
Disable the workflow or clear the workflow content.

Thank you for reporting the issue and apologies for the late response.
Fixed and will be part of the next (patch) release.


Add output format option

Output format can be set to gpx, tcx or original to allow
export different file formats (e.g. fit).

Thanks for the new feature!

Thanks for doing the initial work!


VoIP, WASAPI - audio capturing

I'm working on VoIP application and i use the code of VoIP sample for recording audio.  Unfortunately, after the latest update of Windows 10 Mobile audio capture through WASAPI not working. On desktop everything works well, the problem is only with the phone(Lumia). What can I do to solve this problem?

Did you solve the issue?
I have same issue but in my case it doesn't work on PC either

2 Years later, I think no one cares :(


Move DISTRO_VERSION declaration to KAS repository

As the KAS repository is responsible for the overall versioning, the
DISTRO_VERSION should be defined there as well. Unfortunately, it is not
possible to set the variable outside of the distro config, however one
can refer to the value of an arbitrary variable from the local configs global
scope.

Merge with https://github.com/iris-GmbH/meta-iris-base/pull/16


ci(cirrus): reduce matrix down to 6 instances

Ref: https://github.com/saltstack-formulas/template-formula/pull/121

:tada: This PR is included in version 0.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Way to easily dismiss watermarks

Refs #16560
There should be a way to easily dismiss the watermark. Optimally this would be a setting
'workbench.tips.visible', and there would be an explicit action on the start screen which would flip this setting.
The users which would appreciate this:

expert users that do not need help and like the clean UX that vscode is known for
users that have already read through all the watermark tips we give them

Users benefit from the commands and their shortcuts always being shown so the shortcuts can be memorized over time. The design was chosen to be unobtrusive. More actions on the start screen will just clutter that and new users inadvertently dismissing the commands will loose the watermark's benefit. No need to make it more complicated.

Why do you say that users benefits from the same commands being shown over and over?
The design is obtrusive for how I use vscode, I understand it is not obrusive for all the users. However for users like me we should provide a simple way to dismiss these watermarks. I do not see a downside to that.
At the very least please provide a setting.
@Microsoft/vscode for opinons / feedback

I would use a setting to hide it, but I think an explicit action to hide it, like an X button, would be unnecessary visual clutter.

@isidorn New and occasional users will appreciate being reminded of some of the basic keyboard shortcuts.
Clearly we need to gear the out-of-the-box experience more towards the new to intermediate user than the expert user while still allowing the expert user to tailor the product to specific needs.
Before we introduce yet another setting: How would the watermark become useful to expert users?

I agree with @roblourens; I'd use a setting to beautify the workbench a little more but a button on the UI would be overkill.
On being useful to power users, I was tossing around an idea of using it for a means for me to learn keybindings by allowing me to either set a list or showing random ones similar to a tip of the day. I've found I just completely ignore it at this point though so I'm not sure I would use it.

I was thinking about that too but I don't see it often enough to be useful for power-user tips. I usually have a file open on top of it.

@chrmarti do you accept PRs that introduce a setting to hide the watermark?
Not sure how to make the watermark useful to expert users. But to users like me who like the clean look of vscode the only solution is to hide it IMHO.

Sure, go ahead.


How to not show "three two one go"

What is your question?
NO I DONT WANNA SEE SOMONE SAYING JUST MUTE THE SOUND OR REPLACE PNGS TO TRASPARENT
So do you know how?
No. So does anynone know how to? PLS RESPOND

introSoundsSuffix and setProperty('sprite.visible', false)

setProperty('skipCountdown', true)
it skips only for the song

it will be disabled afterwards


IOS 7 SUPPORT?

Hi!   First Congrat for your Framework.
Why it has only support for ios 8+?
It's Any way to add support for IOS 7???
Thank you!

I didn't test it with iOS 7, but I bet it will work fine. Just try pulling the source files into your project and it will probably just work.


Using the diff function, output is not escaped.

Using vite/slinkity to develop eleventy site along with this plugin.
Vite is fine with:
{% highlight "html" %}
  {% include "html.html" %}
{% endhighlight %}

But not with ...
{% highlight "diff-html" %}
  {% include "html.html" %}
{% endhighlight %}

I get a vite error, because vite doesn't like non escaped html code (via vite error)
The vite error:
Unable to parse HTML; Illegal tag name. Use '<' to print '<'.

<h6>Heading 6</h6>

                                                                                ^

I've tried putting a safe filter around the include to ensure nunjucks escapes the included html file, but no dice.
Not using the diff and just using the shortcode normally, outputs escaped code without the vite error.
Does this mean the diff function is processing the 'output' content differently?

This is fixed in v5.0.0 and I included an automated test for this, thanks!

@zachleat Unfortunately, just having tested this in my local it's still producing an HTML illegal tag error. Again output is fine when using the non diff tag, but when using for example "diff-html" it produces this error for me. It claims I should be using < to print '<'.
Referring to the above code error and cutting it down to the identified character in question ...
<h6>Heading 6
                                   ^

I think the syntax highlighter is causing this by adding a '<' somewhere in the output.
This is with v5.0.0

Okay, I think I've identified when/why this is happening. For context: I created a re-useable include file which was using {% highlight 'diff-html' %} by default. But not every instance of content/code being included within the highlight had a + or a -. After adding these indicators, the output seems to be fine and i get no error.
So the issue is when there are no diffing indicators used within {% highlight 'diff-html' %} it should continue to output valid html, but currently does not.
To get around this, using flag on the include i.e {% set diff = true %}  to use diffing if it is needed otherwise just use normal highlighting.


Route Provider doesn't specify 'dev' in route-ethX config file

On Redhat based systems, the Route provider writes config files in /etc/sysconfig/network-scripts/route-ethX, but it doesn't specify a device. On CentOS 6 at least, if you don't specify a device, the static route will be bound to eth0.
I have the following route resource:
route '<IP_ADDRESS>/19' do
gateway '<IP_ADDRESS>'
device 'eth1'
end
The corresponding line in route-eth1 is
<IP_ADDRESS>/19 via <IP_ADDRESS>
when it should be
<IP_ADDRESS>/19 via <IP_ADDRESS> dev eth1
So the route gets created bound to eth0.
On top of this, the route provider doesn't check device when checking to see if a route exists. So when chef runs, it doesn't fix the route because it already sees it existing (even though it's bound to eth0 instead of eth1).
The first time that Chef runs with a new route resource, the route WILL be created correctly. It only gets messed up when the system is rebooted and the network scripts bring up the route on the wrong device.

Tagging with https://github.com/chef/chef/issues/3862.  We should fix this as part of the route provider overhall.

Tagging this for sustaining to see if we can replicate it. @thommay did a good chunk of work cleaning this up.


Should we pass the initial value of the reducer to mapValues?

Hi,
I was trying to test the assumption that "mutate reducer arguments" (state especially) is very wrong (http://gaearon.github.io/redux/docs/Troubleshooting.html) and I was expecting that when doing this in my reducer:
export function reducer(state = {}, action) {
//...
  switch (action.type) {
    case 'SOME_ACTION':
      state.frozen = false
      return state
//...

I would leave very soon shallowEqual method of react-redux @connect  on this test: https://github.com/gaearon/react-redux/blob/master/src/utils/shallowEqual.js#L3
But actually, I was surprised to find out that this test is almost always false and we end up leaving shallowEqual only after comparing all props (https://github.com/gaearon/react-redux/blob/master/src/utils/shallowEqual.js#L22). This is due to combineReducers https://github.com/gaearon/redux/blob/v1.0.0-rc/src/utils/combineReducers.js#L42 not passing the actual state to mapValues . Doing so, mapValues always provides a new object (https://github.com/gaearon/redux/blob/v1.0.0-rc/src/utils/mapValues.js#L5) and hence, shallowEqual test cannot leave early when testing current state and new state.
I was then wondering if this was the desired behavior (and if it is, why?) or if there could be some place for improvement by passing the state as initialValue to mapValues.
Thanks!

Can you provide a full failing example with actual and expected output? I don't understand what the issue is exactly. If you do something like connect(state => state) you're indeed going to have view redrawn every time. However if you connect(state => state.something) and that something has been mutated by its reducer, the view won't update.
Regardless, react-redux seems like the right place to file this issue. It isn't really related to Redux itself, but to react-redux update semantics. Please file it there.

Sorry if I wasn't clear, I should have said in the beginning that it's not a bug.
I was indeed using this connect code:
@connect(state => state)
export default class SomeComponent extends React.Component {
//...

and I know that it is wrong, it was for pure curiosity purpose.
Actually what's funny, is that the view was not redrawn every time. react-redux logic was correctly doing its job through shallowEqual function but I was expecting to leave shallowEqual on the first test https://github.com/gaearon/react-redux/blob/master/src/utils/shallowEqual.js#L3 since I was mutating the state and returning it from @connect. But because of redux combinerReducers::mapValues providing a new hash each time it is called, I only leave shallowEqual after all props have been compared (and since I mutated the state, all props are actually equals).
Again, this is not an issue, and using connect(state => state) is not advised but maybe using the current state as the initial value of mapValues of combineReducers could make it obvious that mutating the state + using connect(state => state) is a complete no way that will result in your views never updating (because you would then compare the same state object ref). That's why I posted this issue here instead of in react-redux because the fix would be done in combineReducers::mapValues. The only benefit of this fix would be to avoid comparing all props of state when someone is mutating the state and using connect(state => state).
What troubled me at first was that mutating the state can work (it can result in views being updated) if the state is very simple and if we use a correct @connect function:
@connect(state => {
  return {
    frozen: state.reducer.frozen,
  }
})

so I was only able to "feel" the bad effects of mutating state when I used connect(state => state).
Sorry I could not provide a failing test but nothing actually fails :).

I understand your concern now, but there's nothing we can do (I think?)
We need to return a new object every time from combineReducers (otherwise we'd be mutating state ourselves :-)
Am I missing your point?

Am I missing your point?

Absolutely not :D. I am the one that completely missed that non-mutating state meant that we have to change the reference of the state at each update (instead of just returning new slices inside the same state as I thought initially).
Thanks a lot for your clarifications!


Question: How to create an action using a custom Ubuntu docker image

Hi all, I am new to openwhisk, my goal is to create an action using a custom ubuntu docker image, to be more specific: I would like to run some bash scripts within the custom ubuntu docker image.
I read the docs for a while but I haven't found a good solution yet, could any of you give me some advices?

Hi, I build a image using this runtime-docker Currently, I just put a exec file that output a json string using echo and does not modify anything else. But when I create an action and then invoke, it always timeout and then it will report something like "error Request accepted, but processing not completed yet."

There's not information here to help you. You can confirm that your container image is properly set up by running a copy of the container locally and using https://github.com/apache/openwhisk/tree/master/tools/actionProxy to test the image.

thanks, I already solved that.

Great @Tim-C1 - if you can leave a brief comment for what the fix was it might be helpful for future reference.

Sure! Actually this has nothing to do with openwhisk, I follow the dockerskeleton repo and do some modifications to the Dockerfile to build a custom ubuntu image, and it works. The reason why I can not run create and invoke an action is that I first run bash in the image I built then commit it, and this commit will override the default CMD specified in the Dockerfile which is used to the start actionProxy server required by Openwhisk.


Failed to load 'aufs' module

Hi,
Do you know how to fix this error I get when trying to run your portainer script in proxmox shell?
[ERROR] 1@56 Failed to load 'aufs' module
Many thanks!

Add “aufs” or “overlay” (or both) to /etc/modules-load.d/modules.conf in your PVE host and reboot. Check it with lsmod | grep -E

Tthe aufs module is deprecated as of Proxmox VE 7.0. See this article.
create_container.sh needs to be updated and remove the line:
load_module aufs


[ClangImporter] Don't put forward-declared structs in the lookup table.

Otherwise, the lookup table for "CGColor" has two entries, because of this:
typedef struct CGColor *CGColorRef;

and that interferes with our ability to import things as members of "CGColor" (as opposed to "CGColorRef"), which affects the fix-its we generate when you try to use the non-member form.
This isn't necessarily the best long-term solution (as noted in the FIXME) but it is expedient and won't break any current users.
Once this is in, update the CoreGraphics API notes to use non-"Ref" forms for its import-as-member renamings. Since we print the type name in diagnostic text and in fix-its, it's important to use the name that Swift sources use.
rdar://problem/26347297

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

Discussed with @milseman and @DougGregor; this seems like the best way to handle this for now.
@swift-ci Please test

…guess now's not a good time to land patches. Will wait for a clean trunk.

@swift-ci Please test

Linux failure doesn't appear related:
Foundation/NSGeometry.swift:39:25: error: value of type 'NativeType' (aka 'Double') has no member '_toBitPattern'
return Int(self.native._toBitPattern())
~^~ ~~~~~~~~~~~~~

Yeah, but I'll wait for a clean trunk anyway.

@swift-ci Please test

@jrose-apple , this looks great. Thanks!

LGTM. Thanks for getting this in!

Linux failure is in Foundation (unrelated). Merging.


php7 support

composer will now install under php7

Is not working yet.

@rogervila it does :)
in your composer.json:
"repositories": [
    {
        "type": "vcs",
        "url": "https://github.com/younes0/laravel"
    },
]

"require": {
    "backup-manager/laravel": "dev-patch-1"
}

@younes0  thank you!

Thanks. =)


How are complex numbers used?

The complex(a) constructor takes a single argument. In python it's two. How is this meant to work? Is the 1.0j style used and accepted by the compiler?
@sydow guess this is primarily you but maybe @nordlander has an idea?

@mzagozen FYI. Feel free to add on info / questions


chore: Create a data mapper for the 4 main Verax objects

What does this PR do?
Add a DataMapper for the 4 main Verax objects
Related ticket
Fixes #259
Type of change

[X] Chore
[ ] Bug fix
[ ] New feature
[ ] Documentation update

Check list

[ ] Unit tests for any smart contract change
[ ] Contracts and functions are documented

Plus missing examples and implem for write methods

Why not, but it feels like this PR will cover everything at once, no?

Codecov Report

Merging #260 (bf5a2db) into dev (1ddb29f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #260   +/-   ##
=======================================
  Coverage   93.40%   93.40%
=======================================
  Files          13       13
  Lines         197      197
  Branches       46       46
=======================================
  Hits          184      184
  Misses          8        8
  Partials        5        5

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


M3-4489: CMR Header for Object Storage > Buckets & Object Storage > Access Key

Description
Apply CMR Header to Object Storage Access Key and Buckets landing pages
Type of Change

Non breaking change ('update', 'change')

To-Do:

[x] Access Key page

Unrelated, there's a console warning happening here, probably for reasons similar to #6876. Could either be fixed here or filed in a separate ticket.

Thanks @johnwcallahan, I will take care of the console warning issue in a separate ticket & PR.


fix postinstall script

In the CI and locally I see an error related to the husky postinstall script.
➤ YN0009: │ zk-kit.noir@workspace:. couldn't be built successfully (exit code 1, logs can be found here: /private/var/folders/vn/7xm7vmd962x2rg3whjzgp_7r0000gn/T/xfs-c35a9669/build.log)

and then log file shows
Usage Error: Couldn't find a script named "husky".

I am surprised because we are following the official husky setup instructions, aren't we?

@sripwoud we are. Are there any issues about this in their repo? Are we having the same issue in other repos?

Sorry, I had overlooked it is #8 that introduces a breaking change of the husky version.
https://github.com/privacy-scaling-explorations/zk-kit.noir/pull/8#discussion_r1731663068
We can close this


Enable password protected SSH keys.

Fixes: https://github.com/Azure/azure-cli/issues/1773

Refactored to avoid the parameter which I decided felt wonky.

Codecov Report

Merging #2044 into master will decrease coverage by -0.04%.
The diff coverage is 29.41%.

@@            Coverage Diff             @@
##           master    #2044      +/-   ##
==========================================
- Coverage   72.33%   72.29%   -0.04%
==========================================
  Files         323      323
  Lines       18273    18287      +14
  Branches     2701     2702       +1
==========================================
+ Hits        13217    13220       +3
- Misses       4221     4232      +11
  Partials      835      835

Impacted Files
Coverage Δ

...li-acs/azure/cli/command_modules/acs/acs_client.py
54.9% <29.41%> (-5.33%)
:x:

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 2b77c3c...bac7197. Read the comment docs.

What about using the SSH agent?


stdout cluttered with No such file or directory even when that directory is ignored

I am using ag to search something in an old project.
I need to enable following through symbolic links and so I use the -f option.
Couple of directories in that old project have dead symbolic links and I can't delete those. So I added the below to the project root's .agignore:
# Directories
old_dir/

The problem is that when I try
\ag -f 'regex'

from the project root, I get the below on the stdout:
ERR: Skipping ./old_dir/dead_link: Error opening file: No such file or directory

How can I avoid that?
Shouldn't the file existence check be skipped if it's one of the ignored files or if it's in an ignored directory?
I have build ag on RHEL 5.10 using the latest from master today and ag --version gives me
ag version 0.30.0

Features:
  +jit +lzma +zlib

@kaushalmodi, I was able to workaround the issue by putting the .agignore in the same directory as the ag search i'm running.  I also used a relative path to the directory.
ls -la
drwxr-xr-x   6 nromito  staff      204 Jun 13 02:03 .
drwxr-xr-x+ 71 nromito  staff     2414 Jun 13 02:03 ..
-rw-r--r--   1 nromito  staff       23 Jun 13 02:03 .agignore
-rw-r--r--   1 root     staff  1375942 Jun 13 01:11 dtrace.out
drwxr-xr-x   3 nromito  staff      102 Jun 13 01:00 old_dir
drwxr-xr-x   2 nromito  staff       68 Jun 13 01:01 test

cat .agignore
# Directories
old_dir/

The problem seems to be that we match the path in the .agignore against the relative path from where the search was initiated:
DEBUG: path_start . filename old_dir
DEBUG: pattern .DS_Store doesn't match path old_dir
DEBUG: pattern ~/tmp/old_dir/ doesn't match path old_dir
DEBUG: pattern *~ doesn't match file old_dir
DEBUG: pattern .DS_Store doesn't match path old_dir/
DEBUG: pattern ~/tmp/old_dir/ doesn't match path old_dir/
DEBUG: pattern *~ doesn't match file old_dir/
DEBUG: old_dir not ignored

The same thing happens when there is a directory structure deeper than the default value of 25:
ERR: Skipping ./pccsstorefront/web/grunt/node_modules/grunt-contrib-imagemin/node_modules/imagemin/node_modules/imagemin-jpegtran/node_modules/jpegtran-bin/node_modules/bin-wrapper/node_modules/download/node_modules/gulp-decompress/node_
modules/decompress/node_modules/decompress-tarbz2/node_modules/tar-stream/node_modules/readable-stream/node_modules. Use the --depth option to search deeper.
ag did not obey to .gitignore, but the workaround of putting an .agignore in the folder from where I was performing the search solved the issue.

Would be nice to get this fixed...


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>

:tada: This PR is included in version 7.7.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Conflict with MultiplayerHLAPI

It seems this conflicts with the MultiplayerHLAPI package. There may also be other Unity packages that conflict due to a Mono.Cecil package that gets added.
The way I figured out how to fix this is to Remove the Mono.Cecil dll's from FrameRateBooster and instead use the ones provided via package manager by creating an asmdef in the Optimizer.cs Editor folder.
In the asmdef, you need to enable Override References and add Mono.Cecil.dll as an Assembly Reference.
"overrideReferences": true,
"precompiledReferences": [
    "Mono.Cecil.dll"
]

The README would need to be update to say that you must install the Mono Cecil (nuget.mono-cecil) package.
Would you like me to submit a PR for this?

Hi
Thanks for reporting this.
Last time I checked this issue, I found that MPHLAPI was referencing a Mono.Cecil dll it gets from a package from Unity's package manager, but said package was not available to the public. If it was, then it would have solved the issue in a very nice way, since I could reference the same package.
Now, just to be sure I understood what you said: is the Mono.Cecil.dll you are speaking about coming from Unity's package manager or from VS's Nuget pcakge manager? If it is from Unity's package manager, I could not find it. Can you please tell me how to get it?
Thanks

It is still a package intended for internal use<EMAIL_ADDRESS>However, you can just manually add the following to the manifest.json:
{
    "nuget.mono-cecil": "0.1.6-preview"
}

If this was setup as a UPM package, you could list it as a dependency. I played around with trying to get this to work as a UPM package, but ran into problems with the assembly not getting created or not being created in the right order.

Thanks for the explanation. I will not include this change in the Asset Store version of the asset until I find a way to add the package automatically from code. Having an asset not building out of the box, even if everything is explained in the ReadMe/asset description, is a sure way to get 1 star reviews.

While I do agree, it currently does not build out of the box if you have any UPM packages installed that include the nuget.mono-cecil package.
I think part of the issue can be alleviated by including an asmdef for the Editor/Optimizer.cs that lists the Mono.Cecil reference (see original post). This way, if the nuget.mono-cecil package is installed, you could just not import the Mono.Cecil libs included in this asset, and then everything should "just work".

Thanks for the explanation. Please submit the PR and I will take a look at this in the upcoming days.

@skaughtx0r I tried a solution to automatically add the package, using the Package Manager's API, so that users don't have to do a thing. Unfortunately, it is not working reliably. The WIP is available here:
https://github.com/ehakram/FrameRateBooster/tree/WIP_Mono.Cecil_Conflict
I explain (and ask for help) what is not satisfying with this solution here (in the publishers unity forum):
https://forum.unity.com/threads/what-is-the-best-way-to-automatically-install-needed-packages.1083362/
If you don't have access to it, here is the main part:

When a user imports my assets, and thanks to how I setup the assembly definitions, the importer will build first, then the rest of the code that needs the package will build second. That second part will always fail since the package is missing. So the users will get build errors for a moment.

The Client.Add is called too often. I wanted to use the result of Client.Search to see if the asset is already installed in this project before trying to add it, but I can't find how. Neither SearchRequest.Result.resolvedPath nor SearchRequest.Result.resolvedPath.status seem to be a reliable way to get that information

Sometimes, the compiler errors from code needing the package stop the execution of Installer. Some other times, it does not.

It all boils down to this: how to reliably execute some editor code when there are compiler errors.
Any input is welcome.

In the end, I just replaced usage of a mono.cecil dll with its source code. This does not lead to conflicts with nuget.mono-cecil package.
Fixed in commit acb03697ab76f0250e704b422f10f3c482f78eaf

Thanks, sorry I didn't get back to you sooner to look at your earlier changes. I'll try out your latest commit when I get a chance.


Audio must be mono

When runnning , i am getting the error below
File "/usr/lib/python2.6/site-packages/SpeechRecognition-1.1.4-py2.6.egg/speech_recognition/init.py", line 74, in enter
assert self.CHANNELS == 1 # audio must be mono
I confirmed that the input audio is mono.

Can you upload the audio somewhere for testing purposes? I have not been able to reproduce this.

https://www.dropbox.com/s/bhunw6r3ptqzjw7/Notorious BIG - Biggie - Dead Wrong (Acapella).wav?dl=0

I'm not sure how to check whether or not my Wav file is Mono or Stereo, but I'm getting the same error.

Hi @RasJay,
Your file is definitely stereo:

A future version of this library might convert the audio to mono automatically, but for now you can convert the audio to mono manually using tools like Audacity.

Just for reference, sox original_file.wav -c1 mono_file.wav, will convert the file to mono.
Original file can be mp3 or just whatever sox recognizes.

b16101acf9bce667aac0f19c1371cabc5980497e adds support for stereo audio!


Poll timeout

Hi @mhowlett @edenhill
We have a .net core kafka consumer reading messages from a specific topic. The message flow in that topic is very high millions of messages per minute. We are repeatedly getting poll.ms.timeout error. Seems like librdkafka provides messages in batches but when out application is processing one by one and batch size is huge it is taking time more than the configured poll time and eventually timeouts. Can you please suggest how to handle this.
current polltime is 300000 ms which is too much , we dont want to just keep on increasing.
Any suggestions will be appreciated.
thanks

librdkafka provides messages to the application one by one (not in batches), and you must call consume at least as frequently as max.poll.interval.ms, else you will get this error. if an individual message takes longer than this time to process, you could do that in a different thread and pause the consumer, but you must still call consume at least as frequently as max.poll.interval.ms, even though paused. it sounds like that is not the case though if you are doing millions of messages per minute. it sounds more likely an issue with usage that can be fixed, but i can't comment without looking at the code.

librdkafka provides messages to the application one by one (not in batches), and you must call consume at least as frequently as max.poll.interval.ms, else you will get this error. if an individual message takes longer than this time to process, you could do that in a different thread and pause the consumer, but you must still call consume at least as frequently as max.poll.interval.ms, even though paused. it sounds like that is not the case though if you are doing millions of messages per minute. it sounds more likely an issue with usage that can be fixed, but i can't comment without looking at the code.

Hi @mhowlett - if the message is returned one by one, there is no way processing of that message takes more than max.poll set. It is generally in few ms(50 ms). What can be wrong then. Snippet of code . Please suggest

Hi @mhowlett - if the message is returned one by one, there is no way processing of that message takes more than max.poll set. It is generally in few ms(50 ms). What can be wrong then. Snippet of code . Please suggest

Hi @mhowlett - can you please suggest and help.
thanks

Hi @mhowlett - can you please suggest and help.
thanks

@tech7857 we run a loop that declares itself alive like this (note when doing this, we shift the actual work to worker threads, i.e. the thread that does the Consume shifts to the Pause/Consume/Resume loop, while work continues in a different threadpool thread, which will ultimately result in the work queue being drained and consumption resuming)
@mhowlett I'd appreciate if you could advise if there is any better way of doing such placeholder Consume calls to 'touch' the Consumer without being handed a message (I'm concerned the current impl has race conditions in using the Assignment property etc)

@tech7857 we run a loop that declares itself alive like this (note when doing this, we shift the actual work to worker threads, i.e. the thread that does the Consume shifts to the Pause/Consume/Resume loop, while work continues in a different threadpool thread, which will ultimately result in the work queue being drained and consumption resuming)
@mhowlett I'd appreciate if you could advise if there is any better way of doing such placeholder Consume calls to 'touch' the Consumer without being handed a message (I'm concerned the current impl has race conditions in using the Assignment property etc)

@bartelink - thanks for the reply. Can you provide a sample code for this implementation. The other thing is if messages are received one by one, there is no way that message processing can take more than 100 ms. So i am not able to understand why this error will come unless i am missing anything. Please let me know if i am missing something.
thanks

@bartelink - thanks for the reply. Can you provide a sample code for this implementation. The other thing is if messages are received one by one, there is no way that message processing can take more than 100 ms. So i am not able to understand why this error will come unless i am missing anything. Please let me know if i am missing something.
thanks

Hi @mhowlett can you please help and provide some guidance.

the support workload for confluent customers is high atm, they have precedence, and we have people sick. will get back to OS issues later.

@mhowlett - can you please guide on this. Your help will be greatly appreciated.

@tech7857 the issue remains open so no need to assume anything. MHowlett is one person; he has clearly stated his constraints and I would urge you to respect that. Bottom line: if you are a confluent customer, there is a front door; if not, go read the code and/or ask on stack overflow or other places that are geared for answering questions and have tonnes more viewers than this medium

Have you timed the application to make sure it is calling consume()? I've only ever seen that error when consume() is not called. Like mhowlett said you must continue calling consume(), even if you can't process the message. Failing to call consume() also means you won't be notified if kafka gives your assignments away, which could lead to parallel processing of a single message if you have multiple consumers.
Since pause() has the unfortunate side effect of flushing messages in librdkafka's queue, what I've done to work around this problem (consumer is slow / gets stuck for a bit) is to use a queue. When you call consume(), you write to the queue, and if it's full, tell the consumer to seek back to the message you've just consumed.
In psueodo code,
while (!stoppingToken.IsCancellationRequested)
{
    try
    {
        var result = _consumer.Consume(500);
        if (result is null)
            continue;

        if (!queue.TryWrite(result))
        {
            consumer.Seek(new TopicPartitionOffset(result.TopicPartition, result.Offset));
            Thread.Sleep(500);
        }
    }
    catch (ConsumeException ex)
    {
        // log error
    }
}

This approach means you must disable enable.auto.commit, otherwise it will commit messages that are in the queue but haven't been processed yet. The queue can be processed message by message or in batches, I used System.Threading.Channels. I'm also curious if there are better fixes for this problem.


dartanalyser in WebStorm breaks for one specific project in 1.25.0-dev.9.0

See also https://youtrack.jetbrains.com/issue/WEB-28189

I just tried 1.25.0-dev.10.0 with the same result.
I'm still stuck on 1.25.0-dev.8.0 to get analysis results in my project.

Are there steps I can take to try to reproduce the problem?

@bwilkerson I just got an event in IntelliJ where clicking the link opened https://github.com/dart-lang/sdk/issues/new, but the content was just added to the URL bar instead of the input field as usual, this is why the content below is truncated. Not sure if this is of any help.
I don't know how to reliably reproduce this event. I played a bit around to see if something changes, and then the event popped up.
https://github.com/dart-lang/sdk/issues/new?body=Analyzer%20Feedback%20from%20IntelliJ# Version information- `IDEA IU-172.3544.35`- `1.25.0-dev.9.0`- `IU-172.3544.35, JRE 1.8.0_152-release-915-b6x64 JetBrains s.r.o, OS Mac OS X(x86_64) v10.12.6 unknown, screens 1920x1200, 3360x1890  Retina`# ExceptionPlease append the contents of:file:///private/var/folders/k0/l96hhr317hgcfkw1p58qzr880000gp/T/report.txt```Dart analysis server, SDK version 1.25.0-dev.9.0, server version 1.18.3, FATAL error: Failed to handle request: {id: 3, method: analysis.setAnalysisRoots, params: {included: [/Users/zoechi/dart/bwu_product/bwu_product_server], excluded: [/Users/zoechi/dart/bwu_product/bwu_product_server/.idea, /Users/zoechi/dart/bwu_product/bwu_product_server/.pub, /Users/zoechi/dart/bwu_product/bwu_product_server/build]}, clientRequestTime:<PHONE_NUMBER>856}NoSuchMethodError: The method 'configure' was called on null.Receiver: nullTried calling: configure(analysisOptions: Instance of 'AnalysisOptionsImpl', sourceFactory: Instance of 'SourceFactoryImpl')#0      Object._noSuchMethod (dart:core-patch/object_patch.dart:42)#1      Object.noSuchMethod (dart:core-patch/object_patch.dart:46)#2      ContextManagerImpl._checkForPackagespecUpdate (package:analysis_server/src/context_manager.dart:926)#3      ContextManagerImpl._handleWatchEvent (package:analysis_server/src/context_manager.dart:1407)#4      _rootRunUnary (dart:async/zone.dart:1128)#5      _CustomZone.runUnary (dart:async/zone.dart:1012)#6      _CustomZone.runUnaryGuarded (dart:async/zone.dart:909)#7      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:330)#8      _BufferingStreamSubscription._add (dart:async/stream_impl.dart:257)#9      _ForwardingStreamSubscription._add (dart:async/stream_pipe.dart:132)#10     _ForwardingStream._handleData (dart:async/stream_pipe.dart:98)#11     _Forwardi

I added you as collaborator to the project on GitHub https://github.com/zoechi/bwu_product.
If you check it out and open the project root folder or the bwu_product_server directory, you should be able to reproduce the behavior where no analyzer results are shown.

@zoechi Thanks!
https://codereview.chromium.org/3001023002/

@bwilkerson curious what was special with this project to cause this issue?

I don't know. I got to the point of realizing that we start watching the file system before creating the analysis driver and that therefore it's possible for the driver to be null in the methods that handle watch events, and didn't look to see what, if anything, might have made this project different.

@bwilkerson thanks. I just saw that you didn't even need to check out my project.

Is it possible this is still not included in 1.25.0-dev.13.0 ?

If I follow the link to the code review, look for the link following "Committed:" and follow that, I can see that the commit was included in both 1.25.0-dev.13.0 and 1.25.0-dev.12.0. If it's still not working, then we should re-open this issue.

I'm not sure if it's the same issue. The behavior is a bit different. But AFAIR there were a Dart-plugin update in the meantime.
I get
https://github.com/dart-lang/sdk/issues/new?body=Analyzer%20Feedback%20from%20IntelliJ# Version information- `IDEA IU-172.3757.52`- `1.25.0-dev.8.0`- `IU-172.3757.52, JRE 1.8.0_152-release-915-b10x64 JetBrains s.r.o, OS Mac OS X(x86_64) v10.12.6 unknown, screens 1920x1200, 3360x1890  Retina`# ExceptionPlease append the contents of:file:///private/var/folders/k0/l96hhr317hgcfkw1p58qzr880000gp/T/report1.txt```Dart analysis server, SDK version 1.25.0-dev.8.0, server version 1.18.3, error: Exception while using a Visitor to visit a FormalParameterListImpl in GenericFunctionTypeImpl in SimpleFormalParameterImpl in DefaultFormalParameterImpl in FormalParameterListImpl in FunctionExpressionImpl in FunctionDeclarationImpl in CompilationUnitImplNoSuchMethodError: The getter 'name' was called on null.Receiver: nullTried calling: name#0      Object._noSuchMethod (dart:core-patch/object_patch.dart:42)#1      Object.noSuchMethod (dart:core-patch/object_patch.dart:46)#2      Visitor.checkIdentifier (package:linter/src/rules/non_constant_identifier_names.dart:48)#3      Visitor.visitFormalParameterList.<anonymous closure> (package:linter/src/rules/non_constant_identifier_names.dart:61)#4      Object&ListMixin.forEach (dart:collection/list.dart:63)#5      Visitor.visitFormalParameterList (package:linter/src/rules/non_constant_identifier_names.dart:59)#6      FormalParameterListImpl.accept (package:analyzer/src/dart/ast/ast.dart:4811)#7      ExceptionHandlingDelegatingAstVisitor.visitNode.<anonymous closure> (package:analyzer/src/dart/ast/utilities.dart:2756)#8      List.forEach (dart:core-patch/growable_array.dart:267)#9      ExceptionHandlingDelegatingAstVisitor.visitNode (package:analyzer/src/dart/ast/utilities.dart:2754)#10     UnifyingAstVisitor.visitFormalParameterList (package:analyzer/dart/ast/visitor.dart:3020)#11     FormalParameterListImpl.accept (pac

in Dart 1.25.0-dev.13.0 (the same in dev 11, 10, 9)
but works fine in Dart 1.25.0-dev.8.0
(not sure what broke the Open issue submission form? link in IntelliJ, since a while I only get the truncated content of the issue report in the URL bar. Perhaps it is only with this specific error. I didn't have others since a while)

I'm not sure what version exactly I got above output (I forgot that I switched Dart versions several times and didn't pay attention at what version the event was added to the "Event Log" window).
With Dart 1.25.0-dev.13.0 currently no imports are recognized and I get a weird error at the library declaration

Sorry, the library declaration was my fault.
After deleting the ~/.dartServer directory. I got a lot less of the unrecognized imports.
But after opening different files, the list in the "Dart Analysis" window is growing again.
The error just don't come up immediately after restarting the analyzer.

So just to be clear, I think this is the behavior you're seeing. When you open IntelliJ (or restart server directly), you get a few diagnostics that are not valid, but as you write code the number of invalid diagnostics increases. Do I have that right?
If so, are they increasing monotonically, or do they sometimes go away and then come back again?

I think this is just the on-disk cache that wasn't filled immediately. After having opened more and more files, the list of errors grew to until all files of one of the 3 projects were listed, and than stayed the same. After restarting the analysis server the full list appears immediately, the same as before I deleted the disk cache. (the errors are all invalid and go away when I switch back to dev-8)

Are all / most of the invalid errors related to URI resolution (which would include things like undefined names that would have been defined if URIs had been resolved correctly)? If so, are they all package: URIs?
If so, can you check that there are .packages files where you'd expect them at check one of two of the unresolved URIs to see whether the package location is correct in the .packages file?

Yes, it's only package imports. Relative imports are recognized.
The packages file points to the right directory. It also is working perfectly fine once I switch back to dev-8.


AsyncAPI

Bi-directional support for AsyncAPI.
Consideration will be included for various adapters:
https://github.com/asyncapi/bindings

We are commencing a design and research stage of this project. If you are interested in contributing to our research, prototypes and proposals please consider joining our Developer Preview Program and we will soon be reaching out to contribute.


Release/0.8.2

@yangkairui Can you explain the purpose of this PR?  It looks like you may have opened it by mistake.  Without a summary of purpose we will have to close it.

All commits from release/0.8.2 should already be in master.


[kokkos] Work around performance issue by using only 'unsigned long' in AtomicPairCounter

Better workaround than #309, see https://github.com/kokkos/kokkos/issues/4780 for more details.
In addition, this PR adds support for using Kokkos' profiling tools via the KOKKOS_PROFILE_LIBRARY environment variable. (functionality that we were missing because of heavily customized initialization of Kokkos).

Mhm, this doesn't seem to be working as intended, at least in my test on a GTX 1080 Ti:
master
taskset -c 0-15 ./kokkos --cuda --numberOfThreads 16 --numberOfStreams 16 --maxEvents 20000
Processing 20000 events, of which 16 concurrently, with 16 threads.
Processed 20000 events in 2.791611e+01 seconds, throughput 716.432 events/s, CPU usage per thread: 62.2%

master + #309
taskset -c 0-15 ./kokkos --cuda --numberOfThreads 16 --numberOfStreams 16 --maxEvents 20000
Processing 20000 events, of which 16 concurrently, with 16 threads.
Processed 20000 events in 2.793876e+01 seconds, throughput 715.851 events/s, CPU usage per thread: 62.4%

master + #333
taskset -c 0-15 ./kokkos --cuda --numberOfThreads 16 --numberOfStreams 16 --maxEvents 20000
Processing 20000 events, of which 16 concurrently, with 16 threads.
Processed 20000 events in 4.445962e+01 seconds, throughput 449.846 events/s, CPU usage per thread: 61.1%

Though I'm still at my first coffee, so I cannot guarantee there weren't any mistakes...

I ran similar tests (1 thread with 1k events, and 16 threads with 20k events) on
RTX 2080 SUPER
master
Processing 1000 events, of which 1 concurrently, with 1 threads.
Processed 1000 events in 3.812683e+00 seconds, throughput 262.282 events/s, CPU usage per thread: 113.2%
Processing 20000 events, of which 16 concurrently, with 16 threads.
Processed 20000 events in 4.519340e+01 seconds, throughput 442.543 events/s, CPU usage per thread: 51.1%

master + #309
Processing 1000 events, of which 1 concurrently, with 1 threads.
Processed 1000 events in 1.846951e+00 seconds, throughput 541.433 events/s, CPU usage per thread: 122.8%
Processing 20000 events, of which 16 concurrently, with 16 threads.
Processed 20000 events in 1.564639e+01 seconds, throughput 1278.25 events/s, CPU usage per thread: 72.0%

master + #333
Processing 1000 events, of which 1 concurrently, with 1 threads.
Processed 1000 events in 1.848977e+00 seconds, throughput 540.84 events/s, CPU usage per thread: 117.9%
Processing 20000 events, of which 16 concurrently, with 16 threads.
Processed 20000 events in 1.646719e+01 seconds, throughput 1214.54 events/s, CPU usage per thread: 72.9%

GTX 1050 Ti
master
Processing 1000 events, of which 1 concurrently, with 1 threads.
Processed 1000 events in 7.080183e+00 seconds, throughput 141.239 events/s, CPU usage per thread: 96.4%
Processing 20000 events, of which 16 concurrently, with 16 threads.
Processed 20000 events in 1.020225e+02 seconds, throughput 196.035 events/s, CPU usage per thread: 54.9%

master + #309
Processing 1000 events, of which 1 concurrently, with 1 threads.
Processed 1000 events in 6.535374e+00 seconds, throughput 153.013 events/s, CPU usage per thread: 89.3%
Processing 20000 events, of which 16 concurrently, with 16 threads.
Processed 20000 events in 1.004504e+02 seconds, throughput 199.103 events/s, CPU usage per thread: 59.1%

master + #333
Processing 1000 events, of which 1 concurrently, with 1 threads.
Processed 1000 events in 6.933106e+00 seconds, throughput 144.235 events/s, CPU usage per thread: 87.1%
Processing 20000 events, of which 16 concurrently, with 16 threads.
Processed 20000 events in 1.102847e+02 seconds, throughput 181.349 events/s, CPU usage per thread: 58.9%

Earlier I had tested only on RTX 2080 and was happy with the #333 giving similar performance as #309. But my GTX 1050 Ti test reproduces the GTX 1080 result in https://github.com/cms-patatrack/pixeltrack-standalone/pull/333#issuecomment-1062629038, so this appears to be a real effect.

Here is a plot on V100 (~2 minutes running for each point, on the same CoriGPU node)

So on Volta both fixes work, but disabling the "new atomics"  yields a bit higher throughput for >= 3 concurrent events. Perhaps it would be best to go with #309 for now, and rebase this PR on top of that and leave it open for time being.


DBZ-7785 Add example-mariadb container

https://issues.redhat.com/browse/DBZ-7785

@Naros Applied, thanks


[SuperEditor] Caret overlay is not updated when document layout changes due to animations

Steps to reproduce:

Run the demo app
Change the build method of TaskComponent so that it will animate in a subtitle when the node has selection:

  @override
  Widget build(BuildContext context) {
    return Column(
      crossAxisAlignment: CrossAxisAlignment.start,
      children: [
        Row(
          crossAxisAlignment: CrossAxisAlignment.start,
          children: [
            Padding(
              padding: const EdgeInsets.only(left: 16, right: 4),
              child: Checkbox(
                value: widget.viewModel.isComplete,
                onChanged: (newValue) {
                  widget.viewModel.setComplete(newValue!);
                },
              ),
            ),
            Expanded(
              child: TextComponent(
                key: _textKey,
                text: widget.viewModel.text,
                textStyleBuilder: (attributions) {
                  // Show a strikethrough across the entire task if it's complete.
                  final style = widget.viewModel.textStyleBuilder(attributions);
                  return widget.viewModel.isComplete
                      ? style.copyWith(
                          decoration: style.decoration == null
                              ? TextDecoration.lineThrough
                              : TextDecoration.combine([
                                  TextDecoration.lineThrough,
                                  style.decoration!
                                ]),
                        )
                      : style;
                },
                textSelection: widget.viewModel.selection,
                selectionColor: widget.viewModel.selectionColor,
                highlightWhenEmpty: widget.viewModel.highlightWhenEmpty,
                showDebugPaint: widget.showDebugPaint,
              ),
            ),
          ],
        ),
        AnimatedSize(
          duration: const Duration(milliseconds: 100),
          child: widget.viewModel.selection != null
              ? const SizedBox(
                  height: 40,
                  child: Text('some metadata'),
                )
              : const SizedBox.shrink(),
        ),
      ],
    );
  }

Move the selection downwards direction

Expected: the caret is painted in the correct place
Actual: the caret is painted where it was expected to be, if there was no subtitle
https://user-images.githubusercontent.com/31278849/217527882-2707112d-5c3b-478e-91d7-f6c2231085c2.mov

Cc @matthew-carroll

@angelosilvestre I'm gonna take over this ticket because we're gonna need to make some deep alterations to the document layout.

The problem we're dealing with in this ticket is that, in Flutter, child size changes don't trigger rebuilds for ancestors. They do trigger re-layout for some ancestors, but not widget rebuilds.
We need the document layers to rebuild their widget tree when the document changes layout because the layers are likely to inspect the document layout and make positioning decisions based on that document layout inspection.
My plan to resolve this has two steps:
First, I'm going to create a proof of concept where we use SizeChangedLayoutNotifier to notify SuperEditor whenever any of its document layout changes size. Then SuperEditor will explicitly invalidate the layers on top of the document, causing them to rebuild.
This first approach has the downside that we're always 1 frame behind on the document layers.
If the proof of concept solves the fundamental problem, then I'll build a custom RenderObject that runs a build phase during its layout phase, in response to something else changing size.
Such a RenderObject doesn't work in general, because if this new RenderObject runs layout before the thing it's listening to, then we'll either still end up a frame behind, or we'll try to invalidate layout after we've laid out, which would cause Flutter to blow up. But, if we use this new RenderObject tactically, by making sure it appears AFTER the document layout in the widget tree, then it will always run layout after the document, allowing us to rebuild the layer widgets during the layout phase, which saves us from the extra frame problem.

Here's a Flutter issue for fundamental limitations related to this work: https://github.com/flutter/flutter/issues/123305


revert redis alpha

[redis] redis will use old redis service and db == 0 for all envs

 Follow up on PaaS in 10 weeks about Redis


[ISSUE-3503] build Docker Image bug fixed

[ISSUE-3503] build Docker Image bug fixed
What changes were proposed in this pull request
Issue Number: close #3503

Brief change log

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Does this pull request potentially affect one of the following parts

Dependencies (does it add or upgrade a dependency): (yes / no)

cc @tisonkun PTAL, thanks


Documentation error

We are referring to https://developer.ibm.com/tutorials/crisis-communication-chatbot-watson-assistant-webhook-integration-discovery-covid-data/ starter kit and we could find that the documentation has a typo.
We see twcApiKey and url parameters have the same description. The url here should have been the discovery service url.

@emilyannmitchell I opened a PR against the IBM Developer documentation. Please review / merge when you get a chance.

@johnwalicki Merged

Super - Closing ticket


feat: add manual links to device files

Provides the info to integrate a link to the manual, which will close #1551. @marcus-j-davies @robertsLando

@zwave-js-bot fix lint

@AlCalzone These should be ready to merge. The only change is adding the manual link, and anything else that normalizeConfig() caught.

Nice!! :)

I'm just going to revert and re-run without the normalizeConfig for now. I don't have time to go through them all either.

@zwave-js-bot fix lint

@AlCalzone If you can invoke the bot, these are ready to merge. I did a complete revert and removed the normalizeConfig, so we're only adding the manual link and then writing back.

@zwave-js-bot fix lint


FX historical data analysis

Hi,
This is my forex historical data :

row
Timestamp
Open
High
Low
Close
Volume

0
20000530 172700
0.9302
0.9302
0.9302
0.9302
0

1
20000530 173500
0.9304
0.9305
0.9304
0.9305
0

2
20000530 173800
0.9304
0.9304
0.9303
0.9303
0

3
20000530 174300
0.9301
0.9301
0.9300
0.9300
0

4
20000530 174400
0.9298
0.9298
0.9297
0.9297
0

All volumes are 0!
It has about 6.5 million rows!

when I run  "adding all features", I 'm getting this error :
df = ta.utils.dropna(data_df)
df = ta.add_all_ta_features(     df, open="Open", high="High", low="Low", close="Close", volume="Volume")

Traceback (most recent call last):
File "c:/Users/OMID/Desktop/forex/main.py", line 8, in
df = ta.add_all_ta_features(df, open="Open", high="High", low="Low", close="Close", volume="Volume" , fillna=True)
File "C:\Users\OMID\Desktop\forex\venv\lib\site-packages\ta\wrapper.py", line 307, in add_all_ta_features
df = add_volume_ta(df=df, high=high, low=low, close=close, volume=volume, fillna=fillna, colprefix=colprefix)
File "C:\Users\OMID\Desktop\forex\venv\lib\site-packages\ta\wrapper.py", line 72, in add_volume_ta
close=df[close], volume=df[volume], fillna=fillna).negative_volume_index()
self._run()
File "C:\Users\OMID\Desktop\forex\venv\lib\site-packages\ta\volume.py", line 274, in _run
self._nvi.iloc[0] = 1000
File "C:\Users\OMID\Desktop\forex\venv\lib\site-packages\pandas\core\indexing.py", line 669, in setitem
File "C:\Users\OMID\Desktop\forex\venv\lib\site-packages\pandas\core\indexing.py", line 2154, in _convert_to_indexer
self._validate_key(obj, axis)
File "C:\Users\OMID\Desktop\forex\venv\lib\site-packages\pandas\core\indexing.py", line 1993, in _validate_key
File "C:\Users\OMID\Desktop\forex\venv\lib\site-packages\pandas\core\indexing.py", line 2062, in _validate_integer
raise IndexError("single positional indexer is out-of-bounds")
IndexError: single positional indexer is out-of-bounds

where is the problem ?!

So, apparently "df = ta.utils.dropna(data_df)" is clearing all data in the data file. I was having the same problem and realized that and simply removed this line of code and it turned out to work fine. Give it a try


Tokenization in Korean, plus abjad languages

This branch adds support for Korean tokenization via MeCab. It now includes Korean and Japanese MeCab data files as subdirectories of wordfreq/data, instead of assuming the Japanese data is installed system-wide.
Mostly unrelatedly, it also supports tokenizing other languages written with abjad scripts, such as Hebrew and Persian, though there is no frequency data for these languages yet.

Hm, I accidentally included an unrelated change to wordfreq_builder. Let me untangle that.


Problem with arrow keys

Hi,
with an Image selected, if I press down, left, right or up keys, application freeze.
Can you check it?
Thank you.

Hi,
I've been checking this issue and it looks like it's a problem with the focus change when using arrow keys and specifically when trying to focus on the adorner that exists on the selected image or any other drawn object actually. I found a solution for the moment, but I will investigate further to see if I can't find something better. Using this property KeyboardNavigation.DirectionalNavigation="Once" in  MainWindow.xaml will fix the problem.
I will push the fix in some days, but I am writing this comment in case you need this fix sooner.
Thank you for using the library and finding this issue.

Ok it works! Thank you!


fix showItemInFolder return undefined

Description

修复由于 remote.shell.showItemInFolder(fullPath) 没有返回值(undefined)，导致总是提示错误的BUG
Related Issues

Related to #575
Checklist:

[ √ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[ √ ] Have you linted your code locally prior to submission?
[ √ ] Have you successfully ran app with your changes locally?

showItemInFolder 的问题是将 Electron 从 4.x 升级到 7.x 之后出现的，因为 Electron 官方这里搞了个 break change，拿不到调用结果了
https://github.com/electron/electron/pull/17121
由于最近忙着一直没发布新版 Motrix，主要是我还想要调用时传入不存在的 path 能报错，所以这个问题拖着还没处理掉


Missing key in OfferSliceSegment type

Hi,
it seems that the stops key is missing in the OfferSliceSegment type.
Can this be added in the next update?

Thanks for raising this. We will update this shortly!

This is now available in 1.26.0


feat: List.insertP and lemmas

List.insertP is a generalized, tail-recursive version of the insert step in insertion-sort. Sample use:
def insertionSort [LT α] [DecidableRel (α:=α) (· < ·)] (l : List α) :=
  l.foldl (fun s a => s.insertP (a < ·) a) [] -- fully tail recursive and stable!

It will be useful to streamline and optimize ac_rfl once moved to Std.
PS: Perhaps someone has a better name?

The ac_rfl move has been canceled so there is no immediate need for this. It's still a useful addition so I'm reluctant to close but closing might be better than rotting.


[TPC-H] Dask query 11

Part of https://github.com/coiled/dask-engineering/issues/234
Works with dask.dataframe  and dask-expr

@hendrikmakait I think this is good to go, failures are not related.


fix goldenVsFiles

goldenVsFiles behaves funny when more than one files has changed: it first complains about the first file only. Deleting the corresponding .expect file causes it to report that a new golden file has been generated (without an error). Finally, the third run reports a mismatch in the second golden file.

Fixed by 5e6d3f8e6cf307ac301868089b08f65dc788dcc8


Add hgrid column moving samples

The custom styling for the Blazor column-moving-styles sample doesn't seem to be applied for some reason.


Move 'tailscale up' to its own step.

So tailscaled doesn't run with the authkey in its env. It doesn't need to know, so don't provide it. Defense in depth.

(untested)

I guess CI kinda tests it!

CI needs one more thing: its authkey has expired... and I'm aware of the irony in that. I'm working on giving it a new fresh authkey, so we can see it pass.

Gave it an updated authkey.

yay it passed

Hah, @maisem's approval is no good here?

Hah, @maisem's approval is no good here?

Our usual administrator group had not been set up in this repo, though honestly I don't understand why that would make a difference. Nonetheless I just added it.


Select web driver

Add 'geb.driver' parameter to allow configuration of web driver implementation.
This will resolve #1.

@mojavelinux I somehow lost permissions on the screenshot repo. I think I kicked myself out of the screenshot team. Could you help me getting the permissions back. thx.

Fixed!

@sclassen who no merge?

Because of the error you are getting I discovered that setting the variable geb.driver is not sufficient for some of the drivers. They need more configuration. I am having a look at making usage of the geb environment mechanism to allow for more flexibility. Hopefully I will find some time to do this in the next days.

I did some researching today. Here the outcome:

using geb.driver is very limiting in that it does not allow to configure the driver and only 4 driver classes are supported
using the geb configuration. This is the most flexible approach. One can configure almost every aspect of geb. There are 2 drawbacks

one needs to include the config file in the classpath of asciidoctorj
one cannot change the browser during conversion of an asciidoc file

using geb.env does not gain much benefit over the plain configuration. The way geb.env works is that it effects the loading of the config file. Once the file is loaded changing geb.env has no effect.

It looks to me as if using the geb configuration is the simplest and most flexible approach.
I will try to write some examples to show how this can be setup.

I try to implement screenshot or geb within my asciidoc, but building oens just Firefox and then I get a timeout (newer FF problem with selenium).
Any hint or example how to use geb with chromedriver? "it can handle a groovy script" is not that helpful when you have no clue from groovy. ;)
Tried the continous-documentation code from Alex Soto too, but no luck either.
Thanks in advance.


Incorrect npm version on Mac OS X

Having installed node via Homebrew with the --without-npm option and then npm, I have the following setup which works fine:
ixav:~ xp$ which node
/usr/local/bin/node
ixav:~ xp$ node --version
v4.2.1
ixav:~ xp$ which npm
/Users/xp/.node/bin/npm
ixav:~ xp$ npm --version
3.7.2
ixav:~ xp$ which npm-check
/Users/xp/.node/bin/npm-check

but when I run npm-check, i get this:
ixav:~ xp$ npm-check -g

npm   😎  MAJOR UP  Major update available: https://docs.npmjs.com/
                   npm install --save<EMAIL_ADDRESS>to go from 2.14.7 to 3.7.2

Use npm-check -u -g for interactive update.

Not sure where this 2.14.7 version comes from...

Interesting issue!
I use https://github.com/jonschlinkert/global-modules to figure out the path for your global modules. Can you see if that module gives you the correct path? If it doesn't, create a bug for Jon and mention this issue so I can follow the progress.

@dylang: this seems to be a known bug in @jonschlinkert's global-prefix. see Issue

yes, sorry. had the flu this past week, just getting caught up. this issue is near the top of my list. I should be able to fix in the next day or two.

Most likely related to #87 so closing this one just to keep things clean.


Randomize DoAuto timeouts

Having fixed timeouts here can create some kind of pattern in long run which (potentially) could be used by malicious participants (to disrupt one's mixing process for example). Shouldn't be an issue if number of mixing users is large enough but we are not there yet ;)

utACK


Add animation

Add animations when entering the webpage, when clicking on a link and for going from the TOC to the corresponding section

Add exit/2 bullets above the coffee cup


v3.1 Feature (Profile): FireStore setup for profile data

We need to display

Best performing topic (topic with highest score)
most frequently played topic
No of games played
No of topics played
~No of top 1 results in public (trophies)~ (change it to in topics screen, highlight the card that this player is the top scorer)
~No of topics contributed (quick access to directly play)~

UI:

Toggle for control anonymity of player's nickname visible by other players (when in /levels viewing topics)

Database:

played topic information
firebase function auto update some metadata (# of times, etc...)
firebase function auto update public topic metadata (what is the top score?)

Partially implemented by #225
Left features to implement to close:

[ ] /levels page toggle to show best performing player for each topic
[ ] Toggle for control anonymity of player's nickname visible by other players (when in /levels viewing topics)


Cover selectors with types, using Selector

A large chunk of our remaining code that isn't covered with Flow types (#2052) is because most of our selectors don't have appropriate types.
Most of them, it seems like Flow ought to have the information to infer (see #2724 for details). That's likely a problem in Flow, but we can make it not be our problem; we're probably best off writing down explicit types on most of the selectors anyway, for maximum readability.
I worked out a type alias Selector recently that takes care of the complexity in the selector interfaces, and makes this very straightforward: Selector<T> means a selector that returns a T. Let's use that.
Here's an example:
export const getStreamFromId = (streamId: string): Selector<Stream> =>
  createSelector([getStreams], (streams, params) => {

I believe this is now complete! :tada: Thanks to all the contributors who helped out.


Add nitin to BOSH groups

Hi there,
I have been a contributor on BOSH (~2015-2016) and stemcell (on-going for compliance).
I am now a part of the BOSH team.
Cheers,
Nitin

For reference the contributions of @xtreme-nitin-ravindran:
Foundational Infrastructure: VM deployment lifecycle (BOSH) Contributions
PRs Commented on/Reviewed:
Issues that may be relevant:

2016-08-10T13:22:44Z: fixes formatting using parted on centOS
2016-08-30T16:28:13Z: Update progressbar version from 0.9.2 to  0.21.0
2017-08-23T22:09:39Z: Add basic user-add_windows
2017-09-14T16:13:04Z: Update stemcell version to 3445.7 in bosh-dev.yml
2019-05-03T21:52:05Z: [#165802999] Add missing CIS/STIG audit rules and update auditd.conf
2019-05-30T23:25:52Z: Add CIS and STIG audit rules
2019-09-20T20:29:39Z: STIG: Enable cron logging
2019-09-26T22:54:27Z: CIS: Ensure suspicious packets are logged
2019-09-27T18:53:18Z: STIG: pam related hardening rules
2020-02-28T02:00:47Z: [#168964744] make rsyslog and logrotate configs consistent
2020-02-29T02:40:03Z: [#171553414] add CIS, STIG hardening for ssh
2020-03-11T18:41:38Z: [#171751912] lock password if crypted_password is not set
2023-03-23T20:36:25Z: job to add/update windows registry
2024-04-25T08:16:21Z: Noble pam_lastlog is deprecated its succesor is pam_lastlog2
2024-07-09T21:53:18Z: add missing director roles
2024-07-10T18:34:20Z: comment for extra permissions in integration tests

Code contributions:

2015-09-22T17:52:10Z: Bump bosh-agent
2015-09-22T18:34:17Z: Fix failing test due to mocking file directory creation
2015-10-13T13:56:15Z: [#104119296] Fix unit and integration tests, and make tests compatible with new 1.5 vendoring
2015-10-13T14:39:05Z: [#104119296] bumping go agent repo
2015-10-15T19:56:18Z: [#104125746] Director can be deployed with default gateway settings
2015-10-20T19:07:21Z: bump agent
2015-10-20T19:40:06Z: allow passwordless sudo for users created for bosh-ssh
2015-10-20T19:40:06Z: allow passwordless sudo for users created for bosh-ssh
2015-10-20T19:52:58Z: adding bosh_sudoers group to os image
2015-10-21T13:56:43Z: bump ubuntu and centos OS image
2015-10-21T16:39:47Z: bump agent
2015-10-21T16:41:54Z: allow passwordless sudo for users created for bosh-ssh
2015-10-21T20:03:43Z: bump agent
2015-10-23T18:06:53Z: monit should not start any jobs before pre-start is done
2015-10-23T19:46:15Z: refactor of monit start
2015-10-23T20:36:33Z: remove agent start error logging for old invalid case
2015-10-23T20:39:29Z: bump agent
2015-10-26T20:57:06Z: refactored to avoid using @cloud_plugin_is_provided flag
2015-10-30T15:27:26Z: bosh-init raises an error if compiled release does not match given stemcell os / version
2015-10-30T19:39:26Z: Fix a typo in the stemcell os and version comparison
2015-10-30T19:57:34Z: Add one more test to check for incorrect stemcell os and versions for compiled releases
2015-10-30T20:12:30Z: Make sure that the "extract release" method is called for compiled releases
2015-10-30T20:20:30Z: Fix test that makes sure non-cpi compiled releases are being extracted
2015-11-02T14:57:14Z: bosh-init raises an error if cpi release is compiled
2015-11-03T20:35:50Z: refactor. moved monit status related stuff from start & stop actions to
2015-11-03T22:39:39Z: removed stopped file exist check before it's deletion (in
2015-11-06T15:50:01Z: - changed root device number detection a bit to be more readable
2015-11-06T22:12:11Z: bosh-init can use non-cpi compiled releases
2016-08-10T13:19:25Z: fixes formatting using parted on centOS
2016-08-30T16:24:42Z: Update progressbar version from 0.9.2 to  0.21.0
2017-09-14T16:12:26Z: Update stemcell version to 3445.7 in bosh-dev.yml
2019-05-03T21:32:26Z: [#165802999] add missing audit rules
2019-05-30T23:24:31Z: Add CIS and STIG audit rules
2019-09-13T17:54:36Z: [#168486244] enable cron logging
2019-09-20T20:26:29Z: clean up stig_spec
2019-09-26T22:24:43Z: CIS: Ensure suspicious packets are logged
2019-09-27T18:45:05Z: [#168812689] add pam related hardening rules
2019-09-27T18:45:05Z: [#168812689] add pam related hardening rules
2019-09-27T18:46:26Z: add login.patch
2019-09-27T18:46:26Z: add login.patch
2019-09-27T18:51:00Z: [#168812689] add pam related hardening rules
2019-09-27T18:51:11Z: add login.patch
2019-10-04T17:20:47Z: CIS: sync 50-default.conf and logrotate sttings
2019-10-18T16:18:05Z: Revert "CIS: sync 50-default.conf and logrotate sttings"
2020-02-28T01:57:24Z: [#168964744] make rsyslog and logrotate configs consistent
2020-02-29T02:37:50Z: [#171553414] add CIS, STIG hardening for ssh
2020-03-11T18:22:12Z: [#171751912] lock password if crypted_password is not set
2023-03-23T20:29:34Z: job to add/update windows registry
2024-06-05T22:28:06Z: Handles EFI stemcells in mount-image helper
2024-06-10T21:16:44Z: Manully remove jumpbox/director vars stores
2024-07-09T19:48:23Z: correctly set custom gcp role to be used by int test
2024-07-10T18:30:17Z: comment for extra permissions in integration tests
2024-07-18T18:59:40Z: fixup imports
2024-07-18T19:54:39Z: Make tests more explicit
2024-07-18T19:55:31Z: RemoteManager returns errors when closing created shell

@beyhan: Done

IIRC @rkoster is on vacation so I'm merging. Welcome back @xtreme-nitin-ravindran  as approver!


Handling subgroup checks

Currently in the draft there are several key places we need to do subgroup checks on points, for example to validate a public key post de-serialization. However in some places we are implicitly relying on the octet_to_point function to perform this role and other places we are doing it as an explicit step in the procedure. We need to make this consistent, the options are

Consistently rely on the octet_to_point_g* implementation and add a security consideration that documents this function MUST perform the relevant subgroup check.
Explicitly document anywhere in the procedure where a subgroup check is required.

IMO 2 feels like a safer but more verbose option to me.

Labeling as pending close ahead of WG call on the 4th of july

Discussed on WG 8th August, there is potentially some inconsistencies that we need to address before we can close this issue

Discussed on WG call 20th of September, we believe the major inconsistencies around this check are resolved, closing for now and will re-open if some where missed.


RD widgets

Adds "height" event to switchboard to resize the height of the plyfe-widget <div> after the widget loads.

this should be bumped to version v2 for testing in parallel


[Enhancement]: "use Illuminate\Http\Response;" instead of  "use Response;" because phpstan throws error

Can we use the below suggestion in the controllers or wherever it is being used?
"use Illuminate\Http\Response;" instead of  "use Response;"
I am using PHPStan to analyze the code and use Response; fails.

I put it to the ignoredErros list for now but it would be great to do so. BTW, I tried to update stubs but it broked things and I don't remember. But I can reproduce it and provide information about that if you don't know it yet.

"Illuminate\Support\Facades\Response;" instead of "use Response;"

Fixed and not relevant since we are not using the response() function in v5.0.0


[BUG] tf_efficientnetv2_xl_in21k doesnt load

Describe the bug
unable to load efficientNetV2 with xl weights
To Reproduce
Steps to reproduce the behavior:

timm.create_model('tf_efficientnetv2_xl_in21k', pretrained=True, in_chans=3, num_classes=1)
fails to load the model with error:

RuntimeError                              Traceback (most recent call last)
 in
----> 1 timm.create_model('tf_efficientnetv2_xl_in21k', pretrained=True, in_chans=3, num_classes=1000)
~/.local/lib/python3.6/site-packages/timm/models/factory.py in create_model(model_name, pretrained, checkpoint_path, scriptable, exportable, no_jit, **kwargs)
76         create_fn = model_entrypoint(model_name)
77     else:
---> 78         raise RuntimeError('Unknown model (%s)' % model_name)
79
80     with set_layer_config(scriptable=scriptable, exportable=exportable, no_jit=no_jit):
RuntimeError: Unknown model (tf_efficientnetv2_xl_in21k)
Expected behavior
loading large:

timm.create_model('tf_efficientnetv2_l_in21k', pretrained=True, in_chans=3, num_classes=1000)
EfficientNet(
(conv_stem): Conv2dSame(3, 32, kernel_size=(3, 3), stride=(2, 2), bias=False)
(bn1): BatchNorm2d(32, eps=0.001, momentum=0.1, affine=True, track_running_stats=True)
(act1): SiLU(inplace=True)
...
(classifier): Linear(in_features=1280, out_features=1000, bias=True)
)

Desktop (please complete the following information):

OS: centO, running jupyter

@emoen check your version of timm, it works fine with curent master branch and pypi release. You have an old one in your environment

version I had: timm 0.4.12

I am getting the same issue with maxvit_rmlp_base_rw_224.sw_in12k from here. I have also checked and I am using timm version 0.6.12

I am getting the same issue, and solved it with pip install timm --upgrade.