Usage of rfactory in karma runner / jasmine / requirejs

I like direction taken with rfactory RequireJS plugin to provide dependency factories. I'm running unit tests with karma. I've prepared a sample configuration but currently I'm receiving an error "Please ensure that rinject plugin is loaded before any modules you try to resolve". Plese check https://gist.github.com/mstefaniuk/2e12de73e6e0ed548a4e
Could you provide guidelines how to use rfactory with it?

Hi,
The plugin requires 'rfactory' module to be loaded before any test modules or dependencies you try to mock.
In test.main.js configuration:
 - deps: ['rfactory'].concat(tests),
doesn't guarantee you any order. So a test or it's dependency can be loaded before the plugin.
It can be resolved by ensuring the explicit order of files loading.
For example, in test.main.js:
var tests = [];
// your existing logic to collect files
requirejs.config({
    baseUrl: '/base/app',
    paths: {
        rfactory: '../lib/rfactory'
    },
    deps:['rfactory'] // this is loaded before any requirejs module
});
require(tests, function(){
    window.__karma__.start();
}

Or
var tests = [];
// your existing logic to collect files
requirejs.config({
    baseUrl: '/base/app',
    paths: {
        rfactory: '../lib/rfactory'
    }
});
require(['rfactory'], function(){
   require(tests, function(){
      window.__karma__.start();
   }
})

One more suggestion:
For a new project I would recommend replacing 'Jasmine' with 'Mocha' + 'chaijs' + 'sinon'
I choose Jasmine for the example since it's one single library so it's makes the example more compact.
For the real code:

The combination above offers more features then 'Jasmine' solely.
'Mocha' doesn't require workarounds to load tests with requirejs. Workarounds are in https://github.com/achudnovskij/rfactory/blob/master/example/spec/SpecRunner.html I'm not 100% they will work fine with Karma/PhantomJs

I have a small example of Karma + Mocha setup: https://github.com/achudnovskij/mocha-incremental-boilerplate . I's set up to run through Grunt but it has karma configuration

It looks that in my case it was stupid error in under-test.js:
define(['dependency', function(dependency){
    return {
        check: function() {
            dependency.callMe('kuku');
        }
    };
}]);

within define call there was unclosed dependency list square bracket (Angular alike). After fixing it behaves as expected even with deps: ['rfactory'].concat(tests).

Ok,
Anyway, I think your test run logic can benefit from the fix I propose. There is a risk of failing unexpectedly because of script loading sequence. It can potentially make the test runner unstable.
Thanks!

I would explore your approach once more. In example specs of rfactory (in rfactory/blob/master/example/spec/SpecRunner.html) there is:
require(['domready!', 'rfactory','spec/unit/indexView', 'spec/functional/indexView'], function(){
  window.onLoadHandler();
});

and I think that it is the same as deps: ['rfactory'].concat(tests).
Do you expect any differences?

Hi,
No differences, I'll push a change with a comment today.
This should be split into 2 require calls to enforce the sequence.
It's likely to cause instability with real examples.
Please let me know is you face any issues with setting this up with karma.
I can help.
We use this injection technique and run 2k+ tests in karma and in browser.
Thanks!
Andrey
On Mon, Mar 16, 2015 at 7:37 AM, Marcin Stefaniuk<EMAIL_ADDRESS>wrote:

I would explore your approach once more. In example specs of rfactory (in
rfactory/blob/master/example/spec/SpecRunner.html) there is:
require(['domready!', 'rfactory','spec/unit/indexView', 'spec/functional/indexView'], function(){
window.onLoadHandler();
});
and I think that it is the same as deps: ['rfactory'].concat(tests).
Do you expect any differences?
—
Reply to this email directly or view it on GitHub
https://github.com/achudnovskij/rfactory/issues/2#issuecomment-81704300.


Add functionality to open paths with spaces in iTerm2

I'm currently running into the issue where when Finder is open to a directory with whitespace in the path, such as my local/Users/jaroddeweese/Library/Mobile Documents/com~apple~CloudDocs/, I am unable to open the directory in iTerm2 with FiScript extension. This seems to be because $PATH is not escaped when passed to the script.
My terminal will open but prints the following message out:
cd /Users/jaroddeweese/Library/Mobile Documents/com~apple~CloudDocs/Documents; clear
cd: string not in pwd: /Users/jaroddeweese/Library/Mobile

The following is the relevant portion of my Open iTerm Window AppleScript script, my workaround and what I believe to be the solution:
tell current session
write text "cd " & quoted form of $PATH & "; clear"
end tell

Note how I added quoted form of to the script.
If this is the permanent fix to the problem, any other relevant places will need to be updated also, i.e Open Terminal Window, Open Terminal Tab etc.

Same issue here and your suggestion fixes it 👍

Fixed in #8


Is it possible to distinguish between identical keyboard models?

I have two identical keyboards which I would like to be able to use simultaneously, each with its own layout, while also being able to share state between them (i.e. having key presses on one affect the other).
Is it somehow possible to distinguish between them, given that they have the same vendor and product IDs?
$ lsusb
...
Bus 001 Device 009: ID 258a:002a BY Tech Gaming Keyboard
Bus 001 Device 012: ID 258a:002a BY Tech Gaming Keyboard
...

I don't think that this is possible yet. Also referencing #289 since if/when this is implemented, you will presumably also want to bind to keyboard-specific keys.

I don't think that this is possible (yet?).

What would it take to implement?
I'm currently using kmonad with two separate instances, each with the same defsrc but different device-file (under /dev/input/by-path/...). This works and I can have different layouts/layers on each keyboard, but obviously it's lacking the shared state aspect that keyd could provide.
I'm not (yet) familiar with how keyd does its device identification and capturing, but perhaps the config format could be extended to allow for optionally specifying the device path (or id) and an assigned name, which could then be used to refer to that specific keyboard in the rest of the config. For example:
[ids]
# backwards-compatible definition, matches all devices with these IDs
123a:456b

# narrows down to specific device despite sharing IDs
258a:002a:/dev/input/by-path/kbd_L = "Left"
258a:002a:/dev/input/by-path/kbd_R = "Right"

(quotes around the name might make it clearer that it's a made-up name and not a word that's part of the config - just a suggestion)
For the rest of the config, I guess there'd also have to be an optional way to specify if a certain layer or binding should only apply to a specific keyboard.
For example, the layer heading could become:
[main] # standard layer, applies to every ID matched
...

[mylayer1 "Left"] # layer applies only to "Left"
...

[mylayer2 * "Left"] # layer applies to every device except "Left"
...

[mylayer3]
## these apply to all devices
w = C-right
b = C-left

## These are still part of mylayer3
## and only apply to specified devices
["Left"]
a = C-a
s = C-S-s

["Right"]
h = C-left
j = C-down
k = C-up
l = C-right

Obviously there are more issues to iron out and there may be better notations for this, but it's a start.
Thoughts?

I'm not (yet) familiar with how keyd does its device identification and capturing

Devices are currently identified exclusively through their product and vendor ids. Uniquely identifying a device would require a unique id. USB devices are supposed to report a unique serial number, but from what I can tell many manufacturers omit this in the HID report (I will need to do more research).

I'm currently using kmonad with two separate instances, each with the same defsrc but different device-file (under /dev/input/by-path/...).

This is a fickle approach since device paths change depending on things like the USB port they are plugged into, making them unsuitable as unique identifiers. If this gets implemented it would need to be platform agnostic.

For the rest of the config, I guess there'd also have to be an optional way to specify if a certain layer or binding should only apply to a specific keyboard.

There are no plans to support per device key remapping within the same config (#289) for a variety of reason, however I am not opposed to a simple extension of the [ids] section which allows unique device ids to be used in place of vendor/product id pairs.
E.G
[ids]
<vendor>:<product>
<unique id>

Thanks for the explanation, it seems that keyd is unsuitable for my use case then.


better detection of roam studio

If the roam studio extension is also loaded, roam-reference-path must do some adjustments to to the top offset. This is implemented since roam studio was launched, however the automatic detection of roam studio is not working well, as can be seen in this screenshot:

Hi,
It seems that this issue has not been solved yet. The screenshot showase the situation. Could you please help me solve it? Thank you!

The commits mentioned above will now detect if Roam Studio is present and do the necessary adjustments.
However there is another situation where those adjustments are not enough: when the block is H1/H2/H3, Roam Studio will add more styles (margin-top, margin-bottom) to children element, which ruin our computation for the height.

A possible solution: when computing the height, also take into account those elements that have the extra height, which in this case are <div class="controls"><div>. In a normal situation we have current.y - previous.y = 0, but not here. That difference seems to be what we need to have the correct height.


Wrong flag for Norway

The dimensions are off, as per https://en.wikipedia.org/wiki/Flag_of_Norway#/media/File:Flag_of_Norway_with_proportions.svg
The red color should be: #ED2939
The blue color should be: #002664

Should be much better now.. can't be 100% correct as all the flags are adjusted to 4x3 and 1x1

Thanks for the prompt reply, I didn't stop to think about your ratios. (22:16 for .no). :)


Authentication: write out acronyms => more likely to be found by search

Write out acronyms so they are indexed/searchable, e.g. Active Directory (AD)
Minor grammar and layout tweaks

LGTM

LGTM


Data-Sample vs Focal-Loss, which is the better method for data-imbalance task?

Thank you very much @clcarwin

The odds is that it may depends on your dataset.

The odds is that it may depends on your dataset.

Data

Data


Pull to refresh with both from top and from bottom

Is your feature request related to a problem? Please describe.
Current push to refresh works excellent, but only can choose from top or from bottom. In my App, I would like to  pull from top to return to previous page, and pull from bottom to navigate to next page. In current F7 version, there is only one ptr instance
Describe the solution you'd like
Should it possible to separate Top & Bottom ptr instances ?
Thanks for your excellent working for F7~~

in Framework7 verison 1.2.0,This question will not exist,But I recently updated the old version to the new one v5.0.5，Pull to refresh with both from top and from bottom cannot be supported at the same time，Now i want to know when this requirement  will be completed
Thanks for your excellent working for F7~~


Add Windows display names

Per comments in
https://stackoverflow.com/q/53300222/634824
It would be useful for cross-plat scenarios to have a way to return Windows display names.

Also this one:
https://stackoverflow.com/q/54758449/634824

Currently figuring out how to make Windows and Linux-based microservices agree on the timezones we're dealing with, so this is definitely good news to me.

Check out https://github.com/tomkludy/TimeZoneWindowsResourceExtractor;
Or, just grab the output file.
https://raw.githubusercontent.com/tomkludy/TimeZoneWindowsResourceExtractor/master/TZResScraper/tzinfo.json.
I expect this to change occasionally, so I am currently working on some automation to run this once a month or so, and upload the output file to a CDN.  That part isn't done, but let me know if you think this would be worthwhile.

This looks really promising, and is essentially the approach I was considering.  Looks like you've done all the hard work for me. Thanks!
Rather than monthly, I suggest regularly checking the Windows DST/TZ blog and/or subscribing to its RSS feed, to know when there are new time zones or changes to the time zone strings.
Also, while I appreciate you extracting all the information you can, I'd be careful about usage of some of the fields:

BaseUtcOffsetMinutes and SupportsDaylightSavingTime aren't useful by themselves, as one also needs to know exactly when DST applies and the year-over-year changes.  Also, they aren't localized so if one wants them they can be pulled directly from a TimeZoneInfo object (at least on Windows).
StandardName and DaylightName are indeed strings that Windows localizes, but they're very poorly maintained.  I usually recommend they not be used anywhere.  Lots of bad data here, such as "Russia TZ 2 Standard Time" and "GMT Daylight Time".
Id - is in the data twice, and is only needed once
LCID - Doesn't hurt anything, but I'm not sure they are needed beyond the actual lookup

So basically, the resulting data needed (at least for this project) would be:
    {
      "Locale": "en-US",
      "TimeZones": {
        "Dateline Standard Time": "(UTC-12:00) International Date Line West",
        "UTC-11": "(UTC-11:00) Coordinated Universal Time-11",
        "Aleutian Standard Time": "DisplayName": "(UTC-10:00) Aleutian Islands",
        ...

Of course, feel free to leave as is if you like, but that's all that I would want to extract from your output.

Also, please add a license to your project, so I can use it. :)

Great information and suggestions.  I updated to match your suggested format, and dropped the rest (which I also was not using).

Check out https://github.com/tomkludy/TimeZoneWindowsResourceExtractor;
Or, just grab the output file.
https://raw.githubusercontent.com/tomkludy/TimeZoneWindowsResourceExtractor/master/TZResScraper/tzinfo.json.
I expect this to change occasionally, so I am currently working on some automation to run this once a month or so, and upload the output file to a CDN. That part isn't done, but let me know if you think this would be worthwhile.

That's a really nice work @tomkludy, thank you. I also regularly follow Microsoft DST/TZ blog, so whenever they change something, I would like to contribute your TimeZoneWindowsResourceExtractor project.
Again, thanks.

Awesome work!  Thank you!
I will use your data to complete this feature sometime in the next week or so.

This is complete in version 4.0.0.

@tomkludy - thanks again for providing the Windows display names.  I credited you in the readme accordingly.
Feel free to open an issue or PR any time you have updated data.  (which should correspond to announcements on the MS DST/TZ Blog).

Great job @mj1856 and @tomkludy. Well done!

Thanks @mj1856 and @tomkludy !  As the asker of https://stackoverflow.com/q/53300222/634824 I'll hopefully get a chance to use this new feature in our project soon.


Fixed a typo

Fixed a typo in callChannelFeed.
(Edit: moved question about a related feature to #2354)

That's definitely a typo. Thx for finding it.


Add configuration schema

Merging the Config schema to the pipeline branch to generate config.

Successfully merged Config file to pipeline branch


Enable compiler selection in vcpkg.json

I propose adding the ability to select the default compiler directly in the vcpkg.json manifest file. This feature would allow users to specify the compiler for building dependencies and set the CMake variables CMAKE_C_COMPILER and CMAKE_CXX_COMPILER for proper binary linking.
Current Approach:
Currently, setting the compiler requires using overlay triplets, which is not user-friendly. Users must create their own triplet and overlay triplet or use one from GitHub, often needing to fix bugs. This process is cumbersome and slows down the development workflow compared to the otherwise straightforward dependency management provided by vcpkg.
Suggested Improvement:
Allowing compiler selection in the vcpkg.json manifest would:

Simplify Configuration: Users can directly specify the desired compiler in the manifest file, eliminating the need for additional triplet files.
Improve Usability: Make the process more intuitive and accessible, especially for new users.
Enhance Productivity: Reduce setup time and potential errors associated with overlay triplets, leading to a smoother development experience.
This feature would streamline the development process, making vcpkg even more powerful and user-friendly.

Possibly Formats:
{
    "name": "Name",
    "version": "1.0.0",
    "compiler":  "gcc",
    "dependencies":[]
}

{
    "name": "Name",
    "version": "1.0.0",
    "compiler": {
        "name": "gcc",
        "version": "9..10.2",
        "path": "Optional path"
    },
    "dependencies": []
}

IMHO, this is impossible to maintain. And it's vcpkg that must be told to use some toolchain, not the other way around.

@Melnytskyi, Thanks for posting this issue, This feature is very difficult to add and maintain, because different compilers and different versions will affect the installation and compilation of different versions of ports in vcpkg, which is very inconvenient for users and maintainers.


replace putlog with producer sendLoglist interface

Replace the sending interface in the original SDK with the sending interface in go producer

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


aligning README.md and CONTRIBUTING.md to FINOS project blueprint language

aligning README.md and CONTRIBUTING.md to FINOS project blueprint language

Codecov Report

Merging #59 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev      #59   +/-   ##
=======================================
  Coverage   89.65%   89.65%
=======================================
  Files           9        9
  Lines          58       58
  Branches        6        6
=======================================
  Hits           52       52
  Misses          4        4
  Partials        2        2

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 3a23b2e...f63f477. Read the comment docs.


Set up DerBruden.com static site with Gatsby.

Summary
For #1.
Generates a pretty vanilla static site with Gatsby from a starter template: https://github.com/p1t1ch/gatsby-starter-vadyan. It's not the nicest looking but comes with some pretty great bells and whistles out of the box:

💬 Static type checking with Typescript
🥇 Linting environment with ESLint, Prettier, Husky & lint-staged
🎲 Testing environment with Jest, RTL & Cypress
👩🎤 CSS in JS styling with Emotion
📕 Work with components in Storybook
🌀 Transform SVGs into React components with SVGR
✨ Full PWA support
🧠 Apollo Client setup for dynamic data
🚦 Ready to use CI/CD setup with GitHub Actions
📊 Analyze generated build with Webpack Bundle Analyzer
💥 Write pretty imports with Webpack aliases

Set up some basic icons from https://icons8.com/icons/set/football.
Next steps
There's a lot of cruft that came along with this that I didn't completely set up and may not need (Storybook, Cypress, etc)
Next, I'll set up Gatsby's deploy to S3.
It'd be nice to automate this with GitHub Actions.

This broke the out-of-the-box E2E tests but I'm not going to worry about that for now.


NOTIFICATION_WM_MOUSE_ENTER doesn't emit when mouse button is being pressed

Godot version
3.3 Stable, v4.0.dev.custom_build [916c37de0]
System information
Manjaro Linux
Issue description
When normally entering the game with the mouse, the NOTIFICATION_WM_MOUSE_ENTER notification is emitted. But when a file is dragged into the game nothing happens. This issue is present in both Godot 3.3 and Godot 4.0-dev.
https://user-images.githubusercontent.com/47503977/135075827-2f6e89ec-4eb2-400b-960e-2256bc67780b.mp4
.
Steps to reproduce

Create a Node with a script including this code (3.3):

func _notification(what):
  if what == MainLoop.NOTIFICATION_WM_MOUSE_ENTER:
     print("Emitted")

Try to drag a file into the game window, "Emitted" is not printed.

Minimal reproduction project
Test.zip

Might be not caused by file dragging, but the pressed mouse button. See #20881

Then this is a duplicate, I suppose?

Ok I confirmed this is the same problem. But it involves windows not Controls, so it's probably triggered somewhere else. I'd leave it open.

Godot version
3.3 Stable, v4.0.dev.custom_build [[916c37d]

No one can correct this mistake, since no one wants to delve into the problem and does not consider it a problem. Nevertheless, this is a problem :) and this problem leads to unpredictable behavior of your game :)
This problem is solved by the logical layer that I attached in the adjacent branch, but it is not as effective as if it was solved natively, although it works correctly as you expect.

I'm running into this on 4.3 stable on macOS.
func _notification(what: int) -> void:
	if what == NOTIFICATION_WM_MOUSE_ENTER:
		print("enter")
	elif what == NOTIFICATION_WM_MOUSE_EXIT:
		print("exit")

Hold down the mouse anywhere inside the window.
Move the mouse outside of the window and then back in while holding the button down.
enter will not be printed until you release the mouse button 😢


Rails/Validates autocorrect producing questionable syntax

I'm not entirely sure if this is valid Rails syntax, but the autocorrect is switching from
validates_numericality_of :foo, only_integer: true, unless: :blank?
to:
validates :foo, numericality: { only_integer: true, unless: :blank? }
I think that ideally, it should be:
validates :foo, numericality: { only_integer: true }, unless: :blank?
Similarly, it's doing this:
validates_presence_of :foo, unless: :blank? to
validates :foo, presence: { unless: :blank? }
It's entirely possible that somehow this is valid, but it seems a bit off. Looking at the options in Rails' presence.rb it doesn't seem that they are valid options?
Using Rubocop 0.57.2

Those are valid options and they are common for all ActiveModel validators. They allow you to adjust behavior of each validator independently, as opposed to the top-level option which applies to all validators at once. So you can say, for example:
validates :foo,
  numericality: { only_integer: true, unless: :blank? },
  uniqueness: { unless: :duplicates_are_fine? }


MQTT-Broker auto reduce the message-publish number  when user sub some duplicate topics

Describe your question
When a client sub two topic that are related or duplicate
example topic:
"cnc/1/command/fileDispatch/+"  and  "cnc/1/#"
If I use some paid broker (Alibaba service).
And publish the message like this
topic :  "cnc/1/command/fileDispatch/123456",
The client will received message twice

but when I use MQTTnet Broker,  the client will only received the message once.

Does MQTTnet broker has some extra mechanism that make things different?
Which project is your question related to?

MQTTnet.Server standalone
Server
Generic

The specification (http://docs.oasis-open.org/mqtt/mqtt/v3.1.1/os/mqtt-v3.1.1-os.html#_Toc398718037, section 3.3.5 Actions), says:

When Clients make subscriptions with Topic Filters that include wildcards, it is possible for a Client’s subscriptions to overlap so that a published message might match multiple filters. In this case the Server MUST deliver the message to the Client respecting the maximum QoS of all the matching subscriptions [MQTT-3.3.5-1]. In addition, the Server MAY deliver further copies of the message, one for each additional matching subscription and respecting the subscription’s QoS in each case.

So it looks like that MQTTnet does what it MUST do and the Alibaba service does what it MAY do.

The specification (http://docs.oasis-open.org/mqtt/mqtt/v3.1.1/os/mqtt-v3.1.1-os.html#_Toc398718037, section 3.3.5 Actions), says:

When Clients make subscriptions with Topic Filters that include wildcards, it is possible for a Client’s subscriptions to overlap so that a published message might match multiple filters. In this case the Server MUST deliver the message to the Client respecting the maximum QoS of all the matching subscriptions [MQTT-3.3.5-1]. In addition, the Server MAY deliver further copies of the message, one for each additional matching subscription and respecting the subscription’s QoS in each case.

So it looks like that MQTTnet does what it MUST do and the Alibaba service does what it MAY do.

This solve my problem.
Truly appreciate your support.


MacOS/Cmake Supplied-JWT Linker Error

Hi,
I've been attempting to building on MacOS, with the NMOS_CPP_USE_SUPPLIED_JWT_CPP flag to true, causes Linking CXX static library libjwt-cpp.a Error
Assuming Boost/cpprestsdk are installed,
To Replicate:
cmake -B build -DNMOS_CPP_USE_SUPPLIED_JWT_CPP=TRUE -DNMOS_CPP_USE_SUPPLIED_JSON_SCHEMA_VALIDATOR=TRUE -DWEBSOCKETPP_INCLUDE_DIR="Release/libs/websocketpp"
cmake --build build
Thanks

It looks like the section in NmosCppDependencies.cmake that compiles this creates a static library for a header only library. It even has a comment # hm, header-only so should be INTERFACE library?

I've checked and confirmed NMOS_CPP_USE_SUPPLIED_JSON_SCHEMA_VALIDATOR works OK on Linux... it constructs an empty libjwt-cpp.a (8 Byte file). I suspect difference in behaviour with empty archives on macOS. The CI matrix only tests this flag on one platform. As the original author of the "hm" comment after the library was first added, I agree an INTERFACE library would be more elegant anyway... but I note one or two other potential issues - I think missing install(FILES) and I think wrong target_include_directories. I'll look at this when I get a chance.

I believe that is exactly the case, MacOS fails on an empty archive. Thanks for getting back to me!


Fix the type object with its full scoped name

When adding a type object, only the type name is added. Its scope is omitted. That may result in a conflict between two types with the same name but in different scopes. For example, mod1::structA and mod2::structA
module mod1 { struct structA {...}; };
module mod2 { struct structA {...}; };

This should have been addressed in this branch, which will be part of the next major release in upcoming months. Thanks!


Add support for JSON

text format seems to be dead and for proto3 jsonpb seems to be the human-readable encoding of choice.
Please support JSON for proto3.

This is addressed in pull request https://github.com/dinowernli/polyglot/pull/16.


12546-FSS-UI Refresh Batch Attributes Stored In Local Storage On Token Refresh Changes

Hi All, Please review and merge changes related to the refresh batch attributes stored in local storage on token refresh changes for FSS UI in main branch.

Hi Richard, Thanks for reviewing PR. I have done the changes as per comments. Can you please review this.

@gaurav15233 could you please resolve the merge conflicts?

Hi Chris, Thanks for reviewing this PR. I have resolved the merge conflicts. Can you please review again.


Poor performance with pretrained model

Hi @lufficc，we have trained vgg_ssd300 on Pasca VOC for 5000 iterations using pretrained model and configuration in 'vgg_ssd300_voc0712.yaml', the total loss on validation set is 3.231, the map is 0.1641. Is this value all right?

@NewMesc Hi, I ran into the same problem, have you solved it?

If you test directly using the pretrained weights, what's the result?


Handle tasks asynchronously

Review commit by commit.
I will test this branch on the DNPM server before merging

Running flawlessly on the central dnpm servers


API does not errors out when trying to add duplicate projects as targets to existing MultiCluster App.

Rancher server version - Build from master - Feb 25
Steps to reproduce the problem:
Create cluster c1.
Create a MultiCluster app  with target pointing to "Default" project of c1.
Use "View in API" option for this MultiCluster App.
Use "add Project" option and add the same "Default" project of c1 to MultiCluster App.
API does not error out in this case though we do not see duplicate targets being created.
Expected Behavior:
API should error out in this case.

Tested with latest  build from master:
API errors out with following error when trying to add duplicate projects as targets to existing MultiCluster App:
{
"baseType": "error",
"code": "InvalidBodyContent",
"message": "duplicate projects in targets c-ftgzf:p-gtqnk",
"status": 422,
"type": "error"
}


errors

The scripts have serveral errors in 1.19.2
would be cool if you specified a target version or smh

Define "errors"
And for which AP version?

my bad, if it doesn't find it's monitor it errors out weirdly.

close

Alright, I will try to improve that anyways in the future


GitLab data collection failed on v0.10.1

Describe the bug
I tried to collect GitLab data using v0.10.1 but encountered errors. I made two attempts, both with the exact same configuration for the pipeline, but the errors were different (but likely due to the same reason -- rate limit, if I have to guess).
To Reproduce (first attempt)

Use DevLake version v0.10.1
Set up our own GitLab instance
Try to collect GitLab project 11624398
See the following error message at progress 60.87%

The docker log:

To Reproduce (second attempt)

Try to collect GitLab project 11624398
See the following error message also at progress 60.87%

The docker log (pretty long so I had to attach 3 screenshots):

I had add the retry to it if the http StatusCode over 400, in pull request #1838


Update solana.tokenlist.json

{
"chainId": 101,
"address": "DNmxHPgeVLSofyAriirHybKoNx1baM2ufiHKs1W7YyPc",
"symbol": "RPN",
"name": "RoyalPangolins",
"decimals": 9,
"logoURI": "https://user-images.githubusercontent.com/93877972/140663252-e1ec7f7f-0b77-4165-9d2a-4868f8444cd5.png",
"tags": [
  ],
  "extensions": {
    "website": "https://royalpangolins.io/",
    "Discord": "https://discord.gg/XvjxsRzK",
    "twitter": "https://twitter.com/RoyalPangolins"

  }
},

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address: "DNmxHPgeVLSofyAriirHybKoNx1baM2ufiHKs1W7YyPc"
Token Name: "ROYAL PANGOLINS"
Token Symbol: "RPN"
Logo: "https://user-images.githubusercontent.com/93877972/140663252-e1ec7f7f-0b77-4165-9d2a-4868f8444cd5.png"
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

let me know if i have issue so i can make changes


Fix ei nov24 rel

This PR depends on #4
Fix handling of biosphere changes in newer ecoinvent releases
This PR addresses issues with processing biosphere changes in newer ecoinvent releases (3.10.1+) where:

in 3.11 EE Deletions sheet structure has changed to a new format with deletion/replacement exchanges.  (they added some weird multi-column thing)
in 3.10.1 EE deletions was empty which cause an error in 'source_target_biosphere_pair()'  when trying to 'data[0]'
Empty datasets were causing errors when adding to Datapackage due to change in randonneur.

Changes:

Added handling for new column format in EE Deletions sheet
Added data validation before Datapackage creation
Improved handling of empty datasets
Added better logging for format detection
Added early return for cases with no changes

Testing:

Works with older formats (pre 3.10)
Works with newer formats (3.10.1+)
Handles empty sheets correctly
Maintains same output structure for compatibility

Technical notes:

No changes to output data structure/format
Maintains backward compatibility with older versions

Side note also fixed the same error in #4  but for technosphere changes (I forgot about that earlier)

@jsvgoncalves thanks for your review and feedback :) should be ok now


Hotkeys for record screen

To record a video I need to run the command:
scrcpy -r file.mkv
This method is inconvenient if the scrcpy is already running and I need to make a video. I will have to return to the console, close the program, enter the command. After recording the video, I will have to close the scrcpy and reopen it without parameters.
I would like to be able to manage the recording of video from the scrcpy, namely

Start recording
Pause recording
Stop recording

For this, I can use third-party software as OBS, but when I test mobile applications in the scrcpy, switching to between different windows is distracting.

The problem for this is UX: how do you pass the filename when you record at any time (possibly several times per session)?

I think you should not ask the user to manually set the file name. It can be automatically generated. For example:
Scrcpy screen record 2019-03-25 at 7.30.40.mkv
I think we should add a new parameter to specify the directory for recorded videos.
For example:
scrcpy --record-dir ~/android_videos

Any news regarding this request?

Ref: https://github.com/Genymobile/scrcpy/issues/1186#issuecomment-592386916

I understand correctly that this method will solve the problem? Should we expect a feature?

Would be nice!

the client must request to the device to generate a new keyframe (typically, just restart the encoding like on device rotation)

Now we have the requestReset method, it should be easier to add a "restart encoding" command?
https://github.com/Genymobile/scrcpy/blob/665ccb32f5306ebd866dc0d99f4d08ed2aeb91c3/server/src/main/java/com/genymobile/scrcpy/video/SurfaceCapture.java#L21
I had experimented it in my internal fork, it works very well, and doesn't affect mirroring at all.
Another use case I got from a developer is streaming the video to multiple clients (using Scrcpy server and a custom client), without such command, newly connected clients won't have a keyframe to start decoding so there will be several seconds of corrupted image.

how do you pass the filename when you record at any time (possibly several times per session)

SDL3 added an API to open a save dialog, so when stopping recording we can ask the user where to save (or discard) it. But it doesn't exist in SDL2, so I think saving to a user specified folder with auto-generated file names is also ok.


python310Packages.apprise: 1.2.1 -> 1.3.0

Changelog: https://github.com/caronc/apprise/releases/tag/v1.3.0
Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

1 package failed to build:

changedetection-io

5 packages built:

apprise (python310Packages.apprise)
healthchecks
home-assistant-component-tests.apprise
moonraker
python311Packages.apprise

Would you mind updating the license attribute to bsd3?

Thanks for the hint. I will.
Waiting a bit...perhaps changedetection-io is updated as well.

Part of #218614


API deployments should not require changes to the ingestion server

Description

I noticed that we weren't receiving any API deployment messages in Slack, even though we've merged PRs which should have initiated an API deploy. After some investigation, I found that we were requiring changes made to the ingestion server prior to deploying the API:
https://github.com/WordPress/openverse/blob/a2782e8b45300be669c90b4da1abd801caa0263b/.github/workflows/ci_cd.yml#L926
Since these services are distinct, we shouldn't need changes made to the ingestion server in order to deploy the API. This PR removes that requirement.
Testing Instructions

Merge and hope :slightly_smiling_face:
Checklist

[x] My pull request has a descriptive title (not a vague title like
Update index.md).
[x] My pull request targets the default branch of the repository (main) or
a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible
errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

I compared two runs (after the dependabot PRs were merged). One had changes to the API, and the other - to ingestion server.
The ingestion server one did deploy. I think the difference is in the "Publish Docker images": it does not run if the ingestion server tests didn't run.
This needs step is not run, and the API is not deployed if changes are in API only:
https://github.com/WordPress/openverse/blob/1621ed0d68daf62420f997c97745f361191a9825/.github/workflows/ci_cd.yml#L931

Sorry, I was comparing the wrong runs: the "Publish Docker images" only runs on the "push" event, and it does run when there are api changes: https://github.com/WordPress/openverse/actions/runs/4309530451

This run is a good example though - the ingestion server tests didn't run and so a staging deployment of the API wasn't run, even though it should be. At least, that's what I suspect is happening 🙂 GitHub Actions doesn't give you much info beyond "this was skipped" so it's kind of hard to tell haha.

@obulat's understanding is correct, the deploy happens if either API or ingestion server has changed.
We will need to use the expanded form of the if condition which is seen in some other jobs.

What I don't get is why deploy-api was skipped even though all  its dependencies have passed.

Ah okay, thank you both. I was misunderstanding the situation and the code I altered. That said, I think the particular line change I made might still apply, since we don't have ingestion server deployments set up automatically (so no need to deploy the API on ingestion server changes). What do y'all think? I'm also confused about that @dhruvkb, I'll try to look into it more today.

Here are three example runs. All 3 affect the API, but the first 2 do not initiate API deployments. The only difference I can see from the 3rd (which did have a successful run) was the fact that the ingestion server tests were not skipped in this case:

https://github.com/WordPress/openverse/actions/runs/4309189828
https://github.com/WordPress/openverse/actions/runs/4314531114
https://github.com/WordPress/openverse/actions/runs/4310292072

@dhruvkb I think this has to be an interaction with the always() && logic and these lines in the publish-images task:
https://github.com/WordPress/openverse/blob/1621ed0d68daf62420f997c97745f361191a9825/.github/workflows/ci_cd.yml#L854-L855
I suspect this gets us into some weird behavior where the job itself runs but due to its dependencies being skipped it registers as "skipped" for downstream tasks.

Ha, this behavior is exactly described in the next comment on the original suggestion 😅 🤦🏼♀ https://github.com/actions/runner/issues/491#issuecomment-882886236

OKAY I think I got it! Per some of the discussion in that linked thread, I've changed may of our always() calls to something more appropriate, since always() will run even if the job is cancelled or fails. I also added that condition to the API deploy step so that it ignores the skipped state of its grandparents and runs when it should. Fingers crossed that this does it! 🤞🏼 🍀


Merge remote-tracking branch 'CenturyLinkCloud/master' into DBaaS-Getting-Started-Updates-Nov-v2

Updated all screenshots based on cyclops rebranding.  Added new KB on Replication.

@christineparr11 Doesn't appear as though any file changes were committed.  Let us know if we can help.


use backward application operator

This is a small change to replace (\a r -> a r) with the equivalent (<|). This use of (<|) comes up occasionally in package sources and seeing it used in this way really helped me to understand it.
Thanks!

First, this is cool and tweaked my perspective a bit, so thanks!
I'm not sure where it is on the spectrum of clear/too clever, so I'll come back and look again with a less pre-coffee brain.

I think you're right on this being too much of a clever trick, when the original works perfectly well and is immediately clear. It was jarring to see the first time even though it did help. On second thought, this exercise is just right as it is :smile:
Thanks!


[microNPU] add E2E tests with cascader wo striping

This commit adds end-to-end tests using the cascader
w/o striping. It needed few adjustments to the order
in which the arugments are provided to the entry point
function in AoT when both memory pools and devices
are present.
cc @Mousius @lhutton1

This is green now. :)

@ekalda @NicolaLancellotti could you take another look after the further changes to make sure you still approve?

LGTM!

Thanks @manupa-arm @NicolaLancellotti @ekalda!


Inconsistent and broken query-index

There are currently multiple inconsistencies or errors in our index (and/or the source page metadata it was generated from):

[ ] The sheet names should correspond to the media type name e.g. shared-blog should be shared-blogs
[ ] The media-type column is missing the media type on a lot of pages especially infographics and info-sheets
[ ] The media-type for videos is lowercase while the media-type for everything else is capitalized-case. This should be consistent and I would argue lowercase would be the better choice here; this way the case can be easily ignored for search and filters.

@dkuntze Can you please check if this issue is relevant or not because i am not sure about the exact case for which this issue was created.


Add an ability to automatically extract code from the code base into reStructuredText for a macro

Provide the ability to automatically convert source code into reStructuredText that can be included in a macro.
For instance, implement a way to take include code from a function in the comment above INJECTION_TEST:
Code snippet;
int add(int a, int b) { return a + b; }

reStructuredText:
.. code:: cpp

    int add(int a, int b) { return a + b; }

It likely depends on how its implemented, but it would likely be nice to let the user specify which coding language they want the reStructuredText to be in when they instruct VnV to extract code.

What about some living like this.
/**
 The code was
  .. vnv-code-block:: cb1
*/
INJECTION_POINT(...)
.......

INJECTION_CODEBLOCK_START(cb1)

INJECTION_CODEBLOCK_END(cb1)
....

We could also allow passing the function name I stead of a marked code block... that's doable as well but a little harder

I do like the idea of referencing a code block with cb# for code blocks. However, keeping track of the number may unfortunately be a bit difficult for large codebases. For instance, the following header file for the matplotlib-cpp library has 2986 lines of code: https://github.com/lava/matplotlib-cpp/blob/master/matplotlibcpp.h. This example may be an "edge case", but users may have some troubles once they have 20+ code blocks.
I think that having the method of
/**

INJECTION_CODEBLOCK_START(cb1)
<code to render>
INJECTION_CODEBLOCK_END(cb1)
....

should remain, but it would be nice to have extend its capabilities so that the user does not need to keep track of numbers. Enabling the user to input an arbitrary string instead of just a number to identify a code block would help disambiguate the code blocks. For instance, a user can use cb_populatePhaseVector and cb_writeToFile instead of cb3 and cb27. This may also satisfy the "passing the function name" method since the user could put cb_function_readCsv to reference a readCsv function.

Agreed. An arbitrary sting makes sense. I will probably add in the package
name as well so it's consistent with all the other INJECTION macros.
In the larger scheme of things this Is a kind of "static" data collection
(whereas the engine class provides dynamic runtime data collection).
I could see benefit in being able to reference arbitrary "static" data
inside comment blocks. I.e.
/**
The version is :vnv:static.version
The code was
.. vnv-code:: static.cobeblockname
**/
INJECTION_STATIC(version,10)
INJECTION_CODE(codeblockname)
...
INJECTION_CODE_END(codeblockname)
...
This would provide some resilience in terms of not hard coing values into
the comments
On Wed, Jun 22, 2022, 8:57 AM Justin Burzachiello @.***>
wrote:

I do like the idea of referencing a code block with cb# for code blocks.
However, keeping track of the number may unfortunately be a bit difficult
for large codebases. For instance, the following header file for the
matplotlib-cpp library has 2986 lines of code:
https://github.com/lava/matplotlib-cpp/blob/master/matplotlibcpp.h. This
example may be an "edge case", but users may have some troubles once they
have 20+ code blocks.
I think that having the method of
/**
INJECTION_CODEBLOCK_START(cb1)

INJECTION_CODEBLOCK_END(cb1)
....
should remain, but it would be nice to have extend its capabilities so
that the user does not need to keep track of numbers. Enabling the user to
input an arbitrary string instead of just a number to identify a code block
would help disambiguate the code blocks. For instance, a user can use
cb_populatePhaseVector and cb_writeToFile instead of cb3 and cb27. This
may also satisfy the "passing the function name" method since the user
could put cb_function_readCsv to reference a readCsv function.
—
Reply to this email directly, view it on GitHub
https://github.com/vnvlabs/vnv/issues/3#issuecomment-1163064308, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AH3KA44ICFLUO6KBJN225VTVQMESPANCNFSM5ZM67WJQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Allow multiple actions to map to same simplesource command topic(s)

Consider the following SourcingApp setup:
CommandSpec<GenericRecord, CreateReservationSubSaga, AccountKey, AccountCommand> createReservationSpec;

CommandSpec<GenericRecord, ConfirmReservationSubSaga, AccountKey, AccountCommand> confirmReservationSpec;

...

new SourcingApp<>(
        AvroSerdes.actionSerdes(AppShared.SCHEMA_REGISTRY_URL, false), builder ->
        ...
).addCommand(createReservationSpec, builder ->
        builder
                .withTopicNamer(TopicNamer.forPrefix("auction_avro_", "account"))
                .withDefaultConfig(6, 1, 7)
).addCommand(confirmReservationSpec, builder ->
        builder
                .withTopicNamer(TopicNamer.forPrefix("auction_avro_", "account"))
                .withDefaultConfig(6, 1, 7)
)

This results in the following exception:
Exception in thread "main" org.apache.kafka.streams.errors.TopologyException: Invalid topology: Topic auction_avro_account-command_response has already been registered by another source.
	at org.apache.kafka.streams.processor.internals.InternalTopologyBuilder.validateTopicNotAlreadyRegistered(InternalTopologyBuilder.java:569)
	at org.apache.kafka.streams.processor.internals.InternalTopologyBuilder.addSource(InternalTopologyBuilder.java:389)
	at org.apache.kafka.streams.kstream.internals.InternalStreamsBuilder.stream(InternalStreamsBuilder.java:48)
	at org.apache.kafka.streams.StreamsBuilder.stream(StreamsBuilder.java:137)
	at org.apache.kafka.streams.StreamsBuilder.stream(StreamsBuilder.java:97)
	at io.simplesource.saga.action.sourcing.CommandConsumer.commandResponseStream(CommandConsumer.java:21)
	at io.simplesource.saga.action.sourcing.SourcingApp.lambda$addCommand$0(SourcingApp.java:58)
	at io.simplesource.saga.action.sourcing.SourcingApp.run(SourcingApp.java:118)
	at io.simplesource.example.auction.server.SagaActionProcessorApp.main(SagaActionProcessorApp.java:109)

This is a general limitation of the Kafka streams.
When defining a topology, you can only one stream from a source topic (i.e. where the stream is created from the topic).
One solution (maybe you can call this a workaround) is to make a single action for the aggregate and make that polymorphic. So in this case have an account command that handles create and confirm reservation, and other account operations.


Write structured error output details at the end of the log stream

Currently Gradle users have to hunt for the following important pieces of information in the log stream when there is a failure:

What task failed
The causal error message(s)
Where the test/error reports are
The command that was run

(Honestly this is sometimes so long that some of this is lost out of the CLI buffer.)
Expected Behavior
A structured error report at the end of build, even if a limited one, would be a huge experience improvement. Here are some examples:
Test failure
Errors (1)

> Task :foo:integTest [org.gradle.api.tasks.Test] FAILED
Description:   There were failing tests.
Report:           file:///Users/you/src/yourproject/build/reports/integTest/index.html
Command:     ./gradlew :foo:integTest
Build Scan:     https://scans.gradle.com/s/somekey

Here's what is better than the current behavior:

This is much more easily parseable by humans and computers
It is grouped at the end, so that --continue doesn't make the output very scattered
It contains the command to re-run. This is optional, but would be cool
It contains the type of task that failed
The build scan URL is included as a peer to the test report, as it should be.

Current Behavior
As of Gradle 5.2, we write:
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':test'.
> There were failing tests. See the report at: file:///Users/eric/src/eriwen/gradle-digest-plugin/build/reports/tests/test/index.html

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 6s
13 actionable tasks: 1 executed, 12 up-to-date

Publishing build scan...
https://scans.gradle.com/s/occff3ygnj5bs

Context
I consider this my greatest failure so far in Gradle user experience. This is an issue that affects everyone who uses the build tool, including possibly those that use IDEs.  Perhaps a structured log output would be helpful. Regardless, doing this for CLI and continuous integration is enough.
This could be implemented within the logging chain or perhaps as something that subscribes log events more on the outside and maps certain messages to each failure.

Thanks, Eric!
Ideally, I'd like to see Gradle formalize errors in the API, collecting an Error from each failed task - delegating to the tasks to describe their own errors, and falling back to a default error definition. For example, the unit test or checkstyle tasks would register their own instances of Error, describing each failed test or checkstyle violation, and including links to the relevant reports.
It would be ideal if, as part of this effort, Gradle made it easier to inspect the task output (either standard or error) to collect extra information about the error, as described in https://github.com/gradle/gradle/issues/6068.
Build scans could use the information collected through errors to offer the extra detail about build failures, instead of just presenting a callstack and error output (and requiring several clicks). On the client side, error handling can be delegated to plugins, which may dump a description of the errors to the console (as described above), or present to the user through other means (like sending to a CR system, which we've been doing for a few months to remove the need for users to interact with Jenkins entirely).

In the summary gradle report *Where title missed why? Only this. How configure this output ?
gradlew test -Ptarget=local cleanTest test 1>all.log 2>error.log
2 tests completed, 2 failed
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':testSpravochniki'.

There were failing tests. See the report at:

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
Get more help at https://help.gradle.org
BUILD FAILED in 47s

In our project we have "merge" branches where we are merging diverging code of different creators and we get a lot of errors in those branches. And we apply computer processing of compilation errors: parse them, find broken code lines, blame the owners of that code, get urls to the source code with those lines, put all this information into nice html with a table which is sortable and filterable (by owner, by error type, etc.).
Also, we use --continue flag to get as much as possible errors at once (and our project enables org.gradle.parallel flag).
Currently, Gradle is not very friendly to our use case.
Gradle allows to get tasks list that have failed (using org.gradle.api.execution.TaskExecutionListener), and exception stack trace of Gradle itself (using org.gradle.BuildAdapter) which is nice but not enouph for our happyness.
We need Gradle to give us error output of every failed task separately (using task.logging.addStandardErrorListener does not work : https://github.com/gradle/gradle/issues/6068) or may be some kind of api that @cesar1000 proposed or may be some json report where we could read it. It is not comfortable to parse stderr with all of those different failed tasks output with either many parsers either one dreadful parser.


Consumer group

Consumers can't be in the same group and subscribed to different topics.
Updating translator logic.

Codecov Report

Merging #24 (906f3ca) into main (7d8b87c) will increase coverage by 0.22%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##             main      #24      +/-   ##
==========================================
+ Coverage   73.19%   73.42%   +0.22%
==========================================
  Files          36       36
  Lines         347      350       +3
  Branches       17       17
==========================================
+ Hits          254      257       +3
  Misses         93       93

Impacted Files
Coverage Δ

src/infrastructure/stream/consumer.ts
0.00% <0.00%> (ø)

src/app/translators/transaction-translator.ts
100.00% <100.00%> (ø)

...rastructure/stream/translators/block-translator.ts
100.00% <100.00%> (ø)

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 7d8b87c...906f3ca. Read the comment docs.


Partials decided layout

If i define layout_1.hbs and layout_2.hbs then in partials.hbs i want ot controller use layout_1.hbs or layout_2.hbs. How to achieve？

like http://handlebarsjs.com/partials.html#partial-block {{> @partial-block }}

or Dynamic Partials?

+1

+1

+1

+1

No supported :(


Ensure wildcard expansion occurs within shell, rather than within CLI

As an IMPROVER developer, I would like the CLI acceptance tests for the improver-combine CLI to use shell expansion of the wildcards, rather than expand the wildcards within the CLI, because there is no need for the CLI to do this operation.
Related issues: #389, #400
An example of this exists in improver-combine e.g.
input_filenames = []
for filename_globlist in args.input_filenames:
for filename in glob(filename_globlist):
input_filenames.append(filename)

This handling can be removed and quotes can be removed from the wildcarded paths within the CLI acceptance tests, so that shell expansion is used.
Acceptance criteria:

Modify improver-combine CLI to remove expansion of wildcarded paths.
Update or add acceptance tests for this functionality.

Completed.


fix #123 - No autoforward after initiated update

fix #123

@technyon I think this will solve the issue, but it will be hard for me to test if you can do that it will be awsome

Thank you!


Can't build because of APP_CODE_CONTAINER_FLAG=:cached

Info:

Docker version ($ docker --version): Docker version 18.09.2, build 6247962
Laradock commit ($ git rev-parse HEAD): 8ca26e6c06c5841c8ca2c328ea5a0220209a3d71
System info (Mac, PC, Linux): macOS
System info disto/version: 10.13.6 (17G5019)

Issue:
Can't run "docker-compose build", it says:
ERROR: Volume /Users/my-name/Sites/my-project:/var/www:cached:cached has incorrect format, should be external:internal[:mode]

Expected behavior:
Building stuff

Reproduce:
git clone laradock, copy env-example to .env, $ docker-compose build

Relevant Code:
// place a code sample here

Had to empty the APP_CODE_CONTAINER_FLAG in .env to make it work (instead of :cached)

yes after updating the repo i tried to build and i got the same error

What is your APP_CODE_PATH_CONTAINER value ? It should be APP_CODE_PATH_CONTAINER=/var/www


After training, How I do I generate inferences with specific beam width?

I trained with beam_width=0 (which is Greedy Decoding). When I try to generate inferences using:
python -m nmt.nmt \
    --out_dir=/tmp/nmt_model \
    --inference_input_file=/tmp/my_infer_file.vi \
    --inference_output_file=/tmp/nmt_model/output_infer

It obviously loads the saved hyperparameters and decodes using beam_width=0. However, I would like to generate inferences from my trained model with (say) beam_width=10. I tried with setting --override_loaded_hparams=True, but it ends up in the following error:
NotFoundError (see above for traceback): Key dynamic_seq2seq/decoder/multi_rnn_cell/cell_0/basic_lstm_cell/bias not found in checkpoint
	 [[Node: save/RestoreV2_1 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2_1/tensor_names, save/RestoreV2_1/shape_and_slices)]]
	 [[Node: save/RestoreV2_10/_19 = _Recv[client_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device_incarnation=1, tensor_name="edge_56_save/RestoreV2_10", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:GPU:0"]()]]

I would like to know a way out.

@ssokhey
--override_loaded_hparams=True also overrides other configurations such as num_units, num_layers etc. You will need to use your original training command plus the inference arguments in order to override loaded hparams but also keep your trained model's configuration.
For example:
python -m nmt.nmt \
    ...\ # (all your training flags)
    --inference_input_file=/tmp/my_infer_file.vi \
    --inference_output_file=/tmp/nmt_model/output_infer \
    --beam_width=10 \
    --override_loaded_hparams=True

@oahziur Thanks for the prompt reply. It is working. I have just generated instances with your suggestions.
I just wanted to ask if there is any other cleaner way to do the same thing.

@ssokhey
You could try to use one of the standard_hparams, or define something similar.
You can find an example usage here.

@oahziur
These I have already seen. Anyways thank you very much for your time. Really appreciate that.


fix: reduced fields requested

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Correct popup.html

Subject of the issue
popup.html due to some changes, shows the same content twice. This needs to be corrected.

can i do this work

please assign this work to me sir

Go ahead

This issue was occurring because the HTML code was copied twice in the "popup.html". Can be solved just by deleting the duplicate code. @Durgesh4993 if you're not working on this, let me know, I'll raise the PR.


add Integration Testing chapter for the Contoso University example code

The "Getting started with ASP.NET Core and Entity Framework Core using Visual Studio" series is a fantastic tutorial. First of all, thank you for creating this!
I've created this issue to request an additional chapter on "regression testing" the whole application, since I strongly believe this is a very important part of application development.
I found the "regression testing" article by Steve Smith not complete enough to be able to integrate this back into the "Contoso University" application.
Unfortunately I lack the experience with aspnet to write this chapter of the tutorial myself.
Source of the referred "ef-mvc" demo app: /aspnet/Docs/tree/master/aspnetcore/data/ef-mvc
Thank you.

@johanbove  Thanks for the suggestion. Hopefully we can find someone to write the topic or at least write the code. I've add the "up-for-grabs" label.

If Integration tests in ASP.NET Core is missing information, it should be added there.
@daveabrock 	can you outline what you'd add to Integration tests in ASP.NET Core ?

I was thinking about adding a chapter to this demo, with code pertaining to the demo code. I can reference back the regression testing document, and if while writing demo code I notice missing content I can add it there. Does that make sense, @Rick-Anderson?

@daveabrock 	I sure that doc would be welcomed by many developers, but I'm not convinced it's a good long term fit for our docs. I'd be more open to adding a section to  Integration tests in ASP.NET Core and pointing to it from the RP/EF series.
@danroth27 	please review? Do we want a testing doc tightly coupled to the EF/RP tutorial and outside the testing TOC. Perhaps that would be better as a blog rather than maintained docs.
cc @tdykstra 	@scottaddie 	@guardrex

... and note that the Integration tests topic has undergone a major overhaul since this issue was opened (May 29; https://github.com/aspnet/Docs/pull/6086) and uses the new WebApplicationFactory approach for testing. The sample app covers the basic scenarios, and there are additional issues that might be addressed in future work on the topic ...

How to create authenticated request? #6882
How to assert against ApplicationDbContext #8713
How to actually perform post requests? #7526

All makes sense. I'll keep an eye on this. Until then, I see several open GitHub issues throughout the tutorial and maybe can take a look at some of those.

@daveabrock If you find a gap not covered by those existing issues for the Integration tests topic, open a new issue from the "Content feedback" button at the bottom of https://docs.microsoft.com/aspnet/core/test/integration-tests and we'll take a look.

I see several open GitHub issues throughout the tutorial and maybe can take a look at some of those.

That would be great, thanks.

Great @Rick-Anderson. I'll add an "on it!" comment on whatever I'm working on so you guys are aware.

@Rick-Anderson I was just sizing up the Integration Testing issues (and putting them into a project so that I don't lose track of them), and it looks like this one isn't actionable for the docs.
The ask is to add a topic on testing to a couple of the existing multi-topic tutorials (EF w/RP and/or EF w/MVC). That's not something that you wanted earlier.
There's nothing actionable here for the Integration Testing topic itself.
Therefore, this issue can be closed AFAICT.


[Support] MakeStyles crashes with "Can't assign to property" error

I've recently switched from @mui/styles to react-tss and I'm experiencing an error:
Uncaught TypeError: can't assign to property "label" on "ltr": not an object
Tracking the error down I'm lead to the makeStyles.js file, line ~45
  const cssObject = cssObjectByRuleName[ruleName];
  if (!cssObject.label) {
      cssObject.label = `${name !== undefined ? `${name}-` : ""}${ruleName}`;
  }

In this error case, the ruleName is direction resulting in a cssObject of the string ltr. Attempting to write to cssObject.label causes the error above.
I've checked my codebase and I'm not modifying the direction anywhere, so I don't really understand why this is happening and could really use some assistance.
Thanks

Maybe rename your .js files into .tsx files just to let TypeScript tell you what's wrong.

Thats not the easiest solution in such a large codebase. The thing that bugs me is I don't set direction anywhere so it must be coming from the base theme

Not everywhere, just on some files, momentarily to see if typescript can detect an error.
Do you use the theme <ThemeProvider /> as described in the doc?  https://docs.tss-react.dev/readme-1
If I could see the code I would probably be able to spot the error quickly

Just in cas, It may help, https://docs.tss-react.dev/page-1/withstyles

I tried to make a sample to demonstrate the issue I'm having, but it doesn't have the same error.
https://codesandbox.io/s/bold-bird-v2t5pc?file=/src/Component.jsx

I think I figured it out, I had a botched attempt at replicating the withTheme behavior from older MUI. Any tips on how to implement this?

Yup that worked! Thanks, and sorry for the bother

No problem, happy to help


made various fixes to make the percentage function work

updated the function to call from estimate summary it works with no changes to the kpi python
fixed test case to compare the pre-defined test case kpis called from the function with the logic we believe to be correct
small fixes here and tehre

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

I have applied the fixes you recommended. Please give one final review before closing the PR


[HUDI-3603] Support read DateType for both hive2 and hive3

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@danny0405  @nsivabalan  could you pls help me review this pr thanks


Add xbox controller drivers

xone (xbox one wired/rf usb) driver
xpadneo (xbox one bluetooth) driver

Completing this should close  https://github.com/ublue-os/main/issues/46
Related to https://github.com/ublue-os/main/issues/59 as my planned solution includes using xbox driver akmod spec files from https://negativo17.org/


SOLR-15446: Fix three off-by-one ArrayList initialCapacity choices when calling ZooKeeper.multi API.

https://issues.apache.org/jira/browse/SOLR-15446 created but no solr/CHANGES.txt entry planned for this small change.

Not directly related to the PR, but is there a reason for using ZooKeeper.multi when only issuing 1 command?

Not directly related to the PR, but is there a reason for using ZooKeeper.multi when only issuing 1 command?

Interesting question! From a quick look at the ZooKeeper docs I could see no reasons for preferring multi over the single command equivalent.

Another idea might be List.of() which would avoid having to even keep track of the initial capacity at all?

FWIW PR #1232 addressed a part of this. I'll resolve the conflict and use List.of since that makes it so we don't even need to deal w/ sizes.


database migrations

all migrations written to bring the database in line with our planned schema (as on Whimsical)

 As IT, I would like to create a data schema table so that my data from the work orders are organized

Should we add imageurl in the work order table too, I can see you have it in place for the properties table? On the General work order form I am working on it does have a section to add an attachment right from there.

Should we add imageurl in the work order table too, I can see you have it in place for the properties table? On the General work order form I am working on it does have a section to add an attachment right from there.

When we work out how that info is being sent to the back-end, I'll make sure the model pushes attachments into the "image" table. That way, a workOrder can have many images.


Add support for attention softmax in BF16 such as for llama.

What does this PR do?

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@regisss - I hope I have addressed all the nice review comments. Please check now.

@puneeshkhanna I am wondering why we use "--attn_softmax_bf16" arg to control if to run attn_softmax in bf16 or not? why not run attn_softmax_bf16 directly if the model is in bf16/hmp? why we let user control from cmd arg? Thanks

@puneeshkhanna I am wondering why we use "--attn_softmax_bf16" arg to control if to run attn_softmax in bf16 or not? why not run attn_softmax_bf16 directly if the model is in bf16/hmp? why we let user control from cmd arg? Thanks

@mandy-li  - Model code by default has explicit upcast to float32 (torch.nn.functional.softmax(attn_weights, dim=-1, dtype=torch.float32)) so even though model is run in bf16, softmax will always run in FP32. Usually that is the case with softmax in attention block in most of the models because of exponent operations.
Hence we need a user control for it and if desired accuracy and perf can be achieved via softmax in BF16 only then this option can be enabled.


how support divide to insecure site in shenyu

Question
I requested a site based on divide plugin,but the ssl cert of this site is insecure,my service does not work.Who can give me a suggestion? Or how support divide to insecure site in shenyu? Thanks a lot.

Can you paste the specific error message or log? Divide plugin does not require the backend service to enable SSL.


[ISSUE]How to download a browser that supports "all-codecs" and adapt it to the current version

The default downloaded browsers do not have "all-codecs"？

[ ] I understand

You can leverage Google Chrome or Microsoft Edge for that. Only these browsers contain the codecs: https://playwright.dev/docs/browsers#media-codecs

Closing per the response above, feel free to open a new issue if it doesn't work.


GPU option with tensorflow

As a team that is using Planetary Computer but relies on Tensorflow for our ML modeling work, it would be great to have a GPU option with Tensorflow (v 2.4.0 or above) so we can easily access PC data and develop models in one place. I tried installing tensorflow into the current GPU option but it doesn't end up working because the CUDA development and runtime libraries aren't installed. On a related note, it would be great to have Tensorflow in the python CPU option as well.
Thanks!

Thanks for raising an issue. I'll see if I can get an image with TF + Keras + jax, in addition to the usual libraries. Anything else?

On a related note, it would be great to have Tensorflow in the python CPU option as well.

Yeah, I'm trying to keep the image sizes of the base Python image somewhat reasonable. I haven't looked at what just Tensorflow (not cuda) does to the size of the image, but I'm imagining it'll increase their size significantly.

See https://github.com/microsoft/planetary-computer-containers/pull/7
If you could let me know whether you have any other libraries that you'd consider "core" when doing geospatial ML with tensorfow, I'll see if they can be added as well.

This is awesome to hear! I know we end up installing rio-tiler a lot so as of now that would be our other request to this image (and the CPU one) but I respect you want to keep these images light. Tensorflow+GPU is definitely our number one priority.

Yeah, I'm trying to keep the image sizes of the base Python image somewhat reasonable. I haven't looked at what just Tensorflow (not cuda) does to the size of the image, but I'm imagining it'll increase their size significantly.

This makes sense. I think installing tensorflow-cpu ourselves won't be a problem.

And you already opened a PR! This is amazing and will make our lives a lot easier.

Great, I'll add rio-tiler to the images. I'll get that added and merged and we'll see if any issues come up during the build.

Wow! Amazing! Looks like the build worked. So I should Look out for it under the "Choose your environment" option when you login to Hub?

Yeah, that's being added at https://github.com/microsoft/planetary-computer-hub/pull/11 (to our staging deployment). If things look OK with some manual testing I'll deploy to production and it should show up for you.

I'm running into some issues with that in staging unfortunately :( I'll take a look tomorrow or Monday morning though!

Understandable :-) Thanks again for jumping on this so quick!

@TomAugspurger , I saw the new option in the Hub this morning. Wahoo! I do get an authorization error related to pulling the image when I try to launch it:

Yeah I noticed that too 😬. Fixed in https://github.com/microsoft/planetary-computer-hub/pull/14/files#diff-1c45f90b87a9db6b9e68d9a5525d0a664f3c0120562b55f11617a4162fdbf692L27
An unrelated issue is making this difficult to test on the staging cluster, so I'm having to test it in production. Hopefully the current update fixes it.

@zstatmanweil do you want to give it another shot now?
>>> import tensorflow as tf
>>> tf.test.gpu_device_name()

prints out a bunch of stuff, but returns '/device:GPU:0' which I think means the GPU is active. Could you let me know if this covers your needs?

It appears to be working! I was able to launch the environment, and check the GPU is active and all that returned info looks how I would expect. Thank you again. If we have any troubles once we start doing substantial work with it, I will open another issue.


Add the otherLanguages field into the generated schema

Motivation
The otherLanguages field generated for item's variant is not part of the schema definition generated by createTypeNodesSchema() from typeNodesSchema.js.
Proposed solution
The goal of this issue is to add the mentioned field into the generated schema, write test(s), and update documentation.

Version 6 does not contain otherLanguages for every node - primarily because of the performance impact.
It is possible to extend schema like it is showcased on example:

https://github.com/Kentico/kontent-gatsby-packages/tree/master/site#language-variant-relationships


256 equibind doesnt work on mol2 files - now it should

Fixed the issue - the problem was the pattern matching for the output files also depended on _*sdf, it is now extended.
Test with
./plex -tool tools/equibind.json -input-dir testdata/binding/pdbbind_processed_size2 -concurrency=3


Greek translation iex_helpers + debugging

@gemantzu do you want to give @kosmas a chance to look this over?

Sure, I keep forgetting about this.

It should be ok now.

Thank you @gemantzu and @Kosmas 💜


image-reflector-controller cannot pull image list

We upgraded from 0.19.2 to 0.19.3 and our Image Policies are getting spammed with:
image-reflector-controller  Cannot determine latest tag for policy: version list argument cannot be empty

while Image Repositories show:
image-reflector-controller  GCR authentication failed: registry provider not configured for login

We see this happening on images like k8s.gcr.io/sig-storage/snapshot-validation-webhook which do not require auth to be pulled.
Image Repo and Policy:
---
apiVersion: image.toolkit.fluxcd.io/v1beta1
kind: ImageRepository
metadata:
  name: snapshot-validation-webhook
spec:
  image: k8s.gcr.io/sig-storage/snapshot-validation-webhook
  interval: 24h
---
apiVersion: image.toolkit.fluxcd.io/v1beta1
kind: ImagePolicy
metadata:
  name: snapshot-validation-webhook
spec:
  imageRepositoryRef:
    name: snapshot-validation-webhook
  policy:
    semver:
      range: ">=5.0.0"

Hi, thanks for reporting the issue. We did some refactoring of the registry code in the last release and looks like we didn't had a coverage for this scenario. I've created #290 to fix this issue and also added a test for it so that we don't have this bug again in the future.

Image reflector controller v0.19.4 is available now https://github.com/fluxcd/image-reflector-controller/pkgs/container/image-reflector-controller/29796368?tag=v0.19.4 . You can give it a try.

Works, thanks!


StateEncoder object has no attribute 'func_list'

Hi all, Thanks for looking at my question. I'm faced with a problem when using Qiskit with AmplitudeEncoder( ideal with StateEncoder).
When I set the use_qiskit parameter False, it runs well with my own GPU. However, when I run my code on Qiskit, I get the error message "'StateEncoder' object has no attribute 'func_list'".
To solve the problem, I check the "encoding.py", and find that  Amplitude Encoder doesn't have func_list differently from GeneralEncdoer.
Thanks!

Hi 101kyul,
Thank you for your question! For the AmplitudeEncoder, we need to use the Initialize function in the Qiskit. https://qiskit.org/documentation/stubs/qiskit.extensions.Initialize.html


Issue Downloading Since Latest Udemy Update

I can't download courses since the implementation of new interface...
I used to download using -k Cookies method
But it no longer works

@Labeyoo it works fine for me i cannot reproduce. take a look at screen shot below

I could reproduce an error with this stacktrace on Windows 10 with cookies extraced via Chrome (Copy as cURL(cmd)):
Traceback (most recent call last):
  File ".\udemy-dl.py", line 1432, in <module>
    main()
  File ".\udemy-dl.py", line 1091, in main
    udemy.course_download(path=options.output, quality=options.quality, unsafe=options.unsafe)
  File ".\udemy-dl.py", line 425, in course_download
    course = udemy.course(url=self.url, username=self.username, password=self.password, cookies=self.cookies)
  File "C:\Users\chess\git\udemy-dl\udemy\_udemy.py", line 38, in course
    return Udemy(url, username, password, cookies, basic, callback)
  File "C:\Users\chess\git\udemy-dl\udemy\_internal.py", line 44, in __init__
    super(InternUdemyCourse, self).__init__(*args, **kwargs)
  File "C:\Users\chess\git\udemy-dl\udemy\_shared.py", line 63, in __init__
    self._fetch_course()
  File "C:\Users\chess\git\udemy-dl\udemy\_internal.py", line 52, in _fetch_course
    auth = self._login(cookies=self._cookies)
  File "C:\Users\chess\git\udemy-dl\udemy\_extract.py", line 107, in _login
    self._session._session.cookies.update(self._cookies)
AttributeError: 'NoneType' object has no attribute '_session'

I tried manually replacing the access token value in line 106 (self._session = auth.authenticate(access_token=access_token, client_id=client_id)) and it worked, so I guess there's some problem with the parser.

How to login with cookie for Chrome Users with only 4 steps:

Make sure you are already logged in, right click anywhere on the Udemy webpage and click "Inspect" or use shorcut button "F12"

Choose "Application" tab on the top, then choose "Cookies" on the left, what we need here are values of access_token and client_id. You can type into Filter to find exactly what you need. Double click and copy those values.

Create a ".txt" file inside udemy-dl folder, name it whatever you want

Edit the file you just created, copy and paste this line, change yourAccessToken and yourClientId with values from step 2. Save the file.

Cookie: access_token=yourAccessToken client_id=yourClientId

Use the code below to download with cookie, replace courseURL with link of your course, fileTXT with the name of your txt file from step 3

python udemy-dl.py courseURL -k fileTXT


Video download doesn't work for every Video (v.2.0.1)

Hey, idk if this problem is already known, but video download like this (https://www.youtube.com/watch?v=5-xhpcgBMe4&start_radio=1&list=RDMM5-xhpcgBMe4)
doesn't work at the moment.
encryption link calls "Access denied"
Thank You :)

Version 2.0.2 that I recently published should fix it.
Please reopen the issue if it doesn't.
Sorry for taking so long to get to it.

Hey, thank You very very much. It works again, but there is one little problem, the maximum Videoquality is 360p and audio only 96 kbit/s. Maybe you can tell why it is or fix it.
Thanks a lot :)

@MusMuss1 see #108


added ListModuleCommandTest

Pull Request Test Coverage Report for Build 291

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 54.933%

Totals

Change from base Build 290:
0.5%

Covered Lines:
1676

Relevant Lines:
3051

💛  - Coveralls


Display help menus more promptly

Addresses #57 (speed up help menu displays by deferring large imports until called for to execute a command) and #58 (added new help menus to README.md)
With a freshly set up environment before changes:
$ time bioclip -h
usage: bioclip [-h] {predict,embed,list-models} ...

BioCLIP command line interface

options:
  -h, --help            show this help message and exit

commands:
  {predict,embed,list-models}
    predict             Use BioCLIP to generate predictions for image files.
    embed               Use BioCLIP to generate embeddings for image files.
    list-models         List available models and pretrained model checkpoints.

real    0m18.306s
user    0m0.000s
sys     0m0.062s

And subsequently
$ time bioclip -h
usage: bioclip [-h] {predict,embed,list-models} ...

BioCLIP command line interface

options:
  -h, --help            show this help message and exit

commands:
  {predict,embed,list-models}
    predict             Use BioCLIP to generate predictions for image files.
    embed               Use BioCLIP to generate embeddings for image files.
    list-models         List available models and pretrained model checkpoints.

real    0m4.203s
user    0m0.000s
sys     0m0.047s

After changes to lazy loading with a fresh environment:
$ time bioclip -h
usage: bioclip [-h] {predict,embed,list-models} ...

BioCLIP command line interface

options:
  -h, --help            show this help message and exit

commands:
  {predict,embed,list-models}
    predict             Use BioCLIP to generate predictions for image files.
    embed               Use BioCLIP to generate embeddings for image files.
    list-models         List available models and pretrained model checkpoints.

real    0m2.612s
user    0m0.000s
sys     0m0.031s

And subsequently:
$ time bioclip -h
usage: bioclip [-h] {predict,embed,list-models} ...

BioCLIP command line interface

options:
  -h, --help            show this help message and exit

commands:
  {predict,embed,list-models}
    predict             Use BioCLIP to generate predictions for image files.
    embed               Use BioCLIP to generate embeddings for image files.
    list-models         List available models and pretrained model checkpoints.

real    0m0.604s
user    0m0.000s
sys     0m0.046s

Similarly for sub-command menus.

Actually it seems like you believe the tests to belong together with the lazy loading. That seems fine to me, but is another piece of evidence for pulling the change set apart rather than commingling them.


Fix examples in README to pass nil for HandlerOptions

The examples in the README don't compile with the latest slog package because they now require you to pass in either nil or a HandlerOptions ptr to configure the handler. Fixed them.

@samber mind taking a look here?

Thanks for this contrib.


Investigate compile time

Compile time continues to get longer as we add more functionality.  CUDA is
really slow at
compiling template functions with multiple parameters, and we use a lot. There
are something like
384 different scan kernels, for example, and a similar number for segscan.

How can we reduce this code explosion?  Can we give feedback to the CUDA
compiler team? (Emu
mode compiles WAY faster for example).

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 25 Jun 2009 at 12:17

Original comment by<EMAIL_ADDRESS>on 6 Jul 2011 at 2:36

Added labels: Priority-High, Milestone-Release2.1
Removed labels: Priority-Low


[FIX] Some things are not indexed within the update_index command.

I have investigated why some random images do not appear as search results in our backend and tracked it down to this line.
The reason is that there is no default order on the database queryset and since it get's retriggered on each index-chunk, the results are somewhat random and do skip some images (or documents).
Adding a order_by clause fixes that.

... regarding the failing build, I guess that's not my fault? I haven't touched the failing scss file.

Thanks @shredding!
The Drone server was broken for a while, and it's quite probable that some style guide violations sneaked into the codebase while they weren't being tested - I'll get that sorted.

Merged in 9257a4a1dc556b2c9b333f26f36ea41229dff755 - thanks!


style: remove warning about useEffect

This warning was appended to every PR. While the warningn typically makes sense, in this case we intentionally don't want to rerun if desiredState changes.

Thanks for the cleanup!


CLI: node ls output to be more compact

Current implementation wraps easily to the next line because there are lots of whitespace between node name and the next field :
$ kontena node ls
Name                                                                   Status     Initial    Labels
great-node-2                                                      online     yes        region=-eu-west-1,az=eu-west-1c,instance=m3.medium
great-node-1                                                      online     yes        region=-eu-west-1,az=eu-west-1b,instance=m3.medium
great-node-0                                                      online     yes        region=-eu-west-1,az=eu-west-1a,instance=m3.medium

Suggested (field / name length + 3 spaces):
$ kontena node ls
Name           Status   Initial   Labels
great-node-2   online   yes       region=eu-west-1,az=eu-west-1c,instance=m3.medium
great-node-1   online   yes       region=eu-west-1,az=eu-west-1b,instance=m3.medium
great-node-0   online   yes       region=eu-west-1,az=eu-west-1a,instance=m3.medium

this is now good


Finalize system and priority new content

Problem statement
We need updated messaging resources and a more usable system for different stakeholders and teammates to stay aligned and leverage these resources.
What you need to know

planning doc from sprint 100

Acceptance criteria

[x] Structure finalized that contains priority content
[ ] Feedback and input from stakeholders collected and applied as necessary

Chris has reviewed and is ready for publish.

https://cdc.sharepoint.com/teams/ReportStream/SitePages/Brand,-content,-and-communications.aspx


Document patterns with examples

Fixes #12
Documentation on what are patterns, what they are composed of and pattern examples and their behavior.

👍 other than Blade's comments.

I've kicked off the build again. Assuming it passes, 👍 on merging this in.

No output has been received in the last 10 minutes, this potentially indicates a stalled build or something wrong with the build itself.

Seems like our builds are timing out...

Merged in the build fixes. If the build passes this time, we should merge this in.


Unexpected keyword argument:  gpus

Error:  init() got an unexpected keyword argument 'gpus'
TypeError in marathon.models.base in from_json
This error starting occurring after upgrading from DCOS 1.7 to 1.8, which comes packaged with Marathon 1.3.

But they haven't even released it yet publicly?
https://github.com/mesosphere/marathon/releases


README

README
User Story
As a boot camp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
Acceptance Criteria
GIVEN the prework project repository,
WHEN I view the README for the repository,
THEN I can see a professional README that provides information on the project and how to use it

Did it

Whoops didn't actually do it yet

Ok done


Feature request: add SQS support

Story
We wanna add an SQS support and create as many SQS resources as we places into datastores map
Location: sqs.tf
Input
  datastores = {
   # ... #
    sqs = {
      name    = "main"
      sse     = true
      timeout = 900
    }
    sqs = {
      name    = "sqs_reserved"
      sse     = true
      timeout = 900
    }
  }

Important
Please refactor Additional Env Vars in services.tf to support this feature.
Means, for every SQS created we need to place an environment variable here
locals {
  added_env = [
    {
      name  = "QUEUE_URL"
      value = aws_sqs_queue.main.url
    },
    {
      name  = "QUEUE_URL_REVERSED"
      value = aws_sqs_queue.reversed.url
    },
   # ... #
  ]
}

Advice
Please advice if we need to use type in sqs input:
    sqs = {
      name    = "main"
     type = "sqs" # ??? #
      sse     = true
      timeout = 900
    }
    sqs = {
      name    = "sqs_reserved"
     type = "sqs" # ??? #
      sse     = true
      timeout = 900
    }


make bsp build on TestModule success instead of fail

IJ with BSP will make bsp call BuildTargetCompile with all BuildTarget, additional test module (introduced in #3064) will cause the bsp call fail as it falls into third case in the pattern match. This PR makes it does nothing and always success.

reproducer mill project: https://github.com/chikei/mill-bsp-test-only-module
jetbrains issue: https://youtrack.jetbrains.com/issue/SCL-23352


GEODE-6423 availability checks sometimes immediately initiate removal

Do not loop in trying to form a tcp/ip connection to a suspect unless
the next step is to remove the suspect from membership.  In this case
there will be another invocation of the same method that will take the
removal step next.
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

The dunit failure doesn't reproduce.  I ran the test 500 times with no problems.


SPR-4265 use serverless webpack compile results as component deployment artifacts

What did you implement:
Closes #XXXXX

How did you implement it:

How can we verify it:

Todos:

[ ] Write tests
[ ] Write documentation
[ ] Fix linting errors
[ ] Make sure code coverage hasn't dropped
[ ] Provide verification config / commands / resources
[ ] Enable "Allow edits from maintainers" for this PR
[ ] Update the messages below

Is this ready for review?: NO
Is it a breaking change?: NO

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 15 (40.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 92.832%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/validate.js
2
11
18.18%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
2459

Relevant Lines:
2615

💛  - Coveralls


Manager override on confirmation page

Pivotal ticket
Allows users to override an application that has failed via the benefit route.
Still to do

[x] Refactor the logic used on the display, possibly into the decision_override view model
[x] Update the processed applications page to display the new✓ Passed (by manager's decision) message
[x] Increase the number of reason radio buttons (see content task)
[x] Review the content

I'm missing new tests in the process_controller_spec.rb, or am I blind?

Generally speaking, the code looks good! I'm not sure if the ProcessController is the right place, etc. But we've talked about it quite a lot and as it's not bad, let's do it this way..

👍  looks good now!


syncer(dm): remove placement rule table option for tracker

Signed-off-by: lance6716<EMAIL_ADDRESS>
What problem does this PR solve?

Issue Number: close #7493
What is changed and how it works?
as title
Check List
Tests

Unit test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Fix sometime DM task is stopped with error "Unknown placement policy"

this is the fix PR for release-6.1 like https://github.com/pingcap/tiflow/pull/7495 , PTAL
@okJiang @GMHDBJD

/cc @GMHDBJD @lichunzhu

/merge

/merge


[Crazy] Incorporate Simulated Data into PyCMDS Virtualization

Use the same interface to generate simulated data as real data
This is (most likely) dependent on issue  #80 (Hardware Virtualization)
This would also resolve some of the issues with how to generate data for input devices.

Closing as this would be accomplished via yaq, if we ever wanted it at all (actually wouldn't be to hard to do...)


Add support for ktlint fix with -F

This is a follow-up to ktlint support with an action to patch/autocorrect violations where supported in rules using https://pinterest.github.io/ktlint/1.0.1/install/cli/#format-autocorrect and leveraging existing semantics with rules_lint

Changes are visible to end-users: yes

Searched for relevant documentation and updated as needed: yes

Test plan

Covered by existing test cases
Also verified with ./lint.sh --fix --dry-run that hello.ktwould be updated

From bazel-out/k8-fastbuild/bin/src/ktlint.hello_kt.aspect_rules_lint.patch:
--- a/src/hello.kt
+++ b/src/hello.kt
@@ -1,6 +1,6 @@
-import javax.security.*
 import java.util.*
+import javax.security.*

 fun main() {
-  println("Hello, world!")
+    println("Hello, world!")
 }

ktlint acts both as a linter and a formatter. This --format flight is slightly confusing as it goes beyond pure pretty-printing/formatting. For example, custom rules can use this https://github.com/pinterest/ktlint/blob/9b38b37d95fcb0ad90d07d64d581e6059ea8496e/ktlint-api-consumer/src/main/kotlin/com/example/ktlint/api/consumer/rules/NoVarRule.kt#L15 with -F mapped to autoCorrect to suggest custom fixes for the lint rules that'll show up in the diff, so these fixes can also be semantic suggestions/enforcements. That said, I haven't used ktfmt before so if you feel this is redundant, I can close the PR

I don't want to close the PR - we DO want to have auto-fixes for linters that understand them.
It's a pity that the Kotlin ecosystem got into this confused state - Javascript (eslint) and Python (flake8/ruff) were careful to have linting not fight with formatting. It seems like ktlint authors have freely conflated the two.
We could use ktlint for both purposes, but I don't see a flag that allows you to only format and not lint.
I think we should raise a discussion on the ktlint repo to see if there's any recommended solution (and not make it Bazel specific - we just want to follow what other users do)