Another Crash to Desktop...

Slightly different from the other issue that is posted.
I was previously using 2.0.2, and it worked fine. I uninstalled it completely, and cleaned my CPU with a registry cleaner (Piriform/CCleaner). I then installed the new 2.1.1 version directly download from GitHub. From installation, the program immediately closes when I open it. I opened the task manager, and am able to see the task running for a brief moment when I start it, but it closes less than a second later.
There is no splat screen, sound, or error message of any kind. It must be failing on some fundamental level. Maybe I'm missing a dependency? I know you don't claim there were any other than Windows 7 and above. It also may bear worth mentioning that I am a programmer, so there is MVC/ASP.NET/Visual-Studio and other software installed on this computer that the average user may not have. I can't imagine that is causing the issue, just covering my bases here.
It isn't flash, I had to install it for 2.0.2 to work. I also tried running the program as Admin, and disabled my anti-virus.
My stats are beefy:
Win 10 Pro,
i7-5820K @ 3.3Ghz
16 GB DDR4
1/2 TB SSD

If you have Visual Studio can you try cloning the source code and running it.  Then it might throw a semi useful error
Those specs are scarily close to mine.  5820K master race :+1:

same problem here first install ever and just instacrash
specs
nextbook flexx 11
win 10 home x86
2gb ram
atom 3735f quad
64gb SSD

Marking this as Fixed in 3.0.0 because it probably is....  New platform and such so it shouldn't exhibit the same issue

Yeah I'm sorry Google linked me to an old installer for some brilliant
reason i was so embarrassed when i realized i fucked up......that's why I
redacted my post lol..... I have to say great app feels just like play and
no more tab surfing thumbs up
On Feb 6, 2016 4:48 AM, "Samuel Attard"<EMAIL_ADDRESS>wrote:

Marking this as Fixed in 3.0.0 because it probably is.... New platform
and such so it shouldn't exhibit the same issue
—
Reply to this email directly or view it on GitHub
https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/178#issuecomment-180726828
.

This has been tagged with Fixed in 3.0.0 and in preparation for the release of 3.0.0 these issues are being automatically closed.
Get excited for 3.0.0 :cactus: :cake: :tada: :balloon:

Sam Bot

Hey,
I just signed the petition "Mayor Jacob Frey: JUSTICE FOR GEORGE FLOYD" and
wanted to see if you could help by adding your name.
Our goal is to reach 500,000 signatures and we need more support. You can
read more and sign the petition here:
http://chng.it/TdxVq6QqCK
Thanks!
ZACKERY


obsolete: multicellular structure septum development

This is a weird term
id: GO:0080125
name: multicellular structure septum development
namespace: biological_process
def: "The process whose specific outcome is the progression of the multicellular structure septum over time, from its formation to the mature structure. The multicellular structure septum is the thin partition or membrane that divides a cavity or a mass of tissue." [GOC:dhl]
is_a: GO:0009888 ! tissue development
relationship: part_of GO:0048513 ! animal organ development

doesn't seem like a useful grouping. Perhaps septum formation may share mechanisms, but development.
Not sure how this one escaped taxon constraints:
   po GO:0048513 ! animal organ development
       po GO:0080125 ! multicellular structure septum development
        is_a GO:0080127 ! fruit septum development ***

!!!
Only one direct experimental annotation:
http://amigo.geneontology.org/amigo/term/GO:0080125
From this paper
http://www.ncbi.nlm.nih.gov/pubmed/9731532
It seems the desired term is tracheoesophageal septum formation (from the tracheoesophageal folds or ridges)?
See also MP:0003321 ! tracheoesophageal fistula

Very odd. the children terms are ALL plant terms.  The term is one made by Donghui, GOC:dhl, but the relationship to animal organ development was added later in time, most likely not by him.
http://www.ebi.ac.uk/QuickGO/GTerm?id=GO:0080125#term=history
@ukemi - could you look at the mouse annotation referenced above, please?

I think we should

[ ] make a new class tracheoesophageal septum formation
[ ] Move the annotation here
[ ] create a new plant grouping class and move the plant children here
[ ] obsolete GO:0080125

Ok, good plan.  I'll wait for the new UBERON term to show up, then can delegate moving the annotation to @ukemi and will proceed with the other steps.  New term will likely be 'plant septum development' with xp using PO:0000030, septum.

Added:
[Term]
id: GO:1905327
name: tracheoesophageal septum formation
namespace: biological_process
def: "The process that gives rise to the tracheoesophageal septum. This process pertains to the initial formation of a structure from unspecified parts." [GO_REF:0000081, GOC:TermGenie, PMID:9731532]
synonym: "esophagotracheal septum formation" EXACT [GOC:TermGenie]
synonym: "tracheoesophageal ridges formation" RELATED [GOC:TermGenie]
intersection_of: GO:0048646 ! anatomical structure formation involved in morphogenesis
intersection_of: results_in_formation_of UBERON:0036070 ! tracheoesophageal septum
created_by: tb
creation_date: 2016-07-21T18:18:57Z

Also created:
[Term]
id: GO:1905328
name: plant septum development
namespace: biological_process
def: "The process whose specific outcome is the progression of a septum over time, from its formation to the mature structure." [GO_REF:0000080, GOC:tb, GOC:TermGenie, PMID:4562349]
synonym: "dissepiment development" BROAD [GOC:TermGenie]
intersection_of: GO:0048856 ! anatomical structure development
intersection_of: results_in_development_of PO:0000030 ! septum
created_by: tb
creation_date: 2016-07-21T18:25:49Z
The inference pipeline will automatically move the children terms (fruit septum, ovary septum, anther septum development) under this term.

@ukemi - can you please move the mouse annotation that is currently to http://amigo.geneontology.org/amigo/term/GO:0080125 to the new GO:1905327?  Then reassign to me and I'll take care of the obsoletion email and obsoletion itself.  Thanks.

Following up on the moving of the mouse annotation in previous comment.  I think you should have the new term in MGI by now.  Thanks.

Hi Tanya,
The annotation is moved.
-D

Obsoletion email sent:
Dear all,
The proposal has been made to obsolete the following term:
GO:0080125, multicellular structure septum development
The reason for obsoletion is that it is not a useful grouping term.
All details here:
https://github.com/geneontology/go-ontology/issues/12496
The last experimental annotation to this term was removed by MGI today.  Only 1 ISO (RGD) and 6 IEA (Ensembl) annotations remain.
We are opening a comment period for this proposed obsoletion. We'd like
to proceed and obsolete the term above on September 8, 2016.
*** Unless objections are received by September 16, 2016, we will assume
that you agree to this change. ***
On behalf of the GO ontology editors,
Tanya Berardini
TAIR Curator
www.arabidopsis.org

Term made obsolete.


Improve reinstall loading screen status message UI

Related Issue/PR
#278 / #318
The reinstall feature added in the above PR has added the ability to detect when node_modules is missing, and to trigger a reinstall of dependencies from click of a button.
While the reinstall is happening, the standard loading screen is on the screen. The PR also adds a progress bar. There is also a message feature that was added, but this will be commented out until we can come up with a better solution for the UI.
The message looked like this, originally:

User clicks info icon to trigger message:

The main problem with this is the messages are flashing too quickly for the user to see, before they can click the info icon to open it up. So the solution should be come up with a nice UI to incorporate these messages.
I did a quick pass in Photoshop for an idea based on this Dribbble but I'm sure it can be improved.

But if anyone has any other ideas please feel free to suggest.

@melanieseltzer is someone working in this issue? I can do that

@joao-alberto, it's all yours! A couple things to note are that the PR (#318) is actually not merged yet, so you'll need to base your branch off reinstall branch, if you'll be working in the codebase for UI changes.
If you need to trigger the UI to show the loading screen/progress bar while you work, you can input some hard values like isVisible={true} and progress={somevalue} like so:
// components/LoadingScreen/LoadingScreen.js

...
<Window isVisible={true}>
  <ProgressBarWrapper>
    <ProgressBar progress={0.75} reset={this.state.reset} />
...

And finally, I think I remember that we decided not to show a percentage value to the user, which isn't reflected in my Photoshop mockup (there's a 75% value underneath progress bar). So you can ignore that value.
Let me know if you have any other questions!

@melanieseltzer Tks for tips, I'll start now

Closed via #349


minor spelling update

This change is

@chmouel , I don't see the spelling update. Have you seen my review comment?

@devin-donnelly apologies this took more time that it should, corrected the error and add bunch of others (using the 'mispellings' tool)

This is good, but it's going to introduce roughly eight merge conflicts when we launch 1.4. Do you mind if I hold onto it for a couple days?


Multi-class Classification Problem Using QSVC

Tutorial of Sorts for Multiclass Classification Using Qiskit 1.0.2's QSVC
I was going through this tutorial and found it immensely helpful for understanding certain fundamentals on quantum kernels: https://github.com/qiskit-community/qiskit-machine-learning/blob/main/docs/tutorials/03_quantum_kernel.ipynb
Now, there's a use case of mine where I've done feature extraction from a dataset and I've to pass 'train_features' and 'train_labels' to the QSVC for a multi-class classification problem. This is pertinent to a research I'm pursuing to test the performance of some quantum ML methods on different aspects of classical data. Could you provide some tutorial where you could probably showcase one example of a multi-class classification (using any strategies like 'one v/s rest', 'one v/s one' approach) using QSVC. For reference, without divulging too much information of my research, the 'train_features.shape' has 'torch.Size([28248, 7])' and 'train_labels.shape' has a 'torch.Size([28248])'

For such multiclass approaches there is the scikit-learn multiclass functionality. The QSVC is a minimal utility class that extends the scikit-learn SVC to simplify the usage. But as that tutorial shows you can directly use SVC - its really just about how the quantum kernel is passed in, and while it may be computed with quantum, once done it's all the same to SVC so you can leverage tooling from scikit-learn.

This could indeed be useful to many within the community, thanks @pragmatic-philosopher09 for noting this usage and @woodsp-ibm for the suggestion. @pragmatic-philosopher09, could you please post a minimal working example using the scikit-learn multiclass once you've had the chance to delve into it?

Hi @pragmatic-philosopher09 I just got the opportunity to follow up on this topic.
Multi-class classification
Multi-class cases are already supported by QSVC because it directly inherits from the Sklearn SVC, which supports them: see the documentation. in QSVC you can call the 'one-versus-one' decision strategy as
from qiskit_machine_learning.algorithms import QSVC

