Assignment one

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Queries on SELECT, JOIN, AGGREGATE, DATE, etc.
What did you learn from the changes you have made?
Different ways to view the database.
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Yes, I've put comments in the SQL file.
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
I watched the video and the live coding files, it helped.
How were these changes tested?
By executing the queries.
A reference to a related issue in your repository (if applicable)
NA
Checklist

[X] I can confirm that my changes are working as intended

Thank you Niyaz!


feat(sqla-repo): replace **kwargs usage with typed parameters

Replace the **kwargs usages in the repository methods with typed parameters
Add docstrings for the newly added parameters

I'm glad you put this together.  This is something I had been thinking about.


Offline sync condensing logic causes problems for foreign keys

This is a continuation of Azure/azure-mobile-services#871.
Basically,
insert in table A VALUES {id: a, value: a1, foreignkey: null}
insert in table B VALUES {id: b, value: b1}
update table A VALUES {id: a, value: a2, foreignkey: b}

...and syncing breaks. The whole point of a queue for operations is to let actions happen on the server in the same order they happened on the client. This behavior of condensing changes on the client fundamentally breaks that paradigm and renders the operations queue useless for all but the most basic entities with no server-side relationships. This condensing behavior means that the library might as well forget about having a queue and start to parallelize all of the requests to the server in no particular order (or a naive order like inserts first, then updates) to speed things up, because the library can't be relied upon for order-related changes as it currently stands.
The most logical solution is to enable change tracking in Azure Mobile Apps client. I know there is some resistance to this but the library may just have to bite the bullet and do it, especially since related issues like #116 and others are cropping up that would be much easier to solve with change tracking. This change-tracking behavior could even be opt-in to make it a non-breaking change.

This is definitely a big limitation of the SDK that needs to be fixed. The problem is that any fix is expensive and complex, and introduces its own issues, which is why the fix is taking so long.
The condensing logic has problems, but it simplify some things. The main reason it was introduced was to avoid the problem of conflicts when there are multiple edits to the same entity. If you have the following changes:

A.a = "bar"
Other changes
A.b = "foo"

Then what happens if change #1 has a conflict with the server copy?  Then you have to resolve the conflict, and then figure how to merge in change #3. Even if there is no conflict, then A definitely has some changed fields--at least the Version field if not others. So, the SDK needs to then handle field level merges and allow developers to customize the behavior. Plus, the SDK needs to track individual field changes, and not just object changes.
In order to avoid all of this complexity, the condensing logic was introduced, but it has the drawbacks of not keeping things in order when other objects were changing, which is a big problem.
There are other solutions beyond field-level change tracking, however. For instance, if the SDK let you customize the policy of how to do the condensing, that would be sufficient for this example. Right now, the original change is replaced, so in the example above, change #1 and #3 are merged and replace change #1. If it were possible to customize this so that change #3 was replaced, then the other operations would happen first and things would work out fine for this example.
The only case that wouldn’t work is if there were circular dependencies between two objects, which is generally uncommon.

@lindydonna I agree that this looks like a complex change, but I also agree with you that there may be a middle ground that will solve the foreign key issue without tracking of every single field-level change.
Here's a quick thought: what if updates are never condensed into inserts, but updates are allowed to be condensed into other updates? For example:
1. insert in table A VALUES {id: a, value: a1, foreignkey: null}
2. insert in table B VALUES {id: b, value: b1}
3. update table A VALUES {id: a, value: a2, foreignkey: b}
4. update table A VALUES {id: a, value: a3, foreignkey: b}

So what if 3 was never condensed into 1, but 4 was condensed into 3?
It would require adding a column to __operations (only used in insert statements) where a serialized version of the inserted entity is kept. This column represents the entity at the time it was inserted into the local sync table. Then, updates act as they do now and can update the sync tables, add update operations into __operations, and condense subsequent update operations into one update. In our example above, the end result of the local database before syncing would be:
Table A:

id
value
foreignKey

a
a3
b

Table B:

id
value

b
b1

__operations:

...otherColumns
type
table
entityId
serializedEntity

...
insert
A
a
{id: "a", value: "a1", foreignKey: null}

...
insert
B
b
{id: "b", value: "b1"}

...
update
A
a
null

This appears to solve the foreign key problem while still allowing some condensing to avoid the problems you mentioned.

That's an interesting idea, since it would at least solve the conflict problem. You'd still have to do merges though, because the insert would return a new record, with a new version field. I suppose you could use the previously serialized entity to see what fields changed, and merge into the update if only the version column changed. Any other merge would not be possible, since the update doesn't track the changed columns.
It seems like an option for customizing the condensing in terms of which order in the operation queue would be simpler though, and would avoid the changes to the operation queue design. Wouldn't that address your particular situation?

Well, on initial glance that looks like it might solve the issue, but consider the following situation:
insert in table A VALUES {id: a, value: a1}
insert in table B VALUES {id: b, value: b1, foreign key: A}
update table A VALUES {id: a, value: a2}

In this case if we condensed down instead of up it would end up looking like:
insert in table B VALUES {id: b, value: b1, foreign key: A}
insert into table A VALUES {id: a, value: a2}

Which also doesn't work.

Right, in this example you would condense to update the existing record. The actual API would take your dependencies and figure out the right logic, so it would be unlikely that you would have to directly specificity the collapse order for each record type.
The only case that wouldn't work is circular dependencies.

The proposal by @seanfisher is interesting! One issue I see with that is that if the insert (for table A) conflicts, the user would have to resolve the conflict looking at a record value from the past and this looks little unnatural to me. Imagine I having to resolve an old Todo note while pushing the Todo list on my phone to be able to push the latest Todo update.
@seanfisher: If the SDK lets you define foreign relationships (say A foreign references B, B references C and D) and performs push in the following order, will that address your issues:

Push inserts and updates to D, C, B and A in that order
Push deletes in A, B, C and D in that order

It is possible to come up with a linear order as long as there are no cyclic dependencies.

Also, what happens in the following scenario if updates are collapsed into earlier updates?
1. insert in table A VALUES {id: a, value: a1, foreignkey: null}
2. update table A VALUES {id: a, value: a2, foreignkey: null}
2. insert in table B VALUES {id: b, value: b1}
4. update table A VALUES {id: a, value: a3, foreignkey: b}

Closing off a number of issues today.
We will NOT be supporting foreign keys in this library.
I'm happy to accept a PR that adjusts the sequencing for collapsing of the requests.  However, right now, we have no intention of supporting transactions, which is what is really required.


Replacing a tx with the same nonce doesn't put the new tx in the queue

Description
From Discord:

Every time we create a new transaction with edited nonce(13) > SIGN message on connected wallet > this does not show up on Queue. However, when creating new transaction without editing the safe nonce (14) it shows up on Queue.

After signing on wallet, it appears to just refresh but doesn't redirect anywhere. It just closes the wallet pop up.

Seems to have been fixed in 3.19.0.

The original reporter was also able to replace a tx on dev prior to the release. Closing.


npm -> yarn

@barnjamin leaving this for you to help decide whether performing this migration would make anyone's life easier.

I think npm is a good choice. The original yarn 1.x is no longer supported and the latest version has compatibility problems with TypeScript.


style for a tag

update link style
fix packet slack url

@gauravgahlot @mrmrcoleman I'm not allowed to assign any reviewers. So would you review this PR when you have time? thanks


Added premium support for React.js projects

