Can't get a simple example to work?

Hello,
I'm trying to setup a basic example but seem to be failing to get WebMock to actually stub the GET requests my example is trying to make.
I'm using Mac OS X (10.11.5) with a non-system Ruby version of 2.1.2 (set by rbenv).
My reduced test case looks like the following:
require "webmock"
require "faraday"
require "pry"

endpoint = "http://www.beepbeepbeepboop.com"

WebMock.stub_request(:any, endpoint).to_return(:body => "foobar")

uri = URI.parse(endpoint)
http = Net::HTTP.new(uri.host)
http.get(uri.request_uri)      # SocketError: getaddrinfo: nodename nor servname provided, or not known

f = Faraday.new(nil, nil)
f.get endpoint                 # Faraday::ConnectionFailed: getaddrinfo: nodename nor servname provided, or not known

p "done"

Note: I've tried swapping the order of require "webmock" and require "faraday"
but that made no difference to the outcome

You have stubbed "http://www.beepbeepbeepboop.com" with foobar response and you make requests to google.

require 'webmock' does not enable it since version 2.0.0
You have to add WebMock.enable!

Aha, there we go WebMock.enable! did the trick 👍  thank you

@bblimke Thanks very much. I'm really confused why this was needed, though.
This stub did not pass due to the stub not raising the exception as expected:
        before do
          stub_request(:post, url)
            .with(http_params)
            .to_raise(Faraday::ConnectionFailed)
        end

        it { is_expected.to raise_error(MyCustomError) }

But after adding WebMock.enable! to my rails_helper.rb it worked fine. What confuses me is that I use WebMock all over the place in my tests and it worked just fine there, so is this possibly a bug with WebMock ?
Either way, it's working so I'm happy, but am curious why it needed the explicit call to enable WebMock despite other stubs apparently working fine. Thanks a lot anyway !

@bobf I guess you require 'webmock/rspec' which invokes calls Webmock.enable!
Not sure about your single spec as it can be related to your project. If you are able to extract the code to some sample project to reproduce it, then I can have a look.


Package Mirror 64.0.4

Adds package.json to allow the Unity Package Manager to fetch from the repository directly.
Adds custom Unitystation changes.

looks good


qt6Packages.libqaccessibilityclient: 0.5.0 -> 0.6.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/libqaccessibilityclient/versions.
meta.description for qt6Packages.libqaccessibilityclient is: Accessibilty tools helper library, used e.g. by screen readers
meta.homepage for qt6Packages.libqaccessibilityclient is: https://github.com/KDE/libqaccessibilityclient
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.6.0 in filename of file in /nix/store/6p3c4kijrqbzw15vrg3irl5w2w1kbgj1-libqaccessibilityclient-0.6.0

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

72 total rebuild path(s)

72 package rebuild(s)

First fifty rebuilds by attrpath
breath-theme
colloid-kde
graphite-kde-theme
kde-rounded-corners
latte-dock
layan-kde
libsForQt5.applet-window-appmenu
libsForQt5.bismuth
libsForQt5.kde-cli-tools
libsForQt5.kdeplasma-addons
libsForQt5.kdev-php
libsForQt5.kdev-python
libsForQt5.kdevelop
libsForQt5.kdevelop-unwrapped
libsForQt5.khotkeys
libsForQt5.kinfocenter
libsForQt5.kmenuedit
libsForQt5.krohnkite
libsForQt5.kwin
libsForQt5.kwin-dynamic-workspaces
libsForQt5.kwin-tiling
libsForQt5.kzones
libsForQt5.libqaccessibilityclient
libsForQt5.parachute
libsForQt5.plasma-bigscreen
libsForQt5.plasma-browser-integration
libsForQt5.plasma-desktop
libsForQt5.plasma-disks
libsForQt5.plasma-mobile
libsForQt5.plasma-remotecontrollers
libsForQt5.plasma-workspace
libsForQt5.powerdevil
libsForQt5.systemsettings
libsForQt5.xdg-desktop-portal-kde
lightly-boehs
lightly-qt
nordic
plasma5Packages.applet-window-appmenu
plasma5Packages.bismuth
plasma5Packages.kde-cli-tools
plasma5Packages.kdeplasma-addons
plasma5Packages.kdev-php
plasma5Packages.kdev-python
plasma5Packages.kdevelop
plasma5Packages.kdevelop-unwrapped
plasma5Packages.khotkeys
plasma5Packages.kinfocenter
plasma5Packages.kmenuedit
plasma5Packages.krohnkite
plasma5Packages.kwin

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6p3c4kijrqbzw15vrg3irl5w2w1kbgj1-libqaccessibilityclient-0.6.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/e006bc17f87f3a6af15c5096aafc2df7ed8cc2c5#qt6Packages.libqaccessibilityclient

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6p3c4kijrqbzw15vrg3irl5w2w1kbgj1-libqaccessibilityclient-0.6.0
ls -la /nix/store/6p3c4kijrqbzw15vrg3irl5w2w1kbgj1-libqaccessibilityclient-0.6.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

43 packages built:

breath-theme
colloid-kde
graphite-kde-theme
kde-rounded-corners
latte-dock
layan-kde
libsForQt5.applet-window-appmenu
libsForQt5.bismuth
libsForQt5.kde-cli-tools
libsForQt5.kdeplasma-addons
libsForQt5.kdev-php
libsForQt5.kdev-python
libsForQt5.kdevelop
libsForQt5.kdevelop-unwrapped
libsForQt5.khotkeys
libsForQt5.kinfocenter
libsForQt5.kmenuedit
libsForQt5.krohnkite
libsForQt5.kwin
libsForQt5.kwin-dynamic-workspaces
libsForQt5.kwin-tiling
libsForQt5.kzones
libsForQt5.libqaccessibilityclient
libsForQt5.parachute
libsForQt5.plasma-bigscreen
libsForQt5.plasma-browser-integration
libsForQt5.plasma-desktop
libsForQt5.plasma-disks
libsForQt5.plasma-mobile
libsForQt5.plasma-remotecontrollers
libsForQt5.plasma-workspace
libsForQt5.powerdevil
libsForQt5.systemsettings
libsForQt5.xdg-desktop-portal-kde
lightly-boehs
lightly-qt
nordic
qogir-kde
qt6Packages.libqaccessibilityclient
sierra-breeze-enhanced
utterly-nord-plasma
wacomtablet
whitesur-kde

Maintainer pings
cc @Artturin for testing.

Duplicate of #292317.


Capture the output file name in the C++ extractor. [phab:D1373]

Created by fromberger at 2017-02-24 20:44:02:
Addresses an ancillary part of #1754.

jrtom accepted this revision at 2017-02-24 20:47:55

zarko commented inline (context lost during migration) at 2017-02-24 20:57:16:
Also consider -ofoo.

fromberger wrote at 2017-02-24 21:02:21:

Add a TODO about combined arguments.

fromberger commented inline (context lost during migration) at 2017-02-24 21:02:28:
Good point. I will leave a TODO for now, since that mostly occurs in manual configurations rather than build-generated command-lines.

zarko accepted this revision at 2017-02-24 21:02:46

fromberger closed via commit c8e6bcb28fc9f7b8b24ca0b8e079a41045564779 at 2017-02-24 21:34:21


MinGW/Cygwin: Remove checks for ancient gcc/binutils

The versions checked here are 20 years old. Also dllwrap
has started to emit a deprecation warning in the latest release
spamming the build logs.
First steps towards https://github.com/pypa/distutils/issues/34
Commits from: https://github.com/msys2-contrib/cpython-mingw/commit/8ec77b61a8c33a8096882be9edfe0b23c4c62e2c and https://github.com/msys2-contrib/cpython-mingw/commit/825cae18785111ece6ac216335c4b83fccd6034c
cc @lazka

What is the reason for this being a draft?

@lazka When can we expect to see that in MSYS2 builds? I'd try to find some time to test it with a non-GCC compiler that actually work with GCC options (where I think distutils's regex check on the version number previously stumbled over).

It already is in the stdlib version for some time now.

Thanks!


can you add a list of all mapped ports to see the port occupancy？

can you add a list of all mapped ports to see the port occupancy？

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


Optimize the basic CSS for the docs

I really like the project and wanted to help even if I'm not able to contribute to the server architecture. As I noticed that the docs look a little bit "outdated" I decided to take a closer look at it.
This pull requests optimized the CSS styles for the docs. I will open another pull request later where I corrected some spelling and other smaller problems.

