Preserve methods when argument and parameter types are different

When running on a method which references another method, passing in an argument which has a different type than the parameter type, the output file does not preserve the used method, resulting in unintended behavior. For example, when given this input:
import java.util.ArrayList;
import java.util.List;

class Foo {
    void bar() {
        foo(new ArrayList<?>());
    }
    void foo(List<?> baz) {
    }
}

The output only contains the bar() method, removing the foo(List<?>) method entirely:
import java.util.ArrayList;

class Foo {

    void bar() {
        foo(new ArrayList<>());
    }
}

This PR aims to fix this issue by checking the parameters of each method used within the target method with the imports, so they are not removed prematurely and eventually cause an UnsolvedSymbolException, causing the removal of the method.
I wasn't able to make sure that this didn't cause any unintended changes as I have a Windows computer (the tester does not work on Windows), so if someone could verify that, that would be great.

Got it, thanks!


expected at least 1049 bytes but there a re only 54 bytes

fail: MQTTnet.Diagnostics.IMqttNetLogger[0]
ID:,ThreadId:5,Source:MqttHostedServer.MqttServer.MqttClientSessionsManage
r,Timestamp:2019-02-13 10:02:42,Message:expected at least 1049 bytes but there a
re only 54 bytes
Parameter name: _buffer
MQTTnet.Exceptions.MqttCommunicationException: expected at least 1049 bytes but
there are only 54 bytes
Parameter name: _buffer ---> System.ArgumentOutOfRangeException: expected at lea
st 1049 bytes but there are only 54 bytes
Parameter name: _buffer
at MQTTnet.Serializer.MqttPacketBodyReader.ValidateReceiveBuffer(UInt16 lengt
h)
at MQTTnet.Serializer.MqttPacketBodyReader.ReadWithLengthPrefix()
at MQTTnet.Serializer.MqttPacketBodyReader.ReadStringWithLengthPrefix()
at MQTTnet.Serializer.MqttPacketSerializer.DeserializeConnect(MqttPacketBodyR
eader body)
at MQTTnet.Adapter.MqttChannelAdapter.ReceivePacketAsync(TimeSpan timeout, Ca
ncellationToken cancellationToken)
--- End of inner exception stack trace ---
at MQTTnet.Adapter.MqttChannelAdapter.WrapException(Exception exception)
at MQTTnet.Adapter.MqttChannelAdapter.ReceivePacketAsync(TimeSpan timeout, Ca
ncellationToken cancellationToken)
at MQTTnet.Server.MqttClientSessionsManager.RunSessionAsync(IMqttChannelAdapt

Hi,
please provide some more information like sample code or trace output.
The plain exception it not really helpful.
Best regards
Christian

for me it looks like a malformed mqtt packet where some string fields length is bigger than the actual message

In fact, I haven't found any valuable data at the moment, which only happens in a particular network environment, and it's not going to recur at the moment.

If you find more data or if this problem also appears with latest dev and/or preview nugets of 3.0.0 please let me know.


BLE RPC configuration

You need a sauce tag on the new commit

which it looks like should be "fromlist" -- agree @carlescufi ?

I will close it. I used it only for test purposes. I don't know why you were assigned to it


Prevent Play Mode from happening if there are import (compile) errors

Fixes #103

Please check if the pull request fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features) -- TODO: I have no idea what I could do to test
[ ] Docs have been added / updated (for bug fixes / features) -- Not needed; diagnostic (non-functional)
[x] CHANGELOG.md has been updated to describe this change

To update the documentation on yarnspinner.dev, please visit the documentation repository.

What kind of change does this pull request introduce?

[ ] Bug Fix
[x] Feature
[ ] Something else

What is the current behavior? (You can also link to an open issue here)

Currently the importer does not prevent play mode, nor does it refresh the message, which means that you will not be able to pinpoint where the error came from after the initial import. This is fine with a small amount of files, but when the projects becomes larger, having errors in a single location becomes increasingly important.

What is the new behavior (if this is a feature change)?

While there's no way to 100% match Unity semantics for import compiler errors ( >:( ) we can very closely approximate it.

We flash up a toast notification when trying to enter Play Mode:

We cancel play mode, which means the console messages are not cleared.
We can't manipulate log entries directly like Unity C# compiler errors (that class is internal to the core assembly unfortunately) so we emit the errors every time play mode is accessed.

One thing to note is that we will change the error message in the import window to include the file name:

This is technically redundant, but it doesn't get in the way. It's necessary so that our error messages have the correct context to assist the user in finding the correct file.
The way it is designed also means that the registered imports will be lost on domain reload, but this is ok because if domain reload happens, 1) it's probably because another import happened, not because you edited the script, 2) play mode was entered (no errors), 3) files can't magically gain errors without another import happening. So I think it should be fine.

Does this pull request introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:

I haven't fully QAed it because it's like 2AM and I really need to go to sleep, but as far as I can tell it's perfectly functional.

(Please ignore the 'checks have failed' note on this PR; I'd set up automatic building for pull requests, but forgot that PRs don't have access to GitHub secrets, so the build process fails to acquire a Unity license.)

Not really sure what unit tests I can add -- ideally I'd like to test domain reloading but there doesn't seem to be any way to trigger total domain reload.


Integrating with Mayhem

Pull request to submit changes for Mayhem heroes

Edits made- I believe I reverted the go.mod and go.sum in the enola package, and added my own in the fuzzing. My justification for updating the version of go in the docker package is there, and I removed the entrypoint from docker and the space from the target in the mayhemfile.

I believe I've done the updates and added a package stage to the dockerfile.

Added docker file, triple checked that the root-level go.sum is unmodified. It really shouldn't appear as modified, all checks that I've done have said their the same.


client: add debug env var to print selected text

I'm not sure I understand the purpose of this. Could you please explain the rationale? How is this useful for debugging, as opposed to just doing it yourself?


[Subtask] support UpdateColumnType in AlterTable for spark connector

Describe the subtask
please add related code to com.datastrato.gravitino.spark.catalogGravitinoCatalog.java, transfrom Spark TableChange to Gravitino TableChange, you could refer to #2458
  public Table alterTable(Identifier ident, TableChange... changes) throws NoSuchTableException {
    throw new NotSupportedException("Doesn't support altering table for now");
  }

Related Spark SQL should add to SparkIT.java, please refer https://spark.apache.org/docs/latest/sql-ref-syntax-ddl-alter-table.html
Parent issue
https://github.com/datastrato/gravitino/issues/1227

Hi @hiirrxnn  i would like  to confirm again that are you still working on the  UpdateColumnPosition and UpdateColumnType mentioned in #1565

i will work on this.


better error messages with context

Using pandoc with pandoc-citeproc often yields errors that are hard to trace back to source (=bibtex) files. The error messages in pandoc look like this:

pandoc-citeproc: "stdin" (line 241, column 2):
unexpected "M"
expecting "c", "C", "p", "P", "s" or "S"
CallStack (from HasCallStack):
error, called at src/Text/CSL/Input/Bibtex.hs:121:32 in pandoc-citeproc-<IP_ADDRESS>-HIfBMT1iu3x9mOf0xuZHLo:Text.CSL.Input.Bibtex
pandoc: Error running filter pandoc-citeproc
Filter returned error status 1

Since I have multiple, large bibtex files, that I generate automatically and include with --bibliography= to pandoc, it is hard to find the error, since the context of the parsing fail is missing:

stdin > so there is no filename
no context of the failing bibtex entry text

