FFI?

There are two ffi files here, both for array. Does it still make sense to keep them here? Or does it make more sense to move them to purescript-arrays? Near as I can tell, there's no direct dependency on arrays here, and arrays actually depends on this package instead of the other way around.

I can't remember the exact reason, but there was a subtle reason for the current dependency order and having the instance here. Maybe @garyb can recall?

I feel like this is something I ask every so often and forget. Assuming it's decided to keep the instances here, how would you feel about a PR to add the information to the readme?

Sounds great!

The reason they're here is that these instances can't go into purescript-arrays, because then they'd be orphans. The Array type is defined in Prim, so any Array instances must go either in Prim or in the module that defines the class, and we want to keep the amount of code bundled with the compiler as small as we possibly can (which rules out putting them in Prim).

I can't remember why we have organised the core libraries with the current dependency order though. If we were to rearrange it so that foldable-traversable depended on arrays, it might well make sense to move the monomorphic foldlArray etc into arrays. If we did that, the Foldable Array instance would still have to stay in foldable-traversable though.

I can't remember why we have organised the core libraries with the current dependency order though.

It evolved out of the requirements of the relationships between types and classes, especially since orphans were disallowed. It might not be the only valid order there is though, so if you can figure out if the dependencies all work out then doing that seems reasonable to me... although a little odd also if you're looking at the order "semantically".
The fact that array is magic is the problem really. If it wasn't in Prim we could do it properly, but without overloaded literals we can't do anything about it I guess.

I'm closing as this appears to have been resolved.


More lenient nodeId syntax for UserAgents

what do we expect nodeId to be normally?

A host name or an IP address


Issue #1552: Fix method discovery (foreign key)

This fixes the method discovery if the datashape represents a foreign key.  It further entrenches the assumption that datashape.Map always represents a foreign key, see e.g.
https://github.com/blaze/blaze/blob/master/blaze/expr/expressions.py#L176
Compare with the datashape docs ("meant to be useful outside of that context "):
https://github.com/blaze/datashape/blob/master/docs/source/types.rst#maps

Coverage decreased (-0.9%) to 88.738% when pulling e19b0d9ebbcd5a0802c3c2c7473bc7c58baeb840 on skrah:foreign_key into c744e9a9a6db2630788315e776633403269ee726 on blaze:master.

Thanks @skrah, and thanks for documenting the assumptions made here.  The map-in-datashape-represents-foreign-key assumption is pervasive throughout Blaze, which is acceptable given Blaze's data analysis and tabular data focus.


Reaching github recommended file size limit

When trying to push to my repo a new version of the latest data csv file I got:
remote: warning: GH001: Large files detected. You may want to try Git Large File Storage - https://git-lfs.github.com.
remote: warning: See http://git.io/iEPt8g for more information.
remote: warning: File covid-19.data.prod.2020-04-09T111757.csv is 50.48 MB; this is larger than GitHub's recommended maximum file size of 50.00 MB

Instructions are at https://help.github.com/en/github/managing-large-files/versioning-large-files
Probably all the historical csvs should be versioned using git lfs as well.

Maybe can consider adding the data on HDX (https://data.humdata.org/event/covid-19#)?


oauth2 filter: saving id_token and refresh_token in the cookie if they are provided

Commit Message:
oauth2 filter: saving id_token and refresh_token in the cookie if they are provided
Additional Description:
https://github.com/envoyproxy/envoy/issues/17808
Risk Level:
Testing:
Docs Changes:
Need to update doc about oath2_filter. The filter will send over cookies BearerToken, IdToken, RefreshToken to the upstream
https://www.envoyproxy.io/docs/envoy/latest/configuration/http/http_filters/oauth2_filter
Release Notes:
oauth2: handling IdToken, RefreshToken fields if they are provided by IDP provider
Platform Specific Features:

Before last merging all test passed. After the merging there is a one failed test. I took a look in changes and it is not clear for me why it became to failed. I need help. Could anyone look failed test?

This is great, we needed similar functionality. Thanks @Alexcei88 -- I appreciate the contribution and am looking forward to seeing it merged!

Guys, I am sorry, but is there a chance to speed up PR? It need for our project.


Build always uses AVX2 instructions

Problem
You quite possibly don't consider this much of an issue as non-AVX2 processors are becoming increasingly irrelevant, but I thought I should note anyway -
0.11.6 (and 0.11.5 at least), built on x86_64 linux with rust 1.54.0, LLVM 12.0.1, when tremor --test-threads=4 is run on a non-AVX2 machine will result in a SIGILL.
I'm fairly sure this is a separate issue from that you note about simd-json.

Dump of assembler code for function _ZN15linked_hash_map30LinkedHashMap$LT$K$C$V$C$S$GT$6insert17h691937e6791e866cE:
   0x0000555557554940 <+0>:     push   rbp
   0x0000555557554941 <+1>:     mov    rbp,rsp
   0x0000555557554944 <+4>:     push   r15
   0x0000555557554946 <+6>:     push   r14
   0x0000555557554948 <+8>:     push   r13
   0x000055555755494a <+10>:    push   r12
   0x000055555755494c <+12>:    push   rbx
   0x000055555755494d <+13>:    and    rsp,0xffffffffffffffe0
   0x0000555557554951 <+17>:    sub    rsp,0x140
   0x0000555557554958 <+24>:    mov    r12,rcx
   0x000055555755495b <+27>:    mov    r15,rdx
   0x000055555755495e <+30>:    mov    r13,rsi
   0x0000555557554961 <+33>:    mov    rbx,rdi
   0x0000555557554964 <+36>:    cmp    QWORD PTR [rsi+0x30],0x0
   0x0000555557554969 <+41>:    jne    0x555557554986 <_ZN15linked_hash_map30LinkedHashMap$LT$K$C$V$C$S$GT$6insert17h691937e6791e866cE+70>
   0x000055555755496b <+43>:    mov    edi,0x8
   0x0000555557554970 <+48>:    mov    esi,0xa0
   0x0000555557554975 <+53>:    call   QWORD PTR [rip+0xb55fb5]        # 0x5555580aa930
   0x000055555755497b <+59>:    mov    QWORD PTR [r13+0x30],rax
   0x000055555755497f <+63>:    mov    QWORD PTR [rax],rax
   0x0000555557554982 <+66>:    mov    QWORD PTR [rax+0x8],rax
   0x0000555557554986 <+70>:    mov    QWORD PTR [rsp+0xd8],rbx
=> 0x000055555755498e <+78>:    vpermq ymm0,YMMWORD PTR [r13+0x0],0x50
   0x0000555557554995 <+85>:    vmovdqu xmm1,XMMWORD PTR [r13+0x0]
   0x000055555755499b <+91>:    vpxor  ymm0,ymm0,YMMWORD PTR [rip+0x18a67d]        # 0x5555576df020
   0x00005555575549a3 <+99>:    vpxor  xmm2,xmm2,xmm2
   0x00005555575549a7 <+103>:   vmovdqu XMMWORD PTR [rsp+0x58],xmm2

Hi,
yes and no. We build explicitly for the most common subset of CPU features. AVX, AVX2 and SSE4.2. This is set in the .cargo/config:
[build]
rustflags = ["-C", "target-feature=+avx,+avx2,+sse4.2"]

it is possible to change this , AVX2 isn't strictly required, but it makes things faster, (SSE4.2 in contrast is strictly required as it's the minimal instruction set for simd-json). That said it wouldn't make a sense for the project itself, AVX2 has been in CPUs for 10+ years by now so hurting performance for the majority for users to allow some edge cases feels wrong.
That said @mfelsche hat the nice observation that nix builds everything locally on the machine, so setting
RUSTFLAGS="-C target-cpu=native" cargo

will optimize for the exact CPU and if no AVX2 is available still work, if AVX2 is available make use of that and if rust can throw in some more advanced instructions because it's a shiny new CPU go and use those.
It won't make sense for packages as then everyone is bound to the CPU the CI machine runs, but if nix builds the binary on the same machine it runs, cpu-target=native seems like a good way.

Fair enough, thanks for this.


Hyperdimension Neptunia PP [PCSE00400]

Hyperdimension Neptunia PP [PCSE00400]

App summary

App name: Hyperdimension Neptunia PP
App serial: PCSE00400
App version: 1.00

Vita3K summary

Version: v0.1.6
Build number: 3071
Commit hash: https://github.com/vita3k/vita3k/commit/6b2a976a
CPU backend: Dynarmic
GPU backend: OpenGL

Test environment summary

Tested by: pupilplay
OS: Windows 10
CPU: Intel i5-9400F
GPU: NVIDIA GeForce GTX 1050 Ti
RAM: 16 GB

Issues
In the "Viewer" Mode, the character only displays for a instant and disappears soon.
Screenshots

()
Log
vita3k.log
Recommended labels

Ingame +
graphics bug

already exist
https://github.com/Vita3K/compatibility/issues/454


#3857 Removed the API for querying filter candidate values from the dashboard query screen.

Description
Removed the API for querying filter candidate values from the dashboard query screen.
Related Issue : #3857

How Has This Been Tested?

Create a dashboard that uses filters but does not place widgets.
Check if the filter candidate values are inquired during the initial inquiry of the dashboard.
Check that the candidate values are normally displayed in the top filter summary information.

Need additional checks?
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[X] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

deploy to 1

deploy to 1


Migration to SDK 3: facebook login missing user data

Hello,
I am migrating my app from AngularFire1/Firebase2 to AngularFire2/Firebase3 and I have some issues with the facebook login.
I am missing most of the signed-in user info.
Here is the request I am submitting:
providerAuth = new firebase.auth.FacebookAuthProvider();
providerAuth.addScope('public_profile');
Auth.$signInWithPopup(providerAuth).then(function(authData) {
    userInfo = authData.user.providerData[0];
})

In userInfo, I get only:

displayName
email
photoURL
providerId
uid

While with AngularFire1/SDK2
Auth.authWithOAuthPopup('facebook', function(error, authData) {
    userInfo = authData.facebook.cachedUserProfile;
})

This would get me in addition:

first_name
last_name
gender
age_range
locale

Am I missing something or am I not looking at the right place ?

Pretty sure you need to add the appropriate scopes. See step 2 here for an example and link to the FB docs on scopes.

This is actually unfortunately a change in the behavior of the API, although we should be reintroducing provider data like this in an upcoming release. You do need set the appropriate scopes, as Kato suggests, but that will only get you a Facebook access token with the right scpoes which you can then use to request profile data from the Facebook Open Graph. The 3.x.x SDKs do not currently return that data to you. See the Facebook docs here for how to do this. It should be a single request to https://graph.facebook.com/me/ with you Facebook access token.

Thanks @katowulf and @jwngr  :)

Hi again,
I am assuming it is the same for google login. However I just can't seem to recover the email of the user.
Here is what I am doing:
providerAuth = new firebase.auth.GoogleAuthProvider();
providerAuth.addScope('https://www.googleapis.com/auth/userinfo.email');
providerAuth.addScope('https://www.googleapis.com/auth/userinfo.profile');
providerAuth.addScope('email');
providerAuth.addScope('profile');
Auth.$signInWithPopup(provider).then(function(authData) {
    $http.get("https://www.googleapis.com/oauth2/v1/userinfo?access_token="+authData.credential.accessToken).then(function(result) {
    })
})