Updates Bootstrap styles to the latest version
Increases the base font size to 18px (See this article)
Unifies the fonts across all docs
Switches to a modern theme for Code Prettify

Here are two example screenshots:

--

thanks!


Add support for Node.js PATH variable based off of ~/.npmrc

Sets $PATH based off of the Node.js automatic path detection. This pull request also fixes the mode for luarocks_aliases.sh, removing the executability of the file (It's not needed).

I think that belongs in dotfiles.
In particular, that would avoid auto-creating a dotfile withing a user's homedir.

We all agreed that this belongs in shell-etc. If this gets merged, what happens with the people who don't have a ~/.npmrc? They'll have the system setup but not the local one.

@ChickenNuggers Users who want to use npm would indeed have to update their dotfiles.
I don't think that's much of an issue, whereas silently modifying/creating dotfiles for users is:

It means that users who indeed use the git versionning for their dotfiles get screwed over (they silently get a new file that won't get noticed/commited, thanks to .gitignore).
It makes it impossible, in the future, to change that setting: how do you know whether the .npmrc file was created for the user (and thus should not be touched) or not?
This is confusing/unexpected behaviour, and I thought providing users with a somewhat-standard environment is a thing.

Also, the mode change should be in a different commit  ;-)

I'll add a default .npmrc but the path setting will stay in /etc

Works for me

hey i forgot about this
ima fix it up a bit


To Do list

[x] Add Hierarchical Deterministic key support for priv, xpub --> generation of electrum sv and handcash wallet public / private key pairs (derivation path m/0')
[ ] Create a separate HandCash api powered by bitsv in a separate repo.
[ ] Create functions for posting of images / media etc. to blockchain (e.g. bitpaste compatible way)
[x] Add in BitIndex xpub related queries
[ ] Add final BitIndex functions for get used addresses and getting next address
[x] Fix tests
[ ] Fix testnet
[x] Tidy up final re-branding and documentation
[x] Add "confirmations" field to unspents so 0-conf balance versus 1-conf etc. balance can be calculated (may need to contact BitIndex)

Edit 12/04/19 Please comment below with a wishlist of features you want added and I will endeavour to provide! Or recruit help! Donations gratefully received!
Or if any of the above features are important to you... let me know so I can gauge if my time will actually be well spent there.

It would be great if there could be regtest support.

Tests seem to be passing at the moment.

@joshua-s It does have a test module. Currently at 87% coverage. Click on the little button on the ReadMe page that says "coverage 87%"

Thanks for the info. I was actually referring to "Regtest" as in the local testing environment for BSV.
If I understand that properly, it will allow me to develop blockchain applications without paying any fees and without publishing the transactions on a public chain. I suppose testnet support would work as well, but with the caveat of making my dev data public.
If there is something I am missing, please advise.

@teran-mckinney @AustEcon Thanks for the info. I was actually referring to "Regtest" as in the local testing environment for BSV.
If I understand that properly, it will allow me to develop blockchain applications without paying any fees and without publishing the transactions on a public chain. I suppose testnet support would work as well, but with the caveat of making my dev data public.
If there is something I am missing, please advise.

Ah! Okay. I wasn't aware of that. Thanks.
I plan on connecting up testnet as soon as I can. (Or if someone else submits a PR).
I imagine "RegTest" should be very similar to implement in theory?


Incorrect automine label

Am I getting this wrong or the "automine" label here is reversed? if I enable automine it shows "on demand". should be the other way around, right?

That button is to turn off automining.
This whole screen needs a bit of polishing (still prototype). Will add another issue for this!

that doesn't seem to be current behaviour.
When I do have automine (as in, I see the block number updating), this is the UI state:

Clicking "On" makes it stop, reaching the state on the 1st screenshot. button says "Off", which seems correct, but "Mining Status" seems wrong

That's interval mining in your screenshot, no?

hmm sorry. I guess it's my jet lag still kicking in 😅
I was reading automining and registering it as being interval (the name kind of allows for that interpretation I guess)


Update running-code-scanning-in-your-ci-system.md

Fixes #168 with the proposed changes.

Why:

What's being changed:

Check off the following:

[ ] All of the tests are passing.
[ ] I have reviewed my changes in staging.
[ ] For content changes, I have reviewed the localization checklist
[ ] For content changes, I have reviewed the Content style guide for GitHub Docs.

Thank you so much for this fix @casals! I'll leave this to the @github/docs-content-dsp team to review 💖


[tree-view-example]Is there a way to give same icon as the activated iconTheme?

Let's take the FTP Explorer for instance, the resources being displayed are folder, .js, .gz, .html, .ico, .json....
I have an iconTheme activated, so i want to display the corresponding icon in FTP Explorer depends on the resource "type", is it possible?

Yes this is being considered. Please follow the item https://github.com/Microsoft/vscode/issues/27823

@sandy081  Thanks for the information


App fail to respond when running in MacOS with M3 Pro (localhost:8000 failing)

Describe the bug
In MacBook with M3 Pro Chip, when I try to run an application at <IP_ADDRESS>:8000, I get a "Could not reach the server" error.
Upon digging the network logs, I found out the call http://localhost:8000/ws/socket.io/?EIO=4&transport=polling&t=Oy_YIh2 is failing. I suspect the way the socket connection is handled in the chainlit application has to do something with this issue. Any leads to resolve this issue is appreciated?
Desktop (please complete the following information):

OS: MacOS - Apple M3 Pro (2023)
Browser: Chrome 124.0.6367.119

Additional context

The application I am trying to run is build with chainlit for a GenAI use-case.
I cross verified the same application with windows platform and it is working properly without any issue.

Figured out the issue. It is not related to chainlit or platform.
Resolved by setting the following environment variables properly, since the application is running behind the corporate proxy.
export HTTPS_PROXY=
export HTTP_PROXY=
export https_proxy=
export http_proxy=
export no_proxy=
export NO_PROXY=


Data lake storage support phase2

Introduce new end point dfs and added new test cases for it
Updated visual studio extension file and added binary azurite-datalake,...
Updated readme, swagger definition changes
Updated change log
Added dfs test pipelines to azure-pipelines

Note: based on https://github.com/Azure/Azurite/pull/1933 so you can either merge this directly or merge phase 1 first then this later on

We have added a wiki for our requirements and general expectations of PRs that add new ADLS Gen2 to Azurite.
https://github.com/Azure/Azurite/wiki/ADLS-Gen2-Implementation-Guidance
Azurite welcome contribution!
If you would like help to implement ADLS Gen2 in Azurite, please first read the wiki and follow it to design/implement ADLS Gen2 in Azurite (better review the detail design with us first), to get a smooth PR review / merge.


Pull request alert types

User Story Description
As a Minder rule author
I want to write a rule that comments on a pull request if there is an evaluation failure
so that I can alert the PR author in the environment they are already using
Background
We already have existing ruletypes that comment on pull requests (e.g trusty and OSV), but there is nothing in the ruletype definition that explicitly states that commenting will be the action taken if the PR is not compliant.
Furthermore, the logic to comment on a PR is internal to Minder and not available to be reused by external authors of new ruletypes.
Acceptance Criteria

Given that I'm a Minder user
and I've written a rule with the alert type pull_request_comment
and the rule is part of a profile on my project
when the rule evaluation fails
then Minder comments on my pull request
Given I have a rule in a profile with the alert type pull_request_comment
and I've defined the review_message in my alert as "This PR has issues"
then Minder comments on my pull request with the message "This PR has issues"

Additional Resources
No response
Tasks

[x] https://github.com/mindersec/minder/issues/5054
[x] https://github.com/mindersec/minder/issues/5117
[x] https://github.com/mindersec/minder/issues/5120

Hey team! Please add your planning poker estimate with Zenhub @blkt @JAORMX

Closing this as complete. The PR alert type is under the feature flag pr_comment_alert.
Here is an example rule definition using the new alert type:
def:
  in_entity: pull_request
  param_schema:
    properties: {}
  rule_schema: {}
  ingest:
    type: diff
    diff:
      type: full
  eval:
    type: rego
    rego:
      type: constraints
      violation_format: json
      def: |
        package minder

        import future.keywords.in
        import future.keywords.if

        violations[{"msg": msg}] if {
          # Walk all files in the repo
          files_in_repo := input.ingested.files

          some current_file in files_in_repo

          msg := sprintf("File found: %s", [current_file.name])
        }
  alert:
    type: pull_request_comment
    pull_request_comment:
      review_message: |
        {{- range .EvalResultOutput }}
        - {{ .msg }}
        {{- end }}


UI Highlight gets dislocated when shop is closed

Trying to highlight an item while the shop is closed, or closing the shop after an item is highlighted will cause the highlight to be placed incorrectly in the HUD.
The UI Highlight should get an optional parameter: isShopItem, which is a boolean.
When set to true (which you usually would with items located in the shop), panorama will check if the item is shown.

If it is: panorama will constantly check if the shop is open (can be done easily via panorama API Game.IsShopOpen()). If the shop is closed, the highlight panel becomes invisible until the shop is opened again. This repeats constantly until the highlight panel is removed.

If it is not: panorama will constantly check if the shop is open. When it is, it will check again for the path of the item, and then match the element's properties to it. It then behaves the same as above.

The general goal is to make the highlight panel only visible when the shop's open and the panel is in place.

Fixed in #239 (I think)


Can`t find Laser Drill dimmension blacklist.

I looked at config, bit can`t find Blacklist for Laser drill. You sayed you will add it in Ver. 0.8.4.

I apologise, I actually forgot to implement it, outside of mod dev, my schedule is absolutly hectic right now.

implemented in build 0.8.5


Correction for raw affiliation CIRAD, UMR QUALISUD, F-34398 Montpellier, France.

Correction needed for raw affiliation CIRAD, UMR QUALISUD, F-34398 Montpellier, France.
raw_affiliation_name: CIRAD, UMR QUALISUD, F-34398 Montpellier, France.
new_rors: 05kpkpg04;01nfvkq89
previous_rors: 05kpkpg04
works_examples: W3124895353;W4245113100
contact: 1291cd7b67854e350b9a1369f751622c:65aea1c9378081a966781ebe186900edf8 @ supagro.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


Trendline

I'm submitting a...

[ ] bug
[ ] feature
[ ] chore

What is the current behavior
It says in readme that it has option for Trendline but i cant see it in documentation
What is the expected behavior
Please tell us about your environment

Version: 1.1.0
Browser: [all | Chrome XX | Firefox XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari ]

Other information

Is there any follow-up on this?

Before at all Trendlines works on react-financial-charts, just in case if you doubt it.
The entire project has a lack of documentation.
I reccomend you the next steps to infer how to use the functionality:

Examples React Financial Library to try to infer it
The project react-financial-charts is a typescript refactor of react-stockcharts rrag/react-stockcharts, try to use this examples too, in this case i could undertend trendlines thanks those examples
Reverse engineering
Good luck

I see that in TrendLine.tsx. The function seems not implemented yet.

I see that in TrendLine.tsx. The function onSelect seems not implemented yet.

Thats right, i remember now.
I insist, i have used the react-stockcharts example like i told you, the react-stockcharts way use a react selector component that will able to you to select different components, trendlines, prices, etc.
Let me know if you find it


tab open malfunction

Describe the bug
Fail to open website in HTML with attribute <target="_blank">
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):

Device: [e.g. OnePlus 7]
Android Version : [e.g. A11 ]
Kiwi Browser Version [ e.g. 90.0.4577.17 ]

Additional context
Add any other context about the problem here.

Will this test open a google new tab in your browser?
https://codepen.io/scheibo/pen/rWvQYK

Yes, it's ok

then it's generally working, du yo uhave an example where it's not working?

Thanks. It occurrs when I use local homepage in the form of file:///storage/emulated/0/Kivi/index.html, which contains code target="_blank", and all other browsers function formally, including kiwi early versions.

@kiwibrowser  do you think starget="_blank" should work in lokal file as homepage or is it intended?

yes，I have used it for nearly 5 years. In new kiwi version, only Google  and twitter can be openned, others fails. When l cancel the 'target="_blank"' in HTML such as { CSDNITcommunity }, everything is ok.

You can test it in this way:
1, download document in zip  in the https://github.com/Jackie1123/aNavigation
2, unzip the resources and put in a folder Kiwi at internal storage.
3, set up homepage in form of file:///storage/emulated/0/Kivi/index.html
4, In new homepage  you can find you cannot open other tabs except for google and twitter.

I'm really confused that Google and twitter can be openned, and others cannot.

https://github.com/kiwibrowser/src.next/blob/fafdddc2bb152c2799ec9f093df51ec0551cc14b/third_party/blink/renderer/core/page/create_window.cc#L324
Have to loosen this rule to allow more generic way new windows. This is heuristic to prevent hardcoding thousands of popup domains

https://github.com/kiwibrowser/src.next/blob/fafdddc2bb152c2799ec9f093df51ec0551cc14b/third_party/blink/renderer/core/page/create_window.cc#L324
Have to loosen this rule to allow more generic way new windows. This is heuristic to prevent hardcoding thousands of popup domains

It seems that it's unnecessary to do so, ublock origin can do more effective and efficient than we expect in such job.


DFR does not work since DCS 2.9

Since the latest update of DCS today the DFR ingame does not work. It does not track my eyes, but it is more like Fixed Foveated Rendering. I have installed the latest version of Quad Views Foveated and also the latest version of PimaxXR, OpenXR Toolkit, and the QVF Companion App.
My Headset is the Pimax Crystal.
I did everything like in this Video https://www.youtube.com/watch?v=A6g58RVPiUc. But it keeps fixed.
Maybe I have some settings wrong or is there an actual problem with DCS 2.9?
Thanks

see workaround here: https://forum.dcs.world/topic/333666-dynamic-foveated-rendering-everything-in-one-page/?do=findComment&comment=5306921

Thanks that helped! :)


