Minor modification to New Scenario En algún lugar de Massachusetts (v1.0/Base + RN/Español/3 Investigadores) Parte 1

Description of Problem or Question
https://github.com/newusergettingnuts/Massachusetts
Hi i made some minor changes to the scenario because the ending review was not functioning.
But i cant see it taking effect .theres something i may be doing wrong or every update i do you have to add the link again?
Valkyrie Version
Attach Save, Quest or Scenario
Saves:
%appdata%/Valkyrie/MoM/Save
%appdata%/Valkyrie/D2E/Save
Quest:
%appdata%/Valkyrie/D2E/Editor
Scenario:
%appdata%/Valkyrie/MoM/Editor

Update of you file was done 16 hours ago, so it should be fine.
Please note you need to update your scenario in Valkyrie, but going in the "Download section" and pressing "update" button

Ben many thanks, my mystake, sorry.-
It works well now.
I have linked all the possible endings including the defeated event trigger (trigger: defeated......assign $end=1) so as to correctly measure success and failures.
This way if you succeed it triggers the review and if you loose also triggers the review.
again many thanks for the help!
:) moving forward to next scenario.


Fixing atomated generation of documentation.

Reference Issues/PRs
N/A
What does this implement/fix? Explain your changes
Simple PR that fixes various documentation-related issues and ensures that the documentation can be rendered on GitHub Pages.
What testing did you do to verify the changes in this PR?
Extensive local testing and bug-fixing of the Docs were performed. These changes are entirely cosmetic and do not affect any functional aspects of the codebase.
Pull Request Checklist

