Callbacks are slow?

Whenever I'm streaming binary data, either the data is received slowly or the callback is taking a long time to get called.
Just a little context surrounding this, I'm streaming images in binary form from a server and rendering them on screen in Android. I know that there is some 'choking' going on with the Android client because I'm also streaming and rendering to a browser client and on the browser it's delay free. However when I stream to the Android client there is some delay.
Just to clarify, at the start of the stream there is absolutely no delay, but after around 5 seconds, the images received start to slow down (as in, they go in slow motion), which possibly points to the Android library doing some extra processing that slows down the callbacks being called (or my asynchronous LinkedBlockingQueue implementation being just badly done).
Here is some code:
public class MainActivity extends AppCompatActivity {

    private LinkedBlockingQueue<byte[]> linkedBlockingQueue = new LinkedBlockingQueue<byte[]>();
    private boolean keepRunning = true;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        ...
        new MyRenderer().start();
        ...
    }

    // Decodes and renders a bitmap
    private class MyRenderer extends Thread {
        @Override
        public void run() {
            // Keep running this thread
            while (keepRunning) {
                // .take() blocks until we have more data
                final byte[] bitmapArray = linkedBlockingQueue.take();
                final Bitmap bitmap = BitmapFactory.decodeByteArray(bitmapArray, 0, bitmapArray.length);

                // Set bitmap to an imageview
                runOnUiThread(new Runnable() {
                    @Override
                    public void run() {
                        someImageView.setImageBitmap(bm);
                    }
                });
            }
        }
    }

    // We'll create a websocket listener as a class field and
    // set it to a websocket later
    WebSocketAdapter websocketAdapter = new WebSocketAdapter() {
        ....
        @Override
        public void onBinaryMessage(WebSocket websocket, byte[] bitmapArray) throws Exception {
            super.onBinaryMessage(websocket, bitmapArray);
            // Add the binary bitmap to the queue for processing in the above thread
            linkedBlockingQueue.put(bitmapArray);
        }
        ...
    }

   // Create websocket, set the websocketAdapter to a websocket, etc

}

Are there any pointers you could give me?
I would also be interested to know if you will be supporting a streaming websocket. As in one that I could easily perform websocket.read() for example, because that would mean I could pipe the data out to an OutputStream for forwarding or simply to read data in a new thread in an easy manner.

Library version: 1.22
Android compileSdkVersion: 23
Android minSdkVersion: 19
Fantastic library + documentation by the way :+1:

It is needed to find the point that consumes time too much. So, why don't you measure how much time BitmapFactory.decodeByteArray() consumes?

Many thanks for the reply.
You were right, the callbacks are not slow at all, it was actually my producer-consumer (the linkedBlockingQueue) implementation filling up. I have changed the code to a better implementation, namely I have used an ArrayBlockingQueue instead.
Thanks for your help :smile:

I'm glad to hear you've solved the problem.


tests, libstorage: relocate some recently introduced PVC utils

Signed-off-by: Alex Kalenyuk<EMAIL_ADDRESS>
What this PR does / why we need it:
These were introduced in #7324 and would perfectly fit in libstorage.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Looking at pod funcs in the big utils.go, I think we have enough justification to create libpod.
WDYT?
Release note:

NONE

@dankenigsberg @rhrazdil I think this is a good advocation for a small lane that yells at me for adding to an already-huge utils file
BTW, do you think there is justification to create libpod?

/lgtm

/retest-required


[Feature Request] InputNumber adds placeholder attribute

Existing Component
是
Component Name
InputNumber
Description
InputNumber 组件可以添加placeholder属性，输入框里的内容为空时显示的内容

Add PR for the feature, closed.
If you want more support, reopen or create a new issue please.


Xprize Semifinals testing yarpmanager application files updates

@kouroshD can we proceed with this PR?

Yes I think, let me check this again first and I will put for review.

@kouroshD friendly ping

sounds good to me @S-Dafarra . Have a look on it as well, then we merge. thanks.

Rebased the branch, working for the jobs. Then will merge.


Can I open the iphone system address book? If you can write a url how to write quickly and ask you to teach me?

Can I open the iphone system address book? If you can write the url how to write quickly, I beg you to teach me. I want to go to the address book and click on one of the contacts to return the phone number.

I'm not aware of a way to link people to the contacts app. Additionally, that functionality isn't implementable through app linking and this really isn't the right package for that sort of thing. I recommend you take a look at react-native-contacts or something similar so your app can get access to the user's contacts directly and you can display a list within your own app.


Add kube component reference docs

Closes: #8387, #8264, #7943

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For 1.11 Features: set Milestone to 1.11 and Base Branch to release-1.11
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
NOTE: After opening the PR, please un-check and re-check the "Allow edits from maintainers" box so that maintainers can work on your patch and speed up the review process. This is a temporary workaround to address a known issue with GitHub.>
Please delete this note before submitting the pull request.

/close
@zacharysarah is committing a big pr which includes a fix for kube compnents.


Fixed detection of VS SDK and VCTools Versions

Initially the detection of the Visual Studio SDK and VCTools versions were returning null. This was due to case sensitivity. The following changes are case insensitive, allow the removal of the ext.getEnvironmentValue method and remove the need for env to be in the system PATH. This was also tested with Visual Studio 2019 to ensure it will work in the future.

Recreated changed and merged into patch for 9.0.3.  Added trim() to eliminate newline from Version strings.


Support wakatime-chrome extension

Installed chrome extension from wakatime.com. It's seems like I can't set the api endpoint. So the only way is the clone the source code and replace wakatime api with wakapi?
Searched issues found nothing about this.

I think you're right.
It doesn't seem to be adjustable.

Very unfortunate! I opened https://github.com/wakatime/chrome-wakatime/issues/130 in the upstream repo, let's see if some solution can be found. @shyn, are you using wakapi.dev or a self-hosted instance?

Very unfortunate! I opened wakatime/chrome-wakatime#130 in the upstream repo, let's see if some solution can be found. @shyn, are you using wakapi.dev or a self-hosted instance?

I use a  self-hosted instance.
Appreciate the upstream issue. I will watch it, thx!

I ran the chrome extension successfully, but we cannot get the correct os/editor/machine because of the user-agent. The user-agent from the browser looks like Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/93.0.4577.63 Safari/537.36
wakapi cannot parse it.

Thanks for investigating! I'll have a look at the user agent issue soon.

Thanks! We can maintain a chrome-wakatime fork to support wakapi.dev. But it is hard to keep one extension for all self-hosted instances. We need to work around ajax/chrome extension CORS problems.

Okay, so here's the plan:

[x] Fork the official plugin
[ ] https://github.com/muety/chrome-wakapi/issues/1
[ ] https://github.com/muety/chrome-wakapi/issues/2
[ ] https://github.com/muety/chrome-wakapi/issues/3
[ ] Publish chrome-wakapi plugin to Chrome- and Firefox stores
[ ] Add usage instructions to README

Right now, this is blocked by https://github.com/wakatime/chrome-wakatime/issues/135. Unfortunately, I can't properly build the plugin, so I can't continue with the above points. Maybe it's only me, but otherwise we'll have to wait for someone to fix the plugin build.
Also, help is highly appreciated here, especially because the plugin is written in React, which I don't have any expertise with.

Sorry, I don't want to pressure you. If you think you can't afford time for this more or less soon-ish then I'll have a look. However, it's probably gonna take me much longer, as I'm not familiar with React, etc.

I just found wakapi but I am very love it and I am intending to support this task but I found this extension uses manifest v2. I don't know if wakatime has any plans to move to v3 version?🤪🤪

I am developing a small extension, I hope I will release it soon.

@f97 How is going forward with your extension? Is there anything we can help you with?

@muety
hehe, I've had a lot of problems with my life, work, and love lately, so I can't make time for the promise of writing this extension.
I have a bit of free time these days so I quickly code a small extension, but I'm having a problem that I don't know what to push.
I tried a few things but still not satisfied, you can contribute here.
At least it works but it doesn't work properly: 3
https://github.com/f97/chrome-wakapi.git

That is great news! I'm excited to check out what you already came up with. Maybe you can add a brief list of things that would still need to be done / are not working yet - e.g. in the form of issues or as part of your project's README - and I'm sure people will be happy to jump in and contribute to get this thing live! 🚀
And of course, don't feel obliged here by any means!

I like the relay idea.
Just to throw another idea into the room:
if the wakapi host would be provided (i.e. WAKAPI_API_URL), then wakapi could build the extensions on Docker boot (maybe some npm build resulting in an URL dependent file name, so that it won't be rebundled if the hostname or the file hashes didn't change) and put them into the asset directory.
Of course this would mean, that a new wakapi container image must be pulled to get the updated extension but on the other hand this would also make sure that the API always matches.
And I think there's an extra warning when installing extensions from other hosts.
But other than that it would be nice because users know that they don't have to send their activities via a third party.
I'm not

I'll ahve a look into content scripts to see if that could help us by any means. Thanks for the pointer.
Apart from that, if I got your solution approach (building the extension with custom API URL at container build- / run time) correctly, it would mean that everyone, who is self-hosting Wakapi, would have to build and install their own browser extension, no? Of course, this is always an option. But I'd like to have a solution where people can just go to the Chrome / Firefox store and install an official add-on right from the marketplace.
However, the main blocker on this ticket is anyways the extension itself, which would require a few adaptions, but whose code unfortunately seems to be abandoned.
Again, if anyone has good expertise with (a) React and (b) the WebExt API, help is highly appreciated here! 🙂

wakatime/chrome-wakatime#135 finally got resolved in the upstream repo, so no more blockers for us! 🙌
Next steps: as described above.
I'd be super thankful if someone with a bit of React experience could help out in this! @qhantom, maybe? 😇

Sure, gonna have a look at https://github.com/muety/chrome-wakapi/issues/2 :-)

Hi @varac, thanks for reporting this!
To clarify, you don't see your coding time reflected in the Wakapi dashboard at all or is it just classified as unknown? Also, do you see any error messages in the server's logs?
The user agent issue from above most likely still persists. I'll look into it soon!

I fixed parsing the user agent strings, so you should see "Chrome" appearing as an editor now.
However, I realized that domains / web pages are implemented as entities (aka. files), while Wakapi currently does not support file statistics at the moment. I'll have to reconsider this, because otherwise, the browser plugin would be rather useless.

Great! As soon as there's a new image tag I'll test and can give feedback. Thanks!

browser-wakatime now supports sending data to Wakapi. See instructions here. :heavy_check_mark:


Releasing 2.7: are we ready?

I'm thinking of releasing 2.7 soon (by the end of the week, basically).
This includes:

LazyFunction
Traits
Remove Python2.6 support (fitting with the release version :P)

@jeffwidman do you think you're ready? Or do you think we need more docs on one of those topics?

