Migration Issues

Description
We were having trouble updating our CSS in our spectrum implementation. We last updated at about the beginning of the year. There were LOTS! of changes since then. We ended up going through all the components and compiling a list of issues we found.
Additional context
A CHANGELOG file which is updated with changes (i.e. additions, subtractions and changes) would be very much appreciated!
Here is our shorthand list

Action Button: Emphasized, click doesn't work.
Action Group: Removed.
AssetList: Missing > check why
Breadcrumbs: nested (root visible) missing nav root. Same for multiline nested, compact nested, compact nested root visible, title disabled
ButtonGroup: needs changes
Calendar: focused
Color Area: Markup, missing things...
Color Handle: new
Color Loupe: new
Color Slider: new
Color Wheel: new
Combobox: text field changes
Date Picker: structure changed
Dialog: is a mess
Dropdown: renamed to Picker
Link: removed subtle, added secondary
Pagination - explicit: use new textfield
Pagination - page: needs work
Popover: dialog style...
Radio: 2 long radios connected
Rule: renamed to Divider
Search: needs new text field
SearchWithin: needs new text field and dropdown
Side Nav: icon is not in selectionbox
Stepper: needs change and new textfield
Tabs: missing a lot
Tags: messed up
Thumbnail: new
Toast: some are deprecated
Tool: removed
Tooltip: some are deprecated
Treeview: changes
Typography: changes
RTLlayout: new dir='ltr' is now required somewhere up the HTML tree for normal LTR layout

I hope this is helpful to someone...

We've got a changelog for every component (here's Button), but we don't have a changelog with everything in one place. Hope that helps!

@Harbs there are also migration guides in the metadata for some components.

Like @GarthDB, we have some migration guides here: https://opensource.adobe.com/spectrum-css/textfield.html#migrationguide
Please do file separate issues for anything that's messed up!


Support for additional containers

Hi! First off, great work on the Helm chart, I've used it and it works really well!
It would be great to allow to add additional containers to the Traefik deployments. For my use-case, I want to add a side-car that collects the Prometheus metrics from Traefik and sends them somewhere else.
To do this, I think a simple solution is to add an additionalContainers key to the values.yaml file, and just append its content to the containers of the pods spec in the Deployment.

Thanks a lot for your quick action @notsureifkevin


[feat-request] dynamic gateway discovery

Current gateway sources:
https://github.com/labscommunity/arweavekit/blob/11ec2269c936204132a7c8709fc09a469d6f28b8/src/utils.ts#L1-L6
Suggestion

Add ability to detect ARio GWs from the new registry.
Ref: https://github.com/ar-io/wayfinder

Yeah, I saw that. You wanna close this issue?

Hey @rosmcmahon thanks! Closing this issue


SW command does not work

I tried using the new SW V1.4.0, and the switch command does not seem to work even tested with the flag set to true.
Might be some issue with the command not being registered with AddCommand and having a dynamic argument for player.

It will say the command doesn't work, this is also in the known bugs area in the readme, which is incorrect. It does work, it should switch to the next person, or mentioned person. I think its a bug that the first time using it you have to run it twice, but unsure about that.
And yes you are right about the AddCommand not being a thing here, I had to make it a possibility to input usernames, which I believe you can't with the AddCommand. I'm currently just reading console input which might be why it doesn't recognize it, I will try and fix this bug when I have time however

I got 50% fixed, it doesn't spam say the error anymore whenever you're writing "sw" or "s", only does it whenever you write "sw [user]" at the moment. I'll see if I can do anything about it

I will have to check out the thing with usernames later this weekend sadly, I don't have that much time for it right now

This should be fixed in v1.4.1 which will be released sometime soon, if a bug occurs with this, please let me know and I will take a look at it


didChangePlatformBrightness called multiple times flutter

I am using didChangePlatformBrightness to get brightness light/dark. It works fine and I am able to update my theme every time brightness is updated. But this function is Called even when app goes in background and comes back again in foreground. I do store the old brightness and compare it with new one and if it is not same then only update my theme. But this function is being called multiple times and doesn't give correct brightness at once. Mostly its being called two times and first time it gives wrong brightness(this behaviour is random) and in last gives correct brightness.
@override
  void didChangePlatformBrightness() {
    var theme;
    setState(() {
      var brightness = WidgetsBinding.instance.window.platformBrightness;
      if (brightness == Brightness.dark) {
        theme = Brightness.dark.toString();
      } else {
        theme = Brightness.light.toString();
      }
    });
    this.updateAppTheme(theme); // Called multiple times
    super.didChangePlatformBrightness();
  }

In the end I do get correct brightness but since function is called multiple times my theme gets updated multiple times. Basically for every time my app goes in background and comes back in foreground my theme get updated. Which I don't want. Please help!

Hi @Richa0305
can you please provide your flutter doctor -v ,
your flutter run --verbose
and a more complete reproducible minimal code sample.
Thank you

flutter doctor -v
[✓] Flutter (Channel stable, v1.12.13+hotfix.8, on Mac OS X 10.14.5 18F203, locale en-IN)
• Flutter version 1.12.13+hotfix.8 at /Users/Richa/Desktop/Dev/flutter
• Framework revision 0b8abb4724 (3 months ago), 2020-02-11 11:44:36 -0800
• Engine revision e1e6ced81d
• Dart version 2.7.0
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at /Users/Richa/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 29.0.2
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses
[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.3.1, Build version 11C504
• CocoaPods version 1.8.4
[!] Android Studio (version 3.5)
• Android Studio at /Applications/Android Studio.app/Contents
✗ Flutter plugin not installed; this adds Flutter specific functionality.
✗ Dart plugin not installed; this adds Dart specific functionality.
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
[✓] Connected device (1 available)
• iPhone 11 Pro Max • 6C1D79FE-B68F-4F3C-97E6-15AE0AF2D420 • ios • com.apple.CoreSimulator.SimRuntime.iOS-13-3 (simulator)

flutter run --verbose is very huge. I am able to get required result by providing a delay -
@override
void didChangePlatformBrightness() {
setState(() {
brightness = WidgetsBinding.instance.window.platformBrightness;
Timer(Duration(milliseconds: 500), () {
var theme;
if (brightness == Brightness.dark) {
theme = Brightness.dark.toString();
} else {
theme = Brightness.light.toString();
}
store.updateAppTheme(theme);
});
});
}

Hi @Richa0305
I'm glad that you found a work around,
nevertheless your code sample is not reproducible
as isn't representative of your implementation;
also I'd recommend to run flutter upgrade --force
making sure that you are not experiencing bugs already fixed.
Thank you

Without additional information we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


feat: Refactored modal component

Description
Added refactored Modal component.
Fixes #19324
Depends on #334
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Manual on storybook
Manual on main repo
Jest
Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing
https://github.com/appsmithorg/design-system/pull/355#issuecomment-1431034326
Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

Bugs in this PR: (These are in both Modal with footer/no footer)
Functional:

The Modal shifts after appearing on screen.
https://www.loom.com/share/4a77148a6eee4683a5e209f7d4b72138

Clicking outside the Modal does not close it.

No hover state for x icon.

Keyboard focus does not move to the x icon. And since Cancel & Save buttons are not functional, we are not able to test keyboard actions inside the Modal.

When we switch to Docs, the page is not displayed correctly.

Design:

Wrong colour for x icon.
Wrong Padding for the component as a whole. (top/bottom is swapped with right/left).
Wrong padding between the title & body.
Wrong padding between icon & body.

Functional:

The Modal shifts after appearing on screen. -> FIXED
Clicking outside the Modal does not close it. -> FIXED
No hover state for x icon. -> NOT IN DESIGN
Keyboard focus does not move to the x icon. And since Cancel & Save buttons are not functional, we are not able to test keyboard actions inside the Modal. -> We can ignore this for now, since esc will colse modal.
When we switch to Docs, the page is not displayed correctly. -> FIXED
Wrong colour for x icon. -> Made it as per design
Wrong Padding for the component as a whole. (top/bottom is swapped with right/left). -> FIXED
Wrong padding between the title & body. -> Fixed

@shadabbuchh can you re-verify this?

All issues are fixed now. This is good to go.


nixos/matrix-synapse: fix type of settings.url_preview_url_blacklist

Description of changes
Thank you @dasJ for pointing that out :)
Note: I'm considering to backport this. Yes, this is a breaking change and every configuration which uses the old type will get an error, however I explicitly wrote code to provide an error that's actually useful. Also, if you have the option declared right now, every single URL preview will fail without a noticeable error in at least Element (you'd have to check the server logs for that explicitly). cc @NixOS/nixos-release-managers

Actually, it's supposed to be listOf (attrsOf str) because each list-item can match against multiple properties from urlsplit[1]. In fact, listOf str breaks URL previews at runtime:
Sep 14 15:03:47 soost synapse[1100355]: synapse.http.server: [GET-116] Failed handle request via 'PreviewUrlResource': <XForwardedForRequest at 0x7f691bd5f730 method='GET' uri='/_matrix/media/r0/preview_url?url=<redacted>' clientproto='HTTP/1.1' site='8448'>
                                        Traceback (most recent call last):
                                          [...]
                                          File "/nix/store/xk5yksbw09p6qwk0maq2cb2in3z6f4gn-matrix-synapse-1.91.2/lib/python3.10/site-packages/synapse/media/url_previewer.py", line 398, in _is_url_blocked
                                            for attrib, pattern in entry.items():
                                        AttributeError: 'str' object has no attribute 'items'

To make sure that people aren't confused when upgrading their configs, I decided to work with types.coercedTo to "pretend" accepting the old type signature, but then throwing an error explaining what to do (and rejecting the broken configuration).
[1] https://docs.python.org/3/library/urllib.parse.html#urllib.parse.urlsplit

Things done

Built on platform(s)

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

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

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

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

[x] Fits CONTRIBUTING.md.

If it never worked and people expected it to work, then yes, backporting it is the correct thing to do.
If it would regress on a working configuration I'd be opposed.

If it would regress on a working configuration I'd be opposed.

@mweinelt The question here is how do you define "regress"?
If you mean that evaluations (i.e. the ones that declared url_preview_url_blacklist) that worked before will fail now then yes, it's a regression.
If you mean that the configured feature didn't work at all before (and not only that, but url previews in general) and it works as soon as you fix your config, then no, it's not a regression.

I think I'm fine with nudging our users (through eval errors), that how we handled their intent was wrong and that they have to get up and fix things, so their intent gets properly realized.

I think I'm fine with nudging our users (through eval errors), that how we handled their intent was wrong and that they have to get up and fix things, so their intent gets properly realized.

Fully agree with that! But wanted to have a second opinion though considering that this patch will break evaluations on purpose.

a blacklist (sic!).

Is this a reference to https://github.com/matrix-org/synapse/blob/329597022ee02516e5cbee11fcd566e05609b724/synapse/config/repository.py#L240
;-)

