SLSGetWindowBounds gives stale readings

I was doing some improvements of how mff works for my own convenience (#808 plus some more cases) and noticed that sometimes cursor wouldn't follow the window after a swap. Turned out that early exit in window_manager_center_mouse when cursor is already inside window bounds sometimes triggers incorrectly due to CGRect window_frame(struct window *) returning stale data.
I've added some logging statements to illustrate the issue. What I did is swapping kitty window with its right neighbor and back.
==> /usr/local/var/log/yabai/yabai.out.log <==
EVENT_HANDLER_DAEMON_MESSAGE: window --swap east

==> /usr/local/var/log/yabai/yabai.err.log <==
=== ACTING WINDOW: vim .config/yabai/yabairc 557
=== TARGET WINDOW: tail -f /usr/local/var/log/yabai/yabai.* 610
=== BEFORE SWAP: ax_frame is (   3,   25) <-> ( 959, 1077)
=== BEFORE SWAP:    frame is (   3,   25) <-> ( 959, 1077)
===  AFTER SWAP: ax_frame is ( 961,   25) <-> (1917, 1077)
===  AFTER SWAP:    frame is ( 961,   25) <-> (1917, 1077)
=== REFOCUS WINDOW: vim .config/yabai/yabairc 557

==> /usr/local/var/log/yabai/yabai.out.log <==
EVENT_HANDLER_WINDOW_MOVED: kitty 610
EVENT_HANDLER_WINDOW_MOVED: kitty 557

-- here goes swap back --

EVENT_HANDLER_DAEMON_MESSAGE: window --swap west

==> /usr/local/var/log/yabai/yabai.err.log <==
=== ACTING WINDOW: vim .config/yabai/yabairc 557
=== TARGET WINDOW: tail -f /usr/local/var/log/yabai/yabai.* 610
=== BEFORE SWAP: ax_frame is ( 961,   25) <-> (1917, 1077)
=== BEFORE SWAP:    frame is ( 961,   25) <-> (1917, 1077)
===  AFTER SWAP: ax_frame is (   3,   25) <-> ( 959, 1077)
===  AFTER SWAP:    frame is ( 961,   25) <-> (1917, 1077)
=== REFOCUS WINDOW: vim .config/yabai/yabairc 557

==> /usr/local/var/log/yabai/yabai.out.log <==
EVENT_HANDLER_WINDOW_MOVED: kitty 610
EVENT_HANDLER_WINDOW_MOVED: kitty 557

As you can see, SLSGetWindowBounds is sometimes stuck in the past, even after AX api returned correct values.
Changing window_manager_center_mouse to use window_ax_frame fixes the issue.
So the question is what are the downsides of using Accessibility APIs vs SkyLight?
I'm on rMBP + external display; Mojave (10.14.6); SIP fully enabled, not using SA. I've used this patch @ 8777db4 to produce the logs.

SLSGetWindowBounds is no longer a thing in the new version for exactly that reason. The primary motivation before is that the SLS version was more responsive, as it does not need to take a global mutex like the accessibility API does.
The new version of yabai just caches the window frame instead.

OK, I see. Thank you for clarification.


Setting latencyHint breaks Tone.Sequence

await Tone.start()

// comment this line out to fix the sequence
Tone.setContext(new Tone.Context({ latencyHint: 'playback' }))

const seq = new Tone.Sequence((time, note) => {
  console.log(`callback(${time}, ${note})`)
}, [0, 1, 2, 4, 5, 6, 7], '8n').start()

Tone.Transport.start()

i just added a getter for the global singletons like Transport. Since Tone.Transport is tied to the context when you get the reference to Tone.Transport (or even Tone.start()) it no longer refers to the new context that you just set. and so the new context is never started.
The updated way to make this work should be:
// comment this line out to fix the sequence
Tone.setContext(new Tone.Context({ latencyHint: 'playback' }))

// only invoke start after the new context is set to make sure that you're starting the new context
await Tone.start()

const seq = new Tone.Sequence((time, note) => {
  console.log(`callback(${time}, ${note})`)
}, [0, 1, 2, 4, 5, 6, 7], '8n').start()

// get the global Transport singleton
Tone.getTransport.start()

Should be fixed, going to close.


Wiki contributions

Hey,
Is there any chance that I (or anybody) can have write access to the Wiki? It's currently restricted to collaborators.
AFAIK, there's currently no way to contribute improvements to the documentation (I saw quite a few typos), which is less than perfect for an open source project.

Unfortunately we can't open it up to general editing.  There isn't a way to
review changes before they are published so the pages would be vulnerable
to spam, phishing, and other random mischief.
Not the best solution but if you open a github issue with the changes to be
made a contributor will make the updates.
On Tue, Apr 19, 2016 at 3:24 AM, Mathias Lafeldt<EMAIL_ADDRESS>wrote:

Hey,
Is there any chance that I (or anybody) can have write access to the Wiki?
It's currently restricted to collaborators.
AFAIK, there's currently no way to contribute improvements to the
documentation (I saw quite a few typos), which is less than perfect for an
open source project.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/Netflix/SimianArmy/issues/252

You know this isn't going to happen. Most people (including me) won't make the effort to diff the Wiki and paste the diff into an issue so that someone else can apply it.
You could at least add individual people as collaborators (based on GitHub history etc.) to avoid spam.
PS: We wouldn't have this discussion if the documentation were a part of the repo itself.


Westeros tutorials release 2 0

Improving the Westeros tutorials for MESSAGE_ix 2.0 release.
The changes in this PR adress all the points raised in issue #245 (link to specific post).

thanks @jkikstra for your improvements! I slightly edited some small formatting things.
Btw, now I am getting an error with the Plots function from the tools python package. @khaeru I wonder why... I run these tutorials some weeks ago and I was not getting this error (I did NOT update my Python 3.7.3 since then). So, basically I cannot see the plots anymore...

Now I've talked with @behman-zakeri in the bus and I think I can solve it

Okay @francescolovat your changes look good to me. Please let me know if you can run this and then if successful we can merge this in.

Hi @jkikstra,
the comments you've added in the last to commits seem great to me! good level of detail! thank you again 😃
I solved the problem with the plotting, it has taken me way too much time: The issue I was having when plotting was due to 16 recent commits in ixmp I did not merge to my master branch and I did non re-installed from source both message_ix and ixmp packages.
In other words, in my opinion we are ready to merge this PR.


seems like Kapasitor can't run on windows while using path to a config file

I'm trying to run kapacitor on widows like this:
C:\kapacitor-1.5.1-1>C:\kapacitor-1.5.1-1\kapacitord.exe -config C:\kapacitor-1.
5.1-1\kapacitor.conf
I get an error:
2018/08/19 13:51:56 Using configuration at: C:\kapacitor-1.5.1-1\kapacitor.conf
run: create server: invalid configuration: dir must be an absolute path. To gene
rate a valid configuration file run kapacitord config > kapacitor.generated.con f.
I've tried all kind of option with "" under different path etc.
This kinda blocks me from running Kapcitor with an influx node that has authentication enabled.

The issue is not that the conf file path is incorrect. I also had this problem and it also confused me. The error you are seeing is actually due to the paths defined inside .conf file, not the file path you are proving from the terminal. Check the paths and make sure they are not in the form of /var/x/y/...
What you can do is to generate the config file as you did and compare the paths created inside the generated config file to the paths in your config file. Once you correct all the paths to proper Windows directories in your config file, try to run kapacitor again and the path error should go away.


Patterns in trait methods with bodies

Describe the bug
Function signatures in traits with bodies should be allowed to use patterns:
trait Trait {
    fn f([x, y]: [f64; 2]) {
        println!("{} {}", x, y);
    }
}

Running rustfmt on the code above produces:
error[E0642]: patterns aren't allowed in methods without bodies
 --> /home/egor/proj/rust-proto/src/rotate.rs:2:10
  |
2 |     fn f([x, y]: [f64; 2]) {
  |          ^^^^^^
  |
help: give this argument a name or use an underscore to ignore it
  |
2 |     fn f(_: [f64; 2]) {
  |          ^

To Reproduce
Format the above code snippet using rustfmt or rustfmt +nightly.
Expected behavior
No errors
Meta

rustfmt version: 1.4.15-stable (530eadf 2020-06-02) and 1.4.18-nightly (c1e9b7b 2020-06-13)
From where did you install rustfmt?: rustup
How do you run rustfmt: rustfmt

This is due to 1.4.15/18 using an older version of the rustc parser than the version that added support for patterns in this context. On master, the version of the parser is up to date (I think scheduled for 1.4.19 or 2.0 release). @calebcartwright has more context

Correct. rustfmt's broken on nightly ATM, but once available again it will contain the updated versions of the rustc internals that will support the syntax.
Closing as there's no additional action to be taken within rustfmt.

I am seeing this error with the current stage 0 rustfmt in the rustc repository:
$ ./build/x86_64-unknown-linux-gnu/stage0/bin/rustfmt compiler/rustc_middle/src/mir/interpret/pointer.rs
error[E0642]: patterns aren't allowed in methods without bodies
  --> /home/r/src/rust/rustc/compiler/rustc_middle/src/mir/interpret/pointer.rs:55:31
   |
55 |     fn truncate_to_ptr(&self, (val, over): (u64, bool)) -> (u64, bool) {
   |                               ^^^^^^^^^^^
   |
help: give this argument a name or use an underscore to ignore it
   |
55 |     fn truncate_to_ptr(&self, _: (u64, bool)) -> (u64, bool) {
   |                               ~

The version seems recent enough, doesn't it?
$ ./build/x86_64-unknown-linux-gnu/stage0/bin/rustfmt --version
rustfmt 1.4.37-nightly (fdf6505 2021-09-07)

Oh, with --edition=2018 added it works. Sorry for the noise.


Should this library now change to just be a plug-in wrapper around Google's newly released Material Components?

This just happened AFAICT, but just wanted to start a discussion and hear what the thoughts of the maintainers are.
Google has released a preview of their Material Components which effectively offers first class implementations for the web and other platforms, should this library switch directions and throw out its proprietary implementations and just focus on being a pure "wrapper" around that library in the form of an Aurelia plug-in?

MDC really surprised me (this is an understatement)..
That project seems to be basically MDL v2 (Material Design Lite). There's an open question on the MDL repo which asks about the relationship of the two projects.
I had a look and it really looks the same, same maintainers, same principles, even my Aurelia example is copied (and renamed).
While this is really an interesting project, the only relation to Materialize is that both implement Material Design principles. I think Materialize users will expect a "Materialize bridge" to implement wrappers for Materialize, not MDL/MDC (there are some Aurelia plugins for MDL). 😃
MDC would probably be a good candidate for another bridge of its own.
Now that the catalog app infrastructure relies on gist.run for samples this wouldn't even take a lot of time, I guess (apart from writing the docs). If anyone wants to take a shot at that, I'd be glad to help.
But this project here (Materialize bridge) will continue providing Materialize wrappers. 😃

@Thanood Awesome! Thanks for taking the time to do a deeper analysis and for giving such a thoughtful response. I totally agree, based on both your analysis and further analysis of my own later last night, that it makes sense that this lib (AMDB) should continue to provide its own full implementation of Materialize.


Popover content property accepts not just string and function

The popover content option would also accept any content that jQuery's .html() accepts, see https://jsfiddle.net/Herst/bgc3puuf/
Is there any reason why the documentation does not reflect that?
http://getbootstrap.com/javascript/#popovers-options

Hi @Herst!
You appear to have posted a live example (https://fiddle.jshell.net/Herst/bgc3puuf/show/light/), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

line 15, column 7: W013: Bootstrap version might be outdated. Latest version is at least 3.3.4 ; saw what appears to be usage of Bootstrap 3.0.0
line 19, column 7: W013: Bootstrap version might be outdated. Latest version is at least 3.3.4 ; saw what appears to be usage of Bootstrap 3.0.0

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

Oops, here a revisited example: https://jsfiddle.net/Herst/bgc3puuf/

Is there any reason why the documentation does not reflect that?

Nope—if you have a suggested edit, please feel free to propose in a PR. Otherwise, since this isn't a critical fix, I'm closing as a won't fix on our end.

Just to clarify: My question was also about whether there could be any unforeseen side-effects.

Incidentally fixed by c7d8e7a.


Implement native schema inference for PostgreSQL

🗣 Description
Implements native schema inference for PostgreSQL.
Also changes to make the integration tests work on the hosted runner with building the binary on spiceai-runners and running the tests on ubuntu-latest-16-cores.
🔨 Related Issues
datafusion-table-providers PR: https://github.com/datafusion-contrib/datafusion-table-providers/pull/151
Closes #2462
Part of #2147
📄 Documentation Requirements
N/A

Changes look good to me. I triggered a benchmark test on this branch for verifying changes https://github.com/spiceai/spiceai/actions/runs/11505291656.


rustfmt compilation hangs on nightly (f4209651e 2017-05-05)

Not sure if this belongs here, but I've tried to install rustfmt via cargo install rustfmt on two different architectures (amd64 and armv7) and in both cases the compilation hangs when it tries to compile the env_logger crate.
The output of the command is:
$ rustup update
info: syncing channel updates for 'nightly-armv7-unknown-linux-gnueabihf'
info: checking for self-updates

  nightly-armv7-unknown-linux-gnueabihf unchanged - rustc 1.19.0-nightly (f4209651e 2017-05-05)

$ cargo install rustfmt
    Updating registry `https://github.com/rust-lang/crates.io-index`
  Installing rustfmt v0.8.3
   Compiling unicode-xid v0.0.4
   Compiling unicode-segmentation v1.1.0
   Compiling libc v0.2.22
   Compiling multimap v0.3.0
   Compiling bitflags v0.8.2
   Compiling void v1.0.2
   Compiling diff v0.1.10
   Compiling unreachable v0.1.1
   Compiling utf8-ranges v1.0.0
   Compiling log v0.3.7
   Compiling thread-id v3.0.0
   Compiling thread_local v0.3.3
   Compiling rustc-serialize v0.3.24
   Compiling regex-syntax v0.4.0
   Compiling getopts v0.2.14
   Compiling either v1.1.0
   Compiling itertools v0.5.10
   Compiling strings v0.0.1
   Compiling memchr v1.0.1
   Compiling aho-corasick v0.6.3
   Compiling same-file v0.1.3
   Compiling walkdir v1.0.7
   Compiling term v0.4.5
   Compiling rustfmt v0.8.3
   Compiling regex v0.2.1
   Compiling toml v0.2.1
   Compiling syntex_pos v0.58.1
   Compiling syntex_errors v0.58.1
   Compiling syntex_syntax v0.58.1
   Compiling env_logger v0.4.2
# Halts indefinitely, uses 100% of 1 CPU

Again, the exact same thing happened on an amd64 architecture. I'm guessing this may be a compiler error so please let me know if I should report it somewhere else.

This is probably hanging on one of the larger syntex_* crates, not env_logger, it looks like env_logger because Cargo compiles crates in parallel where possible. I presume this is a compiler bug. Will investigate.

I reported this here: https://github.com/rust-lang/rust/issues/41832 I couldn't reproduce locally. Closing this issue since it should be investigated on the Rust repo.


I can only use one FlareComponent at a time

Whenever I try to load 2 FlareComponent like so, I get the following error,
<FlareComponent animationName='idle' width={200} height={260} file={actor} />
<FlareComponent animationName='idle' width={200} height={260} file={actor} />

message: "Expected null or instance of SkPath, got an instance of SkPath"
name: "BindingError"
stack: "BindingError: Expected null or instance of SkPath, got an instance of SkPath↵    at BindingError.<anonymous> (http://localhost:3000/static/js/0.chunk.js:86543:23)↵

Sorry for the delay here, please try with version 1.0.6.


Add ability to deploy Openstack over IPv6 addressing on a standalone VM.

Add ability to deploy Openstack with IPv6 addressing on a stanadalone edpm-compute.
The new make target standalone_ipv6 will create a dedicated ipv6 libvert network for ipv6 control plane.
Deploy the existing ipv6_lab which provides routing and DNS services for ipv6
Create a single edpm-compute VM and deploys Openstack with ipv6 addressing

Hold, I need to rebase


Add desktop & mobile app links to the login screen

Hi developers, I would like to suggest adding some links to desktop & mobile app links to the login screen. I've seen people often get confused about where they can download and install apps to get notifications. Thanks!

@zulipbot  Would love working on this , can you assign these to me?

Can I take this one ?

Thanks for the suggestion, @pjb7687 ! I started a discussion thread in the development community.


WIP breaking: Add event_type to metrics records

Motivation

JIRA: https://issues.jboss.org/browse/AEROGEAR-2250
Description
Add EventType field to metrics, split validation of different event types.
Currently added the existing event types as "init" and "security", these can be changed.
Progress

[x] Add event_type to parsed json payload
[x] Add event_type to table creation script
[x] Add validation for different eventTypes
[x] Update curl examples on readme
[ ] Update unit tests

This approach assumes that the security event type would not contain App or Device metrics, whereas they definitely will very soon.
As the number of event types grows, I can see this approach leading to headaches and lots of code repetition to try fix those headaches.
The Metrics endpoint allows any and all fields defined in the Metric strcut. Instead of explicitly checking the event type and then conditionally checking fields related to that type, we should validate any field if it is present.

Could not say that better! I think that jira ticket is flawed and we going to introduce some bugs with this change.

Could not say that better! I think that jira ticket suggest invalid aproach and we going to introduce some bugs with this change.

I think the JIRA ticket is fine. The only reason we want the event type is so we can filter out particular data when we query the database. The rest of this PR is really good but the validation part makes an incorrect assumption that we should deal with the Metrics payload in different ways depending on the event type. This breaks the API because it creates a situation where the one endpoint accepts different types of payloads.
Different Event Type != Different Payload. The event type is just a string. The Metrics Payload is still the one type of payload and any field that is present must be validated.

The only reason we want the event type is so we can filter out particular data when we query the database.

But we already have the same information as content is in Security namespace.
With type api payload makes no sense.
For example:
"type":"",
"data":{
 "Security":{..},
 "Init":{...}
}

:-1: for having the type field really. How can we require it if we won't have a guarantee that the fields themselves will be present? If queries rely on type, they wouldn't work with records that don't have the appropriate fields in data`.
I think it's time for the app metrics service to implement the fanning out of the keys in data to separate records, I'd say even separate tables that cater directly to the data that we expect to come in each metrics data type.

data.app and data.device to their own tables
data.security to two tables:

One for storing the raw {id, name, passed} check data
Another for storing a consolidated result %. i.e. clientId had x% reliability in the set of checks done at timestamp.

@paolobueno Love your suggestion here!
Storing everything in single table may cause major of problems in the future:

Performance (Single table without indexes on the queries could get really slow.
Scalability (Adding new metrics is painful and it will be hard to track)
Pruning old data
Changing data format for metrics (migrations)

TL;DR - Mobile provides all information that backend needs.
It's up to backend now to get that organized in the way it needs to be done.
Adding type provides conceptual problems in the api.

Going to close this PR for now in favor of some spiking beforehand. I'll probably keep some of the work in these commits anyhow.


TQDM Version breaks training

Description
With tqdm 4.56.2 I get anAtrributeError: __error__ in trainer/_base.py at line 276:
with tqdm(batch_iter, total=num_batches_to_use) as it:
Installing tqdm==4.23.0 fixes this

Hello @DeastinY,
can you share the whole error message and/or code which helps us to reproduce the issue?
Thanks!


Updating cloud callout messaging

References

TODO

Submitter checklist

[ ] The PR request is well described and justified, including the body and the references
[ ] The PR title represents the desired changelog entry
[ ] If the change against WireMock 2 functionality (incompatible with WireMock 3),
it is submitted against the 2.x branch
[ ] The repository's code style is followed (see the contributing guide)

Details: Contributor Guide

Perhaps we should vary the utm_term so it's unique per callout? Or something similar.


Performance optimization, how to reduce the time spent on "collectGarbage" in the Vulkan backend?

Discussed in https://github.com/google/filament/discussions/7042

Originally posted by zy4944796 August  4, 2023
Good day everyone, I am trying to optimize the Vulkan backend to make its performance similar to that of the OpenGL backend. I have added systrace to the engine code and compared the systrace files of the OpenGL backend and Vulkan backend. I found that the main reason why the Vulkan backend is slower than the OpenGL backend is because the Vulkan backend needs to call collectGarbage, which takes 1-2ms. Is there any way to optimize this?
This is a comparison of the execution time of FEngine::execute for testing. Please see the yellow and red lines, which represent the time taken for the Vulkan backend to draw 500 cubes and the OpenGL backend to draw 1000 cubes, respectively. The times are similar.

Through analysis of the Systrace file, it was found that the collectGarbage function called in the commit of the Vulkan backend takes up to 1-2ms of time, with VulkanDisposer::gc accounting for a significant portion of the time, as shown in the following figure:

Why does VulkanDisposer::gc take so much time? Are there any ways to reduce the time taken for this part? For example, by reusing its resources or moving it to another thread?

@poweifeng the code itself seems pretty inefficient:

removeReference does a hash-map lookup while we're iterating the hash-map! That's bad.
there are 2 iterations of the hash-map, maybe we could have only one?
a new hash-map is recreated each time gc is called, by insertion of each element! So even when gc has nothing to do, it will:

iterate the hashmap twice
recreate the hashmap by copy-inserting each element to a new one
destroy the old hashmap
and potentially doing lookups during the 1st iteration

So there are actually at least two issues:

The latency caused by doing gc on the command buffers

This was done to address a crash where the images were being gc'd before the command buffers that contain references to them are even processed. This issue was filed internally at google.
I tried to address it in PR #6855. But it's definitely too aggressive as we note that the fence waiting is on the corresponding command buffers that need to be gc'd. gc() is then 22ms on a pixel4xl.  (I didn't know that it'd be waiting on every frame.)  See Image A
Maybe we don't want to actually gc the command buffers that often, but instead make sure that each resource depending on the submission of a command buffer doesn't get freed/gc'd before the buffer is submitted. Image B shows that even if we don't wait on the fence, free-ing the command buffer is still expensive.

VulkanDisposer issues raised by @pixelflinger and @zy4944796

Inefficient iteration of the hashmap
Inefficient deletion of items from the hashmap
Do we even need to use a hashmap here?
There is about 350 items in the disposer at any frame for the DamagedHelmut.
In Image B disposer::gc takes 1ms. This should be able to go down significantly.

Image A

Image B
Without waiting on the fence of the command buffer.

ah... but I think for (1) we shouldn't wait for the fence, we should just check if it signaled and proceed only if it did. If it didn't, we keep the entry in the list. Alternatively, you could have a gc thread that wait on fences (but can you wait on several fences?) that does the clean-up as needed. But I'm nut sure it'll buy us that much.

As of #7735, all the ref-counting mechanisms have been implemented and should now be relatively stable.


http task output

Hi,
I was trying a simple hello world example. but i was not able to figure out how to configure the output of the microservices in workflow json. example my wf has only 1 task which will call hello world. this would return a json.
Also i was wondering on how to have the inputs mapped to the task and microservices other than sending params through uri.

@amoolya19 do you want to do an HTTP POST with request body?  Take a look at the documentation for http task:
https://netflix.github.io/conductor/metadata/systask/#http

Hi @v1r3n ,
i tried to a scenario where output of task1 (HTTP type) is input of task2 (type HTTP). i tried to place input of task1 in body tag for task2. but task2 fails and output json has null in the body.
worflow json
`[{
"name": "test_workflow",
"description": "test http workflow",
"version": 1,
"tasks": [{
"name": "test_task1",
"taskReferenceName": "tt0",
"inputParameters": {
"http_request": {
"uri": "http://localhost:8081/2",
"method": "GET"
}
},
"type": "HTTP"
},
{
	"name": "test_task1",
	"taskReferenceName": "tt1",
	"inputParameters": {
		"http_request": {
			"uri": "http://localhost:8081/tt1",
			"method": "GET",
			"body" : "${tt0.output.number}"
		}
	},
	"type": "HTTP"
}
]

"schemaVersion": 2

}]`
output json for task 2
{
"taskType": "HTTP",
"status": "FAILED",
"inputData": {
"http_request": {
"uri": "http://localhost:8081/tt1",
"method": "GET",
"body": null
}
},
am i missing anything here?

@v1r3n
i also tried with simpler version for workflow where there is only 1 task which is of type http and tried to output the response like
"outputParameters": {
"number": "${test_task1.output.number}"
} and after execution, still the output was null for number. is there any other configuration that needs to be done?

@amoolya19 the output of the HTTP task is contained in the response field of the output of the task.
In your original workflow example, the configuration you have supplied for the second task is:
"body" : "${tt0.output.number}"

If you look at the output of the HTTP task, the number is located atresponse.body.number.
You have to use the JSON Path of the field from the outputData parameter to inject the value in another task.
Try modifying the parameter config for tt1 task, as follows:
"body" : "${tt0.output.response.body.number}"

OR below should also work, as it is a valid JSON Path.
"body" : "${tt0.output..number}"

@v1r3n,
this worked. actually i had tried this yesterday but not sure why it dint work that time.Any ways thank you for the reply.

I am facing the same issue - not able to extract value from JSON
Both ${tt0.output.response.body.number} and "${tt0.output..number}" returns null.
where as ${tt0.output.} returns the whole string. But ${tt0.output.response} returns null. Any pointers would help.

@v1r3n
Any pointers would help. Thanks

May be I'm missing out on this oint.
One more thing, outputParameters cannot be configured as part of workflow definition, as it is created/updated as the tasks are getting executed.
How else to configure output parameters?

It is working, i was using an old server jar.


Bigger machines?

32x32x32 maybe?

131313 is the max size because one chunk is 161616 meaning only one chunk per machine is loaded. Your suggestion would cause eight chunks to be loaded. Also, why would you need bigger machines?

What XFactHD said. Never going to happen, sry.

Do those chuncks get unloaded if the machine is broken and no one uses that chunk anymore?


Fix issue with entry points for Python 3.6 and 3.7

Description:

Updated entry point interface to not fail for Python 3.6 and 3.7
Added test for custom plugin
Made __init__() a required method in the backend interface definition

Type of change:

[x] Bug fix.
[ ] New feature.
[ ] Documentation update.

Checklist for the reviewer:

This checklist should be used as a help for the reviewer.

[ ] Is the change limited to one issue?
[ ] Does this PR close the issue?
[ ] Is the code easy to read and understand?
[ ] Do all new feature have an accompanying new test?
[ ] Has the documentation been updated as necessary?

Codecov Report
Patch coverage: 83.33% and project coverage change: +0.11 :tada:

Comparison is base (6262e07) 74.31% compared to head (26b52c5) 74.42%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #113      +/-   ##
==========================================
+ Coverage   74.31%   74.42%   +0.11%
==========================================
  Files          14       14
  Lines        1269     1271       +2
==========================================
+ Hits          943      946       +3
+ Misses        326      325       -1

Impacted Files
Coverage Δ

tripper/triplestore.py
82.44% <80.00%> (+0.31%)
:arrow_up:

tripper/interface.py
90.00% <100.00%> (+1.11%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


flutter.io (at least the RN page) looks odd in Safari Mobile

This is actually the embedded webview (inside tweetdeck), but I suspect it looks the same on Safari mobile.

Not a big deal. Just sharing.  @filiph

cc @mjohnsullivan
I can reproduce on iOS simulator, but only for the new ReactNative page. I'll investigate.

Theory no. 1: It's the <table> elements that the authors are using to display iOS and Android output next to each other. Should be a relatively easy fix, just using flex / float elements instead.
Can we ask the authors to rewrite or should we do that for them?
Sidenote: This cannot be reproduced in Chrome, even in the mobile device simulation mode.

Hopefully the authors can fix that.
On 28 February 2018 at 20:59, Filip Hracek<EMAIL_ADDRESS>wrote:

Theory no. 1: It's the  elements that the authors are using to
display iOS and Android output next to each other. Should be a relatively
easy fix, just using flex / float elements instead.
Can we ask the authors to rewrite or should we do that for them?
Sidenote: This cannot be reproduced in Chrome, even in the mobile device
simulation mode.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/15005#issuecomment-369474398,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGQWJrl9sJ3bTChHNXxwphCnpdOwb8Vks5tZ4AlgaJpZM4SXr4R
.

Paging @sanketsahusoft -- could you please fix this? The issue is that on iPhones, the page is effectively broken (text hidden on the left and no way to scroll horizontally).
The solution is to use something else than table for laying out the Android+iOS screenshots. I think the issue is sever enough to warrant (temporarily) abandoning the horizontal layout of the screenshots, and instead put them one after another.
A better solution would be flexbox, or good old float: left + clear: left.

This PR might help us get through this for now. Tested it on iPhone 6 and 7 simulators and it worked just fine.

The PR has landed, so I'm going to close this. Thanks!


New sheet to support The Dragon Prince, Tales of Xadia

This is a new sheet to support the Tales of Xadia game.  It is unofficial and is geared to support what to the best of my ability to guess the actual game will be like once it is released. This contains minimal support for the play test material, but can be done.
Changes / Comments
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Status of KodaDot Github repositories & dynamics

Write an article status quo and activity of KodaDot's GitHub repositories

draft the current contributors culture of how we are operating, from issue, discussion, pull-request, code-review and paying out contributors
the flow of someone makes the issue, bug report, contribute to code review
code review guild - why is it important to have quality code and peer-reviewed codebase
highlight

assign bot (petersopko will be launching writing on that #3010 )
payout bot (that trusted developers does payouts between themselves for PRs)

contributors, forks, stars, growth of the repository
name our popular repositories, so people can contribute elsewhere
mention contributing.md
post it to dev.to

Anything else we should mention?

Ref

#1661

i will take this
@yangwao i feel like article writing might be an edge case for assign bot as it could take quite a bit longer than a code-relateed issue

:wave:

I am working on this please

Almost completed

Completed

https://dev.to/ayushkumar63123/status-of-kodadot-github-repositories-dynamics-59db

Please review and if fine, then please tell how can I get the bounty

@ayushkumar63123
Sorry if this wasn't communicated here but there is already an open draft on medium regarding this topic
https://medium.com/kodadot/contributor-culture-at-kodadot-665243d3d6a6
Also, your article feels very rushed and low effort in terms of doing research on our workflow. Please take your time on such issues as this text should give interested individuals an insight into how we work and lower the burden to contribute in general.


Where is the "EmpatheticDialogue/sys_dialog_texts.train.npy"?

Hi Qintong, I am so happy to see your great job! I have a question about the preprocess.py, I don't see the "EmpatheticDialogue/sys_dialog_texts.train.npy" or "EmpatheticDialogue/sys_dialog_texts.dev.npy" in the according file, where should I get these files? Thanks!

Oh, maybe data from here?

Oh, I know, .npy format files can also store text, so it's all clear，nice work! Hope see you in AAAI 2023.


Different bar widths with multiple datasets using time axis

Hello everyone.
I have searched the documentation but could not find an answer to my problem, maybe someone can help me out.

The temperature bars are thicker than the humidity bars because the temperature has only 4 bars, while the humidity has 6 bars (one is actually being swallowed because of the problem). It seems like the width of the bars are adjusted based on each individual dataset.
Can i change so that all bars have the same width? Any ideas are welcome.

Please ask questions on StackOverflow: https://stackoverflow.com/questions/tagged/chart.js
Also, see the relevant documentation: https://www.chartjs.org/docs/latest/charts/bar.html#dataset-configuration


How to use foreign struct as SimpleObject

HI,
I want to use a struct with lots of field members as a GraphQL object from other crate.
After reading the doc, I know that using scalar! and create a wrapper type can solve this problem.
However, scalar type force client side get the whole object data. In my scenario, client side may query the fields on this large object.
Is there a way to solve this problem?

You can wrap the foreign type to a newtype.
struct LocalObj(ForeignObj);

#[Object]
impl  LocalObj {
    async fn value(&self) -> i32 {
        ....
        }
}

Thanks

Thanks


IPv6: When I set the prefix delegation size to /62 dhcpd6 (DHCPv6 Server) stops and can not restart

Hello,
my english language is not good, I hope you understand what I write here:
Version:
OPNsense 19.7.4_1-amd64
FreeBSD 11.2-RELEASE-p14-HBSD
OpenSSL 1.0.2s 28 May 2019
Hardware: PC-Engines, CPU-Typ: AMD GX-412TC SOC (4 cores),
with 4 interfaces: WAN (connected to the router), LAN, DMZ and WLAN
IPv6 in my router (FRITZ!Box):
Internet, IPv6: connected since 02.10.2019, 14:45 Uhr,
IPv6-Adresse: 2a02:810d::35:2011:9a58:4464:2e3b, Gültigkeit: 5048/2348s,
IPv6-Prefix: 2a02:810d:1a80:700::/62, Gültigkeit: 5048/2348s
WAN configuration in OPNsence (is connected to the first LAN-Interface of my router):
IPv6 configuration typ: DHCPv6
DHCPv6 Clientkonfiguration:
modus: simple
Request only an IPv6 prefix: yes
Prefixdelegationsize: /63 bit <--------------- !!!
Send an IPv6-Präfixnote: yes
IPv6-Prefix: 2a02:810d:1a80:700:: (used by the Router)
WLAN-Interface:
IPv6-configuration-typ: Track Interface
IPv6-Track-interface: IPv6-interface: WAN, IPv6 Prefix ID: 0x0
--> Then the IPv6-Prefix is: 2a02:810d:1a80:702::
LAN-Interface:
IPv6-configuration-typ: Track interface
IPv6-Track-Interface: IPv6-interface: WAN, IPv6 Prefix ID: 0x1
--> Then the IPv6-Prefix is: 2a02:810d:1a80:703::
DMZ-Interface:
No IPv6-Konfiguration because there is no another free IPv6 Prefix ID.
When I set the Prefixdelegationsize to 63 the dhcpd6 runs normal and all works well. I have IPv6 only on the Interfaces WLAN and LAN. the DMZ have only IPv4.
But when I set the Prefixdelegationsize to 62 (what I need to have also IPv6 on DMZ) the dhcpd6 stops and can not restart.
Following IPv6-Configuration set I for Prefixdelegationsize 62:
WAN configuration in OPNsence (is connected to the first LAN-Interface of my router):
IPv6 configuration typ: DHCPv6
DHCPv6 Clientkonfiguration:
modus: simple
Request only an IPv6 prefix: yes
Prefixdelegationsize: /62 bit <------------------ !!!
Send an IPv6-Präfixnote: yes
IPv6-Prefix: 2a02:810d:1a80:700:: (used by the Router)
WLAN-Interface:
IPv6-configuration-typ: Track Interface
IPv6-Track-interface: IPv6-interface: WAN, IPv6 Prefix ID: 0x2
--> Then the IPv6-Prefix is: 2a02:810d:1a80:702::
LAN-Interface:
IPv6-configuration-typ: Track interface
IPv6-Track-Interface: IPv6-interface: WAN, IPv6 Prefix ID: 0x3
--> Then the IPv6-Prefix is: 2a02:810d:1a80:703::
DMZ-Interface:
IPv6-configuration-typ: Track interface
IPv6-Track-Interface: IPv6-interface: WAN, IPv6 Prefix ID: 0x1
--> Then the IPv6-Prefix is: 2a02:810d:1a80:701::
When I do setup this configuration the DHCPv6-Server stops and can not restart.
The IPv6-configuration of the clients can not initialisized!
Best Regards
Andreas Gerlich

I have updated the issue template.

I can confirm the following on test. Primary router set to issue /62 prefixes, secondary router set to request /62 prefixes. DHCP6 server is running on secondary. Setting the secondary dhcp6 server to manual I can see I have a /63 available prefix delegation size.

At 07.10.19 at 11:23 wrote Martin Wasley:

I can confirm the following on test. Primary router set to issue /62
prefixes, secondary router set to request /62 prefixes. DHCP6 server is
running on secondary. Setting the secondary dhcp6 server to manual I can
see I have a /63 available prefix delegation size.

I have a little bit difficulties to understand. Is that a bug? In the
menu of opnsense I can set up /62 prefix delegation size, so I mean that
should be right to set, or?
Best Regards
Andreas Gerlich
--
Dipl.-Ing.(FH) Andreas Gerlich
Dominikus-Zimmermann-Str. 6
D-86956 Schongau
Germany
Tel.: +49(0)8861/6 90 95 11 (phone with voice box)
+49(0) 163/4 36 32 32 (mobile phone)
Email<EMAIL_ADDRESS>Nick im IRCNet: agl
GPG-Key: 5D8EA14A
Fingerprint: 4450 3C20 E8B4 E492 D822 BEB4 3B97 10F8 5D8E A14A
University of Ulm, Germany
open source project "Yet Another Z80 Emulator by AG":
--> http://yaze-ag.de/

I'm checking the dhcp6c responses, which has indirectly led me to an unrelated bug which I need to look at. At present my test system all checks out correctly. I have my WAN set up to request a /62. I then  have three interfaces, LAN2, LAN2, LAN3. I am using Track WAN interface for each and I increment the IPv6 Prefix ID by 1 on each interface, so they are 1,2,3. I have then moved my LAN cable from my PC to each port and the correct IPv6 prefix is given by dhcpd6 on each port. I have checked both the radvd.conf when manual overide is not used and the dhcp6d.conf when it is and both are correct.
I was going to ask you to set the debug option in dhcp6c so we can see what prefix length is being obtained by dhcp6c, but there is a bug in the 'debug' output and it's not showing the prefix info. There is a way around that but you would manually need to run dhcp6c from the shell with  full debug mode '-D' and foreground mode '-f' to see the full debug output.
Do you know what prefix size your ISP is giving you? Sounds to me like they are only giving you a /62, so then that will only be a /63 on the Opnsense router.

Hello Martin Wasley,
Am 12.10.19 um 13:28 wrote Martin Wasley:

I'm checking the dhcp6c responses, which has indirectly led me to an
unrelated bug which I need to look at. At present my test system all
checks out correctly. I have my WAN set up to request a /62. I then have
three interfaces, LAN2, LAN2, LAN3. I am using Track WAN interface for
each and I increment the IPv6 Prefix ID by 1 on each interface, so they
are 1,2,3. I have then moved my LAN cable from my PC to each port and
the correct IPv6 prefix is given by dhcpd6 on each port. I have checked
both the radvd.conf when manual overide is not used and the dhcp6d.conf
when it is and both are correct.

What have you there for the IPv6-Praefix-Length from your ISP?

I was going to ask you to set the debug option in dhcp6c so we can see
what prefix length is being obtained by dhcp6c, but there is a bug in
the 'debug' output and it's not showing the prefix info. There is a way
around that but you would manually need to run dhcp6c from the shell
with full debug mode '-D' and foreground mode '-f' to see the full debug
output.
Do you know what prefix size your ISP is giving you? Sounds to me like
they are only giving you a /62, so then that will only be a /63 on the
Opnsense router.

My Cable-Fritz!Box (Internet-Router) have a fix installed
IPv6-Praefix-Length of /62 in the firmware. In the forum of
"Kabel-Deutschland" (my ISP) there are a lot of people which have the
same problem like I. The IPv6-Praefix-Length can not changed it's fix on
/62.
I have to change the Internet-Provider to get an another
IPv6-Praefix-Length.
Best Regards
Andreas Gerlich
Dipl.-Ing.(FH) Andreas Gerlich
Dominikus-Zimmermann-Str. 6
D-86956 Schongau
Germany
Tel.: +49(0)8861/6 90 95 11 (phone with voice box)
+49(0) 163/4 36 32 32 (mobile phone)
Email<EMAIL_ADDRESS>Nick im IRCNet: agl
GPG-Key: 5D8EA14A
Fingerprint: 4450 3C20 E8B4 E492 D822 BEB4 3B97 10F8 5D8E A14A
University of Ulm, Germany
open source project "Yet Another Z80 Emulator by AG":
--> http://yaze-ag.de/

My ISP does things properly, I get a /48 Prefix.
I would suggest you find a sensible ISP, there is absolutely no reason to limit users to a /62, it's stupid.


Disable doCollect for lookup and set/get API

Improves has/get/set etc. Jsrt API calls ~15% on xplat. It should also boost Windows
numbers too (will check). This change affects only ChakraCore
Why?
Assumption; Js land calls to a user defined function. This function reaches multiple has/get/set calls before finishing its' task. Setting doCleanup = true on each and every API calls effects the performance badly.
Besides, we were updating tickCountNextDispose value regardless from doCleanup is true or not. (SetIsScriptActive)

        if (!threadServiceWrapper || !threadServiceWrapper->ScheduleNextCollectOnExit())

We will not be scheduling IdleGC on these calls?  Seems risky

Refers to: lib/Runtime/Base/ThreadContext.cpp:1561 in 22f31e7. [](commit_id = 22f31e786f625f5f8621f043e6fc719e48e6e959, deletion_comment = False)

We will not be scheduling IdleGC on these calls? Seems risky

These methods are being called under a user defined function. We already schedule it at least once?

I applied these changes to node-chakracore, and I'm not seeing anything like a 15% gain on the acmeair benchmark, more like a 1% gain.

I applied these changes to node-chakracore, and I'm not seeing anything like a 15% gain on the xplat

Perhaps you missread the description...

@MSLaguana see the copy-paste of description;
'Improves has/get/set etc. Jsrt API calls ~15% on xplat. '

We will not be scheduling IdleGC on these calls? Seems risky

These methods are being called under a user defined function. We already schedule it at least once?

But once it ran, it will not run again until you schedule it.  It is there to detect activity that may need clean up.

@MSLaguana see the copy-paste of description;'Improves has/get/set etc. Jsrt API calls ~15% on xplat. '

I think @MSLaguana is pointing out that overall, it doesn't really benefit AcmeAir much.
The concern here is that this case can cause memory usage regression in cases, so we have to wonder whether the change is worth it.

@curtisman I will double check the memory regression (if there is) but I don't agree on gains part. Assuming no memory regression, double digit improvement to API calls is not trivial.

Assuming no memory regression, double digit improvement to API calls is not trivial.

That's is microbenchmarking. In the real world, it depends on how often are these API are called.
Put it in the context of the over all scenario, if it only translate to gain in the noise range, then it is not very useful.

Well, if the whole engine perf assumption translates to AcmeAir, cool closing this one.


record

Surface syntax
record S (x_i : T_i) ...
  a_1 : A_1
  ...
  a_i : A_i

will be converted to a big sigma type when dealing this definition
def S (x_i : T_i) ... => (a_1 : A_1) x ... x (a_i-1 : A_i-1) x A_i

[x] Handle record definition as a huge sigma
[ ] Insert enough renaming to expand field access as tuple indexing

close with 35e3da98adf5e2ed3b6594cb26fe55ac7cf1b7a4
Let dot access syntax for field postpone after multiple tuple and first, second accessement done


Split TestFlushesAfterInterval, remove Timer.sleep

TestFlushesAfterInterval flaps on travis-ci due to relying on wall clock.

Create TestFlushingEmptiesTheBuffer, which tests FlushBuffer.
Remove Timer.sleep from TestFlushesAfterInterval by mocking the timer.

RE: discussions earlier today, should we replace these mocks now with a time mocking library? I feel like it's better to try dropping this in now: https://github.com/benbjohnson/clock

Fixed in #92.


Battery Charge Status missing after switching from Modbus

Hi, I've been using the old method of hard writing into the config.yaml and modbus.yaml to read from my H1.5E inverter for a while and have recently switched over to this much cleaner implementation, but it's missing one of the g features that I used the most previously and I can't figure out how to get it back (or if I'm just missing it)
There used to be a sensor called battery-charge-status which showed as a positive number when charging the batteries or a negative number when discharging. I used this with gauges to instantly see what was going on with energy use in a live way. It easily let me see if my house load was greater than my solar generation etc, and if the battery was depleted it would just show as zero.
It's it possible to bring back this sensor, or is there some way I can add it in myself?
Cheers.

Maybe it was Battery Discharge Power? In which case that's now "Inverter Battery Power" (invbatpower)

Think I was using version 0.9 not sure which implementation, I hacked it in when I first starting using HA, I didn't even know about using HACS 🙄.
The setting above still give a positive number with battery drainage.

I don't think there's ever been a H3 sensor with "status" in the name, looking through the history.
"Inverter Battery Power" should be signed. Battery Charge will be non-zero when Inverter Battery Power is negative, and Battery Discharge will be non-zero when Inverter Battery Power is positive.
Both the old Battery Discharge Power and the new Inverter Battery Power are positive for discharging, negative for charging. That matches what you're seeing, no?
What values do you get when you're charging?
Maybe you created a template sensor in your configuration.yaml which flips the sign?

@tekhiegit I've never known there to be a charge status sensor included as part of the old modbus,
I did write a template sensor some time ago to do similar (and probably shared it on the facebook group)
First check you configuration.yaml to make sure there isn't a template sensor already defined, if there is to get it working just change the source sensor names as follows - sensor.discharge_power to sensor.discharge and sensor.charge_power to sensor.charge
If it isn't there add this to your configuration.yaml templates it will give you a sensor in watts that is positive for charge, negative for discharge.
      battery_charge_status:
        friendly_name: "Battery Charge or Discharge"
        unit_of_measurement: "W"
        value_template: "{{ ( (states('sensor.battery_charge') | float(default=0) - states('sensor.battery_discharge') | float(default=0)) * 1000) | round(0) }}"

If you want it in kW change the * 1000 to * 1, the unit back to "kW" and the round(0) to round(2)

Aha, maybe that explains it.
Note that the old Battery Discharge Power (and the new Inverter Battery Power) are the same as battery_charge_status, but with the sign flipped (battery_charge and battery_discharge are derived from Battery Discharge Power, after all).

@tekhiegit I've never known there to be a charge status sensor included as part of the old modbus,
I did write a template sensor some time ago to do similar (and probably shared it on the facebook group)
First check you configuration.yaml to make sure there isn't a template sensor already defined, if there is to get it working just change the source sensor names as follows - sensor.discharge_power to sensor.discharge and sensor.charge_power to sensor.charge
If it isn't there add this to your configuration.yaml templates it will give you a sensor in watts that is positive for charge, negative for discharge.
      battery_charge_status:
        friendly_name: "Battery Charge or Discharge"
        unit_of_measurement: "W"
        value_template: "{{ ( (states('sensor.battery_charge') | float(default=0) - states('sensor.battery_discharge') | float(default=0)) * 1000) | round(0) }}"

If you want it in kW change the * 1000 to * 1, the unit back to "kW" and the round(0) to round(2)

Ah, Amazing! Yes thank you, it was all done so long ago and when I had basically zero yaml or HA knowlage, so when I added in this modbus I just stripped everything out from config, I was clearly too brutal with the delete key. Thank you both for you help.


Allow to configure library by menuconfig

Patch to make component configurable by make menuconfig

@erfengwelink Any news on this PR?

Hi guys , I’ve not understood what news U wanna get to know , could U tell me some more details please.
Erfeng .
原始邮件
发件人:Ruslan V<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>发送时间:2018年6月11日(周一) 17:35
主题:Re: [erfengwelink/modbus_port_esp32] Allow to configure library bymenuconfig (#1)
@erfengwelink Any news on this PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Anyone SBAT open up a PR that can be merged

Expected Behavior
CircleCI jobs run for PRs opened by anyone
Current Behavior
CircleCI jobs only running for PRs opened by committers

Looks like https://github.com/celo-org/celo-monorepo/pull/343 is having this issue

Enabled "Build Forked Pull requests" at https://circleci.com/gh/celo-org/celo-monorepo/edit#advanced-settings and https://circleci.com/gh/celo-org/celo-blockchain/edit#advanced-settings

https://github.com/celo-org/celo-monorepo/pull/551 failed on Circle CI since our incremental testing setup assumes all the PRs are from the same repository.


[FIX] Unable to connect to site bug

Apparently admins changed order of

If someone is too lazy to build here's debug version. Will not override release in case someone treasures their collections
app-debug.zip

大哥真牛逼

how can I fix it on phone?

I can't find the file path.

大哥真牛逼

大哥发的这个文件路径找不到啊

If someone is too lazy to build here's debug version. Will not override release in case someone treasures their collections app-debug.zip

In this version, I can't download the manga

how can I fix it on phone?

I can't find the file path.

unfortunately it's impossible to fix it directly on device. There are 2 options:

dowload repository on pc, make proposed change and buid an apk
wait for owner of this repository to accept pull request

If someone is too lazy to build here's debug version. Will not override release in case someone treasures their collections app-debug.zip

In this version, I can't download the manga

Same here. Was not able to dowload for months.
From what I understand Android 12.+ changed the way external_storage_permision works. In code methods that check/enable this permission always return false. I tinkered a bit with code but I had 0 success. Basicly this function needs an update from someone who understand a thing or two about this platform


Can run py2puml only on py2puml

Hi,
I'm able to execute your example, but was not able to run it without errors on my own code.
So, I made a small experiment. I placed a minimal class called myminiclass.py with the below content within the folder py2puml/domain
class MyMiniClass(object):
    def __init__(self, instance_name: str):
        self.name = instance_name

    def get_name(self) -> str:
        return self.name

The effect is now when running py2puml py2puml/domain py2puml.domain  I get the error below:
Traceback (most recent call last):
  File "C:\Users\myuser\.conda\envs\myenv\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Users\myuser\.conda\envs\myenv\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\myuser\.conda\envs\myenv\Scripts\py2puml.exe\__main__.py", line 7, in <module>
  File "C:\Users\myuser\.conda\envs\myenv\lib\site-packages\py2puml\cli.py", line 17, in run
    print(''.join(py2puml(args.path, args.module)))
  File "C:\Users\myuser\.conda\envs\myenv\lib\site-packages\py2puml\py2puml.py", line 17, in py2puml
    domain_item_module: ModuleType = import_module(name)
  File "C:\Users\myuser\.conda\envs\myenv\lib\importlib\__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 984, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'py2puml.domain.myminiclass'

When I delete the file everything is working again for the example. Any ideas / suggestions?
Update:
I have one own python package which I installed in my conda environment. For this I'm able to run py2puml as long as I'm in the conda environment it is installed in. But when I create a new conda environment without it I get the same kind of error again.

hello @smarkwart
Thank you for reporting your issue. Unfortunately, I failed to reproduce the error:

I created the 33-myminiclass-issue branch from main to investigate
I pushed this https://github.com/lucsorel/py2puml/commit/fa628d4d0547ba7f9ed7c4952acb6bdb38447fa2 commit where I:

added the myminiclass.py file
updated the expected plantuml file
ran the unit tests (particulary the tests/py2puml/test_py2puml.py::test_py2puml_model_on_py2uml_domain one) and everything works fine

I could also run the poetry run py2puml py2puml/domain py2puml.domain command and everything worked fine. It also worked without poetry by manually activating the virtual environment.
source .venv/bin/activate
py2puml py2puml/domain py2puml.domain

Thus I believe your problem is related with the way you created or used the conda environment. I am not very familiar with them. Can you be more specific about how:

you created the conda environment of your project
you installed py2puml in your project
you used py2puml in your project

In a classical project, here is how you are supposed to use py2puml:
mkdir -p my-project/my_project
cd my-project
python -m venv .venv

# create the myminiclass.py file and then write python code in it
touch my-project/my_project/myminiclass.py

# activate the virtual env and install py2puml
source .venv/bin/activate
pip install py2puml

# run py2puml (my_project appears twice in the command: 1st as folder name, 2nd as a module name - I know this is not optimal)
py2puml my_project my_project

Can you provide such commands for your conda setup, please?

hello @smarkwart
I hope you are doing well. Given that I could not reproduce the issue you faced and the lack of answers, I chose to close the issue for now.
Feel free to reopen it when you have more details :+1:

I have the same issue. I can run py2puml only on its repository.
I added miniclass.py in py2puml/domain but I get the error:
Traceback (most recent call last):
  File "/home/user/.virtualenvs/poetry_virtualenv_name/bin/py2puml", line 8, in <module>
    sys.exit(run())
  File "/home/user/.virtualenvs/poetry_virtualenv_name/lib/python3.10/site-packages/py2puml/cli.py", line 24, in run
    print(''.join(py2puml(args.path, args.module)))
  File "/home/user/.virtualenvs/poetry_virtualenv_name/lib/python3.10/site-packages/py2puml/py2puml.py", line 12, in py2puml
    inspect_package(
  File "/home/user/.virtualenvs/poetry_virtualenv_name/lib/python3.10/site-packages/py2puml/inspection/inspectpackage.py", line 19, in inspect_package
    domain_item_module: ModuleType = import_module(name)
  File "/fn/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1004, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'py2puml.domain.miniclass'

I am working from inside poetry's virtual environment. Maybe the package needs to be installed in the virtual environment?

Yeah after I installed it in my project poetry env as a dev dependency it worked. I would see some problems if any of the dependencies of py2puml conflicts with those of my package, but so far so good.
Thanks!

py2puml has no dependency (except pytest as a dev dependency, but it won't be installed), don't worry :wink:


Add 'Seasonally adjusted' to headings on page and remove from chart title

e.g. Seasonally adjusted unemployment for young people

Done - can be closed if no other comments

@taz-chiles fancy closing this?

@ODILMichelle delighted to :) I think it could make sense to include notes on what seasonally adjusted in the methodology page, do you think? Shall I close this then and we can pick up as a separate ticket


@xtina-starr => Use destination cookie in new onboarding

Tested these three ways to redirect people after onboarding:

Signup from an auth modal on /some/route -> Go through onboarding -> Get redirected back to /some/route from the destination cookie
Signup from onboarding directly with a query param -> Go through onboarding -> Get redirected to the path in the query param
Signup from onboarding directly without a query param -> Go through onboarding -> Get redirected to home

LGTM!


Spelling and naming problems

There are many spelling errors in the code, for example: in the README, '什么是分布式系统中的闹裂？' should be '什么是分布式系统中的脑裂？'. And in the comment for EncodeRaftLogKey, 'encode raft log key with perfix -> RAFTLOG_PREFIX' should be 'encode raft log key with prefix -> RaftLogPrefix'. Additionally, some variable names do not conform to Go's naming conventions. For instance, constants in Go typically use camelCase instead of uppercase snake_case. If you don't have the time to address these issues, I am willing to offer some help

Thank you, I’d more than happy for you to help, create a branch and start your PR, I will help review your commit code.

Thank you, I’d more than happy for you to help, create a branch and start your PR, I will help review your commit code.

I'm glad I can contribute to this project. I have opened a PR to address this issue and look forward to your comments. In fact, I am very interested in the engineering aspects of MIT 6.824, and if given the opportunity, I hope to contribute more in the future.


feat: support async markuplint API via child_process

Codecov Report

Merging #3 (c899750) into main (401657b) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head c899750 differs from pull request most recent head 8afcb85. Consider uploading reports for the commit 8afcb85 to get more accurate results

@@            Coverage Diff            @@
##              main        #3   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            8         9    +1
  Lines           23        76   +53
  Branches         0         8    +8
=========================================
+ Hits            23        76   +53

Impacted Files
Coverage Δ

packages/markup/src/helpers.ts
100.00% <100.00%> (ø)

packages/markup/src/index.ts
100.00% <100.00%> (ø)

packages/markup/src/rules/markup.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Add Bitshares account name to data to be passed to the partner

Just out of completion reasons we could add the bitshares account name as a third information that needs to be passed to the partner to fill out the counterparty data.
Plus I would switch the order from first presenting the public key and then the hash as the user might also insert it in that order.
But we can check with the others what they think as this is just a really minor thing.

Technically we could do that, but a Bitshares account is not changed often and the user should therefore know it by heart. This is also the reason why the Bitshares community decided to introduce accounts.


feat: port skiplist from AgateDB

As titled.
Thanks for AgateDB .

Codecov Report

Merging #31 (8934422) into main (d9721c7) will decrease coverage by 5.39%.
The diff coverage is 60.86%.

@@            Coverage Diff             @@
##             main      #31      +/-   ##
==========================================
- Coverage   94.74%   89.35%   -5.40%
==========================================
  Files          19       22       +3
  Lines        1580     1879     +299
==========================================
+ Hits         1497     1679     +182
- Misses         83      200     +117

Impacted Files
Coverage Δ

storage/src/lsm_tree/utils/coding.rs
86.23% <ø> (ø)

storage/src/lsm_tree/utils/skiplist/key.rs
50.00% <50.00%> (ø)

storage/src/lsm_tree/utils/skiplist/list.rs
58.96% <58.96%> (ø)

storage/src/lsm_tree/utils/skiplist/arena.rs
85.71% <85.71%> (ø)

Continue to review full report at Codecov.

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


[ACR] Deprecate preview command group and implement same commands in parent command group

Description

This PR deprecates the preview az acr manifest metadata command group and moves the 3 commands into the parent az acr manifest command group. This is essentially a renaming or reorganization of the commands after receiving customer feedback.  There is no new functionality added, just name changes.
Testing Guide

helps['acr manifest show-metadata'] = """
type: command
short-summary: Get the metadata of an artifact in an Azure Container Registry.
examples:

name: Get the metadata of the tag 'hello-world:latest'.
text: az acr manifest show-metadata -r MyRegistry -n hello-world:latest
name: Get the metadata of the tag 'hello-world:latest'.
text: az acr manifest show-metadata MyRegistry.azurecr.io/hello-world:latest
name: Get the metadata of the manifest referenced by digest 'hello-world@sha256:abc123'.
text: az acr manifest show-metadata -r MyRegistry -n hello-world@sha256:abc123
"""

helps['acr manifest list-metadata'] = """
type: command
short-summary: List the metadata of the manifests in a repository in an Azure Container Registry.
examples:

name: List the metadata of the manifests in the repository 'hello-world'.
text: az acr manifest list-metadata -r MyRegistry -n hello-world
name: List the metadata of the manifests in the repository 'hello-world'.
text: az acr manifest list-metadata MyRegistry.azurecr.io/hello-world
"""

helps['acr manifest update-metadata'] = """
type: command
short-summary: Update the manifest metadata of an artifact in an Azure Container Registry.
examples:

name: Update the metadata of the tag 'hello-world:latest'.
text: az acr manifest update-metadata -r MyRegistry -n hello-world:latest --write-enabled false
name: Update the metadata of the tag 'hello-world:latest'.
text: az acr manifest update-metadata MyRegistry.azurecr.io/hello-world:latest --write-enabled false
name: Update the metadata of the artifact referenced by digest 'hello-world@sha256:abc123'.
text: az acr manifest update-metadata -r MyRegistry -n hello-world@sha256:abc123 --write-enabled false
"""
History Notes

[ACR] Deprecate az acr manifest metadata command group
[ACR] az acr manifest: Add show-metadata, list-metadata, and update-metadata commands

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x ] I adhere to the Command Guidelines.

[x ] I adhere to the Error Handling Guidelines.

acr

Could you add some tests for those commands?

Could you add some tests for those commands?

They're already tested. The functions that actually execute haven't changed and are tested in this file - https://github.com/Azure/azure-cli/blob/dev/src/azure-cli/azure/cli/command_modules/acr/tests/latest/test_acr_commands_mock.py

@mabenedi Why are there no ScenarioTest for these commands?

Why are there no ScenarioTest for these commands?

They're all data-plane and setting up a scenario would be reliant on external clients like docker etc. None of our commands in az acr repository command group have scenariotests for the same reason

Anything else I need to do to get this merged?


Icy april/update readme

Merge

Is it normal that there is no "Issues" tab to report bugs on this GItHub repository?
It looks like there is a problem with the directive DenyAllButCloudFlare, it only works when the web server is only listening on either HTTP or HTTPS. If a virtual host is configured both for HTTP and HTTPS, the directive DenyAllButCloudFlare is only applied for HTTPS connections. So if we want to access a website that is not automatically redirecting to HTTPS, someone can bypass Cloudflare to access the web server directly on port 80.
Can you confirm this issue and do you have an idea how to fix it?
For testing purpose, I would like to have a domain name available from both HTTP and HTTPS (without automatic redirect).
The only workaround I found is to use Cloudflare flexible SSL instead of full SSL, so all requests are going to the port 80 instead of 80 and 443 (depending on when HTTP or HTTPS is used), which is less secure when since HTTPS is not end-to-end encrypted.

@baptx Well, the module is not supported officially anymore. I had some issues related to this directive too, in particular it was giving error 403 sometimes (not sure if it was directly related to this tho, but refreshing it a couple of times used to help)

@p0358 where did you read that it is not supported anymore? I only read that the Debian packages are not supported anymore and that we have to compile from source.

@baptx
https://support.cloudflare.com/hc/en-us/articles/360029696071-Restoring-original-visitor-IPs-Option-2-Installing-mod-remoteip-with-Apache#12345680

Cloudflare no longer updates and supports mod_cloudflare.

@p0358 is there a way to have the same functionality as DenyAllButCloudflare from mod_cloudflare when using mod_remoteip? I would like to block access to users who try to bypass Cloudflare reverse proxy.


check for null/undefined session in addSessionData

Work around incorrect event ordering between SESSION_PTY_EXIT and SESSION_ADD_DATA. See #1628 for more context.
This is ready to merge. Fixes #1628.

friendly ping? :)

@ppot For what its worth, I have tested this PR locally. Ref discussion in #1628, I am unable to run the current version of Hyper. When testing this fix, I no longer have that issue.

Great work btw, @zsol!

No it won't, notice the return in the then double if. I think it's cleaner
this way.
On Fri, Apr 28, 2017, 22:19 Marty Gentillon<EMAIL_ADDRESS>wrote:

@MartyGentillon commented on this pull request.
In lib/actions/sessions.js
https://github.com/zeit/hyper/pull/1698#discussion_r114021694:

     }

   dispatch({

     type: SESSION_PTY_DATA,

     uid,

     data

   });

Why has this been moved outside of the else? Will it happen twice if it
isn't a URL?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zeit/hyper/pull/1698#pullrequestreview-35463875, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEEtMEvixou2mlIl379cuvoQbqfjSWBks5r0lfGgaJpZM4MouOg
.


grpc_interop_java: Remove unnecessary cruft from container

This reduces the container size from 1.4 GB to 640 MB. 129 MB is
jessie, 489 MB jdk, and 22 MB grpc-java. When we swap from jessie to
stretch, we could swap to openjdk:8-jdk-slim-stretch which would make
the entire image 265 MB.
Python was never needed; it was added by mistake in 0589e533.
Pre-downloading gradle artifacts isn't helpful these days, because we
build on a clean machine. Git isn't needed as cp is sufficient. libapr1
has not been required by tcnative for a long time, and we even use
tcnative-boringssl-static since at least 1.0, which also doesn't need
it. The final cleanup is to remove source and downloaded artifacts when
done compiling.

I used tools/interop_matrix/create_matrix_images.py --language java to
rebuild the container and tested the result with docker_image=gcr.io/grpc-testing/grpc_interop_java_oracle8:master tools/interop_matrix/testcases/java__master.
I did this for grpc-java master and 1.0.x.

The interop tests are failing since it looks like builds of the new docker images haven't been uploaded to dockerhub yet (the interop tests don't attempt to build the base docker images themselves). Need to run:
tools/dockerfile/push_testing_images.sh

@apolcyn, it appears I don't have permission:
The push refers to repository [docker.io/grpctesting/grpc_interop_java_10dfd72669eeb60c3d8721e4aa438c3eb32e8c7d]
d8f986fa80a2: Preparing
abcf5ff38a41: Preparing
9abb3997a540: Preparing
denied: requested access to the resource is denied

@ejona86 look at your IMs :P

#18655 for build_docker_go failing.

All the tests are passing.


issue on keezmovies.py

Hi,
It seems not working:
./youtube-dl --all-formats "http://www.keezmovies.com/video/allison-wyte-kelly-the-coed-17-scene-2-440201"
[KeezMovies] 440201: Downloading webpage
ERROR: Unable to extract video URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Thanks for the report. Videos on KeezMovies.com will be fixed in the next version.


Error on query with end_subscription

Sending the request:
debug: Received request from client: {"request_id":624,"type":"subscribe","options":{"collection":"testA","find":{"id":1}}}
debug: Received request from client: {"request_id":624,"type":"end_subscription"}
debug: Error on request 624:
Error: Operation not permitted.
    at feed.eachAsync (/home/josh/Code/horizon/server/src/endpoint/subscribe.js:25:17)
    at userCb (/home/josh/Code/horizon/server/node_modules/rethinkdb/cursor.js:296:31)
    at /home/josh/Code/horizon/server/node_modules/rethinkdb/cursor.js:342:17
    at tryCatcher (/home/josh/Code/horizon/server/node_modules/rethinkdb/node_modules/bluebird/js/main/util.js:26:23)
    at Promise._settlePromiseFromHandler (/home/josh/Code/horizon/server/node_modules/rethinkdb/node_modules/bluebird/js/main/promise.js:507:31)
    at Promise._settlePromiseAt (/home/josh/Code/horizon/server/node_modules/rethinkdb/node_modules/bluebird/js/main/promise.js:581:18)
    at Promise._settlePromises (/home/josh/Code/horizon/server/node_modules/rethinkdb/node_modules/bluebird/js/main/promise.js:697:14)
    at Async._drainQueue (/home/josh/Code/horizon/server/node_modules/rethinkdb/node_modules/bluebird/js/main/async.js:123:16)
    at Async._drainQueues (/home/josh/Code/horizon/server/node_modules/rethinkdb/node_modules/bluebird/js/main/async.js:133:10)
    at Immediate.Async.drainQueues (/home/josh/Code/horizon/server/node_modules/rethinkdb/node_modules/bluebird/js/main/async.js:15:14)
    at runCallback (timers.js:566:20)
    at tryOnImmediate (timers.js:546:5)
    at processImmediate [as _immediateCallback] (timers.js:525:5)
debug: Sending response: {"error":"Operation not permitted.","error_code":-1,"request_id":624}

Doesn't cause an issue with the client

This is just a side effect of the way queries are cancelled in horizon, it's never reported to the user, but it'd be nice to avoid throwing the error and sending a message over the network

I've decided this is dumb, so closing


aui2s: add rtos i2s audio driver module

An I2S audio driver for freeRTOS.
Tested with:
MAX98357A - i2S CLASS D stereo amplifier
Sparkfun I2S Audio Breakout - MAX98357A SF14809

Builds only for freeRTOS.
We didn't use the baresip makefiles. Instead built it by writing component.mk files for esp-idf.

https://github.com/mrxa13/esp32-baresip/tree/master/esp32phone/components/re_mk
https://github.com/mrxa13/esp32-baresip/tree/master/esp32phone/components/rem_mk
https://github.com/mrxa13/esp32-baresip/tree/master/esp32phone/components/baresip_mk

I suggest renaming the module to just i2s, since the I2S bus
is mainly for audio (Inter-IC Sound)

On Fri 08.11.19  23:45, Alfred E. Heggestad wrote:

alfredh commented on this pull request.

int err;

err  = ausrc_register(&ausrc, baresip_ausrcl(),

	      "aui2s", aui2s_src_alloc);

err |= auplay_register(&auplay, baresip_auplayl(),

	       "aui2s", aui2s_play_alloc);

return err;
+}

+static enum I2SOnMask _i2s_on = I2O_NONE;
+
+
+int aui2s_start(uint32_t srate, enum I2SOnMask playrec)
what about channels ? does it support multiple channels ?

No, currently not. Our goal was a mono device. I tried several settings for
the channel_format and finally it worked with I2S_CHANNEL_FMT_ONLY_RIGHT.
Stereo should be also possible. E.g. using two MEMs mics one with channel
select pin on low and the other on high and I2S_CHANNEL_FMT_RIGHT_LEFT.
I could add this setting in the start function, but currently have no setup for
testing.
r,c

--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/alfredh/baresip/pull/848#pullrequestreview-314546158

Oh sorry. There will follow more corrections and I will make an interactive rebase, sure.
r,c

On Fri 08.11.19  23:58, Alfred E. Heggestad wrote:

I suggest renaming the module to just i2s, since the I2S bus
is mainly for audio (Inter-IC Sound)

Ok. I will rename it. But it is only for freeRTOS.
r,c

On Sat 09.11.19  00:12, Alfred E. Heggestad wrote:

alfredh commented on this pull request.

int err;

err = aui2s_start(st->prm.srate, I2O_RECO);

if (err) {

warning("aui2s: could not start ausrc\n");

return NULL;

}

while (st->run) {

size_t i;

for (i = 0; i + DMA_SIZE / 4 <= st->sampc;) {

	size_t n = 0;

	i2s_read(I2S_PORT, st->pcm, DMA_SIZE, &n,

			portMAX_DELAY);

check return value of i2s_read  ?

In the current implementation from esp-idf always ESP_OK is returned. But to be
save, I will add this check.
r,c

On Sat 09.11.19  04:51, Alfred E. Heggestad wrote:

alfredh commented on this pull request.

bool start = _i2s_on == I2O_NONE;

_i2s_on = _i2s_on | playrec;

if (srate * 4 % DMA_SIZE) {

warning("aui2s: srate*4 % DMA_SIZE != 0\n");

return EINVAL;

}

info("%s start with _i2s_on=%d", func, _i2s_on);
if (start) {

i2s_config_t i2s_config = {

	.mode = I2S_MODE_MASTER | I2S_MODE_RX | I2S_MODE_TX,

	.sample_rate =  srate,

	.bits_per_sample = 32,

	.communication_format = I2S_COMM_FORMAT_I2S |

		I2S_COMM_FORMAT_I2S_MSB,

is it possible to use a communication format that is compatible
with the input from baresip core, i.e. signed 16-bit PCM ?
in this case, we can get rid of the sample format conversion...

I tried to find this. But it did'nt work for the used hardware. I think the
SPH0645LM4H MEMs mics need 32 bit samples. And both, the speaker and the mic
are on the same bus.
Note: The shifting is in order to get a reasonable volume. I will add a
comment.
r,c

On Sat 09.11.19  04:55, Alfred E. Heggestad wrote:

alfredh commented on this pull request.

uint32_t v = st->pcm[j];

uint16_t *o = sampv + i + j;

*o = v >> 15;

/* if negative fill with ff */

if (v & 0x80000000)

	*o |= 0xfffe0000;

}
+}

+static void *read_thread(void *arg)
+{

struct ausrc_st *st = arg;
int err;

err = aui2s_start(st->prm.srate, I2O_RECO);

is the I2S started from the thread on purpose ?
or can it be started from the alloc function?
(if aui2s_start fails, the thread will just exit but the program will continue
as if everything is working).

You are right. It is better to try first i2s_start, then create the thread.
r,c

Ok. Did rework due to Alfred's comments. But we have to test it again on our esp32 phone.

Ok, now we tested the module in the project https://github.com/mrxa13/esp32-baresip. Works like before the rework.

great, looks really good now.
is it okay to add you as the maintainer of the new i2s module?
https://github.com/alfredh/baresip/wiki/Modules

Yes, of course. Thanks!
r,chris
On Mon 11.11.19  05:38, Alfred E. Heggestad wrote:

great, looks really good now.
is it okay to add you as the maintainer of the new i2s module?
https://github.com/alfredh/baresip/wiki/Modules
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/alfredh/baresip/pull/848#issuecomment-552448954

On Mon 11.11.19  05:38, Alfred E. Heggestad wrote:

great, looks really good now.
is it okay to add you as the maintainer of the new i2s module?
https://github.com/alfredh/baresip/wiki/Modules

Hi Alfred,
thank you for the merge! But sorry, I made a mistake in the commit message.
Tested with:
MAX98357A - i2S CLASS D stereo amplifier
Sparkfun I2S Audio Breakout - MAX98357A SF14809

This is the amplifier twice. No hint about the MEMs Mics. Maybe we should add
a short description in the head of i2s.h.
Tested with:
Sparkfun I2S Audio Breakout - MAX98357A SF14809 - CLASS D stereo amplifier
SPH0645LM4H - Mems i2s stereo mic
r,c

it is probably best to add a README.md file in modules/i2s
or add a doxygen header like this:
https://github.com/alfredh/baresip/blob/master/modules/ice/ice.c#L14


Add inshellisense as plugin for powershell not working

Describe the bug
Run the command is init powershell >> $profile for PowerShell didn't work as expected, when I started the new PowerShell, I saw plenty of bugs repeatedly:
A bug sample:

The term '' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At C:\Users\test\WindowsPowerShell\Microsoft.PowerShell_profile.ps1:3 char:1
+

~

CategoryInfo          : ObjectNotFound: (:String) [], CommandNotFoundException
FullyQualifiedErrorId : CommandNotFoundException

To Reproduce
Steps to reproduce the behavior:

Go to PowerShell
Run the command is init powershell >> $profile
Try to open a new PowerShell
See error and inshellisense will not get activated

Expected behavior
Expect the script to run normally without error and inshellisense to get activated
Environment

OS: Window 10
Output of is --version:  0.0.1-rc.16
Nodejs Version:  16.20.2
Shell: PowerShell
Shell Version: [e.g. 5.2.21] 5.1.19041.4894

Additional context
inshellisense still works normally when I type is in any PowerShell, just got the error when I tried to run that command above to start automatically inshellisense  whenever I open a new PowerShell

I have a similar problem but with bash standalone, and bash in vscode integrated terminal. I did is init bash >> ~/.bashrc and I can see the script it put into ~.bashrc but when opening a new vscode integrated bash terminal or a standalone bash, the inshellisense isn't autostarted. No errors though.

So basically we just need to update the command and it will work? 😄


【代码迁移】任务50：Coneheads: Hierarchy Aware Attention

penumbral代码运行结果截图

umbral代码运行结果截图

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.zxlzd seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Replace yarn with pnpm and upgrade typescript version

😀
Changelog

upgrade typescript to 4.9
replace yarn with pnpm
add auth package
transpile packages

My god its finally green


[Tooltip] Add PopperComponent prop

[x] I have followed (at least) the PR section of the contributing guide.

Allows overriding of Popper in Tooltip like Autocomplete.
Use Case: Using a native popper that displays tooltips in native windows. (I don't know how this part works yet :P)  cc @oliviertassinari
Also refactored the tests cc @eps1lon

Use Case: Using a native popper that displays tooltips in native windows. (I don't know how this part works yet :P)

@joshwooding It would be great to have an example, the last time we talk about it, I was leaning toward removing PopperComponent, having no use case for it.

Also, we have frozen all-new feature addition on v4. It would need to be treated as a bug fix.

@olivertassinari I know we were thinking of this but has it been confirmed publicly. I haven’t seen anything but might have missed it?

@joshwooding There are a couple of open pull requests that are on hold because they introduce new features. I think that we need to this feature freeze on the core free us time to work on the v5 depreciation warnings.


Update design systemet's components

Describe the bug
Dropdown and Input components are updated with requested features. Use them.
Solution
Update this lib to the latest.

Could you elaborate some more on this issue?

Update the @statisticsnorway/ssb-component-library to the latest, and try to use dropdown component and input component wherever it possible in the subset-client.
When I tried to use use dropdown last time, i could not control disabled options.
Input for search was not usefull because of the lack of implemented use of keyboard. Now it done. I think it still does not highlight the search string. Så I suppose we will wait with search, but other instances of input/textarea could be revisited.


Translation shortcuts are incorrect, transifex strings have leading 'space' character

Looking into how #129 would affect shortcuts in other languages, I noticed that many shortcuts are incorrect in other languages because the Transifex translation string contains a leading space character. I believe that a linter should be put in place to prevent this from happening in the future,
Affected Translations:  bitcoin_ta.ts, bitcoin_ar.ts<EMAIL_ADDRESS>bitcoin_sr.ts, bitcoin_ml.ts, bitcoin_es_CL.ts,       bitcoin_nl.ts,  bitcoin_ms.ts, bitcoin_lt.ts, bitcoin_es.ts, bitcoin_th.ts bitcoin_th.ts, bitcoin_te.ts, bitcoin_nb.ts,  bitcoin_id.ts, bitcoin_hi.ts, bitcoin_fil.ts, bitcoin_el.ts, bitcoin_bg.ts
To Replicate:

In order to grep the *.ts files we need to set them executable. Under qt/src/locale, run:

chmod +x ./*.ts

now grep for a matching & followed by a 'space' character

grep -n "& " ./*.ts

Optional: To get the files and number of occurrences (Sorted):

grep -c "& " ./*.ts | grep -v ':0$' | sort -n -k2 -t:

Below represents the output you will see. Here is a pastebin link of the full output. In this example, the shortcut for this command will be space instead of G. This is incorrect and needs to be addressed.

 <translation>& Gastar cambio no confirmado</translation>

Using the above example: If we take a look at the transifex page we find that the string does in fact contain a leading space character when it is not supposed to.

true open

@thantkyaw1 open


Added PauseMenu and fixed some map visuals

This PR contains the following added features and bugfixes:
Features:

Added Pause Menu for every map currently added. Open it by pressing the escape button.
Added a "GameWindow" class that I think will work as our "GameManager" class to keep track of the game status.

Fixes:

Fixed so that maps are positioned at the same place on the screen.

I made sure to run the tests and build multiple times on both my laptop and PC and there were no issues.

Map scenes are now also placed inside a "Maps" folder for better structure.


Could you expand on how to send image updates / where to specify where to get new images?

Hi, this is exactly what I was looking for to display images on the eInk display. I got the firmware flashed to the esp32 and the wifi connected, but I am unable to figure out where i can specify the location the esp32 should get the new image, or how the new image should be send to it. Could you please elaborate?
p.s.: i could not access the specified discord channel, that's why I'm trying it here.
Thanks so much.
Ben

Hi @ben68165,
You can use the send.py file (https://github.com/codetiger/ePaper-display-using-ESP32/blob/main/send.py) for sending the images through socket connection. In my case, I am using a RPi home server to send the images every few minutes.


Is there a way to execute a non driver.js function when the Done button is clicked in a tour?

I need to do more and execute other functions on the page when the "Done" button is clicked on the last pop-over Tour. I added a className to the last pop-over, I thought that helps to find and target the Done button on the last pop-over but it doesn't look all steps are available in DOM when the page is loaded.

You can do that using onReset

Thanks, did you mean something like this?
element: '#masonry-drop-down',
	popover: {
    	className: 'last-step-popover',
		title: 'Filter',
      	description: 'This is how you can filter the news by Category.',
      	position: 'right'
    },
	onReset: () => {
		console.log('closed now');
	}

I tried this and it didn't do anything for me.

Never mind, I think I called it in a wrong location, this worked for me:
const driver = new Driver({
	onReset: () => {
		console.log('closed now');
	}
});


netdev: Fix netdev->reconfigure_seq memory leak.

netdev: Fix netdev->reconfigure_seq memory leak after failed to construct netdev.
Signed-off-by: Huanle Han<EMAIL_ADDRESS>

Thanks for the patch.  I applied this to master and branch-2.6.


Update azure-defender-storage-configure.md

The release state for Azure Files and ADLS Gen2 should be GA instead of the current preview

@HasanAboShally-MSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


macOS: update realtime lib for Sierra

The latest version of macOS, Sierra, defines the functions clock_gettime and clock_getres which were previously missing. This PR adds support for Sierra while preserving compatibility with El Capitan using a simple heuristic to either use the system provided functions or define them locally.
Reported-by: Marie-Pier Lessard @mplessard

test osx please

test osx please

Did a red-green test. Fails on master, but it compiles successfully with this branch. On Sierra 10.12.3
:shipit:

Thanks @jpcaissy!
I forgot to brew upgrade on the server after the upgrade to Sierra, so Jenkins...
test osx please

The failing test about openssl is unrelated to this PR and will be fixed as soon as we update the XCode tools on the test server.


[onnx] support ONNXIFI Facebook Extension and update host dependencies

What does your PR fix?
1. Feature for ONNXIFI
Pytorch project is using ONNX with this project's source code. (foxi_loader)
After survey, I found there are differences in ONNX's public headers.
Making it to separate port requires overwrite of those files.
For #17199, replace related source files before build.
Then install the project's LICENSE (MIT) file with ONNX copyright.
By this way, CMake target originally named foxi_* will be built as onnxifi_*.
2. Remove Host dependencies (help/suggestion needed)
In #18073, I used python3 for host dependencies to ensure pybind11 and find_package(Python3 COMPONENTS Development).
#17199 uses NumPy (when find_package(Python3)).
Seems like it should be prepared not by port but from environment...
Which triplets are supported/not supported? Have you updated the CI baseline?
No changes in triplet support.
Does your PR follow the maintainer guide?
Yes

The new feature creats onnxifi.dll and some others. This requires the port to support dynamic build.
Finding a way to support both static/dynamic protobuf

Please ping me if this PR is ready for review or you need some help.

I just read the guideline again, and I  Seems like using these as a feature doesn't fit the guideline... I had better remove foxi part.
I will find another workaround for #17199.

Let me take a look at why dynamic build doesn't work.

In the official readme:

If you are building ONNX from source, it is recommended that you also build Protobuf locally as a static library. The version distributed with conda-forge is a DLL, but ONNX expects it to be a static library. Building protobuf locally also lets you control the verison of protobuf.

Since vcpkg is currently unable to establish a cross-triplet dependency, I think it's good now.

Python3
It seems like we have 3 use-cases for PYTHON3.
Since CMake 3.12, find_package(PythonInterp) and find_package(PythonLibs) are deprecated.
The project is using Python3, so I replaced them with find_package(Python3)
Case 1. Python Interpreter
Mostly port that uses PYTHON_EXECUTABLE are under this category.
The python interpreter is required to invoke their scripts for build, codegen, etc.
Most ports are handling this with acquiring PYTHON3.
vcpkg_find_acquire_program(PYTHON3)

vcpkg_cmake_configure(
    SOURCE_PATH "${SOURCE_PATH}"
    OPTIONS
        -DPython3_EXECUTABLE=${PYTHON3}
        # ...
)

If there is a port which have to solve find_package(pybind11 CONFIG), we can use "pybind11" in dependencies.
Case 2. Python Development
Currently, in Windows environment, vcpkg_find_acquire_program(PYTHON3) setups python executable, but not Python header files.
So, if the CMakeLists.txt wants Development component, it will fail.
find_package(Python3 COMPONENTS Development)

Tagging requires:discussion to resolve the python mess. :/


Unable to reproduce the accuracy

Hi, I got only ~36 accuracy in VQACP2 val set.
Here is the log:
eval_epoch.epoch: 21
[S 2019-08-22 01:47:37] ...trap/engines/engine.py.120: eval_epoch.loss: 9.32
[S 2019-08-22 01:47:37] ...trap/engines/engine.py.120: eval_epoch.loss_mm_q: 4.51
[S 2019-08-22 01:47:37] ...trap/engines/engine.py.120: eval_epoch.loss_q: 4.81
[S 2019-08-22 01:47:37] ...trap/engines/engine.py.120: eval_epoch.accuracy_top1: 36.54
[S 2019-08-22 01:47:37] ...trap/engines/engine.py.120: eval_epoch.accuracy_top5: 73.24
[S 2019-08-22 01:47:37] ...trap/engines/engine.py.120: eval_epoch.accuracy_rubi_top1: 22.48
[S 2019-08-22 01:47:37] ...trap/engines/engine.py.120: eval_epoch.accuracy_rubi_top5: 73.94
[S 2019-08-22 01:47:37] ...trap/engines/engine.py.120: eval_epoch.accuracy_q_top1: 6.13
[S 2019-08-22 01:47:37] ...trap/engines/engine.py.120: eval_epoch.accuracy_q_top5: 47.23

What would be a good way to debug it? Maybe I should compare per type accuracies against the pre-trained model?
The baseline model isn't training well either:
[S 2019-08-22 13:44:22] ...trap/engines/engine.py.120: eval_epoch.epoch: 11
[S 2019-08-22 13:44:22] ...trap/engines/engine.py.120: eval_epoch.loss: 4.28
[S 2019-08-22 13:44:22] ...trap/engines/engine.py.120: eval_epoch.accuracy_top1: 21.74
[S 2019-08-22 13:44:22] ...trap/engines/engine.py.120: eval_epoch.accuracy_top5: 70.43

I just tested with pre-trained weights, and they only gave 35.36. I wonder if something happened with my features, I downloaded them from this link.
[S 2019-08-22 17:12:43] ...trap/engines/engine.py.101: eval_epoch.epoch: 0
[S 2019-08-22 17:12:43] ...trap/engines/engine.py.101: eval_epoch.loss: 9.30
[S 2019-08-22 17:12:43] ...trap/engines/engine.py.101: eval_epoch.loss_mm_q: 4.56
[S 2019-08-22 17:12:43] ...trap/engines/engine.py.101: eval_epoch.loss_q: 4.75
[S 2019-08-22 17:12:43] ...trap/engines/engine.py.101: eval_epoch.accuracy_top1: 35.36
[S 2019-08-22 17:12:43] ...trap/engines/engine.py.101: eval_epoch.accuracy_top5: 73.72
[S 2019-08-22 17:12:43] ...trap/engines/engine.py.101: eval_epoch.accuracy_rubi_top1: 23.05
[S 2019-08-22 17:12:43] ...trap/engines/engine.py.101: eval_epoch.accuracy_rubi_top5: 74.16
[S 2019-08-22 17:12:43] ...trap/engines/engine.py.101: eval_epoch.accuracy_q_top1: 6.27
[S 2019-08-22 17:12:43] ...trap/engines/engine.py.101: eval_epoch.accuracy_q_top5: 47.58

Hi @erobic ,
You should not look at the top1 accuracy to evaluate the model. We use the 'open ended' accuracy. It is not displayed, because it takes time to compute, so it is computed in the background.
You can display it using the compare module:
python -m rubi.compare_vqacp2_rubi -d your/exp/dir

Got it, thank you!


feat(test wallet): Introduces the test-wallet command to the CLI PE-1809

NOTE: Should add some integration/end-to-end tests.

NOTE: We need to approve, merge, and alpha release https://github.com/ardriveapp/ardrive-core-js/pull/205 to test this without building from source.

Why is this one closed? ;-;


Support for IAM Roles for self managed cluster with Kiam

Hey Team,
I am working with IAM Roles to access the services of AWS.  I am building a self-managed cluster on AWS with help of kops tool. All my app deployments uses IAM roles to access AWS Services using Kiam service.
I have added AWS Role in deployment configuration as follows.
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  labels:
    name: kubernetes-external-secrets
  name: kubernetes-external-secrets
  namespace: kubernetes-external-secrets
spec:
  replicas: 1
  selector:
    matchLabels:
      name: kubernetes-external-secrets
  template:
    metadata:
      labels:
        name: kubernetes-external-secrets
        service: kubernetes-external-secrets
      annotations:
        iam.amazonaws.com/role: arn:aws:iam::12312312312:role/Development-Secrets-Manager-Role
    spec:
      serviceAccountName: kubernetes-external-secrets-service-account
      containers:
        - image: "godaddy/kubernetes-external-secrets:1.2.2"
          imagePullPolicy: Always
          name: kubernetes-external-secrets
          env:
            - name: AWS_REGION
              value: ap-southeast-1

It doesn't work. It is throws errors like unable to find the credentials. But, It works with if I provide AWS_ACCESS_KEY_ID & AWS_SECRET_KEYs as environment vars to deployment.
Please can support or help if I am doing something wrong.

We also use kiam and successfully deployed the service in our cluster. Not sure how I can help you, but let me know if you need some information. Your role seems wrong to me, we have something like iam.amazonaws.com/role=dev-main-kubernetes-external-secrets

Seems stale, please reopen if this is still an issue.
Similar to Sytten we also use kiam, with a similar role (just using the role name in the annotation), but I'm guessing kiam supports the full arn as well depending on configuration.


More Serialization changes

More stuff while working on https://github.com/pydantic/pydantic/pull/4978

Codecov Report

Merging #374 (30c3eef) into main (7d7ca9f) will decrease coverage by 0.06%.
The diff coverage is 91.71%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #374      +/-   ##
==========================================
- Coverage   95.37%   95.32%   -0.06%
==========================================
  Files          90       90
  Lines       10236    10372     +136
  Branches        4        4
==========================================
+ Hits         9763     9887     +124
- Misses        473      484      +11
- Partials        0        1       +1

Impacted Files
Coverage Δ

src/serializers/shared.rs
88.80% <ø> (ø)

src/serializers/type_serializers/model.rs
86.66% <0.00%> (ø)

src/serializers/ob_type.rs
86.22% <50.00%> (ø)

src/serializers/type_serializers/format.rs
91.66% <86.36%> (-5.35%)
:arrow_down:

pydantic_core/core_schema.py
99.26% <92.00%> (-0.74%)
:arrow_down:

src/serializers/type_serializers/function.rs
90.44% <98.41%> (+4.17%)
:arrow_up:

src/serializers/extra.rs
95.26% <100.00%> (+0.26%)
:arrow_up:

src/serializers/infer.rs
96.62% <100.00%> (ø)

src/serializers/type_serializers/literal.rs
69.56% <0.00%> (ø)

... and 2 more

Continue to review full report at Codecov.

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