[BUG] Can't change version code if --no-res was used

Information

Apktool Version (apktool -version) - 2.7.0
Operating System (Mac, Linux, Windows) - Linux
APK From? (Playstore, ROM, Other) - Playstore

Steps to Reproduce

Decompile any apk without its resources: apktool d <anyapk.apk> --no-res
Change the version string or version code in apktool.yml
Recompile
Version code will not have changed in the output apk file

I believe this to be expected behavior. I'm not how else we could adapt the version files if we aren't disassembling them. Sure we are reading them, but if you explicitly used --no-res - we aren't rebuilding them.


HyunSueJo

Name: HyunSueJo
GitHub: @HyunSueJo

Completed Courses
Name of the Section

Course
Files

Course Name
link

"You are supposed to comment Issue 206 (scroll down the page to Student Profiles 5), not open a new one"


Editor - Page title buttons

Page wizard: Page Title: The text/background colour buttons never work (no colour picker appears)
By Fay

Seems ok now


Adds PortType to package all

Adds PortType to package all.
Wards off people importing port_for.api.PortType directly.
Chore that needs to be done:

[ ] Add newsfragment pipenv run towncrier create [issue_number].[type].rst

Types are defined in the pyproject.toml, issue_numer either from issue tracker or the Pull request number

@bhelgs released


Create UX-STANDARDS.md

this is only a start, and a space for us to add stuff over time.
this doc is collectively owned and everyone is welcome to propose changes to this doc going forward.

Kind of an aside, but I wonder if there's somewhere else this can live too/be duplicated because right now, it's mostly guidelines that are also true for the service and would prob be helpful as kind of a global set of guidelines. But that's totally just something to keep in the back of our minds, not a problem needing solving now.

@kimberleyamackenzie i agree! the only reason I'm starting here is because the style guide has been somewhat successful in this repo as a resource. if the ux standards end up being a useful tool then we can figure out how to define this for multiple repos without duplication.


Animation for Presets and Comments fields is glitchy

Description
The CollapsibleBox method works great for minimizing the space that the Presets and Comments fields consume within the widget. However, the animation is glitchy, as shown in the screen recording below.

Files
https://github.com/PolarizedLightFieldMicroscopy/napari-LF/blob/1cc187856aa351e8041b799eb10574338decd67d/src/napari_lf/_widgetLF_gui.py#L1172
https://github.com/PolarizedLightFieldMicroscopy/napari-LF/blob/1cc187856aa351e8041b799eb10574338decd67d/src/napari_lf/_widgetLF_gui.py#L1767-L1837
Solution ideas

Make the nearby regions of fixed height instead of expandable height.
Omit the animation effect by using a method other than the CollapsibleBox class, and have the surrounding fields move instantaneously.
The napari-assistant plugin may similarly being using a dropdown area method for their info boxes. I observed what appeared to be an attempted animation when I hovered over the ? in the top right corner and some of the large buttons, as shown in the video below. So this glitchiness may be an issue with napari in general. @haesleinhuepf may also have some ideas on how to fix this glitchiness.

https://user-images.githubusercontent.com/23206511/215195478-71a3448d-79fe-4784-92ef-2a98cab2eb4a.mp4

That jitter looks so much better on your end, here's mine.. will look into your suggestion.
ice_video_20230127-161442.webm

Hi all,
I presume it's a qt-issue or a napari issue. To know which of the two, I recommend reaching out to the napari-core devs to ask them about their opinion. They have weekly open community meetings, the perfect place to show stuff and ask for support:
https://napari.org/stable/community/meeting_schedule.html
I hope that helps. 🌞
Best,
Robert

Thanks!! @haesleinhuepf

@gschlafly with the latest GUI changes, this seems more acceptable now.
ice_video_20230131-201422.webm


Return of MinimaxOPFModel

Is there any reason that MinimaxOPFModel returns PowerModels instead of JuMP.Model? Function MaximinOPFModel returns a JuMP.Model.

Returning the Powermodels instance allows the user not just access to the model, but to the other attributes of the Powermodels instance that is also created. Denoting the return value pm, the user would access the model with pm.model.
On the other hand, in creating the Maxmin problem, dualization() is applied to the same model that would be returned by MinimaxOPFModel(), and naturally there is no Powermodels instance associated with the dualized JuMP.Model instance.
One possibility might be to have both functions return the PowerModels instance that is created, if the MaximinOPFModel function could create an attribute like pm.dual_model.
If the idea is to keep the PowerModels interface under the hood, then both functions should just return instances of JuMP.Model.
While I lean more toward the former approach of having both functions return PowerModels instances with the desired model instantiated as an attribute, it's not perfectly clear to me which way to go.

OK. I think the current implementation makes sense then. Let's not take any further action on this.


Release compatible with Omnipay v2

Hi.
With the recent merge of https://github.com/thephpleague/omnipay-dummy/pull/7, we would like to no longer use the fork at https://github.com/jtgrimes/omnipay-dummy.
Can there be a release please?
Can Omnipay Common be set for V2 or V3 as we use 12 (yep just counted) drivers and they are all on v2.

@barryvdh can I leave this with you please since you appear to know more about what's going on with the 3.0 alphas than I do.

Hi. Please can the merge at cc603bdfb1641701643598c8afff94da571f7f49 be reverted? As things stand, this package cannot be released as a V2 driver due to this change.

What do you mean? There are 2.x tags that you can use for Omnipay V2

In short, every commit after the merge is ONLY for Omnipay V3.

I've setup branch 2.2 and tagged v2.2.0 with that commit reverted. Better now?

Can you do a release, rather than just tagging the repo. Packagist isn't seeing 2.2.0. https://packagist.org/packages/omnipay/dummy
And so you get ...
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - The requested package omnipay/dummy ^2.2.0 exists as omnipay/dummy[dev-master, 3.0.x-dev, v2.0.0, v2.1.0, v2.1.1, v2.1.2, v3.0-alpha.1, dev-feature-additional-request-types] but these are rejected by your constraint.


WIP: IpfsImage Component

Idea here is to create a view component that behaves exactly like the React Native Image, but instead of supplying a URI or base64 string as the image source, you specify an IPFS hash path.
Since the API is totally RN, ObjC/Java based, there are no security issues (like when we had to run a http gateway to serve the images), no request token, auth, etc.
I basically copied all the Image related code from RN and modified it to use our IPFS native module. This guarantees that we're using the same instance of the native module from this new IpfsImage component as well as other places in the app (adding image, for example)... There is only one native module instance created by RN, and therefore only IPFS node instance created.

Super duper. What do you think about calling it TextileImage? Only reason is that folks might try to use this for normal images hosted on IPFS, whereas this does decryption. Similarity, the "node" isn't really an IPFS node... of course it contains one, but the distinction is significant and growing.

We'll have to revisit slightly when the v2 stuff is in because the hash alone won't be enough... you'll need to pass something like <thread_pk_base64>:<content_hash>/<photo|thumb|etc>

For reference, here is a list of all the classes I had to understand/copy/modify in order to get iOS working. We'll have to do the same for Android:
TextileIPFS (to add the RCTBridge extension to expose TextileIPFS)
RCTConvert (to add support converting to/from IpfsImageSource)
RCTImageSource
RCTImageView
RCTImageViewManager
Image.ios.js (RN's iOS JS API)
Image.android.js (RN's Android JS API)

Cool @sanderpick, yea calling it TextileImage is a good idea. I'll do that now.
Adding params to the image source is no problem at all.

Awesome


Big button usage

Hi! How to make this site to open in new tab?
{{< big-button text="Visit Download Page" href="https://github.com/runningstream/hugograyscale/" >}}

You can edit "layouts/shortcodes/big-button.html" and add in target="_blank".  Then that big-button shortcode would look like:
{{ .Get "text" }}
The best way to do this is not to directly edit the "layouts/shortcodes/big-button.html" file directly in the themes directory though...  It's to make a shortcodes directory inside your site's layouts directory (not the theme's layouts directory), then copy "big-button.html" from the theme into your new site-specific shortcodes directory.  The one in your site layouts directory will override the one in the theme layouts directory.  That way - even if you update the theme some time you'll keep your copy of the shortcode separately.


msan support

Feature request
Add LLVM MSan support for JIT code.
In builds that enable MSan for building numba and Python, AOT compiled code is instrumented to add memory sanitization checks. In contrast the JIT code generated by numba is not instrumented, and when calling into the numba or Python runtimes it often passes memory that the instrumented code does not believe has been initialized.
This leads to unavoidable uninitialized-memory-read false positives.

I think this might need support in llvmlite - see numba/llvmlite#862.


Make gossip scores available to users

PR Description
Make gossip scores available to users.
Fixed Issue(s)

Fixes #3557
Documentation

[x] I thought about documentation and added the documentation label to this PR if updates are required.

Changelog

[x] I thought about adding a changelog entry, and added one if I deemed necessary.

I did the improvements. Thank you for the review!

Add the changes. Thank you for the second pass!

Looks great @BogdanStirbat! Just have one more edit to the changelog here: https://github.com/BogdanStirbat/teku/pull/1

Thank you! I've merged the changelog entry.


lower-case badge text

The standard that has largely emerged is fully lower-cased badges... it would be nice if Issue Stats adopted a similar style.
Issues Closed In → issues closed in

I would accept this if someone submitted a PR!

JK - just implemented. Badges are now lower case!

Ha, you got me. I was just about to open my fork.
Sent from my iPhone

On Mar 27, 2015, at 09:42, Hank Stoever<EMAIL_ADDRESS>wrote:
JK - just implemented. Badges are now lower case!

Reply to this email directly or view it on GitHub.


Include java in visual studio 2019

We ask you to include Java within the visual studio 2019 to enable Java programmers to use the features of this IDE to develop applications. Thank you.

Are not enough Intellij IDEA, Eclipse, NetBeans, VSCodecwith Java plugin not enough?
I don't think this is an issue.

I am using visual studio and i want to study java 😜

I used netbeans eclipse intellij for java development and I must say visual studio was superb compared to others, I wrote C# couple of years and the IDE really helped me there, but in open source versions(eclipse,netbeans,intelli) I had problems and IDE was not really helping me to fix those problems.
Eclipse is like a very developed editor. but it never helped me to fix issues related to server or start a project, I may need to run mvn or gradle manually at some step. same applies for intellij and vscode. In visual studio I click on run button and 99 times it worked without issues,
Even in starting a project, whenever I try to start any java project in any IDE it was half way working, but in visual studio it worked without any problems.
After all these years if you ask me, "do we need visual studio to support java" I will say no :) I think everything is working fine for me, but I dont think anyone new can start working in java easily.

My experience is the exact opposite to @ozkanpakdil when comparing Visual Studio to IntelliJ. Rather I found VisualStudio unusable. Must be a subjective thing.

@BenAbdo I'm curious: what is in Visual Studio that stands out so much that you would be willing to study Java in it instead of any other already mature Java IDE? And assuming we start working on this, would you wait for a few years? :-)

Duplicate of #11


refactor!: Refactor Redis Pub/sub topic wild cards to match that of MQTT

BREAKING CHANGE: implement the topic conversion between Redis Pub/sub wild card * and MQTT wild cards + for single level and # for multiple levels
close #166
Signed-off-by: Ginny Guan<EMAIL_ADDRESS>

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/go-mod-messaging/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?) update existing unit tests
[x] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?) N/A, no related docs