A reference to me not accepting that kind of wording anymore.

A reference to me not accepting that kind of wording anymore.

Agreed!
The line I linked was something I found while investigating how synapse behaves in that regard. I'm somewhat surprised that they didn't create an alias in their configuration though (both because of the problematic naming and because of the extreme similarity). I looked at how this was implemented to understand if the listOf str is valid nowadays and stumbled upon blocklist vs blacklist twice at least.


Delete iris interface (and move it to geoviews)

In https://github.com/ioam/holoviews/issues/1892 we decided to move the iris to GeoViews. This can be merged once I've opened a corresponding PR on geoviews.

[ ] Add iris interface to geoviews

I would be happy to see this PR merged for 1.10.1 if it can be quickly fixed...

I'd prefer for this to wait on https://github.com/ioam/holoviews/issues/2580 so I don't have to copy the entire test suite to GeoViews.

Closing, and will open a new PR.


add possibility to use → in route pattern, in addition to ->

XXX: due to infix operation precedence, → has a lower priority than /. So you have to use parentheses in pattern matching when using this operator.
So this works
  case GET → (Root / "test.json") =>

And, this won't
  case GET → Root / "test.json" =>

It gives you something like this
constructor cannot be instantiated to expected type;
 found   : org.http4s.dsl.impl./
 required: org.http4s.Request[cats.effect.IO]
        case GET → Root / "test.json" => true

Do you still like the feature, given the limitation?

Yes, cause on another I use a rule to rewrite -> into →. And, in a sense, parentheses help to have a better delimitation between method and path 😊

scaladoc added.
And I'm currently using the 0.19 series :)


Exceeds block gas limit error to call view functions

@xxorg3 commented on Wed Jul 25 2018
With solidity view function, you can specific a large number of gas to use. Since it is not a state changing transaction, all the gas cost is not charged for, and the client should be able to allow the call function to run with a gas limit larger than the block limit.

"Needs validation" in that we need someone from our team to verify that geth/parity also support view calls in excess of the block gas limit.
Further, we probably don't want to allow infinite gas on view calls, or if we do we likely want to implement some timeout functionality as otherwise a view which never terminates will take out the whole node.

I can add to this issue that I just checked and at least Parity has no problems using more gas for calls than block gasLimit

I can confirm it's the issue with ganache
Parity and geth clients have no problem with calling function with large gas limit even though the real gas limit is 8M. I can easily call geth with 200M gas if it's eth_call method used

FYI, geth has the following option to limit gas usage for calling functions:

--rpc.gascap value                  Sets a cap on gas that can be used in eth_call/estimateGas (default: 0)

(checked with geth 1.9.3-stable)

Closing as this is now fixed in https://github.com/trufflesuite/ganache-core/pull/449.
@TripleSpeeder, The new option, callGasLimit, wasn't added to the readme, i'll do that now.


Added getCommandMode() method to CommandInterface

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

69 of 118 (58.47%)  changed or added relevant lines in 117 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 81.083%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Command/Redis/BloomFilter/BFEXISTS.php
0
1
0.0%

src/Command/Redis/BloomFilter/BFINFO.php
0
1
0.0%

src/Command/Redis/BloomFilter/BFMEXISTS.php
0
1
0.0%

src/Command/Redis/BloomFilter/BFSCANDUMP.php
0
1
0.0%

src/Command/Redis/CountMinSketch/CMSINFO.php
0
1
0.0%

src/Command/Redis/CountMinSketch/CMSQUERY.php
0
1
0.0%

src/Command/Redis/CuckooFilter/CFEXISTS.php
0
1
0.0%

src/Command/Redis/CuckooFilter/CFINFO.php
0
1
0.0%

src/Command/Redis/CuckooFilter/CFMEXISTS.php
0
1
0.0%

src/Command/Redis/CuckooFilter/CFSCANDUMP.php
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
4852

Relevant Lines:
5984

💛  - Coveralls


[sw] Temporarily clear SRAM in bootrom

SRAM must be initialized in order to have valid parity bit.

This reuses the temporary code from sw/device/mask_rom/mask_rom_start.S.

Thanks @alistair23. Yes I think this is not the long term solution, but those should go into mask ROM anyway.
From my point of view, boot ROM should enable other software to run without too much trouble, at least for now.

i'm a tad worried this is going to drastically increase the boot time for DV.  Let me take a quick look.
If that is the case, is there any simple way to bypass it for now by having a slightly different start for DV vs verilator / fpga?

We could also remove the following additional clearing of the stack. If I remember correctly the stack is already quite big, so if we just do the RAM clearing once, the difference should not be huge.

We could also remove the following additional clearing of the stack.

Ditto for the .bss section.
If we do care about performance here in the short term then we could clear all of RAM except the .data section to minimise the number of stores we are doing:
la   a0, TOP_EARLGREY_RAM_MAIN_BASE_ADDR
la   a1, _data_start
call crt_section_clear

la   a0, _data_start
la   a1, _data_end
la   a2, _data_init_start
call crt_section_copy

la   a1, _data_end
la   a1, (TOP_EARLGREY_RAM_MAIN_BASE_ADDR + TOP_EARLGREY_RAM_MAIN_SIZE_BYTES)
call crt_section_clear

There is also potential to optimize crt_section_clear a bit by unrolling the loop it does.

Any follow up?

we are implementing memory clear in #5572 , hopefully we can merge it some time this week.


Pull Request Page UI #1

CLOSED | Error on page's component

lol I forgot there is an error in the [f933504] commit


Can not auth user nor create new user

Description
I can not authenticate nor create new user. In console, it always show this for signin or create new user:
TypeError: this.auth is undefined
	return this._handleSignIn(this.auth.signInWithEmailAndPassword(email, password))

Expected outcome
I can auth user or create new user
Actual outcome

Code
This is my template:
    <firebase-app
      name="xxx"
      api-key="xxx"
      auth-domain="xxx"
      database-url="xxx"
      storage-bucket="xxx"
      messaging-sender-id="xxx">
    </firebase-app>
    <firebase-auth user="{{user}}" on-error="showError"></firebase-auth>

      <form is="iron-form" id="form" method="post" action="/form/handler">
        <paper-card heading="Sign In">

            <paper-input name="email" label="Email" auto-validate required pattern="[a-z0-9._%+-]+@[a-z0-9.-]+\.[a-z]{2,4}$" error-message="Email is invalid" value={{email}}></paper-input>
            <paper-input name="password" label="Password" type="password" auto-validate required value={{password}}></paper-input>

            <paper-button raised on-click="submitForm">Sign In</paper-button>

        </paper-card>
      </form>

My "submitForm" script:
submitForm: function (e){
  if (e) { e.preventDefault(); }
  if (document.getElementById('form').validate()) {
    email = document.getElementById('form').serialize()["email"];
    password = document.getElementById('form').serialize()["password"];
    this.$.auth.createUserWithEmailAndPassword(email, password).then(function(response){}).catch(function(error){});
  }
},

Browsers Affected

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

Remove the"name" from your firebase-app or add it as app-name to your
firebase-auth
On Tue, Nov 15, 2016, 7:01 AM Christoforus Surjoputro <
<EMAIL_ADDRESS>wrote:

Description
I can not authenticate nor create new user. In console, it always show
this for signin or create new user:
TypeError: this.auth is undefined
return this._handleSignIn(this.auth.signInWithEmailAndPassword(email, password))
Expected outcome
I can auth user or create new user
Actual outcome
[image: screenshot from 2016-11-15 21-55-17]
https://cloud.githubusercontent.com/assets/3140815/20310393/6476def2-ab7e-11e6-9d1f-93174c34cad8.png
Code
This is my template:
<firebase-app
  name="xxx"
  api-key="xxx"
  auth-domain="xxx"
  database-url="xxx"
  storage-bucket="xxx"
  messaging-sender-id="xxx">
</firebase-app>
<firebase-auth user="{{user}}" on-error="showError"></firebase-auth>

  <form is="iron-form" id="form" method="post" action="/form/handler">
    <paper-card heading="Sign In">

        <paper-input name="email" label="Email" auto-validate required pattern="[a-z0-9._%+-]+@[a-z0-9.-]+\.[a-z]{2,4}$" error-message="Email is invalid" value={{email}}></paper-input>
        <paper-input name="password" label="Password" type="password" auto-validate required value={{password}}></paper-input>

        <paper-button raised on-click="submitForm">Sign In</paper-button>

    </paper-card>
  </form>

My "submitForm" script:
submitForm: function (e){
if (e) { e.preventDefault(); }
if (document.getElementById('form').validate()) {
email = document.getElementById('form').serialize()["email"];
password = document.getElementById('form').serialize()["password"];
this.$.auth.createUserWithEmailAndPassword(email, password).then(function(response){}).catch(function(error){});
}
},
Browsers Affected

Chrome
Firefox
Safari 9
Safari 8
Safari 7
Edge
IE 11
IE 10

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/firebase/polymerfire/issues/149, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAD_loOFWVlIiP8Zm-YGEFTJgmAzmATks5q-clJgaJpZM4Kyn_D
.

Hi @mbleigh. You're right, but the other problem was firebase-auth need id="auth" properties and I didn't write it. Thank you.


Determine file size in advance

I searched the youtube api and found how to get the size of the video "getVideoBytesTotal ()", but need to run the video to get the size.
My goal is to make an ajax progressbar via php where to display the download progress. I noticed that the youtube-dl does not bring this information.
Is there a way where I can get the size of the video prior to downloading? ssh or via php?

did any one find any solution for this..?


Boxes need margin on mobile view.

The red boxes collapse into one giant box on mobile. They might need a margin, unless this was intentional. In which case, please close this issue :)

