Ensure absence of panics on scripts >4GB and other consensus-invalid data

Bitcoin consensus encoding uses this idiotic thing named VarInt, which can have any value up to 64 bits and is used to encode length of all array data in consensus: number of transaction inputs, outputs, scripts, witnesses etc, etc. This is bad since none of these things in practice can have more than 2^16 items under the consensus rules - but still, an attacker can create a tx which contains octallions of elements in any of the consensus data...
This creates additional problems, since in RGB we'd like to ensure we can run a client-side-validation with clear restrictions on the memory, storage, bandwidth etc known beforehand - thus we have the strict type system and no collection of RGB data can have more than 64k elements or be larger than 16MB. But once we'd like to read Bitcoin tx from AluVM we instantly get into this trouble with octallions of bytes which may come from an attacker with the use of VarInt.
There are three ways of dealing with this situation:

not to care about it (like all Bitcoin wallets) and have an undefined behaviour under various forms of attacks, which one day may lead to global shutdowns like what happened to the lightning network because of the exact same issue;
assume that data must not be larger than a certain size and panic when they do, since it will indicate that they are not coming from Bitcoin consensus but from an attacker - this is what we do today, but it still allows an attacker to crash the software, which is not good;
still assume a size limit - but error and handle errors gracefully. This is the hardest way since it requires a lot of boilerplate code all around the place once we touch any data from Bitcoin - but I think this is inevitable.

This PR puts the foundations for such a thing into bp-consensus crate. Since it changes APIs (many functions will be returning Result now), it is an API-breaking change (but not RGB consensus-breaking) and has to go with a new v0.11 version

Now since we break APIs anyway with v0.11 I think it is time to merge this one as well. @zoedberg what do you think?


Zero Open Handlers in Test Suite

Issue
The test suite fails to mock RPC calls to Solana currently which results in open http handlers that slow down the test suite and prevent jest from exiting cleanly. This issue is tracking the progress of removing open handlers.
yarn test --detectOpenHandles
yarn run v1.19.2
$ jest --detectOpenHandles
 PASS  test/metadata.test.ts (8.52 s)
 PASS  test/metaplex.test.ts
 PASS  test/auction.test.ts
 PASS  test/storage.test.ts
 PASS  test/conversion.test.ts
 PASS  test/vault.test.ts

Test Suites: 2 skipped, 6 passed, 6 of 8 total
Tests:       7 skipped, 27 passed, 34 total
Snapshots:   0 total
Time:        20.482 s
Ran all test suites.

Jest has detected the following 7 open handles potentially keeping Jest from exiting:

https://github.com/metaplex/js/pull/47


GFI - 2 : Upgrade to Protostar 0.4.2 Context: coupled interact with some parts of the codebase

Describe the Feature Request

Once we have Declare V2 implemented from this https://github.com/onlydustxyz/marketplace-frontend/pull/744, we will need a way to store the Sierra program which is passed from the RPC call. Right now, the Sierra program isn't stored anywhere and so we aren't able to return it when the user calls get_class_at. When storing this mapping, we must do it in a way that it is communicated to all the other nodes.
Describe Preferred Solution
We can implement this in the run time but that should be our last option as the runtime should only have the information needed for the chain state. The Sierra program is not essential for the chain state and so we must store them elsewhere. The db in the client isn't gossiped between the nodes and is a local copy. So we need to explore how we can get this done.
If the feature request is approved, would you be willing to submit a PR?
(Help can be provided if you need assistance submitting a PR)

Yes

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
My motivations

I would like to work on this

Can I do it ?

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Can I do it ?

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
d
How I plan on tackling this issue
dd

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
tes
How I plan on tackling this issue
test

hello @pixelfact . Can i contribute to this project

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
yes

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
yes

I want to do this


refactor: backend refactoring

[x] use springboot
[ ] use mysql save data
[x] switch poi
[x] implement a minimum demo

make design doc


stack.yaml: bump resolver to lts-9.20

The failure looks like a flaky download, I'm going to go ahead and merge this.


Added tflint testing

List of changes:
Added TFLint testing and remake Checkov call in ICMK.
How to use:

use Makefile in /examples/simple folder
make terraform.checkov - to test infrastructure with Checkov
make terraform.tflint - to test infrastructure with TFLint
make terraform.test - to test infrastructure with Checkov & TFLint

Testing Done:
make terraform.checkov

make terraform.tflint

make terraform.test

LGTM


Vertex attribute binding mismatch for custom step rate/divisors

Hey, I'm having trouble getting custom step rates/divisors to work when using instanced rendering with MoltenVK version 1.0.43 from the Vulkan SDK version <IP_ADDRESS>.
In MVKGraphicsPipeline::addVertexInputToPipeline where pVertexInputDivisorState is handled getMetalBufferIndexForVertexAttributeBinding is called which ends up returning the Metal buffer index as maxPerStageBufferCount - binding (link). This means that the binding index gets "inverted" and the subsequent call to SPIRVToMSLConversionConfiguration::isVertexBufferUsed fails because the bindings (which for me are in the range 0, 1, ...) gets compared to the "inverted" bindings/indices (in the range 30, 29, ...).
If I skip that isVertexBufferUsed call and thus force-set the vbDesc.stepRate = pVKVB->divisor the rest works as expected. Otherwise the shader vertex data never gets "repeated" and only the first few rendered instances get the data.
So I basically don't understand the maxPerStageBufferCount - binding logic. This could just be me setting up the data incorrectly but everything else seems to work fine.

Does #1042 fix it for you?

That did indeed fix it, cheers!


Add first version of deployment on dev script

@stephan271 , @wakonig this would be the first version of the deployment script. Unfortunately, I've checked the git client hooks and the post-push does not exist.
This script might change (depending on what you need). For now, the idea is that using this script, steps 1, 7, (8 partially) are automated.
This script has to be run from the scilog-ci folder, every time one wants to deploy.
It can be modified to run directly in the scilog (submodule) folder to cover points 2 and 6 (cd commands).
Let me know if it is good enough or needs changing.

Did you think about the option to have no branches for scilog-ci for "normal" development updates ?

I still haven't understood what is the idea on that ....

I guess the idea was to only have a single branch in scilog-ci and automatically deploy whatever is on main. We test locally, discuss the PR on git, merge it, test it on our dev server and potentially later deploy it on production.
By skipping the ci branches, it is always clear what is currently deployed on dev

now the deployment has 3 steps: to dev, to qa, to prod. What you described is actually what happens in qa. I still struggle to understand what it simplifies, it only removes the PR opening need

Okay, now I'm also confused... I understood that there is no automatic deployment from scilog main?

everything it's more or less explained here: https://github.com/paulscherrerinstitute/scicat-ci/#scicat-ci (happy to discuss further ofc)


[BUG] Anvil duplication (severely broken rn)

Please post multiple screenshots on a site like Imgur and post config files on a site like Pastebin or some other alternative.
I have a discord link that a player sent me, it's just him placing an item in an anvil and then exiting and two items move into his inventory (he duped a bow), I'm not sure how to use imgur with videos :/
https://cdn.discordapp.com/attachments/764487669232435200/784941297910939658/video0.mov
Please describe the bug
What is the bug? What do you think the appropriate behavior would be for this feature? (Paste screenshots here)
Anvils now duplicate all items when hitting escape
How to reproduce the error?
What can I do to reproduce the error you have experienced? Please be detailed.
Place any item in an anvil and hit escape, it'll duplicate whatever was in the anvil
What is your Spigot Version and your plugin list?
What version of Spigot are you using? /ver What plugins are you using? /plugins
Paper "git-Paper-261"
I've disabled PCI specifically and the bug is removed so I'm not gonna paste all my plugins since there's a lot (lmk if you do need it regardless lol)
What is the plugins version?
Please run '/pci version' in your console or in game and paste it here.
Latest version - 1.3.9
Anything else I should know?
Is there something else that I should know before attempting to fix this bug?
It's definitely gotta be because of the recent anvil change where it moves items back into your inventory--

Okay so messing around with things a bit more:
The dupe bug was occurring with normal anvils, not /anvil (in 1.3.9)
I moved back to 1.3.8 and the bug isn't present with real anvils but it does still erase items that are being used in /anvil
Hopefully this helps!

Thanks for the report! This is due to the way I had to return the player's items when closing a virtual anvil; there should be a fix in about an hour or so. Apologies for the inconvenience.

Appreciate the quick fix! Love your work ;)

No problem! If you or your players run across any other issues please let me know!


Added working for inserting, saving and rendering Favourite books list

The favourite book ids are stored locally, so even after refresh the list remains the same.
https://github.com/user-attachments/assets/be2a9001-551c-42ec-8b25-bc4ce99cde55
Fixes #52

You are doing great @AdityaBavadekar

Can you resolve the conflicts and push the code @AdityaBavadekar

Sure

I have resolved the conflicts.


[DO NOT MERGE] - Testing XTF issue 536 (same ns)

Description
This is going to be closed, and just used to test https://github.com/xtf-cz/xtf/issues/536 in the same namespace case
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change
[ ] Chore (non-breaking change which doesn't affect codebase;
test, version modification, documentation, etc.)

Checklist

