Can we handle connections among more than two edges?

Something like "im,jm,km->ijk", or in TN graph
- o -   - o -
       |
       o
       |

This is a legal and reasonable einsum expression,  is such thing possible in current setups with TensorNetwork?

Yes but indirectly. It's called a CopyNode.
nodes =[...]
center = tn.CopyNode(...)
for i, node in enumerate(nodes):
  tn.connect(center[i], node[0])
result = tn.contract_copy_node(center)

But this is I believe the 3rd time time someone had requested this. We really should consider supporting these hyper edges.

Yes but indirectly. It's called a CopyNode.

Thanks, I will have a look at CopyNode approach.

But this is I believe the 3rd time time someone had requested this. We really should consider supporting these hyper edges.

It would be great if such connection can be supported in a more natural way. But I think the main issue here is the lack of good documentations and tutorials. For example, there is no explanation on CopyNode even in docstrings. Currently one has to read the source to figure out what is going on for nearly everything. We need more documentations to cover basics as well as contractor, MPS, and quantum subpackages.

That we very much do. The main issue is that this package is still very new and many of the APIs we have are still changing (we haven't felt comfortable to release a 1.0 yet).
Though you're right, the basic tutorial we have isn't nearly enough for things that are stable. I'll update with some better tutorials tomorrow night.

If the request keeps coming up, I guess I was wrong about not wanting #401 , is that one still our preferred option?

No it really isn't. In fact there are many cases where doing the contraction like above is more expensive than just contracting the CopyNode like a regular node.
The reason we haven't done this is because I made a mistake really early in development, and now all of the code has a REALLY strong dependence on the edges being normal edges (and not hyper edges). You can see this in the code base with all of the edge.node1 and edge.node2 usage.
Let's reopen #401

That not a mistake man, that's good old YAGNI.
Do you think we're just adding technical debt by not addressing it now in the best way possible? Even if it is a lot of work to change it, it's never gonna be less work than now. If we decide that it's a top priority, plan it well and divide work over 2-3 of us, it might not be that bad and give us a lot more flexibility?

Yeah you're right, we should fix it now.
The biggest thing is we need to do is remove the attributes axis1, axis2, node1, node2 on the Edge class and replace them with lists axes and nodes. We also will need to add a batched_tensordot operation on all of the backends. Beyond that, the rest is just patching for all of the methods to accommodate this change.
To make the transition a little easier, we can set the above attributes as @property methods, while having the underlying function just work with the lists. Once all of the methods are transitioned away, we can deleted the @property methods.

Some tutorials have been added around copying/contracting and a simple tutorial around building a TN for quantum circuits. Still not enough, but it's a little better.
I think ultimately we should remove CopyNode in favor of hyper edges, though I'm not sure about the timeline.

It's maybe worth pointing out that although np.einsum lets your write it, "im,jm,km->ijk" is actually ill-formed Einstein notation. While since our "tensors" have no metric it's probably harmless, the desired operation would in fact be written by explicitly including a rank-3 identity tensor, with which CopyNode seems closer in spirit.

what is our current strategy regarding CopyNode like contractions?
In general, such things could be handled using batched contractions followed by summation. The former would be very nice to have as well for ncon.

Even though reading it fills me with pain, supporting this notation in ncon is probably simplifying enough to be worthwhile.

The main problem is that copy tensors are fairly inefficient (See #261).
Really, I think the answer is to just add support for hyper edges #401.

yes, the current CopyNode behaviour is unsatisfactory, and I believe we should remove it completely.
From a conceptual viewpoint such things would be very easy to implement once we have a tensordot function that allows batching.
I have now encountered such cases already several times in my own research, and would find it very convenient to have it. In my case, I would favour an ncon which supports batching

I'm a little nervous about encouraging users to draw invalid tensor
networks. It seems to me that, at least conceptually, if this behaviour is
supported it should be documented and explained as implicitly introducing a
Kronecker delta "tensor" at the junction (e.g. the fusing is accomplished
in the conceptual tensor network by a node instead of an edge; this doesn't
however speak to whether or not the implementation should involve some kind
of Edge behind the scenes). For example, the contraction order becomes
ambiguous otherwise.
It's a little unclear to me what actual floating point operations would
occur here. Would e.g. A_{ab} B_{bc} C_{bd} still reduce to a series of
matrix multiplications, or is the idea to do "the Fortran thing", e.g.
for a..
for b...
for c...
for d...
result[a,c,d] += A[a,b]*B[b,c]*C[b,d]
It's not obvious to me how this gets written as batched matrix
multiplications (I believe you that it can be though).
Another concern is that this is somewhat orthogonal to our present goals
w.r.t. getting PEPS working and so on.
On Tue, Jun 2, 2020 at 10:40 AM Martin Ganahl<EMAIL_ADDRESS>wrote:

yes, the current CopyNode behaviour is unsatisfactory, and I believe we
should remove it completely.
From a conceptual viewpoint such things would be very easy to implement
once we have a tensordot function that allows batching.
I have now encountered such cases already several times in my own
research, and would find it very convenient to have it. In my case, I would
favour an ncon which supports batching
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/TensorNetwork/issues/552#issuecomment-637586938,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAINHBVQRN5UZTLIEJYCFUDRUUFORANCNFSM4MTNRL3A
.

That's right, and I'm not suggesting to do it right away, but we should keep it on the agenda. If this is ever going to be a thing for machine learning then batched contractions are a must.
A frequent case is where one has a tensor network where each tensor has a batch index:
A = np.random.rand(batchsize,D1,D2)
B = np.random.rand(batchsize,D2,D3)
C = np.random.rand(batchsize,D3,D1)

and one wants to contract over D1, D2, D3 and sum over batchsize. Typically this is done
using np.matmul (or tf.matmul):
AB = np.matmul(A,B) #shape (batchsize, D1, D3)
ABC = np.matmul(AB,C) #shape (batchsize, D1,D1)
cost = np.sum(np.trace(ABC))

It's easy to extend this to cases where using np.matmul will be a real pain. ncon could handle
this e.g. by using a batchindex argument
cost = ncon([A,B,C],[[4,1,2],[4,2,3],[4,3,1]], batchindex=[4])

Also note that not all tensors neccessarily need to have a batch index, e.g.
A = np.random.rand(batchsize,D1,D2)
B = np.random.rand(D2,D3)
C = np.random.rand(batchsize,D3,D1)
cost = ncon([A,B,C],[[4,1,2],[2,3],[4,3,1]], batchindex=[4])

should also be possible.

I see. Well that makes sense to me, and the meaning seems clear enough. I
do like making the user repeat themself with "batchindex" to prove the y
didn't just make a mistake. Is this the same thing as a "hyper-edge"?
On Tue, Jun 2, 2020 at 11:54 AM Martin Ganahl<EMAIL_ADDRESS>wrote:

That's right, and I'm not suggesting to do it right away, but we should
keep it on the agenda. If this is ever going to be a thing for machine
learning then batched contractions are a must.
A frequent case is where one has a tensor network where each tensor has a
batch index:
A = np.random.rand(batchsize,D1,D2)B = np.random.rand(batchsize,D2,D3)C = np.random.rand(batchsize,D3,D1)
and one wants to contract over D1, D2, D3 and sum over batchsize.
Typically this is done
using np.matmul (or tf.matmul):
AB = np.matmul(A,B) #shape (batchsize, D1, D3)ABC = np.matmul(AB,C) #shape (batchsize, D1,D1)cost = np.sum(np.trace(ABC))
It's easy to extend this to cases where using np.matmul will be a real
pain. ncon could handle
this e.g. by using a batchindex argument
cost = ncon([A,B,C],[[4,1,2],[4,2,3],[4,3,1]], batchindex=[4])
Also note that not all tensors neccessarily need to have a batch index,
e.g.
A = np.random.rand(batchsize,D1,D2)B = np.random.rand(D2,D3)C = np.random.rand(batchsize,D3,D1)cost = ncon([A,B,C],[[4,1,2],[2,3],[4,3,1]], batchindex=[4])
should also be possible.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/TensorNetwork/issues/552#issuecomment-637642757,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAINHBVXIHG3UCML5XGYQADRUUOD3ANCNFSM4MTNRL3A
.

The hyper edge would connect to multiple tensors and do the batch summation.
The ncon API seems more convenient . Consider this case
A = np.random.rand(batchsize,D1,D2)
B = np.random.rand(D2,D3)
C = np.random.rand(batchsize,D3,D1)
cost = ncon([A,B,C],[[-1,1,2],[2,3],[-1,3,1]], batchindex=[-1]) #shape (batchsize,)

One could in this case connect A and C with a hyper edge, but not contract it.
While there is no fundamental problem with it, I don't quite like that the API defines an object that it doesn't really use in the end.

There would be not need to specify a batchindex, and negative number that appears more than once would be treated as a batch index.
Of course, you may choose to include for error catch, but you don't NEED to have it.

@alewis, your comment on how do you contract A_ab, B_bc and C_bd with b being a hyper edge looks like this:
First, you do a batched outerproduct of A and B together to get
D_bac = A_ab @ B_bc
Next, you do a normal tensordot operation of D and C
Result_acd = D_bac @ C_bd
Rule of thumb: If an index is on more than 2 nodes of the network at that instance of contraction, you treat that index as a batch index and the reset as you normally would.

Indeed, it is redundant apart from error catch. I think physicists vs deep
learning types are likely to have opposite instincts wrt to whether or not
this behaviour is sufficiently natural to require said catch, so it's not
clear to me which is best.
On Tue, Jun 2, 2020 at 2:12 PM Chase Roberts<EMAIL_ADDRESS>wrote:

There would be not need to specify a batchindex, and negative number that
appears more than once would be treated as a batch index.
Of course, you may choose to include for error catch, but you don't NEED
to have it.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/TensorNetwork/issues/552#issuecomment-637717730,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAINHBXVEHTUYTACYJHYPWLRUU6HZANCNFSM4MTNRL3A
.

There would be not need to specify a batchindex, and negative number that appears more than once would be treated as a batch index.
Of course, you may choose to include it for error catching, but you don't NEED to have it.

For batch-contracted indices one still needs to specify it. For uncontracted ones one could leave it, but this would lead to confusing behaviour if the user accidentally puts two identical negative numbers when he meant to use two different ones.

Wouldn't a batch-contracted index just be a negative number that appears 3 times? Or am I misunderstanding this?
Can you give an example martin.

actually you're right, one could leave it

Ultimately, ncon has now come full circle to just np.einsum. :)

Though ncon gives you direct control of the contraction order, so that's still useful.


Destroy

Why destroy-routines was deleted?

I can't remember. I guess it could be added back.

Template.afSelect2.destroyed = function () {
  this.$('select').select2('destroy');
};

When removing from afEachArrayItem, this causes now: Exception from Tracker recompute function: Cannot read property 'destroy' of undefined TypeError: Cannot read property 'destroy' of undefined  at b.define.a.fn.select2.a.fn.select2 [as select2]

I have got the same exception as miri-am too
Exception from Tracker recompute function: Cannot read property 'destroy' of undefined
TypeError: Cannot read property 'destroy' of undefined
    at a.fn.select2.a.fn.select2 [as select2] (http://cdnjs.cloudflare.com/ajax/libs/select2/4.0.0-rc.1/js/select2.min.js:2:26761)


Flex Properties Implementation using SCSS/CSS

Is your feature request related to a problem? Please describe.
Eccentric Touch needs a flex-properties system. Classes with various flex properties (Ex: inline, block, flex, etc).

hello is this issue available for assignment?

Hello @nicoolel
Yes, This issue is open for all. Let me tell you what exactly we need to do.
So basically you need to make CSS/SCSS Class methods to implement all the display properties such as flex, inline-flex, block, etc.
First, go through the codebase and understand the project as this will give you a brief idea about the project. The start writing code for this issue in the src/lib/contents/contents.scss or src/lib/contents/contents.css, whichever you are comfortable in.
Also, Test your methods for this task in the src/lib/contents/contentTest.htm file.
Other than this if you have any doubt, you may ask :)
Good Luck!
Also, You should join the discord server, It'll be easy to communicate over there.
https://discord.gg/jdh2AcYm


Feature/all the capital cities in a continent organised by largest population to smallest

@salismt @ayinke93 @BlessingAyoola @VincyAlex06
Can you merge what you have a moment? Thanks!

Codecov Report
Attention: Patch coverage is 95.23810% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 84.75%. Comparing base (59812b3) to head (dae6269).

Files
Patch %
Lines

src/main/java/com/napier/sem/App.java
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##             master      #91      +/-   ##
============================================
+ Coverage     84.48%   84.75%   +0.27%
- Complexity       81       83       +2
============================================
  Files            11       11
  Lines           464      479      +15
  Branches         29       30       +1
============================================
+ Hits            392      406      +14
- Misses           72       73       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Closing a single workspace tab closes all workspace tabs

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.76.2
OS Version: macOS 13.1

Steps to Reproduce:
Requirement: Enable window.nativeTabs: true in the settings.

Open multiple VSCode workspaces and then merge them via "Window -> Merge all windows"
Close a single Tab by clicking the "x" icon in the tab bar.
ALL tabs/instances/windows will be closed – not just the desired single tab.

Can you please follow the steps in https://github.com/Microsoft/vscode/wiki/Native-Crash-Issues to get at more details around the crash and attach the result here? Thanks!

Can you please follow the steps in Microsoft/vscode/wiki/Native-Crash-Issues to get at more details around the crash and attach the result here? Thanks!

I tried, but there is not crash report file being created. I assume this is not a crash but instead VS Code is being shut down "correctly".

/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

/duplicate https://github.com/microsoft/vscode/issues/176945

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


chore: adding changelog

Description

adding changelog to v0.38.11 release branch

Theres two PR's a bit lower in changelog if you can copy over that i put in my comments

ah missed those, updated the changelog to contain and order all changes


Add Anchorage to fintech

Duplicate of https://github.com/marmelo/tech-companies-in-portugal/pull/193


Creating a new Team with the same name should not propagate IntegrityError

When attempting to create a team with the same name as an existing team within the same organization, the API currently returns a HTTP 500 error. While this correctly identifies this, the error handling can be improved to provide a more user-friendly response.

@msmagnanijr I feel we should not add this specific uniqueness constraint validation in Team serialization, otherwise we would need to do the same for every serialization that has a uniqueness constraint field (or joint fields). Let's find out why only this one yields a 500 error. How do others handle django.db.utils.IntegrityError?

@msmagnanijr  Because in the serializer we introduce organization_id to replace the organization in the model, DRF does not auto apply UniqueTogetherValidator based on the model's uniqueness constraint on organization and name jointly. If you switch to use organization in the serializer, you will get a 400 error with a nicely formatted string {"non_field_errors":["The fields organization, name must make a unique set."]}. This is exactly what we see when we use gateway api for the same test.
Since we will continue to use organization_id the same way as we do to all other serializers for foreign keys, I would follow the same general handling as gateway does, adding the following under this line
# from rest_framework.exceptions import ParseError
# from django.db import IntegrityError
if response is None:
    if isinstance(exc, IntegrityError) or isinstance(exc, FieldError):
        exc = ParseError(exc.args[0])
        response = exception_handler(exc, context)

The response will be a 400 error with data {'detail': ErrorDetail(string='duplicate key value violates unique constraint "core_team_organization_id_name_e81b4a73_uniq"\nDETAIL:  Key (organization_id, name)=(1, test-team) already exists.\n', code='parse_error')}. Unfortunately the message is very technical not user friendly.

@bzwei @msmagnanijr We could also add the following in TeamCreateSerializer under Meta:
      validators = [
          UniqueTogetherValidator(
              queryset=models.Team.objects.all(),
              fields=["organization_id", "name"]  # --> changing org field to 'organization_id' here
          )
      ]

which would return the 400 error message @bzwei  mentioned above:
{
    "non_field_errors": [
        "The fields organization_id, name must make a unique set."
    ]
}

@Dostonbek1 @bzwei thanks! I'll work on it now and ask for a new review.

@Dostonbek1 @bzwei thanks! I'll work on it now and ask for a new review.


[x264] build failure

Host Environment

OS: [Windows 19041]
Compiler: vs2019 16.8.4

building x264 with 'vcpkg install x264 --triplet=x64-uwp' does not work.
PS C:\Users\username\Documents\username\opensource\vcpkg> vcpkg install x264 --triplet=x64-uwp
Computing installation plan...
The following packages will be built and installed: x264[core]:x64-uwp -> 161-0d754ec36013fee82978496cd56fbd48824910b3#15 Detecting compiler hash for triplet x64-uwp...
Could not locate cached archive: C:\Users\username\AppData\Local\vcpkg\archives\06\0661a1f5d0e56d348ea6638e281d4c5d772cf081.zip
Starting package 1/1: x264:x64-uwp Building package x264[core]:x64-uwp...
-- Using cached C:/Users/username/Documents/username/opensource/vcpkg/downloads/mirror-x264-0d754ec36013fee82978496cd56fbd48824910b3.tar.gz
-- Cleaning sources at C:/Users/username/Documents/username/opensource/vcpkg/buildtrees/x264/src/48824910b3-cab15fc14a.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source C:/Users/username/Documents/username/opensource/vcpkg/downloads/mirror-x264-0d754ec36013fee82978496cd56fbd48824910b3.tar.gz
-- Using source at C:/Users/username/Documents/username/opensource/vcpkg/buildtrees/x264/src/48824910b3-cab15fc14a.clean
-- Configuring x64-uwp
-- Using msys root at C:/Users/username/Documents/username/opensource/vcpkg/downloads/tools/msys2/23ab37a7454cafa4
-- Using msys root at C:/Users/username/Documents/username/opensource/vcpkg/downloads/tools/msys2/c809757c94447846
-- Configuring x64-uwp-dbg
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:108 (message):
Command failed: C:/Users/username/Documents/username/opensource/vcpkg/downloads/tools/msys2/23ab37a7454cafa4/usr/bin/bash.exe --noprofile --norc --debug -c "V=1 CPP='cl.exe -E' CC='cl.exe' CC_FOR_BUILD='cl.exe' CXX='cl.exe' AR='lib.exe' LD='link.exe -verbose' RANLIB=':' STRIP=':' NM='dumpbin.exe -symbols -headers' DLLTOOL='link.exe -verbose -dll' CCAS=':' AS='C:/Users/username/Documents/username/opensource/vcpkg/downloads/tools/nasm/nasm-2.15.05/nasm.exe' ./../src/48824910b3-cab15fc14a.clean/configure --build=x86_64-pc-mingw32 --host=x86_64-pc-mingw32 --extra-cflags=-DWINAPI_FAMILY=WINAPI_FAMILY_APP --extra-cflags=-D_WIN32_WINNT=0x0A00 --extra-ldflags=-APPCONTAINER --extra-ldflags=WindowsApp.lib --disable-asm --enable-strip --disable-lavf --disable-swscale --disable-avs --disable-ffms --disable-gpac --disable-lsmash --enable-debug ac_cv_prog_ac_ct_STRIP=: gl_cv_double_slash_root=yes ac_cv_func_memmove=yes --disable-silent-rules --verbose --enable-shared --disable-static --prefix=/C/Users/username/Documents/username/opensource/vcpkg/installed/x64-uwp/debug"
Working Directory: C:/Users/username/Documents/username/opensource/vcpkg/buildtrees/x264/x64-uwp-dbg
Error code: 1
See logs for more information:
C:\Users\username\Documents\username\opensource\vcpkg\buildtrees\x264\config-x64-uwp-dbg-out.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_configure_make.cmake:679 (vcpkg_execute_required_process)
ports/x264/portfile.cmake:35 (vcpkg_configure_make)
scripts/ports.cmake:136 (include)

Error: Building package x264:x64-uwp failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: x264:x64-uwp
Vcpkg version: 2020.11.12--unknownhash

Additionally, attach any relevant sections from the log files above.

Please provide failure log C:\Users\username\Documents\username\opensource\vcpkg\buildtrees\x264\config-x64-uwp-dbg-out.log.
Thanks.

Please provide failure log C:\Users\username\Documents\username\opensource\vcpkg\buildtrees\x264\config-x64-uwp-dbg-out.log.
Thanks.

no x264.exe generated.

no x264.exe generated.

@HppZ That will be fixed in #15769.

@HppZ That will be fixed in #15769.

Duplicated to #15744.

Duplicated to #15744.


Add other ways to suppress style rules

Adds preprocessor directive and category suppression specific to each individual code style rule.
Contributes to #24437.
Preview link.

@Youssef1313 If you have time, can you take a peek?

Thanks Bill and Youssef!


Light theme is default theme, but dark should be default

Confirm. If to clear browsing data, open DIAL, then dark theme is set by default.


python312Packages.idna: 3.7 -> 3.8

Description of changes
https://github.com/kjd/idna/releases/tag/v3.8
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

please update the commit message as well

Is that really an improvement? python311Packages.idna is also being updated, right?.
Is it documented anywhere if python3Packages or python3xxPackages is preferred?

https://github.com/NixOS/nixpkgs/blob/54eb3902561d24b0f2aad1a2ecf0fa0ca7ed95c4/doc/languages-frameworks/python.section.md?plain=1#L2007-L2010

Thanks for the reference!

Closed by https://github.com/NixOS/nixpkgs/commit/c2fe962f1919dd52fe06af4cf18ca3a1623b4780.


fix analysis layout

@xavijam could you take a look?
related - https://github.com/CartoDB/cartodb/issues/7503

@viddo, take a look to this...

I'll move these changes to the pending PR I have, since the changed files have been moved/renamed 👉 #7578


[WIP] ios support

I've just started to work on initial ios support and decided to open PR early to discuss some ios specifics and workarounds. btw code is messy now, excuse me for that, I'll clean it up later.
As turns out ios is very different from other platforms and some hacks are required to make it work.
First lets start with UIApplicationMain, it creates app object and sets up event loop. As far as I understand it's mandatory to use and there is no nice workaround. It blocks so I had to setjmp out of it to return control to glutin. After that on every poll_events.next() we can call CFRunLoopRunInMode to give os some time to process events, then put them in our own queue in delegate and take them from the queue in the poll iterator.
Then there are different kinds of events. Some of them, like touch events, can be processed asynchronously, but events like applicationWillTerminate must be sync, otherwise app will quit and you'll have no control over it. So you have to setjmp out of it too. SDL does the same thing. (not yet implemented)
The last thing is EAGL, Apple's version of EGL. And it's somewhat confusing, because it requires to initialize render buffer and then pass it when swap buffer is called. I haven't started working on this one yet, so I guess I'll grok it later.
(https://developer.apple.com/library/prerelease/ios/documentation/OpenGLES/Reference/EAGLContext_ClassRef/index.html)
I'll appreciate any feedback :)
btw it compiles, event polling works, but there is only one event for now WillEnterForeground. I plan to make it possible to draw something and refactor, hope to make it mergeable soon.
ping @cmr

\o/

The last thing is EAGL, Apple's version of EGL. And it's somewhat confusing, because it requires to initialize render buffer and then pass it when swap buffer is called.

You mean an OpenGL render buffer?

You mean an OpenGL render buffer?
Yes

Status update:
Good news: glium's triangle example works
(https://www.dropbox.com/s/sqpndv4ueadprya/Screenshot 2015-05-28 17.28.47.png?dl=0)
Bad news: textures don't. I think it's something to do with format detection, will investigate. It blocks me from proper testing.

Nice work! Keep it up :smile:

@rozaliev Compatibility with OpenGL ES is on my to-do list. I have failing tests waiting to be fixed.
I'll try to work a bit on this.

@rozaliev Hmm, 2D textures work with my local OpenGL ES 2.0 configuration.
According to @fkaa, XCode should have debugging tools.

@rozaliev Ah, nevermind, doesn't work.

@rozaliev Fixed it in https://github.com/tomaka/glium/pull/916

@tomaka still getting GL_INVALID_OPERATION after creating texture, but format seems to be ok now, thx

XCode should have debugging tools.
it does, but to debug ES context it's required to run app on a real ios device, thus app have to be signed and i have my account still being in process of activation. So I'll have to wait a few days before i have proper tools.

Ah, I forgot that textures width/height must be power of two. Everything works fine. https://www.dropbox.com/s/i17siojxzj2brzt/Screenshot 2015-05-29 03.21.27.png?dl=0
Will add rest of the stuff and refactor tomorrow :)

Added app lifecycle events and touches. The only major thing left is orientation change support.
Anyway, I believe anyone interested in testing this backend should probably start. I have my game running with it, but it might not be enough to catch all bugs.

Are still working on this or can I merge it?
Even if it's buggy, it can't hurt to have at least minimal support.

Unfortunately I've got very busy lately and won't have time to finish it properly, sorry. I'll try to get back to it in ~ month.
Current limitations:

it will only work on simulator and aarch64
support for armv7 will require addition of msg_send_struct! to rust-objc, otherwise it will crash every time you receive small struct like CGRect (on real device)
There is no way for now to handle applicationDidEnterBackground message.  You can't setjmp out of it, because it have to return and you can't setjmp back. You can't simple return from it and handle msg later, because app is stuck in event loop.
There is no support for resizing\rotation
There is no support for custom screen size, it will always be native size of the device.

I've not checked rust-objc lately, so maybe some issues have been fixed.

Even if it's buggy, it can't hurt to have at least minimal support.
Agree, it may also inspire someone to contribute more features. So I i think that you can merge it.

@rozaliev, the next version of the objc crate should support ARM and handle stret properly: SSheldon/rust-objc@8ffe610 :)
Still working on a way to test this functionality out on a device. Let me know if anything looks wrong in my intepretation of ARM's calling conventions.


Please update the dependent version to 3.1.

Rxswift 3.1 has been released, the current dependence of this library and other libraries will be in conflict, please update the dependency information, thank you!

Would be great to update the dependence version of RxSwift asap, can't install it because it requires 3.0.1 when RxCocoa and RxRealm both require 3.1.0 …
Any fix while not updated ?

Version 2.3 brings this update, closing it.


Whistle keys visually conflict

The whistle keys can be assigned to the same key and are so by default.
The options menu will mark that as a conflict, but they keys will work as intended.
This visual conflict should probably be patched out.

Implemented in 1.2.3.


loadtest db entry create

remove old get repo test

Codecov Report

Merging #814 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #814   +/-   ##
=======================================
  Coverage   42.57%   42.57%
=======================================
  Files         134      134
  Lines       10583    10583
=======================================
  Hits         4506     4506
  Misses       5491     5491
  Partials      586      586

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 05cd0b0...d3a75a1. Read the comment docs.


⚠ SolarOscp has degraded performance

In 95c0fd4, SolarOscp ($SOLAROSCP_URL) experienced degraded performance:

HTTP code: 200
Response time: 164 ms

Resolved: SolarOscp performance has improved in 01e9fc3 after 14 minutes.


Prism launcher or polymc does not work

Describe the bug
I launch prism launcher or polymc via flatpack andget the error:
QSocketNotifier: Can only be used with threads started with QThread
qt.qpa.qgnomeplatform: Could not find color scheme  ""
QtLockedFile::lock(): file is not opened
QtLockedFile::lock(): file is not opened
What did you expect to happen?
Running program
Output of rpm-ostree status
State: idle
Deployments:
● ostree-image-signed:docker://ghcr.io/ublue-os/bazzite-deck-gnome:stable
                   Digest: sha256:99f91518ceafb75094bd0e5483672af4d13baa2ed4ffc0b009f1a6ff28226dca
                  Version: 39.20240406.0 (2024-04-07T01:32:16Z)
          LayeredPackages: htop libvirt virt-manager
                Initramfs: '"-I /etc/crypttab /etc/modprobe.d/amdgpu.conf /etc/modprobe.d/deck-blacklist.conf" '

  ostree-image-signed:docker://ghcr.io/ublue-os/bazzite-deck-gnome:stable
                   Digest: sha256:b80881eba5a32958b67e9a2197cf1ce1958d6f19b726582d8f11976fe6d87317
                  Version: 39.20240406.0 (2024-04-06T22:25:27Z)
          LayeredPackages: htop libvirt virt-manager
                Initramfs: '"-I /etc/crypttab /etc/modprobe.d/amdgpu.conf /etc/modprobe.d/deck-blacklist.conf" '

Hardware
Jupiter
Extra information or context
No response

Not able to reproduce here, maybe try a flatpak repair?

flatpak repair

Tried it, didn't help


Use hypre_IntArray

This PR provides a temporary fix to issue #2551, and thus it compiles with hypre-master. It is temporary since it relies on a preprocessor variable (hypre_IntArrayData) to switch behavior from an old state (as in mfem-master) to a new state (in line with hypre-master). It would be nice to replace this variable with the new HYPRE_DEVELOP variables discussed in https://github.com/hypre-space/hypre/pull/472

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#2553
@victorapm
@tzanio
@tzanio + @dylan-copeland + @v-dobrev
09/20/21
⌛due 10/04/21
⌛due 10/11/21

If this is the only change, than we should go ahead and merge it.
@wcdawn, can you confirm this fixes it in your applications?
@victorapm, can you mark it as ready-for-review?

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

LGTM

Another question: why is this specific method, HypreParMatrix::ExtractSubmatrix, being modified here? I don't see this method even mentioned in https://github.com/mfem/mfem/issues/2551 which this PR is supposed to fix.

Another question: why is this specific method, HypreParMatrix::ExtractSubmatrix, being modified here? I don't see this method even mentioned in #2551 which this PR is supposed to fix.

I now see that this was the only compilation error with the latest HYPRE.
@wcdawn, in https://github.com/mfem/mfem/issues/2551#issue-999649190 you mentioned issues in "several places". Did you encounter anything beyond the compilation error addressed here? Maybe some run-time issues?

LGTM

I'm seeing an issue running make test on Lassen when both hypre (https://github.com/hypre-space/hypre/commit/c9b1679970335a770978dbc36d83b6394ecc29b8) and this MFEM branch are built with CUDA.
The method that is modified in this PR is not actually used in any of the tests run by make test, so I'll try hypre v2.22.1 to see if that fails too.

@waynemitchell pointed out that the mapping variable passed to HYPRE_BoomerAMGSetDofFunc is assumed to be on the device in hypre-master. Commit https://github.com/mfem/mfem/pull/2553/commits/1264d9906c5fa9137d71854d9144e6dd24ec9e20 updates MFEM accordingly. Note that it relies on hypre_TMemcpy which has been introduced in hypre 2.14.0. Is that fine?

I'm seeing an issue running make test on Lassen when both hypre (hypre-space/hypre@c9b1679) and this MFEM branch (5a9c15e) are built with CUDA.
The method that is modified in this PR is not actually used in any of the tests run by make test, so I'll try hypre v2.22.1 to see if that fails too.

I got the same failure with HYPRE v2.22.1 -- it is in the run mpirun -np 1 punit_tests, at tests/unit/fem/test_eigs.cpp:261. We'll need to investigate this but not necessarily in this PR.

@waynemitchell pointed out that the mapping variable passed to HYPRE_BoomerAMGSetDofFunc is assumed to be on the device in hypre-master. Commit 1264d99 updates MFEM accordingly. Note that it relies on hypre_TMemcpy which has been introduced in hypre 2.14.0. Is that fine?

For HYPRE built with CUDA we require HYPRE >= v2.22.1, see here: https://github.com/mfem/mfem/blob/9a73c2d94c5ecff6dd64c9f7f575e55da0b250cf/INSTALL#L546-L552 so using hypre_TMemcpy in that section of the code should not be a problem.

FYI, the HYPRE_DEVELOP variables are now available on hypre-master.

LGTM

Merged in next for testing...


python311Packages.pytest-harvest: 1.10.4 -> 1.10.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.pytest-harvest is: Store data created during your pytest tests execution, and retrieve it at the end of the session, e.g. for applicative benchmarking purposes
meta.homepage for python311Packages.pytest-harvest is: https://github.com/smarie/python-pytest-harvest
meta.changelog for python311Packages.pytest-harvest is: https://github.com/smarie/python-pytest-harvest/releases/tag/refs/tags/1.10.5
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.10.5 with grep in /nix/store/5cpg0xnd5lihyqrzkvs07wwlgk3l66a7-python3.11-pytest-harvest-1.10.5
found 1.10.5 in filename of file in /nix/store/5cpg0xnd5lihyqrzkvs07wwlgk3l66a7-python3.11-pytest-harvest-1.10.5

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
python311Packages.mne-python
python311Packages.pytest-harvest
python312Packages.pytest-harvest

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5cpg0xnd5lihyqrzkvs07wwlgk3l66a7-python3.11-pytest-harvest-1.10.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.pytest-harvest https://github.com/r-ryantm/nixpkgs/archive/7756a2249f5ab3059cea888698fee0c17a9f70e7.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/7756a2249f5ab3059cea888698fee0c17a9f70e7#python311Packages.pytest-harvest

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5cpg0xnd5lihyqrzkvs07wwlgk3l66a7-python3.11-pytest-harvest-1.10.5
ls -la /nix/store/5cpg0xnd5lihyqrzkvs07wwlgk3l66a7-python3.11-pytest-harvest-1.10.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 45m and timed out

Maintainer pings
cc @mbalatsko for testing.

Result of nixpkgs-review pr 296598 run on x86_64-linux 1

2 packages failed to build:

python312Packages.pytest-harvest
python312Packages.pytest-harvest.dist

4 packages built:

python311Packages.mne-python
python311Packages.mne-python.dist
python311Packages.pytest-harvest
python311Packages.pytest-harvest.dist


Implement decodeHeader function for Gcp Events Filter

Current GCP events convert filter doesn't have a implementation on decodeHeader function only have a dummy body that always return Continue.
This is the key function that will be implemented in the future.
Please checkout PR1 for detail.

Implement the logic for decodeHeader function, follow the logic below.

For any header-only request -> Continue
For any cloud event unrelated request -> Continue
Otherwise, save the current header for future usage -> StopIteration
Please checkout PR3for detail


Hepatic progenitor cell

Hi as finishing my work on the ASCT table for Liver I would like to request to add the hepatic progenitor cell as human adult counterpart of the oval cell that are bipotential progenitors capable of forming hepatocytes or cholangiocytes in rodent adult liver. The equivalent bipotential progenitors in adult human liver are called hepatic progenitor cells.
There is a difference between hepatic stem cell and hepatic progenitor because the last one can only became hepatocyte or cholangiocyte and nothing else. It may be added  as synonym of the oval cell. My only concern in doing this is that in CL there is oval stem cell and this could create a problem of consistency if stem cell is multi potent

Hi @masciam, thank you for suggesting this new cell type.
Could you create an "Add term" issue, and provide the requested information?
For examples of other "new term requests', you could view closed issues with the "new term request" tag or reach out to me for additional support.

Closing this issue as it has been replaced by #1604.


Image size for texture incorrect on Android

Tested under 2.0 20150930 alpha and 1.1 stable.  On windows 7 the attached program shows the texture image size correctly:
texture info [ImageTexture:512] [Image] 4
render size 64 64

Running under Android on a 2013 Moto-X shows the image as 0x0 in size even though the texture does display correctly.
texture type [ImageTexture:507] [Image] 0
render size 0 0

It seems like get_data() on a texture on Android is returning an Image that does not have any actual data in it.  That attached android_image_test.png file is a zip with a project that replicates the problem.

yes, Opengl ES will not give you back textures from video ram so you can't
retrieve back the image data.
If you want to load image data from disk use
var img = Image()
img.load(¨res://somefile.png")
On Sat, Dec 5, 2015 at 6:44 PM, ratsdiov<EMAIL_ADDRESS>wrote:

Tested under 2.0 20150930 alpha and 1.1 stable. On windows 7 the attached
program shows the texture image size correctly:
texture info [ImageTexture:512] [Image] 4
render size 64 64
Running under Android on a 2013 Moto-X shows the image as 0x0 in size even
though the texture does display correctly.
texture type [ImageTexture:507] [Image] 0
render size 0 0
It seems like get_data() on a texture on Android is returning an Image
that does not have any actual data in it. That attached
android_image_test.png file is a zip with a project that replicates the
problem.
[image: android_image_test]
https://cloud.githubusercontent.com/assets/4614207/11610351/6488d754-9b6e-11e5-828b-e19b1175c4ef.png
—
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/issues/2997.

If you just want the width and height, you can use the get_width() and get_height() calls of the Texture class, instead of trying to use the result of get_data().
Is it a lack of glGetTexImage in ES that creates this limitation?  If so, this suggests an FBO workaround that lets you use glReadPixels.  It might be worth implementing just so that we get consistent behavior on all platforms.

closing this, as it's a technical limitation of GLES2.
@romulox-x The file formats allowed for framebuffers are very limited, and some platforms have limits to the amount of them, this probably is too troublesome/hacky. GLES3 backend should allow to get back this data. Also, it's still possible to use the ImageLoader API


Add Remember Me/Logout Functionality

Like a website, a choice to store or clear the user "cookie" and either sign in automatically or require a new log in.

Commit d97a182a416d789a265d25fb143e636f360774c8 adds logout functionality


Atom: Flutter repo refactor/rename analyzer out of sync

I ran into this when trying to refactor the entire flutter repo. My setup is a little strange in that I've got emacs buffers open on many of the affected files. Hopefully the problem can be reproduced with (just) a clone of the flutter repo.
In packages/flutter/lib/src/rendering/flex.dart, try renaming the MainAxisSize enum. This appears to do the right thing but the analyzer reports an error for every line that uses the new name.

FYI @devoncarew

@pq @bwilkerson

cc @scheglov

Fixed in https://github.com/dart-lang/sdk/commit/662a461988b5097a0e7f48628e4330aaf9447b59

w00t! :metal:

Closing because @scheglov says it's fixed :)


Update sbt, scripted-plugin to 1.4.7

Updates

org.scala-sbt:sbt
org.scala-sbt:scripted-plugin

from 1.4.1 to 1.4.7.
Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt" } ]

labels: library-update, semver-patch

Superseded by #23.


Global search (Ctrl+Shift+F) can't go to the correct line in Jupyter notebook

Applies To

[x] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
The global search is unable to locate and jump to the search results in jupyter notebooks, while the file search tab has no problem:

VS Code Version
Version: 1.95.1 (system setup) Commit: 65edc4939843c90c34d61f4ce11704f09d3e5cb6 Date: 2024-10-31T05:14:54.222Z Electron: 32.2.1 ElectronBuildId: 10427718 Chromium: 128.0.6613.186 Node.js: 20.18.0 V8: 12.8.374.38-electron.0 OS: Windows_NT x64 10.0.22000
Jupyter Extension Version
v2024.10.0
Jupyter logs
No output is available during this process.

Coding Language and Runtime Version
Python 3.12
Language Extension Version (if applicable)
Python Extension v2024.18.1
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
Local

I have the same problem!

Update: I spent many hours trying resolve this issue as it is very inconvenient: (1) uninstalled/reinstalled VSCode (along with all extensions), (2) tried different folders/notebooks, (3) used VSCode web version, (4) created a new User account on my mac to get a clean start. etc.
However, none of the above worked: The search across files returns good results but don't allow me to navigate to the correct cell in a notebook; instead, it gets stuck on the cell where I left off when the notebook is last opened. The VS Code's Developer Tools shows no error.
I finally found a workaround: I reinstalled an older version of VSCode (the Sep 2024 version, specifically), and problem solved.


Provide real-time feedback from Journald logs when updating/creating …

During kube-aws up/update, filtered Journald logs can be printed to stdout.
This feature is configurable in cluster.yaml under the cloudWatchLogging section, and requires cloudWatchLogging to be enabled.
( Default values: )
cloudWatchLogging:
 enabled: false
 imageWithTag: jollinshead/journald-cloudwatch-logs:0.1
 retentionInDays: 7
 realtimeFeedback:
  enabled: true,
  filter:  `{ $.priority = "CRIT" || $.priority = "WARNING" && $.transport = "journal" && $.systemdUnit = "init.scope" }`,
  interval: 60

NOTE: Due high initial entropy, .service failures may occur during the early stages of booting.
By default the filter is configured for .service failures and messages flagged as 'critical'.
See the official AWS documentation for more information.
Since some messages are produced frequently, to avoid excessive spam, an 'interval' parameter is provided.
This 'interval' value determines the time between printing two identical messages to stdout.

The filtering feature is neat 👍

Done 👍

@mumoshu It seems the Travis CI builds keeps failing. Could you point me to the relevant tests? Thanks

@jollinshead Thanks for taking your time 👍 The relevant error seems to be core/root/cluster.go:387: result of fmt.Errorf call not used

Codecov Report

Merging #748 into master will increase coverage by 0.08%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##           master     #748      +/-   ##
==========================================
+ Coverage   34.28%   34.37%   +0.08%
==========================================
  Files          55       55
  Lines        3077     3084       +7
==========================================
+ Hits         1055     1060       +5
- Misses       1863     1865       +2
  Partials      159      159

Impacted Files
Coverage Δ

core/controlplane/config/config.go
54.66% <71.42%> (+0.14%)
:arrow_up:

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 0a693ab...626e64f. Read the comment docs.

@mumoshu Is there anything you would like to be done for the LGTM?

@jollinshead Thanks for checking in!
I've left the last nit - I apppeciate it if you could address it but I'll merge this anyway tommorrow!

@mumoshu Done  👍

@jollinshead Great! Thanks as always for your contribution 🍻
Probably we'll address ankon's comment https://github.com/kubernetes-incubator/kube-aws/pull/748#discussion_r126678970 later

@jollinshead This feature has been super handy for me while developing kube-aws. Thank you again for the awesome work 😄


about readme.md preview...

hi, I have been using this plugin useful.
but, I hope to remove the feature that If I click anywhere within the readme.md area, the area is foled.

oops, I miss that write place...I'm sorry.


Unable to Click on sap.suite.ui.commons.networkgraph.ActionButton

Describe the bug
I encountered an issue where I am unable to click on the sap.suite.ui.commons.networkgraph.ActionButton element.
Instead of pressing the action button, its parent element (sap.suite.ui.commons.networkgraph.Node) gets clicked.
Steps to Reproduce

Use the following selector to locate and press the action button:

    const actionButtonSelector = {
        selector: {
            controlType: "sap.suite.ui.commons.networkgraph.ActionButton",
            properties: {
                icon: "sap-icon://menu"
            }
        }
    }
    const actionButton = await browser.asControl(actionButtonSelector);

    const $actionButton = await actionButton.getWebElement()
    await $actionButton.click()

Actual Behavior:
The parent element (sap.suite.ui.commons.networkgraph.Node) is clicked instead of the intended sap.suite.ui.commons.networkgraph.ActionButton.
Additional Information:
When debugging, I noticed that the _metadata of the control obtained based on the selector contains the following discrepancies:

The className is reported as sap.suite.ui.commons.networkgraph.Node instead of sap.suite.ui.commons.networkgraph.ActionButton.
The id is associated with Node rather than ActionButton (verified through DOM inspection).

Please refer to the attached screenshot for visual representation of these properties.
Expected behavior
The sap.suite.ui.commons.networkgraph.ActionButton should be clickable.
Screenshots

Runtime Env:

wdi5/wdio-ui5-service-version: 1.5.1
UI5 -version: 1.96.0
wdio-version: 7.32.1
node-version: 18.12.1
OS: macOS Monterey 12.6
Browser + Version: Chrome 106.0.0

Please let me know if any further information is required to address this bug

I am not familiar with the NetworkGraph but had a look into this sample. Are you aware of the click order? Like "first click on the node to expand the action buttons and click the buttons secondly"?

Hi @dominikfeininger, yes I am aware, and I respected the order in my test.

Hi @borceacristiann,
maybe the interaction selector could help you here. So maybe try something like this:
    const actionButtonSelector = {
        selector: {
            controlType: "sap.suite.ui.commons.networkgraph.ActionButton",
            properties: {
                icon: "sap-icon://menu"
            },
            interaction: "press"
        }
    }

One question: Why are you using the Webdriver.IO native click instead of the built in press event of wdi5?

Hi @Siolto!
It still doesn't work even if I use the press method.
As I mentioned in the bug description, the problem is that I can't even locate the action button of the node.
The _metadata of the control obtained based on the selector contains the following discrepancies:

The className is reported as sap.suite.ui.commons.networkgraph.Node instead of sap.suite.ui.commons.networkgraph.ActionButton.
The id is associated with Node rather than ActionButton (verified through DOM inspection).

Please refer to the attached screenshot for visual representation of these properties.


Second to last child keeps bottom margin after last child is removed

Repro: https://snack.expo.io/@gfpacheco/stacks-bug

@gfpacheco nice catch, thanks for reporting this issue! :)

@gfpacheco it's been fixed and published in v0.8.2 :)

That's great! Thanks for the quick response, I didn't even have time to try and fix it myself


Info about deprecating High concurrency clusters

In the cluster setup screen, it is called out that high concurrency clusters are being deprecated, and it links to this page but there is no information about the deprecation.

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

ID: 06b57bc7-a97e-947c-442f-7ffbf13aa6cb
Version Independent ID: 9dd5e8a5-ce82-6e15-bcfa-939c431cf709
Content: Configure clusters - Azure Databricks
Content Source: databricks/clusters/configure.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

Thanks for the feedback! We have assigned the issue to the content author to further review this and update the document as appropriate.

There is information here: https://docs.microsoft.com/en-us/azure/databricks/clusters/cluster-ui-preview

Thanks for providing feedback that helps improve our documentation. This issue has been resolved and we are closing the issue.
#please-close

#please-close


[compile bug] there is a compilation error cased by bridge.h

Describe the bug
There is a C++ compilation error on the Docker environment hosted by MacBook Pro.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
can fix by specifying the location of file bridge.h
Additional context
Add any other context about the problem here.

https://github.com/oap-project/gluten/pull/614 fixed this issue.


refactor of ContextDependency to es6

What kind of change does this PR introduce?
refactor
Did you add tests for your changes?
current tests should pass
If relevant, link to documentation update:
N/A
Summary
upgrade ContextDependency to es6
Does this PR introduce a breaking change?
No
Other Information
Depends on:

[x] #3803
[x] #3814
[x] #3815
[x] #3816
[x] #3817

i'll quickly trigger a rebuild. hope thats ok :)

Retriggering travis.

Incredible work. Wonder how much left we have now.


AttributeError: module 'pp_sketchlib' has no attribute 'assignThreshold'

poppunk 2.3.0
poppunk_sketch 1.6.5
I have been working on Xanthomonas genus strains, I had successfully created database. Further, I proceeded for --fit-model with bgmm and dbscan both model directories were generated without any error. However I obtained poor plot for both models, so I proceeded for refine and used the following command,
(base) dinesh@dinesh7k:~/Documents/data/xap98/popPUNK$ poppunk --fit-model refine --ref-db database --model-dir dbscan/ end up with the following error,
PopPUNK (POPulation Partitioning Using Nucleotide Kmers)
	(with backend: sketchlib v1.6.5
	 sketchlib: /home/dinesh/miniconda3/lib/python3.8/site-packages/pp_sketchlib.cpython-38-x86_64-linux-gnu.so)

Graph-tools OpenMP parallelisation enabled: with 1 threads
Mode: Fitting refine model to reference database

Loading DBSCAN model
Loaded previous model of type: dbscan
Initial model-based network construction based on DBSCAN fit
Initial boundary based network construction
Decision boundary starts at (0.54,0.53)
Trying to optimise score globally
multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "/home/dinesh/miniconda3/lib/python3.8/multiprocessing/pool.py", line 125, in worker
    result = (True, func(*args, **kwds))
  File "/home/dinesh/miniconda3/lib/python3.8/multiprocessing/pool.py", line 48, in mapstar
    return list(map(*args))
  File "/home/dinesh/miniconda3/lib/python3.8/site-packages/PopPUNK/refine.py", line 179, in newNetwork
    boundary_assignments = pp_sketchlib.assignThreshold(distMat, slope, x_max, y_max, cpus)
AttributeError: module 'pp_sketchlib' has no attribute 'assignThreshold'
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/dinesh/miniconda3/bin/poppunk", line 10, in <module>
    sys.exit(main())
  File "/home/dinesh/miniconda3/lib/python3.8/site-packages/PopPUNK/__main__.py", line 363, in main
    assignments = new_model.fit(distMat, refList, model,
  File "/home/dinesh/miniconda3/lib/python3.8/site-packages/PopPUNK/models.py", line 609, in fit
    self.start_point, self.optimal_x, self.optimal_y, self.min_move, self.max_move = refineFit(X/self.scale,
  File "/home/dinesh/miniconda3/lib/python3.8/site-packages/PopPUNK/refine.py", line 100, in refineFit
    global_s = pool.map(partial(newNetwork,
  File "/home/dinesh/miniconda3/lib/python3.8/multiprocessing/pool.py", line 364, in map
    return self._map_async(func, iterable, mapstar, chunksize).get()
  File "/home/dinesh/miniconda3/lib/python3.8/multiprocessing/pool.py", line 771, in get
    raise self._value
AttributeError: module 'pp_sketchlib' has no attribute 'assignThreshold'

I am getting the same error for using bgmm model directory under refine option as follows,
(base) dinesh@dinesh7k:~/Documents/data/xap98/popPUNK$ poppunk --fit-model refine --ref-db database --model-dir bgmm
bgmm/     bgmm_viz/
(base) dinesh@dinesh7k:~/Documents/data/xap98/popPUNK$ poppunk --fit-model refine --ref-db database --model-dir bgmm/
PopPUNK (POPulation Partitioning Using Nucleotide Kmers)
	(with backend: sketchlib v1.6.5
	 sketchlib: /home/dinesh/miniconda3/lib/python3.8/site-packages/pp_sketchlib.cpython-38-x86_64-linux-gnu.so)

Graph-tools OpenMP parallelisation enabled: with 1 threads
Mode: Fitting refine model to reference database

Loading BGMM 2D Gaussian model
Loaded previous model of type: bgmm
Initial model-based network construction based on Gaussian fit
Initial boundary based network construction
Decision boundary starts at (0.01,0.14)
Trying to optimise score globally
multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "/home/dinesh/miniconda3/lib/python3.8/multiprocessing/pool.py", line 125, in worker
    result = (True, func(*args, **kwds))
  File "/home/dinesh/miniconda3/lib/python3.8/multiprocessing/pool.py", line 48, in mapstar
    return list(map(*args))
  File "/home/dinesh/miniconda3/lib/python3.8/site-packages/PopPUNK/refine.py", line 179, in newNetwork
    boundary_assignments = pp_sketchlib.assignThreshold(distMat, slope, x_max, y_max, cpus)
AttributeError: module 'pp_sketchlib' has no attribute 'assignThreshold'
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/dinesh/miniconda3/bin/poppunk", line 10, in <module>
    sys.exit(main())
  File "/home/dinesh/miniconda3/lib/python3.8/site-packages/PopPUNK/__main__.py", line 363, in main
    assignments = new_model.fit(distMat, refList, model,
  File "/home/dinesh/miniconda3/lib/python3.8/site-packages/PopPUNK/models.py", line 609, in fit
    self.start_point, self.optimal_x, self.optimal_y, self.min_move, self.max_move = refineFit(X/self.scale,
  File "/home/dinesh/miniconda3/lib/python3.8/site-packages/PopPUNK/refine.py", line 100, in refineFit
    global_s = pool.map(partial(newNetwork,
  File "/home/dinesh/miniconda3/lib/python3.8/multiprocessing/pool.py", line 364, in map
    return self._map_async(func, iterable, mapstar, chunksize).get()
  File "/home/dinesh/miniconda3/lib/python3.8/multiprocessing/pool.py", line 771, in get
    raise self._value
AttributeError: module 'pp_sketchlib' has no attribute 'assignThreshold'

Please help me to fix this issue.

This looks like a version mismatch caused by some recent changes we haven't yet synced up, which will be sorted when v2.4.0 of PopPUNK is on bioconda. For now, can I suggest that you downgrade your sketchlib version to 1.6.2:
conda install pp-sketchlib==1.6.2

I can let you know when you've got our next release out, as poppunk 2.4.0 will work with pp-sketchlib 1.6.5 and higher

Thank you @johnlees . Now its works well

This should be fixed with v2.4.0 (and v1.7.0 of sketchlib)


UI: don't allow empty content when editing orchestration template

@mzazrivec Can we got some specs in as well?

@chessbyte specs added.

@mzazrivec Found during testing:
I opened the "Dans Sample" OT for edit, changed the name to "Dans Sample 5", and cleared out the content field.  When I pressed the Save button, I got this screen.  It has the correct flash message, but it restored my content (it should just leave it blank) and it changed the header to say "Editing Dans Sample 5", even tho that name change wasn't saved yet.  The Save button is no longer enabled either, even though the name has been changed.  I think it should simply send back the flash message and leave everything else alone, including the blank content.  Here's the resulting screen shot:

I also noticed similar issues if I change the content to match another OT content, I get the proper error message, but the content is changed back (so the content now shown is NOT the duplicate of the other OT content) and the Save button is also not enabled even though the name was changed.  Header is changed as well, just like above.  Again, should probably just leave things alone and send down the flash message.  Here's the screen shot:

Ping me if you have any questions.  Thx, Dan

@dclarizio We're discussing here how the application should behave when (for whatever reason)
saving the edited orchestration template is not possible.
Though I'm slightly confused about what behavior we desire here.
There are 2 ways to approach this that I see:

User is presented with the original template values (name, description, content) and a flash message. With this approach, the user will have to edit everything over again and also fix
the problem stated in the flash message.

User is presented with the modified values (i.e. the values submitted in previous step)
which then need to be modified (one value or more) so that the next submission succeeds.

Above you describe steps when you modify the name & description and remove all of
the template content. Then you say that when the submission fails (b/c of the blank content),
we should leave everything else alone (does that mean original values?), including the blank
content (which is the modified content value). Which is sort of mix of both 1 and 2.
My take would be to go with 2, but of course, we can discuss this ...

@dclarizio Fixed according to your suggestions. I also fixed the fallback screen with an error
when copying orchestration template (we just need to render a flash message in that case and be done).

@mzazrivec I did some testing and the 2nd approach seems to be working properly, not changing the cell header when there are errors, etc.
@bzwei Are we now allowing blank content or is that only for when the draft flag is on (I know, a different PR).
I'm thinking the UI should not care what's in the content and the model should simply give back errors if it's invalid for whatever reason (i.e. blank if draft is not checked, etc).
I guess I'm wondering if this should be merged and we take care of cleaning it up in a further PR, since it does handle all of the edit errors properly now in this PR.
@mzazrivec @bzwei please comment back . . . thx, Dan

@dclarizio As far as I know, currently our application would allow empty content only as long as
the draft flag is on.
Yes, we could just attempt to save whatever comes in from the user and display whatever
error that would show up. Right now though the backend would only return an error telling
something about MD5 conflict, which says very little about the real problem (as in from
the user perspective, this error message has very little information value).
Allowing one template with empty content and disallowing all subsequent templates with empty
content also seems a bit strange from user's perspective, which is why this PR exists (but otherwise
yes, this is just a special case of conflicting contents).
My take on this right now would be to merge this PR and handle whatever objections would users
give us in future PRs.

When the draft is on, of course we don't constrain the content at all. However when draft is false, we only allow one OT to have empty content. We don't treat empty content a special case. Because of the uniqueness constraint there can be only one empty content.
We don't currently do content validation. Otherwise empty content will not pass validation. I don't have objection if UI forbids empty content for non-draft OT.

@bzwei Can't the model validation just do all of this and return the appropriate errors.  I really think that if there are rules for what gets saved in the DB, it should be done at that level so that no matter who the user is (our UI, the REST API, some other code), the model won't let bad data into the DB.
I would prefer that the UI simply collects the data and displays the errors returned from the model.  We can discuss if needed, let me know.

I'm going to agree with @mzazrivec that we merge this for now and await feedback and discussion if we want to change where certain validations are done.  Thx guys.

@mzazrivec Looks like you need to rebase.

@dclarizio Rebased.

@mzazrivec I see one test failure in the OT draft flag area.  Also some product features failures where the count doesn't match the test count, but since you didn't change any features, I'm not sure how that is failing here.
@h-kataria can you see if you can track down if a PR got merged that broke this?

@mzazrivec can you rebase and repush your branch,  Product Feature count needs to be fixed in spec test

@dclarizio Should be fixed now.


[Enhancement]: Revamping the project card page

Project details

The OSAR  is a download button which should be activated once the reports are available.
Add an appropriate icon next to the OSAR download button, which depicts the report has been attested.
The Envs and Playbooks is a button, which opens up a modal component with environments and compatible playbooks.
The OSAP(Open Source Assurance Provider) will show the name of the entity that performed the assessments, followed by an appropriate icon. The entity can be a user, an organization or a lab.
The tags and languages should be a link, on-click, it should trigger the filter for that tag/lang which updates the poi table.

Assessment details

The scores and assessment counts are modal component which gives you a summary report of its respective assessment.
The OWASP Top Vulnerability Type should display the OWASP top vulnerability types that the vulnerabilities from sast report belongs to.
The Vulnerability Trend gives you the count of vulnerabilities, for this version of the project, over the years.

Fixed in previous release


Temporary pin esmpy to esmpy!=8.1.0 to bring environment in harmony with ESMValCore

Description
esmpy is at odds in Tool vs Core: free env creation grabs 8.1.1 for Core and 8.1.0 for Tool (8.1.1 can not currently be installed in the Tool environment, due to some deps conflict). So when it comes to first installing Tool, then going to install Core in development mode (overriding the conda-forge installed Core while creating the environment), one gets to a deadlock seen in the latest Github Actions test runs. Pinning it this way ensures harmony between Core and Tool via the installation of 8.0.1.

Closes - no issue created

Before you get started

Checklist
It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

[x] 🛠 This pull request has a descriptive title
[ ] 🛠 Tests run successfully
[x] 🛠 Any changed dependencies have been added or removed correctly
[ ] 🛠 All checks below this pull request were successful

I think we have to be rather clear on what is happening where. From what I gather, the problem is with conda(-forge) packaging, not with actual code compatibility. That is completely fair, but means that it should not influence the pinning in setup.py. The change in environment.yml is completely fine; the one in package/meta.yml is not wrong, but hopefully we'll just throw the whole file out soon.
So bottom line: Remove the change in setup.py and I'll have another look.

@zklaus cheers! I keep forgetting about the new way of setting up the package for conda-forge :+1:

Sorry that such a small issue turns into so much fuzz, but there was a bug in the 8.1.1 release of esmpy (wrong version in the setup.py), which has been corrected in the conda-forge package by now (see conda-forge/esmpy-feedstock#49).
As a consequence, I think this is no longer needed.

seems like you are right and this is no longer needed indeed, Klaus! The last time the develop test failed was 4 days ago, it seems all works fine ago - how's this possible though, given that your conda-forge fix PR is 13 days old?

Not sure :shrug:. It may be related to the NCL mess that we cleaned up and that may have pulled in a web of outdated dependencies.

well either way, it works :beer:


[HOLD for payment 2024-02-19] [$500] [Wave 8] [Ideal Nav] The color of Status Bar (top part of the background) is not consistent

Action Performed:

Open the app on a mobile browser
Go to Chat page (Chat icon on the bottom tab)
Go to Settings page (🔧 icon on the bottom tab)
Go to Settings page > Workspaces

Expected Result:
The color should be consistent
Actual Result:
The colors are different across different pages
Workaround:
N/A
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[x] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
screenshots -> https://github.com/Expensify/App/pull/33280#issuecomment-1922009959

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01f7200068e359bb73
Upwork Job ID:<PHONE_NUMBER>673206272
Last Price Increase: 2024-02-02

Please hold a little bit before posting proposals until I confirm that this issue is handled independently (there is a possibility that this issue is combined with this issue)

Posting a proposal before going offline in case we want to handle it separately.
Proposal
Please re-state the problem that we are trying to solve in this issue.
The status bar color doesn't match the page on the chat and workspace list page.
What is the root cause of that problem?
The status bar color is defined by the PAGE_THEMES here.
https://github.com/Expensify/App/blob/8fa0994d09038dd32f2079bf6bd121c999a14d29/src/styles/theme/themes/dark.ts#L94-L98
The chat/HOME page uses productDark200 (for dark mode) as the color, but the page color uses appBG, which is productDark100. It goes the same for the workspace list which uses pink color.
https://github.com/Expensify/App/blob/8fa0994d09038dd32f2079bf6bd121c999a14d29/src/styles/theme/themes/dark.ts#L111-L114
What changes do you think we should make in order to solve the problem?

We can update the color to productDark100 to match the page color or just remove it from the list because the default value is productDark100 (appBG).
https://github.com/Expensify/App/blob/8fa0994d09038dd32f2079bf6bd121c999a14d29/src/components/CustomStatusBarAndBackground/index.tsx#L74-L75
we need to update the light mode color too

defering to @hayata-suenaga's comment to hold https://github.com/Expensify/App/issues/35632#issuecomment-1924192114

still waiting for an answer to this

This one was fixed in this PR.
However, there are other problems with StatusBar:

android insets are broken sometimes: https://github.com/Expensify/App/pull/35564#issuecomment-1926772435,
the status bar is not updating after theme change: https://github.com/Expensify/App/pull/35672#issuecomment-1933807425

Because the issue is fixed in @kosmydel's PR, I'll close this issue.

the status bar is not updating after theme change

@kosmydel, could you handle this issue in your PR?

@hayata-suenaga It's not fixed yet for the home page https://github.com/Expensify/App/issues/35632#issuecomment-1934279437

The offending issue for the deploy blocker liked above was identified and it's not this issue

@hayata-suenaga It's not fixed yet for the home page https://github.com/Expensify/App/issues/35632#issuecomment-1934279437

checking here

@kosmydel could you check this comment when you have time please? 🙇

@kosmydel could you check this comment when you have time please? 🙇

Changing this in config broke the android insets. That's why I haven't changed it. Here is more details:
https://github.com/Expensify/App/pull/35564#issuecomment-1926772435

@kosmydel could you check https://github.com/Expensify/App/issues/35632#issuecomment-1934279437 when you have time please? 🙇

Changing this in config broke the android insets. That's why I haven't changed it. Here is more details:
https://github.com/Expensify/App/pull/35564#issuecomment-1926772435

@kosmydel Thank you for reminding me of that. Are you working on this in a separate issue or should I create a new issue for you?

@kosmydel Thank you for reminding me of that. Are you working on this in a separate issue or should I create a new issue for you?

I can have a look at this, but I found a PR (https://github.com/Expensify/App/pull/34409) that addresses a mWeb status bar issue. The author of the PR suggests that he found a solution for that. In case this doesn't solve our problem, I can surely investigate it.

okay let's see if the PR will solve the issue 👍


Create JSON Schema to utilize with code editors

Description
Create a JSON schema to allow code editors to parse the .vela.yml and help users understand if the format is valid or not.
The simple steps to test the JSON schema:

convert complex .vela.yml from yaml to json
utilize https://jsonschema.net/home to create JSON schema
follow editor specific instructions to utilize the schema for .vela.yml

VScode docs https://code.visualstudio.com/docs/languages/json#_json-schemas-and-settings

Definition of Done
Users able to configure code editor to utilize published JSON schema for file validation
Effort  (Optional)
2 days
Impacted Personas (Optional)
All users modifying .vela.yml within any code editor

Hello 👋
The Vela admins have decided to condense all existing issues into a single repo found @ go-vela/community.
For a direct link to your counterpart issue in that repo, you can find it linked yesterday in the history of this issue.
If you have any questions, please feel free to direct them to the community repo.
Thanks for your input and feedback!


Proc macros / non-deterministic analysis results with webrender

This is also visible on the metrics page:

Here are two analysis-stats runs with the same RA version:
# (some) first runs
Database loaded:     3.57s, 188minstr
  crates: 20, mods: 110, decls: 3292, fns: 2059
Item Collection:     5.65s, 60ginstr
  exprs: 93018, ??ty: 671 (0%), ?ty: 205 (0%), !ty: 14                                                                                Inference:           5.37s, 43ginstr
Total:               11.02s, 104ginstr

# other runs, even after clearing the disk cache
Database loaded:     3.89s, 193minstr
  crates: 20, mods: 110, decls: 3814, fns: 2059
Item Collection:     11.85s, 74ginstr
  exprs: 93018, ??ty: 101 (0%), ?ty: 23 (0%), !ty: 4
Inference:           5.46s, 44ginstr
Total:               17.32s, 118ginstr

Maybe we're not expanding proc macros when they're already built. Running cargo clean made it return the first results again, but not every time.
EDIT: I think this was on the webrender from rustc-perf (the version used for the metrics).

It looks like the webrender crate depends on url 1.5.1, which doesn't build any more:
   --> /home/grayshade/.cargo/registry/src/github.com-1ecc6299db9ec823/url-1.5.1/src/form_urlencoded.rs:251:40
    |
249 | impl<'a> Target for ::UrlQuery<'a> {
    |      -- lifetime `'a` defined here
250 |     fn as_mut_string(&mut self) -> &mut String { &mut self.url.serialization }
251 |     fn finish(self) -> &'a mut ::Url { self.url }
    |                                        ^^^^^^^^ - here, drop of `self` needs exclusive access to `*self.url`, because the type `UrlQuery<'_>` implements the `Drop` trait
    |                                        |
    |                                        returning this value requires that `*self.url` is borrowed for `'a`

I suspect that, depending on the build order, we sometimes end up missing the proc macro (either that, or we don't fill in WorkspaceBuildScripts correctly):

Wait, that doesn't make sense, does it? We don't need to build that dependency for serde.

Repro:
[package]
name = "hello"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
serde = { version = "1.0", features = ["derive"] }
url = "=1.5.1"

use serde::Serialize;

#[derive(Serialize)]
struct S;

fn main() {}

If you run
$ cargo clean; RA_LOG=warn CARGO_BUILD_JOBS=1 rust-analyzer analysis-stats .

it sometimes outputs:
[WARN hir_expand::db] fail on macro_parse: (reason: UnresolvedProcMacro macro_call: #[derive(Serialize)]) parents:

And indeed, the output of CARGO_BUILD_JOBS=1 cargo check --quiet --message-format=json doesn't always contain the serde_derive` proc macro.
We could try to update the url or ws deps, but rustc-perf still seems to be on these versions.


Fix issue with indexPath in didSelectRowAtIndexPath when a cell contains KILabel

Related to #30
It looks like copy-paste issue when a chunk of code at the touchesEnded was copied from touchesBegan and after super's touchesEnded super's touchesBegan is being called.
Main.storyboard was updated to make a cell selectable.
KILabelTableViewController.m updated with logging so you can check issue.
To reproduce the issue revert my changes in KILabel.m and try the following steps:

tap first cell - indexPath is 0-0  (correct)
tap second cell - indexPath is still 0-0 (wrong)
tap second call again - indexPath is 0-1 (correct)
tap first cell - indexPath is 0-1 (wrong)
tap first cell again - indexPath is 0-0 (correct)

Hope you will check this PR asap because it's very useful pod.
ADDED: fix for the crash caused by adding nil color to a dict

... and. nothing?

I fixed the issue by removing for following code from "- (void)touchesEnded:(NSSet *)touches withEvent:(UIEvent *)event. I guess the author copied code from "- (void)touchesBegan:(NSSet *)touches withEvent:(UIEvent *)event" by mistake.
   else     {         [super touchesBegan:touches withEvent:event];     }

@mobilefellow thank you. I have just wasted two hours on this, taking my project (quite large) apart and going more crazy every minute. It was one of the most bizarre and random bugs I have seen this year. I'm so happy it works now.


bug for show_result func

you code:
    def show_result(self, data, result, dataset=None, score_thr=0.3):
        if isinstance(result, tuple):
            bbox_result, segm_result = result
        else:
            bbox_result, segm_result = result, None
        **img_tensor = data['img'][0]** # 经过pipeline处理后的数据
        img_metas = data['img_metas'][0].data[0]
        print('\n', img_metas[0]['filename'])
        imgs = tensor2imgs(img_tensor, **img_metas[0]['img_norm_cfg'])  # 已经经过变换后的图片
        assert len(imgs) == len(img_metas)

        if dataset is None:
            class_names = self.CLASSES
        elif isinstance(dataset, str):
            class_names = get_classes(dataset)
        elif isinstance(dataset, (list, tuple)):
            class_names = dataset
        else:
            raise TypeError(
                'dataset must be a valid dataset name or a sequence'
                ' of class names, not {}'.format(type(dataset)))

        for img, img_meta in zip(imgs, img_metas):
            h, w, _ = img_meta['img_shape']
            img_show = img[:h, :w, :]
            bboxes = np.vstack(bbox_result)
            # draw segmentation masks
            if segm_result is not None:
                **segms = mmcv.concat_list(segm_result)**  # 已经还原到最原始比例的分割结果
                inds = np.where(bboxes[:, -1] > score_thr)[0]
                for i in inds:
                    color_mask = np.random.randint(
                        0, 256, (1, 3), dtype=np.uint8)
                    mask = maskUtils.decode(segms[i]).astype(np.bool)
                    img_show[mask] = img_show[mask] * 0.5 + color_mask * 0.5
            # draw bounding boxes
            labels = [
                np.full(bbox.shape[0], i, dtype=np.int32)
                for i, bbox in enumerate(bbox_result)
            ]
            labels = np.concatenate(labels)
            mmcv.imshow_det_bboxes(
                img_show,
                bboxes,
                labels,
                class_names=class_names,
                score_thr=score_thr)

The img data in data ['img'] has been processed by pipeline fun, but the prediction result of bbox or segm is the original img scale information, which is wrong! When visualizing, it should be the correct way to use the original img data, not after processing。

It is not wrong, you just need to use corresponding (both original sized or both resized) results and images for visualization.
In the model, original sized images are unavailable, and we plot resized results on resized images, please refer to the test code.


[Cleanup] Fix  Props and usage

Small fixes for the <LatestPosts /> component:

Fix for the Props
2. Remove postIds prop
3. Add count prop
Remove postIds from index.astro file

Hi @IlyaKukarkin
We approve your PR that fixes this. Thank you so much.


package.json file was not updated on tag release

Hi there,
At comit 0d408d5fe5a08862522af3aa1bfe1c2166fe9d8d there was a minor bump version from 1.1.20 to 1.2.0. The package.json file was not updated to reflect this version.
This makes an error when https://github.com/igorlino/angular-elevatezoom-plus package is installed from npm using 1.2.0 version.

Thanks!

Gracias!, Tambien he actualizado las version el registro de NPM. (para que aparezca como la version oficial)


Update custom.py

Fix rcf.template_specs.delete If VersionID presented
Description
CLI command for delete Template specs is deleting the template specs instead of the provided version alone, else statement adder
Testing Guide
az ts delete -g RGName --n TSName -v 1.0
this was deleting the template specs resource and not the version needed, now will delete the version alone in case version ID provided
History Notes
[Template Specs] az command Delete: Fix version parameter issue.

ARM

This is a bot spamming our repo.
The change is also against the linter rule.
https://dev.azure.com/azure-sdk/public/_build/results?buildId=652812&view=logs&j=36dd4138-4d53-5e46-00d9-e5cd9744be05&t=1cf3879c-0a42-5ccd-7693-7a4781d739d8&l=107
src/azure-cli/azure/cli/command_modules/resource/custom.py:1900:4: R1705: Unnecessary "else" after "return" (no-else-return)

It was replaced by #16295


etc/resolve ndots set to 1

Wondering why its set to 1 ? I'd like to set it to 3 without writing a new resolver.
https://github.com/amphp/dns/blob/master/lib/UnixConfigLoader.php#L21
Is there a way to extend UnixConfigLoader and continue to use Rfc1035StubResolver ?

You can either edit your /etc/resolv.conf, or set RES_OPTIONS in the environment, which can also be done at the start of the script.
putenv('RES_OPTIONS', 'ndots:3');


DTLS cookie exchange in openssl is different as RFC6347

According to rfc6347, in DTLS, default execute cookie exchange.
However, in OpenSSL, must use SSL_set_options() to set SSL_OP_COOKIE_EXCHANGE.

However, in OpenSSL, must use SSL_set_options() to set SSL_OP_COOKIE_EXCHANGE.

In fact this option is automatically set if you use DTLSv1_listen(). If you are not using DTLSv1_listen() then the server is stateful anyway and there is less benefit to using cookies.

In fact this option is automatically set if you use DTLSv1_listen(). If you are not using DTLSv1_listen() then the server is stateful anyway and there is less benefit to using cookies.

Does OpenSSL plan to DTLS default execute cookie exchange in future versions, no additional settings are required?
Why do I use SSL_set_options() to set up SSL_OP_COOKIE_EXCHANGE instead of DTLSv1_listen(), which leads to less benefits?

Does OpenSSL plan to DTLS default execute cookie exchange in future versions, no additional settings are required?

No.

Why do I use SSL_set_options() to set up SSL_OP_COOKIE_EXCHANGE instead of DTLSv1_listen(), which leads to less benefits?

I would recommend that you use the DTLSv1_listen() approach rather than setting the option explicitly via SSL_set_options(). The main reason to use the latter is that it makes the code slightly simpler - but the code is not stateless. You will still gain some benefit rather than no cookies at all, but DTLSv1_listen() is the best route.


How to reproduce retrieval results?

We didn't perform retrieval in our paper. If you have your implementation please share on github!


http: removing envoy.reloadable_features.unify_grpc_handling

Risk Level: Low (removing deprecated flag)
Testing: n/a
Docs Changes: n/a
Release Notes: inline
Fixes #16014

Snow - think we're good to get rid of this one?


Create a json or jsonc file containing triple quotes between valid entries. Here's a simple example that shows the issue:{
  "hello": "world",
  """
  "asdf": "zxcv",
}

Run biome check on the file.
Get some expected errors, but also a panic that mentions it's a bug in biome, not the file itself.❯ bunx biome lint test.jsonc
test.jsonc:3:5 parse ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  × Missing closing quote

    1 │ {
    2 │   "hello": "world",
  > 3 │   """
      │     ^
    4 │   "asdf": "zxcv",
    5 │ }

  i line breaks here

    1 │ {
    2 │   "hello": "world",
  > 3 │   """
      │
  > 4 │   "asdf": "zxcv",
Biome encountered an unexpected error

This is a bug in Biome, not an error in your code, and we would appreciate it if you could report it to https://github.com/biomejs/biome/issues/ along with the following information to help us fixing the issue:

Source Location: /Users/runner/work/biome/biome/crates/biome_text_size/src/range.rs:50:9
Thread Name: main
Message: assertion failed: start <= end

Expected result
I would expect to see the errors it does report, possibly some others, but not the message about biome encountering an unexpected error. It sounds like Biome bailed out early.
Code of Conduct

[X] I agree to follow Biome's Code of Conduct

This is probably a duplicate of #2357. Do you have nursery lint rules enabled?

I haven't enabled any of them, just the recommended rules. I just tried creating a new config using bunx @biomejs/biome init and get the same error, but I don't actually see a panic for the one reported in that issue.
❯ cat biome.json
{
        "$schema": "https://biomejs.dev/schemas/1.7.1/schema.json",
        "organizeImports": {
                "enabled": true
        },
        "linter": {
                "enabled": true,
                "rules": {
                        "recommended": true
                }
        }
}
❯ cat a.json
{
  "a": """
}
❯ cat hello.json
{
  "hello": "world",
  """
  "asdf": "zxcv",
}

a.json gives me 5 errors and doesn't panic:
❯ bunx biome check a.json
a.json:2:10 parse ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ Missing closing quote

    1 │ {
  > 2 │   "a": """
      │          ^
    3 │ }
    4 │

  ℹ line breaks here

    1 │ {
  > 2 │   "a": """
      │
  > 3 │ }
      │
    4 │

a.json:3:1 parse ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ expected `:` but instead found `}`

    1 │ {
    2 │   "a": """
  > 3 │ }
      │ ^
    4 │

  ℹ Remove }

a.json:2:10 parse ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ Missing closing quote

    1 │ {
  > 2 │   "a": """
      │          ^
    3 │ }
    4 │

  ℹ line breaks here

    1 │ {
  > 2 │   "a": """
      │
  > 3 │ }
      │
    4 │

a.json:3:1 parse ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ expected `:` but instead found `}`

    1 │ {
    2 │   "a": """
  > 3 │ }
      │ ^
    4 │

  ℹ Remove }

a.json format ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ Code formatting aborted due to parsing errors. To format code with errors, enable the 'formatter.formatWithErrors' option.

Checked 1 file in 440µs. No fixes needed.
Found 5 errors.
check ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ Some errors were emitted while running checks.

hello.json bails after the first error:
❯ bunx biome check hello.json
hello.json:3:5 parse ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ Missing closing quote

    1 │ {
    2 │   "hello": "world",
  > 3 │   """
      │     ^
    4 │   "asdf": "zxcv",
    5 │ }

  ℹ line breaks here

    1 │ {
    2 │   "hello": "world",
  > 3 │   """
      │
  > 4 │   "asdf": "zxcv",
Biome encountered an unexpected error

This is a bug in Biome, not an error in your code, and we would appreciate it if you could report it to https://github.com/biomejs/biome
/issues/ along with the following information to help us fixing the issue:

Source Location: /Users/runner/work/biome/biome/crates/biome_text_size/src/range.rs:50:9
Thread Name: main
Message: assertion failed: start <= end

It looks like instead of three double-quotes, I can also do any of these:
""'
"'
'

I'm not quite seeing a pattern but I've found a few other simple test cases that cause it to bail:
{
  "
  '
}

{"hello": "world"
  '
  "}

{"
  '}

whereas these will work fine (reporting a handful of legit errors)
{
  ""'
}

{"hello": "world"
  '
}

{"hello": "world"
  '
"}

{"
'}

but I don't actually see a panic for the one reported in that issue.

Oh yes that is expected. you'll need to enable nursery rules to trigger the panic in that issue. In stable releases nursery rules are disabled by default and one has to set it explicitly in the settings to enable them.

I just tried creating a new config using bunx @biomejs/biome init and get the same error,

So this one is different, because this seems to be triggered by analyzing the keys instead of the values. But I guess it is still related to how we handle unterminated string literals.
Apart from #2357, another similar issue reported is #2410.
And thank you for providing the fail cases.


Movement command H on first character in first row tries to move left 9 times

Description
When the cursor is on the first character of the first line of the file, pressing H causes the terminal to beep 9 times.
It should beep once (and pressing L on the last character of the last row should also beep once), or it should not beep at all (and pressing L on the last character of the last row should not beep either).

agreed, should be fixed now.


Cumulative Changes

Major changes:

Integral Contoller Limit 1 -> 20% of cell_size_ref.  Fixed random number calculation for vfo_suspension_rate INT32_MAX -> UINT32_MAX.
Missing-clock byte A1 and C2 may decay to be 14 and 28 respectively.  This change will identify decayed mising-clock byte and prevent decay of the next byte.
'Exact Cut' in pauline2raw command.

Thank you for your great contribution.


Find a better way to mark all dependencies as externals

Right now, we use esbuild to mark the CDK and Lambda code's dependencies as externals by looking at the package.json. But for some setups, where the dependencies are not listed (nested package.json files), this doesn't work. Two options to investigate:

Does the esbuild external * option work
Use a plugin to mark them all as externals

Can I add that for many setups you actually don't want to externalize all your deps, because many CAN be bundled, and that could save a lot of space ! It's imperative (and I'd say high prio) to let user decide the list of externals they want to bundle. Actually, every esbuild options should be editable, the way we do it in serverless-esbuild, with sane defaults, don't you think ?

I'm not sure if this is relevant anymore since we have support for specify what you want to be external vs bundled.  Closing for now


Add encoding for the jwt key value in the case of HMac signing

Some authorization servers provide a secret which is encoded: auth0 for instance gives a base 64 encoded-url secret. This cannot be handled by Spring Security OAuth at this time (see http://stackoverflow.com/q/37832961/5903220).
This patch adds the possibility to configure an encoding for the key (at this time: base64 or base64url), and uses the MacSigner(byte[]) constructor to avoid data loss when converting back to String.

@cdelmas Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@cdelmas Thank you for signing the Contributor License Agreement!

@cdelmas I'm using Auth0 as well with HMAC. Is there anything else that you did special for Auth0? I'm adding roles to users in Auth0 as well via an Auth0 rule...
Given how popular Auth0 is it's unfortunate a workaround like this is required out of box. Are you still using your custom JwtAccessTokenConverter?

From the 6th of december, this won't be mandatory anymore as they decided to remove the encoding for new keys. A decent workaround is to use RSA.

Okay cool. I suppose that's easier yes because it only requires Auth0 config stuff. Thanks for the timely response.

@cdelmas If you're still looking to get this merged, please add a test and I can review.

@cdelmas I'm closing this PR due to lack of feedback from review.
If you still feel this is an issue, please submit a new PR and address the review feedback.


Fixes the ClientSideRegionScanner to actually respect the nextRaw API

The nextRaw API has the following contract:
nextRaw(List kvs) -> boolean (hasMore)
The intent is that one needs to continue to call nextRaw() repeatedly until the hasMore signal returns false (though it can return false and the caller will still have the last row).
The ClientSideRegionScanner treated any empty kvs list as a sentinel that the scan should be completed for this region. This is incorrect, and breaks pretty badly when filters might cause the nextRaw to return empty results that need to be skipped.
This change moves the client scanner to follow the contract as required by the nextRaw method. We add a private field to track whether hasMore is false, and will repeatedly call nextRaw to get a non-empty result (in order to behave similarly to how it's behaved in the past).

This looks good to me. We generally try to upstream work and cherrypick back into our branch. Someone on the team could help you with that, or we could do it for you

So this only happened on snapshot scan because normal hbase operation just uses its own server side scanner I guess. I read a bit and was surprised at how different they were. Where do filters get applied in the snapshot scan process?

the filters are evaluated in the same way in the underlying RegionScannerImpl class. The reason we're only seeing this in the clientsideregionscanner is that this wrapper class is not using the nextRaw call properly. In a server context it's not using ClientSideRegionScanner and is using the RegionScannerImpl in HRegion which will properly use next() and nextRaw()

This has been resolved upstream and backported to our fork.


Wildcard progressive autocomplete

I do apologise if this is the wrong place as I'm not sure what implements this, but when writing a prompt, I can use the tab key to autocomplete wildcards. This is very useful, except it will auto-complete whichever one is the top one, rather than filling in only what is "common" to the current possibilities. This means I have to type out a considerable amount before I can auto-complete usefully.
For example, if the three files are "foofoo", "foobar" and "foobaz". I would expect the following:

f<TAB> becomes "foo"
foob<TAB> becomes "fooba"

Currently, f<TAB> would just autocomplete whichever happens to be first in the list of possibilities.

I am not sure if this library implements or interacts with auto-completion
Are you thinking of https://github.com/DominikDoom/a1111-sd-webui-tagcomplete, by any chance?

Yes! Sorry, I thought that only provided tag auto-completion but I just checked the commit history and they added in wildcard support, so that's where it is coming from. Please excuse the noise!


Update composer SHA digest for v1.4.1

I have tried to build the Docker image today but it fails because meanwhile composer has been updated.
I have updated the SHA digest to match when downloading composer, to support current version 1.4.1
Maybe it is useful to overcome this check, or to download a specific version of composer instead of latest.

Thanks! :)


Circle CI custom neo4j configuration

Hi!
We use Neo4j with APOC plugin, and we need to change some configuration by mounting conf volume, and this need a custom run script
docker run --rm \
    --volume=$HOME/neo4j/conf:/conf \
    neo4j:3.4 dump-config

How can we use the image with this custom run script?
this is a sample of our config.yml:
version: 2
jobs:
  build:
    docker:
      - image: circleci/php:7.2.3-node-browsers
      # - image: elasticsearch
      - image: digitalcloudsa/neo4j
        environment:
            NEO4J_AUTH=none

    working_directory: ~/repo

any help?

@devmsh did you resolve it?

@piotrjaworski unfortunately no!

@piotrjaworski @devmsh I couldn't solve this through CircleCi so I created my own docker image which simply extends the original one and used it in the CI. It's just a few lines of code. Here's the code https://github.com/pluralcom/docker-neo4j 💪