Define service type using enricher section of the f-m-p config

Define NodePort service type using enricher section of the f-m-p config

@cmoulliard: Thanks you very much! You rock :+1:


Custom markers and position absolute

Hey,
I'm trying to create a marker which will have touchable elements around its center. I use position:'absolute' to get the items positioned properly.
Unfortunately, whenever I add position absolute to any view it becomes unclickable. It makes creating fancy markers very hard and reminds me of a certain MC Hammer song.
A simplified example (which works without position absolute) :
<MapView.Marker coordinate={marker.loc} onPress={() => alert("yo")}>
    <View style={{position:'absolute', height:40, width:40, backgroundColor:'red'}} />
</MapView.Marker>

Interestingly, touchable events become blocked if the view is a TouchableOpacity too :
<MapView.Marker coordinate={marker.loc} onPress={() => alert("yo")}>
    <TouchableOpacity onPress={() => alert("yo")} style={{position:'absolute', height:40, width:40, backgroundColor:'red'}} />
</MapView.Marker>

I'm using the latest version available from npm, 0.7.1
Is this issue a bug?
If yes, could you please give me a hint how to fix it?
If not, what's a reasonable way to get it done?

You should set width/height for MapView.Marker if it has an "absolute" child view, otherwise you'll get a 0x0 marker (try "View Hierarchy" in Xcode).
<MapView style={{flex: 1}} initialRegion={{
      latitude: 37.78825,
      longitude: -122.4324,
      latitudeDelta: 0.0922,
      longitudeDelta: 0.0421,
    }}>
  <MapView.Marker coordinate={{latitude: 37.78825, longitude: -122.4324}} onPress={() => console.log("yo")} style={{height:40, width:40}}>
    <View style={{position:'absolute', height:40, width:40, backgroundColor:'red'}} />
  </MapView.Marker>
</MapView>

Completely unrelated but, is the 'onPress' action working for you on IOS?

I would like to know what @DmsChrisPena asked too, thx :)

Hey @zelll, your suggestion combined with centerOffset solved the issue. Thanks!


Mobile styling

This PR consists of:
-Media Queries for small devices

mobile screenshots comparison

NRCAN
IRCC

alpha badge and federal header is large enough to kind of read
alpha badge and federal header are very hard to read

body text is pretty large here -- headers are emphasized by font weight more than size
body text is pretty small compared to headers

form labels are submit button are huge and easy to click
let's make everything bigger pls. Also the footer links aren't centred

at the iPhone 5/5s size, there is extra padding on the left side that introduces horizontal scrolling on the page - which when scrolled all the way left, cuts off the right-hand side of the form

Killing this since #53 went in.


Please Support Persian ( Farsi ) Language

Please Support Persian ( Farsi ) Language

In the future I will expand the supported languages by integrating new translation and speech recognition models and by separating the supported languages for only text modes and text and speech modes, I will let you know when Persian support will be added.

Is it possible to subdivide translation models into specific language models so that they can be run on demand. I know this may make the translation less intelligent, but there should be no doubt about reducing the size of the model and running the efficiency


ConvBNReLU

There is no ReLU for ConvBNReLU in deeplabv3plus.py. Is that right? @CoinCheung

leaky relu is in batchnorm2d

leaky relu is in batchnorm2d

Thanks for your reply. I have found that. That is a strange way of coding.


Fix copy-paste error in DiGraph.linearize documentation

Checklist

[y] Did you specify the type of improvement?
[y] Did you state the API impact?
[y] Did you specify the code generation impact?
[y] Did you request a desired merge strategy?

Type of Improvement

documentation

API Impact

No changes.
Backend Code Generation Impact

No changes.
Desired Merge Strategy

Rebase: You will rebase the PR onto master and it will be merged with a merge commit.

@sahandKashani, I can merge this PR if you rebase, if that's what you'd like to do.

Hi Albert. Yes, a rebase merge would be great. Thanks!

@sahandKashani I was able to resolve the conflicts local, but I can't push fixes to your remote branch without access, so would you be able to update this?

Sorry, it's the first time I'm submitting a PR to a repo other than one I own. Just to be sure I understand correctly, you'd need write access to my epfl-vlsc/firrtl repo to push your rebase changes, is that correct?

No problem! We are always very grateful to get outside contributions.

Just to be sure I understand correctly, you'd need write access to my epfl-vlsc/firrtl repo to push your rebase changes, is that correct?

Yes

Great! Thanks.
You should have write access now.

 All committers have signed the CLA.

Great! I updated it and marked it for merging.

Awesome. Thanks!


Some queries added and additional documentation

@Springstone,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


fx: error while loading shared libraries: libbz2.so.1.0

Just letting you know I received this error on fedora 38 beta.
fx --version
fx: error while loading shared libraries: libbz2.so.1.0: cannot open shared object file: No such file or directory

sudo ln -s /usr/lib64/libbz2.so.1 /usr/lib64/libbz2.so.1.0

fx --version
felix v2.2.5: Up to date.

On fedora it's provided by bzip2-libs but there isn't a symlink to that version
sudo rpm -ql bzip2-libs
/usr/lib/.build-id
/usr/lib/.build-id/f8
/usr/lib/.build-id/f8/237ff24b622047428b97b1486b393e3a9e38de
/usr/lib64/libbz2.so.1
/usr/lib64/libbz2.so.1.0.8
/usr/share/licenses/bzip2-libs
/usr/share/licenses/bzip2-libs/LICENSE

Thank you for the report.
Internally felix calls cargo search felix --limit 1, so if fx --version does not work out of the box, cargo command may also have a problem.

Let me figure out: You received that error message only by fx --version, or fx also output that? If latter, I have to dig into it.

I get the error with both commands. I noticed it first with fx and then saw it happens with fx --version too. Then I created the symlink to fix it for myself, but thought you'd want to know.

Thank you!
Fedora 38 beta looks pretty new. I'll check it out.

The final release is supposed to come out this week so you shouldn't need to install the beta


well explained

Introduction gives conclusive difference between Holt winter and other methods(that require stationary data) on the other hand time series is usually not. Can we provide more examples on real time case studies on the same to substantiate the statement.

@balachandarganesan Sure, I will provide more details in my upcoming time series projects.


Control + S doesnt set to Shuffle

Describe the bug
Title says it.
To Reproduce
Press C+s
Expected behaviour
should start shuffle mode
Environment

OS: Arch
Application version: latest
Application features:

Additional context
maybe the problem is in front of the screen
edit: the problem was indeed in front of the screen.

Hi @dbt0815, following up, can you provide more details? Otherwise, I'll close this issue

can you read the edit?

Yes, but "in front of the screen" is very vague. Like I already mentioned w/o logs, backtraces, or pictures, I couldn't understand the issue.


Performance

Please read the Julia performance tips
In particular, do not use untyped arrays like a = [ ].
All arrays should be typed.

e.g.
a = Int[ ]

creates an empty array of integers.


TypeError: this.el.select2 is not a function

https://github.com/rkit/react-select2-wrapper/issues/36 is still happening with the latest version 1.0.4-beta.
TypeError: this.el.select2 is not a function when mounting a component in a Jest unit test.

I'm having the same problem. This is what i have in my package.json "react-select2-wrapper": "^1.0.4-beta5". Any solution?

I'm also having the same issue with the latest 1.0.4-beta6
TypeError: this.el.select2 is not a function
        at Select2.initSelect2 (node_modules/react-select2-wrapper/lib/components/Select2.js:94:15)
        at Select2.componentDidMount (node_modules/react-select2-wrapper/lib/components/Select2.js:64:12)

i only got this error when using jest + enzyme

@rkit any help you could provide here? I've spent a week trying to figure this out, building mocks for select2, importing jquery in different places, etc. This is the only issue preventing me from using your library.

solved by adding this
global.$ = require('jquery')
global.$.fn.select2 = require('select2')

on jest setup file

Thank you @agungyuliaji, you're a lifesaver!

Glad to hear that!
anyway it's jest problem. the issue is not related with this plugin.

the issue was resolved for me when I added
global.$ = require('jquery')
global.$.fn.select2 = require('select2')
to app.js
Thanks @agungyuliaji

@rkit this can be closed.


Beds and piston head do not face correctly

Using the latest version (1.0.3), beds are not fading correctly:

Piston heads to not fade correctly too (it's worse when they move)

Video of a piston moving when fading: https://youtu.be/Mvue6cWZUwc
I have not included blocks with faces at non-90° angles (banners, signs, torches, lecterns, ferns, etc...), entities & liquids
(Banners now keep their color, but they still don't face correctly because they fall in the "blocks with faces at non-90° angles" category)
Chests and shulker boxes do not fade correctly, maybe because they fall in the "blocks with faces at non-90° angles" category when the player opens them

Unfortunately these also cannot be fixed right now due to these blocks actually falling in one of the entity rendertypes. For example, beds are rendered as rendertype_entity_solid, which means it's not possible to reliably figure out which corner of a face is where relative to the center of the face. Thank you for the post though!


Error: handlers cannot have an undefined key

I have a problem to create reducer using a Handlers object.
This is my environment version:
"axios": "^0.18.0",
    "jwt-decode": "^2.2.0",
    "react": "^16.4.2",
    "react-dom": "^16.4.2",
    "react-redux": "^5.0.7",
    "react-router-dom": "^4.3.1",
    "react-scripts": "1.1.5",
    "redux": "^4.0.0",
    "redux-logger": "^3.0.6",
    "redux-saga": "^0.16.0",
    "reduxsauce": "^1.0.0"

Here is a project link.
Please I need a help to solve that.
https://github.com/francisrod01/redux-sagas-app

check that the HANDLES Object property keys is not undefined for example :
Types.SAY_GOODBYE should't be undefined or null.

The key it's ok in the reducers/auth.js file
The sagas/index.js is below:
...
import ActionCreators, { Types } from '../actionCreators';

function* signIn(action) {
  console.log('=== call to signIn(...) saga.');
}

export default function* rootSaga() {
  yield all([
    takeLatest(Types.SIGNIN_REQUEST, signIn),
  ]);
}

Here the auth reducers  file:
import { Types } from '../actionCreators';
...
export const HANDLERS = {
  [Types.SIGNIN_REQUEST]: signInRequest,
}

export default createReducer(INITIAL_STATE, HANDLERS);


Long names are displayed as "Anonymous Applicant"

Describe the bug
If your name has more than 2 spaces in it (for example, first, middle and two last names), it is not tokenized correctly and as a result, the name in the top right corner is displayed as "Anonymous Applicant".  Handling of the displayed name is done here: https://github.com/seattle-uat/civiform/blob/56dbc97751b5ade86fed62ae0072232e4c6697eb/universal-application-tool-0.0.1/app/services/applicant/ApplicantData.java#L93-L102
Expected behavior
The user's full name should be displayed.
Screenshots

Hello, I'm interested in this project and would like to tackle this one if possible. I've posted an intro in the slack and mentioned this good first issue. Thanks in advance.

Hey, I'm also interested in doing this. Did you start working on this @jalengg or is it still free?

Hey @alluvm, I've created a fork with my solution-in-progress if you'd like to take a crack at it as well.


GIF Image not animated after resize

Expected behavior
The animated gif image should stay animated after resize
Actual behavior
Gif image is a still image now
Steps to reproduce behavior
The gif image is inserted in Wordpress as a Featured Image. Then I use the following code:

WordPress, PHP and Timber Versions
WordPress 4.4.2, PHP 5.6.10, Timber 0.22.5
How did you install Timber?
Timber is installed via Github and updated to the latest via WP Upgrade Plugins.

Thanks for the detailed info @dkdl - will look at this over the weekend.

@connorjburton Thanks for taking a look later on.
In case it is relevant:
I'm using OSX 10.11.3 and MAMP 3.4 for the localhost

Hi @dkdl - I've looked into this and it seems to work for me on a local installation. Could you provide the GIF you are using please?

Hi @connorjburton
I've attached the original file. And the GIF it generates by resizing.
original:

800 resize:

Is it otherwise possible to ignore GIF files from resizing? That would be cool as well in my case.

Hi @dkdl - that image actually works when I resize, resizing smaller and larger than the source.
http://recordit.co/1UnwTpDQAw
Would it be possible for you to grab the master on github and see if that fixes the issue?
Couple of debug suggestions:

Try a default theme
Try disabling all plugins except Timber

Thanks

Hi @connorjburton
thanks for the video. Unfortunately no difference. For now I've solved it by checking if it is a GIF:
{% if post.thumbnail.src %}

  {% if '.gif' in post.thumbnail.src %}

  {% else %}

  {% endif %}

{% endif %}

Hey,
Sorry I can't help any further, hate to say it but I don't think this is a direct issue with Timber. If you do find more info that could help just comment on here and I will re-open this issue.
Thanks

@dkdl a possibility here is the type of image resizing engine available on your server. The TimberImage processing uses wp_get_image_editor which could use GD or ImageMagick (or something else?) based on what is installed on the server. It's possible that Timber needs one or the other that isn't installed on your server. See if you can identify which library is used; that might help us ID why it works for @connorjburton and I, but not for your setup*
*but warning: it's very possible that a less robust image library simply won't be able to resize animated gifs, in which case the best solution here might just be a warning in the error logs (which is helpful, but unsatisfying)


feat(General): :sparkles: Add global refresh button

Description
Briefly describe the changes made in this Pull Request.
Additional context (optional)
Add any additional context or information about the changes made in this PR.
How to test
Provide clear and concise instructions on how to test the changes made in this PR.

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

Comparison is base (e5ebe57) 89.49% compared to head (a3ef646) 89.76%.
Report is 17 commits behind head on main.

Files
Patch %
Lines

src/core/components/SkUpdateDataButton/index.tsx
91.30%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #378      +/-   ##
==========================================
+ Coverage   89.49%   89.76%   +0.27%
==========================================
  Files         117      117
  Lines        3475     3450      -25
  Branches      779      770       -9
==========================================
- Hits         3110     3097      -13
+ Misses        335      325      -10
+ Partials       30       28       -2

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


Missing validation

Your parser seems to not validate the input files. This makes it easy to crash and possible exploit a program using it. If this is by design it should be documented in the readme.

loader check of its a valid elf64 file before preceeding in https://github.com/finixbit/elf-parser/blob/master/elf_parser.cpp#L202


Removing repotoken flag from goveralls invocation

References:

https://github.com/mattn/goveralls#github-integration
For a public github repository, it is not necessary to define your repository key (COVERALLS_TOKEN).

Coverage remained the same at 66.122% when pulling d3ef97e3befddb4acba66e784cda11b8f93453f1 on fix-goveralls into 4f98dd7dbf66e1997ccedf4ffe0ff05f8dbbf127 on dev.


[PDI-15928] Get Repository Names step hangs when working with the Pen…

…taho
Repository - fix missing subdirectory if path does not root

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f plugins/pdi-pur-plugin/build.xml clean-all resolve jacoco && ant -f plugins/pdi-pur-plugin/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  plugins/pdi-pur-plugin/src/org/pentaho/di/repository/pur/PurRepository.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.repository.pur.PurRepository

Instruction Change: -.06%:small_red_triangle_down:
Line Change: -.02%:small_red_triangle_down:


certificate signed by unknown authority while registering a test broker

Bug Report
What happened: ErrorFetchingCatalog while registering a broker
What you expected to happen: fetch the catalog as expected
How to reproduce it (as minimally and precisely as possible): create a test broker and try to register it
Is there way to skip cert verification or any way to add this cert to trusted list please help me

Hi @yakuza-m

create a test broker and try to register it

could you provide YAMLs that you used to install the broker?
--
As you can see here: https://github.com/kubernetes-sigs/service-catalog/blob/7b32d83b04d4c0f80c96841dcda3a9a009e9bb8d/contrib/examples/walkthrough/test-broker/test-servicebroker.yaml#L11-L25
by default the TLS is commented-out. So the minimal way to register the broker is to create such CR:
apiVersion: servicecatalog.k8s.io/v1beta1
kind: ClusterServiceBroker
metadata:
  name: test-broker
spec:
  url: http://test-broker-test-broker.test-broker.svc.cluster.local

could you provide YAMLs that you used to install the broker?

I used command to register/install  the broker
svcat register test-broker --url http://test-broker.ups-broker.svc.cluster.local --scope cluster


Abstract routing API sorts routes.

Fixes #1879.
Changes proposed in this pull request:

This PR is based on comments in PR #1898
Fix moved from RoutingMiddleware to AbstractRoutingAPI

Let me know if anything else needs adjustments.

Just an FYI that I've confirmed this fixes routing bugs in connexion that exist today.
For example, attempt to add these 2 routes:
paths:
 /users/{user_id}:
    get:
 /users/accept:
   get:

In 3.1.0 GET /users/accept is routed to GET /users/{user_id}
Not sure if there's more needed in this PR to get it merged but we've had to fork connexion until it will be as AsyncApp broke our routing.
LMK if I can help out with some test cases or whatever to get this rolling.

coverage: 94.153% (+0.003%) from 94.15%
when pulling<PHONE_NUMBER>200967155c6a56438350ccc3f1d097 on BorekZnovustvoritel:sort-routes-api
into 7c4fcc4785da8ee6d671315b9e0be5499a980334 on spec-first:main.

@dkoston adding a test would be very much appreciated and will prevent us from accidentally breaking the behavior in the future.

@RobbeSneyders is there any documentation on how to run the test suite?
I added some tests but running the lines from .github/workflows/pipline.yml results in
almost all tests failing even though I haven't changed any code other than adding a test.
pip install --upgrade pip
pip install "poetry<2" "tox<4" "tox-gh-actions<3" "coveralls<4"
tox

@RobbeSneyders is there any documentation on how to run the test suite?
I added some tests but running the lines from .github/workflows/pipline.yml results in almost all tests failing even though I haven't changed any code other than adding a test.
pip install --upgrade pip
pip install "poetry<2" "tox<4" "tox-gh-actions<3" "coveralls<4"
tox

@RobbeSneyders @jmcs LMK if you have some docs on how to run the test suite locally and I'll get these tests completed. Thanks!


A solution for #12 - handling non-default encoding

Rather than ignore troublesome items, this ignores troublesome characters.

Rejected. This modified entries going into the menu and therefore has unknown consequences.


✨(backend) add list and delete endpoints to invitations router

Purpose
Add an invitation router, nested below teams/

create to allow team owners/administrators to invite members
list pending and expired invitations
delete to cancel invitations

Proposal
In a manner similar to #50, continue CRUD actions to complete team/<team_id>/invitations/ route

[x] add route and viewset
[x] add create action
[x]  add uniqueness together to avoid duplicate of email in the same team
[x] permission tests
[x] list
[x] tests list
[ ] test expired invitations are listed too
[ ] delete endpoint
[ ] tests delete

Retrieve and update actions will be dealt with later.

"Merged" this PR into #50


Widgets: Color Picker Changes

If a user can choose a color, don't have them enter the hex, instead they should have a color picker to use.

Instead should be something more like this:

Task breakdown

[x] Implement a color picker interface, should be reusable
[x] Add into congratulations widget

Added a color picker Input type #72


use prettier check in CI

Which issue does this PR close?
Closes #415
Closes #421
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

@Jimexist  this PR has some conflicts -- would it be possible to rebase it?


CORE-8111/salt-passphrase-secret added Kubernetes secret configuratio…

Added functionality for the db.worker.salt and db.worker.passphrase to be passed In by the client's Kubernetes secret.
https://r3-cev.atlassian.net/browse/CORE-8111

@davidcurrie when testing the changes, I noticed that in an override file, You could specify the secret key without the secret name, value and secret key and value and secret name together without the helm chart complaining. Is this something the hem schema should deal with or...

@JasonSinclair95 - the only requirement should be that, if you specify a secret name, then you must also specify the key. The name is the trigger to use the secret rather than the value. So if they specify a key but no name then it's fine to ignore it. Likewise, if they specify a value and a secret name and key then it is fine to ignore the value.


CLOUD-838 cancellable docker tasks

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1298/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1300/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1301/


Invalid encoding

Because I wanted to pass the text to Godot and support Chinese, I used
cur_transcribed_msg["text"] = cur_text.utf8(transcribed[i].text.c_str());
but in some cases, it may trigger invalid encoding,
like this Unicode parsing error, some characters were replaced with ? (U+FFFD): Invalid UTF-8 leading byte (98)
I'm not quite sure

Is it related to the Windows I am using : https://github.com/ggerganov/whisper.cpp/pull/1313
Is it related to the utf8 method?

Perhaps you can help solve this problem?

@bruvzg Do you have any idea how to resolve this to support Chinese?

utf8 is static function, so it should be:
  Dictionary cur_transcribed_msg;
  cur_transcribed_msg["is_partial"] = transcribed[i].is_partial;
- String cur_text;
- cur_transcribed_msg["text"] = cur_text.utf8(transcribed[i].text.c_str());
+ cur_transcribed_msg["text"] = String::utf8(transcribed[i].text.c_str());
ret.push_back(cur_transcribed_msg);

The error is most likely correct and your string is not UTF-8 (might be some unsupported variant like CESU-8). Can use provide a raw content of the source string that is causing this error?

Or it's something wrong with the previous msg.text processing, it seems to be inserting stuff into a string, and probably not taking into account UTF-8 character sizes, so it might be adding stuff in the middle of sequence. Or it's some segments received from whisper_full_get_token_text are incorrectly skipped and cutting part of the sequence.
So ideally, please provide (for the problematic text), otherwise it's hard to deduce what's wrong and at what step:

raw (hex encoded byte sequence as it is in memory) output of whisper_full_get_token_text (all separate segments).
raw content of transcribed[i].text.

Can I output the text you need through UtilityFunctions:: print? Or through other func?
UtilityFunctions:: print ("token", token);
UtilityFunctions:: print ("text", text);
UtilityFunctions:: print ("msg. text", msg. text);
UtilityFunctions:: print ("utf8 msg. text", String:: utf8 (msg. text. cstr()));

Can I output the text you need through UtilityFunctions:: print

No, it will try to print it as a string, and fail it there's something wrong, so you'll need to use custom print functon, something like:
void print_hex(const std::string &p_string) {
    for (i = 0; i < p_string.size(); i++) {
        if (i > 0) printf(" ");
        printf("%02X", p_string[i]);
    }
    printf("\n");
}

ok, I will try to test it

Thank you very much for your help. Indeed, as you said, it was because I overlooked some of the text corresponding to tokens that caused it
if (token.p > 0.6 && token.plog < -0.5) {
  WARN_PRINT("Skipping token " + String::num(token.p) + " " + String::num(token.plog) + " " + text);
  continue;
}
if (token.plog < -1.0) {
  WARN_PRINT("Skipping token low plog " + String::num(token.p) + " " + String::num(token.plog) + " " + text);
  continue;
}

It may be related to this issue, as each token does not necessarily correspond to a complete utf8 byte
https://github.com/ggerganov/whisper.cpp/pull/1313#issuecomment-1875832602
In summary, I removed the step of skipping text corresponding to certain probability tokens, and everything is now working properly.
Thank you again for your help @bruvzg


Can the podspec version be increased when transitive dependencies change?

Hi there! I have an add-to-app Flutter widget that builds on my development machine, but not on CI. I can trace the problem to this commit: https://github.com/googleads/googleads-mobile-flutter/commit/23c37303e542ca7ea16d892a3b06bdc5fa41f0e4
My best guess is that Cocoapods assumes it can cache podspecs locally until their version numbers change.
Plugin Version
2.0.1
Steps to Reproduce
Given an iPhone app with a Flutter add-to-app widget,

Declare a dependency on Google Mobile Ads in Podfile: 'Google-Mobile-Ads-SDK', '~> 9.6'
Declare a dependency on google_mobile_ads in pubspec.yaml: google_mobile_ads: ^2.0.1
pod install
Wait a few weeks (until Google modifies google_mobile_ads.podspec again)
pod install

Expected results:
Podfile.lock reflects a new version of google_mobile_ads, e.g.:
- google_mobile_ads (0.0.2):
    - Flutter
    - Google-Mobile-Ads-SDK (= 9.13.0)

Actual results:
pod install fails to update this block of Podfile.lock from 9.6.0 to 9.13, possibly because the google_mobile_ads version number remained on 0.0.1.
- google_mobile_ads (0.0.1):
    - Flutter
    - Google-Mobile-Ads-SDK (= 9.6.0)

This leads to a build failure:
[!] CocoaPods could not find compatible versions for pod "Google-Mobile-Ads-SDK":
  In snapshot (Podfile.lock):
    Google-Mobile-Ads-SDK (= 9.6.0, >= 8.0, ~> 9.6)

  In Podfile:
    gadzooks (from `Modules/gadzooks`) was resolved to 6.5.0, which depends on
      Google-Mobile-Ads-SDK (~> 9.6)

    google_mobile_ads (from `Modules/native-apps-flutter/games/games_module/.ios/.symlinks/plugins/google_mobile_ads/ios`) was resolved to 0.0.1, which depends on
      Google-Mobile-Ads-SDK (~> 9.13)

Thanks for the report @mnespor
This looks similar to https://github.com/googleads/googleads-mobile-flutter/issues/673 which was fixed recently.
Also, the plugin version you are on is not latest, maybe you can try to upgrade the version to latest and run your use case once again to see if the behavior still persists or not.

Hi @darshankawar , thanks for the review!
#673 is unrelated as far as I can tell. It looks like #673 was an incompatibility between two podspecs. #753 is more so about an incompatibility within one podspec over time, possibly due to a violation of semver in an ecosystem that expects semver.
The key thing that's happening on the CI machine is that I have two ranges: Google-Mobile-Ads-SDK, ~> 9.6 and google_mobile_ads: ^2.0.1. pod could satisfy those constraints with 2.3.0 and 9.13, but something is going wrong, and it's not finding that resolution.
I have two guesses:

It's possible pod assumes a one-to-one relationship between google_mobile_ads podspec versions and Google-Mobile-Ads-SDK versions, and it cached the 0.0.1->9.6.0 relationship. When 0.0.1 was later republished with ~>9.11 (one to many), maybe pod failed to recognize the change.
It's possible install_all_flutter_pods didn't recognize the incompatibility, and didn't update google_mobile_ads from 2.0.1 to 2.3.0 in pubspec.lock.

I think this might be fixed already, actually? Sorry for the bother.
What worked

I switched from Flutter 3.3 to Flutter@master. On master, pod install did update pubspec.lock to include<EMAIL_ADDRESS>
Things I tried that didn't work

flutter clean && flutter pub get
pod cache clean google_mobile_ads
rm Podfile.lock

At this point, the second guess – install_all_flutter_pods – seems most likely to me. There might have been a behavior in install_all_flutter_pods that changed between 3.3 and master?


Read vhd verity footer

Anything new here?

do we only want this for the default VPMem support?

do we only want this for the default VPMem support?

for now yes, I'll need to validate the other path.

Can you add the commit msg in the pr description?


How can I validate a form manually (programmatically) ?

I know that vfg can validate after each change or on load, but how can I use an external element (not the build in submit field) to manually force a validation ?
My exemple is like that :

            <vue-form-generator :schema="schema" :model="model" :options="formOptions" :isNewModel="true"></vue-form-generator>

Quit

Back

Save

Next

What I'm doing is that I'm creating a form with multiple "pages" or "sections" but one model. I load of schema by "page" and users can go back and forth.
What I want is validate the form when someone click on the "Next" button (or "Back").
So users fill the questions and when next is clicked, I launch validation and scroll the container to the question. The "scroll to bad part" is not complicated, since I can just look at the classes. The complicated is trigger the validation process manually.
Thank you for your help!

It's not easy. The validation is inside vfg, so if you want to call, you need a reference to vfg with $refs.
After it you can call the validate() method.
This solution is not too good if you have more vfg on the same page, but currently (I think) there are no alternatives.

Thank you! For my problem it is a good solution. I will test it soon and close when done.


Automate the user Login.

Hi,
It's an amazing program to learn oAuth login with Microsoft. I have a requirement to automatically download inbox for some service email accounts for my organization. How do I automate the system so it won't ask me to login every time with the code?
Thanks,
Mahendra

The best place to ask general questions about Graph is on Microsoft Q&A. For accessing service accounts unattended, a good approach is to use app-only authentication.


remove misleading 'cents' from pricing

You don't want to pull a Verizon ;)
https://www.youtube.com/watch?v=MShv_74FNWU
Per the actual pricing page (https://azure.microsoft.com/pricing/details/key-vault/) it appears the rate is .03 dollars or 3 cents

Hi @BenSeverson, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@BenSeverson
Good catch!
@ggailey777

@BenSeverson thanks for reporting this. We shouldn't have been putting any pricing information in this topic, since such info belongs on the pricing page. We have updated this topic in the private repo, so am going to close this one. We appreciate your help with improving the quality of Azure docs.


AsciiDoc documentation and CI/CD workflow

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


[BUG] Create ILLA Builder error: Docker responded with status code 400: invalid mount

Describe the bug

To Reproduce
Steps to reproduce the behavior:

Install Rust via Homebrew
Install illa cli tool via cargo: cargo install illa
Run command: illa deploy --self
See an error above

Expected behavior
ILLA Builder is deployed ok
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

Macbook Pro, M1
OS: Mac OS 12.4
Browser: Chrome
Version illa cli: 1.26

Additional context

Rust 1.66
Run Docker via Colima
Docker version 20.10.16
Colima version 0.5.2

@Lan-Phan Hi, the Colima with Docker is an untested environment for illa and we haven't tried it. Please try Docker Desktop.
And we found the issue (https://github.com/abiosoft/colima/issues/503), Does the Colima have the mount issue?

Hi @karminski ,
Thanks for your info. Let I dig more into Colima ...
In the meantime, would you please show me the path which is used in mount volume when running illa deploy command?
Thanks.

I confirm this issue is from Colima.
Here are steps I did to make it work:

upgrade MacOS to version 13.1 (Ventura)
upgrade colima/lima to the latest version
remove all colima / lima old data
using vm type vz and virtiofs mount type and mount /tmp/illa-data writable for VM lima
run docker run from script here https://github.com/illacloud/deploy-illa-manually/blob/main/docker/scripts/run.sh , just 1 minor change is to mount /var/lib/postgresql/data in container to sub-directory in /tmp/illa-data (for ex: /tmp/illa-data/pg) (see my comment here https://github.com/abiosoft/colima/issues/54#issuecomment-1400796076 )

It's better if I can change /tmp/illa-data to other path in HOME folder, do you have plan to support it?

@Lan-Phan Great works! Your posts help a lot.
We can make the mount location configurable. I will reply to this issue once it is completed.

@Lan-Phan thanks for your issue. It will lead to some active work on the configurable mount path.

@Lan-Phan Now, you can make the mount location configurable with --mount flag of subcommand deploy. It's available in the new version v1.2.8.

upgrade illa to v1.2.8
Run:

colima start --mount-type virtiofs --vm-type=vz
illa deploy --self --mount ~/illa-data

It works like a charm now, thank @naj1n ,  close this.


cmake-conan on Windows: CMake succeeds, but Intellisense doesn't find includes

Brief Issue Summary
I'm trying to use cmake-conan to bootstrap Conan dependencies into my project. When I click the CMake tools Build button, Conan successfully fetches dependencies and I'm able to successfully use target_include_directories and target_link_libraries to add them to my project. But despite using CMake-tools to configure the C++ extension, Intellisense doesn't recognize any of the headers that came from Conan dependencies.
This is a minimal CMake file I tried with, modified slightly from the cmake-conan readme:
cmake_minimum_required(VERSION 3.5)
project(FormatOutput CXX)

# Bootstrap Conan
list(APPEND CMAKE_MODULE_PATH ${CMAKE_BINARY_DIR})
list(APPEND CMAKE_PREFIX_PATH ${CMAKE_BINARY_DIR})
if(NOT EXISTS "${CMAKE_BINARY_DIR}/conan.cmake")
  message(STATUS "Downloading conan.cmake from https://github.com/conan-io/cmake-conan")
  file(DOWNLOAD "https://raw.githubusercontent.com/conan-io/cmake-conan/v0.16.1/conan.cmake"
                "${CMAKE_BINARY_DIR}/conan.cmake"
                EXPECTED_HASH SHA256=396e16d0f5eabdc6a14afddbcfff62a54a7ee75c6da23f32f7a31bc85db23484
                TLS_VERIFY ON)
endif()
include(${CMAKE_BINARY_DIR}/conan.cmake)

# Configure conan
conan_cmake_configure(REQUIRES catch2/2.13.8
                      GENERATORS cmake_find_package)
conan_cmake_autodetect(settings)
conan_cmake_install(PATH_OR_REFERENCE .
                    BUILD missing
                    REMOTE conancenter
                    SETTINGS ${settings})

# Configure project
find_package(Catch2 REQUIRED)

add_executable(main main.cpp)
target_include_directories(main PRIVATE "${Catch2_INCLUDE_DIRS}")

main.cpp
#define CATCH_CONFIG_MAIN
#include <catch2/catch.hpp>

And in my c_cpp_properties.json, "configurationProvider" is set to "ms-vscode.cmake-tools".
The project builds successfully, producing main.exe. But Intellisense doesn't find the headers I've added with target_include_directories:

If I remove the "configurationProvider" and manually add the paths to the headers to c_cpp_properties.json, then Intellisense works. But the include directories are nested deeply in Conan's cache and it's tedious to add them manually.
CMake Tools Diagnostics
{
  "os": "win32",
  "vscodeVersion": "1.63.2",
  "cmtVersion": "1.9.2",
  "configurations": [
    {
      "folder": "<PROJECT-PATH>",
      "cmakeVersion": "3.16.2",
      "configured": true,
      "generator": "Visual Studio 16 2019",
      "usesPresets": false,
      "compilers": {}
    }
  ],
  "cpptoolsIntegration": {
    "isReady": true,
    "hasCodeModel": false,
    "activeBuildType": "Debug",
    "buildTypesSeen": [
      "Debug",
      "Release",
      "MinSizeRel",
      "RelWithDebInfo"
    ],
    "requests": [
      "file:///<PROJECT-PATH>/main.cpp"
    ],
    "responses": [],
    "partialMatches": [],
    "targetCount": 3,
    "executablesCount": 1,
    "librariesCount": 0,
    "targets": [
      {
        "name": "ALL_BUILD",
        "type": "UTILITY"
      },
      {
        "name": "ZERO_CHECK",
        "type": "UTILITY"
      },
      {
        "name": "main",
        "type": "EXECUTABLE"
      }
    ]
  },
  "settings": [
    {
      "communicationMode": "automatic",
      "useCMakePresets": "auto",
      "configureOnOpen": false
    }
  ]
}

Debug Log
(some file paths have been replaced with "<PROJECT-PATH>")
[main] Building folder: conanbug
[main] Saving open files before configure/build
[main] The folder containing the CMake cache is missing. The cache will be regenerated.
[main] Configuring folder: conanbug
[main] Saving open files before configure/build
[driver] Run _refreshExpansions
[driver] Run _refreshExpansions cb
[driver] Start configure
[driver] Runnnig pre-configure checks and steps
[driver] Run _refreshExpansions
[driver] Run _refreshExpansions cb
[cmakefileapi-driver] Configuring using kit
[cmakefileapi-driver] Invoking CMake C:\Program Files\CMake\bin\cmake.EXE with arguments ["--no-warn-unused-cli","-DCMAKE_EXPORT_COMPILE_COMMANDS:BOOL=TRUE","-DCMAKE_BUILD_TYPE:STRING=Debug","-H<PROJECT-PATH>","-B<PROJECT-PATH>/build","-G","Visual Studio 16 2019","-T","host=x64","-A","x64"]
[proc] Executing command: "C:\Program Files\CMake\bin\cmake.EXE" --no-warn-unused-cli -DCMAKE_EXPORT_COMPILE_COMMANDS:BOOL=TRUE -DCMAKE_BUILD_TYPE:STRING=Debug -H<PROJECT-PATH> -B<PROJECT-PATH>/build -G "Visual Studio 16 2019" -T host=x64 -A x64
[cmake] Not searching for unused variables given on the command line.
[cmake] -- Selecting Windows SDK version 10.0.19041.0 to target Windows 10.0.18363.
[cmake] -- The CXX compiler identification is MSVC 19.29.30038.1
[cmake] -- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Professional/VC/Tools/MSVC/14.29.30037/bin/Hostx64/x64/cl.exe
[cmake] -- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Professional/VC/Tools/MSVC/14.29.30037/bin/Hostx64/x64/cl.exe -- works
[cmake] -- Detecting CXX compiler ABI info
[cmake] -- Detecting CXX compiler ABI info - done
[cmake] -- Detecting CXX compile features
[cmake] -- Detecting CXX compile features - done
[cmake] -- Downloading conan.cmake from https://github.com/conan-io/cmake-conan
[cmake] -- Conan: Detected VS runtime: MDd
[cmake] -- Conan: checking conan executable
[cmake] -- Conan: Found program <USER-PROFILE>/AppData/Local/Programs/Python/Python38/Scripts/conan.exe
[cmake] -- Conan: Version found Conan version 1.41.0
[cmake]
[cmake] -- Conan executing: <USER-PROFILE>/AppData/Local/Programs/Python/Python38/Scripts/conan.exe install . --remote conan-center --build missing --settings arch=x86_64 --settings build_type=Debug --settings compiler=Visual Studio --settings compiler.version=16 --settings compiler.runtime=MDd --settings compiler.toolset=v142
[cmake] Configuration:
[cmake] [settings]
[cmake] arch=x86_64
[cmake] arch_build=x86_64
[cmake] build_type=Debug
[cmake] compiler=Visual Studio
[cmake] compiler.runtime=MDd
[cmake] compiler.toolset=v142
[cmake] compiler.version=16
[cmake] os=Windows
[cmake] os_build=Windows
[cmake] [options]
[cmake] [build_requires]
[cmake] [env]
[cmake]
[cmake] conanfile.txt: Installing package
[cmake] Requirements
[cmake]     catch2/2.13.8 from 'conan-center' - Cache
[cmake] Packages
[cmake]     catch2/2.13.8:5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9 - Cache
[cmake]
[cmake] Installing (downloading, building) binaries...
[cmake] catch2/2.13.8: Already installed!
[cmake] conanfile.txt: Generator txt created conanbuildinfo.txt
[cmake] conanfile.txt: Generator cmake_find_package created FindCatch2.cmake
[cmake] conanfile.txt: Aggregating env generators
[cmake] conanfile.txt: Generated conaninfo.txt
[cmake] conanfile.txt: Generated graphinfo
[cmake] -- Conan: Using autogenerated FindCatch2.cmake
[cmake] -- Found Catch2: 2.13.8 (found version "2.13.8")
[cmake] -- Configuring done
[cmake] -- Generating done
[cmake] -- Build files have been written to: <PROJECT-PATH>/build
[cmakefileapi-parser] Read reply folder: <PROJECT-PATH>\build\.cmake\api\v1\reply
[cmakefileapi-parser] Found index files: ["cache-v2-61448bea6800b504c711.json","codemodel-v2-7ef06f8734580af3a789.json","index-2022-02-04T10-03-09-0006.json","target-ALL_BUILD-Debug-26f43c8a8230e42424ad.json","target-ALL_BUILD-MinSizeRel-26f43c8a8230e42424ad.json","target-ALL_BUILD-Release-26f43c8a8230e42424ad.json","target-ALL_BUILD-RelWithDebInfo-26f43c8a8230e42424ad.json","target-main-Debug-7d9ddb8682898b6b5b34.json","target-main-MinSizeRel-58fa3d3c4f5ed765ee2b.json","target-main-Release-0de0241fcdc7b72cf120.json","target-main-RelWithDebInfo-79fa1c9e93cc1dff4ee1.json","target-ZERO_CHECK-Debug-e331f0d2e3ace846d8c7.json","target-ZERO_CHECK-MinSizeRel-e331f0d2e3ace846d8c7.json","target-ZERO_CHECK-Release-e331f0d2e3ace846d8c7.json","target-ZERO_CHECK-RelWithDebInfo-e331f0d2e3ace846d8c7.json"]
[cache] Reading CMake cache file <PROJECT-PATH>/build/CMakeCache.txt
[cache] Parsing CMake cache string
[build] Starting build
[driver] Start build ALL_BUILD
[proc] Executing command: "C:\Program Files\CMake\bin\cmake.EXE" --build <PROJECT-PATH>/build --config Debug --target ALL_BUILD -j 10 --
[build] Microsoft (R) Build Engine version 16.10.2+857e5a733 for .NET Framework
[build] Copyright (C) Microsoft Corporation. All rights reserved.
[build]
[build]   Checking Build System
[build]   Building Custom Rule <PROJECT-PATH>/CMakeLists.txt
[build]   main.cpp
[build]   main.vcxproj -> <PROJECT-PATH>\build\Debug\main.exe
[build]   Building Custom Rule <PROJECT-PATH>/CMakeLists.txt
[cmakefileapi-parser] Read reply folder: <PROJECT-PATH>\build\.cmake\api\v1\reply
[cmakefileapi-parser] Found index files: ["cache-v2-61448bea6800b504c711.json","codemodel-v2-7ef06f8734580af3a789.json","index-2022-02-04T10-03-09-0006.json","target-ALL_BUILD-Debug-26f43c8a8230e42424ad.json","target-ALL_BUILD-MinSizeRel-26f43c8a8230e42424ad.json","target-ALL_BUILD-Release-26f43c8a8230e42424ad.json","target-ALL_BUILD-RelWithDebInfo-26f43c8a8230e42424ad.json","target-main-Debug-7d9ddb8682898b6b5b34.json","target-main-MinSizeRel-58fa3d3c4f5ed765ee2b.json","target-main-Release-0de0241fcdc7b72cf120.json","target-main-RelWithDebInfo-79fa1c9e93cc1dff4ee1.json","target-ZERO_CHECK-Debug-e331f0d2e3ace846d8c7.json","target-ZERO_CHECK-MinSizeRel-e331f0d2e3ace846d8c7.json","target-ZERO_CHECK-Release-e331f0d2e3ace846d8c7.json","target-ZERO_CHECK-RelWithDebInfo-e331f0d2e3ace846d8c7.json"]
[driver] Run _refreshExpansions
[driver] Run _refreshExpansions cb
[build] Build finished with exit code 0
[cache] Reading CMake cache file <PROJECT-PATH>/build/CMakeCache.txt
[cache] Parsing CMake cache string
[extension] [4117] cmake.build finished (returned 0)

The Diagnostics you shared indicate that CMake Tools does not appear to have the IntelliSense information for your file yet. It is unclear what is going on right now. We'll take your example files and investigate.

If I force the build type for the Visual Studio generator, I can get to the same error as Ninja.
    "cmake.configureSettings": {
        "CMAKE_BUILD_TYPE": "Debug"
    }

@elahehrashedi was able to reproduce your error on 1.9.2, but not 1.10.0.  Can you try a pre-release version and let us know if this problem is fixed for you?

Download the extension here: https://github.com/microsoft/vscode-cmake-tools/suites/5157586847/artifacts/155990654
Unzip the VSIX file from the zip file
Run the "Install From VSIX..." command in VS Code and select the VSIX file from step 2.

If it works with 1.10.0, we'll go ahead and close this issue.

I followed the instructions to add the remote, but I still get:
[cmake] catch2/2.13.8: Retrieving from server 'conancenter'
[cmake] catch2/2.13.8: Trying with 'conancenter'...
[cmake] ERROR: HTTPSConnectionPool(host='center.conan.io', port=443): Max retries exceeded with url: /v1/ping (Caused by SSLError(SSLError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:852)'),))
[cmake]
[cmake] Unable to connect to conancenter=https://center.conan.io
[cmake] 1. Make sure the remote is reachable or,
[cmake] 2. Disable it by using conan remote disable,
[cmake] Then try again.
[cmake] CMake Error at build/conan.cmake:631 (message):
[cmake]   Conan install failed='1'

Do I need some kind of authentication set up too?

I figured out how to disable the SSL verification and now the Configure succeeded for me.  But I also have IntelliSense working correctly on 1.9.2.
From my Log Diagnostics:
  "cpptoolsIntegration": {
    "isReady": true,
    "hasCodeModel": true,
    "activeBuildType": "Debug",
    "buildTypesSeen": [
      "Debug",
      "Release",
      "MinSizeRel",
      "RelWithDebInfo"
    ],
    "requests": [
      "file:///d%3A/test/cmake2359/main.cpp"
    ],
    "responses": [
      {
        "uri": "file:///d%3A/test/cmake2359/main.cpp",
        "configuration": {
          "defines": [
            "WIN32",
            "_WINDOWS"
          ],
          "includePath": [
            "%userprofile%/.conan/data/catch2/2.13.8/_/_/package/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9/include"
          ],

Could you try running the "Delete Cache and Reconfigure" command and let me know if the IntelliSense is still broken?  One last thing you could try is installing CMake 3.21 or newer.  It looks like you're using 3.16.

Updating from 3.16 to 3.21 worked! Thanks!


WindowsTerminal crashes on startup

The application has been working fine for month. But starting today, it crashes on startup.
Repro steps: Start the application from the start menu
Event viewer:
Faulting application name: WindowsTerminal.exe, version: 1.8.2105.24004, time stamp: 0x60ac2e94
Faulting module name: ucrtbase.dll, version: 10.0.19041.789, time stamp: 0x2bd748bf
Exception code: 0xc0000409
Fault offset: 0x000000000007286e
Faulting process id: 0x46d0
Faulting application start time: 0x01d753a7675ca192
Faulting application path: C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_1.8.1444.0_x64__8wekyb3d8bbwe\WindowsTerminal.exe
Faulting module path: C:\Windows\System32\ucrtbase.dll
Report Id: 767a5e38-231f-476f-bc54-ff556bf933db
Faulting package full name: Microsoft.WindowsTerminal_1.8.1444.0_x64__8wekyb3d8bbwe
Faulting package-relative application ID: App
Fault bucket<PHONE_NUMBER>965560009, type 5
Event Name: MoBEX
Response: Not available
Cab Id: 0
Problem signature:
P1: Microsoft.WindowsTerminal_1.8.1444.0_x64__8wekyb3d8bbwe
P2: praid:App
P3: 1.8.2105.24004
P4: 60ac2e94
P5: ucrtbase.dll
P6: 10.0.19041.789
P7: 2bd748bf
P8:<PHONE_NUMBER>07286e
P9: c0000409
P10:<PHONE_NUMBER>000007

I think this is a dup of #10248 because I see exactly the same log in my event viewer

I'm thinking that this is probably #10211, but there's only one way to know for sure. If you could repro this with the Feedback Hub, we could get a stack trace that we can make sure is the same thing.
/feedback

Here's the link from the feedback hub:

I have the same problem: https://aka.ms/AAcr6n5

Same thing for me, starting today. Sorry, it seems my account cannot access the feedback hub.
`-

1000
0
2
100
0
0x80000000000000

32520

Application
fxmerica

WindowsTerminal.exe
1.9.2105.24005
60ac3929
ucrtbase.dll
10.0.19041.789
2bd748bf
c0000409
000000000007286e
6a68
01d753db8718df70
C:\Program Files\WindowsApps\Microsoft.WindowsTerminalPreview_1.9.1445.0_x64__8wekyb3d8bbwe\WindowsTerminal.exe
C:\WINDOWS\System32\ucrtbase.dll
719d39c5-91d4-40c9-8a8f-74c842c0e079
Microsoft.WindowsTerminalPreview_1.9.1445.0_x64__8wekyb3d8bbwe
App

`

Okay yep, this one is /dup #10211


revamp/ui: dashboard not empty

[x] Add a  checkbox (nearly invisible, place discretely) that leverages state (empty), when toggled it show the below ui. By default its shows dashboard[empty] ui
[x] https://www.figma.com/file/HWwY6JKy3bnnu1SfSeRL8d/DeCenter-AI?type=design&node-id=2540-40815&mode=design&t=W42e1cCLzyOrCNn3-0

[ ] on click the model

its suppossed to go to /repository/[id]

Done

Rel #83


Rewrite to pure Elm.Html and Bootstrap

The current implementation of salarycalc uses elm-bootstrap. We want to see how the code looks by using vanilla Elm and Bootstrap.

@tad-lispy: didn't we do this already?

Yes, yes we have: https://github.com/niteoweb/salarycalc/pull/33. So this one can be closed.


abort trap 6

Thanks for posting the code; it looks really interesting! When I run any of the examples I get this sort of error:
Starting optimizing WARNING: Logging before InitGoogleLogging() is written to STDERR F1227 00:45:44.057626<PHONE_NUMBER> inner_product_layer.cpp:144] Cannot use GPU in CPU-only Caffe: check mode. *** Check failure stack trace: *** ./1_activate_output.sh: line 57: 14235 Abort trap: 6           python ./act_max.py --act_layer ${act_layer} --opt_layer ${opt_layer} --unit ${unit} --xy ${xy} --n_iters ${n_iters} --start_lr ${lr} --end_lr ${end_lr} --L2 ${L2} --seed ${seed} --clip ${clip} --bound ${bound_file} --debug ${debug} --output_dir ${output_dir} --init_file ${init_file} convert: unable to open image output/fc8_0643_200_0.99_8.0__0.jpg': No such file or directory @ error/blob.c/OpenBlob/2701.
convert: no images defined output/fc8_0643_200_0.99_8.0__0.jpg' @ error/convert.c/ConvertImageCommand/3258. convert: unable to open image output/fc8_0643_200_0.99_8.0__0.jpg': No such file or directory @ error/blob.c/OpenBlob/2701.
unit: 624  xy: 0
n_iters: 200
L2: 0.99
start learning rate: 8.0
end learning rate: 1e-10
seed: 0
opt_layer: fc6
act_layer: fc8
init_file: None
clip: 0
bound: act_range/3x/fc6.txt
debug: 0
output dir: output
net weights: nets/caffenet/bvlc_reference_caffenet.caffemodel
net definition: nets/caffenet/caffenet.prototxt
`
Any idea what could be causing it?
Thanks so much!

Sorry, I figured it out- a settings.py issue. However, I was also wondering - I noticed commented out code that allows you to supply your own unit number. I was wondering what the syntax is for supplying the unit number at the command prompt? Thanks!


Design update feedback

[x] Add identation if ToC goes to more than one level - for example; everything under 'Attributes' should be slightly indented.

[ ] Left-hand navigation resets to top when you use the ToC:

Visit https://deploy-preview-806--prisma2-docs.netlify.app/guides/database-workflows/import-and-export-data/postgresql - the left hand nav has automatiically scrolled down to highlight the current item
Click on anything in the RIGHT hand nav - left-hand nav either disappears or jumps back to the top (extension of @janpio 's issue where left-hand nav disappears):

[ ] This is more of a feature request, but I think we should make right-hand ToC 'sticky' - should follow you when you a) scroll down the page and b) get to the page via an anchor link.

[x] Remove styling from TopBlock element:

[ ] This is more of a feature request, but I think we should make right-hand ToC 'sticky' - should follow you when you a) scroll down the page and b) get to the page via an anchor link.

This is more of a feature request, but I think we should make right-hand ToC 'sticky' - should follow you when you a) scroll down the page and b) get to the page via an anchor link.

+1. This makes helps accepting the reduction of the main content's space.

Clicking the "API Reference" sub-menu brings you to the main nav's sub-menu. This is highly confusing to me.
I was expecting the top bar submenu to be highlighted and have its own left-hand navigation.

I'd really like to unify the headers across all of our properties. I'm ok with replacing this header with a common one. If you click through the other header items from here the headers across the website, docs and blog are all slightly different which I'd like to avoid

Well, the design is batter, but needs much improvement as per the preview link provided above, few one that i found

on the right content field on the docs are not sticky,
the serach-bar icons used are too small and are not appropriate

Thank you for your feebdak @rahul3v - they're both great suggestions. we'll keep improving the docs both before and after launching this.


Limit number of files allowed for the specific field (Task #16681)

This PR shows an example of how you can limit the number of files you are allow to upload a specific field

@acrias this PR should solve travis problems with JSON validation: https://github.com/QoboLtd/cakephp-utils/pull/224


[ACR] Support pull/push OCI blob and manifest in async client

Resolves: https://github.com/Azure/azure-sdk-for-python/issues/23643
The support in sync client has been published in last release. The async implementation was blocked by aiohttp client which not allowed to rewind stream data's position as aiohttp client closes the file after sending. We find a workaround, in this PR, we change the behavior of aiohttp client to do nothing after request sending out.
Since the implementation is pretty hacky, want to hear concerns/thoughts from reviewers.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-containerregistry

/reopen

Close it as there are much gap with current main, the updates will get reviewed in other PRs.


Hamster Kombat Snapshot

The snapshot will happen on September 20, 18:00 UTC
https://bitpinas.com/business/hamster-kombat-snapshot-date-set/

Do I understand correctly that coin mining will be finished on September 20?

no, season 1 will officially end of September 20, they teased a Season 2 airdrop also.

Just in time to finish https://github.com/masterking32/MasterCryptoFarmBot ASAP and focus on multiple bots i guess.

I hope you guys get the framework and some modules done quickly xD

No, Season 1 will officially end on September 20 and at 18:00 they will decide everyones Season 1 airdrop, they teased a Season 2 airdrop also.

18:00  which is region? russion?

18:00 UTC, convert it to your time zone

1 hour to go


Format string typo in leader

https://github.com/BD2KGenomics/toil/blob/6b22036e1bb4227c6d15f2aeda126dfb5cfab716/src/toil/leader.py#L434

Fixed in #1434, I'll link it in the git commit


Error while distributing rewards

Seen on testnet
2023-02-28 10:44:36 💤 Idle (8 peers), best: #7153780 (0xfb24…88ff), finalized #7153777 (0x5f93…433b), ⬇ 9.3kiB/s ⬆ 13.0kiB/s
2023-02-28 10:44:41 💤 Idle (8 peers), best: #7153780 (0xfb24…88ff), finalized #7153778 (0x8674…852b), ⬇ 11.4kiB/s ⬆ 12.6kiB/s
2023-02-28 10:44:42 Transfering: 25 from contract twin  to foundation account
2023-02-28 10:44:42 error while distributing cultivation rewards <wasm:stripped>

Investigation of the case on Testnet
Looking at block #7153778 we can see the failing extrinsic:
https://polkadot.js.org/apps/?rpc=wss%3A%2F%2F/tfchain.test.grid.tf#/explorer/query/7153778

It corresponds to a failing transaction from this account: 5G3t9VaCQf5M2PawGz3iENYMwzpbMkaqZ1nxavnEaH2oPa3K
which state at this block is:

The error Balances.KeepAlive / Transfer/payment would kill account is triggered because of
  consumers: 1
  providers: 1

which returns false here:
https://github.com/paritytech/substrate/blob/cb4f2491b00af7d7817f3a54209c26b20faa1f51/frame/system/src/lib.rs#L1233
	/// True if the account has no outstanding consumer references or more than one provider.
	pub fn can_dec_provider(who: &T::AccountId) -> bool {
		let a = Account::<T>::get(who);
		a.consumers == 0 || a.providers > 1
	}

while transferring in the transfer() function:
https://github.com/paritytech/substrate/blob/cb4f2491b00af7d7817f3a54209c26b20faa1f51/frame/balances/src/lib.rs#L1546

Some documentation about Account data structures
https://docs.substrate.io/reference/account-data-structures/
https://substrate.stackexchange.com/questions/263/what-is-the-meaning-of-the-account-provider-sufficients-and-consumer

But no idea about the account 5G3t9VaCQf5M2PawGz3iENYMwzpbMkaqZ1nxavnEaH2oPa3K, in which context it is billed and how it get in the consumers: 1 / providers: 1 case
Just found it was a billing for contract ID 716 which is a node contract

I think this error is because we try to distribute rewards from contracts that are in grace but the balance of the user's twin is not enough to cover the costs. We should actually not try to distribute rewards when a contract is in grace period.

We distribute rewards inside handle_lock()
And just before we prevent to do it if contract is in grace period
See here:
https://github.com/threefoldtech/tfchain/blob/fc5e1820d554c9c48ee963dff53388d0514c9011/substrate-node/pallets/pallet-smart-contract/src/lib.rs#L1162
So I am not sure it is the right cause...

Hmm okay, might need to dig deeper then

Closing as issue was fixed in https://github.com/threefoldtech/tfchain/pull/631 and haven't reproduce again on the chain.


wip / PoC of hostnet flags

$ make kots
$ bin/kots pull kots-dex-2 --license-file ~/Desktop/cs-kots-dex-2.yaml  --admin-console-host-network --rootdir .

Enter a new password to be used for the Admin Console: ••••••••
  • Pulling upstream ✓
  • Creating base ✓
  • Creating midstream ✓

$ cat app-slug/upstream/admin-console/minio-statefulset.yaml | grep -i host
      hostNetwork: true

Yeah we need to consolidate, will plan on doing that :)
__
Dexter Horthy
Forward Deployed Engineering
Replicated
replicated.com ( https://www.replicated.com )
On Sun, Feb 02, 2020 at 1:02 PM, Andrew Lavery <<EMAIL_ADDRESS>> wrote:

@laverya commented on this pull request.
In cmd/ kots/ cli/ pull. go (
https://github.com/replicatedhq/kots/pull/299#discussion_r373874775 ) :

CreateAppDir: true, - HelmOptions: v.GetStringSlice("set"), -
RewriteImages: v.GetBool("rewrite-images"), + HelmRepoURI:
v.GetString("repo"), + RootDir: ExpandDir(v.GetString("rootdir")), +
Namespace: v.GetString("namespace"), + Downstreams:
v.GetStringSlice("downstream"), + LocalPath:
ExpandDir(v.GetString("local-path")), + LicenseFile:
ExpandDir(v.GetString("license-file")), + ExcludeKotsKinds:
v.GetBool("exclude-kots-kinds"), + ExcludeAdminConsole:
v.GetBool("exclude-admin-console"), + SharedPassword:
v.GetString("shared-password"), + CreateAppDir: true, + HelmOptions:
v.GetStringSlice("set"), + RewriteImages: v.GetBool("rewrite-images"), +
AdminConsoleHostNetwork: v.GetBool("admin-console-host-network"),

why isn't this also called UseHostNetwork like in kotsadmtypes.DeployOptions
?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub (
https://github.com/replicatedhq/kots/pull/299?email_source=notifications&email_token=AA4OZLMCLSNDTUOBILXBMRTRA4YFJA5CNFSM4KOVQDDKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCT5JPIQ#pullrequestreview-351967138
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AA4OZLKMHFOKCOYDLGDXVC3RA4YFJANCNFSM4KOVQDDA
).

Got us to
kotsadm-685547679c-s9qzt           1/1     Running             0          5m43s
kotsadm-api-569b59df-dwpjr         0/1     ContainerCreating   0          5m43s
kotsadm-migrations-1580678167      0/1     Error               6          5m43s
kotsadm-minio-0                    1/1     Running             0          5m43s
kotsadm-operator-fc5bbcd94-7kqd8   1/1     Running             0          5m43s
kotsadm-postgres-0                 1/1     Running             0          5m43s

Migrations crashlooping because there's no DNS names when you use hostnetwork 😞 it can't lookup postgres:
dex@dex-cni-bootstrap-3:~$ kubectl logs -f kotsadm-migrations-1580678167
dial tcp: lookup kotsadm-postgres on <IP_ADDRESS>:53: no such host
Error: dial tcp: lookup kotsadm-postgres on <IP_ADDRESS>:53: no such host
Usage:
  schemahero apply [flags]

API is stuck due to this mount error, and I still haven't resolved the port conflict:
Events:
  Type     Reason           Age                    From                          Message
  ----     ------           ----                   ----                          -------
  Normal   Scheduled        5m16s                  default-scheduler             Successfully assigned default/kotsadm-api-569b59df-dwpjr to dex-cni-bootstrap-3
  Warning  FailedMount      5m15s                  kubelet, dex-cni-bootstrap-3  MountVolume.SetUp failed for volume "kotsadm-api-token-8csd2" : failed to sync secret cache: timed out waiting for the condition
  Warning  NetworkNotReady  15s (x152 over 5m16s)  kubelet, dex-cni-bootstrap-3  network is not ready: runtime network not ready: NetworkReady=false reason:NetworkPluginNotReady message:docker: network plugin is not ready: cni config uninitialized

Migrations crashlooping because there's no DNS names when you use hostnetwork 😞 it can't lookup postgres:

Sounds like we'll need to have parameterized addresses for minio, postgres, the API pod and the kubectl main pod then - because this won't just be an issue for the migrations pod
Also, we'll need to handle the port forwarding bit somehow - as it is right now, I can't see that working properly. Providing nodeip:hostport as the link and not doing any forwarding at all might be the easiest solution

@laverya @marccampbell gonna close this one for now


select which T1 to use when multiple _T1w.nii.gz are present

For instance if you acquire in the same session a brain T1w which outputs ..._T1w.nii.gz and ..._rec-norm_T1w.nii.gz (scanner normalization) and also a cervical spinal cord T1w named ....loc-spine_T1w.nii.gz, fmriprep seems to try making an average of all these.
Is there an option to select from a BIDS anat directory, the specific T1w to select for anatomical processing using a glob or else?
Thanks!

Ok thanks!
I guess it would be useful to have options --anat_bids_filter and --func_bids_filter to pass to pybids in fmriprep and smriprep, and maybe deprecate the -T option.

Related - #31

Also interested in this issue! i.e. where there are multiple versions of the same T1w scan due to different gradient correction methods applied.
Happy to help as well. In our case, the file names look like
sub-xx_acq-yy_rec-DIS3D_run-01_T1w
sub-xx_acq-yy_rec-DIS2D_run-01_T1w
so maybe just parsing for the tag and adding a command line argument --t1w_select to specify?

I've been watching the issues/PRs about this since asking about it on Neurostars a long time ago, and was currently going to stick with simply adding the anatomicals that I don't want to use to the .bidsignore (as opposed to removing them entirely), in line with the suggestion by @emdupre on the fmriprep PR. I think this will work generally (but haven't tested it yet), but if this feature is close to being implemented in a new release, I might wait on the trouble of doing that... Any updates?

Adding things to .bidsignore only affects the validator, not pybids or fMRIPrep.
I think the selection PRs are nearly there... I can try to have a look this week.

Adding things to .bidsignore only affects the validator, not pybids or fMRIPrep.

Sorry @effigies: Is that an expected situation ? I didn't think so based on discussion in https://github.com/bids-standard/bids-specification/issues/131 and https://github.com/bids-standard/pybids/issues/378. Or is this something particular to *MRIPrep ?

It's what I expect... I can read through those threads to see if there was an alternative consensus, though. Not likely to be tonight.

There doesn't seem to be a clear consensus that came out, but I can't pin down a population vector of any magnitude.
I think one problem we'll face in fMRIPrep is that "ignore" is a blunt instrument and it's encoded in the dataset, not in the execution inputs. I may have two perfectly BIDS-valid T1w images but only want to pass one to fMRIPrep. .bidsignore feels like the wrong way to handle that. Second, ignore in the BIDS-validator context means "do not error on", but in pybids it sounds like we want it to mean "do not index". BUT, we use BIDSValidator errors to mean "do not index", so suppressing errors might intuitively mean "index in spite of errors".
The conceptual ambiguity is probably part of why we haven't moved toward making .bidsignore official for anything but bids-validator. And I think this input form is much more clear. I will see if we can push on this and aim for inclusion in fMRIPrep 20.1.

Has the --bids-filters flag resolved this issue indirectly?

yes!


Farsi and RTL support

dear volkan
hope you well and be out of the politic and military games.
ive provided a rtl and farsi version at the moment. please check it and help me about.
thanks.

Hi @misafer,
Thanks for the nice work. It says there are some conflicts so i can't merge it yet. I'll try to merge it manually this weekend.
If i understand it right, i should include rtl.css when a RTL language is selected right? Otherwise will it affect non-rtl language looks?

hi again, yes you're right. not just rtl.css but whole of SlickGrid at "Serene\Serene.Web\Scripts\SlickGrid".
i suggest to do like this
1- add a new field to languages table names "RTL" type of bit or bool
2- if this flag checks for current language then use rtl.css and second (RTL) version of slickgrid
god bless you

Dear Volkan
hi again. if you are out of time this week, i can help you with the process of rtl-ltr selection on language changes. may i ? and of course can get latest version of project and create a new pull request without these conflicts. is it ok ?
Thanks so

No, thanks i'll try to merge manually today, if any problems i'll let you know.

SlickGrid seems like has many changes, did you implement something yourself or used changes from geriamir branch?

i've change them at my own.

I applied your changes but especially in slickgrid got a non-satisfying look,

Are you sure this works fine on your side?

Headers looks messed up

@misafer i'm stuck atm, do you have a better view?

Yes. My version is OK. Where can I see your code?

see rtl branch in Serene

hi dear. you missed this :
body{direction: rtl;}

put it at the head of rtl.css. everything gonna be ok.

I tried that too but ok i'll have another look.

Hello Dear Volkan
hows going on. is it ok with the RTL css ?
im online at skype with my<EMAIL_ADDRESS>address.
after it i want to translate the serene GitBook too. help me with howtos.
Thanks

Hi Misafer, I already integrated your changes to Serene. So just adding farsi language should be enough. For GitBook, i may get a copy of Serenity guide and give you write access so you can use GitBook to edit.


Feature/servers

In configuration.sql:

Added a generic configuration table -- this can probably be removed once we have a configuration file
Added a servers table for configuring the Servers block.  The table contains a slug and a priority.  The Servers block will contain one item for each slug, and will choose the one with the highest priority (see tests for examples).
Added openapi_server_objects which returns JSON based on the servers table
Added set_server_from_configuration, which updates the servers table based on the configuration values

In main.sql:

Added callable_root function which can be directly called by PostgREST
Added x-software block, for specifying software versions

In fixtures.sql:

For testing, added a pre_config function that sets the pgrst.server_host and pgrst.server_port (and then called it)

Tests:

Added some tests for servers and main

Hi all!  Here's my PR for the servers/x-software stuff.  I'm expecting to be told I've done a number of things wrong.  I probably won't have a chance to look at the corrections for another week.
Thanks!

Interesting!  The tests are failing.  They worked on my local (I'm testing with a pretty recent Postgres, though).  I note the following error which wasn't present on my local.
The postmaster has commanded this server process to roll back the current transaction and exit, because another server process exited abnormally and possibly corrupted shared memory.

I'm not familiar enough with nix to figure out what's going on.
Question: is anyone else able to replicate the test failures on their local?

@wayland Yes, I'm able to replicate the errors locally. (I do realize the CI is not being informative, will fix)
The error happens on pg 14 and 13, the output varies there:
diff -U3 /home/stevechavez/Projects/postgrest-openapi/test/expected/servers.out /home/stevechavez/Projects/postgrest-openapi/output/results/servers.out
--- /home/stevechavez/Projects/postgrest-openapi/test/expected/servers.out	2023-11-04 18:19:12.619939571 -0500
+++ /home/stevechavez/Projects/postgrest-openapi/output/results/servers.out	2023-11-04 18:21:30.704413555 -0500
@@ -53,16 +53,16 @@
 -----------------------------------------------------------------------------
  [                                                                          +
      {                                                                      +
-         "url": "https://www.example.com/api/",                             +
-         "description": "Overriding URL"                                    +
-     },                                                                     +
-     {                                                                      +
          "url": "https://www.example.com/otherapi/?s={s}&browser={browser}",+
          "variables": {                                                     +
              "s": "worms",                                                  +
              "browser": "none"                                              +
          },                                                                 +
          "description": "Additional URL"                                    +
+     },                                                                     +
+     {                                                                      +
+         "url": "https://www.example.com/api/",                             +
+         "description": "Overriding URL"                                    +
      }                                                                      +
  ]
 (1 row)

@wayland Just merged https://github.com/PostgREST/postgrest-openapi/pull/23. Once you rebase you should be able to see the diff with the failures.

Thanks!  I've added the ORDER BY, so it should work now.  Looks like the tests pass now.
HTH,

Thanks for your feedback.  I've done some edits in line with what you've suggested.  Let me know if there are further issues.


[BUG] MINOR: methods not being honored from ferox-config.toml file but working from CLI

Describe the bug
The methods option in ferox-config.toml is not being honored. I tried to double-check this wasn't pilot error, but I don't see much room for me to be screwing this one up as I copied it directly from the example config. If I am screwing this up somehow, I apologize in advance :>
To Reproduce
Steps to reproduce the behavior:

Use a build from the main branch (I build from source but I'm pretty sure the binary release will behave the same- I can test if you want); I used cargo build --release, which I believe is at least correct enough :>
Create a ferox-config.toml like the one below
Create a simple wordlist testing.wl
Run an httpd that makes it easy for you to see the requests being made by feroxbuster (e.g. python2 -mSimpleHTTPServer 9333)
Run feroxbuster -u http://<IP_ADDRESS>:9333
Observe that there are no POST requests made in one of several ways; easiest way is to look at your testing HTTP server output, which logs the verbs/methods

To demonstrate that there is not some other subtle issue at play, using --methods POST on the CLI with this same configuration/invocation does correctly emit POST requests
Wordlist
$ cat > testing.wl << 'EOF'
derp
api
v1
lol

Contents of ferox-config.toml
None of the following will produce POST requests, they all result in GET only:
wordlist = "testing.wl"
debug_log = "debug.log"
user_agent = "Sentinel User-Agent to make it clear the correct config was loaded hehe"
methods = ["POST"]

wordlist = "testing.wl"
debug_log = "debug.log"
user_agent = "Sentinel User-Agent to make it clear the correct config was loaded hehe"
methods = ["POST", "GET"]

Expected behavior

With both of the above configurations, POST requests should be made.  If I have a chance I'll read through and maybe send a PR if it's simple enough, I'm nowhere near fluent in Rust, though I'm guessing for a behavior/bug like this, it's akin to a copy/pasta type issue and may be easy to track down
Also, to confirm I wasn't a dummy and somehow using the wrong configuration file, I put that user-agent sentinel value you in there, to match with the output when feroxbuster boots:
10:03:59 › feroxbuster --url http://<IP_ADDRESS>:9333 --methods POST

 ___  ___  __   __     __      __         __   ___
|__  |__  |__) |__) | /  `    /  \ \_/ | |  \ |__
|    |___ |  \ |  \ | \__,    \__/ / \ | |__/ |___
by Ben "epi" Risher 🤓                 ver: 2.7.1
───────────────────────────┬──────────────────────
 🎯  Target Url            │ http://<IP_ADDRESS>:9333
 🚀  Threads               │ 50
 📖  Wordlist              │ testing.wl
 👌  Status Codes          │ [200, 204, 301, 302, 307, 308, 401, 403, 405, 500]
 💥  Timeout (secs)        │ 7
 🦡  User-Agent            │ Sentinel User-Agent to make it clear the correct config was loaded hehe
 💉  Config File           │ /home/user/projects/feroxbuster/ferox-config.toml
 🪲  Debugging Log         │ debug.log
 🏁  HTTP methods          │ [POST]
 🔃  Recursion Depth       │ 4
───────────────────────────┴──────────────────────
 🏁  Press [ENTER] to use the Scan Management Menu™
──────────────────────────────────────────────────
[####################] - 0s         6/6       0s      found:0       errors:0
[####################] - 0s         6/6       481/s   http://<IP_ADDRESS>:9333

As always, thanks for the great work you've done on this project, and continue to do. Killer stuff

Nice catch!
I just pushed a change that should fix this. Lmk if it's resolved on your end.

Resolving, looks good- I now see the expected behavior when ferox-config.toml contains method = ["POST"] or ["POST", "GET"]
Thanks!

Anytime, glad to hear it!


Update ci.Dockerfile

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

/test kyma-integration

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

Test name
Commit
Details
Rerun command

prow/kyma/kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test kyma-integration

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

/test kyma-integration

/test kyma-integration

/test prow/kyma/kyma-integration

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

Test name
Commit
Details
Rerun command

prow/kyma/kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow/kyma/kyma-integration

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

/test prow/kyma/kyma-integration

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

Test name
Commit
Details
Rerun command

prow/kyma/kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow/kyma/kyma-integration

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

/test prow/kyma/kyma-integration

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

Test name
Commit
Details
Rerun command

prow/kyma/kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow/kyma/kyma-integration

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

/test prow/kyma/kyma-integration

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

Test name
Commit
Details
Rerun command

prow/kyma/kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow/kyma/kyma-integration

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

/test prow/kyma/kyma-integration

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

Test name
Commit
Details
Rerun command

prow/kyma/kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow/kyma/kyma-integration

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

/test prow/kyma/kyma-integration

/test prow/kyma/kyma-integration

/test prow/kyma/kyma-integration

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

Test name
Commit
Details
Rerun command

prow/kyma/kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow/kyma/kyma-integration

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

@suleymanakbas91: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/kyma/kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow/kyma/kyma-integration

prow-kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow-kyma-integration

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

/test prow-kyma-integration

/test prow/kyma/kyma-integration

/test prow/kyma/kyma-integration

@suleymanakbas91: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow-kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow-kyma-integration

prow/kyma/kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow/kyma/kyma-integration

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

/test prow-kyma-kyma-integration

/test prow-kyma-kyma-integration

@suleymanakbas91: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/kyma/kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow/kyma/kyma-integration

prow-kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow-kyma-integration

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

/test prow-kyma-kyma-integration

@suleymanakbas91: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/kyma/kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow/kyma/kyma-integration

prow-kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow-kyma-integration

prow-kyma-kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow-kyma-kyma-integration

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

/test prow-kyma-kyma-integration

@suleymanakbas91: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/kyma/kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow/kyma/kyma-integration

prow-kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow-kyma-integration

prow-kyma-kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow-kyma-kyma-integration

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

/test prow-kyma-kyma-integration

@suleymanakbas91: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/kyma/kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow/kyma/kyma-integration

prow-kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow-kyma-integration

prow-kyma-kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow-kyma-kyma-integration

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

/test prow-kyma-kyma-integration

/test prow-kyma-kyma-integration

/test prow-kyma-kyma-integration

@suleymanakbas91: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/kyma/kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow/kyma/kyma-integration

prow-kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow-kyma-integration

prow-kyma-kyma-integration
50ebe110ce886e93bfe5ea79da47c88a2e785e60
link
/test prow-kyma-kyma-integration

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

/test prow-kyma-kyma-integration


test(wip): removing playwright changes for vite-puppeteer setup

@HunnySajid is it still WIP like the title says?

@HunnySajid is it still WIP like the title says?

I think we can merge this one. I need to discuss changes with @TaniyaBhadauria and have to make a couple of changes later on when SignData.json gets update. Rest for test cases, it is done


False Positive Reason

I know that Windows falsely detects w64devkit as malware, but I was curious of the reason why. Usually a false positive is 1-4 detection but in VT it's 20. Is it just the way the binary is compiled because I had a similar problem before with mingw before?

Maybe 7zip sfx are commonly used to distribute malware, so it's being flagged. Just a guess.

I know that Windows falsely detects w64devkit as malware, but I was curious of the reason why. Usually a false positive is 1-4 detection but in VT it's 20. Is it just the way the binary is compiled because I had a similar problem before with mingw before?

https://www.virustotal.com/gui/url/4bff821231567fbe7765ee563a7953a0380279e83d87e0816baa37d298c977b6

I think original author means the VT analysis of the actual file: https://www.virustotal.com/gui/file/cea23fc56a5e61457492113a8377c8ab0c42ed82303fcc454ccd1963a46f8ce1


Enable retry for HTTP 301 with Retry-After header

Codecov Report

Merging #96 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #96      +/-   ##
==========================================
+ Coverage   87.96%   88.01%   +0.04%
==========================================
  Files          28       28
  Lines         964      968       +4
==========================================
+ Hits          848      852       +4
  Misses        116      116

Impacted Files
Coverage Δ

hca/util/__init__.py
91.56% <100%> (+0.13%)
: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 aa5c7b6...649a9de. Read the comment docs.


Update about.astro

👷 Deploy Preview for famous-marshmallow-cfbd61 processing.

Name
Link

🔨 Latest commit
fc669178c55903e3a6f6ef4fbdb7e73cf097f934

🔍 Latest deploy log
https://app.netlify.com/sites/famous-marshmallow-cfbd61/deploys/64e40b660e4bc60008a6212e


Version 3.3.0 did not exist

idk i've never used gradle. someone else contributed that to the readme. i've never heard of anyone having issues of using it with maven and (afaik) gradle should be compatible with maven, right?
by the looks of it, whoever made the contribution did not add the repository to the gradle instructions. you need to add https://dl.bintray.com/deanveloper/SkullCreator as a repository, however that's done in gradle

nvm i'm dumb and see that it's added in the instructions lol. i was confused by the mavenCentral() thing. can you show me what your build.gradle looks like?

nvm i'm dumb x2...
yea the newest version is 3.0.0... not sure why the build.gradle says 3.3.0. likely just a typo from the dude who added that which i didn't catch.

That repositoriy doesn't have 3.3.0 version. Please edit 3.3.0 to 3.0.0. I already tested that only 3.0.0 is worked.


[server] Fix hanging isolated ingestion server issue caused by race condition in status reporting

[server] Fix hanging isolated ingestion server issue caused by race condition in status reporting
This PR fixes the CI build hanging issue in TestBatchForIngestionIsolation#cleanup() method.
The issue is caused by IsolatedIngestionServer failed to serialize COMPLETE report from forked process to main process and cause ingestion task to fail and test time out. (Most likely happening in testIncrementalPushWritesToRealTimeTopicWithPolicy() as it sets AMP=2 and increase the likelyhood). The root cause of the serailization failure is: ready-to-serve check and COMPLETE report of a same subPartition can actually be invoked from (1) Drainer thread who is processing the records of this subPartition (2) SIT thread checking long running task state (this was introduced long before to avoid the edge case that no data in RT topic and (1) will never be invoked to complete the ingestion). The race condition in StatusReportAdapter counter will lead to one OffsetRecord in user partition being null during serailization and failed.
This PR fixes the issue by introducing a AtomicBoolean state for each subPartition for each status. The status recording counter will only be incremented when the state is flipped from False -> True. Also, this PR fixes another hidden ConcurrentModificationException issue caused by serailization of the OffsetRecord by initialzing PartitionState with VeniceConcurrentHashMap for map field. This fix also simplies the OffsetRecord serailization logic during COMPLETE reporting.
How was this PR tested?
Internal CI running.
Run the specific test 100 times on Azure VM with JDK 8 setup and invoke setup() and cleanup() before/after each test.
1 run failed due to known Router resource unavailable issue and this failure won't block shutdown.
Does this PR introduce any user-facing changes?

[x] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

Thanks a lot Jialin! Was there supposed to be a fix to the map fields in PartitionState to make them threadsafe also? Or is this change sufficient?

Oh nevermind... I missed it on my first reading but I see it now. Thanks!


Mb 13777 update weightticket handler office nt

Jira ticket for this change
Summary
Adds error handling for UpdateWeightTicket handler,and unhappy path tests
Setup to Run Your Code

💻 You will need to use one terminal to test this locally.
Terminal 1
go test ./pkg/handlers/ghcapi --testify.m TestUpdateWeightTicketHandler -v

Additional steps

Verification Steps for Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

Backend

[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 0d6fd212ab3da00934cccc207a8799f0fa57e835


proposal: spec: clarify that map iteration always yields the latest version of an entry

Proposal Details
Consider the following program (https://go.dev/play/p/wd1Ge2PIyAK):
package main

import "fmt"

func main() {
	m := map[string]string{
		"a": "original",
		"b": "original",
	}
	for k, v := range m {
		fmt.Println(k, v)
		m["a"] = "new"
	}
}

Map iteration order unspecified, so if "a" is produced first it will print:
a original
b original

If "b" is produced first, the spec does not specify whether this must print
b original
a new

Or if
b original
a original

is acceptable. In other words, is iteration required to produce the latest version of an entry, or is a stale version OK?
As far as I know, the gc map implementation has always produced the latest version of an entry. In fact, when implementing #54766 I fixed bugs I introduced that produced stale values under the assumption that the spec required it.
The spec does specify "If a map entry that has not yet been reached is removed during iteration, the corresponding iteration value will not be produced." In my opinion it would be odd to require deletion be tracked precisely, but not changes to values.
Thus I propose that the map portion of https://go.dev/ref/spec#For_range be amended to add "Iteration always produces the latest value of a map entry."
cc @griesemer @randall77

Related Issues

spec: clarify deletion of map entries during iteration #8558 (closed)
spec: clarify that modifying map elements in place while iterating is legal #9926 (closed)
spec: clarify range over maps will produce every non-deleted entry avaiable when it starts #35239

Related Documentation

Go 1 Release Notes > Changes to the language > Iterating in maps

Related Discussions

A pedantic question about updating a map during iteration

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

Same as/for slices?


Update .gitignore

Hello, if Geiser is built in its root dir, there appear several files
that are not git-ignored, so this commit will fix it.
Comments to the removed lines: there is no need to ignore every
/.../Makefile.in file as just Makefile.in handles all of them.  Also
I removed /elisp-comp as I don't know what it is I don't see any
sign of this file.  What is it?  Should it be stayed in .gitignore?

merged! i think elisp-comp is a leftover from times where we used a different way of compiling elisp files.  we can always add it back if the file reappears.

Thanks!


Plugin update window doesn't show changelog

When new plugin updates are available a window pops up and shows the new plugin versions available and a dropdown menu to see the changes with an arrowhead to display the menu. When clicking the arrowhead the window enlarges a bit (to make room for the text describing the changelog, I suppose) but no text is displayed.
This is an example for a popup window informing about 3 new plugin updates. The 3 arrowheads have been clicked to show the changelogs, there is extra white space at the bottom of the window to fit the changelog text, but no text shows up.

I can confirm that this used to happen, but I’ve seen the details on the last two plug-in releases. Anyone else?

The last plugin update did that here (Text Manipulation I think). The arrow is visible, but clicking it shows nothing (unless you count weird animation behaviors as "something" ;-)).

I made this, so let me look into this.

Wait, seems to be working here right now...
Anyone else still having problems, or can I close?

Not here. I've had some "no plugin changelog" a few releases ago, but not recently.


[FaceAPI] Mark the user-defined name as mandatory when creating FaceL…

…ists
This should fix issue reported here: https://github.com/Azure/azure-sdk-for-net/issues/4594
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Can one of the admins verify this patch?

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

Hi @lebronJ ,
Can you help review this change?

@TRF258 This is not a right approach to mark the field as mandatory.
Pls check how Python SDK changes will look like:
https://github.com/Azure/azure-sdk-for-python/pull/7046/files
I have to do some checks and will get back to you with a suggestion of proper solution.

Hi @zikalino , Were you able to find any alternative for this change?
Thanks,
Tom

Hi @zikalino ,
Any update on this PR?
Thanks,
Tom

/azp run

/openapi pipeline


chore(docs): feedback tutorial

This looks great. If we're only saying "Weave" - we should only say "Models" for consistency. Other than that, this is ready to go in my opinion


Fail verbosely when no password can be read

On running reload_mixed </dev/null, I get the following response:
Creating strato_smd_1           ... done
Creating strato_nginx_1         ... done
Waiting for STRATO to rise and shine...

Please enter a password:
error: unable to read password, stdin might be closed
Set the PASSWORD environment variable in automated environments


Updated to check that $ref is a string

This is an issue when processing the OpenAPI/Swagger spec, where $ref is an object.
Example on line 315 of the OAPI spec:
"$ref": {
      "type": "string"
},

From what I understand, the spec is a valid document, but beaks the current resolver logic.

@bighappyface - what do you think about this one?

@calcinai thanks for the contribution. To be clear, will this adjustment only allow refs that are strings in this library, or does the check pass through object refs to support the example you provided from OAPI?

From what I understood, $ref is always a string if it's actually a reference.  The issue here is that the current logic eagerly treats any object key $ref as a reference and tries to follow it, which in this case will fail.
The OAPI issue is that in the document it's actually defining what $ref can be.
Apologies if I didn't answer your question, can you elaborate if not?

Just to add - I deliberately made the smallest change possible to correct this to try and have the smallest impact.

I think we are good here. +1