[ ] Code contributed by me aligns with current project license: Apache 2.0
[ ] Pull Request contains a description of the changes
[ ] Pull Request does not include fixes for multiple issues/topics
[ ] Code is self-descriptive and/or documented
[ ] I have implemented unit tests to cover my changes
[ ] I tested my code in OpenShift

/test all


tokens with less than 18 decimals error

pick any token with decimals that are <> 18 (powr, wbtc)
pool -> add liquidity
displayed rates are incorrect.

duplicated issue.


Add Student Loan - Postgraduate repayment threshold - closes issue #3

Closes #3

Thanks for the contribution!


Add noopener and noreferrer attributes to links

Describe the bug
We should add noopener and noreferrer to links that open in a new tab

Do we want to add any tests/presubmits to look for future additions of new tabs that don't have noopener and noreferrer attributes?

Do we want to add any tests/presubmits to look for future additions of new tabs that don't have noopener and noreferrer attributes?

Yeah, that's a great call out - I'll create a separate bug to track that. Thanks!


JSON decode issue when querying database

First of all, thanks for creating flywheel.  Very helpful.
I do have the following issue:
Version is 0.44 running on python 3.43 under Windows 7
The db structure is:
class scrobble(Model):
    artist = Field(hash_key=True)
    ts = Field(data_type=datetime, range_key=True)

Works fine when database is queried through boto or when exploring through the DynamoDB Web interface.  I believe the issue is that for some reason the timestamps have a dozen or more digits to the right of the decimal point when I view them in the DynamoDB.
Whether that's it or not, all queries are throwing an Exception as follows:
>>> z = engine(scrobble).filter(artist="Lucinda Williams")
>>> z.first()
Traceback (most recent call last):
   File "<stdin>", line 1, in <module>
  File "C:\Python34\lib\site-packages\flywheel\query.py", line 155, in first
attributes=attributes, filter_or=filter_or):
  File "C:\Python34\lib\site-packages\flywheel\query.py", line 80, in gen
yield self.model.ddb_load_(self.engine, result)
  File "C:\Python34\lib\site-packages\flywheel\models.py", line 458, in ddb_load_
obj.set_ddb_val_(key, val)
  File "C:\Python34\lib\site-packages\flywheel\models.py", line 450, in set_ddb_val_
setattr(self, key, Field.ddb_load_overflow(val))
  File "C:\Python34\lib\site-packages\flywheel\fields\__init__.py", line 265, in ddb_load_overflow
return json.loads(val)
  File "C:\Python34\lib\json\__init__.py", line 318, in loads
return _default_decoder.decode(s)
  File "C:\Python34\lib\json\decoder.py", line 343, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "C:\Python34\lib\json\decoder.py", line 361, in raw_decode
    raise ValueError(errmsg("Expecting value", s, err.value)) from None
ValueError: Expecting value: line 1 column 1 (char 0)

Note that I truncated all the timestamps to integers and am continuing to see the same type of error in C:\Python34\lib\json\decoder.py

So what appears to be happening is the model is attempting to load values that are not specified in the schema because there are some additional fields present in the data. It will check to see if that field is a int/float/set, and if not, it will do a json.loads. When you look at your data, are there any fields present other than artist and ts?

There are other fields like title and album.  I have just added them to the model and the query does work.  However, since it's a NoSQL database and records could have additional fields, I am not sure why the json.loads fails but, in any event, adding the fields does enable the query to run.  Thank you.

The problem is that when I first wrote the library I tried to be too clever. In addition to saving the declared fields, Flywheel will also save any other properties on the model that don't begin or end with _. This means that when it loads the dynamo data, it will attempt to deserialize any undeclared fields. Which works fine if you only ever added data from the python models, but can cause problems in your case.
If you look over at the 0.5 branch, I'm removing that behavior for the eventual 0.5.0 release. It's too confusing and not very useful.

Thanks for the explanation.  I'll look at the 0.5 branch.  Again, I want to thank you for your work -- using a syntax similar to sqlalchemy's made using flywheel very easy.  I'll close this issue.


how to access peripherals in pre_init?

when I attempt to add the cx context to a pre_init function as indicated as possible in:
https://rtic.rs/1/book/en/by-example/app_init.html
I receive the following error:
error: `#[pre_init]` function must have signature `unsafe fn()`

The piece of documentation I am referring to is:

The init and optional pre-init tasks runs with interrupts disabled and have exclusive access to Cortex-M (the bare_metal::CriticalSection token is available as cs).

My goal here is to try and access the last reset reason to see if it was a software reset.
Am I trying to do something impossible here?

Does your function look something like this?
#[pre_init]
unsafe fn before_main() {
    // do something here
}

The pre_init function must be unsafe, according to the API docs here, because it's called before memory is initialized. Make sure to read the section about undefined behavior, but other than that accessing peripherals should be possible.
Though, if possible, doing this at the start of init is a far better idea.

Hi,
That was unfortunately worded in the documentation, it's not supported to add pre init in RTIC.
One needs to use the cortex_m_rt provided pre_init.
After that, at access peripherals, you need to conjure them using unsafe.

This should be resolved, closing.


fixes joshgillies/node-matrix-file-types#1

Now ignores case on file extension
Updated test.js to now test .JPG

Awesome, much appreciated! 👍

Closes: #1


Equivalent Go syntax to create encrypted storage class

I'm new Pulumi, this page shows how an encrypted storage class can be created using Typescript:
File: themes/default/content/docs/guides/crosswalk/kubernetes/control-plane.md
What is the equivalent Go syntax for it? Since The Go SDK has StorageClassArgs but it does not have any encryption specific argument. Should they be untyped string arguments - using Parameters pulumi.StringMapInput. Any example will be helpful. The default way to create a cluster keeps everything unencrypted and this is not secure for many usecases. So have sane defaults will be really nice.

There is an Encrypted property on StorageClassArgs in the Go SDK. You should be able to use that.
https://github.com/pulumi/pulumi-eks/blob/master/sdk/go/eks/pulumiTypes.go#L2013
Does that work?


Feature/add-redis

add redis plugin for php

Nope, it isn't a new fix version!
Dont edit versions number, edit only CHANGELOG (under next version) and Dockerfile.


Quick suggestions

Love this project, David 💯
A few low-hanging suggestions to improve the repo:

Add a desc to the github repo's metadata (in the upper right corner; same place you edit the URL metadata for the repo). This is really helpful because it's what shows up in opengraph metadata and when shared around the github site itself.
Add a screenshot to the readme like in your tweet 😄
Add a license file; just copy one from an existing repo

You'd be amazed at how much these small details impact developer conversions, especially the image.
Keep up the great work && thanks 🙏

Thank you for the suggestions 😄


Using pyplot fails with latest anaconda

Updating anaconda or using the miniconda package by setting the environment to "" has using pyplot fail with no gui found. Will post the error lot later as I am currently  not at the pc with the updated anaconda.

julia> using PyPlot
C:\Users\micha\.julia\v0.4\Conda\deps\usr\lib\site-packages\matplotlib\__init__.py:1318: UserWarning:  This call to matplotlib.use() has no effect
because the backend has already been chosen;
matplotlib.use() must be called *before* pylab, matplotlib.pyplot,
or matplotlib.backends is imported for the first time.

  warnings.warn(_use_error_msg)
WARNING: No working GUI backend found for matplotlib.
ERROR: InitError: PyError (:PyImport_ImportModule) <type 'exceptions.ImportError'>
ImportError('DLL load failed: The specified module could not be found.',)
  File "C:\Users\micha\.julia\v0.4\Conda\deps\usr\lib\site-packages\matplotlib\pyplot.py", line 27, in <module>
    import matplotlib.colorbar
  File "C:\Users\micha\.julia\v0.4\Conda\deps\usr\lib\site-packages\matplotlib\colorbar.py", line 34, in <module>
    import matplotlib.collections as collections
  File "C:\Users\micha\.julia\v0.4\Conda\deps\usr\lib\site-packages\matplotlib\collections.py", line 27, in <module>
    import matplotlib.backend_bases as backend_bases
  File "C:\Users\micha\.julia\v0.4\Conda\deps\usr\lib\site-packages\matplotlib\backend_bases.py", line 56, in <module>
    import matplotlib.textpath as textpath
  File "C:\Users\micha\.julia\v0.4\Conda\deps\usr\lib\site-packages\matplotlib\textpath.py", line 22, in <module>
    from matplotlib.mathtext import MathTextParser
  File "C:\Users\micha\.julia\v0.4\Conda\deps\usr\lib\site-packages\matplotlib\mathtext.py", line 63, in <module>
    import matplotlib._png as _png

 [inlined code] from C:\Users\micha\.julia\v0.4\PyCall\src\exception.jl:81
 in pyimport at C:\Users\micha\.julia\v0.4\PyCall\src\PyCall.jl:79
 in __init__ at C:\Users\micha\.julia\v0.4\PyPlot\src\PyPlot.jl:250
 in _require_from_serialized at loading.jl:84
 in _require_from_serialized at loading.jl:109
 in require at loading.jl:219
during initialization of module PyPlot

You closed this; is it working for you now?

