Add tool to sync runbooks to openshift/runbooks

What this PR does / why we need it:
This PR adds a tool to automate PR creation for newly updated runbooks in https://github.com/kubevirt/monitoring in https://github.com/openshift/runbooks.
GitHub action to execute the tool will be created in a follow up PR.
Results:

if a runbook does not exist in openshift/runbooks, a PR to add the file is created
if a runbook exists in both repos, it compares the last update date for the runbook, if it is newer in kubevirt/monitoring, a PR to update it in openshift/runbooks is created
removed/deprecated PRs are disabled for now (needing final decision on the process)

If a create/update PR is closed because we might want different wording in this repo, the automation will not try to create a new PR anymore until a new change is made downstream
example PRs on my personal repo: https://github.com/machadovilaca/runbooks/pulls
jira-ticket: https://issues.redhat.com/browse/CNV-28200
/cc @sradco @ctomasko @ousleyp

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note:

NONE

/cc @assafad @avlitman

Added a few comments, but overall looks great!
One general question that came to my mind:
Do you think that many of the automated D/S PRs will be merged as they are?
I'm afraid that as time passes, most of them won't. If that is the case, maybe we should simply the tool, and drop the ability of converting U/S runbooks to D/S. Maybe it would make sense that the the tool only open issues in https://github.com/openshift/runbooks/issues with a link to the U/S PR and assign it to the relevant people, or something like that.
One of the reason I think D/S PRs won't get merged is the following scenario:
We merge a change in an U/S runbook, then a D/S PR was automatically created, but we don't merge it as is, because it needs some manual changes. Then, another change in this U/S runbook was merged, and a following PR in D/S. The second D/S PR will include the old undesired change, right?

Added a small comment, but overall looks great!
One general question that came to my mind: Do you think that many of the automated D/S PRs for changing runbooks will be merged as they are? I'm afraid that as time passes, most of them won't, because U/S content will be suggested in every D/S PR, even if it was declined there in the past (?). If that is the case, maybe we should simply the tool, and drop the ability of converting U/S runbooks to D/S. Maybe it would make more sense that the tool will open issues in https://github.com/openshift/runbooks/issues with a link to the U/S PR and assign it to the relevant people, or something like that.

@assafad I believe that even if they will not merged as is the Docs team would be able to update it and merge. No @machadovilaca ?

Added a small comment, but overall looks great!
One general question that came to my mind: Do you think that many of the automated D/S PRs for changing runbooks will be merged as they are? I'm afraid that as time passes, most of them won't, because U/S content will be suggested in every D/S PR, even if it was declined there in the past (?). If that is the case, maybe we should simply the tool, and drop the ability of converting U/S runbooks to D/S. Maybe it would make more sense that the tool will open issues in https://github.com/openshift/runbooks/issues with a link to the U/S PR and assign it to the relevant people, or something like that.

@assafad I believe that even if they will not be merged as is, the Docs team would be able to update it and merge. No @machadovilaca ?

yes, whenever we update a runbook on KubeVirt we will open a PR on openshift/runbooks to update it, form then they can simply close it or edit it. We won't try to update it again. The automation works based on timestamps and no on the content

yes, whenever we update a runbook on KubeVirt we will open a PR on openshift/runbooks to update it, form then they can simply close it or edit it. We won't try to update it again. The automation works based on timestamps and no on the content

Will the automated PRs drag these undesired U/S changes? My understanding is that this cool tool copies the U/S runbook and converts a few things.
Say we have a runbook alert_name.md, and U/S PR that touches it was merged. Then, a D/S PR was opened automatically, but the docs team decline/edit the new changes. Then, a new U/S for alert_name.md was merged. Will the new D/S PR include the old undesired changes?

yes, whenever we update a runbook on KubeVirt we will open a PR on openshift/runbooks to update it, form then they can simply close it or edit it. We won't try to update it again. The automation works based on timestamps and no on the content

Will the automated PRs drag these undesired U/S changes? My understanding is that this cool tool copies the U/S runbook and converts a few things. Say we have a runbook alert_name.md, and U/S PR that touches it was merged. Then, a D/S PR was opened automatically, but the docs team decline/edit the new changes. Then, a new U/S for alert_name.md was merged. Will the new D/S PR include the old undesired changes?

IIUC your question, you are right, we can't combine upstream new changes, with diffs introduced downstream only. If after the docs team updates a runbook in downstream, and we change after something else in upstream, the PR would overwrite that to use what we have on upstream, is up to the docs team to check again if they went to include those new changes for the downstream or not.
Most of the runbooks, except those changes like kubectl -> oc, KubeVirt -> OpenShift Virtualization, should be the same, so this should not happen frequently.

updated with the deprecation @sradco @assafad

@assafad @avlitman can you please also review this PR?

/lgtm


Hydrazine revision 185 introduces compilation errors (hydrazine/interface/ArgumentParser.h line 242)

From<EMAIL_ADDRESS>on December 05, 2013 04:15:02
Hi,
with the revision 185 of Hydrazine an error is introduced in the code. Line 242 in file "hydrazine/interface/ArgumentParser.h" changed to this:
"if( identifier == str.size() )"
"identifier" and "str" are std::strings, so this comparison leads to compilation errors.
I do not know exactly how the surrounding code works, so I cannot suggest a patch. Please provide a commit to fix this error.
Thanks,
jellysheep
PS: Find the diff here: https://code.google.com/p/hydrazine/source/diff?spec=svn185&r=185&format=side&path=/trunk/hydrazine/interface/ArgumentParser.h PPS: I already posted this issue at the Hydrazine issues, but I do not know if the Hydrazine pages are frequently checked, and the gpuocelot builds are also affected by the error: https://code.google.com/p/hydrazine/issues/detail?id=6
Original issue: http://code.google.com/p/gpuocelot/issues/detail?id=88

From<EMAIL_ADDRESS>on December 12, 2013 05:34:00
Fixed in r186 . Thanks to Greg for providing the fix.
jellysheep


K.set_learning_phase(0)

Hi,
I don't get it why in training.py there's a line 7, which forces the learning_phase to 0 (=test)?
K.set_learning_phase(0)
As far as I know for example BatchNormalization() layer behaves differently in 0 and 1 phase.

K.set_learning_phase (), in the older version of keras, because of BN layers, we must specify the training and inference mode, but in the new version of keras this bug is fixed and don't necessary to set this option.


Files Bug

We need a little more information from you in order to investigate this issue, please reopen this issue once you have filled out the issue template.


(PUP-4533)(DOC-3685) Document http source checksum

The file resource's http source checksum behavior, which relies on headers in the HTTP server response, is not documented in the type. Document this behavior.

CLA signed by all contributors.


added a missing icon import "PlayCircleOutline"

When I recently pulled the changes and ran "npm start", I found an error. The error was a missing icon import that is "PlayCircleOutline" in layout BottomNav section.

@piyush-cosmo but there is a routing issue also.

@Abhijay007 I find no problem in running. Will you elaborate more about the issue you are mentioning?

@Abhijay007 I find no problem in running. Will you elaborate more about the issue you are mentioning?

@piyush-cosmo this one :

as you can see both Home and MyClass pages are the same.

@piyush-cosmo but there is a routing issue also exists.

The routing issue was fixed

@piyush-cosmo but there is a routing issue also exists.

The routing issue was fixed

but @sakshi-choudhary I made this video after pulling the latest changes from the main branch can you please cross-check it once and confirm it. Because I am not sure that something wrong in my code or it exists globally in the main branch.


z-index (set by zIndexOffset) gets reset to -Infinity when the direct parents have no explicit z-index

Expected behaviour
z-index should have the value of the zIndexOffset options property.
Actual behaviour
z-index gets value -Infinity
Datepicker version used
ex. 1.6.1.
Using Math.max on an empy array causes the issue, check for empty array and skip the manipulation of zIndexOffset when empty, to avoid resetting to -Infinity.

Thanks for this bugreport. I've indeed located the code which caused this issue. What should the zindex be when we can't determine it? 0 or ...?

Leave the settings value unchanged if it cannot be determined.


[tool_crash] _CastError: (#0      new WebPlugin.fromYaml (package:flutter_tools/src/platform_plugins.dart:548:34))

Command
flutter --no-color pub get

Steps to Reproduce

...
...
...

Logs
_CastError: (#0      new WebPlugin.fromYaml (package:flutter_tools/src/platform_plugins.dart:548:34))
#0      new WebPlugin.fromYaml (package:flutter_tools/src/platform_plugins.dart:548:34)
#1      new Plugin._fromMultiPlatformYaml (package:flutter_tools/src/plugins.dart:142:21)
#2      new Plugin.fromYaml (package:flutter_tools/src/plugins.dart:78:21)
#3      _pluginFromPackage (package:flutter_tools/src/flutter_plugins.dart:63:17)
#4      findPlugins (package:flutter_tools/src/flutter_plugins.dart:88:28)
<asynchronous suspension>
#5      PackagesGetCommand.usageValues (package:flutter_tools/src/commands/packages.dart:102:36)
<asynchronous suspension>
#6      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1423:15)
<asynchronous suspension>
#7      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1257:27)
<asynchronous suspension>
#8      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#9      CommandRunner.runCommand (package:args/command_runner.dart:209:13)
<asynchronous suspension>
#10     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:283:9)
<asynchronous suspension>
#11     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#12     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
<asynchronous suspension>
#13     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:64:9)
<asynchronous suspension>
#14     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)

[✓] Flutter (Channel stable, 3.7.1, on Microsoft Windows [Version 10.0.19045.2604], locale en-US)
    • Flutter version 3.7.1 on channel stable at C:\src\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 7048ed95a5 (5 weeks ago), 2023-02-01 09:07:31 -0800
    • Engine revision 800594f1f4
    • Dart version 2.19.1
    • DevTools version 2.20.1

[✓] Windows Version (Installed version of Windows is version 10 or higher)

[!] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at C:\Users\User\AppData\Local\Android\sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: C:\Program Files\Java\jdk-17.0.4\bin\java
    • Java version Java(TM) SE Runtime Environment (build 17.0.4+11-LTS-179)
    ! Some Android licenses not accepted. To resolve this, run: flutter doctor --android-licenses

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2022 17.3.1)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.3.32811.315
    • Windows 10 SDK version 10.0.19041.0

[!] Android Studio (version 2022.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] VS Code
    • VS Code at C:\Users\User\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19045.2604]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 110.0.5481.178
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 110.0.1587.63

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 2 categories.