# Assuming you have already built the kernel, here `adhoc_kernel`
qsvc = QSVC(quantum_kernel=adhoc_kernel, decision_function_shape='ovo')
qsvc.fit(...)

From SKlearn, this decision_function_shape  strategy builds n_classes * (n_classes - 1) / 2 classifiers, each training on the data pairwise. In your case, you will have n_classes=7, resulting in 21 classifiers.
Decision strategy, e.g.  'one v/s rest', 'one v/s one'
Similarly, these can be invoked directly in QSVC because of the inheritance from SVC. You should refer to the SKlearn documentation to check which strategies are supported, and what the defaults are. A note about compliance - to ensure compatibility with other classifiers, as of SKlearn version 0.19 it is sometimes advised to convert to
qsvc.decision_function_shape = "ovr"

I hope this information is helpful and don't hesitate to reach out again for questions or input.


Added ProjectInfo.

I implemented the comments. Ready for another pass.

Looks good. Please change the replaces the @code with @link in the places we discussed (where there was already another @link entry) and feel free to merge after.


Create god-mode for testing

We need a god-mode for level testing. This mode will allow you to go through walls, fly and go ahead on the level.
EXPECTED RESULT:
GodMode.cs file on Scripts folder

Free flyping + no clipping mode (camera)
Reboot level
Trigger ending

This issue was completed on the following commit: da4c51da23cf9c80e795f514313f28bb9ea23a65


Add sorting of the list of variables by name

Please, add sorting of the list of variables by name

.
After ARC restarting all my variables aren't sorted.
Expected outcome
There is an opportunity to change the provision of a variable in the list.
After restart the ARC order of variables in the list doesn't change.
Actual outcome
I can't change the position of a variable in the list.
After ARC restart all variables are located chaotically.
Versions
App: <IP_ADDRESS>5-stable
Chrome: 56.0.2924.87
Platform: Linux (-)

Hi,
Thank you for the issue report.
I will do this when I finish fixing files upload.


VSCode considers *.h files as C, not C++, despite files.associations settings

Steps to Reproduce:

Open a C++ project where the C++ header files have a .h extension.
Check the icons in the file explorer.

This is essentially the same issue as #42279, which was closed.
My workspace settings contain the following (copied as JSON from Settings dialogue):
"files.associations": {
  "*.h": "cpp"
}

To open this project, I am using the "Remote - SSH" extension, although I have no reason to think that is related to the problem.
Extensions:

Local:  Remote - SSH; Remote - SSH: Editing Configuration Files
Remote: None

VSCode details (copied from Help > About):
Version: 1.47.3
Commit: 91899dcef7b8110878ea59626991a18c8a6a1b3e
Date: 2020-07-23T15:51:39.791Z
Electron: 7.3.2
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Linux x64 4.14.186-146.268.amzn2.x86_64

I see the following class on the icon:
monaco-icon-label file-icon test.h-name-file-icon h-ext-file-icon ext-file-icon cpp-lang-file-icon explorer-item

So maybe the file extension class is winning over the language mode?

The seti theme explicitly associates the .h file extension with a c icon with a specific color:
https://github.com/jesseweed/seti-ui/blob/719e5d384e878b0e190abc80247a8726f083a393/styles/components/icons/mapping.less#L26
This overrides our language associations.

We are importing the Seti theme from https://github.com/jesseweed/seti-ui.
To request a change please file an issue there. Maybe they can change the .h icon to a H ?


music.youtube.com Support

it is possible to download music / music video from music.youtube.com? Thank you

yes, you simply replace the url of the Youtube video with that of YTMusic.

yes, you simply replace the url of the Youtube video with that of YTMusic.

Thank you ;)


Connect fullstory session url to user props sent with events

See if possible to update the session url for each user session
If not send the fullstory session url on error events as another param

After i've investigated amplitude docs, client source and reviewed our dashboard i found that amplitude doesn't tracks webpage url. it's not sent with events like some 'internal'/'built-in' property. It could analyze url on initialization to track some UTM tags or other stuff but nothing moire
in this case.
As a conclusion - we wouldn't to find ways to dynamically update current url in amplitude during app navigation.  We could just send it where we need it (e.g. in error handling)

committed directly to master

@serdiukov-o-nordwhale this is about "fullstory session url" not the current url.

What means “fullstory session url” ?
You mean “fullstory” not as common word, you mean FS (FullStory) service we also use in the analytics
Should we pass FS.getCurrentSessionURL result instead of url ?
Could you please tell me what actually we should do in the scope of this task?

@sirpy Updated the implementation: https://github.com/GoodDollar/GoodDAPP/pull/2029

@serdiukov-o-nordwhale asked me to test the analytics on gooddev env.
Here is the initialization log of bug tracking systems:
{
  Amplitude: true,
  FS: false,
  Sentry: false,
  Rollbar: false,
}

FS is not present in global scope - it's undefined - what means that FullStory is not initialized via GTM on gooddev.
The FS is available only on the production environment (tested at wallet.gooddollar.org). So I think it should be tested there.

@AnastasiiaOdnoshevna You could test it once production will be released

replaced with #2334


Docker on Windows 10 - Keep having to reset credentials on shared drive

Description

Steps to reproduce the issue:

Share a drive using the docker settings gui in windows 10 pro (x64)
Created a docker image that maps a directory to a windows directory on the shared drive.
Reboot the windows host and start the image if autostart is not set
4: enter image and go to the shared folder/directory - it is not shared...

Describe the results you received:
Directory and folders mapped from host windows machine to Linux image no longer mapped after reboot. I then have to reset the shared drives credentials and re-setup the windows host shared drive. Once done starting the docker image works as expected and the directories are mapped as needed.
Describe the results you expected:
I expect for the shared drive to remain shared after a reboot of the host machine.
Additional information you deem important (e.g. issue happens only occasionally):
Seems to happen after every reboot
Output of docker version:
Client:
 Version:      1.13.1
 API version:  1.26
 Go version:   go1.7.5
 Git commit:   092cba3
 Built:        Wed Feb  8 08:47:51 2017
 OS/Arch:      windows/amd64