No, I was actually going to post but the wonderful comment and close button positions...
Couldn't get it working on my other PC either with the same error.

Are you using the default matplotlib backend (Tk) or have you set up your matplotlibrc to use a different backend?

Could this be the same as #151?

Clean install on a new format on windows using Julia 0.4.
I am not sure. I tried
ENV["LD_LIBRARY_PATH"]="C:/Users/mica/.julia/v0.4/Conda/deps/usr/lib"
using PyPlot

and god the same error.
I haven't changed the matplotlib backend at all.

Hi, I just checked with an old version of anaconda(2.3) and it works with that fine.
It seems to be only an issue with Anaconda 2.4.

What Matplotlib backend are you using?  e.g. if you are using the qt backend maybe you need import Conda; Conda.add("pyside")

This is a dup of https://github.com/stevengj/PyCall.jl/issues/209. It is actually an upstream issue with anaconda which should be fixed in near future. So using an older Anaconda for now seems like a good solution.

Yet another reason to give up on Anaconda.  I am writing up the relative sizes and install times for PSF Python and pip'ing needed packages vs the Julia internal Python using miniconda.  I am still doing some research, but the PSF and pip option looks very good on OS X, but with some issues on Windows that can be solved less easily.

This has reportedly been fixed upstream by Anaconda.


Plugin not working in certain cases

Hi!
I started working a couple of days ago with Cropper but I have a particular problem.
If I initialize the plug in between "script"  tags (outside any function) it works as intended but if I put inside a jQuery or JavaScript event/function(  $("#element").click or   for example ) I get "$(...).cropper is not a function"  error.
What can it be the problem?
Thanks in advance and congratulations for this amazing plug in!
Eneko.

Can you paste a few code of your deno here?

I've been reviewing my code and I've found the error.
I'm working in a Rails app and apparently it auto-imports Javascript and CSS files but what didn't notice is that someone has put another


Alter table command is missing.

Provide support for ALTER Table command

This will be available in the next release of the library.


dist-tags on npm incorrect, showing 5.0.0-beta as latest

I'm updating my dependencies with the yarn upgrade-interactive tool and it's reporting that 5.0.0-beta is the latest version for react-helmet.

Doesn't look like that's an actual release or tag on Github, but I am seeing that on npm. Running yarn info react-helmet (or npm show react-helmet) I'm seeing that the 'dist-tags' are latest: '5.0.0-beta' and next: '2.0.0'.
Seems like latest should be 4.0.0 and next (if it exists) should be 5.0.0-beta

Thanks! I just fixed this. latest now points to 4.0.0.

Nice! Thanks for your quick response and action


Nothing prevents concurrent writes (or concurrent reads)

I may be misunderstanding the code, but given the send() methods aren't taking &mut self, it seems like nothing prevents concurrent or interleaved writing, which can result in corrupted messages.
(I was seeing symptoms of this, wrapped Sender with tokio Mutex and I think it fixed it. But, you know, race conditions, could be unrelated.)

That's correct. I'm not sure if this library should try to do something about this, or if that's up to the user.

I can imagine people wanting their own locking patterns; maybe you want a lock per Sender, or maybe you are parent process writing to all client processes each time, one of my use cases, and so you want a single lock for multiple Senders.
But the current interface is misleading, and can lead to silent data corruption that idiomatically would've been prevented via the type system. So I would suggest Sender::sender() and recv() be changed to accept &mut self, to match the standard Rust pattern (e.g. Read and Write traits).

I really don't like the idea of forcing the this to use &mut self as it makes the type much more frustrating to work with in some common patterns.

@cgwalters what do you think?

Well I got nerd sniped into this https://github.com/mitsuhiko/tokio-unix-ipc/pull/17


Address DH's feedback on FastHotStuffEventLoop structure

Current dependencies on/for this PR:

feature/proof-of-stake

PR #649

PR #767   👈

This stack of pull requests is managed by Graphite.


fixing roundtrip for components

https://github.com/bytecodealliance/wasm-tools/issues/1676
Issue has been raised


Implement respond_to?

I ran into this while trying to set up a message expecation in RSpec and
I got a failure because RSpec thought that the Setting object wouldn't
respond to the attribute I was attempting to stub.
Further reading:
https://robots.thoughtbot.com/always-define-respond-to-missing-when-overriding

Thanks!  I pushed a new commit to bump the patch version so we're at 0.2.2 now.  Are you 👍  to merge and release?

Yup, go for it!


HBASE-27192 The retry number for TestSeparateClientZKCluster is too s…

…mall

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  4s
Docker mode activated.


Questions about the Scenario Outlines

Hey, Guys
I walked through the Scenario Outlines Wiki, but I have some questions about it:
1, For example, as the below shows, the data table values whether support the variables?
Scenario Outline: eating
  Given there are <start> cucumbers
  When I eat <eat> cucumbers
  Then I should have <left> cucumbers

  Examples:
    | start               | eat | left |
    |  <%= cb.prefix %>   |  5  |  7   |
    |  <%= cb.prefix %>   |  5  |  15  |

2, The below format whether right? Or I must create a new variable to replace the last <left>? In other words, how to re-use these variables, <start>, <eat>, <left>?
Scenario Outline: eating
  Given there are <start> cucumbers
  When I eat <eat> cucumbers
  Then I should have <left> cucumbers
  And there are <left> cucumbers left

  Examples:
    | start | eat | left | left |
    |  12   |  5  |  7   |  7   |
    |  20   |  5  |  15  |   7  |

Please use the support forums for questions and discussions. We use GitHub issues to track bugs and contributions exclusively.
Please don't address this community as "guys". I know it is considered gender-neutral by some people, but many people feel it's excluding people who are not male.

@aslakhellesoy Sorry for the bothering, thanks for your suggestion! :)


Upgrade requirements

Codecov Report

Merging #89 (4ae7e23) into main (d358777) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #89   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines           52        52
  Branches         6         6
=========================================
  Hits            52        52

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 d358777...4ae7e23. Read the comment docs.


Update serve packages flagged by snyk

Hi @jaffrepaul
I noticed there is a breaking change in server v13 which changes the default port from 5000 to 3000. Any tests previously relying on the default 5000 would need to add the parameter -p 5000 to the serve command line.
Some of these examples are using yarn so they would need the yarn.lock updating. They currently have yarn classic v1 lock files though and this version went into maintenance mode in January 2020. What do you think about migrating to "modern" yarn (version 3)?

@jaffrepaul
Just to check that my assessment was right, I ended up fixing the issues here, so if you'd like a PR piggy-backed on your draft PR please let me know.
Here are the differences:
https://github.com/cypress-io/github-action/compare/update-snyk-deps...MikeMcC399:github-action:fix/update-snyk-deps

I've submitted PR #682 to fix this one. My PR fails the snyk security check. I don't know why though, since I can't view the details, not being a member of the cypress-io organization.

@jaffrepaul

You'll need PR https://github.com/cypress-io/github-action/pull/690 to get the markdown issue fixed. Adding renovate into the list doesn't help since it's a different root cause.

I suggest you also revert https://github.com/cypress-io/github-action/pull/673/commits/38051500bd75c82fa93c474c8a5351e872b30b4c with the ineffective whitelisting.

Thanks @MikeMcC399 adding it to the whitelist was a simple prayer in the interest of time. Appreciate the follow up here!


BirdFlowR:::get_date()

These two date sets in BirdFlowR, which I expected to be the same, diverge by 1 day after the middle of the calendar year.
Perhaps could fix or sunset get_date()?
This is from a birdflow model based on the current package version.  This should be reproducible with any current birdflow object -- more detailed reprex available upon request.
   It's not used internally anymore and was never intended for external use.  I'll delete it.


Bossbar Crashes

What happened?
The bossbar crashed our server
Steps to reproduce
Create a bossbar
Crash dump
No response
Plugin Version
New
Server Version
New
PHP Version
8.0
Server OS
New

give more proof: pictures and videos


Animation background is white instead of transparent on iOS

Animation file -
loading.json
Example -
https://github.com/ismai117/kottie/assets/7789307/0e014884-f86c-4c12-9294-78f7d79196a2
Please refer to the above animation JSON file and the example video recording. The animation has a transparent background but it renders with a white background when running on iOS. The example video is taken by using the animation file in the sample app.

Hi @rdsarna, transparent background doesn't work on the UIKitview at the moment, so you have to set the background for the KottieAnimation composable the same as the container's background color. Here's an example.
MaterialTheme {
        Scaffold(
            containerColor = MaterialTheme.colorScheme.onSurfaceVariant
        ) { paddingValues ->
            Box(
                modifier = modifier
                    .padding(paddingValues)
                    .fillMaxSize(),
                contentAlignment = Alignment.Center
            ) {

                KottieAnimation(
                    composition = composition,
                    progress = { animationState.progress },
                    modifier = modifier.size(300.dp),
                    backgroundColor = MaterialTheme.colorScheme.onSurfaceVariant
                )

            }
        }
    }

Result:
https://github.com/ismai117/kottie/assets/88812838/5c0d0073-6fc9-467d-b6cd-3f82faf9b168

That fixed the issue. Thanks a lot!