[x] Added a note about the modification or contribution to the ./CHANGELOG.md file (if applicable)
[x] Added appropriate unit test functions in the ./gauche/tests/* directories (if applicable)
[x] Modify documentation in the corresponding Jupyter Notebook under ./notebooks/ (if applicable)
[x] Ran python -m py.test tests/ and make sure that all unit tests pass (for small modifications, it might be sufficient to only run the specific test file, e.g., python -m py.test tests/kernels/test_graph_kernels.py)
[x] Checked for style issues by running black . and isort .

As these changes are minor and just relate to the rendering of docs I will directly merge the PR.


1470 interview notes on recruitmentpositionoverviewpage

Link: http://localhost:3000/control-panel/recruitment/view-applicant/901e8909-401b-4bca-8465-4bb2b5e0bfcb/
Most of the frontend for the linked issue is solved.

Refactored page to use useQuery and useMutation
Used react-hook-form for the notes. Saves using onBlur. The put request is not implemented and will be an own issue.

applicationID vil ikke la seg skrive som applicationId uten at siden crasher. Vet ikke hvorfor. Endret derfor tilbake til applicationID.


mongocxx: 3.8.2 → 3.10.2

Description of changes

Updated mongocxx from 3.8.1 to 3.10.2

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] 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
[x] 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.

@IogaMaster Thanks for your guide. Maybe you can take a look ;)

Based on convention, you should change the title of this PR from Update mongocxx #324374 into mongocxx: 3.8.2 → 3.10.2

Based on convention, you should change the title of this PR from Update mongocxx #324374 into mongocxx: 3.8.2 → 3.10.2 This will make it easier for others to view the specific behavior of the PR implementation

Ok thank's

Sorry, you should spilt the action of add maintainer and update package in two commit. It was my mistake.

And Base on Linking a pull request to an issue you could add following into your Description of changes.
Closes #323974

I am just struggling a bit with git. I don't quite know how the interactive rebase editor works yet.

Closes #323974

I am just struggling a bit with git. I don't quite know how the interactive rebase editor works yet.

Ok got It can you take a look if it's all correct now?

The commit message is correct right now. Great jobs

pkgs.darwin.apple_sdk.frameworks.Security
How can I add this as an dependency only for darwin?

Security

Maybe you could use some kinds of things like
buildInputs = [] ++ lib.optional stdenv.isDarwin darwin.apple_sdk.frameworks.Security;

@Vcele
Maybe you could use some kinds of things like
buildInputs = [] ++ lib.optional stdenv.isDarwin darwin.apple_sdk.frameworks.Security;
I not a kind of darwin user ( i not even have one ), so just a guess.

Me neither, but I can try that. I just saw that the build failed for that system in the checks.

@adriandole if you've got time to give this another review maybe you got time to merge this via the nixpkgs-merge-bot 👍

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

1 package built:

mongocxx

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

1 package built:

mongocxx

@NixOS/nixpkgs-merge-bot merge


RTMP Pull and hls

I've just started using Nginx with rtmp module. I set up nginx.conf to "pull" from exsiting rtmp link
 application stream {
            live on;
            pull rtmp://<IP_ADDRESS>:1937/nocodehd live=1;

            hls on;
            hls_nested on;
            hls_path /mnt/hls/stream/;
            hls_fragment 3s;
            hls_playlist_length 10;
            hls_type event;

        }

Then I config for hls to alias /mnt/hls/stream/  now I can open RTMP but I'm not understand that why If I'm not open RTMP parallel with HLS there is no playlist created in /mnt/hls/stream/ it's mean that If I call only HLS Nginx will return "not found" and I checked that there is no playlist file in path.
What's mistake of configuration ?
How can I config Nginx to create playlist when call only HLS ?
Another question is I configed "secure_link" and "secure_link_md5" to HLS and now it's work for protect playlist.m3u8 but playlist didn't include secure_link after .ts so Nginx return "forbiden" for .ts inside playlist.m3u8.
How can I fix this problem ?
location /test {

            add_header Cache-Control no-cache;

            secure_link $arg_md5,$arg_expires;
            secure_link_md5 "$secure_link_expires$uri$remote_addr PASSWORD";

            if ($secure_link = "") { return 403; }
            if ($secure_link = "0") { return 410; }

            types {
            application/vnd.apple.mpegurl m3u8;
            video/mp2t ts;
            plain/text key;
            }
            alias /mnt/hls/stream/;

        }

pull is only triggered when a client is connected and playing the stream.
When there's no active rtmp client, the stream just terminated, so there no content to save to hls folder.
you can run a exec_static ffmpeg ... to pull the stream automatically

exec_static ffmpeg

pull is only triggered when a client is connected and playing the stream.
When there's no active rtmp client, the stream just terminated, so there no content to save to hls folder.
you can run a exec_static ffmpeg ... to pull the stream automatically

Can I wildcard exec_static ffmpeg pull like normal "pull" command ?

Can I wildcard exec_static ffmpeg pull like normal "pull" command ?

no

Another question is I configed "secure_link" and "secure_link_md5" to HLS and now it's work for protect playlist.m3u8 but playlist didn't include secure_link after .ts so Nginx return "forbiden" for .ts inside playlist.m3u8.
How can I fix this problem ?

I had the same problem myself, which I raised in the StackOverFlow forum a little while ago. But I found the answer myself.
Please see this answer with code snippet - https://stackoverflow.com/a/59241113/1213682


Crashed when create EventProducer Object on iOS 7, Xcode 7.3

Crashed when create EventProducer Object on iOS 7
Xcode 7.3
Crashed here
let tmpSelfReference = Reference(weak: self)

I got this issue in the same point, using Xcode 7.3 also.
Have any solution for this?
Thanks

oh man! me too! haha please tell me you found a solution! @srdanrasic any idea?

Hmm, that is quite puzzling. Any additional information you can provide might help. Under what circumstances does it happen? What transformations are applied? Thanks!

This is a Swift 2.2 bug, update Xcode to 7.3.1 can solve this issue.


Errors when SemanticBot is not authorized to write in results directory

A mapping file is exported in result/ directory when semantization process is over.
When SemanticBot is deployed, it has not always the authorisation to write in source repository.
In this case, it result in a blocking error at mapping generation
To fix this problem:

Make the result directory settable in the settings.py local_settings.py.
Surround file writing code with a try catch

Solved in the next semantic-bot version


baseUrl option for ConfigurationProvider added

Possibility to set baseUrl (https://doc.courtbouillon.org/weasyprint/stable/api_reference.html#cmdoption-u). By default, it sets "." so no intervention with the previous behavior. It helps to locate static assets in Blazor, like fetcing .css files and images

Thank you very much for the contribution!


Update uncompressed-rgb-video-subtypes.md

Changed description of MEDIASUBTYPE_A2B10G10R10.

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


Libre.fm support

Can you add libre.fm integration? Libre.fm is an open source alternative to last.fm.

How would you imagine "integration" is for libre.fm?

Integration = publish the current music on libre.fm profile :)
You can use the same last.fm audioscrobbler api, just need to point to turtle.libre.fm instead of post.audioscrobbler.com.

:+1:

If it's open source, the endpoint should probably also be configurable in case you want to host your own instance.


Update vendor and unpkg code in online.html

Version updates for everything.  Perhaps switch jsDump to node-inspect-extracted

I think, we should turn vendor dependencies into ordinary npm dependencies and build browser bundle from them. For example, we could update CodeMirror to version 6, there even is an example, how to bundle it with rollup.

That makes some sense to me.  I do want to make sure we're using the peggy build that everyone else can get, not bundling it up.  I use that as a sanity check to make sure the minified build is working.


Switch to case-insensitive check to fix "duplicate" parameter error

Rationale
IssueUpdateEmailTemplate used a case-sensitive check to detect existing parameters. Because of field name casing differences, this meant duplicate replacements were being added to the list. Recently added strict test flagged this bug.
Changes

Use a case-insensitive hash map to check existing parameters
For convenience, add a new Collector that produces case-insensitive hash maps
For consistency, use the Replacement class to add parameters within issue's funky custom field handling

Issues suite passed on this branch: https://teamcity.labkey.org/viewLog.html?buildId=1475752&tab=buildResultsDiv&buildTypeId=LabkeyTrunk_ModuleSuites_IssuesPostgres

AssayRequestTest also passed: https://teamcity.labkey.org/viewLog.html?buildId=1475905&buildTypeId=LabkeyTrunk_GitPostgres&tab=testsInfo&branch_LabKey_Trunk_Premium=typeSummaryReport


⚠ Mobitopup has degraded performance

In 19bced3, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 3291 ms

Resolved: Mobitopup performance has improved in 6509842.


chore: add raw filepath for logo in readme

Add the raw file path for the logo
The current path will point to the file in the repository, but not the the path that will return the image to display it. Github itself has some auto correct for this, but when you are referencing the readme from another source, it will fail to load it.

Thanks!

@TimoGlastra I saw there was a linebreak when I added it... The link has to be in the same line and not in a new one... sry for not previewing it first!


radar_camera_calibrator dont work

We appreciate you go through Apollo documentations and search previous issues before creating an new one. If neither of the sources helped you with your issues, please report the issue using the following form. Please note missing info can delay the response time.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 14.04):14.04
Apollo installed from (source or binary):source
Apollo version (1.0, 1.5, 2.0, 2.5, 3.0):3.0

Steps to reproduce the issue:

Please use bullet points and include as much details as possible:
download the calibration.tar.gz
extraxct to modules/calibration
bring up bootstrap usbcam  conti_radar gnss
topics works very well
Sensor | Topic Name | Topic Feq. (Hz)
-- | -- | --
Short_Camera | /apollo/sensor/camera/traffic/image_short | 9
INS | /apollo/sensor/gnss/odometry | 100
INS | /apollo/sensor/gnss/ins_stat | 2

When excuting
bash sensor_calibration.sh camera_camera
It Just hang up there.
CMD shows
"Start to calibrate Radar-Camera extrinsics"
And Nothing else.
Supporting materials (screenshots, command lines, code/script snippets):
Using strace to find out what happens,It Seems like Need a mutex?

@sinopec apologies for the delay in responding to your question. Calibration is currently offered as a service by the Apollo team. The tool that you referred to on Apollo is not supported in the latest versions. If you would like to learn more about the calibration service, feel free to reach out to me at<EMAIL_ADDRESS>


ExtendedAdapterInfo not exported

On current 0.1.0 rend3::renderer::info::ExtendedAdapterInfo is required to construct an InstanceAdapterDevice, but renderer module is private and the EAI is not re-exported publicly. Apparently ambient lights have the same issue
sorry for lazy issue im lazy

Published in rend3 0.1.1


Added Avif and WebP support for slider images

Questions
Answers

Description?
I have added 2 new supported image extensions. Avif and WebP.

Type?
improvement

BC breaks?
no

Deprecations?
no

How to test?
Try to upload Avif or WebP image.

Hi, I have created this PR to add supports for Avif and WebP extensions in slider images.

friendly reminder @buggyzap


Add "Quickstart" instructions to README

In addition to Quickstart, may be useful to outline the project structure.
Happy to do this with a little help from you @aschlosberg .

@aschlosberg should Getting started section also feature npm package?
I would also add sections about how to use some of the libraries. I am having trouble with effectively integrating PRNG and NextShuffler. I can add examples and READMEs if you help me to figure it out, etc: #47

@aschlosberg
Is this the correct usage of PRNG.Source?

contract NextShufflerLazyInit is NextShuffler {
    using PRNG for PRNG.Source;

    uint256[2] private _nextShufflerSourceStore;

    bool private _isRandomnessSourceSet;

    constructor () NextShuffler(1337) {}

    function _setRandomnessSource(bytes32 seed) internal {
        require(
            !isRandomnessSourceSet(),
            "Can't change source after seed has been set"
        );

        PRNG.Source src = PRNG.newSource(seed);

        src.store(_nextShufflerSourceStore);

        _isRandomnessSourceSet = true;
    }

    function _load() internal view returns (PRNG.Source) {
        return PRNG.loadSource(_nextShufflerSourceStore);
    }

    function _store(PRNG.Source _src) internal {
        _src.store(_nextShufflerSourceStore);
    }
}

and then:
contract MetaverseBaseNFT is ERC1155, NextShufflerLazyInit {

    function mint() {
        // ...

        PRNG.Source rndSource = _load();

        for (uint256 i = 0; i < amount; i++) {
            tokenOffset = _next(rndSource);

            // token id is fetched from the random offset
            tokenId = _tokenOffset2TokenId(tokenOffset);

            _mint(to, tokenId, 1, "");
        }

        _store(rndSource);

       // ...
    }
}

https://github.com/buildship-dev/nft-contracts/blob/6a133c70e17297643da213f5bbf0e954aa224d86/contracts/MetaverseBaseNFT_ERC1155.sol#L449

Sorry @caffeinum, I don't know how I missed these. I'm sure it's too late, but in general yes that's how it works although I still recommend not storing the value (I'm considering deprecating that anyway).

@aschlosberg it was late, yeah, we've released that already! https://etherscan.io/address/0xde95471123ce8bd81ad8e7ba553e019da110b654#code
it worked out nice, because of the low gas it was one of the only moments in history where you could have had random ERC1155 mint! But we still got a pushback from a community because the gas cost to mint was 500k+ per token
I needed to store the value to make sure I predictably generate all of IDs, without leaving any of them out

I've set up phone notifications so I shouldn't miss tags again!
I'm glad it worked out well! We've also had issues with the high gas and @cxkoda has since implemented a MWC equivalent that's good for "fun" things that don't require a CSPRNG.


Cannot submit

ADD ISSUE DESCRIPTION HERE
Version: 1.91.0
Commit: ea1445cc7016315d0f5728f8e8b12a45dc0a7286
User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Embedder: codespaces

/duplicate of #220079

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Fix 500 when getting user as unauthenticated user

When doing GET /api/v1/users/{user} as an unauthenticated user,
gitea throws a 500 because it's trying to dereference elements
from the context user. It wants to do this to see whether to
show the primary email and will do that if the logged in user
is admin or the user in question. However, if ctx.User is nil,
go gets really unhappy.

make L-G-T-M work


Incorrect suggestion to remove RecursiveDo

Currently the check to see if RecursiveDo can be removed is if the file contains the mdo keyword. This isn't sufficient as the extension also introduces the rec keyword which can be used inside normal do blocks.

Fixed and released as 1.9.19 - thanks for the report!


[WIP] BPF L4 policy cleanups

test-me-please

test-me-please

test-me-please

@tgraf At this point it's only what's required for the transition.
I realised though that one minor tweak on top may fix #2907: With the current state, l4policy via the embedded maps is only triggered if there's an l4-only entry in the l3/l4 policy map. However, today userspace won't generate such entries. I was thinking that some reworking of the userspace would be necessary to fix the above bug, but we could just unconditionally attempt to use the embedded map if all the other cases fall through. I'll look into that, since it's one of the same core problems blocking #2954.

Confirmed that userspace changes are necessary to fix #2907 without otherwise breaking Label-dependent L4, so I think that we should go ahead with this PR as-is.


Extension convenience

When registering workflows, there is a drop-down for wdl and cwl.
Automatically change the extension on the contents when toggling.

Related to #221 , should probably do the two together


challenge_problem is hard coded as YEAST_STATES

In GitLab by @gzheng on Nov 4, 2019, 20:33
The challenge_problem in the structured request generated by the IP is currently hard coded as YEAST_STATES.  Instead, the IP should derive this from the title of the experiment request.

In GitLab by @mweston on Nov 4, 2019, 22:37
@gzheng I don't think this is correct as stated - YEAST_STATES is the correct challenge problem (see: https://schema.catalog.sd2e.org/schemas/challenge_problem_id.json)
This is derived from the parent folder the experiment request lives in (in this case, YeastStates). Deriving from the title is would be risky and inconsistent.

In GitLab by @gzheng on Nov 5, 2019, 07:38
Ok, I see the parent folder. I will close the ticket and ask Dan to change his in Xplan request.

In GitLab by @gzheng on Nov 5, 2019, 07:38
closed


ERROR Invariant Violation: Tried to register two views with the same name RNCSafeAreaProvider, js engine: hermes

ERROR Invariant Violation: Tried to register two views with the same name RNSVGRect, js engine: hermes
Android Bundled 8978ms (C:\Users\Mello\Documents\Builds\app-noah\apps\app-merchant\index.js)
LOG  undefined undefined
ERROR  Invariant Violation: Tried to register two views with the same name RNCSafeAreaProvider, js engine: hermes
ERROR  Invariant Violation: "main" has not been registered. This can happen if:

Metro (the local dev server) is run from the wrong folder. Check if Metro is running, stop it and restart it in the current project.
A module failed to load due to an error and AppRegistry.registerComponent wasn't called., js engine: hermes

I can't understand the reason for the error above, my package settings related to svg are below:
"react-native": "0.73.6",
"react-native-screens": "3.29.0",
"react-native-safe-area-context": "^4.9.0",
"react-native-svg": "^13.14.0"
The error occurs when I open the expo go application (I have an svg on the welcome screen:
import BackgroundMerchant from "../../assets/img-entrar-merchant.svg";

export function BackgroundWelcomeMerchant() {
  return <BackgroundMerchant width={"100%"} height={"100%"} />;
}

react-app-env.d.ts:
declare module "*.svg" {
  import { SvgProps } from "react-native-svg";
  const content: React.FunctionComponent<SvgProps>;
  export default content;
}
declare module "react-native-url-polyfill/auto";

declare module "blob-polyfill --save";
declare module "react-native-dialog";
declare module "socket.io-client";
declare module "*.png";
// declare module "*.svg";
declare module "*.jpeg";
declare module "*.jpg";
declare module "*.ttf";

my metro.config.js
const { getDefaultConfig } = require("expo/metro-config");

module.exports = (() => {
  const config = getDefaultConfig(__dirname);

  const { transformer, resolver } = config;

  config.transformer = {
    ...transformer,
    babelTransformerPath: require.resolve("react-native-svg-transformer"),
  };
  config.resolver = {
    ...resolver,
    assetExts: resolver.assetExts.filter((ext) => ext !== "svg"),
    sourceExts: [...resolver.sourceExts, "svg"],
  };

  return config;
})();

my babel.config.js:
module.exports = function (api) {
  api.cache(true);
  return {
    presets: ["babel-preset-expo"],
    plugins: [
      ["module:react-native-dotenv"],
      ["react-native-reanimated/plugin"],
    ],
  };
};

We're getting this same error on the latest Expo, but our issue gets fixed by downgrading from react-native-svg 15.2.0 (which is being recommended by expo) to 14.1.0.
Our env is as follows:
  expo-env-info 1.2.0 environment info:
    System:
      OS: Windows 11 10.0.22631
    Binaries:
      Node: 20.13.1 - C:\Program Files\nodejs\node.EXE
      npm: 10.8.1 - C:\Program Files\nodejs\npm.CMD
    IDEs:
      Android Studio: AI-222.4459.24.2221.9971841
    npmPackages:
      expo: ~51.0.14 => 51.0.14
      react: 18.2.0 => 18.2.0
      react-dom: 18.2.0 => 18.2.0
      react-native: 0.74.2 => 0.74.2
      react-native-web: ~0.19.11 => 0.19.12
      react-native-svg: 15.2.0 => 15.2.0to cause the issue)
    Expo Workflow: managed

react-native-svg: 15.2.0 => 15.2.0
This version causes the issue. Downgrading to 14.1.0 fixes the issue. 14.2.0 also appears to cause the issue.
We've also tried deleting node_modules and package-lock.json and reinstalling our dependencies, but we still got the ERROR Invariant Violation: Tried to register two views with the same name RNSVGRect, js engine: hermes error
Hopefully this get's fixed soon because this is still the version expo recommends.

Hello @E-Mello @JPStrydom,
Could you share a simple example of how we can reproduce that issue?

I don't know how to explain why, i simply stopped using svg and put png and the error stopped =/. I believe the problem is with the babel configuration and the use of svg, but, i didn't find anything specifc on the internet , or maybe i didn't know how to look for it.

@E-Mello,
Alright, we have closed that issue. If you have any further questions, please feel free to open a new issue.
Thank you.

I have the same problem using Expo 52. Downgrading to 14.1.0 did not solve the problem, any advice? ✋

Hi @NoyHanan, could you provide a minimal reproduction repository for the issue?


Simplify makefile: Get rid of "install" rules with pure make, recommend cmake and bazel instead.

First step towards simplifying our Makefile is to stop supporting "make install" (in favor or using cmake or bazel).
After that is done, Makefile can be further simplified because it will only be usable for our internal needs (building artifacts etc.).
So we will be able to remove support for different kinds of build modes (e.g. depedencies coming from submodule vs. pre-installed) because we will no longer have a need for them (the cmake build supports these modes properly).
Right now the way each of our dependencies is built via make is a bit different and this specialization makes the entire build system hard to digest.

CC @stanley-cheung  for the PHP changes.

Adhoc artifacts/packages/distribtests: https://sponge.corp.google.com/8dd19d7d-18b4-44f1-bd7a-9010d697a80f

still need to update https://github.com/grpc/grpc/blob/master/src/php/README.md

Known failures: https://github.com/grpc/grpc/issues/23466


Reimplement getImpulseFrame for examples

This is the function that tries to sync the timing of the main thread and the audio thread, so that when you press a key, the impulse is scheduled to play at an exact sample offset, which should be the time the key was pressed plus the duration of one mix buffer.
Right now I have all this code disabled so that all new sounds play at the beginning of the next mix buffer frame.

Moved to TODO file


Xzyaoi/feat/envs for runner

This change is

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: sfermi:x: Abner ZengAbner Zeng seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Austrian German + dative forms added

see https://en.wiktionary.org/wiki/Jänner and https://en.wiktionary.org/wiki/Feber

Please, see https://dateparser.readthedocs.io/en/latest/contributing.html#guidelines-for-editing-translation-data


Support for Balance Assertions

Currently, balance assertions are not supported by hledger-iadd.  Anytime an = sign is present, it gives the following error:
unexpected '='
expecting '+', '-', amount, or white space

This prevents assertions from being added via hledger-iadd, forcing manual addition in the journal after an entry is generated.

You're right, this is something that I've wanted for myself a few times now. I'll implement it when I find some time.

Ran into this a few more times, and am wondering if it's still of interest to you, @hpdeifel.

In theory yes. I just can't commit a lot of time to iadd currently, sorry


Refactor Named Resolvers

clean fix of https://github.com/morpheusgraphql/morpheus-graphql/pull/808

I think you mean #812 (root issue #810)


Support faster-whisper-large-v3

I want to know if faster-whisper-large-v3  is supported ? thanks.

Yes, it is supported. You should be able to load large-v3 once Faster-Whisper is updated to version that supports it.

感谢您的来信，我会尽快阅读的。
Thanks, I would read it as soon as possible~


Rollup count of events on contact errors out

I have a rollup setup to count the number of events related to a contact, the permissions and naming are all correct but it still errors out

@afawcett

So i just realized something, it works in classic but not lightning

So DLRS is having an issue due to some code in Apex class HFT package, class HexaFieldTrackerUtils method createFieldHistory. If you go an make the same edit to the parent record DLRS is attempting to update (per the rollup definition in play) in manually way i suspect you will see the same error. Can you try this? You can also ask the author of the HFT package (see Installed Packages under Setup) to comment on this.

Could this error be occurring because the API version of that HexaFieldTrackerUtils class is 39 and the latest version is 45?

@robertgalex thanks for sharing. Wondering how that class is related though?

@robertgalex let me know if you still want to discuss and we can re-open


unlimited clone depth for license plugin

@mathieucarbou let's not merge if most repos will need unlimited depth - I can just change the default

Tests seem to fail


Upgrade Rails to 4.2

Notes

This is yak-shaving so that we can use thoughtbot/administrate for #555: http://sethgodin.typepad.com/seths_blog/2005/03/dont_shave_that.html

Fix up miscellaneous issues in specs, some of them Rails upgrade-related

Tests pass, looks good on staging too!


Environment variable USERNAME not correctly filled

Description
When I start an executable the environment variable "USERNAME" of that executable shows "SYSTEM" instead of my actual username "z000swdz". This leads to other problems using this variable down the road.
Steps To Reproduce
Start any executable from within Files
Expected behavior
USERNAME should have my actual username "z000swdz"
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22000.856
Windows Version
Edition	Windows 11 Enterprise Version	21H2 Installiert am	09.08.2022 Betriebssystembuild	22000.856 Leistung	Windows Feature Experience Pack 1000.22000.856.0
Relevant Assets/Logs

@Josh-65 did you open a similar issue?

The ones I have are #5648 and #9488. I dont think there the same through.

It's the same as https://github.com/files-community/Files/issues/5648


Adding and updating values for chart indicators, on the fly.

Does this library allow for adding indicators on the fly and the ability to modify their values or remove them once they are added to a chart? Those are basic and necessary features in charting, yet they don't appear to be included in this library, and if they do where's the documentation?
For example, rather than having an ema50 hardcoded as an accessor, is it possible to simply call click add indicator, select and call the ema component, enter the window size in a modal, then click apply to have a visual of the ema with the set window size? I've spent the whole day trying to figure this out and it's driving me nuts that I can't find this feature. I'd rather not have to rebuild an indicator that already exists in the library.

@gldstrrbt i think the prupose of the chart library is to give all the tools and capability, but this type of behavior is part of the frontend implementation (with react is about to make reactive parts and control them how you want inside). Maybe i'm wrong.
Anyway, what you want to do can be accomplished if you change parts of chart component on the fly depending on state or props.

@gldstrrbt i think the prupose of the chart library is to give all the tools and capability, but this type of behavior is part of the frontend implementation (with react is about to make reactive parts and control them how you want inside). Maybe i'm wrong.
Anyway, what you want to do can be accomplished if you change parts of chart component on the fly depending on state or props.

The question was asked because one would think that basic charting capabilities, such as timeframe/interval options and the adding/editing/removing of included indicators, would be included in a charting library. If those capabilities were readily available and someone could've pointed them out, then I would've preferred saving the time spent these last couple days building out these components.

The question was asked because one would think that basic charting capabilities, such as timeframe/interval options and the adding/editing/removing of included indicators, would be included in a charting library. If those capabilities were readily available and someone could've pointed them out, then I would've preferred saving the time spent these last couple days building out these components.


iOS9 it become unstable move when drag

I tested it under iOS9 beta / iphone6 Plus,
When I drag one item, neighboring items will move weirdly when I move dragged item to other place.
It seems compute the wrong coordinate in iOS9.0 beta.

See my fork.

Nice work @chenkun24 !

+1, thank you @chenkun24 for fixing this!

@chenkun24 you're my hero!

@chenkun24 Thanks!

@chenkun24 Thank you!

@chenkun24 You Sir are a Legend! Thanks!


CTS WNS drops in timing repair before ending up roughly at where it started

Describe the bug
Here WNS gets worse in timing repair in CTS before bouncing back up to roughly where it started.

untar https://drive.google.com/file/d/1ICisYfJdyxyJbMKcgs6JkNTB2IT05afA/view?usp=sharing
./run-me-BoomTile-asap7-4.sh

OpenROAD v2.0-16787-gcd519bb5e
Features included (+) or not (-): +Charts +GPU +GUI +Python
This program is licensed under the BSD-3 license. See the LICENSE file for details.
Components of this program may be licensed under more restrictive licenses which must be honored.
[INFO ORD-0030] Using 44 thread(s).
clock_tree_synthesis -sink_clustering_enable -balance_levels
[INFO CTS-0050] Root buffer is BUFx24_ASAP7_75t_R.
[INFO CTS-0051] Sink buffer is BUFx24_ASAP7_75t_R.
[INFO CTS-0052] The following clock buffers will be used for CTS:
                    BUFx24_ASAP7_75t_R
[INFO CTS-0049] Characterization buffer is BUFx24_ASAP7_75t_R.
[INFO CTS-0007] Net "clock" found for clock "clock".
[INFO CTS-0011]  Clock net "clock" for macros has 123 sinks.
[INFO CTS-0011]  Clock net "clock_regs" for registers has 118471 sinks.
[INFO CTS-0008] TritonCTS found 2 clock nets.
[INFO CTS-0097] Characterization used 1 buffer(s) types.
[INFO CTS-0200] 40 placement blockages have been identified.
[INFO CTS-0027] Generating H-Tree topology for net clock.
[INFO CTS-0028]  Total number of sinks: 123.
[INFO CTS-0090]  Sinks will be clustered based on buffer max cap.
[INFO CTS-0030]  Number of static layers: 0.
[INFO CTS-0020]  Wire segment unit: 1350  dbu (1 um).
[INFO CTS-0023]  Original sink region: [(25141, 88188), (1974911, 1973484)].
[INFO CTS-0024]  Normalized sink region: [(18.623, 65.3244), (1462.9, 1461.84)].
[INFO CTS-0025]     Width:  1444.2741.
[INFO CTS-0026]     Height: 1396.5156.
 Level 1
    Direction: Horizontal
    Sinks per sub-region: 62
    Sub-region size: 722.1370 X 1396.5156
[INFO CTS-0034]     Segment length (rounded): 362.
 Level 2
    Direction: Vertical
    Sinks per sub-region: 31
    Sub-region size: 722.1370 X 698.2578
[INFO CTS-0034]     Segment length (rounded): 350.
 Level 3
    Direction: Horizontal
    Sinks per sub-region: 16
    Sub-region size: 361.0685 X 698.2578
[INFO CTS-0034]     Segment length (rounded): 180.
 Level 4
    Direction: Vertical
    Sinks per sub-region: 8
    Sub-region size: 361.0685 X 349.1289
[INFO CTS-0034]     Segment length (rounded): 174.
[INFO CTS-0032]  Stop criterion found. Max number of sinks is 15.
[INFO CTS-0035]  Number of sinks covered: 123.
[INFO CTS-0200] 40 placement blockages have been identified.
[INFO CTS-0027] Generating H-Tree topology for net clock_regs.
[INFO CTS-0028]  Total number of sinks: 118471.
[INFO CTS-0090]  Sinks will be clustered based on buffer max cap.
[INFO CTS-0030]  Number of static layers: 0.
[INFO CTS-0020]  Wire segment unit: 1350  dbu (1 um).
[INFO CTS-0206] Best clustering solution was found from clustering size of 30 and clustering diameter of 50.
[INFO CTS-0019]  Total number of sinks after clustering: 7317.
[INFO CTS-0024]  Normalized sink region: [(7.89957, 8.1), (1439.28, 1464.44)].
[INFO CTS-0025]     Width:  1431.3801.
[INFO CTS-0026]     Height: 1456.3355.
 Level 1
    Direction: Vertical
    Sinks per sub-region: 3659
    Sub-region size: 1431.3801 X 728.1678
[INFO CTS-0034]     Segment length (rounded): 364.
 Level 2
    Direction: Horizontal
    Sinks per sub-region: 1830
    Sub-region size: 715.6900 X 728.1678
[INFO CTS-0034]     Segment length (rounded): 358.
 Level 3
    Direction: Vertical
    Sinks per sub-region: 915
    Sub-region size: 715.6900 X 364.0839
[INFO CTS-0034]     Segment length (rounded): 182.
 Level 4
    Direction: Horizontal
    Sinks per sub-region: 458
    Sub-region size: 357.8450 X 364.0839
[INFO CTS-0034]     Segment length (rounded): 178.
 Level 5
    Direction: Vertical
    Sinks per sub-region: 229
    Sub-region size: 357.8450 X 182.0419
[INFO CTS-0034]     Segment length (rounded): 92.
 Level 6
    Direction: Horizontal
    Sinks per sub-region: 115
    Sub-region size: 178.9225 X 182.0419
[INFO CTS-0034]     Segment length (rounded): 90.
 Level 7
    Direction: Vertical
    Sinks per sub-region: 58
    Sub-region size: 178.9225 X 91.0210
[INFO CTS-0034]     Segment length (rounded): 46.
 Level 8
    Direction: Horizontal
    Sinks per sub-region: 29
    Sub-region size: 89.4613 X 91.0210
[INFO CTS-0034]     Segment length (rounded): 44.
 Level 9
    Direction: Vertical
    Sinks per sub-region: 15
    Sub-region size: 89.4613 X 45.5105
[INFO CTS-0034]     Segment length (rounded): 22.
 Level 10
    Direction: Horizontal
    Sinks per sub-region: 8
    Sub-region size: 44.7306 X 45.5105
[INFO CTS-0034]     Segment length (rounded): 22.
[INFO CTS-0032]  Stop criterion found. Max number of sinks is 15.
[INFO CTS-0035]  Number of sinks covered: 7317.
[INFO CTS-0018]     Created 195 clock buffers.
[INFO CTS-0012]     Minimum number of buffers in the clock path: 34.
[INFO CTS-0013]     Maximum number of buffers in the clock path: 34.
[INFO CTS-0015]     Created 195 clock nets.
[INFO CTS-0016]     Fanout distribution for the current clock = 4:1, 5:2, 6:4, 7:3, 8:1, 9:1, 10:2, 11:1, 16:1..
[INFO CTS-0017]     Max level of the clock tree: 4.
[INFO CTS-0018]     Created 9960 clock buffers.
[INFO CTS-0012]     Minimum number of buffers in the clock path: 46.
[INFO CTS-0013]     Maximum number of buffers in the clock path: 47.
[INFO CTS-0015]     Created 9960 clock nets.
[INFO CTS-0016]     Fanout distribution for the current clock = 1:72, 2:114, 3:112, 4:132, 5:126, 6:166, 7:168, 8:184, 9:175, 10:145, 11:181, 12:254, 13:290, 14:447, 15:680, 16:925, 17:1061, 18:1125, 19:930, 20:573, 21:279, 22:117, 23:41, 24:9, 25:5, 26:1, 28:2, 30:1..
[INFO CTS-0017]     Max level of the clock tree: 10.

[INFO RSZ-0047] Found 148 long wires.
[INFO RSZ-0048] Inserted 533 buffers in 148 nets.

[INFO RSZ-0099] Repairing 1 out of 75145 (0.00%) violating endpoints...
   Iter   | Removed | Resized | Inserted | Cloned |  Pin  |    WNS   |   TNS      |  Viol  | Worst
          | Buffers |  Gates  | Buffers  |  Gates | Swaps |          |            | Endpts | Endpt
---------------------------------------------------------------------------------------------------
        0 |       0 |       0 |        0 |      0 |     0 | -3489.281 | -107827944.0 |  75145 | frontend.bpd.banked_predictors_0.btb.ebtb_ext/R0_addr[1]
       10 |       5 |       0 |        5 |      0 |     2 | -4192.852 | -107095248.0 |  75145 | frontend.bpd.banked_predictors_0.btb.ebtb_ext/R0_addr[1]
       20 |       7 |       3 |        7 |      0 |     6 | -4865.464 | -109202720.0 |  75145 | frontend.bpd.banked_predictors_0.btb.ebtb_ext/R0_addr[0]
       30 |       8 |       6 |       10 |      0 |    11 | -4906.467 | -109337392.0 |  75145 | frontend.bpd.banked_predictors_0.btb.ebtb_ext/R0_addr[1]
       40 |      15 |       8 |       10 |      0 |    12 | -6793.429 | -114285952.0 |  75145 | frontend.bpd.banked_predictors_0.btb.ebtb_ext/R0_addr[1]
       50 |      17 |       9 |       12 |      0 |    18 | -6925.218 | -114462032.0 |  75145 | frontend.bpd.banked_predictors_0.btb.ebtb_ext/R0_addr[1]
       56 |       1 |       5 |       10 |      0 |     2 | -3282.132 | -104430144.0 |  75145 | frontend.bpd.banked_predictors_0.btb.ebtb_ext/R0_addr[1]
    final |       1 |       5 |       10 |      0 |     2 | -3282.132 | -104430144.0 |  75145 | frontend.bpd.banked_predictors_0.btb.ebtb_ext/R0_addr[1]
---------------------------------------------------------------------------------------------------
[INFO RSZ-0059] Removed 1 buffers.
[INFO RSZ-0040] Inserted 3 buffers.
[INFO RSZ-0041] Resized 5 instances.
[INFO RSZ-0043] Swapped pins on 2 instances.
[WARNING RSZ-0062] Unable to repair all setup violations.
[INFO RSZ-0046] Found 927 endpoints with hold violations.
Iteration | Resized | Buffers | Cloned Gates |   WNS   |   TNS   | Endpoint
---------------------------------------------------------------------------
        0 |       0 |       0 |            0 | -148.180 | -347026.844 | core.fp_rename_stage.freelist.br_alloc_lists_0\[4\]$_DFF_P_/D
       10 |       0 |    9619 |            0 | -325.298 | -253532.938 | core.iregfile.regfile_ext/W5_data[46]
       20 |       0 |    9666 |            0 | -325.298 | -253508.125 | core.iregfile.regfile_ext/W5_data[46]
       30 |       0 |    9689 |            0 | -325.298 | -253493.969 | core.iregfile.regfile_ext/W5_data[46]
    final |       0 |    9697 |            0 | -325.298 | -253489.422 | core.iregfile.regfile_ext/W5_data[46]
---------------------------------------------------------------------------
[WARNING RSZ-0064] Unable to repair all hold checks within margin.
[INFO RSZ-0032] Inserted 9697 hold buffers.
 Peak memory: 41517024KB.

Expected Behavior
I don't know what the expected behavior is here, but I tought it looked interesting enough to file a standalone test-case.
Environment
OpenROAD v2.0-16787-gcd519bb5e

To Reproduce
See above
Relevant log output
No response
Screenshots
No response
Additional Context
No response

We do allow some degradation in order to avoid local minima but this seems a bit extreme.

Can you clarify the issue a bit?  Are you concerned about setup WNS degradation during hold fixing?  Or is this about setup WNS QoR trajectory from floorplanning to CTS?

I dont have any concerns or expectations as I dont know the repair setup algorithm, I just thought the WNS dip during setup fixing might be an interesting test case that merited some further study: do we understand what is happening here?

OK.  Understood.   We'll look into WNS fluctuation during setup fixing.

I'm getting an incompatible db schema error after re-building OR with the latest version:
Error: cts.tcl, 4 incompatible database schema revision 0.96 > 0.90

That would suggest you have an older version rather than a newer one.  What commit id does OR show?

OpenROAD v2.0-15833-ge3b4fc0c9

Sorry, I had an issue with git pull which caused the db schema error.  Now I can reproduce the issue.

Here is another plot from various runs, it looks like the test case is indeed a bit peculiar and merits some further study:


Runv subcommand

as runc's subcommand

why introduce the subcommands ? is there other subcommands except start?

Other subcommands will be added.
The goal is to make runv more compatible with the runc as we can.
The user don't need to learn to much when he/her knows runc.

compatible with runc is good, with this patch, can runv successfully run without start subcommand?

Yes, it can, the same as runc.
The older command line still work

had better to add some commit logs to show how to use runv.

runv --help will shows how to use runv

updated, merge it


Plating blocks don't get rendered

Issue description
Industrial plating blocks and industrial glass don't get rendered (i am using sodium and other mods)

Steps to reproduce

place mentioned before blocks, 2. nothing rendering

Minecraft version
1.21 (Latest)
Fabric loader version
0.16.0
Fabric API version
0.100.8+1.21
Oritech version
0.80
Other relevant versions
No response
If a (crash)log is relevant for this issue, link it here: (It's almost always relevant)
No response

Please check the list of required mods, you're missing athena

i did have athena
but alright

Ah sorry for closing this too early. Athena also need indium if you have sodium installed, maybe that's the issue here?

I'll close this issue for now, as based on the sodium mention my last comment was most liklely the issue. Feel free to reopen it if you're still running into this issue


TypeError: chunk_batch() argument after ** must be a mapping, not NoneType

Hi, I have tried to train dreamfusion-if but the following error was encountered.

I think it's mostly because you have an empty geometry, so in validation, you get None from the geometry module. You can tweak your prompt or try with different loss lambdas, especially
    lambda_orient: [0, 10., 1000., 5000]
    lambda_sparsity: 1.
    lambda_opaque: 0.0

Do you run it with your own prompt?
I think it's mostly because you have an empty geometry, so in validation, you get None from the geometry module. You can tweak your prompt or try with different loss lambdas, especially
    lambda_orient: [0, 10., 1000., 5000]
    lambda_sparsity: 1.
    lambda_opaque: 0.0

Thanks, I test with the example prompt in the repo (i.e., bunny pancakes) and I will try as you suggest.


List tables

Tables whose cells contain block-level content (multiple paragraphs, lists, code blocks) can't be represented as pipe tables.  For these cases we might want to provide "list tables" as in RST. These could be rerpresented as a list in a div with attributes.
::: table aligns="lc" widths="25 50"
- * one
  * two
- -----
- * three
  * ^
- * five

    multi-paragraph
  * ~

Any content below the list is the caption.
~ means: merge with the cell above.
^ means; merge with cell to left.
:::

This list-like syntax seems like it would be very messy if a cell were to contain a list.
If you can specify the dimensions of the table at the top (number of rows and cols), then it's predetermined how many cells you have, and you can just specify that many items for your list. How about:
::: table rows=3 cols=2 headings=true
Flavor
----------
Description
----------
lemon
----------
Sour but good. You can use this flavor to make:

 * lemon meringue
 * lemon icing
----------
cherry
----------
Tastes like real cherries! I'd suggest using
this one to make

 * cherry ice cream
 * cherry soda
----------
Blueberry
----------
Tastes pretty much like you'd expect.
Use this flavor for:

 * blueberry pie
 * blueberry gum
:::

@jgm I don't like the "table" keyword since it's natural language specific. I'd rather use a specific delimiter like ===.
Also alignments could be shown symbolically with punctuation
=== <|> 50 25 25

Where aligns are symbolized with

< left
| center
> right

My Pandoc filter repeats the rightmost explicit alignment when there are more columns than explicit alignments. That's handy e.g. when you want the leftmost column to be left aligned and the rest to be right aligned.
Are widths supposed to be percentages of the total available width? I think that would be the easiest conceptually and notationally
Perhaps also a # prefix to indicate a <th> cell, e.g. for a stub.
-   # Header cell

BTW my filter accepts numbered lists and produces them by default, giving the header row if any index 0. That is very useful to keep track on where you are in a large table.
@uvtc I have used nested lists for and in tables for a long time with my filter. They are not confusing if you indent properly and use a variety of list markers. Also you can wrap the cell contents in a div to disambiguate further. I think complicating the syntax would be really confusing.
===
1.  1.  Foo
    2.  :::list-in-table
        -   Bar
        -   Baz
        -   Quux
        :::
===

I think === looks good as a delimiter for a list table. And the parallel lines kind of reminds me of a table.

They are not confusing if you indent properly and use a variety of list markers.

Yeah. I see what you mean.
Incidentally, I don't like calling these "list tables", since the list item markers aren't exactly working like regular list markers (e.g. they have two in a row). (Maybe "list-like tables"...)
Would this syntax break design goal num 7 (friendly to hard-wrapping)?

Do we need nested lists here? We already make blank lines between list elements meaningful, so we can, asciidoctor-style, use blanks to delimit the rows?
::: table aligns="lc" widths="25 50"
- one
- two

- three
- ^

- five

  multi-paragraph
- ~

Any content below the list is the caption.
~ means: merge with the cell above.
^ means; merge with cell to left.
:::

if we need to delimit the header-row, we can re-use thematic break

What about cells with more than one paragraph, or with content with something else than a paragraph? IMNSHO the whole point of list tables is to allow that, but it won't work if you use blanks to delimit rows.

I think it would?
- cell with complex content

  more complex content

  - this is still
  - part of this cell
  - due to indentation

- a new cell, and a new row, due to a preceding blank

  more content
- a new cell in _this_ row, because there's no blank

That is, blanks between the cells seems unambiguous with blanks within the cells

This is pretty similar to wikipedia table syntax.
{|
!align="center" width="15%"| Centered Header
!width="13%"| Left Aligned
!align="right" width="16%"| Right Aligned
!width="35%"| Default aligned
|-
|align="center"| First
| row
|align="right"| 12.0
| Example of a row that spans multiple lines.
|-
|align="center"| Second
| row
|align="right"| 5.0
| Here’s another one. Note the blank line between rows.
|}

Inspired by https://github.com/jgm/djot/issues/128#issuecomment-1344556958, what if we do the following for all tables:

a table is a sequence of  blocks
blocks fill the table left-to-right, to-down
the | is a "block-break" character, which is parsed in inline context, but splits a given block into two. Leading and trailing | are ignored.
| softbreak | or hardbreak | terminates the row
the --- block is used to delimit the header
the ~~~ is used to delimit the table
cell/column alignment is determined "naturally": |left | center | right|

Examples
~~~~~~~~~
| 1 | 2 |
~~~~~~~~~

~~~~~~~~~~~~~~
| two | rows |
| 1   | 2    |
~~~~~~~~~~~~~~

Table with header.
I think we need a blank after `|`.
We don't want inline-parser to recursively call block parser,
so we can't see that `---` is a line break

~~~
|fruit   |  price| {% if there's a header, it sets cell aligmet %}

---
| apple  |     4 |
| banana |    10 |
~~~

Table with cell-per-line

~~~
|fruit |  price|

---
| apple
| 4

| banana
| 10

~~~

Complex table, unreadable, but possible

~~~
|code |  list|

---

```rust
fn main() {}
```

- one
- two
- three

|

```zig
fn main() void {}
```

i. A
ii. B
iii. C
~~~

This ... probably has all kinds of ambiguities, and requires a fair amount of "pattern matching" from the convertor to infer the structure from syntax, but on the first glance it seems like it is both convenient for short tables and reasonable for large tables.
I think the main thing we've lost is specifying aligment on per-row basis, but that doesnt' seem crucial.


chore(env): update default value for span's attributes/links/events count #1675

Updating default values to respect the spec: https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/sdk-environment-variables.md#span-collection-limits

I added breaking label, but i'm actually not sure. It obviously doesn't break any users but it might change the output of the telemetry for high-volume users

I added breaking label, but i'm actually not sure. It obviously doesn't break any users but it might change the output of the telemetry for high-volume users

never too safe :)

@vmarchaud Does it makes sense to add info about it to upgrade guidelines?

Does it makes sense to add info about it to upgrade guidelines?

I dont think since we didn't change the env vars, just their values. If users used them that don't impact them


[FEATURE] Add actions-cookie

added cookie icon

@TonyMarc thx for the pr, I've added two new icons for cookie and cookie-bite to match the current icon styling


VS Code: Error after creating blazor project: "The name 'routeData' does not exist in the current context."

REPRO CONFIGURATION: Win10 x64 ENU 20H2 + SDK 7.0.100-preview.2.22152.5
Latest SDK Version: 7.0.100-preview.2.22152.5
Latest Runtime Version: 7.0.0-preview.2.22128.3
INSTALL STEPS:

Clean machine: Win 10 x64 ENU 20h2
Install the sdk 7.0.100-preview.2.22152.5 from: https://aka.ms/dotnet/7.0.1xx-preview2/daily/dotnet-sdk-win-x64.exe
Install VS Code from: https://code.visualstudio.com/Download
Install C# Extensions version: v1.24.1.vsix from: https://github.com/OmniSharp/omnisharp-vscode/releases

Testcases Affected
SimpleBlazorRazorEditingWithVSCode
Platform

[x] Windows
[x] macOS
[x] Linux

Repro Steps

Open a normal Terminal Emulator
Create blazor project and open it using VS code

  md Blazor
  cd Blazor
  dotnet new blazorserver
  code .

Go to https://github.com/OmniSharp/omnisharp-vscode/releases > Download the csharp-1.24.1.vsix.
In VS Code and go to the Extensions Tab, select "Install from VSIX" > install the file you downloaded.
Navigation to the problems window will encounter a error: "The name 'routeData' does not exist in the current context."

Note:
1. Error will disappear if you open the App.razor file
2. This issue works fine on C# Extensions is v1.24.0.
Actual Result

Expected Result
The problems window does not have error.

This is still happening and occurs in visual studio 2022 as well

@using Microsoft.AspNetCore.Components.Routing
try including above solved mine

Still happening - is this error being ignored?

Balzor/Razor support is implemented in the https://github.com/dotnet/razor repository. Closing this issue as work will be tracked in https://github.com/dotnet/razor/issues/9020


chore(deps): update dependency webpack to v4.42.0

This PR contains the following updates:

Package
Type
Update
Change

webpack
devDependencies
minor
4.41.6 -> 4.42.0

Release Notes

webpack/webpack
v4.42.0
Compare Source
Bugfixes

Disable constant replacements in "use asm" scope
Update schema to disallow functions for output.hotUpdateChunkFilename as this doesn't work
Hoist exports in concatenated module to handle circular references with non-concatenated modules correctly
Flag all modules as used in DLLs to fix missing exports

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

PR has been edited
:construction_worker: This PR has received other commits, so Renovate will stop updating it to avoid conflicts or other problems. If you wish to abandon your changes and have Renovate start over you may click the "rebase" checkbox in the PR body/description.

:tada: This PR is included in version 3.1.0 :tada:
The release is available on:

v3.1.0
GitHub release

Your semantic-release bot :package::rocket:


Feature Request: Add env var options ARCHIVEBOX_USERNAME and ARCHIVEBOX_PASSWORD to initialize admin user on first run without prompting

I have succesfully deploied via docker using portainer a container on my Synology NAS.
I can access the web gui but i haven't found in the documentation the Admin Password to be used to acces.
How can I find the password or what ENV variable I must set up?
Thank.
Marco.

Run archivebox manage createsuperuser using the CLI.

I'm currently working on creating a yunohost package for archivebox, and this feature will also be needed for that.
Or some other way, which I can supply an initial admin and password for archivebox during setup (perhaps as CLI arguments?), without an interactive prompt with the user (since the app is installed via a GUI, without a user directly interacting with the terminal)

for now I was able to achieve what I needed using expect, but having CLI args for archivebox would probably be cleaner. e.g.
usr/bin/expect<<EOF
set timeout -1
spawn sh -c "cd $datadir && archivebox manage createsuperuser --username $admin --email $admin_mail"
match_max 100000
expect "*?assword: "
send -- "$password\r"
expect "*?assword (again): "
send -- "$password\r"
expect eof
EOF

Those are the default Django password validators kicking in (see settings.py) to try and keep people safe, I suppose we can disable them with another env var or specifically in the case that the pass is loaded via env vars.

This would be good for Ansible also.
I tried using the expect plugin (which has worked for me before when no other options were available), and it actually worked, but only during the first run of the playbook:
- name: Create Archivebox superuser account
  ansible.builtin.expect:
    command: archivebox manage createsuperuser
    timeout: 10
    responses:
      'Username': "{{ archivebox_admin.user }}"
      'Email address': "{{ archivebox_admin.email }}"
      'Password': "{{ archivebox_admin.pwd }}"
      'Password (again)': "{{ archivebox_admin.pwd }}"
  become: true
  become_user: "{{ ansible_env.USER }}"
  args:
    chdir: "{{ archivebox_pwd }}"

For some reason beyond my understanding, subsequent runs of this task would just freeze indefinitely (this is not a request for support, just noting the weird behaviour and moving on).
Anyway, so that's another reason why it would be great to be able to set the admin user non-interactively. The expect command can be rather fragile at times.

Note that you can also insert a USER row directly into the index.sqlite3 db as a temporary hacking (instead of using expect).

Thanks for the suggestion! If I get the time, I'll look into doing that.
PS. I just pushed the Ansible role I used to install ArchiveBox to Codeberg. The role is quite tailored to my needs, but the more Ansible roles, the merrier, right? :-)

For people here looking for a temporary solution, I posted instructions on how to insert a User row directly into the SQLite3 DB here: https://github.com/ArchiveBox/ArchiveBox/issues/928#issuecomment-1075758078.

Chiming in to say that I'd love this feature. The SQLite workaround is cool but I'd rather not mess with that in case the implementation eventually changes.
I'm deploying to https://fly.io/ and for most services I host there, I just need a single Dockerfile and a fly.toml without ever needing to start a remote shell to run a startup/config command.

This is done and merged into dev, thanks @benmuth!
I'll update the docs to reflect the new feature.  For anyone that wants to use this immediately you can run the dev branch like so: https://github.com/ArchiveBox/ArchiveBox#install-and-run-a-specific-github-branch


[BUG] $KWEEN on SOL not displaying prices

Describe the bug
The SPL token $KWEEN is not displaying its token price but value 0
Expected behavior
Display proper token price
Screenshots

Additional context
If possible we can verify more prices for smaller SPL tokens

Hello @realpaaao  A support ticket has been opened for you. Kindly Talk to Support on the Live support page so that your issue can be looked into. CLICK HERE


chore(master): release 2.8.3

:robot: I have created a release beep boop
2.8.3 (2024-10-20)
Bug Fixes

esm import error (97dbd78)

Performance Improvements

set context.retry to be deprecated (e8b76a4)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/keq-request/keq/releases/tag/v2.8.3 :sunflower:


Uninstallation Not Completed in Full

After realizing that there was not much functionality available (to include lack of ability to send text to speech notifications), I uninstalled the alexa_media integration from the HACS store, Integrations page. Now everytime I restart my Home Assistant, I get the following message:
"The following integrations and platforms could not be set up:
alexa_media
Please check your config."
There is nothing related to this integration visible in the config/ folder, configuration.yaml, or in the custom_components folder.
My entities are now also flooded with media players, since I use a Family Amazon account. All my family's Alexa devices are showing up on my Home Assistant.
Steps to reproduce the behavior:

Restart Home Assistant after uninstalling the Alexa addon from the HACS store.
Click on 'Notifications'
Error appears as:
"The following integrations and platforms could not be set up:

alexa_media
Please check your config."

Found the fix, I think. There was still the entire collection of Alexa devices in Configuration > Integrations > Alexa Media Player

I see this is closed, but might want to add it to the instructions or add an uninstallation script...

Thanks. People will either find this issue or you can update the wiki.


Update documentation for Volar > 2.0.0

v2: applied conventional commit

v3: rebased to current master.

@WhyNotHugo is the documentation still accurate? with<EMAIL_ADDRESS>


Remove 2.1 specific comment

Putting release notes in Javadoc goes out of date fast
@garydgregory

Codecov Report

Merging #1091 (0dee4cd) into master (aa74b73) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1091   +/-   ##
=========================================
  Coverage     92.13%   92.13%
  Complexity     7504     7504
=========================================
  Files           195      195
  Lines         15722    15722
  Branches       2897     2897
=========================================
  Hits          14486    14486
  Misses          665      665
  Partials        571      571

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Terraform: Error after plan if variables required

Extension: Terraform
When using the plan command and passing required variables through Additional command arguments the task will succeed, but the plan binary and JSON files are not created.
For example with:

Command: plan
Additional command arguments: -var key_vault_name=mykeyvault123

The plan will be displayed:
##[debug]exec tool: /opt/hostedtoolcache/terraform/0.12.8/x64/terraform
##[debug]arguments:
##[debug]   plan
##[debug]   -var
##[debug]   key_vault_name=votingapp-dev-50735032
[command]/opt/hostedtoolcache/terraform/0.12.8/x64/terraform plan -var key_vault_name=mykeyvault123
Refreshing Terraform state in-memory prior to plan...
The refreshed state will be used to calculate this plan, but will not be
persisted to local or remote state storage.
...
...

But the output of the binary plan file will fail:
##[debug]exec tool: /opt/hostedtoolcache/terraform/0.12.8/x64/terraform
##[debug]arguments:
##[debug]   plan
##[debug]   -out=/home/vsts/work/r1/a/plan-binary-970e7cc1-563e-47f7-956f-764aa7d6aa48.tfplan

Error: Required variable not assigned

The variable "key_vault_name" is required, so Terraform cannot proceed without a defined value for it.

The subsequent show command then fails too:
##[debug]exec tool: /opt/hostedtoolcache/terraform/0.12.8/x64/terraform
##[debug]arguments:
##[debug]   show
##[debug]   -json
##[debug]   /home/vsts/work/r1/a/plan-binary-970e7cc1-563e-47f7-956f-764aa7d6aa48.tfplan
Terraform couldn't read the given file as a state or plan file.
The errors while attempting to read the file as each format are shown below.

State read error: Error loading statefile: open /home/vsts/work/r1/a/plan-binary-970e7cc1-563e-47f7-956f-764aa7d6aa48.tfplan: no such file or directory

Plan read error: open /home/vsts/work/r1/a/plan-binary-970e7cc1-563e-47f7-956f-764aa7d6aa48.tfplan: no such file or directory

It appears the additional command arguments are not passed to the second call of the plan command.

Seeing this issue too.  Running the command on the build machine works fine, in my case..
C:\agent\_work\_tool\terraform\0.12.8\x64\terraform.exe plan -var-file=C:\agent\_work\_temp\devtest-secrets.tfvars -var-file=C:\agent\_work\r25\a\_Data-Devops\drop\terraform\develop\data.tfvars
This works fine, I get the planned output.  However, when running with the task..
It completes the plan, then proceeds to complain about all the variables, that it obviously CAN access, or it wouldn't have been able to do the plan...
 2019-09-17T16:49:46.3354183Z Plan: 18 to add, 0 to change, 0 to destroy. 2019-09-17T16:49:46.3354546Z  2019-09-17T16:49:46.3354888Z ------------------------------------------------------------------------ 2019-09-17T16:49:46.3355189Z  2019-09-17T16:49:46.3355426Z Note: You didn't specify an "-out" parameter to save this plan, so Terraform 2019-09-17T16:49:46.3355625Z can't guarantee that exactly these actions will be performed if 2019-09-17T16:49:46.3355831Z "terraform apply" is subsequently run. 2019-09-17T16:49:46.3355975Z  2019-09-17T16:49:46.3730627Z [command]C:\agent\_work\_tool\terraform\0.12.8\x64\terraform.exe version 2019-09-17T16:49:47.1312199Z Terraform v0.12.8 2019-09-17T16:49:47.1313143Z + provider.azurerm v1.28.0 2019-09-17T16:49:47.1313749Z + provider.random v2.2.0 2019-09-17T16:49:48.7593339Z  2019-09-17T16:49:48.7594419Z Error: Required variable not assigned 2019-09-17T16:49:48.7595294Z  2019-09-17T16:49:48.7595972Z The variable "arm_password" is required, so Terraform cannot proceed without a 2019-09-17T16:49:48.7596628Z defined value for it. 2019-09-17T16:49:48.7597222Z  2019-09-17T16:49:48.7597816Z  2019-09-17T16:49:48.7598892Z Error: Required variable not assigned 2019-09-17T16:49:48.7602309Z  2019-09-17T16:49:48.7603322Z The variable "arm_principal" is required, so Terraform cannot proceed without 2019-09-17T16:49:48.7604048Z a defined value for it. 2019-09-17T16:49:48.7604842Z  2019-09-17T16:49:48.7605753Z  2019-09-17T16:49:48.7606453Z Error: Required variable not assigned 2019-09-17T16:49:48.7607071Z

@brent-robinson
 you can easily get output in json file.
Can you please ping you skypeId to<EMAIL_ADDRESS>we will have a quick call to discuss this ?

@brent-robinson I am closing this issue. If you still face any issue please ping us on<EMAIL_ADDRESS>

I've just stopped using the Terraform command wrappers..  It works fine using PowerShell

Could this be re-opened please?   I am experiencing the same issue.
Here is a sample of my output:
2019-10-09T09:09:48.4672883Z Releasing state lock. This may take a few moments... 2019-10-09T09:09:48.7558936Z [command]C:\agent\_work\_tool\terraform\0.12.10\x64\terraform.exe version 2019-10-09T09:09:49.5374147Z Terraform v0.12.10 2019-10-09T09:09:49.5375147Z + provider.azurerm v1.35.0 2019-10-09T09:09:49.5375862Z + provider.random v2.2.1 2019-10-09T09:09:54.0367253Z  2019-10-09T09:09:54.0368069Z Error: Required variable not assigned 2019-10-09T09:09:54.0368424Z  2019-10-09T09:09:54.0368760Z The variable "**secret**" is required, so Terraform cannot proceed without a 2019-10-09T09:09:54.0369169Z defined value for it.
We do not have a value for the variable assigned in a .tf file and prefer to pass this over as a command line argument.  This method causes the plan and "validate and apply" phases to fail.
On that note, could we decouple "validate and apply" and instead have separate phases?

Please do decouple the validate and apply. That is not required. If we can get through an init and plan then we don't need to validate AND apply.
There is a gist which has the output from my job showing the hidden validate command ignoring the command line options I passed to it in the job.
This is a blocker for me.

Paging @liamfoneill who opened https://github.com/microsoft/azure-pipelines-extensions/issues/685 but never replied and had his issue closed.

@ausfestivus I have a workaround but haven't had time to investigate why it's needed. The following works in 'additional command arguements'. According to Terraform docs, the -var parameter should be followed by the expression in single quotations, but with this task only double quotes work
-no-color -var-file="dev.tfvars" -var "app_service_plan_sku=$(app_service_plan_sku)"

This is probably related to both #729 and #741.

Merged PR ..Will release updated version by this week

@Roshan-Kumar-Microsoft any updates on the release? How are we expected to see the new features in azure devops? Does it need to be reinstalled in the organization or the Task's version dropdown will automatically show next version.


Enforce contact list use

TODO - Finalise copy of error message

Reviewed and approved @alan - fine to deploy for those 10 prisons who are already using the contact list


Inject properties

If you're using Spring Framework you can inject a value to any of the components in this way:
@Value("${mongodb.url:<IP_ADDRESS>}")
private String mongodbUrl;

It would be great to have the similar property injection in Finatra.
class SomeController @Inject("prop_file#some.key}")(value: String)
extends Controller {...}

@Krasnyanskiy finatra already supports injection via Guice, so is this proposal to do it via spring instead?

Hmm... I've tried to inject the value via Guice:
class AssetsController @Inject()(@Named("foo") path: String) extends Controller {...}

In my application.conf file I have the next prop: foo = "Barmaley". But I got this error:
com.google.inject.ConfigurationException: Guice configuration errors:

1) No implementation for java.lang.String annotated with @com.google.inject.name.Named(value=foo) was bound.
  while locating java.lang.String annotated with @com.google.inject.name.Named(value=foo)
    for parameter 0 at test.finatra.playground.web.controller.AssetsController.<init>(AssetsController.scala:15)
  while locating test.finatra.playground.web.controller.AssetsController

1 error
	at com.google.inject.internal.InjectorImpl.getProvider(InjectorImpl.java:1004)
	at com.google.inject.internal.InjectorImpl.getInstance(InjectorImpl.java:1009)
	at com.twitter.inject.Injector.instance(Injector.scala:12)
	at com.twitter.finatra.http.routing.HttpRouter.add(HttpRouter.scala:89)

What did I miss?

Use @Flag("flagname")
See http://twitter.github.io/finatra/user-guide/getting-started/#flags
-Steve
On Saturday, December 19, 2015, Sasha<EMAIL_ADDRESS>wrote:

Hmm... I've tried to inject the value via Guice:
class AssetsController @Inject()(@Named("foo") path: String) extends Controller {...}
In my application.conf file I have the next prop: foo = "Barmaley". But I
got this error:
com.google.inject.ConfigurationException: Guice configuration errors:

No implementation for java.lang.String annotated with @com.google.inject.name.Named(value=foo) was bound.
while locating java.lang.String annotated with @com.google.inject.name.Named(value=foo)
for parameter 0 at test.finatra.playground.web.controller.AssetsController.(AssetsController.scala:15)
while locating test.finatra.playground.web.controller.AssetsController

1 error
at com.google.inject.internal.InjectorImpl.getProvider(InjectorImpl.java:1004)
at com.google.inject.internal.InjectorImpl.getInstance(InjectorImpl.java:1009)
at com.twitter.inject.Injector.instance(Injector.scala:12)
at com.twitter.finatra.http.routing.HttpRouter.add(HttpRouter.scala:89)
What did I miss?
—
Reply to this email directly or view it on GitHub
https://github.com/twitter/finatra/issues/287#issuecomment-166027788.

--
-Steve
Sent from Gmail Mobile

Still get this error:
com.google.inject.ConfigurationException: Guice configuration errors:

1) No implementation for java.lang.String annotated with @com.twitter.finatra.annotations.Flag(value=foo) was bound.

Hi Sasha,
The flag must be defined inside a TwitterModule to use the @Flag
functionality. We'll update the docs to make this more clear.
-Steve
On Sunday, December 20, 2015, Sasha<EMAIL_ADDRESS>wrote:

Still get this error:
com.google.inject.ConfigurationException: Guice configuration errors:

No implementation for java.lang.String annotated with @com.twitter.finatra.annotations.Flag(value=foo) was bound.

—
Reply to this email directly or view it on GitHub
https://github.com/twitter/finatra/issues/287#issuecomment-166165846.

--
-Steve
Sent from Gmail Mobile

Thanks.

@Krasnyanskiy have a look at the twitter-clone example, specifically the FirebaseModule to see how to handle flags.

I added a module:
object PropertyProviderModule extends TwitterModule {
  val _flag = flag("foo", "", "foo")
}

Then I added the module to
override val modules = Seq(PropertyProviderModule)

And now I can use it in my controller
class AssetsController @Inject()(
  @Flag("foo") path: String
) extends Controller {...}

But only if i specify a program argument
-foo "Test"

This is not exactly what I'm looking for. Originaly I was trying to get the propery value from application.conf.

Typesafe's config is not explicitly supported in finatra, AFAIK. However, there is a way to do this(there are a few conversations about this on the google group).

Add these two dependencies to your build:

"com.github.racc" % "typesafeconfig-guice" % "0.0.2",
"com.typesafe" % "config" % "1.3.0",

Copy this config to your project

import com.github.racc.tscg.TypesafeConfigModule
import com.twitter.inject.{Logging, TwitterModule}
import com.typesafe.config.ConfigFactory

object ConfigModule extends TwitterModule with Logging {
  override def configure() = {
    info("Loading Config file")
    val config = ConfigFactory.load()
    info(s"App Name from Config: ${config.getString("app.name")}")
    install(TypesafeConfigModule.fromConfig(config))
  }
}

Add the new module to your list of modules in the server.

override val modules = Seq(ConfigModule)

Pass in the config file using JVM arguments like this:

-Dconfig.file=path/to/config/application.conf

Profit!

Yahoo! It woks.
@fayimora Thanks for the hint :+1:

No worries @kramasamy , glad I could help!


Add clusterid description to README and also fix error strings.

Signed-off-by: Humble Chirammal<EMAIL_ADDRESS>

/assign @rootfs

@rootfs PTAL :)

@rootfs done.. PTAL :). Thanks !!!

/lgtm
cc @childsb for merge


Question about FiLM layer

As it's mentioned in the 3.2 DENOISING NETWORK PARAMETRIZATION and Appendix B, a FiLM layer is used to incorporate edge feature and global feature. Why do you use FiLM layer and What other layer did you try for incorporation?

Hello,
FiLM layers are a general way to aggregate information coming from different sources, and it has been used successfully in many settings. We didn't try anything else. There are so many ways to create architecture that manipulate both node level, edge level and graph level information that we had to make choices without trying different options.
Clement


exploration: experimentalist active learner

Description
Mikako's experimentalist – active learner – working code.
Type of change:

New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)

We'll leave this branch open, but the PR can be closed now.


Sampler velocity still seems not to work, even in current dev build

I have just rebuilt the DEV branch with all the latest commits and am still seeing this issue.
mySampler.triggerAttackRelease(frequency, duration, Tone.now(), velocity);//always plays max volume, even when velocity is 0 or very small.

Hmmm. did another rebuild and it DOES appear to work. Closing this issue.


activity indicator bugfixes

The activity indicator only worked on the first image when showing multiple images, because the activity indicator's frame was set incorrectly. This fixes that issue

@fresswolf thanks for your pr! latest master or 1.9.0 can use this bug fixes


getJtreg curl seems flaky

Describe the bug
In the nightlies, there is a stage in openjdk-tests/TKG/scripts/build_test.xml that gets jtreg from our dependencies builds.
However, every now and again, the curl transfer occasionally fails to pull from this link (see screenshot). Although I'm not sure if it's related or not, the upstream jtharness job, has been failing for a while.
To Reproduce
Steps to reproduce the behavior (or Grinder rerun link):

Execute a pipeline with tests enabled at https://ci.adoptopenjdk.net/job/build-scripts/
Look at the downstream test jobs
The error is intermittent in builds (see https://github.com/AdoptOpenJDK/openjdk-build/issues/1634 for reports of this occuring) across all platforms

Expected behavior
jtreg is pulled from the job
Screenshots
18:44:57  getJtreg:
18:44:57      [mkdir] Created dir: /home/jenkins/workspace/Test_openjdk8_hs_sanity.openjdk_ppc64le_linux/jvmtest/openjdk
18:44:57       [echo] Getting Jtreg from https://ci.adoptopenjdk.net/job/jtreg/lastSuccessfulBuild/artifact/jtreg-4.2.0-tip.tar.gz
18:45:14       [exec] curl: (18) transfer closed with 1648942 bytes remaining to read
18:45:14
18:45:14  BUILD FAILED

This should be in a retry loop for a few times

From https://curl.haxx.se/docs/manpage.html:
--retry <num>

If a transient error is returned when curl tries to perform a transfer, it will retry this number of times before giving up. Setting the number to 0 makes curl do no retries (which is the default). Transient error means either: a timeout, an FTP 4xx response code or an HTTP 408 or 5xx response code.

When curl is about to retry a transfer, it will first wait one second and then for all forthcoming retries it will double the waiting time until it reaches 10 minutes which then will be the delay between the rest of the retries. By using --retry-delay you disable this exponential backoff algorithm. See also --retry-max-time to limit the total time allowed for retries.

Since curl 7.66.0, curl will comply with the Retry-After: response header if one was present to know when to issue the next retry.

If this option is used several times, the last one will be used.

Added in 7.12.3.

@karianna Would a transfer closed error count as a transient error?

@M-Davies - yes I believe so


[RFC] Ha 0.99.3 integrations

This is an RFC PR to show the way I see adding new/restoring/updating old integrations.
The commits should have [integrations] tag, it would be good if base HA will have [base] or [core]
tags to distinguish.

I like the idea, i think it makes it easier.
One side note. I think we can stop doing the python3 packages with include files as python2 is soon eol anyway.

Well, I do support python2 drop whenever possible, but if I borrow an existing recipe and update it to HA needs then I prefer to keep the original implementation. For instance, if recipe comes from OE and is implemented with .inc then we should better keep it as is for easier sync in the future. All new components should be created without .inc.

Agree. But this leads to the question how we want to deal with packages from other layers. Do we want to update them in meta-pythonnfor example, or should we keep copies of the required versions in this layer?

I was first thinking that copying recipes from other meta layers is a bad idea (and it actually is).
But at the end of the end we might end up with a bunch of meta layers being included providing "existing recipes we do not want to have a copy of" (I expect conflicts so hello BBMASK hustle)
So, for example, we may have local copies of the updated recipes form meta-python and might want to upstream those. But, once those merged we might not be immediately able to use those as usually those go into master or release we are not yet ready or can use. E.g. we are at warrior, but new recipes in some other new release.
So, IMO:

Have copies and have those as close to their initial implementation as possible.
Try to upstream changes.

...And it is also possible that the maintainers of those repos we want to upstream to might not want to accept the changes for many reasons. Stability, testing etc.

As co-founder and co-maintainer of meta-python, I would strongly recommend upstreaming and not copying recipes from at least that layer. Pascal and I had a long discussion at ELCE about whether to put the meta-homeassistant recipes in meta-python, but decided  to keep the homeassistant-specific functionality in this layer.

Also, I have plans to create a meta-python2 layer for all the folks that insist they cannot change. meta-python itself will therefore become python3 only and we will merge the .inc files. They make devtool upgrade a mess to work with...and we only went down that path because of the need to support two python major versions. Until python4 comes out of course ;)

As co-founder and co-maintainer of meta-python, I would strongly recommend upstreaming and not copying recipes from at least that layer.
Agree, that is always the right goal to have everything upstream, so you have less work maintaining your meta layer

The questions I cannot answer here are:

For example, we need a specific version of a recipe - should we create a local copy then?
If a recipe we need is not yet available in warrior, but is available in master - what do we do?
If a recipe belongs to some other maintained meta layer, should we add that to the project dependencies and make sure we only use those recipes we need (BBMASK)?

@moto-timo As I don't have a lot of time anymore to maintain this I'm not opposed to bring this to meta-python if this is still desired?

@bachp May I ask you what was the reason for closing this PR?

@andr2000 It was automatically closed as it targeted a branch (bachp:ha_0.99.3) that no longer exists.
I can't change the target branch so I can't reopen it. Can you?

superseded by #36


Roll Flutter Engine from afa7ce19bca8 to fbd92055f3a6 (1 revision)

https://github.com/flutter/engine/compare/afa7ce19bca8...fbd92055f3a6
2024-06-25<EMAIL_ADDRESS>Roll Skia from 335200e57c26 to 3029f75db702 (2 revisions) (flutter/engine#53554)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jimgraham@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Prompt lost after answering to not refresh the context

I've got a url in the loaded context.
After plandex t and writing the new prompt, I get a question about whether I want to refresh the updated url context. If I answer n, the whole step is forgotten and I don't have access to the prompt anymore.
This was annoying to lose after I wrote a whole paragraph :(
(Slightly similar to https://github.com/plandex-ai/plandex/issues/109 but in this case I would expect the action to continue - just without the refresh.)

Thanks for reporting @viraptor. Aborting if the context isn't updated was done to avoid edge cases with files that could lead to tricky-to-resolve conflicts. The concern doesn't apply if only urls (or other non-file data) are in context though, so there could be different handling in that case.
As a small step, we could also output the prompt to the terminal in case of cancellation so it doesn't get lost. I definitely understand the frustration there.


KeyError: accept / accept-encoding in _get_blob

Traceback:
File "/env/local/lib/python2.7/site-packages/azure/storage/blob/baseblobservice.py", line 1932, in get_blob_to_stream
_context=operation_context)
File "/env/local/lib/python2.7/site-packages/azure/storage/blob/baseblobservice.py", line 1659, in _get_blob
operation_context=_context)
File "/env/local/lib/python2.7/site-packages/azure/storage/storageclient.py", line 279, in _perform_request
raise ex
Happens infrequently with an incomplete traceback.
pip freeze | grep azure
azure==2.0.0
azure-batch==3.0.0
azure-common==1.1.6
azure-datalake-store==0.0.12
azure-graphrbac==0.30.0
azure-keyvault==0.3.5
azure-mgmt==1.0.0
azure-mgmt-authorization==0.30.0
azure-mgmt-batch==4.0.0
azure-mgmt-cdn==0.30.3
azure-mgmt-cognitiveservices==1.0.0
azure-mgmt-compute==1.0.0
azure-mgmt-containerregistry==0.2.1
azure-mgmt-datalake-analytics==0.1.6
azure-mgmt-datalake-nspkg==2.0.0
azure-mgmt-datalake-store==0.1.6
azure-mgmt-devtestlabs==2.0.0
azure-mgmt-dns==1.0.1
azure-mgmt-documentdb==0.1.3
azure-mgmt-iothub==0.2.2
azure-mgmt-keyvault==0.31.0
azure-mgmt-logic==2.1.0
azure-mgmt-monitor==0.2.1
azure-mgmt-network==1.0.0
azure-mgmt-nspkg==2.0.0
azure-mgmt-rdbms==0.1.0
azure-mgmt-redis==4.1.0
azure-mgmt-resource==1.1.0
azure-mgmt-scheduler==1.1.2
azure-mgmt-sql==0.5.3
azure-mgmt-storage==1.0.0
azure-mgmt-trafficmanager==0.30.0
azure-mgmt-web==0.32.0
azure-nspkg==2.0.0
azure-servicebus==0.21.1
azure-servicefabric==5.6.130
azure-servicemanagement-legacy==0.20.6
azure-storage==0.34.3

Hi @dhollsten,
The storage client has its own Github repository and tracker here: https://github.com/Azure/azure-storage-python
Please open your issue there, since there is no dev from this client here.
Thanks!


Support fake clustering method

balanace, configure etc. don't work.
The table systems are also absent. We could generate fake data.

Also meta tables


[time-service] Add TimeService for mocking async time

Motivation
TL;DR: got fed up with flaky tests for Peer rpc timeouts... wrote this as an experiment to mock async time.
Testing time-related things in network is kind of a pain. We get around this problem for interval-like things using a ticker mpsc channel, but that's kind of a hack and doesn't cover sleeping, timeouts, or querying the clock.
This problem is not unique to network; two other attempts exist in secure/time and consensus. The first is not async-aware, but works great for normal threading-only code.
Compared to the consensus time service, this one returns concrete Futures, which integrate more cleanly with the async ecosystem. The consensus time service also mandates spawning every sleep as a task, which sucks when we're setting/registering/polling timers on every rpc and socket read/write. It's also more convenient for the socket rate limiting work we're doing, which needs manual AsyncRead/AsyncWrite poll implementations.
This implementation also comes as a convenient enum so we don't have to infect literally every struct in network/ with a T: TimeService trait bound. Though perf is not really a goal here, it should be noted that the interfaces here are zero cost when we're running without testing features.
With this mock time service, I think it's actually not too far of a step to mock out the whole runtime so we can start fuzzing the full execution trace deterministically.
As an example, here's a test using the MockTimeService:
#[test]
fn peer_recv_rpc_timeout() {
    let mut rt = Runtime::new().unwrap();
    let time = MockTimeService::new();
    let /* .. */ = build_test_peer(rt.handle().clone(), time.clone().into(), ConnectionOrigin::Inbound);
    let (mut client_sink, mut _client_stream) = build_network_sink_stream(&mut connection);

    let send_msg = NetworkMessage::RpcRequest(RpcRequest { /* .. */ });
    let recv_msg = PeerNotification::RecvRpc(InboundRpcRequest { /* .. */ });

    let test = async move {
        // Client sends the rpc request.
        client_sink.send(&send_msg).await.unwrap();

        // Server receives the rpc request from client.
        let received = peer_notifs_rx.next().await.unwrap();
        assert_eq!(received, recv_msg);

        // Pull out the request completion handle.
        let mut res_tx = match received {
            PeerNotification::RecvRpc(req) => req.res_tx,
            _ => panic!("Unexpected PeerNotification: {:?}", received),
        };

        // The rpc response channel should still be open since we haven't timed out yet.
        assert!(!res_tx.is_canceled());

        // Advance time to timeout the inbound rpc.
        time.advance(INBOUND_RPC_TIMEOUT_MS).await;

        // The rpc response channel should be canceled from the timeout.
        assert!(res_tx.is_canceled());
        res_tx.cancellation().await;

        // Client then closes connection.
        client_sink.close().await.unwrap();
    };
    rt.block_on(future::join(peer.start(), test));
}