Neither the object returned by signInWithPopup nor the one returned by the googleapi request contains the email of the user. Actually:

'signInWithPopup' returns a null for email
'https get' does not return email while it returns familyname, givenname, gender...

Any idea why ? Not sure if I'm adding the right scopes :/


Finish RichCard conversions to Embeds and RichEmbeds

[ ] HeroCard
[ ] ReceiptCard
[x] AnimationCard
[x] VideoCard
[x] AudioCard
[ ] ThumbnailCard
[ ] AdaptiveCard
[ ] SigninCard

There are enough similarities between what BF offers and what discord offers as cards/embeds that this may be worth exploring again.


documentation

do you have documentations how to use this thanks. Sorry if  lame question

hi @anthonyjohnhinay - Could you check this ? - https://github.com/deepanprabhu/duckduckgo-images-api/blob/master/duckduckgo_images_api/api.py

https://github.com/deepanprabhu/duckduckgo-images-api/blob/master/duckduckgo_images_api/api.py#L84 - line has a sample call - for you to use,


[Humble] Add tutorial docker CI job and remove ci-testing job

Description
This PR adds the tutorial docker CI job specific to Humble, which was removed from main in https://github.com/moveit/moveit2/pull/3081.
It also removes the ci-testing Docker build from this branch, since the osrf/ros2:testing image is now on Ubuntu 24.04 and does not have the humble ROS binaries.
Checklist

[ ] Required by CI: Code is auto formatted using clang-format
[ ] Extend the tutorials / documentation reference
[ ] Document API changes relevant to the user in the MIGRATION.md notes
[ ] Create tests, which fail without this PR reference
[ ] Include a screenshot if changing a GUI
[ ] While waiting for someone to review your request, please help review another open pull request to support the maintainers

@sjahr Do you have the powers to remove the "Required" tag from the 2 CI jobs that I've removed here?

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 51.25%. Comparing base (ccf7e5c) to head (2456deb).
Report is 6 commits behind head on humble.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           humble    #3082      +/-   ##
==========================================
+ Coverage   51.15%   51.25%   +0.10%
==========================================
  Files         382      382
  Lines       31902    31865      -37
==========================================
+ Hits        16317    16328      +11
+ Misses      15585    15537      -48

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


Where is the "open in new window" right-click menu option in the latest Ver 5.0?

Where is the "open in new window" right-click menu option in Ver 5.0?
ScottPlot Version:5.0.33
It is there by default in Ver 4.1.61.

Does not exits (yet): https://github.com/ScottPlot/ScottPlot/issues/3730

Hi @kareem469, I updated #3730 to be high priority so it'll be added soon. I'll close this issue because it's a duplicate.
Note that you can add a right-click "open in new window" option pretty easily, and I posted the code here: https://github.com/ScottPlot/ScottPlot/issues/3730#issuecomment-2097117433


Interactive dependency graph

Fixes #10.
In development mode, run moon run visualizer-browser:serve to serve the static files, and then cargo run -- project-graph and then open http://localhost:8000 to see the project graph. Similarly cargo run -- dep-graph for the tasks graph.
In production mode, the rendered template expects the static files to be served from a CDN.
Here is what the moon graph looks like:
Project Graph

Dep Graph

(I have not spent any time on styling the graph, that can come later once functionality is complete.)

I decided to go with react (and not vanilla JS) because we would want to add advanced functionalities later down the line which would be easier to write if we use a framework to handle rendering.

@IgnisDa Not sure where my comment went, but what's the separation of the 2 packages for?

I put the core functionalities in the core package and the browser related one in the other one. We may need to add refactor this once we add this to the vscode extension, but this is what I chose for now.

@IgnisDa If my understanding of how vscode will work, I think this can be 1 package.
Local development will use vite, while vscode (and rust prod) will used the published code in npm.
Let's combine this into a single visualizer package. If it doesn't work, we can always split again later on.

Created a common package visualizer.

@IgnisDa I'm going to include this in the 0.21 release. I'm a bit behind on 0.20 so I'm going to focus on that this week, so I'll merge this once that lands.
In the meantime, if you want to polish it at all, feel free. Maybe mess with the graph design/colors or something.

@IgnisDa Can you rebase this off the develop-0.21 branch, and then change the base to that branch.
I also heavily refactored the graphs here: https://github.com/moonrepo/moon/pull/484 So this will affect this PR.
Once it's good, we can merge it.

Alright will do.
What do we plan to do about publishing the bundled prod version of the visualizer? We need a separate GH action to publish it to a CDN right?

@milesj There are 2 errors in the build that I am unable to resolve since I do not have the time to look into it. Would you be able to fix those?

@IgnisDa Yeah, if you want to give me contributor to the fork, I can work on the branch.

@milesj it is possible without adding as a contributor https://tighten.com/blog/adding-commits-to-a-pull-request/

Thanks for the awesome PR. It pretty much just worked once I fixed the rebase. Great work!


API: Integrate the create class list

Check endpoints here: https://orderlies-555f0367f3ea.herokuapp.com/docs#/
Endpoint: /api/v1/{department_id}/classes

@Chaos10001 Please assign me this issue

Can i be assigned to this issue?


[Feature Request] Source Login

Hi,
I would like to request source login.
Namely, I would like to update my Aniwave view list from the app.
I understand synchronization is a lot (e.g. fetching list from Aniwave and updating it) and this is not what I need.
I want just the Aniwave account to understand I watched some videos.
Of course I do not expect cross-source sync e.g. updating Aniwave when I use another source.
I  just want both my Aniwave and Anilist account updated at the same time.

Will never implemented.

We try as much to do request anonymously, If you login to aniwave then they detect you use AnimeTV, they will ban your account
It is too much work, process and code just for single source feature
Tracking already available through anilist & MAL


fix log message format

string log message does work as expected :

This is the output for this pull request :

Hi @GaneshSPatil, let me know if this is a duplicate fix. Thanks :-)

@japzio -- Many thanks for the fix.
The changes LGTM. Merging it...


Retrofit Issue with POST method

Not working while calling POST method, however GsonConverterFactory works.. Wasted many hours behind find this issue..

I'm sorry for that. Can you post more details, please?
Thank you very much.

When I call WCF POST service using fastjson and FastJsonConverterFactory in retrofit, response from that service not as correct as I tested the same service in POSTMAN rest client. And same service using default converter "com.squareup.retrofit2:converter-gson" working. GET Service completely working for fastjson. I can't send u example.. Just test own self, by calling WCF POST service using fastjson..


[FEAT] Support for Fedora IoT

I see that you have a Fedora CoreOS image. Any chance for also supporting Fedora IoT?
Despite its name, its not just for IoT devices. Its an immutable server OS similar to Fedora CoreOS but the main difference is that with Fedora CoreOS you can only install it using Ignition files and you update it with Zincati (which SecureCore doesnt support) whereas Fedora IoT has a traditional installer and updates are installed the same way as other immutable distros like Fedora SilverBlue. Additionally, CoreOS comes with both Docker and Podman whereas IoT only has Podman which is better IMO. There are other differences in the packages included as well, such as firewalld in IoT but not CoreOS  and cups in CoreOS but not in IoT.
Overall, my experience was that Fedora IoT is the server version of SilverBlue that made more sense to me than CoreOS. Its just unfortunately named I think. A SecureIoT image would be a great addition to the family.

I always assumed IoT meant it was specifically geared towards IoT devices. Very unfortunate naming. In reality it seems more similar to a "server silverblue" than coreos...
I agree it would be a great addition, or even potentially a replacement base image depending on the compatibility.
Something to investigate for sure, although not the top priority right now (lots of stuff going on 😄)

firewalld in IoT but not CoreOS and cups in CoreOS but not in IoT.

we had to layer firewalld for this reason... 🤦


Add NPM workspaces to repo

NX does not always resolve local package versions even with TS Config paths. This PR removes the path aliases and converts the repo to NPM workspaces. This allows you to reference repo packages using a version of "*", which will default to the local version.

@ScriptedAlchemy - it's interesting reading NX documentation, this repo is more of an integrated repo, which suggests we follow this:
https://nx.dev/getting-started/tutorials/integrated-repo-tutorial#local-linking-of-packages
I'll close this PR since it seems we don't want to use this approach.


[AAP-8337] Make inventory optional

https://issues.redhat.com/browse/AAP-8337
Since ansible-rulebook can be used with Automation Controller we don't need to make inventory mandatory. This is needed for the eda-server to launch the ansible-rulebook and execute a job on the Automation Controller.
Check if the rulebook contains any actions with run_module or run_playbook and raise an exception
ansible_rulebook.exception.InventoryNeededException: Rule r1 has an action run_playbook which needs inventory to be defined

I think this is not enough, according to the ticket and discussions related, we should take care if the rulebook contains run_playbook or run_module actions

@Alex-Izquierdo @ttuffin Please review


Question: share depthbuffer between views

I have 2 views: one main view and one debugview. Into the debugview i draw 3D debugstuff like lines and I would like to have those lines depth-clipped by the mainview's geometry.
If I wasnt using filament, I'd probably solve this by simply using the same depthbuffer for both views.
Is there a way to achive something similar with filament (ideally without rendering models twice) ?
What I've tried so far is to create/set manually created rendertargets that share a depthtarget and attach them to the views. But this gives me only a black screen (I guess nobody writes the swapchain anymore?)
        shared_depth.reset(
            filament::Texture::Builder()
                .width(window_size.x)
                .height(window_size.y)
                .format(filament::Texture::InternalFormat::DEPTH32F)
                .usage(filament::Texture::Usage::DEPTH_ATTACHMENT)
                .build(*engine));

        {
            color.reset(
                filament::Texture::Builder()
                    .width(window_size.x)
                    .height(window_size.y)
                    .format(filament::Texture::InternalFormat::RGB16F)
                    .usage(filament::Texture::Usage::COLOR_ATTACHMENT)
                    .build(*engine));

            rendertarget.reset(
                filament::RenderTarget::Builder()
                    .texture(filament::RenderTarget::AttachmentPoint::COLOR, color.get())
                    .texture(filament::RenderTarget::AttachmentPoint::DEPTH, shared_depth.get())
                    .build(*engine));

            view->setCamera(camera.entity.camera);
            view->setScene(scene.get());
            view->setRenderTarget(rendertarget.get());
            view->setViewport(window_viewport);
        }

        {
            dbg_color.reset(
                filament::Texture::Builder()
                    .width(window_size.x)
                    .height(window_size.y)
                    .format(filament::Texture::InternalFormat::RGB16F)
                    .usage(filament::Texture::Usage::COLOR_ATTACHMENT)
                    .build(*engine));

            dbg_rendertarget.reset(
                filament::RenderTarget::Builder()
                    .texture(filament::RenderTarget::AttachmentPoint::COLOR, dbg_color.get())
                    .texture(filament::RenderTarget::AttachmentPoint::DEPTH, shared_depth.get())
                    .build(*engine));

            dbg_view->setCamera(camera.entity.camera);
            dbg_view->setScene(dbg_scene.get());
            dbg_view->setViewport(window_viewport);
            dbg_view->setRenderTarget(dbg_rendertarget.get());
            dbg_view->setPostProcessingEnabled(false);
            dbg_view->setBlendMode(filament::View::BlendMode::TRANSLUCENT);
            dbg_view->setShadowingEnabled(false);
        }

