valgrind: 3.16.1 -> 3.17.0

Version 3.17.0 contains support for sched_getattr and sched_setattr.
This fixes an issue where the following messages can be logged:
WARNING: unhandled amd64-linux syscall 315

See: https://bugs.kde.org/show_bug.cgi?id=369029
In addition to the version bump, this commit enables doCheck (as the
tests now pass) and provides a separate pname and version.
The upstream 3.17.0 also reorganized the output, placing the
default.supp file in libexec rather than lib. The postInstall hook was
adjusted accordingly.
A patch file which has not been used since upgrade to 3.15.0 in 2019
was also removed.

Motivation for this change
Fix above-mentioned issue, as well as other fixes and enhancements.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[ ] Fits CONTRIBUTING.md.

Note: testing of binaries was cursory with the exception of valgrind.
cc @edolstra as maintainer

/rebase staging

@tollb please rebase manually on staging

Thanks @symphorien and @SuperSandro2000. I am in the process of rebasing. I'm trying to rebuild/test on staging and it is taking a while....

Sorry about all of the review requests... They were added automatically when I changed the branch from master to staging. I'm not sure how to clear them.

As a note, I encountered the above-mentioned valgrind message "WARNING: unhandled amd64-linux syscall 315" while working with a program that uses the pango library. The pango library was updated in 1.48.3 to "Do costly fontconfig calls in threads". See: https://github.com/GNOME/pango/blob/1.48.3/NEWS.
The warning can be seen with valgrind 3.16.1 and pango 1.48.4 as found in a recent nixos-unstable:
$ nix-shell -I nixpkgs=https://github.com/NixOS/nixpkgs/archive/e1f8852faac7638e88d5e8a5b9ee2a7568685e3f.tar.gz -p pango -p valgrind --run "valgrind pango-list" >/dev/null
==467965== Memcheck, a memory error detector
==467965== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==467965== Using Valgrind-3.16.1 and LibVEX; rerun with -h for copyright info
==467965== Command: pango-list
==467965==
--467965-- WARNING: unhandled amd64-linux syscall: 315
--467965-- You may be able to write your own handler.
--467965-- Read the file README_MISSING_SYSCALL_OR_IOCTL.
--467965-- Nevertheless we consider this a bug.  Please report
--467965-- it at http://valgrind.org/support/bug_reports.html.
==467965==
==467965== HEAP SUMMARY:
==467965==     in use at exit: 798,534 bytes in 19,125 blocks
==467965==   total heap usage: 35,869 allocs, 16,744 frees, 4,282,916 bytes allocated
==467965==
==467965== LEAK SUMMARY:
==467965==    definitely lost: 139,264 bytes in 544 blocks
==467965==    indirectly lost: 167,665 bytes in 6,645 blocks
==467965==      possibly lost: 1,092 bytes in 4 blocks
==467965==    still reachable: 487,745 bytes in 11,899 blocks
==467965==         suppressed: 0 bytes in 0 blocks
==467965== Rerun with --leak-check=full to see details of leaked memory
==467965==
==467965== For lists of detected and suppressed errors, rerun with: -s
==467965== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

The warning does not occur with valgrind 3.16.1 and pango 1.45.5 found in a recent nixos-20.09:
$ nix-shell -I nixpkgs=https://github.com/NixOS/nixpkgs/archive/59b8d9cf24e9fcf10341a0923c9bdca088dca8c8.tar.gz -p pango -p valgrind --run "valgrind pango-list" >/dev/null
==467617== Memcheck, a memory error detector
==467617== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==467617== Using Valgrind-3.16.1 and LibVEX; rerun with -h for copyright info
==467617== Command: pango-list
==467617==
==467617==
==467617== HEAP SUMMARY:
==467617==     in use at exit: 782,902 bytes in 19,015 blocks
==467617==   total heap usage: 35,755 allocs, 16,740 frees, 4,220,243 bytes allocated
==467617==
==467617== LEAK SUMMARY:
==467617==    definitely lost: 139,360 bytes in 545 blocks
==467617==    indirectly lost: 167,713 bytes in 6,646 blocks
==467617==      possibly lost: 1,420 bytes in 20 blocks
==467617==    still reachable: 474,145 bytes in 11,802 blocks
==467617==                       of which reachable via heuristic:
==467617==                         length64           : 224 bytes in 5 blocks
==467617==                         newarray           : 1,568 bytes in 18 blocks
==467617==         suppressed: 0 bytes in 0 blocks
==467617== Rerun with --leak-check=full to see details of leaked memory
==467617==
==467617== For lists of detected and suppressed errors, rerun with: -s
==467617== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

The warning should be resolved with an update to valgrind 3.17.0, as provided in this PR.
cc fyi for pango development: @7c6f434c @jtojnar

Updated to address a conflict introduced after PR was filed. NOTE: OfBorg test on darwin seems to have stalled.


Tag and Untag API for cluster.

Issue #, if available:
ACK MemoryDB controller doesn't provide tags update after creation of cluster.
Description of changes:
Add functions so that customers can update tags of cluster after creation.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@jljaco and I discussed about the READ_MANY vs GET issue. For now we don't have a precise answer and we'll try to discuss tomorrow.
merging as is
/lgtm


Added bg position and toastify

Added position for bg
Added toastify, but for registration it isn't working? I don't know why(. Could you get a look on it, please ?
Trouble with ts at thunks

На далі перевріряй ПР на конфлікти


feat(ci): add codecov

@vdice Any idea why Jenkins is having syntax errors on this pull request?

@Joshua-Anderson hmmm I don't know why it would complain... I'm not aware of the bash <(<bash script>) syntax and/or how it differs/compares to the classic <bash script> | bash approach... but perhaps try the latter approach?

@vdice The script works for me locally and test-cover does successfully generate a coverage support (coverage.txt). However, in CI, codecov says it can't find any coverage files. Any ideas? Could this be related to the file being created with root permissions? If so, why doesn't travis-ci have this problem?

Current coverage is 5.25% (diff: 100%)

No coverage report found for master at c655ea0.

Powered by Codecov. Last update c655ea0...678013c

As you can see, the PR works! Ready for reviews!

This passed e2e but the result wasn't properly logged


Add ability to select display when creating window

Passing display: window-id will create the window on that display, with any x and y coordinates relative to that displays bounds. However, this removes some default behavior where the window would display in the center of the screen if no x or y coordinates were passed in. This has been changed to [0, 0] to keep it consistent between containers.
Also added clean:cache as a script in api-tests, which deletes %LOCALAPPDATA%/OpenFin/cache, which is run before the tests. It is not run before npm run test:ci: ... as the cache doesn't persist on CI.

@DevAndyLee Im still working on getting the window to display in the middle, the tests are failing, saying the height is 5px off :confused:


Add number of marked/sweeped objects to GC stdout

Resolves #5226.

@Jonathan-Rosenberg - thanks for the review, PTAL. Also note two additional changes:

[Breaking] Removed "deleted.parquet" from the path of the saved report (it causes issues when trying to read it in Spark).
Not failing on any exception when printing the count.


Match UI To Existing Wireframes

This bug list covers items in the UI that need to be updated to match the wireframes created by UXD. If an item/area has not been covered by wireframes, it should be run past UXD for review. All views should be created for mobile first, as the design process is mobile first.

[ ] Mobile First on all screens / interactions

[ ] Review visuals to match PatternFly

PatternFly Pattern Library and PatternFly Styles

[ ] Add Work Item needs to be fixed to the bottom of the screen

https://redhat.invisionapp.com/share/798MZ3GG5#/185961156_AddAWorkItem-1-v3-

[ ] Desktop list view needs updating

https://redhat.invisionapp.com/share/CJ8W68V2T#/193330603_List

[ ] Details View needs to be a panel

https://redhat.invisionapp.com/share/UV8W6AEHX#/193330600_Panel_Edit

[ ] Username removed from navigation bar (there is no concept of a user ATM)

[ ] Remove Status from List View

[ ] Kabob Menu Implementation

https://redhat.invisionapp.com/share/YF8W6CC5Z

[ ] Button Alignments (bottom right, left to right: Cancel -> Save)

https://redhat.invisionapp.com/share/648W6CZVP#/194297190_ListView_-_6

[ ] Catalogue with UX team on @almighty-ux any issues that need review / questions on designs and patterns

@mindreeper2420 That is a great list. Can you please reference the existing issues for them or create an issue where one is missing. That will give us a place to discuss the design and track that it is getting worked.

@joshuawilson So individual issues for each on of these bullet points? I know one or two already exist, so I can link them here and create more, if that is the desire.

@joshuawilson I updated the list with proper references (GitHub, InVision and PatternFly) and created Issues in the almighty-ui repo.

@mindreeper2420 thanks, that will help us track the issues and get them worked. :)

Closing as the list has been completed.


Add probability field calc and plots

Codecov Report
Attention: 90 lines in your changes are missing coverage. Please review.

Comparison is base (1fbaf14) 21.29% compared to head (fad0e19) 20.79%.
Report is 1 commits behind head on main.

Files
Patch %
Lines

...rturbation/uncertainty_quantification/surrogate.py
0.00%
56 Missing :warning:

ensembleperturbation/plotting/surrogate.py
0.00%
33 Missing :warning:

ensembleperturbation/parsing/schism.py
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #127      +/-   ##
==========================================
- Coverage   21.29%   20.79%   -0.50%
==========================================
  Files          28       28
  Lines        3762     3851      +89
==========================================
  Hits          801      801
- Misses       2961     3050      +89

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

This is an example of its use. I got better result on a different dataset, but I don't have it now!

@WPringle swapping the order of set small elev to null and set null to depth for model calculation, (i.e.
https://github.com/noaa-ocs-modeling/EnsemblePerturbation/blob/4d82917f3e1fcce456fd7b5a587da07b7b8658e7/ensembleperturbation/uncertainty_quantification/surrogate.py#L676-L678
and
https://github.com/noaa-ocs-modeling/EnsemblePerturbation/blob/4d82917f3e1fcce456fd7b5a587da07b7b8658e7/ensembleperturbation/uncertainty_quantification/surrogate.py#L679-L679
) resulted in these plots:

I think it might make more sense to keep the swapped version. What do you think?

These are the first couple of probability fields for the 30 member Florence run with OFCL track (past forecast, no NWM)


[rlgl] Batch elements overflow