Server:
 Version:      1.13.1
 API version:  1.26 (minimum version 1.12)
 Go version:   go1.7.5
 Git commit:   092cba3
 Built:        Wed Feb  8 08:47:51 2017
 OS/Arch:      linux/amd64
 Experimental: true```

Output of docker info:
Containers: 3
 Running: 0
 Paused: 0
 Stopped: 3
Images: 4
Server Version: 1.13.1
Storage Driver: overlay2
 Backing Filesystem: extfs
 Supports d_type: true
 Native Overlay Diff: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host ipvlan macvlan null overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: aa8187dbd3b7ad67d8e5e3a15115d3eef43a7ed1
runc version: 9df8b306d01f59d3a8029be411de015b7304dd8f
init version: 949e6fa
Security Options:
 seccomp
  Profile: default
Kernel Version: 4.9.8-moby
Operating System: Alpine Linux v3.5
OSType: linux
Architecture: x86_64
CPUs: 24
Total Memory: 11.45 GiB
Name: moby
ID: 4LZ7:PMRV:JDGJ:UVXJ:CAC6:YG2O:JJK4:AA6V:QWMJ:5NUP:6IN6:NY3G
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: 13
 Goroutines: 22
 System Time: 2017-03-02T11:57:15.7370858Z
 EventsListeners: 0
Registry: https://index.docker.io/v1/
Experimental: true
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

Additional environment details (AWS, VirtualBox, physical, etc.):
Windows 10 pro (x64) docker host is running on a physical HP Z600 workstation

Thanks for reporting @smithma01 - it looks like this issue is specific to the Docker for Windows package; could you reopen this in that issue tracker? https://github.com/docker/for-win/issues ?


Show 'default' cursor on certain list views.

Showing a hand cursor on screens that dont have links to other screen was confusing, changed to show 'default' cursor on screens when @embedded is set.
https://bugzilla.redhat.com/show_bug.cgi?id=1178697
@dclarizio please review/test.

Verified fix.


Add ability to generate and use test files instead of spreadsheet.

This opens up the possibility of measuring, optimizing and testing the monolithic generate.js.

This will clean up generate.js a little so that I can optimize specific parts of it and able to test it.
It does not include any tests yet, that will be in a different PR


Cannot set LibreOffice input filter (i.e. --infilter argument)

Hello,
It seems impossible to set an input filter for the conversions, i.e. LibreOffice argument called --infilter .
This parameter is strictly needed for instance for the conversion of PDF files to other formats like DOCX (with value "writer_pdf_import"), otherwise, the input document is parsed as Draw document (instead of Writer) and the conversion does not succeed.
Example of a successful conversion command:
soffice --headless --infilter="writer_pdf_import" --convert-to "docx" "document.pdf"
Example of a broken conversion command:
soffice --headless --convert-to "docx" "document.pdf"
Thank you

Added in v3.1.0


about the comparasion

Hi, How do you reproduce CMSA-MPTP?

I have replicated the CMSA code, and the results are 1-2 % lower than those presented in the paper. However, the results I cited in my paper are those presented by the authors in their paper.

I have replicated the CMSA code, and the results are 1-2 % lower than those presented in the paper. However, the results I cited in my paper are those presented by the authors in their paper.

Hello, I have also reproduced the code for CMSA-MPTP, but the open acc is 20% lower than the paper result, and the all acc is 10% lower than the paper result. Could you please let me know what changes you need to make during the reproduction process? I didn't modify any parameters, but the results were vastly different, thanks!

I have replicated the CMSA code, and the results are 1-2 % lower than those presented in the paper. However, the results I cited in my paper are those presented by the authors in their paper.

Hello, I have also reproduced the code for CMSA-MPTP, but the open acc is 20% lower than the paper result, and the all acc is 10% lower than the paper result. Could you please let me know what changes you need to make during the reproduction process? I didn't modify any parameters, but the results were vastly different, thanks!

Hello, you need to load the model weights made available by the author for validation.

I have replicated the CMSA code, and the results are 1-2 % lower than those presented in the paper. However, the results I cited in my paper are those presented by the authors in their paper.

Hello, I have also reproduced the code for CMSA-MPTP, but the open acc is 20% lower than the paper result, and the all acc is 10% lower than the paper result. Could you please let me know what changes you need to make during the reproduction process? I didn't modify any parameters, but the results were vastly different, thanks!

Hello, you need to load the model weights made available by the author for validation.

ok，I get it ,thanks!


ubuntu 20.04 running hello_triangle fails with "Failed to find a create function for"

I built the samples project per the linux build guide. I have vulkan-sdk from the lunarg repo. Build did not show any errors.
$ ./build/linux/app/bin/Release/x86_64/vulkan_samples hello_triangle
[info] Logger initialized
[info] Window created
[error] [build/linux/app/main.cpp:73] Failed to find a create function for

(Yes, the last line ends with "for")

My vulkaninfo
I have both clang and gcc installed. I'm not sure which one cmake picked.
Xubuntu 20.04
nvidia 1080 using proprietary driver 460.80
radeon x570 using mesa (I think)

FWIW, issue is due to there being code with side-effect inside assert() statements, here is a little one-line fix for whats causing your issue...
index d41ad7b..d97fb27 100644
--- a/framework/platform/parser.h
+++ b/framework/platform/parser.h
@@ -217,7 +217,8 @@ class CommandParser
                auto values = get_command_value(command);

                Type type{};
-               assert(convert_type(values, &type) && "Failed to retrieve value. Type unsupported");
+               auto status = convert_type(values, &type);
+               assert(status && "Failed to retrieve value. Type unsupported");
                return type;
        }

@@ -301,4 +302,4 @@ inline bool CommandParser::convert_type(const std::vector<std::string> &values,
        return true;
 }

Thanks @jamesdolan . That got it past that error, so I consider the above patch to be a fix, and suggest that the repo maintainers include it. I am on the next error now.
$ ./build/linux/app/bin/Release/x86_64/vulkan_samples hello_triangle
[info] Logger initialized
[info] Window created
[info] Initializing vulkan instance.
[info] Enabled Validation Layers:
[info] Found GPU: AMD RADV POLARIS10 (ACO)
[info] Found GPU: GeForce GTX 1080
[info] Initializing vulkan device.
[error] [samples/api/hello_triangle/hello_triangle.cpp:319] Did not find suitable queue which supports graphics, compute and presentation.
Segmentation fault (core dumped)

At least this error means something.

index d41ad7b..d97fb27 100644
--- a/framework/platform/parser.h
+++ b/framework/platform/parser.h
@@ -217,7 +217,8 @@ class CommandParser
                auto values = get_command_value(command);

                Type type{};
-               assert(convert_type(values, &type) && "Failed to retrieve value. Type unsupported");
+               auto status = convert_type(values, &type);
+               assert(status && "Failed to retrieve value. Type unsupported");
                return type;
        }

diff --git a/samples/api/hello_triangle/hello_triangle.cpp b/samples/api/hello_triangle/hello_triangle.cpp
index 3f2b8e7..78ea885 100644
--- a/samples/api/hello_triangle/hello_triangle.cpp
+++ b/samples/api/hello_triangle/hello_triangle.cpp
@@ -277,8 +277,9 @@ void HelloTriangle::init_device(Context &                        context,
        std::vector<VkPhysicalDevice> gpus(gpu_count);
        VK_CHECK(vkEnumeratePhysicalDevices(context.instance, &gpu_count, gpus.data()));

-       // Select the first GPU we find in the system.
-       context.gpu = gpus.front();
+       for(size_t i = 0;i < gpu_count && (context.graphics_queue_index < 0);i++) {
+
+         context.gpu = gpus[i];

          uint32_t queue_family_count;
          vkGetPhysicalDeviceQueueFamilyProperties(context.gpu, &queue_family_count, nullptr);
@@ -313,6 +314,7 @@ void HelloTriangle::init_device(Context &                        context,
                  break;
                }
            }
+       }

        if (context.graphics_queue_index < 0)

This will be fixed with PR #288

There's two issues reported here.  The first is indeed addressed in PR #288 however, the second, fixing the assumption that the first GPU found in the system is the correct one to use is indeed another issue that needs to be fixed by a PR.  I'd like to suggest submitting a PR if you're willing @memory-thrasher to ensure you maintain proper recognition and attribution for both your issue discovery and fix submission.

@gpx1000
I am. I want to look into disposing of the unused gpu objects correctly, and then apply it to all the samples. Should be ready before the weekend.


Adding images

Hi !
I would love to help you out with this implementation but once I started looking at the code I realize I know nothing about XLS formatting.
Is something you would like to add in the near future?
The reason I use your implementation over PhpSpreadsheet is that yours is lightweight, simple and also they require PHP >= 7.2
Thank you!

no, Ezequiel, any drawings in tables not our way


bug: publishing to routes has assorted issues

We're noticing some issues with developing/publishing to routes with wrangler 2. Using this as an umbrella issue because some of them may be related, but we can investigate them all together.

https://github.com/cloudflare/wrangler2/issues/1003
https://github.com/cloudflare/wrangler2/issues/652
https://github.com/cloudflare/wrangler2/issues/651
https://github.com/cloudflare/wrangler2/issues/603

Assigning this to @caass as she is working on the last issue in this list.

All the sub issues have been fixed, so I'm closing this parent issue. Great work everyone!


Azure file SDK - Put Range method cannot be modified to add content-type header

Package Name:
Azure.Storage.Files.Shares,  version 12.12.17
Details:
When putting Azure File endpoint behind Azure App gateway WAF, the requests must have content-type as multipart/form-data to be identified as a legit upload.
Looking at https://learn.microsoft.com/en-us/rest/api/storageservices/put-range, there is no support for modifying the content-type header as it will always be octet-stream.
I understand that content-type can be modified while creating the file but AppGW needs the subsequent Put Range API call to have content-type as multipart as wel..

Hi @mahmoudsamy27.  Thank you for reaching out and we regret that you're experiencing difficulties.   Would you please share the Azure SDK package that you're using and its version?

This is the SDK version https://www.nuget.org/packages/Azure.Storage.Files.Shares/12.12.1?_src=template - .NET SDK.

@mahmoudsamy27 Apologies for the late reply. Thanks for reaching out to us and reporting this issue. We are looking into this issue and we will provide an update.

@mahmoudsamy27 If the content-type isn't specified at the time of creating the file, it will default to application/octet-stream. When you call Create File, you only initialize the file. To add content to a file, you call the Put Range operation.

Content-Type or x-ms-content-type
Optional. The MIME content type of the file. The default type is application/octet-stream.

I tried to do a repro using Azure Portal and Storage explorer and they all behave as per the design. Content-type appears only during create and not for Put range operation.
If you would still like to change the content-type of the file after uploading it to the storage account, you could go to tyhe properties and change it type as shown below:

Note: I am using storage explorer here to edit the file properties.


Wrapped Select and Chevron in a div for NativeSelect component

Fixes #305

Should I remove this relative class here?

https://www.shadcn-svelte.com/examples/forms/appearance
src\routes\examples\forms\appearance\appearance-form.svelte
I think this was added to solve the same issue discussed in #305.

Yeah if we're already adding a relative container into it we shouldn't need it. When you try removing it as long as everything still looks good for that example form I'm cool with dropping it.

All good. Added the Removed class commit.


feat(dialog): allow focus restoration to be disabled

Allows for the dialog focus restoration to be opted out of, in case the consumer wants to handle it themselves.
Fixes #12515.

Whats the version for this feature?


Bug in Tilemap.createFromObjects

I think there might be a bug in createFromObjects where it iterates over the objects searching for a matching gid, id, or name.  The variable found is only set to false once outside of the loop and so if there is a match it will then continue to create a sprite for every object after the match even if those objects don't have a matching gid/id/name.  I've made the changes and tried it out and it seems to work more as expected with found being set to false at the start of each loop.
I could make a pull request if this is actually a bug.

Yup good call - have fixed that method up and optimised it quite a bit now too.

Awesome, thanks! :+1:

Please test it - I've only got a couple of test cases here I can use and there are so many versions of Tiled out there, outputting all kinds of files it would be good to get more coverage.

Thank you, I can confirm this fixes the issue for me as well, I can't wait for you to push this release to npm

Ok great. I'll be pushing it out this week.

Sorry, I didn't see a notification for your last response and Ludum Dare has kept me busy this weekend.  It fixed the issue for me as well.


2094-10 "tone-mapping-param" messing up brightness and saturation

Hi, I have a problem with the "tone-mapping-param" with 2094-10. It is constantly shifting around and there is no value to fix it. It somehow resets for every new scene. If you are above (+)0 it effects some scenes and of you are under (-)0 then it effects the others. I just want to not be effected by this "feature".
And this should be possible, because when you use glsl tone mapping (which I don't have for 2094-10), then "tone-mapping-param" never effects the image.
There are conversations, where the one person is "normal" and when the camera goes to the other person in the dialog, then it messes with the brightness, which desaturates the picture.
The way it should look

boosted brightness and desaturated picture

report mpv's issue to mpv, tone-mapping-param or st2094-10 not my works.

solved: set "tone-mapping-param" to 0.400-0.500


Include DDSec example solutions

I attended the DDSec workshop at DDD Europe where we used this app to discuss how domain driven design can help secure the application.
I think all the vulnerabilities that are mentioned in the README were discussed but I'm not sure if we covered all the possible solutions. I can fork the project to implement my own solutions but it would be nice to have reference included with the project.
If you don't want to make them too obvious, you can obscure them by putting them on a separate branch?

There are some possible solutions crafted, but they have not made it into the repo yet. They will probably end up as a branch, or several.


adds sourcemap support

I followed the gulp-concat plugin and added support for sourcemaps.
I had to add concat-with-sourcemaps as a dep.
Please review and let me know if anything has to be changed.

:+1:

This is great but it won't work with other sourcemaps in the pipeline. Check out how gulp-concat does it - you need to use applySourceMaps

@madhums I'd suggest following @contra's advice on this.. beyond that, it looks like the test are failing.  Please add/update the tests accordingly.

@contra thanks, I will do it tonight. @tracker1 some tests are failing in the newly released iojs 3.0.0. It works well in 2.5.x, but I will try to fix it.

@contra I added the Flush stream method where the sourcemapped content is pushed. I was thinking that concat-with-sourcemaps would do the applySourceMaps stuff (as this is how it is done in gulp-concat).
Let me know if this seems right. If not, should I use an ast parser and then applySourceMaps?
I tested the above in the following task, it seems to work fine.
var header = require('gulp-header');

return gulp.src(files)
  .pipe(sourcemaps.init())
    .pipe(concat('app.js'))
    .pipe(header('window.locales = { en: {} }'))
    .pipe(gulpif(minify, uglify({ mangle: false })))
    .pipe(header(banner, { pkg: pkg }))
  .pipe(sourcemaps.write('./'))
  .pipe(gulp.dest(dist + '/'));

Ah yeah, concat-with-sourcemaps does handle that - my bad. LGTM

Ok great! @tracker1 can you review the PR?

LGTM...

closes #7

@madhums Thanks for your great work on this.

:+1:


How to append userAgent, not replace?

I want to append some properties to user-agent, but if I set userAgent in initialOptions，the user-agent will only has the value that I set，So，is there a way to append userAgent, not replace?

@hyden-tan
Hi, there. I am also looking for a way to the same exact question.
Have you found any solution for this?


fix lint test

Which problem is this PR solving?

(https://github.com/jaegertracing/jaeger/issues/5506)

Description of the changes

fixes the make lint check that was failing

How was this change tested?

Checklist

[ ] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
[ ] I have signed all commits
[ ] I have added unit tests for the new functionality
[ ] I have run lint and test steps successfully

for jaeger: make lint test
for jaeger-ui: yarn lint and yarn test

there are no lint rules that are currently failing on the main branch. Where do these come from?

there are no lint rules that are currently failing on the main branch. Where do these come from?

when I run make lint locally I was getting these error's
INFO [runner] linters took 727.619655ms with stages: goanalysis_metalinter: 702.156146ms
cmd/anonymizer/app/anonymizer/anonymizer.go:129:12: G306: Expect WriteFile permissions to be 0600 or less (gosec)
        if err := os.WriteFile(filepath.Clean(a.mappingFile), dat, os.ModePerm); err != nil {
                  ^
cmd/internal/flags/admin_test.go:59:2: negative-positive: use assert.Positive (testifylint)
        assert.Greater(t, port, 0)

you may have different version of the tools, try make install-test-tools

Most of the changes proposed are much worse than what we have.


Document return value of uv_get_constrained_memory if no limit is set.

Document that we return UINT64_MAX if the cgroup limit is set to the max. For cgroupv2, that happens if we encounter max, while cgroupv1 returns<PHONE_NUMBER>854771712 when no limit is set (which according to https://unix.stackexchange.com/questions/420906/what-is-the-value-for-the-cgroups-limit-in-bytes-if-the-memory-is-not-restricte is derived from LONG_MAX and PAGE_SIZE).
Addresses https://github.com/libuv/libuv/pull/3744/files#r974062912

Rebased on top of https://github.com/libuv/libuv/pull/3754, but apparently I can't change the base to my fork.
@bnoordhuis Is this something we want? It seems useful to document what we currently return already for cgroup2's "max" (and make sure we do the same for cgroup1).


ISIS-1967: Add tests from Diffblue Deeptest

The test cases contained in this PR were automatically generated by Diffblue's Deeptest software.
These tests, and others similarly generated, can be viewed at http://diff.blue/demo.
As discussed with Andi Huber on<EMAIL_ADDRESS>this relatively small selection of tests are provided for assessment and we are prepared to supply further test cases via subsequent PR's if you are happy to receive them.
We would be very interested to hear any feedback you have on these tests as we aim to continuously improve them.

Thanks Peter!
We are now able to review your submission and discuss internally how to proceed. Hopefully just within a few days, we should be able to give you feedback.
Best regards, Andi

Thanks Andi
I really appreciate you taking the time to consider my PR and providing feedback on the tests proposed for inclusion in isis.
We are on the way to resolving some of your concerns:

Naming of tests is something we had already identified for improvement and, in fact, the latest analysis includes these changes.
As an example:
public void org_apache_isis_applib_internal_base__Bytes_prepend_001_8ef33eec105cda50() throws Throwable {
is now:
public void prepend1() throws Throwable {

Deeptest software aims to identify all paths through the code and produces a test for each case. The idea is to maximize coverage and remove the possibility that an area of code is not covered. This means a large number of tests will be generated - these are compiled and run by the software with only those which pass being made available by default. It is not suggested that each of these is manually inspected because they represent the current working code and serve to track whether that original behaviour changes in the future (due to a bug or re-design)

We do not currently support dependency injection but we are working on it and can let you know when it is available.

Thanks again for your time. We would be happy to provide further tests for your consideration, after the naming and appearance-related changes which have been made recently, and to discuss any issues remaining you may have.
Best regards
Peter


If there are 2 fszip, which one to select?

Needs to add an input for that

I would select always the latest, if they don't want that, then they will
need to be more restrictive or specific on the wildcard, no?
On Mon, Jun 17, 2024 at 7:22 PM Yntiglei @.***> wrote:

Needs to add an input for that
—
Reply to this email directly, view it on GitHub
https://github.com/garikoitz/launchcontainers/issues/74, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABCZAV3A7SHD7DUG6ZQLJSDZH4LNHAVCNFSM6AAAAABJONOFFCVHI2DSMVQWIX3LMV43ASLTON2WKOZSGM2TOOBUGYZDCMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I agree, I implemented to select new


question about d435 pointcloud origin and Intrinics matrix

Required Info

Camera Model
{  D435 }

Operating System & Version
{ Linux (Ubuntu 16)

Kernel Version (Linux Only)
(16.04)

Issue Description
Hi,guys.I have encountered some difficulties when using the camera to locate the object on a planeI So I want to confirm the following facts.
1.The X-Y depth origin coordinates of D435 is from centerline of  1 / 4 - 20 to left imager,which means that if I want to generate point cloud  with left imager as the origin, I have to add 17.5 to the x component of each point. Am I right ?
2.The intrinics Matrix I get from rs2 have a little different from the one I get from Matlab. which one should I choose to use?

RealSense stereo cameras tend to rely on the left imager to generate the image, so you probably would not have to use a special procedure to generate data from the left imager.
The RealSense 400 Series data sheet documents says:  "The left and right imagers capture the scene and sends imager data to the depth imaging processor, which calculates depth values for each pixel in the image by correlating points on the left image to the right image and via shift between a point on the Left image and the Right image".
The dynamic calibration guide for the 400 Series, meanwhile, says: "Assume left camera is the reference camera and is located at world origin".
If you want to translate 2D coordinates to 3D point cloud coordinates then you can do so with an SDK instruction called rs2_deproject_pixel_to_point.
You can read about coordinate systems, intrinsics and extrinsics in the SDK's 'Projection' documentation.
https://github.com/IntelRealSense/librealsense/wiki/Projection-in-RealSense-SDK-2.0?language=en_US

Thanks for the generous help.But I still confused about 4.9.1 depth origin x-y coordinates and table 4-15 in d400-series datasheet.What does it mean?Sorry for not pasting picture because I don't know how to do it by mobile phone.

A note below the table says that 1/4 - 20 refers to the center-point of the screw-hole on the underside of the camera that a camera tripod stand can be attached to.  It looks as though the hole is the approximate center-point of the length of the camera.
I have never known it to be necessary to do a calculation with the 17.5 value to generate a point cloud. with a RealSense 400 Series camera  The rs2_deproject_pixel_to_point instruction usually handles most of the work on behalf of the user.  The SDK's 'Measure' sample program is a good starting point to learn about this instruction.
https://github.com/IntelRealSense/librealsense/tree/master/examples/measure#converting-between-pixels-and-points-in-3d
Unless there is a particular reason why you need to convert 2D coordinates to 3D, you might find it easier to just generate a point cloud directly.  Below are a couple of example programs, a simple one and a more complex one.
https://github.com/IntelRealSense/librealsense/wiki/Projection-in-RealSense-SDK-2.0#point-cloud
https://github.com/IntelRealSense/librealsense/tree/master/examples/pointcloud

I think I probably understand, thank you again for your patience.

What is the deafultt Depth Origin if I dont specfically do anything.So far I assumed RGB Camera is the Origin of XYZ Coordinates.You mentioned like this "Assume left camera is the reference camera and is located at world origin"."What does this mean??By default left imager is the Orgin

The 0,0,0 depth origin by default is the center-line of the left infrared sensor.  If depth-to-color alignment is performed though then the color sensor will become the origin.


Feature group detail

Can create calendar of each group and manage the event together.

The branch runs correctly, can merge


build for windows only connecting

OK i think i got this installed right but i can Not get it to stream to my roku
all it does is saying  state: connecting....  and then closed....
i have used this code:
 castnow --myip <IP_ADDRESS> --address <IP_ADDRESS> Untitled.mp4
i have also tried:
castnow https://www.youtube.com/watch?v=pcVRrlmpcWk --myip <IP_ADDRESS> --address <IP_ADDRESS>
I have tried my roku over LAN And wifi  and i can curl into it to check channels  on port 8080 and get channels back so i know that i can connect to roku..
is this a firewall issues on my windows machine?
or is this NOT using the old Dial protocol that chromecast 1 used?

castnow is for the chromecast (made by google). The roku is a whole other device, so this will probably not work for roku.

I think this does not have anything to do with windows. Are you able to test against a chromecast?

i dont have a chromecast but i am, sure it says that it works with the roku.. roku uses the dial protocol as well


Possible to remove the sidebar?

Hi,
I really like this theme's appearances and functions, but I want to include my profile photo in the main text instead of side bar, like academic-minima theme (https://github.com/academicbio/academic-minima). Is it possible to remove the sidebar?

We can turn on/off the side bar by setting the boolean author_profile to true/false. They are in the _config_yml at top level of the repo. I think you can also go to each specific page, and overwrite that boolean at beginning of the html code.

Thank you. I turned off the sidebar!


Add support for Codecov

This change is

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@c55cba5). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             dev      #69   +/-   ##
======================================
  Coverage       ?   60.02%
======================================
  Files          ?        1
  Lines          ?      988
  Branches       ?        3
======================================
  Hits           ?      593
  Misses         ?      392
  Partials       ?        3

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 c55cba5...a8b04c1. Read the comment docs.

:LGTM :

Reviewed 5 of 6 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Null handling is incorrect with $in operator

Hello,
We've found a bug with the usage of the $in operator when we pass in a null value. Normally, passing a null to the in operator would indicate that absence of the field should be reported back by the query, as documented here. Fongo disregards the null value though and does not behave the same as MongoDB on the null handling. The following test would fail in ExpressionParserTest with the latest code base:
@Test
public void testInOperatorWithNullValue() {
  // Querying for null should return positive if the field is absent.
  // See: http://docs.mongodb.org/manual/faq/developers/#faq-developers-query-for-nulls
  DBObject query = new BasicDBObjectBuilder().push("a").add("$in", asList(2, 3, null)).pop().get();
  List<DBObject> results = doFilter(
      query,
      new BasicDBObject("a", asList(1, 3)),
      new BasicDBObject("a", 1),
      new BasicDBObject("a", 3),
      // Include an element with no 'a' field: this should get returned.
      new BasicDBObject("b", 1)
  );
  assertEquals(Arrays.<DBObject>asList(
      new BasicDBObject("a", asList(1, 3)),
      new BasicDBObject("a", 3),
      new BasicDBObject("b", 1)
  ), results);
}

I will open a pull request with this test along with the one-liner fix that should make it pass. Feedback on it would greatly be appreciated as we forked the project for now to patch a past version we need to use.
Thanks a lot for this library as this saves us lot of troubles for testing our MongoDB layers at Expedia!

Thanks !


gbc cpp generates ambiguous references when a struct and enum val share a name

When compiling a bond file containing a struct and an enum value with the same name, gbc generates C++ that makes unqualified references to that name.
For example, this bond file:
namespace conflict;
struct Foo {};
enum E { Foo; };

builds with the latest master. Trying to compile the resulting conflict_apply.cpp with clang causes a number of errors like this:
./conflict_apply.h:28:31: error: reference to 'Foo' is ambiguous
    bool Apply(const bond::To<Foo>& transform,
                              ^
./conflict_types.h:35:12: note: candidate found by name lookup is 'conflict::Foo'
    struct Foo
           ^
./conflict_types.h:97:13: note: candidate found by name lookup is 'conflict::_bond_enumerators::E::E::Foo'
            Foo
            ^

Bond version 4.3.0 is now live on NuGet.org. These changes are included in that version.


Show a more helpful error message when plugins cannot be imported

Describe the bug
If a dbt plugin cannot be imported, dbt will show a message about invalid credentials, eg:
Running with dbt=0.14.3
Error importing adapter: cannot import name 'collections_abc'
Encountered an error while reading profiles:
  ERROR Runtime Error
  Credentials in profile "<profile name>", target "dev" invalid: Runtime Error
    Could not find adapter type bigquery!
Defined profiles:
 - ....

If this is an error that occurs at import-time, dbt should not indicate that the credentials are invalid. dbt should also not show a list of defined profiles in this error message.
Desired output:
Running with dbt=0.14.3
The dbt plugin 'bigquery' could not be imported: cannot import name 'collections_abc'

dbt version: v0.14.3. I need to test against 0.15.0 to determine if this behavior has recently changed.

closed by #2022


Dasky thoughts for awkward

Following #263, it's now possible to pull big data into awkward of a form that outsiders from the project will appreciate and allow, for instance, parquet and orc reading (this may already exist). This raises the interesting question of how people in the wider community (i.e., non-root) might interact with awkward for nested dataset processing, especially where that data is big enough to need parallel and/or out-of-core processing with dask. This is different from the original question me and @jpivarski talked about, how to make use of Dask within awkward.
Firstly, in my opinion, an awkward array is best seen as a one-dimensional array of things, where those things have structure (whether variable-length components, structs or maps), and so maps best into existing python-land as a pandas series or column. I understand that much of the current coding style is numpy-like, and it may seem that of the dask collections, it most resembles the dask-array codebase. However, usage, to my mind, will be along the lines of
df = ak.read_parquet(awkward_columns=[...])  # other columns are simple pandas types; or we expose columns for extraction
df[filter_multi_column]
df.col1.map(numba_func)
df.col1.agg(numba_iterating_func)

where the map could be written df.map_partitions(lambda d: d.map(numba_func) but the aggregation required the same combination logic as any dataframe combinations currently do. The filter step would require the ability to iterate over the awkward array(s) as well as ordinary pandas series, together.
A few notes:

the pandas codebase and indeed the dask.dataframe one respects the numpy protocol, so that (g)ufuncs are sensibly dealt with, and f(akarray)->akarray should work fine
wrapping awkwards arrays, or indeed seeing the whole arrow structure as a top-level record in awkward, is a very light-weight process
to enable the conversion to dask.dataframe, you would need simply to run arrow->awkward extension array on each piece, and not have to worry about chunks in awkward at all
conversely, if you happen to be starting with a chunked akarray (from root?), then one chunk becomes one partition, and, again, none of the partitions themselves are chunked; the only information you would currently loose is the row-count of each piece
caching of virtual arrays would not be handled in the first instance, but it would not be hard to do something sensible at the dask worker level; the scheduler will tend to keep tasks where an array piece was previously loaded into memory and round-robin pieces that have no dependencies on each other

--
In addition, to kick off this conversation of awkward integration, I was confused in trying to build examples, so I have a couple of gists to share:

https://gist.github.com/martindurant/6a95e20bce620471462062083d1ac1ff#file-time1-py shows that although len mapped on a awkward pandas series is a few times faster than when mapped on a list-of-lists series; but only as fast as a python comprehension on the original lists, and much slower than it should be with numpy
https://gist.github.com/martindurant/6a95e20bce620471462062083d1ac1ff#file-ak-ipynb I couldn't actually get complex numba-jit functions to run, perhaps due to confusion over map and struc-like things when converting from python/json. I would have thought you could process maps too by iterating on the keys or key/value tuples.

This is a good line of thought, and I agree that the best way to think of these things is as one-dimensional arrays, like they way they're represented in Pandas.
I added some comments to the gists.

This will still be searchable, but I don't see any action items in it, so I'm going to close it.

I spent some time thinking about this and playing, and I believe ultimately awkward arrays should be a new collection type in Dask. Here is why the existing ones won't work out:

DataFrame: the conclusion of #350 is that it is inappropriate to make awkward arrays a special Pandas column dtype, and anyway as I understand, there was some issue with conflicts in the awkward extension of pandas and the dask extension of pandas. Certainly the fact that pandas is extensible is enough of a feat to engineer that having two separate packages extend it without conflict is even harder. (c.f. also jax extensions of numpy vs. awkward)
Array: Dask makes great use of the fact that the full shape is known to design partitions and chunk work appropriately. We immediately have to toss all of this out since only the first dimension shape is known. (an exception is when all nodes are Record or RegularArray, so maybe those can be made compatible)
Bag: This is the first collection type that awkward could conceivably extend, because it is conceptually very similar to the awkward PartitionedArray albeit with no knowledge of partition sizes. The downside is that all bag operations are geared towards the assumption that both an outer loop over partitions and an inner loop over entries in partitions is carried out in pure python. There is an escape valve via map_partitions but clearly we are not able to use most of the Bag functions (such as max, mean, repartition, etc.) in an efficient manner out of the box.

I sketched a simple bag extension:
import numbers
from operator import getitem, add
from numpy.lib.mixins import NDArrayOperatorsMixin
from dask.bag import Bag, map_partitions, from_delayed

class AwkwardBag(Bag, NDArrayOperatorsMixin):
    def __array_ufunc__(self, ufunc, method, *inputs, **kwargs):
        if "out" in kwargs:
            raise ValueError("awkward dask ufuncs cannot use 'out'")

        if method != "__call__":
            raise ValueError("awkward1 only supports ufunc call method")

        for x in inputs:
            if not isinstance(x, (numbers.Number, AwkwardBag)):
                return NotImplemented

        return map_partitions(ufunc, *inputs, **kwargs)

    def __getitem__(self, key):
        if isinstance(key, tuple):
            if isinstance(key[0], numbers.Number):
                raise NotImplementedError("We need to know partition sizes to find the right position")
            elif isinstance(key[0], slice):
                if key[0] == slice(None):
                    pass  # We can do this
                else:
                    raise NotImplementedError("We need to know partition sizes to correctly slice")
            else:
                raise NotImplementedError("Maybe its an array? we could handle that if we partition it to match")
        return map_partitions(getitem, self, key)

    def __getattr__(self, key):
        if key in dir(type(self)):
            return super(DaskAwkwardArray, self).__getattribute__(key)
        # it would be nice to have the Form to check an item is available
        return map_partitions(getattr, self, key)

    @classmethod
    def from_bag(cls, bag):
        """Convert a bag of json-like python objects to awkward arrays"""
        return bag.map_partitions(ak.from_iter)

    @classmethod
    def from_delayed(cls, items):
        out = from_delayed(items)
        out.__class__ = cls
        return out

with an example of use:
import random
import numpy as np
import awkward1 as ak
from dask import delayed

@delayed
def gen_partition(i):
    return ak.zip({
        "x": i * np.ones(100),
        "y": ak.from_iter([range(random.randint(0, 6)) for _ in range(100)])
    })

abag = AwkwardBag.from_delayed(map(gen_partition, range(10)))
(abag.y[:, :2] + 3).reduction(lambda x: x, ak.concatenate).compute()

where once the partitions of the bag are generated, any broadcastable operation (such as the add ufunc here) are going to be computed in the fast path for each partition. Then to bring the results back I have to concatenate the arrays, resulting in:
<Array [[3, 4], [3, 4], ... [3, 4], [3], []] type='1000 * var * int64'>

Since this seemed promising, I went further and implemented a custom collection that tracked partition sizes, and am looking into saving the awkward Form (which is known ahead of time for my use case, but could be inferred when loading a dataset similar to how columns would be inferred when reading many csv files). The Form would allow checking that fields exist during the graph construction, and maybe could validate certain structure operations as well.
Many of the ak.* namespace functions that we would want to work with these arrays (think ak.mean(abag, axis=None), etc.) in principle have already been written for PartitionedArray (which is, conveniently, pure python) but the assumption there is that all operations are eager (e.g. the general reduce). Perhaps something as simple as wrapping each such eager op with dask.delayed() is all that's needed.

Please see also https://github.com/martindurant/awkward_extras/issues/1 for discussion on the pandas extension array path - I'd say it certainly can be done for dask-dataframe. Whether it is the right or best model is another matter,


Add new .bbox() to EntityBuilder

.bbox(BoundingShape) which simply creates a new HitBox and delegates to an existing bbox()

Closed by #960


Add cron-send-security-updates-count

@kwoodson gave 👍 in stg PR #2138. Planning to merge on 2017/02/27


Negated binding does not call converter in version 0.9 Preview 7

With the xaml below, the converter is only called by the TextBlock containing IsVisible="{Binding Name ....}, and not by the one containing IsVisible="{Binding !Name ...}"
            <DataGridTemplateColumn Header="Name">
              <DataGridTemplateColumn.CellTemplate>
                <DataTemplate>
                  <StackPanel>
                   <TextBlock IsVisible="{Binding Name, Converter={StaticResource StringToBoolConverter}}" Text="{Binding Name}" />
                   <TextBlock IsVisible="{Binding !Name, Converter={StaticResource StringToBoolConverter}}" Text="No Name Specified" />
                  </StackPanel>
                </DataTemplate>
              </DataGridTemplateColumn.CellTemplate>
            </DataGridTemplateColumn>

Also doesn't seem to work in 0.8.3

The problem here is that the ! (not) is applied to the string value before the converter gets hit.
If the value of Name isn't convertable to bool (i.e. it's not "true" or "false") then the whole binding errors out, meaning that the converter doesn't get called.
I think I'm going to mark this as expected behavior because it seems correct to me
: !Name should produce a boolean value, and that value should be passed to the converter. If the value cannot be converted to a boolean then !Name is in error.

That seems highly non-intuitive to me, and seems to defeat the entire purpose of the converter.  If the value of the string could be converted to a boolean without the converter, then why would I need a converter?  Especially since the version without the negation works.  They should either both fail, or both work.  The lack of consistency is bad.
Perhaps there's a better solution to what I'm trying to do.  If the string is null or whitespace, I want a textblock to appear that says "No Name Specified".  Even if there is a better solution, intuitively, the negation should be applied to the result of the converted value if a converter exists, because that's the way the non-negated version works.

Yeah I understand, but I ask you to think about it like this:
Our property StringValue is "foo". If you have a version without negation or a converter, it obviously won't work.
IsVisible="{Binding StringValue}"
Here, '"foo"` can't be converted to a bool, so the binding fails. Now you add a converter:
IsVisible="{Binding StringValue, Converter={StaticResource FooToBoolConverter}"
The converter is added so that it takes as its input the output of the binding, so in pseudocode it goes boundValue => Converter.Convert(boundValue)
Now lets take out the converter again, and add the ! operator:
IsVisible="{Binding !StringValue}"
Here again, '"foo"` can't be converted to a bool, so the binding fails. Now you add a converter to this:
IsVisible="{Binding !StringValue, Converter={StaticResource FooToBoolConverter}}"
What should be passed to the converter? Remember that !StringValue cannot be done, and the converter takes as its input the output of the binding.