Duplicate of issue #3145


fix: the inputText actions seems to resolve too fast

Description
On Android if you have a user flow with an inputText that comes right after clicking on a button, it can't properly resolve on time and therefore the next action after it will be executed.
Steps To Reproduce

Have an flow for pressing on a button, that button then navigates to a screen with input field
Add an action that taps on the input field to focus it and then do an inputText with some text
Add another action to press a button that goes back to the previous screen
If you execute this flow on an Android device it will press the button, wont wait for the text to be filled, and press the next button

Expected Behavior
I expected that it would wait until all the text has been filled in before execution the next action.

This will be fixed in #70, currently doing an experimental build of the driver and it has this issue solved.


Wrong "seconds_remaining" amount on backup start | Unnecessary folders listed in verbose logs |

Output of restic version
restic 0.15.2 compiled with go1.19.8 on linux/amd64
How did you run restic exactly?
Inside official docker container
docker run --rm -ti --entrypoint=sh -e RESTIC_PASSWORD=qwe -e GOOGLE_APPLICATION_CREDENTIALS=/app/service-account/backup-service-account-key.json --mount type=bind,src=[some/host/folder]/scripts/local/service-account-key.json,dst=/app/service-account/backup-service-account-key.json --mount type=bind,src=[some/host/folder]/tmp/test_file,dst=/app/backup/name/data/restic/test_file restic/restic:0.15.2

And then this command with and without --verbose flag
restic backup --json --verbose --cleanup-cache --cache-dir /app/backup/name/restic-cache --repo gs:[google/cloud/bucket/path] /app/backup/name/data/restic

What backend/server/service did you use to store the repository?
Google cloud bucket
Expected behavior
I will describe in next section.

Actual behavior
Without --verbose flag (full log of secondary run with cache)
/ # restic backup --json --cleanup-cache --cache-dir /app/backup/name/restic-cache --repo gs:[google/cloud/bucket/path] /app/backup/name/data/restic
{"message_type":"status","seconds_remaining":9223372036854775808,"percent_done":0,"total_files":1,"total_bytes":2024800256}
{"message_type":"status","percent_done":1,"total_files":1,"files_done":1,"total_bytes":2024800256,"bytes_done":2024800256}
... [several identical lines]
{"message_type":"status","percent_done":1,"total_files":1,"files_done":1,"total_bytes":2024800256,"bytes_done":2024800256}
{"message_type":"summary","files_new":0,"files_changed":0,"files_unmodified":1,"dirs_new":0,"dirs_changed":0,"dirs_unmodified":5,"data_blobs":0,"tree_blobs":0,"data_added":0,"total_files_processed":1,"total_bytes_processed":2024800256,"total_duration":0.8174002,"snapshot_id":"9323c5518c8bbb8ec0f856e50ec849791a55caabf0dff6ee95544c81ab160bdd"}

With --verbose flag (full log of secondary run with cache)
/ # restic backup --json --verbose --cleanup-cache --cache-dir /app/backup/name/restic-cache --repo gs:[google/cloud/bucket/path] /app/backup/name/data/restic
{"message_type":"status","action":"scan_finished","item":"","duration":0.6854036,"data_size":2024800256,"data_size_in_repo":0,"metadata_size":0,"metadata_size_in_repo":0,"total_files":1}
{"message_type":"status","seconds_remaining":9223372036854775808,"percent_done":0,"total_files":1,"total_bytes":2024800256}
... [several identical lines]
{"message_type":"status","seconds_remaining":9223372036854775808,"percent_done":0,"total_files":1,"total_bytes":2024800256}
{"message_type":"verbose_status","action":"unchanged","item":"/app/backup/name/data/restic/test_file","duration":0,"data_size":0,"data_size_in_repo":0,"metadata_size":0,"metadata_size_in_repo":0,"total_files":0}
{"message_type":"status","percent_done":1,"total_files":1,"files_done":1,"total_bytes":2024800256,"bytes_done":2024800256}
{"message_type":"verbose_status","action":"unchanged","item":"/app/backup/name/data/restic/","duration":0,"data_size":0,"data_size_in_repo":0,"metadata_size":0,"metadata_size_in_repo":0,"total_files":0}
{"message_type":"verbose_status","action":"unchanged","item":"/app/backup/name/data/","duration":0,"data_size":0,"data_size_in_repo":0,"metadata_size":0,"metadata_size_in_repo":0,"total_files":0}
{"message_type":"verbose_status","action":"unchanged","item":"/app/backup/name/","duration":0,"data_size":0,"data_size_in_repo":0,"metadata_size":0,"metadata_size_in_repo":0,"total_files":0}
{"message_type":"verbose_status","action":"unchanged","item":"/app/backup/","duration":0,"data_size":0,"data_size_in_repo":0,"metadata_size":0,"metadata_size_in_repo":0,"total_files":0}
{"message_type":"verbose_status","action":"unchanged","item":"/app/","duration":0,"data_size":0,"data_size_in_repo":0,"metadata_size":0,"metadata_size_in_repo":0,"total_files":0}
{"message_type":"status","percent_done":1,"total_files":1,"files_done":1,"total_bytes":2024800256,"bytes_done":2024800256}
... [several identical lines]
{"message_type":"status","percent_done":1,"total_files":1,"files_done":1,"total_bytes":2024800256,"bytes_done":2024800256}
{"message_type":"summary","files_new":0,"files_changed":0,"files_unmodified":1,"dirs_new":0,"dirs_changed":0,"dirs_unmodified":5,"data_blobs":0,"tree_blobs":0,"data_added":0,"total_files_processed":1,"total_bytes_processed":2024800256,"total_duration":0.9900696,"snapshot_id":"dcde12a0ae93b23c286f517fd6f34a0ee2fc1638011455f4583c1f97309a9272"}

With --verbose flag (start of log of secondary run without cache)
/ # restic backup --json --verbose --cleanup-cache --cache-dir /app/backup/name/restic-cache --repo gs:[google/cloud/bucket/path] /app/backup/name/data/restic
{"message_type":"status","action":"scan_finished","item":"","duration":1.276226,"data_size":2024800256,"data_size_in_repo":0,"metadata_size":0,"metadata_size_in_repo":0,"total_files":1}
{"message_type":"status","seconds_elapsed":1,"seconds_remaining":9223372036854775808,"percent_done":0,"total_files":1,"total_bytes":2024800256,"current_files":["/app/backup/name/data/restic/test_file"]}
{"message_type":"status","seconds_elapsed":1,"seconds_remaining":1243,"percent_done":0.0008037908900778014,"total_files":1,"total_bytes":2024800256,"bytes_done":1627516,"current_files":["/app/backup/name/data/restic/test_file"]}
{"message_type":"status","seconds_elapsed":1,"seconds_remaining":1243,"percent_done":0.0008037908900778014,"total_files":1,"total_bytes":2024800256,"bytes_done":1627516,"current_files":["/app/backup/name/data/restic/test_file"]}
{"message_type":"status","seconds_elapsed":1,"seconds_remaining":459,"percent_done":0.002169511776276662,"total_files":1,"total_bytes":2024800256,"bytes_done":4392828,"current_files":["/app/backup/name/data/restic/test_file"]}
{"message_type":"status","seconds_elapsed":1,"seconds_remaining":317,"percent_done":0.003141393814600545,"total_files":1,"total_bytes":2024800256,"bytes_done":6360695,"current_files":["/app/backup/name/data/restic/test_file"]}

I noticed several things that can be a bug (the first one differently is)
If needed i will create separate issues for additional mentioned problems.

For some reason first message contain wrong approximate time - "seconds_remaining":9223372036854775808
Reproduced in all 3 examples
It can be some backward overflow of variable, because this number can be represented as one and 63 zeroes

In verbose (only second example) mode output contain several strings with field "item" which contains /app/backup/name/data/restic/test_file, /app/backup/name/data/restic/, /app/backup/name/data/ and so on.
But restic backup started for /app/backup/name/data/restic/ folder.
Files from listed folders not appear in backup, so i think this can be logging system problem

In verbose mode appears additional lines and most of them have "message_type":"verbose_status", except string with "action":"scan_finished", which have "message_type":"status".
I am not sure but it may be mistake in logging system.

Steps to reproduce the behavior
Run in docker container and try save backup into google cloud bucket.
At least in my tests, data not have size about 2GB, contains only one file and not changed between tests

Do you have any idea what may have caused this?
In comments to Actual behavior
Do you have an idea how to solve the issue?
In comments to Actual behavior
Did restic help you today? Did it make you happy in any way?
Yes of course, restic helps me and i think it is a grate backup instrument.
Thanks for your work!

For some reason first message contain wrong approximate time - "seconds_remaining":9223372036854775808
Reproduced in all 3 examples
It can be some backward overflow of variable, because this number can be represented as one and 63 zeroes

This is likely fixed by https://github.com/restic/restic/pull/3563 .

2. In verbose (only second example) mode output contain several strings with field "item" which contains /app/backup/name/data/restic/test_file, /app/backup/name/data/restic/, /app/backup/name/data/ and so on.
But restic backup started for /app/backup/name/data/restic/ folder.
Files from listed folders not appear in backup, so i think this can be logging system problem

The log output is actually correct. As restic also includes information about the parent folders in the backup, this means that those are also processed and logged.

3. In verbose mode appears additional lines and most of them have "message_type":"verbose_status", except string with "action":"scan_finished", which have "message_type":"status".
I am not sure but it may be mistake in logging system.

Has been fixed by https://github.com/restic/restic/pull/4176 .


WDI P1 from Ian

@IanCostello FYI the resolver fails on populations like
Node: gen_pop_country/MLT_dcid:WorldBank/AG_LND_PRCP_MM
location: dcid:country/MLT
typeOf: dcid:StatisticalPopulation
when there is no populationType, which is required for StatPops.
We may need to fiddle with the CSV schema approach for direct place obs.

Could you add a description on the changes?


AttributeError: module 'pymesh' has no attribute 'snap_rounding'

Your work is very interesting, and I was trying to run it however, I ran into the issue:
AttributeError: module 'pymesh' has no attribute 'snap_rounding'
I wasn't able to "pip install PyMesh==0.2.1" (as it was not found), so I ended up download the .whl from https://github.com/PyMesh/PyMesh/releases and pip installed the .whl file.
I did check my site-packages' pymesh, and confirm that I snap_rounding doesn't exist. I can find snap_rounding2 from this link tho: https://github.com/PyMesh/PyMesh/blob/ddbcff950954e59deb0f2284b3e529134a6e7cf0/tools/CGAL/SnapRounding2.cpp
It would be great if you could help look into this.

