How to access header fields in response

In abucoins, the page position of the orders returned is returned as part of the header in the HTTP response.  These values are needed to place additional orders calls using the paging values to retrieve different portions of the order book.
Is there an example of where I can intercept the HTTP response so I can extract those header fields?  The body contains JSON but I'd like to also return the values of these header fields as they are technically part of the response.

Do you mean the HttpResponseAware stuff?

Did we come to a consensus on adapting the code in order to provide access to the header fields.
I'm looking at implementing some stuff in gdax and to do it properly I need access to the cb-after field in the header which is required for paging.
There is a work around so it's not urgent.

I've fixed https://github.com/mmazi/rescu/issues/103. You can just implement HttpResponseAware on your response classes where you need access to headers. Let me know if this works for you.

Thanks.  Not quite sure where to use HttpResponseAware though.  I've search for an example but come up empty.

There is an example in tests, committed today.

Thanks. In which repo?

Here's the commit:
https://github.com/mmazi/rescu/commit/e28befa57720598c8d918908f94e6bf4cdc18122

Thanks.

Tried it out but the setResponseHeaders method never got called.  I think we're using rescu 2.0.2 - do we need a more recent version maybe?

2.0.2 was released a few hours ago, that's as recent as it gets :)
The test in the mentioned commit is supposed to demonstrate that it works. Please check your code, or show it if you need more help.


List linter doesn't recognize resource field

Proto example:
message Person {
  option (google.api.resource) = {
    type: "people.googleapis.com/Person"
    pattern: "people/{person}"
  };
}

service PeopleService {
  rpc ListContactPeople(ListContactPeopleRequest )
      returns (ListContactPeopleResponse ) {
    option (google.api.http).get = "/v1/people:listContactPeople";
    option deadline = 10.0;
    option fail_fast = true;
  }
}

message ListContactPeopleRequest {
}

message ListContactPeopleResponse {
  repeated Person people = 1;
}

Linter gives an error on the ListContactPeopleResponse  repeated Person people = 1; field.
List responses should only contain fields explicitly described in AIPs.
https://linter.aip.dev/132/response-unknown-fields

This is because of the mismatch between Person (the name of the resource) and ContactPerson (which is what the List request is using).
I am having trouble deciding whether this is a bug, or working as intended. In general, we advise against having subtype list methods like this, preferring filtering instead. I think for now I am going to err on the side of closing this as WAI, but if it comes up in the future, we may reconsider that.


Fire change event doesn't work on inputs with type="date"

react-testing-library version: ^6.0.3
react version: 16.8.2
node version: 10.11.0
npm version: 6.4.1

Relevant code or config:
test('Should be able to edit audit shift date', () => {
  const { container } = render(<AuditShiftForm />);
  const testValue = '03/29/2019';
  const input = getByTestId(
    container,
    'audit-date-input',
  ) as HTMLInputElement;

  fireEvent.change(input, { target: { value: testValue } });

  expect(input.value).toEqual(testValue);
});

