gitignore: Updated for venvs with other python versions

What does this PR do?
Updated gitignore to allow all venvs with different python versions

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


plugins: Add capability to dis-/enable them per buffer

From current end user perspective it's not really clear why plugins can't be disabled for local buffers and receiving an invalid option doesn't tell that much. So I tried to block at least the callbacks triggering actions in the plugins.
Close #2835

Tested with patch. Works as expected on my end. 👍

Stumbled across this (merged) PR while figuring out how to address the problem described in https://github.com/zyedidia/micro/issues/2973#issuecomment-1776317378, and found a couple of dubious things in this PR. Please see my comments.


Official Docker image for InfluxDB?

Is there any official Docker image for InfluxDB? The best one I could find on Docker Hub was this one.

closing in favor of #1129


[BUG] FATA[0000] ros upgrade cannot be supported

RancherOS Version: (ros os version)
v1.5.2
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
baremetal
Command
sudo ros os upgrade
or
sudo ros os upgrade -i rancher/os:v1.5.3
or
sudo ros os upgrade -s -i rancher/os:v1.5.3
or
sudo ros os upgrade -i rancher/os:v1.5.0
Result for all
FATA[0000] ros upgrade cannot be supported

If you didn't use ros install to install ROS, ros os upgrade cannot be supported.
Looks like that you are using autoformat?

@niusmallnan Thanks for your time
I used ros install from official docs and don't use autoformat!
mkfs.ext4 -FL RANCHER_BOOT "${diskPath}1" || printf '\xe2\x9d\x8c Error'
mkfs.xfs -fL RANCHER_OS "${diskPath}2" || printf '\xe2\x9d\x8c Error'

ros install \
  --image rancher/os:v1.5.2 \
  --no-reboot \
  --install-type noformat \
  --force \
  --device $diskPath \
  --partition "${diskPath}1"

I force to installing with noformat because I want to format RANCHER_STATE as XFS, not ext4 and XFS can't use RANCHER_STATE as a label because of length! and its another bug/issue from the ROS (not important).
How can I skip this situation and forced to update or do it manually?

Your problem is ros install or ros upgrade?
I cannot get your points.
Here is a doc about using the custom partition layouts. https://rancher.com/docs/os/v1.x/en/about/custom-partition-layout/
XFS as a STATE partition may not be feasible because the RANCHER_STATE label is necessary.

@niusmallnan Thank you for your time, As I mentioned clearly in the main post and issue title! I cannot update the RancherOS to the latest version!
You asked:

If you didn't use ros install to install ROS, ros os upgrade cannot be supported.

And I answered that by explaining how I installed the RancherOS and why.

Is there any plan to fix this issue. I too hit the same issue. The rancher OS is installed from Rancher UI template
[root@rancher-etc-control1 ~]# sudo ros os upgrade
FATA[0000] ros upgrade cannot be supported


v2.14.1 [release]

🐛 Bug Fixes

router

#7843 Optional dynamic routing in children routes (resolves #7571)
#7845 Remove optional params in routes ending with index (resolves #5874)

vue-app

#7801 Handle server-side router errors (resolves #7798)

core, config

#7839 Disabe esm cache (resolves #7838)

builder

#7789 Add types for build.corejs option and use number

crawler

#7806 Skip external links with no schema (resolves #7805)

webpack

#7808 Avoid grouping with default name (resolves #7795)

cli

#7802 Generate still runs when cache disabled
#7796 Normalize win32 paths for nuxt generate cache (resolves #7792)

📝 Examples

tailwindcss-purgecss

#7800 Update tailwindcss from 0.6 to 1.6 using purgecss

🏡 Chore

stale

#7828, #7844 Ignore feature request issues

💖 Thanks to

Ahad Birang (@farnabaz)
@ChangJoo-Park
Matthieu Sieben (@matthieusieben)
Rafał Chłodnicki (@rchl)
@lihbr

Codecov Report

Merging #7816 into 2.x will increase coverage by 0.00%.
The diff coverage is 52.94%.

@@           Coverage Diff           @@
##              2.x    #7816   +/-   ##
=======================================
  Coverage   68.88%   68.89%
=======================================
  Files          90       90
  Lines        3834     3838    +4
  Branches     1037     1038    +1
=======================================
+ Hits         2641     2644    +3
- Misses        969      970    +1
  Partials      224      224

Flag
Coverage Δ

#unittests
68.89% <52.94%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

packages/cli/src/utils/generate.js
0.00% <0.00%> (ø)

packages/config/src/config/build.js
100.00% <ø> (ø)

packages/generator/src/generator.js
86.22% <0.00%> (ø)

packages/webpack/src/config/client.js
62.50% <50.00%> (+1.32%)
:arrow_up:

packages/webpack/src/config/base.js
64.41% <80.00%> (ø)

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 60f6863...5909de7. Read the comment docs.

@pi0 Do you think we can publish https://github.com/nuxt/components/pull/84 in this release?


Init contract module

@tds-1 : please

Update Readme.
update changelog
Address the comments

pr base change

Created on Main Repo


Zone Changelog: filter and paginate

Most of what I'd consider controversial takes place in models/zone.php:list_changesets():

It now takes an optional parameter for a list of filters (search comment, date range, pagination)
It now returns two values (changes as before, plus total number of pages) => this is a breaking (internal) change, so all usages were updated.
Items per page is for now hardcoded in there to 200. I'm open to improve this (config.ini knob?) if this is a problem.
The SQL statement to select changelog entries is now relatively complex. To allow for the optional filter parameters, they have to be passed twice. This is a bit ugly, but I don't see a nicer alternative (assembling the SQL query as a string with if/else seems even worse).

I'm also a bit unhappy with the amount of <?php...?> blocks for pagination in templates/zone.php but there's just a lot of cases to check.
commit message:
Implements:

filter by comment (default all)
filter by date range (default all)
paginate results (200 items per page)

Filtering by author, while possible, is not implemented in this patch,
since we don't care about it. Note that the changelog displayed on a
user's profile is not touched; it continues to be unpaginated and
unfilterable for the same reason.
fixes #179

I hope this doesn't come across as pushy, but do you have the time to review this please, @thomas-pike?

I hope this doesn't come across as pushy

Not at all, thanks for reminding me to look at this. I'll take a look now.

Thank you for your continued contributions!


TypeError in AddReservoirConstraintWithActive

The AddReservoirConstraintWithActive in Python does not work, it returns a TypeError for actives "has type IntVar, but expected one of: int, long". Active should indeed be of type IntVar.
Below is a simple code to reproduce the bug
model = cp_model.CpModel()
model.AddReservoirConstraintWithActive(
    [model.NewIntVar(0,10, f'var_{i}') for i in range(3)],
    [2 for _ in range(3)],
    [model.NewBoolVar(f'active+{i}') for i in range(3)],
     0,
     10)

Yes, this is a bug.
It will be fixed in 7.0.
In the meantime, you can patch the cp_model.py file
model_ct.reservoir.actives.extend([self.GetOrMakeIndex(x) for x in actives])
or build from source.
Thanks for the report.
You can also copy the AddReservoir... code in your file, fix it and use it
until 7.0 is out.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le jeu. 3 janv. 2019 à 21:09, philippegr<EMAIL_ADDRESS>a
écrit :

The AddReservoirConstraintWithActive in Python does not work, it returns
a TypeError for actives "has type IntVar, but expected one of: int,
long". Active should indeed be of type IntVar.
Below is a simple code to reproduce the bug
model = cp_model.CpModel()
model.AddReservoirConstraintWithActive(
[model.NewIntVar(0,10, f'var_{i}') for i in range(3)],
[2 for _ in range(3)],
[model.NewBoolVar(f'active+{i}') for i in range(3)],
0,
10)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1001, or mute the thread
https://github.com/notifications/unsubscribe-auth/AKj17QSaZg1X8mBZucRQs45673-4BAcrks5u_mOXgaJpZM4Zosix
.


myarray.includes is not a function

myarray = new Array;
myarray.push("pls","send","help");
myarray.includes("pls");
->myarray.includes is not a function[...]

Tested same thing in Browser AND NodeJS, works fine

nvm

found a workarround by adding
Array.prototype.includes = function(t){return this.indexOf(t)!="-1"}
to the start of the file, to manually re-add the function of .includes, still think this should be fixed