One question before doing a more serious review -- is there anyway to make this crate supersede secure/time such that async and/or tokio is built on top?

My understanding of secure/time is that it assumes only one thread right? So in secure/time, when we call MockTimeService::sleep we can just increment the current time because our thread is the only one running and the only one that can observe the time change.
Here, we're managing time across many concurrent tasks. You could get the equivalent functionality like in secure/time with:
impl TimeServiceTrait {
    fn sleep_blocking(&self, duration: Duration) {
        let delay = self.sleep(duration);
        futures::executor::block_on(delay);
    }
}

Compared to secure/time, this would require some other thread calling time.advance. This is why consensus mock time service has an "auto advance with limit" option, but I haven't added that yet (this PR was already getting big : p). The auto advance just automatically completes and advances time on a sleep (while time is under some limit). That way, the above sleep_blocking would work exactly the same as secure/time does today.

Conceptually far better than what we're currently doing.
I will throw out there though, that what we'll want to eventually have is an api that models the behavior of a network of nodes, not just controls the clock....   ideally this clock would only be used to drive state changes in transaction (awaiting -> timed out).... which would have a better api represented as something like:
let cluster = cluster::Builder().hasSlowNode("x").build();
cluster.accept(message).timeOutOnNode("x").proceedOnNodes().assertQuorumContains(message).assertNode.....
But you get the idea...  we effective want a queryable monad as our api and when executing in process we should use direct state manipulations ignoring the clock, or only moving it forward to keep the transition logic happy....
Each step in the monad/builder above could be a separate struct with separate capabilities, but after describing the state you want the app to transition to you should end up back at the cluster type and be able to query it's state.... did the transaction reach quorum, did the lagging leader get evicted from leadership?  etc....
Hopefully most tests doent