Thanks for this. It's unclear if we're going to keep that feature in our final homepage design. But if we do we'll need to sort it out.

I think the switch to red on white we've made in the homepage branch settles this issue.


Some formatting fixes

r? @topecongiro
The second commit is a bit big, sorry. The basic outline of the refactoring is to layout attributes iteratively rather than recursively, then get derive and non-derive code to share as much code as possible (unfortunately that is a lot less than I hoped).
The results are still not ideal - when we do full vertical formatting for a derive it is kind of silly looking (see one of the test fixups), but you need a lot of derives until you hit that. I'm not sure what would be better either - it is pretty consistent with functions, at least. It would be nice to combine attributes like we do for function attributes and to handle attributes with formatting strings like println!.
Long term, attributes will be allowed to be just tokens, so we'll have the same problems as macros.

LGTM, thank you!

It would be nice to combine attributes like we do for function attributes and to handle attributes with formatting strings like println!.

I think we could try to use overflow::rewrite_with_parens, but that could be a separate PR.


GPG error "gpg: error retrieving<EMAIL_ADDRESS>via WKD: No data"

I'm facing and error that for some reason I can't solve, I've looked at all the issues that look similar and all the solutions in those issues do not work at all.
currently I'm running on a Mac:
System Version: macOS 13.4.1 (c) (22F770820d)
Model Name: MacBook Pro
Model Identifier: MacBookPro15,1
Processor Name: 6-Core Intel Core i7
I'm able to decrypt all files of my repo but I'm not able to encrypt and get this error:
WARNING: This will overwrite any unencrypted files laying about.
Press CTRL-C now to stop. ENTER to continue:
========== RE-ENCRYPTING FILES:
========== PROCESSING "terraform/1-base/roles.tf"
========== Encrypting: terraform/1-base/roles.tf
gpg: error retrieving<EMAIL_ADDRESS>via WKD: No data
gpg<EMAIL_ADDRESS>skipped: No data
gpg: terraform/1-base/roles.tf: encryption failed: No data
I'm added as an admin and I can see my self there:
$ blackbox_list_admins | grep dfranco
<EMAIL_ADDRESS>and my gpg key is also there:
$ gpg --homedir=.blackbox --list-keys | grep -C3 dfranco
gpg: WARNING: unsafe permissions on homedir '/Users/daniel/github/secureauth/willow/.blackbox'
pub   rsa4096 2023-07-19 [SC]
D4F87815ommittedE91F15DF
uid           [ unknown] Daniel Franco (Secureauth GPGKey<EMAIL_ADDRESS>sub   rsa4096 2023-07-19 [E]
and I have that key on my local keyring:
$ gpg --list-keys
[keyboxd]
pub   rsa4096 2023-07-19 [SC]
D4F87815ommittedE91F15DF
uid           [ultimate] Daniel Franco (Secureauth GPGKey<EMAIL_ADDRESS>sub   rsa4096 2023-07-19 [E]
My gpg version is:
$ gpg --version
gpg (GnuPG) 2.4.3
libgcrypt 1.10.2
Copyright (C) 2023 g10 Code GmbH
License GNU GPL-3.0-or-later https://gnu.org/licenses/gpl.html
So from what I understand is the blackbox should grab the gpg keys on .blackbox/pubring.kbx and do all decrypting and encrypting from that information, but it looks like is grabbing the local keyboxd information. I was able to confirm this because I was having that same error:
gpg: error retrieving<EMAIL_ADDRESS>via WKD: No data
with another user and as soon as I imported the private key for that user to my local gpg keyring I stopped getting that error for that user and started getting it for the other one.
The next step was to try to use the /.blackbox as the home for gnupg with:
export GNUPGHOME=~/github/secureauth/willow/.blackbox
This did not work and gave me some other error
I set the enviromental value to the default one and went back to the same error..
Then I tried to export the blackbox pubring.kbx into my local with:
gpg --import .blackbox/pubring.gpg
gpg --no-default-keyring --keyring .blackbox/pubring.gpg  --export -a | gpg --import
this did not work because there is no such file, so I changed it to:
gpg --import .blackbox/pubring.kbx
gpg --no-default-keyring --keyring .blackbox/pubring.kbx  --export -a | gpg --import
this did not work either. So what am I missing here?... I still keep getting the same error, so the fix is to get all the private keys for all the admins, but I don't think that is doable. So if someone has a solution for this I would appreciate if I could be guided in the correct direction.

So I was having just this error as well and it turns out that my local private keys where corrupted / not working properly and the blackbox keys had expired so I had to remove all admins with expired keys and add them again.

So I was having just this error as well and it turns out that my local private keys where corrupted / not working properly and the blackbox keys had expired so I had to remove all admins with expired keys and add them again.

So far I deleted all my keys and I was taken out from the admis and then I was put in again to the admins with a whole new different key, I can decrypt just fine, but encrypting is the thing I can't do

Yeah, based on the error you have it looks like the problem is not your key but the public key of<EMAIL_ADDRESS>is somehow incorrect/corrupt

Same issue here. I tried the below without success
gpg --no-default-keyring --keyring .blackbox/pubring.kbx \
  --with-subkey-fingerprint --quick-set-expire <fingerprint> 6m

any idea how to fix this?

We ran into a similar issue, and we were able to resolve by removing the use-keyboxd option that fresh gpg installs now seem to add by default in ~/.gnupg/common.conf.
Release notes for this change: https://dev.gnupg.org/source/gnupg/browse/master/NEWS$82

Noteworthy changes in version 2.4.1 (2023-04-28)

If the ~/.gnupg directory does not exist, the keyboxd is now automagically enabled.  [rGd9e7488b17]

And docs from gpg indicating behavior of this flag, which seems to cause gpg to totally ignore arguments specifying which keyring should be used:

Note that if the option use-keyboxd is enabled in common.conf, no keyrings are used at all and keys are all maintained by the keyboxd process in its own database.

No idea if that's the "best" solution - or of any potential downsides to keeping keyboxd disabled - but it fixed our blackbox issue..


tuple[][] types

Working on #588
With a regex change, codegen succeeds for  tuple[][] types. but I cant say anything for the actual code it generated. Naively deploying it I get
Subgraph instance failed to run: Failed to process trigger: Unknown function "Contract::bulkGetPlanetArrivals" with signature `bulkGetPlanetArrivals(uint256,uint256):(tuple[][])` called from WASM runtime wasm backtrace: 0: 0x11ef - <unknown>!<wasm function 43> 1: 0x1b2f - <unknown>!<wasm function 73> 2: 0x1b79 - <unknown>!<wasm function 75> , code: SubgraphSyncingFailure, id: QmTcm4ZAB3U7g7oXgAxUEz6yjwqx4DUScbno5xNPg5b8h1

Generated code is
export class Contract__bulkGetPlanetArrivalsResultValue0Struct extends ethereum.Tuple {
  get id(): BigInt {
    return this[0].toBigInt();
  }

  get player(): Address {
    return this[1].toAddress();
  }

  get fromPlanet(): BigInt {
    return this[2].toBigInt();
  }

  get toPlanet(): BigInt {
    return this[3].toBigInt();
  }

  get popArriving(): BigInt {
    return this[4].toBigInt();
  }

  get silverMoved(): BigInt {
    return this[5].toBigInt();
  }

  get departureTime(): BigInt {
    return this[6].toBigInt();
  }

  get arrivalTime(): BigInt {
    return this[7].toBigInt();
  }
}
...
export class Contract extends ethereum.SmartContract {
....
  bulkGetPlanetArrivals(
    startIdx: BigInt,
    endIdx: BigInt
  ): Array<Contract__bulkGetPlanetArrivalsResultValue0Struct> {
    let result = super.call(
      "bulkGetPlanetArrivals",
      "bulkGetPlanetArrivals(uint256,uint256):(tuple[][])",
      [
        ethereum.Value.fromUnsignedBigInt(startIdx),
        ethereum.Value.fromUnsignedBigInt(endIdx)
      ]
    );

    return result[0].toTupleArray<
      Contract__bulkGetPlanetArrivalsResultValue0Struct
    >();
  }

  try_bulkGetPlanetArrivals(
    startIdx: BigInt,
    endIdx: BigInt
  ): ethereum.CallResult<
    Array<Contract__bulkGetPlanetArrivalsResultValue0Struct>
  > {
    let result = super.tryCall(
      "bulkGetPlanetArrivals",
      "bulkGetPlanetArrivals(uint256,uint256):(tuple[][])",
      [
        ethereum.Value.fromUnsignedBigInt(startIdx),
        ethereum.Value.fromUnsignedBigInt(endIdx)
      ]
    );
    if (result.reverted) {
      return new ethereum.CallResult();
    }
    let value = result.value;
    return ethereum.CallResult.fromValue(
      value[0].toTupleArray<Contract__bulkGetPlanetArrivalsResultValue0Struct>()
    );
  }

Im not too sure what multidimensional access should look like, anyone have an example?

OK So I think I understand whats needed now. Though I dont think I need this currently and wont continue. Just for anyone who follows. The goal is to generate something to be consumed by ethabi. Graph uses a fork of ethabi though
So I think that needs to be extended for multidimension
https://github.com/graphprotocol/graph-node/blob/master/graph/Cargo.toml#L24
but also keep your eye on the original because it could be done there too
https://github.com/rust-ethereum/ethabi


Add dependabot

Overview
Add dependabot to keep packages up to date.
Checklist

[x] This PR includes tests, or tests are not required/relevant for this PR
[ ] README.md updated and added information about my change
[x] CHANGELOG.md updated and added information about my change

Tested in my master branch


Fix bugs leading to servers not shutting down

This includes #2713 and #2726 which should be merged first.
Interesting commits are 1191e21, d1c0034, 98371d9.
Fixes #2730

@nicolasnoble - could you reassure googlebot?

I have reviewed the code in the three interesting commits and left my comments.

I have nearly LGTMed #2713, just waiting on a comment fix. When you get the chance, could you answer the question I had here about the shutdown flag? Thanks! I'd otherwise be ready to merge this.


Regarding experts.w1 and experts.w2 gradients

Hi @lucidrains
I am a little confused about how the parameters experts.w1 and experts.w2 are updated. The top1 operation is non-differentiable and therefore the gradients of these two parameters would be None. To confirm i even ran the following:
moe = MoE(
    dim = 512,
    num_experts = 16,               # increase the experts (# parameters) of your model without increasing computation
    hidden_dim = 512 * 4,           # size of hidden dimension in each expert, defaults to 4 * dimension
    activation = nn.LeakyReLU,      # use your preferred activation, will default to GELU
    second_policy_train = 'random', # in top_2 gating, policy for whether to use a second-place expert
    second_policy_eval = 'random',  # all (always) | none (never) | threshold (if gate value > the given threshold) | random (if gate value > threshold * random_uniform(0, 1))
    second_threshold_train = 0.2,
    second_threshold_eval = 0.2,
    capacity_factor_train = 1.25,   # experts have fixed capacity per batch. we need some extra capacity in case gating is not perfectly balanced.
    capacity_factor_eval = 2.,      # capacity_factor_* should be set to a value >=1
    loss_coef = 1e-2                # multiplier on the auxiliary expert balancing auxiliary loss
).cuda()
inputs = torch.randn(4, 1024, 512).cuda()
out, aux_loss = moe(inputs) # (4, 1024, 512), (1,)
aux_loss.backward()
for name, param in moe.named_parameters():
    if param.grad is None:
        print(name)

which gave the following output:
experts.w1
experts.w2

It would be really helpful if you could clarify my understanding. Thanks

same question + 1


IIB Doesn't Work Properly with Syslink Folders

It took me a while to realize, but the reason my WebUI linking doesn't work is because my image output is a syslink folder leading to a different location. It seems to fail to find the folder for some reason? But it starts working fine again if I manually add the folder itself. Like all the txt2img and img2img folder options appear as well on their own.
Problem being I have to do it each time I launch the app, which makes it a pain to use.

Hello, we tried to solve the issue.
This is what we did:
Modified the get_valid_img_dirs function to follow symlinks when scanning for image directories. This should resolve the issue with symlinked folders not being recognized automatically.
You can review changes in this commit: https://github.com/mark-addams001881/zanllp-sd-webui-infinite-image-browsing-766/commit/1de11f6f48782203fa1aca3c2cab795a38401090.

[!CAUTION]
Disclaimer: The concept of solution was created by AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

Latta AI seeks to solve problems in open source projects as part of its mission to support developers around the world. Learn more about our mission at https://latta.ai/ourmission . If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.


Question: Log transport

Hi,
since loggers like winston are instrumented, does this mean all logs are transported to Azure? I haven't found a way to access them in the portal.
Thank you

Hi @visiongeist It should be transporting them to Azure. Which version of winston are you using? We support winston 2.x and 3.x. 3.x support was added in 1.2.0 of this library. If this all sounds fine to you and it is still not working can you post a repro of how you are setting up winston and logging with it?

Hi,
Thank you for your response!
I'm using 2.4.1 and I'm not seeing any logs under Search even though application map works!
`
const appInsights = require('applicationinsights');
appInsights.setup().start(); // using ENV variable
const winston = require('winston');
winston.log('info', Start server ${process.env.npm_package_version})
`

@visiongeist I'm unable to repro this -- it seems to be showing up for me. Are you requiring in winston somewhere else first in your app (e.g. in a different file)? For the auto transporting to work, App Insights needs to be the first thing you require (or at least be before everything that you want it to monitor).
const winston = require('winston');
setInterval( () => winston.log('info', `Starting server (my winston logger)`), 10000);

Hi,
I have same issue.
I can see the dependencies in the portal but not the traces from winston:

I can see Winston logs in the console and the file

Our first require is applicationinsights
"applicationinsights": "1.3.0",
"body-parser": "1.18.3",
"compression": "1.7.3",
"cors": "2.8.5",
"express": "4.16.4",
"ioredis": "4.6.2",
"socket.io": "2.2.0",
"socket.io-redis": "5.2.0",
"spdy": "4.0.0",
"uuid": "3.3.2",
"winston": "3.2.1"
index.js
const appInsights = require('applicationinsights');
const spdy = require('spdy');
const fs = require('fs');
const path = require('path');
const app = require('./app');

const config = require('./config');

appInsights
  .setup(config.INSTRUMENTATION_KEY)
  .setAutoDependencyCorrelation(true)
  .setAutoCollectRequests(true)
  .setAutoCollectPerformance(true)
  .setAutoCollectExceptions(true)
  .setAutoCollectDependencies(true)
  .setAutoCollectConsole(true)
  .setUseDiskRetryCaching(true);
appInsights.defaultClient.context.tags[appInsights.defaultClient.context.keys.cloudRole] =
  'sockets-server';
appInsights.start();

const { SERVER_KEY_PATH, SERVER_CRT_PATH, SERVER_INTERM_PATH } = process.env;
let serverOptions = {};
if (SERVER_KEY_PATH || SERVER_CRT_PATH || SERVER_INTERM_PATH) {
  serverOptions = {
    key: fs.readFileSync(SERVER_KEY_PATH),
    cert: fs.readFileSync(SERVER_CRT_PATH),
    ca: fs.readFileSync(SERVER_INTERM_PATH),
  };
} else {
  serverOptions = {
    key: fs.readFileSync(path.join(__dirname, '../server.key')),
    cert: fs.readFileSync(path.join(__dirname, '../server.crt')),
  };
}

const server = spdy.createServer(serverOptions, app).listen(app.get('port'), (error) => {
  if (error) {
    console.error(error);
  }
  console.log(`Server listening on port ${app.get('port')}`);
});

require('./sockets/startSocketsServer')(server);

loggers.js
const { createLogger, format } = require('winston');

const { combine, timestamp, label, printf } = format;

const prettifyMessage = (message) => JSON.stringify(message, null, 2);
const myFormat = printf(
  (info) =>
    `${info.timestamp} - ${info.id} | [${info.label}] (${info.level}): ${prettifyMessage(
      info.message,
    )}`,
);

const myLogger = createLogger({
  level: 'info',
  format: combine(label({ label: 'myLogger ' }), timestamp(), myFormat),
});

const otherLogger = createLogger({
  level: 'info',
  format: combine(label({ label: 'otherLogger ' }), timestamp(), myFormat),
});

module.exports = {
  myLogger,
  otherLogger ,
};

transports.js
const path = require('path');
const fs = require('fs');
const { transports } = require('winston');

const logsPath = path.join(process.cwd(), 'logs');

if (!fs.existsSync(logsPath)) {
  fs.mkdirSync(logsPath);
}

const infoLogFilename = path.join(logsPath, 'info.log');
const errorsLogFilename = path.join(logsPath, 'errors.log');

const fileTransports = [
  new transports.File({
    filename: infoLogFilename,
    maxsize: 5000000,
    maxFiles: 20,
  }),
  new transports.File({
    filename: errorsLogFilename,
    handleExceptions: true,
    level: 'error',
    maxsize: 5000000,
    maxFiles: 20,
  }),
];

const consoleTransports = [new transports.Console()];

const productionTransports = [...fileTransports];
const developmentTransports = [...consoleTransports, ...fileTransports];

module.exports = { productionTransports, developmentTransports };

initlogger.js
const { loggers } = require('winston');
const { productionTransports, developmentTransports } = require('../utils/logger/transports');
const { myLogger, otherLogger } = require('../utils/logger/loggers');

const addTransports = () => {
  if (process.env.NODE_ENV !== 'production') {
    loggers.options.transports = developmentTransports;
  } else {
    loggers.options.transports = productionTransports;
  }
};

const addLoggers = () => {
  loggers.add('MY', myLogger);
  loggers.add('OTHER', otherLogger );
};

module.exports = function initLogger() {
  addTransports();
  addLoggers();
};

We init express and the logger in the app.js
const express = require('express');

const app = express();

require('./init/initExpress')(app);
require('./init/initLogger')();

module.exports = app;

We use the logger for instance:
const { loggers } = require('winston');

const logger = loggers.get('MY');
const id = 'myId';
const myRequest = 'my request';
logger.log({ level: 'info', id, message: { type: 'REQUEST', myRequest } });

Hey @aparralorenzo thank you so much for the repro! I believe this is a different issue than OP's, since app insights uses different patches for winston v2 and v3. To give some context, I add an app insights transport whenever you call createLogger or configure, as well as adding it to the "root" winston object, e.g. winston.log('this winston has the app insights transport').
With your repro, I believe when you set your transports, the app insights one is being overwritten here.
loggers.options.transports = developmentTransports;

You can try adding these transports to your loggers via a call to .configure(...) or when you call createLogger(...). This way, it will preserve the app insights transport.
Alternatively, you can add our transport to your code and modify it to just call trackTrace on  appInsights.defaultClient instead.
https://github.com/Microsoft/node-diagnostic-channel/blob/ea26dc35f7a27e7e00644f764e7b875cf435711d/src/diagnostic-channel-publishers/src/winston.pub.ts#L58-L89

Thank you @markwolff for the response! Can you provide an example adding AppInsightsTransport to logger.options.transport, please

@aparralorenzo Sure thing. Feel free to modify the call to trackTrace to modify how you want the message to look. Also note that this will only work on winston 3.x
// App Insights setup
const appInsights = require('applicationinsights');
appInsights.setup('')
    .setInternalLogging(true,true)
    .start();
appInsights.defaultClient.config.maxBatchSize = 1;

// Require in Winston 3.x
var winston = require('winston');
var {createLogger, transports, loggers} = winston;

// App Insights custom Transport
class AppInsightsTransport extends winston.Transport {
    mapLevelToKind(winston, level) {
        let levelKind;
        if (winston.config.npm.levels[level] != null) {
            levelKind = "npm";
        } else if (winston.config.syslog.levels[level] != null) {
            levelKind = "syslog";
        } else {
            levelKind = "unknown";
        }
        return levelKind;
    };

    constructor(winston, client, opts) {
        super(opts);
        this.winston = winston;
        this.client = client;
    }

    log(info, callback) {
        const {message, level} = info;
        const levelKind = this.mapLevelToKind(this.winston, level);
        const meta = {};

        if (typeof Symbol["for"] === "function") {
            const metaSearch = info.meta ? [info.meta] : undefined || info[Symbol["for"]("splat")];
            if (metaSearch) {
                for (const key in metaSearch[0]) {
                    if (meta.hasOwnProperty) {
                        meta[key] = metaSearch[0][key];
                    }
                }
            }
        }

        this.client.trackTrace({message: `${level}: ${message}`, properties: meta});
        callback();
    }
}

// Create the loggers
const myLogger = createLogger({
    level: 'info'
});
loggers.add('MY', myLogger);
loggers.options.transports = [new AppInsightsTransport(winston, appInsights.defaultClient)];

const logger = loggers.get('MY');
logger.info('hello world');

u requiring in winston somewhere else first in your app (e.g. in a different file)? For the auto transporting to work, App Insights needs to be the first thing you require (or at least be before everything that you want it to monitor).

Hi. Sorry for the late answer. That worked for me.
I am investigating why it does not work in production

Closing this since it seems both issues are resolved.
@visiongeist please reopen if you are still having issues in prod
@aparralorenzo Did adding the transport work out for you? Let me know!

@markwolff it works! thanks!

Not to revive an old thread, but I ran into this just now and it's because I was running the app in vs code.
VS code adds 'APPLICATION_INSIGHTS_NO_DIAGNOSTIC_CHANNEL' environment variable which prevents logging the traces.  If you add the following to the top of your entrypoint script, you will get winston logs in AI.
delete process.env['APPLICATION_INSIGHTS_NO_DIAGNOSTIC_CHANNEL'];
I did not have to make any modifications to Winston or the transports to make this work, it worked out of the box.


Address vs. nid types

The Address and nid types are both defined as aliases of nat. This is fine as far as it goes, but they seem also to be used as if they always are in fact the same type. For example, we have
Parameter genProof : Address -> Blockchain -> option VProof.

But the genProof function is then used in the procInt function as follows:
let: attempt := genProof n bc in

where n is nominally of type nid.
Wouldn't it make more sense to let Address be an arbitrary Type, eqType, or ordType? That Address and nid are really the same type here seems accidental and restrictive for anyone implementing genProof. Another solution that at least makes things clearer is to explicitly assign nid to be Address or eliminate nid altogether.

Yes, it all makes sense. A pull request for eqType instead of nat would be very welcome.

I looked a bit closer at this now, and the problem with simply switching addresses and nids to eqType is the following in States.v:
Definition StateMap := union_map [ordType of nid] State.
Fixpoint initState' n : StateMap := if n is n'.+1 then (n \\-> Init n) \+ (initState' n') else Unit.

What appears to be needed for a fully satisfactory solution is to make Address both an ordType (to be able to use union_map) and a finType (to be able to initialize states for all nodes). Then, one has to reestablish the necessary lemmas, e.g., initStateValidDom.
This is above my bandwidth threshold for the moment, but maybe I can look into it in the near future. In the meantime, I will submit a simple pull request that simply replaces nid with Address (which should remove the confusion I had when reading the source).


Add number_with_delimiter to long counters

Makes long counters easy to read:

@fxn What do you think?

By now I'll leave it as it is. Thanks for the patch anyway @claudiob ❤ .


Fix documentation Link

Link goes to 404 currently

Closed in favor of Monorepo PR: https://github.com/spiral/framework/pull/832


H2 - Demographics Page Dropdown

Selecting a different category via the dropdown menu should display only the relevant footnote and hide others.

This looks good, closing.


Fix valgrind errors in bootstrap daemon.

Signed-off-by: David Antliff<EMAIL_ADDRESS>

[valgrind]

[valgrind]

retest this please

retest this please

[valgrind]

retest this please

[valgrind]


fix: Fix security findings

Update to use tempfile instead of the deprecated tempdir
Update Python dependencies

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Files
Coverage Δ

src/resources/trigger.rs
79.50% <100.00%> (ø)


Edge cases in when_all

Fix the case where you call when_all with coroutines whose return values have an overloaded comma operator. The existing code would accidentally comma the coroutine results together instead of discarding them as intended. Fix by explicitly comma-ing with void. You cannot overload   the comma operator when void is one of the parameters, so this forces the built-in comma operator to be used.

Fix the case where you call when_all with no parameter. The existing code would end up with an empty body, which   lacked any co_await or co_return statements and therefore   wasn't a coroutine any more. Fix by adding a co_return.

/azp run

D'oh, I found an easier solution this morning:
        (void(co_await async), ...);

but the version I originally submitted still works, so I guess not worth tweaking it now.

Always open to improvements that make it simpler. 👍


MLDB-1738 tensorflow builtin functions

Adds the ability to call a Tensorflow function as a builtin by appending tf_ to its name.

Ignore the test failure which is simple to fix

Ignore the test failure which is simple to fix

Are there any stability implications for this PR on existing functionality?

There shouldn't be.  The only code paths changed are in the processing of embeddings but they were essentially bug fixes.

Ok. Just to be on the safe side, since I want to do a release Tuesday so let's not merge into master until then. That'll give us more time to shake out bugs if any arise.

OK, 10-4 on that

Test failure is once again spurious.

+1


i18n(zh-cn): Update community-resources/content.mdx

Description (required)
Update community-resources/content.mdx
Upstream PR: #8294
Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

zh-cn
community-resources/content.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


RD2: Fix Callable API service to return installments when End Date is not provided

Critical Changes
Changes
Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

Tracking W-8524563


Buy and sell.

Hello !
I gave the two permissions at my players : shopchest.sell & shopchest.buy but they can't buy in the shops.
Why ?
(Using 1.11.2 and the latest dev build).
Thanks !

If you're getting "You can't buy/sell anything here", it's because of
WorldGuard. Check the FAQ of this plugin's spigot page.
Am 11.02.2017 10:46 schrieb "TheIntelloBoX"<EMAIL_ADDRESS>Hello !
I gave the two permissions at my players : shopchest.sell & shopchest.buy
but they can't buy in the shops.
Why ?
(Using 1.11.2 and the latest dev build).
Thanks !
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/EpicEricEE/ShopChest/issues/78, or mute the thread
https://github.com/notifications/unsubscribe-auth/AG26mJaF5HkKesvOXgxYA4DtzXVcZeCHks5rbYN5gaJpZM4L-IgC
.


Update index.js

Issue: #168
Description
We used next/router for solving this.

📝 Changed routes:

/

(view full size)

Commit 18a2e568767baa2ab3b1f9d465771437642f209b (https://javaistic-lchfvmht8-javaistic.vercel.app).


Should Run use the visible code or only the "Dart" tab's code?

The Run button should either run the code that's currently visible (e.g. the code in the Solution tab) or somehow indicate (by switching tabs?) that it's running the user's code.
To reproduce:

Go to https://jt-ww-1.firebaseapp.com/codelabs/async-await#exercise-2
Click Hint.
Click Show solution.
Click Run. The tests fail. This is where I stopped, assuming there was a bug in the codelab. :)
Copy the solution code into the Dart tab and then click Run. The tests work.

/cc @legalcodes

Should be easy enough to implement, possibly at the same time as #1119.

Thanks for filing this issue! It's on my todo list for a while.
Since this may affect analytics (test results), I can think of two ways:
Option 1: The Run button runs the code that's currently visible (e.g. the code in the Solution tab)
In the analytics, we add an extra tag to track the which tab is running because the solution code (without edits) should always pass the test. We may filter out this test result.
Option 2: The Run button run the users's code
This means we need to somehow indicate (by switching tabs?) that it's running the user's code
Also maybe add a button to allow users quickly replace their code with solution?
For analytics, we may want to know whether a solution (and hint) is presented while users run the code.
@jayoung-lee Do you have any suggestion from the analytics perspective?

I'd vote for option 1, and yes I think separating hint-run and code-run is a good idea.
Either way, Analytics should match whatever the user thinks he/she is doing :)

Since https://github.com/dart-lang/dart-pad/issues/2808 dropped support for multiple files, there is only a "Dart" tab now.


Action 'Move closest gridline to mouse cursor' does not work with time signature marker without tempo information

Hello,
I have a problem with the behaviour of action:
'SWS/BR: Move closest grid line to mouse cursor (perform until shortcut released)' .
In case I insert just 'time signature marker' without tempo information in the marker, I get weird behaviour like on the attached gif.

I would expect the time signature marker to not interfere with tempo markers, probably just ignore them while using this action?
Thank you for great work!
Peter

It looks like if the previous point does not set a BPM, then updating its BPM does nothing.
Perhaps this line needs to be more discerning about which point to choose (presumably the previous point that actually sets a tempo).
If we do that, there's another corner case that might need handling here to ensure there's always a point in the tempo envelope that actually sets tempo.

I couldn't find a way to determine whether a tempo envelope point explicitly sets a BPM, so perhaps the only fix would be to add a bpm to the time signature and use that as an anchor.
I was able to handle the case where the very first point is a time signature by simply providing its BPM in InitTempoMap(). However, I haven't had any luck doing similar things for subsequent points like in the original report.

I think I found a fix, unclear if it's a good idea but here it is: https://github.com/reaper-oss/sws/pull/1738

Try the action Set default mouse modifier action for "Project tempo/time signature marker left drag" to "Move project tempo/time signature marker, adjusting previous and current tempo", it seems to do what you want.


Return @mentions preceeded by @

Such as @@kn
These changes bring the library into alignment with the actual Twitter
website and API as shown here:
https://twitter.com/StreetGeekEnt/statuses/554800426768277504
https://twitter.com/kung_fu_mike/status/570024458640973824
This commit only fixes the Ruby, Java, and JavaScript Libraries. I
don't have an environment for Objective C development.

ping?

@codemonkey3045 Any update on this? I guess this is a bit too outdated since the release of version 2+? Thanks in advance!

Pretty sure we can close this request. I will look to add unit tests showing support for @@mentions


Scheduler implementation

This pull request aims to implement the scheduler

Currently implementing gRPC interfaces for the Agent.
RPC "RegisterAgent" interface complete + test client
RPC "ReportHealthStatus" interface complete + currently working on test client (after dodo :hurtrealbad:)

See next steps here: https://github.com/orgs/dev-sys-do/projects/2

The interfaces themselves are mostly done, + integration tests (on interfaces) with some mock examples (to expand)
Started coding some Agent pool logic for registration

@hugoponthieu
I finished implementing the missing interface to send an Action. I am not yet able to test it, since I need a mock controller/agent, but as far as the gRPC interfaces go, it works.
I'll try to write some grpcurl commands if I can get my hands on a mock Agent. And I'll deploy this version on my remote server so we can test it properly and connect your Agent to it.
You can review the code, don't hesitate to ask any question if you are having trouble understanding some stuff, or if it lacks comments in some parts
Lastly, controller_interface contains crude logic code, which will need to be put in its own module; for clarity

The main issue now is that probably lots of error handling is missing. Like some checks on grpc interfaces ; I would need help with that @dev-sys-do/sealci-scheduler

Nice code base

I think you can review and resolve your requested changes? :)


New features

git client support
repository substitution
transitive exclusion

Coverage increased (+1.7%) to 98.818% when pulling d063324d712811cbe2deb5dc0dd36d097b7b415e on git into 9a0f480855a44adfced984b73d162b4da75b197b on master.


fix: only positive numbers in product-quantity

PR Type

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] CI-related changes
[ ] Documentation content changes
[ ] Application / infrastructure changes
[ ] Other:
What Is the Current Behavior?
When you enter negative numbers as product quantity, the form will display a warning and not set the value.