Testing Instructions

Update device-virtual system event topic to edgex/system-events/core-metadata/device/+/device-virtual/#
Build and run device-virtual
Receive system events correctly

New Dependency Instructions (If applicable)

Codecov Report

Merging #211 (caadc50) into main (c8fa4c0) will increase coverage by 0.05%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #211      +/-   ##
==========================================
+ Coverage   75.22%   75.27%   +0.05%
==========================================
  Files          20       20
  Lines         896      898       +2
==========================================
+ Hits          674      676       +2
  Misses        172      172
  Partials       50       50

Impacted Files
Coverage Δ

internal/pkg/redis/client.go
88.18% <100.00%> (+0.21%)
:arrow_up:

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

@jinlinGuan the docs here need to be update now that this inconsistency is fixed. THX!
https://docs.edgexfoundry.org/3.0/microservices/general/messagebus/#multi-level-topics-and-wildcards

PR for docs: https://github.com/edgexfoundry/edgex-docs/pull/953


Downloader terminates if it encounters a track with no artist

My uploads music library is a bit of a hodge podge of files and includes some items like voice notes which don't have an artist.
The Downloader exits with an error on line 125 when trying to pass the artist into addcslashes:
addcslashes(): Argument #1 ($string) must be of type string, null given
To get round this, I manually edited the metadata.json file to replace "artist":null with "artist":"Not known"

Yes, or you can edit the script and replace:
addcslashes($song['artist'], ':'),

By:
addcslashes($song['artist'] ?? 'Unknown artist', ':'),


feat: stylish-haskell support

You linked to the fourmolu formatter, is that intended?

No that is not intended 😂
Will get that fixed

Was not intended lol
Fixed

Implemented in #435


1.0.pre release

Checklist

[X] Used a personal fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[X] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

redo to allow changes in base branch


What is going on with the examples ?

Hi,
I am trying to use this library, but half of the functions and structs seem to be missing...
There is no Fc::new, no Saw, etc...
The examples are unusable...
What is going on ?

Hi, I'm actually in the middle of fixing it right now.  Give me like an hour and a half (maybe sooner).  The issue is I changed the API of the fon crate to work with the wavy crate but completely forgot to fix the twang examples.  Not really sure how this escaped me. ---Jeron Aldaron Lau   17.10.2020, 10:53, "Frédéric Nieto"<EMAIL_ADDRESS>Hi,I am trying to use this library, but half of the functions and structs seem to be missing...There is no Fc::new, no Saw, etc...The examples are unusable...What is going on ?—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.

Oh ok, great, keep me posted, i'll try it later.

@Wicpar The examples should be fixed now.

It works now, but performance of the synthesis is abysmal...
126,564,240 ns per iter (only the synth.gen bit, not the setup or audio write) on my xeon E-2176M
I need to encode data frames for ultrasonic communication, so I cannot afford this much overhead...
Thanks for the quick fix nevertheless.

@Wicpar That doesn't seem right.  I have not optimized it very much yet, but even on a debug build I can generate 5 seconds of a sawtooth wave, and write it out my SSD in 0.120 seconds.  Even including the file I/O, that is 240000 iterations per 120 milliseconds, which is 500 nanoseconds per iteration.  My machine is a 10th gen Intel Core i5, but I don't think even large machine differences could cause that huge of a performance difference.  Can you double check your measurements?

Your numbers are correct, i mean each 5 sec clip generation from your example takes 126ms, which is way too high...
The system needs to work on old crappy smartphones...

I guess I don't know much about ultrasonic communication, but as long as you're not generating audio way faster than you play it, I would think the current amount overhead wouldn't be an issue and probably run fine even on my first smart phone (which was very crappy).  Although, if you can come up with a way to make it faster, I am definitely accepting pull requests.  Good luck to you, though, the algorithm is pretty bare-bones besides just a lot of newtypes.


cmake: Fix generated header file dependency

Fix race condition in build when creating libraries in application and
depending on zephyr_interface.
lib/CMakeLists.txt:
    add_library(lib STATIC src/lib.c)
    zephyr_append_cmake_library(lib)
    target_link_libraries(lib PUBLIC zephyr_interface)

Gives following error message:
  from path/src/lib.c:6:
  zephyr/include/syscall.h:11:10: fatal error: syscall_list.h: \
  	No such file or directory
     11 | #include <syscall_list.h>
        |          ^~~~~~~~~~~~~~~~
  compilation terminated.
  [13/165] Generating include/generated/syscall_dispatch.c, \
  	include/generated/syscall_list.h

    add_library(lib STATIC src/lib.c)
    zephyr_append_cmake_library(lib)
    target_link_libraries(lib PUBLIC zephyr_interface)

when depending on generated headers, then targets should add a dependency to: zephyr_generated_headers.
That is:
add_library(lib STATIC src/lib.c)
zephyr_append_cmake_library(lib)
target_link_libraries(lib PUBLIC zephyr_interface)
add_dependencies(lib zephyr_generated_headers)

https://github.com/zephyrproject-rtos/zephyr/blob/master/CMakeLists.txt#L652-L655
unfortunately it seems @SebastianBoe never got the time to update the docs.: https://github.com/zephyrproject-rtos/zephyr/pull/22143#issuecomment-577763234
Will need a little time to fully ensure that changing dependencies has no side-effects.
Changed to enhancment, as this is not a bug.

DNM until I have had time to completely ensure there are no side-effects to changing the dependencies.

@tejlmand Note that in this case it wasn't me that depended on the generated the header files, it is the zephyr headers itself.
I got the same issue when using zephyrs own wrappers to create header (because they exactly the same).
zephyr_library_named(lib)
zephyr_library_sources(lib_src.c)

I have seen the question about how to properly create libraries in the app and link them in the zephyr build system pop up many places.
The samples only have externally built static library, I think adding application libraries sample would be helpful.

@joerchan thanks for a little more details.
Note using Zephyr's macros zephyr_library AFTER find_package(Zephyr) also has limitations.
One limitation is that libraries created after find_package(Zephyr) will not be included inside the --whole-archive linker flags, and this may cause unintended side effects for the user.
So if wanting to use zephyr_library macros, then it is recommended to get the library included into the build system as a Zephyr module.
This limitation on Zephyr macros is something I also would like to improve in the future.

I have seen the question about how to properly create libraries in the app and link them in the zephyr build system pop up many places.
The samples only have externally built static library, I think adding application libraries sample would be helpful.

I agree. The guides has too little information on how to properly create libs outside Zephyr itself.
Especially when taken into consideration the limitations when creating a lib as part of the app.

DNM until I have had time to completely ensure there are no side-effects to changing the dependencies.

Did you have any chance to look at this?
If you have a task for static library improvements, then link order of libgcc could be added as well.
Had another issue where -lgcc had to be added after the extra library to resolve link errors.

DNM until I have had time to completely ensure there are no side-effects to changing the dependencies.

Did you have any chance to look at this?

Unfortunately not.

If you have a task for static library improvements, then link order of libgcc could be added as well.
Had another issue where -lgcc had to be added after the extra library to resolve link errors.

I believe that is related to:
https://github.com/zephyrproject-rtos/zephyr/issues/28650

I believe that is related to:

Seems like the same fix atleast, we had failure to link with libgcc in the last library in the link command. Because -lgcc was before it.
Adding it both before and after worked.

Hello,
I've just tested this patch since I ran into the same bulid error as described and it works very well.
With the patch I got consistent bulids with external libraries.
I wolud like this patch to be merged :+1:
Best regards,

@tejlmand Rebased and fixed your comment.
Can you remove the DNM label?

Can you remove the DNM label?

Done.


Create colors page

Create page to present the colors of the project

Best way is to generate the colors divs with javascript and SASS. I will think in a good way to do that.

Closed 76e4ee0f450fc9ea3bf3213be7e72c4389ae0639


Tag externally visible objects

Put some bytes at the beginning of objects so we can recognise wrong pointers being passed

only relevant for C api


fix: adding textInput videos

In the process of adding the native text input videos, I realized the "magic" of how the videos were being output on the page was not going to allow us to do native videos with different states like Debbie was hoping. I thus reworked the logic of how these output based on doing a "fuzzier" search on the slug and the category (native or web). This definitely needs a review and clean up and another round.

@silverli If this PR is still relevant I'm wondering why we wouldn't want this functionality for Web too? It seems to be working but maybe can just be generalized so we could have states in Web too. I like the long drawn out if statement instead of regex - more readable to me.


Update SafeAreaView import

READ ME PLEASE!
TL;DR: Make sure to add your changes to versioned docs
Thanks for opening a PR!
The docs cover several versions of react-navigation, and in some cases there are several files (for version 1, version 2 and etc.) that all describe a single page of the docs (eg. "Getting Started").
Please make sure that the edit you're making in docs/file-you-edited.md is also included in the file for the correct version, eg. /versioned_docs/version-3.x/file-you-edited.md for version 3. If such file doesn't exist, please create it. :+1:

Thanks for the PR, but we don't recommend using SafeAreaView from react-native since it only works on iOS.


Set HCaptchaSiteKey on Link Account pages

When using HCaptcha on link account pages the site key needs to be passed
in. This PR ensures that HCaptchaSiteKey is set in the data.
Fix #14766
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

Codecov Report

Merging #14834 (24afae8) into master (cf29cb3) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master   #14834   +/-   ##
=======================================
  Coverage   42.22%   42.22%
=======================================
  Files         771      771
  Lines       82181    82188    +7
=======================================
+ Hits        34702    34706    +4
- Misses      41826    41830    +4
+ Partials     5653     5652    -1

Impacted Files
Coverage Δ

routers/user/auth.go
12.20% <0.00%> (-0.04%)
:arrow_down:

models/repo_list.go
77.87% <0.00%> (-0.89%)
:arrow_down:

models/repo.go
52.57% <0.00%> (+0.16%)
:arrow_up:

services/pull/pull.go
42.64% <0.00%> (+0.49%)
:arrow_up:

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 cf29cb3...24afae8. Read the comment docs.

🚀

Happy for anyone to send backport