Hello after update RayLib my program not work.
INFO: FILEIO: [Assets/font/DejaVuSansMono.ttf] File loaded successfully
WARNING: TEXTURE: Failed to load texture
INFO: FILEIO: [Assets/font/DejaVuSansMono.ttf] File loaded successfully
WARNING: TEXTURE: Failed to load texture
INFO: FILEIO: [Assets/font/DejaVuSansMono.ttf] File loaded successfully
WARNING: TEXTURE: Failed to load texture
INFO: FILEIO: [Assets/font/DejaVuSansMono.ttf] File loaded successfully
WARNING: TEXTURE: Failed to load texture
INFO: FILEIO: [Assets/font/DejaVuSansMono.ttf] File loaded successfully
WARNING: TEXTURE: Failed to load texture
INFO: Initializing raylib 3.5
INFO: DISPLAY: No graphic card set, trying card1
INFO: DISPLAY: Selected DRM connector mode FIXED_MODE (800x480p@60)
INFO: DISPLAY: Device initialized successfully
INFO:     > Display size: 800 x 480
INFO:     > Render size:  800 x 480
INFO:     > Screen size:  800 x 480
INFO:     > Viewport offsets: 0, 0
INFO: GL: OpenGL device information:
INFO:     > Vendor:   Broadcom
INFO:     > Renderer: V3D 4.2
INFO:     > Version:  OpenGL ES 3.1 Mesa 19.3.2
INFO:     > GLSL:     OpenGL ES GLSL ES 3.10
INFO: GL: Supported extensions count: 76
INFO: GL: VAO extension detected, VAO functions initialized successfully
INFO: GL: NPOT textures extension detected, full NPOT textures supported
INFO: GL: ETC1 compressed textures supported
INFO: TEXTURE: [ID 1] Texture created successfully (1x1 - 1 mipmaps)
INFO: TEXTURE: [ID 1] Default texture loaded successfully
INFO: SHADER: [ID 1] Compiled successfully
INFO: SHADER: [ID 2] Compiled successfully
INFO: SHADER: [ID 3] Program loaded successfully
INFO: SHADER: [ID 3] Default shader loaded successfully
INFO: RLGL: Internal vertex buffers initialized successfully in RAM (CPU)
INFO: RLGL: Render batch vertex buffers loaded successfully
INFO: RLGL: Default state initialized successfully
INFO: TEXTURE: [ID 2] Texture created successfully (128x128 - 1 mipmaps)
INFO: FONT: Default font loaded successfully
INFO: RPI: Opening input device: /dev/input/event0 (multitouch touchscreen )
WARNING: RPI: Failed to open Gamepad device, no gamepad available
WARNING: RPI: Failed to change keyboard mode (SSH keyboard?)
INFO: TIMER: Target time per frame: 16.667 milliseconds
'lo': <IP_ADDRESS>
'eth0': <IP_ADDRESS>
UI TEST
Load From File Assets/img/cruscotto.png
INFO: FILEIO: [Assets/img/cruscotto.png] File loaded successfully
INFO: IMAGE: [Assets/img/cruscotto.png] Data loaded successfully (800x480)
INFO: TEXTURE: [ID 3] Texture created successfully (800x480 - 1 mipmaps)
INFO: FILEIO: [Assets/font/DejaVuSansMono.ttf] File loaded successfully
INFO: TEXTURE: [ID 4] Texture created successfully (256x256 - 1 mipmaps)
INFO: FILEIO: [Assets/font/DejaVuSansMono.ttf] File loaded successfully
INFO: TEXTURE: [ID 5] Texture created successfully (256x256 - 1 mipmaps)
INFO: FILEIO: [Assets/font/DejaVuSansMono.ttf] File loaded successfully
INFO: TEXTURE: [ID 6] Texture created successfully (1024x1024 - 1 mipmaps)
INFO: FILEIO: [Assets/font/DejaVuSansMono.ttf] File loaded successfully
INFO: TEXTURE: [ID 7] Texture created successfully (512x512 - 1 mipmaps)
INFO: FILEIO: [Assets/font/DejaVuSansMono.ttf] File loaded successfully
INFO: TEXTURE: [ID 8] Texture created successfully (512x512 - 1 mipmaps)
INFO: FILEIO: [Assets/font/DejaVuSansMono.ttf] File loaded successfully
INFO: TEXTURE: [ID 9] Texture created successfully (512x512 - 1 mipmaps)
ERROR: RLGL: Batch elements overflow

I find cause DrawRing.

@dginu85 Please, could you provide a minimal example illustrating the issue?

Platform RPI4 DRM
BeginDrawing();
DrawRing({400, 400}, /*INTERNO*/ 190, /*ESTERNO*/270, /*PAR*/map(VAL, 0, 100, 270, 90), 270, 100, Fade({0, 139, 240}, 0.9));
EndDrawing();

@dginu85 Please, could you provide a complete example file to test? That line seems to be using some custom functions.

BeginDrawing();
DrawRing({400, 400}, 190,270, 100, 270, 100,  Fade(MAROON, 0.3));
EndDrawing();

@dginu85 that piece of code does not work in C99, are you using C++?

YES

@dginu85 That piece of code works ok for me. I can't reproduce.

You can try review rlCheckRenderBatchLimit(4*segments); on DrawRing() on shapes.c and increase that value, just to make sure the issue is on the buffer limits for that draw call.

have you tried on raspbery?

@dginu85 I tried on desktop using RPI batch limits configuration. If it fails only on RPI DRM, that function is probably not the issue.

