[$250] Mark down - Copying and pasting part of a code block doesn't work

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.82-0
Reproducible in staging?: Y
Reproducible in production?: N
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Navigate to staging.new.expensify.com
Open any chat report
Send a code block using opening three back ticks and closing three back ticks at the end
Select part of the sent code block and press ctrl + c to copy and paste it on the compose box

Expected Result:
The copied code block is pasted
Actual Result:
Nothing gets pasted in the compose box
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/6728d623-7b1b-4ddf-8d57-604d9d52ff73

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01bd2b97089b315a6c
Upwork Job ID:<PHONE_NUMBER>148739899
Last Price Increase: 2024-06-12

Issue OwnerCurrent Issue Owner: @

@alexpensify I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

I think this was caused by https://github.com/Expensify/App/pull/43270
Reverting that PR solved the issue for me locally.

@mollfpr @rlinoz @SzymczakJ can you please check what's going on here? Seems related to https://github.com/Expensify/App/pull/43270

Indeed reverting the PR fixes it

Not adding the excludeStyles as a dependency here https://github.com/Expensify/App/pull/43270/files#diff-a1120b846ebeebcc99d15542e13bd9537c273751a97bce6a0de03295ec98d2baR89 fixes it, but I don't understand why

On it. I'll try to find solution ASAP

Ok, I've updated the assignments here to reflect everyone who is helping to address this issue.

Removing blocker label since it was CP'd

@situchan—I've sent you the offer via Upwork. Please accept it. Then, I'll complete the payment process. Thanks!

@situchan - I've sent over an offer via Upwork. Please accept, and then I can complete the payment process. Thanks!

All set here, I've completed the payment process. All actions are complete here -- closing!


Issue #12951 Add grafana integrations PR | Finished

Related Issue: https://github.com/zulip/zulip/issues/12951
Testing Plan:
I have completely follow the incoming-webhooks-walkthrough
I have ran manually test, automated test and test-all script in my dev environment. Please feel free to double check
GIFs or Screenshots:
No UI changes

Thanks for working on this @Zhenye-Na!
This had several issues:

The commits were not squashed properly.  For commit history, we're not interested in the history of iteration of getting it working; just the series of logical changes make to the codebase.
Check out the Zulip commit guidelines for more details: https://zulip.readthedocs.io/en/latest/contributing/version-control.html
The docs had some sloppy English.
It seems like you modified various SVG files that were not part of the new integration; how did that happen?

I took care of making those changed and pushed back to this PR; please fetch my changes before doing further work on this PR (./tools/reset-to-pull-request 13384 may help).
I think there's a few things that could likely be cleaner in the message template as well; it's very verbose.  I would recommend making the link to details be over a single word (either a sentence "details" or probably better, having the ruleName be the link to the rule).  And the text about "abnormal metrics" feels like fluff as well.
Can you try revising that and then post an example of what the message looks like so we can discuss further?

Hi @timabbott !
Thank you for reaching out! This is the first time I made contribution to open source project and I definitely would like to make it better.

I look through the guide of git commit, but unfortunately did not came across the documentation you sent to me. I will definitely stick to the docs for future contribution
I will change the docs, sorry for that
I do changed several other SVG which does not belong to my integration since the ci test keep complaining that other files need compression and wont let me pass the test. so I optimized all of them

As for the message template, I will make another cleaner version. What kind of message example are you looking for here? The message template for notification in Zulip or native json data in grafana?

If you look at the examples from our other integrations, we try to avoid duplicate information or filler text, and just present the notification.

Would be great if you could include the alert image (if set) as well! See the imageUrl field.

I will make this changes tonight maybe, does not have the develop env machine with me today!
Is there any documentation concerning alert image? I dont know that it stand for.
Will remove duplication message definitely.
Thank you for the advice!

Is there any documentation concerning alert image? I dont know that it stand for.

See https://grafana.com/docs/alerting/notifications/#webhook and https://grafana.com/docs/installation/configuration/#external-image-storage. Basically Grafana takes a screenshot of the alerting graph and uploads it to an external host (eg. s3).
Thank you for your work! I'm looking forward to using your integration :)

No problem! I will definitely check those resources out this evening, hopefully I can make a new PR tonight!

Closing in favor of the active effort at #13491.


JQ behavior differs from RHEL-8

Hello,
Not sure if this is issue, or this is working as designed, but we noticed one big difference in jq behavior between Oracle Linux 8 and RHEL 8. Up to now, we were using RHEL 8, and we decided to evaluate Oracle Linux, and transition isn't smooth. In our code, we are assuming, that performing jq command on empty string will result with exit_code=0, whereas in OL8, it returns 4.
Simplest example:
[rhel-8]# echo "" | jq -e
[rhel-8]# echo $?
0
[ol-8]# echo "" | jq -e
[ol-8]# echo $?
4
What happend?
[ol-8]# echo "" | jq -e
[ol-8]# echo $?
4
What would we expect to happen?
[ol-8]# echo "" | jq -e
[ol-8]# echo $?
0

Thanks for identifying this issue.  I will open an internal ticket to have it looked at.

I can't duplicate this.  What version of jq are you running?
$ echo ""|jq -e;echo $?
0
$ rpm -q jq
jq-1.6-6.el8.x86_64

Rhel: jq-1.6-7.el8.x86_64
OL: jq-1.6-7.0.1.el8.x86_64 (the newest?)

Got it now!

It seems that the issue can be reproduced with "jq-1.6-7.0.1.el8.x86_64" but not with "jq-1.6-7.el8.x86_64".
Those examples are all executed on Oracle Linux 8:
[root@localhost ~]# rpm -qa |grep jq
jq-1.6-7.el8.x86_64
[root@localhost ~]# echo ""|jq -e;echo $?
0
[root@localhost ~]# rpm -qa |grep jq
jq-1.6-7.0.1.el8.x86_64
[root@localhost ~]# echo ""|jq -e;echo $?
4

It seems that the issue is due to the "jq -e" output.
With 1.6.7, the same specifies the "jq" release on the first line:
[root@localhost ~]# jq -e
jq - commandline JSON processor [version 1.6]
with <IP_ADDRESS>.1, the release is not showed on the first line:
[root@localhost ~]# jq -e
jq - commandline JSON processor [version -]

@lpankaz this is clearly a bug. we already opened an internal bug to get the same fixed.
Thanks for reporting.

This is fixed in version JQ-1.6-7.0.3.EL8_9, which should be released shortly.


PBR Fragment Shader Cleanup

Opening a PR for discussion on #44
Fairly hefty cleanup and reorganization attempt based on feedback in #44.

Moved unused shading factor implementations out into the readme.  With PBRInputs struct, these can be easily swapped in for comparison
Lots of code documentation, tracked down references mostly to the Epic Paper
Renamed some core functions, with influence from @javagl's implementation
Minor change/correction to calculation of diffuseColor.  Model lighting looks too dark and changed it to a more intuitive implemenation.  f0 is the reflection contribution at grazing angle, so the diffuse contribution is the inverse of that, and then adjusted by the inverse of the metallic.
Switched the geometricOcclusion to use the SmithG1 sub-function.  I also removed the additional sub-function and inlined the implementation within the geometricOcclusion function for clarity.  We were using the SmithG1_var2, but I could not track down a reference.

TODO:

[x] Track down reference for SmithG1_var2, select a preferred implementation
[ ] Cleanup Appendix, determine if this is even necessary or if we simply provide a single and simple reference implementation.

@toji @javagl Can both of you take a look at this to see how well it addresses your concerns?
/cc @moneimne @snagy Please review when you get a chance, but let's wait for feedback before we merge.

Of course, I won't even dare to claim the ability to review it from a really technical point of view. But from looking at the structure as a whole, it now seems to be well-documented and far more readable. So in this regard, this would be a 👍  from me

Left a few comments, but this definitely strikes me as an improvement to readability! Thank you!

Thanks to all reviewers!  @abwood I see you pushed an update, is this ready for final review?

I've hopefully addressed everyone's comments.  I pointed out two code changes in this PR that definitely need review.
In my opinion, the appendix needs some more work or we just get rid of it altogether.  The intent was to try to stay away from forcing us to endorse a single implementation.  The vision was that we had a place to provide more in depth background on why one implementation is chosen over another, possible offer up performance comparisons or screenshots showing image differences.  That's all possible there in README.md, but I'm hoping that is out of scope for this PR!  Would like some opinions on this, as it could be too big of a scope for this repository (are we a reference implementation or a tutorial).

I think the appendix is a great start.  Perhaps it just needs an introduction about how this is only a potential implementation, and we can add more information over time.
We also need to fill in the PBR 'example implementation' appendix in the actual glTF spec, and I think your appendix is a good start for that.  I'll try to review it more deeply next week and add some additional details.

If you've been comparing this branch to master and wondering what happened to the specular highlight on the front visor of DamagedHelmet, if fixed a bug introduced in this branch.  Updated to correct our diffuse() calculation based on @snagy's review.
All that is left is an intro for the Appendix unless there's any other input from the group here.

Sounds good to me.  I'll merge this and we can update the appendix separately.
Thanks for all the work here, @abwood !


Overwriting FS

I'm working on some project that overwrites all of the I/O functions, but the only thing I could not figure out is at what point do I intercept byte data that is coming from the user?

What do you mean?

I would like to pass bytes, received from user to my own function, that will write them into a file

Not sure what you mean.

Anyway, I figured it out. I'm creating a tempfile and writint user's data there and then mess with it how I desire. Sorry if my question was unclear, but It would be cool to have a byte buffer, that user writes into, and then it dumps to the file, so that people (me) could rewrite dump method the way they need. Currently user writes directly into file through fs.open and that's a little inconvenient, imo

If I understand your original question correctly:

I'm working on some project that overwrites all of the I/O functions, but the only thing I could not figure out is at what point do I intercept byte data that is coming from the user?
...you're (possibly) talking about client upload (from client to server). The way that's implemented is:

we read data from DTPHandler's socket (chunk = sock.recv())
the chunks of data are written to a file object (file.write(chunk))
the file object is previously returned by AbstractedFS.open method which simply is a wrapper on top of open() builtin