(or I'll do it at some point tomorrow)


Fixed exposure of attributes marked as safe when using hash models.

The valid_exposure check now also accepts Hash objects that have a key matching the attribute's name.
Fix includes spec that demonstrates the issue and fails without the changes.

Thanks.


Cannot connect to marketplace

Issue Type: Bug
CORS request error

VS Code version: VSCodium 1.58.2 (c3f126316369cd610563c75b1b1725e0679adfb3, 2021-07-16T00:15:55.196Z)
OS version: Linux x64 5.10.55-1-lts
Restricted Mode: No

System Info

Item
Value

CPUs
AMD Ryzen 5 3600 6-Core Processor (12 x 3959)

Load (avg)
1, 1, 2

Process Argv
--disable-extensions

Screen Reader
no

VM
0%

DESKTOP_SESSION
plasma

XDG_CURRENT_DESKTOP
KDE

XDG_SESSION_DESKTOP
KDE

XDG_SESSION_TYPE
x11

Extensions disabled

installed from vscodium-bin

Just found I have to install vscodium-bin-marketplace as well, sorry guys

not helped me, i installed amd64.deb from github as it works in my laptop


Improve the AttentionMechanism base class

Description
The AttentionWrapper takes AttentionMechanism instances as arguments. However, it used methods that were not defined in the AttentionMechanism base class.
This PR improves the base class definition with the actual interface expected by AttentionWrapper.
Type of change

[ ] Bug fix
[ ] New Tutorial
[x] Updated or additional documentation
[ ] Additional Testing
[ ] New Activation and the changes conform to the activation contribution guidelines
[ ] New Callback and the changes conform to the callback contribution guidelines
[ ] New Image addition and the changes conform to the image op contribution guidelines
[ ] New Layer and the changes conform to the layer contribution guidelines
[ ] New Loss and the changes conform to the loss contribution guidelines
[ ] New Metric and the changes conform to the metric contribution guidelines
[ ] New Optimizer and the changes conform to the optimizer contribution guidelines
[ ] New RNN Cell and the changes conform to the rnn contribution guidelines
[x] New Seq2seq addition and the changes conform to the seq2seq contribution guidelines
[ ] New Text addition and the changes conform to the text op contribution guidelines

Checklist:

[x] I've properly formatted my code according to the guidelines

[x] By running Black + Flake8
[ ] By running pre-commit hooks

[ ] This PR addresses an already submitted issue for TensorFlow Addons
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] This PR contains modifications to C++ custom-ops

How Has This Been Tested?
Ran the existing tests.

Thanks for the update and sorry for the late reply.
The concern I have for this PR is that the implementation of the attention is now split into two classes, especially self.key and self.value are created by used in two different class. When only looking at the AttentionMechanism, it is hard to reason about what is the self.key/value, unless we look at how they are initialized in _BaseAttentionMechanism.setup_memory().
If you feel AttentionMechanism should declare more methods for AttentionWrapper to use, maybe we should just create them but still keep the implementation in _BaseAttentionMechanism?

Thanks for the comment.
The initial observation was that AttentionWrapper expected a _BaseAttentionMechanism and not a AttentionMechanism. So my idea was to move some methods in the base class to fix this inconsistency and at the same time make it easier for users to write custom attention mechanisms. By following this route, would it be enough to define a base implementation of setup_memory in AttentionMechanism to clarify how the attributes are initialized?
Or do you think it's better for AttentionMechanism to just be the interface?

I guess the AttentionWrapper won't work if user just implement the method in AttentionMechanism, so I would expect _BaseAttentionMechanism to just graduate to be a public class to override. Maybe we should just rename the _BaseAttentionMechanism and remove the AttentionMechanism?

Ok, let's rename _BaseAttentionMechanism to AttentionMechanism. I will make a new PR. Thanks!


initcouch2.sh doesn't set to single-node, other issues

Hello, I think I'm having an issue with the initcouch2.sh script. It's the same whether I start by installing CouchDB 2.1.1 with apt-get, or use a docker image via "docker pull apache/couchdb:2.1.1". I get this output if I run the script:

It appears the script wants to setup in single-node mode (please correct if I'm wrong, I'm going off the comment in the script and in the previous commit where that code was added) but actually the command on line 10:
curl -X POST $HOST/_cluster_setup -H 'Content-Type: application/json' -d '{"action":"enable_cluster","username":"couchadmin","password":"test","bind_address":"<IP_ADDRESS>","port":5984}'
is applying the "enable_cluster" setting. The manual says "enable_cluster" is to "Configure the local or remote node as one node, preparing it to be joined to a new CouchDB cluster." I changed it to "single_node" (and removed the following line as it's not necessary to "finish_cluster" anymore) and I get the following output:

still getting errors on the later authentication settings being applied. I changed on line 27 and 28 the "couchdb@localhost" to "_local" and now these are working for me as well and I can confirm the settings are being applied by checking in the CouchDB management app. The manual says "The literal string _local serves as an alias for the local node name". The manual also says "the default node name has changed from couchdb@localhost to<EMAIL_ADDRESS>so that would probably also work. I had been looking at the manual for 2.1.0 found here: media.readthedocs.org/pdf/couchdb/latest/couchdb.pdf.
OS and Browser:
Ubuntu 16.04
If you think these changes should be committed, I will open a PR. Thanks.

@MatthewDorner Hi, Matthew. After the modifications that you did to the script, you got the application running ?

I've only just begun looking at the application so I can't say comprehensively that it's "working" but I get CouchDB installed now and I don't get any CouchDB login error when launching "npm start" the way I would before. Is there a test suite I can run for CouchDB functionality or anything?

It's a good thing!
There is a set of test scripts, that is more comprehensive, that can be run.
I believe that is triggered by calling ember test.

Yes, the tests do pass.

here are changes I made to my copy: comparison


Ability for affiliation and name to be blank on registration.

Closes #603

The validation might need an update, too, if it considers these fields required as opposed to optional, but it's just a matter of the "service layer."

Hmm, in that case, given the already existing lines of:
    user.affiliation = reqBody.affiliation ? reqBody.affiliation :  ''
    user.name = reqBody.name ? reqBody.name :  ''

in userService.js it seems like it this is already accounted for in which case, perhaps this branch should be abandoned and the #195 issue be closed? @WrathfulSpatula

@vprusso That sounds right. Good work on forming a plan ahead, if only for due diligence of gaining context in the code base and technological framework.

@vprusso That sounds right. Good work on forming a plan ahead, if only for due diligence of gaining context in the code base and technological framework.

Thanks for the kind words and explanation. With that, I'll go ahead and close this out. Thanks again for the feedback!


ConnectionDragThreshold

Implement a connectionDragThreshold, so onClick and onDrag events can be distinguished better.

Related: we added a paneClickDistance in 12.0.0 which lets you configure the max distance between mousedown/up that will trigger a click.

@moklick is the paneClickDistance prop on ReactFlow instance the one that should be used if we want to still trigger clicks when users accidentally make a small drag movement? We are experiencing an issue with our users where they click but it is not treated as a click on inner elements within a node because of a slight drag movement.
We are on "@xyflow/react": "^12.0.0-next.16", using Typescript, but that prop is not being recognized as a valid prop option for some reason.

I see nodeDragThreshold as a prop option, but unfortunately that still doesn't allow it to trigger the onClick of an inner element whenever a small drag movement is made under the threshold.

You need to update to 12.0.3 to use paneClickDistance. If you have interactive elements inside your nodes, you might want to use the class name "nodrag" in order to prevent dragging at all.

ah okay I thought the 12.0.0 "next" build was newer than the 12.0.3. The problem with nodrag is that we do in fact want the whole node to be draggable, but we would just like to have it trigger normal click events when the drag amount is below the drag threshold.

Updating and adding paneClickDistance did not trigger either a onNodeClick action or the native onClick when doing a slight drag movement below the specified distance. Am I missing something here?

https://github.com/xyflow/xyflow/issues/2341 seems related but still not clear to me how we can achieve the native onClick event to trigger for small drag movements.

In v12.0.2 we added nodeClickDistance. Does that solve your issue @c0d3ster ?

nodeClickDistance resolved our issue, but it is not yet in the documentation. This combined with nodeDragThreshold set to the same pixel value (10 in our case) allows small drag movements to be treated as clicks still and ignores any custom drag behavior below that threshold 👍

That's great to hear @c0d3ster . I just added the missing docs!


SME is blocking video-downloads

It's appearing some errors with some videos which is saying that because of SME, the video can't be obtained on my country

Carefully read new issue template and provide all requested information.


=, in url escaped

https://www..com/list=1,2
it seems that the fetch function replace
https://www..com/list%3D1%2C2
resulting in a url access error, can it be unescaped?

proc path*(url: Url): string =
  ## Returns the paths combined into a single path string.
  ## @["foo", "bar"] -> "/foo/bar"
  if url.paths.len > 0:
    for part in url.paths:
      result.add '/'
      result.add encodeURIComponent(part)

Is it possible to add an optional parameter not encodeURIComponent
or not escaped or to exclude =, from encodeURIComponent

Thank you for the bug report! You have the correct behavior, we will fix ours to match soon.

This is fixed in release 2.1.2. Feel free to re-open if things are still not working as expected.


python cli_demo.py 下载模型时候报错

需要微调模型，必须时候sat下的模型，跑cli_demo.py 时候，sat/resources/download.py下载模型时候报错
报错信息：
botocore.exceptions.ClientError: An error occurred (400) when calling the HeadObject operation: Bad Request

https://github.com/boto/boto3/issues/3135
Seems like your client region is wrong. Try upgrading your boto3.
pip install -U boto3

boto/boto3#3135
Seems like your client region is wrong. Try upgrading your boto3.
pip install -U boto3

it not work

Then it's your network problem... I have no idea.


App is unresponsive after opening

Occasionally after opening the app, the JS thread locks up and the app becomes unresponsive. It eventually clears up after about 1 minute.
Session logs:
1:38:56 PM | DEBUG : [AccountsStore] Initiating account 0xf0EA7663233F99D0c12370671abBb6Cca980a490
1:38:56 PM | DEBUG : [Datasource] Initializing datasource for converse-5e4c1732-3a8b-43cc-a8f3-9497af60b5cc.sqlite
1:38:56 PM | DEBUG : [Async Updates] Last version open: 1.4.9
1:38:55 PM | DEBUG : [AccountsStore] Initiating account TEMPORARY_ACCOUNT
1:38:56 PM | DEBUG : Initializing Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490
1:38:59 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:38:59 PM | DEBUG : Database initialized for 0xf0EA7663233F99D0c12370671abBb6Cca980a490
1:43:58 PM | DEBUG : Database optimized for 0xf0EA7663233F99D0c12370671abBb6Cca980a490
1:43:58 PM | DEBUG : Running migrations for 0xf0EA7663233F99D0c12370671abBb6Cca980a490
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | WARN : [API] HTTP Error - No response received - https://backend-prod.converse.xyz/api/user
{
  "UNSENT": 0,
  "OPENED": 1,
  "HEADERS_RECEIVED": 2,
  "LOADING": 3,
  "DONE": 4,
  "readyState": 4,
  "status": 0,
  "timeout": 0,
  "withCredentials": true,
  "upload": {},
  "_aborted": false,
  "_hasError": true,
  "_method": "POST",
  "_perfKey": "network_XMLHttpRequest_https://backend-prod.converse.xyz/api/user",
  "_response": "The network connection was lost.",
  "_url": "https://backend-prod.converse.xyz/api/user",
  "_timedOut": false,
  "_trackingName": "unknown",
  "_incrementalEvents": true,
  "_performanceLogger": {},
  "_requestId": null,
  "_headers": {},
  "_responseType": "",
  "_sent": true,
  "_lowerCaseResponseHeaders": {},
  "_subscriptions": [],
  "__sentry_xhr_v3__": {}
}
1:43:58 PM | DEBUG : Migrations done for 0xf0EA7663233F99D0c12370671abBb6Cca980a490
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : loading base64 key
1:43:58 PM | DEBUG : Got 1 new messages from notifications:
[
  {}
]
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : get client from base64 key
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | DEBUG : Database for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 not yet initialized
1:43:58 PM | INFO : Balance for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 already refreshed less than 1800s ago
1:43:59 PM | INFO : [XmtpRN] Syncing 0xf0EA7663233F99D0c12370671abBb6Cca980a490
{
  "lastSyncedAt":<PHONE_NUMBER>905,
  "knownTopics": 842
}
1:43:59 PM | DEBUG : [Hydration] Took 302.444 seconds total
1:43:59 PM | DEBUG : Client for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 already instantiating
1:43:59 PM | INFO : [XmtpRN] Instantiated client for 0xf0EA7663233F99D0c12370671abBb6Cca980a490
1:44:49 PM | DEBUG : [XmtpRN] Imported 832 exported conversations into client in 49.127s
1:44:49 PM | INFO : Trying to send 1 pending messages...
1:44:49 PM | WARN : Did not find the conversation for topic /xmtp/mls/1/g-e2bfe34830662f11061ad87afdb226e9/proto, will retry...
1:44:50 PM | INFO : Trying to send 1 pending messages...
1:44:50 PM | WARN : Did not find the conversation for topic /xmtp/mls/1/g-e2bfe34830662f11061ad87afdb226e9/proto, will retry...
1:44:51 PM | DEBUG : Internet reachable: true
1:44:51 PM | DEBUG : [Async Updates] Account: 0xf0EA7663233F99D0c12370671abBb6Cca980a490
1:44:51 PM | DEBUG : [Async Updates] lastAsyncUpdate: 2
1:44:51 PM | INFO : [Notifications] Subscribing to 14 topic for 0xf0EA7663233F99D0c12370671abBb6Cca980a490
1:44:51 PM | INFO : Trying to send 1 pending messages...
1:44:51 PM | WARN : Did not find the conversation for topic /xmtp/mls/1/g-e2bfe34830662f11061ad87afdb226e9/proto, will retry...
1:44:53 PM | INFO : Trying to send 1 pending messages...
1:44:54 PM | DEBUG : [XmtpRN] Listing 832 conversations for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 took 54.436 seconds
1:44:54 PM | DEBUG : [XmtpRN] Backed up 832 conversations into secure mmkv in 0.005s
1:44:54 PM | INFO : STREAMING CONVOS
1:44:54 PM | INFO : STREAMING GROUPS
1:44:54 PM | DEBUG : [XmtpRN] Stopped streaming messages for 0xf0EA7663233F99D0c12370671abBb6Cca980a490 1:44:54 PM | INFO : [XmtpRN] Streaming messages for 0xf0EA7663233F99D0c12370671abBb6Cca980a490
1:44:54 PM | DEBUG : RECALLING syncConversationsMessages / syncGroupsMessages
1:44:54 PM | INFO : Syncing 10 groups...
1:44:55 PM | DEBUG : synced group /xmtp/mls/1/g-21bb894493eeee79521a5d4baef0d152/proto
1:44:55 PM | DEBUG : synced group /xmtp/mls/1/g-c14a992ea7e2cf5b3adc5d2626f02220/proto
1:44:55 PM | DEBUG : synced group /xmtp/mls/1/g-8d6584e3555ebdb20cc836cd2b9b47e2/proto
1:44:55 PM | DEBUG : synced group /xmtp/mls/1/g-d458c25b7e9b1ccf3b324061add11468/proto
1:44:55 PM | DEBUG : synced group /xmtp/mls/1/g-eda09e68588080e1705b6d93de610a8b/proto
1:44:55 PM | DEBUG : synced group /xmtp/mls/1/g-615ee407d4f49cd373255b9733071677/proto
1:44:55 PM | DEBUG : synced group /xmtp/mls/1/g-e2bfe34830662f11061ad87afdb226e9/proto
1:44:55 PM | DEBUG : synced group /xmtp/mls/1/g-02fa49a58d279e2777435cd0ae9f16b9/proto
1:44:55 PM | DEBUG : synced group /xmtp/mls/1/g-7f84219d9224180826614c7fdf7cc40e/proto
1:44:55 PM | DEBUG : synced group /xmtp/mls/1/g-2923178a5dfe29a4689d2612864aee8a/proto
1:44:55 PM | INFO : 10 groups synced! 1:44:55 PM | INFO : 8 groups messages pulled
1:44:56 PM | INFO : [Notifications] Subscribing to 14 topic for 0xf0EA7663233F99D0c12370671abBb6Cca980a490

I think the latest perf improvements should fix / mitigate a lot of these issues. We should keep an eye out for this one in the next few days and close if we think it's fixed.


Make sure https://git.io/* points to the right place

We use git.io URLs in some places

https://git.io/now-global-config redirects to a wrong url https://github.com/zeit/now#global-config. It has to be https://github.com/zeit/now#global-configuration

✅ 340b8721ac91f47fc694a74e55b6032e70f41f39


Firefox autoselection of chat box does not fill first key

This is a Bug Report
Description
On Firefox, when the chat box is not selected in the Davis Web UI and a key is pressed, the chat box is selected but the pressed key is not written to the box. This is working in other web browsers.
For bug reports:

Description of the issue
Steps to reproduce

Log in to web UI in firefox
Deselect the chat box
Type Hey davis

Desired behavior

"Hey davis" is typed in the box

Actual behavior

"ey davis" is typed in the box

Mike mentioned this issue last week, it should already be fixed in my filters branch.

76ae26e73d4186d276f15929d0aabed37d5aed17


Making my own summary file?

I recently merged a lot of single fast5s into multi_fast5 and in the process I kept track of the read_ids and the filenames.
Is it possible to make my own fake "sequencing_summary.txt" formatted as follows and expect it to work with nanopolish index?
filename        read_id
/path/to/gridion_181008_CHM13.fast5  1d101641-548d-4c9a-b3a7-4b407f65b757
/path/to/gridion_181008_CHM13.fast5  63f5baa8-320d-4d8d-b27f-e64c44857949
...

Thanks in advance!

that should be fine, indexing only uses those two columns and the order doesn't matter.


Webbing attachement with hook

This style of attachment seems printable and would be useful for feedbags and rando-style, quick access closures.

A 25 and 40mm option would be nice

[ ] Test print

[ ] Measure common sizes

[ ] This is something too.


Update project-08-molecular-representation.md

Welcome @schenkch
Now we have a full team!

That's great!


Basic schema validation tests for MSFT_xCertReq

Added basic schema validation tests for MSFT_xCertReq. More specific tests coming soon.

Sorry for the hassle. Not sure why GitHub considers the PR as non-mergeable. The AppVeyor build worked on my fork after fixing up the appveyor.yml file - https://ci.appveyor.com/project/Dan1el42/xcertificate/build/<IP_ADDRESS>.

Thanks for your contribution @Dan1el42
This PR is marked as non-mergable due to existing conflicts.
Please resolve them and tests should be able to run
Step 1: From your project repository, check out a new branch and test the changes.
git checkout -b Dan1el42-basic_schema_validation_tests dev
git pull https://github.com/Dan1el42/xCertificate.git basic_schema_validation_tests

Step 2: Merge the changes and update on GitHub.
git checkout dev
git merge --no-ff Dan1el42-basic_schema_validation_tests
git push origin dev

PR obsolete once the basic schema validation tests are merged for DscResource,Tests.

@KarolKaczmarek, Thanks very much for the instructions.


Build guidelines to avoid missing P224

Kubernetes build issues such as https://github.com/kubernetes/kubernetes/issues/29534 propagate into bootkube since kubernetes is a vendored component. RHEL/Fedora don't include P224 for some license reasons and builds in this environment fail
Name        : golang
Version     : 1.6.3
Release     : 2.fc24
Architecture: x86_64

make install
mkdir -p _output/bin/linux/
GOOS=linux go build -ldflags "-X github.com/kubernetes-incubator/bootkube/pkg/version.Version=v0.2.0" -o _output/bin/linux/bootkube github.com/kubernetes-incubator/bootkube/cmd/bootkube
# github.com/kubernetes-incubator/bootkube/vendor/k8s.io/kubernetes/pkg/util/certificates
vendor/k8s.io/kubernetes/pkg/util/certificates/csr.go:96: undefined: elliptic.P224
# github.com/kubernetes-incubator/bootkube/vendor/github.com/google/certificate-transparency/go/x509
vendor/github.com/google/certificate-transparency/go/x509/x509.go:342: undefined: elliptic.P224
vendor/github.com/google/certificate-transparency/go/x509/x509.go:355: undefined: elliptic.P224
vendor/github.com/google/certificate-transparency/go/x509/x509.go:1461: undefined: elliptic.P224
Makefile:27: recipe for target '_output/bin/linux/bootkube' failed
make: *** [_output/bin/linux/bootkube] Error 2

Using the binary from the go1.7.1 linux/amd64 tarball does seem to allow builds to succeed as reported upstream. It doesn't appear much could be done within bootkube to address this except re-vendoring when upstream reaches a conclusion.
Perhaps I'll add a Go version note to the README for now?
cc @gtank

Also not exactly a solution, but we could have the make release target use the containerized build to mitigate this (https://github.com/kubernetes-incubator/bootkube/blob/master/build/build-release.sh) -- but that also assumes host has rkt installed.

Using the binary Go releases will always solve this problem. The problem is Fedora packagers explicitly removing ECC support from the Go that ships in their repos. They do the same for openssl - my understanding is it's something to do with patent risk (cc @mjg59?)

@aaronlevy yes! rkt-based builds would be nice

Gonna close this, but let me know if it should be re-opened


Trained model not working

I have trained the PPO2 model on Walker2d-v2 environment with following command
with nminibatches=64
python -m baselines.run --alg=ppo2 --env=Walker2d-v2 --num_timesteps=1e6 --seed=30 --network=mlp --num_env=1 --save_path="/home/surabhi/Downloads/github/baselines/result/walker2d/30/ppo2" --log_path="/home/surabhi/Downloads/github/baselines/result/walker2d/30/"

But when i run the trained model it is showing return of ~9.
python -m baselines.run --alg=ppo2 --env=Walker2d-v2 --num_timesteps=0 --seed=30 --network=mlp --num_env=1 --load_path="/home/surabhi/Downloads/github/baselines/result/walker2d/30/ppo2" --play --save_video_interval=1 save_video_length=1000
if args.play:
        logger.log("Running trained model")
        roll_rew = [0.0]
        state = model.initial_state if hasattr(model, 'initial_state') else None
        for roll in range(10): #number of rollout
            obs = env.reset()
            dones = np.zeros((1,))
            episode_rew = [0.0]
            eplen=0
            for ts in range(1000): #max rollout length (path len)

                if state is not None:
                    actions, _, state, _ = model.step(obs,S=state, M=dones)
                else:
                    actions, _, _, _ = model.step(obs)
obs, rew, done, _ = env.step(actions)
                print(rew[0])
                episode_rew[-1] += rew[0]
                eplen+=1
                #env.render()
                done_any = done.any() if isinstance(done, np.ndarray) else done
                if done_any or eplen>=1000: #maxlen of rollout
                    #for i in np.nonzero(done)[0]:
                    #    print('episode_rew={}'.format(episode_rew))
                        #episode_rew[i] = 0
                    obs = env.reset()
                    episode_rew.append(0.0)
                    eplen=0
            #print("#mean100ep_reward",episode_rew[-100:])
            print(round(np.mean(episode_rew[-100:]),1),len(episode_rew))
            roll_rew.append(round(np.mean(episode_rew[-100:]),1))
        print("rollout avg ",sum(roll_rew)//10)

Please tell me how can we evaluate or simulate the trained model to know its performance?
Thanks

Waiting for the answer.
Please reply. This happens with all the models trained with this repository.  Training curves are somewhat closer to that of research paper but testing on seed on which training was done are too less.

Waiting for the answer.
Please reply. This happens with all the models trained with this repository. Training curves are somewhat closer to that of research paper but testing on seed on which training was done are too less.

Hi, have you figured out the reason?

No
Sent from Outlook Mobilehttps://aka.ms/blhgte

From: tmu<EMAIL_ADDRESS>Sent: Tuesday, May 26, 2020 11:42:41 AM
To: openai/baselines<EMAIL_ADDRESS>Cc: SURBHI GUPTA<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [openai/baselines] Trained model not working (#1054)
Waiting for the answer.
Please reply. This happens with all the models trained with this repository. Training curves are somewhat closer to that of research paper but testing on seed on which training was done are too less.
Hi, have you figured out the reason?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/openai/baselines/issues/1054#issuecomment-633827037, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMH64BNDTZYPUFFBFGEUYIDRTNMVTANCNFSM4JZJ3SIQ.

I had some problems with the parser and it took me one hour to find out that the condition if load_path is not None in ppo2.py was not satisfied. Hope you did not make the same mistake as I did. :(


Feature/UI ds 272 update checkbox groups

Closes #272
[ ] Update CheckboxButtonGroup variants to better match Figma
[ ] Add new stories to CheckboxButtonGroup to better match Figma
[ ] Update checkbox font weights to 300

Thanks Rachel, this is looking a lot betterl! Leaving my notes below:
General feedback

Update all helper text to be font-weight:100 and $ux-gray-900
Is there an option within each variant to turn on the border for the entire group like we do under CheckboxButtonGroup/Default?

CheckboxButton / Description

Include helper text next to label
Update children copy to be font-weight-100
When selected, can we change the children copy text color to $ux-blue-700? And update border color to $ux-blue-200 with fill set to $ux-blue-100?
Can we also carry these changes over to CheckboxButtonGroup/Description & Description Row variants?

CheckboxButtonGroup / Default Row

Add in margin-right: 0.5rem to FormControlLabel__label

Thanks @jojolee1996 ! Sorry for the delay - I think I got all your feedback:
[ ] Updated helper text to font-weight: 100 color: ux-gray-900
[ ] Is there an option within each variant to turn on the border for the entire group like we do under CheckboxButtonGroup/Default?

----> Yes there is, though technically it is on the FormGroup component

[ ] Include helper text next to label (Done on the last two stories)
[ ] Update children copy to be font-weight-100
[ ] When selected, can we change the children copy text color to $ux-blue-700
[ ]And update border color to $ux-blue-200 with fill set to $ux-blue-100?

---> These two were already set so not sure if I missed where you are talking about!

[ ] Add in margin-right: 0.5rem to FormControlLabel__label

----> It was actually easier to increase the margin on FormControlLabel to 1rem since it was already set at 0.5

Thanks for making those changes, they look good to me! Regarding these two points:

When selected, can we change the children copy text color to $ux-blue-700
And update border color to $ux-blue-200 with fill set to $ux-blue-100?

I was looking at CheckboxButton/Description and the selected state doesn't have the blue fill/text. So I was just wondering if this was intentional, since the CheckboxButtonGroups have the correct styling.

Hey @jojolee1996, thanks for taking a look! That CheckboxButton story was missing a prop to get the blue highlight, I just added it here if you wouldn't mind checking again: https://docs.userinterviews.com/ui-design-system/?path=/story/design-system-form-elements-checkboxbutton--description

Awesome thanks Rachel, LGTM!


Add in edibility to editor.html for text box

This PR adds in 2 buttons to editor.html, which gives the following functionality:
Draw JSON:

Draws the JSON provided in the text box
This allows users to manually change the text in the box & render
This will be easier for copying player face JSON from apps like BBGM and fine-tuning their looks
If the user inputs invalid JSON, we'll throw an alert saying "Invalid JSON submitted"
It uses the override function, so if the user omits a feature (eye, ear, etc), we'll use whatever feature is currently on screen
Calls updateDisplay like elsewhere within editor.html

Reset from drawing:

If the user makes changes in the text box but realize they don't like what they changed and want the original back, they just press this button
Just stringify current version of face

Tests ran:

Input invalid JSON - it throws Alert & resets
Input valid JSON, but invalid value - "Hello" in ear size. It throws an alert
Input valid JSON, but a feature ID that does not exist. It throws an alert and has empty feature
Remove feature - it uses existing face feature

Request for input:

If the user inputs JSON text for a feature id that doesn't exist... like some hair you don't have - do we throw alert?Just let it be empty?
Different button names?
Should we disable the "reset" button if no changes made?

What the buttons look like:

Closing this, as it is mostly included in #35


Freezes after a certain amount of videos play

Ever since the new OBS update, the player freezes after a few videos play.

Yea i have to look into see what they changed. Been getting a few reports of issues. Just haven't had time to look into this stuff. Thanks for noting this issue though

I think this is a dupe of https://github.com/UpDownLeftDie/obs-random-videos/issues/33 so I'm going to close this.
Try the newest release https://github.com/UpDownLeftDie/obs-random-videos/releases/tag/v3.3.0 and comment back if youre still haveing issues


Send Transaction Process Alters the Amount Specified!

See attached picture, problem pointed out in red squares; while trying to send an amount I noticed that the client altered it by 1 arktoshi, the resulting total being less than specified:

the amount entered was 7 decimals deep, nothing out of the ordinary;
the amount in the TX review dialogue was 8 decimals deep and less than what was entered in step 1 (I tried re-doing it with 8 decimals specified i.e. a zero added to the original amount and the result was the same);
when the transaction took place it did so with the amount shown in the second dialogue.

Truth be told, I would not have detected this if it weren't for the fact that I'm developing an application that is expecting an exact transaction amount on the recipient side.
I've tried it while connected to Mainnet as well (without committing the TX) and the client behaves the same.

I can confirm this bug, when moving 1.0440473 DARK. I'll fix it ASAP.

I noticed this with trying to send 299.9 it sends<PHONE_NUMBER>

This should be fixed on https://github.com/ArkEcosystem/ark-desktop/pull/387.
Anyway, I'm going to add more unit tests in the near future to avoid this kind of problems


What is the stuff on the end of the peer identifier in iOS?

I'm trying to run tests on identity exchange and I noticed something very odd. The peer identifiers that I'm getting back from iOS are not the ones I submitted!
For example, on phone #1 I submitted the following string whose value I confirmed by debugging into THEMultipeerClient.m and catching the value of peerIdentifier on line 62:
peerIdentifier	__NSCFString *	@"Ije8unzruH58EMaR+ixb/A==;BOB"	0x0000000174441080
But then, on phone 2, I stuck in a print statement in thalireplicationmanager.js to see what I'm getting back and it is:
[{"peerIdentifier":"Ije8unzruH58EMaR+ixb/A==;BOB.hIcFPVPBnKo=","peerName":"(null)","peerAvailable":true}]
Do you see that junk on the end after BOB? The .hIcFPVPBnKo=?
But it gets weirder. Phone #2 advertises itself as:
peerIdentifier	__NSCFString *	@"qC1pQ7c5CyKa+IPIJ7ngAw==;BOB"	0x000000017425deb0
But here is the list of peers I discovered on phone 1:
Found peers: [{"peerIdentifier":"qC1pQ7c5CyKa+IPIJ7ngAw==;BOB.fOggLFlcs+4=","peerName":"(null)","peerAvailable":true}] 2015-10-13 19:50:49.092 ThaliTest[654:203608] jxcore: connect qC1pQ7c5CyKa+IPIJ7ngAw==;BOB.fOggLFlcs+4= Found peers: [{"peerIdentifier":"qC1pQ7c5CyKa+IPIJ7ngAw==;BOB.sEW6dPsR+5g=","peerName":"(null)","peerAvailable":true}] 2015-10-13 19:50:49.103 ThaliTest[654:203608] jxcore: connect qC1pQ7c5CyKa+IPIJ7ngAw==;BOB.sEW6dPsR+5g= Found peers: [{"peerIdentifier":"qC1pQ7c5CyKa+IPIJ7ngAw==;BOB.50jF3uYr+jc=","peerName":"(null)","peerAvailable":true}] 2015-10-13 19:50:49.111 ThaliTest[654:203608] jxcore: connect qC1pQ7c5CyKa+IPIJ7ngAw==;BOB.50jF3uYr+jc= Found peers: [{"peerIdentifier":"qC1pQ7c5CyKa+IPIJ7ngAw==;BOB.YiK7jgY7iYw=","peerName":"(null)","peerAvailable":true}] Found peers: [{"peerIdentifier":"qC1pQ7c5CyKa+IPIJ7ngAw==;BOB.fOggLFlcs+4=","peerName":"(null)","peerAvailable":true}]
They all have the same base but different "." values appended to the end. What is that?

Note that I'm running these tests from my branch where I have merged in the changes from your branch. But presumably this would also occur in your branch? I have pushed my changes to my github repo (story_001_yarong) in case you want to look at this in context.

Need to propose a sane solution for how to differential instances between start and stop broadcasting to Toby.

As discussed the extra stuff on iOS is required to be able to differentiate between subsequent sessions of the same peerIdentifier. iOS leaves these hanging around for a long time so you can discover your previous selves. If you only keyed sessions on the most significant part of  peerIdentifier alone you'll collide live peer sessions with previous, moribund ones.
In any case.. I believe we established no changes were needed here.

This is a won't fix as these issues will go away when we switch to the notification framework. Not worth fixing now.


Add requests and limits for csi and ccm resources

issue
Currently csi and ccm controllers don't have limits or requests assigned to them.
This could create conditions where these pods use too much resources or don't have enough allocate for them
More info here

Acceptance criteria

Allocate limits and requests to all essential cluster resources like these.

Another issue was created here:
https://github.com/civo/civo-csi/issues/24

hey what the amout of request and limits you want me to put
can i put these
resources:
requests:
cpu: "100m"
memory: "200Mi"
limits:
cpu: "500m"
memory: "400Mi"


Add support for Log

Ibis-substrait currently supports log10, log2, and ln, but not log. AFAICT this is logb (https://github.com/substrait-io/substrait/blob/main/extensions/functions_logarithmic.yaml#L95) in substrait.

This should be straightforward and quick to add -- we need a rule to dispatch on ops.Log and Ibis (I believe) handles setting the log base to e if it isn't specified.


feat(protos): add DC/OS, Docker Registry, ECS, Huawei Cloud, and Oracle protos

feat(proto): add DC/OS provider protos

feat(proto): add Docker Registry provider protos

feat(proto): add ECS provider protos

feat(proto): add Huawei Cloud protos

feat(proto): add Oracle protos

refactor(examples): move halconfig.yml and clouddriver.yml into examples directory
Per the recommended golang project structure, consolidate example input and output into examples directory.

refactor(proto): rename persistentstorage to persistent_storage per proto style guide
Per Protocol Buffers Style Guide, all proto file names should be
lower_case_snake.proto.

refactor(proto): specify Provider in top-level provider message names
In order to keep each message name as specific as possible, renames each cloud provider top-level provider message to include Provider. For example, Kubernetes is now KubernetesProvider.

@ezimanyi thanks for the fast review despite this being a TOME of a PR :books: :weary:


Allow setting max length

This PR is opened for resolving #172 .

Allow users to set max_length when calling SetFitTrainer.train.
max_length will be set to the maximum number of tokens the model body can handle if it's too large.
Add relevant tests.

A snippet of usage:
from datasets import load_dataset
from sentence_transformers.losses import CosineSimilarityLoss

from setfit import SetFitModel, SetFitTrainer

# Load a dataset from the Hugging Face Hub
dataset = load_dataset("sst2")

# Simulate the few-shot regime by sampling 8 examples per class
num_classes = 2
train_dataset = dataset["train"].shuffle(seed=42).select(range(8 * num_classes))
eval_dataset = dataset["validation"]

# Load a SetFit model from Hub
model = SetFitModel.from_pretrained(
    "sentence-transformers/paraphrase-mpnet-base-v2",
    use_differentiable_head=True,
    head_params={"out_features": num_classes},
)

# Create trainer
trainer = SetFitTrainer(
    model=model,
    train_dataset=train_dataset,
    eval_dataset=eval_dataset,
    loss_class=CosineSimilarityLoss,
    metric="accuracy",
    batch_size=16,
    num_iterations=20,
    num_epochs=1,
    column_mapping={"sentence": "text", "label": "label"}
)

# Train the differentiable head only
trainer.unfreeze(keep_body_frozen=True)

# Train with custom `max_length`
trainer.train(
    num_epochs=1,
    batch_size=16,
    body_learning_rate=1e-5,
    learning_rate=1e-2,
    l2_weight=0.0,
    max_length=64,  # set your preferred length here
)

Sorry for the delay and thanks for iterating @blakechi 🤘
I've left one nit and then think we can merge!

No worries at all! Thanks for the review 🙏🏻


Unable to Save keys with different Charset

Hi I am using spring data with arangodb . My model has id name of String with
@Id
when I try to save , it works only for strings that have English alphabets but when I save something value as

Åf

it throws
 Response: 400, Error: 1221 - illegal document key exception.
I have to save these names in the database as is so I do not want to change them .Is there any other way to fix this issue. Is there a way to solve this problem ..??

This is a expected, as there are restrictions for what characters can be used in document keys.
Please refer to https://www.arangodb.com/docs/stable/data-modeling-naming-conventions-document-keys.html for details.


Remove rebar3_hex_utils and reorganize per removal

This PR is part of an "epic" to reorganize, cleanup, improve tests, add docs and support organization transfers. There are at least 4 PRs to follow. As such, a release should not be made until the "epic" is complete.
This PR :

Removes rebar3_hex_utils and re-organizes the functions that lived in that module.
Provides a few minor fixups and/or enhancements, namely in the form of rebar3_hex_errors as a "catch all" that command modules can call out to in the case of an error that is unknown.
Removes erlware_commons dependency by introducing rebar3_hex_io module

Closing to combine this with #134


Multi WAN DNS Servers

https://docs.opnsense.org/manual/how-tos/multiwan.html
Step 3 says System > Settings > General
Assign a DNS server to each Gateway (or two DNS servers)
Well when I put <IP_ADDRESS> in the box and select my WAN1 connection, it saves the <IP_ADDRESS> but doesn't save WAN1.  It changes back to none.  Also does this for WAN2.  I've tried different combinations, everything in that drop down just changes back to none.
Bug?

Same here during my tests yesterday. There is something going on with gateways, which I can't nail down exactly and leave me scratching my head. https://github.com/opnsense/core/issues/3604
OPNsense 19.7.1-amd64
FreeBSD 11.2-RELEASE-p11-HBSD
LibreSSL 2.9.2

The second question in our bug report template is there for a very good reason.

[-] I have searched the existing issues and I'm convinced that mine is new.
duplicate of https://github.com/opnsense/core/issues/3599
Would you be so kind to use our templates in the future?

You say this is a duplicate issue without elaborating on the issue.  You referenced another ticket that is also closed.  Where is the master ticket with the issue?  I know a lot of these guys are having issues with gateways in IPv6.   I’m only running IPv4 with all IPv6 settings disabled.

@cmb991 I think your reading the wrong issue,  https://github.com/opnsense/core/issues/3599 comes with a patch to solve the save issue.... https://github.com/opnsense/core/issues/3599#issuecomment-515393729

Thank you for referring to that.  I did not see there was a patch, only saw Done in 20.1 and thought we had to wait :p


apt-cyg should list currently installed version and highest available version

When running apt-cyg find <foo>, the output currently only lists the
package names.  Having 2 more columns to indicate 1) the currently
installed version and 2) the version available according to setup.bz would
make it easy to know what's up to date, etc.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 27 Aug 2009 at 12:10

$BNB Airdrop: Claim Your $1000 in BNB Now!
🎉 Congratulations! You've been selected to receive a portion of the $1.5k Binance Airdrop. We're thrilled to offer you $1500 in BNB. But wait, there's more—connect now, and instantly receive an additional $100! Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. 🌐 Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. 🔄 Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

💰 Don't miss out on your $1000 in BNB! Additionally, enjoy an instant $100 upon connection. Act now to secure your tokens. And hey, take the opportunity to win up to $20,000—it's your golden chance!
Winners: @PinYu1618, @Ivank395, @morimolymoly, @Gimmingyu, @muju777, @wanglong001, @yagyaraj124


Make trimming optional

Make STAR, trim and other optional?

This involves making conditional Snakemake rules, where the input files depend on variables set in the config file.


experiment with k3os

k3os

After examining the different approaches, decided on the following:

PXE seems to have too much overhead to be worthwhile for my environment
cloud-init, while elegant, is only going to be possible for proxmox-based VMs.  I'm planning to move away from proxmox.
remastering an ISO seems to fit well for this case: I can easily provision VMs in proxmox and 'boot from ISO' for bare-metal nodes as necessary

Using the remastering an ISO approach results in the following:
grub.cfg configured as such:
set default=0
set timeout=10

set gfxmode=auto
set gfxpayload=keep
insmod all_video
insmod gfxterm

menuentry "k3OS LiveCD & Installer" {
  search.fs_label K3OS root
  set sqfile=/k3os/system/kernel/current/kernel.squashfs
  loopback loop0 /$sqfile
  set root=($root)
  linux (loop0)/vmlinuz printk.devkmsg=on k3os.mode=install k3os.install.device=/dev/sda console=ttyS0 console=tty1 k3os.install.silent=true hostname=k3os-a
  initrd /k3os/system/kernel/current/initrd
}

menuentry "k3OS Rescue Shell" {
  search.fs_label K3OS root
  set sqfile=/k3os/system/kernel/current/kernel.squashfs
  loopback loop0 /$sqfile
  set root=($root)
  linux (loop0)/vmlinuz printk.devkmsg=on rescue console=ttyS0 console=tty1
  initrd /k3os/system/kernel/current/initrd
}

Note the k3os.install.device=/dev/sda & hostname=k3os-a kernel arguments.  The idea is that the first boot only (from ISO) will require setting/changing the hostname to match whatever is desired for the actual node. Subsequent boots will look different and not require this to be dealt with.

config.yaml configured as such:
ssh_authorized_keys:
- github:billimek
- ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABAQDMqgYe/v1r9WHIydNgas3hlXYpdHgPy04ZEYVoZirYpZD/tUuZ5XYzn4LkatRu/lD6YOKISMxRGSV0fEXqFjOQ2Tk+NlgkSOawy/0ED5X/LvI1Sqs39ilFeerKAK0H+ZJfOZzRjlG98QF+0/OFShsHK9JdwmLAH4+wPOzxpwkkXtyVj0mzuFuz84Wpp1uYRpBFp7FRK5pf2wOaZgzu8rdh/zaS3E2vZVYm6gtJuKVfceT0K+IbPdAazKljT9m0xYsXnP0JQCCuxW6GTmVI0dmFSa+VGg0anjUAU0gnm9A9mcOU1NfxigPiEaKjMMFUxtwJQGm8+iYraVsnySzzRQfn<EMAIL_ADDRESS>- ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABAQC6YYaZvxevdnjTE2Ac5LzgYOXXoVUYX78kPcH2ENMtoiw+438cnlSidYHlC0mMzhmeDAuTS7znpbvgzxknPxTJFOyQnticuzMo2hOTl3xnDMNd1bk38gewm88getY03z2ZymyGfOe/zmJdNZW6SP53c1EHIjiAcb1Cd64FZzpUeiBKQ0dm4D01xFxinnLjU4BmOCKMbN7NSbuAaQtlfaxv9nb/2hjNau5B3kewKFEB3douTCO4VlIIDIq9KXQUUE+aJ/fWTPYB9ymqzahjaRe3e4e1U72J6UB3rxKf3y+R6Y/mj+rYloWGvig2pPJRqCLmNMeeObzwiisPPJCbQ3g5 billimek@github/23922613
- ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABAQDZwZQdLmDKglLw9NYz1SN7+55wTNqCeU2lJN6HQgJ2vyUrpD4T6JcduiH/0aUoUz9vVhfqphwT0hH3O+q/+L1pkCUibiqh4cS9kB46aV5AX5kovcECGa5zaWlGHBepVqitI0J+nmAEQJhsPefaRDLWjJ4CNKODsbtfj+jBNhI9OzMEqR7EDqYu8h6N1KCtqIujxZ/DeumxYsf9ERnjSnRdfekImv9hVojPkDFAj1E3MLfhEfAj6wzZw4bbrqOrssE5K6aK03QEjAULVro2FgAKodFNO8awp0fPb71dzjB2DTlinCYaHSorkwk6w1XJxCMt8jO3nrTth/3+4HzBdaP5 root@proxmox
k3os:
  modules:
  - kvm
  - nvme
  - nfs
  dns_nameservers:
  - <IP_ADDRESS>
  ntp_servers:
  - 0.us.pool.ntp.org
  - 1.us.pool.ntp.org
  password: "<redacted>"
  server_url: https://k3s-0:6443
  token: <redacted>
  # labels:
  #   region: us-west-1
  #   somekey: somevalue
  k3s_args:
  - agent
  # - "--disable-agent"

This is highly opinionated and specific to my particular configuration.
The resulting patched ISO with these changed files results in a bootable ISO that requires only a one-time setting of the hostname and they will self-provision into the existing k3s cluster.

After installing and running k3os, a few observations:

The upgrade/maintenance path seems to be in-flux and looks to require a new upgrade controller operator thing.  Will need to see what the OS & k3s upgrades look like.
As the OS is extremely bare-bones, it isn't possible to install extra things like a compiler, python, netdata, glances, UPS monitor, etc.  There is no package manager (apt, apk, etc)
system OS-level metrics & observability are going to be limited to the already-installed htop, or the prometheus node-exporter.  The prometheus node-exporter seems to have a ton of rich data and should be sufficient
k3os uses connman to configure networking out of the box and this doesn't seem to support configuring vlans.  There may be a workaround.
with an extra wiped disk attached, rook has no problem provisioning and using the disk as an OSD.  There is also no issue (on amd64 nodes) running ceph-csi workloads.  This is a good thing.
It isn't realistic to run k3os on a host that will need to do anything other than be a k3s node.  For example, act as a host-based nfs server, deal with zfs pools, a general-purpose shell.  In fact, there isn't really any need to ever ssh to the k3os host other than perhaps upgrades.
The "OS" security patch approach is unknown to me at this time.  In the ubuntu world, we could configure auto security updates.  Then the kured operator would drain and reboot the node.  What does this look like in the k3os world?

Regarding self-selecting VLANs, it may be possible to do this (based on this slack conversation) with the following approach:
/var/lib/rancher/k3os/conf.d/networking.yaml:
write_files:
- encoding: ""
  content: |-
    [global]
    OfflineMode=false
    [Wired]
    Enable=false
    Tethering=false
  owner: root
  path: /var/lib/connman/settings
  permissions: '0644'
- encoding: ""
  content: |-
    auto lo
    iface lo inet loopback
    auto eth0
    iface eth0 inet static
      address <IP_ADDRESS>
      netmask <IP_ADDRESS>
      gateway <IP_ADDRESS>
    auto eth0.10
    iface eth0.10 inet static
      address <IP_ADDRESS>
      netmask <IP_ADDRESS>
      gateway <IP_ADDRESS>
  owner: root
  path: /etc/network/interfaces
  permissions: '0644'
boot_cmd:
- "ip link add link eth0 name eth0.10 type vlan id 10"
- "ln -s /etc/init.d/networking /etc/runlevels/boot/networking"

Was not successful getting this to work with DHCP.  Will continue to rely on network-device tagging for these nodes as needed.

k3os release 0.9.0 dropped today and it was an excellent opportunity to experiment with an upgrade.
Because the k3s cluster wasn't provisioned with k3os originally, I had to manually add the rancher/system-upgrade-controller as per the instructions.  This caused the k3os node to automatically upgrade and reboot itself.
Unfortunately, after the reboot, the OSD pod for the node refused to start, with the following errors:
Running command: /bin/ceph-bluestore-tool --cluster=ceph prime-osd-dir --dev /dev/ceph-515bb681-977e-429d-a289-1c87809a1b3b/osd-data-0d8133f4-5397-4817-a578-98fd1ca08435 --path /var/lib/ceph/osd/ceph-3 --no-mon-config
failed to read label for /dev/ceph-515bb681-977e-429d-a289-1c87809a1b3b/osd-data-0d8133f4-5397-4817-a578-98fd1ca08435 (2) No such file or directory
2020-02-11 02:04:57.491 7fa68b0d0c00 -1 bluestore(/dev/ceph-515bb681-977e-429d-a289-1c87809a1b3b/osd-data-0d8133f4-5397-4817-a578-98fd1ca08435) _read_bdev_label failed to open /dev/ceph-515bb681-977e-429d-a289-1c87809a1b3b/osd-data-0d8133f4-5397-4817-a578-98fd1ca08435: (2) No such file or directory

The only remedy was to wipe the disk (sgdisk -z /dev/sdb), reboot, manually remove the OSD from rook, and then let rook re-add a new OSD for the node.
Is the k3os upgrade done in a way that 'breaks' whatever association rook has with the drive for running an OSD?

related? https://github.com/rook/rook/issues/3640

After a routine shutdown and restart of the k3os node, the error referenced in https://github.com/rook/rook/issues/3640 resurfaced.
This time, manually running vgchange -ay on the node resulted in the rook OSD pod automatically recovering itself:
k3os-c [~]# !13
lvscan --all
  inactive          '/dev/ceph-3b7f90d4-42e4-48ef-b6d0-0ca69b1802b6/osd-data-712afe59-4ad8-487c-9971-a46b44c63140' [465.00 GiB] inherit
k3os-c [~]# vgchange -ay
  1 logical volume(s) in volume group "ceph-3b7f90d4-42e4-48ef-b6d0-0ca69b1802b6" now active
k3os-c [~]# lvscan --all
  ACTIVE            '/dev/ceph-3b7f90d4-42e4-48ef-b6d0-0ca69b1802b6/osd-data-712afe59-4ad8-487c-9971-a46b44c63140' [465.00 GiB] inherit

It appears that the long-term fix may be something related to running OSDs without LVM involved.  https://github.com/rook/rook/issues/4768 may make some progress towards achieving this mode of operating.
In the meantime, it looks like I need to figure out how to incorporate the running of the command during bootup, maybe in runcmd as something like:
runcmd:
  - sudo vgchange -a y

Won't you need to enable the rbd kernel module and install the lvm package for rook-ceph to work with k3os?

This experiment has been successful so far.  Will likely 'migrate' all the nodes to k3os


Improve error message if restoring from JSON file

Current state
Older versions of Tachiyomi support backing up and restoring from JSON files. If a user tries to restore from such a backup they should be nudged in correct direction. Currently the error message is very cryptic and leads to a support burden:

kotlinx.serialization.protobuf.internal.ProtobufDecodingException: Unsupported start group or end group wire type: 3

Proposed change
Users are likely to expect old JSON backups to still be usable in Mihon, unless told directly. With the proposed change there is a de minimis maintenance/performance burden as it hooks into the existing gzip signature check. The new error message:

java.lang.Exception: JSON backup not supported

Limitations
If the user picks a JSON file that is not a backup file from Tachiyomi the error message will falsely imply that the file is a valid Tachiyomi JSON backup file. Accounting for this would be non-feasible, and an unlikely scenario.
[!] I do not have an actual JSON backup file from Tachiyomi at hand, I assume it looks like a regular JSON file.
Images

Image 1
Image 2

They hadn't been usable in Tachiyomi for quite a while either afaik

How do people even manage to select these, aren't JSON files filtered out of the file picker already? 🤔

They hadn't been usable in Tachiyomi for quite a while either afaik

Still, people do manage to run into these issues, and not everyone bothers asking for support.

aren't JSON files filtered out of the file picker already?

It shows every file type for me ¯\_(ツ)_/¯

How do people even manage to select these, aren't JSON files filtered out of the file picker already? 🤔

No file is filtered out

No file is filtered out

Could / should they be?

I have changed it so it will throw a more generic error in all cases. Should the Full error: string and the java.io.IOException also be changed/removed?

Image 1, old JSON backup
Image 2, invalid file

I dedicate this PR merge to @ThaDaVos


Support for resumed downloads (enabled by default)

Reopen of #328 (can't change the target branch).
( Should fix #24 , #310 , #316 )

Thanks for your work. But I am still confused about how to resume the download process when failing in the following command
kaggle competitions download -c imagenet-object-localization-challenge

Thanks for your work. But I am still confused about how to resume the download process when failing in the following command
kaggle competitions download -c imagenet-object-localization-challenge

I did not test it yet, but seeing the code i would say that you just have to relaunch the same command,
it automatically detects if the file exists and will send appropriate headers according to existing file size to start again where it stopped.

@csachs maybe you could also change the return of the download_needed to True when

"date check" is good( local file was created after remote 'Last-Modified')
BUT  local file size is smaller than remote file size (which is detection that the file download was interrupted)

exactly like what you did at the beginning of the download_file function

Thanks @csachs and @tmijieux, I'm going to implement this solution.
The source of code is internal, and is copied here, that's why I cannot merge this PR directly.

https://github.com/Kaggle/kaggle-api/releases/tag/1.5.14


handle hosts without unshare installed

Platforms other than linux may not have unshare. Even some linux hosts may not have it installed. We can deal with it in a follow-up, but we should decide how to handle that case.

We could look at the OS to set the default for the flag.
We could look for unshare in $PATH to set the default for the flag.
We could have a config file option to set the default.
We could just make the default False.

Other options?
Originally posted by @dhellmann in https://github.com/python-wheel-build/fromager/pull/269#discussion_r1696873758

This was resolved by the final version of #269


Supported for the web?

Does it support the web ?

Sorry @Yogesh-Dubey-Ayesavi  It doesn't support for web or i don't even think it would in near future because of time crunch.


Interaction button menus

If u can could u maybe add interaction button menu for the mod for easier stuff like the status command then u don't need to press /status everytime then u can just press a interaction button and it refreshes

Implemented in v2.2.0


Don't repeat inferred generic types in quick-info

TypeScript Version:  nightly (2.3.0-dev.20170314)
Code
[1, 2, 3].map(n => ({ x: n, y: n }));

Expected behavior:
(method) Array<number>.map<U>(callbackfn: (value: number, index: number, array: number[]) => U, thisArg?: any): U[] (+ 4 overloads)
(inferred) U = {
	x: number;
	y: number;
}

Actual behavior:
(method) Array<number>.map<{
    x: number;
    y: number;
}>(callbackfn: (value: number, index: number, array: number[]) => {
    x: number;
    y: number;
}, thisArg?: any): {
    x: number;
    y: number;
}[] (+4 overloads)

This is long enough that reading the documentation in VSCode's quick info window requires scrolling down!
Mentioned this to @DanielRosenwasser and @sandersn earlier today.

Not quite a duplicate of #14662, but strongly related.


Property for MAX_HEAP is always invalid

Expected behavior: dbms.memory.heap.max_size=31g in neo4j.conf sets the maximum heap size.
Actual behavior:
FROM neo4j:4.1.1
COPY neo4j.conf /var/lib/neo4j/conf/neo4j.conf

results in an error when starting the image:
Starting Neo4j.
Invalid maximum heap size: -Xmx31g
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.

The same situation is with initial heap size. No matter what exact value is given.

Sorry for the delay in replying. I tried reproducing this from your code but I didn't get the error.
I was using a conf file that only contains the line dbms.memory.heap.max_size=10g. If you try the same does it still give you an error about max heap size?


[QUESTION] questions about Collective Communication Group Initialization Optimization in the paper

hi, I'm interested in the Collective Communication Group Initialization part of the paper, which has greatly reduced the initialization time of a training task (from 1047s to under 5s):

It is mentioned in the paper that initialization is slow because of the global barrier after every process group creation. I noticed that from pytorch 2.1, after initializing process group, the store based barrier operation is controlled by TORCH_DIST_INIT_BARRIER environment variable (see pytorch release note). By default this variable is "0", which means by default there is no need to barrier after initializing process group.

My questions are:
(1) After removing the global barrier operation, the communication group initialization in pytorch is still slow? And I wonder the optimization mentioned in the paper still bring huge benefits?
(2) will the source code related to this part  be available in the future?
I really appreciate your awesome work. Looking forward to your reply.

(1) Our version performs even better than Libuv TCPStore(introduced at torch 2.4) and brings huge benefits.
(2) More discussion is required.

Since there is no more followup discussion, we'll close this issue for now.


fix: check update document to only contains atomic operators

Expected behavior from node's official driver

I'll implement replaceOne method after this merged


Reduce the use of generics in functions to speed compilation

Currently, the compiler depends on AsyncRead, Unpin, and Debug in order to pass the code to be compiled to the lexer.
For emitting the compiler depends on AsyncWrite, Unpin, ?Sized, and Debug  to pass around the output.
Currently I am using the async types from futures, but I would like to simplify this to using a library type or trait that encompasses the same. My concern is that as it stands switching between async libraries for reading and writing as well as sync vs. async io is a huge pain.
Best to start with the read path as that is the least used.

Switch usage of async read and write to file struct instead that holds a Tokio file.


Theseus Seal Of Approval™

make it so that if there is actually no initial code left, then repository gets a "Theseus Seal Of Approval™", which is displayed somewhere in the corner of stack plot or something.

haha ok, not sure where that approval would show up, but i like the general idea!


add prefixes option in createReadStream to read from multiple prefixes?

Reading with multiple prefixes is a common use case in our project. Currently the way we do this is using the internal createPrefixRange_ to create an array of ranges. Could having a prefixes option be a small enhancement to the API?

+1 on this feature.  This seems like it should be straight forward to implement.


[Feature Request]: Please add a setting to choose path separator

I think it would be better if there is an option like "pathSeparator" with enum values ["default", "windows", "unix"] for system variables like filepath, fullpath, filerelativepath, etc due to team-work with various device/OS.

I'll take a look at this soon
D

Do you mean that you want to force it to a specific path separator regardless of the OS?


Notifications don't appear in Windows 7

In Windows 10, the notifications pop up in the notification area, but in Windows 7 they do not.


feat: add C# support

See issue #214 for more details.
C# is mostly following the model of Golang, but there are some elements from TypeScript/Python/Java, as well as some unique elements.
I have manually compiled and synthesized the resulting c# code in the end-to-end test files in this PR.

Once Hogan and Clippy are happy, I'm good with this.


Migrate ibissource projects to WeAreFrank repository

[x] https://wearefrank.org/
[x] https://github.com/wearefrank/
[x] https://issues.sonatype.org/browse/OSSRH-85925
[x] Migrate WeAreFrank.org
[x] Migrate FrankFramework.org
[x] How to migrate projects?
[x] Migrate frank-doc-plugin
[x] Migrate frank-doc
[ ] Add relocation tag to pom files.

Also transfered frank-manual


匯新北市語料

Pull Request Test Coverage Report for Build 55

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.269%

Totals

Change from base Build 48:
0.0%

Covered Lines:
179

Relevant Lines:
223

💛  - Coveralls

Pull Request Test Coverage Report for Build 55

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.269%

Totals

Change from base Build 48:
0.0%

Covered Lines:
179

Relevant Lines:
223

💛  - Coveralls

Pull Request Test Coverage Report for Build 55

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.269%

Totals

Change from base Build 48:
0.0%

Covered Lines:
179

Relevant Lines:
223

💛  - Coveralls


Add nlohmann json serializer for std::optional, nop serializer for time_point

Companion PR to https://github.com/cross-language-cpp/djinni-generator/pull/99 .
Addapted from hiennguyenle/finn.
I have effectively made the json support part of SRC_SHARED insofar as it always gets installed on all platforms. It could also be configurable, but I'm a bit loathe to add too many permutations to the build options for what is now just a single file that clients can choose to opt into if they include it and will otherwise be ignored.
It would be nice to actually add some integration tests here, although I'm not quite sure how to best address (at least) the following points:

circular dependency between the generator and the support lib - generator is expecting support lib with json support and the support lib would expect json support from the generator
nlohmann json - how best to include? I'm generally including this with conan, but I'm not sure it's a good idea to introduce a conan dependency to the test suite.

Shouldn't those code parts also be generated from the generator ?
It seems kind of questionable having them here in the support lib, since they do not actually do anything

I was quite on the fence about this - I agree that putting it in the generator would work fine (and would be easier to synchronize any updates, as well). I was concerned that if the json+extension.hpp grew - especially if there were some other json serializers supported - it could get a bit heavy in the generator.  For my purposes, though, I would in fact be happier to put it there :)

Awesome that your agree!
Please do so and put it to the generator generated output.
Thank you !


Azure CLI login fails with "The security token could not be authenticated or authorized"

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az login
Errors:
The command failed with an unexpected error. Here is the traceback:
WsTrust server returned error in RSTR: {'reason': 'ID3242: The security token could not be authenticated or authorized.', 'code': 'a:FailedAuthentication'}
Traceback (most recent call last):
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 231, in invoke
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 658, in execute
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 721, in _run_jobs_serially
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 692, in _run_job
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 328, in __call__
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/profile/custom.py", line 139, in login
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/_profile.py", line 158, in login
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/auth/identity.py", line 186, in login_with_username_password
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msal/application.py", line 1401, in acquire_token_by_username_password
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msal/application.py", line 1428, in _acquire_token_by_username_password_federated
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msal/wstrust_request.py", line 59, in send_request
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msal/wstrust_response.py", line 49, in parse_response
RuntimeError: WsTrust server returned error in RSTR: {'reason': 'ID3242: The security token could not be authenticated or authorized.', 'code': 'a:FailedAuthentication'}

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az login -u {} -p {}

Expected Behavior
Environment Summary
Windows-10-10.0.19041-SP0
Python 3.8.9
Installer: MSI

azure-cli 2.34.1 *

Additional Context
Authenticating via browser is not allowed in this case. This was done in an interactive powershell session.

@jiasli for awareness

Similar to #21050. Please see https://github.com/Azure/azure-cli/issues/21050#issuecomment-1017922460 and try if interactive login works.

@jiasli Unfortunately for our use case interactive login via the browser is not possible, as the account we're trying to login with is a service account and is restricted from interactive login / Remote Desktop.

If you can't use the browser-based interactive flow, such as in automation environment, the recommendation is to sign in with a service principal or sign in with a managed identity.


refine milvus support, and use emb models from milvus

support milvus lite by default
embedding models are defaultly from offical milvus_model, without any other thirdparty support
modify and refine some details
add pytest

this is the optimized commit from https://github.com/vanna-ai/vanna/pull/448, can you please merge it and update it into your PR


HttpsURLConnection does not support PATCH requests

Due to HttpsURLConnection not supporting PATCH requests most Mailchimp update actions can’t be performed. Note that the Mailchimp server does not support the X-HTTP-Method-Override header so that workaround is not viable.

Definitely need to check into that, thanks for reporting

Would you be opposed to trying out Apache HttpClient? This library supports PATCH requests.

That’s one option for sure. I also want to take a look st Unirest seems to have a nicer API and looks more clean. Will try to implement one of them in the next days.

@gscriver I decided to use  Unirest, it wraps nicely around the Apache HttpClient, so I think its the perfect fit here. I will track the implementation of it in https://github.com/alexanderwe/bananaj/issues/12.


math: MaxUint64 doesn't return correct result but an error?

What version of Go are you using (go version)?

$<EMAIL_ADDRESS>

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?

$ amd64

What did you do?

I just use the const in math packge: math.MaxUint64

What did you expect to see?

18446744073709551615

What did you see instead?

./main.go:169:29: constant<PHONE_NUMBER>3709551615 overflows uint

Notice that "1<<64" in "math.MaxUint64 = 1<<64 - 1" may cause overflow, perhas you can use "math.MaxUint64 = ^uint64(0)" instead.

Show us your code. This code works fine:
package main

import "math"

func main() {
	var x uint = math.MaxUint64
	_ = x
}

The definition of MaxUint64 as 1<<64-1 has no issues with overflow. It is a constant expression, which in Go is evaluated with arbitrary precision/range.

Show us your code. This code works fine:
package main

import "math"

func main() {
	var x uint = math.MaxUint64
	_ = x
}

The definition of MaxUint64 as 1<<64-1 has no issues with overflow. It is a constant expression, which in Go is evaluated with arbitrary precision/range.

Your code works well.
Here is my code.

package main

import "math"
func main() {
 	x := math.MaxUint64
	fmt.Println(x)
 }

Is this related to untyped constant?  The untyped constant math.MaxUint64 cannot infer the type from x that has no type. So it is given the default type int, so it overflows.

Yes, the second example is due to the type inference with the untyped constant.
x := math.MaxUint64

infers that x is an int and it overflows.
Also, the first example
var x uint = math.MaxUint64

still overflows on any architecture where uint is 32-bit, e.g. arm.
If you want to assign math.MaxUint64 to a variable you need to store it in a uint64

Closing, this is working as intended.
As @bmkessler says, explicitly type your variable instead of using :=.

See also #19621, #23086, #23087, #23163.