Perhaps there's a better solution to what I'm trying to do. If the string is null or whitespace, I want a textblock to appear that says "No Name Specified".

The most obvious solution would be to write your converter so that it takes the string value and outputs the value you need, without the !.

That's the solution that I ended up with as a workaround.  It works, but seems...I don't know...wrong.  I guess I'm having problems seeing the point of converters given that the converter takes the output of the binding.  That seems backwards to me, it seems that the binding should be the output of the converter.
Maybe an alternate solution would be to allow passing parameters to the converter.  Or maybe I'm trying to shoehorn functionality that was never intended out of converters.

it seems that the binding should be the output of the converter

The final binding value that is sent to the property is the output of the converter, but something needs to be sent to the converter, and that something can only come from the Path part of the binding (i.e. Name or !Name in this case). I think what you're asking for is the converter to be inserted between the ! and the Name, which would be confusing in different ways!

an alternate solution would be to allow passing parameters to the converter.

There's already ConverterParameter for that! :D

I think what you're asking for is the converter to be inserted between the ! and the Name, which would be confusing in different ways!
yes, that seems intuitive to me.

There's already ConverterParameter for that! :D
I wasn't aware of these dark arts! :D

That sounds like what I actually want, but then I'm confused as to what I would put in the IsVisible tag above.
Can I just:
IsVisible="{Binding Converter={StaticResource FooToBoolConverter}"

