Review Request - CAF Lifecycle Diagram

Dear Team,
The diagram in the section https://docs.microsoft.com/en-us/azure/cloud-adoption-framework/overview#understand-the-lifecycle doesn't state the position of Organize phase. Could you please have a look and see if it makes sense to have Organize phase also in the diagram?
Thanks,
Shekhar

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

ID: 66744a3f-7ab3-80ae-4c76-639ee78601bf
Version Independent ID: 081c5a49-8bd7-778e-da3f-11ad4667c4a2
Content: Microsoft Cloud Adoption Framework for Azure documentation - Cloud Adoption Framework
Content Source: docs/overview.md
Service: cloud-adoption-framework
Sub-service: general
GitHub Login: @BrianBlanchard
Microsoft Alias: brblanch

@shekhag-zz - thanks for your inquiry, and our apologies for the long delay.
The lifecycle is designed such that the Organize phase can be optional for companies. For some customers, it’s a critical component of adoption, whereas others will not require organizational changes. The diagram reflects this, so as to not suggest that customers are expected to reorganize as part of Azure adoption. We hope this helps.
#please-close


Eliminated mapping of native types to built in types

This PR eliminates internal resolution of native types to built-in types. This change should make the ecosystem of types more consistent by eliminating hardcoded types. It'll be possible to change native type by changing what's in-scope of definition module.
Changes

isPrimitive - now returns true only when a type has composed types.
microstates now exports import { Number, String, Boolean, Array, Object as ObjectType } from 'microstates'
removed getType

Note:  babel uses a function that calls a function on Object, when I import Object it overwrites the default Object and throws an error.

I'm using import { Object as ObjectType } from 'microstates', should we use import ObjectType from 'microstates' instead?


Feedback v1.4.1 gpu utilization suddenly drops to 0

Organization Name:
Hangzhou Innovation Institute, Beihang university

Short summary about the issue/question:
After I upgraded the cluster version from 1.2.1 to 1.4.1, the gpu utilization of all tasks on a host suddenly dropped to 0, but this phenomenon did not occur in the previous version. When viewing the physical host, no abnormalities were found. When I forcibly restart the host, the problematic gpu can be used normally again.

Brief what process you are following:

How to reproduce it:

OpenPAI Environment:

OpenPAI version: 1.4.1
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release): ubuntu1604
Kernel (e.g. uname -a):
Hardware (e.g. core number, memory size, storage size, GPU type etc.): 8card v100
Others:

Anything else we need to know:

Please confirm if node-exporter is working properly by: kubectl get po | grep node-exporter.
Also, is the problem solved after you restart the host?

Please confirm if node-exporter is working properly by: kubectl get po | grep node-exporter.
Also, is the problem solved after you restart the host?

sorry,I have restarted the machine. This problem does not appear after restarting, and it has not reproduced yet

OK. Close this issue for now.


Send file image doesn't work

Every time i wan't to send some images, it looks broken.
Thanks.

Can you be a little more specific on this? A screenshot would also help.

Yes, when using this functionality

I didn't mean a screenshot of the button which looks perfectly fine, but a screenshot of what looks broken. Sorry, I don't get it.

