Introduce HTML viewer to display HTML file on Azkaban server

Currently html files are treated as text files on Azkaban server, its source code is displayed, which for most of the html files, is too complicated to work with. There are also needs for people to see the rendered html page instead of source code. This patch introduces a html viewer and modifies the template so that html file content is rendered. To protect people from malicious html files, the content is rendered within a sandboxed iframe, which prevents any javascript execution.
Previous PR not forbidding javascript execution: https://github.com/azkaban/azkaban-plugins/pull/280

How does this solution work if the file contains images for example?

@qil026 could you describe in more detail your choices of the iframe parameters?

I don't see unit tests exercising the JS functionalities. Did I miss them?

@HappyRay

How does this solution work if the file contains images for example?

Images are usually referenced with a url or base64 encoded, either way, it will be rendered correctly within an iframe.

could you describe in more detail your choices of the iframe parameters for future references?

iframe's sandbox parameter allows browser to completely sandbox the content stored in the iframe, the content cannot execute any javascript, cannot read cookies and is completely separated from the parent page. By default, sandbox makes the iframe be treated as a different origin, and due to origin policy, browser does not allow one origin to interact with another origin, but we have to write content into it, so I had to assign it the value "allow-same-origin" so they are treated as the same origin and thus allows our javascript to write html into the iframe.

I don't see unit tests exercising the JS functionalities. Did I miss them?