What Is the New Behavior?
When you enter negative numbers as product quantity, they get inverted to their positive counterparts and the value can be set. This prevents negative numbers from being used in the first place as this doesn't make sense for quantity values.
Does this PR Introduce a Breaking Change?

[ ] Yes
[x] No
Other Information
Possible solution to #849

I have an idea of a better way to fix this. Please don't merge yet as I want to test this other idea

I also just thought about the pattern based approach. Might try this tomorrow an create another PR

Just checking: https://caniuse.com/input-pattern It actually seems supported pretty well. So please, go ahead 👍

I added the pattern to the input element. But since pattern is only a guide for the browser, I also added additional functions to validate the input. This prevents the user from typing or pasting non numerical characters in the input field. However it does not check if the actual value of the input field is within the given boundaries. Because of this it does not qualify as a fix for #849


BUG: Repetition of icons

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The same icons are present twice on the webpage
SS :

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

I would like to work on this issue please assign this task to me

@arghadipmanna101 Please assign me this issue


fix: enable semantic releae

Impact: minor
Type: chore
Solution
Added semantic-release
Breaking changes
None
Testing
The PR has been tested locally and was pushing the right versions. This has also been tested with the --dry-run flag and everything looked fine. But with such PRs there is always a risk of something going wrong with permissions and thus failing. The plan to address those is to fix the issue and create a next patch release.

