Declarative support for LUIS dynamic lists and external entities

LUIS supports both dynamic lists and external entities which are not accessible through the declarative system.  In order to support this:

Add dynamicList as a property to the LUIS recognizer.  It should take an expression which evaluates to an object like this: { : [ {cannonicalForm: , synonyms: [, ...]}, ..], ...} that is passed to LUIS.
Add the ability to specify a child recognizer.  The child recognizer would be called and the recognizer results would be passed to LUIS as external entities.  This would allow us to use local regex recognizers or more advanced algorithmic matching using things like edit distance.
Create a UnionRecognizer which can union together the results of multiple child recognizers.  For intents, if the names collide pick the maximal score.  For entities, include them all.

3 is done, RecognizerSet is the union of child recognizers.
1 and # 2 will be done by @chrimc62


double-beep: Blacklist life\W*keto

double-beep requests the blacklist of the keyword life\W*keto. See the MS search here and the Stack Exchange search here.

life\W*keto has been seen in 49 true positives, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


Migrating TCPClient Actor

I have encountered a problem migrating TCPClient Actors.
I am running the tcp_client.calvin script (found among the sample scripts) on the develop branch and try to migrate the actor which generates the following log entry:
2016-08-04 12:08:50,700 ERROR    15-calvin.calvin.runtime.north.calvincontrol: Migration failed
Traceback (most recent call last):
File "/calvin-base/calvin/runtime/north/calvincontrol.py", line 1159, in handle_actor_migrate
callback=CalvinCB(self.actor_migrate_cb, handle, connection))
File "/calvin-base/calvin/runtime/north/actormanager.py", line 343, in migrate
actor.will_migrate()
File "/calvin-base/calvin/actorstore/systemactors/net/TCPClient.py", line 54, in will_migrate
self.cc.disconnect()
File "/calvin-base/calvin/calvinsys/network/socketclienthandler.py", line 111, in call
return self.f(self.func, *args, **kwargs)
File "/calvin-base/calvin/calvinsys/network/socketclienthandler.py", line 102, in _call
return func(self._handle, *args, **kwargs)
File "/calvin-base/calvin/calvinsys/network/socketclienthandler.py", line 75, in disconnect
self._connections[handle]['connection'].stop()
AttributeError: 'TCPClientProtocolFactory' object has no attribute 'stop'
Tracing the error I have noticed that there are a few "TODO"-comments in calvin-base/calvin/runtime/south/plugins/async/twistedimpl/client_connection.py which I believe has to do with how the connection should be closed in twisted.  Nonetheless, my question is whether or not this should work and my setup is somehow flawed or if it is a known issue.
Thanks in advance!

This should work it, should be disconnect() instead of stop() in the socketclienthandler.py
I will fix the issue and push it ASAP, and post back here when its fixed.
Thanks for finding it!
//R

Great!
I actually tried that fix myself without success. I also noticed that self._connector in calvin-base/calvin/runtime/south/plugins/async/twistedimpl/client_connection.py is never set in the TCP case as opposed to the UDP case so if I understood the code correctly disconnect() will not do much.
Thanks for the quick reply!

Yes, you are/where right there where more thing missing.
I created a fix on develop branch, you can find more info here:
https://github.com/EricssonResearch/calvin-base/commit/38f05e023f45cb6048bcdfffb05b07d55fd79738
I tested the script you used with success!
//R

Thanks a lot for the help!
Unfortunately I ran into a follow up problem which I haven't managed to solve yet. The migration problem described above works fine in the sense that Calvin does not complain and the Actor can be migrated. However, when I run the sample script and use a basic TCP echoserver it doesn't work as I would like it to. When the actor is migrated the flow of tokens through the server connected to the current runtime stops as expected. However, the flow of tokens is not resumed at the second server connected to the new runtime, nor is it resumed if the actor is migrated back. Furthermore, the production of tokens in the source also stops, which I guess is due to congestion in Calvin caused by the TCP_client being unable to perform its function? In other words, there seems to be something wrong with how the actor disconnects and/or connects to my servers. The problem might very well be on the server side but I think its odd that Calvin doesn't connect to the new server when migrated. Did you test the behavior on/after migration as well?
Thanks in advance!

I did not check if the connection really resumes or not in my simple testcase when solving the other issue.
I can look in to it during the day and get back to you!
//R

On 08 Aug 2016, at 15:05, Mejvik<EMAIL_ADDRESS>wrote:
Thanks a lot for the help!
Unfortunately I ran into a follow up problem which I haven't managed to solve yet. The migration problem described above works fine in the sense that Calvin does not complain and the Actor can be migrated. However, when I run the sample script and use a basic TCP echoserver it doesn't work as I would like it to. When the actor is migrated the flow of tokens through the server connected to the current runtime stops as expected. However, the flow of tokens is not resumed at the second server connected to the new runtime, nor is it resumed if the actor is migrated back. Furthermore, the production of tokens in the source also stops, which I guess is due to congestion in Calvin caused by the TCP_client being unable to perform its function? In other words, there seems to be something wrong with how the actor disconnects and/or connects to my servers. The problem might very well be on the server side but I think its odd that Calvin doesn't connect to the new server when migrated. Did you test the behavior on/after migration as well?
Thanks in advance!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/EricssonResearch/calvin-base/issues/51#issuecomment-238229010, or mute the thread https://github.com/notifications/unsubscribe-auth/ALlyxIq7O797XWOk5aoa7sOAvp7aTCeYks5qdymKgaJpZM4Jcq5I.

Fixed the issue it was related to unicode strings and sending over sockets, and not the connect disconnect.
Check commit 6f894fa789fb4cc2926b83bf43a60aeb2bc7d0f7 for more info.
Simple test case:
$ ncat -l 8124 --keep-open --exec "/bin/cat” &
$ PATH=$PATH:calvin/Tools PYTHONPATH=. calvin/Tools/csruntime.py --name erobema_runtime_1 -n [IP] -p 5003 -c 5004 --keep-alive &
$ PATH=$PATH:calvin/Tools PYTHONPATH=. calvin/Tools/csruntime.py --name erobema_runtime_2 -n [IP] --keep-alive ./calvin/examples/sample-scripts/tcp_client.calvin
//R

On 08 Aug 2016, at 15:05, Mejvik<EMAIL_ADDRESS>wrote:
Thanks a lot for the help!
Unfortunately I ran into a follow up problem which I haven't managed to solve yet. The migration problem described above works fine in the sense that Calvin does not complain and the Actor can be migrated. However, when I run the sample script and use a basic TCP echoserver it doesn't work as I would like it to. When the actor is migrated the flow of tokens through the server connected to the current runtime stops as expected. However, the flow of tokens is not resumed at the second server connected to the new runtime, nor is it resumed if the actor is migrated back. Furthermore, the production of tokens in the source also stops, which I guess is due to congestion in Calvin caused by the TCP_client being unable to perform its function? In other words, there seems to be something wrong with how the actor disconnects and/or connects to my servers. The problem might very well be on the server side but I think its odd that Calvin doesn't connect to the new server when migrated. Did you test the behavior on/after migration as well?
Thanks in advance!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

Tried your example and got the following error when running the last line:
Starting runtime failed:
Traceback (most recent call last):
File "calvin/Tools/csruntime.py", line 324, in
sys.exit(main())
File "calvin/Tools/csruntime.py", line 286, in main
dispatch_and_deploy(app_info, args.wait, uris, control_uri, runtime_attr, credentials_)
File "calvin/Tools/csruntime.py", line 190, in dispatch_and_deploy
rt, process = runtime(uri, control_uri, attr, dispatch=True)
TypeError: 'int' object is not iterable
2016-08-09 14:52:18,771 INFO     361-calvin.calvin.runtime.north.calvin_node: Quitting node "['calvinip://<IP_ADDRESS>:5000']"
Since I am not familiar with what ncat does and what results I should expect (what output and where?) I also tried running my own test and end up with the following error in the log of the runtime I am migrating to.
2016-08-09 15:00:12,317 ERROR    16-calvin.calvin.runtime.north.scheduler: Data must not be unicode
Traceback (most recent call last):
File "/calvin-base/calvin/runtime/north/scheduler.py", line 123, in fire_actors
action_result = actor.fire()
File "/calvin-base/calvin/actor/actor.py", line 227, in wrapper
x = wrapped(*args, **kwargs)
File "/calvin-base/calvin/actor/actor.py", line 484, in fire
action_result = action_method(self)
File "/calvin-base/calvin/actor/actor.py", line 143, in condition_wrapper
action_result = action_method(self, *args)
File "/calvin-base/calvin/actor/actor.py", line 205, in guard_wrapper
retval = action_method(self, *args)
File "/calvin-base/calvin/actorstore/systemactors/net/TCPClient.py", line 70, in send
self.cc.send(token)
File "/calvin-base/calvin/calvinsys/network/socketclienthandler.py", line 113, in call
return self.f(self.func, *args, **kwargs)
File "/calvin-base/calvin/calvinsys/network/socketclienthandler.py", line 104, in _call
return func(self._handle, *args, **kwargs)
File "/calvin-base/calvin/calvinsys/network/socketclienthandler.py", line 89, in send
return self._connections[handle]['connection'].send(data)
File "/calvin-base/calvin/runtime/south/plugins/async/twistedimpl/client_connection.py", line 185, in send
self.protocol.sendLine(data)
File "/usr/local/lib/python2.7/dist-packages/twisted/protocols/basic.py", line 635, in sendLine
return self.transport.write(line + self.delimiter)
File "/usr/local/lib/python2.7/dist-packages/twisted/internet/_newtls.py", line 191, in write
FileDescriptor.write(self, bytes)
File "/usr/local/lib/python2.7/dist-packages/twisted/internet/abstract.py", line 348, in write
raise TypeError("Data must not be unicode")
TypeError: Data must not be unicode
Btw, I am running all of this inside docker containers if that makes any difference (other stuff works fine)
//J

The first ncat is only a echo server running on port 8124.
The error message "TypeError: 'int' object is not iterable” is not very informative but this means you probably have specified a already used port or wrong ip to listen on.
The test case with print numbers that are sent to the echo server and then back, and i was able to migrate the actor with out the flow stopping.
I created a patch to be more informative when this issue occurs.
/R