I couldn't figure out a way to unit test the JS, since unit test environment does not have web/js engine and it's not actually trying to render the html. I did many offline testing and verified that the javascript, regardless of how it is written (inline with html, standalone

Images are usually referenced with a url or base64 encoded, either way, it will be rendered correctly within an iframe.

Do you have an example?

but we have to write content into it, so I had to assign it the value "allow-same-origin" so they are treated as the same origin and thus allows our javascript to write html into the iframe.

Why is it necessary to inject contents into iframe via javascript? Can't the server generate the content of the iframe?

I couldn't figure out a way to unit test the JS, since unit test environment does not have web/js engine and it's not actually trying to render the html. I did many offline testing and verified that the javascript, regardless of how it is written (inline with html, standalone

@HappyRay

Do you have an example?

I have uploaded a couple of html files to HDFS and opened them up, I was able to see all the images rendered correctly.

Why is it necessary to inject contents into iframe via javascript? Can't the server generate the content of the iframe?
Good question, I tried writing the content directly inside the iframe tag, something like this document.getElementById("iframe").html = "my html file"; or generate the html source inside the iframe while building the template, like this: my html file, none of these worked, so I had to resort to using javascript to write the content into the iframe, as if it were coming from an actual url.

I was referring to the code in the hdfs-file-js.vm for example.
I was not aware of anyway to test that file. I didn't find existing tests for template files, if you have any pointers for me, I'd be happy to add some unit test in a follow up PR. Thanks!

I have uploaded a couple of html files to HDFS and opened them up, I was able to see all the images rendered correctly.

How does it work? Could you point me to the server code that returns the image data?

none of these worked, so I had to resort to using javascript to write the content into the iframe, as if it were coming from an actual url.

What problems did you run into?

I was not aware of anyway to test that file. I didn't find existing tests for template files, if you have any pointers for me, I'd be happy to add some unit test in a follow up PR. Thanks!

There are gaps in existing test coverage. That's why the new coding standard requires unit tests. See the dev guide at the main AZ project.
You can also take a look at the new JS unit tests @kunkun-tang added not long ago.

How does it work? Could you point me to the server code that returns the image data?
The java code is not responsible for fetching image. Chrome or Safari is. The server code simply returns the html source code to the browser, and browser can go fetch any image resources that's needed.

What problems did you run into?
For example, writing source code directly into iframe like this my html file, instead of seeing "my html file" in h1 font size&style (as you'd expect), the iframe shows an empty page, it doesn't recognize the inner html at all.

The image files are stored in HDFS. Right? How does it become accessible by the clients?
If image files are stored on HDFS, then no, it won't be accessible. It only works when image file is either base64 encoded or stored on other http servers, local images are not supported. This patch is not aimed to design a complete solution for rendering HTML files on azkaban, the goal is to support the rendering of simple HTML pages.

This is strange. Wouldn't the content be the same when you inject the content via Javascript?

Nope, based on my experiment, iframe doesn't work like that..

Where is the documentation?
please make sure the limitations are included. Given the limitations in features, lack of automated test coverage, I would consider this an experimental feature. The support from the Azkaban team will be limited. Please indicate that in the doc.

Do you have an explanation for your observations?


add database migration + seeder scripts

add support for migrations
convert existing single schema into a migration that can be applied or undo when needed
add commands into make file to make it convenient to apply / reset / undo migrations
add scripts to seed dummy data

will add database seeding in separate task, need to discuss whether it should be implemented as Go code or raw INSERT statements in .sql files.


Fail to send trace of last message before shutdown producer

BUG REPORT

Please describe the issue you observed:
Fail to send trace of last message before shutdown producer

What did you do (The steps to reproduce)?
send message and enable trace

    public static void main(String[] args) throws MQClientException, InterruptedException, UnsupportedEncodingException, MQBrokerException, RemotingException {
        DefaultMQProducer producer = new DefaultMQProducer("ProducerGroupName", true);
        producer.setNamesrvAddr("localhost:9876");
        producer.start();

        Message msg = new Message("TopicTest", "TagA", "OrderID188",
                "Hello world".getBytes(RemotingHelper.DEFAULT_CHARSET));
        SendResult sendResult = producer.send(msg);
        System.out.printf("%s%n", sendResult);

        producer.shutdown();
    }

What did you expect to see?
trace is sent sucessfully
What did you see instead?
trace sent fail:

Exception in thread "MQ-AsyncTraceDispatcher-Thread-c2c2798a-6f33-449d-a62e-f3243429c979" java.util.concurrent.RejectedExecutionException: Task java.util.concurrent.FutureTask@15bafbba rejected from java.util.concurrent.ThreadPoolExecutor@1b7d177d[Terminated, pool size = 0, active threads = 0, queued tasks = 0, completed tasks = 0]
	at java.util.concurrent.ThreadPoolExecutor$AbortPolicy.rejectedExecution(ThreadPoolExecutor.java:2063)
	at java.util.concurrent.ThreadPoolExecutor.reject(ThreadPoolExecutor.java:830)
	at java.util.concurrent.ThreadPoolExecutor.execute(ThreadPoolExecutor.java:1379)
	at java.util.concurrent.AbstractExecutorService.submit(AbstractExecutorService.java:112)
	at org.apache.rocketmq.client.trace.AsyncTraceDispatcher$AsyncRunnable.run(AsyncTraceDispatcher.java:258)
	at java.lang.Thread.run(Thread.java:748)

Please tell us about your environment:

Other information (e.g. detailed explanation, logs, related issues, suggestions how to fix, etc):

Why this happens:

    class AsyncRunnable implements Runnable {
        private boolean stopped;

        @Override
        public void run() {
            while (!stopped) {
                List<TraceContext> contexts = new ArrayList<TraceContext>(batchSize);
                for (int i = 0; i < batchSize; i++) {
                    TraceContext context = null;
                    try {
                        //get trace data element from blocking Queue - traceContextQueue
                        context = traceContextQueue.poll(5, TimeUnit.MILLISECONDS);
                    } catch (InterruptedException e) {
                    }
                    if (context != null) {
                        contexts.add(context);
                    } else {
                        break;
                    }
                }
                if (contexts.size() > 0) {
                    AsyncAppenderRequest request = new AsyncAppenderRequest(contexts);
                    traceExecutor.submit(request);
                } else if (AsyncTraceDispatcher.this.stopped) {
                    this.stopped = true;
                }
            }

        }
    }

In AsyncRunnable::run , traceContextQueue.poll use 5ms to get data in queue, during the 5ms, traceContextQueue is empty but data in queue is placed to contexts,   waiting for traceExecutor.submit.
so, if call traceExecutor.shutdown() in this 5ms, the data is lost, and exception is thrown.

that's a checkpoint. and
what's circumstances would call shutdown immediately after call send

@francisoliverlee In demos, and sendMessage command.

yes, we need a grateful shutdown hook


Update to latest version of libmill

note: don't forget that libmill wasn't updated to work with pipes and FIFO files

and doesn't have filesize function

I have crash on linux [probably] due to outdated sources
https://github.com/sustrik/libmill/issues/143


Add prob_map option to CUBDataset

https://github.com/chainer/chainercv/pull/399#issuecomment-334373228
I will explain the rationale behind the name prob_map.
The name has not been introduced in ChainerCV, and usually the name prob is used instead for an pixel-wise probability.
Although this is fine in the case of semantic segmentation, it is not the case when used together with image-wise labels.
In this case, CUBLabelDataset uses image-wise labels, so it seemed better to assign name to the probability that can be recognized as pixel-wise probabilities easily.
prob_map seems to be good because it implies that this is a pixel-wise annotation and range is [0, 1].

Let me confirm.

In semantic segmentation, we use label for pixel-wise data because users can know it from the context and the definition of SemanticSegmentationDataset.
Without any contexts, we use *_map for pixel-wise data to show it is pixel-wise.

Yes. That is what I wanted to say.

In detection, we are using score for confidence value(s). We should define the difference between score and prob.
I think these definitions are good.

score: confidence value(s) whose range is not limited.
prob: probability value(s) whose range is [0, 1].

I think that rule is good.
As you said, some detection functions use scores where probs should be used.

As you said, some detection functions use scores where probs should be used.

Yes. But we should use scores in detection context because the API uses scores.

Should we change that to probs as well?

Should we change that to probs as well?

I think we should not because some detection models return confidence values which exceed 1.

OK.
Then how about using prob when we always want it to be in range [0, 1].
Therefore, prob is a subset of score.

Then how about using prob when we always want it to be in range [0, 1].

It is OK. Please note that prob implies not only the range is [0, 1] but also the sum of probs is 1.

Therefore, prob is a subset of score.

Yes.

It is OK. Please note that prob implies not only the range is [0, 1] but also the sum of probs is 1.

OK.
In the case of CUBDataset, the probability map should be called fg_prob_map because it does not sum to one.

Well, I guess it is ok to call fg/bg probability object as prob.


Links without an href wrongly refocus an autofocus input

Describe the bug
Obviously an anchor without an href is ... wrong. But it looks like they produce incorrect behavior in SvelteKit, at least with respect to re-focusing an input with autofocus.
An anchor with no href re-focuses your input with autofocus.
An anchor with an href, but which calls preventDefault behaves correctly, and does not
Reproduction
https://github.com/arackaf/sveltekit-autofocus-bug-repro
Logs
n/a

System Info
System:
    OS: macOS 13.0.1
    CPU: (16) x64 Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
    Memory: 77.12 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.11.1 - /usr/local/Cellar/nvm/0.38.0/versions/node/v16.11.1/bin/node
    Yarn: 1.22.17 - /usr/local/Cellar/nvm/0.38.0/versions/node/v16.11.1/bin/yarn
    npm: 8.18.0 - ~/Documents/node_modules/.bin/npm
  Browsers:
    Chrome: 108.0.5359.98
  npmPackages:
    @sveltejs/adapter-auto: next => 1.0.0-next.90
    @sveltejs/kit: next => 1.0.0-next.581
    svelte: ^3.54.0 => 3.54.0
    vite: ^4.0.0 => 4.0.0

Severity
very minor annoyance
Additional Information
No response

The problem is that while normalizing/constructing the URL, there's no "ignore this event because no href set", instead the URL is resolved to the current page (which isn't wrong), but that results in some follow up logic, which is different to native browser behavior

Obviously an anchor without an href is ... wrong

It is not wrong and I have used a elements without href as well, though it can be abused. E.g. this SO question points to a library that uses a elements instead of just buttons.


New Pre-release update issue

App keeps crashing when doing anything

I updated the app , please delete ur current one and reinstall the one from the same release. Thanks ^^


Email upon order

Setup email system to allow:

email notifications upon order (to admin)
email receipt/summary to customer upon order
email upon fulfillment

Sites like shopify allow composition and sending of emails to customers adhoc, plus integration with stuff like mailchimp.
This is more general so Ive started a separate app to cover it: https://github.com/JamesRamm/wagmail
Development of wagmail is required to fulfill this issue...


Added Italian translation

Awesome, thank you so much for your help @Giancarmine! 👍

May you please put the translation on Crowdin too?

Crowdin updated :rocket:
Please check out this to hacktoberfest-update

Thank you so much. It worked. 😎
This repo should be Hacktoberfest-ready. Does your dashboard say something else?

If you like you can translate Pimple Popper into Italian too. 😉

Look nice :+1:


Improve examples

From PR substance/examples#21

Form Example: entity-tool needs an icon then the toolbar will not be offset.
Input Example: Maybe this should not set the cursor immediately. You will loose the ability to click and select the whole node and delete.
Focused: alien does not change color
Table Example: Not possible to insert content into empty cells. Maybe not have them sparse but with empty cells for now

Focused example works for me:

Now Alien also changes color for me.


OR-Tools integer variable changed to be binary internally in SCIP, Change not reflected by OR-Tools state

Thank you for providing OR-Tools! The interface is nice!
What version of OR-Tools and what language are you using?
Version: v9.5 (stable branch HEAD)
Language: C++
Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi)
SCIP
What operating system (Linux, Windows, ...) and version?
Linux
What did you do?
std::unique_ptr<gor::MPSolver> solver_{
      gor::MPSolver::CreateSolver("SCIP")};

// note: it's an integer variable, but currently 0/1, so SCIP changes the type to BINARY
auto const var = solver_->MakeIntVar(0.0, 1.0);

// doesn't do anything, or-tools believes it's a INTEGER
// (doesn't know about the internal change in SCIP)
var->SetInteger(true);

// not good! SCIP has var stored as BINARY
var->SetUB(2.0);

What did you expect to see
MakeIntVar should make an integer variable, independent of the provided bounds.
But even if creating a [0,1] valued integer variable leads to a BINARY variable, at least var->SetInteger(true) should change it back to INTEGER.
What did you see instead?
 /home/.../project/cmake-build-debug/_deps/scip-src/src/scip/var.c:7043: varProcessChgUbGlobal: Assertion `var->vartype != SCIP_VARTYPE_BINARY || SCIPsetIsEQ(set, newbound, 0.0) || SCIPsetIsEQ(set, newbound, 1.0)' failed.

Anything else we should know about your project / environment
I would like to change the model incrementally. The goal is to optimize seat reservation (in buses, trains, flights, etc.). So it would be great to solve the model incrementally (for each booking: check feasibility). That's why, in the beginning, the values can be 0 <= x <= 1 and later it changes to 0 <= x <= N  for N=2, 3, .. as more and more bookings come in.
Also regarding this, there is the following code comment:
https://github.com/google/or-tools/blob/92eab40155d5566039c92662632f196174f8da47/ortools/linear_solver/scip_interface.cc#L686-L687
This matches my measurements: creating a completely new solver object vs. reusing and adapting constraints as new bookings come in does not improve the performance. This is an indicator that the solver starts from scratch instead of reusing information from previous runs.
As I understand the SCIP homepage, they advertise the C++ API to support "modifying and iterated solving":

So I guess that this feature of SCIP was not available at the time of the implementation of the scip_interface.cc and therefore cannot be expected to be supported by OR-Tools?

Ah, now I understood: since booleans (1/0) are already integers, what I tried to do doesn't make sense. It's just a limitation that SCIP changes the type to binary if the range is 0-1 and then I cannot change it back to SCIP_VARTYPE_INTEGER easily. But maybe this has to do with the fact, that OR Tools assumes that SCIP doesn't support incrementally anyway. So maybe I need to try using SCIP directly without the wrapper for now.

AFAIK, incrementality will be a feature of the incoming math_opt

Thank you for your fast response! I will look into it. I'm also trying to use CP SAT now.

Only available if using the Bazel based build and only available in C++.
It is already production ready BUT api/implem is still subject to heavy breaking change...

Closing the bug now.
Good luck.

We encountered this bug as well. I'd like to reuse solver interfaces without having to call .Reset() on them to avoid the overhead of copying over all the variables/constraints to a new solver. Changing the variables, and updating the bounds seems like the best plan, but because of this bug, this is not possible ATM. I've tried setting a variable to 0, 10 and updating the bounds afterwards, but as soon as I .Solve() the first time, it will convert the variables to binary, and updating the bounds afterwards will cause an ABNORMAL output. I'm hoping there's something the or-tools team can implement to mitigate this.

We cannot. The scip interface is corrupted.

Is there any tracking about this issue on SCIP's end?  Is this something they don't ever intend to fix? Not exactly sure where to go.


fix: LlamaTokenizerFast to AutoTokenizer at flash_mistral.py

What does this PR do?
A few cases where you're using a mistral structure or mixtral structure but not a llama tokenizer, why not make it to call the AutoTokenizer in exception handling.
Similar PR #619
@Narsil

These are usually here because TGI code is merged before any Transformers PR land.


Missing property chargeBearer in periodic payments

In the table which describes all properties for each payment type there is a property chargeBearer which is missing in the model periodicPaymentInitiation_json but exists in the model paymentInitiation_json.
Either we remove that property from the description or we add it to the model.

In the NextGenPSD2 XS2A Framework yaml, the tables describing the models are different, as periodicPaymentInitiation_json and paymentInitiation_json have different properties. The tables in the swiss-ng-api.yaml have been copy-pasted and are identical, which is why the model isn't described correctly.
As the tables in the swiss-ng-api.yaml seem to be out of date anyway (see #44), we will thoroughly check the table and model of  paymentInitiation_json in a first step. After that, we will update the tables of periodicPaymentInitiation_json and paymentInitiationBulkElement_json to correctly represent the models.
This will also fix the issues #35, #36, #37, #38, #39, #40 and #42


Achieve a linear animation on line graph

How do I set up configurations for animations for a graph that tends to the last point on the data?
Example of chartjs here this is what I would like to achieve.

This might be helpful, https://apexcharts.com/javascript-chart-demos/line-charts/realtime/

Yes, there isn't an inbuilt one.


Allow webui to attach CD images with extensions other than .iso

Info

Which version of Pi are you using: Zero W
Which github revision of software: 6670fdf
Which board version: 2.4a
Which computer is the RaSCSI connected to: Mac SE

Describe the issue
RaSCSI on the command line is able to mount CD images with file name extensions other than .iso, such as .img and .toast
However, the webui currently only filters .iso files for listing in the Image File Management table.
For some examples of such CD images see http://shorturl.at/wKNY9

Created PR with fix https://github.com/akuker/RASCSI/pull/205


Rename *Filter types to *Matcher

Hey,
this is targeting issue #91.
It's nothing special but a big renaming commit to avoid confusion regarding the Matcher types.
Hope it helps. :)

Thx for the contribution @rabuu ! Did you close this PR by accident, or do you no longer want it to be considered for merge?

Oops, sorry. This was an accident. See #115.


zh: add more restrictions for shard support

What is changed, added or deleted? (Required)
add more restrictions for shard support
Which DM version(s) do your changes apply to? (Required)

[x] master (the latest development version, including v3.0 changes)
[x] v2.0 (TiDB DM 2.0 versions)
[ ] v1.0 (TiDB DM 1.0 versions)

What is the related PR or file link(s)?

This PR is translated from:
Other reference link(s):

Do your changes match any of the following descriptions?

[ ] Delete files
[ ] Change aliases
[ ] Have version specific changes
[ ] Might cause conflicts after cherry-picked

@lance6716 @CharLotteiu PTAL

LGTM


[feature request]: automatic markdown documentation generation

A pretty useful feature Typer has is automatic documentation generation. It's really helpful to have this as part of pre-commit hooks and in the CI pipeline.
I wonder if cyclopts could also have this?

I think we can do that! I think a limitation is that the output docs format will have to be the same format as App.help_format. I'll try and get to this!

@BrianPugh This feature would be very useful to us as we're building out a new CLI app. Were you able to chip away at this?

I haven't started this effort yet; I'll begin it after I finish up my dictionary-support efforts.

Hey @BrianPugh, do you need help with this?
I would really like to switch to Cyclopts in my projects, this is the only thing holding me back.
Let me know if I could do anything.

Thanks for offering! I'm in the middle of a big refactor that should make this feature pretty easy for a v3.0.0 cyclopts. Still targeting about a month away.

I've been looking at for the capability too to switch away from typer entirely.  Current we use https://github.com/bruce-szalwinski/mkdocs-typer for the doc's gen rather than typer's own, I would if a similar plugin would be possible.

this hasn't fallen of my radar; I'm just currently prioritizing getting CLI completions out, then I can work on this!


v5.0.0rc3

Merge pr #30 by @neizod
Fix wrong function name from pr #30
Add more tests for pr #30
Update dependencies

Coverage remained the same at 95.122% when pulling 25ce9c19ebfc71dd0f7a4c90c819259483fa39cb on v5.0.0rc3 into 6778f2d27b7f091510ffa810cb69227f1f3a30fa on main.

Coverage remained the same at 95.122% when pulling 25ce9c19ebfc71dd0f7a4c90c819259483fa39cb on v5.0.0rc3 into 6778f2d27b7f091510ffa810cb69227f1f3a30fa on main.


Doesn't show drawer contents for non Oak wood drawers

Looks like the One Probe shows me a grid of icons and quantities when I'm looking at an Oak Storage Drawer but when i'm looking at another non-vanilla wood-type drawer (Lime, for example) I don't see those item icons or the quantities.

Hmm aren't you looking at an empty barrel? Can't really see clearly

No but I'll include a better picture. This may be something with non oak drawers anyway because I can't use lime drawers in some recipes either. For example, I can't make compacting drawer with lime drawer as part of the recipe. Does that mean it's an ore dictionary issue with storage drawers?
Sent from my iPhone

On Dec 24, 2017, at 3:06 AM, McJty<EMAIL_ADDRESS>wrote:
Hmm aren't you looking at an empty barrel? Can't really see clearly
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

McJty - here's a better set of pictures.  Also I'm running Direwolf20 1.12 v1.3 - should have provided that information earlier.


1.17.0 doesn't compile on Xcode 16 Beta 3

Describe the bug
The library doesn't compile on Xcode 16 Beta 3 due to a change in the swift-testing Issue.record() function arguments. Specially the error is: Extra arguments at positions #2, #3 in call
The Issue.record() function uses sourceLocation instead of filePath and line. It looks like we have to construct an instance of SourceLocation and provide in addition to filePath and line also column and fileID  the latter can be created using fileID()  function so it looks like the recordIssue() function of swift-testing will need some changes to pass in an existing SourceLocation or fileID
To Reproduce
Open project in Xcode 16 beta 3 and compile.
Expected behavior
Compiles ;-)
Screenshots

Environment

swift-snapshot-testing version 1.17.0
Xcode 16.0 beta 3 (16A5202i)
Swift 6.0
OS: 15.0 Beta (24A5279h)

🎉


[WIP] Padding binary data with '0x'

[WIP] Should fix #179 / #180

This has been changed in Truffle 2.0, but the code that does deployment as well as padding the binary (EtherPudding now ensures it's padded). Thanks for the PR, however!

Nice, so this should fix #179, correct?

Confirmed that this is fixed in 2.x.  Thanks @thiagodelgado111 for diving in!


Opencv rotate point cloud example

Hi there.
I added example for rotate point cloud with opencv.

Hi @Allius27
Very nice, thank you


Add queries using JOIN and Reference

This pull request adds more queries using JOIN and Foreign Key Reference. It also adds more tables to the database. The various queries that were performed are provided in screenshots below.
- What animals belong to Melody Pond?

- List of all animals that are pokemon (their type is Pokemon).

- List all owners and their animals, remember to include those that don't own any animal.

- How many animals are there per species?

- List all Digimon owned by Jennifer Orwell.

- List all animals owned by Dean Winchester that haven't tried to escape.

- Who owns the most animals?

Hello,
Here are the screenshots of all the required changes (insertions to data.sql).
Thanks 😄


[SPARK-39016][TESTS] Fix compilation warnings related to "enum will become a keyword in Scala 3"

What changes were proposed in this pull request?
There are some compilation warnings:
[WARNING] spark-source/core/src/test/scala/org/apache/spark/internal/config/ConfigEntrySuite.scala:172: [deprecation @  | origin= | version=2.13.7] Wrap `enum` in backticks to use it as an identifier, it will become a keyword in Scala 3.
[WARNING] spark-source/connector/avro/src/test/scala/org/apache/spark/sql/avro/AvroSuite.scala:553: [deprecation @  | origin= | version=2.13.7] Wrap `enum` in backticks to use it as an identifier, it will become a keyword in Scala 3.

this pr just rename the variable name to fix compilation warnings.
Why are the changes needed?
Fix compilation warnings.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Pass GA

Merged to master

thanks @srowen


Avoid "Unknown report extention" and default to report.json

There is probably a far better way to handle this. Just pushing here to hear about your suggestions.

Thanks, it is indeed annoying to get that error. I would suggest to use the following data types:
enum ReportFormat {JSON, BINARY}
struct ReportRequest {
  path: PathBuf,
  open: bool,
  format: ReportFormat
}

And use that instead of the current (&PathBuf, bool) argument. That way we can parse the format early on and avoid the error/default format.

Actually open is not relevant here because we can always serve a report without writing it to disk. See #127

That could be a followup to #132 132

This is now implemented in #132


Rethink typing constraints

At the moment there are lots of method ambiguities lurking in the shadows. As a result, everything up to the backend argument and even sometimes beyond must be types rather severely as Number or AbstractArray. I wonder if we could get around that

For now, Aqua.test_all ignores ambiguities, I think that might be enough


flex-launcher does not wait long enough for a program to start.

If a program takes longer to start, the flex launcher remains visible in the foreground. I have that, for example, with emulationstaion. Because i have a big rom collection it takes a moment to start.

How long does your emulationstation typically take to start? The launcher assumes that the launch application has failed to start if it doesn't grab the window focus after 7 seconds
-------- Original Message --------
On Mar 10, 2023, 4:59 AM, xslushedx wrote:

If a program takes longer to start, the flex launcher remains visible in the foreground. I have that, for example, with emulationstaion. because i have a big rom collection it takes a moment to start.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

With a fresh Windows 10 start, emulationstation actually needs seven to eight seconds to start. That is of course very long. I have now created a batch file to launch it and flex-launcher is working fine.

is the waiting time configurable?

is the waiting time configurable?

No, but it would be relatively easy to add one. Are you having this same issue?

Yeah, unfortunately I have a game that takes around 10 seconds and even the screen is showing the game, the focus is kept on flexlauncher :/

If you could guide me on where to look around for implementing this feature, I would be more than happy to submit a PR! :)

@sekmo The timeout value is controlled by a macro defined here. You will need to convert this into a config option.

Add a new int variable to the Config struct declaration (in launcher.h)
Define the setting key and default value in config/config_settings.cmake
Make those CMake variables available to the program in config/launcher_config.h.in
Initialize the new variable in launcher.c
Add the setting and default value to the default config file in config/config.ini.in
Parse for the new config setting in util.c. The setting facing the user should be in seconds, but the program uses milliseconds internally so you'll need to multiply it by 1000.

If you are not able to do this, then let me know and I can do it.

Omg thank you so much!
(Sorry if I didn't get back on this, I was eager to help out but I didn't have in the last period :/)


fix #910 change package name from org.apache.gearpump to gearpump

Hi Huafeng,
This is a requirement from Apache Foundation before incubation.
We need to make sure we have retained all the history when this rename. Please double check this and do some experiments to see whether the history is still there.

ok, I'll do that.

how is this going?

Waiting for other big pull request to be merged into master.

flink has done this before when they rename from stratosphere https://github.com/apache/flink/commits/master?page=70

Will the new namespace to be "io.gearpump" or "gearpump"? I vote for "io.gearpump"

+1 for io.gearpump

Lets wait for PR

Connector framework.
Dynamic DAG.
Additions?

After these, we will restart this process.

The connector framework including refactoring external modules would require more time and we already have the interface in so I suggest we restart this right after Dynamic DAG.

shall we restart this since those two PRs are in ?

Agree, I'll do that when kafka_hdfs is merged.

Close this pull request, will submit another one since the code has been refactored a lot.

we have scalajs to merge which has dependencies on backend due to shared messages.


Dynamically set Element public data

See original issue on GitLab
In GitLab by [Gitlab user @tristanvb] on Apr 28, 2017, 09:27
Public data
is exposed on an Element so that other Elements which depend on the given element can read it
It is only interesting to read public data from dependencies
when an Element assembles its artifact. Current use cases of
this include integration commands and artifact splitting rules.
In some cases, like when deploying debian packages from imported
debian supporting sources, it will be important to populate the
splitting rules (essentially file manifests) at the time when the
artifact is built, as we will only really know which files to put
into which package after having run the build.
This task will consist of:

ensuring that public data is recorded in an artifact's metadata
ensuring that the public data is read back into the BuildStream data model whenever the pipeline is loaded and the artifact is found to already be cached
adding some API to the Element allowing the Element to programmatically assign public data at Element->assemble() time.

In GitLab by [Gitlab user @juergbi] on Jul 4, 2017, 14:25
mentioned in merge request !45

In GitLab by [Gitlab user @juergbi] on Jul 4, 2017, 14:32
assigned to [Gitlab user @juergbi]

In GitLab by [Gitlab user @tristanvb] on Jul 5, 2017, 13:14
Fixed by Jürg's branch !

In GitLab by [Gitlab user @tristanvb] on Jul 5, 2017, 13:14
closed


feat: extend application endpoint with service user details

Describe your changes
Ticket: https://pins-ds.atlassian.net/browse/ASB-2164
Updates the GET [/api/v1/applications/{caseReference} for back office integration to show user details from service user table.
Type of change 🧩

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[x] I have performed a self-review of my own code
[x] I have double checked this work does not include any hardcoded secrets or passwords
[x] I have made corresponding changes to the documentation
[x] I have provided details on how I have tested my code
[x] I have referenced the ticket number above
[x] I have provided a description of the ticket
[x] I have included unit tests to cover any testable code changes

:tada: This PR is included in version 2.17.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.22.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Issue 22

prof concept for opencl caffe on android

Looks fine, but minor comment - rather than changing target exe for caffe-time to caffe, I would suggest actually to keep caffe-time.cc and target bin as caffe-time, not to mix up with official "caffe" binary which can do lots of stuff ... What do you think?


LXD init fails to correctly parse hybrid network configs (v2&v1)

Required information

Distribution: Ubuntu
Distribution version: 18.04
The output of "lxc info" or if that fails:

Kernel version: 4.15.0-91-generic
LXC version: 4.0.0
LXD version: 4.0.0
Storage backend in use: Ceph Octopus

Issue description
While attempting to "lxd init" a new cluster enabled host and create a Ceph remote storage pool, instead of creating the pool, this error is given:
Error: Failed to create storage pool 'remote': Failed to run: ceph --name client.admin --cluster ceph osd pool create lxd 32: unable to parse addrs in '[v2:XXX.XXX.XXX.XXX:3300/0,v1:XXX.XXX.XXX.XXX:6789/0]'
Steps to reproduce

Create Ceph cluster
lxd init
Say yes to remote storage

Information to attach
As the lxd init process fails at that point, there is no other useful output at this time :(

@monstermunchkin can you take this one in the morning?

@foobazbar123 Would it be possible for you to share your ceph.conf? I'm unable to reproduce your issue. My ceph.conf is as follows:
[global]
fsid = <id>
mon_initial_members = ceph-node1
mon_host = [v2:<ip>:3300/0,v1:<ip>:6789/0]
auth_cluster_required = cephx
auth_service_required = cephx
auth_client_required = cephx

@foobazbar123 Would it be possible for you to share your ceph.conf? I'm unable to reproduce your issue. My ceph.conf is as follows:
[global]
fsid = <id>
mon_initial_members = ceph-node1
mon_host = [v2:<ip>:3300/0,v1:<ip>:6789/0]
auth_cluster_required = cephx
auth_service_required = cephx
auth_client_required = cephx

Unfortunately, in the meantime I needed a working cluster so downgraded to Nautilus which is working fine. I memory serves though, mine was very much like yours. However, my global section didn't have the auth bits yours mentions after the mon_host field. My mon_host field was identical to yours, I believe.

That's odd. I tested it on both Nautilus and Octopus, and didn't get any error.

Oh, could just be an issue of the ceph client version.
@foobazbar123 If you can get into a broken state easily enough, would you mind refreshing to latest/beta as that channel has a much more recent version of the ceph client.

Our stable/candidate channels have Ceph 10.2.11, our edge/beta have 12.2.13

Our stable/candidate channels have Ceph 10.2.11, our edge/beta have 12.2.13

Getting into a broke state is probably easy enough. Getting back out would be less fun :D My Ceph is running on bare metal so can't just revert a snapshot.
That said, I suspect that you're probably right about the version difference in the client being likely the core issue. Especially if @monstermunchkin is using latest/beta since I definitely wasn't using it

Ok, let's assume that's the issue for now then.
The error is coming from the ceph command line tool, so that's unlikely to be LXD's fault.
I'm working on fixing a number of issues with the snap ahead of moving everyone onto the new base including the new ceph release, so should just be a matter of days or a couple weeks at worst.


Remove date from custom posts

I'm using the custom widget to add several extra pages (linked from the tabs at the top) and each of them shows the date at the top of the post. A date makes sense for a post/publication, but not for an 'about' style page. Is there any way to not show the date?

Widgets should be added to Widget Pages rather than ordinary pages - refer to the documentation:  https://sourcethemes.com/academic/post/managing-content/#create-a-widget-page

ok, and how can I get an ordinary page without a date?


If I just don't wanna draw in the canvas, just wanna click.

I know that to set the compositeOperation options can change the "draw" or "erase" mode .
what if i just wanna do something else in the canvas,
can you make another options "none" or something to not do anything in the canvas

Composite operation can be any of these https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D/globalCompositeOperation


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter enet for common errors and appropiate suggestions to keep this adapter up to date.
This check is based the current head revisions (master / main  branch) of the adapter repository
Please see the result of the check below.
ioBroker.enet
   -
ERRORS:

[ ] :heavy_exclamation_mark: [E254] Versions "1.2.2, 1.2.0" listed at common.news at io-package.json do not exist at NPM. Please remove from news section.
[ ] :heavy_exclamation_mark: [E952] .npmignore not found

WARNINGS:

[ ] :eyes: [W127] Missing suggested translation into ru,pt,nl,fr,it,es,pl,uk,zh-cn of "common.titleLang" in io-package.json.
[ ] :eyes: [W134] Missing suggested translation into pl,uk,zh-cn of "common.desc" in io-package.json.
[ ] :eyes: [W135] "common.tier" is required in io-package.json. Please check https://github.com/ioBroker/ioBroker.docs/blob/master/docs/en/dev/objectsschema.md#adapter.
[ ] :eyes: [W154] Missing suggested translation into it,es,pl,uk,zh-cn of some "common.news" in io-package.json.
[ ] :eyes: [W173] Potential sensitive data "password" not listed at "protectedNative" in io-package.json
[ ] :eyes: [W174] Potential sensitive data "password" not listed at "encryptedNative" in io-package.json
[ ] :eyes: [W181] "common.license" in io-package.json is deprecated. Please define object "common.licenseInformation"
[ ] :eyes: [W184] "common.main" is deprecated and ignored. Please remove from io-package.json. Use "main" at package.json instead.
[ ] :eyes: [W184] "common.materialize" is deprecated for admin >= 5 at io-package.json. Please use property "adminUI".
[ ] :eyes: [W184] "common.title" is deprecated and replaced by "common.titleLang". Please remove from io-package.json.
[ ] :eyes: [W252] Version of package.json (1.2.2) doesn't match latest version on NPM (1.0.1)
[ ] :eyes: [W513] "gulpfile.js" found in repo! Think about migrating to @iobroker/adapter-dev package

SUGGESTIONS:

[ ] :pushpin: [S522] Please consider migrating to admin 5 UI (jsonConfig).
[ ] :pushpin: [S526] Consider adding plugin "@alcalzone/release-script-plugin-manual-review".

Please review issues reported and consider fixing them as soon as appropiate.
Errors reported by repository checker should be fixed as soon as possible. Some of them require a new release to be considered as fixed. Please note that errors reported by checker might be considered as blocking point for future updates at stable repository.
Warnings reported by repository checker should be reviewed. While some warnings can be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
Suggestions reported by repository checker should be reviewed. Suggestions can be ignored due to a decision of the developer but they are reported as a hint to use a configuration which might get required in future or at least is used be most adapters. Suggestions are always optional to follow.
You may start a new check or force the creation of a new issue at any time by adding the following comment to this issue:
@iobroker-bot recheck
@iobroker-bot recreate
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence
Last update at Fri, 13 Sep 2024 10:33:05 GMT based on commit 1f48c7cc742b3d16250ad0d0c58f4ebd0f9a181b
ioBroker.repochecker 3.0.5

This issue has been updated by ioBroker Check and Service Bot
The following issues have been fixed
[W184] "common.main" is deprecated and ignored. Please remove from io-package.json. Use "main" at package.json instead.
:thumbsup:Thanks for fixing the issues.
The following issues are new and have been added
[W184] "common.main" is deprecated and ignored. Please remove from io-package.json. Executable is defined by entry "main" at package.json.


 This is why domains associated with the "Mercado Pago" aren't present in some account deletion entries. In this case, only the domains related to "Mercado Libre" are present.

Thanks for your contribution!


Revert "chore: Enable proguard and add release app to Travis"

Reverts fossasia/open-event-orga-app#126

@iamareebjamal instead of reverting and making another commit, can you fix this directly ?

Doing that @niranjan94
Sorry again

No problem. it's alright


Update debugging-in-vscode.md

Add note how to set source maps with Vue CLI 3

This is great! I've also talked with the VS Code team, they're going to update their cross post as well. Thanks!


Report number of duplicate trie nodes received during state sync

https://github.com/ethereum/py-evm/pull/1097#discussion_r205439467

In order to implement this we would need to keep track of the hashes of all nodes we've received, and for a full state sync that means more than 5GB of data, so we should probably reuse the existing LevelDB nodes cache from StateDownloader


Create settings page

Is there a way for me to make him locally stored, as a parameter similar to a global variable

It would be great that this settings are stored internally.

Is there a way for me to make him locally stored, as a parameter similar to a global variable

Hi bilisheep, how do you set your own gpt-4o? I tried to use openAIApiKey = "my_api_key" and openAIBaseURL = "my_end_point" in the file config.toml, but it doesn't work


Buffer diff from previous commit not working properly.

This happens when I try to display diff from previous commit. On commit selection part it displays changes correctly (see image 1) but when I open it with split (with <CR>) it does not properly what has changed. (see image 2)

This is intended. The default action <CR> will open a diff between the current file and the selected commit. Doing this action on the last commit, there is a chance it won't show any differences with the currently open file. <CR> won't open a diff of the commit's changes, <C-e> is bound to the functionality you want, this will open the entire commit.
Hope that answers your question.


[APIView] Create Isolated Test Environment

Incorporate Storage Emulation with existing Cosmos Emulator in dev environment to create consistent predictable data to allow for isolated test environment.

This work is done. I will now use https://github.com/Azure/azure-sdk-tools/issues/4863 to track further APIView testing work.


Wrapped $user in preg_quote function

This pull request is to fix #3364 where a username with square brackets will prevent encryption of the password. Thanks to ozh for showing me the way!

Nice and fast, thanks :)
What would be really cool would be to update unit tests as well, in tests/data/auth/config-test-auth.php, so we make sure that in the future this edge case is still correctly dealt with

Just so I'm understanding the test enough, is it sufficient to add another line with a user with brackets and a random password into the array?

I'm quite lagging behind several PR's, sorry for that. Will look into this asap


Tell the user that an existing project cannot be used

I tried to access pre-existing projects, and nothing happened -- the screen remained blank, and there was no error message.

Yes @eschnett this was the reason that this issue was created! :smile:


Consolidate child process handling code

This is no functional improvement, the code works as-is, but for a different reason that originally intended / commented. This attempts to make the code more expressive and correct.
Commit message:
`communciate()` calls `wait()` behind the scenes, but only
`wait()` is actually what is required here to populate
the `process.returncode` object (the `wait()` system call
is doing exactly that: wait for process termination and
reap the zombie state and read the exit status). With that
change `poll()` is not actually required because the exit
status is known to be known.

I am throwing this over the fence a bit, let's discuss on Slack if desired :-).

This looks good to me. The StackOverflow issue seems to refer to a full buffer that occurs when the pipe is not drained. This is not a problem in this case.
Having just reviewed the subprocess documentation, I notice that return code can be negative when a signal terminates the process. Not directly related to this change, but should if process.returncode > 0: use != 0 instead?

Not directly related to this change, but should if process.returncode > 0: use != 0 instead?

Yes! :)

Thanks for the change and the lesson, both!


mouse events fail after sprite removed from parent and added again

Hi, I'm making a solitaire card game in Phaseer.
I have a problem when cards are being dragged and dropped in the game.
I might be over complicating things, but couldn't think of a better way:
Given I have 3 cards dragged to pile, eg. 7hearts, 6clubs, 5diamonds (7h, 6c, 5d)
I want to be able to drag 6c & 5d together and drop them on another pile.
To implement this, when the 6c drag start is detected, I add the 5d as a child of the 6c sprite.
When they are dropped on the other pile, I then removeAllChildren from 6c sprite, and re-add the 5c sprite to the game. As it was before. and re-enable drag on it.
The trouble is, after this process the 5c becomes invisible to the mouse, and only the 6c is draggable.
Could this be a bug in the framework, a problem with my code.. or can anyone think of a workaround which might help?
The important code for this functionality is below:
        onDragStart: function(sprite) {
            console.log('{Card} onDragStart :: pileCardsLen=', this.pileCards.length);
            var self = this;
            if (this.pileCards.length) {
                this.attachPiledCards();
            }
        sprite.bringToTop();
        self.cardPicked.dispatch(self);
        this.dragInterval = setInterval(function() {
          self.onDragUpdate(sprite);
        }, 100);
       },

      attachPiledCards: function() {
        var i, piledCard, len;
        len = this.pileCards.length;
        for (i = 0; i < len; i++)
        {
          piledCard = this.pileCards[i];
          console.log('{Card} attachPiledCards :: pileCardName= ', piledCard.name);
          //piledCard.disableClick();
          //piledCard.disableDrag();
          var sp = piledCard.sprite;
          this.sprite.addChild(sp);
          this.sprite.tint = 0.2 * 0xff0000;
          sp.x = 0;
          sp.y = 17 + (i * 17);
        }
      },

      deattachPiledCards: function() {
        var i, piledCard, len;
        len = this.pileCards.length;
        this.sprite.removeChildren();
        console.log('{Card} deattachPiledCards :: len', len);
        for (i = 0; i < len; i++)
        {
          piledCard = this.pileCards[i];
          var sp = piledCard.sprite;
          this.game.stage.addChild(sp);
          sp.x = sp.x + this.sprite.position.x;
          sp.y = sp.y + this.sprite.position.y;
          //this.sprite.tint = 0xffffff;
          piledCard.softEnableDrag();
        }
      },

Sorry it's taken a while to respond to this, but it's more the sort of question best suited for the forum or stackoverflow than the issues list. If you can create a small reproducible test case that shows a bug in effect then we'll happily investigate.
I notice you're adding children directly to the stage for some reason, which is outside the boundary of the game world, so it may be that this isn't helping matters. Also you don't need to remove a child from a Group before adding it to another one - just add it immediately to the other Group and it will re-parent itself, stopping it from becoming unparented.


Marks now update if not reverting to incomplete and changed

This occurred when a submission was viewed, and reverted from released, had the mark changed, and then re-releasing. The average would not change since it assumed that it was reverted back to 'incomplete' first.


nitrite-explorer jar not found

The nitrite-explorer 3.2.0 jar is not present on the release page. On clicking on the file provided it says not found.

I am able to download it fine. Can you try again? May be a temporary github glitch.


zcompare:1: parse error near `)'