Array.prototype.includes is an ECMASCript6 feature ; ES6 support in nashorn (java's JS engine) can be found (partially, it seems not everything is supported yet) in java 9. See here for more informations

+1 What @Gloorf said. includes is an ES6 function. ScriptCraft depends on the Javascript engine provided by the Java Runtime. Right now, Java doesn't have full support for ES6 (the latest version of Javascript) so some functions and features of ES6 will be missing.


Failing composer platform check in version 5.23.4 and higher

As soon as I install captainhook/captainhook-phar in version 5.23.4 or 5.23.5 and run vendor/bin/captainhook -f install, I get the following error:

Composer detected issues in your platform: Your Composer dependencies require a PHP version ">= 8.2.0". You are running 8.1.30. in phar:///var/www/html/vendor/bin/captainhook/vendor/composer/platform_check.php on line 24

We are running a project inside docker, the mentioned PHP version 8.1.30 is correct, but I don't see any dependency in the whole project on PHP 8.2.
As soon as I downgrade to 5.23.3, the error is gone and the configuration is properly set up.
Any idea, what might be wrong here?

I think the PHAR builder Box requires a minimum version of 8.2. I will check if it can be manipulated somehow to run under some older PHP versions.

I just released 5.23.6 build on php 8.0 with an older Box version.
Please check if that works for you again.

@sebastianfeldmann , unfortunately, it does not.
The installation command now works, but as soon as the first hook runs, I get the following error:
Box Requirements Checker
========================
> Using PHP 8.3.6
> PHP is using the following php.ini file:
  /etc/php/8.3/cli/php.ini
> Checking Box requirements:
  ....E.E
 [ERROR] Your system is not ready to run the application.
Fix the following mandatory requirements:
=========================================
 * The application requires the extension "xml". Enable it or install a
   polyfill.
 * The package "sebastianfeldmann/git" requires the extension "simplexml".
   Enable it or install a polyfill.

Strangely it now fails as well, when I use 5.23.3, at least in a new project.

New php setup?
Looks like you are missing some php-extensions  like xml and simplexml.
Please check your php info if you have the necessary extensions enabled.
Maybe your CLI php.ini does not include them.

Ah, interesting, found the issue.
It is related to https://github.com/captainhookphp/captainhook/issues/261.
This one here can actually be closed, this is indeed fixed with your new update.
Let me share details in the correct issue description.
Thanks as always for your amazingly fast help.


MythX cannot be executed after installed.

MythX CLI version: 0.6.22
Python version: 3.9.1
Operating System: macOS 12.1 Monterey

Description
I followed mythx-cli documentation to install mythx-cli. No matter what command typed in, mythx-cli only shows the Traceback message.
What I Did
~:pip3 install mythx-cli
~:mythx analyze --help

Traceback (most recent call last):
  File "/opt/homebrew/bin/mythx", line 5, in <module>
    from mythx_cli.cli import cli
  File "/opt/homebrew/lib/python3.9/site-packages/mythx_cli/cli.py", line 21, in <module>
    from mythx_cli.render.command import render
  File "/opt/homebrew/lib/python3.9/site-packages/mythx_cli/render/command.py", line 7, in <module>
    import jinja2
  File "/opt/homebrew/lib/python3.9/site-packages/jinja2/__init__.py", line 12, in <module>
    from .environment import Environment
  File "/opt/homebrew/lib/python3.9/site-packages/jinja2/environment.py", line 25, in <module>
    from .defaults import BLOCK_END_STRING
  File "/opt/homebrew/lib/python3.9/site-packages/jinja2/defaults.py", line 3, in <module>
    from .filters import FILTERS as DEFAULT_FILTERS  # noqa: F401
  File "/opt/homebrew/lib/python3.9/site-packages/jinja2/filters.py", line 13, in <module>
    from markupsafe import soft_unicode
ImportError: cannot import name 'soft_unicode' from 'markupsafe' (/opt/homebrew/lib/python3.9/site-packages/markupsafe/__init__.py)

I have this issue as well.

MythX CLI version: 0.6.22
Python version: 3.8.3
Operating System: Windows 10

Updated: The version of package "markupsafe" has been upgraded to 2.1.1 and it's not compatible to mythx-cli
This command will fix the problem:
pip3 install markupsafe==2.0.1

Still getting the error @santai-wang-7senses


The token cannot be intercepted

[+] Waiting for multicast token from app...
I can't solve the issue of getting a token
the firmware of my switch
Tuya-Generic_W602--BLE-Light-Switch-2gang--1.3.10.bin.zip
You can help create a profile?

This is switch firmware version 1.3.10 which is already on the known patched firmware list.  A profile cannot be added for this device and serial flashing will be your only option.
This isn't a TuyaMCU device, so the schema won't do you any good.
If you're looking for the device configuration for upk2esphome, it is
{
	"bt1_lv": 0,
	"bt1_pin": 24,
	"bt2_lv": 0,
	"bt2_pin": 22,
	"bt_self_type": 1,
	"ch_cddpid1": 9,
	"ch_cddpid2": 10,
	"ch_dpid1": 1,
	"ch_dpid2": 2,
	"ch_num": 2,
	"crc": 102,
	"cyc_dpid": 43,
	"ffc_select": 0,
	"inch_dp": 44,
	"init_conf": 38,
	"jv": "110.0.0",
	"key_rst_en": 1,
	"module": "T34",
	"net_trig": 4,
	"net_type": 0,
	"netled1_lv": 0,
	"netled1_pin": 8,
	"netn_led": 1,
	"nety_led": 0,
	"on_off_cnt": 10,
	"onoff_clear_t": 10,
	"onoff_rst_m": 0,
	"onoff_rst_type": 2,
	"onoff_type": 2,
	"rand_dpid": 42,
	"remote_add_dp": 49,
	"remote_io": 6,
	"remote_list_dp": 50,
	"remote_select": 1,
	"reset_t": 5,
	"rf_width": 345,
	"rl1_lv": 1,
	"rl1_pin": 14,
	"rl2_lv": 1,
	"rl2_pin": 28,
	"series_ctrl": 0,
	"total_bt_lv": 0,
	"total_bt_pin": 10,
	"total_stat": 2,
	"zero_select": 0
}

thanks!


Operations with graphs

Is there a way to realize operations with graphs? I have two graphs and I need to subtract them.

The Ruby RDF toolset doesn't specifically have such a tool, or an RDF Diff tool, which would be useful, however you may be able to achieve what you want using a SPARQL MINUS or SPARQL UPDATE DELETE WHERE, both available using the SPARQL gem. Note that the problem is that BNodes in two different graphs are different, and a simple removal doesn't work. Even if you load the same serialization into two different graphs, the BNodes aren't equivalent.
However, if you can put the graph you want to do in a SPARQL WHERE clause, BNodes act like existential variables, so that they will match BNodes that have the same pattern.
Lastly, on the books to implement is RDF Normalization, which might work, as it can create canonical serializations of both graphs allowing a text diff to work, or by skolemizing the BNodes in each graph using normalized identifiers allowing the use of something like RDF::Mutable.delete.

See issue #209 on how rdf-normalize might add a diff command to the rdf executable.

It might be good to consider roads to interoperability with something like https://github.com/monora/rgl or the (apparently dead) Ruby igraph bindings. Those won't be bnode aware, but

(closed by mistake)

This seems like a punt for 2.0 so I'm removing the tag.
I have some ideas about supporting graph operations through FFI to C and/or Rust, but think this is a larger work chunk than we can take on on the 2.0 timeline. It would be useful for any interested parties to chime in on use cases.


test: add hack/verify-gomod.sh

What type of PR is this?
/kind test
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #181
Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

/ok-to-test

/lgtm
/approve
thanks @Sakuralbj can you add this tool to other 3 CSI drivers?

OK


Request current user for MFA and email verification required users

This means the session expiry time is set properly.

The change makes sense to me, and I've got it working on all account types locally 👍


fixed JWTAuthBackend.get_auth_token

self.leeway and self.expiration_delta are already converted to timedelta at line 153&155 in JWTAuthBackend.__init__().

Coverage remained the same at 85.556% when pulling 1e3cd223d8a1dbc36420e158d5595472d3f7572e on solarsail:master into 6c2ea2ac1d082791e21a8ea53cc8fed0ddcdf73e on loanzen:master.

@solarsail Thanks for this. you are finding this useful.

I'm trying this in a new REST service project, and it works like a charm. Thanks for the nice work.


vagrant ssh user and password

if an ssh user and password is defined in the config section of the node definition, beaker will add the proper ssh options to the generated Vagrant file instead of forcing the use of pre-seeded keys

Can one of the admins verify this patch?

Thanks for your PR @isaiahfrantz ; we have some PR guidelines written up here that you should apply to this work. I'd prefer this not be a MAINT PR but one with an actual ticket created in the BKR project in JIRA. Along with those changes, I'd like to see some spec tests for this PR as well.
Let us know if you have any questions about those steps.

@isaiahfrantz last comment is still current

@isaiahfrantz last comment is still current. If this hasn't been touched in a week, it'll be closed due to inactivity.


added mattermost icon

Ill sugest to that the skill is renamed to mattermost-skill and that title in readme.md just is Mattermost.
That makes it look niser in the store as everything in the store is for mycroft :)

Sorry, I somehow missed the notification on this PR...


Dependency context from require_optional needs more information

Feature request
Summary
The context generated by the ContextModuleFactory from a require_optional call is not adequate to handle the dependency expression that generates it.
Details
Is this a bug report?  Is this a feature request? It's ambiguous.
Wherever require_optional appears in the code (e.g. in MongoDB node driver), Webpack prints a warning Critical dependency: the request of a dependency is an expression and bundles all of its possible dependencies.

An example of the full warning

WARNING in ../node_modules/require_optional/index.js 84:18-42
Critical dependency: the request of a dependency is an expression
@ ../node_modules/mongodb/lib/core/utils.js
@ ../node_modules/mongodb/lib/bulk/common.js
@ ../node_modules/mongodb/index.js
@ ../lib/MongodbService.js
@ ./gauth.js

Common advice for this warning is to use the ContextReplacementPlugin to indicate which dependency is correct (or even just suppress the warning)
The context objects passed to ContextReplacementPlugin by the ContextModuleFactory at this stage does not have enough information to respond. The request field value is always only '.' and every other field value is likewise identical, because the Context instance here refers to dependency requests in require_optional and not the underlying dependency that uses it.

e.g. The identical context object passed by every require_optional call

{
  "addon": "",
  "resource": "/absolute/path/node_modules/require_optional",
  "context": "/absolute/path/node_modules/require_optional",
  "dependencies": [
    {
      "module": null,
      "weak": false,
      "optional": true,
      "loc": {
        "start": {
          "line": 84,
          "column": 18
        },
        "end": {
          "line": 84,
          "column": 42
        }
      },
      "options": {
        "request": ".",
        "recursive": true,
        "regExp": false,
        "mode": "sync"
      },
      "userRequest": ".",
      "critical": "the request of a dependency is an expression",
      "hadGlobalOrStickyRegExp": false,
      "range": [
        2560,
        2584
      ],
      "valueRange": [
        2568,
        2583
      ]
    },
    {
      "module": null,
      "weak": false,
      "optional": true,
      "loc": {
        "start": {
          "line": 92,
          "column": 20
        },
        "end": {
          "line": 92,
          "column": 44
        }
      },
      "options": {
        "request": ".",
        "recursive": true,
        "regExp": false,
        "mode": "sync"
      },
      "userRequest": ".",
      "critical": "the request of a dependency is an expression",
      "hadGlobalOrStickyRegExp": false,
      "range": [
        2864,
        2888
      ],
      "valueRange": [
        2872,
        2887
      ]
    },
    {
      "module": null,
      "weak": false,
      "optional": false,
      "loc": {
        "start": {
          "line": 99,
          "column": 35
        },
        "end": {
          "line": 99,
          "column": 67
        }
      },
      "options": {
        "request": ".",
        "recursive": true,
        "regExp": false,
        "mode": "sync"
      },
      "userRequest": ".",
      "critical": "the request of a dependency is an expression",
      "hadGlobalOrStickyRegExp": false,
      "range": [
        3091,
        3106
      ],
      "valueRange": [
        3091,
        3106
      ]
    }
  ],
  "resolveOptions": {},
  "request": ".",
  "recursive": true,
  "regExp": false,
  "mode": "sync"
}

If there is a single require_optional in all of the dependencies, then ContextReplacementPlugin can properly handle that single call (or resolve.alias). If there are many, it's not clear which dependency is needed: e.g. one call is ambivalent between bson and bson-ext, and another wants to throw an error if mongodb-extjson is unavailable, and yet another wants to halt if kerberos is not installed.

