Bring back the upgrade module

Taken from #142

Currently inside Desmos we were not making usage of the governance and upgrade modules. The first one was not used at all, while the second one was plugged in a couple of versions ago but never properly tested.

I've decided to remove them temporary inside v0.4.0 to avoid having any problem inside our testnets, but we should definitely bring them back in the future.

With this issue I want to track the bringing back of the upgrade module.

Note that when bringing back this module, a micro test should also be performed locally in order to test the upgrading feature, and such test steps should be provided to everyone that might want to try it out as well.

After discussing with @bragaz, we've found out that the x/upgrade module has some problems currently:

It will not work unless the nodes run the binary with the --prune=nothing option that makes them archives. This problem is tracked inside issue #5746 inside Cosmos repository.

Currently there are still some minor bugs, especially with transactions commands. This is tracked inside issue #6284 and PR #6285.

It looks like the overall module has some issues open related to it (see here and it's tarted to be released inside Cosmos 0.39.
That being said, I'm going to put this issue on hold until those things have been resolved.

@RiccardoM I have been testing the upgrade module from quite a time now. We have tested it on multiple testnets. Meta issue and test scripts need to be handled, but they are not blockers. The critical issue is with pruning. 0.39 should have a fix for this issue. I can help in testing the features

@anilCSE Is There a date set for the 0.39?

Hi @anilCSE, thanks for your willingness in helping us. I've tried implementing the upgrade module inside our code (this has been done inside the riccardo/upgrade-module branch). Unluckily, I got the following errros when trying to create a proposal:
$ desmoscli tx gov submit-proposal software-upgrade "test" --upgrade-height=100 --from jack
ERROR: invalid proposal content: proposal title cannot be blank [cosmos/cosmos-sdk@v0.38.5/x/gov/types/content.go:38]

$ desmoscli tx gov submit-proposal software-upgrade "test" --upgrade-height=100 --from jack --upgrade-info --help
ERROR: unknown flag: --upgrade-info

Do you have any idea why this might happen? The Cosmos SDK version we're using is v0.38.5

You just need to add "title" flag, it's a mandatory flag for this command.
desmoscli tx gov submit-proposal software-upgrade "test" --title "test software upgrade proposal" --description "something about the proposal here" --upgrade-height=100 --from jack
Then name and title looks bit ambiguous but they have different functionality here.
name suggests the upgrade-name and it will be used inside upgrade-hanlder and should be an unique active proposal name. Title is meta for the proposal.

@anilCSE Thank you very much! :pray: Do you by any chance have the reference to some piece of code that performs some migrations inside an upgrade handler that you used in the past? I would like to understand how to handle possibly breaking changes migrations

@RiccardoM you can check this PRs for some example upgrade handlers:
https://github.com/regen-network/regen-ledger/pull/98
https://github.com/ovrclk/akash/pull/598
These are just example handlers, we don't use them to mint/update stake in general cases. Most of the times the handler will be empty as in: https://github.com/ovrclk/akash/pull/561/files
We can use the handler to update params as well.

@anilCSE Thank you! Do you by any chance have also some upgrade proposals examples that allow to auto-download the new binary?

@anilCSE Thank you! Do you by any chance have also some upgrade proposals examples that allow to auto-download the new binary?

I didn't try that but you can keep the binary information inside info object I guess and cosmosd can take care of the upgrade process automatically.

@anilCSE I've tried setting up cosmosd to properly handle the upgrades, and I've set DAEMON_RESTART_AFTER_UPGRADE=on. When the upgrade height is reached, the new binary is started and I get the following:
E[2020-07-20|15:40:34.636] BINARY UPDATED BEFORE TRIGGER! UPGRADE "test" - in binary but not executed on chain module=main
panic: BINARY UPDATED BEFORE TRIGGER! UPGRADE "test" - in binary but not executed on chain

Do you have any idea why that might happen?

Are you running multiple validators?  Or else, you might be facing issues with pruning. If you are using 0.38.x, make sure to use pruning: nothing and otherwise you will face state load issues for upgrades(https://github.com/cosmos/cosmos-sdk/issues/5746)

@anilCSE I was running a local, single validator node chain. You were right, it was due to me starting the node without specifying --pruning nothing. After setting it the problem was solved and the test upgrade was succesful, Thank you very much! :pray:


Lock contention in task creation/finish handling

When runnning Asakusafw on M3BP on a many core machine, a situation
was observed where system CPU got high.  The shape of system CPU rise
is application dependent, one example is at 80-90% (all cpu busy=100%)
for 5 seconds.
By inspecting the situation, two lock contentions in task
creation/finish handling are identified.
First one is a lock in GLibc code [1] which is called from boost log
to local time local time. This lock is somewhat infamous [2] [3].
Linux perf call graph was recorded, some part of report is at [4].
After replacing local time to utc time by ad hoc patch like [5],
second lock contention of m_physical_graph_mutex in scheduler was
observed. Linux perf output is at [6].
These locks can be highly contended when many short tasks are
created/finished. If these locks can be removed or made fine-grained,
M3BP will be faster in some application executions.
[1] https://sourceware.org/git/?p=glibc.git;a=blob;f=time/tzset.c;h=f65116ce24577db3aca8d3d368c162ac71a8cd13;hb=HEAD#l605
[2] 16145 – localtime_r etc holds lock via __tz_convert https://sourceware.org/bugzilla/show_bug.cgi?id=16145
[3] [IMPALA-3316] convert_legacy_hive_parquet_utc_timestamps=true makes reading parquet tables 30x slower - Cloudera Open Source https://issues.cloudera.org/browse/IMPALA-3316
[4] https://gist.github.com/shino/27be5cf09d64a190975caa12e3c58160
[5] diff on top of commit 9517aa8 :
diff --git a/src/logging/general_logger.cpp b/src/logging/general_logger.cpp
index dc81963..19b4357 100644
--- a/src/logging/general_logger.cpp
+++ b/src/logging/general_logger.cpp
@@ -82,7 +82,7 @@ BOOST_LOG_GLOBAL_LOGGER_INIT(
 {
        auto logger = boost::log::sources::severity_logger_mt<LogLevel>();

-       logger.add_attribute("TimeStamp", log_attr::local_clock());
+       logger.add_attribute("TimeStamp", log_attr::utc_clock());
        logger.add_attribute("ThreadID",  log_attr::current_thread_id());

[6] https://gist.github.com/shino/9290371381ad2e8a3bf743b0ff99918f

Thank you for the detailed report!
First, your patch to reduce impact from timezone conversion is effective to make processing faster.
I will fix the logger to avoid __tz_convert calls.
For the second issue, I wrote a simple benchmark that creates many physical tasks:
https://gist.github.com/logicmachine/f713d79f300e528c3d3ba6abea7c852f
When I run and profile this benchmark on a machine with 16 CPU cores, I confirmed that _raw_spin_lock accounts for several percent of execution time.
I think it is possible to reduce the impact from lock contention in m3bp::Scheduler, because current implementation uses naive algorithms.
I will try to fix for using better structures and algorithms to manage physical tasks.


Bad event hubs connection makes entire host unusable

Steps:

Start with empty site
Create JS http webhook and hit test to verify it's all healthy
Create new function using EventHubTrigger template
Click on the + sign to get at the connections, then click Add New Connection
Create a new connection called foo, with value Endpoint=sb://foo.servicebus.windows.net/;SharedAccessKeyName=mysas;SharedAccessKey=Hello
Complete the Function creation
Go back to your webhook function and try to run it, and it's now busted. You get this alert:

The host log file looks like this:
2016-03-24T00:21:14.818 Reading host configuration file 'D:\home\site\wwwroot\host.json'
2016-03-24T00:21:14.929 Generating 2 job function(s)
2016-03-24T00:21:14.929 Found the following functions:
Host.Functions.EventHubTriggerNodeJS1
Host.Functions.GenericWebhookNodeJS1
2016-03-24T00:21:14.945 A ScriptHost error occurred
2016-03-24T00:21:14.945 No such host is known

I'll take a look at this one this morning.

Closing this because there is nothing that can be done in this repo to improve this futher. PR https://github.com/Azure/azure-webjobs-sdk-script/pull/167 did however improve the error message you'll see. It will now be "Microsoft.ServiceBus: No such host is known."

Much better error now!


Paragraph maxIntrinsicWidth incorrect when text wraps

Steps to Reproduce
Flutter Engine computes incorrect maxIntrinsicWidth when ParagraphConstraints cause the paragraph to wrap. IOW, the following test should fail, but it passes:
  testWidgets('odd Ahem width', (WidgetTester tester) async {
    final ui.ParagraphBuilder builder = ui.ParagraphBuilder(ui.ParagraphStyle(
      fontFamily: 'Ahem',
      fontStyle: FontStyle.normal,
      fontWeight: FontWeight.normal,
      fontSize: 10.0,
    ));
    builder.addText('Test Ahem');
    final ui.Paragraph paragraph = builder.build();
    paragraph.layout(ui.ParagraphConstraints(width: 50.0));
    expect(paragraph.maxIntrinsicWidth, 80.0);

    paragraph.layout(ui.ParagraphConstraints(width: 1000.0));
    expect(paragraph.maxIntrinsicWidth, 90.0);
  });

Specifically, we should be getting maxIntrinsicWidth of 90.0 for both constraints. However, the engine incorrectly returns 80.0 for constraint 50.0.
/cc @Hixie @GaryQian

I could also understand a result of 40.0 given a constraint of 50.0, though we'd have to update the docs to be clearer.

Huh. Looks like updating my fork also automatically applied the Fixes flutter/flutter#21965 from one of the commits again for some reason. Reopening.

Framework tests have been updated during roll. This can be closed.


Fix CI and generate the artifacts for different target databases

Need to generate the artifacts for different target databases in the build script.
Wait for the logger to send telemetry.

Find something interesting.


Parse file version correctly

Use a NumberFormatInfo to parse file version so it can work on any machine

@andreminelli, your request is making a full file modification, you can fix this and make a new pr?

Hi, Fabio.  As a matter of fact, only line 737 was changed (I removed a 'string.Replace' and used a NumberFormatInfo instead).
The remaining must be only a change from spaces to tabs (or vice versa).

If you can adjust this, i´ll tnks
If i merge this, will be as if full modify file


Should there be a default user register mutation?

I noticed there's auth mutations for login and logout, but there doesn't seem to be any to register a new user. I think there should be a register mutation that performs a django validate_password, and let the developer choose between the usual  create mutation and the register one.

Thanks for reporting this interesting idea. This is good question.
User registration is a bit more complex operation. Registration field may have various number of fields. Registration also typically needs some kind of activation. What do you think? What would the register mutation provide and how the activation would be handled?
We should also discuss as a team that where we would like to draw the line. Which features are implemented into strawberry and which are left out. We need to find out which features we want to focus on.
Feel free to leave any comments :)

My idea is that the register mutation should create a user with its is_active field set to False as default, so activation can be handled separately by the developer. It would perform Django's validate_password function so no user with a weak password can be saved in the database (at least through the register mutation). It would be a nice shortcut and variation of  the create mutation.
Regarding the multiple registration fields, I haven't come up with any other solution than letting the developer manually create a User strawberry type.

I saw some attempts to implement JSON Web Token (JWT) authentication for Django Strawberry GraphQL, for example, https://github.com/KundaPanda/strawberry-django-jwt. Any guidance or suggestion on utilising this approach?

My idea is that the register mutation should create a user with its is_active field set to False as default, so activation can be handled separately by the developer. It would perform Django's validate_password function so no user with a weak password can be saved in the database (at least through the register mutation). It would be a nice shortcut and variation of the create mutation.
Regarding the multiple registration fields, I haven't come up with any other solution than letting the developer manually create a User strawberry type.

Technically this would work well. I'd like to hear your and other's opinions whether should we have login, logout, registration, password reset and other auth related mutations as an example and add sample code to the documentation or implement and push them to the project.

I saw some attempts to implement JSON Web Token (JWT) authentication for Django Strawberry GraphQL, for example, https://github.com/KundaPanda/strawberry-django-jwt. Any guidance or suggestions on utilising this approach?

Nice to see that community is evolving. :)
I haven't tried that library. I think it would be good to have generic jwt solution which would work well with both plain strawberry and strawberry-django.

Technically this would work well. I'd like to hear your and other's opinions whether should we have login, logout, registration, password reset and other auth related mutations as an example and add sample code to the documentation or implement and push them to the project.

Making mutations for registering users and logging in and out are something I do pretty much every time I start a new project, so having them as a shortcut would be nice. I think it also makes sense having them implemented, as strawberry_django already has some of these "shortcuts", like the create mutation.

Making mutations for registering users and logging in and out are something I do pretty much every time I start a new project, so having them as a shortcut would be nice. I think it also makes sense having them implemented, as strawberry_django already has some of these "shortcuts", like the create mutation.

Makes sense. We can add signup/register mutation. Would you like to contribute the project? :)

Makes sense. We can add signup/register mutation. Would you like to contribute the project? :)