Mac OS 10.15 beta
My current shell is Fish-shell

.zimrc content

#################
# CORE SETTINGS #
#################

#
# Zim settings
#

# Select what modules you would like enabled.
# The second line of modules may depend on options set by modules in the first
# line. These dependencies are noted on the respective module's README.md.
zmodules=(directory environment git git-info history input utility custom \
          prompt completion syntax-highlighting history-substring-search )

###################
# MODULE SETTINGS #
###################

#
# Prompt
#

# Set your desired prompt here
zprompt_theme='steeef'

#
# Completion
#

# Set an optional host-specific filename for the completion cache file. If none
# is provided, the default '.zcompdump' is used.
#zcompdump_file=".zcompdump-${HOST}-${ZSH_VERSION}"

#
# Utility
#

# Uncomment to enable spelling correction prompt for commands. See:
# http://zsh.sourceforge.net/Doc/Release/Options.html#Input_002fOutput
#setopt CORRECT

# Set custom spelling correction prompt
#SPROMPT='zsh: correct %F{red}%R%f to %F{green}%r%f [nyae]? '

#
# Environment
#

# Set the string below to the desired terminal title format string.
# The terminal title is redrawn upon directory change, however, variables like
# ${PWD} are only evaluated once. Use prompt expansion strings for dynamic data:
#   http://zsh.sourceforge.net/Doc/Release/Prompt-Expansion.html#Simple-Prompt-Escapes
# The example below uses the following format: 'username@host:/current/directory'
ztermtitle='%n@%m:%~'

