refactor: cleanup files page

This declutters the FilesPage file a lot.

Moves the modals to their own directory and the info boxes to their own directory too 5409447
Cleans up files.extra on preview c2055cc
Quickly fixes a message f8a3059
Cleans up context menu 63b4164
Moves header outside of files page ea6b761

License: MIT
Signed-off-by: Henrique Dias<EMAIL_ADDRESS>

@olizilla @fsdiogo for the best reviewing experience, I think it'd be better to go commit by commit as their subject-focused 😄

@hacdias the context menu modal doesn't close if you click off it, but does on master, can you take a look pls?

@olizilla could you take a look at https://github.com/tableflip/react-dropdown/pull/5 please?

Thanks for the review @fsdiogo. Just waiting for the updated react-dropdown. /cc @olizilla  😄

thanks @hacdias<EMAIL_ADDRESS>is now a thing

@olizilla updated 😄

Ping @olizilla


Added a .gitattributes to configure how files are handled in git.

Checking out flate2-rs under windows with autocrlf=true results in some of the integration tests failing. This is because the uncompressed text files are undergoing end-of-line normalization while the archive contents still contain the LF line-endings, this results in the test assertions failing.
By using a .gitattributes file it is possible to ensure the integration test text files do not undergo end-of-line normalization.

Thanks!


Introducing Provider, implements #53

MobX can now pass stores around via context! Use Provider to provide them, and @observer([store names]) to make them available in your component's props
Beta build is available: 3.4.0-beta.1
Docs: https://github.com/mobxjs/mobx-react/blob/provider/README.md#provider-experimental

Very nice. This is just to get a value from the context right? But that value can be a mobx instance or not right? Just curious.

@royriojas yes can be anything. But in combination with MobX powered stores it is really powerful, as observer components will then automatically reflect anything that happens in those context stores nicely.

First, thank you for this PR. It's a very valuable piece that's a mild annoyance wiring up, compared to how little boilerplate MobX requires.

How do collisions work? As in, my component uses both foreground.color and background.color?

Because of this, would an object reference be better?
@observer({
 fColor: "foreground.color",
 bColor: "foreground.color",
})

Forgive me if I missed something. I'm in Disneyland Paris checking up on tech :)

I understand some people got used to having <Provider> components, but I wonder how is this better than simply using React's ContextTypes? Having defined contextTypes, a function similar to redux' @connect is trival.

@ericclemmons I think it is a bit simpler than that, provider just puts literally the same things in  using components props as you put into the provider. If providers are nested, the inner one wins.
So you could do something like:
const colors = observable({
   foreground: '#000',
   background: '#fff'
})

const App = () => {
  <Provider colors={colors}>
     <app stuff... />
  </Provider>
}

const Button = observer(["colors"], ({ colors, label, onClick }) =>
  <button style={{
      color: colors.foreground,
      backgroundColor: colors.background
    }}
    onClick={onClick}
  >{label}<button>
)

// later..
colors.foreground = 'blue';
// all buttons updated

Does that answer the question?

@antitoxic It just avoids some boilerplate around defining contextTypes on producer and consumer, created a getChildContext and has clearly defined semantics and checks around the presence of stores. This just makes using context simpler (e.g. hopefully you don't need the manual each time), not different. It's a convention over configuration approach

@mweststrate what are your thoughts on tweaking the API to take an arbitrary number of parameters instead of an array?
example: @observer('colors', 'posts', 'users')
It would be a simpler interface instead of the extra syntax introduced using an array.

@arahansen yes, thought about that, but it might introduce hard to disambiguate scenarios in the future. I can imagine for example introducing a first displayName parameter at some point.


Division by 0 bug is because of having 0 batches for your eval data.

Division by 0 bug is because of having 0 batches for your eval data. I
assume your fine tuning samples are very few, resulting in 5% being rounded
down to 0 batches. Supposing you use batch_size=32 your overall finetuning
samples are around 600 samples?
To overcome that, set "test_size" to None and "test_batches=10" for example
or whatever number of batches you want to use for validation. That should
do.
i have 200 samples ,and i set  tacotron_test_size = None, and tacotron_test_batches = 10，and the error log is "test_size=640 should be either positive and smaller than the number of samples 191 or a float in the (0, 1) range"
so i set tacotron_test_size = None, and tacotron_test_batches = 1, now it is work ,but I don't know if that's right.Does this cause a Over-fitting?

your training data is not enough to create eval samples. try smaller batch sizes, it should be fixed.


Improve Mac OS compatibility

Some parts of GNU libraries were never ported to mac.
This will allow the code to compile on mac OS without any issues.

Thank you for this PR, would like to echo my support for merging this.


feat(drag-drop): allow connecting containers via string ids, attaching data to drop instances and consolidate global event listeners

Adds the ability to pass in strings to connectedTo when connecting multiple containers together. Previously we only accepted CdkDrop instances which can be inconvenient to pass while inside an ngFor.
Adds the ability to attach extra data to a CdkDrag, similarly to CdkDrop. This is mostly for consistency and convenience.
Introduces the CdkDragDropRegistry which is used to keep track of the active CdkDrag and CdkDrop instances. It also manages all of the events on the document in order to ensure that we only have one of each event at a given time. This will allow us to handle dragging multiple items at the same time, eventually.

Feedback has been addressed.


Removed deprecated order attributes

1. Why is this change necessary?
Cleanup old stuff
2. What does this change do, exactly?
Removes the deprecated properties o_attr_1 - o_attr_6 of the class \sOrder, which should actually be removed in version 5.3
3. Describe each step to reproduce the issue or behaviour.
None
4. Please link to the relevant issues (if any).
None
5. Which documentation changes (if any) need to be made because of this PR?
None
6. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Note: I hope the code style changes are fine, my editor does them by default, if not I will reconfigure it and change the commit.

Hey @aragon999,
thank you for your contribution.
We have removed these properties already in commit https://github.com/shopware/shopware/commit/904f74f29260169b9886814cf825d2c0384f8d37#diff-fb2aa687ec624674107b47cabb315496


SPManagedMetaDataServiceApp: No termstore matching to the proxy name 'Managed Metadata Service' was found

Details of the scenario you tried and the problem that is occurring
Problem on line:
$session = Get-SPTaxonomySession -Site $centralAdminSite.Url
if invoked in shp server with role "application with search" return only "OfflineTermStoreNames" and null for "TermStores".
If I try it manual in powershell simple session the first execution do same things, if I repeat the command it works, but only in powershell not in dsc.
Verbose logs showing the problem
In test and set of "SPManagedMetaDataServiceApp":
Leaving EndProcessing Method of Get-SPTaxonomySession.
No termstore matching to the proxy name 'Managed Metadata Service' was found
PowerShell DSC resource MSFT_SPManagedMetaDataServiceApp  failed to execute Set-TargetResource functionality with error message: The name of the Managed Metadata Service Application Proxy
'Managed Metadata Service' did not return any termstore.
Suggested solution to the issue
Find a solution to the bug.
The DSC configuration that is used to reproduce the issue (as detailed as possible)
SPManagedMetaDataServiceApp ManagedMetadataService
            {
                TermStoreAdministrators = @($Credssp_admin.UserName, $Credssp_farm.UserName);
                Ensure = "Present";
                PsDscRunAsCredential = $Credssp_admin;
                ProxyName = "Managed Metadata Service";
                ApplicationPool = "SharePoint Hosted Services";
                ContentTypeHubUrl = "";
                DatabaseName = "SP_MetaData";
                Name = "Managed Metadata Service";
                DefaultLanguage = 1033;
                Languages = @(1033,1040);
                DatabaseServer = "sqlsrv";
            }

The operating system the target node is running

Version of SharePoint that is used (e.g. SharePoint 2016)
SharePoint 2019
Version and build of PowerShell the target node is running
PSVersion                      5.1.17763.134
Version of the DSC module that was used ('dev' if using current dev branch)
v3.1

I just ran into the same error.  In my case, I had two AD principals in the TermStoreAdministrators array and I noticed that after receiving the "No termstore matching..." error, I could navigate to managed metadata in CA, it seemed to load fine, but only the first AD principal was shown in the UI.  I tried deleting the service app and re-running once for each of these config changes:

Remove the second TermStoreAdminstrators array item that never showed up in the UI.
Make sure all items in TermStoreAdministrators don't match the default term store admin account SP adds.
Remove the TermStoreAdministrator property.

Only the last one worked.  Once that completed successfully, I re-added the TermStoreAdministrators property with the original two items, re-ran, and the two principals were added to the service app without error.
So my workaround is to check for existence of the service app and if it doesn't exist, run the config without the TermStoreAdministrators property.  Then, whether the service app exists or not, I run it again with the property.  I'm using Invoke-DscResource orchestrated by another mechanism so having a condition around the first run is trivial.  Not sure how someone would do that with a pull server and a composite resource but worst case, you could just include both configs without a condition.


Midstream Multi Arch Build Targets

Mirroring the work done for #115 .
If any changes are needed please let me know.

As I noted in #115 can this please be converted to an overlay?


Async multiplex tending to be alot slower than sync multiplex

Summary.
I ran a simple http request call's to benchmark various wrappers that Niquests provided, where server supports h2 protocol.
The async version with multiplex is almost 5-6 times slower compared to sync version with multiplex. Would request for help if this could be done a better way.
Expected Result - N/A
What you expected. - Usually Async version will be quite faster than sync version, when performing multiple network requests.
Atleast, thats what I been observed with async/await calls with http1.1.
Actual Result: Here the Async multiplex was  5-6 times slower when compared to Sync multiplex calls.
What happened instead.
Reproduction Steps
from niquests import Session, AsyncSession
from asyncio import run as asyncio_run
from asyncio import gather as asyncio_gather
from aiohttp import ClientSession

#: You can adjust it as you want and verify the multiplexed advantage!
REQUEST_COUNT = 20
REQUEST_URL = "https://documenter.getpostman.com/view/4858910/S1LpZrgg"

def make_requests(url: str, count: int, use_multiplexed: bool):
    before = time()
    responses = []
    with Session(multiplexed=use_multiplexed) as s:
        for _ in range(count):
            responses.append(s.get(url))
            # print(f"request {_+1}...OK", end=", ", flush=True)
    end = time()
    print([r.status_code for r in responses], end=", ")
    print()
    print(f"{end - before} seconds elapsed ({'multiplexed' if use_multiplexed else 'standard'})")

async def async_make_requests(url: str, count: int, use_multiplexed: bool):
    before = time()
    responses = []
    async with AsyncSession(multiplexed=use_multiplexed) as s:
        for _ in range(count):
            responses.append(await s.get(url))
            # print(f"request {_+1}...OK", end=", ", flush=True)
        await s.gather()
    end = time()
    print([r.status_code for r in responses], end=",")
    print()
    print(f"{end - before} seconds elapsed ({'multiplexed' if use_multiplexed else 'standard'})", end=",")
    print()

async def aio_each_session(url: str, session: ClientSession):
    async with session.get(url) as res:
        r = res.status
        await res.text()
        return r

async def aio_make_requessts(url: str, count: int):
    before = time()
    responses = []
    async with ClientSession() as s:
        for _ in range(count):
            responses.append(aio_each_session(url=url, session=s))
        res = await asyncio_gather(*responses)
    end = time()
    print([r for r in res], end=",")
    print()
    print(f"{end - before} seconds elapsed ('standard')", end=",")
    print()

async def _main():
    #: Let's start with the same good old request one request at a time.
    print("-------------------Sync---------------")
    print("> Without multiplexing:")
    make_requests(REQUEST_URL, REQUEST_COUNT, False)
    #: Now we'll take advantage of a multiplexed connection.
    print("> With multiplexing:")
    make_requests(REQUEST_URL, REQUEST_COUNT, True)
    print("---------------------------Async--------------------")
    print("> Without multiplexing:")
    await async_make_requests(REQUEST_URL, REQUEST_COUNT, False)
    #: Now we'll take advantage of a multiplexed connection.
    print("> With multiplexing:")
    await async_make_requests(REQUEST_URL, REQUEST_COUNT, True)
    print("------------------aiohttp--------------------------")
    await aio_make_requessts(url=REQUEST_URL, count=REQUEST_COUNT)

    print("------------------aiohttp--------------------------")
    await aio_make_requessts(url=REQUEST_URL, count=REQUEST_COUNT)
    print("---------------------------Async--------------------")
    print("> Without multiplexing:")
    await async_make_requests(REQUEST_URL, REQUEST_COUNT, False)
    #: Now we'll take advantage of a multiplexed connection.
    print("> With multiplexing:")
    await async_make_requests(REQUEST_URL, REQUEST_COUNT, True)

    #: Let's start with the same good old request one request at a time.
    print("-------------------Sync---------------")
    print("> Without multiplexing:")
    make_requests(REQUEST_URL, REQUEST_COUNT, False)
    #: Now we'll take advantage of a multiplexed connection.
    print("> With multiplexing:")
    make_requests(REQUEST_URL, REQUEST_COUNT, True)

asyncio_run(_main())

System Information
$ python -m niquests.help

{
  "charset_normalizer": {
    "version": "3.3.2"
  },
  "http1": {
    "h11": "0.14.0"
  },
  "http2": {
    "jh2": "5.0.3"
  },
  "http3": {
    "enabled": true,
    "qh3": "1.1.0"
  },
  "idna": {
    "version": "3.10"
  },
  "implementation": {
    "name": "CPython",
    "version": "3.12.4"
  },
  "niquests": {
    "version": "3.8.0"
  },
  "ocsp": {
    "enabled": true
  },
  "platform": {
    "release": "10",
    "system": "Windows"
  },
  "system_ssl": {
    "version": "300000d0"
  },
  "urllib3.future": {
    "cohabitation_version": null,
    "version": "2.9.900"
  },
  "wassima": {
    "certifi_fallback": false,
    "enabled": true,
    "version": "1.1.2"
  }
}

###Results obtained -
####Summary - I did requests call in following order and one more requests in reverse order as shared below, just to make sure last wrappers making the requests are not receiving requests from throttling CPU.  Turns out its not the case, as the time was quite comparable making irrespective of the order.

Niquests

Sync calls (Standard) - 7.295527696609497 seconds elapsed (standard)
Sync call (with Multiplex) - 0.2068800926208496 seconds elapsed (multiplexed)
Sync calls (Standard) - 6.933176517486572 seconds elapsed (standard)
Async calls (with Multiplex) - 1.4766261577606201 seconds elapsed (multiplexed)

Aiohttp

Async call (Standard) - 2.1907169818878174 seconds elapsed ('standard')

Total output obtained
-------------------Sync---------------
> Without multiplexing:
[200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200],
7.295527696609497 seconds elapsed (standard)
> With multiplexing:
[200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200],
0.2068800926208496 seconds elapsed (multiplexed)
---------------------------Async--------------------
> Without multiplexing:
[200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200],
6.933176517486572 seconds elapsed (standard),
> With multiplexing:
[200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200],
1.4766261577606201 seconds elapsed (multiplexed),
------------------aiohttp--------------------------
[200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200],
2.1907169818878174 seconds elapsed ('standard'),
------------------aiohttp--------------------------
[200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200],
2.2599029541015625 seconds elapsed ('standard'),
---------------------------Async--------------------
> Without multiplexing:
[200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200],
5.6602489948272705 seconds elapsed (standard),
> With multiplexing:
[200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200],
1.3796124458312988 seconds elapsed (multiplexed),
-------------------Sync---------------
> Without multiplexing:
[429, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200, 200],
5.500334024429321 seconds elapsed (standard)
> With multiplexing:
[429, 429, 429, 429, 429, 429, 429, 429, 429, 429, 429, 429, 429, 429, 429, 429, 429, 429, 429, 429],
0.2084817886352539 seconds elapsed (multiplexed)

Also want to say I did discover Niquests yesterday, it is a beautiful gift for python. And also would requests for help in integrating this for Fastapi.
Thank you.

It's great to see that Niquests showed interests.
After careful reading of your case, we think that what you saw is perfectly normal.

A) using "async" does not make necessarily things go faster.
B) Doing "multiplex" in sync is basically doing what "asyncio" do but without the huge overhead.
C) You don't create tasks when issuing get, so they are basically running synchronously.
D) Even if you fix "C", it won't matter, as the written size (to socket) is very small, asyncio won't get you boost there.
E) Warning on measuring perf on WAN, the endpoint is probably using DNS round robin
F) You are getting some rate limit errors