Hello @raysan5 today i compile raylib in my desktop.
I try to start my application
external/glfw/src/input.c:516: glfwSetInputMode: Assertion `window != NULL' failed
To understand where I was wrong, I decided to build examples
make: *** [Makefile:525: shapes/shapes_draw_ring] Errore 1
linuxstation:~/Scrivania/Librerie/raylib/examples$

@dginu85 first issue is related to GLFW library but without more context it's very difficult to know the reason.
Examples issue is related to some changes I did to raygui yesterday.


Add health checker implementation

*Populate the healthchecker Ip
*Ping the healthchecker IP to check the remote
gateway status
*Update the gateway satus if the ping fails
Fixes: submariner-io/submariner#821
Signed-off-by: Aswin Surayanarayanan<EMAIL_ADDRESS>

Some comments,

I believe ping should be running in the background all the time to get meaningful statistics and reliable detection of the connection going down.
Better integration to get the connection list from memory, or from the syncer that checks the connection status periodically instead of querying the k8s API for this.
I will try if this is possible with the library and  refactor the code based on  the findings.


⚠ main has degraded performance

In ac8c4a5, main ($MAIN_API_URL) experienced degraded performance:

HTTP code: 200
Response time: 21897 ms

Resolved: main performance has improved in a099d8e.


Switching between search index

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [x ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Hi,

I’m looking to switch between different indexes in the Azure AI search from the frontend. Like a dropdown menu: Index1, Index2, Index3. Could you guide me on how to do this? Specifically, which file should I edit to modify the API call for selecting different indexes?
I’m currently editing the chatReadRetrieveread file, but I’ve struggled to adjust the frontend code to facilitate this switching. Any tips on what parts of the code I should focus on would be greatly appreciated!
Thanks for your help!
Any log messages given by the failure

Expected/desired behavior

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

azd version?

run azd version and copy paste here.

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

@daptatea - I've implemented a feature similar to what you might considering for switching between search indexes. In my case, I use a single index (named "gptkbindex") which contains categorized data, let's say category1(for Past Inquiries), category2(for Product Manuals), category3(etc.). To facilitate this, I introduce an "include category"(*) option in the developer settings as a dropdown menu for selecting categories, with "All" as the default option.

(*) Check azure-search-openai-demo:RAG_Hack_Project for details.
Currently, we have an "exclude category" text box where specifying category1 filters for category2 and category3. However, this setup isn't very user-friendly, I guess. A more intuitive approach, like the multiselect list suggested in #1333, might be beneficial.
To utilize the "category" parameter, it should be included during the data ingestion process. I think there isn't any documentation in repo on this right now, but you might find relevant information in previous issues like this one: #1715

This is great . Would you please create a Pull Request

@pamelafox Is there anything you'd like to add? I am not entirely sure how we can approach this yet.

This functionality would be very useful and is definitely worth exploring for integration into the main branch.

I've seen this implemented on a colleague's version of this app. He put the dropdown in the same spot where @bnodir put the optional language picker, as opposed to putting it in the Developer Settings. The question is whether you consider this more of a "user setting" or more of a "developer setting", or if you end up exposing the developer settings to end users anyway. I don't know what is most common in production.

@pamelafox We've enabled Microsoft login for the app via Azure App Services, and based on that, we prefer this feature to be a user setting rather than a developer setting. This approach will enhance the user experience by allowing users to easily switch between different data sources / indexes. This flexibility helps ensure that the AI Search retrieves relevant data without being overwhelmed by excessive or irrelevant information, leading to a smoother interaction with the app.
However, either approach works for us, as we can always adapt it to our specific needs later.

It would also be helpful to include a "None" option to disable RAG, allowing the chat to rely solely on the model's general knowledge.


Validate FirebaseAnalytics.podspec.json explicitly separately

In the run FirebaseAnalytics.podspec.json was invalid but the error was obscure. Let's have an explicit validation for the spec.
#no-changelog

Closing the PR so far because --include-podspecs cannot include JSON podspec:
scripts/pod_lib_lint.rb FirebaseAnalytics.podspec.json
bundle exec pod lib lint --sources=https://github.com/firebase/SpecsDev.git,https://github.com/firebase/SpecsStaging.git,https://cdn.cocoapods.org/ --include-podspecs={FirebaseCore.podspec,FirebaseCoreDiagnostics.podspec,FirebaseInstallations.podspec,GoogleAppMeasurement.podspec.json} --analyze FirebaseAnalytics.podspec.json

 -> FirebaseAnalytics (7.9.0)
    - ERROR | [iOS] unknown: Encountered an unknown error (No podspec found for `GoogleAppMeasurement.podspec` in `.`) during validation.


Environment

Store declarations in an environment, and look them up therein.

This doesn’t actually allow for lookups in the environment just yet but I want to switch to datatypes for the moment, so merging.


NBDserver per vm instead of per vdisk

Change vm actor to update the disks config with all disks and send hub signal to nbdserver to reload.
Make sure this applied for both update and create of multiple disks.

SV
691e5e8d9bcac157d47dca636bac177527e034b4


error(1,0): Invalid member for expression: .xyzz; In shader code that uses swizzling.

Operating system or device:
Arch Linux: 4.4.5-1-ARCH x86_64 GNU/Linux
Graphics card info:
00:02.0 VGA compatible controller [0300]: Intel Corporation Haswell-ULT Integrated Graphics Controller [8086:0a16] (rev 09) (prog-if 00 [VGA controller])
    Subsystem: Toshiba America Info Systems Device [1179:fa48]
    Flags: bus master, fast devsel, latency 0, IRQ 44
    Memory at f7800000 (64-bit, non-prefetchable) [size=4M]
    Memory at e0000000 (64-bit, prefetchable) [size=256M]
    I/O ports at f000 [size=64]
    Expansion ROM at <unassigned> [disabled]
    Capabilities: <access denied>
    Kernel driver in use: i915
    Kernel modules: i915

Godot verson:
Godot_v2.0.1_stable_x11.64
Issue description::
When using the following shader code in Godot that demonstrates 'swizzling'...
// Blank line #1
vec3 a = vec3(1,2,3);
vec3 b = a.zyx; // b will contain vec3(3,2,1)
vec2 c = a.xy; // c will contain vec2(1,2)
vec4 d = a.xyzz; // d will contain vec4(1,2,3,3)

I was given the following error: error(1,0): Invalid member for expression: .xyzz.  Also notice that the error message points to line 1 instead of line 5.
Note: This sample code came from the Godot Docs: http://docs.godotengine.org/en/latest/reference/shading_language.html#swizzling
It appears that the vec4 dot notation doesn't work at all without throwing an error.
At first, I thought it was due to the double 'z' character, but Godot doesn't like vec4 d = a.xyzw; either.
It appears that the error message originates from shader_language.cpp at line 1892, but my brief look at the code didn't reveal why vec4s don't work and vec2s and vec3s do.
https://github.com/godotengine/godot/blob/master/servers/visual/shader_language.cpp#L1892
Steps to reproduce:

Open a new project
Add a new textureframe to the scene graph
Add a new CanvasItemMaterial to the Materials property of the textureframe
Add a new CanvasItemShader to the Shader property of the CanvasItemMaterial
Edit the new shader and add the sample code above to the textbox that is shown
Wait for the shader to be compiled and the error message to be shown below the text area

shader compiler will get a rewrite in 3.0, so awaiting for this version

shader compiler got a rewrte in 3.0, so you don't have to await any longer, reduz of a year ago.


Assign multiplayer star rating

[ ] Depends on ppy/osu/pull/23551

This retrieves beatmap's difficultyrating from the db and uses it to populate MultiplayerPlaylistItem which is then used by the client to show latest information about room's star rating .

Seems fine, but given the fact that this needs a whole lot of things to happen before merge (merge of https://github.com/ppy/osu-server-spectator/pull/174 first, then a game package with https://github.com/ppy/osu/pull/23551, which is going to be in the release-after-next), I'm just going to let this hang unapproved until CI can be actually unblocked.


Inappropriate error message

The following module includes an error (a duplicate cell in the rule), but the error message is total inappropriate. In a big definition it is not easy to see such an error. I think that it can be easily detected.
module TEST

  configuration
    <k> 1 </k>
    <env> .Map </env>

  rule  <k> 1 => .K ...</k>
        <env> E:Map </env>
        <env> .Map </env>

endmodule

Here is the error mesage:
$ kompile test.k
[Error] Internal: Uncaught exception thrown of type AssertionError.
Please file a bug report at https://github.com/kframework/k/issues

I think I know where that assert is


Update the version since 1.0.0-alpha3 is published

@davezych I messed up. 1.0.0-alpha3 is published. I'm just updating this to 1.0.0-alpha4 so when we push it, it'll update the version in the feed.

:shipit:

I'm sure it'll build successfully. YOLOMERGE!


Don't Merge! Regression PR perf testing with metadata validation

Docs Build status updates of commit e7e717f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

developer/TestingPullRequest.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 4939a5b:
:warning: Validation status: warnings

File
Status
Preview URL
Details

reference/5.1/CimCmdlets/TestingPullRequest.md
:warning:Warning
View (powershell-5.1)
Details

developer/TestingPullRequest.md
:white_check_mark:Succeeded

reference/5.1/CimCmdlets/TestingPullRequest.md

[Warning] Metadata with following name(s) are not allowed to be set in Yaml header, or as file level metadata in docfx.json, or as global metadata in docfx.json: content_type. They are generated by Docs platform, so the values set in these 3 places will be ignored. Please remove them from all 3 places to resolve the warning.
[Warning] Missing attribute: title. Add a title string to show in search engine results.
[Warning] the value invalid_type is undefined for metadata content_type
Line 8: [Warning] Invalid file link:(~/5.1/nonexisted1.md).
Line 9: [Warning] Invalid file link:(~/5.1/nonexisted2.md).
Line 10: [Warning] Invalid file link:(~/5.1/nonexisted3.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add user-properties info to jira_user table. closes #12

Integration test logs

Logs
N/A

Example query results

Results
N/A

Closing this PR due to staleness, will re-open if this feature request comes up again.


engine: better messaging around the fact that we only support live update for the first container on a pod

Addresses #1522

This is the wrong approach, #1530 replaces this. Closing.


Appveyor 4 windows

Added Xenon build to appveyor at https://ci.appveyor.com/project/NLeSC/xenon
Refs #190

Added badge
push/pr hooks added
using appveyor.yml
shows test results in tab
for now only unit tests

The testing seems to work fine, but some tests are failing under windows.
I guess I should merge the pull request and try to fix the windows specific bugs ?

Fixing the windows issues now

Just added the local integration tests as well. I would like to keep the develop branch green, so fix the windows errors here or in another branch.

OK, will do.

I've created a 'windows-quircks' branch that contains a few fixes. Two more bugs to go I think.

I think I managed to fix the bugs. As it turns out, we did not close directory streams in some test and utility classes. This is not a problem on linux/mac, but on windows the directory remains locked in some way, which results in an exception if you try to delete it later.
Should be fixed now in windows-quicks branch.


Graphs are movable but I don't think they're supposed to be

On the player pages and the team pages, the graphs can be moved and I don't think they should be able to.
https://github.com/user-attachments/assets/b03b4a5a-4765-4564-9c48-08452ffe913e

Hey, I will look into this. I agree that the zooming is generally not helpful, especially when it accidentally happens on scroll


Document solution to No GPU found. Conversion on CPU is very slow.

I think it would be cool to add information to the "Install" section of the README that in order to use the GPU, one needs to install CUDA 11.7 or 11.8 and pytorch according to https://pytorch.org/get-started/locally/ (so something like pip3 install torch torchvision torchaudio --index-url https://download.pytorch.org/whl/cu117) on Windows before running pip install nougat-ocr (which otherwise installs only the CPU versions). I don't know though if these steps are only needed on Windows or also on other platforms.
Thanks a lot for developing this, it performs much better than the previous stuff I dabbled with.

I agree with that. And how to reinstall in order to enable the GPU ? I tried to reinstall but there is still no GPU found.

I think you need to pip uninstall torch (and the same for torchvision) before installing them again from the pytorch index.

I've added a notice in the readme

@lukas-blecher Thanks a lot!


No rule to make target 'source/ppiclf_user.f'

Trying to compile the library gives:
$ make
cd ./3rd_party/gslib; \
./install

***********************
*** INSTALLED GSLIB ***
***********************

make: *** No rule to make target 'source/ppiclf_user.f', needed by 'getObjs'. Seis.

Am I supposed to create ppiclf_user.f?

This is the same issue as here: https://github.com/dpzwick/ppiclF/issues/25.

I am going to close this issue since I think following the website will solve this issue for you. If it doesn't please comment back here and I will reopen it.


Querying type with a query variable not working while it works with a string

EdgeDB Version: alpha 2

Steps to Reproduce:

Create an enum
Try the following query

query($enum:String!){
  __type(name:$enum){
    enumValues{
      name
    }
  }
}
# variable
{ "enum": "Position" }

But this one works fine:
query{
  __type(name:"Position"){
    enumValues {
      name
    }
  }
}

Here is a similar query that works for reference https://serve.onegraph.com/short/FHRKVT

@tailhook @vpetrovykh

This is fixed in #1294 . We can probably wait for that to land, instead of patching this issue twice.

@haikyuu, this should already work on master/nighly.

Thanks @tailhook


DockerComposeEnvironment missing validation for wait strategies?

Expected Behaviour
I would expect that test containers would log error when I use withWaitStrategy with not existing container name
OR
that the documentation highlights where should I serviceName vs containerName explaining difference between those two to avoid confusion and related issues.
Actual Behaviour
Testcontainers 'silently fails' to override the wait for strategy for given container, causing the startup to fail.
Testcontainer Logs
N/A
Steps to Reproduce

docker-compose.yml

version: '3.7'
services:
...
  mockServer:
    image: mockserver/mockserver:mockserver-5.13.0
    command: -serverPort 1080 -logLevel DEBUG
    ports:
      - "1080:1080"
...

Run the code

const environment = await new  ('./', 'docker-compose.yml')
        .withBuild()
        .withWaitStrategy('mockServer', Wait.forLogMessage(/.+started on port.+/))
        .up(['mockServer']);

Observe error

Error: Port 1080 not bound after 60000ms

Notes:
The documentation under https://node.testcontainers.org/features/compose/ uses exactly the same value for service name and container name (both using suffix -1) instead of highlighting that those are different things. As a result users might use the same value in their code, which might lead to confusing results. It appears as if the user is not able to override the default strategy of waiting until port is bound - no error saying that the container with given name does not exist, but error about port not being bound.
Switching the code to use withWaitStrategy('mockServer-1', Wait.forLogMessage(/.+started on port.+/)) fixes the issue.
On the other hand, changing the code to use .up(['mockServer-1']) results in error no such service: mockServer-1.
Thus, couple of improvements that could be considered:

Log error/warning if a wait strategy was assigned to a container that does not exist, e.g. no such container: mockServer.
Update the documentation to highlight difference between service name and container name
Switch the API to only allow use either service name and container name

Environment Information

Operating System: MacOS 14.2
Docker Version: 27.2.0
Node version: 20.11.0
Testcontainers version: 10.13.1

Good point, PR welcome


Autocomplete fires on "."

If you have a variable that is named config and you want to use this variable, the config of chai wants to be imported.

Is kind of a hard achievement 😐

'.' does fire the command "accept code completion" which then fires the import.
So the completion provider should filter local variables of the file.

First task should be to add the document offset of the tokens (classes etc) to the nodes
-> document.positionAt(offset)


update(cmake): bumped libs to 0.17.0-rc1 and falcoctl to v0.8.0-rc6.

What type of PR is this?
/kind release
Any specific area of the project related to this PR?
/area build
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,LucaGuerra]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.38.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,LucaGuerra]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

We triggered some errors:
Error: engine_v4_k8s_audit_rules.yaml: Invalid
        	            	1 Errors:
        	            	In rules content: (engine_v4_k8s_audit_rules.yaml:0:0)
        	            	    rule 'ClusterRole With Pod Exec Created': (engine_v4_k8s_audit_rules.yaml:266:2)
        	            	    rule condition: (engine_v4_k8s_audit_rules.yaml:268:13)
        	            	    condition expression: ("kevt and (role or...":269:61)
        	            	------
        	            	... clusterrole) and kcreate and ka.req.role.rules.resources contains "pods/exec"
        	            	                                                          ^
        	            	------
        	            	LOAD_ERR_COMPILE_CONDITION (Error compiling condition): filter error: 'contains' operator not supported list filters

cc @jasondellaluce (?)

Opened pr to fix: https://github.com/falcosecurity/testing/pull/58


Movingto golang1.9.1

Moves to golang 1.9.1

This change is

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks broke.

Comments from Reviewable

closing this, did code change in bump-external-storage branch and PR


Update kfp manifests for 1.9.0

Context
Each charm has a set of manifest files that have to be upgraded to their target version. The process of upgrading manifest files usually means going to the component’s upstream repository, comparing the charm’s manifest against the one in the repository and adding the missing bits in the charm’s manifest
What needs to get done
See the release handbook
Definition of Done
Manifests and images are updated to the corresponding version for 1.9.0

The only change that was introduced in kfp manifests between 1.9.0-rc.0 and 1.9.0 release is the following. However, our KFP Pods don't have an istio sidecar, thus we don't need any AuthorizationPolicy to configure the pod of KFP. Thus, we don't need to bring any changes to kfp.


Seems to mess with internal API somehow

I was encountering https://github.com/gaearon/react-deep-force-update/issues/11 until I uninstalled it :(

Hi @wmertens, do you have any code you could share? Or a screen shot of the error? Thanks!


Fix: Crash caused by useless predicate.

Resolved / Related Issues

[X] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Not approved, but I think a crash should be fixed👀, this closes #15124

Validation
How did you test these changes?

[X] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[X] Are there any other steps that were used to validate these changes?

Open settings
Toggle the option

https://github.com/files-community/Files/assets/52143624/f9e3f7c6-522d-48e9-af53-4a72fc379151
Additional info
This is the stacktrace of the crash
2024-04-05 20:19:11.0801|Error|System.Runtime.InteropServices.COMException (0x80131509)
   at WinRT.ExceptionHelpers.<ThrowExceptionForHR>g__Throw|39_0(Int32 hr)
   at ABI.Files.App.Server.Database.ILayoutPreferencesDatabaseClassMethods.ResetAll(IObjectReference _obj, LayoutPreferencesFilterPredicate predicate)
   at Files.App.Server.Database.LayoutPreferencesDatabase.ResetAll(LayoutPreferencesFilterPredicate predicate)
   at Files.App.Helpers.LayoutPreferencesDatabaseManager.ResetAll(Func`2 predicate)
   at Files.App.ViewModels.Settings.LayoutViewModel.ResetLayoutPreferences()
   at Files.App.ViewModels.Settings.LayoutViewModel.set_SyncFolderPreferencesAcrossDirectories(Boolean value)
   at Files.App.Views.Settings.LayoutPage.LayoutPage_obj1_Bindings.UpdateTwoWay_15_IsOn()
   at Files.App.Views.Settings.LayoutPage.LayoutPage_obj1_Bindings.LayoutPage_obj1_BindingsTracking.<RegisterTwoWayListener_15>b__19_0(DependencyObject sender, DependencyProperty prop)
   at ABI.Microsoft.UI.Xaml.DependencyPropertyChangedCallback.Do_Abi_Invoke(IntPtr thisPtr, IntPtr sender, IntPtr dp)