#
# Input
#

# Set to vi or emacs
zinput_mode='emacs'

# Uncomment to enable double-dot expansion. This appends '../' to your input for
# each '.' you type after an initial '..'
#zdouble_dot_expand='true'

#
# Syntax-Highlighting
#

# This determines what highlighters will be used with the syntax-highlighting module.
# Documentation of the highlighters can be found here:
#   https://github.com/zsh-users/zsh-syntax-highlighting/blob/master/docs/highlighters.md
zhighlighters=(main brackets)

#
# SSH
#

# Load these ssh identities with the ssh module
#zssh_ids=(id_rsa)

#
# Pacman
#

# Set (optional) pacman front-end.
#zpacman_frontend='powerpill'

# Load any helper scripts as defined here
#zpacman_helper=(aur)

#################
# CORE SETTINGS #
#################

#
# Zim settings
#

# Select what modules you would like enabled.
# The second line of modules may depend on options set by modules in the first
# line. These dependencies are noted on the respective module's README.md.
zmodules=(directory environment git git-info history input utility custom \
          prompt completion syntax-highlighting history-substring-search )

###################
# MODULE SETTINGS #
###################

#
# Prompt
#

# Set your desired prompt here
zprompt_theme='steeef'

#
# Completion
#

# Set an optional host-specific filename for the completion cache file. If none
# is provided, the default '.zcompdump' is used.
#zcompdump_file=".zcompdump-${HOST}-${ZSH_VERSION}"