What you did:
Trying to write a test that makes sure a date input is being updated properly.
What happened:
Jest fails the test with this output:
  ● components/AuditShiftForm › Should be able to edit audit shift date

    expect(received).toEqual(expected)

    Difference:

    - Expected
    + Received

    - 03/29/2019
    +

      21 |     });
      22 |
    > 23 |     expect(input.value).toEqual(testValue);
         |                         ^
      24 |   });
      25 |
      26 |   test('Should be able to listen for close-button clicks', () => {

      at Object.test (src/components/__tests__/AuditShiftForm.test.tsx:23:25)

Reproduction:
Here's a code sandbox for reproducing the issue. I added two tests: one that updates and checks a text input and another that updates and checks a date input. The text input test succeeds, but the date input test fails.
https://codesandbox.io/s/92pnxn5kro
Problem description:
It should be possible to fire change events against date inputs.
Suggested solution:
It looks like this library uses dom-testing-library to handle firing events. Let me know if I should post this in the dom-testing-library issues.
EDIT:
For now, my solution is to not use type="date" in test environments:
import styled from '../styled';
import Input from './Input';

const DateInput = process.env.NODE_ENV === 'test'
  ? Input
  : styled(Input).attrs({ type: 'date' })``;

export default DateInput;

The problem is that you're providing an invalid date string. If you try to do this in the browser (dateInputNode.value = '03/29/2019') then you'll see the following warning:

The specified value "03/30/2019" does not conform to the required format, "yyyy-MM-dd".

Here's a working version: https://codesandbox.io/s/rln7q07o0m
You'll notice that the display value may be 03/29/2019, but the value property formats it according to the specification mentioned. Good luck.


[stable/concourse] makes Web probes configurable

What this PR does / why we need it:
Following the example set by the grafana chart (see [1]), this commit
turns web probes into configurable entities.
It also makes use of /api/v1/info as the default endpoint for
performing the requests against so web doesn't return the whole
initial page as a return value (instead, just a very small json from
/api/v1/info).
Which issue this PR fixes
Special notes for your reviewer:
I wondered if I should do the same for the worker, but for now I left that as not configurable given that under the next release of Concourse (soon!) those probes could get replaced by regular httpGet with the improvements that are planned for worker healthchecking.
I bumped a minor as it changes the default value for the initial delay (which was 120s). Please let me know how you feel about that.
thx!
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

cc @frodenas @william-tran


Server management with Windows Tasks

@pakrym @Tratcher WRT ...
https://twitter.com/pakrym/status/996424063466668032
A few options:

New topic in either the fundamentals/servers or host-and-deploy area and linked from the fundamentals/servers/index topic Server startup section: Surface using a task and setting the priority using either or both of ...

For Windows Task Scheduler: Manually set the priority in an exported/imported task.
For a task created with PowerShell: Use the -Priority switch.

New section in fundamentals/servers/index topic Server startup section: Mention task scheduling and highlight the default (low) priority but don't show how to use Windows Task Scheduler or PS to set up a task with a given priority.
Continue to not discuss tasks in the docs.

If we surface anything that speaks to setting the priority, we probably need to broach the subject of what priority is recommended, which might get thorny (i.e., environment-specific or app-specific conditions may apply).
For Windows Services, I read that Service Control Manager doesn't offer a way to control the priority. Perhaps, the user would need to set it directly using Task Manager (or a 3rd party tool). Any info on this should go in the Host ASP.NET Core in a Windows Service topic.

@guardrex  I'm guessing that's not a common scenario and all we need to do is mention it in a doc.

I assume that with good services support these days and docs that we'll steer clear of describing the use of Windows Tasks. We'll apparently also avoid speaking to Windows Service priority management. Re-open if we need to discuss further.


use granian for wsgi

Consider using https://github.com/emmett-framework/granian for your wsgi server.  More proven and this library can then focus more on the web framework functionality.

Looks interesting, a PR is welcome. I looked around the crate but haven't found an obvious way to launch a WSGI app. They seem to have pretty extensive Python wrappers.


[SPARK-47708][CONNECT] Do not log gRPC exception to stderr in PySpark

What changes were proposed in this pull request?

Currently if there's any gRPC exception, instead of just handling it, the PySpark's gRPC error handler is going to print it out to the stderr, not allowing the user to cleanly ignore the exception by using try/except control flow statement.
In this PR we are removing the logger.exception call and we rely on the downstream exception mechanism to report this to the user.
Why are the changes needed?

Without this change, there's no way that the user ignores the gRPC error without piping the stderr to /dev/null or equivalent.
Does this PR introduce any user-facing change?

Yes, the stderr will not have the exception trace written twice.
How was this patch tested?

Existing tests.
Was this patch authored or co-authored using generative AI tooling?

No.

Merged to master.


Add support for macos aarch64

Also throw error, when artifact is not found for current os/arch

closed by https://github.com/hetalang/HetaSimulator.jl/pull/60


removing JsonSerializable

ISSUE-21

@TharinduDG please resolve the conflict and merge the master

Coverage decreased (-0.06%) to 47.41% when pulling fa5005fb2e9f48f7cff38c64f3ec35f73e3b85bc on TharinduDG:remove_jsonSerializable into 25689ed3d8f33c409d859d27bd06c5ed95804f5c on ScorexFoundation:master.

@kushti I resolved the conflicts. Please check.

@ceilican Updated. please check.

Thanks, @TharinduDG ! This PR became outdate w.r.t. master. Could you solve the conflicts, so that we can merge it automatically?

@ceilican Its ready to merge now. Please check.

@ceilican @TharinduDG lets remove JsonSerializable from modifier interfaces at all, let an application to decide whether JSON serialization is needed at all

@ceilican @kushti
Sorry, I confused my self with examples module and core.  Please review.

@TharinduDG thanks for the contributions!


Recover Ruby 2.0 code analysis using TargetRubyVersion: 2.0

Follow up https://github.com/rubocop/rubocop/pull/10632#issuecomment-1125909580.
Reverts part of https://github.com/rubocop/rubocop/pull/4787.
Only the Ruby version (2.0) to runtime should have been dropped, not code analysis.
This PR makes Ruby 2.0 code analysis with TargetRubyVersion: 2.0.
It aims to solve essentially the same problem as #10626, #10632, #10640, #10644, and #10662.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thanks!


Other playable models-Text2Image

playable models

dalle-mini &  craiyon
https://github.com/borisdayma/dalle-mini

待添加

No pretrained models

imagen
https://github.com/lucidrains/imagen-pytorch

待添加

If we have enough time, we will try to migrate. However, I hope that Baidu official can release an open source model of  text to image on paddlepaddle.
I also know a popular model trained by Tsinghua University, although it is also a pytorch version.
CogView2: https://github.com/THUDM/CogView2

If we have enough time, we will try to migrate. However, I hope that Baidu official can release an open source model of text to image on paddlepaddle. I also know a popular model trained by Tsinghua University, although it is also a pytorch version. CogView2: https://github.com/THUDM/CogView2

我刚刚找了一下，文心 ERNIE-ViLG 文本生成图像的能力在开放领域公开数据集 MS-COCO 上进行了验证。评估指标使用 FID(该指标数值越低效果越好), 在 zero-shot 和 finetune 两种方式下，文心 ERNIE-ViLG 都取得了最佳成绩，效果远超 OpenAI 发布的 DALL-E 等模型。他们提供 ERNIE-ViLG API 体验调用的入口，也许你可以联系作者团队，找他们要预训练模型？
I just found it, and the ability of Wenxin ERNIE-ViLG to generate images from text is verified on the open domain public dataset MS-COCO. The evaluation index uses FID (the lower the value of the index, the better the effect). In both zero-shot and finetune methods, Wenxin ERNIE-ViLG has achieved the best results, and the effect is far superior to the models such as DALL-E released by OpenAI. They provide an entry to the ERNIE-ViLG API experience call, maybe you can contact the author team and ask them to pre-train the model?
文心 ERNIE-ViLG
https://wenxin.baidu.com/wenxin/modelbasedetail/ernie_vilg/

Another project with code and models

ERNIE-SAT
类别文心·跨模态大模型
应用语音编辑、语音生成、语音克隆、带语音克隆的语音到语音翻译

ERNIE-SAT 采用语音-文本联合训练的方式在中文和英文数据集上进行预训练。使得模型学到了语音和文本的对齐关系，并且生成频谱的精度更高，合成声音的质量更高。
https://wenxin.baidu.com/wenxin/modelbasedetail/ernie_sat/


Create a Github Issues Bot

Describe the solution you'd like
Configure a Bot to provide automatic replies and actions to:

New Issues
Inactive issues

WIP

In the end, it was decided that only issues are closed after the inactivity threshold (not PRs / discussions).


Editor: Ensure font image rows/columns are positive

Add a check on the import for valid Rows/Columns.
Add some restrictions to the GUI to disallow values <= 0.
Fixes #91812

Did you forget to add your changes? You pushed an empty update

Pushed the correct change this time.

Thanks!


Error for migrations and seed

php artisan migrate --tenant=all
The "--tenant" option does not exist.

Hello @xyrintech what version of this package do you have installed?

2.x-L-5.4

That branch is not supported. Please use the beta of 3.x instead: hyn/multi-tenant:3.*, see:
https://github.com/hyn/multi-tenant/issues/215#issuecomment-312799344

Also see the discussion here; https://github.com/hyn/multi-tenant/issues/187

I have installed this but I don't see the documentation for 3.x
May I have the documentation for this?

You can use the Repositories to create websites, hostnames etc.. The documentation is still on the to-do. As to settings you can check the inline documentation in the configuration files themselves, they are quite extensive.

Thanks.


Steps to Reproduce
What are the steps to reproduce this bug? Please add code examples,
screenshots or links to GitHub repositories that reproduce the problem.
Possible Solutions
If you have already ideas how to solve the issue, add them here.
Otherwise remove this section.

I installed ffmpeg on D:\software\ffmpeg folder
and my config is
$config = [
'ffmpeg.binaries'  => 'D:/software/ffmpeg/bin/ffmpeg',
'ffprobe.binaries' => 'D:/software/ffmpeg/bin/ffprobe',
'timeout'          => 3600, // The timeout for the underlying process
'ffmpeg.threads'   => 12,   // The number of threads that FFMpeg should use
];

i also add it to environment path which i can run ffmpeg and ffprobe  at any path

windows must specify exact path
fuck windows

😀


change ansible 2.5.Latest

ansible 2.5.0

fixed with https://github.com/network-automation/linklight/commit/b71eebf18197a59bb99445938e0e3edc0c4eae48


Move Django Debug Toolbar to Django Commons

I'd like to transfer the project over to https://github.com/django-commons
Matthias and I both plan on helping with the organization and are the leads on the project here with Jazzband. I don't believe it's a reduction in maintenance quality. There may be a few growing pains with releasing as there are no actual projects with Django Commons. The other issue is that links to https://github.com/django-debug-toolbar/django-debug-toolbar/ may not resolve with the latest destination.
It should also allow us to bypass https://github.com/jazzband/help/issues/360 and revert #1916

I'm seconding this request.

By the way, I only now see that this is in our project; maybe this would better be opened in https://github.com/jazzband/help/issues
On the other hand, since it's possible that we will wait for a long time we should have a backup plan. We could look into migrating issues and comments (or maybe only links to old issues) using the GitHub API and replace jazzband/django-debug-toolbar with only a bare README which points everyone to the new location. A work colleague has transferred hundreds of issues from our decomissioned https://taiga.io/ instance into GitHub and I could probably ask him for pointers or code on how to do this.

Oh, we definitely need to open a roadies help question to move it out. But I also wanted to have an issue here for a history of the project.

Is this still something you'd like doing, is Django Commons ready to take it on? Let's do a proper transfer so that redirects keep working.

Hi Jannis, yes it is. Django Commons is nearly ready. I suspect early August we can move the toolbar over if that works for you.

Sounds good to me, thanks for the quick response

@jezdez alright, we're formally ready to do the transition: https://github.com/jazzband/help/issues/369
There isn't a major rush on this. @cunla is going through the process on September 4th for django-fsm2. There may be some bumps for our first few transitions.

This process is in motion. Updates should be followed on https://github.com/jazzband/help/issues/369 and https://github.com/django-commons/membership/issues/34

@elineda @VeldaKiara @living180 Do you mind joining Django Commons so we can grant you your elevated permissions on the project when it moves over?
https://github.com/django-commons/membership/issues/new/choose

done

Just joined as a member.


Using vgo instead of dep

Pull Request Test Coverage Report for Build 42

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+6.8%) to 72.368%

Totals

Change from base Build 33:
6.8%

Covered Lines:
275

Relevant Lines:
380

💛  - Coveralls


updating jshintrc file to remove deprecated options

@RackHD/veyron
removing deprecated hound options

BUILD on-http #166 : FAILURE

jenkins, test this please


[Suggestion] [sokol_app] proxy javascript function accessing DOM to the main thread

This is the PR mentioned in https://github.com/floooh/sokol/pull/997 and is the last change I had locally that I would like to suggest upstream.
What does this PR do?
This commit changes the javascript functions defined by sokol_app.h to allow proxying them to the main thread when necessary and enable the use of the emscripten proxy_to_pthread option. Specifically, it replaces a few call to the EM_JS macro to the EM_JS_MAIN_THREAD
What problem does it solve?
When compiled with proxy_to_pthread (generally recommended by emscripten documentation), the "main" app thread is running separately from the browser event loop. This guarantee the browser won't become unresponsive because of a deadlock. This option is particularly convenient when porting code that might block for any event on the main thread.
With this option, the main app thread is not able to access a few javascript features such as the DOM window object. The emscripten solution for that is called proxyinghttps://emscripten.org/docs/porting/pthreads.html#proxying), where javascript function calls are moved to the browser event loop and the return value eventually sent back to the worker thread.
Potential caveat
Since there is no utility EM_JS_MAIN_THREAD in emscripten (that would define a javascript function callable from c++ proxied to main), I had to define the corresponding function in pure C and make them call inline asm via the MAIN_THREAD_EM_ASM.
Example of such a change:
EM_JS(void, sapp_js_remove_dragndrop_listeners, (const char* canvas_name_cstr), {
...
}

becomes
void sapp_js_remove_dragndrop_listeners(const char* canvas_name_cstr) {
    MAIN_THREAD_EM_ASM({
      ...
    });
}

This means that the generated function in the javascript side won't be named with the name defined in C, instead they will have random name.
Hence, if those function were intended to be called from the javascript side, this PR would be a problem. I don't see any call like this so far, but the fact that they were not prefixed with an underscore, as other internal function, make me doubt.

Hmm, at first glance this change is a bit too radical for my liking. Do you know if MAIN_THREAD_EM_ASM intoduces overhead if the C side is already running on the main thread?

I suspected this would require a bit more discussion, hence why I kept this PR for the end. :)
I think the ability to run with the proxy to pthread option is valuable as it actually eliminates a few nasty gotchas in multithreaded emscripten builds and make the behavior of the web version more similar to the native build.
Unfortunately, I don't have good answer for the overhead, and did not find anything on the emscripten github repo before posting the PR.
I find the emscripten comment on MAIN_THREAD_EM_ASM is informative :
// Runs the given JavaScript code synchronously on the main browser thread, and returns no value back.
// Call this function for example to access DOM elements in a pthread when building with -s USE_PTHREADS=1.
// Avoid calling this function in performance sensitive code, because this will effectively sleep the
// calling thread until the main browser thread is able to service the proxied function call. If you have
// multiple MAIN_THREAD_EM_ASM() code blocks to call in succession, it will likely be much faster to
// coalesce all the calls to a single MAIN_THREAD_EM_ASM() block. If you do not need synchronization nor
// a return value back, consider using the function MAIN_THREAD_ASYNC_EM_ASM() instead, which will not block.
// In single-threaded builds (including proxy-to-worker), MAIN_THREAD_EM_ASM*()
// functions are direct aliases to the corresponding EM_ASM*() family of functions.
#define MAIN_THREAD_EM_ASM(code, ...)

Another informative bit is looking at the generated javascript for the inline ASM call:
var runMainThreadEmAsm = (code, sigPtr, argbuf, sync) => {
 var args = readEmAsmArgs(sigPtr, argbuf);
 if (ENVIRONMENT_IS_PTHREAD) {
  return proxyToMainThread.apply(null, [ -1 - code, sync ].concat(args));
 }
 return ASM_CONSTS[code].apply(null, args);
};

So, In term of overhead I see two potential sources:

From the proxying part: the fact that now the javascript part will start by checking if we are already on the main thread, if so it call the function directly in a sychronous way. This overhead is zero in non pthread build.
From the inline ASM vs EM_JS part: the javascript function is now a field of the ASM_CONSTS table with a code number, so calling it introduce an extra indirection.

I think if 2 is a problem, another option could be to detect and dispatch on the C side instead and avoid inline ASM:
EM_JS(void, cfunc_impl, (void));

void cfunc(void) {
  if (!emscripten_is_main_runtime_thread()) {
    cfunc_impl();
  } else {
    emscripten_sync_run_in_main_runtime_thread(SIG, cfunc_impl, args)
  }
}

but honestly I don't know enough about the internal of emscripten and javascript to see if that could lead to a measurable difference overhead.
Note that I realized my PR is incomplete, but my guess was that the functions needing to be changed  were not performance sensitive:
void sapp_js_add_beforeunload_listener()
void sapp_js_remove_beforeunload_listener()
void sapp_js_add_clipboard_listener()
void sapp_js_remove_clipboard_listener()
void sapp_js_write_clipboard(const char* c_str)
void sapp_js_add_dragndrop_listeners(const char* canvas_name_cstr)
void sapp_js_remove_dragndrop_listeners(const char* canvas_name_cstr)
void sapp_js_init(const char* c_str_target)

although my current implementation is likely not the most efficient and could be improved either with MAIN_THREAD_ASYNC_EM_ASM when correct or with the c version above.


Does this support abtract states for ui-router ?

Hi there,
I was wanting a way to include a certain CSS file (which is dynamic) on each and every state. I was thinking of doing this via an abstract state and having all other states use the abstract.
Does this support abstract states?
The css I need to include on every page would be available in a variable but I just can't insert it onto the page as it changes depending on certain this.
Is there support for this ?

From digging into the source and testing on my project it currently does not support abstract states.
This would be a killer feature as I suspect most people will have a abstract states that define a base layout that other child states will populate the ui-view.
I'm currently working on a project with an abstract base 'frontend' state (home, login, register pages) and an abstract base 'dashboard' state. It would be great if I could define a set of css files for both abstract states whenever a child state is instantiated.
However, I do not see an easy way to go up the state tree.

Hi @ianjohnmadrid and @kenny-f
I can see the value of adding this as a feature. I'm going to look into it and see how we can approach this.
Thanks!

Great news. Thanks

Thanks!
My current workaround is to have a controller on the base abstract state and load the css from there.

That is a good workaround for the time being. I'll still look into it so we can do this properly.

I have an abstract state that contains a navbar, a ui-view and a footer. It is abstract because there should always be content inside the ui-view. But the navbar has its own stylesheet that I need to load... please add this feature.

What is that status of this feature?

Looks like this PR added support for inheritance via abstract states when using the 'data' property: https://github.com/castillo-io/angular-css/pull/35

I believe this was resolved in #35.
Releasing soon.

Fixed in v1.0.8


[Feature] add control for slideshow playback speed

Resolves #570

looks good to me. thank you for your contribution

I found the design of the speed selector to be inconsistent with design of the app, so I moved to the submenu on the top.
Sorry about that. Please consult next time before sending a PR.

Thanks for merging and fixing those things you highlighted! I had some trouble aligning the dropdown with those other buttons and 'io' added some weird position offset that affected height. Just got it to work like that, but I like the position you moved it to a lot, so doesn't really matter anymore I guess.
I'm happy to contribute, this was actually my first time ever :partying_face: What do you mean by 'consult first'? I thought opening a PR and an issue is just that.
I do have a few improvements that I'd like to see. You removed some logic by refactoring the dropdown at a new location. Should I open separate issues for that or would you like to discuss those here and I open issues/PRs afterwards?
What I'd like to re-add:

Set cookie for playback speed. Especially since it's now in the submenu this would be very handy, because it's a little hidden now and would be great as a set-and-forget kind of setting. At least per client and if possible I'd set expires to 10 years or something, but I could not get that to work, at maximum the cookie was created with 7 days validity.

This feature was there already, but since you are using two-way-data binding now, my function that also set a cookie isn't called anymore. I have no idea why changing playback speed works while having an active slideshow running. The logic you added is also in that function and I just can't see it called anywhere and if this works, the cookie should be set aswell. Maybe you see something I don't.

Selecting a playback speed should start the slideshow (arguable, but I personally think if someone is setting a playback speed, they most likely want to start it next)

Thank you for your work on this! I'm more than happy to fix those things myself and create a PR after we decided what to do

Hi
By consulting I meant it is better to have some back and forth mail before someone puts effort into implementing something. (I hope I did not sound harsh, that was no my intention).
The May reason behind it is that it's basically only who has the whole code base in my head.
Also I'm relatively picky on the design. Like I won't submit half solutions:)
About the issues:

Sorry I did not see that the change broke the cokie settings. I will need to double check if directory ordering uses local storage. If yes this feature should also do so.

I'm not 100% that settings is at the right place, but could figure out any better . I might move it back to the play button but just something super minimalistic. Or make the play button behave like the FB like button where longpress/hover opens the other like options.
I do not see this feature to be used often. As you said it should be set and forget. I think the default value should be also set in the config


local.wix.com added to the hosts file (Windows & Mac) in the tests prerun

The solution is just to add local.wix.com to the grunt-build.sh. and we also should have a build step to add local.wix.com to the hosts of our dockers


Document how to develop under Windows 11

Wiki is updated and a script to simplify builds under Windows is deployed.

Got it working based on the wiki with a few modifications:
-open a command prompt and run `npm config set msvs_version 2019`
+open a command prompt and run `npm config set msvs_version 2017`

also I needed to change the build scripts to --max-old-space-size=8096 from 4095, using 4095 threw out of memory errors.

@JacksonKearl did you install the 2019 version of the Visual Studio build tools?
And which error did you receive when using npm config set msvs_version 2019

I'm unable to get back into the broken state, but I found that while I did have the VS 2019 build tools installed, the "Desktop Development with C++"  was not checked, so that's probably the source of the issue. I
t's a 7GB download so I'll wait until I'm broken before putting it on my VM.

@JacksonKearl makes sense. And thanks for getting back.


add nginx rate-limit

