make deb on ARM is broken in 1.12.0-rc5

With docker 1.12.0-rc5 running a make deb on ARM is broken as it tries to pull and run the golang:1.6.3-alpine image for amd64 generating the manpages.
Output of docker version:
Client:
 Version:      1.10.0
 API version:  1.22
 Go version:   go1.4.3
 Git commit:   590d5108
 Built:        Thu Feb  4 21:04:51 2016
 OS/Arch:      linux/arm

Server:
 Version:      1.10.0
 API version:  1.22
 Go version:   go1.4.3
 Git commit:   590d5108
 Built:        Thu Feb  4 21:04:51 2016
 OS/Arch:      linux/arm

Output of docker info:
Containers: 22
 Running: 1
 Paused: 0
 Stopped: 21
Images: 363
Server Version: 1.10.0
Storage Driver: overlay
 Backing Filesystem: extfs
Execution Driver: native-0.2
Logging Driver: json-file
Plugins:
 Volume: local
 Network: bridge null host
Kernel Version: 4.5.2-std-2
Operating System: Ubuntu 16.04 LTS
OSType: linux
Architecture: armv7l
CPUs: 4
Total Memory: 1.975 GiB
Name: jenkins-slave01
ID: DFJ3:7BSF:SOGE:A5CN:HZT3:3Y35:UITQ:EEH7:YDHI:FFLX:RUXV:EBUA
WARNING: No cpu cfs quota support
WARNING: No cpu cfs period support
Labels:
 provider=scaleway

Additional environment details (AWS, VirtualBox, physical, etc.):
Scaleway C1 with Docker preinstalled
Steps to reproduce the issue:

Use eg. a Scaleway C1 ARM machine
git clone https://github.com/docker/docker
cd docker
make deb

Describe the results you received:
---> Making bundle: dynbinary (in bundles/1.12.0-rc5/dynbinary)
Building: bundles/1.12.0-rc5/dynbinary-client/docker-1.12.0-rc5
Created binary: bundles/1.12.0-rc5/dynbinary-client/docker-1.12.0-rc5
Building: bundles/1.12.0-rc5/dynbinary-daemon/dockerd-1.12.0-rc5
Created binary: bundles/1.12.0-rc5/dynbinary-daemon/dockerd-1.12.0-rc5
Building: bundles/1.12.0-rc5/dynbinary-daemon/docker-proxy-1.12.0-rc5
Created binary: bundles/1.12.0-rc5/dynbinary-daemon/docker-proxy-1.12.0-rc5

---> Making bundle: build-deb (in bundles/1.12.0-rc5/build-deb)
---> Making bundle: .integration-daemon-start (in bundles/1.12.0-rc5/build-deb)
INFO: Waiting for daemon to start...
+++ exec dockerd --debug --host unix:///go/src/github.com/docker/docker/bundles/1.12.0-rc5/build-deb/docker.sock --storage-driver overlay --pidfile bundles/1.12.0-rc5/build-deb/docker.pid --userland-proxy=true
.
---> Making bundle: .detect-daemon-osarch (in bundles/1.12.0-rc5/build-deb)
docker build -t docker-manpage-dev -f man/Dockerfile .
Sending build context to Docker daemon 557.1 kB
...
Sending build context to Docker daemon 157.5 MB

Step 1 : FROM golang:1.6.3-alpine
1.6.3-alpine: Pulling from library/golang
e110a4a17941: Pulling fs layer
...
25994cc11974: Pull complete
Digest: sha256:8b9e1e4a137e7663c1dc52a33c41699661ea4da9aca6ab0771b1fcec16a87535
Status: Downloaded newer image for golang:1.6.3-alpine
 ---> 6b317e0ae72c
Step 2 : RUN apk add -U git bash curl gcc musl-dev
 ---> Running in 23bd447afd78
standard_init_linux.go:175: exec user process caused "exec format error"
The command '/bin/sh -c apk add -U git bash curl gcc musl-dev' returned a non-zero code: 1
Makefile:119: recipe for target 'manpages' failed
make: *** [manpages] Error 1
---> Making bundle: .integration-daemon-stop (in bundles/1.12.0-rc5/build-deb)
+++++ cat bundles/1.12.0-rc5/build-deb/docker.pid
++++ kill 5433
Makefile:86: recipe for target 'deb' failed
make: *** [deb] Error 1

Describe the results you expected:
Building the ARM deb's should work without a problem :-)
I had a look at man/Dockerfile to see if we can add eg. a arch specific man/Dockerfile.armhf but found out that glide does not provide a precompiled ARM binary. So there is some more work to make it work.
Another idea is to generate the man pages and check them into GitHub with a [ci skip] commit message.

ping @dnephin @tiborvass ^^

Ya, we could generate the man pages first, instead of doing it as part of the build-deb script. I don't like the idea of checking it into git though.

I understand. Building man pages must be done on the ARM machine as it is a separate Jenkins slave  building the deb's for ARM.
Just some other ideas. Adding the go tools/libs needed to build the man pages into the vendor folder of docker itself? Just like the dev-dependencies in Node.js projects to build the app/man/package. So there is no need to build the manpage inside another docker container. I assume we are running make deb in CI and are already running in a container, so everything should be there to build the manpages.
Or adding the golang:1.6.3-alpine to the frozen images that have to be pulled before the docker-daemon. There we have a chance to pull different images for each architecture. But it feels complicated to solve this small build step.
Maybe there is a much simpler solution.

I can compile glide for arm and put it on bintray. Then if we create a Dockerfile.armhf it should work ?

Probably it should work. We should use an more or less official base image in the FROM line. There is no golang/alpine combination, but armhf/golang (7 months old) could be a first starting point.

I would rather use just alpine:3.4 and apk add -U go, I'll see if that works.

#25192 seems to be working for me on scaleway. Let me know if you have any problems with it.


winston-daily-rotate-file not registered on npm

DailyRotateFile removed  from winston core, but winston-daily-rotate-file not yet registered on npm.
Thanks!

Do not open that issue here. Please open it on winston-daily-rotate-file.


Ability to use custom/non-Ahem/CJK fonts in flutter_tester

Currently, Flutter tester seems to lack the ability to use custom fonts.
There are many situations where varied fonts are required (such as to test how flutter handles different types of glyphs in different combinations) that the uniform-ness of Ahem makes impossible or very difficult to test.

#17700 looks related

See also https://github.com/flutter/flutter/issues/17910#issuecomment-431884965

Though the functionality isn't documented well, flutter_tester can use the font loader API. Is that sufficient for this issue ?


youtube-dl doesn't work with "limited" videos

What is the purpose of your issue?

[x ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
youtube-dl -v -F https://www.youtube.com/watch\?v\=Q2Ki6XwaQ94
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'-F', u'https://www.youtube.com/watch?v=Q2Ki6XwaQ94']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.07.02
[debug] Python version 2.7.10 - Darwin-14.5.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.3.2, ffprobe 3.3.2
[debug] Proxy map: {}
[youtube] Q2Ki6XwaQ94: Downloading webpage
[youtube] Q2Ki6XwaQ94: Downloading video info webpage
Traceback (most recent call last):
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/usr/local/bin/youtube-dl/__main__.py", line 19, in <module>
  File "/usr/local/bin/youtube-dl/youtube_dl/__init__.py", line 465, in main
  File "/usr/local/bin/youtube-dl/youtube_dl/__init__.py", line 455, in _real_main
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1927, in download
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 762, in extract_info
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 433, in extract
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1595, in _real_extract
ValueError: invalid literal for int() with base 10: ''
...
<end of log>

Description of your issue, suggested solution and other information
Herr Schmidt's YouTube now has a "limited" mode for videos espousing wrongthink according to the ADL (such as painting a swastika on a helmet for the purpose of battle reenactments). These "limited" videos cannot currently be downloaded by youtube-dl.

[debug] youtube-dl version 2017.07.02

Please try it with the latest version.

According to brew, I have the latest version

youtube-dl -v -F https://www.youtube.com/watch\?v\=Q2Ki6XwaQ94\&bpctr\=1503822371
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'-F', u'https://www.youtube.com/watch?v=Q2Ki6XwaQ94&bpctr=1503822371']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.08.23
[debug] Python version 2.7.10 - Darwin-14.5.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.3.2, ffprobe 3.3.2
[debug] Proxy map: {}
[youtube] Q2Ki6XwaQ94: Downloading webpage
[youtube] Q2Ki6XwaQ94: Downloading video info webpage
Traceback (most recent call last):
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/usr/local/bin/youtube-dl/__main__.py", line 19, in <module>
  File "/usr/local/bin/youtube-dl/youtube_dl/__init__.py", line 465, in main
  File "/usr/local/bin/youtube-dl/youtube_dl/__init__.py", line 455, in _real_main
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1958, in download
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 776, in extract_info
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 434, in extract
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1596, in _real_extract
ValueError: invalid literal for int() with base 10: ''

Latest version is youtube-dl 2<IP_ADDRESS>

Feel free to try the same line.

You are using an outdated version of youtube-dl. Read this FAQ entry if you have troubles updating.


Jetson xavier nx ERROR!