#
# Utility
#

# Uncomment to enable spelling correction prompt for commands. See:
# http://zsh.sourceforge.net/Doc/Release/Options.html#Input_002fOutput
#setopt CORRECT

# Set custom spelling correction prompt
#SPROMPT='zsh: correct %F{red}%R%f to %F{green}%r%f [nyae]? '

#
# Environment
#

# Set the string below to the desired terminal title format string.
# The terminal title is redrawn upon directory change, however, variables like
# ${PWD} are only evaluated once. Use prompt expansion strings for dynamic data:
#   http://zsh.sourceforge.net/Doc/Release/Prompt-Expansion.html#Simple-Prompt-Escapes
# The example below uses the following format: 'username@host:/current/directory'
ztermtitle='%n@%m:%~'

#
# Input
#

# Set to vi or emacs
zinput_mode='emacs'

# Uncomment to enable double-dot expansion. This appends '../' to your input for
# each '.' you type after an initial '..'
#zdouble_dot_expand='true'

#
# Syntax-Highlighting
#

# This determines what highlighters will be used with the syntax-highlighting module.
# Documentation of the highlighters can be found here:
#   https://github.com/zsh-users/zsh-syntax-highlighting/blob/master/docs/highlighters.md
# For (u)rxvt, termite and gnome-terminal users,
# removing the 'cursor' highlighter will fix the disappearing cursor problem
zhighlighters=(main brackets cursor)

#
# SSH
#

# Load these ssh identities with the ssh module
#zssh_ids=(id_rsa)

#
# Pacman
#

# Set (optional) pacman front-end.
#zpacman_frontend='powerpill'

# Load any helper scripts as defined here
#zpacman_helper=(aur)

#################
# CORE SETTINGS #
#################

#
# Zim settings
#

# Select what modules you would like enabled.
# The second line of modules may depend on options set by modules in the first line.
# These dependencies are noted on the respective module's README.md.
zmodules=(directory environment git history ssh input utility meta custom \
          syntax-highlighting history-substring-search spectrum prompt completion)

###################
# MODULE SETTINGS #
###################

#
# Prompt
#

# Set your desired prompt here
zprompt_theme='pure'

#
# Completion
#

# set an optional host-specific filename for the completion cache file
# if none is provided, the default '.zcompdump' is used.
#zcompdump_file=".zcompdump-${HOST}-${ZSH_VERSION}"

#
# Utility
#

# Uncomment to enable command correction prompts
# See: http://zsh.sourceforge.net/Doc/Release/Options.html#Input_002fOutput
#setopt CORRECT

#
# Environment
#

# Set the string below to the desired terminal title format string.
# The terminal title is redrawn upon directory change, however, variables like
# ${PWD} are only evaluated once. Use prompt expansion strings for dynamic data:
#   http://zsh.sourceforge.net/Doc/Release/Prompt-Expansion.html#Simple-Prompt-Escapes
# The example below uses the following format: 'username@host:/current/directory'
ztermtitle='%n@%m:%~'

#
# Input
#

# Uncomment to enable double-dot expansion.
# This appends '../' to your input for each '.' you type after an initial '..'
#zdouble_dot_expand='true'

#
# Syntax-Highlighting
#

# This determines what highlighters will be used with the syntax-highlighting module.
# Documentation of the highlighters can be found here:
#   https://github.com/zsh-users/zsh-syntax-highlighting/blob/master/docs/highlighters.md
# For (u)rxvt, termite and gnome-terminal users,
# removing the 'cursor' highlighter will fix the disappearing cursor problem
zhighlighters=(main brackets cursor)

#
# SSH
#

# Load these ssh identities with the ssh module
#zssh_ids=(id_rsa)

#
# Pacman
#

# Set (optional) pacman front-end.
#zpacman_frontend='powerpill'

# Load any helper scripts as defined here
#zpacman_helper=(aur)

@jitendravyas, for the second error, looks like there's a permission issue with your .zcompdump.zwc.old file. Not sure if the the first error is related to this too.

@jitendravyas

My current shell is Fish-shell

Your are aware this a plugin framework for ZSH right?


Nothing seen on Firefox 62.0

None of the samples work on Firefox 62.0.  However they do work on Chrome 69.0 and Safari 10.0.
Steps:

https://erik-larsen.github.io/
Try any of the samples under emscripten-sdl2-ogles2.

Specific to macOS Sierra, not reproducible on High Sierra.

I think it's because you need to call SDL_CreateWindowAndRenderer intead of SDL_CreateWindow. I don't know why, SDL_CreateWindow doesn't work for me.

Fixed at least since Firefox 108.0.2 👍


Rationalize text input widgets

After this patch, there are three major text input widgets:

EditableText. This widget is a low-level editing control that
interacts with the IME and displays a blinking cursor.

TextField. This widget is a Material Design text field, with all the
bells and whistles. It is highly configurable and can be reduced down
to a fairly simple control by setting its decoration property to
null.

TextFormField. This widget is a FormField that wraps a TextField.

This patch also replaces the InputValue data model for these widgets
with a Listenable TextEditingController, which is much more flexible.
Fixes #7031

This patch is stacked on https://github.com/flutter/flutter/pull/9074, so you probably want to just look at the second commit independently.
Also, I need to add a bunch of docs, but this patch should otherwise be ready for review.
/cc @Hixie

This looks great. LGTM.
If you add or edit docs in this patch and want that reviewed too, add them as a separate commit and I'll be happy to review those too.

