Installing Open Source Environment

I’m running into issues when trying to install Open Source Package Management tools on my Power 10.
I’ve been following this guide: Getting started with Open Source Package Management in IBM i ACS
I login to STRTCPSVR *SSHD, and then in ACS I go to “Open Source Package Management”.
Then I get this error message printed in ACS: Error: bsh: /QOpenSys/pkgs/bin/yum: not found

So then next, following this guide https://ibmi-oss-docs.readthedocs.io/en/latest/troubleshooting/SETTING_PATH.html I adjusted my PATH to the "Best" option /QOpenSys/pkgs/bin:/QOpenSys/usr/bin:/usr/ccs/bin:/QOpenSys/usr/bin/X11:/usr/sbin:.:/usr/bin and now I get this error

The MSGGEN003 error message was mentioned in the guide I’m following (linked above or here IBM i ACS Open Source Package Management Auth Fail Error). It says that the cause is that “The OpenSSH authentication mechanism on IBM i OS has extra environment items that can cause this issue.”
So now I’m on Step 2 of the Auth Fail Error guide:
Add the line,
"ibmpaseforienv PASE_USRGRP_LIMITED=N"
to the sshd_config file.
A default location for this file is "/QOpenSys/QIBM/UserData/SC1/OpenSSH/etc/sshd_config
So I logged into 5250 Emulator as QSECOFR and used the following command to add the line as instructed:
EDTF ‘/QOpenSys/QIBM/UserData/SC1/OpenSSH/etc/sshd_config’
Then I save and exit.
But I still get the same error message in ACS even after restarting the SSH server.

I'm on 7.5. After I removed ibmpaseforienv PASE_USRGRP_LIMITED=N from my sshd_config, waited for around 15 minutes and tried again I got the first error again, but now I'm getting the second error.

Yeah on 7.5, you definitely shouldn't need the PASE_USRGRP_LIMITED, since that's the default. I have no idea why adding it would have broken anything, though.
Additionally, the PATH adjustment shouldn't be needed for the ACS package manager, but I don't think that should break anything. If yum can't be found then that would indicate the bootstrap is not installed, but then it should detect this and offer to install it for you. For either issue, you're going to have to open a ticket with IBM service to investigate further, since the issue is with ACS.

I've been trying to install this on an IBM i OS partition on my Power 10 but I just had a thought. Do you know if it has to be a Linux partition?

@alanseiden Yes. Thank you.


Pointer access violation in remove_tree(Node *node)

Hi Blai,
When you call remove_tree(child) while iterating over a node's children, you (recursively) delete the child (line 371 in node.h) before accessing its sibling (end of for loop in line 369), implying a segfault.
You need to make two (obvious) small modifications to correct the bug. Basically, it consists in saving a child's sibling before deleting the child and actually moving to the sibling.
Since I have forked your project and substantially changed it, I prefer copying/pasting the correction here rather than doing a pull request with my fork (too far from your project now).
The first modification is in the function remove_tree which becomes:
inline void remove_tree(Node *node) {
    Node *sibling = nullptr;
    for( Node *child = node->first_child_; child != nullptr; child = sibling ) {
        sibling = child->sibling_;
        remove_tree(child);
    }
    delete node;
}

The second modification is in the advance(Action action) function where you need to replace lines 103-110 with the following lines:
        Node *selected = nullptr;
        Node *sibling = nullptr;
        for( Node *child = first_child_; child != nullptr; child = sibling ) {
            sibling = child->sibling_;
            if( child->action_ == action )
                selected = child;
            else
                remove_tree(child);
        }
        assert(selected != nullptr);

Hope it helps.
Cheers,
Florent

Hi Florent. Thanks for the update on this. Bug fixed following your advice!


Attack Docs Missing

http://cleverhans.readthedocs.io/en/latest/source/attacks.html
What happened? Please restore.

Any response?

@nottombrown Do you know what happened?

Odd. I'll take a look and see if I can track down the issue

Thanks for the PR @nottombrown
Do we have to trigger something to rebuild the docs? They don't seem to have refreshed yet despite the PR being merged

There was a second configuration issue with the hosted docs.
Resolved now and the docs will auto-rebuild whenever there is a push to master.
http://cleverhans.readthedocs.io/en/latest/source/attacks.html


fix: load lu content need add system local

Description
The defaultLanguage in luis setting is used as fallback local. And fallbacklocal is used to set default recognizer when we use multi language.
When we load the lu content, the local is used to check the file local.
lubuild will check the filename, if the filename is 'a.en-us.lu', the local will set to 'en-us'.
If the file name is 'a.lu', lubuild will use the passed local.
But if the file is 'a.en-us.lu', and the local is 'it-it', the function will throw an error.

Task Item
#minor
refs #3274

Screenshots

Coverage decreased (-0.001%) to 38.905% when pulling d65b738159a6aabaf24c087ed99c7f3b9cb54184 on lei9444:publishluis into 74299ee0138136d2b3c70c8d6333891302b5edb5 on microsoft:master.


Ipopov/weird cache bugfix

#490
#533

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: IgorPopov-MS sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Adding ARX model from pull #51 with fixed predict().

Grabbed the AutoregressionX and AutoregressionXSuite (tests) from pull request #51 (from @josepablocam) and fixed predict() function.

Thanks a ton for working on this @dmsuehir !  Adding some comments inline.  Let me know if you're interested in iterating on this, or if you'd prefer I take it from here.

@sryza - I can make the updates that you've noted.  Thanks!

Hey @dmsuehir, looks good, merging this in.  Thanks!

Thanks @sryza!  Do you know when there will be a new release that includes this ARX model?

@dmsuehir I am working on that issue , can't give exact estimate , but will try with @sryza to finalize first version in this month

@dmsuehir I'll probably push out a release in a month to give some space since the last one.  Do you need it sooner?
@mbaddar1 Dina contributed an ARX model in this pull request.  Feel free to submit or suggest any improvements that you think are necessary though.

@sryza  The sooner the better.  For now, my team uploaded a version of spark-timeseries with ARX to our maven repo, but after your release we'll delete that, and I'll switch over to using your release.
Another question - would you be able to update the version of Spark and breeze that are being used?  I'm a developer for https://github.com/trustedanalytics/atk and we're using Spark 1.5 and MLlib 1.5.0, which uses breeze 0.11.2.  When I started using spark-timeseries, I ran into an issue with a breeze API not being found in another part of our ATK code, because it was trying to use breeze 0.10 (since that's what spark-timeseries uses).  I was able to work around the issue using exclusions in our pom file, but it would be nice not to have to do that.

@sryza @dmsuehir Thanks for this update , will give a look at the changes and see if there is a room for improvements

@sryza @dmsuehir I think the current implementation have the following issue :
1-It uses OLS regression with a time series Y , and X. As mentioned in
https://onlinecourses.science.psu.edu/stat510/node/72
and
https://www.otexts.org/fpp/9/1
in time series problems , OLS regression with result residual have AR or MA structure. OLS regression assumes white noise residuals (~N(0,sigma^2) ) . So we need to use statistical test (Durbin watson for AR structure) to check if residuals are ~N(0,sigma^2). if test failed , we need to reiterate and update the estimate of regression coefficient using some "feedback" function that updates estimated regression coefficients.
2-Before doing regression we need to test stationarity of Y , else we need to difference all Y and regressors before performing regression
Brockwell approach (which is an extension to cochrane Orcutt) alleviates these problem (1) by working as follows :
Let's assume the following model for regression with ARMA errors
Y = BX+W -----> (1)
phi(B) W = theta(B)Z -------> (2)
Where
Y : is target time series
X : covariates
W : ARMA errors
Z : white noise error ~N(0, sigma^2)
phi(B) : auto regression polynomial
theta(B) : Moving average polynomial
i) Apply mathematical transformation on (will detail that on a blogpost as it has many mathematical notation) on (1) using the innovations algorithm to make sure residuals are white noise
ii) Apply OLS on Y and X and get B_OLS
iii) estimate ARMA (p,q)  model on residuals
iv) use estimated phi and theta from iii , to estimate B_GLS (GLS assumes some level of residual auto correlation) using the inverse of the formula used in i)
v)Re-estimate the residuals Y-B_GLS*X and go back to iii). Stop when B estimation converges or with max defined #iterations
I will work out some mathematical details and try to detail then in a blog post (Git comments will not help too much in that)
What do you think ? Any comments would be helpful before i start the implementation
Regards

I might be butting in here since I haven't worked on this in a long time,
so please feel free to dismiss. But I made some comments below that i
thought might be worth considering
On Sat, Mar 5, 2016 at 1:12 PM, Baddar , M<EMAIL_ADDRESS>wrote:

@sryza https://github.com/sryza @dmsuehir https://github.com/dmsuehir
I think the current implementation have the following issue :
1-It uses OLS regression with a time series Y , and X. As mentioned in
https://onlinecourses.science.psu.edu/stat510/node/72
and
https://www.otexts.org/fpp/9/1
in time series problems , OLS regression with result residual have AR or
MA structure. OLS regression assumes white noise residuals (~N(0,sigma^2) )
. So we need to use statistical test (Durbin watson for AR structure) to
check if residuals are ~N(0,sigma^2). if test failed , we need to reiterate
and update the estimate of regression coefficient using some "feedback"
function that updates estimated regression coefficients.

As far as my experience with other libraries has been, the onus of checking
residuals is on the user (and subsequently adjusting the autoregressive lag
as necessary).

2-Before doing regression we need to test stationarity of Y , else we need
to difference all Y and regressors before performing regression