Hi JK,
I've use on Jetson xavier nx,Jetpack SDK 4.4 .
got error. pls help. Thank you.
jetson@xavier:~/tensorrt_demos$ python3 trt_ssd_async.py --model ssd_mobilenet_v1_coco --video_looping --video /home/jetson/Videos/traffic_960x540.mp4
TrtThread: loading the TRT SSD engine...
[TensorRT] ERROR: INVALID_CONFIG: The engine plan file is generated on an incompatible device, expecting compute 7.2 got compute 5.3, please rebuild.
[TensorRT] ERROR: engine.cpp (1546) - Serialization Error in deserialize: 0 (Core engine deserialization failure)
[TensorRT] ERROR: INVALID_STATE: std::exception
[TensorRT] ERROR: INVALID_CONFIG: Deserialize the cuda engine failed.
Exception in thread Thread-1:
Traceback (most recent call last):
File "/home/jetson/tensorrt_demos/utils/ssd.py", line 88, in init
self.context = self.engine.create_execution_context()
AttributeError: 'NoneType' object has no attribute 'create_execution_context'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/lib/python3.6/threading.py", line 916, in _bootstrap_inner
self.run()
File "trt_ssd_async.py", line 97, in run
self.trt_ssd = TrtSSD(self.model, INPUT_HW)
File "/home/jetson/tensorrt_demos/utils/ssd.py", line 92, in init
raise RuntimeError('fail to allocate CUDA resources') from e
RuntimeError: fail to allocate CUDA resources
Exception ignored in: <bound method TrtSSD.del of <utils.ssd.TrtSSD object at 0x7f64db2cf8>>
Traceback (most recent call last):
File "/home/jetson/tensorrt_demos/utils/ssd.py", line 99, in del
del self.cuda_outputs
AttributeError: cuda_outputs
PyCUDA ERROR: The context stack was not empty upon module cleanup.
A context was still active when the context stack was being
cleaned up. At this point in our execution, CUDA may already
have been deinitialized, so there is no way we can finish
cleanly. The program will be aborted now.
Use Context.pop() to avoid this problem.

[TensorRT] ERROR: INVALID_CONFIG: The engine plan file is generated on an incompatible device, expecting compute 7.2 got compute 5.3, please rebuild.

You need to rebuild the TensorRT engine on the Jetson Xavier.

[TensorRT] ERROR: INVALID_CONFIG: The engine plan file is generated on an incompatible device, expecting compute 7.2 got compute 5.3, please rebuild.

You need to rebuild the TensorRT engine on the Jetson Xavier.

[TensorRT] ERROR: INVALID_CONFIG: The engine plan file is generated on an incompatible device, expecting compute 7.2 got compute 5.3, please rebuild.

You need to rebuild the TensorRT engine on the Jetson Xavier.

Thank you for very fast response. I'll try rebuild that.

[TensorRT] ERROR: INVALID_CONFIG: The engine plan file is generated on an incompatible device, expecting compute 7.2 got compute 5.3, please rebuild.

You need to rebuild the TensorRT engine on the Jetson Xavier.

Thank you for very fast response. I'll try rebuild that.

Has this issue been fixed?

Closing due to no response.


Add public memberwise initializers to layers.

Add a memberwise initializer to every layer. Resolves #30.
Notes:

In some layers, some stored properties are not public, we should consider making them public.
All initializers that deal with integers take Int, which is good. However, some layers expose a public Int32 stored property. This is inconsistent with the initializers and is a leaky abstraction that should be fixed. One potential solution is to rename the Int32 property, make it private, and define a computed property that has a getter (and setter, if necessary) that mutates the private property with a conversion. The other potential solution is to use Int32 everywhere. Neither is pretty.

How about marking the initializers with @differentiable?

TF-305, thanks to your suggestions...


Specify which monitor to use for dynamic island

Hey. I have started building a new setup where i am using my macbook as my secondary display. is there a way to only keep sketchbar on the macbook even if the primary display is the monitor.

I thought this would be an easy implementation I could do by changing display property of SketchyBar. Although this technically positions the Dynamic Island to the correct display, it seems like the animation only plays whenever the monitor is in the focus. I believe this is happening because I'm using a "popup" item for the islands. I always thought of not using "popup" item in hindsight, it seems like now is the right time to actually make the change.
I believe making this change also has a potential in resolving #39 as well as better config.

even if it doesn't work while its unfocused, i just want to keep it on the macbook because it looks weird on the monitor lol

As of right now, this feature cannot be used with the current configuration file. I don't plan on specifically implementing this feature and making changes to the configuration file at the moment, mostly because I'll be making major configuration changes anyway (with the changes that will fix the animations completely).
If you'd like to have the island shown on your second screen right now, you'll need to modify the source code directly. You can do this in item.sh. Under sketchy_bar, add the following
display=2

Where 2 is the id of your display (so in this case it would be using your secondary display).
You may also need to remove the line where associated_display for the island is set (same item.sh file).

This has now been implemented properly with the latest config changes. This, however requires you to update the configuration file entirely. A clean reinstallation is recommended.


Placeholders are not being replaced

Hey there!
First off, great presentation at NordicJS. Really inspired me to give Electron a shot =)
However, I'm having some issues getting the Accelerator to work properly. I go through the wizard entering name, description etc., but package.json and config.json still contain placeholders like {{application-name}}. This causes the bootstrap and build scripts to fail, naturally. I've tried this on both Mac and Windows, getting the same result.
When replacing these placeholders manually I can get a Windows executable out of it, but it crashes referring to "Error: Update URL is not set at SquirrelUpdater.checkForUpdates()" etc. This happens no matter what I enter into the optional variables. Any ideas on how to get past this?

I go through the wizard entering name, description etc., but package.json and config.json still contain placeholders like {{application-name}}

Hi @Tickolas this should indeed not be happening. Are you able to please run electron-accelerator -v and let me know which version you are running?

Sure thing, @ammeep. Looks like version 0.0.4.

Thank you @Tickolas. Honestly I'm not sure why this is not working for you, so I'm working on an update which will allow you to pass in a --debug flag so we can see what is going on.
In the meantime, could you please give me an example of how you are running the command?

Happens here too.
Version 0.0.4
Mac OSX Yosemite 10.10.5
Tried on node v0.10.38, v4.1.1 and iojs-v2.5.0
Latest on master works well though.

@AndreasArledal thanks for testing out master! I have one small improvement I want to ship before pushing an update to npm. Hopefully I'll have out 0.0.5 over the weekend.

Hey @ammeep, I am another one that got inspired to give Electron a shot after your NordicJS presentation. Awesome stuff!
Have you pushed any updates to NPM?
I am having the issue too...

I'm working on fix for this here https://github.com/ammeep/electron-accelerator/pull/24
It's named terribad, but essentially I'm ripping out the old way of gathering args from the CLI. I  have also added a debug option so we can get a glimpse if this happens again. I'll get this out as fast as I can and let you all know :+1:

Hey there folks. Sorry for the delay - life got in the way... you know how it is.
I've overhauled the way the scripts work, to make way for other updating mechanism. And as part of that, this will have been fixed. 0.0.5 is up on NPM now. https://www.npmjs.com/package/electron-accelerator. Please check the read me for CLI changes :+1:


ILCompiler nuget package support

Below is a working implementation - CoreRT can be consumed by adding a
 <PackageReference Include="Microsoft.DotNet.ILCompiler" Version="X" />

where X is the version of the ILCompiler package produced by build-packages.cmd or once its published with
dotnet add package Microsoft.DotNet.ILCompiler

Ideally, I'd like to add the assemblies produced by the project references in Microsoft.DotNet.ILCompiler.pkgproj to have their TargetPath property overriden when creating this package specifically, but the paths seem to be dynamically generated somewhere in the compiler. Can anyone more familiar with the NuGet package publishing pipeline take a look?
Additionally, the implementation below doesn't include symbols for the framework, sdk and tools assemblies to the final package.

@A-And, thank you!
If you don't mind, I would suggest to check in the sample as a separate PR. Basically:

Merge the compiler nuget package support
Make sure we have an official build published
Make the sample work against the latest officially published build and check the sample in.

@sergiy-k Sure thing.

FWIW I gave the package (1.0.0-alpha-25930-03, from yesterday) a try and it seems to be working fine, a native executable is produced. There are a number of linker warnings like:
bootstrapper.lib(main.obj) : warning LNK4099: PDB 'bootstrapper.pdb' was not found with 'bootstrapper.lib(main.obj)' or at 'D:\Projects\cda\bin\x64\Release\netcoreapp2.0\win-x64\native\bootstrapper.pdb'; linking object as if no debug info [D:\Projects\cda\cda.csproj]
Runtime.lib(MiscHelpers.obj) : warning LNK4099: PDB 'Runtime.pdb' was not found with 'Runtime.lib(MiscHelpers.obj)' or at 'D:\Projects\cda\bin\x64\Release\netcoreapp2.0\win-x64\native\Runtime.pdb'; linking object as if no debug info [D:\Projects\cda\cda.csproj]

Presumably the .pdb files for native static libraries like runtime.lib should be included in the package as well. The same way that VC++ installs both .lib and .pdb files for its static runtime libraries.

@mikedn Thanks for testing it out! The motivation behind excluding them was to keep the package as minimal as possible, but we should avoid linker warnings as possible.

@A-And, there should be no warning while compiling an app (i.e. the ones that @mikedn mentioned). We should either ship the pdb files or disable the warnings in the target files with a comment. But this should not block merging your PR. It can be done as a separate change. Thank you!

@sergiy-k I've manually added the debug symbols for static libraries for now. All .pdbs are already published as part of the nuget symbol package, so we have to consider how a user debugging scenario should work.

All .pdbs are already published as part of the nuget symbol package

Hmm, it's unlikely that static library .pdbs need to be published, normally the debugger does not look for them, only the linker.
Those static .pdbs might be needed during debugging if the /DEBUG:FASTLINK linker option is used and ILC probably doesn't do that.
And even if it does, this is something that's supposed to work on the developer's machine, where static .pdbs already exist. In "release" scenarios, the .pdb produced by ILC should be a standalone .pdb. The debugger using it is not supposed to need static library .pdbs, .objs or any other intermediate build files.

Sigh, the Windows build is failing with Method '[MSBuild]::IsOSPlatform' not found. Static method invocation should be of the form. The CI machines are not upgraded to latest VS/msbuild yet, and so we cannot remove the workaround.
Could you please keep the workaround in place? (Keep the new proper OS detection code around commented out.)

@jkotas Thanks for confirming, will do.


[WIP] Remove asyncRoute hoc to use react-loadable

This is a first step towards SSR. Also this is much cleaner as we stop setting states, reducers and locales in render methods.

Codecov Report

Merging #461 into master will decrease coverage by 0.21%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #461      +/-   ##
==========================================
- Coverage   20.59%   20.38%   -0.22%
==========================================
  Files         161      163       +2
  Lines        1248     1261      +13
==========================================
  Hits          257      257
- Misses        991     1004      +13

Impacted Files
Coverage Δ

...outes/CatalogsList/components/CatalogsListRoute.js
0% <0%> (ø)