I just got the same issue so let me describe:
When I click on the upload image button and go to a picture, click on the space bar Goofy crashes. Besides that after I restarted I can select any file and it just doesn't upload, really weird.
Another question, can we add the old upload feature back next to this? Was a lot easier in quite some situations :) (and can we add support for PDF's)

Oh, now I understand. The image doesn't "look broken" as @jopereyral said, but the upload isn't working at all. Yes, I'll look into that.

Replicated this myself on Mac OS X 10.10.3.
Selecting a photo in the finder window then pressing space bar (in order to preview the image) crashes the app. A snippet of the error log if that helps..?
Double clicking or selecting and then pressing "Open" closes the upload window with no visual feedback indicating error, other than the photo not uploading at all.

Still happens in 2.1.3. When you hit the picture icon a file browser opens with which you can select an image, however upon selecting the image nothing happens. You must log on to messenger.com to actually send an image

Confirming that uploading is failing as of today on 2.1.3...

+1

Fixed, created pull request here https://github.com/danielbuechele/goofy/pull/131


Reconfigure test runner to run in node (v4+)

And get them passing on travis-ci

This is done, see https://travis-ci.org/kevinbarabash/debugger/builds/101373115.


[BACKPORT-0.6][Quantization] Fix annotation for multiply op (#4458)

https://github.com/apache/incubator-tvm/pull/4458
@yzhliu

I'll fix the failure in ci.


Size of generated libCardIO.a is too large (155.9 MB).

After building the project in XCode, the generated libCardIO.a in the Product folder of DerivedData is too large (155.9MB), while the static library in https://github.com/card-io/card.io-iOS-SDK/tree/master/CardIO is only 13.8MB.
How do I properly generate the static library? What am I missing here?
Thanks!

@pandeyGitHub did you try this?

@pandeyGitHub did you try the instructions for "Unofficial card.io-iOS-SDK release"?

Thanks for the response.
As per your suggestion I tried running --> fab build:outdir=~ . I get the following error.
$ fab verbose build:outdir=~
[localhost] local: xcode-select -p
Setup
All .strings files appear to be correct.
Will save release sdk to /Users/Pandey
[localhost] local: git describe --match='iOS_[0-9].[0-9]*' --tags --always --dirty
Fatal error: local() encountered an error (return code 128) while executing 'git describe --match='iOS_[0-9].[0-9]' --tags --always --dirty'
Aborting.
local() encountered an error (return code 128) while executing 'git describe --match='iOS_[0-9].[0-9]' --tags --always --dirty'

@pandeyGitHub try entering that git describe... command directly on the command line yourself. Hopefully this will produce a more detailed error message that will help point to the source of the problem.

@dgoldman-ebay
Thanks!

It's working now.  I tried git describe... command as suggested by you but it didn't work because I was not using the cloned version of the repo. I had directly downloaded the card.io-iOS-source.zip file. So I cloned the repo into my machine henceforth. And everything seemed to work out fine.

One more question though. I'm trying to modify the project to suit my own needs. When I add new Classes in the card.io-iOS-source project, and use the generated libCardIO.a in my own separate SampleApp, I'm getting the linker error, during the build process of my SampleApp.
Undefined symbols for architecture arm64:
"_OBJC_CLASS_$_TestViewController", referenced from:

objc-class-ref in HomeViewController.o
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation).
On the other hand, when I generate the libCardIO.a doing a normal build (from XCode IDE) instead, I'm able to build and use the static library without any such linker error issues. What could be the problem?

@pandeyGitHub When building a release version of card.io, we do a moderate amount of finicky work to limit the number of symbols which the library exposes. This is because card.io uses the OpenCV library, which resulted in symbol conflicts for developers whose apps independently included OpenCV.
The symbols which are to be publicly exposed are listed in the file CardIO_exported_symbols.txt.
I think that your problem might be solved if you add OBJC_CLASS$_TestViewController to that file. (You might need a leading underscore character, as with the other symbols already listed in the file. Experiment and learn! :smile_cat:)


simulations get slow when the number of simultaneous signals reach 40k

Now, the performance is aceptable until the number of simultaneous particles is above 200k, which is more than the number of axoms
closed


Bluetooth: Controller: Fix BT_CTLR_FAST_ENC Kconfig help text

Fix BT_CTLR_FAST_ENC Kconfig help text to reflect the
current Encryption Setup Procedure behavior.
With the split architecture, Encryption Setup Procedure
will take 4 connection events when BT_CTLR_FAST_ENC is
enabled, in comparison to 3 connection events in the
legacy Controller architecture. This is due to split
architecture processes control procedures in the lower
priority Upper Linker Layer execution context.
Signed-off-by: Vinayak Kariappa Chettimada<EMAIL_ADDRESS>

Not backporting because this is not a real bugfix.


Proposal: zaptar rune for statically typed compilation and evaluation

@benjamin-tlon @pilfer-pandex @philipcmonk @joemfb @eglaysher
I propose a new rune for typed runtime compilation and evaluation. This is not obviously possible in a statically typed language without dependent types, but due to Hoon's compilation to Nock, we can do it safely and without performance penalty. For an intriguing (and much more complicated) analog in System F, see Self-Quotation in a Typed, Intensional Lambda-Calculus.
This does have to be done through a rune to get the static proof of type safety, and it requires adding a new biblical. I have a partial implementation, but thought I'd present it early on to get feedback before putting real work into it.
The rune compiles a Hoon AST passed in at runtime against a statically specified subject type and result type, then evaluates the resulting Nock using a subject specified at runtime.
The syntax looks like this:
!*  result-example  subject  some-hoon

If the compilation fails, the expression crashes. Deterministic crashes can be caught by virtualizing with +mule, as usual.
If the compilation succeeds, we have a new value whose type is statically known, even though it was compiled at runtime. This would remove the need for vases in several parts of the system, which I believe will spark joy.
I've written a toy example of how Gall could use this if we had a static type for apps. The Arvo kernel could also use this if vanes had a static type. As @frodwith suggested, +eve itself could be jetted as a way for the runtime to make use of Hoon types for optimization. The runtime could also memoize it, replacing the Ford cache. This rune does not require any runtime support, though.
Note the conspicuous absence of vases:
::  |app-runner: like Gall, but statically typed
::
=>  |%
    +$  app-event   [%foo @]
    +$  app-effect  [%bar ^]
    +$  app-core
      $_  ^?
      |%
      ++  poke  |=(app-event [(list app-effect) app-core])
      ++  peek  |=(path (unit (unit [mark *])))
      --
    --
::
|_  apps=(map @tas app-core)
++  app-runner  .
::
++  poke-app
  |=  [app-name=@tas =app-event]
  ^-  [(list app-effect) _app-runner]
  ::
  =/  app=app-core  (~(got by apps) app-name)
  =^  effects  app  (poke:app app-event)
  =.  apps          (~(put by apps) app-name app)
  ::
  [effects app-runner]
::
++  peek-app
  |=  [app-name=@tas =path]
  ^-  (unit (unit [mark *]))
  ::
  =/  app=app-core  (~(got by apps) app-name)
  (peek:app path)
::
++  install-app
  |=  [app-name=@tas source=@t]
  ^+  app-runner
  ::
  ::  compile the source to an +app-core, catching failure with +mule
  ::
  =/  eval-result=(each app-core tang)
    %-  mule  |.
    ~&  %compiling-app^app-name
    ::
    !*  *app-core  ..zuse  (ream source)  ::  <----  where the magic happens
  ::
  ::  print and no-op on failure
  ::
  ?:  ?=(%| -.eval-result)
    %-  (slog p.eval-result)
    app-runner
  ::
  ::  install the new app in our state
  ::
  =.  apps  (~(put by apps) app-name p.eval-result)
  ::
  app-runner
--

As for implementation, the rune's type signature looks like this:
[%zptr result=hoon subject=hoon =hoon]

:subject should evaluate to the subject against which the resulting Nock should be evaluated. Its type will be analyzed statically and used as an argument to +mint:ut at runtime. :result is an example result whose type will be extracted at compile time and passed into the +mint:ut runtime call to constrain the output type of the compilation. :hoon is an expression that should produce an AST to be compiled and run against :subject.
This ends up calling code that looks like this at runtime (+eve is the new biblical):
++  eve
  |=  [=result=type subject=[=type value=*] =hoon]
  ^-  *
  ::
  .*  value.subject
  +:(~(mint ut type.subject) result-type hoon)

The compilation of the rune in +mint:ut looks something like this:
  ?-  gen
  ...
        [%zptr *]
      ^-  [type nock]
      ::  obtain type from :result.gen, ignoring type from rest of expression
      ::
      :-  (play result.gen)
      ::  produce nock that runs the compiler using +eve
      ::
      =<  +
      %_    $
          gen
        ^-  hoon
        :+  %cncl  [%limb %eve]
        :~  [%tsld [%$ 2] %zpbn result.gen]
            [%zpbn subject.gen]
            hoon.gen
        ==
      ==
  ...
  ==

Inb4 we define +mint in terms of !* and use jets to break the circularity.  This would allow us to claim a lot fewer lines of code!
But seriously, I would love to see gall written in this way.  The types will still be non-trivial, though.  +app-core has to be a function of at least its state type if you ever intend to update it, since you'll need to either pull it out directly or make it produce its state with a ++stay arm.  But that means your (map @tas app-core) has to be heterogeneous, because it'll actually be (app-core talk-state), (app-core dojo-state), etc.  And wet cores don't nest, so (map @tas app-core) doesn't work because nothing nests in app-core.
I suppose you could get around this by requiring the app to (de)serialize to * or something, but we don't really want to make everyone write serialization code.

I thought about this a bit. It's true that the naive implementation of this idea loses the type of the app state for app upgrades.
One, somewhat lame, option is to have the new app clam its old state from a raw noun. The +stay code would be exactly the same, and +prep would just have an added call to a mold to validate the old state to a type before migrating it to a new type.
There are major downsides, though. Type validation is at least O(n) in the number of cells in the state, which could be quite large, and we also try not to force user apps to do their own validation.
There is an alternative solution to this problem. Instead of Gall recompiling a new modified app, Gall passes the source code to the app, and the app recompiles itself and performs its own type migration. This may sound like too much power for userspace code, but I think static type safety legitimizes it.
The app statically knows the type of its own state, so as long as the new code passed in provides a function from the old state type to the new state type, the new app can use the static type of its state as the argument to a function that produces a new generic app core from the old state.
This generic app core, of course, contains in its compiled Nock a representation of the type of its state, since it was compiled from Hoon that included a !* rune that closes over its own type. This effectively implements a chain of state types whose correctness is proven coinductively by the compiler.
This does mean an app would need to be compiled against a subject that includes the previous version of the app, so maybe there would be bootstrapping difficulties there? I'm not sure if that would actually cause problems in practice or not.
::  app door
::
|_  state
++  stay
  ^-  app-state-v1
  state
::
++  reload-from-source
  |=  source=@t
  ^-  app-core
  ::
  =*  self  ..reload-from-source
  ::
  =/  migrate
    !*  |=(app-state-v1 *app-core)
      self
    (ream source)
  ::
  (migrate state)
...
--

Does this work when for upgrading the kernel?  I don't see how we could swap in a new kernel under an app in this scheme.

One thing we could do is drop down to clamming using the +stay/+prep interface when the kernel changes. Arguably it's awkward to have two different upgrade mechanisms, but one is much more drastic than the other. Recompiling an app is a much smaller change from recompiling the compiler.
I'm not sure that's so terrible, actually. When the type of type changes, we usually have to do a recursive traversal of the type to convert it to the new format. In this case, we wouldn't bother traversing types at all; instead, we'd traverse the data to validate it.
Converting data instead of converting types might actually be easier to maintain; not sure.
I think it would also usually be easy to implement for apps:
++  prep
  |=  old-state=*
  ::
  =*  self  ..prep
  ::
  :-  moves=~
  self(state (app-state-v1 old-state))

imo introducing any kind of clamming seems like a dealbreaker - for
example, app state could contain gates. (It shouldn't, to avoid leaking the
kernel / losing updatability, but one obvious takeaway from the ph-bind
discussion is people are going to do it anyway)
From a more direct performance perspective, the vast majority of hoon.hoon
changes /don't/ change the type of type; which means types are not
traversed, and the cost is O(1)
On Tuesday, 16 April 2019, Ted Blackman<EMAIL_ADDRESS>wrote:

One thing we could do is drop down to clamming using the +stay/+prep
interface when the kernel changes. Arguably it's awkward to have two
different upgrade mechanisms, but one is much more drastic than the other.
Recompiling an app is a much smaller change from recompiling the compiler.
I'm not sure that's so terrible, actually. When the type of type changes,
we usually have to do a recursive traversal of the type to convert it to
the new format. In this case, we wouldn't bother traversing types at all;
instead, we'd traverse the data to validate it.
Converting data instead of converting types might actually be easier to
maintain; not sure.
I think it would also usually be easy to implement for apps:
++  prep
|=  old-state=*
::
=*  self  ..prep
::
:-  moves=~
self(state (app-state-v1 old-state))
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/urbit/arvo/issues/1146#issuecomment-483894927, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABxXhlgICcbBzwFMczXYwdA49miroqWUks5vhm8agaJpZM4cutbb
.

One old idea I do want to throw in here is the vane-only %meta .^
mechanism: .^($-(vase (unit foo)) %$ /arvo/devase/foo), for any zuse arm
++foo, can be implemented in arvo by the simple expedient of synthesizing a
nest check against the relevant arm. (It does constitute a sizable type
hole which allows you to create a mistyped noun from a malformed vase, so
don't do that; in particular do not run it on untrusted vases you got from
userspace)
On Tuesday, 16 April 2019, Anton Dyudin<EMAIL_ADDRESS>wrote:

imo introducing any kind of clamming seems like a dealbreaker - for
example, app state could contain gates. (It shouldn't, to avoid leaking the
kernel / losing updatability, but one obvious takeaway from the ph-bind
discussion is people are going to do it anyway)
From a more direct performance perspective, the vast majority of hoon.hoon
changes /don't/ change the type of type; which means types are not
traversed, and the cost is O(1)
On Tuesday, 16 April 2019, Ted Blackman<EMAIL_ADDRESS>wrote:

One thing we could do is drop down to clamming using the +stay/+prep
interface when the kernel changes. Arguably it's awkward to have two
different upgrade mechanisms, but one is much more drastic than the other.
Recompiling an app is a much smaller change from recompiling the compiler.
I'm not sure that's so terrible, actually. When the type of type changes,
we usually have to do a recursive traversal of the type to convert it to
the new format. In this case, we wouldn't bother traversing types at all;
instead, we'd traverse the data to validate it.
Converting data instead of converting types might actually be easier to
maintain; not sure.
I think it would also usually be easy to implement for apps:
++  prep
|=  old-state=*
::
=*  self  ..prep
::
:-  moves=~
self(state (app-state-v1 old-state))
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/urbit/arvo/issues/1146#issuecomment-483894927, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABxXhlgICcbBzwFMczXYwdA49miroqWUks5vhm8agaJpZM4cutbb
.


fixing up #258

just a few fixes to captions and references.
built locally, everything looks good.

Ah, thanks - I guess that means I don't need to go do that!
On Fri, Aug 14, 2015 at 4:07 PM, Joe Hamman<EMAIL_ADDRESS>wrote:

just a few fixes to captions and references.
built locally, everything looks good.
You can view, comment on, or merge this pull request online at:
https://github.com/UW-Hydro/VIC/pull/259
Commit Summary

fixing up #258

File Changes

M docs/Overview/ModelOverview.md
https://github.com/UW-Hydro/VIC/pull/259/files#diff-0 (42)

Patch Links:

https://github.com/UW-Hydro/VIC/pull/259.patch
https://github.com/UW-Hydro/VIC/pull/259.diff

—
Reply to this email directly or view it on GitHub
https://github.com/UW-Hydro/VIC/pull/259.

OK - I am going to assume this looks OK and merge


queryParams are not recognized in URL transition

When I pass URL into transitionTo, it seems not to recognize query params.
https://github.com/rwjblue/ember-router-service-polyfill/blob/master/vendor/ember-router-service-polyfill/index.js#L28-L31
It does the transition but the queryParams are ignored.

It was a typo :man_facepalming:


Jump to last button

Add a button like Hangouts or WhatsApp to scroll to the bottom of the message list.

Closing as implemented.


Script for Windows Server AD Creation results in password conversion error

(Windows Server 2016)
https://docs.microsoft.com/en-us/azure/active-directory/hybrid/tutorial-password-hash-sync#create-a-windows-server-ad-environment
Using the default powershell script that is provided I received the following error:
Can not bind parameter safemodeadministratorpassword. Cannot convert the "Pass1w0rd" value of type system string to type system.security.securestring.
The SafeModeAdministratorPassword parameter seems like it needs to be converted first.
Example fix:
#Declare variables
$DatabasePath = "c:\windows\NTDS"
$DomainMode = "WinThreshold"
$DomainName = "contoso.com"
$DomaninNetBIOSName = "CONTOSO"
$ForestMode = "WinThreshold"
$LogPath = "c:\windows\NTDS"
$SysVolPath = "c:\windows\SYSVOL"
$featureLogPath = "c:\poshlog\featurelog.txt"
$Password = (ConvertTo-SecureString -String "Pass1w0rd" -AsPlainText -Force)

#Install AD DS, DNS and GPMC
start-job -Name addFeature -ScriptBlock {
Add-WindowsFeature -Name "ad-domain-services" -IncludeAllSubFeature -IncludeManagementTools
Add-WindowsFeature -Name "dns" -IncludeAllSubFeature -IncludeManagementTools
Add-WindowsFeature -Name "gpmc" -IncludeAllSubFeature -IncludeManagementTools }
Wait-Job -Name addFeature
Get-WindowsFeature | Where installed >>$featureLogPath

#Create New AD Forest
Install-ADDSForest -CreateDnsDelegation:$false -DatabasePath $DatabasePath -DomainMode $DomainMode -DomainName $DomainName -SafeModeAdministratorPassword $Password -DomainNetbiosName $DomainNetBIOSName -ForestMode $ForestMode -InstallDns:$true -LogPath $LogPath -NoRebootOnCompletion:$false -SysvolPath $SysVolPath -Force:$true

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

ID: 938c7dba-41d9-92c4-fb2f-feeba1878c13
Version Independent ID: b221ae33-5625-b912-839c-14aeef5ac744
Content: Tutorial:  Integrate a single AD forest to Azure using password hash sync (PHS)
Content Source: articles/active-directory/hybrid/tutorial-password-hash-sync.md
Service: active-directory
GitHub Login: @billmath
Microsoft Alias: billmath

@illgitthat
Thanks for your feedback! We will investigate and update as appropriate.

@illgitthat Thanks for reporting this. I have updated the doc accordingly and it will be published shortly.


UnboundLocalError for 'pairs' - pcap_utils.py

Whilst trying to train a model I encountered the following error:
root@d3499990ed37:/app/NodeClassifier# python train_OneLayer.py ./Training firsttest.pickle
Reading data
Reading ./Training/WebTraffic-test.pcap as WebTraffic
Traceback (most recent call last):
  File "train_OneLayer.py", line 31, in <module>
    model.train(data_dir)
  File "/app/NodeClassifier/utils/OneLayer.py", line 133, in train
    labels=self.labels
  File "/app/NodeClassifier/utils/training_utils.py", line 67, in read_data
    capture_source = get_source(binned_sessions)
  File "/app/NodeClassifier/utils/pcap_utils.py", line 140, in get_source
    _, ip_mac_pairs = get_indiv_source(session_dict)
  File "/app/NodeClassifier/utils/pcap_utils.py", line 88, in get_indiv_source
    if is_private(source_address) or is_private(destination_address):
  File "/app/NodeClassifier/utils/pcap_utils.py", line 24, in is_private
    if pairs[0] == '10': private = True
UnboundLocalError: local variable 'pairs' referenced before assignment

Why would the pairs variable not be assigned?  From what I gathered reading pcap_utils, the address needs to be equal to or greater than 4 for 'pairs' to get set, would would cause the address argument to be less than that?

@jaiken06 this has been fixed in the overhaul branch, which isn't quite ready for merging into master yet as it's a big structural change.  let me push that fix into master in the meantime.


Entity AI has no dimension sanity checks (MC-149563)

I am currently running

SpongeForge version: 1.12.2-2768-7.1.6-RC3671
SpongeVanilla version: 1.12.2-7.1.6-RC165
Java version: Java 8
Operating System: Windows 10
Plugins/Mods: None

Issue Description
Entity AI has no dimension sanity checks, this can lead to players being attacked by mobs in other world, more detailed report on: https://bugs.mojang.com/browse/MC-149563
This is more plausable on server software that supports multiple worlds

Can reproduce using this code (not a valid plugin by itself; patch as needed). The smoke shows the location of the zombie in another dimension. Once the zombie dies from sunlight, both dimensions will be deleted. (Note that the entity activation range plugin must be turned off or patched in this method.)


fix: Add eks and base data models

Summary
Resolves: #58
Resolves: #33
Changes
This change introduces the new submodule models including models.eks and models.base.  The base submodule includes base models relevant to creating reusable models specific to this tool and AWS services.
User experience
This provides a cleaner API for interacting with resources within the tool both from CLI and programmatically (enabling further automation using this tool and library in the future).
Checklist
If your change doesn't seem to apply, please leave them unchecked.

[x] I have performed a self-review of this change
[x] Changes have been tested
[ ] Changes are documented

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Codecov Report
Base: 11.74% // Head: 19.50% // Increases project coverage by +7.75% :tada:

Coverage data is based on head (d6396bd) compared to base (b29d596).
Patch coverage: 81.60% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #59      +/-   ##
==========================================
+ Coverage   11.74%   19.50%   +7.75%
==========================================
  Files          10       12       +2
  Lines        1711     1907     +196
==========================================
+ Hits          201      372     +171
- Misses       1510     1535      +25

Impacted Files
Coverage Δ

eksupgrade/src/eks_get_image_type.py
14.54% <11.11%> (+0.81%)
:arrow_up:

eksupgrade/src/k8s_client.py
21.68% <50.00%> (ø)

eksupgrade/models/eks.py
86.41% <86.41%> (ø)

eksupgrade/models/base.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Someone please create a wiki page for "Family Plan" so I can populate it with information

I either don't have permission to create wiki pages or edit/add to them! Thank you.

Here it is :) Let me know if you still have trouble. https://github.com/HTBox/TwoWeeksReady/wiki/Family-Plan