Finding and correcting failing/erroneus entries is very hard this way.
I would like to see some of the parse context (lines before/after failing read) to identify the bibtex entry based on its content (e.g. based on bibkey, title, authors or any other fields).
Basically any error in a bibfile can lead to a bail out. I might currently be seeing errors due to @comments (as in #306), but I am not sure about it.
Cleaning up bibfiles is more or less luck, and usually only feasible, if the latest added entries fail and therefore can be found by log/date/diff.
I usually call pandoc from script or commandline with quite a few options, at least like this and with multiple bibliography options:
pandoc --latex-engine=xelatex -fmarkdown+autolink_bare_uris+lists_without_preceding_blankline --template=/path/to/t.tex --bibliography=/path/to/bib --csl /path/to/csl -o out.html in.md

What version of pandoc?
With recent versions you should be getting the filename, at least.

OK, my pandoc was a bit outdated (installed via package manager on ubuntu 18.04: bionic:pandoc packages): pandoc <IP_ADDRESS>, pandoc-citeproc <IP_ADDRESS>
With a freshly compiled pandoc-citeproc (pandoc 2.2.1, pandoc-citeproc <IP_ADDRESS>) the filename of the bibfile is given and that points to the error precisely, just as you explained.
So an upgrade to a recent version solves this issue, sorry for the noise.
I just stumbled across a different error, where a filename is not given. Bailout due to non-utf8 encoded bibfiles just names the reason, not the file:

pandoc-citeproc: Cannot decode byte '\xf6': Data.Text.Internal.Encoding.decodeUtf8: Invalid UTF-8 stream
return/exit value 83

OK, encoding could be checked with external tools (e.g. file guesses it), if you have the list of input files (in my case a bash script adds multiple bibfiles, so I can look at the verbose output of the bash execution). Anyway, it would be nice to have a kind of verbose flag for pandoc/citeproc that lists the processed file(s) or input. This might be a different issue, since the original issue is solved.
Thank you for looking into this and the great software!

It's a different issue. If you want to create a separate issue for this, go ahead.


monaco init command for project initialisation

Common feedback about Monaco is that people don't always know how to get started with the folder structure.
A monaco init command should be available that creates:

Sample environments.yaml file
Project directory
For each supported config type, a folder should be created with a base yaml in there
After the command executes it should print the exact command needed to apply that project

Would empty files be OK inside the folders? If so, I have some code.

@Dynatrace-Adam-Gardner this is great stuff! For the yaml file, do you have the base structure there as well, something like this:
`config:

application: "application.json"

application:

name: "application-one"`

That would make it super easy.
If then that application.json also exists, all the user has to do is paste in the json body from the api explorer and they are good to go!

Done, this now creates a YAML file with content like:
config:
- alerting-profile: alerting-profile.json
alerting-profile:
- name: alerting-profile-one

From the above it is not clear if it creates a sample environments.yaml as well?

It doesn't yet, but I'll add it

This is now complete. It generates files and folders for all APIs returned by the api.NewApis() which references the apiMap in /pkg/api/api.go so new APIs will automatically get folders and content.
One important note: You must delete folders that you're not using otherwise monaco will error.
Perhaps it's worth ignoring blank files? I'll start a new discussion for this.


Using healthchecks component from Vert.x #39

Coverage increased (+0.003%) to 98.639% when pulling 91081babe6194fafeb50f9323540df5398bb4f1b on issue_39 into 4b2c90bbaecc7abaefad965b72ede646e46da0d8 on master.


Counter overflow inconsistency in ChaCha20

When the 32 bit block counter overflows, the nonce is incremented in some of the implementations.
This isn't a problem for OpenSSL's use of the algorithm but applications that set the counter to a value that might wrap could be a problem.

@paulidale could you please elaborate on what the proposal is here?  Is it that the nonce should never be incremented on a wrap, or something more subtle?

ping @paulidale


matmul and dot

I've been looking, but I didn't find implementations for those two.
Is there a plan to leverage e.g. BLAS or similar libraries for those operations?
- Wolf

We have a plan to support

dot
tensordot
einstein sums.

The latter with a template-based syntax, instead of strings à la numpy.
The use of SIMD-accelerated versions of blas for certain memory continuous versions is something we have experience with, but we  don't have a general approach for tensor / einstein products.

@SylvainCorlay any update about this feature?

@edgarriba The dot and tensordot will probably be added in the version 0.9. We have urgent features to release in the next version (0.8) before we can focus on dots.
SIMD-accelerated support will be added after version 1.0 has been released.

Also, we make release about every other week. Some of the foundations for how we want to make tensor dots were introduced in 0.7.

@edgarriba I saw the discussion on tiny-dnn about adopting xtensor. Is this a blocking point for you guys?

@SylvainCorlay not by the moment since we are still evaluating xtensor as our tensor backend. However, we have some codes were we have implemented dot by hand, so we could use them in the meantime.

Also about batched interfaces and http://www.netlib.org/utk/people/JackDongarra/WEB-PAGES/Batched-BLAS-2016/Day1/05_Ahmad_vbatched.pdf

Ongoing work in xtensor-blas for the curios.

xtensor-blas has been released.


controllers

Work on homeroutes.js;

Having trouble calling {session} into the code, can't figure this out

Reverted everything to express-session functions. Seems to be working fine for now. Only issue is line 7, when I uncomment the withAuth part, the localhost/webpage crashes due to TOO MANY REDIRECTS. Not sure why this is.


Remove errant whitespace

A gem this beautiful shouldn't be marred with misaligned code. :)

At least make a functional improvement in your commit, you bum! :hankey:

Functionally it is pristine? How could I improve on something so sublime?

Plus you're always telling me to keep the formatting changes separate from the functional ones. So don't be throwing :hankey: at me when I'm just following your own guidelines.

Someone has to keep your rampant pedantry in check!
Dean
On Fri, Oct 30, 2015 at 10:45 AM, Scott Haug<EMAIL_ADDRESS>wrote:

Plus you're always telling me to keep the formatting changes separate from
the functional ones. So don't be throwing [image: :hankey:] at me when
I'm just following your own guidelines.
—
Reply to this email directly or view it on GitHub
https://github.com/kerrizor/fizz-buzz/pull/3#issuecomment-152599165.

18 months later...


fix(datepicker): inconsistent focus restoration timing in touchUi mode

MatDatepicker has focus restoration which happens (almost) synchronously, whereas MatDialog restores focus asynchronously when the animation is finished. This means that in touchUi mode focus ends up being restored both by the datepicker and the dialog within a 150ms window. The problem is that if the consumer of MatDatepicker decides to move focus inside the closed event, it'll be ovewritten by the dialog 150ms later. These changes disable the dialog's focus restoration because it's unnecessary.
Fixes #17560.

Rebased.


Flag user during event

It might be nice to have a quiet way of signaling the event organizer that there's someone inappropriate/that you just dated someone inappropriate.
Then the event organizer can choose to deal with this as they see fit.

Also some sort of application wide ban, though with our current Auth-Setup that's not going to do much.


The bridge you looking for does not exist. It should be at path ...

Hey there,
since a few commits (I'm always on master), I have the weird Issue, that all my bridge feed URLs don't work anymore. Example:
Link to the bridge that I used (for like forever): https://my.host.name/rss-bridge/?action=display&bridge=SoundcloudBridge&u=conscious-embassy&format=AtomFormat
Error message:

The bridge you looking for does not exist. It should be at path /var/www/my.host.name/rss-bridge/lib/../bridges/SoundcloudBridgeBridge.php

If I use the current UI to get the feed URL, it returns: https://my.host.name/rss-bridge/?action=display&bridge=Soundcloud&u=conscious-embassy&format=AtomFormat (notice: bridge=Soundcloud instead of bridge=SoundcloudBridge). With this URL it actually works.
That's a rather breaking change, don't you think? This basically means I have to adjust EVERY (rss-bridge) feed URL inside of my feedreader. Is this an error or am I missing something?
Cheers & Thanks
Thomas

@LogMANOriginal https://github.com/RSS-Bridge/rss-bridge/commit/e26d61ec0ac0659244ac68ccb0f832e89f043c7a#diff-828e0013b8f3bc1bb22b4f57172b019d
See lines 115-119 in previous version of index.php

Thanks for reporting this issue!
@em92 is correct, the issue was likely caused by that commit. Actually, bridge names are now sanitized from within the Bridge class: https://github.com/RSS-Bridge/rss-bridge/blob/dd95ec6200df1cfbcf0ef0bce06e167a72ed7c41/lib/Bridge.php#L136-L167
Technically lines 149-152 should take care of this:
https://github.com/RSS-Bridge/rss-bridge/blob/dd95ec6200df1cfbcf0ef0bce06e167a72ed7c41/lib/Bridge.php#L149-L152
I'll see what I can do.

Should be fixed!

I can confirm that! Thanks! :)
Cheers
Thomas


Change direction

Is it possible for me to change the swipe direction?
Currently when I swipe to the right it moves to next page, I need to go to next page by swiping to the left.

no, it's not possible presently but you can fork it and alternate the booleans in lines
https://github.com/chitezh/react-native-swiper-animated/blob/master/src/Swiper.js#L279 and https://github.com/chitezh/react-native-swiper-animated/blob/master/src/Swiper.js#L282
Or even better, use a prop to control that

Thanks you for getting back to me so quickly~ forked it and done it.. Thanks!


Replace bundlesize with bundlewatch

Unfortunately bundlewatch also doesn't seem to be super actively maintained however at least the server-side service seems to be working at the moment so we git diffs w.r.t. master directly in the Github check.
Since I can't find any drawbacks w.r.t. bundlesize I'd give it a try.

I'm watching http://npm.broofa.com/?q=@ampproject/filesize as well which has an even smaller dependency graph, however it doesn't support reporting differences w.r.t. master out of the box yet.

BTW apparently during the last few hours there was a problem on GitHubs side with verifying signed commits which has now been resolved, however commits need to be rewritten in order for GitHub to pick up the changes. This is why I had to rebase once more…


pull request for e2e test at 11/08/2016 10:26:04

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

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.


Add demo for qwen_doc_search_QA based on llama_index and bce_embedding

Change Summary

Add demo for qwen_doc_search_QA based on llama_index and bce_embedding. LLM qwen1.5-4b-chat is used in the example.
Related issue number

fix #386
Checklist

[ ] The pull request title is a good summary of the changes - it will be used in the changelog
[ ] Unit tests for the changes exist
[x] Run pre-commit install and pre-commit run --all-files before git commit, and passed lint check.
[ ] Some cases need DASHSCOPE_TOKEN_API to pass the Unit Tests, I have at least pass the Unit tests on local
[ ] Documentation reflects the changes where applicable
[ ] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review


Relationship with polymorphic: true doesn't set base_class in field_type

I have the following code:
class Parent < ActiveRecord::Base
  include Tenacity
  t_has_many :cars, as: :owner
end

class Child < Parent
  attr_accessible :type
end

class Car
  include Mongoid::Document
  include Mongoid::Timestamps
  include Tenacity

  t_belongs_to :owner, polymorphic: true
  attr_accessible :owner
end

Running the command on console: car = Car.new(owner: Child.new) I have the result:
car.owner_type: "Child"
But the expected behavior from polymorphic using ActiveRecord is:
car.owner_type: "Parent"

Did you solve it?


Dtyi edits

This is my last work for tonight, complete up to the last code block (In [7] in solarsystem).

First note-- When you git add a whole folder, you also end up adding lots of undesirables, such as auto-backup files (hello.py~ and the like).  Please git rm these files and push this branch again.  In the future, you should be more explicit about what files you want to add; e.g. git add dtyi/*.py, etc.

sentence.py
Great-- Just one note here, so you know----you can say while(word not in escape_chars) if you want, which is a bit more rolling-off-the-tongue than while(not word in escape_chars).
sum_digits.py
Nice-- for this one, the only note to make is so that you know integer division with a single / is only default in Python 2; in Python 3, 5/2 will equal 2.5.  So it's better to get used to forcing integer division with // so 2//5 will always equal 0, not dependent upon which Python is being run.
wordfreq.py
Ooh... big no-no--- don't use dict as a variable name--- while not an officially reserved word, it's the name of the type itself!  That is, one of the ways you can construct a dictionary is d = dict(a=2, b=3).  So definitely give that a new name before I pull this in.
Also, instead of using f.readline(), a common pattern to loop through a file is:
for line in open(filename):
     [do stuff]

Also, a good function to know is the string split method, e.g.: words = line.split(), which automatically splits on whitespace.
solarsytem.py

Style comment:  when splitting commands over multiple lines, following lines should be indented relative to the code block (see here).

For putting variables in strings, what you've done works (using the % sign), but also checkout the .format() string method.  That's a bit more common these days, and has significantly more functionality, so I recommend you become familiar with it.

In the System constructor, don't have an empty string (or any other mutable object) be the default value of a keyword argument.  I don't remember exactly why, but nasty things can happen.  Instead, give it a default value of None and have a if planets is None test within the method to set it to [] if need be.

When you're building strings, you can use the += operator.  Also another big no-no, don't use str as a variable name!  Same issue as dict above.

Otherwise, looks great!  Make these touch-ups and I'll pull it in.

Oh also-- I don't see a single documentation string.  Please put documentation strings in all functions & class definitions, even if minimal.  Good habit.


Unable to setup tables.

When I click "setup tables" for the Pathfinder database, the page refreshes but no tables are created (verifiable my loading the mySQL database).
With debugging set to 3, there is no related error in the log for mysql, nginx, or pathfinder.
I had previously created a  database with the same name ("pathfinder"), but for certain reasons I needed to delete it and remake it. Upon re-creation, with nothing changed, the app is unable to setup tables.
I suspected some sort of caching issue, but upon clearing /tmp/cache, the problem persisted.
I have also attempted to change the name of the db to be targeted in environment.ini and the same problem occurs.
/setup page shows both DBs (pathfinder and SDE db) connecting properly.

Seems related to #483

Hi @avree I think I may have found why this happens, on a standard nginx install with php the try_files thing is modified to include the php file&args, that is usually done by simply putting replacing nginx's default with something like this: try_files $uri $uri/ /index.php?$args; however, this doesn't seem to work for everything. I then noticed that exodus4d uses a different try_files on his .conf file. so try and replace whatever you have on you try_files in nginx with try_files $uri $uri/ /index.php?$query_string;


IntelliSense does not work in new "Native Built-in Interactive Window"

Environment data

VS Code version: 1.58.0
Jupyter Extension version (available under the Extensions sidebar): 2021.8.1013163132
Python Extension version (available under the Extensions sidebar): 2021.6.944021595
OS and version: Windows_NT x64 10.0.19041
Python and/or Anaconda version: Python 3.8.8 | Anaconda 2021.05
Type of virtual environment used: conda
Jupyter server running: Local

Expected behaviour
IntelliSense should work in the new interactive window like it does with the old interactive window, show below:

Actual behaviour
IntelliSense dow not work in the new interactive window, show below:

Steps to reproduce:

Configure "jupyter.enableNativeInteractiveWindow": true
Reload window in VS Code to apply the setting.
Run some piece of code in the interactive window with object definitions, import packages, etc.
Try to access objects atributes or packages functions in the interactive window with "."

@Diogo-Rossi IntelliSense for the built-in interactive window is available in the Python extension's Insiders Daily builds. If you run the 'Python: Switch to Insiders Daily Channel' command from the command palette, that should download a version of the extension with preliminary IntelliSense. Please let us know if that works for you 😊

@joyceerhl Yes, that worked!
So, for now the "Default Channel" does not have it yet, am I right?

That's right. The Python extension will be releasing a stable build with IntelliSense for the interactive window sometime next week.


Script does not get executed

The addition of
# Call the function Install-VSCode-Windows does not exist in the script and when I add it manually  i get error: Install-VSCode-Windows : The term 'Install-VSCode-Windows' is not recognized as the name of a cmdlet, function, script file, or operable program.

There is an error in the README. The function should be Install-VSC-Windows. I will correct it in the next pull request.

Changes have are now in the main branch


WIP : Implement a sortedmulti_a descriptor

This is mostly the copy of sortedmulti for now with some changes in the Terminal being MultiA instead of Multi and some max_satisfaction_witness_element and max_satisfaction_size as implemented in core.
Some Question that I have and I think we can implement it on the way in this PR.

How can we build_template and what exactly is a template?
How to calculate the script_size of sortedmulti_a.

Some other questions that I have about implementing sortedmulti_a inside tr like tr(c6...., sortedmulti_a(2, 5f...., 6e....). For now we build a taptree and consider the trees inside the tr like : tr(c6..., {pk(), pk()}. This means when I trying to build a tree for tr(c6...., sortedmulti_a(2, 5f...., 6e....) I am hitting an error with
`Result::unwrap()` on an `Err` value: Unexpected("sortedmulti_a(3 args) while parsing Miniscript")

One solution that I for now think is to implement a TrInner as we have inside Sh(). Am I on the right path?

@sanket1729 @apoelstra sorry for the ping. But can you please tell me if this is the right way of approaching the porblem or is there a better aproach to this?
Thanks.


Integration with Vuejs: not detecting translations

Hi there,
I have a bunch of projects with Vuejs waiting to be translated by some translators.
I set up everything on one of them, however no translations are found when I run the CLI. It won't detect any translations in the template or script tags in a .vue file, even in a .js.
As it doesn't work out of the box, I created a quick plugin that works when I initialise, but does not detect any translations:
import Vue from 'vue'
import { tx, t } from '@transifex/native'

Vue.use({
  install (Vue) {
    Vue.tx = Vue.prototype.$tx = tx
    Vue.tx.init({
      token: '[REDACTED TOKEN]'
    })
    Vue.t = Vue.prototype.t = t
  }
})

My wish is obviously to use things like this.t() or something similar on a <script> tag, and {{ t() }} on a <template> tag.
In another file I set the lang with setCurrentLocale.
Any idea how I can localize my Vuejs app by using the cloud-based localization solution with Transifex?
Thanks!

The issue is related to the fact that @transifex/cli is using Babel, doing AST analysis on the code, looking for t functions and locating phrases.
So for plain .js files it should work, if you are using the code as t('Hello') and not this.t('Hello'). As a matter of fact, tx and t are singleton objects, so no need to bind them to this.
I am not familiar with Vue, but I think we could probably investigate adding vue-template-compiler in the @transifex/cli package, and update it to be able to parse Vue templates but it will requiring some digging, or even better a full fledged @transifex/vue package.
But in the meantime, one (intermediate) solution you can use to unblock you, is create a temporary phrases.js file and copy the phrases you want to push to Transifex like:
t('Hello');
t('Another phrase');
...

then, push the phrases using npx txjs-cli push phrases.js.
Assuming that the quick plugin you created works, you should see your templates translated when running the code.

Thanks @nbasili! I will dig that.
I am surprised Transifex doesn't have a @transifex/vue package, as vue is pretty much very popular too!
Either way, thanks for your quick reply.


Rails 4 ?

turbo-sprockets-rails4 ???

We upgraded our app to Rails 4 last year and were pretty disappointed at how slow asset precompilation was, especially considering the speed and caching issues were supposed to have been solved. We ended up writing some code to precompile assets in parallel, which sped up precomp by quite a bit. I just extracted and published the code as turbo-sprockets-rails4, which is available here. Your mileage may vary 😉


nnfs c++ part 6

Hi,
Thank you for your submission!
I noticed that there are 2 files, possibly because you tried to rename it? Also could you keep the file naming from Python's implementation?
Additionally, @shreeviknesh - are you going to continue with your implementation, if not, there is also @muha0143 who made submissions to the C++ branch - are you going to continue here? If not, I'll accept @HippozHipos 's solution.

It could be because I tried to rename it, this is my first time doing anything in github, so I don't really know what I am doing, sorry. I have edited the name to the the same as the video name (part 6) and the code is better overall as well (improved variable names and comments).


Update helm chart

Run as non privileged user

Adds securityContext with a user id (8888 by default)
Changes ports to 8080, 8443
Adds config to ensure service_port setting is added as its not the default 80 now

Add exporter.enabled setting and set to false to not include statsd exporter by default
Fix single namespace typo - needs to be string

Sorry for the delay getting back to you on this one! I like the direction here, but I think we need to keep the default prior to 0.50 as running as root, listening on 80/443. Easy for you to tweak this to preserve the default but allow easily deploying as non-root?

I think we can close this as the new helm/charts chart covers this :)

I think you're right. @cliveseldon, many thanks for getting this rolling way back when! Please check out the new Ambassador chart in helm/charts. :smile:


Test Proxy asset transition script should enforce minimum test proxy version

I just tried running the asset transition script, and since I've been using/testing proxy via docker, my actual test proxy binary version is many months old. This resulted in the restore command "hanging" because the sub-command was ignored and it started just running the test server instead (but the logs were swallowed so it just appeared to hang).
We should check a minimum version via test-proxy --version and log a link and/or command to update to the latest version (or install it automatically).
CC @scbedd @JimSuplizio

Minimum test-proxy version should be 1.0.0-dev.20221007.3 IMO.
This is the latest version that has the fixed buffer problems AND Jim's --tags fix.

Minimum test-proxy version should be 1.0.0-dev.20221007.3 IMO.
This is the latest version that has the fixed buffer problems AND Jim's --tags fix.

@scbedd We're going to need the test-proxy that uses the CLP version changes that were checked in today in order to do the version check. Pre-CLP version, the --version will spit out the version but also start the proxy which we don't want here.


You do not have the rights to carry out this action.

even though I have the roles "Admin, Content Manager" and I see "TV Shows" & "Movies"...
EDIT: i have already opened sooo many issues :laughing:

Where? What are you doing?

I am on the admin page and clicking "TV Shows" or "Movies" and I get the error message in the lower right corner

oh yea.. that is a springsecurity "feature", changing roles does not apply until you logout and back in.

and I relogged an I have two "Admin" buttons on the main page... lol

yep that is fixed, like 2 minutes ago

... :D

does it work if you log out and back in?

trying

double admin is gone

and i can acess tv shows & movies, either it works directly, or add a warn message!

yea.. good idea. or forcing a logout.

but please with a warning ... :)

okay anyways, good night! :D

fixed with https://github.com/dularion/streama/commit/8893cba44ce455c0027030595d7d7b322243799b

Fixed in a README.MD commit? Lol

haha oops
https://github.com/dularion/streama/commit/3de424b1c2910c1eb04a3c4d8b578630b572d4ba


Context "I am logged in as an administrator" failing

After installing a new project, and launching all the commands, then running the behat tests with Selenium, the context "I am logged in as an administrator" doesn't seem to be working.
I can see that a user is created in the database, but I am not logged in : the 3 example scenarios accessing the admin part of the website are failing because I am redirected to the login page.
Expected to be on "http://localhost:8080/admin/products/new/simple" but found "http://localhost:8080/admin/login" instead (Sylius\Behat\Page\UnexpectedPageException)
Is there a configuration that I am missing somewhere ? For the sessions ? The cache ? Something else ? The weird thing is that if I launch the website on http://localhost:8080, I can log in, and go to the given pages without problem.

Thank you, Camille, for reporting this, let the discuss happen in https://github.com/Sylius/PluginSkeleton/issues/138 :)


Add a claimable balance feature

When sending a payment to a user's contact, if the contact doesn't have a trustline for that asset the transaction will fail. That's because we're only using the payment operation. We should detect this case, and offer to create a claimable balance for the user.

@ElliotFriend can I work on this

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I am a Full Stack developer with 6 years of experience in the industry, I am super excited to be able to contribute to Web 3 projects, this to continue paving my way as an Open Source Contributor. I am from Dojo Coding Community
How I plan on tackling this issue
Detect Missing Trustline Before Payment:
Before sending a payment, check if the recipient has an established trustline for the asset. If the trustline is missing, proceed to the next step.
Offer to Create Claimable Balance:
If the recipient does not have a trustline, prompt the user with an option to create a claimable balance. This would allow the recipient to claim the payment once they establish the necessary trustline.
Fallback to Claimable Balance Creation:
Upon user confirmation, automatically create a claimable balance instead of proceeding with the regular payment operation. This ensures that the transaction won't fail, and the recipient can claim the asset later.
Provide Clear User Feedback:
Communicate clearly with the user, explaining why the payment could not be completed directly and offering the claimable balance as an alternative solution.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I would like to take this task, and it will be delivered within (26, September, 2024-30, September, 2024). I am a programmer who has intensive knowledge in Python and Javascript. I have worked on a lot of problems which has given me insights in detecting problems and coming up with optimized solutions.
How I plan on tackling this issue
I would  use an if statement to check if the user's contact doesn't have a trustline and if  this is the case; the if block would handle it by creating a claimable for the user.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Hello @ElliotFriend I am Joseph I am a Software developer and blockchain developer and I am also an active contributor here on only dust here is my profile https://app.onlydust.com/u/josephchimebuka. This is my first time to contribute to this repo ill appreciate the opportunity to contribute.
How I plan on tackling this issue
I will ensure the issue is solved by adding a claimable balance feature I will complete this in 2-3 days

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I am a smart blockchain developer.
I can leverage on my background to solve this issue by implementing an automated trustline check and Claimable Balance creation system.
Design a smart contract that verifies the recipient's trustline status before initiating a payment, and if absent, automatically generate a Claimable Balance.
This approach combines your knowledge of state management and conditional execution from smart contract development with the Stellar network's Claimable Balance feature, providing a seamless and efficient solution to the trustline problem
How I plan on tackling this issue
I will take the following steps;
1. Implement a smart contract that checks if the recipient has a trustline for the asset being sent.
2. If the trustline exists, proceed with the normal payment operation.
3. If the trustline doesn't exist, automatically create a Claimable Balance instead.
4. implement periodic cleanup of unclaimed balances to manage network efficiency.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Please can I be assigned this issue? I'll love to contribute to this project please. I am a blockchain Developer, and my experience includes html, css, react, JavaScript,TypeScript and solidity, python and Cairo.
How I plan on tackling this issue
To solve this issue, I'd take the following steps:

I'll detect trustline error when the payment fails.
I'll prompt the sender to create a claimable balance if the recipient lacks a trustline.
I'll create the claimable balance using Stellar SDK, allowing the recipient to claim the asset once they add a trustline.
I'll notify the recipient to add a trustline to claim the balance.

Please assign me.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Hi my name is Joseph Gimba a computer engineer graduate that has passion for web3 and blockchain development, I have experience in writing smart contracts as well as using Javascript to a create Frontend for my projects built.
How I plan on tackling this issue
Using try & catch if the user does not have a Trustline then the catch block prompts the user to create a claimable balance to be able to receive payment. another approach would be asking the user if he wants us to automatically create a trustline for him automatically to receive the funds.

@ElliotFriend  Hi, can you give me more details about this issue? Steps to reproduce the use case?

Hi, there @PabloVillaplana! Thanks for taking the issue on! Hope all this info helps, and if not feel free to reach out and ask more questions!
The current state of BasicPay allows sending a regular payment operation of any asset that your account holds. That will work as long as the receiving account has a trustline for that particular asset. But, in the case where the recipient account does not have a trustline, the regular payment operation will fail.
So, we should do something like the following:

We're already checking if the account exists when we select it for a payment in /src/routes/dashboard/send/+page.svelte, so we should hook into that logic to get the recipient account's balances array.
When selecting the asset to send, see if the recipient holds the asset we're trying to send
If not, give the user an opportunity to select "create a Claimable Balance instead"
If they choose this option, we should add both the recipient account and the user's account as claimants on the CB (in case the user needs to claim it at some point in the future).
I'm fine using unconditional claim predicates for both of these, just for simplicity's sake.

Helpful Docs and Links

claimable balance docs
trustline docs
Create Claimable Balance operation docs
Stellar Quest 0103 - Change Trust

Note about Stellar Quest: The code examples might be a little outdated, compared to current SDK versions/syntax. But, the concepts and principals should still be useful in researching how to accomplish the task.

Stellar Quest 0303 - Claimable Balances

See note on above item

Hi, @PabloVillaplana! Thanks for the interest in this issue. Since I haven't seen a PR yet for this issue, I'm going to un-assign it and make it available for ODHack9. You should definitely feel free to apply again, though!
Thanks again!

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I am applying to this issue via https://app.onlydust.com/hackathons/odhack-90.
I am Collins Cxaton, a fullstark developer with a strong background in integrating APIs and designing responsive UI components based on Figma designs. I have experience in working with cryptocurrency wallets and displaying real-time wallet balances by integrating financial APIs. My skills in React and JavaScript allow me to effectively manage currency conversions and data handling.
How I plan on tackling this issue
First, when a user attempts a payment, check if the recipient has a trustline for the asset. This can be done by querying the user’s account and assets.
Integrate a conditional check before initiating the payment operation.
If the recipient lacks a trustline, prevent the payment and display an appropriate message to the sender; if If the recipient lacks a trustline, offer the sender the option to create a claimable balance for the recipient. And Implement a function that allows the sender to create a claimable balance, using an appropriate API or smart contract method.
lLasty, notify the sender when a claimable balance is created; also i will create a test case and update the documentation

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
would love to work on this

I’d like to resolve this.

My background and how it can be leveraged:
I'm a Computer Science student with experience in JavaScript, and I’m eager to apply my skills to web3 projects. While this is my first time working with decentralized systems, I have a solid foundation in software development and problem-solving. My experience with JavaScript will be especially useful when working with payment flows and handling the detection logic required for this issue.
How I plan on tackling this issue:
1.I will review how payments are currently processed and identify where the transaction fails when the contact lacks a trustline for the asset.
2. Using JavaScript, I’ll add logic to detect if the contact doesn’t have a trustline for the asset before attempting the payment operation.
3. If the trustline is missing, I will provide an option to create a claimable balance for the user instead of letting the transaction fail.
4. I will test the new functionality to ensure the flow works smoothly and no other parts of the payment system are affected.
5. Lastly, I will submit a pull request explaining the changes, ensuring the payment process is more resilient.

Would love to tackle this!

I am a Full Stack Developer specializing in Next.js, TypeScript, Node.js, Cairo and Rust . With over 31 contributions across projects in the OnlyDust ecosystem, I’ve developed strong proficiency in delivering high-quality solutions and resolving complex issues within tight deadlines. My experience spans frontend, backend, smart contracts, and the optimization and maintenance of scalable codebases.
To implement the claimable balance feature, I would approach it with the following steps:

The first step in the process would be checking if the recipient has an active trustline for the asset being sent. This can be done by querying the blockchain to see if a trustline exists between the recipient's account and the asset in question. If the trustline exists, the payment proceeds as usual. If it doesn't, the system would trigger the creation of a claimable balance.

If no trustline is detected, the system would present the sender with an option to create a claimable balance instead of immediately failing the transaction. This can be handled at the user interface level, where the user will be informed that the recipient lacks a trustline and offered the ability to send the payment as a claimable balance.

If the user chooses to proceed, the system will generate a claimable balance operation. This involves creating a transaction that specifies the recipient as the claimant, with the condition that they must establish a trustline to claim the asset. In Stellar, the CreateClaimableBalance operation can be used to hold the asset in a special balance until the recipient meets the claim conditions.

Once the claimable balance is created, the recipient should be notified that there is a pending balance waiting for them. This notification can be triggered through the platform's notification system or by a more direct message like email. The recipient will be instructed to set up a trustline for the specified asset to claim the balance.

To implement this, the user interface will need to be adjusted to account for both the detection of the missing trustline and the offer to create a claimable balance. Also, the user should be able to view pending claimable balances and manage the process from the app, giving full visibility into the status of the transaction.

Finally, thorough testing will be essential to ensure that this feature works across different assets and account types. Special care should be taken to handle edge cases, such as when the recipient establishes a trustline but does not claim the balance promptly or when a recipient fails to set up the trustline altogether.

Is it okay if I tackle this?

Can I start working on this?

Can I take this from here?

Could I take a shot at this?

Can I handle this task?

I’d like to work on this, im pretty sure it will be over in a day

Can I contribute to this one?

hey @fricoben Can I contribute to this one? Let me handle this issue!

Mind if I try this one?

I'd like to take this issue.

May I try my hand at this?
About me:
Software Engineer with 10 years of experience. For the past 5 years, I’ve specialized in HR Tech solutions. I’m passionate about exploring new technologies and currently looking to deepen my expertise in blockchain, smart contracts, and decentralized applications. I have also contributed to open source development at OnlyDust (My OnlyDust Profile).
How I plan on tackling this issue:
Before sending a payment, I'll check if the recipient has a trustline for the asset. If not, I'll prompt the user to create a claimable balance instead of the regular payment. Upon the user's confirmation, I'll automatically create the claimable balance, ensuring the transaction doesn't fail. I'll also provide clear feedback to the user, explaining why the direct payment couldn't be completed and offering the claimable balance as an alternative

Can I start working on this?

I'd like to take this issue.

Can I take this issue?

Please can I work on this, haven't gotten issues to contribute to yet

Can I be assigned to this?

im a newbie contributor, i'd be grateful if you'd give me the opportunity to work in this.

Can I try solving this issue?

I @Ellliot I am koxy, a stellar ecosystem contributor, can you pls assign this task to me?. I would work on this by implementing the change trust operation on stellar's SDK.

I'm a returning contributor and I'd like to try this one.
I will carefully study the structure of the project and provide a solution based on the description of the issue.

Could I take on this issue? I've worked on this repository in the previous ODHack and I'm confident that I'll be able to resolve this issue.
ETA: 1 Day


Improve error handling and messaging

Need to support a number of types of errors:
Error messages sent to AT:

Unsupported request type
Unsupported property
Exception handling request
Cancelled from timeout

Error messages sent to AT and content

Request or response does not validate

Just landed support for 'No js2at observer for this request'


Fix build on Rocky Linux. (fixes #57)

Originally the build of the poplog commander failed due to the lack of -std=c99 and the lack of visibility of PATH_MAX.

It's a shame to have that magic 512 constant in there. But [1] I don't have a better idea [2] this isn't worth putting a vast amount of effort into and [3] it is undoubtedly better than what we had previously. Accepting on that basis.

And it's great to have the right libraries sorted out for a RHL variant!


.NET core - route behavior difference from Windows and Linux

I am using this route on Windows with .NET core:
routes.MapRoute("api", "{action}/{id?}", new { controller = "API", action = "Index" });

Everything works great in console app or in IIS. Example url: http://localhost/action/parameter, calls into APIController controller class.
When running the same code on Linux (Ubuntu 18.04 in this case), I get a 404 error for every call. Changing the route to this fixes it:
routes.MapRoute("api", "/{action}/{id?}", new { controller = "API", action = "Index" });

Note that the new route starts with a forward slash, which is required on Linux. It would be nice if the forward slash was optional on all platforms.
Thanks very much!

cc @rynowak \ @kichalla

Interesting. I would imagine we would have seen many failures in our tests too if this were the case.
@jjxtra Could you share a standalone repro too? (Want to make sure you don't have a setup which is unique for your scenario)

Sure I can conjure something and post it, will do tomorow.

Here is my startup class. I am using newtonsoft json nuget package as well.
    public class APIStartup
    {
        public static void Main(string[] args)
        {
            CreateWebHostBuilder(args).Build().Run();
        }

        public static IWebHostBuilder CreateWebHostBuilder(string[] args)
        {
            return WebHost.CreateDefaultBuilder(args).
                UseKestrel(c => c.AddServerHeader = false).
                UseStartup<IPBanAPIStartup>();
        }

        public IPBanAPIStartup(IConfiguration configuration)
        {
            Configuration = configuration;
        }

        public IConfiguration Configuration { get; }

        // This method gets called by the runtime. Use this method to add services to the container.
        public void ConfigureServices(IServiceCollection services)
        {
            services.AddResponseCompression();
            services.AddMemoryCache((o) =>
            {
                o.CompactionPercentage = 0.9;
                o.SizeLimit = 1024 * 1024 * 64; // 64 MB
                o.ExpirationScanFrequency = TimeSpan.FromMinutes(10.0)
            });
            services.AddMvc().SetCompatibilityVersion(CompatibilityVersion.Version_2_1).AddJsonOptions(options =>
            {
                options.SerializerSettings.NullValueHandling = NullValueHandling.Ignore;
            });
        }

        // This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
        public void Configure(IApplicationBuilder app, IHostingEnvironment env)
        {
            if (env.IsDevelopment())
            {
                app.UseDeveloperExceptionPage();
            }
            else
            {
                //app.UseHsts();
            }
            app.UseResponseCompression();
            app.UseMiddleware<RateLimitMiddleware>(app.ApplicationServices.GetService<IMemoryCache>());
            //app.UseHttpsRedirection();
            app.UseMvc(routes =>
            {
                // this is the line where adding a forward slash before {action} fixes 404 error on Linux
                routes.MapRoute("api", "{action}/{id?}", new { controller = "API", action = "Index" });
            });
        }
    }

and my api controller:
[APIController.APIControllerExceptionAttribute]
public class APIController : ControllerBase
{
        private static readonly JsonSerializerSettings jsonSerializerSettings = new JsonSerializerSettings { NullValueHandling = NullValueHandling.Ignore };

        private readonly IMemoryCache cache;

        public class APIControllerExceptionAttributeAttribute : ExceptionFilterAttribute
        {
            public override void OnException(ExceptionContext context)
            {
                JObject obj = new JObject
                {
                    ["message"] = (context.Exception.Message + Environment.NewLine + (context.Exception.InnerException == null ? string.Empty : context.Exception.InnerException.Message)).Trim(),
                    ["error"] = true
                };
                context.Result = new ContentResult
                {
                    Content = obj.ToString(Formatting.None),
                    ContentType = "application/json",
                    StatusCode = (context.HttpContext.Response.StatusCode == StatusCodes.Status200OK ? StatusCodes.Status500InternalServerError : context.HttpContext.Response.StatusCode)
                };
            }
        }

        public APIController(IMemoryCache cache)
        {
            this.cache = cache;
        }

        public IActionResult Method(string id)
        {
            return Content("hello world");
        }
}

rate limit middleware:
public class RateLimitMiddleware
    {
        private class CacheEntry
        {
            public int Count;
        }

        public const int MaxRequestsPerMinute = 60;
        public const string MaxRequestsPerMinuteString = "60";

        private const string acceptEncodingHeader = "Accept-Encoding";
        private readonly RequestDelegate nextDelegate;
        private readonly IMemoryCache cache;

        public RateLimitMiddleware(RequestDelegate nextDelegate, IMemoryCache cache)
        {
            this.nextDelegate = nextDelegate;
            this.cache = cache;
        }

        public async Task InvokeAsync(HttpContext context)
        {
            context.Response.Headers["Vary"] = acceptEncodingHeader;
            string key = "RateLimit_" + context.Connection.RemoteIpAddress.ToString();
            if (!cache.TryGetValue(key, out CacheEntry count))
            {
                count = new CacheEntry();
                cache.Set(key, count, new MemoryCacheEntryOptions
                {
                    AbsoluteExpirationRelativeToNow = TimeSpan.FromMinutes(1.0),
                    Size = (key.Length * 2) + 16 // 12 bytes for C# object plus 4 bytes int
                });
            }
            int currentCount = Interlocked.Increment(ref count.Count);
            context.Response.Headers["X-RateLimit-Remaining"] = (MaxRequestsPerMinute - currentCount).ToString(CultureInfo.InvariantCulture);
            context.Response.Headers["X-RateLimit-Limit"] = MaxRequestsPerMinuteString;
            if (currentCount >= MaxRequestsPerMinute)
            {
                context.Response.StatusCode = 429;
                context.Response.Headers["Content-Type"] = "application/json";
                await context.Response.WriteAsync("{ \"message\":\"Rate Limit Exceeded\", \"error\":true }");
                return;
            }
            await this.nextDelegate(context);
        }
    }

Thanks for contacting us, @jjxtra.
Sorry for our delayed response. Can you please advice whether you still experience this issue or whether this issue can be closed already.

Hi. We're closing this issue as we have heard no response from you for some time now. If you have more details and are encountering this issue please add a new reply and re-open the issue.


fix: rename InternalAnchor/ExternalAnchor prop types

PR Checklist

[x] Addresses an existing open issue: not quite an issue, but this was discussed here
[x] That issue was marked as status: accepting prs
[x] Steps in CONTRIBUTING.md were taken

Overview
This was identified as a typo in https://github.com/SquiggleTools/boston-ts-website/pull/284#discussion_r1656539369

@JoshuaKGoldberg do I need to do anything about this GitHub check?

Nope :)

Nope :) it's complaining about the lack of fixes #... but failing to report that because of https://github.com/mtfoley/pr-compliance-action/issues/368.


feat: Redo: Rename Lambda related interfaces

What does this PR do?
Redoes https://github.com/DataDog/datadog-cdk-constructs/pull/288:

Rename: DatadogProps -> DatadogLambdaProps
Rename: DefaultDatadogProps -> DatadogLambdaDefaultProps
Rename: DatadogStrictProps -> DatadogLambdaStrictProps
For backward compatibility, add:

export interface DatadogProps extends DatadogLambdaProps {}
export const DatadogDefaultProps = DatadogLambdaDefaultProps;
export interface DatadogStrictProps extends DatadogLambdaStrictProps {}

Add an example Python stack and an example Go stack that use the new APIs (DatadogLambda and DatadogLambdaProps) to examples/. Also add these two stacks to integration_tests/stacks for snapshot tests.

Motivation
To make it cleaner to add DatadogStepFunction class later. Details in [RFC] Changing API for Datadog CDK Construct

Testing Guidelines
1. Pass snapshot tests
aws-vault exec sso-serverless-sandbox-account-admin -- scripts/run_integration_tests.sh
The stacks can be processed without error.
2. Notes on mixed API use
One use case will be broken by this PR: it's when the user uses Golang and uses DatadogProps (old) to construct DatadogLambda (new, added on Aug 29 in https://github.com/DataDog/datadog-cdk-constructs/pull/285), i.e.
	datadog := ddcdkconstruct.NewDatadogLambda(
		stack,
		jsii.String("Datadog"),
		&ddcdkconstruct.DatadogProps{
			...
		})

because NewDatadogLambda() can only take DatadogLambdaProps.
However, I think this case should be rare because:

The Go example code in README.md is still using the old Datadog instead of DatadogLambda
Go support was just added on Aug 23, so the user base should be small.

I'm also unable to find a good way to make this case happy. I'd just go ahead and release it, and ask users to update their code if the rare case does exists.
3. Notes on other types/constants
I don't think people are supposed to use DefaultDatadogProps and DatadogStrictProps directly, so I didn't test the various combinations around those consts/interfaces. There's a chance that user code will be broken if DefaultDatadogProps or DatadogStrictProps are use directly. Let's closely monitor the issues after the release. In case anyone complains, let's fix forward (revert this PR then release), like what we did last time.
Additional Notes

Types of Changes

[ ] Bug fix
[x] New feature
[ ] Breaking change
[ ] Misc (docs, refactoring, dependency upgrade, etc.)

Check all that apply

[x] This PR's description is comprehensive
[ ] This PR contains breaking changes that are documented in the description
[x] This PR introduces new APIs or parameters that are documented and unlikely to change in the foreseeable future
[ ] This PR impacts documentation, and it has been updated (or a ticket has been logged)
[ ] This PR's changes are covered by the automated tests
[ ] This PR collects user input/sensitive content into Datadog

@purple4reina Added snapshot tests for Go and Python in the last few PRs. This PR is now ready for review again.


Short solution needed: "How to set data frame index" (python-pandas)

Please help us write most modern and shortest code solution for this issue:
How to set data frame index (technology: python-pandas)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

How to set data frame index


Suggestion: Multi-List Scans

I really like that we can out together our own lists, and that the color of the indicator changes depending on the color of the list.
I think it'd be pretty cool if you could search using two or more lists at the same time, using the color from each list respectively.
Hope this feedback is helpful.
Keep up the good work, and have a wonderful day!!!

Hey! This is actually already supported — just swipe left on the search bar, and you can add as many search bars as you like! Find will use the list's color as the highlight indicator.
https://user-images.githubusercontent.com/49819455/221392925-8c78eee0-f7c7-4760-8550-82767ae74830.mp4


Add replicas handling in Stream editing

It seems nats str edit with new replica count was not handled. It may be better to add some validation, but kept it simple for now, as I assume the server will validate the input.

We now support this in the Server which was not supported when you initially opened the PR. We also updated the CLI to support this in the manner the server supports, I will close this PR, thanks for the contribution


[wip] Compiler switch

Summary:

switched to conda compilers
deprecated support for python 3.4, 3.5
added support for 3.7
added some meta-data

I've switched to the conda-compilers for building the conda package as it is ubiquitously done on conda-forge. This is really nice, since it makes it more likely that packages are binary compatible.
This is work in progress (wip) with respect to that I still have to try these recipes on macos and windows.
Some questions on what I can do here:

I would like to remove the pybind11 submodule and add it to the conda dependencies. I presume this would break the gentoo-build. Do you mind? Should I remove the gentoo recipe in this case?
at some point I'd like to submit a recipe to conda-forge so that we will not have to build it ourselves anymore :D

I'd like to keep a normal build with cmake working (i.e. no conda). if you can guarantee that I don't mind if the submodule is replaced with something else.


fix(curriculum): require space after keyword

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Requires space after the keyword when needed:

await
function
new

well done !


Login endpoint gives a 404

https://api.robinhood.com/api-token-auth/ results in a 404 error.
See https://github.com/Jamonek/Robinhood/issues/135.

Should be fixed with #36. Feel free to reopen if that's not the case.


Error: module loader

Hello dear contributors and maintainers,
With Angular v1.5.8 and yeoman generator gulp-angular, there's an error when loading your module ngInfiniteScroll.
With angular.module('myModule', ['infinite-scroll']); in my code, neither the ngInfiniteScroll bower package v1.3.4  nor the ng-infinite-scroll npm package v1.3.0 can be loaded by Angular. The page just stays blank and the console full of errors.
I had to install the bower package v1.2.2 so that my app can load correctly.
Let me know if you need more infos and if this issue is due to a specific version / tool.
Keep me posted.

Can you create an example app on github with the failure?

You got it.

I've conducted additional tests.
It appears that the gulp-angular generator can't work with npm packages (tested with npm firebase). On the other side bower packages are loaded as they should.
In conclusion, I think I've made a typo or entered an other package name with bower which caused this behaviour: bower ngInfiniteScroll (v1.3.4) works fine.
¯_(ツ)_/¯

@jbouhier looks like gulp-angular is deprecated. Consider webpack/rollup/browserify.


qps axis - any way to delete it when it's off (or blank out the data)

when turning off (by clicking) the QPS data set, the right side changes to [0 - 1.0] which is a bit confusing - is it possible to change some setting in the axis itself (maybe there is a flag/option) or actively remove it if not ?

if not delete just have no 0, 0.1, 0.2,... 1.0 there


Can't run the WebHost project

I set the "WebHost" as the starting project and tried to run it. I got an error popup from Visual Stuido: "Unable to start debugging on the web server. The debugger cannot connect to the remote computer. The debugger was unable to resolve the specified computer name."
What do I need to do to be able to run the "WebHost' project?

As I said I tried to run the "WebHost" without any modifications and I got the error that I mentioned. In the Web tab of the properties page of the "WebHost" project I see that the Servers is set to External Host with the following url:
https://web.local/wa
Could somebody tell me what I need to do to at least run the sample?
When I tried to run the "SelfHost" I get an exception: "{"Sequence contains no elements"}" which I believe has to do with the following line:
SigningCertificate = X509.LocalMachine.My.SubjectDistinguishedName.Find("CN=sts").First()
Any help would be appreciated.

I used web.local on my machine. You have to change that to your machine name.
That's just basic asp.net 101.

OK, thank you @leastprivilege.
I was able to run it using IIS Express (http://localhost:6739/).
Quesition 1. Is IIS Express OK for testing?
In order to run it I had to comment out the the following section in web.config because I was getting errors:
<!--<security>
  <authentication>
    <anonymousAuthentication enabled="true" />
    <windowsAuthentication enabled="true" />
  </authentication>
</security>-->

Question 2. Could you please comment if the above section is needed in web.config. Of course the project wouldn't run in IIS Express with that section present.

These settings are for full IIS - you need to enable Windows and anonymous authN in IIS Express.

OK, thank you.


Migration examples.

[X] I have added or updated the specs.
[X] I have verified that the specs pass on my computer.
[X] I have not attempted to bump, or alter versions.
[X] This is a documentation change.
[ ] This is a source change.

Description
Thanks for jekyll-assets!
I ran into a few issues migrating from 2.x -> 3.x. This includes some before and after examples to help with migrating.

Thanks :heart:


Neovim: AttributeError("'LegacyVim' object has no attribute 'bindeval'",)

I'm trying to set this up using Neovim(0.1.4) and deocomplete and seeing this error when I trigger the omnifunc method with <C-x><C-o>
Error detected while processing function omni_swift#complete[6]..<SNR>177_complete[3]..autocomplete_swift#complete[9]..sourcekitten_daemon#complete[5]..<SNR>120_get[6]..<SNR>120_request[33]..13[94]..provider#python#Call:
line   18:
AttributeError("'LegacyVim' object has no attribute 'bindeval'",)
Traceback (most recent call last):
  File "/Users/jakecraige/Library/Python/2.7/lib/python/site-packages/neovim/msgpack_rpc/session.py", line 185, in handler
    rv = self._request_cb(name, args)
  File "/Users/jakecraige/Library/Python/2.7/lib/python/site-packages/neovim/api/nvim.py", line 151, in filter_request_cb
    result = request_cb(self._from_nvim(name), args)
  File "/Users/jakecraige/Library/Python/2.7/lib/python/site-packages/neovim/plugin/host.py", line 74, in _on_request
    rv = handler(*args)
  File "/Users/jakecraige/Library/Python/2.7/lib/python/site-packages/neovim/plugin/script_host.py", line 75, in python_execute
    exec(script, self.module.__dict__)
  File "<string>", line 2, in <module>
AttributeError: 'LegacyVim' object has no attribute 'bindeval'

Any ideas on how to fix this?

Did you ever solve this because I have the same problem?

@codelogicws No I didn't. I haven't had time to look much more into it.

@jakecraige @codelogicws Sorry for late reply. I start investigating now, but I recommend to use with deoplete if you are not using it, which provides asynchronous autocompletion.
@codelogicws What the version of neovim are you using?

@jakecraige

I'm trying to set this up using Neovim(0.1.4) and deocomplete

Sorry, I misunderstood. Omnifunc always uses the implementation in Vim script even if you use Neovim.
In addition, omnifunc avoids to send request to SourceKitten/SourceKittenDaemon when the regular expression doesn't match the current context (surrounding text). This regular expression is same as the one used for neocomplete/deoplete.
So, omnifunc is useless in the case that deoplete is installed.

@jakecraige @codelogicws
This problem is reproduced in the case of python-client for Python 2.x. Some workarounds may resolve this:

Uninstall python-client for 2.x
Use the 3.x version of python-client

I keep finding the fundamental resolution ... please wait for a while.

@jakecraige @codelogicws Is the problem solved? Please reopen this issue whenever the problem happens again.


chore: Prettier on entire repo

Now that we have a .prettierrc, better to format the repo once so any future tweaks dont cause entire diffs full of prettification

Closing this as it could cause git conflicts in current site which is quite hard to fix. Will take care of this later in Config-less architecture and when the remix version is up


urllib.error.URLError: <urlopen error [Errno 11001] getaddrinfo failed>

Can you help me in resolving this issue ? Is internet necessary to evaluate/ test the image. If Yes, can you suggest some workaround so that internet can be avoided to infer the text ?
Thanks in advance.

You have a problem with your internet connection.

@baudm That's what I was trying to point out. How can I use this model in offline mode that is to eliminate internet dependency.

Download the pretrained weights and load the model manually:
https://github.com/baudm/parseq/blob/ed3d84720c3f2a7b6c463a40492e0dad93920294/strhub/models/utils.py#L80


⚠ Bofrost.lu has degraded performance

In d761edd, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 912 ms

Resolved: Bofrost.lu performance has improved in 5d3871f after 6 minutes.


Typo in apostrophes

In the templates in "tacred.relation.config.json", for the relation "per:date_of_birth", there is the following template.
"{subj}\u00e2\u0080\u0099s birthday is on {obj}."
Shouldn't it be as below?
"{subj}'s birthday is on {obj}."
It looks like the apostrophe character (') is replaced with \u00e2\u0080\u0099. Is that a typo or is it intentional?
There is also the same issue in the following template as well.
"{obj} is the cause of {subj}\u00e2\u0080\u0099s death."

Thank you @mahrahimi1 for pointing it out!
I think that was caused by some encoding issue, I corrected the configuration files.


Adding code build script for smlog travis ci tests

Code build script to run travis-ci
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@RedbackThomson


reduced db calls

reduced the db calls to no duplicate
the MessageController had several points where the code of ´getConversations() was used, replaced the duplicated code with a function call

resolves #398


hwclock & fake-hwclock support

The Raspberry Pi doesn't have an RTC so if the unit fails to get the correct time, it can have trouble connecting to the OpenVPN server. I had that happen on a few networks where NTP was blocked and the unit was not able to connect.
For the Raspberry Pi units that don't have a physical hardware clock, the fake-hwclock package is very useful as it ensures that the clock moves forward after reboot and prevents the time going back to the year 2000.
An even better solution is to add this little device: https://www.ebay.co.uk/i/253056413179?chn=ps&dispItem=1&adgroupid=41920510205&rlsatarget=pla-325751485803&abcId=1128926&adtype=pla&merchantid=109797754&poi=&googleloc=9045006&device=c&campaignid=856841495&crdt=0
To set it up on the RPI, you run:
echo ds1307 0x68 | tee /sys/class/i2c-adapter/i2c-1/new_device

After that, the command hwclock starts to work:
root@resin:/# hwclock
Sat Jan  1 23:18:01 2000  0.000000 seconds
root@resin:/# hwclock -w
root@resin:/# hwclock
Mon Aug 14 17:19:11 2017  0.000000 seconds

It would be great if support for the above was added to ResinOS out of the box.

Hi, thanks for the report.
@shaunmulligan would you consider we look into adding fake-hwclock for cases where the network blocks NTP? Or we consider NTP unblocking as a hard dependency which we absolutely need and hence we don't cover cases where this happens?

@floion I don't really know enough about these mechanisms, but it sounds reasonable to me. I would defer to @agherzan and @petrosagg to make the actual decision though

ping @agherzan @petrosagg

@agherzan @petrosagg any info on the above? I am adding RTC DS3231 clock to the resin device that should operate in the following way:

use RTC clock on boot
sync clock via NTP when online

Hi @SloMusti. Did you try to setup the hwclock for the user container? Did you encounter any issues with that?

@agherzan above instructions work in the user container, but it be great to have hwclock support on the host

@SloMusti what kind of device is that, one of the Raspberry Pi family? If so, you can try to use dtoverlay=i2c-rtc,ds1307 in the config.txt (which you can set remotely as well: https://docs.resin.io/reference/resinOS/advanced/#modifying-config-txt-remotely)
As much as I checked, that should support DS3231 as well.
If not a Pi, then probably need some extra investigation.

@imrehg One doubt. For me it seems to work. But i have one doubt, is the RTC being updated by some NTP related task running on the background of Balena OS? Or do we have to be responsible for maintaining correct RTC time (RTC can lag depending on quality) using a specific service like this one: https://github.com/balena-io-playground/balena-rtc-multicontainer

You can answer but it seems to be answered here, sorry.
https://forums.balena.io/t/current-state-of-hardware-rtc-support/6727/17


fix: add support for updating an individual field with pojo in all update method

Fixes #126

@BenWhitehead This looks good to me, but we should try to replace the FooModel usage with SingleField before merging.


[DATA-897] Remove unused last from BinaryDataByIDsResponse

Cleanup PR for a breaking API change to remove a field that was never used.
last is used for pagination when requesting data by filter in Binary/TabularDataByFilterRequest.
We also support requesting binary data by IDs, but this doesn't include any pagination in the request and thus last is unused.
Removed in app here (which was simply populating the field with an empty string) and was never used in RDK (which only uses the data field of BinaryDataByIDsResponse)

@tessavitabile Since this is a breaking change, would need to be force-merged if possible. Thanks!

@tessavitabile Would it be possible for you to force-merge this today? Confirming that this field is not used in app or RDK.

Thanks for pinging me, I missed the last message on this thread.

@tessavitabile Thank you!


Increment version for maps releases

Increment package version after release of Azure.Maps.Routing

API change check
API changes are not detected in this pull request.


Panic on empty input for Secure Cell

It does not make much sense to use an empty master key for encryption, or encrypt or decrypt empty messages. Secure Cell does not allow such inputs and returns errors if the user insists. Additionally, context imprint mode of Secure Cell requires a non-empty user context.
The programmers should never really use Secure Cell in this way. If an empty key, or message, or token, or user context is used anywhere then it's most likely to be a programming error. Therefore treat it as such — like using an invalid index with a slice — and panic instead of returning an error code if we encounter empty inputs.

Are you sure that the code should panic rather than return error if input parameters are not valid?
I've read only one chapter of Rust book about panicking, but I'm not sure how typical this behavior is for Rust-devs.
Few usage examples:

Wrong configuration. Developers sets up SecureCell without master key. On the runtime SecureCell returns panic to indicate that master key is required. Panic looks appropriate, developers add master key in code.

Data got from network. Application got master key / message from network requests and called SecureCell to decrypt message. Due to network encoding or failure, key or message might be corrupted. SecureCell responds with panic.

To make sure that code works properly, developers should handle both panics and errors. Panic looks as (unexpected) additional check-to-make. Is it ok for Rust ecosystem?

Are you sure that the code should panic rather than return error if input parameters are not valid?

No, I'm not completely sure.
The approach depends on the severity of the error condition which I would like to discuss.
On panics
Libraries should rarely panic because this is a decision that affects the whole application rather than only the library code and its direct users. By panicking the library forces the whole program to handle an error in a certain way and does not give the programmer an option for different behavior.
One does not really handle panics (though there are mechanisms for that). Instead panics are to be prevented. So the idea is that the developer has to either perform necessary safety checks before calling the code that might panic, or use some other means to prevent a possible panic (e.g., iterator API makes it impossible to use invalid array indices in the first place).
Errors signaled by panics should be so severe that killing the thread justifies the prevented harm. Now, after considering your example with keys obtained externally I'm not really sure that we need to panic at all, not just in this situation. Using invalid or corrupted data is certainly a preventable error condition. It can be avoided by performing a check. It can be handled after-the-fact by inspecting the returned result. And it does not really jeopardize the process as a whole if it happens, in contrast with using an out-of-bounds array index for example.
So it seems it would be fine to leave the situation as it is now (without panics). The user is technically allowed to pass empty keys, tokens, contexts, messages to Secure Cell. However, they will get an error in result. Even if that's a configuration error then it has to be handled explicitly.
Opinions?
On further actions
One of my concerns was about the master key. It is currently possible to construct a Secure Cell with an empty master key, but the error will be reported only when the cell is actually used to encrypt or decrypt data. The construction site and the usage site may be quite distant in code and time which can make it hard to track this error.
I think it would be nice to prevent this kind of errors earlier. I'd like to change the Secure Cell constructor to return a Result instead which should be an error if the key is empty. This will force the users to handle this possible error at the construction site.
That can be done in a different pull request and this one can be closed if we choose to not add any new panics to the code.
Alternatives
Some libraries provide “dual interface”. For Secure Cell it would be an encrypt() method that returns Vec<u8> directly and panics on any error whatsoever, accompanied by its cousin try_encrypt() that returns an explicit Result<Vec<u8>> and never panics.
It looks a bit nicer in the code but I find this approach to have poor discipline. It's not that hard to add an .unwrap(), or .expect(), or even just ? to the line to handle the error in a suitable way.

The user is technically allowed to pass empty keys, tokens, contexts, messages to Secure Cell. However, they will get an error in result.

agree

I'd like to change the Secure Cell constructor to return a Result instead which should be an error if the key is empty

agree.
instead of panicking, can we add length-checks? If no token/context/masterkey/message - return Error as result?

@vixentael, I've opened a new pull request with early length checks for master key during Secure Cell construction. This one PR will be closed as there will be no expected panics in the code.
Rust-Themis already reports an error for invalid length of other parameters. The checks are actually performed by Themis core. We have tests that verify this (e.g., for message).


[Network] az network vnet subnet list-available-ips: Get list of available IPs for subnet

Related command

az network vnet subnet list-available-ips
Description

close #13853
Add new command 'az network vnet subnet list-available-ips'
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Network

you guys rock. this is perfect timing.

Why does it list only some of the available ips?

@necusjz I was expecting this new command to report on all available IPs. Should I create an issue for this?

@webmandman, actually az network vnet subnet list-available-ips implemented a functionality similar to az network vnet list-available-ips (but at different levels). It'll list some available IPs in the VNet.
For your needs, we already have a similar issue for that. Does it meet your expectation? It'll return the number only (not the specific IPs).
Or could you explain the usage scenario for returning all IPs. We need to evaluate the feasibility. Thanks.

@necusjz no you are right. I actually do not need to know all available IPs. I thought maybe this limit was overlooked. Thanks.


Use database instead of config.toml calls

Every toot/tweet/mail should in the end be written to a database, so the bots can communicate with each other over a database, not inside the code, and so everything is trackable and some stuff reversible.
This would also enable to blacklist certain authors.

database city_ticketfrei

table reports

columns:

author
full_author (full email/username as a reference to block them)
report text
link to tweet (or tweet object?)
link to toot (or toot object?)
timestamp of email (or full email object?)

Cool, this is pretty much done. If there is something left, it will be solved by other efforts to get features running. No reason to leave this open.


Traefik does not kill persistent connections when removing a service

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
If we register a service in Traefik's config, have a client create a persistent websocket connection to the service's backend, and then update Traefik's config to remove the service, new connections to the service's backend via Traefik will fail (as expected), but Traefik does not kill any existing persistent connections to the service (that were established prior to the service being deleted from Traefik's config).
What did you see instead?
I believe this is an expected behavior in Traefik (as part of its graceful handling of client connections with services), but is there a way to configure a service in Traefik, such that Traefik will kill all persistent connections to that service when its deleted from the Traefik config?
What version of Traefik are you using?
2.9.1
What is your environment & configuration?
Docker
If applicable, please paste the log output in DEBUG level
No response

Hello @thecooltechguy and thanks for your interest in Traefik,

I believe this is an expected behavior in Traefik (as part of its graceful handling of client connections with services), but is there a way to configure a service in Traefik, such that Traefik will kill all persistent connections to that service when its deleted from the Traefik config?

As you said, this behavior is expected, and there is no way to configure Traefik to close the persistent connections when a corresponding service is deleted from the config.
The GitHub issue tracker is dedicated to bug and feature requests. For questions, please join our Community Forum and reach out to us on the Traefik section.


Handle Quote (specifically Quote3)

[x] Look at pulling nom in for converting bytes to the types

Discussion from 2022-08-16:

Create a rust type that holds a Vec<u8>. This will be the raw quote.

The need to have this raw representation is to be able to verify the signatures on parts of these raw bytes.

Create a rust type for each of the SGX quote types, sgx_quote_t, sgx_quote3_t, and sgx_quote4_t.

Initial thought was a view, but with https://github.com/rust-lang/rust/issues/82523 we think we'll need to copy out instead of being able to pass references.
sgx_quote_t looks to be EPID, sgx_quote3_t looks to be DCAP, and sgx_quote4_t looks to be for TDX

Thoughts as of today's sprint-planning: Quote3 is a rustic structure that contains the parsed Quote bytes, as well as the underlying bytes (as slice or vec).

Implemented via #190, #196, #197, #200, #202, #203, #204, #205, #208


[Access Request] report 106 "Customer Detailed Aging"

Can you please change the acces scope of the following variables to protected?
protected var
    EndDate: Date;
    OnlyOpen: Boolean;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


can't install it

can you give me test(example) fully working project (compile,run and watch video) ?

@spartanatreyu can you upload the code that u used in angular to do that?... I have the same problem... I have a video background for my app, and in iOS works fine, but in Android I can't view the video :(.
Please can u help me? Thanks

@linkito My video code is a little over the place but I put a gist here for you:
https://gist.github.com/spartanatreyu/dd7bc772a5ed85ff512a

Hello
I cant install this plugin like in README.md you indicate. I have this error:
Error: Failed to fetch plugin https://github.com/dawsonloudon/VideoPlayer.git via registry.
Do you know why it could be? thanks

@NereaElorza This repo hasn't been updated in 4 years. Probably not the video player you want to use considering how much android, cordova and the android platform code for cordova have changed.

Hey everyone, you should really look at this repo to solve your issue - https://github.com/jaeger25/Html5Video/blob/master/README.md as it looks more actively maintained. It works on a similar method to bypass the local chrome browser restrictions on video files (android.resource://). If that does not work, I'd recommend investigating Crosswalk (https://crosswalk-project.org/documentation/cordova.html)


GString-closure not supported

A GString may contain a closure as one of its expressions, written in a particular style, like
"GString ${ -> heavy() }"
Example from the console:
println "${ -> println "1 - lazy" ; 1 } - ${ println "2 - eager" ; 2 }"
Gives error:

1 compilation error:
no viable alternative at input 'println"${->' at line: 1, column: 13

I'm trying to fix it.
在 Jesper Steen Møller<EMAIL_ADDRESS>20:03写道：
A GString may contain a closure as one of its expressions, written in a particular style, like
"GString ${ -> heavy() }"
Example from the console:
println "${ -> println "1 - lazy" ; 1 } - ${ println "2 - eager" ; 2 }"
Gives error:
1 compilation error:
no viable alternative at input 'println"${->' at line: 1, column: 13
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHubhttps://github.com/jespersm/groovy/issues/12

The statements are not supported in the GString-closure, I'll fix them together.

The issue is fixed.
The related PR: https://github.com/jespersm/groovy/pull/14


whitelist bridge tokens to remove warnings

whitelist is already implemented, please avoid creating hanging constants all over.
https://api.zilstream.com/tokens


Integrate CHIP as a Submodule

We should follow the style provided by CHIP and include them in this repo under third_party/chip/repo as a git submodule.
Furthermore, our setup script should call out to ./third_party/chip/repo/scripts/activate.sh to ensure our environment is configured in a compatible way.
Things that are not part of this ticket:

Setting up our own gn files to call out to CHIP
Being able to build CHIP from our base directory

since there is no setup script yet, is its implementation part of this issue?

Yes. But for now it should only forward to the CHIP repo setup script.


[bug] When the domain name is accessed with a port, it cannot be used

Search before asking

[X] I had searched in the issues and found no similar issues.

Environment
https://laf.dev
What happened
https://fire.juukee.cn/firewall   Accessible
https://fire.juukee.cn:443   Accessible
https://fire.juukee.cn:443/firewall  Not accessible  --want support
When the domain name is accessed with a port, it cannot be used
How to reproduce
When the domain name is accessed with a port, it cannot be used
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

I tried to reproduce it and nothing happened. Unless you are using a non-80 or non-443 port, the cloud function only supports binding to ports 80 or 443
Also, deliberately adding port 443 to https like "https://fire.juukee.cn:443/firewall" doesn't make any sense


Arm64 is now Supported

Arm64 is now Supported by WebRTC
https://code.google.com/p/webrtc/issues/detail?id=3898

Thanks! I'm working on updating our own stuff to a much newer revision of WebRTC (including arm64). I'd like to confirm that it's all working as expected first, but I plan to update this once I've done that.

@hjon hey there. i've been looking into the webrtc stack and the TLK otalk libraries.
since 64-bit is a new requirement, i rebuilt the webrtc binaries with these instructions:
https://github.com/pristineio/webrtc-build-scripts
i've started doing some updates to the TLK libraries as well. i can send a few pull requests your way if it helps. feel free to email me, if you'd like to talk more or coordinate, wasn't sure how to contact you.

you can see my changes here, https://github.com/piemonte/TLKWebRTC

For anyone interested instead of using
pod "webrtc-ios"
use:
pod "libjingle_peerconnection"
Then in TLKWebRTC rename RTCSessionDescriptonDelegate to RTCSessionDescriptionDelegate
And change [peer addStream:self.localMediaStream constraints:[self mediaConstraints]]; into [peer addStream:self.localMediaStream];

Given that the modifications we were using have now made their way into the main WebRTC library, I've accepted the changes to the other libraries (TLKWebRTC, etc.) to get up to speed with recent changes and will point others to libjingle_peerconnection in the README.


Color thresholds for values below 0 (plus optionally allowing >/< syntax)

Is your feature request related to a problem? Please describe.
With HA Companion app 2022.8 for Android the battery power sensor has been introduced. This gives the current power usage, e. g.:

Currently a color threshold can not be set for negative values (<0). Syntactically it's also not possible to work with "great than" and "lower than".
Describe the solution you'd like
At least allow negative values to work with color_thresholds. >/< would be an optional nice thing. So either:
    color_thresholds:
      - value: <0
        color: red
      - value: 0
        color: yellow
      - value: >0
        color: green

or at least
    color_thresholds:
      - value: -0.01
        color: red
      - value: 0
        color: yellow
      - value: 0.01
        color: green

Currently for negative values the color_threshold is not applied, even if value is set as 0.
Additional context

Release notes HA Companion App for Android
Details on new battery power sensor
Similar issue related to color threshold setting

What if you use multiplier? I think it should work but of course it will modify the displayed value.
multiplier: -1

Does it solve your problem?

Catering for -ve values would be very helpful as I would like to use the color_gradient for my BLE sensor signal strength, which measures in -dBm vales, however the color_threshold does not work for these values.

In v3 you have the option to convert value to absolute and convert it to desired percentage via KString/thresholds function.

In v3 you have the option to convert value to absolute and convert it to desired percentage via KString/thresholds function.

@maxwroc Unfortunately I neither see a "lowerthan" option (only "{state|greaterthan(10,100)}") nor a single example visualizing negative values.
So even the technical possibility might exist now - it's way to complicated to use without a proper example in the docs.

TBH It would be quite hard/impossible to put examples for all of the cases and combinations in the doc.

Unfortunately I neither see a "lowerthan" option

The name of it is "lessthan" (Less-than_sign)

nor a single example visualizing negative values

This is not true. Here is the example with negative values: https://github.com/maxwroc/battery-state-card#rssi-sensors-signal-strength
The way to achieve it is the following:

"{state|abs()|greaterthan(10,100)}"
This converts the state value to the absolute and later changes the value to 100 if the state value is greater than 10.
"{state|abs()|lessthan(10,0)}"
Similar as above but changes value to 0 if state is lower than 10
You can combine both as well
"{state|abs()|lessthan(10,0)}|greaterthan(10,100)}"
Documentation states this as well:

To use the above and change the shown entity state value you need to use the "value_override" in your config (please take a look at the example which I have linked above)
I will think about adding more examples to the doc, but as I have said, card is getting more and more features and it would be quite hard to cover all of their combinations in the doc.
If the above solved your problem feel free to contribute and add an example in the doc. If you're still confused and don't know how to achieve the expected result please let me know.
BTW Thanks for such detailed description of the issue with HA doc/pr links etc!

I played a bit with the value_override.  The problem here is, that

the override changes the values itself which is completely irrelevant here and creates other visualization issues, it's just about the color_thresholds
so I can not find a override template which keeps the current values. In detail:

Few examples:

normal with value_override:

with value_override: "{state|abs()|lessthan(0,-10)}"

--> why the hell are the (actually negative, below 0) values now shown as positive ones?

The main issue is that the color formatting is dependent on the value. And the value_override stuff is mighty, but obviously not working well (or I'm using it wrong).
In other words: how to show entities in green, yellow and red depending on their range from e. g. this to that?

There was an attempt to add such support but there were some issues and it had to be reverted https://github.com/maxwroc/battery-state-card/pull/599
It is possible to do it but it just requires little bit more thinking.
For example I don't want to enable setting colors for any value ranges and then later hear people complaining that the gradient option doesn't work with them etc.
@amsterjanis maybe you would be interested to tackle this problem again?


[WIP] Strhelpers

This pull request is opened to show a progress of the implementation of some string helpers from Laravel.
This will address https://github.com/confetti-framework/confetti/discussions/102

This looks good! This is very useful. 🥇

Great changes! Do you mind merging main into this branch? Then we get a better idea of whether code coverage has improved.

@octoper feel free to continue working on this PR. I don't know how you can easily work on this with @Wulfheart. Perhaps can fork Wulfheart:strhelpers and build on it. I think it would be good if this PR is not open for too long. Maybe it would be better if few functions work, we already merge them.

@octoper I can give you write access to repository if you want and we can continue this fork together.

Yeah @Wulfheart  if I would love to help!

I hope the invitation has been sent. How can we avoid duplication? How would you like to solve it? How can we coordinate it?

Do you have a Discord account or something else so we can talk there or Twitter maybe?

@octoper @Wulfheart I have given you write rights. You should have received an invitation. I will create a branch for the above.

New pull-request:
https://github.com/confetti-framework/support/pull/12
@Wulfheart and @octoper you can develop on branch #5-string-helpers

@reindert-vetter Thank you!