Search for the current host in nodes

To ensure we find the current host we should search for the hostname
without domain in the response array of nodes, and then proceed to
collect metrics from that dict.
Addresses: #610

Looks okay to me. :+1:

Tested with AIO and works for me. Discussed with @git-harry that run_queue may not be the right thing here, but that predates this PR.

I logged a bug [1] to track the issue referenced by @hughsaunders
[1] https://github.com/rcbops/rpc-openstack/issues/626


Currently on a successful ActionResult (start / stop) we return nil

Returning the result of the api call would at least be a positive result
https://bugzilla.redhat.com/show_bug.cgi?id=1628658

Hammer backport details:
$ git log -1
commit 266b6c550dcb461206993916828ea9c5295b4e4a
Author: Nick Carboni<EMAIL_ADDRESS>Date:   Mon Oct 1 15:56:11 2018 -0400

    Merge pull request #18036 from bdunne/action_result_shouldnt_be_nil

    Currently on a successful ActionResult (start / stop) we return nil

    (cherry picked from commit 2fb626056c1b7e53203ac7d727780693a1c619c1)

    https://bugzilla.redhat.com/show_bug.cgi?id=1628658

Gaprindashvili backport details:
$ git log -1
commit a44cae5a04cd9965eee660d0cef5864fb4a3ee07
Author: Nick Carboni<EMAIL_ADDRESS>Date:   Mon Oct 1 15:56:11 2018 -0400

    Merge pull request #18036 from bdunne/action_result_shouldnt_be_nil

    Currently on a successful ActionResult (start / stop) we return nil

    (cherry picked from commit 2fb626056c1b7e53203ac7d727780693a1c619c1)

    https://bugzilla.redhat.com/show_bug.cgi?id=1635764


SITE_URL not being read

My deployment was working earlier, not sure what affected the sudden issue. Here is my netlify.toml (with GRAPH_OPTIONS omitted):

[build]
command = "rm -rf __obsidian __site && mkdir __obsidian && mv * __obsidian || true && git clone https://github.com/ppeetteerrs/obsidian-zola.git __site && __site/run.sh"
publish = "public"
[build.environment]
# (Optional) Text in landing page button
LANDING_BUTTON = "Enter"
# (Optional) Site description on landing page
LANDING_DESCRIPTION = ""
# (REQUIRED) The Markdown page linked to landing page button (e.g. put home for ./home.md)
LANDING_PAGE = "home"
# (Optional) Site title on landing page
LANDING_TITLE = "Spring 2023 Notes"
PYTHON_VERSION = "3.8"
# (REQUIRED) Site repo URL
REPO_URL = "https://github.com/ndoniyor/notes-spring-2023/"
# (Optional) Site title in navbar
SITE_TITLE = ""
# (Optional) Site title in browser tab (leave blank to use SITE_TITLE)
SITE_TITLE_TAB = ""
# (REQUIRED) Netlify site URL
SITE_URL = "https://dnim-notes.netlify.app/"
# (Optional) Site Timezone
TIMEZONE = "America/New_York"
ZOLA_VERSION = "0.15.2"
# (Optional, default title) How to sort notes inside each section on the right sidebar ("date" or "title"). Subsections on side bar are sorted by title, the sort order cannot be changed.
SORT_BY = "title"
# (Optional) Google Analytics Measurement ID
GANALYTICS = ""
# (Optional, default true) Slugify URLs. Put "" to disable.
SLUGIFY = "y"
# (Optional, default true) Shows knowledge graph on home page. Put "" to disable.
HOME_GRAPH = "y"
# (Optional, default true) Shows knowledge graph on every page. Put "" to disable.
PAGE_GRAPH = "y"
# (Optional, default false) Whether sidebar sections should be collapsed by default.
SIDEBAR_COLLAPSED = ""
# (Optional, default blank) Additional footer content.
FOOTER = ""
# (Optional, default main) Root section name.
ROOT_SECTION_NAME = "main"

Netlify returns the following error:

Build failed due to a user error: Build script returned non-zero exit code: 2

But when I run the build command on my own I get the following:

raise Exception(f"FATAL ERROR: build.environment.{key} not set!") Exception: FATAL ERROR: build.environment.SITE_URL not set! __site/run.sh: line 24: zola: command not found

I dont understand why since SITE_URL is set.

I think this morning Netlify changed something because we're all ultimately getting the same error:
line 24: zola: command not found
https://answers.netlify.com/t/error-127-when-deploying-a-zola-website/86029/4

It's confirmed that it was an issue on Netlify's side, it's fixed. It works now!
https://answers.netlify.com/t/jekyll-build-failing-today-exact-source-built-successfully-yesterday/86038/10?u=orditeck


Add additional terminal path traversal commands

This pull request is intended to enhance directory traversal in terminals. It adds the capability to:

Traverse not only to the parent directory (katie up), but an arbitrary number of directories upwards by appending an ordinal: katie up third results in cd ../../.. in POSIX shells. This behavior differs a little from the usual use of ordinal suffixes. The commands in repeater.talon usually repeat the last command N times, but in the context of changing directories in a terminal, this repetition is more sensibly applied to the directory traversal itself. I've been using this extensively on my machines and found it pretty ergonomic to use.
Toggle-traverse between the most recent directories (cd -)
Navigate back to previous directories, similar to a browser back button. This is not possible in POSIX shells, but the shells usually used in developer terminals like bash or zsh provide this capability. This is why I introduced an additional tag.

I don't have any experience with windows terminals and no access to OSX. The latter should default to zsh shells, so we should be good there. I could use some contributions regarding implementations for other shells/OSes.

Wouldn't katie up third have the same effect without your modification? (It would result in cd ..\n cd ..\n cd ..\n)

@DonnerWolfBach : You know what, in vanilla knausj_talon, you are perfectly right. I didn't even realize that. In my local setup, I don't have key(enter) in the implementation. In this case, you can't really repeat the command with ordinals. While my implementation here still leads to a cleaner bash command history, its value is definitely debatable in vanilla. Would you say that this pull request still has merit this particular command is removed? So only cd - and popd?

Yeah, I think having cd - and popd in knausj still makes sence. We can even copy and paste the code for powershell since it's (rougly) the same there: https://superuser.com/questions/593987/change-directory-to-previous-directory-in-powershell
Unfortunately, there is no equivalent in windows command prompt to cd - - but that's not a big deal, while not officially deprecated it kinda is.


What is SPKeyStore used for?

I'm trying to understand why we need to provide a certificate to create AuthnRequests. So far, I've generated a random one using dsig.RandomKeyStoreForTest(), and haven't imported this anywhere into my IDP, but everything is working smoothly - so I'm wondering what purpose it serves.
I'm assuming its for a more complex use case?

Good question (and probably something that should be better documented). SAML supports signing of AuthnRequests, but support for this among IdPs is pretty variable and I'm not aware of a specific threat model it defends against. Some IdPs simply ignore signatures on requests, but others may return an error if they get a signed request and don't have a certificate, so if you're not using it I'd suggest setting SignAuthnRequests to false, which should let you simply use a nil keystore.

Those that do support signed requests will have an option for you to upload a cert, correct? I just haven't seen it yet, thats why I ask

I'm having trouble with signing AuthnRequests. https://www.samltool.com/validate_authn_req.php thinks the signature is invalid, as does one IdP, but testshib (in the past), ipsilon and azure ad saml have accepted authnrequests.


Chinese display problem in integrated terminal

VSCode Version: 1.2.0-insider
OS Version: 10.0.10586

All Chinese characters huddle together when I open the terminal, and if I resize the window , the space will change too.