On 09 Aug 2016, at 17:07, Mejvik<EMAIL_ADDRESS>wrote:
Tried your example and got the following error when running the last line:
Starting runtime failed:
Traceback (most recent call last):
File "calvin/Tools/csruntime.py", line 324, in
sys.exit(main())
File "calvin/Tools/csruntime.py", line 286, in main
dispatch_and_deploy(app_info, args.wait, uris, control_uri, runtime_attr, credentials_)
File "calvin/Tools/csruntime.py", line 190, in dispatch_and_deploy
rt, process = runtime(uri, control_uri, attr, dispatch=True)
TypeError: 'int' object is not iterable
2016-08-09 14:52:18,771 INFO 361-calvin.calvin.runtime.north.calvin_node: Quitting node "['calvinip://<IP_ADDRESS>:5000']"
Since I am not familiar with what ncat does and what results I should expect (what output and where?) I also tried running my own test and end up with the following error in the log of the runtime I am migrating to.
2016-08-09 15:00:12,317 ERROR 16-calvin.calvin.runtime.north.scheduler: Data must not be unicode
Traceback (most recent call last):
File "/calvin-base/calvin/runtime/north/scheduler.py", line 123, in fire_actors
action_result = actor.fire()
File "/calvin-base/calvin/actor/actor.py", line 227, in wrapper
x = wrapped(args, *kwargs)
File "/calvin-base/calvin/actor/actor.py", line 484, in fire
action_result = action_method(self)
File "/calvin-base/calvin/actor/actor.py", line 143, in condition_wrapper
action_result = action_method(self, args)
File "/calvin-base/calvin/actor/actor.py", line 205, in guard_wrapper
retval = action_method(self, *args)
File "/calvin-base/calvin/actorstore/systemactors/net/TCPClient.py", line 70, in send
self.cc.send(token)
File "/calvin-base/calvin/calvinsys/network/socketclienthandler.py", line 113, in call
return self.f(self.func, *args, *kwargs)
File "/calvin-base/calvin/calvinsys/network/socketclienthandler.py", line 104, in _call
return func(self._handle, args, *kwargs)
File "/calvin-base/calvin/calvinsys/network/socketclienthandler.py", line 89, in send
return self._connections[handle]['connection'].send(data)
File "/calvin-base/calvin/runtime/south/plugins/async/twistedimpl/client_connection.py", line 185, in send
self.protocol.sendLine(data)
File "/usr/local/lib/python2.7/dist-packages/twisted/protocols/basic.py", line 635, in sendLine
return self.transport.write(line + self.delimiter)
File "/usr/local/lib/python2.7/dist-packages/twisted/internet/_newtls.py", line 191, in write
FileDescriptor.write(self, bytes)
File "/usr/local/lib/python2.7/dist-packages/twisted/internet/abstract.py", line 348, in write
raise TypeError("Data must not be unicode")
TypeError: Data must not be unicode
Btw, I am running all of this inside docker containers if that makes any difference (other stuff works fine)
//J
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/EricssonResearch/calvin-base/issues/51#issuecomment-238583440, or mute the thread https://github.com/notifications/unsubscribe-auth/ALlyxDM19XcQBOwa2ieSTIKwNgHjTVNhks5qeJedgaJpZM4Jcq5I.

Ok, thanks. I will see if I manage to get it to work.

I managed to get your test above to work outside docker containers with the runtimes on the same host.  I will try to get it to work in the setup I am using aswell.


Support podman for cmdfndoc

Fixes #3298
I was attempting to run the local tests via 'make test' and saw failures when I didn't have docker installed and was only using podman.
Setting KPT_FN_RUNTIME="podman" make test was ignored for the cmdfndoc test.
With this PR I can now run the local tests with podman.

Thank you for the PR @jwmatthews
@mengqiy can help with the review since he is our function-runtime expert :)


Update README.md

tiniest typo

Thanks @sr-ix - we will fix this! I'll close this PR and add a ticket unless you particularly want to sign the CLA: https://github.com/snowplow/snowplow/wiki/CLA

This is likely part of a professional usage, and I am not prepared to sign any CLA's on behalf of my company at this time.

np, #109


Change pressure from bar to GPa

Resolves #186

Changes units from bar to GPa in MD and geomopt (CLI)
Moves setting units for scalar_pressure when carrying out geometry optimization into calculation module. Previously this was only set for the geomopt CLI command, but this way it's set for anything that calls geomopt e.g. phonons, MD, EoS etc. and for the Python interface, so it's more consistent.

Is there a good way of signposting this? As I mention in #186, for any arbitrary kwargs passed to functions, ASE units should be assumed, so this is an exception.
When it's an explicit variable we make it clear, but is there an obvious way when it's passed via (potentially quite convoluted) kwargs e.g. --minimize-kwargs "{'filter_kwargs': {'scalar_pressure' : x}}"?
Also worth checking if I've missed obvious cases for other inconsistencies.

@harveydevereux once this is in you will need to keep an eye... since there is a small change of units from bar to GPa as defaults. also is consistent now when passed to ase.


FileGroup do not apply for Transaction Log file

FileGroup do not apply for Transaction Log file

Learn Build status updates of commit 3e09edc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/relational-databases/system-stored-procedures/sp-add-log-file-recover-suspect-db-transact-sql.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
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.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@MashaMSFT,
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment 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

awesome, thanks for your contribution @vikasrana2006! #sign-off


Work Type codes

Hello
Work Type codes - can be defined as selected on the timesheet. How does this affect the system? For example Absentees, what happens in the back end when this work type is selected? Does this accrue somewhere? What the ramifications on using work type codes?
Thank you
[Enter feedback here]

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

ID: dd49f3e2-3072-8a24-dcba-bc7aaba2625d
Version Independent ID: 00d7ace5-1e2a-7eab-ee6c-de453653635a
Content: Use Time Sheets - Business Central
Content Source: business-central/projects-how-use-time-sheets.md
Service: dynamics365-business-central
GitHub Login: @SorenGP
Microsoft Alias: edupont

Thanks for your feedback.
The main purpose of Work Type Codes is for costing and pricing resources working on jobs.
You can check the following links for more information:
Set Up Jobs, Prices, and Job Posting Groups - Business Central | Microsoft Learn
Manage Employee Absence - Business Central | Microsoft Learn


Support to not display icons in 'short' display format

Add a new 'long' optional value for the --icon flags. If set
only 'long' display formats (long or tree) will show icons.

Close #267

Hi @JrCs , thanks for this PR.
I think this feature is too custom and could be resolved with an alias like:
alias ll='lsd -l --icon=never'

I clone this PR but don't hesitate to exchange on the subject on the issue #267


Diagnostics.IsConnected

Firewalls or Pi-Holes often block communication to App Center. It would be great to check if App Center can connect or is connected.
I don't find a workaround, tried a HttpRequest to in.appcenter.ms but this failed.
Have someone an idea for a workaround?

Its not a beauty code... but it works (for me) 🦄
private static bool IsAppCenterConnected()
{
  try
  {
    string url = @"https://in.appcenter.ms";
    var httpWebRequest = (HttpWebRequest)WebRequest.Create(url);
    httpWebRequest.GetResponse();
    return true;
  }
  catch (WebException e)
  {
    return e.Response != null;
  }
}

Any update?

Bump

Bump

Hi Johann,
I would like to know more about your feature request.
Could you please share more details about the behaviour you want to achieve by using such API?

Sure 💖
I am using appcenter in my .NET apps. So it would be nice a Property like Appcenter.IsConnected to check if the client can reach Appcenter through proxies etc.

Thank you for the details.
Could you elaborate on this. Could you describe an example of usage of this API? How would your app react to the result returning by calling this API?
I just want to get a better picture of a possible use cases of such API.

In a desktop application the user can opt-in for a, we called it, "Appcenter logging". Then all critical events are logged into app center.
Our enterprise customers often have proxies in their companies which are blocking https://in.appcenter.ms.
So we want to give back a feedback in the UI if the connection fails.

Got it, thanks for the clarification! Looks like it is not a standard way of using AppCenter SDK. Since you expose the usage of AppCenter to users, I would rather recommend to handle such connectivity checks on your side.
Actually, this API won't work consistently, because there are many reason which can affect the connection stability. On the other hand, on your end you would be able to check if there is any proxy used on the device and handle it as you need. In case of proxy, you can use SetLogUrl API, see: https://github.com/microsoft/appcenter-sdk-dotnet/issues/624.

The user don't see anything about AppCenter. Its just a checkbox for "cloud logging". I described it for you directly to better understand :)

Hi @suplanus! Thank you for sharing the use case and starting this thread. It's an opportunity for us to learn better about our customers, and I'm sure the community will find your case informative.
From the feature request perspective, this can be tricky. I'm afraid this feature won't be as expandable for some scenarios, and might end up leading to confusion. For example, The API won't work as expected when the network connection on the end user side is not stable at the application start. Due to this discrepancy, the API usage will be very limited to specific scenarios. For this reason, I echo what @DmitriyKirakosyan suggested above to handle connectivity on the application side. Thanks again for sharing your feedback with us.


Reminders seem to be deliberately obtuse

You ask it to remind you in 10 minutes, it says it'll remind you in 9! What cheek!

I'm going to guess this is because the minute after you write the message it's 9 minutes and 59 seconds so it says 9 minutes. The logic isn't broken, just the output... not worth fixing. :)


Is there a way to provide the root user & passwd and the user root&passwd to the testcase?

@liangchenye
When I write the json file ,I met the problem, we usally have the both root and the user rights in the same programme, and always have to switch between both of them. So , Is there a way to provide the root user & passwd and the user root&passwd to the testcase?
A example is:
in Spectest for runc, we have to use root power to run runc, also ,we need user power to build the guest programe.

resovled.


Tag.equals fails when Tag name is null

Why do you have a tag with a null name?

List items can have a null tag name. Do you recommend an empty string instead?

Yes. I'm pretty sure that's how I do it when reading tag lists.

Alright :) thank you


fuzz: store the corpus

We have a few fuzzers running.
To maximize their impacts, we should store the corpus to be reused in a new run.
Maybe we could use a separate repo to store the corpus.
More info on:
https://github.com/rust-fuzz/cargo-fuzz/issues/194

I have a simple idea for using the GitHub action to store artifacts as the below doc described,
https://docs.github.com/en/actions/using-workflows/storing-workflow-data-as-artifacts
then every time the action runs it

download the gziped corpus
running the test
upload a gziped corpus

what do you think?

exactly what I had in mind :)

ok I'll try to create a PR for this 🙇🏻

Thanks!


dns.staticHosts parsing failed in jsonv5 format

What version of V2Ray are you using?
https://github.com/v2fly/v2ray-core/releases/tag/v5.2.1
What's your scenario of using V2Ray?
Setting up host mapping for dns servers.
What problems have you encountered?
The dns config failed to parse.
What's your expectation?
Passing the config parse.
Please attach your configuration here
Client configuration:
{
  "log": {
    "access": {
      "type": "File",
      "path": ".\\access.log",
      "level": "Warning"
    },
    "error": {
      "type": "File",
      "path": ".\\error.log",
      "level": "Warning"
    }
  },
  "dns": {
    "staticHosts": [
      {
        "type": "Full",
        "domain": "doh.opendns.com",
        "ip": "<IP_ADDRESS>"
      }
    ]
  },
  "inbounds": [],
  "outbounds": [],
  "router": {}
}

