Speaker automatically selected on callkit

If it is an audio call then it should use iPhone's built-in receiver (output).
How to make it possible?
https://stackoverflow.com/questions/50502518/issue-with-opentok-and-callkit

@mrazam110 thanks for this concise report. We'll work on that change.

Hi, any updates on this?

Hello, everyone :)
If you are using this example in your app and experiencing issue with speakerphone instead of headset, you need to do some changes in the custom audio driver - OTDefaultAudioDevice.

Remove from audio session setup - setupAudioSession and setupAudioSession:(AVAudioSession *)audioSession setting of default speaker option:

audioOptions |= AVAudioSessionCategoryOptionDefaultToSpeaker;

In same session setup methods change audio session mode from
Video

[mySession setMode:AVAudioSessionModeVideoChat error:nil];

to Voice
[mySession setMode:AVAudioSessionModeVoiceChat error:nil];

If you need to turn on headset by default (after previous steps it should be chosen by default, but if not...) you can force set it manually by calling

[self configureAudioSessionWithDesiredAudioRoute:AUDIO_DEVICE_HEADSET];

You can do this also in the setupAudioSession or before creation and connection to the OpenTok session.
Here is how setupAudioSession looks like after changes:
- (void) setupAudioSession
{
    AVAudioSession *mySession = [AVAudioSession sharedInstance];
    _previousAVAudioSessionCategory = mySession.category;
    avAudioSessionMode = mySession.mode;
    avAudioSessionPreffSampleRate = mySession.preferredSampleRate;
    avAudioSessionChannels = mySession.inputNumberOfChannels;

    [mySession setPreferredSampleRate: kSampleRate error: nil];
    [mySession setPreferredInputNumberOfChannels:1 error:nil];
    [mySession setPreferredIOBufferDuration:kPreferredIOBufferDuration
                                      error:nil];

    NSError *error = nil;
    NSUInteger audioOptions = 0;
#if !(TARGET_OS_TV)
    audioOptions |= AVAudioSessionCategoryOptionAllowBluetooth ;
    [mySession setCategory:AVAudioSessionCategoryPlayAndRecord
               withOptions:audioOptions
                     error:&error];
#else
    [mySession setCategory:AVAudioSessionCategoryPlayback
               withOptions:audioOptions
                     error:&error];
#endif

    [mySession setMode:AVAudioSessionModeVoiceChat error:nil];

    if (error)
        OT_AUDIO_DEBUG(@"Audiosession setCategory %@",error);

    error = nil;

    [self setupListenerBlocks];
    [self configureAudioSessionWithDesiredAudioRoute:AUDIO_DEVICE_HEADSET];
    [mySession setActive:YES error:&error];

    if (error)
        OT_AUDIO_DEBUG(@"Audiosession setActive %@",error);

}

Don't forget to pre-heat audio session before starting CallKit's AudioSession as TokBox provided in example - in CXStartCallAction and CXAnswerCallAction

Minor explanation why it's all happening with us :)

When Audio Session configured with AVAudioSessionCategoryOptionDefaultToSpeaker it's obviously uses speaker by default :)
When you choose AVAudioSessionModeVideoChat mode, session obviously (for Apple :D) uses speaker by default and you can't switch because... it's Apple's logic, I suppose.
And if session is configured with AVAudioSessionModeVoiceChat mode, iOS and CallKit behaves as voice call with ability yo switch between speakers (and also turning screen off when user brings phone to ear).

BTW, from Apples documentation:
/* Only valid with kAudioSessionCategory_PlayAndRecord. Reduces the number of allowable audio
routes to be only those that are appropriate for video chat applications. May engage appropriate
system-supplied signal processing.  **Has the side effect of setting
AVAudioSessionCategoryOptionAllowBluetooth and AVAudioSessionCategoryOptionDefaultToSpeaker.** */
AVF_EXPORT AVAudioSessionMode const AVAudioSessionModeVideoChat API_AVAILABLE(ios(7.0), watchos(2.0), tvos(9.0)) API_UNAVAILABLE(macos);

And also, from documentation, Apple suggests to use AVAudioSessionModeVoiceChat for both audio and video calls.


Update TupleParentheses usage