Flutter Application Metadata
Type: malformed
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: null
Creation framework version: null
Plugins
call_log-4.0.0
cloud_firestore_web-3.2.0
connectivity_plus-3.0.2
device_info_plus-4.1.3
device_info_plus_linux-3.0.0
device_info_plus_macos-3.0.0
device_info_plus_web-3.0.0
device_info_plus_windows-4.1.0
file_picker-5.2.5
firebase_auth-4.2.3
firebase_auth_web-5.2.3
firebase_core-2.4.0
firebase_core_web-2.0.2
firebase_messaging-14.2.0
firebase_messaging_web-3.2.10
flutter_contacts-1.1.5+1
flutter_phone_direct_caller-2.1.1
flutter_plugin_android_lifecycle-2.0.8
flutter_secure_storage-5.1.2
flutter_secure_storage_linux-1.1.2
flutter_secure_storage_macos-1.1.2
flutter_secure_storage_web-1.1.1
flutter_secure_storage_windows-1.1.3
flutter_udid-2.0.1
flutter_web_auth-0.4.3
fluttertoast-8.1.2
hexcolor-2.0.7
integration_test
path_provider-2.0.11
path_provider_android-2.0.22
path_provider_ios-2.0.11
path_provider_linux-2.1.7
path_provider_macos-2.0.6
path_provider_windows-2.1.3
permission_handler-10.2.0
permission_handler_android-10.2.0
permission_handler_apple-9.0.7
permission_handler_windows-0.1.2
phone_state-1.0.2
shared_preferences-2.0.15
shared_preferences_android-2.0.14
shared_preferences_ios-2.1.1
shared_preferences_linux-2.1.2
shared_preferences_macos-2.0.4
shared_preferences_web-2.0.4
shared_preferences_windows-2.1.2
sqflite-2.2.2
url_launcher-6.1.10
url_launcher_android-6.0.22
url_launcher_ios-6.0.17
url_launcher_linux-3.0.1
url_launcher_macos-3.0.1
url_launcher_web-2.0.13
url_launcher_windows-3.0.1

Closing this as a duplicate of https://github.com/flutter/flutter/issues/115738


[W4.5c][W15-A3] Yash Chowdhary

Application displays appropriate message to user and starts over.

Some code adapted from #44 (restarting the application).

@yash-chowdhary Please close PR after reading comments.


Allow options to bubble upstream, support 304s.

On a V1 model, when doing something like Comment.all(post, { cache: true }), $.ajax() doesn't get the cache option passed to it. This means we can't use the ETag/If-None-Match caching built into ajax on APIs that support it. There is also a ifModified option for $.ajax() with similar functionality that doesn't get sent up through RestModel V1.
Additionally, when an ETag is still valid, we don't need to update the cached response. In both V1 and V2, we now abort processing and updating in the case of 302 (not modified) responses.

Ported over from jclem/rest-model#26.

Oops, duplicate.


Allow a user to remove their own calls

Add a destroy endpoint to calls controller.
Add delete button to calls in call log that's created recently by the current user.

Fixes https://github.com/colinxfleming/dcaf_case_management/issues/524

@colinxfleming sending this your way while I write some tests and hopefully find a way to get the tests running locally.

Codecov Report

Merging #913 into master will decrease coverage by -0.75%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master     #913      +/-   ##
==========================================
- Coverage   98.46%   97.71%   -0.75%
==========================================
  Files          37       37
  Lines         783      789       +6
==========================================
  Hits          771      771
- Misses         12       18       +6

Impacted Files
Coverage Δ

app/controllers/calls_controller.rb
75% <33.33%> (-25%)
:x:

Continue to review full report at Codecov.

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

Tests added! Ready for reviews!

ahhhh the gif

Kind of a BOLD action for something that may not be used much. Could we convert it from a button to just an underlined red link?

my 2c: buttons and links are semantically different, a button is an action in the current context and a link navigates the user to a different resource. If the concern is the big red button is too bold, we can style it differently, but a button makes more sense here.

Clutch part of this was getting the functionality in place -- if we need to tweak the styling let's record that in a separate issue and fix with a separate PR.
Thanks so much @yf6 , hope you're caught up on sleep by now!


Add a fixed-top navigation

With links to
Logo
Home
Jobs
Network
About Us
Contact Us
FAQ

I can do this for you!


Hist training with checkpointing is non-deterministic based on subsample

Hi there,
We have found that despite setting a seed for our hist training, we get non-deterministic results when resuming training from a checkpoint. A reproducer can be seen below:
import numpy as np
import xgboost as xgb
from sklearn.datasets import make_classification
from sklearn.model_selection import train_test_split
from xgboost.callback import TrainingCallback

# Generate random sample data
X, y = make_classification(n_samples=1000, n_features=20, n_informative=10, n_redundant=10, random_state=42)

# Split the data into training and testing sets
X_train, X_test, y_train, y_test = train_test_split(X, y, test_size=0.2, random_state=42)

# Ensure model has correct epoch upon restart
class XgbCheckpointCallback(TrainingCallback):
    def __init__(
            self,
            start_epoch: int
    ):
        super().__init__()
        self.start_epoch = start_epoch

    def before_training(self, model: xgb.Booster):
        self._prev_update = model.update

        def update(
                dtrain, iteration: int, fobj=None
        ) -> None:
            return self._prev_update(dtrain, iteration + self.start_epoch, fobj)

        model.update = update
        return model

    def after_training(self, model):
        model.update = self._prev_update
        return model

# Set the parameters for XGBoost training
params = {
    'silent': True,
    "tree_method": "hist",
    "seed": 1,
    "base_score": -0.9,
    "max_depth": 3,
    "learning_rate": 0.02,
    "lambda": 500,
    "subsample": 0.9,
}

# Train the XGBoost model using the hist algorithm
dtrain = xgb.DMatrix(X_train, label=y_train)
dtest = xgb.DMatrix(X_test, label=y_test)

# Train the model for the first run
bst1 = xgb.train(params, dtrain, 10, evals=[(dtrain, 'train'), (dtest, 'test')], early_stopping_rounds=5, feval=None)

# Save the model after the first run
bst1.save_model('first_run_model')

# Train the model for the second run
bst2 = xgb.train(params, dtrain, 5, evals=[(dtrain, 'train'), (dtest, 'test')], early_stopping_rounds=5, feval=None)

# Save the model after the second run
bst2.save_model('second_run_model')

# Resume training from the second run and run for 5 more epochs
bst3 = xgb.train(params, dtrain, 5, evals=[(dtrain, 'train'), (dtest, 'test')], early_stopping_rounds=5, feval=None,
                 xgb_model=bst2, callbacks=[XgbCheckpointCallback(5)])

# Save the model after the third run
bst3.save_model('third_run_model')

# Make predictions on the test set for each run
preds1 = bst1.predict(dtest)
preds2 = bst2.predict(dtest)
preds3 = bst3.predict(dtest)

# Evaluate the model
print("Test set accuracy for run 1: {:.5f}".format(np.mean(preds1)))
print("Test set accuracy for run 3: {:.5f}".format(np.mean(preds3)))

We make use of XgbCheckpointCallback to fix a similar issue whereby restarting from a checkpoint ignores the epoch the checkpoint got up to. You can remove it, but then setting any of the colsample_* params to a value below 1.0 will produce the same issue.
When tree_method is set to exact, the uninterrupted model and the interrupted model are identical. When tree_method is set to hist and subsample is set to 1.0, they are also identical. When running with hist and subsample < 1.0 however, the results differ.
I've seen #6711, but that seems to be somewhat different in nature.

Thank you for raising the issue. We will revisit this after https://github.com/dmlc/xgboost/issues/6711 is resolved by removing the global random engine.

Hi again,
Just wondering if there's anything I can do to help out with resolving this? Like I mentioned earlier my C++ abilities are somewhat lacking, but if there is something I can help with I'd gladly have a crack.

We are currently working on it among other items https://github.com/dmlc/xgboost/pull/10354 We want to use a booster-local random state instead of a global random state. The problem is that we can' reliably preserve the random state in the model file. As a result, after the PR is finished, the result would be deterministic. However, it's not necessarily true that training two smaller boosters would be the same as training a single large booster. We are still investigating.


RC Spot-Check Test: Gases Intro 1.1.0-rc.2

RC Spot-Check Test
Mentions: @arouinfar @kathy-phet @KatieWoe @Nancy-Salpepi
Simulation links

github repository for issues - Please create issues in gas-properties.
phet top-level directory
sim: all_phet.html
phet-io wrapper index
link to phet-io zip file

Test Matrices
General Test:

[x] Tester = NS , Platform = mac + safari , Time = .75
[x] Tester = KW, Platform = Win 11 Chrome, Time = .5

PhET-iO Test:

[x] Tester = NS , Platform =mac + safari , Time =
[ ] Tester = , Platform = , Time =

Additional Features Test:

[ ] Tester = , Platform = , Time =
[ ] Tester = , Platform = , Time =

Features included

PhET-iO
Dynamic Locale
Alternative Input
UI Sound
Sonification
Description
Voicing

Focus and Special Instructions
This sim is part of a suite of 3 sims, all being released and tested at the same time. These QA issues should therefore be done together:

https://github.com/phetsims/qa/issues/1123
https://github.com/phetsims/qa/issues/1124
https://github.com/phetsims/qa/issues/1125

Issues to Verify
These issues have the "status:ready-for-review" label. Unless an issue says to close after verifying, assign the issue back to the developer.
See the last comment in each issue for instruction describing how to verify.

[x] https://github.com/phetsims/gas-properties/issues/289
[x] https://github.com/phetsims/gas-properties/issues/288
[x] https://github.com/phetsims/gas-properties/issues/285
[x] https://github.com/phetsims/gas-properties/issues/283
[x] https://github.com/phetsims/gas-properties/issues/276
[x] https://github.com/phetsims/gas-properties/issues/221

For QA...

General features

What to Test

Click every single button.
Test all possible forms of input.

Test all mouse/trackpad inputs.
Test all touchscreen inputs.

If there is sound, make sure it works.
Make sure you can't lose anything.
Play with the sim normally.
Try to break the sim.
Test some query parameters. (See QA Book for a
list of query parameters.)
When making an issue, check to see if it was in a previously published version.
Try to include version numbers for browsers.
If there is a console available, check for errors and include them in the Problem Description.
As an RC begins and ends, check the sim repo. If there is a maintenance issue, check it and notify developers if there
is a problem.
As the RC ends, notify the developer of any new QA credits that need to be added.

PhET-iO features

What to Test