ES Module loading with absolute path fails on windows

Frodo CLI version
Version: v0.23.0
Platform: Windows 10 64-bit
Node: v16.16.0
What steps will reproduce the bug?
After installed frodo, input any frodo command e.g. 'frodo help' 'frodo -v'
Describe the issue
Error [ERR_UNSUPPORTED_ESM_URL_SCHEME]: Only URLs with a scheme in: file, data are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'c:'

Hi @cmcneilniu , did you install the frodo binary or the NPM version?
The NPM version can be a bit tricky at times and honestly I never used it on Windows. The binary (.exe) wraps it all into one nice package and you should never run into any of these types of issues.
In case you must run the NPM version, I will try to help you figure out what's happening here. I'm currently traveling but will look into it as soon as I can.

@cmcneilniu would you like me to look further into this or did you find a solution?

Hi @vscheuber, yes please look further into this. I have to use NPM version. From my understanding, update the absolute path could be a fix.

+1 on this one. Can't use NPM version in git bash because of this bug. Admins lock things down so that binaries can't be installed (deletes new binaries that show up in filesystem too) without approval - npm and node are approved of course.

@cmcneilniu and @jgdtech I have not been able to reproduce this in the latest 1.x and 2.x versions. I never saw this issue so I am not sure if my env is immune to it and it's still lurking out there or has been addressed? We have made several changes that could have resolved this for Windows but I really don't know for sure. Please re-open if this is still an issue for you.


ROX-19636: Fix bulk suppress/reobserve CVEs in VM 1.0

Description
Problem to be solved (explanation from bug ticket investigation):
An incorrect request is sent from the frontend to the backend. The /v1/suppress and /v1/unsuppress APIs expect CVE name not the ACS ID of the CVE. The bulk operation seems to use database ID instead of CVE name.
Solution in this PR:
The container where the handler for globally suppressing or re-observing CVEs only has access to the list of selected rows, which are the full IDs of the CVEs. (The actual CVE data for the rows is retrieved in a child component; this is a legacy of the infinite nesting nature of pages in VM 1.0.)
Therefore, the simplest solution to get back to the CVE names is to take the array of selected CVE IDs, split them on the separator ("#"), and take the first part, which is the CVE name.
Note that this reveals the somewhat incongruous edge case that if you have selected more than one CVE by its unique ID that start with the same name, then all the CVEs that start with that ID will be suppress or re-observed.  "Sunt lacrimae rerum / et mentem mortalia tangunt"
Checklist

[ ] Investigated and inspected CI test results

Testing Performed
Here I tell how I validated my change
Manual testing
Select some CVEs

Choose to "Defer and Approve" those CVEs for some duration

See the success message

Select some CVEs

Choose to "Reobserve" those CVEs for some duration

See the success message

Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

/test gke-ui-e2e-tests

/test ocp-4-13-ui-e2e-tests


included heatmaps + minor fixes

Codecov Report

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