Sorry for the delay here. I have no objections, let's pull the trigger.

Awesome, I'll handle the release tomorrow (too busy last week :/)

Released as f1ed74e06dfb6851bc691ebfd8135c875154ad50 \o/


ERROR when upload to AppStore

ERROR ITMS-90056: "This bundle Payload/X.app/Frameworks/Then.framework is invalid. The Info.plist file is missing the required key: CFBundleVersion."

@devjia, which version did you use?

Thanks!
I solved this issue by manually modify the info.plist file of Then.framework.
And, the new prebuilt binary looks no problem.
But,
carthage update will broken and throws Incompatible Swift version - framework was built with 4.0 (swiftlang-9<IP_ADDRESS> clang-900.0.37) and the local version is 4.0 (swiftlang-900.0.65 clang-900.0.37).
I think I should update Xcode to version 9.0.1 to solve this.
Is there a way not use prebuilt binary ?
Do you have any suggestion?

I suggest you to update Xcode to 9.0.1.

Thanks!


[CWS] reduce the mount fallback

What does this PR do?
This PR change the key used for the mount fallback rate limiter and the rate itself. Previously it was 75 allowed fallback per mountID each 5sec. This can cause some overhead and it is not really useful. This PR uses a key composed of PID/MountID with a limit of 1 per 5sec.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

How'd you find the increase in overhead?

@modernplumbing locally spawning a lot of container, I can see that a fallback for the same mount ID/PId is called multiple times. It generates useless syscalls/parsing, etc.


follow up to previous issue

Because the previous issue was closed I don't know if you will see the comment there that's why I create anew one. Please excuse me if this is troubling you!

This is how my code, data and result look like the image that I predict after training is the same one that I have annotated beforehand so I want to compare the baseline prediction to the prediction after training.
Can you give me some clue what might be wrong?
Thanks in advance!
Originally posted by @kristiyan-hristov in https://github.com/weecology/DeepForest-pytorch/issues/81#issuecomment-878864485

Hi @kristiyan-hristov - @bw4sz's main point in the last issue was that this is the wrong repository to be using both for installing software and opening issues. Active development is now in https://github.com/weecology/DeepForest. I am transferring this issue over there so it can be addressed.
To get us started can you please let us know how you installed (pip, conda, or from source) DeepForest? Thanks.


Bitwise binary operators needlessly slow?

Just from reading the source code, I see two problems with the bitwise binary operators (&, |, ^).

The assign variants do not support a reference RHS. So if you want to compute a ^= &b where a, b are bitvecs a needless clone is necessary.

As far as I can see, the implementation calls down to the incredibly slow bitwise iterator approach. When you have two bitvecs with the same underlying storage you can directly &/|/^ the limbs and get a 8 to 64 times speedup (depending on the size of your limbs).

This is a mixture of limitations in the language and underexploration of the design on my part. It is eligible for performance acceleration, at the cost of generality both on Rust's end and on mine. However, it is probably more important to accelerate the common case of operating between like-kinded data structures than it is to enable operating with arbitrary iterators.

This acceleration is feasible, but breaks public API. Since I consider performance in the common cases to be an important factor in crate behavior, and we are still in the 0.X development series, I will do this break.
Please see the branch feature/better-bitops for a draft implementation. Especially please patch your Cargo.toml to use it:
[patch.crates-io.bitvec]
git = "https://github.com/myrrlyn/bitvec
branch = "feature/better-bitops"

I am not confident that my implementation is correct, or that I have a satisfactory test suite to find edge cases. I have had defect reports in the past from attempting to try this specialization route, and I would like to test this in the wild before publishing.
Once I am confident in the implementation, I will publish this as a patch version, not as 0.21. While this is technically a semver breaking change, I strongly suspect that nobody is using the APIs that will produce a compile failure, and I have a simple fix for them if I'm wrong.

The v1 series replaces the bit-wise operator implementations with batched integer operations. I apologize for completely forgetting to follow up on this in the past ... year ... but it's been a rough one and I haven't been working on bitvec as much as I'd have liked.
Anyway. Thanks for prompting me to do this. As the behavior is present in current v1 code, I'm going to close this issue. Please feel free to re-open it, or file a new one, if you experience any problems with the code. While I have some tests for it, I am never confident that I have enough, and real-world usage is always more useful than what I am able to contrive myself.


[DOC] - Add environment requirements for dashboards

Preliminary Checks

[X] This issue is not a question, feature request, RFC, or anything other than a bug report. Please post those things in GitHub Discussions: https://github.com/nebari-dev/nebari/discussions

Summary
The tutorial on creating new dashboards should include the required packages for creating dashboards of each type.
For example, if you don't include
      - cdsdashboards-singleuser >=0.5.6
      - jupyter_bokeh

Then panel serve will fail with ERROR: The Bokeh notebook application handler requires Jupyter Notebook to be installed.
Steps to Resolve this Issue

...

Also add FAQ item


Carousel - no animation when edit in plunker.

There's the nice animated example of carousel on plugin homepage:

But when I click 'Edit in plunker' button there's no animaton in plunker example:

Same issue I have in my project. Tested with Angular 1.3.* and angular 1.4.*.

That's because since 0.13, you need to include angular-animate in your project if you want animations:
http://plnkr.co/edit/0k1HyBcT87MXSRZ8pQXD?p=preview


WIP: UI Fixes

This branch fixes two UI issues.

DataPackGridItem description overflow gets hidden behind zoom buttons.  Added a z-index to the div that holds the description text to ensure it displays on top of the zoom buttons.

DataPack fields don't update their size and character counts when backing out of confirm section. Character counts reset to original values.  Added a componentWillMount function that checks to see if the exportInfo props currently have any text in the name, description and project fields.  If they do have text already, the counter is updated to be the max - text.

Updated this PR to handle things a little differently. MUI was a real pain on this one :/
The CustomTextField now checks the length of defaultValue or value and if present, reduces the initial charsRemaining by that amount. Since we reference the props defaultValue and value we should be listing them in the prop types. If we list them in the prop types as NOT required we should give them default values. The problem is, we would want those default values to be undefined (unset) and the propTypes to be any (what MUI does). Having those values as undefined seems to stop the hint text from disappearing on the Component, so we just skip the whole thing and disable the eslint rule for it in this case.
The other issue was that when the TextField renders with a long default value, it wont adjust height to that default value. So you end up with one line of text in the field and you must "scroll" down to see the other lines. The TextField WILL adjust height to the value so we should use that instead. To use value we need to rework how we are handling changes, we cannot debounce user input before saving to the state.
So yea, thats that.


RAID F1 and SHR may coexist

After setting support_syno_hybrid_raid to "yes" and supportraidgroup to "no", it appears that RAID F1, SHR-1 and SHR-2 can coexist.  One can choose RAID F1 on SSD drives and SHR-1/2 on HDD drives on the same machine.
This seems to run against your README statement "Changing to RAID F1 disables SHR, and changing to SHR disables RAID F1."  Is there a reason to impose this mutual exclusiveness?

Thanks. And thank you for including screenshots.

This seems to run against your README statement "Changing to RAID F1 disables SHR, and changing to SHR disables RAID F1." Is there a reason to impose this mutual exclusiveness?

From what I read on the Xpenology forum I thought they said that in DSM you could have RAID F1 or SHR but not both. I've never had 3 SATA SSDs to test it with.
Now I'm wondering if my script is just disabling and enabling SHR - and RAID-F1 is always enabled (on models that support RAID F1). Which raises the question of how to enable RAID F1 on other models.

https://xpenology.com/forum/topic/9394-installation-faq/?tab=comments#comment-131458
This xpenology post suggests that "Note that SHR can be enabled on these platforms by modifying /etc.defaults/synoinfo.conf, with no impact to RAIDF1 functionality."

"The MDRAID modifications that enable RAIDF1 are compiled into the DSM kernel. The consumer-oriented DSM platforms do not contain those changes,"

That makes sense.
Before I saw your last comment I found some incorrect information here https://www.reddit.com/r/synology/comments/jueg8z/comment/gcbv9wp/ that said "RAIDF1 is an option compiled into their customized version of mdm, and therefore there is no way create a RAIDF1 on unsupported device."
After checking 112 different model's mdadm I found that all models include raidf1 in mdadm (even single drive Synology NAS!) so I knew that was wrong.
So my raidf1_shr_switch script needs an update to make it just shr_switch

How are you planning to change the script? 😄
I noticed https://kb.synology.com/en-us/DSM/tutorial/What_is_RAID_Group
DSM 7.0 and above

On models with support for RAID groups
- RAID F1

On models without support for RAID groups
- RAID F1, SHR

If "models with support for RAID groups" can be signified by supportraidgroup="no",
is it fair to assume the following outcomes?

supportraidgroup="no" & support_syno_hybrid_raid="yes": both F1 and SHR are available
supportraidgroup="yes" & support_syno_hybrid_raid="no": only F1 is available
supportraidgroup="yes" & support_syno_hybrid_raid="yes": only F1 is available

This KB article https://kb.synology.com/en-us/DSM/tutorial/What_is_RAID_Group
especially the row "On models without support for RAID groups" where F1 and SHR are mentioned together
seems to suggest that the mutual exclusivity is between RAID group and SHR, not between F1 and SHR.

seems to suggest that the mutual exclusivity is between RAID group and SHR, not between F1 and SHR.

It does seem to suggest that.

How are you planning to change the script? 😄

I was going to remove all reference to RAID F1 and change it to either disable or enable SHR. But having read that page you linked I think I'll just change the RAID F1 text to RAID Group. So it will actually be Synology_RAID-Group_SHR_switch

I noticed https://kb.synology.com/en-us/DSM/tutorial/What_is_RAID_Group
DSM 7.0 and above

On models with support for RAID groups
- RAID F1

On models without support for RAID groups
- RAID F1, SHR

If "models with support for RAID groups" can be signified by supportraidgroup="no",
is it fair to assume the following outcomes?

supportraidgroup="no" & support_syno_hybrid_raid="yes": both F1 and SHR are available
supportraidgroup="yes" & support_syno_hybrid_raid="no": only F1 is available
supportraidgroup="yes" & support_syno_hybrid_raid="yes": only F1 is available

The only "model" that has supportraidgroup="no" is VirtualDSM.
The models that support RAID groups all have supportraidgroup="yes"
The models that don't support RAID groups don't have any supportraidgroup setting in synoinfo.conf

It definitely looks like:

You can have RAID Groups or SHR but not both.
RAID F1 is always available (on models that support RAID F1).

I would take your scenario one step further to say:

supportraidgroup="no" & support_syno_hybrid_raid="yes": both F1 and SHR are available, but not RAID groups
supportraidgroup="yes" & support_syno_hybrid_raid="no": both F1 and RAID groups are available, but not SHR
supportraidgroup="yes" & support_syno_hybrid_raid="yes": both F1 and RAID groups are available, but not SHR