Thank you for getting this page created. I don't however have permission to edit the page :(

Trying to figure that out now.
On Tue, Mar 2, 2021 at 4:53 PM MaximilianDixon<EMAIL_ADDRESS>wrote:

Thank you for getting this page created. I don't however have permission
to edit the page :(
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/HTBox/TwoWeeksReady/issues/40#issuecomment-789338747,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGKUVE4CCAOIASFHZLDQM3TBWB7FANCNFSM4YP2XRQA
.

--
Pascal Schuback
<EMAIL_ADDRESS>206.414.8799
@schuback


SQL queries not being logged on Application Insights for Azure Cloud Services

I have application Insights Agent installed on web role and worker role instances in Azure Cloud Service. The Dependency telemetry logs the full SQL query text for web roles but not for worker roles.
For web roles, the sdk version logged is rddp:2.9.1-26132
For worker roles, the sdk version logged is rddf:2.9.1-26132
The application insights agent is installed for both web roles and worker roles by adding a startup tag to call the install agent PowerShell script.
The environment variables for profiling on the worker roles are as follows:
COR_ENABLE_PROFILING=1
COR_PROFILER={324F817A-7420-4E6D-B3C1-143FBED6D855}
MicrosoftInstrumentationEngine_Host={CA487940-57D2-10BF-11B2-A3AD5A13CBC0}
Actual Behavior
Everything is logged correctly on Application Insights (Requests, Traces, Dependencies), but I don't see SQL Queries on SQL Server dependencies for Worker roles. Web roles have the SQL Queries logged.
Expected Behavior
SQL Queries should be displayed for Worker roles.
Version Info
SDK Version : rddf:2.9.1-26132
.NET Version : 4.8
Hosting Info: Azure Cloud Services. Web role and Worker Role

Worker roles are not supported, unfortunately. Only IIS based apps are supported.
https://learn.microsoft.com/en-us/azure/azure-monitor/app/asp-net-dependencies#advanced-sql-tracking-to-get-full-sql-query

I found this repo https://github.com/MohanGsk/ApplicationInsights-Home/tree/master/Samples/AzureEmailService
I have a similar code which does not log the SQl queries

Check my comment above. This is unsupported

Thanks! Could you please point to the exact section in the document which mentions worker role is not supported. I only see mention of Azure Cloud Service being supported.


Zero instances calls opengl generate errors.

Description

When i try call draw with 0 instances renderdoc shows errors.
This my render file:
Render.txt
GlDebugOutput isnt called. So i dont see errors in my program.
Opengl reference pages: https://www.khronos.org/registry/OpenGL-Refpages/gl4/html/glBufferData.xhtml
https://www.khronos.org/registry/OpenGL-Refpages/gl4/html/glDrawElementsInstanced.xhtml
I see zero is valid arg.
This renderdoc show:

Steps to reproduce
load this capture:
https://drive.google.com/file/d/1MdQlHLN-LT6uLGAkrt-UE2_EvJgLZYsK/view?usp=sharing

Environment

RenderDoc version: v1.11
Operating System: Windows 10
Graphics API: Opengl 4.6

AMD card.

The text of these errors does not mention the instance count at all. The errors are pointing out that the vertex buffer setup is invalid.
That commit will not check for validity if the draw is empty and has 0 vertices or instances.


ERROR: Protocol initialisation failed!

Have been trying for 3 hours to flash TWRP to a SMA500FU, and I don't know if it's a problem with the fact it's an A5, or if it's the build I'm using or what?
Here's the output of detect, and download-pit.
aliensonneptune@tinfoilhats:/$ sudo heimdall detect
Device detected
aliensonneptune@tinfoilhats:/$ sudo heimdall download-pit --output pit.pit
Heimdall v1.4.1

Copyright (c) 2010-2014 Benjamin Dobell, Glass Echidna
http://www.glassechidna.com.au/

This software is provided free of charge. Copying and redistribution is
encouraged.

If you appreciate this software and you would like to support future
development please consider donating:
http://www.glassechidna.com.au/donate/

Initialising connection...
Detecting device...
Claiming interface...
Setting up interface...

Initialising protocol...
ERROR: Protocol initialisation failed!

Releasing device interface...

aliensonneptune@tinfoilhats:/$

Can anyone help me?
PS, I'm an android noob, only been using it for 4 weeks.
PPS, I'm on a USB3, and the phone is on the 6.0.1 UD.

Use version 1.4.2 - I recommend compiling yourself

@JonnyTech I've built it from the repo, and I still get 1.4.1, built yesterday? When was 1.4.2 pushed?

Have you tried rebooting the phone and another USB port and/or cable?
Which version of libusb do you have installed? Version 1.0.21 works for me:
user@computer:~$ dpkg -l libusb-1*
Desired=Unknown/Install/Remove/Purge/Hold
| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend
|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)
||/ Name           Version      Architecture Description
+++-==============-============-============-=================================
ii  libusb-1.0-0:a 2:1.0.21-2   amd64        userspace USB programming library
ii  libusb-1.0-0-d 2:1.0.21-2   amd64        userspace USB programming library
ii  libusb-1.0-doc 2:1.0.21-2   all          documentation for userspace USB p

