Fix deployment on OCP 4.6

Check against changing the Service's clusterIP, and retain the existing value as OCP 4.6 does not tolerate that. Later versions just ignore the field.

Nothing to document or no support impact, but let's have qe approve the PR.

/label qe-approved
Without the pr, the operator installation will fail due to "Error creating metrics service/secret","error":"Service \"metrics\" is invalid: spec.clusterIP: Invalid value: \"\": field is immutable","stacktrace""; with the pr, the installation will succeeded.
Test details seen from: http://pastebin.test.redhat.com/1075104
 $ oc get clusterversion
NAME      VERSION                             AVAILABLE   PROGRESSING   SINCE   STATUS
version   4.6.0-0.nightly-2022-09-20-111416   True        False         96m     Cluster version is 4.6.0-0.nightly-2022-09-20-111416
$  make images && make push
...
$ make catalog && make catalog-deploy
make: go: Permission denied
/usr/local/bin/opm index add --container-tool podman --mode semver --tag quay.io/xiyuan/compliance-operator-catalog:0926 --bundles quay.io/xiyuan/compliance-operator-bundle:0926
...
podman push quay.io/xiyuan/compliance-operator-catalog:0926
...
$ oc get sub
NAME                      PACKAGE               SOURCE                CHANNEL
compliance-operator-sub   compliance-operator   compliance-operator   alpha

$ oc get sub
NAME                      PACKAGE               SOURCE                CHANNEL
compliance-operator-sub   compliance-operator   compliance-operator   alpha
$ oc get ip
NAME            CSV                           APPROVAL    APPROVED
install-bzxll   compliance-operator.v0.1.55   Automatic   true
$ oc get csv
NAME                          DISPLAY               VERSION   REPLACES   PHASE
compliance-operator.v0.1.55   Compliance Operator   0.1.55               Succeeded
$ oc get pod
NAME                                              READY   STATUS    RESTARTS   AGE
compliance-operator-78dbd98fc8-b9q6c              1/1     Running   1          5m15s
ocp4-openshift-compliance-pp-75dc6b78-d7vpv       1/1     Running   0          4m57s
rhcos4-openshift-compliance-pp-548f8f8586-tlhkb   1/1     Running   0          4m57s

/label qe-approved


Add Edit Task to UserGuide.md

Update UG to include the edittask feature.

Codecov Report

Merging #43 (0a4f30e) into master (23c74ad) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #43   +/-   ##
=========================================
  Coverage     72.98%   72.98%
  Complexity      505      505
=========================================
  Files            85       85
  Lines          1536     1536
  Branches        172      172
=========================================
  Hits           1121     1121
  Misses          362      362
  Partials         53       53

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 23c74ad...0a4f30e. Read the comment docs.


app.keyboard.wasPressed(pc.KEY_DELETE) always returns false

The wasPressed function never returns true on releasing the DELETE key.

I notice our keyboard handling code reference the keyIdentifier property. According to MDN, this is deprecated so I guess we should update:
https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/keyIdentifier

On Chrome (Windows), the keypress event has a keyIdentifier of 'U+007' but when was pressed is called, it's looking for 'Delete' in the keymap, not 'U+007'.

Should be fixed in https://github.com/playcanvas/engine/commit/90b53a1854d213455a6ae336acee52d89cad7706

Confirmed.


Fixed breaking didRefreshRegistrationToken problem

When building my flutter app, I ran into this problem:
https://stackoverflow.com/questions/44116626/fira-appdelegate-messagingdidrefreshregistrationtoken-unrecognized-selecto
(not my post, but exactly the same problem)
And wrote this minimalistic fix. Since I do not think I am proficient enough concerning the whole plugin and FCM, I did not implement an actual body for the new function. Is there an expected behaviour, which should be implemented?
Take care!

Trying to fix cla by commenting.

Hi @BenSower we fixed the Travis issue in https://github.com/flutter/plugins/pull/220, so if you merge with master hopefully it will pass now.

Travis passing now :-)

I don't think the changes proposed in this PR are enough to adopt to the underlying changes in the Firebase SDK. For reference, see https://github.com/firebase/quickstart-ios/commit/d1ee0c62e522313c25f4af903ea0da3fb8d4d6cc and later changes to learn how the Firebase Quickstart Examples got updated. We will have to do similar changes to this plugin.
/cc @kroikie, who might be able to help with this.

Hi @BenSower thanks for the PR, @goderbauer is correct there would be a bit more required for this change.

Implement the didRefreshRegistrationToken callback. This should do what tokenRefreshNotification currently does. There is no longer any need for the observer so it can be removed.
Remove connectToFcm. All FCM messages both notification-messages and data-messages are now sent via APNs so there is no need to 'connect' unless you are sending upstream messages which this SDK does not yet support.
We are still using iOS 9 handlers for incoming messages and should add the iOS 10 handlers but that can/should be done in a separate PR.

Hi @kroikie, I

moved the logic from 'tokenRefreshNotification' to 'didRefreshRegistrationToken',
removed 'tokenRefreshNotification' as well as the observer registration
removed the 'connectToFcm' method and all related calls

what do you think?

@collinjackson I updated the changelog and the pubspec. Just a general question:
What is the best practice concerning the author(s) field? Do I have to add myself to the authors field, since it's not only Flutter Team<EMAIL_ADDRESS>anymore?

@BenSower do you mean the authors field in pubspec.yaml? I believe we have been using that for only the main/original author, but please do add yourself to the AUTHORS file: https://github.com/flutter/plugins/blob/master/AUTHORS

@mit-mit Okay, I added myself to the Authors file.

@kroikie this one looks ready to go. Can you give it a last look, please?

@kroikie @mit-mit @collinjackson : Heyho, not trying to be rude, just wondering: Might there be another reviewer, so this pr doesn't get too old? :-)

Sorry for the delay, this looks good to me.

Perfect, thank you for your time and have a nice weekend!

This is live now: https://pub.dartlang.org/packages/firebase_messaging


Arrays and negative indexes in Perl

Hi,
i ran with this command:

gsat graphCorrection -conf  mytest1.conf

im getting this error:

BLAST ran out of memory: Query sequence buffer

Reading gfa file ...
Reading path file ...
Reading reads file ...
Constructing index for reads  ...

Modification of non-creatable array value attempted, subscript -1 at /public/home/softwares/GSAT-main/bin/../lib/GSAT/graphIO.pm line 21.

what could be the reason?
thank.

It seems that the problem occurred during the blast process: the input query and subject files are too large which made the blastn program ran out of the memory and failed to output the alignments.
Please change the minimap2 option (in your config file, e.g., mytest1.conf) to use the minimap2, instead of blastn, for large sequencing dataset.


DataMapper support with Doctrine tests

Now you can use have FactoryMuffin to work with DataMapper pattern, especially with Doctrine.
To have FactoryMuffin to be initialized with Doctine you should use RepositoryStore instead of ModelStore class:
$em = EntityManager::create($dbParams, $config);
$fm = new FactoryMuffin(new RepositoryStore($em));

Unlike model store you can't use factory prefix to create associated models. factory creation method is supported to create a record and recturn its id, while Doctrine expects to receive a complete entity into association. That's why you should use entity to get a record into association
'user_id' => 'factory|User', // to receive id of a record
// vs
'user' => 'entity|User', // to receive a record

Also Doctrine added as dev dependency and tests for it added.

Thanks. I've left a few comments.

Could you skip the failing tests on hhvm too please?

Done.

updtd

done

Ping-ping?

Done. But I hope that was the last part of codestyling and dockblock bureaucracy )

Thanks for all your work here. I appreciate it very much. I'll do a final review of this at the weekend and then merge. :)

Ok, will be waiting for that and for final release of 3.0 :+1:

@GrahamCampbell yep, about 3.0. With this PR I have plans to include factory-muffin in Codeception 2.2 but I need to know your release plans so to decide If I can stick to it. It's not a nice idea to rely on beta dependency (

Thank you @DavertMik.

@DavertMik v3.0.0 is released now, even with incomplete upgrading docs. That was the main reason I was holding off, but it'a already like 18 months late, so I'm just going to ship it.

Factory Muffin 3.x will be maintained into the future, and follows semver.

Awesome!


Integrated Terminal 's Right click menu disappeared!

VSCode Version: Code - Insiders 1.9.0-insider (9486bf472c5c558e4f1793e0e746da88d071b1cc, 2016-12-29T07:11:17.628Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

githistory
donjayamanne
0.1.4

output-colorizer
IBM
0.0.8

cpptools
ms-vscode
0.9.3

vscode-icons
robertohuertasm
4.3.0

Steps to Reproduce:

This was intentional, it now behaves similar to cmd.exe where right click when you have a selection will copy (and drop the selection), and right click without a selection will paste. You can turn it off with "terminal.integrated.rightClickCopyPaste": false. See #14202.


State set race conditions

When calling this.refs.toastContainer.clear() calls to setState() in ToastContainer::_handle_toast_remove() enter a race condition causing the state to reset instead of removing all elements.
The end result is the last toast being removed and the rest remaining on screen.
A solution would be to wait for the next digest cycle with something like this:
setTimeout(() => {
  this.state.toasts[operationName]((found, toast, index) => {
    if (found || toast.toastId !== toastId) {
      return false;
    }
    this.setState(update(this.state, {
      toasts: { $splice: [[index, 1]] },
    }));
    return true;
  }, false);
}, 0);

If memory serves the setState function uses an internal setTimeout to execute updates which would mean this.state will be the same value for all calls in the loop.

@NullDivision Thank you for notifying this. A fix will be provided for this ASAP.

@NullDivision @RangarajKaushikSundar I think the correct pattern is:
this.setState((state) => {
 ...
 return update(state, {
    toasts: { $splice: [[index, 1]] },
  }));
});

This enqueues the state modification such that the result of the previous setState feeds in to this one. That was you can do a series of modifications using the last set state as input.