Then add whatever xaml is needed for ConverterParameter?

I wasn't aware of these dark arts! :D

It's the same in pretty much all XAML frameworks, so I wouldn't exactly call it a dark art ;)

I'm confused as to what I would put in the IsVisible tag above.

You could take advantage of the built-in StringConverters for that:
<TextBlock IsVisible="{Binding Name, Converter={x:Static StringConverters.IsNotNullOrEmpty}}" Text="{Binding Name}" />
<TextBlock IsVisible="{Binding Name, Converter={x:Static StringConverters.IsNullOrEmpty}}" Text="No Name Specified" />

Nice, that looks like exactly what I'm looking for.  Keep in mind, this is the first XAML framework I've ever worked with.  I've been a C++ developer in Unix for the last 25 years.  ALL of this XAML stuff is dark arts to me! :D

Yeah, and we need a lot more documentation for this stuff! I will add info about the built-in converters to the docs.

Thanks for helping me out, and for spending the, I'm sure, vast amounts of time you spend on this project.
It's awesome to finally have a good way to write xplat UI and code.

Thanks for the kind words @lenonk, it really is appreciated!


Error while screen sharing: WGL: Failed to make context current: The requested transformation operation is not supported.

I get the error bellow while displaying images in a for loop. It happens very often when I share my screen via Microsoft Teams. The program terminates after that. Happens to my colleagues too.
We all use julia 1.6.3 and GLMakie 0.4.6
julia --sysimage ..\..\julia_sysimages\glmakie_sysimage1.so .\core\player\player_test.jl