:exclamation: Current head b85ec87 differs from pull request most recent head 2b03142. Consider uploading reports for the commit 2b03142 to get more accurate results

@@          Coverage Diff          @@
##             dev      #5   +/-   ##
=====================================
  Coverage       ?   0.20%
=====================================
  Files          ?       8
  Lines          ?     954
  Branches       ?       0
=====================================
  Hits           ?       2
  Misses         ?     952
  Partials       ?       0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Evaluator Review: FACINGS NFT Collection Manager Stage 2 - Milestone 1 delivery

Review, approve, or deny https://github.com/eosnetworkfoundation/grant-milestones/pull/85

Closed due to creating https://github.com/eosnetworkfoundation/grant-milestones/issues/89 in wrong repo. Issue replaced with https://github.com/eosnetworkfoundation/grant-framework/issues/203

Closed https://github.com/eosnetworkfoundation/grant-framework/issues/203 as unplanned due to issue in framework repo. Re-opened https://github.com/eosnetworkfoundation/grant-milestones/issues/89 as it's in milestone repo.


Setup Kankan for react redux group project

We are two Micronauts in this project: @kazaneza and @rachaouldbabaali
Here is the link for the Kanban:
https://github.com/rachaouldbabaali/React-Redux-GP/projects/2

Hi Team,
Good job so far! 👍🏻
There are some issues that you still need to work on to go to the next project but you are almost there!

Required Changes ♻

[ ] As per this requirement, kindly make sure all cards are fairly distributed between the team members.

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Thank you @ShadyShawkat, problem fixed 👍🏻

Hello @Team    😃 🥳 🎉 🎈
Great Job 😄 👏 You've done it. Your project is complete!  There is nothing else to say  👍🏻 ✅
To Highlight 💯

Descriptive issue👍
Good implementation of the required changes 👍

Congratulations! 🎉

Cheers, and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Thank you @divinecharlotte 😃


IL2BOS Horizontal and vertical FOV does not match with HDK2/steamOSVR

Description of issue
In Il2 BoS the horizontal and vertical FOV are not matching, Jauges are not circular and head rotation introduce tearing instead of view rotation. The game is working well in VR with Vive and Rift.
My config is working without trouble with DCS world and war thunder.
I opened a ticket at Il2 support site, they answered that the problem is on headset provider side...
Same problem with extended or direct mode, and a new empty steamvr.setting file. Tested with beta and no beta steamVR version. Tested with 0.7.0, 0.8.0 AIO installer and the build 356 of OSVR core.
Other people have posted the same problem on Il2 Bos forums.
Steps to reproduce the problem
How can the problem be reproduced?
just use Il2 Bos and a HDK2 !
System configuration
Please specify any relevant system configuration information.

Graphics card(s): gtx 1070
Operating system: win 10 64b
Version of SteamVR: latest beta version
Version of SteamVR-OSVR: 0.8.0 installer.

Attached files
Please attach the following files:

The osvr_server configuration file.
I added the default 0.8.0 config file that I use now
osvr_config_HDK_2_default.zip

The vr*.txt files from the Steam logs directory.
I added the report of steamVr, a picture of in game steamVR miror illustrating the FOV problem.
OSVR_HDK2_PB.zip

The SteamVR-OSVR driver gets its FOV information from the OSVR server.
If you want to try adjusting the FOV values, you may do so in the displays/OSVR_HDK_2_0.json file.
Documentation for the FOV parameters in that file may be found here.

I tried different FOV settings in the OSVR_HDK_2_0.json file but did not manage to change the problem. The The view was still deformed, but in different way...

On forum, it looks like Pimax users had the same problem, but also that the last "piplay" version (software suite of pimax) fixed them...

Is there something that can be done ? This game is still unplayable with the HDK2...

oops, I closed by mistake this issue !

same as 118


make nstime support float eg. nstime=1488361308.424999936

support "nstime" Float Type
e.g.  nstime=1488361308.424999936

I think this patch is not needed because fluentd's event time should be Integer(v0.12 compat) or EventTime(v0.14 or later for nanosecond).
Client should not send Float type.
If fluent-logger-node sends Float type unexpectedly, fluent-logger-node should be fixed.


Week 2: Network Configuration (Re) Routing

Topics: Ensure that the compute node routes its traffic through the head node for improved control and monitoring. The compute node's next hop will be configured to point to the head node.

Tasks:

Configure Network Routing on Compute Node:

Modify the route table of the compute node so that its next hop for all outbound traffic is the head node.
Ensure that the compute node does not communicate with external networks directly but routes its traffic through the head node.

Test the Routing Configuration:

Ensure that the compute node cannot reach the internet or other internal/ external resources directly.
Verify that all traffic from the compute node goes through the head node by running network diagnostics like traceroute or netstat.

Enable IP Forwarding on the Head Node:

On the head node, enable IP forwarding so that it can route traffic between the compute node and the wider network (if needed for further configurations or internet access).
Update the network configuration to persist after a reboot (e.g., using sysctl on Linux).

Apply Firewall Rules (Preliminary):

Restrict traffic between the head node and compute node to necessary ports (e.g., SSH, web ports).
Block any unintended external traffic on the compute node until routing through the head node is confirmed.

Deliverable:

Configuration files showing updated route tables and network configurations.
Evidence of successful routing, such as output from traceroute and tests confirming all traffic goes through the head node.

Hi Team @lisabeeaan @thinker-bell @JCheney20 @Ghost-3342
As week 2 is ending, I am just checking in under week 2 task, hopefully your exams are going well. Please don't hesitate to ask any questions should you have any
Remember to @ me when you need me to attend to something so that I can get a notification and I will do the same.
Good Luck

Image 1 - updated route table

Image 2 - updated network config showing that the default route is via the head node at ipaddr: <IP_ADDRESS>

Image 3 - traceroute to google.com showing that the head node ip is being used as the gateway
@zsanda1, Hi please see above the deliverables for week 2, sorry for the delay on that, will also being uploading the deliverables for week 3 shortly.

Hi Team @lisabeeaan @thinker-bell @JCheney20 @Ghost-3342
Well done team for managing to get this far, we understand that it is exam times and I hope they are going well so far.
Can you please screenshot and send me the whole trace route to google output from your compute node.
I would also like to apologies for not adding IP forwarding evidence as a deliverable. Can you also please show me evidence that show IP forwarding is enabled ,and show your firewall rules.

Hi @zsanda1
No problem at all.
Here is an updated screenshot of the complete trace route to google

Then for the evidence of IP forwarding that also persists through reboot on the head node

Finally for the firewall rules, we used nftables and our table is seen below: is also updated for the requirements for Week 3:


Service Fabric Open Source Roadmap

Hi All,
I am opening this thread hoping I can aggregate some useful information about the open source roadmap for Service Fabric.
SF team has been advertising the plans for open sourcing service fabric for a while, we've got access to the source code and now we can see how it works behind the scenes. But it is clear that many things are missing in this journey, just the access to the source code does not add much benefit to the platform.
My doubt is, when Service Fabric will really adopt the open source culture?
What mean is, bring the full development outside the Microsoft silos, create competence teams for different platform features, accept the public involvement to discuss feature proposal, accept community support and listen for users feedback.
Today the only thing we see is a new release coming and a dump of the source published here, without any clear visibility on what is going on, what to expect and without any confidence on the future of the platform.
Microsoft has many teams already doing open source, I think the knowledge shouldn't be an issue here, .Net Core is an amazing example, the people involved on Kubernetes are also doing a really nice job.  I understand that many process might be tied to Microsoft internal tool, but is clearly visible that Open Source is not one of the priorities.
Analyzing the SF Mesh preview releases, it is painful to see the so many basic issues taking ages to be fixed, some of them planned to be released late next year.
I really love the platform, but is a shame see it being left behind and probably having the same destiny as other platforms becoming another Kubernetes wannabe implementation like Docker Swarm, Mesos, OpenShift and other dozens.

We are working hard to go the last mile in moving our development to the public repo. Early in the new year we will post an overview of the work remaining and the timeline to get us there. Rest assured we are committed to get this done.

Any updates?


add example for "ssl usage" and "sharing port usage"

Hi, I found another way not to modify system library and implement port sharing of HTTP Request and WebSocket Request, which required a deep re-implementation of handler.
This code is also an example of websocket using SSL.
Ghostplant<EMAIL_ADDRESS>

Any comment?