src/routes/Catalogs/index.js
0% <0%> (ø)
:arrow_up:

src/routes/Events/index.js
0% <0%> (ø)
:arrow_up:

src/routes/Dataset/components/DatasetRoute.js
0% <0%> (ø)

src/components/App.js
0% <0%> (ø)
:arrow_up:

...s/CatalogHarvest/components/CatalogHarvestRoute.js
0% <0%> (ø)

src/routes/Publication/index.js
0% <0%> (ø)
:arrow_up:

src/routes/Search/components/SearchRoute.js
0% <0%> (ø)

src/routes/Events/components/EventsRoute.js
0% <0%> (ø)

src/routes/Catalogs/components/CatalogsRouter.js
0% <0%> (ø)
:arrow_up:

... and 13 more

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 412864f...78adf59. Read the comment docs.

TODO (reminder): The route components should implement shouldComponentUpdate() { return false; }


Refactor the service loop in order to only wake up the process when it has to perform house-keeping chores.

I implemented something like this a few months ago


[Forge 1.0.2+1.19.2] Epileptic JEP Villagers

Installing Just Enough Professions makes the villagers in the recipe view exhibit less-than normal behavior.
https://github.com/emilyploszaj/emi/assets/43504436/108ad34f-8918-44eb-a427-1c5b6d9c883b

I am somewhat confident that this is an issue with Fresh Animation and JER itself
I had this issue without EMI and now without Fresh Animations I don't have it with EMI

Because of that I don't think this is on EMI to fix, going to close.


Update: dreamus.flo v1.0.8 (#22605)

Resolves #22605
Resolves #22744
Resolves #22947
The url actually installs 1.0.8 for some reason

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.0 schema?

Microsoft Reviewers: Open in CodeFlow

Cached hash since the filename is the same..
Actual hash:   74AA1C28F136A1C2D9BD47C2476F3B64C9E69595D6257DC8AB7AAF49450FF0F7
Expected hash: 3CED2E7E044BE4865B35A866447F06056F2F4036E2A5D6E0AC68746704F0CF28

#23376…


Multiple quality improvements

This pull request is focused on resolving occurrences of Sonar rules squid:S2325 - "private" methods that don't access instance data should be "static"
squid:HiddenFieldCheck - Local variables should not shadow class fields
squid:S1118 - Utility classes should not have public constructors
You can find more information about the issues here:
https://dev.eclipse.org/sonar/coding_rules#q=squid:S2325
https://dev.eclipse.org/sonar/coding_rules#q=squid:HiddenFieldCheck
https://dev.eclipse.org/sonar/coding_rules#q=squid:S1118
Please let me know if you have any questions.
M-Ezzat

In facet, this is done manually, that's after scanning HiveRunner using the SonarQube tool to just facilitate figuring out the issues based on some rules.

You're welcome :)

I don't personally feel any great benefits from these changes. Could you provide some arguments for why these changes are important. I often feel that these code quality rules are generally quite old school.
/Pelle

Generally, it's the best practice to apply these rules, and I don't think that it harms the code, you can also refer to the upper links if you would like, which has more info. Thanks.

Sorry, very sloppy read on my part
/Pelle


Cant jump forward under certain conditions

I can't say it's a bug. Maybe this is intentional behavior.
Steps to reproduce:

press forward
press jump
wait some time to gain speed
rotate camera by 90 degrees
character will jump to right or left but never forward in the direction of camera view

Not sure if I understand how to reproduce. To move you will need to strafe like in the Source Engine. So release W and move A and D in the direction of your camera.

Just tested in half life 2, behaviour is same, so its not a bug. But if you hold space button in half life 2 you jump once, in this fps controller you jumping continuously, this makes the described behavior more visible.

The continuous jumping is intentional, but I should probably make it an option. Would be worth opening another issue for that.


Add flake8 CI, and drop exceptions

We should try to match Matplotlib in flake8 config. The exceptions in Matplotlib are only for backwards-compatibility with existing code and to avoid mass changes, and this code doesn't require any of them, so all exceptions can be dropped.
This depends on #16 and #17, and should not be merged before rebasing.

@QuLogic  ready for rebase.


tradeEnv Issue

May I ask a question? In the end of save_profile() of tradeEnv.py, why we need to reset step_data when (invest_profile['ep'] == self.config.num_epochs), meaning that when it comes to final episode like 50, step_data will always be the detail info of episode_50 and then be saved. Maybe we should save step_data in the IF instead of reset it? I not sure, did I misunderstand something?

I got it, the columns saved each time are different, excellent!!!

Its weird that in the final valid_profile and test_profile, adjacent two episode got the same result after a certain episode(3 in my situation DJIA-MASA-DC). This situation still occurs when I re-executed entrance.py and I haven't found the reason yet. By the way is it convinient for you to chat in Chinese hah. Thank you for you kind reply!!

I got it, the columns saved each time are different, excellent!!!

Good! But please be aware that those results are just for analyzing the model. When reporting the results, it should use the best model validated in the validation set to evaluate and report the performance in the test set. (e.g., 'xxx_policy_validbest' columns in the test_stepdata.csv, or the results in ep='x' in the test_profile.csv and 'x' comes from the ep in the valid_bestmodel.csv.)
Thank you!

  Its weird that in the final valid_profile and test_profile, adjacent two episode got the same result after a certain episode(3 in my situation DJIA-MASA-DC). This situation still occurs when I re-executed entrance.py and I haven't found the reason yet. By the way is it convinient for you to chat in Chinese hah. Thank you for you kind reply!!

It may come from the settings of TD3 in the stable-baseline3. The update frequency of the action network is set to 2 epoches such that you may observe the results updated by the same frequency.
Sure, both Chinese and English work for me. But I may not regularly check github and immediately reply questions since the manpower is limit and I am super busy in these days.
Thank you for your kind attention again!

由衷地感谢，祝你生活愉快！！！

我发现了一个小问题：在tradeEnv.py的save_profile()中（上图），假设X_best_epi表示在当前数据集中表现最佳的epi号，目前的逻辑当valid_best_epi < test_best_epi时，会导致最后保存的test_stepdata中"best"相关的列信息对应的是test集下最佳的epi，而不是valid集下的
因此我进行了简单的修改，或许能够帮助到这一小问题（下图）
另外我的训练过程也有一个奇怪的现象：三种市场下，三种模型（masa-dc,masa-mlp,masa-lstm）中，valid_best_epi经常是2或者49，即模型要么早早地就达到了“最优”或是即使epi完全结束也可能没有达到“最优”，这与我的数据源有关吗（目前没有使用60m级数据，仅使用了1d数据），我还在思考这个问题。
Please feel free to respond at your convenience. I truly appreciate your valuable time. Thank you!

您好，关于第一个问题，test_stepdata的'xxx_best'中储存的是test集下的最优ep，这个只是用于分析性能。而best validated model在test set下的结果是在'xxx_validbest'中，您可以直接查看该列以获得想要的观察数据。
关于第二个问题，这与数据，模型训练等因素有关系，也与金融数据的特殊性质有关，这都可能增加训练难度。您可以尝试增加epoch，更换不同kernel model等去观察。
谢谢！


[CORE] Remove the html reporter from gemini test command.

It looks like Gemini removed the html reporter recently:
https://github.com/gemini-testing/gemini/commit/e2613e4c80f013902f3f8df3a92e0bd37fa1d2c4
I'm not sure if we need/use the html reporter but this PR will allow builds to run gemini tests again until we change the gemini setup.
Signed-off-by: Matt Hippely<EMAIL_ADDRESS>

@jeeyun - I would like to discuss this when you get back to make sure its inline with what you want for gemini.


email: Open a single SMTP connection to send email batches.

Previously the outgoing emails were sent over several SMTP
connections through the EmailSendingWorker. In order to improve
the efficiency of this process the EmailSendingWorker worker is
now defined based on the LoopQueueProcessingWorker which allows to
handle batches of events from the queue workers. This permits to
open a single SMTP connection to send the whole batch of emails.
This was implemented directly in the consume_batch function
inherited from the LoopQueueProcessingWorker's abstract method.
This concrete implementation first creates and opens a new SMTP
connection from the default Django SMTP server then sends each
email of the batch through this connection and finally closes the
connection.
A first retry mechanism is used to redo the opening of the
connection until it works or until three attempts failed. A second
retry mechanism tackles errors that could happen when sending a
particular email from the batch and for which the process is
retriable (e.g. an SMTPServerDisconnected error). If the error
doesn't fall in this category then the particular email is just
dropped (e.g. an SMTPRecipientsRejected).
Help/Feedback required
This second part about the retry mechanisms is trickier than what
I first thought and I must say I am a bit confused about how I
could adapt the "retry_send_email_failures" decorator to be able
to use it in the new consume_batch function. I'm then open to any
suggestions to improve the current workaround that I tried to
implement but that still makes the backend tests fail.
Testing plan:
Automatic tests.
Fixes: #17672.

This is a hard, complicated problem! Thank you for digging into it.
I've left a bunch of comments inline -- let me know if they clarify things, or any further questions that they bring to mind.

Yes indeed at least harder than I thought in the first place but your comments helped me to learn a lot of things. I will update the pull request with the changes I mentioned earlier but without any changes to the tests for now (which might be required). I will implement them later.
You can still comment any of the changes I already did following your previous remarks.
@alexmv

@alexmv I pushed some modifications to account for your second wave of comments and I started to work on the tests but I can't find a way to test this code snippet :
if isinstance(connection, EmailBackend):
        try:
            status = connection.connection.noop()[0]
        except Exception:
            status = -1
        if status != 250:
            # Close and connect again.
            connection.close()
            connection.open()
    return connection

I tried many different possibilities with the mock module but I can't find one that is able to test this correctly. My first problem occurs at the first line : the connection should be of type smtp.EmailBackend to enter the if statement. The best way I found to do this in the tests is to create an object of this class and to mock its methods with patch.object. This is necessary because in the tests any calls to get_connection create a locmem.EmailBackend object and not an smtp.EmailBackend one. Is it an elegant way of doing it ?
My second issue is that I can't find a way to mock the connection.connection.noop() call. When I instantiate the smtp.EmailBackend class its connection is set to None and thus when I try
mock.patch.object(smtp.EmailBackend.connection, "noop", side_effect=Exception) an AttributeError is thrown :
AttributeError: None does not have the attribute 'noop'. This is logical but I can't call smtp.EmailBackend.open() as it uses an internet connection so is there a way to initialize the connection attribute in order to be able to mock noop ?

On the mocking front, I think MagicMock(spec=...) may help.  Here's a quick demo:
from unittest.mock import MagicMock
import smtplib
from django.core.mail.backends.smtp import EmailBackend

c = MagicMock(spec=EmailBackend)
c.connection = MagicMock(spec=smtplib.SMTP)

assert isinstance(c, EmailBackend)

c.open.return_value = False
c.connection.noop.return_value = [250]  # Or give it a side effect, to test the `except`
initialize_connection(c)

assert c.open.call_count == 1
assert c.connection.noop.call_count == 1

You can mock out get_connection to return that, for he case where you call initialize_connection(None).

Thank you once again for your feedback. I updated the changes and implemented the test for initialize_connection. I don't think we need any additional tests as send_email in send_email.py and the EmailSendingWorker class are tested through numerous other test suites that use them in the process.
I remain open to any additional comments if you consider that the test suite I implemented is not sufficient.
@alexmv

Merged, with minor edits, as 9afde790c6a1855de1714378b1ee3e467a7b0fb1!  Thanks for this fix, @pletinckxc!

This has at least one bug: https://chat.zulip.org/#narrow/stream/3-backend/topic/SMTP.20exceptions/near/1170218
I haven't done any further investigation.


Cannot get working with Vue -  Cannot read property 'LocalNotificationsPlugin' of undefined

I can't seem to get this plugin working with vue on playground:
https://play.nativescript.org/?template=play-vue&id=1i7BTX
[Nexus 5X]: 'Error in LocalNotifications.addOnMessageReceivedCallback: TypeError: Cannot read property \'LocalNotificationsPlugin\' of undefined'
[Nexus 5X]: 'NativeScript-Vue has "Vue.config.silent" set to true, to see output logs set it to false.'
[Nexus 5X]: 'Error in LocalNotifications.schedule: TypeError: Cannot read property \'NotificationClickedActivity\' of undefined'
[Nexus 5X]: 'ERROR' [TypeError: Cannot read property 'NotificationClickedActivity' of undefined]

Yeah that's not possible with Playground; this plugin requires an external jarfile which you can't upload.

fair enough :)


