backtrace [n] should start from the current frame

(I think this is more like a thing to improve instead of a bug)
When using backtrace [n], it always start from the top frame (index 0), even if the user has moved to another frame (e.g. frame 5). For example, given these frames
=>#0    Object#foo(i=0) at target.rb:3
  #1    Object#foo(i=1) at target.rb:5
  #2    Object#foo(i=2) at target.rb:5
  #3    Object#foo(i=3) at target.rb:5
  #4    Object#foo(i=4) at target.rb:5
  #5    Object#foo(i=5) at target.rb:5
  #6    Object#foo(i=6) at target.rb:5
  #7    Object#foo(i=7) at target.rb:5
  #8    Object#foo(i=8) at target.rb:5
  #9    Object#foo(i=9) at target.rb:5
  #10   Object#foo(i=10) at target.rb:5
  #11   Object#foo(i=11) at target.rb:5
  #12   Object#foo(i=12) at target.rb:5
  #13   Object#foo(i=13) at target.rb:5
  #14   Object#foo(i=14) at target.rb:5
  #15   Object#foo(i=15) at target.rb:5
  #16   Object#foo(i=16) at target.rb:5
  #17   Object#foo(i=17) at target.rb:5
  #18   Object#foo(i=18) at target.rb:5
  #19   Object#foo(i=19) at target.rb:5
  #20   Object#foo(i=20) at target.rb:5

If we type bt 10, it'll list frame #0 ~ #9, which is expected. But if we move to #9 with frame 9 and then run bt 10, it'll still show #0 ~ #9.
(rdbg) bt 10    # backtrace command
=>#0    Object#foo(i=0) at target.rb:3
  #1    Object#foo(i=1) at target.rb:5
  #2    Object#foo(i=2) at target.rb:5
  #3    Object#foo(i=3) at target.rb:5
  #4    Object#foo(i=4) at target.rb:5
  #5    Object#foo(i=5) at target.rb:5
  #6    Object#foo(i=6) at target.rb:5
  #7    Object#foo(i=7) at target.rb:5
  #8    Object#foo(i=8) at target.rb:5
  #9    Object#foo(i=9) at target.rb:5
  # and 12 frames (use `bt' command for all frames)
(rdbg) f 9    # frame command
=>   5|     foo(i-1)
=>#9    Object#foo(i=9) at target.rb:5
(rdbg) bt 10    # backtrace command
  #0    Object#foo(i=0) at target.rb:3
  #1    Object#foo(i=1) at target.rb:5
  #2    Object#foo(i=2) at target.rb:5
  #3    Object#foo(i=3) at target.rb:5
  #4    Object#foo(i=4) at target.rb:5
  #5    Object#foo(i=5) at target.rb:5
  #6    Object#foo(i=6) at target.rb:5
  #7    Object#foo(i=7) at target.rb:5
  #8    Object#foo(i=8) at target.rb:5
=>#9    Object#foo(i=9) at target.rb:5
  # and 12 frames (use `bt' command for all frames)
(rdbg)

IMO, the debugger should show #9 ~ #18 instead. Because when we jump to a frame, it's natural to use it as a new starting point for follow up operations. And the current behavior is inconvenient because

To see next n frames, the user needs to compute this: bt [current frame index + n].
It also means that the first few frames will always be included in the result, which would be annoying when debugging from deeper frames (like frame 100).

So I think backtrace [n] should take the current frame index into consideration too.
Example
Script
def foo(i)
  if i == 0
    binding.b
  else
    foo(i-1)
  end
end

foo(20)

Command
$ rdbg -e "c" target.rb

Could you survey other debuggers?

byebug always start from the top frame: https://github.com/deivid-rodriguez/byebug/blob/ddad523427d4ccb5c94aa4858b64b4bbf3d0399c/lib/byebug/commands/where.rb#L57-L59
(Only the unreleased master version accepts argument for bt command).
I also tried to observe gdb's bt command, but I just couldn't get it work without the error 😞
[New Thread 0x1c0b of process 3387]
[New Thread 0x1e03 of process 3387]
During startup program terminated with signal ?, Unknown signal.

But from its description:

.... starting with the currently executing frame (frame zero), followed by its caller (frame one), and on up the stack.

and the fact that it supports both n and -n argument, I assume that it starts from the current working frame.
So here's the question: should bt with no argument also start from the current frame? I also think that having -n argument will be useful here.
Perhaps a more complete bt command would look like this?

bt - starting from the current frame and up the stack.
bt [n] - starting from the current frame and up to the next [n-1] frame.
bt [-n] - starting from the current frame and down to the next [n-1] frame or bottom frame.
bt full - starting from the bottom frame and up the stack.
bt full [n] - starting from the bottom frame and up to the [n] frame.

on gdb:
(gdb) bt
#0  0x00007ffff790c55b in kill () at ../sysdeps/unix/syscall-template.S:78
#1  0x0000555555776e48 in rb_f_kill (argc=<optimized out>, argv=0x7ffff72b4048) at ../../src/trunk/signal.c:481
#2  0x00005555557f4d7b in vm_call_cfunc_with_frame (ec=0x555555982f30, reg_cfp=0x7ffff73b3f90, calling=<optimized out>) at ../../src/trunk/vm_insnhelper.c:3037
#3  0x0000555555809da9 in vm_call_method_each_type (ec=0x555555982f30, cfp=0x7ffff73b3f90, calling=0x7fffffffddf0) at ../../src/trunk/vm_insnhelper.c:3639
#4  0x000055555580a674 in vm_call_method (ec=0x555555982f30, cfp=0x7ffff73b3f90, calling=<optimized out>) at ../../src/trunk/vm_insnhelper.c:3750
#5  0x0000555555802fd2 in vm_sendish (block_handler=<optimized out>, method_explorer=<optimized out>, cd=<optimized out>, reg_cfp=<optimized out>, ec=<optimized out>)
    at ../../src/trunk/vm_callinfo.h:349
#6  vm_exec_core (ec=0x555555982f30, initial=11) at ../../src/trunk/insns.def:778
#7  0x0000555555808ba3 in rb_vm_exec (ec=0x555555982f30, mjit_enable_p=true) at ../../src/trunk/vm.c:2211
#8  0x000055555562297b in rb_ec_exec_node (ec=ec@entry=0x555555982f30, n=n@entry=0x7ffff72af928) at ../../src/trunk/eval.c:280
#9  0x0000555555628f7a in ruby_run_node (n=0x7ffff72af928) at ../../src/trunk/eval.c:321
#10 0x0000555555582f53 in rb_main (argc=<optimized out>, argv=<optimized out>) at ../../src/trunk/main.c:47
#11 0x00007ffff78ed0b3 in __libc_start_main (main=0x555555582de0 <main>, argc=5, argv=0x7fffffffe1d8, init=<optimized out>, fini=<optimized out>,
    rtld_fini=<optimized out>, stack_end=0x7fffffffe1c8) at ../csu/libc-start.c:308
#12 0x0000555555582e1e in _start () at ../../src/trunk/main.c:56
(gdb) f 5
#5  0x0000555555802fd2 in vm_sendish (block_handler=<optimized out>, method_explorer=<optimized out>, cd=<optimized out>, reg_cfp=<optimized out>, ec=<optimized out>)
    at ../../src/trunk/vm_callinfo.h:349
349         return cc->call_;
(gdb) bt 3
#0  0x00007ffff790c55b in kill () at ../sysdeps/unix/syscall-template.S:78
#1  0x0000555555776e48 in rb_f_kill (argc=<optimized out>, argv=0x7ffff72b4048) at ../../src/trunk/signal.c:481
#2  0x00005555557f4d7b in vm_call_cfunc_with_frame (ec=0x555555982f30, reg_cfp=0x7ffff73b3f90, calling=<optimized out>) at ../../src/trunk/vm_insnhelper.c:3037
(More stack frames follow...)
(gdb) bt -3
#10 0x0000555555582f53 in rb_main (argc=<optimized out>, argv=<optimized out>) at ../../src/trunk/main.c:47
#11 0x00007ffff78ed0b3 in __libc_start_main (main=0x555555582de0 <main>, argc=5, argv=0x7fffffffe1d8, init=<optimized out>, fini=<optimized out>,
    rtld_fini=<optimized out>, stack_end=0x7fffffffe1c8) at ../csu/libc-start.c:308
#12 0x0000555555582e1e in _start () at ../../src/trunk/main.c:56

on byebug:
Stopped by breakpoint 1 at /mnt/c/ko1/src/rb/ruby-debug/target.rb:9

[4, 13] in /mnt/c/ko1/src/rb/ruby-debug/target.rb
    4: end
    5: def bar
    6:   baz
    7: end
    8: def baz
=>  9:   p :baz
   10: end
   11: foo
   12:
   13: __END__
(byebug) bt
--> #0  Object.baz at /mnt/c/ko1/src/rb/ruby-debug/target.rb:9
    #1  Object.bar at /mnt/c/ko1/src/rb/ruby-debug/target.rb:6
    #2  Object.foo at /mnt/c/ko1/src/rb/ruby-debug/target.rb:3
    #3  <top (required)> at /mnt/c/ko1/src/rb/ruby-debug/target.rb:11
(byebug) f 2

[1, 10] in /mnt/c/ko1/src/rb/ruby-debug/target.rb
    1:
    2: def foo
=>  3:   bar
    4: end
    5: def bar
    6:   baz
    7: end
    8: def baz
    9:   p :baz
   10: end
(byebug) bt
    #0  Object.baz at /mnt/c/ko1/src/rb/ruby-debug/target.rb:9
    #1  Object.bar at /mnt/c/ko1/src/rb/ruby-debug/target.rb:6
--> #2  Object.foo at /mnt/c/ko1/src/rb/ruby-debug/target.rb:3
    #3  <top (required)> at /mnt/c/ko1/src/rb/ruby-debug/target.rb:11
