Unable to estimate gas on "Creating a new 1155 contract and token with Secondary Markets"

Hi! I'm facing this issue:
Code ref: https://docs.zora.co/protocol-sdk/creator/onchain#creating-a-new-1155-contract-and-token-with-secondary-markets

 useEffect(() => {
    (async function () {
      if (!wallets[0]) return;
      const provider = await wallets[0].getEthereumProvider();
      if (provider) {
        setProvider(provider);
        console.log("executed");
        // await wallets[0].switchChain(iliad.id);
        // console.log("executed");
        const walletClient = createWalletClient({
          transport: custom(provider),
          chain: baseSepolia,
        });

        const publicClient = createPublicClient({
          transport: custom(provider),
        });

        setWalletClient(walletClient);
        setPublicClient(publicClient);
      } else {
        console.error("Please install MetaMask!");
      }
    })();
  }, [ready, authenticated, wallets]);

// contract deployment
  const test = async () => {
    if (!publicClient) {
      console.error("Public client is not defined");
      return;
    }

    console.log(chainId, "chainId");
    const creatorClient = createCreatorClient({ chainId, publicClient });

    const { parameters, contractAddress } = await creatorClient.create1155({
      // the contract will be created at a deterministic address
      contract: {
        // contract name
        name: "testContract",
        // contract metadata uri
        uri: "ipfs://DUMMY/contract.json",
      },
      token: {
        tokenMetadataURI: "ipfs://DUMMY/token.json",
      },
      // account to execute the transaction (the creator)
      account: loggedInAddress!,
    });

  //  const { writeContract } = useWriteContract();
    const tx = await walletClient?.writeContract({
      ...parameters,
      account: loggedInAddress as `0x${string}`, // Ensure loggedInAddress is properly set
      chain: baseSepolia,
    });
    console.log(tx, "tx");

  //  writeContract(parameters);
  }

this is usually caused by an error in the contract call.  I would suggest simulating the contract call first with await publicClient.simulateContract - this will throw an exception with info in the error.

Another common reason this error comes up is if you have no balance on the contract.


gpt-4 support

Thank you for the package!
Are you planning on supporting /v1/chat/completions and gpt-4 too?
Thank you!

Assistant and thread are also missing

@Jany-M GPT-4  is supported basically for use.

@HamzaYslmn see #112


Implement silent deploy & upgrade

This PR implements the option to silently deploy and upgrade the AOE and allows as such for a more automated deployment or upgrade. I saw that the SilentDeploymentSettingsPath parameter was already defined but never implemented. This PR does just that.
At the same time, it implements a SQL password validation (only used with the silent deploy option at this time) to catch possible errors before deployment.
Fee free to provide additional remarks or suggest possible optimizations before merging.
One possible optimization is to generate a random SQL password instead of providing it in the deployment file and storing this password immediately in an Azure Key Vault. But I prefer to not over complicate the PR and would be willing to implement via a new PR if valuable.

Many thanks for your valuable and appreciated contribution, @wpouseele. Silent deployments were a pre-requisite for a GitHub Actions AOE deployment pipeline I hadn't had the time to implement so far. With your help, this will become a reality sooner and the community may implement their own upgrade pipelines at their will.
I have some questions, though:

Why are SilentUpgrade/DoPartialUpgrade mutually exclusive with SilentDeploymentSettingsPath? 1) I would expect a partial upgrade (which does not redeploy the ARM template) to be able to run silently if needed; and 2) if I turn the SilentUpgrade flag to true, I am expecting to have the SilentDeploymentSettingsPath filled in.
It is not clear why we have SilentUpgrade and silentDeploy variables. What is the difference and why we need both?

I do agree with you, in that we should store the Azure SQL password in a Key Vault, but this can be added in a separate PR, to keep things simple at this moment.

Hi @helderpinto,

I made an effort to maximize reuse of code from the original source, which led me to avoid creating a separate file for SilentDeploymentSettings. Instead, I've integrated it with the existing $deploymentOptions and $lastDeploymentStatePath. Please note that the SilentDeploymentSettingsPath parameter is only relevant during the initial deployment. Thus, specifying SilentDeploymentSettingsPath alongside a SilentUpgrade or DoPartialUpgrade parameter is nonsensical since it holds no value at that juncture. My intention was to emphasize this through mutual exclusivity. The thrown error might help elucidate this distinction a little more. 😉
The SilentUpgrade parameter is used to specify a silent upgrade from a perspective of a user running the script. Otherwise we don't have any way to indicate that we want the upgrade to be done silently (like without any further prompt asking to re-use the previous deployment settings).
The silentDeploy variable is used within the script to bypass various prompts or selections when deploying silently and gets set automatically when specifying a SilentDeploymentSettingsPath parameter.

Hi, @wpouseele. Thanks for the clarifications.
The SilentDeploymentSettingsPath parameter was initially intended to be used both at initial deployment and upgrade times. It would be great if users were also able to set up a continuous deployment pipeline based on your implementation. This would of course require removing the mutual exclusivity between DoPartialUpgrade and SilentDeploymentSettingsPath and renaming the SilentUpgrade parameter to a more generic term, e.g., SilentMode, or simply removing it and making the SilentDeploymentSettingsPath parameter an implicit choice for a silent mode execution. Both approaches are fine for me.
Allowing SilentDeploymentSettingsPath to be used both at initial deployment and upgrade times would require only a few changes in your code, maybe making it simpler, and would bring even more value to your work. Does it make sense?

Hi @helderpinto, sure makes total sense, assuming that the SilentDeploymentSettingsPath always overrules any last-deployment-state.json file.
Please find the needed changes in the additional commit.

Thanks for the update, @wpouseele! It looks good and thanks for taking care of the optional Benefits Usage workbooks deployment. I am going to merge your code. Thanks a lot for your contribution.


[Show Item Sources And Uses] Usage type "Combat Spells"

It might be worth adding combat spells to the list of usages. Currently, only alt-magic processes are recognized and displayed, while combat spells are completely missed out.

