add note to include jqueryui before angularjs

First of all, thanks for the contribution. It's really great to have wide community participation.
I created a pen example that includes jquery-ui before angularjs and seems to work fine.
Can you include an example showcasing your use case that caused the error?

Oops typo. I meant "angularjs before jquery-ui".

Not sure what the best tool is to create an example for this. I currently using requirejs for loading dependencies and also using jqueryui-amd version. If I don't have 'jqueryui' as a dependency for 'angular' in the config.js then I get the previously error mentioned in the issue.

What if you just define jquery as a dependency to angular, as shown by the
requirejs example, found in the README.md ?
Please feel free to fork that code example (or any other applicable) for
your demonstration purposes.
On Tue, Mar 10, 2015, 15:49 Harshil Shah<EMAIL_ADDRESS>wrote:

Not sure what the best tool is to create an example for this. I currently
using requirejs for loading dependencies and also using jqueryui-amd
version. If I don't have 'jqueryui' as a dependency for 'angular' in the
config.js then I get the previously error mentioned in the issue.
—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/ui-sortable/pull/342#issuecomment-78055771
.

I already have jquery as a dependency for angular. doesn't work though.
Best Regards,
Harshil
On Tue, Mar 10, 2015 at 12:34 PM, Thodoris Greasidis <
<EMAIL_ADDRESS>wrote:

What if you just define jquery as a dependency to angular, as shown by the
requirejs example, found in the README.md ?
Please feel free to fork that code example (or any other applicable) for
your demonstration purposes.
On Tue, Mar 10, 2015, 15:49 Harshil Shah<EMAIL_ADDRESS>wrote:

Not sure what the best tool is to create an example for this. I currently
using requirejs for loading dependencies and also using jqueryui-amd
version. If I don't have 'jqueryui' as a dependency for 'angular' in the
config.js then I get the previously error mentioned in the issue.
—
Reply to this email directly or view it on GitHub
<
https://github.com/angular-ui/ui-sortable/pull/342#issuecomment-78055771>

.

—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/ui-sortable/pull/342#issuecomment-78092308
.

If you are able to provide a code example to show your case I will be happy
to merge this PR.
But my experience till now does not show what you are reporting.
On Tue, Mar 10, 2015, 18:44 Harshil Shah<EMAIL_ADDRESS>wrote:

I already have jquery as a dependency for angular. doesn't work though.
Best Regards,
Harshil
On Tue, Mar 10, 2015 at 12:34 PM, Thodoris Greasidis <
<EMAIL_ADDRESS>wrote:

What if you just define jquery as a dependency to angular, as shown by
the
requirejs example, found in the README.md ?
Please feel free to fork that code example (or any other applicable) for
your demonstration purposes.
On Tue, Mar 10, 2015, 15:49 Harshil Shah<EMAIL_ADDRESS>wrote:

Not sure what the best tool is to create an example for this. I
currently
using requirejs for loading dependencies and also using jqueryui-amd
version. If I don't have 'jqueryui' as a dependency for 'angular' in
the
config.js then I get the previously error mentioned in the issue.
—
Reply to this email directly or view it on GitHub
<
https://github.com/angular-ui/ui-sortable/pull/342#issuecomment-78055771

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/angular-ui/ui-sortable/pull/342#issuecomment-78092308>
.

—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/ui-sortable/pull/342#issuecomment-78094172
.

yep. will try and recreate a sample.
Best Regards,
Harshil
On Tue, Mar 10, 2015 at 1:46 PM, Thodoris Greasidis <
<EMAIL_ADDRESS>wrote:

If you are able to provide a code example to show your case I will be happy
to merge this PR.
But my experience till now does not show what you are reporting.
On Tue, Mar 10, 2015, 18:44 Harshil Shah<EMAIL_ADDRESS>wrote:

I already have jquery as a dependency for angular. doesn't work though.
Best Regards,
Harshil
On Tue, Mar 10, 2015 at 12:34 PM, Thodoris Greasidis <
<EMAIL_ADDRESS>wrote:

What if you just define jquery as a dependency to angular, as shown by
the
requirejs example, found in the README.md ?
Please feel free to fork that code example (or any other applicable)
for
your demonstration purposes.
On Tue, Mar 10, 2015, 15:49 Harshil Shah<EMAIL_ADDRESS>wrote:

Not sure what the best tool is to create an example for this. I
currently
using requirejs for loading dependencies and also using jqueryui-amd
version. If I don't have 'jqueryui' as a dependency for 'angular' in
the
config.js then I get the previously error mentioned in the issue.
—
Reply to this email directly or view it on GitHub
<

https://github.com/angular-ui/ui-sortable/pull/342#issuecomment-78055771

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/angular-ui/ui-sortable/pull/342#issuecomment-78092308

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/angular-ui/ui-sortable/pull/342#issuecomment-78094172>

.

—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/ui-sortable/pull/342#issuecomment-78108022
.

@harshil07 I'm closing this for now.
Feel free to reopen with a code example and then we can merge this.


sarkar/Update test_examples.py

What does this PR do?
creating pr on behalf of @yeonsily

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


increment tds driver version in dependency

Hi @mjaric, just to clarify, i have approved so you can go ahead and merge it. :)
Btw, we should release Ecto v3.4 in one or two days. Are you ok with this or would you like to wait for the release a bit more?

There is one thing I'm looking at the moment. It is about named queries, required for caching and streaming.
Tds.prepare/3 lacks name argument and it should be Tds.prepare/4. Since this is braking change maybe it is better if I now create new tds version that implements prepare/4 and ignores name argument. So later ecto_sql can work with any tds 2.2+.
Make sense?

second option is to ensure in ecto_sql deps that tds is >= 2.2.0 and release this now

second option is to release this now as is and later when we add caching and streaming we should ensure that ecto_sql tds_deps requires >= 2.2.0

Exactly. We can just bump the requirement later, as long as you don't break the API in 2.1.X, we are good.

Then we are good to go. I will merge this.
Thanks!


Fix another reference to go1.x

One more reference to go1.x runtime was remaining in the readme. This fixes that up

@triarius I dont have permission to add reviewers, but can you look at this?


Not listed yet: software.ac.uk

The website at this url is unlisted: https://software.ac.uk/

Thanks for the help, the website will be listed soon.


feat: new optional loadFonts  parameter

Description
fix memory issue along multiple call to loadFonts.
https://github.com/Betterment/alchemist/issues/76
As a developer, I just need to call the following code before my multiple goldenTest().
setUpAll(() async {
    await loadFonts();
  });

This fix also increase test speed, it's insane.

Type of Change
New optional parameters

[x] ✨ New feature (non-breaking change which adds functionality)
[x] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[x] 📝 Documentation
[ ] 🗑 Chore

I find it a bit cumbersome to have to specify goldenTest(loadFonts: false) every time when you might want to disable it globally.
Maybe you could consider injecting the loadFonts value with an environment variable 🤔
For example:
flutter test --dart-define=LOAD_GOLDEN_FONTS=false

The implementation in golden_test.dart could be:
const _loadFontsDefault = bool.fromEnvironment('LOAD_GOLDEN_FONTS', defaultValue: true);

// ...

Future<void> goldenTest({
  // ...
  bool? loadFonts,
}) async {
  // ...

  goldenTestAdapter.setUp(
    () => _setUpGoldenTests(mustLoadFonts: loadFonts ?? _loadFontsDefault),
  );
}

I find it a bit cumbersome to have to specify goldenTest(loadFonts: false) every time when you might want to disable it globally. Maybe you could consider injecting the loadFonts value with an environment variable 🤔
For example:
flutter test --dart-define=LOAD_GOLDEN_FONTS=false

The implementation in golden_test.dart could be:
const _loadFontsDefault = bool.fromEnvironment('LOAD_GOLDEN_FONTS', defaultValue: true);

// ...

Future<void> goldenTest({
  // ...
  bool? loadFonts,
}) async {
  // ...

  goldenTestAdapter.setUp(
    () => _setUpGoldenTests(mustLoadFonts: loadFonts ?? _loadFontsDefault),
  );
}

I think we can find a better solution. I create this PR as a quick fix because i (and my team) couldn't use this package.
Defining env variable is maybe not perfect. If someone forgot it or just run flutter test, he can encounter issues.

Defining env variable is maybe not perfect. If someone forgot it or just run flutter test, he can encounter issues.

True, maybe adding a property to AlchemistConfig or PlatformGoldensConfig should be the way to go 🤔

Don't close please

Sorry about that! We have to make some time to dig into this

commenting for visibility

What I'm gather from this is, loadFonts only needs to be called once per group/test/instance (which one is it?). So then shouldn't we move loadFonts into some setup logic that is guaranteed to only be run once per group/test/instance?
This doesn't seem like something users should handle, but instead something that Alchemist needs to keep into account by default. I don't see a use-case for a feature where users can toggle loadFonts on or off per test or group -- it should either happen once or not at all (if a user desires the latter).
@samandmoore @btrautmann @EArminjon Can any of you confirm this thinking?

What I'm gathering from this is, loadFonts only needs to be called once per group/test/instance (which one is it?). So then shouldn't we move loadFonts into some setup logic that is guaranteed to only be run once per group/test/instance?
This doesn't seem like something users should handle, but instead something that Alchemist needs to keep into account by default. I don't see a use-case for a feature where users can toggle loadFonts on or off per test or group -- it should either happen once or not at all (if a user desires the latter).
@samandmoore @btrautmann @Kirpal @EArminjon Can any of you confirm this thinking?

That is exactly it. Thank you!
If we could make sure it runs one per every flutter test that would be ideal as devs don't tend to change fonts per test/group.

What I'm gathering from this is, loadFonts only needs to be called once per group/test/instance (which one is it?). So then shouldn't we move loadFonts into some setup logic that is guaranteed to only be run once per group/test/instance?
This doesn't seem like something users should handle, but instead something that Alchemist needs to keep into account by default. I don't see a use-case for a feature where users can toggle loadFonts on or off per test or group -- it should either happen once or not at all (if a user desires the latter).
@samandmoore @btrautmann @Kirpal @EArminjon Can any of you confirm this thinking?