I will throw out there though, that what we'll want to eventually have is an api that models the behavior of a network of nodes, not just controls the clock.... ideally this clock would only be used to drive state changes in transaction (awaiting -> timed out)....
But you get the idea... we effective want a queryable monad as our api and when executing in process we should use direct state manipulations ignoring the clock, or only moving it forward to keep the transition logic happy....
Each step in the monad/builder above could be a separate struct with separate capabilities, but after describing the state you want the app to transition to you should end up back at the cluster type and be able to query it's state.... did the transaction reach quorum, did the lagging leader get evicted from leadership? etc....
Hopefully most tests don't need to access the clock, aside from the transition applied by the monad auto updating the clock for them. But one step at a time.

Oh definitely, that would be super useful! A few points here:

In the short term, this time service is more about increasing test determinism. Along those lines, the next source of non-determinism is the tokio scheduler itself; mocking the scheduler would allow us to execute most of network's tests deterministically.
Of course, a time service is also useful for full cluster simulation. It would make sense to me that each virtual node gets its own mock runtime (including a mock time service) and the cluster harness drives each virtual node according to the test profile.
I think the big thing missing that's blocking your proposal is actually full message intermediation hooked into the network layer. Consensus has a NetworkPlayground which does this, but at a layer above network. In this case, the cluster harness would probably init each virtual node with a mock transport. The mock transport would intermediate new connections through the cluster harness, who can now control the full message flow and network topology. Some API validating constraints probably include: how to support twins (how do we id nodes in the harness)? do we support json-rpc? do we serialize/encrypt messages across the wire? can we ensure determinism? etc...