Thank you for reporting!
I briefly checked that PyMesh==0.2.1 doesn't currently exist in pip distribution, and 'snap_rounding' has already removed in newest pymesh.
So my code doesn't seem to work currently... :(
OK, I'll find a way to avoid this error! Please wait for a while.


Proactive message example issue

SDK Platform

Node SDK
URL of Code Sample

https://github.com/Microsoft/BotBuilder-Samples/tree/master/Node/core-proactiveMessages/startNewDialog
Problem Description

Hi,
I'm trying to use the proactive message example and I'm dealing with an issue regarding the 'resume' option allowed by the patched beginDialog method(cf. botadapter.js).
function startProactiveDialog(address) {
  bot.beginDialog(address, "*:/survey", {}, { resume: false });
}

Actually, when it is set to 'false', the bot crashes considering 'resume' option as 'undefined'. When set to 'true', it simply won't trigger the proactive message.
Otherwise, when not using botadapter.js and the 'resume' option, proactive message works but stopping the current conversation.
Thanks for your help.

Running into this exact behavior.  Did you figure anything out or get a response?

The link to that code sample is now giving a 404.

Here is an updated link to the code in question:
https://github.com/Microsoft/BotBuilder-Samples/tree/v3-sdk-samples/Node/core-proactiveMessages/startNewDialog

Thanks @benbrown. For anyone else Googling this, here is the GitHub canonical link (permalink) to the code sample, in case it disappears again: https://github.com/Microsoft/BotBuilder-Samples/tree/109b176ea3791687e31f6ab31e7ea06f380a9186/Node/core-proactiveMessages/startNewDialog

@tdurnford, can you please assist?

I apologize we didn't get to this issue sooner.
Looking at the updated source code, it seems the beginDialog method behaves differently than it did in previous versions of the v3 SDK. The method currently accepts an address and a dialogId like it did before; however, it looks like it also accepts dialogArgs which get passed along to the dialog and a done callback function which are both optional parameters. Using the function call in the issue above now would result in a TypeError since the done parameter is expected to be a function and not the behaviors you are describing.
    /**
     * Proactively starts a new dialog with the user. Any current conversation between the bot and user will be replaced with a new dialog stack.
     * @param address Address of the user to start a new conversation with. This should be saved during a previous conversation with the user. Any existing conversation or dialog will be immediately terminated.
     * @param dialogId ID of the dialog to begin.
     * @param dialogArgs (Optional) arguments to pass to dialog.
     * @param done (Optional) function to invoke once the operation is completed.
     */
    beginDialog(address: IAddress, dialogId: string, dialogArgs?: any, done?: (err: Error) => void): void;

Closing as it appears to be an old issue.


[Port] Fix table designer reloading when only descriptions are being updated

https://github.com/microsoft/azuredatastudio/issues/20366
STS port PR: https://github.com/microsoft/sqltoolsservice/pull/1645
STS main PR: https://github.com/microsoft/sqltoolsservice/pull/1644
https://github.com/microsoft/sqltoolsservice/compare/<IP_ADDRESS>...<IP_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
8 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 41.658%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/common/ports.ts
4
74.14%

extensions/notebook/src/jupyter/serverInstance.ts
4
75.5%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
27860

Relevant Lines:
62301

💛  - Coveralls


BUG/#1285 As a user, changing an opportunity from a null amount to a positive amount should create a payment

Hello.

Your branch does not meet the feature branch naming requirements.  The name must use the format feature/NNN-description-of-feature where NNN is a valid GitHub issue number from the main repository.  Please rename your branch and submit a new pull request.

Hi @bermager. Thanks for contributing to our repo! As you saw, our robot posted regarding the naming convention to follow. This is just so it can be automatically picked up and run against our test org. Once that happens, you'll see your branch being build at http://ci.salesforcefoundation.org/. Let me know if you have any questions.

Aha, now I got it -))
Thanks, but still didn't understood how should I name it :D tried 5 times -))

So I don't need to recreate pull request anymore? You got the changes now?

There were a lot of emails coming through, so I got a little lost... I think maybe you are missing the "feature/" part at the beginning. For example:
feature/1285-add-opp-amount

Ok, sorry for creating so many requests, first time is always first time -))

No problem. But I don't think we got the final pull request yet. If we did it'd be listed under https://github.com/SalesforceFoundation/Cumulus/pulls

well, I tried another one. "feature/1285-opp-add-amount" didn't worked either.


i'm gonna rename this to various-js or various or random or stuff or something.

maybe even something or something.
i dont like the -web suffix. since i just threw logger together, and that DOES not run in the web.

i'd suggest utilikit since it's kind of a kit of JS utilities, but various sounds fun too

it wouldn't be wrong to call it all utilities but it doesn't feel right to me.


refactor(telemetry-utils): Make MockLogger internal

This class was never intended for external use.
This is a breaking change, and no replacement API is given. Existing users should either create their own mock logger implementation, or copy and adapt MockLogger from fluid-framework.

It would be great if we could remember why we switched it to Alpha to begin with:  https://github.com/microsoft/FluidFramework/pull/20657/files#diff-61c6613935614633863a5a9e1389b5f61cdfa591909facc136cd3403376378f0R20
I pinged Daniel offline, we'll see if we get any insight here.

It would be great if we could remember why we switched it to Alpha to begin with: https://github.com/microsoft/FluidFramework/pull/20657/files#diff-61c6613935614633863a5a9e1389b5f61cdfa591909facc136cd3403376378f0R20
I pinged Daniel offline, we'll see if we get any insight here.

Presumably Loop is using it somewhere, and they'll need to migrate. But as you noted, in this case they can just copy the code into their repo and use it there.

Actually, this really isn't urgent. Since this is @alpha (i.e. "legacy"), we will be able to make the breaking change later. For now, it might be better to just mark this class as deprecated (for external users). I'm going to proceed with that plan instead.


⚠ Bofrost.es has degraded performance

In ac45d3b, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1560 ms

Resolved: Bofrost.es performance has improved in 00b6ede after 13 minutes.


Trying to set up deployment on separate github repo.

New configuration for doctr to push the docs into gh-pages on this repo:
https://github.com/AFQ-Browser/AFQ-Browser.github.io
This would mean that the docs would be in: https://AFQ-Browser.github.io (I think).

Unfortunately, we would have to merge this PR to see if it works as hoped for.

If all the changes are just to travis.yml, I don't see any harm in merging to test if it works.

Here goes...


