Regrowth time addition to Time Tracker plugin

Currently, the time tracker will tell you the approximated time of when something you've planted will be fully grown. Once the plant has fully grown and been harvested, if the patch is not automatically cleared - aka it is not an herb or allotment patch - the time tracker will begin telling you how much the patch has regrown, such as the coconut tree above. As can be seen in the image, the regrowth is already tracked, but no approximate time is given for when it will be done regrowing.
Could a toggle feature be added to the time tracker to add regrowth time approximations?

so to be succinct, you would like a timer to show when the fruit is completely regrown and can be fully harvested again

Yes, that is correct. I tend to be awful at being succinct so thank you for the assist there @Alexsuperfly

I've now found that this is actually already done on the overview window, which makes it extra strange to me that the timer doesn't show on the specific patches.


⚠ Bofrost.de has degraded performance

In 3802915, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 972 ms

Resolved: Bofrost.de performance has improved in c710011 after 7 minutes.


Fix url parsing bug

Fixed the issue of Nil pointers in certain URL parsing, such as http://hostname/xxx?a=1&b

Thanks for the awesome bugfix!


INC-739: PATCH /sync/booking/{bookingId}/id/{id} endpoint

Adds endpoint to update an existing IEP review (PrisonerIepLevel)
As per ticket:

MAINTAIN_IEP role and write scope needed to perform this operation
only comment, iepTime and current fields can be changed
publishes relevant events:

"incentives.iep-review.updated" domain event (domain event types are now part of the IncentivesDomainEventType enum, e.g. this endpoint publishes IncentivesDomainEventType.IEP_REVIEW_UPDATED)
also publishes AuditType.IEP_REVIEW_UPDATED audit message
made existing code that publishes domain/audit events more explicit about what events are published

revamped Swagger docs so that:

example requests bodies are useful (courtesy of @Schema's implementation field)
example values are consistent across all the swagger examples, so that requests tend to match responses and they're less likely to confuse people
I guess this was technically part of INC-740 but I needed to have docs for this endpoint (...and I may have been carried away a little 😅)

tests to endpoint and also to the corresponding service

FYI: I've fixed the few linting problems hopefully

@Mjwillis FYI: This PR adds the sync PATCH endpoint.
Regarding domain/audit events:

I've added an enum for domain events published by this service IncentivesDomainEventType (new domain event type is "incentives.iep-review.updated", named consistently with the existing "incentives.iep-review.inserted" one)
I've also added the IEP_REVIEW_UPDATED type to the existing AuditType
nothing too exciting but hopefully it's a bit tidier using enums for both events types

CC: @neilmendum


can't find in the plugins

i gti clone the kinetic version and catkin_make it as the tutorial from wiki.ros,however i can't find launchtree in the plugins list

and I tried
rosrun rqt_launchtree rqt_launchtree
,but get
qt_gui_main() found no plugin matching "rqt_launchtree.launchtree_plugin.LaunchtreePlugin"
as result.I don''t know what to do .I really need to use this plugin to understand how the launch file works.Is there anyone who know how to use it?

I solve the problem just now.
run rm ~/.config/ros.org/rqt_gui.ini
then 'rqt',the plugin will be find


Ability to close the fd of an .asar we're using

My problem is:
On Windows, I use my own auto-update framework which is supposed to replace the .asar file of an app. This works fine for Linux, OS X, but it requires a hack for Windows with an external script doing the work, because Electron keeps the .asar file open at all times.
What I'm asking is a way to close any asar files programatically we've used in Electron, therefore allowing to move them/delete them on Windows.
Looking at https://github.com/atom/electron/blob/master/atom/common/lib/asar.coffee, we have a getFD function, so this shouldn't be too hard. It would be even possible to do it only by exposing cachedArchives.
The reason I'm not submitting a PR is that I don't know what's the cleanest way to expose an API like that.

@zcbenz how does this work?
https://github.com/atom/electron/blob/0d50e08ed106ae2897d6320966b47ac0357b8da1/atom/common/asar/archive.cc#L117-L132
Archive::Archive(const base::FilePath& path)
    : path_(path),
      file_(path_, base::File::FLAG_OPEN | base::File::FLAG_READ),
#if defined(OS_WIN)
      fd_(_open_osfhandle(
              reinterpret_cast<intptr_t>(file_.GetPlatformFile()), 0)),
#elif defined(OS_POSIX)
      fd_(file_.GetPlatformFile()),
#else
      fd_(-1),
#endif
      header_size_(0) {
}

Archive::~Archive() {
}

So MSDN says
To close a file opened with _open_osfhandle, call _close. The underlying handle is also closed by a call to _close, so it is not necessary to call the Win32 function CloseHandle on the original handle.

Do I just do this: ?
Archive::~Archive() {
#if defined(OS_WIN)
   _close(fd_)
#endif
}

@zcbenz how does this work?
https://github.com/atom/electron/blob/0d50e08ed106ae2897d6320966b47ac0357b8da1/atom/common/asar/archive.cc#L117-L132
Archive::Archive(const base::FilePath& path)
    : path_(path),
      file_(path_, base::File::FLAG_OPEN | base::File::FLAG_READ),
#if defined(OS_WIN)
      fd_(_open_osfhandle(
              reinterpret_cast<intptr_t>(file_.GetPlatformFile()), 0)),
#elif defined(OS_POSIX)
      fd_(file_.GetPlatformFile()),
#else
      fd_(-1),
#endif
      header_size_(0) {
}

Archive::~Archive() {
}

So MSDN says
To close a file opened with _open_osfhandle, call _close. The underlying handle is also closed by a call to _close, so it is not necessary to call the Win32 function CloseHandle on the original handle.

Do I just do this: ?
Archive::~Archive() {
#if defined(OS_WIN)
   _close(fd_)
#endif
}

Do I just do this: ?

Yeah.

I'm closing this since we don't have plan to work on it, feel free publishing a PR if anyone is interested.


Bugs on creating account okx

Not sure what that is, but it has nothing to do with this project


Hi Team,

Browsers and versions affected
Steps to reproduce
Expected results
Actual results

Hi Team
For some reason the tool can't access my audio and camera.  I tested fine yesterday and there have been no changes.  Any ideas?
Thanks

Can you specify your environment setup in this issue ? Thanks.

Sorry guys,
There are no issues.  The browser was blocking the access.  All is working.  Thanks for responding back.


from keras.backend import tf as ktf

from keras.backend import tf as ktf fails on keras.version = 2.0.8
therefore the interpolation layer gives an error.
Any solution?

I have met the same problem. My solution is to not activate tensorflow in the terminal, i.e., not use "source activate tensorflow". Just use "python pspnet.py -m pspnet101_cityscapes -i example_images/cityscapes.png -o example_results/cityscapes.jpg", it will work.

@cardwing what do you mean activate tf in the terminal? I guess it is to import tf in the python shell?
@aliericcantona  Can you please provide some logs or error stack

Apparently this issue is caused by the marshal serial format having changed a bit between Python 3.5 and 3.6 as discovered in https://github.com/keras-team/keras/issues/9595 . Converting the weights from numpy with python 3.6 should work around it.

I have met the same problem. I use python 3.5 and tensorflow-gpu 1.14.0.
I don't understand "not use tf in the terminal is just not execute "source activate tensorflow"" from @cardwing.
how can i fix it?

Just use "python pspnet.py -m pspnet101_cityscapes -i example_images/cityscapes.png -o example_results/cityscapes.jpg".

@cardwing Could you please be clearer? The README.md asks us to run the file just the way you have mentioned. What does don't activate tensorflow in terminal mean?

from keras.backend.tensorflow_backend import tf as ktf

im having the same issue. tensorflow-gpu 2.0.0 abd keras 2.4.3
from keras.backend.tensorflow_backend import tf as ktf doesn't work either


Standardize swift-ring finalizer name

The swift-ring finalizer is created by a curl API request and not using lib-common. This patch standardizes the finalizer name by adding the missing domain name.
Related: https://github.com/openstack-k8s-operators/lib-common/pull/519

/test swift-operator-build-deploy-kuttl


Activity Screen

I have completed the activity screen using hardcoded text, will make it dynamic as we progress

Best.


Endpoint deprecation warnings in .18.1 & Terraform AWS > 5.36 (or so)

terraform-local: .18.1
terraform: 1.74, 1.8.0
AWS provider:  > 5.36?  Wasn't present in < 5.35.
When running tflocal plan, or tflocal apply:
Warning: Invalid Attribute Combination
│
│   with provider["registry.terraform.io/hashicorp/aws"],
│   on providers.tf line 19, in provider "aws":
│   19: provider "aws" {
│
│ Only one of the following attributes should be set: "endpoints[0].configservice", "endpoints[0].config"
│
│ This will be an error in a future release.
│
│ (and one more similar warning elsewhere)

update terraform to v1.8.1 and this should go away.

Even with Terraform v1.8.1 the warning is shown:
$ terraform --version
Terraform v1.8.1
on linux_amd64
+ provider registry.terraform.io/hashicorp/aws v5.46.0

$ tflocal apply
╷
│ Warning: Invalid Attribute Combination
│
│   with provider["registry.terraform.io/hashicorp/aws"],
│   on localstack_providers_override.tf line 2, in provider "aws":
│    2: provider "aws" {
│
│ Only one of the following attributes should be set: "endpoints[0].elasticsearch", "endpoints[0].es",
│ "endpoints[0].elasticsearchservice"
│
│ This will be an error in a future release.
│
│ (and one more similar warning elsewhere)
╵

Hi @mbarr-hrp,
Thanks for reporting this issue. The root of this warning is the endpoint generation in the script which done dynamically without any checks for such condition currently, in the near future I'll provide a fix which addresses this reciprocal exclusiveness.


Grid helper puts lines in horizon

Just with the default arguments!

Can confirm this is an issue with the line material shaders.

Zoom in and get....

As you can see the box helper is also affected.

Thin lines are not affected.


identity: improve the documentation for Principal.Name()

Summary
Clarifies that Name must match the proof of possession.
@haydentherapper let me know if you can thing of a better name for the method. Happy to add that in here. Thought I'd at least improve the docs while I remembered to.

Codecov Report

Merging #579 (3b7c790) into main (8d307b5) will decrease coverage by 0.67%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #579      +/-   ##
==========================================
- Coverage   44.29%   43.62%   -0.68%
==========================================
  Files          21       21
  Lines        1517     1513       -4
==========================================
- Hits          672      660      -12
- Misses        771      781      +10
+ Partials       74       72       -2

Impacted Files
Coverage Δ

pkg/ca/x509ca/common.go
0.00% <0.00%> (-32.61%)
:arrow_down:

pkg/ca/googleca/v1/googleca.go
52.25% <0.00%> (ø)

pkg/challenges/challenges.go
29.71% <0.00%> (+0.93%)
:arrow_up:

Continue to review full report at Codecov.

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


changing transient values

As @matthiaskoenig noted, it is currently not possible to change transient values. Though of course it could be useful to do so. All task have a flag use_initial_values defaulting to True, that the initial values would be used. But to continue from the current state, it makes sense to be able to change the transient values as well.

this is now available with release 0.18


A Scan has a failure.. Please approve to proceed

Approval required for following Workflow to proceed.

WorkflowName
Workflow#
BranchName
TriggerEvent
LastCommitUser

debug-Api
107
refs/heads/debug
push
Yog4Prog

Issue Details
Found an Issue while performing SCA Scan
:point_right: You are required to Approve (or) Reject the issue
Approve or Reject by commenting in the below section with any of the following keywords
:green_book:  Approved Keys: approved approve lgtm goahead proceed yes
:closed_book: Reject Keys: denied deny reject rejected no abort cancel exit
:point_right: Note: If left uncommented the issue will be closed after the timeout period and the workflow will be marked unsuccessful.

approved


Extra relative path in map comment

Gulp task:
sourcemaps = require('gulp-sourcemaps');
stylus = require('gulp-stylus')

gulp.task('test', function() {
    return gulp.src('src/**/*.styl')
    .pipe(sourcemaps.init())
        .pipe(stylus({sourcemap: {inline: false}}))
    .pipe(sourcemaps.write('.'))
    .pipe(gulp.dest('dist'));
});

This creates CSS files with the following comment: /*# sourceMappingURL=../styles/screen.css.map */ The first part of the path, ../styles, is redundant, because the sourcemap is written to the same directory as the file (sourcemaps.write('.')). The comment should be /*# sourceMappingURL=screen.css.map */.
The same thing happens with the gulp-coffee plugin, so this looks like an issue with gulp-sourcemaps, which is common to both.

Same problem for me with:
var path = "web/app/themes/white";
return gulp.src([path + '/js/vendor/*.js', path + '/js/white.js'])
    .pipe(sourcemaps.init())
      .pipe(concat(path + '/js/white.min.js'))
      .pipe(uglify())
    .pipe(sourcemaps.write('.', {
      includeContent: false
    }))
    .pipe(gulp.dest('.'));

I'll just add to this that setting sourceMappingURLPrefix to ./ also failed to fix this problem.

I'm experiencing exactly the same problem here.

In case newer versions don't fix this, try the new sourceMappingURL option, which allows full control over the URL.


Request for RandomIllumination layer in keras

Something like this already is possible with ImageDataGenerator but with corners in it (only applies brightness scale).
What is needed is a full keras layer that randomize illumination differences spatially across the image (or even a single scaling factor to the whole image).
i think more ideas can be added to something like this if it was implemented properly.

@MohamedAliRashad ,
Please post this issue on keras-team/keras repo.
To know more refer to:
https://discuss.tensorflow.org/t/keras-project-moved-to-new-repository-in-https-github-com-keras-team-keras/1999


[Publisher][UX][Listing] Styling issue in the api listing card in tenant mode.

fixed in 3.0.0 beta


PushNotification is undefined

I have updated my project to phonegap 7.0.1 and using the push plugin version 2.
In my index.js file, after the device is ready, I run this code:
if (typeof PushNotification !== 'undefined') {
          var push = PushNotification.init({
              android: {

              },
              ios: {
                  alert: "true",
                  badge: "true",
                  sound: "true"
              },
              windows: {}
          });

          push.on('registration', function (data) {
              // data.registrationId
              alert(data.registrationId);
              DevExpress.ui.notify("Device registered " + data.registrationId, "success", 3000);
          });

          push.on('notification', function (data) {
              // data.message,
              // data.title,
              // data.count,
              // data.sound,
              // data.image,
              // data.additionalData
              DevExpress.ui.notify(data.message, "info", 10000);
          });

          push.on('error', function (e) {
              // e.message
              DevExpress.ui.notify(e.message, "error", 10000);
          });
      }
      else
          alert("else!!!");

This used to work like charms, now it doesn't. Why?
This is also my confing.xml file
<widget id="com.devexpress.apptemplate" version="1.0" versionCode="1">
  <name>ApplicationTemplate</name>
  <description>Template</description>
  <preference name="phonegap-version" value="cli-7.0.1" />
  <preference name="permissions" value="none" />
  <preference name="prerendered-icon" value="true" />
  <preference name="android-windowSoftInputMode" value="adjustPan" />
  <preference name="SplashScreen" value="splash" />
  <preference name="SplashScreenDelay" value="60000" />
  <preference name="AutoHideSplashScreen" value="false" />
  <preference name="SplashShowOnlyFirstTime" value="false" />
  <preference name="FadeSplashScreen" value="false" />
  <preference name="ShowSplashScreenSpinner" value="false" />
  <preference name="DisallowOverscroll" value="true" />
  <preference name="StatusBarOverlaysWebView" value="false" />
  <preference name="StatusBarBackgroundColor" value="#000000" />
  <preference name="android-minSdkVersion" value="15" />
  <preference name="android-targetSdkVersion" value="22" />
  <plugin name="cordova-plugin-geolocation" />
  <plugin name="cordova-plugin-splashscreen" onload="true" />
  <plugin name="cordova-plugin-whitelist" />
  <plugin name="cordova-plugin-ios-longpress-fix" />
  <plugin name="cordova-plugin-statusbar" onload="true" />
    <plugin spec="https://github.com/phonegap/phonegap-plugin-push.git" source="git" />
    <platform name="android">
    <resource-file src="google-services.json" target="google-services.json" />
  </platform>
  <platform name="ios">
    <resource-file src="GoogleService-Info.plist" />
  </platform>
  <access origin="*" />
</widget>

The first if in my code goes in the else branch. This because PushNotification is undefined. Why is it undefined? Am I missing something?

@pieruz what version of cordova-android or cordova-ios do you use? Run cordova platforms or phonegap platforms to find out.

@macdonst  Sorry for the dumb question, but how can I do this check using DevExtreme?

@pieruz No clue. I've never used DevExtreme. Probably should ask them.

@macdonst  I asked, I will come back here with the answer

@macdonst  Ok, here I am.
Cordova-android version is 6.2.3
Cordova-ios version is 4.4.0

@pieruz those are the supported versions for 2.0.0. Not sure why you are getting undefined. Are you building locally?

@macdonst  I created the template in the phonegap online utility and then I created the package with the DevExtreme tool. Opening the generated package, I see the version I wrote above, with the push plugin included in the plugin list

@pieruz I'm really sorry but I don't have time to research the DevExtreme tool. You would probably be better off asking them for support. Everything you are doing looks correct.

@macdonst  You're right, finally I solved with them the issue. Now it works like charms!

@pieruz can you post what the solution was so that other can benefit if they actually search the old issues?

I wrote a summary here: https://programmingistheway.wordpress.com/2017/07/19/devextremephonegap-how-to-manage-push-notifications-with-fcm/


[Test tools] Add modules' endpoints

Is your feature request related to a problem? Please describe.
Module endpoints should be exposed in the withServerpod helper. It probably makes sense to keep them in a separate namespace just like on the client: endpoints.modules.myModuleExample (as opposed to endpoints.example for non-module endpoints).

Closing as duplicate of: https://github.com/serverpod/serverpod/issues/2848


Changes required to the Public/Proponent export

Issue #: https://github.com/bcgov/epic-engage/issues/17
Description of changes:

Updated logic to download the file in excel format instead of csv.
Updated the template to add formatting

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #27 (0321cfd) into main (bd4931e) will decrease coverage by 0.09%.
The diff coverage is 3.44%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #27      +/-   ##
==========================================
- Coverage   68.54%   68.46%   -0.09%
==========================================
  Files         472      472
  Lines       15593    15610      +17
  Branches     1193     1191       -2
==========================================
- Hits        10689    10687       -2
- Misses       4695     4714      +19
  Partials      209      209

Flag
Coverage Δ

metapi
76.53% <3.44%> (-0.22%)
:arrow_down:

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

Files
Coverage Δ

...rc/met_api/services/document_generation_service.py
80.55% <ø> (ø)

met-web/src/apiManager/httpRequestHandler/index.ts
57.14% <ø> (-7.57%)
:arrow_down:

...rc/components/comments/admin/textListing/index.tsx
80.70% <ø> (+0.70%)
:arrow_up:

met-web/src/services/commentService/index.tsx
37.50% <ø> (+2.20%)
:arrow_up:

met-api/src/met_api/services/comment_service.py
58.71% <3.44%> (-6.60%)
:arrow_down:

... and 1 file with indirect coverage changes


[Image] Cache never expires for the same uri

Even the image from the same uri is changed, <Image> still returns cached data. The code is here: Libraries/Image/RCTImageDownloader.m#L91
There should be a parameter that makes <Image> simply return a fresh copy of the image.

I wonder know too.

I'm pretty sure this is to encourage you to use fingerprints, I may be wrong though. Does your server return cache headers? I'm not sure react-native takes them into account regardless

It's not actually my server. I'm showing a satellite image from other source: weather.gc.ca/data/satellite/goes_nam_1070x_100.jpg
I can proxy the url but it doesn't make sense that cache never expires and there is no way to clear the cache from js code.

If you need a quick fix you can add
var url = 'weather.gc.ca/data/satellite/goes_nam_1070x_100.jpg'
url += '?random_number=' new Date().getTime()

This will however, always download the image.

Thanks @JohnyDays. That should work. But now when the image is downloaded and I want to refresh the file, the obj-c code doesn't update the image... 😓

@umhan35 there might be a bug in the image component, but for a temporary workaround (emphasis on temporary! if there is a bug in react we should fix it), you could specify <Image key={url} /> which will replace your old image with a new one each time the URL changes.

@ide thank you! There is key to rescue. It works.
So, in addition to the bug that the title says, <Image> doesn't refresh itself after loaded.

@sahrens - looked at the git attributions for this and it looks like you are the right man for this issue, feel free to re-assign if I got it wrong :smile:
@umhan35 - can you elaborate on that last point?

@brentvatne I think his last point may be referring to the same problem in #1417, if I understand correctly

@brentvatne when Image is rendered, assign its uri to another image url doesn't make it show the new image.

Oops, sorry about the assign mess, I was trying to assign @a2... Damn buggy chrome.

@tadeuzagallo - Github Issue Assigner - I had the same problem so I made this extension.

From IRC

potomushto:
Is there any plans to introduce a cache policy for images? For example, to be able to set expirationDate for RCTCache keys (<Image expire={week} />) or add methods to manual invalidating (this.res.image.reload())?

@brentvatne it was my message on IRC. I use Image a lot in my app and hardly need to somehow control the cache. I've made a PR #1491 but it's a just silly concept. Any thoughts how it could be done without risks to broke something in <Image/> for others? I see these options:

totally own component as an external library
use new <Image>  feature "displaying local files". So the part responsible for downloading and caching goes to javascript side.
react-native fork <- I'm here

You can also add a cache buster to the url from JS - not super elegant, but might work with no native changes?

On Jun 2, 2015, at 4:04 AM, Dmitry<EMAIL_ADDRESS>wrote:
@brentvatne it was my message on IRC. I use Image a lot in my app and hardly need to somehow control the cache. I've made a PR #1491 but it's a just silly concept. Any thoughts how it could be done without risks to broke something in  for others? I see these options:

totally own component as an external library
use new  feature "displaying local files". So the part responsible for downloading and caching goes to javascript side.
react-native fork <- I'm here

—
Reply to this email directly or view it on GitHub.

The image bug is a separate issue - changing the url should change the image, that definitely needs to be fixed.  Closing this because I don't think we want to change the internals since a cache busting param in the url should solve this particular issue (you can change the URL whenever you want to refresh the image...once that bug is fixed of course.)

@ide react-native 0.8.0 'key' param method seems to have been ineffective?

What's the status of this bug?
I still have this bug on real machine(release), but not happen on emulator(debug).
Am I missing something ?

I ended up adding
[_URLCache removeAllCachedResponses];
to RTCImageLoader.m where it checks for cached response. I added it just to clear the cache then commented it out. I'm still new to all of this so I'm not sure where to change to have my React Native app to set a flag to clear all image cache. For now, this works while I'm still building my app.

I use
<img src={/public/uploads/filename?${Math.random()}}/>

Hi guys, I've the same issue but with local pictures, I do something like that:
        <Image
          source={ object.selected ?
            require('../../images/filter/CheckmarkHighlight.png') :
            require('../../images/filter/Checkmark.png')
          }/>

I don't really understand what's happen, because the picture CheckmarkHighlight is a little bit smaller than the other, and when I select the element the picture change to the size of CheckmarkHighlight but with the picture of Checkmark. So I think it's the same problem as above.
I just find a way to solve it by adding a different key to the picture when it's selected or not, but that's re-render all the Image, and I don't think it's the best way to solve this problem.
What do you think ?

Any progress on this?  I'm getting duplicate images getting rendered when adding images to the end of a ListView.  It only happens when the new row in the ListView is adjacent to another row that also has an Image in it.

I read the entire thread and I am still a little confused.

Is the Image bug solved eventually?

If I use the key={url} trick will that work eventually (by changing the URL?)? Or not?

Thanks!

Seems that local packager images (such as <Image source={ require('./img.png') } />) cached and there is no way to reset cache (on device).
I have to delete app and run it again or change filename to fix image to actual version. Also, setting key prop didn't help in that case (and i think key have another goal)
RN 0.35.0

To reload local images like javascript <Image source={require('./pic.jpg')} /> just kill terminal process which is always opened to new terminal window when simulator is started. Then restart simulator with react-native run-ios for example :)

Bug still present.
I'm fetching photos from a server that's under development, where a request to e.g. /photo/5 can resolve to a different photo than last time (resetting server or doing a new photo upload).
There should be some way to delete the image cache when the app is restarted.

Yea I'm still encoutering this in RN 0.49

@JulianKingman The issue with the Image component still persists. Why was this closed ?

I think the solution is to set some kind of development/production flag, where the caching only happens on production.

I was going through react native node modules and in react-native->Libraries->Image->ImageSource.js, following is defined.
`/**

Copyright (c) 2015-present, Facebook, Inc.
All rights reserved.

This source code is licensed under the BSD-style license found in the
LICENSE file in the root directory of this source tree. An additional grant
of patent rights can be found in the PATENTS file in the same directory.

@providesModule ImageSource
@flow
*/
'use strict';

// This is to sync with ImageSourcePropTypes.js.
type ImageURISource = {
uri?: string,
bundle?: string,
method?: string,
headers?: Object,
body?: string,
cache?: 'default' | 'reload' | 'force-cache' | 'only-if-cached',
width?: number,
height?: number,
scale?: number,
};
export type ImageSource = ImageURISource | number | Array;
`
Which makes me think that there is prop called 'cache', possible values to which are given above. I added to source prop like follows.
<Image source={{uri : myImage, cache: 'reload'}} />
According to its definition, it should always reload the image from the server and not the cache. But it is not working.

@prakharrke I've tested cache: 'relaod',  it's work for me. You can compare with                          cache: 'force-cache' to see the difference.

cache: 'reload is not working in Android.

I am having a flat list with images. When my images updates I am trying to update my flat list images but they were not updating, I have looked into the directory and found that image is already changed there but not reflecting on the flat list. I have also tried key solution as well as a cache='reload solution but nothing worked for me.
Is there any way to solve this?
react: 16.2.0,
react-native: 0.53.0,


Unit tests for Infrastructure

The logging methods are not tested, as the hope is to phase them out in favor of a more robust logging solution


fix: Update Discord invitation

Things added/changed:

Update Discord invitation to http://discord.eddiehub.org.

Nice catch 👌🏼


Nativecall function callback syntax

https://docs.perl6.org/language/nativecall#Function_arguments
has an example like this:
my sub SetCallback(&callback:(Str --> int32)) is native('mylib') { * }

I'm not sure, but that syntax seems wrong to me.  I think it should be:
my sub SetCallback(&callback (Str --> int32)) is native('mylib') { * }

(No ':', space instead).
Can someone who actually knows verify that?  (The space seems to work when I use it, while I get errors with the ':', but I don't actually know why)

Well, it might be that the first has too many parentheses... But you are right, that syntax is wrong. However, we will have to look and check what's actually right...

Everywhere else the colon is used to constrain the signature of a callable, isn't it? The no-colon-but-space syntax reminds me of destructuring which is not at all what is intended.
But I can reproduce what Curt said: with the colon I get an exception, and without the colon it works as expected, e.g. https://gist.github.com/taboege/2e95ce969d26d00342669723c1a11dd1.
I am so surprised that I bugged rakudo/rakudo#2878 about it.

Closed via 1abe655

This is insufficient, @JJ — it makes the example work, but it has no explanation anywhere. Like @taboege noted, the “fixed” example should be, by Signature’s doc, a destructuring, and a nonsensical one.
I don’t think there’s harm in leaving the “fixed” example, but I think this needs to stay open at least until we have some clarity from the rakudo bug on how this syntax is doing what it’s doing.
I’ll annotate rakudo/rakudo#2878 about this.

@JJ I’m suggesting something along the lines of the pullreq #2791 — please take a look.

Closed after merging #2791


Autoscaling AMI: use in PROD

Follow this PR https://git.speag.com/oSparc/osparc-ops-deployment-configuration/-/merge_requests/1098


when save dashboard, 400error occured. I use superset original source and almost do not change source.

Bug description

Screenshots/recordings
No response
Superset version
master / latest-dev
Python version
3.9
Node version
16
Browser
Chrome
Additional context
No response
Checklist

[ ] I have searched Superset docs and Slack and didn't find a solution to my problem.
[ ] I have searched the GitHub issue tracker and didn't find a similar bug report.
[ ] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

python version 3.10
node version 20

This suggests that the error occurs due to missing or invalid query context data when attempting to save the dashboard

error message is from backend， error is {
"message": {
"json_metadata": [
{
"map_label_colors": [
"Unknown field."
],
"shared_label_colors": [
"Not a valid mapping type."
]
}
]
}
}

python version 3.10 node version 20

This suggests that the error occurs due to missing or invalid query context data when attempting to save the dashboard

error message is from backend， error is { "message": { "json_metadata": [ { "map_label_colors": [ "Unknown field." ], "shared_label_colors": [ "Not a valid mapping type." ] } ] } }

python version 3.10
node version 20
This suggests that the error occurs due to missing or invalid query context data when attempting to save the dashboard
@dosu
error message is from backend， error is {
"message": {
"json_metadata": [
{
"map_label_colors": [
"Unknown field."
],
"shared_label_colors": [
"Not a valid mapping type."
]
}
]
}
}

request data ：
certification_details
:
""
certified_by
:
""
css
:
""
dashboard_title
:
"[ untitled dashboard ]"
json_metadata
:
"{"chart_configuration":{"1":{"id":1,"crossFilters":{"scope":"global","chartsInScope":[]}}},"global_chart_configuration":{"scope":{"rootPath":["ROOT_ID"],"excluded":[]},"chartsInScope":[1]},"map_label_colors":{"SUM(net_sales_outside_tax)":"#1FA8C9"},"color_scheme":"","positions":{"DASHBOARD_VERSION_KEY":"v2","ROOT_ID":{"type":"ROOT","id":"ROOT_ID","children":["GRID_ID"]},"GRID_ID":{"type":"GRID","id":"GRID_ID","children":["ROW-3ewow1zW_T4gLZE--DRbf"],"parents":["ROOT_ID"]},"HEADER_ID":{"id":"HEADER_ID","type":"HEADER","meta":{"text":"[ untitled dashboard ]"}},"CHART-S5hNAliZjz5xkRZKM5YZ7":{"type":"CHART","id":"CHART-S5hNAliZjz5xkRZKM5YZ7","children":[],"parents":["ROOT_ID","GRID_ID","ROW-3ewow1zW_T4gLZE--DRbf"],"meta":{"width":4,"height":50,"chartId":1,"sliceName":"TRN_POSヘッダ--純売上外税金額"}},"ROW-3ewow1zW_T4gLZE--DRbf":{"type":"ROW","id":"ROW-3ewow1zW_T4gLZE--DRbf","children":["CHART-S5hNAliZjz5xkRZKM5YZ7"],"parents":["ROOT_ID","GRID_ID"],"meta":{"background":"BACKGROUND_TRANSPARENT"}}},"refresh_frequency":0,"color_scheme_domain":[],"expanded_slices":{},"label_colors":{},"shared_label_colors":[],"timed_refresh_immune_slices":[],"cross_filters_enabled":true,"default_filters":"{}","filter_scopes":{}}"
owners
:
[1]
0
:
1
slug
:
null

@dosu
those are data and source.

@dosu thank you , i continue to check source

Hello @q631599119 thanks for reporting the issue. However, I could not repro this on Superset. I am closing it for now. Feel free to re-open with reproducible steps if you are still experiencing the problem.

Hey :raised_hand: I'm encountering the same issue. I cannot edit dashboards or add new ones.
The issue arises as soon as I define a new dashboard and add any layout element (e.g., a divider bar). Saving is not possible.
In the payload being sent, I do not see these new fields.
This issue occurs on the commit: 827fe0690327604338125adcf2eafd9bc7701ec0.
It was working correctly on a previous version I used (Commits on Nov 6, 2024):
Commit f704b0f5568bc5d07360d8d365510369a6088011.
Could you please reopen the issue? I am happy to provide additional details if necessary.


Add a base64 or encoding feature and make the base64 crate optional

Hi @dullbananas,
What’s your use-case for making the base64 dependency optional?


New role: Configure Ansible logging

Ansible should write logs to /var/log/stereum/ansible.log.
https://docs.ansible.com/ansible/latest/reference_appendices/logging.html

done


Update README.md

What does the program do ?

In what programming language it is written?
###shubham

This pull request do not contains files in specific folders. Please fix it and reopen for help see https://help.github.com/articles/moving-a-file-to-a-new-location/


Update api lint checks for migration-tools

Attempts to fix api linting for the migration-tools package.  It kind of works, but I'm still seeing some oddities in task order, want to see if this will work in CI.
AB#13807

Does this have public types?
(Feels better to me not to have those exports if there aren't any, but no biggie.)

Longer-term it will, so figured it would make sense to go ahead and prep for it.

Does this have public types?
(Feels better to me not to have those exports if there aren't any, but no biggie.)

Longer-term it will, so figured it would make sense to go ahead and prep for it.

Won't types go to beta after alpha, before public? We have all the things that should make the transitions to more supported easy. An advantage of not public now is that you can't accidentally use it. For example, this package doesn't have API reports turned on, which would be the way to see that public exposure is happening.

Does this have public types?
(Feels better to me not to have those exports if there aren't any, but no biggie.)

Longer-term it will, so figured it would make sense to go ahead and prep for it.

Won't types go to beta after alpha, before public? We have all the things that should make the transitions to more supported easy. An advantage of not public now is that you can't accidentally use it. For example, this package doesn't have API reports turned on, which would be the way to see that public exposure is happening.

OK! Updated in latest


Fix update docker cache error

Fix update docker cache error: issue comment.
If /etc/docker/daemon.json doesn't exist or is an empty file, the script will fail.

Coverage remained the same at 33.681% when pulling 5843ce68819ba36e08ec48556f90e7d6cef614e1 on siaimes:patch-2 into 34fc8f600d8a040c901225606660bd1c73b54154 on microsoft:master.


fix(): header and footer position

[!NOTE]
I don't build the source to avoid conflicts from other PRs.

Before fix

After fix

Thanks for bug report. Unfortunately your fix doesn't work if header is bigger than top margin. Have fixed it in another way in my latest commit.


i18n example messages not in library

Studying the library for an implementation I'm doing, I noticed that some pages import standard messages from ./baseMessages and these messages are ignored in .gitignore. Could you remove these files from .gitignore` to have a value reference for custom messages?
This is important considering that some pages, in the context of keycloak, are difficult to reproduce, eg expired login page.

Hi @bralandealmeida,
Only primary source of truth should be tracked by Git. Those messages are fetched from the Keycloak repo.
You get them when installing Keycloakify from NPM though.
You can also clone the repo and yarn install and you will get them.

This is important considering that some pages, in the context of keycloak, are difficult to reproduce, eg expired login page.

Keycloakify features a mock mechanism that enables to test any pages outside of Keycloak: https://github.com/keycloakify/keycloakify-starter#quick-start
We also have a storybook: https://storybook.keycloakify.dev/storybook


Feature request: parse-able output (JSON?)

We are doing offsite backups from a VPS to a residential based server where the internet can be spotty at times... what I am noticing is that the backups will get synced the next time a successful remote connection is made from the host to the residential side... however, it would be nice to be able to indicate X snapshots transferred in Y seconds when these cases happen.
I push the results from btrfs-backup -P to a variable and then pipe that variable to pushbullet for a poor man's monitoring system... this results in a message like:
snapshotname=At snapshot 20180430-174503
Or when the remote connection fails...
snapshotname=
Would love to be able to output more meaningful logs to my client.
I'll take a stab at this myself as well, but python is not my forte these days.
Thanks again for an awesome utility!

Hmmm actually, just double checked this... .outstanding_transfers never gets updated so my remote backups are not staying synced once connections return :(

Hi,
You use ssh:... as the destination, right? What does the .outstanding_transfers file contain after all backups have been synced? This file just contains the snapshot/destination pairs of transfers that have been started but not completed successfully, so that they can be removed and re-transferred at the next run. After a snapshot has been transferred successfully to a remote destination, the "lock" for that particular snapshot/destination pair is removed from the .outstanding_transfers file.
Please note that the file only works reliably when you specify the exact same destination string at each and every invocation of btrfs-backup.
Best regards
Robert

And retry failed transfers independent of the normal backup procedure, you could run
btrfs-backup [your retention options] --no-snapshot --locked-dests <SOURCE>

regularly. It will only transfer existing snapshots that are not successfully sent yet.

I've never seen the file contain anything, despite the destination being offline for > 6hrs (we do hourly snapshots for 24hrs locally and keep 30 days on the remote end). Below is the command that is ran every hour. -- So maybe doing local + remote causes an oddity here?
btrfs-backup -P --num-snapshots 24 --num-backups 744 --snapshot-folder ../snapshots/lib --ssh-sudo /home/project/lib ssh://backups/media/backups/project/snapshots/lib
In this case, backups is a hostname defined in the ssh-config to keep the ssh process simple.

Ok, can you please check whether the file contains something during the transfer is in progress?

I'm currently adding the keeping of local snapshots to backwrap, it should be available later today.

.outstanding_transfers does get updated during transfers.
up to ~500 for 2 different snapshots and so far no noticeable negative side effects -- these are almost never accessed directly on the remote system. Generally the 24 hourly backups are sufficient for our needs.
Will checkout backwrap and the other tools.


Copy a file into container

add function copy_into to container
add example

I updated the PR, providing a better API.

Ci failure is unrelated to your changes. Thanks for implementing this feature!


W-11934998 Removed erroneous == and added information on entryTTL behaviour.

Writer's Quality Checklist
Before merging your PR, did you:

[ ] Run spell checker
[ ] Run link checker to check for broken xrefs
[ ] Check for orphan files
[ ] Perform a local build and do a final visual check of your content, including checking for:

Broken images
Dead links
Correct rendering of partials if they are used in your content
Formatting issues, such as:

Misnumbered ordered lists (steps) or incorrectly nested unordered lists
Messed up tables
Proper indentation
Correct header levels

[ ] Receive final review and signoff from:

Technical SME
Product Manager
Editor or peer reviewer
Reporter, if this content is in response to a reported issue (internal or external feedback)

[ ] If applicable, verify that the software actually got released

Signing the CLA failed with the following error:

I tried emailing the address mentioned<EMAIL_ADDRESS>but the email bounced.
Now if I try to sign the CLA again I get the following issue:

@mike-conroy - Updated with clarification from dev. See also https://docs.mulesoft.com/object-store-connector/1.2/object-store-connector-reference#objectStorev2

@hannanelson Thanks for the clarification, I hadn't seen that. Interestingly, that doesn't seem to match actual results in the runtime when I've tested this.
I have a simple flow with an object store which has an entryTTL of -1 seconds & an Expiration Interval of 1 second. The flow will store the value "testPayload" in the OS, wait 12 seconds (arbitrary) and then retrieves the value again. On retrieval, it returns the OS Default Value "nullPayload". I would expect the Retrieve operation to return the stored value "testPayload" if the expiryTTL defaulted to the maximum (30 days).
<?xml version="1.0" encoding="UTF-8"?>

<mule xmlns:ee="http://www.mulesoft.org/schema/mule/ee/core"
	xmlns:os="http://www.mulesoft.org/schema/mule/os"
	xmlns="http://www.mulesoft.org/schema/mule/core"
	xmlns:doc="http://www.mulesoft.org/schema/mule/documentation"
	xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.mulesoft.org/schema/mule/core http://www.mulesoft.org/schema/mule/core/current/mule.xsd
http://www.mulesoft.org/schema/mule/os http://www.mulesoft.org/schema/mule/os/current/mule-os.xsd
http://www.mulesoft.org/schema/mule/ee/core http://www.mulesoft.org/schema/mule/ee/core/current/mule-ee.xsd">
	<os:object-store name="os" doc:name="Object store"
		doc:id="8caff084-ee2f-4d62-933e-d4d7758080db" expirationIntervalUnit="SECONDS" entryTtl="-1"/>
	<flow name="object-store-ttlFlow"
		doc:id="9c56b56b-6828-447d-a664-25bb8e24ba10">
		<scheduler doc:name="Scheduler"
			doc:id="bcea3c2b-93c6-4d0d-b18d-a6aee0927266">
			<scheduling-strategy>
				<fixed-frequency frequency="1000000"/>
			</scheduling-strategy>
		</scheduler>
		<logger level="INFO" doc:name="Log Storing Value" doc:id="3f97441c-ed5a-4e26-b111-e95d02ea2c9b" message="Storing Value: testPayload"/>
		<os:store doc:name="Store"
			doc:id="40fda9f0-e7ea-4d29-8476-87a3c9ade5e1" key="testKey"
			objectStore="os">
			<os:value ><![CDATA[#["testPayload"]]]></os:value>
		</os:store>
		<logger level="INFO" doc:name="Log Waiting 12 Seconds" doc:id="7e20acc6-0edf-42b7-b5cb-90ffe3e5547d" message="Waiting 12 Seconds"/>
		<ee:transform doc:name="Wait 12 Seconds" doc:id="b600eaff-0ca6-4324-b50a-45f60d9e6ccb" >
			<ee:message >
				<ee:set-payload ><![CDATA[%dw 2.0
import * from dw::Runtime
output application/java
---
{} wait(12000)]]></ee:set-payload>
			</ee:message>
		</ee:transform>
		<os:retrieve doc:name="Retrieve"
			doc:id="00921ce3-8de0-4aff-8cff-e9077098b053" key="testKey" objectStore="os">
			<os:default-value><![CDATA[nullPayload]]></os:default-value>
		</os:retrieve>
		<logger level="INFO" doc:name="Log Retrieved Value" doc:id="7513451d-d3dd-44ca-b933-be259b148563" message="Retrieved Value: #[payload]"/>
	</flow>
</mule>

Deployed to CH 2.0 on 4.4.0:

Hi @mike-conroy , the key should have a ttl of 30 days. I tested the same flow you mentioned but instead of relying on logs, I looked at the actual object store that's created for the app and see the key exists even few minutes after creation.

If you see different behavior, can you please create a support ticket so that the team can look into it? Thank you!
fyi @yunmei-zhang

Thank you @GouthamiKondakindi for checking and validating. I'm not sure why we see different behaviour, I will spend some more time looking at it.

Hi @mike-conroy, If the issue is reproducible, the following info needs to be collected for further troubleshooting.
1.Application jar file exported from the Anypoint Studio
2.OSv2 debug https://help.mulesoft.com/s/article/How-to-enable-DEBUG-level-logging-for-Anypoint-MQ-Mule-4
Kindly have the OSv2 debug enabled when deploying the app so it captures all the info needed.


Page: BrowserApp-Summary is not finished

Just leaving a placeholder reminder

Cleaning up issues before we move to Splunk GitHub, please re-open if still applicable


Extend NuGet.Core to expose v2 feed

Bypass search hijacker, add created and last edited fields

I was able to find a work-around without modifying NuGet.Core.


The redis-master-controller should be stopped

The redis-master-controller should be stopped, and the redis-master will be deleted automatically.

LGTM, thanks!

Shippable is a flake.


[WIP] Using Docker #22

For #22

I'd prefer to use published ROBOT releases rather than BBOP Jenkins. Unfortunately GitHub doesn't provide a URL for a file attached to the latest release -- you have to specify the tag. But people have figured out one-liners that should do the job, such as: https://gist.github.com/steinwaywhw/a4cd19cda655b8249d908261a62687f8

Actually, I think it's fine to specify the tag, that way the dockerfile can be versioned and tied to a particular version of ROBOT. Agreed about the jenkins!

To build:
docker build -t dougli1sqrd/osk:v0.0.1 .
To Run:
docker run --rm -ti -v target:/opt/starter-kit/target  dougli1sqrd/osk:v0.0.1 ./seed-my-ontology-repo.pl -c -d po ro pato -u dougli1sqrd -t "Triffid Behavior ontology" triffo


Fixed #34034 -- Allow setting HTML attributes on Select widget options

https://code.djangoproject.com/ticket/34034

Hello @khink, thanks for this work! Could you fix the code style issues reported by the failed "Linters / black" test?
Alternatively, you could automate the code formatting in your repo by configuring the pre-commit checks.

Thanks for the suggestions, I'll take a look next week!
On Fri, 2 Jun 2023, 18:22 nessita, @.***> wrote:

@.**** requested changes on this pull request.
Great start!
I think this is on the right path but I noticed that we need to be more
generous with the tests that are added. Particularly, I don't think this
proposal is completely testing all the branches if self.option_attrs: ... elif
self.option_inherits_attrs: .... More so, we should have tests for this
change on every widget that inherits from ChoiceWidget(those areSelectand
RadioSelect`).
The Select widget particularly sets option_inherits_attrs = False and
there are no new tests for this class. As an example, I've extended some
checks in test_options and added a new test_options_with_option_attrs to
make the tests more robust (you can check how these fail if some of the new
code is removed):
--- a/tests/forms_tests/widget_tests/test_select.py+++ b/tests/forms_tests/widget_tests/test_select.py@@ -111,6 +111,28 @@ class SelectTest(WidgetTest):
),
)

def test_constructor_option_attrs(self):+        """+        Select options shouldn't inherit the parent widget attrs.+        """+        widget = Select(+            attrs={"class": "super", "id": "super"},+            option_attrs={"data-test": "custom", "class": "other"},+            choices=[(1, 1), (2, 2), (3, 3)],+        )+        self.check_html(+            widget,+            "num",+            2,+            html=(+                """+              1+              2+              3+            """+            ),+        )+
def test_compare_to_str(self):
"""
The value is compared to its str().@@ -332,14 +354,33 @@ class SelectTest(WidgetTest):
self.assertEqual(options[0]["value"], "J")
self.assertEqual(options[0]["label"], "John")
self.assertEqual(options[0]["index"], "0")+        self.assertEqual(options[0]["attrs"], {"selected": True})
self.assertIs(options[0]["selected"], True)
# Template-related attributes
self.assertEqual(options[1]["name"], "name")
self.assertEqual(options[1]["value"], "P")
self.assertEqual(options[1]["label"], "Paul")
self.assertEqual(options[1]["index"], "1")+        self.assertEqual(options[1]["attrs"], {})
self.assertIs(options[1]["selected"], False)
def test_options_with_option_attrs(self):+        options = list(+            self.widget(choices=self.beatles, option_attrs={"class": "other"}).options(+                "name",+                ["J"],+                attrs={"class": "super"},+            )+        )+        self.assertEqual(len(options), 4)+        for option, (i, (value, label)) in zip(options, enumerate(self.beatles)):+            self.assertEqual(option["name"], "name")+            self.assertEqual(option["value"], value)+            self.assertEqual(option["label"], label)+            self.assertEqual(option["index"], str(i))+            self.assertEqual(option["attrs"]["class"], "other")+            self.assertIs(option["selected"], value == "J")+
def test_optgroups(self):
choices = [
(

In django/contrib/admin/widgets.py
https://github.com/django/django/pull/16929#discussion_r1214501886:

@@ -580,8 +580,8 @@ def media(self):

class AutocompleteSelect(AutocompleteMixin, forms.Select):

pass

option_attrs = None

These two changes does not seem to be tested?
In django/forms/widgets.py
https://github.com/django/django/pull/16929#discussion_r1214503105:

@@ -689,9 +690,14 @@ def create_option(
self, name, value, label, selected, index, subindex=None, attrs=None
):
index = str(index) if subindex is None else "%s_%s" % (index, subindex)

   option_attrs = (

       self.build_attrs(self.attrs, attrs) if self.option_inherits_attrs else {}

   )

   if self.option_attrs:

       option_attrs = (self.build_attrs(self.option_attrs, self.option_attrs))

The outer parenthesis are not necessary. Also, and after checking what
does build_attrs do, I think this call does not need the two parameters?
⬇ Suggested change

       option_attrs = (self.build_attrs(self.option_attrs, self.option_attrs))

       option_attrs = self.build_attrs(self.option_attrs)

In django/forms/widgets.py
https://github.com/django/django/pull/16929#discussion_r1214503434:

@@ -689,9 +690,14 @@ def create_option(
self, name, value, label, selected, index, subindex=None, attrs=None
):
index = str(index) if subindex is None else "%s_%s" % (index, subindex)

   option_attrs = (

       self.build_attrs(self.attrs, attrs) if self.option_inherits_attrs else {}

   )

   if self.option_attrs:

       option_attrs = (self.build_attrs(self.option_attrs, self.option_attrs))

   elif self.option_inherits_attrs:

       option_attrs = (self.build_attrs(self.attrs, attrs))

Same here, the line could be:
⬇ Suggested change

       option_attrs = (self.build_attrs(self.attrs, attrs))

       option_attrs = self.build_attrs(self.attrs, attrs)

In django/forms/widgets.py
https://github.com/django/django/pull/16929#discussion_r1214565527:

     super().__init__(attrs)

     # choices can be any iterable, but we may need to render this widget
     # multiple times. Thus, collapse it into a list so it can be consumed
     # more than once.
     self.choices = list(choices)

   self.option_attrs = option_attrs

I think this should mimic what Widget does in init for attrs:
⬇ Suggested change

   self.option_attrs = option_attrs

   self.option_attrs = {} if option_attrs is None else option_attrs.copy()

—
Reply to this email directly, view it on GitHub
https://github.com/django/django/pull/16929#pullrequestreview-1457759874,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABL3OWKPQ4RJCCVLNPUCRLXJIHMPANCNFSM6AAAAAAYYGV5CA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@nessita Thanks for your remarks. I have copied your patch as-is (see last commit) for easy reviewing, thanks for this. I agree all widgets need to be tested. Do you think the commits should be squashed at some point?
And is there any place where we can see which lines in a PR are covered by tests, and which are not?
Looking forward to your reply!

Do you think the commits should be squashed at some point?

Yes, commits should be eventually squashed into one or perhaps two commits, depending on logical units (it's usually only one commit).

And is there any place where we can see which lines in a PR are covered by tests, and which are not?

I'm not sure, what I do is comment out the code block that look suspicious, and I run the tests. If no test fail, then that's a clear indication that the commented code is either unnecessary or is untested.
I'll re-review before this week ends!

And is there any place where we can see which lines in a PR are covered by tests, and which are not?

I'm not sure, what I do is comment out the code block that look suspicious, and I run the tests. If no test fail, then that's a clear indication that the commented code is either unnecessary or is untested.

You can use coverage.py for this, here are the instructions: https://docs.djangoproject.com/en/4.2/internals/contributing/writing-code/unit-tests/#code-coverage

I've shortened the comment as suggested by @shangxiao.

Hi @nessita and @shangxiao, is there anything else i can to to further the progress on this?

Hi @nessita and @shangxiao, is there anything else i can to to further the progress on this?

For me, I think this question would need checking

@shangxiao Thanks, that would be (please correct me if i'm wrong):

To nessita, felix or carlton: can we not make AutocompleteMixin just honour the contract of init()? It's rare to have a mixin break the contract like this and I feel like if subclasses & mixins want to do this then they must behave properly.

I think @nessita @felixxm or @carltongibson would be the ones to comment on that.
If there's anything i myself can do to further progress on this issue, please let me know.

There's no just here... -- Folks are already using (and extending) this in the wild, so we'd need to consider any migration path.
I'm afraid I haven't looked in depth at this specific question, but there have been a few related tickets in this area, that reviewing might help move forward. (Like maybe it would be OK, but without measuring twice I wouldn't like to just say yes.)

I mean… there are grades of modifications from just to not-just 😛
To be clear what I'm referring to is making AutocompleteMixin behave according to what I consider its responsibilities; ie making sure it initialises a complete instance that its supers expect. If a subclass purposefully avoids the "normal" initialisation flow then it should make sure it manually initialises all expected attributes properly:
--- a/django/contrib/admin/widgets.py
+++ b/django/contrib/admin/widgets.py
@@ -490,6 +490,7 @@ class AutocompleteMixin:
         self.db = using
         self.choices = choices
         self.attrs = {} if attrs is None else attrs.copy()
+        self.option_attrs = {}
         self.i18n_name = get_select2_language()

     def get_url(self):

@shangxiao Yes, makes sense. (My worry was only breaking existing usage, but squinting, it's likely OK...)

@shangxiao @carltongibson Thanks for the input! I changed this in<PHONE_NUMBER>8a9c9d64ed9c2b7a8e95ebdfb9eecb. I think this is what you meant, right?

I've pushed an update for the docs, i'll look into the other comments later.

@khink Do you have time to keep working on this?

@felixxm Good of you to check, i had forgotten. Unfortunately the answer is no, i have no more time to spend on this.

@felixxm Good of you to check, i had forgotten. Unfortunately the answer is no, i have no more time to spend on this.

Thanks for letting us know.


Dynamic Integration of bpftrace Scripts into Kernel Collector

Is your feature request related to a problem? Please describe.

Users can define custom telemetry logic based on their specific requirements.
Dynamic loading allows for runtime changes without restarting the kernel collector.
Encourages the community to share and contribute bpftrace scripts tailored for different use cases. See examples: https://github.com/iovisor/bpftrace#tools

Describe the solution you'd like
Enable users to dynamically integrate and execute bpftrace scripts as part of the OpenTelemetry-eBPF kernel collector. This feature aims to provide a flexible and extensible way for users to define custom tracing logic using bpftrace scripts, allowing them to gather specific telemetry data tailored to their application's needs.
Introduce a new plugin mechanism within the OpenTelemetry-eBPF collector that allows users to specify and load bpftrace scripts dynamically. The integration can be achieved through the following steps:
Script Loading:
Extend the kernel collector to recognize a designated directory or configuration file where users can place their bpftrace scripts.
Compilation and Execution:
Implement a compilation step within the kernel collector that compiles the bpftrace scripts into eBPF bytecode, leveraging the existing bpftrace project for this purpose.
Develop a mechanism to dynamically load and execute the compiled eBPF bytecode within the kernel collector.
Configuration:
Introduce configuration options for users to specify the directory or file path containing their bpftrace scripts.
Allow users to enable or disable specific scripts based on their requirements.
Lifecycle Management:
Implement mechanisms for script lifecycle management, including reloading scripts without restarting the kernel collector.
Example Usage
# Set environment variable for bpftrace script directory
export BPFTRACE_SCRIPT_DIR=/path/to/bpftrace/scripts

# Run kernel collector with bpftrace integration
sudo kernel-collector --log-console --enable-bpftrace

# Example bpftrace script: trace disk I/O latency
# /path/to/bpftrace/scripts/io_latency.bt
bpftrace -e 'tracepoint:block:block_rq_issue { printf("%d %s %d\n", pid, comm, args->bytes); }'

Describe alternatives you've considered
No response
Additional context
Security: Implement security measures to ensure that only authorized scripts are executed.
Performance: Evaluate the performance impact of dynamically loaded bpftrace scripts on the kernel collector.

👍
How would you envision processing the messages that are generated?

I think that it might be possible to generate bpftrace code to serialize messages so scripts can encode in the right format.
The collector's message handling code might be configured to forward some messages to the reducer without further processing

Not sure how the new messages might be processed in the reducer, do you have a plan for that? Would you run processing bpftrace scripts in the reducer?


⚠ Cioos National CKAN has degraded performance

In 556d2fd, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 12841 ms

Resolved: Cioos National CKAN performance has improved in eb5e69c after 6 minutes.


Accommodate team member website URL without scheme?

@andersy005 , on https://xarray.dev/team, your website link is set incorrectly as  https://xarray.dev/blog.andersonbanihirwe.dev currently. Cool team page though. It looks like the ones that work correctly have the scheme included in their URL on their GitHub, whereas your GitHub does not.

thank you for reporting this, @zmoon! The issue should be resolved by now.


Teams Android Meeting UI Licensing Documentation Incorrect

As of Microsoft Teams phone app version <IP_ADDRESS>3060906 released June 15, the E3 and E5 licenses mentioned in the article can no longer be used to enable the Meeting UI. The article says:

An E3 or E5 license supports all sign-in modes.

However, this is not true as of the latest app version. As soon as the app is updated to the latest version, the UI reverts from the Meeting UI to the User UI, even though the IPPhonePolicy SignInMode is set to "MeetingSignIn".
Microsoft Support has confirmed for us in ticket #35771137 that E3 and E5 licenses are not supported mechanisms to use the MeetingSignIn mode, contrary to what the documentation says.
I hope that this is a bug and support is incorrect, but as of right now with the latest Teams desk phone app, the documentation is incorrect.

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

ID: 62d58962-9b2f-0231-187b-58c845e52a0c
Version Independent ID: b3045cca-8bcb-8d30-0dda-49aed3009172
Content: Set Microsoft Teams Android devices user interface - Microsoft Teams
Content Source: Teams/devices/Teams-Android-devices-user-interface.md
Service: msteams
Sub-service: itpro-devices
GitHub Login: @tonysmit
Microsoft Alias: tonysmit

Hello @hh-trj I'm working on this

I was let known that a document update is on the works


Add deserialization of Bytes -> Decimal

Arrow2 already has support for Parquet FixedLenByteArray -> Decimal conversion
This PR adds support for Parquet (variable-length) ByteArray -> Decimal conversion, re-using most of the logic from FixedLenByteArray conversion

This PR adds support for Parquet (variable-length) ByteArray

I don't understand. Why would decimal be encoded in variable length binary?

Hi @ritchie46, apologies for the late reply!
Going by the Parquet spec, decimals are actually able to be encoded as int32, int64, fixed_len_byte_array and binary.
See: https://github.com/apache/parquet-format/blob/master/LogicalTypes.md#decimal

binary: precision is not limited, but is required. The minimum number of bytes to store the unscaled value should be used.

Also need to impl for nested https://github.com/jorgecarleitao/arrow2/blob/main/src/io/parquet/read/deserialize/nested.rs


Fix project structure list numbering in README.md

Noticed the readme has number 1 repeated under the Project structure for Shopify API environment variables when it should be 2.

Check out that Contributor badge 😎


Polymer breaks if another project file extends the Object prototype

Description
Polymer breaks if another script extends the Object prototype.
Live Demo

n/a
Steps to Reproduce

Include a script file in Polymer-based project that extends the Object prototype with any method. For example:
Object.prototype.liqid = function () {  return this; }

Expected Results
Polymer elements load without error.
Actual Results
Console is spammed with errors, most appear multiple times. Here are the unique errors, from what I can discern:
polymer.html:1476 Uncaught TypeError: fx.push is not a function
at Object.addPropertyEffect (polymer.html:1476)
at HTMLElement._addPropertyEffect (polymer.html:1683)
at HTMLElement._addAnnotationEffect (polymer.html:1799)
at HTMLElement._addAnnotationEffects (polymer.html:1783)
at HTMLElement._prepEffects (polymer.html:1688)
at HTMLElement._finishRegisterFeatures (polymer.html:4075)
at HTMLElement._ensureRegisterFinished (polymer-micro.html:252)
at HTMLElement.ensureRegisterFinished (polymer-micro.html:237)
at HTMLElement.registerCallback (polymer-micro.html:203)
at desugar (polymer-micro.html:71)
polymer.html:1484 Uncaught TypeError: fx.sort is not a function
at Object.createBindings (polymer.html:1484)
at HTMLElement._prepBindings (polymer.html:1691)
at HTMLElement._finishRegisterFeatures (polymer.html:4078)
at HTMLElement._ensureRegisterFinished (polymer-micro.html:252)
at HTMLElement.ensureRegisterFinished (polymer-micro.html:237)
at HTMLElement.registerCallback (polymer-micro.html:203)
at desugar (polymer-micro.html:71)
at window.Polymer (polymer-micro.html:55)
at [[some-project-file.js:1]]
polymer.html:790 Uncaught TypeError: Cannot read property 'length' of undefined
at Object.add (polymer.html:790)
at HTMLStyleElement._listen (polymer.html:1175)
at HTMLStyleElement.listen (polymer.html:470)
at HTMLStyleElement._listenListeners (polymer.html:459)
at HTMLStyleElement._marshalBehavior (polymer.html:4109)
at HTMLStyleElement._marshalBehaviors (polymer-micro.html:536)
at HTMLStyleElement._initFeatures (polymer.html:4102)
at HTMLStyleElement.__initialize (polymer-micro.html:234)
at HTMLStyleElement.createdCallback (polymer-micro.html:217)
Browsers Affected

[x ] Chrome
[ ] Firefox
[ ] Edge
[ ] Safari 9
[ ] Safari 8
[ ] IE 11

note: Project is eventually built with Electron, so other browsers have not been tested.
Versions

Polymer: v1.9.2
webcomponents: v0.7.24

Confirmed with http://jsbin.com/cixusejuba/edit?html,console,output Not sure how easily we can fix Polymer 1 though, we still advise you to migrate to Polymer 2 which does (should) not have this issue.


Brakets missing on singelton cells

I have a related Issue to #1.
I want to convert a cell array of strings to json and transfer it to an external api which expect an array.  If the cell is not singelton everything works fine. But if i have a singelton cell it is convertet to string data
e.g:
 >> savejson({'A'})
 ans =
 {
      "root": "A"
 }

But the external API  expecting
 ans =
 {
      "root": [
	"A"
]
 }

There is the option 'NoRowBracket' for numeric arrays and I think a second option for cell arrays will be needed, e.g. 'NoSingeltonCellBracket'. The default value should be 0, because if you use a cell array in matlab is more likely that you realy want a array.

This has been requested more than one time, let's add that support. will work on it in the next few days.

I would also like to request this feature if you have not implemented it already.  It would be very valuable.

Here is my fix to this issue
https://github.com/fangq/jsonlab/commit/7dd018bdd9e6edc29985c42ffa49382725c887ab
Basically, I added an option called "SingletCell". By default, it is 1, which force printing [] even a cell has only one element.
I also retired NoRowBracket, and replaced it with SingletArray. SingletArray is set as not(NoRowBracket). By default, SingletArray is 0, which removes [] for a single scalar.
Now, the remaining question is how to deal with struct arrays. Currently, I am using SingletArray flag to control the behavior, as if a struct array is a numerical array - i.e. a 1x1 struct will be printed as
"astruct": { "field1": value1, "field2":value2,...}

instead of
"astruct": {[ "field1": value1, "field2":value2,...]}

What do you think?

Thank you for your effort Qianqian.
Will this preserve the first example file in #15 if you load and save it? In that example you have both a single struct and a single struct within an array. In that issue there is also an example of a file which contains both a key with a scalar value and a key with a scalar value inside an array.
My issue is that the json files saved by savejson is going to be used as input to another system (C++/python) where there is a difference between a single value and and single value in an array, in addition to a single  map/struct and a single map/struct in an array. Therefore it can be some situations where I do not want brackets on all single elements, hence a global flag will not do the job. If for instance a json file is generated by some python or C++ software, and I want to load it in Matlab, change something and write it back, I need load-/save-json to not alter the layout.
It might be to tedious, but currently loadjson is differentiating between a single struct and a single struct in an array by placing the latter 1x1 struct-array inside a cell-array. Could one do the same with a single value in an array to?
So,
{
    "a":[0],
    "b":1,
    "c":[2, 3]
}

will in Matlab look like
a : {[1x1 array]}
b : [1x1 array]
c : [2x1 array]

I'm not sure if this is a good solution, but somehow I need the information about which elements who was originally scalar values (or struct/map) or a value (or struct/map) inside an array to be preserved. So that savejson can store it in the correct way.
My suggestion is that the save json functions automatically adds square brackets to single elements matlab arrays which are contained within a cell-array. It should probably not be the default behaviour, but rater a compliant-mode if the load/save functions are used in combination with other programming languages.
If you have another solution that accomplish the same I'll be fine with that. ;-)

@jpaasen

Will this preserve the first example file in #15 if you load and save it?

try it, it works on my end

My issue is that the json files saved by savejson is going to be used as input to another system (C++/python) where there is a difference between a single value and and single value in an array,

if you have access to the C/C++/python source code, you can make your code more flexible to deal with situations like this. For example, I did the same for a C code I wrote to handle single element with or without bracket
https://github.com/fangq/mmc/commit/08da9f9cf72ce42b2b9bfe0fd6fbccf84b93a099
I used cJSON library to access the JSON structure. If you can implement something like mine, I am sure you can deal with situations like this
{
    "a":[0],
    "b":1,
    "c":[2, 3]
}

however, at this point, JSONLab can not preserve the above form with loadjson/savejson because SingletArray flag will force "a" and "b" to use a consistent format, either with bracket, or not, but not mixed.

@fangq

try it, it works on my end

Good. I'll test it.

if you have access to the C/C++/python source code, you can make your code more flexible to deal with situations like this. For example, I did the same for a C code I wrote to handle single element with or without bracket

Sure, but I my opinion it makes much more sense to handle, what is basically a Matlab issue, within the Matlab json load/save functions instead of everywhere else. Sometimes you have access to the other source code, but sometimes you don't.
I still wish for a mixed bracket mode for single arrays which can be turned on if needed ;-)

here is a workaround. first download the latest version with my above patch. then, try below commands
>> ! cat test.json
{
    "a":[0],
    "b":1,
    "c":[2, 3]
}

>> tt=loadjson('test.json','FastArrayParser',0)

tt =

    a: {[0]}
    b: 1
    c: {[2]  [3]}

>> savejson('',tt)

ans =

{
	"a": [
		0
	],
	"b": 1,
	"c": [
		2,
		3
	]
}

>> loadjson(savejson('',tt),'FastArrayParser',0)

ans =

    a: {[0]}
    b: 1
    c: {[2]  [3]}

The original version of loadjson, derived from below upstream works
http://www.mathworks.com/matlabcentral/fileexchange/20565-json-parser
http://www.mathworks.com/matlabcentral/fileexchange/23393--another--json-parser
http://www.mathworks.com/matlabcentral/fileexchange/25713-highly-portable-json-input-parser
parses any array into a cell array with each array element into a cell element. I added something I called "fast array parser" (https://github.com/fangq/jsonlab/commit/5b82753c0ffe8dba0ed72125beee0495c1b66e62), which groups array elements as much as possible into an array instead of a cell. This alone accelerated the parsing process by 50-80 fold.
Later on, I added an option "FastArrayParser" to set the min dimension beyond which the grouping happens (https://github.com/fangq/jsonlab/commit/9b4c3ba254cb4d9bd5a6fe0cba56538252e8e1ca). Setting it to 0 disables the fast parser entirely.
The difference between [0] and 0 is only distinguishable when you disable the fast array parser.
If you deal only with small JSON objects and wants to keep the depth level of all elements, you may consider setting FastArrayParser to 0 as in the above example. Just be aware that you will have a performance penalty when dealing with large JSON inputs.

I still wish for a mixed bracket mode for single arrays which can be turned on if needed ;-)

of course I open to the idea of generating consistent output. currently, the nature of the difficulty comes from the ambiguity of the data mapping between JSON data structure (numbers, arrays and objects) to MATLAB data structures (scalars, arrays, structs, struct arrays, cells, cell arrays, classes, and objects). Unless we define a mapping that uniquely label the input data, to reproduce any JSON input is going to be difficult.
As I suggested in this reply (https://github.com/fangq/jsonlab/issues/1#issuecomment-63371585), the current goal is to do round-trip convergence after one or two iterations into the loadjson/savejson loop.

Closing this bug for now. please reopen if something comes up.


CSS #regions folding

Part of #45899
For example, when you have
/* #region */
.foo {
 color: red
}
/* #endregion */

You want to have the folding in it.
I don't think this feature is very helpful. Just to keep record of it.
If people do want it we can consider adding it.

@aeschli I fixed it only for comments using #region and #endregion. However in #47300 the screenshots show /* footer */. Should those be supported too or we only support #region and #endregion?

I'd say just #region and #endregion for now (there are requests to make that user configurable)


Multiplatform implementation

Changes

Changed the whole code architecture in order to allow multiplatform usage of the library.

Additions

Added the ripemd160 module
Added the keccak module
Added the extensions module

Removals

I had to remove Gradle Witness as I don't know how to make it work inside multiplatform projects

I see failing tests:
khash.sha256.Sha256Tests > testHashRawBytes FAILED
    java.lang.AssertionError at Sha256Tests.kt:36

khash.sha256.Sha256Tests > testDigest FAILED
    java.lang.AssertionError at Sha256Tests.kt:26

Should be fixed now. I think this was due to a wrong mockkObject call that made the Sha256 object return a different result in subsequent tests. With the last commit this should be fixed.

dam - just saw now after the merge that the module structure is wrong - sha256 is in sha3 - this is wrong - can you fix @RiccardoM

@ligi Implemented the fix inside GH-8


Fix the location of the tutorial of control flow operators.

Description
The tutorial of control flow operators are placed under Module API, but it's actually written for Gluon.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

Thanks for your contribution @zheng-da
@mxnet-label-bot[pr-awaiting-review]

@aaronmarkham for review

@sandeep-krishnamurthy  Please change the label to pr-awaiting-merge


feat: enable yaml-based workflow

Main improvements in this PR:
This PR adjusts HumanGEM model structure and relevant functions to enable a Yaml-based workflow, as proposed in #27

Add enableYamlBasedWorkflow.m script to remove fields (compOutside,rxnMiriams and metMiriams) with duplicated or unnecessary information, and modify elements of rxnReferences and rxnConfidenceScores that affect Mat-Yaml format conversion
Modify importHumanYaml and testYamlConversion functions to comply with recent model changes

Note that this PR has to be implemented after #140
I hereby confirm that I have:

[X] Tested my code on my own computer for running the model
[X] Selected devel as a target branch

@JonathanRob me too.
Actually this PR only adjusts the model structure for allowing lossless Mat-Yaml conversion. Yes there should be another PR, or more, to activate the Yaml-workflow implementation, hopefully from v1.3.1 onward.


SupportID

The SupportID is missing from this doc. Can this ID be used for my own purposes to identify sites?

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

ID: 45804e55-39f4-8929-f245-88b72e627ac2
Version Independent ID: ccd9d1c5-e677-cac9-4219-0145e1a5e6a9
Content: SMS_Identification Class - Configuration Manager
Content Source: sccm/develop/reference/core/servers/configure/sms_identification-server-wmi-class.md
Product: configuration-manager
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Thanks @Garthmj for the feedback. We unfortunately have a large backlog of work for the PowerShell & SDK content, but are looking at ways to improve it in the future.

Thanks again for the feedback. We recently moved this content to a new GitHub repository, so this issue is no longer tracked with the original article. As part of that process, we reviewed our backlog of feedback, and made some tough decisions on items that we don't think we'll realistically ever address. If you're passionate about this specific feedback, please refile it on the current article, now under docs.microsoft.com/mem/configmgr.


Extend API: Allow generated name to be queried and deleted by ID

This will allow API users to query the generated names by providing an ID in the URL.

Hi @theneedyguy ,
Thank you for the PR!  These are great changes and will be integrated into the next version (3.0.1). In reviewing the change, I found a number of areas in the API that were not properly returning the right information (example: If an ID for component doesn't exist).  v3.0.1 will update all APIS to resolve this, as well as add your suggested changes.
I'm trying to determine the best way to merge these changes with your edits. I will update the PR once that process is compelte.
Thank you again for the contribution!
-Bryan

@BryanSoltis Thanks for reviewing my changes. I too found that some areas would not return an empty array if the index was out of bounds but would instead return the last item of the array.
P.S.
The reason for my contribution was to be able to use the naming tool with Terraform together Mastercard's rest_api module. My changes would allow the "creation" and "destruction" of names using the API. What I also found is when using the module, the API would not be fast enough to create multiple names in parallel. This might be because using JSON as the storage is not suited for parallel writes. The workaround is setting the Terraform parallelism to 1 when applying the config. Maybe using a proper DB for data storage could fix this issue or maybe adding some kind of queueing mechanism to make sure the index is already used and to move on to the next free index. This an extremely niche case and the workaround works for me so that might be something for the future.

Hi @theneedyguy,
Thank you for your PR! These changes will be merged into the project, then updated with new functionality to align with recent API updates.
-Bryan

You can test out the changes in the pre-lease here:
https://github.com/mspnp/AzureNamingTool/releases/tag/v3.0.1
Please let me know if you have any issues!
-Bryan


adds init_app method to HealthCheck and EnvironmentDump classes

Adds support for Flask application factory. No public tests so I could only test with available methods.

if you let me know what happened I can try to fix and re-submit.

Hi Frank, I think the problem is with our internal (private) Jenkins server—it doesn't work correctly on third party forks. Nothing to worry about. The Jenkins server does pep8 and pylint validation on the code, but otherwise I don't think there are any unit tests for this project. We can always fix those after it's merged, if that's necessary.

think init_app is breaking add_check methods, I'm getting None for app when I try to add them.
Do you guys happen to have an example for SQLAlchemy, or a similar extension. I think I'm missing something when trying to create a checker.

closing in favor of https://github.com/Runscope/healthcheck/pull/17

Ah man, so sorry. I forgot about this.

No worries; I appreciate the time you put in.


Use editor.document.uri for URI

This is should fix #2. It uses editor.document.uri as URI when calling IssueCollection.set()

Thank you! I'll try and cut a release this evening.

This should now be fixed in v0.2.0, which was just published.


PD: Air gap on dispense

Background
As a user I may want to air gap after a dispense to prevent drips/contamination between wells (in a multi dispense) and during pipette movement across the deck.
Acceptance Criteria

[ ] Add a checkbox for air gap after Blow out (https://www.figma.com/file/0dTEbGNsKYgNwKEW6LLfzJ/Airgap-Delay?node-id=50%3A850)
[ ] Add tooltip on hover (https://www.figma.com/file/0dTEbGNsKYgNwKEW6LLfzJ/Airgap-Delay?node-id=104%3A0)
[ ] Min validation Air gap default is the pipette's min. If there is not enough room left in the tip the default will be the largest value possible between the pipette's min and 0. If a user goes beneath the pipette's recommended minimum the user will get a dismissible warning.
[ ] Max validation
The remaining volume in a tip after the first (or only) dispense is as follows:
(Tip capacity - occupied volume of tip before the dispense) + first dispense volume + air gap volume.
If the user is doing a multi aspirate or 1:1 Transfer then the remaining volume in the entire tip capacity. The max is therefore the tip's capacity.
If a user is doing a multi dispense then the space for an airgap after the first dispense is any left over room in the tip after the aspirate, plus what was just dispensed (first dispense volume + air gap volume). This number gets bigger as the tip does more dispenses, but to keep things simple we will set the max at the smallest air gap.
[ ] Order of operations for single transfer (https://www.figma.com/file/0dTEbGNsKYgNwKEW6LLfzJ/Airgap-Delay?node-id=453%3A6016)
[ ] Order of operations for multi dispense (https://www.figma.com/file/0dTEbGNsKYgNwKEW6LLfzJ/Airgap-Delay?node-id=453%3A6016)
[ ] Order of operations for multi dispense (https://www.figma.com/file/0dTEbGNsKYgNwKEW6LLfzJ/Airgap-Delay?node-id=453%3A6016)

In terms of breaking this down into more technical, bite sized pieces, I see the tickets as:

Add checkbox + tooltip for air gap on dispense (UI)
Add air gap dispense min validation for air gap on dispense (all three paths)
Add air gap dispense max validation for 1:1 transfer and multi aspirate
Add air gap dispense  max validation for multi dispense
Add atomic commands for air gap dispense for single transfer
Add atomic commands for air gap dispense for multi aspirate
Add atomic commands for air gap dispense for multi dispense

Closing this as it is broken down into:

#6347
#6500
#6508
#6510
#6512

Note, still need a ticket for max air gap volume validation for multi dispense. Need to clarify whether it is okay if we clamp the maximum value down to whatever the max air gap aspirate value is. @howisthisnamenottakenyet


@cavvia => Bring back new rails

Reverts artsy/force#2552
Should wait on https://github.com/artsy/gravity/pull/11746

Great thanks - will probably wait till the Gravity patch is out.


USHIFT-1328: document how to pass options to the test runner

Since we have a wrapper script for running the robot runner, it is not
necessarily obvious how to pass arguments through to the runner to
control which tests are run. Add a couple of examples to the
documentation.

/test microshift-e2e

/lgtm


pyright: Fix a 2nd batch of issues reported by pyright

This PR fixes a 2nd batch of issues reported by pyright:

Raise a ValueError on unsupported key types so that applications using a non-RSA or non-ECDSA don't use the key; this helps to subsequently add restrictive type annotations to ima/file_signatures.py
Add type annotations to ima/file_signatures.py
Add type annotations to cmd/convert_ima_policy.py
Enable pyright checking on files that are clean

@stefanberger can you rebase on the latest master?

@stefanberger can you rebase on the latest master?

Done. Also revised some Dict's to make them more concrete...


[BUG] Possible Bloxstrap Error during installation(?)

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
problem
i got a manifest error during installation but i don't know if its something to do with my pc or not 🤷♂
error
install location
C:\Users\myuser\AppData\Local\Bloxstrap
logged error
System.NotSupportedException: Unexpected package manifest version:  (expected v0!)

Is this happening every time you launch? If so, go to https://setup.rbxcdn.com/version-94a1a798754e4385-rbxPkgManifest.txt in your browser, and take a screenshot of what you see.

it was. i ended up downgrading dotnet back to 6.0 and it ended up working. thank you for actually responding Pizzaboxer, i really apreciate it when the devs actually respond!


Use only one tab color picker for all tabs and delay load the picker

Summary of the Pull Request

We only ever have 1 color picker now, instead of each tab having its own
TerminalPage constructs this color picker (upon first request for it)
TerminalPage attaches the color picker to the tab that requested for it
TerminalTab detaches the color picker when it is done with it, so that TerminalPage can attach it to another tab later on

References
#5907
PR Checklist

[ ] Closes #xxx
[x] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Documentation updated. If checked, please file a pull request on our docs repo and link it here: #xxx
[ ] Schema updated.
[x] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Validation Steps Performed
User-end behaviour is the same

YAS

TerminalPage constructs this color picker (upon first request for it)

You should use x:Load for this. XAML already supports building an entire UI and then not actually wasting memory on it until it's been requested!

You should use x:Load for this. XAML already supports building an entire UI and then not actually wasting memory on it until it's been requested!

I'm not entirely sure what this looks like - does that mean in TerminalPage.h and TerminalTab.h we can just declare the variable as ColorPickupFlyout _tabColorPicker instead of ColorPickupFlyout _tabColorPicker{ nullptr }? (I assume this is the desired outcome because otherwise I don't see how the x:Load is helping).
When trying x:Load="False" on the <Flyout> at the root of the xaml file, we get these compiler errors (more info on the restrictions on x:Load here):

We can put x:Load="False" in the StackPanel within the Flyout, but wouldn't that mean we are still doing the rest of the overhead for initializing the flyout before we need to? (Unless we have the members initialized with nullptr in which case, again, not sure how the x:Load changes anything)


AudioSystem - Weird sound at start

Whenever the background music starts, I hear some kind of weird 'popping' sound. Not sure what it is, or how to fix it. Sounds almost like the speakers turning on.
Anyone else hear that sometimes?

Can either of you confirm (or refute), @paked @matiwinnetou  ?

@EtienneBruines I hear it too.

Up for grabs.

on Mac Osx - El Capital I hear this too.

This also might be a godoc.org/github.com/google/mobile issue.

I can confirm this issue. Hear it too.

Closed in #494


Enable managed service in Python tests

Looks like the test failure is still there/the file is not getting auto-created

Yep the test relies on the connector to create the table dynamically. This feature is present in 2.61.0 (and HEAD), but Managed Iceberg version is pinned at 2.60.0.
We can wait and rerun the tests when 2.61.0 is released and Managed Iceberg version is bumped accordingly.

The service config version was bumped and tests are green now! will merge

Awesome - thanks!


Cleanup polyphase classes

Fixes #232, #231

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

Comparison is base (64168c8) 75.37% compared to head (897a08e) 75.30%.

Files
Patch %
Lines

src/sdr/_filter/_design_multirate.py
54.54%
5 Missing :warning:

src/sdr/_filter/_polyphase.py
70.00%
3 Missing :warning:

Additional details and impacted files
@@                Coverage Diff                @@
##           release/0.0.x     #234      +/-   ##
=================================================
- Coverage          75.37%   75.30%   -0.07%
=================================================
  Files                 48       48
  Lines               2575     2588      +13
=================================================
+ Hits                1941     1949       +8
- Misses               634      639       +5

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


⚠ Bofrost.be has degraded performance

In 78cbd9a, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2122 ms

Resolved: Bofrost.be performance has improved in dcf6c81.


Hadoop-18330

https://issues.apache.org/jira/browse/HADOOP-18330
Description of PR
Added extra client attribute to the S3ClientCreation Parameters so that the full s3a path can be accessed!
How was this patch tested?
For code changes:

[X] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

look in the aws testing policy doc to see how to test the patch. Afraid you need to test with your own s3 bucket and credentials.
https://hadoop.apache.org/docs/stable/hadoop-aws/tools/hadoop-aws/testing.html
we cannot change the client factory interface as that breaks every implementation and makes people unhappy, especially applications creating mock clients for testing. This is why we now have a S3ClientCreationParameters Parameter class which is extensible (HADOOP-13351). Add the path as a new field to it and set it in filesystem init

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 37s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
38m 37s

trunk passed

+1 :green_heart:
compile
0m 46s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
compile
0m 39s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
0m 34s

trunk passed

+1 :green_heart:
mvnsite
0m 45s

trunk passed

+1 :green_heart:
javadoc
0m 30s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
0m 34s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
1m 19s

trunk passed

+1 :green_heart:
shadedclient
20m 57s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

-1 :x:
mvninstall
0m 23s
/patch-mvninstall-hadoop-tools_hadoop-aws.txt
hadoop-aws in the patch failed.

-1 :x:
compile
0m 26s
/patch-compile-hadoop-tools_hadoop-aws-jdkPrivateBuild-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>.txt
hadoop-aws in the patch failed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>.

-1 :x:
javac
0m 26s
/patch-compile-hadoop-tools_hadoop-aws-jdkPrivateBuild-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>.txt
hadoop-aws in the patch failed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>.

-1 :x:
compile
0m 22s
/patch-compile-hadoop-tools_hadoop-aws-jdkPrivateBuild-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07.txt
hadoop-aws in the patch failed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07.

-1 :x:
javac
0m 22s
/patch-compile-hadoop-tools_hadoop-aws-jdkPrivateBuild-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07.txt
hadoop-aws in the patch failed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07.

-1 :x:
blanks
0m  0s
/blanks-eol.txt
The patch has 1 line(s) that end in blanks. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

-0 :warning:
checkstyle
0m 19s
/buildtool-patch-checkstyle-hadoop-tools_hadoop-aws.txt
The patch fails to run checkstyle in hadoop-aws

-1 :x:
mvnsite
0m 25s
/patch-mvnsite-hadoop-tools_hadoop-aws.txt
hadoop-aws in the patch failed.

+1 :green_heart:
javadoc
0m 15s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
0m 24s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

-1 :x:
spotbugs
0m 23s
/patch-spotbugs-hadoop-tools_hadoop-aws.txt
hadoop-aws in the patch failed.

+1 :green_heart:
shadedclient
24m 25s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
0m 28s
/patch-unit-hadoop-tools_hadoop-aws.txt
hadoop-aws in the patch failed.

+1 :green_heart:
asflicense
0m 38s

The patch does not generate ASF License warnings.

92m 37s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4548/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4548

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 31d8b8e5242e 4.15.0-169-generic #177-Ubuntu SMP Thu Feb 3 10:50:38 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 83e0cce7129325027e4e34d192c1a99e3fb4666c

Default Java
Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS> /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4548/1/testReport/

Max. process+thread count
749 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4548/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

why did you close this?
Created another branch so closed this!
https://github.com/apache/hadoop/pull/4551