Make sure that you do not run Heimdall as root/sudo - I have had problems:
user@computer:~/Heimdall-master/build/bin$ ./heimdall info
Heimdall v1.4.2

Copyright (c) 2010-2017 Benjamin Dobell, Glass Echidna
http://www.glassechidna.com.au/

This software is provided free of charge. Copying and redistribution is
encouraged.

If you appreciate this software and you would like to support future
development please consider donating:
http://www.glassechidna.com.au/donate/

Heimdall utilises libusbx for all USB communication:
    http://www.libusb.org/

libusbx is licensed under the LGPL-2.1:
    http://www.gnu.org/licenses/licenses.html#LGPL

What are the output of these commands:
user@computer:~/Heimdall-master/build/bin$ ./heimdall detect
user@computer:~/Heimdall-master/build/bin$ ./heimdall download-pit --output pit.pit
user@computer:~/Heimdall-master/build/bin$ cat pit.pit

Tried reboot, other USB cable, ./heimdall detect came up with nothing, tried the 1st one again, and like before it detected on the original cable. Turns out my backup cable is faulty. After a complete sudo apt update && sudo apt upgrade, IT GOT THE PIT FILE! Here's my proof, though the .pit looks messy as hell, don't know if that's good?
aliensonneptune@tinfoilhats:~/musicphone/Heimdall/bin$ ./heimdall detect
Device detected

aliensonneptune@tinfoilhats:~/musicphone/Heimdall/bin$  dpkg -l libusb-1*
Desired=Unknown/Install/Remove/Purge/Hold
| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend
|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)
||/ Name                    Version          Architecture     Description
+++-=======================-================-================-====================================================
ii  libusb-1.0-0:amd64      2:1.0.20-1       amd64            userspace USB programming library
ii  libusb-1.0-0-dbg:amd64  2:1.0.20-1       amd64            userspace USB programming library development files
ii  libusb-1.0-0-dev:amd64  2:1.0.20-1       amd64            userspace USB programming library development files
ii  libusb-1.0-doc          2:1.0.20-1       all              documentation for userspace USB programming

aliensonneptune@tinfoilhats:~/musicphone/Heimdall/bin$ ./heimdall download-pit --output pit.pit
Heimdall v1.4.2

Copyright (c) 2010-2017 Benjamin Dobell, Glass Echidna
http://www.glassechidna.com.au/

This software is provided free of charge. Copying and redistribution is
encouraged.

If you appreciate this software and you would like to support future
development please consider donating:
http://www.glassechidna.com.au/donate/

Initialising connection...
Detecting device...
Claiming interface...
Setting up interface...

Initialising protocol...
Protocol initialisation successful.

Beginning session...

Some devices may take up to 2 minutes to respond.
Please be patient!

Session begun.

Downloading device's PIT file...
PIT file download successful.

Ending session...
Rebooting device...
Releasing device interface...

aliensonneptune@tinfoilhats:~/musicphone/Heimdall/bin$ cat pit.pit
v�4COM_TAR2MSM8916<EMAIL_ADDRESS>��FSG
�� secsec.dat
                �� TPAD
@pEFSefs.img.ext4�MODEMST1nvrebuild1.bin�MODEMST2nvrebuild2.bin�hBOOTboot.imgHxRECOVERYrecovery.img�(FOTA��7BACKUP�FSC�7SSD8@PERSISTpersist.img.ext4xHPERSDATApersdata.img.ext4�KSYSTEMsystem.img.ext4�P@CACHEcache.img.ext4W<EMAIL_ADDRESS>MD5md5.imgI���!SGPTsgpt.imgI�W!����%C��4=��
                                                        	��o!���p�%�<\T��Rq�
                                                                                �I']enw��T?8�Z�{E��cV	��E7O&���Q��B��4��4�����=
                         ό�'d�T}?��O���28��~�ڐe�V)G��<��`���y���d�"���A�9e�!Q�'���)�&!ũ��Գ�M��Mq}�iah���H�
�2�樦E�<�K6��OP�`G8�qА�?Ȁ ؖ,���ͫ����d5�+����0�

I sent pretty much that command to my phone, and it rebooted, actually I tried doing it using -- instead of partition name, and accidentally flashed to 18, which was BACKUP, the phone still boots though. Here is the output of the console command, now how do I boot into the recovery?
aliensonneptune@tinfoilhats:~/musicphone/Heimdall$ ./bin/heimdall flash --RECOVERY ort.img
Heimdall v1.4.2

Copyright (c) 2010-2017 Benjamin Dobell, Glass Echidna
http://www.glassechidna.com.au/

This software is provided free of charge. Copying and redistribution is
encouraged.

If you appreciate this software and you would like to support future
development please consider donating:
http://www.glassechidna.com.au/donate/

Initialising connection...
Detecting device...
Claiming interface...
Setting up interface...

Initialising protocol...
Protocol initialisation successful.

Beginning session...

Some devices may take up to 2 minutes to respond.
Please be patient!

Session begun.

Downloading device's PIT file...
PIT file download successful.

Uploading RECOVERY
100%
RECOVERY upload successful

Ending session...
Rebooting device...
Releasing device interface...

aliensonneptune@tinfoilhats:~/musicphone/Heimdall$

Well done! To boot to recovery after flashing press Power, Volume Up and Home Buttons at the same time. If the phone boots normally then you pressed the buttons too late. At this point your newly flashed firmware has been overwritten by the system. So you need to flash it again from Heimdall in bootloader mode then hold the the buttons as it finished and restarts.
Or you can use the --no-reboot flag to not reboot automatically which makes it easier to get ready and power down the device manually and boot up holding down the keys mentioned above.

I have tried booting, but it just shows the boot logo of my phone normally and some text in the top left. I thought it was loading TWRP in, but I left it for 20-25 mins then lost hope and held down all the buttons to boot normally? Is it the image or what? PS I'm using the latest image from here, have no idea if that's what I'm meant to be using?

Okay, found an Odin-flashable TWRP .tar.md5, unpacked it, and found an .img file, whick looked flashable. Flashed it, worked flawlessly, booted TWRP, found my SuperSU .zip, installed.... aaaaaaaaand now I'm stuck on the loading screen :/

Has anyone else got this issue? I am having exactly the same thing:
./heimdall print-pit --verbose --no-reboot
...
--- Entry #28 ---
Binary Type: 0 (AP)
Device Type: 2 (MMC)
Identifier: 73
Attributes: 5 (Read/Write)
Update Attributes: 1 (FOTA)
Partition Block Size/Offset: 61071327
Partition Block Count: 33
File Offset (Obsolete): 0
File Size (Obsolete): 0
Partition Name: SGPT
Flash Filename: sgpt.img
FOTA Filename:

Ending session...
WARNING: Empty bulk transfer after sending packet failed. Continuing anyway...
ERROR: libusb error -7 whilst receiving bulk transfer. Retrying...

Releasing device interface...

And my libusb
ii  libusb-0.1-4:amd64                                          2:0.1.12-30                                       amd64        userspace USB programming library
ii  libusb-1.0-0:amd64                                          2:1.0.21-1                                        amd64        userspace USB programming library
ii  libusb-1.0-0:i386                                           2:1.0.21-1                                        i386         userspace USB programming library
ii  libusb-1.0-0-dev:amd64                                      2:1.0.21-1                                        amd64        userspace USB programming library development files
ii  libusb-1.0-doc                                              2:1.0.21-1                                        all          documentation for userspace USB programming
ii  libusb-dev                                                  2:0.1.12-30                                       amd64        userspace USB programming library development files

Heimdall buit from git, ubuntu 17.04.
The phone is a Samsung Galaxy Note 3, SM-N900V. Any other solution would be amazing!

Wiped dalvik/cache/tried hard resetting, SOMEBODY HELP, PLEASE, MY PHONE ISN'T LOADING IN.

Did it boot after flashing TWRP, before installing SuperSU?

Yes, and in the time since I last commented, I figured out that if TWRP still booted in, I could use ADB sideload to load in a new ROM, which could either solve my problems or make them worse.
I'm an optimist.
SO.
I sideloaded a Lineage OS 14 ROM for the A500xx series, crossed my fingers and booted. Soon enough I was greeted by the amazing sight of a Lineage OS welcome screen. Currently sideloading opengapps for my device.
Somehow flashing TWRP to root, and failing, then trying to fix it gave me face unlock and a stock (finally) theme? Anyway, couldn't have done it without your help, thanks a lot @JonnyTech

@aliensonneptune - great result, glad that I could help, please close this issue now
@spmp - do not hijack issues, please start your own using the new issue button

@JonnyTech Sorry buddy, I thought it was the same issue as having the same error etc. Will start a new issue 8)

Same problem here, "Protocol initialisation failed".
I've noticed though that the first heimdall command did execute successfully... so I kept rebooting the phone out of download mode (hold all buttons for ~5 seconds) then back into download mode, then issue the next heimdall command.
TL;DR only the first command after booting into download mode works!

Thanks dandv! Indeed, only first command after booting into download works.

um, help? I have the same error but I'm trying with the kernel sudo heimdall flash --kernel zImage but it told me :
`Initialising protocol...
ERROR: Failed to send handshake!ERROR: Protocol initialisation failed!
Releasing device interface...
`
is there any solution please? I'm trying to root my samsung A2 core

Did you issue that command directly after booting the phone into download mode? If you read this thread you will learn that only one command after each reboot works.

owh, okay

I have tried everything here, I always get:
Initialising protocol...
ERROR: Protocol initialisation failed!

Releasing device interface...

Are you sure that you are in odin/download mode?: https://www.androidsettings.net/samsung-galaxy-a2-core-download-mode/

You might also want to try a newer build from a fork with a few fixes. It may or may not help...

I am trying "only" to print-pit for now a Samsung Galaxy S21 (G991B), under Debian Bookworm, and I also get the same libusb errors:
Initialising protocol...
ERROR: libusb error -7 whilst sending bulk transfer. Retrying...
ERROR: libusb error -7 whilst sending bulk transfer. Retrying...
ERROR: libusb error -7 whilst sending bulk transfer. Retrying...
ERROR: libusb error -7 whilst sending bulk transfer. Retrying...
ERROR: libusb error -7 whilst sending bulk transfer. Retrying...
ERROR: libusb error -7 whilst sending bulk transfer.
ERROR: Failed to send handshake!ERROR: Failed to receive handshake response. Result: -7
ERROR: Protocol initialisation failed!

Heimdall version 1.4.2
$ heimdall version
v1.4.2
$

and libusb versions:
$ dpkg -l libusb*
Desired=Unknown/Install/Remove/Purge/Hold
| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend
|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)
||/ Name                     Version      Architecture Description
+++-========================-============-============-=====================================================================
ii  libusb-0.1-4:amd64       2:0.1.12-32  amd64        userspace USB programming library
ii  libusb-1.0-0:amd64       2:1.0.26-1   amd64        userspace USB programming library
ii  libusb-1.0-0-dev:amd64   2:1.0.26-1   amd64        userspace USB programming library development files
ii  libusb-1.0-doc           2:1.0.26-1   all          documentation for userspace USB programming
ii  libusb-dev               2:0.1.12-32  amd64        userspace USB programming library development files
ii  libusbredirhost1:amd64   0.13.0-2     amd64        Implementing the usb-host (*) side of a usbredir connection (runtime)
ii  libusbredirparser1:amd64 0.13.0-2     amd64        Parser for the usbredir protocol (runtime)

Heimdall can see the device:
$ heimdall detect
Device detected

A bit weird here but it's there
$ lsusb
Bus 002 Device 008: ID 04e8:685d Samsung Electronics Co., Ltd GT-I9100 Phone [Galaxy S II] (Download mode)

Anything else I can try?

I have written more about this error here: Troubleshooting: Heimdall Fails at Initialising Protocol
In short:

Scenario 1: the first command in bootloader/download mode works, and subsequent commands fail -> either it's a limitation of the device, or properly using Heimdall's "--no-reboot" and "--resume" options may overcome it.
Scenario 2: every command (except "detect") in bootloader/download mode fails and you're on an OS similar to Ubuntu (that may well include Debian) -> there is a fix that's not in v1.4.2 in this repo, but that is available in various other people's updated versions of Heimdall, such as Grimler's Heimdall .

Possibly same issue as fixed by PR #478


Wallet connect dialog v2

connected with:

#4144
#4303
https://github.com/kodadot/nft-gallery/issues/4720

Basics

this is how it will look like when you open the sidebar on
darker screen, black bg with 17% opacity
I experienced with background blur but its not that usable when then displaying wallet.

Phases
warning

confirmation

primary button there

Connect wallet
Wallet providers as dropdowns, you can only see wallet you have downloaded.
Other wallet are hidden under "more option"

these wallets have icon next to it co it indicates you can download them.

At the bottom there is dialog with introducing into wallets and link to a educational video in case you are a beginner.

Wallet

we are moving it to the sidebar

hover states
wallet provider:

same as the search results, when you hover over, the background turns into k-accentlight2 (lighter version than k-accentlight), in dark mode its k-accentlight2dark

sub-select:

when you click the chevron turns down and reveals your accounts.
there is a name and short adress
next to the icon there is line, like in vscode etc. (its clear that its sub section)
the line is only big as the max height of the content
hover effect starts at the line but is slightly bigger vertically, should have same size as the wallet provider hovers.
there are different screens that show it.
this shows the hover on account:

More options:
default state is k-grey, when hovering it turns into k-black

when clicking it displays the other options/wallets you can download

Cross icon:
turn into k-hovergrey, in case of darkmode its k-shade.
there is a close button at the top right in every sidebar + a clickable button (with left chevron) you can use to close it as well.

Mobile
on mobile the wallet section in menu stays the same. We wont change it.
We are going to change only the connect part.
It will popup over the whole page - it should be a overlay so once you close it you stay where you had been last time.
Darkmode
pretty straightforward, hover on the bigger areas have k-accentlightdark instead of the k-accentlight2.
If you have any questions about the dark mode feel free to ping.
Figma
designs are now available in gallery item handoff.

https://www.figma.com/file/gHSzqGPeQ9qK4L0qIDcTjY/gallery-item-handoff?node-id=1270%3A5615&t=s0dzfVDc0DFz66R4-1
Thanks!

@exezbcz will soon drop the hand-off file for this, and we can start work on this 💪

is this only for mobile?

is this only for mobile?

Nope

I guess time for @exezbcz to make issue with design and description this week so we can start working on this for next release

updated ✔

@Jarsen136 let's take this one

👋

I wonder which URL should we open when clicking this one. 👀  @yangwao

oh, seems onboarding materials are missing
I found only this available on docs https://docs.kodadot.xyz/tutorials/how-to-kodadot-phone-ios.html#what-we-need
oh ok, probably we can put it this one there https://docs.kodadot.xyz/tutorial-overview.html


merge 2.12.x to 2.13.x, unfreeze, refreeze

notably, this adds the configs/unfrozen-refs.conf file from 2.12.x (but with appropriate 2.13.x contents), so we have both the frozen and unfrozen states always in version control

queued: https://scala-ci.typesafe.com/job/scala-2.13.x-integrate-community-build/2631/
PR doesn't include a refreeze commit yet (since the freeze script needs a Jenkins run to use as reference)

I'm hitting a tough problem with scala-parallel-collections where if I use sbt 1.3.2, junit somehow isn't on the classpath, but I can't downgrade to 1.2.8 because the latest mima doesn't work there
it were some random third-party project I'd probably just freeze or fork it, but scala-parallel-collections is one way where it seems pretty important that we track master
I will dig into the junit classpath thing and try and find a solution or workaround, I already tried and failed once a week or two ago, but I'll try again.

I found a workaround, useCoursier := false, surely I tried this at least once before? but perhaps with some other combination of settings that made it fail? anyway, seems to be okay for now, 🤞

needs revisiting/reporting

scala-parallel-collections freeze

done
akka
test failure reproducible outside dbuild, reported upstream
worked around it by excluding test
maybe done
scalameta
weird coursier dependency problem, let's just freeze scalameta
and worry about it some other time
monix
https://scala-ci.typesafe.com/job/scala-2.13.x-integrate-community-build/2641/artifact/logs/monix-build.log
[monix] [error] **** Missing dependency: the library io.monix#implicitbox is not provided (in space "scala.main") by any project in this configuration file.
[monix] [error] In order to control which version is used, please add the corresponding project to the build file
ah, just need to add this:
// dependency of monix
${vars.base} {
name: "implicitbox"
uri:  ${vars.uris.implicitbox-uri}
extra.projects: ["implicitboxJVM"]
}
twitter-util
weird ParRange compilation error
investigation steps:

try to reproduce outside of dbuild

first try 2.13.1: ++2.13.1 util-stats/Test/compile

result: compiles

then try 2.13 nightly: ++2.13.2-bin-520a88e

result: compiles

then switch to sbt 1.3.2
try 2.13.1 again: ++2.13.1 util-stats/Test/compile

result: compiles

then try 2.13 nightly: ++2.13.2-bin-520a88e

result: compiles

okay, let's see what we can do within dbuild

first cut the config file down:

depends on: cloc-plugin, jackson-module-scala, scala, scala-collection-compat, scala-parallel-collections, scala-parser-combinators, scala-xml, scalacheck, scalatest
just build util-stats, not other subprojects
run it and verify it fails. DONE

now try rewinding both t-u and s-p-c to sbt 1.2.8

possible next steps:

rewind twitter-util to old sbt?
rewind s-p-c to old SHA + sbt 1.2?

if we use old sbt, we have to either rewind SHA or fork to get rid of mima
rewinding all the way to v0.2.0 runs into a source incompatibility
let's try rewinding to effa33489bc02723a1aba1e36911128a500dfc68 (May 2019)
which is where the needed source change was made
aha, that fixed it!  okay, so continue bisecting

use extra.test-tasks: ["compile"] to speed up bisect
using git bisect, but manually pasting the SHAs into configs/project-refs.conf,
I don't think it's worth automating?
the bisect is a bit tricky because I need to adjust the s-p-c entry in the config
file to include the right workarounds for the sbt version in question

gah, during bisect I ran into java.lang.NoClassDefFoundError: xerial/sbt/Sonatype$
during dependency extraction of s-p-c. under what circumstances does this happen?
I still don't know, despite having seen it a lot.  I'll try git clean -ffxd --
but does that fix it? no. neither does setting sbt version to 1.3.0. I guess it's
a problem with sbt-scala-module" % "2.1.0"?  I'll git bisect skip it

There are only 'skip'ped commits left to test.
The first bad commit could be any of:
156ed3073bda636b51cbeeca7072acc9a63d14b0
dab76f9c69bcf0a45ae8904acc8469ae4d8bdaee
a2dd4c1ced7d89af34fd8781ea8e8d6a190285e2
We cannot bisect more!

what about looking at the s-p-c JAR and seeing if the classes are missing?
hmm, what about trying to reproduce it outside dbuild using publishLocal?
scala-js
https://scala-ci.typesafe.com/job/scala-2.13.x-integrate-community-build/2641/artifact/logs/scala-js-build.log
[scala-js] [error] /home/jenkins/workspace/scala-2.13.x-integrate-community-build/target-0.9.16/project-builds/scala-js-a976b60efd6a19472ddcddc97bf7c03e98baaa8f/linker/jvm/src/main/scala/org/scalajs/linker/frontend/optimizer/ParIncOptimizer.scala:40:44: Symbol 'type scala.collection.generic.ParMapFactory' is missing from the classpath.
[scala-js] [error] This symbol is required by 'object scala.collection.parallel.mutable.ParTrieMap'.
[scala-js] [error] Make sure that type ParMapFactory is in your classpath and check for conflicting dependencies with `-Ylog-classpath`.
[scala-js] [error] A full rebuild may help if 'ParTrieMap.class' was compiled against an incompatible version of scala.collection.generic.
[scala-js] [error]     def emptyParMap[K, V]: ParMap[K, V] =  ParTrieMap.empty
[scala-js] [error]                                            ^

huh, this looks similar to the twitter-util thing!
fast-string-interpolator
https://scala-ci.typesafe.com/job/scala-2.13.x-integrate-community-build/2641/artifact/logs/fast-string-interpolator-build.log
[fast-string-interpolator:error] <set>:1: error: not found: value bintrayReleaseOnPublish
[fast-string-interpolator:error] bintrayReleaseOnPublish := false

avro4s
https://scala-ci.typesafe.com/job/scala-2.13.x-integrate-community-build/2641/artifact/logs/avro4s-build.log
wtf is this
[avro4s] [error] /home/jenkins/workspace/scala-2.13.x-integrate-community-build/target-0.9.16/project-builds/avro4s-6ffddf2be2c194c8810f3d8e50b63abc26c2ad0b/avro4s-core/src/test/scala/com/sksamuel/avro4s/record/decoder/EnumDecoderTest.scala:17:47: class AvroEnumDefault does not extend scala.annotation.Annotation
[avro4s] [error] case class ScalaAnnotatedSealedTraitEnumList(@AvroEnumDefault(List(CuppersAnnotatedEnum)) cupcat: List[CupcatAnnotatedEnum])
[avro4s] [error]                                               ^

can we reproduce outside dbuild? avro4s-core/test:compile

with 2.13.1: yes, reproduced

upstream PR submitted

https://scala-ci.typesafe.com/job/scala-2.13.x-integrate-community-build/2644/

rebased.
split the scala-parallel-collections trouble out as #981

https://scala-ci.typesafe.com/job/scala-2.13.x-integrate-community-build/2644/artifact/logs/monix-build.log monix needs older cats, I think?
https://scala-ci.typesafe.com/job/scala-2.13.x-integrate-community-build/2644/artifact/logs/akka-http-build.log akka-http test failure, perhaps intermittent? but seen twice so far, maybe it's consistent

queued: https://scala-ci.typesafe.com/job/scala-2.13.x-integrate-community-build/2648/


Avionics power check type error (int to float)

After a refactor inn #9193 a type error was introduced. This was truncating 4.95v down to 4.00v.

Thanks @dakejahl. Why don't we go one step further and use system_power.voltage5V_v directly?

FYI I think your github and local git email address is out of sync.


⚠ Bofrost.fr has degraded performance

In 28f695a, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1045 ms

Resolved: Bofrost.fr performance has improved in cfa9230 after 17 minutes.


Adding AWS S3 Signed URL example

Adding AWS S3 Signed URL example
S3 Signed URLs can be used to allow clients to upload files to AWS S3 without needing or exposing AWS credentials.
More info on using AWS S3 Signed URLs for file uploading at http://docs.aws.amazon.com/AmazonS3/latest/dev/PresignedUrlUploadObject.html
Example GraphQL query request
{
  SignedUrl(filePath: "filePath.jpg") {
    fileName
    signedUrl
    getUrl
  }
}

Example return
{
  "data": {
    "SignedUrl": {
      "fileName": "graphcool_uploads/1504353705891-filePath.jpg",
      "signedUrl": "https://s3.amazonaws.com/_AWS_BUCKET_/graphcool_uploads/1504353705891-filePath.jpg?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=_KEY_ID_%2F20170902%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20170902T120145Z&X-Amz-Expires=300&X-Amz-Signature=d7370f599fe9feaef643baf62f70d867cf9d5b094998a28e201067c5a53fbcac&X-Amz-SignedHeaders=host",
      "getUrl": "https://_AWS_BUCKET_.s3.amazonaws.com/graphcool_uploads/1504353705891-filePath.jpg"
    }
  }
}

ping

🎉 thanks a lot!


Can't add any options to the filter block

I have a brand new install of C5 and the Community Store.
I've added a product list block and a product list filter block to a page.
I can't add an attribute to the filter. When I click on one of the Available Attributes to move it to the Displayed Attributes nothing happens except a hash is added to the page url. Am I missing something or is there a bug here?

I don't know if it's related but I'm getting an error message as I seem to be missing some files ...
`        $select2 =  $al->getAssetGroup('select2');
    if (!$select2) {
        $al->register('css', 'select2', 'vendor/select2/select2/select2.css', ['version' => '3.5.4', 'position' => Asset::ASSET_POSITION_HEADER, 'minify' => false, 'combine' => false], $this);
        $al->register('javascript', 'select2', 'vendor/select2/select2/select2.js', ['version' => '3.5.4', 'position' => Asset::ASSET_POSITION_FOOTER, 'minify' => false, 'combine' => false], $this);

        $al->registerGroup('select2',
            [
                ['javascript', 'select2'],
                ['css', 'select2']
            ]
        );
    }`

Yep that was it.
I downloaded and added the following files and now the filter is working
/packages/community_store/vendor/select2/select2/select2.css
/packages/community_store/vendor/select2/select2/select2.js

I just took a look at the latest release zip, for 2.4.6, and the vendor folder and those files appear to be in it.
So I'm a bit confused why you've had to manually add them. Do you have a vendor folder with select2 in it at all?

No I don't have the vendor folder at all.  I downloaded a new copy of Community Store (today at 15.59) just to check that as well and it did not contain the vendor folder either.

I'm getting another issue with the filter block so don't know if I'm missing more files. If I add the option to filter by price I get the following error...
jquery.js?ccm_nocache=51e590e5cc3b549e8a6133bc65f5b5b61e20630a:2 Uncaught TypeError: $(...).ionRangeSlider is not a function     at HTMLDocument.<anonymous> (<anonymous>:3:55)     at e (jquery.js?ccm_nocache=51e590e5cc3b549e8a6133bc65f5b5b61e20630a:2:30038)     at t (jquery.js?ccm_nocache=51e590e5cc3b549e8a6133bc65f5b5b61e20630a:2:30340)

Are you downloading release zips?
On Sun, 9 Jan 2022, 16:51 rosie607, @.***> wrote:

I'm getting another issue with the filter block so don't know if I'm
missing more files. If I add the option to filter by price I get the
following error...
jquery.js?ccm_nocache=51e590e5cc3b549e8a6133bc65f5b5b61e20630a:2 Uncaught
TypeError: $(...).ionRangeSlider is not a function at
HTMLDocument. (:3:55) at e
(jquery.js?ccm_nocache=51e590e5cc3b549e8a6133bc65f5b5b61e20630a:2:30038) at
t (jquery.js?ccm_nocache=51e590e5cc3b549e8a6133bc65f5b5b61e20630a:2:30340)
—
Reply to this email directly, view it on GitHub
https://github.com/concrete5-community-store/community_store/issues/644#issuecomment-1008334685,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIHSMFJ3IGTBYI6BNH7SPDUVG4HPANCNFSM5LR7M6AA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***
com>

I'm downloading from this link

Use the latest release zip from
https://github.com/concrete5-community-store/community_store/releases
Releases have all files ready to go and are meant for production use,
unlike downloading from the master branch (which has dependency install
steps).
On Sun, 9 Jan 2022, 18:11 rosie607, @.***> wrote:

I'm downloading from this link
[image: Screenshot 2022-01-09 at 18 10 26]
https://user-images.githubusercontent.com/11563269/148694815-7bf790ab-7808-48e4-92c6-cc45d2a86000.png
—
Reply to this email directly, view it on GitHub
https://github.com/concrete5-community-store/community_store/issues/644#issuecomment-1008347613,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIHSMA4PG2XWDM27DLQNODUVHFVXANCNFSM5LR7M6AA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***
com>

Use the latest release zip from https://github.com/concrete5-community-store/community_store/releases Releases have all files ready to go and are meant for production use, unlike downloading from the master branch (which has dependency install steps).
…
On Sun, 9 Jan 2022, 18:11 rosie607, @.> wrote: I'm downloading from this link [image: Screenshot 2022-01-09 at 18 10 26] https://user-images.githubusercontent.com/11563269/148694815-7bf790ab-7808-48e4-92c6-cc45d2a86000.png — Reply to this email directly, view it on GitHub <#644 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIHSMA4PG2XWDM27DLQNODUVHFVXANCNFSM5LR7M6AA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you commented.Message ID: @. com>

Thanks!

I'm getting another issue with the filter block so don't know if I'm missing more files. If I add the option to filter by price I get the following error...
jquery.js?ccm_nocache=51e590e5cc3b549e8a6133bc65f5b5b61e20630a:2 Uncaught TypeError: $(...).ionRangeSlider is not a function at HTMLDocument.<anonymous> (<anonymous>:3:55) at e (jquery.js?ccm_nocache=51e590e5cc3b549e8a6133bc65f5b5b61e20630a:2:30038) at t (jquery.js?ccm_nocache=51e590e5cc3b549e8a6133bc65f5b5b61e20630a:2:30340)

I've installed the release version. Thanks for pointing that out for me!
I'm still getting the above error and the price filter isn't working, is this another 'me' thing?

If this is on V9.0.1, this sounds like you're hitting a core bug where a block doesn't load in css and js files correctly.
You'll need to apply this fix directly (which will be in v9.0.2)
https://github.com/concrete5/concrete5/commit/694c6e4b146f329225cde7a1623c75854855d962

That worked great, thank you!


Add bsg

Add bsg to JetBrains

@Tiremisu First stage of verification passed, official domain is nordangliaeducation.com as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://www.nordangliaeducation.com/bsg-guangzhou/contact-us

@Tiremisu You might miss my previous comment. Please react.

@philipto Here is the info:

the school official website URL -- bsg.org.cn

the school street address, including city and country
The British School of Guangzhou- South Lake Campus
983-3 Tonghe Road
Baiyun District
Guangzhou 510515
China

an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
https://www.nordangliaeducation.com/cn/bsg-guangzhou/academic-excellence/curricula-guide/a-levels

a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@Tiremisu Thank you very much for the clarification and your patience while waiting for a review and processing your request. Pull request merged.


Added scale-up functionality for MySQL and PostgreSQL DBaaS clusters

Adding a new type object to the Managed MySQL Database Update and Managed PostgreSQL Database Update operations.
The new object is unique to these two operations, so I've included it in the root requestBody content for both of these operations. (The template $ref content that all of the other objects link to already contains a type object. Looks like it's used for GET operations.)

Waiting on approval from DBaaS team for the release of this functionality. (Parity concerns: UI issues are preventing release for the API.)


Prevent Ship When Requirements Aren't Passing

Right now when I ship a branch, it will merge regardless of the branch protection.
When branch protection is enabled on GitHub, and my PR doesn't fulfill the requirements of the branch protection (i.e. passing CI), I want a fail message to appear. Protection options should be similar to the options listed below.

Now that we git ship can merge via the github api (released in 4.2.0), the branch protection provided by github should now work with git ship


make an option for dark bkg

some people do not like a white background.
we should give them an option.

related: https://github.com/twosigma/beaker-notebook/issues/2692

extend/use codemirror themes? https://codemirror.net/demo/theme.html#ambiance

please add a persistent preference for codemirror color theme.
colors should apply to plots, tables, menus, dialogs, the whole UI.


[Enhancement][OpenSearch & OpenSearch Dashboards] Request for Helm Repo

Describe the solution you'd like
Can we have the helm repo for OpenSearch projects? So that we can install charts easily by adding the repo.
Something like:
helm repo add opensearch <https://helm repo URL>
helm repo update
helm install opensearch/opensearch-dashboards

We already have one setup. You can refer this

We already have one setup. You can refer this

Oh thank you!! I missed that.


angular-utils-pagination isn't handling data the same as ng-repeat

The following two code snippets should result in the same data, but one is loaded prior to the state change and the other is loaded asynchronously. They're not.  ng-repeat works with both of them, but I switched to angular-utils-pagination, and the latter code works, but the former code does not.
However, I don't see a difference in the arrays that are being generated, and they work with ng-repeat.
I'm going to crosspost this issue to angular-utils-pagination and ui-router as well, as it's not handling the difference properly, as I'm not certain whether uirouter or restangular is in error for giving the wrong information in the first place.
doesn't work with angular-utils-pagination
.state( 'users', {
    url:'/users',
    controller: 'UsersCtrl',
    templateUrl: 'users/Users.tpl.html',
    data:{ pageTitle: 'Users' },
    authenticate: true,
    resolve: {
      users: ['Users', function(Users){
        return Users.getList({embed:'roles,user_certifications'});
      }]
    }
  })
.controller( 'UsersCtrl', function UsersController( $scope, users ) {
  $scope.users = users;
})
.factory('Users', function(Restangular){
  return Restangular.service('users');
})
;

works with angular-utils-pagination
.state( 'users', {
    url:'/users',
    controller: 'UsersCtrl',
    templateUrl: 'users/Users.tpl.html',
    data:{ pageTitle: 'Users' },
    authenticate: true
  })
.controller( 'UsersCtrl', function UsersController( $scope, Users ) {
  Users.getList().then(function(result){
   $scope.users = result;
   });
})
.factory('Users', function(Restangular){
  return Restangular.service('users');
})
;

Here's the other two issues I created for this issue.
https://github.com/angular-ui/ui-router/issues/2308#issuecomment-147878147
https://github.com/mgonto/restangular/issues/1252

Hi there,
Thanks for the code, makes the problem clear. What I am missing is what exactly you mean when you say that the pagination module doesn't work in the first instance. Does it throw an exception? Any output in the console? What do you see that is not what you expect?
Fill me in a bit more - best would be to reproduce the issue in Plunker - and I'll check it out.

It doesn't give an error, but doesn't show the list, despite ng-repeat working in the same instance. I'll write a plunker to reproduce.

Okay. Also I just pushed some changes that may have something to do with your issue today. make sure you test against the latest version of dirPagination.js in the master branch.

It looks like your changes fixed my issue.

I would like to know what the changes were that caused this and why it was fixed, as I have another ticket in restangular because the two patterns return different data.

You can see the diff of the change that may have solved your issue here: https://github.com/michaelbromley/angularUtils/commit/6944504cdc3ab52d89edcef5e955e4a57ca2133d
It has to do with using a pagination-id that resolves to a string containing a non-alphanumeric character such as a period or hyphen. This confuses Angular's $parse service which then means the directive cannot internally figure out the current page, which was causing it to display an empty list.


RuntimeException: Crosswalk's APIs are not ready yet

Fatal Exception: java.lang.RuntimeException: Crosswalk's APIs are not ready yet
       at org.xwalk.core.XWalkView.setZOrderOnTop(XWalkView.java:1569)
       at org.crosswalk.engine.XWalkWebViewEngine.initWebViewSettings(XWalkWebViewEngine.java:200)
       at org.crosswalk.engine.XWalkWebViewEngine.access$000(XWalkWebViewEngine.java:59)
       at org.crosswalk.engine.XWalkWebViewEngine$2.run(XWalkWebViewEngine.java:96)
       at org.xwalk.core.XWalkActivityDelegate.onActivateCompleted(XWalkActivityDelegate.java:172)
       at org.xwalk.core.XWalkLibraryLoader$ActivateTask.onPostExecute(XWalkLibraryLoader.java:349)
       at org.xwalk.core.XWalkLibraryLoader$ActivateTask.onPostExecute(XWalkLibraryLoader.java:317)
       at android.os.AsyncTask.finish(AsyncTask.java:660)
       at android.os.AsyncTask.-wrap1(AsyncTask.java)
       at android.os.AsyncTask$InternalHandler.handleMessage(AsyncTask.java:677)
       at android.os.Handler.dispatchMessage(Handler.java:102)
       at android.os.Looper.loop(Looper.java:154)
       at android.app.ActivityThread.main(ActivityThread.java:6682)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1520)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1410)

This is accounting for 58% of crashes in my Android app according to Crashlytics.
Full crash log: issue_9_crash.log

Same here for 2.3.0. Four crashes in one week.

Hello there, has anyone found a workaround to this issue? Please let me know if you did find a fix to this. Thanks.

Hello there, has anyone found a workaround to this issue? Please let me know if you did find a fix to this. Thanks.

By any chance did you find a fix for this. I am also facing the same thing on multiple devices.

@cosmopolit Have you found any fix for this. Thank you.

Does someone has resolved the issue?

Both this plugin and the Crosswalk project are deprecated and no longer maintained.
Therefore it's highly unlikely this will ever be fixed.

Ok, thanks!


Just warn about multiple packages

There are many projects outside that have some helper binaries with a
"// +build ignore" clause and "main" package in a directory, where the
rest of files have some other package name.
So far I found cznic/ql, coreos/rkt and golang.org/x/net/html/atom (found packages atom (atom.go) and main (gen.go) in /home/kv/projects/coreos/GOPATH/src/golang.org/x/net/html/atom)
This depends on #346, as both modify the code around the same place.

Closing this. It is not enough and I have a problem fixing it. Will file an issue.

That's actually already filed - #345


Wrong file of jstree.js used

Hi,
in the last commit, you changed the used jstree.js file. The one from src is included instead of the one from dist. The plugins aren't working anymore as they are not included in the src/jstree.js file.

Thanks for reporting.
Version 0.1.1 should fix this issue.


Feature/3.2 rest protocol provider

What is the purpose of the change
RestProtocol provider  restHandler & restProtocol
Brief changelog
##Related issues
#10803
@AlbumenJ
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #11640 (f0177c9) into 3.2 (ea7af7a) will decrease coverage by 0.72%.
The diff coverage is n/a.

:exclamation: Current head f0177c9 differs from pull request most recent head 7cd8650. Consider uploading reports for the commit 7cd8650 to get more accurate results

@@             Coverage Diff              @@
##                3.2   #11640      +/-   ##
============================================
- Coverage     69.25%   68.53%   -0.72%
+ Complexity      340        2     -338
============================================
  Files          3256     1558    -1698
  Lines        153181    64480   -88701
  Branches      25560     9404   -16156
============================================
- Hits         106084    44193   -61891
+ Misses        37681    15993   -21688
+ Partials       9416     4294    -5122

see 1807 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Feature Request : Multiple windows when dragging files out of editor

For instance when an editor tab is dragged out of the current vs code editor it will instantiate a new window for that file.
Thanks

This issue has already been reported. Please use the search function next time before reporting an issue and/or this time if you are curious about the progress of the issue.


Backport PR #1369 on branch 2.15.x (Add label checker CI)

Backport PR #1369: Add label checker CI

Note the Label related CI won't appear until this is merged because pull_request_target requires the CI to exist as is in the repo itself to run for security reasons (it gives limited write permissions so it will not be run from forks like this one if it gets changed). We should see it operate on the next backported PR.


No module named dbus.service

On Ubuntu 16.04 Unity, I ran into the above mentioned issue so I thought python-dbus might be added to the package dependency list.
sudo apt-get install python-dbus

@teobaluta thanks for noticing this.


Add ts-reset

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

Describe the solution you'd like to see
I think there should be an d.ts file with ts-reset to make our Typescript so much better
Describe alternate solutions
Do the stuff from ts-reset self
Additional information
No response

I think we had this before. Personally I'm leaning no - it makes TS behave in non default ways which most people will not expect, and adding it yourself takes a few seconds.
Would also suggest watching this https://www.youtube.com/watch?v=_-RyV65J4kE

we decided a while back that this is something we don't want to prescribe since you need to be aware of the changes it does. we use it on this monorepo, and i use it on some other projects but it's not something we want to include in every app by default


Move fav/touch icons to CDN

http://www.webpagetest.org/result/170123_QP_6AN/1/performance_optimization/#compress_text

Nevermind.


Using refPackSet causes trouble when resolving packDir if refFile is absolute

Using refPackSet causes trouble when resolving packDir if refFile is absolute. In this case two absolute paths are combined, leading to "Invalid file" exception.
With this pull request absolute and relative paths are handled.
Testcase and test installer files prove the change. Running the testcase on current master will lead to:
com.izforge.izpack.api.exception.CompilerException: Invalid file: D:\projects\izpack\izpack-compiler\target\test-classes\samples\refpackset\D:\projects\izpack\izpack-compiler\target\test-classes\samples\refpackset\..\refpackincludes\izpack-module.xml while processing refpackset refpackset         at com.izforge.izpack.compiler.CompilerConfig.addPacksSingle(CompilerConfig.java:1018)         at com.izforge.izpack.compiler.CompilerConfig.addPacks(CompilerConfig.java:771)         at com.izforge.izpack.compiler.CompilerConfig.executeCompiler(CompilerConfig.java:360)         at com.izforge.izpack.compiler.CompilerConfigSamplesTest.installerShouldResolveRefPackSetCorrectly(CompilerConfigSamplesTest.java:83)
For sure, path on your machine will be differnt.

@gerhardhiller could you also open an according JIRA issue?

Jira-Issue: https://izpack.atlassian.net/browse/IZPACK-1747


conditionize initing features and components.

Resolve: https://github.com/istio/istio/issues/16045

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

Test name
Commit
Details
Rerun command

operator-lint-dind
d9bb6fa2a41338f91dd9f75b2fec662a51fff42e
link
/test operator-lint-dind

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

This will also fix https://github.com/istio/istio/issues/16041

/retest

@morvencao: PR needs rebase.

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

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

Test name
Commit
Details
Rerun command

operator-lint-dind
d9bb6fa2a41338f91dd9f75b2fec662a51fff42e
link
/test operator-lint-dind

operator-lint
9241e83820a0a69a59717a1ea4a3fdec33c3ea7c
link
/test operator-lint

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

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

Test name
Commit
Details
Rerun command

operator-lint-dind
d9bb6fa2a41338f91dd9f75b2fec662a51fff42e
link
/test operator-lint-dind

operator-lint
82f241a4cc30f186643b1873da3e8f3834da3e22
link
/test operator-lint

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

/retest

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

Test name
Commit
Details
Rerun command

operator-lint-dind
d9bb6fa2a41338f91dd9f75b2fec662a51fff42e
link
/test operator-lint-dind

operator-test
6315ec18d293fcfc0447d8236331b1dfa4d73f8a
link
/test operator-test

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


PDF reflow (not an issue)

This is not an issue, but a notification of a new feature.
I used PDFJS to reflow PDFs in the browser.
You can search for PDFs from https://docushow.com and see an example document here:
https://docushow.com/viewdoc?url=https%3A%2F%2Fwww.microsoft.com%2Fen-us%2Fresearch%2Fwp-content%2Fuploads%2F2016%2F02%2FMobileHCI2011-Panjwani-reflow.pdf&title=Script-Agnostic Reflow of Text in Document Images - Microsoft#0
Would it be something useful for PDFJS?
Laurent

I don't think this should be included in PDF.js by default, but please feel free to add your project to https://github.com/mozilla/pdf.js/wiki/Gallery-of-user-projects-and-modifications. It may be useful for other people that want to work on a mobile/responsive viewer. Thank you!

@ldenoue is there a PR with your changes anywhere? I am interested in this.

@rmhrisk No, I decided to continue work on this on my own at the moment.

This would be an amazing feature!
It's incredibly rare to find PDF viewers which do this (I only know of one, and it's was discontinued 5 years ago), but it makes reading documents far more enjoyable. I hate having to continually reposition the page while reading, and many PDFs are unusable on certain devices (mobile, ebook) without this feature.

What was that app that was discontinued 5 years ago?
For now you can use DocuShow.com to view your PDFs on mobile.
But I'm also working on a native iOS app. Would Android be useful?

The app is called QuickOffice: https://play.google.com/store/apps/details?id=com.quickoffice.android
You activate reflow mode by going to "Reading View" in the hamburger menu. Google bought up the company that made it, then ditched it a few years later.
DocuShow is very nice! It works well on the desktop, but I was having trouble running it in Brave on Android. It was coming up blank.
One immediate feature I'd like is the ability to upload a PDF without supplying the URL.
(Maybe we should take this conversation to the DocuShow issue tracker... :)

Has the situation changed at all on this feature? @ldenoue , was the stuff you were working on open source at all? I am interested in something basic that works for relatively simple cases. Basically if it can keep paragraphs separate then it would be reasonably fit-for-purpose. I also came across https://github.com/JSTOR-Labs/reflowit but that doesn't really do what I need (I need HTML like PDF.js produces, not another PDF or an image), and also hasn't seen any activity in half a decade...

@AntonOfTheWoods not open sourced and my tech cuts out page images into small word images to perform reflow, adding back a text inside each.
I since moved the web demo and released an iOS app called PDF Reflow here https://www.appblit.com/pdfreflow


Add separate targetTemperature sensor on washing machines and washer-dryers.

The targetTemperature (0) sensor was used for washing machines in the same way as for other appliances (fridges, ovens, etc.).
Create a separate sensor used only by washing machines and washer-dryers in order to display target temperature as an integer. Sensor key is still "targetTemperature" so it's not a breaking change.
Target temperature 1 and 2 are not in use by any model of these classes.

If I understand you correctly the intention with this PR is to display target temperature without decimals and to remove two hidden target temperatures.

Yeah, you understood correctly.

I think the target without decimals is a valid change for all devices with target temperature.

Is this true also for fridges / wine cabinets? I don't have these device classes, so I can't verify this. In that case, this change could be applied also to zones 2 and 3 for all device classes.

It was a practical solution to create deactivated sensors for appliance types with different number of zones. There is no simple way do deduct from the API how many zones there are for a specific device, except by creating and maintaining specific capabilities tables for each device model. If you are sure that there is no and never will be any multizone devices of type washer or dryer, well then it is OK to remove zone 2 and zone3

Of course it's an assumption that zones 2 and 3 will never be used by washers / washer-dryers, but it seems reasonable. It's not a problem to revert this and leave deactivated sensors, it's more generic and practical.

I think the target without decimals is a valid change for all devices with target temperature.

Is this true also for fridges / wine cabinets? I don't have these device classes, so I can't verify this. In that case, this change could be applied also to zones 2 and 3 for all device classes.

The API data model specifies targetTemperature as an integer when setting the target, so I think it is reasonable do display the returned value without decimals. I can confirm that fridge and freezer only can be set on integer temparatures on the panel.
I suggest that the received targetTemperature is converted to an int for all instances.
If we should separate washers from the other devices with targetTemperatures when there are no material differences is a matter of opinion. You can decide!
And when you are in the sensor.py file. Please remove state_class=SensorStateClass.MEASUREMENT from all targetTemperature sensors.

Very well, so I'll update this also for other target temperature sensors.

If we should separate washers from the other devices with targetTemperatures when there are no material differences is a matter of opinion. You can decide!

So, probably it's better previous solution with deactivated sensors... so if something changes on Miele side, it's easier to maintain because other 2 sensors are already available.

Thank you!


python311Packages.apprise: fix build when cross compiling

Description of changes
setuptools was trying to install babel as it is a setup requirement (listed in setup_requires in setup.py) but failed as pip is not available in the build environment.
I'm not 100% sure why this builds fine when not cross compiling, but this change fixes it.
Fixes #304841.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Looks like this was fixed by #309680 (specifically in 75f054f52c95d167e567300aaa6d2e9dac00a275).


Building ruby artifacts is broken on Linux

https://grpc-testing.appspot.com/view/Artifacts/job/gRPC_build_artifacts/361/architecture=x64,language=ruby,platform=linux/console
src/core/iomgr/pollset_set_windows.c: In function 'grpc_pollset_set_create':
src/core/iomgr/pollset_set_windows.c:40:19: error: type of 'pollset_set' defaults to 'int' [-Werror]
 grpc_pollset_set* grpc_pollset_set_create(pollset_set) { return NULL; }
                   ^
src/core/iomgr/pollset_set_windows.c:40:1: error: number of arguments doesn't match prototype
 grpc_pollset_set* grpc_pollset_set_create(pollset_set) { return NULL; }
 ^
In file included from ./src/core/iomgr/pollset_set_windows.h:37:0,
                 from src/core/iomgr/pollset_set_windows.c:38:
./src/core/iomgr/pollset_set.h:46:19: error: prototype declaration
 grpc_pollset_set *grpc_pollset_set_create(void);
                   ^
cc1: all warnings being treated as errors
make: *** [/tmp/objs/opt/src/core/iomgr/pollset_set_windows.o] Error 1

cc @ctiller
That looks like a core bug to me.

Still a problem on the release branch:
https://grpc-testing.appspot.com/view/Artifacts/job/gRPC_build_artifacts/450/architecture=x64,language=ruby,platform=linux/console

Fix looks trivial, I can send a PR.

This is now fixed.


Merged changes from master into develop branch

I'll be looking to make changes on the code, but will be working on develop branch. I suggest to make this branch the primary for development and then promote to master when stable.
Regards,

Sounds great, thanks for your help! 👏