(byebug) bt 2
*** NoMethodError Exception: undefined method `bt' for main:Object

I'm familiar with gdb's manner.


bug: v2 search not working in prod

Context
https://data-sources-v2.pdap.dev → search works as expected
https://data-sources-v2.pdap.io → search clears the search and does not proceed to the next page
console says:
 Failed to load resource: the server responded with a status of 500 ()

https://data-sources-v2.pdap.io/api/search/search-location-and-record-type?state=Pennsylvania&county=Allegheny
{"message": "Please provide an API key in the request header in the 'Authorization' key with the format 'Basic <api_key>'"}
Requirements
ID the bug, and fix it

Let's check back in on this after the following pull request is complete:
https://github.com/Police-Data-Accessibility-Project/data-sources-app-v2/pull/151


[guide] Datamap

Swate / ARCitect now allows to "Add Datamap".
This concept has not been introduced to standard users at all.
I would suggest to

[ ] add an article about data map

[ ] what is it?
[ ] why do we need it?
[ ] how does it relate to other ISA / ARC / CWL concepts and entities
[ ] add example table

[ ] add a guide on creating a data map

I know that it's not all final (see https://github.com/nfdi4plants/ARC-specification/pull/113), but if we offer it to users, we should at least link to a little place holder ("under construction") article.

@Freymaurer ?

To be 100% honest with you, i just followed specification, as @HLWeil is on vacation 😅
It is meant to further describe data files.

Ah great, I missed that in addition to the ARC specification there is another one for ISA-XLSX
https://github.com/nfdi4plants/ARC-specification/blob/main/ISA-XLSX.md#datamap-table-sheets

Maybe use motivation from https://github.com/nfdi4plants/ARC-specification/pull/104?

#474

Currently covered here /content/docs/start-here/06-datamap.mdx.
This is draft: true and won't show in build.


Is there a migration path for System.EnterpriseServices?

We have a lot of code in .NET 4.0 using System.EnterpriseServices. Can you suggest a migration path to .NET 5?

@jeffschwMSFT do you know who the best contact is for answering this question?

One library you might try is Polly, but it really depends on what features you were using in EnterpriseServices.
@Pilchie do you have any advice here?

Also tagging @glennc.  Unfortunately, I'm not very familiar with EnterpriseServices.

EnterpriseServices is COM+ (dynamic com with remoting).  There is not a straight equivalence in .NET Core.  I am seeking the right set of folks to include.


Stackoverflow exception with Animations (when pc sleeps and wakes)

https://github.com/zkSNACKs/WalletWasabi/issues/724

@jmacato

Seems to be already fixed! https://github.com/zkSNACKs/WalletWasabi/issues/724#issuecomment-430498295


minute and second values not adjusted for timezone

I've included a setLocation in my code but when I use the standard functions like minute(), second(), etc, they are showing UTC instead of being adjusted for my timezone.  Do I need to include another line in my code to fix this situation?

I should have read ALL the way down the README... then I would have found the setDefault function, problem solved.


Change MessageDispatcher to have ownership of the batch queue.

Change MessageDispatcher to have ownership of the batch queue.
One of many to try to resolve #4485
Parent is #4545

Closing, going to split all of these up and submit one at a time.


Added a new book section for security programmers

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
I have added CompTIA Security+ books to the list of resources.
Why is this valuable (or not)?
The CompTIA Security+ certification is valuable for individuals looking to pursue a career in cybersecurity. Adding resources related to this certification enhances the repository's value for those seeking to prepare for the exam.
How do we know it's really free?
I have verified that the resources added are freely available.
For book lists, is it a book? For course lists, is it a course? etc.
These are books.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Please read docs/CONTRIBUTING.md guidelines!

Sorry for the misunderstanding.. creating another PR.. please merge that one.. Did some research and came up with that idea to keep my file. Hope it helps you out.. Thank you.. @eshellman


multiply_dimensionwise: mortars etc.

Once https://github.com/JuliaLang/julia/pull/37444 is merged into mainstream Julia, we should revisit the choice of MortarMatrix in DG methods. Currently, we use mortar_forward_upper::SMatrix etc. which basically disables LoopVectorization in @tullio there. Once the performance of getting pointers to views is increased, we should check whether it's better to use mortar_forward_upper::Matrix with LoopVectorization instead.
Xref https://discourse.julialang.org/t/tullio-loopvectorization-pointer-becomes-slow-for-views-because-of-base-memory-offset-for-ndims-5/46178

Sounds good to me! By the way, exceptional work on the performance of Trixi, @ranocha ! :+1:

That's great to hear, @chriselrod! We'll definitely want to make more use of your nice work.

@chriselrod To be a bit more specific: We will operate on large arrays, where we know the first few dimensions (say, three or four dimensions) but the last dimension is only known at runtime. Is that something for the new version of PaddedArrays.jl?

HybridArrays: https://github.com/mateuszbaran/HybridArrays.jl is also a good solution when you have arrays with both static and dynamic indices.

Thanks for the hint, @mateuszbaran! Does HybridArrays interact well with Tullio.jl and LoopVectorization.jl?

No, not at the moment. You can however wrap any array type in HybridArray and use for example Tullio.jl on the wrapped array when it's faster and HybridArray for other operations. Generally HybridArray should have very good performance for operations on statically-sized parts of arrays.

Sorry, I didn't notice the word "well" in your message. HybridArrays shouldn't obstruct any operations from other libraries, so in that sense is should interact well. If you have any problems, let me know. As I wrote, HybridArrays can wrap around any array and you can perfectly well use other libraries to work on the wrapped array and use HybridArray only for a part of your workflow.

The current version already handles that, but as the changes will be extremely breaking.
The ArrayInterface changes should make it easy for other libraries (like HybridArrays) to support LoopVecotrization.

Okay, thanks for the info. We'll consider that. It would just be nice to not have to switch between too many representations of the same data for some general access routines (where HybridArrays.jl can give improvements) and other parts making use of Tullio/LoopVectorization.

The current version already handles that, but as the changes will be extremely breaking.
The ArrayInterface changes should make it easy for other libraries (like HybridArrays) to support LoopVecotrization.

In that case, I think we'll just wait a bit longer until everything has stabilized a bit. Thank you all for your nice suggestions and the interest!

Okay, thanks for the info. We'll consider that. It would just be nice to not have to switch between too many representations of the same data for some general access routines (where HybridArrays.jl can give improvements) and other parts making use of Tullio/LoopVectorization.

HybridArray isn't actually a different representation, it just generates more efficient code for operations on the wrapped array.  That said, having to un-wrap and re-wrap is not great I will definitely look into updating HybridArrays to the new version of ArrayInterface.jl (once its ready).

That said, having to un-wrap and re-wrap is not great I will definitely look into updating HybridArrays to the new version of ArrayInterface.jl (once it's ready).

This is the PR in question.
I'll add docs describing everything needed, but the most important bits would be to support pointer (assuming the wrapped array does), and also providing methods for sdsize, sdstrides (, and sdoffsets) that use ArrayInterface.Static for static sizes known at compile time, and regular integers otherwise.
Performance should be better if you have leading dimensions that are large enough to SIMD.


Possible typescript type improvements

I was thinking about how we could better handle consensus types changing across different forks, fork capabilities, and also blinded and unblinded variants.
We can/should use generics to help switch between both forks/capabilities and blinded/unblinded if we restructure types a bit.
Here's a proposal using an example:
export type BlindedOrFull = "full" | "blinded";

export type ExecutionFork = ForkName.bellatrix | ForkName.capella;

type forkTypes = {
  [ForkName.phase0]: {
    BeaconBlockBody: phase0.BeaconBlockBody;
  };
  [ForkName.altair]: {
    BeaconBlockBody: altair.BeaconBlockBody;
  };
  [ForkName.bellatrix]: {
    BeaconBlockBody: bellatrix.BeaconBlockBody;
    BlindedBeaconBlockBody: bellatrix.BlindedBeaconBlockBody;
  };
  [ForkName.capella]: {
    BeaconBlockBody: capella.BeaconBlockBody;
    BlindedBeaconBlockBody: capella.BlindedBeaconBlockBody;
  };
};

export type BeaconBlockBody<
  F extends ForkName = ForkName,
  B extends BlindedOrFull = "full"
> = F extends ExecutionFork
  ? B extends "full"
    ? forkTypes[F]["BeaconBlockBody"]
    : forkTypes[F]["BlindedBeaconBlockBody"]
  : B extends "full"
  ? forkTypes[F]["BeaconBlockBody"]
  : never; // <- we can exclude non-execution blinded blocks :)

//
export function isBlindedBlockBody<F extends ExecutionFork>(
  body: BeaconBlockBody<F, BlindedOrFull>
): body is BeaconBlockBody<F, "blinded"> { // <- the fork type isn't lost if its known :)
  (body as BeaconBlockBody<F, "blinded">).executionPayloadHeader !== undefined;
  return true;
}

So functions could be gated on different forks or unions of fork names:
export function doSomethingWithBellatrix(body: BeaconBlockBody<ForkName.bellatrix>): void {}

export function doSomethingWithExecution(body: BeaconBlockBody<ExecutionFork>): void {
  body.executionPayload;
}

And functions that deal with blinded or unblinded blocks can also be kept typesafe:
export function doSomethingWithBlindedOrUnblinded(body: BeaconBlockBody<ExecutionFork, BlindedOrFull>): void {
  if (isBlindedBlockBody(body)) {
    body.executionPayloadHeader;
  } else {
    body.executionPayload;
  }
}

And we get nice things that we didn't have before:
function foo (body: BeaconBlockBody<ForkName.eip4844, BlindedOrFull>): void {
  // since the fork is already typed as eip4844, the type assertion returned by isBlindedBlockBody will still have that fork type
  // isBlindedBlockBody doesn't have to return some sort of "allForks" type as is the current pattern or be redefined with per fork variants
  if (isBlindedBlockBody(body)) {
    body.executionPayloadHeader;
    body.blobKzgCommitments
  }
}

What's nice about this is that this easily works with future forks with minimal changes as we add new forks. We'd just add the new fork to ForkName, add that fork to any capabilities / union types (eg: ExecutionFork), and add the fork-specific types to forkTypes.
Using a generic for the fork has better devex since its easier to combine forks into capabilities using a union type. (switching on namespaces isn't allowed :( )
Using a generic for blinded/full blocks is also pretty nice, esp if we set the default BlindedOrFull to "full". It stays out of the way most of the time, until we need it.

As a general look the ergonomics of this proposal are pretty nice


Support different slack channels per aws service

Hi I love what you have built here and recently set up a codebuild notification pipeline with ease. I was hoping there was a way to support sending messages to different slack hook urls on a per service basis, i.e. send codedeploy messages to a different channel. I could work on writing the code myself or if theres a way to do this already then please advise.

I considered something similar but opted against.
Since AWS doesn't charge for defining multiple Lambda functions, it makes a little more sense to deploy the stack multiple times with different channel endpoints.  Personally, I created "notice" / "warning" / "alert" stacks that deliver to different channels, then configured CloudWatch/etc.al. to deliver alerts to the appropriate channel based on the severity of the alarm or message.  This kept the code clean, and makes reviewing alerting configurations in AWS super-easy!
The downside is re-deploying Lambda multiple times after an update to the repo.  I documented a simple multi-deploy pattern in the readme, if that helps!

Very helpful thanks, I went ahead and just set up another lambda. It was quick and easy. Thanks for responding. I might also look at doing something similar to your cloudwatch alerts going to different channels depending on severity.


Unable to deploy to Azure from VSC

Does this occur consistently?
Repro steps:

Action: appService.Deploy
Error type: ETIMEDOUT
Error Message: request to https://management.azure.com/subscriptions/c840b027-4a10-4c79-8473-ca47f200fdad/providers/Microsoft.Web/sites?api-version=2019-08-01 failed, reason: read ETIMEDOUT
Version: 0.20.0
OS: darwin
OS Release: 19.6.0
Product: Visual Studio Code
Product Version: 1.51.1
Language: en

Call Stack
ClientRequest.<anonymous> extension.bundle.js:166:16389
ClientRequest.emit events.js:228:7
TLSSocket.socketErrorListener _http_client.js:406:9
TLSSocket.emit events.js:223:5
emitErrorNT destroy.js:92:8
emitErrorAndCloseNT destroy.js:60:3
processTicksAndRejections task_queues.js:81:21

Please provide repro steps.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


[CodeQuality] Skip inline comment on DataProviderArrayItemsNewLinedRector

array with inline comment should be skipped instead of re-printed, it currently produce invalid change:
     public function provideData(): array
     {
         return [
-            ['content123', 11], // a comment inline
+            ['content123', 11],

Fixed with verify next array item start line is equal with current array item line https://github.com/rectorphp/rector-phpunit/pull/291/commits/0aade184eaaf620717cfdd3079984d3cdcfc9b6a

All checks have passed 🎉 @TomasVotruba I am merging it ;)


[Bug]: locked and lockStatus

Ask your question
hey bropat,
I have two use cases for two different locks and receiving contradicting results, please help me to understand to find out correct attribute between locked and lockStatus. So, checking below information, do you think T8510 is giving wrong information?
Model: T8510 - https://github.com/fuatakgun/eufy_security/issues/349#issuecomment-1199374452
In Locked State;

locked: false
lockStatus: true <-------------this was supposed to be number
In UnLocked State;
locked: false <-------------- this does not change at all
lockStatus: false <-------------this was supposed to be number

While T8520 seems to be giving correct output.
Model: T8520 - https://github.com/fuatakgun/eufy_security/issues/402#issuecomment-1198608651
In Locked State;

locked: true
lockStatus: 4
In UnLocked State;
locked: false
lockStatus: 3

All clients are using 0.9.3 and latest schema version

This is already fixed in the current develop branch of eufy-security-client.

Cool, let's wait for next version then

Fixed in release 0.9.4

Thanks for update on this.
What would be the final state? Are these events (or state attributes) be in line with each other or should i prefer one over another?
If i should rephrase, should i use locked event or lockStatus event to present it to final user?

If i should rephrase, should i use locked event or lockStatus event to present it to final user?

lockStatus can represent more than locked, but unfortunately I have not yet been able to decipher all the values.
I would provide both ;)

Thanks for great work


Slice index out of bounds when parsing upcase table for malformed FS

fn main() {
    let data = [
        235, 82, 144, 78, 84, 70, 83, 32, 32, 0, 0, 0, 0, 0, 0, 128, 32, 128, 0, 255, 15, 0, 0, 0,
        0, 0, 0, 32, 0, 0, 0, 0, 0, 0, 0, 255, 7, 0, 0, 0, 0, 0, 0, 149, 0, 0, 0, 8, 0, 0, 0, 120,
        183, 16, 124, 224, 39, 74, 127, 0, 0, 0, 0, 14, 31, 190, 113, 124, 172, 34, 192, 116, 11,
        86, 180, 14, 187, 7, 0, 205, 16, 94, 235, 240, 50, 228, 205, 22, 205, 25, 235, 254, 84,
        104, 105, 115, 32, 105, 115, 32, 110, 111, 116, 32, 97, 32, 98, 111, 111, 116, 97, 98, 108,
        101, 32, 100, 105, 115, 107, 46, 32, 80, 50, 101, 97, 115, 101, 32, 105, 110, 115, 101,
        114, 116, 32, 97, 32, 98, 111, 111, 116, 97, 98, 108, 101, 32, 102, 108, 111, 112, 112,
        121, 32, 97, 110, 100, 13, 10, 112, 114, 101, 115, 115, 32, 97, 110, 121, 32, 107, 101,
        121, 32, 116, 111, 32, 116, 114, 121, 32, 97, 103, 97, 105, 110, 32, 97, 110, 121, 32, 107,
        101, 121, 32, 116, 111, 32, 116, 114, 121, 32, 97, 103, 97, 105, 110, 32, 46, 46, 46, 32,
        13, 10, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
        0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
        0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
        0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
        0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 255, 255, 255,
        255, 255, 255, 2, 183, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
        0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
        0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
        0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0, 128, 0, 0, 0, 0, 0, 0, 0, 0, 0,
        0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
        0, 0, 0, 0, 85, 170,
    ];
    let mut data = std::io::Cursor::new(data);
    let mut fs = ntfs::Ntfs::new(&mut data).unwrap();
    fs.read_upcase_table(&mut data);
}

Error: thread 'main' panicked at 'range end index 4 out of range for slice of length 0', /home/jess/.cargo/registry/src/github.com-1ecc6299db9ec823/ntfs-0.1.0/src/record.rs:108:9

FYI, I didn't forget about your report, thanks for opening it!
I implemented some parts like NtfsFile::new infallible when earlier called code should have indeed checked that the used sizes are reasonable. I initially didn't handle the case when a malformed filesystem reports zero sizes, but this has been fixed by d3fcffd4244475ad87e80a6b00bb1b9532dccb27.
In one of the next commits, I want to tighten the record and sector sizes even more to reasonable limits to combat OOM situations. While working on that, I also found larger cluster sizes to require slightly different handling in index records, which I also want to tackle.
All of this taken together should make malformed filesystems bail out early and thereby fix this bug.


Potential bug: missed EDS update for headless services

https://github.com/istio/istio/pull/16845 added some new code for headless services. I think this may have introduced a bug. However, I haven't experienced this, just was looking at the code and noticed this:
https://github.com/istio/istio/blame/master/pilot/pkg/serviceregistry/kube/controller/controller.go#L975-L991
Normally we trigger and EDSUpdate with the endpoints, which updates the data structures and triggers a push.
The new code instead just triggers a ConfigUpdate, but never updates the internal EDS data structures, because we return early

We need full push for headless service, because it requires add/delete listeners per service instance.

but never updates the internal EDS data structures, because we return early

This does not matter, as the Cluster type is ORIGINAL_DST rather than EDS.

Why even build the endpoints object in the first place, should we just stop early if it's  headless?

Good catch, will fix it.


Typing fix

Type comments are not prevalent in the codebase, but some type inference can be done automatically (e.g. using class definitions). This type comment is syntactically incorrect in that it causes mypy to throw an error. python 2 backward compatible comments either have to be on a line with a variable assignment, on a line with a function signature, immediately after a function signature, or on each line of a multiline function signature. The existing comment is neither. This small PR moves the comment to right after the function signature, and adds the type for the itr parameter too.
Longer term it would be really nice to have a stub file, if only the auto-generated one, so that mypy can check the types of the classes used in this package.
@astooke

Codecov Report

Merging #147 into master will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #147   +/-   ##
=======================================
  Coverage   22.56%   22.56%
=======================================
  Files         128      128
  Lines        7987     7987
=======================================
  Hits         1802     1802
  Misses       6185     6185

Flag
Coverage Δ

#unittests
22.56% <0.00%> (ø)

Impacted Files
Coverage Δ

rlpyt/samplers/base.py
80.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


code changes required for image size128

Hi,
I wanted to run the code for images of size 128 as input instead of 64.
I see that image size 64 is hardcoded in "train.py" while transforming into tensors.
Is there any other place across the code where I need to change the image size (or other parameters dependent on image size) to run it on 128 size input image.

@loseway12138 You can change the resolution by modifying the data loader and the model definition (in model.py).


Updated thanks reputation regex to include some more multilingual exp…

…ressions

As per the contributing guidelines, don't open pull requests to the master branch:
https://github.com/botlabs-gg/yagpdb/blob/3d93bf65bf71112fee0df77d407e1c3de0e05766/CONTRIBUTING.md?plain=1#L5


Fixed incorrect string/int/float detection on labels

Labels such as "mystring1234" or "1234mystring" were being incorrectly classified as integers. I've corrected the regex to make it insist whole string is a number.

thanks!


Using Edge with MySQL

How can edge be migrated so that we can use MySQL as a backend rather than sqlite?

Hi @sokser
I have written a more general answer to your question in the new documentation.

@sokser was the answer clear for you?

@arocks
I tried it with Postgres, but after sign up and trying to see the Profile page got errors.
https://gist.github.com/anonymous/ce287fe4b7f916ce93d1

Just found the problem, I was using migrate tools instead of syncdb
migrate auth
migrate

working commands:
migrate auth
syncdb


Support event statistics view in Theia

This is the type of categorical view that can be used as base output

[x] Make sure the data is provided through a data provider in Trace Compass
[ ] Support this data type with pie chart in Theia

Pie charts remain, I would suggest they are blocked by D3's progress. @Rodrigoplp-work thoughts?

Exactly! With the structure we are already deploying in the y-axis plus the examples discussed in the "0003-D3-charts" ADR, this would fall perfectly in line for expanding our use of the library.


Integration with django-filer for resizing copy/pasted pictures using the mouse in the editor

This is an enhancement suggestion.
CKeditor allows to resize images using the mouse. However, the size to which it is uploaded to media using filer is the original one.
It would be "nice" to have the size of the image to be recorded or the image to be resized automagically upon saving the plugin.
Thanks!

closed in favour of #285


Struct clone behaviour causes errors with readonly properties

PHP Version: 8.2   | Shopware Version: <IP_ADDRESS>  | Affected area / extension: Platform(Default)

Actual behaviour:
Example scenario: Say an event subscriber has added a struct extension to the ProductEntity on the product detail route (store-api). The extension contains readonly properties holding onto some other data objects. A CMS page is associated, which will cause the product entity struct to be cloned.
Since every struct and entity use the CloneTrait, which defines custom behaviour for cloning Struct objects.
Herein lies the problem: The __clone() code by default clones all associated objects (and arrays). This does not take into account that readonly properties can only be written once. Thus, a PHP error is emitted:
{
    "errors": [
        {
            "code": "0",
            "status": "500",
            "title": "Internal Server Error",
            "detail": "Cannot modify readonly property ExampleStruct::$exampleProperty"
        }
    ]
}

Expected behaviour:
No error occurs. Clones of structs with readonly properties are handled correctly.
Alternatively, usage of readonly properties in such a scenario should lead to a more informative error messages, because finding the source of such an error is not trivial, depending on the code-base.
At least the pitfall should be mentioned in the documentation.
How to reproduce:
use Shopware\Core\Framework\Struct\ArrayStruct;
use Shopware\Core\Framework\Struct\Struct;

class ExampleStruct extends Struct
{
  public function __construct(
    public readonly ArrayStruct $exampleProperty,
  ) {}
}

$instance = new ExampleStruct(new ArrayStruct());
$clonedInstance = clone $instance; // <-- causes the error

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-40100
[created from NEXT-40100, comment 528741]

[public]

Priority changed to: Low

Thank you for your report. After careful consideration, we've concluded that this issue does not require immediate attention or isn't something we can address right now. We prioritize openness and clear communication with our community, so we want to be transparent. We may not resolve all reported issues, and this one might remain unresolved. To stay effective, we periodically clean our backlog, and this issue could be closed in the future.
For now, we'll keep this ticket open to invite further feedback from the community in case we've underestimated its impact and to allow us to revisit it later.
[created from NEXT-40100, comment 530284]


Use of generics returns a confusing error

We should consider returning a neater error message stating that generics aren't supported, instead of the confusing error it currently returns.
Especially since, it's perfectly valid code without the #[near_bindgen] attr.
#[near_bindgen]
impl MyType {
    pub fn method<T>(arg: T) -> T { ... }
}

$ cargo build
   Compiling contract v0.1.0 (/path/to/contract)
error[E0401]: can't use generic parameters from outer function
  --> src/lib.rs:10:28
   |
10 |     pub fn method<T>(data: T) -> T {
   |            ---------       ^ use of generic parameter from outer function
   |            |      |
   |            |      type parameter from outer function
   |            help: try using a local generic parameter instead: `method<T, T>`

For more information about this error, try `rustc --explain E0401`.
error: could not compile `contract` due to previous error

But technically, it's clear what the hurdle will be here. It's understandably tricky disambiguating between generic identifiers and explicitly typed ones. Would be interesting to investigate.

One option is to just disable the declaration of generics in public functions altogether. Unsure if there's a use-case for them anyway. I can't personally think of any patterns that require them.
impl<N> MyType<N> already returns Impl type parameters are not supported for smart contracts.

Yeah just feels like an oversight to me, opened a quick PR


log incoming orderbook data to the console

added a printMessage method that the message returned from the websocket and logs it to the console.
Let me know what you think 💯

Hey @cyzanfar, while this is a great implementation, I am hesitant to commit changes that include unnecessary functions, such as printMessage(). Please move this logic into the "main" method so that the skeleton of the class ispreserved. Let me know if this doesn't make sense!

That makes total sense! Will update it now.

Im on mobile so I am unable to edit messages. Please add your changes into a class that inherits OrderBook that we can use in the documentation as an example. Please see the websocket.py for something similar in the '__ main__' function.

you got it ;)

Sure thing. Let me update that :)

I'm a bit confused. In the WebsocketClient.py onOpen in overwritten in the myWebsocketClient object. How is that different from the WebsocketOrderBook object I wrote ?


Using Option::from, instead of Option::just, to wrap map's function call

if the mapped function returns null, it will become a None, instead of throwing an exception.
Not sure if this is overstepping the map functions responsibility, but I'd rather a None than an Exception.

I'll reopen this if you're interested later.


_identifier

From<EMAIL_ADDRESS>on October 21, 2009 15:05:09
Move children to symbol and remove
Original issue: http://code.google.com/p/information-artifact-ontology/issues/detail?id=65

From<EMAIL_ADDRESS>on November 03, 2009 09:11:10
done

From<EMAIL_ADDRESS>on November 03, 2009 09:23:54
identifier: a symbol that is a name for an entity or class of entities
eg model number names class of instruments

From<EMAIL_ADDRESS>on November 09, 2009 14:20:54
We had a discussion on how to define _identifier, label and symbol during November
3rd call.
The result summarized:

symbol: denotes something. cannot be decomposed into parts that
denote the same thing. Examples: letters, words, chinese characters.
identifier: a symbol that denotes something and is created with the
intend to pick out that something from a larger set. Example: Lot
number, SSN, Pubmed ID
label: a symbol or textual entity that describes an entity

Some points of the discussion:

identifiers should be linked to their objective: identifier would be
a defined class
symbols are unitary information artifact: they cannot be decomposed
into parts denoting the same than the symbol as a whole
a label doesn't have to be unitary, it can be decomposed. A label is
descriptive, and is not always meant to identify something.

Owner: jonathan.rees
Cc: alanruttenberg mcourtot

From<EMAIL_ADDRESS>on November 10, 2009 12:18:16
This looks like progress, thanks.
URIs and file names in hierarchical file systems are created with the intent to pick something out, yet they
can be decomposed into parts.  Even SSNs have independently meaningful subparts.  Does this make them
not identifiers?  (because identifiers are symbols, and symbols can't be decomposed, that is.)
HLA nomenclature has a similar nature: B*010301 has B, 01, 03, 01 as meaningful subparts.
Is a symbol a kind of textual entity? (I guess I could look this up but am lazy)  If so you can just say "a textual
entity" instead of "a symbol or textual entity".  If not there ought to be a natural category subsuming them
both.
Is denotation contextual?  That is: 1553937 can be a lot number or a Pubmed id; is a textual entity the
occurrence of the number 1553937 (an individual), such that you can tell by context or history what's
denoted, or the quality of 1553937 denoting a Pubmed record, so that it denotes the record by its nature, or
what?  If the denoting is inherent, then what is like a symbol, but does not denote (i.e. what does the "denotes
something" in the above description of symbols distinguish symbols from)?  I find this all a bit confusing.
A nice word for "not decomposible" or "without parts" is "atomic" ("a-" + "tomos"), FWIW.

From<EMAIL_ADDRESS>on November 16, 2009 13:51:57
Symbols can be decomposed into parts, however the parts don't denote the same thing
as the whole: HLA B*010301 denotes the HLA allele, while the digits (or groups of)
refer to its type or subtypes.
Anything could be an identifier: identifier would become a defined class "anything
used with the intent to pick out something". You are right however and the identifier
definition shouldn't start with "is a symbol".
Currently symbol and textual entities are siblings in the hierarchy. Any suggestion
as to which class could subsume both? Would that ought to include labels as well?
Maybe we could start there and then try and go down in the hierarchy to precise
symbol/label/textual entity and the identifier classes?

From<EMAIL_ADDRESS>on November 17, 2009 02:16:21
Are there other use cases written down somewhere that I can examine? I'm not sure what the consequences of
the desired definitions are supposed to be. Something about curation consistency and integration with other
(possibly future) ontologies, right?
All textual constructs, symbols or not, have the property that the part and the whole mean different things,
so that's not a useful criterion.
It is going to be difficult to distinguish "symbol", "identifier", and "textual entity" in any way that is actionable.
More border cases: linguistic collocations such as pairs of Chinese characters whose meaning is unrelated to
the meanings of the individual characters; book and article titles (the whole string means the book, which is
not a lawful combination of the meanings of its constituent words); query URIs having contiguous characters
that are combined lawfully, forming a little language.
It might be better to focus on the relationships into which these things enter (or can't enter).
I'm still very disturbed by the use of "denotes" as if it were objective. This makes the terminology seem a bit
cultish. Denotation is not obviously testable, so all arguments over statements of the form "x denotes y" will
end in "because I said so".  Perhaps you have already written down your simplifying assumptions (your theory
of meaning) somewhere, but they're not captured in the above discussion.
You could go down the road of metamodeling (e.g. creating a "denotes" process) and maybe come up with
something interesting, but that seems out of scope for this project. Without that, however, you'll have to be
very up-front about the shortcuts you're taking. If you want to take "unmodeled" or "let the user figure it out"
or "whatever is common usage" or "because I say so" approaches, that's OK, if you put your cards on the table.
Otherwise this talk of "denotation" is just mysticism, or at best a naive entry into a long and bloody
conversation in philosophy (Russell, Quine, Skinner, ...) that is at best unresolved.

From<EMAIL_ADDRESS>on November 23, 2009 15:25:37
Current summary of email discussion regarding "CRID" proposal
As OBI development is driving the initial use case of 'identifier', I want to propose
a more limited definition of 'centrally registered identifier' (CRID for short in
what follows). I believe the approach outlined below will address all that OBI needs.
It does not however attempt to cover all ways of identification. I am re-using
multiple ideas raised by others on this thread before.
Definitions:
creating CRID system =def: a planned process with the objective to establish a system
that allows to refer to specific entities of a certain kind and store information
about them, by establishing a CRID registry and plan specifications for the process
of 1) assigning a CRID and 2) looking up a CRID.
Example: Introduction of a personal identity card system in the UK. OBI agreeing on
using a sourceforge for issue tracking, and registering a sourceforge account for
that purpose. Agreeing on an alphanumeric format for strings which will be tracked in
an Excel sheet to track blood samples received for use in bjoern's swine flu study.
assigning a CRID =def: a planned process in which an a new CRID is created and stored
in the CRID registry along with information about an entity associated with the CRID.
example: A journal publisher receiving a Pubmed ID for a newly published article. Me
applying for license plates for a new car. A manufacturer assigning a batch number to
the antibodies produced today.
looking up a CRID=def: a planned process in which a request to the CRID registry is
made to return the information associated with the CRID.
example: Searching for a pubmed ID on http://pubmed.org . Looking up the date in an
Excel sheet which a blood sample arrived based on a donation ID. A policeman looking
up the registration information associated with a vehicle license plate.
CRID registry =def: a bearer of an ICE comprising a list entries pairing a CRID with
an associated ICE
Example: Piece of paper, Excel sheet, MySQL database
CRID =def: an ICE which specifies a CRID registry and allows obtain information about
an entity associated with the CRID.
Example: 'Pubmed ID:12345'. 'ISBN:12345'.
Counter example: '12345'
Some notes:

A CRID system implicitly defines a 'denotation relation' between the CRID and the
entity about which information is stored. This denotation requires to specify both
the CRID and the CRID system in which it was created.

CRIDs can be used outside without looking up the CRID in the registry, e.g. in JR's
example for railroad car barcodes used by a machine to see if it already loaded a
certain car. In such secondary uses, it is assumed though that the CRIDs are well
formed (e.g. unique, readable in a certain format), which enables their use to 'denote'.

No claim is made that the information stored about the entity associated with the
CRID allows to identify it (e.g. a batch number won't help you find where a produced
antibody is now).

The CRID approach may be useful to less formal identifiers such as names: 'Birth
certificates' are CRIDs. Normal use of human names can be taken as a shorthand for a
CRID.

Some comments (copied from emails addressing directly the proposal - apologies if I
missed something)

MC: While I believe in most cases we will indeed have entities of the same kind I
am not sure this is a requirement for the system.
BP: True, there could be CRID systems where limitations to certain entities is not
useful, and it can be removed from the definition. It will be a useful way to
distinguish different subclasses of CRID systems, e.g. those identifying individual
human beings vs. those identifying railcars etc.

MC: I also believe we refer to entities or collection of entities, e.g. for lot
number.
BP: I was assuming a 'collection of entities' is_a entity.

MC: Maybe even add nomenclatures, classifications... (to example of usages for
CRID system)
BP: I was thinking about that, and agree that they fit, but hesitated putting them in
as examples, as they sound like 'ontologies', and that would alert Alan's instincts
against self-reflection.

MC: I think the CRID can be created prior to the assignement. I can create a
barcode system, print out 10 000 tags and stick them on my products only later on. I
would put CRID creation within the system creation. Similarly for storage - assigning
a CRID is really only associating  the existing identifier to the entity it denotes.
I thought maybe that is where the role/function of the CRID is realized?
BP: I thought we would only call something a CRID that is recorded in the registry
with an association to an entity. In that view, the printed out tag is not a CRID
yet. Similarly, a number that confirms to the credit card format is not a credit card
number until it is registered and can be used for transactions.

DN: I like the definition of CRID, especially as it speaks to the point that  an
identifier cannot uniquely identify something without knowing the system within which
it works.  My reading of the CRID definition is that it does imply an identifier
points to a unique entity.  Is this correct?

BP: I can see three possible senses of 'unique'. My thoughts:

the same CRID cannot be assigned twice. So there is a unique entity that it is
associated with it during assignment

that entity can be a set of things. For example, a CRID can point to a batch of
antibodies produced that is split up into multiple aliquots which have one batch number.

it is possible that the same entity gets two CRIDs assigned. In many CRID systems,
it is desired to avoid this, and it is part of the plan specification not to assign a
new CRID to an entity that has already been assigned one. At the same time, many CRID
system realize that duplication errors happen quite a bit, and have explicit ways of
'merging' two CRIDs that are found to be duplicates (e.g. most issue tracking system;
we even found several duplicate pubmed IDs for the same journal article; Barry's RTS
also mentions this specifically). As in OBI the plan specification is always right,
we can distinguish CRID systems that guarantee single IDs per entity, those that have
formal processes to remove them, and those that don't care.

DN: I already assumed the second point, so agreed there.  In fact, I agree
with everything you wrote.  I see no problem in having two CRIDs
assigned to a single entity, only in having two entities assigned to the
same CRID.

BS sent doc on referent tracking system.
BP:The one addition I would have to my previous email is to add a relation

domain: CRID,
range: entity
label: CRID_refers_to
definition: a relation between a CRID and an entity about which information is stored
in the CRID registry associated with the CRID.
MC: what is the difference between "CRID_refers_to" and using the ...

From<EMAIL_ADDRESS>on November 23, 2009 15:25:37
..."denotes" relation?

MC: The more I think about it, the more there seems to be some "identifying"
objective. It would maybe be safer if we were to try and build upon those, I guess
lots of things can be used as identifiers. I liked the very initial proposal which
said identifiers are ICE with a specific objective (making it a defined class)- and I
don't think it is contradictory with the current proposal?

From<EMAIL_ADDRESS>on February 23, 2010 10:16:00
The question of defining CRIDs loosely and therefore allow, under the definition, common names as CRID
systems came up. The counter to this is that common names don't satisfy the unique assignment criteria that a
CRID system would - that two same identifiers not be assigned to two different things.

From<EMAIL_ADDRESS>on October 13, 2010 08:03:36
Note to self/IAO - An alternative to CRID are systems designed to create e.g. serial numbers, where each individual gets a unique number, but there is no central registry of all of them. Such identifiers can be later used in registries - e.g. to associate information with the identifiers as in warrantee registration, or in DRM efforts.

From<EMAIL_ADDRESS>on January 26, 2011 20:56:43
see discussion at http://groups.google.com/group/information-ontology/browse_thread/thread/464df25363149068

From<EMAIL_ADDRESS>on January 27, 2011 12:03:59
Proposed updates at this point:

add a restriction to CRID "has_part some (ICE and is_about some CRID registry)" to match the textual definition "An information content entity that consists of a CRID symbol and additional information about which CRID registry it belongs." (there is already a restriction has_part some CRID symbol)
add restriction CRID_symbol part_of some CRID
add some examples to CRID/CRID symbol based on the discussion thread to clarify, such as "OGMS_000031 is not a CRID symbol if it is not part_of a larger piece of information communicated that identifies the CRID. Writing OGMS_000031 on the IAO mailing-list is a CRID symbol, as from all kinds of previous communication the intended information recipients know how to interpret the 'OGMS' part." (adapted from Bjoern)


Translate between Core.Nothing and Core.Void

@timholy added code to perform special serialization of Nothing/Void, but we're still reconstructing these types when reading JLD files created with older versions of JLD instead of mapping them to their new names (e.g. JuliaLang/julia#10289). We could do this with a regexp hack like what we have for UInt, but it would be better to have a general solution that doesn't have the potential to cause problems with user-defined types. The same probably holds for String/AbstractStrings. Related to #204.

Yes, I've been also thinking we need a general translation infrastructure. Might also help with #209.


Add tests cases for, and fix, #513

This patch addresses both leaks discussed in #513.
First leak
The changes in https://github.com/migueldeicaza/SwiftGodot/commit/d1d6ca47f0fc5bb16948017e1a6b2c6c8838b3b5 made several Geometry2DTests fail.
The actual unbalanced retain causing the leak in VariantRepresentable.swift:
extension ContentVariantRepresentable {
    public init? (_ variant: Variant) {
        guard Self.godotType == variant.gtype else { return nil }

        var content = Self.zero
        withUnsafeMutablePointer(to: &content) { ptr in
            variant.toType(Self.godotType, dest: ptr) // <-- THIS IS THE UNBALANCED RETAIN
        }

        self.init(content: content)
    }
}

The variant.toType function calls into Godot to copy out the built-in type stored in variant. If that built-in type has an internal reference count, Godot increments it. Then, the call to self.init(content: content) increments the count again. The retain performed by variant.toType is never balanced.
This patch fixes the bug by generating a new init(alreadyOwnedContent:) initializer for each Godot builtin class type. This needs to be on the builtin wrappers (like GDictionary, GArray, etc.), rather than on Variant which is where https://github.com/migueldeicaza/SwiftGodot/commit/d1d6ca47f0fc5bb16948017e1a6b2c6c8838b3b5 put it.
This patch also adds a test case to check for the leak by looking at Godot's MEMORY_STATIC performance counter, which is only enabled if Godot was built with DEBUG_ENABLED.
Second leak
This patch adds a test for the second leak described in https://github.com/migueldeicaza/SwiftGodot/issues/513, and fixes the leak.
Without this patch, the leak happens because Variant.asObject has an unneeded call to rc.reference() which increments the RefCounted object's reference count. As far as I can tell, nothing balances this increment.
Variant.asObject calls lookupObject(nativeHandle:), which returns a Swift wrapper for an object whose reference count has already been incremented (if the object is RefCounted). The Swift wrapper balances that increment with a decrement in its deinit.

You are absolutely right about these observations, I ran into these while cooking the tests, and noticed the 2D tests fail and spent some quality time trying to figure it out, when I saw that I decided to check on the bug again and saw your analysis.
Love your patch, and the additional insight on the differences on those built-in types.


Ability to batch move documents/images to collections

Is your proposal related to a problem?

Consider an existing project with many documents and images. At some point the admins/editors realize that they should make use of Wagtails collections feature. The admins/editors will now have to edit every single document and image separately.
Describe the solution you'd like

I would be great if you could apply the same change to multiple documents/images at the same time.
E.g. in the index view each row could have a checkbox on the left. Checking the checkbox activates an action toolbar. The checkbox can be activated for multiple rows at once.
Potential actions for the action toolbar could be "Move to collection", "Delete", "Add tag"
Describe alternatives you've considered

Manually changing each separate document/image is of course possible but can get labor intensive in larger projects that have ignored the collections feature for too long.
It would also be possible to create a one off script to perform the batch action. This requires developer expertise though. Then communicating all the document/image ids from the editors to the devs is basically the same labor that it takes to manually edit each element.

This sounds a lot like #7573 :) Coming in Wagtail 2.15.


Run as a desktop app, how to hide or remove the close button on the browser ？

Hello,
Run as a desktop app, how to hide or remove the close button on the browser  ？@almarklein

That button is out of control of Flexx.


Twilio integration: "+" sign isn't passed for "From" and "To" phone numbers

It generates the situations form log like
"The 'From' number is not a valid phone number, shortcode, or alphanumeric sender ID."
or
"The 'To' number is not a valid phone number."

Add checking if number is starting with "+" and add description to phone number fields that full phone number with country code is required.


Invalid syntax in TSR unit tests

See this build log.

Fixed in the current version. Unfortunately, by removing the offending tests.


[SPARK-12632][Python][Make Parameter Descriptions Consistent for PySpark MLlib FPM and Recommendation]

made changes according to indentation setting  done in intellji

Can one of the admins verify this patch?

ok to test

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/51165/
Test FAILed.


Remove css/*.css from repository

Since they're either built from Sass or moved from node_modules, there's no need to handle them in the repository and could be ignored: the main build script (webext) would add them to packages.
Need to be checked.

Not needed, IMHO.


Adding in sdk fixes, like #1112

I need that config to tweak kind, for nginx. I am not sure that is the problem though.

@blampe do you know what I am doing wrong here?

@rshade run make locally in the root of the repo, that will re-generate all of the checked-in fixtures. It helps us lint things and see what will eventually get pushed to repos.

with needing a blank config is something I am going to need to test on cert-manager, before we apply this.


[bn] Adding shortcode upcoming-releases.html

Adding shortcode upcoming-releases.html in bn version to solve this error

/cc @dipesh-rawat

@reylejano Requesting for approval

/lgtm cancel
This is the wrong way to do it; instead, I'd prefer to see either:

a PR merging main into dev-1.24-bn.1
the localized text for the Bengali patch releases page being a localized version for the merge base of dev-1.24-bn.1
In less technical language: localize an older version of that page that doesn't rely on this missing shortcode

If taking option #2, someone will need to update the patch releases page in Bengali after the localization has launched. This is OK; after any localization launches, there is still lots of work that people can do to improve it.

@sftim Thanks for suggestion , we are going with option 2


GroupRowsBy "Age" works incorrectly work titanic dataset

I am using following code for kaggle titanic dataset:
let titanic = Frame.ReadCsv("c:\\tmp\\titanic.csv")
let byAge = titanic.GroupRowsBy<int>("Age")

And it produces some unexpected results - age in rows keys does not match age in rows values:

I suspect this issue takes place because some age values are missing because this code works correctly:
titanic.DropSparseRows().GroupRowsBy<int>("Age")

I discovered the same behaviour. MWE:
//expected behaviour
Frame.ofValues [ (1,"foo","a"); (1,"bar","b");  (3,"foo","d"); (3,"bar","e"); (4,"bar","f")]
|> Frame.groupRowsByString "bar"

//unexpected behaviour
Frame.ofValues [ (1,"foo","a"); (1,"bar","b"); (2,"foo","c"); (3,"foo","d"); (3,"bar","e"); (4,"bar","f")]
|> Frame.groupRowsByString "bar"

Doh! I see what's wrong now! The latest version is now hosted at: https://fslab.org/XPlot/chart/plotly-line-scatter-plots.html

Fixed by #405


You may need an appropriate loader to handle this file type

I'm using babel-preset-env and taking a look at your code, I'm using the same features throughout my code base.. but somehow I still get this. Any ideas?
./node_modules/react-share-button/components/ShareBtn.js
Module parse failed: Unexpected token (62:6)
You may need an appropriate loader to handle this file type.
|     const gmailURL = `https://mail.google.com/mail/u/0/?view=cm&ui=2&tf=0&fs=1&su=${this.props.text}&body=Check out this awesome property %0A${this.props.url}`
|     return (
|

|         {this.props.shareModalOpen && <Visibility ref={(node) => { this.visibility = node }} />}
|         <a className='sp-tab' href={`whatsapp://send?text=${text}`} onClick={this.whatsappClicked}>
 @ ./node_modules/react-share-button/components/index.js 1:0-33

Please provide more context, lel


Can not extract file accurately use FileHeader when some file has the same fileName

i create SplitZipFiles with some files, and among them some file  has the same file name but from different path：
A/B/C/fileName.txt
A/F/G/fileName.txt
A/B/C/P/fileName.txt
E/B/C/fileName.txt
....
i want to extract  single file,like A/B/C/P/fileName.txt,and i use this api code:
FileHeader fileHeader = zipFile.getFileHeader("fileName.txt");
zipFile.extractFile(fileHeader, parentFile.getAbsolutePath(), mergeName);
but i got a not correct file.i also use the full path to get FileHeader:
FileHeader fileHeader = zipFile.getFileHeader("A/B/C/P/fileName.tx");
zipFile.extractFile(fileHeader, parentFile.getAbsolutePath(), mergeName);
but they returns null,i debug the source code，and i found Zip FileHeader only show file names,her like this:

how can i extract  single file  accurately? i really need your help,may you give me some helpful ideas？thank you very much

I was successfully able to extract with the scenario you mentioned. This is the structure of the files in the zip file:

I generated the zip file like this:
ZipFile zipFile = new ZipFile("Test.zip");
zipFile.addFolder(new File("FOLDER_TO_ADD"));

I attached the generated zip file. I extracted the text file with this code:
ZipFile zipFile = new ZipFile("Test.zip");
FileHeader fileHeader = zipFile.getFileHeader("test/a/b/123.txt");
zipFile.extractFile(fileHeader, OUTPUT_DIR);

Test.zip

I was successfully able to extract with the scenario you mentioned. This is the structure of the files in the zip file:

I generated the zip file like this:
ZipFile zipFile = new ZipFile("Test.zip");
zipFile.addFolder(new File("FOLDER_TO_ADD"));

I attached the generated zip file. I extracted the text file with this code:
ZipFile zipFile = new ZipFile("Test.zip");
FileHeader fileHeader = zipFile.getFileHeader("test/a/b/123.txt");
zipFile.extractFile(fileHeader, OUTPUT_DIR);

Test.zip

yeah,thanks for your response,i will show my prroject code:
    because i have many files，i split the zip file,like this:

    ZipParameters zipParameters = new ZipParameters();
    zipParameters.setEncryptFiles(true);
    zipParameters.setCompressionLevel(CompressionLevel.FAST);
    zipParameters.setIncludeRootFolder(false);
    zipParameters.setSymbolicLinkAction(ZipParameters.SymbolicLinkAction.INCLUDE_LINK_AND_LINKED_FILE);
    zipParameters.setEncryptionMethod(EncryptionMethod.ZIP_STANDARD);

    ZipFile zipFile = new ZipFile(packOutputFile, keyword.toCharArray());
    zipFile.setRunInThread(true);
    zipFile.createSplitZipFile(backFileList, defaultZipParameters, true, SPLIT_SIZE);

    note: the parameters passed "backFileList"  all is the files， not contains folder
    and here is output zip file,the encryption password is 111111,you can input password to unzip the test file.

    so,whether i got a wrong step code? please help me correct the wrong code settings, thank you very much.

1658803896900.zip
:


[BUG] tslearn version 0.6 introduces torch dependency

Describe the feature or idea you want to propose
tslearn (used in clustering) introduced new dependencies in version 0.6, including torch, which we are not ready to include as yet. We have pinned it at <0.6,
https://pypi.org/project/tslearn/
Describe your proposed solution
this issue is here to just remind us of this, and to explore removing the bound when we include torch as a softdep.
Describe alternatives you've considered, if relevant
No response
Additional context
No response

torch is now a regular soft dependency I believe with the addition of Hydra and Quant.


Add an explicit license, attribute to Ben Gamari

Relates to issue #5
I copied the license (BSD-3) from the Dhall prelude, but attributed it to Ben Gamari. Feel free to close the PR if I'm overstepping.

As a past contributor, I agree with adding an exllicit license, assuming it's what @bgamari wants.

As a past contributor, I agree with adding an exllicit license, assuming it's what @bgamari wants.

@bgamari did you have time to check this?

This is a fairly important usage blocker - @bgamari could you please take a look at this and either add a license (this one or some other one) or explicitly mark that it's not for public use? As it stands, this falls into technically-unusable territory, per GitHub's docs:

However, without a license, the default copyright laws apply, meaning that you retain all rights to your source code and no one may reproduce, distribute, or create derivative works from your work. If you're creating an open source project, we strongly encourage you to include an open source license.

Whoops, my apologies, everyone! I somehow missed this.
Thanks, @MasseR !


Support java.util.Properties in Java 8

Hashtable's readObject() uses unsupported ObjectInputStream#readFields(), and so did Properties up to Java 8.
Issue: #24765

@bot-gradle help

Currently, the following commands are supported:
@bot-gradle test <BuildTrigger1> <BuildTrigger2> ... <BuildTriggerN>

A trigger is a special build for this PR on TeamCity, common triggers are:

SanityCheck/CompileAll/QuickFeedbackLinux/QuickFeedback/PullRequestFeedback/ReadyForNightly/ReadyForRelease
Shortcuts: SC/CA/QFL/QF/PRF/RFN/RFR
Specific builds:

BD: BuildDistributions/BuildDocs so that you can preview the generated docs/distribution.
PT: PerformanceTest, all performance tests for Ready For Nightly stage.
APT: AllPerformanceTest, all performance tests, including slow performance tests.
AST: AllSmokeTestsPullRequestFeedback
AFT: AllFunctionalTestsPullRequestFeedback
ASB: AllSpecificBuildsPullRequestFeedback
ACC: AllConfigCacheTestsPullRequestFeedback
ACT: AllCrossVersionTestsReadyForNightly
AFTN: AllFunctionalTestsReadyForNightly
ACTR: AllCrossVersionTestsReadyForRelease
AFTR: AllFunctionalTestsReadyForRelease

@bot-gradle test and merge

queues this PR for testing and merges if all tests pass by:

Creating a merge commit from your PR branch HEAD and the target branch
Running a ReadyForNightly build against the merge commit
When it passes, fast-forward the target branch to this merge commit (i.e. merge the PR)

The merge commit is called a pre-tested commit, which means that it fully tests the integration
of your branch HEAD and latest master, instead of only testing your branch HEAD.
@bot-gradle cherrypick to <branch>

Cherrypicks the current PR to another branch.
A new PR will be created and a build will triggered automatically if there is no conflict.

@bot-gradle cancel

cancel a running pre-tested commit build or remove it from queue

@bot-gradle clean

clear the conversation history

@bot-gradle help

display this message

To run a command, simply submit a comment.  For detailed instructions see here.

@bot-gradle test QF

I've triggered the following builds for you:

QuickFeedback build

@bot-gradle test ACC

I've triggered the following builds for you:

AllConfigCacheTestsPullRequestFeedback build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you.

Your PR is queued. See the queue page for details.

@bot-gradle test and merge

Pre-tested commit build failed.

I've triggered a build for you.


HDDS-2768. security/SecuringS3.md

What changes were proposed in this pull request?
translation to https://hadoop.apache.org/ozone/docs/0.5.0-beta/security/securings3.html
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-2768
How was this patch tested?
hugo server

LGTM.  + 1.
Thanks @iamabug for the contribution.


Map of France's regions is outdated

The regions of France have been modified in 2016, they now correspond to this: https://en.wikipedia.org/wiki/Regions_of_France
I'd like to contribute by modifying the regions of France to the up to date version but do not understand how the instructions are setup.
I have an SVG file with the new information which I am trying to use in a project. Is there any chance you could explain how you arrive to the instructions string ?

Goodmorning,
I have made a simple flutter app for converting SVG’s to the instruction Strings.
However it is not bug free…
Could you send me the SVG of the update regions? Then I can try to convert it to a working version 😊
Best regards,
Lau
Van: Nael M @.>
Datum: vrijdag, 4 augustus 2023 om 11:22
Aan: simplewidgets/countries_world_map @.>
CC: Subscribed @.***>
Onderwerp: [simplewidgets/countries_world_map] Map of France's regions is outdated (Issue #17)
The regions of France have been modified in 2016, they now correspond to this: https://en.wikipedia.org/wiki/Regions_of_France
I'd like to contribute by modifying the regions of France to the up to date version but do not understand how the instructions are setup.
I have an SVG file with the accurate version which I am trying to use in a project. Is there any chance you could explain how you arrive to the instructions string ?
—
Reply to this email directly, view it on GitHubhttps://github.com/simplewidgets/countries_world_map/issues/17, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALYVEUBDGNA2YA7E7MOCNY3XTS5O7ANCNFSM6AAAAAA3D57PSM.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thanks for the response, here's an accurate version of France's regions.

Could be  great to keep old an new border of France's region :). (Personally, i build an app which use and need the old one).


Inference on custom images

Hi, congratulations on your excellent work!
Would it be possible for you to share the code for running inference on custom images?
Thank you!

Please refer infer_pano_sem.ipynb to provide an idea how to run inference on (single) custom image. You can then extend the code for multiple images.


Detach process

HI,
I already worked with bass.dll, but under Delphi 7 on Win2K, +10 years ago.
I have a problem today with Python, I start on this language.
I created a wx GUI, and when I press a button, an mp3 file is played. Nice!
The problem is that as long as the file is not finished, the button remains pressed and the program freezes (I'm on Mac OS and the OS tells me "This program does not answer")
I must definitely detach the process, but how? with a Thread?
Thanks,
Mickael

Yes, a thread would be good idea. You can download WyliodrinSTUDIO, create a new Visual project and drag the blocks for multimedia. That should show you the code.


The methods/pytorch_net folder is missing files

The methods/STExplainer/pytorch_net folder is missing files, can you add them? Thank you!

Thank you for your interests on our work! You can clone the github repos from https://github.com/DSE-MSU/DeepRobust and https://github.com/tailintalent/pytorch_net to get the source code of DeepRobust and pytorch_net. Hope my answer is help for you. Wishing you an early Merry Christmas!


Convert from .npy to .ckpt format

I just created a repo to convert models from .npy to .ckpt (checkpoint) format.
It would be cool if  we could integrate it someway in the caffe-tensorflow repo. The thing is that I needed to change some lines to use Python 3 and TensorFlow 1.

Is your converter only able to deal with googlenet?

No, you can change the parameters of your network (width, height, channels of the input image) and run it.
I have added another example in the repo to make things clearer. It would be easier if you didn't need to deal with these parameters, but I don't know how to avoid it.


BUG: actor self reference oddity

See:
https://forum.dfinity.org/t/what-is-special-about-actor-wrapper-and-this-motoko/36049

Line 94 in definedness.ml skips the expression in ActorUrlE.. so we ignore any this reference from within an the actorUrlE expression.
Maybe it was generalized from a string lit to an expression at some point but not fixed here


MULTIARCH-4296: Add more pre-checking logic in the webhook to skip pods that will not get a patched nodeAffinity

Changes proposed in this PR:

Improve the code of the webhook by using the Pod model embedding v1.Pod so that we can share further methods among the controller and the webhook
Ensure the pod placement controller also runs the shouldIgnore verification: We remove the scheduling gate for pods that no longer meet the criteria for architecture-aware scheduling due to changes between the webhook patch at creation time and the pod's processing by the pod placement controller.

A pod may be gated but unprocessed when the operator's configuration changes, making the pod ineligible for architecture-aware scheduling based on the new rules.
Alternatively, the pod may have been modified by another webhook in the admission chain (e.g., setting the nodeAffinity for the kubernetes.io/arch label), rendering it unsuitable for further processing by the operator.
In both scenarios, we must not set the nodeAffinity, and the gate should be removed.

Ignore pods that have nodeSelectors/affinity set earlier in the webhook and before even catching pull secrets
Finally, we stop ignoring hypershift—* namespaces. At this stage, it should mostly be a testing effort to validate that hyper shift works. Also, namespaces with hypershift—* are not a constraint for hypershift management clusters.

/test e2e-gcp-multi-operator e2e-gcp-multi-operator-olm

/lgtm

/retest "Red Hat Konflux / multiarch-tuning-operator-on-pull-request"

/test e2e-gcp-multi-operator-olm

/lgtm

/lgtm

/approve


Set up 404 checker

Generate RSS (antwar rss plugin).
Run build with RSS.
Run hyperlink against the build.

Made some progress on this. Waiting for https://github.com/Munter/hyperlink/issues/104 and https://github.com/Munter/hyperlink/issues/105 to be resolved. After that we are good to go with link checking.

Hyperlink 2.5.0 should have what you need now

Yeah, working great. Just setting this up. 👍


x/build/vcs-test: certificate is expiring

The vcs-test.golang.org certificate will expire on July 16th, which suggests renewal is failing.
The most likely reason is that it might be built with an old version of x/crypto/acme/autocert which still uses ACVEv1, which Let's Encrypt recently turned off. In that case we should look into why we didn't get any emails from Let's Encrypt.
Regardless, we maybe should have alerts for expiring certificates and for error logs.
/cc @golang/release

Thanks for catching this.

In that case we should look into why we didn't get any emails from Let's Encrypt.

I believe it's because the autocert.Manager.Email field is simply unset. We should find an email address to use for this, and set it.

Redeploying vcs-test fixed the problem; it was ~2 years old. Leaving this open to track adding the email.

@FiloSottile do you happen to know how to get the email address change to take effect? Autocert doesn't support explicit account updates from what I could see. If I blow away the cache directory and the private key along with it, does that mean a new account will be created with the email associated? I don't want to burn a ton of time preventing a problem that might at worst cause us a little bit of annoyance in a few years.

@heschi Yeah, you can blow away the cache and let it re-register.

http2.golang.org also has expired. It probably needs redeploying as well.

Right you are. Done. (No email there either yet.)

I've reregistered h2demo and vcs-test. Hopefully that does it. I don't see an easy way to view account details using autocert.


Package tidy-up

This pull request mostly just tidies up the package to follow best practices for Python packages.

For versioning we'll use zest.releaser rather than versioneer
Updated package metadata in *setup.py
Added CHANGES.rst, AUTHORS.rst
Move scripts that run the Brie model outside of the package and into a separate folder called scripts
Set up continuous integration on Travis
Added a CONTRIBUTING.rst file that outlines how one can contribute

@anardek I think it would be fine if you added Jaap to the copyright. More importantly, I think, I would add him to the AUTHORS file.
At some point we will have to fill out the README file with a description of the model. That would be a good place to say something about how this code is based on his work.


New url format for RuTube.ru

Could you also add a test for it maybe?

Is this test correct?

:+1: Nice


When considering last_build and build_status on a repo's branch, consider API builds, too

:heart:


added transform-class-properties to fix issue with parcel

I know this is not a final solution but I think it solves this problem for now.

Released as v1.0.1


make a yarn package

add the ability to run yarn add yarn-completion to run this script

@chiptus Just confirming: Are you requesting that this be published to the npm registry so that you can install using yarn add ...?
If so, although I can certainly do that, running yarn add yarn-completion won't add the file to the correct location.
I can perhaps make a package.json script to install the file to the right place however. But having said that, there are a few different ways you can use install it:

Add to /etc/bash_completion.d/
Copy directly to  .bashrc.
Source from file in .bashrc.
Source in the active session.

Let me know. Thanks 👍

@chiptus Updated the readme with two-step install instructions.
Closing this since I haven't heard back. Feel free to reopen if this doesn't work for you. 👍


Verifying a signed message

What's this issue about?
Verifying a self signed message wrong strings.
Expected behavior
Show the name/email of the person that signed the message.
Actual behavior
Show undefined .
Steps to reproduce
Create key, sign a message, verify it.
Environment
macOS Sierra.

was the user part of your "Friends" list?

I was the user. I signed a message, and I verified. That screenshot is the verification part.

Yes , we need to account for this. I know if you add yourself it will work.
If you'd like to submit a PR to fix this that would be great!
On Tue, Oct 25, 2016, 8:47 PM Fastidious<EMAIL_ADDRESS>wrote:

I was the user. I signed a message, and I verified. That screenshot is the
verification part.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/henryboldi/felony/issues/88#issuecomment-256220029,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFu9R9R6Ru5DHZx2fcCIOQSHchPbSGekks5q3qM2gaJpZM4Ke0cj
.

@frankcash if I knew, I would have submitted one to begin with. :-)

I just tried to reproduce this following the steps above and it seemed to work so we should close this, unless someone else can verify it as not working

@ldabiralai "it seemed to work?" This either works, or doesn't. This is still an issue. When someone produces a PR to account for this, that someone will close the issue.

@Fastidious I said it seemed to work because I'm not that well versed in the expected behaviour.
I followed the steps you described above and received this message:

I added my own public key to my contact list and then received this:

To me, both of these behaviours look correct..

@ldabiralai I should not need to be on the contact list to sign a message. Why will I need to be on my contact list? That is exactly what needs to be accounted for, which @frankcash noted on his comment.

Working on this if that's okay ~

npm-debug.log
Unfortunately couldn't even get this to run.. :\


Add Typescript bindings

We should add Typescript bindings to this. Currently we only support Javascript (untyped).

Is this up for grabs? :)