I'm unsure about this approach. I'm building a startup and moving to a new place with my family, so unpaid OSS work doesn't rank very high on my priorities right now.
I don't want to ship an example that includes a HTTP parser. That stuff belongs to a library, not a code sample.
I'm planning to assess your proposal when I find time, likely in July.

Okay. This is the only way to share ports without changing existing libraries. After all, websocket is mainly used by JS from an HTML client. If they don't share ports, it's easier for users to get into trouble on configurating multiple paisr of PEM certificates.
Just like issues/111, the current example "showtime.html" using SSL mode doesn't work in firefox, and "Malformed HTTP message" will be reported. After sharing port in the added example, this problem gets fixed.

I looked at this again today.
I appreciate that you took the time to submit this example as a pull request.
Unfortunately it's very far from how I would approach these problems and what I consider best practices. I wouldn't put this in production and I'm not comfortable recommending this approach, the main reasons being:

there's a bunch of security vulnerabilities in the HTTP parsing (look at the countermeasures in websockets/http.py for some examples)
I don't think port sharing is a good idea in general, or even useful, and if you want to do it I'd rather you use Twisted than websockets
I don't want to teach people about TLS that way.

@aaugustin Thanks, it doesn't matter since I already dropped dependency with websockets a year of because of some other bugs in this module (easily crash under huge quantities of connection, etc). Currently I found that python3-tornado seems to be a better choice since it is a lot more faster.


Custom CSS  not working in angular 7

I want to change the size of modal to full page. so i used custom CSS . but it is not changing the size. throughing error

Hi @arunkumarbrt,
can you give us an example of your code? You can use stackblitz.
However, it would be great if you could use the issue template that was displayed when you open your issue. 👌


stream_select error suppression

Currently we suppress errors for stream_select, but that might also hide useful error messages like the one shown in https://github.com/ratchetphp/Ratchet/issues/300:
PHP Warning:  stream_select(): You MUST recompile PHP with a larger value of FD_SETSIZE.
It is set to 1024, but you have descriptors numbered at least as high as 1123.
 --enable-fd-setsize=2048 is recommended, but you may want to set it
to equal the maximum number of open files supported by your system,
in order to avoid seeing this error again at a later date.

Dunno whether we can do something about that without a high performance impact.

It should be comparatively rare that stream_selects returns 0/false at all. (typically only when interrupted due to a signal, which means that it was anyway idling)
We may just check the error message at that place (right before the return; statement in NativeLoop) without performance penalty under normal operation.

See #162 for a PR.


Create TEST_QRDA_3

Information

Fixes #_.
JIRA story _.

Changes proposed in this PR:

(Please list what you changed.)

Checklist

[ ] All JUnit tests pass (mvn clean verify).
[ ] New unit tests written to cover new functionality.
[ ] Added and updated JavaDocs for non-test classes and methods.
[ ] No local design debt. Do you feel that something is "ugly" after your changes?
[ ] Updated documentation (README.md, etc.) depending if the changes require it.

@vabihsahu There is already a test QRDA-III file under the qrda-files directory labelled valid-QRDA-III-latest.xml The measure data is still references 2022 measures but that will be updated this coming week.


Cask request: Electron Markdownify

Cask details
(Please fill out as much as possible)
Name - Electron Markdownify
Homepage - http://www.amitmerchant.com/electron-markdownify/
License - MIT
Download URL - https://github.com/amitmerchant1990/electron-markdownify/releases/download/v1.1.3/Markdownify-darwin-x64.zip
Description - This is a minimal Markdown Editor desktop app based on Electron.

I added a first version here - but it seams the .app is broken from the original source.
https://github.com/wingsuitist/homebrew-cask/commit/f36cc4406f51b423c5d5c28b5e9666eac9eec374

it seams the .app is broken from the original source

What do you mean? Broken how?
Submit it anyway, so we can all check the PR and comment/help.

Closing due to no reply and apparently no further interest in this cask.


Images load with a white rectangle outline.

We upgraded to Next13. Next13 replaces the next/image component that we heavily depended on with what was previously next/image/next. Whereas the previous next/image is now next/image/legacy.
The new one has some improvements we'd like to use, but it also suffers from a white outline bug in Safari. Let's wait with the upgrade until it's fixed.
https://github.com/vercel/next.js/issues/40615
I have given the proposed workaround a shot but it doesn't appear to work.
Using this issue to track.

Unconfirmed fix in: https://github.com/ultrasoundmoney/frontend/commit/0220058008dc8024ff34de0414f859f5a2988af1


Refactor charts, BarChart and LineChart layout fixes

Description
This is a refactoring of the chart code with some minor fixes and improvements, continued from https://github.com/MudBlazor/MudBlazor/pull/2209. Summary of changes:

Moved constants to the top
Split up the very long OnParametersSet method into smaller separate methods
Removed the ILineInterpolator.InterpolationRequired and NoInterpolation classes
Minor fixes, e.g. replaced ((int)(maxY / gridYUnits)) + 1; by (int)Math.Ceiling(maxY / gridYUnits);. The former rounds up even if maxY is a multiple of gridYUnits, which is probably undesired here.
The variables numHorizontalLines and numVerticalLines now really do denote the respective number of lines, before it was number of lines - 1
The "safeguard against millions of gridlines which might arise with very high values" which only existed in LineChart has been copied to BarChart
Performance optimization: avoid population of XValues and YValues arrays when interpolation is disabled
Performance optimization: use StringBuilder to construct the SVG data

How Has This Been Tested?

Tested the changes visually using the Unit test viewer charts page
The existing unit tests have been adapted

Types of changes

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

Before:

After (note the y axis range of the bar chart and the x axis range of the line chart):

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

Hi.
I don't see the justification of removing InterpolationRequired and NoInterpolation, even if they are not used / not required anymore, removing any public API (property, classes etc) is a breaking change, which means your PR will be on hold until next major version, I would value more pull requests that are not breaking.

Hi. I don't see the justification of removing InterpolationRequired and NoInterpolation, even if they are not used / not required anymore, removing any public API (property, classes etc) is a breaking change, which means your PR will be on hold until next major version, I would value more pull requests that are not breaking.

I guess we could mark that property and class as deprecated and remove it with version 7. I'd prefer that over deferring this PR until version 7.
I wonder if maybe all the interpolation functionality should better not have been made part of the public API. It might be useful if somebody wants to write their own chart control and reuse the interpolation functionality but otherwise, this logic could be seen as an implementation detail of the inbuilt charts, no?

I guess we could mark that property and class as deprecated and remove it with version 7. I'd prefer that over deferring this PR until version 7.

Yeah, that would be the best way to address it. What do you think @henon?

Sorry for the late reply, but yes, obsoleting and removing in v7 is the way to go.


Update action to Node 16 and update node modules

This PR:

Updates the action to use Node 16
Updates dependencies to latest versions
Updates dev dependencies to latest versions

ToDo

[ ] update eslint settings
[ ] explicitly declare chalk as a dependency (rather than an implicite devDependency)

Version 12 still showing in .github/workflows/ci.yml . This mismatch causes the CI build to fail.
By the way, thanks for this work. I've pulled your changes into my form at https://github.com/stevenbitner/autograding since I don't think any merges happen on this repo any more.

The CI update is in PR #55.
I have a combined branch in my fork with successful workflow runs: https://github.com/markpatterson27/autograding/actions/runs/3310743415

There's still a few changes that should be made:

eslint rules should be updated, but I'm not sure what the recommended GitHub coding styles are.
Chalk should be added as an explicite dependency. There are some breaking changes going to v5 of Chalk though which I haven't had the bandwidth to look at properly.

I suspect this PR would also fix the Issue #71 I just opened as it updates the action/core version.

CI workflow run against node 16.x (see PR #55) for these recent commits is in my fork: https://github.com/markpatterson27/autograding/actions/runs/4794051331

Confirmed tests running with node version 16 on fork.


Dockerfile: strip binary

Decrease image size by stripping docker-buildx binary
Using docker/buildx-bin as binary source mitigates the issue with wget behind HTTP/1.1 proxies
Use multistage build to keep size at a minimum
Use specific tags (not latest) for base images according to best practice recommendations

Signed-off-by: Joakim Roubert<EMAIL_ADDRESS>

I will take a look at this later today

@joakimr-axis -- how often do you build this image locally vs pulling down from docker hub/gcr?

I'll merge by next week or earlier unless I hear any pushback

@joakimr-axis -- how often do you build this image locally vs pulling down from docker hub/gcr?

That is certainly a valid question: When I want to build with some other base image than docker:latest (ideally, you should probably have a specific tag such as docker:20.10.7 or similar for security reasons) and/or a BUILDX_VERSION other than the one in the Dockerfile.

Approved!
@joakimr-axis -- I can merge if/when you look over the change (one commit) that I made as well and tell me what you think

Works well and looks good to me. Two questions though:

Why did you ditch ARG BASE_IMG=docker:latest?

When do you need to have a tag for the last multistage image? From what I can see that

FROM docker:latest as buildx_image
could just as well be
FROM docker:latest
When building, the target will always be that last one unless e.g. buildx_download is stated.

Ok, I now did some additional update(s) that got line numbers and complexity down a bit.

@joakimr-axis now that #7 was created, do you think this PR should be closed?
I think that one now covers all the bases :-) (it is awesome!)