Hi @ismai117
I tried what you said, but it kinda doesn't work. Here's an example:
Left button = iOS, Right button = Android
You can see on iOS the white empty square at the "first" animation frame that Android doesn't have
https://github.com/ismai117/kottie/assets/39140585/6f28e572-2c30-4a1c-871a-47e92ca95fd6
I guessed there was something wrong with my color management, but even when I set the Kottie background to Color.Black, I initially still saw a white square at the first animation frame. Only afterwards could I see the animation start on a black background

My code:
KottieAnimationView(
    type = KottieCompositionSpecType.JsonString(animationJsonString),
    animationSettings = KottieAnimationSettings(backgroundColor = Color.Black),
    onCompleteCallback = onCompleteCallback,
    modifier = Modifier
        .size(24.dp)
        .background(Color.Black)
)

KottieAnimation(
    modifier = modifier,
    composition = composition,
    progress = { animationState.progress },
    backgroundColor = animationSettings.backgroundColor
)

I don't know exactly how to handle a transparent background on iOS, but maybe this could help?
class TransparentViewController: UIViewController {
    override func viewDidLoad() {
        super.viewDidLoad()

        // view
        self.view.backgroundColor = UIColor.clear

        // subview
        let subview = UIView()
        subview.backgroundColor = UIColor.clear
        self.view.addSubview(subview)
    }
}

https://github.com/JetBrains/compose-multiplatform/issues/3154


Fix git_add rule

Coverage decreased (-0.5%) to 90.341% when pulling 41707b80c61acadb7c87b0efcbf10f4186dc5937 on scorphus:fix-git-add into 3a39deb485995e67afb1919972cd1c9aaedf4c32 on nvbn:master.

Thanks!


W2D2 Typos

Tutorial 3, Section 2, second plot: labels (‘simulation’, ‘analytic solution’) are swapped.

In addition:
Small typo in T3 Exercise 1: ‘Distribution of bateria positions’
T4 Exercise 1: len(x2) = T-2, should be T-1 from description

Also:
T2

system express either

system expressed either

Also:
T3
Spare code chunk at end.

Spare code chunk at end.

That's pretty much unavoidable, so I wouldn't worry about it when you see it.

Spare code chunk at end.

That's pretty much unavoidable, so I wouldn't worry about it when you see it.

I don't think other tutorials had it 😕
(Not that it is at all an issue, just looks slightly messy)

Adding another note: in T1, the parenthesis in args=(params) should be removed (it is already a list).

Hello, just finish the W2D2 Tutorial 3
I realized in the helper function "var_comparison_plot" the xlabel and ylabel were swapped. Since we plot empirical variances at the x axis and analytical variances at the y axis, the set_xlabel should be "empirical equilibrium variance" and set_ylabel should be "analytic equilibrium variance" instead.
Thank you!


AndroidView is offsetted and clipped on some Xiaomi devices with a notch

When running the webview_flutter example on a device with a notch, a blank bar apears on the top of the native view, below the app bar. The platform view responds to touches with a vertical offset (one must touch above the point). #19030

I can't reproduce this on an emulator with a notch.
@yoavrofe which device are you seeing this on?
Can you post the output of flutter doctor.
Do you see the issue on latest master?

I can't reproduce this on a Pixel 3 XL, either --- I did a sync to head this morning and a flutter run of the webview example, and it worked. @amirh and I are wondering if it's a device-specific issue?

same problem #25152 .

View will have a padding on this Android screen.

code example is

https://github.com/CaiJingLong/androidview-notch-error

And touching events can also be offset.
Other androidview is also have the problem . I just extracted the smallest repository of code.
flutter doctor -v
flutter doctor -v
[✓] Flutter (Channel dev, v1.0.0, on Mac OS X 10.13.6 17G65, locale zh-Hans-CN)
    • Flutter version 1.0.0 at /Users/cai/fluttersdk/flutter
    • Framework revision 5391447fae (10 days ago), 2018-11-29 19:41:26 -0800
    • Engine revision 7375a0f414
    • Dart version 2.1.0 (build 2.1.0-dev.9.4 f9ebf21297)

[✓] Android toolchain - develop for Android devices (Android SDK 28.0.3)
    • Android SDK at /Users/cai/Library/Android/sdk
    • Android NDK at /Users/cai/Library/Android/sdk/ndk-bundle
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Volumes/Evo512/application/develop/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 9.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.2, Build version 9C40b
    • ios-deploy 2.0.0
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 29.0.1
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[✓] Android Studio (version 3.2)
    • Android Studio at /Volumes/Evo512/application/develop/Android Studio.app/Contents
    • Flutter plugin version 30.0.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[✓] IntelliJ IDEA Ultimate Edition (version 2018.2.5)
    • IntelliJ at /Users/cai/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 31.1.3
    • Dart plugin version 182.5124

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

[✓] Connected device (1 available)
    • MI 8 • b6ec3684 • android-arm64 • Android 8.1.0 (API 27)

my device photo

I will test the master channel.

My device is a Pocophone F1.
The problem exists when using the master channel, but does not exist when not using the AppBar.
flutter doctor:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, v1.0.1-pre.15, on Mac OS X 10.14.1 18B75, locale en-IL)
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
[✓] Android Studio (version 3.2)
[✓] VS Code (version 1.29.1)
[✓] Connected device (2 available)
• No issues found!

I upgraded to Android 9.0, and could not reproduce the problem. It seams like a problem related to the app bar before 9.0.

The Pocophone F1 I got yesterday to investigate the issue auto-upgraded to Android 9 over night (without asking for my confirmation), and with Android 9 the issue is indeed fixed.
I'm trying to reach out to relevant folks to see if there's a better workaround than allocating a bigger virtual display and clipping it.
As we learned that this is a vendor specific bug, and as it seems to have been already fixed by the vendor I'm de-prioritizing this issue and moving out of the January 2019 milestone.
We should still look for a reasonable workaround for the affected Xiaomi devices that are still running older Android versions.

If you can, try to contact Xiaomi Development Engineer?

I met same issue, older Android versions have any solution to avoid it? @amirh

me too。

older Android versions have any solution to avoid it? @amirh

In addition to upgrading android9, is there any other way to solve it temporarily? Simple, quick way?

See the proposed workaround at https://github.com/flutter/flutter/issues/24657#issuecomment-448394975 if you're willing to fork the engine you can probably give it a shot.

Is there a solution?

I have update Android version to 9，and this problem has solved

Current solution is
Modify SingleViewPresentation which is create PlatformView
code:
int flag = 0x00000100 | 0x00000200 | 0x00000400;
try {
    Method method = Window.class.getMethod("addExtraFlags",
            int.class);
    method.invoke(getWindow(), flag);
} catch (Exception e) {
    Log.i(TAG, "addExtraFlags not found.");
}

document: https://dev.mi.com/console/doc/detail?pId=1293#_3_3

Current solution is
Modify SingleViewPresentation which is create PlatformView
code:
int flag = 0x00000100 | 0x00000200 | 0x00000400;
try {
    Method method = Window.class.getMethod("addExtraFlags",
            int.class);
    method.invoke(getWindow(), flag);
} catch (Exception e) {
    Log.i(TAG, "addExtraFlags not found.");
}

document: https://dev.mi.com/console/doc/detail?pId=1293#_3_3

android:screenOrientation="landscape" ,This problem has not been solved

AndroidManifest.xml
<meta-data             android:name="notch.config"             android:value="portrait"/>
document: https://dev.mi.com/console/doc/detail?pId=1293#_3_3

AndroidManifest.xml
<meta-data             android:name="notch.config"             android:value="portrait"/>
document: https://dev.mi.com/console/doc/detail?pId=1293#_3_3

Bumping this issue down to a P2 since it is fixed with a later version of Android and has a posted workaround.


dont compute git info when not cloud not enabled and warn message improvement

This is a 🐞 bug fix.

[ ] I've added tests (if it's a bug, feature or enhancement)
[ ] I've adjusted the documentation (if it's a feature or enhancement)
[x] The test suite passes (run bundle exec rspec to verify this)

Summary
Don't compute vcs info when terraspace cloud is not in use. Also provide a user a warning when unable to get info from local git repo.
Context
Closes #254
How to Test
Sanity test
Version Changes
Patch

Released in 2.1.5


change service name to match readme

Change is required for demo to work as per README instructions.

I think the change should be in the README because the change to the manifest will break the deploy to bluemix

I'm happy to merge a fix for the README

[Current coverage][cc-pull] is 100%