Go for it!
On Mon, Sep 19, 2022 at 2:20 PM maciek_grzybek @.***>
wrote:

Is this up for grabs? :)
—
Reply to this email directly, view it on GitHub
https://github.com/cereallarceny/gradient-path/issues/2#issuecomment-1251443472,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJ44CTF6HDVKXNCY5CKWRLV7C4I5ANCNFSM4H5IO2WA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add defaults to gRPC LoadOptions

According to https://developers.google.com/protocol-buffers/docs/proto3#default, default values should not be serialized:

Also note that if a scalar message field is set to its default, the value will not be serialized on the wire.

This causes things like empty arrays, false values, etc to all come back as null from the mesh as they are not present in the response from gRPC. This can be fixed by passing defaults: true to proto-loader but this results in a config validation warning:

GraphQL Mesh Configuration is not valid:
data.sources[0].handler.grpc.protoFilePath.load should NOT have additional properties

@mjbcopland Thanks!


Fix missing master accumulator in dockerfile

What was wrong?
Default master accumulator wasn't available in final docker image, causing docker images to fail when launching trin.
How was it fixed?
Explicitly copy over the default master accumulator to the expected location.
To-Do

[ ] Add entry to the release notes (may forgo for trivial changes)
[ ] Clean up commit history

Why do we need the default master accumulator in the final docker image?
If it is because Trin fails when running the code and looking for the file, I think it is better to include those files in the binary itself. We can use rust_embed or include_dir crates to achieve this.