I agree with the thinking here and agree with @pedromassango that this makes sense once per "suite run". This actually feels similar in terms of granularity to what I mention here, where we're wanting to detect the architecture once and then resolve the golden paths based on that. I mentioned some sort of "global caching mechanism" that may make sense for this use case as well, so maybe in whatever issue we tackle first we could be sure to support additional use cases (if it makes sense to do so).


Tie creature bulk to STR score

Currently, there is no way to adjust the maximum bulk for a creature, which is tied to size. Would appreciate being able to tie bulk to a creature's Strength score (not modifier).

The code (for bulk calculation) is bounded to the str mod (and the size of course).
https://github.com/p4535992/foundryvtt-variant-encumbrance-dnd5e/blob/3b8436d385021a469b3f38450d8a79cdf8f79902/src/scripts/VariantEncumbranceBulkImpl.ts#L586
did you have some suggestion for a better behaviour ?

I was asking about being able to tie it purely to STR score, not to STR modifier or size.

OK, I'll try that -- thanks for the suggestion!

version 0.11.7 has this module setting now


Generate / save / pass PKCE verifier and challenge

For now the identity provider wrappers just ignore the challenge, that'll come later (other than OrcID, which doesn't support it)

Looks like codcov is having issues, although the upload worked


Question: Would you be open to some CI/CD?

Hey @exilon,
I have this thing that I kinda hate having tags without a GitHub official release.
I'm aware that this can be a me thing and that not everyone is on board.
I'm the author of setup-lazarus, and I did this to allow me to have proper support for CI/CD with Free Pascal. Sorry, have tried to bug the Embarcadero peeps to give me something to do the same with Delphi, but haven't had any luck due to the fact that the Community Edition does not ship with a command line compiler.
If you tell me what is the project that you use for Unit testing, I can produce a PR that will compile and run it, at least with Free Pascal's lazbuild.
I could then do the same with QuickLogger, once we've had this one settled.
Many thanks in advance for your attention.
Cheers,
Gus

Hey John(@jkour),
After I dropped this issue, I realised that your code is more akin to a package than an application.
That left me scratching my head a bit, on how I would do the CI/CD of it.
I guess that if you have a CLI unit testing app, I could compile and run that and that's the CI part done.
Then, the CD part is kinda just zipping the code... There's no cross compiling for many platforms... And that is already done on every release by default. So yeah, scratching my head :smiley:
I guess we could automate the release process. Using something like a release_notes.md file on the repo to feed to the automated release action.
I don't quite remember if you have an example application. If so, that could be the thing that we can use for the CD part. An example, readily compiled into the 3 major platforms would be nice, right. If that even makes sense, of course.
In conclusion, I think I jumped the gun a tad.
I'll let you read my suggestions above and when you decide what you want, we can tailor it to your needs.

I couldn't really understand how you use github actions with CI/CD

Please elaborate on this. You kinda lost me here...
Cheers,
Gus

Hi @gcarreno ,
I see. That's why I got confused as well 😃
For the record, I am not the owner of the repo. I just asked out of curiosity.
@exilon is the owner

Hey John(@jkour),

I see. That's why I got confused as well 😃

Yeah, my deepest apologies for the confusion. I guess I shot my mouth without engaging my brain first :rofl:

For the record, I am not the owner of the repo. I just asked out of curiosity.
@exilon is the owner

Yeaps, gotcha :wink:
Nonetheless, I can still add the CI part ( if a CLI unit test thing is done ).
And on the CD part, I can add an automated release upon a git push --tags that will read the release notes from a file, e.g. release_notes.md.
So, @exilon, please give me a poke if you're up for it.
Cheers,
Gus


implement built-in function str_to_date

Closes #518
This PR implements the STR_TO_DATE MySQL function.
In places where the spec is ambiguous, I'm attempting to match the behavior of MySQL version 8 from my manual testing. I need to implement a few more parsers and add more test cases to cover the expected behavior.
go test  -cover github.com/dolthub/go-mysql-server/sql/parse/dateparse

ok  	github.com/dolthub/go-mysql-server/sql/parse/dateparse	0.163s	coverage: 89.3% of statements

cc @zachmu

@zachmu it doesn't seem I can tag reviewers, but this is ready for a first pass while I finish up the final few format specifiers.

I missed the timezone setup in the test, I get it now :)
I'm going to go ahead and merge this in, please add an engine test as a followup. You can implement the missing week specifiers, or leave them as TODO for now. Those are pretty obscure for the purpose of date parsing.


Upgrade DMN editor and support more current version of the DMN spec

First of all, thank you for making such a helpful tool available. We have found it to be the most featured and stable environment available for collaborating and testing DMN models.
We are wondering if there are any plans to update the DMN editor, as there have been many improvements made to the library, including features like copy-paste, more robust row and column insertion, etc. (summary here https://github.com/kiegroup/kie-tools/pull/1562)
As well, are there any plans to support more recent DMN specs? There are some new FEEL functions, such as round up that are not supported in the declab tool, which came with DMN 1.4 that is supported in recent versions of Drools.
Thank you in advance.

Thank you very much for reaching out and for your feedback. We are pleased to hear that you find it useful!
We are currently working hard on declab 3.0, which will encompass a multitude of improvements and new functionalities. That being said, we are currently planning to release a patch to declab 2.10, to address the topics you highlighted among some bug fixes.
I will keep the issue open and will reach out again once the updated version is published. In the meantime, as we are actively working on the new version, should you have any further suggestions or wishes, please do not hesitate to share them either using GitHub issues by contacting me at jonas.tamimi [at] materna.de!


Reactions API Preview has been updated.

https://developer.github.com/changes/2016-06-07-reactions-api-update/
This is a breaking change that we'll need to update go-github for. It looks like a straightforward change on our side. And (from a cursory look) IMO this is a better API; at least for my use cases. 🎉
/cc @gmlewis

Assigning myself to this issue; I'll have a PR that fixes it soon.


How to implement this?

Great that you give us all these considerations, but how do I actually apply these settings in Office 365?

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

ID: c354faae-625c-2b36-2a77-4ed2962feff3
Version Independent ID: 1dd96166-8bae-fc8c-491f-99ca45f5b058
Content: Password policy recommendations for Office 365
Content Source: Office365-Admin/misc/password-policy-recommendations.md
Service: o365-administration
GitHub Login: @KwekuA
Microsoft Alias: kwekua

@Haribo112 - Thank you for submitting feedback.
I think the best way forward is if you can open a service ticket in your tenant so this can get resolved ASAP. Based on outcome, let me know if it is something that can be called out in the docs.
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.


Source Location Problem

Modify the code to identify the source location

Solved with commit "receptor location identified and a map created" on July 12, 2023


Partof

Summary
Implement the capability to merge multiple compose services into a single deployment or statefulSet (using pods with multiple containers). See internal ticket APPX-93.
This is a breaking change! Previous versions of k8ify produced one ingress resource per port; this new implementation will combine all ports of the same compose service in a single ingress (as it is done for the service). This changes the behavior even if one single ingress was in use previously, because the port number is not part of the ingress name anymore. To migrate existing deployments you will have to manually delete the old ingress resource (the one with the port number in the name); k8ify will create the new one.

LGTM, using it on project worked without issues :)
Thanks !


Update 2020-05-04_05-00-00_community-en.md

updating broken links and adding an updated version
What? (required)
What has changed/been added?
Why? (optional)
Why has this change occurred if applicable?
How can this be tested? (optional)
Describe how this change can be tested
Screenshots (optional)
Upload some screenshots of the changes

fix for issue 171


[BugFix] update error msg when join cannot be implemented by NLJ

What type of PR is this：

[X] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

In the previous version, when a join cannot be implemented by Nestloop Join (NLJ), error msg is 'no executable plan for this sql'. In this pr, we change the error msg more explicit like this ' Unsupported '%s' clause with this ['%s'] correlated condition.' For example,
> select v1 from t0 where 1 in (select v4 from t1 where 1 = v2 + v5)
Unsupported 'LEFT SEMI JOIN' clause with this ['add(2: v2, 5: v5) = 1'] correlated condition.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

[FE PR Coverage Check]
:heart_eyes: pass : 7 / 7 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/optimizer/rule/implementation/NestLoopJoinImplementationRule.java
7
7
100.00%
[]

run starrocks_admit_test

run childrenOutputProperties

@Mergifyio rebase

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 7 / 7 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/optimizer/rule/implementation/NestLoopJoinImplementationRule.java
7
7
100.00%
[]


chore: bump for 0.4.8 release

Prepare for the 0.4.8 release.
Update Cargo.lock file.

We need two reviewers for PRs with the possible stability impact tag


[Build] Better fix for break

This is an improvement on #867, which addressed a build issue by excluding the latest release of llama-cpp-python when setting up the GitHub runner machines. The assertion was checking for a self-consistency issue, but it need not be fatal (as evidenced by the fact that we turn it off for Phi3 models). However, if the check fails, it would be helpful to know in bug reports from subsequent failures. Hence, switch to warnings.warn().

Codecov Report
Attention: Patch coverage is 77.77778% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 56.76%. Comparing base (cb263a8) to head (c1d67ba).

Files
Patch %
Lines

guidance/models/_model.py
77.77%
6 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #871      +/-   ##
==========================================
- Coverage   56.91%   56.76%   -0.16%
==========================================
  Files          63       63
  Lines        4570     4570
==========================================
- Hits         2601     2594       -7
- Misses       1969     1976       +7

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

As I think about this more, I'm getting some apprehension about universally changing this to a warning -- failing this cross-assertion could lead to pretty dramatic model performance implications that are almost silent, if people ignore the warning.  I agree it wasn't the nicest user experience to just error out here, but imo every model that has this issue should be investigated by hand (as we did for Phi-3). It's not an error in a code-failure/segfault sense, but it usually is indicative of real problems in how guidance (or the underlying model) is tokenizing text.
Maybe the best thing to do here is to just update to have much stronger language on asking for a github issue? Curious about thoughts here from @slundberg and @hudson-ai .