Port release of existing integrations for Agent 7.61 (#19160)

What does this PR do?

port https://github.com/DataDog/integrations-core/pull/19160 to master
Motivation

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
[ ] If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

@dkirov-dd is looking into the teradata failure, it's unrelated.


Organization ID requires a RHEL server

It would be nice to find the Organization ID within the customer portal to avoid creating a RHEL server to check the "org_id" paramter.

By creating a activation key, you get what you need.


In webpack.config.dev.js use localhost instead of <IP_ADDRESS>

On windows (tested on 10), use publicPath: 'http://localhost:8000/', instead of publicPath: 'http://<IP_ADDRESS>:8000/'.  On line 26 of webpack.config.dev.js.  Shouldn't affect other platforms.

Yes, also for me (Windows 7), it can give issues to the HMR runtime


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 11:33:29 on 2023.06.09
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 3, errors out: 0
Original problem ID: 28473
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=28473

Problem has been resolved in 18m 0s at 11:51:29 on 2023.06.09
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 28473


Allow operator length to be defined in msec

Scrumwise Status:Ready for sprint or Kanban
Currently the length of the operator is controlled by the number of zero crossings from which we calculate the percentage energy captured.
Matt Wingham (BP) suggested to specify operator length in msec. Can we provide both? From zero-crossing calculate the length in msec and vice-versa?

Not a straightforward thing to do.
Put on hold for the time being.


feat: replace deadpool with poolnoodle

This rips out deadpool for spinning up Cyclone instances and replaces the pool logic with PoolNoodle.
The Problem

Deadpool only creates new instances when the pool is empty, so we have no way to warm up instances in advance
Deadpool pushes off the recycle task until an instance is requested, so dead instances would linger in the mean time
We never want to reuse an instance, so the pooling mechanism of Deadpool is completely redundant

The Solution

Remove Deadpool as the interface for getting Cyclone instances
Replace it with a similar implementation using PoolNoodle
a. The core loop of PoolNoodle is effectively unchanged, except it now looks out for Drop messages from dead instances
b. Instances requested from PoolNoodle are given back in a wrapper (LifeGuard) that implements Drop so we can handle them going out of scope. On drop, we send a message on a channel that is handled in the core loop, as mentioned above. Deref is also implemented to allow referencing the inner Instance
c. We now call the native Instance methods, so we are no longer hardcoded to only working with Firecracker
d. We now always try to fill the entire pool (so pool sizes should be kept lower). When doing so, we start the instance, effectively making our pool warm! Not that this works with both the Tokio and Firecracker runtimes

I also modified the existing tests and added a few more to test the pool mechanics.
I do think that there is an issue with the core loop. It feels weird to be both checking vecs and reading messages from a channel. I definitely think there is a more event-driven approach that can be taken here, but I'm too close to this solution right now to walk it out.

bors try

bors try

bors try-

bors try


extras parameter supported in this plugin???

extras parameter supported in this plugin???

Notice: this issue has been closed because it has been reported for branch 2.x or later. It could be already fixed in v3 or is not relevant anymore. You may reopen this issue if it has been closed in error.


#84 generating gitignore

done differently; in Go / aem init cmd
https://github.com/wttech/aemc/blob/main/pkg/project/project.go#L151


.Net Core tests take a long time to report a missing SDK.

Description
When I run a .Net Core test where the sdk being targeted is not installed on the box we do nothing for about a minute after which we show the following message in the output pane
 Microsoft.VisualStudio.TestPlatform.ObjectModel.TestPlatformException: Testhost process exited with error: It was not possible to find any compatible framework version
The specified framework 'Microsoft.NETCore.App', version '2.1.0-preview2-26406-04' was not found.
  - Check application dependencies and target a framework version installed at:
      C:\Program Files\dotnet\
  - Installing .NET Core prerequisites might help resolve this problem:
      http://go.microsoft.com/fwlink/?LinkID=798306&clcid=0x409
  - The .NET Core framework and SDK can be installed from:
      https://aka.ms/dotnet-download
  - The following versions are installed:
      2.1.0-preview1-25929-02 at [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
      2.1.1 at [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]

   at Microsoft.VisualStudio.TestPlatform.CrossPlatEngine.Client.ProxyOperationManager.ThrowExceptionOnConnectionFailure(Int32 connTimeout)
   at Microsoft.VisualStudio.TestPlatform.CrossPlatEngine.Client.ProxyOperationManager.SetupChannel(IEnumerable`1 sources)
   at Microsoft.VisualStudio.TestPlatform.CrossPlatEngine.Client.ProxyExecutionManager.StartTestRun(TestRunCriteria testRunCriteria, ITestRunEventsHandler eventHandler)
[7/6/2018 4:08:48 PM Error] Microsoft.VisualStudio.TestPlatform.ObjectModel.TestPlatformException: Testhost process exited with error: It was not possible to find any compatible framework version
The specified framework 'Microsoft.NETCore.App', version '2.1.0-preview2-26406-04' was not found.
  - Check application dependencies and target a framework version installed at:
      C:\Program Files\dotnet\
  - Installing .NET Core prerequisites might help resolve this problem:
      http://go.microsoft.com/fwlink/?LinkID=798306&clcid=0x409
  - The .NET Core framework and SDK can be installed from:
      https://aka.ms/dotnet-download
  - The following versions are installed:
      2.1.0-preview1-25929-02 at [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
      2.1.1 at [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]

   at Microsoft.VisualStudio.TestPlatform.CrossPlatEngine.Client.ProxyOperationManager.ThrowExceptionOnConnectionFailure(Int32 connTimeout)
   at Microsoft.VisualStudio.TestPlatform.CrossPlatEngine.Client.ProxyOperationManager.SetupChannel(IEnumerable`1 sources)
   at Microsoft.VisualStudio.TestPlatform.CrossPlatEngine.Client.ProxyExecutionManager.StartTestRun(TestRunCriteria testRunCriteria, ITestRunEventsHandler eventHandler)

This is precise and helpful messaging. Is there a way we could report this earlier then having to wait for the connection timeout? This isn't very important but would give a better experience.

@pvlakshm Please triage.

I also raised this feedback ticket for the same issue. And IMHO this feels like a broken scenario. I installed the latest VS 15.8 and opened an existing solution and tried to run tests. If Test explorer window has hard dependency on some sdk version then should that be included as part of installer or Test explorer should take dependency on latest sdk which is part of the installer itself... either way we shouldn't expect customers to install those specific sdk versions separately for us to be working properly...

@jainaashish : Test Explorer or the platform does not take a hard dependency on an sdk. It picks up whatever the test app specifies. The VS installer AFAIK downloads only the latest stable sdk. you might want to look at additional components to install 1.* sdks or pull the one the app requires from the downloads page.

@AbhitejJohn Is this still an issue? I thought this was fixed a long time ago.

@singhsarab : Tried this out on a simple MSTest .NET Core project and I did not see the delay. So I guess this is fixed - Don't really know what fixed this though.

Thanks for the confirmation. I think I remember we rolled our a fix for this.
Closing.


runtime: support DT_GNU_HASH in GNU/Linux VDSO

Currently we only recognize DT_HASH in runtime/vdso_linux_amd64.go.  However, my understanding is that the VDSO would work fine (other than for Go) if it were generated with only the newer DT_GNU_HASH tag.  For future proofing, we should recognize DT_GNU_HASH as well.
The format of DT_GNU_HASH is defined at https://sourceware.org/ml/binutils/2006-06/msg00418.html and probably other places as well.

Is there a reason to use the hash table at all? There are fewer than a dozen symbols in the VDSO.

The format of DT_GNU_HASH is defined at https://sourceware.org/ml/binutils/2006-06/msg00418.html and probably other places as well.

I don't think that's the format that was settled on. It certainly doesn't match up with what's in the VDSO. This seems to be an up-to-date description: https://flapenguin.me/2017/05/10/elf-lookup-dt-gnu-hash/

It speeds up linking at some point but the vDSO, let's say on amd64, has only 4 functions so this perfomance tweak wold be minimal.
At the same time it's not easier to parse than the standard DT_HASH.

Yes, sorry, by "at all" I didn't mean "why do these .so's have these pesky hash tables at all" :) I just meant why is Go bothering to implement hash table parsing when the only thing we use that code for is an object with only a handful of symbols?

We don't need the hash table to speed up lookups, we need the hash table because that is what the dynamic linker looks at.  The dynamic linker does not fall back to searching the symbol table if there is no hash table, it just punts.  I suggested that we add DT_GNU_HASH because I would not be surprised if at some point the GNU/Linux dynamic linker starts to require it.

Oh, wait, that was the wrong answer.  Sorry.  We need to recognize DT_GNU_HASH because the VDSO doesn't always have a symbol table.  We used to just look through the symbol table, but that broke.  Similarly I expect that at some point the VDSO will drop the DT_HASH table.

We need to recognize DT_GNU_HASH because the VDSO doesn't always have a symbol table.

Your point over on the CL pointed me in the right direction. If I understand correctly, it's not exactly that it "doesn't always have a symbol table" since even the hash tables have to index into something. It's that the only way to know the size of the dynamic symbol table is if there's a section for it, and there isn't always a section table.


Updated the labels for the access control set

These labels were changed a while ago. Not referencing the correct information can be confusing.

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

Hi @iamalexmang Could you respond to the feedback as we review your pull request? Thanks!

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

Hi Kristine. I applied the change, as per the latest updates of the UI. However, I believe the screenshots may benefit from another update as well. I'll be happy to cover them as well, but likely in a different PR. Would you agree?

@chasewilson The PR has been updated. Could you take a look? Thanks.

Removed chase and assigned to @juliakm

#sign-off
@iamalexmang Thanks for your contribution! Sorry for the delay on this one.

@juliakm: I'm sorry - only the author of this article, @chasewilson, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Comparing List

Hi quick question.
List1: {1, 2, 3}
List2: {1, 2}
Is it possible for it to tell me that List 1 has '3' but List 2 doesnt, instead of telling me List1.count!= List2.count

@wireless90 Sorry it does not have that ability.  You can certainly add it and I will be happy to merge in the pull request.

@wireless90 I must have been tired.  Yes it does support that:
https://github.com/GregFinzer/Compare-Net-Objects/wiki/Comparing-Lists-of-Different-Lengths


Observable of actions that completes interferes with other observables of actions

tl:dr: if you pass your store into the subscribe() method of an Observable that emits Actions, then completes, it improperly stops actions from other Observables of Actions that were similarly added to the store.
See this example:
https://plnkr.co/edit/ijTa6E7AY6z64MbjhqQ6?p=preview
The important code is in main.ts.
Clicks on "add person" emit events that are mapped to an action, then store subscribes to that stream of events, as an Observer.
Clicking "Kill store" creates an Observable that emits one action (that does nothing itself), then completes; store subscribes to this Observable. After you click it, clicks on "Add person" will no longer work.
By contrast, the "do not kill store" button does almost the same thing as the "kill store" button, except that its Observable does NOT complete.
Note that this bug only affects subscriptions if BOTH are added by passing store to the subscribe() method of an Observable of Actions. If you call store.dispatch() directly, it doesn't seem to be an issue.

Please check this against NgRx v4, and if it’s still an issue, please reopen on https://github.com/ngrx/platform. Thanks!


Update UG

I made the 'benefits' box purple :D

Codecov Report

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

@@            Coverage Diff            @@
##             master     #148   +/-   ##
=========================================
  Coverage     51.49%   51.49%
  Complexity      549      549
=========================================
  Files           113      113
  Lines          2416     2416
  Branches        293      293
=========================================
  Hits           1244     1244
  Misses         1091     1091
  Partials         81       81

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 4c4db2e...e27d218. Read the comment docs.


Don’t italicise keywords in syntax highlighter

We had a user run into an issue where they mistook | for / because it was being italicised by our syntax highlighter (see Discord support thread). That doesn’t seem great, so I suggest removing that italic styling.
Here’s the offending code sample:

before
after

It does impact elsewhere, like this below. Anyone know if there’s a way to target those things separately?

before
after

I have to agree. I was just going over a code sample with || and it really did look like (weak) //.
So, I'm +1 on the change.

Do you think we should wait for @bholmesdev’s eyes — given he led on setting the theming up — or do we just take an executive decision?

@delucis yes, this dethemeitely looks good!

This seems like a small but impactful detail! It should clear things up while not looking off elsewhere...if that makes sense 😅


Date picker bugs

Prerequisites

[x] I've searched for existing issues

Describe the bug

The “day” is pre-populated with "undefined". If I click into it, it doesn’t disappear and I can’t type anything. (However, if I tab into it and it selects the “undefined”, I can edit it). The only way to get rid of it is to click/focus in, and then click out - then it disappears

I can't reproduce this, @cherylhjli — I'll need more information about where this happens. I've tried creating a new project and new meeting and not seeing "Undefined" on either.

@greatislander while testing #1315 I saw this too. I signed in as the community organization user. I filled out the org page, and approved (as an admin) the org before publishing. I then created a new project and saw the issue mentioned here.


Feature/debug log tasks

Create new CCI tasks for enabling and downloading Debug Logs.

Closing for now because I don't have a need for this functionality coming up soon and it sounds like the improvements needed are largeish.


fix(): useRuntimeConfig should use the .public property

Nuxt v3.4 introduced a warning for using public environmental variables without the .public property. In v3.5, it will be removed entirely.
Simply adding the .public property should solve this.

Thanks, @Adamkillander96, for adding this deprecation fix. I love that you helped us to keep it up to date!


Consumer fails to read messages properly

When running the Quarkus Kafka Quickstart, when a consumer is added to the prices topic, message encoding is borked (consumer expects UTF-8, producer simply serializes integers before sending to the topic):

I study quickly how to fix this issue, and I think we should use an avro js library to do that.
I tested quickly with https://www.npmjs.com/package/avsc and if key is a long we can write:
import { types } from "avsc";

message.key = new types.LongType().decode(message.key).value;

I wonder if we must use  https://github.com/apache/avro/tree/master/lang/js#avro-js instead?
Need some investigation...

I wonder if we must use https://github.com/apache/avro/tree/master/lang/js#avro-js instead?

See https://github.com/mtth/avsc/issues/259 in other words, we must use avsc

This issue will be fixed with the issue https://github.com/jlandersen/vscode-kafka/issues/112

Fixed with #112


Pip not installing dependencies

First time opening an issue like this, please let me know if I do something wrong!
Pip seems to be not installing the dependencies for this package:
$ pip install memery
Defaulting to user installation because normal site-packages is not writeable
Collecting memery
  Using cached memery-0.0.2-py3-none-any.whl (14 kB)
Installing collected packages: memery
Successfully installed memery-0.0.2

The pip install command completes very quickly.
rob@tortuga ~/Pictures$ memery . 'music' -n 2
Traceback (most recent call last):
  File "/home/rob/.local/bin/memery", line 5, in <module>
    from memery.cli import __main__
  File "/home/rob/.local/lib/python3.9/site-packages/memery/cli.py", line 6, in <module>
    import typer
ModuleNotFoundError: No module named 'typer'

I then installed python-typer (from pacman), and tried again:
rob@tortuga ~/Pictures$ memery . 'music' -n 2
Traceback (most recent call last):
  File "/home/rob/.local/bin/memery", line 5, in <module>
    from memery.cli import __main__
  File "/home/rob/.local/lib/python3.9/site-packages/memery/cli.py", line 7, in <module>
    from .core import queryFlow
  File "/home/rob/.local/lib/python3.9/site-packages/memery/core.py", line 8, in <module>
    from .loader import get_image_files, archive_loader, db_loader, treemap_loader
  File "/home/rob/.local/lib/python3.9/site-packages/memery/loader.py", line 18, in <module>
    import torchvision
ModuleNotFoundError: No module named 'torchvision'

So I installed that, and now it doesn't have annoy. At this point I opened the issue.
Some other info:
rob@tortuga ~/Pictures$ uname -a
Linux tortuga 5.4.116-1-MANJARO #1 SMP PREEMPT Sun May 2 11:10:35 UTC 2021 x86_64 GNU/Linux
rob@tortuga ~/Pictures$ pip --version
pip 20.3.1 from /usr/lib/python3.9/site-packages/pip (python 3.9)
rob@tortuga ~/Pictures$ python --version
Python 3.9.4

Thanks, and let me know if there's any other info that would be helpful

Thank you! You did great. This was definitely a big error on my part and you finding it helped a lot :sweat_smile:
The package downloaded was just my tiny amount of code without dependencies. This was due to the nbdev build practices which use settings.ini, instead of requirements.txt. I found the right way to package things for the nbdev+pip build system, so if you start over at pip install memery it should grab all the dependencies this time.
NB I'm not sure how memery will interact with your python 3.9 environment. i've been testing with 3.7 but i don't remember why

I reinstalled and it seems like the installation worked properly this time, thanks!
I'm now having a different problem, which I'll open a separate issue for.


Remove ErrorResponse from Generator.Shared

Summary
ErrorResponse<T> is not returned by any DPG APIs.  Per https://github.com/azure/azure-sdk-for-net/issues/27852, we would like to move to using NullableResponse<T> instead.  ErrorResponse<T> should be removed.
Tasks involved

[ ] ErrorResponse<T> is removed from Generator.Shared
[ ] ProcessHeadAsBoolMessage is updated to return NullableResponse<T>
[ ] Generator is updated if changes need to be made when head-as-boolean: true.

Blocked by https://github.com/azure/azure-sdk-for-net/issues/27852

Let's start by removing ErrorResponse<T> and ignoring the tests that fail.


Limit height of choices accordion to fit within its container

Description
This PR shortens the max height at which an accordion can open.
Previously, it could expand to be the entire height of its container (from the top of the containing rect).
This meant that the dropdown would often expand past the viewbox and users would have to scroll down to see all the options in the dropdown.
This PR fixes that by making the max height 75% of the height of its container, so that it is always contained within the viewport.
Before

After

https://zapierorg.atlassian.net/browse/CORE-1770

@stevemolitor Good questions, and thanks for the review, I'm glad to have your opinion on this!

Does this affect formatic dropdowns outside the editor? If so is everyone ok with those changes?

It does, so I'm hesitant to make this change. We could very well have people out there who prefer the current behavior 🤔

If the above is a concern, can we plugin in the magic 0.75 value, or do it with a max-height css value or something?

We can -- though the manner feels a bit hacky to me-- we'll could pass it through the field.meta object. It feels strange to me to accept changes from this.props.field.meta but since this is such a nested component I'm not sure how else to pass the prop for our use case.
I've got another working solution pushed now. I'm not completely pleased with it but it does work 🤔 It's a bit complicated because this component already uses JS to manage the maxHeight by evaluating the size of the viewport:
https://github.com/zapier/formatic/blob/bcc6bb0064191c8a07378663c4fd6e5c38a86e75/src/components/helpers/choices.js#L253-L262
So here's the potential solution I've come up with in
Via a prop passed in meta:

See if the prop choicesMaxHeight exists in the field.meta. It must be a decimal between 0 and 1.
If so, ensure it's above 0 and less than 1, default to 1
Multiply the actual height by the desired height (calculated in the previous step)
If the actual height and desired height are not equal, set the maxHeight of the element

Let me know what you think of this, thanks again @stevemolitor !

Alrighty @stevemolitor the latest changes are ready for another look whenever you have time. Thanks so much for the great suggestions!

Thanks @stevemolitor !


interactive does not work with Visual Studio

This issue was originally created at: 2013-09-24 04:35:00.
This issue was reported by: chfast.
chfast said at 2013-09-24 04:35:00

Using interactive mode (--interactive) my project can be compiled with Visual
Studio compiler (cl) only once. The second time the build is called builder output
is incorrect: the source file is missing after "/c" option.

No repro was provided.
Closing


(PWK) pods status pending, while the same command is running in katacoda

steps to reproduce :
pwk

open https://labs.play-with-k8s.com
login
add new instance

kubeadm init --apiserver-advertise-address $(hostname -i)
kubectl apply -n kube-system -f "https://cloud.weave.works/k8s/net?k8s-version=$(kubectl version | base64 |tr -d '\n')"
kubectl apply -f https://k8s.io/docs/user-guide//nginx-app.yaml
kubectl get pods

NAME                        READY     STATUS    RESTARTS   AGE
my-nginx-569477d6d8-56sdx   0/1       Pending   0          1m
my-nginx-569477d6d8-lrklk   0/1       Pending   0          1m
my-nginx-569477d6d8-xrcp7   0/1       Pending   0          1m
katacoda

open https://www.katacoda.com/courses/kubernetes/playground
login

kubectl apply -f https://k8s.io/docs/user-guide//nginx-app.yaml
kubectl get pods

NAME                        READY     STATUS    RESTARTS   AGE
my-nginx-569477d6d8-wln2k   1/1       Running   0          21s
my-nginx-569477d6d8-wzv29   1/1       Running   0          21s
my-nginx-569477d6d8-xrlpg   1/1       Running   0          21s

@sugizo this is because kubernetes master node by default doesn't schedule any pods. You need to either add a second "worker" node to schedule or taint (https://kubernetes.io/docs/concepts/configuration/taint-and-toleration/) the master node so it can schedule pods.

tried to learn from the https://kubernetes.io/docs still not clear what to do to make simple tutorial work in play-with-k8s.com, perhaps give some additional info in bootstrap a cluster in play-with-k8s.com, like how to add a second "worker" node to schedule or taint the master node so it can schedule pods


[pigeon] Pigeon example invocation should mention experimental_kotlin_out flag

The example of running Pigeon available here, mentions that Swift and Kotlin generation are experimental.
A flag for Swift generation is included in the example (experimental_swift_out), but Kotlin ones are not. Moreover, I couldn't find the exact name for the experimental Kotlin generation flag without looking at Pigeon's source code. In fact, the section Swift / Kotlin Plugin Example  links an example project that still hasn't the Kotlin flag in its run_pigeon.sh.

Closing this since https://github.com/flutter/packages/pull/3260 has been merged


winget update reports a Hashicorp.Terraform update is available, but winget update Hashicorp.Terraform says none is found

Brief description of your issue
Running winget update reports a Hashicorp.Terraform update is available, but winget update Hashicorp.Terraform says none is found:

See logs:
WinGet-2024-08-28-09-37.zip
Steps to reproduce
On a system with Hashicorp.Terraform version 1.9.0 installed, run winget update Hashicorp.Terraform
Expected behavior
Hashicorp.Terraform version 1.9.5 should get installed
Actual behavior
winget update Hashicorp.Terraform reports No installed package found matching input criteria.
Environment
Windows Package Manager v1.8.1911
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19045.4780
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.23.1911.0

Winget Directories
-------------------------------------------------------------------------------------------------------------------------------
Logs                               %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir
User Settings                      %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\settings.json
Portable Links Directory (User)    %LOCALAPPDATA%\Microsoft\WinGet\Links
Portable Links Directory (Machine) C:\Program Files\WinGet\Links
Portable Package Root (User)       %LOCALAPPDATA%\Microsoft\WinGet\Packages
Portable Package Root              C:\Program Files\WinGet\Packages
Portable Package Root (x86)        C:\Program Files (x86)\WinGet\Packages
Installer Downloads                %USERPROFILE%\Downloads

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Admin Setting                             State
--------------------------------------------------
LocalManifestFiles                        Disabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Disabled
LocalArchiveMalwareScanOverride           Disabled
ProxyCommandLineOptions                   Disabled
DefaultProxy                              Disabled

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

What is the output from winget list terraform?
Can you share verbose logs from the failure?
winget upgrade Hashicorp.Terraform --source winget --verbose-logs --logs

Here's the output:
PS> winget list terraform
Name                Id                  Version Available Source
----------------------------------------------------------------
Hashicorp Terraform Hashicorp.Terraform 1.9.0   1.9.5     winget

Here are the logs:
WinGet-2024-08-28-16-5X.zip

It looks like we're seeing multiple correlations. I'm not sure if there are extra registry entries or something else causing the failure.
I think this is the same issue as:

#4751
#3994

winget search hashicorp.terraform
Name                Id                        Version              Source
--------------------------------------------------------------------------
Hashicorp Terraform Hashicorp.Terraform       1.9.5                winget
Hashicorp Terraform Hashicorp.Terraform.Alpha 1.10.0-alpha20240807 winget
Hashicorp Terraform Hashicorp.Terraform.Beta  1.7.0-beta2          winget
Hashicorp Terraform Hashicorp.Terraform.RC    1.8.0-rc1            winget

2024-08-28 16:54:58.417 [YAML] Detected UTF-8
2024-08-28 16:54:58.489 [REPO] Performing search: Query:[none] Filter:Id='Hashicorp.Terraform'[CaseInsensitive]
2024-08-28 16:54:58.498 [REPO] Finding installed package from available package using system reference search: Query:[none] Include:NormalizedNameAndPublisher='hashicorpterraform'+'hashicorp'[Exact] Include:NormalizedNameAndPublisher='terraform'+'hashicorp'[Exact]
2024-08-28 16:54:58.498 [REPO] Performing search: Query:[none] Include:NormalizedNameAndPublisher='hashicorpterraform'+'hashicorp'[Exact] Include:NormalizedNameAndPublisher='terraform'+'hashicorp'[Exact]
2024-08-28 16:54:58.499 [REPO] Performing search: Query:[none] Include:ProductCode='hashicorp.terraform_microsoft.winget.source_8wekyb3d8bbwe'[Exact] Include:NormalizedNameAndPublisher='hashicorpterraform'+'hashicorp'[Exact]
2024-08-28 16:54:58.499 [REPO]  ... installed package [ARP\Machine\X64\Hashicorp.Terraform_Microsoft.Winget.Source_8wekyb3d8bbwe] had multiple correlations and is being ignored as a match for [Hashicorp.Terraform]

From the log:
2024-08-28 16:54:58.489 [REPO] Performing search: Query:[none] Filter:Id='Hashicorp.Terraform'[CaseInsensitive]
2024-08-28 16:54:58.498 [REPO] Finding installed package from available package using system reference search: Query:[none] Include:NormalizedNameAndPublisher='hashicorpterraform'+'hashicorp'[Exact] Include:NormalizedNameAndPublisher='terraform'+'hashicorp'[Exact]
2024-08-28 16:54:58.498 [REPO] Performing search: Query:[none] Include:NormalizedNameAndPublisher='hashicorpterraform'+'hashicorp'[Exact] Include:NormalizedNameAndPublisher='terraform'+'hashicorp'[Exact]
2024-08-28 16:54:58.499 [REPO] Performing search: Query:[none] Include:ProductCode='hashicorp.terraform_microsoft.winget.source_8wekyb3d8bbwe'[Exact] Include:NormalizedNameAndPublisher='hashicorpterraform'+'hashicorp'[Exact]
2024-08-28 16:54:58.499 [REPO]  ... installed package [ARP\Machine\X64\Hashicorp.Terraform_Microsoft.Winget.Source_8wekyb3d8bbwe] had multiple correlations and is being ignored as a match for [Hashicorp.Terraform]
2024-08-28 16:54:58.499 [CLI ] Search result size: 0
2024-08-28 16:54:58.499 [CLI ] No app found matching input criteria

2024-08-28 16:54:58.499 [REPO]  ... installed package [ARP\Machine\X64\Hashicorp.Terraform_Microsoft.Winget.Source_8wekyb3d8bbwe] had multiple correlations and is being ignored as a match for [Hashicorp.Terraform]
The package manager downloads a compressed copy of all manifests in the repo, then matches this data against the data in the Registry. It seems that the data in Registry might be matching to more than 1 manifest, and so the package manager ignores it.
The next step would seem to be reducing the correlations to 1 through removing data that correlates to other manifest versions.

One way to do this might be uninstalling the application, then installing again with the package manager. This might obliterate all Registry data about the application, including the unwanted corellation.
Other options might be available.

That matching issue looks like it is with portables specifically, and is probably not due to other packages with ids that are longer, but rather with other packages that are so similar we can't tell them apart.  Ultimately we should be able to go from portable directly to an ID and source but we aren't.  I suspect that we just need some "enlightening" given to the ARP reading for portable specific info we write in.  This should chain through the tracking DB back to the original package without any confusion 🤞

It looks like we're seeing multiple correlations. I'm not sure if there are extra registry entries or something else causing the failure.

Feel free to ask me to look up registry entries that winget relies upon.

Seems that the registry only contains one entry:
PS> Get-InstalledApp.ps1 -AppName '*terraform*' -MatchAll

ComputerName    : <REDACTED>
AppID           : Hashicorp.Terraform_Microsoft.Winget.Source_8wekyb3d8bbwe
AppName         : Hashicorp Terraform
Publisher       : HashiCorp
Version         : 1.9.0
UninstallString : winget uninstall --product-code Hashicorp.Terraform_Microsoft.Winget.Source_8wekyb3d8bbwe
Scope           : Machine
InstallDate     : 2024-07-01 00:00:00
Architecture    : 64-bit

Problem still present for version 1.9.6:

Exact same problem on another machine with RawTherapee.RawTherapee:


Pipeline-based script loading and retries

This PR builds on https://github.com/luin/ioredis/pull/1497 (first three comments)
The last two commits are relevant for the new script load behavior.
Changelog

🚀 Scripts are loaded into the pipeline instead in advance

It improves performance during first load
Improves cluster support with scripts

🐛 Automatically retry failed evalsha commands if the script is missing in redis, this improves the resiliency
🐛 Use the same script caching logic for standalone execution of scripts
🐛 Only send the scripts to a socket exactly once

Hey @marcbachmann 👋,
Sorry for the delay. I'm planning v5 in https://github.com/luin/ioredis/pull/1504. So do you mind change the target branch of this PR to v5 and resolve the conflicts?

@luin I've rebased this.
At least one cluster.disconnect test is missing as I've removed the script based ones:
https://github.com/luin/ioredis/pull/1499/files#diff-25757dbbc9670af28c244c85ae7b36c1224b1963223299f4df48963785141a6fL415

Awesome! Nice and clean improvements!


ControlGroup: Make it easier to customize layout (width) for items

This update makes it easier to customize layout (width) rendering for control items within ControlGroup.
ControlGroup now accepts a templateColumns prop, which accepts CSS grid-template-columns values.
Passing templateColumns into ControlGroup will render the component with a Grid vs a Flex.

For example, here is the code to render the Grid-based layout seen in the above screenshot:
<ControlGroup templateColumns="auto 1fr 1fr auto">
  <Select options={[{ label: 'Ms.', value: 'ms' }]} />
  <TextInput placeholder="First name" />
  <TextInput placeholder="Last name" />
  <Button variant="primary">Submit</Button>
</ControlGroup>

Resolves https://github.com/ItsJonQ/g2/issues/151

Merging it up 😊


Memory leak in -[NSApplicationAWT postRunnableEvent:]

zcash4mac 1.0.4-208
Steps:
Launch zcash4mac.
What happened:
Memory leak in -[NSApplicationAWT postRunnableEvent:]
https://www.dropbox.com/s/pf54fy2ge7yx5o9/zcash4mac.trace.zip?dl=0
Expected result:
No leaks.
Notes:
Unzip and open file with Apple Instruments (part of Xcode tools).

This project is dead so retracting.


Register experiment_ids for CovidMIP (#973)

I've added some changes to the writeJson.py file to insert six new DAMIP experiments. Note: we still need final confirmation for the ssp245-cov-aer and ssp245-cov-ghg experiments -- I'll revise this PR if necessary.
Other than adding the new experiment data I have

tweaked the import of urllib to try to import urllib2 first (urllib wouldn't retrieve the contents of the README.md file for me for some reason)
pulled the author and author institution out into a variable next to commit message

@matthew-mizielinski this looks good, will wait until tomorrow (Friday) and merge. We'll leave #973 open until @martinjuckes (Data Request) and @charlottepascoe (ES-DOCs experiment documentation) have managed to pull across these 6 new experiments

6d11b95 corrected capitalisation inssp245-cov-GHG experiment id.

This PR was deprecated by #976, so will close


Gross Store: Add additional test case for AddItem

The current test cases for AddItem in the concept exercise gross-store do not cover the case that an item is added more than once. That means, if a student uses bill[item] = units[unit] instead of bill[item] += units[unit] in their code, they still pass all the tests.
An additional test case should be added that tests the behavior in case an item is added multiple times.

Closing this as the PR was already merged,


Add command option sub sheets

Possible functionality to make really popular commands even more meaty
Curl cheat.sh/podman run
Would give specific sheets for a run option command, purely optional behaviour that would only exist for a subset of the most useful bash utilities
Absolutely love this repo, it is amazing work

It is a good idea. Accepted.


Issues when triggering a workflow in another repository

I'm having some issues when triggering a workflow in another repository.
Let's say that I'm developing something in Org/A repository and I want to trigger a workflow in Org/B repository from some Org/A workflow's job:
      - name: "Trigger B Workflow"
        uses: benc-uk/workflow-dispatch@v1
        with:
           repo: "Org/B"
           workflow: "Workflow B"
           token: ${{secrets.XXX}}
           inputs: '{ "someInput": "someValue" }'

The problem is that if this workflow is executed in branch "XYZ" in Org/A I get the following error:
No ref found for: refs/heads/XYZ

I'm really not sure anymore whether I wrote it wrong or whether this is a bug in benc-uk/workflow-dispatch?

I am experiencing the same issue:
Run benc-uk/workflow-dispatch@v1
  with:
    workflow: Java CI
    repo: yooksi/java-workflows
    token: ***
Workflow id is: 5761042
Error: No ref found for: refs/heads/master

I experienced this same error when triggering from the master branch of one repo and targeting a remote repo that doesn't have a master branch (the "target" repo uses main instead).
I was able to resolve the issue by hard-coding the ref as refs/heads/main, which feels wrong but appears to be a workaround:
      - name: Pin new image version in the k8s deployment repo
        uses: benc-uk/workflow-dispatch@v1
        with:
          ref: refs/heads/main # <-- this action is triggered in a repo with a 'master' branch, but targets a repo that uses 'main'
          workflow: Update pinned image tag
          repo: my-org/my-remote-repo
          token: ${{ secrets.WORKFLOW_TRIGGER_TOKEN }}
          inputs: '{ "tag": "${{ steps.vars.outputs.sha_short }}"}'

My action is simply triggering the GitHub API, and for the default ref I use the standard variable github.context.ref
When triggering remotely and the two repos have different default branches, you probably will need to set the ref yourself


Fail to build sdk namespace-related DTO in clone mode.

version
"@nestia/sdk": "^2.0.4",
"typescript": "^5.2.2",
Bug Report

every type in namespace is missing.
a dto with the same name as the namespace is missing.

Bug Test

clone project (branch: develop)

TEST Project

project setup & build sdk

// download dependency
npm i
// build sdk to ./sdk
npm run sdk

check bug

please check ./sdk/functional/users/normals/index.ts
related router code

@samchon bug is not completed in  "@nestia/sdk": "^2.0.5",
related @TypedException();

Upgrade to v2.0.6, then be fixed.


MINOR: Double timeout passed to producer.close in sendAndVerifyTimestamp

We have had transient failures in this method when Jenkins is
overloaded.

Review by @ewencp.

@ijuma LGTM


Make concurrent connections respect limits

When I tried to use aiohttp.ClientSession to make many concurrent requests to the same server, I found that I ended up with more connections than I expected.
Using the current master branch, the script below makes 3 connections for 3 requests, despite setting a limit of 1 on the connector. The problem seems to be that connections aren't counted against the limit until after the underlying trasport has been created. This creates a race condition where connect yields from _create_connection before recording anywhere that one of the available connections has been consumed. Subsequent calls to connect that begin before the earlier _create_connnection call has returned are able to create an unlimited number of connections.
# This script should make only 1 connection, but it actually makes 3.

import asyncio
import logging

import aiohttp

logger = logging.getLogger(__name__)

class LoggingTCPConnector(aiohttp.TCPConnector):
    async def _create_connection(self, req):
        try:
            connection_id = self._connection_id
        except AttributeError:
            connection_id = 0
        self._connection_id = connection_id + 1
        logger.debug("CREATING CONNECTION %s", connection_id)
        transport, protocol = await super()._create_connection(req)
        transport.connection_id = connection_id
        logger.info("CREATED CONNECTION %s", connection_id)
        return transport, protocol

    def _release(self, key, req, transport, protocol, *, should_close=False):
        connection_id = transport.connection_id
        logger.debug("RELEASING CONNECTION %s", connection_id)
        super()._release(key, req, transport, protocol, should_close=False)
        logger.info("RELEASED CONNECTION %s", connection_id)

async def make_many_requests(url, num_connections, num_requests):
    http = aiohttp.ClientSession(
        connector=LoggingTCPConnector(limit=num_connections),
    )

    async def make_one_request(request_id):
        logger.debug("MAKING REQUEST %s", request_id)
        response = await http.request("GET", url)
        await response.release()
        logger.info("MADE REQUEST %s", request_id)

    with http:
        tasks = [
            asyncio.ensure_future(make_one_request(request_id))
            for request_id in range(num_requests)
        ]
        await asyncio.wait(tasks)

logging.basicConfig(level=logging.INFO)
loop = asyncio.get_event_loop()
try:
    loop.run_until_complete(
        make_many_requests("http://example.com/", 1, 3)
    )
except KeyboardInterrupt:
    loop.stop()
finally:
    loop.close()

This pull request includes a new test that fails on the current master branch, but passes with my changes to the connector. I'm also including the first version of the test that I wrote below, because I was surprised when it actually passed on current master. I want to highlight the use of a mock that returns a done future in place of a coroutine, in this case _create_connection. I copied this pattern from test_connect_with_limit and it broke my test because while a function that returns a done future can be yielded from as if it were a coroutine, it never actually yields control back to the event loop. The result in this case was that all tasks ran sequentially rather than concurrently as they would in real use. I replaced the mock with a real coroutine and the test failed as expected. I mention all this because I'm concerned about this pattern potentially masking other problems elsewhere in the tests.
# This test passes, but it shouldn't.

def test_connect_with_limit_concurrent(self):

    @asyncio.coroutine
    def go():
        tr, proto = unittest.mock.Mock(), unittest.mock.Mock()
        proto.is_connected.return_value = True

        class Req:
            host = 'host'
            port = 80
            ssl = False
            response = unittest.mock.Mock(_should_close=False)

        conn = aiohttp.BaseConnector(loop=self.loop, limit=1)
        conn._create_connection = unittest.mock.Mock()
        conn._create_connection.return_value = asyncio.Future(
            loop=self.loop)
        conn._create_connection.return_value.set_result((tr, proto))

        @asyncio.coroutine
        def f():
            connection = yield from conn.connect(Req())
            connection.release()

        tasks = [asyncio.async(f(), loop=self.loop) for i in range(10)]
        yield from asyncio.wait(tasks, loop=self.loop)
        self.assertEqual(1, conn._create_connection.call_count)

    self.loop.run_until_complete(go())

@asvetlov I updated the check for available connections based on your feedback.
My original test didn't catch the problem because I was only checking the case where all connections were requested up front. I've now updated it to make more than limit connections up front and then also follow up with more connections after the initial ones are released.
Sorry it took me a while to get back to this. I had prepared the changes, but then our new baby decided it was time to arrive. I got a little distracted for a few days. :-)

I wish happy long life for your baby.

Thanks a lot!


Bump steve and [indirectly] apiserver to latest

Bumping steve and apiserver to pickup fixes for https://github.com/rancher/rancher/issues/39413

Looks like I'll need to make a few adjustments to get it working with the new steve version


Doesn't find Clang in Linux when using the Build.sh tool

Describe the bug
When I try the following command:
/home/blasco/Programs/UnrealEngine/Engine/Build/BatchFiles/Linux/Build.sh -project=/home/blasco/Documents/Unreal/MyProject/MyProject.uproject -mode=GenerateClangDatabase UE4Game Linux Development

It fails with:
Fixing inconsistent case in filenames.
Setting up Mono
Building -project=/home/blasco/Documents/Unreal/MyProject/MyProject.uproject...
ERROR: Clang must be installed in order to build this target.

But Clang is installed, I was actually able to build the engine. Building the project from the engine also works, but I want to use the Build.sh tool to generate the compile_commands.json. Where is it looking for clang? Why doesn't find it?
clang --version
clang version 10.0.0-4ubuntu1
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin

clang++ --version
clang version 10.0.0-4ubuntu1
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin

Desktop (please complete the following information)

OS: Ubuntu

Hi,
I'm not sure how this error is related to CodeChecker. Where is the CodeChecker command executed?
Can the Build.sh script generate a compile_command.json or you want to try to log the compiler calls with CodeChecker?
In CodeChecker there is a log command which can be used to generate a compile_command.json file.
Probably something like this could work:
CodeChecker log -o compile_command.json -b "/home/blasco/Programs/UnrealEngine/Engine/Build/BatchFiles/Linux/Build.sh -project=/home/blasco/Documents/Unreal/MyProject/MyProject.uproject -mode=GenerateClangDatabase UE4Game Linux Development"

but I'm not familiar what the Build.sh script does.

This issue is connected to #2987.


How will pybids evolve as the MEG/EEG/iEEG specs come into fruition?

It seems like pybids does a couple of things, some of which are more sharabed between specifications (e.g., parsing folders and returning metadata about them, such as grabbids), and some that are less shared (e.g., loading and creating design matrices)
I'm curious if the plan is for pybids to be generalized across all of the specifications, or unique to the MRI specs, or some hybrid between the two. E.g., I was gonna write some kind of I/O patch to MNE-python, but I'd really prefer to recommend using a different open tool like grabbids instead of reinventing the wheel, but I'm not sure whether that's a reasonable assumption

Just to clarify - BIDS Extension Proposals are meant to be temporary
discussion tools with the intention to merge each of them into the main
specification. This is what is happening now for MEG.
I predict that pybids will grow organically according to the needs of the
community - with contributions from people that need support for a given
data type.
On Feb 23, 2018 4:12 PM, "Chris Holdgraf"<EMAIL_ADDRESS>wrote:

It seems like pybids does a couple of things, some of which are more
sharabed between specifications (e.g., parsing folders and returning
metadata about them, such as grabbids
https://github.com/INCF/pybids/tree/master/bids/grabbids), and some
that are less shared (e.g., loading and creating design matrices)
I'm curious if the plan is for pybids to be generalized across all of the
specifications, or unique to the MRI specs, or some hybrid between the two.
E.g., I was gonna write some kind of I/O patch to MNE-python, but I'd
really prefer to recommend using a different open tool like grabbids
instead of reinventing the wheel, but I'm not sure whether that's a
reasonable assumption
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/INCF/pybids/issues/132, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOkp3aF2xJZfz4qjF70Eq41IhoDEK6dks5tX1P5gaJpZM4SRpdM
.

Yup!
On Feb 23, 2018 4:22 PM, "Chris Holdgraf"<EMAIL_ADDRESS>wrote:

I'm taking this to mean "pybids is friendly to PRs that add support for
non-fmri data but this will likely come from people from those respective
communities", yeah?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/INCF/pybids/issues/132#issuecomment-368177317, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAOkp1CcZCLU_iCRLYbz-0FIeclFG7wdks5tX1ZHgaJpZM4SRpdM
.

linking https://github.com/mne-tools/mne-bids/issues/54 to this issue.
cc: @jasmainak


Add multi-KDMA targets for testing

Adds multi-kdma targets for testing
Ingroup Bias-lowhigh means IB is low and MJ is high whereas Moral judgement-lowhigh means MJ is low and IB is high, i.e the KDMA that will be scored by the TA1/3 server is listed first

The targets look good, I think the names are a little confusing. Could we put both the KDMAs in them? Maybe something like IO-low_MJ-high?

Since these are not the "final" targets from TA1s I don't think the naming is super important if we just want to get this merged

Target files have been renamed


runPlutipContract stalls then errors with small utxo values

If a utxo distribution provides a value lower than ~800000 lovelace (rough estimate), runPlutipContract will stall then eventually error with ClientHttpError There was a problem making the request: request failed. Reproduce with:
  let
    distribution =
        [ BigInt.fromInt 2_000_000_000
        , BigInt.fromInt 800000
        ]
  runPlutipContract config distribution \alice -> pure unit

This is caused by user error (I think?), but it should probably be handled better.

It works for me on be0c81b6dde21fa5370740b642aa549e93f74048

@klntsky Testing this out on https://github.com/Plutonomicon/cardano-transaction-lib/commit/be0c81b6dde21fa5370740b642aa549e93f74048 it still shows the behaviour I described. For example, if I make the following change at that commit:
modified   test/Plutip.purs
@@ -69,7 +69,7 @@ import Examples.MintsMultipleTokens
   , mintingPolicyRdmrInt2
   , mintingPolicyRdmrInt3
   )
-import Mote (group, test)
+import Mote (group, only, test)
 import Plutip.Server
   ( startPlutipCluster
   , startPlutipServer
@@ -93,7 +93,7 @@ main = launchAff_ do
   Utils.interpretWithConfig
     -- we don't want to exit because we need to clean up after failure by
     -- timeout
-    defaultConfig { timeout = Just $ wrap 30_000.0, exit = true }
+    defaultConfig { timeout = Just $ wrap 30_0000.0, exit = true }
     suite

 config :: PlutipConfig
@@ -146,12 +146,13 @@ suite = do
           _ -> false
         liftEffect $ Console.log $ "stopPlutipCluster: " <> show stopRes

-    test "runPlutipContract" do
+    only $ test "runPlutipContract" do
       let
         distribution :: InitialUTxO /\ InitialUTxO
         distribution =
           [ BigInt.fromInt 1_000_000_000
           , BigInt.fromInt 2_000_000_000
+          , BigInt.fromInt 800000
           ] /\
             [ BigInt.fromInt 2_000_000_000 ]
       runPlutipContract config distribution \(alice /\ bob) -> do

I get:
  ✗ runPlutipContract:

  (ClientHttpError There was a problem making the request: request failed)

Thanks


Some sort of easter egg or hidden feature

This is extremely important. If we do not include this, PlanHub will never succeed.
But it can't be a snake game.
...
@medensqui @medensqui

That second mention should be @nywillb

@thatoddmailbox Y?

@thatoddmailbox How about the spinny hex security hole?

What about a pacman type of thing? Or we could have the Konami Code change the background color. See http://www.theverge.com/2013/5/17/4340740/google-hangouts-easter-eggs-include-shy-dinosaurs-and-stampeding

@nywillb Obviously we make the Konami code trigger pacman that, when you win, changes the background color and spins the page.

@thatoddmailbox NO SPINNY HEX BUG

@medensqui No as in "no, don't do this?" or no as in "no, do this instead?"

@thatoddmailbox DO IT

@thatoddmailbox that is awesome. But if you type /ponies into the planner box, it should make a ponies go across the screen.

@nywillb IT DIDNT WORK ALSO GRAMMAR
"IT SHOULD MAKE A PONIES" "IT SHOULD MAKE A PONIES" WHAT? TSK TSk

@medensqui Try it in google hangouts. It will explain everything.

@nywillb @medensqui
For the lazy, I found an image online: (not my phone)

@thatoddmailbox @medensqui Your idea is the best:

@nywillb Obviously we make the Konami code trigger pacman that, when you win, changes the background color and spins the page.
except for one revision. Every time you get a point it does that stuff. This is ready to implement.

@willweinstein Did you implement this?

@nywillb the spinny thingy? yes. type in ey.hex(u);;;;; and click away.  TO GET RID OF IT DELETE THE CODE AND RELOAD THE PAGE. I AM NOT RESPONSIBLE FOR ANY PERMANENT SPINNYNESS

Obviously this must be re-opened
duh
cause memes

Umm @em-p
I'm not sure of any need for memes right now. @PlanHubMe what do we think?

@RobotXLabs Reopening because we do need more easter eggs (but you are right about the memes)

K cool

Closing because this issue is sort of vague. If you really think there should be another easter egg, please make another issue for it.