@joakimr-axis now that #7 was created, do you think this PR should be closed?
I think that one now covers all the bases :-) (it is awesome!)

Unfortunately, the binary in the Docker container is not stripped. So as you can see, I have scaled down the scope quite a bit in this one thanks to the Docker bin container, but the strip step is still needed if one wants to shave off a couple of MBs from the resulting image.

So I think that #7 with your nice latest touch-ups should be merged first. Then let me update this one to be based on that one and you can then see if the hassle to get the binary stripped is worth the image footprint savings or if the simplicity of what is in #7 trumps image size.

So I think that #7 with your nice latest touch-ups should be merged first. Then let me update this one to be based on that one and you can then see if the hassle to get the binary stripped is worth the image footprint savings or if the simplicity of what is in #7 trumps image size.

Sure!
I am not 100% sure if the strip is actually removing anything /shrug

So I think that #7 with your nice latest touch-ups should be merged first. Then let me update this one to be based on that one and you can then see if the hassle to get the binary stripped is worth the image footprint savings or if the simplicity of what is in #7 trumps image size.

Sure!
I am not 100% sure if the strip is actually removing anything /shrug

Oh, but it does! If I build the current on main (with #7, although it is the very same container output size as previously) the resulting image is 307 MB (not compressed), whereas the one with the stripped binary is 291 MB.

I have now rebased this PR, so it's ready to review (again).
(Even if you choose not to strip the docker-buildx binary, I strongly suggest to use specific tags and not the "latest" ones.)

@joakimr-axis
Should we rethink how the images are named?
Now that docker is configurable along with the version of buildx? (As I have in my PR for #8 we probably don't need alpine)
Idk a good scheme to name them
Any ideas?
Just naming after the buildx version is probably not the best approach.
Should I turn this into a GitHub discussion maybe?

Moved above discussion here: https://github.com/maniator/dind-buildx/discussions/9

I have now rebased this PR, so it's ready to review (again).
(Even if you choose not to strip the docker-buildx binary, I strongly suggest to use specific tags and not the "latest" ones.)

I will update #8 to use the specific docker tags now.
We also should discuss #9
Also, after I update #8 I am not 100% sure we need to strip the exec

@joakimr-axis  -- do you think it is worth to rebase this and keep the strip?
Or do we not need it?

@joakimr-axis -- do you think it is worth to rebase this and keep the strip?
Or do we not need it?

It's up to you, actually. :-) Is it worth the added complexity to strip the binary to get a two digit MB decrease in the final image as compared to the unstripped one?


sokol_app.h: support for settings x11 cursor

Add a new api call, sapp_set_cursor
Add some standard cursors, same as GLFW, e.g. sapp_set_cursor(&SAPP_IBEAM_CURSOR)
Only implemented on x11, tested on my own (zig) project locally

FIY I'll start working on this PR now in a branch created from the PR.
One thing I noticed and which I'm going to change is that I'll turn the sapp_cursor_t struct into a simple sapp_cursor enum.
Next I'll look around how to do the Windows and macOS implementation.
Finally I'll change the imgui-samples to use the cursor images.

@floooh sapp_cursor_t contains the actual platform-independent cursor data/handle, that needs to be a struct not an enum.
_sapp_x11_create_standard_cursor is the equivalent to GLFW's glfwCreateStandardCursor, but I chose not to expose that to the user & instead just create all the cursors at startup & put them in global vars. Maybe this doesn't suit sokol, in which case remove the globals & expose an sapp_create_standard_cursor function like glfw, which switches on an enum value
The 10 'standard' cursors I initialize at startup are taken from the defines at glfw3.h:1135, possibly this is more up to date than the docs? limiting sokol's support to the same as GLFW's official docs is probably reasonable though. You can see these are used in glfw/tests/cursor.c

...ah ok, I've been looking at GLFW 3.3, not 3.4. Ok, I'll use the GLFW 3.4 cursor image types.
I'll still change the public cursor type to an enum, wrapping the platform specific cursor handle in a public type isn't really needed (note that in GLFW, the cursor 'handles' are also just defines: https://github.com/glfw/glfw/blob/da6713cd096a40a4512f468b34c189017e73f987/include/GLFW/glfw3.h#L1161)
I'm still ending up rewriting most of the PR unfortunately, because of the complex relationship between sapp_show/hide_mouse() and the currently set cursor type (on X11, hiding and showing the cursor would always end up calling XUndefineCursor() which I think would switch back to the system default cursor.
The functions _sapp_x11_show_mouse() and _sapp_x11_set_cursor() will basically be merged into a single _sapp_x11_update_cursor() function.

You can track my changes here:
https://github.com/floooh/sokol/compare/master...tomc1998-master
I have also added the mouse cursor update to sokol_imgui.h, on Linux this is working fine. Over the next few days I'll try to look into Windows, macOS and HTML5 (I think updating the cursor should also be possible on the web).
This is branched from your PR, so after merging your original contribution should still be properly attributed in the history :)

Ah go ahead, i don't understand how sokol does other cursor stuff so my code is likely wrong :+1:

note that in GLFW, the cursor 'handles' are also just defines

That's what i was trying to explain, these aren't handles! they're just parameters that you pass into glfwCreateStandardCursor to get a handle, which contains the platform specific stuff. sapp_cursor_t is the equivalent of GLFWcursor. That's the difference with my impl, I already call 'create_standard_cursor' for all the standard cursors at startup & then expose them as global vars - glfw doesn't do this, and lets you call 'create_standard_cursor' in your own time.

Ah ok, I prefer the enum approach though, less private information leaking into the user side of the API. In a later version we could add another function to associate a custom cursor image with a cursor image, a bit similar to the sapp_set_icon() function (something like
sapp_set_mouse_cursor_image(sapp_mouse_cursor cursor, const sapp_mouse_cursor_desc* desc);

PS: another nice effect of this PR is that we can hopefully remove the somewhat weird "user cursor" feature (sapp_desc.user_cursor and SAPP_EVENTTYPE_UPDATE_CURSOR), I never particularly liked this, but it was a necessary workaround for switching the cursor image on Windows.

Ok, macOS implemented: https://github.com/floooh/sokol/commit/8c1bc368b0b8b13c471d7bdab609e7cd3947908c
I guess the reason why GLFW didn't have the 'diagonal' cursor types was because macOS doesn't expose those in the public API (but everybody is using the private methods anyway)

Windows implemented: https://github.com/floooh/sokol/commit/454972323d5902aa38373735283d1b71e5c4bc3e

Reminders:

[ ] HTML5
[ ] minimal UWP hack (just fix the compilation issue)
[ ] can remove SAPP_EVENT_
[ ] update header documentation
[ ] update changelog

HTML 5: https://github.com/floooh/sokol/commit/56d09103606208a84f2496a55dc7c26034997dc6

UWP: https://github.com/floooh/sokol/commit/dfc01d45ce79479ced4be413f345adb34e30741e

Ok, my branch which I created from your PR has been merged, but this time Github didn't detect the association with the original PR and didn't mark the PR as merged. Your commits are still recorded in the history though.
So I gonna need to "Close" this PR, even though it has been actually merged.

...I'll update the changelog next and see how the new enum looks in the Zig and Nim bindings, I priobably need to manually tweak the name mapping.


Example code from mario_gpt should possibly be mario_gpt.lm

Hi,
First of all thanks for your novel implementation! This is very cool to see.
When running the minimal code snippet provided in the readme, I got the following error:
C:\Users\Mossly\Desktop>py mariotest.py
Traceback (most recent call last):
  File "C:\Users\Mossly\Desktop\mariotest.py", line 1, in <module>
    from mario_gpt import MarioLM
ImportError: cannot import name 'MarioLM' from 'mario_gpt' (C:\Users\Mossly\AppData\Local\Programs\Python\Python310\lib\site-packages\mario_gpt\__init__.py)

I resolved it by changing the line:
from mario_gpt import MarioLM
to
from mario_gpt.lm import MarioLM
Perhaps this is just a simple syntactical mistake? Or it could be user error on my part...
Thanks for your attention!

Hey! So I actually updated the code to make this importing possible, but I realized that the pypi package has not really been updated yet. I reverted this back in the README so it should be good :)

Thanks for your assistance!
I'm pretty close to getting things working but hit another error I've been unable to debug.
generated_level = mario_lm.sample(
File "lm.py", line 113, in sample
[self.prompter.output_hidden(prompt) for prompt in prompts]
File "lm.py", line 113, in
[self.prompter.output_hidden(prompt) for prompt in prompts]
File "prompter.py", line 117, in output_hidden
.mean(0)
AttributeError: 'list' object has no attribute 'mean'
This is using the default prompts = ["many pipes", "many enemies", "some blocks", "high elevation"]
Any help would be greatly appreciated!

Hey! So multiple prompts should now work. You should probably clone the repo or update from pypi. On another note, when using default prompts make sure you're using:
prompts = ["many pipes, many enemies, some blocks, high elevation"]
instead of
prompts = ["many pipes", "many enemies", "some blocks", "high elevation"]
If you split the categories the model could still work, but I haven't tested that out extensively.

Also, make sure you update your transformers package :)

Thanks for pointing this out though, lmk if there's any other errors!

I updated the repo, transformers, fixed up the prompts as per your advice, and now everything is working on Windows 11 with Python 3.10.
Thanks again for all your help! :)


Fix doctest failure related to __array_function__ support

np.allclose(x_gpu, [1, 2, 3]) now returns True without any errors by __array_function__ support.
https://jenkins.preferred.jp/job/chainer/job/cupy_pr/227/TEST=cupy-doc,label=mn1-p100/console

Jenkins, test this please.

Jenkins CI test (for commit 8fe0b3003d32eed332cc3fbca747763d36b902c2, target branch master) succeeded!


Trouble with installation

It says some modules are missing!

just means that there is no file on that route. it works fine for me, so check if there's actually a file on that route and adjust it as needed.


Make this work with the Inventory Setups plugin.

This plugin should detect when the bank is being filtered by the Inventory Setups plugin, so that custom layouts can be set.
I think using the bank title is the best way to detect this, since the Inventory Setups plugin modifies this when it is filtering the bank.
Should I require custom layouts to be enabled for these inventory setups? I'm already not 100% sure doing that for regular tabs was the best idea, and I'm also not sure how to let people configure whether the custom layout should be enabled for a particular inventory setups. A checkbox in the config to enable this (probably set to true by default) would be good enough, I assume.

Should it be possible to import/export layouts for these?

I would love this feature. I mainly use inventory setups.
Besides using Inventory Setups + Bank Tag custom layouts, there is a bug when opening the bank.
I've added an image for if you're using both plugins. A tab is active (from previous bank-attendance or clicking it whilst you're using the bank) and an inventory setup is active.
Most of the items aren't in the 'teleport tab', but are from the inventory setup. Yet the placeholders makes it a bit difficult to use the inventory setup plugin.

@MathieuDR  how exactly are you making this happen? I've made it happen before but either it only happens randomly or I've forgotten how I did it last time.

This is a Runelite bug (source: Runelite discord). Will probably be fixed next time Runelite updates.

An initial version of this feature exists on branch inventory-setups-support. Before publishing this, I'd like to add auto-layout to this, based on the contents of the inventory setup.

PR is finally open! https://github.com/runelite/plugin-hub/pull/1331
No import/export currently exists, would like this to be done by Inventory Setups rather than by my plugin, as it's more convenient.
Also there is currently no way to transfer layouts between bank tags and inventory setups.

PR has been accepted. Closing this issue and making new ones for the remaining work.


[CommonJS] Future-proof way of passing the default logger

Hey, is it OK to regard whatever createLogger() (with no arguments) returns to be the same as the default logger? I'd prefer not to use require('redux-logger').default as it's a babel-specific ES module workaround.

You can import default logger, if you're prefer CommonJS.
const { logger } = require('redux-logger')

or, of course,
const { createLogger } = require('redux-logger')

const logger = createLogger()

But we decided to drop support because it's not friendly with named exports. More information: https://github.com/evgenyrodionov/redux-logger/issues/210#issuecomment-288672170


🔧 ChainMoney composition api

PR Type

[x] Bugfix
[x] Refactoring

Context

[x] Closes #5577
[x] Requires #5821

Before submitting pull request, please make sure:

[ ] My contribution builds clean without any errors or warnings
[ ] I've merged recent default branch -- main and I've no conflicts
[ ] I've tried to respect high code quality standards
[ ] I've didn't break any original functionality

Screenshot 📸

with Unit fix

Copilot Summary

🤖 Generated by Copilot at f10572b
Refactored ChainMoney component to use Vue 3 and support multiple chains. Replaced unit prop with chainSymbol and used useChain hook.

🤖 Generated by Copilot at f10572b

Sing, O Muse, of the skillful coder who updated
The ChainMoney component, a shining display of value,
To use the new features of Vue 3, the framework divine,
And to support different networks, the realms of many chains.

should these be KSM?

Yup

pay 30 usd


Fix icons in menu

Since some of the codepoints have been moved around this change is needed so that it renders correctly. See this issue for more information: https://github.com/ryanoasis/nerd-fonts/issues/1190

Someone already has made this change, my bad.


NotifyIcon remains in Tray even after application closing but disappears on Mouse Hover

NotifyIcon remains in Tray even after application closing but disappears on Mouse Hover.
Code of the NotifiyIcon can be found in Bootstrap.cs https://github.com/teun25/C-sharp-clipboardquestionanswer-/blob/master/Bootstrap.cs at line 31 until 33 and 185 until 220

Make sure to actually set notifyIcon.Visible = false; or (notifyIcon.Icon = null;) before you call notifyIcon.Dispose().
Also keep in mind that debugging the program (and quitting with the debugger) will likely still cause this issue, but your release shouldn't.


Test for QuasiQuotes

fix #59
The purpose of refactoring
My purpose of 2c889cf is...

to use a more natural way.

To abort the computation, use fail.
https://hackage.haskell.org/package/template-haskell-<IP_ADDRESS>/docs/Language-Haskell-TH.html#g:2

to make error states easier to handle.

Side effect of the refactoring
By 2c889cf , compile-time error messages are changed.
I think but this change is good. They get more simple.
Before:
/Users/ikngtty/Projects/src/github.com/realglobe-Inc/hrafnar/server-test/Hrafnar/ParserSpec.hs:69:25: error:
    • Exception when trying to run compile-time code:
        multiple characters
CallStack (from HasCallStack):
  error, called at util/Hrafnar/Util.hs:29:20 in hrafnar-<IP_ADDRESS>-Jqfmt167IwoGCyqBIvYRaG-hrafnar-util:Hrafnar.Util
      Code: Language.Haskell.TH.Quote.quoteExp rawC "ab"
    • In the quasi-quotation: [rawC|ab|]
   |
69 |             Lit' (Char' [rawC|ab|])
   |                         ^^^^^^^^^^

After:
/Users/ikngtty/Projects/src/github.com/realglobe-Inc/hrafnar/server-test/Hrafnar/ParserSpec.hs:69:31: error:
    • multiple characters
    • In the quasi-quotation: [rawC|ab|]
   |
69 |             Lit' (Char' [rawC|ab|])
   |                               ^^^^