The one exception to this is the HD6500 which officially does not support SHR or RAID F1. It has supportraidgroup="yes" but does not have support_syno_hybrid_raid.

FYI, my current setup is as follows:
DS3617xs, DSM 7.2, and /etc.defaults/synoinfo.conf has
supportraidgroup="no"
support_syno_hybrid_raid="yes"

Screenshot of Storage Manager:


Add hoarder extension

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Could you add at least one screenshot of the extension 🙂

Yes, gonna prepare something!


Formatting is off

@bram209 as discussed on discord here is the weird formatting:

                    {result_string} <Show when=is_finished>

{conclusion}

                    </Show>

You can find the whole file here: https://github.com/hiveboardgame/hive/blob/main/apis/src/components/molecules/game_row.rs
Thanks
leex

closing this as a duplicate of #112

I don't know if this is related somehow but I observed that commenting out line
macro_names = ["leptos::view, view"] # Macro names which will be formatted
in leptosfmt.toml file ironically fixes formatting inside !view macro.

I don't know if this is related somehow but I observed that commenting out line macro_names = ["leptos::view, view"] # Macro names which will be formatted  in leptosfmt.toml file ironically fixes formatting inside !view macro.

Not sure where you get that line from, but macro_names should be ["leptos::view", "view"] and not ["leptos::view, view"].

Ah, I had  old version which I copied some time ago, Probably I wouldn't notice this, thanks!


MLP activation functions

Team members:

Keresh Arman
Konyspay Aidos
Meiramova Sabina

Also we created github page. https://bombex.github.io/kbtu-ml-book/mlp/activations.html


[AIRFLOW-3447] Add 2 options for ts_nodash Macro

Make sure you have checked all steps below.
Jira

[x] My PR addresses the following Airflow Jira issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-3447

Description

[x] Here are some details about my PR, including screenshots of any UI changes:
Add Option to use ts_nodash and ts_nodash_with_tz macro so it gives any option to users to use it Filename. Previous users have been using ts_nodash for file and folder names.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
Added tests

Commits

[x] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.
All the public functions and the classes in the PR contain docstrings that explain what it does
Updated the docs

Code Quality

[x] Passes flake8

Codecov Report

Merging #4323 into master will increase coverage by 0.49%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4323      +/-   ##
==========================================
+ Coverage   77.59%   78.09%   +0.49%
==========================================
  Files         201      201
  Lines       16470    16471       +1
==========================================
+ Hits        12780    12863      +83
+ Misses       3690     3608      -82

Impacted Files
Coverage Δ

airflow/models.py
92.31% <100%> (ø)
:arrow_up:

airflow/www/views.py
69.37% <0%> (+0.12%)
:arrow_up:

airflow/hooks/dbapi_hook.py
83.06% <0%> (+1.61%)
:arrow_up:

airflow/hooks/hive_hooks.py
75.26% <0%> (+1.84%)
:arrow_up:

airflow/utils/sqlalchemy.py
78.57% <0%> (+5.71%)
:arrow_up:

airflow/operators/mysql_to_hive.py
100% <0%> (+100%)
:arrow_up:

airflow/operators/mysql_operator.py
100% <0%> (+100%)
:arrow_up:

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 9d87552...f3d0f9e. Read the comment docs.

Actually - what does this do to the filename that logs are written to?

I don't think it affects the log filename as the log filename uses ts. I will add details to UPDATING.md
# Log filename format
log_filename_template = {{{{ ti.dag_id }}}}/{{{{ ti.task_id }}}}/{{{{ ts }}}}/{{{{ try_number }}}}.log
log_processor_filename_template = {{{{ filename }}}}.log
dag_processor_manager_log_location = {AIRFLOW_HOME}/logs/dag_processor_manager/dag_processor_manager.log


Update sbt-travisci to 1.2.0

Updates com.dwijnand:sbt-travisci from 1.1.3 to 1.2.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a nice day!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [{ groupId = "com.dwijnand", artifactId = "sbt-travisci" }]

Codecov Report

Merging #11 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #11   +/-   ##
=======================================
  Coverage   54.54%   54.54%
=======================================
  Files           1        1
  Lines          66       66
=======================================
  Hits           36       36
  Misses         30       30

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 c71631d...969affa. Read the comment docs.


[locust] Update Readme, url and logo

Add url and logo to chart description

I just noticed the README still mentions this chart being in incubator - but it was already moved to stable

I just noticed the README still mentions this chart being in incubator - but it was already moved to stable

Yes, please update the readme to reference the stable repo too in this PR.

rebased and updated to include the changes requested

@prydonius - let me know if there is anything still missing

LGTM, thanks a lot @so0k!


Whether zipwith can be rewritten

Whether zipwith can be rewritten ?
var zipWith =  curry(function(f, list1, list2, init) {
if(!list1.length || !list2.length ) return init;
init[init.length] = f(list1[0], list2[0]);
return zipWith(f, tail(list1), tail(list2), init)
});
export default zipWith(R_, R_, R_, []);

The very early history of Ramda involved @buzzdecafe and me writing nice elegant recursive code for everything, in a library called Eweda.  It failed miserably at being practical; even when our code was eligible for tail-call optimization, the engines wouldn't provide it.  Ramda was a rewrite with an emphasis on the practical.
So, while that looks elegant to me, I'd be extremely wary of including it in Ramda.


glob expansion in zap :delete and uninstall :delete

Done in https://github.com/Homebrew/brew/pull/2042. We now need to:

[x] Add it to the documentation.
[ ] Add it to casks (many have it in a commented TODO).

Hello @vitorgalvao I would like to get stuck into this, but I am not 100 % certain what is required.
For example, looking at the balsamiq-mockups Cask, it what you have on mind as simple as it states?
Finally, would it be better to have glob expansions everywhere where possible, like in an uninstall :pkgutil for example?

I would like to get stuck into this, but I am not 100 % certain what is required.

You mean you want to get assigned? Done. You should also be able to self-assign (or unassign) with the options to the right of the conversation.

For example, looking at the balsamiq-mockups Cask, is what you have on mind as simple as it states in the commented TODO?

Yes. In those cases, removing the # TODO: expand/glob for  should suffice.

Finally, would it be better to have glob expansions everywhere where possible, like in an uninstall :pkgutil for example?

No. If we can accurately pinpoint the files and there’s only a few of them, we should spell them out. When there are too many/we don’t know them all and we can target them specifically with a glob, without catching what we shouldn’t, we use the glob.

@vitorgalvao all done except for the hipchat Cask, which states:
# TODO: expand/glob for '~/Library/<userid>/HipChat/'

I am not sure this makes sense?

@miccal '~/Library/*/HipChat/' might work, but it’d have to be tested.

@vitorgalvao thanks for that - I do not have a testing machine at the moment, but I can test it in a few days.

# TODO: expand/glob for '~/Library/<userid>/HipChat/'

does not make sense to me either. These files were present after installing, starting, but not really doing anything, and then stopping HipChat:
~/Library/Application Support/com.apple.sharedfilelist/com.apple.LSSharedFileList.ApplicationRecentDocuments/com.hipchat.hipchat.sfl
~/Library/Application Support/com.hipchat.HipChat
~/Library/Caches/com.crashlytics.data/com.hipchat.HipChat
~/Library/Caches/com.hipchat.HipChat
~/Library/Caches/Homebrew/Cask/hipchat--4.29.0-732.zip
~/Library/Logs/HipChat
~/Library/Preferences/com.hipchat.HipChat.plist

While '~/Library/*/HipChat/' works fine (removes e.g. ~/Library/Logs/HipChat, which I had removed from the zaplist before testing), it is not needed.

@neutric by "stopping" to you mean just quitting the app, for something more?
Do you mean that ~/Library/<userid>/HipChat/ was created while the app is running, and deleted when stopped?