Make sure that public files do not have password protection. Use a private browser for this.
Make sure that private files do have password protection. Use a private browser for this.
Make sure standalone sim is working properly.
Make sure the wrapper index is working properly.
Make sure each wrapper is working properly.
Launch the simulation in Studio with ?stringTest=xss and make sure the sim doesn't navigate to youtube
For newer PhET-iO wrapper indices, save the "basic example of a functional wrapper" as a .html file and open it. Make
sure the simulation loads without crashing or throwing errors.
Load the login wrapper just to make sure it works. Do so by adding this link from the sim deployed root:/wrappers/login/?wrapper=record&validationRule=validateDigits&&numberOfDigits=5&promptText=ENTER_A_5_DIGIT_NUMBER

Further instructions in QA Book
Conduct a recording test to Metacog, further instructions in the QA Book. Do this for iPadOS + Safari and one other
random platform.
Conduct a memory test on the stand alone sim wrapper (rc.1).
Test one platform combination with ?phetioDebug=true on the Studio and State wrapper.
If Pan/Zoom is supported, make sure that it works when set with PhET-iO State.
Test that the sim works offline:

Click the link to the phet-io zip file (at top of issue) to download the zip file.
Unzip it to a spot locally.
Open index.html by double clicking it on your desktop or in a Finder-view.
It should look like the standalone version of the sim in PhET-iO brand.

Accessibility features

What to Test

Specific instructions can be found above.

Make sure the accessibility (a11y) feature that is being tested doesn't negatively affect the sim in any way. Here is
a list of features that may be supported in this test:

Alternative Input
Interactive Description
Sound and Sonification
Pan and Zoom
Mobile Description
Voicing

Test all possible forms of input.

Test all mouse/trackpad inputs.
Test all touchscreen inputs.
Test all keyboard navigation inputs (if applicable).
Test all forms of input with a screen reader (if applicable).

If this sim is not in this list or up to date there, make an
issue in website to ask if PhET research page links need updating. Please
assign to @terracoda and @emily-phet.

Screen Readers
This sim may support screen readers. If you are unfamiliar with screen readers, please ask Katie to introduce you to
screen readers. If you simply need a refresher on screen readers, please consult the
QA Book, which should have all of the information
you need as well as a link to a screen reader tutorial made by Jesse. Otherwise, look over the a11y view before opening
the simulation. Once you've done that, open the simulation and make sure alerts and descriptions work as intended.

Platforms and Screen Readers to Be Tested

Windows 10 + Latest Chrome + Latest JAWS
Windows 10 + Latest Firefox + Latest NVDA
macOS + Safari + VoiceOver
iOS + Safari + VoiceOver (only if specified in testing issue)

Critical Screen Reader Information
We are tracking known screen reader bugs in
here. If you find a screen reader bug,
please check it against this list.

Keyboard Navigation
This sim supports keyboard navigation. Please make sure it works as intended on all platforms by itself and with a
screen reader.
Magnification
This sim supports magnification with pinch and drag gestures on touch screens, keyboard shortcuts, and mouse/wheel
controls. Please test magnfication and make sure it is working as intended and well with the use cases of the
simulation. Due to the way screen readers handle user input, magnification is NOT expected to work while using a screen
reader so there is no need to test this case.

FAQs for QA Members

There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.

How should I format my issue?

We typically assign the developer who opened the issue in the QA repository.

My question isn't in here... What should I do?
You should:

Consult the QA Book.
Google it.
Ask Katie.
Ask a developer.
Google it again.
Cry.

QA is done

Thanks QA! 1.1.0 has been published.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_5125bc53-5d95-4cc1-b5f5-58fbbeb06dba/pull/1 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target_5125bc53-5d95-4cc1-b5f5-58fbbeb06dba.git

Docs Build status updates of commit ac8929b:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-04-20-06-17-02 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-04-20-06-17-02

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Windows Defender Activation