Code:
while (.....)
  fig, ax, plotobj = heatmap(rotr90(img_data), colormap=:gray1)
  heatmap!(rotr90(img_data2), colormap=(:gray1, 0.5))
  heatmap!(rotr90(ix3), colormap=(:prism, 0.5), colorrange=(1/30, 1/5))

  screen = display(fig)
  wait(screen)
end

Error:
GLFWError (PLATFORM_ERROR): WGL: Failed to make context current: The handle is invalid.

How are you screen sharing? Maybe that will help to at least reproduce it...

Well, the usual screen sharing in Microsoft Teams, in a meeting. Are there more settings behind it? We have also set the vsync to false, via setwindowconfig and the error still appear. We have nvidia gpu-s, windows 10. I'm wondering if the nvidia driver is used via screen sharing. Maybe the Microsoft Teams switches to the integrated intel graphics driver while screen sharing.
I do remember from the past that I was unable to work with nvidia cuda devices with the Remote Desktop from Microsoft Windows. I had to use another remote desktop app for this.

Hm switching GPUs in the middle of things could certainly lead to such behaviour, so that is a good theory....

I haven't seen it happening for a while, we can close the issue.


Vim9: can shadow script-local variable with function argument

Describe the bug
In Vim9 script, we can shadow a script-local variable with a function argument.
To Reproduce
Run this shell command:
vim -Nu NONE -S <(cat <<'EOF'
    vim9script
    var name = 'script-local'
    def Func()
        var name = 'function-local'
    enddef
    defcompile
EOF
)

E1054 is raised:
E1054: Variable already declared in the script: name

This is expected.  Shadowing is disallowed in Vim9 script:

As  mentioned,  in  Vim9  script  shadowing  is  disallowed. This  is  to  avoid
mistakes. Since the declarations  are in one file, it should  always be possible
to rename one of the variables. Shadowing global variables, which can be defined
and unlet  any time, is allowed. But  since you need  to use the g:  prefix this
isn't real shadowing.

Source.
Now, run this other shell command:
vim -Nu NONE -S <(cat <<'EOF'
    vim9script
    var name = 'script-local variable'
    def Func(name: number)
        echom name
    enddef
    defcompile
EOF
)

No error is raised.
Expected behavior
E1054 is raised in the second command:
E1054: Variable already declared in the script: name

Just like it is raised in the first one.
Environment

Vim version: 8.2 Included patches: 1-2529
OS: Ubuntu 16.04.7 LTS
Terminal: xterm(366)

If this is working as intended, then maybe we should document what are the rules regarding what is shadowing exactly, and/or which kind of shadowing is allowed and which is not.

E1054 is raised in the second command:

Because it seems that allowing a function argument to use the same name as a script-local variable can lead to mistakes.

Because it seems that allowing a function argument to use the same name as a script-local variable can lead to mistakes.

That was less of an issue in Vim script legacy with the explicit scopes s: and a:.  But in Vim9, a: is invalid, and s: can be omitted most of the time (if not all the time).


Consolidate getSubscriptions

Remove getAllSubscriptions and consolidate logic into a single function.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 11 (9.09%)  changed or added relevant lines in 7 files are covered.
4 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.005%) to 41.356%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/azurecore/src/azureDataGridProvider.ts
0
1
0.0%

extensions/azurecore/src/azureResource/commands.ts
0
1
0.0%

extensions/azurecore/src/azureResource/tree/flatAccountTreeNode.ts
0
1
0.0%

extensions/azurecore/src/azureResource/tree/flatTreeProvider.ts
0
1
0.0%

extensions/azurecore/src/azureResource/utils.ts
0
1
0.0%

extensions/azurecore/src/azureResource/services/subscriptionService.ts
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

extensions/azurecore/src/azureDataGridProvider.ts
1
0%

extensions/azurecore/src/azureResource/tree/flatTreeProvider.ts
1
0%

extensions/azurecore/src/azureResource/services/subscriptionService.ts
2
0%

Totals

Change from base Build<PHONE_NUMBER>:
0.005%

Covered Lines:
26433

Relevant Lines:
57945

💛  - Coveralls


[5.2] Update v5.2.5 -> 5.2.6 Uncaught ReflectionException: Class log does not exist

After the update from 5.2.5 to 5.2.6 I get the following error:
PHP Fatal error:  Uncaught ReflectionException: Class log does not exist in /home/vagrant/code/xxx/vendor/laravel/framework/src/Illuminate/Container/Container.php:736

Any suggestion? Thanks.

You have a syntax error in your .env file.

Laravel was previously silently ignoring all entries after the syntax error. Now, we throw an exception.

Thanks! For others having this issue, I placed some quotes around several env variables.

You'll only need to do this when they have spaces, as documented.

https://github.com/vlucas/phpdotenv

This affected me too.. lost a lot of time until i found this issue. Maybe should be more well documented?

The error message is a different bug in laravel's exception handler.

i can't fix this issue, really bugging me

This affected me too.
But my issue is not something wrong with .env file. It is because i missed a ; in a config file.

Which config file? Mine was working okay until after composer update and this error comes out

in my case only is the :: in the line:
'Datatables' => Yajra\Datatables\Facades\Datatables::class,


chore: format codebase with Prettier

and introduce a CI check that the codebase is properly formatted
Closes #226

[x] I have updated CHANGELOG.md
[x] I have added unit tests to demonstrate the contribution is correctly implemented
[x] I have run all the tests locally and no test failure was reported
[x] I did not introduce unrelated formatting/whitespace changes in the code base
[x] I did not do unrelated and/or undiscussed refactorings

Everything is the default, except I kept the 4 spaces indentation level for TS and JS files (see .editorconfig) and excluded the auto-generated and vendored files that are not in .gitignore, which Prettier ignores as well (see .prettierignore).

@byakuren-hijiri thanks! [if only there was a way to automatically check all the other bullet points, lol]


Need tests

Need tests