Add Nikola Vaptsarov Naval Academy

@dmitov Pull request merged. Thank you!

Awesome! Thank you too!


Update missing CHANGELOG from v0.3.0 to v0.3.5

Found the CHANGELOG is outdated so took some time to added them from the commits: https://github.com/taskrabbit/makara/compare/v0.3.0.rc3...v0.3.5
Please feel free to pointing out if I've missed or misspelled anything, or any preferred format of CHANGELOG. :grin:

I'm using v0.3.0.rc3 in current project and it's yanked recently.. makara has been so great and helpful, and what it does is critical so feeling a bit nervous just hitting bundle update without knowing what's changed. Thanks to all the hard works from the author and other contributors, hope we could keep this CHANGELOG updated as well :smile:

Thanks!


bip39.c: optional API to avoid static vars, make mnemonic_generate optional

As discussed in #136, this creates new entry points where the output buffer is provided as a pointer to caller's memory. I've also included compatibility code that matches the operation of the old code.
The new function names have _p suffix. The version with internal static memory retains the old name.
mnemonic_from_data_p()
mnemonic_from_data()
mnemonic_from_data_indexes()
mnemonic_from_data_indexes_p()

I didn't follow through with corresponding changes to mnemonic_generate() so I made it optional.
This change saves 272 bytes of RAM, if you use the _p versions and avoid linking the other versions of these functions.

This is not worth complicating the design.


This does not reflect the current Windows 10 theme. If Windows 10 theme is set to Dark it should be also set to Dark in Edge, so when a websites implements Auto Theme feature it reflects the current Windows 10 theme. This works fine in Google Chrome, but not in Edge

Edit: This post was labelled "needs more info" so I made a short GIF to showcase what I mean, please have a look at it:

You see, this is the chrome browser. When I change the system theme from dark to light or the opposite, the google chrome browser also changes its theme reflecting the system theme. In my blog, I implemented a feature called "Auto Theme" that reflects the system theme using CSS media query "prefers-color-scheme". This works fine in Chrome but not Edge. The browser Edge does not reflects the system theme whenever it changes like Chrome does.

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

ID: fc385dd2-fb44-9553-7350-9813d104b3dc
Version Independent ID: a3db4e1e-87ce-d35c-435a-8ff4a59d4dd3
Content: Force Microsoft Edge DevTools Into Color Scheme Preview Mode (CSS Prefers Color Scheme) - Microsoft Edge Development
Content Source: microsoft-edge/devtools-guide-chromium/accessibility/preferred-color-scheme-simulation.md
Product: microsoft-edge
Technology: devtools
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

Any updates on this?

Hi @abbasaleid - This should work as expected in current versions of Microsoft Edge. Can you confirm that the "Default Theme" is set to "System Default" in Microsoft Edge (Settings -> Appearance)? Note that you may need to reload the page to see the dark theme take effect.
If this still doesn't work as expected, can you file a browser bug via the "Send Feedback" menu item in Microsoft Edge (preferably on the site where the issue reproduces)? That will help our platform team investigate further.
Thanks!

Hi @abbasaleid - This should work as expected in current versions of Microsoft Edge. Can you confirm that the "Default Theme" is set to "System Default" in Microsoft Edge (Settings -> Appearance)? Note that you may need to reload the page to see the dark theme take effect.

This does work! I think Edge should be set to "System Default" from install. Anyway, thank you @kypflug.


[feature request] define an input file and map files/rules/severity

It would be most useful a to have an optional input file that would be used to help configure the test to be run.
This file could be in the same folder as the 'azuredeploy.json' and if detected, would help configure how test are run.
For a given file,  list which tests should be "skipped",  or have the option of reporting a test as warning instead of an error.
`
[
{
" file": "filenameA.json"
"skipTests: "rule1, rule2, ... "
},
{
"file": "filenameA.json"
"testAsWarning: "rule1, rule2, ... "
}
}
`

are you finding that tests are "inaccurate" for some files?  or just want to have the option to be "less strict" for certain files?