Great! It might be pleased in Haskell community, if you summarize to an article.

I think that too.  And I want to bundle that code into a package and publish it to Stackage.
 I'll do it when I have plenty of time.


Make the style-applying engine easily overridable

With v2.0, a new diffing algorithm calculates the styles that needs to be applied, and the props that need to be removed.
This allows the style applying engine to be easily replacable.
Using something like Styletron, for example instead of using Element.style.
Currently the change set looks as follows:
{
  addStyles: {
    border: "none",
  },
  removeProps: ["font-size"]
}

The only change needed though, is to separate styles generated by container units from regular ones, since the former changes a lot more. (On every single container size change.)
This means that you may want to treat those differently.
For example:
Styletron would create a great number of classes for all values generated by container units, like:
.a { font-size: 10px;}
.b { font-size: 10.123px;}
.c { font-size: 10.345px;}
.d { font-size: 10.567px;}
etc ...

Which basically means that for values, using styletron would not be a good solution, while using it to update normal styles would work very well.

Note: in order for styletron to be able to remove properties, it needs the values too, nod just the prop names.


flutter adds https git remote repo __flutter_version_check__

Steps to Reproduce

run flutter -v help.  The command gets stuck.

My local flutter repo will have a new remote __flutter_version_check__ repo https://github.com/flutter/flutter.git.  But my dev environment can only use the git: protocol.
https://github.com/flutter/flutter/blob/05dadb0120b4aa3e5c0ccb1cf302632bf563d104/packages/flutter_tools/test/general.shard/version_test.dart#L103

