cli commands (add, update) remove empty lines from package.json

From @gkalpak on May 18, 2018 11:11
Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Area
- [x] devkit
- [x] schematics

Versions
All
Repro steps
Some commands that modify package.json (e.g. ng update ... or ng add @angulatr/pwa (probably other ng add commands as well), remove an extra empty lines at the end of the file.
Desired functionality
Modifications in package.json should preserve empty lines. (Ideally, all whitespace should be preserved.)
Having an empty line at the end of each file is a common convention and removing such lines creates unnecessary noise in diffs. (Also note that other tools (e.g. yarn install) tend to add them back, which creates more unnecessary noise :grin:)
Copied from original issue: angular/devkit#940

Duplicate of #11744


Nimbus update

Pull Request
What problem does this PR solve?
Update the opensoruced codes for Nimbus: Secure and Efficient Two-Party Inference for Transformers.
Possible side effects?
No side effects.

I have read the CLA Document and I hereby sign the CLA


Moving timestamp components to global components

Signed-off-by: Matan Schatzman<EMAIL_ADDRESS>
📝 Description

Add a brief description

🎥 Demo

Please add a video or an image of the behavior/changes

/retest

/lgtm


Welcome page

Introduction
Modify contents on the first page users see as they arrive on the website

[ ] Add logos from the different organizations of the MobiliseYourCity partnership  @TTalex
(-> it's possible to change the format of the provided logos keeping their order and the groups. inputs : https://cloud.fabmob.io/s/npBq9SFMFksoWYc)
[ ] Add a world map with MYC partners countries and cities @maloufabmob
[ ] Add description of the MobiliseYourCity partnership @maloufabmob

I added a static map for now.
I also redesigned the page in the process.


Add MIQ

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[x] Filename in the lowercase format
[x] Image format: png
[x] Image size: 256 by 256 px

What is contract address for this logo ?

Also, you can use these graphics to say that your ERC20 token fully compatible with Trust Wallet (https://trustwallet.com/logo-assets). Tokens will show up automatically once the user receives them.
Feel free to share this on social media and tag Trust Wallet:

Twitter: @trustwalletapp

Facebook: @trustwalletapp

Contract address = 0x704bddaa18f3b4eef33cdfec5e80daf5847c7d41

Added


Delete unnecessary spaces

When copying from the web page, there is an unnecessary space after the code.

@changeworld : Thanks for your contribution! The author, @abshamsft, has been notified to review your proposed change.

#sign-off


Using next() breaks with JQuery validate. Works perfectly with nextAl…

We froze the current jQuery-Text-Counter on our site with these changes. The counter stops working when a JQuery validate error is thrown because it places the error element right after the input. Switching to nextAll has your plugin work even after the validator error element is added.

Thanks for the pull request to resolve this issue - it has been merged with the project.


Gzip ledger assets

Store the ledger assets as Gzipped JSON (instead of just JSON)
Based off benchmark:
json       | 788 ms   | 1208 bytes
json-gzip  | 12112 ms | 405 bytes
gob        | 3956 ms  | 1391 bytes
gob-gzip   | 17545 ms | 602 bytes

tested on a CompositeTraintuple
time column: serialize/deserialize 50k times
weight column: weight of a serialized CompositeTraintuple
benchmark source : https://github.com/SubstraFoundation/substra-chaincode/commit/e37e3a7e078d18d8bc1bf7df5d552770d6226233

I haven't tested with make test because tests currently don't pass on master on my machine :'(

@AurelienGasser @thibaultrobert keep in mind that this won't work if we switch the storage to CouchDB. So we should maybe add a reminder comment or make this feature optional.

@inalgnu Good point! I looked into it and I found this:

If you have a mix of JSON and binary data values, you can still use CouchDB, however the binary values can only be queried based on key, key range, and composite key queries.

So I think we should be good.

@thibaultrobert @inalgnu I made some changes, please let me know what you think!

Gzipping can be turned on or off + add tests
Introduce InitSettings and LedgerSettings to configure the chaincode at instantiation time + add default values for settings + add tests

There's a bug, the substra-tests don't pass.
Error with registernode:
400 Client Error: Bad Request for url: http://substra-backend.node-1.com/data_manager/: error while ungzipping data for key MyOrg2MSP gzip: invalid header

@inalgnu @thibaultrobert  This is ready for another review.
I tested with substra-tests: it's all :heavy_check_mark:

Note: I asked a question in the HLF chat:

In our chaincode, we implemented two different ways to serialize data to be added to the ledger: json and gzipped json. We can control whether or not to gzip assets using a global setting, which is essentially a constant; it doesn't change during the lifecycle of the chaincode. Is there a way to pass arbitrary settings to the chaincode - at install-time for instance? The Init method isn't a good fit because it is not run on every peer. The solution we have now is to hardcode the setting, but that isn't very elegant because the only way to modify the setting is to modify the chaincode source code.

The only response (so far) isn't great:

this is just a guess, but have you tried loading a file from the peer's filesystem? if that works you could just put it in a docker volume and read the file at runtime

@inalgnu Would you min giving this another review?

Hello @AurelienGasser,
I'm wondering if we are not missing something here, with HLF we have the ability to instantiate multiple chaincodes accessing the same state DB, and I see a limitation for that with this feature. I think the way we instantiate the chaincode should not impact the way the assets are stored in the state DB.
Shouldn't we use the state DB to store weather the asset is gzipped or not? this should be the case for the entire state DB lifetime. Also, maybe we could add a correspondence table (asset_type | gzipped (true/false)) and rely on this for any chaincode, or an asset struct wrapper {asset asset, gzipped bool}.
Maybe we could take some time to discuss that this afternoon.
Otherwise, If we need this to have a successful load test, I'm fine with merging and take the time to rethink the way we design asset compression afterward.

@inalgnu we could do the following:

at chaincode instantiation time, pass a parameter to Init describing whether or not to gzip assets. In the peer executing Init, check the state DB for an existing setting already saved:

if there's already a setting in the state DB, use that setting and ignore the setting passed to Init
if there's no setting in the state DB, insert the setting passed to the Init function into the state DB

In all the peers, on each query/invoke, if the settings haven't been loaded from the state DB, load them.

But I'm not sure having the ability to choose the gzip behavior is desirable: why would we want to turn it on in some ledgers and off in some others? Or on for some assets and off for some others?

Closing until we validate we have a need for this feature


Update internal Workbench version number

Bump mWorkbenchVersionCode to 5:
   var mWorkbenchVersionCode = 5

On this line:
https://github.com/evothings/evothings-studio/blob/evothings2/hyper/server/hyper-server.js#L45

Done.
Updated in this commit: https://github.com/mikaelkindborg/evothings-studio/commit/aa5c01b8c0fdc6533c6f4221787d81bc85729de6
Also incremented var mProtocolVersion:
var mProtocolVersion = 4


Add slide to download energy flows CSV

Downloads a CSV containing the inputs and outputs for every carrier, for each node in the graph.
Needs a couple of translations added before merging, please and thank you.

config/locales/nl_slides.yml: data_export_energy_flows (line 26)
db/migrate/20171012131529_add_energy_flow_csv_slide.rb: (lines 22-27)

Closes #2503

please and thank you


Multiple associations in factories

Hi,
i have a question that i think may extend issue https://github.com/thoughtbot/factory_girl/issues/382. I have following code:
FactoryGirl.define do
  factory :issue do
    association :project
    summary "MyString"
    description "MyText"
    association :priority
    association :issue_type
    association :issue_state
    association :created_by, factory: :user
    assigned_to { created_by } # as you may see - i've used trick from #382 :)
  end
end

FactoryGirl.define do
  factory :issue_state do
    name "MyString"
    association :project
  end
end

FactoryGirl.define do
  factory :issue_type do
    name "MyString"
    association :project
  end
end

FactoryGirl.define do
  factory :project do
    name "name"
    slug "megaslug"
    association :organization
  end
end

FactoryGirl.define do
  sequence(:name)    { |u| "test#{u}" }
  sequence(:slug)    { |u| "test#{u}" }

  factory :organization do
    name
    slug
  end
end

I have following problem:
1 - each time i run my test suite - i have a lot of db records. But in my case - i'd like to have each of generated issue_type and issue_state both referencing to the single project. Is it possible?
2 - currently on each run i see error (for project factory as i understand) Validation failed: Slug has already been taken - because of each project in scope of his organization should have unique slug. But now - there are problem :(  I've tried to create sequence slug for projects factory, but i saw another error
Sequence already registered: slug (FactoryGirl::DuplicateDefinitionError)

How can i solve this issues?

1 - I think you're going to need to create the project outside of the factory and then pass it in via the options hash like so:
project = FactoryGirl.create(:project)
issue_type = FactoryGirl.create(:issue_type, project: project)
issue_state = FactoryGirl.create(:issue_state, project: project)

Of course, you would probably want to extract that out to a helper method.
2 - If you define your sequences inline, they will be defined locally to the factory they were defined in. So:
FactoryGirl.define do
  factory :organization do
    sequence(:slug) { |n| "organization_#{n}" }
  end

  factory :project do
    sequence(:slug) { |n| "project_#{n}" }
  end
end

You can check out the docs for more info.
Let me know if this helps.

Hi @alec-c4, @ajm188 gave a perfect answer for number 2.
I have a solution that may be helpful for your first issue. I've used this in a few scenarios that are similar to your case:
factory :issue_type do
  name "MyString"
  after :build do |model|
    if model.project.blank?
      model.project = Project.first || build(:project)
    end
  end
end

This allows you to default to the first project but still allows you to override it in the cases you want a custom one.
Hope that helps!


Add other platforms

Trying to add cryptography is hard :(

So sorry! meant to try this against my repo first.

Hi @troykelly, I'm not super familiar with GitHub actions. Appreciate any improvements you can make, thanks!

@zaxbux I'm not an expert by any measure, sadly, but I needed a multi-platform build (well, for RPi at least). It's working, I just can't work out why the caching doesn't appear to be working reliably. I will send a PR over once I know what's going on there.


Mistake

The text says the function returns EINVAL but it returns nothing.
[Enter feedback here]

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

ID: e932ee57-662b-dd17-a113-a70f8cb8f86f
Version Independent ID: 79dd5ab5-fdf5-f6c1-c266-7668bd523da3
Content: _splitpath, _wsplitpath
Content Source: docs/c-runtime-library/reference/splitpath-wsplitpath.md
Product: visual-cpp
Technology: cpp-ucrt
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

Yup.  I think that is a generic remark that doesn't apply here.  I presume it sets errno but the function simply returns if continuation is allowed.

Fixed. Will be live tomorrow. Thanks for flagging that.


Use formatted float in dataframe_to_wiki

What does the code in this PR do / what does it improve?
In the previous implementation, straxen.dataframe_to_wiki will give you '0.5' when float_digits=2, which gives the output different lengths.
Previous:
import pandas as pd
import straxen

df = pd.DataFrame({'x': [0.51], 'y': [0.50]})
print(straxen.dataframe_to_wiki(df, float_digits=2))

gives output:
^ Awesome table ^^
^ x ^ y ^
| 0.51 | 0.5 |

Can you briefly describe how it works?
Can you give a minimal working example (or illustrate with a figure)?
Now the new code gives output:
^ Awesome table ^^
^ x ^ y ^
| 0.51 | 0.50 |

Please include the following if applicable:

[ ] Update the docstring(s)
[ ] Update the documentation
[ ] Tests to check the (new) code is working as desired.
[ ] Does it solve one of the open issues on github?

Notes on testing

Until the automated tests pass, please mark the PR as a draft.
On the XENONnT fork we test with database access, on private forks there is no database access for security considerations.

All italic comments can be removed from this template.

Hey Dacheng, I read through the commits and they look fine to me. Correct me if I'm missing something but, it seems like the failing tests are unrelated to your commits?

The error seems to be related to the holoviews displays, but I do not know why.

coverage: 93.521% (-0.01%) from 93.531% when pulling 2bef011704fd8d4f247a1b885430b35502e8cc29 on format_float into 5f232eb2c1ab39e11fb14d4e6ee2db369ed2c2ec on master.


How do I train this model on a custom dataset?

Hi @soo89,
Wonderful work.
Could you please help understand how can I train this semi-supervised model on my own dataset. I only have images and xml files. I have not worked with VOC/COCO datasets before.  Could you please give some suggestions about where should I start? Do I need to split my dataset into labelled and unlabelled datasets to perform semi-supervised learning?
I appreciate your response.

Thanks for your interest.
I suggest you try to train the basic CSD model with PASCAL VOC first.
And, please check the "voc07_consistency.py" file in the data folder.
If you understand how it works, you can apply this algorithm to your dataset.

@soo89
Thank you for the prompt response. I will check this "voc07_consistency.py" out.  I have trained your model on VOC2007 and VOC2012 datasets. But I am having difficulty in understanding how semi-supervised learning is working. For semi-supervised learning, one should be having a set of labelled and unlabelled images as shown in Fig. 1 (d) of your paper. But VOC2007 and VOC2012 are fully labelled.  How did you use partial labels in your code to perform semi-supervised learning? Could you please explain this part to me? Thank you.

please check 'voc07_consistency.py' in detail.
I did not use VOC2012 labeled data.
In other words, it is used the VOC07 image and labeled data and only VOC12 image.
    if (img_id[0][(len(img_id[0]) - 7):] == 'VOC2007'):
        target = ET.parse(self._annopath % img_id).getroot()
        img = cv2.imread(self._imgpath % img_id)
        semi = np.array([1])
    elif (img_id[0][(len(img_id[0]) - 7):] == 'VOC2012'):
        img = cv2.imread(self._imgpath % img_id)
        target = np.zeros([1, 5])
        semi = np.array([0])

@soo89  Got it. Thanks. But I am still not sure how to train on my own dataset. I guess, the easiest method that comes to my mind is to replace XML and JPG files in their respective folders with my own files and create trainval.txt files (filenames) for both VOC2007 and VOC2012. Also, I have to change my filenames to 000001.xml (and .jpg). What you think, am I missing something?

I also think that it seems the easiest way.
There seems to be no problem. Good luck.

@soo89  One last question before you close this thread. After training this model using train_csd.py and evaluating using eval.py, what will be the output? Will I get label files (.xml) in terms of bounding boxes for unlabelled data?

it does not need a pre-trained supervised object detection model.
It means that you can train only with 'train_csd.py'.
The output of 'eval.py' is the mAP score for the VOC07 test.
To get a box and score, you need to find the code or make the code.

Thanks for all your answers to my questions. I will contact you again if i have any other questions. You can close this thread.


Add HSplit between inspector and viewport

This was separated from #115 .
The viewport does not resize yet.

This was added in 117c889


fix SoundFile.processPeaks()

Fixes https://github.com/processing/p5.js-sound/issues/412
There is a if (!self.panner) return; check at https://github.com/processing/p5.js-sound/blob/b291d88d9bcb45905852fabbcbb2d8fff0999c00/src/soundfile.js#L1547 which always fails, because self variable is undefined.
Not sure if this is proper way to fix this, but it seems pretty obvious and the callback starts to work. 🙂

@therewasaguy it seems to be a reasonable and effective  approach!

@therewasaguy Hi, as I mentioned in https://github.com/processing/p5.js-sound/pull/548#issuecomment-751517090 I don't use this library anymore, so I really have no opinion one way or the other. 🙂

Hey there!
As I understand processPeaks is no longer a thing. Is there a substitute? We used to be able to generate an array of timestamps of peaks/beats of a soundFile beforehand. It seems like you can only get peaks of the live audio output now with peakDetect, is that correct?


bugfix: remove space from keys in theme and themeType

Keys having nonmatching space so space should be removed.

closing this issue since it's working


The Add Bookmark Control dialog box appears.

The Add Bookmark Control dialog box appears.
No it doesnt

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

ID: 89eb61fa-81bf-878b-148e-52df4634d8d9
Version Independent ID: 5494bd57-78e4-ceff-f990-74e9412bd7be
Content: Walkthrough: Insert text into a document from an actions pane - Visual Studio
Content Source: docs/vsto/walkthrough-inserting-text-into-a-document-from-an-actions-pane.md
Product: visual-studio-dev15
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@davepile Thanks for your feedback. We are aware that the documentation is outdated; it's in our backlog queue to update. Meanwhile, if you require immediate assistance, please consider perusing the StackOverflow website (and note that the link I've included goes directly to a collection of questions and answers that are tagged with "action pane" and "VSTO").

The document is correct.
There should be a "Add Bookmark Control" dialog that states:  "Make a selection or place the insertion point where you want to add a bookmark control, and click OK."

#please-close


[OLGA-67] fixed size for metrics blocks

Description: fixed size for metrics blocks in the Geography tab
Youtrack: https://youtrack.raccoongang.com/issue/OLGA-67

Codecov Report

Merging #231 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #231   +/-   ##
========================================
  Coverage    97.48%   97.48%
========================================
  Files           23       23
  Lines          477      477
========================================
  Hits           465      465
  Misses          12       12

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 18d3b29...26269d7. Read the comment docs.


tests() and find_prog()

Traceback (most recent call last):
  File "/home/brain/git/upstream/meson/meson_test.py", line 176, in <module>
    run_tests(options, datafile)
  File "/home/brain/git/upstream/meson/meson_test.py", line 165, in run_tests
    drain_futures(futures)
  File "/home/brain/git/upstream/meson/meson_test.py", line 125, in drain_futures
    print_stats(numlen, tests, name, result.result(), i, logfile, jsonlogfile)
  File "/usr/lib64/python3.4/concurrent/futures/_base.py", line 402, in result
    return self.__get_result()
  File "/usr/lib64/python3.4/concurrent/futures/_base.py", line 354, in __get_result
    raise self._exception
  File "/usr/lib64/python3.4/concurrent/futures/thread.py", line 54, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/home/brain/git/upstream/meson/meson_test.py", line 96, in run_single_test
    env=child_env)
  File "/usr/lib64/python3.4/subprocess.py", line 858, in __init__
    restore_signals, start_new_session)
  File "/usr/lib64/python3.4/subprocess.py", line 1456, in _execute_child
    raise child_exception_type(errno_num, err_msg)
OSError: [Errno 8] Exec format error
FAILED: '/usr/bin/python3' '/home/brain/git/upstream/meson/meson_test.py' '/home/brain/tmp/json-parser/buil/meson-private/meson_test_setup.dat'
ninja: build stopped: subcommand failed.

I've never seen an exec format error before. Some googling says that it is caused by trying to run a binary with the wrong architecture (e.g. 64 bit exe on 32 bit os). Can you verify that the binary it finds is the correct one and runnable and that if it is a script that its shebang line is correct?

Binary is python file. I'll check shebang tomorrow

it was wrong shebang, so closing it


GPII-4270: Upgrade to 1.13.11-gke.15 - fixes Istio CVEs

This PR upgrades GKE clusters to 1.13.11-gke.15. This version includes Istio 1.1.17 which fixes known security issues, see https://istio.io/news/security/istio-security-2019-007/ for details. (GPII-3850).
The fixed GKE version should be removed, once the 1.13.11-gke.15 or newer becomes default (comment added in the code).
Changes:

Bump GKE to fixed version to 1.13.11-gke.15 -  includes Istio 1.1.17

Deployment:
This is a no-downtime deployment.
Testing:
This change has been tested in dev cluster.

LGTM


Ion-range responds extremely slowly when bound and inserted via *ngFor

Ionic version:  (check one with "x")
[ ] 1.x
[ ] 2.x
[x] 3.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
I have the following prop in my data provider/service...
 get Comparisons(): number[] {
    if (this.data == null)
      return [];
    // console.log('PWCInputProvider------------>>Comparisons', r);
    return this.data.HDM.Level[1].E[2].w;  // returns an array of 3 numbers (for now)
  }

Here is the html
<p>
    Comparisons....
  </p>
  <ion-list>
    <ion-item *ngFor="let w of PWCInputProvider.Comparisons; let i = index">
      <ion-range min="0" max="100" pin="true" debounce="50"  [(ngModel)]="PWCInputProvider.Comparisons[i]"  >
        <ion-label range-left class="small-text">Apples</ion-label>
        <ion-label range-right>Oranges</ion-label>
      </ion-range>
    </ion-item>
  </ion-list>

If I enable the console.log I will see a zillion calls to the Comparisons property while I try to move the slider (range) and it responds very very slowly and the range pin won't even display, rendering this user control not usable.
I would like the 2-way binding between the ranges and some corresponding model in my data provider class (an array of  integers representing the latest values).
Expected behavior:
range sliders responds to human interaction
Steps to reproduce:
make a provider return an array of 3 numbers and use it to insert 3 range selectors bound to each element of this collection.  The insertion in the view should be done via an *ngFor  (ie. dynamic)
For Ionic 2 issues - http://plnkr.co/edit/z0DzVL?p=preview
-->
Related code:
Other information:
If I remove the ngModel altogether I get a normal UI response from the range selectors, smooth sliding, the pin pops, shows value and has an animation, but of course all ranges inserted in this manner are just vanilla, unbound and no initial value.
UPDATE
It appears as if inserting ranges via the ngFor AND making them bound is the problem.  My current test data returns an array of 3 numbers, if I insert an instance of the range bound to the second element as below explicitly before the ngFor, then I get the following behavior on the 4 resulting ranges in the view. This first instance responds just fine to human interaction and also causes the 3rd instance to sync (expected).  However, all 3 instances (2nd, 3rd and 4th) of the range control inserted via the *nfFor do not respond to human interaction (I guess busy making calls to the bound model).
<ion-range min="0" max="100" pin="true" [(ngModel)]="PWCInputProvider.Comparisons[1]">
    <ion-label range-left class="small-text">Apples</ion-label>
    <ion-label range-right>Oranges</ion-label>
  </ion-range>

  <ion-list>
    <ion-item *ngFor="let w of PWCInputProvider.Comparisons; let i = index">
      <ion-range min="0" max="100" pin="true" [(ngModel)]="PWCInputProvider.Comparisons[i]">
        <ion-label range-left class="small-text">Apples</ion-label>
        <ion-label range-right>Oranges</ion-label>
      </ion-range>
    </ion-item>
  </ion-list>

Why doesn't ion-range work like other ionic components eg. label, textbox, etc...  why is ngModel needed?
Could it be simplified to something like
<ion-range>{{w}}<ion-range>
other parameters could be supplied as attributes. including the initial/bound value like so:
<ion-range min="0" max="100"  value="{{w}}" ><ion-range>
Thank you.
global packages:

    @ionic/cli-utils : 1.0.0
    Cordova CLI      : 7.0.0
    Ionic CLI        : 3.0.0

local packages:

    @ionic/app-scripts              : 1.3.7
    @ionic/cli-plugin-cordova       : 1.0.0
    @ionic/cli-plugin-ionic-angular : 1.0.0
    Ionic Framework                 : ionic-angular 3.2.0

System:

    Node       : v7.2.0
    OS         : Windows 7
    Xcode      : not installed
    ios-deploy : not installed
    ios-sim    : not installed

Hello, thanks for opening an issue with us. Would you mind posting a plunker or minimal repo we could use to reproduce this issue? Thanks!

Here is the plunker you requested
http://plnkr.co/edit/ps2YEfwdHhgC0YcU0lJ2?p=preview
It should render like this....

Any news on this issue?


Arm builds

This adds triggering and display of ARM builds.
The table doesn't look great with the long label, in particular on mobile but I guess we can tweak that in another pass.
Leaving this in draft so I can merge it when I'm around to monitor the runners.
Fixes #605

The table display issues should be mitigated by #566

The table display issues should be mitigated by #566

Ignore that, I see the issue now. It's not in that view, it's in the Package show page. I'll have a think about what we can do with that.


Make travis great again

Modified the travis build to have a parallel structure to scikit-learn.

Coverage increased (+0.2%) to 96.141% when pulling 8f9d2ea4791582182d1ae0688691537e6890b810 on make-travis-great-again into 98b29b06dd147ce7f7db537c07603cee5b0e4b15 on develop.


Broken build

https://ci.eclipse.org/microprofile/job/OpenTracing-maven-snapshots/
I disabled the build, since no one was fixing it. It had been failing for about 2 weeks.

See #18

Jenkins builds for microprofile-opentracing are now succeeding


Can't remove image when image was placed in land claimed with GriefPrevention

I cannot remove a image i have placed when the image is in a area claimed with GriefPrevention.

Hello @MarcelskyXD,

What server are you running? SpigotMC, Bukkit, PaperMC, etc.
What Minecraft version are you on? 1.19.2, 1.19.3, etc.
What version of ProtocolLib are you using? 4.8.0, 5.0.0-SNAPSHOT-b608, etc.
What version of GriefPrevention are you using?
What error message are you getting?
What error trace do your server logs contain (if any)?

Hello @MarcelskyXD,

What server are you running? SpigotMC, Bukkit, PaperMC, etc. Purpur (paper fork)
What Minecraft version are you on? 1.19.2, 1.19.3, etc. 1.19.3
What version of ProtocolLib are you using? 4.8.0, 5.0.0-SNAPSHOT-b608, etc. 5.0.0-SNAPSHOT-b608
What version of GriefPrevention are you using? 16.18
What error message are you getting? no error, i just cant break it
What error trace do your server logs contain (if any)? none

No error when trying to remove it? That's weird...
Please paste here the output from the /image describe command of the image you cannot remove.

Hi @MarcelskyXD,
This should be fixed in the next pre-release (v1.2.8-SNAPSHOT).
Give it a try and let me know if it worked :)

For now i will close this issue, i will re open it if the pre release version does not work.


Add network-<IP_ADDRESS> to 18.13 snapshot

This pulls in a fix for the package configuration files to recognize arm64-apple systems, see: https://github.com/haskell/network/pull/510

Ah, I saw #14 but now see that that was special because the snapshot was utilized.

Sorry for the noise, I had thought this exception when updating the resolver to use this branch was related to the resolver field in the YAML and not the path itself:
Exception while reading snapshot from https://raw.githubusercontent.com/freckle/stackage-snapshots/corey/13-FR1/lts/18/13-FR1.yaml:
Specified snapshot as file path with "lts-18.13-FR1", but not reading from a local file
Type help for available commands. Press enter to force a rebuild.

Sure!


Running a WDL workflow twice in the same directory results in a super-long start

Bug Report
Expected behaviour
Can run WDL workflow in the same directory twice without issue with the same runtime
Actual behaviour
The second time you run a workflow, the zipDirectory method blows up as input files get mistaken for workflow descriptors
git clone https://github.com/DataBiosphere/topmed-workflows.git
cd topmed-workflows/
cd aligner/u_of_michigan_aligner-checker
git checkout 1.32.0
dockstore workflow launch --local-entry u_of_michigan_aligner_checker.wdl --json u_of_michigan_aligner_checker.json
# can abort the previous command after input files are downloaded (roughly 9GB)
dockstore workflow launch --local-entry u_of_michigan_aligner_checker.wdl --json u_of_michigan_aligner_checker.json

In a separate window look at the directory mentioned that looks like Created temporary working directory at '/tmp/1566933878086-0' and watch directory.zip grow
Discovered while working on #2746
┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Issue Number: DOCK-939

Can confirm with 1.7.0-rc.0


How to make support other?

How to make support other accessories like desoldering iron on brand Weller like DSX 80 or JBC

In the Dangerous Prototypes thread there's some information, for example: http://dangerousprototypes.com/forum/index.php?topic=7218.780


Up to date  dnscrypt-resolvers.csv

Is the list dnscrypt-resolvers.csv maintained? It seemed that the resovled ip of the Swiss d0wn server is not correct anymore.

It is! But dnscrypt providers should submit a pull request if they change something!


Using Web Proxy

Hi team,
We're looking for a solution to use qlik cli via web proxy settings, in order to bypass corporate firewall. Is that possible achieve it with some qlik cli flags or smth like that?
Thank you

Hi @Justformoment, sorry for the late reply. There is some support for using qlik-cli behind a proxy that you can try.
To use qlik-cli when working behind a proxy. Set the operating system environment variables called HTTPS_PROXY/HTTP_PROXY with the hostname or IP address of the proxy server.
The environment values may be either a complete URL or a "host[:port]", as seen in the example below.
HTTPS_PROXY=https://<IP_ADDRESS>:1234 (or) HTTPS_PROXY=http://proxy.example.com
You must add the port number if the proxy server uses a port other than the defaults 80 or 443, as seen in the example below.
HTTPS_PROXY=https://proxy.example.com:3000
If the proxy server requires a username and password, you must include the credentials e.g<EMAIL_ADDRESS>NOTE HTTPS_PROXY takes precedence over HTTP_PROXY for https requests. Qlik CLI supports only Basic Authentication.

Please reopen this issue if the reply above does not resolve your issue.


Update README.md - frontmatter Link Error Fixed

frontmatter is now pointing to the correct link - https://middlemanapp.com/basics/frontmatter/

Thanks!


Mazars - Missing "Targetare.ro" button

ENVIRONMENT: production
URL: https://scrapers.peviitor.ro
Browser: Chrome
Device: Laptop
OS: Windows 10
STEPS TO REPRODUCE:
1.Open the URL in the browser
2.Search for "Mazars " Company
3.Go to logo - Click on Targetare.ro
4.Verify "Targetare.ro" button
EXPECTED RESULTS:
The "Targetare.ro" button is functional . Targetare link opens correctly
ACTUAL RESULTS:
Missing "Targetare.ro" button. Targetare link doesn't open targetare.ro

Fixed!

Data is now correct after fixing the bug.
But verify this one: The number of jobs on scrapers peviitor.ro (26) is not the same as the number of jobs peviitor.ro (24).

check on "targetare.ro" if it exists


Let the shortcut-guide  support more software

Description of the new feature / enhancement
Now we can only see system shortcuts.
If you are in MSOffice, can you display their(word,excel,powerpoint) shortcut keys?
Scenario when this would be used?
Display shortcut keys for each software
Supporting information
something like this:
CheatSheet  for MacOS
or
https://github.com/BetterWorld-Liuser/XiaoChao

/dup #890


New tab defaults to first item in list

Environment
Windows build number: OS Build 19041.508
Windows Terminal version (if applicable): 1.3.2651.0

Steps to reproduce
Add newTab command
{ "command": { "action": "newTab", "index": 0 }, "keys": "ctrl+T" }

Set default shell to anything other than the first in the array
{
    "$schema": "https://aka.ms/terminal-profiles-schema",
    "defaultProfile": "{ubuntu-guid}",
    "copyOnSelect": false,
    "copyFormatting": false,

    "profiles": {
        "defaults": {},
        "list": [
            {
                "guid": "{powershell-guid}",
                "name": "Windows PowerShell",
                "commandline": "powershell.exe",
                "hidden": false
            },
            {
                "guid": "{ubuntu-guid}",
                "hidden": false,
                "name": "Ubuntu",
                "source": "Windows.Terminal.Wsl",
            },
        ]
    },
    // ...
}

Create new tab with command
ctrl+T

Expected behavior
Expect it to take the default shell
Actual behavior
It takes the first in the list, in this case Windows PowerShell. It's not really an issue that would block anyone as you simply move the default object to the first item in the list, its just unexpected.

{ "command": { "action": "newTab", "index": 0 }, "keys": "ctrl+T" }
                                   ^^^^^^^^^^
                                   this part means "first profile in the list"

{ "command": { "action": "newTab", "index": 0 }, "keys": "ctrl+T" }
                                   ^^^^^^^^^^
                                   this part means "first profile in the list"

Stupid moment... thanks, without thought I figured that was tab index, not array index

Hopefully #1564 goes some way towards helping! :smile:


DoubleTransition on ProgressBar value causes a memory leak

Hi,
I'm trying to animate the ProgressBar's Value property when it changes.
The Value is bound to property on the ViewModel.
Here is the xaml:
<ProgressBar Maximum="{Binding MainProgressBarMaximum, FallbackValue=100}" Value="{Binding MainProgressBarValue, FallbackValue=0}">
    <ProgressBar.Transitions>
        <DoubleTransition Property="ProgressBar.Value" Duration="0:0:0.2" />
    </ProgressBar.Transitions>
</ProgressBar>

Property on the ViewModel looks like this:
private double _mainProgressBarValue = 0;
public double MainProgressBarValue
{
	get => _mainProgressBarValue;
	set => this.RaiseAndSetIfChanged(ref _mainProgressBarValue, value);
}

After the ViewModel is loaded I do this:
MainProgressBarValue = 50;
As soon as I do that, memory usage skyrockets, reaching 2GB in a matter of seconds.
When I stop with a breakpoint in the setter of 'MainProgressBarValue' I can see that the value is repeatedely set to 50, than to 0, then 50... (repeat).
As you can see on the attached screenshot, the ProgressBar's Value stays at 0 when this problem occurs.

Is that a bug, or I'm misusing the api?
Here is xaml from WPF that works just fine, and animates the value of progress bar smoothly.
<ProgressBar
    Maximum="{Binding MainProgressBarMaximum, FallbackValue=100, NotifyOnTargetUpdated=True}"
    Value="{Binding MainProgressBarValue, FallbackValue=0, NotifyOnTargetUpdated=True}">
    <ProgressBar.Triggers>
        <EventTrigger RoutedEvent="Binding.TargetUpdated">
            <BeginStoryboard>
                <Storyboard>
                    <DoubleAnimation
                        Storyboard.TargetProperty="Value"
                        To="{Binding MainProgressBarValue, NotifyOnTargetUpdated=True}"
                        Duration="0:0:0.30" />
                </Storyboard>
            </BeginStoryboard>
        </EventTrigger>
    </ProgressBar.Triggers>
</ProgressBar>

@MarchingCube I need some help in profiling this. This could be a huge memleak coming somewhere from animations subsystem

@jmacato Do you also experience this behavior in latest master?

Yeah I agree that's probably the best way forward.

Any news for this issue? Same here

Since this causes a memory leak, is there any other way of achieving a smooth progress bar animation?

@RyouBakura the workaround would be an attached property that changes the direct one.
public class ProgressBarWorkaround
{
    public static AvaloniaProperty<double> ValueProperty =
        AvaloniaProperty.RegisterAttached<ProgressBarWorkaround, ProgressBar, double>("Value");

    public static void SetValue(ProgressBar pb, double value) =>
        pb.SetValue(ValueProperty, value);

    static ProgressBarWorkaround()
    {
        ValueProperty.Changed.Subscribe(ev =>
        {
            ((ProgressBar) ev.Sender).Value = (double) ev.NewValue;
        });
    }
}

    <StackPanel Margin="30">
        <StackPanel.Styles>
            <Style Selector="ProgressBar">
                <Setter Property="Transitions">
                    <Transitions>
                        <DoubleTransition
                              Property="(local:ProgressBarWorkaround.Value)"
                              Duration="0:0:0.5"/>
                    </Transitions>
                </Setter>
            </Style>
        </StackPanel.Styles>
        <ProgressBar local:ProgressBarWorkaround.Value="{Binding Progress}"/>
    </StackPanel>


Add pre-commit hook to lint test files.

Two things:

run prettier --write test/**/*.js
add a pre-commit hook to lint-staged to do the same thing

Fine for now, but we should stop using husky. It doesn't work right with git. We should check the formatting with a CI action.


Prettify output when installing Taylor

When running make install use xcpretty if available.

Current coverage is 93.83%

Merging #29 into master will not affect coverage as of df2cbdc

@@            master     #29   diff @@
======================================
  Files           57      57
  Stmts         1607    1607
  Branches         0       0
  Methods          0       0
======================================
  Hit           1508    1508
  Partial          0       0
  Missed          99      99

Review entire Coverage Diff as of df2cbdc

Powered by Codecov. Updated on successful CI builds.

Thank you! :wink: :dancers:


HAWQ-682 Include ignore-bad-hosts option in hawq init master command …

…to avoid syncup failures during setting default bucket number

Looks good to me. +1


fix: flag required params as missing when explicitly undefined

Currently, we are checking for "truthyness" when evaluating if required parameters have been set or not. This is flawed - when required params are set to false, they are incorrectly being flagged as missing.
This PR fixes that bug by explicitly checking for the undefined value. Test case added to support the fix.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


chore: bdd-spring-1580823272 to 0.0.1

chore: Promote bdd-spring-1580823272 to version 0.0.1

[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


fix(empty): fix automatically inherited warn

Types of changes

[ ] New feature
[x] Bug fix
[ ] Enhancement
[ ] Component style change
[ ] Typescript definition change
[ ] Documentation change
[ ] Coding style change
[ ] Refactoring
[ ] Test cases
[ ] Continuous integration
[ ] Breaking change
[ ] Others

Background and context
empty 组件在全局配置 ConfigProvider 设置了自定义空状态后会出现继承警告问题

[!WARNING]
[Vue warn]: Extraneous non-props attributes (xxxx) were passed to component but could not be automatically inherited because component renders fragment or text root nodes.

当使用 configCtx.slots.empty 自定义空状态时无法确定是否是单个根节点
https://github.com/arco-design/arco-design-vue/blob/faf3ce8cdb10bdda25b5217d6c13c0d7ce2e95c7/packages/web-vue/components/empty/empty.tsx#L41-L47
Solution
直接添加一个层包裹可以解决但层级结构变了 怕影响大
return <div>{configCtx.slots.empty({ component: 'empty' })}</div>

该 PR 采用手动选择性继承，影响范围与之前一致
How is the change tested?

Changelog

Component
Changelog(CN)
Changelog(EN)
Related issues

empty
修复属性自动继承的错误
fix automatically inherited warn
Closes #3034

Checklist:

[x] Test suite passes (npm run test)
[x] Provide changelog for relevant changes (e.g. bug fixes and new features) if applicable.
[x] Changes are submitted to the appropriate branch (e.g. features should be submitted to feature branch and others
should be submitted to main branch)

Other information

嗯，增加层级这种属于bk，能回避时最好回避下


Allow sync storage locations other than web

Web sync storage backends might be the right solution for many users, but they're inflexible and won't work for everyone. For example, users who work in internet-restricted environments, or whose configs contain sensitive or proprietary information (e.g. in build tasks) won't be able to use public sync backends, and the companies they work for might not be willing to dedicate the time to running a central sync backend server.
A simple solution would be to allow for non-web sync backends: initially it should be possible to sync your settings to a local directory, and eventually maybe ssh, sftp, etc could be supported (though those features could potentially be provided by extensions).
Many other dev tools such as Alfred and iTerm2 allow users to save their settings to a custom location, this would be similar. This approach would put the power in the hands of the user to build sync solutions that work the way they need them to.

/duplicate

https://github.com/microsoft/vscode/issues/92357

@sandy081 I don't understand how this is a duplicate of #92357. All of the discussion in #92357 is around custom backend servers, whereas this issue is suggesting a settings sync option that uses no backend at all. Solving #92357 requires establishing a generalized protocol with which VSCode can communicate with arbitrary backend servers, solving this issue requires teaching the Settings Sync feature how to talk to filesystems.

A backend can be anything.. it can be a remote server or on local disk. #92357 is allowing to support a custom backend which can be a server on web or local disk. Hence marked as dup.


Support aggregation on NullType in RunningWindowExec

Fixes #2715.
#2596 extended WindowExec's meta to allow for aggregations on NullType input. E.g.
SELECT MIN(null) OVER (PARTITION BY foo ORDER BY bar ROWS BETWEEN UNBOUNDED PRECEDING AND CURRENT ROW)...

test_window_running() and test_window_running_no_part() integration tests were added to exercise this path.
#2596 did not include a change to enable NullType aggregations in the Databricks Shims, causing the test_window_running*() tests to fail.
This commit should resolve the failure.
Signed-off-by: Mithun RK<EMAIL_ADDRESS>

build

Do we know if Adobe will be using NullType?  Just wondering about 21.06

Do we know if we need NullType support for 21.06?

#2596 didn't ship in 21.06, so I think it should be alright for this to be left out of 21.06 as well.


tests: add out of balance test

Codecov Report

Merging #518 (1f26fab) into main (fdbc124) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #518      +/-   ##
==========================================
+ Coverage   18.06%   18.16%   +0.10%
==========================================
  Files         136      136
  Lines       18853    18878      +25
==========================================
+ Hits         3406     3430      +24
- Misses      15447    15448       +1

Impacted Files
Coverage Δ

core/executor/src/tests/mod.rs
100.00% <100.00%> (ø)

core/mempool/src/pool.rs
87.93% <0.00%> (-0.58%)
:arrow_down:

The test has been migrated to #522


Update manifest.json to support app tools

Add xwalk_package_id  in manifest
Update version, icon in manifest

https://crosswalk-project.org/jira/browse/XWALK-5737

I'm not sure how much say I have in this, but I'd prefer these changes to be made upstream :
https://github.com/01org/webapps-hangonman
https://github.com/01org/webapps-memory-game-older-kids
Also, why does one have a key field, and the other not? ...and one has 'start_url' as 'app/index.html' and the other is simply 'index.html'. That doesn't seem quite right to me - they should be the same, iinm.
I'm not too keen on a different 'package_id' to their upstream ones either.
Can you tell me what the motivation is behind these changes?

hi @davidmaxwaterman crosswalk will replace make_apk.py with app tools, the app tools need a package id in manifest.json.
I don't update the upstream code because this changes of manifest.json align to app tools requirement. I see that there is single manifest.json in each demo, so update it.

Ok, fair enough - it's quite a bit more work to switch the upstream to use app tools, but I have it on my todo list.
What about the other questions I had...why are the changes not 'the same'? Ie one has 'app/index.html' vs 'index.html' and one has a 'key' but the other not?

I just update the existed items in the manifest.json, and remove not support/useless items. I do not clear whether the "key" in MemoryGame/manifest.json have special use, so keep it.
I have updated "index.html" to "app/index.html".

OK, thanks for app/index.html change.
From the diff, it still looks like you've added the key...is that not the case?

Oh sorry, I diff a incorrect version of manifest.json, not notice that the origin repo has not this key. It may be added in our local test code by manual, I will remove it soon.

lgtm


Update README.md

(Task 1) Add my name to the README Ahwaban

Create a new branch and send pull request. This is incorrect


Building freetype:arm-uwp fails

I'm trying to build freetype:arm-uwp v2.8 on Windows 10, de-de locale, using VS 2017. The build fails just before linking with the following error in package-arm-uwp-rel-out.log:
unrecognized file format in 'C:/Users/patrick.SPRINGFIELD/Documents/development/vcpkg/buildtrees/freetype/arm-uwp-rel/freetype.dir/Release/autofit.obj'
x86-uwp and x64-uwp are building without problems. package-arm-uwp-rel-err.log is empty.
Here is the complete build log:
package-arm-uwp-rel-out.log.txt

You should use CMake with version >3.8.2. Easiest way to workaround it is download it from cmake.org and make in-place replace in folder <vcpkg_root>\downloads\cmake...

Many thanks! In-place replace of cmake 3.8.0 with 3.9.0-rc3 fixed building freetype for arm-uwp.

We'll definitely update to 3.9.0 as soon as the main release comes out. Thanks for providing the current workaround, @zabulus!

Hi @pdehne , thanks for reporting this issue!
Please open a new issue if this is still a problem for you.
Thx.


doubt  | disableClusteringAtZoom spiderfyOnMaxZoom

[ ] I'm reporting a bug, not asking for help
[ ] I'm sure this is a Leaflet.MarkerCluster code issue, not an issue with my own code nor with the framework I'm using (Cordova, Ionic, Angular, React…)
[x ] I've searched through the issues to make sure it's not yet reported

"leaflet": "^1.2.0",
"leaflet.markercluster": "^1.1.0",

is there any way to access the clusters and with a method one can display the bookmarks separated in this way

what I want to achieve is that when I reach the maximum zoom automatically it will show the ungrouped markers in the form of a spider I could not achieve this with the configuration
disableClusteringAtZoom: 17,
spiderfyOnMaxZoom: false,
the markers appear high but very close together and do not look good. something like this.

Hi,
This sounds to me similar to:

When you set disableClusteringAtZoom the spiderfy does not work #339
Spiderfy overlapping markers if zoomlevel is below disableClusteringAtZoom #311

apparently can not implement this feature I can not find a solution.

Oh you mean the cluster would spiderfy automatically once reaching max zoom, instead of having to click on it? Then it sounds more like:
Spiderfy all clusters at a particular view #744
That being said, how to know which cluster to spiderfy, if you have more than one on screen?
Of course you could try to spiderfy all of them, as proposed in #744.
But then what if there are many markers being spiderfied? the spirals would overlap…
If you are ready to implement it yourself and need a little bit of help, feel free to use Stack Overflow.

Duplicate of https://github.com/Leaflet/Leaflet.markercluster/issues/744 either way


[SPARK-21685][PYTHON][ML] PySpark Params isSet state should not change after transform

What changes were proposed in this pull request?
Currently when a PySpark Model is transformed, default params that have not been explicitly set are then set on the Java side on the call to wrapper._transfer_values_to_java.  This incorrectly changes the state of the Param as it should still be marked as a default value only.
How was this patch tested?
Added a new test to verify that when transferring Params to Java, default params have their state preserved.

Test build #80813 has started for PR 18982 at commit 1203056.

Test build #80813 has finished for PR 18982 at commit 1203056.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80813/
Test PASSed.

cc @jkbradley @holdenk

Thanks for reviewing @holdenk !  You brought up some good points, let me know if you prefer me to change them.

@BryanCutler sorry my slowness, responded with a note on the tests. If you have a chance to update the test and update to master I'd love to try and get this in.

Test build #81484 has started for PR 18982 at commit 482c025.

Test build #81484 has finished for PR 18982 at commit 482c025.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81484/
Test FAILed.

Jenkins retest this please

Test build #81487 has started for PR 18982 at commit 482c025.

Test build #81487 has finished for PR 18982 at commit 482c025.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81487/
Test FAILed.

No problem @holdenk, I updated using transform() on the test.  See if it looks ok to you now (pending Jenkins). Thanks!

Hmmm, I can repeat the error with Python3, I'll look into it tomorrow

Cool, let me know if you want a hand reproing the error.

Test build #81574 has started for PR 18982 at commit 1a9fa46.

Test build #81574 has finished for PR 18982 at commit 1a9fa46.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81574/
Test FAILed.

Merged build finished. Test FAILed.

@holdenk , the error was because I was calling setDefault with a separate Param and value. So for something like a seed param, the Long value would get converted to an Int in the exchange to Java if using Python 3 and then cause a cast error.  The only solution I could find was to create another method in Scala that takes a single ParamPair value because I just couldn't get the existing setDefault for a list of ParamPairs to work. Let me know if this sounds ok or you have another idea. Thanks!

Test build #81577 has started for PR 18982 at commit 088ee52.

Test build #81577 has finished for PR 18982 at commit 088ee52.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81577/
Test FAILed.

retest this please

Test build #81647 has started for PR 18982 at commit 088ee52.

Test build #81647 has finished for PR 18982 at commit 088ee52.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81647/
Test PASSed.

Test build #83691 has started for PR 18982 at commit 088ee52.

Test build #83691 has finished for PR 18982 at commit 088ee52.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83691/
Test PASSed.

Can we update this to master?

Test build #84035 has started for PR 18982 at commit bef3fb5.

Test build #84035 has finished for PR 18982 at commit bef3fb5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84035/
Test PASSed.

Merged build finished. Test PASSed.

Hi @holdenk , this is updated.  Look ok to you?

ping @MLnick @holdenk , seems like this issue popped up a couple times recently.  Can you please take a look here?

Test build #87228 has finished for PR 18982 at commit 339c793.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87228/
Test PASSed.

So between https://github.com/apache/spark/pull/20410 & this one which path do we want to go down?

retest this please

From my rough reading and based upon what I know, this seems fine.

Test build #88369 has started for PR 18982 at commit 339c793.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1604/
Test PASSed.

Test build #88369 has finished for PR 18982 at commit 339c793.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88369/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1667/
Test PASSed.

Test build #88452 has started for PR 18982 at commit 9162944.

Test build #88452 has finished for PR 18982 at commit 9162944.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88452/
Test FAILed.

cc @viirya too

retest this please

Test build #88470 has started for PR 18982 at commit 9162944.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1682/
Test PASSed.

I think I figured out how to use params setDefault(paramPairs: ParamPair[_]*) from python so we won't need to add setDefaultPair and the MiMa exclusion.  Let me just do a little more testing to make sure.

Test build #88470 has finished for PR 18982 at commit 9162944.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #88475 has started for PR 18982 at commit 24a1dbf.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1684/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88470/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1687/
Test PASSed.

Test build #88481 has started for PR 18982 at commit 2eaf1a7.

Test build #88481 has finished for PR 18982 at commit 2eaf1a7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88481/
Test PASSed.

Thanks @holdenk and @HyukjinKwon !  I made a small change so that this can call the existing setDefault method and can avoid adding anything new to the Scala side.  It's a little less clean, but I think better not to add anything if it can be avoided for this case.  Please take another look when you can.

Merged to master.

Thanks @holdenk @HyukjinKwon and @viirya !


update models for sdwandemo items

Pull Request summary:
multiple fixes
Description of changes:
Checklist:

[x] Make sure to run pre-commit before committing changes
[x] Make sure all checks have passed
[x] PR description is clear and comprehensive
[x] Mentioned the issue that this PR solves (if applicable)
[x] Make sure you test the changes

This issue was migrated from an old repository: https://github.com/CiscoDevNet/catalystwan/pull/499


update test to handle connect error when tserver killed

Adds a check in the test when trying to read the from TabletStateStore iterator.
Upfront, this seems to fix the test.  An alternative would be to handle the connection failure on the TabletStateStore implementations side of things.  Changing the test seems like fixing it "on the client side" and I am unsure if it may be preferable to handle it "on the server side".   The situation in the test is triggering a server failure to force table reassignments.  If the connection error is seen when reading values so the test knows when to proceed and specific to the test, then modifying the test seems the best approach.  However, if this is a more generic exception handling situation for a server that dies, then it may be better to strengthen the TabletStateStore implementations.  Currently only this test seems to be triggering the connection error / failure.  (This also may be a timing issue with the dead server check and the way the test is using it)
The change tries to get the TabletStateStore iterator, catching any Exception and retrying until the store is available.  The issue is caused when trying to talk with a recently kill server. When the tserver is killed, by design as part of the test, occasionally the TabletStateStore iterator will see an exception that the socket is closed.  The exception thrown is an EOFException, wrapped in a RuntimeException.
With the test code changes:
2023-09-22T17:20:09,789 [functional.ConfigurableMacBase] DEBUG: Failed to get TabletStateStore iterator - will retry
java.lang.RuntimeException: java.io.EOFException: Socket is closed by peer.
        at org.apache.accumulo.core.clientImpl.TabletServerBatchReaderIterator.hasNext(TabletServerBatchReaderIterator.java:198) ~[accumulo-core-2.1.3-SNAPSHOT.jar:2.1.3-SNAPSHOT]
        at org.apache.accumulo.server.manager.state.MetaDataTableScanner.hasNext(MetaDataTableScanner.java:125) ~[accumulo-server-base-2.1.3-SNAPSHOT.jar:2.1.3-SNAPSHOT]
        at org.apache.accumulo.test.ManagerRepairsDualAssignmentIT.test(ManagerRepairsDualAssignmentIT.java:120) ~[classes/:?]

Catching the exception allows the test to continue.  There is a background thread that results in
2023-09-22T17:20:10,800 [rpc.ThriftUtil] WARN : Failed to open transport to ip-xx-xx-xx-xx:45665
2023-09-22T17:20:10,905 [rpc.ThriftUtil] WARN : Failed to open transport to ip-xx-xx-xx-xx:45665
2023-09-22T17:20:10,909 [rpc.ThriftUtil] WARN : Failed to open transport to ip-xx-xx-xx-xx:45665

Once that clears (I'm assuming that it eventually finds the remaining running server) the test seems to pass reliably. I have not seen a case where the retry is attempted more than once.
If this is the correct way to fix the test, this fixes #3762

Not sure if this strengthens the case for improving the error handling in TabletStateStore - but in the test, the bad server scan sees the communication error - but it does not reach the threshold to mark the server as bad (it takes 3 failed attempts).  It seems that depending on which server is used by TabletStateStore the test passes if it picks the "good" server.  If it picks the dead, but not declared dead server, the socket closed / EOF exception triggers the test failure.
It seems that trying to talk to a recently deceased server should be an exceptional condition that we should handle correctly rather than working around it in the test.

An alternate approach is provided in #3776 that counts the number of dead tservers as a check before allowing the test to continue - this provides an positive check that the killed tserver has been declared dead.
Only one of these should be merged - if the solution is to modify the test.  A new PR should be created if the solution it to improve the exception handling in the TabletStateStore iterator code.

Set this to draft becuase #3771 is likely to be the preferred approach.  The question of handling the exception upstream in the code remains open.

Closing in favor of #3789


Add const to public interfaces where appropriate

I've noticed that there are some missing consts on copied inputs, for example int64_t* shape for omTensorCreate(). This appears to be leading to unnecessary casting away constness for users. Is there any reason not to change this? If not, I can do a more thorough review of parameters where they could be added.

Mostly because we weren't paying attention in the beginning and now we want to keep them unchanged as much as possible for compatibility reasons. But I don't think adding const would be a problem. @cjvolzka agree?

Agreed. We're already doing some other API changes for the next release so this would be minor by comparison. If possible it'd be nice to include this by the end of the week or it's all part of the same release.

Thanks so much for the resolution @cjvolzka! I'm going to close this.


Speed up HashProbe against and the resizing of HashTable

Description
Currently, HashTable employees a single char** table_ = nullptr; and memory::ContiguousAllocation tableAllocation_; to hold all TagPointer pairs.
This structure, when it has reached a large size, has the limitations, that:

For HashProbe, if a probe row has no match, the worst case lookup loops through the entire table. (https://github.com/facebookincubator/velox/blob/main/velox/exec/HashTable.cpp#L159-L160)
For HashAggregation resizing, the entire table_ entires need to be re-probed and re-inserted, which is expensive.

This ticket proposes two changes, that would improve the above limitations:

Deprecate table_ and instead use a preconfigured or dynamically determined number of  table_partitions_'s to hash partitioned shards of the TagPointerPairs that would otherwise all reside in a single table_. This would help HashProbe look up as for each probe row, only a partition needs to be checked, vs the entire table. Further, for resizing, only a partition is resized as opposed to the entire table.

Optimize HashTable resize logic. The current logic rehashes, reprobes and reinserts all RowContainer rows into the newly sized table. Given that table_ is power of 2 sized, a row's slot number in the old table_ can be mapped to a new slot number in the newly allocated table; thus, the tag and pointer values can be copied over direct into new slot. No hash and reprobe is necessary. This change would make HashTable resize much faster; while also possibly reduce memory allocation failures due to smaller size being asked.

If the above two sound good, I can implement them.
@mbasmanova @xiaoxmeng @oerling

@shiyu-bytedance
Re: (1) - Would this be equivalent to partitioning the input to aggregation using LocalPartition?
Re: (2) - Is this similar to https://facebookincubator.github.io/velox/develop/hash-table.html#resizing ?

If a probing is running through the whole table, there is something terribly wrong and need to fix.  Sub-partitioning the table would just hide the real problem to some extent.

a row's slot number in the old table_ can be mapped to a new slot number in the newly allocated table; thus, the tag and pointer values can be copied over direct into new slot. No hash and reprobe is necessary.

How to map to the new table without hashing and probe? We have no idea a slot in old table was conflicted or not. If yes, it may have probed to the end and start in the beginning, in such case we can't directly map the slot into the new table because it will cause incorrect result.

@waitinfuture
Assuming we are only discussing kHash mode, we can make resize work without hashing and probing with the following changes:

Make hash value (int64) stable across resize. This is already the case.
Make Tag value stable across resize. This is already the case. Currently, Tag is the top 8 bits of hash, we will switch to using the lowest 8 bits.
Use the top K bits of hash as the bucket index(where K = log2(number of buckets)). This guarantees that after resize, which typically doubles the table size, the new bucket index and old bucket index will share a common prefix.
We store the stable hash value of each row in RowContainer, exactly how NormalizedKey is stored.

An example:
Assuming old table has 4 buckets, after resize,  new table should have 8 buckets.
old_table.bucket[2] entries will exist in new_table.bucket[4, 5] in the new table.
For each old_table.bucket[2] entry:

Compute its new bucket from the stored hash to determine whether it will be in bucket[4] or bucket[5].
Copy the TagPointerPair to new bucket. Tag and pointer are stable.

This achieves no rehash and probing.

Hi @shiyu-bytedance
old_table.bucket[2] can contain conflicted entries that belongs to bucket[1], so when mapping to new table, the stored hash tells that it should be put to bucket[2] or bucket[3], and at this time conflicts can happen again and we have to probe to find an empty slot.


Does social login/register work on localhost?

Might be a bit of a noob question but does social login/register work on MAMP localhost?
Currently I have it setup for twitter google and facebook neither of them work.
Twitter I get - EAuthException: Error in retrieving token.
Facebook I get - Setting read-only property: common\models\User::authKey
Google I get - Error: redirect_uri_mismatch
Update I forgot to add
public $authKey;

I have exactly that problem now : )


Add url revision helpers

Moved the url stuff from #21
Added url_without_revision, needed in workflows

I'll look after @joni-herttuainen has a chance to look.


We had executed the scanner tool manually with suggested setting.It has scanned 348 sites out of  1941 sites from the tenancy and got aborted in the middle[BUG].

Thank you for reporting a bug. We appreciate your feedback - to help the team to understand your needs, please complete the below template to ensure we have the necessary details to assist you. Thanks!
Problem Area
[ ] Page Transformation: Error during the use of page transformation from PnP PowerShell
[ ] Page Transformation: Error during the use of page transformation from .Net
[ ] Page Transformation: Page is not looking correct after transformation
[ ] Modernization Scanner: something went wrong...
Expected or Desired Behavior
If you are reporting a bug, please describe the expected behavior. If you are suggesting an enhancement please
describe thoroughly the enhancement, how it can be achieved, and expected benefit.
Observed Behavior
If you are reporting a bug, please describe the behavior you expected to occur when performing the action. If you are making a suggestion, you can delete this section.
Steps to Reproduce
If you are reporting a bug please describe the steps to reproduce the bug in sufficient detail to allow testing. Only way to fix things properly, is to have sufficient details to reproduce it. If you are making a suggestion, you can delete this section.
Submission Guidelines
Delete this section after reading

All suggestions or bugs are welcome, please let us know what's on your mind.
If you checked the "Page Transformation: Error during the use of page transformation from PnP PowerShell" checkbox then please provide:

The output you see in your PowerShell session
The output generated by calling Get-PnPException
The PnP PowerShell version. Run get-module -Name SharePointPnPPowerShellOnline and copy the version info

If you checked the "Page Transformation: Error during the use of page transformation from .Net" checkbox then please provide:

The exact error message you get
The stack trace of the error
The version of the SharePointPnP.Modernization.Framework.dll assembly

Thanks for your contribution! Sharing is caring.

Hi @Alok-sys123 ,
The scan run can be terminated due to various reasons (throttling, network connectivity issues,...). I would advice to split up the scan and run in an performance optimized mode. See https://docs.microsoft.com/en-us/sharepoint/dev/transform/modernize-scanner-faq for details on how to that. If you're still not able to complete a smaller scan run then please create a new issue sharing the exception that happens. Without exception details there's not much I can help with.

Hi @Alok-sys123 ,
The scan run can be terminated due to various reasons (throttling, network connectivity issues,...). I would advice to split up the scan and run in an performance optimized mode. See https://docs.microsoft.com/en-us/sharepoint/dev/transform/modernize-scanner-faq for details on how to that. If you're still not able to complete a smaller scan run then please create a new issue sharing the exception that happens. Without exception details there's not much I can help with.


Help setting up API key in HL2RP

Hi, two questions. One, I have my workshop collection, how do I hook the API key up to my server running off of NFOServers? Also, when setting the map rotation, will simply having the map in the collection work? Thanks!

This isn't a question for the Clockwork github, as it isn't Clockwork related. Put in a support ticket and or how to edit your server.cfg to link your server to your collection. This isn't the place for it.


feat: add allow empty coverage flag

:zap: Summary
Add an option to support empty coverage files without returning an error code.
:ballot_box_with_check: Checklist

[x] Add specs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

21 of 25 (84.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 89.439%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/coverage_reporter/cli/cmd.cr
15
19
78.95%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
686

Relevant Lines:
767

💛  - Coveralls


Managed Identity with Contributor role is unable to execute Microsoft.ContainerInstance/register/action

Use case: Run a deployment script post creating an Azure Key Vault to prevent public access to it. These network level settings are currently not offered in the schema for the Key Vault resource and so a PowerShell or AZ CLI command is the only option (besides the portal).
See gist of the template at https://gist.githubusercontent.com/indranil-chakraborty-thd/a00fcde354e9baadf9afac3a9b32f3c0/raw/989a4ff614adea8d9d5f704a75dd69f3cfa103a8/azuredeploy.json
Azure documentation here indicates that one can use use a Deployment Script to run a post-deployment PowerShell or AZ CLI command/script.
However, that involves using a Managed Identity that requires the 'Contributor' role on the subscription. Creating the managed identity and assigning it the Contributor role requires it to be assigned the ServicePrincipal as documented here or one runs into the timing issue documented here.
After overcoming these issues, I ran into an error stating that my Managed Identity did not have the authorization to perform the Microsoft.ContainerInstance/register/action, which lead me to read the Limitations as described here. At this point, I gave up and decided it would be easier to simply run an AZ CLI command to secure the vault.

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

ID: b5c4c8c5-7757-a969-5800-17f2f2312649
Version Independent ID: 9ced8dd5-43ca-6d1f-c054-842299db8327
Content: Use deployment scripts in templates - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/deployment-script-template.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @mumian
Microsoft Alias: jgao

@indranil-chakraborty-thd You might have tried this option, but I am cross verifying again. "Microsoft.ContainerInstance/register/action" registers the subscription for the container instance resource provider and enables the creation of container groups.
Have you tried manually registering the container instance using these steps and then deploy the template ?

Quick update: The 'Microsoft.ContainerInstance' resource provider was NOT registered and I did so manually. It is however confusing since the documentation here seems to suggest that "A user-assigned managed identity with the contributor's role at the subscription level is required if the subscription has not registered the Azure storage account (Microsoft.Storage) and Azure container instance (Microsoft.ContainerInstance) resource providers."

I have not yet tested the results, will update this thread after I do so.

@indranil-chakraborty-thd From the template you have provided, I could see that the managed identity has contributor access at resource group scope level. That might be the cause of the error you were prompted with. If that not the case, we might need to investigate further to understand the cause of the issue.

Also, have you had chance to test the template after registering "Microsoft.ContainerInstances" resource provider?

I have now tested the template after making 2 changes.

Registered the container instance manually as described here
Made the depends on explicit on the resource ID.

changed to

It now works as a standalone script but I am continuing to test to see if it works as part of a longer orchestration.
Thank you for you assistance.

@indranil-chakraborty-thd We will close the issue for now. If there are further questions , please revert and we will be glad to assist you.


Update window-styles.md

Typo correction

@sylistine : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Make ui of Arito better

Make more interactive and responsive page for good user experience

i am unable to push the branch can u plz grant me permission to push the changes or is there any other problem.

I'm a GSSoC contributor ..
I would like to work on this section...
would you please assign me to this task ??

I'm a GSSoC contributor .. I would like to work on this section...
would you please assign me to this task ??

Hey, @Aliza30 do you want to work on this issue?
If yes then explain to us in detail how you will implement this feature and also provide some design prototypes.
For now, I am temporarily closing this issue. If you want to work on this issue reopen it and start contributing.


kata-runtime installation fails due to unsatisfiable vanilla-qemu dependency

Description of problem
Executing:
BRANCH=stable-1.11 bash -c "$(curl -fsSL https://raw.githubusercontent.com/kata-containers/tests/master/cmd/kata-manager/kata-manager.sh) install-packages"

on Debian 10 results in:
The following packages have unmet dependencies:
 kata-runtime : Depends: qemu-vanilla (= 4.1.1+git.99c5874a9b-5) but 4.1.1+git.99c5874a9b-2 is to be installed

Expected result
qemu-vanilla=4.1.1+git.99c5874a9b-2 should be sufficient - the commit hash is the same, so the package should have the exact same content. (Wondering, if this caused by a typo in the dependencies somewhere?)
Actual result
see Description of problem

Just found that issue in the packaging repo: https://github.com/kata-containers/packaging/issues/1051
Is this related?

I think so, since I don't see Debian 10 here https://build.opensuse.org/package/show/home:katacontainers:releases:x86_64:master/qemu-vanilla, I guess it's time to check if QEMU 5 works on Debian 10

@devimc there is Debian 10 build inside http://download.opensuse.org/repositories/home:/katacontainers:/releases:/x86_64:/stable-1.11/ and this references qemu-vanilla in the kata-runtime package, but the difference between the unmet dependency and the available dependencies is only the -5 / -2:
4.1.1+git.99c5874a9b-5 #kata runtime depends on -5
4.1.1+git.99c5874a9b-2 #debian 10 provides -2

So, the -5/-2 indicates the QEMU version?
And can I install kata containers in Debian 10 with another runtime instead of vanilla-qemu?

So, the -5/-2 indicates the QEMU version?

good catch, I have no idea why the -2 in qemu-vanilla
cc @rhafer @jodh-intel

And can I install kata containers in Debian 10 with another runtime instead of vanilla-qemu?

vanilla-qemu is not a runtime, it's the hypervisor, kata support other hypervisors like , cloud-hypervisor, firecracker and ACRN, but if you still want to use QEMU, install it form the debian repos (apt-get install qemu-system-${arch}) and change the path in the configuration file /usr/share/default/kata-containers/configuration.toml

@devimc thanks for clarifying! I was thinking of "vm runtime" ;-) hypervisor is the better term ... way too many things are called runtime these days and I have been reading way too much documentation the last couple of days :-D
Regarding firecracker, I would go straight for this, but the dependency problem with qemu-vanilla when installing kata-runtime persists, unless I compile kata-runtime myself from source? (which in turn excludes the installation from simple package manager updates - is there a guide on how to manually update a from-source-installation somewhere?)

Regarding firecracker, I would go straight for this, but the dependency problem with qemu-vanilla when installing kata-runtime persists, unless I compile kata-runtime myself from source? (which in turn excludes the installation from simple package manager updates - is there a guide on how to manually update a from-source-installation somewhere?)

try adding  --ignore-missing to the apt-get command

go get -d -u github.com/kata-containers/runtime
package github.com/kata-containers/runtime: no Go files in /root/go/src/github.com/kata-containers/runtime

this is a warning, not an error
maybe you want to take a look to the kata containers snap https://snapcraft.io/kata-containers

@devimc --ignore-missing seems to have no effect, I still can't install kata-runtime using: apt-get --ignore-missing kata-runtime results in same depdency error as before.
Continuing the build from source with make despite the warning works. If there is a migration guide to switch to the package manager version of kata-runtime later, once it works again, I would consider using it in production already. If not, it's probably better to stay on debian 9 and wait for the issue to be resolved in the package dependencies before doing a dist-upgrade + a kata-runtime update.
Regarding snaps, there is quite some critism in the different linux communities, so I am hesistant to use it - and https://forum.snapcraft.io/t/disabling-automatic-refresh-for-snap-from-store/707/282 is still an ongoing discussion (which would make me not use it in production environments yet). Besides, installing your stack at least once using the existing documentation teaches quite a bit about dependencies and helps in tracking down problems later on. From the snap website it is also hard to see, what's inside the snap (e.g. is firecracker also in there? If not, how is it possible to use different hypervisors using the snap?).

@sray thanks for the link, yeah maybe you should wait a little bit more before using Debian 10 + Kata in production, maybe @amshinde or @chavafg can help us with the packaging issue

good catch, I have no idea why the -2 in qemu-vanilla
cc @rhafer @jodh-intel

I am not entirely sure what that is as well. It seems to be a the Debian dsc equivalent for the Release Tag in RPM spec files. Though I am not sure why the deb package are including that in their dependencies. The specfiles don't. See:
https://github.com/kata-containers/packaging/blob/master/obs-packaging/runtime/kata-runtime.dsc-template#L22
vs:
https://github.com/kata-containers/packaging/blob/master/obs-packaging/runtime/kata-runtime.spec-template#L31
Also note that the latest build of qemu-vanilla for debian 10 failed: https://build.opensuse.org/package/live_build_log/home:katacontainers:releases:x86_64:stable-1.11/qemu-vanilla/Debian_10/x86_64
That one would have gotten the -5 in its version string. At least according to the dsc file (https://build.opensuse.org/package/view_file/home:katacontainers:releases:x86_64:stable-1.11/qemu-vanilla/qemu-vanilla.dsc?expand=1)

thanks @rhafer , I think you're right it's using qemu -2 because qemu -5 failed, I hope this issue gets fixed in kata 1.12 since it uses QEMU 5

The issue has been fixed as far as we know.


Fixed firewall in N-S scenario, Added nfp in place of nsf

@njagadish @rajendramachani @YogeshRajmane

Changes are looking fine. Some pep8 warnings regarding visual indentation are there. They can be fixed.

Closing this PR, raised with april_15 branch - https://github.com/oneconvergence/group-based-policy/pull/214


feat: add support for compiler options

https://testcafe.io/documentation/402638/reference/configuration-file#compileroptions

Thank you for your contribution to TestCafe. We will review this PR.

Please inform us if you want to keep working on this PR. For now, I close it to keep the repository clean.


Remote - SSH not available

Remote - SSH not available
after updating the Visual Studio Code V1.70.0, it doesn't work the Remote - SSH with connecting and installing the VS Code Server in downloading.
it stuck in downloading the VS Code Server.
The target is the Raspberry pi OS Buster kernel 5.10.103.

It installed the VS Code Server in RPi for long time.
I tried it on the Windows 11.
Is it the downloading server was busy?


Import * from models doesn't work

This isn't super important, but it would be nice to do this in a pcms shell sometimes: "from pyramidcms.db.models import *", however it complains that the list of models in __all__ must be in string form, not a list of actual classes.
It should be fine to change this to a list of strings, e.g __all__ = ["User", "Group", "Permission"], however any code that is still expecting a list of models in __all__ has to be fixed up too.

There is a bit of code in modes/__init__.py where it loops over the list of models in __all__ to dynamically add the SomeModel.objects.model property.
If we make the list of models in all strings instead, we are going to have to fix that code that adds the property to Model.objects, but we need to be able to retrieve the model class from the string with the model in this loop for example "User" should get the User model.
Since these models are all imported in this same module anyway, you might be able to use getattr() on the module to get the model class from the model string.

Maybe storing the models as a list of strings in __all__ then converting them back to models from string is the wrong approach.
Maybe best to go the other way... store a second list in models/init.py of all the model classes, then build __all__ form this list by getting the class name (probably model.__class__.name).

I've already got a fix for this, pushing in a second.


Default lineHeight setting not accounted for

Need to account for fontSize values like [ '0.75rem', { lineHeight: '1rem' } ] when normalising values.

Partially fixed in e18c905, but some changes to account for v2 font-size configs could still be needed, namely the default lineHeight for a fontSize.

--line-height-px should be replaced with --line-height-offset.


can't compute cl_00 of a map with healpy nside smaller than 32

Tried to use the code "example 1" on [https://namaster.readthedocs.io/en/latest/sample_simple.html] to compute cl_00 (TT) of temperature maps with different nside. The code failed to run and gave error "zsh: segmentation fault" (when ran on jupyter notebook the kernel died) with nside = 4, 8 16, but ran successfully at nside=32 and 64. This is confusing as NaMaster should work for any nside.

Humm, I'm not able to reproduce this error. Did you install namaster from pip?

Humm, I'm not able to reproduce this error. Did you install namaster from pip?

Below is the code I used. The map I used is just random values with different nside. Again it works for nside>=32.
namaster-example1.txt

Below is the code I used. The map I used is just random values with different nside. Again it works for nside>=32.
namaster-example1.txt

@damonge, I can reproduce the error on a mac-os based system with a conda install. I'll take a deeper look at it (it breaks in the apodization part already).

@damonge, I can reproduce the error on a mac-os based system with a conda install. I'll take a deeper look at it (it breaks in the apodization part already).

OK, thanks! I can see how apodization might fail. Can you check if that's the only place it fails?

OK, thanks! I can see how apodization might fail. Can you check if that's the only place it fails?

Yeah, the rest works. Here's what it says when it breaks (and you set OMP_NUM_THREADS=1.
//anaconda3/lib/python3.7/site-packages/pymaster/utils.py in mask_apodization(mask_in, aposize, apotype)
    128     """
    129     return lib.apomask(mask_in.astype("float64"), len(mask_in),
--> 130                        aposize, apotype)
    131
    132

RuntimeError: Not enough memory in listir

Yeah, the rest works. Here's what it says when it breaks (and you set OMP_NUM_THREADS=1.
//anaconda3/lib/python3.7/site-packages/pymaster/utils.py in mask_apodization(mask_in, aposize, apotype)
    128     """
    129     return lib.apomask(mask_in.astype("float64"), len(mask_in),
--> 130                        aposize, apotype)
    131
    132

RuntimeError: Not enough memory in listir

I'm checking the code and I'd say that the error comes from here but I don't know under what circumstances it would break with just one thread...

I'm checking the code and I'd say that the error comes from here but I don't know under what circumstances it would break with just one thread...

@kateyang0 you should be able to use the example with Nside=4 if you change the apodization radius to a larger value. For example:
mask = nmt.mask_apodization(mask, 7.7, apotype="Smooth") (anything larger than 7.7 should work).
TLDR;
I continued digging and the problem is that the apodization radius is too small for Nside=4 and that generates problems downstream. In particular, with 1 deg the variable lenlist0 in here
takes the value 0. Then, it gets divided by 2 in the for loop that follows (so anything lower than 2 can give potential problems). And even when it's larger than 2, it continues breaking further downstream. I guess that we should probably raise an exception if the apodization radius is "too small". I don't see an easy way around this really. Thoughts @damonge?

@kateyang0 you should be able to use the example with Nside=4 if you change the apodization radius to a larger value. For example:
mask = nmt.mask_apodization(mask, 7.7, apotype="Smooth") (anything larger than 7.7 should work).
TLDR;
I continued digging and the problem is that the apodization radius is too small for Nside=4 and that generates problems downstream. In particular, with 1 deg the variable lenlist0 in here
takes the value 0. Then, it gets divided by 2 in the for loop that follows (so anything lower than 2 can give potential problems). And even when it's larger than 2, it continues breaking further downstream. I guess that we should probably raise an exception if the apodization radius is "too small". I don't see an easy way around this really. Thoughts @damonge?

I continued digging and the problem is that the apodization radius is too small for Nside=4 and that generates problems downstream. In particular, with 1 deg the variable lenlist0 in here takes the value 0. Then, it gets divided by 2 in the for loop that follows (so anything lower than 2 can give potential problems). And even when it's larger than 2, it continues breaking further downstream. I guess that we should probably raise an exception if the apodization radius is "too small". I don't see an easy way around this really. Thoughts @damonge?

I continued digging and the problem is that the apodization radius is too small for Nside=4 and that generates problems downstream. In particular, with 1 deg the variable lenlist0 in here takes the value 0. Then, it gets divided by 2 in the for loop that follows (so anything lower than 2 can give potential problems). And even when it's larger than 2, it continues breaking further downstream. I guess that we should probably raise an exception if the apodization radius is "too small". I don't see an easy way around this really. Thoughts @damonge?

@fjaviersanchez What is the best way to estimate the reasonable apodize radius for each nside? Is there a relation between the two that we can use? I tried apodize radius of 7.7 for nside=4 but it still didn't work.I also tried apodize radius of 10 or 20, neither of them worked.  But the code worked if I remove the apodizing line of code and use the original mask.
Thanks for pointing this parameter out!

@fjaviersanchez What is the best way to estimate the reasonable apodize radius for each nside? Is there a relation between the two that we can use? I tried apodize radius of 7.7 for nside=4 but it still didn't work.I also tried apodize radius of 10 or 20, neither of them worked.  But the code worked if I remove the apodizing line of code and use the original mask.
Thanks for pointing this parameter out!

@fjaviersanchez thanks a lot. This is definitely a problem with the C-level apodizing function and we should just propagate the error to python (which for some reason isn't happening, I'll look into that).
@kateyang0 I'd suggest you work without any apodization. At these super large scales apodization shouldn't really matter (and if it does, you should be using a higher resolution map and mask, where this problem won't appear).

@fjaviersanchez thanks a lot. This is definitely a problem with the C-level apodizing function and we should just propagate the error to python (which for some reason isn't happening, I'll look into that).
@kateyang0 I'd suggest you work without any apodization. At these super large scales apodization shouldn't really matter (and if it does, you should be using a higher resolution map and mask, where this problem won't appear).

@kateyang0 this issue will be closed by #124, but let me know if you keep having trouble


Similar to how react work, or on events

Do proper state management.
Be able to call a function when a specific state changes.

I think this is referring to #299. It's so sad that we had thought of this so long ago but never implemented it. ):


Response empty with only Q&A

Hi Guys, i hope you can help me with a situation that is occurring when I call the /chat end-point. When i call the end-point, the response is always empty and I really don't know why.
Response:
{     "success": true,     "message": null,     "data": {         "nlu": {             "text": "Hi",             "intent": {                 "id": -5668266704242435470,                 "name": "greet",                 "confidence": 0.9999999403953552             },             "entities": [],             "intent_ranking": [                 {                     "id": -5668266704242435470,                     "name": "greet",                     "confidence": 0.9999999403953552                 },                 {                     "id": -1894405930850988402,                     "name": "new",                     "confidence": 6.276561492768451e-08                 },                 {                     "id": -6269350766581444242,                     "name": "bye",                     "confidence": 2.6635090577542542e-08                 }             ],             "response_selector": {                 "all_retrieval_intents": [],                 "default": {                     "response": {                         "id": null,                         "responses": null,                         "response_templates": null,                         "confidence": 0.0,                         "intent_response_key": null,                         "utter_action": "utter_None",                         "template_name": "utter_None"                     },                     "ranking": []                 }             }         },         "action": [             {                 "action_name": "utter_greet",                 "max_confidence": 0.9937130212783813,                 "policy_name": "policy_1_TEDPolicy"             },             {                 "action_name": "action_listen",                 "max_confidence": 0.9982038140296936,                 "policy_name": "policy_1_TEDPolicy"             }         ],         "response": [],         "slots": [             "doc_url: None",             "document: None",             "video: None",             "audio: None",             "image: None",             "order: None",             "kairon_action_response: None",             "bot: 65985df50ac8038d3104f6ae",             "session_started_metadata: {'tabname': 'default', 'is_integration_user': False, 'bot': '65985df50ac8038d3104f6ae', 'account': 2, 'channel_type': 'chat_client'}"         ],         "events": null,         "tabname": "default",         "agent_handoff": {             "initiate": false,             "type": null,             "additional_properties": null         }     },     "error_code": 0 }

Could you help-me with this issue?
Thank you very much.

Hi, it looks like your bot is hallucinating while trying to predict action. Can you verify if you have any intents or actions that are not linked to any story. You can check this by going to components screen:
1.

Verify if each intent has at least 1 story attached (see Corresponding Stories on right side of screen) by selecting each intent.

Similarly, verify for utterances and actions too.

If there are intent/actions/utterances that have no corresponding stories, then either create stories with those or delete them.
Let us know anyway so we can debug further.

Hi @udit-pandey, thank you very much for you awnser.
I change all my actions, utter and intentions. But the aplication still returns empty reponse :(
I create a new chatbot with the default project, but this new chatbot return empty too.
Attached is the downloaded model
659bed57ee3e1ffabfd45422.zip
Thank you very much.

Did you retrain the bot after removing the intents?

Yes, i retrained the bot. But it's don't work.
I really doin't know whats going on.

Thats strange because I trained a new bot based on the training data you shared and its working.
Will it be possible to share this bot with me (udit.pandey@digite.com)?
If yes, then please add me to the bot by going to Deploy >> Administration >> Add Members.


Use graphhopper in exiting project

Hi,  i'm try to use graphhopper for ios in existing prject.
i follow the instruction how to use it in Xcode but when im drag&drop the Translations and Libraries groups Xcoode is crashing.
Any idea how to solve it?
i also try to create the lib by build it separately (it build successfully) and drug the lib to my project, Xcode didn't crush, but i couldn't use the lib. it doesn't recognise it when i try to add GraphHopper.h to my bridge file (i'm using swift)
Any idea?
Thanks

Would you mind to create a new topic on our forum?

sure


Error with db:seed

When I make the command db: seed I get the following error: 'Undefined index: eloquentChecks'

Will fix it.  In a few hours
Thanks for report,  man.

try out the latest version. v2.2.30.
it should be fixed