One of the example that comes to mind;  is the rule about "empty" object;  sometime you would want to skip that because you want the deployment to "reset" the values.
Another example is is for a logic apps, some values are user defined, not schema related, then it would simply easier to skip a rule check for that file, or just leave, but report it as a warning instead of blocking the deployment pipeline.
Also had some weird cases with apiVersion that were still in preview but were not "cached" yet;  or the opposite, dont want to use the preview;  and the last api dates from 2016.
There are many more....
fyi:  I have integrated the arm-ttk as part of our pipelines at work;  for now it`s on an "opt-in" basis;  but we have at least 5 pipelines using them now.    I am pushing to get all of them... 50+ projects.

🥇 I can tell you're giving this stuff a lot of thought (these never seem to be simple conversations)...  Anyway.
So far this sounds like a way to configure certain tests... take for example the empty property test, that has exceptions but what if you want your own list?  And then your scenario sounds more complex that says maybe for a given resource type in a given file this is the test I want?
That would be pretty complex to do generically, but perhaps for each test there is a way to deliver something that scales.
re: The apiVersion test, I thought we had a provision in that the required only the latest non-preview version.  Or are you saying that you may want that just for a given resource type?
Good the hear the adoption is picking up steam...

Going to close this one so we can track in #309


Show deprecated warning for current commands

Commands to show deprecate warning:-

[x] player state
[x] player toggle
[x] track info
[x] track skip
[x] track replay

Done in 0.3.0


PubSub support

As I currently understand, this feature is discussed in #176 and might be get implemented in the future.
But in the mean time, is there a support for events (like in pg module: https://github.com/brianc/node-postgres/wiki/Client) or does Massive expose the client (pg client)? Sorry I couldn't find in the docs.
Best Regards :)

@deoxen0n2: check out: https://github.com/posix4e/jsoncdc and: https://github.com/JarvusInnovations/
What is your use case? Is it full-blown publish/subscribe that you are looking for? (or would you just like to attach event handlers to tables and stream changes sequentially?)
Do you need at least once delivery? How about in order? There are a lot of options depending on your requirements.

I might recommend you look into EventEmitter and pub/sub your own events maybe? We used to have events in there but they didn't make much sense.

My use case is streaming changes to clients via WebSocket (say, incoming new order or existing order is updated). So my plan is to use NOTIFY query on Postgres side when the new row is inserted or the existing row is updated and then streaming those changes via WebSocket.
I'm very new on PostgreSQL so apologize in advance if I missed something.

No apologies needed :). You can do LISTEN/NOTIFY with an EventEmitter but... that's a long way round. Are you thinking of using socket.io or something?
Another idea: you could use Firebase and keep things ultra simple :).
Anyway: yeah you might need to use node-pg directly for this.

OK then, I think I would go with node-pg directly for this functionality. Thanks you both of you and nice module by the way :)


Install fails

I followed the install instructions, but received an error about installing potrace.
I followed the instructions here: https://github.com/bsdz/pypotrace. Then the install worked. I would add this to the README 😃
System/Versions
OS: MacOSX Big Sur
Python: 3.7

Ah yes sorry about that. I should have made that clear. As mentioned in another issue I am archiving project so not much point in adjusting the README now.


Refresh button

Hi,
Could you add a refresh button similar to how Muximux(https://github.com/Tenzinn3/Managethis) has integrated to their dashboard?
Currently if I want to reload a just one tab, I would have to hit the refresh button on the browser, which will take me to the default tab. It would be great if you could implement a refresh button that refreshes only the selected frame/tab.
Overall great dashboard, keep it up. Looking forward to what more you could add it, if you need any suggestions let me know.

Double click the tab name.  Would love something like multiple user support.
Thanks for getting back to me.

Yea, i'm planning more stuff, i just need to find more time for it.  Hopefully soon, keep the suggestions coming :)


[BUG] url was provided more than once, but cannot be used multiple times

Describe the bug
Seems to be a parsing issue when reading in the command line arguments
To Reproduce
Steps to reproduce the behavior:
The following command triggers this
feroxbuster --thorough --parallel 2 -o ferox.txt -v --url http://<IP_ADDRESS>:8080/
Expected behavior
A clear and concise description of what you expected to happen.
Traceback / Error Output
feroxbuster --thorough --parallel 2 -o ferox.txt -v --url http://<IP_ADDRESS>:8080/
error: The argument '--url <URL>' was provided more than once, but cannot be used multiple times

USAGE:
    feroxbuster [OPTIONS]

For more information try --help

Environment (please complete the following information):
# Version
feroxbuster -V
feroxbuster 2.7.0

# OS
lsb_release -a
Distributor ID: Kali
Description:    Kali GNU/Linux Rolling
Release:        2022.2
Codename:       kali-rolling

Additional context
Just going off the help output, it does not seem indicate any specific positional order to passing in arguments. If there is a desired or required order then the help screen should properly show the format of the expected invocation

This appears specifically related to specifying the --parallel option, as reording the parameters resulted in the same message. Removing --parallel ultimately allowed the scan to run.

hey there, thanks for the report! Fortunately, this isn't a bug.
From the docs:

--parallel spawns a child process for each target passed in over stdin (recursive directories are still async within each child process).

The normal invocation of a parallel scan is something like this:
cat large-target-list | ./feroxbuster --stdin --parallel 10 --extract-links --auto-bail
Which would produce a process list something like this:
feroxbuster --stdin --parallel 10
 \_ feroxbuster --silent --extract-links --auto-bail -u https://target-one
 \_ feroxbuster --silent --extract-links --auto-bail -u https://target-two
 \_ feroxbuster --silent --extract-links --auto-bail -u https://target-three
 \_ ...
 \_ feroxbuster --silent --extract-links --auto-bail -u https://target-ten

If you want to control the amount of threads / speed while scanning a single url (which appears to be what you were doing given the command you listed), you would use -t|--threads

Thanks, that makes sense. Love the tool!


[2.0-wip] new mini-button style

Tiil now there exists a couple of different sizes of buttons. If you want to use buttons in tables, all the buttons are stretching the table, so it doesn't look good.
If you are using the small buttons it looks like this: http://www.abload.de/img/button.small4pp3n.png
Adding a new button class, something like "mini" would allow to use buttons even in tables.
I copied the mini declaration and edited the padding to "1px 4px;"
Using this it would look like: http://www.abload.de/img/button.minij7rfj.png
Sure the use of it would be limited and you should only use it, when you need it, but i think it would complement bootstrap.

+1

Closing for #1400.


training problem with imbalanced data

Hi,
Have you ever faced a training problem with imbalanced data? For example, when the labels for fetch latency are mostly 0 or 1, and the range of labels spans from 0 to over 1000. Some labels appear frequently, while most others are very rare. Additionally, the distribution can vary significantly across different workloads or even within different parts of the same workload.
When I try to train a model, it often fails to converge—the loss value stabilizes at a certain point and doesn't decrease further. I've tried several tricks, such as label smoothing, loss re-weighting, and combining regression with classification, but none of them have worked.
I'm not very experienced in model training and don't know where to go. Could you give me some suggestions?
Thank you!

Thanks for your interest to our work.
Imbalance distribution of latencies should be common in the training data. It didn't necessarily create a problem in our training. Nevertheless it could potentially result in better models if you manage to improve it.
About loss convergence, there are many possible causes. We observed loss stablizes too and it does not necessarily mean the trained model is not accuarte if you test it.

Thanks for your reply!
I successfully reproduced your code by following your tutorial.
I tried training "InsLSTM(256, 2, narchs=1, bias=False)" from scratch using training data selected from the gemm60x70x80 program, which contains 1,551,224 execution instructions.
Config file:
input_length = 51
tgt_length = 1
cfg_num = 1
seq_length = 256
ori_tgt_length = 16

def sel_batch_out(y):
  y = y.reshape((-1, cfg_num, ori_tgt_length))
  y = y[:, :, 0] # fetch latency
  return y

Training command:
python -m ML.train --cfg=com_shixl --train-size=1550848 --epochs=200 --batch-size=2048 --sbatch "InsLSTM(256,2,narchs=1,bias=False)" --lr=0.001 --lr-step=50

The model get best checkpoint at epoch 20 with Training Loss 295.509224, Test Loss 173.282750
Sim result:
...
Test loss: 315.740453   Time: 43.9 s
(1536000, 1, 1)
Config 0
0 :
        target: [800.   5.   0. ...   5.   0.   5.]
        norm output: [440.   1.   9. ...   3.  -1.   4.]
        errors: [360.   4.  -9. ...   2.   1.   1.]
        Error abs avg, norm abs avg, norm avg, and norm std: 5.360873    1.9813309       -1.5350392      5.9195485
...
Run 1551224 instructions.
trace
Use the maximum size of 3029 instead of 3030
Target: tensor([[11909115.]])
Output: tensor([[11659610.]])
Error: tensor([[-0.0210]])
Mean error: tensor([0.0210])
Mean normalized error: tensor([0.0210])
Loss: 0.104162  Time: 40.8

The result is remarkable! However, I still have some confusion:

Is the training loss acceptable, or could the issue be caused by the dataset size or insufficient epochs?
I got an absolute average error value of 5.36, but when summing the latency of each instruction, the error decreased to 0.021 per instruction. This indicates that the errors for individual instructions may have both positive and negative values, which mostly cancel out during the summation process. Is this a coincidence, or is it an inherent effect of the model?

For your questions:

It's hard to tell what loss is acceptable or not as it depends on many factors (e.g., batch size, dataset).
These errors have different units. 4.909 is for instruction latency, and 0.0084 is the percentage error for the whole program. So they are not directly comparable. We do expect instruction errors to cancel out each other at the program level to reach better accuracy.

@DF-erlian I am instrested in the performance modeling and predicting. It semms that you already run the project well, Do you know how to tune the configurations of arm platform when doing performance predicting?
The project is so nice. If you are free, please reply. Thanks. @lingda-li

I reproduced the code by following the markdown files under Tutorials, such as gem5.md for generating input traces, learn_rep.md for obtaining instruction representations, predict.md for generating simulation results, and train.md for training a model from scratch.

I reproduced the code by following the markdown files under Tutorials, such as gem5.md for generating input traces, learn_rep.md for obtaining instruction representations, predict.md for generating simulation results, and train.md for training a model from scratch.

Yes, i also reprodeced the steps successfully except train.d as my GPU server is not ready yet. But, did you know how to tune parameters of platform such as the size of L3 cache?

Chapter VI of the author's paper "Learning Generalizable Program and Architecture Representations for Performance Modeling" may be helpful, which introduces the applications of PrefVec.

@liuqinfei You can change the configuration in gem5. PerfVec requires a learned microarchitecture representation in that case to make prediciton.


improved xgboost wandb_callback

https://wandb.atlassian.net/browse/WB-NNNN
https://wandb.atlassian.net/browse/CLI-NNNN
https://github.com/wandb/client/issues/NNNN
Description
What does the PR do?
Testing
How was this PR tested?
Release Notes
Below, please enter user-facing release notes as one or more bullet points.
If your change is not user-visible, write NO RELEASE NOTES instead, with no bullet points.
------------- BEGIN RELEASE NOTES ------------------
------------- END RELEASE NOTES --------------------

Here's an example dashboard: https://wandb.ai/ayush-thakur/xgboost-housing-demo

@morganmcg1

Maybe we need to keep the existing code and add a warning that it will be deprecated and they should use the new callback

Added UserWarning that it will be deprecated. One user warning is coming from our side, the other one is XGBoost's.

Maybe call the new callback WandbCallback to differentiate from wandb_callback

I have named the new callback XGBoostCallback for now. If you want to use WandbCallback for consistency I will put that.

Also a colab to compare old and new callbacks: https://colab.research.google.com/drive/1dvAQmqkLP51IP8AXaqzDdbdYh7ruOQ5Z?usp=sharing

Lets keep the name as WandbCallback as it aligns with all of our other integrations callbacks


Simple NIO tests

Make sure we find the provider and it recognizes paths as expected.
These tests don't actually connect to any Google Cloud Storage
bucket.

now in a separate repo


south-africa/module6

https://github.com/school-of-data/GIS-curriculum/tree/main/south-africa/module6
Missing media

many-polygons.png

missing media (add new image)

example: https://github.com/school-of-data/GIS-curriculum/blob/main/ethiopia/module6/media/many-polygons.png

selected-canvas.png

missing media (add new image)

example: https://github.com/school-of-data/GIS-curriculum/blob/main/ethiopia/module6/media/selected-canvas.png

Update

https://github.com/school-of-data/GIS-curriculum/blob/0cc64b893d4d4380c4d7f4c1a2d9c3ca186008b3/south-africa/module6/module6.md#L29-L32

replace with GDrive link
see: https://github.com/school-of-data/GIS-curriculum/blob/main/south-africa/module6/module6.md?plain=1#L29-L32

https://github.com/school-of-data/GIS-curriculum/blob/0cc64b893d4d4380c4d7f4c1a2d9c3ca186008b3/south-africa/module6/module6.md#L113

replace with GDrive link
see: https://github.com/school-of-data/GIS-curriculum/blob/main/south-africa/module6/module6.md?plain=1#L113

https://github.com/school-of-data/GIS-curriculum/blob/0cc64b893d4d4380c4d7f4c1a2d9c3ca186008b3/south-africa/module6/module6.md#L185-L188

replace with GDrive link
see: https://github.com/school-of-data/GIS-curriculum/blob/main/south-africa/module6/module6.md?plain=1#L185-L188

https://github.com/school-of-data/GIS-curriculum/blob/0cc64b893d4d4380c4d7f4c1a2d9c3ca186008b3/south-africa/module6/module6.md#L241-L242

outdated link, replace link to 2.18 with link to 3.16
example: https://github.com/school-of-data/GIS-curriculum/blob/main/ethiopia/module6/module6.md?plain=1#L240-L241

Localize

https://github.com/school-of-data/GIS-curriculum/blob/0cc64b893d4d4380c4d7f4c1a2d9c3ca186008b3/south-africa/module6/module6.md#L113

replace with correct locale (says PHL provinces)

Commit links:

https://github.com/school-of-data/GIS-curriculum/commit/44935a20e3a5951c014b58c3dbe1e7e93010d1b6
https://github.com/school-of-data/GIS-curriculum/commit/2419999a9cc4c953dfc8930630c1ea61885d0ca0


Fix glibc locales on cross endian

Description of changes
When cross-compiling between systems which have different endianness, the locale-archive from glibcLocales is unusable since it follows the host system's endianness.
cc @shlevy if you see any drawback of undoing your commit aebb024b2f35d826f99d3e97caeff8522ff6e3c7
Not sure if I should target staging but since it basically will trigger rebuilds of all NixOS systems...
Currently tested only by comparing the hexdump of ${glibcLocales}/lib/locales/locale-archive and  ${glibc}/lib/locales/locale-archive.
Things done

Built on platform(s)

[x] x86_64-linux
[x] cross x86_64-linux -> ppc64
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Not sure if I should target staging but since it basically will trigger rebuilds of all NixOS systems...

If it would be only that then it would be totally fine but we also trigger 5000+ rebuilds so staging must be targeted.

This PR rebuilds a lot of packages which means we must target staging. Please follow the contributing guide to not potentially ping a lot of people.

Yes, I thought packages would depend only on glibc since it has a locale archive, and not glibcLocales but I was wrong!

In general diff looks ok, but I don't really like that sed call being abused to add 'make flags'. Maybe it could be split out somehow?

@Mindavi let me know what you think of this method. The drawback is that I couldn't use the $TMPDIR variable anymore in the localedef arguments.

Tested with m68k too, like @trofi suggested, and after some time cross-compiling, it seems to work:
minijackson@machine ~ % LOCALE_ARCHIVE_2_27=/nix/store/...-glibc-locales-m68k-unknown-linux-gnu-2.35-163/lib/locale/locale-archive  qemu-m68k /nix/store/...-glibc-m68k-unknown-linux-gnu-2.35-163-bin/bin/locale
LANG=en_US.UTF-8
LC_CTYPE="en_US.UTF-8"
LC_NUMERIC="en_US.UTF-8"
LC_TIME=en_DK.UTF-8
LC_COLLATE="en_US.UTF-8"
LC_MONETARY="en_US.UTF-8"
LC_MESSAGES="en_US.UTF-8"
LC_PAPER="en_US.UTF-8"
LC_NAME="en_US.UTF-8"
LC_ADDRESS="en_US.UTF-8"
LC_TELEPHONE="en_US.UTF-8"
LC_MEASUREMENT="en_US.UTF-8"
LC_IDENTIFICATION="en_US.UTF-8"
LC_ALL=

minijackson@machine ~ % LOCALE_ARCHIVE_2_27=/nix/store/...-glibc-locales-m68k-unknown-linux-gnu-2.35-163/lib/locale/locale-archive  qemu-m68k /nix/store/...-glibc-m68k-unknown-linux-gnu-2.35-163-bin/bin/locale -a
# ...
# Lots of locales
# ...

:tada:


cli: change stacks deploy helper to wait on deployment state

Fixes #2523
The stack deployment is not guaranteed to ever see any service_deploys, if it goes into an error state before starting those (raise from remove_services). This changes wait_for_deployment_to_start to just expect the state to change from created. The first thing the server StackDeployWorker does is update the state to ongoing.
This means that the Waiting for deployment to start... phase is now shorter, and the wait_for_deploy_to_finish has to wait for the first service deploy to start. I think it can deal with that as-is?

NTS: un-skip the e2e spec in #2524 after this is merged

The broken #2524 is now fixed:
stack upgrade
  for a stack that is linked to externally
 [done] Creating stack links-external-linked
 [done] Triggering deployment of stack links-external-linked
 [done] Waiting for deployment to start
 [done] Deploying service foo
 [done] Deploying service bar
 [done] Reading stack links-external-linked metadata from Kontena Master
 [done] Upgrading stack links-external-linked
 [done] Creating external-linking-service service
 [done] Triggering deployment of stack links-external-linked
 [fail] Waiting for deployment to start
 [error] RuntimeError : deploy failed
         See /root/.kontena/kontena.log or run the command again with environment DEBUG=true set to see the full exception
 [done] Removing service external-linking-service
 [done] Removing stack links-external-linked
    fails to deploy if linked


Captive Portal not redirecting to login page

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x ] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x ] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
Captive Portal has different problems on different devices:

Windows machines - Network is connected with right parameters but either no action to login is displayed or any redirect occurs when trying to navigate, as a result, users can't provide credentials to surf the web.
Android devices - We made lot of tests and figured out that on certain models, things go well, while on others, a notification to authenticate to Captive Portal only happpens the first time.  If that android phone disconnects the network and re connect, it will never asked to login to Captive Portal, as a result users are unable to surf the web as mentioned at point 1.

Already searched online and figured out that is not an https redirect problem, because that happen also when trying to search over http.
Followed this tutorial to setup the Captive Portal: https://docs.opnsense.org/manual/how-tos/guestnet.html
To Reproduce
Steps to reproduce the behavior:

Use this tutorial to setup the Captive Portal: https://docs.opnsense.org/manual/how-tos/guestnet.html

Expected behavior
I expect to get a notification everytime I connect to CaptivePortal that brings me to Login Page or get redirected to the Login Page if session is not authenticated yet.
Environment
OPNsense 21.7.7-amd64, OpenSSL.
Intel(R) Xeon(R) Gold 5120 CPU @ 2.20GHz (4 cores)

I have that problem, too. can't automatically enter login page
can access login page with  enter manually gateway address with port 8000

I have this problem as well

hmmm... this is not good, i have recognized the same problem as well! i'm not sure on which point this happens, any suggestion to debug this problem?

i have that problem too

Same issue still

Ditto here. Can't really figure out why it wouldn't pop up with the captive portal screen. It used to work but stopped when an update got installed a few months back. I'm not using it often but when I have guests over I would like them to use that functionality.


Delete rows

i did not triggered any delete statement explicity from dbeaver when connected postgres dba i triggered only select statements but when i see few of rows deleted from table and it is showing records are deleted with my userid. is it anyway to delete records through dbeaver without passing delete statement

Please provide more information.
You can delete data not only in SQL editor (by running DELETE FROM queries) but also in data editor by deleting rows from UI and then saving changes to a database.

Actually explictly i did not passed delete statement from aql editor.is any
possible ways by accidentally deleting data by unknown short cuts in dbeaver
On Thu, 28 Mar 2019, 21:28 Serge Rider<EMAIL_ADDRESS>wrote:

Please provide more information.
You can delete data not only in SQL editor (by running DELETE FROM
queries) but also in data editor by deleting rows from UI and then saving
changes to a database.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dbeaver/dbeaver/issues/5597#issuecomment-477658258,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Auu7_5DSeRt5R0Mi8EG_-qZA0nL33Q6gks5vbOaTgaJpZM4cOrO6
.


Duplicate requests in development through AsyncConnect

I'm noticing that when I dispatch my http requests inside of asyncConnect, they calls are duplicated when in development mode.  This does not happen when I compile and run for production.

Components also seems to be getting mounted twice.  I put a console.log inside of component will mount (that only logs if __CLIENT__ and it gets executed twice.


Should we allow TSDataset to hold non-continuous series?

Checklist

[X] I have searched the issues.
[X] I have searched the docs.

❓ Questions and Help
Some tasks require us to delete some time periods from dataset (for example, the beginning of COVID in 2020). Should we allow our users to upload datasets with missing values as well?

Will work on it in the separate track


fix: ci config

fix: ci config
Description
update ci config

Codecov Report

Merging #28 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #28   +/-   ##
=======================================
  Coverage   53.32%   53.32%
=======================================
  Files         123      123
  Lines        1652     1652
=======================================
  Hits          881      881
  Misses        771      771

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 2b1cd36...46126ca. Read the comment docs.


Add dark mode support

Even though the editor window's CSS can be customized (to be documented according to issue #20), we should add a simple way to switch the editor window's color to a dark theme.
Maybe this could be done by adding an options parameter to the constructor for MJXGUI which will take a JSON object containing key value pairs for customizing the MJXGUI editor's behavior, and one of the supported keys could be theme.
This fix would need a major version bump.

Hey @hrushikeshrv This is Tej from Discord. I would like to work on this. I have gone through the issue.

Great, thanks! I'll assign this to you.

Is this feature still under development or can I try my hand at it?

Is this feature still under development or can I try my hand at it?

Hi, I’m working on it.

Hi @uttej-vsk, what approach are you taking to fix this issue? Since #29 was just fixed, the CSS for the editor is now included separately in src/mjxgui.css. Now users will have to link the CSS file on their webpage separately, and the CSS is no longer hardcoded in the ui.js file like it used to be.
To add support for dark mode, IMO we should modify the new src/mjxgui.css file with the relevant CSS for dark colors, and maybe define variables in the CSS as well.


Dialog component

@JaredBourget Did you want me to merge this before or after yours? It will have conflicts.

@chellimiller You can go ahead and merge it - the only thing holding me back is a couple of tests... I'm going to tackle a different issue and then circle back


Add LIBRARY_TASK_MAPPING_EXCLUDING_TRANSFORMERS

Context:  https://huggingface.slack.com/archives/C02ERUY9TKK/p1669743642830359

Thanks Omar for the detailed review!


Add OCP on OSP scaleup into scale.yml playbook

Currently the scaleup procedure will work for AWS because of its implementation with 3 availability zones, when we deploy OCP on OSP we have a single availability zone so the scripts need to be modified to allow this to work in both environments.  Furthermore it will need to be review/checked with OCP on Azure.

#80 handles this.


tests/usbredir_test.go should be decoupled from virtctl

What happened:
The test code in tests/usbredir_test.go relies on kubevirt.io/kubevirt/pkg/virtctl/usbredir to test the USBRedir API of KubeVirt.
What you expected to happen:
The test code in tests/usbredir_test.go should not rely on any virtctl code to test the USBRedir API of KubeVirt. If the virtctl usbredir command is tested in functional test, the tests should live in tests/virtctl/usbredir.go and should use newRepeatableVirtctlCommand as entry point.
How to reproduce it (as minimally and precisely as possible):
Look at tests/usbredir_test.go, the file uses usbredir.NewUSBRedirClient.
Additional context:
Add any other context about the problem here.
Environment:

KubeVirt version (use virtctl version): N/A
Kubernetes version (use kubectl version): N/A
VM or VMI specifications: N/A
Cloud provider or hardware configuration: N/A
OS (e.g. from /etc/os-release): N/A
Kernel (e.g. uname -a): N/A
Install tools: N/A
Others: N/A

/cc @victortoso

/assign
I'll work on it early January.

The test code in tests/usbredir_test.go should not rely on any virtctl code to test the USBRedir API of KubeVirt

The idea behind it was to have both virtctl+api+severside being tested and the changes I'll propose will remove virtctl from it. That means some code duplication. Also, it would potentially remove the need of https://github.com/kubevirt/kubevirt/pull/12977
Happy holidays.


Gauss Newton with Line Search

TODOs

[ ] Fix Tests
[ ] Wait on https://github.com/SciML/NonlinearSolve.jl/pull/265
[ ] Upstream patch to allow non-square function operators for VecJac https://github.com/JuliaDiff/SparseDiffTools.jl/pull/272
[ ] Remove Manifest

Any reason to not use LineSearches.jl?

It is using LineSearches internally

This is done. just need the previous PRs to be merged

would it be possible to pass an initial guess for alpha and line search? example? InitialStatic(), InitialPrevious(), etc...

would it be possible to pass an initial guess for alpha and line search? example? InitialStatic(), InitialPrevious(), etc...

Not currently. I have no clue how that API currently works.


Known Issue appears to still be active

[Enter feedback here]
Encountered the initial issue described here on 3/17/23 with a report that what previously available, removed from a workspace, and re-uploaded to the same space.
Manual regrants of access described in the workaround are not consistently successful, and some share emails are not being received. In most cases, granting and then removing build permissions was what appeared to resolve access concerns. RLS for the report now also appears non-functional.

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

ID: bead40a7-0f26-dc4b-0e4f-9427dd1296c2
Version Independent ID: bead40a7-0f26-dc4b-0e4f-9427dd1296c2
Content: Known issue - Accessing Power BI app report is denied due to no dataset access - Power BI
Content Source: powerbi-docs/troubleshoot/known-issues/known-issue-284-accessing-app-report-denied-due-to-no-dataset-access.md
Service: powerbi
Sub-service: pbi-troubleshooting
GitHub Login: @mihart
Microsoft Alias: mihart

Thanks for your comment, @mmBPCS. This issue has been assigned to the doc author for review.

Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. Both the product and documentation has changed since this issue was opened. Therefore, we are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.


Allow 'etcdctl' and 'fleetctl' to work on the worker nodes according to CoreOS Docs

As it is now, etcdctl and fleetctl to not work properly on the worker nodes since they don't know how to find the master node's etcd.  Following instructions in the CoreOS docs I was able to get this to work on the worker nodes with the addition of a few environment variables.  Perhaps you can bake this into your (very helpful) tool for everyone?
The documentation for this was found in the 'Easy Development/Testing Cluster' section of this page:
https://coreos.com/docs/cluster-management/setup/cluster-architectures/
Based on that here are the env vars I manually added on each worker node to get things to (so far) work for me:
export ETCDCTL_PEERS="http://<IP_ADDRESS>:4001"
export FLEETCTL_ENDPOINT=unix:///var/run/fleet.sock
export FLEETCTL_EXPERIMENTAL_API=true

After this I am able to run any fleetctl or etcdctl commands on the nodes without seeing errors like:
core@k8snode-01 /run/systemd/system $ fleetctl list-machines
2015/03/15 22:50:33 INFO client.go:291: Failed getting response from http://<IP_ADDRESS>:4001/: dial tcp <IP_ADDRESS>:4001: connection refused
2015/03/15 22:50:33 ERROR client.go:213: Unable to get result for {Get /_coreos.com/fleet/machines}, retrying in 100ms
2015/03/15 22:50:33 INFO client.go:291: Failed getting response from http://<IP_ADDRESS>:4001/: dial tcp <IP_ADDRESS>:4001: connection refused
2015/03/15 22:50:33 ERROR client.go:213: Unable to get result for {Get /_coreos.com/fleet/machines}, retrying in 200ms
...

For a more permanent solution I also followed the directions in that documentation to add the 'write_files' section to automatically add this config to every worker machine.  Here is a working gist (see the end of the file for the changes):
https://gist.github.com/grempe/92704b8bdecf19530607

@grempe sure we can include that etcdctl and fleetctl from workers can connect to control peer.
I have set that in my development and production clusters but which are not k8s based yet :-)
But the changed will not make any harm to have set it for k8s clusters.
And yes this one https://coreos.com/docs/cluster-management/setup/cluster-architectures/ must be read before setting up any CoreOS cluster :-) as I see many bad examples setting CoreOS clusters where all cluster peers are part of etcd cluster :)

PR merged and released

Agreed, I am relatively new to CoreOS and that document was very helpful. Initially I was thinking that there was an etcd cluster between your three VM's and I was trying to get that to work (and fun with discovery URL's etc).  That document helped me understand the no-cluster approach you were taking in this tool (and how to get everything to work as expected). The no-cluster approach makes a lot of sense for a dev env where you don't want to be hunting down discovery URL's every time you re-launch.
Good learning experience for me troubleshooting it.
Cheers.

I got bitten badly setting up many CoreOS clusters for production and having problems then.
When I moved everything to 3x control/etcd based cluster (GCE g1-small instances) + many workers, there I have grouped servers by required work, everything became so easy to manage and upgrade the cluster.
E.g even for my development I use just one control/etcd node (GCE g1-small) and 5 workers and it works fantastic too.


Point MariaDB to MySQL install framework flow

Summary
For the MariaDB's quickstart, the new MySQL install framework flow is shown.

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


Add touch

Need a comment on how to implement tests better, because this pr will result 200ms delay in tests since it's testing if it changes the access and modification time of an existing file. Also is using os.Remove() okay to remove files after each test? Or file system should be mocked somehow?

Thank you for the PR @wralith


Inconsistent URLs for "tools" vs. "tests"

There is inconsistency with our URLs for tools and tests. The tab is labeled tests but the URL for NDT is:
https://www.measurementlab.net/**tools**/ndt/
But then the URL for the index of all tests is:
https://www.measurementlab.net/**tests**/
Suggest changing all URLs to tests and adding redirects. At the very least, we should be consistent.

+1

+1 We've already updated text (on the FAQ page) standardizing on tests

How are we redirecting URLs we used to provide to their new ones? ie: redirecting /tools/ndt to /tests/ndt  ?

I believe we can just use a Jekyll plugin add metadata to the page:
https://github.com/jekyll/jekyll-redirect-from

We also need to add a redirect from /measurement-lab-tools. It's our third most popular landing page according to Google Analytics and it currently 404s.

@georgiamoon I've attempted to test redirects using the jekyll-redirect-from plugin but it doesn't seem to work for me. It generates the page (for example, when I add redirect_from: "/measurement-lab-tools/"  to the front matter of _pages/tools/03-tests.md a page is created when the site gets built: _site/measurement-lab-tools/index.html but the refresh meta tag isn't populated with a URL. The result is a page trying to constantly refresh to a blank location.
I've done a workaround in my branch, using the method we used to redirect to the viz site, that is adding a redirect_to: variable to the front matter of a new page: _pages/measurement-lab-tools/index.md  which you can preview.
I'm wondering if this solution is sufficient or if someone else can get the redirect-from plugin working right we could use it. In this case, to add redirects, we would simply add a stub page like _pages/measurement-lab-tools/index.md for any page that needed redirecting, or for 404s we see a lot of, etc.
So for example, if we were to use this method to redirect pages like the test pages currently in /tools/ we would make the folder _site/tests/, copy all the test pages from _site/tools/* into it, change their permalinks, and add the redirect-to:  URLs to all the old pages in _site/tools/

Tagging @nkinkade in here too to take a look

Tagging @nkinkade in here too to take a look

Pinging this thread with an update on redirects. We can do redirects in the way we've done it for the new visualization, redirecting from this page: https://github.com/m-lab/m-lab.github.io/blob/master/_pages/02-visualizations.md  to the viz.measure... site. I believe you can also do the redirect in AWS, though that is less straightforward.

I've documented how to do the redirect in AWS here: https://github.com/m-lab/m-lab.github.io/issues/65#issuecomment-314542474
Two ways to do redirects actually. First, is to have a page like https://github.com/m-lab/m-lab.github.io/blob/master/_pages/02-visualizations.md which specifies an external URL to redirect to in the YAML front matter. The second is described in the link at the top of this comment, which is to upload a zero-byte file in the location where your redirect is needed, and then add the redirect metadata item to the object in S3.

Resolved with 301 redirects added in AWS.


Implement function to get Context Preview when a Piece is selected

The user should be able to view the contextual information based on the selected snippet.
import * as pieces from<EMAIL_ADDRESS>
async function main() {
    const assetsApi = new pieces.AssetsApi();
    const assetApi = new pieces.AssetApi();

    const indicies = await assetsApi.assetsIdentifiersSnapshot({});

    const firstAssetId = indicies.iterable?.[0].id;

    // this should only happen if u got no assets saved
    if (!firstAssetId) return;

    const asset = await assetApi.assetSnapshot({asset: firstAssetId})

    const websites = asset.websites?.iterable;

    const tags = asset.websites?.iterable;

    console.log('WEBSITES!', JSON.stringify(websites, undefined, 2));

    console.log('TAGS!', JSON.stringify(tags, undefined, 2));
}
main()

@shivay-at-pieces
hey there, iwould like to work on it..


Impossible to change generator binaryTargets

Hi,
I tried Prisma 2 on my Ubuntu(Bionic), but the binaryTargets field in generator doesn't seem to work.
Here's my setting:
generator photon {
  provider = "photonjs"
  binaryTargets = ["debian-openssl-1.1.x"]
}

Here is what it says, when I try to use photon.js in my code:
Error: Photon binary for current platform linux-glibc-libssl1.1.0 could not be found.
Photon looked in null but couldn't find it.
Make sure to adjust the generator configuration in the schema.prisma file:
generator photon {
provider      = "photonjs"
binaryTargets = ["native", "debian-openssl-1.1.x"]
}
Please run prisma2 generate for your changes to take effect.
Note, that by providing `native`, Photon automatically resolves `linux-glibc-libssl1.1