Feel free to re-ping me after the change to eg~ rust_embed

closing this due to #468


Fix FIRCLSContextInitData crash

I'm getting a crash during startup using 10.13.0, 10.12.0, 10.3.0 and master.
This happens on an iPad (6th generation) running 16.6.

It happens all the time with our app when the Address Sanitizer is on and Detect use of stack after return is on.

The Problem
In [FIRCLSContextManager setupContextWithReport:settings:fileManager],

A instance of the FIRCLSContextInitData struct is assigned to initDataObj but it's still a struct.
The pointer of it is assigned to initData.
The initData pointer is passed to asynchronous functions. The pointer is captured by not what it points at.
FIRCLSContextInitData returns
When the async functions run they try to access the stack and crash.

The Fix
In this PR I've made FIRCLSContextInitData a class and copy the strings into it instead of storing a pointer to the private buffer inside NSString that UTF8String returns. Being a class it will get captured in the async blocks and dealloc'ed after the last use.
Related tickets.
#8883
#10104
#9254
PR #10750

Thanks @kgrigsby59! Let me know if you run into any issues with signing the CLA. I'll trigger the CI workflows.

2023-08-18 10:24:50.860815-0400 DiamondKinetics[8456:688425] [Firebase/Crashlytics] Version 10.14.0
=================================================================
==8456==ERROR: AddressSanitizer: stack-use-after-return on address 0x00016545cd50 at pc 0x0001012ebea0 bp 0x00016fece570 sp 0x00016fece568
READ of size 1 at 0x00016545cd50 thread T7
    #0 0x1012ebe9c in __FIRCLSContextInitialize_block_invoke_2+0x6d8 (/private/var/containers/Bundle/Application/7B6201BC-4D43-4116-BF87-2EC8466ED5C9/DiamondKinetics.app/DiamondKinetics:arm64+0x100febe9c) (BuildId: 233e923b4bbd3a87b5f79d100334e1cc32000000200000000200000000001000)
    #1 0x106df2c90 in __wrap_dispatch_group_async_block_invoke+0xb8 (/private/var/containers/Bundle/Application/7B6201BC-4D43-4116-BF87-2EC8466ED5C9/DiamondKinetics.app/Frameworks/libclang_rt.asan_ios_dynamic.dylib:arm64+0x3ec90) (BuildId: eab26a7029d83ad09a494b0296e24d4525000000100000000000090000041000)
    #2 0x107923f04 in _dispatch_call_block_and_release+0x14 (/usr/lib/system/introspection/libdispatch.dylib:arm64+0x3f04) (BuildId: 8dab90719e243c99820ad15d63901de732000000200000000200000000061000)
    #3 0x10792579c in _dispatch_client_callout+0xc (/usr/lib/system/introspection/libdispatch.dylib:arm64+0x579c) (BuildId: 8dab90719e243c99820ad15d63901de732000000200000000200000000061000)
    #4 0x1079281b4 in _dispatch_queue_override_invoke+0x5c8 (/usr/lib/system/introspection/libdispatch.dylib:arm64+0x81b4) (BuildId: 8dab90719e243c99820ad15d63901de732000000200000000200000000061000)
    #5 0x107937a00 in _dispatch_root_queue_drain+0x160 (/usr/lib/system/introspection/libdispatch.dylib:arm64+0x17a00) (BuildId: 8dab90719e243c99820ad15d63901de732000000200000000200000000061000)
    #6 0x10793836c in _dispatch_worker_thread2+0xcc (/usr/lib/system/introspection/libdispatch.dylib:arm64+0x1836c) (BuildId: 8dab90719e243c99820ad15d63901de732000000200000000200000000061000)
    #7 0x1e25d9b90 in _pthread_wqthread+0xdc (/usr/lib/system/libsystem_pthread.dylib:arm64+0x1b90) (BuildId: de6f7fd7ca453635af3274bc843f8d2932000000200000000200000000061000)
    #8 0x1e25d971c in start_wqthread+0x4 (/usr/lib/system/libsystem_pthread.dylib:arm64+0x171c) (BuildId: de6f7fd7ca453635af3274bc843f8d2932000000200000000200000000061000)

