Is there any plans to develop a "Table" component?

Is your feature request related to a problem? Please describe.
I need to develop a management type page, and this time the "table" component is essential.
Describe the solution you'd like
Add a "table" component
Describe alternatives you've considered
Using a third-party "table" component?can be yes, but the overall feeling of style is not very good.

#96
Just working on that component at this time, would be created today in 2 days or so or 3 days for the complexity you want to achieve
As soon as I finish I update to add it and I can review it
thank you very much

＃96此时
正在处理该组件，将在2天左右或3天内创建，以实现您希望实现的复杂性
我一完成就更新添加它，我可以查看它
非常感谢你

#96
Just working on that component at this time, would be created today in 2 days or so or 3 days for the complexity you want to achieve
As soon as I finish I update to add it and I can review it
thank you very much

Thank you.

#96
Just working on that component at this time, would be created today in 2 days or so or 3 days for the complexity you want to achieve
As soon as I finish I update to add it and I can review it
thank you very much

Thank you.

Hi, to add to the table request, I hope it handles complex datas like datatable plugin. When our team are trying new frameworks, we always check their table components because that is the most tricky one. :)

Added the vs-table component in the latest version, thank you very much
https://lusaxweb.github.io/vuesax/components/table.html#default

Hey, @luisDanielRoviraContreras is there any problem on site? Your link redirects to 404 (and some other links), correct link is https://lusaxweb.github.io/vuesax/vuesax/components/table.html

@ridvankaradag Thank you very much for letting us know it has been fixed so that the links work as before, thanks for letting us know
this should work again
https://lusaxweb.github.io/vuesax/components/table.html#default

Are you planning to add search, filtering, and server-side pagination?


Error while reading manifest.json: The file could not be found in the app package.

I've followed the deployment guide, but can't upload the app to Teams - either through sideloading or through the Teams admin portal (we don't usually allow sideloading in our tenant, but I've done it for this purpose).
Error while reading manifest.json: The file could not be found in the app package.
I've rerun the deploy script a couple of times to re-generate the app package, but no dice.
Any ideas where I start?

Update: after digging through some other threads & finding the MS Teams App Validator, I've solved the issue: https://dev.teams.microsoft.com/appvalidation.html

the language json files needed to be added to the manifest zip
the outline.png file needed to be 32x32px not 20x20px

After fixing both those things, I was able to upload the app to Teams with no further issues.


build error:

using "mvn clean install" to install, but find 1 build error. please give me some advise. Thank you.
error:

/Luyten-0.5.4/src/us/deathmarine/luyten/Model.java:[648,74] unresolved compilation problem: java.util.Enumeration<javax.swing.tree.TreeNode> cannot convert to java.util.Enumeration<javax.swing.tree.DefaultMutableTreeNode>
my java -version:
java version "12.0.2" 2019-07-16
Java(TM) SE Runtime Environment (build 12.0.2+10)
Java HotSpot(TM) 64-Bit Server VM (build 12.0.2+10, mixed mode, sharing)

Java 12?
This is a compatibility issues. Beyond Java 8 I cannot guarantee anything. I don't even know if procyon can be built on this Java version.


Desired Behavior
Please add 'the' before the campaign name and update our member count to 5.3 million.
It should read:
By joining the [campaign] campaign, you've teamed up with 5.3 million other members who are making an impact on the causes affecting your world.

for context


[Ministery Admin] - add functionality to craete admin ministery

create admins wrapper component

add banner that has dropdown button with options to create three types of admins

add create-admin component

add basic functionality to create admin of ministry (all fields, but "city" is not allowed yet, you can just mock it for now)

the component should be added to the data module

don't forget about mobile view
you can find an example in provider-admins/ create provider admin
endpoints:

Done


Start trying to make netlify dev work WIP

@BrianMMcClain I've almost got this. There's a final redirect that doesn't seem to work. For some reason the path in auth-start.js is /.netlify/non-existent-path and I don't think it's supposed to be so it ends up with an infinite redirect loop. Maybe you can figure it out, I'll look more another time as well.

@BrianMMcClain I found this https://github.com/netlify/cli/blob/main/src/utils/proxy.js#L144
I think this is what's causing the problem. The nf_jwt cookie...

needs to be set without the domain parameter for localhost (easy fix similar to what I did with the other cookie)
needs to be set earlier I think. It's currently getting set in auth-callback instead of auth-start based on looking at this code. I think since token in the netlify cli code is not being set it never sets the URL. May have to adjust the flow a little bit for localhost...

Also a note, we should move this line https://github.com/vmware-tanzu/tanzu-dev-portal/blob/main/netlify/functions/auth-callback.js#L124 to a separate file ideally and read it in to a variable. This is pretty rough to maintain as is...

@bryanfriedman This sets the jwt token now, but still getting the redirect loop. Looking into if it needs to be set earlier like you mentioned

@BrianMMcClain Can you give this a really quick once over? I think it's good to merge. I tested in the deploy preview and workshops launch okay, nothing else should really be affected in prod, only locally, where it also seems to work. If it looks good, feel free to approve and merge.


Style: conversion of 3 column design to 2 column design

What's the suggested design idea?
The feed page is currently having 3 columns. The left most column only contains a user button and a bookmark button. I was thinking we could navigate those button at the top in the navbar and extend the feed panel to make a consistent 3 column structure with 75-25 ratio of main and side column.
Add screenshots
Removal of mention buttons

Record

[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue

assign me

Assigned

how is it

also do you want bookmark should also need to sync with feed like profile section is..?
current bookmark

how is it

It's great and is the expected result

also do you want bookmark should also need to sync with feed like profile section is..?
current bookmark

Currently not have a plan for this


Form hideErrors prop should not be passed down to html form element

The doc site has a js error caused by the hideErrors prop getting passed down to the html form element:

Fix should just be adding hideErrors to the blacklist in this line like so:
const othersFiltered = blacklist(others, "onSubmit", "hideErrors");
That will prevent it from getting added to the  dom element, since it only gets othersFiltered.
You'll need to update snapshot test as well before putting in a PR.

Fixed with https://github.com/DigitalRiver/react-atlas/pull/986


Change Spain Central row location to Europe

@guywi-ms
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Progress of codes?

Thanks for your great work!
What is the plan to make code publicly available?
The proposed methods and modified dataset OPUS-100 are really beneficial to the NLP community.

Same question here

Released now, sorry for the delay

Released now, sorry for the delay


Cant make coexist two env in the same aws profile

First of all, I wanted to thank you for your work on this tool.
I am having trouble making coexist two envs in the same AWS account, since the tables of our envs are differentiated by name.
Ex:
TableOne-Dev
TableTwo-Dev
TableOne-Prod
TableTwo-Prod
I understand the tool is thought to use the --profile dev/stage/prod. But it doesn't work in our case.
But when we want to run migrations the tool cant know what migrations were already run in Dev and what were run in Prod, since all are stored in the same place (MIGRATIONS_LOG_DB).
This forced us to duplicate the migrations naming them:<PHONE_NUMBER>190-migration-one-DEV.mjs and<PHONE_NUMBER>190-migration-one-PROD.mjs, so the same database can store history for both ends in parallel.
Could you please add a config that allows us to specify the env? For example:
dynamo-data-migrations up --env [Prod] to allow us to differentiate different envs.
The same could happen for other users that use your tool with two different apps. They should be careful of not having the same name in two files of their apps (unlikely, but possible)
That might be very helpful. The tool could use that env parameter to use a different table like: MIGRATIONS_LOG_DB_DEV and MIGRATIONS_LOG_DB_PROD. It might also work if the differentiation is done by adding a field like ENV to the current MIGRATIONS_LOG_DB table like:
ENV: Dev
FILE_NAME:<PHONE_NUMBER>190-migration-one.mjs
APPLIED_AT: 2024-05-30T11:32:19.478Z

ENV: Prod
FILE_NAME:<PHONE_NUMBER>190-migration-one.mjs
APPLIED_AT: 2024-05-30T11:32:19.478Z

It would also do the trick for us, If you allow us to define the name of the table where the migrations log is stored like:
dynamo-data-migrations up --log-table MIGRATIONS_LOG_DB_DEV
dynamo-data-migrations up --log-table MIGRATIONS_LOG_DB_PROD

I'm currently facing the same challenge. Any workarounds for this perhaps?

I'm currently facing the same challenge. Any workarounds for this perhaps?

I'm afraid not, @ChipiKaf . I still haven't figured out how to solve it. I have considered forking from this project and adapting it to my needs, but I'm still hopeful to find another solution.

@ChipiKaf I forked from this project and created a package with the new feature of changing the prefix of the migrations table:
https://github.com/dservidie/dynamo-data-migrations
https://www.npmjs.com/package/@dservidie/dynamo-data-migrations

Hi @dservidie .
Thanks. Will give it a try!


CLIP different text and image embedding sizes

As per the CLIP paper's pseudo-code implementation, it has the feature to use different embedding sizes for image encoder and text encoder. To do this -

They have two additional linear layers - One for Text Encoded to Text embedded and another for images.
These multimodal embeddings are L2 Normalized
The scaling factor used in the current implementation (dim**-0.5) is different from pseudo-code, which uses a learned temperature parameter.

Partial Pseudo-Code -
I_f = image_encoder(I) #[n, d_i]
T_f = text_encoder(T) #[n, d_t]
# joint multimodal embedding [n, d_e]
I_e = l2_normalize(np.dot(I_f, W_i), axis=1)
T_e = l2_normalize(np.dot(T_f, W_t), axis=1)
# scaled pairwise cosine similarities [n, n]
logits = np.dot(I_e, T_e.T) * np.exp(t)

@asvskartheek Thank you! Let me know if this looks good https://github.com/lucidrains/DALLE-pytorch/commit/10ada06a4cf60de3446ec257078f8ef029f7547f

@asvskartheek Thank you! Let me know if this looks good https://github.com/lucidrains/DALLE-pytorch/commit/10ada06a4cf60de3446ec257078f8ef029f7547f

I'm always amazed by your lightspeed implementations! This looks very much in line with the pseudo-code and closing the issue.

I'm always amazed by your lightspeed implementations! This looks very much in line with the pseudo-code and closing the issue.


17.1.0 apparently breaks automat (and thus Twisted)

  File "/home/travis/build/fusionapp/entropy/.tox/py27/lib/python2.7/site-packages/xmantissa/port.py", line 52, in <module>
    from twisted.application.strports import service
  File "/home/travis/build/fusionapp/entropy/.tox/py27/lib/python2.7/site-packages/twisted/application/strports.py", line 14, in <module>
    from twisted.application.internet import StreamServerEndpointService
  File "/home/travis/build/fusionapp/entropy/.tox/py27/lib/python2.7/site-packages/twisted/application/internet.py", line 542, in <module>
    class _ClientMachine(object):
  File "/home/travis/build/fusionapp/entropy/.tox/py27/lib/python2.7/site-packages/twisted/application/internet.py", line 842, in _ClientMachine
    outputs=[_connect])
  File "/home/travis/build/fusionapp/entropy/.tox/py27/lib/python2.7/site-packages/automat/_methodical.py", line 62, in upon
    self.machine._oneTransition(self, input, enter, outputs, collector)
  File "/home/travis/build/fusionapp/entropy/.tox/py27/lib/python2.7/site-packages/automat/_methodical.py", line 255, in _oneTransition
    tuple(outputTokens))
  File "/home/travis/build/fusionapp/entropy/.tox/py27/lib/python2.7/site-packages/automat/_core.py", line 74, in addTransition
    (inState, inputSymbol, outState, tuple(outputSymbols))