If V2Ray cannot start up, please attach output from --test command
v2ray test -c configtest.json -format jsonv5
V2Ray 5.2.1 (V2Fly, a community-driven edition of V2Ray.) Custom (go1.19.4 windows/amd64)
A unified platform for anti-censorship.
Test failed: main/commands: failed to load config: [configtest.json] > infra/conf/v5cfg: unable to build config > infra/conf/v5cfg: failed to parse DNS config > common/registry: unable to parse json content > json: cannot unmarshal string into Go value of type []json.RawMessage

duplicate with #2107


Normalize Windows paths for forward slashes

Fix to address #26. Essentially using the solution suggested by @javi7.

@chouhanyang, good to merge?

setup.py was failing under windows because it was trying to copy the bash completion stuff into an absolute path meant for Linux. I just added a fix so bash completion stuff is ignored when not on a Posix box. This project is now totally Windows compatible as far as I can tell!

Hi,
It is not clear to me how your change would work since you only deal with upload? How about download and other commands?


Low accuracy on custom coco format dataset

你好，感谢你们工作的开源，我把自己的数据转换成coco格式后，在大约20epoch的时候，训练100个epoch准确度还是很低，total_loss基本就不变了，请问下会是什么原因导致的呢

这个是我的训练命令：python tools/train.py -n yolox_tiny -f /YOLOX-main/exps/example/yolox_voc/yolox_my.py -d 2 -b 16 --fp16 -o

Plz load the pretrained COCO weights using -c /path/to/the/weights

Plz load the pretrained COCO weights using -c /path/to/the/weights

OK, thank you very much. I'll try this method.But if I want to train from scratch with my own dataset, is there any solution to this problem

I trained yolox tiny on my datasets and meet the same problem, maybe caused by init setting about bn: "m.eps = 1e-3 m.momentum = 0.03", I also try to init weights with coco pretrained model , waiting for  eval results now.

@BowieHsu @JasenWangLab Training from scratch is not recommended, unless your training data is as large as COCO dataset.

你好，感谢你们工作的开源，我把自己的数据转换成coco格式后，在大约20epoch的时候，训练100个epoch准确度还是很低，total_loss基本就不变了，请问下会是什么原因导致的呢

这个是我的训练命令：python tools/train.py -n yolox_tiny -f /YOLOX-main/exps/example/yolox_voc/yolox_my.py -d 2 -b 16 --fp16 -o

I am also suffering from this trouble. Even when I load the pretrained model of yolox-s, the loss stabilizes around 10.0 and the eval map is approaching 0, which is even worse than the beginning (about 0.45). @Joker316701882 @ruinmessi Do you know what causes this phenomenon ? @JasenWangLab Did you manage to solve this problem?

Same problem here as @TUDelftHao

兄弟，有好的解决方案吗


Thank you for signing the Anthem Opensource Software CLA.


[Feature] Option "enabled" in MoveToMapPokemon

New option in config like  rename function have.
{
    "type": "MoveToMapPokemon",
        "config": {
            "enabled": true,
            "...": "..."
        }
    }
}

This is already available for all functions, you just need to add the enabled flag


[Improvement] Big Partition require buffer fail, and improve it.

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

What would you like to be improved?
When there is big partition require buffer, require buffer fails and may wait.

Until there is flush:

How should we improve?
Provide better default arguement and advise.
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Machine 128G memory， rss Xmx 75G, -XX:MaxDirectMemorySize=20G.

Now,
rss.server.shuffleBufferManager.trigger.flush.interval
is default value, and we want to set it to 180.

Now the huge partition is default value 20G, and it is bigger for me,  I want to adjust to 8G according to the log:
(Make it flush early)

Above is First Appear.


Copying LinearForm objects leads to seg fault

I happened to notice that the following code leads to a segmentation fault.
{
   ParLinearForm lf1(fespace_q);
   lf1.AddDomainIntegrator(new DomainLFIntegrator());
   lf1.Assemble();

   ParLinearForm lf2(fespace_q);

   lf2 = lf1;
}

The intention was to copy the contents of lf1, viewed as a Vector, to lf2.  What actually happened, as seen in TotalView, was that the vector data was copied as desired but the arrays that contain LinearFormIntegrators were also copied.  When these objects fell out of scope they both attempted to delete the LinearFormIntegrators and caused a segmentation fault.
The question is, do we want lf2=lf1 to simply copy the vector data or should it copy everything?  I'm not sure we can support the latter without smart pointers or some sort of reference counting.

We can define operator=(const Vector &) for (Par)LinearForm to simply call Vector::operator=(const Vector &) -- this is what is done in class (Par)GridFunction.

Yes, that sounds best to me as well.  I'll take care of it.  Thanks!


Fix IRQ EOI issue

Currently the IRQ handler doesn't send EOI, but expects the defered delegate handler to do so. When that happens just the non-specific EOI is sent, which means (if I understand the whitepaper correclty) that only the highest priority IRQ gets ended.  We need to be able to disable specific IRQ lines, either by switching to auto-mode, sending EOI right away, or by sending EOI to the specific IRQ line.

This is complete since #446 as far as I can get with the PIC. We're now using auto-EOI for the master PIC, but everything freezes if I try to use auto-EOI for the slave PIC as well. According to whitepaper atuo-EOI-mode for slave-PIC wasn't supported until 1985 so I guess we can't expect that to work for virtual machines in 2016. But maybe in 2017.


Release 0.13.1

HEADS UP: This is actually a roll up of everything up to fa235193, plus one commit. I realize this procedure is a bit odd for a dot release, but @brendanburns left a lot of stuff out of 0.13.0, so we decided to do a quick turn .1 that caught up.

Sorry, wrong target here. Obviously can't merge this into master.


New release

Hi!
Would you consider releasing a new version?
Thanks!

Ping @Nyholm
It would be nice to have a release that contains https://github.com/FriendsOfApi/localise.biz/commit/3668d3ae4af5959703707906280c8099f5adc656


make sure IncreaseFrameIndex happens right after draw

Doing things like InvalidateVisual in response to input requires that
the UpdateManager.FrameIndex was increased right after drawing, because
otherwise the frame index will be the same when the input happens,
causing elements that were validated the same frame to not be
invalidated.
This problem was introduced in 81eae3e, where another similar issue
was worked around. This should fix both cases.
Fixes issue #182
Unfortunately, this doesn't seem to be possible to write a test-case for,
because the logic happens in AppBase, and we don't have an instance
of that during uno test.

If this has been manually verified to fix #182, then this LGTM

Yeah #182 is def fixed; that was verified.


Please keep github release tags up to date

Please add to your release process to update github tags.  In particular I'd like to know when there are new shoal-agent versions since I include that in the frontier-squid rpm.  Also please tag the latest release.

I see that the shoal-agent-1.0.0 has been tagged, thank you.  I did a diff against agent-0.9.9 and noticed that the rpm spec still says 0.9.9 so that's another thing that needs to be included in a release procedure.

Okay, we will keep this in mind, thought we do not support a rpm for the agent.


[Enhancement]: add store readiness gate to CN

Is there an existing issue for enhancement?

[X] I have checked the existing issues.

What would you like to be added ?
Add store readiness gate to CN Pod
Why is this needed ?
Currently, cn rolling-update relies on Pod state to judge the readiness, however, an CN would not be available for compute before it appears in HAKeeper with Working state. The mis-judged readiness would result the count of actual ready CN store below the desired count during automated process like rolling-update.
Additional information
No response

closed by #397


Populate sample data

Populated DebugCommand with sample data

Coverage decreased (-0.5%) to 68.636% when pulling e2f2908b4fc6eb9f88267d197a0994061255f8ff on zhicaizack:master into 93c5478751f9594755d4c8c43c48e171bde1116f on CS2113-AY1819S1-T16-1:master.


feat: move file hashes to a dedicated lockfile

Codecov Report

Merging #281 (36b4b54) into main (e7ee837) will decrease coverage by 1.35%.
The diff coverage is 85.24%.

@@            Coverage Diff             @@
##             main     #281      +/-   ##
==========================================
- Coverage   95.14%   93.78%   -1.36%
==========================================
  Files          16       17       +1
  Lines         926     1013      +87
==========================================
+ Hits          881      950      +69
- Misses         45       63      +18

Impacted Files
Coverage Δ

src/ops2deb/cli.py
92.38% <58.33%> (-4.53%)
:arrow_down:

src/ops2deb/fetcher.py
92.30% <58.33%> (-6.89%)
:arrow_down:

src/ops2deb/parser.py
95.48% <90.90%> (-1.21%)
:arrow_down:

src/ops2deb/lockfile.py
97.82% <97.82%> (ø)

src/ops2deb/exceptions.py
100.00% <100.00%> (ø)

src/ops2deb/generator.py
94.21% <100.00%> (ø)

src/ops2deb/updater.py
89.84% <100.00%> (ø)

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


feat(data-preset): add data-preset button locale

For: https://github.com/Yellow-Dog-Man/FrooxEngine/pull/653

Thanks!


Fixed bug that wouldn't allow containers to touch top and left border

Fixed the bug that wouldn't allow containers to touch the top and left borders of the svg canvas.
Still have to decide how we want to deal with container text overflowing the right border of the svg.
It currently disappears behind the right edge, not necessarily a problem.  But we might want to look into enforcing a text border.