Merging [#26][cc-pull] into [master][cc-base-branch] will increase coverage by +9.10%

@@           master       #26   diff @@
=======================================
  Files           3         3
  Lines          33        69    +36
  Methods         0         7     +7
  Branches        5         5
=======================================
+ Hits           30        69    +39
+ Misses          2         0     -2
+ Partials        1         0     -1

2 files (not in diff) in config were modified. more

Misses -1
Partials -1
Hits +20

File app.js (not in diff) was modified. more

Misses -1
Partials 0
Hits +19

Powered by Codecov. Last updated by 76cf63f
[cc-base-branch]: https://codecov.io/gh/watson-developer-cloud/natural-language-classifier-nodejs/branch/master?src=pr
[cc-pull]: https://codecov.io/gh/watson-developer-cloud/natural-language-classifier-nodejs/pull/26?src=pr

Thank you for your feedback. I have reverted the initial change I made to the manifest file and edited the create service command, example training output, and sample manifest file to reflect the original manifest file.

thanks @flosal-bccs !


update codebase to match Replicate guide

This updates the codebase to match what users will get when they go through the steps in this guide: https://replicate.com/docs/get-started/nextjs
It also updates the README with real content: https://github.com/replicate/getting-started-nextjs/pull/5/commits/dd997d5c6f9f92af27107eeecd978acd3ee5655c
Followup to https://github.com/replicate/replicate-web/pull/2753

@bfirsh do you know what's up with this License Compliance check? Is this something you installed?
When I tried to view the details of the failing check, I logged in with my GitHub account and I now see this onboarding page:

Yeah, just ignore that. I'll figure out how to switch it off.

I haven't thoroughly reviewed this, but it's presumably a lot more functional than what is currently in the repository. :)

No feedback from my review! This is great. Thank you!


Icon requests

Would you mind of making Perl, Less, Stylus and Toml (rust dependency manager) icons?
Thanks in advance :smile:

No problem! If you have other language icons to propose me, do not hesitate, I will try to make them as quickly as possible

Oh, wait, I forgot the Stylus icon, I'm doing it now

Done


CyclonDX SBOM vulnerabilities not found

I tried using osv-scanner with the latest available docker image and a CyclonDX SBOM using the following command
~/test/osv$ docker run -it -v ${PWD}:/src ghcr.io/google/osv-scanner --sbom=/src/bom.cdx.json
Scanned /src/bom.cdx.json as CycloneDX SBOM and found 2 packages
No vulnerabilities found

With the following CyclonDX SBOM (bom.cdx.json):
{
  "bomFormat": "CycloneDX",
  "specVersion": "1.4",
  "version": 1,
  "components": [
    {
      "type": "library",
      "name": "wolfssl",
      "version": "v5.6.0-stable",
      "purl"<EMAIL_ADDRESS>    },
    {
      "type": "library",
      "name": "NanoPb",
      "version": "0.4.1",
      "purl"<EMAIL_ADDRESS>    }
  ]
}

The osv-scanner tool reports "No vulnerabilities found", but if I check for vulnerabilities of the used libraries using the web interface, I can see several known vulnerabilities. For example:
https://osv.dev/list?ecosystem=&q=wolfssl
https://osv.dev/list?ecosystem=&q=nanopb
Also, if I use the API, I receive a list of vulnerabilities. For example:
curl -d '{"package": {"name": "wolfssl"}, "version": "v5.6.0-stable"}' "https://api.osv.dev/v1/query"
curl -d '{"package": {"name": "nanopb"}, "version": "0.4.1"}' "https://api.osv.dev/v1/query"
I tried to follow the usage documentation here https://google.github.io/osv-scanner/usage/
So I assume that I either missed something in the documentation on how to use the CLI tool, or that there is a potential bug.
If there is an error in the SBOM itself I would expect an error or at least no feedback that the two packages were found.
Used docker image: ghcr.io/google/osv-scanner    latest    725aa024bca8    8 weeks ago
docker run -it -v ${PWD}:/src ghcr.io/google/osv-scanner --version
osv-scanner version: 1.3.6
commit: b5f7502d2b76b1b84201937aa759299d23e02c48
built at: 2023-07-19T05:04:48Z

Nevermind, after having had another look at the API responses I realized that the purl's are not the same as in the SBOM, namely the package type is different "purl":"pkg:pypi/nanopb" and "purl":"pkg:generic/wolfssl" instead of pkg:github.
As both projects are hosted on github, that leaves me wondering how one determines the right package type to use for the purl in the SBOM.
For example, some vulnerabilities reported for pypi are not specific to the nanopb python package, but are generic nanopb issues...
If I understand the documentation right, SBOMs can currently only be used with purl, and not only based on name and version.

indeed, we currently don't have support for github package URLs. In general it's also challenging to build a reliable mapping between github repos and ecosystem specific ones like PyPI etc.
That said, there are probably some things we can do here to give more results in some cases. E.g. leveraging our git commit scanning to match github PURLs.
@another-rex thoughts?

I think long term we can get some matching with the GitHub URLs.


Including non-python files to package

Hi,
I am developing some Django plugin and I keep my static files (like templates, less and js) in my source directory (i.e. in each app I have templates directory).
The issue is that I can't figure out how to tell PyBuilder to include my static files into distribution.
I wrote some monkey-patching task which adds include_package_data = True to setup.py arguments and created a MANIFEST.in file which looks more or less like this:
recursice-include src *
recursive-exclude * __pycache__
recursive-exclude * *.py[co]

It is not very convenient though. Can this be done in a simpler way?

I am aware of copy_resources plugin but it doesn't do what I want. I want to keep my directory structure, not copy everything to the same directory.

Yeah sorry, this is not well documented currently. We're working on replacing the documentation.
There are two possibilities to add data files, both involve using the project object in your build.py initializer :

project.include_file(package_name, filename)
project.install_file(destination, filename)

Install_file is for a static destination (/tmp for example) and include_file is for using the package as destination.

Thanks!
It looks like I can't add whole directory and need to declare each file separately which is not really nice.
It will be great to have a function like project.include_directory(directory, file_names) that would generate in MANIFEST.in line:
recursive-include directory file_names

i.e. recursive-include src/static *.min.js *.css. Would you accept such a PR if I prepared some code for that functionality?

I think this would be a great enhancement. You can do this in the build.py with shutil of course but it would be much better to have this logic in pybuilder to avoid cluttering of the build descriptor.

@mriehl, just to inform you - I am going to write this feature today evening. In case of lack of time I will finish it tomorrow evening.

Great to hear, thank you for working on this!

Yeah, some documentation about this would be really good.

Hi, is there a plan to merge this change? This would be very useful

Afaik there's no pull request? Let me check..

I've created pull request for this issue.

@glujan if you could, please squash the commits and reword the commit message with fixes #127 to link up PR with the Issue.

@arcivanov Here you are.

Hi guys,
I am having an issue with this, and since there is no documentation, I don't know whether I am using it wrong or there is a bug.
How am I supposed to call the method?
I tried the following:
1 - project.include_directory(project.expand_path('$dir_source_main_python/testdata/arr'), ['*.pkl', '*.pickle']) but this will replace all slashes in the full path with . and the package doesn't exist (I truncated the path and it would work)
2 - project.include_directory('testdata/arr', ['*.pkl', '*.pickle']) but relative path doesn't work. Somehow it packages the files, but setup.py will not install them.