Duplicate of https://github.com/Microsoft/vscode/issues/7062


Water Sensors- DDQ

This site went completely offline on 07/13/2022. Rep 1 and 2 Bare were last to go offline on 07/13 while Rep 1 Cover went down last month and has never sent surface sensor data. Rep 2 Cover sent data for a few hours on 6/17 and then went off again.
Are the sensors not sending data connected correctly? Does this site need a reset? Other attention?

This gateway was updated with a 4G SIM card this afternoon and connected with the gateway after a reset, but has not communicated with the dashboard any time since then.
The gateway has only sent two successful time updates and a message the same as VQQ (#245):

{"new_imei": "865456055988016", "prev_imei": "014682005382220", "msg": "Device 1724909 IMEI change from<PHONE_NUMBER>82220 to<PHONE_NUMBER>88016"}
Wednesday, May 18th 2022, 2:37:58 PM


ttf-noto-emoji: wrong size

This is a github comment with emojis and ttf-noto-emoji-kagayaki:

and this is the same comment without ttf-noto-emoji-kagayaki:

Both on Firefox 51.0.1
Moreover, I think this font shouldn't be in base package.

That's really strange, I can't reproduce it. Do you get a correct rendering on http://getemoji.com/ ?
Btw, it is already in the base group.

I think this font should NOT be in base package. It should be in extra or better alone.

I've tested with the official package of noto-emoji and it occurs too. I think the bug can be in Firefox.

There is something wrong with Firefox, on Chromium it works as expected, but on Firefox I get some emojis, but not the one from Noto Color Emoji font. Are you using latest fontconfig and noto emoji PKGBUILDs ?
As I'm expecting to render emojis using the base config, I think Noto Emoji package belongs to base group.

Forget it. Firefox, since version 50, brings native emoji. I think it's a Firefox problem.


rye: 0.16.0 -> 0.17.0

Description of changes
Changelog: https://github.com/mitsuhiko/rye/releases/tag/0.17.0

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

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

1 package built:

rye


React Native compatibility

Sweet library, amazing work!
Are there any plans to make it RN compatible?

Not at the moment but it does seem interesting -- I might explore this in the future.


Executing check on client

Hello, please what is the supposed way of executing check on client, currently items are type  "trapper" so I should use some scheduler to execute script on client every minute?

Hi.
You have to run the script this way:
client$ python /etc/zabbix/scripts/mini_ipmi_lmsensors.py get "Example host"
With task scheduler or cron.
If it involves smartctl - it must be under admin privileges because of disk access.
Or, it might be more convenient to change it from the server: Template mini-IPMI v2 active -> Discovery list/CPU discovery -> Update interval.
But don't go with low values - the data might clog up. You might want to lower TIMEOUT (delay).

Hope that's solved. Let me know if you have other questions.


RTL support for DataTable

Just some very minor tweaks to remove subtle LTR bias.
We use the same arrow rotation animation in RTL and LTR, but I think
that's correct. Usually, rotations are either clockwise or
anitclockwise, which are the same in RTL and LTR. We might need to check
with someone who reads an RTL language to confirm.
Fixes #11845


Make app a bit more responsive

Suggestions for quick wins on mobile responsiveness

keep only the "m" of the logo, remove the "Moving Motivators" title
change language selector from English / Français to EN/FR
place the left menu (full screen / swap / zoom in / zoom out) as a horizontal bottom line
reduce card size to have two cards in a row

(less quick)

keep only the drawing on card recto. Tap on card > swaps to the verso where card text is displayed

Thanks for your suggestions 👍
I'm not sure about this one reduce card size to have two cards in a row 👉 it will be really small and completely unreadable unless we found another way to display the text (like your suggestion keep only the drawing on card recto. Tap on card > swaps to the verso where card text is displayed).

The best way to do it on mobile would be to change completely the card layout and opt for a list-like design...


Deploy March 2021 Release

Docs Build status updates of commit 39a181a:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-PullRequestMergeConflict] Cannot sync git repo to specified state. Please make sure the pull request is mergeable.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit b3014df:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-PullRequestMergeConflict] Cannot sync git repo to specified state. Please make sure the pull request is mergeable.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit bf5388c:
:warning: Validation status: warnings

File
Status
Preview URL
Details

sysinternals/downloads/accessenum.md
:warning:Warning
View
Details

sysinternals/downloads/diskmon.md
:warning:Warning
View
Details

sysinternals/downloads/notmyfault.md
:warning:Warning
View
Details

sysinternals/downloads/pagedefrag.md
:warning:Warning
View
Details

sysinternals/downloads/shareenum.md
:warning:Warning
View
Details

sysinternals/downloads/shellrunas.md
:warning:Warning
View
Details

sysinternals/Announce/SHA1Deprecation.md
:bulb:Suggestion
View
Details

sysinternals/Announce/SiteUpgradeDec2019.md
:bulb:Suggestion
View
Details

sysinternals/Announce/TLSDeprecation.md
:bulb:Suggestion
View
Details

sysinternals/downloads/procfeatures.md
:bulb:Suggestion
View
Details

sysinternals/downloads/psgetsid.md
:bulb:Suggestion
View
Details

sysinternals/license-faq.md
:bulb:Suggestion
View
Details

sysinternals/bread/toc.yml
:white_check_mark:Succeeded

sysinternals/downloads/accesschk.md
:white_check_mark:Succeeded
View

sysinternals/downloads/adexplorer.md
:white_check_mark:Succeeded
View

sysinternals/downloads/adinsight.md
:white_check_mark:Succeeded
View

sysinternals/downloads/adrestore.md
:white_check_mark:Succeeded
View

sysinternals/downloads/autologon.md
:white_check_mark:Succeeded
View

sysinternals/downloads/autoruns.md
:white_check_mark:Succeeded
View

sysinternals/downloads/bginfo.md
:white_check_mark:Succeeded
View

sysinternals/downloads/bluescreen.md
:white_check_mark:Succeeded
View

sysinternals/downloads/cacheset.md
:white_check_mark:Succeeded
View

sysinternals/downloads/clockres.md
:white_check_mark:Succeeded
View

sysinternals/downloads/contig.md
:white_check_mark:Succeeded
View

sysinternals/downloads/cpustres.md
:white_check_mark:Succeeded
View

sysinternals/downloads/ctrl2cap.md
:white_check_mark:Succeeded
View

sysinternals/downloads/debugview.md
:white_check_mark:Succeeded
View

sysinternals/downloads/desktops.md
:white_check_mark:Succeeded
View

sysinternals/downloads/disk2vhd.md
:white_check_mark:Succeeded
View

sysinternals/downloads/diskext.md
:white_check_mark:Succeeded
View

sysinternals/downloads/diskview.md
:white_check_mark:Succeeded
View

sysinternals/downloads/du.md
:white_check_mark:Succeeded
View

sysinternals/downloads/efsdump.md
:white_check_mark:Succeeded
View

sysinternals/downloads/file-and-disk-utilities.md
:white_check_mark:Succeeded
View

sysinternals/downloads/filemon.md
:white_check_mark:Succeeded
View

sysinternals/downloads/findlinks.md
:white_check_mark:Succeeded
View

sysinternals/downloads/handle.md
:white_check_mark:Succeeded
View

sysinternals/downloads/hex2dec.md
:white_check_mark:Succeeded
View

sysinternals/downloads/index.md
:white_check_mark:Succeeded
View

sysinternals/downloads/junction.md
:white_check_mark:Succeeded
View