Is it possible to replace the native for another value like "debian-openssl-1.1.x" so I don't get this error?
Am I doing it wrong?

I am unable to reproduce this. Can you please tell me which version of photon are you using. In some parts of the output that you have shared the binary names are out of the date. I faintly remember that we fixed a bug regarding this recently.
I used<EMAIL_ADDRESS>binary version: e7579bd35e0938dbf773f1706c098a0d14a5a038 to reproduce this error.

@Prattjames What version of Photon did you use? The error indeed looks incorrect, it lists linux-glibc-libssl1.1.0 in the content but debian-openssl-1.1.x in the fix suggestion.
linux-glibc-libssl1.1.0 was the old name of the binary and shouldn't exist anymore. Maybe you are using a newer CLI version in conjunction with an older Photon version?
Can you please confirm?

Also, how to synchronize CLI version and Photon version ?

The CLI should warn you if the versions are out of sync. You can bring them to the same version manually via
yarn add -D prisma2
yarn add -D @prisma/photon

These commands seems to resolve the initial pb. Thanks.
Now I have an other error looks like i didn't generate schema, even i have. ("no such table: dev.mytable"). But it's an other subject.

Sure, please feel free to close this issue and open a new issue for that one. Thanks.


[Sprint: 62] XD-3690 Updating Database Configuration section and sink/source/job docs