Made Changes to the yaml parser. State is no rendering networks correctly.
Changed CSS for D3Wrapper to render buttons in the center on initial load (still think we should fix the centering of the form, but wasn't able to do what I wanted with it so I left it as is)
Still working on dynamically changing borders with screen width. But wanted everyone to have changes to state moving forward.


Personalization group (s)pgid is missing in request call for top level categories

In current version, an API call is there from method getTopLevelCategories in categories.service.ts to fetch top level categories to display on home page.
Actual Behavior
Mentioned call doesn't include personalized group parameters in URL (s)pgid. If I change line 47 of categories.service.ts file to include sendSPGID parameter as
return this.apiService.get('categories', { **sendSPGID: true**, params }).pipe(
spgid  gets added to the request, but with undefined value.

Expected Behavior
Personalization parameter should be included in top level categories call with correct value.
Steps to Reproduce the Bug

Go to PWA homepage
Login with valid account.
Navigate to home page again and check top level categories call in developer console.
Notice URL is missing with personalization group (s)pgid parameter.

To me it seems pgid get set in store after the call to top level categories made. So for mentioned call pgid is always undefined. But for subsequent calls to sub-categories and products valid pgid is present in store and Urls.

Hi Kapil,
thank you for raising this bug. Currently we only support personalization of CMS content in PWA out of the box. Personalization of products, prices and categories is one of the next topics we want to address but currently it is not in progress. We are going to handle this issue once we work on personalization in a larger context.

Converted to feature request.


REX-Ray 0.11.0 doesn't start on machines w/o docker w/o special config

Summary
I think this is a change as of 0.11.0 since we created the CSI module backend, but if you try to start REX-Ray on a machine without Docker installed, it fails unless you specify a custom CSI_ENDPOINT. I'm not sure this is by design.
Bug Reports
Version
# rexray version
REX-Ray
-------
Binary: /usr/bin/rexray
Flavor: client+agent+controller
SemVer: 0.11.0
OsArch: Linux-x86_64
Commit: 508023f4b0e4974f1844c26b39af7c4219f4c7a9
Formed: Mon, 16 Oct 2017 00:07:06 PDT

Expected Behavior
Be able to start REX-Ray with minimal config, even w/o Docker installed
Actual Behavior
REX-Ray fails to start with: error: service startup failed: agent: mod start failed: invalid implied sock file: /run/docker/plugins/rexray.sock: open /run/docker/plugins/rexray.sock: no such file or directory
Steps To Reproduce
rexray start will make things fail because it tries to create the .sock file in /var/run/docker which does not exist. Setting CSI_ENDPOINT to a valid path makes it work: CSI_ENDPOINT=/var/run/rexray/csi.sock rexray start
Configuration Files
I was using a ScaleIO system at the time:
rexray:
  logLevel: debug
libstorage:
  service: scaleio
scaleio:
  insecure: true
  thinOrThick: ThinProvisioned
  endpoint: https://<IP_ADDRESS>:443/api
  userName: admin
  password: ****
  systemName: scaleio
  protectionDomainName: default
  storagePoolName: default

Logs
https://gist.github.com/codenrhoden/08b00b4258f03f6d79362365b7cb89bc

Hi @codenrhoden,
This is a side-effect of that change, yes. The old Docker module would mkdir -p the parent directory.

I am experiencing the same problem on machines with docker swarm running as AWS EC2 VM on Amazon Linux 2: AMI amzn2-ami-hvm-2.0.20190313-x86_64-gp2 (ami-0de53d8956e8dcf80).
rexray.service - rexray
   Loaded: loaded (/etc/systemd/system/rexray.service; enabled; vendor preset: disabled)
   Active: failed (Result: exit-code) since Thu 2019-05-02 14:36:32 UTC; 6min ago
  Process: 2648 ExecStart=/usr/bin/rexray start (code=exited, status=1/FAILURE)
 Main PID: 2648 (code=exited, status=1/FAILURE)

May 02 14:36:29 swarm-1-manager-1 systemd[1]: Started rexray.
May 02 14:36:29 swarm-1-manager-1 systemd[1]: Starting rexray...
May 02 14:36:32 swarm-1-manager-1 rexray[2648]: error: service startup failed: agent: mod start failed: invalid implied sock file: /run/docker/plugins/rexray.sock: open /run/docker/plugins/rexray.sock: no such file or directory
May 02 14:36:32 swarm-1-manager-1 systemd[1]: rexray.service: main process exited, code=exited, status=1/FAILURE
May 02 14:36:32 swarm-1-manager-1 systemd[1]: Unit rexray.service entered failed state.
May 02 14:36:32 swarm-1-manager-1 systemd[1]: rexray.service failed.

I found a work-around which is to modify /etc/systemd/system/rexray.service to read as follows:
[Unit]
Description=rexray
Wants=scini.service
Before=docker.service
After=scini.service
StartLimitInterval=200
StartLimitBurst=5

[Service]
EnvironmentFile=/etc/rexray/rexray.env
ExecStart=/usr/bin/rexray start
ExecReload=/bin/kill -HUP $MAINPID
KillMode=process
Restart=always
RestartSec=30

[Install]
WantedBy=docker.service

This gives the rexray service five chances to start properly with a 30 second wait between restarts.
This doesn't address the underlying problem, but at least it provides a functioning system that doesn't require manual intervention when a swarm node boots up.
Hope this is helpful/useful.


Add docker-based CI testing for compiled codes

For instance, it would be nice to have a CI test that runs a real GULP calculation rather than mocking the output.

Closing as not worth the testing time overhead.


Bump flower==0.9.2 (#4263)

This PR cherry-picks https://github.com/apache/incubator-superset/pull/4263 which bumps the version of the flower package. In the Flower UI @michellethomas and I noticed an UI bug which according to the forums may have been resolved in 0.9.2.
to: @graceguo-supercat @michellethomas

lgtm


Optimized topbar for iPad

@sreilly can you make sure that on iPad the search-field and segmented tabs have a maximum width of 710px/355pt. The dropdown is slightly wider at 750px/375pt (same width as an iPhone 6 screen). In terms of measurements we're trying to mimic the measurements on iPhone here, so the home/settings icon would appear in the same place. This means that the search-field gets slightly less wide to make place for the house/settings icon and the "Cancel" button on either side.

@thm question: where should the cancel button go when the field has the focus?  If it should be next to the field, should it animate in all the way from the side of the screen?
Also, what about the home button when the user is more than one level deep: where should that go, and how should the animation go?

@sreilly Can you make it fade in while it slides in with the changing width of the search field?
Quick demo here: http://cl.ly/0v3J3r0N3o0M/iPad.mov

will do!

@thm the latest build includes the search field that works closer to what you describe here, aside from the popover.  One problem is that on the iPhone 6+ 355 pts isn't wide enough to fill the top nav bar, so I set it to a maximum of 414pts.  Is that ok?
Also, for the popover, would you like to have the default iOS behaviour of taps outside of the popover dismissing it, or should it stay visible whenever the search field is active, as on the iPhone?

Yup, a tap outside should dismiss the popover, so the same result as the Cancel button I think! Also make sure the popover overlays the current page. So, when you search on the Stories page for example, you'll be able to see the stories behind the popover. For the overlay let's try ~35% black for now
I'll check the 415pt search-field later today on an iPad, curious how it looks there. But it'll probably be fine!

@thm when in the search tab, should that still have a popover or just use the search field contents to filter the contents of the main view (as it does on the iPhone)?

@sreilly Good question! I think we should put everything in the popover on the Search page, for consistency. So that would look exactly like the screenshot posted above, until you start typing and auto-complete takes over

I also noticed the Cancel button and home icon don't fade in yet, they do slide in, but ultimately they should start at 0% opacity and go to 100% opacity during the sliding animation

I also noticed that when switching pages on the iPad the blue progress bar is briefly visible on the upper-right

@thm ok, I'm just about to push an update that implements all of this.  The cancel and home button animation is in there, but it's a quick one (I believe it's 0.25 seconds).

The cancel button still doesn't fade in. Again, it slides in, but it doesn't go from 0 opacity to 100 over the course of the sliding animation. It pops into view, and then slides to the left. Here's a slowed-down demo: http://cl.ly/1d3I3r3T2K2h/comparison.mov

You forgot to add the overlay behind the popover. Can you add an overlay of 35% black so that the popover doesn't blend in as much with the stories underneath?

And in the image below is more alignment and visuals feedback:

This should be all fixed as of the next build!

:+1:

Can you make tapping the overlay dismiss the popover? So, same functionality as the "Cancel" button. Right now when you tap the overlay, it activates the story underneath

Do you think a better solution for the popover's arrow would be adding the same 50px margin at the top of the list as we have for iPhone?
Current (when scrolling the list down, the white background becomes apparent):

Proposed:

@thm Personally I prefer to omit the empty grey margin at the top as it's just wasted space.  I think the current solution where the line disappears as it touches the triangle looks pretty good.
I do think that tapping to dismiss having the same effect as the cancel button (ie resetting and taking the focus away from the search field) might be a bit frustrating.  I can imagine people accidentally tapping outside of the popover (quite a large target) and being upset about losing their half-typed query.  Maybe we could simply ignore taps outside of the popover in the dimmed area?

I don't really like the white arrow tip, in the list's resting state I like the way it looks, but when scrolling it just looks awkward. The argument for it being wasted space is less of a concern to me than it looking slightly akward with the grey space inbetween. So, below is the solution I've come up with:

...and then when scrolling down the list:

Below is the final spec sheet for the visuals. In addition, can you round off the tip of the arrow with 1-2pt more? (top = current, bottom = new):

In addition, I think tapping the overlay should dismiss the popover. It's default/expected behavior in iOS

@thm This should be done now, and the resulting build on hockey.  I think the result is a tiny bit weird because touches on the dimmed view dismiss the popover (and aren't passed through), but taps above the dimmed view don't.
Tapping the dimmed view performs the same action as the Cancel button, but do we want to clear the contents of the search field upon tapping outside the popover?

@sreilly we should never clear the search-field, unless the user taps the clear-icon. In any other case (ie. tapping "Cancel" or tapping the overlay) we should only pause the search. So, the current query remains, whether it's typed out completely or not, and whether the user has actually hit "Search" or not.
Also, I'm sorry for the extra work this has brought you, but can you please remove the arrow all-together and make the first item flush with the top of the popover? See screenshot below:

Also found this on the Favorites and Recents pages. The overlay isn't stretched to the top.

@thm cool... can you give the latest (in about 3 minutes) build to see how it feels?

We seem to only show the 5 most recent recents and favorites in the popover. That should be the 5 most recent recents and all favorites.

The icons' measurements are off again:

Can you make the margin inbetween the topbar and the popover 16px/8pt again? It's currently ~3px

@thm all set, thanks!  Changes made and pushed.

@sreilly perfect, thanks! on just the Favorites and Recents pages can you make sure the distance between the popover and the search-field is 16px/8pt instead of 32px/16pt? That would make it properly cover the segmented tabbar. Though please note, only on the Favorites and the Recents pages.
Feel free to move this to the "iteration 1" milestone if it takes up too much time!

@thm piece of cake!  this change is baking now

@sreilly Thanks a lot!


Add support for .inc files

ArmA 3 uses .inc files as a way to create "include" file/library that shouldn't be merged into the config file. This is similar to ACE/CBA's script component files. This PR simply allows the .inc files to be treated as a ArmA configuration file.

Thank you for the contribution!


Luis extension should support patterns, key phrases and sentiment

Push out due to design dependency and LUIS dependencies


relative-pointer-v1: move pointer iteration in wlroots

This makes sure we always send events to the right client.

LGTM aside from the notify_relative_motion which doesn't really make sense now IMO


Support reusable workflows

This PR adds support for creating documentation of reusable workflows.

I just realized it must also support outputs from reusable workflows and a bug I introduced.. Will update PR.

@andreas-aman Thanks I’ll take a look later today

@andreas-aman Thanks I’ll take a look later today

@jackton1 Had time to look at this yet?

I'm currently in need of this feature, but for the life of me I can't figure out how to make @andreas-aman's fork work. There seems to be hardcoded references to tj-actions in the project, and I tried to get go to use my cloned fork by using go mod edit -replace github.com/tj-actions/auto-doc=/my/cloned/path, but if I go build then it seems to just build the code from this repository as opposed to the forked repository.
Apologies for my newbie-ness when it comes to Go, but I wanted to test this feature out.

I'm currently in need of this feature, but for the life of me I can't figure out how to make @andreas-aman's fork work. There seems to be hardcoded references to tj-actions in the project, and I tried to get go to use my cloned fork by using go mod edit -replace github.com/tj-actions/auto-doc=/my/cloned/path, but if I go build then it seems to just build the code from this repository as opposed to the forked repository.
Apologies for my newbie-ness when it comes to Go, but I wanted to test this feature out.

@Makeshift Not sure how to achieve this with the git cli. But if you have the github cli you can simply checkout this PR and build it. gh pr checkout 405

@all-contributors please add @andreas-aman for code and docs


Target WASM runtimes where wasm-bindgen is not available

Is your feature request related to a problem? Please describe.
I'm trying to run an RDF database and expose a SPARQL endpoint on an Internet Computer canister. For this reason, the features I'd need the most from oxigraph are the database and the SPARQL parser.
However, but I keep getting this error when deploying it:
Module imports function '__wbg_getRandomValues_3774744e221a22ad' from '__wbindgen_placeholder__' that is not exported by the runtime.

From what I understand, this means that the canister runtime doesn't expose wasm-bindgen library.
Describe the solution you'd like
I'm not that expert in Rust and compiling it to WASM, hence I don't know exactly if it would be possible to avoid the usage of wasm-bindgen and what effort it could it take.
Describe alternatives you've considered
I've tried sophia, which compiles and runs smoothly, but is missing the SPARQL parser. Is there a way to use oxigraph's parser with sophia?
Additional context
I'd suggest you to dive into the Internet Computer documentation, because I see a great potential here to enable decentralized RDF databases that can have various applications like decentralized social media platforms, etc.

The issue is not with wasm-bindgen (you will likely need that if your compile target is WASM), the problem is with the usage of getrandom for generating random values for Blank Nodes.
See https://docs.rs/getrandom/latest/getrandom/#webassembly-support for more context. It looks like your WASM runtime environment (the IC canister) doesn't expose a Crypto.getRandomValues function to the WASM module.
AFAIK if the js feature flag would be removed here (https://github.com/oxigraph/oxigraph/blob/main/lib/Cargo.toml#L51-L53), it would be possible to register a custom random number generator for non-JS WASM runtimes. (I guess oxigraph would need to introduce a feature flag that's turned on by default to allow this in a backwards compatible manner).
From what I know from working with Web3 systems in the past, random numbers are notoriously hard/slow to generate in the runtimes of those systems (I would assume that IC is no different there). So if you want to embark further on this topic, there is a good chance that this could be quite complicated and/or may not yield a result that has a good performance profile.

Thanks @hobofan!
Given the complications that could come out from porting oxigraph to the IC by registering a custom random number generator and probably also changing other things, you wouldn't recommend to spend time on it, right?
In this case, do you have some ideas and/or suggestions on how to create and expose an RDF database on the IC without rewriting everything from scratch?

Hi @Luca8991! thank you for trying Oxigraph.
Hi @hobofan! Thank you for your detailed answer.
Oxigraph compiled to wasm32-unknown-unknown relies on 2 JS method: Crypto.getRandomValues for randomness (used to generate random variable and blank node ids and to power some SPARQL functions like RAND and UUID) and Date.now for the SPARQL NOW function.
It would be great to have a no_js feature Oxigraph to allow compiling without JS dependencies on wasm32-unknown-unknown. The main challenge imho is to replace the random blank node generation with some system that would generate unique blank node identifiers. For variables, it's only used in the parser so knowing which variable names are already used in order to properly generate new variable names should not be hard. And for the SPARQL functions RAND, UUID and NOW we could maybe just disable them.  For blank nodes, using a global counter and hope for no conflict might be a way to go.

Given the complications that could come out from porting oxigraph to the IC by registering a custom random number generator and probably also changing other things, you wouldn't recommend to spend time on it, right?

I mean in the end it depends on your motivation. As I don't have any experience with IC concretely (and also don't want to spend time digging into it), I can't judge how much effort it would ultimately be. (Who knows, maybe IC does something really clever with random numbers and exposes a great interface for them 🤷)

In this case, do you have some ideas and/or suggestions on how to create and expose an RDF database on the IC without rewriting everything from scratch?

I guess an alternative route (which is probably also quite some work) would be to fork oxigraph and rip out the parts that require random number generation. AFAIK that would "only" be the support for generating anonymous blank nodes. I'm unsure how many parts of the oxigraph internals rely on the generation of those, and for how many of those there is an adequate workaround, or functionality would have to be sacrificed.
Another possible alternative would be to ensure unique anonymous blank nodes via a mechanism that doesn't rely on random numbers (e.g. https://github.com/Stebalien/snowflake).

Thanks for the precious advices @Tpt and @hobofan! I've created a fork at omnia-network/ic-oxigraph in which I ported the necessary functions to make Oxigraph run on the Internet Computer runtime. Now it compiles and apparently runs correctly!
I'd be really thankful if you could give it a quick look and tell me if I need to adjust something. Here's the relevant commit: https://github.com/omnia-network/ic-oxigraph/commit/fb8406e1bbf300486f7510ffd2cb99674179c72a

@Luca8991 It looks great! For the next breaking Oxigraph release we might:

add to oxigraph and oxsdatatypes a js feature to enable usage of js-sys for random and time. It should be disabled by default.
introduce something like register_custom_getrandom for the "now" operation to allow providing a custom implementation of now.
I believe it would cover the "internet computer" use case.
What do you think?

add to oxigraph and oxsdatatypes a js feature to enable usage of js-sys for random and time. It should be disabled by default.

Does this mean that if I don't enable the js feature I won't target WASM?

introduce something like register_custom_getrandom for the "now" operation to allow providing a custom implementation of now.

Sounds great! Will there also be the possibility to "register" a custom Date::now() implementation, so that we can use the ic_cdk::api::time function when using oxigraph on the Internet Computer?
Also, the maximum compiled WASM size supported by the IC is 2MB (they're working on extending it as far as I know), and this makes the compiled WASM that uses the oxigraph library pretty "huge" (~ 4.5MB). It can anyway be deployed on the IC with gzip, but is there a way to reduce the library size? Maybe by adding optional features that one can choose?
If you've also took a look at the fork I created, you've probably noticed that I've removed a lot of folders that I don't need (oxrocksdb-sys, python, server, etc.) and this probably will make it hard to merge oxigraph again into it in the future to get updates, etc. Is there a better way to keep the repo simple but also easily updatable?
I'm also getting some errors when running cargo test in the repository where I use the ic-oxigraph library, and I think that's because the test doesn't target wasm32-unknown-unknown by default. The errors come from rocksdb.rs file and are related to libc missing or similar. Do you have suggestions on how to solve this?

add to oxigraph and oxsdatatypes a js feature to enable usage of js-sys for random and time. It should be disabled by default.

Does this mean that if I don't enable the js feature I won't target WASM?

It means that it will target WASM but without asusming that a JS runtime is availlable. So, it would work with e.g. the "Internet Computer".

introduce something like register_custom_getrandom for the "now" operation to allow providing a custom implementation of now.

Sounds great! Will there also be the possibility to "register" a custom Date::now() implementation, so that we can use the ic_cdk::api::time function when using oxigraph on the Internet Computer?

Yes, exactly. It's the goal!

Also, the maximum compiled WASM size supported by the IC is 2MB (they're working on extending it as far as I know), and this makes the compiled WASM that uses the oxigraph library pretty "huge" (~ 4.5MB). It can anyway be deployed on the IC with gzip, but is there a way to reduce the library size? Maybe by adding optional features that one can choose?

Yes, optional features are a great idea. For example we might allow to not compile all parsers and serializer. I am not sure which component takes the most space. Here are some tips on reducing the compiled code size.

If you've also took a look at the fork I created, you've probably noticed that I've removed a lot of folders that I don't need (oxrocksdb-sys, python, server, etc.) and this probably will make it hard to merge oxigraph again into it in the future to get updates, etc. Is there a better way to keep the repo simple but also easily updatable?

That sounds like a good idea. I already do something similar in Oxigraph RocksDB fork. I wrote a script that removes the files I do not care about. This way I can easily pick a new rocksdb release, apply on top of it some patches and then do a final commit with the cleanup script.

I'm also getting some errors when running cargo test in the repository where I use the ic-oxigraph library, and I think that's because the test doesn't target wasm32-unknown-unknown by default. The errors come from rocksdb.rs file and are related to libc missing or similar. Do you have suggestions on how to solve this?

I believe cargo-test does not work with wasm. There are multiple tools, one for WASM+JS and one for WASM+WASI. I run the WASI one in the CI.

Here are some tips on reducing the compiled code size.

Thanks a lot for this resource! I reduced the WASM size by almost 50% and I didn't use all the optimizations available! So I guess the WASM size won't be a problem for now.

I believe cargo-test does not work with wasm. There are multiple tools, one for WASM+JS and one for WASM+WASI. I run the WASI one in the CI.

I can't use WASI because I'm getting errors regarding some methods imported from the IC SDK, that are not available in that WASM+WASI tests runtime.
I solved the problem by removing every line that was targeting another target than WASM (basically every line with #[cfg(not(target_family = "wasm"))] attribute) and removed every #[cfg(target_family = "wasm")] attribute to make the related line being compiled by default. You can see it in this diff.
So now running cargo test works again. However, I don't feel it's good, also because writing a script that removes the unneeded parts like the one you mentioned becomes quite hard. It would have to go through all the code to find the lines with a different target and remove/adapt them as I did manually. Do you have some suggestions on how improve this?

Done in 0.4.0-alpha.1. The "js" feature needs to be enable for integration into js-sys and a hook point exists for "now" support: https://github.com/oxigraph/oxigraph/tree/main/lib/oxsdatatypes#datetimenow-behavior


Native TLS

This PR adds the native TLS support from Vouch into this Helm chart.
It also contains modifications from https://github.com/vouch/helm-charts/pull/40 as I use them both, disregard these changes for this PR.

I was looking at the TLS and saw that it was missing. It would be really nice to have it ... :)


sip.parser | error parsing header "m"

SIP.js is not able to handle parsing of 200 OK packet while m header contains any string with # character.
Thu May 24 2018 11:22:55 GMT+0530 (IST) | sip.transport | received WebSocket text message:
SIP/2.0 200 OK
v:SIP/2.0/WSS 8r1po5314s38.invalid;branch=z9hG4bK9420222;received=X.X.X.X;rport=36874
From: "Ghanshyam Katriya" <sip:10001080@pbx.xyz>;tag=cv333cb995
t:"Ghanshyam Katriya"<sip:10001080@pbx.xyz>;tag=46UQm7ggKt1ZS
Call-ID: ve9co05i7flonpbq1p337s
CSeq: 83 REGISTER
m:<sip:34098f8f@8r1po5314s38.invalid;transport=ws;received=X.X.X.X:36874>;expires=600,<sip:10001080@x.x.x.x.x:65080;transport=tcp;received=x.x.x.x:62580>;expires=2353204,<sip:PBX#1000#905448737777@x.x.x.x>;expires=2519533,Date:Thu, 24 May 2018 05:52:55 GMT
User-Agent:FreeSWITCH-mod_sofia/1.6.20~64bit
Allow:INVITE,ACK,BYE,CANCEL,OPTIONS,MESSAGE,INFO,UPDATE,REGISTER,REFER,NOTIFY,PUBLISH,SUBSCRIBE
k:timer,path,replaces
l:0

Hi,
This is actually not an error- the "#" character is not supported in the grammar for the contact header outlined in https://tools.ietf.org/html/rfc3261#page-228

But same thing is handled by all IP Phones & Soft-phone like linphone etc. what about them? Can't we handle it in SIP.js? @JimGreenberg

Ultimately, we are an RFC compliant library- if we stray from the spec we could inadvertently break something else in the library.
I've dealt with phones that which are strict in this sense, and would break in this instance- keeping RFC compliance makes it much easier to maintain as well.
That said, we'd be open to a pull request that allows this optional functionality, probably as a configuration option on the UA.

Modifying the grammar to optionally support this is not going to be an easy task. I believe that it would be much simpler to fix the UA that is generating the offending contact. Additionally, if you escape the # in the contact, that would also solve your issue.


No human-readable message when scanning wrong QR code with vaccine certificate scanner

Technical details

Device name: Samsung Galaxy A70
Android version: 11
App version: 2.3.2

Describe the bug
When scanning some random QR code which is not a vaccination certificate, you get an error message saying "Dieser QR-Code ist kein gültiges Impfzertifikat - VC_PREFIX-INVALID". When clicking on "Details", a StackTrace is shown (probably intentional). To me, a StackTrace containing lines like
 kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:3)

is an information disclosure which in my opinion is a vulnerability and should be avoided.
Steps to reproduce the issue

Click on "Hinzufügen" for "Impfzertifikat hinzufügen"
Scan a random QR code which is not a valid vaccination certificate
In the resulting error message "Fehler - Dieser QR-Code ist kein gültiges Impfzertifikat" click on "DETAILS"
StackTrace will be shown:

Expected behaviour
A human readable explanation should be shown which does not disclose any implementation details.

Internal Tracking-ID: EXPOSUREAPP-7709

@Michael-635 Could you elaborate why this is a disclosure? The code is public on GitHub anyways. BTW AFAIK a StrackTrace like this is shown for all errors.

Well, ok, then I guess it's just my training to never ever show a StackTrace to a user, which of course might not be valid for open source apps.
But still, even if it's not a disclosure, is it acceptable from a user experience perspective to present messages like this without an explanation to an end user, even after pressing details?

@Michael-635
I think you have a very valid point as the app is inconsistent in how it handles QR code scanning errors.

Scan an incorrect QR code with Register Your Test / Scan QR Code and the error message is "QR code is invalid" with no Details / Stack Trace.
Scan an incorrect QR code with Add Vaccination Certificate and the message includes the Details selection which shows the stack trace.

This could be considered an enhancement request to harmonize the error messages when an incorrect QR code is scanned.
There is a related, approved, wishlist item https://github.com/corona-warn-app/cwa-wishlist/issues/343 "Improve response to scanning non-CWA lab QR code".

Well, ok, then I guess it's just my training to never ever show a StackTrace to a user, which of course might not be valid for open source apps.

True; implementation details can't be leaked, because they are already public ;) rather, a stacktrace is useful for debugging, not only for project members, but also for the general public, because they can also contribute fixes!
However, when a specific error is shown that only has one known cause (e.g. QR code incompatible) and is not expected to show up in unexpected places, then it is rather unnecessary to bother the user with this message which is cryptic to them.

FYI: This will be improved with a hot fix: https://github.com/corona-warn-app/cwa-app-android/pull/3400
To me it seems like the stack trace then will not be available to the user anymore.

@Michael-635
Showing the stack trace if fine as a fallback for now. As the other have pointed out, the code is already open, so nothing to worry about regarding information leakage.
The error will be caught in CWA hotfix 2.3.3 which should release soon. If you scan an invalid QR code it will show an explanation and link to an FAQ entry with an explanation.

Corona-Warn-App Open Source Team

Hi, if the hotfix is not released yet, I'd propose to quickly add the same fix to the other error dialog, when scanning the SAME QR code again:

/cc: @chiljamgossow @harambasicluka @mtwalli

PS.: probably it's already enough to take out the "details" in this case here - imho no FAQ entry is needed
cross ref: #3400

@vaubaehn Can you make a new issue for that? :)

@heinezen
I can do, eventhough it's only exchanging the word "wrong" with "same" from the title of this issue ;)
I just wanted to be quick, in case there is a chance to add a fix before 2.3.3 is released...

@vaubaehn Thx :)

@heinezen done: #3413

@Michael-635
Would you like to check that this is fixed for you?
"Hot fix faq invalid vc (EXPOSUREAPP-7709)" PR https://github.com/corona-warn-app/cwa-app-android/pull/3400 was released with the v2.3.3 Android update. In the meantime v2.3.4 has also been released to fix other issues quickly.
If I select

Impfzertifikat hinzufügen
HINZUFÜGEN
using release 2.3.4, and I scan in a false code, I get the following screen shown:

The text "FAQ ZU IMPFZERTIFIKATEN" links directly to an explanation of the error message on https://www.coronawarn.app/de/faq/#vac_cert_invalid (or to https://www.coronawarn.app/en/faq/#vac_cert_invalid for English language users). There is no DETAILS option offered and no stack trace.
I think this resolves your concerns and it is definitely an important improvement in the app during the transition period when most vaccination QR codes are not yet compatible with CWA or the CovPass-App. As time goes on, more of the QR codes issued will be compatible and some pharmacies starting today have the capability to issue compatible certificates. (More detail is availble in the https://www.coronawarn.app/en/faq/#vac_cert_invalid quoted above.)

@MikeMcC399
Yes, it's fixed for me and the issue can be closed.
Thank you!

@Michael-635 @MikeMcC399 @vaubaehn and community. Thanks for reporting and contributing. Closing this issue! Best wishes, DS

Corona-Warn-App Open Source Team


New endpoint: protocol-parameters

TODO:

Add support for the new protocol-parameters endpoint and mark the legacy protocol-params endpoint deprecated.

Completed via https://github.com/maestro-org/haskell-sdk/issues/67.


surface: accept commits with buffer size not divisible by scale

There are still many situations where the buffer scale is not
divisible by scale. The fix will require a tad more work, so
let's just log the client error for now and continue handling
the surface commit as usual.
Closes: https://github.com/swaywm/sway/issues/6352

LGTM


Better bitwise right shift doc

New PR, continuation of https://github.com/google/jax/pull/21892

Looks good, thanks! Please squash changes into a single commit and we should be good to go.

It turns out this introduces test failures in testShiftOpAgainstNumpy for numpy versions older than 2.0. The reason is that the implements decorator sets the __name__ of the wrapped function to the implemented function, and older numpy versions don't have bitwise_right_shift.
So to land this, we'll have to change the test to account for the functions not existing in older NumPy versions.

Ahh, that was the reason. Let me know if this works.


Clarification regarding DVB-S2 multistream operation

Hi all,
Does anybody know if the DVB support in Linux is such that multiplexed DVB-S2 BBFRAME streams with different ISI can be received at the same time?
I understand that the dvb plugin of tsp supports specification of the input stream identifier ISI. However, could it specify multiple ISIs to be received at the same time? And would the kernel/OS support that? Also, is this supported by the DVB-S2 standard at all, that is, the concurrent reception of multiple streams with distinct ISI?
Thanks

This is clearly a question for the Linux DVB mailing list.
That being said, tsp processes a suite of TS packets which collectively create a consistent transport stream. If you receive TS packets from multiple independent streams, how would you rebuild a consistent stream? When receiving a packet with, say, PID 0, which PAT (or part of PAT) is this?
If the kernel allows receiving two independent streams which are demodulated from the same physical source, then the API shall be designed to discriminate between the two (maybe a different dvr device under the same adapter). In all cases, tsp will read only one of them.
This is different from T2-MI for instance where the two streams are not independent. One is inside the other. tsp reads the outer TS and the plugin t2mi extracts the inner TS and replaces  the outer TS with the inner one. But, in no circumstances, one single tsp instance can handle two independent TS.

Thanks, @lelegard.
I'm experimenting with a DVB-S2 system with two types of receivers: Linux USB receivers and software-defined receivers. For the software-defined receiver, I'm experimenting with tsp -P mpe.
I am particularly trying to understand more about the two layers of multiplexing, at MPEG TS level (by PID) and at BBFRAME level (by ISI). The goal is to transmit IP traffic over MPE, within MPEG TS. So the entire encapsulation chain is:
UDP -> IP -> MPE -> MPEG TS -> DVB-S2
On transmit side, I'm transmitting two UDP streams, which are distributed to two distinct MPEG TS and correspondingly DVB-S2 streams. The two schemes I'm exploring are summarised below:
Scheme 1 (same ISI, different PID)
Stream 1:   ISI 1; PID 1;
Stream 2:   ISI 1; PID 2;
Scheme 2 (different ISI, different PID)
Stream 1:   ISI 1; PID 1;
Stream 2:   ISI 2; PID 2;
Now, to the questions.

If you receive TS packets from multiple independent streams, how would you rebuild a consistent stream?

With USB receivers, the approach I'm currently using is to instantiate two dvb net interfaces. Each interface is configured to a different PID.
With the software-defined receiver (SDR) approach, the SDR application decodes the MPEG TS stream and feeds into tsp -P mpe. The stream has both PIDs mixed (multiplexed together), but the mpe plugin successfully extracts the UDP datagrams regardless. I launch it as follows:

-P mpe --pid 32-33 --udp-forward --local-address <IP_ADDRESS> -O drop

Everything works as desired on Scheme 1, i.e. if the ISI is the same. However, there is a specific reason why Scheme 2 (distinct ISI) would be actually advantageous in the system I'm exploring.
Using Scheme 2, the SDR receiver feeding tsp -P mpe should remain OK. Nothing will change from the point of view of the mpe plugin, since it will continue to see the same MPEG TS stream (with multiplexed PIDs).
The real problem is on Linux USB receivers. What I wanted to accomplish in order to make Scheme 2 function is to be able to receive both streams (with distinct ISI) at the same time. This really isn't dependent on tsp. Instead, it depends on drivers and hardware as you suggested. However, my question really comes from the fact that it could be up to the user space application to configure this.
I've been trying to use dvbv5-zap. I tried to set the ISI equal to the NO_STREAM_ID_FILTER , such that all ISIs were allowed to go through at the same time. With one of the devices I'm testing, this did allow two ISIs to go through, but one of the two streams was barely passing (the adapter was probably dropping data). The other stream was with normal rate.
So I tried to explore the tsp -I dvb plugin from tsduck to see if it could configure the demodulator/tuner differently. I asked when I was wondering whether multiple ISI could be configured at the same time. However, by looking at the kernel implementation, it does seem that NO_STREAM_ID_FILTER (i.e. ISI =<PHONE_NUMBER>) is the only possibility of allowing multiple ISI.
I guess I really need to ask on Linux DVB group.
Many thanks for the help.

The central concept here is the "logical consistent transport stream". As soon as you have a consistent TS, TSDuck can work on it.
As far as I understand (which may not be the exact reality), multiple ISI - independent stream identifiers - mean distinct logical TS being transported side by side on the same modulated carrier. Am I right? Any DVB-S2 specialist to confirm or correct me?

However, by looking at the kernel implementation, it does seem that NO_STREAM_ID_FILTER (i.e. ISI =<PHONE_NUMBER>) is the only possibility of allowing multiple ISI.

If I am right, you cannot get two ISI at the same time in the same pipe. Each TS packet with a given PID number belongs to a given TS (in a given ISI) and you need to know which one. Maybe you can demodulate two or more ISI at the same time and send each one to a distinct pipe (the dvr devices on Linux). But in that case, you have several TS sources and each source produces a distinct TS. It you want to use TSDuck on them, then one tsp instance would be needed for each ISI.
Assuming that some external program is able to demodulate 3 ISI on /dev/dvb/adapter0, then maybe the 3 different independent TS would be readable from /dev/dvb/adapter0/dvr0, /dev/dvb/adapter0/dvr1 and /dev/dvb/adapter0/dvr2. But this is purely hypothetical.
Would this be true, then you would need to run three instances of tsp, one on each dvr device.

I guess I really need to ask on Linux DVB group.

Please let us know any valuable information you get from them.
As a comparison, MPE and T2-MI are quite different from multiple ISI. With MPE or T2-MI, the two TS are not independent. One TS - the inner one - is encapsulated inside one PID of the other TS - the outer one. You may have multiple inner TS in different PID's of the outer TS. You may even have several inner TS in the same PID of the outer TS (with different IP addresses on MPE or different PLP on T2-MI).
So, you first demodulate the outer TS and then you decapsulate another TS  from a PID. Using tsp plugins such as mpe or t2mi, you can extract one - and only one - inner TS. You may select to process the inner TS as a replacement of the outer TS or just save it away. But tsp cannot process the two TS at the same time.


editing video error

What version are you running?
21w51a
What went wrong
i tried to do auto-editor and it didnt work
Paste the command you've tried.
auto-editor C:\Users\fleur\Videos\mad.mov

Describe what went wrong
it gave me an error
Paste the terminal output here.
stderr: Input #0, rawvideo, from 'pipe:':
  Duration: N/A, bitrate: 1492992 kb/s
    Stream #0:0: Video: rawvideo (I420 / 0x30323449), yuv420p, 1920x1080, 1492992 kb/s, 60 tbr, 60 tbn, 60 tbc
Stream mapping:
  Stream #0:0 -> #0:0 (rawvideo (native) -> h264 (h264_amf))
[h264_amf @ 000001ce0c5e2e40] DLL amfrt64.dll failed to open
Error initializing output stream 0:0 -- Error while opening encoder for output stream #0:0 - maybe incorrect parameters such as bit_rate, rate, width or height
Conversion failed!

stdout:
Error! FFmpeg Error!

What conditions must be met for it to be wrong
Does the example video work? (if applicable)
What's your platform? (if applicable)
Run: auto-editor --debug
windows 10
Paste output here.
Python Version: 3.9.2 64-bit
Platform: Windows 10 amd64
FFmpeg path: C:\Users\fleur\AppData\Local\Programs\Python\Python39\Lib\site-packages\auto_editor\ffmpeg\Windows\ffmpeg.exe
FFmpeg version: 4.3
Auto-Editor version 21w51a

What properties your input file(s)? (if applicable)
You can use  aeinfo ./path/to/video.mp4 or ffmpeg -hide_banner -i ./path/to/video.mp4 to get standard info.
Paste output here.
- video tracks: 1
   - Track #0
     - codec: h264
     - fps: 60
     - resolution: 1920x1080 (16:9)
     - bitrate: 11524 kb/s
 - audio tracks: 1
   - Track #0
     - codec: aac
     - samplerate: 48000
     - bitrate: 195 kb/s
 - container:
   - duration: 00:09:36.12
   - bitrate: 11723 kb/s

Are you using a headless version of Windows? I've seen that exact error message before in GitHub Actions running Windows.

How do i see if i am?

headless means there's no GUI or any graphics capabilities. Headless versions are often used in servers and in CI/CL systems. You probably have the regular version of Windows.
I would recommend installing FFmpeg then using the --my-ffmpeg flag with the auto-editor command.

headless means there's no GUI or any graphics capabilities. Headless versions are often used in servers and in CI/CL systems. You probably have the regular version of Windows.
I would recommend installing FFmpeg then using the --my-ffmpeg flag with the auto-editor command.

Thanks it worked

Thanks it worked

i cant edit my videos type in ever and it gave me that errorstderr: Input #0, rawvideo, from 'pipe:':
Duration: N/A, bitrate: 1492992 kb/s
Stream #0:0: Video: rawvideo (I420 / 0x30323449), yuv420p, 1920x1080, 1492992 kb/s, 60 tbr, 60 tbn, 60 tbc
Stream mapping:
Stream #0:0 -> #0:0 (rawvideo (native) -> h264 (h264_amf))
[h264_amf @ 000001ce0c5e2e40] DLL amfrt64.dll failed to open
Error initializing output stream 0:0 -- Error while opening encoder for output stream #0:0 - maybe incorrect parameters such as bit_rate, rate, width or height
Conversion failed!
stdout:
Error! FFmpeg Error! and idk what to do to fix it

i cant edit my videos type in ever and it gave me that errorstderr: Input #0, rawvideo, from 'pipe:':
Duration: N/A, bitrate: 1492992 kb/s
Stream #0:0: Video: rawvideo (I420 / 0x30323449), yuv420p, 1920x1080, 1492992 kb/s, 60 tbr, 60 tbn, 60 tbc
Stream mapping:
Stream #0:0 -> #0:0 (rawvideo (native) -> h264 (h264_amf))
[h264_amf @ 000001ce0c5e2e40] DLL amfrt64.dll failed to open
Error initializing output stream 0:0 -- Error while opening encoder for output stream #0:0 - maybe incorrect parameters such as bit_rate, rate, width or height
Conversion failed!
stdout:
Error! FFmpeg Error! and idk what to do to fix it

im confuse

im confuse

if i download that dll where do i put it

if i download that dll where do i put it

You might find this link helpful:
https://github.com/obsproject/obs-amd-encoder/issues/91

Are you using a headless version of Windows? I've seen that exact error message before in GitHub Actions running Windows.

I don't think that's the issue, I had the same problem in my regular windows installation
it may be because of the h264
It was fixed for me when i copied my installation of ffmpeg instead of the one from auto-editor
this is a comparison of my installation of ffmpeg and the one bundled with auto-editor
ffmpeg-compare.txt
and a comparison of ffmpeg.exe -encoders | find "264"
ffmpeg-encoders-compare.txt
this is probably the same issue described in here:
https://johnnn.tech/q/ffmpeg-dll-amfrt64-dll-failed-to-open-on-windows-solved/

I fixed this in 22w03a creating an FFmpeg that is static.

headless means there's no GUI or any graphics capabilities. Headless versions are often used in servers and in CI/CL systems. You probably have the regular version of Windows.
I would recommend installing FFmpeg then using the --my-ffmpeg flag with the auto-editor command.

bro you save my time, thanks


implemented basic planting cursor icon

I timeboxed the effort and implemented a basic plant icon which follows the cursor while planting.
Basics

[x] I added a line to changelog.md
[x] The PR is rebased with current master.

[ ] References to issues, e.g. close #X, are in the commit messages and changelog.
[ ] The buildserver is happy.

Checklist

[ ] I have installed and I am using pre-commit hooks
[ ] I fully described what my PR does in the documentation
[ ] I fixed all affected documentation
[ ] I fixed the introduction tour
[ ] I wrote migrations in a way that they are compatible with already present data
[ ] I fixed all affected decisions
[ ] I added automated tests or a manual test protocol
[ ] I added code comments, logging, and assertions as appropriate
[ ] I translated all strings visible to the user
[ ] I mentioned every code or binary not directly written or done by me in reuse syntax
[ ] I created left-over issues for things that are still to be done
[ ] Code is conforming to our Architecture
[ ] Code is conforming to our Guidelines
[ ] Code is consistent to our Design Decisions
[ ] Exceptions to any guidelines are documented

Review

[ ] I've tested the code
[ ] I've read through the whole code
[ ] I've read through the whole documentation
[ ] I've checked conformity to guidelines

This is perfectly as we imagined! :heart:
Pls also make CI happy etc.

jenkins build please

jenkins build please

@4ydan E2E fail here, too.

E2E tests dont have a cursor, and it seems there is a problem with that.

In general I am skeptical about having any difference during testing, as this makes it much harder to reason about what we actually test. But maybe it is the right thing to do for the cursor.
We could merge it for the release, revert it, and then find a proper solution for the e2e tests?


Couldn't install kubectl-ks correctly via README.md

In README.md, I was trying to execute command hd install -t 8 linuxsuren/ks/kubectl-ks to install k8s and ks, but failed eventually. Full error log as follow:
start to fetch the config
From https://github.com/LinuxSuRen/hd-home
 * branch            master     -> FETCH_HEAD
HEAD is now at 48ee4a8 Add cgit in the package list (#27)
start to download from https://github.com/linuxsuren/ks/releases/latest/download/kubectl-ks-windows-amd64.tar.gz
Error: cannot download from https://github.com/linuxsuren/ks/releases/latest/download/kubectl-ks-windows-amd64.tar.gz, response error: failed to download from 'https://github.com/linuxsuren/ks/releases/latest/download/kubectl-ks-windows-amd64.tar.gz': status code: 404, content length error: <nil>

I found that github releases' detail didn't list any assets which name started with "kubectl-ks".

The README file is outdated. Please use hd install linuxsuren/ks instead.

Already done.


Data enhancement

Hello, I found that the original second code has a data enhancement method that adds noise, but now I don’t find it in second.yaml and kiiti_datase.yaml. Did you delete it?

Object-level data augmentations are removed since they have not contribute to the performance.

thank you for your reply


remove v4

Problem solved
Short description of the bug fixed or feature added

#4627  👈
main

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

Join @jnsdls and the rest of your teammates on  Graphite


Rename & log corrupt tsm files on load

During a run of megacheck the following issues were discovered:
/tmp/232802298/src/github.com/influxdata/influxdb/tsdb/index/inmem/inmem_test.go:127:29: func (*seriesFileWrapper).reopen is unused (U1000)


How do I listen to element events?

Describe the bug
How do I listen to element events?
There appears to be no event handlers on element component
https://stripe.com/docs/js/element/events/on_loaderror
I get 'Unhandled payment Element loaderror' error
Reproduction
How do I listen to element events?
There appears to be no event handlers on element component
https://stripe.com/docs/js/element/events/on_loaderror
I get 'Unhandled payment Element loaderror' error
Severity
serious, but I can work around it
Additional Information
No response

Hi @shkvoretz!
The issue is now resolved.
Here is an example of using the events:
<PaymentElement
  on:loaderstart={e => console.log(e.detail)}
  on:loaderror={e => console.log(e.detail)}
  />

@joshnuss Legend! Thank you! 🙏


Task - dump ProjectSources as BibTex library

Add a simple task that generates a bibliography for all BibTex references in the project
Major consideration is polling for time for longer dumps, perhaps do this as a DelayedJob
Secondary consideration is caching the result

This is largely handled now with the filter, but a full dump may still be useful.

Exists in source filter -> download -> BibTeX.


Documentation for the BAR sample scripts

Hello, i have docs to share.
They contain descriptions for the sample scripts on  https://github.com/IBM/db2-samples/tree/master/BARVendor/ACS_scripts
They were once published on developerworks but got missing in action.
I'm ready to open a PR for this.
Thanks and Regards,
Martin

That's a great idea, @martinjungfer .  Is there anything you need?


Update for Self Hosting Idris2SH?

Any hope of updating your derivation for the current instructions and repo for the installation of idris2 from here: https://github.com/edwinb/Idris2-SH ?
Would love to be able to do it myself on Nixos, but too new and having enough trouble just getting the regular things to work. If you have the time, other newcomers like myself would appreciate it.

Is the request to create a derivation for idris2boot or update to the newest version of idris2?
I'll look into both with the goal to create a derivation for the newest idris2 version. I don't know if it would matter that the build happens with idris2-boot. I'm open to be convinced, though.

I am imposing on your generosity so I don't want be demanding. While working through the type driven development book in idris I noticed that idris is being deprecated in favor of idris2. Seemed like it would be nice to try and use the new version, but I am also a nixos noob and don't feel confident to write the derviation. An update of your derivation to current idris2 would be appreciated, but going forward if idris2-boot is to be the future stand alone version then a derivation for that version might be more useful to more people in the community, eventually. Thanks for considering this request

As far as I can tell, idris2-boot is only useful to compile the code in the idris2 repository. Since there is a way to bootstrap without idris2-boot (by using scheme), it is not strictly necessary. I played around a bit, yesterday, and I couldn't get it to work with the scheme version. Probably there's a bug somewhere, but I'll try it tonight using idris -> idris2-boot -> idris2 if I can't get the scheme version to work.
Meanwhile the best I can offer is the current derivation that gives you an old version of idris2. Maybe you'll be able to work through the first few chapters of the book until I figure out the derivation here.

Alright, I think I got it to work. Edwin upgraded the scheme yesterday night which made the most obvious derivation compile.
Hope that helps; let me know if you run into any issues.

Just confirming it works perfectly. Thanks.


Mysterious "Repeated record added outside of an array" error when inserting BigQuery records including Clojure/Scala Lists

Oops! Pressed RET to fast. Sorry, I'm writing up a description!

I started writing a minimal reproducer and figured out that it wasn't BigQuery anyway. Sorry :)


HDDS-11591. Copy dependencies when building each module

What changes were proposed in this pull request?
(This PR extracts the code change common between HDDS-11450 and HDDS-11588, to make it easier to review.)
When building each Ozone component, its dependencies are listed in a classpath descriptor file, to be used at runtime.
Dependencies are copied to the share/ozone/lib directory only when building the ozone-dist module.  It depends on all other Ozone modules directly or transitively.  Since ozone-dist is a single module, third-party transitive dependency versions are resolved by Maven to a single version.  Only that version is copied to share/ozone/lib.  However, various Ozone components may depend on different versions of the same third-party module.  These different versions can end up in the classpath files, but will not be found at runtime.
The problem is mitigated by the dependency convergence check we already have in place.
This PR fixes the root cause by copying dependencies when building each module.
Further improvements:

move build-classpath execution to the root POM to avoid duplication
copy the classpath files instead of extracting them from module jars
restrict execution to the specific modules for which we need the classpath (controlled by classpath.skip property)

https://issues.apache.org/jira/browse/HDDS-11591
How was this patch tested?
CI:
https://github.com/adoroszlai/ozone/actions/runs/11379873039

@kerneltime this PR addresses your question from HDDS-7372:

since we use classpath files to start services up, wouldn't it be more robust to include both versions vs. forcing convergence?

With this change we'll have the ability to include different versions of the same dependency (at most one per module).  We still have dependency convergence enabled, but we can consider disabling it in the future.

Thanks @kerneltime, @swamirishi for the review.

The change looks good at a high level. I will need to check out the PR and try it out locally. I will try to get to this this week itself.

Please feel free to report any further observations as sub-task of HDDS-11276, I can check in follow-up


Field Configuration

The field configuration is different in the example then the official configuration (like angular-formly, ng-formly). We would like the configuration similar to them.
https://github.com/formly-js/ng2-formly/blob/master/src/core/components/formly.field.config.ts

got it, thanks Mohammed, I'll start working on this soon

Hey Mohammed, can you give me write access to vue-formly please? I just went to create a new branch and it said I don't have the correct access rights.

Can you try now?

Nope, still nothing. I can't even edit anything through the web interface so it's not just something on my machine.

Can you instead fork this project and make a PR?

I changed some settings, can you try now?

Perfect! thanks Mohammed.


Single word change for typo in microsoft-edge-for-business.md

Changes the text from "Also switches automatically swirtches" to "Also switches automatically from". The word 'swirtches' appears to be a typo, but updating to the word 'switches' does not correct the issue. Proposing change to 'from' instead.

Thanks for the PR!  We are currently making some updates to the doc and will make this fix.


Postmaster Limits (Old 283)

In gitlab by @A-Lawrence on Aug 31, 2015, 01:08
Given our outbound SMTP server(s) have upper limits over a month, it's important that we consider how we prevent postmaster breaching these.
One ideas is as follows:

Emails are given a priority (completed in issues #59 and #204).
Postmaster dispatches IMMEDIATE or HIGH priority emails regardless of any other limits.
Postmaster calculates: [Remaining emails in month] / [ Remaining days in month] = DailyLimit
Postmaster then only sends UP TO the DailyLimit of medium or low priority emails (med first, then low).

In gitlab by @A-Lawrence on Aug 31, 2015, 01:13
Added allDispatched scope to postmaster queue.  Refs issue #488449 (283).
ÔåÆ <<cset 143e0de4c847>>
ALawrence66 - 2015-02-27T20:35:20.554491+00:00

In gitlab by @A-Lawrence on Aug 31, 2015, 01:13
A priority queue would be good (so, high priority get sent first, then medium, then low).
The limits must be dynamic for the implementation to be effective, (i.e. your calculation of remaining emails / remaining days).
So basically, the suggestion is perfect, sounds good to me
NFarrington - 2015-02-27T12:23:21.433435+00:00

In gitlab by @A-Lawrence on Aug 31, 2015, 01:13
@NFarrington what do you think?
ALawrence66 - 2015-02-26T22:43:53.249838+00:00

In gitlab by @A-Lawrence on Aug 31, 2015, 01:50
mentioned in issue #303


Added management command to download & cache all Freshdesk tickets.

This change is

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Add support for copying objects with metadata replace

Hi, thanks a lot for S3rver!
Our S3-based application uses object copying with metadata replacement, and we realized that it's not supported by S3rver. I just added the missing support. The relevant S3 operation is PUT Object - Copy, in particular, the x-amz-metadata-directive header.

Hi @MartinSoto  - thank you for your contribution!


3/20 Publish

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-autogen/TOC.md
:white_check_mark:Succeeded
View

docs-ref-autogen/acs.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/acs/dcos.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/acs/kubernetes.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/appservice/web.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/network/application-gateway/url-path-map.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/refTOC.md
:white_check_mark:Succeeded

docs-ref-autogen/sql.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/sql/db.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/sql/db/replica.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/sql/dw.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/sql/elastic-pool.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/sql/server.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/sql/server/firewall-rule.pycliyml
:white_check_mark:Succeeded
View

docs-ref-conceptual/get-started-with-azure-cli.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/install-azure-cli.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Feature/components

In this PR, I:

Set up a new React application using Create React App.
Configured linters for the project.
Created a directory 'components' for my reusable components.
Added React Router Dom.
Set up my router.
Created Navbar component.
Created Categories page.
Created BookCard component.
Created Form component.
Updated README.md file.
Added LICENSE file

Thank you @Qoosim 😊

You're most welcome @hansderly :handshake:


Content-Disposition info for attachments missed

Without this we can't determinate is attachment really attachment or it must be inlined. Ofc we have Content-Id but it may be availible also for not-inline attachments.
anyway thank you guys for awesome parser!

Hi @byaka,
I will release this change in next version.

Awesome, thx!