Hi @zjiekai
Can you please provide your flutter doctor -v?
Thank you

I fixed my git https issue.  Now I can not reproduce this anymore.

Hi @zjiekai
Given your last message I feel safe to close this issue, if you disagree please write in the comments and I will reopen it.
Thank you


Use case: exit-early computation (akin to Maybe monad)

Description:

When a value in the middle of computation evaluates to null, terminate the computation and return the default value

NOTE: The same may be achieved through exceptions
Similar cases:

Retry on null

Issues:

Properties as suspension points (#24)

val e = maybe(default = 5) {
    // would like to do .mb, but have to do .mb() instead
    val x = foo?.bar().mb()
    val y = x.baz.mb()
    x + y
}

Library:
fun <T : Any> maybe(default: T, coroutine c: () -> Coroutine<MaybeController<T>>): T? {
    val ctl = MaybeController<T>()
    c().entryPoint(ctl).resume(Unit)
    return ctl.result ?: default
}

class MaybeController<T : Any> {
    var result: T? = null

    suspend fun <V: Any> V?.mb(c: Continuation<V>) {
        if (this == null) return // TODO: execute finally blocks here
        c.resume(this)
    }

    operator fun handleResult(v: T) {
        result = v
    }
}

Is it really a use-case for coroutines? It is natively handled by exceptions. You can implement it extremely efficiently with exceptions. Coroutine-based implementation will unlikely to ever be as efficient. See the following that properly supports all kinds of try-finally blocks (including catch-exception-rethrow style finally blocks) and defines mb as extension property:
private val ABORT = Throwable("Abort")

public val <V: Any> V?.mb: V get() = this ?: throw ABORT

public fun <T: Any> maybe(default: T, block: () -> T): T {
    try {
        return block()
    } catch (t: Throwable) {
        if (t == ABORT) return default
        throw t
    }
}

Closed. Obsolete. (actually Arrow has it now on top of coroutines, I think)


Style fixes

Fixed up some syntax errors in the wallet css and changed the button styles to go with the color theme.

thanks!

Does this fix #747?

No, I'm still trying to figure out how to replicate that bug and since I'm fairly new to Sia I need to do some playing around so I can learn all the features


Vulnerability alert CWE-23: Relative Path Traversal

** removed **

Hello @guisaulo and thanks for the report. I have removed it since I haven't heard of this report and we have a security policy (https://github.com/expressjs/express/security/policy) set for how to report Express.js vulnerabilities.
If you can send me all the details and such I can help look into the issue you are reporting. The bits I saw on there look like it is something to with the a third-party module in use, and not in Express.js itself, but without really more details it is hard to really say. I can say that I don't believe something that has been reported to us in the past, though.

Just to add a bit more reading the linked references on there, it probably makes sense why this particular issue likely hasn't been reported to Express.js directly: it seems that the issue was in a third-party dependency. The reference states that "This issue has been patched in [non-express dependency] versions 4.1.2, 4.0.14, and later.". So the version of Express.js you're using would seem to be irrelevant to the issue, you just need to update that other module (which is not an express dependency, so express cannot update it for you). I hope that helps. Feel free to still contact via the security policy if you disagree or have further information that demonstrates there is an issue in Express.js itself.


[Feature Request] Add devcontainer configuration

Tell us about your feature request
I usually work with dev containers either in Vscode or devpod which use the devcontainer.json standard. Will be interesting to initiate the project with this feature.
Disclaimer

[X] I agree

I think this is a great tool. I will personally explore it and give it a shot... but implementing this into the blueprint is a bit too opinionated.
A more suitable approach would be Docker as an advanced flag. Currently, we have a compose file for databases. Adding configuration that can containerize, create, or update (case when a database is used)  docker-compose file is something that we will accept.


Add spaces between values and units

Punctuation of units is improved. Now values are more readable because of an extra space between the value and the unit. By the way, it's the SI standard:

https://english.stackexchange.com/questions/2794/punctuation-with-units
https://www.bipm.org/en/publications/si-brochure/section5-3.html

Makes sense to me.

@raiguard In the SI standard (the second web page), there is also a rule says that percent (%) sign must be spaced. I can do this. What do you think?

I’ve always written % without a space, and that is what I see 99% of the Rainmeter community do it. So I think we should leave that as-is.
Plus, I think it looks better.

OK.


CI zip file deployment version flag fix

Fixes #6030

Thanks @ggetz!  Since you're on a fork, I used your repository to verify this (since that is also set up for CI).


Using @auth directive with @key directive

Which Category is your question related to?
GraphQL transform.
Amplify CLI Version
3.17.0
What AWS Services are you utilizing?
dynamodb, appsync, cognito, s3
Provide additional details e.g. code snippets
Here's the use case:

new users create account through wizard like pages
want to capture email in db right away in case new user exits before completing all steps
allow new user to create customer record via public auth (apiKey)
want to check for email existence in case where user exited before finishing process and returns to complete
BUT don't want to open up customer type for anyone to read.
there is a byEmail key which will be used.

Can I put the public read auth on the key and limit the fields returned to just the email was entered? All I need is to know if it exists or not.
Or do I need to create my own cusom resolver (create customer) where I do the check?

GraphQL transformer supports field level auth. If you rely on subscription on your user type, the behavior changes when you add field level auth. This is documented here


CheckHealth() task crashes when running REST interface

Client.getXQConnection(), run from directXQuery, run from checkHealth results in an "IO Exception: Access Denied"
Not sure why this is happening

However the server itself runs fine

The server runs fine in unit tests / RestX specs but not from the web browser - trying to debug this right now

This 665d91c867dd272508629ac8c6ae2a5f658e1e4d introduced the bug


Referenced Assembly doesn't have a strong name

First of all, kudos on building such an amazing library!! Thank you to you and everyone involved in the project!
Onto the issue:
I'm tyring to include this package directly into one of my projects. But when I'm trying to build it, I'm facing this issue:
CSC : error CS8002: Referenced assembly 'Clipper2Lib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' does not have a strong name.
Could you please publish a signed version of the package on nugget so that I can directly use without having to manually sign it myself every time there's an update?

That's what I would like to see.

Thanks for the feedback. I'm afraid my knowledge of C# is quite limited and I'm only barely competent in producing nugget packages. To do what you're requesting, it seems I'd need a certificate from a Signing Authority, which usually require payment. And I'm rather disinclined to spend money just so I can share my open source freeware.


Update alga + nalgebra version.

We found that nalgebra and alga can be updated.

nalgebra 0.18
alga 0.9
one test is slightly different (less than 10e-13)
change deprecated function into_subview into index_axis_move

I have made test with these commands:
cargo build --release --features ndarray_volumes
cargo test --release --features ndarray_volumes
cargo build --release --features nalgebra_affine
cargo test --release --features nalgebra_affine
cargo build --release --features ndarray_volumes,nalgebra_affine
cargo test --release --features ndarray_volumes,nalgebra_affine

everything looks fine.

LGTM. Disclaimer: @czotti is my colleague.

Thanks!


Seems to get confused with wildcard domains?

When typing in our company domain (vaultdragon.com) on the dnstwist.it site I have noticed variations such as vaultd.ragon.com, vaultdr.agon.com and so on.
These seem to not actually be set subdomains, but actually wildcard domains. I think it gives the wrong impression that these were actively set subdomains for malicious intent as opposed to wildcards that would respond regardless of the subdomain entered.
Is this a concern? It seems like it would be beneficial to have some denotation that certain subdomains being returned are due to a wildcard DNS settings so it's more obvious that it might not be anything too concerning?

I personally describe this as "collision" of brands/domains, when permutation of a legitimate domain turns into another very similar and also legitimate brand. In this particular case it's due to the wildcard domain. In general, all domains reported by the tool should be initially considered only as suspicious or potentially phishing.