Sorry guys (I hope writing in a close issue still sends notifications :) ), but this packaging is confusing at so many levels :)
I need to elaborate my problem (I'll focus on include_directory).
1 - If I pass the full path, /home/user/proj/src/main/python/testdata/files, this will end up in the manifest, but my tar.gz will contain only testdata/files, so this will be broken. The same, but with dots, will appear in setup.py, which is broken as well.
2 - If I pass the relative path testdata/files, this will end up correctly to the manifest, but apparently os.walk cannot work with it and will never reach package_data in setup.py.
To add to my confusion (but maybe it's just my limited knowledge here), @glujan mentioned above recursive-include src/static *.min.js *.css, but from a conventional naming structure, that would break everything as much as a full path.

@cowst a bit late but for what it's worth, and for anyone else who's also losing hair over the ambiguity of this implementation, this worked for me:
project.package_data.update({'mypkg': ['rel/path/to/data/files/*' ]})

It's 2020 and I still scratch my head to make it work. The solution that worked for me to combine two posts above:
install_file, and update package_data
sort of:
project.files_to_install.append(('give it a name', [list files relative to src/main/python]))
project.package.data.update({'mypkg': [same thing as above list]})

Does this help? https://github.com/pybuilder/pybuilder/blob/5dd1ced52dfa7167bb84b871fad5382eb37666fe/build.py#L139-L141

I appreciate your feedback. I am actually going through this now.
Tested the solution above, and it worked. I am looking at the include_directory function. It works to package into sdist, but not bdist_wheel. TBH, I am still trying to figure out what it is trying to do, in particular the last line https://github.com/pybuilder/pybuilder/blob/4f25fb307fa2cb03c8e034b20723fb96126d6d33/src/main/python/pybuilder/core.py#L399

Although I am new to python packaging, I would still suggest renaming include_file to something like include_package_data. It would be more consistent with setuptools.

Unfortunately we can't rename the stable API until a major version. I should probably should do a blog article write up about how to use these two functions though.

Sure, I was trying to say provide a new function name and deprecating the old name. Just a suggestion to consider.
Also, it would be great if you could provide more documentation about this topic. Currently, there are different plugins/functions for doing this: copy resource, install_file, include_file, MANIFEST.in. They are confusing to me.
Great appreciation for the work you've done!


Add standardization/write to BQ step for CTP

Implements write_to_bq for the CTP data source
Adds two standardized race categories, API and INDIGENOUS
Adds unit tests

#116

Thanks for the reviews! I created issues for the items we discussed

Thanks for the reviews! I created issues for the items we discussed


client crashes when attempting reconnect

Describe the bug
The server was unavailable due to iptables and the client process failed with:
Aug 18 17:17:26 overcloud-controller-2 directord[18220]: ERROR Heartbeat failure, can't reach server
Aug 18 17:17:26 overcloud-controller-2 directord[18220]: WARNING Reconnecting in [ 2 ]...
Aug 18 17:17:28 overcloud-controller-2 directord[18220]: DEBUG Running reconnection.
Aug 18 17:17:28 overcloud-controller-2 directord[18220]: Process Process-1:
Aug 18 17:17:28 overcloud-controller-2 directord[18220]: Traceback (most recent call last):
Aug 18 17:17:28 overcloud-controller-2 directord[18220]:   File "/usr/lib64/python3.8/multiprocessing/process.py", line 315, in _bootstrap
Aug 18 17:17:28 overcloud-controller-2 directord[18220]:     self.run()
Aug 18 17:17:28 overcloud-controller-2 directord[18220]:   File "/usr/lib64/python3.8/multiprocessing/process.py", line 108, in run
Aug 18 17:17:28 overcloud-controller-2 directord[18220]:     self._target(*self._args, **self._kwargs)
Aug 18 17:17:28 overcloud-controller-2 directord[18220]:   File "/opt/directord/lib64/python3.8/site-packages/directord/client.py", line 128, in run_>
Aug 18 17:17:28 overcloud-controller-2 directord[18220]:     self.driver.heartbeat_reset()
Aug 18 17:17:28 overcloud-controller-2 directord[18220]:   File "/opt/directord/lib64/python3.8/site-packages/directord/drivers/zmq.py", line 444, in>
Aug 18 17:17:28 overcloud-controller-2 directord[18220]:     self.poller.unregister(self.bind_heatbeat)
Aug 18 17:17:28 overcloud-controller-2 directord[18220]: AttributeError: 'Driver' object has no attribute 'bind_heatbeat'

To Reproduce
Steps to reproduce the behavior:

The directord server needs to be on a node with iptables restricting network access
Run bootstrap
Clients install but cannot connect
See error

Expected behavior
reconnect should continue

This should now be resolved with the following commit: https://github.com/cloudnull/directord/commit/85d9a0e53e97c98f3d0f0ef355fcd428190ed2fa


[SPARK-14298] [ML] [MLlib] Add unit test for EM LDA disable checkpointing

What changes were proposed in this pull request?
This is follow up for #12089, add unit test for EM LDA which test disable checkpointing when set checkpointInterval = -1.
How was this patch tested?
unit test.
cc @jkbradley

Test build #55474 has started for PR 12286 at commit acf7455.

Test build #55474 has finished for PR 12286 at commit acf7455.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/55474/
Test FAILed.

Test build #55475 has started for PR 12286 at commit fa36cde.

Test build #55475 has finished for PR 12286 at commit fa36cde.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/55475/
Test PASSed.

Merged build finished. Test PASSed.

We could maybe make this a more complete test and actually check that their are no checkpoint files generated? Although that might be over kill.

@holdenk Thanks for your comments! Actually model.getCheckpointFiles.isEmpty checked that their are no checkpoint files generated.

@yanboliang that makes sense. I was thinking maybe checking the actual checkpoint directory to catch anything else (or even just clearing the checkpoint directory so checkpointing fails), but if getCheckpointFiles was correct even before the fix than that is probably a fine way to test it to.

@holdenk I agree with you, but I'm worried it could fail if checkpoint deletion/cleanup does not happen quickly enough across tests in this suite.  I'd prefer to leave it as is.

@yanboliang Thanks!  LGTM
Merging with master


Look for an alternative to pm2

We currently have to completely stop/kill and start pm2 to get the front-end to use the updated data files (see #475). We need to look for alternatives to pm2 for zero downtime deployments

Currently looking into using Kubernetes as an alternative

@GopiGugan reported that since we are using a single system, it  makes more sense to use Docker Swarm

VM webserver has 32GB RAM and is running on Intel Xeon Gold 5218 CPUs (2.3GHz, 4 cores)

We could ask IT services to reprovision this VM with more RAM

@ArtPoon to inquire about RAM upgrade

Old webserver (coreceptor, now ccr5) is back up and on the network.
I have also installed OS for a backup server (cxcr4) that was once part of the cluster.

Records updated

Waiting on validation of mongoDB to bring up second instance

@GopiGugan created Docker file for MongoDB, as well as CoVizu Docker files, so have everything needed to run Swarm

go ahead and set up a test environment on Paphlagon

Looking into create a service (using Docker Compose) to avoid having to use a container within a container

Using containers within a container impacts performance. Currently setting up a service using docker compose to deploy the front-end (with mongodb) on the web server

@GopiGugan reports that front-end should be updated by this afternoon

Front-end has been updated


Deviceinfo resolution r2

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.6 out of 7 committers have signed the CLA.:white_check_mark: karuna2git:white_check_mark: anand-ky:white_check_mark: steveford-obs:white_check_mark: cmurug728:white_check_mark: maheshkumar4:white_check_mark: MFransen69:x: liginPunooseYou have signed the CLA already but the status is still pending? Let us recheck it.

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 20 files pending identification.

Protex Server Path: /home/blackduck/github/ThunderInterfaces/299/rdkcentral/ThunderInterfaces

Commit: 651874250bb94b73831ca341103c9e11f6ca317b

Report detail: gist


Upgrade AspectJ 1.9.22 -> <IP_ADDRESS>

This PR contains the following updates:

Package
Type
Update
Change

AspectJ (source)
compile
patch
1.9.22 -> <IP_ADDRESS>

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade AspectJ 1.9.22 -> <IP_ADDRESS>

See:
- https://github.com/eclipse-aspectj/aspectj/releases/tag/V1_9_9_1
- https://github.com/eclipse-aspectj/aspectj/releases/tag/V1_9_20_1
- https://github.com/eclipse-aspectj/aspectj/releases/tag/V1_9_21_1
- https://github.com/eclipse-aspectj/aspectj/releases/tag/V1_9_21_2
- https://github.com/eclipse-aspectj/aspectj/releases/tag/V1_9_22_1
- https://github.com/eclipse/org.aspectj/compare/V1_9_22...V1_9_22_1


Implemented basic theming

Hey @adalfarus 🙋♂
Thanks a lot for spending your valuable time for contributing to ZenNotes, and I very much appreciate it.
Can you explain how can I build the aplustools from source? I can't find any whl files anywhere.
Once again, thanks a lot. I'll merge this as soon as the above issue is fixed.

Clone the repository (https://github.com/adalfarus/aplustools) and execute the build.bat file or use similar commands for your os, do the same for the install.bat file, but I would recommend install2.bat.
Alternativly you can just use the source code directly found here (https://github.com/adalfarus/aplustools/blob/main/aplustools/io/environment.py#L341), or just use version <IP_ADDRESS> (from PyPI) with the import "from aplustools.system import System". I opted to use the package to avoid cluttering the project files with general source code, which can be cumbersome and messy. Should I include this in the fork for you?


Add kitty terminal support

cool, i will try this out before merching, but it looks good. There is one thing though, and this is on me, i have never mentioned this, but the documentation (includes man page and readme.md) is generated from the stuff in docs/ . i just include the """"compiled""" manpage/readme for convenience and also remove make dependencie go-md2man , but if you have that command, i think you can generate new Readme and manpage with , make readme , make manpage

i just include the """"compiled""" manpage/readme for convenience and also remove make dependencie go-md2man, but if you have that command, i think you can generate new Readme and manpage with , make readme , make manpage

At first I started changing all the files by hand, then I dug into the Makefile a bit and made new readme and man page that way. If you look on my fork's readme, it should reflect the changes I made. This one is on me, but I should've split the commits into feature and documentation. I got kind of rusty with my git-fu and out of frustration I commited this as just one thing.


2.0.0rc4 clean install: Conflicting dependencies and VisibleDeprecationWarnings

Rasa version: 2.0.0rc4 on Windows 10 1909
Python version: 3.8
After doing a fresh install in a clean venv, numpy shows some VisibleDeprecationWarnings during training a model:

lib\site-packages\rasa\nlu\classifiers\diet_classifier.py:648: VisibleDeprecationWarning: Creating an ndarray from ragged nested sequences (which is a list-or-tuple of lists-or-tuples-or ndarrays with different lengths or shapes) is deprecated. If you meant to do this, you must specify 'dtype=object' when creating the ndarray
model_data.add_features(key, sub_key, [np.array(_features)])
lib\site-packages\rasa\utils\tensorflow\model_data.py:587: VisibleDeprecationWarning: Creating an ndarray from ragged nested sequences (which is a list-or-tuple of lists-or-tuples-or ndarrays with different lengths or shapes) is deprecated. If you meant to do this, you must specify 'dtype=object' when creating the ndarray
final_data[key][sub_key].append(np.concatenate(np.array(f)))
lib\site-packages\rasa\utils\tensorflow\model_data_utils.py:197: VisibleDeprecationWarning: Creating an ndarray from ragged nested sequences (which is a list-or-tuple of lists-or-tuples-or ndarrays with different lengths or shapes) is deprecated. If you meant to do this, you must specify 'dtype=object' when creating the ndarray
attribute_features = {MASK: [np.array(attribute_masks)]}

Also note that a clean install is producing dependency conflicts:

pip check
tensorflow 2.3.1 has requirement numpy<1.19.0,>=1.16.0, but you have numpy 1.19.2.

Thanks for raising this issue, @JustinaPetr will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗

Tested and the warnings are visible in Rasa==2.0.0 as well because of the numpy version being installed is 1.19.2 while the maximum version supported by tensorflow is 1.18.5. Will create a PR to pin the numpy version.

tensorflow==2.3.1 explicitly specifies that the support version of numpy should be <1.19.0 thus running poetry update updates the dependencies and includes the following line to the lockfile:
numpy = ">=1.16.0,<1.19.0"

I will prepare a PR that updates the dependencies.


xram location

Why is the xram located at the start of the ocram instead of using the dedicated xram
From the included linker scripts:
        xram_memory (!rx) : ORIGIN = 0x22020000, LENGTH = 16K

From the datasheet?
XRAM(EMI)     | 0x40000000            | 16KB  | Shared RAM mainly for data communication among M0/LP/D0                                                   ```

It comes from Bouffalolab. They may just use ocram that with cache disabled to exchange data. Also memory started from 0x40000000 can of course be used due this link script.


network node crashes with exit code 137

Description
The local node's network-node container crashes and exits with code 137(which presumably means too much memory). I don't recall having this issue before. I'm running both the hedera local node and a local graph node(using same docker-compose.yml as subgraph-example with a newer image i.e. image: graphprotocol/graph-node:v0.28.2) in docker pointing to the local-node's json rpc relay.
My docker resource settings are as follows:
CPUs: 6
Memory: 8GB
Swap: 1GB
Disk image size: 59.6GB
If it is in fact as memory issue, then there might be a memory leak as the recommended memory is 5GB for the local node and the graph node requires <3GB so 8GB should be sufficient.
Steps to reproduce
I've attached the entire network node's container logs in the following link(if useful please copy offline as it's set to expire in 1 day, paste in VS Code for easy viewing/finding):
https://ctxt.io/2/AAAQzVxXFA
Going through the logs I notice a few issues towards the end.

A bit after the above 312 error logs their appear logs which seem to indicate some kind of rate limiting:

2022-12-06 13:15:01.395 INFO  233  GasPriceMultiplierSource - The new cutoffs for EVM utilization congestion pricing are :
  (Gas) When EVM gas/sec exceeds:
    13500000.00 gas/sec, multiplier is 10x
    14250000.00 gas/sec, multiplier is 25x
    14850000.00 gas/sec, multiplier is 100x

The last logs indicates that the network node was killed:

/opt/hgcapp/services-hedera/HapiApp2.0/entrypoint.sh: line 51:    10 Killed                  /usr/bin/env java ${JAVA_HEAP_OPTS} ${JAVA_OPTS} -cp "data/lib/*" com.swirlds.platform.Browser > >(tee stdout.log) 2>&1

I've also attached the entire json-rpc-relay container logs below in case that might be helpful as well:
https://ctxt.io/2/AAAQ-e-iFg
Additional context
No response
Hedera network
other
Version
<EMAIL_ADDRESS>Operating system
macOS

@georgi-l95 I think this might be related to the evm rate limiting issue I experienced previously, however I don't recall the network-node crashing so likely a different issue altogether.

@dimitrovmaksim I did, I'm on v4.15.0, will see if the roll back works for me, thanks!


Android ShareViaInstagram opens Send To Direct Message

iOS opens the main dialog asking to share Story/Feed and successfully posts. Android 6 & 7 are both popping open straight to the Direct Message dialogs. It successfully posts the DM, but that's not what I want it to do. I can't seem to get to the Android regular sharing. I can't find any threads of anyone else having this issue. Any ideas?
cordova-android: 6.3.0
cordova-plugin-x-socialsharing: 5.2.1
ionic: 3.20.1
ionic-angular: 3.9.2
@ionic-native/social-sharing: 4.5.2

Same problem I am facing

Same problem here.

Same problem here.

Same problem here. I would appreciate any kind of suggestion/advice.
Thanks in advance.

FYI. I never got around this. I had to upgrade the entire app to the latest Ionic starter and re-import all project files and plugins and make necessary updates along the way. It was a pain. The ShareViaInstagram works as expected now though. Good luck.
cordova-android: 7.1.4
cordova-plugin-x-socialsharing: 5.4.6
ionic: 5.2.0
ionic-angular: 4.6.0
@ionic-native/social-sharing: 5.8.0

Thank you @jkufrin! We have lost our mind with debugging this.

for me shareViaInstagram open up only direct message
"cordova-plugin-x-socialsharing": "^5.6.8",
Ionic:
Ionic CLI                     : 5.4.16 (C:\Users\effev\AppData\Roaming\npm\node_modules\ionic)
Ionic Framework               : @ionic/angular 4.7.1
@angular-devkit/build-angular : 0.13.9
@angular-devkit/schematics    : 7.3.9
@angular/cli                  : 7.3.9
@ionic/angular-toolkit        : 1.5.1
Cordova:
Cordova CLI       : 9.0.0<EMAIL_ADDRESS>Cordova Platforms : android 8.0.0, browser 5.0.4
Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 4.1.2, (and 7 other plugins)
Utility:
cordova-res (update available: 0.15.1) : 0.6.0
native-run (update available: 1.0.0)   : 0.2.8
System:
Android SDK Tools : 26.1.1 (C:\Users\effev\AppData\Local\Android\Sdk)
NodeJS            : v10.15.3 (C:\Program Files\nodejs\node.exe)
npm               : 6.14.5
OS                : Windows 10

See the commit and pull request for the solution this problem. You should be able to remove the existing plugin with
cordova plugin remove cordova-plugin-x-socialsharing
and install this version with
cordova plugin add https://github.com/bennimmo/SocialSharing-PhoneGap-Plugin

See the commit and pull request for the solution this problem. You should be able to remove the existing plugin with
cordova plugin remove cordova-plugin-x-socialsharing
and install this version with
cordova plugin add https://github.com/bennimmo/SocialSharing-PhoneGap-Plugin


Fix some typos, update from upstream, simple test

Another fix for IE9 hack

Thanks for this, I don't have time to fully test this, but I'm going to pull it into the current version based on 1.4.0 and release as I work on a new version based on Bootstrap 2.x and above.
These tests are going to be great as I work on #6, can I ask how you generated the original CSS file?

Thanks, I went ahead and pulled 0d72d05 only.


Fix failing master

Black formatting initially
Need to fix failing tests

Note to self:
Ask Brian questions about dtypes tomorrow

https://www.tensorflow.org/probability/api_docs/python/tfp/distributions/Bernoulli

Ready for review

Thanks!


Problem

After i make my choices , the generator automatically closes. what do i do?

the same problem

Check .txt file named codes.txt
If you have Code Check: on then you need to check working-codes.txt


the group by shard is incorect

There are many places we allow to do show the graph by shard.
But what we are doing is combining all shards from all the nodes.
I'm not sure this is correct.
Instead, we should display per instance per shard.

@amnonh agree its confusing, but I do not understand your solution.

An example of the current situation, setting graph by shard (two nodes)

Same graph, but with a group by instance and shard (not just shard)

@tzach see my comments and example, let's decide, either to fix or to close the issue

Not sure its better, this will explode with a larger cluster of big machines.
Thinks 10 nodes and 64 shards per each.

but, then don't choose view per shard with all instances, what does that mean to show the sum of all shards by instance?
why shard 0 on one instance should be added to shard 0 on another instance?

Fixed by #606


Tips - Mneumonics for password backup is always displayed as invalid

While generating the account , at the verification stage for the back up mneumonics Tips is displayed as "Your mnemonic words backup is invalid. Please, carefully select the words in the correct order.".
I have not yet entered any words, but is this display right? kindly review

@bassemmagdy

Fixed by changeing tooltip.


 This is maybe not the best idea. From https://github.com/golang/dep/issues/221#issuecomment-277169036:

We had a CI dependency which was being pulled with go get in our Makefile, and it eventually lead to a broken build because the tool changed upstream to depend on 1.8 beta features.

Would this involve looking at build tags?
The thing is, some libraries will have source files for both old and new versions of Go, in which case you don't need to do anything. It just works™
Maybe the library should declare in the manifest what the minimum Go version is for the whole repository?

Would this involve looking at build tags?
The thing is, some libraries will have source files for both old and new versions of Go, in which case you don't need to do anything. It just works™

It would probably have to factor in, yeah - but as you note, the interesting thing about having release tags is that it's more likely to make toolchain version NOT actually matter.

Maybe the library should declare in the manifest what the minimum Go version is for the whole repository?

I think this is probably simplest. Unfortunately, it's also error-prone :(

Possibly doing the former evaluation on dep init to put the appropriate Go version in the manifest? Still a bit error prone though, as that value can change. Also not sure if it's worth the effort.
I think it's okay to demand a little more of library authors than end-users.

This is something I'm interested in as well.  For App Engine, we'd like to have some way to let users express to us the version of the go toolchain to use for builds.
Godep gives us a chance to set it here:
https://github.com/tools/godep#file-format

@JustinBeckwith help is suuuuuper welcome 😉

For App Engine, we'd like to have some way to let users express to us the version of the go toolchain to use for builds.

Yeah, this certainly is one use case. I know that @freeformz has been clear about the benefits of being able to include additional information in these files for similar purposes.
Just musing, but - I think there's (at least) two interrelated concerns, here - indicating a minimum required version for a package, and allowing users to specify an exact, or perhaps just maximum, version for...well, it'd probably have to be for a given solve run. That's probably then information we'd also want to store in the lock.

Yeah, this likely entails manifest/lock changes, so I'm reorganizing it accordingly.


toplevel out-of-order objects.

Segfault.  Looks like some extra care is needed with toplevel objects.
Repro:
(define some-object (object-maker))
;;... use some-object ...
(include "object.scm")

Need to look at reorder-fix and/or top[N] references...

This is caused by cycles in the dependency graph.
At the very least it should generate a warning.


Generated SystemVerilog includes index access on 1-bit signals

Describe the bug
If a signal is defined as 1 bit in SystemVerilog:
logic x;

Then you should not access the 0'th bit of it like this:
x[0]

However, ROHD can generate this.
To Reproduce
Access the 0'th bit of a Logic and generate SystemVerilog for it.
Expected behavior
When a signal is 1 bit, treat a 0-index into it as a nop and generate accesses as just the raw signal.  It's probably even better to just short-circuit this and return the signal itself immediately to save the extra work of a nop module.
Actual behavior
Generated verilog has x[0]-like behavior for 1-bit signals
Additional: Dart SDK info
No response
Additional: pubspec.yaml
No response
Additional: Context
This bug likely applies to both slice/BusSubset via constant indexing and IndexGate.

I can work on this one


Question: Path().write_text failed | playwright._impl._api_types.Error: TypeError | not sure if its because of my own setting

Question

[ x] I have read the comment above and have completed each step

steps:
I open python on my terminal for testing, while jupyter notebook is running in the background.
Package importing was successful, but the function call failed after running for a few seconds.
An empty SVG file was created.
As mentioned in the title, I suspect this situation has something to do with my environment.
environment:
macOS Catalina 10.15.7
python3.9 (Homebrew)
Traceback:
>>>Path("outPath.svg").write_text(trace("inPath.png", True), encoding="utf-8")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.9/site-packages/svgtrace/__init__.py", line 50, in trace
    svg = page.evaluate("(element) => element.innerHTML", element)
  File "/usr/local/lib/python3.9/site-packages/playwright/sync_api/_generated.py", line 8611, in evaluate
    self._sync(
  File "/usr/local/lib/python3.9/site-packages/playwright/_impl/_sync_base.py", line 104, in _sync
    return task.result()
  File "/usr/local/lib/python3.9/site-packages/playwright/_impl/_page.py", line 410, in evaluate
    return await self._main_frame.evaluate(expression, arg)
  File "/usr/local/lib/python3.9/site-packages/playwright/_impl/_frame.py", line 277, in evaluate
    await self._channel.send(
  File "/usr/local/lib/python3.9/site-packages/playwright/_impl/_connection.py", line 44, in send
    return await self._connection.wrap_api_call(
  File "/usr/local/lib/python3.9/site-packages/playwright/_impl/_connection.py", line 419, in wrap_api_call
    return await cb()
  File "/usr/local/lib/python3.9/site-packages/playwright/_impl/_connection.py", line 79, in inner_send
    result = next(iter(done)).result()
playwright._impl._api_types.Error: TypeError: Cannot read properties of null (reading 'innerHTML')
    at eval (eval at evaluate (:197:30), <anonymous>:1:22)
    at UtilityScript.evaluate (<anonymous>:204:19)
    at UtilityScript.<anonymous> (<anonymous>:1:44)
>>>

Hey tried this out and seems to happen if the source file doesn't exist. That is a bug and is fixed in 2023.0.1. Would you be able to confirm this when you get the chance?
Thank you :)

Here's what I tried
 % git checkout tags/2023.0.1
HEAD is now at 39c1165 release 2023.0.1
 %python3
Python 3.9.9 (main, Nov 21 2021, 03:22:47)
[Clang 12.0.0 (clang-1<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from svgtrace import trace
>>> source=trace("../img.png")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/liangzhenning/Desktop/test_f/SvgTrace/svgtrace/__init__.py", line 53, in trace
    svg = page.evaluate("(element) => element.innerHTML", element)
  File "/usr/local/lib/python3.9/site-packages/playwright/sync_api/_generated.py", line 8611, in evaluate
    self._sync(
  File "/usr/local/lib/python3.9/site-packages/playwright/_impl/_sync_base.py", line 104, in _sync
    return task.result()
  File "/usr/local/lib/python3.9/site-packages/playwright/_impl/_page.py", line 410, in evaluate
    return await self._main_frame.evaluate(expression, arg)
  File "/usr/local/lib/python3.9/site-packages/playwright/_impl/_frame.py", line 277, in evaluate
    await self._channel.send(
  File "/usr/local/lib/python3.9/site-packages/playwright/_impl/_connection.py", line 44, in send
    return await self._connection.wrap_api_call(
  File "/usr/local/lib/python3.9/site-packages/playwright/_impl/_connection.py", line 419, in wrap_api_call
    return await cb()
  File "/usr/local/lib/python3.9/site-packages/playwright/_impl/_connection.py", line 79, in inner_send
    result = next(iter(done)).result()
playwright._impl._api_types.Error: TypeError: Cannot read properties of null (reading 'innerHTML')
    at eval (eval at evaluate (:197:30), <anonymous>:1:22)
    at UtilityScript.evaluate (<anonymous>:204:19)
    at UtilityScript.<anonymous> (<anonymous>:1:44)
>>> import os
>>> os.path.isfile(f"../img.png")
True

also I've tried importing using SvgTrace.svgtrace, the result was the same.

Hmm honestly not too sure, though admittedly I do not use jupyter notebooks. Does this happen when it is not running in the background out of curiosity?

Fixed in https://github.com/FHPythonUtils/SvgTrace/releases/tag/2023.1
Note using svgtrace.trace/ asyncTrace raises an OSError as it is not supported in Windows Jupyter Notebooks. (https://github.com/jupyter/notebook/issues/5916)
However, you can use skimageTrace for basic conversion (based on edge detection)
def skimageTrace(image: Image.Image) -> str:


⚠ Bofrost.at has degraded performance

In 5fb6182, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2428 ms

Resolved: Bofrost.at performance has improved in 0a098f1.


Parties Entering/leaving cities

Verify MoveTo synchronization works for entering and leaving cities.

Works with manual testing


Addclass not working

Apologies for the newbie question, but i am trying to add a class to the #cs-content element on my page, upon start of the tour.
Tried this with onStart, which is I believe correct syntax, so is this a bug?
// Define the tour!
    var tour = {
      id: "hello-hopscotch",
      steps: [
        {
          title: "Week 1",
          content: "This week is about this new thing",
          target: "title1",
          placement: "bottom"
        },
         {
          title: "Look at this",
          content: "You can select any of the three modules",
          target: "title-step1",
          placement: "top"
        },
        {
          title: "Open the menu here",
          content: "You can select any of the three modules",
          target: "column1",
          placement: "right"
        }
      ],
      onStart: function() {
      $('#cs-content').addClass('nosnap');
      }
    };

fixed it. Newbie JS mistake


Escaping quotes breaks SPL

Basic informations

OS: Windows 11
VSCode version: 1.79.0 (user setup)
Extension version: 5.3.10

To Reproduce
It appears that escaping quotes in functions seems to break the SPL Server / Code Auto completion.
Uncommenting the PrintToChatAll function causes auto complete to stop working, and quickly causes the SPL server to break.
Code to reproduce the behaviour
https://paste.msws.xyz/?l=c#haste/pMa6TNu2
semanticTokensProviderStyling.ts:115 Semantic token with invalid length detected at lineNumber 14, column 7
warnInvalidLengthSemanticTokens @ semanticTokensProviderStyling.ts:115
notificationsAlerts.ts:42 The SourcePawn Language Server server crashed 5 times in the last 3 minutes. The server will not be restarted. See the output for more information.
c @ notificationsAlerts.ts:42

This does not reproduce anymore, so I assume it was fixed when I revamped the lexer.


Add first mkdocs setup, add contributing guidelines

Still need to test if readthedocs support the theme

btw, what should we do with circleci?


Allow non square compressed texture update support.

Allow for none square compressed texture update to be supported. The OpenSage is crashing when updating non-square compressed textures.

This looks like a good fix to me; thanks. Do you have a simple repro that we can also add as a test case to TextureTests.cs, based on what is crashing in OpenSAGE?

This looks like a good fix to me; thanks. Do you have a simple repro that we can also add as a test case to TextureTests.cs, based on what is crashing in OpenSAGE?

Great, I do not have any test code associated with it. But I can perhaps later add some more unit tests.

Great, I do not have any test code associated with it. But I can perhaps later add some more unit tests.