Fair points - this is why I'd like to have:

A clear list of 'core' models, where we will (if necessary) 'special case' guidance code to get them working
More 'unit-like' tests of things like tokenisers, to catch behaviour like this, where a 'round trip' test fails (again, per many other threads, I know that only the bytes->tokens->bytes round trip is expected to work; tokens->bytes->tokens may well not)


Running inference with Trits is much slower than running model Pytorch directly

Description
A clear and concise description of what the bug is.
Running inference with Trits is much slower than running model directly
Triton Information
What version of Triton are you using?
Triton 20.03 Pull from nvcr
Describe the models (framework, inputs, outputs), ideally include the model configuration file (if using an ensemble include the model configuration file for that as well).
Python 3.6
GPU: 1660Ti
Expected behavior
A clear and concise description of what you expected to happen.
When I run pytorch model directly, performance is about 25fps, but when running on Trion, performance is only ~9fps

The communication between Triton and its client goes through network, so the network overhead will have impact on the inference latency. It will be more helpful to compare the results without those overhead. If you are using perf_client for profiling Triton performance, you can take a look at the request latency reported by the server, that is more meaningful when comparing with native performance.

Closing. Please provide perf_client output if you have followup questions.


Project using go-prompt (abs-lang)

Hello there,
We are working on a new scripting language using go-prompt, and we want to add it to "Projects using go-prompt" session.
ABS is a scripting language that works best on terminal. It tries to combine the elegance of languages such as Python, or Ruby, to the convenience of Bash.
Thanks for this amazing lib 👍

Amazing! I added abs at https://github.com/c-bata/go-prompt/commit/ea6035601114c4fb96ea02b79935762df24a99e5


Generate Benchmarks for Sk

If we can generate benchmarks for the in circuit ECC operations which use zerocaf they can be formatted and stored. As this can be the best way to compare to both other embedded curve(s) and the cost of doing it without an embedded curve.
If we can get it started with the Sk proof, that would be ideal.

Once #5 is closed you will be able to start.
Anyway, there are a few considerations regarding the constrains that we need to think about.
Also, the code can be optimized, now it's just prototyping the solution.


Update to be compatible with RTFM

Semi-reverts https://github.com/nrf-rs/nrf52-hal/pull/95.
This updates the interfaces to be compatible with RTFM. RTFM does not give the user code access to the NVIC, and enables interrupts automatically.
I suggest we maintain this pattern moving forward for peripherals that have their own interrupt enable bits.
CC @hannobraun @bjc

This is fine, but a bit clunky, IMHO. Since these are the only two peripherals that have enable_interrupt methods, maybe it'd be better to just remove the method entirely?
In the case of rtc, we're needlessly enabling the interrupt potentially multiple times, for each possible event type, and in the case of timer it hides the use of the comparator, which isn't necessarily desirable either.
I don't have a problem with this merge, but it does make it more important to figure out how to abstract the NVIC, I think.

Hi, can we decide on this and move forward?
Currently I am extending the HAL manually for our use, and it would be nice to have support for systems with external interrupt setup (such as RTFM).

To clarify: I'd like to give others the chance to comment for a few days. No need to leave this open for a few more months :-)

Sounds good :)

I have nothing agindt this (yet). Imo we should merge this. Most people use RTFM I guess and it shouldn't hinder others if I understand right.

Okay, I stumbled over something today that turns out to be relevant to this issue: NVIC::enable and disable have been deprecated in the latest cortex-m version, and the replacement methods don't need an NVIC instance.
So the correct thing to do here is to upgrade to the latest cortex-m version and figure out whether we want to do the NVIC stuff in those methods. One way or another, it won't interfere with RTFM, if I understand correctly.
That said, this pull request has been open for months (+ 2 days), so I'm just going to merge this now to get @korken89 unblocked, and leave other improvements for later.

Thanks!


hprc browser doesn't know it's own DNS name

if you do a view table schema on one of the tracks, you see
http://<IP_ADDRESS>/hub/HG00621.1/consensus.cmg-chm13.gencode38.bb
I am not 100% sure, but I think this has the risk of breaking sessions

I think this was fixed in commit 1d76d3f. We will see when we switch to the updated browser. I'll leave this ticket open as a reminder to check.


ref #65, new formula function: DBCS

PR Details

Description

Related Issue

Motivation and Context

How Has This Been Tested

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (9b07898) 99.06% compared to head (968d3e7) 99.06%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1791   +/-   ##
=======================================
  Coverage   99.06%   99.06%
=======================================
  Files          32       32
  Lines       28560    28584   +24
=======================================
+ Hits        28293    28317   +24
  Misses        177      177
  Partials       90       90

Flag
Coverage Δ

unittests
99.06% <100.00%> (+<0.01%)
:arrow_up:

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

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


I wish /docs/admin/admission-controllers/ documented the recommended set of admission controller plug-ins to use for Kubernetes 1.2

I wish /docs/admin/admission-controllers/ documented the recommended set of admission controller plug-ins to use for Kubernetes 1.2
That page currently documents the set for version 1.0

http://kubernetes.io/docs/admin/admission-controllers/#is-there-a-recommended-set-of-plug-ins-to-use
Done and Done :)


Person Component

first name last name? two separate http://element.eleme.io/#/en-US/component/input


Adds ClickedEntityGroup events

This adds the click events for web; they pretty much have the same schema as the tap events. I also added a context module for the new topWorksRail on the artist pages that FX is working on. One question - I did not keep module_height for the click events. Do we think we need it on web?

@abhitip on 'module_height' i think design is best to consult on that ("will we be interested in the visual treatment of modules and their effectiveness of converting traffic?") but i also think if we include it at all, it should be optional. not noticing a lot of early appetite to track it.

Okay got it! I'll leave it out for now and we can revisit later.

:rocket: PR was released in v1.4.0 :rocket:


Strange OpenGL output in place of toc?

I am getting this strange output in place of my toc when trying the read the README.md file in text editor:
<!-- toc -->

- [Enable OpenGL](#enable-opengl)
- [Tell Xorg to use the nvidia driver (also valid for Wayland)](#tell-xorg-to-use-the-nvidia-driver-also-valid-for-wayland)

<!-- tocstop -->

These are not the contents or headings of my README.md.
The code I ran was:
$ ./node_modules/.bin/markdown-toc -i README.md

Am I doing something strange or wrong?

So Apparently the issue was that my markdown editor added code block () around <!-- toc -->. which caused that weird output to appear. Removing the  fixed the issue.

So Apparently the issue was that my markdown editor added code block (```) around <!-- toc -->. which caused that weird output to appear. Removing the ``` fixed the issue.


Set self.abstract_class = true

base_class in ActiveRecord returns
PublicActivity::ORM::ActiveRecord::Activity incorrectly. Since
PublicActivity::ORM::ActiveRecord::Activity is really an abstract
class for all intents and purposes, it should be declared so that
base_class returns PublicActivity::Activity instead.
My use case was using the unread gem to track who had read an
activity. It uses base_class to record the polymorphic type of the
readable which is the Activity in this case.
All the tests passed with this addition but I’ve added another to
ensure that base_class returns the correct value.

I've hopefully fixed the test so that it only runs with ActiveRecord :)

The warnings in the test suits are insane BTW :) Probably a case for a seperate PR. Or we could just turn them off with a rake config change.

Yeah, PublicActivity has a bucket load of legacy in it. I'm working slowly on a completely rewrite to target only ActiveRecord and Rails 5+ now.
Thanks for this!

No worries :D

I assume you'll also support 4.2 if Rails 6 hasn't been released yet? ;)

Yeah I think supporting 4.2 makes sense, since it doesn't have that many changes compared to 5 :) Only < 4.0 is very problematic.

Haha, that's good since I'm not off 4.2 yet :)


Have Trouble importing datasets

I'm failing to import transformers (v4.0.0-dev), and tracing the cause seems to be failing to import datasets.
I cloned the newest version of datasets (master branch), and do pip install -e ..
Then, import datasets causes the error below.
~/workspace/Clone/datasets/src/datasets/utils/file_utils.py in <module>
    116 sys.path.append(str(HF_MODULES_CACHE))
    117
--> 118 os.makedirs(HF_MODULES_CACHE, exist_ok=True)
    119 if not os.path.exists(os.path.join(HF_MODULES_CACHE, "__init__.py")):
    120     with open(os.path.join(HF_MODULES_CACHE, "__init__.py"), "w"):

~/.pyenv/versions/anaconda3-2020.07/lib/python3.8/os.py in makedirs(name, mode, exist_ok)
    221             return
    222     try:
--> 223         mkdir(name, mode)
    224     except OSError:
    225         # Cannot rely on checking for EEXIST, since the operating system

FileNotFoundError: [Errno 2] No such file or directory: '<MY_HOME_DIRECTORY>/.cache/huggingface/modules'

The error occurs in os.makedirs in file_utils.py, even though exist_ok = True option is set.
(I use Python 3.8, so exist_ok is expected to work.)
I've checked some environment variables, and they are set as below.
*** NameError: name 'HF_MODULES_CACHE' is not defined
*** NameError: name 'hf_cache_home' is not defined
*** NameError: name 'XDG_CACHE_HOME' is not defined

Should I set some environment variables before using this library?
And, do you have any idea why "No such file or directory" occurs even though the exist_ok = True option is set?
Thank you in advance.

I'm sorry, this was a problem with my environment.
Now that I have identified the cause of environmental dependency, I would like to fix it and try it.


add: .vscode, rustfmt, development setup for nix

Hey, in this PR I wanted to add configurations for vscode, rustfmt and easy development setup for nixos. All of my future PRs will require this.

Incase you dont know what nixos is? Check https://nixos.org/ 😄

@scopsy Mind reviewing? :D


Options Editor crashed

Problem Description
mitmproxy has crashed. in Options view
Steps to reproduce the behavior:

Build mitmproxy from source code:

python -m venv venv
venv\Scripts\pip install -e .[dev]
run success

go to Options view -> Edit options

System Information
Paste the output of "mitmproxy --version" here.

maybe