/land

:broken_heart: Test Failed - Build images and run cluster test

/land

:broken_heart: Test Failed - Build images and run cluster test


fix(tracing): allow 256 chars in path tag

Description:
Increases the max_path_length for the :path tag in tracing to 256. This returns the length to the capacity that was supported before #444 . The issue of truncating the path has been raised in a few places recently, most notably in: https://github.com/istio/istio/issues/12561.
I inquired as to the reasoning behind the switch to 128 from 256, but did not receive an answer. If there are valid reasons to keep 128, I would appreciate feedback there.
Risk Level: Low
Testing: Unit testing
Docs Changes: N/A
Release Notes: N/A
cc: @Stono @objectiser
Signed-off-by: Douglas Reid<EMAIL_ADDRESS>

No objections

As you are changing this code, could you change to constexpr? Theoretically it could enable optimization when accessing the value.

As you are changing this code, could you change to constexpr? Theoretically it could enable optimization when accessing the value.

I'm pretty sure the optimizer will do the same thing either way.


Let Nosetests be configured with other options

Fixes bug where kwargs weren't being passed to the parent Helper class.

Coverage remained the same at 90.5% when pulling 420dfd1df4eef05fde5382f2ab3c655739b5e2f3 on blaix:accept_nosetest_options into fa2cd3af6f7c9a3211d0630783e45e93c9c16253 on thomasw:master.

