1.19 <IP_ADDRESS> Night Vision Flickering

Hello on every instance of the mod i run, the night vision power flickers on and off constantly.

it also seems like no datapacks in general are working

also probably part of these issues is i frequently cant load into a world and get the error message resource root power missing or something along those lines

hey ive tried out the 1.19.2 version and it doesnt seem to have the problems that im experiencing so i was wondering if youd just try backporting <IP_ADDRESS> to 1.19 i think thatd work

Short answer: No.
Long answer: The changes mojang did for 1.19.1 & 1.19.2 are minimal (I had to change 1 method call over the entire codebase), most mods can and have updated, if you don't want chat reporting, have it be disabled on the server, or use mods/plugins that do the job. The changes forge did however, are required since they fixed Jar-in-Jar (which origins relies on since <IP_ADDRESS>) in 1.19.1, as such Origins cannot be backported prior to 1.19.1, and since that version had a flaw critical enough to be hotfixed, there is no good reason not to use 1.19.2.


Fix WorkingDirectory path in RHEL 7 install guides

Addresses https://github.com/mattermost/platform/issues/4133
The RHEL 7 install guides don't appear to use the correct data folder, and hence Mattermost cannot read uploaded profile pictures or attachments after installation.
This PR should resolve the issue.

Would anyone in the community like to try and confirm if the issue reported in https://github.com/mattermost/platform/issues/4133 is fixed after the change here?

I submitted the original issue. This fix looks good to me. Thanks!

Thanks @InterHmai for the confirmation!

By the way, @InterHmai, if you're not a member yet, please consider joining our Mattermost developers channel. It's a good place to ask questions and/or share feedback with other open-souce developers and the Mattermost core team.

Thanks! This change is actually included in another PR #600. In order to avoid conflicts I'm going to close this one.


Incorrect JSONRPCRequest.params format