@hez2010 Can you take a look?

I think this has occurred during debugging and clicking Continue multiple times fix this issue.

This occurs even when not in debugging. Removing the predicate would be sufficient here, but #14911 seems to cause this issue in the first place.

I think this has occurred during debugging and clicking Continue multiple times fix this issue.

No, I found the crash at the Preview version (the video shows the problem fixed)

This is caused by the predicate cannot be converted to Expression as the predicate contains invocations to clients.
Can you also remove the unused predicate in ApplyToAll as well?

Can you also remove the predicate parameter from ApplyToAll as well? Then I think we can remove the type LayoutPreferencesFilterPredicate as well (from both the code and Package.appxmanifest).

Done


Improve accessibility functions of Menus

Describe what feature you'd like. Pseudo-code, mockups, or screenshots of similar solutions are encouraged!
It would be nice to also support up and down key events in menus and having click events toggle the menu open and closed (currently the menu only opens when the menu button is clicked, but it does not close it again on a second click).
According to this page https://www.w3.org/WAI/ARIA/apg/patterns/menu/ , Tabs and Shift+Tabs should also close the menu.
I could work on this as well.
What type of pull request would this be?
Enhancement
Any links to similar examples or other references we should review?
Headless UI has a nice example.

Absolutely Benny. There's plenty to be done on the a11y front. I'd be glad to have folks help tackle these one by one or in a more dedicated sweep through the project. If you get a chance send a PR and I'll take look!

Great, I'll try to tackle this one for now and maybe have a look at some other components afterwards as well. Should I use a chore/* branch for this?

Nice! Yeah I'm on the discord channel as well. Will check it more often from now on :)

@bennymi awesome, same username? I'll keep an eye out for you. Don't be afraid to pop into the channel and say hi!


Unable to Install

It says opusfile package not found. Any idea which package to install? This is on Windows Subsystem Linux.
go build -o transcribe-server ./cmd/transcribe-server/main.go
# pkg-config --cflags  -- opus opus opus opusfile opus opusfile
Package opusfile was not found in the pkg-config search path.
Perhaps you should add the directory containing `opusfile.pc'
to the PKG_CONFIG_PATH environment variable
No package 'opusfile' found
Package opusfile was not found in the pkg-config search path.
Perhaps you should add the directory containing `opusfile.pc'
to the PKG_CONFIG_PATH environment variable
No package 'opusfile' found
pkg-config: exit status 1
Makefile:7: recipe for target 'default' failed
make: *** [default] Error 2

Yes - exactly the above.

@abhinavkulkarni I started using this instead: https://github.com/vin-ni/Google-Cloud-Speech-Node-Socket-Playground.git
You might find it useful.

I came across the same problem while building this repo in a container.
In my case, after
sudo apt install libopus-dev libopusfile-dev
building succeeded.

Thanks @JonghunBok  :) this worked for me - I have spent hours trying to get this working, I use this for running unit tests for Vector the Robot using the OSKR - (open source software development kit)
Thank you :)


fix potential panic with SpiderSubnet feature

For auto-created IPPool, I think there's no need to inherit the application matchLabel(podAffinity).
Because for IPAM or spiderpool-controller components, we just use the label to search the IPPool and no need podAffinity to do a filter.
Signed-off-by: Icarus9913<EMAIL_ADDRESS>What this PR does / why we need it:
fix bug
Which issue(s) this PR fixes:
https://github.com/spidernet-io/spiderpool/issues/1252

Spidernet needs 'spec.podAffinity' to identify whether an IPPool is a static pool and which workloads have affinity to it. An automatically created IPPool should be a static pool.


Load-balancer optional

This commit changes the default behaviour of kube-vip to use VIP clustering only.
The flag --lbEnable will default to false, which in turn will result in the kubernetes manifest generated by the kube-vip kubeadm init having a disabled load-balancer in the manifest.

/lgtm


New pipeline save/load features

An ideal feature but not sure if it is better for app or pipeline. A proper save/load can make the limbus serve as ONNX.
1. Save and Load
Requesting a feature to export/save and import/load a built Pipeline.
Format Candidates:

joblib, recommended by scikit-learn. https://scikit-learn.org/stable/model_persistence.html

MyPipeline().save("myapp.someformat")

Then one can load and use with
Load("myapp.someformat").run(1)

2. Input/Output Information
In order to integrate the exported app/pipeline to other applications, it is better to include the input and output information like pipeline.graph.input and pipeline.graph.output.
3. Accepts I/O
Accepting input and output for the built app/pipeline.
4. A flag with if packaging models into the output or not.
We support a very big xxx.pipeline file.
Or a file structure like:

XXX.pipeline

model_a.onnx
model_b.tensorrt

A Proposal
Maybe having Input and Output Component.
For example:
self.c1 = Input("c1", shape=(B,))
self.t1 = Input("t1", shape=(B, 3))
self.t2 = Input("t2", shape=(B, 3))
self.stack = stack("stack")
self.out = Output("o", shape=None)

self.c1 >> self.stack.inputs.dim
self.t1>> self.stack.inputs.tensors.select(0)
self.t2>> self.stack.inputs.tensors.select(1)
self.stack.outputs.out >> self.out

pipeline = Pipeline()
pipeline.add_nodes([c1, t1, t2, stack, show])

pipeline.save("mypipeline.pipeline")

# Adding some ONNX-ish APIs
pipeline.graph.input  # returns [c1, t1, t2] and their shapes
pipeline.graph.output  # returns [o] and its shape

out = pipeline.exec([torch.tensor([0.]), np.array([[1, 2, 3.]]), ...])

Format Candidates:

joblib, recommended by scikit-learn. https://scikit-learn.org/stable/model_persistence.html
MyPipeline().save("myapp.someformat")

That's an interesting feature. Is there any detail for how to implement this feature?

Format Candidates:

joblib, recommended by scikit-learn. https://scikit-learn.org/stable/model_persistence.html
MyPipeline().save("myapp.someformat")

That's an interesting feature. Is there any detail for how to implement this feature?

Not so complex if all the classes are serializable. It is more or less a pickle file only. Well, since maybe some model files are involved, we may also append those ONNX files into the pickle object.

i think @lferraz was somehow serializing already pipelines


Add Round to showcase

Adding Round, a design-focused medication reminder and pill tracking app built almost entirely in React.

@facebook-github-bot shipit


Inspect/Reproduce error on live