:tada: This PR is included in version 4.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


feat: add transformScale

the PR: https://github.com/STRML/react-grid-layout/pull/987  is to fix grid item drag to error place when parent node has css-style tranform: scale(0.x)
this PR to fix grid item resize error when parent node has css-style tranform: scale(0.x)
Please contact me if  any questions.

@STRML have a look. pls

@STRML pls merge this pr & upgrade react-resizable version


Test directory is wrong?

Hi,
I think that the test directory has wrong name.
Today it's name is "it", but need be by default "test".
The problem is that commands like this sbt test didn't found the tests.
One solution can be add this line in build.sbt file:
scalaSource in Test := baseDirectory.value / "src/it/scala",

Example:
lazy val djinni = (project in file("."))
  .configs(IntegrationTest)
  .settings(
    name := "djinni",
    Defaults.itSettings,
    scalaSource in Test := baseDirectory.value / "src/it/scala",
    libraryDependencies += "org.scalatest" %% "scalatest" % "3.0.5" % "it",
    libraryDependencies += "org.scala-lang.modules" %% "scala-parser-combinators" % "1.1.2",
    libraryDependencies += "org.yaml" % "snakeyaml" % "1.26",
    libraryDependencies += "com.github.scopt" %% "scopt" % "4.0.1",
    libraryDependencies += "commons-io" % "commons-io" % "2.11.0",
    assemblyOutputPath in assembly := { file("target/bin") / (assemblyJarName in assembly).value },
    assemblyJarName in assembly := s"${name.value}",
    assemblyOption in assembly := (assemblyOption in assembly).value.copy(prependShellScript = Some(defaultUniversalScript(shebang = false))),
    test in assembly := {}
  )