Summary
Implements a potentially more re-usable naming convention for TupleParentheses.
Subscript case  -> Preserve - "Preserve the tuple's parentheses if they are there"
SkipInsideLambda case  -> Never - "Parentheses are never needed with this parent"  (#5806; comment)
Comprehension case -> Never - "Parentheses are never needed with this parent" (#5790; comment)
StripInsideForLoop case -> NeverPreserve - "Parentheses are never preserved in special cases" (I interpret "Preserve" as "not always necessary but keep them anyway")
The counter to this I'd think is that there may be benefit to having node-specific differences.
Test Plan
Existing snapshots.

Questions

Where would TupleParentheses::Expr(...) be helpful in some of these cases (if at all)? (answered)

Related:

Handle parenthesized Argument

Really appreciate all of this direction. I'm going to wrap up #5806 and then #5790. Then I can come back here with everything you need to see if this is even something that's worth merging.

I'm not in love with NeverParentheses. I'm going to think a bit more about some good variant names.

I'm not in love with TupleParentheses::NeverPreserve. I'm going to think a bit more about some good variant names.

@konstin likes the name a lot. I believe it has the same semantics as Parenthesize::IfBreaks (which naming I find okay, but not good)

Thank you. Enabling auto merge. Hopefully, this flushes through.


ARROW-8005: [Tools] Update apache mirror links

Change urls from www*.apache.org/dist to downloads.apache.org.

@fsaintjacques seems like we have more occurrences, could you please update them as well?

I believe that it only applies to the www* sub domain.

Yeah, I think this only applies to verification of keys and checksums of releases, so it's fine for us to keep using https://dist.apache.org/repos/dist/dev/arrow in our scripts


Run tobiko pod as root with NET_ADMIN,NET_RAW capabilities

Tobiko requires to run as root in order to be able to run tcpdump command and needs NET_ADMIN and NET_RAW capabilities. Tcpdump command is currently being used by only one test in tobiko:

tobiko/tests/scenario/neutron/test_qos.py::QoSNetworkTest::test_ping_dscp

/lgtm


ci: Configure Dependency CI

Configures DependencyCI to ignore flaky maven checks.
This allows DependencyCI status to be re-enabled and to provide meaningful results.

Contributor License Agreement adherence:

[x] This Contribution is under the terms of Individual Contributor License Agreements (and also Corporate Contributor License Agreements to the extent applicable) appearing in the Apereo CLA roster.

:construction: :white_check_mark: Example passing build https://dependencyci.com/github/ChristianMurphy/angularjs-portal/builds/4


fix: use provider credentials if they are set in extractClusterApiKeyAndApiSecret

Current behavior:
With the following API Key based provider configuration:
# Option #1: Manage multiple clusters in the same Terraform workspace
provider "confluent" {
  cloud_api_key    = var.confluent_cloud_api_key    # optionally use CONFLUENT_CLOUD_API_KEY env var
  cloud_api_secret = var.confluent_cloud_api_secret # optionally use CONFLUENT_CLOUD_API_SECRET env var
}

CRUDing a KafkaACL results in the KafkaACL's referenced credentials for authentication instead of the provider block's configured credentials.
Expected behavior:
I would expect the creation, reading, deletion, or update of a kafka_acl to use the provider's configured APIKey/secret defined in the provider "confluent { .. } block.
However, because isKafkaMetadataSet checks for the kafka_rest_endpoint to be set in order for the provider's APIkey/secret to be used for CRUD of a kafka_acl, this is not happening. Instead, the CRUD operations of a KafkaACL are using the API Key referenced in the kafka_acl's credentials block. Essentially this block is executed instead of this one.
This PR changes the conditional block so that the api key and secret defined in the provider's configuration block are used for CRUD operations instead of the APIKey referenced in the KafkaACL's credentials block.
fixes #329

@linouk23 Let me know if this change makes sense to you, or if you have any questions/concerns regarding the change. Thanks!


Add skip minification block

A html comment. It would fix very odd edge cases.

With the rewrite I've instead solved the root to the problem. <textarea tags within <textarea> tags will not be minified for example.


T2 submission

confirmed:T2
account1:441fdk62aLbDmE8ggkbqvNXfQ6nsWbFx9fifn2EXqF3T8KKwbC
account2:3M8XZJA1arZV42zy1x9UtxJeW8ri3r6mqPSwVcgU1jZeqX5Y5q

confirmed:T2
account1:441fdk62aLbDmE8ggkbqvNXfQ6nsWbFx9fifn2EXqF3T8KKwbC
account2:3M8XZJA1arZV42zy1x9UtxJeW8ri3r6mqPSwVcgU1jZeqX5Y5q


riscv: b9x: save bin size for dlm.

seperate dlm_bss and dlm_data to save bin.
load dlm_data and reset dlm_bss .

it only have the B9X soc family, so if you finish TLX , you can modify the code just like what i did in B9X , or later , you can just assign to me if you fin

Apply your new PR into next Andriy's TLX branch (based from develop_new) https://github.com/telink-semi/zephyr/tree/tlx


Toolbar overlay slider

Toolbar (http://prismjs.com/plugins/toolbar/) overlay slider in Firefox and Chrome.
Also, there should be a space between two buttons in toolbar, and the toolbar should be more visible.

How wide is the scroll bar? I'm on a Mac so I can't test, but we should be able to shift that over a little.

I'd say ~15-20px.

Yes. And there should be space between two buttons.

Hi team, Does this issue still need to be fixed?
*  gap between buttons
*  fix scroll bar overlay

@hemmingson Yes. If you are interested in fixing this, go for it.
#2138 might be a good starting point. I tried to fix the "scroll bar overlay" issue but it didn't go too well. Maybe it helps.

Hi, I would like to take a shot at this issue, if it is still relevant.

@RunDevelopment hi, what do you say to this alternative

How's this? Do let me know there's anything else.

P.S: I am interested in this issue.

is this still open i would like to resolve it?

Can you please assign it to me?


2x slower with k=18

I've done some first a bit complex benchmark for one of my DBs for diagnostics of resistance (~661 executions of ProphAsm).
ProphAsm2 is really fast!!!
However, I noticed that from some reason, with k=18 it's approx 2x slower compared to k=31. Is there any reason for this? With ProphAsm 1, I didn't observe this.
ProphAsm, k18

real	7m54.141s
user	43m45.501s
sys	1m28.739s

ProphAsm, k31

real	8m29.515s
user	48m24.024s
sys	1m25.909s

ProphAsm 2, k18

real	4m35.686s
user	21m11.361s
sys	1m44.798s

ProphAsm 2, k31

real	2m51.831s
user	11m32.846s
sys	1m25.608s

However, I noticed that from some reason, with k=18 it's approx 2x slower compared to k=31. Is there any reason for this?

Interesting. My guess is that this is because of the dBG structure (much more branching / disconnected for $k=18$). Actually what is the dataset you're running it on?

Would be interesting to see the number of simplitigs for $k=18$ and $k=31$. Not sure what else could cause such a big difference...

This doesn’t explain. In case of ProphAsm 1 there was no such effect.
Also,we  dont construct dbg explicitely so dbg topology should play no role.
Maybe an effect of deletions in khash? This could in theory explain that….
(If answers for previously deleted kmers are slow)
Should be reproducible with two pneumococcal genomes, while computing
intersection and both sym diffs
On Thu, Mar 14, 2024 at 5:36 PM Pavel Vesely @.***>
wrote:

Would be interesting to see the number of simplitigs for $k=18$ and $k=31$.
Not sure what else could cause such a big difference...
—
Reply to this email directly, view it on GitHub
https://github.com/prophyle/prophasm2/issues/12#issuecomment-1997868273,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACC7KJR7PGFZT7JJNNNFQJTYYHGXDAVCNFSM6AAAAABEV5OGTSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJXHA3DQMRXGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Never seems to become active

I installed the part manager using the "Add new" button on the CKAN Plugins page.  Then I hit the Activate button.
Something happened, but I have no idea what.  It starts showing a tab which says "Installed mods that contain parts", but nothing was listed.
I then tried to unload it, but the tab remained.  It even remained after I restarted.  I had to find where the dll was copied to (the Plugins directory) and delete it manually.
I'm not sure if this is a PartManager issue or a CKAN issue, I'm going to post this in the CKAN issues as well.

I have the same issue.  Part Manager 1.2 CKAN 1.16.0

I believe that after CKAN's registry system change this pkugin stopped working


TargetsdkVersion > 23 and ANIMATABLE_XWALK_VIEW  cause a Build Crash

I'm using Webview as Textureview ((XwalkPreferences.setValue(XWalkPreferences.ANIMATABLE_XWALK_VIEW, true); flag) but I'm not able to build project with targetSdkVersion > 23.
Caused by: java.lang.RuntimeException: java.lang.UnsupportedOperationException: TextureView doesn't support displaying a background drawable         at org.xwalk.core.ReflectConstructor.newInstance(ReflectConstructor.java:54)         at org.xwalk.core.XWalkView.reflectionInit(XWalkView.java:2082)         at org.xwalk.core.XWalkView.<init>(XWalkView.java:261)
(without this flag, Webview is SurfaceView - it's hard to create bitmap from it)
I need this flag to grab screenshot of Webview. Is there other way to grab screenshot and build with target > 23 ?

Ok, so I used Surfaceview (ANIMATABLE_XWALK_VIEW=false) and create new API method called captureBitmapAsync. For all of you, check my fork:
https://github.com/Krbz/react-native-crosswalk-webview-plus/commit/f5a9b28f82b7da957547fea55a2cb7d137d42dd1


can you support item2 version 3.0.0

When i upgrade my iterm2 to 3.0.0, the itermocil not word any more.
The error shows:

This is an unsupported beta build of iTerm.
Try the latest nightly, or the 2.1.1 stable build.
See Readme notes for more info. Sorry!

Can you make your itermocil support iterm2 3.0.0?

This was resolved in #44. You will need to update itermocil.


[SPARK-39134][SS] Add custom metric of skipped null values for stream join operator

What changes were proposed in this pull request?

This proposes to add a custom metric of skipped null values for stream-stream join operator.
Why are the changes needed?

When STATE_STORE_SKIP_NULLS_FOR_STREAM_STREAM_JOINS is enabled, Spark will skip null values in hash-based stream-stream joins. But currently we cannot track if this is happened and how many it is. For our customers, they have an interest to know the number of values skipped for better tracking purpose.
Does this PR introduce any user-facing change?

Yes. One custom metric is added to stream-stream join if the config is enabled. As the config is disabled by default, it is 'no' for users by default.
How was this patch tested?

Updated current test.

Re-ran all CI tasks and they were passed.

cc @HeartSaVioR @dongjoon-hyun @sunchao

LGTM

Thank you @dongjoon-hyun @anishshri-db

Thanks @HeartSaVioR !

Merging to master.


Development: Fix a date conversion error and quiz cypress tests

Checklist
General

[ ] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines and ensured that the layout is responsive.
[x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[x] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[x] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

Motivation and Context
Fixes the Cypress tests that were failing for quiz exercises.
Description
Small bugfix in complaint service that flooded console with errors and caused component not to render.
Steps for Testing

Prerequisites:

1 Instructor
1 Student

Log in to Artemis
Navigate to Course Administration

Review Progress
Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Test Coverage

Class/File
Branch
Line

Screenshots

I will merge this now as important bugfix, additional improvements can be made in a follow-up PR


Implement dispatcher stream id in the WebSocket protocol.

Description:
The purpose of this is to address the issue where, if two custom remote functions return the same message type, the generated AsyncAPI client cannot determine which response corresponds to which request. So the plan was to do the validation at the compile time. To give an error if two remote functions have the same return type and there isn't a dispatcher stream defined.

Validation at compile time doesn't work in this case. The service configuration annotations are defined in the first service, which is dispatched for the initial HTTP request. However, the actual remote functions that are dispatched after the connection upgrades to WebSocket reside in the second service. Therefore, validation should occur at the second service. To verify if the user has defined dispatcherStreamId, the service configuration should be accessible at the second service level, but this is not possible.


fix: add # to regex to match text before cursor

Thanks for this.  It is unrelated to #139, but it needs to be fixed anyway.


feat: Pass the resource key role as a module input (LogDNA)

Description
Pass the resource key role as a module input
https://github.com/terraform-ibm-modules/terraform-ibm-observability-instances/issues/206
Types of changes in this PR
No release required

[ ] Examples or tests (addition or updates of examples or tests)
[ ] Documentation update
[ ] CI-related update (pipeline, etc.)
[ ] Other changes that don't affect Terraform code

Release required

[ ] Bug fix (patch release (x.x.X): Change that fixes an issue and is compatible with earlier versions)
[ ] New feature (minor release (x.X.x): Change that adds functionality and is compatible with earlier versions)
[ ] Breaking change (major release (X.x.x): Change that is likely incompatible with previous versions)

Release notes content
Replace this text with information that users need to know about the bug fixes, features, and breaking changes. This information helps the merger write the commit message that is published in the release notes for the module.

Checklist for reviewers

[ ] The PR references a GitHub issue.
[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

Merge by using "Squash and merge".

Use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author.
The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).

@jor2 - Hi Jordan, this one got closed without being merged - is this expected?


Active Mode

Add active mode, where the DPAE is in-line with the flow until a classification determination is made. This requires the DPAE to be able to send packets back to the switch, and the switch pipeline to skip learning and classification for these packets. Needs a policy directive to choose active vs passive mode

Closed by release 0.2.0


Duplicate rows when joining two joined streams

Describe the bug
I wrote a pipeline which reads data from 4 ports, joins two streams at a time into one stream, and then joins the two resulting streams again. When fetching the resulting rows, each row is read up to four times.
To Reproduce

Checkout https://github.com/AntonOellerer/SpringQL/tree/bug/duplicate_rows_when_joining_joined
Run cargo run --example test_join

Expected behavior
Each timestep one row with the results from the 3 joins is read
Actual behavior
Each timestep up to 4 rows with the results from the 3 joins are read (often also with one column missing)
e.g.:
[motor_average_readings]        2022-01-01 13:00:00.000000000   5       -1      15      19
[motor_average_readings]        2022-01-01 13:00:00.000000000   5       -1      15      19
[motor_average_readings]        2022-01-01 13:00:10.000000000   50      -1      70      -1
[motor_average_readings]        2022-01-01 13:00:10.000000000   50      -1      70      -1
[motor_average_readings]        2022-01-01 13:00:10.000000000   50      -1      70      -1
[motor_average_readings]        2022-01-01 13:00:10.000000000   50      -1      70      -1
[motor_average_readings]        2022-01-01 13:00:30.000000000   15      36      17      -1
[motor_average_readings]        2022-01-01 13:00:30.000000000   15      36      17      -1
[motor_average_readings]        2022-01-01 13:00:30.000000000   15      36      17      -1
[motor_average_readings]        2022-01-01 13:00:30.000000000   15      36      17      -1

Environment
SpringQL

springql-core: 0.18.1

your machine

Device: Tuxedo Pulse 15
OS: Fedora 37

Additional context

Kernel: 6.1.15
rustc: 1.68.0

There generally seems to be a problem with the output produced by joins.
When looking at the springql-web-console, it seems as if each pump produces far more messages than necessary:

In this example, each sink received one message per second for 30 seconds


Scanning your network hang on

I've been getting this issue lately
root@Retr0:~/kickthemout# ./kickthemout.py
SCanning your network, hang on...
ERROR: Default Gateway MAC Address could not be obtained. Please enter MAC manually.
kickthemout> Enter your gateway's MAC Address (MM:MM:MM:SS:SS:SS): ScAnning your^Canning your network, hang On...
Thanks for dropping by.
Catch ya later!
i pressed control c to stop it. It seems to go on forever and i couldn't type the mac address..
Please help.

I'm getting the same issue.
`sudo python kickthemout.py
Scanning your network, hAng on...
ERROR: Default Gateway MAC Address could not be obtained. Please enter MAC manually.
^Canning your network, hang oN... MAC Address (MM:MM:MM:SS:SS:SS): Scanning your network, haNg on...
`
What details do you need? I'm more than happy to help.
Cheers

Hi there @barthol0. Please create a new issue and include the information in the issue template. Thanks!

Closing this due to inactivity.


enh/issue: performance optimisation

Is your feature request related to a problem? Please describe.
The performance of long chat threads is really bad. I don't know if it is a browser client side issue or an issue on the server side.
Describe the solution you'd like
A performance fix would be great, but even the ability to do a shallow clone where only the last context window or so of the chat is cloned would help.
Describe alternatives you've considered
I am resorting to starting a fresh chat with memory items added and trying to use the prompt to pick up where the last chat left off.
Additional context
I have tried a few different browsers, but the best fix so far has been to start a new chat and try my best to get it back on roughly the same track.

If you could share an export of your chat so that we can diagnose which part of the UI is being the bottleneck, that would be sublime!

I generated a chat large enough to trigger the same behavior since it would be a pain to organically create one I am fine with sharing big enough just for this, and have included it along with the generator.py file (as a .txt since uploading .py isn't allowed) in case you want to play around with different generated content and sizes.
generated.json
generate.py.txt
To reproduce the behavior I imported the chat and sent a message through the UI. The client side browser locked up for a bit, but after waiting it will eventually continue and display the new message UI and eventually fill in the assistant message.
The performance is much, much better with a shorter chat. Let me know if that doesn't work for you, or you need more information.

Encountered similar issues, I controlled the token generation for each dialogue to only come from the cache, while cutting the chat context length to 0 to prevent speed issues with LLM serving.
However, in long threads, the interface lag was very noticeable.
When page streaming output occurred, the page became unresponsive and uncontrollable.

PR Welcome

Just to add to this. I don't know if this might be browser specific. I see horrible lockups and slow downs in Safari on macOS but not in Edge on the same machine.

Not sure if this really helps?

I'm experiencing performance issues in Edge, Chrome, and Arc Browser on macOS, specifically when the chatbot has long conversations. These issues occur particularly when the LLM is responding, which might be related to how the front end handles streaming chunks of data. Additionally, when using Arc Browser on iOS, the browser frequently crashes under similar circumstances, and I have to manually force close the app for it to work again.

In my case I get one core on 100% on the safari tab, seemingly consumed by a JavaScript process, but the profiler doesn't tell me what is using all that cpu time.

Same here, Chrome on ChromeOS Flex version 129. It gets very slow as the chat length grows. I often get a prompt from chrome asking me if I want to kill the page or wait for it to finish.

I've carried on stress-testing Edge on macOS which has slightly better profiling and found slowdown, but not quite as extreme as Safari. Note the following:

Note the "animation time" which is the ui's placeholder box after hitting the Send message button. I'm using Groq here, so the thinking time is too long for it to be groq's fault. Rendering the response uses up quite a lot of CPU, which is articulated in more detail here:

Hope this helps

I completely agree with the previous comments regarding the lag in Open-WebUI, especially with long chat threads. The slowness and high CPU usage make the experience quite frustrating. I've found that restarting Firefox seems to help temporarily, but it's inconvenient. It's good to know this isn't just a Firefox issue. Hopefully, this will be resolved soon, as I sometimes even switch to using Firefox on my phone to continue chatting when the lag becomes too severe.

I'll start my thorough investigation shortly but I'd greatly appreciate some help from the community here!

An interesting observation is that in the latest version 0.3.22, there isn't a noticeable lag or blocking in page interactions with long threads, but the model token output still significantly slows down to ~5 token/s.

I upgraded to 0.3.22 also but I still think long threads are a problem, I've looked at Firefox / devtools / console and there seems to be a awful lot of logging happening in there, but I am not a developer so maybe that's normal.
What I have discovered is that if I go to Open-WebUI settings and set "Stream Chat Response" to "off" is that I still get the reply but it's just in a single dump but my cpu is now idle and I don't get any lock ups. Only down side is you don't get to see the response output in real time. This might be a work around for some ?

hey tjbck, just tell me what you need, I can compile/test/send reports as much as you need.

pagination/lazyloading may be a general optimization here

I've found it's parsing the whole chat history into markdown every time when a new token arrives.

Hi I'm coming from #4881 and wanted to add that actually I think it might be every time a token is rendered and not received. As from my very limited testing it seems that turning off "Fluidly stream large external response chunks" while mermaid charts are rendered seems to make the generation noticeably faster.

Testing wanted in latest dev! Several optimisation has been applied to message rendering process (e.g. infinite scrolling), so significant performance gain is expected here.

After testing git-822c47c, I noticed a significant improvement. The chat experience feels much smoother, especially in longer conversations. Considering I was using Arc Browser on iOS, the lag has almost completely disappeared. It was previously unusable, so this is a major enhancement.

So this fixed my mermaid issue as completions are now just as fast as regular text but I'm suspecting a regression when dealing with vision models. I've tested it with a local Ollama model as well as 4o from OpenAI and Cloud Sonnet. And in all cases, the generation is suspiciously slow, just like for the mermaid before the fix. This was on Brave Browser and on 0.3.26. Am I the only one?

After testing git-822c47c, I noticed a significant improvement. The chat experience feels much smoother, especially in longer conversations. Considering I was using Arc Browser on iOS, the lag has almost completely disappeared. It was previously unusable, so this is a major enhancement.

Same for me. Experience in Arc on iOS improved x100 🥳

Hi @tjbck, thanks for resolving this out, it runs perfect with Chromium based browsers, but the issue still exist on Safari, for both Mac and iPhone.

Hi @tjbck, thanks for resolving this out, it runs perfect with Chromium based browsers, but the issue still exist on Safari, for both Mac and iPhone.

I can confirm the issue still arises on Safari. After long chats or prompts with much context, the page hangs until the answer is streamed. Switching between chats with much chats/context is very slow as well.
Please let me know what I can do to help triage the issue on Safari.


add dependency version control

See discussion started on the mailing list
Currently the SuperBuild uses master at our CCPPETMR forks of dependencies such as STIR and Gadgetron. This means that it is impossible to go back to a specific version with all dependencies that same as when we released.
One possibility is to create a SIRF/sirf_version_config.cmake (recording some git-tags) as that'd would be version tracked. A difficulty with this is that the Superbuild clones/builds dependencies first, and hence doesn't have SIRF and so its version info yet. I think it's therefore better to keep this version_config.cmake in the Superbuild. SIRF's CMake files would just have to set minimum requirements as usual (and would conform to how our external dependencies Gadgetron are treated).
However, hard-wiring specific SIRF git-hashes makes it inconvenient for developers. On the other hand, setting it to master means we cannot rely on this to get a specific version after all. (we could have a release branch that modifies the version_config.cmake, but that will create complications/conflicts etc). Maybe we could let these be overridden by CMake variables?
Not sure what the best option is.

I think there are a few options:

have SuperBuild/version_config.cmake and SIRF/version_config.cmake

superbuild refers to SIRF tag/commit hash. Slightly inconvenient for developers
SIRF refers to its own dependencies
SIRF CI triggered by pushes to SIRF:master
SuperBuild/VM CI triggered by pushes to SuperBuild:master

have SuperBuild/version_config.cmake

superbuild oversteps itself a bit by referring to tags/commits for SIRF and all sub-dependencies.
SIRF CI triggered by pushes to SIRF:master
SuperBuild/VM CI triggered by pushes to SuperBuild:master

have SIRF/version_config.cmake

superbuild always refers to SIRF:master, but exposes this so the end-user can change it to a certain SIRF tag/commit.
SIRF and SuperBuild/VM CI triggered by pushes to SIRF:master
SuperBuild/VM CI also triggered by pushes to SuperBuild:master

I vote for 3.

This was discussed at the Software tcon 15 June 2017 (minutes to get into the website). Brief summary:

A general preference was felt for option 3 presented by Casper (specify git hashes in SIRF/version_config.cmake), however this conflicts with the current SuperBuild mechanism which builds STIR etc before it even clones SIRF. Option 2 (SuperBuild/version_config.cmake) is therefore our recommended solution.

SIRF will use normal CMake mechanisms to ensure minimum versions of dependencies but not specify git hashes of dependencies.

The developer can override the tags by specifying variables, and we also provide a “developer flag” that just uses “master” for everything.

version_config.cmake file might look like this
Boost_VERSION=1_63_0
Boost_MD5=dsfsf

option(DEVEL_BUILD "Use current versions of major packages" OFF)

if (DEVEL_BUILD)
default_SIRF_GIT_TAG=master
else()
default_SIRF_GIT_TAG=v0.9.0
endif()
option(GIT_TAG_SIRF "git tag for SIRF" ${default_SIRF_GIT_TAG})

Do we have a list of required versions for all the dependencies?
I found the following.
Armadillo => 7.800.2 (release)
Boost => 1.63 (release)
FFTW3 => 3.3.5 (release)
Gadgetron => ?? (git)
googletest (rename to Gtest)
HDF5 => 1.10.0-patch1 (release)
ISMRMRD => ?? (git)
SIRF => release-0.9.0 (git)
STIR => ?? (git)
SWIG => 3.0.12 (release)
So I just miss STIR, Gadgetron and ISMRMRD.

would be closed with https://github.com/CCPPETMR/SIRF-SuperBuild/pull/29

@paskino, it seems we need a few more changes after PR #29. Current version_config.cmake has
if (DEVEL_BUILD)
set (SIRF_TAG master)
else()
set(SIRF_TAG v0.9.0)
endif()
option(GIT_TAG_SIRF "git tag for SIRF" ${default_SIRF_GIT_TAG})

This means that the option GIT_TAG_SIRF actually doesn't do anything. The first few lines need to set the default tag, the option then needs to set the one to be actually used (presumably SIRF_TAG).
Also, I believe we need these options for STIR, Gadgetron and ISMRMRD as well. Otherwise we can not use the superbuild to test with a newer version. I'm not 100% sure what the prefered default would be for DEVEL_BUILD. We could either use master or keep it at the "stable" git tag. @evgueni-ovtchinnikov, what would you prefer?
Currently we set the git repos to the CCPPETMR forks of the above. That prevents a developer from using a version that isn't merged into the fork. It was suggested above to use the original repos. (commit-hashes should remain the same). To make this entirely flexible, we should make the URLs into options as well. I think it'd be fine to just say
option(STIR_URL "git repository for STIR" https://github.com/UCL/STIR)

Finally, I think we should use mark_as_advanced on all these options, except for the DEVEL_BUILD option.

I totally agree with you.
I understand the DEVEL_BUILD option would make us use the latest release (master).
But I still do not understand what is the meaning of the GIT_TAG_SIRF, and default_SIRF_GIT_TAG, which aren't used anywhere.
I can clearly add the project url as advanced variable to be set here. Is this what you mean?

I created a new branch (version_control) and with https://github.com/CCPPETMR/SIRF-SuperBuild/commit/e66e17c48ceea824d9a9d4d4b0bb536d4b205ced it is now possible to use the option DEVEL_BUILD to set the URL/TAG of Gadgetron, Stir and ISMRMRD to upstream/master. I've tested on my VM and it builds on travis.
I wait the merge of the https://github.com/CCPPETMR/SIRF-SuperBuild/pull/32 before issuing a PR from version_control branch, as it branches from that.

Fixed by PR #36


allow for identity POST_TRANSFORM

In https://github.com/ludwig-ai/ludwig, we require access to the raw logits since we already have our own "post-transform" logic. With this PR, a user can predict raw logits by using the following extra_config:
from hummingbird.ml import convert
from hummingbird.ml.operator_converters import constants

lgbm_model = ...
convert(lgbm_model, "torch", extra_config={constants.POST_TRANSFORM: None})

@ksaur would you have bandwidth to review this small PR? Many thanks!

@microsoft-github-policy-service agree company="Predibase, Inc."


BeatAlign

Hi Ruilong,
I was wondering whether the kinematic beats are computed based on the whole body joints?
I cannot find these codes.

@YIN95 Hello, Wenjie, did you find the corresponding code?

@YIN95  @wang-zm18  I have same question, where does the kinetic velocity come from?

Hi, please refer to the code here for beat metric calculation:
https://github.com/liruilong940607/mint/blob/main/tools/calculate_beat_scores.py
Sorry that I forgot to merge updates from my personal branch to this official repo. I just created a merge request but would need to wait for my collaborators to approve it.

Hi Ruilong, where can I download the data in '/mnt/data/aist_paper_results/*.pkl'?


Dashboard page 1.1

Basic Dashboard page with a navbar and file upload

Looks good, you can merge.


dictionary takes input from the user and gives the word meaning

refer to issue#251

refer to issue#251

Nice work @AmmyK369! But there are a few changes to be made before I merge your Pull Request:

Rename the dictionary folder to dictionary_python.
Rename the read.me file to readme.md.
The name of your code file is too log, which is not professional. Try keeping it short and simple yet understandable. Something like word_meanings.py would do good.

Make these changes and then your Pull Request will be good to merge.

@AmmyK369 it's actually readme.md but you named it as readme.me. I'll take care of it for this time, but keep this in mind for future contributions. "md" file refers to markdown file.


Fix possible issues with Gamma?

It seems as if Konami messed up the gamma levels of MGS2 and 3 when porting it over to The Master Collection.  Possibly add in a fix for this?  Seems like it would be easy to do so.
Examples at 1:17 and 4:24, not the most professional I know, but it's my first github post.
https://www.youtube.com/watch?v=gNBuXuHr6qs

This mod is a reshde preset that fixes the gamma ideally something like this could be added as an ini toggle
https://www.nexusmods.com/metalgearsolidmastercollection/mods/22

Updated link to the gamma mod:
https://www.nexusmods.com/metalgearsolid3mc/mods/29
It also works perfectly for MGS2.

Kind of a dupe of #58. @ShizCalev's mod looks like the way to go


Version 2.3.2

Landed in https://github.com/metarhia/impress/commit/1eb421121ef965ae614b86a64456300156484ca1


Only print the last component of the filename

In particular, on Windows MSVC, the ASSERT macros use __FILE__ (with the full pathname) instead of __FILE_NAME__ (with only the last component).

@swift-ci Please test

@swift-ci Please smoke test


⚠ Bofrost.at has degraded performance

In e4325c9, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1264 ms

Resolved: Bofrost.at performance has improved in 5c99cf3 after 36 minutes.


Error when deleting files

Description
Received a error message when deleting 1 .mph and 3 .jpg files
Steps To Reproduce
Selected 4 files and hit the delete key
Expected behavior
Files deleted with no errors
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.19044.1586
Windows Version
Device name	DW7B3B3 Full device name	DW7B3B3.meansindustries.local Processor	11th Gen Intel(R) Core(TM) i7-1185G7 @ 3.00GHz   1.80 GHz Installed RAM	16.0 GB (15.7 GB usable) Device ID	210381D3-6B61-43D8-BD27-048480AFD152 Product ID	00331-20160-00000-AA055 System type	64-bit operating system, x64-based processor Pen and touch	No pen or touch input is available for this display
Relevant Assets/Logs
debug.log
debug_fulltrust.log

Is this an issue in the preview version? https://files.community/download/preview


Remove @tf.function in tfa.image.equalize

tfa.image.equalize uses an internal scale_channel function which triggers excessive retracing when using it with datasets and probably in other cases as well. To illustrate this point the code snippet below benchmarks a naive tf.data loop once with the current implementation and once with a copy of the code in tfa.image.equalize with @tf.function removed.
On my machine I get:
[Insert a gazillion retracing warnings here]
With @tf.function: 0.6684637546539307
Without @tf.function: 0.08414626121520996

import time
import statistics
from functools import partial

import tensorflow as tf
import tensorflow_addons as tfa
import tensorflow_datasets as tfds

from tensorflow_addons.image.utils import to_4D_image, from_4D_image

def equalize_image(image, data_format: str = "channels_last") -> tf.Tensor:
    """Implements Equalize function from PIL using TF ops."""

    # @tf.function  <-- This triggers retracing.
    def scale_channel(image, channel):
        """Scale the data in the channel to implement equalize."""
        image_dtype = image.dtype

        if data_format == "channels_last":
            image = tf.cast(image[:, :, channel], tf.int32)
        elif data_format == "channels_first":
            image = tf.cast(image[channel], tf.int32)
        else:
            raise ValueError(
                "data_format can either be channels_last or channels_first"
            )
        # Compute the histogram of the image channel.
        histo = tf.histogram_fixed_width(image, [0, 255], nbins=256)

        # For the purposes of computing the step, filter out the nonzeros.
        nonzero = tf.where(tf.not_equal(histo, 0))
        nonzero_histo = tf.reshape(tf.gather(histo, nonzero), [-1])
        step = (tf.reduce_sum(nonzero_histo) - nonzero_histo[-1]) // 255

        def build_lut(histo, step):
            # Compute the cumulative sum, shifting by step // 2
            # and then normalization by step.
            lut = (tf.cumsum(histo) + (step // 2)) // step
            # Shift lut, prepending with 0.
            lut = tf.concat([[0], lut[:-1]], 0)
            # Clip the counts to be in range.  This is done
            # in the C code for image.point.
            return tf.clip_by_value(lut, 0, 255)

        # If step is zero, return the original image.  Otherwise, build
        # lut from the full histogram and step and then index from it.

        if step == 0:
            result = image
        else:
            result = tf.gather(build_lut(histo, step), image)

        return tf.cast(result, image_dtype)

    idx = 2 if data_format == "channels_last" else 0
    image = tf.stack([scale_channel(image, c) for c in range(image.shape[idx])], idx)

    return image

def equalize(
    image, data_format: str = "channels_last", name=None
) -> tf.Tensor:
    with tf.name_scope(name or "equalize"):
        image_dims = tf.rank(image)
        image = to_4D_image(image)
        fn = partial(equalize_image, data_format=data_format)
        image = tf.map_fn(fn, image)
        return from_4D_image(image, image_dims)

dset = tfds.load(name='mnist', data_dir='tensorflow_datasets', split='train')
dset = dset.map(lambda data: data['image']).cache()

# Caching for a fair comparison.
for image in dset:
    img = image
print('Cached dataset')

function_times = list()
for _ in range(10):
    start = time.time()
    for image in dset.take(10):
        augmented_image = tfa.image.equalize(tf.constant(image))
    function_times.append(time.time() - start)
print(f'With @tf.function: {statistics.mean(function_times)}')

non_function_times = list()
for _ in range(10):
    start = time.time()
    for image in dset.take(10):
        augmented_image = equalize(tf.constant(image))
    non_function_times.append(time.time() - start)
print(f'Without @tf.function: {statistics.mean(non_function_times)}')

The same is true for tfa.image.translate_xy.

Just a link to https://github.com/tensorflow/addons/pull/2264. Some discussions there.


passwordPolicies InputClaim DisableStrongPassword

In the final XML "Putting it all together" there is an InputClaim that is not referenced before:
<InputClaims>
        <InputClaim ClaimTypeReferenceId="passwordPolicies" DefaultValue="DisablePasswordExpiration, DisableStrongPassword" />
</InputClaims>

Is it necessary to set DisableStrongPassword to make the PIN password work?

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

ID: ddd28e85-f3db-fcf9-7b6b-c45fe9dfc9e6
Version Independent ID: 75e4906f-3704-e416-79df-3e1565e33ab6
Content: Password complexity in custom policies in Azure Active Directory B2C
Content Source: articles/active-directory-b2c/active-directory-b2c-reference-password-complexity-custom.md
Service: active-directory
GitHub Login: @davidmu1
Microsoft Alias: davidmu

@bibliothek Thanks for your feedback! We will investigate and update as appropriate.

@bibliothek No, PIN Password is an InputValidation element which validates a pin of any length but it is not used in the current sample as it is not referenced in inside ClaimType. Also, passwordPolicies specifies password policies for the user and is an enumeration of possible values like DisablePasswordExpiration, DisableStrongPassword.

Thanks for the clarification. It seems odd that the example only shows the validation of the PIN, but ignores it completely for authentication.


Sign out not clearing session

I'm having a couple of issues with sign in/out, when users sign in sometimes it just redirects them to the sign in page again to go through an endless loop unless they go to the forum root and try again. Another issue is sign out takes you to the signed out page but upon returning to index you're still signed in. If someone wants to take a closer look at this let me know, I should also mention my forum is using ONLY steam as  login authentication (Normal Sign In/Out was disabled/hidden completely)

i had the same issue, check here for my fix
https://github.com/Lavoaster/IP.Board-Steam-Authentication-Method/issues/68

Unfortunately I'm already running from the Github version, (Latest)

Apache or Ngnix?

Apache

Are you using a modified htaccess? Or the stock one from the friendly URL
area from the ACP?

Stock one


Cannot get a marker to show up at a GPS location?

I followed your Sceneform step-by-step directions, but I can't seem to get anything to show up when I place a marker at a GPS location.
onCreate()
// build and set renderable
CompletableFuture<ModelRenderable> arrowStage = ModelRenderable.builder().setSource(this, Uri.parse("Arrow.sfb")).build();

        CompletableFuture.allOf(arrowStage).handle(
                (notUsed, throwable) -> {
                    if(throwable != null) {
                        Log.d(TAG, "Unable to load renderable");
                        return null;
                    }

                    try {
                        arrowRenderable = arrowStage.get();
                        hasFinishedLoading = true;
                    } catch (InterruptedException | ExecutionException ex) {
                        Log.d(TAG, "Unable to load renderable");
                    }

                    return  null;
                });
 // set update listener on the AR scene
        arFragment.getArSceneView()
                .getScene()
                .addOnUpdateListener(
                        frameTime -> {

                            if(!hasFinishedLoading) {
                                return;
                            }

                            if(locationScene == null) {
                                locationScene = new LocationScene(this, this, arFragment.getArSceneView());
                                locationScene.mLocationMarkers.add(
                                        new LocationMarker(
                                                BULLPEN.latitude,
                                                BULLPEN.longitude,
                                                getArrow()));
                            }

                            if (locationScene != null) {
                                locationScene.processFrame(arFragment.getArSceneView().getArFrame());
                            }
                        });

^BULLPEN is just a pre-set location I am using.
getArrow()
private Node getArrow() {
        Node base = new Node();
        base.setRenderable(arrowRenderable);
        return base;
}

^is the setOnTapListener necessary? I would like it to be active on runtime without any interaction.
onResume() and onPause() also have their respective versions of:
if (locationScene != null) {
        locationScene.resume(); // locationScene.pause();
}

I was able to figure out my issue, thanks!


Error indicator on span in /Traces is hard to see in dark mode

The contrast between the red and the dark background doesn't meet accessibility standards, which might be ok if we could rely on the shape as well. But the outer shape is always a circle and the inner shape is hard to tell also, in part because of the size and in part because of the contrast:

It's a little better in light mode since the red/white contrast is better, but the shape of the indicator in the circle is still hard to see unless it is big enough:

This isn't just an a11y issue since it is hard to tell what the icon is at 100% scaling, but accessibility is definitely a part of how we'd address it.
A few thoughts:

We should see if we can adjust the icon in some way to make it easier to decipher (use regular instead of filled? use Size16 instead?). If not, we should consider if there's a different way to display that information
These individual blocks could be light-colored even if in dark mode (see screenshot below) if we wanted to keep the error red the same. But we should also see about trying to make an error red color that works better on the dark background across the site.

Just make the --error var red slightly lighter in light mode.

Option 3 is overriding the error color to a shade that we know is accessible against both backgrounds. I prefer this


Create an SAP Fiori App Using SAP Business Application Studio

Tutorials: https://developers.sap.com/tutorials/appstudio-fioriapps-create.html
In Step 4 (Run the App Locally in the Dev Space) the new tab shows an empty screen instead of the title suppliers:

There are also a lot of errors in the console:

Hi @GertSchneider ,
A fix was deployed.
Please re-check - You need to re-do the tutorial from the start.
Regards - Raz

Hi @raz-korn ,
the tutorial works now fine completely. Thanks for your help!
Best regards, Gert


Move plotting dependencies as extras

Plotting does not seem as a core functionality of u8timeseries (and it's just bloating our builds and dependencies) so mark them as optional/extra requirements, i.e. to make use also of plot stuff:
pip install .[plots]

instead of
pip install .

(and once published, pip install u8timeseries[plots])
But everything else (so >95%) will still just work without them.
Notes

like ~20s faster CI
IPython and tqdm also seem optional? if that's the case, could add those as optional in the same extras bundle, e.g. u8timeseries[visuals]

I'm not too sure... I think plotting will likely be very often useful, and it's a bummer to have to launch a special pip command to have it. The goal of the library should be simplicity of use (including install), and "getting everything out of the box". I think that should have precedence over CI tests time.

IMO if library does not work 100% without plots then we should not remove it.
Library is intended to compare timeseries, which does not say we need to visualise them, so I would recommend remove all visual related things from core library and allow use to chose ploting lib. We can provide ways to integrate library with matplotlib, but this is something on top of core lib.

That is one way of seeing it. However I think 80% of the time when you work with time series, you want to visualise them at some point. We could see this like the functionality on Pandas Series or Dataframe objects: although Pandas is not a visualisation tool per-se, you can still call my_series.plot(), which is extremely useful, and IMO should come out-of-the-box with u8timeseries.
Perhaps we could have the reverse? pip install u8timeseries[core] to install only the core (for instance for server use)?

pandas also doesn't include matplotlib by default. Users are required to install it separately.
By making it an extra, we would actually do more (compared to no extra at all and just core lib) by providing recommended and possibly tested visualization libs (matplotlib, plotly, etc.)
>>> df.plot()
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/Users/radujica/anaconda3/envs/pandas-plot/lib/python3.8/site-packages/pandas/plotting/_core.py", line 767, in __call__
plot_backend = _get_plot_backend(kwargs.pop("backend", None))
File "/Users/radujica/anaconda3/envs/pandas-plot/lib/python3.8/site-packages/pandas/plotting/_core.py", line 1666, in _get_plot_backend
  raise ImportError(
ImportError: matplotlib is required for plotting when the default backend "matplotlib" is selected.

OK I see your point @radujica. I'm hesitating.. because on the other hand, why removing plotting from core, if anyway most users will plot (at least I think)? @TheMP @pennfranc @endrjuskr any opinion?


Async Websocket Initializers?

Description of the feature
Currently async_graphql::http::WebSocket expects a synchronous initializer function however it would be useful to allow async results from that function so that asynchronous database calls can be made to authenticate the user based on a connection init payload.
Code example (if possible)
I'm looking for something approximately like this:
impl<S, F, Query, Mutation, Subscription> Stream for WebSocket<S, F, Query, Mutation, Subscription>
where
    S: Stream,
    S::Item: AsRef<[u8]>,
    F: FnOnce(serde_json::Value) -> Future<Result<Data>>,
    Query: ObjectType + Send + Sync + 'static,
    Mutation: ObjectType + Send + Sync + 'static,
    Subscription: SubscriptionType + Send + Sync + 'static,
{

I added this to the master branch, please help me test it, thank you. 🙂

I added this to the master branch, please help me test it, thank you. 🙂

Thanks again @sunli829, I've updated my codebase to master and it's compiling fine after a little hacking.
Oddly I needed to increase my type length limit to #![type_length_limit="15873664"] in order for my future returning initialization closure to compile. I'm not familiar with type_length_limit although it sounds fairly self descriptive - I suspect that's more an issue in my code?
Oh btw a little note of possible interest: I'm using this for something a little novel here, AFAIK this is the first async-graphql over WebRTC data channel implementation anyone's put together:
https://github.com/tegapp/teg/blob/feature/rust_dec_2020/crates/server/src/main.rs#L123

Thanks again @sunli829, I've updated my codebase to master and it's compiling fine after a little hacking.
Oddly I needed to increase my type length limit to #![type_length_limit="15873664"] in order for my future returning initialization closure to compile. I'm not familiar with type_length_limit although it sounds fairly self descriptive - I suspect that's more an issue in my code?
Oh btw a little note of possible interest: I'm using this for something a little novel here, AFAIK this is the first async-graphql over WebRTC data channel implementation anyone's put together:
https://github.com/tegapp/teg/blob/feature/rust_dec_2020/crates/server/src/main.rs#L123

Oddly I needed to increase my type length limit to #![type_length_limit="15873664"] in order for my future returning initialization closure to compile. I'm not familiar with type_length_limit although it sounds fairly self descriptive - I suspect that's more an issue in my code?

I will find out the cause of this problem.

Oh btw a little note of possible interest: I'm using this for something a little novel here, AFAIK this is the first async-graphql over WebRTC data channel implementation anyone's put together:

This is amazing! 🙂

Oddly I needed to increase my type length limit to #![type_length_limit="15873664"] in order for my future returning initialization closure to compile. I'm not familiar with type_length_limit although it sounds fairly self descriptive - I suspect that's more an issue in my code?

I will find out the cause of this problem.

Oh btw a little note of possible interest: I'm using this for something a little novel here, AFAIK this is the first async-graphql over WebRTC data channel implementation anyone's put together:

This is amazing! 🙂

This is amazing! slightly_smiling_face

Thanks! ^_^

I will find out the cause of this problem.

Even more thanks! I read here that nesting closures can cause this kind of issue with long monomorphised type names causing type_length_limit issues in stable Rust.
One thing I noticed that might allow us to reduce the level of nesting - because I'm expected to instantiate the async_graphql::Data instance inside the data_initializer I pass in a closure so that I can capture relevant application-scoped / not-per-request variables such as my database connection.
If WebSocket::with_data took a data instance and then passed it to the data_initializer then I wouldn't need the closure and could pass a reference to a function for the data_initializer instead. My hope being that this would reduce the monomorphized type length and resolve our issue. Something like this:
WebSocket::with_data
    pub fn with_data(
        schema: Schema<Query, Mutation, Subscription>,
        stream: S,
        data: Data,
        data_initializer: impl FnOnce(Data, serde_json::Value) -> R + Send + 'static,
        protocol: Protocols,
    ) -> Self {
    }

Example Usage
async fn initializer(mut data: Data, connection_init: serde_json::Value) {
    // Per-request-scoped data goes here
    let user = User::auth(data, connection_init).await?
    data.insert(user)
    Ok(data)
}

let mut data = async_graphql::Data::default();
// Application-scoped data goes here
data.insert(db)
data.insert(jwt_private_key)

let connection = async_graphql::http::WebSocket::with_data(
    schema,
    message_stream,
    data,
    initializer,
    async_graphql::http::WebSocketProtocols::GraphQLWS,
)

This is amazing! slightly_smiling_face

Thanks! ^_^

I will find out the cause of this problem.

Even more thanks! I read here that nesting closures can cause this kind of issue with long monomorphised type names causing type_length_limit issues in stable Rust.
One thing I noticed that might allow us to reduce the level of nesting - because I'm expected to instantiate the async_graphql::Data instance inside the data_initializer I pass in a closure so that I can capture relevant application-scoped / not-per-request variables such as my database connection.
If WebSocket::with_data took a data instance and then passed it to the data_initializer then I wouldn't need the closure and could pass a reference to a function for the data_initializer instead. My hope being that this would reduce the monomorphized type length and resolve our issue. Something like this:
WebSocket::with_data
    pub fn with_data(
        schema: Schema<Query, Mutation, Subscription>,
        stream: S,
        data: Data,
        data_initializer: impl FnOnce(Data, serde_json::Value) -> R + Send + 'static,
        protocol: Protocols,
    ) -> Self {
    }

Example Usage
async fn initializer(mut data: Data, connection_init: serde_json::Value) {
    // Per-request-scoped data goes here
    let user = User::auth(data, connection_init).await?
    data.insert(user)
    Ok(data)
}

let mut data = async_graphql::Data::default();
// Application-scoped data goes here
data.insert(db)
data.insert(jwt_private_key)

let connection = async_graphql::http::WebSocket::with_data(
    schema,
    message_stream,
    data,
    initializer,
    async_graphql::http::WebSocketProtocols::GraphQLWS,
)

This seems to be a problem with the rust language itself, so I decided to observe it for a while. 😁

This seems to be a problem with the rust language itself, so I decided to observe it for a while. 😁


{jam,ftjam}: set target-specific macros correctly when cross compiling

Description of changes
Jam embeds some information about the target platform, namely the OS and architecture, directly into the binary. We need to re-evaluate the macros for the target platform when Jam is being used to cross compile. I can successfully cross-compile p4 with this change, so I think this is hopefully the last change required here.
I'm not super pleased about having to bring pkgsBuildTarget into the mix, so if anyone has any better ideas, I'm all ears... :)
Things done

Built on platform(s)

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

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

It is so wonderful the way we support open source abandonware


Support for programmer setting?

Does ano support the Arduino IDE's "programmer" setting, found under the Tools->Programmer menu?
I'm trying to use ano to upload a sketch using the "Arduino as ISP", but I can't find any setting in the help docs to specify programmer.

Can you describe a little more what you are trying to do? I'm guessing you're using the Arduino to flash another device.
You might be able to get things working by copying /usr/share/arduino10/examples/ArduinoISP/ArduinoISP.ino over to a temporary directory and run ano/ino and seeing what happens. Granted, it's been a while since I've done this, and I had to review the Arduino website to check, and I forgot about the special "burn bootloader" command that needs to be ran. That's probably just some avrdude commands with a few arguments that are for this situation. I also looked at the documentation for a bit and couldn't find any remarks on using the Arduino as an ISP, so maybe that's a feature that could be added.
You might be able to get things working by playing with avrdude directly.


Second .msi launched during install of 1.0.3

Maybe this is not as scary as it looked to me, but I saw the following during install of JenkinsTray_v1.0.3:

I immediately cancelled as this looked like malware. The .msi did not have the same name the first time I tried it.

Hi @Reppinger, the digital certificate is correct.
I work for this company and they support me with license for DevExpress and development time for this project.
Due to Windows 8.1's blockade for unsigned applications, my company is kind enough to provide authenticode signing support to smoothen things out. See https://github.com/zionyx/jenkins-tray/issues/47
Btw, all releases are produced on build environments at work. Feel free to scan it at https://www.virustotal.com/ to boost your confidence a little.
Here's the scanned outcome: https://www.virustotal.com/en/url/4c68fe396e3cbf7028b1b4bac6073a67405a9006109c8dec5cf66ee70ac98148/analysis/1443740363/

Maybe i should include the publisher information in the release notes.

Btw, it is not the 2nd msi file, it prompts to confirm if you trust the publisher before proceeding with install.

Thanks. It was the first version where I noticed this. I mainly posted it so you could respond and set all of us at ease that it is not a problem.  Thanks for doing that!


Fix shapes in planar_normalizing_flow

In planar_normalizing_flow, log_prob is not flattened as done in other variables, which causes an error when sample has more than one (leading) axis with length >1.
e.g. Currently, the normalizing flow example crashes after 10 iterations (first test).
This PR fixes it.

Thanks. You are the first PR since public release! @wmyw96 will have a look.

Fine & Add test for the bug.


Fixed libra icon is blur and update components style.

Update list

New Libra icon (old seem blur).
New banner logo with testnet badge.
Fixed word line break when select "Special Drinks".
Updated some styles for better looks.

Nice, Thanks


We need to automate migration for added version key rules and added dependency notations

Ideally, the tests that already make sure we don't remove any dependency notation should also fail on additions, after generating migration resources to be picked up by migration code, and the tests would pass again after the migration resources add edited.
A manual edit would also be required to ensure just running the tests twice doesn't succeeds.
I think a pair-programming session with you @jmfayard would be great to tackle this issue with a great solution that helps us move faster and safer regarding bundling dependency notations.
Finally, we could also handle dependency notations removals and renames to be able to remove support for some of them if needed without breaking users badly.

done


angular cli use

I try to use your lib inside my angular-cli project your last version and I get :
 UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: VirtualScrollModule is not an NgModule

Looks like this is a generic issue caused by a problem with<EMAIL_ADDRESS>Temporarily workaround that works for me is to downgrade typescript to 2.0.10.

Hi Guys, I am getting this error today as well without the unhandled promise rejection ... just simply ... "ERROR in VirtualScrollModule is not an NgModule" at compile time.  I am on<EMAIL_ADDRESS>and have ensured that I have<EMAIL_ADDRESS>Wondering if it has anything to do with "NgModule" vs "ModuleWithProviders" but not really sure how to test that.  Seems to work fine when I build from source and run the demo.

Fixed. Please check now. Feel free to reopen if you still face this issue.


Can I run the code in horizontal_playground or just connect blocks so far ?

you might be looking for LLK/scratch-vm

Ah I see, great stuff, looking forward to it!

Unless I am missing something scratch-vm playground is only vertical blocks but I would like to test the horizontal blocks.

@rhulha All of this code -- both in scratch-blocks, and in scratch-vm -- is extremely experimental and unfinished. We're developing it in the open over the course of the next few months, but there is a lot more to do (especially with the VM, which just barely works).
You could try running horizontal blocks in the VM playground by swapping out the compiled scratch-blocks JS it includes (i.e., blocks_compressed_vertical -> blocks_compressed_horizontal, blockly_compressed_vertical -> blockly_compressed_horizontal, etc.) The VM should be basically compatible with running horizontal blocks right now. We made the playground to show vertical blocks, as we have a lot of restructuring to do to get it running that type of block :)
Let us know if you have any questions or run into problems. Eventually, we'll have this all working and documented for people to use.

Thanks for your kind response!
The reason I am trying to get this running is that I have also developed a horizontal semi Visual Language. Now I would like to decide if I should abandon my project and maybe help on Bocks instead:

Should I close this ticket or leave it open until it is easy to test and run the new Blocks ?

@rhulha I'll close it for now since I think we've answered the question. But feel free to raise new bugs if you find them, or continue commenting here (or in scratch-vm) if you run into trouble - we'd be happy to try to help.
Your project looks interesting! Does it have some kind of windowing system? I'd also note that it might be more appropriate to use our parent project, Blockly: https://developers.google.com/blockly/ - which is more of a general-purpose toolkit than Scratch Blocks.

It's written in Java and therefore has access to AWT and Swing.


Some fields are not specific to S2 and should be in separate extensions

As in https://github.com/stac-extensions/landsat/issues/2, many of the fields should be made more general and be placed in other extensions (many of the percentages, for example). Currently, this looks like everything was dumped into this extension.
Generally, I'm planning to talk to ESA to get an official extension in some way, e.g. through an open call proposal. Would you be interested in a collaboration on this?
PS: I'm opening these issues because I heard concerns about these kind of extensions from some people in the CEOS community, and also in the recent ARD BoF at BiDS.

@m-mohr Yes, certainly interested in collaboration here and share your concerns.
Like the Landsat extension, the goal here was to capture the fields that are currently used in the community for Sentinel-2 (which was not using an extension at all, and we needed to document them).   The percentages are really useful to search on but there's no appropriate extension right now.   I think it would be a good new extension so that providers could define their own list of calculated coverage stats.

Hi @matthewhanson this is a great initiative, thank you! Are you working on creating similar extensions for S1 and S3?

@vprivat-ads If we end up reindexing/indexing S1/S3 into Earth Search we'd probably create extensions first, but we have no immediate plans to do so.

ESA would like to discuss which fielda of this extension are important to the users / the STAC community and which we might be able to ditch. Also, we want to discuss whether there are better places for some fields that are not overly specific to the mission.
The document where you can leave comments can be found here: https://eoframework.esa.int/display/CDSE/Copernicus+Data+Space+Ecosystem+(CDSE)+STAC+catalogue
Alternatively, leave a comment here and I'll move them over to the Confluence page.
Thanks!


Fix kernel segfaults using staggerU with tensor sizes > 2^32

For these kernels, the calculation of the stagger offsets need to be augmented to 64 bit calculations to ensure that tensors in col-major format (N) with sizes > 2^32 can be handled.
This PR will address the 64 bit calculations in both unroll loop and tail section addressing updates of the kernels. Will also add tests for regression covering these cases where skinny tensors can get > 2^32 in size.

This PR broke rocBLAS build.  The instruction s_mul_hi_i32 is not supported on gfx803.  Please either fix this issue right away or revert this PR for now.  Feature Complete day is Wednesday.  We can't afford to have Tensile breaking rocBLAS build.  Thanks!

It looks like we really need some TensileCreateLibrary CI test against some subset of asm_ci or asm_full.

It looks like we really need some TensileCreateLibrary CI test against some subset of asm_ci or asm_full.

Indeed.  Currently we encourage all Tensile developers to make sure their siginficant PRs can build rocBLAS (and pass some subset of rocblas-test) before merging, which is a pain.  At any rate, in this case, testing only asm_ci won't catch the build failure because it's related to gfx803, which isn't covered by asm_ci.  For that matter, as @sdquiring pointed out, no CI tests gfx803 - other than when building rocBLAS with asm_full.


libgda: fix build with GCC 14

Fix #369438.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

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

x86_64-linux

:x: 5 packages failed to build:

glom
glom.dev
glom.devdoc
glom.doc
glom.lib

:white_check_mark: 8 packages built:

gnomeExtensions.pano
gtkd
libgda
libgdamm
libgdamm.dev
planner
sequeler
tilix

Since this is my first contribution to nixpkgs (as noted by the automatically added label), please let me know if I’ve made any mistakes. Thanks!


Tests fail with pipeline_error

Hello 👋
I tried to run the tests locally and got the following exception:
dyld[33619]: Library not loaded: @rpath/libhighs.1.5.dylib
  Referenced from: <C448DE4D-4EDC-3C46-AFE8-4AD8DF198300> /Users/david/Projects/Dotfiles/julia/.julia/artifacts/f5a7ecea4a8a9ee26d48f4fcce9af54d211adcb
4/bin/minizinc
  Reason: tried: '/Users/david/Projects/Dotfiles/julia/.julia/artifacts/f5a7ecea4a8a9ee26d48f4fcce9af54d211adcb4/bin/../lib/libhighs.1.5.dylib' (no suc
h file), '/Users/david/Projects/Dotfiles/julia/.julia/artifacts/f5a7ecea4a8a9ee26d48f4fcce9af54d211adcb4/bin/../lib/libhighs.1.5.dylib' (no such file),
 '/Users/david/Projects/Dotfiles/julia/.julia/juliaup/julia-1.6.0+0.x64.apple.darwin14/lib/julia/libhighs.1.5.dylib' (no such file), '/Users/david/.jul
ia/artifacts/ffe0a8aaa74bd3b3f610096503c0683b32e8ceec/lib/libhighs.1.5.dylib' (no such file), '/Users/david/Projects/Dotfiles/julia/.julia/juliaup/juli
a-1.6.0+0.x64.apple.darwin14/bin/../lib/julia/libhighs.1.5.dylib' (no such file), '/Users/david/Projects/Dotfiles/julia/.julia/juliaup/julia-1.6.0+0.x6
4.apple.darwin14/bin/../lib/libhighs.1.5.dylib' (no such file)
test_highs_feasibility: Error During Test at /Users/david/Projects/MiniZinc.jl/test/runtests.jl:16
  Got exception outside of a @test
  failed process: Process(`/Users/david/.julia/artifacts/f5a7ecea4a8a9ee26d48f4fcce9af54d211adcb4/bin/minizinc --solver highs --output-objective -o /va
r/folders/3b/0ks_42x93jvfw6f3xnkh_lqr0000gn/T/jl_UrjTPR/model.ozn /var/folders/3b/0ks_42x93jvfw6f3xnkh_lqr0000gn/T/jl_UrjTPR/model.mzn`, ProcessSignale
d(6)) [0]

  Stacktrace:
    [1] pipeline_error
      @ ./process.jl:525 [inlined]
    [2] run(::Base.CmdRedirect; wait::Bool)
      @ Base ./process.jl:440
    [3] run
...

I'm running Julia 1.9.2 on MacOS with the Apple Silicon built. I got the same exception with 1.8 and 1.6.
It seems that there is something wrong with the binary. ProcessSignaled(6) should indicate SIGABRT, meaning there is some critical error. There is also a message about a missing libhighs library.
Steps to reproduce:

Fresh clone
Start a Julia REPL
instantiate and resolve
test

The issue persists even when I remove all artifacts from .julia.
We observed something similar the first time when our CI suddenly started to fail: https://github.com/RelationalAI/SolverAPI.jl/actions/runs/6337329320/job/17401990379

Hmm. This is probably because of the new HiGHS release

What is Pkg.status()?
It looks like MiniZinc_jll uses compat correctly:
https://github.com/JuliaBinaryWrappers/MiniZinc_jll.jl/blob/7cc3dd33d76582bc5ffc33b5dbb7ea0c2c7bed6d/Project.toml#L15

Pkg.status() gives me the following:
(MiniZinc) pkg> status
Project MiniZinc v0.3.3
Status `~/Projects/MiniZinc.jl/Project.toml`
  [b8f27783] MathOptInterface v1.20.1
  [77125aae] Chuffed_jll v0.10.4+0
  [3677d96b] MiniZinc_jll v2.7.4+3

Weird. And ] status -m?

Can reproduce https://github.com/jump-dev/MiniZinc.jl/actions/runs/6456636021/job/17526520855?pr=49

MiniZinc_jll [3677d96b] MiniZinc_jll v2.7.4+3
was installed with [8fd58aa0] HiGHS_jll v1.6.0+0, but there is a
=1.5.3 compat constraint:
https://github.com/JuliaBinaryWrappers/MiniZinc_jll.jl/blob/7cc3dd33d76582bc5ffc33b5dbb7ea0c2c7bed6d/Project.toml#L15

I'll make a new Yggdrasil release: https://github.com/JuliaPackaging/Yggdrasil/pull/7513. Maybe that'll fix it. Otherwise we'll need to patch the registry. I've also asked in the #pkg channel on slack.

The new release will work, and I'm patching the registry for older versions: https://github.com/JuliaRegistries/General/pull/93084

Awesome, thank you very much!


[MM-15015] cherry-pick #65 from jira2 ...icons to ... menu items

Merging from jira2. @crspeller please note the diff in the icon.jsx

Add jira Icons to create and attach post menu items
place .png in /assets and share for create and attach
add url-loader to webpack

use svg instead of png for icon

Updating icon

Add line separator

Approving.  We've merged this into Jira2 branch for v2.0

@levb Build failure.
Also what am I supposed to notice about icon.jsx?

@crspeller iconStyle = {flex: '0 0 auto', was not there before, I didn't know what it was for, thought you may be using this icon your code and may be affected.
Fixed the build issue.


Add expand_coverage_report option to for #270

Description
Issue
This PR provides a work-around that fixes #270
The issue reported there is that diff data contains all modified lines, while most coverage tools report by statement and not lines.
As an example, let's take this code change:
429        if worker is None:
430            metrics.UNEXPECTED_RETURN_VALUE_COUNTER.labels(
- 431                code=self._UNKNOWN_WORKER_RESPONSE["returnvalue"]["code"],
+ 431                code=self._UNKNOWN_WORKER_RESPONSE["code"],
432                method=request.method,
433                endpoint=request.path,
434            ).inc()
435            return Response(self._UNKNOWN_WORKER_RESPONSE)

Diff file will report line 431 as being changed.
However, most coverage tools will report lines 429, 430 and 435 being hit or not by test, but nothing for line 431.
As a result, the Violation reporters do not report line 431 has a violation, even if it does not run during tests.
But diff reports don't report line 430 has being changed. So in the end, this changes goes silent in the diff-cover analysis.
Fix implemented
Following this suggestion: https://github.com/Bachmann1234/diff_cover/issues/270#issuecomment-1017563602, it seems most coverage reports use the line opening a statement as the reference. Also, untested statements are reported in the coverage files, with 0 hits (they are not just missing).
The implemented solution is to:

add an option --expand-coverage-report (not on by default because this workaround might not work in all situations, to be further tested with different coverage reporters and languages.)
When the option is set, all line numbers not listed in the XML reports are extrapolated from the previous line reported.

In the example above, line 431 is not in coverage.xml. But line 430 is, so we add line 431 in the data extracted from the XML report, with the same number of hits as 430.
Risks & limitations

The option is only implemented for XML reports.
This fix works as expected under the assumptions that:

The coverage tool reports untested statements too, with hits set to 0.
The coverage tool reports statements based on the opening line.

Blank lines/empty lines might be reported?
There are no unit/integration tests

Tests performed
I tested this PR on the example presented above (so in Python), and with other changes covered in the same file ; with:

pytest --cov=. --cov-report=xml
diff-cover coverage.xml --compare-branch=origin/develop --markdown-report diff-cover-report.md --exclude 'test*.py' --fail-under=50 --expand_coverage_report

When line 430 is not tested

coverage.xml contains

						<line number="424" hits="0"/>
						<line number="426" hits="1"/>
						<line number="427" hits="1"/>
						<line number="428" hits="1"/>
						<line number="429" hits="1"/>
						<line number="430" hits="0"/>
						<line number="435" hits="0"/>
						<line number="436" hits="1"/>
						<line number="438" hits="1"/>
						<line number="439" hits="1"/>

and diff-coverage returns:

Diff Coverage
Diff: origin/develop...HEAD, staged and unstaged changes
saas/views.py (95.5%): Missing lines 431
Total:   22 lines
Missing: 1 line
Coverage: 95%
The markdown:

Diff Coverage
Diff: origin/develop...HEAD, staged and unstaged changes

saas/views.py (95.5%): Missing lines 431

Summary

Total: 22 lines
Missing: 1 line
Coverage: 95%

saas/views.py
Lines 427-435
  427         params["id"] = job_id
  428         worker = settings.WORKERS.get(worker_name.lower())
  429         if worker is None:
  430             metrics.UNEXPECTED_RETURN_VALUE_COUNTER.labels(
! 431                 code=self._UNKNOWN_WORKER_RESPONSE["code"],
  432                 method=request.method,
  433                 endpoint=request.path,
  434             ).inc()
  435             return Response(self._UNKNOWN_WORKER_RESPONSE)

When line 431 is tested

coverage.xml contains

						<line number="424" hits="0"/>
						<line number="426" hits="1"/>
						<line number="427" hits="1"/>
						<line number="428" hits="1"/>
						<line number="429" hits="1"/>
						<line number="430" hits="1"/>
						<line number="435" hits="0"/>
						<line number="436" hits="1"/>
						<line number="438" hits="1"/>
						<line number="439" hits="1"/>

and diff-coverage returns:

Diff Coverage
Diff: origin/develop...HEAD, staged and unstaged changes
saas/views.py (100%)
Total:   22 lines
Missing: 0 lines
Coverage: 100%
The markdown:

Diff Coverage
Diff: origin/develop...HEAD, staged and unstaged changes

saas/views.py (100%)

Summary

Total: 22 lines
Missing: 0 lines
Coverage: 100%

I'm gonna need a bit of time before I get to this. But I see it. Just a busy time this week.

This is pretty interesting. I feel a bit iffy about trying to interpret data not in the report. But this seems sensible.
The flag should be documented in the readme
and ill need a few tests before I can merge this

Thank you for the feedback. I won't have time right now for the test but I'll try to do something about this in the upcoming weeks for sure!

Annoyingly my readme formatting check had some nitpicks, I made the suggestions that I think will fix these.
Pr looks good, once we get this merged ill try to get it out there asap.

Good catch, thanks. It should be good for the 2 feedbacks you shared now 👍

Alright. Let's merge this in. I'll get it released in the next day or so

Thank you! Try out https://pypi.org/project/diff-cover/9.2.0/


OLS-109: Better docstring for cache

Description
Better docstring for cache
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[x] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library

Related Tickets & Documents

Related Issue #OLS-109

Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] PR has passed all pre-merge test jobs.

/approve

/lgtm


Multiple results for the same part

Example :when searching for atmega168, the last includes 4 results for ATMEGA168-20 with no indication what the differences might be.

Sorry to open an old topic, but still happens to some ambiguous names. Sometimes the same part appear waive because of a different manufacturer or it contains part of the code.

For latter, you could search for exact matches and two other options by using the dropdown list (Spinner in Android terms) on the search page. See Search.Mode
For the former, should the result page include manufacturer? I found no use for it, but if it's necessary, I could include it I guess (but it's not trivial with that soup of HTML tags that they call website).


8tracks playlists with periods at the end of the title break in windows

Example: http://8tracks.com/emilybrock101/ladies-aren-t-supposed-to-talk-about-those-things

Fixed by acdaa485f9906d221ff4dafc4c20c7f33d9c89d9


makeself: backport megastep/makeself#142

Currently, a self-extracting archive created by makeself will fail to properly execute on NixOS because the boilerplate Bash code it uses to clean up the temporary directory it extracted its contents into
assumes that the rm command is installed at /bin/rm, which is not the case on NixOS.
This commit, a backport of a pull request I made to the upstream repository at megastep/makeself#142, fixes the issue by causing the boilerplate code to call rm without specifying an absolute path, which allows the version of rm from one's current Nix environment to be used instead.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build makeself

Success on x86_64-linux (full log)
Attempted: makeself
Partial log (click to expand)
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/6vinvl5va09gwd5zi5s7g54xs9ns0l93-makeself-2.4.0
gzipping man pages under /nix/store/6vinvl5va09gwd5zi5s7g54xs9ns0l93-makeself-2.4.0/share/man/
strip is /nix/store/dxf1m7dhc4qb655bdljc1fsd74v1nag3-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/6vinvl5va09gwd5zi5s7g54xs9ns0l93-makeself-2.4.0/bin
patching script interpreter paths in /nix/store/6vinvl5va09gwd5zi5s7g54xs9ns0l93-makeself-2.4.0
/nix/store/6vinvl5va09gwd5zi5s7g54xs9ns0l93-makeself-2.4.0/bin/makeself: interpreter directive changed from "/bin/sh" to "/nix/store/nii7pk6pv4x4as7vsxbvwyzjn67vax6r-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/6vinvl5va09gwd5zi5s7g54xs9ns0l93-makeself-2.4.0...
/nix/store/6vinvl5va09gwd5zi5s7g54xs9ns0l93-makeself-2.4.0

Success on x86_64-darwin (full log)
Attempted: makeself
Partial log (click to expand)
building
no Makefile, doing nothing
installing
post-installation fixup
gzipping man pages under /nix/store/wjyafajb8px90z6qcc7vysf5w2w3cgh4-makeself-2.4.0/share/man/
strip is /nix/store/9xjkb4xz0b5lmizij9ppxy7lkxdxhx6b-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/wjyafajb8px90z6qcc7vysf5w2w3cgh4-makeself-2.4.0/bin
patching script interpreter paths in /nix/store/wjyafajb8px90z6qcc7vysf5w2w3cgh4-makeself-2.4.0
/nix/store/wjyafajb8px90z6qcc7vysf5w2w3cgh4-makeself-2.4.0/bin/makeself: interpreter directive changed from "/bin/sh" to "/nix/store/f2lp21azk4w88f17zgf4mji64agb18ga-bash-4.4-p23/bin/sh"
/nix/store/wjyafajb8px90z6qcc7vysf5w2w3cgh4-makeself-2.4.0

Success on aarch64-linux (full log)
Attempted: makeself
Partial log (click to expand)
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/c41siiz03raj8m1frql25gq7b8xry4kr-makeself-2.4.0
gzipping man pages under /nix/store/c41siiz03raj8m1frql25gq7b8xry4kr-makeself-2.4.0/share/man/
strip is /nix/store/428gs2z4b8f9byvghzlpbjwjb3a7jwxx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/c41siiz03raj8m1frql25gq7b8xry4kr-makeself-2.4.0/bin
patching script interpreter paths in /nix/store/c41siiz03raj8m1frql25gq7b8xry4kr-makeself-2.4.0
/nix/store/c41siiz03raj8m1frql25gq7b8xry4kr-makeself-2.4.0/bin/makeself: interpreter directive changed from "/bin/sh" to "/nix/store/017xymp5i03qzigafnfx7sgf6602gmzq-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/c41siiz03raj8m1frql25gq7b8xry4kr-makeself-2.4.0...
/nix/store/c41siiz03raj8m1frql25gq7b8xry4kr-makeself-2.4.0

backport: 37564578ace

this one is the actual backport: 536f6e86cf1


Crash: invalid entity

Dragging over a site that had ghosts being built by robots and some miners running already
The mod YARM - Resource Monitor (0.8.203) caused a non-recoverable error.
Please report this error to the mod author.

Error while running event YARM::on_tick (ID 0)
LuaEntity API call when LuaEntity was invalid.
stack traceback:
	[C]: in function '__index'
	__YARM__/resmon.lua:412: in function 'scan_current_site'
	__YARM__/resmon.lua:1293: in function 'update_players'
	__YARM__/resmon.lua:1346: in function 'on_tick_internal'
	__YARM__/resmon.lua:1380: in function <__YARM__/resmon.lua:1375>

I think this one is also #180


reformat docker build phase

use golang build image to build swan bianry
add docker version to docker tag

Signed-off-by: Deshi Xiao<EMAIL_ADDRESS>

Codecov Report

Merging #380 into master will decrease coverage by -0.46%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #380      +/-   ##
==========================================
- Coverage   42.67%   42.22%   -0.46%
==========================================
  Files          21       21
  Lines       10764    10764
==========================================
- Hits         4594     4545      -49
- Misses       4984     5036      +52
+ Partials     1186     1183       -3

Continue to review full report at Codecov.

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

👍

LGTM


here is code run is not properly please fix the bug.it throw the error that .exe file is stop to working.

Environment (please complete the following information):

OS (e.g., Windows 10):
Extension version (e.g., 1.2.3):

Describe the bug:

Repro steps:

Expected behavior:

Additional context:

Yeah, that's unparseable and we don't have anything to do with any .exe


UBY: repo clone links as a href, copy to clipboard

this

[ ] git clone https://URI.URI/uri/uri -b branchname
[ ] git clone git://URI.URI/uri/uri -b branchname [clipboardicon]

instead of this:
git clone <URI> -b branchname

See: #5


New command: m365 pp solution publisher get

Usage
m365 pp solution publisher get
Description
Get information about the specified publisher
Options

Option
Description

-e, --environment <environment>
The name of the environment

-i, --id [id]
The ID of the publisher. Specify either id or name but not both.

-n, --name [name]
The name of the publisher. Specify either id or name but not both.

--asAdmin
Run the command as admin for environments you are not a member of

Examples
Get publisher in the environment named Default-d87a7535-dd31-4437-bfe1-95340acd55c5 using the name.
m365 pp solution publisher get --environment Default-d87a7535-dd31-4437-bfe1-95340acd55c5 --name Contoso

Get publisher in the environment named Default-d87a7535-dd31-4437-bfe1-95340acd55c5 using the id as an administrator.
m365 pp solution publisher get --environment Default-d87a7535-dd31-4437-bfe1-95340acd55c5 --id 8b839fb7-288f-ec11-b400-000d3a26c374 --asAdmin

Additional Info
See #3766 for inspiration on the query
Use friendlyname to query against.

LGTM 🚀

Can I have the honor to work on this one 😃 ?

The honor is ours! 😊


Update python.nanorc

Updating python.nanorc to better match cython.nanorc (where relevant).
Cython had a more complete syntax highlighting definition, and this just tries to bring python syntax highlighting to match it.

Looks good!


"list" argument must be an Array of Buffers

I was trying to use @simplewebauthn/server in cloudflare workers, but getting this error when verifying attestation
TypeError: "list" argument must be an Array of Buffers
    at Function.a.concat (index.js:376)
    at Object.convertCOSEtoPKCS [as default] (convertCOSEtoPKCS.js:19)
    at Object.verifyAttestationPacked [as default] (verifyPacked.js:50)
    at verifyAttestationResponse (verifyAttestationResponse.js:166)

Cloudflare workers provide access to most browser api's including cypto api. Is this possible to run this package in cloudflare workers with browserifying ?

@GagnDeep Can you provide an example of how you're calling verifyAttestationResponse, including the arguments you're feeding it?

async function handleVerifyAttestation({attestationRes, user, rpID, expectedOrigin}) {
    const expectedChallenge = user.currentChallenge;
    try {
      const opts = {
        credential: attestationRes,
        expectedChallenge: `${expectedChallenge}`,
        expectedOrigin,
        expectedRPID: rpID
      };

      return await verifyAttestationResponse(opts);
    } catch (error) {
      throw error;
    }
  }

Here attestationRes will be the response of calling startAttestation in browser, similar to in example.
Here's a codesanbox link. Worker i cannot share easily but running it here provides similar enviroment of what i have in worker. And here also it throws same error "list" argument must be an Array of Buffers
When running this example just don't use the embedded browser. Click on open in new tab and open console to see that error

Thank you for providing the CodeSandbox. It appears this is an issue with attestation statement verification. The following direct attestations generate that error:
FIDO-U2F: SimpleWebAuthn debugger link
Packed: SimpleWebAuthn debugger link
I'll dig into this and report back :)

@GagnDeep I ran those two attestation responses through unit testing and they verified fine for me. Upon close examination of that CodeSandbox example, it appears to me that it's polyfilling enough of Node's APIs to mostly enable execution of @simplewebauthn/server in the browser. I hope you'll understand when I say that that's not a supported use case of that package - it's Node or nothing 😂
As for your Cloudflare Worker issue, I imagine your problem there is that Cloudflare Workers execute JavaScript in an environment that is more similar to browsers than Node. CF Workers and Node are both V8-powered, but offer separate collections of APIs and so are not equivalent runtime environments. The issue with "list" argument must be an Array of Buffers is probably somehow related to the fact that Buffer values, which @simplewebauthn/server make extensive use of, are a NodeJS data type that browser-like runtime environments lack native support for.
Browser polyfills to support NodeJS Buffers exist, so there's a chance you could get something to work, but at this time this is not a supported use case of @simplewebauthn/server.


Instructions for compiling libclang_rt.builtins-wasm32.a?

I'm trying to use clang-9 to compile and use wasi-sysroot, but additionally the libclang_rt.builtins-wasm32.a file is required to link into the final wasm module. It is not obvious how to build this file without building the llvm/clang submodules first, so it would be much appreciated if you can add relevant instructions. Thanks a lot!

@TerrorJack Did you ever find out how to build libclang_rt.builtins-wasm32.a from source?

libclang_rt.builtins-wasm32.a built a build of llvm's compiler-rt.
You can see how we build it as part of wasi-sdk here: https://github.com/WebAssembly/wasi-sdk/blob/1a953299860bbcc198ad8c12a21d1b2e2f738355/Makefile#L98-L124

If you don't want to build it from source you can download a prebuilt version of this library from one of the releases.  e.g. https://github.com/WebAssembly/wasi-sdk/releases/tag/wasi-sdk-12
Then can most likely just copy this library into ./lib/clang/9.0.0/lib/wasi/ inside your existing clang installation.

Is it possible to build libclang_rt.builtins-wasm32.a this with -phread?

I have successfully compiled Clang. Can I use this Clang to individually compile libclang_rt.builtins-wasm32.a in wasi-sdk?


[CARBONDATA-1209] add partitionId in show partition result

add partitionId in show partition result
change example to avoid truncate of result.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Build Failed with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/493/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3082/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3083/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3089/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/499/

Build Failed with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/543/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3135/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/554/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3147/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/573/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3168/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/574/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3169/

LGTM

LGTM