I updated the docs, but I squashed the commit before seeing your note.  :(


UI elements don't work on Firefox 60.8.0 ESR x64

https://www.reddit.com/r/ffxiv/comments/d3s91t/universalis_crowdsourced_crossworld_market_board/f895ss0/?utm_source=share&utm_medium=web2x
Making the website functional on old browsers represents a significant amount of work for little gain, but fixing issues like these will improve experience for some number of users and potential users.

The website has been rewritten entirely; this is probably no longer applicable.


Add tooltip refill

We'll need to update the code samples to stretch, since we won't always have js.

Looks pretty 🆒 . We may want to add a dropdown menu later but that should be separate from a regular old tooltip.

Thanks! This could use a re-review, since I switched it to use the aria-label's content

Updated the description and example to focus on graphical elements.


Fatal error with Kirki and Bootstrap shell.

Warning: Illegal string offset 'body' in E:\Web\wp-content\plugins\kirki\includes\class-kirki-style.php on line 140
Fatal error: Cannot use string offset as an array in E:\Web\wp-content\plugins\kirki\includes\class-kirki-style.php on line 140

Fixed in https://github.com/aristath/kirki/commit/1bbd9db6fd399c40a82150f1eb0805dbfc22162d


chore(deps): update React peer dependency

Same as recent react-query PR, update React versions in peer dependency.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Cant use ng-class on accordion group since 1.33.3

You can no longer use ng-class on an accordion group since this commit.
https://github.com/angular-ui/bootstrap/commit/ead15e37
https://github.com/angular-ui/bootstrap/pull/4152
https://github.com/angular-ui/bootstrap/issues/3419

Saldy, I'm closing this due to the responses on here
https://github.com/angular-ui/bootstrap/issues/4172

Sadface.

  We're actually looking out for you and your app's best interests here.  Did you read the comments in this link?  Merging classes can yield very unpredictable behavior - this is one of the reasons replace: true can be so evil.

Temporary workaround:
Put your ng-classes in your plain "class" attribute:
Before:
ng-class="{'expanded': status.open}"

After:
class="{{status.open ? 'expanded':' '}}"

@leledev, this is not a workaround, temporary or otherwise.  CSS classes are not applied in the order they appear in your DOM element's class attribute, but rather, where they are defined in your CSS file(s), and thus, the order in which they are loaded.

@icfantv
Sorry, I wanted to post the 'workaround' in / for this issue: https://github.com/angular-ui/bootstrap/issues/4172 , but I pasted it here by mistake

It's a workaround for who wants to make things works without using ng-class, and it should work in most of the cases


unicode url not working correctly for the Navlink

When the Navlink is dealing with unicode urls it will not work correctly. My guess is it doesn't encode the url before sending out the ajax calls. So the API server or the backend will not see the correct url.
Currently I do
href = encodeURI(href);
inside the dispatchNavAction function. But I am not sure if this is a temp solution or is it correct. Could you guys take a look at it? Thanks.

Hmm, are you sure this is a router problem or does this have to do with fluxible-plugin-fetchr? Fluxible router does not do AJAX, but it does call history.pushState. Maybe the issue is there. I will see if I can reproduce an issue with routes have unicode.

not really sure.  But without the one line code href = encodeURI(href) I will see when I click on the Navlink the website is calling the backend API server with weird routes the API server cannot read.

I tried in my app and i have no issues with unicode. I suspect it's something in @ssmlee04 backend API not parsing unicode when receiving XMLHttpRequest calls.

Currently I kind of put all the links that involves Unicode characters with href = encodeURI(href);
I tried to make API calls to the backend API server using both Postman and links from Navlinks.
The test link that contains the unicode characters are:
https://localhost:8000/places/777咖啡-taiwan-taipeicity-556e76e00cfaa9fe78080b7a
those characters are Chinese btw.
the request sent using Postman looks like this on the backend. Response status is 200.

the request from Navlink doing href = encodeURI(href) also give the same result.
But if I don't do it the request would look like this

the response status is 500. Express even failed to parse the route correctly
Because when I paste the same route into Postman and from Navlink the behavior on the backend is different. This is why I think the problem isn't from the backend?


1.2.0: LV2 plugin fails on lv2lint (regression)

Hi! Upon building 1.2.0 for Arch Linux I ran lv2lint against the lv2 plugin. This now fails with many errors, such as those from the below excerpt:
  <http://sfztools.github.io/sfizz#cc006>
    [FAIL]  Parameter Label
              rdfs:label not found
              seeAlso: <http://lv2plug.in/ns/lv2core#Parameter>
    [FAIL]  Parameter Range
              rdfs:range not found
              seeAlso: <http://lv2plug.in/ns/lv2core#Parameter>
  <http://sfztools.github.io/sfizz#cc000>
    [FAIL]  Parameter Label
              rdfs:label not found
              seeAlso: <http://lv2plug.in/ns/lv2core#Parameter>
    [FAIL]  Parameter Range
              rdfs:range not found
              seeAlso: <http://lv2plug.in/ns/lv2core#Parameter>
  <http://sfztools.github.io/sfizz#cc001>
    [FAIL]  Parameter Label
              rdfs:label not found
              seeAlso: <http://lv2plug.in/ns/lv2core#Parameter>
    [FAIL]  Parameter Range
              rdfs:range not found
              seeAlso: <http://lv2plug.in/ns/lv2core#Parameter>
  <http://sfztools.github.io/sfizz#cc002>
    [FAIL]  Parameter Label
              rdfs:label not found
              seeAlso: <http://lv2plug.in/ns/lv2core#Parameter>
    [FAIL]  Parameter Range
              rdfs:range not found
              seeAlso: <http://lv2plug.in/ns/lv2core#Parameter>
  <http://sfztools.github.io/sfizz#ui>
    [WARN]  UI Instance Access
              usage of instance-access is highly discouraged
              seeAlso: <http://lv2plug.in/ns/ext/instance-access>

lv2lint log: sfizz-1.2.0-lv2lint.log
Full build log: sfizz-1.2.0-build.log

Hi, I've changed the manifest on develop, seems better now but my lv2lint version on Ubuntu acts a bit weird. I'll try on some other computer, if you have time to check too, thanks !

@paulfd thanks, https://github.com/sfztools/sfizz/pull/1114/commits/dac20eac20cbbcb14c6d25d1575b85d6dce7ee93 fixes the issue.
As there hasn't been a release in over a year, I guess it would be great to cut one! :)


PR: Improve message about new releases (Main Window)

This PR improves the message we show at startup about new releases. Its purpose is to simplify that message significantly, especially for people that will start using our new installers. I also changed the title of that dialog to make it clearer.

Before, using Anaconda:

Before, not using Anaconda:

After, using Anaconda:

After, not using Anaconda:

Fixes #12677

@juanis2112, I'd like to know your opinion about this one. What do you think about my suggested changes?


Single pin encoder

Hello,
Fist of all thankyiu for your incredible contributions to the open source community. I've made extensive use of your encoder and motor / motor2040 code. Unfortunately the encoders / motors I'm using are very poor quality and are causing a lot of erratic behavior.
I'm intended to switch the motors / encoders to a unit with a built in driver which provides a single encoder output. I.e the the "motor" has 5 pins, power, ground, pwm input, direction input and encoder output.
My question is if the encoder library supports this?
Thanks!
Ben

Hi Ben,
Glad to hear you have found our motor / motor2040 code of value.
I am not familiar with encoders that would output just a single PWM signal. It must be an absolute angle, as you would not be able to determine the direction of travel otherwise. Do you have a datasheet you can link so I can confirm?
In any case, our Encoder library does not support this type of encoder, as it is specifically for those that output a quadrature signal. The RP2040 seems to support reading of single PWM signals, but this is not exposed nicely in Micropython. The best I have found with 10 seconds of googling is this repo: https://github.com/phoreglad/pico-MP-modules/blob/main/PWMCounter/PWMCounter.py
As a side note, you mention the motor is controlled by a PWM and a direction input. If you have not already seen, our Motor class supports this mode (but is not the default). https://github.com/pimoroni/pimoroni-pico/tree/main/micropython/modules/motor#driver-type
Hope that helps

@ZodiusInfuser Hey, sorry I don't think I was clear in what i meant. The encoder feedback is just a single channel, it's not PWM. This is an example of a similar motor (and one I'm using for development) -
https://cdn.robotshop.com/rbm/a00a7635-653b-4220-aac9-b0c23c5c5e2c/d/d86e25ee-9cba-4fd3-93ee-8538436c9ad4/3f672a79_25SG-2418BL-12V.pdf
https://www.robotshop.com/products/24mm-brushless-dc-gear-motor-12v-100rpm

@Chick92 Thanks for those links.
PWM was my guess of what the output could have been given it was on a single wire (though it could have been UART or analog too). Looking at that datasheet I was not far off, as that encoder outputs a PFM signal, which is a pulse signal with the frequency proportional to the speed of the motor (if I am inferring the meaning of FG correctly)
Our library does not support this kind of output, but that PWMCounter.py I linked to before may be able to decode it.
Also, our enviro grow product has moisture sensors that produce a similar output, which we decode using this function:
https://github.com/pimoroni/enviro/blob/b5bff82f9001409c9683be43b9f2114e7da8b430/enviro/boards/grow.py#L27-L64
Hope that helps.

That's really helpful, thanks very much!


Binary files size too big ?

There is commit 1d239bc already, but latest release 1.2.0 binaries are still around 3.4 MB. I want to try this on mips router.
Previous discussion #68
edit: I see the files are already packed and I understand that go binaries are fat.

就这么大，go编译的文件一般都不小


Fix init wait on tls

Fixes #4051
The failure was due to the hostname getting passed as <IP_ADDRESS> which did not match the hostname of the certificate.
Changes

Added --tiller-tls-hostname flag for init command.
Fixed a debug statement I stumbled upon (additional commit). I can remove the commit on debug statement from this PR if it should not be included.

 Can you validate that the issue was solved with that PR and let us know if there was anything we still need to address?

@thomastaylor312 Its still needed. The flags added addFlagsTLS is not for the init command here. And I think the reason is the difference in the flag names. Normally, addFlagsTLS adds flags as --tls-key as in here, however on init command it is --tiller-tls-key as in here (Its prefixed with tiller).
The above mentioned PR only adds tls-hostname to all the subcommands after a tiller has been initialized but not to the init command itself and that was causing the issue #4051 . This PR fixes that.

Mind rebasing this PR? Once that's done we can merge this. Thanks!

@bacongobbler Done


FindHiredis.cmake has a case sensitivity issue

Describe the bug
In cmake_modules/FindHiredis.cmake#L28, drogon framework try to find package hiredis with wrong case Hiredis, which would cause a dev warning in cmake process.
To Reproduce
Steps to reproduce the behavior:

Create a CMake project with a vcpkg.json in root
Add drogon with redis feature in vcpkg.json, example:

"dependencies": [
    {
        "name": "drogon",
        "default-features": false,
        "features": [
            "redis"
        ]
    }
]

In your CMakeLists.txt, add these lines before project() :

set(CMAKE_TOOLCHAIN_FILE "${CMAKE_CURRENT_SOURCE_DIR}/vcpkg/scripts/buildsystems/vcpkg.cmake" CACHE STRING "Vcpkg toolchain file")
if (WIN32)
    message(STATUS "Platform is Win32")
    set(VCPKG_TARGET_TRIPLET "x64-windows" CACHE STRING "Vcpkg target triplet" FORCE)
elseif (UNIX)
    message(STATUS "Platform is Unix")
    set(VCPKG_TARGET_TRIPLET "x64-linux" CACHE STRING "Vcpkg target triplet" FORCE)
endif ()

In your CMakeLists.txt, add these lines after add_executable() or add_library() :

find_package(Drogon CONFIG REQUIRED)
target_link_libraries(${PROJECT_NAME} PRIVATE Drogon::Drogon)

Run CMake command in your build directory, then you can see vcpkg is automaticly downloads and installs packages.

When proceed to find_package(Drogon CONFIG REQUIRED) command, it raises a de warning:

CMake Warning (dev) at C:/dev/Jetbrains/Toolbox/apps/CLion/ch-0/212.5284.51/bin/cmake/win/share/cmake-3.20/Modules/FindPackageHandleStandardArgs.cmake:438 (message):
  The package name passed to `find_package_handle_standard_args` (Hiredis)
  does not match the name of the calling package (hiredis).  This can lead to
  problems in calling code that expects `find_package` result variables
  (e.g., `_FOUND`) to follow a certain pattern.
Call Stack (most recent call first):
  cmake-build-debug-msvc2019/vcpkg_installed/x64-windows/share/drogon/FindHiredis.cmake:27 (find_package_handle_standard_args)
  vcpkg/scripts/buildsystems/vcpkg.cmake:786 (_find_package)
  C:/dev/Jetbrains/Toolbox/apps/CLion/ch-0/212.5284.51/bin/cmake/win/share/cmake-3.20/Modules/CMakeFindDependencyMacro.cmake:47 (find_package)
  cmake-build-debug-msvc2019/vcpkg_installed/x64-windows/share/cmake/redis++/redis++-config.cmake:31 (find_dependency)
  vcpkg/scripts/buildsystems/vcpkg.cmake:786 (_find_package)
  CMakeLists.txt:35 (find_package)
This warning is for project developers.  Use -Wno-dev to suppress it.

Expected behavior
No dev warning should produced during cmake process.
Screenshots

Desktop (please complete the following information):

OS: Windows 11 Pro
Compiler: Visual Studio 2019

Weird. For me on Linux, changing the case as suggested caused the warning
❯ git diff
diff --git a/cmake_modules/FindHiredis.cmake b/cmake_modules/FindHiredis.cmake
index 03e7fc50..20623db5 100644
--- a/cmake_modules/FindHiredis.cmake
+++ b/cmake_modules/FindHiredis.cmake
@@ -25,7 +25,7 @@ else ()

     include(FindPackageHandleStandardArgs)
     find_package_handle_standard_args(
-            Hiredis DEFAULT_MSG HIREDIS_LIBRARY HIREDIS_INCLUDE_DIR)
+            hiredis DEFAULT_MSG HIREDIS_LIBRARY HIREDIS_INCLUDE_DIR)

     mark_as_advanced(HIREDIS_LIBRARY HIREDIS_INCLUDE_DIR)
 endif ()
@@ -37,4 +37,4 @@ if(Hiredis_FOUND)
             "${HIREDIS_INCLUDE_DIRS}"
             INTERFACE_LINK_LIBRARIES
             "${HIREDIS_LIBRARIES}")