By stopping I mean quitting, yes. What I wanted to say is that all I did was install, start the app so it can create its files, and then quit it. It is possible that more files are created once you really use the app, which I didn't. Then I searched with find . -iname "*hipchat*" which produced the above list of files. Nothing that matches ~/Library/<userid>/HipChat/, so a '~/Library/*/HipChat/' in the zap list is probably not necessary.

Ah OK - I guess that the <userid> is a user's login name. You did not create an account and log into the app, I assume?

Correct. But why on earth would it create a folder <userid>/HipChat/ under ~/Library? HipChat/<userid> would make sense to me, but the other way around?

But why on earth would it create a folder <userid>/HipChat/ under ~/Library/?

Bad design?

-.-

I can conform @neutric's finding - there is no ~/Library/<userid>/HipChat/ created when installing, and creating a login is too complicated.
@Tenzer, @chaseroden, @cliffcotino you have all updated this Cask in the past - are you able to shed any light on this?

[jeppe@reventon ~/Library]$ find . -iname '*hipchat*'
./Application Support/com.apple.sharedfilelist/com.apple.LSSharedFileList.ApplicationRecentDocuments/com.hipchat.hipchat.sfl
./Application Support/com.hipchat.HipChat
./Application Support/Google/Chrome/Default/Local Storage/<website>_0.localstorage
./Application Support/Google/Chrome/Default/Local Storage/<website>_0.localstorage-journal
./Caches/com.apple.nsurlsessiond/Downloads/com.hipchat.HipChat
./Caches/com.crashlytics.data/com.hipchat.HipChat
./Caches/com.hipchat.HipChat
./Caches/HipChat
./Caches/io.fabric.sdk.mac.data/com.hipchat.HipChat
./HipChat
./Logs/HipChat
./Preferences/com.hipchat.HipChat.plist
./WebKit/com.hipchat.HipChat
./WebKit/com.hipchat.HipChat/WebsiteData/LocalStorage/<website>._0.localstorage
./WebKit/com.hipchat.HipChat/WebsiteData/LocalStorage/<website>._0.localstorage-shm
./WebKit/com.hipchat.HipChat/WebsiteData/LocalStorage/<website>._0.localstorage-wal

Note that I have normalised the lines which contain <website>. They originally contained the URL of the internal, self hosted HipChat instance we use at work, and I don't want to make the hostname of that public.

Thank you @Tenzer for your help.
@vitorgalvao it seems that there is no glob expansion required for the hipchat Cask, so I think we can call this completed?

Thank you @miccal!

You're welcome as always @vitorgalvao.


Pull Request UI: Missing Hover / No Action On Click

While there is an action on click, I think the hover could say what it means or what would happen when I click on it.
Maybe "Change reviewers" would do it.
Showing a hover just for the icon is confusing, as it is unrelated to the click action:

Icon alignment will be fixed by https://github.com/microsoft/vscode-pull-request-github/pull/5101, as will the not helpful hover for the icon.

Fixes have been merged.


Error thrown when trying to turn on/off the screensaver (or other switches in the integration)

I have two Fully Kiosk's running here.
One is the browser version, the other is the single app version.
The error is thrown for both versions.
HA is 'core' 2022.05.05
Integration is 1.02
The integration shows quite a few switches and buttons.
In the later versions of Home Assistant all the switches are starting to show errors like the one below
I'm not sure if this change has anything to do with it?
https://developers.home-assistant.io/blog/2022/04/20/saying-goodbye-to-event_time_changed/
Error shown in log:
Logger: homeassistant.helpers.script.websocket_api_script
Source: custom_components/fullykiosk/sensor.py:115
Integration: Fully Kiosk Browser (documentation, issues)
First occurred: 15.05.06 (1 occurrences)
Last logged: 15.05.06

websocket_api script: Error executing script. Unexpected error for call_service at pos 1: Unknown string format: 13/05/2022 11.42.24 PM
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 447, in _async_step
    await getattr(self, handler)()
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 680, in _async_call_service_step
    await service_task
  File "/usr/src/homeassistant/homeassistant/core.py", line 1627, in async_call
    task.result()
  File "/usr/src/homeassistant/homeassistant/core.py", line 1664, in _execute_service
    await cast(Callable[[ServiceCall], Awaitable[None]], handler.job.target)(
  File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 204, in handle_service
    await service.entity_service_call(
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 677, in entity_service_call
    future.result()  # pop exception if have
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 964, in async_request_call
    await coro
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 714, in _handle_entity_call
    await result
  File "/config/custom_components/fullykiosk/switch.py", line 91, in async_turn_off
    await self.coordinator.async_refresh()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 168, in async_refresh
    await self._async_refresh(log_failures=True)
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 270, in _async_refresh
    update_callback()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 533, in async_write_ha_state
    self._async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 571, in _async_write_ha_state
    state = self._stringify_state(available)
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 539, in _stringify_state
    if (state := self.state) is None:
  File "/config/custom_components/fullykiosk/sensor.py", line 115, in state
    time = parser.parse(self.coordinator.data[self._sensor])
  File "/usr/local/lib/python3.9/site-packages/dateutil/parser/_parser.py", line 1368, in parse
    return DEFAULTPARSER.parse(timestr, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/dateutil/parser/_parser.py", line 643, in parse
    raise ParserError("Unknown string format: %s", timestr)
dateutil.parser._parser.ParserError: Unknown string format: 13/05/2022 11.42.24 PM

Same issue (I think) as https://github.com/cgarwood/homeassistant-fullykiosk/issues/103#issue-1240433427.
Rolling back to 1.0.1 makes it work again.

same issue here... but looks like that it is working... when turning screens on or off the above error messages are logged.
rolling back to 1.0.1 didnt help...
HA: Home Assistant OS - core-2022.5.5

That didn't work for me, just tried it, if I go to 1.0.2, I can't disable the last app restart entity, only delete it, and after a reboot, it is recreated. So the error is still thrown

hat didn't work for me, just tried it, if I go to 1.0.2, I can't disable

That did the trick for me.... Never the less this should be fixed


Implementation and readme changes for StrongOracle probe #134

(also fixes a typo in README.md)

Looks good, merging.

Thanks Howard.


postgresqlPackages.pgvector: 0.1.2 -> 0.1.4

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 123614 at dcd14935 run on x86_64-linux 1

2 packages built successfully:

postgresqlPackages.pgvector (postgresql11Packages.pgvector)
tests.trivial

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/servers/sql/postgresql/ext/pgvector.nix:16:3:
   |
16 |   installPhase = ''
   |   ^


t = S / V

lolo

We do not want the gh-pages branch merged to the master branch.

We do not want the gh-pages branch merged to the master branch.


input tails (fluent-bit) causing logs to stop being generated entirely and 2) buffers not being filled up

hello, I have been facing some issues configuring the logging operator specifically  1) input tails (fluent-bit) causing logs to stop being generated entirely and 2) buffers not being filled up. My environment is disconnected openshift.  I have installed the operator CRDs from the official helm chart.
I have a clusterflow, clusteroutput on my cluster as well which sends log to kafka, adding of input tails as per the below stops logs from being generated entirely and  my /buffer does not seem to be filling up despite debugging the collector pods to check.
I have to admit i'm quite new to using helm and k8s and in general and have been very confused as to how to edit the fluentbit.conf file / my logging operator configuration.
What steps can I take to solve these 2 issues?  And what would be the best practice of adjusting configurations using helm? would it be editing the kind: logging resource or otherwise?
Charts: https://github.com/kube-logging/logging-operator/tree/master/charts/logging-operator
my logging operator:
||apiVersion: logging.banzaicloud.io/v1beta1 kind: Logging metadata: spec: ...   fluentbit:     bufferStorage:       storage.path: /buffers     # inputTail:     #   Path: /var/log/containers/*.log     #   Path: /var/log/audit/kube-apiserver/*.log     #   Path: /var/log/audit/oauth-apiserver/*.log     #   Path: /var/log/audit/openshift-apiserver/*.log     security:       securityContext:         privileged: true     image:       repository: <private repo>       tag: 1.9.5-debug||
my fluentbit.conf file:
`||[SERVICE]
Flush        1
Grace        5
Daemon       Off
Log_Level    info
Parsers_File parsers.conf
Coro_Stack_Size    24576
storage.path  /buffers
[INPUT]
Name         tail
DB  /tail-db/tail-containers-state.db
DB.locking  true
Mem_Buf_Limit  5MB
Parser  cri
Path  /var/log/containers/.log
Refresh_Interval  5
Skip_Long_Lines  On
Tag  kubernetes.
[FILTER]
Name        kubernetes
Buffer_Size  0
Kube_CA_File  /var/run/secrets/kubernetes.io/serviceaccount/ca.crt
Kube_Tag_Prefix  kubernetes.var.log.containers
Kube_Token_File  /var/run/secrets/kubernetes.io/serviceaccount/token
Kube_URL  https://kubernetes.default.svc:443
Match  kubernetes.*
Merge_Log  On
Use_Kubelet  Off
[OUTPUT]
Name          forward
Match         *
Host          ocp-fluentbit-logging-root-fluentd.fluentbit.svc.cluster.local
Port          24240
Retry_Limit  False||`

Logs from my collector pods
Fluent Bit v1.9.5

Copyright (C) 2015-2022 The Fluent Bit Authors
Fluent Bit is a CNCF sub-project under the umbrella of Fluentd
https://fluentbit.io

[2023/08/15 07:46:58] [ info] [fluent bit] version=1.9.5, commit=9ec43447b6, pid=1
[2023/08/15 07:46:58] [ info] [storage] version=1.2.0, type=memory+filesystem, sync=normal, checksum=disabled, max_chunks_up=128
[2023/08/15 07:46:58] [ info] [storage] backlog input plugin: storage_backlog.1
[2023/08/15 07:46:58] [ info] [cmetrics] version=0.3.4
[2023/08/15 07:46:59] [ info] [input:storage_backlog:storage_backlog.1] queue memory limit: 95.4M
[2023/08/15 07:46:59] [ info] [filter:kubernetes:kubernetes.0] https=1 host=kubernetes.default.svc port=443
[2023/08/15 07:46:59] [ info] [filter:kubernetes:kubernetes.0]  token updated
[2023/08/15 07:46:59] [ info] [filter:kubernetes:kubernetes.0] local POD info OK
[2023/08/15 07:46:59] [ info] [filter:kubernetes:kubernetes.0] testing connectivity with API server...
[2023/08/15 07:46:59] [ info] [filter:kubernetes:kubernetes.0] connectivity OK
[2023/08/15 07:46:59] [ info] [sp] stream processor started
[2023/08/15 07:46:59] [ info] [input:tail:tail.0] inotify_fs_add(): inode=107293714 watch_fd=1 name=/var/log/containers/dns-default-vkvtl_openshift-dns_dns-3105f9d30d6ffcc83f8a6f24e6052d63cda486d5e14babd193d0a039c5b7c9bb.log
[2023/08/15 07:46:59] [ info] [input:tail:tail.0] inotify_fs_add(): inode=107430649 watch_fd=2 name=/var/log/containers/dns-default-vkvtl_openshift-dns_dns-48d5869d06a665154b9a743df42e7f60b7837384a5c9eb8831896bfc0db85efd.log

kubernetes version: 1.25
logging-operator version: 1.95-debug
helm chart version: 3.17.10

@dariuslwh95 I would recommend upgrading to fluentbit 2.x. I had similar issues with the 1.x series and there were tons of reports on the fluentbit repository.
The only workaround I found was regular restarts of the fluentbit agents.

I'm still setting this happen as of fluent-bit 2.10.  I had also set storage.type: filesystem for InputTail, but it doesn't appear to actually set this in the fluent-bit.conf in the configMap, so I'm wondering if that setting might be broken, so it's defaulting to the in memory buffer which is getting overrun.

Please note that it is an old version and we are committed to support the last stable version only which is 4.4
As far as we are concerned rancher is not commited to upgrading the logging operator, but we have written a blog post on how you can upgrade your rancher based logging app, see: https://axoflow.com/get-the-latest-logging-operator-in-rancher/
@dariuslwh95 the chart you linked is the latest chart version which is very different from the 3.17 version.
On the other hand I wouldn't recommend to configure the logging resource from the chart. There is limited support to do that, but I would recommend putting the logging manifests into your own chart templates and only depend on the logging operator chart to install the operator itself. For example you wouldn't install cert-manager certificates with the cert-manager chart, would you?
If you insist on using the chart then you can use the logging field to configure the logging resource and logging.fluentbit to configure the fluentbit spec, see:

https://github.com/kube-logging/logging-operator/blob/master/charts/logging-operator/templates/logging.yaml
https://github.com/kube-logging/logging-operator/blob/master/charts/logging-operator/templates/logging_fluentbit.yaml

If you want to have a chat please reach out over the project discord: https://discord.gg/sXFdDKEKK8


Spacing in header

Can you match the spacing of the 2 orange spacers:

Top space still looks too big


Safer common.js module system detection

Sanctuary is using this code to detect common.js module system:
   if (typeof module !== 'undefined') {
      module.exports = f(require('ramda'), require('sanctuary-def'));
   }

But this is definitely not enough. Because 'module' is a common variable name and could easily be in use not by the common.js module system. For example in my angular application it is used by 'angular-mocks.js' to inject angular module dependencies. So sanctuary causes an error in my environment.
My decision to check for 'module.exports' was inspired by:
http://code.jquery.com/jquery-2.2.2.js
https://raw.githubusercontent.com/lodash/lodash/4.6.1/dist/lodash.js
https://github.com/jashkenas/underscore/blob/master/underscore.js

Thank you, @kahless! Would you like to open similar pull requests for sanctuary-def and sanctuary-int?


THRIFT-5494: fix cpu full caused by infinite select() when frameSize < maxReadBufferBytes but readBufferBytesAllocated.get() + frameSize always greater than MAX_READ_BUFFER_BYTES

We encountered a problem that caused the CPU to fill up (24 thread, 40 core), it seems to be due to the readBufferBytesAllocated not being properly calculated and the frameSize is large.
So i think that even if the variable is correctly evaluated, it is necessary to add a separate parameter limit to the frameSize, rather than reusing the maxReadBufferBytes.
ps. this large frame should be come from security check detection, also it could be attack
24 core is full

big request caused inifinite loop

[ x ] Did you create an Apache Jira ticket?  (not required for trivial changes)
[ x ] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[ ] Did you squash your changes to a single commit?  (not required, but preferred)
[ ] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[ ] If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

Is it a good idea to use TConfiguration maxFrameSize？

static final constant DEFAULT_MAX_FRAME_SIZE is not able for customization.
so i have to use trans_.getConfiguration().getMaxFrameSize(), but that means many class need to be modified(TNonblockingSocket, TNonblockingServerSocket, AbstractServerTransportArgs and NonblockingAbstractServerSocketArgs)

For the sake of consistency, we should aim to use
DEFAULT_RECURSION_LIMIT = 64;
DEFAULT_MAX_MESSAGE_SIZE = 100 * 1024 * 1024; // 100 MB
DEFAULT_MAX_FRAME_SIZE = 16384000; // this value is used consistently across all Thrift libraries

For the sake of consistency, we should aim to use
DEFAULT_RECURSION_LIMIT = 64;
DEFAULT_MAX_MESSAGE_SIZE = 100 * 1024 * 1024; // 100 MB
DEFAULT_MAX_FRAME_SIZE = 16384000; // this value is used consistently across all Thrift libraries

Is it a good idea to use TConfiguration maxFrameSize？

static final constant DEFAULT_MAX_FRAME_SIZE is not able for customization.
so i have to use trans_.getConfiguration().getMaxFrameSize(), but that means many class need to be modified(TNonblockingSocket, TNonblockingServerSocket, AbstractServerTransportArgs and NonblockingAbstractServerSocketArgs)

if i want use it in 0.9.1, i must

Using 0.9.1 is discouraged anyways, there are a number of CVEs between that version and today. If you make the decision for yourself, that's perfectly fine. But you also live with all the consequences of such decision, because you work on your own fork then. Wouldn't recommend it, but it's a free country, you know.

static final constant DEFAULT_MAX_FRAME_SIZE is not able for customization.

Right, that's why it is called DEFAULT_xxx and not "current frame size" or the like. It's the default when you don't specify anything else.

+1 LGTM

Starting a Gradle Daemon (subsequent builds will be faster)

This failed because AbstractServerTransportArgs.maxFrameSize is not init.
But there seems to be some checks that cannot be pass. Could you please help to check it?

Using 0.9.1 is discouraged anyways

Our project have many clients exist in projects that other people are responsible for. So each upgrade takes a very long time. Most likely it won't upgrade for now.

Do they also use old log4j versions for the same reason? Anything else vulnerable you want to make publicly known?

Actually, I think this problem also exist in the latest version. But I don't have much time to reproduce. So If anyone else is experiencing this problem(CPU is full and consumed by select()), they could try use this PR to fix it.


axis tick values and tick size is not exposed

as a result you cannot set explicit tick values, or configure the length of ticks.

fixed by #27


Dictionary error

Everytime I try to use in-plugin commands (or rocket commands like, /i) I get a dictionary error and it doesn't complete the command.
Example with Rocket "/i"
Example with ZaupShop "/buy"
And pretty much this error shows up in any command (that uses item names and item IDs) that is not from Vanilla Unturned itself. (Rocket Build: 798, 800 both have the error.)

Same.

I was able to fix it on my end, it's the references Rocket.Unturned and zaupshop uses, they're out of date, it works properly when it's compiled against the newest references.

Same here
capture

I uploaded a build of Rocket.Unturned and ZaupShop with the updated references. Built against build 801.
Note: Rocket.Unturned is in a windows only build in this, this build of Rocket.Unturned will most likely not work on a Linux server.
http://sc.cartman-2000.net/files/updates.7z

@cartman-2000 I had an outdated version of zaupshop, which caused the library error, but now I updated, and the error doesn't show up anymore, just for /i though... going to use uEssentials /item though... thanks for the help anyways :)

The Rocket.Unturned one should work if you replaced the one in the managed folder properly.

Still an issue on:
Unturned <IP_ADDRESS>
Rocket 801
However, works with cartman-2000's Unturned.dll.

Fixed since yesterday


前置阅读 | Pre-reading

[X] 文档/Doc
[X] Issues
[X] FAQ

PicList的版本 | PicList Version
v1.9.6
系统信息 | System Information
Windows
问题重现 | Bug reproduce
拖动图片到上传区域上传成功后，在预览显示的相册界面无法正常预览，查看文件名称后发现在文件名尾部追加了“undefined”字符，我不太懂js代码，不知道在哪里修改这个。

相关日志 | Logs
http://maskmaskmask/TLS握手协议图.webpundefined
2023-07-11 11:39:53 [PicList INFO] [PicList Server] upload result http://maskmaskmask/TLS握手协议图.webpundefined
2023-07-11 11:48:11 [PicList INFO] Compress or convert image
2023-07-11 11:48:11 [PicList INFO] Before transform
2023-07-11 11:48:11 [PicList INFO] Transforming... Current transformer is [path]
2023-07-11 11:48:11 [PicList INFO] Before upload
2023-07-11 11:48:11 [PicList INFO] beforeUploadPlugins: renameFn running
2023-07-11 11:48:11 [PicList INFO] Uploading... Current uploader is [qiniu]
2023-07-11 11:48:12 [PicList SUCCESS]
http://maskmaskmask/算法协商图1.webpundefined
2023-07-11 11:48:15 [PicList INFO] Compress or convert image
2023-07-11 11:48:15 [PicList INFO] Before transform
2023-07-11 11:48:15 [PicList INFO] Transforming... Current transformer is [path]
2023-07-11 11:48:15 [PicList INFO] Before upload
2023-07-11 11:48:15 [PicList INFO] beforeUploadPlugins: renameFn running
2023-07-11 11:48:15 [PicList INFO] Uploading... Current uploader is [qiniu]
2023-07-11 11:48:16 [PicList SUCCESS]
http://maskmaskmask/算法协商图2.webpundefined

我大概能确定问题在哪，会在下个版本修复

非常感谢😀


[CRITICAL] [$250] #announce room should default to 'Admins Only' for posting

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: v1.4.73-7
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers): N/A
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL: N/A
Issue reported by: @garrettmknight
Slack conversation: https://expensify.slack.com/archives/C05RECHFBEW/p1715886822520169
Action Performed:

Create a workspace
Navigate to the #announce room for your new workspace
Open the chat settings

Expected Result:
Who can post should be defaulted to Admin only as it was designed in this doc
Actual Result:
Who can post is set to All members
Workaround:
You can set it to admins only.
Platforms:
All

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

Upwork Job URL: https://www.upwork.com/jobs/~01d20a5670a3bdd03e
Upwork Job ID:<PHONE_NUMBER>450966528
Last Price Increase: 2024-05-21
Automatic offers:

nkdengineer | Contributor | 102491174

Issue OwnerCurrent Issue Owner: @eVoloshchak

Proposal
Please re-state the problem that we are trying to solve in this issue.
Who can post is set to All members
What is the root cause of that problem?
When we create a new workspace, the announce room has writeCapability as undefined in optimistic data and then BE returns it as all.
https://github.com/Expensify/App/blob/e8ae3c5acedf0e6788dc574c7b6f3043ca37092a/src/libs/ReportUtils.ts#L4710
What changes do you think we should make in order to solve the problem?
We should pass writeCapability as CONST.REPORT.WRITE_CAPABILITIES.ADMINS here to fix the offline case for announce room
https://github.com/Expensify/App/blob/e8ae3c5acedf0e6788dc574c7b6f3043ca37092a/src/libs/ReportUtils.ts#L4710
And BE also need to return writeCapability of announce room as admin in CreateWorkspace API
What alternative solutions did you explore? (Optional)
NA

The fix is pretty straightforward, @nkdengineer's proposal looks good to me!
Changes to BE would also be needed
🎀👀🎀 C+ reviewed!

@nkdengineer's proposal LGTM.
I'll take care of the BE changes.

We have a draft PR here
@lakchote please let me know when BE is done

🎀👀🎀 C+ reviewed!

Lucien is on parental leave. @miljakljajic Let's find another internal engineer to sort out the BE changes.

🎀👀🎀 C+ reviewed!

🎀👀🎀 C+ reviewed!

@eVoloshchak can you type this again, Lucien is on leave and we're trying to trigger a new internal engineer:
🎀👀🎀 C+ reviewed!

Following this one as it's a very big pain point for Recorded Future that their non-admins can post to the #announce room by default.

🎀👀🎀 C+ reviewed!

@danieldoglas are you good to assist with the BE changes required here?

Yes, I will.

Backend PR created

This is WIP, we need to change some automated tests on web so we can deploy it.

work continues on this one

Not overdue, latest update: https://github.com/Expensify/App/issues/42417#issuecomment-2175828711

Yep, still working, other things took priority for now.

still WIP

Still need to work on the Web part of the changes, Auth is done.

Still working

Yep. Lower priority compared to fast APIs now

I'll be OOO during the next 2 weeks - I'll finish this one once I'm back.

back from OOO

@danieldoglas friendly bump

Sorry, there are some other competing priorities which are taking some time before this.

All backend PRs were created. We need to wait for the reviews/deploys - probably done by the end of the week

Last backend PR was merged, should be deployed Monday and we can close this.

This is done in the backend. @nkdengineer all yours to finish the app pr

@danieldoglas thank you, I'm working on this PR now

@danieldoglas Currently, after creating a new workspace, I am unable to access the announce room.
https://github.com/user-attachments/assets/2f7cf737-e4a9-4144-9f2e-d680b5f96365

You need your workspace with at least 4 people to have an announce room. This was changed recently.

@eVoloshchak this PR is ready for review

Merged, awaiting delpoy - should be deployed to staging today.

seems like this PR went out over 7 days ago, so I've paid @nkdengineer and providing a payment summary below for @eVoloshchak
@eVoloshchak is owed 250 for their work reviewing this issue.

If there are any further actions to take, please assign another BZ team member - I am out on extended leave from the end of the day.

The PR that introduced the bug has been identified. Link to the PR: no PR that has caused this, this is an initial implementation on the back end
The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: N/A
A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: Not needed, it isn't a bug per se, rather a change in expected behavior
Determine if we should create a regression test for this bug.
Is it easy to test for this bug? Yes
Is the bug related to an important user flow? Yes
Is it an impactful bug? No
We don't need a regression test here, this will no break as long as back end fix is not reversed

$250 approved for @eVoloshchak

Opening back up for the regression test.

Milja is out so I'll take this.


Conditions: cron "0/20 * * * " is not same as "/20 * * * *"

Describe the bug
"0/20 * * * " cron pattern launches my tasks hourly while "/20 * * * *" launches tasks every 20 minutes
There is a screenshot of my log stats from heroku for 0/20 * * * * pattern

there is one for */20 * * * *