Address 0x00016545cd50 is located in stack of thread T0 at offset 80 in frame
    #0 0x1012f7680 in -[FIRCLSContextManager setupContextWithReport:settings:fileManager:]+0xc (/private/var/containers/Bundle/Application/7B6201BC-4D43-4116-BF87-2EC8466ED5C9/DiamondKinetics.app/DiamondKinetics:arm64+0x100ff7680) (BuildId: 233e923b4bbd3a87b5f79d100334e1cc32000000200000000200000000001000)

  This frame has 1 object(s):
    [32, 104) 'initDataObj' (line 52) <== Memory access at offset 80 is inside this variable
HINT: this may be a false positive if your program uses some custom stack unwind mechanism, swapcontext or vfork
      (longjmp and C++ exceptions *are* supported)
SUMMARY: AddressSanitizer: stack-use-after-return (/private/var/containers/Bundle/Application/7B6201BC-4D43-4116-BF87-2EC8466ED5C9/DiamondKinetics.app/DiamondKinetics:arm64+0x100febe9c) (BuildId: 233e923b4bbd3a87b5f79d100334e1cc32000000200000000200000000001000) in __FIRCLSContextInitialize_block_invoke_2+0x6d8
Shadow bytes around the buggy address:
  0x0001364cb950: f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5
  0x0001364cb960: f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5
  0x0001364cb970: f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5
  0x0001364cb980: f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5
  0x0001364cb990: f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5