sysinternals/downloads/ldmdump.md
:white_check_mark:Succeeded
View

sysinternals/downloads/listdlls.md
:white_check_mark:Succeeded
View

sysinternals/downloads/livekd.md
:white_check_mark:Succeeded
View

sysinternals/downloads/loadorder.md
:white_check_mark:Succeeded
View

sysinternals/downloads/logonsessions.md
:white_check_mark:Succeeded
View

sysinternals/downloads/media/adexplorer/adexplorer.jpg
:white_check_mark:Succeeded
View

sysinternals/downloads/media/adinsight/adinsight.jpg
:white_check_mark:Succeeded
View

sysinternals/downloads/media/autoruns/autoruns_v13.png
:white_check_mark:Succeeded
View

sysinternals/downloads/media/bginfo/BGInfo-Dialog.jpg
:white_check_mark:Succeeded
View

sysinternals/downloads/media/bginfo/BgInfo.jpg
:white_check_mark:Succeeded
View

This comment lists only the first 50 files in the pull request.

sysinternals/downloads/accessenum.md

Line 22, Column 1: [Warning-alt-text-bad-value] Alt text 'AccessEnum' is the same as the image file name. This redundancy adds no value for users of screen readers. For accessibility, add more descriptive alt text.

sysinternals/downloads/diskmon.md

Line 51, Column 1: [Warning-alt-text-bad-value] Alt text 'DiskMon' is the same as the image file name. This redundancy adds no value for users of screen readers. For accessibility, add more descriptive alt text.

sysinternals/downloads/notmyfault.md

Line 32, Column 1: [Warning-alt-text-bad-value] Alt text 'NotMyFault' is the same as the image file name. This redundancy adds no value for users of screen readers. For accessibility, add more descriptive alt text.

sysinternals/downloads/pagedefrag.md

Line 45, Column 1: [Warning-alt-text-bad-value] Alt text 'PageDefrag' is the same as the image file name. This redundancy adds no value for users of screen readers. For accessibility, add more descriptive alt text.

sysinternals/downloads/shareenum.md

Line 34, Column 1: [Warning-alt-text-bad-value] Alt text 'ShareEnum' is the same as the image file name. This redundancy adds no value for users of screen readers. For accessibility, add more descriptive alt text.

sysinternals/downloads/shellrunas.md

Line 28, Column 1: [Warning-alt-text-bad-value] Alt text 'ShellRunas' is the same as the image file name. This redundancy adds no value for users of screen readers. For accessibility, add more descriptive alt text.

sysinternals/Announce/SHA1Deprecation.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

sysinternals/Announce/SiteUpgradeDec2019.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

sysinternals/Announce/TLSDeprecation.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

sysinternals/downloads/procfeatures.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

sysinternals/downloads/psgetsid.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

sysinternals/license-faq.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


UI Plugin: Route client HTTP calls to management-cluster plugin

The existing Tanzu CLI management cluster plugin includes an HTTP server and REST layer that can be leveraged for several UI functions. Some of these functions include authentication with a cloud provider, discovery of resources on a cloud provider, and starting deployment of a management cluster.
The new TCE UI plugin should attempt to route any HTTP calls related to the above functionality from its own UI to the management cluster plugin so that existing APIs can be leveraged.
This may require that the TCE UI plugin starts the management cluster plugin's HTTP server in a browserless state at a specified IP address and port. The TCE UI plugin's backend should field HTTP requests from its own UI and then forward calls to the management cluster plugin's HTTP server that is running on the specified IP and port.

We may want to think about either duplicating (with the goal of eventually migrating?) some of this functionality from the management-cluster plugin to the new UI plugin so we don't need to launch a second web server to serve these calls. Or importing the code so we can have one library that contains the logic, but can be used by two different services.

@stmcginnis absolutely and let's maybe chat offline to see how we can go about this. Having a single library sounds ideal, and I would like to talk through how we can make that happen.


Candles in websockets?

I notice that Kraken's WS API provides ohlc but as far as I can see ExchangeSharp only exposes the REST version.
Is this a deliberate design or perhaps it's not seen as useful?
I wouldn't be surprised if there's already a utility class which will generate candles from the raw trades feed... I keep starting to write things then stumbling across existing implementations!

Streaming tickers is implemented in ExchangeSharp for many exchanges but not Kraken. If you'd like you submit an implementation as a Pull Request, we'd be happy to merge it!

@vslee I'm not referring to tickers, but to OHLC candles.
I thought that WS functionality for Kraken is supported, e.g. OnGetTickersWebSocketAsync / OnGetTradesWebSocketAsync in ExchangeKrakenAPI?
But my question is that in IExchangeAPI, there is no signature defined at all for streaming OHLC data. Perhaps KRaken is the only exchange who added this (https://docs.kraken.com/websockets/#message-ohlc) or maybe everyone builds their own candles from raw trades data so it's not considered important?
So to submit a PR I would need to add a new API method on the interface and presumably dummy versions for every other exchange, as well as doing the implementation?

I see what you're saying, ExchangeSharp only had REST support for Candles, but not Websocket support. I just made a PR which adds Websocket support for candles. After my PR is merged, you can then add OHLC support to Kraken by overriding the OnGetCandlesWebSocketAsync() method in ExchangeKrakenAPI.cs.


Fix/ROE-2558: Fix broken back-links when REDIS removal flag is turned on

Fixes broken back-links when REDIS removal flag is turned on
JIRA link
https://companieshouse.atlassian.net/browse/ROE-2558
Change description
Fixes broken back-links when REDIS removal flag is turned on
Work checklist

[ ] Tests added where applicable
[ ] UI changes meet accessibility criteria

Merge instructions
We are committed to keeping commit history clean, consistent and linear.  A BREAKING CHANGE can be part of commits of any type,
types other than fix: and feat: are allowed, for example build:, chore:, ci:, docs:, style:, refactor:, perf:, test:, and others,
footers other than BREAKING CHANGE: <description> may be provided.

Presume that tests to check correct back link is being used will be added before/when this moves out of 'draft' status?

Presume that tests to check correct back link is being used will be added before/when this moves out of 'draft' status?

Yes, I'm trying to get the journey working from end-to-end with the correct back-links in the first instance, then add the tests in after.

This PR is now ready for a full review.
cc: @dlloyd1-ch @mwestacott @markpit

See analysis details on SonarQube

Sonar is reporting a code smell and I'm also wondering if you're able to address the code coverage issue?


Add install script for swift protoc gen tools

@Natels Could you please push this to a branch on this repo (make-files/makefiles), instead of from a fork? That will allow us to test it in a real repo by using rm -rf .makefiles && MF_BRANCH=<branch name> make makefiles.

I have pushed a nate-install-swift-protoc branch to the main repo. After this PR I'll close the fork and just use branches 😃


Allow overriding default servingruntime.

As a stopgap until we fully support multiple custom runtimes, we want to allow a backend only configuration to override the default custom Runtime
Description
Fixes bug where UI would die when a custom runtime has not buildInAdapter (which is the case on any custom runtime)
Reads override-config from servingruntimes-config and uses that serving runtime if available
How Has This Been Tested?
add the following key/value to servingruntimes-config
  override-config: |
    apiVersion: serving.kserve.io/v1alpha1
    kind: ServingRuntime
    metadata:
      # metadata will be overwritten by the model's metadata
      name: ''
      namespace: ''
      labels:
        name: ''
        opendatahub.io/dashboard: 'true'
      annotations:
        openshift.io/display-name: 'Watson NLP Custom'
    spec:
      supportedModelFormats:
        - autoSelect: true
          name: chris-watson-nlp-custom
      # replicas will be overwritten by the model's replica
      replicas: 1
      protocolVersions:
        - grpc-v1
      multiModel: true
      grpcEndpoint: 'port:8085'
      grpcDataEndpoint: 'port:8001'
      containers:
        - name: watson-nlp-runtime
          image: 'us.icr.io/watson-runtime/fmaas-runtime-ansible:0.0.3'
          env:
            - name: ACCEPT_LICENSE
              value: 'true'
            - name: LOG_LEVEL
              value: info
            - name: CAPACITY
              value: '28000000000'
            - name: DEFAULT_MODEL_SIZE
              value: '1773741824'
            - name: METRICS_PORT
              value: '2113'
            - name: GATEWAY_PORT
              value: '8060'
            - name: STRICT_RPC_MODE
              value: 'false'
            - name: HF_HOME
              value: /tmp/
          resources:
            limits:
              cpu: 2
              memory: 16Gi
            requests:
              cpu: 1
              memory: 16Gi
      storageHelper:
        disabled: false

You should be allowed to generate a Watson NLP Custom runtime, although pulling that image will not currently work.  The normal configmap should also work.
fixes #904

Merge criteria:

[ ] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

@cfchase fwiw -- if you want to put your PR into WIP, prefix the title of your PR with [WIP]. I put a hold on it for now.

We can have a discussion about this, I think this won't do any harm and it could be good for the user, we need to talk with docs just to document this flow of override-config -> default-config -> fallback-config we have here.


No unit test exists testing multiple persistence units

We don't have a JPA unit test that proves that multiple persistence units can be found and injected using @PersistenceContext(unitName = "xyz").
PR coming shortly.

This issue is tangentially related to #2500.


ui revamp

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)