I'm starting to see this as pretty close to ARIMAX. If the goal is simply
ARX, then seems like the onus of differencing should be on the user (by
either using a helper to do that, or simply using ARIMAX....when that is
implemented, if there are plans for that).

Brockwell approach (which is an extension to cochrane Orcutt) alleviates
these problem (1) by working as follows :
Let's assume the following model for regression with ARMA errors
Y = BX+W -----> (1)
phi(B) W = theta(B)Z -------> (2)
Where
Y : is target time series
X : covariates
W : ARMA errors
Z : white noise error ~N(0, sigma^2)
phi(B) : auto regression polynomial
theta(B) : Moving average polynomial
i) Apply mathematical transformation on (will detail that on a blogpost as
it has many mathematical notation) on (1) using the innovations algorithm
to make sure residuals are white noise
ii) Apply OLS on Y and X and get B_OLS
iii) estimate ARMA (p,q) model on residuals
iv) use estimated phi and theta from iii , to estimate B_GLS (GLS assumes
some level of residual auto correlation) using the inverse of the formula
used in i)
v)Re-estimate the residuals Y-B_GLS*X and go back to iii). Stop when B
estimation converges or with max defined #iterations
I will work out some mathematical details and try to detail then in a blog
post (Git comments will not help too much in that)
What do you think ? Any comments would be helpful before i start the
implementation
Regards
—
Reply to this email directly or view it on GitHub
https://github.com/sryza/spark-timeseries/pull/129#issuecomment-192700417
.

--
Jose Pablo Cambronero

@josepablocam I agree with your points. But if we are planning to make library to be used by IT developers with few statistical background , the stationarity and auto correlation test should be done automatically (as in R) to avoid wrong coefficient values

Fair enough. I think there should be an option to tune the threshold used
for some of the tests though. For example, there might be some
domains/applications where non-normality in errors can be tolerated and
certain tests might be too stringent. Here's a good stats stackexchange
thread
http://stats.stackexchange.com/questions/29731/regression-when-the-ols-residuals-are-not-normally-distributed
discussing some of this.
On Sun, Mar 6, 2016 at 7:54 AM, Baddar , M<EMAIL_ADDRESS>wrote:

@josepablocam https://github.com/josepablocam I agree with your points.
But if we are planning to make library to be used by IT developers with few
statistical background , the stationarity and auto correlation test should
be done automatically (as in R) to avoid wrong coefficient values
—
Reply to this email directly or view it on GitHub
https://github.com/sryza/spark-timeseries/pull/129#issuecomment-192884924
.

--
Jose Pablo Cambronero

@sryza I am using spark within 4 months, recently I have been used ARIMA, Holt Winter from spark-ts. But now i required ARIMAX implementation in java spark, as ARIMAX.scala is available in repository but ARIMAXModel is doesn't available under com.cloudera.sparkts.models._
Please let me know is there any alternative or any other option to implement ARIMAX?

Hi @111992-07, which jar are you using?  ARIMAX was added in September, so it's possible you're running off of an earlier version?  It should be included in the 0.4.1 version.

@sryza I have been used this dependency in pom

com.cloudera.sparkts
sparkts
0.4.1

but still in maven i didn't found ARIMAX and ARIMAXModel
others are available ARXModel and ARModel which i used in autoregression.
Now what can I do any suggestions?


Make Markdown read a bit more robust

Patch for the failing CI on master. Was introduced by #123.

Although this now runs successfully, master shouldn‘t have failed. I‘ll further look into this tomorrow.

Ok, for some reason tutorial 8 (multionomial-regression) failed on master and the stacktrace wasn't visible because one of the folder names didn't match the jmd name. That last issue is now fixed.
The failure of tutorial 8 is not fixed, because I cannot reproduce it. However, the next time that it happens, there will at least be a stacktrace thanks to this PR.


Write summary document

I like how this is presented: https://github.com/kubeflow/mpi-operator/blob/master/proposals/scalable-robust-operator.md

We will have a proper paper hopefully early to mid next year.


Add TypeScript type declarations file

This project is looking great and seems to be gaining a bit of attention already. I know some people who'd like to use this and would like TypeScript support. This PR adds a TypeScript declaration file and updates the package.json to point to the declaration file.
Note: When this project is publishe to NPM (see #14) and the library structure is changed, some adjustments to the declaration file might be neccessary.
I packed this project via npm pack and installed the tarball in a TypeScript project, it picks up the declarations correctly.

Also just for reference, this might be how this project would look "converted" (with types added inline) to TypeScript: https://github.com/blutorange/Coloris/blob/demo-ts-convert/src/coloris.ts

Hello @blutorange, thank you for your contribution, however I will not be adding TypeScript support to this tool.
I feel bad to reject your PR after all the work you have done, but I have a very strong opinion about this.
I don't like TypeScript, I don't plan to use it, and I think it's one of the worst things that happened to web development along with NPM. This is of course my opinion, but I will stand by it.
I would like this tool to stay as simple as possible, and to be written only in standard technologies that are natively supported by modern web browsers. I will eventually remove babel and the ES5 version and only provide the ES6 source and the minified file.
If you would like to maintain a fork of this project focusing on TS, I will do what I can to help.
P.S: I will fix the typos and mistakes you pointed out in the README file. Thank you for that :)

@melloware thank you for reviewing the changes :)

Ok based on your two posts you are ok if I fork it to push to npm and provide typescript?  As long as I have your permission I don't mind doing it 😁

Of course! Feel free to do so. I will link to your fork in the README file for those who are interested in NPM/TypeScript.

@mdbassit Thanks for your feedback, I respect your opinion. I understand not everybody likes TypeScript. It didn't take that long and it was a nice exercise, so feel free to close this PR if you want.
Two things I'd like to point out regarding TypeScript: (1) it is pretty widely adopted these days, unlike some other languages like Flow (even e.g. the Chrome Dev tools are written in TypeScript); (2) TypeScript nowadays has a strong policy about not adding any non-standard runtime features not supported by the EcmaScript standard; rather, it strives to be simply a type checker, i.e. writing function parameters und return types after a colon (:) directly in the code instead of in the doc comment.

@blutorange Thank you for the additional information. My point was that it doesn't run natively in the browser without transpiling  to JS.

@melloware let me know if you do it 😄 . I'll be glad to help 😉

OK thanks guys I will post back here when I have it stood up and on NPM.


[WIP] Direct headers announcement

This is a cherry-pick of https://github.com/bitcoin/bitcoin/pull/7129.
I just needed the updates to mininode testing support, but rebased the sendheaders support as well.  sendheaders.py does not pass, due to a run-time conflict with thin blocks.

Of interest https://github.com/bitcoin/bitcoin/pull/7919

Closing in favor of #159.


Setup downstream CI tests

It would be nice to setup CI to test all the downstream packages in the JuliaNonconvex org for new PRs.

Done.


post_hooks for release provider don't run.

In the rebar.config, pre_hooks for release work, but post_hooks don't run. There are no error messages.
rebar.config:
...
{pre_hooks, [{release, "touch /tmp/pre"}]}.
{post_hooks, [{release, "touch /tmp/post"}]}.
...

Then:
$ rebar3 release
===> Verifying dependencies...
...
===> release successfully created!
$ ls /tmp
pre
$

Edit:
Tried it on a rebar new release and it worked.
The presence of this plugin causes it to break, though:
{plugins, [
    { provider_asn1, ".*", {git, "git@github.com:knusbaum/provider_asn1.git", {tag, "0.1.1"}}}
]}.

I can't reproduce.

Oops, my bad, I was running pre hooks. I can reproduce with post hooks.

Argh, I see the issue. relx will init:stop(0). Will have to refactor relx so it has an entry that can be run in command_line mode (so the output looks right) but doesn't call init:stop/0


use matching backticks in docstrings.

We autogenerate a shortcut list in IPython docs using docstrings, the non-matching backticks make RST to complain.
We could fix it, but it seem cleaner to do it upstream :-)

Thank you for this! I'll merge it ASAP. It makes sense to fix it here. Don't hesitate to create pull requests for stuff like this.

Merged in https://github.com/jonathanslenders/python-prompt-toolkit/commit/021e6cc963570e1bc645defcdeb2da10d0c83a9e + also pushed to the 2.0 branch.
Thanks!


python311Packages.greeneye-monitor: 4.0.1 -> 5.0

Diff: https://github.com/jkeljo/greeneye-monitor/compare/refs/tags/v4.0.1...v5.0
Changelog: https://github.com/jkeljo/greeneye-monitor/blob/v5.0/CHANGELOG.rst
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] Fits CONTRIBUTING.md.

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

5 packages built:

home-assistant-component-tests.greeneye_monitor
python310Packages.greeneye-monitor
python310Packages.greeneye-monitor.dist
python311Packages.greeneye-monitor
python311Packages.greeneye-monitor.dist


F# triple-quoted strings are not highlighted correctly

e.g. in
"""A triple-quoted string can contain quotes "like this" anywhere within it"""

the part like this is not highlighted as a string.