Of course! I would love to! <3

I think this PR can now be closed since https://github.com/strawberry-graphql/strawberry-graphql-django/pull/45 has now be merged 🤔
@bellini666

I think this PR can now be closed since #45 has now been merged thinking @bellini666

As far as I can tell, you are right. Closing it!


request to not ignore exceptions in S3Helper

Hi!
I'm troubleshooting some intermittent failures with rules involving S3 downloads. It took me a while to figure out that these were S3 download errors, which are silently ignored by snakemake. It'd be great if you could raise the exception or at least print a warning when a S3 download fails....
see: https://github.com/snakemake/snakemake/blob/master/snakemake/remote/S3.py#L280-L292
Thanks!

Fixed with commit 889f2630.

awesome. thanks!


Unable to add syft.js as a dependency to app

Description
Unable to add Syft.js as a dependency in my application.
How to Reproduce

I have added this to my app's package.json

"dependencies": {
<EMAIL_ADDRESS>"https://github.com/openmined/syft.js#eb3564",
    "@tensorflow/tfjs": "^1.3.2"
  },

I keep getting this error.

$~/Desktop/Cryptly$ yarn build
yarn run v1.22.4
warning package.json: License should be a valid SPDX license expression
$ parcel build src/background.js -d dist/src/ -o background --no-minify && npm run copy
:rotating_light:  /home/sachin/Desktop/Cryptly/node_modules/@openmined/syft.js/dist/index.js:2:124: Cannot resolve dependency '@OpenMined/threepio'
  1 | (function (global, factory) {
> 2 |   typeof exports === 'object' && typeof module !== 'undefined' ? factory(exports, require('@tensorflow/tfjs-core'), require('@openmined/threepio')) :
    |                                                                                ^
  3 |   typeof define === 'function' && define.amd ? define(['exports', '@tensorflow/tfjs-core', '@OpenMined/threepio'], factory) :
  4 |   (global = global || self, factory(global.syft = {}, global.tf, global.threepio$1));
  5 | }(this, (function (exports, tf, threepio$1) { 'use strict';
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Expected Behavior
A beginner friendly guide to how to integrate syft.js into there application.
System Information

OS: PopOS
OS Version: 18.04 LTS
Package Manager Version: NPM 6.14.5, yarn 1.22.4

@sachin-101 are you good after https://github.com/OpenMined/threepio/pull/90 fix?

@sachin-101 I'm closing this, presuming there is no longer an issue.

Yeah. It is solved @cereallarceny  :smiley:  Thaks for the quick fix @vvmnnnkv.


Batch normalization and inference in the DCGAN model

I am using the DCGAN code and pretty happy with the results. However, I am curious, should one not treat the Batch Normalization operation in a special way when doing inference (after training is completed) ?
i) the original BatchNorm paper mentions that we need to freeze the mean and variance when doing inference with the model https://arxiv.org/pdf/1502.03167v3.pdf , algorithm 2
ii) the DCGAN does not use this fixing of the statistics of the batch, so when we generate new samples with the _gen function it seem we calculate on the fly the batch norm statistics. This still works and produces nice images, to my surprise
iii) now here is a case when it does not work: start with a black image X and optimize it with respect to the discriminator function to make it close to the "true" images. With few iterations of gradient descent I can get an X image which is predicted as 1 (true), but it looks pretty much also black. So basically, the discriminator seems to be pretty bad in that case, even though the images I can generate are quite good. My guess would be that the batch normalization fails in that case, since the statistics of the single black image are totally different than the statistics of a proper random minibatch.
iv) has anyone implemented a fixing of the mini batch parameters for inference, as advocated in the original paper? This might be an useful option for the DCGAN  code.
v) as next experiment, I will try to remove batch normalization and train without it, and than see whether my black image experiment will work correctly
if anyone has more insights about the use of batch normalization in the DCGAN it will be really helpful to discuss that, or to get the code for a simple modification of DCGAN in order to use fixed batch normalization operation when doing inference.
thanks a lot
Nikolay

You're correct that generation just uses minibatch-statistics (as long as minibatch is non-zero it works fine). Inference is used for the semi-supervised results - an example of inferring/calculating these values can be found in https://github.com/Newmu/dcgan_code/blob/master/svhn/svhn_semisup_analysis.py


[OSF-6968] Removed spurious png file with backslash in name

Purpose
The \.png file cannot be cloned by git on windows. Because the file is unused, it is safe to remove, enabling better platform compatibility.
Changes
Removed the website/static/img/ \.png file.
Side effects
If the seemingly blank file is actually used somewhere, it would not be found. But could easily be recreated with a less offensive name,
Ticket
https://openscience.atlassian.net/browse/OSF-6968

Failed tests all seem to do with preprints. Have nothing to do with the extraneous file being removed.

Suuuuper complicated change, but I think I follow the details. :) I've restarted the second test, since it really shouldn't be failing based on this change. Once it passes, this is ready to go. 🎉 🐧 🎉


Indentation size and Tab changed to 3 after comment block

Type: Bug

Create a C source file, blah.c, make sure that tab size is 4

Create a function with the following content:

int32_t remove_path(const char* oldPath) {

}

Add an indented block comment (indented by one tab that is 4 spaces in my setting), so this becomes:

int32_t remove_path(const char* oldPath) {
    /*
    */
}

Put cursor after the last '/', as if just typed, press Enter

Observe that the cursor of next line does NOT follow indentation (should be 1 tab/4 spaces), instead it is indented by 3 spaces.

Press Backspace to move cursor to beginning of this new line, press Tab

Observe that Tab is also reduced to 3 spaces

VS Code version: Code - Insiders 1.77.0-insider (833ac685084e2028b09753392b82c641c7025bbc, 2023-03-21T05:25:51.950Z)
OS version: Linux x64 5.15.0-67-generic snap
Modes:
Sandboxed: Yes

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7600U CPU @ 2.80GHz (2 x 2904)

GPU Status
2d_canvas: unavailable_softwarecanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: disabled_softwaremultiple_raster_threads: disabled_offopengl: disabled_offrasterization: disabled_softwareraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_softwarewebgpu: disabled_off

Load (avg)
1, 1, 0

Memory (System)
3.83GB (2.15GB free)

Process Argv
--no-sandbox --force-user-env --unity-launch --crash-reporter-id f4c6a06a-ab6d-43a5-9dab-aab506eef877

Screen Reader
no

VM
100%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
x11

Extensions (12)

Extension
Author (truncated)
Version

language-x86-64-assembly
13x
3.0.0

isort
ms-
2022.8.0

python
ms-
2023.4.1

vscode-pylance
ms-
2023.3.25

jupyter
ms-
2023.2.1200692131

jupyter-keymap
ms-
1.1.0

jupyter-renderers
ms-
1.0.15

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

cpptools
ms-
1.15.0

hexeditor
ms-
1.9.11

markdown-all-in-one
yzh
3.5.0

A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256198
vslsvsres303:30308271
pythonvspyl392:30422396
pythontb:30258533
pythonptprofiler:30281269
vshan820:30294714
pythondataviewer:30285072
vscod805:30301674
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30404738
cppdebug:30492333
vsclangdf:30492506
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30618038
pylantcb52:30590116
pyindex848:30611229
nodejswelcome1:30587009
pyind779:30611226
pythonsymbol12:30651887
6233i204:30659907
pythonb192cf:30661257
defaultazurewalk:30687958
pythonms35:30671666
0bg3d529:30680467
ecj1e332:30687743

@markus-zhang I cannot reproduce. Does this reproduce when disabling all extensions? Press F1 and then choose Developer: Reload Window with Extensions Disabled
https://user-images.githubusercontent.com/5047891/230508346-17faf618-c456-4ebd-a9af-2c881f691763.mp4

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


tmp remove the include labels

Label should be added back when https://github.com/ome/omero-model/pull/86 is merged and the change integrated in openmicroscopy

Merging so we can fix the build


bireme是怎样保证数据一致性?

kafka消费存在指定多个Partition时, 会不会在执行update时导致先后顺序问题?
bireme消费过程中出现异常,  好像没有看到数据重新打回到kafka, 那么就可能会造成数据丢失不一致的情况.

Good questions.
If you use Debezium as the CDC and configure Kafka with multiply partitions for a topic. Bireme cannot  guarantee the loading order and will throw exception.
It is ok to use Maxwell with multiply Kafka partitions since Maxwell hash binlong entry with table name.
Bireme only commit to Kafka after committing to database. So it is ok to stop/interrupt Bireme at any point and continue later.