Some params received in JSONRPCRequest.params are not in the correct format:
Expected
For example on method eth_signTransaction (https://docs.walletconnect.com/2.0/advanced/rpc-reference/ethereum-rpc#eth_signtransaction), params are sent by the dApp in the following format:
[
  {
    "from": "0x0f364Bac605ba88200ac0c740d5484E6634B59d6",
    "to": "0x0f364Bac605ba88200ac0c740d5484E6634B59d6",
    "data": "0x",
    "nonce": "0x0",
    "gasPrice": "0x4a817c800",
    "gasLimit": "0x5208",
    "value": "0x0"
  }
]

Note that value, nonce, gas and gasPrice are strings.
Actual behavior
But JSONRPCRequest.params is in the following format:
[
  {
    "from": "0x0f364Bac605ba88200ac0c740d5484E6634B59d6",
    "to": "0x0f364Bac605ba88200ac0c740d5484E6634B59d6",
    "data": "0x",
    "nonce": 0,
    "gasPrice":<PHONE_NUMBER>0,
    "gasLimit": 21000,
    "value": 0
  }
]

nonce, gasPrice, gasLimit and value are integers instead of strings, see the documentation here: https://docs.walletconnect.com/2.0/advanced/rpc-reference/ethereum-rpc#eth_signtransaction.
I have not tested it but I suspect this will also cause an issue if value exceed a Kotlin Long (2^64) since it can go up to 2^256.
SDK Version

Client: Kotlin
Version 2.0.0-rc.0

@Azkae RC.1 was released today with a fix for the serializing


Nashorn JavaScript engine used for adaptive authentication causes memory leak with JDK 11

Describe the issue:
Nashorn JavaScript engine used for adaptive authentication has been removed/deprecated in newer JDK versions. Also, there are bugs in JDK 11 which cause memory leaks when using the Nashorn JavaScript engine. These JDK bugs are not fixed[1]
How to reproduce OOM issue due to JDK bug[1]:

Configure a Service provider with the following adaptive authentication script.

var onLoginRequest = function onLoginRequest(context) {
  executeStep(1,
      {
        onSuccess: function (context) {
          var fedUser = context.currentKnownSubject;
          var currentIdp = context.currentStep.idp;
          var endUser = fedUser;
          fedUser = null;
          currentIdp = null;
          endUser = null;
        }});
};

Login/SSO/logout to the service provider repeatedly with high load. The server would go OOM.

Expected behavior:
If the JDK bug[1] is not fixed and JDK does not support it, look for alternatives for the Nashorn JavaScript engine.
Environment information :

Product Version: [IS-5.11.0]

[1]https://bugs.openjdk.java.net/browse/JDK-8229011

duplicate https://github.com/wso2/product-is/issues/6404

I think this has three concerns mixed.

Nashorn is not supported on JDK : Here Nashorn  has been moved out from JDK to its own library and is maintained under LGPL. Hence there is no question on this regard.
Bug 8229011 , I do not see that is a memory leak. It creates lots of engines inside the for  loop. That supposed to consume memory indefinitely and should cause OOM. Try the same with any other object which has a lifecycle, you will notice this. e.g. try reading a web page with new HTTP client inside an infinite loop.
The code block stated here also related to above (2). Engine is created per each request (for context isolation) if you execute very high number of requests. I do not see this is memory leak, as long as we allocate enough memory for the purpose and it does not re-claim after the high number of requests are reduced.

An alternative script execution engine in introduced with https://github.com/wso2/carbon-identity-framework/pull/5279.


Modify Runtime Component project template to take advantage of converged VCLibs.UWPDesktop framework package

The runtime component project template creates a Universal Windows dll, with linkage to UWP CRT binaries (vcruntime140_app.dll, etc).  To host such a dll in a Desktop app requires a reference to the VCRT Forwarders nuget, which provides adapters from vcruntime140_app.dll to vcruntime140.dll, etc.  Visual Studio will soon ship with a converged VCLibs.UWPDesktop framework package, which provides the complementary behavior.  A dll with desktop CRT linkage (vcruntime140.dll) can be hosted by a UWP app, with VCLibs.UWPDesktop providing forwarders in the opposite direction.  New runtime component projects should begin using this technique instead.

I've been running into this problem, whats the right setup to author and consume a winrt library in a desktop application?

Changing the runtime component template to link against the desktop CRT imports is not sufficient - consuming projects also need to reference the latest VCLibs.UWPDesktop.  So the C++/WinRT app template should also be updated.

Feel free to reopen if there's interest in exploring this further.


[go_router v9.1.0] How can I navigate to shell Route from a root Route.

Is there an existing issue for this?

[x] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
I am trying to access a shell Route (Home Screen with bottom nav bar) from a Root Route ( Splash Screen ). If I navigate to home Screen the Nav Bar doesn't shows as the nav bar is the part of Shell Routes page Builder.
final GoRouter router = GoRouter(
  navigatorKey: _rootNavigatorKey,
  initialLocation: '/${Routes().splash}',
  debugLogDiagnostics: true,
  routes: <RouteBase>[
    GoRoute(                                              // Navigating from Here
      name: Routes().splash,
      path: '/${Routes().splash}',
      builder: (BuildContext context, GoRouterState state) {
        return const SplashScreen();
      },
    ),

 ShellRoute(

      parentNavigatorKey: _shellNavigatorKey,
      builder: (BuildContext context, GoRouterState state, Widget child) {
        return NavigationScreen(child: child);
      },

  routes: [
        GoRoute(                                                 // Navigating to this page and no Nav Bar appears
          name: Routes().home,
          path: '/${Routes().home}',
          builder: (BuildContext context, GoRouterState state) {
            return const HomeScreen();
          },
        ),

My Home page is a shell route.
Do I need to add a Key or something to make it work ? I can't figure this out if anyone has any idea how to do it please answer.
Thank You
Actual results
   if (context.mounted) {
      context.pushNamed(Routes().home);
    }

This takes me to home screen with no Nav Bar
Logs

Logs

<!-- Paste your logs here -->

Flutter Doctor output
[√] Flutter (Channel stable, 3.10.6, on Microsoft Windows [Version 10.0.19045.3208], locale en-US)
• Flutter version 3.10.6 on channel stable at D:\Development\Flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision f468f3366c (9 days ago), 2023-07-12 15:19:05 -0700
• Engine revision cdbeda788a
• Dart version 3.0.6
• DevTools version 2.23.1
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
• Android SDK at D:\Android
• Platform android-34, build-tools 34.0.0
• ANDROID_HOME = D:\Android
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.6.4)
• Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
• Visual Studio Community 2022 version 17.6.33815.320
• Windows 10 SDK version 10.0.22000.0
[√] Android Studio (version 2021.3)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
[√] VS Code, 64-bit edition (version 1.80.1)
• VS Code at C:\Program Files\Microsoft VS Code
• Flutter extension version 3.68.0
[√] Connected device (3 available)
• Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19045.3208]
• Chrome (web)      • chrome  • web-javascript • Google Chrome 114.0.5735.199
• Edge (web)        • edge    • web-javascript • Microsoft Edge 114.0.1823.67
[√] Network resources
• All expected network resources are available.
• No issues found!

Hi @hyderali0889. Thanks for filing this. It seems you are asking for assistance on personal code. As for the case you described above, if I got you correctly, you can try this https://stackoverflow.com/a/76077870 if it works in your case. also, please see https://flutter.dev/community for resources and ask questions like this,
you may also get some help if you post it on Stack Overflow
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


The file size increases abnormally when saving

The original file size is 8KB, and it contains nothing but only one empty row (row# 1048576), the file size will become 2.8MB when saving.
origin.xlsx
Unzipped both files and you can find that the file size of sheet1.xml has increased from 1KB to 23MB.
package main

import (
	"log"
	"os"

	"github.com/xuri/excelize/v2"
)

func main() {
	fi, err := os.Stat("origin.xlsx")
	if err != nil {
		log.Fatal(err)
	}
	log.Printf("Origin file size: %v", fi.Size())
	file, _ := excelize.OpenFile("origin.xlsx")
	file.SetCellValue("Test", "A1", "test")
	file.SaveAs("copy.xlsx")
	fi, _ = os.Stat("copy.xlsx")
	log.Printf("Copy file size: %v", fi.Size())
}

The output is
Origin file size: 8382
Copy file size: 2759163

Thanks for your issue, I have fixed it. Please upgrade to the master branch code, and this patch will be released in the next version.

Awesome! Thanks for your efforts!


Picocli 3.9.1

Upgrade to picocli 3.9.1 and spring boot 2.0.7.RELEASE
Allow single MainCommand to be loaded by Spring. Otherwise, throw the exception
Fix test case of Nested commands

Hi, Do you have time to publish new release version? If you need any help, I'd like to give you a hand.

Sorry guys I was totaly busy + some personal stuff drained my time. I'll plan that MR that next week

@colinkuo Thank you I'll merge it on branch 1.0.0-beta-1 that I'll merge to master as soon I polished other things


Training data to support for different language

Hello! I'm trying to add support for different language here. I have training data with about 100 000 sentences and can increase it to 1M or so. How many sentences I need to start training and how I need to update ./run.sh file in my case (input file name updated already)? (I've tried to use total and half number of lines already and got these errors:
...
Step 1/3
Step 2/3
[nltk_data] Downloading package punkt to /Users/350d/nltk_data...
[nltk_data]   Package punkt is already up-to-date!
Skipped 11383 lines
Step 3/3
head: illegal line count -- -50000
head: illegal line count -- -25000
Cleaning up...
...

Update: ok, OSX head and tail don't accept negative values, fixed with update to coreutils.
Will let you known about my progress here...

Is there an upper limit for the training data, or the more the better?


Looking for a new maintainer

I now only rarely use Atom and am looking for a new maintainer for this project. Please chime in.

@jazzpi do you feel like this could be of interest to you?

I still use Atom regularly and I think I know how this package works by
now, but I don't really work on ex-mode too often anymore. I guess I
could at least merge PRs and test them for a bit though since I use the
package pretty much daily, so I would take it up if no one else wants too.
On 25.11.2015 14:39, Loic Nageleisen wrote:

@jazzpi https://github.com/jazzpi do you feel like this could be of
interest to you?
—
Reply to this email directly or view it on GitHub
https://github.com/lloeki/ex-mode/issues/116#issuecomment-159610625.

What do you use instead? :D

@jazzpi I added you as a collaborator :-)

Alright, thanks :smile:

Thanks @jazzpi, now that I know about the need for maintenance I'll peer in sometimes. I mostly wondered what the upgrades were after 0.8.0 came out. :smile:

It would be cool if Atom would pick up this package formally, since vim-mode is directly supported.

DUDE!! I'll pick it up!  I came here to file a bug for ex-mode, and was shocked to find that you had written it.  I keep swapping back and forth between vim and atom.

Hah, cool :-) Super busy right now but if @jazzpi doesn't oppose, I'll add you as a collab for now.

No objections, more people working on this can't hurt :smile:

Welcome, @CHIEFnChowCombo.

Hai , i am entirely new to atom. In my VIM world I do use :column , :sort , :set nowrap, etc commands very often . Can i request you guys to add them to the ' : ' please.
I am new to .js and .coffee too , but to make atom the best ever editor (One day when I can just copy my vimrc to atom) , this seems to be a good enough motivation to pick up a new language :)

@vivek-nandyala Hello. You can open new issues for your requests and PR are welcomed


Add shortcut tour commands

What about double elimination? Triple elimination? 822262515-Tuple elimination?

Also, what about a  player cap?

Why not manually make the tour with /tour then?

True


[benchmark] Adjust RuBen's StructOpt

Summary:
One of many preparing steps for a new binary that linear search the max throughput.

Rename Opt to RuBenOpt. The future new binary will reuse all the arguments in RuBenOpt.
Remove free lunch as it is never used.

Test Plan:
Existing tests related to Benchmark package.

I'm strongly in favor of non-capital "B" in the name. It's already a name on its own, no need to break it down to syllables.

Haha, sure. Name adjusted.


Add readable_xticklabels

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

Comparison is base (f2055dc) 100.00% compared to head (7a12e2d) 100.00%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #189   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           67        67
  Lines         1542      1553   +11
  Branches       151       153    +2
=========================================
+ Hits          1542      1553   +11

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


Can't Add a Wiki Page

I'm not seeing the button to create a new page. This is the task table I was going to post.

Project name
Language (py or js)
analyze? (X for yes) <-- either
or --> develop? (X for yes)

Fractionauts
Python

X

Lemonade-Stand
Python

X

Fractionators
JavaScript
X

FoodChain
JavaScript
X

On Tue, Apr 14, 2020 at 01:12:39PM -0700, Ian Clare wrote:

I'm not seeing the button to create a new page.

https://github.com/ritjoe/sugarizer-lite/issues/4#issuecomment-611853624
https://github.com/ritjoe/sugarizer-lite/issues/1#issuecomment-613106450
rit-foss.2020-04-06-13.00.log.txt:13:10:29  I've issued github collaborator invitations to everyone
rit-foss.2020-04-10-13.02.log.txt:13:26:29  jrtechs: I am trying to give everyone access to ritjoe/sugarizer-lite/wiki via invitations to collaborate

This is the task table I was going to post.

I took the liberty of pulling a task table from your journal and tried to incorporate it as best I could in the overall assignments table, here:
https://github.com/ritjoe/sugarizer-lite/wiki/final-projects-20200414

Strange, I never received a collaborator invite for multiple repos this week. The notifications never came through on GitHub and I never received emails about them.

Closing since this seems to be an issue on my end / github


Save without formatting

What do you think of a key combo to bypass formatting and save? Sometimes I have to temporarily disable the Format on Save setting to make patches for OSS/projects that don't use prettier.

@tacomanator I'm not sure how I would go about coding that one. Maybe it would be better if we had a whitelist or blacklist of globs. I did a PR for a related project that does blacklist globs:
https://github.com/kentcdodds/prettier-eslint-atom/pull/11/files
Then you would just blacklist the projects you didn't want it for. What do you think of that solution?

@robwise that would definitely help. I haven't thought of a better way either.

I just had this exact issue recently. It would be nice to ignore souce.min.js type files.

I'm currently work on a PR that will integrate prettier-eslint-atom where I wrote this feature, so stay tuned!

@Anthonyzou @tacomanator We now have the ability to both whitelist and blacklist certain globs so I'm closing this for now. Maybe in the future we should add a keymap to toggle format on save on and off.


Added functionality to get raw body

// Added
To get Bi/Report raw data, Plentymarkets provides a download-like request via "/rest/bi/raw-data/file".
To work with the content, json_decode must not be applied to the response and must be disabled accordingly.
This is enabled with enableJsonDecode / disableJsonDecode.
// Nice to Have
A single run solution would also be possible, but is not needed especially for the bi-report raw data.

Thanks!

I've changed your naming a little bit to be consistent with the rest & updated the README


Update Help for both Master and Node

Need to have better usage and help commands

Merged better help command


Sole Survivor issues

Apologies if I shouldn't ask here, since I don't know if cnc-ddraw is meant for Sole Survivor, but seems like it's the only solution to run it.

Only Windowed (but not maximized, aka only small resolution window) and Fullscreen do NOT have an issue where right side of the screen won't scroll to the right. It's like there are a few vertical lines of pixels where the cursor "disappears", so it doesn't register as scrolling to the right. This happens with or without aspect ratio/scaling/windowboxing, and with or without locking the cursor.

Mouse sensitivity fix isn't applied in Fullscreen - probably because it's literary 640x480, but the mouse sensitivity is what I use for 1080p. However, what does this fix do then?

D3D9 always applies bilinear filtering/scaling in Fullscreen Upscaled or Borderless, even with integer scale - why? GDI is sharp, and OGL always applies the selected shader (also, any reason for lack of no shader for OGL option?).

Only Windowed (but not maximized, aka only small resolution window) and Fullscreen do NOT have an issue where right side of the screen won't scroll to the right. It's like there are a few vertical lines of pixels where the cursor "disappears", so it doesn't register as scrolling to the right. This happens with or without aspect ratio/scaling/windowboxing, and with or without locking the cursor.

That bug was fixed already, here is the latest build:
cnc-ddraw.zip

Mouse sensitivity fix isn't applied in Fullscreen - probably because it's literary 640x480, but the mouse sensitivity is what I use for 1080p. However, what does this fix do then?

sensitivity will always match the size of the window, in normal fullscreen the size of the window is not changed and the sensitivity equals the original resolution of the game. You must use Fullscreen Upscaled or Borderless or a resized window to enable the fix.

D3D9 always applies bilinear filtering/scaling in Fullscreen Upscaled or Borderless, even with integer scale - why? GDI is sharp, and OGL always applies the selected shader (also, any reason for lack of no shader for OGL option?).

There is currently no option in the config program, but you can disable the bilinear filtering for d3d9 via ddraw.ini "d3d9linear=false".
For OpenGL set your shader to "nearest-neighbor.glsl" to disable shaders, this is actually not a shader, just a dummy file


Lock related problem in on travis-ci but not on local machine

There is a KeyError at xarray.backends.file_manager.CachingFileManager for the acquire method on these lines:
        with self._lock:
            try:
                file = self._cache[self._key]

which does not happen when I test on my macos.
Let me know if you require further testing on my part.
Full log:
=================================== FAILURES ===================================

________________________________ test_qtl_xarr _________________________________

self = <xarray.backends.file_manager.CachingFileManager object at 0x7f59e2cab978>

    def acquire(self):

        """Acquiring a file object from the manager.

        A new file is only opened if it has expired from the

        least-recently-used cache.

        This method uses a reentrant lock, which ensures that it is

        thread-safe. You can safely acquire a file in multiple threads at the

        same time, as long as the underlying file object is thread-safe.

E   OSError: [Errno -101] NetCDF: HDF error: b'/tmp/tmprj8lwlat/xarr.hdf5'

netCDF4/_netCDF4.pyx:1752: OSError

===================== 1 failed, 50 passed in 65.82 seconds =====================

The command "bash <(curl -fsSL https://raw.githubusercontent.com/horta/ci/master/travis.sh)" exited with 1.

Done. Your build exited with 1.

Thanks for the report!
This might be an xarray issue, or it might also be a netCDF4 issue -- I wonder if you might have different versions of libnetcdf installed locally and on Traivs-CI?

I am having the exact same problem. Have you found a solution/workaround?

Sorry guys. I've found the problem and solution.
The problem is that filesystem not supporting lock mechanism. The solution is to export the following variable: export HDF5_USE_FILE_LOCKING=FALSE.

OK, I think I understand what's going on here and why this issue only appears with xarray v0.11.
The problem is related to how you are opening files with xarray but not closing them:
https://github.com/limix/limix/blob/8bc0861035cc60b3ce7bcbf7f147bcc828580828/limix/qtl/test/test_qtl_xarr.py#L20
With v0.11, xarray introduced a least-recently-used cache for netCDF files. This means xarray's LRUCache is holding on to a reference to your files, so they never get garbage collected, and automatically closed by netCDF4-Python. Hence the HDF5 locks never get released.
Although it is indeed a good practice to always explicitly close files, I think we should explore using weak references to keep track of files in our cache so we don't hold on to them for holding than necessary.

If either of you have time, it would be great if you could test out https://github.com/pydata/xarray/pull/2595 to see if that resolves your issue without requiring the environment variable.

I have done a bit more testing on this, and I believe the issue may not necessarily be with xarray but with rasterio (disclaimer: I haven't tested your pull request yet).
I can reproduce the following on Ubuntu and on Travis CI. On Mac OS none of these errors occur.
Minimum example:
import xarray as xr
import numpy as np
import rasterio  # The rasterio import makes the last line of this code fail.
ds = xr.Dataset()
ds['data'] = (('y', 'x'), np.ones((10, 10)))
ds.to_netcdf('test.nc', engine='netcdf4')

I was able to fix the error by prepending a
import netCDF4

at the very top of the script.
A very similar thing happens with engine='h5netcdf'. The same script works without rasterio, fails with rasterio, and can be fixed by inserting import h5netcdf at the top of the script.

See also https://github.com/pydata/xarray/issues/2535 for rasterio/netCDF4 issues.


[WIP] making inpaint compatible with grayscale

Results are weird. I think after changing to grayscale there is problem with applying patches.

@jiviteshjain you can commit to this PR
The tests needs to be changed for grayscale images

Results are weird. I think after changing to grayscale there is problem with applying patches.

The tests needs to be changed for grayscale images

Couldn't figure out where exactly this is happening, but using images with Gray{Float64} colourant works. Could the limited precision of fixed decimal types be the cause? Just converting the images to Gray{Float64} could be a solution, not sure if it's the best one.

Results are weird.

Okay It is because of storage precision. Float precision works fine gives exactly previous result.

What is the status of this PR @mgautam98 ? Any updates?


Use dotnet/runtime-deps as the base image for SelfContained scenarios

Self-contained applications only need the runtime-deps (regardless of aspnet-ed-ness) so we can use that base image as the default. This image has the same tags as the other base images, so it should be a safe change.

@BenVillalobos I like the message for telling the user the base image chosen - can we say something like Building <imageName>:<imageTag> based on <sourceImageFullRepoNameAndTag>? I suppose that could happen early in the CreateNewImage target.

I'll come back and add regression tests to this (and to all the permutations) when the PR with the test evaluation gets merged in.

@baronfel it's sort of built-in today. After ParseContainerConfig we log a message displaying everything it parsed. Then there's the CreateNewImage task inputs.
I suppose the only use for logging the container info early on is to let them know what the default was, but that can be inferred from what ParseContainerProperties logged.
Hmm. imo that makes the purpose of this message for context. If we tell the user what the default was and why, maybe even what they can do to change that default, that might be most helpful. Thoughts? Unless that isn't too important considering the other logging that already exists.


Remove new_with_config constructors where possible

I think it might just be cleaner to have new always take a config, users can always just pass Default::default().
Related to #2010 and #2321

IMHO we should instead do new().with_config() but the compiler will first apply a default config so it's slightly less efficient...

IMHO we should instead do new().with_config() but the compiler will first apply a default config so it's slightly less efficient... But I think I prefer the code bloat over the "required optional" parameter.

I agree, but with a small tweak. It should be apply_config and it can be applied anytime. (Related: #2416 )

We should probably put this in the API-GUIDELINES, but in general I think with_X makes sense on construction, and apply_ or other verb-based methods make sense post creation.

re "the compiler will apply a default config" we can also choose to roll with hardware default initially.

We should also think about what to do with SPI: currently we don't have an SpiDevice equivalent, which means we can't automatically reconfigure the bus for the devices' individual needs. A configuring version of ExclusiveDevice would align well with config structs and apply_config. So I guess there are pros and cons for each approach, the question is whether keeping a small API surface here is a good enough motivator to pick between them?

We should also think about split peripherals: what does apply_config(uart::config::Config) mean for UartTx, what does it mean for UartRx? What does it mean to change the baud rate for half of a peripheral, for example?

We should also think about split peripherals: what does apply_config(uart::config::Config) mean for UartTx, what does it mean for UartRx? What does it mean to change the baud rate for half of a peripheral, for example?

This is a good question, in embassy they treat it like a full apply on the whole Uart. I think I would also go down this route. Having to do Uart::unmerge(rx, tx).apply_config() or something like that would be quite terrible, for a number of reasons.

I agree, I'll close this now.


feat: general implement

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: leo220yuyaodog:x: semantic-release-botYou have signed the CLA already but the status is still pending? Let us recheck it.


file deficiency

Can't find dict_utils.py and map_vis_lanelet2.py in utils

Thank you for pointing this out, I have commented out those lines. Please let me know if you are still having issues!


Jenkins dashboard issue

Hi Team,
One of my jenkins job have 369 builds. Also mongo db shows build history(screenshot attached)
But in my hygieia dashboard shows only 2 builds 368 & 369 information(screenshot attached). why?
Could you please help me to fix this issue.
-Vijay

By looking at your build start times - I think they are more than 14 days old and the dashboard shows build only for past 14 days.

Yeah. I agree you can close this tkt.


fix PHP E_NOTICE errors

fix PHP E_NOTICE errors, change constructor to PHP5 style, format code to adhere to PSR

https://github.com/google/ReCAPTCHA/pull/11 https://github.com/google/ReCAPTCHA/pull/13 https://github.com/google/ReCAPTCHA/pull/16

@yingliu-google @google can someone please merge this PR? There are at least 10 other PRs that are duplicates of this one...

This is getting ridiculous :disappointed:

If @google-admin would just transfer this repo to me...

Friendly ping for somebody to merge this :)

/cc @haidongs

As this got too annoying, we now offer a maintained fork where we accept pull requests (even for Ruby like in #4 and #6 ), offer better docs, Composer support (like requested in #2 ) and tagged releases with meaningful commit messages.
Currently the fork fixes all the errors in #2 #3 #5 #9 #10 #11 #13 #14 #15 #16 #20 #21 #22 #23
The general goal will be to provide a PSR-4 compatible rewrite with version 1.0 (including the suggestions from #7 #17 #18 ).

Hi all, thank you for all the feedback here! I know it's been a while, but there is now a new v1.1 published which hopefully resolves a lot of these issues for you. Also, and perhaps more interesting for a bunch of you on this thread, there is a proper contribution process in  place now and I'll keep an eye on new pull requests.


Searchability is problematic

File: engine/reference/builder.md, CC @mstanleyjones
The site's resident search is limited. Recently tried searching for the term, entrypoint, and was disappointed at the weirdness. Perhaps your site needs a search-able glossary to hold all its concepts in one place.

The very first search result I get from https://docs.docker.com/search/?q=entrypoint is https://www.google.com/url?q=https://docs.docker.com/engine/reference/builder/, which seems to be the right one. The second and third results, which are both on forums.docker.com, also seem to be high quality results. If you want to narrow it down to only docs.docker.com content, you can click the "Docs" tab in the search results.  The top four search results are IMO very high quality.
If you think we should have a definition for Entrypoint in the glossary, I can definitely see an argument for that.

The search results brings up the article and not the definition or synopsis.

From: Misty Stanley-Jones<EMAIL_ADDRESS>Sent: Sunday, April 9, 2017 11:37:14 AM
To: docker/docker.github.io
Cc: George; Author
Subject: Re: [docker/docker.github.io] Searchability is problematic (#2691)
The very first search result I get from https://docs.docker.com/search/?q=entrypoint is https://www.google.com/url?q=https://docs.docker.com/engine/reference/builder/, which seems to be the right one. The second and third results, which are both on forums.docker.com, also seem to be high quality results. If you want to narrow it down to only docs.docker.com content, you can click the "Docs" tab in the search results. The top four search results are IMO very high quality.
If you think we should have a definition for Entrypoint in the glossary, I can definitely see an argument for that.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/docker/docker.github.io/issues/2691#issuecomment-292793201, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAJpM91Y7Q_CunVwIMfmnPHeVbjntgsNks5ruPsqgaJpZM4M35rQ.

@gdennie You'll be happy to know we're working on it: https://github.com/docker/docker.github.io/pull/1387/files#diff-8fd00abf6369bddcce9b428356326a1aR13


New interface icons

@sephcoster uploaded icon assets; let me know if these work.
Unfortunately they're not currently included in the minicon font, but hopefully can be added if they go through the full design approval process and asset generation.

@sephcoster new keys for the two ACS layers are up!

:+1:  Looks good!


[Accessibility][Usable - New Dialog] : Once the error appears it can't remove again even after the removal of the filed which have error.

User Experience
The screen reader dependent user will get confused as well as it effects low vision user as the error message didn't disappear even after correcting the field where the error appears.
Test Environment
Browser: Microsoft Edge Canary Version: 82.0.425.0 (Official build)
OS build: 2004 (19564.1005)
Screen Reader: Narrator
URL: Bot Framework Composer
Prerequisite Here
Repro Steps:

Open the above mentioned URL in EDGE.
Select 'Design Flow' from left navigation.
Now, navigate to 'New Dialog' button available in the filter dialog pane and activate it..
Type the name of the dialog which is already created ex- Actions.
Observer the issue after correcting the name of the dialog.

Actual:
The error message i.e. "Duplication of the dialog" is not gets disappeared even after correcting the name in the dialog.
Expected:
As the user delete the previously entered dialog name the error message should gets disappeared.
MAG Guideline: MAG
Attachment:
Usable_error message can't gets corrected even after correcting the error field.zip

This issue will be solved once the form-validation thing goes through; it's an artifact of the same problem (#2074 ).

As checked and verified on below environment, the issue is fixed now. So, we are closing the issue.
Test Environment
Browser: Microsoft Edge Dev {Version 83.0.467.0 (Official build) dev (64-bit)}
OS build: 2004 (19588.1000)
Screen Reader: Narrator
URL: Bot Framework Composer
Prerequisite Here


(Suggestion) Add Skinrestorer support

Skin Restorer :

Spigot Store
Github Link
Api Mention
Issue I commented on for future improvements:

Summary
This feature would be great to show the players head avatars for on the fly skin swapping , Offline users, and GeyserMC players. This is supported in Dynmap and would be a great feature to add to Pl3xmap (IMO).
We had a brief discussion about it in the discord.
Original Message

It seems as they do for SkinRestorer specifically.
Camotoy also threw in some $.02 about how it could possibly done.
1st comment
2nd comment
These seem to be the configuration options that would help support this in dynmap's configuration.txt file:
# custom-colors-support: if true, Custom Colors in texture packs is enabled (default)
custom-colors-support: true

# Control loading of player faces (if set to false, skins are never fetched)
fetchskins: true

# Control updating of player faces, once loaded (if faces are being managed by other apps or manually)
refreshskins: true

# Customize URL used for fetching player skins (%player% is macro for name)
skin-url: "http://skins.minecraft.net/MinecraftSkins/%player%.png"

# Enable skins via SkinsRestorer plugin instead of internal legacy implementation (disabled by default)
skinsrestorer-integration: true

Dynmap's original commit for Skinrestorer (i think)
https://github.com/webbukkit/dynmap/commit/578c66e2fe6de9e83f9e9ad1b8508a1def97aae1
P.S. Thanks for taking the time to read BIlly , appreciate the work that you're doing. <3

I already knew how to do this without using their API. I was just reluctant on doing so because it opens up the possibility for cracked/offline servers to use. I do not condone the use of piracy in any way shape or form, and try to actively block it where I can.
With that said, I went ahead and made a small addon because Geyser is indeed a valid use-case.

Hey Billy, installed and its working TYSM.
1 small issue and 1 requests .

looks like the image is set to save at 8x8 instead of 16x16 as I think you intended.  So it comes out really smol.

Is there a way to add a boolean option in the config  to add the player avatar with the helm layer as well?

I believe Dynmap does it here in their implementation to grab the layers of the skin
https://github.com/webbukkit/dynmap/blob/578c66e2fe6de9e83f9e9ad1b8508a1def97aae1/DynmapCore/src/main/java/org/dynmap/PlayerFaces.java#L101


Foundry tutorial complete missing steps

Replacing https://github.com/ava-labs/avalanche-docs/pull/1217. This PR has been QAed and is good to merge.


more feedback for the examples

I am trying to use a QUIC library for measuring the time the connection establishment is taking.
I have tried to use Quinn, but it seems to not work for me. The lack of feedback from the command is frustrating.
➜ quinn git:(master)>cargo run --example server ./
    Finished dev [unoptimized + debuginfo] target(s) in 0.17s
     Running `target/debug/examples/server ./`

➜ quinn git:(master)>cargo run --example client https://localhost:4433/Cargo.toml
    Finished dev [unoptimized + debuginfo] target(s) in 0.19s
     Running `target/debug/examples/client 'https://localhost:4433/Cargo.toml'`
ERROR: failed to connect: timed out
➜ quinn git:(master)>

I wished the server would tell me where it's listening to, e.g. what port.
I tried to force it myself:
➜ quinn git:(master)>cargo run --example server --listen 4433
error: Found argument '--listen' which wasn't expected, or isn't valid in this context

USAGE:
    cargo run --example <NAME>...

For more information try --help
➜ quinn git:(master)>

I forgot the path. But... the error message is not very indicative.
➜ quinn git:(master)>cargo run --example server ./ --listen 4433
    Finished dev [unoptimized + debuginfo] target(s) in 0.19s
     Running `target/debug/examples/server ./ --listen 4433`
error: Invalid value for '--listen <listen>': invalid IP address syntax
➜ quinn git:(master)>

Hrmpf. What is the syntax? 0:4433 does not work either.
I continued to try with another, supposedly good server:
➜ quinn git:(master)>cargo run --example client  quic.westus.cloudapp.azure.com
    Finished dev [unoptimized + debuginfo] target(s) in 0.16s
     Running `target/debug/examples/client quic.westus.cloudapp.azure.com`
error: Invalid value for '<url>': relative URL without a base
➜ quinn git:(master)>

what does it even mean to not have a base?
I'd appreciate examples of calls that are supposed to be working.

Hey, sorry you're having a bad first experience! I'll make sure we add more logging to help debug.

Also, might be an IPv4 vs IPv6 issue? The client argument has to be a full URL including scheme.

The server listens on IPv6 by default (see cargo run --example server -- --help) so if your system isn't dual-stack that might be the issue; we should probably tweak the example invocations not to be sensitive to that.

error: Found argument '--listen' which wasn't expected, or isn't valid in this context

Cargo is trying to parse the --listen argument itself. To prevent this, put a -- before argument to the example, like this: cargo run --example server -- ./ --listen <IP_ADDRESS>:4433. Note that --listen takes ip:port syntax.

what does it even mean to not have a base?

Try e.g. http://quic.westus.cloudapp.azure.com/index.html, similar to the example.

error: Found argument '--listen' which wasn't expected, or isn't valid in this context

Cargo is trying to parse the --listen argument itself

hm. that's bad. I guess there is nothing that can be done on Quinn's side :-/
But it'd be helpful if it was possible.

Note that --listen takes ip:port syntax.

except it seems to be very picky. But I guess that printing an example of an invocation that is supposed to work in the error message guides towards a solution and would have helped me to manage my frustration.

what does it even mean to not have a base?

Try e.g. http://quic.westus.cloudapp.azure.com/index.html, similar to the example.

may I suggest to add such an example to the error message?
From URIs I know this as scheme rather than base.

From URIs I know this as scheme rather than base.

I suspect the URL library is parsing your input as a path, rather than a domain name. I agree that the errors here are not ideal; structopt makes it a bit awkward to override them, but it's probably worth the effort.

https://github.com/djc/quinn/pull/279 will provide feedback on the server's listening address.

@muelli you might want to check out a smaller example, if you're only interested in connection speed: https://github.com/djc/quinn/pull/246/files#diff-b12592f14356bd3fb49dda7e8ae22e6e ;)


Automatically update the cache path

The gradle runner step seems to automatically update the cache path but this step does not. I think either this step should be deprecated in the favor the runner or update the cache path.

Hey @ansman !
Thank you for reporting this issue, cache collection will be added soon!
🚀

Hey @ansman !
Sorry for the delay, the new version is released including the cache path collection!
https://github.com/bitrise-io/steps-gradle-unit-test/releases/tag/1.0.4
Please feel free to re-open this issue if you encounter any issue with the new version


[Twitter] 404 error for adult videos

Checklist

[ ] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-J', u'--verbose', u'https://twitter.com/JaqGomesOficial/status/1489003590190571521']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 2.7.18 (CPython) - Linux-5.11.0-1027-raspi-aarch64-with-Ubuntu-21.04-hirsute
[debug] exe versions: ffmpeg 4.3.2-0, ffprobe 4.3.2-0
[debug] Proxy map: {}
ERROR: Unable to download JSON metadata: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "./youtube-dl/youtube_dl/extractor/common.py", line 634, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "./youtube-dl/youtube_dl/YoutubeDL.py", line 2288, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python2.7/urllib2.py", line 435, in open
    response = meth(req, response)
  File "/usr/lib/python2.7/urllib2.py", line 548, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python2.7/urllib2.py", line 473, in error
    return self._call_chain(*args)
  File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
    result = func(*args)
  File "/usr/lib/python2.7/urllib2.py", line 556, in http_error_default
    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

Description

Twitter prohibits viewer access to adult videos. Only logged in users have access to the video.
The solution is to add Twitter login support.
The temporary solution is to use browser cookies.

I have an alternate tool that works, no login needed:
PS C:\> twitter<PHONE_NUMBER>190571521
POST https://api.twitter.com/1.1/guest/activate.json
GET https://api.twitter.com/1.1/statuses/show/1489003590190571521.json?tweet_mode=extended
GET https://video.twimg.com/ext_tw_video/1489003473421152261/pu/vid/480x270/JoLJH5YhcyyXItmH.mp4

https://github.com/89z/mech

Twitter prohibits viewer access to adult videos. Only logged in users have access to the video.

Possibly, but the link works for me with Py2.7 and 3.9 and also with yt-dlp using 3.9

@dirkf I took a new test and realized that the error only occurs in some countries (Brazil)
Proxy Brazil (doesn't work)

Proxy by: https://freeproxylists.net/br.html

./youtube-dl --verbose -J --proxy https://<IP_ADDRESS>:5005 https://twitter.com/JaqGomesOficial/status/1489003590190571521
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'-J', u'--proxy', u'https://<IP_ADDRESS>:5005', u'https://twitter.com/JaqGomesOficial/status/1489003590190571521']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 2.7.18 (CPython) - Linux-5.11.0-1027-raspi-aarch64-with-Ubuntu-21.04-hirsute
[debug] exe versions: ffmpeg 4.3.2-0, ffprobe 4.3.2-0
[debug] Proxy map: {u'http': u'https://<IP_ADDRESS>:5005', u'https': u'https://<IP_ADDRESS>:5005'}
ERROR: Unable to download JSON metadata: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "./youtube-dl/youtube_dl/extractor/common.py", line 634, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "./youtube-dl/youtube_dl/YoutubeDL.py", line 2288, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python2.7/urllib2.py", line 435, in open
    response = meth(req, response)
  File "/usr/lib/python2.7/urllib2.py", line 548, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python2.7/urllib2.py", line 473, in error
    return self._call_chain(*args)
  File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
    result = func(*args)
  File "/usr/lib/python2.7/urllib2.py", line 556, in http_error_default
    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

Proxy United States (It works)
./youtube-dl --verbose -J --proxy https://<IP_ADDRESS>:3128 https://twitter.com/JaqGomesOficial/status/1489003590190571521
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'-J', u'--proxy', u'https://<IP_ADDRESS>:3128', u'https://twitter.com/JaqGomesOficial/status/1489003590190571521']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 2.7.18 (CPython) - Linux-5.11.0-1027-raspi-aarch64-with-Ubuntu-21.04-hirsute
[debug] exe versions: ffmpeg 4.3.2-0, ffprobe 4.3.2-0
[debug] Proxy map: {u'http': u'https://<IP_ADDRESS>:3128', u'https': u'https://<IP_ADDRESS>:3128'}
[debug] Default format spec: bestvideo+bestaudio/best
{"display_id": "1489003590190571521", "extractor": "twitter", "protocol": "https", "description": "Eu adoro, eu me amarro. RT \ud83e\udd74\ud83d\ude08 https://t.co/f2FbtjLEX1", "tags": [], "timestamp":<PHONE_NUMBER>, "format": "http-2176 - 1280x720", "tbr": 2176, "height": 720, "like_count": 6711, "uploader": "Jaqueline Gomes", "duration": 15.0, "format_id": "http-2176", "upload_date": "20220202", "id": "1489003590190571521", "requested_subtitles": null, "playlist": null, "thumbnails": [{"url": "https://pbs.twimg.com/ext_tw_video_thumb/1489003473421152261/pu/img/RX7hnj0W_7PGzBMr.jpg?name=thumb", "width": 150, "resolution": "150x150", "id": "thumb", "height": 150}, {"url": "https://pbs.twimg.com/ext_tw_video_thumb/1489003473421152261/pu/img/RX7hnj0W_7PGzBMr.jpg?name=small", "width": 680, "resolution": "680x383", "id": "small", "height": 383}, {"url": "https://pbs.twimg.com/ext_tw_video_thumb/1489003473421152261/pu/img/RX7hnj0W_7PGzBMr.jpg?name=medium", "width": 1200, "resolution": "1200x675", "id": "medium", "height": 675}, {"url": "https://pbs.twimg.com/ext_tw_video_thumb/1489003473421152261/pu/img/RX7hnj0W_7PGzBMr.jpg?name=large", "width": 1280, "resolution": "1280x720", "id": "large", "height": 720}, {"url": "https://pbs.twimg.com/ext_tw_video_thumb/1489003473421152261/pu/img/RX7hnj0W_7PGzBMr.jpg?name=orig", "width": 1280, "resolution": "1280x720", "id": "orig", "height": 720}], "title": "Jaqueline Gomes - Eu adoro, eu me amarro.  I get different results and still WFM in UK.
You have a work-around for the moment.
In due course we'll unify the extractor with the yt-dlp extractor; maybe that will address the problem without more hard work.


Add numpy_seed and torch_seed

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

Project coverage is 100.00%. Comparing base (0d70984) to head (d65a2ad).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #533   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           66        66
  Lines         1768      1788   +20
  Branches       206       208    +2
=========================================
+ Hits          1768      1788   +20

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


118 builder requests constructed from spawnRequestArgs

If half-functional RCL 6

It rose proportionately to stored energy. A static maxCreeps value was added to ensure absurd values are not reached


rocm5.2.0 torch run test_inference.py error undefined symbol: hipblasHgemm

(llama) root@:/data2/02_exllamav2/exllamav2# python test_inference.py -m /data2/models/exllamav2_models/turboderp_Llama2-7B-exl2-4.0bpw -p "Once upon a time,"
Traceback (most recent call last):
File "test_inference.py", line 2, in
from exllamav2 import(
File "/data2/02_exllamav2/exllamav2/exllamav2/init.py", line 3, in
from exllamav2.model import ExLlamaV2
File "/data2/02_exllamav2/exllamav2/exllamav2/model.py", line 17, in
from exllamav2.cache import ExLlamaV2CacheBase
File "/data2/02_exllamav2/exllamav2/exllamav2/cache.py", line 2, in
from exllamav2.ext import exllamav2_ext as ext_c
File "/data2/02_exllamav2/exllamav2/exllamav2/ext.py", line 14, in
import exllamav2_ext
ImportError: /root/miniconda3/envs/llama-fac/lib/python3.8/site-packages/exllamav2-0.0.7-py3.8-linux-x86_64.egg/exllamav2_ext.cpython-38-x86_64-linux-gnu.so: undefined symbol: hipblasHgemm

run "python setup.py install" no problem; but error in "test_inference.py"

This project currently only support ROCm 5.6 and above.


How to hide one or two of my all apis?

Hi, I would like to hide some of my REST apis. I don't want to make docs for them.
Is there any way to do this through some configuration?

Hi, pls tell more about your case usage.


Fixed Improper tags for Meshery Docker Extension #6396

@aditi5050 there is a merge conflict here...

This has been convered in another PR


Added url getter for mutation record

@n1zea144 @angelicaochoa
getUrlForRecord is now being used throughout.

Merged now, but if relocation of isHgvspNullClassifications() to another class is desired it can be done in a separate PR
@n1zea144


Probability Skewed

The probability of a certain wallpaper can be defined by the function P(x) = (R/N) * (1/N2), where R is the residing folder of the image, N is the number of possible folders and N2 is the number of images in R. As per the current algorithm, the probability of a given wallpaper is dependent on both R and N2. An algorithm needs to be designed that delegates probability independently of any variables besides the total number of images available.

Solved by first getting all the potential file names using the Directory.GetFiles(ROOT_DIRECTORY, "*", SearchOption.AllDirectories) function. Then the program generates a random number, using it to pick a file path from the potential list and setting it that way.


Milestone 09: Add Devise

In this PR, I @FaranoshAmini and my partner @samlorlah Implemented the following tasks:

[x] Installed and set up devise to:

[x] register a new user.
[x] Log in user with a combination of email and password.
[x]  Store hashed password in the database.
[x]  Ask for confirmation of email.
[x]  Allow user to reset password.

[x]  Modified views for the registration and login.

@Reem-lab Thanks for your kind review.


Improve Tasks

This is an issue to improve the task object. I've gathered the following use-cases:

[ ] Return correct error exit codes when running tasks

[x] serial playbook
[ ] parallel playbook

[ ] to omit certain tasks from auto-completion and/or being called directly (mainly tasks which are called by other tasks)
[ ] to repress certain task output if exit code is 0, otherwise displayed
[ ] a summary of task execution at the end
[ ] to be able to pass environment variables between tasks
[ ] to be able to access the previous exit code of the previous task
[ ] conditional task execution

[ ] tags/servers filtering https://github.com/alajmo/sake/issues/6
[ ] ensure command (check existence of file, software, etc.)

[ ] on-error/on-success task execution
[ ] a cleanup task that is always ran
[ ] to log task execution to a file
[ ] to abort if certain env variables are not present (required envs)
[ ] want a --step mode flag or config setting to prompt before executing a task

A separate pull-request will be created to implement the necessary changes.

Currently it is possible to have duplicate task without error.

Yes, I had plans to rectify that, however, it makes sense to keep it as it is since you might want to override tasks.
So it would be possible to have a task defined in your home config (~/.config/sake/config.yaml) and then override that task if you're working with a config that defines the same task.

I get that point. But then there should be a way to see which one is overwritten (maybe a flag to check the config).
The worst case scenario is when you have a sprawling config and somebody edits the config and nothing changes because it is overwritten somewhere else.

I get that point. But then there should be a way to see which one is overwritten (maybe a flag to check the config). The worst case scenario is when you have a sprawling config and somebody edits the config and nothing changes because it is overwritten somewhere else.

I've had some second thoughts here, initially, I thought the use case was valid (users sharing the same code base could override tasks), however, I don't see that use case happening a lot, and that it should take precedence over debugability or worst case, you think you're running a task, but it's some other task you defined in your user config.
Will add it to the roadmap.


[BACKEND] List all minified URLs from Database to /minify/all route

Create GET route at /minify/all to list all URLs, minified Alias as JSON from the database.

I would like to be assigned this issue on Feb 1st if possible.

I am interested

I want to work on this issue under JWOC

@luxunator Go ahead

@luxunator Any updates on the issue?


Secret/map

Closes #1774

Please re-open this against the new repo structure, apologies we didn't realise changing head would close the PR.


How to disable/enable zooming

How can I disable/enable

pinch zooming

Currently there is no such possibility.

Add this code in onCreate:
Constants.Pinch.MINIMUM_ZOOM = 1;
Constants.Pinch.MAXIMUM_ZOOM = 1;
For disable double tap:
pdfView.fromAsset(SAMPLE_FILE)
.......
.enableDoubletap(false)
.load();


Add support for custom logging behaviour.

I would like this PR to be merged because I can use it to replace the log what discordgo does with something like logrus.
Feel free to add comments for adjustments that might need to be made.

Please PR into develop.

Fixed.


New version: DashHtmlComponents v1.1.3

Registering package: DashHtmlComponents
Repository: https://github.com/plotly/dash-html-components
Created by: @rpkyle
Version: v1.1.3
Commit: 422c952ed2c86f6c3dbec04150da27688d464a57
Reviewed by: @rpkyle
Reference: https://github.com/plotly/dash-html-components/issues/186#issuecomment-818993062

We would like to request manual merging of this package (in spite of the gap in the version sequence), since we have already created releases for those releases which remain unpublished to the Registrator, and we routinely delete branches containing earlier versions within our repository.


Align goals for CCM for official upstream cloudstack kubernetes provider?

Hi guys,
Thanks for starting the work. The upstream kubernetes project will eventually remove the CloudStack k8s provider soon, see https://github.com/kubernetes/kubernetes/pull/68199
In that case, we'll need to host/maintain something similar to your ccm, probably as part of the upstream project. Can you advise your goals for the ccm and if they can align such that this can be developed and maintained under an upstream official CloudStack project repo?

Ping @cezarsa @andrewsmedina @andrestc any updates guys?

cc-ing for reference @ustcweizhou @sebgoa @PaulAngus @nvazquez @DaanHoogland

Sorry for taking so long to address this.
tl;dr: I don't really think this project is a good replacement for the deprecated cloudstack CCM, at least not how it is right now.
First of all this project was created to handle some very specific issues with our (Globo.com) private Cloudstack cloud, and with each release it ended up becoming more specific to our reality yet.
Each of these changes either add more complexity that wouldn't exist in a common Cloudstack installation (if such a thing exists) or break compatibility altogether.
I'll try listing the changes we have in this CCM compared to the old cloudstack one.
1 - We have multiple Cloudstack API endpoints with multiple Project IDs, and we wanted a CCM being able to talk to all these multiple instances on the same kubernetes cluster. Because of this constraint we require a few labels to be present on every node identifying its API endpoint and also identifying which projectID it's in.
Related to this same issue, every LoadBalancer service must include annotations identifying which endpoint will be used and which projectID will be used.
https://github.com/tsuru/custom-cloudstack-ccm/blob/c3cb42cb7ef236a524f32ed7260eeafea5b7714a/cloudstack/cloudstack.go#L41-L42
2 - We have the concept of node pools on our infrastructure, a pool label in each node will identify which pool it belongs. Pools are completely segregated on a network level. Because of this we wanted LoadBalancers to only include nodes from a single pool as their destinations, this adds another big change to how load balancers are created because we now also require a pool annotation on each LoadBalancer service.
https://github.com/tsuru/custom-cloudstack-ccm/blob/c3cb42cb7ef236a524f32ed7260eeafea5b7714a/cloudstack/cloudstack.go#L38
3 - Cloudstack API is kind of a non-standardized mess, and it's our fault that we added to this mess by changing some API command names and arguments on our private installation. To support our non-standard version we allow custom command names for some operations used in LoadBalancer creation, including associate-ip and dissociate-ip.
https://github.com/tsuru/custom-cloudstack-ccm/blob/c3cb42cb7ef236a524f32ed7260eeafea5b7714a/cloudstack/cloudstack.go#L54-L56
4 - This is related to 3, but the way we create LBs is really messed-up right now, our custom implementation accepts a additionalPorts argument to load balancer rules and a single rule is created, instead of creating a new rule for each port as the original Cloudstack CCM did.
All that said, it would be possible to change this CCM to work in a similar fashion to the old Cloudstack CCM if no custom configuration is provided. However, due to our unique constraints the code would probably look overly complex.
It was not our intention for this to become the default Cloudstack CCM when we started this project and it shows on how we didn't care for breaking compatibility in the past, changing this now would be feasible but not without some heavy work and I'm afraid we do not have enough hands to work on this right now. If someone from the community would like to take this however I would love to provide some guidance.

@cezarsa when you say "some heavy work", do you have an idea of the amount? (lines of code/methods/blocks to change, days/weeks/.. anykind of measure)

SWISS TXT also prepared a CloudStack CCM: https://github.com/swisstxt/cloudstack-cloud-controller-manager
We intend to keep it as generic as possible and possibly move it back under the k8s umbrella at some point.
For the moment, it's basically the code base of the old CCM, with some modifications from us to make it work more reliably.
I'm happy to discuss any possibilities for a common project; even with custom integrations, it may be useful to work on the CCM together.
The old cloud provider will be removed from the k8s tree soon, so I think it's important to have a good replacement: https://github.com/kubernetes/kubernetes/pull/72178


minimize: remove Option wrap from translate_stmt

Removed the Option in translate_stmt by instead returning an empty vector.
I tried to remove the Option from translate_rvalue as well, this would entail that BinAdd is supported.
Adding this support for Int is relatively easy, but support for Bool is required as well (possible via casting to Int etc. but I don't know if I should just hack a solution.)
Removed the Option in translate_stmt by returning an empty vector instead.
I tried to remove the option from translate_rvalue as well, this would require support for the BinAdd operation.
Adding this support for Int is easy, but support for Bool is also required (possibly by casting to Int and doing the operation there, but I don't know if hacking a solution together is the right thing.)

I tried to remove the option from translate_rvalue as well, this would require support for the BinAdd operation.
Adding this support for Int is easy, but support for Bool is also required (possibly by casting to Int and doing the operation there, but I don't know if hacking a solution together is the right thing.)

So we already have code that generates BinAdd but we currently just do nothing there?
Wow okay that's bad. But this means that removing the Option is blocked on at least figuring out what the BinAdd is for, and maybe implementing it.

this would entail that BinAdd is supported.

You mean BinAnd, I assume?

So we already have code that generates BinAdd but we currently just do nothing there?

We have code that turns BitAnd into None and then ignores it.

No I mean we have tests that need BitAnd. But then we ignore it.
That's pretty bad. Good catch!


Multiple API-specifications

Description
I try to add multiple api-specifications:
swagger1.yaml
swagger2.yaml
and in the application code:
app.add_api("swagger1.yaml")
app.add_api("swagger2.yaml")
Expected behaviour
Expect to start api with one long swagger documentation
Actual behaviour
AssertionError: A name collision occurred between blueprints <flask.blueprints.Blueprint object at 0x7f8e1ee2ea90> and <flask.blueprints.Blueprint object at 0x7f8e1ee2e6d0>. Both share the same name "". Blueprints that are created on the fly need unique names
Steps to reproduce
mentioned above
Additional info:
Maybe this is expected behavior, but the documentation does not specify anything about multiple swagger docs. If this is expected behavior, im sorry for raising the issue.
Output of the commands:

python --version
pip show connexion | grep "^Version\:"

https://github.com/zalando/connexion/issues/254#issuecomment-497194240

Hi @msoodb - it sounds like you're expecting that those specs would be merged?
If so, I don't think that's the intended behavior in connexion.
If what you want is two separate apis, each with a different spec, then you should be able to do that by passing in distinct base_path routes to app.add_api()

For merging multiple API-specifications, please check #254.
For header-based or media-type-based versioning, please check #1346.

Thanks @RobbeSneyders :)


List enumeration render difference between mobile and desktop app

Summary
Enumerated lists are treated differently in Desktop app and mobile client
Environment Information

Device Name: LG G4 - H815
OS Version: 6.0.0
Mattermost App Version: 1.5.1 (Build 74)
Mattermost Server Version: 4.3.0 (Build 4.3.2)

Steps to reproduce
Post the following to a channel:
0. Zeroth point of enumeration
1. First point
2. Second point

Expected behavior
This should render the same in desktop app and mobile app.
Observed behavior
In mobile app, this renders to:
1. Zeroth point of enumeration
2. First point
3. Second point

In desktop app, this renders to:
0. Zeroth point of enumeration
1. First point
2. Second point

Possible fixes
No idea.

Hi @th-joerger,
The behaviour in the desktop app is correct. I've filed a ticket here to go look at it for the mobile app

Hi @th-joerger, I'm not able to reproduce this issue on the latest beta version of the app available from the Play Store. I think it was fixed with https://github.com/mattermost/mattermost-mobile/pull/1304. Do you still see the issue if you upgrade your app to latest?

Hey @hmhealey, I can confirm it as fixed on my device (on which I discovered the bug) with Mattermost Android App version 1.7.1 (Build 92). Thank you!


RFC: Enable Kotlin's new approach to incremental compilation

Expected Behavior
Kotlin's incremental compilation can be used alongside Gradle's build cache.
Current Behavior
We use build cache, which disables Kotlin's incremental compilation. This is a problem in Kotlin 1.6.21 and earlier.
Context
Kotlin 1.7 has a new approach to incremental compilation. Initial impressions are positive, and an issue with occasional corruption of the IC status has been fixed in Kotlin 1.7.10.
This promises significant savings when compilation is required, and likely will help with https://github.com/detekt/detekt/issues/4908, since there will be fewer dependent tasks running which will avoid triggering test tasks.
However, this feature is experimental, and JetBrains recommend using it for evaluation only. I think the pros outweigh the cons, we should enable this, and if we find new issues we can easily turn it off and wait for fixes in newer Kotlin versions, but looking for comments first.

I agree. This doesn't affect our users.


[Docs] Suggesting a section in the Readme file which tells how to add name to contributers list after their PR has been merged.

Category of documentation update
New Category addition to the Readme file of this repository.
Describe the change you think might work
My PR submit regarding Recipe Cost Calculator has been accepted and merged. However after this I am struggling to get my name added in contributers list.
I went on a search throughout this repository's Readme file, hoping to find instructions/help on how to add our name/picture to all the existing contributers list. I couldnt find it, so I am requesting this category to be started so that it can help upcoming contributers. I also request you to publish the instructions on how to add our name to the contributer list.
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

The contributors list is automatically fetched and being added @EshaalakshmiDS


iree/hal/cts:allocator_test, command_buffer_test and device_creation_test fail on real GPU hardware (Quadro P1000)

These three tests fail on real GPU hardware and they are all passing on swiftshader.
All of them hit this assertion:
15: iree_hal_cts_allocator_test: tpp.c:82: __pthread_tpp_change_priority: Assertion `new_prio == -1 || (new_prio >= fifo_min_prio && new_prio <= fifo_max_prio)' failed.

note: just file an issue for record when I was running tests on real GPU hardware. This doesn't block anything.

Interesting. These tests all work for me on Windows (NVIDIA 2080TI, updated drivers and Vulkan SDK). On Linux (NVIDIA Quadro P1000, old drivers, old Vulkan SDK, using extension layer for timeline semaphore support) I see an assertion during test teardown, after the test "passes":
[vk_mem_alloc.h : 15777] RAW: vmaCreateAllocator
I iree/hal/vulkan/vulkan_driver.cc:295] Created Vulkan Device: Quadro P1000
I ./iree/hal/cts/cts_test_base.h:48] Created device 'Quadro P1000'
I iree/hal/cts/device_creation_test.cc:26] Device details:
[DeviceInfo]
  Name: Quadro P1000
  Supported features: []
  Device ID: 32541024
[VulkanDevice]
  Command Queues: 3
    - Dispatch Queues: 2
    - Transfer Queues: 1
[vk_mem_alloc.h : 15787] RAW: vmaDestroyAllocator
DEBUG: Unloading layer library /usr/local/google/home/scotttodd/code/iree/build/third_party/vulkan_extensionlayer/layers/./libVkLayer_khronos_timeline_semaphore.so
[       OK ] AllDrivers/DeviceCreationTest.CreateDevice/vulkan (263 ms)
[----------] 2 tests from AllDrivers/DeviceCreationTest (263 ms total)

[----------] Global test environment tear-down
[==========] 2 tests from 1 test suite ran. (263 ms total)
[  PASSED  ] 2 tests.
external/bazel_tools/tools/test/test-setup.sh: line 310:    14 Segmentation fault      "${TEST_PATH}" "$@" 2>&1
================================================================================
Target //iree/hal/cts:device_creation_test up-to-date:
  bazel-bin/iree/hal/cts/device_creation_test
INFO: Elapsed time: 0.680s, Critical Path: 0.41s
INFO: 1 process: 1 linux-sandbox.
INFO: Build completed, 1 test FAILED, 2 total actions
//iree/hal/cts:device_creation_test                                      FAILED in 0.4s

I see the same type of failure through both CMake and Bazel.

Yes, I see the exactly same error message with CMake. I didn't try with Bazel.

I think we don't have such failures anymore. Closing this. Feel free to reopen if you still see them.


Masking policy state not properly registered with using <<-EOT EOT notation

Provider Version
0.51.0
Terraform Version
v1.3.6
Describe the bug
While defining masking policies in Snowflake we decided to store the masking expression in a properly formatted SQL file instead of always placing the masking expression as a one-liner in the configuration. Having a one-liner worked alright in the beginning, but as masking expressions started to grow it became a pain to review, so we placed the masking expression in a SQL file and used the terraform file function (ref) to load the masking expression from the SQL file.
With this approach, the definition of the masking expression is defined as something like:
<<-EOT
            CASE
              WHEN <some_expression>
                THEN <masked result>
              ELSE val
            END
        EOT

instead of the usual oneliner"CASE WHEN <some_expression> THEN <masked result> ELSE val END".
This is expected as it's the notation for terraform to handle multiline strings. Although, while using this approach terraform always considers that the masking policies need to be changed even when no code was changed, seeming that this notation is not properly registered in the terraform state.
Expected behavior
terraform plan/apply to only detect changes when code is actually changed.
Code samples and commands
Resource definition example:
resource "snowflake_masking_policy" "mask_varchar" {
  for_each           = snowflake_database.market
  name               = "MASK_VARCHAR"
  database           = each.value.name
  schema             = "PUBLIC"
  value_data_type    = "VARCHAR"
  masking_expression = file("./masking_policies/mask_varchar.sql")
  return_data_type   = "VARCHAR(16777216)"
  comment            = "Masking policy for `varchar` columns"
}

./masking_policies/mask_varchar.sql content:
CASE
  WHEN system$get_tag_on_current_column('masked_markets') IS NULL
    OR ARRAYS_OVERLAP(SPLIT(system$get_tag_on_current_column('masked_markets'), ','),
                      TO_ARRAY(system$get_tag_on_current_column('market')))
    THEN IFF(ARRAYS_OVERLAP(SPLIT(UPPER(system$get_tag_on_current_column('public.authorized_roles')), ','),
                            PARSE_JSON(current_available_roles())),
             val, SHA2(val))
  ELSE val
END

terraform plan result returned every time, even when no code changes were made:
  # snowflake_masking_policy.mask_varchar["nl-prod"] will be updated in-place
  ~ resource "snowflake_masking_policy" "mask_varchar" {
        id                 = "PICNIC_NL_PROD|PUBLIC|MASK_VARCHAR"
      ~ masking_expression = <<-EOT
            CASE
              WHEN system$get_tag_on_current_column('masked_markets') IS NULL
                OR ARRAYS_OVERLAP(SPLIT(system$get_tag_on_current_column('masked_markets'), ','),
                                  TO_ARRAY(system$get_tag_on_current_column('market')))
                THEN IFF(ARRAYS_OVERLAP(SPLIT(UPPER(system$get_tag_on_current_column('public.authorized_roles')), ','),
                                        PARSE_JSON(current_available_roles())),
                         val, SHA2(val))
              ELSE val
            END
        EOT
        name               = "MASK_VARCHAR"
        # (6 unchanged attributes hidden)
    }

Result of terraform state show 'snowflake_masking_policy.mask_varchar["nl-prod"]'
# snowflake_masking_policy.mask_varchar["nl-prod"]:
resource "snowflake_masking_policy" "mask_varchar" {
    comment            = "Masking policy for `varchar` columns"
    database           = "PICNIC_NL_PROD"
    id                 = "PICNIC_NL_PROD|PUBLIC|MASK_VARCHAR"
    masking_expression = <<-EOT
        CASE
          WHEN system$get_tag_on_current_column('masked_markets') IS NULL
            OR ARRAYS_OVERLAP(SPLIT(system$get_tag_on_current_column('masked_markets'), ','),
                              TO_ARRAY(system$get_tag_on_current_column('market')))
            THEN IFF(ARRAYS_OVERLAP(SPLIT(UPPER(system$get_tag_on_current_column('public.authorized_roles')), ','),
                                    PARSE_JSON(current_available_roles())),
                     val, SHA2(val))
          ELSE val
        END
    EOT
    name               = "MASK_VARCHAR"
    qualified_name     = "\"PICNIC_NL_PROD\".\"PUBLIC\".\"MASK_VARCHAR\""
    return_data_type   = "VARCHAR(16777216)"
    schema             = "PUBLIC"
    value_data_type    = "VARCHAR"
}

Additional context
The behaviour is the same If I paste the masking expression using the EOT notation directly into the resource definition instead of using the file() function.

We found a workaround that is to do a trim of \n after loading the SQL file content (using file() terraform function).
It appears that the problem does not come from the EOT notation or the fact that we're using a multiline string, but from the fact that the SQL file has a newline in the end of the file.

Duplicated https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/1097

Hi, we've released v0.96.0, which includes adjusting and fixing masking policies. Please upgrade with the migration guide.

Closing the issue due to inactivity. Please create a new one if the issue persists in the newest version of the provider.


Fix test for tensorsolve

I found axes argument in tensorsolve is not tested.

i fixed it

Thank you.
Cloud you resolve conflict?

done

LGTM!


Cut v0.17.0 on September 3, 2021

Steps

This issue can be closed when we have:

[X] Created the release branch.
[X] Run the Tag workflow on the relevant release branch with the proper release version (link).
[X] Run the Tag workflow on the master branch with the next release candidate tag (Tag).
[X] Run the CI workflow on the release branch.
[X] Run the Promote workflow with channel stable on the release branch.
[X] Published a new release for the tagged version, with the same name as the version and descriptive release notes (link).
[X] Ensured that users have been notified of the release on all communication channels:

[X] Slack (link)
[X] Twitter (link)

Notes
As a sanity check, I've successfully tested CRUD operations via a PostgreSQLServerConfiguration MR after a successful installation of provider-azure v0.17.0 using kubectl crossplane install provider command. The PostgreSQLServerConfiguration MR is not available in older versions of the provider.

Thank you @hasheddan for the announcements!


Proposal: re-define S_0 -> S_0 / Q^2

If we re-define S_0 as S_0 / Q^2, not much changes but the peak power at omega = omega_0 is no longer a function of Q. This is pretty much what I always want. Any reason why this might be a bad idea?

That sounds like a good idea. Here are a couple of considerations:
1).  The peak power is at \omega_0*\sqrt{1-1/(2Q^2)} (I had defined this as "x_max" in a prior version of the paper where I had fit the peak with a single Lorentzian, not the symmetric double Lorentzian).  So, it's almost the peak, but not quite, I think.... so the peak power will be something like ~S_0*Q^2/(1-1/(4Q^2)) (this is the old S_0).  See the attached notebook.  The width also differs slightly from 1/(2Q).
sho_peak.nb.pdf
2). With this redefinition, the kernel then has an ~\omega_0/Q dependence at \tau = 0, I think. This is actually nice since the damped term is proportional to \omega_0/Q as well, so if you transform \tau to \omega_0*\tau/Q, then the amplitude at zero would be just proportional to your new S_0.

Good point in (1) there. Perhaps we should use: S_0 * 4*Q^4/(4*Q^2 - 1).

I've reverted this change but let's come back to it later because it might be good to choose a more useful parameterization in practice.


Reverse discovered beacon UUID, fixes h2zero/NimBLE-Arduino#86

@lknop Have you been able to test this and is it ready to merge?

Hi, yes it seems it ticks all the boxes, at least in the scope that I need. I consider it ready to merge.

Great, thanks!


Updates to typings

My Typescript compiler complains if I try to use the DDragon endpoints since they are not included in the typings file.
I've added them, but the only way I could get it to accept the DDragonRequest return type was by converting it to a class. If you would prefer that to not be touched, I can have them return any. I wasn't sure which one was best, but thought it would be better to keep it specific rather than falling back on the any type.
I also added the rune bonuses to the MatchV4ParticipantStatsDto. These are not on the official documentation page yet, but they are returned in the data.

Thank you for the pull request! This is nice.
DDragonRequest
I don't mind converting it to a class at all if we are able to restrict the types effectively.
Adding manual types
I left one comment regarding using yarn swagger, do you mind running it just to double-check? I know the new match stuffs is not on the documentation though, and so it'll probably fail, but it's more of a sanity check.
I do not mind adding manual types, so this is fine.

I did use the DTO typing in my project. Honestly I just want them to make the VS Code Intellisense work :smile:
I have only used the Version endpoint of DDragon, so have nothing written up to test the other bits. My only real concern is that even though I managed to get the properties/fields of the endpoints, the return type is always any since it is coming from a plain .js file. I didn't notice this at first. Not sure I can call it a "typing" if they don't return any types!

Is it possible for you to make the branch merge destination dev instead of master? If you don't mind :). I will test it tomorrow

I now have an open issue with the Riot Developer Relations to get the rune statPerks added to the documentation. Once they do that, swagger should be able to update and get them in officially. If it takes them forever, I may do a pull request to the riotapi-swagger project to get that override in.


Refactor/lib restructure

name example/simple is misleading. This is the most complex example with all crates included

Can you make it so that example imports crates it need to use. Now in example contract the rmrk crate is included which then includes all other.
Perhaps creating one more example with only minting and nesting will show the best way to do that.

Can you make it so that example imports crates it need to use. Now in example contract the rmrk crate is included which then includes all other. Perhaps creating one more example with only minting and nesting will show the best way to do that.

IMO it's better to keep to a single entry crate (much like what ink! has moved to) rather than users having to depend on crates individually. Maybe we could put crates behind features flags so users only pull in the sub-crates they need, but still only depend on the single rmrk crate?

@boyswan agree. Let's create features with logical group of crate as you mentioned and keep a single entry crate


Move from request-promise to axios

What problem does the pull request solve?

Requests has been deprecated and so we replace it with a more modern
library. This will mean some small breaking changes for users which are
noted by the version bump and changelog records.
Also fixes the User-agent header to be User-Agent as per the
docs:
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/User-Agent
How to review

Please review line by line for a general code review
Have a think if there is anything missing from the changelog or ways we could improve the experience for developers upgrading
Acknowledge the decision that the published docs will be out of date with older versions of the client (but that appears to be a pattern we already use)
General thumbs up that assuming the code is all 'correct', this we still want to make this change as it will mean a major breaking change for our users

Question
Our docs (https://docs.notifications.service.gov.uk/node.html#response) say that If the request to the client is successful, the promise resolves with an object and show properties on that object like subject, body, date_created etc. However, for a user to access this I think they instead need to look at response.body which we do not make clear. Is that correct? If so, then it also means that with this change they will instead need to look at response.data instead? If so, should our docs be making that clear that people need to look at that. I've never heard any comments from users about it before so maybe they don't have any trouble with it?
Checklist

[x] I’ve used the pull request template
[x] I’ve written unit tests for these changes
[x] I’ve updated the documentation in

DOCUMENTATION.md
CHANGELOG.md

[x] I’ve bumped the version number in

package.json

[ ] I've added new environment variables in

CONTRIBUTING.md
notifications-node-client/scripts/generate_docker_env.sh

@tombye Thanks Tom. I believe I answered your questions and added commits based on some of them. Let me know if there is anything further you can think of and then I will ask Karl to take a look :)


TreeBuilder: fix chargeback.

@GregP : fix for your issue, please, test.

including an explanation sent to @GregP :
TreeBuilder includes Sandbox as a way to access session data. Same as all controllers.
So naturally inside controllers you access Sandbox methods directly -- like 'x_node =  .... '.
Inside TreeBuiler code you also use 'x_node' and friends.
Because methods from included modules are public by default, tree.x_node worked. Although it was not ever meant as par of TreeBuilder public API.
With recent changes to TreeBuilder classes I wrapped the Sandbox methods a bit more, so that those are no longer accessible from outside. And
that change broke the code.
So yes, recent changes to TreeBuilder made this stop working. But the method should never have been called this way in the first place and I
hope there's no other place like this one.

Is there a way we can add an automated test for this? @GregP / @martinpovolny

@tenderlove : here we go

Review? Someone?  @GregP ? @mzazrivec ?


Improve markdowncli-lint integration with renovate

Description
Improve markdowncli-lint integration with renovate
How can this be tested?
Run: make prerequisites/markdownlint
Checklist

[x] Unit tests have been updated/added
[x] PR is labeled accordingly with a single label
[x] I have read and understood the contribution guidelines

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

Comparison is base (c93b150) 64.70% compared to head (67a9f1a) 64.70%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2418   +/-   ##
=======================================
  Coverage   64.70%   64.70%
=======================================
  Files         245      245
  Lines       16229    16229
=======================================
  Hits        10501    10501
  Misses       4929     4929
  Partials      799      799

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


Fix is_dir under Python 3

It is no longer returning True for files with binary content.
Fixes #23.

is this PR planned to be merged soon?


Adding new property for Target

Can you please add a new property ('name', data type - string) for the target model.

existing target_name will be used


Saltern support

About the PR
Added Saltern into rotation and Nyano-ified it.
Changelog

made the old salvage room the mail room
replaced singulo room with salvage
removed singulo
thats all the major stuff everything else is minor

@Elijahrane DNM for now

Ready for review


Update changelog and release changelog for v1.12.2.

Description
Update the changelog and release changelog for v1.12.2.

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Gonna hold off on this for a bit to get some more testing in of the iavl upgrade stuff. I'll need to update the date once read for it again.


Camera flight issues

The below Sandcastle example is a simple save/restore camera view that exposes two issues in Cesium.

Flights that start far away from their destination and end up on the ground will cause the camera to collide with terrain while it is in flight. This causes the camera to end up wherever the terrain was loaded when it collided and not the end destination.

I think the minimum distance is being taken into account differently during camera flights than it does during camera motion. This means if you move the camera close to the ground with a horizon view, save the settings, and then do a flight; you end up in a difference location.

var viewer = new Cesium.Viewer('cesiumContainer');
var cesiumTerrainProviderMeshes = new Cesium.CesiumTerrainProvider({
    url : 'https://assets.agi.com/stk-terrain/world',
    requestWaterMask : true,
    requestVertexNormals : true
});
viewer.terrainProvider = cesiumTerrainProviderMeshes;

var view;
var camera = viewer.scene.camera;

Sandcastle.addToolbarButton('Store View', function(){
    view = {
        destination : camera.positionWC,
        orientation : {
            heading: camera.heading,
            pitch: camera.pitch,
            roll: camera.roll
        }
    };
});

Sandcastle.addToolbarButton('Load View', function(){
    camera.flyTo(view);
});

Using the above code, fly to San Francisco and create a horizon view right up against the ground.  Storing and loading the view won't match.
Stored view

Result after flight

@mramato thanks for narrowing this down.
@bagnell can you take a look soon since this is affecting demos.  Perhaps it is as simple as disabling collision detection and minimum distance during flights?


Kh nav login modal

This update only affect the Nav partial, and comments out some features we may use in the future, but as of right now have no plan.
The person icon launches a modal that is coded into the Nav partial for login fields
There is some weird styling issues that I need to solve, but I wanted to get the modal up ASAP

This does not overlap or interfere at all with my other pull request!


How to communicate between viewmodels

not only show

There is no built in mechanism. If you need one, create one yourself

okay, thanks a lot.


fix(netlify-cms-core): fields metadata

Summary
This PR would prepend field's name for field's metadata. Currently, the kitchen sink relationKitchenSinkPost widget custom preview only works for the first top-level widget but does not work for the other two nested relationKitchenSinkPost widget. This PR fixes the issue. Plus the fix for  #1982 depending on this PR.

<!--
Demonstrate the code is solid.
Example: The exact commands you ran and their output, screenshots / videos if the pull request changes UI.
--

@barthc I just tried out the relation widgets in the kitchen sink from this PR preview and from the current cms demo, and the previews for both behave the same - not present on load, but shows when you edit. Are you seeing anything different?

@erquhart  that is a known issue #1541. The simple fix for that is to allow the preview to update on fieldsMetaData change.

I wasn't able to reproduce the issue you mentioned in the OP, but I spent a little more time and found it 👍👍


Consul discovery function buildServiceDiscoveryMiddleware puts consul server itself as target.

I configure tree gateway with consul with steps mentioned here:
https://github.com/cloudhrm/frontend
But when I try to call /auth endpoint of gateway, I get Invalid gateway. GW log is as follows:
error: [Tree-Gateway] Error occurred while trying to proxy request / from localhost:8000 to undefined (ETIMEDOUT) (https://nodejs.org/api/errors.html#errors_common_system_errors)
2019-05-23T17:02:36.153Z - error: HTTP GET - 502 - /auth (23790ms)

I am not function buildServiceDiscoveryMiddleware should return consul or service IP. But it sets consul IP as target.

And with debug I see that actual error is failing to connect with "auth" api within consul hostname and registered service port :(

I have also the same issue, after  flush redis and reconfigure gateway and api then restarted, now its working

also open the server at routed api , that might be a reason for your error


Shift and CRTL+A group selection does not work on the AnimationPlayer Copy Tracks pop up

Godot version:
3.2 alpha 3
OS/device including version:
All
Issue description:
When you hold the Shift key and click the bottom track and then the top track or vica versa, and just those tracks are selected rather than all of the tracks which are inbetween those two.
In addition, CTRL+A does not work either and rather should have just the sellect all functionality as already coded in for the Sellect All/None button.
Steps to reproduce:
Create AnimationPlayer, create animation, create at least 3 tracks, press Edit, press Copy Tracks, hold the Shift key and click on the bottom track, then the top track, or vica versa.

Still valid in bf37ab5


[NIFI-13053] Cluster page

NIFI-13053

New Cluster page accessible from the main menu
Listings for:

Nodes
System
JVM
FlowFile Storage
Content Storage
Provenance Storage
Versions

All table here extend a base table ClusterTable and leverage the same filter component. The "Storage" pages all share the same same table implementation.

Will review...


Fix UTC Timestamps in PostgreSQL

According to the docs, timestamps(type: :utc_datetime) should result in PostgreSQL columns of type timestamp(0) with time zone.
However, those options result in columns with unexpected types (no time zone):
   Column    |              Type
-------------+--------------------------------
 inserted_at | timestamp(0) without time zone
 updated_at  | timestamp(0) without time zone

This change fixes those options to result in columns with the expected types (with time zone):
   Column    |            Type
-------------+-----------------------------
 inserted_at | timestamp(0) with time zone
 updated_at  | timestamp(0) with time zone

utc_datetime does not include timezone pretty much on purpose. naive_datetime and utc_datetime are about the same representation in the database but different representation on the Elixir side of things. I have improved the docs here, thank you!

Thank you for the explanation and for improving the docs.
By using @timestamps_opts [type: :utc_datetime] in the schema, I do get DateTime rather than NaiveDateTime. But that doesn't affect how the timestamps are stored in the database. My expectation was that by putting [type: :utc_datetime] in the migration (not just in the schema), the timestamps in the database would be (explicitly) in UTC. If not, why have that option in migrations?

The option for migration is only for you to keep both formats consistent but they are the same internally (at least in MySQL / Postgres).

@mmaa you can force migration to using type of your choice by doing:
create table(:posts) do
  timestamps(type: :"timestamp(0) with time zone")
end

or setting it to be the default:
config :myapp, Repo,
  # ...
  migration_timestamps: [type: :"timestamp(0) with time zone"]

@wojtekmach Thanks! It hadn't occurred to state the type explicitly like that. That works perfectly for me.
@josevalim Internally, the stored timestamps are almost identical but not the same. Postgres doesn't actually store a time zone, but it does explicitly store the timestamps as UTC rather than being ambiguous. It can then display the times in the local time zone.
timestamp(0) without time zone:
     inserted_at     |     updated_at
---------------------+---------------------
 2019-03-29 13:03:08 | 2019-03-29 13:03:08

timestamp(0) with time zone:
      inserted_at       |       updated_at
------------------------+------------------------
 2019-03-29 08:03:08-05 | 2019-03-29 08:03:08-05


Link yourself with friends by sharing your meal plan on social medias

Provide a Share feature where users can share their thoughts, meal activities, and meal agenda with others on Twitter, Facebook, and Instagram.

featured


Monokai theme - C functions are green instead of blue

VSCode Version: 1.11.2
OS Version: Windows 7 Home Premium
Theme: Monokai
Language: C

Hello coders, I recently updated the editor to version 1.11.2 and I noticed that the Monokai theme has changed. In every previous version the function names were coloured in blue, and now they're green. I must admit that I don't like the green. Can I somehow change it back to blue? Thank you in advance.

Used to be blue:

Steps to Reproduce:

Open a ,c file and set the workbench theme to "Monokai"
Now check how functions are colored

The green matches the Monokai theme in Atom:

Sublime out of the box seems to use blue though:

Hello @mjbvz , Thank you for your reply. It seems that every editor have its own color-scheme of the Monokai theme. So there isn't a consistent scheme among those editors. But from the looks I'd say that green is the "right" color. It'll probably take a while for me to get used to it, but that's how it was when Github introduced the black navbar. Once again, thank you very much, Cheers Steven.

Some releases ago we made fixes to engine that matches colors to text mate scopes. That resulted in some color changes.
The variant of Monokai we have (it's from http://colorsublime.com/?q=monokai#) colorizes entity.name.function.c as green. We have no plans on changing this.
You might want to check out other Monokai themes in the Marketplace, or think about creating your own variant of Monokai.


Add Documentation

Codecov Report

Merging #12 (74564ff) into master (5ec6487) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #12   +/-   ##
=======================================
  Coverage   76.81%   76.81%
=======================================
  Files           3        3
  Lines          69       69
=======================================
  Hits           53       53
  Misses         16       16

Impacted Files
Coverage Δ

structoscope/lib.py
65.21% <ø> (ø)

Continue to review full report at Codecov.

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