There is some usecase where async+multipex perform better, but clearly not on your presented case. See https://github.com/Ousret/niquests-stats for a viable example.
And yes, sync + multiplex can make asyncio http calls look terrible. Look at<EMAIL_ADDRESS>section (II) multiplex.
I am closing this one, as it is not a performance issue.
Regards,


add a bunch more glossary words

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#126  👈
#125
#124
main

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

Join @dyc3 and the rest of your teammates on  Graphite

Merge activity

May 12, 6:42 PM EDT: @dyc3 merged this pull request with Graphite.


ACME server was unable to reach...

I am able to browse acme challenge file in browser, but it is not reachable in letsencrypt-win-simple.
I am using 1.9.1, fighting from 2 3 days with it.

You need to see it from the internet not from the PC it's running on.
I would suggest you use something like http://www.infobyip.com/httpservertest.php
It will try to connect to the URL you give and give you the error or a HTTP/1.1 200 OK and the Content-Type: it gets (which has to be json)
Also for ACME the URL must be http and must NOT have a port number.

yeah thank you so much. my DNS records were not pointing to my domain name.


Add API functions to initialize Haveno account

commit 54b74fcda76c30d1947e5e0cc2f312ef0dca145f
Author: muscleman<EMAIL_ADDRESS>Date:   Sun Nov 21 15:09:24 2021 -0600
WIP

commit f6079499731fb6b7e83e8d49893f3e5d6e9f2452
Author: muscleman<EMAIL_ADDRESS>Date:   Sat Nov 20 20:01:23 2021 -0600
stuck at restoreAccount

commit 6546534ceec333eef2d6a7bbe23f2d17a9e7a62b
Author: muscleman<EMAIL_ADDRESS>Date:   Thu Nov 18 10:40:25 2021 -0600
Added AccountClient

This seems to be a duplicate of #176

This is the frontend of account management, while #176 is the backend.

yeah, i wondered why you deleted it

Sorry, my bad. Haven't noticed i was on the other repo :)


Problems to setup Facebook Plugin in the browser

I'm getting this error:
TypeError: Cannot read property 'browserInit' of undefined

I'm using this:
// Only required for development in browser, not cordova!
module.config(function($cordovaFacebookProvider) {
  var appID = 123456789;
  var version = "v2.0"; // or leave blank and default is v2.0
  $cordovaFacebookProvider.browserInit(appID, version);
});

I'm trying to run the project with cordova run browser, so, I'm not sure what could be wrong.

I think browserInit need to be inside platform ready.

this.browserInit = function (id, version) {
this.appID = id;
this.appVersion = version || "v2.0";
  facebookConnectPlugin.browserInit(this.appID, this.appVersion);
};

    this.browserInit = function (id, version) {
      var appID = id;
      var appVersion = version || "v2.0";

      ionic.Platform.ready(function () {
        facebookConnectPlugin.browserInit(appID, appVersion);
      });
    };

I had the exact same problem!
And solved it your way!
Well if you think about it, in the Ng-Cordova,  the FacebookCordovaProvider is called to initiate the FB SKD even before its files are launched.
I think this problem existed from the creation of the ng-cordova.
It is a bug related only to ng-cordova as if we used purely the facebook connect plugin that the ng-cordova uses, then this would not be an issue, as it doesn't use a provider to set the appid.
I really don't like editing the ng-cordova files to change this, but in this case it seems that there is no alternative in order to make it work!
I hope the source files of ng-cordova will be updated soon to fix this bug.

Thanks man. Solved it the same way.

Hi,
@robotvert could you make a PR maybe?

@blckshrk this issue is closed, hasn't the problem been fixed? in any case, sorry but I won't do a PR because I'm not using ionic or cordova anymore, it was a one time gig for me and frankly the experience was so miserable I'm definitely not going back.

@robotvert I dropped out the idea of using FB connect on web app... I use the simulator or connects the Chrome inspector to my smartphone. I think this is the more correct way to develop cordova apps. Thanks for your reply :)

Hello, can you specify the file that you modify?


Write documentation about how to maintain xmtp.org

Great first draft provided! We'll keep adding to it as we go.


Steps for making Config Web client for puck-server