Hey, while trying to install this code (Thanks for the work by the way, it's awesome)
My windows automatically remove the code for this reason: https://www.microsoft.com/en-us/wdsi/threats/malware-encyclopedia-description?name=HackTool%3AWin32%2FCrack!MTB&threatid=2147745913.
I'm using the latest version of Windows without a specific security setting or anitvirus.
Is there a workaround?
Thanks again

Same.

https://www.virustotal.com/gui/file/508cc5597dca3aa69190f610c3bd055d2930266b03baaafa857a9ce7d58fb67f
Not just Windows Defender, Virus Total says many security vendors are reporting the version.dll file as a Trojan or Trojan Variant, wish the devs would comment on this.

fixed on my fork: https://github.com/Speretta/lovely-injector/releases/latest

fixed on my fork: https://github.com/Speretta/lovely-injector/releases/latest

It didn't worked on my windows. Still automatically removed.

It didn't worked on my windows. Still automatically removed.

whats your windows version and whats the flag name? works fine on my windows 11

Integrating code signing into the release workflow should ameliorate these issues for a little bit.

It didn't worked on my windows. Still automatically removed.

whats your windows version and whats the flag name? works fine on my windows 11

Sorry for late reply. I checked on my windows pc again, and it worked!
I might have missed with wrong file of original repo.
The windows version was 23H2 os build: 22631.4169

had the same issue chrome/edge wont even let u download it and even when u do it gets auto flagged and deleted by windefender, the suggested fork did not have this issue.
also, not to be 'that guy' but this issue seems to be a month old, meanwhile noone can download ur shizzle ??

I had the same issue of it being flagged on download on Firefox. Windows Defender identifies it as a trojan. Specifically Trojan:Win32/Fauppod!md


Fix autoplay issue.

I had an issue, that the slider won't slide automatically. I feel that the condition is either wrong, of the comment above it. Nevertheless, after I inverted the condition, the automatic sliding works as I expect it to.

duplicate of https://github.com/akiran/react-slick/pull/651


Add the rutime dependency to the project, not compilations

This should resolve IDE issues in a multiplatform use.
Also fix the test which was wildly broken and add one for no supported plugin.
Closes #136

It was a test inside a test fixture. We already test the library normally. The fixture only is for guaranteeing the runtime is added automatically.


[Sample 51][C#] Added README note about updating LuisConfiguration

Fixes #1236
Proposed Changes
Added note about ensuring LuisConfiguration is updated in each of the appropriate files. Bot breaks, otherwise.

refer to the comment in #1236


ROX-23070: Add routing for Platform CVE cluster page

Description
Adds routing and placeholder component for Platform CVEs: Cluster single page.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
Via direct visit in URL:

Click the breadcrumb link and ensure the page navigates back to the cluster overview list:

/retest


LAI fapar and Soil moisture notebook

Notebook description
Percentage of missing values for each pixel map and histogram
Notebook link or upload
Soil moisture_ Completeness_rev.zip
fAPAR_ LAI_Completeness(1).zip
Anything else we need to know?
Hi Mattia,
I have to reopen this issue. I noticed your question in the closed issue about the fact if the barplot should have the percentage of NA on the world or respect to a specific Continent num of pixel. When I show the map of Europe, the histogram should tell me the total number of pixel with 100 percent missiing value respect to the total num of pixel for that specific continent, not respect the globe. Now it is respect the globe area?
ps. They set a limitation on the number of plots we can create so I am showing at the end just Europe.
Thanks!
Environment

I believe what you are doing is correct. The barplot and the map use the exact same dataarray as input.
I guess the comment you are referring to is this: https://github.com/bopen/c3s-eqc-toolbox-template/issues/34#issuecomment-1547500187
You asked how to change the x/y axis of the map, so I added this feature in the template (looks like you are not using it though). For example, if you plot Europe, then you change the limits of the map using set_xlim/set_ylim, the bars won't change. Therefore you are showing a subregion, but the bars are showing the statistics for the whole continent.


Deploy product  on Association infrastructure (int.catena-x.net)

As product
I want to deploy the product on new infrastructure
so that we are able to participate in release 24.8
Link

relates to https://github.com/eclipse-tractusx/sig-infra/issues/515
relates to https://github.com/eclipse-tractusx/sig-infra/issues/516

Hints / Details

ArgocCD https://argocd.int.catena-x.net
Vault https://vault.int.catena-x.net

Acceptance Criteria

[ ] Product is  deployed and running on association infrastructure
[ ] Rename branch catenax-environments to helm-environments

Out of Scope

...

IRS is deployed and reachable via this URL https://irs.int.catena-x.net/api/swagger-ui/index.html
It is not yet functional, since the shared services URLs are not available yet, but this is not part of the ACs
New Helm Environments branch for association config: https://github.com/eclipse-tractusx/item-relationship-service/tree/helm-environments

LGFM: PO acceptance in behalf of @jzbmw .


all: make all benchmarks report allocations using b.ReportAllocs()

It is necessary to trivially see how many allocations are being churned out by code, of which the closest to easily quantifying the allocations is by using benchmarks. A couple of the benchmarks in this repository don't have b.ReportAllocs() e.g.
goos: darwin
goarch: amd64
pkg: github.com/cosmos/iavl
cpu: Intel(R) Core(TM) i7-7920HQ CPU @ 3.10GHz
BenchmarkImport-8   	      79	  16952531 ns/op
BenchmarkImport-8   	      72	  17366706 ns/op
BenchmarkImport-8   	      73	  17971052 ns/op
BenchmarkImport-8   	      70	  16451840 ns/op
BenchmarkImport-8   	      64	  16419086 ns/op
BenchmarkImport-8   	      75	  17330117 ns/op
BenchmarkImport-8   	      72	  17079898 ns/op
BenchmarkImport-8   	      64	  17366409 ns/op
BenchmarkImport-8   	      66	  17114097 ns/op
BenchmarkImport-8   	      73	  15890291 ns/op
BenchmarkImport-8   	      70	  17459452 ns/op
BenchmarkImport-8   	      67	  16169899 ns/op
BenchmarkImport-8   	      66	  16764476 ns/op
BenchmarkImport-8   	      74	  17083379 ns/op
BenchmarkImport-8   	      72	  15838747 ns/op
BenchmarkImport-8   	      74	  16469723 ns/op
BenchmarkImport-8   	      75	  17212699 ns/op
BenchmarkImport-8   	      62	  16596012 ns/op
BenchmarkImport-8   	      67	  16189477 ns/op

Kindling cc-ing @erikgrinaker

@odeke-em  - do you think we should add it with b.ReportAllocs? I think the main way to decide about capturing memory allocations is by adding -benchmem cmd parameter.


Update for 0.7.1

This change includes the following changes to work with the new OpenStack SDK v0.7.1

Connection now uses rackspaceauth, as the upstream SDK now depends on keystoneauth.
service_filter was relocated out of openstack.auth
The addition of _find to BaseProxy means the find implementations in the database and backup proxy classes is slightly simplified, and falls in line with how every other proxy method is called.

@WalterLenuzza can you have a look at this, at least around the database and backup things you did? We added a few things in OpenStack SDK that made the find calls in proxies a little more clean, and there's a change in database/database_service.py around a service_type argument that is now required due to some under-the-hood stuff we get out of keystoneauth. I did a quick check against public cloud and it appeared fine.

@briancurtin The new _find methods are awesome, and the Cloud Backup code seems to work fine.
But Cloud Databases resources are not being loaded: looks like services are not overridden in the profile if their names don't match (makes sense). Any way that we can inherit from openstack.database even though the service name is different?
>>> from rackspace import connection
>>> import sys
>>> import openstack
>>> openstack.utils.enable_logging(debug=True, stream=sys.stderr)
>>> conn = connection.Connection(username="XXX", api_key="XXX", region="XXX")
2015-11-25 10:00:53,817 DEBUG: openstack.profile Overriding object-store with {'service_type': 'object-store', 'interface': None, 'version': None, 'region_name': None, 'service_name': None}

>>> db_baks = [x for x in conn.database.backups()]
Traceback (most recent call last):
  File "<input>", line 1, in <module>
AttributeError: 'Proxy' object has no attribute 'backups'
>>> conn.database.__module__
'openstack.database.v1._proxy'

Good find - I'll look into it. We don't provide a great way from upstream to override resources downstream like we're doing with databases, so it's a bit manual. At the very least, I can make it work for the time being while we improve it up there.

If you took out that "import openstack" line, you'd have done what I tested, which works. Ugh, that's not great.
>>> from rackspace import connection
>>> conn = connection.Connection(username="", api_key="", region="")
>>> conn.database
<rackspace.database.v1._proxy.Proxy object at 0x10dc4ef98>
>>> conn.database.__module__
'rackspace.database.v1._proxy'
>>> list(conn.database.backups())
[] # I don't have any of these, but at leas the call worked properly.

Ok, this is more odd. import openstack by itself usually does nothing, and you would not be able to call openstack.utils.enable_logging as none of that is plumbed all the way through to be supported off of importing just the top level namespace. However, if you do that after already having done from rackspace import connection, what you did works, but only in that case. Whatever is allowing that to work seems to also be messing up the service issue.

Alright, well this is weird, but not entirely weird, but also avoidable. At least for now. If you change your import to from openstack import utils and then adjust accordingly, it works as expected.
>>> from rackspace import connection
>>> from openstack import utils
>>> utils.enable_logging(True, path="sdk.log")
>>> conn = connection.Connection(username="", api_key="", region="")
>>> list(conn.database.instances())
[] # again this is working, i just have no instances
>>> conn.database
<rackspace.database.v1._proxy.Proxy object at 0x104e96f60>

The import openstack line by itself only works because under the hood, most of what it contains has already been loaded in the from rackspace import connection line. I don't know exactly why it's stomping on the database service that gets loaded, but for right now I'm going to say that the way you did it is "not supported". None of this is setup to work when imported only as a top-level name.
Additionally, for fun, if you did import openstack first and then from rackspace import connection second, nothing on the openstack name would work until the second line has been interpreted, but after that, everything would work as we need because the proper rackspace.database gets loaded and doesn't end up being stomped on.

Can't say I understand why this happened, but I've noticed the weirdness before. I'll carry on testing my code how you suggested. Would integration tests be the right place to verify that all plugins are loaded properly?
:+1: for merging this


Feature Request: To also clear cache when widgets/menus/themes configs are changed

Hello guys. I've noticed than when the site is too big and you have edited widgets or menus or something at theme customizer, the cache is not cleared, yet it should.
I found a plugin that solved this: W3TC Auto Pilot, but given its implementation is by far simple, I would love to see its features integrated within W3TC, so you can mark a checkbox to Clear cache when widgets changed, when theme options changed, when a new theme is activated, and when menus have changed. But the mentioned plugin have not received updates since 1 year ago, so it may be abandoned:
https://es.wordpress.org/plugins/w3tc-auto-pilot/
Is there any possibility that you can consider adding those features to the plugin core config?
Thanks in advance!

@MarceloPedra w3tc flush the cache on some wordpress events, see Util_AttachToActions::flush_posts_on_actions() on https://github.com/szepeviktor/w3-total-cache-fixed/blob/b225070f2c68b45411cb8ba69d2c007627909678/Util_AttachToActions.php#L18 .
w3tc-auto-pilot seem handle the same events of flush_posts_on_actions().

@MarceloPedra any news?

Mmmhmm. W3TC will flush the cache even when you edit/change widgets or menus?
Also, is it working for you on custom post type categories and tags? In example: a WooCommerce product is commonly listed in the frontend using its categories or tags. So, when you update a product, its related cached categories and tags pages should be cleared as well. I'm not seeing this happenning... always. Sepecially when it comes to big stores with hundreds of products. Do you?

@MarceloPedra these are all  w3tc-auto-pilot features (YES/NO if already implemented on W3TC):

w3tc-auto-pilot
w3tc

customize_save_after
NO

wp_update_nav_menu
YES

sidebar_admin_setup
NO

switch_theme
YSE

wp_ajax_widgets-order
NO

save_post
YES

Mmmhmm. W3TC will flush the cache even when you edit/change widgets or menus?

for the menu, both plugins, flush the cache on wp_update_nav_menu
for the widgets only auto-pilot flush the cache on sidebar_admin_setup.
in /wp-content/plugins/w3-total-cache/Util_AttachToActions.php in the function flush_posts_on_actions() add these lines after edit_user_profile_update:
add_action( 'sidebar_admin_setup', array(
    $o,
    'on_change'
), 0 );

this enable flushing on widget change...

Hello. I still didn't test your code recommendation BUT I found another plugin that does the cache purge, and does it well: Clear Cache for Me:
https://wordpress.org/plugins/clear-cache-for-widgets/
It has enough hooks to clear cache on almost any modification to the WP configs.
The problem is that this plugin will purge the entire cache!
So I kept looking, until I discovered this, that probably you will love as much as I did: the official CloudFlare plugin has a beautifully implemented method to purge only the cache for items and taxonomies related to the post being modified:
https://github.com/cloudflare/Cloudflare-WordPress/blob/master/src/WordPress/Hooks.php
Look at these functions: purgeCacheByRevelantURLs and getPostRelatedLinks
Hope it helps to improve the plugin.
Thanks!

Ah thanks! i see both. Stay tuned.


Trying to build torchvision 0.13.0 from source to run YOLOv5 natively on Jetson nX

Trying to build torchvision==0.13.0 from source to run YOLOv5 natively on Jetson nX
when I run pyrhon3 setup.py install is throws follwoing error
Traceback (most recent call last):
File "setup.py", line 9, in
import torch
ModuleNotFoundError: No module named 'torch'
I had installed torch 1.12.0 from a python wheel file downloaded from jetsonzoo
https://developer.download.nvidia.com/compute/redist/jp/v50/pytorch/torch-1.12.0a0+2c916ef.nv22.3-cp38-cp38-linux_aarch64.whl
Please help

I had installed torch 1.12.0 from a python wheel file downloaded from jetsonzoo

That seems to have not worked out. Try python -c 'import torch'. As long as this fails, torch is not correctly installed and in turn torchvision cannot be build from source.

Can you please elaborate
I am running python3 setup.py install
where should I use -c?

You need to make sure that torch is correctly installed before you try to install torchvision. And the error you are getting states that this is not the case.
I'm going to close this issue, since this seems like an issue with your environment rather than torchvision. For questions please use user forum. Feel free to re-open this or open a new issue if you are experiencing issues after the environment is sorted out.


added show withDelay option in order to delay immediate show of indic…

Following will allow users to show indicator with delay. This could be useful when user is on high-speed internet connection and operation takes less than let's say 200 mili seconds. User will constantly see blinking(show and immediate hide) of loading screen, while his connection is so good that he/she could not even understand network calls are made and just give him/her the appropriate content/action.
With new method showWithDelay if system call hide before desired delay, the activity won't be shown to user. Also if call is made to show, this delayed call will be discarded as we need activity immediately. It could be done as method with default param to show, however I see it that way more readable.
Regards,
Hristo Todorov

thanks!


Compiler complains that diesel::Expression isn't implemented

I'm trying to use the enum as an input into a sql_function, but the compiler complains:
error[E0277]: the trait bound `models::RoomUsage: diesel::Expression` is not satisfied
  --> src/db_helpers.rs:55:23
   |
55 |     let returned_rooms = get_available_rooms(start_date, end_date, room_usage);
   |                          ^^^^^^^^^^^^^^^^^^^ the trait `diesel::Expression` is not implemented for `models::RoomUsage`
   |
   = note: required because of the requirements on the impl of `diesel::expression::AsExpression<models::RoomUsage>` for `models::RoomUsage`

But a quick glance into the source code shows that you do implement AsExpression, so I'm wondering what I'm missing here.
Cheers

Oh wow, turns out I forgot to use the Mapping suffix in the sql_function! macro.
I.e. Changing sql_function!(fn some_fn(a: BigInt, b: BigInt, room_usage_in: RoomUsage)); to sql_function!(fn some_fn(a: BigInt, b: BigInt, room_usage_in: RoomUsageMapping));
Fixed it


Test timeouts waiting for human input

Tests occassionally fail in dashboard/api/pkg/test TestEndToEndWithComputeNodeHook/TestCanSeeJob with the following behaviour
DBG pkg/compute/endpoint.go:53
> asked to bid on: {ExecutionMetadata:{ExecutionID:e-827e9921-5853-4da6-a1fa-5d1245b3401b
      JobID:5d436651-0f49-4e31-a608-397a9dabf450}
      RoutingMetadata:{
          SourcePeerID:QmcHV27bYoprc45QN69pzPqbWwvoqFU1xok83SckVTbzSX
          TargetPeerID:QmcHV27bYoprc45QN69pzPqbWwvoqFU1xok83SckVTbzSX}
     Job:5d436651-0f49-4e31-a608-397a9dabf450} [NodeID:QmcHV27b]
DBG pkg/bidstrategy/semantic/chained.go:53
> bidding strategy *semantic.StatelessJobStrategy returned should not bid due to: stateless jobs not accepted
DBG pkg/bidstrategy/chained.go:64
> bidding strategy *semantic.ChainedBidStrategy returned should not bid due to: stateless jobs not accepted
DBG pkg/bidstrategy/semantic/chained.go:53
> bidding strategy *semantic.ExternalHTTPStrategy returned should wait due to: Awaiting human approval [NodeID:QmcHV27b]
DBG pkg/bidstrategy/semantic/chained.go:53
> bidding strategy *semantic.ChainedBidStrategy returned should wait due to: Awaiting human approval [NodeID:QmcHV27b]

followed by mutitple logs of
    end_to_end_test.go:152: Did not see job "5d436651-0f49-4e31-a608-397a9dabf450" after 200ms

before the tests finally time out.

Human approval has been deprecated until insulated jobs is re-designed


Contributing: Separate unit tests from integration tests?

I've started contributing to draggable mostly by writing unit tests. Hopefully my work so far has been helpful! 😄
As I add unit test coverage, it strikes me that there are good reasons to start separating out unit tests from integration tests that rely on simulating user input (eg. triggering mouse events), or incorporate interactions between multiple components of the library (eg. testing that dragged elements collide).
Does this make sense? If so, what organizational approach for grouping like tests would be idiomatic to Jest?

Thank you for your contributions! We have been working on a bunch of improvements in the v1.0.0-beta.2 branch, including a new folder structure. They address some critical bugs we wanted to get out of the way first.
The new folder restructure looks something like:
- Draggable/
  - Sensors/
    - MouseSensor/
      - tests/
        - MouseSensor.test.js
      - MouseSensor.js
      - index.js
      - README.md
    - ...
  - ...
  - tests/
    - Draggable.test.js
  - Draggable.js
  - index.js
  - README.md

Have a look here: https://github.com/Shopify/draggable/tree/v1.0.0-beta.2/src
I would be happy to accept & merge your PRs if you rebase them off v1.0.0-beta.2 and point them against the v1.0.0-beta.2 branch (instead of master).
I completely agree with separating the two, I am just not set on how to structure them yet. Open to suggestions 🙂

Max,
The work of rebasing my PRs and getting the tests to pass again should
now be complete.
Thanks, by the way, for your kind words about me to your internal
recruiter, Julia Hurrelmann. She reached out to me this week to set up a
quick chat. I'm not actively looking for work, but however things shake
out, Shopify is certainly an organization that is worthy of my
attention.
Kind regards,
Tim
On 2017-10-09 19:02, Max Hoffmann wrote:

Thank you for your contributions! We have been working on a bunch of improvements in the v1.0.0-beta.2 branch, including a new folder structure. They address some critical bugs we wanted to get out of the way first.
The new folder restructure looks something like:

Draggable/
Sensors/
MouseSensor/
tests/
MouseSensor.test.js
MouseSensor.js
index.js
README.md
...
...
tests/
Draggable.test.js
Draggable.js
index.js
README.md

Have a look here: https://github.com/Shopify/draggable/tree/v1.0.0-beta.2/src
I would be happy to accept & merge your PRs if you rebase them off v1.0.0-beta.2 and point them against the v1.0.0-beta.2 branch (instead of master).
I completely agree with separating the two, I am just not set on how to structure them yet. Open to suggestions 🙂
--
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [1], or mute the thread [2].

Links:
[1]
https://github.com/Shopify/draggable/issues/31#issuecomment-335321394
[2]
https://github.com/notifications/unsubscribe-auth/AHw7ygdV8KE3TZlswgEVrNe0cxMKM8aUks5sqrQNgaJpZM4PprVL

Max,
Just following up. I've targeted my PRs to master, but it looks like
there's a new tag already made for the next beta release, v1.0.0-beta.3.
This caught me by surprise. Should I retarget my PRs? Why is master
behind the latest tagged version?
Thanks!
Tim
On 2017-10-09 19:02, Max Hoffmann wrote:

Thank you for your contributions! We have been working on a bunch of improvements in the v1.0.0-beta.2 branch, including a new folder structure. They address some critical bugs we wanted to get out of the way first.
The new folder restructure looks something like:

Draggable/
Sensors/
MouseSensor/
tests/
MouseSensor.test.js
MouseSensor.js
index.js
README.md
...
...
tests/
Draggable.test.js
Draggable.js
index.js
README.md

Have a look here: https://github.com/Shopify/draggable/tree/v1.0.0-beta.2/src
I would be happy to accept & merge your PRs if you rebase them off v1.0.0-beta.2 and point them against the v1.0.0-beta.2 branch (instead of master).
I completely agree with separating the two, I am just not set on how to structure them yet. Open to suggestions 🙂
--
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [1], or mute the thread [2].

Links:
[1]
https://github.com/Shopify/draggable/issues/31#issuecomment-335321394
[2]
https://github.com/notifications/unsubscribe-auth/AHw7ygdV8KE3TZlswgEVrNe0cxMKM8aUks5sqrQNgaJpZM4PprVL

@tsov is this resolved?

Let's close this for now, so far the convention is:

unit tests go with describe('#methodName')
and integration tests co-exists in the same test file, but live at the bottom of the file and has good test descriptions


Added support for the C11 "_Noreturn" keyword to Ghidra

This patch resolves issue #4202 by adding support for the "_Noreturn" keyword added in the C11 specification to the C header parser.

I was about to cherry pick the important commit from this PR into our patch branch but noticed your commit username/email on it might not be what you want.  Would you like to fix up your branch?

Nevermind, I am fixing it up for you.


Add ability to pass along output of one step to another and execute it

Adds:

skip_checks() optional method to task drivers to allow for bypassing checks they would fail incorrectly
updates dvflow to support icarus
add testcases for new heartbeat_sim and core changes.

@nmoroze I incorporated the changes we discussed and removed the skip_checks all together

One more thought, did you consider making exe a 'builtin' function, like maximum/minimum ? It still feels a little out of place to have it in the tools directory.

@aolofsson That was my initial thought, but the builtins don't actually execute anything but instead they are flowgraph execution modifiers (not real tools), so this didn't fit in there very well. I'll merge this in once the CI is back to functional

ok
On Thu, Jun 22, 2023 at 9:22 PM Peter @.***> wrote:

@aolofsson https://github.com/aolofsson That was my initial thought,
but the builtins don't actually execute anything but instead they are
flowgraph execution modifiers (not real tools), so this didn't fit in there
very well. I'll merge this in once the CI is back to functional
—
Reply to this email directly, view it on GitHub
https://github.com/siliconcompiler/siliconcompiler/pull/1679#issuecomment-1603505141,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARAM4D64EQ46F7FOZEYA3LXMTVWLANCNFSM6AAAAAAZJSKP6Y
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
**** All email information shared by Zero ASIC corporation in this email
should be considered proprietary and confidential  ****


How to make layouts? here?

Im using the card component but on the documentation there are no layouts available and dialogs wil lthis be implemented? or is there a layout control but not documented?

What do you mean by layout? Like a CSS Grid System (with rows and columns)?

yeap

Duplicated #78 #93 #37 #35


ClassNotFoundException: sbt.util.ShowLines

I'm getting the following build failure. Not sure if I'm missing something to get things to compile.
[INFO] --- gatling-maven-plugin:3.0.5:test (default) @ ***---
[WARNING] Missing POM for org.scala-sbt:compiler-interface:jar:1.3.1
[WARNING] Missing POM for org.scala-sbt:util-logging_2.12:jar:1.3.0
[WARNING] Missing POM for org.scala-sbt:zinc-classpath_2.12:jar:1.3.1
[WARNING] Missing POM for org.scala-sbt:compiler-interface:jar:1.3.1
java.lang.NoClassDefFoundError: sbt/util/ShowLines
	at java.base/java.lang.Class.getDeclaredMethods0(Native Method)
	at java.base/java.lang.Class.privateGetDeclaredMethods(Class.java:3139)
	at java.base/java.lang.Class.getMethodsRecursive(Class.java:3280)
	at java.base/java.lang.Class.getMethod0(Class.java:3266)
	at java.base/java.lang.Class.getMethod(Class.java:2063)
	at io.gatling.mojo.MainWithArgsInFile.runMain(MainWithArgsInFile.java:43)
	at io.gatling.mojo.MainWithArgsInFile.main(MainWithArgsInFile.java:33)
Caused by: java.lang.ClassNotFoundException: sbt.util.ShowLines
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:582)
	at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:185)
	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:496)
	... 7 more
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------