Yikes. Thanks.


Unhandled rejection TypeError: reg.regexp.exec is not a function (when receiving a message)

Unhandled rejection TypeError: reg.regexp.exec is not a function
    at /home/nick/Desktop/telegram-bot/node_modules/node-telegram-bot-api/src/telegram.js:90:33
    at Array.forEach (native)
    at TelegramBot._processUpdate (/home/nick/Desktop/telegram-bot/node_modules/node-telegram-bot-api/src/telegram.js:88:32)
    at null.<anonymous> (/home/nick/Desktop/telegram-bot/node_modules/node-telegram-bot-api/src/telegramPolling.js:36:14)
    at Array.forEach (native)
    at null.<anonymous> (/home/nick/Desktop/telegram-bot/node_modules/node-telegram-bot-api/src/telegramPolling.js:33:15)
    at tryCatcher (/home/nick/Desktop/telegram-bot/node_modules/bluebird/js/main/util.js:26:23)
    at Promise._settlePromiseFromHandler (/home/nick/Desktop/telegram-bot/node_modules/bluebird/js/main/promise.js:507:31)
    at Promise._settlePromiseAt (/home/nick/Desktop/telegram-bot/node_modules/bluebird/js/main/promise.js:581:18)
    at Promise._settlePromises (/home/nick/Desktop/telegram-bot/node_modules/bluebird/js/main/promise.js:697:14)
    at Async._drainQueue (/home/nick/Desktop/telegram-bot/node_modules/bluebird/js/main/async.js:123:16)
    at Async._drainQueues (/home/nick/Desktop/telegram-bot/node_modules/bluebird/js/main/async.js:133:10)
    at Immediate.Async.drainQueues [as _onImmediate] (/home/nick/Desktop/telegram-bot/node_modules/bluebird/js/main/async.js:15:14)
    at tryOnImmediate (timers.js:534:15)
    at processImmediate [as _immediateCallback] (timers.js:514:5)