LGTM.  Merged as d7d2abfb9ae801e7165902cb57bfa124af6c66e2


What if true K is 1?

Hi! Thanks for the very useful software, hope you will keep developing the project.
My question is - how robust is the TI algorithm implemented in Maverick, when the real K is 1?

Hi,
Thanks, I haven't been back to this software for a while so good to know people are still using it!
The rmaverick method works fine for K=1 (unlike some other methods), and in fact in this situation the model evidence can be computed exactly so it is extremely fast.

In my experience rmaverick outperforms Structure. It is faster and more accurate.
So, I follow the best practice and do hierarchical clustering for my data, but I found the following.
For one group I got this Q-matrix and best K is 2:
     deme1      deme2

ind1  0.9485547 0.05144530
ind2  0.9183966 0.08160337
ind3  0.9498344 0.05016556
ind4  0.9321735 0.06782647
ind5  0.9352606 0.06473937
ind6  0.9562050 0.04379501
ind7  0.8168223 0.18317774
ind8  0.9202123 0.07978767
ind9  0.9397267 0.06027329
ind10 0.9631617 0.03683831
as you see, there is no biological reason to agree with K=2. With "no admixture" model the best K is 1 (which is reasonable), but "admixture" model is strongly supported over "no admixture " model in this analysis. Would be great if you can comment on it.