My redacted pom.xml:
<?xml version="1.0"?>
<project
    xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd"
    xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">

    <modelVersion>4.0.0</modelVersion>

    <groupId>***</groupId>
    <artifactId>****</artifactId>
    <version>1.0</version>

    <properties>
        <maven.compiler.source>1.8</maven.compiler.source>
        <maven.compiler.target>1.8</maven.compiler.target>
        <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
        <gatling.version>3.3.1</gatling.version>
        <gatling-plugin.version>3.0.5</gatling-plugin.version>
        <maven-jar-plugin.version>3.2.0</maven-jar-plugin.version>
        <!-- <scala-maven-plugin.version>4.3.1</scala-maven-plugin.version> -->
    </properties>

    <dependencies>
        <dependency>
            <groupId>io.gatling.highcharts</groupId>
            <artifactId>gatling-charts-highcharts</artifactId>
            <version>${gatling.version}</version>
            <scope>test</scope>
        </dependency>
    </dependencies>

    <build>
        <testSourceDirectory>simulations</testSourceDirectory>
        <plugins>
            <plugin>
                <artifactId>maven-jar-plugin</artifactId>
                <version>${maven-jar-plugin.version}</version>
            </plugin>
            <plugin>
                <groupId>org.codehaus.mojo</groupId>
                <artifactId>exec-maven-plugin</artifactId>
                <version>1.6.0</version>
                <executions>
                    <execution>
                        <goals>
                            <goal>exec</goal>
                        </goals>
                    </execution>
                </executions>
                <configuration>
                    <executable>bash</executable>
                    <arguments>
                        <argument>run.sh</argument>
                        <argument>${env.NS}</argument> <!-- will be empty if running with services -->
                    </arguments>
                </configuration>
            </plugin>
            <plugin>
                <groupId>io.gatling</groupId>
                <artifactId>gatling-maven-plugin</artifactId>
                <version>${gatling-plugin.version}</version>
                <executions>
                    <execution>
                        <goals>
                            <goal>test</goal>
                        </goals>
                    </execution>
                </executions>
                <configuration>
                    <!-- the name of the single Simulation class to run -->
                    <simulationClass>${env.SIMULATION}</simulationClass>
                    <runMultipleSimulations>false</runMultipleSimulations>                   <!-- if the plugin should run multiple simulations sequentially -->
                    <includes>                                                               <!-- include filters, see dedicated section below -->
                        <include></include>
                    </includes>
                    <excludes>                                                               <!-- exclude filters, see dedicated section below -->
                        <exclude></exclude>
                    </excludes>
                    <reportsOnly></reportsOnly>                                              <!-- to only trigger generating HTML reports from the log file contained in folder parameter -->
                    <runDescription>****</runDescription>             <!-- short text that will be displayed in the HTML reports -->
                    <failOnError>true</failOnError>                                          <!-- report failure in case of assertion failure, typically to fail CI pipeline -->
                    <continueOnAssertionFailure>false</continueOnAssertionFailure>           <!-- keep on executing multiple simulations even if one fails -->
                    <useOldJenkinsJUnitSupport>true</useOldJenkinsJUnitSupport>             <!-- report results to Jenkins JUnit support (workaround until we manage to get Gatling support into Jenkins) -->
                    <!-- pass extra parameters to the Gatling JVM -->
                    <jvmArgs>
                        <jvmArg>-DIAM=${env.IAM}</jvmArg>
                        <jvmArg>-DStarterkitsDir=${env.STARTERKITS_DIR}</jvmArg>
                        <jvmArg>-DServicesDir=${env.SERVICES_DIR}</jvmArg>
                        <jvmArg>-DGROUP=${env.RESOURCE_GROUP}</jvmArg>
                        <jvmArg>-DGORG=${env.ORG_GUID}</jvmArg>
                        <jvmArg>-DSPACE=${env.SPACE_GUID}</jvmArg>
                        <jvmArg>-DURL=${env.URL}</jvmArg>
                        <jvmArg>-DUsers=${env.USERS}</jvmArg>
                        <jvmArg>-DDuration=${env.DURATION}</jvmArg>
                    </jvmArgs>
                    <overrideJvmArgs>false</overrideJvmArgs>                                 <!-- if above option should override the defaults instead of replacing them -->
                    <propagateSystemProperties>true</propagateSystemProperties>              <!-- if System properties from the maven JVM should be propagated to the Gatling forked one -->
                    <!-- pass extra parameters to the Compiler JVM -->
                    <compilerJvmArgs>
                    </compilerJvmArgs>
                    <!-- if above option should override the defaults instead of replacing them -->
                    <overrideCompilerJvmArgs>false</overrideCompilerJvmArgs>
                    <!-- extra options to be passed to scalac -->
                    <!-- <extraScalacOptions>
                        <extraScalacOption></extraScalacOption>
                    </extraScalacOptions> -->
                    <disableCompiler>false</disableCompiler>                                 <!-- if compiler should be disabled, typically because another plugin has already compiled sources -->
                    <simulationsFolder>${project.basedir}/simulations</simulationsFolder> <!-- where the simulations to be compiled are located -->
                    <resourcesFolder>${project.basedir}</resourcesFolder> <!-- where the test resources are located -->
                    <resultsFolder>${project.basedir}/target/gatling</resultsFolder>         <!-- where the simulation log and the HTML reports will be generated -->
                </configuration>
            </plugin>
        </plugins>
    </build>