What is the expected behavior?
The context object that ContextModuleFactory creates as a result of require_optional should be detailed enough to properly resolve the ambiguity in dependency, perhaps by including the warning's stack trace of the warning that includes the files that call require_optional.
What is motivation or use case for adding/changing the behavior?
I think it's probably answered under the Details heading, above, but I'm happy to clarify if anything is unclear.
How should this be implemented in your opinion?
Some naive possibilities (given that I'm naive about Webpack internals):

Enhance the Context instance passed from ContextModuleFactory, perhaps to include the stack trace if possible, or otherwise enable Context related plugins to distinguish among identical contexts
If this ambiguity-about-dependencies is passed to plugins via some other ... ModuleFactory?... mechanism, maybe it would be better to create a plugin that uses that

Are you willing to work on this yourself?
Definitely. I'd like some feedback or insight, but I am willing to do the heavy lifting.
First, is this write-up clear? Do you, reading this, understand what the issue is?
Second, if so, is enhancing the Context instance even possible? Is there a better way? I don't know if the stack trace is even available to the ContextModuleFactory.
Edits: nervously clarifying with links and examples

bump


Add custom handler for jump button on click?

Currently the normal onClick handler gets triggered when clicking on jump button. How do you navigate to the first or last page directly with these jump buttons?

The point of these jump buttons are to do jumps and not go to last page, that's why there's no controller for these buttons, you can use first and last page buttons to implement what you're explaining


sequence preprocessing

Notes for reviewers

added the visit filter under the patientloader class, since I felt it was more a patient filter than a diagnosis filter, but no strong feelings about it. also, should we add some tests for this filter, and the before2013 one?

TODO: make sure min_n_visits=5 in BEHRT loader

@MartinBernstorff something is wrong?


refactor: test variables

Adds variables to tests that will be changed in #387 to simplify the review for the latter.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 68.98%. Comparing base (3476994) to head (4acf8bc).

@@            Coverage Diff             @@
##             main     #408      +/-   ##
==========================================
+ Coverage   68.41%   68.98%   +0.57%
==========================================
  Files          70       70
  Lines       11838    12056     +218
  Branches    11838    12056     +218
==========================================
+ Hits         8099     8317     +218
  Misses       3482     3482
  Partials      257      257

Files with missing lines
Coverage Δ

pallets/nfts/src/tests.rs
99.88% <100.00%> (+<0.01%)
:arrow_up:


[13.0.x] Backport ISPN-13497 and subtasks

https://issues.redhat.com/browse/ISPN-13589
https://issues.redhat.com/browse/ISPN-13600
https://issues.redhat.com/browse/ISPN-13497

Integrated into 13.0.x, thanks @pruivo !


Replace :method: with :func: in Python init docstring.

Otherwise the following failure occurs in tox -e docs:
Warning, treated as error:
/usr/local/google/home/brianwatson/src/gapi-dev/gapi-pubsub-python/.tox/docs/lib/python2.7/site-packages/google/pubsub/v1/publisher_api.py:docstring
of google.pubsub.v1.publisher_api.PublisherApi:15: ERROR: Unknown
interpreted text role "method".

ERROR: InvocationError:
'/usr/local/google/home/brianwatson/src/gapi-dev/gapi-pubsub-python/.tox/docs/bin/sphinx-build
-W -b html -d docs/_build/doctrees docs docs/_build/html'
_________________________________________________ summary
_________________________________________________
ERROR:   docs: commands failed

Pre-push hook installed.
Change-Id: Ieced602e803e05a6acfa056f8e43700b9831e91e

LGTM


Run only selective tests

Hi team , the filters thing that is mentioned in the documentation. Is it also possible if I am not using junit/java for my tests and if i just have the yaml files -> mention it in the specmatic.json and then run it through command line. I tried multiple ways but was not able to do so
"start": "cross-env-file -p ./token.json  specmatic test --testBaseURL=https://abc.com  --junitReportDir=./test-output && node cleanup"

This is the start command I am using, need urgent help

And when we write --filter-name "xyz" , where should this xyz be present for that endpoint (example) to be hit or not hit. Please provide example. It will be great as I am stuck for this implementation
@harikrishnan83 @nashjain @rakeshpai

@GagandeepSharma060 thanks for sharing issue you are facing. We will update you shortly.

@pranavgawri found out that the filters are applied on the path name itself. It is great but can we have some tagging kind of mechanism like we have in Cypress.
Because filtering on the path names does not seem to be a great idea when dealing with a lot of services.
Just a thought.

@GagandeepSharma060, Thank you for sharing your experience and suggestion. We're glad you found a workable solution with path name filtering.
Your idea about implementing a tagging system similar to Cypress is interesting and could indeed be beneficial for filtering tests across multiple paths. We've noted this as a feature request for future consideration.
We appreciate your feedback as it helps us improve Specmatic. Thanks.

Hi @GagandeepSharma060 ,
We have enhanced our --filter option and I hope it should be able to help you better in your use case. Here is a quick overview:
The new filtering system provides powerful capabilities that align well with API contract testing:

Filter by HTTP Method and Path:

specmatic test --filter="METHOD=POST" --filter="PATH=/api/employees"

Filter by Status Codes (including pattern matching):

specmatic test --filter-not="STATUS=4xx"  # Excludes all client errors

Advanced Combinations:

specmatic test --filter="PATH=/api/employees" --filter="METHOD=POST" --filter-not="STATUS=400,401,403"

These filters work directly with your API contract structure, making it natural to organize and run your tests based on your API's design.
Please find the full details in our documentation here: https://specmatic.io/documentation/contract_tests.html#selectively-running-tests-in-ci
Let us know how this works for you!
Thanks,
Pranav

Thanks for the quick response and change team, will try it out and get back to you for feedback or any issues .
I believe this feature would help others too
Just a question, can we filter through description of the test .... as in that way it will give a huge flexibility

Hi @GagandeepSharma060,
Thanks for the feedback about filtering by description/summary fields. I've opened a discussion thread to explore this further: Filtering Tests by Description/Summary Fields
Please take a look and share your thoughts there. Once you've reviewed the discussion, please feel free to go ahead and close this original issue.
Appreciate you helping us improve Specmatic!
Thanks,
Pranav

Hi @pranavgawri ...Thanks for the response. Justwanted to let you know that filtering by the Example name is not working for me..
Let me explain
--filter="EXAMLPE_NAME=ABC" is not working however. --filter-not="EXAMLPE_NAME=ABC" is working fine.
Can u please look into it

Hi @GagandeepSharma060 ,
Thanks for sharing the issue you are facing. We are able to replicate it at our end, and will push the fix soon.
I will update you on the same.
Thanks,
Pranav


⚠ Bofrost.de has degraded performance

In 8be7573, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1726 ms

Resolved: Bofrost.de performance has improved in ab80e41.


Implementation of Inventory API

I wrote the Sponge port of BuycraftX, which centers around a GUI functionality. Little did I know, the Inventory API has not yet been implemented. Thus, I offer the following challenge:

Whoever contributes the most to the implementation of inventories in SpongeVanilla and/or SpongeForge (likely both through SpongeCommon) will be entitled to a $10 bounty.

The bounty can be claimed here (https://www.bountysource.com/issues/31041996-implementation-of-inventory-api).
This is purely an added bonus, and because I think Sponge developers are so amazing I might up that in the future. I have already become a $30 Patreon donor (funded purely with money I make off of my plugin(s)) and this is an added incentive for something that will be implemented no matter what.

:moneybag: :moneybag: :moneybag:

The next phase of inventory is blocked on some features of Mixin needed to support it properly for mods. Once Mixin 0.6 is completed the remaining inventory will follow shortly afterward.


RHINENG-7094 Adjust regex to handle advisory ids correctly

## Secure Coding Practices Checklist GitHub Link
- https://github.com/RedHatInsights/secure-coding-checklist

## Secure Coding Checklist
- [ ] Input Validation
- [ ] Output Encoding
- [ ] Authentication and Password Management
- [ ] Session Management
- [ ] Access Control
- [ ] Cryptographic Practices
- [ ] Error Handling and Logging
- [ ] Data Protection
- [ ] Communication Security
- [ ] System Configuration
- [ ] Database Security
- [ ] File Management
- [ ] Memory Management
- [ ] General Coding Practices

@rexwhite Hmm the labeler is still failing 🤔 Not sure why because the new signature looks good to me.

@marleystipich2 It looks like the GitHub labeler has been updated and our config needed updating so I pushed a new config - let's see if that fixes it 😬
The labeler just scans for changes to any static playbook and adds a 're-sign' label to the PR so we don't accidentally deploy stale playbooks.

@rexwhite Sorry, I was on PTO but thanks for fixing the labeler and merging this! 🙂


Murmur Hashing losses significant number of kmers

Tried to use the Murmur Hashing on 1 Million sequences containing 2,601,505 unique canonical kmers.

MQF with Q{27,28,29,30} + Murmur Hashing stored 112,624 Kmers
MQF with Q{27,28,29,30} + Integer Hashing stored 2,601,505 Kmers
PHMAP + Murmur Hashing stored 117071 Kmers
PHMAP + Integer Hashing stored 2,601,505 Kmers

Should not the numbers change by the change of q?

@drtamermansour As far as I know, yes. However, I tried many Q values with the same result each time.

if kmer Decoder using the Murmur hash function to hash the kmers and then insert the hash value in the kdataframe then collisions are not handled correctly

@shokrof Can you explain more, please?

Hash Functions sometimes give the same hash value to two different inputs. this is called a collision. Hashtables are designed to resolve collisions by different methods like probing or open addressing. if you are creating the flat_map<string, int> then the flat map will be able to resolve the collisions. However, If you created the flat_map<uint64_t,int> and you hash the kmers first using Murmur hashing and then inserting the hashes to the map. flat_map wont be able to resolve the collisions and this will make the kDataframe approximate not exact

Reversible Hash functions dont have collisions but they are limited for small ksizes <32


parenthesis in the return clause of the function definition

What version of Go are you using (go version)?
go1.10
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
amd64
What did you do?
Don't think this is a bug, it is more of an enhancement. I am not too strong about it and I recognize I probably miss background, but:
I keep forgetting to add parenthesis to the return clause of the function definition:
func FunctionName(arg argType) string, bool, int {
...
}

syntax error: unexpected comma after top level declaration
I am not sure why they are needed though.

I am not sure why they are needed though.

func f() func() int, int { ... }
How many return values does f have?

well that is horrible, but is good enough to educate me thanks.


Add an API route to load occurrence data for the map

It needs to be possible to get occurrence data from the graphql API endpoint, using an appropriate query. The initial fields we care about are:

site.location
sample.n_all
These two fields should give us enough information to start plotting on the map. The query used should initially just extract these fields.
Editing the occurrence.resolver class is a good starting point. To populate your db, use the occurrence.csv template file in the API public folder.

Acceptance Criteria:

[ ] It is possible to extract the above data from a populated db, through the vector-api/graphql endpoint, using an appropriate query
[ ] The query is able to get paginated data
[ ] The query is documented on this story

Extending the test/get_nested branch, sample export added to resolver with queries tested on 'http://localhost:1234/vector-api/graphql' using a POST from Postman:
site location/co-ordinates can now be queried:
query {
allGeoData{
site{
longitude,
latitude
}
}
}
sample n_all can now be queried:
query {
  allGeoData{
     sample{
         n_all
        }
      }
  }

All AC met, test TC-3.7 created in STS.


swagger mixin bug

i have latest swagger binary installed from this repo
i have common.swagger.json that have empty "definitions": {},
and i mixin to it another swagger json file that have
"/api/central/v0/volumes/{volume}": {
  "get": {
    "operationId": "VolumeExport",
    "responses": {
      "200": {
        "description": "A successful response.(streaming responses)",
        "schema": {
          "$ref": "#/x-stream-definitions/volumeVolumeExportRsp"
        }
      }
    },
.....

"x-stream-definitions": {
  "volumeVolumeExportRsp": {
    "type": "object",
    "properties": {
      "result": {
        "$ref": "#/definitions/volumeVolumeExportRsp"
      },
      "error": {
        "$ref": "#/definitions/runtimeStreamError"
      }
    },
    "title": "Stream result of volumeVolumeExportRsp"
  }
}

after swagger mixin run i don't have in resulted file x-stream-definitions

Could you please join your full specs?

i'm join full specs, for issue i provide only it parts

I am asking because I don't quite understand what you want to achieve from the snippet you are giving. It looks like you are expecting a merger of arbitrary json objects (like x-stream-definitions)
but this is not what the tool provides.
However, swagger mixin only supports merger of swagger specs, e.g. it understands a definitions sections.
On the other hand this x-stream-definitions could be an extension we would expect to be merged, but this would not be defined at the top level.

hm, x-stream-definitions added by grpc-gateway protoc-gen-swagger tool
// http://swagger.io/specification/#swaggerObject
type swaggerObject struct {
  Swagger             string                              `json:"swagger"`
  Info                swaggerInfoObject                   `json:"info"`
  Host                string                              `json:"host,omitempty"`
  BasePath            string                              `json:"basePath,omitempty"`
  Schemes             []string                            `json:"schemes"`
  Consumes            []string                            `json:"consumes"`
  Produces            []string                            `json:"produces"`
  Paths               swaggerPathsObject                  `json:"paths"`
  Definitions         swaggerDefinitionsObject            `json:"definitions"`
  StreamDefinitions   swaggerDefinitionsObject            `json:"x-stream-definitions,omitempty"`
  SecurityDefinitions swaggerSecurityDefinitionsObject    `json:"securityDefinitions,omitempty"`
  Security            []swaggerSecurityRequirementObject  `json:"security,omitempty"`
  ExternalDocs        *swaggerExternalDocumentationObject `json:"externalDocs,omitempty"`
}

ok now I see clear. The swagger object is extensible. This should work.
Maybe this is just an issue with nesting the object? Like:
{
  "x-stream-definitions": {
  ...
 }
}
```

Oh when checking in the code I can see this is not supported... This is indeed a bug. Nice catch.

Oh when checking in the code I can see this is not supported... This is indeed a bug. Nice catch.

do you have plan how to fix this issue ?=)

Well I think this is rather straightforward in here. I am very busy at doing other things these days...
If you can't wait, I'll certainly welcome a PR on go-openapi/analysis.

Any updates on this issue? @fredbi any chance you are less busy these days?

@scevallos yes doing this now


Add commands

Add tp command
Add locatebiome command
Add structures to locate command
Add spectate command
Add loot command

Great work! I personally have started to move a way from atom, so if your interested I could hand over the repos to you^^

I don't know enough about Atom to continue developing the extension. I know the JSON format well because I make datapacks and I have that the CSON was quite close to it. That's why I did the PR

Haha, same tbh, except I'm more pron to use cson then json. I really have no idea how this actulay works, I just forked the now archived "mcfunction".  Anyways thanks again^^


LL-2776 Backport device desktop flows into mobile

Type

Context

Parts of the app affected / Test plan

TODOs

[x] DeviceAction/rendering renderBootloaderStep
[x] DeviceAction/rendering renderLoading with deviceStreamingProgress
[x] style closer to current modal
[x] adopt device animation depends on the connection type (cable vs bluetooth)
[x] fix type errors on DeviceAction/index.js
[x] lock navigation while device action
[x] prevent The connection to the device was lost... error flash
~[ ] format file names with step num~
[x] change step num in header accordingly

Device Action Flows
Here's the result of grepping by good old "DeviceJobs".

[x] cosmos/ClaimRewardsFlow
[x] cosmos/DelegationFlow
[x] cosmos/RedelegationFlow
[x] cosmos/UndelegationFlow
[x] tezos/DelegationFlow
[x] tron/VotingFlow
~[ ] EditDeviceName~
~[ ] RepairDevice ? not sure about this. Do we even have the repair feature on mobile?~
[x] AddAccounts
[x] ClaimRewards
[x] Exchange/CoinifyWidget
[x] Exchange
[x] FreezeFunds
[x] MigrateAccounts
~[ ] Onboarding/steps/pair-new~
[x] ReceiveFunds
[x] SendFunds
[x] UnfreezeFunds
[x] Revert Manager (actions/manager isn't ready yet for mobile)

Probably https://github.com/LedgerHQ/ledger-live-mobile/pull/1328/commits/c843a8f3978e7969f2207ddb272456837c68c5b5 is already handled by this logic. We need to check and revert if necessary. IMO, we only need to lock navigation when device is prompting for user confirmation.

Oops, I forgot about converting Algorand flows.

[ ] algorand/Rewards/ClaimRewardsFlow/index.js
[ ] algorand/OptInFlow/index.js

@LFBarreto can you help on this PR? If possible for algorand but also for the bridge.receive usage?

[ ] Adopt design


cobalt-0.0.5.pom xml error for Gradle but not for Maven

This
This page contains the following errors:
error on line 1 at column 7: XML declaration allowed only at the start of the document
Below is a rendering of the page up to the first error.

what one can see at https://repo1.maven.org/maven2/com/github/auties00/cobalt/0.0.5/cobalt-0.0.5.pom
Something like that one can see attempting to build on Gradle as part of Java project, or part of Android project. For example in Android mobile app:
> Task :prepareKotlinBuildScriptModel UP-TO-DATE
[Fatal Error] cobalt-0.0.5.pom:3:1: Content is not allowed in prolog.
[Fatal Error] cobalt-0.0.5.pom:3:1: Content is not allowed in prolog.
[Fatal Error] cobalt-0.0.5.pom:3:1: Content is not allowed in prolog.
[Fatal Error] cobalt-0.0.5.pom:3:1: Content is not allowed in prolog.
[Fatal Error] cobalt-0.0.5.pom:3:1: Content is not allowed in prolog.

BUILD SUCCESSFUL in 1s

 <?xml version="1.0" encoding="UTF-8"?>
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
    <modelVersion>4.0.0</modelVersion>
    <groupId>com.github.auties00</groupId>
    <artifactId>cobalt</artifactId>
    <version>0.0.5</version>
    <name>${project.groupId}:${project.artifactId}</name>
    <description>Standalone fully-featured Whatsapp Web API for Java and Kotlin</description>
    <url>https://github.com/Auties00/Cobalt</url>

    <developers>
        <developer>
            <name>Alessandro Autiero</name>
<EMAIL_ADDRESS>        </developer>
    </developers>

    <licenses>
...

The essence is that there is a space character in the first line of pom. It is incorrect. Can you remove it

Patched on #477, clone this repo and run:
sh mvn clean install -DskipTests=true

When will this be updated on maven?

When will this be updated on maven? Would be great to be able to use this with Gradle without any problems

Idk, maybe on 0.0.6

You can use doing what I said and setting mavenLocal() on the top of your repos

Patched on #477, clone this repo and run: mvn clean install -DskipTests=true

That works, thanks

fixed in 0.0.6


ST-1507-refactoring

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/ST-1507
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ x] No

Serkan did you change backend as well?

Serkan did you change backend as well?

yes

Serkan did you change backend as well?

yes


Use libc where possible and fs2 to lock file

Fixes #24.

Thanks for your contribution, Bastian.
Can you please take a look at failed checks?

I do not know what the problem is. Maybe you can figure it out. If not, feel free to close this PR. The commits f09bad6 and 43302d1 should be usable without getting the error.


added comments for encode method

Comments added to ParameterEncoder.swift file encode method

This documentation should be inherited from the protocol itself and so doesn't need to be duplicated at each implementation point.


TCP connections, request rate and ulimit

Would it possible to limit the TCP connections with Vegeta and still achieve a desired request rate by decreasing the ulimit?

 # allow 2 telnet connections per client host
 iptables -A INPUT  -p  tcp  --syn  --dport  23  -m
connlimit --connlimit-above 2 -j REJECT


UI Enhancements for Color Mode Management

Summary
This Pull Request improves the UI with a focus on color mode management, as noted in [Auto Dark/Light mode #91].
Changes Made

Color Mode Component

Added a Vue component for color mode management.
Improved code readability.

Dark Mode Support

Enhanced styling for dark mode.
Improved accessibility.

Code Cleanup

Streamlined the codebase for color mode selection.

Refactor Color Mode Selection

Changed color mode selection to a button for toggling modes.
Added @vueuse/components as a new dependency.

Screenshots
Light Mode View

Dark Mode View

Impact
These changes enhance color mode support and improve the overall user experience.

Thanks for the PR!


Etap 1

[ ] text is in C#
[ ] build succeeded
[ ] all UT are green
[ ] Data layer is clearly stated using language terms only (no database, no file)
[ ] Data API is clearly stated
[ ] Data API is abstract
[ ] Logic layer is clearly stated using language terms only
[ ] Logic API is clearly stated
[ ] Logic uses only the abstract Data layer API
[ ] Presentation layer is clearly stated using language terms only
[ ] Presentation uses only the abstract Logic layer API
[ ] MVVM and XAMLare applied to implement GUI
[ ] reactive and interactive user interaction for user (operator) communication
[ ] reactive and interactive programming is used for the communication of layers
[ ] Presentation - object model representing process data
[ ] Fulfill functional requirements of the task
[ ] Unit Test - layers are tested independently using abstract API
[ ] Dependency injection (additional framework is not required)
[ ] Mock is used for testing purposes (expected but not required)

🙆🏼♂


Files in MEI_DIRECTORY do not auto populate the document list.

Perhaps intended, but needs further exploration.

It does actually, its just an upload issue as in #105


[GHSA-vj3f-3286-r4pf] Path Traversal in Docker

Updates

Affected products
Severity
Source code location

Comments
github.com/moby/moby is not a valid Go package; github.com/docker/docker is still the canonical name.

👋 This pull request has been marked as stale because it has been open with no activity.  If you do nothing this pull request will be closed eventually by the stale bot. Please see CONTRIBUTING.md for more policy details.

I am unable to add the Keep label, but this PR is still "fresh" and I would like to merge it.


spirv-val: Multiple interface var with same SC

This was added in https://github.com/KhronosGroup/SPIRV-Tools/pull/4969 but then removed for what I pointed out was a DXC and not an issue with this validation (if anything, this caught a bug in DXC which means it is doing it's job in spirv-val)

@cassiebeckley you reverted the previous incarnation of this PR. Is DXC ok with this validation? It sounds like a DXC bug as Spencer said.

(dxc tracking bug: https://github.com/microsoft/DirectXShaderCompiler/issues/6265)

If the DXC fix won't take long I don't mind waiting to merge this.

I'm also in no rush to get this merged, I just noticed this was reverted awhile ago and there was never any tracking to get it back into spirv-val

The DXC fix has been merged


DEV: Integrate chat with experimental user menu

This PR adds support for the chat_mention and chat_invitation notifications type in the experimental user menu. A future PR will look at adding tests and support for chat_message and chat_quoted.

@OsamaSayegh Can you review this? Thank you!

Thank you @OsamaSayegh


Robertson-Walker flat metric

Implement detailed features for expanding universe metric and simulate geodesics for the same. Along with geodesics calculation, show signature plots for energy shifts.

#42 Addressed this!

Still some more efforts are needed, so keeping it open!

More theoretical clarity needed. Temporarily moved files to bin. Would come back after sufficient knowledge.
See #43

I believe this issue will also be tackled in the next release. More clarity is needed.


谁成功在jetson上使用了 flash_attn

我刚才在我jetson上成功编译了这个库，但是运行llm推理的时候，报错
RuntimeError: CUDA error: no kernel image is available for execution on the device
CUDA kernel errors might be asynchronously reported at some other API call, so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
Compile with TORCH_USE_CUDA_DSA to enable device-side assertions.
jetpack：6
cuda：12.2
python：3.10
torch：用的英伟达的轮子torch-2.3.0a0+6ddf5cf85e.nv24.04.14026654-cp310-cp310-linux_aarch64.whl

Yes, I did. If you still want to know solution, you can leave a message. Maybe I will write a blog sometime later.

Yes, I did. If you still want to know solution, you can leave a message. Maybe I will write a blog sometime later.

Yes, please, I ran into the same issue

Yes, I did. If you still want to know solution, you can leave a message. Maybe I will write a blog sometime later.

Yes, please, I ran into the same issue

OK, I will do it ASAP, however, since it is written in Chinese, maybe you need use translator to read it.

Yes, I did. If you still want to know solution, you can leave a message. Maybe I will write a blog sometime later.

Yes, please, I ran into the same issue

OK, I will do it ASAP, however, since it is written in Chinese, maybe you need use translator to read it.
PS: OK, I read your bio. Obviously, you can read Chinese

Thank you, I am Chinese myself haha, I'll try to translate your blog into English for others to reference

Having the same issue.

I wrote the approach here install vllm on jetson, and there is a chapter about how to install vllm-flash-attn on Jetson. Although the name is different, but vllm-flash-attn is just the fork of flash-attn, they share almost all source code.

The blog article is written in Chinese, translation is welcomed.

The blog article is written in Chinese, translation is welcomed.

I note that there is no info about jetpack,cuda,python version.
Would you mind supply these info.
Cuz I found a lot of problems especailly when compile things about cuda according to your blog.

JetPack 6, with the built-in CUDA 12.2. And the torch is 2.3.0, you need build it yourself.

您好，请问您博客中的方法，适用于jetpack6.1 CUDA12.6吗


Install fails: No targets specified and no makefile found. Stop.

Hi there,
just tried your script but the installation fails:
[root@host src]# sh opentracker/compile.sh
Installing opentracker with libowfat 0.29 ...
Updating package sources ...
Installing build dependencies ...
Reading package lists... Done
Building dependency tree
Reading state information... Done
build-essential is already the newest version.
bzip2 is already the newest version.
git is already the newest version.
0 upgraded, 0 newly installed, 0 to remove and 51 not upgraded.
Already up-to-date.
make: *** No targets specified and no makefile found.  Stop.

Cheers

Hi, this was not the first run, right? Could you dump an ls -l of the checked out opentracker directory? Obviously the makefile was not found. Thanks for reporting! :+1:

Hi,
here's the whole output: http://paste.calypso.pm/oVXfZuPZ
ls -la returns:
[root@calypso opentracker]# ls -la
total 48K
drwxr-sr-x 3 root 4.0K Feb  5 11:05 ./
drwxrwsr-x 4 root 4.0K Feb  5 11:05 ../
-rw-r--r-- 1 root 6.1K Feb  5 11:05 compile.sh
-rw-r--r-- 1 root  221 Feb  5 11:05 .editorconfig
drwxr-sr-x 8 root 4.0K Feb  5 11:05 .git/
-rw-r--r-- 1 root   19 Feb  5 11:05 .gitattributes
-rw-r--r-- 1 root    6 Feb  5 11:05 .gitignore
-rw-r--r-- 1 root 5.3K Feb  5 11:05 opentracker
-rw-r--r-- 1 root  395 Feb  5 11:05 README.md
-rw-r--r-- 1 root 1.2K Feb  5 11:05 UNLICENSE

Do you have my repository in /usr/local/src? I think that this is the problem. I'll rename my repository to opentracker-compile in order to avoid this problem. Could you try again after renameing it as well if you have it in /usr/local/src?
It reports at the end that the repository is up to date (of course, since you just cloned it) and that it couldn't find a Makefile (of course, since my repository does not contain one).

I did indeed clone opentracker to /usr/local/src. I've now move to repo to opentracker-compilebut I'm still running into some issues:
http://paste.calypso.pm/gE9F5Qrw
When I do service opentracker startafter I've created a config file, nothing happens. Seems that the command just loops forever.

That was my fault with the last commit. This should fix all your issues together with the renamed repository.

Still failing (hangs at [FAIL] opentracker not found! ... failed! install -m 755 opentracker /usr/local/sbin
http://paste.calypso.pm/i3xal5Lb

Okay, that one is interesting. I have to set up a new Linux and debug it. I'll get back to you asap.

No worries, thank you for your commitment to open-source software :+1:

It was actually a false positive because I try to stop the service before I attempt to install, the shell redirect was only catching the STDERR output, but the script was actually printing to STDOUT.
I tested it on a very limited and old Debian Linux, it compiled like a charm after I added zlib1g-dev to the build dependencies and removed the -m64 flag from the compiler options (now added if the OS reports to be x64).
Thank you for reporting the issues. It makes me happy to see that people can benefit from what I do. :+1:

Well, bummer:
http://paste.calypso.pm/SbRxqE59
Host system is running Wheezy 7.7 along with kernel 3.16.0-0.bpo.4-amd64.
Sorry for all the trouble.

No, don't be! It's actually very nice of you to keep testing and coming back, despite my incompetence to test correctly. I really should invest some time into learning some shunit2 scripting.

It's actually very nice of you to keep testing and coming back, [...]
Well that's what open-source software is all about, isn't it? :smile:
Name collision is now fixed but compile.shhangs at ```install -m 755 opentracker /usr/local/sbin````

Cheers.

That is because it is already installed and running, the file at /usr/local/sbin/opentracker is busy. Execute the following command ps aux | grep opentracker, this outputs the following on my VirtualBox:
nobody	2442	0.0	0.2	26944	872	?	S1	17:35	0:00	/usr/local/sbin/opentracker -f /etc/opentracker/opentracker.conf
root	2447	0.0	0.2	3560	808	tty1	S+	17:36	grep opentracker

The first line tells us that opentracker is already running. We could also issue service opentracker status which outputs [ ok ] opentracker is running. Now to stop it we simply issue service opentracker stop and to uninstall it from the system we do:
rm --force -- /usr/local/sbin/opentracker
# Delete init script
rm --force -- /etc/init.d/opentracker
# Delete configuration
rm --force --recursive -- /etc/opentracker

Now you have a totally clean system again and can compile a new one. I will add a check to the script.

Just realized that I already have that situation covered, the script does not hang on my side, even if opentracker is already running. Hmmm …

What happens if you go to /usr/local/src/opentracker and issue make install yourself?
The line in the Makefile of opentracker only contains:
install:
        install -m 755 opentracker $(BINDIR)

I cannot imagine how this command might hang. You could also try to issue that command by yourself: install -m 755 opentracker /usr/local/sbin

Thank you very much, worked like a charm this time. :grin:

Great! :+1: Open a new issue if you have any problems or questions.


Need option to mount tabs that are not active

I have a use case where it would be more convenient if hidden tabs were mounted (but still hidden). Some of the tabs we're showing make AJAX calls to fetch data and it's expected that users will be switching between the tabs. So having the tabs mounted and hidden makes the UX better as the user can immediately see what they are expecting when they click on a tab.
I feel the default should be the current behavior. If there is agreement on this feature and it's config I can put together a pull request.
I think it should be configured on Tabs and not an individual Tab. I can think of two ways to configure this:

hideMode: PropTypes.oneOf(["display", "unmounted"]). default: "unmounted"
mountHidden: PropTypes.bool. default: false

What do you think?

In my opinion having the choice is (almost) always better. But I think that what you want to do is perfectly doable from outside the tabs component. You can have a container wrapping the Tabs that perform that operation for you and saves the state into an external component. I would love more opinions on this because the same issue may happen for other components since in most cases we are trying to move from hidden to unmounted...

@javivelasco I too would have preferred if unmounted worked for my case. But we have at least one case that would still feel slow: An IFrame in a tab. Am I right in thinking that since it's removed from DOM it feel be another request...
In addition it feels like a high bar to make all components that should be rendered in a tab stateless (i.e. state is managed by the container that wraps the Tabs). We would have to refactor a lot of components that "just work" otherwise.

This relates to #739 There are some components that are totally stateless and forces you to create wrappers. The Input with label is an example. I think it's a better approach to wrap them with an HOC like recompose does to add state that get's synchronized with the props. This allows to make handling the state totally optional.
This work should be taken in both directions: making components that are stateless optionally stateful and viceversa. In any case if adding a prop to switch unmounted/mounted works for you I'd have no problem accepting a PR so feel free to do so

Just saw this issue. I think that if we separate the actual tabs from the content that should be rendered when the tabs are active this issue is gone, since everyone can handle the tab content as they see fit depending on their particular use case.
Adding more options to the existing tab component only makes it more complicated, where I would like to see us move in the opposite direction (make the tab component simpler and more flexible by separating concerns). Please comment in #739 if you have thoughts on the matter (either against or in favor of such a refactor).


Width of selections in search dropdown varies when scrolling through list

When using the up or down arrow to move through the dropped down list in a search dropdown, the width of the list varies with each item as it gets focus.  This is distracting to users.  The width for all items in the dropdown menu should be the same when they get focus.

I just updated to 2.2.13 and this issue seems to have gone away.  @Banandrew Can you confirm that this has been fixed?

@DanAtShenTech No, it’s still there, there were no commits to address it.


Add tests for string and omitempty tags, and fix issues found

This fixes:

https://github.com/pquerna/ffjson/issues/76  (Omitempty on pointer to value should only omit if not nil)
https://github.com/pquerna/ffjson/issues/81 (Different encoding of value)
https://github.com/pquerna/ffjson/issues/75 (Pointer to string with tag "string" has a closing quotation.)

It adds (disabled) test for these issues:

https://github.com/pquerna/ffjson/issues/82 (Decoder: Attempting to decode null value)
https://github.com/pquerna/ffjson/issues/80 (Support "string" tag in decoder)

Sorry about the big blob, but keeping tests in sync was important for me.

Interesting - it seems like "string" tag was changed in Golang 1.3 -> 1.4 to also include pointers.
This PR is equivalent to how 1.4 behaves. How should we proceed?

1.4 behavior going forward makes sense for me.  Maybe we add a test.Skip() for go 1.3?

ok, changed test to exclude go 1.2 and 1.3 - go version is a bit tricky, but this should catch most cases.

huh, go version is not as nice as i would of hoped. bleh.


[cinder-csi-plugin] Add doc for volume expansion

What this PR does / why we need it:
In the past I saw this question bubbling up several times and it is hard
to find an good answer on the internet.

Which issue this PR fixes(if applicable):
fixes #
Special notes for reviewers:

Release note:

NONE

/lgtm
/approve


ERC-721 Metadata hook

Closes #79
Introduced changes

Changed token_uri() mutability to view
Created ERC721MetadataHooksTrait containting single function custom_uri()
Calling custom_uri() internally by erc721_metadata_component::token_uri().
Working implementation:
https://github.com/rsodre/512karat/blob/main/dojo/src/systems/karat_token.cairo#L271-L287

Breaking changes
Contracts that do not implement this hook are required to import ERC721MetadataHooksEmptyImpl.
use token::components::token::erc721::erc721_metadata::erc721_metadata_component;
use token::components::token::erc721::erc721_metadata_hooks::ERC721MetadataHooksEmptyImpl;

Checklist

[x] Linked relevant issue
[ ] Updated relevant documentation
[x] Updated relevant presets
[x] Added relevant tests
[ ] Add a dedicated CI job for new examples
[x] Performed self-review of the code

@notV4l , what do you think about this?
custom uris are necessary.
if you have a better alternative let me know.

Origami tokens were removed, this PR is no longer needed.


fslit seems to no longer work very well

Looking at https://fstarlang.github.io/lowstar/html/LowStar.html#some-valid-low-constructs it appears that code blocks for F* are no longer rendered properly
Thanks,
Jonathan

Do you know what changed (a sphinx update maybe?)

no we pinned sphinx to the previous version -- I'm observing this with my local machine:
jonathan@absinthe:~/Code/mitls-fstar (no_hsl) $ sphinx-build --version
sphinx-build 2.4.3

and also on the CI build
jonathan@absinthe:~/Code/everest-ci (master) $ git grep sphinx
server-infra/linux/.docker/Dockerfile:#Install sphinx (for the Low* tutorial)
server-infra/linux/.docker/Dockerfile:RUN pip3 install sphinx==1.7.2 sphinx_rtd_theme

Maybe sphinx_rtd_theme, then? Can you try pinning it to a previous release and seeing if that still happens?

indeed, it looks like sphinx_rtd_theme now slaps white-space: nowrap on  HTML tags
any way you could skip the  tag? right now the structure is

stuff
over
multiple
lines but with white-space nowrap

the
, per sphinx-rtd-theme, has the right CSS styling so just getting rid of the extra  would likely be enough

actually, is there any way that fslit could generate the equivalent of

.. code-block:: fstar
  <contents of the F* code here>

? I'm enjoying syntax higlighting via pygments for code-blocks I manually wrote, but the fslit-generated code blocks don't seem to be enjoying this formatting

Sorry, I missed your messages.  I'd use a custom stylesheet to work around this problem; let me know if you need help writing it.

actually, is there any way that fslit could generate the equivalent of …

Yes, definitely; the reason they are not currently highlighted is that they're intended to be highlighted in js, along with running fstar.js.


pre-commit-ci fails when automatically created CHANGELOG PRs are merged

In repos where auto is set up for auto-releasing and pre-commit-ci is enabled, when a release is triggered (by a PR merge), another PR is automatically created which updates the changelog. This PR is then merged automatically, meaning pre-commit-ci does not have time to run on it. We think this is what's causing it to fail upon the merge to main (e.g., https://results.pre-commit.ci/run/github/567874792/1713298808.705SJF3VSvec-P8euGQopw).
One potential solution is to configure the auto-release workflow to include in the commit message for the changelog something like the following (from pre-commit-ci logs) to skip pre-commit CI:

skipping push runs: skip a run by putting [skip ci], [ci skip], [skip pre-commit.ci], or [pre-commit.ci skip] in the commit message.

See also: https://intuit.github.io/auto/docs/generated/changelog

Re-opening as adding single additional skip keyword [skip pre-commit.ci] did not do the trick.
Will try updating commit message to include all skip keywords.


docker-proxy does not die / hangs around / blocks port

I killed/stopped one of my containers and could not restart because its port had been taken.  It happened for Jenkins running inside Tomcat 7.0.50 on Ubuntu 14.04.1
As it turned out, docker-proxy did not die or go away when I docker killed/stopped jenkins.
docker-proxy -proto tcp -host-ip <IP_ADDRESS> -host-port 65150 -container-ip <IP_ADDRESS> -container-port 8080
Docker Info
Client version: 1.2.0
Client API version: 1.14
Go version (client): go1.3.1
Git commit (client): fa7b24f
OS/Arch (client): linux/amd64
Server version: 1.2.0
Server API version: 1.14
Go version (server): go1.3.1
Git commit (server): fa7b24f

@coderlol this should be fixed now, please ping me if you are able to reproduce on latest version and the exact steps for reproducing and i will be happy to reopen

Still happening with 1.6.0 on CentOS 7.

Seeing this on CentOS 7 as well, even after upgrading docker to version 1.6.2.

Can anyone confirm this has been fixed? And what is the fix version of docker?

I think I've just bumped into the same bug.
After a computer restart, I can't start any docker container with port bindings (it says that the port is already allocated). So the situation looks as follows:
docker ps -a doesn't show any container BUT the docker-proxy process already blocks the ports specified in the containers I want to run (8080, 8998).
I'm just curious how to reset docker's state so it frees all ports?
Neither system reboot or service docker restart helps.
OS: KUbuntu 14.04,
docker -D info output:

Containers: 0
Running: 0
Paused: 0
Stopped: 0
Images: 167
Server Version: 1.12.0
Storage Driver: aufs
Root Dir: /var/lib/docker/aufs
Backing Filesystem: extfs
Dirs: 178
Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
Volume: local
Network: null host overlay bridge
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: apparmor
Kernel Version: 4.2.0-42-generic
Operating System: Ubuntu 14.04.5 LTS
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 15.57 GiB
Name: nekrasov-ubuntu
ID: G5B2:YAU2:DPBU:DEK2:JG6Y:GXAY:P5MJ:YK3O:WQMB:CU4D:GM7Z:FKBS
Docker Root Dir: /var/lib/docker
Debug Mode (client): true
Debug Mode (server): false
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Insecure Registries:
<IP_ADDRESS>/8

Curious, after disabling userland proxy, the port is still blocked (but now it is owned by dockerd process instead of docker-proxy):
$ sudo netstat -tulpn | grep 8998
tcp6       0      0 :::8998                 :::*                    LISTEN      4153/dockerd

So the question remains: how to tell docker to free those ports?

We met the same problem. Even if restart the docker daemon.
To fix this, stop the docker daemon and remove /var/lib/docker/network/files/local-kv.db
Then restart daemon again.
This problem comes out very often. Please fix it ASAP.

We ran into the same issue today after starting and stopping a complex docker-compose application.
After removing all containers and stopping docker the docker-proxy taking the port was gone but after restarting docker (still with no containers - docker ps -a is empty) the docker-proxy process has been restarted.
After removing the network which has been created by docker-compose the proxy was gone and we managed to restart the application.
docker 1.12.0, ubuntu 16.04

I have the same issue. Please fix.

Same issue.
Running docker multicontainer docker (AWS Beanstalk) latest 1.11.2.
In AWS context:
Happens when you want to e.g. mount EFS instances, because you have to restart docker after mounting. And when you run docker restart, the ports become locked to the previous docker instance. So bascially the fixes I am using now is:
commands:
  # Here i do mounts etc ...
  # ...
  06stopdocker:
    command: "service docker stop"
  07killallnetworkbindings:
    command: 'sudo killall docker  > /dev/null 2>&1 || /bin/true'
  08removenetworkinterface:
    command: "rm -f /var/lib/docker/network/files/local-kv.db"
    test: test -f /var/lib/docker/network/files/local-kv.db
  09restart:
    command: "service docker start && start ecs"

Same issue with docker 1.12.1 on ArchLinux
Thanks @himulawang for the fix

@jfrazelle can you please open again this issue?

Please can you open a new issue, and give details of how to reproduce, (and link here). A lot of code has changed since the original issue, it is better to open a new one.

Same issue here. Stopped a container but docker-proxy didn't release the port. Killed the relevant docker-proxy process, netstat says the port is free now but can't start the container nevertheless:

$ docker start mycontainer && docker logs -f mycontainer
Error response from daemon: driver failed programming external connectivity on endpoint mycontainer
(9a1493d....2227d65): Bind for <IP_ADDRESS>:443 failed: port is already allocated
Error: failed to start containers: mycontainer

This is clearly a lie as the port is not allocated.

I just reproduced this on Ubuntu 16.04.1 LTS with kernel 4.4.0-47-generic #68-Ubuntu and Docker version 1.12.2, build bb80604. There were no containers at all, and the docker-proxy processes reappeared after shutting down Docker and restarting it. Wish I had more data, but I wanted to confirm that this could be reproduced.

This issue still happens for us, mainly during continuous integration were we spin up/down test containers regularly on the same Vms. Currently happening on Ubuntu 14.0.4 with a recent docker-ce version
jenkins@ip-10-0-12-99:~/workspace/dockerdebug$ docker version
Client:
Version:      17.03.1-ce
API version:  1.27
Go version:   go1.7.5
Git commit:   c6d412e
Built:        Fri Mar 24 00:40:33 2017
OS/Arch:      linux/amd64
Server:
Version:      17.03.1-ce
API version:  1.27 (minimum version 1.12)
Go version:   go1.7.5
Git commit:   c6d412e
Built:        Fri Mar 24 00:40:33 2017
OS/Arch:      linux/amd64
Experimental: false

@amalaowala please open a new issue with details and steps to reproduce (e.g. how is docker started etc)

I am locking the conversation on this issue, because the original issue reported here was resolved over two years ago; many things have changed and any issue encountered with current releases is very unlikely related to the original issue reported here.


Bump copyright year to 2022 in constants.ts

Thanks David.
Does this PR "overrule" the previous one on LICENSE  #934 ?
So can I close or delete  #934

Codecov Report

Merging #935 (4622c56) into master (e767bdf) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #935   +/-   ##
=======================================
  Coverage   76.96%   76.96%
=======================================
  Files         135      135
  Lines        3486     3486
  Branches      634      634
=======================================
  Hits         2683     2683
  Misses        796      796
  Partials        7        7

Impacted Files
Coverage Δ

src/app/constants.ts
100.00% <100.00%> (ø)

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 e767bdf...4622c56. Read the comment docs.

Does this PR "overrule" the previous one on LICENSE #934 ? So can I close or delete #934

No, both need to be updated.

Thanks David


Fix spelling typo

Thanks for the contribution!

Not sure why I missed seeing this PR for so long. Sorry for the delay merging it.


mnist datasets error

Hi,
I am trying to run this code(using summerschool2015's ipython notebook)
from fuel.datasets import MNIST
train = MNIST( which_sets=('train',), subset=slice(0, 50000))
It give following error,
TypeError                                 Traceback (most recent call last)
 in ()
2
3
----> 4 train = MNIST( which_sets=('train',), subset=slice(0, 50000))
5 valid = MNIST(which_sets=('train',), subset=slice(50000, 60000))
6 test = MNIST(which_sets=('test',))
TypeError: init() takes exactly 2 arguments (1 given)
I have tried various other way with no luck..Any idea about this error? I have mnist.hdf5 inside fuel_data directory..

Updated:
I just restarted the ipython notebook , deleted the fuel-data folder and freshly created from the notebook and it works again :) . Not sure what was the issue!!

Glad to see the issue is solved! By the way, you can always ask these sorts of questions on the mailing list. This way you won't have to to open an issue on Github.


Support SoM/ClassicEra

[x] Build ability GSheet
[x] Update Build.py for BCC+SoM/ClassicEra
[x] Support multi-toc
[x] Only load expansion-specific ability data

Foundation added in #22

Nice, I'll get the reviewed data over to you soon!

#26 should have finished support for SoM.
Will be released with v1.2.0

So 1.13.0/1 had spellID in combat log, then 1.13.2 removed it. https://wowpedia.fandom.com/wiki/Patch_1.13.2/API_changes

The Combat Log no longer has access to Spell IDs, they are always returned as 0, since they were only added to the combat log in patch 2.4
UNIT_SPELLCAST_* events only fire for the player.

https://wowpedia.fandom.com/wiki/COMBAT_LOG_EVENT

Spell IDs are always 0 on classic_era.

Trimmed functionality for SoM, using https://wowpedia.fandom.com/wiki/UNIT_SPELLCAST_SUCCEEDED #27


Validate that Project URLs are actually URLs on Upload

As the title says, validate the project url metadata on upload, probably with rfc3986.

Grabbing this. PR incoming.


chore(main): release 1.1.0

:robot: I have created a release beep boop
1.1.0 (2022-05-09)
Features ✨

add main upload and default prop logic (d321943)
add the ability to just run with a fake url for test runs (bd00d8f)
adding glob finder to search for files in a path (b271e54)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/sekwah41/upload-release-asset/releases/tag/v1.1.0 :sunflower:


Update MD file on new branch 2017-25-04_03-12-06

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef.md
:white_check_mark:Succeeded

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


Looking to Add 'return' functionality to functions.

Just as the title says. Working on it through a local version downloaded from the repo. Don't know if opening an issue is the right way to do this, but that's what it says on the ReadMe, so that's what I did.

This is something that has been on my mind for a long time. Just can't think of a way to implement it. I will find a way eventually.

I'm close to having it working, if you're interested.
On Mon, 14 Mar 2022, 11:17 am TechStudent10, @.***>
wrote:

This is something that has been on my mind for a long time. Just can't
think of a way to implement it. I will find a way eventually.
—
Reply to this email directly, view it on GitHub
https://github.com/TechStudent11/Skate/issues/2#issuecomment-1066239493,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXUC2WGJ26X74ZZPV7Q37VTU72OT5ANCNFSM5QUAYE3Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Could you share your implementation, please?

If you do have an early (working) version, you could push it to your fork for me to read.

Closing due to inactivity.


uv_poll_init() doesn't work for kqueue fds

Versions affected: since 1.2.2 (reproduced on 1.9.0)
Platform: BSDs and OS X
Problem
uv_poll_init() enforces that socket is set to non-blocking mode, but ioctl is N/A for kqueue pollfd so it returns an error. This blocks nesting/embedding other event loops or polling mechanisms in libuv where kqueue is used.
Steps to reproduce

Create a kq = kqueue() pollfd
Create uv_poll_t and call int uv_poll_init(loop, &some_cb, kq)
Get ENOTTY (Inappropriate ioctl for device)

While we can fix that in libuv (and we probably should), I don't think it would help you much. You can't usefully nest kqueue file descriptors (on OS X anyway) because the nested fd is immediately reported as readable, regardless of whether there are actual events available.  It's not like epoll, where events bubble up.
It's possible Apple fixed that in the most recent XNU release but a quick look at bsd/kern/kern_event.c suggests they haven't.

It'd help me a lot. It is indeed reported as readable and that's fine as the nested loop finds out there are no events and yields. Next wakeup happens when there are actual events on the kqueue. Right now I had to hack it a little bit, as in - I have to open the uv_poll_t on a throwaway file descriptor and then switch it with the kqueue before uv_poll_start(...), but that requires fiddling with uv_poll_t internals. I don't feel great about it, but it works for now.

@bnoordhuis I guess we could ignore the error here: https://github.com/libuv/libuv/blob/v1.x/src/unix/poll.c#L62 kind-of setting the fd non-blocking best-effort.

Neat, thank you!


Upload django-elasticsearch to Pypi

Hi, there is no release yet because the goal was to have python3, es2 support and a few important functionalities first (term queries, multi indexing, maybe a "raw query" type of thing), but the package is not moving forward really fast right now so who knows when it will be ready.


Bug: node.setFormat and selection.formatText leave empty style and class attributes

Lexical version: v0.12.4
Bug 1
Steps to reproduce

Enter the text ‘abc’ in the editor:

  abc

Select the text ‘abc’ in the editor.
Set and delete the text-align CSS property of the p element:

const selection = $getSelection();
const spanNode = selection.anchor.getNode();
const pNode = $getNearestBlockElementAncestorOrThrow(spanNode);
pNode.setFormat('center');
pNode.setFormat('');

The current behavior
The style attribute of the p element is empty:

  abc

The expected behavior
The style attribute of the p element should not be present:

  abc

Bug 2
Steps to reproduce

Enter the text ‘abc’ in the editor:

abc

Select the text ‘abc’ in the editor.
Set and delete the text-decoration CSS property of the span element:

const selection = $getSelection();
selection.formatText('underline');
selection.formatText('underline');

The current behavior
The class attribute of the span element is empty:
abc

The expected behavior
The class attribute of the span element should not be present:
abc

Yea, we could probably change this, but I'm not sure it's a bug. What's the practical problem this is causing for you?


Add Changelogs

Add Changelogs and/or links to change logs to the PR description

https://github.com/dylang/changelog


The TypeScript language service died unexpectedly 5 times in the last 5 Minutes. Please consider to open a bug report.

I am repeatedly getting the following warning after installing 1.8:

The TypeScript language service died unexpectedly 5 times in the last 5 Minutes. Please consider to open a bug report.

VSCode Version: Code 1.8.0 (38746938a4ab94f2f57d9e1309c51fd6fb37553d, 2016-12-13T17:38:28.425Z)
OS Version: Darwin x64 15.6.0
Extensions:

Extension
Author
Version

vscode-eslint
dbaeumer
1.0.6

python
donjayamanne
0.3.21

text-transform
florianloch
0.1.0

Python-autopep8
himanoa
1.0.2

docthis
joelday
0.3.5

cpptools
ms-vscode
0.8.1

csharp
ms-vscode
1.4.1

autoimport
steoates
1.2.2

cordova-tools
vsmobile
1.2.4

Steps to Reproduce:

Start VS Code - warning appears after several seconds.

@chipaubry Can you please share your workspace so that we can investigate? Does this happen for all projects you open or just one?

@mjbvz Unfortunately I can't share the workspace. It appears to have been an issue with OSX because after a reboot I am no longer seeing the behavior. I didn't make any changes to the workspace.

@chipaubry Ok. Closing since the issue seem to have been resolved.
Please let us know if you see this issue again so that we can investigate.
Thanks


Gap between right side segments using nerd-fonts

Hi,
I'm using nerd-fonts, and this config:
POWERLEVEL9K_MODE="nerdfont-fontconfig"
POWERLEVEL9K_LEFT_PROMPT_ELEMENTS=(os_icon ssh dir vcs )
POWERLEVEL9K_RIGHT_PROMPT_ELEMENTS=(vi_mode status history background_jobs time)
POWERLEVEL9K_VI_MODE_INSERT_FOREGROUND='teal'
POWERLEVEL9K_TIME_FORMAT="%D{%H:%M \uF073  %d.%m.%y}"
I see one-pixel gaps between the segments on the right, but not on the left:

I'm using 14pt Meslo LG M Regular for Powerline Nerd Font Complete in iTerm2 profile.
Problem goes away if I use regular Meslo Powerline version (but I lose the additional icons so..)
I'm not sure if this is a powerlevel9k issue, or a nerd-fonts issue, please let me know.
Tks.

Hmm. We have something like that on our troubleshooting wiki page, but I am not sure if that helps you.. I would say this is an issue in Nerdfonts (like here)

From my testing I'm only noticing this when using non-mono fonts, when using mono fonts it works as intended.

Looks like this is a known issue with nerd-fonts. https://github.com/ryanoasis/nerd-fonts/issues/26

Hi.  Thanks, I commented in https://github.com/ryanoasis/nerd-fonts/issues/26

Hello. I think the problem hasn't been fixed. I have the same issue: a gap of 1 pixel between arrows on the right prompt.
I'm running under Archlinux, I use urxvt with nerd fonts (I installed nerd-font-complete and use Hack Nerd Font Mono)
Seeking for a solution, It turns out that the AUR provides a package "rxvt-unicode-pixbuf" that fixes the issue (although it's not meant for it). I was wondering if that could come from urxvt ?


How to test high resolution imgs？

I want to train a matting model by my own dataset. It will be used in high resolution imgs.
Why is it not clear. How can I train my model to use in high resolution imgs.
By the way, why the img size use 640?

The image size is not set to be 640 and I think there is no limitation on the training image size. You can upload the error information you encountered.

Sorry. I mean，How to train a model for high resolution imgs. I  want to use model for 2k resolution imgs

If you want to train on 2k images without any croping or resizing, it is a little difficult. In the training phase each 2k image may require more than 10G GPU memory.
You can try to edit this line or codes in dataloader.
https://github.com/Yaoyi-Li/GCA-Matting/blob/5a999be4b00bb482658fa7013a74a2b8acddc78e/utils/config.py#L45

@Yaoyi-Li  what is your recommended size for the images resolution as input to your network during training? please give me a range that you recommend, what is in your view the ideal size of images to input to your gca training, or the one that you used or that you recommend, thank you very much :)

@Yaoyi-Li  what is your recommended size for the images resolution as input to your network during training? please give me a range that you recommend, what is in your view the ideal size of images to input to your gca training, or the one that you used or that you recommend, thank you very much :)


Increase tserver ingest memory buffer for SummaryIT

Fixes #4021

@keith-turner  - if/when you look at this, please look at the comments in #4021 . This fixes the test, but there is a behavior difference, minc starting earlier I think. I looked at the code in LargestFirstMemoryManager, and nothing appeared to change between main and elasticity.


added replace() function

added my implementation of the replace() function along with the test_str.py

Sure, I'll get working on that asap.

I have updated my code for a bug that I found. Also, I've added more test cases like you asked.

@AakashSasikumar As there is a merge conflict, could you rebase this on/merge in latest master branch please? I'm not too sure what's going on but there seem to be changes for rstrip/lstrip/strip included in the last few commits, which is a little unexpected.

sure thing

Why is the merge conflict still there? I've changed the test_str.py.

@AakashSasikumar could you rebase/merge with the latest master and update this PR. After that are there still missing tests?

Closing, as this was implemented in #501.


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/opstest2/docs-archive-test-source.git

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

File
Status
Preview URL
Details

:x:Error

Details

[Error: RepositoryUnauthorizedWithSourceCommit] Git data access to 'https://github.com/opstest2/docs-archive-test-target' (branch: Release_Archive_main_2022-02-21-02-03-40) is not authorized.. It looks like this might be the first time user: v-alji submitted a pull request to this repo. Before user can run builds in a pull request, he/she can enable 'v-alji' GitHub account in Docs by visiting Docs Portal (https://ops.microsoft.com). After enabling his/her account (v-alji), close and then reopen this pull request to start another build.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


The model for GReaT

Thanks for the great work! I am a newbie in NLP so this might be a silly question. I am wondering how to use the GPT-2 model for GReaT? Is it
model = GReaT(llm='gpt2', batch_size=32, epochs=50)

or are there any examples that run GReaT instead of Distill-GReaT?

Hey, thanks for your question!
Until now, we only used "distilgpt2" and "gpt2-medium" in our experiments.
But, you should be able to use all the pre-trained models from the HuggingFace Model Hub (https://huggingface.co/models) as large language model (llm) - also "gpt2".


DCorlib build fails [Win 7, DMD v2.069.2]

If I do what's suggested in these errors and few other changes lib builds but CsNativeVisual outputs 'gibberish' D code and there are many path errors because they are hardcoded in sources somewhere.
Error	4	Error: class System.Nullable__G.Nullable__G!(__UNBOUND).Nullable__G use of object.Object.opCmp(Object o) is hidden by Nullable__G; use 'alias opCmp = Object.opCmp;' to introduce base class overload set	D:\Dlang\SharpNative\DCorlib\System\Nullable__G.d	4
Error	5	Error: class System.Nullable__G.Nullable__G!(__UNBOUND).Nullable__G use of object.Object.opEquals(Object o) is hidden by Nullable__G; use 'alias opEquals = Object.opEquals;' to introduce base class overload set	D:\Dlang\SharpNative\DCorlib\System\Nullable__G.d	4
Error	2	Error: class System.Collections.Generic.InternalStringComparer.InternalStringComparer use of System.Collections.Generic.EqualityComparer__G.EqualityComparer__G!(String).EqualityComparer__G.GetHashCode(NObject obj, IEqualityComparer __j = null) is hidden by InternalStringComparer; use 'alias GetHashCode = EqualityComparer__G.GetHashCode;' to introduce base class overload set	D:\Dlang\SharpNative\DCorlib\System\Collections\Generic\InternalStringComparer.d	7
Error	3	Error: class System.Collections.Generic.InternalStringComparer.InternalStringComparer use of System.Collections.Generic.EqualityComparer__G.EqualityComparer__G!(String).EqualityComparer__G.Equals(NObject x, NObject y, IEqualityComparer __j = null) is hidden by InternalStringComparer; use 'alias Equals = EqualityComparer__G.Equals;' to introduce base class overload set	D:\Dlang\SharpNative\DCorlib\System\Collections\Generic\InternalStringComparer.d	7
Error	1	Error: class System.Collections.Generic.EqualityComparer__G.EqualityComparer__G!int.EqualityComparer__G use of System.NObject.NObject.Equals(int x, int y) is hidden by EqualityComparer__G; use 'alias Equals = NObject.Equals;' to introduce base class overload set	D:\Dlang\SharpNative\DCorlib\System\Collections\Generic\EqualityComparer__G.d	10

Hi wyrex, can i have a sample of the code you are trying to compile ? Seems the repo might be out of sync and the new changes ive made locally are incompatible. So i could either get you set up or you could wait till i release the first beta version ...

Also do note that quite a few things have changed in dmd so theres going to be alot of incompatibility ... could you zip and post the generated code ? Should be in %AppData%/Local/Temp/SharpNative

Those errors were what I got when trying to compile latest commit, most likely it broke badly when I modified it to build (without knowing what I was changing..), I'd send the output but laptop is in my second home where I won't be going in a week.


GregTech aluminium compatiblity issue

GregTech unifies Mariculture aluminium ingots to GregTech aluminium ingots.  For most other mods, the Unification config would allow me to change the unification target to a given mod's ingot type instead, but apparently Mariculture aluminium ingots aren't recognized as valid targets.  Also, even when I use Minetweaker to mirror ingotAluminium and ingotAluminum, GregTech aluminium ingots aren't usable in Mariculture recipes, even though GregTech titanium ingots and magnesium ingots are.  I can set up Minetweaker recipes for the items I'm interested in, but I shouldn't have to.

The standard set by the community for the spelling was ingotAluminum. Being british I spell it the other way, but that is what everyone settled on, so it's what I use. It's what I ask for in my recipes. Why greg decided to try and get everyone to use the other spelling in the ore dictionary is beyond me, considering how much he'd moan about people not using the ore dictionary correctly. It's not my fault he didn't want to follow the standard. Which causes this issue. And no you shouldn't have to. However it is still not my issue.

Then why didn't mirroring the ore dictionary entries, so that GT aluminium ingots show as both ingotAluminium and ingotAluminum (according to NEI Integration) work?

No clue. I've used other mods aluminium to make things before.

Some further experimentation, and apparently Minetweaker's "mirror" function doesn't work that well.  Neither
ore:ingotAluminum.mirror(ore:ingotAluminium);
nor
ore:ingotAluminium.mirror(ore:ingotAluminum);
worked, but
ore:ingotAluminum.addAll(ore:ingotAluminium);
allows me to use GT aluminium ingots in Mariculture recipes.


I need a way to calculate and show token counts for each message

I've tried the old tokenizer, it doesn't work with React Native. Because the old tokenizer depends on tiktoken and although the recent version of RN added support for TextEncoder, it still doesn't have TextDecoder with its default js engine Hermès. And tiktoken uses TextDecoder. I've tried polyfill with @bacons/text-decoder and it also failed with errors like: invalid escape characters...
so here I am. what are my options? I understand that I can get .usage from the response. but...how do I estimate the message generated by the user before sending to the API?

There is now a new API for getting token counts! https://docs.anthropic.com/en/api/messages-count-tokens


IDOC Satellite

Pull Request
Description
Add option for IODC satellite pull
Main changes are in app.py, other changes are small refactoring and small changes needed for IODC
Refactor use_backup to use_hr_serviri in many places
#279
Example of image -

and windy.com weather roughly at the same time

Q: do we see this white band in the satellite image that we trained on?
Im trying to work out the right balance of re-writing lots of things, and get this working. Have discussed with @devsjc about re-writing this consumer completely
How Has This Been Tested?
CI + extra tests

[x] Yes

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked my code and corrected any misspellings

So for reference this is what the IODC image looked like that we pulled from HF and what we cropped down to to get just India to train on

So for reference this is what the IODC image looked like that we pulled from HF (sorry not got a better screenshot of the one that shows India a bit more clearly on it) and what we cropped down to to get just India to train on

Thanks, that looks different enough, I think somethin may not be right with the PR, let me look in to it

So for reference this is what the IODC image looked like that we pulled from HF (sorry not got a better screenshot of the one that shows India a bit more clearly on it) and what we cropped down to to get just India to train on

Thanks, that looks different enough, I think somethin may not be right with the PR, let me look in to it

This is now sorted, thank you @Sukh-P for this input


INT B-19044 remix electric boogaloo

NOTE - THIS PR IS TWO PREVIOUS PRs addressing a BL item and an issue:
graph TD;
    B-19044-->INT-B-19044;
    B-19044-->INT-B-19044-I-13328;
    INT-B-19044-I-13328-->THIS_PR;
    INT-B-19044-->THIS_PR;

LINK TO FIRST INT PR
LINK TO SECOND INT PR
NOTE
Was failing client test code coverage even though all changes have updated tests. Went ahead and created a test file for ConfirmCustomerExpenseModal.jsx and added some tests since one did not exist.
Agility ticket
Summary
These are to make the counts on the left nav and top nav bar more useful to the office users. Daniel did the implementation here and had some back and forth with the gov via slack, if we need to reference it.

Counselor remarks are no longer included in the count for tasks
Actions that block the office user from processing are now red notifications
Other required actions are yellow

Testing AC 1:
GIVEN that I am a Service Counselor viewing "Shipments" on the "Move details" page
WHEN there are no counselor remarks,
THEN I will not be required to enter counselor remarks,
AND this item will not be included in the count of items that need my attention.

Create a PPM move as the customer, request an advance
Log in as the counselor
See that the counselor remarks no longer have a yellow mark and are not included in the total count

Testing AC 2:
GIVEN that I am a Service Counselor viewing "Shipments" on the "Move details" page
AND a customer has a PPM shipment,
WHEN the "Advance requested?" field has not been reviewed,
THEN I will see a red indicator that the item needs my attention,
AND this item is included in the count of items that need attention for the "Shipments" section in the left nav bar,
AND this item is included in the count of items that need attention for the "Move details" page in the top nav bar.

Use the move you just created as long as it has a requested advance (if you forgot, just make a new one)
Notice that since the Advance Rqeuested field has not been reviewed by the counselor, the Shipments left nav tag will have a red '1' by it.

Testing AC 3:
GIVEN that I am a TOO viewing "Shipments" on the "Move details" page
AND a customer has a PPM shipment that requires GHC Prime Counseling,
WHEN the "Advance requested?" field has not been reviewed,
THEN I will see a red indicator that the item needs my attention,
AND this item is included in the count of items that need attention for the "Shipments" section in the left nav bar,
AND this item is included in the count of items that need attention for the "Move details" page in the top nav bar.

Switch to the TOO role
Search for the same move you just looking at, by move code
Notice that since the Advance Rqeuested field has not been reviewed, and the Shipments left nav tag will have a red '1' by it.

Testing AC 4:
GIVEN that I am a Services Counselor,
WHEN I am viewing "Allowances" on the "Move details" page,
THEN I will not see orange indicators for Pro-gear, Spouse pro-gear, Required medical equipment, OCIE, and Gun Safe.

Log in as a services counselor
Confirm you don't see orange indicators on the Allowances section

Screenshots

Agility ticket
Issue ticket
Summary
Here we are again. You may be asking yourself, "Why, Daniel? Why are you doing this again?"
I admit, I do have a problem, but this issue ticket is not one that I created. It was confirmed by the gov via approving a screenshot that they didn't want unapproved shipment counts in the left tab nav, but wanted it in the top nav.
Then I made a huge mistake - I went on vacation thinking they wouldn't change their mind. I was so naive...
Anyway, this PR does the following:

removes unapproved shipment count entirely for services counselors
since this is not an action they need to complete, there should not be a flag for them at all
or at least until some minds are changed again - who knows?

How to test

Access MM as a services counselor or multi role
Find a move that has unapproved shipments
Confirm that the unapproved shipments do not show up as a count/flag for services counselors, but does for TOO

Screenshots
Move for SC with unapproved shipments

Same move for TOO that does show flags since shipments are not approved

Test coverage on client failed.
Last run: Mon 30 Sep 2024 02:45:50 PM UTC
Refer to this report to see coverage details for the client app.
Documentation for test coverage can be found at this link.

Warnings

:warning:
This PR does not include changes to storybook, even though it affects component code.

  Generated by :no_entry_sign: dangerJS against 1c7f5a9a8c55802593ef55619afed95532d95134