=>0x0001364cb9a0: f5 f5 f5 f5 f5 f5 f5 f5 f5 f5[f5]f5 f5 f5 f5 f5
  0x0001364cb9b0: f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5
  0x0001364cb9c0: f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5
  0x0001364cb9d0: f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5
  0x0001364cb9e0: f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5
  0x0001364cb9f0: f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5 f5
Shadow byte legend (one shadow byte represents 8 application bytes):
  Addressable:           00
  Partially addressable: 01 02 03 04 05 06 07
  Heap left redzone:       fa
  Freed heap region:       fd
  Stack left redzone:      f1
  Stack mid redzone:       f2
  Stack right redzone:     f3
  Stack after return:      f5
  Stack use after scopAddressSanitizer report breakpoint hit. Use 'thread info -s' to get extended information about the report.
e:   f8
  Global redzone:          f9
  Global init order:       f6
  Poisoned by user:        f7
  Container overflow:      fc
  Array cookie:            ac
  Intra object redzone:    bb
  ASan internal:           fe
  Left alloca redzone:     ca
  Right alloca redzone:    cb
Thread T7 created by T0 here:
    <empty stack>

==8456==ABORTING

@ncooke3 I'm having trouble with the CLA. My employer signed the CLA and added me to the authorized contributers with my work email. That email is not my primary GitHub one but is listed in my emails.