This is very interesting. Not immediately obvious to me what's going on, I'd have to do a dive into the maths and the data to properly work it out. But one possibility relates to the fact that admixture model sort of conflates two distinct situations - an allocation probability 0.9 because we are confident the individual belongs in that deme at every locus, and a probability 0.9 because we are very confident (almost certain) that 90% of loci belong in deme1 while the remaining 10% we are very confident belong in deme2. Obviously all middle points between these extremes are possible too. So it might be that there is a strong signal of admixture, but the majority of loci are from the non-admixed population. You could look into individual locus-level allocation plots (and probabilities) to explore this.


Fix wind index. Add debug logging. Get it working with the http dep

oops

hej @notnullxyz,
nice! 👍 thank you for taking this into account.
please let me know when this is done. I'm happy to merge this in.

hej @notnullxyz,
nice! 👍 thank you for taking this into account.
please let me know when this is done. I'm happy to merge this in.

Hello @comes
It works, imports and keys correctky.
You could give it a test if you'd like.
Ill try to update the README at some point to mention the --debug as well.


Added mailid at the required place

Issue that this pull request solves
Closes: # (issue number)
Proposed changes
Brief description of what is fixed or changed
Types of changes
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] My changes does not break the current system and it passes all the current test cases.

Screenshots
Please attach the screenshots of the changes made in case of change in user interface
Other information
Any other information that is important to this pull request

Hey @Sloth-Panda you don't need to open up a new PR for the requested changes, pushing on the same branch works


ja: Make docs/concepts/extend-kubernetes/api-extension/custom-resources.md follow v1.18 of the original text

This is a Feature Request

What would you like to be added
Update docs/concepts/extend-kubernetes/api-extension/custom-resources.md to follow v1.18 of the original (English) text.
Why is this needed
content/ja/docs/concepts/extend-kubernetes/api-extension/custom-resources.md is outdated.
Comments
/language ja
/good-first-issue
File to update
https://github.com/kubernetes/website/blob/dev-1.18-ja.1/content/ja/docs/concepts/extend-kubernetes/api-extension/custom-resources.md
Original
https://github.com/kubernetes/website/blob/fb6364d/content/en/docs/concepts/extend-kubernetes/api-extension/custom-resources.md
diff between translated and v1.18
https://gist.github.com/b35d11ab9ce54a059aa5683e509d1684
Note
Currently, we use dev-1.18-ja.1 branch. Please open a PR targeting the branch.
ref
https://kubernetes.io/ja/docs/contribute/localization/
If you have a question, feel free to ask us at slack.k8s.io #kubernetes-docs-ja channel.

/milestone 1.18-ja.1

/assign

/close


slide.addText() with "line.pt" option does not work

Hi,
congrats great lib, just ran into small issue.
using: slide.addText() with option: "option.line.pt: xx" does not work, only "option.lineSize: xx" does the job.
Documentation says: Adding Text
Source Code says: deprecated
thx
Christian

Hi @npgeomap
You ran into this on the cusp of v3.3.0 being released.
Documentation is currently being updated, please consult type defs file for correct properties.


Release v13.14.2

I hope that at least some of the recent release hurdles are not directly caused by the NUKE migration and we can see at least some benefits at some point 😞

Function x wallet.pdf


processor/sinkmanager(ticdc): refine error handle and skip empty event

What problem does this PR solve?

Issue Number: close https://github.com/pingcap/tiflow/issues/7674
What is changed and how it works?
processor/sinkmanager(ticdc): fix error handle bug
processor/sinkmanager(ticdc): fix error filtered events bug
processor/sinkmanager(ticdc): fix data race in event sorter
Check List
Tests

Unit test

Questions
Will it cause performance regression or break compatibility?
No
Do you need to update user documentation, design documentation or monitoring documentation?
No
Release note
None

/merge

/run-all-tests

/run-verify

@ti-chi-bot wake up~


Opensea: Mainnet and Matic store confusion

ASSET collection on Matic (https://matic.opensea.io/) isn't showing the polygon assets we mint, they're getting minted to https://opensea.io/collection/asset-v3 instead.
This is the correct address:
0x1E9dC2903cF07A047eBDdb28B14174eA3A86A775
https://github.com/webaverse/contracts/blob/b09d68a29892c602314f7b808525b1ad5f40db2f/config/addresses.js#L45
Need to poke opensea again
Solving this may depend on Cosmos work with migrations

https://opensea.io/assets/0x42fae5a6dbd93cf63ed9eea0449825ddc8234a17/86
The tokens one is allowed to offer are WETH, DAI, USDC on the things we already minted on mainnet ASSETs. These tokens are what people would use to bid on Matic NFTS.

Naming suggestion: mASSET (matic ASSET)
Moving sidechain -> ETH mainnet = https://opensea.io/collection/asset
Moving sidechain -> Polygon = https://opensea.io/collection/masset

Yeah I think we need to update the contract address. Once migrations are done we can add the Ownable interface to the contract and do it one more time.


Display issue - conflicting with Architect Plugin?

Description
Getting some funky characters in my side bar whenever I turn this plugin on whilst having Architect enabled (v.2.2.8)

Also tried removing the custom svg icons from the new entries in the sidenav to see if it was those messing it up, and makes no difference either.
Steps to reproduce

Install CP Nav
Turn CP Nav on
Install Architect
Turn Architect on

Additional info

CP Nav Plugin version: 2.0.4
Craft version: 3.0.2

That bug is happening with other plugins that have an option on the sidebar. I have SEOmatic and Sprout Forms, but not Architect and it's happening to me.

Same with SEOmatic.

It actually only looks to be happening on my staging and local environements now, which is odd as it was doing it on all 3 at one point. Here are the environment details (all the same across each one)

PHP version 7.1.17
MySQL 5.5.60
Imagick 3.4.3 (ImageMagick 6.8.9-9)
Craft Pro 3.0.5
Yii <IP_ADDRESS>
Twig 2.4.8
Guzzle 6.3.3
Imagine 0.7-dev

Plugins:

Browser is Firefox Quantum 59.0.2 (64-bit), also occuring on Safari Version 11.0.1 (126<IP_ADDRESS>.1) & Chrome Version 66.0.3359.139
Running MacOS Sierra 10.12.6 (16G29)
Issues occur instantly after other plugins, but only seems to affect those items in the sidebar that are below a custom one added via your plugin. If you disable your plugin, the issue goes.
No errors in the console log either :/

Fixed in 2.0.6.


Fix typos in README

I just fixed a few typos in the sample code. The sample still isn't directly runnable (variables declared but not used), but I think the intent is obvious.

thanks. lgtm!
(red tests are expected)