Expected behavior
I expect these patterns to behave in the same way
Desktop (please complete the following information):

OS: Windows 10 Professional
Python version: 3.11.6

requirements.txt:
pydantic==1.10.10
fastapi
rocketry
requests
boto3
uvicorn[standard]

Hey, man, "0/20 * * * * *" is not standard cron. Heroku ignores "/20".
rontab.guru

@zzl221000
Thanks for your response. I've already changed it to the standart pattern and it works as expected


Bug: failed pushChanges with OpenAPI Connection

Describe The Bug
Can't access to any SwitchBot Devices (bot, meter, etc.) from homekit
To Reproduce
I've reset the hub and both Bot from the Switchbot app and reinstalled everything. Then I set the new token in Homebridge. Still errors below.
Expected behavior
I should be able to access my switchbot devices
Relevant log output
[07/12/2021, 19:21:55] [SwitchBot] Initializing SwitchBot platform...
[07/12/2021, 19:21:55] [SwitchBot] Using Default Refresh Rate (2 minutes).
[07/12/2021, 19:21:55] [SwitchBot] Using Default Push Rate.
[07/12/2021, 19:22:09] [SwitchBot] Total SwitchBot Devices Found: 3
[07/12/2021, 19:22:09] [SwitchBot] Restoring existing accessory from cache: Mario DeviceID: C2492CF8A79E
[07/12/2021, 19:22:09] [SwitchBot] Restoring existing accessory from cache: Jean DeviceID: D00DB4DE8135
[07/12/2021, 19:22:09] [SwitchBot] Total IR Devices Found: 0
[07/12/2021, 19:22:09] [SwitchBot] Bot: Mario2 Displaying as Outlet
[07/12/2021, 19:22:09] [SwitchBot] Bot: Jean2 Displaying as Outlet
[07/12/2021, 19:22:20] [Timer Chaufferette] Starting the Timer