Any plan to add meta pseudo labels?

We have implemented meta pseudo labels before, you can find it in TorchSSL.
The reason we didn't add it here is because the re-implementation performance is quite far from those reported in the paper.

I see, thanks!


Consider removing Apple Inc link

Unless this project is getting revenue from Apple, the URL should be replaced in the docs.

Removed :)


add spaces, increase indentation, and fix number order to Pareto notebook

Description

Related Issue

[ ] Closes #
[ ] Related to #

Checklist

[ ] Checked that the pre-commit linting/style checks pass
[ ] Included tests that prove the fix is effective or that the new feature works
[ ] Added necessary documentation (docstrings and/or example notebooks)
[ ] If you are a pro: each commit corresponds to a relevant logical change

Modules affected

[ ] MMM
[ ] CLV

Type of change

[ ] New feature / enhancement
[ ] Bug fix
[ ] Documentation
[ ] Maintenance
[ ] Other (please specify):

📚 Documentation preview 📚: https://pymc-marketing--651.org.readthedocs.build/en/651/

Fixing two things I noticed while opening up the docs here and here

There needs to be only one Header 1 or they show up outside along with the other notebooks
The latex was rendering differently in the notebook than in the sphinx docs. Hence my issue #650

Before:

After:


Missing hover info on "Connect to GitHub Codespaces" under "connect to..." picker

Testing #177702
When I don't have any of the remote extensions installed, I see a "Also installs dependencies - " when hovering over all the items, expect the Connect to GitHub Codepsaces one. Is that intentional?
I was also wondering if there could be a more descriptive text for hover  (e.g. "Install dependencies and connect")?

The Codespaces extension does not define any dependencies which is why it didn't show up.
I love the recommendation "Install dependencies and connect" but one of the feedback items from the team was to ensure we're showing information about all the extensions we might download.
Having said that, the extensions page also does not show all the dependencies when we hit "Install". It's up to the user to open up the extensions pack. So I'm leaning towards not having a tooltip at all.
cc @bamurtaugh

I like that we show which other extensions each path might download. For Codespaces, could there be a hover like Installs the Codespaces extension or Installs the Codespaces extension and no other dependencies (not sure the best wording). Or if you add "Connect" to all of the hovers, they could look something like:

Install dependencies (list dependencies here) and connect for extensions with other dependencies
Connect for extensions without other dependencies

Moving this to next milestone as part of issue tracking and cleaning for the current milestone as we are close to the release.

Leaving this experience as is. We have also added a Learn more info link in the picker to help users. We have also not seen any user issues related to this. Closing this issue.


"!>" (not greater than) generates exception not caught by UdonSharp

This line of code: if (otherNeso.isPanicing && Vector3.Distance(transform.position, otherNeso.transform.position) < 0.4 && interactionType != 3 && interactionStep !> 0)
THe last comparison in this if statement generates an exception:  Apparently !> is not a symbol recognized by UdonSharp.  This generates an unhelpfull error: Error: Could not load signature of VRC.SDK3.Components.MultipleDisplayUtilities:GetRelativeMousePositionForDrag due to: Could not load file or assembly 'UnityEngine.UI, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' or one of its dependencies. assembly:UnityEngine.UI, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null type: member:(null) signature:
It seems that this symbol is not a standard operator, but the errors generated by its use are very unhelpful.  It could be good to insert at least some sort of error message...I spent a few days trying to figure out what the heck was going on >_<

This is a Unity error when loading assemblies and has no bearing on U# as far as I am aware.

What I was trying to say was the error fell through UdonSharp entirely and landed on UnityEngine.UI.

That is a syntax error and shows up fine for me in compiles as expected, make sure you aren't hiding errors in the console.

That's interesting.  Visual Studio Code doesn't seem to mark it as bad syntax...

Generally VS code's C# support for Unity is pretty lacking. I'd suggest Visual Studio, or if you can pay for it, Rider is absolutely worth it.

I've never really run into anything like this error with VSCode before and I'm been doing some advanced things with U#.  In this case I can just use "==" instead anyway


against: [:some_method] doesn't work

Example
class Post < ActiveRecord::Base
  belongs_to :author

  include PgSearch
  multisearchable against: [:title, :author_name]

  def author_name
    author.name
  end
end

When I go to the rails console and run: PgSearch::Multisearch.rebuild(Post), I get the following error:
ActiveRecord::StatementInvalid: PG::UndefinedColumn: ERROR:  column posts.author_name does not exist
LINE 5: ...lesce("posts".title::text, '') || ' ' || coalesce("posts".au...
                                                             ^
: INSERT INTO "pg_search_documents" (searchable_type, searchable_id, content, created_at, updated_at)
  SELECT 'Post' AS searchable_type,
         "posts".id AS searchable_id,
         (
           coalesce("posts".title::text, '') || ' ' || coalesce("posts".author_name::text, '')
         ) AS content,
         '2014-02-07 17:59:01.858489' AS created_at,
         '2014-02-07 17:59:01.858489' AS updated_at
  FROM "posts"

Environment

ruby 2.0.0p353 (2013-11-22 revision 43784) [x86_64-darwin12.5.0]
rails 4.0.2
pg 0.17.1
pg_search 0.7.3

Analysis
This seems to be a bug in https://github.com/Casecommons/pg_search/blob/v0.7.3/lib/pg_search/multisearch/rebuilder.rb#L13-21. If you don't have a rebuild_pg_search_documents method in your model, or if you don't have a condition (if or unless), it always assumes that ":against"  is a list of db columns. This is incorrect since author_name is not a db column.
Moreover, if I add a fake condition to my code, it seems to work fine. i.e: multisearchable against: [:title, :author_name], if: proc{|p| true}. The condition forces the looping and therefore author_name is properly evaluated.
Possible fix
Option 1
I'm not sure if this is the right way to go, but we can add an additional check: if there's a custom method in the against list, then run through the loop for evaluation. Something similar to this, perhaps?
def rebuild
  if model.respond_to?(:rebuild_pg_search_documents)
    model.rebuild_pg_search_documents
  elsif !columns.map{|c| model.column_names.include?(c.to_s)}.all? ||
    model.pg_search_multisearchable_options.key?(:if) ||
    model.pg_search_multisearchable_options.key?(:unless)

    model.find_each { |record| record.update_pg_search_document }
  else
    model.connection.execute(rebuild_sql)
  end
end

Option 2
Move custom methods to a separate "against" list:
multisearchable against: [:title], against_methods: [:author_name]
References

I created a sample rails app with some seeds in case you want to test the error, you can download it here: https://github.com/hesham/pg_search_bug. Just follow README.md for a quick setup.
There's also a StackOverflow discussion around this bug here: http://stackoverflow.com/questions/21395704/pg-search-using-a-dynamic-method-as-the-against-parameter

The same seems to be true using pg_search_scope. Is there any update on how to possible solve this?

+1

+1


Customize Remote SL midi script for Ableton

Hello,
I don't know how to send a pm so i try it this way..
I am a musician and i am building a Ableton Live 9 live rig with my Novation SLMK2 61 key keyboard..
I am looking for someone who can modify the Remote SL midi script that comes default with Ableton Live 9.
I am willing to pay for this!
I find that the default Remote SL midi script is not good enough and i would like to make it simpler, i think my version is very easy to make, but i know nothing of python scripting..
Here is the way i would like it to work:
-I want the 8 endless encoders AND the 8 pots to control the 8 blue hand parameters of a selected Ableton device. So i can use the endless encoders for slower precision moves and the pots for fast moves (like filter cutoff for example)
-I need the SL display to always show the blue hand parameter info of a selected Ableton device. So i don't need mixer information or track names on the display when i touch a fader on the SL.
-I want the 8 SL drum pads to select bank 1-8 of the blue hand parameters of a selected Ableton device. This way i can go instantly to bank 8 without paging up/down. The SL display should display the parameters of the selected bank! I don’t need the bank up/down buttons from the default Remote SL script.
-I don’t need automap, i am not going to use it.
So that's it!
Is this possible?
I tried doing it by using both the default RemoteSL script and by adding a userconfiguration.txt script:
-I replaced the 8 parameter cc's in the user script to the cc's of the 8 Novation SL pots (so now both the endless encoders and the 8 pots controlled the 8 blue hand parameters..this worked fine)
-I added the 8 cc's of the drumpads for the BANK1-8 buttons in the user script. This worked fine and i could switch banks but the SL display didn't follow the selected bank, only the bank up/down button (from the default Remote SL script)
-The Novation display keeps on switching between mixer/track name information (when i touch a fader on the SL) and the 8 blue hand parameters (when i touch a endless encoder or a pot) and i want it to stay on the blue hand parameters. I couldn't fix this.
So thanks, let me know what you think! It looks like a simple modification to me, but i give up on trying this myself..So i am willing to pay for this mod.
I have Ableton 9.7.5 and i am on Mac and Windows.
You can always contact me on my mail<EMAIL_ADDRESS>Thanks again! Jan

Hi Jan!
Thanks for your interest in the project. I have no SLMkII keyboard at hands and there are some small differences how the displays are controlled in comparison to the ZeRO SLMkII. This would be impossible for me to debug as I even had some trouble getting the displays to work with my modification. As you can read in the description of this project the displays are not used at all.
Here are some alternatives that might help you:
http://www.deeflash.com/custom-ableton-live-remote-sl-mk-ii-remotescripts/
https://remotify.io
Also learning python is not that hard and might benefit you on other projects. So if you want to dig into it, here are some links that might be of interest:
http://remotescripts.blogspot.de/2010/03/introduction-to-framework-classes.html
https://github.com/gluon/AbletonLive9_RemoteScripts
https://docs.cycling74.com/max7/vignettes/live_object_model
https://github.com/wiffbi/Selected_Track_Control
Best of luck on your journey :)