</project>

Looks like a corrupted jar in your local maven cache. Or something wrong with your pom.
But https://github.com/gatling/gatling-maven-plugin-demo works our of the box with Java 11 or Java 14.
Problem is on your side.

Since no one here was willing to help a bit more, posting this in case others have the same issue: I changed my pom.xml to use the scala-maven-plugin to do the compilation and setting disableCompiler in the gatling-maven-plugin configuration to true.
Before, I was letting gatling-maven-plugin do the compilation. It seems to me something is wrong in gatling-maven-plugin since it couldn't compile, but I guess we'll never know.

Since no one here was willing to help a bit more

This is a bug tracker, to be used for software bugs and feature requests.
If you want help with your own set up, please use the community mailing list.
As I already said, your problem is most likely one jar is corrupted in your local maven repository.
A quick search on the internet would give you that the jar containing the class sbt.util.ShowLines (from your stacktrace) is org.scala-sbt/util-logging_2.12.
The fact that scala-maven-plugin worked most likely means they don't use the same version and you have 2 different version of this jar in your local maven cache: one corrupted that gatling-maven-plugin is trying to use but can't unzip and one fine that scala-maven-plugin uses successfully.
Again, please investigate the content of your local maven cache.

You’re assuming I didn’t. I deleted the scala-sbt and scala-lang  folders in my local maven cache. Still had the same issue afterwards.
If they don’t use the same version, that sounds problematic at the very least. My gut tells me I should be able to use either method of compilation successfully out of the box.

You posted a question on SoF  stating that now your issue is with com.trueaccord.scalapb.GeneratedEnum which is in a different jar, neither in org.scala-sbt not org.scala-lang organisations. You have multiple corrupted jars. Cleaning the scala-sbt fixed the first corrupted jar and helped you get rid of the ClassNotFoundException about sbt.util.ShowLines and now you hit a second one.

If they don’t use the same version, that sounds problematic at the very least. My gut tells me I should be able to use either method of compilation successfully out of the box.

No.
scala-maven-plugin:4.3.1 depends on zinc:1.3.2 while gatling:3.3.1 depends on zinc:1.3.1. Different versions, different jars.
Maven installed some corrupted jars when installing zinc:1.3.1 dependencies and valid jars when installing zinc:1.3.2 ones.


Improve OptunaSearchCV (accept Mapping in addition to dict for param_distributions and allow use of OptunaSearchCV with cross_val_predict)

Motivation

I have two motivations.

Accept Mapping in addition to dict for param_distributions.
Allow use of OptunaSearchCV with cross_val_predict.

Description of the changes

1. Accept Mapping in addition to dict for param_distributions.
Changed the accepted type from dict to Mapping.
MappingProxyType and other types cannot be made to pickle, which causes an error during parallelization, so the specification has been changed to change to dict after accepting the Mapping type.
2. Allow use of OptunaSearchCV with cross_val_predict.
In scikit-learn's HasMethods, predict (and related methods) get error because of 'property', so define them as method.

@nabenabe0928 Could you review this PR?

@yu9824
So, I, first of all, found the exceptions where your changes do not work:

https://scikit-learn.org/stable/modules/generated/sklearn.tree.DecisionTreeClassifier.html#sklearn.tree.DecisionTreeClassifier.predict_proba
https://scikit-learn.org/stable/modules/generated/sklearn.tree.ExtraTreeClassifier.html#sklearn.tree.ExtraTreeClassifier.predict_proba

It means we cannot merge this PR as is, so I will close this PR for now.
However, please feel free to open this PR again if you find any other solutions!
Or you can alternatively post an issue for this regard with the steps to reproduce.

Thank you for your review and comment.
I'll create issues alternatively to show you how to reproduce errors.

I've created the issues (#171, #173) and the PRs (#172, #174).
I cannot reproduce your error below. Please tell me in the new PRs.

@yu9824
So, I, first of all, found the exceptions where your changes do not work:

https://scikit-learn.org/stable/modules/generated/sklearn.tree.DecisionTreeClassifier.html#sklearn.tree.DecisionTreeClassifier.predict_proba
https://scikit-learn.org/stable/modules/generated/sklearn.tree.ExtraTreeClassifier.html#sklearn.tree.ExtraTreeClassifier.predict_proba

It means we cannot merge this PR as is, so I will close this PR for now. However, please feel free to open this PR again if you find any other solutions! Or you can alternatively post an issue for this regard with the steps to reproduce.

I appreciate you taking the time to review this.


Trim extra data from iden8mebi test case

This test case was originally generated through a fuzzer. Apparently the resulting test vector wasn't normalized when added to test.c, and extraneous data was left tacked onto the end.

Fixed #194


Improved error handling for composed property wrapper mismatches

The purpose of this pull request is to remedy a confusing error associated with wrapped value type mismatches to give more context for the wrapped value type in the error message. So that when running this block of code:
@propertyWrapper struct Wrapper<Value> {
    var wrappedValue: Value
}

struct User {
    @Wrapper<String> @Wrapper var value: Int
}

the new error message for the above example would be
composed wrapper type 'Wrapper<Int>' does not match the type of 'Wrapper<String>'.wrappedValue, which is 'String'

Should this change not have been made, the error would have looked like the following:
composed wrapper type 'Wrapper<Int>' does not match former 'wrappedValue' type 'String'

which is rather confusing.
NOTE: There is an existing diagnostic, called wrapped_value_mismatch that should be combine with this diagnostic, called composed_property_wrapper_mismatch. There are too many separate modifications to include in this PR in order to combine these two diagnostics, so that will be done in a separate PR.
Resolves:
https://bugs.swift.org/browse/SR-14130
Thank you @hborla for all of your help with this!

Overall looks great! I have left a couple comments about documentation and code but logic-wise this is on point.

Thank you so much! I really appreciate your comments I will definitely go back and implement the changes you suggest :)

A few more small changes.
Ill get to working on these, thank you for the feedback :)

Happy to help, thank you for making this work!

Happy to help, thank you for making this work!

Of course, it's been awesome to have the chance to work on this project, I've been learning so much!

@swift-ci please smoke test

Perfect!

Awesome, thank you!!!! In that case then, in regards to @LucianoPAlmeida's comment, @xedin, should we make the removal of CTP_ComposedPropertyWrapper a project for another PR, or attempt to delete it in this one?

in regards to @LucianoPAlmeida's comment, @xedin, should we make the removal of CTP_ComposedPropertyWrapper a project for another PR, or attempt to delete it in this one?

Apparently I didn't press "Comment" :/ I think it's okay to clean that up separately.

@swift-ci please smoke test Windows platform

@swift-ci please smoke test Windows platform

@Strieker Looks like these changes regressed validation-test/compiler_crashers_2_fixed/sr11599.swift, you'd have to take a look at the before we can merge.

Looks like these changes regressed validation-test/compiler_crashers_2_fixed/sr11599.swift, you'd have to take a look at the before we can merge.

@Strieker you can run the validation tests locally by passing --validation-test to build-script. Looks like the test just needs to be updated with the new error message!

you can run the validation tests locally by passing --validation-test to build-script

Or -T which would run both tests and validation-tests.

in regards to @LucianoPAlmeida's comment, @xedin, should we make the removal of CTP_ComposedPropertyWrapper a project for another PR, or attempt to delete it in this one?

Apparently I didn't press "Comment" :/ I think it's okay to clean that up separately.

No problem, sounds good! Thank you :)

you can run the validation tests locally by passing --validation-test to build-script

Or -T which would run both tests and validation-tests.

@xedin and @hborla I will make sure to get on that, thank you for telling me how to work on fixing these validation tests :)

you can run the validation tests locally by passing --validation-test to build-script

Or -T which would run both tests and validation-tests.

@xedin and @hborla I will make sure to get on that, thank you for telling me how to work on fixing these validation tests :) Also as a heads up, my .zshrc file is currently corrupted/something is wrong with it. I can't run ls, git, or anything else from terminal right now at the moment. I have to talk to a tech support person or something to figure out what's wrong, since I've been debugging for a hot minute to try and figure out what went wrong, so it might be sometime tomorrow before I can run the commands you told me to run, sorry about that.