WHEN I PRESS THE BUTTON IN HOMEKIT:
[07/12/2021, 19:22:41] [SwitchBot] Bot: Jean2 failed pushChanges with OpenAPI Connection

WHEN I PRESS THE OTHER BUTTON IN HOMEKIT:
[07/12/2021, 19:22:53] [SwitchBot] Bot: Mario2 failed pushChanges with OpenAPI Connection

Config for homebridge-switchbot
{
            "name": "SwitchBot",
            "credentials": {
                "openToken": "XXXXc182a41fadc8aeaea298856e818c3c904de2dc885c568f94f8f9c711a24b4cd75f7f464a22c42a541ea76404f364",
                "notice": "Keep your token a secret!"
            },
            "platform": "SwitchBot"

Screenshots
No response
Device & Model
iPhone X, HomePod Mini, Switchbot hub mini, 2 switchbots
Node.js Version
v16.13.1
NPM Version
v8.1.2
Homebridge Version
v1.3.8
Homebridge Switchbot Plugin Version
v1.3.0
Homebridge Config UI X Plugin Version
v4.41.3
Operating System
Debian on Raspberry pi

You can close the issue. I needed to configure the "options", like this:
"options": {
"devices": [
{
"deviceId": "D00DB4DE8135",
"configDeviceType": "Bot",
"bot": {
"mode": "press",
"deviceType": "switch"
}
},


Fork() and Waitpid() Simulator

About
This is an animation to help beginner C programmers visualize the fork() and waitpid() system calls. It includes multiple types and functions that are helpful for creating animated line segments with text labels using Reanimate. If included in the examples folder, programmers may find my code useful for creating systems programming related teaching resources or more generally, animations with multiple line segments and text labels.
Applications
This is one of the 6 animations created for the website: fork-resource.

That's very cool. I have a few suggestions:

Use signalA (curveS 2) to get smoother animations.
Draw text with withStrokeWidth 0 to get sharper text. Drawing text with latex with a non-zero stroke width tends to be less sharp.
Use & strokeLineCap .~ pure CapSquare to make the lines meet up neatly.
Use oFadeIn instead of staticFrame to make every line appear smoothly.

Here's a quick mockup (text should ideally be bigger):
https://user-images.githubusercontent.com/1443107/173179602-3e87b2cf-6282-40fd-833c-d691824d8d4c.mp4

Thank you so much for your feedback! I incorporated your changes but I had some trouble with using withStrokeWidth 0. I'm a Haskell Beginner (and a Reanimate Beginner) so please bear with me if my questions/confusion seems self-explanatory!
When I included withStrokeWidth 0 in my outlinedText function, my white outline disappeared. When I added  withStrokeWidth 0 to my mkTextLabel function, my text disappeared. So then, I changed the stroke width to a smaller value to make my text appear thinner and more similar to the animation that you created. I think I may have misunderstood what you meant by use withStrokeWidth 0 since it hasn't given me the desired effect. I'd appreciate any feedback/advice you have.
Thank you so much for your time!

You can change center $ latex txt to withStrokeWidth 0 $ withFillOpacity 1 $ center $ latex txt. Then the text will be filled rather than stroked.

That worked for me!

I appreciate all of your help and feedback. I look forward to hearing any other improvements you may have!

Nice!


Drop CSS driven order of PLP slots

So far, the various slots on listing pages (category and search) have been ordered with a combination of a layout configuration and order rules in CSS (flexbox). There is no good reason to do so, and it breaks accessibility. Using the keyboard is breaking the experience, since the re-ordered visual layout(CSS) does not align with the real DOM.
As part of accessibility improvements and a larger refactor of the facet navigation (#6581) we will change it to a simplified setup (simple DOM, simple CSS).
BREAKING CHANGE:

the b2cLayoutConfig will change, which will produced a change in the DOM for the category and search pages
the CSS layout for the category and search pages will no longer use specific "order" for the layout.

QA

open PLP (category and search) in list and grid mode, ensure that all slots are rendered in the expected order
ensure that the order is not driven by CSS, you could verify it in the DOM and validate that the DOM order is the order of slots.

Tested on dev10.
PLP from category and search, looks ok, for both list and grid view,
Slots in DOM are in this order:
PLP from category:

PLP from search:


Error about  TimeDistributedDense

model.add_node(TimeDistributedDense(2), name='T', inputs=[ 'c', 'w'], model='concat')
It raise the error  'ValueError: total size of new array must be unchanged'
How to deal with the problem like this

I just want to first point out that I seems it should be 'mode' instead of 'model' at last.


APIGOV-19290 - call out env var when template is not valid

Looks good. Makes very good sense for the error variables.  Easier check 🤙


normalize and document data models

Review all models in the ledger branch, and make suggestions for normalization. Document the before and after normalization schemas, including graphics depicting foreign keys and other relationships.

Git Guild Bounty Program

Total
Dev
Sales
Reporter
Tester(s)
Maintainer
Documenter

N/A
assignee(s)
@isysd
@isysd
@cindy-zimmerman, @d3brouille
@cindy-zimmerman
@d3brouille

$1665
$999
$83.25
$83.25
$333
$83.25
$83.25


ci: fix release

Fixes no issue
Description

Checklist

[x] PR is rebased to/aimed at branch develop
[x] PR follows Contributing Guide
[x] Added tests (if necessary)
[x] Extended README/Documentation (if necessary)
[x] Adjusted versions of image and Helm chart in values.yaml and Chart.yaml (if necessary)

@dependabot squash and merge


Support running run.sh from any current working directory

When stable package installs to user's $HOME/jitsi, currently, run.sh starts successfully only when CWD is $HOME/jitsi.
Don't know for which other cases run.sh is used, please take care to check.

I'm not sure if cd in a script is a good idea. @damencho ?

I think it's better to run everything within the script relatively, e.g.:
export SCRIPTPATH=$( cd $(dirname $0) ; pwd -P )
then every subsequent relatively pathed script invocation/variable can be prefixed with SCRIPTPATH, and you don't trash your cwd. no pushd or popd, either.

@ibauersachs I agree with @narfman0, in jicofo and jvb we already have:
SCRIPT_DIR="$(dirname "$(readlink -f "$0")")"

@andrey-utkin could you please rework this to not use cd?

Feel free to rework in any way you like without worries about my authorship.

I don't have time to work on this. Please reopen or create a new PR if you can do the changes as suggested in the comments.


Dispatch projection update async

We don't need to wait for this to complete and doing so diffuses the
metrics on how long the TakerConnected handlers takes. It also blocks
things like the OfferParams message.

bors r+


WIP: Add template parameter for endinanness in Serial.hpp

Description
Also refactor Serial.hpp a bit.
Fixes #34

Codecov Report
Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (8675469) 77.20% compared to head (796bd15) 77.22%.

Files
Patch %
Lines

Sts1CobcSw/Serial/Serial.hpp
69.56%
7 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #110      +/-   ##
==========================================
+ Coverage   77.20%   77.22%   +0.01%
==========================================
  Files          16       16
  Lines         351      382      +31
==========================================
+ Hits          271      295      +24
- Misses         80       87       +7

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Added new participant

Not sure if you did that with a local branch, or directly on the GitHub UI. But just to be clear, I don't care much about the changes to the README file, what I really care about is that you learn how to deal with branches, forks, conflicts... with very simple changes, so when you have bigger PRs, you're very familiar with it. :)


teleop_twist_joy: 0.1.3-1 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository teleop_twist_joy to 0.1.3-1:

upstream repository: https://github.com/ros-teleop/teleop_twist_joy.git
release repository: https://github.com/ros-gbp/teleop_twist_joy-release.git
distro file: noetic/distribution.yaml
bloom version: 0.9.7
previous version for package: null

teleop_twist_joy
* Use industrial ci
* Don't crash with invalid axes.
* Make sure to not crash when the Joy message buttons is too small.
* Don't get the axis twice.
* Add ROS Wiki link (#26 <https://github.com/ros-teleop/teleop_twist_joy/issues/26>)
* Contributors: Chris Lalancette, Julian Gaal, Rousseau Vincent, vincentrou

holding for the upcoming Noetic sync


Error with cocoa pods plz help

Hi, im new in Xcode, im trying to install google firebase with cocoa pods and when going to compile I have this issue
Apple Mach-O Linker (ld) Error Group
clang: error: linker command failed with exit code 1 (use -v to see invocation)
when I run pod install I have this warnings too
[!] The reactApp [Debug] target overrides the FRAMEWORK_SEARCH_PATHS build setting defined in Pods/Target Support Files/Pods-reactApp/Pods-reactApp.debug.xcconfig'. This can lead to problems with the CocoaPods installation     - Use the $(inherited)` flag, or
- Remove the build settings from the target.
please be very specific with yours answers, remember im noob with Xcode

Same issue I can't find the problem.


[ARPA P&E Reporting] Enhanced Workbook Data Validation Checks

Submitted Description:

Implement error flag or notification when Total Cumulative Expenditures exceed Total Cumulative Obligations in the EC tab.
Implement error flag or notification when subaward IDs are duplicated in the Awards >50000 tab.
Implement error flag or notification when Subaward Award Date in the Awards>50000 tab occurs after the Expenditure Start Date in the Expenditures >50000 tab.
Implement warning flag or notification when the sum of Current Quarter Expenditure/Payments in the Aggregate Awards 50000 tab do not equal the Current Period Expenditures in the EC tab.

(Optional) Uploaded file - click to download:

when reviewing #394 determine if this is a duplicate or not as this is an improvement for upload validation


how to get the request totaluse stream request

the stream chat completion request method(openAiService.streamChatCompletion) can only get the stream content, but not this request total token. which can be got by the OpenAiService#createCompletion(CompletionRequest) method. I want to use the stream method, and get the consumed total token, anyone could tell me how to do it?


Add a test to verify that redactions are sent out over federation without having the original event

This is a test to verify that synapse #13813 works. Tests that redaction events are sent over federation even if the originating server does not have the event to be redacted. More discussion on this can be found at https://github.com/matrix-org/synapse/issues/12795.

Irritatingly the synapse run is beset with flakes but the test in question is passing. I'm also unsure if this test is something that makes sense to merge, given that Dendrite does not currently implement this behavior? My main goal with this was a POC of synapse #13813

I'm also unsure if this test is something that makes sense to merge, given that Dendrite does not currently implement this behavior?

Good question. I guess the pertinent question is: is this behaviour (being able to redact something you've not seen yet) important enough for all HSes to do?

My main goal with this was a POC of synapse #13813

Related: https://github.com/matrix-org/complement/issues/226

Discussed yesterday in a meeting.

is this behaviour (being able to redact something you've not seen yet) important enough for all HSes to do?

Probably. It helps moderation (https://github.com/matrix-org/synapse/issues/12795). It also ensures that an event we send out locally is sent out over federation, which seems like a very sound property to uphold. (I hadn't grokked that redactions were being sent locally in #13813.)
We agreed that we'd like to land this in complement (skipping for dendrite if necessary).


I need to set ignoreCase true in Filter ( JS Grid )

I have Done this Point. No need any Answer,
Thanks


Job Status Widget with progress bar

Bold Job Status with progress % + progress bar for job which is building.

Nice widget, can be useful for long running jobs.


求问Material的m_emission和Kd项是怎么给出的

虽然这部分是代码模板已经给出的不用完成，但还是想知道。。
Metrial的m_emission和Kd项是怎么给出的？遵循什么规则？求给几个关键词就好。
// main.cpp
    Material* red = new Material(DIFFUSE, Vector3f(0.0f));
    red->Kd = Vector3f(0.63f, 0.065f, 0.05f);
    Material* green = new Material(DIFFUSE, Vector3f(0.0f));
    green->Kd = Vector3f(0.14f, 0.45f, 0.091f);
    Material* white = new Material(DIFFUSE, Vector3f(0.0f));
    white->Kd = Vector3f(0.725f, 0.71f, 0.68f);
    Material* light = new Material(DIFFUSE, (8.0f * Vector3f(0.747f+0.058f, 0.747f+0.258f, 0.747f) + 15.6f * Vector3f(0.740f+0.287f,0.740f+0.160f,0.740f) + 18.4f *Vector3f(0.737f+0.642f,0.737f+0.159f,0.737f)));
    light->Kd = Vector3f(0.65f);

找到了
http://www.graphics.cornell.edu/online/box/data.html


Letterboxing removal broken as of patch 1.0.3

Seems as of 1.0.3 which came out December 3rd letterboxing has reverted to default.

I can't seem to replicate this. Can you post your STALKER2Tweak.log file? Also is it any particular cutscene that's displaying letterboxing/pillarboxing? If you start a new game and skip the first pre-rendered video, does the in-game cutscene display letterboxing/pillarboxing?

What location is the output of the log usually in?

I can't seem to replicate this. Can you post your STALKER2Tweak.log file? Also is it any particular cutscene that's displaying letterboxing/pillarboxing? If you start a new game and skip the first pre-rendered video, does the in-game cutscene display letterboxing/pillarboxing?

What location is the output of the log usually in?

It'll be in Stalker2\Binaries\(Win64/WinGDK)\ next to the main shipping executable.

I can't seem to replicate this. Can you post your STALKER2Tweak.log file? Also is it any particular cutscene that's displaying letterboxing/pillarboxing? If you start a new game and skip the first pre-rendered video, does the in-game cutscene display letterboxing/pillarboxing?

What location is the output of the log usually in?

It'll be in Stalker2\Binaries\(Win64/WinGDK)\ next to the main shipping executable.

No such log generates. Attempted to reinstall and restart the game to generate it and no-dice.

I can't seem to replicate this. Can you post your STALKER2Tweak.log file? Also is it any particular cutscene that's displaying letterboxing/pillarboxing? If you start a new game and skip the first pre-rendered video, does the in-game cutscene display letterboxing/pillarboxing?

What location is the output of the log usually in?

It'll be in Stalker2\Binaries\(Win64/WinGDK)\ next to the main shipping executable.

No such log generates. Attempted to reinstall and restart the game to generate it and no-dice.

Can you show me a picture of the contents of your Win64 or WinGDK folder? Just to make sure you have STALKER2Tweak installed properly.

I had it installed incorrectly, apologies.


E2.02/adapt notification per project

Moves creation of E-Mail Content out of the use case
Adds NotificationContent DTO object used by...
...newly implemented use case, that collects the information for each project and subscriber, which samples have been updated

tests are not working yet due to the pom not being set up correctly (we think), but feedback is appreciated!

notification tests now work, there are some problems with the other use case tests, but I'm not sure this is needed anymore, anyway


Didn't set up the boot script for the XHYVE_EXPERIMENTAL_NFS_SHARE

Reported from https://github.com/zchee/docker-machine-driver-xhyve/issues/18#issuecomment-172080769

host: OS X
guest: boot2docker
If changed guest IP such as delete /var/db/dhcpd_leases and restart host, but does not improve the IP for NFS shared folder in /etc/exports.
In this case, folder sharing will not be enabled.


feat(Gamut): Allowed general text inputs for GridForm

Overview
I have a need for password. @jakemhiller mentioned recently that we could just use a default case for these input types. Works for me!
PR Checklist

~[ ] Related to Abstract designs:~
[x] Related to JIRA ticket: GM-9
[x] I have run this code to verify it works
[x] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-labs@1.7.4-alpha.34b45e.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5ec54f984b767101b88fcc37--codecademy-styleguide.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@1.7.5-alpha.cb5fd1.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5ec553f14ef74471f0df12bd--codecademy-styleguide.netlify.appDeploy Logs


change from sass to css-variables

Variables havent been replaced with css vars

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


app/data/local/temp/ij4_nlog_11.log

cant install the update or install the app from scratch keep getting the same error app/data/local/temp/ij4_nlog_11.log

Please copy the whole error! I can not guess the error from the file name...

C:\users\iamtheeggman\appData\local\temp\ij4_nlog_12.log
sorry I copied it sloppily thanks for your help

What is the error or does it say 'C:\users\iamtheeggman\appData\local\temp\ij4_nlog_12.log'

the file doesn't exist yet? its an error while installing so it isn't on my hard drive.
it might help you to look at the last patch or the patch before it because this same error happened to me then and you guys fixed it.

oh above the line it says an internal error occurred error 34

I am really sorry but i still can not imagine what the error looks like.
Btw the patching process is not done by us but by a program called install4j

everytime I try it the nlog_# goes up one

Honestly what is the message you get it certainly wont be a file path. I mean I never had an error installing it and so I have no idea what the error says or reads... Please tell me. But I am pretty doubtful it only states a path...

Does the C:\users\iamtheeggman\appData\local\temp\ij4_nlog_12.log not exist. If it does please upload it here

the error says error 34. the filepath does not exist on my comp yet

error code 34

Maybe start with installer with debug option as described here
So

Windows+R
Type cmd
cd pathToTheInstaller
downlordsxxxxx.exe /create-i4j-log

It should open a window which tells you where it writes it log to. Please forward that log to me

thanks it was a good idea but for some reason when I look in my downloads folder the download doesn't appear, it just appears in downloads on my internet explorer.

Well that is a problem you need to solve yourself can not really help you there

hmmm I finally just reinstalled the .3 client and downloaded the patch through the client again and it worked for some reason. must have been an issue with ie and chrome or an ai just messing with me. thanks for trying to help me.


Search feature

Two suggestions:
-Create new Search button, that works same as enter key.
-Change title of current search button to "Adv. Search", with tooltip of "Advanced Search".
e.g.
[______________________]    [Search]  [Adv. Search]

not sure whats the intent here, we cannot create a new search button that works same as enter key, thats for filtering.
search and filter are two different functionality

Clicking search opens the search dialog
Typing on filter text-box and pressing enter filter out the links on the navigation bar

feel free to recreate another issue if there is something missing in my explanation

please provide the actual intent of the UI change
explain the problem it solves
provide the pain point of the existing UI


Remove unused js-combinatorics types package

It looks like this package wasn't removed when we switched from
js-combinatorics to just-permutations.
Change-type: patch
Signed-off-by: Lucian Buzzo<EMAIL_ADDRESS>

@balena-ci I self-certify!


feat(scope-zone): new scope manager to support async operations in web

Which problem is this PR solving?

Fixes #435

Short description of the changes
This PR adds new ZoneScopeManager for web that will support async operation - will keep the reference to scope (span) between asynchronous operations. This will allow to build more plugins for web. It is created into 2 packages. One is bundled together with zone.js and second allows to use your own zone.js if user already has it (for example when building angular apps)
Updated the corresponding readme too.

cc @draffensperger

Codecov Report

Merging #461 into master will decrease coverage by 95%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #461       +/-   ##
=========================================
- Coverage      95%     0%   -95.01%
=========================================
  Files         125      4      -121
  Lines        5981    136     -5845
  Branches      497      6      -491
=========================================
- Hits         5682      0     -5682
+ Misses        299    136      -163

Impacted Files
Coverage Δ

...ages/opentelemetry-scope-zone-peer-dep/src/util.ts
0% <0%> (ø)

.../opentelemetry-core/test/metrics/NoopMeter.test.ts

...metry-core/src/trace/instrumentation/BasePlugin.ts

...entelemetry-plugin-dns/src/enums/AttributeNames.ts

...ry-plugin-dns/test/functionals/dns-disable.test.ts

...entelemetry-exporter-jaeger/test/transform.test.ts

...entelemetry-core/test/common/ConsoleLogger.test.ts

...try-core/test/trace/fixtures/test-package/index.js

...opentelemetry-base/test/resources/resource.test.ts

packages/opentelemetry-tracing/src/config.ts

... and 114 more


glob 8.x breaks Windows

On Windows, glob 8.x has changed how results are returned and fantasticon 2.0.0 is unusable.
><EMAIL_ADDRESS>icons-font
> fantasticon

No SVGs found in ./icons
ERROR: "icons-font" exited with 1.

I suggest that you either downgrade glob or normalize backslashes yourself. I personally just don't update glob to 8.x.

Confirmed, I got a "No SVGs found in xxx" error on Windows.
Version: 2.0.0

The bug is, that glob requires slashes, but path.join() creates backslashes on windows systems. In the file src/utils/assets.ts, function loadPaths(dir), create globPath as join(dir, **/*.${ASSETS_EXTENSION}).replace(/\\/g, '/'); and it should work.

The bug is, that glob requires slashes, but path.join() creates backslashes on windows systems. In the file src/utils/assets.ts, function loadPaths(dir), create globPath as join(dir, **/*.${ASSETS_EXTENSION}).replace(/\\/g, '/'); and it should work.

I just tested this, and it fixed the issue.

The same thing is happening to me. Downgrading the version to 1.2.3 helped temporaraly.

The bug is, that glob requires slashes, but path.join() creates backslashes on windows systems. In the file src/utils/assets.ts, function loadPaths(dir), create globPath as join(dir, **/*.${ASSETS_EXTENSION}).replace(/\\/g, '/'); and it should work.

I did the same! this works..

Followed same steps but still getting that error :/

For now i downgraded it to 1.2.3, and Since I was using global instance, I just installed it globally
npm i -g<EMAIL_ADDRESS>

For anyone encountering this bug: you can try migrating to @twbs/fantasticon instead. The bootstrap team has forked fantasticon and worked around this issue. It seems like this original fantasticon package is no longer maintained. I migrated my project 2 months ago and it's been working well so far.

I was really hoping we could upstream our patches, since they are mostly trivial but improve things nevertheless (comparison). Unfortunately, last time I tried upstreaming the patches, I didn't succeed so I have to keep the fork around for the time being.

For now i downgraded it to 1.2.3, and Since I was using global instance, I just installed it globally
npm i -g<EMAIL_ADDRESS>
this worked to me! thanks

Do I get this right, that the PR to fix this was not completed yet?
https://github.com/tancredi/fantasticon/pull/510
So the only way to get this to work is to downgrade to v2? The override of the glob dependency did not work for me...

2024, still not working

Soon 2025...


Bugfix in event decoder (?)

Background
While running this decoder on a sequence of tokens (midifile -> NoteSequence -> event_tokens), I realised that after every note is processed and we return to processing shifts, the cur_time is reset (since start_time is 0). This then causes an exception to be thrown from note_sequences#decode_note_event:
if time < state.current_time:
    raise ValueError('event time < current time, %f < %f' % (
        time, state.current_time))

I believe the cur_time should be an offset of state.current_time, not of start_time - this allows the decoding sequence to pick up where it left off along the timeline.

I've signed the CLA, can that build be rerun?

Can you provide a little more information on how you're using the decoder? I'm guessing it's for some kind of custom setup.
The way we're using it (as illustrated in metrics_utils.event_predictions_to_ns), the ground truth for the current time offset comes from the start_time passed into decode_events. That's then used to set state.current_time. We do this because we're decoding independently-inferred chunks of the full sequence.

We do this because we're decoding independently-inferred chunks of the full sequence.

Right, I had suspected as such. I was just trying to understand how the MT3 + note-seq libraries work and wrote some code to split a midi file into subsequences just to see if I could then reconstruct the original midi file.  So my input would be a sequence of events corresponding to the entire midi file (several minutes worth of events).
I can see how this function would work as-is for a small slice containing only a single note event + some shift events, but it seems like it will fail if it encounters multiple notes (unless there are increasingly more shifts between subsequent notes).
So perhaps it's by design - but I believe this change is still an improvement, as it should not change functionality for the 'small slice' use-case and will prevent errors in a longer slice use-case.
I have updated the description with my relevant code.

I think it still doesn't work for our case because state.current_time at the end of one chunk isn't necessarily the right start time of the subsequence chunk. For example, what if there are several chunks in a row with no note events? The start time of the chunk needs to come from some external source.


Update deploy.yml

Updating build files to account for repurposing "internet" section to "fundamentals."

@jrf0110 Who should we work with to approve these types of requests?

Merged commit into https://github.com/cloudflare/cloudflare-docs/pull/958


Deployments pictures: lifecycle and selection algorithm diagram.

ChangeLog:none
Signed-off-by: Peter Grzybowski<EMAIL_ADDRESS>

ok thanks @mirzak ! I corrected inprogress, good catch!
lets do the rest ina follow up PR, as you proposed.

Probably good to rebase this one on top of master (which might resolve the CI errors)

ok thanks!
and thanks @drewmoseley for approval as well!


Redesing paginations

Created PaginationStrategy subset of classes
Moved creation of links and meta to them (as each of those will create those links in different ways)
Twin PR of: https://github.com/infinum/infinum-json-api-setup/pull/18

[ ] Add Kaminari
[ ] Add pagy-cursor
[ ] Add order_query
[ ] Add tests

Used the other paginator encupsulation suggestion.


Dashboard says there is no packages

Describe the bug
We have one user in verdaccio, and several scoped packages published by this user. Some time ago verdaccio was restarted and and then i can see all packages in storage, everything works fine in npm cli but i can see no packages in web interface
To Reproduce

deploy verdaccio with stable helm chart
publish some packages
restart node which hosts verdaccio pod
See no packages in web interface, but everything works in cli

Kubernetes

Docker verdaccio tag: verdaccio:3.8.1
Kubernetes version: 1.11
Helm chart version: 0.5.0

Configuration File (cat ~/.config/verdaccio/config.yaml)
# This is the config file used for the docker images.
# It allows all users to do anything, so don't use it on production systems.
#
# Do not configure host and port under `listen` in this file
# as it will be ignored when using docker.
# see https://github.com/verdaccio/verdaccio/blob/master/docs/docker.md#docker-and-custom-port-configuration
#
# Look here for more config file examples:
# https://github.com/verdaccio/verdaccio/tree/master/conf
#

# path to a directory with all packages
storage: /verdaccio/storage/data

web:
  # WebUI is enabled as default, if you want disable it, just uncomment this line
  #enable: false
  title: Verdaccio

auth:
  htpasswd:
    file: /verdaccio/storage/htpasswd
    # Maximum amount of users allowed to register, defaults to "+infinity".
    # You can set this to -1 to disable registration.
    max_users: 1

# a list of other known repositories we can talk to
uplinks:
  npmjs:
    url: https://registry.npmjs.org/

packages:
  '@*/*':
    # scoped packages
    access: $authenticated
    publish: $authenticated
    proxy: npmjs

  '**':
    # allow all users (including non-authenticated users) to read and
    # publish all packages
    #
    # you can specify usernames/groupnames (depending on your auth plugin)
    # and three keywords: "$all", "$anonymous", "$authenticated"
    access: $all

    # allow all known users to publish packages
    # (anyone can register by default, remember?)
    publish: $authenticated

    # if package is not available locally, proxy requests to 'npmjs' registry
    proxy: npmjs

# To use `npm audit` uncomment the following section
middlewares:
  audit:
    enabled: true

# log settings
logs:
  - {type: stdout, format: pretty, level: http}
  #- {type: file, path: verdaccio.log, level: info}

@r-moiseev It looks like the storage location is non-standard, so I might have had this same issue whilst moving the storage folder around during Sinopia to Verdaccio migration. It seemed that when I manually moved all the packages I didn't copy the .sinopia-db.json and when Verdaccio started up it created a new one. This results in there being no listed packages and it showing the blank dashboard, despite being able to query the cli for uploaded packages.
This might not be the same issue but it's worth checking.

@r-moiseev  access: $authenticated your packages are protected. Have you log in in the UI?
@josh-green that depends on which sinopia you are migrating. If it is legacy lower than<EMAIL_ADDRESS>then you might get odd results.

Ok, thing is there is only packages pushed since last container restart.
But if I go on package page i can see it. but not in list.
Ive also found .sinopia-db.json in storage. Which is also empty

Ok, thing is there is only packages pushed since last container restart.
But if I go on package page i can see it. but not in list.

You hit this bug, so, I was aware of this one.
https://github.com/verdaccio/verdaccio/issues/1005

Ive also found .sinopia-db.json in storage. Which is also empty
PS I never had sinopia installed

Version 3 still use .sinopia-db.json for backward compability. In<EMAIL_ADDRESS>will be named .verdaccio-db.json instead. But for those still use sinopia named, it will be compatible, I would not worried about it. Ref https://github.com/verdaccio/verdaccio/pull/954
I close this one, please follow up on #1005

@juanpicado Thanks but seems it is not same bug. I CAN see packages published before restart, but CAN NOT after restart

Oh ... ok, I'll check it then.

Lokks like this list is cached somewhere outside storage and dies with container

If you bind the volume to some external location or don't persist data, then issue might be in your side.

@juanpicado /verdaccio/storage is persisted and all packages in place. Only page that does not work is repos list

I tried to emulate your environment using our examples https://github.com/verdaccio/docker-examples/blob/master/kubernetes-helm-example/ and works for me with v3.8.1
I'd recommend following https://github.com/verdaccio/verdaccio/issues/1005#issuecomment-445493069 that advice and give it a try.
That's all I can do from my side since my knowdlege on k8m or helm is limited.


READMEs: Formatted and removed all other links to azure.github.io API docs

#6311
Due to workspace settings, the formatter also ran on some of the files.
I replaced all of the azure.github.io API reference links with docs.microsoft.com, except for eventhubs-checkpointstore, which doesn't have a docs.microsoft.com page.

Any chance you can wait to merge this until after #6353 ?

@richardpark-msft Lets merge this first as it runs a whole lot more CI than #6353