https://www.nginx.com/blog/rate-limiting-nginx/
An unknown IP started scraping pages at a fast enough rate that @alanpost was paged for the site being down. (It didn't crash, just got unresponsive.) We should pick and add limits for the app and more relaxed limits on assets (js, css, avatars).

Do you have a rough idea on the number of daily IPs used to visit Lobsters? This is to size the in-memory buffer holding the IPs. It's probably not more than 100k/day.
We also should define a req/s for pages and for assets. I propose something like 5/s with a burst to like 10(I don't think there are lots of concurrent calls for that) for the app and something like 10-15/s with a burst at like 25 for the assets.
What do you think about it? (To be honest, I have no experience with that, I would have taken the logs and write a script to approximate it, maybe some tools exist for that)

Last we checked it was around 15k unique IPs per weekday, but there are substantial spikes (2-3x? less certain here) when we get linked off YC News or a comment goes viral on Twitter.

I wrote a little script (probably unmaintainable at this step) in ruby (because I assume it'll be easier to integrate at some point, but also because you're probably more used to it than Go or Python).
I did some tests locally and here is the current output:
$ ruby nginx_log_stats.rb -f access.log
{:number_of_ips=>3,
 :max_reqps=>16,
 :min_reqps=>1,
 :num_reqs=>137,
 :avg_reqps=>2,
 :avg_reqps_app=>2.8055555555555554,
 :avg_reqps_assets=>0.9166666666666666}

The useful bits for this PR use case would be avg_reqps_app and avg_reqps_assets. I didn't handle anything linked to how many days are in a log file because I assume there's a daily logrotate in place.
If you have any useful idea to make this match a bit more what you'd think we need, please let me know! If you intend to test it, just be sure not to run it on the server as it may cripple CPU and FS for the time of processing.

@pushcx Can we close this now that we've merged lobsters/b8d91ca?


add elapsed field to logger

Hi @leaxoy! :wave:
It's definitely useful to be able to collect timings about how long a piece of middleware takes to execute, but in my opinion this is best solved through some kind of contextual logging where we can choose what blocks of code to wrap for timings.

Wrap the user logic maybe useful. Then we can add some code around RequestContext.next call with insert time field to Request.extensions, which can be used whenever it's needed.

Hi @leaxoy! Does #222 resolve this? :)

Looks cover this.


[G7T5-36] Add CD pipeline

Description
Added CD pipeline for backend
Type of change

[ ] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

[ ] Deployed G7T5 branch to staging

Checklist:

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

I would say we can squash and merge this


S3 access changed to virtual hosted style

Changed S3 addressing from path-style to virtual-hosted style
Clearly describe the problem and solution
Amazon has deprecated path-style access (https://s3.amazonaws.com/bucket/key) to S3 buckets. Buckets created after September 30, 2020, will support only virtual hosted-style requests (https://bucket.s3.amazonaws.com/key).
References:
AWS Blog
S3 Documentation
Changing S3 access to virtual hosted-style to comply with the deprecation.
Relevant issue number(s) if applicable
N/A
Did you run npm run lint from the viewer or parliament directory (whichever you are making changes to) and correct any errors
N/A
Did you Ensure that all tests still pass by navigating to the tests directory and running ./tests.pl --viewer
N/A
License
I confirm that this contribution is made under an Apache 2.0 license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Does this work for bucket names with dots in them? I seem to recall that there are TLS certificate issues....

Does this work for bucket names with dots in them? I seem to recall that there are TLS certificate issues....

Yes indeed there is an issue with bucket names containing dots - upload fails due to hostname mismatch in SSL wildcard certificate:
For example, for bucket name fj,moloch:
Dec  5 18:48:19 http.c:584 moloch_http_curl_watch_open_callback(): Connected 1/0 - https://fj.moloch.s3-eu-west-2.amazonaws.com   35142-><IP_ADDRESS>:443 - fd:8
* Connected to fj.moloch.s3-eu-west-2.amazonaws.com (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering http/1.1
* Cipher selection: ALL:!EXPORT:!EXPORT40:!EXPORT56:!aNULL:!LOW:!RC4:@STRENGTH
* successfully set certificate verify locations:
*   CAfile: /etc/pki/tls/certs/ca-bundle.crt
  CApath: none
* SSL connection using TLSv1.2 / ECDHE-RSA-AES128-GCM-SHA256
* ALPN, server did not agree to a protocol
* Server certificate:
*  subject: C=US; ST=Washington; L=Seattle; O=Amazon.com, Inc.; CN=*.s3.eu-west-2.amazonaws.com
*  start date: Nov  9 00:00:00 2019 GMT
*  expire date: May 20 12:00:00 2020 GMT
*  subjectAltName does not match fj.moloch.s3-eu-west-2.amazonaws.com
* SSL: no alternative certificate subject name matches target host name 'fj.moloch.s3-eu-west-2.amazonaws.com'
* Closing connection 0

This is documented by AWS:

When using virtual hosted–style buckets with SSL, the SSL wild-card certificate only matches buckets that do not contain periods. To work around this, use HTTP or write your own certificate verification logic.

What should be done about it?

Ignore deprecation and leave the code as is with path-style access? Or...
Add a note about bucket names with dots not supported? Or...
Disable certificate verification in libcurl call? Or...
As per Amazon's suggestion, write custom certificate verification logic?

no custom cert logic please. :)  I would suggest we leave both methods in and just either add a switch for which method to use, or if there is a dot in the name choose the old method otherwise use the new method.

I would suggest we leave both methods in and just either add a switch for which method to use, or if there is a dot in the name choose the old method otherwise use the new method.

This means that uploads to buckets with dots created after September 30, 2020 would not work.

correct.  isn't it kind of a well known issue now to not use dots unless its for static host serving?

I believe it is well-known. I'll see if I can add a new switch to the code. Still new to GitHub - this is only my second OSS contribution. Thank you for your help!

seems ok to me, @pjsg look ok?


[DO NOT MERGE] Transit visa update

Supersedes https://github.com/thomasjhughes/smart-answers/pull/1
Description
Added callout to let users with a Marriage Visitor or Standard Visitor visa know that they don’t need to apply for a separate Visitor in Transit visa.
Trello
https://trello.com/c/TjJAN82A/769-1-update-outcomes-in-check-if-you-need-a-uk-visa
Fact check
Current
New
Before

After

Examples of affected outcomes

https://smart-answers-preview-pr-3254.herokuapp.com/check-uk-visa/y/afghanistan/transit/somewhere_else/yes
https://smart-answers-preview-pr-3254.herokuapp.com/check-uk-visa/y/china/transit/somewhere_else/yes
https://smart-answers-preview-pr-3254.herokuapp.com/check-uk-visa/y/democratic-republic-of-the-congo/transit/somewhere_else/yes
https://smart-answers-preview-pr-3254.herokuapp.com/check-uk-visa/y/estonia/alien/transit/somewhere_else/yes
https://smart-answers-preview-pr-3254.herokuapp.com/check-uk-visa/y/israel/provisional-passport/transit/somewhere_else/yes
https://smart-answers-preview-pr-3254.herokuapp.com/check-uk-visa/y/latvia/alien/transit/somewhere_else/yes
https://smart-answers-preview-pr-3254.herokuapp.com/check-uk-visa/y/syria/transit/somewhere_else/yes
https://smart-answers-preview-pr-3254.herokuapp.com/check-uk-visa/y/turkey/transit/somewhere_else/yes
https://smart-answers-preview-pr-3254.herokuapp.com/check-uk-visa/y/venezuela/transit/somewhere_else/yes

 (1) Update outcomes in 'Check if you need a UK visa'


Localization

Pretty cool, it's in a need of localization though.
I will be glad to help btw.

@mshaaban0, feel free to submit PR, thanks.


missedmessage_emails: Backoff the background worker retries.

2024-05-06 19:13:58.985 ERR  [] Exception in MissedMessage background worker; restarting the loop
2024-05-06 19:14:00.015 ERR  [] Exception in MissedMessage background worker; restarting the loop
2024-05-06 19:14:02.059 ERR  [] Exception in MissedMessage background worker; restarting the loop
2024-05-06 19:14:06.089 ERR  [] Exception in MissedMessage background worker; restarting the loop
2024-05-06 19:14:14.124 ERR  [] Exception in MissedMessage background worker; restarting the loop
2024-05-06 19:14:30.155 ERR  [] Exception in MissedMessage background worker; restarting the loop
2024-05-06 19:15:00.192 ERR  [] Exception in MissedMessage background worker; restarting the loop

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Looks good, merged, thanks @alexmv!


Make Args a Named Record

This Issue need to be discussed further

@calderinisystemorph could pls elaborate a bit more on this issue? Thanks


Hello World example fails

I'm not sure if this is a documentation bug or an actual bug--I'm new to this project and can't quite figure that out.
I'm trying to run the hello world example from the docs, with the dockerfile from the docs, but on the object building phase I get the error:
root@da558b6501ef:~/hello# lucetc-wasi -o hello.so hello.wasm
Error: Lucet Module: Unknown module for symbol `wasi_unstable::fd_prestat_get

I expected this line to successfully complete and produce hello.so. It did not.
Steps to repro:

Follow the compiling Lucet from scratch guide (with docker). For the rest of this, I'll be inside the docker image.
Side note: to anyone else who encounters this, the dockerfile needs a CMD /bin/bash or something at the end, or else it won't run.
cd /root/
git clone https://github.com/bytecodealliance/lucet.git (Latest on master at time-of-filing is:  12ae2a0cec4a7726d5d023b9ac227af336c59e7d)
git submodule update --init --recursive: Get all the submodules, as requested by the compiling for linux guide.
make install: Build lucet
source /opt/lucet/bin/setenv.sh: This is needed to put the tools below in $PATH
mkdir /root/hello/ && cd /root/hello
printf '#include <stdio.h>\nint main(void){ puts("Hello World"); return 0;}' > hello.c
wasm32-wasi-clang -Ofast -o hello.wasm hello.c
lucetc-wasi -o hello.so hello.wasm
Error:

~/hello# lucetc-wasi -o hello.so hello.wasm
Error: Lucet Module: Unknown module for symbol `wasi_unstable::fd_prestat_get

Let me know what other information might be helpful!
I would love to be just a documentation bug, so if there's some extra modules I need to put into my path I'd love to know what I need to do.
Thanks for taking the time to look at this!

Thanks for submitting this issue.  I appreciate your clear repro steps!  After following your steps, I see exactly what you see:
$ wasm32-wasi-clang -Ofast -o hello.wasm hello.c
$ lucetc-wasi -o hello.so hello.wasm
Error: Lucet Module: Unknown module for symbol `wasi_unstable::fd_prestat_get

More to come.


PEP8 cleanups

PEP8 cleanups in HwndWrapper  and uia_element_info tests
Small re-work on mouse tests

Codecov Report

Merging #428 into master will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #428      +/-   ##
==========================================
- Coverage    94.5%   94.42%   -0.09%
==========================================
  Files          49       49
  Lines       19797    19797
==========================================
- Hits        18710    18693      -17
- Misses       1087     1104      +17


XMLStats: some questions for params

-val: (fictional)
Is it possible / can it be added to show only defined values? Like "xmlstats -val "owner, area" -f myfile.xml
-f:
I get a (summary) result, but don't know what it contains. Please add in header the files which are used
-f -v:
I defined three files - a.xml, b.xml, c.xml.
The result file shows me:

title A
data A
title B
data B
title C
summary data

Is this intended?
Thanks!

Bump up ...

So to be clear the -f option should point to a file containing a list of file names and not myfile.xml.
For example:
cd D:\git\lovettchris\XmlNotepad\src\Application\Samples
dir /b *.xml > list.txt
xmlstats -f list.txt

This will report a combined report of everything found in all the XML files listed in list.txt.

Now for your -val option it sounds like you want a kind of "selector" that is saying only report the status on matching nodes. So along those lines, what do you think of making this "selector" an XPath expression?  Something like this:
xmlstats Hamlet.xml -select "//SPEECH"

would output only the stats for those matching elements in the hamlet.xml file.  Then I would allow multiple -select arguments so you can specify different xpath expressions if needed.
Would this be useful?

would output only the stats for those matching elements in the hamlet.xml file. Then I would allow multiple -select arguments so you can specify different xpath expressions if needed.

Yes, I think this would to the job. Thanks.

This will report a combined report of everything found in all the XML files listed in list.txt.

Yes, I did it in two versions:
xmlstats  -f list.txt > -f.txt
xmlstats -v -f list.txt > -v-f.txt

The results are attached
-f.txt
-v-f.txt
f.txt contains:

no info about the files which are analysed
a lot a statistics

v-f.txt contains:

path for basket.xml
data for basket.xml
path for employee.xml
data for employee.xml
path for hamlet.xml
data for hamlet.xml
path  for rss.xml
data for ????. It seems that data.rss.xml is missing and is followed by the same (summarized) data package which is in -f.txt

Ah, ok, I see what you mean, I added the file names in the -f case with this fix: https://github.com/microsoft/XmlNotepad/commit/a003cae254b86acb0f32a658ca1215b15f06aec1

XPath turned out to be too complex, so I went for a simpler option, similar to what you proposed, the new argument is "-s" for "select" and you can select element names (using local names, so you don't have to worry about namespace prefixes).  You can try it out here: https://github.com/microsoft/XmlNotepad/commit/7a767e2b8fee7f056d63651cd9612b6c0c56ca50

Fixed in https://github.com/microsoft/XmlNotepad/releases/tag/<IP_ADDRESS>


Multi-dataset Line chart shows the wrong tooltip in crowded charts

While single line charts have their own problem rendering multiple tooltips on crowded charts, I've been able to work around that with the customTooltips option. However, multi-dataset line charts can't be worked around since they only show 1 tooltip, and it's wrong. The following gif illustrates that it starts being wrong at the halfway point, when it flips sides.

Let me know if you need more details.

Hello, today I am closing all issues that are only affecting version 1 of Chart.js as WONTFIX.
If this issue does affect version 2 as well, I apologize for the error. Please create a test case against Chart.js 2 using on of the below websites and we will be happy to reopen the issue and update its classification:

http://jsbin.com/
http://jsfiddle.net/
http://codepen.io/pen/
Premade template: http://codepen.io/pen?template=JXVYzq


btTransform wrapper

Hi!
I'm trying to port some C++ code using BulletPhysics to C# using BulletSharp.
colMapObject = new btCompoundShape();
colMapObject->addChildShape(btTransform(btQuaternion(0, 0, 0, 1), btVector3(x, y, z), sphere);

For btCompoundShape I use CompoundShape class, similarly for Quaterion and Vector3. But what's need to be used for btTransform?

It depends on the version of BulletSharp, but it's usually a class called Matrix.
Your IDE should suggest the right type when you type in:
colMapObject = new CompoundShape()
colMapObject.AddChildShape(

Yeah, I know that the Matrix should be passed as an argument. I use bullet sharp 3 and VS2017. But I didn't found a method in Matrix that will create it from Quaternion and Vector3 as in original btTransform. That's why I asked it.

I mean it depends on whether it's the generic version or a version that uses types from a graphics library like OpenTK/SharpDX/etc.
The generic version has no special Matrix constructors, but you can always multiply a rotation matrix and a translation matrix.
Matrix rotation = Matrix.RotationQuaternion(new Quaternion(0, 0, 0, 1));
Matrix translation = Matrix.Translation(x, y, z);
Matrix transform = rotation * translation;
colMapObject.AddChildShape(transform, sphere);

See here about multiplying matrices:
https://docs.microsoft.com/en-us/windows/desktop/direct3d9/transforms#concatenating-matrices
With a Matrix class from some other framework, you can do the same, except if the matrix layout is different (like in OpenTK), you may have to reverse the multiplication order to translation * rotation.
In this particular case, since (0, 0, 0, 1) is an identity quaternion that specifies no rotation, you could just use the translation matrix: colMapObject.AddChildShape(Matrix.Translation(x, y, z), sphere);
The generic version of BulletSharp has math classes based on SlimDX:
https://slimdx.org/docs/#T_SlimDX_Matrix
However, the Matrix.Transformation constructor that takes a rotation quaternion and a translation vector has been removed since it depends on DirectX.

Thank you very much for you rapid and detailed answer! It helped!


update pull_request_template

In pull_request_template.md
Change
Issue Number
with
Fixes #(issue number)
so that issue automatically get close on successful merge of PR

@QAZIMAAZARSHAD Assign this to me, please.

@satyampgt4 Thanks for suggesting this update!


Update SwiftRuntimeSupport to a version from 2021

Fixes https://github.com/xamarin/xamarin-macios/issues/13605

All of the cake builds fail for me locally due to this so I created a package manually with these steps reverse engineered from the cake file:
msbuild /t:Build /p:Configuration=Release  SwiftRuntimeSupport.sln
msbuild /t:Pack /p:Configuration=Release /p:PackageOutputPath=/Users/donblas/Programming/XamarinComponents/iOS/SwiftRuntimeSupport/output/ /p:PackageRequireLicenseAcceptance=true /p:IncludeBuildOutput=false

@SotoiGhost can you have a look? this should enable Swift async support

@chamons In case you need it, we are using an old version of Cake. Run dotnet tool install Cake.Tool --version 0.33.0 to make it work.

/azp run

When will this be fixed? It is currently blocking some of us like shown here https://developercommunity.visualstudio.com/t/Xamarin-iOS-codesign-failed-with-exit-co/1631600?space=8&entry=problem&q=codesign+failed&sort=newest&page=8

/azp run

Hi, it's seem we have an anomaly since this version : https://github.com/xamarin/XamarinComponents/issues/1358


New dev ux: Add a docker build doc and improve error message

Sorry didn't see this when you posted it! Thanks for adding this!


Provide a "Open with..." for settings.json

Similar to the search editor, I think we should have an enry "Open with... > Settings editor" for settings.json inside .vscode folder.
//cc @roblourens @sandy081

Feel free to close this if not straightforward, I don't think this is very important, just nice to have.

Closing due to the amount of complexity this would add to a niche use case.


Unexpected content getting parsed by Stylelint

Team Stylelint:
Love your ecosystem! I've been trying to add more tooling to help our junior devs, and I've been having some issues getting Stylelint to behave.
When I run stylelint 'src/**/*.{js,jsx}', I'm currently getting errors. It seems that stylelint is trying to parse code outside of my styled-component template strings.
src/components/Button/components/ButtonWrap.js
 75:21  ✖  Unknown word   CssSyntaxError

src/components/AppHeader/components/Tab/components/TabLink.js
 25:23  ✖  Missed semicolon   CssSyntaxError

ButtonWrap

TabLink

import { NavLink } from "react-router-dom";
import styled from "styled-components";

import { spacingScale, contrastColor } from "style/styleFunctions";
import {
  COLOR_ALT_BACKGROUND,
  COLOR_WHITE,
  COLOR_HIGHLIGHT
} from "style/styleVariables";

const TAB_WIDTH_BASE = "1%";
const COLOR_TAB_BACKGROUND_BASE = contrastColor(COLOR_ALT_BACKGROUND, 0.175);
const COLOR_TAB_BACKGROUND_ACTIVE = contrastColor(COLOR_ALT_BACKGROUND, 0.3);
const COLOR_TAB_HIGHLIGHT = COLOR_HIGHLIGHT;
const TabLink = styled(NavLink).attrs({
  exact: true
})`
  border-top: 1px solid transparent;
  font-weight: 600;
  flex: 1 1 ${TAB_WIDTH_BASE};
  margin: 1px 1px 0 0;
  padding: ${spacingScale(0.5)} ${spacingScale(1)} ${spacingScale(1)};
  position: relative;
  transition: all 0.15s ease;
  &:after {
    border-radius: 1px;
    box-shadow: 0 0 1em 1px ${COLOR_TAB_HIGHLIGHT.string()};
    color: inherit;
    content: '';
    height: 1px;
    left: 0;
    opacity: 0;
    pointer-events: none;
    position: absolute;
    right: 0;
    top: -2px;
    z-index: 100;
  }

  &:hover {
    background-color: ${/* sc-block */ COLOR_TAB_BACKGROUND_BASE.mix(
      COLOR_TAB_BACKGROUND_ACTIVE,
      0.3
    ).string()};

    &:after {
      opacity: .5;
    }
  }

  &.active {
    background-color: ${COLOR_TAB_BACKGROUND_ACTIVE.string()};
    &:after {
      background-color: ${COLOR_TAB_HIGHLIGHT.mix(COLOR_WHITE, 0.3).string()};
      opacity: 1;
      transition: .1s;
    }
  }

  &:active {
    &:after {
      background-color: ${COLOR_TAB_HIGHLIGHT.mix(COLOR_WHITE, 0.3).string()};
      opacity 1;
      transition: .1s;
    }
  }

  &:focus {
    outline: ${COLOR_TAB_HIGHLIGHT.string()} auto 3px;
  }
`;

export default TabLink;

.stylelintrc file at root of project
{
  "processors": ["stylelint-processor-styled-components"],
  "extends": [
    "stylelint-config-standard",
    "stylelint-config-styled-components"
  ],
  "syntax": "scss",
  "plugins": [
    "stylelint-no-unsupported-browser-features"
  ],
  "rules": {
    "plugin/no-unsupported-browser-features": [true, {
      "ignore": ["rem"],
      "severity": "warning"
    }],
    "at-rule-empty-line-before": null,
    "block-opening-brace-space-after": null,
    "block-closing-brace-space-before": null,
    "color-hex-length": null,
    "comment-empty-line-before": null,
    "declaration-block-semicolon-newline-after": null,
    "declaration-colon-newline-after": null,
    "declaration-empty-line-before": null,
    "indentation": null,
    "no-extra-semicolons": null,
    "rule-empty-line-before": null,
    "selector-pseudo-element-colon-notation": null,
    "value-list-max-empty-lines": null,
    "value-list-comma-newline-after": null
  }
}

Hey!
Just to rule out the obvious first, are you using the latest version of the processor? Since we recently published a fix that was causing multiline interpolations to break the sourcemaps, and therefore report errors in wrong places?

Updated to 1.2.1 and updated appropriately. One of the errors now seems to be an erroneous "Missing semicolon" error, but that's probably now a stylelint thing. ButtonWrap seems to behave the same.

Okay, I'll try and run it locally and see if I can debug it a bit, I'll be back!

For one I just noticed that you have written &.active instead of &:active here:
&.active {
    background-color: ${COLOR_TAB_BACKGROUND_ACTIVE.string()};
    &:after {
      background-color: ${COLOR_TAB_HIGHLIGHT.mix(COLOR_WHITE, 0.3).string()};
      opacity: 1;
      transition: .1s;
    }
  }

Okay I think I identified at least the error in TabLink, it's an error introduced by the new fixes I mentioned previously, and it might cover both your cases. I'll see if I can get around to it tonight, but yep, definitely our bad. Thanks for the report!

Hey, a bit more information. Firstly, to fix your css you currently have 2 bugs in TabLink:

You're using the wrong interpolation tag in your multiline interpolation, it should be /* sc-value */ not /* sc-block */. Check out [this vocabulary list] if it wasn't just a typo and you want to see where you went wrong.

In the second to last block you have forgotten a colon in your opacity statement.

This should make your code run.
As you can see in the issue I just referenced this one from there's apparently a problem with running source maps when a CssSyntaxError occurs, which sucks! Hope we can find a way around that. I also think I might've spotted another bug which is what I was talking about above, but in the TabLink case it seems that it didn't affect you.

If you would like me to take a look at ButtonWrap I'll need the whole file since the sourcemaps aren't working, so the problem is probably simply in your CSS (though it could also possibly be some issues with our implementation of interpolations) but just in a completely different place, probably in the 75th line of CSS present in your file (skipping all non css stuff)

Emil, thank you so much for your keen eye! TabLink is behaving again, and now that I understand that the syntax error is related to the non-sourcemapped CSS string, I think I should be able to track things down in ButtonWrap. This file is essentially a big ball of styles and mixins that we refactored from SCSS earlier this week, so it might take a bit. I'll report back when I've tracked down the culprit.

After spending a few hours refactoring things, I've found and fixed a handful of syntax errors. However, I still am getting the error at approximately the same line number as before, so I likely missed the major one (not surprising, as I am a quite a CSS amateur) . Since I see that you're working on submitting a PR upstream, I'm thinking I'll stand down until that's worked it's way through the ecosystem. I'm happy to either test your candidate fix or provide you the full breaking code if you think that will help.

Hey! If you want you can replace the lib folder in your node_modules/stylelint folder with the one from here: https://github.com/emilgoldsmith/stylelint.
I believe I have it working now, just gonna finish up tests and gotta go through code review etc.

Oh sorry, forgot I hadn't pushed + wasn't linking to the branch (that didn't exist then either). Here's the correct link: https://github.com/emilgoldsmith/stylelint/tree/feature/apply-resultprocessors-on-CssSyntaxError

That seemed to do it! I went to the line and column suggested and discovered a malformed comment. Fixing it seems to have resolved things, so I think you're on the right track.
The only oddity I encountered was that your default formatter seemed to be json for some reason, so I had to change my npm script to manually set it back to to string via "lint-css": "stylelint --formatter string 'src/**/*.{js,jsx}'",
You can feel free to close out this comment if you want. I'll keep track of your progress in your other issue.
Thanks so much!

The only oddity I encountered was that your default formatter seemed to be json for some reason

Didn't change anything there so that sounds weird, I'm pretty sure the problem should be at your end.
And yep I'll close this one

We just published v1.3.0 that together with stylelint v9+ should now apply sourcemaps correctly to css syntax errors!


Replace ReturnChecker and UnreachableCodeChecker with the Behavior machinery from the spec

Replace ReturnChecker and UnreachableCodeChecker with the Behavior machinery from the spec
(Port https://bugs.webkit.org/show_bug.cgi?id=193487 to JavaScript)

Done a long time ago now.


chore change branch-2.0 version to 2.0.7

Proposed changes
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Added libi2c-dev rosdep rule

This package includes headers that allow for i2c devices to be accessed from userspace programs.
The particular use case that this was necessary for is a ROS node that fetches data from a Lidar-Lite v2 laser rangefinder.
On Arch, the required header is contained in this package:
https://www.archlinux.org/packages/core/x86_64/linux-api-headers/
On Debian and Ubuntu respectively:
https://packages.debian.org/jessie/libi2c-dev
https://packages.ubuntu.com/xenial/libi2c-dev

I looked for the fedora version, and didn't seem to find one.  Most I could find was this old bug report: https://bugzilla.redhat.com/show_bug.cgi?id=484128

Fixed the formatting, it's using the compact version now


nitmproxy v2.0.0

Merge HttpListener and FowardListener into NitmProxyListener
Initial support for response modification
Add listener.onInit (called on channel first initialized)
Add listener.onConnect (called on server channel connected)
Add listener.close (called on channel closed, to provide the change for users to release the resources)
Rename websocket functions to onWsRequest, onWsResonse (it's more clearly)

@abbasvalliani
Example code for http response intercept can be found here
It looks complicated, I will create some template interceptors to simplify the intercept process in separate PR.

Yes, I will check it. My setup is a local MITM proxy and an Android project.

@chhsiao90 My initial gut feeling looking at this is the level of complexity for the user. Any chance we can simplify this while still providing the control (if needed). I don't have an issue using the API as is, but could be cumbersome for someone new.

@chhsiao90 Any chance we can do a release of 2.0? Thanks!

nitmproxy v2.0.0 just released


Add Ability test golem

To playtest abilities in game, it'd be really handy to start with a max level golem. This golem would be useful just so I can snag some powers and test them out.
This golem would also need access to all jobs.

We could modify BobcatShark for this purpose. It's already a test-only golem (not available in-game by default, but can be explicitly set).
Although, now that I am thinking about it, we might need to keep BobcatShark lower-level to unit test leveling mechanics.


Replace events chan with an io.Writer

I think we're kinda sold into the jsonmessage package (CLI uses it to decode deploy output) so this just let's us remove a lot of code.

Sweet :+1:


Reflecting UI changes

Reflecting UI changes as of today for policy options.
Also adding a link to how the user can control the caller id if override is enabled.
https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/4626

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@LanaChin PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

Confirmed with PM. Made some minor changes and merged. Looks great @get-itips! Thanks for making the updates.


Translation file for zh-CN not found. Using default translation en-US.

can u help me to solve the issue when I  run demo.py>zh_CN ?
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

can u help me to solve the issue when I run demo.py>zh_CN ?
Describe the bug A clear and concise description of what the bug is.
To Reproduce Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior A clear and concise description of what you expected to happen.
Screenshots If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context Add any other context about the problem here.

我是通过更新chainlit版本后解决了这个问题，在容器终端执行：pip install --upgrade chainlit

version of chainlit-1.3.2 give
2024-11-20 00:10:57 - Translated markdown file for zh-CN not found. Defaulting to chainlit.md.


Add support for mdx 3

Describe the solution you'd like
Support mdx 3 please :-)
Additional context
I was planning to use mermaid within Astro but got this error instead:
[@mdx-js/rollup] Cannot read properties of undefined (reading 'call')
file: /home/millette/wip/mermaid-mdx/src/pages/page-2.mdx:undefined:undefined
  Stack trace:
    at Object.transform (file:///home/millette/wip/mermaid-mdx/node_modules/.pnpm/@astrojs+mdx@3.0.1_astro@4.9.1_@types+node@18.19.33_typescript@5.4.5_/node_modules/@astrojs/mdx/dist/vite-plugin-mdx.js:54:15)
    at file:///home/millette/wip/mermaid-mdx/node_modules/.pnpm/rollup@4.18.0/node_modules/rollup/dist/es/shared/node-entry.js:19774:40

Hi, currently I have no plans to implement support for mdx 3. A PR doing this is welcome


Add "NLP-Overview" project

Added a 1 line description that points to the NLP-Overview  website
Contribution Guidelines
Please ensure your pull request adheres to the following guidelines:

Search previous suggestions before making a new one, as yours may be a duplicate.
Make an individual pull request for each suggestion.
Use the following format: Bookmark Title: Description.
The title usually consists of several words. For bookmarks of an article or tutorial, use the title of the article.
The description consists of one-two sentences about the bookmark. It is required.
If the resource has 2 or more links please provide the Github repo link.
Additions should be added to the bottom of the relevant category.
New categories or improvements to the existing categorization are welcome.
Check your spelling and grammar.
Make sure your text editor is set to remove trailing whitespace.
The pull request and commit should have a useful title.

Thank you for your suggestions!

Thanks for the PR @omarsar
I've added a suggestion to the PR. Please review and accept/discuss?

@NirantK Excellent! Even better description!


test_detect_deadlock_when_shuffle_region failed

Bug Report
test_detect_deadlock_when_shuffle_region failed

[2019-12-04T03:00:58.887Z] thread 'server::lock_manager::test_detect_deadlock_when_shuffle_region' panicked at 'message: "stale command"', tests/integrations/server/lock_manager.rs:38:5
[2019-12-04T03:00:58.887Z] stack backtrace:
[2019-12-04T03:00:59.449Z]    0: backtrace::backtrace::libunwind::trace
[2019-12-04T03:00:59.450Z]              at /cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.35/src/backtrace/libunwind.rs:88
[2019-12-04T03:00:59.450Z]    1: backtrace::backtrace::trace_unsynchronized
[2019-12-04T03:00:59.450Z]              at /cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.35/src/backtrace/mod.rs:66
[2019-12-04T03:00:59.450Z]    2: std::sys_common::backtrace::_print
[2019-12-04T03:00:59.450Z]              at src/libstd/sys_common/backtrace.rs:47
[2019-12-04T03:00:59.450Z]    3: std::sys_common::backtrace::print
[2019-12-04T03:00:59.450Z]              at src/libstd/sys_common/backtrace.rs:36
[2019-12-04T03:00:59.450Z]    4: std::panicking::default_hook::{{closure}}
[2019-12-04T03:00:59.450Z]              at src/libstd/panicking.rs:200
[2019-12-04T03:00:59.450Z]    5: std::panicking::default_hook
[2019-12-04T03:00:59.450Z]              at src/libstd/panicking.rs:214
[2019-12-04T03:00:59.450Z]    6: std::panicking::rust_panic_with_hook
[2019-12-04T03:00:59.450Z]              at src/libstd/panicking.rs:477
[2019-12-04T03:00:59.450Z]    7: std::panicking::continue_panic_fmt
[2019-12-04T03:00:59.450Z]              at src/libstd/panicking.rs:384
[2019-12-04T03:00:59.450Z]    8: std::panicking::begin_panic_fmt
[2019-12-04T03:00:59.450Z]              at src/libstd/panicking.rs:339
[2019-12-04T03:00:59.450Z]    9: integrations::server::lock_manager::must_acquire_pessimistic_lock
[2019-12-04T03:00:59.450Z]              at tests/integrations/server/lock_manager.rs:38
[2019-12-04T03:00:59.450Z]   10: integrations::server::lock_manager::must_deadlock
[2019-12-04T03:00:59.450Z]              at tests/integrations/server/lock_manager.rs:46
[2019-12-04T03:00:59.450Z]   11: integrations::server::lock_manager::test_detect_deadlock_when_shuffle_region
[2019-12-04T03:00:59.450Z]              at tests/integrations/server/lock_manager.rs:136
[2019-12-04T03:00:59.450Z]   12: integrations::server::lock_manager::test_detect_deadlock_when_shuffle_region::{{closure}}
[2019-12-04T03:00:59.450Z]              at tests/integrations/server/lock_manager.rs:116
[2019-12-04T03:00:59.450Z]   13: core::ops::function::FnOnce::call_once
[2019-12-04T03:00:59.450Z]              at /rustc/c6e9c76c59e3c10acd63ca9ec157a8894ea1a068/src/libcore/ops/function.rs:227
[2019-12-04T03:00:59.450Z]   14: <alloc::boxed::Box<F> as core::ops::function::FnOnce<A>>::call_once
[2019-12-04T03:00:59.450Z]              at /rustc/c6e9c76c59e3c10acd63ca9ec157a8894ea1a068/src/liballoc/boxed.rs:922
[2019-12-04T03:00:59.450Z]   15: __rust_maybe_catch_panic
[2019-12-04T03:00:59.450Z]              at src/libpanic_unwind/lib.rs:80
[2019-12-04T03:00:59.450Z]   16: std::panicking::try
[2019-12-04T03:00:59.450Z]              at /rustc/c6e9c76c59e3c10acd63ca9ec157a8894ea1a068/src/libstd/panicking.rs:275
[2019-12-04T03:00:59.450Z]   17: std::panic::catch_unwind
[2019-12-04T03:00:59.450Z]              at /rustc/c6e9c76c59e3c10acd63ca9ec157a8894ea1a068/src/libstd/panic.rs:394
[2019-12-04T03:00:59.450Z]   18: test::run_test::run_test_inner::{{closure}}
[2019-12-04T03:00:59.450Z]              at src/libtest/lib.rs:1413
[2019-12-04T03:00:59.450Z] note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
[2019-12-04T03:01:01.964Z] test server::lock_manager::test_detect_deadlock_when_shuffle_region ... FAILED

It's unstable. I'll take a look today.


Allow single file uploads

Basically, allow for mutations instead of just like this:
updateAvatar(userId: String!, file: [UploadedFile!]!): Avatar

Also like this:
updateAvatar(userId: String!, file: UploadedFile!): Avatar

Clearer semantics.
Better validation for mutations requiring only one file in a variable.
It is sometimes easier to get a single File object to send as a mutation parameter:

handleSubmit = event => {
    event.preventDefault()
    const formData = new window.FormData(event.target)
    this.props.mutate({
      variables: {
        userId: this.props.userId,
        file: formData.get('avatar') // Instance of File (not FileList), from <input type='file' name='avatar' required />
      }
    }).then(({data}) => {
      console.log('Avatar updated', data)
    })
  }

To get this to work, handle File as well as FileList here and here.

Not certain how all this works, but here in graphql-server-express-upload also looks relevant.

For those who find this pressing, I ended up supporting single File objects, File arrays, and FileList objects intuitively in apollo-upload-client.


Restrict workload selection

Is your feature request related to a problem? Please describe.
One might have a huge cluster and want to focus on optimizing just a particular workload.
Prometheus queries while analysing all worklaods in a cluster/namespace might take too long otherwise.
Describe the solution you'd like
Introduce a new command line argument as kubectl has:
    -l, --selector='':
	Selector (label query) to filter on, supports '=', '==', and '!='.(e.g. -l key1=value1,key2=value2). Matching
	objects must satisfy all of the specified label constraints.

to limit the workload selection. E. g:
krr simple --context my-cluster --namespace kube-system --selector app-instance=metrics-server
# or
krr simple --context my-cluster --namespace kube-system --selector owner=devops

Describe alternatives you've considered
krr simple --context my-cluster --namespace kube-system --workload Deployment/metrics-server

Additional context
Thanks for this wonderful tool. Happy to learn if there is alternative solution for me.

Now I hit an issue when Prometheus is unable to give me the results and the request fails with 431. I don't understand the details krr, but would implementation of feature above help as well?

PrometheusApiClientException: HTTP Status Code 431 (b'431 Request Header Fields Too Large')

Thanks for reporting it @dohnto-s1
I think this issue is similar to #60
Does limiting krr to a specific namespace using krr simple -n default can be a workaround until we fix it?

This unfortunately happens when running with -n.

And I really like your proposal for selector flag, we should 100% implement it

@LeaveMyYard Looks awesome, thanks!


Typescripts enumsAsTypes option should derive their unions from a tuple with all the values

Is your feature request related to a problem? Please describe.
I like the option enumsAsTypes because I like working with string unions more in my code. However sometimes I need to be able to loop through all the values. That isn't possible unless the union type was derived from a tuple with the values.
an example:
enum Color {
    RED
    BLUE
}

produces
export enum Color {
  Red = 'RED',
  Blue = 'BLUE'
}

with this I could do Object.values(Color) and this would give me an array ['RED', 'BLUE']. However when I use enumsAsTypes: true I get
export type Color =
  | 'RED'
  | 'BLUE';

and I can't loop over these.
Describe the solution you'd like
when enumsAsTypes is true I would like for the input to not be:
export type Color =
  | 'RED'
  | 'BLUE';

but:
export const ColorValues = ['RED', 'BLUE'] as const;
export type Color = typeof ColorValues[number];

that way I can use ColorValues to loop over the values
Describe alternatives you've considered
No response
Is your feature request related to a problem? Please describe.
No response

That would reduce a ton of boilerplate we need to write right now.


improve About section

For this project , about section is poorly designed . I can redesign it to make it look good .

Hello! Could you assign this to me? @Anuj3553

Hello! Could you assign this to me? @Anuj3553

What you want to work. Give me some samples.

Hello! Could you assign this to me? @Anuj3553

Can you give star to this repo?

@Anuj3553 Can I work on this issue ?

@Anuj3553 Can I work on this issue ?

Yes sure

@Anuj3553 Do you have any specific concept about how to redesign the about us section ? Or anything to give me a head start.

@Anuj3553 Do you have any specific concept about how to redesign the about us section ? Or anything to give me a head start.

Well for now i have no idea but you can give me sample then we can discuss about this. You can also connect with my social links.


listening incoming connection

Hi,
Is it possible to listen for incoming connection ?
for example with a reverse vnc and vcnviewer I can use this command:
vncviewer -listen <IP_ADDRESS>:5500.

No, that's not implemented


ghostscript: fix broken rpath on Darwin

Dynamic library name on Darwin contains only 'maj.min' eg "9.53";
the build however used $version to set rpath;
this broke on 2029ca37 when $version went from "9.52" to "9.53.3".
Signed-off-by: Sirio Balmelli<EMAIL_ADDRESS>

@ofborg eval

@veprbl thank you for help cleaning this up. Re-rolled as a single commit and force-pushed.


Tiles are not shown if used in another instance of FlutterMap

For details look at the screen record from current version of example app.
To reproduce just start the example app and switch between 'MapController' and 'AnimatedMapController'. Unfortunately it’s blocking the app in my use case of FlutterMap because I push multiple instances of FlutterMap with ineractive: false to the navigation stack and you can just remove the 'grayscreen' by zooming in or out. Would be awesome to have a solution for that.

Best Regards

I think it has something to do with the CachingNetworkTileProvider because the issue does not appear on the ZoomButtonPlugins Page and there the NonCachingNetworkTileProvider is used.

You may be able to fake it by adding a move or slightly changing the zoom to something and back again, which will force a refresh ?

Thanks for that hint. Thought about that earlier but didn't came back to it. Tried it now and works fine :) added the following code:
@override
  void didUpdateWidget(SimpleMapView oldWidget) {
    super.didUpdateWidget(oldWidget);
    if (widget.position != oldWidget.position || widget.zoom != oldWidget.zoom)
      _mapController?.move(widget.position, 18);
    Future.delayed(Duration(milliseconds: 200)).then(
      (value) => _mapController?.move(widget.position, widget.zoom),
    );
  }

With flutter_map version 8, tiles are correctly displayed.
Seems that a commit from version 9 break something.
Need to explore git history and find the issue

@johnpryan PR available
Fix the race condition

⚠ After many tests, rotation is broken too since the commit d4aefaecc44d0c0b431375fd7bc97b2ef1a214e7
I'm working also to fix that

PR #715 is an easy applicable fix, try it out.


[Bug] – Fix Download link in README

The download links in the README should contain the whole version, e.g., https://github.com/dephraiim/minimal-youtube/releases/download/v1.0.0/minimal_youtube-1.0.0.zip.

Fixed it now. Thanks


Not receiving the callback from enqueueMapBuffer

I'm new to OpenCL. I've tried following the code from the readme and the tests, but I'm not getting a callback when calling enqueueMapBuffer. The promise exists, but no callback ever gets called, and I use the syntax .then(function(){...}).
I think some really simple examples would help with understanding how to use it, far simpler than a particle simulation. Being able to add two vectors of numbers together would be helpful to get going.
Is there a way to use enqueueMapBuffer asynchronously, using callbacks rather than promises?

Thanks for the feedback! I'm trying to get some time to add some basic examples to the readme. Until then, please do not hesitate to put your code there, I will help you to make it work for sure.
Yust for the record, promises are always asynchronous. Those like callbacks encapsulated to an object to have the ability to make them composable. You can use a promise by the good old node way:
...promise.nodeify(function(err, result) {
  // Whoa, finally a callback!
})

I don't get the time to document everything either, within my own project. I'm very much in favour of developers publishing what they have, even if it's not ready to be used. I think having OpenCL bindings in node is important, and I'm keen on helping to improve this project. This project looks as though the engineering is solid, but it's particularly hard to get started with because of the sparse documentation.
I have not actually got any OpenCL to run using this project. I did get OpenCL running through a C++ addon using https://www.olcf.ornl.gov/tutorials/opencl-vector-addition/, and have now modified that somewhat. Would everything there translate OK using NOOOCL?
The way that I think I'd proceed to get NOOOCL to work, so I could document it, would be to modify a test file so that it just executes some code. I did not get to actually run the tests though, what do I use to do that?
If you were to provide me with just one example of NOOOCL just doing something simple (with no test code) that would be a great help, but if you don't have time for that soon then I'd need to change some of the test code to only carry out the OpenCL.
I'm also interested in what patterns would make for efficient calling of OpenCL. With the current example I'm working on in C++ it compiles the kernel each time it's called. I think I could have it cache a whole load of things (particularly the compiled kernel) in order to actually run quickly. I'd like to develop and showcase patterns that enable OpenCL to be used efficiently in a variety of use cases.
Also, I think some simplified  wrappers would be of great help. That sounds like it should probably be outside of this project's scope, but it's something we could work together on if you are interested.
Are you still lacking a GPU that supports OpenCL 2.0? And if you were given one, would you have the time to work on the 2.0 API any time soon?

 Please take a look at the new readme.
Code, example or documentation contributions are welcome of course! Please do not hesitate to PR anything that you got, I'll review an merge them ASAP.
Regarding caching of the compiled kernel binaries, it is of course supported in NOOOCL. You can query compiled program binaries, and can make program from them later, which process is much faster than to compile them from source. See: https://github.com/unbornchikken/NOOOCL#build
I think abstractions of NOOOCL library are already simplified compared to native bindings, so simplify them more is kinda out of scope. But I can imagine a library on top of NOOOCL to do it (like OpenCLTemplate for .NET).
As soon as I get a capable card, I'm gonna do OCL 2.0 support for NOOOCL. It's not too hard, just kdiff cl.h 1.2 vs cl.h 2.0, then write some ffi binding code, and done. Right now getting a new card is on low  priority on my family budget, I have two small child you know. :)

I'm still waiting for your sample code describing the original issue regarding enqueueMapBuffer. Please put it there, I can help you with this for sure.

We've handled this issue privately. No work have to be done at NOOOCL side.


docs: docker core

Fixes #2866.
cc @jpadams. Please merge #2880 first

rebased

Now that:

https://github.com/dagger/dagger/pull/2880 is merged
addressed one warning https://github.com/dagger/dagger/pull/2884
fixed a bunch of related content https://github.com/dagger/dagger/pull/2885

I can finish reviewing this.

Having taken a step back and looking at it from further away, I don't think that this universe.dagger.io/docker page belongs under the Core Concepts section. This section was meant to cover the high-level, e.g. client, secrets, actions, plan, etc. This specific page seems to be a reference for the Docker package. As great & essential as this doc page is, I would put it under References, in a place that people can see it easily (I assume that most users will find it via Search).
WDYT @jpadams? Happy to defer the merge to you.

@gerhard I see what you mean. Perhaps detailed foundational docs like the FS doc and the Docker doc should be in References.
I think there is a conceptual doc that we're missing that explains how to think about using Dagger at a high level with some basic illustrations. Something like this (needs a lot of work):

An automation (plan) is typically implemented by pulling in source, strings, secrets, or artifacts from the client, git, or a container registry in the form of strings (can be a whole file), filesystem trees (FS, think directory structure), or container images, then building, testing, deploying by processing them in actions steps.
These actions themselves run in containers. Many actions employ simple alpine images (with more packages added if needed like bash, curl, jq) for the execution environment, but you can use any image that contains tools/commands/content that you need. The actions act on filesystems and their contents (e.g. strings in files): reading (copying or mounting), writing (modifying, exporting), executing (e.g. commands, scripts, API calls).
The actions create new filesystems or container images as output (optionally writing out to the client or pushing images/artifacts to a registry/artifact repo) and pass those to other actions. This continues until your build, test, or deploy has been accomplished.

@gerhard @jpadams. What is the next actionable step that I shall do ?

@jpadams Perhaps detailed foundational docs like the FS doc and the Docker doc should be in References.

Yes,  I think so too.

@jpadams: I think there is a conceptual doc that we're missing that explains how to think about using Dagger at a high level with some basic illustrations. Something like this (needs a lot of work):

Yes! That sounds like a great follow-up PR to this one.

@grouville: What is the next actionable step that I shall do?

My interpretation of @jpadams' last comment is to move this page under References & follow-up with another PR which adds a new page that expands on the quote from his comment. I am ready to review & merge that change as soon as you're ready @grouville.

If you go to the current redesign preview in #2853, you'll see that the "How it Works" section is a great spot for placing some sort of high-level explanation like the one @jpadams mentioned.

explanation

@jpadams, where should I move it ? How-it-works ?

Any updates on this?
Reading something like this may point us in the right direction.
I can see two possible paths:

making a "Background" category inside Core Concepts, that explains Dagger's inner workings (explanatory content is not a guide). We can add several explanatory guides here, and give them more visibility on the how it works page or even create a new card in the Overview page if it's necessary.
We can also make a Recommended section inside Guides, containing what we consider must-read content.

Having taken a step back and looking at it from further away, I don't think that this universe.dagger.io/docker page belongs under the Core Concepts section. This section was meant to cover the high-level, e.g. client, secrets, actions, plan, etc. This specific page seems to be a reference for the Docker package. As great & essential as this doc page is, I would put it under References, in a place that people can see it easily (I assume that most users will find it via Search).
WDYT @jpadams? Happy to defer the merge to you.

@gerhard I see what you mean. Perhaps detailed foundational docs like the FS doc and the Docker doc should be in References.

I think that base reference should inform the user on how to use the API: inputs, outputs, what does it do.
Which is what we're doing right now: https://docs.dagger.io/1234/dagger-types-reference
I think that a technical deep dive should have its own category. It could be under reference, (what tailscale does with its technical overview which is listed in reference: https://tailscale.com/kb/reference/)
So I think the current dagger.#FS documentation could be in Reference > Tech Deep Dive.
But we should have a simple documentation as well explaining the dagger.#FS API succintly. (with link to the deep dive)
For this current documentation (docker), it looks more like a deep dive as well, so I would put it there as well.

I think that base reference should inform the user on how to use the API: inputs, outputs, what does it do. Which is what we're doing right now: https://docs.dagger.io/1234/dagger-types-reference
I think that a technical deep dive should have its own category. It could be under reference, (what tailscale does with its technical overview which is listed in reference: https://tailscale.com/kb/reference/)
So I think the current dagger.#FS documentation could be in Reference > Tech Deep Dive. But we should have a simple documentation as well explaining the dagger.#FS API succintly. (with link to the deep dive)
For this current documentation (docker), it looks more like a deep dive as well, so I would put it there as well.

+1
I really like the idea of a "Tech Deep Dive" inside References.
If we want to highlight this section so it does not loose visibility (References category is the last in the sidebar) we can do any of the following things:

Add a CTA for the Deep Dive in the Overview
Add a CTA for the Deep Dive in "How it Works"
Add a CTA in related documentation that links to the Deep Dive section (for example at the end of Building container images)

@jpadams WDYT?

We didn't manage to find an arrangement. Closing it for now


Windows (Visual Studio 2013) build support for pack-asmjs and unpack-asmjs

This adds a workaround for C++11 unconstrained unions (not yet available in Visual C++) and adds project files for the two command line packing utilities.

Let me know if those changes aren't sufficient.

Thanks!

:heart:


Initial lock acquisition with timeout implementation.

@OliveGerste: Could you please check if that's what you expect?
Signed-off-by: Jérôme Benoit<EMAIL_ADDRESS>

I tested it in my local environment and it worked.
To see if it is working in real life, we could use it in the smart charging integration and start two transactions at the exact same time.
If smart charging runs twice, it works.

I will try to find time to add some unit tests, which might be a bit tricky to implement, unless you need it urgently and then I create another issue to add the unit test later on.
I'm not very happy with the busy loop wait and retry (CPU hog) but I think it should be enough. The other idea I have was to wait and retry by looping over an array of promises trying to get the lock.


Add fun widgets to the README

Thanks to @aliberts for the pointers!

This is how it renders

We'll need to provide more information in the releases (like Python versions we support, and the fact we're in alpha) to have more info in the display :)

It will pulled from the next release, that's why I edited the pytoml :)


Does not load following DVD queue redesign

Netflix (very) recently updated the DVD queue design and the sorter no longer loads.

Thanks for reporting the issue--I expect to take a look at this tomorrow.

Fixing this is non-trivial. The APIs have changed and I'll need to figure out how to use the new ones (without documentation) in the hopes they support changing the position of many rows with just one request. A slower alternative is to programmatically invoke the "Move to top" many times to move rows in their new position. This will be as slow as the sorts that need info from the movie details page in order to do their job.
I hope to reserve some time to take a look this weekend and get it fixed.

Have same issue, any progress?

Sorry, haven't had time...

Any progress?  I miss being able to actually use Netflix queues properly.  I don't know about the rest but I would pay for something that fixes Netflix's queue hotmess


External gBenchmark Build Broken

What version of gRPC and what language are you using?
C/C++, release 1.4.2 and master
What operating system (Linux, Windows, …) and version?
Linux, Debian 8.8 "jessie"
What runtime / compiler are you using (e.g. python version or version of gcc)
gcc/g++ 4.9.2
What did you do?
Building master with all-external dependencies for packaging.
cmake \
  -DgRPC_ZLIB_PROVIDER=package \
  -DgRPC_PROTOBUF_PROVIDER=CONFIG \
  -DgRPC_SSL_PROVIDER=package \
  -DgRPC_CARES_PROVIDER=package \
  -DgRPC_GFLAGS_PROVIDER=package \
  -DgRPC_BENCHMARK_PROVIDER=package \
  ..

fails with
CMake Warning at CMakeLists.txt:285 (find_package):
  By not providing "Findbenchmark.cmake" in CMAKE_MODULE_PATH this project
  has asked CMake to find a package configuration file provided by
  "benchmark", but CMake did not find one.

  Could not find a package configuration file provided by "benchmark" with
  any of the following names:

    benchmarkConfig.cmake
    benchmark-config.cmake

  Add the installation prefix of "benchmark" to CMAKE_PREFIX_PATH or set
  "benchmark_DIR" to a directory containing one of the above files.  If
  "benchmark" provides a separate development package or SDK, be sure it has
  been installed.

What did you expect to see?
Ideally, the library should also be build-able with all-external libraries.
This is quite important for packing the library, e.g. for spack or linux distributions.
What did you see instead?
The error above :)
The best way to fix this build error is to improve the install of benchmark itself: https://github.com/google/benchmark/issues/423
Alternatively, we will need a FindBenchmark.cmake script inside this repo (deprecated solution).
Anything else we should know about your project / environment?
I used CMake 3.0.2
This is a follow-up to #11140 which already solved a lot of issues (thanks @randomguy3 !)

Update: the master branch of gbenchmark already adds the proper files, just a new gbenchmark release (current stable: v1.1.0)  would be awesome :)


Add change_message_visibility

First of all thanks for this library! This PR adds support to change_message_visibility. Calling this method today will fail with ReceiptIsInvalid given it contains the RECEIPT_HANDLER_MATCHER.

Bump :) This fix would be great to have. @edumucelli I notice you haven't added the decorator on the Message object, does calling change_visibility() on a message work with fix?

@Urist, I am using that in prod and that is working fine. I have not seen a
need to change the Message.
On Thu, Nov 16, 2023 at 8:29 PM Joseph Barcham @.***>
wrote:

Bump :) This fix would be great to have. @edumucelli
https://github.com/edumucelli I notice you haven't added the decorator
on the Message object, does calling change_visibility() on a message work
with fix?
—
Reply to this email directly, view it on GitHub
https://github.com/QuiNovas/sqs-extended-client/pull/14#issuecomment-1815180891,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHKJB5PHZO6STZGAWEN4YDYEZSTPAVCNFSM6AAAAAA2W23IQCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJVGE4DAOBZGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Eduardo Mucelli R. Oliveira -- http://eduar.do

Awesome, thanks for the update!
On Fri, Nov 17, 2023 at 6:31 PM Joseph Wortmann @.***>
wrote:

Merged #14 https://github.com/QuiNovas/sqs-extended-client/pull/14 into
master.
—
Reply to this email directly, view it on GitHub
https://github.com/QuiNovas/sqs-extended-client/pull/14#event-10997211368,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHKJB7UK2AYTG3JIP6Z7WTYE6NQHAVCNFSM6AAAAAA2W23IQCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJQHE4TOMRRGEZTMOA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Eduardo Mucelli R. Oliveira -- http://eduar.do

@edumucelli Thank you for the PR. It is merged and should be available in PyPI under version 0.0.11.

@edumucelli Also note that if you have forked this the build process has changed to the modern python -m build


Add configuration option for authentication session duration

Currently the session duration is set to 30 days. This means if someone was to leave the company they may have access to the service for 30 days longer than they should. This should be reduced to something more like 1 day, and should be configurable.

Closed by #97
Use AUTH_EXPIRY_SECONDS env var to adjust time


Syllable Desktop LiveCD 0.6.6 (iso.xz) worked for me if runs via Syllable in a Virtual Machine (VMware, Virtual PC, VirtualBox) -> Syllable in VirtualBox (no SMP/ACPI), but it have some graphical issues

@SuperMaxusa Nice find. Indeed it works after adding disable_smp=true to the kernel command line in /boot/boot/grub/menu.lst.
Will add a profile to the website soon.

https://copy.sh/v86/?profile=syllable
A bit too large to put on the frontpage now, but still an interesting demo.


[MM-19208] Allow hidden tabs to still receive unread/mention updates

Summary
Desktop v4.3-rc2 was not updating mention/unread status in inactive tabs - seems Chrome related. This PR changes the mechanism for showing/hiding tabs to allow them to continue updating while inactive.
More details: it seems that setting the width/height on inactive tabs' webview to 0px/0px is causing Chrome to stop updating them, preventing the scraping mechanism from updating the count on the tabs.
Issue link
https://mattermost.atlassian.net/browse/MM-19208

Removing QA label, they've agreed to test afterwards


Manipulate request URL

How do you manipulate request URL before it is sent?
My guess is that I need to do override a method in the WebViewClient but I have no way of doing that since you do not expose the WebViewClient.
Is there a workaround?
Thanks
EDIT: Found "setWebViewClient" method.

Thanks for your question and posting the solution :)
Yes, the normal setWebViewClient(...) from the WebView class can be used.
There was some effort to extend the WebView class but at the same time keep all the behaviour and available methods the same. So you should be able to use all things that a normal WebView exposes.

Thank you very much!
I had to do this:
webView.setWebViewClient(new WebViewClient(){
        @Override
        public boolean shouldOverrideUrlLoading(WebView view, String url) {
            view.loadUrl(url, getExtraHeaders());
            return true;

        }
    });

to get my custom headers to show up at my backend.
Normally I would have used the "webView.addHttpHeader" method, but for reason, I did't set the custom headers at my backend.
You might need to look at it.
Thanks again!

This looks perfectly fine and is a clever solution!
Indeed, the values from addHttpHeader(...) are only used when calling loadUrl(...), i.e. for your initial request. Do you feel that this should be changed?

I think the design of addHttpHeader is pretty good because I don't have to implement a WebViewClient callback to add custom headers.
My point is: I used it at first, but it seem not to work so I went the custom WebViewClient way.
Calling addHttpHeader is pretty more convenient but it seem not to be working.

Yes, the issue is that the values from addHttpHeader(...) are only used when you call loadUrl(...).
This is typically your first request -- where you set the page from code.
Whenever the user follows a link or redirect then, loadUrl(...) will not be called. You see?
So maybe we should incorporate your fix with shouldOverrideUrlLoading(...) into this library. It seems that will help.
Setting the headers there seems to work every time. But we have to check against side effects and unexpected problems this could cause, of course.

Yeah, very good. I think it will also be nice to have a switch to enable or disable the sending of the headers on every request just in case it is not desirable in some other scenarios (overheads and what not).
In my scenario, I want to have the headers sent with every request to different the requests by device and version at my backend.
Thank you.

Great, we'll leave this issue open and implement a fix as soon as possible :)
Probably, we don't even need a switch, because looking at the method names and documentation, it really is a bug. So the behaviour that you requested should really be what everyone actually wanted in every use case.
Otherwise, one may call removeHttpHeader(...) after the initial request to remove the headers again.

Hi, I have the same problem.
I imported it in android studio, but don't recognize any function... i have a folder in my project called Android-AdvancedWebView and 3 file:
Android-AdvancedWebView.iml
Android-AdvancedWebView.jar
build.gradle

@scheccia1 Thanks, but doesn't really look like you have the same problem described in this thread. Your problem is that you simply can't get the library working, right? Can you try copying the AdvancedWebView.java source file to a package im.delight.android.webview in your own source code instead?

Excuse me,I thought I had answered to other iussue where there was a logcat, and you suggested to install jar.

@scheccia1 No problem! So what about copying the source code instead of including the JAR? Does it work?

@oderayi and everyone interested: This issue has been resolved now: The fix is in https://github.com/delight-im/Android-AdvancedWebView/commit/e90fc8243ca68a23088eb4f12dd53317873d4c4c and can be used by updating your dependency (see README). Sorry for the long delay!
@scheccia1 You can include this library via Gradle now (instead of JAR).


CMakeLists.txt file

I've written a CMakeLists.txt file to build libmxml with CMake, and was wondering if you'd be interested in adding it to the project.

Please attach it, will consider it...

Never attached, closing.