@NullDivision- Well actually, this bug is because the toastr currently assumes that the default behaviour of it is to prevent duplicates. We just need one small condition check whether it is a clear all call before the splice. So as you mentioned, it would not splice one by one, but remove it completely.

I agree with @plemarquand's solution. I don't see why you'd want to use the same state and rely on the index. I think a UID would solve a lot of problems for this case.

I agree! I am working on fixing these issues. If there are devs available to refactor code to be stable with latest versions of React, kindly comment here. Also, any suggestions on new features/ optimizations on existing functionalities are welcomed.


Setup error

This is how I call the action:
      uses: mamba-org/provision-with-micromamba@main
      with:
        channels: conda-forge
        environment-file: ""
        extra-specs: ""

and this is what i get from the runner:
Run mamba-org/provision-with-micromamba@main
  with:
    channels: conda-forge
    micromamba-version: latest
    channel-priority: strict
    cache-downloads: false
    cache-env: false
    log-level: warning
    installer-url: https://micro.mamba.pm/api/micromamba
    post-deinit: auto
  env:
    pythonLocation: /opt/hostedtoolcache/Python/3.10.6/x64
    PKG_CONFIG_PATH: /opt/hostedtoolcache/Python/3.10.6/x64/lib/pkgconfig
    Python_ROOT_DIR: /opt/hostedtoolcache/Python/3.10.6/x64
    Python2_ROOT_DIR: /opt/hostedtoolcache/Python/3.10.6/x64
    Python3_ROOT_DIR: /opt/hostedtoolcache/Python/3.10.6/x64
    LD_LIBRARY_PATH: /opt/hostedtoolcache/Python/3.10.6/x64/lib

Error: EISDIR: illegal operation on a directory, read
node:fs:723
  handleErrorFromBinding(ctx);
  ^

Error: EISDIR: illegal operation on a directory, read
    at Object.readSync (node:fs:723:3)
    at tryReadSync (node:fs:433:20)
    at Object.readFileSync (node:fs:479:19)
    at main (/home/runner/work/_actions/mamba-org/provision-with-micromamba/main/dist/main/index.js:64061:34)
    at run (/home/runner/work/_actions/mamba-org/provision-with-micromamba/main/dist/main/index.js:64093:11)
    at /home/runner/work/_actions/mamba-org/provision-with-micromamba/main/dist/main/index.js:64100:1
    at /home/runner/work/_actions/mamba-org/provision-with-micromamba/main/dist/main/index.js:64104:3
    at Object.<anonymous> (/home/runner/work/_actions/mamba-org/provision-with-micromamba/main/dist/main/index.js:64107:12)
    at Module._compile (node:internal/modules/cjs/loader:1101:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10) {
  errno: -21,
  syscall: 'read',
  code: 'EISDIR'
}

Is this a self hosted runner? Can you provide a link to your repo?

no, it wasn't self-hosted. and sorry, i already developed it away (was a re-usable one so developing was done by re-running the importer, hence no log file remained).
will try to reproduce and get back to you.

Any new input here?


RISC-V __mulsf3: handle exponent increase due to rounding

Originally reported by @Panda381. I thought this case was unreachable for multiply, but it's unfortunately just rare. This costs one cycle.

Ah, this needs to also check overflow after exponent adjustment. I'll add another testcase

...and do that without making underflow slower. Ready for review now


Pytorch Yolov5: Add synchronous() to measure time, and remove filtering class

Issue #, if available:
Description of changes:
Pytorch operations are asynchronous (referring to this), and it requires to do torch.cuda.synchronize() to correctly measure the total execution time.
Also we remove class filtering in postprocessing (NMS) by default.
TODO: It'd better to have app and metrics collection into separate class.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

LGTM


TypeLoader: Consider fast-pathing CustomAttributeData.AttributeType.

This issue has a dependency on this corefx issue:
https://github.com/dotnet/corefx/issues/31614
Once that's done and propagated (i.e. CustomAttributeData.AttributeType is virtual), just delete this line:
https://github.com/dotnet/corefxlab/blob/d9aede5d9500412ea8265b51d667a439422b09c6/src/System.Reflection.TypeLoader/src/System/Reflection/TypeLoading/General/NetStandardBridge.cs#L136
This will light up the optimization on NetCore builds.

Still holding hope I can get this pushed through this week.


Display Exception

As shown in the screenshot：

Looking forward to your reply。

I have the same problem.

Hello @stargrey, do you know when this issue might be fixed?

Sorry, I don't want to show off, but, since I didn't get a response, I created my own plugin. It is waiting for Obsidian team to review it. @be-studio feel free to beta-test it. https://github.com/mugiwara85/CodeblockCustomizer


gitstatus: trigger rebuild (avoid poisoned darwin entry)

Latest build on hydra has unexpected entries when trying to access grep:
 $ nix log /nix/store/blbb8bhxqhpaif73xqnw0n0msf3k5fhl-libgit2-1.8.4
 ...
 /<<NIX>>/cmake-3.30.5/nix-support/setup-hook: line 77: grep: command not found

The change adds a minotr change to the environment to force the rebuild.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 359133

x86_64-linux

:white_check_mark: 2 packages built:

gitstatus
zsh-powerlevel10k


Version 0.9.1 addressing minor issues on species identification and fasta files handling

Implemented better species identification error handling for cases when accession number
is not found in the assembly stats. This is especially important for custom
MASH sketches that might have accession numbers not found in assembly_summary.txt  stats file
Corrected issues #76 and #77. Improved behaviour in case of no --verify switch is
specified and MASH distance to RefSeq genomes fails
Added additional test cases for non-E.coli genomes especially for the accession numbers that became deprecated in RefSeq but are still present in current MASH sketch.
Implemented minimum requirement of 100/1000 shared hashes with the reference genome for successful species identification. Otherwise user would be warned to improve their MASH sketch or to look closer at the submitted sample.

Oh yes, I forgot to uncomment line 146 in test_ectyper_integration.py that was responsible to extract all accession numbers from the sketch. I had corrected that and now all checks pass. This is more of a test function for me to check how two sets of accessions (from mash sketch and refseq assembly stats meta data) intersect.

Thank you, Chad! I am glad to make a progress slowly but steadily  :-)


Added new implementation, based on pre-calculated Lucky number

Tried to solve FizzBuzz in shortest possibly byte size, found this way.
Think it's interesting one.

Wow. That's a really interesting one! It takes time to comprehend the solution.
Nice to have approaches like that in your toolkit.  It definitely expands the way we solve the problems.
Slava, thank you for contributing!


[elliptic] Take min instead of max across ranks for allNeumann flag.

allNeumann should be 1 only if it is 1 on all ranks.  If any rank sets
it to 0, then it should be 0.  That means we need a min operation, not a
max.
This closes #29.

Nice find!


[Distributed] NFC: Disable remoteCall tests due to random failures

/cc @ktoso

@swift-ci please smoke test


clientv3: document error handling in README

/cc @xiang90 Please review. Forgot to include this when updating godoc.
Thanks.

lgtm

thanks.


Release 1.2.6

Pull requests.

https://github.com/cookpad/LicenseToolsPlugin/pull/67
https://github.com/cookpad/LicenseToolsPlugin/pull/78
https://github.com/cookpad/LicenseToolsPlugin/pull/80

and bump up libraries version PRs by Renovate.

😢  https://github.com/cookpad/LicenseToolsPlugin/actions/runs/595185177
後で直す

released: https://plugins.gradle.org/plugin/com.cookpad.android.plugin.license-tools


History Plugin and Angular

Which @ngneat/elf-* package(s) are the source of the bug?
state-history
Is this a regression?
No
Description
I receive the following error after I installed the stateHistory plugin and included it into my code as follows:
export const store = new Store({ name: 'laneGraph', state, config });
export const todosStateHistory = stateHistory(store);

@Injectable({ providedIn: 'root' })
export class LaneGraphStore {
... operations on store ...
}

Please provide a link to a minimal reproduction of the bug
No response
Please provide the exception or error you saw
Error<EMAIL_ADDRESS>164:22
Module parse failed: Unexpected token (164:22)
File was processed with these loaders:
 *<EMAIL_ADDRESS> *<EMAIL_ADDRESS>You may need an additional loader to handle the result of these loaders.
|     }
|
>     this.subscription?.unsubscribe();
|   }
|

Please provide the environment you discovered this bug in
Angular 10

Anything else?
No response
Do you want to create a pull request?
No

Addition: Using in compiler options target: es2015 and 'module': es2020. Currently I'm trying to understand more of the bundler setup which seems to be the issue here

I can't reproduce it.

Thanks for trying though, I will create a new Angular test project this evening to see if I myself can reproduce it or if the setup of the real project is somehow broken (everything else works tho, except for this plugin).:dagger:

I think the problem is with the ?operator in this.subscription?.unsubscribe();.

For some reason, one of the webpack's loaders is not familiar with it.

Probably takes you to upgrade TS version since you mentioned Angular 10.

You should use target: es2020


Bug when using tooltips: { controls: false, seek: true }, (no tooltips on seek controls)

Expected behaviour
Tooltips should show on the seek icons only
Actual behaviour
No tooltips on any icons
Steps to reproduce
See here: https://codesandbox.io/s/plyr-bug-seek-control-go61y

initialize Plyr with this option (exactly as documented) tooltips: { controls: false, seek: true },
see that no tooltips show up on the seek controls, nor any other controls
when using tooltips: { controls: true }, then tooltips show up on all controls, but the desired behavior is to have tooltips on only the seek controls

Environment
All that I try - See code sandbox link above
Console errors (if any)
None
Link to where the bug is happening
See code sandbox link above

I'm struggling to replicate this. Using this:
tooltips: {
  controls: false,
  seek: true,
},

I get this:
https://user-images.githubusercontent.com/719092/102710459-8ed32880-4306-11eb-9d36-d3ea5ed97f4a.mp4

ohhh @sampotts I think there is a misunderstanding on my part.
My intention was to show tooltips:

on the seek bar
on the fast forward/rewind seek controls
nothing else


Fix typo in example

Nice that I finally get to check out this library :)
Not sure how I feel about the requirement that values are mutable, as update doesn't take a return value.
That wouldn't work for our current models at all, which are all marked @immutable.
I like the analogy to setState, though I think the use is a bit different here, as many children can gain access to the update method, not just the current widget itself.
I guess that could all be alleviated by forcing the usage of updateUser instead of user.update though. I am just not clear how well that would be enforceable in projects.
(Or worse, accidental mutations of user in this case, which would then only show up after something else called update.)

Thanks for your contribution. ;)
Actually, there is no requirement for mutable values. You can always just set .value = .... The Value.update() helper is specifically provided for mutable data because (in contrast to setting .value) it's more ergonomic, it doesn't require implementing a comparison operator and you can avoid the comparison (sometimes unnecessarily inefficient). For the immutable case the code doesn't become meaningfully more compact:
user.value = user.value.set('name', 'Adam');
// vs. replace() that is like update(), but taking return value
user.replace((u) => u.set('name', 'Adam');

Do you think this is still worthwhile to add?
BTW, after having worked with immutable and mutable state management solutions I have a slight preference for safe, controlled access to mutable data for best ergonomics, compatibility with third-party classes, and efficiency. While immutable looks nice at first and works well for simple cases, I think it doesn't play well with the ecosystem and hits a wall once the complexity goes up. Once you have the slightest level of nesting in your data structures you need something like a lens/cursor/reference for ergonomic updates. However, the immutable + cursor abstraction breaks once you integrate third-party packages that aren't designed with cursors or immutability in mind. Also, you might run into efficiency problems due to high GC/allocator pressure or conversion between the immutable and mutable world. I've had to fight these problems and I'd rather find a solution that tries to embrace the existing (mutable) ecosystem using a safe and simple mutation operation.
My current solution doesn't provide any helpers for working with deeply nested data structures and partial updates, though. At some point I'd like to extend Value for these use-cases.

Do you think this is still worthwhile to add?

Since you mentioned that value is still there and fine to be used, I don't think a replace helper is needed (as both ways of updating would have the same semantics anyway).
I just thought value ought not to be used as I didn't see it in the examples anywhere, though since it's always there I just read too much into that omission. (As the example just talks about what's new, not what's all the same).
Thanks for the reminder and clarification.


Help with Python game implementation: MCTS reaches state with no legal actions remain

Hi OpenSpiel team!
I'm working on understanding how OpenSpiel works, and I've been implementing the card game Battle Line (in Python) to learn about the game and state objects OpenSpiel uses.
I've gotten my implementation to a point that I believe the game logic is correct: I can sample random actions manually and rollout an entire game trajectory, but when I try testing my game implementation with the MCTS bot, I get an error from open_spiel/python/algorithms/mcts.py:69
<ipython-input-12-0f348d9ccb6f> in <module>()
     50         # Decision node: sample action for the single current player
     51         bot = bots[current_player]
---> 52         action = bot.step(state)
     53         action_str = state.action_to_string(current_player, action)
     54         print("Player {} sampled action: {}".format(current_player, action_str))

3 frames
/usr/local/lib/python3.7/dist-packages/open_spiel/python/algorithms/mcts.py in step(self, state)
    292
    293   def step(self, state):
--> 294     return self.step_with_policy(state)[1]
    295
    296   def _apply_tree_policy(self, root, state):

/usr/local/lib/python3.7/dist-packages/open_spiel/python/algorithms/mcts.py in step_with_policy(self, state)
    266     """Returns bot's policy and action at given state."""
    267     t1 = time.time()
--> 268     root = self.mcts_search(state)
    269
    270     best = root.best_child()

/usr/local/lib/python3.7/dist-packages/open_spiel/python/algorithms/mcts.py in mcts_search(self, state)
    409         solved = self.solve
    410       else:
--> 411         returns = self.evaluator.evaluate(working_state)
    412         solved = False
    413

/usr/local/lib/python3.7/dist-packages/open_spiel/python/algorithms/mcts.py in evaluate(self, state)
     67           action = self._random_state.choice(action_list, p=prob_list)
     68         else:
---> 69           action = self._random_state.choice(working_state.legal_actions())
     70         working_state.apply_action(action)
     71       returns = np.array(working_state.returns())

mtrand.pyx in numpy.random.mtrand.RandomState.choice()

ValueError: 'a' cannot be empty unless no samples are taken

It seems that when doing a lookahead rollout, the MCTS bot reaches a point where it tries to sample actions, but there are no legal actions to be found - which should mean the game has terminated, but apparently, it hasn't. I'm guessing I've missed something small in my implementation of the Python state interface, but I can't see my mistake.
My code is in a collab notebook - I would greatly appreciate if anyone more familiar with the project had time to take a peek and let me know if you can see the problem with my game implementation.
If I can get this working, I'd be happy to contribute this implementation as an unofficial demo of the python API.
Thank you,

Hi @aaronsnoswell,
The best way to debug this is to insert an if statement on the offending line that checks if the legal_actions are empty. When it is, print out the state.to_string() and state.history().
Then, separately, construct a new program that plays through that history, printing out ever state and every action (you can use state.action_to_string to get the action's string representation).
Then manually inspect what's happening and whether it makes sense. This should find where the logic went wrong or a case that is not being covered.

Actually, this is such a common case for debugging that I'll add a helper that does this (one that can take the history string and game string and just play it out (but will only be in the next few weeks). There are a few options that would be good too, like the ability to make it an interactive session walk through the game.

Hi @aaronsnoswell, just touching base -- were you able to find the problem?

Hi @lanctot! Thank you for following up! I have been busy and haven't had a chance to review this yet :D Please feel free to close and I'll report back if there is any actual issue here (other than my own incompetence :D)


deprecate chargepoint-ID in CSMS-URL config-field

So far it was required to include the chargepoint_id in the configured CentralSystemURI (v16) or ocppCsmsUrl (v201, as part of the NetworkConnectionProfiles variable). This is indeed redundant so I like this initiative!
For v16 we can append the internal configuration key ChargePointId (like implemented in this PR) and for v201 we need to use the variable "Identity" from the "SecurityCtrlr", so this line is currently not correct:
https://github.com/EVerest/libocpp/blob/main/lib/ocpp/v201/charge_point.cpp#L585
Please add this to the PR.

Next I'll use a Uri::set_path to allow backwards-compatibility with current URLs which contain the CP-ID as path.

@Pietfried It's ready now. I'll do the returning an error/std::expected in a follow-up PR.


Error page

A new inspection was created.

The inspection completed: No new issues

And I correct some mistakes in docs.

Current coverage is 72.27% (diff: 35.71%)

Merging #162 into develop will decrease coverage by 1.56%

@@            develop       #162   diff @@
==========================================
  Files            28         29     +1
  Lines           990       1017    +27
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            731        735     +4
- Misses          259        282    +23
  Partials          0          0

Powered by Codecov. Last update 604bf56...be4d0c6

A new inspection was created.

The inspection completed: No new issues

A new inspection was created.

The inspection completed: No new issues

A new inspection was created.

The inspection completed: No new issues

A new inspection was created.

The inspection completed: No new issues

23:44 ≡ python2
pyenv: python2: command not found

The `python2' command exists in these Python versions:
  2.7.10
  2.7.11

23:44 ≡ pyenv local 2.7.11
23:44 ≡ python2
Python 2.7.11 (default, Apr  3 2016, 20:54:08)
[GCC 4.2.1 Compatible Apple LLVM 7.3.0 (clang-703.0.29)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import html
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: No module named html

A new inspection was created.

The inspection completed: No new issues

A new inspection was created.

The inspection completed: No new issues

A new inspection was created.

The inspection completed: No new issues

A new inspection was created.

The inspection completed: No new issues

A new inspection was created.

The inspection completed: No new issues

A new inspection was created.

The inspection completed: No new issues

A new inspection was created.

The inspection completed: No new issues


Local pull request success status testing

Docs Build status updates of commit 7dcebd9:
:clock10: Pending: waiting for processors (15 builds ahead of you)
:warning: Docs Build is busy, currently there are 15 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 7dcebd9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

E2E_DocsBranch_Dynamic/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Add integration with itazuraneko

As the title says it, add support for the novel section of itzuranekko like for the ttsu app.
Right now when you try selecting all the text to parse, you either get: text too long, Error: Cannot read properties of undefined (reading 'parentNode'), or I remember getting something about reaching end of document.

These bugs with the right click selection parsing should be fixed now, hopefully :+1:
Currently I don't think I'll be adding integration for Itazuraneko. For the full reasoning, you can see the pull request from mikzorz, but TL;DR Integrations should only be used for websites that actually need it (IMO), rather than for avoiding bugs, and I also don't want to add integrations for piracy websites.


Add key path related guide & code to the documentation.

Friendly ping :)

Looks great, thanks Ivy. Sorry for the review delay. But great work as always!

No worries Matt, thank you so much for reviewing!


Add support for running test

We shouldn't be regularly running against our test set, but eventually will need to add support for running test with a trained model.

distinct container


You can develop IITCm in Android Studio

I imported a classical project which uses ant + eclipse and changed
something to build in Android Studio which is a latest stable one.
CHANGES
In mobile-as/app/build.gradle, I deleted ‘compile
'com.android.support:appcompat-v4:19.0.3’ ‘ and added
‘ compile 'com.android.support:appcompat-v7:25.1.0' ‘ and
‘compile 'com.android.support:support-v13:25.1.0' ’
And I added the next row in the file.
‘useLibrary 'org.apache.http.legacy' ’
After Marshmallow, Apache.HttpClient was deprecated.
We have better to improve around these implements.

Thanks for the pull request. However, this change adds over 51,000 lines of code. There are a lot of files which seem like they shouldn't be part of the repository at all, such as lock files, the built version of IITC, as well as various binaries. It also adds a complete copy of the IITC mobile codebase, which would be hard to keep in sync.
It seems like it could be reasonably straightforward to add Android Studio support to the existing Android codebase without needing to duplicate anything. I'd also be fine with moving to Android Studio and Gradle from the current Ant solution.


[BUG] Case when types incorrect in certain cases

What happened:
As seen in https://github.com/dask-contrib/dask-sql/blob/main/tests/integration/test_rex.py#L24 the optimizer adds a projection step to reduce compute when multiple case-when statements have common conditionals. It seems like the type inferencing during that projection step might be incorrect either on the dask-sql planner side or datafusion side.
What you expected to happen:
No error
Minimal Complete Verifiable Example:
SELECT
   CASE WHEN (a=3 and b=4) then 1 ELSE 0 end as col1,
   CASE WHEN (a=3 and b=5) then 2 ELSE 0 end as col2
from table1

Anything else we need to know?:
Environment:

dask-sql version: 2022.10.1
Python version: 3.9
Operating System:
Install method (conda, pip, source): source

On initial inspection it looks like the latest release with type coercion fixes resolves this issue. Will still keep open until the pytest is not skipped


Split vite config for Server and Client

Describe the problem
I make a site with Golang server and frontend is Svelte Kit complied. After research, problem with compile script to ES5.
Vite navtive support @vitejs/plugin-legacy for legacy browser
But, I don't want to use it on the server
Describe the proposed solution
My soulution is split vite config with server and client
It here
	/** @type {import('vite').UserConfig} */
	const vite_config = config.kit.vite();

Alternatives considered
No response
Importance
would make my life easier
Additional Information
No response

I think the better solution would be to have a plugin which does not compile the server code to legacy JS, only the Frontend. Not sure how that would look like.

It can is a options for Kit. eg legacy
With legacy is Object, builder will include plugin @vitejs/plugin-legacy with option is value of legacy
In source of client config (https://github.com/sveltejs/kit/blob/master/packages/kit/src/core/build/build_client.js#L88)
It maybe is:
import legacy from '@vitejs/plugin-legacy';

// other config

plugins: [
	svelte({
		extensions: config.extensions,
		emitCss: !config.kit.amp,
		compilerOptions: {
			hydratable: !!config.kit.hydrate
		}
	}),
        config.kit.legacy &&  legacy(config.kit.legacy)
]

// ...

Is there a legacy output for the server build when used today? I just checked @vitejs/plugin-legacy and it is all guarded with if (config.build.ssr) (checks whether is an SSR build), which means it skips SSR build entirely. If the issue indeeds persists, it should be fix upstream instead. Vite plugins are meant to be interoperable in different builds.


Rename floxpkgs catalog namespace to "flox"

Breaks existing flox (through nix) installations' upgrade url
Requires Docs to use the new name

This is not applicable any longer


Fix ESM issues

Coverage decreased (-0.8%) to 81.301% when pulling 6304ddcbfdc6373bec426484c116463069c3772a on fix-esm-issues into 9602a3ef9fb46d49fda0c583247f968f6f75455c on develop.


Pub should return semantic error codes

Currently, Pub returns an error code of 0 even when it fails to install or update. It should return 1 in those cases.

This issue has been moved to dart-lang/pub#88.


TBI - Secure storage is not being read when creating a service provider instance for an on-premise ABAP system

Description (include screenshots)
Extend the createAbapServiceProvider method to support both credentials and service keys if found in the system secure store.
Existing functionality;
https://github.com/SAP/open-ux-tools/blob/main/packages/deploy-tooling/src/base/deploy.ts#L106
Value
Provides a secure way to deploy to ABAP using service keys.
Provides an alternative to the user to deploy to an on-premise ABAP system.
Architecture Elaboration
No
Notes
Found when testing on-premise ABAP deplyment where the secure store had the service keys for the target URL.
Tasks

[ ] Update createAbapServiceProvider to determine if the secure store also has service keys, if so, then generate a provider sinstance using these keys
[ ] Update test cases

I don't think we should do that because createAbapServiceProvider is only called if the configuration defines that it is not a cloud system. If it is a cloud system then we execute createAbapCloudServiceProvider (which is also checking the store for credentials).
What we could and should do instead, is to check in both functions if the returned system is of the incorrect type, and if yes, throw an exception informing the user that either the deployment configuration or the stored system information are wrong. We could even recommend how to fix it.

If I am not mistaken, then this one is also done, correct @cianmSAP @longieirl ?

Yes, this is complete. Closing


Fix testing errors

Tests have been failing for the last couple of days. The most suspicious changes are pyparsing 3.0.4 => 2.4.6 and coverage 5.5 => 6.1.1, and since the error is coming out of MDTraj, I'm starting with pyparsing as the suspect.

Full diff comparing "Versions" section of CI
8c8
< argon2-cffi               21.1.0           py37h5e8e339_1    conda-forge
---
> argon2-cffi               21.1.0           py37h5e8e339_2    conda-forge
18c18
< brotlipy                  0.7.0           py37h5e8e339_1002    conda-forge
---
> brotlipy                  0.7.0           py37h5e8e339_1003    conda-forge
25c25
< cffi                      1.14.6           py37h036bc23_2    conda-forge
---
> cffi                      1.15.0           py37h036bc23_0    conda-forge
31,33c31,33
< coverage                  5.5              py37h27cfd23_2
< coveralls                 3.2.0              pyhd8ed1ab_0    conda-forge
< cryptography              35.0.0           py37hf1a17b8_1    conda-forge
---
> coverage                  6.1.1            py37h5e8e339_1    conda-forge
> coveralls                 3.3.0              pyhd8ed1ab_0    conda-forge
> cryptography              35.0.0           py37hf1a17b8_2    conda-forge
38c38
< debugpy                   1.4.1            py37hcd2ae1e_1    conda-forge
---
> debugpy                   1.5.1            py37hcd2ae1e_0    conda-forge
111c111
< libpq                     13.3                 hd57d9b9_2    conda-forge
---
> libpq                     13.3                 hd57d9b9_3    conda-forge
163c163
< packaging                 21.0               pyhd8ed1ab_0    conda-forge
---
> packaging                 21.2               pyhd8ed1ab_1    conda-forge
180,181c180,181
< prompt-toolkit            3.0.21             pyha770c72_0    conda-forge
< prompt_toolkit            3.0.21               hd8ed1ab_0    conda-forge
---
> prompt-toolkit            3.0.22             pyha770c72_0    conda-forge
> prompt_toolkit            3.0.22               hd8ed1ab_0    conda-forge
193c193
< pyparsing                 3.0.4              pyhd8ed1ab_0    conda-forge
---
> pyparsing                 2.4.7              pyhd8ed1ab_1    conda-forge
199c199
< pyrsistent                0.17.3           py37h5e8e339_3    conda-forge
---
> pyrsistent                0.18.0           py37h5e8e339_0    conda-forge
218c218
< setuptools                58.4.0           py37h89c1867_1    conda-forge
---
> setuptools                58.5.2           py37h89c1867_0    conda-forge
224c224
< terminado                 0.12.1           py37h89c1867_0    conda-forge
---
> terminado                 0.12.1           py37h89c1867_1    conda-forge
228a229
> tomli                     1.2.2              pyhd8ed1ab_0    conda-forge

So, the problem is actually with coverage, not pyparsing. This doesn't cause a problem in the OPS core library because we don't count coverage of the experimental subpackage.
I haven't had time to do a deep dive here and figure out exactly what's going wrong, and I haven't been able to reproduce it locally yet. It would be nice to figure this out and get it fixed upstream, but for now we may just need to hope that coverage fixes it by the time we need coverage 6+.
This is ready for review and comment. I will leave it open for at least 24 hours, merging no earlier than Mon 08 Nov 16:00 GMT (11:00 my local).

LGTM, do you mind opening an issue reminding us to try unpinning somewhere in the future?

Done (#14). Also added similar #13 to remind us to undo extra CI steps added in #10.


Crash error in 18.2

Hello since the 18.2 I can't start my server anymore

I can trigger this with 18.2 but not with 18.1. It fails when you call uWS.SSLApp(). Don't know why this function wouldn't be defined anymore - will check

Fixed with v18.3.0. Thanks for reporting


Add jabref installer

I added the bibtex manager JabRef, which publishes .deb files at https://builds.jabref.org/main/

I would very much like to be able use deb-get to fetch the latest build of JabRef from builds.jabref.org/main .  As far as I can see, this is what this installer did, originally, but in April last year it was switched to fetch the public stable release from Github.  Could and installer with the original behaviour be added, perhaps with a different name (jabref-test perhaps?).

The change was made to make the stable version available from deb-get.
That is the same version that a user would download from the upstream website (https://www.jabref.org/#download).
This is the better default, keeping in mind that it can be overrided (as @philclifford mentioned).


Adds details about new non-blocking logging mode

This functionality was added in docker/docker#28762.
It allows a user to specify what to do in cases when a log message can't
be sent, block the container's stdio or drop messages.
Also removes an erroneous entry about a nats log driver, which does
not exist.

ping @thaJeztah

LGTM pending tech review. Where in the world did the nats driver content come from??

Not sure but it looks like it got cherry-picked in somehow in #1187

The NATS stuff was added in #369. Should I just revert that change? It is my fault because I merged it without realizing that the feature was rejected.

I can't revert #369 cleanly so I'll just merge this one, which effectively does the same thing.


useParseQuery subscription

I'd like to request a feature / bug useParseQuery works good in mount/unmout scnearios, but with react-navigation they dont have this lifecycles so it is impossible to use useParseQuery  because it will keep requesting / updating other Screens, and I think with this will bring other level of customization, ex: click on somthing and being able to stop requesting live.
It could actually be a solution to use the enableLiveQuery to start/stop at any time
Is it possible to change enableLiveQuery on options and will it start / stop
The main objective is something like this
import { useIsFocused } from '@react-navigation/native';
import React,{ useEffect} from 'react';
// ...

function App() {
  const isFocused = useIsFocused();
  const {results,liveQuerySubscription} = useParseQuery(query);

useEffect(()=>{
if(!isFocused ){
// stop requesting
 liveQuerySubscription.unsubscribe();
}

},[isFocused ])

  return <Text>{isFocused ? 'focused' : 'unfocused'}</Text>;
}

But exporting the liveQuerySubscription maybe would not be the greatest solution because when we return to the same srceen we, mount event will not be trigered and we want to start requesting again...
What you guys think would be the best solution?

should this work to stop / start live requesting on demand?
  const isFocused = useIsFocused();
  const {results} = useParseQuery(query,{enableLiveQuery  :isFocused });

When I navigate between two screens it gives me this error on the console. I'm using useParseQuery in both but for different requests.
https://user-images.githubusercontent.com/96492663/146973458-35fb7279-129f-4e43-b1d6-3674e0155c87.MOV

@claudiarsl Unfortunately, I have exactly the same problem. Were you able to solve the error somehow?


Fix #1017

Codecov Report

Merging #1031 into master will decrease coverage by 0.2%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1031      +/-   ##
==========================================
- Coverage   94.97%   94.77%   -0.21%
==========================================
  Files          39       39
  Lines        3485     3673     +188
  Branches      303      545     +242
==========================================
+ Hits         3310     3481     +171
- Misses        175      180       +5
- Partials        0       12      +12

Impacted Files
Coverage Δ

browser/pages/install/controller.js
71.3% <0%> (-2.91%)
:arrow_down:

browser/pages/location/controller.js
92.3% <0%> (-2.86%)
:arrow_down:

browser/model/helpers/downloader.js
94.7% <0%> (-1.85%)
:arrow_down:

browser/services/credentialManager.js
93.02% <0%> (-1.72%)
:arrow_down:

browser/main.js
95.58% <0%> (-1.39%)
:arrow_down:

browser/model/devstudio.js
96.8% <0%> (-0.97%)
:arrow_down:

browser/pages/selection/controller.js
98.52% <0%> (-0.47%)
:arrow_down:

browser/model/jdk-install.js
96.61% <0%> (-0.42%)
:arrow_down:

browser/services/data.js
97.43% <0%> (-0.23%)
:arrow_down:

browser/services/openpgp.js
100% <0%> (ø)
:arrow_up:

... and 25 more

Continue to review full report at Codecov.

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


Tests react components

PR in progress
Added and updated unit tests for components:

InputField2
CrowdsaleStartTime
CrowdsaleEndTime
Supply
TokenSupply
MinCap
processReservedTokens
processWhitelist
CrowdsaleRate
DisplayField
Error
DisplayTextArea
Header
Footer
Loader
CrowdsaleList

Black Duck Security Report
Branch 2.0 has no Black Duck results, and could not be compared to #1078. Analyze branch 2.0 to get a change comparison.
Removed Components
Medium Risk: 14
Clean:       1488
Click here to see full report

@dennis00010011b please resolve merging conflicts

Pull Request Test Coverage Report for Build 2856

0 of 0   changed or added relevant lines in 0 files are covered.
158 unchanged lines in 5 files lost coverage.
Overall coverage increased (+2.0%) to 20.414%

Files with Coverage Reduction
New Missed Lines
%

src/components/Common/StepNavigation.js
1
66.67%

src/components/Common/CrowdsalesList.js
2
72.73%

src/components/crowdsale/index.js
15
1.26%

src/stores/utils.js
31
2.82%

src/components/crowdsale/utils.js
109
3.17%

Totals

Change from base Build 2854:
2.0%

Covered Lines:
828

Relevant Lines:
3553

💛  - Coveralls


panic: descriptors reported by collector have inconsistent label names or help strings for the same fully-qualified name

Cannot start postgres_exporter
./postgres_exporter --version
postgres_exporter v0.4.5-1-g36e13ca (built with go1.10.1)
OS: CentOS 7.4, Postgres 10.3
Please advise. The full error is below. Yesterday, i was complaining about pg_settings_checkpoint_timeout_seconds
# tail -20f postgres_exporter-10.log
time="2018-04-10T14:35:55-04:00" level=info msg="Established new database connection." source="postgres_exporter.go:995"
time="2018-04-10T14:35:55-04:00" level=info msg="Semantic Version Changed: 0.0.0 -> 10.3.0" source="postgres_exporter.go:925"
panic: descriptors reported by collector have inconsistent label names or help strings for the same fully-qualified name, offender is Desc{fqName: "pg_settings_shared_buffers_bytes", help: "Size of shared_buffers", constLabels: {}, variableLabels: []}

goroutine 1 [running]:
github.com/wrouesnel/postgres_exporter/vendor/github.com/prometheus/client_golang/prometheus.(*Registry).MustRegister(0xc4200b8380, 0xc4201eea40, 0x1, 0x1)
        /home/ipolishchuk/go/src/github.com/wrouesnel/postgres_exporter/vendor/github.com/prometheus/client_golang/prometheus/registry.go:354 +0x9e
github.com/wrouesnel/postgres_exporter/vendor/github.com/prometheus/client_golang/prometheus.MustRegister(0xc4201eea40, 0x1, 0x1)
        /home/ipolishchuk/go/src/github.com/wrouesnel/postgres_exporter/vendor/github.com/prometheus/client_golang/prometheus/registry.go:153 +0x53
main.main()
        /home/ipolishchuk/go/src/github.com/wrouesnel/postgres_exporter/cmd/postgres_exporter/postgres_exporter.go:1129 +0x298```

Yesterday, on the same system, it complained about a different metric - pg_settings_checkpoint_segments.
I've installed postgres_exporter on another identical system, it complains about it too:
panic: descriptors reported by collector have inconsistent label names or help strings for the same fully-qualified name, offender is Desc{fqName: "pg_settings_checkpoint_segments", help: "Number of checkpoint segments", constLabels: {}, variableLabels: []}


Make dockerUtils use container name for test exec

Provides easier to follow logs also we reduce the log output via custom log config for the integration tests.

We are overriding  that , we are reducing the number of log line elements since we are not looking at multithreaded server components but single threaded test instances, also we are making the log files easier to follow bu using the containername

we are reducing the number of log line elements since we are not looking at multithreaded server components but single threaded test instances

well, I think it is debatable. more information is generally helping with debugging things. is there any particular reasons that you have to reduces the log line elements?

also we are making the log files easier to follow bu using the containername

how does the log configure help that?

the log configure does not, the DockerUtils changes fixes that issue.

can you please comment inline and address the comment individually? because my comments are for different places. especially for the log configuration file, I don't see it is really needed.


Properties not generating Sketch V 53.1

Unable to generate all 4 sides of properties and also not able to select the generated properties.

@mohanvamsi219 Please give this a try:
536


Return data type options for query

Currently queries return a SQLalchemy object.
We could add an option to convert this to a dict, or a pandas dataframe to make the output more familiar to work with.

Addressed in #66


Looking up annotation's destination

Problem description
Given an InternalLink annotation with some id such as 2R how do you most efficiently look up what page this annotation is pointing at?
My solution
Given an id and a page, the following code does the lookup:
pdfDocument.getPage(page).then((p) => {
    p.getAnnotations((annots)=> {
// Just iterate the annotation and find the one with the same ID as the ID provided
        let a = utils.find(annots, id, (a) => a.id);
        if(a) {
            let dst = a.dest; // Some long ID like 'YSCDB916BIB1'
            pdfDocument.getDestination(dst).then((lookingFor) => {
                // By now we've got {num,gen} object and need to
                // iterate through all the pages and access the .pageInfo property
                // to find which one is equal to lookingFor
            });
        }
    });
});

There must be a better way of doing this, I can't find the equivalent code in the example viewer.

pdfDocument.getDestination(dst).then((lookingFor) => {
  // By now we've got {num,gen} object and need to
  // iterate through all the pages and access the .pageInfo property
  // to find which one is equal to lookingFor

There's no need to iterate anything here, since assuming that you've managed to obtain a Reference (i.e. an object with num and gen properties) you can simply use the getPageIndex method; please refer to https://github.com/mozilla/pdf.js/blob/c2cbeaa34d81bbb7cced856e2888867df587a1fa/src/display/api.js#L612-L620

@jsjolen Is the above enough to answer this question or is there more you want to know here?

@timvandermeij  I am satisfied with this answer! Thank you.
@Snuffleupagus Thank you for the link!


Database ingestion and functional API

This PR handles both database ingestion and the creation of the related API. A few points to keep in mind:

I have provided a populateAll resolver as a convenience to seed all tables. However, seeding pki_events can take a long time, so in order to play with this locally more quickly, you can individually run all of the resolvers defined in src/resolvers/db/index.js except populateAll and populateDailyCron. Then search for the comment // Can use the following line for testing when needed: in src/resolvers/db/utils/addToDB.js. Right below this you can edit the number of rows you want to add to pki_events. I find a few hundred to be sufficient to demonstrate the full range of functionality. There will be a console log every 200th iteration so you can keep track of where things are without blowing up your terminal.
The correct contents of the eventTypeKey object in src/resolvers/db/utils/addToDB.js is still being sorted out. I emailed Mark about this today and copied you (Josh) on it.
cron functionality exists and works but is commented out for now so ESLint doesn't yell at me.
Additional remaining work, such as Jest tests, is tracked in this Trello board. I emailed you (Josh) an invite to edit. Let me know if you do not receive that. I try to keep it up to date but I make no assertions that this board reflects all additional work remaining with project as there might be things I'm missing.

Let me know if you have any questions. Looking forward to review.

i'm excited about this

Please note that I am making additional commits addressing PR feedback just to make it easier to keep track as review is in progress, but happy to squash those when we're ready to merge.


docs(faq): added a link to unofficial Chinese documentation

Thanks again to the Chinese DockerOne community for this great work! See #2852 for the details.
I used Google translate to spot-check several sections, and the content seems good. It would be nice to include multiple translations at http://docs.deis.io/ directly, but for now providing a link is better than nothing.
Closes #2852.

LGTM!

docs LGTM

Full version: http://deis.heheapp.com/


Fix for brunch/brunch#1549.

Find plugins with names that do not match common convention.
For example: babel-root-import

Related:

https://github.com/brunch/brunch/issues/1549
https://github.com/michaelzoidl/babel-root-import/issues/63

LGTM, very clean & appreciate the comment.
However, Travis is failing.

@shvaikalesh the cause of failed build is not correlated with this PR. The problem in single-var notation:
Expected:
var a = c.a;
var b = c.b;

But got actual:
"use strict";

var c = {};
var a = c.a,
    b = c.b;

I can open another PR with corresponding fix, if you don't mind.

@denysdovhan Thanks for digging it up.

PR with corresponding fix

Please do.

Tests are fixed in #46. Build is greeen.


Use correct casing in HTTP headers

HTTP headers are case-insensitive, per RFC 7230 Section 3.2:

Each header field consists of a case-insensitive field name followed by a colon (":"), optional leading whitespace, the field value, and optional trailing whitespace.

However, we should be consistent when we refer to the headers, and use X-WOPI- in all caps as the prefix, and camel-casing for the rest of the header name, e.g. X-WOPI-OldLock.
Unfortunately, due to the behavior of sphinxcontrib-httpdomain in version 1.3.0, the header values are automatically title-cased in the REST parts of the docs. This issue is actually already fixed in the next version of sphinxcontrib-httpdomain, and I have asked the developer to publish an updated version since 1.3.0 is quite old at this point. Until that's done we'll have inconsistencies.

I went ahead and published a new version of the lib: sphinxcontrib-httpdomain-update.


Presentation & presenter notes to PDF not rendered correctly

Is there an elegant way to print the presentation and presenter notes to a PDF? I know that you have a way with printing the presentation using Chrome or Safari. Thanks.

I had the same problem, too. I found that as a quick fix, it helps to add a custom style sheet to your presentation containing:
@media print {
 .remark-container.remark-presenter-mode .remark-slide-notes {
  margin-top:700px;
 }
}

I imagine that the exact number of pixels (700 roughly works for me) may be different for various paper sizes. You may have to try different values.

It would be worth looking at the print CSS stylesheet to get the slides to print out to PDF.

this is now covered on the front page.

Closing this. @jknyght9  please re-open if Decktape doesn't fix this for you.


After downloading from wp.org - activating plugin: Fatal error: Class 'Timber\Timber' in /wp-content/plugins/timber-library/init.php on line 5

Expected behavior

Downloading Timber from http://downloads.wordpress.org/plugin/timber-library.zip I expect to get the latest release
When clicking the "Activate" link for Timber in WP admin I expect the plugin to be activated.

Actual behavior
A PHP error is thrown:
Fatal error: Class 'Timber\Timber' in /wp-content/plugins/timber-library/init.php on line 5

It seems the version you get when you download "timber-library.zip" to get the latest version from downloads.wordpress.org does not work.
Steps to reproduce behavior

wget http://downloads.wordpress.org/plugin/timber-library.zip
unzip and see Timber listed in admin as 1.0.3
Try to activate and get error

Steps to fix

wget https://downloads.wordpress.org/plugin/timber-library.1.0.3.zip
unzip and see Timber listed in admin as 1.0.3
Activation works

If I list the directory I can see the zips downloaded are not identical (although both plugins identify as version 1.0.3 in WP admin)
-rw-r--r--   1 bjorn  staff   729K May 14 19:12 timber-library.1.0.3.zip
-rw-r--r--   1 bjorn  staff   787K May 14 19:11 timber-library.zip

What version of WordPress, PHP and Timber are you using?
Timber 1.0.3
WordPress 4.5.2
PHP 5.6.20
How did you install Timber? (for example, from GitHub, Composer/Packagist, WP.org?)
wget http://downloads.wordpress.org/plugin/timber-library.zip

One for you @jarednova ?

Yeah I'll take a deep-dive

Hi, just some additional info .. when pushing this site to the staging server just now I got the same "Class not found Timber\Timber in init.php" error.
I fixed it by adding this to the top in init.php:
if ( ! class_exists( 'Timber' ) ) {
    require_once( 'lib/timber.php' );
}

Not sure if this is related to the original issue as I got that error also on my local install but thought I'd add some more context. Might it have something to do with my directory structure? I have WP installed in /vendor/wordpress while the plugins are located in /webroot/wp-content/plugins

@bjornbjorn: thanks so much for this info. I was able to recreate the problem and encounter your issue.
It looks like there was an SVN problem with files that were checked into trunk (but for some reason not downloading). I nuked everything and started fresh. After everything was (re)checked into SVN; I ran through the steps and it works!

Did a fresh install of latest WP/timber-library.zip and can confirm it works now. Thanks a lot for the fix! :-)


[cosmetic] remove border from table viz

before:

after:
/
@williaster @graceguo-supercat what do you think ?

a little hard to tell how it would hold up with multiple columns but overall I think it looks good 👍


Forces are not correct

Parsed forces are not correct when the multiplicity of the atom is not 1. This bug was present in the original code I wrote and was copied during he rewrite. The problem is that when one expands the forces with the atomic multiplicities, one needs to apply proper rotations (the atoms are equivalent but the forces in cartesian coordinates are not in the same direction). This is however non-trivial as it seems the needed rotation matrices are not printed anywhere.
This expanding could be done by the Arrows utility from the Wien2k package, however it is very heavy hammer. It does full symmetry detection and uses that info to properly transform the forces, we probably don't want to have similar code in the parser?

This will be a problem also for the atomic positions of the equivalent atoms. The parse now gets it from the *.struct file and that is OK for the single run calculations, however if support for geometry optimization is added as some point, the positions along the optimization trajectory are printed only for nonequivalent atoms similarly to the forces, so it will need to be expanded as well.


Add limitations for mobile apps and solution flows

Add known limitation that the Power Apps mobile app and Power Automate mobile app do not currently list flows created in a solution.

Can you give me an approximate timeline for when this fix will be officially posted in the public information?

@KumarVivek Are any additional checks/reviews needed for this to be merged?


Compatibility with pg database version

Hi，I want to know the compatible pg database version？

Hi, all versions since 8.0 I think but sure since 9.0.


Will Java always be the primary language of ghidra?

Compared to other tools such as IDA ghidra feels quiet sluggish even on an i9 machine. I can only assume this is because it has roots is in Java - but maybe its just an algorithm issue?
Anyway I'm not trying to start a flame/language wars here but is Java always going to be the main/go forward language?
If there was enough community backing to port to say C++/Qt for instance would it be considered and would it be accepted?
Also I'd be interested to know why this was the original language of choice. I'd assume its just because its what the NSA internal devs where most familiar with at the time?

I think that the subsequent modules may be developed independently, and perhaps the core modules will gradually turn to other languages, such as C++.

There are many reasons Ghidra might appear sluggish; the visible plugins that are displaying and updating information, background analysis on a program (running 6 decompilers in the background), the complexity of analysis being performed (fixing damage from non-returning functions), or the size of the program.  Even with very large programs, I hope you will find Ghidra performs essentially the same.
One startup cost when first loading a program is the automated re-compilation of language modules from changes.  Sometimes when unzipping the files, the date time stamp on the file will be reset.  This causes the processor module for the binary being opened to be re-compiled to a .sla file.  This should only happen once for that processor and subsequent uses of the processor will use the latest compiled .sla file.  We've tried to limit this re-compile by providing pre-compiled .sla files, but occasionally the time stamps are reset or timestamps that are too close are put on the files at build time.
Another potential issue on the i9 is the use of background threads that JAVA uses for garbage collection.  On a processor with many available cores/threads we've seen some issues with GC thrashing.  If this is the case, there is a setting to restrict the GC threads that might not be working in your case.
Occasionally plugins display information that must be constantly updated due to changes in the background or are costly to compute need to be re-designed, for example the data type archives have historically had performance problems that have been addressed when processing large programs.  An attempt is made to maintain interactivity and reduce unnecessary updates with good event re-action management.  The code is profiled from time to time when sluggishness is noted to identify the culprit.
If you note some sluggishness it may be there is computation in the background going on to display the data.  For example, the code browser is doing an amazing number of controlled complex tasks to provide maximum configurabilty, maintain interactivity during analysis, all while smoothly scrolling with many types of plug-able fields displaying unknown information.  These costs would be born in any GUI language chosen.
There are many advantages and some sacrifices to using JAVA.  A few advantages come to mind that I depend on:
Ghidra rarely loses work due to bugs in both Ghidra base, contributions, and scripts, for example out of memory errors.  Previously libraries of C/C++ code using JNI were used, but it was found that bugs in code caused crashes and loss of data at the worst time.  Using C/C++ directly has been discouraged.
Scripts (and Ghidra) are fully interactively debuggable in modern development environments with the ability to push changes to code on the fly while debugging without re-setting up the issue under inspection.
Originally the portability of the GUI was an issue (Windows, Linux, Mac), however Qt has gotten better on that front.  Also at the time of choice (not trying to get into a comparison of other options) JAVA was the only language that could provide the on the fly extendability necessary for Ghidra to do what it does.
Normally Ghidra runs with only 1Gig of available memory to Ghidra and I've found I rarely need to bump it up unless there is a greedy algorithm that needs to be run.  The memory in JAVA is normally mostly just empty overhead to allow processing to be quick without excessive garbage collection.  I have 125M in use with several programs up, Java is using 450M with about 300M free.
Ghidra provides a tremendous amount of capability using JAVA.  The advantages and disadvantages of moving to another architecture would need to be very carefully weighed, to include first a re-design of sluggish algorithms, in favor of another option to merit the time and potential loss of capability, vice just enjoying using the tool ;-)

As I am using an i9 system any chance you could provide more details on how to prevent the GC thrashing? (Btw that wouldn't be an issue with C++ ;)).
Also I think most users probably come from IDA or the likes which just feels a lot more snappier in general. Not saying it can't be fixed without not using Java but it is an observation worth noting.
IDA has a freeware version if you'd like to compare using some samples.

You can control the number of GC threads with the -XX:ParallelGCThreads= JVM argument.  If you look at support/analyzeHeadless, you will see we restrict it to 2 because it may be a common use-case to start up several instances of the headless analyzer concurrently, and if we didn't restrict the number of GC threads, each process would create a GC thread for every available core on the system.  For a system with a lot of cores, this can cause system-wide performance problems.
If you are launching the Ghidra GUI with the ghidraRun script, the number of GC threads is not explicitly defined, so by default it will start one thread per core.  Since you most likely have only one instance of the GUI running at a time, this shouldn't be a problem, but it's certainly a setting you can play with to see if it helps you.


Roadmap

Examples
Component Playground

Add support for component-playground
http://projects.formidablelabs.com/component-playground/

Test
Integration Tests
On examples/gh-pages

https://github.com/kriasoft/react-starter-kit
https://github.com/vvo/selenium-standalone
https://github.com/webdriverio/webdriverio/blob/master/examples/webdriverio.with.mocha.and.chai.js
https://github.com/tatablack/karma-webdriverjs-launcher

https://docs.npmjs.com/files/package.json#files

Script loader
https://tomchentw.github.io/xrossref/#ZGVkL3NjcmlwdC5qcywgc3lzdGVtanMvc3lzdGVtanMsIFNsZXhBeHRvbi95ZXBub3BlLmpzLCBnZXRpZnkvTEFCanMsIHdlc3NtYW4vZGVmZXIuanM=

Closed by #95 and #96


Configure as default dashboard

Hi there! I'm wondering if there are any configuration tricks that can be done to make this dashboard the default one in HA.
Currently, the default HA dashboard pops up because I've deleted my other dashboards, but I'm not sure if there's a way to change that. Is this something that can't be done, as I can't select it as the default dasbhoard, or are there some hidden ways to do it? Thank you so much!

There's not, this is not technically a ha dashboard, just an addon.

Hi there! I'm wondering if there are any configuration tricks that can be done to make this dashboard the default one in HA.
Currently, the default HA dashboard pops up because I've deleted my other dashboards, but I'm not sure if there's a way to change that. Is this something that can't be done, as I can't select it as the default dasbhoard, or are there some hidden ways to do it? Thank you so much!

https://github.com/matt8707/ha-fusion/issues/228#issuecomment-1902672896


'VortoValue' does not contain a contructor that takes 0 arguments

Updated to nuget version 1.5.3 and exspected the VortoValue to be public, as changed about 3 months ago.
Does not appear to be public.
The .dll from the zip-package does have a constructor on VortoValue that is public
But doesn't have the #63 fix.

I'm running Vorto 1.5.3 and VortoValue is both public and has a parameterless public constructor.
I can confirm though that the #63 fix is missing from that release.

I believe this should be fixed now as the #63 fix is in the codebase. Feel free to re-open if that appears not to be the case.


Create and validate a build pipeline for the nodejs collection

Tested the dev-pipeline in the kabanero-pipelines repo to verify it works with a nodejs appsody stack.


New channel due to missing replaces (open liberty)

Signed-off-by: Martin Vala<EMAIL_ADDRESS>Hi @leochr,
there are two options, maybe you forgot to set replaces for the latest version or it was intended.
We added a version to separate channel as it actually fall off the update graph and are orphaned and not installable / upgradable anymore due to missing replaces in the path. After this PR, it will be installable from this channel.
The main reason for this update is that we are currently preparing to enable bundle based catalogs which involves background conversion, it’s currently failing for the reason stated above.
If you don't agree with the proposal, feel free to open a new PR:)
Thanks submitting your Operator. Please check below list before you create your Pull Request.

Flat operator directory structure is obsolete from 23-rd of October 2019, only nested directory structure will be accepted.

New Submissions

[ ] Does your operator have nested directory structure?
[ ] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Is your submission signed?

Updates to existing Operators

[ ] Is your new CSV pointing to the previous version with the replaces property?
[ ] Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[ ] Is your submission signed?

Your submission should not

[ ] Modify more than one operator
[ ] Modify an Operator you don't own
[ ] Rename an operator - please remove and add with a different name instead
[ ] Submit operators to both upstream-community-operators and community-operators at once
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description about the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

Our PR is just technically correct workaround, but only you know if everything can be replaced by higher version or not. If replaces is missing somewhere, we have 2 options. Create new channel or add replaces where it is missing. The last option is better, but only possible for someone who knows operator very well, so you as an author. Feel free to open a new PR if you have different opinion, we can communicate and get it merged.

@mvalahtv @J0zi Separate channel for the initial release of Open Liberty operator 0.0.1 is fine as it was helm based and not possible to upgrade from that to the go-based releases 0.3.0+. The only suggestion is to name the channel alpha instead of original. I'll submit a PR for that. Thanks.

Submitted PR https://github.com/operator-framework/community-operators/pull/2050 to update the channel name.


Error publishing to Azure App Service

Hello,
Need help. This runs well locally (on IIS Express), so I'm confident that I cloned and put in the right configurations.
However, I couldn't seem to publish. I get a "publish error" with the following error logs.
The command "npm install" exited with code 1.	MyHealth.Web
D:...\src\MyHealth.Web\node_modules\node-sass\vendor\win32-x64-48\binding.node	MyHealth.Web

This is an issue related to npm install step where node-sass module version that is installed does not have a correct win32-x64 binding.  My team has to hard-code the node-sass to version "3.2.0" to make that work.


If required attribute is set to false, do not prefix as is-required

Hey there,
This was just something I came across when using your library (thanks!). If you are actually setting the value of the required to true and false rather than including/excluding the attribute, the field will get marked as required regardless of the value.
Cheers

I am hesitant to apply this pull request. Setting the "required" html5 attribute should never be done with true/false as a value as that is incorrect usage, the required attribute is a boolean value in itself. Setting a "true" value suggests that the "false" value will make the form control optional, which is not the case.
<input type="text" required>

The only time you should add a value is if your document must be XHTML in which case it will be:
<input type="text" required="required">

Thanks for the feedback - learn something new every day!


.NET Designs Repository Created

.NET Designs Repository Created
Summary
.NET designs now have a home on GitHub at dotnet/designs. This repository and the process it uses is intended to be very similar to dotnet/csharplang. We want to share .NET design documents and get much earlier feedback on plans.
Details
The repository was just created. You can see two designs that you can participate on. More designs will show up over time. These two designs were selected as the initial set since the current behavior for both has received significant feedback.

.NET Core 2+ Naming and Versioning
.NET Core 2+ Version Binding

A separate repository was created for a few reasons:

.NET designs may need to be implemented in multiple repos, including runtime, framework and language repos.
.NET designs may or may not be specific to .NET Core.

Thanks in advance for your participation in this repository.

Reopening according to process.


Update to Python 3.10

https://aws.amazon.com/blogs/compute/python-3-10-runtime-now-available-in-aws-lambda/

[ ] Test
[ ] Update CFN template once available in GovCloud (https://docs.aws.amazon.com/govcloud-us/latest/UserGuide/govcloud-lambda.html#govcloud-lambda-diffs)

https://aws.amazon.com/about-aws/whats-new/2023/06/aws-lambda-python-3-10-aws-govcloud-regions/


fix(mock): Fixed error handling on preserveResolvers (#149)

TODO:

[x] Update CHANGELOG.md with your change (include reference to issue & this PR)
[x] Make sure all of the significant new logic is covered by tests
[x] Rebase your changes on master so that they can be merged easily
[x] Make sure all tests and linter rules pass

Yeah, interaction of mocks and default resolvers is pretty tricky. I think this is a decent solution. 👍

We can ship it once it's rebased.


Add my new pixel

Checklist

[x] I ran npm test locally and it passed without errors.

[x] I only edited the _data/pixels.json file.

[x] I entered the username in the pixels.json that I'm also using to create this pull request.
[x] I acknowledge that all my contributions will be made under the project's license.

@Mergifyio refresh

@Mergifyio update

@Mergify refresh

@Mergifyio refresh

@Mergifyio update

Closing to retry step-by-step


The unexpected appearance of elastic etcd service

Rancher versions:
rancher/server:
v1.6.11
rancher/agent:
v1.2.7
Infrastructure Stack versions:
healthcheck:
v0.3.3
ipsec:
v0.13.2
network-services:
metadata(v0.7.18),dns(v0.15.3),network-manager(v0.7.18)
scheduler:
v0.8.2
kubernetes (if applicable):
v1.8.3-rancher1
Docker version: (docker version,docker info preferred)
v1.12.3
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
Ubuntu 16.04.3 LTS
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
VirtualBox
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node rancher
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Kubernetes
Steps to Reproduce:

Add two or more hosts to kubernetes cluster
Just remain only one host in kubernetes cluster and deactive/delete the other hosts

Results:
Scheduler was always going to start one more etcd container on this left host, however it should be got a failure because the scheduling policy of etcd service was set to "the host must not have a host label of io.rancher.stack_service.name=$${stack_name}/$${service_name}"

Close in favor of https://github.com/rancher/rancher/issues/10411


Administrator can assign a curator to a work

Enable the ability to assign an individual as a curator to a deposit directly within the system. This information should be stored internally, accessible to administrators and curators, but not displayed publicly.

[ ] Modify the database schema to include a field for curator assignment associated with each deposit.
[ ] Add a new option in the admin view of works to assign a curator
[ ] Define appropriate permissions for admins to access and modify curator assignments
[ ] Audit trail for curator assignments

[ ]  Modify the database schema to include a field for curator assignment associated with each deposit.

@bdezray I'm still learning the scholarsphere terminology/structure, does deposit exclusively apply to works or does this need to be added to collections as well?

@Smullz622 My bad! I should be consistent with terminology here. We curate right now at the work level, so we would only need it applied to works and not to collections. However, we may want it applied on the project level if that is implemented.

@bdezray thanks! Follow up- is there a scenario when there would be more than one curator assigned to a work?

@Smullz622 Great question! Yes, but probably not at the same time. We would want to be able to transfer curation between people throughout the curation process. For the main purpose of the assignment of enabling notification, the most important information would be who is currently assigned, but we likely would want to keep a log of who all has worked on curation for a work.


Defined new revisions for golang dep: github.com/dgryski/go-sip13

Ran component through the scanners since I noticed it had not been harvested yet. PR matches curation site. Closing.


Console shows delete option for super admins when email username is enabled

Describe the issue:
In the console, there is a hardcoded check to figure out if the user is super admin [1]. This will fail if email username is enabled and also if the super admin username is changed from the deployment.toml.
Also it would be best to avoid danger actions for the currently logged in user to avoid unexpected problems.
[1] https://github.com/wso2/identity-apps/blob/master/apps/console/src/features/users/components/users-list.tsx#L305
How to reproduce:

Enable email username.
Login to console.
Go the users list.
Super admin will have the delete option in the listing and other danger actions in the edit page.

Expected behavior:
Logged in user should not have the delete and other danger actions enabled.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: IS 5.11.0-RC1
OS: Mac
Database: H2
Userstore: LDAP

Optional Fields
Related issues:

Suggested labels:

Super-admin name should come from an API. Preferably in Server Configurations API

Let's use this issue https://github.com/wso2/product-is/issues/10702 to track the API progress of the super-admin retrieval.


Add support for MIME types to Java and Objective-C clients

The Java and Objective-C clients currently hard-code the MIME type for attachments to "application/octet-stream". The clients should be updated to pass the actual MIME type of a file, if possible.
In Java, this can be done via the URLConnection#guessContentTypeFromName() method. The Objective-C client can use the approach outlined here:
http://stackoverflow.com/questions/24250475/post-multipart-form-data-with-objective-c

This is done.


docs: Example of Designation of Exempt Person (FinCEN DOEP | Form 110)

On our docs site let's have a page showing how to generate a DOEP report. It can include sample code and values. A full Go example would be ideal.
https://github.com/moov-io/fincen/blob/main/exempt_designation/activity.go

https://github.com/moov-io/fincen/pull/22


Achievement Bugs

Hi,
These achievements don't exist and aren't achievements and one of them is a duplicate. Would you be able to remove them please.
Thank you.

Duplicate #485 #487 #488 #491 #493 #497 #498 #500 #504 #513 #517 #524 #532 #537 #538 #539 #540 #552 #557 #558 #560 #565 #580


Sample working sty files

Hello,
I've tried playing and converting a number of style files, none of them seem to work, do you have example files, or a repository of files known to work? Thanks.

YAML export works great for me with original factory style from Genos.
I am running this in the Windows Ubuntu shell. Below are my installation notes.
git clone https://github.com/bures/sff2-tools
sudo apt-get update
sudo apt-get install python3-pip
sudo pip3 install pyyaml
sudo pip3 install construct==2.8.21
sudo apt-get install libasound2-dev
sudo apt-get install libjack-dev
sudo pip3 install python-rtmidi
python3 sty2yml.py /mnt/e/Music/Genos/Ordinary\ World/CalifornianCountry.T554.prs Cali.yml

Great. Thanks for the instructions. The prints don't work because they require Python 3.5. However they are not needed anyway. So commenting them out and using it on Python 3.4 is fine too.

Thanks, Thomas, awesome tools :-D So far I have not found any other tools that write SFF2 CASM format.
Greetings from LA to Praha, Markus

I was able to compile executable files with pyinstaller. This works on both Linux and Windows and possibly also MacOS.
sudo pip3 install pyinstaller
python3 -m PyInstaller --onefile sty2yml.py
python3 -m PyInstaller --onefile yml2sty.py
Attached sty2yml.exe and yml2sty.exe for 64bit Windows.
sff2-tools-windows-64bit.zip

I was able to compile executable files with pyinstaller. This works on both Linux and Windows and possibly also MacOS.
sudo pip3 install pyinstaller
python3 -m PyInstaller --onefile sty2yml.py
python3 -m PyInstaller --onefile yml2sty.py
Attached sty2yml.exe and yml2sty.exe for 64bit Windows.
sff2-tools-windows-64bit.zip

Please Please ! Could you make an .exe of styplay and ymlplay, I cant use pyinstaller because error
Thanks a lot !!!

@markusthegeek - Excellent. Thanks.

@markusthegeek - Excellent. Thanks.

Sorry Markus, I have not been able to order the files correctly. Can you compile styplay. py and ymlplay. py PLEASE !

@markusthegeek - Excellent. Thanks.

Your  sff2 tools can play any chord of any style. Yamaha Sty ?
I mean not only the CMaj7 factory base, but any other such as Csus, Cdim, etc etc.
Sorry for this intrude, but im very interested in your code to work with . Sty files. Thanks

@CIRCE-EYES - the *play.py tools only approximate what you are going to hear from the keyboard when you upload the style there. The *play tools don't handle any transposition based on selected chord, thus they don't have to understand the chords. You can create styles in any base, but the transposition of the style based on the selected chord is up to the keyboard.

Thanks Tomas for the information,
I thought you could create an application, for example a form full of buttons, each associated with an event, for example
button 1 play Csus
button 2 Cdim, etc etc,
I have misinterpreted your code, I thought I could transpose the base style made in CMaj7 to any other chord, my intention was not to modify the style to load it on a keyboard. my intention was to play the style through buttons associated with chords.
I have almost finished an app made in vbnet, which triggers the Mains.
I send a screenshot. Thank you very much for answering !!!

Anyway if someone could compile styplay.py and ymlplay.py to .exe I would be very happy. And greatefulllly
Thanks

yml-sty-players.zip
At least I get compile the players, but no luck  using them. Give me errors on init.py


chore(repo): Use correct install snippet in snapshot comment

Description
In our snapshot comment we were missing the --save-exact flag. With our current snippets you installed it with a ^ caret, making it possible for another version to be installed.
I also moved the name out of the code block so that when you copy it to your terminal the comment is not also copied.
Checklist

[ ] npm test runs as expected.
[ ] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[x] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[ ] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[ ] @clerk/backend
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/fastify
[ ] @clerk/chrome-extension
[ ] gatsby-plugin-clerk
[x] build/tooling/chore

This is for the installation inside the test project. You'll want to install the exact version, not the caret version - otherwise you might install another snapshot

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


Update cats-effect-laws, cats-effect-std to 3.5.5

About this PR
📦 Updates

org.typelevel:cats-effect-laws
org.typelevel:cats-effect-std

from 3.3.14 to 3.5.5
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.typelevel" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, version-scheme:early-semver, commit-count:1

Superseded by #744.

Superseded by #745.


Exported JSON cannot be imported

Fixes #586
Replaces #617, #623 (GitVersion issued due to the branch name, source of the branch?)
This PR is an attempt to fix the export/import of JSON messages.
There's a whole slew of broken things with the current code base 😞

Serialization

❌ Incorrectly stored body - the body is stored as an object rather than raw JSON. To work properly, the body has to be saved as an escaped JSON string.
❌ Incorrectly stored properties/headers - headers are stored as an object where it should be an array of key/value objects

Deserialization

❌ Incorrectly deserialized body - deserialized property is called content rather than body
✖ Incorrect merging of deserialized properties/headers and custom provide via app properties/headers - properties provided via app are lost. Don't think it matters since a restored message can be tweaked via modifying the serialized text.

To make it worse, there are multiple paths of invocation and reuse for XML & JSON.
Frankly, I don't know if it's worth spending time fixing it; fixing one thing will break something else.
I'm going to attempt fix JSON as this is the most common message serialization type used. No guarantees it won't break XML or other formats the application claims to support.

[x] Fix body serialization
[x] Fix body deserialization
[x] Fix property serialization
[x] Works with a single exported message (queue)
[x] Works with multiple exported messages (queue)
[x] Works with DLQ-ed single message (queue)
[x] Works with DLQ-ed multiple messages (queue)
[x] Works with a single exported message (topic/subscription)
[x] Works with multiple exported messages (topic/subscription)
[x] Works with DLQ-ed single message (topic/subscription)
[x] Works with DLQ-ed multiple messages (topic/subscription)
[x] Investigate what "Template" stands for - no longer relevant
Serialization/deserializaton from to/files other than JSON is not working --> Raised as https://github.com/paolosalvatori/ServiceBusExplorer/issues/619
[x] Save application properties types and do not hardcode to String

Changes in action
Preserving types
https://user-images.githubusercontent.com/1309622/158310081-586550e7-970b-4b13-b172-a92f02f5ff6e.mp4
Single message export/import
https://user-images.githubusercontent.com/1309622/158314129-82c0d2b7-4bc3-471a-8455-5f094b4f48e3.mp4
Multiple messages export/import
https://user-images.githubusercontent.com/1309622/158316558-23f6f0e7-d186-4a21-8918-2a4558563eed.mp4

:white_check_mark: Build ServiceBusExplorer 5.1.0-exported-json-cannot-be-imported.84.build.924 completed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/ec1ddc527b by @SeanFeldman)

@paolosalvatori @ErikMogensen please review/approve. PR build is green. Branch is red but it refers to the wrong branch.

Which branch should we review?

Which branch should we review?

The one associated with this PR, coming from https://github.com/SeanFeldman/ServiceBusExplorer/tree/feature/exported-json-cannot-be-imported


New formsflow.ai env in d1***

DOD

[x] KC Setup
[x] Build of forms-flow-forms in tools
[x] Build of forms-flow-forms in tools
[x] Build of forms-flow-forms in tools
[x] Deployment of ffa in dev
[x] Re-configuration of new ffa to dev
[x] Re-configuration of old ffa to dev-marshal

to close after todays testing