@wesleywiser  or @cprieto Are you interested in taking a look at this (triple-quoted string support in CodeMirror's F# mode)?

@marijnh I will take a look as soon as I have some time this week.


feat: checkbox component

Summary
Features
Bugs
Check List

[x] Your code builds clean without any errors or warnings.
[x] Check coding convention.
[x] Updated Storybook components and stories to reflect any changes made to the UI.
[x] Test covers all edge cases and coverage is greater than or equal to 80%.

Proof of Completeness

https://user-images.githubusercontent.com/124558477/230290553-97a66282-ea53-4476-a796-15d60c4c53a5.mp4

Please move the magic number and magic string
Use theme for default value of metrics and color.

updated

could you provide your test case result ?

could you provide your test case result ?

Done

@ngochuyduong should we write test case for lines 132 and 136?

@ngochuyduong should we write test case for lines 132 and 136?

update test cases


Precompiled binary not available for 1.16.0

Expected Behavior

Up through 1.15.0, the detekt binary is available as a release artifact.  It should also be available for 1.16.0.
Observed Behavior

It is not available.
Steps to Reproduce
See release artifacts here: https://github.com/detekt/detekt/releases/tag/v1.16.0

Context
We run detekt by running the precompiled binary.  Without this, we can't use version 1.16.0.
Additionally, the quick start instructions in the Readme won't work for version 1.16.0.

That was a decision we took once we bumped to Kotlin 1.4.21 here https://github.com/detekt/detekt/pull/3254#issuecomment-761624329
Please use the -all.jar files we ship with every version with java -jar. If that doesn't fit your needs, please explain what's your specific use case 👍

We'll be able to adapt when we update the version of the library we use.
Please update the quick start section of the readme, though.  I imagine it will be jarring for new folks.

Good point, yes readme was left not updated.


[SUGGESTION] add separate font option for chat

Acknowledgement of preliminary instructions

[X] I have read and acknowledged the preliminary instructions.

What idea do you have?
I would like a separate font option for chat to have more user choice, if it's possible.
This hypothetical option will be next to the current font option if it's implemented.

Not possible. We can only globally apply fonts.


Fix error with Pyinstaller not finding files inside the static

check the dot(.) syntax

Can confirm 1.6 fixes this issue. Filesize also drops to 21.1mb from 37mb.

@jwbn Thank you. Closing now


[WIP] Implement ROM API Interface structure

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
See PR #1432
What are the significant changes in functionality due to this change request?

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[ ] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

@dylanjm  let me know if you have questions

Closing this PR as the new implementation was taken over by @wangcj05


[Bug] New file open in top sidebar folder

1. Summary
I can to create file via File Manager in top folder of my sidebar, but not in current folder. Some features into GIFs don't work for me.
2. Settings
My sidebar:

Киролайна — top folder of my sidebar,
KristinitaLuckyLink/KristinitaLuckyLink.py — opened file.

3. Steps to reproduce
I reproduce the problem in a version of Sublime Text without plugins and user settings.
I install File Manager → I restart Sublime Text → I open file KristinitaLuckyLink.py → Ctrl+Shift+P → File Manager: New File.
4. Expected behavior
Create file in current folder.
5. Actual behavior
See image in 2 section. I create file in top folder, not current.
6. Environment
Operating system and version:
Windows 10 Enterprise LTSB 64-bit EN
Sublime Text:
Build 3126
Thanks.

This is not a bug. I've never had the idea of doing this, it is just a coincidence in a GIF as you said (could you let me know which one is it?).
It is a good idea, though, I'll add this on the todo. 👍

Oh wait, I've done this, see details here: https://github.com/math2001/FileManager/issues/16#issuecomment-308361859. Maybe there should be an example somewhere in the docs though. I'll close this anyway :slightly_smiling_face:


Unclear ALLOWED_HOSTS error message when running 'python manage.py runserver'

This is the error message I get when running
python manage.py runserver
/home/graham/Virtual-Environments/mezz_test/local/lib/python2.7/site-packages/mezzanine/utils/conf.py:51: UserWarning: You haven't defined the ALLOWED_HOSTS settings, which Django requires. Will fall back to the domains configured as sites.
  warn("You haven't defined the ALLOWED_HOSTS settings, which "
/home/graham/Virtual-Environments/mezz_test/local/lib/python2.7/site-packages/mezzanine/utils/conf.py:51: UserWarning: You haven't defined the ALLOWED_HOSTS settings, which Django requires. Will fall back to the domains configured as sites.
  warn("You haven't defined the ALLOWED_HOSTS settings, which "

It is given twice and ends with the unnecessary line
warn("You haven't defined the ALLOWED_HOSTS settings, which "

That's a combination of what Django and the warnings lib are doing, there's no issue here.

On your local_setting.py  add  something like this:
ALLOWED_HOSTS = [ '<IP_ADDRESS>:8000', ]
donot forget the comma at the end


Add new company: Okta

Company Name?
Okta
What's the company main business today?
Identity Management
What did the company do in the early days?
monitoring and reporting on the reliability of cloud services
What caused the company to pivot?
customer demand, "it was unscalable"
Source?
https://www.entrepreneur.com/article/375105

Hey thanks for the summary. I changed the source to be this tweet as it provides first-hand source of the why they pivoted: https://twitter.com/toddmckinnon/status/1266004294911721480
Let me know if you have better source. Thanks!

Hey thanks for the summary. I changed the source to be this tweet as it provides first-hand source of the why they pivoted: https://twitter.com/toddmckinnon/status/1266004294911721480
Let me know if you have better source. Thanks!


Create Helm chart of "Faber" university

Description
As a developer I want to create "Faber" university helm chart so that I can deploy it on a kubernetes cluster using BAF.
Prerequisites:

Indy nodes should be already running on Kubernetes
Indy agent should be able to get the node ledger

Acceptance Criteria

Helmchart created
Documentation created

Helm Chart is created and tested on Minikube. I created init container for getting endorser's seed from vault and store into volume from where the main container read and put to environment variable. Seed from file is removed after put as environment variable.


⚠ Thumbnails API Endpoint has degraded performance

In 4544c28, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 1080 ms

Resolved: Thumbnails API Endpoint performance has improved in 273b7e0 after 6 minutes.


minor changes

I've made a couple of minor changes:

Removed the require of a less file as it caused issues.
Added an ability/prop to pass an on focus event handler to the telephone input field.
Added an event listener to trigger setting focus on telephone input field (trigger by creating an firing an event from the parent node)

Removing the less file is not a minor change :) You can do something instead like surrounding the require with a if (typeof document !== "undefined").
You should add the onFocus handler to the actual element instead of creating a global one.

Thanks for the quick feedback.
The reason I commented it out instead of wrapping it with an if is because when I did that, it throws an exception on the client: Error: Cannot find module "../less/default.less".
Also, can you please elaborate on the onFocus comment? There was an event attached to the element already, I just added a way to pass through an external handle function:
handleInputFocus(e) {
        // if the input is blank, insert dial code of the selected country
        if(this.refs.numberInput.getDOMNode().value === '+') {
            this.setState({formattedNumber: '+' + this.state.selectedCountry.dialCode});
        }
        if(typeof this.props.onFocus === 'function') {
            this.props.onFocus(e);
        }
    },

The 2nd change I've made was to allow setting focus on the nested input element from external source (i.e. a button click from a different react component).

I'm assuming you're getting an error with less because you don't have less compilation setup for your project?
We should probably compile the less to raw css to make it simpler for people to import as more people have their webpack/browserify projects setup to work with css than less.

Closing this since there are other ways around it.


How do I get access to the payload?

Thanks for providing this example. Since I want to save user specific data to a database I need access to the userId - which is the sub in the jwt payload. How do I get access to the payload data in the route?
Although "dependencies" injection contains the validate_token method which should return the jwt payload I can't get access to it.

Closing the issue


[OCU-89] change registry from docker to github

Can you please create a ticket for this?

can you add the ticket to the commit message please?

@northdpole can you please rebase this on the latest main? there's a lot of noise atm from the changes in the main's commit history

done @ptzianos


feat: Bump OTel image version to 0.115.0

Description
Changes proposed in this pull request (what was done and why):

Bump OTel image version to 0.115.0
Bump telemetrygen image version to 0.115.0

Changes refer to particular issues, PRs or documents:

https://github.com/kyma-project/opentelemetry-collector-components/pull/177

Traceability

[x] The PR is linked to a GitHub issue.
[ ] The follow-up issues (if any) are linked in the Related Issues section.
[ ] If the change is user-facing, the documentation has been adjusted.
[ ] If a CRD is changed, the corresponding Busola ConfigMap has been adjusted.
[x] The feature is unit-tested.
[x] The feature is e2e-tested.

You missed the load test, there we also use an image and telemetrygen.. how can we simplify that?


Modifying Index.html

Added a hero section. Now the page contains a hero section along with the hamburger menu.
This leads to a better user experience
solves #68

Since you have made some changes, I believe I should review them.

I noticed those changes. It's not a music app, but thanks for your contribution.


Add Gameforge Client to SDK

SteamDB app page links to a few games using this

Nostale
Metin2
Runes of Magic
Swords of Legends Online
AION Free-to-Play
Swords of Legends Online Beta

Brief explanation of the change
find gameforge_api.dll.

After running changed rules in this pull request (this bot only tests rules.ini changes):
Changed technologies: SDK.Gameforge_Client
These apps now match: (10)<PHONE_NUMBER>80<PHONE_NUMBER>00<PHONE_NUMBER>90 1418100 1541840 1558640 1619130
Please confirm that all these changed apps are correct.


Incorrect JSON parsing for float

import ceylon.json {
    parse
}
test
shared void parseFloatTest(){
    String s = (parse("1.01"))?.string else "";
    assertEquals(s,"1.01");
}

Test fails because parse returns 1.1 (skeeping leading zeros after dot)

    // Parsing floats is tricky if you want to get all the corner cases like very small
    // or very large floats, and get every single digit correct. Rather than re-implement
    // we will build a string then use the built-in native parseFloat() function.

Tricky indeed, even if you cheat ;)

By the way, in Firefox and Chromium, `JSON.parse("010")` gives a parse error, so `parseDigits` should treat leading zeroes specially even before the dot.

Yes, ceylon.json::parse is indeed ignoring leading zeros after the decimal:
Integer digitsAfterDecimal = parseDigits(tokenizer);

@lucaswerkmeister we don't have to give an error for JSON.parse("010") though, do we? Is there some spec for this?

Yes, the spec is called JSON :P and it clearly states (emphasis added):

A number is represented in base 10 with no superfluous leading zero.

ECMA-404, § 8 Numbers. You can also see this in the short summary on json.org:

int

digit
digit1-9 digits

digit

digit1-9 digits

ok :)

Thanks @MikhailMalyutin. Fixed.

@jvasileff I don't think you added a test for this, did you?

I think he did, in the same commit that fixed the issue (8fe58cf).

Oops, you're right, thx

New ceylon.json module has been published: https://herd.ceylon-lang.org/modules/ceylon.json/1.3.0-1


Chunk records should have a start and end stamp

With a start and end stamp on chunks, I could cheaply identify if I need to read a chunk or decompress a chunk based on the time I want to read messages at. Without the start and end stamp I have to process a chunk before knowing it contains messages for my timestamp.

+1

Captured in the branch here: https://github.com/foxglove/mcap/tree/roman/spec-format/docs/specification


Updates AnimatedRegion component

This PR doesn't directly address any issues; rather, it's more of a cleanup PR.
Changes

cleans up the spring and timing methods of the AnimatedRegion component
implements a more readable flow in the constructor when setting latitude, longitude, latitudeDelta, and longitudeDelta via the setInitialValue method

Testing
Run the example apps to confirm functionality.

@jasonmichaels I'm not sure why those files int the android directory were creating but I don't think it was from merging in master. Either way, do you mind checkout out those files? I think the only files that need to be changed are AnimatedRegion.js  and index.d.ts

@jasonmichaels any updates here ?

@jasonmichaels can you please give an update to this PR?

@christopherdro I updated the PR to infer/coerce useNativeDriver from per-spring/timing calls in the config. Also updated the docs and added notes to example files AnimatedMarkers and AnimatedRegions indicating that while useNativeDriver is set to true in said examples, it defaults to false if not specified/given.

@miallo I removed the instance method, setInitialValue to a standalone function, getAnimatedValue. Not sure if there's a better place for that (I'm considering a PR to consolidate constants and common functions).