@ncooke3 Is the failing check about the name being too long? FIRCLSContextInitData.h If so what would suggest. It's named the same as the original struct which wasn't in it's own file.

@kgrigsby59 Sorry about the trouble with the CLA. Have you worked through the suggestions at https://github.com/firebase/firebase-ios-sdk/pull/11699/checks?check_run_id=16024717651?
We can review and respond to other questions once the PR's CLA is sorted out.

It looks like we need a CLA for the email address -<EMAIL_ADDRESS>

Yes, we're trying. My employer signed the agreement yesterday. And i'm added via a google group. But when I go to the link you provided it says the CLA is still missing.

Can you check if the corporate agreement is still being processed?

We'll check next week. cc: @morganchen12

It looks like the CLA is sorted out now.

Just to verify, your commit email<EMAIL_ADDRESS>is included in the google group associated with the corporate CLA? https://cla.developers.google.com/about#add-contributors

Yes

Is there something I’m supposed to be doing to move this along?

@kgrigsby59 sorry for the delay. Going to run a couple smoke tests and review this today


DBeaver closes on clicking on No button in "Are you sure" window

Description
DBeaver is closing even if option No is clicked in "Are you sure" dialog window (appears on close dbeaver)
DBeaver Version
<IP_ADDRESS>406231636
Operating System
MacOS Sonoma 14.5
Database and driver
No response
Steps to reproduce
Open DBeaver.
Click on close button (x)
Closing dialog window with "Are you sure" appears.
Click on "No"
DBeaver terminates anyway
Additional context
No response