Hello, thanks for the answer!
I understand, it seems like a difficult task..
Deeflash already tried ☺  and he doesn't have a Novation to check so it's
hard to debug for him
I also contacted John Carney from Remotify..so far no answer   (and
Remotify doesn' provide display feedback i think?)
Now i am trying to get in contact with the guy that wrote the Ultimate
Control script (from the Ableton forum)..maybe he can help me..
Or i have to find sombody that works with Ableton that can help me..
Strange it is such a undocumented world and there are no manuals or
tutorials, nothing..
Learning Python is indeed something i really would like to do..
But i need this to get up and running fast, and if i can find somebody that
can fix this i can learn from this project!
And it doesn't seem like a hard thing to do!
It's like taking the existing RemoteSL script and just change a couple of
things:
-add the 8 bank select buttons
-make the Novation display follow these bank select buttons (the default
RemoteSL script has a bank up/down button and the display follows that)
-make the 8 novation pots control the 8 blue hand parameters (the default
RemoteSL script has the 8 endless encoders mapped to the blue hand
parameters)
*-all other stuff (automap, mixer controls, ..) can be removed   *
I have built a nice setup around a Novation (see the attached picture) and
i want to start gigging with it..☺
So it would be hard for you to do as you have the Remote Zero mk2 and the
display behaves differently?
I really need help!
Ok i will stop stalking you but if you know anybody that can help me i
would be very thankful!
Anyway thanks for the help!
Jan
On Fri, Feb 23, 2018 at 2:42 PM, wobbleboxx<EMAIL_ADDRESS>wrote:

Hi Jan!
Thanks for your interest in the project. I have no SLMkII keyboard at
hands and there are some small differences how the displays are controlled
in comparison to the ZeRO SLMkII. This would be impossible for me to debug
as I even had some trouble getting the displays to work with my
modification. As you can read in the description of this project the
displays are not used at all.
Here are some alternatives that might help you:
http://www.deeflash.com/custom-ableton-live-remote-sl-mk-ii-remotescripts/
https://remotify.io
Also learning python is not that hard and might benefit you on other
projects. So if you want to dig into it, here are some links that might be
of interest:
http://remotescripts.blogspot.de/2010/03/introduction-to-
framework-classes.html
https://github.com/gluon/AbletonLive9_RemoteScripts
https://docs.cycling74.com/max7/vignettes/live_object_model
https://github.com/wiffbi/Selected_Track_Control
Best of luck on your journey :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/wobbleboxx/ZeRO-SLMkII-MIDI-Remote-Script/issues/1#issuecomment-368012122,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ai_pZ7fTB1XAlNnkjkt600XbDswdnCVzks5tXsBfgaJpZM4SPLFT
.


Open a new page will lock the scroll of the old page

Describe the bug
Open the codesandbox below. The web page should be able to scroll normally. Then click on one of the cards. It'll open a new page. Switch back to the sandbox page and you'll find the whole page can't be scrolled.
Seems like the scroll lock doesn't get released… Also I found this bug only occurs when preventScroll is set to true and in @use-gesture/react v10.2.13.
I think this bug was introduced by 6896094b, which solves #485.
Sandbox or Video
For preview: https://1b8n4s.csb.app/
With editor: https://codesandbox.io/s/late-sound-1b8n4s?file=/src/App.js
https://user-images.githubusercontent.com/16526078/168585768-23ea3ba3-eae3-46c3-b828-81440f00b8fe.MP4
Information:

Use Gesture version: 10.2.13
Device: iPhone 12 mini
OS: iOS 15.4.1
Browser iOS Safari

Checklist:

[x] I've read the documentation.
[x] If this is an issue with drag, I've tried setting touch-action: none to the draggable element.

Indeed. This is must be due to this commit: https://github.com/pmndrs/use-gesture/commit/6896094b312da0efadbdad15e81f4fc0f8dc0b26#diff-bdb226cd6a706170fbb3438b6eefc36f52e23458ac4190aace4c5d11c89d3a11, I knew this would fire back somehow.

v10.2.14 works now. Thanks!


apply http.DefaultTransport DialContext Timeout and KeepAlive to prox…

…y and watch
Rancher frequently loses its proxy and watch connections to AKS clusters leading to timeouts while browsing the UI as well as delayed Catalog App deployments. The likely root cause of these failures is the Basic Azure Load Balancer limitation where it doesn't send tcpReset (RST) after an idle timeout. This leads to zombie client connections.
The fix is to use the defaults from http.DefaultTransport for DialContext Timeout and KeepAlive similar to the fix made for kubectl proxy to address the same issue.
This fix goes along with https://github.com/rancher/norman/pull/247 to address AKS/rancher integration issues.

The change you did here I believe will break something because DialContext is used before Dial and we use a lot of custom dialers that are assigned to the Dial field. I believe all you wish to accomplish is just set KeepAlive field (in general we already set Timeout in a lot of cases I believe, but that is done through the call to DialTimeout, not DialContext). In order to keep the existing mess of custom dialers in place, you just need to change the places in our code that use raw net.Dial. I believe all that needs to change is the following. This will address all connections that are sent directly to the cluster API endpoint or proxied through our tunnel. (Note this diff has changes to norman.)
diff --git a/pkg/dialer/factory.go b/pkg/dialer/factory.go
index 62ab5ebc6..6f6355f34 100644
--- a/pkg/dialer/factory.go
+++ b/pkg/dialer/factory.go
@@ -171,9 +171,10 @@ func hostPort(cluster *v3.Cluster) string {
 }

 func native() (dialer.Dialer, error) {
-       return func(network, address string) (net.Conn, error) {
-               return net.DialTimeout(network, address, 30*time.Second)
-       }, nil
+       return net.Dialer{
+               Timeout:   30 * time.Second,
+               KeepAlive: 30 * time.Second,
+       }.Dial, nil
 }

 func (f *Factory) DockerDialer(clusterName, machineName string) (dialer.Dialer, error) {
diff --git a/vendor/github.com/rancher/norman/pkg/remotedialer/client_dialer.go b/vendor/github.com/rancher/norman/pkg/remotedialer/client_dialer.go
index 1ab6615ab..00675fe55 100644
--- a/vendor/github.com/rancher/norman/pkg/remotedialer/client_dialer.go
+++ b/vendor/github.com/rancher/norman/pkg/remotedialer/client_dialer.go
@@ -16,7 +16,10 @@ func clientDial(dialer Dialer, conn *connection, message *message) {
        )

        if dialer == nil {
-               netConn, err = net.DialTimeout(message.proto, message.address, time.Duration(message.deadline)*time.Millisecond)
+               netConn, err = net.Dialer{
+                       Timeout:   time.Duration(message.deadline) * time.Millisecond,
+                       KeepAlive: 30 * time.Second,
+               }.Dial(message.proto, message.address)
        } else {
                netConn, err = dialer(message.proto, message.address)
        }

Thanks for the input. I will give this a shot.

.

https://github.com/rancher/rancher/issues/14354

@ibuildthecloud @cjellick I'm not having a whole lot of luck with the suggested changes. I removed my changes and applied the changes in factory.go/client_dialer.go, but the result is not the same. Clusters created from within rancher have improved reliability of managing catalog apps, but I'm still seeing errors while browsing the UI. Imported clusters saw no improvement, still seeing degraded management of catalog apps and UI errors.
FWIW, my changes result in the UI and Catalog Apps issues fixed for both created from rancher and import cases, but I understand there is likely some changes in expected behavior that I'm not observing. Any other ideas on what might be missing?

Hmmm, I can't accept the PR as is because it will break RKE based, and "custom" clusters in Rancher.  Let me think more about what might be different here.

@juan-lee Sorry, did you test with your norman changes + my suggested patch?  Your norman changes are still needed, its just this rancher PR that I had an issue with.

@ibuildthecloud I totally understand that we don't want to regress any other cases. I hope there is something we can do to address the custom cluster scenario that doesn't violate your layering.
Regarding testing w/ my norman changes, I did test the proposed changes with and without my norman PR. Same result I described before.

@juan-lee Thanks.  We definitely want this fix, just trying to figure out the best approach.  Let me look at this further.  Are you testing on master or a specific release of Rancher?

@juan-lee Can you confirm.  I believe with this change imported clusters will not use the tunnel anymore.  Are you still seeing traffic going through the tunnel for imported clusters?

@ibuildthecloud I am in the process of developing a new cluster driver for Rancher.  I am seeing an similar issue as @juan-lee except that my cluster-agent is never able to connect to my newly created cluster unless the DialContext is set. Instead I see  "Waiting for API server...":
With this single change, things work:
$ git diff  pkg/clustermanager/manager.go
diff --git a/pkg/clustermanager/manager.go b/pkg/clustermanager/manager.go
index e2b0e33dc..6d69e1067 100644
--- a/pkg/clustermanager/manager.go
+++ b/pkg/clustermanager/manager.go
@@ -256,7 +256,10 @@ func ToRESTConfig(cluster *v3.Cluster, context *config.ScaledContext) (*rest.Con
                Timeout: 30 * time.Second,
                WrapTransport: func(rt http.RoundTripper) http.RoundTripper {
                        if ht, ok := rt.(*http.Transport); ok {
-                               ht.DialContext = nil
+                               ht.DialContext = (&net.Dialer{
+                                       Timeout:   30 * time.Second,
+                                       KeepAlive: 30 * time.Second,
+                               }).DialContext
                                ht.DialTLS = tlsDialer
                                ht.Dial = clusterDialer
                        }

Cleaning up!
This PR should have been closed as we adapted the findings from it in these PRs:

https://github.com/rancher/rancher/pull/17951
https://github.com/rancher/norman/pull/254

The issue we were tracking for testing was this: https://github.com/rancher/rancher/issues/14354


Use docker-compose to deployment get POST http://<IP_ADDRESS>:20080/api/conversation/ 400 (Bad Request)

Deploy use docker-compose.yml, but when I sent a message I got error messages:
index.90447dd7.js:57          POST http://<IP_ADDRESS>:20080/api/conversation/ 400 (Bad Request)

Here's my docker-compose.yml
version: '3'
services:
  client:
    image: wongsaang/chatgpt-ui-client:latest
    environment:
      - SERVER_DOMAIN=http://backend-web-server
      - NUXT_PUBLIC_APP_NAME='ChatGPT UI'
      - NUXT_PUBLIC_TYPEWRITER=true
      - NUXT_PUBLIC_TYPEWRITER_DELAY=100
    depends_on:
      - backend-web-server
    ports:
      - '20080:80'
    networks:
      - chatgpt_ui_network
  backend-wsgi-server:
    image: wongsaang/chatgpt-ui-wsgi-server:latest
    environment:
      - APP_DOMAIN=<IP_ADDRESS>:9000
      #      - DB_URL=postgres://postgres:postgrespw@localhost:49153/chatgpt # If this parameter is not set, the built-in Sqlite will be used by default. It should be noted that if you do not connect to an external database, the data will be lost after the container is destroyed.
      - DJANGO_SUPERUSER_USERNAME=admin # default superuser name
      - DJANGO_SUPERUSER_PASSWORD=password # default superuser password
      -<EMAIL_ADDRESS># default superuser email
      # If you want to use the email verification function, you need to configure the following parameters
#      - EMAIL_HOST=SMTP server address
#      - EMAIL_PORT=SMTP server port
#      - EMAIL_HOST_USER=
#      - EMAIL_HOST_PASSWORD=
#      - EMAIL_USE_TLS=True
#- HTTP_PROXY=http://<IP_ADDRESS>:7890
#      - HTTPS_PROXY=http://<IP_ADDRESS>:7890
    ports:
      - '8000:8000'
    networks:
      - chatgpt_ui_network
  backend-web-server:
    image: wongsaang/chatgpt-ui-web-server:latest
    environment:
      - BACKEND_URL=http://backend-wsgi-server:8000
    ports:
      - '9000:80'
    depends_on:
      - backend-wsgi-server
    networks:
      - chatgpt_ui_network

networks:
  chatgpt_ui_network:
    driver: bridge

Have you set up the API key?

Yes, you are right, I re-deploy my environment, and forgot to mapping sqlite database outside container. After I set my openapi_key and http_proxy in backend-wsgi-server container, seems everything works.

how to set API key please?


Fix documentation of to_simple_string()

It is 16 octets but each octet takes two hex characters to represent, so the function returns
32 characters.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @huonw (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Thanks!


[WIP][SPARK-42298] Assign name to _LEGACY_ERROR_TEMP_2132

What changes were proposed in this pull request?
The PR assigns a more descriptive name to the error class _LEGACY_ERROR_TEMP_2132.
Why are the changes needed?
This change Improves the error framework by making the error name more descriptive.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Tested with existing test cases.

@asl3 Can you reproduce the error from user space? If so, please, add a test otherwise we should convert this to an internal error.

marked as internal error since error is difficult to reproduce from user space

It seems this is a duplicate of https://github.com/apache/spark/pull/40632


Update how-to-grafana-enterprise.md

@maud-lv
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Instructions for standalone version of 3D-DART

Thank you for sending me your software folders. I was able to install 3D-DART on my Mac, however jobnr8-PDBeditor was not created. What is the reason? I now need instructions for running 3D-DART. How do I specify the parameters? In the examples shown in https://haddock.science.uu.nl/dna/DARTusage.html#introduction, under "Download Results,"
I do not see a parameter file that matches the description on the website.
A couple of comments that might help the next person who wishes to install the standalone version of 3D-DART:

When installing the X3DNA files, you need version 1.5. These must be obtained directly from Xiang-Jun Lu.
In the bin directory, you must include EnergyPDNA.exe. This is a 3D-DART file, not X3DNA. The source code and header must be obtained from Brian Jimenez-Garcia.

Is the folder jobnr8-PDBeditor not being created for the example provided in this repository?

I looked more carefully at the directories that were constructed using your repository. The directory names that were created do not match the ones listed on https://github.com/haddocking/3D-DART. Directory jobnr7-PDBeditor was created. There is no jobnr8-PDBeditor.
If I pick a random pdb file in jobnr7-PDBeditor, the DNA is not complete; for example it may only consist of one strand with bases and just the sugar-phosphate backbone of the other strand with no bases. Can you provide instructions on how to run the standalone version of 3D-DART? In the example, what should I expect to see?

I confirm that in the example there is no  jobnr8-PDBeditor, that is dependent of the actual workflow used by 3D-DART. Indeed, the folder description is from a different example, so maybe documentation should be modified accordingly.
I confirm the problem when generating the pdb structures in the last step. this is one of the structures with Pymol:

I'm not the original developer and I don't really have experience on the chemistry behind this problem, so I mention here @amjjbonvin and @marcvdijk which probably could give you a better insight of the problem.
Attached the output of running the example dart.out.zip:
./RunDART.py -w NAensemblebuild -f example/struct_*.pdb > dart.out 2>&1

I would not worry about the way pymol displays the structure.
This is software-dependent. I sticks representation might give a better picture.
Also when introducing bends, distortions will be present.
Those models are usually meant to be refined.


Broaden README use case

This reverts back to the modern_browser? intent outlined in the README
before @ai's changes in PR #4.
In doing so it shows an error unless the browser is matched explicitly.
For example in an app that doesn't support any version of IE, a message
will always be shown.

@dsalahutdinov yeap, seems like it is correct fix


On clicking on the image of Key account manager FTE details are available

Describe the bug
In the client listing page under the key account manager column clicking on the image, results in displaying FTE details of the key account manager.
Refer to screenshot for error

To Reproduce
Steps to reproduce the behavior:

Go to CRM
Click on Clients
Click on the image available under key account manager

Expected behavior
FTE details of the Key account manager should not be displayed to the user

Tested and working fine. Closing


Timeline for productionizing this DB

Hi @xujiajun,
Do we have a checklist, which is being tracked to make this production-ready? What are the major issues we are facing to have it production-ready?

@ishail ,  yes we have checklist, but not public to visit,  and i has completed all short-term issues, the next arrangement is mainly maintenance and repair bugs, etc., as well as planning distributed solutions.
Actually the version 0.4.0 is production ready. Someone has used it in the production environment.

Cool, Thanks for quick update! 👍


s390x native library fails to load

It looks like, after https://github.com/xerial/snappy-java/pull/416, the native library fails to load on a few s390x machines.
Failing OSes include (ones that I was able to check):

RHEL 7, RHEL 8
Ubuntu 16, Ubuntu 18, Ubuntu 20
SLES15 SP3, SLES 15 SP4

The error message says:
libsnappyjava.so: /lib64/libc.so.6: version 'GLIBC_2.32' not found (required by libsnappyjava.so).
Previously used version, <IP_ADDRESS>, was working properly and already included s390x native library:
snappy-java-<IP_ADDRESS>\org\xerial\snappy\native\Linux ├───aarch64 ├───android-arm ├───arm ├───armv6 ├───armv7 ├───ppc ├───ppc64 ├───ppc64le ├───s390x ├───x86 └───x86_64
so I'm thinking: is the change done in https://github.com/xerial/snappy-java/pull/416 really needed?
Thanks for looking into it.

One important addition: <IP_ADDRESS>, which is currently used, is prone to three high-score vulnerabilities:
https://github.com/advisories/GHSA-pqr6-cmr2-h8hf
https://github.com/advisories/GHSA-fjpj-2g6w-x25r
https://github.com/advisories/GHSA-qcwq-55hx-v3vh
which is the reason I've tried updating to a version that has those vulnerabilities resolved.

Without #416, I can't build any native libraries for s390x by myself.
I can accept a PR for adding native libraries built with an older version of Linux like RHEL7. Run make native and create a PR that contains an updated native library for s390x.

The latest version might have fixed this issue.

PR #481 would fix this issue on s390x.
@xerial Could you please take a look? Thank you very much!

Thanks all for the contributions! I just released snappy-java <IP_ADDRESS> with this fix

Thanks @xerial ! I just tested the newly released libsnappyjava.so for s390x, it worked fine on Linux distributions mentioned in this issue as well as newer ones.

@xerial @kun-lu20 I also checked on RHEL 7 and Ubuntu 20 - works as expected. Thanks a lot!

@xerial , could you please help fix this issue on ppc64le as well? Opened #512


add logo generator

This is sort of finished externally but needs to be integrated into the tool.

this is implemented and live


understand why the CGI masterVar files have fewer non-reference alleles than the corresponding output from cgatools mkvcf

I think this is due to the fact that cgatools mkvcf is using not only the masterVar data but also some additional data as input.  But understand this better and describe it in the data story.

This is obsolete since we are now loading the data directly from the CGI masterVar files.
http://googlegenomics.readthedocs.org/en/latest/use_cases/discover_public_data/pgp_public_data.html#bigquery-pgp-tables


Add Cros-nest to persistent peers

Signed-off-by: David Pierret<EMAIL_ADDRESS>

Closing since I added it manually


buglets with documentation of units in results.json

The results.json file contains, after a fashion, documentation on all of the measured quantities.  Phew!
The documentation is in places inconsistent or incomplete.
Printing of benchmarks/cpu/raw prints far more digits of precision than the underlying system is capable of measuring.
Some times are reported in ms, and other times in us, and rates in per-second.  I'd pick either ms or us, and stick with per-second.
Pedantic: if you report in us, perhaps use UTF8 mu symbol rather than Latin u.
Many, but not all, longDescriptions have units.  All should have units, replicating what's already in shortDescription.  See benchmarks/build-time, benchmarks/start-time, benchmarks/published-size where the units are omitted.
Some shortDescriptions use '(ms)' but the longDescription says 'in ms'. Use the style '(...)' uniformly.
Are sizes (such as benchmarks/swap) reported in SI Mega (1e6) or Mibi (1024*1024) units.  This may be tricky.
The "format" is always given as "n0".  I had to look n0 format up. That's irrelevant, since this is json, and json doesn't support using ',' to group digits.  I don't see the point in reporting "format".
What happens if the number being reported is mis-computed upstream as Inf or NaN. What happens when it is printed in the json file?  (JSON doesn't support Inf or NaN...)

The results.json file contains, after a fashion, documentation on all of the measured quantities. Phew!

You can exclude these from the results file by using --no-metadata and --no-measurements respectively

The documentation is in places inconsistent or incomplete.

Thanks for your recent help on that. Please file specific issues when you find some.

Printing of benchmarks/cpu/raw prints far more digits of precision than the underlying system is capable of measuring.

https://github.com/dotnet/crank/issues/100

Some times are reported in ms, and other times in us, and rates in per-second. I'd pick either ms or us, and stick with per-second.

I am following what dotnet counters export. And some clients (bombardier, wrk) use varying units too.
But filed this issue to follow up, we might be able to decide what common unit to use https://github.com/dotnet/crank/issues/101

Pedantic: if you report in us, perhaps use UTF8 mu symbol rather than Latin u.

Noted

Many, but not all, longDescriptions have units. All should have units, replicating what's already in shortDescription. See benchmarks/build-time, benchmarks/start-time, benchmarks/published-size where the units are omitted.
Some shortDescriptions use '(ms)' but the longDescription says 'in ms'. Use the style '(...)' uniformly.

Created https://github.com/dotnet/crank/issues/102
I just copied the dotnet counters descriptions. Could be better

Are sizes (such as benchmarks/swap) reported in SI Mega (1e6) or Mibi (1024*1024) units. This may be tricky.

Noted

The "format" is always given as "n0". I had to look n0 format up. That's irrelevant, since this is json, and json doesn't support using ',' to group digits. I don't see the point in reporting "format".

This format is not for description purpose, but for output formatting on the controller. It's just UI metadata.
Note that right now it also has data with n02. And json is also used for Bombardier's raw results.

What happens if the number being reported is mis-computed upstream as Inf or NaN. What happens when it is printed in the json file? (JSON doesn't support Inf or NaN...)

It should be stored as a string in this case.


$(configurationName) not replaced before checking for illegal characters

Do you want to request a feature or report a bug?
Report a bug
What is the current behavior?
Using the same configuration as with Unicorn 3, I get the following exception in Application_Start():
Illegal character $ in tree root physical path D:\Company_Web\Sitecore\serialization$(configurationName)\T_Company. If you're using Unicorn, you may need to specify a 'name' attribute on your include to make the path a valid name.
If the current behavior is a bug, please provide the steps to reproduce.
Upgrade from Unicorn 3 to Unicorn 4 while keeping existing configuration files that worked under Unicorn 3. Nothing else changed except for updating to Unicorn 4 with Nuget package manager console.
What is the expected behavior?
I expect the $(configurionName) placeholder to be replaced by the actual configuration name. This way, it should no longer contain a dollar character when checking for illegal characters and thus it should no longer throw an exception.
Please mention your Sitecore version and Unicorn version.
Sitecore 8.1 rev. 151207 and Unicorn version 4.0.0-Pre04

Fixed in 4.0.0-pre05

This will be in 4.0.


Improvement for Mac OS

This is to close #45
Thanks!

Looks like something's weird with zsh! I haven't had a lot of time to peek at this, but there may be some other way ($(()) maybe?) which will work on that shell as well

zsh is indeed weird!
I couldn't make it work with math expressions.

Nice, I wouldn't have even thought of that! Looks good to me 🎉


Help me please

Mac OS big sur 11.3
-python setup.py install

I installed the package pycrypto manually (from repository https://github.com/pycrypto/pycrypto):
git clone git@github.com:pycrypto/pycrypto.git
cd pycrypto
python setup.py install


Require fewer things in the MergeAdapter

https://play.prx.org/e?tt=Testing Standalone&ts=test test test test&ui=%2F%2Fs3.amazonaws.com%2Fproduction.mediajoint.prx.org%2Fpublic%2Fcomatose_files%2F4625%2Fprx-logo_large.png&us=https:%2F%2Fbeta.prx.org%2Fstories%2F219680&ua=https:%2F%2Fcms.prx.org%2Fpub%2F989e07002296698426cb2fb2f64b7f9d%2F0%2Fweb%2Faudio_file%2F1529757%2Fbroadcast%2F278_The_Athletic_Brassiere.mp3

not sure what is broken, assigning an estimate of 3

Original issue:

https://play.prx.org/e?tt=Testing Standalone&ts=test test test test&ui=%2F%2Fs3.amazonaws.com%2Fproduction.mediajoint.prx.org%2Fpublic%2Fcomatose_files%2F4625%2Fprx-logo_large.png&us=https:%2F%2Fbeta.prx.org%2Fstories%2F219680&ua=https:%2F%2Fcms.prx.org%2Fpub%2F989e07002296698426cb2fb2f64b7f9d%2F0%2Fweb%2Faudio_file%2F1529757%2Fbroadcast%2F278_The_Athletic_Brassiere.mp3


Added fixes related to some issues

#132 #127 #128 #126
@kenvandine

@kenvandine  Tell me man, what you need to accept these updates. They really improve performance.

@adderly Ken VanDine, [18.08.16 10:52]
Just looked at it again, not github complains there's a conflict.  Please fix the conflict and I promise to review it :)


How to run this repo in Windows?

Please share the steps to run this repo on WIndows

I do not have access to a Windows machine. Perhaps one of the other authors has, but please share any instructions, if you have it figured out.

We tried cloning the repo and run the make file using git command prompt, but we are getting the error.


Support universal pathlib paths for cloud storage solutions like s3

The API is written to use Path but the impl uses things like shutil.copy2 which require it to be type(object)==Path not just a subtype isinstance(object,Path). mypy static type analysis won't complain, but the type hints aren't quite specific enough in the downstream libraries.
Definitely a MemoryPath object fails to be copied by shutil.copy2.

closed with #21


HtmlElementView Scroll Onclick event conflict

Is there an existing issue for this?

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

Steps to reproduce
This is an inevitable problem, I wrote a demo for reference.https://github.com/zjrwdl/html_element_view_scroll_issue
In this issue https://github.com/flutter/flutter/issues/68903, Although the sliding problem can be solved, it leads to the problem that the player cannot be played. @bwysistemas
In my sample, You can use the sd parameter usePointerIntercept to control whether to enable point_intercept
 IframeYoutubePlayer(       url: url,       usePointerIntercept: false,   ),
@ditman
Expected results
My expectation is that when the mouse stays in the youtube list, it can scroll normally, and at the same time click the youtube video play button to play normally
Actual results

The default result is that the mouse stays on the youtute list and cannot scroll
If the point_intercept component is used, the scrolling problem can be solved, but the youtube player cannot be clicked to play normally

Code sample
https://github.com/zjrwdl/html_element_view_scroll_issue
Screenshots or Video
before use point_intercept

after use point_intercept:

Logs
No response
Flutter Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.10.2, on macOS 11.6 20G165 darwin-x64, locale zh-Hans-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.1)
[✓] Connected device (2 available)
[✓] Network resources

Hi @zjrwdl. Thanks for filing this. I am able to reproduce the conflict on the latest master and stable channels. This might be similar/related to https://github.com/flutter/flutter/issues/124799. Can you confirm it is?

你好@zjrwdl。谢谢您提交此文件。我能够在最新的频道上消除master冲突stable。这可能与#124799类似/相关。你能确认是吗？

I read the case you mentioned, and I don't feel exactly the same.
First of all, I hope to solve the sliding problem of htmlelementview; but after using point_intercept, the problem of click event conflict is introduced.

Hi @zjrwdl. Thanks for confirming. I am able to reproduce this issue on the latest master and stable channels. I can also see that scrolling works without PointerInterceptor on Safari browser.
On Chrome web, using PointerInterceptor enables scrolling but conflicts with onClick event
https://github.com/flutter/flutter/assets/53122008/bfead112-60aa-43af-b7a6-65c16a0d2051
stable, master flutter doctor -v

[✓] Flutter (Channel stable, 3.10.6, on macOS 13.0 22A380 darwin-arm64, locale
    en-NG)
    • Flutter version 3.10.6 on channel stable at
      /Users/damilolaalimi/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f468f3366c (4 weeks ago), 2023-07-12 15:19:05 -0700
    • Engine revision cdbeda788a
    • Dart version 3.0.6
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.80.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (4 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        •
      android-arm64  • Android 14 (API 34) (emulator)
    • iPhone 14 (mobile)          • 1A122DE2-0CAB-4C3E-A395-691BF27D626F • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)             • macos                                •
      darwin-arm64   • macOS 13.0 22A380 darwin-arm64
    • Chrome (web)                • chrome                               •
      web-javascript • Google Chrome 115.0.5790.170

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[✓] Flutter (Channel master, 3.14.0-2.0.pre.19, on macOS 13.0 22A380 darwin-arm64, locale en-NG)
    • Flutter version 3.14.0-2.0.pre.19 on channel master at /Users/damilolaalimi/fvm/versions/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision a50776e5b8 (2 hours ago), 2023-08-10 02:29:24 -0400
    • Engine revision 9117ff27d9
    • Dart version 3.2.0 (build 3.2.0-53.0.dev)
    • DevTools version 2.26.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.80.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (4 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        • android-arm64  • Android 14 (API 34) (emulator)
    • iPhone 14 (mobile)          • 1A122DE2-0CAB-4C3E-A395-691BF27D626F • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)             • macos                                • darwin-arm64   • macOS 13.0 22A380 darwin-arm64
    • Chrome (web)                • chrome                               • web-javascript • Google Chrome 115.0.5790.170

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Hi @zjrwdl. Thanks for confirming. I am able to reproduce this issue on the latest master and stable channels. I can also see that scrolling works without PointerInterceptor on Safari browser.
On Chrome web, using PointerInterceptor enables scrolling but conflicts with onClick event
Screen.Recording.2023-08-10.at.08.59.09.mov
stable, master flutter doctor -v

Is there any solution, I feel this is a tricky problem

I understand that this might not be a great solution but have you tried to display a thumbnail image of the video instead of the HtmlElementView, and when the user clicks, replace the image by the actual embedded video?


[WIP] - add list assets / asset_pairs 🥳🥳🥳

Codecov Report
Patch coverage: 65.11% and project coverage change: -6.69 :warning:

Comparison is base (71f5be2) 86.57% compared to head (825d5a5) 79.88%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #2      +/-   ##
==========================================
- Coverage   86.57%   79.88%   -6.69%
==========================================
  Files           5       10       +5
  Lines         767      676      -91
==========================================
- Hits          664      540     -124
- Misses        103      136      +33

Impacted Files
Coverage Δ

src/stm.rs
10.00% <10.00%> (ø)

src/stm/state_unknown.rs
53.12% <53.12%> (ø)

src/ui/core.rs
88.72% <55.88%> (-11.28%)
:arrow_down:

src/stm/state_search.rs
58.78% <58.78%> (ø)

src/stm/state_help.rs
62.50% <62.50%> (ø)

src/stm/state_debug.rs
67.44% <67.44%> (ø)

src/stm/state_home.rs
77.55% <77.55%> (ø)

src/app.rs
97.77% <100.00%> (-2.23%)
:arrow_down:

src/stm/events.rs
100.00% <100.00%> (ø)

src/ui/list_stateful_widget.rs
100.00% <100.00%> (ø)

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

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


vertical option is not working...

https://jsfiddle.net/954eve2L/2127/
the slider fails to load properly if I set vertical option to true.

If you don't assign an explicit height to the container div on which the slider lives, then it can't work properly.

... I was hoping that there would be a simple way of toggling between vertical and horizontal orientation... oh well. Thanks.


github automerge added for all contributors

What is the change?
GitHub auto-merge action added for all contributors
Related issue?
issue: #139
How was it tested?
In my Github project.
You can change the below code according to your requirements.
name: Node.js CI
uses: "actions/checkout@v2"

Checklist:
Before you create this PR, confirm all the requirements listed below by checking the checkboxes [x]:

[x] Have you followed the Contribution Guidelines while contributing.
[x] Have you checked there aren't other open Pull Requests for the same update/change?
[x] Have you made corresponding changes to the documentation?
[x] Your submission doesn't break any existing feature.
[x] Have you tested the code before submission?

Screenshots or Video:
Add a screenshot or demo video if appropriate.

It will work for all contributors.

Also the base branch of the PR is master which I had advised against in the previous PR, base branch for PR needs to be develop.


C6 support

Add CM 6 support.
Add install_python27.sh.
Add install_psycopg2.sh.
Add add services/c6/install_mysql.sh.

@jprostko Review?


Marlin Compatibility

Do I have to use the Direct Step branch of marlin? is that the only branch that will work?
Changing monitoring state from "Offline" to "Opening serial connection"
Connecting to Step Daemon
Step Daemon is still updating.
Changing monitoring state from "Opening serial connection" to "Error: Connection error, see Terminal tab"
Changing monitoring state from "Error: Connection error, see Terminal tab" to "Offline (Error: Connection error, see Terminal tab)"
Unexpected error while connecting to serial port /dev/ttyUSB0, baudrate 115200 from hook stepd: Exception: 'Step Daemon is still updating.' @ comm.py:_open_serial:3670
Changing monitoring state from "Offline" to "Opening serial connection"
Connecting to Step Daemon
Changing monitoring state from "Opening serial connection" to "Connecting"
Connected to: <octoprint_stepd.StepdService.StepdService instance at 0x6119a328>, starting monitor
Send: N0 M110 N0*125
Recv: warn:failed to load ./bedlevel.json: open ./bedlevel.json: no such file or directory
Send: N0 M110 N0*125
Recv: info:config processed
No answer from the printer within the connection timeout, trying another hello
There was a timeout while trying to connect to the printer
Changing monitoring state from "Connecting" to "Offline (Error: )"
Connection closed, closing down monitor
Changing monitoring state from "Offline" to "Opening serial connection"
Connecting to Step Daemon
Changing monitoring state from "Opening serial connection" to "Connecting"
Connected to: <octoprint_stepd.StepdService.StepdService instance at 0x6c83f1c0>, starting monitor
Send: N0 M110 N0*125
Recv: warn:failed to load ./bedlevel.json: open ./bedlevel.json: no such file or directory
Recv: info:config processed
Send: N0 M110 N0*125
Unexpected error while reading serial port, please consult octoprint.log for details: ValueError: 'I/O operation on closed file' @ comm.py:_readline:3831
Changing monitoring state from "Connecting" to "Offline (Error: ValueError: 'I/O operation on closed file' @ comm.py:_readline:3831)"
Connection closed, closing down monitor
Changing monitoring state from "Offline" to "Opening serial connection"
Connecting to Step Daemon
Changing monitoring state from "Opening serial connection" to "Connecting"
Connected to: <octoprint_stepd.StepdService.StepdService instance at 0x6c464490>, starting monitor
Recv: warn:failed to load ./bedlevel.json: open ./bedlevel.json: no such file or directory
Send: N0 M110 N0*125
Recv: info:config processed
No answer from the printer within the connection timeout, trying another hello
Send: N0 M110 N0*125
There was a timeout while trying to connect to the printer
Changing monitoring state from "Connecting" to "Offline (Error: )"
Connection closed, closing down monitor

Sorry, i just realized the README still points to the dev branch. Any new 2.0.x version is fine. The README does need some additional cleanup.

I got the exact same error on Marlin <IP_ADDRESS> and 2.0.7.
It seems like not every version is compatible.
Does anyone know wich version to use or how to fix this?

Hmm very interesting indeed. The version im running now is: https://github.com/MarlinFirmware/Marlin/commit/131ddeaeba96addcf573cbeff86b65225d3b8320
It'd definitely be interesting to see if the problem persists on that version. If it does, then there might be some other config interaction at work here.

Just installed the marlin version you have provided, but can't get it to work..
Keep getting those two errors:
Changing monitoring state from "Offline" to "Opening serial connection"
Connecting to Step Daemon
Changing monitoring state from "Opening serial connection" to "Connecting"
Connected to: <octoprint_stepd.StepdService.StepdService instance at 0x6857d558>, starting monitor
Send: N0 M110 N0*125
Recv: warn:failed to load ./bedlevel.json: open ./bedlevel.json: no such file or directory
Recv: info:config processed
Send: N0 M110 N0*125
Unexpected error while reading serial port, please consult octoprint.log for details: ValueError: 'I/O operation on closed file' @ comm.py:_readline:3831
Changing monitoring state from "Connecting" to "Offline (Error: ValueError: 'I/O operation on closed file' @ comm.py:_readline:3831)"
Connection closed, closing down monitor
Changing monitoring state from "Offline" to "Opening serial connection"
Connecting to Step Daemon
Changing monitoring state from "Opening serial connection" to "Connecting"
Connected to: <octoprint_stepd.StepdService.StepdService instance at 0x69294cd8>, starting monitor
Send: N0 M110 N0*125
Recv: warn:failed to load ./bedlevel.json: open ./bedlevel.json: no such file or directory
Send: N0 M110 N0*125
Recv: info:config processed
No answer from the printer within the connection timeout, trying another hello
Send: N0 M110 N0*125
There was a timeout while trying to connect to the printer
Unexpected error while writing to serial port: IOError: '[Errno 32] Broken pipe' @ comm.py:_do_send_without_checksum:4686
Changing monitoring state from "Connecting" to "Offline (Error: IOError: '[Errno 32] Broken pipe' @ comm.py:_do_send_without_checksum:4686)"
Connection closed, closing down monitor
Changing monitoring state from "Offline" to "Opening serial connection"
Connecting to Step Daemon
Changing monitoring state from "Opening serial connection" to "Connecting"
Connected to: <octoprint_stepd.StepdService.StepdService instance at 0x69282198>, starting monitor
Send: N0 M110 N0*125
Recv: warn:failed to load ./bedlevel.json: open ./bedlevel.json: no such file or directory
Send: N0 M110 N0*125
Recv: info:config processed
Unexpected error while reading serial port, please consult octoprint.log for details: ValueError: 'I/O operation on closed file' @ comm.py:_readline:3831
Changing monitoring state from "Connecting" to "Offline (Error: ValueError: 'I/O operation on closed file' @ comm.py:_readline:3831)"
Connection closed, closing down monitor

Can you provide a config that is known to work?

I'm hitting the same thing.  Any idea what might be up?  I'm on an Ender 3 Pro

I'm hitting the same thing.  Any idea what might be up?  I'm on an Ender 3 Pro

Sorry for the delay, finally got around to updates, including fixed support for python2/3 octoprint. This has been tested with the latest 2.0.x version of Marlin.
You should be able to grab the new OctoPrint plugin here: https://raw.githubusercontent.com/colinrgodsey/maven/master/step-daemon/octoprint-plugin/latest.zip and you should be able to install it over the existing plugin and reset.
Let me know if there's still issues!

Sorry for the delay, finally got around to updates, including fixed support for python2/3 octoprint. This has been tested with the latest 2.0.x version of Marlin.
You should be able to grab the new OctoPrint plugin here: https://raw.githubusercontent.com/colinrgodsey/maven/master/step-daemon/octoprint-plugin/latest.zip and you should be able to install it over the existing plugin and reset.
Let me know if there's still issues!


Google Tag Manager pushState/replaceState

From @koenpunt on January 25, 2016 16:31
On a regular page load, bothe pushState and replaceState are called, which causes duplicate page views in the Google Tag Manager for the event "History change".
Is this intentional? Or is there something I missed in the configuration?
Copied from original issue: rails/pjax_rails#79

Could the error be on this line: jquery.pjax.js#L291
Where it maybe should be:
if (!options.push || options.replace) {

Sorry, I don't know what Google Tag Manager is! But you're right that both pushState and replaceState are called. This is for technical reasons. This should't be a problem because the library works fine this way.

Maybe not a problem for the library, but can be for where it's implemented. I can't recall right away how I worked around this, since it has been more than a year, but I'll get back to this topic.
And google tag manager is the de facto standard of managing tracking scripts, analytics and other measuring tools. So having duplicate impressions there due to a library is a problem.
https://www.google.com/analytics/tag-manager/

@koenpunt  @mislav Any updates on how you used Pjax and Tag Manager? I'm facing the same problem and would like to fix it for future projects.
Thanks


Support Riak 2.0

/cc @jaredmorrow
New config - need specification.
Suggest that we stop supporting puppet 2.7 and its hiera in this next version.

@danieldreier, as mentioned on #48, this is a major update and - to me - the main goal. Let's use this (overdue) conversation to keep track of progress on 2.x

Requires feedback from #51 before it can progress to ready.

I put together an experimental riak 2-only module last night to see what could be done if we only support puppet 3.7 but without a future parser dependency. It installs riak 2 on CentOS 6 and Debian Wheezy and has pretty extensive rspec-puppet test coverage and basic beaker test coverage. I don't intend that as a fork of this project, but part of me wonders whether it might make sense to build a new riak 2 module in parallel to maintaining this one.
I'm hesitant to make sweeping PRs that make breaking changes to this one, because I'm assuming that 100% of the users of this module use it for Riak 1, and nearly 100% of the new users who are looking for a riak module will be looking to use Riak 2.

@danieldreier I'd be glad to cut a version as it stands right now with 1.3.x support and then move forward with the 2.x goal. That's absolutely the intention from a PM POV.

I agree with moving forward with a 2.x only module, but a little concerned about only supporting puppet 3.7+. We are still using 2.7, although we are planning to upgrade soon.
So If I were to contribute, is this danieldreier/riak2 module the one to use now?
I'm a bit new to the Github workflow. Would these be the basic steps to follow?

Fork this repo
Hack
Commit
Push
Add an Issue to discuss/approve the change
Submit a PR

Is that correct?

@Cornellio I'm really only using my danieldreier/riak2 module as a development testbed, to see what I can do without being constrained by riak 1 or older puppet version compatibility, and without having to add tests onto older code; I'm hoping to wholesale forklift that code into this one after meeting feature-parity and incorporating the features that your and other modules have that I lack. For example, I've been realizing that the system settings are less optional than I had thought.
I should really add a CONTRIBUTING.md file documenting this. The workflow I'd suggest is:

Fork repository
Create feature branch
Hack
Run rspec-puppet tests locally (rake spec)
Run beaker tests locally (BEAKER_set=debian-76-x64 BEAKER_destroy=onpass BEAKER_provision=yes rake beaker)
Commit
Push
Submit a PR (discussion can happen on the PR, but if there is a related issue link to it from the PR)
travis-ci will run rspec-puppet tests against a wider range of puppet versions and settings and will post the result on your PR.

Regarding puppet 3.7, puppet 4 is going to be here sooner than people think. I don't have a specific timeline but I think we're looking in the several-months-out timeframe, because there are very usable pre-release versions and the updated language is (basically) feature-complete in puppet 3.7 if you enable future parser. If you're coming from puppet 2.x, you might considering just jumping straight from your current codebase to getting compatible with 3.7 with future parser enabled, because in 4.x that'll be the only option.
Does that make sense?

I think that makes sense. For the rspec and beaker tests, they ca be run in vanilla Debian and CentoOS Vagrant boxes?
On Mar 26, 2015, at 10:04 AM, Daniel Dreier<EMAIL_ADDRESS>wrote:
@Cornellio I'm really only using my danieldreier/riak2 module as a development testbed, to see what I can do without being constrained by riak 1 or older puppet version compatibility, and without having to add tests onto older code; I'm hoping to wholesale forklift that code into this one after meeting feature-parity and incorporating the features that your and other modules have that I lack. For example, I've been realizing that the system settings are less optional than I had thought.
I should really add a CONTRIBUTING.md file documenting this. The workflow I'd suggest is:
Fork repository
Create feature branch
Hack
Run rspec-puppet tests locally (rake spec)
Run beaker tests locally (BEAKER_set=debian-76-x64 BEAKER_destroy=onpass BEAKER_provision=yes rake beaker)
Commit
Push
Submit a PR (discussion can happen on the PR, but if there is a related issue link to it from the PR)
travis-ci will run rspec-puppet tests against a wider range of puppet versions and settings and will post the result on your PR.
Regarding puppet 3.7, puppet 4 is going to be here sooner than people think. I don't have a specific timeline but I think we're looking in the several-months-out timeframe, because there are very usable pre-release versions and the updated language is (basically) feature-complete in puppet 3.7 if you enable future parser. If you're coming from puppet 2.x, you might considering just jumping straight from your current codebase to getting compatible with 3.7 with future parser enabled, because in 4.x that'll be the only option.
Does that make sense?
—
Reply to this email directly or view it on GitHub.

rspec-puppet and beaker tests are actually run on your host machine. Beaker creates its' own vagrant boxes to run tests inside of, and rspec-puppet uses the puppet you have installed on your development environment to build catalogs. rspec-puppet basically compiles the puppet code into a catalog and then inspects the catalog for certain conditions, whereas beaker actually tries to set up a virtual machine and then use serverspec to verify that the VM is running the expected services, has the right ports listening, etc.
One other step I forgot to add was to add test coverage for any new changes. If it's a straight up refactor of existing code, no new tests are needed. If there's new conditional logic being added or new features, the tests should fail without the code changes and pass with. Conditional logic is best tested in rspec-puppet and full-on integration testing is best done in beaker. If you need assistance in figuring out how to get that test coverage I'd be glad to help.

Thank you, that makes sense and gives me a good opportunity to adopt a proper testing framework.
On Mar 26, 2015, at 10:44 AM, Daniel Dreier<EMAIL_ADDRESS>wrote:
rspec-puppet and beaker tests are actually run on your host machine. Beaker creates its' own vagrant boxes to run tests inside of, and rspec-puppet uses the puppet you have installed on your development environment to build catalogs. rspec-puppet basically compiles the puppet code into a catalog and then inspects the catalog for certain conditions, whereas beaker actually tries to set up a virtual machine and then use serverspec to verify that the VM is running the expected services, has the right ports listening, etc.
One other step I forgot to add was to add test coverage for any new changes. If it's a straight up refactor of existing code, no new tests are needed. If there's new conditional logic being added or new features, the tests should fail without the code changes and pass with. Conditional logic is best tested in rspec-puppet and full-on integration testing is best done in beaker. If you need assistance in figuring out how to get that test coverage I'd be glad to help.
—
Reply to this email directly or view it on GitHub.

@danieldreier Riak2 initial version looks good.
In trying to setup my environment for beaker-rspec, I already have Vagrant with the vmware fusion provider, but I see most module tests are geared toward virtual box. I'm trying to get the fusion provider working but have no luck so far following this guidance. Not sure if it's worth the effort as virtual box is so much more common in terms of collaboration.

The beaker docs are pretty spotty; I'm sorry you're having a hard time with it. I've never used it with vmware fusion, but my vagrant - vmware fusion experiences have been poor enough that I switched back to virtualbox.
If you're using either my riak2 module or the PR I just made to merge that into this repo, you should be able to:
bundle install
BEAKER_set=centos-66-x64 BEAKER_destroy=onpass BEAKER_provision=yes bundle exec rake beaker

You will probably want to disable the virtualbox vagrant plugin first; I've had nothing but trouble with it. I'd also recommend you use the vagrant-vbguest and vagrant-cachier vagrant plugins.
The beaker tests will be slow (10 minutes?) with virtualbox or vmware because it has to provision and destroy a VM for each test run. I'd suggest running it once at the beginning of the development session to validate that tests pass when you start, and another run right before you make a PR.
When I get it set up myself, I'll add docs to the CONTRIBUTING.md on how to get it set up with GCE or AWS, since that may be a little faster.

Just FYI to this thread that a PR is up for review: https://github.com/basho-labs/puppet-riak/pull/52

@danieldreier Thank you, I switched to virtualbox instead of fusion and beaker runs now. It's an impressive piece of automation.
On your riak2 module the test process runs completely using bundle exec rake beaker, although I am seeing failures in the module itself (both on CentOS and Ubuntu.) I'll explore more for issues on my end.
I'm also able to run a test cycle with beaker-rspec (against puppetlabs-apache module) with
bundle exec rspec spec/acceptance

I'm not sure the issue is necessarily on your end. I encountered issues with it as well. For context,
I've been doing most of my work using the internal puppetlabs VM pooler, which is a godsend -- it keeps a constantly-running pool of about 1000 VMs running that beaker pulls out of the pool, provisions, tests and sends back, which makes beaker tests take about 45 seconds.
For some reason it's been passing on that platform but not in virtualbox. Just a guess but I suspect that there's a timing issue - the pooler hardware is incredibly fast. My hunch is that riak starts a lot faster on the pooler hardware than it does on a local virtualbox system, and in virtualbox it's not ready for tests to run by the time they start, even though I tried to build in a delay for that.

@Cornellio I think I figured out what was breaking tests in virtualbox and fixed it - look at commit https://github.com/danieldreier/puppet-riak/commit/045a240c356adbf2ee1b49f3ca384d53bdff38d8 in https://github.com/basho-labs/puppet-riak/pull/52

Yes indeed, it works fine now.
I'm thinking of adding system tuning settings with the kernel params and ulimits. It'll take me some time to get accustomed to rspec though...

https://github.com/basho-labs/puppet-riak/pull/52 was merged, so this is resolved. The module now only supports raik 2.


fix audit logs and introduce actual http errors instead of only handling 400's

yes this is tested
but nah fr, this is tested and should close #231 (except for the fact that i can't reproduce the 400 bad request)
it wasn't my code causing the issues either which is a first. now any response code which is not 429 or 400 creates a neatly-formatted ResponseException which you can respond to by using CompletionStage#exceptionally.
i'd recommend creating a new tag for this as 0.13.3 is effectively broken when dealing with audit logs

An example of the stack trace of a ResponseException:
java.util.concurrent.CompletionException: com.mewna.catnip.rest.ResponseException: HTTP Error Code: 403 | JSON Message: Missing Permissions | JSON Error Code: 50013
	at java.base/java.util.concurrent.CompletableFuture.encodeThrowable(CompletableFuture.java:331)
	at java.base/java.util.concurrent.CompletableFuture.completeThrowable(CompletableFuture.java:346)
	at java.base/java.util.concurrent.CompletableFuture$UniApply.tryFire(CompletableFuture.java:632)
	at java.base/java.util.concurrent.CompletableFuture.postComplete(CompletableFuture.java:506)
	at java.base/java.util.concurrent.CompletableFuture.completeExceptionally(CompletableFuture.java:2088)
	at com.mewna.catnip.rest.requester.AbstractRequester.handleResponse(AbstractRequester.java:274)
	at com.mewna.catnip.rest.requester.AbstractRequester$1.lambda$onResponse$1(AbstractRequester.java:209)
	at io.vertx.core.impl.ContextImpl.executeTask(ContextImpl.java:320)
	at io.vertx.core.impl.EventLoopContext.lambda$executeAsync$0(EventLoopContext.java:38)
	at io.netty.util.concurrent.AbstractEventExecutor.safeExecute(AbstractEventExecutor.java:163)
	at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:404)
	at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:462)
	at io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:897)
	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
	at java.base/java.lang.Thread.run(Thread.java:834)
Caused by: com.mewna.catnip.rest.ResponseException: HTTP Error Code: 403 | JSON Message: Missing Permissions | JSON Error Code: 50013
	at com.mewna.catnip.rest.requester.AbstractRequester.handleResponse(AbstractRequester.java:272)
	... 9 more


Citation link is broken

DOI link has an extra  ' at the end.

Hi Nabil,
Do you mean the link on the landing page? It's working for me.
Simon

hi.
this is what I get when i click on the link from the README shown on https://github.com/jameshadfield/phandango

Thanks Nabil. I think it's fixed now.


Adding backup/restore back for pgsql db and restore for server in dashboard

This PR fixes #12974
Backup restore button got hidden for PGSQL due to explicit check for MSSQL only - adding PGSQL back.  Thanks @kisantia for help.

@gupta1 - this shouldn't affect Kusto but please do take a quick test pass after this checkin

Pull Request Test Coverage Report for Build 324939909

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.001%) to 39.748%

Totals

Change from base Build 324712565:
-0.001%

Covered Lines:
20536

Relevant Lines:
47410

💛  - Coveralls

mixed use of && and || without parentheses is not preferred, but I guess this works for now.

I tried the (MSSQL|PGSQL) trick but didnt work - suggest please.

even the regex is not preferable, because the later part mssql:xxx is still sql server specific. we have to support condition grouping to fully address the issue, definitely not something you want to do in this PR :)


Medoo::buildRaw preg_replace_callback invalid regexp

Medoo::buildRaw has bad regexp /((FROM|TABLE|INTO|UPDATE)\s*)?\<([a-zA-Z0-9_\.]+)\>/i
For example query
INSERT INTO test ('TEXT FROM <.> TEXT')

Generate error Incorrect column name "." in file /vendor/catfan/medoo/src/Medoo.php at line 521
Why do this change? (where is using this table name syntax <table_name>?)
Suggests fixed:

Delete this replace
Make this preg_replace optional
Fix regexp in column name capture (for example ([a-zA-Z0-9_\.]+) replace to ([a-z][a-zA-Z0-9_\.]*)) .....
But bug is still exists after fix (partial fixed only)
Fix regexp by add first ^[a-z\s]*
Replace to: /^[a-z\s]*((FROM|TABLE|INTO|UPDATE)\s*)?\<([a-zA-Z0-9_\.]+)\>/i

Hello, this fix broken MariaDB compatibility.
I have update composer but still use my repo, because this fix broken my code. This is my query to test the issue:
$db->query("SELECT <table1.id>, table22.a_table2 FROM
LEFT JOIN (
SELECT CONCAT('[', GROUP_CONCAT(JSON_OBJECT('lid', <table2.lid>) ORDER BY <table2.id>, <table2.lid>),']') as a_table2, <table2.id>, <table2.lid>
FROM
WHERE <table2.pid> = 1
GROUP BY <table2.id>
) table22 ON <table1.id> = table22.id")->fetchAll();

@BrainStormDevel Change your single quote to double quote for
SELECT CONCAT("[", GROUP_CONCAT(JSON_OBJECT("lid", <table2.lid>) ORDER BY <table2.id>, <table2.lid>),"]")

I will test more for the regex for complex case. But this may fix your case from now.

Hello,
I also have a problem with the RAW function in the current version:
https://github.com/catfan/Medoo/issues/912
Can anyone here help me? Looks like a bug.