[x] initialize the project with react.js & typescript
[ ] Create forms for setting server password/(username?) & universal path
[ ] Create a handler for submission that sends a PUT request to serve with updated vars
[ ] Use a fetch API such as AXIOS (https://www.npmjs.com/package/axios) to send the PUT request
[ ] Contact @tspython  to make sure the necessary routes are handled properly on the server
[ ] Have error/success displays
[ ] You should also use a GET to server to retrieve the current values of the universal path & password/username
[ ] Please also consult with @tspython regarding how we can make this client only run on localhost and a different port than other clients

Helpful links: https://reactjs.org/docs/forms.html

PR: https://github.com/qds-team/puck-web-client/pull/2


Pass options to UglifyJsPlugin & use -p & not get an error

I'm need to pass drop_console: true option to UglifyJsPlugin. But every time I do that it throws an error:
Cannot read property 'sections' of null
As far as I can understand, this is due to me using -p flag in package.json.
Thus, it leaves me two options, remove -p, or don't pass drop_console: true option .
Neither of which I want to choose. I want to use -p, because it adds some magic which is important for prod build.
Is there a way to pass drop_console: true option to UglifyJsPlugin and use -p?

uglifyjs-webpack-plugin version ?
The -p flag sets this.minimize to true (html-loader, css-loader) and pushes new UglifyJSPlugin() into the plugins {Array}. I personally discourage it's usage, handling it in the webpack.config.js via env === 'production' ? x : y  is the more explicity and predictable way
webpack.config.js
const env = process.env.NODE_ENV
...
{
  loader: 'css-loader'
  options: { minimize: env === 'production' ? true : false }
}
...
plugins: [
   env === 'production' ? new UglifyJSPlugin() : false
].filter(Boolean) // <=

It looks like you just deleted/ignored our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please open a new issue with filled issue template. Also make sure your issue is not a question. Questions should be posted on Stack Overflow.

Fair enough.


begin and end extraction fails if when and end but not start

additional condition for: when and end and not start
fixes undefined date output if xml node holds when and end but not start attributes

1 out of ~50 to 100 examples: lk-data/listperson.xml@xml:id=pmb56751


Problem with multiple connections

I'm having difficulty maintaining more than 20,000 simultaneous connections. After this number, the script does not respond to new connections.
There is a perl script for mining ZEC that supports multiple connections:
https://github.com/mtve/yazecminer
Is there any configuration to make it work?

duplicate https://github.com/x25/coinhive-stratum-mining-proxy/issues/38


datetime widget format is not respected

- Do you want to request a feature or report a bug?
Bug
- What is the current behavior?
A field in the config.yml  such as
{label: "Publish Date", name: "date", widget: "datetime", format: "YYYY-MM-DD hh:mma"}

leads to an output:
date: 2017-08-15T13:33:26.984Z

The output does not match the given format.
- If the current behavior is a bug, please provide the steps to reproduce.
Check out these repos:

output: https://github.com/beta-europe/beta-europe.github.io/commit/d0e8dd21e1d94d29f4cb8c0e84ac0541e9ab68ee#diff-e743088db72d471d9ac3936dbfe70940R3
config: https://github.com/beta-europe/cms-admin/blob/7d2feb0d2ea36cda4b924f88877713118f04b35b/config.yml#L19

- What is the expected behavior?
date: 2017-08-15 13:33Wed

- Please mention your node.js, and operating system version.
Firefox Nightly (57) with<EMAIL_ADDRESS>

@Benaiah is working on this, turned out to be non-trivial.
Benaiah can you give a quick update here?

Actually respecting the datetime format is pretty doable, and already implemented by myself and Matt. The UI, on the other hand, has some issues due to the limited API of the DateTime component we're wrapping. At this point I'm OK with merging the PR Matt and I have been working on (#842) once it's ready and living with the UI issues until we can fix the dependency.

Yeah I ended up merging and releasing that right after my comment, we can close this.


Model and Test values not the same

Hey I have a really weird problem which does not make any sense to me.
My environment is Factory_Girl_Rails(1.2.0) on jruby 1.6.3, rails 3.0.10
What the issue is I create a dummy object with factory_girl and some how the objects attributes are different within the tests and in the model.
What I mean by it is in the test the value of an attribute is 4.5 and in the model the same attribute is 5.0 on the same object.
Can some help me to determine if its a active record problem or factory_girl or something else?
Thanks

Can you post the factory and the section of the db/schema.rb that creates its table?

Here is the factory:
Factory.define(:object) do |f|
f.name "Test Object"
f.set_spacing 4.5
end
Here is the schema.rb:
create_table "object", :force => true do |t|
t.string   "name"
t.decimal  "set_spacing"
end
In the schema.rb it says decimal but I run a migration which changes the type to float:
change_column :object, :set_spacing, :float, :precision => 8, :scale => 1

If you create the model without factory girl, what happens?
define Object, "with spacing set" do
  subject { Object.create!(:set_spacing => 4.5) }
  its(:set_spacing) { should == 4.5 }
end

When I create the object without the factory the set_spacing is 4.5:
it "should give set_spacing as 4.5" do
object = object.create!(:set_spacing => 4.5, :name => "Test Object")
object.set_spacing.should == 4.5
end

When i do the following test the set_spacing is 5.0 and not 4.5:
it "should give set_spacing as 4.5" do
	 object = Object.create!(:set_spacing => 4.5, :name => "Test Object")
	  objects.set_spacing.should == 4.5
	  object_action = ObjectAction.create( :name_past => "replaced", :object_counter => "", :object_name => "Pipe Section")
	  job                = Job.create(:object_id => object.id)

      job.object_counter_string.should == "9.0m"
	end

It fails on the last line and give answer as "10.0m" meaning the set_spacing in the model side gets round to 5.0

Have you overwritten the setter for set_spacing, per chance? When FactoryGirl assigns attributes, all it does is loop through each one and essentially call instance.send("#{attribute_name}=", attribute_value). What's returned when you call FactoryGirl.attributes_for(:object)?

Can it be that the following is overwriting the value as it comes into the model
validates :set_spacing, :presence => true, :numericality => true

The output of FactoryGirl.attributes_for(:object) is:
{:name=>"Test Object", :set_spacing=>4.5}

Wonderful! That means it's not an issue with FactoryGirl. I'd look through your model code and try to trace down where the value is changing. I'd look in any callbacks, setters, or anything else that'd modify set_spacing. If you call FactoryGirl.build and it's set correctly to 4.5, but you know it doesn't work with FactoryGirl.create, it's pretty safe to assume it's somewhere in your validations, callbacks, or in the database layer. I'd also tail the test log to see what's actually inserted into the database, since that may help provide more clues.

Thanks for the help

No problem - let me know what you end up finding!

If look if its a validations but nothing, next i went and looked at my callbacks nothing the only thing that I can think of is that my database which is Microsoft SQL Server 2008 is rounding the numbers.


ERROR IN PLUGIN when running action new-tab

Describe the bug
When running zellij action new-tab with -l and -c set, zjstatus fails to start.
To Reproduce
My settings:
config.kdl
default_shell "/opt/homebrew/bin/fish"
default_layout "zjstatus"
simplified_ui true
pane_frames false
theme "catppuccin-latte"
keybinds {
    normal {
        bind "Alt p" {
            Run "/opt/homebrew/bin/fish" "-c" "$HOME/.scripts/open-project" {}
        }
    }
}

layouts/zjstatus.kdl
layout {
    pane split_direction="vertical" {
        children
    }
    pane size=1 borderless=true {
        plugin location="file:$HOME/.config/zellij/plugins/zjstatus.wasm" {
            format_left   "{mode} #[fg=#1e66f5,bold]{session}"
            format_center "{tabs}"
            format_right  "{datetime}"
            format_space  ""

            border_enabled  "false"
            border_char     "─"
            border_format   "#[fg=#6C7086]{char}"
            border_position "top"

            hide_frame_for_single_pane "true"

            mode_locked "#[fg=#dce0e8,bg=#e64553,bold] LOCKED "
            mode_normal "#[fg=#dce0e8,bg=#1e66f5,bold] NORMAL "
            mode_resize "#[fg=#dce0e8,bg=#179299,bold] RESIZE "
            mode_pane   "#[fg=#dce0e8,bg=#ea76cb,bold] PANE "
            mode_tab    "#[fg=#dce0e8,bg=#8839ef,bold] TAB "

            tab_normal   "#[fg=#1e66f5] {name} "
            tab_active   "#[fg=#8839ef,bold] {name} "

            datetime        "#[fg=#6C7086,bold] {format} "
            datetime_format "%A, %d %b %Y %H:%M"
            datetime_timezone "Europe/Berlin"
        }
    }
}

When running:
zellij action new-tab -l zjstatus -c ~/Development/personal/turboprop/
The new tab is opened with the correct working directory, but zjstatus fails to start.

Interestingly, I cannot find any entry about this in the zellij logs.
Expected behavior
zjstatus starts.
Screenshots
See above.
Desktop (please complete the following information):

OS: [e.g. iOS] macOS 14.5
Zellij version [e.g. v0.39.0] 0.40.1
Version [e.g. v0.9.0] 0.15.0

Layout
How does the layout look like? Please copy it into a code block.
See above.
Cheers

Hi and thanks a lot for the detailed bug report. I think this is an error in zellij and not zjstatus.
When I try your layout, the following log lines appear when running the command.

This hints at some inconsistent behaviour when resolving the $HOME environment variable.
If I test your layout with an absolute path or ~ instead of $HOME, it seems to work.
https://github.com/dj95/zjstatus/assets/4348959/876628e0-cde1-4b12-a9c6-9adf1d50aeec
Can you test this please?

Hey @dj95, thanks for the swift reply! Replacing $HOME with ~ in the layout worked. I'll recreate this bug in the zellij repo then, sorry for misplacing it :)

No worries, at first glance it looked like an issue in zjstatus. I'm glad that it works for you :) In addition, I addressed the issue and asked about it in the zellij discord.


add sorting feature

I need to have the sorting option in the JobList where JobItems are rendered. Get the sorting icon from 'react-icons' library (already installed)

hey madina,
Could you assign this issue to me?

Sure!


Fix operational credentials fabric count attributes

Problem
The SupportedFabrics and CommissionedFabrics attributes of operational credentials cluster are always 0, when accessed via the read attribute command.
Change overview
The attribute values were not getting updated. The code to write SupportedFabrics was missing. CommissionedFabrics value was not written due to FabricID check.
Testing
Added a new cluster test to read and verify these two values.

@andy31415 , @Damian-Nordic , @jepenven-silabs do you have any review feedback?

rebased.. trying to re-trigger CI.

@andy31415 , @Damian-Nordic , @jepenven-silabs do you have any review feedback?

@saurabhst ?

@Damian-Nordic @hawk248 ?

@andy31415 ?

TE5 cherrypick failed due to changes in src/darwin/Framework/CHIP/templates/clusters-tests.zapt:
#9215 and #9287 changed the same file

I did a manual merge of this - the change seemed to be only the test list.


The timestamp of the captured packet in sniffer mode is not updated for a long time (IDFGH-2412)

When I use sniffer mode to capture some WiFi packets, I find that sometimes the timestamps of dozens of consecutive packets are the same, lasting even 300ms. Is this normal? Time is not accurate enough? Or ampdu? But I use Wireshark to capture packets on Linux. The time of each packet is different. I want to make sure. Thank you.

@wgcod Please describe how you call the sniffer API, and provide the logs of your test.

The test is very simple. I used the sniffer demo to output the time stamp of each package in the callback function, as you can see.

@wgcod Have you enabled the modem sleep or light sleep? There is description in IDF:
https://github.com/espressif/esp-idf/blob/30372f5a4ff2c0dfdaad95f544dc36bcdda30b75/components/esp_wifi/include/esp_wifi_types.h#L335

No, I test with demo without any modification. Save the data to SD card and open it with Wireshark. Sometimes, a dozen or more packet timestamps are the same. When I capture packets with Wireshark at the same time, I can see that the time of these packets is different, but only 2-3us is related between packets. Is it because the clock is not accurate. If so, is it configurable? You can try it. Just use demo.

hi @wgcod ,
I have tested with simple_sniffer example in IDF, and open the capture packets with Wireshark, for some packets the timestamps are the same, but only about a dozen milliseconds. And the time inaccuracy is  acceptable for ESP32.
If you want the timing to be particularly accurate, you can try our new chip ESP32S2.
https://www.espressif.com/en/products/socs/esp32-s2

Thanks for reporting and sorry for slow turnaround, feel free to reopen if the issue still happens. Thanks.


Cannot read property 'autoUpdate' of undefined

Version: 3.1.4.
Setup: Webpack + Angular 2.
Whenever i try to call $(element).mCustomScrollbar("destory") i get an error: Cannot read property 'autoUpdate' of undefined.

+1

+1

+1

+1

+1

Make sure you use deps correct version
"@angular/common": "^2.4.0",
    "@angular/compiler": "^2.4.0",
    "@angular/core": "^2.4.0",
    "@angular/forms": "^2.4.0",
    "@angular/http": "^2.4.0",
    "@angular/platform-browser": "^2.4.0",
    "@angular/platform-browser-dynamic": "^2.4.0",
    "@angular/router": "^3.4.0",

+1

+1

+1

#392 works
you should add
if(!mCSB_container.length) return;
right after
_autoUpdate=function(rem){
    var $this=$(this),d=$this.data(pluginPfx),o=d.opt,
          mCSB_container=$("#mCSB_"+d.idx+"_container");

I have the same issue . But the pluggin doesn't contain these lines and its version 3.1.5
autoUpdate=function(rem){
var $this=$(this),d=$this.data(pluginPfx),o=d.opt,
mCSB_container=$("#mCSB"+d.idx+"_container");

I had to uncomment or delete line 1108:
clearTimeout(mCSB_container[0].onCompleteTimeout); _delete(mCSB_container[0],"onCompleteTimeout");


[WIP] New content - TKGm STIG Hardening Guide

Anyone in particular that we need to review this @anthonyvetter?
Thanks for putting it together!

@alexbarbato, Nitin Ravindran and Brian Rieger are point for the reviews. I am also trying to get clarity from the #tkg-airgapped-docs channel if we are still going this route since it looks like they ported this one to Docs already.

Roger, thanks :)


Background image having complete width

Make it so that the background image fills the entire width of the screen.

Doesn't actually look better.


Rename standard fields in logs

Hi!
Is it possible to rename fields from logs?
For example: to rename threadName to thread?

Yes, take a look at https://quarkiverse.github.io/quarkiverse-docs/quarkus-logging-json/dev/index.html


Support Serving Latest Offline Segments Immediately

Overview
For hybrid setups, Pinot splits/filters broker queries to both the offline and realtime tables. It does so based on a time value which represents the latest available offline segments. Pinot assumes the consumer is uploading partial or intermediate segments throughout the current day, so it only serves time value - 1 from the offline table and the rest from realtime.
As an explicit example, if I upload a segment with the date 1/2/2000, the time value for the table will be 1/1/2000. This means, a query for all time, will be modified to query (*, 1/1/2000] to the offline table and (1/1/2000, *) to the realtime table. In this scenario, the 1/2/2000 segment is not being served queries. Once another segment is uploaded for 1/3/2000, then the data from 1/2/2000 will be served.
Problem Statement
In hybrid table setups where offline segments are only uploaded once per day, it is desired that those offline segments immediately start serving over the realtime segments. Today, this has to be achieved via a "hack" where an empty segment is pushed for a future date to trick Pinot into serving the latest offline segment with actual data.
Requirements

This should be configurable per table.

Proposed Solutions
Update BrokerRequestHandler to Partition Queries Differently
We could use a table-level config in the request handler to modify the time filtering behavior to include the latest offline segments based on the time boundary (already available in the request handler).
Update TimeBoundaryManager
Using a table-level config we could modify the time boundary manager to return current time value for inquiries about the current time boundary rather than the current behavior of time value - 1.
Open Questions

At what point is the time value refreshed for a table? Segment uploads may contain more than one segment file. Does it wait until all are uploaded to increment or does this happen asynchronously and the time value may refresh while the others are still uploading? If the latter, then we risk creating a state of bad data where only some segments have been uploaded, but queries are already being diverted to the offline table.

Here is my suggested approach.

Create a config in table that indicates that for a table we need the time boundary to be next unit up. (i.e. if daily push, next day, if hourly the next hour. etc.)
Change the time boundary code to move up the time boundary when new segments are added to the table.

In this case, there is no new API, but for use cases that have multiple segments, the time boundary will be advanced when the first segment is pushed. This will lead to inconsistency.
Pinot currently has the inconsistency problem only if RT is not there, but we will be introducing a new one now.
We can, however, introduce an API as well, which can send out a message to the brokers to advnce time bondary. This API will be invoked by the push job when all segments are pushed. This iwll reduce the inconsistency window to some narrow cases such as:
(1)  Three segments are being pushed. After the first one, a broker restarts, and lokoing at the table config, and the recent timestamp, advances the boundary. Since the other two segments are not pushed yet, the broker has no way of knowing that a message is going to come by later. Corner case, but maybe we can leave that for the day when we support "atomic" push of data.
(2). Three segments are being pushed. The push complets, but the external view is not yet updated (slow server, slow helix, whatever). The broker meanwhile gets the message to move time boundary.
The second one is more likely to happen, but either one is a bad experience for the application.
Some mitigation may be to include the segment names in the controller api and the controller waits for the EV of these segments to be ONLINE before issuing a message, but that is also not complete. Broker may still not see the new EV
Ideas?

@mcvsubbu the time boundary should be saved somewhere in ZK so that all brokers use it and it handles all the failure and restarts scenarios.
The user still has to wait for the servers to load the segment before invoking the time boundary.

@kishoreg I didn't follow. We don't store the time boundary now. Why do we need to keep it in ZK?
Perhaps you are trying to support an arbitrary time boundary that a user can set. I don't see a use case for that now. My thought is that we can support one of two time boundaries -- either include the latest <day/hour> or exclude it. I think this will cover most of the use cases.
What is the use case to cover up to some arbitrary user-specified time?

the latest day will clearly not work unless there is one segment for that day and it is too limiting.
The use case I am thinking is the same one listed here. If we allow this to be part of ZK, then time boundary is
max(max(segments) - 1 , value in ZK)
Eventually, we want to avoid users having to manage offline jobs. Users are already asking for features where we provide a minion task that moves realtime segments to offline nodes automatically after a day. Having this API will allow us to control the time boundary.

Summarizing the offline discussion between @mcvsubbu @snleee @sajjad-moradi and @mayankshriv:

Since there are use cases that may push partial data for the day (UTC vs PST), a table level config is warranted to ensure where this feature can be enabled.
We can leverage the primitive in https://github.com/apache/incubator-pinot/pull/5712 to write the time boundary in ZK once the segments have been pushed.
We will need to replace the existing EV based time-boundary computation with one that is based on the Znode listener. We might also need a periodic auto-correct task (in case updating the Znode fails).

Let us avoid combining this with #5712 . That was replacing segments, and this is about adjusting time boundary after segments are pushed. A single primitive to adjust the time boundary, invoked by the segment pusher should be enough

I also thought some more about znode and watches. Introducing a new node may be fine, but having the broker set a watch on that needs some thought. I remember in the HLC case, the controlller was setting too many watches, and that caused zookpeer re-connects to fail. A re-connect requires that the zk client send the entire watch list in one message to the zk server. We had to increase the message size setting in order to overcome this...

It seems discussing here may not be as effective as desired. My proposal is to have a design doc within initial proposal detailed out. We can use that as a starting point to discuss the pros/cons.
@sajjad-moradi @snleee is one of you still planning to start that design doc?

@mayankshriv sure, I can start the initial design doc.

Here is the design doc proposing three different solutions for this problem:
https://docs.google.com/document/d/1KykbUIstFI7F4IOOlUVnXWzpgKazwRAJdok0RT1ae64
Please review and leave your comments.

Here is the design doc proposing three different solutions for this problem:
https://docs.google.com/document/d/1KykbUIstFI7F4IOOlUVnXWzpgKazwRAJdok0RT1ae64
Please review and leave your comments.

Thanks, added to ticket.

@ianvkoeppe The agreement on design doc is to go with approach 1 as the short term solution. Since this approach is very similar to what you provided in PR #5745, could you please reopen the PR and apply the new changes.
Basically the boolean flag that you have defined in TableConfig need to be changed to a number indicating how many days (or hours) need to be deducted from the time boundary. Default value should be one for backward compatibility. Follow PR #4156 to make the changes on time boundary calculation.

@sajjad-moradi we would prefer to discuss table config items in a design doc. If you can update the design doc with the proposed config, then @ianvkoeppe  can go for it. thanks.

@sajjad-moradi we would prefer to discuss table config items in a design doc. If you can update the design doc with the proposed config, then @ianvkoeppe can go for it. thanks.

The doc is updated with config suggestion. Please review and leave your comments.

Thanks all! I appreciate the rapid iteration on the design. @sajjad-moradi I'm happy to iterate on that PR and make necessary changes.

@ianvkoeppe Just checking if you are still planning on iterating your PR as per the new design?

@mayankshriv, I would like to still get this done, but it's not something I can prioritize right now.


Incorrect python example in Text-to-speech quickstart guide

Given the previous import statement the following example section:
speech_config = SpeechConfig(subscription="<paste-your-speech-key-here>", region="<paste-your-speech-location/region-here>")
Should actually read like:
speech_config = speechsdk.SpeechConfig(subscription="<paste-your-speech-key-here>", region="<paste-your-speech-location/region-here>")

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

ID: 064d7074-3ea4-21e4-60b4-b09976bbdf7e
Version Independent ID: 1fdefc18-e765-9fe8-0652-ca18122cb0fa
Content: Text-to-speech quickstart - Speech service - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/get-started-text-to-speech.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @eric-urban
Microsoft Alias: eur

@nath-thomas Thanks for the feedback! We will review and update accordingly.

@nath-thomas Here is the link to QuickStart sample.
https://github.com/Azure-Samples/cognitive-services-speech-sdk/blob/master/quickstart/python/text-to-speech/quickstart.py
I have assigned to content author to update the document.
@eric-urban Could you please update the document.

@nath-thomas - Thanks for letting us know! We will update the documentation.

#please-close


Update golang versions from 1.17.10 to 1.17.12 and 1.18.3 to 1.18.4

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

What does the PR accomplish, why was it needed?
Fixes the following CVEs in the golang packages:
2022-1705, 2022-1962, 2022-28131, 2022-30630, 2022-30631, 2022-30632, 2022-30633, 2022-30635, 2022-32148
Change Log

From the security announcement: https://groups.google.com/g/golang-announce/c/nqrv9fbR0zE
CVE-2022-1705
CVE-2022-1962
CVE-2022-28131
CVE-2022-30630
CVE-2022-30631
CVE-2022-30632
CVE-2022-30633
CVE-2022-30635
CVE-2022-32148
Packages uploaded to blob storage
core/go1.17.12.src.tar.gz
core/go1.18.4.src.tar.gz
Does this affect the toolchain?

YES
Links to CVEs
https://nvd.nist.gov/vuln/detail/CVE-2022-1705
https://nvd.nist.gov/vuln/detail/CVE-2022-1962
https://nvd.nist.gov/vuln/detail/CVE-2022-28131
https://nvd.nist.gov/vuln/detail/CVE-2022-30630
https://nvd.nist.gov/vuln/detail/CVE-2022-30631
https://nvd.nist.gov/vuln/detail/CVE-2022-30632
https://nvd.nist.gov/vuln/detail/CVE-2022-30633
https://nvd.nist.gov/vuln/detail/CVE-2022-30635
https://nvd.nist.gov/vuln/detail/CVE-2022-32148
Test Methodology

Toolchain build: 213065 succeeded
Buddy build: 212961 succeeded

I'm going to go ahead and close this PR, which is superseded by #3561 and #3562.


GH-35404: Temporarily pin sphinx to 5.x to fix the doc build

See https://github.com/apache/arrow/issues/35403 about requiring some fixes on our side to support sphinx 6. Until then, let's pin sphinx to <6.

Closes: #35404

@github-actions crossbow submit preview-docs

@github-actions crossbow submit preview-docs

This seems to have worked: http://crossbow.voltrondata.com/pr_docs/35405/


How to implement anti spoofing ?

Face liveness detection will be soon added, check this issue and the other for updates.

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."


first test omitted many sets of factors

This first test for this exercise did not agree with the one in the JS track, omitting most of the factors for the first palindrome.
There was also a small typo in the description for the second one.

I see what you're saying: the broken test worked because the example.js was broken.
On closer inspection, I don't think the JS track is set up sensibly.  I think what you actually want, for both tracks, is a test like this:
expect([[1, 9], [3, 3]]).toContain(largest.factors);
[1,9] and [3,3] are both legitimate answers to the question 'what are the factors of 9?'
So, are you proposing to change the example.js yourself?  I can change the test and resubmit.  Let me know how you want to proceed.

Feel free to change example.js. That will make the PR more balanced. And I think you have the context of the code, so you're in a better position than me.
The code is all yours.

OK, I think it's set.  Please check it out and let me know if it looks good.

It looks nice! Great job!
Would you mind to rebase the commits so that there is only one commit to merge?

OK, I think I did that correctly.  It's been a while since I've used rebase.

The idea is to have only one commit in the pull request. But now, there are four :cry: I struggle a lot with rebase. Sometimes I prefer to discard my changes and start from scratch.
I think the problem was that you worked on master, so merged changes on exercism/xecmascript master branch affected your work. And now, you have more commits in this PR.
Don't worry, I will merge this PR. It's not a big deal.
Next time, to avoid these problems, I suggest to follow this workflow:

Be sure your master branch is synchronized with exercism/xecmascript. I have a git remote called exercism referencing that repository. Locally, I am in master branch. I run git pull exercism master.
Create a new branch: git checkout -b <new-branch>. This prevents my work to be affected by changes in master branch. It can cause your work to be out of date, so don't work for too long in a branch.
Work on that branch. Commit as many times as you want.
Rebase and squash all commits on that branch into a single commit with all changes. git rebase -i <commit-just-before-first-commit-on-branch>
Push that branch to your GitHub repo: git push origin <new-branch>
Go to GitHub and create a pull request
Once the pull request has been merged.  And update with exercism repo: git pull exercism master.
Repeat :wink:

Thanks for the detailed instructions.  I did in fact forget to branch after forking.
BTW I have not seen the new exercises you have committed appearing.  Should they be?

You should see them directly in the repository. Then, when there is a deploy, new exercises should be available to practice.


Debug trigger algo

Debug modulus operation for window_start_time==0, which causes window_start_time to move back by one window_step_size.

Thanks @kmtsui


beacon file script

References
Add references/links to any related issues or PRs. These may include:

Fixes #issue-number (if this fixes an issue ticket)

Description
Working state of our Script to generate beacon files of some identifier based on the beacon draft (https://gbv.github.io/beaconspec/beacon.html) (our implementation in dspace 5  was mentioned in some earlier PR https://github.com/4Science/DSpace/pull/218)
Script for generating files according to the beacon standard (see https://gbv.github.io/beaconspec/beacon.html). It get all (visibile) entities of some identifier and prints them into one process result file or optionally dumps them in the handler log. The usage is mainly popular among GLAM-Systems using the gnd-identifier in the DACH-Area. The header lines can be configured in the configuration.
Instructions for Reviewers
This is mainly some working solution for institutions in the DACH-Area (Germany, Austria, Switzerland) , e.g. archives or libraries to generate beacon files with gnd-identifiers. Feel free to pick it up and modify to your needs.
List of changes in this PR:

introduced new beacon script.
It works over the whole site on discoverable entities.
Identifier, main resolver and additional resolvers as well as the header lines can be configured in beacon.cfg

Include guidance for how to test or review your PR. This may include: steps to reproduce a bug, screenshots or description of a new feature, or reasons behind specific changes.
Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome). If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[ ] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[ ] My PR passes Checkstyle validation based on the Code Style Guide.
[ ] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[ ] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[ ] If my PR includes new libraries/dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[ ] If my PR modifies REST API endpoints, I've opened a separate REST Contract PR related to this change.
[ ] If my PR includes new configurations, I've provided basic technical documentation in the PR itself.
[ ] If my PR fixes an issue ticket, I've linked them together.

aligned to 2023.02.02 Release


Adopt to compose 1.0.0-beta09

New AnimatedContent composable is introduced in compose-animation 1.0.0-beta09. (I2c3df)
So I think, I need to change the implementation.

I took a quick look at AnimatedContent and tried it out.
But, there are no scale function in EnterExitTransition.
So we must wait more until that time. :sob:
If anyone wants material motion based on compose-animation, please click the star in the link below. 🙇♂
https://issuetracker.google.com/issues/191325593
Well..but I'm going to create an experimental feature before supporting it. 😆


Default language mode when new file opened

VSCode Version: 1.2.1
OS Version: Windows 10 64x

Steps to Reproduce:

When create new file, before save file with '.js' extension and the manual language selection, need to have option to default language e.g. JavaScript, not plaintext as default

@isidorn

We would need to introduce a workbench configuration such that there is a default language mode for new files opened. Such that it is being picked up before a user manuly chooses a mode or sets the file extension.

I would think this could also be done nicely from an extension.

This is one of my key uses from PowerShell ISE that is hard to convert over to VS Code. I use the Untitled page often to construct commands and scripts rather than doing it in terminal because its much easier to edit and I'm not wanting to save it just yet.
The trick though is this Visual Studio Code and versatile and sometimes I will want the default to be different based on the language I'm focusing on in a given period.
The two ways to handle this would be:

Command Line option to select default language on startup.
File New --> Language menu options

The first option allows me to create a shortcut focused on my language. VS Code PowerShell and VS Code C#. It would also make it easier to for extensions to configure the environment more for their use if they knew someone was starting it up as their language.
The second option should not be defaults, but rather configurations for me to add my own list to the File menu.
I like both approaches but if I had to choose, the first I think could be more useful.

@bpasero I just looked into making an extension for this but there doesn't seem to be an API to change the language ID of a TextDocument.  I tried passing the language ID as a parameter to workbench.action.editor.changeLanguageMode but it always displays the quick pick dialog.  Any ideas?

I think this can be done via a new command using the new API: http://code.visualstudio.com/updates/v1_9#_new-api-to-open-an-untitled-file-with-optional-language

That doesn't work for new files that the user creates using "File -> New", "Files: New File" or double-clicking in the tab area.  Though maybe I should just send a PR to VS Code to add an official setting for default language mode :)

Oh yeah, but I would start with an extension first to see how it works. You can easily map it to Ctrl+N for "New File" by default.

All in all, also not opposed to seeing a PR on this, e.g. could be something like editor.defaultLanguage or file.defaultMode.

file.defaultLanguage maybe?  Not sure that most users know what a "mode" is.

I'm going to work on this today!

Verify:

a new setting files.defaultLanguage allows to configure the default language mode a new untitled file should get
you can still change the mode from the picker and the untitled file will keep this association even between restarts


Cannot build

C:\Users\micro\Downloads\rwkv-cpp-cuda\build>cmake ..
CMake Warning:
  Ignoring extra path from command line:

   ".."

CMake Error: The source directory "C:/Users/micro/Downloads/rwkv-cpp-cuda" does not appear to contain CMakeLists.txt.
Specify --help for usage, or press the help button on the CMake GUI.

Hi! the example project now lives in ./examples/terminalchat
you can now build from there


Traceback stack does not show exact place of error

Discussed in https://github.com/fastapi/fastapi/discussions/8428

Originally posted by NewSouthMjos December  5, 2022
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
import uvicorn
from fastapi import FastAPI, Depends

app = FastAPI()

def get_something_sync():
    yield True

async def get_something_async():
    yield True

@app.get('/1')
def router_func(dependency=Depends(get_something_sync)):
    raise ValueError
    return

@app.get('/2')
def router_func(dependency=Depends(get_something_async)):
    raise ValueError
    return

if __name__ == "__main__":
    uvicorn.run("main:app", host="<IP_ADDRESS>", port=5600, workers=1)

Description
If dependency function uses yield, injecting dependency with sync def function (get_something_sync), the error traceback will broke - it doesn't locate the place where it was rise (should be "/app/main.py", line 24)
calling endpoint /1:
fastapi_dependency  | INFO:     <IP_ADDRESS>:60070 - "GET /1 HTTP/1.1" 500 Internal Server Error
fastapi_dependency  | ERROR:    Exception in ASGI application
fastapi_dependency  | Traceback (most recent call last):
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/uvicorn/protocols/http/httptools_impl.py", line 419, in run_asgi
fastapi_dependency  |     result = await app(  # type: ignore[func-returns-value]
fastapi_dependency  |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/uvicorn/middleware/proxy_headers.py", line 78, in __call__
fastapi_dependency  |     return await self.app(scope, receive, send)
fastapi_dependency  |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/fastapi/applications.py", line 270, in __call__
fastapi_dependency  |     await super().__call__(scope, receive, send)
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/starlette/applications.py", line 124, in __call__
fastapi_dependency  |     await self.middleware_stack(scope, receive, send)
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/starlette/middleware/errors.py", line 184, in __call__
fastapi_dependency  |     raise exc
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/starlette/middleware/errors.py", line 162, in __call__
fastapi_dependency  |     await self.app(scope, receive, _send)
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/starlette/middleware/exceptions.py", line 79, in __call__
fastapi_dependency  |     raise exc
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/starlette/middleware/exceptions.py", line 68, in __call__
fastapi_dependency  |     await self.app(scope, receive, sender)
fastapi_dependency  |   File "/usr/local/lib/python3.11/contextlib.py", line 222, in __aexit__
fastapi_dependency  |     await self.gen.athrow(typ, value, traceback)
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/fastapi/concurrency.py", line 36, in contextmanager_in_threadpool
fastapi_dependency  |     raise e
fastapi_dependency  | ValueError

When injecting dependency with async def function (get_something_async), the error traceback seems to be right
endpoint /2:
fastapi_dependency  | INFO:     <IP_ADDRESS>:60067 - "GET /2 HTTP/1.1" 500 Internal Server Error
fastapi_dependency  | ERROR:    Exception in ASGI application
fastapi_dependency  | Traceback (most recent call last):
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/uvicorn/protocols/http/httptools_impl.py", line 419, in run_asgi
fastapi_dependency  |     result = await app(  # type: ignore[func-returns-value]
fastapi_dependency  |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/uvicorn/middleware/proxy_headers.py", line 78, in __call__
fastapi_dependency  |     return await self.app(scope, receive, send)
fastapi_dependency  |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/fastapi/applications.py", line 270, in __call__
fastapi_dependency  |     await super().__call__(scope, receive, send)
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/starlette/applications.py", line 124, in __call__
fastapi_dependency  |     await self.middleware_stack(scope, receive, send)
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/starlette/middleware/errors.py", line 184, in __call__
fastapi_dependency  |     raise exc
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/starlette/middleware/errors.py", line 162, in __call__
fastapi_dependency  |     await self.app(scope, receive, _send)
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/starlette/middleware/exceptions.py", line 79, in __call__
fastapi_dependency  |     raise exc
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/starlette/middleware/exceptions.py", line 68, in __call__
fastapi_dependency  |     await self.app(scope, receive, sender)
fastapi_dependency  |   File "/usr/local/lib/python3.11/contextlib.py", line 222, in __aexit__
fastapi_dependency  |     await self.gen.athrow(typ, value, traceback)
fastapi_dependency  |   File "/app/main.py", line 13, in get_something_async
fastapi_dependency  |     yield True
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/fastapi/middleware/asyncexitstack.py", line 21, in __call__
fastapi_dependency  |     raise e
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/fastapi/middleware/asyncexitstack.py", line 18, in __call__
fastapi_dependency  |     await self.app(scope, receive, send)
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/starlette/routing.py", line 706, in __call__
fastapi_dependency  |     await route.handle(scope, receive, send)
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/starlette/routing.py", line 276, in handle
fastapi_dependency  |     await self.app(scope, receive, send)
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/starlette/routing.py", line 66, in app
fastapi_dependency  |     response = await func(request)
fastapi_dependency  |                ^^^^^^^^^^^^^^^^^^^
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/fastapi/routing.py", line 235, in app
fastapi_dependency  |     raw_response = await run_endpoint_function(
fastapi_dependency  |                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/fastapi/routing.py", line 163, in run_endpoint_function
fastapi_dependency  |     return await run_in_threadpool(dependant.call, **values)
fastapi_dependency  |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/starlette/concurrency.py", line 41, in run_in_threadpool
fastapi_dependency  |     return await anyio.to_thread.run_sync(func, *args)
fastapi_dependency  |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/anyio/to_thread.py", line 31, in run_sync
fastapi_dependency  |     return await get_asynclib().run_sync_in_worker_thread(
fastapi_dependency  |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/anyio/_backends/_asyncio.py", line 937, in run_sync_in_worker_thread
fastapi_dependency  |     return await future
fastapi_dependency  |            ^^^^^^^^^^^^
fastapi_dependency  |   File "/usr/local/lib/python3.11/site-packages/anyio/_backends/_asyncio.py", line 867, in run
fastapi_dependency  |     result = context.run(func, *args)
fastapi_dependency  |              ^^^^^^^^^^^^^^^^^^^^^^^^
fastapi_dependency  |   File "/app/main.py", line 24, in router_func
fastapi_dependency  |     raise ValueError
fastapi_dependency  | ValueError

On python 3.10 there were no such a problem.
Created repo for reproduce the problem: https://github.com/NewSouthMjos/fastapi_dependency_test
So, should the sync way to inject dependecy show exact plase of where is was raised?
Operating System
Linux
Operating System Details
Running in a docker
FastAPI Version
0.88.0
Python Version
3.11
Additional Context
uvicorn[standard]==0.20.0

I think the issue is not what it's described here, but I'll not spam another one to not create confusion.
The issue is actually this one: https://github.com/fastapi/fastapi/discussions/12999
The one mentioned in this issue is very similar, but it was solved.


Build error on Ubuntu 12

I have trouble building usrsctp-neat on Ubuntu 12 on Travis CI. This is the output:

CFLAGS=-fPIC cmake -DCMAKE_INSTALL_PREFIX=/home/travis/build/rawrtc/rawrtc/build/prefix -DSCTP_DEBUG=1 ..
make install -j12
Scanning dependencies of target usrsctp-static
Scanning dependencies of target usrsctp
[  2%] [  2%] [  4%] [  5%] [  7%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet6/sctp6_usrreq.c.o
[  8%] [ 10%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_callout.c.o
[ 11%] [ 13%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_input.c.o
[ 16%] [ 16%] [ 17%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet6/sctp6_usrreq.c.o
Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_sha1.c.o
Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_userspace.c.o
Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_input.c.o
Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_callout.c.o
Building C object usrsctplib/CMakeFiles/usrsctp.dir/user_mbuf.c.o
Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_sha1.c.o
Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_userspace.c.o
Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_asconf.c.o
[ 19%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/user_mbuf.c.o
[ 20%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_cc_functions.c.o
[ 22%] [ 23%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_asconf.c.o
[ 25%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_cc_functions.c.o
[ 26%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_output.c.o
Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_output.c.o
[ 28%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_ss_functions.c.o
[ 29%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_usrreq.c.o
[ 31%] [ 32%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/user_recv_thread.c.o
Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_auth.c.o
[ 34%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_crc32.c.o
/home/travis/build/rawrtc/rawrtc/build/dependencies/usrsctp/usrsctplib/user_recv_thread.c: In function ‘recv_icmp’:
/home/travis/build/rawrtc/rawrtc/build/dependencies/usrsctp/usrsctplib/user_recv_thread.c:408:29: error: ‘struct udphdr’ has no member named ‘uh_sport’
/home/travis/build/rawrtc/rawrtc/build/dependencies/usrsctp/usrsctplib/user_recv_thread.c: In function ‘recv_icmp6’:
/home/travis/build/rawrtc/rawrtc/build/dependencies/usrsctp/usrsctplib/user_recv_thread.c:845:29: error: ‘struct udphdr’ has no member named ‘uh_sport’
make[2]: *** [usrsctplib/CMakeFiles/usrsctp-static.dir/user_recv_thread.c.o] Error 1
make[2]: *** Waiting for unfinished jobs....
[ 35%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_ss_functions.c.o
[ 37%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_usrreq.c.o
[ 38%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/user_recv_thread.c.o
[ 40%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_auth.c.o
[ 41%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_crc32.c.o
/home/travis/build/rawrtc/rawrtc/build/dependencies/usrsctp/usrsctplib/user_recv_thread.c: In function ‘recv_icmp’:
/home/travis/build/rawrtc/rawrtc/build/dependencies/usrsctp/usrsctplib/user_recv_thread.c:408:29: error: ‘struct udphdr’ has no member named ‘uh_sport’
/home/travis/build/rawrtc/rawrtc/build/dependencies/usrsctp/usrsctplib/user_recv_thread.c: In function ‘recv_icmp6’:
/home/travis/build/rawrtc/rawrtc/build/dependencies/usrsctp/usrsctplib/user_recv_thread.c:845:29: error: ‘struct udphdr’ has no member named ‘uh_sport’
make[2]: *** [usrsctplib/CMakeFiles/usrsctp.dir/user_recv_thread.c.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [usrsctplib/CMakeFiles/usrsctp-static.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
make[1]: *** [usrsctplib/CMakeFiles/usrsctp.dir/all] Error 2
make: *** [all] Error 2

You can find the full log here which also includes precise information about the build system.

The other Travis CI environment also fails with no member named uh_dport.

There is no problem when using a more recent version of Ubuntu. On Buntu 16.10 you find in /usr/include/netinet/udp.h:
struct udphdr
{
  __extension__ union
  {
    struct
    {
      u_int16_t uh_sport;               /* source port */
      u_int16_t uh_dport;               /* destination port */
      u_int16_t uh_ulen;                /* udp length */
      u_int16_t uh_sum;         /* udp checksum */
    };
    struct
    {
      u_int16_t source;
      u_int16_t dest;
      u_int16_t len;
      u_int16_t check;
    };
  };
};

Can you post the corresponding definition from your system including all relevant #ifdefs around that? Possibly, an additional compile flag can resolve your issue.

It looks like this in Ubuntu 12.04:
/* UDP header as specified by RFC 768, August 1980. */
#ifdef __FAVOR_BSD

struct udphdr
{
  u_int16_t uh_sport;           /* source port */
  u_int16_t uh_dport;           /* destination port */
  u_int16_t uh_ulen;            /* udp length */
  u_int16_t uh_sum;             /* udp checksum */
};

#else

struct udphdr
{
  u_int16_t source;
  u_int16_t dest;
  u_int16_t len;
  u_int16_t check;
};
#endif

So, it seems that if you define __FAVOR_BSD it should be fine?

It was already set in sctp_output.c but not in sctputil.c and user_recv_thread.c. I fixed it in the usrsctp-neat fork for rawrtc - you can cherry-pick the commit from there.

Would it make sense to define this in the CFLAGS variable in the build environment? I guess that might be better?

Sure, that might be a better way to prevent further build errors in case other files start using the BSD style struct. Feel free to close my PR. :)


Deprecate DeferredConfigurable

Remove it from the publishing plugins, as they no longer require it.

@adammurdoch The native plugins were actually depending on the deferred configurable behavior. I think we can replace this with Provider (for the scalar values) and DomainObjectSet (for the set of variants). Does that sounds like a good approach to you?

@marcphilipp those should be fixed now.
I also discussed this with Adam and we decided to not change ComponentWithVariants for now, since we're likely going to change the way this laziness works in the near future.


Falco metrics do not populate in Prometheus with falco-exporter

Describe the bug
I've partnered with Udacity to create a course on microservices security. Falco is front and center per the good work of our community :wink:
Working through demos for the very last lesson in the course on runtime monitoring and incident response.  Intent is to teach students how to use falco for runtime monitoring and incident response. Roughly following Leo's awesome blog- https://falco.org/blog/falco-kind-prometheus-grafana/#install-prometheus
After much debugging, I'm not seeing the falco metrics in Prometheus and subsequently not in Grafana.
I'm under a huge time crunch, we need to ship the course, this is the last technical blocker. I would greatly appreciate your help folks. Hard blocked on completion and created alot of content for this already.
How to reproduce
To repro:

Create a two node (node1) RKE cluster via Vagrantfile and cluster.yaml

SSH into node1 and node2 and install kernel drivers for falco

rpm --import https://falco.org/repo/falcosecurity-3672BA8F.asc
curl -s -o /etc/zypp/repos.d/falcosecurity.repo https://falco.org/repo/falcosecurity-rpm.repo
Install kernel headers: zypper -n install kernel-default-devel

Generate certs manually and Deploy falco as a daemonset. note: Certs are generated manually and stored in certs/  directory on the machine I'm running helm from

helm install falco falcosecurity/falco \
  --set falco.grpc.enabled=true \
  --set falco.grpcOutput.enabled=true \
  --set-file certs.server.key=certs/server.key \
  --set-file certs.server.crt=certs/server.crt \
  --set-file certs.ca.crt=certs/ca.crt \

Deploy falco-exporter via falco-exporter helm chart with reference certs/

helm install falco-exporter \
	--set-file certs.ca.crt=certs/ca.crt,certs.client.key=certs/client.key,certs.client.crt=certs/client.crt \
	falcosecurity/falco-exporter

Deploy Prometheus via kube-prometheus-stack helm chart

helm install prometheus prometheus-community/kube-prometheus-stack

All relevant pods, everything is up separately, falco generates logs

kubectl get pods
NAME                                                     READY   STATUS    RESTARTS   AGE
alertmanager-prometheus-kube-prometheus-alertmanager-0   2/2     Running   0          105m
falco-86mzz                                              1/1     Running   1          3d4h
falco-exporter-jq869                                     1/1     Running   51         3d1h
prometheus-grafana-66c946f558-7j9hq                      2/2     Running   0          106m
prometheus-kube-prometheus-operator-779574c749-gb2h8     1/1     Running   0          106m
prometheus-kube-state-metrics-685b975bb7-xbbw5           1/1     Running   0          106m
prometheus-prometheus-0                                  2/2     Running   1          48m
prometheus-prometheus-1                                  2/2     Running   1          48m
prometheus-prometheus-kube-prometheus-prometheus-0       2/2     Running   1          105m
prometheus-prometheus-node-exporter-vxwst                1/1     Running   9          106m

Port forward all relevant pods:
kubectl --namespace default falco-exporter-jq869 9376
kubectl  --namespace default port-forward prometheus-grafana-66c946f558-7j9hq 3000
kubectl  --namespace default port-forward prometheus-prometheus-kube-prometheus-prometheus-0 9090

I see falco metrics in the falco-exporter endpoint http://<IP_ADDRESS>:9376/metrics

# HELP falco_events
# TYPE falco_events counter
falco_events{hostname="falco-86mzz",k8s_ns_name="<NA>",k8s_pod_name="<NA>",priority="4",rule="Mount Launched in Privileged Container",source="SYSCALL"} 1

Now configure an additional scraper for falco-exporter. Create prometheus-additional.yaml file

apiVersion: v1
kind: Secret
metadata:
  name: additional-scrape-configs
  namespace: default
stringData:
  prometheus-additional.yaml:
    - job_name: falco_exporter
      static_configs:
      metrics_path: /metrics
        - targets: ["falco-exporter:9376"]
type: Opaque

Use that file to run this command and generate a new file with secret additional-scrape-configs.yaml

kubectl create secret generic additional-scrape-configs --from-file=prometheus-additional.yaml  > additional-scrape-configs.yaml

Apply the additional-scrape-configs.yaml via kubectl  apply -f additional-scrape-configs.yaml

Create custom ServiceMonitor file falco_service_monitor.yaml and apply kubectl  apply -f falco_service_monitor.yaml`

apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
  name: falco-exporter
  labels:
    release: prometheus
    app: falco-exporter
spec:
  endpoints:
  - port: '9376'
    path: '/metrics'
  namespaceSelector:
    any: true
  selector:
    matchLabels:
      app: falco-exporter
      release: prometheus

From grafana, import falco grafana panel -> https://grafana.com/grafana/dashboards/11914

grafana dashboard is empty

The falco-exporter ServiceMonitor populates in Prometheus but no metrics.
prometheus service discovery ok for falco-exporter

prometheus metric source, no falco_events

Expected behaviour
Expect to see falco_events in Prometheus under and Grafana
Screenshots
falco-exporter

prometheus service discovery

prometheus metric source

grafana dashboard

Environment
RKE

Falco version:

docker.io/falcosecurity/falco:0.28.0

System info:

root@falco-86mzz:/# falco --support | jq .system_info
Mon Apr 26 02:36:32 2021: Falco version 0.28.0 (driver version 5c0b863ddade7a45568c0ac97d037422c9efb750)
Mon Apr 26 02:36:32 2021: Falco initialized with configuration file /etc/falco/falco.yaml
Mon Apr 26 02:36:32 2021: Loading rules from file /etc/falco/falco_rules.yaml:
Mon Apr 26 02:36:35 2021: Loading rules from file /etc/falco/falco_rules.local.yaml:
{
  "machine": "x86_64",
  "nodename": "falco-86mzz",
  "release": "5.3.18-lp152.72-default",
  "sysname": "Linux",
  "version": "#1 SMP Wed Apr 14 10:13:15 UTC 2021 (013936d)"
}

Cloud provider or hardware configuration:

Vagrantbox running openSUSE Leap hosted on macOS Catalina
NAME="openSUSE Leap"
VERSION="15.2"
ID="opensuse-leap"
ID_LIKE="suse opensuse"
VERSION_ID="15.2"
PRETTY_NAME="openSUSE Leap 15.2"
ANSI_COLOR="0;32"
CPE_NAME="cpe:/o:opensuse:leap:15.2"
BUG_REPORT_URL="https://bugs.opensuse.org"
HOME_URL="https://www.opensuse.org/"

Kernel:

Linux localhost 5.3.18-lp152.72-default #1 SMP Wed Apr 14 10:13:15 UTC 2021 (013936d) x86_64 x86_64 x86_64 GNU/Linux

Installation method:

See above
Additional context

Hi
Thank you for reporting this!
I'm linking the original discussion on Slack as references:

https://kubernetes.slack.com/archives/CMWH3EH32/p1619138180109800
https://kubernetes.slack.com/archives/CMWH3EH32/p1619144955110600

Moreover, it seems to me this could be an issue related to how the falco-exporter is deployed by its helm chart. If it were confirmed, I would move this issue to the charts repository.
Finally, just a question:
Instead of manually creating the service monitor, have you tried to use the chart's option intended for that (i.e. serviceMonitor.enabled)?
Thanks

thanks @leogr for your quick reply, I tried to redeploy falco-exporter with serviceMonitor.enabled switch, with no luck. Also tried to apply a custom ServiceMonitor config with no luck.
apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
  name: falco-exporter
  labels:
    release: prometheus
    app: falco-exporter
spec:
  endpoints:
  - port: '9376'
    path: '/metrics'
  namespaceSelector:
    any: true
  selector:
    matchLabels:
      app: falco-exporter
      release: prometheus

I will destroy the cluster and start over.

Hi @leogr , I rebuilt the cluster following the steps verbatim and don't see falco-exporter under targets and service discovery in prom on http://<IP_ADDRESS>:9090. I'm not sure how to further troubleshoot at the moment. Trying to get help from folks with depth on Prom.

Can you confirm this has been solved with the solution described by this comment https://kubernetes.slack.com/archives/CMWH3EH32/p1619753500281500?thread_ts=1619144955.110600&cid=CMWH3EH32 ?

It seems to me this issue belongs to the falco-exporter repository. Moving it there


stop using System::String to set shader variables

I'm seeing mucho garbage creation from using the Dotnet::System::String class. This pr address creating many new strings when setting shader variables, it is also more desirable to use ids anyways.
Usually string interning  would save us from the garbage creation but it only works with string literals.
I would suggest not creating any .NET strings at all if we could help it.
Why am I nitpicking GC? Keeping runtime allocations low are critically important for low power devices like a hololens2 as there is noticeable frame rate drop when the garbage collector runs. Ideally, there would be 0 runtime allocations, aside from resource creation that can't be helped like, meshes, and textures. Although we should probably be pooling the meshes as well instead of creating/destroying.
using shader id's is more performant than using strings.

Thanks @ZackOfAllTrades, this looks like a very valuable change! Can you please sign the Contributor License Agreement so that we can review and merge it? We don't seem to have one from you yet.
https://github.com/CesiumGS/cesium/blob/master/CONTRIBUTING.md#contributor-license-agreement-cla

Okay will do thanks!

I can confirm that @ZackOfAllTrades has signed the CLA as of 2023-02-22. Thanks once again for your contributions!

Thanks @ZackOfAllTrades, this is great!


Add CI For un-encrypted tunnels

What would you like to be added:
As a part of  https://github.com/submariner-io/submariner/issues/674 we are adding support for new Cable driver using vxlan . We need to have a dedicated CI job for this. Since there proposal to have a different kind of tunnels we could also have CI job in which one tunnel in un-encrypted.
Why is this needed:
To automate testing of  https://github.com/submariner-io/submariner/issues/674

[ ] Add vxlan option to shipyard Make.inc
[ ] Add a temporary GH workflow (e2e-vxlan.yml) in Submariner to test vxlan only on PRs with vxlan label
[ ] Once https://github.com/submariner-io/submariner/pull/1171 is merged, remove e2e-vxlan.yml and add vxlan testing to e2e .yml

bump

Add E2E tests to test VxLAN cable driver

Add vxlan option to shipyard Makefile.inc https://github.com/submariner-io/shipyard/pull/553


Add pyproject.toml for Custom Node Registry

We are working with dr.lt.data and comfyanon to build a global registry for custom nodes (similar to PyPI). Eventually, the registry will be used as a backend for the UI-manager. All nodes go through a verification process before being published to users.
The main benefits are that authors can

publish nodes by version and users can safely update nodes knowing ahead of time if their workflows will break or not
automate testing against new commits in the comfy repo and existing workflows through our CI/CD dashboard

Action Required:

[ ] Go to the registry. Login and create a publisher id. Add the publisher id into the pyproject.toml file.
[ ] Write a short description.
[ ] Merge the separate Github Actions PR and run the workflow.

If you want to publish the node manually, install the cli and run comfy node publish
Check out our docs if you want to know more about the registry. Otherwise, feel free to message me on discord at haohao_81202 or join our server if you have any questions!

hey @azure-dragon-ai, wanted to send a small nudge and ask if you had any questions!

@azure-dragon-ai, wondering if there's any way we can support you on this PR!


Add transpose for multi-select

Resolves one of the polish items in #323

Deepest apologies for taking so long to review. I had meant to earlier, but was very focused on the textplane stuff and found it difficult to focus mentally.
I like the fact that this captures the desired semantics (including preserving selections) without being a huge code change. However, the change to transform_selections makes me deeply uncomfortable; I don't feel confident I could write down exactly what it's supposed to do after this change. (I absolutely should write down what it's supposed to do now, but that's another question).
My preference would be to have the keep_selections logic as it is now, but to have clean separation between the two cases; if true, then the edit logic is entirely responsible for computing the new selection (which mostly means moving the carets to the "after" position), if false then it's as now, the update is done by OT.
There's an additional invariant which I think should probably be written down explicitly; an edit that's done with keep_selections on must appear in its own commit without other edits before or after. In other cases, it's possible to aggregate multiple edits together in a commit, though this isn't exploited in the current code (cmd_postlude is called after the handling of each notification).
Thanks again for taking this on!


Packet queues implementation

Description
Implementation for (FIFO and timed one) queues are added.
How Has This Been Tested?
Unit tests and integration tests.
Process
I know the process and did my best to follow it

Codecov Report

Merging #155 (361aff1) into main (d582052) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head 361aff1 differs from pull request most recent head 78d4403. Consider uploading reports for the commit 78d4403 to get more accurate results

@@            Coverage Diff            @@
##              main      #155   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           34        34
  Lines         1893      1957   +64
=========================================
+ Hits          1893      1957   +64

Flag
Coverage Δ

integration-tests
88.80% <41.17%> (-2.16%)
:arrow_down:

unit-tests
100.00% <100.00%> (ø)

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

Impacted Files
Coverage Δ

uds/utilities/__init__.py
100.00% <ø> (ø)

uds/utilities/common_types.py
100.00% <ø> (ø)

uds/message/uds_message.py
100.00% <100.00%> (ø)

uds/packet/__init__.py
100.00% <100.00%> (ø)

uds/packet/abstract_packet.py
100.00% <100.00%> (ø)

uds/packet/can_packet.py
100.00% <100.00%> (ø)

uds/packet/can_packet_record.py
100.00% <100.00%> (ø)

uds/transport_interface/abstract_packet_queue.py
100.00% <100.00%> (ø)

uds/transport_interface/packet_queues.py
100.00% <100.00%> (ø)


dot-repeating ct hangs

When editing, e.g.,
foo =    bar
buzz =     bazz

to become
foo = bar
buzz = bazz

I move my cursor like so (indicated by X)
foo =X   bar
buzz =    bazz

then type ct=<Space>
foo = bar
buzz =    bazz

then I move to the next occurence
foo = bar
buzz =X   bazz

and press . to repeat the sequence ct=<Space>. Instead of simply doing that like normal, Neovim hangs until i <C-c>.

caused by failure to properly install repeat.vim


~/.local/share/code-server/User/settings.json is not respected in code-server v4.0.0

OS/Web Information

Web Browser: Firefox 95.0 (64-bit)
Local OS: macOS 10.14.6 (Mojave)
Remote OS: Debian bullseye
Remote Architecture: 64-bit
code-server --version: 4.0.0 0f395955515207f9b461ffdd2a595f35311dd05f

Steps to Reproduce
Log in at https://vscode-r.jupyter.b-data.ch.
Expected
The following settings should be applied:
~/.local/share/code-server/User/settings.json
{
  "editor.tabSize": 2,
  "telemetry.enableTelemetry": false,
  "gitlens.advanced.telemetry.enabled": false,
  "r.bracketedPaste": true,
  "r.plot.useHttpgd": true,
  "r.rterm.linux": "/usr/local/bin/radian",
  "r.rterm.option": [],
  "r.workspaceViewer.showObjectSize": true,
  "workbench.colorTheme": "Default Dark+",
  "terminal.integrated.fontFamily": "MesloLGS NF",
  "r.sessionWatcher": true
}

Actual
Settings are not applied.
ℹ It seems v4.0.0 is using the browser storage instead of the file system to read/set settings.
Logs
On demand only.
👉 Issue may be reproduced any time by @code-asher, @jsjoeio or @bpmct at https://vscode-r.jupyter.b-data.ch.
Screenshot
None.
Notes
See https://github.com/cdr/code-server/issues/2274 for an identical issue with previous versions.
This issue can be reproduced in VS Code: No

I don't mind that the state is stored in the browser's storage (https://github.com/cdr/code-server/issues/4212).
But one should habe the possibility to pre-populate ~/.local/share/code-server/User/settings.json with default settings (https://github.com/cdr/code-server/issues/2274#issuecomment-722285783), e.g. when creating a docker image.

On a similar note, "bind-addr:" in ~/.config/code-server/config.yaml is not respected either.
All other default options are still respected though.

Might be another patch that was not ported over when we hard reset to upstream's server.  Hopefully I can just apply the old patch and it will work.  This is probably why https://github.com/cdr/code-server/issues/2306 has cropped up again too.
I think we may want to enforce e2e tests for various patches we make to VS Code since otherwise we tend to lose them when major refactors due to upstream changes like this happen.  Individual patch files instead of a fork would also make the changes more obvious which might be worth discussing.

Individual patch files instead of a fork would also make the changes more obvious which might be worth considering.

I was thinking about this too. Could we combine the workflows somehow? I.e. keep the fork (so we can submit upstream) and then maintain patches in that repo?

Could definitely do.  One advantage to keeping patches here is that we would not need to make double PRs for every fix.

Let's plan to have a discussion offline in January and then I'll share notes in a Discussion afterwards with the community.


[ACM-5057] Use cluster_version metric for cluster label_values (#1177)

Switch to using cluster_version metric instead of the expensive kube_pod_info metric to obtain cluster labels

/hold

/retest

/retest

/retest-required

/retest

/lgtm


Make enums hashable

Closes #236

@idavis I was wondering if there will be a new release for pyqir anytime soon incorporating this change.


visual studio code...cannot open vs code....Unable to write program user data.......A system error occurred (EPERM: operation not permitted, mkdir 'C:\Users\ ......Please make sure the following directories are writeable:

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

VS Code Version:
OS Version:

Steps to Reproduce:

Is this a new installation, or one that used to work OK?
Did you install from the Windows User Installer (the default download) or from the System Installer?
If the User Installer did you simply run it, or use "Run as administrator"?
What Windows version?


Missing parameter in SAML assertion.

We have implemented the Identity Server 4 with SAML2p and noticed that SAML assertion does not contain these below parameters. This entire block is missing, are we missing anything to enable this? please help.
    <saml:Conditions NotBefore="2018-05-31T16:07:28.717Z"
                     NotOnOrAfter="2018-05-31T16:09:28.717Z"
                     >
        <saml:AudienceRestriction>

Anybody has nay idea?

@scottbrady91

Please contact support at Rock Solid Knowledge for this, as it's a feature of that product not IdentityServer itself. Thanks.

@frjaffery Is this the same as the issue you have already reported? Or something new? If this is different, please contact<EMAIL_ADDRESS>

Yes, this is the same as we have already reported.


Synchronize .ssh/authorized_keys on request

There is no way to synchronize authorized key in web and the .ssh/authorized_key.
I want to make sure there is no other key in the authorized_key except the keys from gogs. the problem was I reinstall gogs in my machine and I get this error :
 Gogs: you are not the owner of SSH key

Another thing is if this file is out of sync, then the ssh add/delete dose not work correctly. but I'm not sure about this.

fixed by #818


Cannot bump node count for K8s clusters

Latest rancher/server:master.
Create a cluster on DO with say 3 master/etcd nodes and 5 worker nodes.
Now try to bump up worker node count or etcd node count.
It does not work. Instead all nodes are deleted and the same number of nodes are recreated. And the resulting cluster is completely broken.

Also, tested by creating 4 other clusters where the etcd nodes and worker nodes were incremented[Through edit cluster].  Did not see any issues.

Tested with master 3/1
k8s clusters created from node pools only allows incrementing nodes from the node pools within "edit cluster".
When bumping up nodes, only new nodes are created.
Any issues with a resulting cluster not being able to go back to active is RKE related.