Please implement hiprog/loprog

Please implement the hiprog and loprog opcodes, which are useful for those of us who use SFZ live and want to have more than 16 sfz instruments available, and/or to simplify programming our master keyboard to select an instrument using a patch button on the master keyboard.  Without this, I'll have to stick to Sforzando.
Ideally, this keyword should be scoped to the <global>, <master>, or <group> section in which it appears.

I see at https://sfz.tools/sfizz/development/status/opcodes/?v=2 it says "not supported by design."  (It also says that for lochan/hichan.)  I'd be interested to hear why.
Seems to me it should be fairly simple to implement, factoring into the filter for a region.

For my purposes, hichan/lochan are superfluous. I can manage MIDI channels just as easily in the live host or DAW, and frankly I find it better to do it there and let the player run in omni mode. MIDI program is different.

That's exactly the point actually. It's conceptually simple but kind of multiplies the size of some data structures by a factor of 16 which is wasteful in 99% of the situations SFZ is used for. In most cases as you said it just makes more sense to do channel dispatching at the host level. There's also really nothing stopping anyone from implementing for sfizz what the ARIA player is to sforzando.
loprog/hiprog is probably less costly because it seems like it's only an additional playing condition, but since I don't really know how program works and how people use them I decided not to support this.

For efficient, seems some developers is using internal converter trick.
My MIDI controller also has its own 'Extended MIDI CC' like we got in SFZ.
For example, the Mono Aftertouch or Channel Pressure message, is implemented as CC 144 in this device :

As we know we have Mono Aftertouch as CC 129 in SFZ, which is the same concept.
And it also treated Program Change message as CC 145.
So which I guess the trick is, use an alias as a CC number, then transmitted it as message that compatible for Program Change.
That probably can save quite amount of unnecessay CPU resources maybe?
In reverse process for sfz, we receive the Program Change message and read it as CC message then apply that to a new CC message (said CC 170 or CC 5000). And then implement locc5000 - hicc5000 as alias to  loprog' and hiprog`
I wonder if sforzando also use this kind of trick...

Thanks Kinwie, though I'm not sure I follow it all!
Paul, here's some background on prog from an old guy.  It dates back to hardware keyboards.  A keyboard would have a row of buttons for choosing the sound (program).  And usually a "bank change" method so that if there are only 8 buttons you can access more than 8 programs.  This appeared with the first digital keyboards, prior to MIDI.  When MIDI appeared, they just included this behavior, so that if I choose program 10 on my master keyboard, and if my slave keyboard was configured to receive prog change messages, that would also choose program 10 on the slave.  Therefore, pushing one button on the master keyboard could affect the sound for that keyboard and any slaves.  VERY helpful for live use, to push one button rather than one on each keyboard, for a given song or passage.
Here's how I use it today.  I currently have one keyboard, a Yamaha CP4.  I use a number of its internal sounds, plus I have it connected to a Windows laptop (that folds back on itself so it sits on my music rest with keyboard hidden.)  The CP4 has 16 patch keys that are programmable, so that whenever I select one it selects any internal sounds (and key split/layer) and also selects the output for MIDI for that patch, which can be different for different zones.  For each zone (I usually need only one), I can program the MIDI channel and the MIDI program number.
On the laptop, for most MIDI channels, I have one plugin setup to respond to that channel.  These are my "bread-and-butter" sounds.  Of these, 3 aren't usable for unimportant reasons, and 3 are assigned to Hammond, so I have 10 left over.  But I want more than 10 sounds.  So, I put all the "extra" sounds (ones that I won't layer with each other) in one big SFZ and use the MIDI prog feature to select the sound.  Personally I always have hiprog and loprog set to the same number.  Using ranges allows clever tricks for layering, but I use "clever" as a pejorative: like write-only code IYKWIM.