I imagine you want to do something else other than writing on disk, so what you want to do is write a custom file-like class which provides the same interface of open builtin and which provides the same methods (read(), write(), close(), seek(), tell(), etc.). Specifically, if you want to do something else other than writing on disk the central method you want to customize is write()·


Add example strings to sql constructor documentation

Is this PR related to a proposed Issue?
Proposed by me in the gitter.im channel. It was not clear to me what format the text constructors were expecting. Hopefully this will clarify it for future users.
What changes were proposed in this PR?
Add some example text input strings to the documentation of ST_FromText constructors. I also fixed one typo on line 176 -> 192.
How was this patch tested?
I install mkdocs, ran mkdocs serve, and verified that the documentation works as intended.
Did this PR include necessary documentation updates?
This is a documentation-only update.

@jiayuasu I discovered a small error, Line 19 should not have a comma separating the latitude and longitude. That is
SELECT ST_GeomFromWKT('POINT(40.7128,-74.0060)') AS geometry
should be
SELECT ST_GeomFromWKT('POINT(40.7128 -74.0060)') AS geometry
So sorry. Would like another pull request or can you change this easily?

@jordanperr No worries. I will fix that. Thanks for letting me know.


[semver:minor] Add junit_report input

This PR adds a junit_report input, so that you can build a JUnit XML report of the tests that were run.

we should add it to the datadog-ci command options list then.

@lefebvree I had a look and we actually don't have a command options list. We only have this "Global configuration file options" section, so that's why the --jUnitReport flag only exists in the Reporters section.


Update layers/meta-balena digest to 98b6efa

This PR contains the following updates:

Package
Update
Change

layers/meta-balena
digest
c06fe03 -> 98b6efa

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

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

This PR has been generated by Renovate Bot.

lgtm


feat: deal proposal: makeDealProposal accepts struct ContractDealProposal instead of bytes

Building off of https://github.com/filecoin-project/fevm-hardhat-kit/pull/87 update DealClient.sol to use solidity abi based types including those in the filecoin library instead of raw bytes.
see https://github.com/lotus-web3/client-contract/issues/13 for extra details

Superseded by https://github.com/filecoin-project/fevm-hardhat-kit/pull/88.


Remove old batman-adv

example from slybek
sylsbek ➜  ~ dkms status
batman-adv, 2021.2, 5.10.0-10-amd64, x86_64: built
batman-adv, 2021.2, 5.10.0-12-amd64, x86_64: built
batman-adv, 2021.2, 5.10.0-9-amd64, x86_64: built
batman-adv, 2021.3, 5.10.0-12-amd64, x86_64: built
batman-adv, 2021.3, 5.10.0-13-amd64, x86_64: built
batman-adv, 2022.0, 5.10.0-13-amd64, x86_64: built
batman-adv, 2022.0, 5.10.0-14-amd64, x86_64: built
batman-adv, 2022.1, 5.10.0-18-amd64, x86_64: built
batman-adv, 2022.1, 5.10.0-19-amd64, x86_64: installed (WARNING! Diff between built and installed module!)
batman-adv, 2022.2, 5.10.0-18-amd64, x86_64: installed
batman-adv, 2022.2, 5.10.0-19-amd64, x86_64: built

sylsbek ➜  ~ ls /usr/src
batman-adv-2021.2  linux-headers-5.10.0-18-amd64
batman-adv-2021.3  linux-headers-5.10.0-18-common
batman-adv-2022.0  linux-headers-5.10.0-19-amd64
batman-adv-2022.1  linux-headers-5.10.0-19-common
batman-adv-2022.2  linux-kbuild-5.10

([0-9]{4}\.[0-9]{1})

to find versions in dkms status
then compare to current version, call via ansible
remove via dkms remove
then remove directories in /usr/src

Cleanup for /opt implemented in<PHONE_NUMBER>58953b75ad64be379e3f61ca5a838f
Works easily in this case since the directories and archives with the current version are not needed and can be deleted after unpack and install

Solved with:
ac4eb322cf8deb957f92ac607c35ed541b46a15f
5b9a18184efe879eddd330bcd31fc5a4fe032f09
938e428eaf5117c3a541697e8c327358b05ecca9


markdown-mode and company-mode conflict

This only started happening after I updated packages recently, so we can bisect pretty easily.

Fixed temporarily since I downgraded markdown-mode. I'll re-open when we need to upgrade.


fix: Make tls email required if tls is enabled

Resolves  #22

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign chrismellard
You can assign the PR to them by writing /assign @chrismellard in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@chrismellard: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr-build
204b14d6a90d552c9b3847997dfa2967bf35834b
link
/test this

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

/retest

@chrismellard: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr-build
204b14d6a90d552c9b3847997dfa2967bf35834b
link
/test this

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

@chrismellard: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr-build
ec15907c4721c7d845db957528c0009057afd914
link
/test this

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

/test this

@chrismellard: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr-build
ec15907c4721c7d845db957528c0009057afd914
link
/test this

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


Bug with multiple selection of the 31st of current month and the next week of the next month

Version 7.1.5. iOS 10 ONLY

Steps to reproduce:

Select 31st of August
Select 8th of September
Select next month
Unselect 31st of August
Go back to previous month
-> None of 31st, 1st, 7th, and 8th cell are selectable
-> no delegates are getting called

I am using next settings:
        self.calendarView.allowsMultipleSelection = true
        self.calendarView.isRangeSelectionUsed = true

Temporary I've solved it by using next code when I go back to the previous month:
        self.calendarView.scrollToSegment(.previous, triggerScrollToDateDelegate: false, animateScroll: true, extraAddedOffset: 0.0, completionHandler: {
            self.calendarView.reloadData()
        })

Whole code is here: https://github.com/NadzeyaStsepanchuk/JTAppleCalendarTestExample

checkign this out now

What did you mean by

Temporary I've solved it by using next code when I go back to the previous month:

What is next code?

I am using iOS 11.4 and it is working correctly.
Based on the comment you used in your app ---> //FIX for iOS 10:
Are you saying you are only seeing this in iOS 10 ?

One thing i would say is this.
I notice you have implemented the shouldSelect function.
Are you saying that even the shouldSelect delegate is not getting called?

It's working fine on iOS 11, right. Bug is only reproducing on iOS 10.
Yep, shouldSelect is also not getting called.
You can test it using my code on git.

updated master branch.
Closing issue now.


improve obi stop pkill regex

This changes the pkill regex to use the calculated full path of the executable target. This commit depends on dff3734, because that commit removes relative paths from obi go

This change should address all of the concerns in #32:

won't accidentally kill important processes
works without dependencies
will hit debug runs too, though LLDB at least doesn't stop from SIGINT #80

As @EchoAbstract mentioned in slack, we need to preserve the ability for one user to stop the obi go of another user.

The bcbac5e commit injects a simple username-matching regex into the full path to the executable, when the username is part of the default remote path to the executable (/tmp/arice/porject). This makes the remote pkill match apps run by other users. I believe this still hits the other three requirements as well.

to use the calculated full path of the executable target

Why not just use the relative path from the project root directory?

I don't have a reason to prefer relative path over absolute, except that it was easier for me to code against the absolute path at the time.
I don't think killall would hit debug runs where the actual process is gdb or something.

I have updated this branch to be free of merge conflicts and to more closely resemble the code in #86.

I don't think killall would hit debug runs where the actual process is gdb or something.

It would kill the application but leave gdb/tmux running, that's true. That could be seen as a feature, though.

I'd like to personally test the heck out of this before merging to dev so I'm holding off until I have some time.


Update to new base image jdk8u202-b08_openj9-0.12.1.

Update to a new base image with security fixes and some updates.

Hi @mpmunasinghe, @rabbah ,
I added a CHANGELOG.md file and I updated this runtime to the most current version of the base image. The previous base image had several security issues (CVE-2019-3462, CVE-2018-16890, CVE-2019-3822, CVE-2019-3823, CVE-2018-16839, etc.). Most of them are related to the packages apt, curl, libcurl3-gnutls, libssl1.0.0, libsystemd0 and systemd.
They are fixed with the new version.
Can someone of you or maybe another committer do a review to see if did all correct :-).
Thanks in advance.
Regards, Falk.


shuffle stacked loss

Consider replacing
tf.random.shuffle(loss)
with
loss = tf.gather(loss, tf.random.shuffle(tf.range(tf.shape(loss)[0])))

Hi @cfifty , May I ask why not replacing with loss=tf.random.shuffle(loss)?

In non-eager mode, tf.random.shuffle(loss) is never called, so the loss list is not shuffled if you use graph mode TensorFlow.

If you use loss=tf.random.shuffle(loss), the backwards pass of tf.random.shuffle is not defined. Thus, you can't compute gradients through this operation and an error is thrown. See https://stackoverflow.com/questions/55701407/how-to-shuffle-tensor-in-tensorflow-errorno-gradient-defined-for-operation-ra for additional context.

Thank you very much for your detailed explanation!

The loss list is not shuffled if using tf.random.shuffle(loss). For the reason, I think, tf.random.shuffle is not an inplace operation, and thus the input argument loss is not shuffled.
It seems loss=tf.random.shuffle(loss) do not throw an error with tf 1.15.3. Maybe in the new version, the gradient for this operation is registered.
Overall, I think loss = tf.gather(loss, tf.random.shuffle(tf.range(tf.shape(loss)[0]))) is a greater choice for compatibility,

I am sorry that I made a mistake, the gradient operation is still not defined for loss=tf.random.shuffle(loss) in tf.15.3.

We should consider use loss = tf.gather(loss, tf.random.shuffle(tf.range(tf.shape(loss)[0]))) instead.


Fix filtering on boolean columns, broken with SQLAlchemy 1.2 upgrade

SQLAlchemy 1.2 does not allow comparing string values to boolean
columns. This caused errors like:
sqlalchemy.exc.StatementError: (builtins.TypeError) Not a boolean value: 'true'

For more details see http://docs.sqlalchemy.org/en/latest/changelog/migration_12.html#boolean-datatype-now-enforces-strict-true-false-none-values
But in all honesty, the filtering logic is quite ugly overall and deserves to be rewritten.
I'm not too proud of my "fix" either.

Thanks for the PR Marti. It looks like a test is failing, can you take a look?

@castrapel Doh, fixed now. I had reverted one line to check that my unit test was testing the right thing and forgot to restore it. :)