In case there are users, who are still finding some issues regarding solving problems. You could check out a service I started to solve fixes for React.js websites called Quickly react. (https://quicklyreact.com)

Sorry, this could potentially confuse people that this is a first-party support option from the react team


Error is not received after trying to play an "unsupported' song on Netcast TV

I have in our app a list of songs, when I try to play a song that seems to be unsupported it enters in the callback for successful playing, even thought the song did not start on the TV.
It would be really helpful to have an indication if the song actually starts or not in order to indicate it correctly in the app.
If needed I can provide the song for easy testing.

@softvision-teodorabadea, I tried playing a few ogg files on Netcast TVs, which are apparently unsupported, and errors are returned to the app.
Yes, please provide the song, and also the TV model you are using.

Hi,
Thank you for your quick response. I uploaded at the following location https://www.dropbox.com/sh/ary304hz3ou5k9f/AABCpzxSG1bprmgBs6PY2VN1a?dl=0 a picture with the information about TV model, firmware, etc.
Additionally I uploaded two songs, which make the issue reproduce on my side consistently. I first play successfully "17. Puya - Vestu salbatic [www.ten28.com].mp3" which correctly starts on the TV. After it startes playing I tap "02. Alb Negru feat. Ralflo & Rares - Zile [www.ten28.com].mp3", the "unsupported" song. I receive the callback of success, but on TV the previous song continues playing.
Additional note, before each song I also perform on Netcast TV a:

(void) closeMedia:(LaunchSession *)launchSession success:(SuccessBlock)success failure:(FailureBlock)failure;

One issue found so far: if mediaInfo.title or other properties (such as URL) contain special XML chars, they are not properly escaped, so the TV returns an error and doesn't play the second song. It happens here: https://github.com/ConnectSDK/Connect-SDK-iOS-Core/blob/38a3f16d0aac1672df8cf506aa09259a27a8dfad/Services/DLNAService.m#L985

The XML escaping has been fixed in https://github.com/eunikolsky/Connect-SDK-iOS-Core/tree/bugfix/138_dlna_xml_encoding and will be merged in dev soon.
Back to the main issue, our Netcast TV behaves strangely when switching songs. Eg, I play song 1, switch to song 2 (works fine), then switch to song 1 – we send the SetAVTransportURI request, the TV doesn't reply anything for 20 seconds, then the Play request fails with an error. This scenario works on newer WebOS TVs via DLNA. So the issue may be with the old Netcast TVs or our DLNA implementation.

Turns out that for old Netcast TVs the app needs to close the LaunchSession of the first media, wait for a response (the success block), and then play another media. It works fine this way on two Netcast TVs I've tested.

The DLNA fix is merged into dev: https://github.com/ConnectSDK/Connect-SDK-iOS/commit/da709546d653bd66bf9a759fbc3446351f7f6e70
@softvision-teodorabadea, please check out the latest code from the dev branch, and try with your Netcast TV. Make sure you call -[MediaPlayer closeMedia:success:failure:] on the current song, then play another one once it's done (in the success block) – that is required by those old TVs.

Thank you, I will give it a try.

The fix is available in the latest release: https://github.com/ConnectSDK/Connect-SDK-iOS/tree/1.4.3.
Closing this issue. @softvision-teodorabadea, please reopen if you encounter this issue again.


Can add watermark on image?

I need add watermark on image when take photo. Can someone help me?

You would have to write custom image logic for that. Look at SkiaSharp


How do I get plain text in a WebView

How do I get plain text in a WebView,I don't think there is such a way.Now RichEditor. GetHtml () is getting the tagged content, I want to get the plain text directly, can you provide this method?thank you

I guess you sent the  message to the wrong person
------------------ 原始邮件 ------------------
发件人:                                                                                "wasabeef/richeditor-android"<EMAIL_ADDRESS>发送时间: 2020年9月21日(星期一) 下午5:21
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS>主题: [wasabeef/richeditor-android] How do I get plain text in a WebView (#246)
How do I get plain text in a WebView,I don't think there is such a way.Now RichEditor. GetHtml () is getting the tagged content, I want to get the plain text directly, can you provide this method?thank you
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


review.py wrong config file location

review.py has the wrong file location for the .config file. It's looking for it in .../.config/.config and not in .../pycasso/.config
pycasso@raspberrypi:~/pycasso/examples $ python3 review_screen.py WARNING:root:Unable to find file at '/home/pycasso/pycasso/.config/.config' or at backup path '/home/pycasso/pycasso/.config/examples/.config-example' INFO:root:Loading config Traceback (most recent call last):   File "/home/pycasso/pycasso/examples/review_screen.py", line 33, in <module>     display_type = config.get("EPD", "type")   File "/usr/lib/python3.9/configparser.py", line 781, in get     d = self._unify_values(section, vars)   File "/usr/lib/python3.9/configparser.py", line 1149, in _unify_values     raise NoSectionError(section) from None configparser.NoSectionError: No section: 'EPD'

Fixing up script. We might end up removing the review_screen.py as test mode in pycasso is doing its job better.

Fixed in cab144c


[BUG]: basic sample project in repo on iOS doesn't show textTracks

Version
v6 (Beta)
What platforms are you having the problem on?
iOS
Architecture
Old architecture
What happened?
I ran the basic sample app (on master) and was trying to find a working example of the textTracks on iOS since I can't get it to work on my own sample repo.
I expected to see subtitles/captions since the title of this one says subtitles.

Reproduction
repository link
Reproduction
Step to reproduce this bug are:

Run git clean -xdf && yarn xbasic install && cd examples/basic/ios && pod install && cd - && yarn xbasic ios*

At this point I noticed my Podfile.lock had some differences with git diff COCOAPODS: 1.15.2

Click channel up until reaches 'sintel with sideLoaded subtitles', watch until audio occurs.

Observe no captions/subtitles.

@freeboub am I correct that this video in the sample should display textTracks? I am just presuming based off its title.

Additional info:
iOS simulator used: iPhone SE (3rd Generation) iOS 17.4
MacOs Sonoma 14.4.1, Apple M1 Pro

I can confirm that this.srcList[this.state.srcListId]?.textTracks is equal to [{"language": "en", "title": "test", "type": "text/vtt", "uri": "https://bitdash-a.akamaihd.net/content/sintel/subtitles/subtitles_en.vtt"}] on render

@freeboub am I correct that this video in the sample should display textTracks? I am just presuming based off its title.

Yes I think it should and I also think this was working on 5.2 version

Even when I set selectedTextTrack={{type: 'index', value: 0}}, it does not show them.

Same result for selectedTextTrack={{ type: SelectedTrackType.INDEX, value: 0 }}

@paulrinaldi are you able to see the text tracks on the dragon video? its the sintel one.

@paulrinaldi are you able to see the text tracks on the dragon video? its the sintel one.

I do not.
https://github.com/react-native-video/react-native-video/assets/107127397/7eb24310-a315-47d0-ad61-e930c3fbea86

(sintel with sideLoaded subtitles) also does not show subtitles

with
textTracks={this.srcList[this.state.srcListId]?.textTracks}
selectedTextTrack={{type: SelectedTrackType.INDEX, value: 0}}

Are you able to use SelectedTrackType.Language value: 'en' ? or do you have to use the index?

I started to investigate yesterday, looks like a regression from swift promise removal ( ping @KrzysztofMoch ) I will push my changes today even if I am not sure everything works fine. (I will have very limited time next week)

I just push a branch with a first fix. Still need to understand why side loaded textTracks cannot be selected

@freeboub @paulrinaldi I have good news - I have fixed sideload for subtitles! I will make PR soon. One thing that is very important is that sideload subtitles will not work for HLS streams (m3u8)

@freeboub @paulrinaldi I have good news - I have fixed sideload for subtitles! I will make PR soon. One thing that is very important is that sideload subtitles will not work for HLS streams (m3u8)

I was also able to make it work on MP4 local file. There is a test to avoid adding sideloaded text tracks on m3u. I don't understand why this limitation has been introduced, and I cannot confirm from google or apple documentation that this feature is not feasible... @KrzysztofMoch did you find such information ?
I would like to test the behavior with 5.2, but no time yet ...

We do some magic with AVMutableCompositionTrack to add subtitles to video - and we cannot use streams such as HLS. I found this information going through the forums like stackoverflow & apple forum (not sure if it is somewhere in docs)

Ok, thanks! Your looks good, I think you should add this information in documentation (for the m3u) !
Btw , please let me cross check the behavior on 5.2

It's already here


running app with docker by docker-compose

Thanks. But, please without the part of giving an option to enable/disable the authentication, as this part needs to be document, coded, tested and reviewed.


Refactored CMake configuration with better iOS integration

This is a major improvement to the organization of our build system! It's a big diff, but the changes are mostly all following one of these strategies:

Apply properties to CMake targets rather than entire directory subtrees. This ensures that compile options and include directories are scoped to only the targets that require them.
List source files instead of globbing directories. Collecting files with GLOB is not recommended in CMake scripts (see the docs). When you change the set of source files for a target you should also change the CMake file for it so that CMake knows that the build files need to be regenerated.
Use PUBLIC and PRIVATE keywords to declare usage requirements. This allows CMake to correctly propagate include directories and compile options as usage requirements between targets.
Remove CMake logic that isn't used. There was a lot of it.
Reorganize platform-specific configuration files. Here's how this works after this change:

Builds always start with the CMakeLists.txt in the project root.
When invoking the root CMake file you must include a TANGRAM_PLATFORM parameter, like -DTANGRAM_PLATFORM=linux.
The root CMake looks for a file at platforms/$TANGRAM_PLATFORM/config.cmake and includes it if found.
The platform config.cmake file then sets any global toolchain-related variables required and configures platform-specific targets, linking them against tangram-core. At this point, tangram-core has not yet been configured, but we can still use it as a target for linking.
After running the platform config.cmake script, the root CMake adds the core subdirectory, whose CMakeLists.txt configures the tangram-core target.

Rewrite the iOS build system to work within an Xcode workspace. Turns out we don't need the iOS CMake toolchain file we've been using? Instead we can just set a few global variables to adjust the SDK used by Xcode and the configuration "just works". Here's how this works now:

Start an iOS build with make cmake-ios. This generates an Xcode project for the iOS framework target in the iOS build folder and also applies some settings to the workspace file at platforms/ios/Tangram.xcworkspace.
The workspace file platforms/ios/Tangram.xcworkspace contains a reference to the generated framework Xcode project and to the demo app Xcode project platforms/ios/demo/TangramDemo.xcodeproj.
Through the workspace, the demo app project contains a reference to the framework output from the generated Xcode project.
In the workspace you can build and debug either the framework or the demo app using the typical Xcode configurations.

TODO:

[x] Consolidate git history
[x] Update build instructions in README files
[x] Add comments to explain non-obvious parts of Makefile and CMake files
[ ] Improve source file grouping for core project in Xcode

Future changes:

Add iOS static library target
Probably remove Tizen targets?
Just write built-in shaders as raw strings in header files - generating data headers with a script is more complicated than it needs to be

@tallytalwar Check out https://github.com/tangrams/tangram-es/pull/1827/commits/71582e71c44141881d94b4a8f7e3e2d340b25efa for more information than you ever wanted about the Xcode workspace setup haha

Haha, I am always hungry for information. Checking thanks.

With respect to my previous review question of make android building all architectures, I remember now that we added an option to specify ABI Targets using command line which gets applied to the abiFilter in tangram's android library's build.gradle. And if no abi target is specified, build.gradle is configured to build all (armeabi-v7a, arm64-v8a, x86).


Rate limit does not apply after a period of inactivity

The rate limiter will not limit after a period of inactivity.
Expected behavior
If there is no activity on the rate limiter for a while, it should be filled to maximum capacity but not allow more than capacity tokens to be consumed at once.
Example:

Rate limiter set to 1 call per second (Ratelimiter(1, 1, 1))
No calls for 30 seconds
Rate limiter should still only allow one call per second.

Actual behavior

Rate limiter set to 1 call per second (Ratelimiter(1, 1, 1))
No calls for 30 seconds
Rate limiter will allow 31 calls before applying rate limit again

Comment
Because we increment self.next to be self.next = self.next + self.tick, the rate limiter will be less than now() when all the capacity is not used. Rate limiting only applies again after self.next catches up with now().
Steps to reproduce the behavior
I wrote a simple test to reproduce this. This could use some refactoring to use a mocked timesource for the tests but here it goes for now.
#[cfg(test)]
mod tests {
    use std::{thread::sleep, time::Duration};

    use crate::Ratelimiter;

    #[test]
    fn should_not_burst_beyond_capacity_gh() {
        let rl = Ratelimiter::new(1, 1, 1);

        // Sleep 10s
        sleep(Duration::from_secs(10));

        let mut tokens = 0;
        while rl.try_wait().is_ok() {
            tokens += 1;
        }

        assert_eq!(tokens, 1);
    }
}

Fails with:
thread 'tests::should_not_burst_beyond_capacity_gh' panicked at 'assertion failed: `(left == right)`
  left: `11`,
 right: `1`', ratelimit/src/lib.rs:386:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Thanks for the detailed issue. Will take a look at the logic tomorrow.

This was fixed by #37


[CORE-4546] jenkinsfile: Parallelize tests

This is ready for review. I've sped up client tests by 10ish minutes, or 30% of overall runtime.


Fixes for TFO Tests in ASAN build

ASAN reports a heap overflow and a leak when running tfo tests.
The heap overflow might be spurious (not sure what the C standard says to that case), but we can simply fix that by using the correct member of a union instead of copying all bytes of said union including bytes that might not belong to the structure written into the union.
The leak is due to a missing freeaddrinfo call.

What branches does this apply to?

What branches does this apply to?

Found it on master.  Did not check other branches.  But I do not think this needs to be backported.

What branches does this apply to?

Found it on master. Did not check other branches. But I do not think this needs to be backported.

TFO was not ported to 3.0

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Pushed. Thanks.


fix: passing branch ids with 8 digit suffix should work

Closes: #84

Sorry, I noticed your pr a bit late, was hurrying to deploy a fix!
I left the rule less strict to account for future possible changes. But I'd really want to merge your tests! So can you please rebase the latest changes?

@duskpoet I rebased the PR so only the tests are left here 👍 thanks for the quick release of the fix

Thanks for your assistance!


VLAN should be WLAN

I think VLAN should be WLAN in the following sentence: When the user is connected to a VLAN or Ethernet connection, no Internet traffic is routed over the cellular connection.
This would make sense in combination with the paragraph after that.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 007029d8-c597-7a41-ecf9-577078ada719
Version Independent ID: 3e097f5a-0353-71b5-7932-9d5ac4cba772
Content: Understanding and configuring Windows Connection Manager - Windows drivers
Content Source: windows-driver-docs-pr/mobilebroadband/understanding-and-configuring-windows-connection-manager.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @duncanmacmichael
Microsoft Alias: dumacmic

Hi @SasStu, thanks for pointing this out. This does look like a typo, so I've gone ahead and fixed it. Those changes should be live in the next few minutes.
Also, for small changes like this in the future you can feel free to edit the page directly using the "Edit" button in the upper right corner, then submit a pull request. :)
Thanks!


Flask extension broken for Flask-Login 0.3.0

Specifically the following line in the extension will raise a TypeError:
is_authenticated = current_user.is_authenticated()
Switching to:
``is_authenticated = current_user.is_authenticated`
as well as two other .is_anonymous() calls, resolves the problem.

Ohp, thanks @dcramer! – was just pulling things together for a PR but looks like you have it.
Note here is the relevant changelog for Flask-Login: https://github.com/maxcountryman/flask-login/blob/93a5e20ba7289c3c0ed440faaa5b195d36f392cb/CHANGES

Fixed in latest release


Added warnings into React Panel on the Graph page

Good time of the day,
I added warnings to be displayed in React Graph panel similarly as to how they are displayed in the old UI as suggested in #8030. The change seemed small enough to not require individual test. It's my first time contributing to Prometheus, so please let me know if you want me to change anything.

Thank you. I have tested the functionality and it works. Pending @juliusv 's review about the tests etc.

Thank you. I have tested the functionality and it works. Pending @juliusv 's review about the tests etc.

:+1: Thank you :)

:+1: Thank you :)


[Feature] Supports Kafka with canal CDC format ingestion

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
TiDB is also an importation database. It outputs changelog to Kafka with canal CDC.
Solution
No response
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

I have interest to work for this issue. Please assign this issue to me. @JingsongLi

I have interest to work for this issue. Please assign this issue to me. @JingsongLi

Is there any progress? I have plenty of time, please allow me to continue this pr

I have interest to work for this issue. Please assign this issue to me. @JingsongLi

Is there any progress? I have plenty of time, please allow me to continue this pr

I've almost finished this issue


Piracy incident

Hey nunu, Ive read about the piracy incident on your Patreon. I can understand your frustration but have you considered that individual is probably just frustrated with you closing the source of your project and even advertising it on an open source Platform?
Also, whats the latest Patreon firmware? I am a bit confused with what the state of this project is and do you still maintain the open sourced repo?
Best, Olli

For context, I am not a Patreon member anymore and am considering whether to resubscribe or not as you have been focusing on the ESPR project which I was not too interested in
And I could imagine whoever pirated your software felt similar with the state of your firmware because they feel like you're wasting lots of potential by not publishing it. Publishing a premium distributed version is of course not the way to dispute this though lol, but just as a thought given to you, maybe communicate more clearly which features in the Documentation are premium features and where to get the version v21+ as that is not clear from this repo.

Thanks for your input.


The result for atari enduro in the paper is not reproduced

Hi Danijar,
I am trying to work on your codebase, but I found your latest codes are not working for the atari enduro game.
(I checked it is working for Atari Bank Heist).
Could you check it?
I leave the Tensorboard logs here.

Bests, Jaesik.

I ran it with precision=16, I will re-run with precision=32 config and share the result.

I checked it is okay with precision=32.


Restore IAP on install of second device

If a user already purchased Notey Pro and they attempt to install Notey on another device, IAP are not transferred.

Already implemented. User just has to click Upgrade to Pro the either Pro or Tip.  If they select the one they purchased, pro features will enable. If they press the opposite, a purchase window will appear, but backing out of it enables pro features too.
Will leave open but put to future milestone for possible better implementation. Possibly check on first install so user doesn't have to go through all those steps in regain pro features.

Duplicate, #29 will solve this.


Auto corrected by following Lint Ruby Layout/EmptyLines

Auto corrected by following Lint Ruby Layout/EmptyLines
Click here to configure it on awesomecode.io

Coverage increased (+0.09%) to 96.266% when pulling 3736ef457ee507d1fc5364cc85435a12a7645b69 on awesomecode-lint-ruby-layout-emptylines-41844 into 9c055d175a7c54f6dacbccff2f14edd7f1f88666 on master.


Fix "'NoneType' object has no attribute 'cast'" error when model is freed

This patch fixes the error by storing the null pointer in the constructor, which is used to set ctx.ptr when the model is freed.

I don't really understand how this works. Is it possible to provide a minimal snippet that reproduces the exception and is fixed by this PR?

This error happens on all of the examples in my environment. The simplest way I'm able to reproduce it is by modifying the generate_completions.py script to have it read one token and generate one token. The error it produces is:
Loading 20B tokenizer
System info: AVX=1 AVX2=1 AVX512=0 FMA=1 NEON=0 ARM_FMA=0 F16C=1 FP16_VA=0 WASM_SIMD=0 BLAS=0 SSE3=1 VSX=0
Loading RWKV model
1 tokens in prompt

--- Generation 0 ---

Hello[,]

Took 0.151 sec, 151 ms per token
Exception ignored in: <function RWKVModel.__del__ at 0x7fa783d80ee0>
Traceback (most recent call last):
  File "./rwkv.cpp/rwkv/rwkv_cpp_model.py", line 198, in __del__
  File "./rwkv.cpp/rwkv/rwkv_cpp_model.py", line 193, in free
  File "./rwkv.cpp/rwkv/rwkv_cpp_shared_library.py", line 235, in rwkv_free
AttributeError: 'NoneType' object has no attribute 'cast'

I'm running it on Python 3.10.6.


Link broken on AppTypeRegistry

Objective

The idea is to fix a dead link in the docs
Fixes #6281

Solution

I replaced the link to the one suggested by the users in the  thread

this is still a dead link for me when building doc locally

this is still a dead link for me when building doc locally

What's seems strange to me is that if we reference something inside the same namespace we have no problem, while if it's something related to another it seems to break, like in our case, there might be something I'm missing?

I think I remember seeing a few issues in rustdoc about links between re-exported crates when the link wasn't available as is through the main crate... we have that issue in other parts of Bevy doc. But it may not be the same thing...
Sorry, I don't have better advice than "try different path for the type and build locally to check it works" 😞

@mockersf Alrighty! I'll check some of this issues asap 😊

no dead links for me anymore, this was probably fixed by rustdoc


Enable advanced ES6 features in ClientFunctions

Enable the following ES6 features in ClientFunctions code:

[ ] async/await
[ ] generators
[ ] spread with non-arrays (https://github.com/DevExpress/testcafe/issues/1596)

request - "new" std library methods, e.g. Object.entries ?

It looks like there's more to it then simply enabling these ES6 features in ClientFunctions: I had to replace every occurrence of spreads on iterables in the test target webpages to have my pages properly rendering using testcafe...

@AlexKamaev Could you provide an explanation why this isn't supported? What does TestCafe do with user scripts that causes them to fail? I'd imagine running them through Babel with some configuration that's specific to the target browser. But if that's so, then why does this fail?

Hi @amitzur ,
Please see this comment:
https://github.com/DevExpress/testcafe/issues/1596#issuecomment-314135764


More Type Primitives Support Request

Hi, I'm just opening this issue to get to know more about the current state of type primitives support.
I've managed to use UInt's and such inside of #[cube] kernels but it would be nice to have some more support over these primitives. For example for image manipulation purposes I would find very useful the possibility to work with something like:
use cubecl::prelude::*;

struct Pixel {
    r: u8,
    g: u8,
    b: u8,
    a: u8
}

#[cube(launch_unchecked)]
fn random_image_manipulation_kernel<P: Pixel>(input: &Array<P>, output: &mut Array<P>) {
    ...
}

#[cube]
fn some_operation<P: Pixel>(x: P) -> P {
    ...
}

I've seen you can already work with custom structs if every element inside of them is a cubecl primitive like UInt or Float, but what about a situation like this.
Technically I could definitely work with just a UInt but currently bitwise operations are very hard to execute since the bitwise or and bitwise and operations are not supported by UInt.
Is there currently already a way to achieve this sort of result? If so it would be very nice to include this in the examples folder since there is nothing regarding custom types. Otherwise I would consider implementing UInt bitwise operations or even some form of support for 8 and 16 bits integers.
Thanks for reading this through, hope I'm not missing anything here, I haven't had the time to read through much of the source code, hope you might help.
Also, if you wish I might take some time to read through the source code and try implement any of this myself, if you find it valuable and in line with the project.

The problem with u8 is that you can't send those directly to the GPU via an array, so we would need to refactor that part before making U8 a primitive type.


New WebUI metadata duplicates first seed for all images

Describe your environment

GPU: mps
CPU arch: arm
OS: macOS
Python: Anaconda
Branch: development
Commit: 0ab5f2159d5d7b25cffd96b03fed0ecd305db5b6

When generating multiple images, all generated images have the same seed in the metadata instead of the seed actually used for that image. This also breaks the use seed buttons, as they read the metadata and thus always use the same seed from the first image.
Expected behaviour is that each image has the pseudo-random seed it was generate with.
To replicate, generate a few images from a random seed & check details.

I've found the issue, I'll make a PR.


[Bug] When username is a subset of an already existing user then invite user prompt does not appear

Description
When tagged user email address is a subset of an already existing user then "Invite user" prompt is not displayed
Steps to reproduce the behaviour:

In comment mode, try tagging a user with email ID which is a subset of current user, e.g<EMAIL_ADDRESS>(current user) tags<EMAIL_ADDRESS>(user to be invited to new organization)
Invite user option does not appear and user is unable to add user to organization through comment mode.

Important Details

Version: Cloud
OS: Win10
Browser: Chrome
Environment: Release

Hey @pranavkanade, the suggestion to add user only pops up right at the end of typing out the entire email. Would it be possible to dynamically check as the user types the email so the Invite user prompt appears a bit earlier rather than at the absolute end?
https://www.loom.com/share/8f382cdca212420fa56c3395e60025db

@ramsaptami are you suggesting we should always show the invite new prompt? Showing an invite prompt before user has entered email leads to an error. Try this - go to comments mode, add a comment @asdf click on invite user; you'll see the error please enter a valid email address.

Ah I see, there's a pattern validation. That explains why we need to type out the email entirely before getting that option. I was thinking of a different approach. But this works too 👍


Extend the test suite to cover the CLI

The CLI first was meant as a simple example. User feedback however indicates that it's another selling point to easily use the MythX API from the shell. To provide the same level of quality, we should extend our test suite to cover all CLI functionality.

Closed, moved to new CLI at https://github.com/dmuhs/mythx-cli/


PR Template: Add a (non-enforcing) checklist of items to consider for…

… PRs

What this PR does / why we need it:
It introduces a checklist for PR templates.
We want it to help PR authors to take care of all the external dependencies of PRs, such as user-guide, documentation, design.
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:
Release note:

NONE

/lgtm
Looks pretty good, thanks @fabiand !

/uncc

/retest

Thanks, fixed

There were different opinions about when e2e tests are needed.
I've restored the old wording, but we migh twant to pickup this topic async in order to see what the right rule of thumb could be.
To me personally bug fixes require at least on e2e test is misleading, if there is a code generator bug - no e2e test is required.

/lgtm
Good PR to see flakes :)

@AlonaKaplan , FYI.
In order to make sure all are in-sync about this (and agree to enforce it), I think it will be effective to ask a maintainer from each SIG to lgtm such cross-project things in the future.


Return empty object or null cursor

Is there a way for a find function to return an intentionally empty set of documents? For example, I have a parent publication retrieving the User document and a child find function returning a cursor only if the User document contains the requisite permission.
If the child find function is passed an invalid User document, there's currently no good way to process that. When using Meteor.publish, we can return an empty set of values as an empty Array or simply by calling this.ready() if the parent subscription doesn't check out. That don't appear to work within Meteor.publishComposite and result in the follow traceback instead:
I20141105-22:52:42.905(0)? Exception from task: TypeError: Object  has no method '_getCollectionName'
I20141105-22:52:42.907(0)?     at Publication._getCollectionName (packages/reywood:publish-composite/publish_composite.js:171)
I20141105-22:52:42.908(0)?     at Publication.publish (packages/reywood:publish-composite/publish_composite.js:105)
I20141105-22:52:42.909(0)?     at null.<anonymous> (packages/reywood:publish-composite/publish_composite.js:180)
I20141105-22:52:42.910(0)?     at Array.forEach (native)
I20141105-22:52:42.910(0)?     at Function._.each._.forEach (packages/underscore/underscore.js:105)
I20141105-22:52:42.911(0)?     at Publication._publishChildrenOf (packages/reywood:publish-composite/publish_composite.js:177)
I20141105-22:52:42.911(0)?     at Object.observeHandle.cursor.observe.added (packages/reywood:publish-composite/publish_composite.js:118)
I20141105-22:52:42.912(0)?     at observeChangesCallbacks.added (packages/minimongo/observe.js:153)
I20141105-22:52:42.913(0)?     at self.applyChange.added (packages/minimongo/observe.js:53)
I20141105-22:52:42.914(0)?     at packages/mongo/observe_multiplex.js:188
I20141105-22:52:42.944(0)? Exception from sub PLdN4pDMc5NfWi82h TypeError: Object  has no method '_getCollectionName'
I20141105-22:52:42.944(0)?     at Object.Future.wait (/home/vagrant/.meteor/packages/meteor-tool/.1.0.35.ftql1v++os.linux.x86_64+web.browser+web.cordova/meteor-tool-os.linux.x86_64/dev_bundle/lib/node_modules/fibers/future.js:326:15)
I20141105-22:52:42.945(0)?     at _.extend.runTask (packages/meteor/fiber_helpers.js:79)
I20141105-22:52:42.946(0)?     at _.extend.addHandleAndSendInitialAdds (packages/mongo/observe_multiplex.js:47)
I20141105-22:52:42.946(0)?     at MongoConnection._observeChanges (packages/mongo/mongo_driver.js:1108)
I20141105-22:52:42.947(0)?     at Cursor.observeChanges (packages/mongo/mongo_driver.js:783)
I20141105-22:52:42.947(0)?     at Function.LocalCollection._observeFromObserveChanges (packages/minimongo/observe.js:177)
I20141105-22:52:42.947(0)?     at Cursor.observe (packages/mongo/mongo_driver.js:777)
I20141105-22:52:42.948(0)?     at Publication.publish (packages/reywood:publish-composite/publish_composite.js:108)
I20141105-22:52:42.948(0)?     at null._handler (packages/reywood:publish-composite/publish_composite.js:12)
I20141105-22:52:42.948(0)?     at maybeAuditArgumentChecks (packages/ddp/livedata_server.js:1599)
I20141105-22:52:42.949(0)?     - - - - -
I20141105-22:52:42.950(0)?     at Publication._getCollectionName (packages/reywood:publish-composite/publish_composite.js:171)
I20141105-22:52:42.951(0)?     at Publication.publish (packages/reywood:publish-composite/publish_composite.js:105)
I20141105-22:52:42.951(0)?     at null.<anonymous> (packages/reywood:publish-composite/publish_composite.js:180)
I20141105-22:52:42.952(0)?     at Array.forEach (native)
I20141105-22:52:42.953(0)?     at Function._.each._.forEach (packages/underscore/underscore.js:105)
I20141105-22:52:42.954(0)?     at Publication._publishChildrenOf (packages/reywood:publish-composite/publish_composite.js:177)
I20141105-22:52:42.955(0)?     at Object.observeHandle.cursor.observe.added (packages/reywood:publish-composite/publish_composite.js:118)
I20141105-22:52:42.956(0)?     at observeChangesCallbacks.added (packages/minimongo/observe.js:153)
I20141105-22:52:42.957(0)?     at self.applyChange.added (packages/minimongo/observe.js:53)
I20141105-22:52:42.958(0)?     at packages/mongo/observe_multiplex.js:188

In the native Meteor publish method, returning an empty cursor (i.e. []) works (although this.stop() is best practice) but doing this in publishComposite results in an error, it might be good to replicate this behavior? Although a return statement works. as described above.


fix: css loader options editable (#10)

Make css loader options editable, such as mode: 'pure' to mode: 'local'

This plugin uses the same css-loader config that Next.js uses intentionally.
If this customization exists in Next.js, then applying it in Next's config will also make it available in the less configuration.
If not, and this config is absolutely needed, you can create another plugin, e.g. withCssLoaderConfig that is similar to this one and apply it before withLess, that way it'll clone the customized css-loader.
Generally Next.js is opinionated about how CSS is loaded...


question about vocabulary and tokenization

Hello, thanks for a great dataset release!
I've been trying to build the vocabulary for the dataset, but am a little confused since there seem to be several sources from which we can obtain vocab - from the triples in the dataset itself, from jericho using env.get_dictionary(), and from the files in symtables/ .
From what I can understand, the vocabulary that we can get from jericho with env.get_dictionary() seem to be vocabulary that comes from a trained bpe - would this be correct?
Also, would it be possible to share the vocabulary list used for the paper? It would be much appreciated!

Update: I ended up using the objects from env.get_world_objects() as I think they cover most (if not all) of the objects appearing in the triples. Closing!


Sample weights in gradient boosted trees

Is there support for sample weights in daal4py.gbt_classification_training().compute? Adding weights has no effect on training.

Gradient boosted trees don't support sample weights. gbt_classification_training has this argument in compute method because it's default behavior for classification training template.


[ERROR] - JSON POST BODY

Is this a question?
When using it with express server and trying to post json body, it seems like anything above 1KB since request fails. I tried using express.json() middle also body parser but it does not seem to go along smoothly with the proxy middleware. Any help regarding this issue would be helpful. Thanks in advance for the time.
this.app
.use(this.parseRequestBody.bind(this))
.use('*', apiProxy)
.use(express.json())
.listen(proxyPort);

Is this a bug report?
Yes

Is this a feature request?
No

http-proxy-middleware: 1.0.4
http-proxy-middleware configuration
{
// This value is not used but this is mandatory.
target: proxyPort.toString(),
changeOrigin: true,
followRedirects: true,
preserveHeaderKeyCase: true,
pathRewrite: this.rewritePath.bind(this),
router: this.rewriteHost.bind(this),
onError: function onError(err, _req, _res) {
// TODO: In future expose it as a callback.
},
onProxyReq: this.onProxyReq.bind(this)
}
server: express
other relevant modules

(Paste the link to an example project and exact instructions to reproduce the issue.)

Can't really do anything without minimal reproducible example.

@chimurai Sorry forgot to update here. I was able to figure out a fix for the issue from my side. Thanks for taking time to review the issue.

Would be nice if you could share your solution for developers running into the same issue.

I tried this from https://github.com/chimurai/http-proxy-middleware/issues/320#issuecomment-600470764 and it worked for me.

You just need to reorder the middlewares;
put bodyParser after all proxy middlewares and routes without proxy after bodyParser
ex:
app.use('/proxy/routex', myProxy);
app.use('/proxy/routey', myProxy);

... //then, after all proxys
app.use(bodyParser.json());

//and now, all no proxy routes
app.use('/api', (req,res)=>{
//TODO
});

app.get('/api/test', (req,res)=>{
//TODO
});

@chimurai @geoangelotti Sorry for the delay in responding. Back i already had in that order even after that it was causing me issues, for me it was a specific issue of post body being greater than 1KB. For some reason something was adding a header i had to delete below headers before passing the request object to the proxy middleware and everything worked fine after that.
delete req.headers['expect']
delete req.rawHeaders['expect']


CZ: invalid iat field

Affected Country: CZ
Issue Description
All json test data provided by CZ contain an iat that is later than the contained VALIDATIONCLOCK
Proposed Solution
Please amend the test data to contain iat field earlier than the VALIDATIONCLOCK

Can you please clarify what iat stands for.  Thank you.

The time when the certificate was created, as defined by the spec here

Hello, can you recheck the issue. The pull should fix that. Thx

@JChrist Can we close the issue?

I checked again with latest main branch (commit: 3b8e92998a7bd23418d53e9a24c4df54e3acc82d), however I still had failures for CZ iat field in 1.json and 3.json.
For 1.json:
Certificate is NOT issued YET! now:2021-05-31 14:11:37.501563Z iat:2021-05-31 17:11:38.000
For 3.json:
Certificate is NOT issued YET! now:2021-05-31 14:04:23.803752Z iat:2021-05-31 17:04:24.000

Hello are you sure that when you are loading time that you are loading it as UTC not your local time as the diff between times are +3h (Greece local). Thanks for check.

While it is printed as Greek local time, it is not loaded as such, since iat field is seconds since epoch

Ok, looks like it's problem with rounding secs in epoch, I will fix the validationclock time. Change will be available right after builds will work.


Trying to resolve DashRenderer issue

Trying to resolve the issue Uncaught ReferenceError: DashRenderer is not defined

No longer usefule


Update to Aspire 9

This updates Aspire to the upcoming 9.0-rc.1 release and retargets the projects to net9.0.
It also moves the Python inference app into a container so that Python doesn't need to be installed on the host machine when developing. The container bind-mounts the python app files and enables Uvicorn reloading when running in dev.

It also moves the Python inference app into a container

I think this is absolutely great and we definitely should do it.
However I also think we shouldn't consider this to be a complete solution for #42. Since getting this branch, I've been trying for about 30 mins to make the app run but haven't yet got the Python container to build properly. It took over 10 mins to download the Python dependencies then stopped for a long time saying exporting to image. Eventually it continued but seems to have failed with this at the end of the Docker log:
2024-10-14T14:23:01 #9 DONE 45.0s
2024-10-14T14:23:01
2024-10-14T14:23:01 View build details: docker-desktop://dashboard/build/desktop-linux/desktop-linux/vwqseg2b22idn618phjplc0xm
2024-10-14T14:23:03
2024-10-14T14:23:03 Error response from daemon: No such image: sha256:b40b44cfaec46ee67768396149dc00c5df75c3ddf76e7f57ab3759b2cb38a109
2024-10-14T14:23:07 Error response from daemon: No such image: sha256:b40b44cfaec46ee67768396149dc00c5df75c3ddf76e7f57ab3759b2cb38a109
2024-10-14T14:23:12 Error response from daemon: No such image: sha256:b40b44cfaec46ee67768396149dc00c5df75c3ddf76e7f57ab3759b2cb38a109

I don't know what's going on with that, and will next try clearing out the Docker image it built and try starting it again. This is not to criticise anything in the implementation, which looks entirely correct, but just to acknowledge that even with this automation it remains prone to be a sticking point for new users.
So although I definitely think it's better to run this in Docker than not do, it's probably better still if we also make the Python dependency optional (defaulting to not used).


Extend CxG conversion to cover - uns (general metadata)

It doesn't look like any data in uns is expected as in the schema.
These are the columns in uns:
'hvg', 'markers_louvain_resolution_0.1', 'markers_louvain_resolution_0.1_filtered', 'markers_louvain_resolution_0.3', 'markers_louvain_resolution_0.3_filtered', 'markers_louvain_resolution_0.5', 'markers_louvain_resolution_0.5_filtered', 'markers_louvain_resolution_0.7', 'markers_louvain_resolution_0.7_filtered', 'markers_louvain_resolution_1.0', 'markers_louvain_resolution_1.0_filtered', 'markers_louvain_resolution_2.0', 'markers_louvain_resolution_2.0_filtered', 'markers_louvain_resolution_3.0', 'markers_louvain_resolution_3.0_filtered', 'markers_louvain_resolution_4.0', 'markers_louvain_resolution_4.0_filtered', 'markers_louvain_resolution_5.0', 'markers_louvain_resolution_5.0_filtered', 'neighbors', 'pca', scrublet

Schema values: title, batch_condition, {column}_colors, default_embedding, X_approximate_distribution

We may have to mine some of the other files for general metadata.

Only the title is required. Got this from the IDF file.

Add default_embedding: "X_umap_neighbors_n_neighbors_20

Add dataset curie
e.g.
SCXA:E-CURD-2
This is sufficient for rolling links to dataset files in general on FTP, SCXG web site and the CxG standard h5ad file that we produce.
TBD: Do we store this in uns or do we just make it the ID of DataSet?
Decision: Save as property in dataset object.


OVN DB not work when recovers from space full

Expected Behavior
When disk space is not full, ovn db should work normally
Actual Behavior
The disk space is not full, but ovn db still can not work.

after reboot  pod ovn-central-xxxxxx. everything is Ok now.

we may need a automated reboot procedure.


How to build dynamic window with controls list?

I try to build the window using this package when contents of a window should be changed on runtime.
I have the list of "file downloads". I want to display this list in the window. But i can not find how to do this correctly.
Is there some best practice or recomedation on how to do a window containing list of repeated blocks with same structure but different data?
In my case each file is Name,Size, SavePath, Progress bar and Cancel button. On design time i do not know how many records (files) will i have.
The list must be updated every second. And on each update number of rows can be different .
What i did, i created the box on the window and displayed that window. And i have another child box which will be created in this one
Then in a loop (in other goroutine ) :

if child box was already created, i remove it from parent box with
parentBox.Remove(childBox)

Create child box again and add labels, progresbar, button in it
childBox = gtk.BoxNew()

Add back to parent
parentBox.Add(childBox)

And this works. But only some random iterations and then window becomes empty (no any content) and not responsive. But that goroutine till works and try to repeat all this process.
Other issue is that refresh of widgets blinks
What can cause this way doesn't work?
maybe there is better, more correct way to do such things?

Are you trying to update the UI directly from a goroutine? All modification of GTK objects should run in a block like this:
glib.IdleAdd(func() {
    // Modify GTK stuff here
})

Anything NOT touching GTK should of course be in a goroutine and just sync back with IdleAdd (I recommend a wait group if you have to wait for the UI update to complete. Using channels to wait may give you some deadlocks like you describe. Example:
var wg sync.WaitGroup
wg.Add(1)
glib.IdleAdd(func() {
    defer wg.Done()
    // Modify GTK here
})
wg.Wait()

If you already do that, then check for deadlocks with channels or similar.
Regarding flickering, a few suggestions:

Try to group as many updates into the same IdleAdd call
First build full elements, then swap them with what is on screen (less work in the background between Remove() and Add())
Re-Use elements where possible. You could have a struct holding a reference to all your elements, i.e. label, progressbar, button and keep them around until the next update. Remove excess, create missing, repurpose existing elements.

Aside from these ideas you probably should look into the GtkListBox for this kind of window structure. Above suggestions still apply, but it may give you some other benefits like sorting & the likes. This is a nice explanation: https://python-gtk-3-tutorial.readthedocs.io/en/latest/layout.html#listbox


memory and cpus settings require fresh minikube vm

@johnelm Please feel free to re-open this PR after signing the CLA.

okay.
It was a very small edit to the wiki, to clarify something that would have saved me a bit of time troubleshooting.
It’s not worth the effort of going through the CLA process.

John
<EMAIL_ADDRESS>On May 31, 2018, at 3:44 PM, Zach Corleissen<EMAIL_ADDRESS>wrote:
@johnelm https://github.com/johnelm Please feel free to re-open this PR after signing the CLA.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/kubernetes/website/pull/8550#issuecomment-393706149, or mute the thread https://github.com/notifications/unsubscribe-auth/AB5yFRAZae3_mg-iknJxuVGM6e2h_hf0ks5t4HJngaJpZM4UAN9p.


v3 off thigh angle is a hot mess

because pi & angles and such, it wraps around to 0. fixing interval to be 0-2pi instead of -pi to +pi


Migrate to AndroidX

Hey, how about to migrate to AndroidX libraries?

This requires migrating the underlying library from the support libraries (https://github.com/LiquidPlayer/LiquidCore/tree/master/LiquidCoreAndroid) to AndroidX.
If you can migrate it to AndroidX and generate a clean stable pull-request, then I'm sure the author would be more than happy to merge it.
Once that's done, I can migrate the flutter wrapper to AndroidX as well.

Actually, we don't need a new version of that version, all you need to do is migrate your deps to AndroidX and use
android.enableJetifier=true
android.useAndroidX=true

in a gradle.properties
More info here
https://flutter.dev/docs/development/packages-and-plugins/androidx-compatibility

The main underlying dependency Liquidcore still references support library classes:

https://github.com/LiquidPlayer/LiquidCore/blob/0bceda115f5932a9c0b61b38a1d2159045e5309a/LiquidCoreAndroid/src/main/java/org/liquidplayer/service/MicroService.java#L12
https://github.com/LiquidPlayer/LiquidCore/blob/0bceda115f5932a9c0b61b38a1d2159045e5309a/LiquidCoreAndroid/src/main/java/org/liquidplayer/javascript/JNIJSValue.java#L9

So unless that flag changes all the compiled class references to use androidx.* ones, I'm not sure it's going to work?
I tried a quick migration a couple minutes ago, but it kept crashing whenever I ran the example project (due to missing support classes).

Actually, just read the documentation a little more carefully:

android.enableJetifier: When set to true, the Android plugin automatically migrates existing third-party libraries to use AndroidX by rewriting their binaries. The flag is false by default if it is not specified.

I've been out of the loop with the current android-dev ecosystem for a bit, but is there a reason your own project doesn't just specify android.enableJetifier and automatically migrating the flutter_liquidcore and Liquidcore dependencies to AndroidX?

Jetfier only works in aars and jars. Your plugin is delivered as source code. If your code depends on libs that are aars and they use support library there is no problem. But your code that is compiled in the client developer machine needs to be using AndroidX libraries.

Okay, I've gotten it to work now.
The missing class issue must've been because I/Android Studio stripped out the com.android.support:support-v4 dependency during the refactor.
One really annoying thing is that the AndroidX update requires upgrading gradle from 3.2.1 to 3.3.0 (assuming to fix this bug)
However the update to 3.3.0 starts throwing deprecated API warnings on every clean build requiring you to rebuild a second time in order to bypass the warning.
https://github.com/flutter/flutter/issues/28713 - This particular issue was addressed but reverted back.
I think it's fine to move up to 3.3.0 as that's the version which all the AndroidX compatible plugins are using as well.
I always wanted to keep the version parity of this plugin compatible with the main library, but that doesn't look very likely now given that this change will require a major version bump 😢.


Is there a limit how many Vnet rules  that can be added

Currently I have one Vnet rule that allows our prod VM to communicate with a subnet. I want to add a separate sub net where our dev test server are running  that I want  to communicate to this Mysql  server. From UI it lets me add , but want to check for experts that adding this new rule and subnet will not cautions any issues to  our prod rule

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 27c517a9-3ff8-4082-8905-e8007158d91f
Version Independent ID: 3a59e804-c4db-1024-5f4d-fc974a310e69
Content: Create and manage Azure Database for MySQL VNet service endpoints and rules using the Azure portal
Content Source: articles/mysql/howto-manage-vnet-using-portal.md
Service: mysql
GitHub Login: @mbolz
Microsoft Alias: mbolz

@sunilprabha Thank you for the detailed inquiry. Are you planning on making this a temporary configuration or a permanent configuration item? Just because you can doesn't mean you should and I don't know your environment thoroughly. I suggest you have the means to test your network routing before and after this change is made, and be prepared to roll-back the change if unforeseen issues arise.

@sunilprabha The answer your question specifically about the total number of VNet rules, the answer is 128. There is some good information available here: Limitations

Each Azure SQL Database server can have up to 128 ACL entries for any given virtual network.

The MySQL specific documentation does not include this detail.

Thanks guys, I am sure of  adding a vnet rule using this configuration permanently.  By adding a new Vnet rule should not interfere with the existing Vnet rule.

We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Add new configuration object

Add new configuration object, to try and clean up the parsing class a little.
This isn't ready to merge yet - but could be a good discussion point

Coverage decreased (-0.66%) to 89.42% when pulling 4f2fa21e8709a87df21b59bbb09aa020ac7ab728 on configurationObject into 73a5fabad6ed2c9c8b7515f6cce1f547ac29d897 on master.


Question: How to set width and height of the component in Streamlit app?

How to set the width and height of the component in Streamlit app? The new version seems remove the two parameters.

Hi @kidoln
At that time, height and width were only used to adjust the height and width of the HTML iframe, and could not actually adjust the height and width of the pygwalker UI.
Therefore, after using the Streamlit component to render the pygwalker UI, these two parameters were removed.

Hi @kidoln
There is currently no way to set the height of pygwalker.
Because the height of graphic-walker ui is fixed, even if we adjust the height to 500, this will show part of the ui and the user will need to slide to see different parts of the ui.
Can you give an case where you need to use the height parameter?


Enable modin-on-omnisci for modin cloud

We need to try Modin Cloud with new experimental engine which is based on OmnisciDB
At the moment https://github.com/intel-go/modin/tree/omnisci-on-ray branch should be used
In general instructions for using omnisci engine are here - https://github.com/modin-project/modin/issues/1652#issuecomment-679925921
(current PR in master branch - https://github.com/modin-project/modin/pull/1893)

REF: draft enabling the feature: https://github.com/intel-ai/modin/pull/109/ (not complete, needs cluster yml fixed)


MinCount validator to be present a level above or below the PropertyRequirements

In line 297 of OCPDoorHeatExchangerHardwareManagement Profile, MinCount Validator to be Present a level above or below the PropertyRequirements as per Redfish Interoperability Profile Specification.
Reference: https://github.com/DMTF/Redfish-Interop-Validator/issues/129

The MinCount property (297) cannot be within the PropertyRequirements property.  It should be place between 290 and 291, to specify the requirement of two Temperatures array property elements.
290        "Temperatures": {
291          "PropertyRequirements": {

Hi, @mgthompson @jcleung5549 can this be resolved quickly?

If you are in a rush, feel free to create a pull request.  I don't know when Michael will get to this.

I created two pull requests with the fixes.
On Thu, Jan 6, 2022 at 11:43 AM John Leung @.***> wrote:

If you are in a rush, feel free to create a pull request. I don't know
when Michael will get to this.
—
Reply to this email directly, view it on GitHub
https://github.com/opencomputeproject/HWMgmt-OCP-Profiles/issues/32#issuecomment-1006737951,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAVIHTPSYT5UWTQ7EZLHFDUUXBE7ANCNFSM5JWYXO6Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Michael Thompson

Fixed in #36


Updated Readme

Added reference to an article about Super Star Trek in C++.  (I'll make a GitHub repo for it shortly)

I like this idea of linking other implementations, as we do have to narrow down the languages we support based on memory safety and popularity, etc, per recent discussion! Good idea!


Question: Would this library support dragging a new grid item into it?

Hello,
I am wondering if I have a drag and drop outside of this library if it would be possible to use it. i.e.- I would like to have a menu of sorts where a user can drag off a box (lets say a 1x1) and drag it into the grid area and drop it in, with the grid reacting to the new item. Would something like this be possible?
Thanks very much!

Addressed in #289.


feat: Support Client Certificate Authenticate

Motivation
We want to use rest-api-client for the subdomain in which Client Certificate Authentication is enabled
NOTE: This feature is available only in Node.js environment.
Overview
We haven't fixed the interface and need to discuss, but something like the following: The client can accept the client certificate file and its password.
const client = new KintoneRestAPIClient({
  baseUrl: "https://example.cybozu.com",
  auth: {
    clientCert: {
      file: fs.readFileSync('PATH_TO_CERT_FILE'), // or the path to certificate file
      password: 'PASSWORD'
    }
  }
});

Released in v1.2.0.


Fix lcd non-random ROIs not returned

Codecov Report

Merging #17 (6f4a45d) into master (ce24e62) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #17      +/-   ##
==========================================
+ Coverage   83.29%   83.31%   +0.01%
==========================================
  Files          18       18
  Lines         862      863       +1
  Branches      188      188
==========================================
+ Hits          718      719       +1
  Misses         99       99
  Partials       45       45

Impacted Files
Coverage Δ

src/dicom_image_tools/image_quality/lcd.py
93.54% <100.00%> (+0.10%)
: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 ce24e62...6f4a45d. Read the comment docs.


Update New-SPEnterpriseSearchCrawlRule.md

Fix for #3638

Docs Build status updates of commit c889096:
:clock10: Full build: the average full build time is 13 min(s) 4 sec(s), this is based on the last 534 build(s) for this repository.

Docs Build status updates of commit c889096:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/New-SPEnterpriseSearchCrawlRule.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Spring layout 1

Description
A summary of the change
Layout 1 for Deployed Client Website.
Type of change

[ ] Bug fix
[x] New feature
[ ] Refactoring
[ ] Documentation

Problem
Briefly describe the root cause and analysis of the problem
Solution description
Describe your code changes in detail for reviewers.
Explain the technical solution you have provided and how it fixes the issue.
Tests
All unit tests pass?
Builds successfully?
Yes.
Attach screenshot of the expected result (e.g. UI of the website/ postman api call result/ database changes)

Provide steps in detail for reviewers to reproduce the expected result of the code changes.

Go to base1 folder under scholly folder.
Using npm run dev, the layout can be run in development mode.

Relevant document/ link (if any)
Any related documents could help reviewers to review this PR
Risk
Identity the risk level (low, medium, high) and indicate which component(s) will be affected if bugs exist.

Please combine scholly1 into scholly. We should have only one base app ideally.

Agreed - Team Spring & Kunj are currently working on this and figuring out how to integrate layouts, colours and make the selection process dynamic under one base folder.

Please combine scholly1 into scholly. We should have only one base app ideally.

Agreed - Team Spring & Kunj are currently working on this and figuring out how to integrate layouts, colours and make the selection process dynamic under one base folder.

SGTM. Can we then do reviews after that's been implemented? We won't be able to merge this anyhow

Please combine scholly1 into scholly. We should have only one base app ideally.

Agreed - Team Spring & Kunj are currently working on this and figuring out how to integrate layouts, colours and make the selection process dynamic under one base folder.

SGTM. Can we then do reviews after that's been implemented? We won't be able to merge this anyhow

Yep, valid point. Spring has its meeting tomorrow, so we'll make the necessary changes then and update accordingly.

As discussed over Slack, please integrate into the existing base app. Closing for now.


window half black

Issue Type: Bug
Often, after installing the latest version, the screen becomes half grey (right part of the screen) and it's unusable. In order to use VSC I have to quit and reopen the app.
VS Code version: Code 1.27.2 (f46c4c469d6e6d8c46f268d1553c5dc4b475840f, 2018-09-12T07:04:11.329Z)
OS version: Darwin x64 17.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz (4 x 2700)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 3, 3

Memory (System)
16.00GB (0.24GB free)

Process Argv
/Applications/Visual Studio Code.app/Contents/MacOS/Electron

Screen Reader
no

VM
0%

Extensions (17)

Extension
Author (truncated)
Version

vscode-eslint
dba
1.6.0

vscode-babel-coloring
dza
0.0.4

vscode-pull-request-github
Git
0.1.6

beautify
Hoo
1.4.4

Kotlin
mat
1.6.0

rainbow-csv
mec
0.6.0

mssql
ms-
1.4.0

python
ms-
2018.8.0

vsliveshare
ms-
0.3.709

material-icon-theme
PKi
3.6.0

live-sass
rit
3.0.0

LiveServer
rit
5.1.1

code-settings-sync
Sha
3.1.2

react-beautify
tai
0.3.0

vscode-react-native
vsm
0.6.17

quokka-vscode
Wal
1.0.149

JavaScriptSnippets
xab
1.7.0

(1 theme extensions excluded)

We are building exploration builds that use a much newer version of Electron (our UI framework). I wonder if this issue reproduces with one of these builds, could you try? Download:

macOS: Download
Linux: Download
Windows: Download

Hi Benjamin,
Thanks for the message. I try and let you know. The hard part is to
reproduce the issue, that appears randomly.
Alberto
On Mon, 24 Sep 2018 at 19:08, Benjamin Pasero<EMAIL_ADDRESS>wrote:

We are building exploration builds that use a much newer version of
Electron (our UI framework). I wonder if this issue reproduces with one of
these builds, could you try? Download:

macOS: Download
https://az764295.vo.msecnd.net/exploration/86ff36a1cca2c5763c9c57bda2ec8d3443538c89/VSCode-darwin-exploration.zip
Linux: Download
https://az764295.vo.msecnd.net/exploration/86ff36a1cca2c5763c9c57bda2ec8d3443538c89/code-exploration_1.28.0-1536650436_amd64.deb
Windows: Download
https://az764295.vo.msecnd.net/exploration/86ff36a1cca2c5763c9c57bda2ec8d3443538c89/VSCodeUserSetup-x64-1.28.0-exploration.exe

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/59197#issuecomment-423914523,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdIBmTAmwTLPZDry3umZ3RpqLeUuhDfhks5ueKEngaJpZM4W1-hV
.

--
Best,
Alberto Camillo, Software Engineer
Terem Technologies
(p) 02 9430 6877
(e<EMAIL_ADDRESS>(w) terem.com.au http://www.terem.com.au
Suite 2.02, Level 2, 55 Clarence Street, Sydney NSW 2000.
This email and any files transmitted with it are confidential and intended
solely for the use of the individual or entity to whom they are addressed.


FluentDataGrid: Improve data refresh logic in FluentDataGrid

Pull Request
📖 Description
This PR improves the data refresh logic in the FluentDataGrid component to better handle state changes, particularly those related to pagination. It addresses potential inconsistencies that could occur during rapid state changes or when other events trigger state updates.
This change is a fix that enhances the reliability and consistency of the FluentDataGrid component.
🎫 Issues

👩💻 Reviewer Notes
Reviewers should focus on:

The changes in the OnParametersSetAsync method, particularly the mustRefreshData calculation.
The updated RefreshDataCoreAsync method, noting the new position of the _lastRefreshedPaginationStateHash update.

📑 Test Plan

Manual testing has been performed.

✅ Checklist
General

[ ] I have added tests for my changes.
[x] I have tested my changes.
[ ] I have updated the project documentation to reflect my changes.
[x] I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

[ ] I have added a new component
[ ] I have added Unit Tests for my new component
[x] I have modified an existing component
[ ] I have validated the Unit Tests for an existing component

Add fix for #2512

@dvoituron I updated PR you may want to review again.


Add initial implementation of remove all cache

A shot at https://github.com/nickoneill/Pantry/issues/33

👍


Index out of range exception

Code:
/proc/flick_overlay(image/I, list/show_to, duration)
    for(var/client/C in show_to)
        C.images += I
    addtimer(CALLBACK(GLOBAL_PROC, /proc/remove_images_from_clients, I, show_to), duration, TIMER_CLIENT_TIME)

BeeStation

Exception fixed by f5975c3849e0cb2143334283d93a869d58a652bc, issue with code snippet fixed by 610f69ea209ea8b30613303decaab5b82d1f0f8b.


async behaviour of an Isolate's top level function

Aye aye good people,
I'm experiencing a weird behavior
when using the top level function of an isolate asynchronously;
you can find example code here,
as top level function of an isolate this works:

String _syncHandle(int data) {
return 'done';
}

and this doesn't:

Future _syncHandle(int data) async {
return 'done';
}

can anybody explain me why?
(or if should work, why isn't doing so in my code?)
I noticed that I'm not the first one to ask this question,
which anyway was never answered
https://github.com/flutter/flutter/issues/17671.
I'm begging for help also in Stackoverflow and I can say that I'm not the only one with this problem.
thank you in advance for any help you can give me
Francesco

I can second that problem same here! Simple setup: I have an async encryption method called inside of my isolated method.
how I call/create the Isolate:
`ReceivePort receivePort =ReceivePort(); //port for this main isolate to receive messages.
_isolate = await Isolate.spawn(_encryptVault,receivePort.sendPort);
receivePort.listen((data) {
print("Continue with my work");
_refresh();
});`
now what happens:
Empty Method:
String _encryptVault(SendPort port) {       print("encrypting vault");       return "test"; }
This works like charm.
Async Method:
String _encryptVault(SendPort port) async {       print("encrypting vault");       String key = await Encoder.encrypt();       print("vault encrypted");       return key; }
This does not work
the outout is

encrypting vault

Sync Method with "then":
String _encryptVault(SendPort port) {       print("encrypting vault");       String key = await Encoder.encrypt().then((key) {             print("vault encrypted");             return key;       }); }
This does not work
the outout is

encrypting vault

##And in case there is a need. Thats the output of my flutter doctor:

C:\src\flutter\bin\flutter.bat doctor --verbose
[√] Flutter (Channel stable, v1.2.1, on Microsoft Windows [Version 10.0.17763.107], locale de-AT)
• Flutter version 1.2.1 at C:\src\flutter
• Framework revision 8661d8aecd (8 weeks ago), 2019-02-14 19:19:53 -0800
• Engine revision 3757390fa4
• Dart version 2.1.2 (build 2.1.2-dev.0.0 0a7dcf17eb)
[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at C:\Users\mga\AppData\Local\Android\sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-28, build-tools 28.0.3
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
• All Android licenses accepted.
[√] Android Studio (version 3.3)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin version 34.0.1
• Dart plugin version 182.5215
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
[√] Connected device (1 available)
• FRD L09 • 73QDU17517001330 • android-arm64 • Android 7.0 (API 24)
• No issues found!
Process finished with exit code 0

Hi, sorry for the delay, I just want to check if you could solve this problem.

I was young and inexperienced....
and I was passing "send.send" in the isolate pretending to get back complex data


Fix the ng-table.min.js and ng-table.css url

I think this is what you expected. Fix for issue #7

You will need to fix the actual project. Not the overlay. We want the fix to be in, forever. Not just for the overlay.
https://github.com/apereo/cas/tree/master/webapp-mgmt
and I think this is already fixed. Thanks for the patch though. A for effort :)


🔧 Add earthlyignore file

What this PR does / why we need it:
We need to add .earthlyfile to enhance build speed locally. Without this we are sending a lot of unneeded
files to buildkit, especially true for large ISOs under build/.
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 #693

upgrade tests seems to use build https://github.com/kairos-io/kairos/actions/runs/4057373981/jobs/6983975639

upgrade tests seems to use build kairos-io/kairos/actions/runs/4057373981/jobs/6983975639

weird, that downloads the required iso into the root dir. I would instead change the job to move the ISO into the test dir or a special dir instead of removing the earthly file on those tests...

TBH, I think you are correct. It's the right thing to do, but I don't want to mess further with the Earthlyfile.
Every commit push cycle is really slow, and we can do this in a separate PR to make all the tests use the same directory.
Right now, I just want all the team to use the earthlyignore in the most basic version.
Once this is the master, everyone uses it, so we can incrementally improve it.


Modifications to enable packaging and deployment via pypi

Hi,
I made a few changes to enable packaging of metk using the standard python packaging structure that can be added to pypi and installed via pip install
To make the wheel file that can be used with pip install
make dist

Please feel free to reject, or request other changes.
thanks,
chris

Merged a pull from coleslaw481, thanks Chris!


Project creation using JSON fails

Describe the bug

Project creation using JSON always gives Please specify record path first
To Reproduce
Steps to reproduce the behavior:

Go to 'Create Project and Click on clipboard'
Paste this data

[
    {
        key1: value1,
        key2: value2
    }
]

Select Parse Data as JSON Files
Click on Create Project & see error

Expected behavior

Create Project should work smoothely.
Screenshots

Additional context

When I click on Update Preview and then click on Create Project it works smoothly. But it alwasy gives this error when I try to create JSON Project.
Also there is dummy button with label Please specify a record path first and clicking on it makes no change.

As GitHub's syntax highlighting suggests, your JSON payload is invalid.

@wetneb  The same error occurs on this dataset too!
{
    "glossary": {
        "title": "example glossary",
		"GlossDiv": {
            "title": "S",
			"GlossList": {
                "GlossEntry": {
                    "ID": "SGML",
					"SortAs": "SGML",
					"GlossTerm": "Standard Generalized Markup Language",
					"Acronym": "SGML",
					"Abbrev": "ISO 8879:1986",
					"GlossDef": {
                        "para": "A meta-markup language, used to create markup languages such as DocBook.",
						"GlossSeeAlso": ["GML", "XML"]
                    },
					"GlossSee": "markup"
                }
            }
        }
    }
}

Not for me, I can import this without a problem.

@wetneb Please see this Screen Recording for the bug
https://drive.google.com/file/d/1NzWsR5Oz9Qg9GclK0nPWCMVm3FzR8dst/view?usp=sharing

@kushthedude You have to use your mouse to hover around over "some" part of the JSON that you wish OpenRefine to try to treat as a Record.  In your screen recording, something like the GlossEntry would probably be useful, especially if you had many GlossEntry nodes inside the GlossList.
So that part is not a bug.
However, we COULD create a new issue to validate the JSON during preview and warn the user when it is invalid. (although huge JSON files might be a slight problem to check for complete validity, but still)

Closing because this is based on a misunderstanding of how the JSON importer works.


Verify unified fides can run without a fides/ctl/ops.toml file

Bug Description
We currently have several reports of issue with running fidesops when no fidesops.toml file is present. There is ongoing work to fix this, and we should verify that it is still possible to run without the file present once the repos are merged.
Steps to Reproduce

docker-compose up when no fides/ctl/ops.toml file is present
Server will error instead of loading.

Expected behavior
Server should be able to start with environment variables and not require a toml file.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

Version:
OS:
Python Version:
Docker Version:

Additional context
Add any other context about the problem here.

#1718 and #1982 have verified this works now.


TAKS: Don't require a specific neos version for master branch

:+1:

:+1: Makes sense I guess. Thanks for taking care


[SCB-667] fix gracefully shutdown is not work in some case

Signed-off-by: zhengyangyong<EMAIL_ADDRESS>Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Run mvn clean install to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.1%) to 87.225% when pulling 4d595f7d555296883976cab5d8f54ec81d0e76ac on zhengyangyong:SCB-667 into b81196f6eedf220dfb87b77b3a548cd201a7f3ac on apache:master.


Everything on Explorer blurs out

VS Code Version: 1.54.3
OS Version: macOS Mojave 10.14.6

Steps to Reproduce:

Have a code that downloads a large amount of files (download then delete)
Everything will start blurring

Does this issue occur when all extensions are disabled?: Yes/No Yes. I was able to reproduce this after disabling all the extenstions

I can not reproduce. Thanks for filling this issue. Can you reproduce using Visual Studio Code Insiders https://code.visualstudio.com/insiders/

Isn't it a problem with the hardware of your machine @rpai9 ? maybe the GPU has a problem, because i had the same problem, and because of it i refunded a 1 year old laptop i used from ACER...

I can not reproduce. Thanks for filling this issue. Can you reproduce using Visual Studio Code Insiders https://code.visualstudio.com/insiders/

I don't seem to be able to replicate it on the insider edition

Isn't it a problem with the hardware of your machine @rpai9 ? maybe the GPU has a problem, because i had the same problem, and because of it i refunded a 1 year old laptop i used from ACER...

I'm using a 2019 Macbook pro with Radeon Pro 555X 4 GB Intel UHD Graphics 630 1536 MB. I know it's not the best GPUs but it works fairly well for all the other things. But it's entirely possible that GPU might be having a hard time keeping up with the changes. I'm not sure if our IT is going to replace the laptop just because of this issue.

If you can not reproduce with insiders than it is probably fixed in latest, thus closing


hunspellDicts: add Czech and Slovak dictionary

Also bumps libreoffice dicts version <IP_ADDRESS> -> <IP_ADDRESS>.
Motivation for this change

missing Czech dictionary

Things done

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

[x] NixOS
[ ] 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.

cc @vlaci

@GrahamcOfBorg build hunspellDicts.cs-cz hunspellDicts.hu-hu hunspellDicts.ru-ru hunspellDicts.sk-sk