Originally posted by @Dominux in https://github.com/Dominux/spherical-go/issues/13#issuecomment-1198044182

Lib


Mention the feed url:
https://threedots.tech/index.xml

closed by https://github.com/murtuzaalisurti/rssed/commit/3c62d126f89df0458d771b6257233bbbe1296d2e


Add support for non-standard cabal.project file locations to cabal cradles

Add parser tests for non-standard cabal.project file locations
Add bios tests for projects with non-standard cabal.project file locations
Follows the same structure as stackYaml feature for stack cradles.
Fixes #263

@fendor what's the status on this PR? Do you think you can spare some time to solve the conflicts / rebase it?

@fendor I personally don't, but a post on the Discourse could certainly gauge the interest for this feature :)


Clip correctly in linear_scale_range when inputMax < inputMin

Clip correctly in linear_scale_range when inputMax < inputMin
Found by the new test suite and want to make you aware. I won't have time to add tests, this is more a bug report rather than a full PR.

@m-mohr where is this special behaviour defined? https://processes.openeo.org/#clip


super broken within class static methods

The following doesn't typecheck.
class A {
  static a(): number {
    return 5;
  }
}

class B extends A {
  static a() {
    return super.a() + 10; // NG by Flow
    // Fails with "Method cannot be called on ...empty super object"
  }
}

The super.a() should resolve to A.a() (see http://www.2ality.com/2015/02/es6-classes-final.html#the_prototype_of_a_subclass_is_the_superclass or trace DefineMethod(constructor, proto, functionPrototype=constructorParent) from 14.5.12, ES6 release candidate 3).

This has been fixed, closing.


MGMT-16709: Add ignition headers to Agent CR

https://issues.redhat.com/browse/MGMT-16709

/hold depends on https://github.com/openshift/assisted-service/pull/5934

/unhold


*[Don not merge, this is for demo deployment ONLY]

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

re openning the PR as we need it for demo deployment

Not needed


Reverse date as partition key

Hi Karell,
may you remember our discussion about how to sort/partition the items in table storage.
may this could be an option for you to integrate my solution in your version of the appender.
This based on my documentation pull request #21 and the relvant changes are in the last commit.
Regards
Erik

Had to remove nameof since I'm not of VS 2015


Add search funtionality in homepage

Version info

Milligram:
Other (e.g. normalize.css, node.js, npm, bower, browser, operating system) (if applicable):
Test case

Steps to reproduce

Expected behavior
Add search funtionality to documentation homepage, so that users can search what they want to get it quickly

Actual behavior

hello sir can you assign this issue to me

if no issue, both of us can work for it.


[Dev] change badge link

Now badge link refers to builds page

Codecov Report

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

@@           Coverage Diff            @@
##             master     #51   +/-   ##
========================================
  Coverage      53.5%   53.5%
  Complexity      510     510
========================================
  Files            98      98
  Lines          2471    2471
  Branches        137     137
========================================
  Hits           1322    1322
  Misses         1067    1067
  Partials         82      82

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 9705beb...348ac76. Read the comment docs.


t.expose* requires 2 arguments.

Hey everyone,
    "@pothos/plugin-prisma": "^3.53.0",     "@prisma/client": "^5.2.0",
I'm running into an issue when defining my schema where everything runs perfectly, however when i go to build it, I get these typescript errors:
From what i can tell in the documentation, there is no second argument.

`src/schema/PatientProfile/queries.ts:6:11 - error TS2554: Expected 2 arguments, but got 1.

6     id: t.exposeID('id'),
            ~~~~~~~~~~~~~~

src/schema/PatientProfile/queries.ts:7:18 - error TS2554: Expected 2 arguments, but got 1.

7     firstName: t.exposeString('firstName'),
                   ~~~~~~~~~~~~~~~~~~~~~~~~~

src/schema/PatientProfile/queries.ts:8:17 - error TS2554: Expected 2 arguments, but got 1.

8     lastName: t.exposeString('lastName'),
                  ~~~~~~~~~~~~~~~~~~~~~~~~

src/schema/PatientProfile/queries.ts:9:15 - error TS2554: Expected 2 arguments, but got 1.

9     gender: t.exposeString('gender'),
                ~~~~~~~~~~~~~~~~~~~~~~

src/schema/PatientProfile/queries.ts:11:20 - error TS2554: Expected 2 arguments, but got 1.

11     phoneNumber: t.exposeString('phoneNumber'),`

Totally right, it was a version mismatch. Updated all my pothos versions and works perfectly now.


Use super.key instead of manually passing the Key parameter to the parent class

Use super.key instead of manually passing the Key parameter using super(key: key) in the constructors.
Since if you create a widget the new default will use super.key instead of Key? key : super(key: key) this small change is to maintain the consistency, it has no semantic change
also there are some other places that might need to be updated:

this file for example generate l10n project and it has all the dart code as String, it might have tests that validate the output somewhere that I might miss, also there are some other places like the _Segment class where it require ValueKey instead if Key so I didn't update them (even though it's possible)
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] I followed the breaking change policy and added Data Driven Fixes where supported.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Thanks for the clean-up. Looks like the analyzer is not happy, though: https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8749165137680142961/+/u/run_test.dart_for_analyze_shard_and_subshard_None/stdout
Could you look into that and fix whatever it is complaining about?

Thanks a bunch for the contribution!
Sorry to make a bunch of suggestions at once, but hopefully we can get all those tests to start passing soon 😄

Thank you, too, for all your amazing work in this framework, I wasn't planning on contributing to Flutter, but I found one of the widget in flutter source code that still usr the old syntax so I searched using the IDE and found all of this. I will do what you suggested as soon as I can

Thanks for the clean-up. Looks like the analyzer is not happy, though: https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8749165137680142961/+/u/run_test.dart_for_analyze_shard_and_subshard_None/stdout
Could you look into that and fix whatever it is complaining about?

I will as soon this process finish:

For some context: There is a use_super_parameters lint that is supposed to encourage the use of super. everywhere. Unfortunately, it has a bug and doesn't trigger when the type of the parameter changes (as is the case with all the instances in this PR), see https://github.com/dart-lang/linter/issues/4614.

For some context: There is a use_super_parameters lint that is supposed to encourage the use of super. everywhere. Unfortunately, it has a bug and doesn't trigger when the type of the parameter changes (as is the case with all the instances in this PR), see https://github.com/dart-lang/linter/issues/4614.

Thank you for the info. I didn't know about it


Allow aliased file paths when importing through scss files

Add configuration for custom module paths similar to ESM. A feature when using JavaScript modules is that you can add something like this to jsconfig/tsconfig:
{
    "compilerOptions": {
        "baseUrl": ".",
        "paths": {
            "@constants/*": [
                "./src/constants/*"
            ],
            "@utils/*": [
                "./src/utils/*"
            ]
        }
    }
}

This will allow the compiler to use these paths to resolve imports in your different modules. Could something similar be done with sass imports?
I've run into a problem where I have many scoped scss files in my project, at different levels in the project structure. And something like this would be very useful.
Describe the solution you'd like
Instead of importing an scss file like this: @import '../../../Styles/_global.scss'; ,
you could do something like: @import '~globals/_global.scss' or just @import '_global.scss'
Additional context
It appears webpack has a way to do this using something like this:
// Webpack config
{
  test: /\.scss$/,
  loader: 'sass-loader',
  options: {
    sassOptions: {
      includePaths: [path.resolve(__dirname, '<path to styles>')],
    },
  },
},

which you would then use like this:
@import 'filename.scss'; // Imported from <path to styles> folder.

.style {
  //...
}

Hi @axekan , thanks for reporting the issue. As we use the dart-sass cli underneath to compile the scss files, we can make use of the --load-path option to do basically the same thing as what you described in your webpack example.
I will try to add a specific setting for this in the next few days. In the meantime, you can actually already do this yourself by making use of the "Arguments" setting (alongside your other SassCompiler settings) like so:
{
    "Arguments": "--load-path Styles"
}


added Fairseq supported language list to english readme

@ROBERT-MCDOWELL 👍 ?

I was looking at putting the help command modification but only after your push lol 🤷

it's ok you can push as long as it's only the readme..
I'm almost good, testing with real ebooks with various language and length, fix the metadatas parsinng,  tough tough work.... :-\

Your messing with the metadata parsing code?😅😅😅
XD
My code I wrote for that was barley hanging on by a thread XD😅😅😅
🫡🫡🫡

metadata was nothing to fix, the most hard is to filter all text that are in a book like technical editor stuff etc....

Ohhhhhh 👀✨
Fancy 👀✨

Just realized that the xtts version we are using using supports hindi also!
✨
Languages
XTTS-v2 supports 17 languages: English (en), Spanish (es), French (fr), German (de), Italian (it), Portuguese (pt), Polish (pl), Turkish (tr), Russian (ru), Dutch (nl), Czech (cs), Arabic (ar), Chinese (zh-cn), Japanese (ja), Hungarian (hu), Korean (ko) Hindi (hi).
Source


Use ParquetReadSupportWrapper instead of OapReadSupportImpl #591

What changes were proposed in this pull request?
In current version, we use ParquetReadSupportHelper hold a ParquetReadSupport instance and proxy method used by OapReadSupportImpl, as #591 said, it a little redundancy, so in this pr use ParquetReadSupportWrapper class in parquet package hold a ParquetReadSupport instance and proxy method we need, then del ParquetReadSupportHelper and OapReadSupportImpl.
How was this patch tested?
existing test

@linhong-intel As we discussed on WeChat, I refactor this part of the code.

@linhong-intel fix confilct~


Keyboard does not pop up when entering wallet PIn

Description of problem
OS keyboard does not appear when at enter PIN screen. App worked well before and this is a recent issue.
From customer support email:
"I am unable to enter my wallet PIN because the number pad does not pop up to allow me to enter it.  Please let me know what I can do to fix the problem.  I have an Iphone 12. "
"The app did work for me before, and it was only in the past few days that the keyboard does not pop up for the PIN. ... even uninstalling and re-installing the app and adding the credentials again did not help. It was only a temporary fix and the keyboard popped up directly after the re-installation, but now it does not work again.  When I try to get back on the BC Wallet app again, the keyboard is missing again"
Expected behavior
keyboard should appear automatically or appear when tapping the PIN text input
Steps to reproduce
See above description
Screenshots and/or log output
Environment

[ ] Occurs on Android
[X] Occurs on iOS

Build #: TBD
Android Device Model:
iOS Device Model: iPhone 12
Workaround
only a temporary fix: uninstalling/reinstalling the app. The keyboard does not appear a second time.
Severity