[Bug Report] Theme Chalk library gone from unpkg.com

Element UI version
2.15.11
OS/Browsers version
Mac Os Ventura
Vue version
2.7.14
Reproduction Link
https://jsfiddle.net/r8ofj5eL/
Steps to reproduce
I was loading the chalk theme from here:
https://unpkg.com/element-ui/lib/theme-chalk/index.css
which redirected to the newest version here:
<EMAIL_ADDRESS>Now, this is gone. Why? Why would you delete it from there?
What is Expected?
I expect the file to stay on upkg.com
What is actually happening?
It is gone from unpkg.com

try 2.15.12

try 2.15.12


Unable to build due to conan profile not existing?

Out of the box, I get this error when building through the janus script:

Step 7/13 : RUN 	export CC=`conan profile get env.CC ue4` && 	export CXX=`conan profile get env.CXX ue4` && 	conan install . && 	mkdir /tmp/MediaIPC/build && 	cd /tmp/MediaIPC/build && 	cmake -DCMAKE_BUILD_TYPE=Release .. && cmake --build .
 ---> Running in d7bba8a9ca4f
ERROR: Key not found: 'env.CC'
ERROR: Key not found: 'env.CXX'

Then later:
boost/1.67.0@conan/stable: WARN: Lib folder doesn't exist, can't collect libraries: /home/ue4/.conan/data/boost/1.67.0/conan/stable/package/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9/lib
Then:

CMake Error at conanbuildinfo.cmake:644 (message):
  Incorrect 'clang', is not the one detected by CMake: 'GNU'
Call Stack (most recent call first):
  conanbuildinfo.cmake:227 (conan_check_compiler)
  CMakeLists.txt:19 (conan_basic_setup)

This is on Ubuntu 18.04. I believe I have installed all the correct dependencies.

I've fixed this so that it builds successfully with the latest changes in how conan-ue4cli handles profile generation, but I'd still strongly recommend using Pixel Streaming for Linux rather than the old UE4Capture plugin that this demo is built around.


Get error on default configuration with default nextjs 13 installation

Experiencing the same issue. ☹
This is my components.json file
{
  "$schema": "https://ui.shadcn.com/schema.json",
  "style": "new-york",
  "rsc": true,
  "tailwind": {
    "config": "tailwind.config.js",
    "css": "app/globals.css",
    "baseColor": "slate",
    "cssVariables": true
  },
  "aliases": {
    "components": "@/global/components",
    "utils": "@/lib/utils"
  }
}

Restarting the server after the installation causes 404 Error on any route I visit.

This issue can probably solve the problem https://github.com/shadcn/ui/issues/755#issuecomment-1612617783.

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


Fixes 17413: Fix one sided tests for columnValueLengthsToBeBetween and columnValuesToBeBetween

Describe your changes:
Fixes https://github.com/open-metadata/OpenMetadata/issues/17413

Type of change:

[x] Bug fix

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

[x] I have added a test that covers the exact scenario we are fixing. For complex issues, comment the issue number in the test for future reference.

not sure if py-tests failures are relevant though, they seem we might need to take a look


none Unique index.

It seems that if you index a field as not unique and submit many documents with the matching index field that those _ids are not listed in an array but instead are concat into one field. Creating a very long string.
This is not happening in the persisting of the data but in the actual indexing in the AVL tree.
New Tests need to be written testing non-unique issues. on TeDB and BTT

Looks like this issue is with TeDB. when inserting an index, need to insert  (key, [value]) not (key, value)


E2E Error Output Tests

This adds the start of a testing of the error output from spin. The end vision is an easy way for developers to add tests for some sort of broken Spin experience and test the actual stdout/stderr output against some expected output.
We're not quite there, but this PR makes significant changes to the e2e testing framework to allow for test Spin apps to fail with (necessarily) failing the test. I've added one test as well to give an idea of how these tests might work in the future. However, I expect the experience of adding new tests to improve with time as we bake in more convention on how to write such tests.
Note: I did quite a bit of clean up along the way which might make this somewhat difficult to review. I apologize for this, but I hope the intention is still clear. Please let me know if you need more explanation of the changes, and I can write up a more thorough summary.

Annoyingly the broken test consistently works locally. Looks like for some reason there's some extra whitespace produced by spin when running on the CI server.


Logging User in before routing?

I've a number of routes for my website, which check for special permissions.
The most basic being isAdmin
routes.php
Route::group(array('before'=>'isAdmin'), function() {
	Route::resource('users', 'UserController');
	Route::resource('user', 'UserController');
});

filters.php
Route::filter('isAdmin', function($route)
{

	if (User::isAdmin() == false) {
		return Response::make('Unauthorized', 401);
	}

});

Obviously, if I pass this this through as  ['before'=>'isAdmin|oauth'] it will fail because Auth is not defined.
How would I go about checking for this as well as authenticating the access_token at the some time?
BaseController.php
<?php

class BaseController extends Controller {

	protected $currentUser;
	protected $isAPI;