I was aware to what written at sfizz page about loprog/hiprog that "Not supported by design" note. I was making my own guess that it might be because of so many MIDI features was stripped-out from VST3 format, which might caused MIDI Program Change can't be supported in sfizz. Of course I might be totally wrong as it just a wild random guess.
So I just gave some examples, using trick like how it seems done on my MIDI controller. Treated MIDI ProgChange like a regular MIDI CC and implement a way to read it as ProgChange.
My Midi controller has a dedicated ProgChange buttons along with the LSB/MSB (Bank select) buttons to do the GM style presets selection. But it's a two-step way pressing. So you press the Up/Down (+/-) ProgChange button and then Enter to confirm the selection. This way, the Bank Select message also need to be matched (Bank number and Perc/Melodic patch number). Sounds complicated hahh! Just like the old GM keyboard way.
But I can assign a new CC to a rotary knob, so it can control the ProgChange with just one-time turn of that knob and with no LSB/MSB affected. It just change the Program number regardless the Bank selection. It's a cool additional hack, imho.
So, from that above MIDI CC list I posted previously, I assign MIDI CC 145 (which is the ProgChange's extended CC number in my keyboard) to a knob and it transmitted that CC message as ProgChange message to be received by sforzando.
*I tested it with a SFZ GM I made years ago, the Fluid R3 GM SFZ bank
Never mind... I just show-off :p

So #1058 has loprog/hiprog, but apparently midi program changes and VST3 do not mix well. In LV2 it works fine. I don't know which one you use @jlearman .

Thanks!  I tried VST3.  I haven't tried any LV2 plugins yet, but maybe now is the time.
Just curious, is this a flaw in VST3 definition, or just an issue with Sfizz implementation?
In any case I am satisfied with this resolution.

Yes VST3 basically removes alot of the MIDI "Legacy" things. It's hard to do a MIDI only plugin in VST3 afaik...


3D reconstruction result is not good

Hi,
I followed your direction to run 3D reconstruction on south-building dataset. I got the mean_reproj_error: 1.400.
However, when I rebulit the same scene using COLMAP with default settings, I got the mean_reproj_error: 0.591.
I do not understand why there is such a huge gap between these two results. I think the 3D reconstruction result of superpoint+superglue should have been better, as accurate 3D model can do good to image relocalization.
Looking forward for your help.

These numbers are expected. Other metrics should be looked at, such as the track length or the number of 3D points. We do not claim that SP+SG is better than SIFT for all SfM metrics, especially with this easy dataset. Keypoint detectors that do not have subpixel localization have a larger reprojection error, which is indeed detrimental for localization (see this paper).

Thanks for your reply.
The other metrics also perform worse, such as number of 3D points and mean track length. I do not report these statistics as I think the detected feature number is different.
Besides, I conduct sfm on large-scale Aachen outdoor dataset following your direction.  The SIFT model has the following statistics: Points 1652687; Mean track length 6.387; Mean reprojection error 1. And the SP + SG model has the following statistics: Points 1310593;  Mean track length 6.116; Mean reprojection error 1.452. For 3D reconstruction, SIFT model performs totally better. I do not understand how can a less accurate 3D model lead to better image pose estimation.
Thanks a lot for your attention.

As far as I know, on Aachen the SIFT model was obtained with exhaustive matching while here the SP+SG model uses only the 20 most covisible image pairs - so the results are not really comparable. You can also improve the results by resizing the images to a higher resolution, as done in the dev branch for Aachen v1.1.
Regarding your question: for day-night localization, invariance to illumination changes is more important than 3D model accuracy. SP+SG might produce 3D models that are less accurate than SIFT, but it is much more invariant and robust than SIFT, and thus overall better suited for day-night localization. There are some recent works (Dusmanu et al., Zhou et al.) that refine keypoints and obtain more accurate 3D models, which further improves the localization accuracy.

Thanks a lot for your very detailed response!


CSP violation in modern builds

Version
v2.4.0
Reproduction link
https://codesandbox.io/s/v271z453
Steps to reproduce

Enable modern builds and CSP in the configuration
Load the webpage
Check the devtools console for a CSP violation

What is expected ?
The Content-Security-Policy is expected to be satisfied.
What is actually happening?
A CSP violation is occurring.
Refused to execute inline script because it violates the following Content Security Policy directive: "script-src 'self' 'unsafe-eval' 'sha256-Q8VZ3JY1e/6NFowuA+3pjpynK6AzVNhbH31DIqVDFVM='". Either the 'unsafe-inline' keyword, a hash ('sha256-4RS22DYeB7U14dra4KcQYxmwt5HkOInieXK1NUMBmQI='), or a nonce ('nonce-...') is required to enable inline execution.

Notes
It's better to test locally than in the codesandbox. Codesandbox seems to have additional CSP errors that don't happen in a local environment.

This bug report is available on Nuxt community (#c8558)

!function(){var e=document,t=e.createElement("script");if(!("noModule"in t)&&"onbeforeload"in t){var n=!1;e.addEventListener("beforeload",function(e){if(e.target===t)n=!0;else if(!e.target.hasAttribute("nomodule")||!n)return;e.preventDefault()},!0),t.type="module",t.src=".",e.head.appendChild(t),t.remove()}}();

the script is appended to body and its hash are sha256-4RS22DYeB7U14dra4KcQYxmwt5HkOInieXK1NUMBmQI=.
It is workaround for Safari 10.1 nomodule bug. See Safari 10.1 nomodule support.
in v2.4.3, here appends the script.
Although we might not modify CSP header in webpack plugin, renderer appends another CSP script hash. Should we append there?

And current workaround is to add hash in nuxt.config.js.
export default {
  render: {
    csp: {
      hashArgorism: 'sha256',
      policies: {
        'script-src': [
          "'sha256-4RS22DYeB7U14dra4KcQYxmwt5HkOInieXK1NUMBmQI='" // this line resolves the violation
        ]
      }
    }
  }
}

@il-m-yamagishi good catch! Would you like to submit a PR?

@il-m-yamagishi any progress on the PR?
Also wouldn't having the option to disable inline-scripts (currently possible in vue-cli https://github.com/vuejs/vue-cli/issues/2570) mitigate this issue and expand on the http2 capabilities of Nuxt?

I'm also looking into something related to this issue, I'm wanting to hash or nonce the scripts and styles appended to the template so I can get CSP rules to stop blocking resources, how is this going and if there is a way how can I check and edit it?
Thanks!

As @il-m-yamagishi said here, this is likely related to the safari "nomodule" fix.

Adding the hashes doesn't work for me.

I am having the same trouble, its ruin my life.
18842af40923cd48144b.js:2 Refused to connect to 'https://my-api.com/api/tag/' because it violates the following Content Security Policy directive: "default-src 'self' 'unsafe-inline'".  Seems entire vendors.app.js are made of eval() so probably all will fail:

I use Helmet CSP on the server/index.js, and I imagine adding 'unsafe-eval' to script-src would "fix", but then, isn't that a bad thing (adding eval as acceptable)?

Is there any progress? Without this being fixed I cannot use a newer version than 2.9.2.

Up! :)

I suppose it might be related to source map, could you try to use source-map in devtool ?
https://nuxtjs.org/api/configuration-build/#extend

I just opened a pr a fix, could you please try if it solves your problems ?

      hashArgorism: 'sha256',

Should this not be "hashAlgorithm"?


fix: Name of undefined error in datasource catch Block

Description

fix for name of undefined error

Fixes #6195
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Manually
Checklist:

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

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: fix/name-of-undefined-datasource

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
54.96 (-0.01)
36.63 (-0.02)
34.68 (0)
55.47 (-0.01)

:red_circle:
app/client/src/sagas/DatasourcesSagas.ts
13.29 (0)
2.33 (-0.07)
8.7 (0)
15.75 (0)

:red_circle:
app/client/src/selectors/commentsSelectors.ts
83.61 (-1.64)
61.76 (-2.95)
73.33 (0)
88.24 (-2.35)

:green_circle:
app/client/src/utils/WorkerUtil.ts
89.76 (0)
72.55 (1.96)
100 (0)
93.33 (0)

:red_circle:
app/client/src/utils/autocomplete/TernServer.ts
52.71 (-0.23)
40.83 (-0.84)
36.21 (0)
56.74 (-0.25)

/ok-to-test sha=605fb1962a2d91a49cb1fb1b42cabaf2095a7f94

/ok-to-test sha=c15a1ea

/ok-to-test sha=c15a1ea

/ok-to-test sha=60904c6

/ok-to-test sha=bf3b140

/ok-to-test sha=a181258


I can't hit the notes of a song

Describe the bug
When playing a song that I converted (Blocked by the Vs Dave mod) only the first notes are hittable for a strange reason..
Error message
No error message
Screenshots:
I will attach a video if I can
Update: no, I can't sorry, my pc lags af when trying to record
Program version:
v1.1.2
Platform:
Windows 11
.exe file
Used files:
Files.zip
Additional context:

Never seen notes likes this, that why the program tries to make long notes with negative length instead of normal notes.
[
  86799.0,
  4.0,
  -86799.0
],

To fix, I'll consider that when you have a negative value as the 3rd argument of a note (supposed to be the length of the note), I'll consider it as a normal note (length = 0)

Fixed with 1.1.3 release.


.csv file busy

Implement a loop that's wait for the file to be close or create a new one, thus avoiding the loss of data.

Or it could be great to ask the user the name of the csv file and keep asking until it's not a busy file. With a warning if the file already exists

Fixed in #14


How to prevent the children redraw when pop to a TabBarView?

In a TabBarView,when navigate to a new view,then pop,the TabBarView will redraw the TabBarView's children and call initState function.
Here's my demo.
flutter-zhihu_zhuanlan.zip
Thanks flutter,it's very wonderful

Please consider asking support questions in one of the other channels listed at http://flutter.io/support .
Just cache the result of the code in initState so that it doesn't matter when initState is called again.
If you need more details I'd suggest you ask on StackOverflow.


Add second page about education

These changes build on top of #1 with a page about coursework.

This PR should only have commits relating to the education page now (reformatting commit was taken out)


feat: Init changeset

#55
#54  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @AhmedBaset and the rest of your teammates on  Graphite


Make it possible to build charm++ git without autotools

Original date: 2019-05-06 21:52:26
Original PR: https://charm.cs.illinois.edu/gerrit/5166

Adds the autotools-generated files to the repo, so they don't need to be generated by the user. Also splits off the script to rerun the autotools from the package tarball creation, such that the autotools-generated files can be refreshed without creating the tarball.
Change-Id: Iba0b6afb34069950f54c38cb5ef9d431c6d7adcd

this is ready for review

Can you elaborate on how this avoids the need of using autotools? It seems like it just does the same thing as before, just spread over multiple files now.

Basically, this PR adds the files generated by the autotools to the repo, so they don't need to be generated by the user. It also splits off the script to rerun the autotools from the package tarball creation, such that the autotools-generated files can be refreshed without creating the tarball.
I'll add this to the commit message.

Basically, this PR adds the files generated by the autotools to the repo, so they don't need to be generated by the user. It also splits off the script to rerun the autotools from the package tarball creation, such that the autotools-generated files can be refreshed without creating the tarball.
I'll add this to the commit message.

Ah, gotcha. Thanks for the explanation.

this doesn't seem to include a way to automatically determine when these should be rebuilt. It would be handy to have something automatically warn us that a change has necessitated these be rebuilt.

Yeah, I wouldn't merge this for now. It will be for the most part superseded by the cmake build system.

Should this be abandoned now that cmake has been merged?

Should this be abandoned now that cmake has been merged?

We'll still need a (potentially simplified) form of this patch. Even the cmake build currently requires autotools for the integrated hwloc.

This should also adjust the check for the presence of autoreconf and aclocal in build.

The presence check is only done if src/scripts/configure does not exist (which this PR adds). So I guess this doesn't have to be adjusted?

this doesn't seem to include a way to automatically determine when these should be rebuilt. It would be handy to have something automatically warn us that a change has necessitated these be rebuilt.

I've added a check to determine if configure is out of date, and rerun autoconf in that case.

#2661 eliminated the need for autotools when using the CMake build system. Standard practice for autotools build systems is to require them to be run when building from version control. Only tarballs will come pre-generated (see package-tarball.sh).


Async Clipboard table disappears shortly after page load (Cannot read property 'sourceData' of null)

If I view this page on Chrome (v86) or Firefox: https://caniuse.com/async-clipboard then the compatibility table appears for a moment and then goes away. Here's the table if I stop the page loading before it gets a chance to disappear:

And here's an error from the console:

Yes, this feature was never officially published which is why it's not working as expected. Looks like it was added quite some time ago but fell off my radar as a feature to review. The issue #4680 exists for that.

Yep, third result for me here https://www.google.com/search?q=can+i+use+async+clipboard

Curious... thankfully searches for other unpublished features don't show up so maybe that one was briefly there and got caught by the googlebot. Anyway, I've started researching the feature so hopefully will get it updated and appearing on the site properly soon. Thanks for reporting!

Alright, added now and should be working as expected.

https://caniuse.com/async-clipboard
It's not working

https://caniuse.com/async-clipboard
It's not working

error message
ciu-option-tag.ts:66 Uncaught (in promise) TypeError: Cannot read property 'sourceData' of null
    at HTMLElement.getFeatTitle (ciu-option-tag.ts:66)
    at HTMLElement.makeTitle (ciu-option-tag.ts:79)
    at HTMLElement.render (ciu-option-tag.ts:89)
    at HTMLElement.update (lit-element.ts:293)
    at HTMLElement.performUpdate (updating-element.ts:774)
    at HTMLElement._enqueueUpdate (updating-element.ts:724)

error message
ciu-option-tag.ts:66 Uncaught (in promise) TypeError: Cannot read property 'sourceData' of null
    at HTMLElement.getFeatTitle (ciu-option-tag.ts:66)
    at HTMLElement.makeTitle (ciu-option-tag.ts:79)
    at HTMLElement.render (ciu-option-tag.ts:89)
    at HTMLElement.update (lit-element.ts:293)
    at HTMLElement.performUpdate (updating-element.ts:774)
    at HTMLElement._enqueueUpdate (updating-element.ts:724)

Ugh, looks like I reverted the fix soon after I fixed it... fixed again now.

Ugh, looks like I reverted the fix soon after I fixed it... fixed again now.

Ugh, looks like I reverted the fix soon after I fixed it... fixed again now. Thanks for pointing that out @carnoxen.

Comfirmed. Thanks.

Ugh, looks like I reverted the fix soon after I fixed it... fixed again now. Thanks for pointing that out @carnoxen.

Comfirmed. Thanks.


Document folder conventions

One of the ideas @juanazam and I have is to have some conventions on where to generate the object pages.
We would like to propose and document these conventions so everyone can take advantage of them.
Folder structure

Put your Page Objects in <root>/tests/pages/ folder
Put you custom components in <root>/tests/pages/components/ folder

An example could be
myApp/tests
└── pages
    ├── team.js
    ├── users.js
    ├── users-add.js
    └── components
        ├── search-box.js
        └── gear-menu.js

Naming conventions (?)
This is something we didn't discuss yet and we would like to hear ideas.
For simple routes like users.index it's easy to pick a name for the page object <root>/tests/pages/user.js, but when you have complex nested routes it's no so trivial to pick a name. It would be great to have some guidelines.

:+1: A good questions would be were to put those components which the developer wants to separate from the page to make it smaller but are not re used elsewhere, the components general folder might become a bit messy. I guess we could apply naming conventions or that case (as well as in pages themselves for nested routes).

#35 adds the last two blueprints needed to complete this feature.


Adds setup instructions to the homology modelling notebook

I've added:

A very very basic README file
A setup.md file that details what they should look at in the notebook
A call to install notebook in the setup phase
Fixed some typos

Seems fine...


[stdlib] Remove the unnecessary compactMap overload

This particular overload of compactMap was mechanically copied from the flatMap while implementing SE-0187, but as community members correctly noticed on the forum thread here, there is no code that we should be backward compatible with, sincecompactMap has only been introduced recently.
After applying this change, the following code is correctly ambiguous again:
[].compactMap { _ in nil } // error: 'nil' requires a contextual type

as opposed to:
[].flatMap { _ in nil } // r0 : [String] = []

Fixes: https://bugs.swift.org/browse/SR-7052

@swift-ci Please smoke test

@swift-ci Please Test Source Compatibility

Simplest solution ever :D

Cherry-picked into https://github.com/apple/swift/pull/14779 and https://github.com/apple/swift/pull/14804


Use Travis instead of Testling

Testling has been super flakey recently. I can't even figure out why it's not showing anything for new PRs.

We can use Zuul and Saucelabs. I'm on this.

fixed


Update Demo.java

Create a new release repo

I have looked at all lines of code

I have looked at all lines of code

okay

...


SS-788: fix performance issue loading profile details

Lazy load the profile detail tab components, only query for data once you click on the tab ui

Closing in favor of https://github.com/bcgov/sheriff-scheduling/pull/234


docker build error

I get this error when building with docker:
Sending build context to Docker daemon 190.5 kB
Sending build context to Docker daemon
Step 0 : FROM golang as builder
Invalid repository name (golang as builder), only [a-z0-9-_.] are allowed

nevermind it was because the docker version is too old
Docker version 1.7.1, build 786b29d/1.7.1


Base mixin

Maintaining the cleanest compiled output with cleaner input.

My PR's better :wink: https://github.com/mdo/css-output/pull/1
...because gzip size really makes the case for mixins.


[Feature] : time boxing

I was wondering if the features available in this mac app or this android app or this ios app could be implemented. Given that, timeboxing is similar to pomodoro as the two have countdown timers.
The features I'm interested in are :

To manually set duration for each tasks with the option of not using breaks in between.
The time when I'll finish the task based on the duration set (automatically generated)
Notification when the duration time is finished

There is a similar feature request in the day planner plugin, but there is no active development for it, so I thought maybe this plugin could do it.

Maybe you can use the dataview to calculate the behavior in the log. Use a template to automate statistics once a day

Of course, I think if this plugin can set the total duration of the task, and automatically count it. That would be a great feature. I thought maybe we could make a label. Something like a task.


Typo in repository About sidebar text

The About sidebar text of this repository:

Build animated charts in Jupyter Notebook and in many other environments with a simple Python synthax.

contains a typo: synthax needs to be syntax.

Thanks, fixed.


Why do the maven repos have scala 3 version of "core" but not "ext-scalatags"?

There is a Scala 3 version available for scalacss "core".
See here.
but for "ext-scalatags", there is only: this scala 2.13 version.
Since they are in the same source tree, I would have assumed they would both have Scala 3 versions.

Seems scala 3 did not compile this.
I tried to fix this in a version for me.
See: Fork of ScalaCSS


s2u-36 <IP_ADDRESS> Announcements: Ability to assign announcements to roles -

fix

Does this fix cover bullhorns and also emails ?

About email, I dont know, but the bullhorns yes

@JuanDavid102 Check emails please, I think they are complaining about emails.

Ok

@JuanDavid102 Please review the build, is not building well.

Ok. I'm trying now with the complete deploy, but it looks like there is no problem, in my localhost

Ok. I'm trying now with the complete deploy, but it looks like there is no problem, in my localhost

Do a push --force again

It looks like fail in

Okay, the build looks broken because of #11440 , we'll review that in a different PR.

Please, rebase with the latest changes from #11460 to check that builds fine, thanks!

This is very nice!


class JSONParser bug

[version 1.1.12]
java.lang.StringIndexOutOfBoundsException would be found when try to parse a Long value in class
com.alibaba.druid.support.json.JSONParser
there is bug in method  scanDigit()

This issue belongs to https://github.com/alibaba/druid


fix: delete duplicated trainee profile placements caused by missing ID.

TIS21-3171

We could add a test that the MongoException is caught rather than being thrown.


How do you make it work on Windows 7 Youtube-dl-gui / Youtube=dl

Windows 7 is EOL and no longer supported. Get off that.


Change docutils literal notranslate color

class="docutils literal notranslate" looks pink, and I've never really liked the color scheme. Nothing against pink, but I don't feel it belongs in technical documentation (nor would something like red unless in a big bold warning).

cc @PipKat for her expertise.

@akaszynski The pink was chosen because so many of the Python packages we reference use this color for literal tags--and it passed accessibility checks for readability in dark and light modes. @Revathyvenugopal162, I will let Tim Duffy, our UX expert know that Alex would like to see us use something else that also passes the accessibility checks. I couldn't tag Tim here, but I'll chat him the information.

Thanks @PipKat. Ideally something that's not link color (blue) or a header color (black, matches text).

@akaszynski I relayed this to Tim--but if you want inline literal tags be black text in a gray box, just like in code, Revathy can probably just implement it. We wouldn't have to wait for Tim's input. @Revathyvenugopal162?

@akaszynski @Revathyvenugopal162  Tim said that he thought we had already agreed to use black text in a gray box for literal tagging. I told him that we discussed so much in that meeting, all I clearly remember is that we all agreed that we didn't want to keep the gold! Anyhow, @Revathyvenugopal162, feel free to change the font color to black. Thanks!


Falco in 2020 (blog post)

Signed-off-by: Leonardo Di Donato<EMAIL_ADDRESS>
What type of PR is this?
/kind content
Any specific area of the project related to this PR?
/area blog
What this PR does / why we need it:
During my personal yearly self-review, I started reflecting back on Falco in 2020.
The number of things I remembered was so big that I had to separate the artifact in another blog post that is now suitable for the Falco blog, imho.
Which issue(s) this PR fixes:
NONE
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leodido after the PR has been reviewed.
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leodido after the PR has been reviewed.
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 483576f9af496f10b1a1fec41e14fd5effda976c

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 483576f9af496f10b1a1fec41e14fd5effda976c


Support datafusion functions true_unless_null,  invert and binary_length.

Can use these functions in the filter expression.

[x] true_unless_null
[x] invert
[ ] binary_length

Since binary_length is the only one remaining and is already in #898 i'm going to close this issue as completed


Automatically close row doesn't work

Describe the bug

My items that are rendered, aren't automatically closing when another one is opened, and aren't closing as soon as they are pressed (even when i manually set the options on the component). They all open and don't close until they are manually closed with a swipe in the opposite direction to the way they were opened

To Reproduce
Steps to reproduce the behavior:
Create a component with all the basic options, and use section list, with section list data.
Screenshots
If applicable, add screenshots to help explain your problem.

Environment (please complete the following information):

OS: [e.g. iOS or Android]
RNSLV Version: [e.g. v2.0.2]
RN Version: [e.g. v60.0.5]

@jainshreyp 99% of the time this is due to the objects not having a key property or your keyExtractor being implemented incorrectly. Can you please double check these two things and if you don't find any problems please post some example code I could look at, thanks!

Ahhh your right, forgot to pass my keyExtractor haha! Thanks for your help, sorry for not being more informed, I didn't see the keyExtractor on the swipe list (but i realized thats cuz u had the key property). Thank you!


Prevent Windows going to sleep while restic operations in progress

What does this PR change? What problem does it solve?
From #3216:

To save power my PC goes into sleep state after an hour of inactivity. However opertions like restic prune could last a lot longer.
Currently powercfg /requests does not show any request when restic is running.
I'd like that restic informs Windows not to go into sleepstates when restic is performing operations.

Some popular console apps like Wget and Rar already do this.
In this PR I prefer to use Power*Request API instead of XP-era SetThreadExecutionState: new API has a clear scope (owned kernel object instead of a global flag) and allows describing the reason of sleep blocking.
Was the change discussed in an issue or in the forum before?
Issue #3216
Checklist

[x] I have read the Contribution Guidelines
[x] I have enabled maintainer edits for this PR
- [ ] I have added tests for all changes in this PR
- [ ] I have added documentation for the changes (in the manual)
[x] There's a new file in changelog/unreleased/ that describes the changes for our users (template here)
[x] I have run gofmt on the code in all commits
[x] All commit messages are formatted in the same style as the other commits in the repo
[x] I'm done, this Pull Request is ready for review

Hi DRON-666 thanks for creating a pull request for my suggestion for preventing windows going to sleep while restic operations run.
Sorry I don't understand the code fully, so I have to questions:
const (
	PowerRequestDisplayRequired PowerRequestType = iota
	PowerRequestSystemRequired
	PowerRequestAwayModeRequired
	PowerRequestExecutionRequired
)

I see PowerRequestDisplayRequired in changed code. Does that mean display would stay on? If so: Display is not needed, I prefer turning it off.
If I understand correctly, the pull request just turns it on for prune operation. However a restic backup or restore could also take longer and so these operations should also be covered. So would be great if you could fix this. My suggestion. Just issue wake lock when starting restic and releasing when restic closes. Overhead should be minimal for operations like just printing help or version.

Hi DRON-666 thanks for creating a pull request for my suggestion for preventing windows going to sleep while restic operations run.
Sorry I don't understand the code fully, so I have to questions:
const (
	PowerRequestDisplayRequired PowerRequestType = iota
	PowerRequestSystemRequired
	PowerRequestAwayModeRequired
	PowerRequestExecutionRequired
)

I see PowerRequestDisplayRequired in changed code. Does that mean display would stay on? If so: Display is not needed, I prefer turning it off.
If I understand correctly, the pull request just turns it on for prune operation. However a restic backup or restore could also take longer and so these operations should also be covered. So would be great if you could fix this. My suggestion. Just issue wake lock when starting restic and releasing when restic closes. Overhead should be minimal for operations like just printing help or version.

I see PowerRequestDisplayRequired in changed code. Does that mean display would stay on?

This is just a declaration of constants, see lines 18-19 for their use.

Just issue wake lock when starting restic and releasing when restic closes.

That's how it's already done in this PR.

I see PowerRequestDisplayRequired in changed code. Does that mean display would stay on?

This is just a declaration of constants, see lines 18-19 for their use.

Just issue wake lock when starting restic and releasing when restic closes.

That's how it's already done in this PR.

If this were to be added, please do not make it on by default. Please give it a command line option to enable. On-by-default result in an unexpected behavior people IMHO.
For me, I want Windows to go to sleep during restic operations. Sleeping / saving power is more important to me than restic finishing a background operation. When I close my laptop lid, I expect my machine to go to sleep. If it doesn't, I can end up with unexpected situations such as my machine burning power while enclosed in a bag, overheating the machine, coming back to a laptop that has drained it's battery when I need it most, etc. Windows has enough problems preventing it from suspending properly, I'd like not to add another.
I personally think this should not be a feature of restic, but instead should be left to the person to implement something in a script to prevent Windows from sleeping. I understand that others may have a different viewpoint there, however.
FYI - Windows going to sleep during a backup (or other operation) doesn't cause any data loss, corruption or other errors in my experience.

When I close my laptop lid, I expect my machine to go to sleep.

This PR does not interfere with this behavior. Please read this:

There are three kinds of requests that can be used through the Power Request API: a system request, a display request, and an “away-mode” request. The first type requests that the system not automatically go to sleep due to the idle timer (although the user can still close the lid to enter sleep, for example), while the second does the same for the display.

For me, I want Windows to go to sleep during restic operations. Sleeping / saving power is more important to me than restic finishing a background operation. When I close my laptop lid, I expect my machine to go to sleep.

Thats a very good point and I'm quite sure, that closing the lid will still get windows into sleep mode.
For example I had the problem that firefox prevented automatic sleep after 60 minutes.

However when going to Start and selecting powersave PC immediately went to sleep mode, as expected. When closing the lid behaviour should be identical. So this PR should only change the requests. When users explicitly put PC into sleep mode like via closing the lid, then IHMO this is still respected.
PS: I prevented firefox from preventing my PC going into sleepmode via powercfg -requestsoverride process firefox.exe display execution system awaymode

Thanks for the explanation. I understand how it should behave. My worry around this issue is mostly that Windows power management sleep behavior has been pretty bad in the past. So much so that I've had supposedly well-behaved applications prevent multiple laptops that I manage from going to sleep when the lids are closed. This is has led me not to trust Windows power management. Given that, I'm generally opposed to making this change without giving the user the option to enable/disable this behavior through a command-line switch or environment variable. And my preference would be off by default.

So much so that I've had supposedly well-behaved applications prevent multiple laptops that I manage from going to sleep when the lids are closed.

Sorry, but I don't think this is very valuable information without a proof. And you can easily test this PR: download sources, compile go run build.go, execute dummy command restic init -r nul, check (from another console window) power requests powercfg -requests (you will see something like SYSTEM: [PROCESS] \Device\HarddiskVolume5\Restic\BIN\restic.exe Restic init in progress.) and finally close the lid.

Given that, I'm generally opposed to making this change without giving the user the option to enable/disable this behavior through a command-line switch or environment variable. And my preference would be off by default.

It makes no sense to add this feature disabled by default, and no one does it. I've given Rar and Wget as examples, but only because a direct link to the source code is available. Many other programs like players, network clients, media converters, installers, etc., use this feature, and I doubt they have a switch for that.

I've been pondering quite a bit about what to do with this PR (and the corresponding issue). In some cases it may be quite useful to let restic ensure that the system stays awake. But I also see the point that this behavior may be unexpected or even unwanted. Another such example I can think of would be the mount command (once it exists on windows). I don't restic should prevent the system from sleeping in that case.
In general my feeling is that whatever component is scheduling restic should also manage the wake lock and not restic. So, I'm inclined to close this PR.

Also, the performance/speed of prune has increased since the issue of this PR was opened, might have made the initial problem a bit smaller.

Also, the performance/speed of prune has increased since the issue of this PR was opened, might have made the initial problem a bit smaller.

Actually with compression performance of prune is quite slow, see https://github.com/restic/restic/issues/3869
However I'm not so sure anymore if restic really should prevent OS from sleeping. I have seen this behaviour with a lot of other programs and to be honest, I'm now quite annoyed of it. Especially Firefox with a lot of open tabs prevents system from sleeping which is not what I want and you need powercfg /requestsoverride to save energy, especially I'm using
[SERVICE]

[PROCESS]
firefox.exe DISPLAY SYSTEM AWAYMODE EXECUTION
MoUsoCoreWorker.exe DISPLAY SYSTEM AWAYMODE EXECUTION
JDownloader2.exe DISPLAY SYSTEM AWAYMODE EXECUTION

[DRIVER]
High Definition Audio Device SYSTEM
Realtek High Definition Audio SYSTEM

So I'm fine with closing this pullrequest.


Out of bounds CUDA error

Using compute-sanitizer, I encounter the error that I haved condensed into a log I provide.
error (1).log
The repro script requires LongLoRA's shifted short attention modifications.
the `llama_attn_replace.py`` file can be found here.
import torch
from transformers import AutoModelForCausalLM, AutoTokenizer
from datasets import load_dataset
from torch.utils.data import DataLoader
from transformers import AutoModelForCausalLM, AutoTokenizer

from llama_attn_replace import replace_llama_attn

replace_llama_attn()

tokenizer = AutoTokenizer.from_pretrained(pretrained_model_name_or_path="meta-llama/Llama-2-7b-chat-hf")
tokenizer.pad_token = tokenizer.eos_token

model = AutoModelForCausalLM.from_pretrained("meta-llama/Llama-2-7b-chat-hf", torch_dtype=torch.float16, device_map="auto")

model.resize_token_embeddings(len(tokenizer))
optimizer = torch.optim.AdamW(model.parameters())

dataset = load_dataset("databricks/databricks-dolly-15k", split="train", streaming=True)
dataset = dataset.map(lambda x: tokenizer([x["instruction"]], truncation=True, padding='max_length', max_length=4096, return_tensors="pt"))
train_dataloader = DataLoader(dataset)

model.train()

for step, batch in enumerate(train_dataloader):
    output = model(
        input_ids=batch["input_ids"].to(model.device).squeeze(1), attention_mask=batch.get("attention_mask").to(model.device).squeeze(1)
    )
    loss = torch.sum(output.logits - output.logits)
    loss.backward()
    optimizer.step()
    optimizer.zero_grad()
    print("step: ", step)

I'm running this on 8xA100s with CUDA 12.1 if that helps.
Let me know if more info or further experiments on my side are needed to sort this one out!
Thanks for providing this library.

Thanks for the bug report, let me investigate. Does it crash with the provided script? Do you know the shape of the tensors (batch, seqlen, nhead, headdim) that cause the crash?

Thanks for the quick response @tridao
It crashes with the provided script on my end.
I'll come back with the tensor shapes in a bit.

Just reran the code. It crashes on the first call.
The shapes are the following:
** qkv:  torch.Size([16, 3, 16, 128])
** cu_seqlens:  torch.Size([4])
** max_seqlen:  1024

@tridao Sorry for the ping, do you need more info?

I just tried to run the script and it didn't crash (i waited till step 400).
Do you have the cmd to run compute-sanitizer that produces that error log?

Thanks for responding @tridao !
I'm using. compute-sanitizer --target-processes=all python repro.py
Heads up: With the newest transformers commits, they also changed how they treat the attention mask in the flash attention 2/non-flash attention case is that there is now use a utility called _prepare_4d_causal_attention_mask, so setting
transformers.models.llama.modeling_llama.LlamaModel._prepare_decoder_attention_mask like LongLoRA does, has no effect now. You can just set transformers.models.llama.modeling_llama.LlamaFlashAttention2.forward instead of transformers.models.llama.modeling_llama.LlamaAttention.forward and enable flash attention 2 in the llama 2 config and it should be fine if you are running into this.

I can still reproduce with the script I posted and the newest HF transformers version btw.

Did you use the provided code as-is? I wonder what the cap between our runs of this script is.

I got it to crash now. However, it seems it's because the cu_seqlens are wrong.
I put a breakpoint to print out:
     79     breakpoint()
---> 80     print(cu_seqlens_q, cu_seqlens_k)
     81     out, q, k, v, out_padded, softmax_lse, S_dmask, rng_state = flash_attn_cuda.varlen_fwd(

ipdb> cu_seqlens_q
tensor([  0,   8, 520,  16], device='cuda:0', dtype=torch.int32)
ipdb> (q).shape
torch.Size([16, 16, 128])

In this case Q has total of 16 tokens. However, cu_seqlens_q is telling the flash-attention kernel that there are 3 sequences, from indices 0 to 7, from 8 to 519(?!?), then from 520 to 16 (?!?). This causes flash-attention kernel to try to read elements beyond the 16 tokens provided, and hence memory error.
I think the problem is in producing the cu_seqlens that are wrong.

Thanks for spotting this. This appears to be an issue with the LongLoRA forward function used here.


Error when use maxDate:  Uncaught RangeError: Maximum call stack size exceeded.

Here is the steps to reproduce:
<paper-date-picker min-date="August 10, 2016" max-date="January 13, 2019" date="Sep 20, 2016"></paper-date-picker>
Click the year number from calendar and choose "2019" from year list,  the  new date will be "Sep 20 2019".  The console will continue to output warning messages "Date outside of valid range: " (paper-calendar.html line 628).  After thousands of warnings,  a new error "Uncaught exception" will output ,  later another error "Uncaught RangeError: Maximum call stack size exceeded.".
This problem is coming from https://github.com/bendavis78/paper-date-picker/blame/master/paper-calendar.html#L629.

Fixed in 86ecebc


Retrain model with another parameters

Description
{"lstm_units": "122", "dense_1_units": "230", "activation": "relu", "learning_rate": 0.0001, "epochs": "18", "batch_size": "112"}
How has this been tested?
Please describe the tests that you ran to verify your changes.

[x] UnitTest
[x] Benchmark

Thank you  for the PR. The experience was used in #610


Add the Apache Pulsar helm chart repository

The Artifact Hub is another place you can list your repos. To do that, you login (or create an account), create an organization, and list your repository. You can link it to a user or organization.

@mattfarina I am one of the PMC members of Apache Pulsar. Pulsar team maintains an official project chart for each release. I have submitted a pull request #394 to add the official chart.


Install and Postinstall hooks may need to know the root install target.

This is like a node thing, I think, maybe useful with other lang specific sub tools.
A give node module may declare a "bin" field like so in the package.json
  "bin": "./bin/cli.js"

And that will be symlinked either to node_modules/.bin if a local install or /usr/local/bin if a global install.
However if we gx install gx-foo-js and gx-foo-js depends on gx-quux-js and both declare "bin"s then the gx postinstall hook has no way to determine which one was the "root" target of the gx install command, because we only really want to symlink the binary for direct imports or installs not every package.
By the way, gx-js is coming along, thanks for the --global flag that worked great.

Maybe the path to the install target would be sufficient?  Then it's just a string compare?

@sterpe does it help that the post-install hook gets executed in the working dir of the 'root' install target?

I will take a look.
On Wed, May 4, 2016 at 5:07 PM, Jeromy Johnson<EMAIL_ADDRESS>wrote:

@sterpe https://github.com/sterpe does it help that the post-install
hook gets executed in the working dir of the 'root' install target?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/whyrusleeping/gx/issues/65#issuecomment-217043770


The same variable name in Cookie and Path used in route and depends causes conflict

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from typing import Optional
from fastapi import FastAPI
from fastapi import Cookie, Path
from fastapi.param_functions import Depends

app = FastAPI()

async def current_user(
    user_id: Optional[str] = Cookie(
        None, alias="USER_ID", min_length=1, max_length=256
    ),
    cookie: Optional[str] = Cookie(
        None, alias="SESSION_ID", min_length=1, max_length=256
    ),
):
    return {
        "user_id": user_id,
        "session_id": cookie,
        "user_name": "Ivan",
    }

@app.get("/{user_id}")
async def some_job(
    current_user: dict = Depends(current_user),
    user_id: str = Path(...),
):
    return {
        "user_id": user_id,
        "user_id_from_cookie": current_user['user_id'],
    }

Description

Run code, and see at generated documentation. Note: no USER_ID in cookies

Try to pass USER_ID, SESSION_ID and user_id to test function some_job. You'll see that cookie user_id  is overriden by path user_id.

Try to change cookie variable name to _user_id. This resolves name conflict and generates correct documentation

Operating System
Linux
Operating System Details
No response
FastAPI Version
0.63.0
Python Version
Python 3.7.3
Additional Context
No response

I don't know if that's a bug.
But I think it's not a good way to do this:
@app.get("/{user_id}")
async def some_job(
    user_id: str = Path(...),
    user_id: Optional[str] = Cookie(
        None, alias="USER_ID", min_length=1, max_length=256
    ),
    cookie: Optional[str] = Cookie(
        None, alias="SESSION_ID", min_length=1, max_length=256
    )
):

I think this should be treated as usage error.
Is it possible to put an assert to avoid such name conflicts?

Yes, may be do an enhance to do this.
The name conflict in two function, may be forget it.
I will have a try, thanks for your advice.

Thank you


refactor: __init__ file

Description
On a new refactoring effort, I simplified the main __init__ file.
Issue linked
NA
Checklist

[x] I have visited and read Develop PyMAPDL.
[x] I have tested my changes locally
[x] I have added necessary documentation or updated existing documentation.
[x] I have followed the PyMAPDL coding style guidelines.
[x] I have added appropriate unit tests that also ensure the minimal coverage criteria.
[x] I have reviewed my changes before submitting this pull request.
[x] I have linked the issue or issues that are solved to the PR if any.
[x] I have assigned this PR to myself.
[x] I have marked this PR as draft if it is not ready to be reviewed yet.
[x] I have made sure that the title of my PR follows Commit naming conventions (e.g. feat: adding new MAPDL command)

@pyansys-ci-bot LGTM.


Conflicting information on deployment options for Outlook add-ins

The table Deployment options for Outlook add-ins implies that Sideloading, AppSource, and Exchange Server are the only deployment options for Outlook add-ins.
However, the table at What controls are available on the Integrated Apps portal? shows that Outlook add-ins "can be managed from the Integrated apps portal on the Microsoft 365 admin center." Indeed this method of deployment seems to work. So the Deployment options for Outlook add-ins should be corrected.

Hi @johnthad,
Thank you for pointing this out. Outlook add-ins can be deployed through Integrated Apps. I'll update the documentation to reflect this.


Include a Cancel Button On All Update Forms

As a general request it would be useful to users if as well as a save or update button all forms also had the option of a cancel button as well. Users might often encounter validation errors where they will need to go and find the missing information so would not always be able to save their details at that point.

This issues has already been fixed.


How to increase size of edges in sigma js ?

I have snippet of code.
I have this snippet of code.
s = new sigma({ graph: g, container: 'network-graph', renderer: { container: document.getElementById('network-graph'), type: 'canvas' }, settings: { maxNodeSize: 16, minNodeSize: 45, minEdgeSize: 5, maxEdgeSize: 5, minArrowSize: 25 } });
But the size of arrow is very small .
Can anyone anyone help ?

Same thing happening to me, put the value in 100, 200. I can see no change at all.

That's probably because of the autoRescale set to true by default. If all of your edges has the same size, it will be rendered with the same size.


Feature Request: Frecency + find_files

Currently when we invoke frecency it only lists files "frecently" opened. Which means that we need to have two different keybindings for finding files, one for frecency and another one for list_files and we need to make that choice everytime we want to open a file.
It would be great to have only one keybind to navigate through files but have them sorted by frecency.
Basically, list the frecency files, but also show the results from list_files.
If I'm not mistaken (never actually used it much) VSCode does exactly that with their file finder, except that it is restricted by the current project I think, which is worse than the current frecency approach which can navigate through any file recently opened which I think is much more powerful, specially if combined with something that changes the current directory to the project root of the selected file

@sunjon currently provides this tag feature with an show_unindexed option which should be true pre default. See README.
So what i am doing is. Open frecency type :LSP: (has tab completion) which currently requires builtin-lsp but it will get the current workspace and all unindexed files. So basically a find_files. It still sorts with the calculated value tho. But its pretty useful. And it will remember the tag. So the next time opening :LSP: is already applied.
You can also define your own tags. So if i wanna look at a different project i can just write/complete that tag and get all files.

Like Connie said, I think this functionality already exists.
I can try make the readme/gifs better if all the features aren't clear enough.
Let me know if you don't get the :foo<tab> filtering working.

@sunjon that would be useful!

@sunjon that would be useful!


new release

It has been nearly a year and no new release of crate2nix is out. nixpkgs now has a one year old crate2nix :( I'm selfishly waiting for https://github.com/kolloch/crate2nix/pull/123 to be fixed in nixpkgs.

I also ran into this in https://github.com/kolloch/crate2nix/issues/168

@zseri's comment means my CI is broken :(

Valid point! On it :)

I released 0.9.0

https://github.com/kolloch/crate2nix/releases/tag/v0.9.0

Thanks!


how did you get category about dataset train_3.5M_CN_With_Category?

here
https://huggingface.co/datasets/BELLE-2/train_3.5M_CN_With_Category
Thank you for sharing this excellent data. May I ask how you obtained the category for each conversation? Did you train a classification model for this purpose? If so, would it be possible to open-source the model? Thank you.

Hi, thanks for you attention, we did train a category classification model to classify the data. And for the categories, We defined these categories with reference to instructgpt, traditional NLP tasks, and other classification logic.


ETA on data and models?

Hi,
As title says, looking for any ETA.

Hi Pavan,
If you shoot me an email, I can send you the data right away. It'll take a couple of months to release the models though.

Both data and models are released now.


Update README

Include following example screenshot:

Instead of putting everything in the README, just show the expected output and then setup a wiki to showcase all the features.


created solution of #113 issue

created programming meme extension.

While submitting a PR

[ ]  Create a folder of all codes
[ ]  Add a readme file with a proper description of the extension, screenshot, and video of extension implementation
[ ]  In PR add what extension you worked by tagging issue and ss of that extension

Kindly complete 2nd and 3rd point

but i already make a PR
should  i change it ??
i can add a readme  file
but i don't know how to add tags
please help me @harshita214

@MeetaHaldar you have to add a readme file describing your extension and then add ss , video of extension implementation

@MeetaHaldar ?

@MeetaHaldar kindly do the changes

sorry for the delay but there is another repo name "awesome chrome extension" i had made a PR on that repo about this same issue . and that PA told me to not contribute same thing to both places . if you are agree to make a PR on this repo too then i can do PR i have already made that chrome extension. @harshita214

Yes that's correct, in that case I am closing this issue. Kindly contribute to different issue


implied oncurve points for interpolatable glyphs

we have a drop_implied_oncurves function inside ttGlyphPen, but that being a pen only works for a single glyph.
Thus I modified this to work on one or more (interpolatable) glyphs and to only prune oncurve points that are impliable for all of the input glyphs, such that the modified glyphs maintain interpolability.
I also moved it to the glyf table module. I'd like to use this in ufo2ft to massage the glyphs of master TTFs before it calls varLib.build.
fontc does a similarly eager pruning of oncurve points and I'd like to have this done (optionally) in fontmake-py as well.

LGTM. Thanks!


Update slurm.sh

Codecov Report
Base: 70.26% // Head: 70.26% // No change to project coverage :thumbsup:

Coverage data is based on head (fee6ed4) compared to base (bbe1525).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #200   +/-   ##
=======================================
  Coverage   70.26%   70.26%
=======================================
  Files          93       93
  Lines        2327     2327
  Branches      293      293
=======================================
  Hits         1635     1635
  Misses        646      646
  Partials       46       46

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.


Add the 'Graceful outfit: Adventurer' to the list of allowed graceful sets for the master clue step to run a lap of the Rellakka course with it.

Fixes #15999
Bug description
"Complete a lap op the Rellekka rooftop agility course whilst sporting the finest amount of grace."
The game allows the set 'Gradeful outfit: Adventurer' -which is gained through quest speedrunning- to be used to complete the lap.

Added in 318c910d0d4378f15b42fa3c65b3eb261b67a2e5


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/62626

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/63803

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/66928

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/100362

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/103082

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/106461

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/108694

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/108971

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/110640


Verify That There Are No Sample Pipelines After Creating Pipeline Server

Added Automation Test for RHOAIENG-4382

Verified success with Jenkins Build #5069


PostCSS plugins are ignored

Hey, I've a problem with postcss-loader and have created the smallest repo to reproduce it.
https://github.com/ertrzyiks/happypack-postcss-example
First of all I've found that not all options defined in my config are passed to the loader, so updated SERIALIZABLE_OPTIONS array with my key, but it still doesnt work. Looks like PostCSS plugins are ignored, because they are passed as an array of functions and can not be serialized here in launchAndConfigureThreads.
Any clues how to resolve it?

How about we give you a chance to deserialize custom options in such a way that you can reproduce this list of functions on the background threads?
The API I'm thinking of is something like this:
{
  postcss__happy: [
    {
      path: 'autoprefixer',
      options: {
        browsers: ['ie 10']
      }
    }
  ],

  plugins: [
    new HappyPack({
      customOptionSerializer: function(compilerOptions) {
        return { postcss: compilerOptions.postcss__happy };
      },

      customOptionDeserializer: function(rawOptions) {
        return {
          postcss: function() {
            return rawOptions.postcss.map(function(postcssModule) {
              return require(postcssModule.path)(postcssModule.options);
            });
          }
        };
      }
    })
  ]
}

This will work with the only condition that customOptionDeserializer does not depend on the closure (i.e. it can be serialized using toString()). And the reason we use a different option name instead of postcss so as to not conflict with the original option in case you're not using happypack for a certain build.

Honestly, I know this isn't a pretty solution but it's the best I can think of in terms of solving your use-case and not causing regressions. There are good reasons why we don't try to serialize function options.
It might be worth noting that this could've been solved more reasily on the loader's end since if they were accepting paths to functions and arguments (like [{ path: 'autoprefixer', options: { browsers: [ 'ie10' ] } }])  and then materialize them into a list of functions then we could've serialized postcss without any modifications.
If you like the suggested solution or can think of a different approach, let me know.

Thanks for the fast reply @amireh
Your suggestion looks fine and might be enough for most use cases, but it will not help me. I mean, it would increase a complexity of my code to the level I think is not worth implementing. It's good enough to know I did not miss something obvious and it is actually a problem in 3rd party, not my usage.
I was very surprised that there is no way to configure postcss-loader or even postcss itself using a plain object. Unfortunately fixing it in the loader will be replacing one problem with another, since in the latest version it uses internal this._compilation https://github.com/postcss/postcss-loader/blob/master/index.js#L86 and crash when used with happypack.
In the end, I will try to exclude postcss from parallelization in my case and will try to fix it on the loader side, since it's way easier as you said.
Cheers!

Looks like there is a discussion around postcss to support plain object config, see postcss/postcss#813
Meanwhile I will suggest postcss-loader to support one of helper libraries: https://github.com/postcss/postcss/issues/813#issuecomment-229815437

Hey @amireh, the latest postcss-loader support alternative configuration options which work with happy pack 🎉 Can we close or do we need an action point before? Like update wiki

Hi @ertrzyiks. I have been trying to get postcss-loader with autoprefixer plugin working with HappyJack but haven't had any luck. I've tried the postcss-loader config option for Webpack 1 here: https://github.com/postcss/postcss-loader/blob/1.1.0/README.md#webpack-1x-config. I've also tried with a postcss.config.js file. Would you mind sharing how you were able to get it to work?

@brandondoran I've updated my example repo with the latest postcss-loader and you can see it working: https://github.com/ertrzyiks/happypack-postcss-example

That's great! I'm glad to hear, and thanks a lot @ertrzyiks for providing an example. I will incorporate it into the repository or perhaps make a Wiki article for it. Which do you think is best?
(Closing since it's resolved.)

@amireh I see you have a lot of examples in the repository, feel free to incorporate my example as well. I would just add a note to the wiki to clear that postcss-loader is fully supported since v1.1.0 (master readme does not even mention problematic webpack-1 config approach anymore)

Yeah, I think we can do a better job at reporting known issues in the docs. Thanks again, I'll look into this.

@ertrzyiks Thanks so much for the example! I've noticed that when I add ExtractTextPlugin usage I get the following warning:
WARNING in ./~/happypack/loader.js?id=sass!./client/assets/styles/account.scss
this._compilation is not available thus `postcss-loader-before-processing` is not supported

However, the css file does seem to build correctly. It looks like you had a commit to postcss-loader to emit the warning instead of crashing. Is it safe to ignore this warning?
Out of curiosity I cloned https://github.com/ertrzyiks/happypack-postcss-example and added ExtractTextPlugin and I get the warning there too.

Hey @brandondoran I'm glad that you find it useful ) It's totally safe to ignore that warning as long as everything works for you. I've added this warning to avoid crash and leave a hint so in case of any strange problem with postcss-loader and happypack you have at least one thing to check.

Hey @ertrzyiks, awesome, that is what I figured. Thanks again for your help on this!

@ertrzyiks I'm also using the ExtractTextPlugin and I'm getting those warnings. It is possible to make them less verbose? Maybe only one warning instead of 200 :wink: (depending on the amount of files in my case)?

@acidicX this warning comes from postcss-loader, not ExtractTextPlugin. It is triggered because happypack does not support this._compilation property and some loader/plugins are not fully functional. Maybe we could add an option to acknowledge lack of _compilation and explicitly silent this warning...

We definitely need to suppress that warning if result is successful and is correct.

@creage @acidicX I've created an issue in the postcss-loader repo, lets see what can be done to mitigate this problem. See postcss/postcss-loader#179

Seems to be fixed with 1.3.3 release of postcss-loader. I've tested it and have no warnings any more.

So I guess the postcss-loader row in https://github.com/amireh/happypack/wiki/Loader-Compatibility-List could be updated now?

webpack: 4.x
happypack : 5.0.1
还是有这个问题（have this problem）


I can't stub request with OHHTTPStubs

Description:
Im trying in my test target to stub request made by OAuthSwift. I dont know why but i can't stub those requests. If i made a simply URLSession.dataTaskWithURL the stubs just works, but if i do the request with oauth no. I see in the code that internally those request are made with URLSession so its very strange
OAuth Provider (Twitter, Github, ..):
OAuth Version:

[ ] Version 1
[x] Version 2

OS (Please fill the version) :

[x] iOS :
[ ] OSX :
[ ] TVOS :
[ ] WatchOS  :

Installation method:

[ ] Carthage
[x ] CocoaPods
[ ] Manually

Library version:

[ ] head
[x] v1.0.0
[ ] v0.6
[ ] other: (Please fill in the version you are using.)

Xcode version:

[x ] 8.0 (Swift 3.0)

[ ] 8.0 (Swift 2.3)

[ ] 7.3.1

[ ] other: (Please fill in the version you are using.)

[ ] objective c

no code provided
no error message
invalid


How windows drawn?

Hello
i am learning c++
now i want to learn "HOW I CAN MAKE A GUI"
for example game engine draw images and etc. with opengl or directx ,...
how nanapro renders ui? which file is the base of draw in nanapro?

Nana is drawing UI using native OS's APIs. Check "paint" directory.
You can draw your own shapes using previously mentioned APIs.
For Windows https://docs.microsoft.com/windows/win32/api/wingdi/
GNU/Linux https://xcb.freedesktop.org/XcbApi/
macOS https://developer.apple.com/documentation/coregraphics
Writing even simple GUI for a beginner could be overwhelming task. I'd recommend using system already established solutions like WPF or GTK. Using existing multi platform libraries is the best option here.

nice
thanks


[enhancement] Removed org field snapshot #128

Removed organization field from snapshot model.
Fixes #128

Coverage decreased (-0.0008%) to 99.128% when pulling 0ec96dc46e233dc87396eb009237c55e052793a1 on Aryamanz29:issue-128/remove-org-field-snapshot into c1219845d377825587665ba689c43b10924fca0c on openwisp:master.


feat: adding deployments fetching

We are doing two things here:

We are adding new fetching methods that call the /deployments endpoint
We are using a global time API when creating new entities

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Should the context from the view be fetched in the mixin instead of the base HxRequest 🤔

The point would be to decouple HxRequests some what from the view. Currently because it calls the view's context data in the class itself it is 100% coupled to a view. Calling the context in the mixin and passing it to the HxRequest would decouple it because then even without a view an HxRequest would still work.

It will also break if the view does't have get_context_data defined.


Added a default ordering so get_queryset will not query the database again

I tried customizing the basket view by adjusting the basket_content.html to display the discounts of each line. I encountered that the discounts were not there, however, the summary with the totals below it shows the discounts.
After some debugging I figured out that the queryset which is used (basket.all_lines()) which contains the discounts applied as it uses a cached resultset:
# oscar/apps/basket/views.py
class BasketView(ModelFormSetView):
    model = get_model('basket', 'Line')
    ...
    def get_queryset(self):
        return self.request.basket.all_lines()

This Formset which is used in the view is using this queryset, so the question is why the discount values are empty:
# oscar/apps/basket/forms.py
class BaseBasketLineFormSet(BaseModelFormSet):

    def __init__(self, strategy, *args, **kwargs):
        self.strategy = strategy
        super(BaseBasketLineFormSet, self).__init__(*args, **kwargs)
        ...

The BaseModelFormSet is using the following code for getting the queryset:
# django/forms/models.py
class BaseModelFormSet(BaseFormSet):
    ...
    def get_queryset(self):
        if not hasattr(self, '_queryset'):
            if self.queryset is not None:
                qs = self.queryset
            else:
                qs = self.model._default_manager.get_queryset()

            # If the queryset isn't already ordered we need to add an
            # artificial ordering here to make sure that all formsets
            # constructed from this queryset have the same form order.
            if not qs.ordered:
                qs = qs.order_by(self.model._meta.pk.name)

            # Removed queryset limiting here. As per discussion re: #13023
            # on django-dev, max_num should not prevent existing
            # related objects/inlines from being displayed.
            self._queryset = qs
        return self._queryset

So when the queryset is not ordered, it will query the database again with an order_by, so then all discount values are lost.
So that's the reason of this pull request :)

[Current coverage][cc-pull] is 80.67%

Merging [#2062][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master      #2062   diff @@
==========================================
  Files           323        323
  Lines         15469      15469
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits          12480      12480
  Misses         2989       2989
  Partials          0          0

Powered by Codecov. Last updated by [1bf028a...04b1b72][cc-compare]
[cc-base-branch]: https://codecov.io/gh/django-oscar/django-oscar/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/django-oscar/django-oscar/compare/1bf028aa819c309ecd106b1e60818fcc9e67c5c8...04b1b7213243cf9ffa3b46736e6b4dae436bc5c4
[cc-pull]: https://codecov.io/gh/django-oscar/django-oscar/pull/2062?src=pr

This seems like a reasonable solution, given that the Line model already orders to id by default.
@mvantellingen would a unit test be needed here to make sure this is not broken, or just a small comment to mention why the extra .order_by() is there?

@vdboor I added some tests for it

Looks good to me!


Setting g:neotex_delay delays other neovim plugins

Specifically deoplete. Completions don't show up until 5 seconds after typing when g:neotex_delay is set to 5000
Deoplete resumes normal behavior after calling :NeoTexOff

Sorry it took so long, but a56c5db0fd495f6fe41627cd14af44de0fa3816a finally fixes this.


Extend dev testing workflow to include probeinterface

Dev tests failing past couple days due to mismatch in our environment
Hopefully this fixes it

Ups, I messed your suggestion, man : (
The correct one would be :
pip install --no-cache-dir "spikeinterface[test_core] @ git+https://github.com/spikeinterface/spikeinterface@main"

Following example 7 from the documentation:
https://pip.pypa.io/en/stable/cli/pip_install/#examples


Add missing credscan suppression download

eng\pipelines\templates\stages\archetype-java-release.yml's was missing the cred scan suppression download which needs to be part of the Publish to Maven Central job. As you can see, eng/pipelines/templates/stages/archetype-java-release-batch.yml correctly has this. What's happening is that older style release (data track) is failing because in the release because the cred scan suppressions aren't downloaded.

I'm going to override check-enforcer, nothing running in this PR will actually test these changes

/check-enforcer override


chore: Update CODEOWNERS file to use new team

Description of changes: Update CODEOWNERS file to reference new amplify-uibuilder team
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Discarding after discussing naming w/team


disable jsp and jspx by default

jsp and jspx is dangerous. likes spring4shell and others hacker,they use uplaod jsp or write a webshell to disk.
If project need jsp or jspx, they pack web.xml in war with jsp mappings by themself.
secure by default.
thx!

This is a bad idea for so many different reasons. To name a few:

"Spring4Shell" allows arbitrary file uploads. All an attacker has to do to bypass this change is to upload a web.xml file that re-enables the mapping
It does nothing to help users that want/need to use JSPs.
Users that had followed the documented security recommendations and set OS file permissions appropriately would have been protected not only against "Spring4Shell"but against any similar vulnerability as well


"reached the recursion limit during monomorphization (selection ambiguity)" with join_all()

I expect the following program to compile without error.
extern crate futures; // version 0.1.9
use futures::{Future, future};

fn main() {
    let _ = future::join_all(vec![
        future::lazy(|| {
            future::ok::<(),()>(()).join(
                future::ok(()).and_then(|()| {
                    future::ok(()).map(|_| ())
                })
            )
        })
    ]);
}

Instead, rustc (stable, beta, and nightly) gives me
error: reached the recursion limit during monomorphization (selection ambiguity)

This problem seems to be related to https://github.com/rust-lang/rust/issues/38033.
I am not sure whether this should be considered a rustc bug or a futures-rs bug. I think the problem could be sidestepped on the futures-rs side by making join_all()'s I::Item be an actual Future rather than an IntoFuture.

What in the wrong is wrong with join_all?!

I applied both https://github.com/alexcrichton/futures-rs/pull/286 locally and removed IntoFuture in the implementation of JoinAll, but it still hit the recursion limit :\

Closing, as this error no longer happens.


Importing ICSS modules into ts files does not work

🐞 Bug report
Command
- [x] build
- [x] serve

Is this a regression?
I'm not sure. I've only tried with v8.
Description
When attempting to import an ICSS module into a ts file the entire file is imported as a string rather than just the objects defined in the :export block.
The behaviour is also inconsistent, dependant on whether you're attempting to import from a component .scss file or the global styles.scss. Component scss files import as strings and the global style file imports as an empty object.
This seems to be something to do with the way the sass loader is configured by the cli since this works as expected in a normal webpack project.
🔬 Minimal Reproduction
The following repo contains an example of the issue:
https://github.com/despian/icss-import-test-angular
This repo contains an example of the expected behaviour:
https://github.com/despian/icss-import-test-webpack
🌍 Your Environment
Angular CLI: 8.0.6
Node: 10.16.0
OS: darwin x64
Angular: 8.0.3
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.800.6
@angular-devkit/build-angular     0.800.6
@angular-devkit/build-optimizer   0.800.6
@angular-devkit/build-webpack     0.800.6
@angular-devkit/core              8.0.6
@angular-devkit/schematics        8.0.6
@angular/cli                      8.0.6
@ngtools/webpack                  8.0.6
@schematics/angular               8.0.6
@schematics/update                0.800.6
rxjs                              6.4.0
typescript                        3.4.5
webpack                           4.30.0

This looks more like a feature request. I don't recall having support for CSS imports in TypeScript files. @clydin and @alan-agius4 would be able to give more details.

I did consider raising it as a feature request but the basic components used by the CLI do support this. Seems like it's something to do with the way they are configured that breaks support. But yeah, maybe it should be a feature request.

Yeah, we don't support this feature yet.

Hi @despian,
Thanks for this feature request, it doesn't' appear that there is a great push for this feature from the community and hence currently this doesn't fall in the CLI roadmap.
You can use third party builders such as https://github.com/manfredsteyer/ngx-build-plus which extend the Angular CLI to add this functionality. NB: by doing so, you will leave the supported path.


Why choose us nav pills overflows in small devices.

Description
The nav pills in why choose us section overflows from the screen in small width devices.

Expected Behavior
It is a width issue. I would like to fix that. Kindly assign.

Screenshots

Enviroment:

OS: Windows
Browser: Chrome
Version: Latest
Device: Desktop

Hey @Devil1205 , I don't think this issue exist, it is only when u check it through the responsive scroll thingy, but when u choose particular , it doesn't. And when u lower the width still more, it will obviously be cut, but that much small width will not be used mostly


Search for pthread or pthreadVC2 lib in CMakeLists

Following issue 421
pthread is a POSIX library. So Windows can't compile project that include pthread.h.
Windows use threads included in windows.h.
To use pthread, Windows users have to get pthreads-win32.
The library provided by pthreads-win32 is called pthreadVC2.lib.
So CMake have to search for pthread and pthreadVC2
I'm new to GitHub and CMake, tell me if I'm doing something wrong.

Can one of the admins verify this patch?

ok to test

@peci1 will it break something on your side?

Looking through the repo, I'm not even sure pthread is really needed. I only see usages of boost::thread. But I'm not sure if boost pulls in pthread automatically or not. But definitely, boost::thread should not rely on pthread on windows.
The standard way of getting pthread is find_package(Threads) and then linking against Threads::Threads imported target. This should correctly handle all operating systems and pull in their respective threading dependencies. But the imported target is only available since CMake 3.1. As libpointmatcher tries to support 2.8, it should instead use variable CMAKE_THREAD_LIBS_INIT when linking targets that require threading support.
So, I'd suggests something like
find_package(Threads REQUIRED)
if (CMAKE_VERSION VERSION_LESS 3.1.0)
  set(EXTERNAL_LIBS ${EXTERNAL_LIBS} ${CMAKE_THREAD_LIBS_INIT})
else()
  set(EXTERNAL_LIBS ${EXTERNAL_LIBS} Threads::Threads)
endif()

Is that PR still valid or #437 has priority on that one?

Is that PR still valid or #437 has priority on that one?

#437 is a nicer solution because the user doesn't have to download pthread

#437 is a nicer solution because the user doesn't have to download pthread


What is the expect behavior of Path<Vec<T>> in openapi

I describe an API like this:
#[OpenApi]
impl Api {
    /// Return status code or random status code if more than one is given
    #[oai(
        path = "/status/:codes",
        method = "get",
        tag = "ApiTags::StatusCodes"
    )]
    async fn status(&self, codes: Path<Vec<u16>>) -> StatusRes {
        if codes.len() > 1 {
            let mut rng = rand::thread_rng();
            let idx = rng.gen_range(0..codes.len());
            StatusRes::from(codes[idx])
        } else {
            StatusRes::from(codes[0])
        }
    }
}

I expect usage like /status/200,300,400 or so. And the above code will lead to array[integer] in swagger.
But when really receiving requests like 200,300, it seems there is no parse_from_parameter(&str) for Vec<T> and come across this error:
failed to parse path `codes`: failed to parse "integer(uint16)": invalid digit found in string (occurred while parsing "[integer(uint16)]")

Did I misunderstand or is this a bug?

This is not supported right now and I need to think about how to do it.


Contribution to "jackson-databind before 2.13.0 allows a Java..."

Updates

Affected products
Severity
Summary

I didn't mean to revert the CWE number here, so feel free to override that change.


Consider adding usage examples for the <include> xml element

This is about the <include> section specifically: https://learn.microsoft.com/en-us/dotnet/csharp/language-reference/xmldoc/recommended-tags#inheritdoc
There should be examples of how to structure the target xml file, as well as different ways of using the file, and path attributes, as well as the @name and id.
Unfortunately, I could not find any official documentation showing how to use this, and I had to resort to going deep into the google rabbit hole to find some examples from years ago. ☹
The good news is that I am currently investigating this, so as soon as I have some working examples, I will share them here.
cc @gewarren

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

ID: 3bcb925f-ef72-9a64-3762-c0b4a96f9dc4
Version Independent ID: d135d0a6-53d2-1567-0b1b-7aae5e07739f
Content: Recommended XML documentation tags for a class and its members
Content Source: docs/csharp/language-reference/xmldoc/recommended-tags.md
Product: dotnet-csharp
Technology: csharp-language-reference
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

I agree @carlossanlop
This is an area where our teams have created some great examples of using this in the product repos. Either the runtime, roslyn, or ASP.NET Core.
If I added pointers to those examples that already exist, would that be helpful? Or would it be better to create new (smaller) examples that demonstrate the feature in different scenarios?

There's plenty of examples in the .NET Framework code, e.g. this.


Question: simultaneous capture and playback

Probably the right place to ask this would be mini_al repo, but maybe Go bindings do it differently.
I would like to have an app that does sound capture and playback at the same time through the same device. My approach of making a config with two callbacks (send and recv) doesn't seem to work on Linux/PulseAudio and Linux/Alsa. If I initialize device with Capture mode - only capture happens, if I do it in Playback mode - only playback happends. If I use a bitmask - nothing changes (because Playback mode = 0). My attempt to open the same default device in two goroutines results in various crashes.
So I wonder what is the correct way to do this? I haven't found any hints in the docs yet, and the examples only demonstrate mutually exclusive capture and playback.
Thanks, and sorry if the question is irrelevant here.

Currently mal_context and mal_device are globals (in mini_al.c), so that does not work with Go routines.
I am not sure if it can be configured like that, better to ask at mini_al repo.

@zserge there is a pull request https://github.com/gen2brain/malgo/pull/5 you might want to check, and if you have some suggestions please comment.


nixos/oci-containers: add preRunExtraOptions

podman and docker each have global options, that must be specified before run.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

podman and docker each have global options, that must be specified before run.

What are these global options?

What are these global options?

In the case of docker: (output trimmed from docker help)
Global Options:
      --config string      Location of client config files (default "/home/benley/.docker")
  -c, --context string     Name of the context to use to connect to the daemon (overrides DOCKER_HOST env var and default context set
                           with "docker context use")
  -D, --debug              Enable debug mode
  -H, --host list          Daemon socket to connect to
  -l, --log-level string   Set the logging level ("debug", "info", "warn", "error", "fatal") (default "info")
      --tls                Use TLS; implied by --tlsverify
      --tlscacert string   Trust certs signed only by this CA (default "/home/benley/.docker/ca.pem")
      --tlscert string     Path to TLS certificate file (default "/home/benley/.docker/cert.pem")
      --tlskey string      Path to TLS key file (default "/home/benley/.docker/key.pem")
      --tlsverify          Use TLS and verify the remote
  -v, --version            Print version information and quit

These only work in the form docker <globaloptions> run ..., and cannot be used as docker run <globaloptions> ....  I haven't encountered a need to use any of these, but @tomfitzhenry must have.


Support PPO with value model, not value head

There's a lot of effort brewing to do thorough comparisons of DPO to PPO and other methods, but to the best of my knowledge many of the best PPO successes in public (and mostly private) use entire value models instead of just value heads.
Any chance we can have help introducing this? Anyone? 🤗

I think it'll be an additional option / parameter (non-default) then everything needs to be validated again. It's tough, but worth it?
The value network is interesting. Really odd, not sure why it's so bad.
Maybe the normalization is more important with a value model rather than a value head, given it has less regularization by "More different" parameters?

Related, @vwxyzjn shared this photo from the appendix of the summarize work with me

Adding a note here:
To use separate value head for PPO, we might not be able to use AutoModelForSequenceClassification without refactoring transformers.
The reason is that AutoModelForSequenceClassification only returns the pooled_logits corresponding to EOS tokens (modeling_gpt_neox.py#L1214), there isn't any way to get the actual logits corresponding to each token, but for separate value network we need to model the value at each token.

A current workaround is something like

class ScalarModelConfig(PretrainedConfig):
    def __init__(
        self,
        base_model: str = "EleutherAI/pythia-160m",
        base_config: PretrainedConfig = AutoConfig.from_pretrained("EleutherAI/pythia-160m"),
        hidden_size: int = 768,
        bias: float = 0.0,
        **kwargs,
    ):
        super().__init__(**kwargs)
        self.base_model = base_model
        self.base_config = base_config
        self.hidden_size = hidden_size
        self.bias = bias

class ScalarModel(PreTrainedModel):
    config_class = ScalarModelConfig

    def __init__(self, config: ScalarModelConfig):
        super().__init__(config)
        self.config = config
        self.lm_backbone = AutoModel.from_pretrained(
            config.base_model,
            config=self.config.base_config,
            trust_remote_code=True,
        )
        self.scalar_head = layer_init(
            nn.Linear(self.config.hidden_size, 1),
            std=1 / np.sqrt(self.config.hidden_size + 1),
        )

    def forward(self, **kwargs):
        output = self.lm_backbone(**kwargs)
        reward = self.scalar_head(output.hidden_states[-1]) - self.config.bias
        return reward

Related, @vwxyzjn shared this photo from the appendix of the summarize work with me

Source: https://openai.com/index/learning-to-summarize-with-human-feedback/


up_squared:tests/portability/cmsis_rtos_v2 failed.

To Reproduce
Steps to reproduce the behavior:

west build -b up_squared -p auto tests/portability/cmsis_rtos_v2
west flash ~/up_squared.sh
See error

Screenshots or console output
*** Booting Zephyr OS build v2.3.0-rc1-210-gac898bc49ea8  ***
Running test suite test_cmsis_v2_apis
starting test - test_kernel_apis
PASS - test_kernel_apis
starting test - test_delay
PASS - test_delay
starting test - test_thread_apis
PASS - test_thread_apis
...
starting test - test_mempool_dynamic
PASS - test_mempool_dynamic
starting test - test_messageq
E: Page fault at address 0xd25ff8 (error code 0x3)
E: Access violation: supervisor thread not allowed to write
E: PML4E: 0x0000000000d15827 Writable, User, Execute Enabled
E: PDPTE: 0x0000000000d14827 Writable, User, Execute Enabled
E:   PDE: 0x0000000000d0d023 Writable, Supervisor, Execute Enabled
E:   PTE: 0x8000000000d25001 Read-only, Supervisor, Execute Disable
E: RAX: 0x00000000001a4c04 RBX: 0x0000000000d1cd60 RCX: 0x000000000000000c RDX: 0x0000000000d26020
E: RSI: 0x0000000000d1cd60 RDI: 0x0000000000d1dbc8 RBP: 0x0000000000d26090 RSP: 0x0000000000d26000
E:  R8: 0x0000000000d26020  R9: 0x0000000000d4d280 R10: 0x0000000000000000 R11: 0x0000000000000000
E: R12: 0x0000000000d26020 R13: 0x0000000000d1dbc8 R14: 0x0000000000d1dbc8 R15: 0x0000000000000000
E: RSP: 0x0000000000d26000 RFLAGS: 0x0000000000010003 CS: 0x0018 CR3: 0x0000000000115000
E: RIP: 0x0000000000103f87
E: >>> ZEPHYR FATAL ERROR 2: Stack overflow on CPU 0
E: Current thread: 0x0000000000d18f90 (send_thread)
E: Halting system
Environment (please complete the following information):

OS: Fedora28
Toolchain: Zephyr-sdk-0.11.2
Commit ID: ac898bc49

get around the stack overflow with
diff --git a/tests/portability/cmsis_rtos_v2/prj.conf b/tests/portability/cmsis_rtos_v2/prj.conf
index ccdf0628a9..bebe7530e0 100644
--- a/tests/portability/cmsis_rtos_v2/prj.conf
+++ b/tests/portability/cmsis_rtos_v2/prj.conf
@@ -14,3 +14,5 @@ CONFIG_CMSIS_V2_MEM_SLAB_MAX_DYNAMIC_SIZE=128
 CONFIG_CMSIS_V2_THREAD_MAX_COUNT=23
 CONFIG_CMSIS_V2_THREAD_DYNAMIC_MAX_COUNT=10
 CONFIG_TIMEOUT_64BIT=n
+CONFIG_CMSIS_V2_THREAD_MAX_STACK_SIZE=1024
+CONFIG_CMSIS_V2_THREAD_DYNAMIC_STACK_SIZE=1024
``
But then there is a test assert..

START - test_thread_join
 - Creating thread B...
 - Creating thread A...
 * Thread B started.
 - Waiting for thread B to join...
 + Thread A started.
 * Thread B joining...
 - Thread B joined.
 + Thread A joining...
 - Waiting for thread A to join...

    Assertion failed at WEST_TOPDIR/zephyr/tests/portability/cmsis_rtos_v2/src/thread_apis.c:305: test_thread_join: (status not equal to osOK)
osThreadJoin thread A failed!
 FAIL - test_thread_join

This bug is only partially resolved by the fix tests: cmsis_rtos_v2: Fix race condition #28490. There are still assertion errors to this testcases.

Can you please help me understand why we are reporting this issue? I see that up_squared is excluded for this test case, so are we trying to resolve something in order to include it in testing? https://github.com/zephyrproject-rtos/zephyr/blob/master/tests/portability/cmsis_rtos_v2/testcase.yaml

Can you please help me understand why we are reporting this issue?

the issue was reported before testcase.yaml was changed :-)


Enabling Envoy gRPC Access Log Service (ALS)

Describe the feature request
Envoy gRPC access log service is another option for the access log. A user could use it to send access logs to a central log inventory or AMP system. Apache SkyWalking as an oss APM project has supported this protocol, so I submit this proposal to enable it in istio.
I submit the following PR to support this feature.

https://github.com/istio/api/pull/911
#13638

The referenced PRs are now in the repo. Closing this issue as the functionality has been added.

How to enable ALS in already installed Istio? Where is it configured?

How to enable ALS in already installed Istio? Where is it configured?

Did you send skywalking from envoy successfully


Support QPIK and TSID QP problems with no constraint

This is actually a osqp-eigen issue, but it is a good thing to have a placeholder for this here. fyi @Gio-DS @LoreMoretti @S-Dafarra

I experienced the same issue implementing a class that performs the IK for a human in biomechanical-analysis-framework; setting the priority of all the task to highPriority or lowPriority I get the following error:
[OsqpEigen::SparseMatrixHelper::eigenSparseMatrixToTriplets] The eigenSparseMatrix is empty.
[OsqpEigen::Solver::updateLinearConstraintsMatrix] Unable to evaluate triplets from the old hessian matrix.
[2023-12-14 12:24:27.020] [thread: 46797] [blf] [error] [QPInversekinematics::Impl::updateSolver] Unable to set the constraint matrix.
[2023-12-14 12:24:27.020] [thread: 46797] [blf] [error] [QPInverseKinematics::advance] Unable to update the QP solver.

Thank you all @davidegorbani for adding the details. I'm currently working on fixing this issue. I hope I will open a PR within today
Stay tune 🎶

Hi @davidegorbani https://github.com/ami-iit/bipedal-locomotion-framework/pull/784 should fix your issue, you can give it a try: https://github.com/ami-iit/bipedal-locomotion-framework/pull/784
In the meantime, I'm writing the test in blf to avoid this happening again

I tested and now I do not get the error anymore; thank you @GiulioRomualdi!

Great!