No rush! You can try running /bin/sh from your terminal to see whether it would allow you to run commands again, if so - it’s something with your PATH. Run env to see what bindings you currently have and unbind broken variables.

No rush! You can try running /bin/sh from your terminal to see whether it would allow you to run commands again, if so - it’s something with your PATH. Run env to see what bindings you currently have and unbind broken variables.

Thank you so, so much for this advice! Yes, I was assuming it was something having to do with my PATH, I will try your suggestions and see what happens, because I've been trying to change my PATH and I can't seem to get it working.

@swift-ci Please smoke test


Add meganopperman1's file

created a branch, created a file and made a commit, and opened a pull request


Adding a Zipline Binder?

I was just looking through some JupyterLab repos (among others) and was reminded of Binder. I think it'd be pretty cool to have a Binder in the README for people to play around with Zipline's beginner tutorial before they decide whether or not they'd like to install it.

Thoughts @richafrank @ssanderson @llllllllll?

closing because:

zipline is very data intensive and not a great fit for ephemeral notebooks
quantopian is free hosted zipline :)


can't get psh modules to work

I'm having problems getting a response from the psh command when using provided modules.
Tried on two different Win10 machines as beacon (1909 build 18363.720 and 18363.592)
Standard ps commands are working e.g. psh -c Write-Output $PSversionTable
But when running modules I don't receive any response
shad0w(User@WINDEV2002EVAL) ≫ psh -m GetHash -c Invoke-GetHash
[i] Hosting module 'GetHash' (22216 bytes) => /KEiePaTceM
[i] Beacon (b810bd86ec9ec08516b4a609d8d85899) received task

Supplied ps1 scripts alone do work when I execute them directly on the target. I'm not sure how to debug this, so maybe someone could verify it's not a local problem.

Ah, I got it. It wasn't clear for me I had to start the listener with the -e flag
I'll add a warning to psh command if this flag is missing

Okay cool, will implement a version of #55 once I have finished rewriting the way modules will be called by the team server


Enhancement suggestion

I hesitate to ask this but if I take a few days off the -c parameter requires that I remember which day I last made a note for. It would be useful to have it where if there were no value for the c parameter textnote would cycle back and find the last day's note. I have not looked at the code for textnote so don't know if the filename is guaranteed to match the date for that note, but if it is cycling back should work. I really enjoy using textnote. Thank-you for both writing it and making it available.

Hi @imarsman, thanks for the suggestion and for the kind words.  I'm really glad to hear that you are using and finding value in textnote!
I think the suggested enhancement is a good one and makes a lot of sense.  I also find myself wanting a similar feature.  I did not implement it initially because there wasn't a clear way to do it efficiently.  Because the date format can be specified in the configuration, we are not guaranteed that sorting the string file names will give us the most recent file.  I also considered decrementing the time.Time object by one day and checking if that file exists - if it doesn't decrement again and keep going.  In the case where textnote hasn't been used for a long time, this would result in many calls to os.stat.  Let me think on the best way to approach this and I will see about implementing it in the next few days.
FWIW - because I do use lexicographically sortable date conventions for my file names (textnote's default), we can rely on sorting to get the latest.  So, my current workflow for accomplishing what you are asking about is:
textnote open --copy $(tnlatest) -s TODO -x

where tnlatest is the following function:
tnlatest () {
	ls -lhFG $TEXTNOTE_DIR | grep -v 'archive' | awk '{print $9}' | sort -r | head -n 1 | cut -f 1 -d '.'
}

While this works, I would like to make it a proper part of textnote as you suggested and make it robust to any date formatting.
Thanks again and I'll update here in a few days!

Thanks! That's great!

I believe I have a reasonable implementation, look for a new release in a day or two (maybe even sooner)!

The enhancement, along with several other changes are available in the latest v1.2.0 release.
Thank you again for opening this issue and for using textnote!


server.start(config) crash

Hi, I am trying to use moquette code for custom MQTT based system.
When I run it, server.start(config) crashes with both
default
IConfig config = new FilesystemConfig();
and my custom
File configFile = new File("broker/config/moquette.conf");
which is config file from project source code and is read ok.
Here's the stack trace:
Exception in thread "main" java.io.IOError: java.io.IOException: storage has invalid header
at org.mapdb.StoreDirect.checkHeaders(StoreDirect.java:254)
at org.mapdb.StoreWAL.checkHeaders(StoreWAL.java:115)
at org.mapdb.StoreDirect.(StoreDirect.java:207)
at org.mapdb.StoreWAL.(StoreWAL.java:74)
at org.mapdb.DBMaker.extendStoreWAL(DBMaker.java:981)
at org.mapdb.DBMaker.makeEngine(DBMaker.java:758)
at org.mapdb.DBMaker.make(DBMaker.java:701)
at io.moquette.spi.persistence.MapDBPersistentStore.initStore(MapDBPersistentStore.java:98)
at io.moquette.spi.impl.ProtocolProcessorBootstrapper.init(ProtocolProcessorBootstrapper.java:77)
at io.moquette.server.Server.startServer(Server.java:153)
at io.moquette.server.Server.startServer(Server.java:135)
at io.moquette.server.Server.startServer(Server.java:127)
at io.moquette.server.Server.startServer(Server.java:94)
at io.moquette.server.Server.main(Server.java:72)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:606)
at com.intellij.rt.execution.application.AppMain.main(AppMain.java:147)
Caused by: java.io.IOException: storage has invalid header
... 19 more
Code in MapDB project has changed considerably: there is NO StoreDirect.java file any longer.
Please help. Any ideas are appreciated!

Probably the MapDB file is corrupted,is not a fail from Moquette side,
probably the process was filled instead of clean shutdown and this
corrupeted the file. The only road I can think, is to drop that mapdb
files and restart the process.
Andrea
On Thu, Oct 20, 2016 at 10:42 AM, George Matveev<EMAIL_ADDRESS>wrote:

Hi, I am trying to use moquette code for custom MQTT based system.
When I run it, server.start(config) crashes with both
default
IConfig config = new FilesystemConfig();
and my custom
File configFile = new File("broker/config/moquette.conf");
which is config file from project source code and is read ok.
Here's the stack trace:
Exception in thread "main" java.io.IOError: java.io.IOException: storage
has invalid header
at org.mapdb.StoreDirect.checkHeaders(StoreDirect.java:254)
at org.mapdb.StoreWAL.checkHeaders(StoreWAL.java:115)
at org.mapdb.StoreDirect.(StoreDirect.java:207)
at org.mapdb.StoreWAL.(StoreWAL.java:74)
at org.mapdb.DBMaker.extendStoreWAL(DBMaker.java:981)
at org.mapdb.DBMaker.makeEngine(DBMaker.java:758)
at org.mapdb.DBMaker.make(DBMaker.java:701)
at io.moquette.spi.persistence.MapDBPersistentStore.initStore(
MapDBPersistentStore.java:98)
at io.moquette.spi.impl.ProtocolProcessorBootstrapper.init(
ProtocolProcessorBootstrapper.java:77)
at io.moquette.server.Server.startServer(Server.java:153)
at io.moquette.server.Server.startServer(Server.java:135)
at io.moquette.server.Server.startServer(Server.java:127)
at io.moquette.server.Server.startServer(Server.java:94)
at io.moquette.server.Server.main(Server.java:72)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(
NativeMethodAccessorImpl.java:57)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(
DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:606)
at com.intellij.rt.execution.application.AppMain.main(AppMain.java:147)
Caused by: java.io.IOException: storage has invalid header
... 19 more
Code in MapDB project has changed considerably: there is NO
StoreDirect.java file any longer.
Please help. Any ideas are appreciated!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/andsel/moquette/issues/222, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA_Y0RPFJrgWgmrcii8v11EgMuu2tDfyks5q1ymGgaJpZM4Kb3LJ
.

Really funny things happen.
I believe done it before in Eclipse and it did not work.
I removed  mapdb stuff: rm moquette_store.mapdb*
and launched the project again and now it Works!
Thank you Andrea! Very nice project/code!
P.S. It seems choice of IDE makes a lot of difference.


added hightrust.id

Hi @akolio,
are you able to sign your commit so the DCO check can pass? Then we are able to merge your PR :)

Thanks @cre8!
Luckily GitHub had straightforward steps to add the signoff.