Duplicate https://github.com/dbeaver/dbeaver/issues/34462


Expense media upload

Add expense media file support for receipts to expenses

@kcotugno @ahuth Can I get a look at this P.R. when you guys get a chance?

Looks good


[Do Not Merge] Adding the structure of urlConfiguration in rewriteRuleActionSet

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[X] I have reviewed the documentation for the workflow.
[X] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[X] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Do Not Merge this request until september branch is cut. These changes are in NRP 127 with NRP api 2019-08-01 and we will be merging the changes in September Branch only. This is for early review.

Can one of the admins verify this patch?

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

@zikalino could you pls review this PR?

@zikalino @MikhailTryakhov please review the PR

@zikalino gentle ping

Since those changes are for 2019-08-01, closing this PR for now to avoid confusion for 2019-07-01 release.


Cannot find module '@heroicons/react/solid' or its corresponding type declarations.ts(2307)

I am trying this in a brand new Next.js 13 app with TypeScript, and I've followed the installation guide: https://www.tremor.so/docs/getting-started/installation
When I do:
import { Icon } from "@tremor/react";
import { HomeIcon } from "@heroicons/react/solid";

I get:
Cannot find module '@heroicons/react/solid' or its corresponding type declarations.ts(2307)

In my package.json I have:
{
  "name": "config-server",
  "version": "0.1.0",
  "private": true,
  "scripts": {
    "dev": "next dev",
    "build": "next build",
    "start": "next start",
    "lint": "next lint"
  },
  "dependencies": {
    "@headlessui/react": "^1.7.13",
    "@tremor/react": "^2.1.0",
    "@types/node": "18.15.10",
    "@types/react": "18.0.30",
    "@types/react-dom": "18.0.11",
    "eslint": "8.36.0",
    "eslint-config-next": "13.2.4",
    "heroicons": "1.0.6",
    "next": "13.2.4",
    "react": "18.2.0",
    "react-dom": "18.2.0",
    "typescript": "5.0.2"
  },
  "devDependencies": {
    "autoprefixer": "^10.4.14",
    "postcss": "^8.4.21",
    "tailwindcss": "^3.2.7"
  }
}

Any idea what causes this?

I can solve it with:
yarn add @heroicons/react

and then:
import { Icon } from "@tremor/react";
import { HomeIcon } from "@heroicons/react/24/solid";

So I guess the installation instructions are outdated?

Hey @slhck! Thank you for reaching out. You are right, the installation guide has just been updated!

Thanks for the quick fix!
So I understand it's pinned to<EMAIL_ADDRESS>Because I used the latest version and it seems to work fine.


GH-1878: Display Bot Badge in Auto Complete For Focalboard

Summary

Bot Badge being used in Focalboard. Bot Badge is required to be exported here.
Ticket Link

Required for https://github.com/mattermost/focalboard/pull/1906
Related Pull Requests

Has server changes (please link here)
Has mobile changes (please link here)

Screenshots

Release Note

NONE

@hmhealey can you confirm if that is ok? We need that, but maybe we just want to replicate the styles for this or copy over the component.

This one should be fine. Most of the widgets are pretty simple, so I'm not as worried about breaking them randomly

/cherry-pick release-6.3

Cherry pick is scheduled.


Serializing Illegal IRIs

Using the ValueFactory interface it is possible to create an illegal IRI. However, RIO writers/parsers fail if some of these illegal IRIs are present. The writers/parser should not fail on these illegal IRIs when verify IRI syntax is disabled.

Well, per Postel's Law... perhaps the writer should fail (at least produce a warning), but the parser should be able to process them ?

If the writer fails, the user will have no easy way to recover, which is the case with rdf4j 2.2.

If verify IRI syntax is disabled the parser will (with what's in develop branch) percent encode illegal characters in IRIs


Feature: add the example for using HuggingFace text generation filter

Description
Add the example about the repositories minimum dataset and query the result by using huggingface_text_generation filter.
Additional Context

The screenshot will be updated after https://github.com/Canner/vulcan-sql/pull/266 is released.
Update the Table Question Answering README to add the steps for adding the Hugging Face token

LGTM :)


Code analysis updates for .NET 7

Contributes to #29176.
Base branch is release-dontet-7.

Thanks @BillWagner