Attendee Tracking: Created AttendeeSelectionPage

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/47171
$ https://github.com/Expensify/App/issues/47172
PROPOSAL:
Tests

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@ZhenjaHorbach please review it too, this is an important PR that needs to be merged by Monday (hopefully).

@ZhenjaHorbach please review it too, this is an important PR that needs to be merged by Monday (hopefully).

Got it ! In that case I'll check today

Thank you very much!

@ZhenjaHorbach did you still need access to the docs?

@ZhenjaHorbach did you still need access to the docs?

Yes, please !
My email<EMAIL_ADDRESS>
I've added you! Let me know if you need anything else.

hi @ZhenjaHorbach, thank you for the review! :)
About the previous message, I fixed the first bug, however I'm not sure about the second one as you don't exit the creation of expense, so the transanction-draft object is not cleaned 🤔

And I found another bugs

After we unselect submitter of the Request, we will no longer be able to find it in the sections

But I'm not sure how best to fix this Maybe we need to create a new section 🤔 I think we need help from designers on where to best display a submitter of the Request
@Expensify/design Could you please check this case and help us how to fix it better?

In Recents we can find a workspace and after we select the workspace, the list will break

2024-09-<IP_ADDRESS>.mov

Regarding pt 2 I am wondering if this shouldn't be handled by BE and if Workspaces shpuldn't be excluded from the recents list? Otherwise we have to filter received data somehow
cc @Julesssss WDYT?

Regarding pt 2 I am wondering if this shouldn't be handled by BE and if Workspaces shpuldn't be excluded from the recents list?

We'll need to find a way to exclude workspaces here, as attendees can only ever be people.

I shared the above questions here with the design team

Hey @filip-solecki, as Jon mentioned here, after unselection let's show users their own profile item row always at the top of recent.
Also, FYI I'm going to switch the base branch to a feature branch (not sure if non-org members have the ability to do that)

Hey @filip-solecki, as Jon mentioned here, after unselection let's show users their own profile item row always at the top of recent.
Also, FYI I'm going to switch the base branch to a feature branch (not sure if non-org members have the ability to do that)

Done, when creator is unselected it goes to Recents section.

Ok, thanks I'll work on them

Both fixed, please check again @ZhenjaHorbach

@ZhenjaHorbach again all yours 😄

Looks like a bug, let me check

@ZhenjaHorbach can you upload a video?

Thanks all, this is looking good. As we haven't yet made the backend changes available to you I think we are in a good place to merge to the feature branch. Then we can create bugs for the remaining issues.
Once main is merged I'm going to add a C+ reviewer to try to locate other bugs that may have been mixed.

@ZhenjaHorbach don't worry! it was not very elegant but for sure quick fix 😊

And when I open screen Recents list is empty, but when I unselect submitter of the Request
I have full Recents list
About this bug, I couldn't reproduce it, did you do anything special? 😅

When we open screen, we couldn't find the submitter using search
But if we unselect, submitter will be always visible using search
This one I managed to reproduce and indeed it's very weird... I will look into this right now 🕵♀

Okay, this is looking good. Let's merge these changes to the feature branch so we can get the backend changes in place for you. Then we can handle the remaining bugs (recent empty, workspaces showing in list, ect)


Generate a new cert

/kind feature
Hi. I didn't found a command or something else in the documentation about this.
Is it possible to regenerate a certification with a CLI command?
By default, certificate are provided for 3 months. But how to do if we want to regenerate the next rotation before the end? For testing or prevent a vacation for example.
Thank you

ok. Thank you.


How to consume the ML.Model in project in an ASP.Net .Net Framework 4.8 project in the same solution.

Problem encountered on https://dotnet.microsoft.com/learn/ml-dotnet/get-started-tutorial/generate-code
Operating System: windows
Using Windows 10, Visual Studio 2019 Version 16.6.1.
The Model Builder created a ConsoleApp and ML.Model in my ASP.net Solution, but I am unable to run the Predict function because in the PredictionEngine function on the line
ITransformer mlModel = mlContext.Model.Load(modelPath, out var modelInputSchema);
it says that it can not find Microsoft.ML.FastTree or 1 of its dependencies. I can confirm they are installed.
I tried running the console app as the starting project and it works fine in consuming the Model.
So I think it is incompatibility between .NetFramework 4.8 and the ML.MOdel project.
Do you have any suggestions how I can overcome this?
sincerely
Darren

I believe this is not possible and so we are going to create a .net core webservice to consume the Model which we can then call.

@Darren-Wood Sorry you ran into this problem! We do have a bug with .Net Framework 4.8. Is your project in the SDK style? We do have a work around if you don't want to create a .Net core webservice.

Thank you for your answer. We are going to consume it in the Core Web.Service.
Sorry for not getting back sooner


Update to Swift 5

Should resolve #7

Thank you for your contribution!
Unfortunately, we need to make some more changes to make it work. Continuous integration build failed.

Fix failing test due to wrong Swift version for test target.
Update tools versions using bundle update.
Update podspec.
Update README (and version if you want).


Find documentation and and then re-document application/mflux and request/response structure

Using Wireshark, Hector figured out that an application/mflux header is used for file uploads, along with a particular body structure. A google search only finds one instance of this string, in the University of Melbourne's python client:
        header += 'Content-Type: application/mflux\r\n'

It is not referenced in the "Mediaflux Developer Guide" or the "Mediaflux Services Documentation".
(Just sent a help request to CSES.)

[ ] Find authoritative documentation from Arcitecta on the request and response structure for application/mflux
[ ] Add what we learn to the "Missing Manual" that we will write to summarize those parts of the Mediaflux API that we actually use.

On Mediaflux Developer Guide p. 8:

DIME Encoding
• NOTE: Support for DIME has been depreciated in Mediaflux. This documentation exists purely for historical reasons
Mediaflux uses Direct Internet Message Encapsulation (DIME) encoding for all messages. Knowing about DIME is only relevant if any of the following applies:
• You are using web-services, in which case DIME encoding must be specified when setting up the web-service client application.
• You need to write a native client in a language other than Java (and cannot use web-services).

Further down:

The native format is used for TCP/IP transport. Native Mediaflux messages are DIME encoded. The first record in the DIME message contains a Mediaflux service request or response, described below. The subsequent DIME records contain attachments, if any.

Reading https://en.wikipedia.org/wiki/Direct_Internet_Message_Encapsulation

… the limit being the size information for each chunk, which was 32 bits so 1 gigabit …

(ie, the zero bytes are there because the file length was less than 2^16… if it were longer, there would be fewer zeros.)
So there seem to be mixed messages here: It’s deprecated… but it’s used for encoding all messages?
… and there is a Ruby library for it: https://formats.kaitai.io/dime_message/ruby.html
(But maybe we shouldn’t go too deep here? My sense is that we’re probably not going to be supporting upload through the app … just moving around files that are already on disk? But it will still probably come in handy… :shrug: )