exceptions.TypeError: unhashable type: 'MethodicalState'

cough https://github.com/python-attrs/attrs/issues/136#issuecomment-277881702

You'll have to opt into being hashable by using `attr.s(hash=True)' since MethodicalState is mutable. Or make it frozen, then you'll get hashability by default.

Oh I see, it's fixed in automat master but not 0.5.0 I think. Ping @glyph I guess?

As the above reference shows, this is now fixed in Automat and therefore Twisted.  But we should probably work out a system for compatibility testing in advance so this doesn't result in a fire drill while everyone is on planes for PyCon ;-).  Thanks very much to @markrwilliams for the SUPER rapid response fix & release.

Yay Mark! 🎉
Also yay software that's always ready for release.

Closing this since automat is fixed and it seems like this isn't an attrs bug insofar as the compat break was deliberate.


What you think about FastAPI Amis Admin?

a very similar project with a reach components library:
https://github.com/amisadmin/fastapi-amis-admin
can compare with your library, are there any plans for SSR?

a very similar project with a reach components library:
https://github.com/amisadmin/fastapi-amis-admin
can compare with your library, are there any plans for SSR?

I think Amis docs has a problem, it doesn't have English translation but it's fantastic.

translating Fastapi Amis Admin fully to English - in progress, but the Chinese version will be present anyway optionally
for rendering using Baidu Amis Framework, yep - there is China language (this little confuses me, right, I am from Ukraine :) ). Post a few issues and use English languages, no issues - guys quickly fix and answer in the English language
kill feature - designer, like in Delphi IDE!!!
in the real world - requires read sources and reading docs at any keys, component library - huge and very cool

The principle behind FastUI is that you can use it to build admin interfaces, but also more custom UIs.
But we could/should build abstraction that make FastUI even easier to use for admin interfaces.

are there any plans for SSR?

Yes, but not in the immediate future - there's quite a lot of work to do first.


⚠ Samsung has degraded performance

In 9ba22fc, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2464 ms

Resolved: Samsung performance has improved in 65ac363.


Latest Version Troubles

Diving into the rabbit hole here. 😆
Looks like Blazor has updated its entire bits lately and this project's dependencies are out of date.
For instance, it seems that a new Blazor project from this resource utilizes .razor files whereas this project uses .cshtml files.
I am a total newb looking through this, but if it's a matter of updating file extensions and dependencies, I would be interested in submitting a PR to update everything. 👍

Wow that will be great!! Note that to integrate reactive UI this is not your vanilla Blazor.

OK COOL. 😁  I will see what I can do and update here if I can/can't. 👍

Sure enough, I have a nasty one that appears to be due to DynamicData's System.Reactive.
I see a System.Reactive.dll checked into source... is the idea to copy that in somehow?
Here's the stack I am seeing:
Uncaught (in promise) Error: System.TypeLoadException: Could not find method '.ctor' due to a type load error: Could not load type of field 'DynamicData.ObservableCache`2[TObject,TKey]:_changes' (0) due to: Could not resolve type with token 0100002b from typeref (expected class 'System.Reactive.Subjects.Subject`1' in assembly '') assembly: type:System.Reactive.Subjects.Subject`1 member:(null) assembly:DynamicData.dll type:ObservableCache`2 member:(null)
  at NethereumBlazor.Services.AccountsService..ctor (NethereumBlazor.Services.IWeb3ProviderService web3ProviderService) [0x0001a] in C:\- Remote\NethereumBlazor\NethereumBlazor\Services\AccountsService.cs:35
  at NethereumBlazor.Startup.ConfigureServices (Microsoft.Extensions.DependencyInjection.IServiceCollection services) [0x00012] in C:\- Remote\NethereumBlazor\NethereumBlazor\Startup.cs:14
  at (wrapper managed-to-native) System.Reflection.RuntimeMethodInfo.InternalInvoke(System.Reflection.RuntimeMethodInfo,object,object[],System.Exception&)
  at System.Reflection.RuntimeMethodInfo.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) <0x2538fb0 + 0x000d2> in <e904f5829c7444acab57d4661a39f734>:0