-endif(Hiredis_FOUND)
\ No newline at end of file
+endif(Hiredis_FOUND)

❯ cmake ..
...
CMake Warning (dev) at /usr/share/cmake-3.21/Modules/FindPackageHandleStandardArgs.cmake:438 (message):
  The package name passed to `find_package_handle_standard_args` (hiredis)
  does not match the name of the calling package (Hiredis).  This can lead to
  problems in calling code that expects `find_package` result variables
  (e.g., `_FOUND`) to follow a certain pattern.
Call Stack (most recent call first):
  cmake_modules/FindHiredis.cmake:27 (find_package_handle_standard_args)
  CMakeLists.txt:388 (find_package)
This warning is for project developers.  Use -Wno-dev to suppress it.


Update backup for analyseData.json

In this PR due to today's technical issue, I have made a change on the server deploy, updating the backup file used to start the project in localhost without internet connection.
The file had been added to the .gitignore file because for testing is not necessary to keep the file updated, just is important for the deployed site.
The file name was changed because IDK why, with the last name .gitignore ignored ignore the file.

@brifemu When I execute npm run build in this branch, it creates an untracked file src/data/analyse_data_no_internet.json
so I think something needs to change in this PR.
npm run build should not cause any changes that git status would report about.

You are right. I forgot to change the path, sorry. Already fixed.

@brifemu
There is no problem with git status now, after your latest change. 🙂


Remove "edit" in advanced options for "reject transactions"

Web app, v3.2.0
In reject transactions every input in the advanced option is not editable, So having the edit there may confuse the user of why he cannot edit the parameters.
I propose remove the "edit" link only for the transaction rejections

Why wouldn't it be editable?

Eth tx params should certainly be editable. It works for me.
Closing this, feel free to reopen in case I missed anything.


Run app.py Error：cannot import name 'build_lama_model' from 'lama_inpaint'

DESC
hi, i am so surprised by your latest feature, the local web ui, so i quickly run the app.py for a test, however i get an error like this: cannot import name 'build_lama_model' from 'lama_inpaint'
please help me solve the problem, or give me more information about this, thanks!
COMMAND
python app.py \
>       --lama_config  /home/lcc/github.com/Inpaint-Anything/pretrained_models/big-lama/config.yaml \
>       --lama_ckpt  /home/lcc/github.com/Inpaint-Anything/pretrained_models/big-lama/models/best.ckpt \
>       --sam_ckpt /home/lcc/github.com/Inpaint-Anything/pretrained_models/sam_vit_h_4b8939.pth

ERROR
Detectron v2 is not installed
Traceback (most recent call last):
  File "/home/lcc/github.com/Inpaint-Anything/app/app.py", line 13, in <module>
    from lama_inpaint import inpaint_img_with_lama, build_lama_model, inpaint_img_with_builded_lama
ImportError: cannot import name 'build_lama_model' from 'lama_inpaint' (/home/lcc/github.com/Inpaint-Anything/lama_inpaint.py)

MY ENV
centos7
GCC9.5
python3.9
cuda 11.3
torch 1.12.1

Hi, we updated the code and instruction. Please use the latest version. It should work. I just tested it.

I have same issue with latest version

I have same issue with latest version

Did you follow the latest instruction to run the demo web?

I found my mistake, I did't enter the app folder
python ./app/app.py
use this code not work,need to execute in app folder

@distort5871 In your context, kindly try this:
python app.py \
       --lama_config ./lama/configs/prediction/default.yaml \
       --lama_ckpt  /home/lcc/github.com/Inpaint-Anything/pretrained_models/big-lama \
       --sam_ckpt /home/lcc/github.com/Inpaint-Anything/pretrained_models/sam_vit_h_4b8939.pth

thanks so much, it works for me.


fix typo in nftables default ruleset

Description
Fixes typo in FORWARD chain of nftables default ruleset.
Issues Resolved
Currently no open issue.
Check List

[X] A summary of changes made is included in the CHANGELOG under ## Unreleased
[X] New functionality includes testing.
[X] New functionality has been documented in the README if applicable.

Released as: 6.2.18


any way to use this with an encrypted channel?

Hello
Is there any way using this with an encrypted channel?
Best regards

Hi @horstepipe,
Unfortunately there is not yet a way to use encrypted channels with the webhook gateway. I would like to further investigate supporting this, because I think it is a very valuable feature to have, but that will take some more time.

hi,
thanks for the answer!
are you aware of that project?
https://github.com/immanuelfodor/matrix-encrypted-webhooks
maybe you could grab some ideas/code from there.

Maybe this can be archived with low development efford by adding support for pantalaimon?
Looks like this slack appservice is doing that: https://matrix-appservice-slack.readthedocs.io/en/stable/bridge-encryption/
Looked at the code and it seems to me that they only added some conditionals to use pantalaimon url in some cases and the actual matrix server url in other cases.
See e.g. https://github.com/matrix-org/matrix-appservice-slack/blob/3b17bf851ea910d91d2a5bfe212d2e6a1f744077/src/Main.ts#L480
I tried setting pantalaimon as homeserver_url but that resulted in auth errors (M_UNKNOWN_TOKEN when accessing http://pantaaimon:8008/_matrix/client/r0/profile/..../avatar_url).

hi @jthomaschewski, thanks for the suggestion! I can't quite manage to get pantalaimon configured locally yet (I'm having some trouble with pactl), but I've pushed a small update that might allow the gateway to communicate with it.
If you'd like to test this, fetch the latest development release (docker pull geluk/matrix-webhook-gateway:dev or just clone the repository) and set pantalaimon_url in your config file (see here: https://github.com/geluk/matrix-webhook-gateway/blob/master/templates/gateway-config.yaml#L30).

Hi @geluk thanks for the quick response and development release!
I tested it but unfortunately it didn't work. Same behavior as with :latest Tag:

webhook-gateway start with pantalaimon:
webhook-gateway_1  |     href: 'http://pantalaimon:8008/_matrix/client/r0/profile/%40bot_webhook%3Amyserver.tld/avatar_url',
webhook-gateway_1  |     req: [Circular *11],
webhook-gateway_1  |     ntick: true,
webhook-gateway_1  |     timeoutTimer: null,
webhook-gateway_1  |     response: [Circular *11],
webhook-gateway_1  |     originalHost: 'pantalaimon:8008',
webhook-gateway_1  |     originalHostHeaderName: 'host',
webhook-gateway_1  |     responseContent: [Circular *11],
webhook-gateway_1  |     _destdata: true,
webhook-gateway_1  |     _ended: true,
webhook-gateway_1  |     _callbackCalled: true
webhook-gateway_1  |   },
webhook-gateway_1  |   toJSON: [Function: responseToJSON],
webhook-gateway_1  |   caseless: Caseless {
webhook-gateway_1  |     dict: {
webhook-gateway_1  |       'access-control-allow-headers': 'Origin, X-Requested-With, Content-Type, Accept, Authorization',
webhook-gateway_1  |       'access-control-allow-methods': 'GET, POST, PUT, DELETE, OPTIONS',
webhook-gateway_1  |       'access-control-allow-origin': '*',
webhook-gateway_1  |       'content-type': 'application/json; charset=utf-8',
webhook-gateway_1  |       'content-length': '69',
webhook-gateway_1  |       date: 'Thu, 03 Feb 2022 23:48:04 GMT',
webhook-gateway_1  |       server: 'Python/3.8 aiohttp/3.7.4.post0',
webhook-gateway_1  |       connection: 'close'
webhook-gateway_1  |     }
webhook-gateway_1  |   },
webhook-gateway_1  |   body: {
webhook-gateway_1  |     errcode: 'M_UNKNOWN_TOKEN',
webhook-gateway_1  |     error: 'Unrecognised access token.'
webhook-gateway_1  |   }
webhook-gateway_1  | }
webhook-gateway_1  | 2022-02-03 23:48:04.492  FATAL [webhook-srv] Could not start webhook-gateway

I have synapse+pantalaimon+webhook-gateway within the same docker network and pantalaimon is working fine.
I use it with matterbot which authenticated by user/password - it's not an appservice though
Pantalaimon does seem to have issues/quirks when used with appservices :(
see e.g. https://github.com/matrix-org/pantalaimon/issues/107 and https://github.com/matrix-org/pantalaimon/pull/84 (possible fix)
It seems to try to "sync" on startup which doesn't work for appservices due too missing access token -
As I have no knowledge of the Matrix protocol, I have no clue if its possible to workaround it somehow - or whatever matrix-appservice-slack is doing to make it work...
Btw here is my pantalaimon docker config, maybe it helps you if you wanna try:

docker-compose.yml for pantalaimon:
  pantalaimon:
    image: matrixdotorg/pantalaimon:v0.10.3
    restart: unless-stopped
    volumes:
      - pantalaimon-data:/data
      - ./pantalaimon.conf:/data/pantalaimon.conf:ro

pantalaimon.conf:
[Default]
LogLevel = Warning
SSL = True

[local-matrix]
Homeserver = https://my.homeserver.tld
ListenAddress = <IP_ADDRESS>
ListenPort = 8008
SSL = False
UseKeyring = False
IgnoreVerification = True

With this config all docker services within the same network as pantalaimon can configure http://pantalaimon:8008 as their homeserver_url and will reach https://my.homeserver.tld with encryption support

Thanks for testing! I looked into it a bit further and dug around in the source code of the Slack bridge and pantalaimon, but I couldn't find any more details on how this is supposed to work with appservices. I think we may just need to have a bit more patience while appservice support in pantalaimon is implemented. It does seem like using pantalaimon is the intended way of enabling encryption in the Matrix appservice library, which is good to know.
There might be a way to get it to work yet, but finding that out could take a fair amount of digging around, which I'd rather not do. I do appreciate the suggestions, so if someone knows what else is missing here, let me know.
I'll leave the issue open for now, if there are new developments I'll take another look.


Copy pasting is losing the formating

I don't know if this behavior was ever present in Quill. I think it was if I correctly recall some comments. But for sure I want it in the Visual Editor version. Including copying of custom attributes. I'm using custom attributes to inject various DB ids in the text at precise positions. So I want to preserve them on copy paste.
Join on discord to get advice and help or follow us on YouTube Visual Coding to learn more about the architecture of Visual Editor and other Flutter apps.

When implementing the copy paste feature I'll have a look to check how to preserve rich text from other editors as requested in this PR. Apparently Quill JS has an entire module dedicated to converting HTML styling to delta. I'll have a look to see what we can use as inspiration.

I can confirm that this functionality was working in the pure JavaScript Quill Editor and I also think this feature is quite important. Otherwise it can be very frustrating for users when the formatting gets lost all the time.

Overall, no need to change any existing source code to build this feature, just build on top of existing source code of quill.
4 steps here,

quill use the flutter‘s native selection popup menu under the hood, which has different action callbacks, I have some sample code using this feature with quill for a different purpose, but the idea is the same, you can intervene what data to pass down on onPasteText callback, so no need to change source code of quill for this
you need to detect the format for xml/html/markdown/wordFormat/delta/plainText on your onPasteAction call back, so again no need to change quill's existing source code,
transform all these different format into delta and insert the delta descriptions one by one using the controller. I have writen some code converting between html/markdown/delta because my workflow involves displaying quill edited text in both html and quill correctly,
use the controller to generate the final correct delta output, and update your valueNotifier which holds the json String

Over here step 4 is for eliminating the need of using setState, many people here are using setState with quill after controller update, I do not recommend its usage here,
begin by looking at QuillEditor's argument
textSelectionControls: MyNoteTextSelectionControls(
focus: focus,
generateNote: generateNote,// this is my own function, you can pass other functional arguments to fullfill your own purpose.
),
for which class MyNoteTextSelectionControls extends MaterialTextSelectionControls
you also need to override onCopy, onCut if you are to put json instead of text onto the clipboard when copying/cutting from quill editors' content.
when you are up to implementing this, give me a shout, I can give you some half-baked code to begin with.

the quill-delta to markdown translation package on pub.dev is no longer maintained and due to null safety migration, no longer works, I have a working version of this code against flutter3.0 somewhere in my repo which I have added a few more features and heavily refactored, if someone is willing to open and manage a new repo on github, I can pass on my modified code to him (not fully refactored, just good enough to work atm), so you don't have to start from scratch.

you to detect the format for xml/html/markdown/wordFormat/delta/plainText - Any particular recommendation what technique to use for this purpose?
Again, this could be a plugin. You download barebone quill/VE, then you download the Copy/Paste extensions and they do the plugin registration behind the scenes and that enables the copy paste from various doc types. In this way you can keep the core lightweight and help to not so skilled users to enable these features. I doubt there are many users that have the skills required to setup such complex setups. Most users want to enable rich text and that's all they care about. If they can get copy paste from other doc formats that's even better. But them spending time to setup the flow, highly unlikely for most devs. So a middle ground solution would be best.


use setTimeout instead of setImmediate

setImmediate is  a non standard feature and the behavior is non stable. In the src/utils.js I have replaced the setImmediate function call with an this alternative
setTimeout(async () => {
 await push()
}, 0)

Current coverage is 81.81% (diff: 100%)

Merging #39 into master will not change coverage

@@             master        #39   diff @@
==========================================
  Files             3          3
  Lines            55         55
  Methods           4          4
  Messages          0          0
  Branches         12         12
==========================================
  Hits             45         45
  Misses           10         10
  Partials          0          0

Powered by Codecov. Last update 09442c1...e6b06c3

I also would like to know if this locks functionality will work when we are using multiple processes. What are the alternatives in case we have multiple processes of one micro function running. Can we instead implement an increment functionality in the DB. Db such as redis provides us with this increment function which is atomic.

We've thought about moving the atomicity out to the database adapters, see #14 and related issues. Not sure yet if we'll go down that route, /cc @sean-roberts

I don't think changing this is necessary. This is for sure standard in node. Maybe not in browsers according to mdn, but it's been in node quite a long time. :man_shrugging:  As far as the locking convo, let's move that to a discussion ticket

Want to give your thoughts on https://github.com/micro-analytics/micro-analytics/issues/40 ?

Closing this since we'll probably move forward with #40!
Thanks for taking the time to submit a PR and kicking off this discussion @abdulhannanali!


Depth chart

@tiltom @rick23p I believe you have permissions to merge feature branches to other feature branches?

@tiltom @rick23p I believe you have permissions to merge feature branches to other feature branches?

Yes we do :))


no possible to build shared library

Hi !
I tried to build boost as shared library, but I get this kind of errors :
cannot find crtbegin_so.o
cannot find -lrt
Does anybody has a solution ?
I built static libraries using boost 1_49 and ndk8b without any problems.
Gabriel

Passing ./b2 the
target-os=android
switch will prevent the -lrt flag from being passed.


geneus: 2.2.1-0 in 'jade/distribution.yaml' [bloom]

Increasing version of package(s) in repository geneus to 2.2.1-0:

upstream repository: https://github.com/jsk-ros-pkg/geneus
release repository: https://github.com/tork-a/geneus-release.git
distro file: jade/distribution.yaml
bloom version: 0.5.19
previous version for package: 2.2.0-0

geneus
* [geneus_main.py] fix when pkg_map does not have geneus
* [geneus_main.py] Fix unexpected function resposes caused by python gabage collection algorightm
* [geneus_main.py] fix typo for warning
* [geneus_main.py] Yellow color for warnings
* Contributors: Kei Okada, Kentaro Wada

@k-okada That was a fast first release after being added as a default message generator :wink: I hope no further patching is necessary for some time.

yes, I understand the situation and hope that we do not find any further
patching, anyway thank you for adding us to the default generator!
--
◉ Kei Okada
On Sat, Apr 25, 2015 at 2:11 AM, Dirk Thomas<EMAIL_ADDRESS>wrote:

@k-okada https://github.com/k-okada That was a fast first release after
being added as a default message generator [image: :wink:] I hope no
further patching is necessary for some time.
—
Reply to this email directly or view it on GitHub
https://github.com/ros/rosdistro/pull/8147#issuecomment-95994662.


[Enhancement] Move cronjob ui to the admin dashboard

The Cronjob UI being in setup makes it impossible to see without leaving /setup exposed on nginx servers.
It would be helpful for the UI to be visible in the admin dashboard to allow in-deployment monitoring of the cron status

+1

+1 But there would need to be some improvements on the admin dashboard for large installations
Currently im unable to load the Corporations Management part, looks like "failed login attempts" still generate a config file for that corporation

@november781 You can expose /setup on nginx and protect it by a login. Check:  pathfinder.conf. That´s the way I do it.
  # Protct /setup with password
  location /setup {
   auth_basic "Setup Login";
   auth_basic_user_file /etc/nginx/.setup_pass;
   try_files $uri $uri/ /index.php?$query_string;
  }

http://nginx.org/en/docs/http/ngx_http_auth_basic_module.html
@Meganitrospeed I´ll do some testing before the next release and check the Corp. Management section for any errors.

@november781 You can expose /setup on nginx and protect it by a login. Check:  pathfinder.conf. That´s the way I do it.
  # Protct /setup with password
  location /setup {
   auth_basic "Setup Login";
   auth_basic_user_file /etc/nginx/.setup_pass;
   try_files $uri $uri/ /index.php?$query_string;
  }

http://nginx.org/en/docs/http/ngx_http_auth_basic_module.html

Doing some messing around with this again: I'm using proxypass and a docker container and it is not having fun with using auth_baisc at all


✅ Refactor OpenAPI tests, prepare for Pydantic v2

✅ Refactor OpenAPI tests, prepare for Pydantic v2
During the migration, it will be very useful to check the tests with insert_assert from https://github.com/samuelcolvin/python-devtools
This prepares the code for that.

Before this change, there was normally a variable with openapi_schema, containing all the OpenAPI schema for each app tested. Then that was checked in a test for that.
But this doesn't allow easily inlining the result with insert_assert. By moving that schema to inside of the test, I'll be able to explore with insert_assert and verify in the diff what is wrong (later during the migration to Pydantic v2).
This needed to be done here, before that, because it only moves code/data around. But the diff is gigantic, so it shouldn't be mixed with code changing actual logic.
I also wanted to do it, to be able to keep using insert_assert, but that needed many consecutive hours of work, and a reason to go through all that work. But now this is a good reason to do it. :nerd_face:

lol
On Mon, May 8, 2023, 4:37 PM github-actions[bot] @.***>
wrote:

📝 Docs preview for commit 8ff855c
https://github.com/tiangolo/fastapi/commit/8ff855cbe5820ab7e335ed3986fc9b2221b7fc89
at: https://64595de4b0b5812debbcfff4--fastapi.netlify.app
—
Reply to this email directly, view it on GitHub
https://github.com/tiangolo/fastapi/pull/9503#issuecomment-1539019542,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7VCCCSJK6WYYKHNTK7CDR3XFFRRVANCNFSM6AAAAAAX2NKQXY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Rich-text editor improvements

This PR adds few improvements to the rich-text editor that I found useful when working with our new Pages app. It introduces few additional formatting options:

converting image links to <img> tags
text alignment options
clear formatting

Backend sanitizer settings have been moved to settings.py.
Screen below shows how the editor looks like after these changes, including a charming example of a page that may be created using it.

Pull Request Checklist
(Please keep this section. It will make maintainer's life easier.)

[ ] Privileged views and APIs are guarded by proper permission checks.
[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[x] Python code quality checks pass: pycodestyle, pydocstyle, pylint.
[x] JavaScript code quality checks pass: eslint.

Editor does not work fully on Firefox - alignment is not being applied.
Also maybe I am doing something wrong, but selecting text and choosing image picture removes text and does nothing. We should include some instructions how to embed an image.

@patrys Some of the tags that I included there are defaults from Bleach. They're generally adjusted to match with the tags and attributes that the rich-text editor produces.
@akjanik There is indeed a bug with alignment in Firefox, I'll check that out. Images are working fine for me, though.

Could we add some padding to paragraphs in editor's textarea? Right now those look like linebreaks and give false impression of text being cluttered.