I don't think this is necessary, like 'Air Rune'.
The list would be too long if we added combat spells.;(
Also, when I play the game, I only use very few spells.

True, now that you mention air runes and thinking about all the types of spells... those lists would be huge. And pretty much anyone will now the use case of combat in general, so a generic, grouped entry "Combat Spells" is quite redundant as well.


update with-reasonml dependencies

update dependencies (remove webpack which shouldn't be there)
be more explicit with subdir setting

Thanks!


handle status histories like 1,1,1,1,1,1,1,2,0

Currently handlers ignore RESOLVE (status 0) after an error if there's a 2 (or anything else) in the way.
This isn't a perfect solution, but I believe it solves >90%
Open question: what should sensu do with a status history like 1,2,1,2,1 ?

Just noticed this PR. Seems like it's a dupe of https://github.com/sensu-plugins/sensu-plugin/issues/47, which was resolved in https://github.com/sensu-plugins/sensu-plugin/pull/48 but then regressed in https://github.com/sensu-plugins/sensu-plugin/pull/50.


Schema Mismatch Between Cached Constraint Run and Action Response

The cached results in the constraint_run table stores the names of the used resources under "resourceNames". The response from the Constraints action stores them under "resourceIds"
Additionally in the violationspart of results, the violating windows are stored as windows in the action response and as violationWindows in the cached results
Also, type in the response vs constraintType in the cache

If constraint_run should not be queried directly, consider hiding it from the hasura schema


Setup docs app

This is only the boilerplate of the docs app, copied from ember-toucan-core, and enabling CI/CD. Writing the actual documentation will follow in separate PR(s).

@NullVoxPopuli this message seems to suggest we need to create a new project for this? Do you have the permissions, I don't have any cloudflare access on behalf of CS, right?

@NullVoxPopuli this message seems to suggest we need to create a new project for this?

I just made it 🎉

I don't have any cloudflare access on behalf of CS, right?

Right, we can get you added if you want (UX has its own cloudflare account). But for this kind of setup you also would need to be a GitHub org admin. :(

@NullVoxPopuli I took the freedom to refactor the docs CD workflow a bit, in this commit. Previously we had a separate PostPreviewURLascommenttoPR job, which seemed too much to me tbh. It did a pnpm i and pnpm build, which seems unnecessary for the sticky-pull-request-comment action. And if you remove those, what is left then is a single step that takes 1s, which IMO doesn't justify a separate job. So I merged the two together.
Btw, that sticky-pull-request-comment action wouldn't really be needed, if coudflare would merge and release this bugfix, as the cloudflare action will create a deployment Github event that makes GH render it as a deployment in its native UI (which due to the bug doesn't work correctly for PRs rn). We could still leave the automated comment in here if we think that's useful though.


[CLOSED] Add govuk-lint

 Issue by gpeng
Thursday 24 March 2016 at 17:54 GMT
Originally opened as https://github.com/alphagov/specialist-publisher/pull/684

Setup Jenkins to run govuk-lint to avoid lint-esque commenting on PRs, improve code consistency and respect the gov.uk style guide.
All the juice is in the first commit. The last two are noisy 'fix all the things' commits.

gpeng included the following code: https://github.com/alphagov/specialist-publisher/pull/684/commits

 Comment by gpeng
Tuesday 29 March 2016 at 10:17 GMT

Thanks @danielroseman. I've addressed your comments in f851641


Ray seems not be running at http://localhost:23517

Hi, nice looking package!
I'm having trouble getting it up and running. When calling ray() I'm receiving the error Ray seems not be running at http://localhost:23517.
Any reason why I'd be getting this error?
Thanks!

I can't seem to reproduce this. Would you mind recreating this issue at the spatie/ray repo using the issue template? Make sure to include version numbers and such. Thanks!

I can't seem to reproduce this. Would you mind recreating this issue at the spatie/ray repo using the issue template? Make sure to include version numbers and such. Thanks!

I think it's because I'm running through Docker. Is there any documentation around this? Thanks in advance!

I think it's because I'm running through Docker. Is there any documentation around this? Thanks in advance!

I don't have much experience with Docker myself, but other users are investigating:
https://github.com/spatie/ray/pull/41/files

I don't have much experience with Docker myself, but other users are investigating:
https://github.com/spatie/ray/pull/41/files

I'm getting this issue in CI running my tests.
There was 1 error:

Tests\Feature\Http\Controllers\Auth\LoginControllerTest::updating_a_legacy_password_to_bcrypt
Exception: Ray seems not be running at http://localhost:23517

I'm getting this issue in CI running my tests.
There was 1 error:

Tests\Feature\Http\Controllers\Auth\LoginControllerTest::updating_a_legacy_password_to_bcrypt
Exception: Ray seems not be running at http://localhost:23517

I'm getting this issue in CI running my tests.
There was 1 error:

Tests\Feature\Http\Controllers\Auth\LoginControllerTest::updating_a_legacy_password_to_bcrypt
Exception: Ray seems not be running at http://localhost:23517

Maybe call ray()->disable() in test setup?

I'm getting this issue in CI running my tests.
There was 1 error:

Tests\Feature\Http\Controllers\Auth\LoginControllerTest::updating_a_legacy_password_to_bcrypt
Exception: Ray seems not be running at http://localhost:23517

Maybe call ray()->disable() in test setup?

$ray = new Ray($settings, $client);

if (! $settings->enable) {
    $ray->disable();
}

Seems like the Client will be invoked even if it is ->disabled().
Try changing Spatie\LaravelRay to
if (! $settings->enable) {
    $ray = new Ray($settings, $client);

    $ray->disable();
}

to test.

$ray = new Ray($settings, $client);

if (! $settings->enable) {
    $ray->disable();
}

Seems like the Client will be invoked even if it is ->disabled().
Try changing Spatie\LaravelRay to
if (! $settings->enable) {
    $ray = new Ray($settings, $client);

    $ray->disable();
}

to test.

@chriischambers This is fixed in the latest release.

@chriischambers This is fixed in the latest release.


test

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test gencheck

@luksa: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_sail-operator_main
bac12f48086c845ab7efad066edb795485b6216f
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/test gencheck

@luksa: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_sail-operator_main
a0f2fcb6da9d0a40c410d268bf196f9124805b7e
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/test gencheck

@luksa: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_sail-operator_main
5935fd753882c7be75efdf846d0961ba0aa1a49a
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/test gencheck


X-Plane magneto switch "start" position does not work (won't hold)

The starter key switch works otherwise fine, but the "start" position just briefly engages the starter and it is not enough to start the engine. I would suspect there needs to be some "hold until switch is released" for a dataref or command.
Unfortunately I don't yet know more, but wanted to log this down so it gets noted, and there is a place to add extra information we might find.

this was redundant, so closing.


[BUG] When selecting default values in a form, these values will still be selectable inside the dropdown (zod)

Description
When i set defaultValues, I expect these values to be removed from the dropdown, but instead i can select them once more.
When I edit an Object inside a form, I pass default values. It doesnt work with zods typical register{...prop}, instead i tried it with the default values prop.
The default Values will still be selectable inside the dropdown -> I can have the same Tag selected twice
<ControlledSelect<FormData, SkillOptions, true>
          isMulti
          name="skills"
          label="Fähigkeiten"
          placeholder="Suchen..."
          openMenuOnFocus
          closeMenuOnSelect={false}
          useBasicStyles
          control={control}
          tagVariant='solid'
          colorScheme='blue'
          selectedOptionStyle="check"
          options={skillsForSelect}
          defaultValue={transfSkills}
          hideSelectedOptions={true}
/>

chakra-react-select Version
^4.8.0
Link to Reproduction
No response
TypeScript?

[X] Yes I use TypeScript

Steps to reproduce

Define mockData as Options inside Select
Take some Objects of that data and put them as default values into the Select Comp.
See that the Default Values and in addition to that the same Objects inside the mockData will be displayed in the dropdown

Operating System

[ ] macOS
[X] Windows
[ ] Linux
[ ] iOS/iPadOS
[ ] Android

Additional Information
No response

Hello again, i really need a solution for that

Also, when you say, "It doesnt work with zods typical register{...prop}", are you actually talking about react-hook-form? If so, you also don't want to be using defaultValue on the select component. Instead, you'd want to be passing it into the defaultValues config in the useForm hook:
  const {
    control,
    handleSubmit,
    register,
    reset,
    formState: { errors, isSubmitting },
  } = useForm<FormValues>({
    defaultValues: {
      mySelectValues: [
        { label: "V...", value: "v..." }
      ]
    },
  });

During the construction of the sandbox i realised the skillsForSelect are made toLowerCase()and the transfValues arent. Thats why they werent seen as the same values
Now its working, thank you for making me realise <3!
And yes i didnt realise the register prop wasnt needed and the default values would be automatically inserted i defined in defaultValues!
For anyone in the future thats how the code looks now:
 defaultValues: {
      skills: defaultSkills,
      ...
}
---
<ControlledSelect<FormData, SkillOptions, true>
          isMulti
          name="skills"
          label="Fähigkeiten"
          placeholder="Suchen..."
          openMenuOnFocus
          closeMenuOnSelect={false}
          useBasicStyles
          control={control}
          tagVariant='solid'
          colorScheme='blue'
          selectedOptionStyle="check"
          options={skillsForSelect}
          hideSelectedOptions={true}
        />


Ability to simulate a manipulator attached to a USV

Hi Prof. Fossen,
Would it be possible to simulate a serial manipulator attached underneath a USV with this library in such a way that the hydrodynamics is applied to the manipulator as well?
Thank you very much.

The toolbox does not contain multibody systems. However, you can use the Otter USV model (scale it up and down) and add a robot manipulator model. Then, you need to understand the interaction forces and moments between the two rigid bodies. Concerning hydrodynamic added mass and damping for a robotic manipulator, publications on the Internet describe these terms. However,  these forces are minor in practice, so a practical approach is to use a standard rigid-body model for the robot manipulator. Since it is submerged, you should still add buoyance forces corresponding to the displaced water volume of the manipulator.

The toolbox does not contain multibody systems. However, you can use the Otter USV model (scale it up and down) and add a robot manipulator model. Then, you need to understand the interaction forces and moments between the two rigid bodies.

Thank you very much for your reply.
Would you mean that I need to code the calculation of the interaction forces and moments between the Otter and the manipulator?

Yes, you need to calculate the interaction forces using other tools/programs.

Thank you very much, Professor!


Reshape data for cascade indicators

Reshaping data from glamr output around cascade indicators

@baboyma - see the cascade package which generates cascade plots + dfs

Got it. Will look at this next week.


Add support for advertising a MXID

Is your feature request related to a problem? Please describe.
I'd like to be able to specify my Matrix contact address on my READMEs. Adding a field for this would be useful.
Describe the solution you'd like
#162
A clear and concise description of what you want to happen.
Describe alternatives you've considered
None
Additional context
N/A

😄
@Half-Shot I don't know matrix.to . Could you provide some context?
My thought is that most of the readme-md-generator users won't have a matrix id, so I don't want to bother them with an extra field.

Understood, I'm wondering if there is a way we can include this as an extra without having to prompt the users.
For some context, Matrix is a pretty active chat protocol which tends to get new projects being created each week by the community (some have lent their :+1: to the post above :smile:). matrix.to is just a permalink service to direct users onto their chat client of choice.
Given we tend to have a high number of new projects each week, my goal was to direct them to this tool so that we can start to have some consistent and high quality READMEs.
I'm quite happy to work on whatever is required to make this feature available but hidden from users who don't care about it (command line flag perhaps?)

Thanks for your interest, but for now I don't see any solution to embed this feature without impacting other users 😕


Compilation Error in angular project when I try to upgrade my pdfjs-dist from 3.11.174 to 4.0.189

Discussed in https://github.com/mozilla/pdf.js/discussions/17480

Originally posted by Srinivasan0311 January  5, 2024
Attach (recommended) or Link to PDF file here: Any basic pdf can be used
Configuration:

Web browser and its version: chrome 120.0.6099.199
Operating system and its version: windows 11
PDF.js version: 4.0.189
Angular project version 16
Is a browser extension:

Steps to reproduce the problem:

upgrading pdfjs-dist using the command "npm i<EMAIL_ADDRESS>npm start

What is the expected behavior? (add screenshot)
Expected behaviour is should compile successfully without any errors.
What went wrong? (add screenshot)

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

Closing since this is a duplicate of #17245; please refer to the discussion in that issue.


installing on debian 8.8

Hi
I tried to install gitfs on debian, but without a luck
error is:
File "/usr/lib/python2.7/dist-packages/cffi/ffiplatform.py", line 51, in _build
raise VerificationError('%s: %s' % (e.class.name, e))
cffi.ffiplatform.VerificationError: CompileError: command 'x86_64-linux-gnu-gcc' failed with exit status 1
any hints?

To compile/package gitFS is needed libgit2-dev (>= 0.24)
I could do this with Debian 9 as root:
apt install build-essential checkinstall automake python-virtualenv libffi-dev python3-dev libgit2-dev
wget https://github.com/PressLabs/gitfs/archive/master.zip
unzip master.zip
rm master.zip
mv gitfs-master gitfs
cd gitfs
# Fix subversion reference for virtualenv:
sed -i -e 's|python3.4|python3|g' Makefile

Write a "gitfs.spec" file with a line:
Release: 1
Write a "description-pak" file with a line:
Version controlled file system
Continue to compile and package:
make
checkinstall -D --fstrans --default --reset-uids "--maintainer=MyName" --pakdir=.. --pkgname=gitfs --requires=python3-minimal,python-fusepy,python-pygit2,python-setuptools --pkgrelease=1 --install=no make install
cd ..
rm -r gitfs

Package dependencies should be fixed about version of python-pygit2 (>= 0.24) because gitfs requires this version at least.
Anyway, you will need to backport dependencies not available at plain Debian 8 repositories.

We've just released 0.5.1 with updated dependencies and fresh new debs.
Can you please try again, with the newer version?
Sorry for the delay.


Neo Async features

Add all of these functions:
pick
pickSeries
pickLimit
rejectLimit
selectLimit
someLimit
sortByLimit
transform
transformSeries
transformLimit
timesLimit

Based pick and transform tests off of neo-async tests https://github.com/suguru03/neo-async/tree/v1.x/test/collections

Updated @crispy1989


Use user-preferred browser by default

Currently gluton hardcodes a few browsers, however is someone is using a different browser like for example a custom chromium build or browsers like opera or brave this fails.
A decent solution would be:

use the system preferred browser

windows:

HKEY_CURRENT_USER\SOFTWARE\Microsoft\Windows\Shell\Associations\URLAssociations\https\UserChoice
ProgID - REG_SZ
HKEY_LOCAL_MACHINE\SOFTWARE\Classes\${ProgID}\shell\open\command
(Default) - REG_SZ

linux: ???
macOS: ???

get all available browsers for the OS

??

for windows the best module for navigating regedit seems to be winreg, not sure how to do this on linux/macos

Would be cool, currently is just in order of most preferred/usable by Gluon. Would likely use reg.exe directly to avoid external dependencies.

Would be cool, currently is just in order of most preferred/usable by Gluon. Would likely use reg.exe directly to avoid external dependencies.

winreg doesnt have any external dependencies, so I think it's a safe go, it's also been stable for a few years

I meant having a dependency at all, as you can make a wrapper for reg.exe in ~5 lines

I meant having a dependency at all, as you can make a wrapper for reg.exe in ~5 lines

hm, too bad, wanted to PR this using that lib, but making a wrapper for a windows executable and parsing errors isn't my forte, here's the code using winreg, util.promisify doesn't seem to work on it :/
import Registry from 'winreg'

const preferred = new Registry({
  hive: Registry.HKCU,
  key: '\\SOFTWARE\\Microsoft\\Windows\\Shell\\Associations\\URLAssociations\\https\\UserChoice'
})
preferred.get('ProgID', (_err, item) => {
  const ProgID = item.value
  const browser = new Registry({
    hive: Registry.HKLM,
    key: `\\SOFTWARE\\Classes\\${ProgID}\\shell\\open\\command`
  })
  browser.get(Registry.DEFAULT_VALUE, (_err, item) => {
    console.log(item)
    const shell = item.value
    console.log(shell)
  })
})

Oh, apologies. Code looks good and understandable, using a custom wrapper allows using promises as well so lol.

Possibly could be an easier idea to use file associations like .html instead. Dedicated command line tool for it on Windows so would be much simpler but unsure if some people/browsers don't set file associations.

Preferred file associations for .html would be a generally safe bet, but accessing based on https preferences may be the best to go? Windows is a "little" aggressive with their promotion of Microsoft Edge, so they don't reliably change all file types when you update your preferences.

Preferred file associations for .html would be a generally safe bet, but accessing based on https preferences may be the best way to go? Windows is a "little" aggressive with their promotion of Microsoft Edge, so they don't reliably change all file types when you update your preferences.

.html is bad as it also lists text editors like vscode which are also in theory chromium, so i just went with https and browser detection

Yeah changed my mind, have an almost complete implementation which I'm leaving for 0.10.0, 0.9.0 should be releasing today with some new APIs.

On macOS I have found a way to obtain the default browser.
You can use plutil -convert json -o - ~/Library/Preferences/com.apple.LaunchServices/com.apple.launchservices.secure.plist to obtain the mapping of schemes/protocols to bundle IDs. [^1] [^2]
From that, you get something like:
{
  "LSHandlers" : [
    { /* Other options */ }
    {
      "LSHandlerPreferredVersions" : {
        "LSHandlerRoleAll" : "-"
      },
      "LSHandlerRoleAll" : "org.mozilla.firefox",
      "LSHandlerURLScheme" : "https"
    },
    { /* Other options */ }
  ]
}

Parsing that JSON, you can search for the object whose property LSHandlerURLScheme equals "https" and obtain the value of the property LSHandlerRoleAll, which is the bundle identifier of the application -- in this case "org.mozilla.firefox".
Then, you can use mdfind kMDItemCFBundleIdentifier = "org.mozilla.firefox", which will return the path of the application that that identifier belongs to. In this case, for my setup, it returned /Applications/Firefox.app.
From this information, you can access the Info.plist, which is located in /Applications/Firefox.app/Contents/Info.plist and once again use plutil to fetch the executable file that should be run using plutil -extract CFBundleExecutable raw /Applications/Firefox.app/Contents/Info.plist.
This will return firefox, which means the final path is /Applications/Firefox.app/Contents/MacOS/firefox.
You can also use plutil -extract CFBundleName raw /Applications/Firefox.app/Contents/Info.plist to obtain the application name, which will return "Firefox" for Firefox and "Chrome" for Google Chrome. This can be useful to compare against the supported browsers for gluon.
[^1]: Taken from here. Note that the path differs for older versions of macOS.
[^2]: You can add the -r to get a prettified JSON response.

On Windows getting exe path from this is most universal: HKEY_LOCAL_MACHINE\SOFTWARE\Classes\${ProgID}\shell\open\command. It will also work when users have non-standard path. Which would be more common for Firefox then Chrome probably.

On Windows getting exe path from this is most universal: HKEY_LOCAL_MACHINE\SOFTWARE\Classes\${ProgID}\shell\open\command. It will also work when users have non-standard path. Which would be more common for Firefox then Chrome probably.

its legit in the original post


don't commit SVG files

These should be created when importing from raku/doc, not rendered and stored in the repo.

will also avoid issue #11

closed #11 as the issue moved here.
All it needs is to delete the svg. They are (should be) regenerated if the typegraph plugin is run and the svgs are missing.
I'll delete them now from the repo, and close this issue


Retrieve RateLimits

Hi.
When requesting https://api.openai.com/v1/chat/completions in streaming mode, the HTTP response contains a set of headers with service information like X-Ratelimit-Remaining-Tokens:.
Is it possible to add a method to retrieve all headers from stream?
Or make the $response property in the OpenAI\Responses\StreamResponse class public?

I agree.  We need this for things like calculating how many tokens were used.

I submitted this PR that would accomplish this:  https://github.com/openai-php/client/pull/187

You can now use the latest release which gives you access to the header / meta information on every response: https://github.com/openai-php/client#meta-information


IDX20803: Unable to obtain configuration from: '[PII is hidden]

Hi
I encounter a very strange problem. I put identityserver 4 server and client in the same web project. In the LAN environment, they can work normally. When I publish to the external network environment, and configure the verification address to the external network IP address, I can get token normally, but the verification can not pass. The specific error information is as follows. Please help me see this problem, I can't get token. Accurate positioning of this problem
IDX20803: Unable to obtain configuration from: '[PII is hidden]'.
at Microsoft.IdentityModel.Protocols.ConfigurationManager1.GetConfigurationAsync(CancellationToken cancel)    at Microsoft.AspNetCore.Authentication.JwtBearer.JwtBearerHandler.HandleAuthenticateAsync()    at Microsoft.AspNetCore.Authentication.JwtBearer.JwtBearerHandler.HandleAuthenticateAsync()    at Microsoft.AspNetCore.Authentication.AuthenticationHandler1.AuthenticateAsync()
at Microsoft.AspNetCore.Authentication.AuthenticationService.AuthenticateAsync(HttpContext context, String scheme)
at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
at Microsoft.AspNetCore.Cors.Infrastructure.CorsMiddleware.Invoke(HttpContext context)
at IdentityServer4.Hosting.BaseUrlMiddleware.Invoke(HttpContext context)
at Microsoft.AspNetCore.StaticFiles.StaticFileMiddleware.Invoke(HttpContext context)
at Util.Webs.Middlewares.ErrorLogMiddleware.Invoke(HttpContext context)

IDX20803: Unable to obtain configuration from: '[PII is hidden]'.

Typically points at HTTPS configuration problems on the server.

All set on this issue -- can we close?


tortoise bug fix + ut

solves a bug we had initialising votes to pbase where we didn't delete all previous votes

@barakshani please comment with any interesting assertions you think off


Better Logging

I got logging work, check it at https://github.com/flynnoct/chatgpt-telegram-bot/commit/87bed9320c41f0c881ad4bad2c1b4c9fdde6e20c


Support timeout on fetch requests

Is your feature request related to a problem? Please describe.
Running multiple fetch requests in parallel may end up causing denial-of-service attacks if they are long-lived.
Describe the solution you'd like
Regular XHR requests support a timeout that will fail the request after a specific amount of time.
There is currently no clear spec for it in the fetch API, but some discussion around it exist (see links below)
Would be great to have a first draft implementation in the library to offer a decent enough implementation until the spec is finalized.
Describe alternatives you've considered
This can alternatively be implemented consumer-side by using timeouts and promises, and rejecting the promise before the request ends, but the socket is still left open until the fetch actually finishes so it doesn't fully removes the issue
Additional context
See:

https://github.com/whatwg/fetch/issues/20
https://github.com/whatwg/fetch/issues/179
https://github.com/whatwg/fetch/issues/180

supported in v1.1.0


refactor: prefer map_or_else

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Digital signature invalid on Windows

I was notified by security that I used a tool with an invalid signature on Windows. I've tried both 8.2 and 8.3.
Would be great if this can be fixed for future versions.

这是来自QQ邮箱的假期自动回复邮件。您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

Can someone show me the output of:
signtool verify /pa /v <app>

Tested with JMC 8.3:
 .\signtool.exe verify /pa /v 'C:\temp\JDK Mission Control\jmc.exe'

Verifying: C:\temp\JDK Mission Control\jmc.exe

Signature Index: 0 (Primary Signature)
Hash of file (sha1): 49973BE351B250FD818DFB44C2FD50BE1E62F45B

Signing Certificate Chain:
    Issued to: DigiCert Assured ID Root CA
    Issued by: DigiCert Assured ID Root CA
    Expires:   Mon Nov 10 01:00:00 2031
    SHA1 hash: 0563B8630D62D75ABBC8AB1E4BDFB5A899B24D43

        Issued to: DigiCert SHA2 Assured ID Code Signing CA
        Issued by: DigiCert Assured ID Root CA
        Expires:   Sun Oct 22 13:00:00 2028
        SHA1 hash: 92C1588E85AF2201CE7915E8538B492F605B80C6

            Issued to: Eclipse.org Foundation, Inc.
            Issued by: DigiCert SHA2 Assured ID Code Signing CA
            Expires:   Thu May 19 00:59:59 2022
            SHA1 hash: 899BF5E9EE7FD5AA2F4DDAB20FD9632ABDC58A64

The signature is timestamped: Tue Apr 19 17:03:12 2022
Timestamp Verified by:
    Issued to: USERTrust RSA Certification Authority
    Issued by: USERTrust RSA Certification Authority
    Expires:   Tue Jan 19 00:59:59 2038
    SHA1 hash: 2B8F1B57330DBBA2D07A6C51F70EE90DDAB9AD8E

        Issued to: Sectigo RSA Time Stamping CA
        Issued by: USERTrust RSA Certification Authority
        Expires:   Tue Jan 19 00:59:59 2038
        SHA1 hash: 02D65B95E28370C1570095FA88F923DD937FAD8F

            Issued to: Sectigo RSA Time Stamping Signer #2
            Issued by: Sectigo RSA Time Stamping CA
            Expires:   Fri Jan 23 00:59:59 2032
            SHA1 hash: 951137101D882F31BD513F949ADA4C68AD8C08F5

SignTool Error: WinVerifyTrust returned error: 0x80096010
        The digital signature of the object did not verify.

Number of files successfully Verified: 0
Number of warnings: 0
Number of errors: 1


needed "vested balance" for harvesting wrong?

0.6.25-BETA
When you try to start harvesting for an account with insufficient funds, a warning is displayed saying that you need 1.000 XEM to harvest. Truth seems to be 10.000 XEM though?

Also it should say vested XEM instead of just XEM. This is a common source of confusion for people.

https://github.com/NewEconomyMovement/NemCommunityClient/pull/398


psycopg.errors.UndefinedTable: relation "journal" does not exist

Using timescaledb-latest-pg16 docker image. When PG Logger gets a message on a topic, when trying to insert it into the DB, it throws this error.
mqtt-pg-logger-mqtt-pg-logger-1  | 2024-09-25 11:51:32,499 [   ERROR] src.proxy_store: relation "journal" does not exist
mqtt-pg-logger-mqtt-pg-logger-1  | Traceback (most recent call last):
mqtt-pg-logger-mqtt-pg-logger-1  |   File "/mqtt-pg-logger/src/proxy_store.py", line 94, in run
mqtt-pg-logger-mqtt-pg-logger-1  |     if self._store_messages():
mqtt-pg-logger-mqtt-pg-logger-1  |   File "/mqtt-pg-logger/src/proxy_store.py", line 161, in _store_messages
mqtt-pg-logger-mqtt-pg-logger-1  |     self._message_store.store(messages)
mqtt-pg-logger-mqtt-pg-logger-1  |   File "/mqtt-pg-logger/src/message_store.py", line 65, in store
mqtt-pg-logger-mqtt-pg-logger-1  |     with cursor.copy(copy_statement) as copy:
mqtt-pg-logger-mqtt-pg-logger-1  |   File "/usr/local/lib/python3.7/contextlib.py", line 112, in __enter__
mqtt-pg-logger-mqtt-pg-logger-1  |     return next(self.gen)
mqtt-pg-logger-mqtt-pg-logger-1  |   File "/usr/local/lib/python3.7/site-packages/psycopg/cursor.py", line 908, in copy
mqtt-pg-logger-mqtt-pg-logger-1  |     raise ex.with_traceback(None)
mqtt-pg-logger-mqtt-pg-logger-1  | psycopg.errors.UndefinedTable: relation "journal" does not exist

I think I have removed references and the ability to specify the table ("journal"), I think you want to use pgqueuer instead


Better support for Server-Side Rendering & client-only code in React client-only lifecycle hook

Hey @timbrandin!
First of all, thanks: that's great to see this package getting better and better!! I love the latest updates! 👍
Problem
I'm using it on a server-side rendered app, and I noticed that if the form is on the page on initial render, there is an issue between its initial state on the server & the client, causing a performance loss:

Note: it's also showing the user's password in the console 🙄
Solution
➡ I just moved this handy method this.getDefaultFieldValues to the componentDidMount hook, which is not triggered on the server, from the constructor which is called on both env.

So here is my PR to fix that 😄
We have now 3 calls to this.setState on componentDidMount, I was thinking of grouping them into one call at the end of the method potentially?

We have now 3 calls to this.setState on componentDidMount, I was thinking of grouping them into one call at the end of the method potentially?

➡ I've changed the setState arguments to be a function so:

Updates will be queued and later executed in the order they were called.

https://twitter.com/dan_abramov/status/824309659775467527

Hey, another update on this PR 😄
If you don't use reactrouter:react-router-ssr (code)   or fast-render to monkeypatch Meteor.subscribe on the server, the LoginForm component spit an error about the user status / tracker-related / subscription to oauth services code.
In the vein of this PR, I moved this code updating the state of LoginForm from the constructor to componentDidMount 🚀
PS: This was highlighted by @comus in https://github.com/TelescopeJS/Telescope/pull/1561.

oki,

Updates will be queued and later executed in the order they were called.
that's a sound reason! 👍

Version 1.2.18 is out. // T

Yay!! 🎉🎉🎉


Only add the text selection canvas when the textbox is focused

What does the pull request do?
TextSelectionCanvas will now only be created and added to the visual tree when its TextBox is focused, and be removed when unfocused.
What is the current behavior?
TextSelectionCanvas is currently created when a textbox is shows, even when not active. This occurs for every textbox in the visual tree. Even if not visible, it add some layout overhead.
What is the updated/expected behavior with this PR?

How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/avalonia-docs with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues

LGTM!


Show some sort of indicator showing the win/loss ratio against the current opponent

This might take into account just the last N games or everything in the crosstable. Or even replace it completely and show it on mouseover.

I know it doesn't showed the win loss ratio, but if you hover over your opponents username it does show the number of W/D/Ls that you have against that opponent.  I'm certain you are aware of this though 😺

Pointless feature. LiChess Tools is not here to add fancy social visuals.


Commit and Contributor shield additions to README.md

How Many Days Until Halloween PR
Type of Change was Made?
What type of change does your code introduce? After creating the PR, tick the checkboxes that apply.

[ ] Small bug fix (non-breaking change which fixes an issue)
[ x ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Description of the Change Being Made.
Added the following shields from https://shields.io:

Custom shield: Friendly - True
** Intended for marking repo as friendly to contributors
Github Contributors
** Reinforces the friendly badge before it by showing the number of contributors
GitHub commit activity the past week, 4 weeks, year
GitHub last commit

Issue Number
#174: Improvement: Add sheilds to README.md
Potential Performance Issues
None
Tests/Checks
Tested via python -m SimpleHTTPServer, no issues found
New Dependencies
No

References #174


Add an API for Consumer to read all slots?

Now, I read all slots from Consumer by this:
  let result = consumer.read_chunk(consumer.slots());

But sometimes I don't care how many slots hold by RingBuffer, I just want to read all slots.
Do you think it's a good idea to add a method for Consumer like this:
  let (all_slots_size, chunks) = consumer.read_all();

You could also have a look at https://github.com/agerasev/ringbuf, which has a very different API, and it does not require specifying the number of slots, AFAICT.

I don't think that all_slots_size is necessary, because chunks have a .len() method. And is chunks intentionally plural? I would expect one chunk to be returned, or am I missing something?
I agree with you, all_slots_size is not necessary. And should be chunk instead of chunks.

You could also have a look at https://github.com/agerasev/ringbuf, which has a very different API, and it does not require specifying the number of slots, AFAICT.

Thank you. :smile:

I would suggest a couple of things:

Instead of forcing the user to choose between getting a chunk of a fixed size (the current read_chunk implementation) or getting a slice of any size (the proposal), why add instead of the latter a function that returns a chunk of at least a given size?
This way if one doesn't care about the chunk's size (like @eval-exec) one could just specify 0 as a minimum, while if one requires to have at least a certain size (while also accepting more), they can have the option to do so.
I'm not a native english speaker, but I would suggest as a name for such method read_chunk_least(min: usize).
Such a function would behave like @eval-exec's read_all() proposal when min is 0, return an error if there weren't enough available slots and return a chunk whose length is >= min in any other case.
Add a symmetric interface to the producer.

Thanks for the suggestions!

if one requires to have at least a certain size (while also accepting more)

Is this a real use case that you have?
If yes, can you please elaborate?

Is this a real use case that you have? If yes, can you please elaborate?

I'm using this library in a small side project of mine as a channel between an IO-thread and a parser thread.
In order to keep the commits and system calls low, I would like to do vectored reads with the slices as large as possible.
And since doing a sytem-call with an excessively small buffer would be counterproductive, I would like to specify a minimum size.

Thanks, that's an interesting use case!
So if I understand correctly, you need something like this (using existing API):
let chunk = c.read_chunk(c.slots()).unwrap();
if chunk.len() >= 25 {
    // do something
}

... but with less noise:
if let Ok(chunk) = c.read_chunk_least(25) {
    // do something
}

This new function could then also be used for simply getting all slots by using 0 (but this would still need unwrap()):
let chunk = c.read_chunk_least(0).unwrap();
// do something

I don't love the name, and I think it's not good to have two similar functions that both take the same arguments (a single number). I think this might be confusing.
For comparison, if we had a function to get all slots unconditionally (let's call it max_read_chunk() for now), your use case would look like this:
let chunk = c.max_read_chunk();
if chunk.len() >= 25 {
    // do something
}

This would still be more than a one-liner, but at least the .unwrap() could be avoided.
I think having to type a few characters less is not reason enough to add a new function, but avoiding unwrap() might be?
If we have to decide between the two options shown above, I think I would vote for the second one, which avoids unwrap() in all cases.
Another way to look at it is that both read_chunk() and read_chunk_least() would have the same signature (usize) -> Result<ReadChunk, ChunkError>, while max_read_chunk() would have () -> ReadChunk. Looking at this, I think that the function signatures on their own are more understandable than the names of the functions!

There is also a subtle performance-related question: should read_chunk_least(25) try to return as many slots as possible (which requires touching an atomic variable) or would it be OK to return fewer slots (but of course still at least 25) if that could be done without an atomic operation?
Latter cannot be achieved with the current API.

So if I understand correctly, you need something like this (using existing API):
let chunk = c.read_chunk(c.slots()).unwrap();
if chunk.len() >= 25 {
    // do something
}

Mostly, but as the slots() documentation mentions there's a data race in there, which could mean I would get less than all availabe slots

... but with less noise:
if let Ok(chunk) = c.read_chunk_least(25) {
    // do something
}

Yes.

For comparison, if we had a function to get all slots unconditionally (let's call it max_read_chunk() for now), your use case would look like this:
let chunk = c.max_read_chunk();
if chunk.len() >= 25 {
    // do something
}

Doesn't seem like a bad idea to me. It would even work for my usecase since I'm wrapping it all anyway

There is also a subtle performance-related question: should read_chunk_least(25) try to return as many slots as possible (which requires touching an atomic variable) or would it be OK to return fewer slots (but of course still at least 25) if that could be done without an atomic operation?

A bool perhaps? It's a simple solution but probably not exactly elegant

as the slots() documentation mentions there's a data race in there

Just to clarify, that's not a data race, at least I hope not!
If it were a data race (as Rust uses the term: https://doc.rust-lang.org/nomicon/races.html), it would be Undefined Behavior and rtrb would be unsound.
It is natural that once one thread has finished reading an atomic variable, the other thread can change it again. Whether this happens within a single function or whether another function is called (in this case c.slots()) doesn't make any real difference.
So even if we implement a new function read_chunk_least() or max_read_chunk() or whatever, the fact would remain that at some point we have to read from an atomic variable, and from that point on, we don't have any knowledge what happens to that variable.
From a thread-communication point of view, there is no difference between c.read_chunk(c.slots()).unwrap() and c.max_read_chunk(). In both cases, the atomic tail variable is read exactly once and when we get our ReadChunk, the Producer might have already committed new elements.
In the current implementation, c.read_chunk(c.slots()).unwrap() might result in a few redundant instructions, because none of the functions is marked #[inline]. It would be interesting to benchmark this and see if it makes a difference.

Is there still interest in this feature?
I'm still not convinced that it is worth implementing, since it can be written as c.read_chunk(c.slots()).unwrap(), which doesn't seem that bad to me.


Add Basic Recipes

This is basically adding in the vanilla recipes for certain metals and objects that won't be apart of the multiblock recipes.

Lets get this started.

OK I'm done with the basic recipes for now. These are vanilla recipes that are crafted in a crafting table. I've taken a few pictures of the current list of recipes that are in the mod.
Crafting Fire Block

Crafting Water Block

Crafting Air Block

Crafting Earth Block

Crafting 9 Fire Dust

Crafting 9 Water Dust

Crafting 9 Air Dust

Crafting 9 Earth Dust

Crafting Void Stone

Note: Totem Elemental Heads might be changed in the
Upcoming update. I'm thinking of replacing it with
Elemental Dust Blocks
Crafting Fire Totem Head

Crafting Water Totem Head

Crafting Air Totem Head

Crafting Earth Totem Head

Crafting Void Glass.

Void Glass looks cool

Crafting Void Sticks.

Crafting Void Tool

Crafting Machine Core

The next section contains both elemental iron and elemental diamond. To get these
you'll need to create an elemental totem which hasn't been implemented yet. The next
update will have the totems completed hopefully
Crafting Fire Iron Block

Crafting Water Iron Block

Crafting Air Iron Block

Crafting Earth Iron Block

Crafting Fire Diamond Block

Crafting Water Diamond Block

Crafting Air Diamond Block

Crafting Earth Diamond Block

And they all can be revered back to ingots and diamonds.

Whew that was a lot of pictures. I'm going to keep this open for basic vanilla recipes for documentation purposes. I guess I'll need some sort of guide in the mod to help with starting
out. But for now I'll uses the git issues track as documentation. I'll setup the wiki later for
documentation later well night.

I've added in a new recipe for Void Dust Block. Its going to be used as a way to revert element iron and diamond blocks back to there original state. Because who know when you might be low on diamonds and iron :D.
Void Dust Block Recipe

Also I've removed the elemental totem heads. I'll be using Elemental Dust Blocks as the top of the elemental totems for generating elemental iron and diamond blocks. I'll explain those today when I finish the void totem. I'll also add a new issue for totem recipes.

I'm going to recreate new issue similar to this because I've rewrote the mod for 1.8.9.


Fix issue with DateTimePicker layout in IE 11

Q
A

Bug fix?
[X]

New feature?
[ ]

New sample?
[ ]

Related issues?
fixes #301

What's in this Pull Request?
This fixes the layout problem mentioned in issue #301.
The implementation adds flex-basis: 0%; to the picker css class. This fixes the layout issues seen in IE 11 and is needed because IE 11 does not fully comply with the flexbox standard.

Codecov Report

Merging #302 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #302   +/-   ##
=======================================
  Coverage   72.36%   72.36%
=======================================
  Files          17       17
  Lines         760      760
  Branches      157      157
=======================================
  Hits          550      550
  Misses        163      163
  Partials       47       47

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 69a8337...4b4ee8c. Read the comment docs.

Thanks @robert-lindstrom for fixing this


Added support for dropdown-menu being right aligned, see angular-ui/bootstrap#3820

Bootstrap has a 'dropdown-menu-right' class, which is used to display a dropdown menu aligned to the right edge of the toggle, rather than the left.
This PR adds a check in the case of append-to-body, and calculates the correct right-aligned position of the dropdown menu element.
This would then be able to close angular-ui/bootstrap#3820.

This looks good - here is a Plunker with your changes for reference.


Add remote domain support

Some TV's use a a remote domain on/off switch to toggle standby mode (The Roku TV integration for example).
I simply allowed the configuration to accept remote domain as a valid option.
I see no errors and everything works as if it was using a switch entity.

Thanks for pr, Nice addition. Could you also update readme accordingly. There is reference for supported platforms

I was looking over the code and I see no reason to limit fixed mode domains at all. The logic is actually really well done and should work with most domains to be honest.
Is there a reason to limit domains with fixed mode? Do you mind if I add media_player as well?

I was looking over the code and I see no reason to limit fixed mode domains at all. The logic is actually really well done and should work with most domains to be honest.
Is there a reason to limit domains with fixed mode? Do you mind if I add media_player as well?

One of the reasons I filtered domains was to limit selection of entities when you are using VS code addon.
It makes no sense to setup a power sensor for a entity not having an on/off state (for example sensor).
I did a quick look through possible domains and picked the ones which seem to fit to me. However I missed some, media_player and remote can be turned on and off, so they can be used for the fixed power mode.

Yes please add media_player also, than I will merge the changes.

Done, I even added logic to detect media_player standby state and function accordingly. Works great with google Chromecast devices.

Thanks


tree-sitter-sql: fix source hash

Description of changes
Patched tree-sitter-sql hash, which is incorrect as it stands.
nurl https://github.com/derekstride/tree-sitter-sql b8175006d9c8120d41cf40a4ef3711bbbbc08973
$ nix flake prefetch --extra-experimental-features 'nix-command flakes' --json github:derekstride/tree-sitter-sql/b8175006d9c8120d41cf40a4ef3711bbbbc08973
fetchFromGitHub {
  owner = "derekstride";
  repo = "tree-sitter-sql";
  rev = "b8175006d9c8120d41cf40a4ef3711bbbbc08973";
  hash = "sha256-ZURUEd7TokBIVVejRoBiXMQ1XwUEgMnzOhQiV+Tdpk0=";
}

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] 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.11 Release Notes (or backporting 23.11 and 24.05 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.

@ofborg build tree-sitter-grammars.tree-sitter-sql

Help me understand the process - we're waiting on the queued jobs to finish? Are they still queued due to a lack of free workers?

Well that's fun, there must be some casing differences in some filenames so the hash ends up different on mac and linux :(
NixOS:
❯ nurl https://github.com/derekstride/tree-sitter-sql b8175006d9c8120d41cf40a4ef3711bbbbc08973

$ nix flake prefetch --extra-experimental-features 'nix-command flakes' --json github:derekstride/tree-sitter-sql/b8175006d9c8120d41cf40a4ef3711bbbbc08973
fetchFromGitHub {
  owner = "derekstride";
  repo = "tree-sitter-sql";
  rev = "b8175006d9c8120d41cf40a4ef3711bbbbc08973";
  hash = "sha256-idQB8Wqw7lvU192y7+UgFvcwlmY71/mu9jJ4hRc4ud4=";
}

macOS:
❯ nurl https://github.com/derekstride/tree-sitter-sql b8175006d9c8120d41cf40a4ef3711bbbbc08973

$ nix flake prefetch --extra-experimental-features 'nix-command flakes' --json github:derekstride/tree-sitter-sql/b8175006d9c8120d41cf40a4ef3711bbbbc08973
fetchFromGitHub {
  owner = "derekstride";
  repo = "tree-sitter-sql";
  rev = "b8175006d9c8120d41cf40a4ef3711bbbbc08973";
  hash = "sha256-ZURUEd7TokBIVVejRoBiXMQ1XwUEgMnzOhQiV+Tdpk0=";
}

@ofborg build vimPlugins.nvim-treesitter.grammarPlugins.sql

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

1 package failed to build:

vimPlugins.nvim-treesitter-parsers.sql

1 package built:

vimPluginsUpdater

Here is the current error I'm getting on mac, it looks to me as if this hash is indeed different on macOS:
error: hash mismatch in fixed-output derivation '/nix/store/yaaf6yx9d3z5ym19j2pf9zkrizhirvqr-source.drv':
         specified: sha256-idQB8Wqw7lvU192y7+UgFvcwlmY71/mu9jJ4hRc4ud4=
            got:    sha256-ZURUEd7TokBIVVejRoBiXMQ1XwUEgMnzOhQiV+Tdpk0=
error: 1 dependencies of derivation '/nix/store/gkj82kn7lv4npd2y312dxf47q6l07mb7-sql-grammar-0.0.0+rev=b817500.drv' failed to build

Here is the current error I'm getting on mac, it looks to me as if this hash is indeed different on macOS:

How is the hash obtained? Is there a wrapper script that could have a platform-specific bug?
I assume the archive itself is identical on both platforms, since it is fetched from github using the same repo & revision. If so, either the hashing process or the download/copying process must have a platform-specific bug :thinking:
I guess any bug here would likely be in the nix binary itself though, so I'd expect it to affect more packages. Maybe I'm barking up the wrong tree...

Converting this back to a draft as from the discussion it seems this wouldn't work if merged.

perhaps something upstream got fixed as I just update my flake this morning and now I am not having this issue on macos anymore.

Can confirm, it builds fine on macOS now!


Docker Monitoring - no Subject on Email

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

Description
I tested the new beta version and tryed it with some docker containers.
Works like a charm just one small "polishing" thing :)
👟 Reproduction steps
Get an Email Notification from Uptimekuma after a docker container is up or down.
👀 Expected behavior
We see an Email with a subject name like the friendly name of the docker container monitor we setup in Uptimekuma.
😓 Actual Behavior

🐻 Uptime-Kuma Version
1.18.0 (beta)
💻 Operating System and Arch
Ubuntu 22.04.01
🌐 Browser
Edge 104
🐋 Docker Version
Docker version 20.10.14, build a224086
🟩 NodeJS Version
No response
📝 Relevant log output
No response

I will have a look into this tomorrow. It is likely that the SMTP notification provider hasn't been updated with the new monitor type

@nicedevil007 Did you set a custom subject in the notification settings? Something like {{NAME}}

@nicedevil007 Could you please comment what your custom subject is?
I cannot reproduce this in https://github.com/louislam/uptime-kuma/pull/3414


View/Toogle Word Wrap not working in lasted version 1.14.1

Please check this issue, I'm using win 10 Pro. Tks

I have the same issue on 1.14.1. I'm running Windows 10 Enterprise (build 15063.483). I think I also had this issue in the previous version too.

Already fixed and will ship with next recovery build.

Hi,
file > preferences > settings
"editor.wordWrap": "on",
and smile ;-)


Bump execution extensions with Siddhi core 5.1.2 or higher

Description:
This is to make sure all the functions are properly validated based on the fix #1401 No-arg functions are not properly validated

Actually, above issue is fixed in Siddhi version 5.1.0 ... But, since map uses siddhi version 5.0.2, you have seen that issue. But, do you think your PR is required to solve this ?
[1] https://github.com/siddhi-io/siddhi/pull/1375

Actually, above issue is fixed in Siddhi version 5.1.0 ... But, since map uses siddhi version 5.0.2, you have seen that issue. But, do you think your PR is required to solve this ?
[1] #1375

Yep, there is another validation added in that which helps. Will add to other extensions as well.

All extensions bumped with the following PRs,
https://github.com/siddhi-io/siddhi-execution-math/pull/50
https://github.com/siddhi-io/siddhi-execution-time/pull/57
https://github.com/siddhi-io/siddhi-execution-json/pull/26
https://github.com/siddhi-io/siddhi-execution-unitconversion/pull/35
https://github.com/siddhi-io/siddhi-execution-reorder/pull/49
https://github.com/siddhi-io/siddhi-execution-unique/pull/55


Fix polish translations

@sbadura I have no idea how to evaluate this as I don't speak Polish, but can you explain the changes you've made?

@zachfeldman I'm Polish. Current translations are wrong. You can compare them with proper translations from: https://github.com/tigrish/devise-i18n/blob/master/rails/locales/pl.yml

Good enough for me!


Add support for EXTRACT syntax and converts it to appropriate Pinot expression

Description
This PR adds support EXTRACT syntax and converts it to its Pinot expression.
This PR will solve the following issue --  https://github.com/apache/pinot/issues/9075
Testing
Verified the desired behavior locally by running CalciteSqlCompilerTest

thanks, @Jackie-Jiang for providing context. Very helpful.
I am trying to understand how ExtractTransformFunction will be used in the overall flow (write path and read path). I have a few follow-up ques from my investigation:

Do we store the transformed data (after applying transformations as defined here on raw data) into segments or only raw data into segments and apply the transformations on the fly during Pinot query execution?

ExtractTransformFunction will only be applied at query time. If we want to support ingestion transform, we need to add a ScalarFunction for the extract (you may take a look at DateTimeFunctions` class).
To support this feature, we need 2 parts of the changes:

Calcite parser change to support extract syntax and parse the query into extract(field, expression)
Add ExtractTransformFunction or ScalarFunction to support query/ingestion time transform

Hey @Jackie-Jiang , I have completed part 1 of the task.
For Part2:
Can you please help me point to where ‘DateTimeTransformFunction’ (or other query time transformers defined here) are getting applied? I was able to locate TransformFunctionFactory but couldn’t locate where transformations are actually getting applied.

PTAL @Jackie-Jiang when you get a chance. Thanks!


Git branch string

Could a string token be added for showing the current git branch?

The extension doesn't have git support yet, just the button that takes you to the repo.
In the future, maybe I'll add this feature.😅

@LeonardSSH I have started implementing git branches into this extension.
@exiva Should I also add a repository string, and if so should it be extracted from the folder name or the remote URI?
Example: If I would to clone the myProject-docs repo into the docs folder of myProject (also a git repo).
Should the git repository string be myProject, myProject-docs or docs


Lose terminal scrollback after resizing in vim

Testing https://github.com/Microsoft/vscode/issues/19071
Not sure if this is a bug or expected, but it doesn't repro in the terminal
In Ubuntu 14.04

Have some output in the terminal like from ls -l
Open vim
:q
You can still see the output from ls, good
Open vim again
Resize the terminal
:q
The vim text is still on screen, and I can't scroll to see the output from ls.

Wow great find! Looks like resize needs to be made aware of the alt buffer.

Upstream issue https://github.com/sourcelair/xterm.js/issues/510

Should be fixed


feat: use pyproject.toml more heavily

Using pyproject.toml more heavily, now that mypy supports it.
I might remove setup.py from the regular setuptools version, as it's no longer needed, and encourages bad practice, like python setup.py <X>.

I need this for the next project here, so will merge - happy to take feedback!


tensorstore may not be necessary

Wanted to share this here as it may be useful and was inspired by your PR (https://github.com/napari/napari/pull/3155)
I found that a user of napari can get a zarr.Array to work in a labels layer by wrapping it like so:
import napari

import zarr
import wrapt
import numpy as np

class zarrWrapper(wrapt.ObjectProxy):
    """Add fancy indexing to zarr."""

    def __getitem__(self, key):
        """Get some data inplace maybe using numpy indexing"""
        try:
            return self.__class__.__getitem__(self, key)
        except IndexError as e:
            try:
                return self.vindex[key]
            except Exception:
                raise e

    def __setitem__(self, key, value):
        """Edit data in-place allowing for fancy indexing"""
        try:
            self.__class__.__setitem__(self, key, value)
        except IndexError as e:
            try:
                self.vindex[key] = value
            except Exception:
                raise e

z_arr = zarrWrapper(zarr.zeros((50, 512, 512), dtype=np.uint16))

viewer = napari.Viewer()
viewer.add_labels(z_arr)
napari.run()

Thanks @ianhi, this is very nice! I will say that (a) try/except is actually quite expensive in Python, so best to avoid it for something interactive like painting, but (b) I actually added fancy indexing to zarr arrays by default recently and this will be in release 2.11, so I'm just waiting for that to happen then we'll drop the tensorstore dependency. 😊

I actually added fancy indexing to zarr arrays by default recently and this will be in release 2.11, so I'm just waiting for that to happen then we'll drop the tensorstore dependency. blush

Amazing!


What should expect for windows and linux for var app = new Application({   path: '/Applications/MyApp.app/Contents/MacOS/MyApp' })

Hi. I would like to know what is suppose to remplace this part below for windows and linux;
var app = new Application({   path: '/Applications/MyApp.app/Contents/MacOS/MyApp' })

@xelfe it will be the path to the main executable.
So it would be something like C:/Users/kevin/Downloads/MyApp/myapp.exe on Windows and something like /home/kevin/Downloads/MyApp/myapp on Linux.
It is different for every application since name styles vary but it will be the path that you launch when you run the app normally.
Closing this out.


Add Contact Us details at the Footer

I would like to work on this issue can you please assign this to me

Hey @RahulSharma4329 i am searching for projects and issues for my very first hacktoberfest contributions .... can you please assign it to me .
Thanks in advance!

hey!I would love to work on this issue, can you please assign it to me

I also love to work on it.

Whoever wants to work, Create a PR for the Same.

please assign this to me


DNS Issue on Kubernetes (ndots=5 + search domain query)

Expected behavior
DNS Resolution works always
Actual behavior
Some hosts are not resolving and keep getting DnsNameResolverTimeoutException.
SearchDomainUnknownHostException: Search domain query failed. Original hostname: 's3-eu-central-1.amazonaws.com' failed to resolve 's3-eu-central-1.amazonaws.com.default.svc.cluster.local' after 2 queries
    at io.netty.resolver.dns.DnsResolveContext.finishResolve(DnsResolveContext.java:845)
    at io.netty.resolver.dns.DnsResolveContext.tryToFinishResolve(DnsResolveContext.java:806)
    at io.netty.resolver.dns.DnsResolveContext.query(DnsResolveContext.java:333)
    at io.netty.resolver.dns.DnsResolveContext.query(DnsResolveContext.java:322)
    at io.netty.resolver.dns.DnsResolveContext.access$500(DnsResolveContext.java:62)
...
(17 additional frame(s) were not displayed)

DnsNameResolverTimeoutException: [/<IP_ADDRESS>:53] query timed out after 5000 milliseconds (no stack trace available)

The issue is not only relevant for cluster external dns entries, but also when querying <service-name>, which should resolve when using search path. using the FQDN <service-name>.default.svc.cluster.local makes the issue less noticable, except for external names due to the ndots.
The DNS server (coredns pod) is located on the same kubernetes node. performing dns queries on the shell using dig +search [...] or nslookup always yield the correct result
Steps to reproduce
inside a kubernetes pod, try resolving internal names and external names. doesn't seem to be a network issue, I've seen it on weave (with the tc fix applied) and flannel-vxlan and all other non-netty pods don't have any issues with DNS
this is the /etc/resolv.conf
nameserver <IP_ADDRESS>
search default.svc.cluster.local svc.cluster.local cluster.local eu-central-1.compute.internal
options ndots:5

this rules out the fix applied for #8261 because it's a single dns server
Minimal yet complete reproducer code (or URL to code)
working on that part right now. Really hard to make it reproducable. I've tried ruling out everything else, see above
Netty version
4.1.30 (through Vert.x 3.6.3)
JVM version (e.g. java -version)
8
OS version (e.g. uname -a)
CentOS 7.6 (Docker)

@m1schka let me know once you have a reproducer

Also please try to update to latest Netty version and see if this still exists.

@m1schka any update ?

@m1schka Actually, this might be the problem with kube-dns not with Netty. Here is a long discussion about performance problems with the default settings of search domains + ndots=5. Is this the case?

sorry for the delay.
@normanmaurer it's hard for me to write a reproducer since I'm not directly using netty but vert.x. any help is appreciated.
It still occurs like ~1-2 times per day but significantly less after changing to FQDN for kubernetes services (i.e using .<namespace>.svc.cluster.local).
@kachayev I'm not using kube-dns but coredns. I'm aware of issues with kube-dns. Also I don't use an alpine based image (which shouldn't matter anyway, since the dns query is not done by musl) and I shouldn't be affected by conntrack issue because I installed weave-tc.
Next step for me is disabling the ipv6. maybe that helps

I'm closing this issue until I've got a full picture on this issue. also watching https://github.com/weaveworks/weave/issues/3287 in the meantime

disabling ipv6 seems to fix this issue

@m1schka
Seems like I have the same issue. Also using kubernetes + vertx + netty.
disabled IPv6 with ENV JAVA_OPTS="$JAVA_OPTS -Djava.net.preferIPv4Addresses=true -Djava.net.preferIPv4Stack=true" but still see the issue time to time (~ once a day for now).
Any suggestions? Did you fix the issue completely?

We had this same issue and found it was related to a linux kernel udp bug, detailed here:
https://blog.quentin-machu.fr/2018/06/24/5-15s-dns-lookups-on-kubernetes/
What we ended up doing was adding a single-request-reopen dnsC=onfig for our deployments to ensure we had sequential lookups.  This 100% fixed our problem, e.g.
    spec:
      dnsConfig:
        options:
          - name: single-request-reopen

Incidentally the alpine linux base images we were using use libmusl rather than glibc, so the single-request-reopen option for resolv.conf did nothing until we changed our base images to debian:stretch (since it is glibc based)

https://pracucci.com/kubernetes-dns-resolution-ndots-options-and-why-it-may-affect-application-performances.html


I want to save the result md files to my disk

can you give me an example code?

Its just markdown, do you can write to disk like any text in python?
with open("file.txt", "w") as f:
  f.write(document.text)


[BUG] Metabase/SendGrid email integration uses Branded Link with misconfigured certificate

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Metabase emails sent through SendGrid integration redirect users to a URL with an invalid certificate. Users are unable to proceed, as most modern browsers block progression.
When receiving a welcome email, password reset email, or other communication from Metabase, links in the email are formatted as follows:
https://url9996.simplereport.gov/ls/click?upn=<snipped>
When navigating to a link like this, the user is presented with the following error screen:

Further details show that the certificate is issued to the *.sendgrid.net domain:

We should research how Branded Links are configured, and ensure that certificate issuance isn't interfering with user interactions.
This was reported by another user a few weeks ago, when they attempted to access a contact form. @emmastephenson may be able to provide further context on this.
Steps To Reproduce

Receive a link to reset password or create account from Metabase.
Click the associated action link in the email.

Expected Behavior
A user should be properly redirected to Metabase without a certificate error.
Additional context
No response
Does this issue have a priority label?

[X] I have added a priority label

Has issue been assigned to a team?

[X] I have added a team label

Additional note: This behavior is reproducible in multiple browsers on Windows 11, including Brave, Edge, and Firefox, as well as Chrome on Ubuntu 21.

Possible research items for fixes:
https://github.com/sendgrid/docs/issues/5496
https://stackoverflow.com/questions/61169431/sendgrid-link-branding-gives-ssl-issue
https://stackoverflow.com/questions/52052585/sendgrid-brand-link-with-nginx-proxy
https://www.digitalocean.com/community/questions/sendgrid-link-branding-custom-ssl-with-nginx

Can we also check to make sure we have followed all the steps in the branded setup guide, located at the URL below? Sender Authentication is not available to me, but I'm not sure whether that's due to permission levels or missing pieces.
https://docs.sendgrid.com/ui/account-and-settings/how-to-set-up-link-branding

@emmastephenson to sync with @timothybest-usds on why we redirect from sendgrid.net in the first place, and if it can be taken off.

Tim's vote is to just remove the link branding entirely, which should fix this problem.

Removed link branding and that seems to have taken care of it. Thanks for all the investigation you did on this one @rin-skylight!


Step 6.Install APD DC-DC co......

When doing "6.Install APD DC-DC components (Q1 and others). Check "APD_V" - it must change by manual commands from utility."
I found I only could adjust the APD Volt in scope 0 ~ 59.8V, like below pictures, whatever I changed the APD PWM and APD Targ.Volt. Should it be correct?

Have you measured APD voltage by voltmeter. Is the value the same as displayed in utility?
Also you can check this (type of inductor): https://github.com/iliasam/OpenTOFLidar/issues/5#issuecomment-690827160

Yes, it is same although some difference, for example, 12.5V in utility, 13.6V in voltmeter.
Let me check #5

"In any case, it is not OK that measured APD voltage (63V) is not the same as set target voltage (128V). It is look like APD step-up converter or APD voltage measurement circuit are working wrong." as your repled in #5, let me check if it is .

13.6 / 12.5V = 1.088, -> ~9% error, which is big. Can you measure difference at higher voltage  (40-50V)?

when  47.3V in utility, 46.4V in voltmeter

"Don't try to switch to Auto mode until you get well working manual mode." you mean first use the PWM duty to get higer voltage then use the AUTO mode to set?

And one more : when 13.8V in utility, 11.9V in voltmeter

"Hello.The APD voltage's problem solved. The problem is the inductor L5 type not value.
The manufacturer gave a multilayer inductor, but the actual winding type. There is no description about L5 type in the BOM."
I think it is same issue, I will buy this one and to try.

"you mean first use the PWM duty to get higer voltage then use the AUTO mode to set?"
You need to get stable high voltage  (< 110V) by setting PWM duty manually.

Hi, iliasam,
Thanks for your advice, after changing to the leading inductor, I got below picture:

And after changing to the high precise resister, I got the 103.7V on voltmeter and 103.1V on PC utility.

but what max volt I could achieve is 122V, is it correct?

Breakdown voltage of MTAPD-07-013 is in 120-130V.
When APD is installed, you can the that APD is close to breakdown by seeing more noise at the TIA output.
Also, APD voltage would increase slowly (comparing to low voltage - not breakdown mode) as shown here: https://github.com/iliasam/OpenTOFLidar/blob/develop/Images/APD_voltage_vs_PWM.png (for AD500-8).

But it is strange that you can't get 130V. Do you have APD installed or not?

No, I havent installed the APD, and my APD tupe is MTAPD-07-007.
What is the maximum PWM value that you have set?
200 steps, as above picture.

MTAPD-07-013 is on the delivery way

The plot that I mentioned earlier, was measured with AD500-8 which have breakdown voltage ~95V. This plot is for information only.
This image is from "Issue 5": https://user-images.githubusercontent.com/48154419/92838938-f75e5f80-f411-11ea-96e4-dbe952ec430b.png
You can see that "Anesthesia-RW" can get 124.5V at 72 PWM steps.
I get 130V at 73V: https://ibb.co/vjFfwVD
All these measurements were made with APD installed, but I don't think that it is important.
Cleaning PCB could be important.
Check R8, R9 values, check that C10, C11 are high-voltage capacitors.

After clearing the PCB, I could get the maximum volt is 126V.
I just put here my soldered components, maybe you have some clue.

Your Q1 and D1 are looking OK.
You can try to shorten L4, but I don't think that this will help.
Maybe it is really problem with C10,C11 working woltage?

Because in china market, the  MTAPD-07-013 needs too long to get, and there is no AD500-8, So I buy a AD500-9 to try.

Do you think the AD500-9 will work?

Why there is no AD500-8? https://aliexpress.com/item/32396227457.html
Also I can see them in Taobao.
It looks like AD500-9 have higher breakdown voltage, than AD500-8. Also, you need to remade PCB for any leaded APD.
I think you need to find out why your circuit can't produce higher voltage.

Oh, thank you for your reminder, after several times of searching in Taobao, I found one AD500-8.
"I think you need to find out why your circuit can't produce higher voltage."
I will try more.


performance enhancement

i have been using storio from yesterday and i don't fully understand why DefaultPutResolver#performPut uses a Cursor to find out whether to call insert or update which degrades the performance a bit, why not internally use storIOSQLite.internal().insertWithOnConflict() with android.database.sqlite.SQLiteDatabase#CONFLICT_REPLACE conflictAlgorithm? that way mapToInsertQuery and mapToUpdateQuery could be unified maybe?

Simply because logical check on our side gives more predictable result,
especially if you have multiple logical keys which is possible.
Feel free to change implementation to insertWithOnConflict!
On Mon, Nov 23, 2015, 17:51 pskink<EMAIL_ADDRESS>wrote:

Closed #566 https://github.com/pushtorefresh/storio/issues/566.
—
Reply to this email directly or view it on GitHub
https://github.com/pushtorefresh/storio/issues/566#event-472126095.

--
@artem_zin


fix: Disable Zoom feature for tables on Markdown files

:tada: This PR is included in version 3.13.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


This adds multiline text widget support.

This adds support for multi-line text widgets, making a new TextInputAction.newline enum, and supporting a new TextInputType.multiline enum value.
There are still some issues with cursor placement in multi-line input, but those may go away when we switch to a non-Blink text renderer.
Fixes #8028

 modulo comments

Nice!

modulo comments too.


ETCD configuration is ignored Issue with k8s.io/docs/setup/independent/high-availability/

This is a...

[ ] Feature Request
[X] Bug Report

Problem:
HA Bootstrapping with stacked control plane does not work, etcd options are ignored.
Proposed Solution:
The document says that the configuration has to look like:
etcd:
local:
extraArgs:
listen-client-urls:
.....
I had to remove one level and change the configuration to:
etcd:
extraArgs:
listen-client-urls:
.....
It looks like the 1.11 configuration is not in sync with kubeadm 1.11
Page to Update:
https://kubernetes.io/docs/setup/independent/high-availability/

Kubernetes Version:
kubeadm version:
kubeadm version: &version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.0", GitCommit:"91e7b4fd31fcd3d5f436da26c980becec37ceefe", GitTreeState:"clean", BuildDate:"2018-06-27T20:14:41Z", GoVersion:"go1.10.2", Compiler:"gc", Platform:"linux/amd64"}
rpm -qa:
kubelet-1.11.0-0.x86_64
kubectl-1.11.0-0.x86_64
kubernetes-cni-0.6.0-0.x86_64
kubeadm-1.11.0-0.x86_64

/assign @detiber @rdodev @chuckha

/close
this may have been broken for 1.11 but we've vetted for 1.12 and the versions are no longer mismatched


[Feature]: [validations] Extracting requirements from JSON

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
When we get the form config, we have to extract the validation config and create validation paths.
As an FE Coder, I should be able to extract the validation keys from the config JSON and set them as validation paths, so that the validation service can run the validations on the components.
Corner cases: None
Test cases: additional test cases will be required
To be planned separately
Documentation required: Yes
To be planned separately
Note: this is part of the larger validation story, details here.
Why should this be worked on?
As a Plugin developer, I should be able to enable real time validations on the query and ds forms, so that the developer can know when they have entered something wrong

Waiting for backend

Verified as a part of regression


Not able to sign with "i" or "." in the signature

Thanks for the code.
Im trying to sign with  "i" or "." in the signature but not able to do .

Please refer new release 1.0.2 💯


add support for Projects;

Checklist for your PR

[ ] please start with a draft PR and mark it as ready for review when you are confident that your implementation works as expected.
[ ] add a changelog entry in CHANGELOG.md with a link to this PR.
[ ] make sure that all tests pass and add new test functions if your code is not covered by existing tests. Please use mock tests when network requests are required.
[ ] let someone review & approve your code, make changes where necessary.

Hints

name your branches accordingly: use the prefixes bugfix/, feature/, or release/ and then a short title for your PR in kebab-case (lowercase words separated with dashes), e.g. feature/my-awesome-feature.
Please make sure that pre-commit hooks are enabled on your system to avoid common mistakes before you make a commit.
feel free to delete this template and replace it with your own description of the PR.

Codecov Report
Base: 86.71% // Head: 84.43% // Decreases project coverage by -2.27% :warning:

Coverage data is based on head (6a310c8) compared to base (3cd11f6).
Patch coverage: 47.05% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #30      +/-   ##
==========================================
- Coverage   86.71%   84.43%   -2.28%
==========================================
  Files           7        7
  Lines         843      893      +50
==========================================
+ Hits          731      754      +23
- Misses        112      139      +27

Impacted Files
Coverage Δ

src/personio_py/__init__.py
100.00% <ø> (ø)

src/personio_py/client.py
71.56% <18.51%> (-7.90%)
:arrow_down:

src/personio_py/models.py
89.30% <79.16%> (-0.58%)
:arrow_down:

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.

The PR is missing tests with mocking the API, would be great to know where to add them best or what the intended structure should be. Thanks!


⚠ Bofrost.at has degraded performance

In 31eef0c, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1173 ms

Resolved: Bofrost.at performance has improved in b02b646.


Create a Technoshard

Description: This is a MobileEntity, so it will need to inherit the MobileEntity class, and it will be stored in an entity pool separate from the actual GridSpaces or map scenery. Make it a single GameObject prefab and we’ll load them in automatically and spawn them at will using the MapGenerator. As a result, it will need to be self-contained within that GameObject
(e.g., all scripts and particle systems and models and such should be components to the
GameObject or children of it)
Graphical:
- get a model, texture, prefab for the shard
- stick a particle system on it so it emits a blue techno fog
Sound:
- should have a function collect() that destroys it and plays a noise
Implementation:
- should float and spin, possibly (not necessary, do it however you want)
- touching it will end the game for now (you can literally close the game upon collision with a Player object)

So far it is not a mobile entity but rather a prefab from : https://www.assetstore.unity3d.com/en/#!/content/73764
and a particle system from :
https://www.assetstore.unity3d.com/en/#!/content/32351

Needs to be packaged up into a prefab and made into a MobileEntity so we can spawn it with the MapGenerator eventually
for now it needs a nice particle system attached to it that is by default off, so i can turn activate it when we grab the Technoshard and it will spew out a few particles for a second to show we grabbed it
this will satisfy one of the Particle System requirements for demo 2 in case we don't get the turret particles done in time. You can just use one of the ones they gave us with the pack.


added largest subarray sum algorithm, AVL Tree remove function, egg dropping, all permutations algorithms

Hey guys I would like to contribute a few famous algorithms.  Here's the first one.

@arslan-ashraf
Thank you for your PR. But it looks like it contains duplicated work. Please take a look at the algorithms that already exists in the repo. For example we already have Kadane's algorithm, permutations and AVL tree removal.
Currently I'm closing this PR but feel free to create a new one if you will commit some new functionality. Keep PR focused so that only one algorithm should be suggested in one PR. Thank you!

Useful


New plugin options and SMIL support

Hi,
At eventials.com our live streaming is primarily based on SMIL manifests. In order to support this format, I'm using OSMF's SMILPlugin. It handles all the steps (loading, parsing and creating the DynamicStreamingResource) to pass to OSMF media player.
Since now there are plugin specific options, I created rtmpConfig with the options:

swfPath (default //cdn.jsdelivr.net/clappr.rtmp/latest/assets/RTMP.swf) - Path to the SWF responsible for the playback.
scaling (default letterbox) - Path to the SWF responsible for the playback.
playbackType (default live if the source contains live on its URL, vod otherwise).
bufferTime (default 0.1) - How long to buffer before start playing the media.
startLevel (default 0) - Initial quality level index.

Without these new options the player will still work, but we can change the default values if you want.
Hope you enjoy these changes.

woww!! Sorry for taking so long to accept the PR, thanks for your effort @gfronza! :beers:

@gfronza since I'm a bit busy with other projects, I added you as commiter on this plugin so you can keep up the good work. Thanks and feel free to add/fix/change stuff on this plugin.

@flavioribeiro Outstanding! Thanks for accepting it and for adding me as a committer. I'll do my best to push this plugin even further! BTW great job you guys are doing with Clappr!! :+1:


Mention of sumner

The following sentence in the README is JAX-specific
On sumner, these scripts can be run in the singularity directory with

should we remove this (or alternatively provide a bit more explanation of how to run this with singularity?)

Eliminated sumner from the offending sentence.


Version 0.5.0 changelog

What this PR does:
Which issue(s) this PR fixes:

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

Codecov Report

Merging #379 (ebbf498) into develop (246bb65) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #379   +/-   ##
========================================
  Coverage    39.61%   39.61%
========================================
  Files           54       54
  Lines         3463     3463
========================================
  Hits          1372     1372
  Misses        1929     1929
  Partials       162      162

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 246bb65...ebbf498. Read the comment docs.


Fixed NPE when suite does not contain started tests.

The error of converting a Long object to a long primitive if the Map does not contain an object.
Fixes #1931
Closes #1931

[x] Add test case(s)
[x] Update CHANGES.txt

Fix bugs in TestNG

@juherr  review please.


Update README.md

Added ThatsMyAI providing APP listing as well as AI certifications

Nice one. You can also add it on Find AI Tools on reddit


Revisit dropdown component to update styles

The dropdown was an early component and since then I think we've changed our approach. There are some styles that can be cleaned up.
The dropdown toggle when just an icon is also only 28px high. We should look at increasing this so it matches all other actionable icons (pf-c-button.pf-m-plain).
This also includes the custom select component as it pulls a lot of styles from the dropdown.

@matthewcarleton can we close this one out? the plain toggles in the dropdown components are all a consistent height now.

Thanks for the update @mcoker. I'll pull the trigger and close :)


Uptime Kuma v2 support

This issues is meant to keep track of support for the upcoming version 2.0 of uptime-kuma, currently available as a beta.
Since the api of v1 and v2 have api incompatibilities there's a separate version of autokuma for v2. This is made available as docker tags with the prefix uptime-kuma-v2- e.g. to get the latest dev version with v2 support use
docker pull ghcr.io/bigboot/autokuma:uptime-kuma-v2-master

to pin a specific commit:
docker pull ghcr.io/bigboot/autokuma:uptime-kuma-v2-sha-23287bc

For source builds v2 support can be enabled using a feature flag, i.e.:
cargo install --git https://github.com/BigBoot/AutoKuma.git --features uptime-kuma-v2 kuma-cli

Status
Notes

Monitor
✅

Docker Host
✅

Notification
⚠*
Same issues as v1

Status Page
❓
Untested

Maintenance
❓
Untested

Tags
✅
There currently seems to be some problems with assigning tags/values to monitors in V2, this also happens using the UI

This works perfectly. Thank you for implementing it so quickly.

One thing I noticed with my current config is when I did a fresh compose up -d on uptime kuma itself, autokuma created a second copy of the monitors I currently had instead of detecting them. They were created both times by Autokuma.

I've also noted a lot of this:

WARN [kuma_client::util] The server rejected the login: Too frequently, try again later.
WARN [kuma_client::util] Error while handling 'Info' event: The server rejected the login: Too frequently, try again later.
WARN [kuma_client::client] Timeout while waiting for Kuma to get ready...
WARN [autokuma::sync] Encountered error during sync: It looks like the server is expecting a username/password, but none was provided
WARN [kuma_client::util] The server rejected the login: Too frequently, try again later.
WARN [kuma_client::util] Error while handling 'Info' event: The server rejected the login: Too frequently, try again later.
WARN [kuma_client::client] Timeout while waiting for Kuma to get ready...
WARN [autokuma::sync] Encountered error during sync: It looks like the server is expecting a username/password, but none was provided
WARN [kuma_client::util] The server rejected the login: Too frequently, try again later.
WARN [kuma_client::util] Error while handling 'Info' event: The server rejected the login: Too frequently, try again later.
WARN [kuma_client::client] Timeout while waiting for Kuma to get ready...
WARN [autokuma::sync] Encountered error during sync: It looks like the server is expecting a username/password, but none was provided
WARN [kuma_client::util] The server rejected the login: Too frequently, try again later.
WARN [kuma_client::util] Error while handling 'Info' event: The server rejected the login: Too frequently, try again later.
WARN [kuma_client::client] Timeout while waiting for Kuma to get ready...
WARN [autokuma::sync] Encountered error during sync: It looks like the server is expecting a username/password, but none was provided
WARN [kuma_client::util] The server rejected the login: Too frequently, try again later.
WARN [kuma_client::util] Error while handling 'Info' event: The server rejected the login: Too frequently, try again later.
WARN [kuma_client::client] Timeout while waiting for Kuma to get ready...
WARN [autokuma::sync] Encountered error during sync: It looks like the server is expecting a username/password, but none was pro

Thanks, I just bumped up the sync interval.
Re: the first point, this is a net new test setup with only those containers. I just brought the stacks up, down, then up again. There isn't an easy migration path from v1 to v2 with current historical data, they're basically stating due to bandwidth reasons they won't officially support it, but others have posted info on how to create a mysql database and populate it with converted sqlite data via an export.

Hi!
The latest update seems to have broken autokuma for ukv2. I recreated my uptimekuma server but ak can't create any monitors. It throws out the following logs repeatedly:
autokuma-1  | WARN [kuma_client::util] [backups-nasty.toml] No monitor named backups could be found
autokuma-1  | WARN [kuma_client::util] [backups-nasty-other_devices.toml] No monitor named backups-nasty could be found
autokuma-1  | WARN [kuma_client::util] [backups-vivo.toml] No monitor named backups could be found
autokuma-1  | WARN [kuma_client::util] [backups-nasty-self.toml] No monitor named backups-nasty could be found
autokuma-1  | WARN [kuma_client::util] [backups-asus_viki.toml] No monitor named backups could be found
autokuma-1  | WARN [kuma_client::util] [backups-nasty-db.toml] No monitor named backups-nasty could be found
autokuma-1  | WARN [autokuma::entity] Cannot create monitor uptime-kuma because referenced monitor with services is not found
autokuma-1  | WARN [kuma_client::util] Error while parsing uptime-kuma-auto-kuma: data did not match any variant of untagged enum EntityWrapper!
autokuma-1  | WARN [autokuma::sync] Encountered error during sync: Error while trying to parse labels: data did not match any variant of untagged enum EntityWrapper

The referenced groups are supposed to be created  by ak too. For the enum problem I haven't had time to actually debug the code, but the latest uk update did have a pr merged for monitor tags: https://github.com/louislam/uptime-kuma/pull/5298
I didn't change any of the already working monitor definitions and am running the images louislam/uptime-kuma:beta (sha256:752118f891ea991180124e3fc7edbc1865a58cb03e15e612ecbc68065b1d4b9f) and ghcr.io/bigboot/autokuma:uptime-kuma-v2-master (sha256:74bccf145554cce2acf63676d4b98fafdf1e710e60150733fcac8b5b1c364301)
Thanks for the help and all the good work you do!

Hi @bnctth, I don't think there's any breaking change, this looks rather like a problem with your labels, let's try to break this down.
autokuma-1  | WARN [kuma_client::util] [backups-nasty.toml] No monitor named backups could be found
autokuma-1  | WARN [kuma_client::util] [backups-nasty-other_devices.toml] No monitor named backups-nasty could be found
autokuma-1  | WARN [kuma_client::util] [backups-vivo.toml] No monitor named backups could be found
autokuma-1  | WARN [kuma_client::util] [backups-nasty-self.toml] No monitor named backups-nasty could be found
autokuma-1  | WARN [kuma_client::util] [backups-asus_viki.toml] No monitor named backups could be found
autokuma-1  | WARN [kuma_client::util] [backups-nasty-db.toml] No monitor named backups-nasty could be found

These are "expected" when creating nested setups, autokuma knows about these dependencies but they have not been created yet, as a result they are skipped till later.
autokuma-1  | WARN [autokuma::entity] Cannot create monitor uptime-kuma because referenced monitor with services is not found

this one says that you have a monitor referencing a parent monitor with the autokuma id "services", however there doesn't seem to exist any such monitor definition.
autokuma-1  | WARN [kuma_client::util] Error while parsing uptime-kuma-auto-kuma: data did not match any variant of untagged enum EntityWrapper!
autokuma-1  | WARN [autokuma::sync] Encountered error during sync: Error while trying to parse labels: data did not match any variant of untagged enum EntityWrapper

This error unfortunately isn't as clear, but it basically means you have a definition (uptime-kuma-auto-kuma) with a missing or invalid "type".

@BigBoot  thanks for your reply! Turn out I had a pretty trivial problem mixed with some red herring-y error messages: I had a typo in a snippet (notificationIDList instead of `notificationIdList˙, small d in Id), so I had the correct definition for the parents, but because of the error it never got to their definition.