I'm not opposed to adding Hightrust.id, but wondering why the entry is basically blank.
@akolio, is there a reason you didn't provide more information? If you're affiliated with Hightrust, it should be relatively easy for you to find out information about supported credential formats, signature algorithms, etc.
At least the urlWebsite would be nice (it's mandatory in the schema) so that the audience has a pointer on where to look for more information.
@cre8, I think we shouldn't accept blank entries. They provide very little value to the audience. I removed many entries from the previous iteration due to missing data.

We were planning on adding fields as we go. Company URL was added as CompanyUrl which is what the "add wallet" tool on https://openwallet-foundation.github.io/digital-wallet-and-agent-overviews-sig/ wanted. But I see it might be deprecated.
I noticed there were other differences in this json too compared to the ones on GitHub. E.g. there's no "linkToApp" (instead there are fields such as "urlAppStore", "urlGooglePlayStore" and "urlWebsite").
The add wallet tool might need a little refreshing 🙂

Thanks!
@cre8, will you check that the attributes in the add component correspond to the schema?

@samuelmr I commented the check out since we had the old schemas included, see: https://github.com/openwallet-foundation/digital-wallet-and-agent-overviews-sig/blob/main/.github/workflows/main.yml#L41
I will add it again


All elements

Feature description
We need to add all the extra element types. Most are fairly trivial to add, but e.g. adding all the desired shapes to the shape library require a decent level of abstraction to reduce code duplication.
This includes the five main elements:

Image (special element for Gif perhaps - with extra settings)
Video
Text
Shape (new, probably SVG based)
Embed

And some elements not necessarily duplicated and definitely less important:

Source code
Preformatted
Quote
List
Table
Verse
Post author
Post date
Post title
HTML

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance criteria

A suggestion is made based on the findings

Implementation brief

QA testing instructions

Demo

Changelog entry

Time to close this catch-all issue


league-of-moveable-type: fix raleway integration

The raleway fonts need to be copied from their /share folder to be
included properly.
Found out that I could test it via fc-list | grep -i rale, and everything seems to work now.
cc @Mic92 since you merged the first PR.

Like @rycee said, I would do the following:
diff --git a/pkgs/data/fonts/league-of-moveable-type/default.nix b/pkgs/data/fonts/league-of-moveable-type/default.nix
index 4228105..684f837 100644
--- a/pkgs/data/fonts/league-of-moveable-type/default.nix
+++ b/pkgs/data/fonts/league-of-moveable-type/default.nix
@@ -21,10 +21,10 @@ stdenv.mkDerivation rec {
   sourceRoot = ".";

   installPhase = ''
-    mkdir -p $out/share/fonts/truetype
-    cp */*.otf $out/share/fonts/truetype
+    mkdir -p $out/share/fonts/opentype
+    cp */*.otf $out/share/fonts/opentype
     # for Raleway, where the fonts are already in /share/…
-    cp */share/fonts/truetype/*.otf $out/share/fonts/truetype
+    cp */share/fonts/truetype/*.otf $out/share/fonts/opentype
   '';

   meta = {

@Mic92 @rycee Yeah, we can do that. I only put it in there because it was like this before.

I changed both packages to opentype folders.

Thanks!


Sending current value of perPage select on datatable.perpage emit

I need to listen for perPage dropdown changes and do something with the selected value.
I'm currently doing:
dataTable.on("datatable.perpage", function(){
    // Do something here
});

At the moment the value isn't passed in - I'd love it to be if possible :)

ie: Change this:
that.emit("datatable.perpage");

into maybe this?:
that.emit("datatable.perpage", o.perPage);


In which file should the mainWindowController reference be created in?

https://docs.microsoft.com/en-us/xamarin/mac/app-fundamentals/xibless-ui
The article does not specify in which file these lines should be added in
private MainWindowController mainWindowController;
...
mainWindowController = new MainWindowController ();
mainWindowController.Window.MakeKeyAndOrderFront (this);

Thanks for submitting an issue and we apologize for the delayed response. The requested updates haven't been made since the creation of the issue, and the timeline for resolution may vary based on resourcing, so the issue is being moved to an internal repo to better track it.


Failing to clone multiple BundleInstances of the same Bundle

This kinda code had been failing (silently), I think for some time:
def test_nested_bundle_conn2():
    """Test some bundles which have multiple instances of the same bundle as attributes."""

    @h.bundle
    class Ab:
        a, b = h.Signals(2)

    @h.bundle
    class FourAbs:
        i0 = Ab()
        i1 = Ab()
        i2 = Ab()
        i3 = Ab()

    @h.module
    class HasAbPort:
        ab = Ab(port=True)

    @h.module
    class HasThose:
        four_abs = FourAbs()
        i0 = HasAbPort(ab=four_abs.i0)
        i1 = HasAbPort(ab=four_abs.i1)
        i2 = HasAbPort(ab=four_abs.i2)
        i3 = HasAbPort(ab=four_abs.i3)

    h.elaborate(HasThose)

    assert HasThose.instances["i0"].conns["ab_a"].name == "four_abs_i0_a"
    assert HasThose.instances["i0"].conns["ab_b"].name == "four_abs_i0_b"
    assert HasThose.instances["i1"].conns["ab_a"].name == "four_abs_i1_a"
    assert HasThose.instances["i1"].conns["ab_b"].name == "four_abs_i1_b"
    assert HasThose.instances["i2"].conns["ab_a"].name == "four_abs_i2_a"
    assert HasThose.instances["i2"].conns["ab_b"].name == "four_abs_i2_b"
    assert HasThose.instances["i3"].conns["ab_a"].name == "four_abs_i3_a"
    assert HasThose.instances["i3"].conns["ab_b"].name == "four_abs_i3_b"

E.g. https://github.com/dan-fritchman/Hdl21/actions/runs/3753792417/jobs/6377386972#step:5:1902
There was no dedicated test for this, and it didn't throw any exceptions (until the assertions in the test above).

This boils down to how we flatten Bundles and clone their Signals.
There was a key step which made a per-Bundle-instance mapping from "original signal" to "new signal". That logic fell over with multiple instances of the same Bundle, in the same scope.
That whole bit had bugged me for some time as being too complicated. I think it was. Commit 6299f34, now in #85, re-did it. I think it's aways simpler. (If the line-count is any indication - it's a lot simpler.) In short:

FlatBundleDef, which tried to be a cacheable "half-step" for each Bundle definition, goes away.
FlatBundleInst, the analogous thing per-instance, also goes away
Each BundleInstance and AnonymousBundle now get transformed into a BundleScope. They nest, with each level having different keys to look up sub-scopes.
The PathStr also goes away, and is replaced by a hashable version of Path.

flatten_bundle_inst is the core of this, and becomes like 20 lines. The key is that for every BundleInstance, for every signal we encounter, no matter what - clone it.


Fix #99

Refer to this link for build results (access rights to CI server needed):
http://guh.guru:8080/job/guh-webinterface-pull-request/67/


[Android] - NullPointerException: Attempt to read from field 'java.lang.Object com.google.maps.api.android.lib6.gmm6.util.g.c'

I'm submitting a ...  (check one with "x")

[ ] question
[X] any problem or bug report

OS:  (check one with "x")

[X] Android
[ ] iOS
[ ] Browser

cordova information:  (run $> cordova plugin list)
cordova-plugin-device 2.0.3 "Device"
cordova-plugin-geolocation 4.0.2 "Geolocation"
cordova-plugin-googlemaps 2.6.2 "cordova-plugin-googlemaps"
cordova-plugin-ionic-keyboard 2.2.0 "cordova-plugin-ionic-keyboard"
cordova-plugin-ionic-webview 4.1.1 "cordova-plugin-ionic-webview"
cordova-plugin-splashscreen 5.0.3 "Splashscreen"
cordova-plugin-whitelist 1.3.4 "Whitelist"

If you use @ionic-native/google-maps, please tell the package.json (only @ionic-native/core and @ionic-native/google-maps are fine mostly)
@ionic-native/core :  4.20.0
@ionic-native/google-maps :  4.21.0

Current behavior:
Sometimes when entering one of my map pages. The app would crash.
Fatal Exception: java.lang.NullPointerException: Attempt to read from field 'java.lang.Object com.google.maps.api.android.lib6.gmm6.util.g.c' on a null object reference
I don't know the steps to reproduce it.
The page does not do anything complex. It interact with the map plugin using the following function

creates map
show/hide map
animate camera (every second, based on location from Geolocation plugin)

Expected behavior:
Does not crash
Screen capture or video record:
N/A
Related code, data or error log (please format your code or data):
Fatal Exception: java.lang.NullPointerException: Attempt to read from field 'java.lang.Object com.google.maps.api.android.lib6.gmm6.util.g.c' on a null object reference
       at com.google.maps.api.android.lib6.gmm6.util.h.a(:com.google.android.gms.dynamite_mapsdynamite<EMAIL_ADDRESS>(100400-0):4)
       at com.google.maps.api.android.lib6.gmm6.util.h.c(:com.google.android.gms.dynamite_mapsdynamite<EMAIL_ADDRESS>(100400-0):7)
       at com.google.maps.api.android.lib6.gmm6.vector.bx.a(:com.google.android.gms.dynamite_mapsdynamite<EMAIL_ADDRESS>(100400-0):18)
       at com.google.maps.api.android.lib6.gmm6.vector.gl.drawable.q.a(:com.google.android.gms.dynamite_mapsdynamite<EMAIL_ADDRESS>(100400-0):6)
       at com.google.maps.api.android.lib6.gmm6.vector.gl.drawable.s.a(:com.google.android.gms.dynamite_mapsdynamite<EMAIL_ADDRESS>(100400-0):81)
       at com.google.maps.api.android.lib6.gmm6.vector.gl.drawable.ae.a(:com.google.android.gms.dynamite_mapsdynamite<EMAIL_ADDRESS>(100400-0):53)
       at com.google.maps.api.android.lib6.gmm6.vector.bd.a(:com.google.android.gms.dynamite_mapsdynamite<EMAIL_ADDRESS>(100400-0):7)
       at com.google.maps.api.android.lib6.gmm6.vector.bt.a(:com.google.android.gms.dynamite_mapsdynamite<EMAIL_ADDRESS>(100400-0):179)
       at com.google.maps.api.android.lib6.gmm6.vector.av.run(:com.google.android.gms.dynamite_mapsdynamite<EMAIL_ADDRESS>(100400-0):60)

Support this plugin activity
I appreicate if you give me a beer :beer: from here

Please try to upgrade to 2.7.0 at least.

And it's hard to detect the problem. Please provide an example project repo that reproduce this issue.

@wf9a5m75
No worries Masashi. It is happening in production, and it is compiled with the plugin version 2.6.2.
I'll update to 2.7.0 in dev, and see if I encounter the issue.
However, I can't reproduce at will.
Because the logcat error do not provide any hints of source of bug, I created this issue to see if there are others who may be experiencing the same issue. If there are others, we maybe able to collect additional information. And hopefully it can help me deduce the cause, and debug the issue.

Yeah, I understand. If this error is occurred many times, please ask to Google.
At least this issue is occurred in Google Maps SDK for Android itself.
(I guess the end user's Google Play Services was old version)
https://stackoverflow.com/questions/34188394/nullpointerexception-from-google-maps

Thanks mate. I saw that SO question as well.
I will update this thread if I get any more information

BTW, what is the recommended play service for this plugin?
I'm currently using 17.0.+ in my package.json setup

I'm using
"GOOGLE_MAPS_PLAY_SERVICES_VERSION": "17.0.0"
also
"ANDROID_SUPPORT_V4_VERSION": "27.1.1"
and these settings work for me.
On an aside note, I'm also using the cordova-android-play-services-gradle-release plugin as I have many other cordova plugins where "PLAY-SERVICES-MAPS": "17.0.0" is set.

Thanks mate.
I've got similar setup.
Play Service: 17.+
Support V4: 27.+


feature: pass initial value to search components

Feature Request
Ability to pass an initial value through an input property to both search components.
What is the expected behavior?
It should be possible to do something like:
<td-search-box
backIcon="search"
[placeholder]="'TERM.FILTER' | translate"
[showUnderline]="true"
[value]="filter$ | async"
[debounce]="500"
(searchDebounce)="handleSearch($event)">
  </td-search-box>

https://github.com/Teradata/covalent/pull/1041 will add ngModel support to both search-input and search-box


The provided scripts generate invalid json schema

I have following installation:
python 2.7.9
jsonschema 2.5.1
If I build the package with the script provided ./scripts/build.sh, an invalid json is generated.
The first entry of the packages looks like this:
{
  "packages":[
    {
      "versions":{}
    },

What am I doing wrong?

@aquamatthias What branch are you running this against?

Branch is version-1.x

Am 10.07.2015 um 20:40 schrieb José Armando García Sancio<EMAIL_ADDRESS>@aquamatthias What branch are you running this against?
—
Reply to this email directly or view it on GitHub.

Have you tried building this recently? I just tried and it worked for me :/

Clean checkout resolved the problem. Closing this ticket.