reg.regexp is a regular expression provided by telegramBot.onText(regexp, callback). So it looks like you used onText somewhere without a regexp object.
There's also a debug statement right before reg.regexp.exec gets called, so starting node with the DEBUG=node-telegram-bot-api environment variable will give you some more information about whats going on.

Yep, this was my fault, my editor was at fault here.

what was the problem?

@Besatnias : The problem was, @ZudoMC  didn't use Regular Expression on onText..


cholesky not working with vmap on complex arrays

I tried to use cholesky on complex arrays with vmap but I get Unimplemented error.
A = np.eye(3).astype(np.complex64)
A_batched = np.stack([A, A])
jax.vmap(np.linalg.cholesky)(A_batched)

RuntimeError: Unimplemented: Complex types are not implemented in Cholesky; got shape c64[2,3,3]:

This isn't a problem if I don't use vmap.
np.linalg.cholesky(A)
DeviceArray([[1.+0.j, 0.+0.j, 0.+0.j],
             [0.+0.j, 1.+0.j, 0.+0.j],
             [0.+0.j, 0.+0.j, 1.+0.j]], dtype=complex64)

Is this a bug or am I doing anything wrong? This looks really strange.

Thanks for the question!
What backend are you using (CPU/GPU/TPU)? (I'm guessing CPU.)
It looks like you're hitting this unimplemented error in XLA.
I think the reason it's only coming up when you use vmap is that JAX falls back to an XLA-based implementation, rather than using a LAPACK one in the un-batched case.
@hawkinsp we could just write a loop in our lapack.pyx bindings to handle batch dimensions, right? I think that's how we handle batch dimensions in other kernels.
I'm going to mark this as a "good first issue" because there are plenty of examples in lapack.pyx and lax_linalg.py to pattern-match off of.

(Also I un-assigned myself because I'm not working on this right now.)

will attempt this!

Thanks for the quick responses! @mattjj @hawkinsp @am-khan Just tested on GPU and it is a CPU-only issue. Look forward to the fix.

@am-khan Oops! Sorry: I didn't see you wanted to work on this. Thanks for volunteering!
PR #1956 fixes this, and also improves the efficiency of batched Cholesky on GPU at the same time.

@am-khan while @hawkinsp got the jump on you for this issue, it looks like there's a very similar TODO for batched triangular solves.

PR #1956 fixes this but it requires a jaxlib rebuild (or waiting until we push new jaxlib wheels). Hope that helps!

thanks @mattjj is there a corresponding issue or can I just go ahead?

You can just go ahead, though if you feel the same amount of satisfaction from closing issues that I do, feel free to open one and mention that you're working on it :)