[ ] High
[X] Medium
[ ] Low

@knguyenBC Are they using a TestFlight build or the most recent release?

Most recent release.


Add more examples and containerize them for E2E tests

Include Langchain RAG

Added two new examples with dockers:

image-echo
langchain-rag


Simple fuzzing project doesn't work (pypi__enum34 issue)

Expected Behavior
I've copy-pasted the example from https://github.com/bazelbuild/rules_fuzzing#readme into a trivial project.  I should be able to bazel run --config=asan-libfuzzer //:fuzz_test_run but I can't get it to work.
Actual Behavior
bazel run --config=asan-libfuzzer //:fuzz_test_run
Loading:
Loading: 0 packages loaded
Analyzing: target //:fuzz_test_run (0 packages loaded, 0 targets configured)
ERROR: /home/matt/.cache/bazel/_bazel_matt/d58dd01668b145fbe0a002f4953a11e2/external/fuzzing_py_deps/pypi__absl_py/BUILD:5:11: no such package '@fuzzing_py_deps//pypi__enum34': BUILD file not found in directory 'pypi__enum34' of external repository @fuzzing_py_deps. Add a BUILD file to a directory to mark it as a package. and referenced by '@fuzzing_py_deps//pypi__absl_py:pypi__absl_py'
ERROR: Analysis of target '//:fuzz_test_run' failed; build aborted:
INFO: Elapsed time: 0.062s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded, 3 targets configured)
ERROR: Build failed. Not running target
FAILED: Build did NOT complete successfully (0 packages loaded, 3 targets configured)

Steps to Reproduce the Problem

git clone https://github.com/matta/bazel-fuzz-example
cd bazel-fuzz-example
bazel run --config=asan-libfuzzer //:fuzz_test_run

Specifications
bazel version
Bazelisk version: v1.10.1
Build label: 5.2.0
Build target: bazel-out/k8-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Tue Jun 7 16:02:26 2022<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>

cat /etc/os-release
PRETTY_NAME="Debian GNU/Linux bookworm/sid"
NAME="Debian GNU/Linux"
ID=debian
HOME_URL="https://www.debian.org/"
SUPPORT_URL="https://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"

clang --version
Debian clang version 14.0.6-2
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin

Thanks for reporting this issue! Just wanted to first rule out a Bazel compatibility issue.
If you pin the Bazel version to 4.2.0 (use this string in a .bazelversion file), are you still getting this error? (Make sure to bazel clean --expunge first.)

Pinning to Bazel 4.2.2 doesn't change the result:
# echo 4.2.2 > .bazelversion

# bazel run --config=asan-libfuzzer //:fuzz_test_run
2022/08/15 09:24:04 Downloading https://releases.bazel.build/4.2.2/release/bazel-4.2.2-linux-x86_64...
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
Loading:  (1 packages loaded)
Loading: 1 packages loaded
Analyzing: target //:fuzz_test_run (2 packages loaded, 0 targets configured)
Analyzing: target //:fuzz_test_run (5 packages loaded, 5 targets configured)
Analyzing: target //:fuzz_test_run (19 packages loaded, 68 targets configured)
ERROR: /home/matt/.cache/bazel/_bazel_matt/d58dd01668b145fbe0a002f4953a11e2/external/fuzzing_py_deps/pypi__absl_py/BUILD:5:11: no such package '@fuzzing_py_deps//pypi__enum34': BUILD file not found in directory 'pypi__enum34' of external repository @fuzzing_py_deps. Add a BUILD file to a directory to mark it as a package. and referenced by '@fuzzing_py_deps//pypi__absl_py:pypi__absl_py'
ERROR: Analysis of target '//:fuzz_test_run' failed; build aborted: Analysis failed
INFO: Elapsed time: 7.769s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (24 packages loaded, 74 targets configured)
ERROR: Build failed. Not running target
FAILED: Build did NOT complete successfully (24 packages loaded, 74 targets configured)

# bazel version
Bazelisk version: v1.10.1
Build label: 4.2.2
Build target: bazel-out/k8-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Thu Dec 2 18:15:58 2021<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>

...same result after a bazel clean --expunge.

Thanks for confirming! Will take a closer look and follow back here.

So, I installed Debian 11 (stable) in a VM, installed git, cloned my example repo, installed homebrew, then bazelisk from there, then clang from Debian's repo, then installed Debian's build-essential and python3-distutils packages, and it works!
I suspect that the problem I am having on my original Debian Testing workstation is some interaction with the way Python is installed there.
Any idea how I might set up a Bazel project to effectively use a completely hermetic Python installation?

rules_python uses a hermetic Python interpreter in it's latest versions: https://github.com/bazelbuild/rules_python#toolchain-registration
I don't know whether rules_fuzzing works with these versions yet though.

For fun I just tried this in a new Fedora 36 install and get the same error that I got with Debian Testing.

For fun I just tried this in a VM running a new Ubunu 22.04.1 LTS install.  I get the same error I got with Debian Testing.

I've just created #208 to bump up the Python dependencies to the latest version. Can you depend on this revision in particular and let me know if you're still running into the issue?

That fixes it, thanks @stefanbucur!

Awesome! I'll cut a new release then once this gets in.

This is causing an issue for me and at least a few others trying to build TCMalloc, and it looks like TCMalloc is actually using the 0.1.1 release of rules_fuzzing.
Is the solution to wait until rules_fuzzing cuts a new release, and then update TCMalloc to reference it?
Cross reference: https://github.com/google/tcmalloc/issues/127

Edit: @matta can you explain what you did to bypass this error? I tried changing the rules_fuzzing dependency in TCMalloc so that it pointed directly at the master branch, but I'm getting the same error.

Hi @emesterhazy,
First a disclaimer: I am only a novice when it comes to this part of Bazel (third party package downloads, etc.).
Simplest fix would be to get tcmalloc to depend on a newer rules_fuzzing.  Both https://bazel.build/docs/external#shadowing-dependencies and https://bazel.build/docs/external#overriding-repositories might let you use a local git clone of the tcmalloc project for debug/test purposes.
I am using the fuzzing rules directly in a small hobby project of mine, not indirectly through the tcmalloc project. Fetching the latest version in my WORKSPACE with git_repository worked for me.  I have this:
# For now, depend on a specific git revision of rules_fuzzing (that is
# still a PR in review!) to get newer Python treatment.  See:
# https://github.com/bazelbuild/rules_fuzzing/pull/208
# https://github.com/bazelbuild/rules_fuzzing/issues/207
git_repository(
    name = "rules_fuzzing",
    remote = "https://github.com/bazelbuild/rules_fuzzing.git",
    commit = "77bb2af4ee076e0706bb14fb5d62b108043624e1",
    shallow_since = "1660592161 +0000",
)

Also it looks like https://github.com/bazelbuild/rules_fuzzing/commit/23f9676510995ebf0de2ef3aa7da41045308c171#diff-ab4ccd094f3a70b4e98b0d81099360aac970e620fc0e5c258792bc2572714d64 bumped the rules_python dependency in a call to a maybe rule.  It "may be" that this call to maybe isn't taking effect in your project?  See this search https://github.com/bazelbuild/rules_fuzzing/search?q=maybe -- hmm... we have this: https://bazel.build/rules/lib/repo/utils#maybe.
I wonder you can load the newer rules_python before the tcmalloc project in in your project's WORKSPACE like this:
    # Work around https://github.com/bazelbuild/rules_fuzzing/issues/207
    # and https://github.com/google/tcmalloc/issues/127
    load("@bazel_tools//tools/build_defs/repo:utils.bzl", "maybe")
    maybe(
        http_archive,
        name = "rules_python",
        sha256 = "c03246c11efd49266e8e41e12931090b613e12a59e6f55ba2efd29a7cb8b4258",
        strip_prefix = "rules_python-0.11.0",
        url = "https://github.com/bazelbuild/rules_python/archive/refs/tags/0.11.0.tar.gz",
    )

The idea is; the older rules_fuzzing might see the newer rules_python is already present and skip loading the older rules_python.

@stefanbucur FWIW I've confirmed that this fixes things. I'll leave the issue open in case you are using it as a reminder to push a new labeled release.

Thanks @matta. It turns out there were a couple of issues. First, it turns out there was actually an explicit rules_python load in the WORKSPACE file to work around an earlier bug in rules_python:
# This is explicitly added to workaround
# https://github.com/bazelbuild/rules_python/issues/437.
http_archive(
    name = "rules_python",
    urls = ["https://github.com/bazelbuild/rules_python/releases/download/0.1.0/rules_python-0.1.0.tar.gz"],
    sha256 = "b6d46438523a3ec0f3cead544190ee13223a52f6a6765a29eae7b7cc24cc83a0",
)

I ended up fixing my problem by replacing this with rules_python-0.11.0 as you suggested. Unfortunately simply using a newer rules_fuzzing doesn't resolve the problem because an older rules_python gets pulled in by protobuf.
By the way, did you ever figure out the root cause of this python issue? I'm running Ubuntu 22.04.1 LTS and don't remember doing anything in particular to my python installation. I guess Debian 11 is the only distro that worked for you?

I guess Debian 11 is the only distro that worked for you?

Yes, in a VM Debian 11 worked. Distros that user newer software broke.
My understanding is that the newer rules_python bundles its own python interpreter, which makes it hermetic with respect to the host's python.

I ended up fixing my problem by replacing this with rules_python-0.11.0 as you suggested. Unfortunately simply using a newer rules_fuzzing doesn't resolve the problem because an older rules_python gets pulled in by protobuf.

Ugh.
https://bazel.build/build/bzlmod aims at making this less of a problem. The problem statement in the bzlmod design doc describes this kind of problem clearly ("diamond dependency problem", with unclear resolution if there is a version conflict).
Until bzlmod matures, I guess a game of whack-a-mole with PRs bumping rules_python versions is the best way forward.


moss-moon-003-sft-plugin-int8 可以精调吗

solving the problem in this project , https://github.com/yangzhipeng1108/moss-finetune

https://github.com/yangzhipeng1108/moss-finetune-and-moss-finetune-int8


Base.object_to_hash should be moved

We should experiment with moving jsonify/object_to_hash to a another class. Whether it should be in an existing class like ViewCollection  or whether it should be in a new class specific to transforming the object to a hash and json.

@philipqnguyen
Do you think this can be closed now that this has been moved to base_helpers?
https://github.com/procore/blueprinter/blob/master/lib/blueprinter/helpers/base_helpers.rb#L46-L51

Yes.