--- End of stack trace from previous location where exception was thrown ---

  at Microsoft.AspNetCore.Blazor.Hosting.ConventionBasedStartup.ConfigureServices (Microsoft.Extensions.DependencyInjection.IServiceCollection services) <0x2523480 + 0x0007e> in <91b564433c704544acd7894656c3aff9>:0
  at (wrapper delegate-invoke) System.Action`1[Microsoft.Extensions.DependencyInjection.IServiceCollection].invoke_void_T(Microsoft.Extensions.DependencyInjection.IServiceCollection)
  at Microsoft.AspNetCore.Blazor.Hosting.WebAssemblyHostBuilderExtensions+<>c__DisplayClass1_0.<ConfigureServices>b__0 (Microsoft.AspNetCore.Blazor.Hosting.WebAssemblyHostBuilderContext context, Microsoft.Extensions.DependencyInjection.IServiceCollection collection) <0x2523020 + 0x00014> in <91b564433c704544acd7894656c3aff9>:0
  at (wrapper delegate-invoke) System.Action`2[Microsoft.AspNetCore.Blazor.Hosting.WebAssemblyHostBuilderContext,Microsoft.Extensions.DependencyInjection.IServiceCollection].invoke_void_T1_T2(Microsoft.AspNetCore.Blazor.Hosting.WebAssemblyHostBuilderContext,Microsoft.Extensions.DependencyInjection.IServiceCollection)
  at Microsoft.AspNetCore.Blazor.Hosting.WebAssemblyHostBuilder.CreateServiceProvider () <0x24fffd0 + 0x000e8> in <91b564433c704544acd7894656c3aff9>:0
  at Microsoft.AspNetCore.Blazor.Hosting.WebAssemblyHostBuilder.Build () <0x24ecb98 + 0x00050> in <91b564433c704544acd7894656c3aff9>:0
  at NethereumBlazor.Program.Main (System.String[] args) [0x00001] in C:\- Remote\NethereumBlazor\NethereumBlazor\Program.cs:9
    at Object.callMethod (http://localhost:52910/_framework/blazor.webassembly.js:1:35424)
    at Object.callEntryPoint (http://localhost:52910/_framework/blazor.webassembly.js:1:35002)
    at http://localhost:52910/_framework/blazor.webassembly.js:1:31090
    at http://localhost:52910/_framework/blazor.webassembly.js:1:29898
    at Object.next (http://localhost:52910/_framework/blazor.webassembly.js:1:30003)
    at i (http://localhost:52910/_framework/blazor.webassembly.js:1:28749)

doh... I created a pull request, but then I realized, that I didn't need to do that.  Oh well... it's there anyways. 😁  Still not proficient with git and its workflows yet.

Reactive UI is a fork and a submodule so you need to do git submodule init (the idea is that any changes that are made in the fork to support Blazor can be pushed eventually to ReactiveUI)

Ah ok... well the mystery deepens, then.  I do have ReactiveUI locally and everything compiles fine and launches.  It's only when trying to run the application that it throws that error at runtime.
I did run the above command, as well as git submodule update --init --recursive from this article, but that did not seem to fix this issue, unfortunately.

I modified ReactiveUI slightly did you used that version?

I am using what is in the repo. 😁
To assist, this reproduces the error:
	public class Program
	{
		public static void Main(string[] args)
		{
			new System.Reactive.Subjects.Subject<object>();
		}
	}

I am confused if we are talking about ReactiveUI or System.Reactive. 😆

hmm I might have been doing some experimental shit that got checked in by mistake :) Let me pull out master again and give it twirl.

Great, thank you.  To be sure this looks like a System.Reactive issue and not ReactiveUI.
I do see a System.Reactive.dll in the root of NetereumBlazor.csproj.  Is that used or should we delete that?

Ok let me try to do the port as it is not as straight forward.. as there a few breaking changes on blazor, before I get to that part :)

@Mike-EEE  Done :) hopefully it is ok for you.
I had some issue on startup on porting, mainly not happy to start processing the timer based services until fully completed the startup.
Other than that, changes.. etc
I have removed that System.Reactive.dll not needed
I still use the old Reactive.UI fork (that had no issues).
I did a dotnet restore / dotnet build directly in the Reactive.UI project folder.

Woohoo!  It works!  Although, I have no idea why it was broken on my machine.  I do not see anything in your commit that is different from my pull request, except that you didn't upgrade DynamicData whereas I did.  I did try to revert that, but I still got the same error.
Sorry to make you go through that, that's exactly that I tried to avoid. 😆  I will go ahead and close #8 and probably flatten my fork, too. :P

Oh yeah, we can close this one as well.  Thanks again!

@Mike-EEE  oh I did not see your pull, I thought it was a mistake.


ReplicateError input: prompt is required

HI!
I am exploring the capabilities of Llama 2 70b and found a bug.
I wrote the temp solution, but I think many people are extremely curious about llama-2-70b,
so it would be great to get a working fix (no temp one).
replicate==0.11.0
langchain==0.0.250
python 3.10
I tried to run that code and it worked.
But, with:
model="meta/llama-2-70b:8570a72c77eeae584bbeded5b0928704a44246ba77728bd5223fbc3d6ef2888e"

I got "ReplicateError input: prompt is required".
So, I debugged and found that in the file:
/usr/local/anaconda3/envs/llm/lib/python3.10/site-packages/replicate/prediction.py
at class "PredictionCollection(Collection)", inside the method "create" we have such part:
resp = self._client._request(
            "POST",
            "/v1/predictions",
            json=```
,
        )

where replicate_weights value (inside input, which is inside body) is replaced with promt.
So, after I added that code before "resp = self._client._request" - the error is gone:
if 'prompt' not in body['input']:
    body['input']['prompt'] = body['input']['replicate_weights']
    del body['input']['replicate_weights']

Hi @kzaho. Thanks for the detailed write-up. This is a problem in LangChain's integration with Replicate, rather than the Python library itself (even though you can work around this issue by patching the client library).
The bug comes from the following lines, where the prompt field is derived by selecting the first ordered field in the schema rather than, say, a field named "prompt":
https://github.com/langchain-ai/langchain/blob/01e9d7902d4bdd0de88bc8d63c96d8d8894b8c4c/libs/langchain/langchain/llms/replicate.py#L118-L125
Working on a solution for that now.

@mattt, thanks for a quick response!
I will follow up on updates in integration)

@kzaho A workaround was introduced upstream by https://github.com/langchain-ai/langchain/pull/10512. You can now set the prompt_key argument to "prompt":
llm = Replicate(
    model="a16z-infra/llama13b-v2-chat:df7690f1994d94e96ad9d568eac121aecf50684a0b0963b25a41cc40061269e5",
    model_kwargs={"temperature": 0.75, "max_length": 500, "top_p": 1},
    prompt_key="prompt"
)
prompt = """
User: Answer the following yes/no question by reasoning step by step. Can a dog drive a car?
Assistant:
"""
llm(prompt)

Hopefully this unblocks you. I don't personally like this extra step, so I'll try to contribute a different fix upstream to make the parameter unnecessary in most cases.
On Replicate's side, replicate_weights will now show up last in the list of inputs, so this won't be an issue for new models created as of a few days ago.

@mattt Thanks so much for the update!
I am closing the issue because it is solved for now)
Thank you for contributing; I hope you have time to do what you mentioned.


Running stack build is displaying an ExitFailure1

Hi,
I'm getting the following error when I run stack build...
$ stack build
servant-quickcheck-sandbox-<IP_ADDRESS>: build (lib + exe)
Preprocessing library servant-quickcheck-sandbox-<IP_ADDRESS>...
Preprocessing executable 'check-tags' for
servant-quickcheck-sandbox-<IP_ADDRESS>...
[1 of 1] Compiling Main             ( app/Main.hs, .stack-work/dist/x86_64-osx/Cabal-<IP_ADDRESS>/build/check-tags/check-tags-tmp/Main.o )

/Users/joshuagorospe/haskell/gorospe5_servant-quickcheck-sandbox-master/app/Main.hs:39:38: error:
    • Couldn't match type ‘Maybe a0’ with ‘[Char]’
      Expected type: [a0] -> String
        Actual type: [a0] -> Maybe a0
    • In the second argument of ‘(.)’, namely ‘listToMaybe’
      In the second argument of ‘maybe’, namely ‘(T.pack . listToMaybe)’
      In the first argument of ‘fmap’, namely
        ‘(maybe "" (T.pack . listToMaybe))’

/Users/joshuagorospe/haskell/gorospe5_servant-quickcheck-sandbox-master/app/Main.hs:39:52: error:
    • Couldn't match type ‘[String]’ with ‘Maybe [a0]’
      Expected type: IO (Maybe [a0])
        Actual type: IO [String]
    • In the second argument of ‘fmap’, namely ‘getArgs’
      In a stmt of a 'do' block:
        search <- fmap (maybe "" (T.pack . listToMaybe)) getArgs
      In the expression:
        do { search <- fmap (maybe "" (T.pack . listToMaybe)) getArgs;
             manager <- newManager
                          (defaultManagerSettings {managerModifyRequest = logIt});
             tagRes <- runClientM
                         (queryTags (SearchTerm search)) (ClientEnv manager baseUrl);
             case tagRes of {
               Left err -> fail (show err)
               Right theTags -> mapM_ (TIO.putStrLn . name) theTags };
             .... }

--  While building package servant-quickcheck-sandbox-<IP_ADDRESS> using:
      /Users/joshuagorospe/.stack/setup-exe-cache/x86_64-osx/Cabal-simple_mPHDZzAJ_<IP_ADDRESS>_ghc-8.0.2 --builddir=.stack-work/dist/x86_64-osx/Cabal-<IP_ADDRESS> build lib:servant-quickcheck-sandbox exe:check-tags --ghc-options " -ddump-hi -ddump-to-file"
    Process exited with code: ExitFailure 1

Thanks @jg8481it


Add python 3.13 support

This PR is just a draft to see if torchjd already works with python 3.13, and with which OS.

It looks like we need:

scs to start supporting python 3.13. Their maintainer said "Oh also we don't have the wheels published for 3.13 yet I think, it's on my todo list!" 3 weeks ago so it should come relatively soon.
torch to support python 3.13 on MacOS and Windows. See here for the status. This should come with torch 2.6.0.


Publish autogenerated API docs for the base check to Readthedocs

What does this PR do?
Adds a sphinx build to autogenerate API docs for the base check.
Readthedocs was set up to run the build and make the docs public, here's the preview:
https://datadog-checks-base.readthedocs.io/en/massi-rtd/#
Motivation
API docs are extremely helpful and we don't have anything like that yet.
Review checklist

[ ] PR has a meaningful title or PR has the no-changelog label attached
[ ] Feature or bugfix has tests
[ ] Git history is clean
[ ] If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo

How is this doc going to be updated? Should we add something to our build process?

@gzussa repo automatically pulled from readthedocs at every commit

I guess we need to better document our code, because this is weird: https://datadog-checks-base.readthedocs.io/en/massi-rtd/datadog_checks.checks.html#module-datadog_checks.checks.base


Adds the ability to receive pickles over UDP

One of the developers I work with was wondering why he couldn't send pickles over UDP, while he could send line data over UDP. This adds the ability, in case anyone else wants to do that.

@hufman So the intent here is not to add client UDP/pickle support (for inter-daemon communications), just the listener service for developers tooling their stuff accordingly?

@hufman After thinking about it a bit more, it seems like a benign addition -- but I worry about the confusion of Yet Another Listener, specifically one that few people will use. TBH I'm not even a fan of the UDP listener (I want deliverability guarantees for my metrics) but I can see why some might need it.
As it stands I'm going to close this out but if you can make a strong case for its inclusion I'll put in the work to merge the conflicts.


Memory fault (core dumped) on QNX using a large Capacity Vector or other container

Required information
Test Env:
RaspberryPi4B aarch64le
CPU：ARM Cortex-A72 1.5GHz
QNX：qnx710
Mem:2G
Operating system:
QNX 7.1.0
Compiler version:
QCC 8.3.0
Observed result or behaviour:
Process 13226012 (iox-test-pub) terminated SIGSEGV code=1 fltno=11 ip=00000057020678a0(/mnt1/iceoryx/bin_test/./iox-test-pub@isdigit+0x00000000000615e4) mapaddr=000000000007a8a0. ref=0000003802b815c0
Memory fault (core dumped)
and then, roudi will print a warning log:
2022-07-04 10:56:18.792 [Warning]: Application iox-test-1to1-pub not responding (last response 1543 milliseconds ago) --> removing it
Expected result or behaviour:
communicate like using a small cap container
Conditions where it occurred / Performed steps:
I using header file "msg_img_data.hpp" which defined a struck below:
struct imgdata{
    iox::cxx::vector<uint8_t, 350000> data;
};

which caused a Memory fault (core dumped), but when i reduce the size of vector like 340000 or smaller, core dumped won't happen.
and when i change the cap to a biger cap like 34000000 and pub can't even get into the main() function
I also tried using dumper trace the source of the fault, it lead to file:
iceoryx_hoofs/source/posix_wrapper/unix_domain_socket.cpp :line 284
which cased fault: Program terminated with signal SIGSEGV, Segmentation fault.
here is the gdb info:
#0  0x000000507ad5b440 in iox::posix::UnixDomainSocket::timedReceive (
    this=this@entry=0x507ad98bf8 <iox::runtime::PoshRuntime::defaultRuntimeFactory(iox::cxx::optional<iox::cxx::string<100ul> const*>)::instance+408>, timeout=...)
    at /home/tusen/iceoryx/iceoryx/iceoryx_hoofs/source/posix_wrapper/unix_domain_socket.cpp:277
#1  0x000000507ad2fff0 in iox::runtime::IpcInterfaceBase::timedReceive (
    this=this@entry=0x507ad98b68 <iox::runtime::PoshRuntime::defaultRuntimeFactory(iox::cxx::optional<iox::cxx::string<100ul> const*>)::instance+264>, timeout=...,
    answer=...) at /home/tusen/iceoryx/iceoryx/iceoryx_posh/source/runtime/ipc_interface_base.cpp:91
#2  0x000000507ad32744 in iox::runtime::IpcRuntimeInterface::waitForRegAck (
    this=this@entry=0x507ad98ae8 <iox::runtime::PoshRuntime::defaultRuntimeFactory(iox::cxx::optional<iox::cxx::string<100ul> const*>)::instance+136>,
    transmissionTimestamp=transmissionTimestamp@entry=3712276960700) at /home/tusen/iceoryx/iceoryx/iceoryx_posh/source/runtime/ipc_runtime_interface.cpp:223
#3  0x000000507ad348c0 in iox::runtime::IpcRuntimeInterface::IpcRuntimeInterface (
    this=0x507ad98ae8 <iox::runtime::PoshRuntime::defaultRuntimeFactory(iox::cxx::optional<iox::cxx::string<100ul> const*>)::instance+136>, roudiName=...,
    runtimeName=..., roudiWaitingTimeout=...) at /home/tusen/iceoryx/iceoryx/iceoryx_posh/source/runtime/ipc_runtime_interface.cpp:109
#4  0x000000507ad0cd5c in iox::runtime::PoshRuntimeImpl::PoshRuntimeImpl (
    this=0x507ad98a60 <iox::runtime::PoshRuntime::defaultRuntimeFactory(iox::cxx::optional<iox::cxx::string<100ul> const*>)::instance>, name=..., location=<optimized out>)
    at /home/tusen/iceoryx/iceoryx/iceoryx_hoofs/include/iceoryx_hoofs/internal/cxx/optional.inl:239
#5  0x000000507ad0bc6c in iox::runtime::PoshRuntime::defaultRuntimeFactory (name=...) at /home/tusen/iceoryx/iceoryx/iceoryx_posh/source/runtime/posh_runtime.cpp:48
#6  iox::runtime::PoshRuntime::defaultRuntimeFactory (name=...) at /home/tusen/iceoryx/iceoryx/iceoryx_posh/source/runtime/posh_runtime.cpp:48
#7  0x000000507ad0bd08 in iox::runtime::PoshRuntime::getInstance (name=...) at /home/tusen/iceoryx/iceoryx/iceoryx_posh/source/runtime/posh_runtime.cpp:67
#8  0x000000507ad0be5c in iox::runtime::PoshRuntime::initRuntime (name=...)
    at /home/tusen/iceoryx/iceoryx/iceoryx_hoofs/include/iceoryx_hoofs/internal/cxx/optional.inl:239
#9  0x000000507acebedc in sending<WaypointVec150> () at /home/tusen/iceoryx/iceoryx/icetest/one2one/src/iox_test_pub.cpp:37
#10 0x000000507ace952c in main (argc=1, argv=0x42acc1b9a8) at /home/tusen/iceoryx/iceoryx/icetest/one2one/src/iox_test_pub.cpp:131

if you need any further info,please let me know.
thanks

@JR-Rong I suspect that the stack size of QNX is maybe insufficient (I think it is around 500kb and imgdata requires already 350kb).
Could you try to set the stack size to unlimited in qnx with ulimit -s unlimited and check if this solves your issue?
But in general I would suggest not storing such large data on the stack, especially in QNX. Do you have a copy or did you store the last received example somewhere of imgdata.
You have two options now:

Either only keep the imgdata in the shared memory, this means you loan it with the publisher and then populate it and send it out. If you would like to keep the older data in the subscriber, keep the sample.
If you would like to perform a copy create a imgdata on the heap like: std::unique_ptr<imgdata> myData = std::make_unique<imgdata>(); and copy your contents in there.

thanks for your answer
1.I tried to use ulimit -s unlimited but nothing change
2. I decided to use option 2, here is my code:
std::unique_ptr<ImageData> obj = std::make_unique<ImageData>(GetNs());
        myPublisher.publishCopyOf(*obj)
            .or_else([&](auto) { std::cout << "Failed sending: " << std::endl; });
        std::cout << "Sending Image with timer "
        << obj->header.stamp.nsec
        << " and data size is:" << sizeof(ImageData) << std::endl;

ImageData contains the BIG vector, and in ImageData, i added a timestamp.  When I try to run, roudi gived the log:
[ Fatal ]: The following mempools are available:  MemPool [ ChunkSize = 168, ChunkPayloadSize = 128, ChunkCount = 10000 ]  MemPool [ ChunkSize = 1064, ChunkPayloadSize = 1024, ChunkCount = 5000 ]  MemPool [ ChunkSize = 16424, ChunkPayloadSize = 16384, ChunkCount = 1000 ]  MemPool [ ChunkSize = 131112, ChunkPayloadSize = 131072, ChunkCount = 1000 ]  MemPool [ ChunkSize = 1048616, ChunkPayloadSize = 1048576, ChunkCount = 100 ]Could not find a fitting mempool for a chunk of size 35000144
So,I edit roudi_config.toml and use -c roudi_config.toml to roudi, and then I got the Memory fault (core dumped) again,where am i doing wrong？
Again,Thank you for your answer!

@JR-Rong
I think GetNs() is already using the stack and returns a copy of your image data which is then stored on the heap. When I take a look at your code it seems that your struct requires 35MB in size which is pretty huge.
So I would recommend the following adjustments:

Do not use publishCopyOf, this performs another copy which will cost a lot of runtime. Use loan instead. To do that add a parameter to GetNs which is of type ImageData& so the signature should look like void GetNs(ImageData&);
After loan is successful use GetNs to populate the data, so your algorithm looks like:

publisher.loan().and_then([](auto & sample) {
           GetNs(*sample); // here the image data is written
           sample.publish(); });

If the mempool is insufficient you can reconfigure our shared memory pool like it is described here: https://github.com/eclipse-iceoryx/iceoryx/blob/master/doc/website/advanced/configuration-guide.md#configuring-mempools-for-roudi

But if you only have 2GB at your disposal you should be cautious to not be too generous. Here I would recommend a sample size with at least 35000144 bytes (better maybe 36MB) with a count of 4.

Thank you very much! Using the method you provided helped me solve the problem very well！

@JR-Rong always happy to help :smile:


Update play, play-ahc-ws, play-netty-server, ... to 2.8.10

Updates

com.typesafe.play:play
com.typesafe.play:play-ahc-ws
com.typesafe.play:play-netty-server
com.typesafe.play:play-test

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

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.typesafe.play" } ]

labels: library-update, early-semver-patch, semver-spec-patch

Superseded by #968.


arg_relations_distance QSR

@PDuckworth is this what you need? Please give me the thumbs up and then I will merge.

I'm not sure.
@Yianni, Does this this commit contain only the distance qsr code? Or does it also contain the ability to return specific object pairs? i.e. Trajectory - object pairs and not all pairs?

arg_relations_distance is only for distance QSR allowing to pass different number of labels/thresholds. The ability to pass specific pairs for QSR generation is in the core of the QSRlib and can be used from arg_relations_distance and RCC3 but not from QTC(b,c) AFAIK.

@hawesie I would like to use these qsrs. They are more descriptive than the QTCb I'm currently using, and the labels/thresholds can be learnt from the data.

Yes it is already merged in upstream (see #33 ). arg_relations_distance supports this.

@PDuckworth ok! if you say this code is ok I will merge it.

Yep, I will use this code if it's merged. I haven't used it or checked it works yet, but it is probably very similar code to rcc3 (so I expect it to work fine).

I won't merge it until you've checked it works! You can pull the code for this PR into your local workspace. Click on the command line instructions in the merge box to see how if you don't know.

In your local copy of strands_qsr_lib:
git checkout -b yianni-master master
git pull https://github.com/yianni/strands_qsr_lib.git master

Yep, I've already done this. I just haven't got around to testing it. I will do this today.
I will also try and make the update so I only call QSRLib once per trajectory.

It most certainly works, although being the developer and since we are following two person testing yes pull from my fork as nick described above. See ros client for usage example (although for specific pairs code I think is commented out) just look for the 'qsrs_for' parameter.

Thanks @yianni
Also, what is the which_qsr_argv I need to call for the distance qsrs? I don't think this is in the example client.

@yianni I have two issues which maybe you could answer/resolve:

When I run the ROS example using arg_distance:

qsrlib_request_message = QSRlib_Request_Message(which_qsr=which_qsr, input_data=world, \
                    include_missing_data=True,\
                    qsr_relations_and_values=qsr_relations_and_values)

I obtain the "correct" output:
Response is:
0: o2,o1:0; o1,o2:0;
1: o2,o1:1; o1,o2:1;
2: o2,o1:2; o1,o2:2;

But I don't require o1,o2 and o2, o1. These will always be the same distance relation. Maybe remove one?

This may (or may not) effect the first point, but when I run your example with a qsrs_for list, as per the commented out code, I don't get any output, but also, no error.

qsrlib_request_message = QSRlib_Request_Message(which_qsr=which_qsr, input_data=world, \
                    qsrs_for=[("o1", "o3"), ("o2", "o3")], include_missing_data=True,\
                    qsr_relations_and_values=qsr_relations_and_values)

Response is:
0:
1:
2:

Yiannis
On 10 Apr 2015, at 12:41, Paul <notifications<EMAIL_ADDRESS>wrote:
@yiannihttps://github.com/yianni I have two issues which maybe you could answer/resolve:

When I run the ROS example using arg_distance:

qsrlib_request_message = QSRlib_Request_Message(which_qsr=which_qsr, input_data=world,
include_missing_data=True,
qsr_relations_and_values=qsr_relations_and_values)
I obtain the "correct" output:
Response is:
0: o2,o1:0; o1,o2:0;
1: o2,o1:1; o1,o2:1;
2: o2,o1:2; o1,o2:2;
But I don't require o1,o2 and o2, o1. These will always be the same distance relation. Maybe remove one?
That's default behaviour. Yes could have default without duplicates in this case.

This may (or may not) effect the first point, but when I run your example with a qsrs_for list, as per the commented out code, I don't get any output, but also, no error.

qsrlib_request_message = QSRlib_Request_Message(which_qsr=which_qsr, input_data=world,
qsrs_for=[("o1", "o3"), ("o2", "o3")], include_missing_data=True,
qsr_relations_and_values=qsr_relations_and_values)
Response is:
0:
1:
2:
Difficult to see why without having the full code of what you are running. For the commented out line try a tuple of o1 and o2.
—
Reply to this email directly or view it on GitHubhttps://github.com/strands-project/strands_qsr_lib/pull/36#issuecomment-91495099.

I am running the commented out line in your ROS client example [here] (https://github.com/strands-project/strands_qsr_lib/blob/master/qsr_lib/scripts/example_ros_client.py#L214)
But I added qsr_relations_and_values=qsr_relations_and_values for the arg_distance.

Yiannis
On 10 Apr 2015, at 17:51, Paul <notifications<EMAIL_ADDRESS>wrote:
I am running the commented out line in your ROS client example herehttps://github.com/strands-project/strands_qsr_lib/blob/master/qsr_lib/scripts/example_ros_client.py#L214
That's from upstream and does not contain arg_distance. Run it with rcc or clone my fork and add an o3 in this Elif https://github.com/yianni/strands_qsr_lib/blob/master/qsr_lib/scripts/example_ros_client.py#L77
But I added qsr_relations_and_values=qsr_relations_and_values for the arg_distance.
See above.
—
Reply to this email directly or view it on GitHubhttps://github.com/strands-project/strands_qsr_lib/pull/36#issuecomment-91581907.

Ok, sorry for the confusion. I did do what you say, I just linked to the wrong place in my message.
It's [here] (https://github.com/yianni/strands_qsr_lib/blob/master/qsr_lib/scripts/example_ros_client.py#L232) that won't return qsrs. I added "o3" also.
If I remove the qsrs_for, it works.

Problems should be fixed now. When @PDuckworth is happy with his tests then drop a message for @hawesie to merge.


Minor fix to broken merge.

At some point when HEAD was rebased or merged with crouton/master, one
conflict went unresolved.  I resolved the conflict and was able to use
crouton to successfully generate an arch chroot.

@mathuin Nice work! If you should write down your steps for others to follow (i.e., a Guide), I'd be super interested. My skill level isn't up for the task. Thanks!

@mathuin : Thanks, sorry for that ,-)
I'm happy to merge it but I'd need you to sign the Individual Contributor License Agreement first, since this will get merged into the main crouton at some point. Thanks!

@drinkcat I have indeed signed the ICLA, how do I prove that to you?
@Watney all I did was use this branch as I would use the master branch of crouton but with "-r arch -t xiwi,cli-extra" as the arguments.  Nothing special. :-)

@mathuin hmm, okay, I'll look closer. Thanks!

Merged. Thanks!

I just learned about the test suite yesterday and ran this branch against the suite.  It failed on the xorg target for arch.  I am investigating this issue, and will create another branch to fix it.


Config for 16" notch?

Been trying to tweak this for 16" mbp, but I can't seem to make the notifications big enough. Anyone have a working config?

userconfig.txt
@crissNb Hey mate !
 Notifications are still broken (seems you know it) and spotify only display a blank island.
Apps names are displaying well tho.
You'll find the config file attached to this, in a txt format (sh wasn't uploading)
Feel free if you wanna know anything, happy to help.
Cheers
Yuki

Thank you. I've added your configuration file (after adjusting one or two defaults to stay consistent) in userconfigs/ directory. As of right now, users need to manually copy the right config file for themselves (see readme installation section for more details).
Maybe in the future I'll make changing between config presets easier. Closing this issue for now.


MSS510X stopped working

Describe The Bug:
All of a sudden my MSS510X light switch doesn't work with Homebridge...
Logs:

[13/06/2021, 11:31:17] [Meross] MSS510X - Failed to refresh status of Bedroom Light: "connect EHOSTUNREACH <IP_ADDRESS>:80" undefined

Homebridge Config:
"platforms": [
        {
            "name": "Meross",
            "devices": [
                {
                    "model": "MSS510X",
                    "name": "Bedroom Light",
                    "deviceUrl": "<IP_ADDRESS>",
                    "channel": 0,
                    "messageId": ****,
                    "timestamp": ****,
                    "sign": ****,
                    "accessory": "Meross"
                }
            ],
            "platform": "Meross"
        },
]

Screenshots:

Environment:

Node.js Version:  v14.15.1
NPM Version:  6.14.8
Homebridge Version:

Initializing HAP-NodeJS v0.9.4...
1.3.4

Homebridge Meross Plugin Version: homebridge-meross v5.0.2
Homebridge Config UI X Plugin Version: homebridge-config-ui-x v4.41.0
Operating System:  Raspbian

For some reason now, it works! Just took quite a few restarts for it to work


Scrape Behind Login

Hello.
So I'm using Scrape it CLI for my project because, it's not working for me from browser and cors limitations.
So my question, can I use Scrap it cli behind a login of Web site ?
And how, please.
Thank you, for your help , I will mention this great tool "scrape it" and You in my md soon.

You should be able to pass the auth cookies. But I don't think that's possible with scrape-it-cli currently.


[PINOT-7088] Emit metrics to track when servers are not found segments during query processing

This change is related to https://github.com/linkedin/pinot/pull/3217.
This change updates broker and server code to indicate the following errors:

If a server gets a request for a segment its not hosting. This can happen when broker hasn't processed EV update indicating that the segment has moved off the server.
On the broker side, when a EV update results in routing table entries that have no servers for some segments. This could happen during table rebalance when Helix removes a segment from all servers temporarily.

Testing done:
Unit tests exercise the new code; I see log messages for both server and broker side issues.

Codecov Report

Merging #3220 into master will decrease coverage by <.01%.
The diff coverage is 76.92%.

@@             Coverage Diff             @@
##             master   #3220      +/-   ##
===========================================
- Coverage     70.21%   70.2%   -0.01%
  Complexity        4       4
===========================================
  Files           957     957
  Lines         44327   44353      +26
  Branches       5976    5982       +6
===========================================
+ Hits          31125   31139      +14
- Misses        11137   11148      +11
- Partials       2065    2066       +1

Impacted Files
Coverage Δ
Complexity Δ

...not/broker/routing/RoutingTableBuilderFactory.java
44.82% <ø> (-0.94%)
0 <0> (ø)

...afkaHighLevelConsumerBasedRoutingTableBuilder.java
96.15% <ø> (+3.84%)
0 <0> (ø)
:arrow_down:

...com/linkedin/pinot/common/metrics/BrokerMeter.java
100% <100%> (ø)
0 <0> (ø)
:arrow_down:

.../broker/routing/HelixExternalViewBasedRouting.java
90.33% <100%> (-1%)
0 <0> (ø)

...com/linkedin/pinot/common/metrics/ServerMeter.java
100% <100%> (ø)
0 <0> (ø)
:arrow_down:

...der/PartitionAwareRealtimeRoutingTableBuilder.java
85.18% <100%> (ø)
0 <0> (ø)
:arrow_down:

...lder/PartitionAwareOfflineRoutingTableBuilder.java
69.33% <100%> (ø)
0 <0> (ø)
:arrow_down:

...ing/builder/BalancedRandomRoutingTableBuilder.java
100% <100%> (ø)
0 <0> (ø)
:arrow_down:

...lder/KafkaLowLevelConsumerRoutingTableBuilder.java
80% <100%> (+1.15%)
0 <0> (ø)
:arrow_down:

...ing/builder/DefaultOfflineRoutingTableBuilder.java
47.05% <100%> (ø)
0 <0> (ø)
:arrow_down:

... and 26 more

Continue to review full report at Codecov.

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


Feature/page order API refactor

Description
Refactor the page order API implementation as suggested here https://github.com/appsmithorg/appsmith/pull/5477#issuecomment-871146676
Fixes #5784
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Unit Tests
Postman locally

Checklist:

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

@trishaanand @sharat87 Ready for review

@trishaanand Ready for review 😀

@trishaanand Ready for review 😀


Add aperture class for computing aperture area

This PR adds a new abstract base class AbstractAperture as well as two additional concrete classes CircularAperture and SlotAperture. As a result, the Channel class now takes in an instance of this aperture class and it uses to compute the geometrical area such that the aperture can be any arbitrary shape. The OpticalDesign no longer takes in the pinhole diameter.
Additionally, this PR adds reprs for optical and instrument designs and also adds a method for combining instrument designs.

Codecov Report
Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (3f898e8) 63.40% compared to head (636420e) 64.64%.

Files
Patch %
Lines

mocksipipeline/instrument/design.py
66.66%
4 Missing :warning:

mocksipipeline/instrument/optics/aperture.py
86.20%
4 Missing :warning:

mocksipipeline/instrument/optics/design.py
66.66%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #27      +/-   ##
==========================================
+ Coverage   63.40%   64.64%   +1.24%
==========================================
  Files          13       14       +1
  Lines         582      628      +46
==========================================
+ Hits          369      406      +37
- Misses        213      222       +9

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


Provide option to version info in generated .sys file

Description
Provide option to Convert-BpfToNative.ps1 to pass resource file for use when generating .sys file.
Testing
CI/CD
Documentation
Added documentation to the script.

Should this PR also include the calls to actually add the version info in the generated sys files (i.e. for the next release), or is it planned in another PR?

Should this PR also include the calls to actually add the version info in the generated sys files (i.e. for the next release), or is it planned in another PR?

Which version number should be included? There is no version number in the input ELF file, so this PR permits the end user to add their own version number / copyright etc into the resulting .sys file.

I'd say we should start using the standard version number capability in the input ELF file.
https://github.com/dthaler/ebpf-docs-1/blob/elf/rst/elf.rst#runtime-version-restriction

Should this PR also include the calls to actually add the version info in the generated sys files (i.e. for the next release), or is it planned in another PR?

Which version number should be included? There is no version number in the input ELF file, so this PR permits the end user to add their own version number / copyright etc into the resulting .sys file.

As discussed offline, agree to leave the decision to the customers, to provide their own resource file, accordingly to their versioning strategy.


Two clock domains

Add a second independent clock domain for RsBm, RsChien and RsForney to have an option to use a clock signal slower than the clock signal of the input AXIS interface.

Implemented.


Instance emotes not always displaying correctly in profile name

Describe the bug
A clear and concise description of what the bug is.

Which site: phanpy.social
Which instance: My account is on shelter.moe, bug seen on account from misskey.io

To Reproduce
Go to any account with custom emojis in their profile name
Expected behavior
All emojis should be displayed the same way as they are on their instance
Screenshots
Profile as seen on Phanpy

Same profile as seen on Misskey

Desktop (please complete the following information):

OS: W11
Browser: Firefox
Version: 119.0

Seems like Mastodon can't "find" the image for that custom emoji 🤔 (the API response doesn't include it). If you view it on your instance, it also doesn't show that emoji https://shelter.moe/<EMAIL_ADDRESS>

Nice catch, I didn't try to check this user's profile from my instance. Not sure there's anything you can do in that case...

@Gyoo yeah, there's not much can be done if the API doesn't return the emoji 😬


Cannot find a way to send signal to a specific ouput

I have a Lexicon Omega Audio interface. I have written a small program from examples to send a sinewave to the output through MME. This works. I have had a go at selecting an ASIO output and using the channel_selector argument, but no luck. I get invalid number of channels. Is there a way I am missing to be able to pass an argument that says I want outdata to go to a specific output of the device I am using?
import sounddevice as sd
import numpy as np
import scipy.stats as ss
import queue

asio_out = sd.AsioSettings(channel_selectors=[1, 2])

out_q = queue.Queue()

global start_idx
start_idx = 0

def callback(outdata, frames, time, status):

    global start_idx
    t = (start_idx + np.arange(frames)) / 48000
    t = t.reshape(-1, 1)
    sine = 0.9 * np.sin(2 * np.pi * 1000 * t)

    outdata[:] = sine
    out_q.put(sine)

    start_idx += frames

with sd.OutputStream(device=24, samplerate=48000, channels=4, blocksize=4096, callback=callback, extra_settings=asio_out):
    while True:
        pass

Thanks in advance.

I guess this issue is resolved, right?


Request for JSON schema

It would be really helpful to have JSON schema (http://json-schema.org/) for the various JSON structures that are used by the APIs and image formats - that way one could syntactically verify one's JSON without having to run it through the API and diagnose the failures that result.

I think this is a duplicate of https://github.com/docker/engine-api/pull/78, but was closed in favor of using Swagger; https://github.com/docker/engine-api/issues/77

Indeed, it appears to discuss the same issue. Shall I close this? I am a bystander so I am not sure of the protocol...

Let me close it here, because this change has to me made in the other repository, so probably no need to keep it open here. Thank!


Upload unit test logs

Describe your changes

Upload unit test logs

Checklist before requesting a review

[x] I have performed a self-review of my code
[ ] I have made corresponding changes to the documentation
[x] I read the contribution documentation and made sure that my changes meet its requirements

Could you please revwie/approve @Pietfried or @corneliusclaussen ?


preceded_by have stasis and process aggregate as domain and range

Currently preceded_by only allows usage with processes. I am requesting that we change the domain and range to include stasis and process aggregate in order to have a more generalized preceded_by to work with more use cases.

preceded_by and precedes are not CCO terms. They are from the Relation Ontology. Therefore we can't modify their definitions.
However, in the old version of RO that CCO is currently importing, there are no domain and range restrictions asserted for either property. Despite use of 'process' in the textual definition, you can use them to link instances of process aggregate.
Important to note: In the new version of RO, the textual definitions have changed, and the domain and ranges are now explicitly asserted as occurrent. We are in the process of updating CCO to use a subset of the newest version of RO, which will be available in a forthcoming release.


add 5.10 ci setup

motivaiton: test for upcoming 5.10 release
changes: add docker setup for stable 5.9 and nightly 5.10

@swift-server-bot test this please


(WIP) add extendable erc721r contract

Created a simple extendable version of ERC721R
WIP: need to handle owner mints differently

For owner mints I would work on what we already have merged in which fixes issues around that.

Note, I'm not sure the actual issue is owner mint. Because even if the owner can't mint, they can still buy 1 off the market and refund indefinitely. So I'm not sure we need it at all. We just need the fixes that were put in here:
https://github.com/exo-digital-labs/ERC721R/pull/9
The core being:
    mapping(uint256 => bool) public hasRefunded; // users can search if the NFT has been refunded
    mapping(uint256 => bool) public isOwnerMint; // if the NFT was freely minted by owner

Could the notContract modifier be removed? I don't see why it should be included in the base contract, an inheriting contract could simply implement it themselves if they need it. (There is a reason you don't see it in the ERC721A contract, for example). Also, it's Bad Practice® (#5).

Note, I'm not sure the actual issue is owner mint. Because even if the owner can't mint, they can still buy 1 off the market and refund indefinitely. So I'm not sure we need it at all. We just need the fixes that were put in here: #9
The core being:
    mapping(uint256 => bool) public hasRefunded; // users can search if the NFT has been refunded
    mapping(uint256 => bool) public isOwnerMint; // if the NFT was freely minted by owner

I have a nice solution for all that I plan to implement to push tomorrow to fix this issue

Could the notContract modifier be removed? I don't see why it should be included in the base contract, an inheriting contract could simply implement it themselves if they need it. (There is a reason you don't see it in the ERC721A contract, for example). Also, it's Bad Practice® (#5).

yeah I see what you're saying. It may be removed soon

Note, I'm not sure the actual issue is owner mint. Because even if the owner can't mint, they can still buy 1 off the market and refund indefinitely. So I'm not sure we need it at all. We just need the fixes that were put in here: #9
The core being:
    mapping(uint256 => bool) public hasRefunded; // users can search if the NFT has been refunded
    mapping(uint256 => bool) public isOwnerMint; // if the NFT was freely minted by owner

I have a nice solution for all that I plan to implement to push tomorrow to fix this issue

a solution was already merged in for this btw. there was discussion around some different options too. may be worth adding comments to that discussion on what you're planning as a few options were already put out

mapping(uint256 => bool) public hasRefunded; // users can search if the NFT has been refunded

Yes the mapping are included to the implementation.
I was thinking about adding another mapping for token id -> purchase price.
This will require modifying _safeMint to add purchase price after minting.
We then require that purchase price for this token id > 0 when requesting a refund for  it.

So I pushed a version like that a few days ago:
https://github.com/exo-digital-labs/ERC721R/pull/9#issuecomment-1097823682
The reason we didn't use it is discussed there. It increases mint price. The other versions increase ownermint price and refund which is probably better


How to modify the background color of this position?

How to modify the background color of this position?

Please ask questions on stack overflow, thanks.
fyi @aeschli

https://code.visualstudio.com/docs/getstarted/theme-color-reference
In fact, you can't find configuration parameters on this url(Theme Color Reference)。I suggest adding the configuration parameters of the background color for this position.


useSuspenseQuery reverts back to variables set in PreloadQuery when search params change

We have a query setup that has some default variables on the server that can then change on the client. As an example:
const QUERY = gql`
    query GetChart ($id: UUID!, $compact: Boolean) {
        # ...
    }
`;

// Server
// =========================================================================

return (
    <PreloadQuery query={QUERY} variables={{ id, compact: true }}>
        {/* ... */}
    </PreloadQuery>
);

// Client
// =========================================================================

const [compact, setCompact] = useState(true);
const { data } = useSuspenseQuery(QUERY, {
    variables: { id, compact },
    fetchPolicy: 'cache-and-network',
});

The user can change the value of compact on the client.
If the search parameters then change (in our case clicking an "Edit" button might open a modal by pushing to the same route with ?edit=true) the compact state will persist with whatever value the user has set, but the value of data will revert back to whatever the initial value was loaded from the server.
In the example above this means that if the user changed compact from true to false and then clicked an "Edit" button, the graphql data would revert back to seeing compact as true while the rest of the UI would still be seeing it as false.

Package
Version

@apollo/client
3.11.8

@apollo/experimental-nextjs-app-support
0.11.5

next
15.0.1

react
18

That really seems curious. I know it's a big ask, but could you maybe please create a small reproduction for this issue?

Sorry, haven't been able to create a reproduction yet, been up against it!
Adding to this, we're also having issues where we update something via a mutation client side, use refetchQueries on the mutation, call refetch on the useSuspenseQuery, set fetchPolicy: 'no-cache', on the useSuspenseQuery, have export const dynamic = 'force-dynamic'; on the page, and cache: 'no-cache' on the fetch function, but the change is lost when refreshing. The only way to see it is, at least locally, to cause a rebuild.


Improve performance for get request

Get data as soon as they are ready

Reopening this since we reverted async get changes in #390.


adapt the GitLab CI to GitHub

The CI was originally made for Gitlab. As we moved the repo to GitHub we then need to adapt the files to be GitHub compatible.

See #25


Fix two bugs

How do I reproduce those bugs?

One of them is just that if the settings"useLocalBranch" is true, it doesn't use local branch and opens a fixed remote branch.
The other one I didn't know existed when I came to fix the previous one but I just saw the code. If the remote url doesn't exist, it was supposed stop with a msg by checking if repeurl undefined but you were checking repopath again

You should see the commit. It's just two lines

Thanks, available in v1.2.1.


i'm trying to install guzzle version 7 and i'm getting this error what should i do

Root composer.json requires guzzlehttp/guzzle 7.0, found guzzlehttp/guzzle[7.0.0] but these were not loaded, likely
because it conflicts with another require.

Looks like you asked for 7.0 instead of ^7.0 by mistake.

Your error messages says different.

that's what i'm writing


Fix PhotoShop Lightroom installation.

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Fixes https://github.com/caskroom/homebrew-cask/issues/27804.

After reading your explanation, I was expecting the fix to be just setting must_succeed: false (imagining that’d be an extra optional argument) or something.

You're right, this would have been the quick fix, but this would still have printed the error message to stderr, even if SafariNotificationAgent wasn't running to begin with.


Updated messaging to display correctly in prod

Existing code already handles binding to a non-localhost address via configuration file and environment variables.
For example:
export MOOSE_HTTP_SERVER_CONFIG__HOST=<IP_ADDRESS>
export MOOSE_HTTP_SERVER_CONFIG__PORT=4000

However, server output to console was hardcoded to say localhost rather than display the address passed in from the user.

@callicles this PR is not a block as I will use the environment variables on Kubernetes.


r2modman: 3.1.50 -> 3.1.54

Things done
Updated r2Modman from version 3.1.50 to 3.1.54.
Changelog

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 358592

x86_64-linux

:white_check_mark: 1 package built:

r2modman

Builds & works as intended. Can you rename the commit to message to r2modman: 3.1.50 -> 3.1.54?

Commit message is renamed


Unable to pass a zod schema to a function and have it be typed correctly by typescript

Suppose I have a function that accepts a zod schema and runs the parse function on some data like:
import { z, ZodTypeAny } from 'zod'

type Opts<Z extends ZodTypeAny> = {
  // ...other db opts
  schema: Z
}
export const findOne<T = any, Z extends ZodTypeAny = never> = async (opts: Opts<Z>) => {
  // ...logic to read db

  const item = await dbQuery()

  if (!item) return
  if (opts.schema) return schema.parse(item)
  return item as T
}

const User = z.object({
  id: z.string(),
  username: z.string(),
  createdAt: z.string()
})

// expect user to be of type `typeof User`
const user = await findOne({ /** opts */, schema: User })

I would expect all zod schemas to extend the ZodTypeAny type, but I get the error Property 'received' is missing in type 'ZodInvalidLiteralIssue & { fatal?: boolean | undefined; message: string; }' but required in type 'ZodInvalidLiteralIssue'. when I pass the User schema into the function.

I recommend checking out the "Writing generic functions" guide: https://github.com/colinhacks/zod#writing-generic-functions

import { z } from "./src";

type Opts<Z extends z.ZodTypeAny> = {
  // ...other db opts
  schema?: Z;
};
export const findOne = async <Z extends z.ZodTypeAny = z.ZodNever>(
  opts: Opts<Z>
) => {
  return "whatever" as z.infer<Z>;
};

const User = z.object({
  id: z.string(),
  username: z.string(),
  createdAt: z.string(),
});

// expect user to be of type `typeof User`
const user = await findOne({});


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[core][fix] fix ArrayType convert for spark2.4.0 and improve the convert method name

Purpose of this pull request
fix the ArrayType translate for spark2.4.0
Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[x] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[x] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

this PR should be after This PR should be after console optimization. I have tested some data types in the local rebase PR # 2516, but I have not written unit tests. I will add some unit tests for this PR later


Using base pot while creating single pot

There is one thing that has been bothering me for a while, and I don't know if it's an issue with documentation or if it is really how it is supposed to work.
So the command pot create has a parameter -b where docs say -b base : the base pot. Ok, so let's create a base pot, right.
pot create-base -r 14.0 -b mybase14
then
pot ls -b
shows
bases: mybase14
It all works up to this point; the base pot is created and updated with the latest patches. Then I continue with creating new pot using a base pot
pot create -p mypot -b mybase14 -t single but I get the error
###>  mybase14 is not a valid release
Going through the code I found
https://github.com/bsdpot/pot/blob/9ac5a5fc15843039e1c266cbe9b083f95dd33952/share/pot/create.sh#L614-L625
And function _is_valid_release
https://github.com/bsdpot/pot/blob/9ac5a5fc15843039e1c266cbe9b083f95dd33952/share/pot/common.sh#L919-L926
That checks if my base pot is actually a release itself. Which is not. Then I thought ok let's go and use -P as this is documented as -P pot : the pot to be used as reference
pot create -p mypot -P mybase14 -t single but I get the error
###>  pot mybase14 not found
From the code, it's obvious since the base pot is not a regular pot.
I would like to fix the doc in a way that can guide me through this, but something isn't right here, or maybe I'm missing something. I can see from the code that both -b and -P can be used simultaneously. But I don't know how the base and this pot should be set up for this to work.
The workaround, for now, is just to create pot from release and use an updated flavour in combination with other flavours. And this takes a bit more time to always do extract and update before anything else can be created.

Any comments on that @pizzamig ?


fix: adds app selector to pod template

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If this PR will release a new chart version please make sure to also uncomment the following line:

/kind chart-release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area falco-chart

/area falco-exporter-chart

/area falcosidekick-chart

/area event-generator-chart

What this PR does / why we need it:
The provided falco-grpc service app selector does not exist in the pod template.
Which issue(s) this PR fixes:
Adds a label to the the pod template so the service can route traffic to that pod.

Fixes #
Special notes for your reviewer:
There are template options to add pod selectors. However, since the falco-grpc servcie is provided in the chart and specifies a app label it seems it would be appropriate to have a matching label in the pod template.
Checklist

[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] CHANGELOG.md updated

@angrieralien: The label(s) area/event-generator-chart cannot be applied, because the repository doesn't have them.

In response to this:

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If this PR will release a new chart version please make sure to also uncomment the following line:

/kind chart-release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area falco-chart

/area falco-exporter-chart

/area falcosidekick-chart

/area event-generator-chart
What this PR does / why we need it:
The provided falco-grpc service app selector does not exist in the pod template.
Which issue(s) this PR fixes:
Adds a label to the the pod template so the service can route traffic to that pod.
Fixes #
Special notes for your reviewer:
There are template options to add pod selectors. However, since the falco-grpc servcie is provided in the chart and specifies a app label it seems it would be appropriate to have a matching label in the pod template.
Checklist

[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] CHANGELOG.md updated

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Hi @angrieralien, thanks for the fix. I think we should update the grpc-service to use the correct label selector. A helper is already in place, called falco.selectorLabels. Could you please update the service to use it?

@alacuku  Sure can. I'll start working on that soon.

@alacuku made the change to use falco.selectorLabels.
I tested the generated templates with
helm template falco  --output-dir out  --set falco.grpc.enabled=true --set falco.grpc.bind_address="<IP_ADDRESS>:5060"  --set falco.grpc_output.enabled=true .

It would be best if you dropped the first commit. And then, bump the version of the chart and update the changelog.
After that, we are good to merge this one!
ps. Here you can find a commit that bumps the chart version and updates the changelog. Ignore the falcosidekick dependencies section, since it was the main goal of the PR: https://github.com/falcosecurity/charts/pull/474/commits/fc371241b5112a842c644624b5bf6d609834b053

@alacuku made the changes you requested. Please let me know if there is anything else you need me to do.

@alacuku sorry about that. Should be good now!

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: alacuku, angrieralien
Once this PR has been reviewed and has the lgtm label, please assign issif for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

LGTM label has been added.  Git tree hash: 84c61a0d9a145259e6134e28c1207295786b4422


Adding new feature to encrypt and decrypt documents with unit test

Encrypt documents by byte
Decrypt documents by byte
Adding dummy files docx, xlsx and png as test files
Unit test of files

Squash and close :)


Target profile page rewrite: Create Gene Ontology summary widget

Use Cases

As a user of the Platform, I want to see how many GO terms for each respective category - Molecular Function, Biological Process, Cell Component - are related to my target of interest.

Summary Widget Views

Full-size version
Design and Interaction Notes

For targets where there is gene ontology data for all three categories (e.g. ESR1), please use the following text in Open Targets Grey - #5a5f5f:

X Molecular Function terms
X Biological Process terms
X Cellular Component terms
Please replace X with the relevant number of terms for a given category.
Also, please colour the icon and the widget container box outline in Open Targets Grey - #5a5f5f.

For targets where there is gene ontology data some but not all three categories (e.g. INTS5 or GADD45B), please display the categories with data in Open Targets Grey - #5a5f5f as per point 1 above. For the categories without data, please display the following text in Open Targets Light Grey - #e2dfdf:

0 GO-CATEGORY terms (e.g. 0 Molecular Function terms)
Also, please colour the icon and the widget container box outline in Open Targets Grey - #5a5f5f.

For targets with no gene ontology data (e.g. HOTAIR or FAM160A1), please use the following text in Open Targets Light Grey - #e2dfdf:

0 Molecular Function terms
0 Biological Process terms
0 Cellular Component terms
Also, please colour the icon and the widget container box outline in Open Targets Light Grey - #e2dfdf.

When a user hovers over the summary widget and the target has either a small molecule and/or an antibody tractability assessment, please show the pointer icon and change the box outline to Open Targets Purple - #7b196a. This will provide users with a visual cue that the summary widget is clickable. For more information, please see issue #429.

Design Assets

Gene ontology icon
Open Targets brand palette colour codes

Looks good @mirandaio - thank you! 👍
UX QC review complete


New App: GitHub Unread

Adds a new app to display the User's current number of Unread Notifications on GitHub. Uses the GitHub Rest API to fetch the notifications, which requires a personal access token from the user as the only necessary argument.
Example Render:

I have read the CLA Document and I hereby sign the CLA

@matslina Hello, just wondering if there is anything else I can do to help move this along?

Hey @ElliottAYoung! This is what will show up in our store and is what @matslina is referring to:

I'm going to merge this a while, but you should consider following up with some form of demo content so users can see what the app could look like if they go through the setup.


Gitea crash when indexer is run (database error, data too long for column language)

Gitea version (or commit ref): Gitea version 1.12.5 built with GNU Make 4.1, go1.14.9 : bindata, sqlite, sqlite_unlock_notify
Git version: 2.28.0
Operating system: Ubuntu 18.04.5 LTS
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No
[ ] Not relevant

Log gist:

./gitea.log:2562:2020/10/02 09:12:07 ...dexer/stats/queue.go:24:handle() [E] stats queue indexer.Index(24) failed: Error 1406: Data too long for column 'language' at row 1

Description
I've upgraded Gitea today the usual way by stopping the service, replacing binary and starting the service again. (From 1.12.4 to 1.12.5)
Gitea crashes when indexer runs. I have no idea why this happened. Maybe failed database migration on my end? That's the only relevant log I could find above.
I've disabled repo indexer and things started working again.

This is a duplicate of #12379
You will need to run the following SQL on the database:
ALTER TABLE language_stat MODIFY COLUMN language VARCHAR(50)

This is fixed in 1.13 with a migration using #12396, (with additional fixes in #12427, #12445) but migrations are not backportable.

Maybe we could add a doctor or fix sub command gitea doctor fix 12379 which is backportable.

closed as duplicated.

closed as duplicated.


How low (in memory capacity) can we go?

My company is planning on working with TMS320F2838x MCUs from Texas Instruments.
It's Connectivity Manager (CM) has 96KB of RAM. I want to know has anyone here has gotten the parser to work in a real application with even lower memory constraints?

There are even possibilities to run this on 8-bit AVR. Primary purpose of this library are small microcontrollers.
This library has these memory requirements

store command template table - on sane architectures this can be completely in flash without problems so it does not consume RAM. On old 8-bit architectures, this needs patching the library (AVR, '51, PIC16), but it is not your case with ARM Cortex M4.
store full incomming command - this is limitation for situations, where you need to pass large data into the device (like signal generator). This preallocated buffer needs to be as big as the largest expected command with data. This is not a big limitation for all other purposes. There is no limitation on the size of outgoing message.
last big memory block is error queue - this is the maximum number of errors the device can memorize.

There are no dynamic memory allocations. There are no extensive stack usages but I don't know exact numbers.

Without any command and a buffer of only 10, I increase the .text of only 11632 bytes and .data of 368.
So it's really small !

That will fit. I use the library with the TI Hercules family.
The drivers of TMS320 are somewhat familiar as the Hercules ones - I'm using the UART one for SCPI exchange. Ping if you need an example.

@jancumps
Are any of those Hercules MCUs the ones that only have 32 KB RAM?

I'm unable to get below roughly 90K binary size, even using -Os. I'm targeting a system with 32K of ram, and slow flash.
I configured the library for SYSTEM_BARE_METAL.
Do you have any suggestions?

And that's just the library. All command patterns and their callbacks consume more flash. This library is written in such a way that it should follow the standards as much as possible. You can sacrifice this functionality by removing some commands.
Writing a simple command handler from scratch is also possible if you don't need these command patterns and these standard SCPI commands. It's just a tradeoff between completeness and size. Then you can say that you support the SCPI-Like interface. The more you remove from the standard functionality, the more your users will get angry. But unfortunately this is common practice for all companies, including those you wouldn't expect (including some industry-leading companies in test and measurement).

...users may express their gratefulness via disappointment through anger :)

It should be possible to get well below 90KB. We have a binary for a AVR8 (32K flash 2.5K ram) which is 20KB, and that includes dozens of quite long commands.  The binary also includes a decent amount of application specific code, a USB library, and a DFU bootloader - I'd be surprised if the SCPI parser component is more than 12KB. Unless I'm missing something?

Thanks @j123b567 for the great library! Good to have some perspective on what is achievable from the comments in this thread. I was able to decrease my code size by using the --gc-sections option in my LD flags to garbage collect unreferenced sections, and -ffunction-sections -fdata-sections in my CC flags. It seems like this is the norm in embedded applications.
I will put together a build with all of my commands and callbacks to see what code size I can achieve. Worst case I can live with the performance hit of linking libscpi exclusively in flash (8x higher read latency in my case).


Updated application port to 3000

Seems puma is running application in default port 3000.
https://github.com/codetriage/codetriage/blob/f020070f0c44c79a90ff4f2c7c0e8ccb956a7f70/config/puma.rb#L14

Thanks!


Omnidirectional Character Sprite Preview

About the PR

Adds all four sprite directions to the character setup UI.
Makes the right lobby panel character preview a bit larger

Why
Easier to customize the character if you can see the character..
Also someone in the discord wanted it
Media

why not make them smaller and put them in a 2x2 box to save space

or that, there are multiple different approaches to this. i personally like it vertical

Wouldn't it be better to just rotate the sprite like borgs do?

Wouldn't it be better to just rotate the sprite like borgs do?

what?

Wouldn't it be better to just rotate the sprite like borgs do?

what?

Thank you for your contribution. While it sounds like there may be ways to do this better, it does seem to me that this is an improvement over what exists.

Morty's going to wrap this into https://github.com/space-wizards/space-station-14/pull/19741 and credit you.

Morty's going to wrap this into https://github.com/space-wizards/space-station-14/pull/19741 and credit you.

oh cool


Add German clustering datasets

This pull request adds German clustering datasets (in reference to issue #115). More concretely, it adds four clustering tasks based two different data sources (blurbs and news articles, a s2s and p2p task per data source).
To do

[x] add tasks to mteb/tasks/Clustering
[x] update README.md
[x] update leaderboard code (fetch German results and show in clustering results)

@Muennighoff Please have a look at my proposed changes and let me know if I should change anything.
I will also open a PR to mteb/results with all the model evaluation results I have (I don't have write access to any of the HF repositories of the models I used).

Is it fine for you to merge this PR?

Is it fine for you to merge this PR?

Yes, please :) PR on HF for eval results follow soon!


[Bug Report] Date Range Picker should not show the month that are not active

Element UI version
2.8.2
OS/Browsers version
Chrome / Mac OS
Vue version
2.6.10
Reproduction Link
https://jsfiddle.net/vkg46rfa/
Steps to reproduce
We do not allow the user to select the date range from the future and we use disabledDate to be able to do that. However; el-date-picker still shows the next month by default even though none of the days are active in that month.
What is Expected?
We should show the previous month if all the values are inactive within a month or provide a way for the user to select the months that are shown in the panel by default.
What is actually happening?
The component shows the inactive month which is confusing to the end users.

read the doc. use default-value (optionally with unlink-panels)
https://jsfiddle.net/3jguxhL7/

@wacky6  I have read the docs, don't worry. :) It looks like there's a bug which prevents the picker to show the correct months. In my case, when I select the range from the previous month, default-value doesn't seem to work. See https://jsfiddle.net/ugmqnapj/


Issue303 and minor other things

This pull request adds bout metric 5:
Bout metric 5 is an modified copy of bout metric 4 (current default), and is different in two ways:

It look for breaks larger than but not equal to a minute when distinguishing multiple bouts. In metric 4 breaks could not last 1 minute, but had to be shorter than a minute.
It requires that the fraction of time in a bout that meets the thresholds should be equal than or greater than the bout.criter value. For metric 4 this fraction of time had to be greater than but not equal to the bout criteria.

The first change is valuable when working with 1 minute epoch data, when we want to allow for 1 minute breaks. If working with 5 second epoch data and using bout.metric 4 then practically the maximum break duration is 55 seconds. With bout.metric 5 this now has become a 1 minute, which is probably more intuitive..
The second change makes it possible to calculate bouts with bout.criter = 1.0, which means that no bout breaks are allowed (see #303).
For the time being I will keep bout metric 4 as the default. I am involved in a project where we use metric 5, once this is published it may be good to reflect on whether bout.metric=5 should be default.
Other minor updates:

Recognition of the number of files that need to be processed in part 5, This looked at part 4, but it should look at part 3.
Updated documentation of part5.definedays.

I will now merge this into the master branch:

This is only an extension of existing functionality, not affecting default GGIR output. Only when users specify bout.metric = 5 they will use the new bout.metric.
The other change is a fix to a variable mismatch between documentation and code.


Add API for checking whether a peer is connected through a relay server

I want this because I want to tell my players why their latency is bad.
It looks hairy, but doable. Not sure what the API will be.

I'm just curious, but why not just an API to negotiate ping?

Ideally, we'd have ping available through the same API, but not critical for me, as I already get it through ggrs.


Log usage unclear

Problem/Motivation
high memory usage
Expected behavior
normal memory usage
Actual behavior

2024-02-18T20:32:15.628659+01:00 [warning] msg: alarm_is_deactivated, mfa: emqx_alarm:do_actions/3(424), name: high_system_memory_usage
2024-02-18T21:30:15.690699+01:00 [warning] msg: alarm_is_activated, mfa: emqx_alarm:do_actions/3(418), message: <<"78.39% mem usage">>, name: high_system_memory_usage
2024-02-18T21:31:15.692591+01:00 [warning] msg: alarm_is_deactivated, mfa: emqx_alarm:do_actions/3(424), name: high_system_memory_usage
2024-02-18T23:31:15.819597+01:00 [warning] msg: alarm_is_activated, mfa: emqx_alarm:do_actions/3(418), message: <<"77.36% mem usage">>, name: high_system_memory_usage
2024-02-18T23:32:15.820445+01:00 [warning] msg: alarm_is_deactivated, mfa: emqx_alarm:do_actions/3(424), name: high_system_memory_usage
2024-02-19T03:31:16.070281+01:00 [warning] msg: alarm_is_activated, mfa: emqx_alarm:do_actions/3(418), message: <<"76.40% mem usage">>, name: high_system_memory_usage
2024-02-19T03:32:16.072468+01:00 [warning] msg: alarm_is_deactivated, mfa: emqx_alarm:do_actions/3(424), name: high_system_memory_usage
2024-02-19T07:50:16.341557+01:00 [warning] msg: alarm_is_activated, mfa: emqx_alarm:do_actions/3(418), message: <<"80.33% mem usage">>, name: high_system_memory_usage
2024-02-19T07:51:16.343575+01:00 [warning] msg: alarm_is_deactivated, mfa: emqx_alarm:do_actions/3(424), name: high_system_memory_usage
2024-02-19T13:31:16.710660+01:00 [warning] msg: alarm_is_activated, mfa: emqx_alarm:do_actions/3(418), message: <<"80.00% mem usage">>, name: high_system_memory_usage
2024-02-19T13:32:16.715605+01:00 [warning] msg: alarm_is_deactivated, mfa: emqx_alarm:do_actions/3(424), name: high_system_memory_usage

Steps to reproduce
just run
Proposed changes
no
Is there any change to give it more memory or find out what is needed?

Hi, still existing
2024-03-20T14:30:54.809230+01:00 [warning] msg: alarm_is_activated, mfa: emqx_alarm:do_actions/3(418), message: <<"76.25% mem usage">>, name: high_system_memory_usage 2024-03-20T14:31:54.819135+01:00 [warning] msg: alarm_is_deactivated, mfa: emqx_alarm:do_actions/3(424), name: high_system_memory_usage 2024-03-20T18:00:55.152595+01:00 [warning] msg: alarm_is_activated, mfa: emqx_alarm:do_actions/3(418), message: <<"78.63% mem usage">>, name: high_system_memory_usage 2024-03-20T18:01:55.154799+01:00 [warning] msg: alarm_is_deactivated, mfa: emqx_alarm:do_actions/3(424), name: high_system_memory_usage 2024-03-20T21:30:55.465854+01:00 [warning] msg: alarm_is_activated, mfa: emqx_alarm:do_actions/3(418), message: <<"70.76% mem usage">>, name: high_system_memory_usage 2024-03-20T21:31:55.467792+01:00 [warning] msg: alarm_is_deactivated, mfa: emqx_alarm:do_actions/3(424), name: high_system_memory_usage 2024-03-21T08:30:56.425733+01:00 [warning] msg: alarm_is_activated, mfa: emqx_alarm:do_actions/3(418), message: <<"76.37% mem usage">>, name: high_system_memory_usage 2024-03-21T08:31:56.427531+01:00 [warning] msg: alarm_is_deactivated, mfa: emqx_alarm:do_actions/3(424), name: high_system_memory_usage

Still need assistance

still need help

Hey, the memory in EMQX should be the memory of your Homeassistant server.  Maybe your got some heavy stuff running on your Homeassistant server?
You check the memory on the webinterface -> Cluster Overview -> OS Memory or   webinterface -> Cluster Overview -> Nodes.  There should be the used and max ram which should be the same as your server. Where you should check which process uses your memory.

do you mean this:

?
perhaps at some time there are special jobs which ran it high:
024-05-30T05:30:30.041024+02:00 [warning] msg: alarm_is_activated, message: <<"75.98% mem usage">>, name: high_system_memory_usage
2024-05-30T05:31:30.042630+02:00 [warning] msg: alarm_is_deactivated, name: high_system_memory_usage
2024-05-30T06:30:30.148226+02:00 [warning] msg: alarm_is_activated, message: <<"83.18% mem usage">>, name: high_system_memory_usage
2024-05-30T06:31:30.150018+02:00 [warning] msg: alarm_is_deactivated, name: high_system_memory_usage
2024-05-30T14:30:30.906953+02:00 [warning] msg: alarm_is_activated, message: <<"84.11% mem usage">>, name: high_system_memory_usage
2024-05-30T14:31:30.911878+02:00 [warning] msg: alarm_is_deactivated, name: high_system_memory_usage
2024-05-30T15:30:30.988444+02:00 [warning] msg: alarm_is_activated, message: <<"86.98% mem usage">>, name: high_system_memory_usage
2024-05-30T15:31:30.990449+02:00 [warning] msg: alarm_is_deactivated, name: high_system_memory_usage
2024-05-30T21:30:31.555579+02:00 [warning] msg: alarm_is_activated, message: <<"73.03% mem usage">>, name: high_system_memory_usage
2024-05-30T21:31:31.557701+02:00 [warning] msg: alarm_is_deactivated, name: high_system_memory_usage
2024-05-30T22:30:31.654884+02:00 [warning] msg: alarm_is_activated, message: <<"81.27% mem usage">>, name: high_system_memory_usage
2024-05-30T22:31:31.656409+02:00 [warning] msg: alarm_is_deactivated, name: high_system_memory_usage
2024-05-30T23:30:31.737991+02:00 [warning] msg: alarm_is_activated, message: <<"86.54% mem usage">>, name: high_system_memory_usage
2024-05-30T23:31:31.739309+02:00 [warning] msg: alarm_is_deactivated, name: high_system_memory_usage

Yeah thats what i mean. With 2.3/3.8G thats already 60% usage.
I guess that just means your system is running already with a pretty high ram usage.
Maybe consider changing the alarm to only trigger at 90 or 95%?
With htop inside a terminal (on HAOS you need to check how to get ssh access) you can also check whats using all your memory.
But that should be no EMQX issue. EMQX just tells you about that so maybe consider closing the issue.

How can i change the alarm to 90 percent?

any news on that?!

still 6 alerts per day ...

Hey sorry.. I totally forgot about that..
Just try to edit that here:

SysMem to 90 or 95% and ProcMem to 10% and check if it works.

ok that should ignore the warnings. Will i get problems with this "high" workload?


app.server.port is not same app.listen(port)

What version of Elysia is running?
1.1.24
What platform is your computer?
Microsoft Windows NT 10.0.22631.0 x64
What steps can reproduce the bug?
set port with app.listen method.
and then, the print app.server.port variable
What is the expected behavior?
The app.server.port value should match the port number specified in the app.listen method
const app = new Elysia()
.listen(3002);

What do you see instead?
always same 3000
$ bun run --watch src/index.ts
🦊 Elysia is running at localhost:3002
Started server http://localhost:3000

Additional information
because of this bug, I cannot run another server using port 3000.
I am unable to start another server using port 3000 at the same time as elysiajs. It seems that elysia is turning on faster and blocking port 3000.
If you think about it, if you position the elysiajs startup order behind another server that uses 3000 ports, it will work.
Have you try removing the node_modules and bun.lockb and try again yet?
yes, I did it.

Tried to reproduce:
import Elysia from 'elysia'

const app = new Elysia()
    .listen(3002);

console.log('app.server.port =', app.server.port)

Output:
app.server.port = 3002

If you could post a minimal code example to reproduce the issue, it would be helpful for diagnosing the issue.

Hi, I want to tackle this problem again.
I'm unable to reproduce this issue and the issue is on stale.
If the problems still occurs, feels free to reopen this issue again.

You are right, export default causes this.
I was not aware, but it appears to be the documented behavior of Bun:

when Bun sees a file with a default export containing a fetch handler, it passes it into Bun.serve under the hood

@SaltyAom @crishoj Hello.
Regarding the current issue, is the following correct?
As stated in this document, this is related to Bun's export default behavior  so please comment out or avoid using the following code:
// export default app;

Indeed, you should avoid using export default app in your main file.
May I know what the motivation for exporting was? In case you need API types for an Eden Treaty client, you can use export type ....

You use the export for test, so you can make calls to the api in the test.

Makes sense.
Then, instead of export default app you could export { app }.

@crishoj OK, understood. I will use export {app}' instead of export default app'.


add missing at the end of azurecli

Missing ``` caused two azurecli blocks and a line to be combined together which was wrong.
It's visible here:

on this link:
https://docs.microsoft.com/en-us/azure/aks/use-network-policies#create-an-aks-cluster-for-azure-network-policies

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

@aysabzevar  - thanks!
@zr-msft
Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@aysabzevar @zr-msft  - It looks like this change has been made (possibly through another PR). So this PR should be closed.

Docs Build status updates of commit fd47e05:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: CannotMergeCommit] Cannot merge commit fd47e055be61c07694cb4207ed643c88efd038aa in branch fix-missing-closing-azurecli of repository https://github.com/aysabzevar/azure-docs into branch main (commit 458100365e23a28d44298857a9cd2aaac81cddae). Please follow this documentation: https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line/ to use git.exe to resolve you content conflicts locally and then push to remote.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel