runtime: apparent false-positive race report for a buffered channel after CL 220419

I got the following race report on CL 258758.
(https://storage.googleapis.com/go-build-log/04d98a9c/linux-amd64-race_596e00e8.log)

--- FAIL: TestScript (0.11s)
    --- FAIL: TestScript/mod_retract (0.34s)
        script_test.go:211:
            # Populate go.sum. (0.087s)
            # 'go list pkg' does not report an error when a retracted version is used. (0.069s)
            # Nor does 'go build'. (0.000s)
            # Neither 'go list' nor 'go build' should download go.mod from the version
            # that would list retractions. (0.000s)
            # Importing a package from a module with a retracted latest version will
            # select the latest non-retracted version. (0.175s)
            > go get -d ./use_self_prev
            [stderr]
            go: downloading example.com v1.0.0
            go: downloading example.com/retract/self/prev v1.1.0
            go: downloading example.com/retract v1.1.0
            ==================
            WARNING: DATA RACE
            Read at 0x00c0002c0ce8 by main goroutine:
              cmd/go/internal/modget.reportRetractions()
                  /workdir/go/src/cmd/go/internal/modget/get.go:550 +0x224
              cmd/go/internal/modget.runGet()
                  /workdir/go/src/cmd/go/internal/modget/get.go:490 +0x931
              main.main()
                  /workdir/go/src/cmd/go/main.go:195 +0xb94

            Previous write at 0x00c0002c0ce8 by goroutine 76:
              cmd/go/internal/modload.addRetraction()
                  /workdir/go/src/cmd/go/internal/modload/build.go:132 +0x424
              cmd/go/internal/modload.ListModules.func1.1()
                  /workdir/go/src/cmd/go/internal/modload/list.go:40 +0x84

            Goroutine 76 (finished) created at:
              cmd/go/internal/modload.ListModules.func1()
                  /workdir/go/src/cmd/go/internal/modload/list.go:32 +0xe4
              cmd/go/internal/modload.ListModules()
                  /workdir/go/src/cmd/go/internal/modload/list.go:46 +0x1fa
              cmd/go/internal/modget.reportRetractions()
                  /workdir/go/src/cmd/go/internal/modget/get.go:547 +0x184
              cmd/go/internal/modget.runGet()
                  /workdir/go/src/cmd/go/internal/modget/get.go:490 +0x931
              main.main()
                  /workdir/go/src/cmd/go/main.go:195 +0xb94
            ==================
            ==================
            WARNING: DATA RACE
            Read at 0x00c000058ac0 by main goroutine:
              cmd/go/internal/modget.reportRetractions()
                  /workdir/go/src/cmd/go/internal/modget/get.go:556 +0x2b9
              cmd/go/internal/modget.runGet()
                  /workdir/go/src/cmd/go/internal/modget/get.go:490 +0x931
              main.main()
                  /workdir/go/src/cmd/go/main.go:195 +0xb94

            Previous write at 0x00c000058ac0 by goroutine 76:
              cmd/go/internal/modload.checkRetractions()
                  /workdir/go/src/cmd/go/internal/modload/modfile.go:163 +0x4a4
              cmd/go/internal/modload.addRetraction()
                  /workdir/go/src/cmd/go/internal/modload/build.go:126 +0x104
              cmd/go/internal/modload.ListModules.func1.1()
                  /workdir/go/src/cmd/go/internal/modload/list.go:40 +0x84

            Goroutine 76 (finished) created at:
              cmd/go/internal/modload.ListModules.func1()
                  /workdir/go/src/cmd/go/internal/modload/list.go:32 +0xe4
              cmd/go/internal/modload.ListModules()
                  /workdir/go/src/cmd/go/internal/modload/list.go:46 +0x1fa
              cmd/go/internal/modget.reportRetractions()
                  /workdir/go/src/cmd/go/internal/modget/get.go:547 +0x184
              cmd/go/internal/modget.runGet()
                  /workdir/go/src/cmd/go/internal/modget/get.go:490 +0x931
              main.main()
                  /workdir/go/src/cmd/go/main.go:195 +0xb94
            ==================
            go: warning<EMAIL_ADDRESS>is retracted: bad
            go: run 'go get example.com/retract@latest' to switch to the latest unretracted version
            Found 2 data race(s)
            [exit status 66]
            FAIL: testdata/script/mod_retract.txt:23: unexpected command failure

FAIL
FAIL	cmd/go	151.822s

The reported race is for a function that uses an N-buffered channel as a semaphore. The function sends to the channel (from the main goroutine) to acquire a semaphore token, and receives from the channel (in a background goroutine) to release the token. Before the function returns, it sends N additional tokens to the channel to ensure that all of the background goroutines have finished.
I believe that this pattern explicitly covered by the memory model:

The kth receive on a channel with capacity C happens before the k+Cth send from that channel completes.

This is the first race we've seen reported for this code, and the timing w.r.t. CL 220419, which changes the semantics of channels under the race detector, seems suspicious to me.

CC @dfava @ianlancetaylor @dvyukov @aclements

package main

const C = 2
const K = 2

var ch = make(chan int, C)

var x int

func main() {
	for i := 0; i < K; i++ {
		ch <- 0
		go func() {
			x++
			<-ch
		}()
	}

	for i := cap(ch); i > 0; i-- {
		ch <- 0
	}
	println(x)
}

This reports race with all versions of Go that I have on my machine (since Go 1.7). So, not the CL above. (I think this resembles your case.)

@cherrymui, your example is spawning N goroutines that all mutate the same global variable, whereas the goroutines in the actual code in question mutate disjoint variables.
The race reported for your example should be a write–write race, whereas the newly-observed false positive is reported as a read-write race.

I've spent some time analyzing the effect of CL 220419 at the time and I concluded it's perfectly in line with MM.

I believe that this pattern explicitly covered by the memory model:
The kth receive on a channel with capacity C happens before the k+Cth send from that channel completes.

This is true, but this relation does not seem to be transitive. Namely, if there are more than C operations, kth receive HB k+Cth send, but it does not HB 2*k+Cth send. So when the code does:
	for n := cap(sem); n > 0; n-- {
		sem <- token{}
	}

It only synchronizes with the completion of the last cap(sem) operations, but not with all the previous ones.

@bcmills you're right. Sorry about the noise.

@dvyukov If I understand you correctly, that sounds like a bug in the memory model.  Perhaps the memory model ought to say that the kth receive on a channel with capacity C happens before the k+Cth+N send from the channel completes, where N is any non-negative integer.  There is no necessary relationship between sends, of course.  But with capacity C, a receive may happen concurrently with up to (but not including) C sends, but it happens before C sends complete, or C+1 sends complete, or C+2 sends complete, etc.

@dvyukov, in this case we should be able to prove that all previous sends happen before the final receive by induction, because all k sends on the channel occur in the same goroutine.

The kth receive happens before the k+Cth send because “[t]he kth receive on a channel with capacity C happens before the k+Cth send from that channel completes.”

In this program all sends are within a single goroutine, so the k+Cth send happens before the i+Cth send for all i ≥ k because “[w]ithin a single goroutine, the happens-before order is the order expressed by the program.”

“Happens before” is transitive, so in this program the kth receive happens before the i+Cth send for all i ≥ k.

In this program there are exactly N receives and exactly N+C sends, so each receive happens before the last send.

@ianlancetaylor, I think the terms “kth receive” and “k+Cth send” are confusing and perhaps misleading here. Identifying the “kth receive” requires a total order among the receives, but the point of HB is that we may only have a partial order.
A more precise definition might be more like: “If k sends on a channel with capacity C happen before C sends on the channel, and those C sends happen before an event e, then the receives of the initial k values also happen before e.”

@dvyukov If I understand you correctly, that sounds like a bug in the memory model. Perhaps the memory model ought to say that the kth receive on a channel with capacity C happens before the k+Cth+N send from the channel completes, where N is any non-negative integer. There is no necessary relationship between sends, of course. But with capacity C, a receive may happen concurrently with up to (but not including) C sends, but it happens before C sends complete, or C+1 sends complete, or C+2 sends complete, etc.

Good question.
The examples we had in CL 258758, did not look like something we need to support and race detector indeed produced false negatives on these examples. This "wait for all" example did not come up at the time.

happens before the k+Cth+N

This feels a bit fishy to me. It seems we are trying to compensate for broken transitivity, where fixing transitivity would cleaner.
I am thinking of:
-The kth receive on a channel with capacity C happens before the k+Cth send from that channel completes.
-This rule generalizes the previous rule to buffered channels.
+The kth receive completion on a buffered channel with capacity C happens before the k+Cth send.
It does not generalize the rule for unbuffered channels anymore: the use of send/recv and send completion/recv completion is swapped. But breaking generalization by introducing new custom rules also feels fishy to me :)
However, my understanding always was that weird splitting to operation and operation completion was introduced specifically for unbuffered channels to break HB cycle. Without that splitting the current rules would work perfectly for buffered channels (support the code in question).

@dvyukov, in this case we should be able to prove that all previous receives happen before the final send by induction, because all sends on the channel occur in the same goroutine.

The kth receive happens before the k+Cth send because “[t]he kth receive on a channel with capacity C happens before the k+Cth send from that channel completes.”
In this program all sends are within a single goroutine, so the k+Cth send happens before the i+Cth send for all i ≥ k because “[w]ithin a single goroutine, the happens-before order is the order expressed by the program.”
“Happens before” is transitive, so in this program the kth receive happens before the i+Cth send for all i ≥ k.
In this program there are exactly N+C sends and exactly N receives, so each receive happens before the last send.

Good point.
With the changes Ian or me proposed above it would work without relying on the fact that all sends happen in the same goroutine, the channel itself would provide the necessary synchronization. But if we take into account that all sends happen in the same goroutine, then it seems that it should work with the current rules as well:

Here channel capacity is 2 and all accesses d0-3 happen before use.
@dfava where is the flaw now?
Also now I wonder if we are dealing with one issue or two :) There seems to be a bug in race detector implementation, but also do we still want to fix the memory model so that similar synchronization is provided even when sends do not happen in the same goroutine? For ListModules is seems inherent that all sends happen in the same goroutine (?).

In this case it actually is important that the initial sends happen before the final loop, so I would be wary about strengthening the model too much.
If the initial sends were moved into the background goroutines, it would be possible for that final channel-fill loop to execute before the background goroutines, causing them to leak (blocked on the sends) instead of execute. That wouldn't be a data race, but only because the mutations would no longer happen at all!

@dvyukov, also note that the phrasing I suggested in https://github.com/golang/go/issues/42598#issuecomment-727416726 does continue to generalize to the unbuffered case.

Given that we're coming up against the beta, should we revert CL 220419?

Yes, I think we need to rollback for now and add this pattern as a new race test. False positives are much worse than false negatives.

I sent a revert CL: https://golang.org/cl/271946.

Sorry I wasn't receiving emails about this thread until the change was reverted just now.  I haven't had a chance to read it, but I will soon and can comment.

For what it's worth, the revert CL is not yet committed.  A simple roll forward fix is still possible, but it would have to happen very very soon.

Is the failure deterministic or intermittent?  I'm not able to reproduce it on commit 5e58ae43bedeae5964e668755049088938320740.
Seems like there has been some refactoring around modules.  I can't find "reportRetractions" in the source tree.  Seems like the function has recently been removed and it used to live in src/cmd/go/internal/modget/get.go.  Is that correct?
@bcmills , is the race still reported after this recent refactoring?
(I haven't had a chance to fully digest the discussion above, just going piece by piece from the beginning.  I agree on reverting the CL so we can drill down into this carefully.  I doubt there is a "bug" in memory model specification; more likely the bug is on my code, or on the code where the race is being reported)

Also, I vaguely recall an earlier discussion on how the race detector handles channel where the element size is zero differently...  Perhaps that special treatment maters now.

Also, I vaguely recall an earlier discussion on how the race detector treats differently channels whose elements are size zero... Perhaps that special treatment maters now.

Good point!
Here is the address we use for synchronization:
// chanbuf(c, i) is pointer to the i'th slot in the buffer.
func chanbuf(c *hchan, i uint) unsafe.Pointer {
	return add(c.buf, uintptr(i)*uintptr(c.elemsize))
}

I assume for struct{} it's always the same location for all elements.
I guess racereleaseacquire fix does not work for such case.
I think we could do:
if c.elemsize != 0 {
  racereleaseacquire(qp)
} else {
  raceacquire(qp)
  racerelease(qp)
}

I agree, we could put a check for elemsize.  It's nice and simple, and it doesn't force us to revisit the assumptions made about channels of elemsize==0 right now.
I'm happy that the memory model specification is fine as it is, and I also see that the code in ListModules is correct.
I also think that my patch is not wrong :) though it did expose the issue with elemsize==0, and, in hindsight, I should have accounted for that when testing.
I'll spend a bit of time thinking about what exactly happens when elemsize==0 interacts with racereleaseacquire().

@dfava

Is the failure intermittent (or "somewhat deterministic")? I'm not able to reproduce it on commit 5e58ae4.

The failure is intermittent.
There have so far been at least three failures in TryBots and one on the build dashboard (2020-11-14T17:24:37-92c732e/linux-amd64-race).

I'll spend a bit of time thinking about what exactly happens when elemsize==0 interacts with racereleaseacquire().

My understanding is that previously HB was completely cumulative, so the fact that the same address was shared between unrelated elements did not matter (we only get more false negatives). But racereleaseacquire is not cumulative, it always drops everything that was there before, so sharing breaks.

I'll spend a bit of time thinking about what exactly happens when elemsize==0 interacts with racereleaseacquire().

My understanding is that previously HB was completely cumulative, so the fact that the same address was shared between unrelated elements did not matter (we only get more false negatives). But racereleaseacquire is not cumulative, it always drops everything that was there before, so sharing breaks.

You are absolutely right.  Here is in a bit more detail, perhaps it's nice to document.
The code essentially does the following, where c is a channel of size 2:
T0                T1        T2
c <-         |           |
go T1        |  wr X     |
c <-         |  <- c     |
go T2        |           |  wr Y
             |           |  <- c
c <-         |           |
c <-         |           |

I'll represent each entry in the channel's buffer as B0 and B1.
Here is a picture of the state of the race detector after T1 and T2 write to variable X and Y:
B0      B1      T0             T1      T2
{}      {}      {}             {wr X}  {wr Y}  => then T1 receives from c
{wr X}  {}      {}             {wr X}  {wr Y}  => then T2 receives from c
{wr X}  {wr Y}  {}             {wr X}  {wr Y}  => then T0 sends onto c
{}      {wr Y}  {wr X}         {wr X}  {wr Y}  => then T0 sends onto c
{}      {wr X}  {wr X, wr Y}   {wr X}  {wr Y}

The end state is, T0 is properly synchronized with T1 and T2:  T0 "knows" of both T1's and T2's writes to X and Y.
However, we currently model channels whose elmsize==0 with one buffer entry, so only B0 and no B1.  The picture then looks like:
B0        T0             T1      T2
{}        {}             {wr X}  {wr Y}        => then T1 receives from c
{wr X}    {}             {wr X}  {wr Y}        => then T2 receives from c
{wr Y}    {}             {wr X}  {wr X, wr Y}  => then T0 sends onto c
{}        {wr Y}         {wr X}  {wr Y, wr Y}  => then T0 sends onto c
{wr Y}    {wr Y}         {wr X}  {wr Y, wr Y}

In the end state now, T0 is unaware of one of the writes, so a race is flagged.

It seems like we are able to keep CL 220419 and add the check that Dmitry suggested.
Let me know if you would like me to do it, or if you prefer to do it yourselves (since you are probably faster at it than me, given your familiarity with the flow and my lack of access to buildbot and etc).  I'm happy either way!

Whoever is doing it, just please add some comments/references and tests ;)

Whoever is doing it, just please add some comments/references and tests ;)

:) Is it possible to force a specific scheduling of goroutines so we hit the scenario exactly?  Maybe using internals/unsafe/something?

(I'm working on a patch)

I submitted a patch which included the check for elemsize, documentation, and a test.
The test fails on top-of-tree (as expected), and passes on the branch with the patch (as expected).


Copy Data from blob to Azure table does not copy columns that have null values

I have a CSV file with two columns that have all empty values. When I copy the blob to a new Azure table the two columns are not visible in the Azure table, even though I have defined explicit mappings. How can I get those columns show in Azure table (with null values)?

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

ID: b8ab834e-461c-b8fc-2056-7d2bfd152630
Version Independent ID: 805724e0-968d-2854-e0a9-6eeae2df9fa2
Content: Copy data to and from Azure Table storage by using Data Factory
Content Source: articles/data-factory/connector-azure-table-storage.md
Service: data-factory
GitHub Login: @linda33wj
Microsoft Alias: jingwang

@h28669 Thanks for the feedback! What are the schema mappings that you are using? Would you please share that with including types?

@h28669 Have you found a solution? This question is a great candidate for the forums MSDN or StackOverflow just make sure to include more details about your specific scenario so the developer community can help. We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

I found this is the standard behavior of Azure Table service. Properties with null values are not stored.
https://docs.microsoft.com/en-us/rest/api/storageservices/understanding-the-table-service-data-model
"The Table service does not persist null values for properties. When querying entities, the above property types are all non-nullable. When writing entities, the above property types are all nullable, and any property with a null value is handled as if the payload did not contain that property."


Razor editor very slow on blazor project

Using razor editor result in a severe slow down of visual studio with an increasing memory occupation. It also affect auto formatting of razor code.
I need to close visual studio and restart to solve.
My vs version is 17.5.4

@Gambero81 , Visual Studio 17.6 released recently with a number of performance improvements in Razor editing.  Can you update and see if the issue still exists?  If so, can you open a VS Feedback ticket from the "Help->Send Feedback->Report a Problem" menu option built into Visual Studio? If you select "Record your actions to reproduce the issue", it gathers a lot of information that helps us investigate the cause of the issue.


ESP32 hci controller timeout in host (IDFGH-11525)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
v5.1.1
Espressif SoC revision.
ESP32-D0WD-V3 (revision v3.1)
Operating System used.
Linux
How did you build your project?
Command line with idf.py
If you are using Windows, please specify command line type.
None
Development Kit.
Custom Board
Power Supply used.
External 3.3V
What is the expected behavior?
ESP32 run hci controller example(BTDM mode), and communicate with BlueZ in Linux via 4 wried UART.
What is the actual behavior?
The ESP32 successfully attach with btattach -B /dev/ttymxc0 -S 921600 -P h4 and work fine in the beginning.
But after running a few minutes or hours, it returns Can't read local name on hci0: Connection timed out (110) with hciconfig -a.

Steps to reproduce.

ESP32 run the hci controller example in idf-v5.1.1.
Linux host attach the ESP32 btattach -B /dev/ttymxc0 -S 921600 -P h4 &, and set hci up hciconfig hci0 up.
Linux host setup a GATT service and characteristic, and start adv using bluetoothctl (here use the BLE).
Linux host try query a BT device's name hcitool name <MAC>, and connect its NAP server bt-network -c <MAC> nap & per five seconds (here use the Classic BT).
Use another ESP device(in my case, ESP32S3) run the gatt client example in idf-v5.1.1, and try connect the Linux Host and write/read the GATT characteristic.

Debug Logs.

Here's the ESP32 serial output:
ESP32 Serial Debug 2023-11-23 121310.txt
Here's the Linux host btmon output:
btmon_20231123_121234.txt
Here's the Linux host hciconfig -a output:
hciconfig_20231123_121253.txt
Actually, i had created an issue in esp-hosted.
But i found the same problem while using esp-idf example for further test.

ESP-Hosted reference : https://github.com/espressif/esp-hosted/issues/279

Thank you for reporting the issue; I will check it.

libbtdm_app.zip
This is a fixed library. The usage involves replacing the file "esp-idf/components/bt/controller/lib_esp32/esp32/libbtdm_app.a" with this updated library.
Both of the methods mentioned above are aimed at resolving the issues caused by the "Read Link Quality" command. Kindly conduct some testing and share your feedback.
thanks!

Hi @zhp0406
I had tried your fixed library, but it seems to cause a core crash.
Logs
ESP32 serial output:
ESP32 Serial Debug 2023-11-28 102942.txt
Linux host btmon output:
btmon_20231128_102842.txt
Linux host hciconfig -a output:
hciconfig_20231128_102930.txt

@MrSumi
I will check the cause of the crash.Have you tried prohibiting the host from sending "Read Link Quality" to the controller when using BLE?

Hi @zhp0406
I guess the "Read Link Quality" may send to the controller by calling hcitool lq with BLE address.
Now, i call hcitool lq only with Classic BT address, and it works fine in the last 12 hours.
To ensure the problem is solved, i will do futher testing.
thinks a lot!

@MrSumi
Looking forward to your further feedback.

Since there was no response from the user closing this issue. If the issue still persists please feel free to reopen this issue or create a new one. Cheers.


Add "expandable" property defining if row can be expanded

Is your feature request related to a problem? Please describe.
I would like to use the expansion feature with nested tables, to expand rows which have a children property.
For rows which don't have children, I would like to disable the expansion feature.
Describe the solution you'd like
Best would be to add a expandable prop, which is a function taking record and index as arguments, and returning a boolean defining wether or not the row is expandable.
<DataTable
      withTableBorder
      withColumnBorders
      columns={[{ accessor: 'name' }, { accessor: 'city' }, { accessor: 'state' }]}
      records={records}
      rowExpansion={{
        allowMultiple: true,
        expandable: ({ record }) => record.children && record.children.length > 0, // 👈 enable expansion only on rows which have children
        content: ({ record }) => (
        ...

#580

That's a good idea, thanks.
I'm up to my ears in a project at the moment, but I'll have a look at how this could be implemented when I'll find a moment.

Wow, that was fast! :-)

Actually I filled the issue while I was already working on the PR :-)

Thanks, I'll merge and publish it later today!

Great, thanks!
I just pushed a new commit with default behavior (when expandable prop is missing) set as "expandable", otherwise it would break existing code!

Released in v7.8.2 🎉


Export 'go_sync_mutex_wait_total_seconds_total' metric

This metric can be useful to debug mutex contention.
Ref: https://github.com/golang/go/issues/49881

We used it (the one exported in downstream projects importing tsdb) as an indicator to assess the impact of https://github.com/prometheus/prometheus/pull/15242
I was trying to follow up and improve the code benchmarking Prometheus itself when I realized it's not exported here.


Add a 'use as template' option

This adds an option to the MenuStyle properties called "use as template". If set, all other menus will inherit their styles from the template menu unless explicitly overwritten. This way you can set the menu style just once instead of every place you use the mini menu extension in your code. The template will still work even if you destroy/close the template menu.
This feels/is a little hacky, but it's a useful feature and I don't want to duplicate all of the style blocks just to support it. I think this is the most minimal way to add it in.
Also I am not responsible for any weirdness that might occur if you set more than one menu as the template. Don't do it!
Thanks @UnsignedArduino for the feature suggestion! Let me know if this covers what you wanted.

also TODO:

somehow the menu/normal styles should be united
maybe these should be some sort of map instead of classes to cut down on code size. probably not a problem just yet

I have not tested this out yet, but this seems reasonable! If there are problems, I will open an issue here/post about it on the forum.


docs(wiki): Add wiki link

Is the readme displaying the same information twice?

Is the readme displaying the same information twice?

Good catch.


show previous / next links on post pages

moved this to the common codebase and deployed to client site


Game Script - Nier: Automata

Attached below is the full Japanese text for Nier: Automata.
I ripped all the text from the game (via unpacking the game's archives) and merged it all into one text file; then I stripped all the blank lines, got rid of all ID/meta info via regex, and finally removed all stray HTML-style tags. If needed, I could provide the unedited file, which is about 4-5x as large and includes the english lines.
In my limited testing with it in this software, it seems to work well, aside from some additional lag from the text search.
(edit 2: appear to have fixed the problems with split lines, apparently the \n's indicated line breaks mid-dialogue, so reverted that change by just removing them instead of also adding a line break.)
(edit 3: was missing the in-game novellas, cleaned them by hand and added them to the end of the file.)
Nier Automata v3.txt

Do you mind removing repeat lines?

……
……９Ｓ。
え？なんでしょう？
敬称は要らない。
え？
……私の名前に『さん』は付けなくていい。
は、はい……分かりました。
……あ、そうだ。
司令官が呼んでたから一緒に行きましょう。
２Ｂさ……いえ、２Ｂ。
９Ｓ……
司令官からの命令で、２Ｂさんのメンテナンスを担当する事になったんです。
これから定期的にチェックしますね。
……そう。
心配は要りませんよ？僕達９Ｓモデルは優秀な事で有名ですから。
……って自分で言う事じゃないか。
……
……９Ｓ。

Removed duplicate lines and updated main comment, replied in the other issue thread with some more thoughts.
https://github.com/mathewthe2/Game2Text-GameScripts/issues/2#issuecomment-844351894

Added to repository
https://github.com/mathewthe2/Game2Text-GameScripts/commit/fadffda517e93a8a33d90a5b3ed8fb0e979bebb5

Played through the majority of the game using v4 of this with no problems; closing this issue now.


update tendermint, ibc-proto deps

I didn't touch the deps on the main ibc-types crate since there were some compatibility issues.


Post refreshes when <3'd

This means that embedded media will reset!

Just noticed this as well. I'll take a look at it.
I think it is due to the onLove event modifying the state and causing a rerender.
https://github.com/ludumdare/ludumdare/blob/master/src/com/content-post/post.js#L136

fixed in c6269e1a27e41647edd9f9593b5cca83d60ef605

This is actually by design.
https://github.com/ludumdare/ludumdare/blob/master/public-api/vx/node.php#L213
The reason for this is to allow a level of interactivity for users that do not yet have accounts. Currently the event is targeted at developers, but this is one method for non-developers to be part of it.
For the sake of security though, anonymous love is limited to 1 love per node per IP address. It's also (at this time) the only thing you're allowed to do without an account.
It can technically be used to give your posts an extra Love, but as far as sorting and filtering is concerned, it just means we up the minimum.
For reference, Love has always worked this way, even on the old site. The minumum threshold before anything happened was 4 love. If "loving" does become a more common thing, this number will change.

It's also partially related to how the Markdown parser is being used. The YouTube video embedder (I'm assuming this was where the issue was found) isn't clever yet.


[21.05] chromium: 92.0.4515.159 -> 93.0.4577.63

https://chromereleases.googleblog.com/2021/08/stable-channel-update-for-desktop_31.html
This update includes 27 security fixes.
CVEs:
CVE-2021-30606 CVE-2021-30607 CVE-2021-30608 CVE-2021-30609
CVE-2021-30610 CVE-2021-30611 CVE-2021-30612 CVE-2021-30613
CVE-2021-30614 CVE-2021-30615 CVE-2021-30616 CVE-2021-30617
CVE-2021-30618 CVE-2021-30619 CVE-2021-30620 CVE-2021-30621
CVE-2021-30622 CVE-2021-30623 CVE-2021-30624
(cherry picked from commit d04f44f7e8d6f475e91e5539becf156676caff5c)

Motivation for this change
Backport of #136373.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

$ nix-build -A nixosTests.chromium
/nix/store/hf4z1gs9jkmkdp71v8mlmfx2cdpl1k79-vm-test-run-chromium-stable


Week numbers mixed in incremental reports

Using pgbadger 11.6 latest with the --iso-week-number and --incremental options.
Clicking week numbers in the global index page opens the week's index page but the dates and data in that page are from the previous week.

Yes there is no backward compatibility with previous weeks reports. If you want to fix that you will have to change the week number in the old sub directory names.

@darold For clarity, this bug occurs even when I produce reports from scratch (no old reports sub-directories exist)

Ok finallly I have understood the problem and was able to reproduce it. Commit 202050f fixes this issue.

@darold I can confirm that this issue appears fixed when using the latest code.

Great, thanks for the feedback, a new release is coming.


opencensus: delete opencensus tracer

Additional Description: This has been deprecated for several releases, and the upstream dependency is no longer maintained.
Risk Level: Low; this was marked as deprecated in 1.28 and was supposed to be removed in 1.30
Fixes #9958
Progress on #11816

/lgtm deps
Yay.


[Event Request] codeunit 550 VAT Rate Change Conversion - Function UpdateTable

Dear ALAppExtensions Team,
We would like to request the following event publisher in codeunit 550 function UpdateTable, of course without the prefix DIN.
begin
    // DINNL <<
    DINIsHandled := false;
    DINOnBeforeUpdateTable(TableID, DINIsHandled);
    if not DINIsHandled then
        exit;
    // DINNL >>
    if not ConvertVATProdPostingGroup and not ConvertGenProdPostingGroup then
        exit;
    RecRef.Open(TableID);
    ProgressWindow.Update(1, Format(RecRef.Caption));
    I := 0;
    ProgressWindow.Update(3, RecRef.Count);
    if RecRef.Find('-') then
        repeat
            I := I + 1;
            ProgressWindow.Update(2, I);
            UpdateRec(RecRef, ConvertVATProdPostingGroup, ConvertGenProdPostingGroup);
        until RecRef.Next() = 0;
end;

local procedure DINOnBeforeUpdateTable(TableIdPint: Integer; var IsHandled: Boolean)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


fix: parse yaml for test in fetch data

@smollweide, sorry I can't verify the snyk because I "do not have permissions to access" it.


tikv-server error when upgrade to last version tikv from version of 2017-8-14

Please answer these questions before submitting your issue. Thanks!

What version of Rust are you using (rustc --version)?

Rustc Version:     1.25.0-nightly (73ac5d6a8 2018-01-11)

What operating system and processor architecture are you using?

centos 7.2 x86_64
cat /proc/version
Linux version 3.10.0-327.13.1.el7.x86_64

What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.

upgrade to lastest version  today (2018-2-8)
last version is downloaded at 2017-8-14 with latest binary version

What did you expect to see?

tikv running ok

What did you see instead?

2018/02/08 09:52:35.411 mod.rs:344: [INFO] Welcome to TiKV.
2018/02/08 09:52:35.412 mod.rs:345: [INFO] Release Version:   1.1.0-dev
2018/02/08 09:52:35.412 mod.rs:346: [INFO] Git Commit Hash:   18ab0e5db93763675525e029533d8ca37f5d3eab
2018/02/08 09:52:35.412 mod.rs:347: [INFO] Git Commit Branch: master
2018/02/08 09:52:35.412 mod.rs:348: [INFO] UTC Build Time:    2018-02-07 09:01:38
2018/02/08 09:52:35.412 mod.rs:349: [INFO] Rustc Version:     1.25.0-nightly (73ac5d6a8 2018-01-11)
2018/02/08 09:52:35.412 tikv-server.rs:503: [ERROR] invalid configuration: StringError("default rocksdb exist, buf raftdb not exist")

hi @aruanruan
Seem you use a too old TiKV version before which didn't support raft RocskDB.
We don't support compatibility before GA version.


Update anuran.html - Added some more details

I have added some more informations about the blog. Please merge my pull request.

@akshitagupta15june Please review my changes

Okay 👍


[Feat] code format & using CSSProperty wrapper

Added the ability to add a font family type.

Hello, thank you for your PR contribution.
Regarding the implementation of font-family, I have already implemented it in PR #29 (taking into account the various cases of CSS font-family and matching with UIFont/NSFont Family).
I will adjust the Code Format and CSSProperty contributions from your PR, extract them, and then merge them.
Thank you again for your contribution.

Fixed a conflict

@zhgchgli0718
Is there anything else we need to do to merge this PR?

I had some code I wanted to add, so I added it and raised it to a new PR!
I removed and uploaded it because I think it's a personal preference, but if you like it, I'll apply it in bulk to #30 PR 🙏
And I'll close this PR!!!


[delta-xds] add nonce to all requests sent to management server

Description: right now in Delta XDS, only ACK request has a nonce attatched to them, we need to attach the most recent nonce to all delta XDS requests.
Repro steps:
enable DeltaXDS connection with CDS/EDS or SRDS&&RDS, new sub-resource (out of cycle) XDS requests created by "master" XDS types will not have a nonce attached to the requests.

cc @htuch

cc htuch@

/cc @htuch

I don't think we need a nonce for out-of-cycle requests, see https://www.envoyproxy.io/docs/envoy/latest/api-docs/xds_protocol#incremental-xds:

Spontaneous DeltaDiscoveryRequests from the client. This can be done to dynamically add or remove elements from the tracked resource_names set. In this case response_nonce must be omitted.

didn't see that.
it's a little bit confusing as earlier in the doc, it also says
https://www.envoyproxy.io/docs/envoy/latest/api-docs/xds_protocol#basic-protocol-overview

All server responses will contain a nonce, and all subsequent requests from the client must set the response_nonce field to the most recent nonce received from the server on that stream. This allows servers to determine which response a given request is associated with, which avoids various race conditions in the SotW protocol variants. Note that the nonce is valid only in the context of an individual xDS stream; it does not survive stream restarts.

Let me close this issue if that's by design.


Add support for excluded_regions property in Datadog::Integrations::AWS

We need to exclude AWS regions from our AWS integration.
After contacting support we were advised to use the API and set the excluded_regions property (link).
Unfortunately the AWS resource does not include it so we can't use it.

Closed by: https://github.com/DataDog/datadog-cloudformation-resources/pull/243

Is this released? It's not currently available in 2.2.1


Revert "[CMake] sync submodule"

This fixes CI. Until upstream CMake is fixed we need to block any further syncing of the CMake submodule. This also warrants a new patch release.
cc: @nim65s @proyan
This reverts commit cf179f7bcd676cc8a489add1e83f4c031c9f09d0.

Thanks for the work @wxmerkt

Thank @wxmerkt :)
@proyan you might want to cherry picks these commits (to boost the merge of the other PR)


Add new MOC planes

Great project you made, here are the new planes from March of the Machines (there are no new phenomenons). Used same dimensions from original images and disabled by default the tricky planes.
Any feedback is welcome :).

Thank you for the PR!
I'll review this first chance I get, but at first glance I don't see anything odd. :)

I've made the result of this PR available at https://planechase.spicyfortress.games/ until it's merged and deployed if that's ok.


small responsive adjustments

Small change per Corinne. Merge at your convenience @ksigler7.

Hello @justinwhall,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


firebase_auth onAuthStateChanged doesn't refresh from Firebase

I know that the whole point of onAuthStateChanged is to get when a user logs in and out. From my understanding, the FirebaseUser is cached in the device.
I'm however facing an issue. When I'm using onAuthStateChanged with the Js/Web SDK it catches all changes even those in Firebase. For example deleting/disabling a user, verifying email etc.
But in Flutter when I delete a user from firebase console or when a user verifies their email it doesn't seem to update. The user has to completely log out and then try to log back in. or use  reload() method

How does this compare to the native iOS/Android behavior?

I haven't used firebase_auth in native IOS/Android before. After posting the issue I researched around the internet and realized that the JS SDK has a different implementation for that and why it wouldn't be ideal to use that for Flutter and native Android/Ios. For now the reload() method on the FirebaseUser  object will do. The only problem is that I have to call currentUser() twice, before reload and after reload, for the FirebaseUser object to actually update.

Sounds like a bug, I think that reload() should probably mutate the current user object to have the correct data.
Filed #32431

@newtonmunene99
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/376. Any further collaboration will be done there.


Need any help?

Hello there, I was going to create a React JS library for UIKit similar to the bootstrap one: https://github.com/react-bootstrap/react-bootstrap/. I prefer UIKit to Bootstrap but I do not want to reinvent the wheel.  If not I will create a new one. Thanks!

Hey, yes I do plan on working on it.I want to draw inspiration from http://material-ui.com/ in addition to react-bootstrap.
You're more than welcome to join in on the effort, no point in duplicating our efforts. I'll add you to the project right away :)

Thanks so much! I thought it silly to create what you obviously were
working on.  I read the material UI spec last week, great stuff in there.
Just to clarify, you are planning on utilizing the UIKit (getuikit.com)
framework right?

Zach Curtis
P:<PHONE_NUMBER>
E<EMAIL_ADDRESS>T: @zachariahCurtis https://twitter.com/#!/@zachariahCurtis
B: http://zachariahtimothy.wordpress.com/
On Wed, Mar 4, 2015 at 4:27 AM, Stian Didriksen<EMAIL_ADDRESS>wrote:

Hey, yes I do plan on working on it.I want to draw inspiration from
http://material-ui.com/ in addition to react-bootstrap.
You're more than welcome to join in on the effort, no point in duplicating
our efforts. I'll add you to the project right away :)
—
Reply to this email directly or view it on GitHub
https://github.com/stipsan/react-uikit/issues/1#issuecomment-77142067.

Yes, I am going to utilize UIkit by YOOtheme :)


Caching breaks the rate limiting

I'm pretty sure the caching expiration logic used is incorrect and causes requests to not be counted or rate limited
I had to remove the caching for it to actually work once deployed. I don't see how you can cache anything and count responses to be rate limited at the same time but I could be misunderstanding it 🤔

Hello @sathoro.
Sorry for the late response, I'm not that active here atm.
The caching is used to check if a request is already being rate-limited, as when the rate-limit is triggered, the DO response contains expire headers which are used to off-load traffic to the DO for identical subsequent requests, for the rate-limiting period.
I don't see how the above behavior can break the rate-limiting, as the requests would indeed be rate-limited for that period of time even with caching disabled and the extra counts would make no difference as they would refer to the same (already rate-limited) time window.
I hope this helps. Let me know if you think I'm missing something here.

@honzabit Hey, no worries!
I'm referring to the caching happening here: https://github.com/honzabit/durable-limiter/blob/main/src/index.ts#L29
Doesn't this cache the rate limiting call, so that subsequent requests aren't counted for rate limiting purposes? We are running a modified version of this in production and it works great but we did remove the caching in that file because in my testing it was making it never actually do any rate limiting
On the other hand, the Cache-Control here is fine: https://github.com/honzabit/durable-limiter/blob/main/src/ratelimiter.ts#L146

Hello again.
Yes, it does, and that's actually the purpose of this code, to see if we already have a cached response from the rate limiter for the given set of keys (the keys shown here) so that when we do, it means that there's no need to consult again the rate-limiter as the response would be 429.
The cached response will expire though when the time window progresses (for fixed time window configuration), or, in the case of the sliding algo, when the "virtual time window" does.
The calculation of the expirations times can be found in the else block here.
Hope this helps.


Ambilight extreme CPU usage and Main monitor selection broken

So testing the ambilight layer on the current beta (v0.6.1-beta3) I get 30% CPU usage (to be fair, 2x Full HD and 1x 2560x1440 screen) then wanting to limit that to a single screen i.e. my main monitor Then only the f1-f12 keys light up and the area seems to be around the mid lower section of the screen.
Trying to limit it to a single process still uses the same large amount of CPU usage.

The issue appears to be as a result of how the current system of taking screenshots works as it will always just stitch the monitors you have beside each other in whatever order they are in windows, whereas the region we use to get the right part is determined using the region that windows make to lay everything out which results in a mismatch. From looking at it it'll likely require a switch in screenshot method for both issues (CPU usage and bad regions) which might be a while.

The performance of ambilight could probably be improved by using the desktop duplication api https://msdn.microsoft.com/en-us/library/windows/desktop/hh404487(v=VS.85).aspx
I use a program that has ambilight as its main feature (https://github.com/psieg/Lightpack) and desktop duplication performs about 4 times better and it doesn't make the windows lag.

hi sorry for reviving this issue, but still cpu usage too high for ambilight and my i5 6600k.
Anyway looking to upgrade, do you believe im better with more threads or more single core power to run amblight without taking much from new cpu?

The desktop duplication was recently merged (thanks @lucasvanhalst ) and I decided to make some UI changes and under the hood improvements. Looking for feedback. @ntreik if you could give this build a try that would be awesome. https://ci.appveyor.com/project/antonpup/aurora/builds/26114907
Please backup your profiles!

Hi just tried reduces cpu usage to almost half but my SteelSeries headset is not changing colors anymore. . its connected alright in device manager but only listening to stelseries app
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

Από: Diogo Trindade<EMAIL_ADDRESS>Στάλθηκε: Wednesday, July 24, 2019 4:52:00 PM
Προς: antonpup/Aurora<EMAIL_ADDRESS>Κοιν.: ntreik<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Θέμα: Re: [antonpup/Aurora] Ambilight extreme CPU usage and Main monitor selection broken (#778)
The desktop duplication was recently merged (thanks @lucasvanhalsthttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Flucasvanhalst&data=02|01||820483471abd4925224a08d7103e1a50|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636995731221165628&sdata=f9UtkIoMplaG1epet2lXSezJ89sLUAonK3YR05wTCSI%3D&reserved=0 ) and I decided to make some UI changes and under the hood improvements. Looking for feedback. @ntreikhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fntreik&data=02|01||820483471abd4925224a08d7103e1a50|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636995731221165628&sdata=DfW3fM8%2BjS3aJLK%2F7PcXOzdWdBg8KcyI3CR2boM3DYI%3D&reserved=0 if you could give this build a try that would be awesome. https://ci.appveyor.com/project/antonpup/aurora/builds/26114907https://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fci.appveyor.com%2Fproject%2Fantonpup%2Faurora%2Fbuilds%2F26114907&data=02|01||820483471abd4925224a08d7103e1a50|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636995731221175636&sdata=rzlnnqF3dTCCRyq8M8o41kUQFqtsVeC0kmz8e%2BlHy5s%3D&reserved=0
Please backup your profiles!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fantonpup%2FAurora%2Fissues%2F778%3Femail_source%3Dnotifications%26email_token%3DALPFD2QGJPDU4QQSIBB5253QBBNABA5CNFSM4DZM2PZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2WM2JQ%23issuecomment-514641190&data=02|01||820483471abd4925224a08d7103e1a50|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636995731221175636&sdata=2UM4yiXl4T%2FMW6Fni1kZlxdrRTWaBdtVZqQo1FnshOk%3D&reserved=0, or mute the threadhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALPFD2SAQFMCK6KDPJQDWU3QBBNABANCNFSM4DZM2PZQ&data=02|01||820483471abd4925224a08d7103e1a50|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636995731221185649&sdata=zCgjr3LqzKENUXuuk7TymF41ZHA7IXTV6nkl%2FkTyzh8%3D&reserved=0.

The Desktop Duplication API is now available with v0.7.0. Please make another issue if it still causes excessive cpu usage

Hi,
I've received this item today : https://www.aliexpress.com/item/32974224895.html?spm=a2g0s.90423<IP_ADDRESS>be4c4dR8UQlI
I've installed it and looks ok but I noticed that Ambibox uses 7 to 10% of CPU ressources thus heat my CPU more than usual when the screen capture mode is ON.
Is there any way to fix that?
Thanks for the help !


OCP is now OCI

There was a renaming from "Project" to "Initiative".

Thanks for your reminder, we will update it soon:)

:)  We were thinking about choosing a cool name @vbatts , but struggling...

hi @vbatts, the project changed to oct (open container testing) now

:-D :octopus:
Thanks y'all


fix: add hugo-responsive-images

Added a new theme module, which I have written and actively maintain.

Thanks!


Do tasks have to be in jobs?

"A task is the building block of a pipeline." But is that really true?
My understanding:
A pipeline is made of one or more jobs.
A job is made of one or more tasks.
I think this page is not successful in communicating these relations.
The following source is along my logic "Every build or deployment pipeline has at least one job." :
https://docs.microsoft.com/en-us/azure/devops/pipelines/process/phases?view=azure-devops&tabs=yaml
For contrast, i think GoCD does an excellent job at describing the basic concepts.
https://docs.gocd.org/current/introduction/concepts_in_go.html

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

ID: 7f8c8539-89a7-a426-868b-96e961c39e29
Version Independent ID: de260a68-aab7-7c6b-6727-90c92c65b225
Content: Azure Pipelines New User Guide - Key concepts - Azure Pipelines
Content Source: docs/pipelines/get-started/key-pipelines-concepts.md
Product: devops
GitHub Login: @steved0x
Microsoft Alias: jillfra

Correction:
A job is made of steps. A step can be a task OR a script. Especially as the focus is shifting from the UI editor to the YAML editor, I think scripts are will be more and more used. I think the followin two stuff are also key concepts, and I suggest to add them to this page:

Steps
Scripts

https://docs.microsoft.com/en-us/azure/devops/pipelines/scripts/cross-platform-scripting?view=azure-devops&tabs=yaml
Please let me know what you think!


October Video workgroup

I added summaries for the video workgroup for October.

https://github.com/SingularityNET-Archive/SingularityNET-Archive/issues/52#issuecomment-1816136575


navigator: allow mission items with same position

This reverts the addition of a check against mission items that have the same position. This breaks existing MAVSDK implementations where a LOITER_TIME item is set right after a WAYPOINT with the same coordinates. It is an interim hack to allow the vehicle to hold still during a photo is captured.
The question is where not having this check would currently lead to trouble in the waypoint navigation logic. If there are potential problems, I would like to address them.
This is just a draft PR to check CI for now.
Would fix #14904.

As I've already explained in the original issue allowing this would force complexity onto the autopilot that would make navigator even more complex. The fact that it was possible before was an oversight of the checks.

Otherwise things like CONDITION_GATE are not properly conditioned and have no proper solution.

Ok, that's the information I was looking for, thanks.


Update version

This won't resolve matter as a transient dependency properly. The current version range resolves correctly.


Require RSpec only when not loaded yet

This PR fixes issue #87
It checks if rspec is already loaded and if so does not require it.
In issue #87 I use the rspec-rails and don't have the rspec gem in my bundle. Bundler already requires the rspec-rails gem and loads rspec-core, which actually contains (most of) the rspec gem.
An alternative solution would be the following below, but I chose not to do this because you also check if Capybara loaded the same way as in this PR
 require 'pry-rescue'
-require 'rspec'
+begin
+  require 'rspec'
+rescue LoadError
+  require 'rspec/core'
+end

@ConradIrwin This fixes the issue for me on rails5 + rspec 3.5.0. Could we merge this in and cut a new release?

This is very peculiar, I was under the impression that when you require the same thing twice it should be a no-op the second time. Happy to merge this if you're sure that's what fixed it

@ConradIrwin Awesome, thanks! Any chance of pushing a new release to rubygems

DoneSent via SuperhumanOn Fri, May 20, 2016 at 11:00 AM, Michael Bianconotifications@github.comwrote:@ConradIrwin Awesome, thanks! Any chance of pushing a new release to rubygems
—You are receiving this because you were mentioned.Reply to this email directly or view it on GitHub


Window Event Handling

Make a system to listen for and handle window events, especially closing.

This has been dealt with fairly simply, as was detailed in the most recent commit.


Lab 1 grader

In this PR we are introducing the meta tags: grader_import and grader_function to grade future exercises. With the introduction of this new system the content for Lab 1 must be reviewed in consequence.
Fix #360
preview: https://platypus-pr-797.ecy0akwlcpw.us-south.codeengine.appdomain.cloud/course/ch-labs/lab-1-quantum-circuits

in attempting to merge/update this branch with changes from main resulted in many of the translation notebooks appear with changes here (but not locally) 🤷🏽  anyhow, any attempts i made to clear these up didnt seem to work. ultimately, i just created a new PR (#1407) with the appropriate changes to replace this one and will close out this one

in attempting to merge/update this branch with changes from main resulted in many of the translation notebooks appear with changes here (but not locally) 🤷🏽 anyhow, any attempts i made to clear these up didnt seem to work. ultimately, i just created a new PR (#1407) with the appropriate changes to replace this one and will close out this one

FYI, @Tansito @frankharkins i closed this in favor of #1407


tdMessage component is duplicated with *ngIf

Do you want to request a feature or report a bug?
bug
Bug Report
Steps to reproduce

Use a td-message inside a <div *ngIf="condition">
Change the condition result multiples time

Screenshots or link to CodePen/Plunker/JSfiddle

see this plunker:  https://plnkr.co/edit/6NRQD9KQjluwEtdrkTbe?p=preview
What is the expected behavior?
The box should not be duplicated.
Which version of Angular and Material, and which browser and OS does this issue affect?
Same as in the plunker<EMAIL_ADDRESS>

Duplicate of #981. Already fixed in #984 :smile:

oups, sorry I only looked in open issues

Yup, this was fixed already and will be available in Rc.1.. you can however use the nightly build for the time being.


⚠ Bofrost.lu has degraded performance

In 7da803d, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2493 ms

Resolved: Bofrost.lu performance has improved in c4ae1ed.


text to speech REST API: silence at the end of generated audio - part 2

@szhaomsft thanks for updates. Will now proceed to close this thread.
Originally posted by @GiftA-MSFT in https://github.com/MicrosoftDocs/azure-docs/issues/47344#issuecomment-583507480

I wasn't able to reopen #47344 so I open a new ticket.
I attach the two audios and an audacity screenshot of the audios from December 17th vs. January 10th. They clearly show the difference of 50ms. If you did not change anything on your silence configuration that means that the change of phonemes caused that change in end silence.
What is a range for silence time at the end of the audio that I can rely on? Can you guarantee it will stay above 650 ms?
[audios] (https://github.com/MicrosoftDocs/azure-docs/files/4185148/audios.zip)

For Reference from the original ticket:
We use the REST API for text to speech (https://westeurope.tts.speech.microsoft.com/cognitiveservices/v1). The generated speech contains silence at the end of the audio. This used to be about 800 ms, since recently (January) it appears to be less.
Is there anyway to control/predict the amount of silence that is generated at the end?
In our setting we record the customer's voice immediately after playing back the generated audio.

if we start recording too early we hear our own synthesized audio
if we start recording too late we will often miss the beginning of the customer's sentence

So it is important for us to know the amount of silence that is produced by the text to speech service.
We use HeddaRUS. Request bodies are of the form
<speak version='1.0' xml:lang='de-DE'><voice xml:lang='de-DE' xml:gender='Female' name='Microsoft Server Speech Text to Speech Voice (de-DE, HeddaRUS)'>Hallo</voice></speak>

@christoph-zenger-hcsg Thank you for the feedback. We will review and update accordingly.

It looks like @szhaomsft replied to this question on original thread. Tagging here for reference.

yes replied

@christoph-zenger-hcsg your question has been answered on the previous help. Do you have any further questions or concerns?


(BKR-6) added privatebindir for windows' old PE and FOSS types

:+1:

For git acceptance, I see the correct /usr/bin prepended to PATH:
$ be rake ci:test:git SHA=4dae4b348ab6942d07c178fd160726e2c7895eaa CONFIG=config/nodes/win2012r2-rubyx64.yaml
...
che2lw9pzylud0n.delivery.puppetlabs.net (agent-2012r2-x86_64-rubyx64) 10:20:21$ env PATH="/usr/bin:${PATH}" cmd /c gem source --clear-all
env: cmd: No such file or directory

The command fails due to BKR-7 (since PATH is corrupted with a semicolon), but this is expected, so :+1:  to merge.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/641/
:red_circle: Test failed.

The failures are because beaker is testing against 3.7.4, which doesn't have the fixes that were made for QENG-1592 in commit 0d9d11392. You may want to backport that commit to puppet#stable and pin against that version.

@joshcooper I'm working on backporting test updates now so that we can get to green.

@kevpl @anodelman should be good to retest now

This won't get exercised with additional testing.  Merging as-is.


added local seller authentication

Looks good! Log the Autorization issue as a bug for someone else to fix.


Error: Cannot find module 'sqlite'

Whenever I run this bot it keeps giving me that error. Please help.

npm i discord.js and npm i sqlite


init and parse_obj should have (virtually) the same behaviour in V2

see https://github.com/samuelcolvin/pydantic/issues/1608#issuecomment-784920520
Container(**x) and Container.parse_obj(x) should have exactly the same behaviour (except parse_obj should allow or raise a ValidationError if x is not a mapping) in v2.
In particular they should both support an argument which is a mapping used in a __root__ type.

.


ImportError: cannot import name 'RasaNLUConfig'

Rasa Core version: 0.8.5
Python version: 3.6.4
Operating system (windows, osx, ...): windows 10
Issue:
I have recently updated to RASA NLU v0.12.0 after the update i'm getting this issue
Traceback (most recent call last):
  File ".\bot.py", line 25, in <module>
    run_bot()
  File ".\bot.py", line 16, in run_bot
    agent = Agent.load('./models/dialogue/default/dialogue_model', RasaNLUInterpreter('./models/nlu/default/nlu_model'))
  File "D:\Softwares\Python\lib\site-packages\rasa_core\interpreter.py", line 219, in __init__
    self._load_interpreter()
  File "D:\Softwares\Python\lib\site-packages\rasa_core\interpreter.py", line 234, in _load_interpreter
    from rasa_nlu.config import RasaNLUConfig
ImportError: cannot import name 'RasaNLUConfig'

I got this error while using the following method
agent = Agent.load('./models/dialogue/default/dialogue_model', RasaNLUInterpreter('./models/nlu/default/nlu_model'))

We haven't released a new version of rasa core yet that works with NLU 0.12.x - the master version from github should be fine though. We are working on a new release that will be compatible, but it will take a couple days.

For the time being, I created an alpha release that should show up on pypi in a couple minutes.
So if you want to use nlu 0.12.x together with core, you can install rasa core version 0.9.0a1.

@tmbo  I see two versions were released with time gap of 30 min which one is stable?

So the 0.8.6 one is the latest stable one and the 0.9.0a2 is a tagged release from the current development version (which is tested and stable as well, but not yet feature complete and that is why we can not yet release the next major version 0.9.0 for core).

I wanted test the new 12.2 but I face same issue. which version of core I have to install now which is working like latest stable but together with 12.2?

I ugraded to 0.9.0a2 and get:
Traceback (most recent call last):
  File "dialogue_management_model.py", line 45, in <module>
    run_bot()
  File "dialogue_management_model.py", line 35, in run_bot
    interpreter = RasaNLUInterpreter('./models/nlu/default/kk_nlu')
  File "C:\Users\chris1\Anaconda3\envs\rasa12\lib\site-packages\rasa_core\interpreter.py", line 221, in __init__
    self._load_interpreter()
  File "C:\Users\chris1\Anaconda3\envs\rasa12\lib\site-packages\rasa_core\interpreter.py", line 237, in _load_interpreter
    self.interpreter = Interpreter.load(self.model_directory)
  File "C:\Users\chris1\Anaconda3\envs\rasa12\lib\site-packages\rasa_nlu\model.py", line 272, in load
    Interpreter.ensure_model_compatibility(model_metadata)
  File "C:\Users\chris1\Anaconda3\envs\rasa12\lib\site-packages\rasa_nlu\model.py", line 264, in ensure_model_compatibility
    rasa_nlu.__version__))
rasa_nlu.model.UnsuportedModelError: The model version is to old to be loaded by this Rasa NLU instance. Either retrain the model, or run withan older version. Model version: 0.12.0a1 Instance version: 0.12.2

It should work, but you have to retrain your NLU model using 12.2.x

i'll close for now, let us know if you encounter any more issues

Hi referenced to this issue because I had the same issue and I don't know how to upgrade to a new version and which.
My error message was :
rasa_nlu.model.UnsupportedModelError: The model version is to old to be loaded by this Rasa NLU instance. Either retrain the model, or run withan older version. Model version: 0.9.0a4 Instance version: 0.12.3

I have the same problem:
rasa-core = 0.9.7
rasa-nlu = 0.12.3

I just followed the installation instruction on Rasa webpage:
pip install rasa_core
pip install rasa_nlu[spacy]
python -m spacy download en_core_web_lg
python -m spacy link en_core_web_lg en

@antoinecomp did you try to retrain after you reinstalled rasa?

working combination  to resolve this error:
rasa_nlu = 0.11.5
rasa_core = 0.10.3

@Ghostvv Yes, Probably !

Any reason why not to upgrade to 0.10 and 0.13? 😄
You usually need to retrain models anyways when upgrading to major versions as there can be breaking changes

I have the same problem
from rasa_nlu.config import RasaNLUConfig
is not working. I am using the 0.14.3 version of rasa_nlu. What could be the problem

@serdarbozoglan it doesn't exist in rasa_nlu 0.14.3

when i run
->from rasa_nlu.converters import load_data
->from rasa_nlu.model import Trainer
they work fine
but when i run -> from rasa_nlu.config import RasaNLUConfig
then i get
cannot import name 'RasaNLUConfig'
why

Also when i run import rasa_nlu.config then it's work also fine

please try RasaNLUModelConfig instead


Wrong program interpreter requested on musl targets

The cc crate requests /lib/ld-linux-aarch64.so.1 as a program interpreter on musl targets when some c++ code is built. Instead the cc crate should request /lib/ld-musl-aarch64.so.1.

I'm not convinced that it's cc-rs's domain/responsibility. To specify the dynamic linker that is. cc-rs merely compiles object modules, and linking is performed by rustc. One can wonder if cc-rs could [and then if it actually should] help steering rustc to target-specific linker, but as far as I can tell it doesn't interfere with this process. I would suggest treating musl as any other cross-compile target and specify musl-gcc as designated linker. One way is to add following lines to your .cargo/config.toml:
[target.aarch64-unknown-linux-musl]
linker = "musl-gcc"

I think requiring users to specify this in .cargo/config.toml is not right. The problem clearly occurs when C++ code is linked. Rust is not responsible for linking C++ as is Cargo, cc is responsible for handling proper C++ linkage.

I think requiring users to specify this in .cargo/config.toml is not right.

Yet it's the way it is. Try to cross-compile for any target and you'll see.

Rust is not responsible for linking C++ as is Cargo, cc is responsible for handling proper C++ linkage.

cc-rs does tell rustc to link with C++ run-time library, but doesn't tell which linker to use. The choice is rustc's. [Do note that I'm not in position to change anything, I merely describe the state of affairs.]

You cannot rely on everyone customizing their config.toml every time they work on must based systems, this crate should just work no matter what target is used as long as the target supports C++.

What defines a musl-based system? If there are systems where musl is the default (or only?) libc run-time, then the expectation would be that default cc would link with it, as well as pick matching dynamic linker. If it's not the case, then can one call it musl-based? Questions are rather rhetorical... In sense that I don't expect answers, nor plan to engage in further discussion. Again, I'm just describing the way things are, and have no power to change anything.

If there are systems where musl is the default (or only?) libc run-time, then the expectation would be that default cc would link with it, as well as pick matching dynamic linker.

That's exactly what I meant, If musl is the only libc (which is the case on Postmarket OS aarch64) then cc should instruct rustc to use the proper linker instead of continuing with the default one and generate a broken binary which is the current state. If there are both libc and musl available let rustc choose what it prefers by default.

But that's not what I meant. I didn't mean that cc-rs crate would have to do anything special, but that the default linker used by rustc would "know" what to do without rustc doing anything special. In other words nobody would have to instruct anybody about anything on a musl-based system. Or maybe more specifically on a system that can actually be classified as musl-based.
Just in case, the misunderstanding arose from the fact that when I wrote "cc" I referred to an actual command, a.k.a. default linker, while you interpreted it as "this crate." My fault, I should have been more clear.

But that's not what I meant. I didn't mean that cc-rs crate would have to do anything special, but that the default linker used by rustc would "know" what to do without rustc doing anything special. In other words nobody would have to instruct anybody about anything on a musl-based system. Or maybe more specifically on a system that can actually be classified as musl-based.
Just in case, the misunderstanding arose from the fact that when I wrote "cc" I referred to an actual command, a.k.a. default linker, while you interpreted it as "this crate." My fault, I should have been more clear.

If I get it right, that means you should open an issue on rustc...

If I get it right, that means you should open an issue on rustc...

It means that someone might choose to open an issue with rustc. Just in case, as implied above, I personally have no plans for doing anything beyond describing the way things are.

If I get it right, that means you should open an issue on rustc...

It means that someone might choose to open an issue with rustc. Just in case, as implied above, I personally have no plans for doing anything beyond describing the way things are.

So that means you're fine with the current state which is generating broken binaries...


Test pipeline v3

adding "test api stage" inside Jenkinsfile. If api return 200, through message "api is valid". If not through "api not found" and stop building


CVAT new UI: Filter property "shape" and operator "~="

In the new UI, it’s not clear how to use a filter with the "shape" property,
I tried the filter "shape == polygon" and the like, but they just hide all the objects.
And also I did not understand in what cases to use the operator "~ =", for example, applying the filter "width ~ = 100" causes an error:

@TOsmanov , please confirm that the problem is resolved.

@TOsmanov
Right filter for shape type is shape == "polygon". We've also added the example to CVAT.

@nmanovic Problem resolved
@bsekachev Thanks, and the operator "~ =" will not be used?

@TOsmanov
There were two issues with this operator. First right operator is =~, not ~=
Second is that operator allows to use regular expressions (according to specification), but technically a library that we use doesn't support this operator.
So, I've just removed it from CVAT.

@bsekachev ok thanks


Tracking issue for adding NIP-13 (POW) to NDK

This is a tracking issue for changes needed to add NIP-13 to NDK.

[ ] Figure out TS webworker
[ ] Turn Miner into class to handle Isomorphic webworker with node.js
[ ] Tests for all files
[ ] Add event ID in the handleMinedPow function (in nip13.ts)

Ref this for workers https://github.com/v0l/snort/blob/a3f139399973df4caa1e2e20b83ce80663fae90b/packages/system/src/pow.ts#L4


Do not depend on latest/untagged dependencies

When building or running any docker command, go fetches all latest dependencies. This might be dangerous if we try to replicate the build from a release branch.
make format                                                                     11:16
Building vendor...
go: finding sigs.k8s.io/controller-tools/pkg/crd/generator latest
go: finding sigs.k8s.io/controller-tools/pkg/crd latest
go: finding sigs.k8s.io/controller-tools/pkg latest
Formatting code...
pkg/util/util_test.go

IIRC this should not be a problem, there should be a hash of each module we are using.


oci-security-health-check-standard output files has wrong path

the output files has wrong path because the links in the file *standard_cis_html_summary_report.html has refered *_standard/cis_Identity_and_Access_Management_1-1.csv but the generated files not has the same path

Issue is resolved


fail fast on unknown options

prevents typos and makes the initializer easier to read + less hash lookups ... might do that for more methods later
needs ruby 2.0+ ... 1.9 was "end of life" February 23, 2015 ... and whoever still runs it will get an old version since it requires 2.0+

@gmmeyer

fixed ... turned out rubocop was linting for 1.9 which did not have keyword args :)
... also changed rake to run rubocop so users see failures before making PRs

ready!

@gmmeyer merge ?


Fix for PR #236

Description
Pulled in customer change and also fixed parameter for specifying the storage type.
Checklist

[x] This pull request was tested on both of:

[x] PowerShell 5.1 (Windows)
[x] PowerShell 6.x (Latest PowerShell)

[x] Scripts do not contain static passwords or other secret tokens.
[x] All Azure resource identifiers which must be universally unique are guaranteed to be so.

Testing information

Platform:
Powershell version: $PSVersionTable.PSVersion
Local:
Major  Minor  Build  Revision

5      1      18362  145
CloudShell:
Major  Minor  Patch  PreReleaseLabel BuildLabel

6      2      2
Az version: (Get-InstalledModule -Name Az).Version
Local:
Major  Minor  Build  Revision

2      5      0      -1
CloudShell:
2.6.0

#sign-off Pulled over the other fixes and changed -AccountType to -SkuName, retested and verified the disk is created correctly.

@sptramer


Generate GroupByHash output in multiple RecordBatches rather than one large one

Is your feature request related to a problem or challenge?

The AggregateExec generates one single (giant) RecordBatch on output (source)
Which is then emitted in parts (via RecordBatch::slice(), which does not actually allocate any additional memory) (source)

This has at least two potential downsides:

No memory is freed until the GroupByHash has output every output row
As we see in https://github.com/apache/arrow-datafusion/issues/9417, if there are upstream operators like TopK that hold references to any of these sliced RecordBatchs, those slices  are treated as though they were an additional allocation that needs to be tracked (source)

Something like this in pictures:
                                                 Output
                           ▲               RecordBatches are
                           │                 slices into a
                  ┌────────────────┐          single large
                  │  RecordBatch   │─ ─ ─ ┐   output batch
                  └────────────────┴ ─ ─ ┐
                           ▲              │
                           │             │        ┌────────────────┐
                  ┌────────────────┐      └ ─ ─ ─▶│                │
  output stream   │  RecordBatch   │     │        ├ ─ ─ ─ ─ ─ ─ ─ ─│
                  └────────────────┘      ─ ─ ─ ─▶│                │
                                                  ├ ─ ─ ─ ─ ─ ─ ─ ─│
                         ...                      │                │
                                                  │                │
                           ▲                      │      ...       │
                           │                      │                │
                  ┌────────────────┐              │                │
                  │  RecordBatch   ├ ─ ─ ┐        │                │
                  └────────────────┘              │                │
                           ▲             │        ├ ─ ─ ─ ─ ─ ─ ─ ─│
                           │              ─ ─ ─ ─▶│                │
                           │                      └────────────────┘
                           │
               ┏━━━━━━━━━━━━━━━━━━━━━━━┓
               ┃                       ┃       Single RecordBatch
               ┃                       ┃
               ┃                       ┃
               ┃                       ┃
               ┃                       ┃
               ┃    GroupByHashExec    ┃
               ┃                       ┃
               ┃                       ┃
               ┃                       ┃
               ┃                       ┃
               ┃                       ┃
               ┃                       ┃
               ┗━━━━━━━━━━━━━━━━━━━━━━━┛

Describe the solution you'd like
If we had infinite time / engineering hours I think a better approach would actually be to change GroupByHash so it didn't create a single giant contiguous RecordBatch
Instead it would be better if GroupByHash produced a Vec<RecordBatch> and then incrementally fed those batches out
Doing this would allow the GroupByHash to release memory incrementally as it output. This is analogous to how @korowa  made join output incremental in https://github.com/apache/arrow-datafusion/pull/8658
Perhaps something like
                            ▲
                            │
                   ┌────────────────┐                    Output
  output stream    │  RecordBatch   │              RecordBatches are
                   └────────────────┘              created in smaller
                            ▲                      chunks and emitted
                            │                          one by one
                            │
                            │
                ┏━━━━━━━━━━━━━━━━━━━━━━━┓          ┌────────────────┐
                ┃                       ┃          │  RecordBatch   │
                ┃                       ┃          └────────────────┘
                ┃                       ┃          ┌────────────────┐
                ┃                       ┃          │  RecordBatch   │
                ┃                       ┃          └────────────────┘
                ┃    GroupByHashExec    ┃          ┌────────────────┐
                ┃                       ┃          │  RecordBatch   │
                ┃                       ┃          └────────────────┘
                ┃                       ┃                 ...
                ┃                       ┃          ┌────────────────┐
                ┃                       ┃          │  RecordBatch   │
                ┃                       ┃          └────────────────┘
                ┗━━━━━━━━━━━━━━━━━━━━━━━┛
                                                    Vec<RecordBatch>

Describe alternatives you've considered
No response
Additional context
@yjshen  notes:

To improve AggExec's mono output pattern, https://github.com/apache/arrow-datafusion/issues/7065 might be similar to the idea of incremental output.

take

This issue is interesting, let me try implement it

FYI this issue may be tricky -- as it will be performance critical -- I will be happy to assist

this issue is a bit tricky for me 😢 , I can only think of the following approaches:
change the GroupedHashAggregateStream::emit() function return a vector Result<Vec<RecordBatch>>, each RecordBatch num_rows equal GroupedHashAggregateStream::batch_size, and change ExecutionState::ProducingOutput(RecordBatch) to ExecutionState::ProducingOutput(Vec<RecordBatch>), then GroupedHashAggregateStream::poll_next() function returns one element of a vector at one loop, like let output = batchs.pop() source

this issue is a bit tricky for me 😢 , I can only think of the following approaches: change the GroupedHashAggregateStream::emit() function return a vector Result<Vec<RecordBatch>>, each RecordBatch num_rows equal GroupedHashAggregateStream::batch_size, and change ExecutionState::ProducingOutput(RecordBatch) to ExecutionState::ProducingOutput(Vec<RecordBatch>), then GroupedHashAggregateStream::poll_next() function returns one element of a vector at one loop, like let output = batchs.pop() source

I think this approach sounds good -- nice proposal
One thing that could help keep the PRs small and manageable would be to switch the APIs as described above but you could avoid having to change all the GroupAccumulators in one PR by returning return a Vec<> of size 1 for most of them.
Then we can make subsequent PRs to switch over other groups accumulators as needed

Actually, I found the slice function is not trivial because the eager computation for the null count.
One way I could think to solve it is make the computation lazy, see arrow-rs pr:
https://github.com/apache/arrow-rs/pull/6155
The alternative may be the way mentioned in this issue.

Maybe I can try it and compare the effect with lazy null count computation.
If @guojidan has not worked for this.

Hi @guojidan, seems there is no update for some months, can I pick this up? I want to try it and compare the effect with lazy null count computation.

Thanks. You can take this. I am working on another project now.


Added desktop folder

Added desktop directory.

No problem, will let you know when I'm done

Seems stalled, let me know when you have more time and we can reopen.

@gaborbernat I've added a fix and the required tests. Please reopen so I can check it against the CI.


Undo bad state

Title:
The undo fails with no elements
Description:
I encountered this error when I tried to delete a column with elements in it, then undo the action.

App Version:
Production app (buildwiththeta)
Device Information:
macOS 13.4 (22F66)
Browser: Google Chrome

Steps to Reproduce:

Create a container with a column inside
Put some widgets inside the column.
Delete the column.
Do the undo action for a few times (CMD+Z for macOS)

Expected Behavior:
I expect the column with previously deleted items in it to be restored.
Actual Behavior:
Currently only the column is recreated without the old elements inside.
This causes errors within the undo state.
Screenshots:
https://github.com/buildwiththeta/buildwiththeta/assets/62183044/b6232e6a-6732-403b-b1ec-0ed6635f74bd

@kasapiniz


doc: fix draft version typo (2012 → 2020)

Hello. Thanks for AJV!
What issue does this pull request resolve?
Doc typo
What changes did you make?
Is there anything that requires more attention while reviewing?

Thank you!


added hover on graph node

Docs for https://github.com/dendronhq/dendron/pull/3060

related to https://github.com/dendronhq/dendron/pull/3060


Add 'name' argument to function

Caught exception:
Traceback (most recent call last):
File "aqt.progress", line 118, in handler
File "aqt.main", line 217, in on_window_init
File "aqt.main", line 316, in setupProfile
File "aqt.main", line 496, in loadProfile
File "_aqt.hooks", line 3809, in call
File "/Users/hoon/Library/Application Support/Anki2/addons21/1889069832/init.py", line 135, in update
create_model(mw.col, nfront, nback, ncss)
TypeError: create_model() missing 1 required positional argument: 'css'

Perfect, thanks!


Interface with MDTraj

Ideally functions should accept MDTraj's Trajectory objects:
import mdtraj as md
from mdentropy.metrics import DihedralMutualInformation

traj = md.load('trajectory.lh5')
dmi = DihedralMutualInformation(types=['phi', 'psi'])
M = dmi.partial_transform(traj)

added in https://github.com/msmbuilder/mdentropy/pull/19


In the example file stream_output_via_api.py, all data from the setting context is None

I reinstalled the metagpt package using git and modified the log function to obtain streaming output.
However, I am currently unable to retrieve any data,
stream_pipe = stream_pipe_var.get(None)
always returns None, even though the context has been set.
When I used the metagpt package installed in June, I was able to get streaming data.

I am using the current version of metagpt and running the stream_output_via_api.py file. However, the streaming data obtained through the log settings is all None.

Using the main branch, the output of stream_output_via_api.py is normal.
 curl http://<IP_ADDRESS>:7860/v1/chat/completions -X POST -H "Content-Type: application/json" -d '{"model": "write_tutorial","stream": true,"messages": [{"role": "user","content": "Write a tutorial about MySQL"}]}'

def stream_pipe_log(content):
print(content, end="")
stream_pipe = stream_pipe_var.get(None)
if stream_pipe:
print(f'{content=}')
stream_pipe.set_message(content)
Hello, thank you for your response. If you log within this method, you'll notice that the content is empty, and I am unable to retrieve any streaming data. However, in the version I installed via Git in June, there was data output.


Filter by userid

Would it be possible to filter the pals by the userid of the owner? Trying to find your lamball out of like 50 lamballs of 5 different players for example is a pain lol.

Coming in next update

v0.48 added this


CNOT gate doesn't work with aqt.sim

Hi there. The following code should return 1.0. It does so with default.qubit, but with aqt.sim it returns a result close to 0.0:
import pennylane as qml

#dev = qml.device("default.qubit", wires=2)
dev = qml.device("aqt.sim", wires=2, api_key="<Insert API Key here>")

@qml.qnode(dev)
def circuit():
    qml.CNOT(wires=[0, 1])
    qml.CNOT(wires=[0, 1])
    return qml.expval(qml.PauliZ(0) @ qml.PauliZ(1))

print(circuit())

drawer = qml.draw(circuit)
print(drawer())

Output:
0.11
 0: ──╭C──╭C──╭┤ ⟨Z ⊗ Z⟩
 1: ──╰X──╰X──╰┤ ⟨Z ⊗ Z⟩

I think I found what the issue is.
In the paper where the formula was taken from in #14, the angle is confusing:

In the picture above, XX(theta) means MS(2*theta), so I think the fix would be to use np.pi / 2 in the decomposition, not np.pi / 4:
        if op_name == "CNOT":
            self._append_op_to_queue("RY", np.pi / 2, [device_wire_labels[0]])
            #self._append_op_to_queue("MS", np.pi / 4, device_wire_labels)
            self._append_op_to_queue("MS", np.pi / 2, device_wire_labels)
            self._append_op_to_queue("RX", -np.pi / 2, [device_wire_labels[0]])
            self._append_op_to_queue("RX", -np.pi / 2, [device_wire_labels[1]])
            self._append_op_to_queue("RY", -np.pi / 2, [device_wire_labels[0]])
            return

Hi @muttley2k,
Thank you for reporting this and for thoroughly looking into a fix here! :slightly_smiling_face: Just to confirm, are results as expected after the change?
Let us know if you'd like to open a pull request with the changes.

Hi @muttley2k,
Thank you for reporting this and for thoroughly looking into a fix here! 🙂 Just to confirm, are results as expected after the change?
Let us know if you'd like to open a pull request with the changes.

@antalszava Yes, locally the results look good to me, now the IsingXX gate as well as UCCSD works as expected, using aqt.sim. It could be a 1-character change, unless unit tests are obligatory (they didn't help before it seems).

I can do the pull request, it will be a 2-character change if the unit test is adjusted too. However, I cannot run the make testcommand on my laptop, unfortunately (nothing to do with the repo, but with my laptop).

Thanks @muttley2k ! Feel free to do the pull request and link this issue there too.


feat: include GitLab project variables

Improves code style with single type of quotes.
Uses pipenv install --deploy --dev on Travis.
Fixes minor issues in .travis.yml

closes #120

@jirikuncar can you deploy this PR on dev so we can test it?


Can't use translate directive on  element

In this case:
   <title translate translate-context="Title">My website title</title>

It is rendered like:
<title translate="" translate-context="Title">
   My website title
</title>

The unexpected  causes the <title> to no longer display in the browser.
If I use a translate filter, I can no longer set a context, and the title appears as {{'My website title' | translate }} until the JavaScript is executed (undesirable behavior).

See https://github.com/rubenv/angular-gettext/issues/191.

Should be fixed with Angular 1.6


Add PaintCode to the list

Checklist

[x] I made something good today 🍰.
[x] I put links in the correct format: [Tool](link) — description.
[x] My description starts with the lower-case letter and ends with the full stop.
[x] I put the tool in the alphabetical order.

Optional

[x] I added the appropriate labels: free, open-source or mac.
[x] I put the tool only to one category.
[x] I followed Contribution Guidelines.

@Tricertops thanks for adding PaintCode!


WIP: New organisation based schema

Rationalises schema to be 3 main models: User, Membership and Organisation
Memberships link Users to Organisations, and define which roles a User performs in each Organisation they belong to
Pundit is used to control access - a User can update their own information but only a User with an admin role in an Organisation can create Users and modify other Users in their Organisation
Access to consuming Applications is defined in terms of organisation types - the Organisations a User belongs_to determines which Applications they can log in to.
Each User can have 1 or more roles for each Application

I really like this set of changes. :+1: from me.
I've commented on one of the changes that looks like it could introduce a SQL injection attack. I've not worked through the 'Add pundit' change 3478965 yet, though. If someone could review that in more detail it'd be great.
I'm not going to merge this - since it's a fairly sweeping change I think it'd be good to have other people on the team comment on it.

That is a LOT of work :clap: , 89 files changed is ridiculous, definitely owed a beer.

Or 7. :beers:


How to cover VictoryScatter's label

Bugs and Questions
Checklist

[x] This is a victory-native specific issue. (Issues that also appear in victory should be opened here)

[x] I have read through the FAQ and Guides before asking a question

[] I am using the latest version of victory-native
v31.0.0

[x] I have checked to make sure that my versions of react-native and react-native-svg are compatible with this version of victory-native. Read about version requirements

[x] I've searched open issues to make sure I'm not opening a duplicate issue

The Problem
I am currently working on a chart using VictoryScatter with a labelComponent.
        <VictoryScatter
          data={players}
          labelComponent={<VictoryLabel dy={17} />}
          labels={datum => datum.jersey}
          size={circleSize}
          style={{
            data: { fill: primaryColor },
            labels: { fill: 'white', fontSize: 14, fontWeight: 'bold' },
          }}
        />

The problem I have is that label is displayed always on top of every point.
Ideally, if two points are displayed in almost same position and they are overlapping I would like next point to cover the previous point's label. Is it possible to achieve?

Reproduction
I'll provide repro soon.

I am closing this, because it also happens in web version of Victory.


Disable Caffe Converter integration tests.

Description
Disable Caffe Converter integration tests. The test is failing because it is not able to download files from caffe server.
This test can be enabled again after https://github.com/apache/incubator-mxnet/issues/8283 is fixed.
 I finished coding on this PR)
[x] All changes have test coverage
[x] For user-facing API changes, API doc string has been updated.
[x] To my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Looks good


Migrating from 0.16.0 to 1.0.12

I'm migratiiong from 0.16.0to 1.0.12
My problem is, that I can't find the old function Utils.restartBluetooth() anymore.
Is this function still available or completely removed?

Hi,
True - there are some breaking changes between pre 1.X  releases and the current versions(1.x and above). One of those changes is Utils removal :/.
It's no longer available as a part of the Estimote SDK.


Support league's 2.0 client

I'm in the process of actually testing this but on the surface and doing some basic unit testing it seems to work.
softonic/guzzle-oauth2-middleware` require the league 2.x branch so this fixes an installation failure trying to install the two packages together.

moved branch and opened new pr #2


Add tier 4 area data

Update to the local restriction data with tier 4 areas.

Some jq sneaky tricks that I'm using to review this:
ruby -rjson -ryaml -e 'puts YAML.load_file("config/local_restrictions.yml").to_json' |
  jq -r '.[] | select(.restrictions[].alert_level == 4) | .name'

Returns a list of 98 LTLAs which are moving into tier 4. I'm just matching them up with the spreadsheet.


pachctl edit/update pipeline fails when setting output_branch that doesn't exist

What happened?:
I used pachctl edit pipeline to change the output_branch on the definition of an existing pipeline to a branch, and got the error "/pachyderm_pfs/branches//  not found"
Same error occurs if I change the spec and update the pipeline.
What you expected to happen?:
Since I can create the pipeline and have it automatically create the output branch, I'd expect edit pipeline or update pipeline to work the same way.
How to reproduce it (as minimally and precisely as possible)?:

Create a pipeline without output branch specified.
Verify it's operating
Use either pachctl edit pipeline or pachctl update pipeline to change the output_branch to one that doesn't yet exist
The update fails with the error message above.

Anything else we need to know?:
It's probably valid to have this fail for updates, but the error message should be more informative.  If we do create the branch, we should make it clear in the docs that the head commit will be blank.  The user should create the branch and set it accordingly.
Environment?:

Kubernetes version (use kubectl version):
Pachyderm CLI and pachd server version (use pachctl version):
Cloud provider (e.g. aws, azure, gke) or local deployment (e.g. minikube vs dockerized k8s):
OS (e.g. from /etc/os-release):
Others:

I think this is a dupe of #4009. Closing in favor of that since it has more info. Feel free to reopen or copy over anything in this issue you feel is missing there.


Issue 5125: stf changes with combined changes from tls changes to pravega cluster and pravega-operator revision in system tests

Change log description
This combines the changes to bring in pravega-operator 0.5.1 to system tests with #5050 pravega pr and the tls based system tests with #5167 pravega pr
Purpose of the change
Fixes: #5025
What the code does
The code merges and resolves all the conficts between:

tls changes to pravega cluster
pravega-operator revision to system tests

How to verify it
The steps to verify remain the same as in the pr 5167

This pull-request is intended only to resolve conflicts with the validated changes from pravega pr 5167 but does not validate pr 5050. Thanks.


Sticky Footer

Is there any way to add a sticky footer?
I can add 2nd table after the original table and make the header contain footer data, but then it scrolls independently horizontally from the original table.

@AppJarBiz there isn't a sticky footer as of yet, but that's a feature I can definitely add!

@mckervinc , have you been able to make any progress on this?

Fwiw, I was hoping for something similar — I need to be able to allow users to add a new item to the table. I can fake a footer at the bottom, and that's probably what I'll do, but it would be ideal if I could make it align with the last row in the table — sort of like how airtable does it (note the + sign).

@AppJarBiz @adampash this should be updated in version 0.4.4. let me know if this is expected

Thanks! However, it's unclear from the documentation how to make the footer a row that lines up with the rows from the rest of the table (for example a row of totals for all the columns)

@AppJarBiz ok I'll re-open. my first guess would be to expose the widths of each column as a property to the footerComponent. let me play around with it

@AppJarBiz ok updated the docs for version 0.4.6. this should be resolved (I hope!)

@AppJarBiz ok so if im understanding you, are you saying that you cannot do the following:
interface CustomFooterProps extends FooterProps {
    rows: any[]
}
const CustomFooter = ({ data, widths }: CustomFooterProps) => {
  // widths is a number[] generated by the library. widths
  // gets calculated for you based on the window size, the table width,
  // and other properties.
  return (

      {widths.map((w, i) => (
        <div className="some-class" style={{width: `${w}px` }}>
          {i === 0 ? calculateTotals(rows) : "Sample Text"}

      )}

}

and then use it like so:
const Component = () => {
  return (

 <CustomFooter rows={data} {...props} />}
    />
  );
}

is that right? after I had pushed 0.4.6, i had the thought of including a footer property inside the column itself, similar to the header. but this should be a good stop-gap until I implement that

@mckervinc the sticky footer works great, thanks!
The syntax I used was:

const FooterCell = ({ val }) => {
   return (
{name}
);
}

const columns = [
		{
			key: "date",
			header: "Date",
			sortable: true,
			footer: (props) => <FooterCell val={`${..myDataRows.length} (Total)`} {...props} />,
		},
                ...

]

        columns={columns}
	stickyFooter={true}
	footerStyle={{	height: "36px",	 ...myFooterStyle}}
        ...

@AppJarBiz im glad!


Documentation for version 0.13.4

Hi Team,
First of all, thank you for your incredible works. I'm using angular-ui 0.13.4 and want to use Modal, but I couldn't find the documentation for that. The samples on https://angular-ui.github.io/bootstrap/ are for latest version, and I got error Unknown provider: $uibModalProvider. I think it due to different versions.
So could you please let me know how to view documentation for older version (0.13.4).
Thanks & Best Regards

Sorry, I found that option to select version.


Fix wrong tiles parameters in Sauvola

Thanks to Robert Sachunsky @bertsky who pointed out the issue.

Adapted from:
https://github.com/DanBloomberg/leptonica/blob/bc21d9f184ce6d9e28e0d37ea4ad38cdbce08297/src/binarize.c#L484
https://github.com/DanBloomberg/leptonica/blob/bc21d9f184ce6d9e28e0d37ea4ad38cdbce08297/src/binarize.c#L885-L889

The CodedQL runner failed at build time.