We're not sure if we'll need this, or if we'll do it a different way.
Docs in the new repo are tracked by

https://github.com/pulibrary/tiger-data-app/issues/12


Add React to Recommended Client-side Libraries

I feel like in the context of a hackathon, trying to point developers to react docs, and expecting them to ramp up during the weekend, might be too big of a jump.


build: add rust-toolchain.toml file

Let me know what you think but, especially for open source, I prefer to pin the version of the language to make sure that all contributors are working with the same one and that no bugs or changes in behaviors they may encounter are due to running a different version of the language
Run cargo install at the project root to download and install the language version and the components we listed.

Can the channel be stable instead of a specific version?

It is not a good idea, it would mean that the project toolchain is automatically updated every time a new version is released, without any human check to see if nothing has been broken during this release. One day it will just refuse to compile because some breaking change has been introduced that we don't handle

That's why I use the nix shells,

I can see the version being defined there: https://github.com/cashubtc/cdk/blob/main/flake.nix
We have to use the same in both, otherwise it loses its interest.
It looks like you are using stable latest in it:
stable_toolchain = pkgs.rust-bin.stable.latest.default.override {
          targets = [ "wasm32-unknown-unknown" ]; # wasm
        };

but with target = wasm32 ??? Is it the only thing you compile too? Not native, only wasm?

but with target = wasm32 ??? Is it the only thing you compile too? Not native, only wasm?

It add the was wasm target, native is already included.
The CI also uses the nix flake, checks both wasm and native on stable and MSRV

So the only thing I would recommend would be to fix the version of the language you are using in the CLI.
And in the future make sure it's the same for both the toolchain file and the nix one

closed for https://github.com/cashubtc/cdk/pull/434
I set a specific rust-version in the flake and toolchain.toml (current stable 1.82.0)


Revert to only setting hostname with hostname

+1
google_set_hostname still miss hostnamectl code block. Tested on CentOS7 (the same in CentOS8, etc.) with no NetworkManager.
hostnamectl=$( which hostnamectl 2> /dev/null )
  if [ -x "$hostnamectl" ]; then
    $hostnamectl set-hostname "$new_host_name"

Static hostname (/etc/hostname) is still not changed and appears equal to VM Image.
I see hostnamectl code block is still in development tree.
Need in production
Best Regards,
Claudio

@kladiv this PR is 2 years old and this repo has since been deprecated. if you have an issue with google_set_hostname, please file against the GoogleCloudPlatform/guest-configs repo


Fix issue with ReflectionParameter (fixes #14)

The original code
$constructorParameters[] = $parameter->getType()->getName() == 'int' ? 'integer' : $parameter->getType()->getName();

has error because not all $parameter has a getType method (which are PHP's native types, such as int, bool etc.); this fixes it, by looping it thru, and it does not have getType, it will just take it from givenParameters; this is fine, because native PHP types are very flexible, and it will not cause error ( passing 3 as int, string or bool doesn't cause error )

Thank you so much for the contribution @starvsion 🙂 I have added credits to you in the changelog and the release 😄


Map Privacy module shows wrong description text

Open a map in the Builder
Open the privacy settings

The following module appears- note that the description should say "map", not "table" (this is how it appeared in the Editor). I would have changed the text myself but it looked like the source was coming from somewhere else.

Good catch. Since this modal is changing completely with the new design I will keep this closed but mention @nobuti just in case he is not aware. Thanks @csobier !!!: :)


Feature: also declare macro would benefit from SerializationConfig

E.g.
#[tsify_next::declare]
pub type Id = u64;

this would output the type alias as number, at the moment I have no way to specify I want it as bigint

If I understand your issue, you want to be able to set large_number_types_as_bigints using declare too ?

Exactly
Il ven 12 apr 2024, 19:23 Pantamis @.***> ha scritto:

If I understand your issue, you want to be able to set
large_number_types_as_bigints using declare too ?
—
Reply to this email directly, view it on GitHub
https://github.com/siefkenj/tsify/issues/28#issuecomment-2052175077, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZXINOJYSMLH7QFPHBIDATY5AJ7ZAVCNFSM6AAAAABGELOZA2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJSGE3TKMBXG4
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add unit tests

What does this PR do?
This PR implements unit tests for the notify extension
Description of Task to be completed?
Ensure that there are unit tests for the core functionalities of the notify extension
How should this be manually tested?
Run the following commands separately:

python2.7 ckanext/notify/tests/test_plugin.py (Tests for plugin.py)
python2.7 ckanext/notify/tests/test_ui_controller.py(Tests for ui_controller)

Any background context you want to provide?
N/A
What are the relevant pivotal tracker stories?
N/A
Screenshots (if appropriate)

Questions:
N/A

This might be the same as #12 ?

Cool. @andela-dhaniabelega. I've merged #12. There are a couple of conflicts with it. Could you fix them?

Sure. I've fixed the conflicts. @DavidLemayian


Unable to run a detached (forked) process in the background using fork from an extension

As an extension author I would like to:

Launch a node process from extension
The process must be alive even after closing vs code
User should need not have node on their machine.

Basically using fork with detached option.
I have tried using
const {fork} = require('child_process');