	function __construct() {

		if (Input::has('access_token')) {
			$user_id           = Authorizer::getResourceOwnerId();
			$this->isAPI       = true;
			$this->currentUser = Auth::loginUsingId($user_id);
		}
		else {
			$this->isAPI       = false;
			$this->currentUser = Auth::user();
		}

	}

I believe the library has it's own filter:
Route::group(array('before' => 'oauth'), function()
{
// your routes go here...
});

@dud3 but that still doesn't solve my problem whereby User is still undefined?

It seems like I didn't understand you're requirements completely.
Do you want to associate the user with the token somehow, or you just want to authenticate the user and then get the token ?
Password grant type example:
'grant_types' => [
        'password' => [
            'class'            => 'League\OAuth2\Server\Grant\PasswordGrant',
            'access_token_ttl' => 604800,

            // the code to run in order to verify the user's identity
            'callback' => function($username, $password) {

                $credentials = [
                     'email'    => $username,
                     'password' => $password,
                 ];

                 if (Auth::once($credentials)) {
                            return Auth::user()->id;
                 } else {
                     return false;
                 }

            }
        ]...

My password grant seemed to be working fine
'grant_types' => [
    'password' => [
        'class' => '\League\OAuth2\Server\Grant\PasswordGrant',
        'callback' => function($username, $password) {
            if (Auth::attempt([
                'email'    => $username,
                'password' => $password,
            ])) {
                return Auth::id();
            }
            else {
                return false;
            }
        },
        'access_token_ttl' => 604800
    ],
    'refresh_token' => [
        'class' => '\League\OAuth2\Server\Grant\RefreshTokenGrant',
        'access_token_ttl' => 604800,
        'refresh_token_ttl' => 604800
    ]
],

I'm already able to authenticate a user, provide a token and give them access with using this token. My issue is that parts of my site already have specific filters such as isAdmin. Obviously these pages can only be served if the user has admin privileges. Essentially I'm trying to authenticate a user using oauth while simultaneously checking (using filters.php) if they have admin privileges

How about you modify the oauth_access_tokens and crate another column let's say user_id and then link the permissions based on that.
But for this to work, you might need to create you're own grant type.

With oauth, you can't access user from Auth anymore.
In your isAdmin filter, you can get access_token from Input::get('access_token') and use that token to get session_id (in oauth_access_token table).
Once you get oauth_sessions from the session_id, there is a field named "owner_id" which is your logged in user that associate with the input token. Use that id to query user table to see if he/she is admin.
Your route can be something like this.
Route::group(array('before' => 'oauth'), function() {
     // Normal Get / Post ...
     Route::group(array('before' => 'isAdmin'), function() {
         // Admin stuff
     }
}

Hope this help.

Thanks for your help teamsoo,
Not sure how I didnt think of this,
Heres what I ended up doing for anyone in the future looking to achieve the same thing
Created a new filter in filters.php
Route::filter('setUser', function()
{
	$user_id = Authorizer::getResourceOwnerId();
	Auth::loginUsingId($user_id);
});

then in routes.php
Route::group(['prefix' => 'api/v1', 'before' => 'oauth|setUser'], function(){

	Route::group(array('before'=>'isAdmin'), function() {
		Route::resource('users', 'UserController');
		Route::resource('user', 'UserController');
	});

	Route::get('projects', ['uses' => 'ProjectController@index']);
	Route::get('project/{project_id}', array('uses' => 'ProjectController@show'));

	// Nested Project Operations
	Route::group(['prefix' => 'project/{project_id}'], function(){

		// Audit Trail
		Route::resource('audit', 'AuditController');

	});
});

In my BaseContoller.php I set isAPI so i can test on this on my controllers and either return a view or json
	function __construct() {

		if (Input::has('access_token')) {
			$this->isAPI       = true;
		}
		else {
			$this->isAPI       = false;
		}
	}


Convert sol-util functions to use sol-buffer

Functions like sol_util_load_file_raw() and sol_util_fill_buffer()should be using sol_buffer instead of own buffer manipulation functions.
This also have nice side-effects like allowing sol_util_load_file_string() getting the sol_util_load_file_raw() buffer and being able to ensure the trailing \0 without the need of a `realloc()``
Also report on functions that would help this task, I already envision sol_buffer_trim() that calls sol_buffer_resize(buf, buf->used) that could be used at the string version before it returns. Maybe sol_buffer_steal(buf) to take buf->data and reset buf->used = 0; buf->capacity = 0;?

@barbieri
...that could be used at the string version before it returns

So, sol_util_load_file_string() should start returning a sol_buffer? Won't it make its usage cumbersome?

I was thinking more in the lines of sol_util_load_file_raw() returning a buffer and sol_util_load_file_string() calling sol_util_load_file_raw(), forcing a trailing \0, calling sol_buffer_resize(buf, buf->used) and stealing the buffer's data, returning it.
Maybe these new calls should be introduced to help make it clear:
int sol_buffer_trim(buf) {
    return sol_buffer_resize(buf, buf->used);
}

void *sol_buffer_steal(buf, size_t *size) {
    void *r = buf->data;
    if (*size) *size = buf->used;
    buf->data = NULL;
    buf->used = 0;
    buf->capacity = 0;
    return r;
}

Ok. Got it.

@barbieri does #589 close this issue? Is there anything missing?

yes.


Versioning and 1.0 pre-release

This PR adds versions to the docs so we can update for 1.0 and keep old docs around for pre-1.0 users. A couple notes for anyone committing edits here:
The /docs/ directory you’re used to updating is pinned to 1.0. Since we’re still in alpha, this is not the content that displays on the published site by default, but it is accessible at /docs/1.0/*, flagged as docs for the alpha release.
To update 0.82 docs in the repo, update the appropriate file in the /versioned_docs/0.82/ directory. Note that sidebars are also versioned in /versioned_sidebars/ if you’re making changes there.
Updates to the 0.82 docs are relatively modest here, but a ton is changing preliminarily on the 1.0 side, most notably the actor-to-component terminology change, reworked “component” and “provider” concept pages, and a revamped policy service page, in addition to some restructuring and many nits.
This PR is designed to be merge-able in advance of wash+wadm releases and any further 1.0 pre-releases—once reviewers are good with it, it should be ready to go.

Under "Workflows" I see "Building Actors"
https://deploy-preview-325--dreamy-golick-5f201e.netlify.app/docs/1.0/developer/workflow
And mention of signing.

Under "Workflows" I see "Building Actors"
https://deploy-preview-325--dreamy-golick-5f201e.netlify.app/docs/1.0/developer/workflow
And mention of signing.

Fixed!


Does not work for dynamic work queues

The AMQP consumer is reusing the same queue/exchange values for every subsequent sampler execution. This does not work for use-cases which have a dynamic work-queue to pull the mq-message from.

Pull requests welcome :)

Has anyone found some fix for that - or is there some other fork?
Even the example RPC_Load_Test from the repo is not working with some normal "RPC-Echo server".
In every new iteration the message is send with the correct "reply_to" header set, but the consumer is not consuming from that new random queue. In Rabbit-Web you also see, that just one random-return queue is created and it never changes.

Have a nice solution for RPC testing now #31

Hi, has any one used it to connect to azure iot hub. please let me know how to connect it to azure, i already tried for rabitmq.


Support custom actions in pretty-select choices

@stevemolitor Is this a stale PR? It looks like a version of this code is already in master. If so, let's close this one


Question: Jobs created with this library are expected to keep running even after app is terminated?

That's the only library that I found to handle jobs specifically on expo. But after some tests, I found that my jobs just stop if I close my app in the middle of the process. That's my code:
React.useEffect(() => {
   ExpoJobQueue.configure({
     onQueueFinish: () => {
       console.log("Queue stopped and executed")
   },
})

ExpoJobQueue.addWorker("testWorker", async (payload) => {
   return new Promise((resolve) => {
     syncAll().then(resolve)
  })
})

ExpoJobQueue.addJob("testWorker", { text: "THIS IS FANTASTIC", delay: 30000 })
ExpoJobQueue.start()

@robsonbbs unfortunately this library doesnt do background handling yet...


support window functions

Which issue does this PR close?

Closes #542 .
Rationale for this change
As stated in #542, datafusion adds SortExec and RepartitionExec based on the window clauses partition by and order by (See enforce_distribution.rs in datafusion and WindowAggExec::required_input_distribution .
From what I can see, this restriction on window functions in ballista was put way back when datafusion was in the same repo as ballista and window function support was still in its infancy. Now that datafusion enforces the right hash distribution by placing repartition nodes, I believe ballista currently already supports window functions.
I have tested partitioned, unpartitioned and sorted window expressions with ballista with this PR and compared them with datafusion, it seems to be working as intended. Parititioned window functions get repartitioned by their partition expressions and are parallelised.
Not sure what is the best way to test this given distribution handling is done by datafusion here.

What changes are included in this PR?
Allow queries with window expressions

Are there any user-facing changes?

@milenkovicm should be good to go. I also enabled a previously ignored test, as the fix it mentions is merged for a while now


glog installation

I think the glog installation part might look like this maybe?
$ sudo apt install autoconf automake libtool libunwind-dev
$ git clone https://github.com/google/glog.git
$ cd glog
$ git checkout v0.5.0
$ mkdir build
$ cd build
$ cmake ..
$ make -j4
$ sudo make install

Hi @mjlee111
Thanks for pointing out this issue. The README has been updated.


Config reference documentation update

Resolves #9036
This PR is a few commits on top of each other.

Inroduction of a JSON reader and writer for Metadata (so we can remove dependency on JSON-P/JSON-B from our tools, such as native image extension, config metadata write, and we can replace the text format in service registry with JSON which is better suited for the task)
A small fix for builder codegen, so it does not generate service.loader file when empty
config-metadata module is re-structured, as it now has multiple child modules; I did not change group id of existing modules for backward compatibility; introduced codegen and docs modules, deprecated processor module
Fix to all pom files to use codegen, add plugin dependencies to correctly order Maven reactor, remove processor dependencies from regular Maven dependencies etc.
A small fix to logging-jul, to correctly print out how logging is configured even when not running in native image
Update to docs pom file, to generate the config reference documentation with profile configdocs
Update to the actual .adoc files based on all of the above

This is also a must-have for 4.1.0, as service registry needs to use it

Closed as this is separated into multiple PRs


Export from station thumbed up tracks

When I run this on the thumbed up tracks from a station, the object is empty. (My station thumbs up do not appear in the profile/thumbs list).
Would it be possible to modify this to work on an individual station?

Hi @tilsen
The script was created to run on the global thumbs up page rather than a specific station. Unfortunately the page structure is different for stations than it is for the global thumbs up page.
While researching this, however, I found that the station page includes both Thumbs Up and Downs, so I'm interested in getting this working.
On the global thumbs up page, you should be able to see which station the thumbs up originated from:

In addition, I'm updating the script today to categorize thumbs up by station! So you should be able to identify which songs are related to which station.
Hopefully that helps for now, until I have time to check out the station pages and get those working. I've marked this as an enhancement.
Thanks for the idea for a station specific script! 🎵

Unfortunately, my global thumbs up page shows none of my station thumb ups, so I don't have the option to export anything from my global thumbs up page. From what I can tell this is an issue that affects people with very old accounts--all of my stations are from more than 10 years ago. In the meantime, I'll be eagerly awaiting your enhancement! Thanks.

@jeffsikes A bump on this feature.  The global thumbs page truncates at 800 total.
I have two stations, one with 849 thumbs up and 131 thumbs down and a "retuned" version with 251 thumbs up.  For a total of 1231 thumbs that should be reported.
I verified that the export stops at 800.  It's not a situation where the 800, means 800+, and the extra data shows up if you scroll down far enough.


Terminal Switcher:  Rename should be reachable with first tab

While testing #31784.

VSCode Version: Code - Insiders 1.15.0-insider (f39385b8326671dc6389f21f8c2e2f6afdb352a1, 2017-08-01T05:07:59.267Z)
OS Version: Darwin x64 16.7.0

Steps to Reproduce:

Open 3 terminals
Select Terminal: Switch Active Terminal from the command palette
ArrowDown to the last terminal in the switcher
Tab to rename the terminal
-> keyboard focus disappears from the input box (and seems to be nowhere). I need a second Tab to move the keyboard focus to the rename action in the selected row. A single Tab should suffice.

This is also an issue with task configuration


YAML error in settings

Hi, I found this as the Hub script to pick up new settings crashed reading the YAML...

I'm working on a PR for this


not  found  preview_store.g.dart

not  found  preview_store.g.dart

run this command -> flutter packages run build_runner build

Hey  guolidaxian, generated files are not pushed into repository.
To generated these files, please run following code in terminal / CMD.

open 100ms-flutter/example folder in terminal/CMD
flutter pub get
flutter packages run build_runner build --delete-conflicting-outputs


Upstream Update (guild-events)

The guild-events branch is 3 commits behind dev.
This pull request was created automatically. If you wish to be excluded from automatic pr creation, open an issue here. Contact DonovanDMC if you find any other issues.

upstream branch name change


Correct return values for lstrcpynW, lstrcpyW and lstrcatW functions.…

… These now return LPWSTR.

Hopefully nobody was using these functions in the first place.


Automated cherry pick of #2287: fix ceph rescue mode migrate

Cherry pick of #2287 on release/2.10.0.
#2287: fix ceph rescue mode migrate

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@06f7589). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             release/2.10.0   #2290   +/-   ##
================================================
  Coverage                  ?   7.56%
================================================
  Files                     ?     410
  Lines                     ?   63620
  Branches                  ?       0
================================================
  Hits                      ?    4811
  Misses                    ?   58398
  Partials                  ?     411

Flag
Coverage Δ

#aFlag
7.56% <ø> (?)

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 06f7589...cb4476f. Read the comment docs.

/lgtm

/approve


chore: replace Marshal with MarshalVT

Description
This PR replaces Marshal with MarshalVT.

[x] CHANGELOG update is not needed
[x] Documentation is not needed

Testing

[x] inspected CI results

Automated testing

[x] modified existing tests
[x] contributed no automated tests

How I validated my change
CI

drop compatibility function
Let's do it with all other functions.

This PR replaces Marshal with MarshalVT.

Could you add the "why" part to the description, please?


Applicant email confirming request is approved

As an applicant, I want an email on whether my request has been approved
Acceptance Criteria
Scenario 1: xxxxxx

GIVEN ... (you or your condition)
WHEN ... (what you do)
THEN ... (what you see)

Scenario 2: xxxxxx

GIVEN ... (you or your condition)
WHEN ... (what you do)
THEN ... (what you see)

Scenario 3: xxxxxx
...
Dependencies? What is the impact of this dependency? (If so, link dependency in the ticket, make it visible in a team´s backlog)
Validation Rules? (If yes, list here)
Design
@xxx - please link the Design here
Definition of Ready

[ ] Is there a well articulated User Story?
[ ] Is there Acceptance Criteria that covers all scenarios (happy/sad paths)?
[ ] If there is a user interface, is there a design?
[ ] Does the user story need user research/validation?
[ ] Does this User Story needs stakeholder approval?
[ ] Design / Solution accepted by Product Owner
[ ] Is this user story small enough to be completed in a Sprint? Should it be split?
[ ] Are the dependencies known/ understood? (technical, business, regulatory/policy)
[ ] Has the story been estimated?

Definition of Done

[ ] Passes developer unit tests

[ ] Passes peer code review
[ ] If there's a user interface, passes UX assurance
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Confirm Test cases built and succeeding
[ ] No regression test failures
[ ] Test coverage acceptable by Product Owner
[ ] Ticket ready to be merged to master or story branch
[ ] Developer to list Config changes/ Update documents and designs
[ ] Can be demoed in Sprint Review
[ ] Tagged as part of a Release
[ ] Feature flagged if required
[ ] Change Management activities done?

Closing - applicants receive an email after submitting a request via webform, All other correspondence methods currently covered in other tickets related to Contacting an Applicant


HTYI: If the game generates a hashtag for a player, they are able to vote for their own submission

If a player runs out of time and the game has to generate a hashtag for them, that player will get the option to vote for that generated hashtag in the voting round, and receive points for the vote.

adding game design label: this label will be used for any bug or issue that creates game design exploitation opportunities. It doesn't break the game, but can kill the play experience

Was unable to reproduce.

Unable to reproduce


ValidationError: relax message type (from String to Any)

In some applications the user may want to separate validation logic from
error generation and i18n. This allows to to return any kind of data as
message so that the UI can use it to do further processing.
This code could look a little unclean. A better solution that doesn't
break backwards compatibility would be to add a new API similar to
getValidationErrorMessage, but that would require a lot of work and code
complexity for a not very important use case.
Fixes: #53

Thx!

Released in https://github.com/gcanti/tcomb-validation/releases/tag/v3.3.0


Feature/custom tagging to Worker Nodes via Launch Template

Issue #, if available: #485
Description of changes:
custom tagging to Worker Nodes via Launch Template
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@shapirov103 All issues are fixed. Please kick e2e

/do-e2e-tests

Hi @praveenperera All these are great suggestions and definitely qualifies for a EKS CDK Blueprints Patterns which many can benefit from. Do you mind submitting a Blueprints Pattern with your idea.

@elamaran11 I'm not sure the blueprints pattern is the correct place.
Because the way it is right now. The new LaunchTemplateProps key isn't flexible enough to do any of this.

@praveenperera Since your request was focussed on diskSize which is not possible to set today, i was thinking its better to create a pattern which does what you are suggesting something like Take diskSize and create a BlockStorage in the launchTemplate.
But on the flip side i do understand where you are coming from. I would recommend you to submit a PR or Draft PR we can review and take this to a path which satisfy your request. Do you have any driver or a compelling need behind this ask?
@shapirov103 Thoughts?

Ya I’ll think about it and try and get a PR this week.
My use case is pretty simple I need to add some tags to my nodes. And one set of nodes needs a 500GB HD.

Sounds good. Looking forward for the collaboration and PR.

@praveenperera @elamaran11 the approach that we took was to simplify launchtemplate creation with custom tags. More involved use cases were supposed to create a launch template outside of the blueprint (e.g. separate stack) and use the launchTemplateSpec attribute of the node group to pass it.  This approach is supposed to be achievable now.
The next approach that I had in mind was registering a custom launch template  (e.g. this) as a resource provider and referencing its ID in the launchTemplateSpec similar to using CDK constructs with resource providers.
However, supporting disk size with launch template ootb seems reasonable to me, we can also extend the framework to make it happen (I assume configuring block devices).

@shapirov103 Makes sense.


Install v0.4 by default

Also change to use configure; make rather than the Makefile directly. And add the build dependencies for HEAD installs.

Fixes #3


Cleanup

[x] renamed heatmap generator
[x] aggretion time is a param
[x] add heatmap generator to launch
[ ] add sim option in factory.launch
[ ] repo structure should be as per THMHO

Closing this in favor of https://github.com/ipa320/THMHO_heatmap_generator/pull/7


ROU-4763: Add ability to to GoogleMaps to add localization by setting the language and region

This PR is for adding the Localization feature to GoogleMaps, StaticMap and SearchPlaces  by setting the language and region through OptionalConfigs on each component.
What was happening

The user to apply a localization on a google maps, he need to add it on APIKey value as workaround. With this, the google maps will have the language and region applied.

What was done

Add the Localization parameter into OptionalConfigs on GoogleMaps, StaticMap and SearchPlaces
Review the TypeScript to add the localization (language and region) into consideration on loaded script by google

Test Steps

Open each Sample Page
Expected: Map will rendered in Japan language (On StaticMap, you can change the language and toogle the map visibility to apply the changes.)

Screenshots
(prefer animated gif)
Checklist

[ ] tested locally
[ ] documented the code
[ ] clean all warnings and errors of eslint
[ ] requires changes in OutSystems (if so, provide a module with changes)
[ ] requires new sample page in OutSystems (if so, provide a module with changes)

Member

Leaflet doesn't support localization?

@rugoncalves The leaflet doesn't have support for localization, this only on google maps.


ifconfig resource does not respect 'target' option properly

It looks like 'target' properly is not respected in resource 'interface'.
While documentatoin says :

target 	The IP address that will be assigned to the network interface. Default value: the name of the resource block. (See “Syntax” section above for more information.)

The code and behaviour looks like the variables are used separately.
for example resource like :
  ifconfig 'test-name' do
    device 'lo:1'
    target '<IP_ADDRESS>'
    mask '<IP_ADDRESS>'
    action [:add, :enable]
  end

will throw error :
           ================================================================================
           Error executing action `add` on resource 'ifconfig[test-name]'
           ================================================================================

           Chef::Exceptions::Exec
           ----------------------
           ifconfig lo:1 test-name netmask <IP_ADDRESS> returned 1, expected 0

           Resource Declaration:
           ---------------------
           # In /tmp/kitchen/cache/cookbooks/XXX/recipes/default.rb

           147:   ifconfig 'test-name' do
           148:     device 'lo:1'
           149:     target '<IP_ADDRESS>'
           150:     mask '<IP_ADDRESS>'
           151:     action [:add, :enable]
           152:   end
           153:

What platform are you on?

Does it really changes anything at this particular problem ? I was testing that on centos-6.5, but the code is broken in the platform-shared part too.
Here is how I expected it to be implemented when digging  though the code: https://github.com/chef/chef/compare/master...rafaltrojniak:ifconfig_target?expand=1

I'm not familiar with this provider's code, but I see what you're saying. Actually, since the resource already takes care of setting target to name when target isn't specified, what's needed here is just to substitute @new_resource.name for @new_resource.target. Want to submit a patch?

@juliandunn  Thanks for pointing that out, now I can see how this works when target is really used.
Apart from simple substitution, the detection logic seams weird too : https://github.com/chef/chef/blob/master/lib/chef/provider/ifconfig.rb#L70 (what does this 'inet_addr' really means ? is there 'inet6_addr' ?)
Really not sure how this supposed to be done, and I still have no CLAs signed.
I'm also not in hurry, so will leave this for someone with CLAs and more knowledge in code.

Yeah, I'm not sure either. Digging through the Git history it looks like this provider code dates right back to Chef's inception in 2009, so I'm not surprised it's wrong. My current Linux boxes don't have such an inet_addr field.
I'll minimally fix the problem you reported in this ticket, but it looks like this provider needs an overhaul generally.

@juliandunn  Yes, the provider does quite weird things and needs to be cleaned up. But fixing that particular problem with target is enough for me now.
Maybe if I will someday sign CLA, that would be my first Pull request :P

@rafaltrojniak that would be great :grinning:


What's the difference between “as list” and ”as iterable”?

I tried the first example
import jsonl

data = [
    {"name": "Gilbert", "wins": [["straight", "7♣"], ["one pair", "10♥"]]},
    {"name": "May", "wins": []},
]

jsonl.dump(data, "file.jsonl")  # as list
jsonl.dump(iter(data), "file.jsonl")  # as iterable

These two approaches both output
{"name": "Gilbert", "wins": [["straight", "7♣"], ["one pair", "10♥"]]}
{"name": "May", "wins": []}

Hello, I want to explain how the dump function from the jsonl library allows for efficient data dumping, whether from a list or any other iterable, such as a generator. This capability is particularly useful because it avoids the need to load all the data into memory before dumping it into a file, which can be beneficial when working with large volumes of information.
Usage Example:
Let’s say we want to dump a large number of records into a JSON Lines file. Instead of first creating a complete list of data (which could consume a lot of memory), we can use a generator that produces the data incrementally. Here’s an example:
import jsonl

def my_generator(count):
    for i in range(count):
        yield {"name": "Gilbert", "wins": [["straight", "7♣"], ["one pair", "10♥"]]}

# In this case, the dumping is done incrementally.
# Even though we generate 10,000 records, memory consumption remains low.

jsonl.dump(my_generator(10000), "file.jsonl")


Log4Net version mismatch

I got such error
FileLoadException: Could not load file or assembly 'log4net, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=669e0ddf0bb1aa2a' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
due to this entry
<Reference Include="log4net, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=669e0ddf0bb1aa2a, processorArchitecture=MSIL">
  <HintPath>..\packages\log4net.2.0.5\lib\net40-full\log4net.dll</HintPath>
  <Private>True</Private>
</Reference>

in https://github.com/csnemes/tracer/blob/master/Tracer.Log4net/Tracer.Log4net.csproj#L34 file

@Janek91 I'd be a bit surprised if version 1.2.15 of log4net was to be found in the 2.0.5 package. I'd try changing the versions so that they match in all and any of your app.config/web.config, packages.config and *.csproj.

@gautelo but it's not an entry in any of mine *.csproj files, but in yours one:
https://github.com/csnemes/tracer/blob/master/Tracer.Log4net/Tracer.Log4net.csproj#L34
In addition it's a bug in 2.0.5 version of log4net NuGet package. So, please, either fix manually path to correct dll in Tracer.Log4net.csproj file or update log4net to version above 2.0.5. I can prepare such commit on my fork, but I don't know which option do you prefer.

@Janek91 Aha. I see. I was under the impression that the snippet was in your csproj. Apparently I need reading lessons. Clear cut bug then. Might be a merging issue or something. In any event, I'm sure @csnemes will fix it for you in a jiffy.

Still no fix. I was able to create package for my own purposes, but others will be having the same problem. As I said previously - it's a bug in log4net ver. 2.0.5 - just upgrade it a bit, please.

Ok. I'll update it to the latest 2.0.8.

Released Tracer.Log4Net.Fody 1.3.3 which is updated to log4net 2.0.8


fix: initialPageParam with type unknown to support string, number, ...

Infinite query page parameter is not always a number

I'm not sure if there's an easy way to test this, but is the intention that the kubb configuration should still contain something like the following?
initialPageParam: 'undefined as string | undefined'

Otherwise, if I don't provide explicit typing, the type of initialPageParam is determined to be undefined when it should be string | undefined.  This creates a typing issue for the queryFn parameter pageParam.


WIP - end-to-end-security with XFSC

I'm closing this, the XFSC demo will be maintained in a dedicated repository now:
https://github.com/stackabletech/marispace-demos


feat(OnyxSelect): initiate input area of OnyxSelect

Relates to #762
Creates the base for OnyxSelect. This includes labels, message, and the central input-like section which will trigger a flyout in the future.
For now, the model is based on strings for showcasing the selections, this will be replaced by whatever types ListBox and ComboBox will need (they are not finished yet).
Also, a minor style discrepency in OnyxInput has been fixed.
Checklist

[x] The added / edited code has been documented with JSDoc
[x] If a new component is added, at least one Playwright screenshot test is added
[x] A changeset is added with npx changeset add if your changes should be released as npm package (because they affect the library usage)

Seems like the width is different between chrome and webkit/firefox?

Whoa that's crazy :O maybe the input has the same issue? in the OnyxInput's playwright tests the width is always declared as inline style...

is that a good idea and I should do the same to OnyxSelect's tests or should a min-width be applied to the component intself?

Seems like the width is different between chrome and webkit/firefox?

Whoa that's crazy :O maybe the input has the same issue? in the OnyxInput's playwright tests the width is always declared as inline style...  is that a good idea and I should do the same to OnyxSelect's tests or should a min-width be applied to the component intself?

I updated it like that. fyi in the follow-up branch I optimized the screenshots to have less variations


Optitype not predicting HLA-C alleles

I ran optitype, and everything looked fine, but the output file did not give any predictions for the HLA-C alleles. Here is the output log:
mapping with 8 threads...

 0:01:14.27 Mapping filtered_fished.fastq to GEN reference...

 0:03:26.66 Generating binary hit matrix.
0:03:26.70 Loading optitype_outdir/2017_04_25_02_02_55/2017_04_25_02_02_55_1.bam started. Number of HLA reads loaded (updated every thousand):
1K...
 0:03:26.97 1255 reads loaded. Creating dataframe...
0:03:27.18 Dataframes created. Shape: 1255 x 11179, hits: 22073 (22250), sparsity: 1 in 630.55

 0:03:33.52 temporary pruning of identical rows and columns

 0:03:33.63 Size of mtx with unique rows and columns: (50, 59)
0:03:33.63 determining minimal set of non-overshadowed alleles

 0:03:33.78 Keeping only the minimal number of required alleles (12,)

 0:03:33.78 Creating compact model...

starting ilp solver with 1 threads...

 0:03:33.83 Initializing OptiType model...
Welcome to the CBC MILP Solver
Version: 2.9
Build Date: Mar  7 2017

command line - /n/sw/fasrcsw/apps/Core/Cbc/2.9-fasrc01/bin/cbc -printingOptions all -import /tmp/tmpext6iepo.pyomo.lp -import -stat=1 -solve -solu /tmp/tmpext6iepo.pyomo.soln (default strategy 1)
Option for printingOptions changed from normal to all
 CoinLpIO::readLp(): Maximization problem reformulated as minimization
Current default (if $ as parameter) for import is /tmp/tmpext6iepo.pyomo.lp
Presolve 103 (-9) rows, 61 (-3) columns and 285 (-15) elements
Statistics for presolved model
Original problem has 37 integers (37 of which binary)
Presolved problem has 35 integers (35 of which binary)
==== 8 zero objective 32 different
==== absolute objective values 32 different
==== for integers 7 zero objective 17 different
==== for integers absolute objective values 17 different
===== end objective counts

Problem has 103 rows, 61 columns (53 with objective) and 285 elements
Column breakdown:
25 of type 0.0->inf, 1 of type 0.0->up, 0 of type lo->inf,
0 of type lo->up, 0 of type free, 0 of type fixed,
0 of type -inf->0.0, 0 of type -inf->up, 35 of type 0.0->1.0
Row breakdown:
0 of type E 0.0, 0 of type E 1.0, 0 of type E -1.0,
0 of type E other, 0 of type G 0.0, 3 of type G 1.0,
0 of type G other, 73 of type L 0.0, 0 of type L 1.0,
27 of type L other, 0 of type Range 0.0->1.0, 0 of type Range other,
0 of type Free
Continuous objective value is -429.083 - 0.00 seconds
Cgl0004I processed model has 102 rows, 61 columns (35 integer (35 of which binary)) and 279 elements
Cbc0038I Initial state - 0 integers unsatisfied sum - 1.44329e-15
Cbc0038I Solution found of -429.083
Cbc0038I Relaxing continuous gives -429.083
Cbc0038I Before mini branch and bound, 35 integers at bound fixed and 5 continuous
Cbc0038I Mini branch and bound did not improve solution (0.01 seconds)
Cbc0038I After 0.01 seconds - Feasibility pump exiting with objective of -429.083 - took 0.00 seconds
Cbc0012I Integer solution of -429.083 found by feasibility pump after 0 iterations and 0 nodes (0.01 seconds)
Cbc0001I Search completed - best objective -429.083, took 0 iterations and 0 nodes (0.01 seconds)
Cbc0035I Maximum depth 0, 0 variables fixed on reduced cost
Cuts at root node changed objective from -429.083 to -429.083
Probing was tried 0 times and created 0 cuts of which 0 were active after adding rounds of cuts (0.000 seconds)
Gomory was tried 0 times and created 0 cuts of which 0 were active after adding rounds of cuts (0.000 seconds)
Knapsack was tried 0 times and created 0 cuts of which 0 were active after adding rounds of cuts (0.000 seconds)
Clique was tried 0 times and created 0 cuts of which 0 were active after adding rounds of cuts (0.000 seconds)
MixedIntegerRounding2 was tried 0 times and created 0 cuts of which 0 were active after adding rounds of cuts (0.000 seconds)
FlowCover was tried 0 times and created 0 cuts of which 0 were active after adding rounds of cuts (0.000 seconds)
TwoMirCuts was tried 0 times and created 0 cuts of which 0 were active after adding rounds of cuts (0.000 seconds)

Result - Optimal solution found

Objective value:                -429.08300000
Enumerated nodes:               0
Total iterations:               0
Time (CPU seconds):             0.01
Time (Wallclock seconds):       0.02

Total time (CPU seconds):       0.02   (Wallclock seconds):       0.03

The output file looks like this:
	A1	A2	B1	B2	C1	C2	Reads	Objective
0	A*11:01	A*11:01	B*07:02	B*07:02			433	429.083

Any idea why this is?

Hi Alexander, can you e-mail me the intermediate bam file? You can tell OT to keep them with deletebam=false in the config file.

Hello,
I encountered the same problem. Wonder if anyone could advise what the problem was and how to fix it? Thank you!

I can look at the intermediate bam file if you e-mail it to me.

Thank you for offering help. Please find the intermediate files and the output attached. ?

From: andras86<EMAIL_ADDRESS>Sent: 30 November 2018 13:38
To: FRED-2/OptiType
Cc: Phuc Hoang; Comment
Subject: Re: [FRED-2/OptiType] Optitype not predicting HLA-C alleles (#49)
I can look at the intermediate bam file if you e-mail it to me.

You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/FRED-2/OptiType/issues/49#issuecomment-443205846, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ak_nDn6HwgM7DgQIgaFaeAHVlKZ1DJMoks5u0TTWgaJpZM4NHCoZ.
The Institute of Cancer Research: Royal Cancer Hospital, a charitable Company Limited by Guarantee, Registered in England under Company No. 534147 with its Registered Office at 123 Old Brompton Road, London SW7 3RP.
This e-mail message is confidential and for use by the addressee only.  If the message is received by anyone other than the addressee, please return the message to the sender by replying to it and then delete the message from your computer and network.


Add aws serverless application template page

Initial draft of the aws-serverless template

@gthuang looks like you need a file at themes/default/content/templates/serverless-application/_index.md, as I'm not seeing anything on the home page. You should be able to copy the one out of static-website and use that as a basis.

We are targeting the keyword aws serverless for the aws template here. That keyword is used throughout and also in the meta title and description. We will have 97th review this for any additional optimization once it is published, but looks to include the recommendations they provided. @SaraDPH for visibility.


Adding Dockerfile for backend

Hello
I would like to create a dockerfile for containerizing the application for deployments

Hello I would like to create a dockerfile for containerizing the application for deployments

Sorry but at present,we don't need docker file for our project


api documentation's font importation lacks of font-weight

Description
Flutter API documentation has accessibility problems, especially on the font-weight.
I've added 100 to all font-weight parameter, which makes it more visible.
In my environment, the docs don't seem to be that invisible, but looking at other environments like the issue I mentioned below, in some environment, it seems to be really invisible.
The image below compares how it looks, so please look at it.
This is my first pull request to flutter, so feedbacks are highly welcome.
Before
1.

2.

After
1.

2.

Related Issues
#47796
Tests
I've generated my docs using what is written here, and I found some points differs to the original one, but I think this is unrelated to this commit because I didn't change any code that is related to the context of the docs.
But no tests added because this is a doc change
Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change.

@googlebot I fixed it.

hi @keidaroo  - i've looked into this and here's some of my findings to help clarify the issue:
I do not see this as a technical issue, but rather a Design issue.  I can use typical accessibility tools built into the browser to increase/decrease the fonts without issue.

It seems your main concern is with the current font choice - Source Code Pro and Roboto Light - which is intentionally designed to be thin sans-serif fonts.  These fonts, when paired with light colors, can be "invisible" to the eye in smaller sizes.
I see in your screens above, you changed the font to Arial or Helvetica, which improved the readability.  To me, this is an issue which would be resolved with a different font choice that will be readable with lighter colors.  Google Fonts has many options other than Arial/Helvetica that will look modern and be readable/accessible.

Note: The thinnest font on the page is actually Roboto Light with a 300 weight.

We should have our docs evaluated for accessibility, both on desktop and  in mobile environments. Just changing the font weight across the board is not how we should approach this.

Thanks for the suggestion @keidaroo!  This is definitely an important issue.
Do you think you'll be able to address @sfshaza2 and @raison00's feedback, or should we close this for now?

Thank you for your feedbacks! (@raison00 @sfshaza2 @dnfield )
I agree with what @sfshaza2 says, this suggestion isn't evaluated enough, and this issue shouldn't be solved as simply changing the font-weight. In the next commit, I'll delete all changes of the weight adding things.
But still, I found another issue about the font-weight, which is related to what @raison00 says.
In the header file of this page, it is written like
@import 'https://fonts.googleapis.com/css?family=Roboto:500,400italic,300,400,100i';
and this means 300i(talic) font isn't imported.

Note: The thinnest font on the page is actually Roboto Light with a 300 weight.

so this is incorrect, actually, it is using 100 italic font instead of 300 italic font. In the browser debugging console, it is written that the text is using a 300-weight font, but this is just showing the css setting, not the actual font-weight.
You can check this by saving the webpage and changing the heading into
@import 'https://fonts.googleapis.com/css?family=Roboto:500,400italic,300,400,300i';
and this is the result.
Before

After

It is obvious from the values that 100-weight font is almost "invisible", and I think this needs to be changed.
Can I fix this bug in this pull-request or should I close this and open a new pull-request?
( @raison00 actually, I didn't change any font-family, I just added font-weight into all text)

Sorry, the last comment was including a mistake.
The thinnest font-weight was actually 100 in the original CSS settings.

Still, Roboto 300-weight font is not imported, so it must be imported.

Two months past, do you have any suggestions about this PR?
Also, I think this "Waiting for customer response" label should be taken off.

Removed the label - will wait for response from @sfshaza2 or @raison00

Does anyone have any update regarding this issue? The docs have pretty much been unusable for me since I started learning Flutter this year. I have to edit the css in my browser every time I go to the documentation site.
Chrome, Edge and Firefox, on Windows 10.
1440p VA monitor and 1080p IPS monitors.

Since this PR is out of date, and has had no activity for some time, and there was some meaningful disagreement about whether it's the right way to fix potential a11y issues, I'm going to close it.
I've filed https://github.com/dart-lang/dartdoc/issues/2374 and https://github.com/flutter/website/issues/4824


Color

I want to change the color how would I do so?

@QBrysonQLive go look into the HTML for HEX and RGBA colours and start changing them, can find the CSS locations by checking out the HTML file too.


Import from real life data not working

All that shows up when I try to use Import from real life.

 That should fix it


Text styling API proposal

The current text texture API (below) doesn't allow some fairly basic settings such as font weight, and requires the family and size to be set each time.
t.text({
  text: `foo bar`,
  align: 'start',
  font: {
    size: 20,
    name: 'Barlow Semi Condensed',
  }
});

Here's my proposal for a new API which provides more options and switches "name" to the more standard "family". See this article for more details on each property.
I would also suggest that every property is optional; the others will cascade from either Replay's defaults or gameProps.defaultFont.
t.text({
  text: `foo bar`,
  font: {
    size: 20,
    family: 'Barlow Semi Condensed',
    weight: 'bold',
    style: 'italic',
    decoration: 'underline',
    baseline: 'alphabetic',
    align: 'start',
  }
});

Yeah this proposal looks good. The only thing I'm not sure on is whether props like align should be within font or not.
I'd also like to find a way to add multi-line text, though this can be a separate effort.

Yeah this proposal looks good. The only thing I'm not sure on is whether props like align should be within font or not.
I'd also like to find a way to add multi-line text, though this can be a separate effort.


How to use Pagination component to implement routing jump paging?

Description
Hello, I use the Pagination component to implement pagination. The code is as follows. This is not a routing jump method for pagination. I am worried that this will affect SEO. How can I optimize it to be SEO-friendly?
`

`

Hi, could you please format the code following this guide ? https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/creating-and-highlighting-code-blocks#syntax-highlighting
Also, I'm not sure that I understand your question, nor how it is related to Nuxt UI given the context you provide.

<script setup lang="ts">
import type { BlogPost } from '~/types'
const page = ref(1)
const pageSize = ref(6)
const { data: posts, pending } = await useAsyncData('posts', () => ($fetch as any)('/api/blog/blog', {
  method: 'POST',
  body: { 'page': page.value, 'pageSize': pageSize.value }
 }), {
   watch: [page]
 })
</script>
<template>
  <UContainer>
    <UPageBody>
      <UBlogList>
        <UBlogPost
          v-for="(post, index) in pending ? [] : posts.data.records"
          :key="index"
          :to="'/blog/'+post.blogId"
          :title="post.title"
          :description="post.description"
          :image="post.image"
          :date="new Date(post.date).toLocaleDateString('zh-CN')"
          :ui="{
            description: 'line-clamp-2'
          }"
        />
      </UBlogList>
    </UPageBody>
    <div class="flex justify-center px-3 py-5">
      <UPagination v-model="page" :page-count="pageSize" :total="pending || !posts.data ? 0 : posts.data.totalCount" show-last show-first />
    </div>

  </UContainer>
</template>

The code you provided is not formatted correctly. ruby after the backticks will apply color syntaxing for a ruby file. For a vue file, you'll naturally use vue instead.
Also, this issue is not related to Nuxt UI, but applies to Nuxt itself. For that kind of question I invite you to join the community's Discord and look for help in there, the community should be more active and give you an answer right away: https://discord.com/invite/nuxt
I can give a first draft of an answer though.
If you need to support pagination with the routing, you can make use of a writable computed:
const route = useRoute()

const page = computed({
  get() {
    return Number(route.query.page?.toString()) || 1
  },
  set(newPage: number) {
    navigateTo({
      query: {
        page: newPage,
      }
    })
  },
})

What it does:

It reads directly for the url to parse the page number, and pass it to your pagination component, and your fetch method
Updates the url when you click on a new page, which updates the page property that was based on the URL query parameters.

If you don't want it to add a new entry to your history on each page change, you can eventually use the replace option in navigateTo


"Text" inside "Main" master slide not working

Issue description
When using the Text component in the Main master slide I get this error:
Uncaught Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.

Steps to reproduce

Use the Main master slide
Add the Text element like this:

    <A>
      <Text>This is some text</Text>
    </A>

Versions

yarn: 1.5.1
node 6.9.1
dekk: 1.0.0

Can you paste a link to the original code or paste a snippet?
It might be that you mixed default with named exports
import Text from '@dekk/text'
// But
import {Title} from '@dekk/text'

I just tested it. please see my previous comment. It's probably something similar.


Sliding Scale Overlapping Styles

For Capacity and ratios (bottom 2 statistics), the horizontal line on the sliding scale is missing missing, and the line indicating position on the scale is white even in light mode.
Current:

Expected:

Additionally, when dealing with numbers that overlap with the markings on the scale, I redesigned a few different styles in order to make sure the chart is always clear to read. Can also be seen here on figma.

done


Refs #26565 -- Errored nicely when using Prefetch with a values() queryset.

Thanks Maxime Lorant for the report and Anssi for the suggestion.

LGTM


Fix #7915: Make Delete consistent with conditional Get

We've decided to postpone this change until after the arch board meeting on Tuesday.  We will likely not do it, but let's keep the PR, in case the arch board decides to do this.

Bug is closed by design, PR not needed anymore


Sound plays 2x as fast if set 40 as FrameDuration

I tried play sound with Discord VC.
Code:
option := &dca.EncodeOptions{
	Volume:           256,
	Channels:         2,
	FrameRate:        48000,
	FrameDuration:    40,
	Bitrate:          96,
	Application:      dca.AudioApplicationAudio,
	CompressionLevel: 10,
	PacketLoss:       1,
	BufferedFrames:   1024,
	VBR:              true,
}
encoder, err := dca.EncodeMem(r, option)
if err != nil {
	return err
}

for {
	frame, err := encoder.OpusFrame()
	if err == io.EOF {
		break
	}
	if err != nil {
		return err
	}

	conn.OpusSend <- frame
}

This plays it 2x as fast. Setting FrameDuration: 20 fix it. It's expected behavior?
Or maybe Discord side problem?(Sorry, but really I'm not familiar with DCA or Opus.)

I increased the return value of PCMFrameLen(https://github.com/jonas747/dca/blob/10e959e9d3e808cd961308760de0c1c36285fed7/encode.go#L61) experimentally but not affect...

I found the reason in discordgo.
https://github.com/bwmarrin/discordgo/blob/8325a6bf6dd6c91ed4040a1617b07287b8fb0eba/voice.go#L411
The 960 is hard coded as the length of frame len.
I close this issue because it's discordgo problem. Sorry.


Wrong number of arguments

There seems to be an error in input_rebind_tick_legacy. I'm not sure what it is, but it's preventing my game from compiling. Please help.
`input_rebind_tick_legacy at line 172, 75 : number of arguments for function input_axis_threshold_get expected 0 got 2
var _value = input_gamepad_value(gamepad, _check);
if (abs(_value) > input_axis_threshold_get(_check, _player_index).mini)
{
_new_binding = new __input_class_binding("gamepad axis", _check, (_value < 0));
}`

That error doesn't make sense as input_axis_threshold_get() supports a variable number of arguments. Could you check that script/function in your project and tell me what it contains? I suspect this may be an import error.

Closing due to inactivity