Coverage increased (+0.1%) to 60.817% when pulling 8e2b2123f1dd80aaba978eb671a8650956e3272b on intgr:fix-boolean-filter into b4b9a913b31a9359de990b59bd09ed318550a256 on Netflix:master.

Thanks! Looks good to me.


feat(api): add functions list command

What kind of change does this PR introduce?
Added functions list subcommand. Also did some small refactors.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

30 of 33 (90.91%)  changed or added relevant lines in 5 files are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.05%) to 61.8%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/functions.go
2
5
40.0%

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
7
7.94%

Totals

Change from base Build<PHONE_NUMBER>:
-0.05%

Covered Lines:
4745

Relevant Lines:
7678

💛  - Coveralls


added intune module updates

Regarding the Intune module:

Added these intune assets: pipeline template, test data, and notebook
Only updated the "notebook" and "pipeline" readme to include up-to-date information

Still to be done for Intune module:

Finalize landing page readme
All other readmes
Rubric update
Add tutorial doc
Double-check/Update all other internal readme links

Reviewed and approved.


Fix typo in ValueBuilder test

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/.
Fix typo in yaml format name

recheck

The PR looks good! Please add the magic CLA agreement phrase as a separate comment (the CLA check bot is not very smart)

Also, please apply the following formatting change
    --- taxi/uservices/userver/shared/src/formats/common/value_builder_test.cpp  (index)
    +++ taxi/uservices/userver/shared/src/formats/common/value_builder_test.cpp  (working tree)
    @@ -14,8 +14,7 @@ struct MyKeyValue {

     // The function must be declared in the namespace of your type
     template <class Value>
    -Value Serialize(const MyKeyValue& data,
    -                formats::serialize::To<Value>) {
    +Value Serialize(const MyKeyValue& data, formats::serialize::To<Value>) {
       typename Value::Builder builder;

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/

Many thanks for the PR!


OIDC/Auth2 integration

This PR contains a POC for a OIDC/oauth2 integration into OpenSlides. The dev setup is based on keycloak.

Or move the auth code to another repo. For example the openslides-auth-repo


Update lxc-librenms.tf

Added memory and core count requirements to match min specs for LibreNMS

Closing in favor of 1bdf8a344488851769ad3c9fd5b8e75f5c667112


Convert src/APIs to TS

Is your feature request related to a problem? Please describe.
Convert src/APIs to TS
Describe the solution you'd like
Convert src/APIs to TS
Avoid making logical changes to these files. These files may need new types or interfaces, so add them whenever necessary. Prettier will format these files which is okay.
Describe alternatives you've considered
N/A
Additional context
N/A

Can I work on this issue?

Can I also work on this @ijemmao

Can I also work on this @ijemmao

Yes @zanio

This ticket has been addressed! Thanks for calling this out @zanio


Infinite loading with arcane fighter wizard in group

I have an Arcane Fighter Wizard in my group. After the update when I try to load the save game it says one of the characters is corrupted and is stuck at loading.

Please always read change log before an update. RESPEC was required before update on Arcane Fighter.

I'm sorry, does it mean i need to basically reset the char before i load the game? (i've read the changelog)


Using original names for images in group, community, and profile pages creates filename collision problem.

Description of the bug
Group, community, and profile images are uploaded with their original filenames. If two users upload images with the same name, they will collide.
Expected behavior
Image filenames should be made unique through hashing.  This is a trivial patch.

@alvacouch I verified this has been fixed on production, so I am closing it, but feel free if you notice any further issues.


Adding AMIUpdater

Overview
This PR Adds AMIUpdater. A README is forthcoming in a future PR.

This PR is superseded by https://github.com/aws-quickstart/taskcat/pull/274

this looks amazing @andrew-glenn, but pretty large, going to need some time for review

what I did to get to the test failures:
git clone git@github.com:andrew-glenn/taskcat.git
cd taskcat
git checkout develop
python setup.py test

I would consider modifying the unittests to not be dependent on valid credentials.
Add a test to this sections  functional tests( the functional tests will only run as part of PR validation from with in travis env). This change allow community developer to run pytest with no dependency on cloud access.
With that said the PR look awesome! Nice work!

Rest in peace the Alchemist. Long live the AMIcorn!

@jaymccon  - Assuming you approve, please DO NOT merge; I want to squash everything down before this hits develop.

LGTM, great stuff @andrew-glenn !


batch mode requests_*.jsonl files doesn't follow GenericRequest data schema

class GenericRequest(BaseModel):
    model: str
    messages: List[Dict[str, Any]]
    row: Dict[str, Any] | str
    row_idx: int
    response_format: Optional[Type[BaseModel]] = None

while in batch=True requests_0.jsonl (which includes in responses_0.jsonl) is
(base) ➜  a2a86c77ae470e3f head requests_0.jsonl -n 1 | jq
{
  "custom_id": "0",
  "method": "POST",
  "url": "/v1/chat/completions",
  "body": {
    "model": "gpt-4o-mini",
    "messages": [
      {
        "role": "user",
        "content": "Write a very long, elaborate, descriptive and detailed shooting script, including a background and dialogues, for a John Waters-inspired film scene that includes a character preparing omelettes. You are free to choose the setting, scenario (it should make sense) and characters (give them names, and describe their appearance and clothing in detail)."
      }
    ]
  }
}

causing issues when access messages fields when doing batch=True frontend

It should be writing generic responses:
https://github.com/bespokelabsai/curator/blob/main/src/bespokelabs/curator/request_processor/openai_batch_request_processor.py#L380-L391


Source code for StaticSitesClient ?

Is the source-code from the StaticSitesClient (which is used in docker image mcr.microsoft.com/appsvc/staticappsclient:stable) public?
And where to find this?

@StefH This component is currently not open source.

Will it be when it's ready to use in DevOps pipeline?

The task is rolling out. You'll be able to create a Static Web App in the portal without GitHub sometime in March. It'll depend on the portal deployment schedule.

Wil that also mean that Azure Static WebApps itself will be out of preview around that time?
And may I ask why the GitHub action was available earlier?

We still have more to do before we can GA.
As for why GitHub Actions was supported first, we had to pick one for the public preview and felt that GitHub Actions allows us to reach more people in our target audience (some of whom are not Azure customers) which, in turn, has allowed us to collect the feedback needed to shape our product.
Closing this issue, but keep an eye on #5 for Azure DevOps support.

Instead of using the limited StaticSitesClient docker container I went over to use the SWA CLI to deploy an Azure Static Web App, because I can also easily deploy to specific (preview) environments.

@bkarstaedt Thanks for sharing about the "SWA CLI".
I had to follow on the same path as I'm using Bun instead of NPM which causes all kinds of trouble.
Here's a snippet of my github action:
jobs:
  build_and_deploy_static_site:
    runs-on: ubuntu-latest
    environment:
      name: dev
      url: ${{ vars.DOMAIN }}
    name: Deploy to Dev Environment
    steps:
      - uses: actions/checkout@v4
      - uses: oven-sh/setup-bun@v1
        with:
          bun-version: latest
      - name: Install dependencies
        run: bun install
      - name: Generate static site
        run: bun run generate
      - name: Install Azure Static Web Apps CLI
        run: bun install -g @azure/static-web-apps-cli
      - name: Deploy to Azure Static Web Apps
        run: swa deploy dist --deployment-token ${{ secrets.AZURE_STATIC_SITE_TOKEN }} --env production


Add equality test on Redis client and conn pool

Currently, there's no __eq__() method defined on the Redis or
ConnectionPool classes.  Therefore, no two instances of either of
these classes will ever be equal.
I have a use case where it would be nice for one Redis client instance
to be equal to another if they have the same connection kwargs; that is,
if they're connected to the same Redis database.
Pull Request check-list
Please make sure to review and check all of these items:

[x] Does $ python setup.py test pass with this change (including linting)?
[x] Does travis tests pass with this change (enable it first in your forked repo and wait for the travis build to finish)?
[x] Is the new or changed code fully tested?
[ ] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

NOTE: these things are not required to open a PR and can be done
afterwards / while the PR is open.
Description of change
Add an __eq__() method on the Redis and ConnectionPool classes that return true if they have the same connection kwargs; that is, if they're connected to the same Redis database.

Codecov Report

Merging #1240 into master will increase coverage by 0.05%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #1240      +/-   ##
=========================================
+ Coverage   92.34%   92.4%   +0.05%
=========================================
  Files          20      21       +1
  Lines        5958    6003      +45
=========================================
+ Hits         5502    5547      +45
  Misses        456     456

Impacted Files
Coverage Δ

redis/connection.py
82.38% <100%> (+0.04%)
:arrow_up:

tests/test_connection_pool.py
97.73% <100%> (+0.13%)
:arrow_up:

redis/client.py
85.32% <100%> (+0.01%)
:arrow_up:

tests/test_client.py
100% <100%> (ø)

Continue to review full report at Codecov.

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

Looks good to me, thanks!

@brainx After thinking about this a bit more, I now believe this isn't the right way to go. Just because two connection pools share the same configuration doesn't make them identical. Clearly they manage a separate set of connections (to the same server). It seems wrong to say pool_a == pool_b when pool_a will never give me one of pool_b's connections or when I can call pool_a.disconnect() but pool_b's connections are still active.
I'd tentatively be OK with saying client_a == client_b if they share the same connection pool instance. I'm not sure if that helps you at all.
I'm going to revert this change for now. If you really need this functionality it's trivial to compare r1.connection_pool.connection_kwargs == r2.connection_pool.connection_kwargs.

@andymccurdy - Thanks for the heads up!  That works for me.


Encountering Webcam Detection Issue on Ubuntu 22.04

Hello,
I've completed the installation process on Ubuntu 22.04. However, when I attempt to run the application, no webcam appears in the selectable device list, indicating that the webcam is not initializing or being recognized.

hi @Limbicnation , I've responded here, please let me know if the SSL would work for you. thanks
https://github.com/radames/Real-Time-Latent-Consistency-Model/issues/10#issuecomment-1806926430

Thanks for the explanation, @radames . This worked perfectly!


Use new dropdown style search box in giphy and emoji popoves

discussion: https://chat.zulip.org/#narrow/stream/101-design/topic/migrating.20giphy.2Femoji.20popovers.20to.20the.20new.20dropdown.20style
If someone else doens't pick it up, I will do it before release.

Hello @amanagr, I'm eager to help resolve this issue. Although I'm new to this repository, could you please guide me on the steps or approach I should take to tackle this problem? Additionally, please assign this issue to me.

@Moksh45 you can claim the issue by commenting "a zulipbot claim"

Thank you, @sayyedarib  I appreciate your guidance on claiming the issue in Zulip. I'll be sure to follow your instructions and comment "a zulipbot claim" to claim the issue.

a zulipbot claim

@zulipbot claim

@zulipbot claim

@zulipbot claim

@amanagr The pr is ready to be reviewed, let me know what you think.

@zulipbot claim


Z[++NEW*S.E.X-VIDEOs++]~ Elif Karaarslan Snimka Video Viral Leaked 2024 LINK Viral on Social Media X Today [WA26KH]

12 sec ago, Elif Karaarslan Snimka, ZFULL Elif Karaarslan Snimka Leaks Video On Social Media X Twitter[WA26KH]

Elif Karaarslan Snimka latest video has taken social media by storm, captivating viewers and sparking widespread discussions. This young digital creator has garnered significant attention for her engaging content, particularly with her recent Elif Karaarslan Snimka-themed video that has gone viral. Fans are eagerly searching for where to watch the full video, turning to platforms like Instagram, X (formerly Twitter), and Discord for access.
As the buzz around Elif Karaarslan Snimka grows, fans are actively sharing their reactions online. The viral success of her video highlights the powerful role of social media in propelling creators into the spotlight. With her unique style and creativity, Elif Karaarslan Snimka has quickly become a notable figure in the digital landscape, leaving many eager to learn about her future projects and content.
...
....
....
...
...
...
....
....
...
...
....
...
...
....
....
...
...
....
...
...
....
....
...
...
....
...
...
....
....
...
...
....
...
...
....
....
...
...
....
...
....
...
[

xzcs


⚠ SceneTime has degraded performance

In a7f3883, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1574 ms

Resolved: SceneTime performance has improved in b665b10 after 19 minutes.


Fix for consumption chart which flickers as well as legend state being lost.

On clicking a legend, the respective dataset in the chart is either hidden or shown. Make sure that this happens and the state of all the legends is maintained when clicking "Show full chart resolution".
Make sure the grid line on the background is always visible as long as a dataset is visible.
On changing date and clicking enter, close the datetime dropdown

@RohanAntony : please put me in the loop for reviewing that PR.
@cabch : Caro, I've just tested that one, and we have vertical scrollbars when viewing the curves. Can you help here as well?

updated with master and fixed dialog size @RohanAntony please tell me if you can see the dialog correctly without scroll bar nor big white space below the chart (you can try resizing your window manuelly, size should follow resizing)
@ClaudeROSSI : sounds fixed for my part

updated with master and fixed dialog size @RohanAntony please tell me if you can see the dialog correctly without scroll bar nor big white space below the chart (you can try resizing your window manuelly, size should follow resizing) @ClaudeROSSI : sounds fixed for my part

@cabch : I still have huge charts

updated with master and fixed dialog size @RohanAntony please tell me if you can see the dialog correctly without scroll bar nor big white space below the chart (you can try resizing your window manuelly, size should follow resizing) @ClaudeROSSI : sounds fixed for my part

@cabch : I still have huge charts

On line 539 in consumption-chart.component.ts, if you set "maintainAspectRatio: false", the chart is set to initial size but the flickering exists.

The flickering issue was due to the css value being set in % instead of vw or vh for the parent container. Also the default parameter for maintainAspectRatio in every chart is true which needs to be set to false inorder to fit the parent container. The size of parent container is set in _charts.scss file.
I have removed the default additional classes that were added to transaction.component.html. If we need this, we need to add custom media queries in the css file to handle when dialog is smaller else the chart overflows.

Let's sync on this.


Bin Gone WRong

JS Bin all the sudden started doing this ..... http://prntscr.com/791sen
410: This bin has gone
This bin has gone and there's no alternative that I'll redirect you to. It's quite possible this bin has been blocked from appearing due to abuse content, so really, I'm just doing you a favour.
what is going on guys? =(

+1 for this issue.

Sorry, this was fixed this morning. Something went wrong after Anonymous decided to use jsbin to make a statement on the web :-\


windows adding CONTEXT type for x86_64 arch.

r? @JohnTitor
(rust-highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit 6d09f2c9016c27f8207c6fc386c703dfaa9ed639 has been approved by JohnTitor

:hourglass: Testing commit 6d09f2c9016c27f8207c6fc386c703dfaa9ed639 with merge d7e8f361c7add23861e72462c516365116733281...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing d7e8f361c7add23861e72462c516365116733281 to master...


Add partner integrations to product list

Purpose of this pull request
This pull request adds partner integrations to the the product list articles from the updates found in PR #4597
Select the type of change

[ ] Minor Changes - Typos, formatting, slight revisions
[x] Update Content - Revisions, updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)
See PR #4597

we can release it, since these are older already published apps.


[FEATURE] Remove uses of old JsonWriter

SanyParserPass still uses the old JsonWriter to output out-parse.json. It should use the new implementation with the standardized JSON encoding (see #595).

Note: Reading from a JSON, with JsonReader breaks on typechecking in the current implementation. For scoping reasons, this change just replaces JsonReader calls with JsonToTla, with the UntypedReader, which does not recover types either. Therefore, until  #780 is resolved, reading JSON is not expected to work.


Remove requirement for C code to use -DIREP_LANG_C

IREP WKT headers have traditionally required some IREP_LANG_* macro to be defined in order for them to pass through the C preprocessor without error. But if you look at the use cases, IREP_LANG_C shouldn't really be needed:

IREP_LANG_FORTRAN: used by irep-generate to generate nearly-complete Fortran code from a WKT header. irep-generate finishes the translation and outputs the completed Fortran module.

IREP_LANG_LUA: used by irep-generate to generate sort-of-complete Lua code from a WKT header. irep-generate finishes the translation and outputs the completed Lua file.

IREP_LANG_GENERATE: used by irep-generate to generate easy-to-parse symbosl that are then used by irep-generate to create table indices.

IREP_LANG_C: has to be defined when compiling any C or C++ code that includes a WKT header. Causes WKT headers to be translated to valid C.

The first three cases are used by tooling, not by client code. It makes sense that the tools might have to fiddle with defines to get the right output. But IREP_LANG_C is used by client code, and there isn't a reason you'd include a WKT header in C code unless you wanted IREP to generate C.
wkt.mk and irep-config.cmake handle defining this macro when building WKT libraries and indices, but codes that include the IREP headers have to do extra work, and this can be a stumbling block for new IREP users.
This PR removes the extra step for C code. IREP_LANG_C is no longer required, and IREP headers included with no IREP_LANG_* definition are assumed to be C.

[x] Make IREP_LANG_C the default in ir_macros.h.
[x] Get rid of error message when no IREP_LANG_* is defined.
[x] Remove IREP_LANG_C from docs and examples.

@stephdempsey FYI

@lee-busby @kennyweiss @tomstitt: what do you think of this change?

ok with me.


⚠ GitLab Server has degraded performance

In 040eb89, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11562 ms

Resolved: GitLab Server performance has improved in d091fab.


Updated docs

Signed-off-by: Michael Mattsson<EMAIL_ADDRESS>

Does the Helm chart need to be rebuilt with the values.yaml change?


articles/azure-monitor/logs/tutorial-logs-ingestion-api.md: corrected a minor mistake

I noticed a minor mistake in the link and text of the Note. Corrected them.

@bwren
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Sometime Debug.WriteLine is ignored.

It happened rarely but see more on 0.8.0

The bug found on FEZCLR

This issue is very hard to reproduce for now. The commit  #183 also try to fix this issue for now.


Option to disable "Allow access requests" with PnP Provisioning Schema

Is it possible to implement another option besides just "RequestAccessEmail" in the provisioning schema?
Some SharePoint Online groups needs specialized custom security whereas external users are invited in each group site.
If the owners of that site does not understand the severity of external users requesting access to files and folders they should not have access to and accepts the request. This is not wanted.
I know that this is a quickfix by doing it manually in the site permission panel.
When there is quite the amount of templates being provisioned, that small amount of manual work could quickly escalate to a lot of unwanted and unneccassary work that could be automated.
My wish is an additional option to add "AllowAccessRequest" (true or false) to the provisioning schema to be implemented in "pnp:WebSettings".

https://github.com/pnp/pnpframework/issues/118

https://github.com/pnp/pnpframework/issues/118

This would be really nice!


Updated the Contributors Section

#11 I have updated the Contributors section showing a badge of the no. of contributions in the current repo. Thank You! Hope it helps.

@all-contributors please add @parthib2004  for content


Line endings and file resources

Description
The way we handle line endings is mildly terrible and inconsistent and confusing.  When we upload cookbooks we use 'binmode' so we get no conversion of line endings.  When we download cookbooks MOST of the file resources will use binmode so that users will get whatever they uploaded.  However the template resource will do line ending conversion so will behave like binmode is false.
Chef Version
11.8.0-ish through 12.7.2
Issues

Sometimes users push files to Windows hosts which are then consumed ultimately on Linux hosts (the windows host may be the TFTP server or whatever for $REASONS).
Sometimes users push files to Linux hosts which are then consumed ultimately on Windows hosts.
When we weren't using binmode on file resources people would push binaries through cookbook_file or remote_file resources which would then be mangled by binmode settings.
Users that aren't using the template resource get the line ending of whatever was uploaded -- so you can't edit Windows cookbook_file resources on Linux hosts and vice-versa
Does anyone want to edit on Linux, push to Linux via chef but to deploy with Windows line endings?  (and vice versa)

Proposal One

Add a binmode flag on all the file resources.
Most file resources will default to binmode false, while template resources will default to binmode true.  This is necessary for backcompat and is in many ways arguably 'correct'.
The 'binmode true' flag on cookbook_file/etc resources will mean the line endings will convert to agree with the target O/S
The 'binmode' on the template resource will need to work around erubis converting everything to windows line ending.

This has at least one issue that the ability to edit on linux, push to linux, but consume windows line endings becomes a lot more difficult.  You could still save the file as windows on the linux workstation and upload it and then use 'binmode true' though.  But its not 'delightful'.
Proposal Two

Add a line_ending property (note: awkward) that is quadruple valued:
nil means no conversion (binmode = true, and this would be the default for cookbook_file, etc)
:target means to convert to the target host (binmode = false, and this would be the default for template resource)
:windows means to convert to windows, even if you're on a linux host
:unix means to convert to unix endings, even if we're running on windows

The benefit of this is its much more clearly descriptive, and would allow writing a file resource which would deploy windows line endings to a linux host -- without having to both set binmode to true and to make damn certain to upload the file to the chef server with windows line endings.
Note
We treat files and templates differently by default right now.  I'm not sure we should fix this, however.  When someone wants to deploy a binary they will never use a template and will always use a cookbook_file.  They will never want line ending conversion.  The default for cookbook_file gets this use case correct.
On the other hand templates are never going to be binaries, and by default you almost always want the line endings to match the target host -- otherwise if someone edits the /etc/resolv.conf on a windows host and uploads it you blow it out with windows line endings, which is what you never want.
Of course cookbook_files that are text files are treated like binaries by default, while cookbook_files are still more often text files than binaries[*].  I guess I could see the argument that both cookbook_files and template files should be considered text files by default and should get the target O/S line endings by default -- but that requires everyone pushing out a binary cookbook_file to have a flag day and drop 'binmode true' onto their resources.
The way I proposed solving this in https://tickets.opscode.com/browse/CHEF-4746 was to inspect the file and determine if it looked like a binary or not and then set binmode based on if the first X thousand bytes of the file looked binary or text.
Either way if we pick proposal #1 or #2 we'd need to think this issue through as well.  There are three sub-options here:

Templates default to text files, cookbook_files default to binaries (this would not be a breaking change, and i'd assumed we'd do one of the proposals above and implement it with this first)
Templates and cookbook_files default to text files, all binaries need to be denoted as such on the resource (this would be hella breaking change)
Magical binary detection nirvana with the option to override in the edge case we guess wrong (this might not be a breaking change except in edge conditions)

[*] binaries are always cookbook_files, cookbook_files are predominantly text files

newline might be a better name for the property in proposal two than line_ending since its shorter, doesn't have the underscore and matches the title of https://en.wikipedia.org/wiki/Newline (which is what pops up when i type line ending into google)

supersedes #1718 and #2466

I'd love to have an update to this. This makes it impossible for us to use chef on windows since we are compiling the cookbooks before using them in Vagrant. It's sad to see statements like "We’re declining to move forward with changes to CRLF handling at this time. By closing this idea, we’re releasing the votes attached to it for use on other product ideas." as found here . Also I vote for your second proposal. Seems like this would be the best way to handle it.

update is that i haven't had time to work on it and other stuff is taking priority...

Thanks for the feedback. Do you happen to have pointers as to where or at which steps one might have to look?

Story description doesn't mention windows-native developers (I used to develop chef cookbooks on windows and use windows machines for the cookbook deployment pipelines) which sounds similar to @stiller-leser environment. I like the readability of proposal 2 (developer sets intended mode by name).


Install issue (Amazon Linux AMI / Elastic Beanstalk)

I'm getting an installation error from node-sass which is one of laravel-elixir's dependencies:
><EMAIL_ADDRESS>install /var/app/ondeck/node_modules/laravel-elixir/node_modules/gulp-sass/node_modules/node-sass
> node scripts/install.js

npm ERR! Linux 3.14.20-20.44.amzn1.x86_64
npm ERR! argv "node" "/usr/bin/npm" "install"
npm ERR! node v0.10.33
npm ERR! npm  v2.7.3
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn

npm ERR<EMAIL_ADDRESS>install: `node scripts/install.js`
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node scripts/install.js'.
npm ERR! This is most likely a problem with the node-sass package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node scripts/install.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls node-sass
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /var/app/ondeck/npm-debug.log

Extract from NPM Debug (let me know if you need the entire file):
https://gist.github.com/RTC1/62438cb9157e52f8b59a/raw/gistfile1.txt

Hi, thanks for the report.
I think it is failing when try to download the binary. To verify, please try npm install<EMAIL_ADDRESS>on same VM and check if it is still broken.
Aside: you are having a very old version of node.js v0.10.33 (current stable version is v0.12.1 and latest in 0.10 branch is v0.10.38: https://nodejs.org/dist/) with the latest version of npm (https://github.com/npm/npm/releases/tag/v2.7.3). That might be causing the underlying issue, but I am unsure at this point.

I ran that command and it worked fine - it only seems to error when npm install is being ran by the build process (however it installs many other packages ok).
The NPM package was updated following a suggestion that it may fix the issue. Unfortunately NodeJS cannot be updated as Amazon have yet to release a new Linux AMI for Elastic Beanstalk PHP.

Rebuilt the environment with previous version of NPM:
><EMAIL_ADDRESS>install /var/app/ondeck/node_modules/laravel-elixir/node_modules/gulp-sass/node_modules/node-sass
> node scripts/install.js

npm ERR! Linux 3.14.20-20.44.amzn1.x86_64
npm ERR! argv "node" "/usr/bin/npm" "install"
npm ERR! node v0.10.33
npm ERR! npm  v2.7.3
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn

npm ERR<EMAIL_ADDRESS>install: `node scripts/install.js`
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node scripts/install.js'.
npm ERR! This is most likely a problem with the node-sass package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node scripts/install.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls node-sass
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /var/app/ondeck/npm-debug.log

With older version:
``

<EMAIL_ADDRESS>install /var/app/ondeck/node_modules/laravel-elixir/node_modules/gulp-sass/node_modules/node-sass
node scripts/install.js

npm ERR! Error: spawn ENOENT
npm ERR!     at errnoException (child_process.js:1001:11)
npm ERR!     at Process.ChildProcess._handle.onexit (child_process.js:792:34)
npm ERR! If you need help, you may report this log at:
npm ERR!     http://github.com/isaacs/npm/issues
npm ERR! or email it to:
npm ERR<EMAIL_ADDRESS>npm ERR! System Linux 3.14.20-20.44.amzn1.x86_64
npm ERR! command "node" "/usr/bin/npm" "install"
npm ERR! cwd /var/app/ondeck
npm ERR! node -v v0.10.33
npm ERR! npm -v 1.3.6
npm ERR! syscall spawn
npm ERR! code ENOENT
npm ERR! errno ENOENT
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /var/app/ondeck/npm-debug.log
npm ERR! not ok code 0

Is it worth trying different versions of gulp-sass?

Please try with node-sass v2.1.1.

Here's the output from 2.1.1:
https://gist.githubusercontent.com/RTC1/ab623c65d69fe27f26b1/raw/gistfile1.txt

I fixed this by adding the HOME & PATH environment variables when calling NPM install (.ebextensions). Not sure why it only effected this plugin.
    "10-install-npm-packages": {
      "env": {
        "HOME": "/root",
        "PATH": "/usr/bin:/usr/lib/node_modules:/sbin:/bin:/usr/sbin:/opt/aws/bin"
      },
      "command": "npm install"

@RTC1 where does that json go?

That goes in the .ebextensions folder if you're using AWS Elastic Beanstalk. If you're not using EB but encountering the above error, just make sure those environment variables are set.

I have a .config and config.yml in those folders, can they be json?


chore: restructure integration tests

Description

Problem*

Resolves
Summary*
This PR reorganizes the integration tests to a structure to be ready to integrate hardhat. I've taken the opportunity to break out helpful functions to utils files.
Documentation

[ ] This PR requires documentation updates when merged.

[ ] I will submit a noir-lang/docs PR.

[ ] I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Ah, whoops forgot to update the circuit paths. One sec


ARROW-8511: [Release] In verify-release-candidate.bat, exit when CMake build fails, use Unity build

The unity build makes things faster, but it also hides the error reported in ARROW-8510. This at least allows the release verification to pass for 0.17.0

+1


using POST to create new user

The library seems to hang when I try to create a new user. Im simply doing
graph.post('/v1.0/users', req.body, (err: any, response: any) => {
        if (!err) {
          res.json(response);
          return next();
        }
});

Where req.body is
{
  "accountEnabled": true,
  "displayName": "Alex Wu",
  "mailNickname": "AlexW",
  "passwordProfile": {
    "password": "Test1234",
    "forceChangePasswordNextLogin": false
  },
  "userPrincipalName"<EMAIL_ADDRESS>}

Anyone else having this issue?
Looking at this for documentation.

Please try forceChangePasswordNextSignIn instead of forceChangePasswordNextLogin.


How I can remove the current selection back to default value or placeholder?

Hi, not sure if it is a bug or a new feature.
Just want to know how I can remove the current selection back to the default value or the placeholder?
Im using the select to select different values in a report filter and I want to give the option to final users to remove all the filters. So I need to remove the selected value on the select lists.
Thanks

+1

Please 👍 and follow #1449.


Adding ability to edit itinerary

Fixes #1331
This is an initial quick pass to enable editing an itinerary. The main purpose is to allow a start/end address to be set for an itinerary to the be used in my work for issue #1102
I'll continue with that issue once this is merged.
Note: I've not included unit tests for the edit itinerary handler yet. I'll open a follow up issue for that.
The validation also needs tweaking to validate the addresses as well. Again, I'll follow that up in a new issue.

@stevejgordon, are we allowing the editing of an Itinerary's date?

@mgmccarthy Currently, yes. This was mostly put in to manage the addresses though.
The editing of the date may have some impact on the notifications I guess. I can remove it from this PR. It does feel reasonable that someone may need to adjust an itinerary date but we'd need to decide on handling the change.
Also, there's not specific validation on this yet. It's a bit quick and dirty to enable the work on the route optimisation.

@stevegordon. Don't remove it until I have a chance to look at the scheduling/hangfire stuff. There might be a way to handle it in the job code.

@stevejgordon, to me, should we have the ability to edit an Itinerary's date is more of a business question than a technical concern, but by allowing the editing of an Itinerary's date at any point in the lifetime of the Itinerary will force us to write more code which will create complexity.
From a technical point of view, I have two concerns that add complication to the system:

Hangfire jobs "in-flight": since we schedule Hangfire jobs based on the Itinerary's date, changing the date could potentially lead to sms messages that are either not sent, or sent on the wrong date.
Itineraries that have had requestor's confirm their request means they confirmed their request based on the date of the Itinerary that the Request was added to. If we were to change the Itinerary date after a requestor(s) confirmed their request with us, now we need to go through a look-up process, and re-send more confirmation messages to requestors notifying them that the date of the Itinerary has changed and they need to confirm the new date with us.

Because we'd have to add code to address these two issues above, which can create more complexity, here is a list of what we could do in order of easiest to hardest.

once an Itinerary has been changed, do not allow it's date to be edited
once an Itinerary has one Request assigned to it (regardless of the source of the Request), do not allow the date to be edited
allow the date to be edited at any point once an Itinerary has been created (I don't think we should choose this option for the reasons I outlined above).

@tonysurma @MisterJames @dpaquette what are your thoughts?

Let's remove editing date to make simplicity a winner.  We can open an issue on feature backlog for date editing and see if it ever gets prioritized and then we're can deal with the complexity noted above.
When you remove that @stevejgordon we can merge this

@tonysurma, had a quick chat with @stevejgordon this morning about this and it sounds like we're all on the same page. Thanks.

@stevejgordon, let me know if I can help in any way

@tonysurma All done and ready to go


archive/tar: FileInfoHeader cannot parse Windows directory symlinks

On Windows, a symlink to a directory is reported as a directory and a symlink at the same time.  This confuses tar.FileInfoHeader, which reports the object as a directory, whereas on UNIX systems it would be reported as a symlink.  I believe it makes much more sense to report a symlink.
What version of Go are you using (go version)?
$ go version
go version go1.7.1 windows/amd64

What operating system and processor architecture are you using (go env)?
$ go env
set GOARCH=amd64
set GOBIN=
set GOEXE=.exe
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=x:\go
set GORACE=
set GOROOT=C:\Go
set GOTOOLDIR=C:\Go\pkg\tool\windows_amd64
set CC=gcc
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0
set CXX=g++
set CGO_ENABLED=1

What did you do?
Run the following program:
package main

import (
	"archive/tar"
	"fmt"
	"io/ioutil"
	"os"
	"path/filepath"
)

func main() {
	tempdir, err := ioutil.TempDir("", "")
	if err != nil {
		panic(err)
	}
	defer os.RemoveAll(tempdir)

	os.Symlink(`c:\`, filepath.Join(tempdir, "directorysymlink"))
	// os.Symlink("/", filepath.Join(tempdir, "directorysymlink"))

	fi, err := os.Lstat(filepath.Join(tempdir, "directorysymlink"))
	if err != nil {
		panic(err)
	}
	hdr, err := tar.FileInfoHeader(fi, "directorysymlink")
	if err != nil {
		panic(err)
	}

	fmt.Printf("Typeflag: %c, Mode: 0x%x, Linkname: %q\n", hdr.Typeflag, hdr.Mode, hdr.Linkname)
}

What did you expect to see?
On Windows, this returns Typeflag: 5, Mode: 0x41ff, Linkname: ""
What did you see instead?
I'd expect to see Typeflag: 2, Mode: 0xa1ff, Linkname: "directorysymlink", as is reported on non-Windows systems.

I have a windows machine. I can look into this.


Facilitate installation on tablets without root access [20-40h]

[ ] Download apps
[ ] Install apps
[ ] Update apps

Cancel APK download in progress if the user leaves the AppListActivity?


Make the use of sudo optional

This allows it to be used inside container images that don't have sudo.

The only instance of sudo I'm seeing is https://github.com/pantheon-systems/terminus-github-actions/blob/main/action.yml#L31... does your container image allow non-root to write to /usr/local/bin ?

Tracking this internally as FEAT-1047.


Horse dual riding.

Like in the boats.

This might be simple or this might be nigh-impossible. Need to look into it.

This might be simple or this might be nigh-impossible. Need to look into it.

curseforge

curseforge


Unit tests are failing when running on Windows.

Path returned is OS dependent, adjusted the expected path according to OS it is running on.

3.25.2


Add organization type to org profile settings.

Adds updating the organization's type to the organization profile admin page. Fixes #21692.
Back end implementation notes:

The Realm.org_type setting was added in #19042, so there was no need to update the models or do a migration.
There was a special RealmAuditLog event type created for updates to a realm's org_type, which has been possible for Zulip Cloud through support admin.
I extended that implementation, which meant it needed a slightly different handling in views.realm.update_realm even though the setting follows the standard structure. Added tests to test_realm for this.
I added an event to actions.do_change_realm_org_type so that other users in the realm would get live updates as they do for other organization settings. Added test to test_events for this.
Questions to confirm / consider:

Do we want to keep the specific RealmAuditLog tracking for changes to an organization's type as I did here? Or do we want to use the standard update log?
When the setting was added, existing realms were updated to be Unspecified. Do we want this to be an acceptable value for organizations to update to? If not, do we want to handle on the back end or front end?

Front end implementation notes:

This first iteration is a simple drop-down menu with a help center link.
Questions to confirm / consider:

The issue mentions providing some information about how the data will be used. Do we want to do this in the help center documentation or on the organization profile admin page?

If on the page, then how / what?
If in the help center docs, where /  what?

The existing registration and Zulip Cloud support pages do not show the Unspecified option. On the one hand, we need that as an option because a large number of existing orgs have that as their org_type value. On the other hand, we may not want to let organizations update to this setting? Or maybe that's okay?
If we do want to limit this, I'm not sure how to do that with handlebars and in a way that allows us to show the current orgs that are set to Unspecified. The other pages are written in html and use Django template syntax to limit the options shown.
Do we want to add this information to the realm log-in page, which is where we show the organization's description and profile photo? If so, we would need to update that image used in the help center documentation (see screenshot below). Part of me thinks no or only if the setting isn't set to Unspecified.

Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[ ] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Hm, I'm thinking adding "type" where you did is good, but let's make a separate documentation page for it. New organizations are already going to have the correct org type; we'd like to encourage existing orgs to update theirs.

We can put the new page under Organization basics, as the second link.
The page intro can say something like: "Setting the organization type lets Zulip customize the experience for users in your organization (e.g. Welcome Bot interactions)."

The existing registration and Zulip Cloud support pages do not show the Unspecified option. On the one hand, we need that as an option because a large number of existing orgs have that as their org_type value. On the other hand, we may not want to let organizations update to this setting? Or maybe that's okay?

Rather than having the menu say "Unspecified", let's have that displayed as the empty string. I think that will be sufficiently discouraging as an option to change your org type to.

Do we want to add this information to the realm log-in page, which is where we show the organization's description and profile photo?

I don't think we need to show it there.

Rather than having the menu say "Unspecified", let's have that displayed as the empty string. I think that will be sufficiently discouraging as an option to change your org type to.

Yeah, that sounds right to me.

Rather than having the menu say "Unspecified", let's have that displayed as the empty string. I think that will be sufficiently discouraging as an option to change your org type to.

Yeah, that sounds right to me.

So, I don't think an empty string is possible due to the internationalization formatting that we use not allowing empty strings. Would a dash, -, work? We don't have any other drop downs in the web app with a blank option. Here's a screenshot of the menu with the dash ...

Updates to PR from front end review comments:

I've added a separate help center page (see screenshot below). It's currently just the short description and instructions block.
I did figure out how to have an empty string for the "Unspecified" organization type, but I kind of think it looks better with the dash. Posting some screenshots to the CZO design discussion for that.

Set organization type help documentation page:

This is still tagged as WIP -- can you comment on what is known to be incomplete here?

This is still tagged as WIP -- can you comment on what is known to be incomplete here?

There are two pending design discussion / decisions:

Where is the drop down in the modal: between the org name and description or after the profile preview link?
Do we want a dash or an empty string for the "Unspecified" description?

I didn't get any feedback on the backend questions here yet. The one about the "Unspecified" setting seems to be using the less descriptive text in the UI will be the extent we stop people from choosing that as their organization type. The one about the RealmAuditLog type I could post as a question in a topic on CZO backend to see if there's any feedback.

For reference, this is the design thread: https://chat.zulip.org/#narrow/stream/101-design/topic/communities.20directory.20.2315844/near/1367930
@laurynmm as a heads up, it's helpful to have a link to the #design conversation right in the comment for @timabbott to refer to, so that he can easily check whether the questions have been addressed.

@alya Thanks for adding that link! I'll be sure to do that moving forward. Those design questions have been clarified, so I've updated for those decisions and removed the WIP tag.
While writing out my thoughts on the RealmAuditLog choice I made in the back end implementation, I feel pretty confident that was the right call. Especially since this is extending the existing implementation used in analytics.support.py.

Cool, after reviewing the PR again, one more point of feedback (suggested by @timabbott ): once you have an option other than "Unspecified" set, the blank line should no longer show up in the drop-down. That is, organizations that entered an option at sign-up will not encounter it, and others will no longer see it after saving their choice.

@alya - I've updated the front end implementation so that the empty string / "Unspecified" dropdown option doesn't appear once an organization has set a different, defined org_type value (see GIF below). I'm not sure if one can create a realm with the org_type as 'Unspecified' for testing in the dev environment. I was doing this manually in the code as I was debugging and testing.
screen capture of updating an organization with 'unspecified':

Yeah, I'm not sure how to test on dev, but the screen capture looks good -- thanks!

This looks great; I applied the following edits:

Renamed /help/ article to "Organization type" and extended its content a bit to make it easier to talk about other things this setting does. I think the main motivation was that the common question is likely "what does organization type do", not "How do I change my organization type", since the vast majority of organizations likely will only set it once during account creation.
Added a bit more detail to the /api/ docs to reassure client developers that they can probably ignore this, and of course like to the /help/ article.

diff --git a/static/templates/settings/organization_profile_admin.hbs b/static/templates/settings/organization_profile_admin.hbs
index e9c240bdf1..66c141925b 100644
--- a/static/templates/settings/organization_profile_admin.hbs
+++ b/static/templates/settings/organization_profile_admin.hbs
@@ -19,7 +19,7 @@
                 </div>
                 <div class="input-group admin-realm">
                     <label for="realm_org_type" class="dropdown-title">{{t "Organization type" }}
-                        {{> ../help_link_widget link="/help/set-organization-type" }}
+                        {{> ../help_link_widget link="/help/organization-type" }}
                     </label>
                     <select name="realm_org_type" class="setting-widget prop-element" id="id_realm_org_type" data-setting-widget-type="number">
                         {{> dropdown_options_widget option_values=realm_org_type_values}}
diff --git a/templates/zerver/help/include/sidebar_index.md b/templates/zerver/help/include/sidebar_index.md
index<PHONE_NUMBER>..7640e43a43 100755
--- a/templates/zerver/help/include/sidebar_index.md
+++ b/templates/zerver/help/include/sidebar_index.md
@@ -128,7 +128,7 @@

 ## Organization basics
 * [Review your organization's settings](/help/review-your-organization-settings)
-* [Set organization type](/help/set-organization-type)
+* [Organization type](/help/organization-type)
 * [Import from Mattermost](/help/import-from-mattermost)
 * [Import from Slack](/help/import-from-slack)
 * [Import from Gitter](/help/import-from-gitter)
diff --git a/templates/zerver/help/organization-type.md b/templates/zerver/help/organization-type.md
new file mode 100644
index<PHONE_NUMBER>..4f8c6ddf52
--- /dev/null
+++ b/templates/zerver/help/organization-type.md
@@ -0,0 +1,24 @@
+# Organization type
+
+An organization's type is used to customize the experience for users
+in your organization, including initial organization settings and
+Welcome Bot messages received by new users.
+
+## Set organization type
+
+{!admin-only.md!}
+
+{start_tabs}
+
+{settings_tab|organization-profile}
+
+1. Under **Organization type**, select the option that best fits
+your organization.
+
+{!save-changes.md!}
+
+{end_tabs}
+
+## Related articles
+
+* [Configure default new user settings](/help/configure-default-new-user-settings)
diff --git a/templates/zerver/help/set-organization-type.md b/templates/zerver/help/set-organization-type.md
deleted file mode 100644
index aa9fc8e8ed..0000000000
--- a/templates/zerver/help/set-organization-type.md
+++ /dev/null
@@ -1,19 +0,0 @@
-# Set organization type
-
-{!admin-only.md!}
-
-Setting the organization type lets Zulip customize the experience
-for users in your organization (e.g. Welcome Bot interactions).
-
-## Set organization type
-
-{start_tabs}
-
-{settings_tab|organization-profile}
-
-1. Under **Organization type**, select the option that best fits
-your organization.
-
-{!save-changes.md!}
-
-{end_tabs}
diff --git a/zerver/openapi/zulip.yaml b/zerver/openapi/zulip.yaml
index b7ec0ca814..9a6d1fdadb 100644
--- a/zerver/openapi/zulip.yaml
+++ b/zerver/openapi/zulip.yaml
@@ -3932,7 +3932,8 @@ paths:
                                     org_type:
                                       type: integer
                                       description: |
-                                        The type of the organization.
+                                        The [organization type](/help/organization-type)
+                                        for the realm.

                                         - 0 = Unspecified
                                         - 10 = Business
@@ -11488,7 +11489,10 @@ paths:
                         description: |
                           Present if `realm` is present in `fetch_event_types`.

-                          The type of the organization.
+                          The [organization type](/help/organization-type) for the realm.
+                          Useful only to clients supporting changing this setting for the
+                          organization, or clients implementing onboarding content or
+                          other features that varies with organization type.

                           - 0 = Unspecified
                           - 10 = Business

I've tagged this to merge once CI finishes.


Typo in Wiki page SQLite

There is a missing bracket in the example code that demonstrates how to use SQLite file. Current version is:
    <script src="alasql.js"></script>
    <script src="sql.js"></script>
    <script>
        alasql(['ATTACH SQLITE DATABASE Chinook("Chinook_Sqlite.sqlite");\
            USE Chinook; \
            SELECT * FROM Genre'].then(function(res){
                console.log("Genres:",res.pop());
        });
    </script>

which should be changed to:
    <script src="alasql.js"></script>
    <script src="sql.js"></script>
    <script>
        alasql(['ATTACH SQLITE DATABASE Chinook("Chinook_Sqlite.sqlite");\
            USE Chinook; \
            SELECT * FROM Genre']).then(function(res){
                console.log("Genres:",res.pop());
        });
    </script>

Note that there is a ) before then.
Link to the wiki page.

You are awesome - thank you!


Credential Issuer Protocol Handlers

Implement handler functions for the credential issuer protocol

[x] Use env var to allow importing custom manifests
[x] Create, read, update credential manifest records in DWN
[x] Create and update responses to applications
[x] Read applications, parse credentials and validate against manifest (presentation exchange)
[x] Request new credentials from trusted 3rd party issuers
[x] Create new credentials from 3rd party data requests
[ ] Create and update invoices as responses to applications

Partially completed by https://github.com/TBD54566975/incubation-dcx/pull/5

Implement handler functions for the credential issuer protocol

[x]  Use env var to allow importing custom manifests
[x]  Create, read, update credential manifest records in DWN
[x]  Create and update responses to applications
[x]  Read applications, parse credentials and validate against manifest (presentation exchange)
[x]  Request new credentials from trusted 3rd party issuers
[x]  Create new credentials from 3rd party data requests
[ ]  Create and update invoices as responses to applications

[ ] Debug runtime errors and handling

Implement handler functions for the credential issuer protocol

[x]   Use env var to allow importing custom manifests
[x]   Create, read, update credential manifest records in DWN
[x]   Create and update responses to applications
[x]   Read applications, parse credentials and validate against manifest (presentation exchange)
[x]   Request new credentials from trusted 3rd party issuers
[x]   Create new credentials from 3rd party data requests
~Create and update invoices as responses to applications~

[x]  Debug runtime and error handling; see #18


Constant missing in version 1.33

I upgrade kanboard to 1.33 today and installed the metadata plugin. I get the following error when I choose the plugins menu item:
PHP Warning:  constant(): Couldn't find constant \Kanboard\Plugin\kanboard_plugin_metadata-<IP_ADDRESS>\Schema\VERSION in /Library/Server/Web/Data/Sites/.../app/Core/Plugin/SchemaHandler.php on line 69, referer: https://.../dashboard

Hi,
thanks for reporting, the schema is not needed at the moment, I renamed the folder, so it is not executed automatically anymore.


Django 1.8 - There is no South database module 'south.db.sqlite3' for your database. Please either choose a supported database, check for SOUTH_DATABASE_ADAPTER[S] settings, or remove South from INSTALLED_APPS.

I'm using Django-celery for the admin-schedule feature. Everything was working fine on Django 1.7.7. I just testes on Django 1.8.1 and got this error:
There is no South database module 'south.db.sqlite3' for your database. Please either choose a supported database, check for SOUTH_DATABASE_ADAPTER[S] settings, or remove South from INSTALLED_APPS.

Oups!! I realized South was still installed in my environment. pip uninstall South just fixed my problems. Closing.

thanks, it works, but my problem of images do not show on PDF was solved by removing a slash from static of STATIC_ROOT = 'static/' directive, it was STATIC_ROOT = '/static/'.

thank you papasax

I have got the same error. I have installed south inside my project and added it to installed_apps list in settings.py file. I get this error on running syncdb or "python manage.py schemamigration App_name --initial"

Remove South from your installed apps. Django 1.7 and higher does not need South. See https://docs.djangoproject.com/en/dev/topics/migrations/#upgrading-from-south

There is no South database module 'south.db.postgresql_psycopg2' for your database. Please either choose a supported database, check for SOUTH_DATABASE_ADAPTER[S] settings, or remove South from INSTALLED_APPS.


huak run fails to run python -c commands

❯ huak run python -c "import sys; print(sys.executable)"
  File "<string>", line 1
    import
          ^
SyntaxError: invalid syntax
zsh:1: unknown file attribute: y

The implementation is pretty raw, so this is actually expected. Tag #128

Tagging this as a semi-good first issue. Hesitant because a good #486 implementation could probably resolve this, but I’m open to a less involved first pass.


Updated Events Page

The table now is using the table from the style guide and presents the
information in a clearer more attractive way to viewers.
Thought I provide one more little gift, enjoy :)  @naomiblack Please review and merge

live and merged.


openCV debug mode LNK2019 / LNK1120 error

Hello,
I'm trying to build a x64-windows project in debug mode but I get the following two errors:
Error | LNK2019 | unresolved external symbol "void __cdecl cv::cvtColor(class cv::_InputArray const &,class cv::_OutputArray const &,int,int)" (?cvtColor@cv@@YAXAEBV_InputArray@1@AEBV_OutputArray@1@HH@Z) referenced in function "public: static void __cdecl IntensityToRGBConverter::WinterColormap(double,double *)" (?WinterColormap@IntensityToRGBConverter@@SAXNPEAN@Z) | AMADEUS | C:\MA_Repo\30_Code\85_Code_Gruenewald\01_RTMaps\MyRTMaps_SDK\AMADEUS.u\Intensity_To_RGB_Converter.obj | 1 |
Error | LNK1120 | 1 unresolved externals | AMADEUS | C:\MA_Repo\30_Code\85_Code_Gruenewald\01_RTMaps\MyRTMaps_SDK\packages\debug\AMADEUS.pck | 1 |
The build errors are disappearing when I switch to release mode.
I use vcpkg version f2e9ef7f259843e116453bbabb7bcc2b13246f13, with
opencv4:x64-windows                                4.1.1-3          computer vision library
opencv4[dnn]:x64-windows                                            Enable dnn module
opencv4[jpeg]:x64-windows                                           JPEG support for opencv
opencv4[opengl]:x64-windows                                         opengl support for opencv
opencv4[png]:x64-windows                                            PNG support for opencv
opencv4[tiff]:x64-windows                                           TIFF support for opencv
opencv4[webp]:x64-windows                                           WebP support for opencv
opencv:x64-windows                                 4.1.1-1          Computer vision library
opencv[dnn]:x64-windows                                             Enable dnn module
opencv[jpeg]:x64-windows                                            JPEG support for opencv
opencv[opengl]:x64-windows                                          opengl support for opencv
opencv[png]:x64-windows                                             PNG support for opencv
opencv[tiff]:x64-windows                                            TIFF support for opencv
opencv[webp]:x64-windows                                            WebP support for opencv
Why is this happening?
Thanks in advance!

Hi @jagrme00
Thanks for posting this issue.
Could you please try to rebuild this based on PR #11130?

Well I can try it, but this changes my local repo to version dbfe6cc1ca3ce3532b768acd2ccd8c60616a412f. Thus the version of the Point Cloud Library (PCL) that I also need for my project gets changed from 1.9.1-11 to 1.9.1-13 and this will break my code since not all features are included in the current PCL version.
Is it possible to run two separate vcpkg versions in parallel one for PCL and the other for OpenCV to overcome this issue or is there a better solution?
Thanks!

@jagrme00
You can merge the two changes to one branch and then try to rebuild. Since there are too many changes in PR #11130. You can update the changes for PCL to PR #11130.

Okay. Since I am not working with git on a daily basis can you explain this with a little bit more details please?
I tried the following:

git clone https://github.com/Microsoft/vcpkg.git

git checkout f2e9ef7f259843e116453bbabb7bcc2b13246f13

git fetch origin pull/11130/head:TEST

git merge TEST

I am 100% sure that these are the wrong steps because I ended up with PCL 1.9.1-13 again...

@jagrme00
you might try to use the following steps:

git clone https://github.com/cenit/vcpkg.git
cd vcpkg
git checkout -b dev/cenit/opencv43 origin/dev/cenit/opencv43
merge PCL updates to this repo manually

Or you can wait for the PR merged into master and then try this again.

@jagrme00
you might try to use the following steps:

git clone https://github.com/cenit/vcpkg.git
cd vcpkg
git checkout -b dev/cenit/opencv43 origin/dev/cenit/opencv43
merge PCL updates to this repo manually

Or you can wait for the PR merged into master and then try this again.

@NancyLi1013 @jagrme00 to do it with less line:

git clone https://github.com/cenit/vcpkg.git  -b dev/cenit/opencv43
cd vcpkg
merge PCL updates to this repo manually

Hi @jagrme00
opencv4 has been updated to 4.3 via PR #11130.
Now you can update vcpkg to the latest version and try to rebuild if this can work for you.

@jagrme00
Please let me know if this is still a problem for you now.

@NancyLi1013 sorry for my delay. Currently I have some other topics which require more attention than this issue.
I'll try the suggestions within the next few days and I'll let you know my results.
Thanks for your help.

My problems are solved now. The above mentioned approach https://github.com/microsoft/vcpkg/issues/12487#issuecomment-664813848 is working for me.
Thank you @NancyLi1013 for you help!

Thanks for your update.


facing problem in installation

hello i have tried all method as you describe in read.me file
can you help me regarding installation
thanks

Hello,
Thank you for reaching out and for trying the installation methods described in the README.
Firstly, I would like to point out that in most situations, it is not necessary to run your own server; you can use our public cloud server for free. For more information on this, please visit: https://sms.capcom.me/getting-started/public-cloud-server/
However, if you are interested in running your own private server, this does require some technical knowledge. You can find more detailed instructions and information in our documentation here: https://sms.capcom.me/getting-started/private-server/
If you encounter any problems during the installation, could you please provide a detailed description of the issue? This should include any logs from your actions or screenshots of the errors you are experiencing. This additional information will greatly assist us in helping you resolve the problem more effectively.

i have tried but not installed properly do you have any other documentation or docker file for centos 7 or ubuntu

Can you provide any details about the installation methods you've attempted, the steps you've taken, and the results?
In our documentation at https://sms.capcom.me/getting-started/private-server/, there are two links with instructions on how to install a private server with Docker:

One is from a user contribution: https://github.com/capcom6/android-sms-gateway/discussions/50
Another is a guide that I've prepared: https://github.com/capcom6/sms-gateway/tree/master/deployments/docker-compose-proxy

Please try following these guides and let me know how it goes. Your feedback would be greatly appreciated.

thanks for your quick reply now i have installed system on my server
does api support receiving replies to the conversation and associating it with a sent message ID
Thanks

Thank you for the update, and I'm glad to hear that you've successfully installed the app on your server.
Unfortunately, the current version of the app does not support receiving replies. The primary focus of the app at this time is sending messages.
I may consider adding this feature in future releases, but I cannot provide any guarantees. For further discussion and updates on this topic, you can follow this issue: https://github.com/capcom6/android-sms-gateway/issues/16
Thank you for your understanding.
Do you have any more questions about the private server installation, or may I close the issue?


Request: Can we have retro theme(s) ?

See the themes at the following:
https://github.com/AlexanderSharykin/RetroUI
This is a bit of fun and low priority, but might find use for Tablet style apps (i.e. low resolution)

Might be possible to do. Is there any documentation for the palette designer, as I have no clue how to use it?

From the Help
Upgrading Palette Definitions
When moving from one version of Krypton to another there are sometimes changes made to the palette XML format. When this happens you can use the Palette Upgrade Tool to convert palette XML files to the latest file format. This utility can be started from the Krypton Explorer program, a shortcut for which should be on your desktop after installation. Figure 5 shows the tool in action.

Figure 5 - Palette Upgrade Tool
You can make use of the tool to upgrade Krypton Palette component instances. Before installing a new version of Krypton you can use the Export option on the palette in order to create a palette definition file. Then perform a Reset on the palette component so it is defaulted for all its values. Once the new version of Krypton has been installed you use the Palette Upgrade Tool to update the XML file and then Import it into the palette component. This is easier than trying to fix the compile time errors that would result from palette properties being removed or renamed.

I'll look into it. Currently working on an XML authoring system for part of a feature that I have integrated into the `Extended Toolkit' project.

@Smurf-IV
Can do if we set the palette to Custom. The only issue is that the KryptonComboBox does not currently support a dropdown BackColor property.

KryptonComboBox -> StateCommon -> DropBack e-> tc.
or
KryptonComboBox -> StateCommon -> Item -> Back -> etc.

Is there a property for it in the Palette Designer?


problem when reading CK2 Exports

i found an issue while converting.
The converter is currently unable to read converted_countries_l_english.yml files from eu4_export.
This file placed in the Converted_Country1444_11_11\localisation\replace folder by default.
But the converter only read localisation files outside of the replace folder.
So,if you convert a save that included nations generated by ck2,it will be "Y30" "Y40" "X20".
If i moved converted_countries_l_english.yml file outside of replace folder,the problem will be fixed.

Same issue as in #153

And thus also should have been fixed by #240


[Forward Port] Drain on delete for nodes in Rancher provisioned clusters fails when node is running Pods with emptyDir volumes

Forward port to be validated on master-head for https://github.com/rancher/rancher/issues/31455

Version:
Rancher | master-6ea7600e6bddcd0483afcf86cd6142920601d4e5-head

Create. cluster with 1etcd, 2cp, 3 worker and enable drain on delete

Create a pod:
apiVersion: v1
kind: Pod
metadata:
name: test-pd
spec:
containers:

image: k8s.gcr.io/test-webserver
name: test-container
volumeMounts:

mountPath: /cache
name: cache-volume
volumes:

name: cache-volume
emptyDir: {}

Scale down the node on which the pod was created

While the node is drained, we see the pod is evicted:

node  kubectl drain response: node/ cordoned
 deleting Pods not managed by ReplicationController, ReplicaSet, Job, DaemonSet or StatefulSet: default/test-pd
evicting pod default/test-pd
pod/test-pd evicted
node/***-w4 evicted


libnotify integration

For Linux users, it'd be great to get a notification on new messages.

You could use the weechat plugin: https://weechat.org/scripts/source/lnotify.py.html/

Sure, I had just noticed that 'growl integration' was on the roadmap. Since this plugin looks like it's trying to become a cohesive solution, it would be nice if libnotify was included.
If that's not the plan, please do close this bug. There's no reason to leave it open if there is no intention to support it.

I actually use this script and while it works great with bitlbee and "pure" IRC, with wee-slack I get a double notification.
I'm using dunst for notification daemon and this is what dust gets:
{
	appname: 'WeeChat notification'
	summary: 'infamous_derp'
	body: 'ok
ok'
	icon: 'info'
	category:
	urgency: 0
	formatted: 'person: ok
ok'
	fg: #000000
	bg: #83c83d
	id: 3
	script: (null)
}

As you can see the body is duplicated on a new line.

I haven't been able to get any notification plugins to work with wee_slack at all, only regular IRC.

The lnotify plugin kind of works for me.  When someone messages me it will send a notification of the backlog from them as well which is kind of annoying but still works.  As long as I am not on the conversation it will notify me upon direct messages and items that get highlighted (I believe)

I see the same thing @mrrsm mentioned sometimes as well. But it's definitely not 100%

It only sends you all messages if you didn't have the buffer open. I only get this issue if I close any of the chat buffers with people writing to me. By default weeslack will open these conversations every time. So the workaround is to not close the conversation buffers.

@terlar that is not a convenient option (for me) though.
I believe that if you add previous messages to the backlog rather then the chat buffer you would get the appropriate messages sent to libnotify (or maybe it loads that for you, I am not really sure)

This may also be (mostly) fixed in #124
I've been testing with my terrible-custom-growl-like system and I no longer get dupes when a DM buffer opens.

Sounds like duplicates are fixed now, and we shouldn't have our own notification system (but integrate with those already out there, which we do), so closing this.


Do not return the user secret

Describe the problem
Currently, there is a class called DelphiUser which is used to both add and return a user. It has an attribute called secret: String which, when adding a user holds the plaintext password, and when returning a user holds the password hash. I do not think it is good practice to return the password hash when asking for users. There is no management use-case where an administrator would want to see the password hash, and there is no additional value in having this data in the management application. Instead it is rather risky to send private information like the password hash around too much, as it might give attackers a chance to break passwords.
Describe the solution you would like
Change the workflow so that no secret is returned when asking for users. Of course we still need the secret attribute for adding a user, so you can not just remove it. You could think of making it an Option[String], or you could just leave it a string and set it to the empty string when returning users.

Closed with #121


Feature: Front-End Design for Medical Record Pages (User and Admin)

Describe the solution you'd like
2 webpages (one for both user and admin) that deal with medical records for the patient. Should be done using HTML and CSS with reference to the Figma drawing.
Additional context

Closed via #22