fork('<path ot js>', { detached: true, env: {...process.env, ELECTRON_RUN_AS_NODE: '1'})

However, when i close or restart VS Code, the forked process also dies.
I have tried initializing execArgv to [] to no avail.
& also tried clearing some of the VSC env variables as follows:
env: {
	...process.env,
	VSCODE_IPC_HOOK: undefined,
	VSCODE_IPC_HOOK_EXTHOST: undefined,
	VSCODE_NLS_CONFIG: undefined,
	VSCODE_PID: undefined,
	VSCODE_HANDLES_UNCAUGHT_ERRORS: undefined,
	VSCODE_LOG_STACK: undefined
}

I have also tried
const proc = spawn(process.execPath, ['<path to js>'], {detached: true, env : <same as previous>}, stdio:'ignore');
proc.unref();

The issue seems to be the fact that detached is not being respected here.

Works when not in debug mode, the process dies only when debugging extensions.
Closing for now.

@DonJayamanne when forking a process, the new process automatically inherits command line arguments from the parent (accessible via process.execArgv). This includes the debug arguments. See https://github.com/microsoft/vscode/issues/27665#issuecomment-305444891
You can remove or modify debug arguments in the process.execArgv property before calling "fork"..


No module named 'rlkit.data_management.load_buffer'

In the rlkit there is no module called rlkit.data_management.load_buffer

https://github.com/vitchyr/rlkit/tree/master/rlkit/data_management

I found the script in this repo, but the original rlkit doesn't have


push-amt not shown as an invoice?

Alice wants to pay Bob 100k sats, but realize there is no path, so Alice decided to open a channel with Bob with a push-amt of 100,000:
lncli openchannel bob-key<PHONE_NUMBER>00
After the channel is opened, Bob can see his fund with lncli channelbalance but Bob don't really have a way to easily figure it out where the money comes from.
If Bob do a lncli listinvoices there is no apparent invoice that would explain the change in balance. This would be the expected behavior like for a keysend?
Or what is the right way to do a per transaction accounting for funds that are sent with push-amt?

The push amount becomes the remote party's initial balance in the channel, there isn't really a record for it other than it being the channel's initial state. One thing Bob could do now is receive a notification for when the channel is opened and inspect the channel state. If there aren't any updates and Bob has a balance, then it was pushed to them. This can be somewhat racy though if an actual payment is attempted right after the channel is opened, as the payment could settle before Bob has a chance to check the initial channel state.

I guess the challenge is, a wallet developer, how do you show to the user where those sats come from. The sum of the payments and invoices would not be equal to the balance.

I have to check, but I would guess the transaction would not appear on sendpayment for the other node neither

cc @carlaKC on accounting for push amts

I guess the challenge is, a wallet developer, how do you show to the user where those sats come from. The sum of the payments and invoices would not be equal to the balance.

The easiest solution here would be to just disallow push amounts with --rejectpush. We could add a feature where these push amounts present as key sends but I'm not sure if that's wise because they aren't actually payments.
@nicolasburtey would having a record of the fact that the push occurred be enough?

key sends but I'm not sure if that's wise because they aren't actually payments.

I'm not sure to understand this point
If I step back, I feel this would make sense to have some rules around sum(initialchannelockup) + sum(list"settled"invoice) - sum(listpayments) = current balance.
This is a first principle in accounting. I see that at list push amount breaks this assumption.

I'm not sure to understand this point

A keysend and a push amount are different concepts. Keysends are invoiceless sends.  They should not be recorded as paid invoices because they're not a payment you're receiving, there is no htlc.

If I step back, I feel this would make sense to have a rules around sum(initialchannelockup) + sum(list"settled"invoice) - sum(listpayments) = current balance.

Indeed, I'm currently working on accounting improvements in lnd. You would need to factor a push amount into this calculation. It unfortunately isn't as simple as this equation, because you'd need to factor in current reserved balance or fees on the channel.

I see that at list push amount breaks this assumption.

Correct, that's why I would recommend disabling push amounts so that these unexpected amounts cannot be sent to your node.

A keysend and a push amount are different concepts. Keysends are invoiceless sends.  They should not be recorded as paid invoices because they're not a payment you're receiving, there is no htlc.

How is push_amt paid then?
I thought that @Roasbeef said on slack that push_amt was not part of the funding transaction, but the amount was pushed after the channel was opened.
More importantly, why have those 2 concepts now that we have keysends? Seems to be the same use case for me.

When a channel is created, the two parties involved sign a 2-2 multi-sig transaction which is called the funding transaction. Before they exchange these signatures, they each sign a commitment transaction which pays out the respective parties their balances. This prevents parties from disappearing and leaving you’re money trapped in the multi sig.
Currently one party funds a channel. If they specify push amount, the first commitment transaction assigns some capital to the remote peer. So the channel does not exist, and when it it is created with a push amount the remote party gets some sats. This is not a payment. It is an amount that you give to the other party.
Keysend, by contrast, is a payment that does not require an invoice. It requires that a channel already exists, and can be sent to a peer that we do not have a channel open with. It has an invoice because a HTLC is sent to our node and we record the payment.
We are not going to record push amounts as payments. I am completely sympathetic to the accounting use case, but a push amount is simply not a payment by definition. If you would like to be able to account for push amount, I think the best course of action is to open an issue to record push amounts for channels on open. I will personally pick it up.
I’m going to close this issue for now, but please do open another issue for recording push amounts if it is relevant for you. I’m also around on the LND slack if you want to chat further. Having a look at the BOLT#2 spec would also be helpful to help your understanding.


Implement Email Sender

Email Should have models and saved to database when sent

Email Should have models and saved to database when sent


Support global prefix + global indentation

This PR implements a new field in the render config called a "global prefix", which will be interpreted by the prompt backend as a prefix that is rendered before each line written out to the terminal.
This will be a breaking change, since the RenderConfig type was altered to allow the user to infer a lifetime other than the 'static lifetime. A non-static lifetime was only added for the global prefix field, but for consistencies sake, the PR can be updated to support the other fields just as easy if needed. Alternatively, the non-static RenderConfig can also be broken out into a separate PR, if it is desired to be handled separately.
Additionally, a bug fix for hidden password prompts is also included in the PR (see commit history). Again, this can be handled in a separate PR if desired.
Feel free to come back with suggestions if the feature and/or implementation is appropriate!
Closes #81

Implemented global indentation as well as a field to the RenderConfig, which allows to indent the prompt and avoid clearing the content within the indentation. This can't be achieved using a global prefix.

Sorry for not giving a response for a very long time, I have not had the time to look into this any further. Given the age of the PR, and the fact that I don't actually need this feature very much anymore, maybe we should close this and give this another go if someone else feels like they need the feature?

Sorry on my end as well that it didn't work out, I'm closing for the sole purpose of clean-up (and the age of the PR), let me know if you'd still like to see this implemented upstream


codet5-base.jackson-core.mid-2546.idx-33440.2.mutant

Here, the writeArray method for double[] arrays in JsonGeneratorDelegate.java is correct and enhances functionality, either setting the array to null after writing it could potentially lead to unintended side effects or null pointer exceptions if the array is used elsewhere. It might be better to reconsider or thoroughly check for such cases.

I am reformatting not to change code function, but make it fit the format of code around it.


All the pages are opening in the same tab

As here all the pages are opening in the same tab  which can be inconvenient for the users sometime. I want to fix this issue and make them open in the new tab.
Please assign this issue to me under gssoc-extd and hacktoberfest.

we want it that way , its fine


Enhance Widget Layout

This update contains two changes to two different files.
I had an issue similar to Issue #3185 and was able to implement a solution that works along with the current voyager configuration but allow for more than three widgets to be on the dashboard window.
This is what the dashboard window looks like with 1 - 3 Widgets

This is what the dashboard window looks like with 5 widgets

Please let me know if there is any more information needed

Thank you very much for the help as well. It's much appreciated.


mark_dead NetworkStream method to support client keep-alives.

When the with_connector client constructor is used, and those connections are being
retrieved from a pool, and re-used as keep-alive connections, the stream
needs some way to be notified that it is going to be closed by the
server because "Connection: close" was provided in the
response.
Then, the stream (which will be closed by the server and the next
request would fail) can be drop()-ped instead of being retained in the
pool.
(Also, brought current with some rust nightly changes.)

@jamwt I'm going to wait and let @seanmonstar merge this since he's worked on the client more and has a better idea of if this is appropriate.

Why use mark_dead instead of handling this outside of the Request class?
On Thu, Jan 1, 2015, 6:47 PM Jonathan Reem<EMAIL_ADDRESS>wrote:

@jamwt https://github.com/jamwt I'm going to wait and let @seanmonstar
https://github.com/seanmonstar merge this since he's worked on the
client more and has a better idea of if this is appropriate.
—
Reply to this email directly or view it on GitHub
https://github.com/hyperium/hyper/pull/220#issuecomment-68506420.

So, you guys facilitate providing a connector which retrieves an stream.  Those streams are coming out of a pool.
This stream is a wrapper as part of the pool implementation that attaches to drop so the stream can be re-added to the pool when it is done--if the stream is still "healthy".  This has different definitions for mysql connections vs. http ones, etc, but the general gist is "can this stream be reused for the next request?"
So the connection trait has a method called "is_healthy(&mut self) -> bool".
Here's some application code using these abstractions:
{
    // client is hyper::client::Client, grabbed one from the pool which may or may not be reused.
    // Easy to do this using `with_connector`.
    let mut client = pool.client();
    let response = client.get(/* something */).send().unwrap();
    /* do something with response, which is regular ol' hyper::client::Response */
}
// drop()

When the response drops, the stream wrapper drops, and it checks to see if it is "healthy" according to the definition of HTTP (keep-alive), and if it should be re-added to the pool for reuse.  This information is derived from the response headers.
Unfortunately, it's not clear to me how else to get access to those response headers without some support from the HTTP framework, or without having to force the user (i.e., the application author) to remember to always explicitly pass the response into something...
Or to completely replace the hyper client/request builder/response layer with wrappers all the way down, so that we were using a custom response object that we could implement Drop and have it check the inner response (the hyper one) for headers, and use into_inner to extract the stream, and then unsafe transmute the stream into the concrete type that we know it is... etc.  That seems like overkill, when this is the one piece of information missing (given with_connector) that's necessary to integrate with connection pools.


Keep webpack

Is it possible to keep webpack for development purposes and use rollup just for production build.

Two different pipelines would be harder to maintain and ensure we're getting the same content in both dev and production -- may I ask why the Webpack preference? The only dev facing change is the lack of webpack-dev-server, but something similar can be implemented with something like rollup-plugin-serve

I wanted to keep webpack-dev-server  for serving so it's easier to access from my phone.  Didn't know about the plugin.

The rollup patch has been bitrotting for a bit so just wanted to get it in, but I can look into adding rollup-plugin-serve as well. In the mean time, running npm run watch with something like simplehttpserver to serve content will get the same functionality as before


Add canView check to Order.

Without an implementation of canView, only administrators could view
order details.
Fixes #322

I agree that we need to add support for this to the shop core, but I wonder if we need to be more robust and actually add a permission for being able to view/edit/etc an Order. I've always done the same as this in our projects using an extension and just given users access to the Orders ModelAdmin in the CMS. That's as far as any of our sites have ever needed and may be all that's required unless one is using a REST API for Orders or something.
@jedateach and @wilr do either of you have more experience with more granular permissions in SS and care to comment on this? I think we should take the simplest route and not over-complicate this.

(also, thanks for noticing this and working up a PR, @jameshoward!)

If it wasn't for canEdit leading the way I would have suggested a new permission or re-use of the CMS_ACCESS_OrdersAdmin that ModelAdmin will have created.

Also worth noting that the same is true of Payment. I can now view orders but the payments show as empty unless an admin. The whole lot could work under a single permission such as CMS_Shop.

(@markguinn no worries, glad to be able to contribute to something we're making good use of)

Interesting. I hadn't realized canEdit was already present. As such I'd be comfortable with merging this but hopefully adding more complete permissions in 1.1 or 2.0 (seems like a border-line backwards incompatible change to me if we're suddenly requiring an additional permissions for users that previously could see an order). I'd like to hear Jeremy's input though since that's a longer term question.

Yes, good point on it potentially being a breaking change.


DrbConnError when Transaction Events Array too large

Hi Appsignal, hi @thijsc,
back to this error from #38 and #39, but now with better knowledge.
Recap the setup -  Appsignal 0.11.11 and Rails with a single vanilla Resque worker that does 35k times a DB lookup, e.g. User.find('dontexist').
Recap the stacktrace:
Exception DRb::DRbConnError
Error Broken pipe
/usr/lib/ruby/2.1.0/drb/drb.rb:611:in `write'
/usr/lib/ruby/2.1.0/drb/drb.rb:611:in `send_request'
/usr/lib/ruby/2.1.0/drb/drb.rb:921:in `send_request'
/usr/lib/ruby/2.1.0/drb/drb.rb:1221:in `send_message'
/usr/lib/ruby/2.1.0/drb/drb.rb:1110:in `block (2 levels) in method_missing'
/usr/lib/ruby/2.1.0/drb/drb.rb:1197:in `open'
/usr/lib/ruby/2.1.0/drb/drb.rb:1109:in `block in method_missing'
/usr/lib/ruby/2.1.0/drb/drb.rb:1128:in `with_friend'
/usr/lib/ruby/2.1.0/drb/drb.rb:1108:in `method_missing'
/app/.bundle/ruby/2.1.0/gems/appsignal-0.11.9/lib/appsignal/ipc.rb:59:in `enqueue'
/app/.bundle/ruby/2.1.0/gems/appsignal-0.11.9/lib/appsignal/transaction.rb:160:in `complete!'
/app/.bundle/ruby/2.1.0/gems/appsignal-0.11.9/lib/appsignal/transaction.rb:26:in `complete_current!'
/app/.bundle/ruby/2.1.0/gems/appsignal-0.11.9/lib/appsignal.rb:92:in `monitor_transaction'
/app/.bundle/ruby/2.1.0/gems/appsignal-0.11.9/lib/appsignal/integrations/resque.rb:8:in `around_perform_resque_plugin'

Unfortunately it does not state what caused the error, but happy debugging revealed there is a default load_limit for messages over drbunix connections of 25MB! I stepped through the code to the point in ruby 2.1.0 drb.rb line 611 with varying lengths of the messages to be send. And tadaa its true to the byte.
In Appsignal terms that means the Appsignal::Transaction (= the object to be marshalled) which is too large (in marshaled form > 25MB) triggers this error.
The transaction object gets too large quite easily - as stated before - when a worker job has to do a bit.
Our workaround was and still is that we clear the events queue of the transaction object as last statement in the resque jobs of which we know to trigger the error
Appsignal::Transaction.current.clear_events!

This load_limit of DRb could theoretically be configured, but not transparently via Appsignal config, at least I didn't find a way.
Cheers,
Baschtl

We've completely overhauled this system in the 1.0 version of our gem. This should now work like a charm.


Test installation status when ideviceid is not installed

Ensure that flutter doctor returns a partial installation status when
ideviceid is not installed.

@tvolkert @goderbauer @xster

Spotted while fixing #10245

The newly added test fails on Travis and AppVeyor

Travis and Appveyor seem rather unhappy with this change :(

Can you paste what the doctor outputs in that case? Just checking if we're explaining to the user what the various dependencies are for in case they fall into edge cases.

@xster This doesn't fix #10245, this simply adds coverage where we had none before. I'll be fixing that in a separate patch.

PTAL

FYI this now does fix #10245. When libimobiledevice is at the incorrect version you now get this:
[-] iOS toolchain - develop for iOS devices (Xcode 8.3.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 8.3.2, Build version 8E2002
    ✗ libimobiledevice is incompatible with the installed Xcode version. To update, run:
        brew update
        brew uninstall --ignore-dependencies libimobiledevice
        brew install --HEAD libimobiledevice
    ✗ ideviceinstaller not available; this is used to discover connected iOS devices.
      To install, run:
        brew update
        brew install --HEAD libimobiledevice
        brew install ideviceinstaller
    • ios-deploy 1.9.1
    • CocoaPods version 1.2.1


Swift Compiler Error with PerfectICONV

I have added the dependency "https://github.com/PerfectlySoft/Perfect-LDAP.git" to the Package.swift file of my project. Esto provoca lo siguiente:

When I compile it causes me the following errors:

Apparently some modules are being included twice

Is there any update regarding this since I also face the same issue?


Discussion: Why not base this off VSCode and Web Components + Observables?

It's not too difficult to build our own VSCode.dev. Which will have search and replace, good formatters, etc
... and be far closer to supporting The TS Language server for web and other extensions.
We could also fork Theia or Gitlab's Web IDE

https://github.com/eclipse-theia/theia (Uses semi colons)
https://gitlab.com/gitlab-org/gitlab-web-ide-vscode-fork (Uses tabs and semi-colons)

Alternatively, if the goal is to keep it small and nimbler... why not use web components and and observables like
https://stackblitz.com/edit/movies-web-components-dbapmu?file=package.json
We already have a 256MB node_modules, 174MB are from things that depend on react...
Currently we are a much worse version of @worbles
Which does run in VSCode.dev
Non-of the following depend on node:
{
  "dependencies": {
    "@webcontainer/api": "^1.1.4",
    "deep-object-diff": "^1.1.9",
    "dockview": "^1.7.3",
    "xterm": "^5.1.0",
    "xterm-addon-fit": "^0.7.0"
  },
  "devDependencies": {
    "typescript": "^5.1.3",
    "vite": "^4.3.9",
    "vite-plugin-node-polyfills": "^0.8.2"
  }
}

The goal is to keep it simple and light. Only a small layer above Monaco.
The current production bundle is 183.82 kB gzipped.
React is needed for the libraries used such as Dockview and RCT, and eventually Radix UI. Preact could possibly be used but not much point with the overall remote dependencies that Monaco and WebContainers incur.
The 256MB~ you see are from development dependencies. Monaco is like a 80MB module (only for dev), SWC, ESBuild, Typescript all add like 30MB. This is not a problem for end users and not really noticeable for maintainers.


$title $title++ $title$ ADD PLS

If the videos in a playlist have the same name, when downloading the playlist, instead of saving all the videos, one is saved, then another is saved in the same file, and the first one is lost etc

$index
I did not see it 😆


Highlight map points from the same structure

When in atom mode, it would be good to highlight on the map other points pertaining to the same structure as the currently selected atom.
A good way to do that would be to dim / grey out other points, using the same mechanism as #3.

I think this could/should go hand-in-hand with the highlighting of the "clickable" atoms in the JSMolWidget. When in environment mode, one should see all the environments highlighted in the JSMolWidget, and the corresponding points should be highlighted in the scatter plot - perhaps just made 2x larger, with thicker outline, and brought to front in the z-index.

and brought to front in the z-index.

This is not easy since we are not using HTML to render the graph but WebGL (which is why we are able to render more than 10k points 😄). Instead we would need the same kind of bookeeping infrastructure as for #16 for coloring.
Bringing points on front would require either using a second plotly trace, with the related performance issue when changing the selected point --- this is why we use a small hack to display selected point in 2D mode --- or duplicating the same hack for all highlighted points.


support get document by id

PR-Codex overview
This PR focuses on adding a new getDoc function to the SDK and implementing the corresponding API endpoint in the IndexerNode service.
Detailed summary

Added getDoc function to the SDK to retrieve a document by its ID.
Implemented the GetDoc API endpoint in the IndexerNode service.
Updated the db_store_v2.rs file to handle the getDoc request.
Added tests for the getDoc function in the SDK and doc_store module.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Codecov Report

Merging #602 (3dd1521) into main (bb44ae3) will increase coverage by 0.53%.
The diff coverage is 83.33%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #602      +/-   ##
==========================================
+ Coverage   61.70%   62.24%   +0.53%
==========================================
  Files           9        9
  Lines         470      482      +12
  Branches       69       71       +2
==========================================
+ Hits          290      300      +10
- Misses        180      182       +2

Flag
Coverage Δ

rust
62.24% <83.33%> (+0.53%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

sdk/src/store/document_v2.ts
84.61% <66.66%> (-2.35%)
:arrow_down:

sdk/src/provider/indexer_provider.ts
75.60% <100.00%> (+4.18%)
:arrow_up:

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

@qa-agent

Unit Test Analysis
:dart:level:Important
:mega:explanation:The pull request includes changes to the getDoc function, which is an important part of the functionality. It is necessary to ensure that the function works correctly and handles different scenarios.
Existing Test Cases

Summary
Category
File

Test that the getDoc function retrieves a document successfully.
positive testing
sdk/tests/client_v2.test.ts

Test that the getDoc function throws an error when the document is not found.
negative testing
sdk/tests/client_v2.test.ts

Recommended Test Cases

Summary
Category

Test that the getDoc function handles edge cases, such as null inputs and large document IDs.
edge testing

@qa-agent

Unit Test Analysis
:dart:level:Important
:mega:explanation:The pull request introduces a new function getDoc that retrieves a document from the database by its ID. This is an important functionality that needs to be thoroughly tested to ensure its correctness.
Existing Test Cases

Summary
Category
File

Test that the new function is called successfully and returns the correct document.
positive testing
sdk/tests/client_v2.test.ts

Test that the new function throws an error when the document is not found.
negative testing
sdk/tests/client_v2.test.ts

Recommended Test Cases

Summary
Category

Test that the new function is called with invalid inputs, such as null ID or non-existent collection.
edge testing


update nav_bar and corresponding tests

I rule and have completed some work on Case Manager that's ready for review!
hops off https://github.com/DCAFEngineering/dcaf_case_management/pull/1932 to fix a real minor test issue. (CI would have nabbed this, but CI's still busted for branches.)
This pull request makes the following changes:

#1932 pretty much

It relates to the following issue #s:

Fixes #1924

godmode because this is fine and my commit is a test fix


graticule lines lost with overlays

Reported by @AnsleyManke on 8 Mar 2017 22:10 UTC
Martin Schmidt reported this, and we see it with LAS plots.
http://www.pmel.noaa.gov/maillists/tmap/ferret_users/fu_2017/msg00048.html
This goes back a ways. Starting with the fix to #2279, we draw axis lines only on the first plot, to prevent over-thickening of lines when drawing with PyFerret. Previously, axes were re-drawn on overlays, to make sure that things like tic marks and graticule lines weren't obscured when overlays were drawn.
Ferret example:
yes? use coads_climatology
yes? shade/l=1/graticule=(color=blue) sst
yes? fill/over/l=1 sst
yes? go fland 20 gray

The graticule lines are visible only in places where there is no data in SST and no land.
Perhaps graticule lines, and maybe also tic marks, but not the axis lines, should be re-drawn with each overlay.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2514

If we drew graticule lines on all overlays, then those graticule lines would get thicker and thicker in PyFerret plots where they show up.  I think the best answer is to allow the user coltrol when to plot graticules -- which are part of axis-line drawing -- on an overlay, so that for a high-quality plot, we would remove axes from all of the layers except the last thing overlaid.  As things stand now, /OVERLAY on a plotting command causes us to never draw axes or graticule lines.
The new behavior will be that /OVERLAY/AXES will draw the axes and any graticules that have been defined.
So,
yes? shade/noaxes my_var
yes? set mode graticule
yes? fill/overlay/nolabel/AXES my_var

Often the last thing drawn is the land masses using the fland.jnl script
yes? shade/noaxes my_var
yes? fill/overlay/nolabel my_var
yes? go fland

If there are graticules, then we want the graticule lines on top of the land masses.  The fland.jnl script could by default make plot axes. It could have a new option to control whether axis/graticule lines are drawn.
I have checked in the use of /OVERLAY/AXES as a combination, and am closing this ticket. We may want to revisit this as we work with it.``


Build fail on unknown type from GraphQL Scheme

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
20
Amplify CLI Version
12.10.2
What operating system are you using?
als2023 (amplify build console)
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
Nope
Describe the bug
Build that worked yesterday will not build today. Suspect it to be CLI version 12.10.2, works with 12.10.1.
In my GraphQL scheme I do have the following custome query:

enum BillingSource {
  CLIENT
  PROJECT
  FLEET
  DEPLOYMENT
}

input QuerySumBillingParamsInput {
  what: BillingSource!
  id: String # required for all except client
  billed: Boolean # null == all , false == unbilled true==billed
  from: AWSDateTime
  to: AWSDateTime
}

type Query {
  sumBilling(params: QuerySumBillingParamsInput!): Float
    @function(name: "billingFunction-${env}")
}

This has not been changed for a long time, and today amplify build fails with:
2024-03-07T10:35:46.418Z [WARNING]: ✖ There was an error initializing your environment.
2024-03-07T10:35:46.424Z [INFO]: 🛑 Unknown type QuerySumBillingParamsInput
                                 Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/
2024-03-07T10:35:46.437Z [INFO]:
2024-03-07T10:35:46.437Z [INFO]: Session Identifier: e0409621-5b2b-4afb-86bb-d38cf0535346
2024-03-07T10:35:46.547Z [ERROR]: !!! Build failed
2024-03-07T10:35:46.547Z [ERROR]: !!! Error: Command failed with exit code 1

Has support for custom input types been removed from queries?
If I downgrade to 12.10.1 it works...
Expected behavior
Should work as previous version 12.10.1
Reproduction steps
Try to build a grahql scheme with a query which has custom input variable
Project Identifier
0e0c35de10e80e85c555ebedce4575e5
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

We have identified a regression in CLI version 12.10.2 and rolled back latest to 12.10.1.


Support binary_sensor and device_tracker in HomeKit.

Description:
Map binary_sensor and device_tracker to HomeKit Binary Sensor, based on 'device_class' attributes:
'co2': 'CarbonDioxideSensor'
'gas': 'CarbonMonoxideSensor'
'moisture': 'LeakSensor'
'motion': 'MotionSensor'
'occupancy': 'OccupancySensor'
'opening': 'ContactSensor'
'smoke': 'SmokeSensor'

If there is no any matched device_class, e.g. device_tracker, then default sensor type is OccupancySensor.
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox.

Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#5143

Thanks :+1:
Looking forward to more PR's

Thank you for your effective advice and guidance.

cong~ merged!

@Yonsm great, thanks a lot! @cdce8p maybe its possible to add a default mapping to binary_sensors with device_class "opening" for all sensors which returns states like "open/tilted/closed"? something similar to the existing temperature sensors. with that change I don't need to create extra template binary_sensors anymore :)

@cdce8p great to hear, I can wait for this 👍 keep up this great work! 🥇

Or maybe it's best to default to ContactSensor since this allows the most flexibility in Homekit and is probably the likely use of a binary_sensor?

Please don't discuss feature requests in closed PRs.

@MartinHjelmare seemed like an omission, what SHOULD I do instead?

If you want to suggest an enhancement please open a feature request in the Feature Requests section of our community forum.
Merged PRs should not be used for enhancement discussion or bug reports. If you've found a bug it's ok to make a review with inline comments and link to an issue that reports the bug.
Thanks!


Appstudio update test-custom-default-branch-qapl

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-wadk-on-pull-request-
creating pipelinerun test-simple-build-wadk-on-pull-request- in namespace build-e2e-qbuy-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-uqzm-on-pull-request-
creating pipelinerun test-simple-build-uqzm-on-pull-request- in namespace build-e2e-qbuy-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-nwtb-on-pull-request-
creating pipelinerun test-simple-build-nwtb-on-pull-request- in namespace build-e2e-qbuy-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-mijr-on-pull-request-
creating pipelinerun test-annotations-mijr-on-pull-request- in namespace build-e2e-qbuy-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-lqle-on-pull-request-
creating pipelinerun test-simple-build-lqle-on-pull-request- in namespace build-e2e-qbuy-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-mcmm-on-pull-request-
creating pipelinerun test-simple-build-mcmm-on-pull-request- in namespace build-e2e-qbuy-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-ufgi-on-pull-request-
creating pipelinerun test-simple-build-ufgi-on-pull-request- in namespace build-e2e-qbuy-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-nyrh-on-pull-request-
creating pipelinerun test-simple-build-nyrh-on-pull-request- in namespace build-e2e-qbuy-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-imul-on-pull-request-
creating pipelinerun test-simple-build-imul-on-pull-request- in namespace build-e2e-qbuy-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


wait_for_arrival error

I am getting this error when I do a move_to on a smoothie (mine is not connected to a robot but that shouldn't make a difference for jogging). I also get it occasionally on the Virtual Smoothie when doing a move_to.
  File "/Users/levkravinsky/Desktop/ot/v2/server/env/lib/python3.4/site-packages/flask/app.py", line 1820, in wsgi_app
    response = self.make_response(self.handle_exception(e))
  File "/Users/levkravinsky/Desktop/ot/v2/server/env/lib/python3.4/site-packages/flask_cors/extension.py", line 161, in wrapped_function
    return cors_after_request(app.make_response(f(*args, **kwargs)))
  File "/Users/levkravinsky/Desktop/ot/v2/server/env/lib/python3.4/site-packages/flask/app.py", line 1403, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/Users/levkravinsky/Desktop/ot/v2/server/env/lib/python3.4/site-packages/flask/_compat.py", line 33, in reraise
    raise value
  File "/Users/levkravinsky/Desktop/ot/v2/server/env/lib/python3.4/site-packages/flask/app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "/Users/levkravinsky/Desktop/ot/v2/server/env/lib/python3.4/site-packages/flask/app.py", line 1477, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/Users/levkravinsky/Desktop/ot/v2/server/env/lib/python3.4/site-packages/flask_cors/extension.py", line 161, in wrapped_function
    return cors_after_request(app.make_response(f(*args, **kwargs)))
  File "/Users/levkravinsky/Desktop/ot/v2/server/env/lib/python3.4/site-packages/flask/app.py", line 1381, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/Users/levkravinsky/Desktop/ot/v2/server/env/lib/python3.4/site-packages/flask/_compat.py", line 33, in reraise
    raise value
  File "/Users/levkravinsky/Desktop/ot/v2/server/env/lib/python3.4/site-packages/flask/app.py", line 1475, in full_dispatch_request
    rv = self.dispatch_request()
  File "/Users/levkravinsky/Desktop/ot/v2/server/env/lib/python3.4/site-packages/flask/app.py", line 1461, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/Users/levkravinsky/Desktop/ot/v2/server/main.py", line 258, in jog
    result = robot.move_head(mode="relative", **coords)
  File "/Users/levkravinsky/Desktop/ot/v2/server/env/lib/python3.4/site-packages/opentrons_sdk/robot/robot.py", line 209, in move_head
    self._driver.move_head(*args, **kwargs)
  File "/Users/levkravinsky/Desktop/ot/v2/server/env/lib/python3.4/site-packages/opentrons_sdk/drivers/motor.py", line 315, in move_head
    return self.consume_move_commands(args_list, increment)
  File "/Users/levkravinsky/Desktop/ot/v2/server/env/lib/python3.4/site-packages/opentrons_sdk/drivers/motor.py", line 330, in consume_move_commands
    self.wait_for_arrival(tolerance)
  File "/Users/levkravinsky/Desktop/ot/v2/server/env/lib/python3.4/site-packages/opentrons_sdk/drivers/motor.py", line 354, in wait_for_arrival
    dist = pow(diff['x'], 2) + pow(diff['y'], 2) + pow(diff['z'], 2)
KeyError: 'x'```

Fixed - this was due to a frontend bug with the Socketio thread asking the backend for the current position in the middle of other commands (getting current position is not thread-safe).


Filled Button Style before iOS 15.0 ⚠

In VerificationViewController.xib, set button style to 'default'  since other styles is only availabe in iOS 15.0 and above

i already solve it and create pull request feat/updateverificationscene but I can't find it here

@RaniaZeid  can you link that PR?


Feedback dialog is not responsive

The feedback dialog from the EN does not look good below about 575px height. They have no resources available to fix it up, so we will have to do it ourselves.

We "fixed" this a different way - by removing the options to submit feedback when the browser is too small. It's still possible to pop it up, then resize the browser, but no one is going to care.


add typings for TypeScript users

Summary
This is needed because type inference isn't that good, I'm afraid. If you have any remarks or criticism, please let me know!
Other Information

Looks great, thank you for taking the time to make everything more accessible. I went ahead and resolved some conflicts with another PR I just merged.
I agree, the lib really needed good typescript definitions.
If you're interested in helping maintain lmk!
Thanks again