But when SBT find the tests, it has problem with dependencies:
sbt test
[info] welcome to sbt 1.4.7 (Oracle Corporation Java 11.0.12)
[info] loading settings for project djinni-generator-build from plugins.sbt ...
[info] loading project definition from /Users/paulo/Developer/workspaces/java/djinni-generator/project
[info] loading settings for project djinni from build.sbt ...
[info] set current project to djinni (in build file:/Users/paulo/Developer/workspaces/java/djinni-generator/)
[info] compiling 3 Scala sources to /Users/paulo/Developer/workspaces/java/djinni-generator/target/scala-2.12/test-classes ...
[error] /Users/paulo/Developer/workspaces/java/djinni-generator/src/it/scala/djinni/GeneratorIntegrationTest.scala:3:12: object scalatest is not a member of package org
[error] import org.scalatest.GivenWhenThen
[...]

What do you think in solve it and move to correct place and solve dependencies problem?

It will solve test problems and format problems.

There is nothing wrong with the directories. it is the default directory for integration tests. Pls have a look at the developer guide on how to execute integration tests.
We currently do not have any unit-tests, that's why the test folder is missing.


Discord bot

I added a supabase table for keeping track of markets and threads via message ids but this means old markets that aren't in the table will be ignore. Perhaps worth waiting until deploying the new version until there's a lull or add back in the old code to handle the old markets for a while.


feat(substrait): support order_by in aggregate functions

Which issue does this PR close?
Closes #13113.
Rationale for this change
Adding support to a new sort of expression (ORDER BY within aggregation functions) in the Substrait consumer.
For example:

SELECT ARRAY_AGG(genre ORDER BY genre) FROM book

What changes are included in this PR?
The wiring to go from Substrait's SortField to DataFusion's SortExpr in the context of aggregation measurements.
The actual conversion already existed at from_substrait_sorts because of window functions - I just hooked up in the context of aggragations as well.
Are these changes tested?
I added a roundtrip test and also added a logical plan to check if everything was decoded correctly. Open to more ideas on how to better unit test this.
Are there any user-facing changes?
n/a

LGTM. Thanks @bvolpato

Thank you! Sorry, I've missed a cargo fmt and we may need to trigger the checks again.

Thanks @bvolpato @vbarua and @andygrove
Sorry for the delay in merging this one


ScaleManager.windowConstraints has the wrong typescript type

The 'bottom' and 'right' properties of windowConstraints are declared as boolean, however they should really be string types.  This is a property for ScaleManager.

Fixed in dev.


Update prepare-tls-1.2-in-office-365.md

Added a statement stating this is completed.
Fixes https://github.com/MicrosoftDocs/microsoft-365-docs/issues/12226

@JohanFreelancer9 please copy review

@KCCross Please merge this PR. Thanks

Closing as per the comment in #12226


feat(components): handle hotkeys by event.code

Description
In the current version hotkeys that use English letters (alt+f, alt+s, alt+d, ctrl+alt+r) are handled by event.key, which makes it impossible to use them in other language keyboard layouts.
Handling keys with event.code fixes this problem.

Additional context

What is the purpose of this pull request?

[ ] Bug fix
[X] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[X] Read the Contributing Guidelines.
[X] Read the Pull Request Guidelines and follow the Commit Convention.
[X] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[X] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).

Problem with code is that it ignores keyboard layouts and try to force a QWERTY layout on the user keyboard :|

@Akryum maybe you can suggest ways of solving this problem?


Dark Mode

Related component
It affects core colours, so will affect most components in some way but perhaps indirectly.
Context
A lot of operating systems now support 'Dark Mode'. Modern browsers can support CSS for prefers-color-scheme but GOV.UK does not provide it as an enhancement. When browsing GOV.UK content in dark mode, the background is still pure white which can be quite harsh on the eyes in low light environments.
Alternatives
N/A
Additional information (if applicable)
Not really requesting the feature be implemented at this stage, just wanting to open up the discussion and to see if it's something we perhaps need to consider.

Hey @abbott567,
Thanks for raising this. There's already a backlog issue for dark mode with some discussion already happening – are you happy to continue this discussion there instead?

Hey @abbott567,
Thanks for raising this. There's already a backlog issue for dark mode with some discussion already happening – are you happy to continue this discussion there instead?

Hi, @36degrees , of course. As usual I just couldn't find anything already open!
Checked Design System and Frontend, but not the Backlog!
I'll add that to my places to check in future!

Hi, @36degrees , of course. As usual I just couldn't find anything already open!
Checked Design System and Frontend, but not the Backlog!
I'll add that to my places to check in future!

Closing in favour of https://github.com/alphagov/govuk-design-system-backlog/issues/177

Closing in favour of https://github.com/alphagov/govuk-design-system-backlog/issues/177


provide iOS .ipa

Could you provide the iOS ipa package for self-signed use?

The ios version of the application is available, you can download the latest version to use it.


jitclass doesn't work when dict class attribute has array as value type

Reporting a bug