Closes #XXXXX

This is the fist attempt to revamp the UI
Hid the dark/light switch for now for simplicity.
Once we update the theme selector on learn we could grab the user preferences from localstorage and show it here.

we could grab the user preferences from localstorage and show it here.

I wanted to do this, but we removed it: https://github.com/freeCodeCamp/freeCodeCamp/pull/43829

Anyone notice anything wrong....

Anyone notice anything wrong....
no :)


Incorrect cardano static binary/attribute name

Followed exact directions on Centos 7.4 to build cardano bin. It fails unless the bin name is changed to 'cardano-sl-static'.
[martin@macedonia cardano-sl]$ nix-build -A cardano-sl-node-static --cores 0 --max-jobs 2 --no-build-output --out-link master
error: attribute ‘cardano-sl-node-static’ in selection path ‘cardano-sl-node-static’ not found
[martin@macedonia cardano-sl]$
'cardano-sl-node-static' may have been valid at some point, but no longer.

Closing as this PR is old and it seems to solve nonexisting problem (as of today at least - release.nix contains cardano-sl-node-static entry)
@MartinMKD did you try to use current develop branch? Are you having the same problems there? If yes, we will reopen and investigate the issue


Update user profile in developer guide

Codecov Report
Base: 72.15% // Head: 72.15% // No change to project coverage :thumbsup:

Coverage data is based on head (2cada48) compared to base (f8feaa3).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #55   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Ruby Homebrew installer is now deprecated and should be migrated to Bash

Cookbook version
5.0.8
Chef-client version
15.8.23
Platform Details
mac_os_x
Scenario:
Error executing action run on resource 'execute[install homebrew]'
STDERR: Warning: The Ruby Homebrew installer is now deprecated and has been rewritten in
           Bash. Please migrate to the following command:
             /bin/bash -c "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install.sh)"

also hitting this issue which is a blocker for users.

@ehanlon and @bcg62 you can workaround as follows:
url_sh = 'https://raw.githubusercontent.com/Homebrew/install/master/install.sh'
node.default['homebrew']['installer']['url'] = url_sh
include_recipe 'homebrew'

HTH?

Fixed by #140


solve issue dockerfile

issue that with remotes::install_github could not find the repo should be resolved. Honestly, no idea why it decided to work again...

Codecov Report
Patch and project coverage have no change.

Comparison is base (f100f18) 95.01% compared to head (04ce191) 95.01%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #25   +/-   ##
=======================================
  Coverage   95.01%   95.01%
=======================================
  Files          14       14
  Lines         662      662
=======================================
  Hits          629      629
  Misses         33       33

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[FEATURE] make the entire issue card clickable and add hover to time since issue creation

Is your feature request related to a problem? Please describe.
When clicking anywhere on the issue card expect the title or the repo link nothing happens. But the hover efect indicates otherwise.
Describe the solution you'd like
Make it so that if you click anywhere on the card (blue marked in the image) not only the title you will be redirected to the issue page and if you click on the repo link (marked red) you will be redirected to the repo page. And it would also be great that if you hover over the time it would show the exact time the issue was created and not only the time elapsed since it was created.
Additional context

@SpatzlHD, would you like to work on it?

@SpatzlHD, would you like to work on it?

Yes. That shouldn't take much time and is a good start for me to work a bit more with the project before starting #46

ok I'm assigning this to you

@anand346 I raised a pr #55

This issue is fixed in #55


[refactor: metrics] fix agent metrics init bug

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #2913 (058c4b1) into master (8f53662) will decrease coverage by 0.48%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2913      +/-   ##
============================================
- Coverage     59.97%   59.49%   -0.49%
+ Complexity     4465     4424      -41
============================================
  Files           693      693
  Lines         20015    20012       -3
  Branches       1785     1784       -1
============================================
- Hits          12004    11906      -98
- Misses         7003     7100      +97
+ Partials       1008     1006       -2

Impacted Files
Coverage Δ

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
68.75% <0.00%> (-31.25%)
:arrow_down:

...henyu/admin/service/impl/UpstreamCheckService.java
42.85% <0.00%> (-20.64%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
70.37% <0.00%> (-13.34%)
:arrow_down:

...yu/sync/data/http/refresh/AbstractDataRefresh.java
70.00% <0.00%> (-13.34%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
52.12% <0.00%> (-11.71%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

...apache/shenyu/admin/utils/ShenyuDictH2Trigger.java
63.63% <0.00%> (-9.10%)
:arrow_down:

...ache/shenyu/admin/utils/PluginHandleH2Trigger.java
63.63% <0.00%> (-9.10%)
:arrow_down:

.../apache/shenyu/admin/model/dto/BatchCommonDTO.java
50.00% <0.00%> (-7.15%)
:arrow_down:

...apache/shenyu/admin/model/dto/AuthPathWarpDTO.java
57.14% <0.00%> (-7.15%)
:arrow_down:

... and 27 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8f53662...058c4b1. Read the comment docs.


chore(main): release 0.4.0

:robot: I have created a release beep boop
0.4.0 (2024-02-28)
⚠ BREAKING CHANGES

return pwd() or source.file dir if we reach file system root
use recursive search logic when in REPL (interactive)

Features

return pwd() or source.file dir if we reach file system root (b559f8a)
support expressions in [@projectroot](https://github.com/projectroot) (b8e302e)
support string interpolation (8a48c81)
use recursive search logic when in REPL (interactive) (415c517)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/jolars/ProjectRoot.jl/releases/tag/v0.4.0 :sunflower:


bug: util.file.is_readable_directory() does not work on Termux

On Termux, the /sys/class/power_supply exists and "is readable", except it, well, isn't.
$ stat --format='%A' /sys/class/power_supply
drwxr-xr-x
#      ^ Everyone can read it

$ ls /sys/class/power_supply
ls: cannot open directory '/sys/class/power_supply': Permission denied
#                                                    ^^^^^^^^^^^^^^^^^ I can't read it?

I have found a method that I think works on both regular Linux and Termux (not Windows though):
$ test -d /sys/class/power_supply && test -r /sys/class/power_supply
#      ^^ Is directory                    ^^ Is readable

This is communicated through the exit code, which is returned by the os.execute function (I think Neovim uses Lua 5.1, so this version of os.execute only returns the exit code)
function is_readable_directory(file)
  return os.execute('test -d '..file..' && test -r '..file) == 0
end

The problem with this is that the input must be trustworthy, because the file could be something like /some/file; rm -rf /

I have created a draft pull request with some proposed changes

Interesting. Since termux is a special case, is there an environment flag we can use to detect Termux and then use a different code path for that?

There is a $TERMUX_VERSION environment variable, and I am looking into whether that is a reliable way of detecting if it is running in Termux

I have made it so that it checks whether it is in Termux, and will use the "old" or "new" version of the function based on that


What's the license for technically-php/callable-reflection?

Can't find it in the repo, nor in de header of the sources files themselves...

Hey @Mrten!
Sorry, I have somehow missed this issue for too long.
The license is MIT, as it was stated in composer.json since day 1: https://github.com/technically-php/callable-reflection/blob/2a6bfe8b739fcedb66ca2d031178af4726c70b02/composer.json#L5
I've added an explicit LICENSE file to leave no room for confusion.
Thank you for reporting it :heart:, and apologies for this taking so long :sweat:

Please, no worries! Thank you for the inspiration!


Support need while trying to run a predicition.

Hey!
Basicly I am trying to train my first model which itself work but now I have Issues with the translator, I continue to get a "The array has not been initialized", and I am unsure how to fix this Issue. I am thankfully for every kind of help!
Current Translator:
        Translator<Image, Classifications> translator = new Translator<>() {

        @Override
        public Classifications processOutput(TranslatorContext ctx, NDList list) {
            // Create a Classifications with the output probabilities
            NDArray probabilities = list.singletonOrThrow().softmax(0);
            List<String> classNames = IntStream.range(0, output).mapToObj(String::valueOf).toList();
            return new Classifications(classNames, probabilities);
        }

        @Override
        public NDList processInput(TranslatorContext ctx, Image input) {
            NDArray array = input.toNDArray(ctx.getNDManager(), Image.Flag.COLOR);
            return new NDList(NDImageUtils.toTensor(NDImageUtils.resize(array, 128 * 128)));
        }

        @Override
        public Batchifier getBatchifier() {
            return Batchifier.STACK;
        }
    };

Model Creator:
`        long inputSize = 128 * 128 * 3;
    SequentialBlock sequentialBlock = new SequentialBlock();

    sequentialBlock.add(Blocks.batchFlattenBlock(inputSize));
    sequentialBlock.add(Linear.builder().setUnits(128).build());
    sequentialBlock.add(Activation::relu);
    sequentialBlock.add(Linear.builder().setUnits(64).build());
    sequentialBlock.add(Activation::relu);
    sequentialBlock.add(Linear.builder().setUnits(outputSize).build());

    Model model = Model.newInstance("mlp");
    model.setBlock(sequentialBlock);`

Trainer:
`        trainer.initialize(new Shape(128 * 128 * 3));
    EasyTrain.fit(trainer, 2, dataset, null);

    saveModel(model);`

If more of my code is required to understand and find the cause of the Issue, I am gladly read to share it.

I have now uploaded my Project on Github, if that helps in any kind of way.
https://github.com/DxsSucuk/PissAI

@DxsSucuk
After trained and saved your model, you need load the parameters into your model for inference.
See our inference example:
https://github.com/deepjavalibrary/djl/blob/master/examples/src/main/java/ai/djl/examples/inference/ImageClassification.java#L58-L63

Thanks it helped and yeah I just forgot to load the model data!


[Feature Request] Remember grouping status

I'm not sure it is possible but can you remember the grouping status, as in who queued with whom from the loading screen and display it in-game afterwards? I often forget but like to know mid-game.

Party detection is possible bot not yet implemented. I provided more details in this issue. I will do it at some point but it's not reliable.


selinux-policy: Fix issues in patches.

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[ ] The toolchain has been rebuilt successfully (or no changes were made to it)
[ ] The toolchain/worker package manifests are up-to-date
[ ] Any updated packages successfully build (or no packages were changed)
[ ] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[ ] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[ ] All package sources are available
[ ] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[ ] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[ ] All source files have up-to-date hashes in the *.signatures.json files
[ ] sudo make go-tidy-all and sudo make go-test-coverage pass
[ ] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary

What does the PR accomplish, why was it needed?
Change Log

Fix missing role associations in cloud-init patch.
Fix missing require in mkinitrd patch.

Does this affect the toolchain?

NO
Test Methodology

Pipeline build id: 382541

cc @cwize1

LGTM


Revert "[Frontend] Factor out code for running uvicorn"

Reverts vllm-project/vllm#6828
@DarkLight1337, we need to revert this and reapply this later to accelerate #6797 to be landed this week. Sorry about the back and forth.

looks like you need to rerun mypi

i'm actually confused why this failed.

I'm guessing that vllm/server was explicitly added to the list of things for mypy to check with a later commit


D3D12: Ability to inspect root signatures

Description
Root signatures are very important in D3D12 and provide bindings from the shader to the currently bound descriptor sets. RenderDoc only allows you to inspect resource bindings from the pipeline, but it would be handy to also be able to view root signatures. Currently it only shows the size of the blob.
There's a few functions that might be of use to you:

D3D12CreateRootSignatureDeserializer
GetRootSignatureDesc

However, I'm not certain if you can obtain the root blob from a created root signature.
Steps to reproduce
Run a D3D12 app and try to inspect the root signature with the resource inspector.
Environment
Win10 x64 D3D12

Yes root signatures are possibly the ugliest wart in D3D12. If they were created through a normal set of structs it would be visible in the resource inspector, but since it's an opaque blob that's all you can see.
A proper implementation would require significant custom UI (either in a separate panel or API-specific-type customisations to the resource inspector) just for viewing root signatures. It's not impossible of course, but as with some other feature requests this might not be worth the implementation and maintenance cost.
I think it will be possible I can add an ugly hack to simulate what it would look like if the root signature API wasn't awful and it was created with a proper descriptor instead of a blob. It won't be very user-friendly but at least the information will be accessible in some form.
I know it's not exactly what you're asking for and it seems like you're aware of it anyway, but RenderDoc does show which root signature element each binding comes from which does give you the relevant parts of the mapping:

Which shows a root CBV at element 1, and two SRV tables at elements 2 and 3.

The problem turned out to be because SRV/UAV/CBV descriptor heaps with shader visiblity are GPU only (so you have to copy them from a different heap). Though it would have been handy to be able to see all root signature parameters in the pipeline (and maybe color them differently if they aren't used), to make sure I knew the root signature was correct.

What problem? I don't understand what you're referring to because you didn't describe what you were trying to investigate in the original report, and I'm not sure what shader visible heaps have to do with examining root signature parameters.
RenderDoc can show all parameters in the root signature. By default unused items are hidden, but there is a toggle Show unused items in the pipeline state which will show any bound-but-unused items from the root signature.

Sorry, I thought my root signature was invalid since it said it couldn't find resources for the pipeline. I inspected the descriptor heap and it had valid data. Problem was that it was marked as "shader visible" aka GPU access only and so the CreateShaderView didn't do anything, even though it showed up in RenderDoc in the creation info (CPU side had data but was never pushed to GPU).
I never knew that's an option, I'll keep that in mind, that's very nice. Regardless, some time in the future an ability to see root signatures and compare them would be nice nonetheless.

That doesn't sound right, it's definitely valid to call CreateShaderResourceView on a shader-visible heap. From the docs:

DestDescriptor (Type: D3D12_CPU_DESCRIPTOR_HANDLE)
Describes the CPU descriptor handle that represents the shader-resource view. This handle can be created in a shader-visible or non-shader-visible descriptor heap.

It sounds like perhaps your application was hitting invalid behaviour somewhere. RenderDoc does not handle validation as that's the domain of the API-specific validation layers.
As mentioned above I'll see if I can hack something to view root signatures, however it sounds like you might want to make sure you're running with API validation. I'm not sure what you mean by comparing them but unless you mean just comparing by eye then that's a separate feature entirely.

Yup, it seemed like it is; but it just didn't send it to the GPU. You have to call CopyDescriptors from a CPU heap to a GPU visible heap. It works now that I do that, but it seems a bit strange to me as well. I am using the validation layer, maybe this is just something they don't report but should; or an invalid driver.

 However you can also create the descriptors directly on a shader visible heap and not do any copy at all, that is valid and does work. If you think that it's not working for you then it's likely that your application has a bug and introducing the copy just avoids the bug.
This has nothing to do with the original request and it doesn't sound like a RenderDoc bug, I'm just pointing this out because it sounds like you still don't quite have things right and you might have a bug lurking somewhere.

I understand, but why would the copy get around it? The only thing I do different in the one that works is create a new descriptor heap that is shader visible and copy the one that isn't to it and then release the old one. It's fine if I don't do that step for RTV and DSV heaps, but otherwise it doesn't work for me. Is there a flush I need to call after filling the CPU-visible heap, that is being done by the copy?

It's impossible for me to know without seeing what your code is doing, and as this does not seem to be a RenderDoc bug this issue tracker is not the right place to ask for help debugging your application. I only wanted to let you know that you were under the mistaken impression that it was impossible to create descriptors directly on a shader visible heap. No copy is necessary, so whatever is going wrong must be something else.

That commit adds a fake extra parameter to the details of CreateRootSignature which contains the unpacked/decoded blob, which allows root signature inspection:

Amazing, looks good!


How to properly search for latest successful hydra build snapshot for a package?

Issue description
haskellPackages.brittany on master is currently failed from missing hspec dependency.
$ nix-shell -I nixpkgs="https://github.com/NixOS/nixpkgs/archive/master.tar.gz" -p haskellPackages.brittany

Setup: Encountered missing dependencies:
hspec >=2 && <2.6

builder for '/nix/store/jj7y7l2fdjy17m82chnvrwq4izyzcqd6-multistate-<IP_ADDRESS>.drv' failed with exit code 1
cannot build derivation '/nix/store/sx3b52dsl4lanvxaplac91b7ygx9gqvh-brittany-<IP_ADDRESS>.drv': 1 dependencies couldn't be built
error: build of '/nix/store/sx3b52dsl4lanvxaplac91b7ygx9gqvh-brittany-<IP_ADDRESS>.drv' failed

I try to search for haskellPackages.brittany on https://hydra.nixos.org/ but there are too many results and I don't know how to read it.
I pick the top one from 18.09, is it the latest one?
$ nix-shell -I nixpkgs="https://github.com/NixOS/nixpkgs/archive/52571d750de0f90205db62afa50dbe723e42be33.tar.gz" -p haskellPackages.brittany

[nix-shell:~]$

Questions

If I want to use the latest success build of a package, how should I do a search for it?
I can't find any documentation on this issue, is there any that I miss?
Is there any tool that do the search like this but for shell.nix instead of a single package (searching for latest snapshot which able to build the entire expression build dependencies )?

I hope this is what you search for:
https://hydra.nixos.org/job/nixos/trunk-combined/nixpkgs.haskellPackages.brittany.x86_64-linux/latest
Just replace haskellPackages.brittany with any other attribute to get latest Hydra build.


Not perfectly alligned course tags. Unwanted margin between course image and course tags

Again I would love to work on this issue.

Hi, i fix this this trouble. But I can't commit
git push origin master remote: Permission to ishandeveloper/Coursify-hacktoberfest.git denied to objNameGit. fatal: unable to access 'https://github.com/ishandeveloper/Coursify-hacktoberfest.git/': The requested URL returned  error: 403


[Candidate 3v3 Nocky Mountains ZH v2] Top left Supply Piles cannot be driven into, but bottom right Supply Piles can not

Observed behaviour
Top left Supply Piles cannot be driven into, but bottom right Supply Piles can not.
Expected behavior
Mirrored Supply Piles offer equal space for unit mobility.
Screenshots

fixed


Update SECURITY.md

Change 1.18 to 1.19. I know this one is small but it makes me feel good.

I didn't test the code. If you have anything to change or something else just comment on this and I will read it.


Can't login to app

Today I 've trying to login, but it return the error code like this:
Exception in PokemonData.php line 63:
Incorrect wire format for field 1, expected: 1 got: 2

Your code maybe ? ^^

I think is same problem with google auth #142

I get approximately the same error. Only one of my user tho, both with token and user/pass. It has worked for a long time, with no changes to the code.
Incorrect wire format for field 1, expected: 1 got: 0

Same problem as #142.


VM service does not hot reload non-main Isolates

Using the vm service package it should be possible to use the reloadSources to hot reload an Isolate from changes made to the src file on disk.
With current master of the Dart SDK this seems to work fine using the DartVM in JIT mode on the commandline with the main isolate but not with any isolate's spawned from the main isolate.
This seems to have been reported in several different Flutter issues:

https://github.com/flutter/flutter/issues/18600
https://github.com/flutter/flutter/issues/20743

This was supposedly fixed for Flutter in this change as part of https://github.com/flutter/flutter/pull/52149 but I don't see how that could actually be working at the moment as I am seeing that it  doesn't work with just the standalone Dart SDK.
I created a modified version of the recharge package that demonstrates this issue.
To reproduce:

clone the above repo and use the background-isolate-test branch, run: dart  --enable-vm-service example/hello.dart
edit hello.dart changing the value returned by mesg() and the print inside worker()
note the pid printed out from step 1 and send it a sigusr1, eg kill -n 10 <pid-of-example>
notice that the output will show the new value of mesg() but NOT of the worker's print()

The version I have been testing with is built locally:
dart --version
Dart SDK version: 2.12.0-edge.30c4006d52fad4ee8b92025933b9f17af7a8740f (be) (Thu Dec 31 00:48:50 2020 +0000) on "linux_x64"

This is due to https://github.com/dart-lang/sdk/issues/36648. We intend to solve as part of https://github.com/dart-lang/sdk/issues/36097

This is due to https://github.com/dart-lang/sdk/issues/36648. We intend to solve as part of https://github.com/dart-lang/sdk/issues/36097

Just to clarify - the issue demonstrated above with recharge package is not that non-main isolates are not hot-reloaded, rather that after a hot-reload, isolates spawned after_that(worker isolate in the repro above) don't have that hot-reload updates, instead spawned with the original source.

This is due to #36648. We intend to solve as part of #36097

Those two issues have been resolved. I just tried the repro recipe above, and confirmed that it no longer reproduces — at step 4, I see the new version of both strings.
That's with:
$ dart --version
Dart SDK version: 2.18.6 (stable) (Unknown timestamp) on "linux_x64"

So I believe this issue can be closed.

yes sorry I forgot about this issue.
I just retested myself with 2.18.6 on linux and concur its now fixed so I'll close this now.


The package doesn't support null safety

I am trying to use your package but I am getting this error when I try to run it:
Error: Cannot run with sound null safety, because the following dependencies
don't support null safety:

 - package:master_detail_scaffold

For solutions, see https://dart.dev/go/unsound-null-safety
Unhandled exception:
Bad state: Unsupported Null Safety mode NonNullableByDefaultCompiledMode.Invalid, in null.
#0      ProgramCompiler.emitModule (package:dev_compiler/src/kernel/compiler.dart:439:9)
#1      JavaScriptBundler.compile (package:frontend_server/src/javascript_bundle.dart:146:33)
#2      FrontendCompiler.writeJavascriptBundle (package:frontend_server/frontend_server.dart:638:47)
<asynchronous suspension>
#3      FrontendCompiler.compile (package:frontend_server/frontend_server.dart:549:9)
<asynchronous suspension>
#4      listenAndCompile.<anonymous closure> (package:frontend_server/frontend_server.dart:1119:11)
<asynchronous suspension>
the Dart compiler exited unexpectedly.

Does this have a timeline? @MaikuB?

I took a look on the code and tried to make some improvements (only on legibility and duplicity), and also fixed the null safety issue on my fork, if you are interested on maintain it

Is this package no longer maintained? @MaikuB, would save a whole lot of issues in the future if the change @aquintiliano made on the fork were merged. Can he create a PR on those changes?
In the meantime @aquintiliano could you please send me the link to your fork, that would be really appreciated coz this package is the last one I need to fully migrate to null safety.

sure, https://github.com/alandoni/emoji_picker
the code remains almost the same, I tried to not change everything haha
this is how I am using it:
 LayoutBuilder(
    builder: (context, constraints) => IntrinsicHeight(
      EmojiPicker(
          bgColor: backgroundLightColor,
          decoration: BoxDecoration(
             border: Border.all(color: borderColor),
          ),
          columns: constraints.maxWidth > 768
            ? 15
            : 6,
          defaultCategoryColor: backgroundLightColor,
          defaultCategorySelectedColor: highlightedSelectedColor,
          defaultCategoryIconColor: iconColor,
          defaultCategoryIconSelectedColor: primaryColor,
          indicatorColor: primaryColor,
          progressIndicatorColor: primaryColor,
          onEmojiSelected: (emoji, category) { },
      ),
   )
)

omg, I made a mistake, the code I have changed was another repo haha, this one I just made the adjustment on the null safety actually, sorry
but the repo is here: https://github.com/alandoni/master_detail_scaffold


Implementation of capabilities ARKit

@Robbendebiene   I created again the Flutter project from scratch because I realized it wasn't setup right, then when I wanted to use it inside the Example project does not work. The correct setup is as a  Flutter Plugin, because of this new configuration some names will change but not work as it is supposed to.


Remove Deref implementation and other refactorings

Deref no longer makes sense for LineBuffer since it can have multiple lines.

Looks good


Start From Template that does not have a matching UseCase

If we can't find the proper useCase then we should extract all the set details (seeks and content) from the "fromNeed" and display their respective pickers instead (if available) ->
if there is content set that does not contain a picker that content should be left unchanged
this should not be happening a lot though, be aware that the details to choose From might hide within a specific useCase (not all the available details are within the customUseCase)
requires #2715

it seems that we do not have a atoms that match multiple usecases anymore, this is no longer relevant


Enabling the interruptions

The datasheet explained that we can write into the Interrupt Configuration Register to enable interrupts. And then read into the Interrupt Status Register.
Is that possible to make that from your library ?

Have to investigate how to keep it simple.
And find some time ☺

Have to investigate how to keep it simple.
And find some time ☺

Straightforward functions are quite easy but possibly allow people to "break" the config too.
I can make a separate branch so you can test, ok?

Straightforward functions are quite easy but possibly allow people to "break" the config too.
I can make a separate branch so you can test, ok?

@Harian-Elyoth
Apparently the functions are already in the library - I didn't recall that they were public.
So, you should use the functions, they are already in the library.
  uint8_t  setRegister(uint8_t reg, uint8_t value);
  uint8_t  getRegister(uint8_t reg);

Details can be found in the PDF  to be found with - Google "mpu-6000 and mpu-6050 register map and descriptions revision 4.2"

@Harian-Elyoth
Apparently the functions are already in the library - I didn't recall that they were public.
So, you should use the functions, they are already in the library.
  uint8_t  setRegister(uint8_t reg, uint8_t value);
  uint8_t  getRegister(uint8_t reg);

Details can be found in the PDF  to be found with - Google "mpu-6000 and mpu-6050 register map and descriptions revision 4.2"

@Harian-Elyoth
https://github.com/RobTillaart/GY521/tree/develop branch has a file with register-names.
This makes working with registers a bit easier as you can use names instead of numbers.
Develop might be merged later this week.

@Harian-Elyoth
https://github.com/RobTillaart/GY521/tree/develop branch has a file with register-names.
This makes working with registers a bit easier as you can use names instead of numbers.
Develop might be merged later this week.

Thanks ! I'll test that out soon

Thanks ! I'll test that out soon

Branch with register names merged and new release made 0.2.3

Branch with register names merged and new release made 0.2.3

Please reopen if there are still problems.


Add ability to bind to attachments property

Unless I'm mistaken, there's currently no way to bind to the attachments property. This would be useful to declare styles for Uranium controls that use attachments.
Currently you have to add the attachment to every instance of the UI control like so:
<mtrl:PickerField
    Title="My Picker"
    ItemsSource="{Binding Items}">
    <mtrl:PickerField.Attachments>
        <Image
	    Margin="0,0,4,0"
	    Source="expand.png"/>
    </mtrl:PickerField.Attachmets>
</mtrl:PickerField>

It would be useful to instead be able to define a style like this:
<Style
    x:Key="PickerWithAttachment"
    TargetType="mtrl:PickerField">
    <Setter Property="Attachments">
        <Image
            Margin="0,0,4,0"
            Source="expand.png"/>
    </Setter>
</Style>

But when I try this I get the following error:

Cannot resolve property "Attachments" on type "PickerField (property missing or missing accessors)".

It seems it's not possible right now, it's a simple IEnumerable<IView> and the same instance should be used always. It's not a settable field.
https://github.com/enisn/UraniumUI/blob/6c2d666149e7bddb4ed4e1bd3babb77dd11e3fd8/src/UraniumUI.Material/Controls/InputField.cs#L70


Add missing dependencies

Signed-off-by: Koenig Fabian (DC/PAR<EMAIL_ADDRESS>

Great, now it also compiles on Ubuntu 22.04; I tested it in a VM.


5.8: set relatedDependenciesBranch to 5.8 in Package.swift

This makes sure that dependencies of Swift Compiler Driver built with SwiftPM and coming from the Swift project use release/5.8 branch.

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test macOS


Scrollbar visibility fix for when auto hide is disabled

Describe your changes
Currently, when ScrollViewBar::setAutoHideEnabled(false) is called to disable the auto-hide of the scroll bar, the scrollbar opacity is set to 0 under certain conditions, when that should not be the case.  The logic in that method seems to be incorrect.
Current implementation:
void ScrollViewBar::setAutoHideEnabled(bool autoHideEnabled)
{
    _autoHideEnabled = autoHideEnabled;
    if (!_autoHideEnabled && !_touching && _autoHideRemainingTime <= 0)
    {
        ProtectedNode::setOpacity(_opacity);
    }
    else
        ProtectedNode::setOpacity(0);
}

If _autoHideEnabled is false, then it should not be checking for _touching or _autoHideRemainingTime, as they would only be relevant if _autoHideEnabled is set to true.
This should be the correct implementation:
void ScrollViewBar::setAutoHideEnabled(bool autoHideEnabled)
{
    if (_autoHideEnabled == autoHideEnabled)
        return;

    _autoHideEnabled = autoHideEnabled;
    if (!_autoHideEnabled)
    {
        ProtectedNode::setOpacity(255); // No point passing _opacity since it is an unknown value, so set it to 255 by default
    }
    else if (!_touching && _autoHideRemainingTime <= 0)
    {
        ProtectedNode::setOpacity(0);
    }
}

If the user requires a different opacity than 255, then it would simply be this call order:
ScrollViewBar::setAutoHideEnabled(false);
ScrollViewBar::setOpacity(requiredOpacity);

Issue ticket number and link
Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[x] I have checked readme and add important infos to this PR (if it needed).
[ ] I have added/adapted some tests too.

Actually, would a default opacity of 128 be better instead of 255 for when auto-hide is disabled?

Would a default opacity of 128 be better instead of 255 for when auto-hide is disabled?

I don't think so.

All good, then this PR is complete, and fixes the bug that has been in there since Cocos2d-x v3, unless you would want me to add a test case for it.


Two versions of localStorate

Just leaving this here for anyone using this plugin and having issues with localStorage not being consistent between page navigations (location.href changes). We were having this issue for quite a while in our app and as a workaround (hack) opted in to save entire localStorage to sqlite and restore from it when the next page loads. There might be already a ticket for the issue on the issue tracker and possibly a solution how to avoid this condition but on a quick search I could not find one there.
I tested the case in isolation on a simple cordova  (HelloWorld app) and was able to recreate the problem on and iOS 9.3, iOS 10.2 but not on iOS 12.

since the files are loaded from file protocol, the webview can think the origin changed and cause this problem.
now you can move to cordova-ios 6 and use a custom scheme, that way the origin won't change and you won't have this problem

@jcesarmobile, would you please elaborate what you mean by "use a custom scheme"? Do you mean using cordova-plugin-customurlscheme ?

Please, don’t ping me.
And please, read the cordova iOS 6 release notes
https://cordova.apache.org/announcements/2020/06/01/cordova-ios-release-6.0.0.html


Excerpt only flag

Create new flag excerpt_only, which allows to exclude post content from feed.
PR to issue: https://github.com/jekyll/jekyll-feed/issues/277
BTW I don't like name excerpt_only, but don't have better idea. IMO this name should get true value for full content and false value for excerpt.

@torrocus First of all, apologies for not reviewing this earlier.
While I don't have an issue with the idea, I'm on the fence regarding the implementation.
Since Parker has already approved this, I won't block this from being merged but would love it if you could clarify the following / make some changes:

| default: false }} is unnecessary. The subsequent conditional is valid even if excerpt_only evaluates to nil.
Why conditionally assign post_content and then immediately test post_content for a truthy? Why not render directly?{% unless excerpt_only %}
  <content type="html" xml:base="{{ post.url | absolute_url | xml_escape }}">
    {{ post.content | strip | xml_escape }}
  </content>
{% endunless %}

@parkr Thanks for dig.
@ashmaroli No problem, it's good to know someone is coming here.
Regarding your attention with | default: false }} you're right. I deleted the redundant default value. I wanted to show what the default value is. But later I included this information in the documentation and it should be enough.
Your second point is not that simple. Post does not always have content. The question is is this correct? View to file spec/fixtures/_posts/2015-02-12-strip-newlines.md
Personally, I would prefer render directly. But I do not know the whole context and did not want to interfere with existing functionalities.

Post does not always have content

To my knowledge, the content attribute is never nil or anything other than a String. It can be just empty "" but in Ruby, that is considered a truthy as well.
Of course, having a test for this would make things clear and concrete.

Right, I corrected it. Previously, one test failed. Now everything is OK, and I already had this code written in this way.

Please squash all commits when you will merge.

@parkr Since the diff changed significantly after you gave your approval, could you do a re-review?
Thanks.

@jekyllbot: merge +minor


Cursor changes automatically between different selects

Hello
I have a page that uses two different select2 and, for some reason, if you try to write in one of them , it will automatically change to the other select and keep writing there. This doesn´t happen if I write in the other one.
I have tried to change manually the focus, but it doesn´t work.
I don´t know what causes this.
Thanks.

This appears to be more of a usage question than a bug report or common feature request.  As we continue to keep the Select2 issue tracker geared towards these two areas, bug reports and common feature requests, we are redirecting usage questions to other communities such as the mailing list, IRC, or Stack Overflow.


Easy hack

This application it doesn't work

easy hack please add search engine for exploitation  in the next update

مش عارف اعمل حاجه


Regression in 0.4.14+ 403 while reporting status

Suspected reason is our move to tokenless flow to no longer depend on digger-hostname and digger-token and digger-organisation, however it now leads to an error of:
Found settings sent by backend in job string, overriding backendApi and policyCheckecd r. setting: (orgName: digger BackedHost: https://*** token: ****)

Failed to report job status to backend. Exiting. unexpected status when reporting a project job status: 403

Impacting PR: https://github.com/diggerhq/digger/pull/1330


chained fk doesn't work if it's parent field is not editable

I have a model that has few fields set as list_editable, one of which is a chained fk, but it's parent field isn't. The fk field doesn't display any choices - however it works on a form view, where all fields are editable. So it probably has something with retrieving "parent" id/fk.
Just raising the issue now - when I have some time I'll try to add some more useful input.

Is this still an issue?

Closing due to inactivity.


Add doesNotThrowAnyExceptionExcept to AbstractThrowableAssert

Check List:

Fixes #3248
Unit tests : YES
Javadoc with a code example (on API only) : YES
PR meets the contributing guidelines

Following the contributing guidelines will make it easier for us to review and accept your PR.

Hi @joel-costigliola
Do you have any comments regarding the wording of javadoc / error message?

@joel-costigliola @scordio
Is there a plan to add this to any of the milestones?

Done @pbacz 😉

Integrated thanks @pbacz!


feat: changed port 53 -> 822

@aj-may please add @rmjames to this repo

This change needs to be only for users using MacOS. Linux distros don’t allow for dns servers running on non standard ports. Additionally we need to change the resolver config file that is written to contain the port. @jtlacci please share my notes. I sent you a lot of info on this.

@codfish great input

AJ's comments:
https://github.com/aj-may/dotdocker/blob/master/src/containers/dnsmasqConfig.js
src/containers/dnsmasqConfig.js
export default {
Image: 'andyshinn/dnsmasq:latest',
name: 'dotdocker-dnsmasq',
Cmd: ['--address=/docker/<IP_ADDRESS>', '--log-facility=-'],
ExposedPorts: {
this will need to be updated to be a function that returns the josn object so that we can pass in the desired port number
and then you will need to refactor the command to use different ports based on distro
53 on linux
random high port on macos
and then update the resolver config file that gets written on macOS to specify port number
ie.
➜ cat /etc/resolver/docker
Generated by dotdocker
nameserver <IP_ADDRESS>
port ${randomNumber}

https://github.com/aj-may/dotdocker/tree/master/src/tasks/setupDNS
this is an example of how i did that kind of distro spicific stuff for the config file
might make sense to do something similer
but also might be overkill

@jtlacci @rmjames any updates on this? Let me know if you have any questions on my notes.