When open /map view on live, markers are not showing up, and the error in console log is
Error: iconUrl not set in Icon options (see the docs).
    at a.Icon.a.Class.extend._createIcon (http://test.loklak.net/js/main.js:6:14854)
    at a.Icon.a.Class.extend.createIcon (http://test.loklak.net/js/main.js:6:14690)
    at a.Marker.a.Class.extend._initIcon (http://test.loklak.net/js/main.js:6:17599)
    at a.Marker.a.Class.extend.onAdd (http://test.loklak.net/js/main.js:6:16555)
    at L.Marker.include.onAdd (http://test.loklak.net/js/main.js:1:29816)
    at a.Map.a.Class.extend._layerAdd (http://test.loklak.net/js/main.js:5:31975)
    at a.Map.a.Class.extend.addLayer (http://test.loklak.net/js/main.js:5:23479)
    at a.Marker.a.Class.extend.addTo (http://test.loklak.net/js/main.js:6:16722)
    at http://test.loklak.net/js/main.js:13:15743
    at Array.forEach (native)

According the debugging instructions link, it is implemented the right way at lines 65-70: https://github.com/loklak/loklak_webclient/blob/master/app/js/services/mapcreation.js
I wasn't able to find the bug on live...yet. Meanwhile, it works fine at local, and remote http://gofullstack.me:3001/map (the related PR was reviewed by multiple peers).
@aneeshd16, @sudheesh001, @prasht63 can you try to run master at local and your remote server to see if the markers are showing on map & you're not getting the error?

I was able to diagnose why markers are not showings on test.loklak.live. Somehow, at the end point, the problem is related from the result of the server. This is an example result from user servlet - a following.
On TEST.LOKLAK.NET
{
....
profile_image_url: "2838735506"
....
}

On my remote server & localhost
{
....
profile_image_url_https : "https://pbs.twimg.com/profile_images/517920769402105856/n22C6FsD_normal.jpeg",
profile_image_url: "http://pbs.twimg.com/profile_images/517920769402105856/n22C6FsD_normal.jpeg"
....
}

Shortly about debugging the initial error (backward):

 iconUrl not set in Icon options => Check the iconUrl value
It is undefined => Check the resource value
The field profile_image_url_https is not in the result from user servlet => Compare with local and other remotes
It happens only on the TEST.LOKLAK.NET server. Also profile_image_url is not used at the moment.

@Orbiter: can you give me some more info about the number value? Should I write separate code for the live server? If you want to inspect the values, you can do some curl on the server, or I also created a global variable on live, which you can just go to the dev console of test.loklak.net and inspect the variable topo.

The live server should produce the same result as your test server, the live server code is identical to the code in git.
You should not write separate code!
If you see that the result from loklak.org is different from your own server, then this is strange. Please drop the url which you used to get this result.

Actually you can test it straight from the browser, since user servlet is available without auth:

Live http://loklak.org/api/user.json?callback=angular.callbacks._1&followers=500&following=500&screen_name=phucdn2005
Same link for local:
http://localhost:9000/api/user.json?callback=angular.callbacks._1&followers=500&following=500&screen_name=phucdn2005

You can see the difference quickly by searching for profile_image_url

I can reproduce this. That is really strange. I just checked if this profile_image_url is overwritten anywhere, but is not. I cannot see a cause why these results are different. Right now I can just suggest that you test if the value from profile_image_url starts with htttp:// and if not, take the url from profile_image_url_https. I will continue looking for a cause, but right now I don't know.

I actually used profile_image_url_https. The field wasn't there yesterday, hence the error. But now it's there so we're good to go now.


Install k3OS: Allow to download cloud-config file self-sign certificates

Hello,
I'm using the ISO file for AMD64 and it seems, its not possible to get the cloud-config file via HTTPS, if the cert is not recognized.  Either I have to download the file with "curl -k", or use HTTP, instead of HTTPS.
It would be great, if there is an option to allow that.
cu denny

Hi @linuxmail,
Unless you want to build a custom ISO with your self-signed cert then curl -k is your best option.


The description of currentDir inside launch.vs.json is wrong

Inside the launch.vs.json schema reference, section Default properties, the description of the parameter currentDir  states:

Specifies the full directory path to the Build Target. The directory is detected automatically unless this parameter is set.

It is not clear to me what this means... However, it is very clear to me what happens if I change that value: the working directory of the exe is changed according to the value provided.
The description should state something like: "The directory where the program will run".
I feel however that I'm missing something: the parameter cwd seems to be what I was looking for (a way to change the working directory of my program), but it doesn't seem to do anything.
I'm using CMake with Visual Studio v17.5.3 on Windows, and I'm developing locally.

cwd is for Linux and currentDir for Windows. Not clear why different attributes are used.
Some attributes are not documented at all like symbolLoadInfo. This is documented for vscode but also works in vs. Both use the Visual Studio MI Debug Engine from what I understand.

Hi, the reason for the different attributes is that they are implementation-specific. As voltamund says, cwd is Linux, currentdir is Windows.
When I get some time, I'll take a look at updating the docs for attributes like symbolLoadInfo.

We have received more issues than we can manage at this time. We are closing issues based on their severity, how broadly applicable they are, and whether there is enough information provided to act on the issue. If you believe that this issue should not have been closed, please add a comment explaining why by 2/23/2024. We appreciate your contribution and encourage you to propose changes using public PRs, or submit feedback through our updated control at the top of the page, as we phase out Git Issues for this repo. For more information, please see Announcing a new way to give feedback on Microsoft Learn


URL for Integrations broken/wrong

Hi there,
Attempting to set this up (via deploying to Heroku) and having some fun. The URL for settings/integrations in the README seems invalid.
I'll probably file some more issues as I go along!

Thanks!


Node: let quinoa use new LTS version 20.9.0

Describe your changes

Fixes Issue

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.

@melloware


Repository "Configurations" class spams the log when using legacy config

Current Behavior
The Configurations class is used to support both the new and legacy vocabulary for repository configurations.
The currrent implementation of org.eclipse.rdf4j.model.util.Configurations.logDiscrepancyWarning unfortunately spams the log on level WARN when using legacy config
Expected Behavior
My expectation would be that the method warns if the new and old property have diverging / contradicting values.
Currently it logs a warning even if there is only an old value (i.e. only the old vocabulary is used, and the new one is emptY).
I suggest to include a condition like
if (preferred.isEmpty()) {
  return;
}

As a workaround, we have set the logging level for the Configurations class to Error to avoid the log spam
Steps To Reproduce
Parse a repository configuration with legacy vocabulary
Example output
2024-06-25 07:56:18,777 WARN [main] org.eclipse.rdf4j.model.util.Configurations - Discrepancy between use of the old and new config vocabulary.
2024-06-25 07:56:18,778 WARN [main] org.eclipse.rdf4j.model.util.Configurations - Discrepancy between use of the old and new config vocabulary.
2024-06-25 07:56:18,778 WARN [main] org.eclipse.rdf4j.model.util.Configurations - Discrepancy between use of the old and new config vocabulary.
2024-06-25 07:56:18,778 WARN [main] org.eclipse.rdf4j.model.util.Configurations - Discrepancy between use of the old and new config vocabulary.
2024-06-25 07:56:18,778 WARN [main] org.eclipse.rdf4j.model.util.Configurations - Discrepancy between use of the old and new config vocabulary.
2024-06-25 07:56:18,782 WARN [main] org.eclipse.rdf4j.model.util.Configurations - Discrepancy between use of the old and new config vocabulary.
2024-06-25 07:56:18,783 WARN [main] org.eclipse.rdf4j.model.util.Configurations - Discrepancy between use of the old and new config vocabulary.
2024-06-25 07:56:18,783 WARN [main] org.eclipse.rdf4j.model.util.Configurations - Discrepancy between use of the old and new config vocabulary.
2024-06-25 07:56:18,783 WARN [main] org.eclipse.rdf4j.model.util.Configurations - Discrepancy between use of the old and new config vocabulary.

Version
5.0.0
Are you interested in contributing a solution yourself?
None
Anything else?
Note that this is a minor issue only

The model vocabulary should automatically be updated to the new vocabulary and written back to disk. So these error messages should only happen once, the first time the model is parsed before it is upgraded?
The logging also isn't very useful when it doesn't log any info that would help to diagnose the issue.
We could use debug log level for when the difference is that the preferred is empty. Warn for when both preferred and legacy have values but they are different.
We should also fix: public static Set<Value> getPropertyValues(Model model, Resource subject, IRI property, IRI legacyProperty) {
We have tests in ConfigurationsTest that actually check logging, so those would need to be updated.

@aschwarte10 Do you want to fix this yourself?

@aschwarte10 Do you want to fix this yourself?

I can do, but would be able only in about two weeks (if that is sufficient). We are currently in release finalization phase which takes all my attention.
I like the suggestion to distinguish the log level based on preferredValue being empty.
Note that in our case it is spammy as we load repository configurations from file during our application startup, i.e. updating them is an explicit migration


pandas Dataframe with 1 row results in AssertionError during call to validate

Describe the bug
When calling validate for a SchemaModel on a pandas's data frame with only 1 row, an AssertionError is raised. This appears to be triggered when coerce=True is includex in the field definition, but seems like a simple fix as detailed below

[x] I have checked that this issue has not already been reported.
[x] I have confirmed this bug exists on the latest version of pandera.
[x] (optional) I have confirmed this bug exists on the master branch of pandera.

Note: Please read this guide detailing how to provide the necessary information for us to reproduce your bug.
Code Sample, a copy-pastable example
from pandera.typing import Series
import pandas as pd
import pandera as pa

class Model(pa.SchemaModel):
    x: Series[int] = pa.Field(coerce=True)

Model.validate(pd.DataFrame({"x": [None]}))

# Results in the following error
# ...
# File "/cluster/home/willems/MethodsDev/vcgt/garbnewenv/pandera-fix/lib/python3.9/site-packages/pandera/engines/numpy_engine.py", line 57, in coerce
#    failure_cases=utils.numpy_pandas_coerce_failure_cases(
#  File "/cluster/home/willems/MethodsDev/vcgt/garbnewenv/pandera-fix/lib/python3.9/site-# packages/pandera/engines/utils.py", line 88, in numpy_pandas_coerce_failure_cases
#    check_output = numpy_pandas_coercible(data_container, type_)
#  File "/cluster/home/willems/MethodsDev/vcgt/garbnewenv/pandera-fix/lib/python3.9/site-packages/pandera/engines/utils.py", line 33, in numpy_pandas_coercible
#    search_list = _bisect(series)
#   File "/cluster/home/willems/MethodsDev/vcgt/garbnewenv/pandera-fix/lib/python3.9/site- packages/pandera/engines/utils.py", line 20, in _bisect
#    assert (
# AssertionError: cannot bisect a pandas Series of length < 2

Expected behavior
The above validation should fail with a SchemaError instead of an AssertionError
Desktop (please complete the following information):
python 3.9.1
pandas             1.3.1
pandera            0.7.1
Additional context
Seems like the numpy_pandas_coercible() fn is missing a base case for a series that contains only 1 element on line 33:
# Current call which triggers the AssertionError as _bisect immediate checks that series.size >= 2
search_list = _bisect(series)

# Proposed fix
search_list =[[series.iloc[0]] if series.size == 1 else _bisect(series)

# I'm not sure if adding a check before line 33 for an empty series is also necessary in case this function is (mis)called on an empty series
if series.empty:
   return pd.Series(dtype="bool")

doh! nice catch

#626 should address this

Great! Feel free to close this whenever #626 is merged


Fix "The path is not of a legal form" during loading VFT solution

This fixes https://github.com/Microsoft/visualfsharp/issues/3006

@vasily-kirichenko merge with master?

Done.


Including copyright in resource RESTful API

We need to look at the possibility of adding  RESTful support for getting those copyright setting out of TRACS. Canvas does have corresponding mapping to each copyright choice in TRACS. But currently there is no info regarding copyright of a content in the RESTful API.

Done. Deployed.

Done. Deployed


Requests for the plugin (Rank upgrade, Buyer protection)

Hey,
I tested your plugin, and it worked really well.
I have a few suggestions, so your plugin gets even better!
First, I would love to see a feature, where you can upgrade your rank (If you already have one, then the price for the more expensive ones go down by the price of your own rank)
Another request would be a feature, that protects you from buying a cheaper rank then you already have.
Thanks for this great plugin!

Done in 0c51549


Fold output

When a player folds (or if more than 2 players functionality is implemented, all but one players fold), the hands should not be evaluated. The player that didn't fold should just take the pot and win the hand.

W18 OK (50 pts)

Solved W18


[GH Permissions] A team of compiler engineers

What you are trying to do...
We are working on this project for miscellaneous improvements and to enable more conversions in pytorch2.0_ttnn.  We have made a couple of pull requests and opened dozens of issues.  It would be great convenience to grant us writing permissions so that we can:

Label issues and PRs
Run CI tests to fix code proactively

Who needs permissions...
@jerrysky3 and me
Optional screenshot etc...

Hi! Working on this. Will update you tomorrow.

Closing this issue after both of us have joined the team


Remove replica scale from 2 to 1 for deployment

From 4.8 we don't need to scale the deployment because it already
support single node configuration and by default deployment replicas
set to 1.

merging only single commit.


Enhance and Structure the Contributor Section

Is there an existing issue for this?

[X] I have searched the existing issues

Issue Description

Updated contributor graph to use the Vaunt API for dynamic contributor visuals.
Structured the contributor section for better presentation.
Added responsive alignment for the contributor images.

Suggested Change

Reformat Contributor Section: Use a grid layout to make the section visually appealing and organized.
Add Images: Display contributor GitHub avatars with a consistent size.
Include Links: Add links to each contributor's GitHub profile directly.
Use Badges: Include badges that showcase contributions such as "Maintainer," "Top Contributor," or specific GitHub activity (stars, forks).
Group Contributors: Categorize contributors based on the type of contribution (e.g., code, documentation, design).
Include Automation: Consider using GitHub tools like all-contributors to automate the addition and maintenance of contributors.

Rationale
N/A
Urgency
High
Acknowledgements

[X] I'm a GSSOC'24-Extd contributor
[X] I'm a Hacktoberfest contributor
[X] I am willing to work on this issue (optional)

@rishicds kindly assign me the task

merge this and #343 into a single issue and close these two


Test that clippy::all is present in all crates

Signed-off-by: Mark Bestavros<EMAIL_ADDRESS>

@mbestavros I'm not quite sure why the lints aren't running...

It's been my experience that disappearing checks are a symptom of a syntax error in the yaml file; though that is often times accompanied by a github notification saying so

@connorkuehl It was indeed that... an indentation error, of all things. ¯_(ツ)_/¯

Having discussed this a bit more with @npmccallum on our daily standup, I agree a template is simpler. Closing this test for now.


Adds logging levels

Addresses #586

@ivotron can you help me with this build error ? It is passing the test here.
I decided to use colorlog which is essentially the logging module with color support; which I did include in setup.py.

Sorry for the delay! PR #563 will remove the pip-less method for installing popper.

@ivotron ready for review!
The log colors are controlled using
log_colors = {
    'DEBUG': 'bold_purple',
    'INFO': 'reset',
    'WARNING': 'bold_yellow',
    'ERROR': 'bold_red',
    'CRITICAL': 'bold_red,bg_white'
}

Hello @akashrajr1! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file cli/popper/utils.py:

Line 159:80: E501 line too long (83 > 79 characters)
Line 165:80: E501 line too long (85 > 79 characters)
Line 171:80: E501 line too long (81 > 79 characters)
Line 220:1: E302 expected 2 blank lines, found 1

@ivotron can you point me towards where this debug info is generated? It has debug color scheme but not the 'DEBUG' prefix

@akashrajr1 thinking a bit more about this feature: we can define a new level on a custom handler that corresponds to messages logged by actions. I'll reference the table with default logging levels from the official doc (reproducing below for convenience):

Level
Numeric value

CRITICAL
50

ERROR
40

WARNING
30

INFO
20

DEBUG
10

NOTSET
0

We can add a new ACTION_INFO level between DEBUG and INFO. This would be the level that runners use to log messages that are coming from actions, whereas the INFO level is used by popper to log messages that come from popper itself. For example, take the following sample output:
$ popper run
[popper] cloning actions from repositories
[install] docker pull node:11.6.0
[install] docker create node:11.6.0 npm install
[install] docker start
audited 195 packages in 2.498s
found 0 vulnerabilities

[test] docker pull node:11.6.0
[test] docker create node:11.6.0 npm test
[test] docker start

><EMAIL_ADDRESS>test /private/tmp/mypaper/github-actions-demo
> mocha ./tests --recursive

Listening on port 8080
  GET /
    ✓ should respond with hello world (48ms)

  1 passing (74ms)

[lint] docker build -t popper/cplee/github-actions-demo/jshint:c824a5a /private/tmp/mypaper/github-action
s-demo/./.github/actions/jshint
[lint] docker create popper/cplee/github-actions-demo/jshint:c824a5a
[lint] docker start
[branch-filter] docker build -t actions/bin /tmp/actions/github.com/actions/bin/./filter
[branch-filter] docker create actions/bin branch master
[branch-filter] docker start
refs/heads/master matches refs/heads/master
[deploy] docker build -t actions/bin /tmp/actions/github.com/actions/bin/./sh
[deploy] docker create actions/bin env
[deploy] docker start
Running 'env'...
POPPER_EVENT_NAME=push
HOSTNAME=1d490843e5ff
PHONY_SECRET=foo
HOME=/Users/ivo
POPPER_WORKSPACE=/private/tmp/mypaper/github-actions-demo
POPPER_SHA=c824a5a
POPPER_ACTOR=popper
POPPER_REF=refs/heads/master
Successfully ran 'env'

Workflow finished successfully.

messages between squared brackets are regular INFO, whereas the ones coming from the action are ACTION_INFO messages. With this, we can turn off messages that come from actions (when --quiet is given), but at the same time show popper messages

Sounds good @ivotron , will incorporate it

@ivotron the update is almost ready, but I can't figure from where the runners are generating their logs. Can you help me?

@ivotron just wanted to confirm, is this pu.info(b(l)) where we print our action info?

yeah, that's for the dockerrunner. In the case of the singularity runner, it is here.
For the docker runner, I already took out the code that was dealing with writing to log files. This can be simplified further by removing the conditional for when quiet and debug are given.
For singularity, we can simplify the logic of the code significantly, as we can remove conditionals that setup log files, as well as those that deal with quiet, verbose and debug.

This can be simplified further by removing the conditionals for when quiet and debug are given.

Already took care of this.
Fyi: The PR is updated with ACTION_INFO level and I am using log.action_info() for that.
And is possible to remove this condition? If we can then we don't actually need to propagate the quiet in Runners. I think debug flag after the last commit isn't used for any condition checking. So I will clean up the code by removing it. Essentialy debug and quiet will be used only once at the beginning to set the logging level.

And is possible to remove this condition?

yes, that can be removed

If we can then we don't actually need to propagate the quiet in Runners. I think debug flag after the last commit isn't used for any condition checking. So I will clean up the code by removing it. Essentialy debug and quiet will be used only once at the beginning to set the logging level.

yeah, that's the ideal, to cleanup the codebase so that the code just uses log.<level> and delegate that to the logging handler

Was the previous logfile code somehow connected to travis ci build logs? By removing it now I am not getting any output for test actions on the log.
I thought Travis CI was represented as stdout, but maybe it is stderr?

thanks you so much for your patience @akashrajr1 !


Refreshing webui page launches a new instance

Would be nice if it gave an option on every refresh if I would like to launch a new instance or not

@saifabid - thanks for the suggestion. for now, you can click on the logo at the top to go back to main page to launch another instance.


Set up Travis CI

Description
I would like to ensure that on every PR submission, the code is sent to travis CI and unit tests are run.

Can I take this?


Issue on split screen view, popup view and landscape view

In the "split screen" view and popup view two problems
a) Might need to not take the system navbar height.
b) While the banner ad is load in split view, it show the following error (tried with "leaderboard" ad type)
Error building request URL: Ad size will not fit on screen. a_w=728,  a_h=90,  s_w=384,  s_h=374

In landscape mode, no need to take the navbar height

A temporary solution is to disable those modes, by adding the following line on the AndroidManifest.xml file
<activity
    android:screenOrientation="portrait"
    android:resizeableActivity="false" <!-- Disables split-screen and popup view -->
</activity>

I don't want this change to bother other users, because it affects (UX), so maybe tomorrow I add a variable to cordova plugin.xml and capacitor, the default value remains true.

I found some new issues in android 15/16, and problems when using plugins cordova-plugin-statusbar or
cordova-plugin-fullscreen
if you have time you can test the above 2 problems

Is the temporary fix working or not? I have not checked it on Android 15 yet, but I will check it later.

I have tested the statusBar still cannot be displayed on android 15/16
the point is how can the plugin display statusBar on android 15/16 so as not to damage emiAdmobPlugin.styleBannerAd

I want to make an option how in android 15/16 the statusBar remains displayed.
like the only way to use hooks is to disable edge-to-edge node:tools = “remove”

I think it doesn't affect the capacitor project. Please confirm by the following image.

Am I correct?

Yes unfortunately I don't have an Android 15/16 phone, I can't see what's happening with banner ads on real devices.
but in the user reviews of my app on play store it says the black empty space above the banner is quite big 😁 on their device android 15/16.

Are you adding a black line above the banner ad? If yes, can I see your JavaScript code?
I am doing like this
	let blackLine = document.getElementById("bannerBottomWhiteSpace");

	if (blackLine)
		blackLine.remove()

	blackLine = document.createElement("div");
	blackLine.id = "bannerBottomWhiteSpace";
	blackLine.style.width = "100%";
	blackLine.style.height = `${extraHeightForNavigationBar+parseInt(bannerHeightPx)+parseInt(lineHeightPx)}px`;
	blackLine.style.background = _bannerTopSpaceColor;
	blackLine.style.position = "fixed";//"relative";
	blackLine.style.bottom = 0;//`${bannerHeightPx}px`; // Position above the banner
	blackLine.style.zIndex = "99999"; // Ensure it's above the banner
	document.body.appendChild(blackLine);

https://github.com/EMI-INDO/emi-indo-cordova-plugin-admob/releases/tag/v2.0.0
I'm off for the next 5 to 7 days.
No updates.
Have a good day.


Popularity Metric for Living Wage

I am a student of Sean Goggins. We needed to finished a metric and create a pull request back to the base repo.
Joshua Hickman

Hey @hickmanjv, thanks for submitting this to the project. Here are a few thoughts:

First, please follow these instructions to sign the DCO
The popularity element of this metric is implied in the definition. I think it's fair to focus on it being a singular (synthetic metric)
I believe you're offering a metric from the perspective of an individual looking to maximize their hiring potential. Though I also see funding potential, which is from the perspective of a corporation of venture capital firm. It'd be helpful to center on a single persona, or clarify when different personas can use the same metric.
Does this metric overlap with any others in the project?

There is an opportunity to either edit this toward that ends, or to consider this a part of another working group's questions. To be fair, it's hard to see that beyond the README since our focus areas are a work in progress.
I hope the feedback is helpful - let me know if you have questions and let me know how you'd like to proceed.

Thanks for looking it over Matt,
I’ve followed those instructions and keep getting rebase errors and I am very new to this process. I cannot get them to clear as to sign them off.
“””
First, rewinding head to replay your work on top of it...
Applying: Update organizational-project-skill-demand.md
Applying: Update Labor Investment (Heading Issue)
Applying: Update social-currency-metric-system.md
Applying: remove trailing whitespace
Applying: Update labor_investment.md
Applying: minor typo fix
Applying: Update social-currency-metric-system.md
Applying: Update organizational-project-skill-demand.md
Applying: move images
Using index info to reconstruct a base tree...
Falling back to patching base and 3-way merge...
error: Your local changes to the following files would be overwritten by merge:
focus-areas/ecosystem/scms-0-circle2.png
focus-areas/ecosystem/scms-1-dashboard.png
focus-areas/ecosystem/scms-2-tool-example.png
focus-areas/ecosystem/scms-3-expanded-comment.png
Please commit your changes or stash them before you merge.
Aborting
error: Failed to merge in the changes.
Patch failed at 0009 move images
hint: Use 'git am --show-current-patch' to see the failed patch
Resolve all conflicts manually, mark them as resolved with
"git add/rm <conflicted_files>", then run "git rebase --continue".
You can instead skip this commit: run "git rebase --skip".
To abort and get back to the state before "git rebase", run "git rebase --abort”.
“””
That is the error I get from the rebase.
I don’t know how to fix it from here. When I use ‘git am —show-current-patch’ I get:
“””
commit 1a4e808427deb2da6a7a61746b638dccff094f71
Author: Georg J.P. Link <linkgeorg<EMAIL_ADDRESS>Date:   Sun Jan 26 16:35:15 2020 +0100
move images

Signed-off-by: Georg J.P. Link <linkgeorg<EMAIL_ADDRESS>
diff --git a/focus-areas/ecosystem/scms-0-circle2.png b/focus-areas/ecosystem/images/scms-0-circle2.png
similarity index 100%
rename from focus-areas/ecosystem/scms-0-circle2.png
rename to focus-areas/ecosystem/images/scms-0-circle2.png
diff --git a/focus-areas/ecosystem/scms-1-dashboard.png b/focus-areas/ecosystem/images/scms-1-dashboard.png
similarity index 100%
rename from focus-areas/ecosystem/scms-1-dashboard.png
rename to focus-areas/ecosystem/images/scms-1-dashboard.png
diff --git a/focus-areas/ecosystem/scms-2-tool-example.png b/focus-areas/ecosystem/images/scms-2-tool-example.png
similarity index 100%
rename from focus-areas/ecosystem/scms-2-tool-example.png
rename to focus-areas/ecosystem/images/scms-2-tool-example.png
diff --git a/focus-areas/ecosystem/scms-3-expanded-comment.png b/focus-areas/ecosystem/images/scms-3-expanded-comment.png
similarity index 100%
rename from focus-areas/ecosystem/scms-3-expanded-comment.png
rename to focus-areas/ecosystem/images/scms-3-expanded-comment.png
“”"
Any help I can get to resolve this would be appreciated.
Yes, from attending the wg-value meeting today this metric would most definitely be a synthetic metric. As I was creating it and thinking through it, I did see how it could be used for individual developers and business investment. I was unsure how to present that inside the metric as this was my first attempt and was following the template like a lifeline.
I was getting a sense that this metric did overlap some others such as job opportunity, however, I didn’t see any of the data analytics in this fashion. I was coming at this metric from a business perspective. Companies have had centuries to flesh out their data analytic schemes and use those for better forecasting. I wanted to try and apply that thinking to a metric that would allow us to forecast the projects with the most potential.
Thank you very much for the feedback.
Josh
On May 7, 2020, at 11:39 AM, Matt Broberg <notifications<EMAIL_ADDRESS>wrote:
Hey @hickmanjvhttps://github.com/hickmanjv, thanks for submitting this to the project. Here are a few thoughts:

First, please follow these instructionshttps://github.com/chaoss/wg-value/pull/83/checks?check_run_id=628351623 to sign the DCO
The popularity element of this metric is implied in the definition. I think it's fair to focus on it being a singular (synthetic metric)
I believe you're offering a metric from the perspective of an individual looking to maximize their hiring potential. Though I also see funding potential, which is from the perspective of a corporation of venture capital firm. It'd be helpful to center on a single persona, or clarify when different personas can use the same metric.
Does this metric overlap with any others in the project?

There is an opportunity to either edit this toward that ends, or to consider this a part of another working group's questions. To be fair, it's hard to see that beyond the README since our focus areas are a work in progress.
I hope the feedback is helpful - let me know if you have questions and let me know how you'd like to proceed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/chaoss/wg-value/pull/83#issuecomment-625365609, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOLA7A22FGFZNYNS7JH4ZTTRQLP3DANCNFSM4MTLTZMA.

can we delete the .DS_store files?

Hey! I know you were on the meeting, but I'll add this here as well:
Thanks again for this metric proposal. Before we can add it in, we're taking the time to make sure we're answering the right questions with the right metrics. You can see more on how in #88 and join us on June 4th to continue that effort.

we'll continue to develop this metric in #104
Closing this PR. When we are done developing the metric, we'll create a new PR.


IsString instance for JSVal

Can we add the following instance to GHCJS base?
instance IsString JSVal where
  fromString = jsval . Data.JSString.pack

This will make it easier to use OverloadedStrings to put JSString into JSVal (eg. as the value of a property of JavaScript.Object).

Sounds good to me.


adding explicit NATS username/password env vars

the build seems to be failing?

@dbason it looks like you updated the drone file?

The drone update was just an adjustment to when we build the base images.
The build is currently failing on the e2e tests.  This is because they're currently pinned to the v0.1.2 versions of the microservices which don't include the new nats wrapper.  @sanjay920 @tybalex we can discuss a strategy for the new API in standup


Coverage Report: rules

Need a better way to quickly see what’s been tagged and what hasn’t been tagged.
Set up rules such as every table or view in this project/dataset should have these three tags.

Hi I see you're dealing with Python
I would love it if you could help me with my homework
I'm currently taking a pretty basic course in Python and the question demands it that way.
The tester, brings me a list of hydrogen carbon compounds:
list = [CH3, CH4, C2H6, C3H20, C2H12 [
Enter any index from 0 to 4, and the program selects a compound from the list. According to its index.
The compounds should adhere to the following formula:
CnHr
r = 2n + 2.
If it holds, some string is printed.
If not, another string is printed.
My problem is with strings where the numbers become double digits.
How do you resolve the issue?

Hi I see you're dealing with Python
I would love it if you could help me with my homework
I'm currently taking a pretty basic course in Python and the question demands it that way.
The tester, brings me a list of hydrogen carbon compounds:
list = [CH3, CH4, C2H6, C3H20, C2H12 [
Enter any index from 0 to 4, and the program selects a compound from the list. According to its index.
The compounds should adhere to the following formula:
CnHr
r = 2n + 2.
If it holds, some string is printed.
If not, another string is printed.
My problem is with strings where the numbers become double digits.
How do you resolve the issue?

Duplicate of ISSUE #13.


IPT public access: Failed

We have three IPT instances and in all the IPT public access appears as failed

The URL's for the IPT installations are:

https://patrimonio.ambiente.gob.ec/iptmae/
https://patrimonio.ambiente.gob.ec/iptmae_test/
https://patrimonio.ambiente.gob.ec/iptmae_permisos/

Someone has a similar problem?

The site appears down from here, and from https://downforeveryoneorjustme.com/
https://check-host.net/check-http?host=patrimonio.ambiente.gob.ec shows it working in the USA and Brazil, but nowhere else.  It could be an issue with your institution's firewall or network.

I just check the IPT and the server is up here and there is no firewall for the webpage.

Unfortunately the server is still not accessible from most of the world, including Denmark.  This is probably something your IT department or network team will need to investigate.
See https://check-host.net/check-http?host=patrimonio.ambiente.gob.ec or https://www.uptimia.com/website-availability-test/141e8f6f or https://www.site24x7.com/tools/check-website-availability.html

We don't have any block from here, we don't know what else to do.

I can access http://mail.ambiente.gob.ec, though it redirects to https://mail.ambiente.gob.ec which is blocked by a firewall.
I cannot access either http://patrimonio.ambiente.gob.ec or https://patrimonio.ambiente.gob.ec
This is nothing to do with the IPT, it is a network problem. You need to ask your IT department or network administrators; the people who control your IP range <IP_ADDRESS>/24, for help. That might be cnt.gob.ec.
GBIF's servers are in <IP_ADDRESS>/24, which is present on the internet as part of Denmark's academic network, AS1835

I just talk with my IT and they told me that they dont have any special firewall for the domain and it is published and propagated.

I dont know what else to do.

Have you shown them the public testing page I've been using, for an HTTP connection (port 80) https://check-host.net/check-http?host=patrimonio.ambiente.gob.ec ? It clearly has many red entries for your server.
I can connect to an Alfresco service running on http://patrimonio.ambiente.gob.ec:8080/, but I cannot connect to http://patrimonio.ambiente.gob.ec/ (port 80) or https://patrimonio.ambiente.gob.ec/ (port 443).
The public checker also reports being able to connect on port 8080: https://check-host.net/check-http?host=patrimonio.ambiente.gob.ec%3A80 but not port 80.

https://downforeveryoneorjustme.com/patrimonio.ambiente.gob.ec also reports the site is down.

Can you have a look to the SSL certificate? the only thing that change is that te ssl certificate was updated on November 5.

I cannot connect at all, so I don't see any certificate.
I'm sorry, but there is nothing more I can do. This is not a problem with the IPT, it is a problem with your institution's network, or your ISP, or a very strange setting that's on your server (TTL, net.ipv4.ip_default_ttl?).
Please show https://check-host.net/check-http?host=patrimonio.ambiente.gob.ec to your IT people and work with them to see why most locations show  "Connection timed out" errors.

I just take off the SSL from the webpage and it works normally. I'll try to see what happen whit the SSL next year.


Generated LICENSE file is misspelled

Should be LICENSE instead of LICENCE :)

Thanks, I'll fix that in an upcoming update, I am going to be migrating the generator to the very latest versio of phaser


Added support for a wider range of AK series Digital CPU coolers

Feature Description
Add support for a wider range of AK series Digital CPU coolers by implementing a list of product IDs in setup.sh. Future support for other coolers can be added to the same list.
The new setup.sh also require two arguments model and sensor. It will then modify the Python script based on the two arguments before copying the Python script to the /usr/bin directory.
This is tested with i7-12700K and AK500S cooler.

@hykelvinlee42  Thank you for this can you update the branch with main branch?

Hi @raghulkrishna, my branch is already in sync with the main branch.


Сordova

Is it possible to use with Сordova? If yes, tell me how best to do it

Hey @pabloid64! From my point if view, I believe it's possible, nevertheless I've never tried it. I've quickly investigated this problem and this approach seems good to me -  https://stackoverflow.com/questions/52718004/kotlin-for-a-cordova-ionic-based-plugin. I believe @Alexey-Verkhovsky might have a more detailed answer.

Hi! Since we don't have an iOS version of the library we haven't considered yet to build a cross-platform plugin.
But Cordova 9.0.0 fully supports Kotlin, so it's not a problem to build the plugin yourself.
The Kotlin plugin file should look like following:
package org.apache.cordova.plugin

import com.fingerprintjs.android.fingerprint.FingerprinterFactory
import com.fingerprintjs.android.fingerprint.Configuration
import org.apache.cordova.CallbackContext
import org.apache.cordova.CordovaPlugin
import org.json.JSONArray
import org.json.JSONException

class FingerprintAndroidCordovaPlugin : CordovaPlugin() {

    val fingerprinter = FingerprinterFactory.getInstance(
        cordova.getActivity().applicationContext,
        Configuration(version = 3)
    )

    @Throws(JSONException::class)
    fun execute(action: String, args: JSONArray, callbackContext: CallbackContext): Boolean {
        if (action == "fingerprint") {
            getFingerprint(callbackContext)
            return true
        }
        if (action == "deviceId") {
            getDeviceId(callbackContext)
            return true
        }

        return false
    }

    private fun getFingerprint(callbackContext: CallbackContext) {
        fingerprinter.getFingerprint { fingerprintResult ->
            callbackContext.success(fingerprintResult.fingerprint)
        }
    }

    private fun getDeviceId(callbackContext: CallbackContext) {
        fingerprinter.getDeviceId { result ->
            callbackContext.success(result.deviceId)
        }
    }

}

Also make sure that you imported fingerprint-android library in coresponding *.gradle files of the plugin.
You need to add a repository:
allprojects {
  repositories {
  ...
  maven { url 'https://jitpack.io' }
}}

and dependencies:
dependencies {
  implementation "org.jetbrains.kotlin:kotlin-stdlib:1.4.30"
  implementation "com.github.fingerprintjs:fingerprint-android:1.2"
}

After that in JS code will be available two actions: fingerprint and deviceId. Use the namespace you've set in plugin.xml
If there are some problems with building plugin - let us know, we'll try to help.

Hello again. Thank you for your responses. I tried making a plugin. It is currently in the repository https://github.com/pabloid64/cordova-plugin-device-fingerprint. The plugin was installed into the project, it is available in the JS code, but it returns undefined (when trying to assign the result of a function to a variable).
To the main project, I added:
<preference name="GradlePluginKotlinEnabled" value="true" />
<preference name="GradlePluginKotlinCodeStyle" value="official" />
<preference name="GradlePluginKotlinVersion" value="1.3.50" />
.....
<plugin name="cordova-plugin-device-fingerprint" spec="https://github.com/pabloid64/cordova-plugin-device-fingerprint#master" />
In JS code I use like this:
const device = fingerprint.fingerprint(function(msg) {           return msg         }, function(err) {           return err         });
I installed the dependencies you mentioned using build-extras.gradle, assuming that I will extend build.gradle in the project itself.
I am also sending a screenshot of the final assembly file structure.

I'm not a mobile developer, so it's hard for me to quickly figure this out and have to do some things intuitively.
If you have any thoughts why such a problem arises, I will be very grateful.

Hey! Sorry for the delay. I took a look, and the only thing I noticed is probably an incorrect android packagename in the plugin.xml file. FingerPrint.ktfile should be in a directory that corresponds to packagename. Packagename is org.apache.cordova.plugin, so the directory should be src/org/apache/cordova/plugin.
Try to change plugin.xml from:
 <platform name="android">
      <config-file target="res/xml/config.xml" parent="/*">
      <feature name="FingerPrint">
        <param name="android-package" value="fingerprint.FingerPrint"/>
      </feature>
    </config-file>

    <source-file src="src/FingerPrint.kt" target-dir="app/src/main/kotlin/fingerprint"/>
    <source-file src="src/build-extras.gradle" target-dir="."/>
  </platform>
</plugin>

to
 <platform name="android">
      <config-file target="res/xml/config.xml" parent="/*">
      <feature name="FingerPrint">
        <param name="android-package" value="org.apache.cordova.plugin"/>
      </feature>
    </config-file>

    <source-file src="src/org/apache/cordova/plugin/FingerPrint.kt" target-dir="app/src/main/src/org/apache/cordova/plugin"/>
    <source-file src="src/build-extras.gradle" target-dir="."/>
  </platform>
</plugin>

Also don't forget to put FingerPrint.kt to src/org/apache/cordova/plugin.

Thanks for the help. Unfortunately, I have not been able to solve this problem. I'll put it off until better times)


CLI Options duplicate names check #173

bootique/bootique#173

I started with your PR, but made significant changes:

moving check into application metadata builder
implemented a marginally faster dupes check
reformatted error message to specifically identifying the first found dupe option
added unit tests


Add(ByIndex(0))

I'm investigating using FLaUI instead of White and was looking for the equivalent of AndIndex in White
For Example in White I have
mainWindow.Get(SearchCriteria.ByControlType(ControlType.Edit).AndIndex(2));
What's the equivalent in FLaUI to the above line, to use And.Index(2)?

The .Index is not supported by the underlying technology (MS UIA) and therefore gives the wrong impression, that this might be efficient. That's why I currently do not support the Index directly.
You can of curse use the following:
mainWindow.FindAllDescendants(cf=>cf.ByControlType(ControlType.Edit))[2];

That would be exactly what White does. Another approach would be to use XPath which "can" be more efficient if you use a lot of indexers:
mainWindow.FindFirstByXPath("(//Edit)[2]");

Does this answer your question and can it be closed?


Importing huge playlist

Getting tracks from Spotify. Traceback (most recent call last):   File "/home/jacob/Desktop/spotify2tidal/importer.py", line 93, in <module>     run_conversion(sp, sys.argv[1], tidal)   File "/home/jacob/Desktop/spotify2tidal/importer.py", line 70, in run_conversion     tracks = load_tracks_from_playlist(sp, spotify_user, playlist['id'])   File "/home/jacob/Desktop/spotify2tidal/spotify_scanner.py", line 50, in load_tracks_from_playlist     songs.extend([   File "/home/jacob/Desktop/spotify2tidal/spotify_scanner.py", line 58, in <listcomp>     if 'isrc' in s['track']["external_ids"] TypeError: 'NoneType' object is not subscriptable
The link to the playlist is https://open.spotify.com/playlist/49hT6xCBA6PLOzgcaM8ZMD

For some reason, two songs in this playlist are not retrieved correctly from the API. Maybe because the tracks are not available anymore. This is the only information that I get:
{'added_at': '2020-12-22T00:44:35Z', 'added_by': {'external_urls': {'spotify': 'https://open.spotify.com/user/1177473688'}, 'href': 'https://api.spotify.com/v1/users/1177473688', 'id': '1177473688', 'type': 'user', 'uri': 'spotify:user:1177473688'}, 'is_local': False, 'primary_color': None, 'track': None, 'video_thumbnail': None}

{'added_at': '2021-06-10T14:06:46Z', 'added_by': {'external_urls': {'spotify': 'https://open.spotify.com/user/1177473688'}, 'href': 'https://api.spotify.com/v1/users/1177473688', 'id': '1177473688', 'type': 'user', 'uri': 'spotify:user:1177473688'}, 'is_local': False, 'primary_color': None, 'track': None, 'video_thumbnail': None}

I have pushed a commit so that the code does not stop in those cases. You can try to find the songs according to the date added and compare them with the new Tidal playlist to see which ones are missing.

Thanks its seems to be working now.
I think they are songs removed by spotify. One seems to be a russian song thats no longer on spotify and the other im not sure of.
Thank you very much for this program btw. I've had this playlist since i was around 14 and 6 years later im glad i can preserve it without having to pay money.


Add terraform

Looks fine, though you might want to simplify a little to:
RUN curl -L https://releases.hashicorp.com/terraform/0.10.7/terraform_0.10.7_linux_amd64.zip > /tmp/terraform.zip && \
    unzip /tmp/terraform.zip terraform -d /usr/local/bin && \
    rm /tmp/terraform.zip

ie, only extract the file terraform - and there's no need to set it as executable - it already is, and there's no need for the pipefail, as we're not piping (we're output redirecting).

Sounds good to me :) Done


Updates to fine-tuning spec

pending -> queued
introduce "validating_files" status
mark finished_at, n_epochs, and all error properties as required

@athyuttamre done!


fix update_lead crash

Thanks for fixing this. Do  you have any idea of the underlying issue that's causing this? We can't start adding band-aids everywhere.

The problem in this case is caused by not received any modelV2 message before radarState,which is not the same as the sensorEvents problem.

Ah, ok. Any reason why you're checking the raw size then instead of the list length?

I made an assumption here, as long as there is data, the length of getZ() and getX() are both TRAJECTORY_SIZE, otherwise I have to check the two lists returned by getZ and getX

In other places we usually check if the recv frame is non zero to see if the message was ever received.

the reason that sensorEvents reads an empty list is possibly that the number of connections exceeds the limit (NUM_READERS) :

Why do you think that could cause this? How many subscribers are there?

There are at least 4(or 5?) , it's likely to exceed the limit  if start more UI processes

When I was testing I only ran sensord an ui instances.
If the number of readers is exceed warning messages are printed. Also a disconnected reader wouldn't be able to read a message anymore, it would just try to reconnect.
When I saw the bug the capnproto message seemed valid and had the correct union set for the specific sensor and also the valid flag was true, however the list with measurements was empty.

OIC.I understood it wrong.


Are there plans to access the "list image row item" for ListTemplate?

I wasn't able to figure out a way to render items in ListTemplate in the same way you would if you use 'CPListImageRowItem'. Are there plans to bridge that ability in the future?

Absolutely. Waiting for iOS 15 specs for further development 😁

Sweet, cant wait!

Were you able to make time to implement this?

Hi, @birkir - Just wondering if this was part of the 2.4.0 release and / or have plans to implement this soon? I can see the feature tag was added on May 25th.
Absolutely love the package and can confirm that expo will be supporting this in their releases going forwards!

@janwiebe-jump implemented this in #177
👍

This will be included in next release


Create additional info blocks

You can make your resume less official by adding more entertaining info, like your hobbies and interest, or what do you like, some interactive content or joke, easter egg, etc.

Good idea to add facts about yourself


Type Information in AST

I'm currently assessing whether I can use quill as an interface for DSH.
To create some useful input for DSH by using quill's AST, I need the types of the queried columns.
For
quote {
  for {
    x <- query[Person]
  } yield x.name
}

quill creates currently this AST:
Map(
  Entity(Person,None,List())
, Ident(x)
, Property(Ident(x),name))

It would would be great if

either Entity holds the type of the table
or Property just the type of the column.

I don't know much about Reflection in Scala, but maybe some kind of reference to the class could be a good extension for the AST and solve that problem too(?).
I am looking forward to your thoughts on that.

@freddie-freeloader Would a string with the full name of the entity type be enough or is it better a Type?

With Type I could easily deduce the names and types of the arguments/columns.
Therefore I'd prefer Type to String.

@freddie-freeloader please re-open if you're still interested in this.