[x] I have tried using the latest released version of Numba (most recent is
visible in the change log (https://github.com/numba/numba/blob/main/CHANGE_LOG).
[x] I have included a self contained code sample to reproduce the problem.
i.e. it's possible to run as 'python bug.py'.

Trying to jitclass a class that has a Dict attribute with an array as the value type fails. Code and traceback below. Removing jitclass results in code that runs fine.
import numpy as np
from numba import int64
from numba.experimental import jitclass
from numba.typed import Dict
from numba.types import DictType

@jitclass(
    [
        ("dict", DictType(int64, int64[:, :])),
    ]
)
class StructureManager:
    def __init__(self):
        self.dict = Dict.empty(key_type=int64, value_type=int64[:, :])

s = StructureManager()
s.dict[0] = np.zeros((2, 2), dtype=int)

Traceback (most recent call last):
  File "/Users/dstansby/projects/brainglobe/cellfinder-core/test.py", line 18, in <module>
    s = StructureManager()
  File "/Users/dstansby/mambaforge/envs/cellfinder-core/lib/python3.10/site-packages/numba/experimental/jitclass/base.py", line 124, in __call__
    return cls._ctor(*bind.args[1:], **bind.kwargs)
  File "/Users/dstansby/mambaforge/envs/cellfinder-core/lib/python3.10/site-packages/numba/core/dispatcher.py", line 468, in _compile_for_args
    error_rewrite(e, 'typing')
  File "/Users/dstansby/mambaforge/envs/cellfinder-core/lib/python3.10/site-packages/numba/core/dispatcher.py", line 409, in error_rewrite
    raise e.with_traceback(None)
numba.core.errors.TypingError: Failed in nopython mode pipeline (step: nopython frontend)
Failed in nopython mode pipeline (step: nopython frontend)
No implementation of function Function(<built-in function getitem>) found for signature:

 >>> getitem(class(int64), UniTuple(slice<a:b> x 2))

There are 22 candidate implementations:
  - Of which 22 did not match due to:
  Overload of function 'getitem': File: <numerous>: Line N/A.
    With argument(s): '(class(int64), UniTuple(slice<a:b> x 2))':
   No match.

During: typing of intrinsic-call at /Users/dstansby/projects/brainglobe/cellfinder-core/test.py (15)
During: typing of static-get-item at /Users/dstansby/projects/brainglobe/cellfinder-core/test.py (15)

File "test.py", line 15:
    def __init__(self):
        self.dict = Dict.empty(key_type=int64, value_type=int64[:, :])
        ^

During: resolving callee type: jitclass.StructureManager#115e0b7f0<dict:DictType[int64,array(int64, 2d, A)]<iv=None>>
During: typing of call at <string> (3)

During: resolving callee type: jitclass.StructureManager#115e0b7f0<dict:DictType[int64,array(int64, 2d, A)]<iv=None>>
During: typing of call at <string> (3)

File "<string>", line 3:
<source missing, REPL/exec in use?>

Unfortunately, use of int64[:, :] style type declarations inside JITted code does not work. You can hoist that definition outside the JIT region like this:
vt = int64[:, :]
@jitclass(
    [
        ("dict", DictType(int64, vt)),
    ]
)
class StructureManager:
    def __init__(self):
        self.dict = Dict.empty(key_type=int64, value_type=vt)


Research Using Third-Party Command-Line Option Parsing Library

Description
JenAI currently parses command-line options manually, which makes the code less legible and maintainable. To improve these aspects, it would be beneficial to leverage a well-established, third-party Java library for command-line option parsing.
Problem Statement
The current implementation of parsing command-line options within JenAI's codebase leads to several issues:

Code Duplication and Repetition: The parsing logic duplicates several parts of low-level processing, making it harder to understand the flow and maintain consistency.
Error Handling Complexity: Manual parsing can lead to more complex error handling, as the application needs to manage and report issues related to option validation and incorrect usage.
Limited Flexibility and Extensibility: Hardcoding parsing logic restricts the ability to easily add or modify options in the future, potentially requiring significant changes to the application's core.

Solution Proposal
To address these challenges, the suggestion is to explore and integrate a reputable, open-source Java library for command-line option parsing. Some popular options include:

Apache Commons CLI
JCommander
Args4J

The chosen library should provide features such as:

Easy option definition and validation
Support for various option types (e.g., flags, values, subcommands)
Built-in help and usage message generation
Error handling and reporting mechanisms
Compatibility with Java 21 and above (JenAI's current target version)

Acceptance Criteria:

The selected library is thoroughly tested in a local environment to ensure compatibility with JenAI's existing architecture and command-line interface.
All manual command-line option parsing logic is refactored to utilize the chosen library, promoting code consistency and readability.
Error handling and reporting mechanisms are updated to utilize the library's features, providing a more robust and user-friendly experience.
The migration to the new parsing library does not introduce any significant performance regressions or compatibility issues with existing use cases.

A good starting point is Apache Commons CLI.
Consider the following resources:

Official documentation
Usage example

This is also a great opportunity to add a suite of tests for the configuration built from the arguments to the main function. Consider either making the parse method public, or testing through main itself (might have to create mock/spy of CLISession in that case).

Another alternative to check out is JCommander.
Some resources:

Official documentation
Tutorial with some examples

After checking out both Apache Commons CLI and JCommander, I chose to pick JCommander from version 1.7.1 onwards.
While both libraries worked fine for parsing the options, JCommander has a more modern approach (using a class to represent the possible options and annotations to configure them) and offered built-in type validation - maybe Apache Commons CLI also can be configured to validate types, but I did not find anything about it while reading the documentation and setting it up in the code.
Also, before checking both libraries I wrote unit tests for all of the current command line options. I tried to do this through the main method initially, but that quickly got quite messy, so instead I made the method that parses the options public and tested it directly. As a bonus, that allowed for changing several static properties into local variables, which I think ended up being a better design overall.


fix: spinner and env injection in upgrade, some bugs in install/link

The spinner and env injection was broken for jspm upgrade, and there were some
potential race conditions in jspm install and jspm link - they called out to
generator.addMappings without using await!
The tests will fail until https://github.com/jspm/generator/pull/213 lands.

Going to merge this for now as tests will pass once we publish a new patch on the generator and bump.


pick command stopped working

I am self hosting the bot, everything was working fine and the pick command stopped working for some reason, plant and award work fine but pick doesn't

Ask for help in #help on Nadeko support server. Sounds like you messed with persmissions


Airflow URL is not injected in the UI when Airflow is installed in the background

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/986
Originally created by @mbergeron on 2019-09-16 13:16:51

We've updated meltano ui to automatically install Airflow in the background, which is a nice improvement for the UX.
Albeit that works locally, when the application runs on an external bind, the Airflow URL won't be injected.
To fix it, we need to refresh the UI template (i.e. so the API can inject the proper context post Airflow installation).
Now, there are multiple ways to fix this: we could always send the Airflow context, even when it is not installed, but that might still fail if users customize the Airflow port in the Meltano config.
A quick fix here would be to force the Orchestration page to issue a browser reload when visited.

View 2 previous comments from the original issue on GitLab


Fix menu order 9_adding-navigation.md

Description

Related Issues

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


PWN-4806-Fix country code names

Jira Ticket
https://p2pvalidator.atlassian.net/browse/PWN-4806
Description of Work
Fix country code names
Screenshots (Optional)

Screenshot #1
Screenshot #2
Screenshot #3

[Paste Pic 1]
[Paste Pic 2]
[Paste Pic 3]

strange fix, but ok


Adding validation on file upload, package updates, css cleaning

Adding validation to prevent non-csv file types from being able to be screened.

This is over and above the 'suggestion' made to browsers by fileInput(..., accept = ".csv") in the ui.R script which only tells a browser to expect a certain filetype, and doesn't prevent this from being overridden. As such, users could previously manually select all files in their file browser window and then upload and attempt to screen invalid file types, causing the app to crash without any feedback to the user as to why.
This PR should stop this from happening by giving a friendly error with useful feedback if an invalid filetype (anything other than csv) is uploaded. It.
The screening button should not appear while a warning is present, and warnings should dissapear if a correct filetype is uploaded.

The CSS has been updated to force the colour of the upload bars.

I have managed to fix the wiping out of the css on the upload bars. UI tests using shinytests weren't really doing much and were causing pain in the GHA workflow so I've dropped them entirely. Will consider RSelenium instead in a seperate branch as this should be stable enough for the time being.
For reference, my change was to add !important into the argument I wanted to make override other ones (.progress-bar(...; background-color: ...)).

I have 'beautified' the css code so it should be easier to track changes in the future.

The css change was impossible to spot unless you rendered the entire file. So I've ran it through a beautifier (https://www.cleancss.com/css-beautify/) so that when lines are edited we can track them better.

renv::update() was ran, updating a number of packages including shiny (to 1.6.0) and renv itself.

Have reverted the package updates for now as they seem to cause an issue with the deployment (have posted in the R Shiny Developers teams channel). So this PR is now for 1,2, and 3 only


Fatal server error: (EE) xf86OpenConsole: Cannot open /dev/tty0 (No such file or directory)

Crouton was working fine on my computer until a week or so ago.  Now whenever I try to open it (sudo startxfce4) or update (sudo sh -e ~/Downloads/crouton -n precise -u) it starts to update various things and then I get this:
Loading extension GLX
(EE)
Fatal server error:
(EE) xf86OpenConsole: Cannot open /dev/tty0 (No such file or directory)
(EE)
(EE)
Please consult the The X.Org Foundation support
at http://wiki.x.org
for help.
(EE) Please also check the log file at "/tmp/Xorg.crouton.1.log" for additional information.
(EE)
(EE) Server terminated with error (1). Closing log file.
/usr/bin/xinit: giving up
/usr/bin/xinit: unable to connect to X server: Connection refused
/usr/bin/xinit: server error
Unmounting /mnt/stateful_partition/crouton/chroots/precise...
What can I do to fix this?
Thank You

Please post the output of sudo sh  ~/Downloads/crouton -n precise -u probably your update is failing for some reason.

Here is the output:
Downloading latest crouton installer...
######################################################################## #######################                                                  ###############################################                          ######################################################################## 100.0%
/usr/local/chroots/precise already exists; updating it...
Preparing chroot environment...
Installing brightness into the chroot...
Installing croutonpowerd into the chroot...
Installing croutonversion into the chroot...
Installing host-dbus into the chroot...
Installing host-x11 into the chroot...
Installing volume into the chroot...
Installing pulseaudio-default.pa into the chroot...
Installing croutoncycle into the chroot...
Installing croutontriggerd into the chroot...
Installing croutonxinitrc-wrapper into the chroot...
Installing setres into the chroot...
Installing xinit into the chroot...
Installing xbindkeysrc.scm into the chroot...
Installing xorg-intel-sna.conf into the chroot...
Installing xserverrc into the chroot...
Installing xserverrc-xorg into the chroot...
Installing xserverrc-local.example into the chroot...
Installing startunity into the host...
Installing crouton-noroot into the chroot...
Installing startunity into the chroot...
Installing gnome-session-wrapper into the chroot...
Installing crouton-unity-autostart into the chroot...
Installing unity-autostart.desktop into the chroot...
Installing unity-profiled into the chroot...
Installing startxfce4 into the host...
Installing crouton-noroot into the chroot...
Installing startxfce4 into the chroot...
Installing enter-chroot into the host...
Installing delete-chroot into the host...
Installing edit-chroot into the host...
Installing mount-chroot into the host...
Installing unmount-chroot into the host...
Installing crash_reporter_wrapper into the host...
Installing target core...
Preparing environment...
Preparing software sources...
 Done
Ensuring system is up-to-date...
Reading package lists... Done
Building dependency tree
Reading state information... Done
Calculating upgrade... Done
The following package was automatically installed and is no longer required:
keyutils
Use 'apt-get autoremove' to remove it.
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Reading package lists... Done
Building dependency tree
Reading state information... Done
sudo is already the newest version.
apt-transport-https is already the newest version.
wget is already the newest version.
ca-certificates is already the newest version.
The following package was automatically installed and is no longer required:
keyutils
Use 'apt-get autoremove' to remove it.
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Generating locales...
en_US.UTF-8... up-to-date
Generation complete.
Generating locales...
en_US.UTF-8... up-to-date
Generation complete.
Syncing timezone...
Installing target audio...
Fetching CRAS (branch 103c37a8bf17ea65d590acd56f5d1a1778d27b1e)...
--2015-06-23 12:50:33--  https://chromium.googlesource.com/chromiumos/third_party/adhd/+archive/103c37a8bf17ea65d590acd56f5d1a1778d27b1e.tar.gz
Resolving chromium.googlesource.com (chromium.googlesource.com)... <IP_ADDRESS>, 2607:f8b0:4001:c02::52
Connecting to chromium.googlesource.com (chromium.googlesource.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: unspecified [application/x-gzip]
Saving to: '/tmp/crouton-cras.Yxt8Jl/adhd.tar.gz'
 0K .......... .......... .......... .......... ..........  933K
50K .......... .......... .......... .......... .......... 1.63M

100K .......... .......... .......... .......... .......... 2.17M
150K .......... .......... .......... .......... .......... 3.19M
200K .......... .......... .......... .......... .......... 2.84M
250K .......... .......... .......... .......... .......... 6.72M
300K .......... .......... .......... .......... .......... 2.01M
350K .......... .......... .......... .......... .......... 3.65M
400K .......... .......... .......... .......... .....      2.08M=0.2s
2015-06-23 12:50:34 (2.12 MB/s) - '/tmp/crouton-cras.Yxt8Jl/adhd.tar.gz' saved [456009]
Reading package lists... Done
Building dependency tree
Reading state information... Done
libasound2:i386 is already the newest version.
libspeexdsp1:i386 is already the newest version.
alsa-utils is already the newest version.
The following package was automatically installed and is no longer required:
keyutils
Use 'apt-get autoremove' to remove it.
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Reading package lists... Done
Building dependency tree
Reading state information... Done
gcc-multilib is already the newest version.
pkg-config is already the newest version.
gcc is already the newest version.
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
libc6-dev : Breaks: libc6-dev:i386 (!= 2.19-0ubuntu6.6) but 2.15-0ubuntu10.12 is to be installed
libc6-dev:i386 : Depends: libc6:i386 (= 2.15-0ubuntu10.12) but 2.19-0ubuntu6.6 is to be installed
Depends: libc-dev-bin:i386 (= 2.15-0ubuntu10.12)
Depends: linux-libc-dev:i386 but it is not going to be installed
Breaks: libc6-dev (!= 2.15-0ubuntu10.12) but 2.19-0ubuntu6.6 is to be installed
libspeexdsp-dev:i386 : Depends: libspeexdsp1:i386 (= 1.2~rc1-3ubuntu2) but 1.2~rc1.1-1ubuntu1 is to be installed
Depends: libspeex-dev:i386 (= 1.2~rc1-3ubuntu2) but it is not going to be installed
E: Error, pkgProblemResolver::Resolve generated breaks, this may be caused by held packages.
Failed to complete chroot setup.
Unmounting /mnt/stateful_partition/crouton/chroots/precise...

Before doing anything, backup your chroot
Then use sudo enter-chroot -n precise to enter the command line of your chroot so you can resolve your issue with libc6-dev : Breaks: libc6-dev:i386 which looks the same like https://github.com/dnschneid/crouton/issues/1694 Maybe following could help. https://bugs.launchpad.net/ubuntu/+source/eglibc/+bug/1365375 ...


Textfield inside Dialog in Safari - carret is hidden

If Textfield is inside of Dialog and initial value is set to '' - there will be no cursor in Safari

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔
Cursor must be visible in Safari like in other browser
Steps to Reproduce 🕹
Add Textfield with initial value '' in Dialog and open it in Safari
https://codesandbox.io/s/material-demo-exxfo
There will be background: none css in input, if you will uncheck it - everything will work as expected in Safari

Tech
Version

Material-UI
v4.0.2

React
16.9.0

Browser
Safari

@koshkarik It seems to also be related to the position: relative of the parent elements. Do you have an idea of how we can fix the problem? Do you think that you could open an issue on Safari side? It seems like a problem with their rendering engine.

Yep, i will open the issue on Safari side 👌

No luck to reproduce the problem with pure html and css in safari - so not enough facts to open issue on Safari side(

@oliviertassinari I didn't figure out how to fix this problem in mui( I do understand I can simply overwrite input styles (background) and it will solve the problem for me, but in our project we have a lot of inputs in modals, so this style fix will make a lot of crutches. Do you have any idea how to solve it in mui?

@koshkarik Copy & pasting the HTML output seems to be enough: https://codepen.io/anon/pen/agmmMR to reproduce. Maybe we can clean the codepen as much as possible to be minimal?

I have the same bug. But I found temporary solution.
default state for input:
input: focus {
outline: 0;
}
I set it up:
input:focus {
outline: 1;
outline-offset: 100000px
}

@oliviertassinari I have finally opened an issue on Safari side with your example https://bugs.webkit.org/show_bug.cgi?id=199188 If it will be needed i will clean up the example from the rest of unnecessary things. I hope they will reply soon about this issue, but there is a big chance that fixing this bug in Safari will take a while and a lot of people will face this problem for that period of time... May be it's possible to make quick fix in material ui, what do you think of it?

May be it's possible to make quick fix in material ui, what do you think of it?

@koshkarik How? Do you know a fix that can be deployed without breaking people code?

@oliviertassinari actually didn't figure out yet :(

I hope that the Safari team will be able to help. Given they have acces to advanced debugging tools, they might be able to explain what's going on.

@oliviertassinari how do you think, is it possible do add zIndex: 1 to InputBase somewhere here https://github.com/mui-org/material-ui/blob/master/packages/material-ui/src/InputBase/InputBase.js#L32 ? Because it definitely fixes this issue and i believe (have checked it, but don't know for sure) it doesn't break anything.

Hum. Adding a z-index would move the component into its own stacking context so it might break people. If we try it, it's also possible that we will have to revert it.

Hello! Any updates on this issue?

What's new?

I see this issue in Safari 12.1 on Mojave, but I noticed two things:

While it occurs when using Textfield directly from material-ui in a new react app, it does not appear when viewing the official material-ui page: https://material-ui.com/components/text-fields/ So there is something that is causing this to occur from the application/framework side (that possibly Safari is not handling correctly)

As for the Safari side, it is in fact fixed in Safari 13 in Catalina.

I think this is outdated, I cannot reproduce this in the latest Safari.


Key Feature Section Improvement

Description
The styling of the key Feature section is misaligned and has a basic layout, it may be due to some styling not being applied properly, causing it to misaligned. Here are a few steps to fix this:
Steps to Reproduce
Inspect how the feature list is currently positioned in your code. Look for any CSS properties applied on feature list classes. These could be causing it to be misaligned and  basic styling.
Expected Behavior
The feature section items should be placed on the left side and the image content on the right side, and with styling, greatly improve the user experience and website presentation.
Actual Behavior
the alinement of the feature section items with the image content are all in the left side with default styling
Screenshots

Additional Information
can you assign this issue to me with labels "gssoc-ext" with levels , "hacktoberfest-accepted";
What browser are you seeing the problem on?

[ ] Firefox
[X] Chrome
[ ] Safari
[ ] Microsoft Edge

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

this issue has been solved, so we are closing this issue.


#23 getCountryCode returns undefined

Fixes #23

@rsertelon can you please code review this?

You should be good to go with 1.2.2

@rsertelon thanks a lot!


Validating files against the wrong SHACL file

I used the validation tool to validate a bf:Work description. By accident, I loaded the wrong Shacl file -- the one for bf:Instance and it validated just fine. I had noticed that once before when I accidentally  used the Shacl for the wrong format type. In both cases I noticed simply because I knew that my RDF files SHOULD cause some errors so I looked for the problem. But if I hadn't I would have just assumed that my data is good.
So my question is: When I use the Shacl file for the Instance for a Work, shouldn't it tell me that all kinds of things are missing, such as bf:Instance or bf:Print and the required instance fields? Could we adjust the validation so that it catches such errors?

Email response from Steven: If the SHACL doesn’t have targets that match up with what’s in the data, then it doesn’t assume anything about the data. Maybe we could consider having the validation tool say it was unable to find any descriptions that matched up with the targets defined in the loaded SHACL, and the user should confirm they have the correct SHACL and data loaded.

After doing some research and testing, there is a way to add a SHACL rule to validate for the existence of a particular class(es). For example, the following SHACL nodeShape:
<big:Monograph:Work:Exists> a sh:NodeShape ;
    rdfs:label "Ensure Monograph and Text exists" ;
    sh:property [
        a sh:PropertyShape ;
        sh:path [ sh:inversePath rdf:type ] ;
        sh:minCount 1 ;
        sh:severity sh:Violation
     ] ;
     sh:targetNode bf:Monograph, bf:Text .

will raise a Violation error if incoming resource graph doesn't have a resource with both bf:Monograph and bf:Text classes. If a resource only has a bf:Monograph the validation will still fail because it is missing a bf:Text.
In our current DCTap structure, we could automatically generate this existence SHACL rule but we should only do this for the main BIBFRAME classes and not for the secondary nodeShapes in the DCTap file. The easiest approach would be to add a boolean column (maybe mustExist) that would create this rule. If adding a column is not desired, we would need to come up with a set of conditions for creating the existence SHACL rule from the DCTap row values but that may not be possible.

@jermnelson nice catch! Just to confirm, with sh:NodeShapes like this, we'd get a report that said no data was found for the particular sh:targetNode classes? It could be ignored if it's a bunch of heterogeneous data and may not have instances of the classes?


Migrate deployment to Puppet/Ansible/Chef

Instead of using a deployment script, use an industry standard tool. Consult UGent about institute-specific standards and practices.

See https://trello.com/c/Ek1dE1zV/314-migrate-deployment-to-puppet-ansible-chef