Issue: Override ROOT_PATH

Hi Guys,
I try to override ROOT_PATH with a separate path. Got an error:
~\AppData\Local\Continuum\anaconda3\envs\deep_pavlov\lib\site-packages\deeppavlov\models\squad\squad.py in __init__(self, word_emb, char_emb, context_limit, question_limit, char_limit, train_char_emb, char_hidden_size, encoder_hidden_size, attention_hidden_size, keep_prob, min_learning_rate, noans_token, **kwargs)
     71         self.noans_token = noans_token
     72
---> 73         self.word_emb_dim = self.init_word_emb.shape[1]
     74         self.char_emb_dim = self.init_char_emb.shape[1]
     75

AttributeError: 'NoneType' object has no attribute 'shape'

What I did is:
"ROOT_PATH": "/C/Users/kx764qe/Desktop/deep_pavlov/deep_pavlov_model", In this folder, I have below:

I am using: multi_squad_noans_infer.json.  Can anyone give some insights on how to fix it?
Thanks. Luke

Hi @luke4u,
It's a Windows thing. You can replace it with "C:/Users/kx764qe/Desktop/deep_pavlov/deep_pavlov_model" or "~/Desktop/deep_pavlov/deep_pavlov_model" if you're running this as kx764qe

Thanks @yoptar . problem solved.


Check user signup against class roster in Canvas

When users sign up to Autograder, we should check whether or not they are already listed as student of the course in Canvas. If they are in the class roster we can auto-enroll them into the course without teacher approval. If they use a different name, the teacher should be able to reject their enrollment or change their name accordingly, so that it becomes possible for teachers to map Autograder users to actual students signed up to the course.
@0xf8f8ff Can you check the docs to see if this would be difficult to implement?
https://canvas.instructure.com/doc/api/enrollments.html
https://canvas.instructure.com/doc/api/users.html
PS: Looks like graphql api is the preferred one:
https://canvas.instructure.com/doc/api/file.graphql.html
There are several Go libs for graphql:
https://github.com/graphql-go/graphql
https://github.com/graph-gophers/graphql-go

Reference to case number: INC0136440 with subject Canvas API access. Reopen case file when/if we decide to implement this feature, to request access to relevant courses. This link may also be relevant:
https://canvas.instructure.com/doc/api/courses.html


Gas Profiling

Component
Forge
Describe the feature you would like
Hey everyone, first off thank you for the work on foundry so far, I've been enjoying working with it.
I'd like to ask about gas metering though - what dev workflows do people use/recommend for profiling functions at the moment, and what's the current general plans for future work on this aspect.
Here's the methods I'm presently aware of:

There's forge snapshot which gives you gas costs for test functions. This suffers from not giving per function breakdown, and whilst you can structure your tests to minimise the code running that isn't the function you're trying to profile that's pretty messy.

There's forge test --gas-report which summarises gas costs for function calls as well. At a glance this looks just like what I'm after, but re-running it a few times reveals that the min,avg,max values can differ by significant margins from run to run. It seems that it only tallies the gas cost for a function once per test function, even if the test is fuzzed? In doing so it would appear that it's perhaps inadvertently often using threshold fuzzed input that makes the function behave more drastically different, perhaps. I think I read in passing that only profiling a single run from a fuzzed test was a deliberate decision to keep execution times short, but I think it would benefit from a cli flag that enabled the slower but more representative profiling every run.

Then there's the vm cheatcodes to pause and resume gas consumption. In theory you can use these to ignore the gas cost for the code that sets up test scenarios, only metering the gas consumed by the function call that the test is testing. Unfortunately, judging by several other issues (https://github.com/foundry-rs/foundry/issues/4523, https://github.com/foundry-rs/foundry/issues/4370, etc) this doesn't always work as expected.

There's also the gasleft() approach where end users can manually keep track of gas consumed between two arbitrary points in a test function. This is perhaps the most reliable approach I know of right now, but even that isn't flawless unfortunately. Warm/cold access, memory expansion, and probably some other EVM details mean that even if you're ignoring the gas cost of the test setup code, it's still impacting the execution cost of the function you're trying to profile.

In order to address the shortcomings with approach 4 it seems to me that it requires fairly extensive work: some sort of cheatcode that can be used to tell forge to run the subsequent code as if it were a new transaction, giving it fresh state to work with. I'm not remotely familiar with forge internal working so no idea how practical attempting that even is, alas.
So to recap:

what approach do people find works best currently?
what are the general plans for how forge should do gas profiling in the future?
is forge ever likely to support more accurate function profiling, or would it be recommended to use other tools (eg. hardhat) if that extra accuracy is important?

Additional context
No response

Any progress on this

Any Answer for this... it is important for developers...

We've since added the option to run tests in isolation: https://book.getfoundry.sh/reference/cli/forge/test?highlight=isolate#forge-test, this greatly improves the accuracy of tracking gas.
--gas-report now also uses this flag.
Other improvements like those developed in https://github.com/marktoda/forge-gas-snapshot/ can potentially be upstreamed
We also support the vm.lastCallGas cheatcode: https://github.com/foundry-rs/forge-std/blob/8948d45d3d9022c508b83eb5d26fd3a7a93f2f32/src/Vm.sol#L453-L454 to be able to get gas information about calls directly inside of your tests.
Converting this ticket into a discussion as there are no direct actionable items.


Invalid signature for com.prolific.driver.PL2303, will not load.

I have a clean install of El Capitan (10.11.1), it looks like the pl2303 cask doesn't work...
linickx:~ $ brew cask info pl2303
pl2303: 1.5.1
Prolific USB-Serial Cable driver
http://www.prolificusa.com
/opt/homebrew-cask/Caskroom/pl2303/1.5.1 (3 files, 1.5M)
https://github.com/caskroom/homebrew-cask/blob/master/Casks/pl2303.rb
==> Contents
  PL2303_MacOSX_v1.5.1.pkg (pkg)
linickx:~ $
linickx:~ $ brew cask audit pl2303
audit for pl2303: passed
linickx:~ $
linickx:Extensions $ pwd
/System/Library/Extensions
linickx:Extensions $
linickx:Extensions $ sudo kextutil ProlificUsbSerial.kext
Diagnostics for /System/Library/Extensions/ProlificUsbSerial.kext:
Warnings:
    Dependency lacks appropriate value for OSBundleRequired and may not be availalble during early boot:
        com.apple.iokit.IOSerialFamily - Safe Boot

Code Signing Failure: not code signed
Warnings:
    Dependency lacks appropriate value for OSBundleRequired and may not be availalble during early boot:
        com.apple.iokit.IOSerialFamily - Safe Boot

Untrusted kexts are not allowed
ERROR: invalid signature for com.prolific.driver.PL2303, will not load
linickx:Extensions $

This looks like a possible alternate vendor: http://www.prolific.com.tw/US/ShowProduct.aspx?p_id=229&pcid=41

This is an upstream issue. Nothing we can do about it, unless remove the cask or make it not install in El Capitan.


Add new cluster command MYID

Redis 3.0-rc5 introduced a CLUSTER MYID to return current nodeId.

Coverage decreased (-0.02%) to 95.53% when pulling c79ef2c34b4c5476c0adbc15f1215d4209933ff4 on taer:clusterMyId into 6591a0407ab03508c17d5aabfaed01d8339ace4f on mp911de:master.

Great, thank you. Will also help to improve tests which try to find out the current node id.


Did the data-do-not-track Tracker Configuration get removed in 2.0?

Describe the Bug
In versions below 2.0, there used to be a data-do-not-track option for the tracker configuration that would honour the user’s "Do not track" setting. This seems to no longer be the case in post v2.0 – or is the option simply missing from the docs?
Database
MySQL
Relevant log output
No response
Which Umami version are you using? (if relevant)
No response
Which browser are you using? (if relevant)
No response
How are you deploying your application? (if relevant)
No response

It was removed because it is not a standard and not supported in all browsers. You can trivially add it to your own code, but we didn't want to bloat the tracker code with something not commonly supported.

Thanks for letting me know, perhaps this should be added to the document about how to migrate to v2.


MypyStub doesn't recognize Decimal

Describe the bug
When I run mypy on this code snipped I got this error
Argument "Item" to "put_item" of "Table" has incompatible type "Dict[str, object]"; expected "Mapping[str, Union[bytes, bytearray, str, int, Decimal, bool, Set[int], Set[Decimal], Set[str], Set[bytes], Set[bytearray], Sequence[Any], Mapping[str, Any], None]]"  [arg-type]

To Reproduce
Steps to reproduce the behavior:

Install boto3-stubs[...]
Run mypy/pyright on the following code sample

        self,
        history_id: HistoryID,
        latitude: float,
        longitude: float,
        accuracy: float,
        timestamp: int,
    ):
        item = {
            "history_id": str(history_id),
            "activity_id": str(uuid.uuid4()),
            "location": {
                "latitude": self._make_decimal(latitude),
                "longitude": self._make_decimal(longitude),
            },
            "accuracy": self._make_decimal(accuracy),
            "timestamp": timestamp,
        }
        boto3.resource(
            "dynamodb",).Table("measurements").put_item(Item=item)

    @staticmethod
    def _make_decimal(value: float) -> Decimal:
        return Decimal(str(value))
...

Additional context
Windows Machine with poetry

Thank you for the report!
I do not think that this is because of Decimal, because Decimal is accepted as Item mapping value. You can check if HistoryID.__str__ method returns a string, this could be an issue. Otherwise, I would try to comment item keys one by one and find a conflicting value this way.
Let me know if this helps.

Hi :) I've double checked and it works when I comment all decimal property. HistoryID is a UUID field.

Well, I still cannot reproduce it with Decimals. It would be helpful if you could send a small sample that I can run to reproduce the issues.

You can create a new branch and try to reproduce this bug here: https://github.com/youtype/stubs-playground


PBA Transcripts from Week 1

Basic plan:

[x] Convert lecture videos to text files
[x] Upload transcripts to a Google drive
[x] Rough edit
[ ]  Polish edited text
[ ] Integrate (or replace) existing topics with the new material

Completed


Command PhaseScriptExecution failed with a nonzero exit code. Xcode 13.1 + SwiftUI

Hi. I create a project in Xcode 13.1 with SwiftUI, add a script and get this error. Install via cocoapods

If I create a project without using SwiftUI, then everything works fine.

The current version of R.swift requires that a Info.plist file exists. Xcode 13 / SwiftUI projects don't have this file by default.
Make sure to disable the "Generate Info.plist File" build setting, and add a manual Info.plist file.
The next version of R.swift will not require for the Info.plist file to exist.

R.swift 6.0.0 was just released, this removes the requirement for an Info.plist file to exist.
Thank you


There's no reliable way to read and suppress terminal output

When dealing with VT terminals I need to be able to Read-Host from the terminal input, without waiting for {Enter}, and on Windows, we use $Host.UI.RawUI.ReadKey("NoEcho") for that, but it's failing elsewhere.
Steps to reproduce
On Windows, the following allows me to read responses from the terminal (without having them show up):
function Read-Terminal {
[CmdletBinding()]
param([string]$EscSequence)
  $Host.UI.Write($EscSequence)
  -join $(while ($Host.UI.RawUI.KeyAvailable) {
    $Host.UI.RawUi.ReadKey("NoEcho,IncludeKeyDown").Character
  })
}

So for instance, the CSI code 6n causes the terminal to report the cursor location, and we can read our X, Y from the terminal like this:
$answer = Read-Terminal "`e[6n"
"Row {0}, Column {1}" -f @($answer -split ';' -replace '\D')

Will get something like:
Row 29, Column 1"

Actual behavior
But when I try to run the same code on linux (even in the WSL terminal) the $Host.UI.RawUI seems to echo no matter what I do, so the terminal's response is completely unsuppressable.
PS /root> cls; $answer = Read-Terminal "`e[6n"
^[[1;1R

I would be remiss not to point out that in addition to this being unreadable, the escape sequence is being shown as ^[ instead of the expected `e
Environment data

Name                           Value
----                           -----
PSVersion                      6.2.2
PSEdition                      Core
GitCommitId                    6.2.2
OS                             Linux 4.4.0-18362-Microsoft #1-Microsoft Mon Mar 18 12:02:00 PST 2019
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Just for context, using the read built-in with bash (wsl or native linux), we'd use something like read < <(echo -e -n "\e[6n"); echo "The reply is ${REPLY}"
(REPLY is the default token when you don't specify one in read)

But when I try to run the same code on linux (even in the WSL terminal) the $Host.UI.RawUI seems to echo no matter what I do, so the terminal's response is completely unsuppressable.

Before core 3.0, terminal input echo was always enabled on Unix, unless ReadKey(intercept: true) was running.  We had a similar issue in PowerShellEditorServices, so I stripped out the native code corefx uses to disable echo (and otherwise prep termios) and put it in the UnixConsoleEcho package.  That works, but I'd recommend just requiring PowerShell 7 instead, shipping native binaries isn't fun. Plus you have to be very careful about reverting, because it'll affect child processes.
Note that the problem isn't $Host.UI.RawUI.ReadKey, it's that ReadKey isn't running when input is pressed.  Since you're waiting until a key is available to read it, input will still be enabled at the time of the key press.
Also, can't text on *nix atm, but avoiding KeyAvailable should work:
$Host.UI.Write("$([char]27)[6n")
$chars = while (($key = $Host.UI.RawUI.ReadKey('NoEcho, IncludeKeyDown')).Character -ne 'R') {
    $key.Character
}

$y, $x = -join $chars.Where($null, 'Skip', 2) -split ';'

Nice workaround Patrick. Pity about needing to know about a terminating character. I suppose another alternative would be to make it timeout based, but spinning up threads while blocking on input causes hell on corefx on windows. I tried to make psreadline's argumentcompleters more robust by having them timeout some years ago but it was deadlock prone and flaky AF.

I suppose another alternative would be to make it timeout based, but spinning up threads while blocking on input causes hell on corefx on windows.


Document all package dependencies for Meltano

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/462
Originally created by @valexieva on 2019-03-07 23:31:47

Problem to solve
Currently, we use different versions for python for different parts of the product and at least on 2 occasions I had to downgrade my packages. Moreover, I didn't realize the error log I was seeing was coming from that.
Additionally, we have dependencies that are not locked down within the Docker instance we spin up and the user may see errors for that which they would not be able to resolve without becoming contributors. We have an open question of whether we should lock these underlying dependencies down to specific version to avoid this, which would have pros and cons.
Target audience
Any user of Meltano, past and future
Further details
People should be aware of package dependencies and we should ensure consistency across features. This would prevent unexpected errors and debugging time. Users should not be impacted negatively by things that they are no able to re-configure/troubleshoot themselves.
Proposal

[X] Review and ensure we are currently consistent across features and discuss what needs to be upgraded
[X] @jschatz1 Update Meltano documentation in the wiki - just a link to an updated requirements.txt
[X] If we move to python version 3.7.2, website should be updated - Decision: not bumping the documentation for now, doesn't seem to be impacting anyone
[ ] Make a decision about whether to lock down other dependencies, and create an issue to complete that work if the decision is to lock down -- Decision: YES, we locking down and @jschatz1 is going implement

What does success look like, and how can we measure that?

All package dependencies documented on a wiki and all feature dependencies to be consistent as of now.
Users not blocked from using Meltano due to dependencies

Links / references

View 10 previous comments from the original issue on GitLab


Weird font glitch

Sometimes when running the engine, the debug text seems to run across the screen occasionally with ASCII characters seen but other times just white horizontal stripes.

Seems to occur more often when there is more font being rendered

Fixed - Ignoring escape characters when recalculating the vertices data seemed to fix it


Module to use Anthropic's Claude

Hello,
After using your tool I dove into the code base to figure out how you structured the cognitive architecture. My only issue was running out of context window. I have a key for Anthropic's API so I went ahead and built a module that lets me run Claude. Its not working fully yet since there are a lot of interactions I am unaware of. I was hoping you'd be willing to check it out. I worked on it for too long today and I cant see my mistakes from exhaustion lol.
I used a fairly inelegant approach and just highjacked the base coder after I confirmed it was Claude. If you want me to refactor it into a more civilized way before I make a pull that would be fine. Mostly I wanted to get it running on my side.
let me know what you think.

Aider is an amazing project - freaking love it - but I too run into context length issues often.
Would love to see Anthropic's API integrated, although I am still waiting for access.

Thanks for trying aider!
There's a FAQ entry that might contain helpful information:
https://aider.chat/docs/faq.html#can-i-use-aider-with-other-llms-local-llms-etc


Does not accept float number

If you try to validate float number like 0.25 it will get exception:

first try float with default multiple
Schema("quantity").number().float().min(0).max(1000).required()
""quantity" should be a multiple of 1"

second try with precision multiple
Schema("quantity").number().float().multiple(0.1).min(0).max(1000).required()
""quantity" should be a multiple of 0.1"

In the second case the problem is:
        if value % self._multiple_of != 0:
            raise MultipleException(self.field, self._multiple_of)

if you test 0.25 % 0.1 it will be "0.04999999999999999" that is different of 0

In the code, the multiple function only accepts integers as its argument, and that's because floating point maths is weird.
You're not supposed to give a float in the multiple() function.
As for the first problem, I'll create a PR for fixing that.

Update: Said PR is at #5


Output variables in downstream job don't exist

Agent Version and Platform
Version of your agent? Hosted VS2017 - Agent version 2.140.0
OS of the machine running the agent? Windows
VSTS Type and Version
VSTS
If VisualStudio.com, what is your account name? https://readify.visualstudio.com
What's not working?
In the guide Output Variables under Examples it describes using a tasks output variables in a downstream job such as from Job_2 using $(Job_1.TaskA_1.AuthToken). In another task of the same job, this works (without using the Job_1 prefix) but doesn't in a separate job.
Setup:

JobA - Hosted VS2017

PowerShell Task

inline script Write-Host "##vso[task.setvariable variable=ChangeFilePath2;isOutput=true]C:\Test"
Output Variables - Reference Name PS1Output

PowerShell Task

inline script Write-Host "This works: $(PS1Output.ChangeFilePath2)"

JobB - Hosted VS2017

PowerShell Task

inline script Write-Host "This doesn't work $(JobA.PS1Output.ChangeFilePath2)"

Agent and Worker's Diagnostic Logs
JobA.PS1Output.ChangeFilePath2 : The term 'JobA.PS1Output.ChangeFilePath2' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

have you tried with yaml?
https://github.com/Microsoft/azure-pipelines-agent/blob/master/docs/preview/outputvariable.md

No we haven't. I did see in other issues that this was originally just supported in yaml definitions, but the docs page didn't mention that limitation so was using our existing pipeline in the visual editor.
Is this still a restriction for this support (yaml only)?

Thanks @chrisrpatterson. How do we explicitly map them across jobs?
There isn't a mention in the linked readme of doing anything explicit. Just access them in JobB as $(JobA.PS1Output.ChangeFilePath2). Do we have to register/mark these in some way?

@vtbassmatt I can't find information about this in the new docs
@damienpontifex here is an example in our old yaml docs

It's there, but not very discoverable. All the way at the bottom of this doc.
I have a PR up (internally) which puts a better example in the conditions topic.

@vtbassmatt that is a different scenario. The scenario here is consuming the variable in a downstream job (on the agent) - not using the output variable to determine the job condition (orchestrator). Consuming the variable on the agent side requires extra steps to map it into the job message that gets sent down to the agent.

Ah gotcha. OK, I need to find a place for that.

Nevermind, it's already there: https://docs.microsoft.com/en-us/azure/devops/pipelines/process/variables?view=vsts&tabs=yaml%2Cbatch#set-an-output-multi-job-variable

Currently in the visual editor, we can only set pipeline variables...correct me if I'm wrong?
Just using pipeline variables doesn't seem to work:
Setting variable:

Name: ChangeFilePath2
Value: $[ dependencies.JobA.outputs['PS1Output.ChangeFilePath2'] ]

Then in the second job from PowerShell running Write-Host This is $(ChangeFilePath2) just echoes $[ dependencies.JobA.outputs['PS1Output.ChangeFilePath2'] ] rather than the variable set in JobA.
Am I correct then that this requires job variables? Which are only currently available in yaml and still coming for the visual editor?

It works within a job, but not across jobs using the visual editor. If you need to flow variables across jobs, you'll have to use YAML. No timeframe for when it might come to the visual editor.

Thanks for all the assistance on this. In the meantime, we will try it out with yaml.
One question: how can I create a release with a yaml definition? I noticed a few projects on GitHub have a .vsts-release.yml file, but creating this in my repo doesn't create a release definition like a .vsts-ci.yml file does with build. Or do I have to create a release with this file somewhere/somehow from the UI?

We don't have our Release product running on YAML yet. I see we're using .vsts-release.yml right here in our own repo, but it's really using Build, not Release.

Yep.  It's using pipelines (build) yaml to do a release.

So TL;DR summary from these discussions:

Using output variables in another job is not possible in releases
Is possible using build and yaml definition with job variables defining the dependency (not build variables)

I'm not sure what the "(not build variables)" part means. Other than that, you've correctly summarized the situation :)

Meaning you have to define the dependency at the job level.
Not sure if it's the same in yaml when defining variables, but like this from a previous comment when I defined the variable for the entire definition

Just using pipeline variables doesn't seem to work:
Setting variable:

Name: ChangeFilePath2
Value: $[ dependencies.JobA.outputs['PS1Output.ChangeFilePath2'] ]
Then in the second job from PowerShell running Write-Host This is $(ChangeFilePath2) just echoes $[ dependencies.JobA.outputs['PS1Output.ChangeFilePath2'] ] rather than the variable set in JobA.

And thanks for confirming 😄

I am confused... can someone from MSFT declare answer to following question?
Does this support designer pipeline to pass output variables set in one job to a task in another job in same pipeline

Designer pipelines don't support passing variables from a task in one job to another job. YAML pipelines do support this.

Thanks for summary of current situation.
There is still confusion as to what the expected target situation should be: is this by design or will Designer Pipelines (especially for Release) be aligned on the YAML behavior?

We don't expect to bring this feature to the designer environments. We're working on making YAML fully capable to support release pipelines.

What do you suggest for organizations like mine, who are planning to start
full on Azure devops usage for CICD pipeline, should we start making our
users adapt to YAML definitions or can give a freehand to have them use
designer at will.
Your statement obviously is a red flag to me, saying that "an year or two
down the timeline, we no longer support designer"
On Wed, Jan 2, 2019 at 10:15 AM Matt Cooper<EMAIL_ADDRESS>wrote:

We don't expect to bring this feature to the designer environments. We're
working on making YAML fully capable to support release pipelines.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/azure-pipelines-agent/issues/1842#issuecomment-450907009,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AV7GICWg5f2ziZIHaWdKqdMFXzmY4BKGks5u_Ns_gaJpZM4WzVJk
.

--
Ganesh Phani Majeti.

Thanks for your response Matt. Looks like it is safe to stick to YAML to
have full access to broadest of features.
On Thu, Jan 3, 2019 at 6:29 AM Matt Cooper<EMAIL_ADDRESS>wrote:

We don't plan on dropping support for the designer. Definitely not in a
year or two - I can't name a feature that this team has removed that
quickly in its entire 15+ year history. It's true that new features will
land in YAML first. Some, but not all, of them will also come to the
designer.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/azure-pipelines-agent/issues/1842#issuecomment-451130016,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AV7GIBaCiqA31D1cM_35SXIU0PK7nbTHks5u_fe7gaJpZM4WzVJk
.

--
Ganesh Phani Majeti.

Please bring this to the designer. I would hope that when release pipelines are running on top of YAML that this wouldn't be too difficult to make happen.

Any update about this functionality in the classic designer?


Support for Kubernetes v1.30

How to categorize this PR?

/area open-source
/kind enhancement
What this PR does / why we need it:
Support for Kubernetes v1.30.
Which issue(s) this PR fixes:
Part of https://github.com/gardener/gardener/issues/9508
Special notes for your reviewer:

Code changes have been tested in the following scenarios:

Create/Delete a new (normal/workerless) Shoot cluster with 1.30.0
Upgrade an existing (normal/workerless) Shoot cluster to 1.30.0

Release note:

Gardener can now support clusters with Kubernetes version 1.30. To allow creation/update of 1.30 clusters you will have to update the version of your provider extension(s) to a version that supports 1.30 as well. Please consult the respective releases and notes in the provider extension's repository.

Gardener can now support clusters with Kubernetes version 1.30. Extension developers have to prepare individual extensions as well to work with 1.30.

/assign

/assign

/assign

cc @rfranzke, Can we go ahead with this PR?

/test all


Class paramter in training promprt

It seems there is no class parameter in training prompt, eg "sks dog" (where dog is a class and sks is an instance name).
I see that there is $PT variable, but can't find definition for it to check.
https://github.com/TheLastBen/fast-stable-diffusion/blame/main/fast-DreamBooth.ipynb#L1032

This colab uses the image filename as an instance name, you can also use manual captioning

Thanks for answering! I already read that it's by design. Closing.


Can´t add/edit items on list

Hello Roger,
thanks a lot for your work!
Reading the lists works fine but I can´t add items or mark them as done.
Any hint (I have set the Tasks.ReadWrite in the API-permissions)?

Did you enable update in the configuration?

Ah, no.
Sorry to ask: which is the configuration file that I have to change?

It’s not in a configuration file, it’s in the configuration of the integration. Click on the there dots and select reconfigure.

Aah, I had overlooked that. Works now - thanks a lot! :)


Add GitHub links to mock component actions

The existence of Wingman is still a bit mysterious. This adds an explicit link from our MockComponentActions to the relevant directory on GitHub. This will allow curious developers to go directly from the mock component on the Developer Site to the Wingman source.

Candy would be proud of you Ryan


Installation under Symfony 4.2 not possible

Installation under Symfony 4.2.x not possible, after entering command "composer require php-http/httplug-bundle" composer says:
Your requirements could not be resolved to an installable set of packages.
Problem 1
- Installation request for php-http/httplug-bundle ^1.14 -> satisfiable by php-http/httplug-bundle[1.14.0].
- php-http/httplug-bundle 1.14.0 requires php-http/cache-plugin ^1.6 -> satisfiable by php-http/cache-plugin[1.6.x-dev] but these conflict with your requirements or minimum-stability.
Installation failed, reverting ./composer.json to its original content.

this seems to work now. i tested:
composer create-project symfony/website-skeleton sf42
cd sf42
composer require php-http/httplug-bundle guzzle6-adapter

and end up with
psr/http-client (1.0.0)
php-http/httplug (v2.0.0)
php-http/guzzle6-adapter (v2.0.1)

the flex recipe is still on httplug 1, but that is a different topic.


Safari currently supports this

In the past I hadn't been able to use this because of Safari.
Today I was looking and saw several snippets where image paste works in Safari:
https://jsfiddle.net/SaschaKluth/togafu5s/1/
http://jsfiddle.net/KJW4E/754/
So I revisited this and found out that the table still doesn't list Safari but that it indeed supports it
in all three modes:

Perhaps you'd like to update the table? I tested with Safari Version 10.1 (126<IP_ADDRESS>.34). Haven't tried with versions older than Sierra's but I could confirm with El Capitan's later today.

Hi @eduo thanks for finding this! I've added the change to READMEs c6487b9 . Cheers.

That's great news! 👍


A good documentation !

Hey thanks for he great work ! went through the paper.
Please include documentation regarding :

Inference
Finetune
Changes for the config file needed

Hope these commits can help

Thank you so much. 😊


Version 0.10 changes

Breaking changes

[x] Rewritten strategies #75
[x] Moved broker.createNewContext
[x] Killed nodeID == null in core modules. #81
[x] changed internalActions to an internal service #81

New

[x] AMQP transporter #72
[x] Better remote call metrics #78
[x] Hot reload services

Fixes

[x] Broker direct call nodeID #67
[x] Runaway promises #71 #77

Docs

[x] update CHANGELOG
[x] update docs on moleculer site
[x] add protocol doc to moleculer site

Released


Tweak sidebar ad priority

This is a tweak to increase the sidebar ad priority (vs footer ad priority) if the sidebar isn't long. Essentially, this change will prioritize showing a sidebar ad if the sidebar ad would be visible.

If I understand correctly, we check

This is correct. Basically if the sidebar ad would be in view, raise the priority of choosing a sidebar ad. If the sidebar ad would require scrolling to see, instead lower the priority. Most likely we'll choose a footer ad.


SECC Simulator: ModuleNotFoundError: No module named 'aiofile'

First of all, thank you for the iso15118 implementation!
I am trying to run the docker environment on a Ubuntu system. However there seems to be an issue with one of the python packages needed by the SECC simulator, namely "aiofile":
secc_1 | Traceback (most recent call last):
secc_1 |   File "/venv/bin/iso15118", line 5, in
secc_1 |     from iso15118.secc.main import run
secc_1 |   File "/venv/lib/python3.10/site-packages/iso15118/secc/main.py", line 5, in
secc_1 |     from iso15118.secc.controller.simulator import SimEVSEController
secc_1 |   File "/venv/lib/python3.10/site-packages/iso15118/secc/controller/simulator.py", line 12, in
secc_1 |     from aiofile import async_open
secc_1 | ModuleNotFoundError: No module named 'aiofile'
I checked the poetry file etc. and the aiofile packages seems to be in there. But it seems that it is not being installed, although another package e.g. "alabaster" is included:
• Installing attrs (21.4.0)
• Installing markupsafe (2.1.1)
• Installing iniconfig (1.1.1)
• Updating packaging (20.9 -> 21.3)
• Installing pluggy (1.0.0)
• Installing py (1.11.0)
• Installing pytz (2022.1)
• Installing alabaster (0.7.12)
• Installing babel (2.10.1)
• Installing click (8.1.3)
• Installing coverage (6.3.3)
• Installing docutils (0.17.1)
• Installing imagesize (1.3.0)
• Installing jinja2 (3.1.2)
• Installing marshmallow (3.15.0)
• Installing mccabe (0.6.1)
• Installing pathspec (0.9.0)
• Installing pycodestyle (2.8.0)
• Installing pyflakes (2.4.0)
• Installing pygments (2.12.0)
• Installing pytest (7.1.2)
• Installing python-dateutil (2.8.2)
• Installing python-dotenv (0.20.0)
• Installing snowballstemmer (2.2.0)
• Installing sphinxcontrib-applehelp (1.0.2)
• Installing sphinxcontrib-devhelp (1.0.2)
• Installing sphinxcontrib-htmlhelp (2.0.0)
• Installing sphinxcontrib-jsmath (1.0.1)
• Installing sphinxcontrib-qthelp (1.0.3)
• Installing sphinxcontrib-serializinghtml (1.1.5)
• Installing asynctest (0.13.0)
• Installing black (22.3.0)
• Updating cryptography (37.0.2 -> 3.4.6)
• Installing environs (9.5.0)
• Installing flake8 (4.0.1)
• Installing freezegun (1.2.1)
• Installing isort (5.10.1)
• Installing psutil (5.9.0)
• Installing py4j (<IP_ADDRESS>)
• Installing pydantic (1.9.1)
• Installing pytest-asyncio (0.18.3)
• Installing pytest-cov (3.0.0)
• Installing sphinx (4.5.0)
Do you have any idea what is going wrong? Thanks in advance

Hi @RadioX90 thanks for the feedback,
This issue was also "reported" here: https://github.com/SwitchEV/iso15118/pull/51
I have a fix for it here: https://github.com/SwitchEV/iso15118/pull/55
However, I believe I actually think that the easiest way is just to install the dev dependencies in the end image as well in docker to solve the issue. I will investigate this, but in the mean time check the comments in the #51 and the code in #55

fix #130


anonymous rest arguments

Since parameter names can be omitted (see Anonymous Parameters), how do I type rest arguments anonymously? How about:
Named:
(...args: Any[]) => Array

Anonymous:
(...Any) => Array

IMO, this adds needless complexity to the spec. Why not simply use the very generic args name?
I'd say, stick with (...args: Any[]) => Array.

If you omit the parameter name in
(...args: Any[]) => Array

you get
(Any[]) => Array

which is not the same type. Another suggestion would be:
(...Any[]) => Array

IMO, this would be more consistent since ... does not really belong to the name of the parameter. It just declares the parameter as rest parameter. If you use the rest parameter, you just write args instead of ...args, which is something different (due to the spread operator).
After all, I would give this issue a low priority :sweat_smile:

... does not really belong to the name of the parameter.

That's the real problem then.
Is (args: ...Any[]) => Array what you want?

Is (args: ...Any[]) => Array what you want?

No. The current correct syntax is:
(...args: Any[]) => Array

I don't see a good reason to complicate parsing just so we can add the potentially confusing shortcut (...Any[]) => Array.

@ericelliott For the record, why is it the "correct" syntax?
@maiermic If we don't change the current syntax here's your answer:
(...: Any[]) => Array

For the record, why is it the "correct" syntax?

Because that's what is currently specified, and because it makes sense.
This makes sense and should work:
(...: Any[]) => Array

Sorry, but it doesn't make sense to me. Due to the colon it looks like ... is the parameter name, which is invalid, since dots are not allowed in the name.

Sorry, but it doesn't make sense to me. Due to the colon it looks like ... is the parameter name, which is invalid, since dots are not allowed in the name.

Since dots are not allowed in names, the ambiguity you mention doesn't exist at all...

From chat:
When I say easy to parse, I'm not just talking about human parsing. I'm also talking about the parser itself. (...Any[]) => Array requires more complex lookahead rules than (...: Any[]) => Array because the former creates ambiguity. When we get to Any are we looking at a parameter name? (The default assumption). At what point can you discover that Any is actually the type annotation? Maybe when you hit the square brackets you can realize you've made a parsing mistake.... at which point you have to rewind the parser state.
In the (...: Any[]) => Array example, the parameter name is already optional, and the : disambiguates and makes it clear that Any[] is the type annotation. Which means there is no lookahead required, and there won't be a need to rewind the parser state.
A human being reading the signature might recognize the built-in Any type and assume that it's the type annotation, but we can't disambiguate that way in the parser, because some type names, such as Any and Predicate are perfectly valid parameter names.
I normally rate user friendliness over parser friendliness, but in this case, I feel that the (...: Any[]) => Array notation is friendlier to users, too, because it's logically consistent with the rest of the spec rules. In fact, if we change nothing in the existing spec documentation, that is currently legal syntax. If we wanted to make it illegal, we'd actually have to add a rule to the spec to do so.
Regardless of all that, I would still recommend that users stick to (...args: Any[]) => Array because there's no ambiguity, the intent is crystal clear, and it's only a few more keystrokes.

If there's no objections we will merge the addition this week.


New option: enable changing /MD to /MT

When building with MS Visual Studio, the /MD command line option links in the dynamic C runtime and /MT links the static runtime.  At the link step, every binary must have used the same runtime link flag or link errors will result.  Some libraries' build systems insist on one flag or the other: CMake only uses /MD, HDF5 strongly prefers /MD for both dynamic and static builds, while GTest changes all occurences of /MD to /MT for static builds.
This commit establishes ENABLE_MSVC_STATIC_MD_TO_MT, defaulting to ON.  This flag has effect only if building static binaries using MS Visual Studio.  If the option is OFF, BLT will leave all /MD flags generated by CMake alone and throw an error if GTest is enabled for a static build under MSVC.  If ON, BLT will change each occurence of /MD in the compiler flags to /MT to match (and enable) GTest.

Note for the future: We should discuss this option in the user docs (when that page exists #324 )

there are other considerations beyond gtest (other TPLS), it would be good for us to understand distill the issues in user docs (after this PR)

Sorry I forgot you can't merge this yourself @agcapps .  Merging now.

Thanks Chris.
From: Chris White<EMAIL_ADDRESS>Sent: Friday, December 6, 2019 2:31 PM
To: LLNL/blt<EMAIL_ADDRESS>Cc: Capps, Arlie<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [LLNL/blt] New option: enable changing /MD to /MT (#323)
Merged #323https://github.com/LLNL/blt/pull/323 into develop.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/LLNL/blt/pull/323?email_source=notifications&email_token=ALV2FMOAHAV6SXNKDIEDE23QXLHB5A5CNFSM4JVNXMD2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOVKSQJJY#event-2862941351, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALV2FMJ65JNY634DFK2VNJDQXLHB5ANCNFSM4JVNXMDQ.


Any Possibilities to RUN with Powerful New GEN IGPUs?

Self Checks

[X] I have searched for existing issues search for existing issues, including closed ones.
[X] I confirm that I am using English to submit this report (我已阅读并同意 Language Policy).
[X] [FOR CHINESE USERS] 请务必使用英文提交 Issue，否则会被关闭。谢谢！:）
[X] Please do not modify this template :) and fill in all the required fields.

1. Is this request related to a challenge you're experiencing? Tell me about your story.
In today's world we have quite powerful igpu's(Integrated Gpu) like amd ryzen 780M and 880M which are performing very well like dedicated nvidia dedicated mid range graphic card like nvidia 1660 and are provided in handheld ROG devices too (You can read and see more news and things about these powerful igpus).
New upcoming laptops are also coming with amd processors with these igpu's which are powerful enough to run 8-13B quantized llm models too.
So is it possible for you to add support for these powerful IGPUs which can run this project without nvidia gpu, if yes then it would be great as more and more people will be able to experience this awesome project.
2. Additional context or comments
No response
3. Can you help us with this feature?

[ ] I am interested in contributing to this feature.

No one in development team have AMD or Intel IGPU, even Discrete Graphics.
So for now, no offical support plan, PR Welcome.


Always getting a 403 when trying the example

Hi,
I've just cloned the example and I followed the instructions. However when running it on localhost I get a 403.
How do I specify the masterKey in my header?
Thank you.

Specify the master key on the command line or in your config file. If you get a 403 error, that probably means there was a typo in your master key or app ID.

Fixed it :-) Thank you


[Question] Text selection colour

Is there a simple SCSS variable defining the text selection colour? Where is it defined in the code?
I'm not too familiar with CSS/SCSS, but I'd expect the ::selection or user-select properties somewhere, but can't seem to find it

No, it's just using the default one.
You can add it in public/stylesheets/styles.scss


Package directory and package.json name must be exactly the same, oth…

…erwise bugs.
Learnt this the hard way, makes it impossible to create a settings button in the packages panel.

Pedantic


Removed duplicate definition of javadoc-plugin

Removed duplicate definition of maven-javadoc-plugin.  Maven was complaining that there were multiple versions of the javadoc plugin.  As the javadoc plugin was declared outside of the quality profile group the other (later) plugin remains.

fixed this slightly differently, but thanks for the reminder :)


Add ChromeCustomTabs

Issue

none

Overview (Required)

I was going to add ChromeCustomTabs and fix to open web page.
I'm super super sorry to do not use awesome helper developed by our FRIEND. 😿

Links

https://developer.chrome.com/multidevice/android/customtabs

Screenshot

Before
After

:eyes:

Thanks! Commented!

I'm super super sorry to do not use awesome helper developed by our FRIEND. 😿

Never mind.

LGTM! 💯 Thanks!


Documentation for large-scale deployent

Multi-Backend would be nice, but that's currently out of scope. Let's assume single server, cloud or server relay.

Currently if you backup to the same backend, you can't view others files, but you can delete other's backups. For multi-user, through a relay server (fwd all storage to cloud, but do not allow clients to delete), for enterprise, introduce a single point of failure - only admins may delete (but with power comes responsibility).
Notifications to admin, when no backup from client x for n time.
Faster cross-platform mass-deployment: (to be addressed in another issue, 6hrs introducing myself to Restic, need sleep)

Currently if you backup to the same backend, you can't view others files, but you can delete other's backups.

Unless I'm missing something, that's no longer true anymore with the --append-only option, as of #784.  See Append-only backups with restic and rclone for implementation details.

@colans --append-only is a client side option. Assume client no trust.
This would probably need a special back-end, what supports enforcing only appending.

Yes, it's a client-side option, but it can be enforced by the server.  Please follow the link I posted for more information.

The first item was addressed above.  For the other two:

Notifications to admin, when no backup from client x for n time.

Scheduling is outside the scope of Restic.  For that, you need something like Backupninja and the Restic patch.  Basically, Restic doesn't even run until you tell it to run.

Faster cross-platform mass-deployment: (to be addressed in another issue, 6hrs introducing myself to Restic, need sleep)

@jtagcat  Please add a link from here to there.
I think we can close this one.

Currently if you backup to the same backend, you can't view others files, but you can delete other's backups. For multi-user, through a relay server (fwd all storage to cloud, but do not allow clients to delete), for enterprise, introduce a single point of failure - only admins may delete (but with power comes responsibility).

I use rest-server for my clients. Each client backs up to one single server running rest-server. This means each have their own account and can only access their own repository.

Notifications to admin, when no backup from client x for n time.

I use a simple little program/script that checks for the latest file under the snapshots/ directory in each repository in the rest-server filesystem. If you do this, you can make it alert you when one of the repositories haven't gotten a new snapshot file for repository X in N days.

Faster cross-platform mass-deployment: (to be addressed in another issue, 6hrs introducing myself to Restic, need sleep)

Not entirely sure how you define this (each have their own use cases). Personally I use a restic script that's wrapped in a .app for my Mac clients. To deploy a new client, all I do is copy the .app bundle to the user's ~, run it to make it populate ~/.config/restic/ with some files, add the rest-server and the repository passwords to the config file, then call crontab with an argument to make it schedule the backups. Takes about 3-5 minutes per client, could probably be scripted in a more serious deployment scenario.
Would the above solve the things you mention? I'm curious.
If you have a more specific feature request that's suitable for this issue format, please update the original post in this issue so we can label it and treat it as such. Otherwise it's more of a discussion and we'll probably close it for maintenance reasons. Thanks!

The one rest-server per client is an idea I didn't think and works around the issue.
Now that I think of it, one more thing: clients should not be able to delete anything (for compliance and protection for more targeted malware). You could in theory do scripting on the server side to make it read-only for rest-server after a backup.
Indeed a script could be created.
These would all be workarouds, I would leave the issue open for a more integrated solution (would actually close this and create more issues, dividing this issue):

configure restic-server:
allow clients to only see their own backups (sso or such user authentication? (multiple devices, client can restore jtag-client2 backups from jtag-client1, but not from rawtaz-client1))
switch to prevent client side snapshot deletion (exploit: clients could back up 100 times, overriding actual backups, depending on retention policy, one could effectively delete some or all backups)
restic clients get a new command to get their configs (something like restic get-config <url>), after getting the config, allow to a) choose from existing configs (users) b) name the device (allow server-side configuration).
Then the client should set up a default cron to call itself until it gets a config from the server. It would also be useful.

Honestly, I think you are a bit misinformed here.. Everything you write, except the last bullet point, is already supported. And you do not need to set up one separate rest-server per client.

The one rest-server per client is an idea I didn't think and works around the issue.

Not needed, rest-server already has support for multiple users. You set them up by adding their passwords to a .htaccess file using e.g. htpasswd. Problem solved.

Now that I think of it, one more thing: clients should not be able to delete anything (for compliance and protection for more targeted malware). You could in theory do scripting on the server side to make it read-only for rest-server after a backup.

Use the --append-only argument to rest-server. This will make your clients backing up to their repositories on the rest server only able to add backups, not delete them. Problem solved. If you want even more protection layers for this, simply snapshot the filesystem where the repositories are stored, then you can go back to that. ZFS is nice for this.

allow clients to only see their own backups (sso or such user authentication? (multiple devices, client can restore jtag-client2 backups from jtag-client1, but not from rawtaz-client1))

The rest-server already supports this, as mentioned above. Create multiple user accounts/passwords for your users, and they will be able to back up to different subfolders on that rest-server, and will only see their own repository, not others.

switch to prevent client side snapshot deletion (exploit: clients could back up 100 times, overriding actual backups, depending on retention policy, one could effectively delete some or all backups)

As mentioned above, already possible, use --append-only in rest-server.

restic clients get a new command to get their configs (something like restic get-config ), after getting the config, allow to a) choose from existing configs (users) b) name the device (allow server-side configuration).
Then the client should set up a default cron to call itself until it gets a config from the server. It would also be useful.

This one is not supported currently. Can you elaborate on what problem you are trying to solve here, and how this is supposed to work in an automated fashion (if that's your intention)?

Indeed I was. Doucmentation needed :)

@jtagcat I'd be happy if you could elaborate on documentation needed. I agree the current docs are in need of improvement. Can you for example suggest where in the manual we should add what? I'd be happy to write it up given some pointers. Your feedback is welcome!

A subtopic under installation or examples is probably the best place to put it.
It seems that a summarization of few of our posts will do, I'd add a link to this issue somewhere.
If you need anything else, ask me. Though I may not reply asap.

related: #2122

a fair idea for a workaround by youam on irc:
okay, so i'll create one repo per host, backup there, and copy the resulting snapshots over to a global repository with longer history
thank you all

@jtagcat How would you/youam copy snapshots from one repo to another repo currently?

@rawtaz just an idea, I'd join irc, if I were you.

@jtagcat Thanks for the suggestion, but I'm already on the IRC, just not in #restic.
Would you mind answering the question I asked? :) AFAIK there's no copy command in restic, and just copying files from one repo to another is not possible either AFAIK.

youam is there some way to set up a repository so that host A and host B can both write, but can't see each other's data? i'd be fine with getting deduplication benefits on prune only
xavierg This is, afaik, not possible with Restic's current design.
youam related, or even better: is there a way to set up a backend which i can only write to, but not read from?
Rasmus Kallas youam: see https://github.com/restic/restic/issues/2414
fd0 youam: no, it's not possible. restic's repo design requires being able to read data
youam: if you want isolation between hosts, you need to use separate repos
youam: okay, so i'll create one repo per host, backup there, and copy the resulting snapshots over to a global repository with longer history
thank you all
fd0 uhm.
youam: please be aware that there's no built-in method for copying the snapshots over yet
youam: how are you planning to do that? :)
dozn_ joined the room._
youam fd0: i seem to remember i've used https://github.com/restic/restic/issues/323#issuecomment-380714687 successfully in the past
I haven't checked on how much of it is true or how easily it works, but it's an idea.

Yeah, so what you wrote earlier (the idea by @youam) is currently not doable with restic. That's why I asked how it was expected to be done, because it was written as if it was doable. Thanks for the clarification.

I will be (already did some work) putting together an interactive / argument-based installer, there's nothing unresolved here (what hasn't forked to an another issue).


Support columns with server_default

At the moment, the following error is produced:
sqlalchemy.exc.ProgrammingError: (psycopg2.errors.DatatypeMismatch) default for column "abc" cannot be cast automatically to type _tmp_cdef_1720

My first idea was to extend alembic_enums.Column to support something like new_server_default (which could be NOT_SET, None or a value).
Have you thought about supporting something like that?

Hey @kreathon, thanks for the report.
Can you please take a look at the PR #3 and let me know if it works for you. As outlined in the documentation in the updated README, you can use the following syntax.
column = Column(
    "resources",
    "state",
    server_default=Change(old="enabled", new="active"),
)

Thank you for addressing this so quickly, I will leave a comment on the PR :slightly_smiling_face: .


Setting up dependabot

Discussion 🗣
Dependabot alleviates that pain by updating your dependencies automatically, so you can spend less time updating dependencies and more time building. Up until now, the Dependabot features we’ve brought to GitHub have focused on automated security updates, which update packages that have known vulnerabilities.
Context
When you check in an insecure dependency, or a new vulnerability is discovered in a dependency you already have, we already let you know with security alerts for vulnerable dependencies. This system also powers our automated security updates, which we announced last November. GitHub users have merged more than 776,000 automated security update pull requests since the announcement.

Which repositories do not have Dependabot that could be benefited from it?

The web-components repository can be benefitted from it. Plzzz have a look upon this PR. Same feature can be implemented to the web components repository https://github.com/accordproject/hlf-cicero-contract/pull/11

@irmerk Any opinions or suggestion you want to add here?

I've now enabled dependabot for all Accord Project repositories. Thanks for the prompt @Cronus1007!

@mttrbrts Have a look upon these issues as well. #25 #26 #24


'Malformed data' on bt_data_parse() for every ble adv packet on bbc_microbit

Describe the bug
I am trying to inspect ble advertising packets on a bbc_microbit. I ran into the problem that for every adv packet bt_data_parse() logs 'Malformed data' and returns. (bt_data_parse() relevant part)
Relevant code:
static bool data_cb(struct bt_data *data, void *user_data)
{
	int err;

	switch (data->type) {

		case BT_DATA_MANUFACTURER_DATA:
			printk("Found!");

			err = bt_le_scan_stop();
			if (err) {
				printk("Stop LE scan failed (err %d)\n", err);
				return false;
			}

			printk("Stopping.");

			return false;
		default:
			return true;
	}
}

static void device_found(const bt_addr_le_t *addr, int8_t rssi, uint8_t type,
			 struct net_buf_simple *ad)
{
	char dev[BT_ADDR_LE_STR_LEN];

	bt_addr_le_to_str(addr, dev, sizeof(dev));

	bt_data_parse(ad, data_cb, (void *)addr);
}

static void start_scan(void)
{
	int err;

	struct bt_le_scan_param scan_param = {
		.type       = BT_LE_SCAN_TYPE_PASSIVE,
		.options    = BT_LE_SCAN_OPT_NONE,
		.interval   = BT_GAP_SCAN_FAST_INTERVAL,
		.window     = BT_GAP_SCAN_FAST_WINDOW,
	};

	err = bt_le_scan_start(&scan_param, device_found);
	if (err) {
		return;
	}

	printk("Scanning successfully started\n");
}

void main(void)
{
	int err;
	err = bt_enable(NULL);

	if (err) {
		return;
	}

	start_scan();
}

prj.conf
CONFIG_BT=y
CONFIG_BT_CENTRAL=y
CONFIG_LOG=y
CONFIG_LOG_MODE_MINIMAL=y
CONFIG_LOG_MAX_LEVEL=4

Environment

Zephyr v3.0.0
Zephyr SDK v0.13.2
Ubuntu 21.10

@atoncetti can you please print the raw adveritising data tha triggers this warning and copy it here so we can analyze it manually?

Hi @carlescufi, I was just doing that yesterday but couldn't point out where is the issue.
[DEVICE]: 35:4C:73:2A:95:07 (random), AD evt type 0, AD data len 19, RSSI -82
Raw adv data: 4C20404340E180E19F9
len: 76, ad->len: 18
Malformed data
[DEVICE]: --:--:--:--:--:03 (public), AD evt type 3, AD data len 28, RSSI -80
Raw adv data: 71E3BFE408820
len: 113, ad->len: 27
Malformed data
[DEVICE]: --:--:--:--:--:03 (public), AD evt type 3, AD data len 28, RSSI -82
Raw adv data: 71E3BFE408820
len: 113, ad->len: 27
Malformed data
[DEVICE]: 35:4C:73:2A:95:07 (random), AD evt type 0, AD data len 19, RSSI -83
Raw adv data: 4C20404340E180E19F9
len: 76, ad->len: 18
Malformed data

len and ad->len refer to the same variables in bt_data_parse()

@atoncetti Could you let us know the peer device that is advertising?
The raw data does not follow the AD Data format of lenght-type-value.

Hi @cvinayak
Here is an example of a scan from the bbc microbit (left) and from my ubuntu machine running a scan with bluetoothctl (right):

Strangely, the bbc microbit picks up a different device address even though there is a single device currently advertising.
Moreover, if I start advertising on my laptop, the bbc microbit picks up the same address as before but the adv data differs for the new device that is advertising:

I hope this helps, please let me know if you need something else.

@atoncetti what do you get when using the samples/bluetooth/central or the samples/bluetooth/central_hr sample? Please use the peripheral_hr sample as the peer and let me know if the advertising data is parsed correctly by the central.
If you share your branch, steps to build your application and the peer device used for advertising, I can try to reproduce at my end.

Hi @cvinayak,
unfortunately at the moment I do not have another device at hand on which I can flash the peripheral_hr (is there a way to simulate one?)
Anyways I've tried to run the samples/bluetooth/central on the bbc_microbit and I got similar results as above, namely, the bbc_microbit picks up a device address that is incorrect:
First test:
bbc_microbit "Device found: CE:BD:AF:BD:FD:03 (random) (RSSI -75)" |  correct device address 55:5C:75:13:56:BF

Second test:
bbc_microbit "Device found: 6E:B4:EA:F1:BF:02 (random) (RSSI -73)" | correct device address 6B:75:A7:57:8D:FA

Third test:
bbc_microbit "Device found: AB:BA:39:7A:22:00 (random) (RSSI -76)" | correct device address 7D:5D:D1:CB:D1:10

For all three tests I've used a device created with the nRF Connect android app with the following configs:
Random address, connectable, Device type: LE only, Advertising type: Legacy

@atoncetti I will will try out with nRF Connect android app and check if I get similar malformed data using samples/bluetooth/central on bbc_microbit board is observed.

Hi @cvinayak,
Thank you for your support!
As a follow up, I have created a temporary repository with the exact same zephyr project I ran at the beginning of this issue.
Here is the code: https://github.com/atoncetti/zephyr_bbc_microbit_ble_fault_example

I am able to reproduce the malformed data, it appears the device address and ad data are corrupt, the length seems correct. I will be investigating as part of the v3.1 release stabilization.

Sounds great! Please let me know if there is anything I can help you with.

Regression introduced in commit https://github.com/zephyrproject-rtos/zephyr/commit/1dcbe73cc8f31b6a207d23a3c825a7ed4a6b15f0 ("Bluetooth:
controller: radio: add setting of S1 byte in radio packet").
Fix posted in #45749.
Thank you for reporting the bug.


Add delete functions to Model/Cluster

Subtask of: #475
Depends on: #508
Right now there is no convenient way of deleting old data.

Fixed with #522


build: release candidate

The previous release failed with this error:
[11:01:58 AM] [semantic-release] › ✘  An error occurred while running semantic-release: Error [ERR_REQUIRE_ESM]: require() of ES Module /home/runner/work/oak-ai-lesson-assistant/oak-ai-lesson-assistant/release.config.js from /home/runner/work/oak-ai-lesson-assistant/oak-ai-lesson-assistant/node_modules/cosmiconfig/dist/loaders.js not supported.
release.config.js is treated as an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which declares all .js files in that package scope as ES modules.
Instead either rename release.config.js to end in .cjs, change the requiring code to use dynamic import() which is available in all CommonJS modules, or change "type": "module" to "type": "commonjs" in /home/runner/work/oak-ai-lesson-assistant/oak-ai-lesson-assistant/package.json to treat all .js files as CommonJS (using .mjs for all ES modules instead).

This is a consequence of us missing a js file which needs to be renamed to .cjs

:tada: This PR is included in version 1.18.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


NPM Script Explorer icon

The NPM Script is currently using the following icon:

Inside the NPM Scripts Explorer scripts are shown with the '{ }' icon.
Wouldn't it be more consistent to also use the '{ }' icon for the explorer?

'{ }' is already a codicon, so I am happy to change to that. @sbatten I think you added the code icon here. Anything that would prevent us from switching to '{ }'?

Isn't this just a .json file icon that look different depending on the installed icon theme?

@usernamehw good point. I'll wait for @misolori's thoughts too.

The { } on the view is indeed a file icon that changes on the theme. If we wanted to keep the icons consistent we could opt to change the { } on the view to the code icon that's used in the activity bar.

If we change the { } in the view, then we'd be breaking a different consistency. Since the view uses the resource for package.json it means that everywhere that the file package.json appears it get the same icon, which is great. If we change the icon just in the scripts view, then that view will be the odd one out.
I think if we change away from the code icon then we should be changing it because have an icon that fits better. Perhaps an icon similar to the icon we use for the build in extension?

Of the existing codicons, I think code and json fit best. @misolori do you have a preference between code, json, and a new npm specific icon?

I don't have a preference, either works. We also have 'tools' that's used in tasks that is similar as well. But any of those work for me.

Alrighty then.  Let's go with json, since it will nicely match the default file icon theme.

Im facing this issue. And I have package.json in all my projects root. Some apears others dont appears.


update to lexibank 2

Requires the PR in https://github.com/lexibank/pylexibank/pull/204

I've also checked and would approve it. Will merge it and start work on the profile.


[v4, new feature] create "up" set of classes in grid system

I like the new "up" feature for responsive classes in v4. It's great!
Would it be possible to have a parallel "up" set of classes for the grid system? This would allow me to specify one setting for x-small devices and just one class for small, medium, large inclusively.
So instead of this

I could do this?

Bryan

v3's and v4's grid .col-* classes already work like "up" classes; they aren't "only" classes.
See also https://github.com/twbs/bootlint/wiki/E029


Java stack trace missing some SourceSection data with eval

When using eval and it throws an error, the java-sided error message is missing some SourceSection data,
for example, calling a file with the following code:
public static void main(String []args) throws IOException {
        try {
            Context context = Context.newBuilder("js").build();
            context.eval(Source.newBuilder("js", new File("./test.js")).build());
        } catch (Exception e) {
            e.printStackTrace();
        }
    }

and in test.js
try {
    eval("causeerror")
} catch(e) {
    console.error("js stack:")
    console.error(e.stack)
    console.error("java stack:")
    throw e
}

creates this result
js stack:
ReferenceError: causeerror is not defined
    at <eval>:1:1
    at test.js:1:1
java stack:
ReferenceError: causeerror is not defined
	at <js> :program(<eval>:1:0-9)
	at <js> :program(Unknown)
	at org.graalvm.polyglot.Context.eval(Context.java:347)
	at xyz.wagyourtail.test.main(test.java:14)

I would expect the java stack trace to be able to have test.js instead of Unknown on the second at <js> :program statement in the java stack trace.
Also it appears that the line number is incorrect for the at test.js:1:1 in the js stack trace.
these are not issues with Polyglot.eval()

@wagyourtail Thanks for reporting the issue
we will check it out and let you know

@wagyourtail Thanks for reporting the issue
we will check it out and let you know

@wagyourtail Bug has been raised for this issue
We will get back to you with a fix or workaround

Thanks for the report! Fixed as of https://github.com/oracle/graaljs/commit/e8420f3ba5b0a9a7aa2cdd63ca672ee84758e2da.


Config path issue

Codecov Report

Merging #24 into master will increase coverage by 0.23%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #24      +/-   ##
=========================================
+ Coverage   97.16%   97.4%   +0.23%
=========================================
  Files           5       5
  Lines         141     154      +13
=========================================
+ Hits          137     150      +13
  Misses          2       2
  Partials        2       2

Impacted Files
Coverage Δ

internal/config.go
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


HTTP/2 may send DATA after RST_STREAM

The function prepare_data/6 may call maybe_terminate_stream/3 and send an RST_STREAM before returning and sending the data buffered. The stream should be terminated AFTER we flush the buffer.

The Gun tests cancel_remote and cancel_remote_connect can be used to reproduce the issue. The tests pass but errors may be logged about this issue. Tracing can otherwise be used to confirm the wrong behavior.

I ran into this again so I will fix it now.

Done.

Hi Loïc,
is this issue released in any version? I am facing the issue, where uploading big files ends crash:
2023-02-21T08:00:23.216675Z	trace	envoy router	[C147][S11823289556390547815] proxying 16384 bytes
2023-02-21T08:00:23.216678Z	trace	envoy http	[C147][S11823289556390547815] decode data called: filter=envoy.filters.http.upstream_codec status=0
2023-02-21T08:00:23.216681Z	trace	envoy http	[C147][S11823289556390547815] decode data called: filter=envoy.filters.http.router status=3
2023-02-21T08:00:23.216683Z	trace	envoy http2	[C147] about to recv frame type=3, flags=0, stream_id=5
2023-02-21T08:00:23.216686Z	trace	envoy http2	[C147] track inbound frame type=3 flags=0 length=4 padding_length=0
2023-02-21T08:00:23.216690Z	trace	envoy http2	[C147] recv frame type=3
2023-02-21T08:00:23.216693Z	trace	envoy http2	[C147] remote reset: 0
2023-02-21T08:00:23.216696Z	debug	envoy http2	[C147] stream 5 closed: 0
2023-02-21T08:00:23.216699Z	debug	envoy http	[C147][S11823289556390547815] stream reset
...

A piece of stack is:
[
  13,
  :no_error,
  ...
  [file: 'lib/upload_service/stream_handler.ex', line: 128]},
  {:cowboy_stream, :terminate, 3,
   [file: '/opt/deps/cowboy/src/cowboy_stream.erl', line: 138]},
  {:cowboy_http2, :terminate_stream_handler, 4,
   [file: '/opt/deps/cowboy/src/cowboy_http2.erl', line: 1200]},
  {:cowboy_http2, :rst_stream_frame, 3,
   [file: '/opt/deps/cowboy/src/cowboy_http2.erl', line: 569]},
  {:cowboy_http2, :parse, 2,
   [file: '/opt/deps/cowboy/src/cowboy_http2.erl', line: 325]},
  {:cowboy_http, :parse, 2,
   [file: '/opt/deps/cowboy/src/cowboy_http.erl', line: 337]},
  {:proc_lib, :init_p_do_apply, 3, [file: 'proc_lib.erl', line: 226]}
  ...
]

I have been debugging it for days, I thought that maybe it is related to this issue?

I have pulled the latest master version but it still crashes.

Your crash seems to happen in stream_handler.ex, not in Cowboy.

The fix is not in a released version in any case but it's in master.

Yes, the stack trace pointed that I did not predict specific terminate function. I handled it but the problem lies itself lies somewhere in http2 stream that is being reset because of no_error reason. This is what I get on the client side:
2023/02/21 10:07:06 http2: Framer 0xc0001920e0: read HEADERS flags=END_HEADERS stream=3 len=52
2023/02/21 10:07:06 http2: decoded hpack field header field ":status" = "502"
2023/02/21 10:07:06 http2: decoded hpack field header field "content-length" = "87"
2023/02/21 10:07:06 http2: decoded hpack field header field "content-type" = "text/plain"
2023/02/21 10:07:06 http2: decoded hpack field header field "date" = "Tue, 21 Feb 2023 09:07:05 GMT"
2023/02/21 10:07:06 http2: decoded hpack field header field "server" = "istio-envoy"
2023/02/21 10:07:06 http2: Transport received HEADERS flags=END_HEADERS stream=3 len=52
2023/02/21 10:07:06 http2: Framer 0xc0001920e0: read DATA flags=END_STREAM stream=3 len=87 data="upstream connect error or disconnect/reset before headers. reset reason: protocol error"
2023/02/21 10:07:06 http2: Transport received DATA flags=END_STREAM stream=3 len=87 data="upstream connect error or disconnect/reset before headers. reset reason: protocol error"
2023/02/21 10:07:06 http2: Framer 0xc0001920e0: read RST_STREAM stream=3 len=4 ErrCode=NO_ERROR
2023/02/21 10:07:06 http2: Transport received RST_STREAM stream=3 len=4 ErrCode=NO_ERROR

You receive HEADERS, then DATA with FIN flag set, then RST_STREAM with reason NO_ERROR. There's no problem there. Cowboy will reset the stream if the request body was large (or of unknown size) and won't be fully read to avoid taking up too much bandwidth for no good reason.
Anyway this ticket is about DATA being received after RST_STREAM so this is not the same.

Thank you for you information. Is any option to turn on debug mode in cowboy to get flow similarly to e.g. https://go.dev/src/net/http/doc.go http2debug?

Another option might be to set the expected header to inform cowboy about length. What would be the best key? I tried with Content-Length, but no success.


Server Neoforge 1.21.1 Game Crash

Neoforge Server 1.21.1 Neoforge Version 21.1.42
When trying to select chunks or make active chunks through Journeymap, the game crashes and exits.
Server Console log:
[00:34:51] [Netty Epoll Server IO #3/ERROR] [minecraft/Connection]: Exception caught in connection
io.netty.channel.unix.Errors$NativeIoException: recvAddress(..) failed: Connection reset by peer
[00:34:51] [Server thread/INFO] [minecraft/ServerGamePacketListenerImpl]: Fly0ver lost connection: Disconnected
[00:34:51] [Server thread/INFO] [minecraft/MinecraftServer]: Fly0ver left the game
crash-2024-09-20_08.34.49-client.txt

Same issue here, only crashed on the client, when claiming using the Integration in Journeymap screen it closes the game.

Seems like there are some breaking changes in recent FTBChunks update, I'll take a look in next few days

Should be fixed in v1.6


1.3.0 Milestone: nsca crypto

Allow configuration of pynsca crypto mode.
Include python-mcrypt and pycrypto in requirements.txt.
Don't assume mode 16 just because a password is defined.

Thanks @jsatterfield. If I have it right we should assume encryption mode '1' unless '16' (mcrypt) is used, does that sound right?

It appears there is some issues with the PyPi python-mcrypt package, adding it to our requirements causes Pip to error on the build:
$ sudo pip -v install python-mcrypt==1.1
Collecting python-mcrypt==1.1
  1 location(s) to search for versions of python-mcrypt:
  * https://pypi.python.org/simple/python-mcrypt/
  Getting page https://pypi.python.org/simple/python-mcrypt/
  Looking up "https://pypi.python.org/simple/python-mcrypt/" in the cache
  Current age based on date: 0
  Freshness lifetime from max-age: 600
  Freshness lifetime from request max-age: 600
  The response is "fresh", returning cached response
  600 > 0
  Analyzing links from page https://pypi.python.org/simple/python-mcrypt/
  Could not find a version that satisfies the requirement python-mcrypt==1.1 (from versions: )
Cleaning up...
No matching distribution found for python-mcrypt==1.1
Exception information:
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/pip/basecommand.py", line 215, in main
    status = self.run(options, args)
  File "/usr/lib/python2.7/site-packages/pip/commands/install.py", line 324, in run
    requirement_set.prepare_files(finder)
  File "/usr/lib/python2.7/site-packages/pip/req/req_set.py", line 380, in prepare_files
    ignore_dependencies=self.ignore_dependencies))
  File "/usr/lib/python2.7/site-packages/pip/req/req_set.py", line 554, in _prepare_file
    require_hashes
  File "/usr/lib/python2.7/site-packages/pip/req/req_install.py", line 278, in populate_link
    self.link = finder.find_requirement(self, upgrade)
  File "/usr/lib/python2.7/site-packages/pip/index.py", line 514, in find_requirement
    'No matching distribution found for %s' % req
DistributionNotFound: No matching distribution found for python-mcrypt==1.1

We will need to ask the python-mcrypt or pynsca developers how to resolve this. It seems the PyPi package is not setup correctly.


Basic setup for client is done

Adds skeleton structure for the client app

Hello @onlinejudge95! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file services/server/migrations/env.py:

Line 24:1: E402 module level import not at top of file

In the file services/server/migrations/versions/001_initial_migration.py:

Line 4:9: W291 trailing whitespace
Line 22:5: E128 continuation line under-indented for visual indent
Line 23:5: E122 continuation line missing indentation or outdented
Line 24:5: E122 continuation line missing indentation or outdented
Line 25:5: E122 continuation line missing indentation or outdented
Line 26:5: E122 continuation line missing indentation or outdented


Add more granularity to nofication settings

As a user who is high risk I want to be able to add a vibrate alert or sound not only for
Danger or too close,
But on "Warning" status.

default for warning should be off, but user can set this.


[opentitantool] HyperDebug firmware with bugfixes

Update the version of the pre-built firmware for HyperDebug shipped with opentitantool.  This should resolve problems with HyperDebug dropping characters if the USB host computer fails to poll for a few milliseconds, while OT is producing data at maximum speed.
Also, bugs in corner cases of GPIO monitoring have been addressed.

It would be nice to cherry pick this to the earlgrey_es_sival branch. @jesultra

As far as I can tell, the earlgrey_es_sival branch also uses the 20240112_01 firmware for HyperDebug (same as was introduced by this PR.)
https://github.com/lowRISC/opentitan/blob/earlgrey_es_sival/third_party/hyperdebug/repos.bzl

Ah sorry my apologies. Thanks for checking.


Create prateek

intro

Your report is in plain text. Learn how to write github markdown. See other reports for reference


Progress not properly displaying for explorer synced health check

The data is coming back correctly but for some reason it's showing as 0.00%.
{"bitcoinStatus":{"blocks":718729,"headers":718729,"verificationProgress":0.9999970594948908,"isSynched":true,"incrementalRelayFee":1.0,"minRelayTxFee":1.0,"externalAddresses":["<redacted>.onion:8333"],"capabilities":{"canScanTxoutSet":true,"canSupportSegwit":true,"canSupportTransactionCheck":true}},"repositoryPingTime":0.0,"isFullySynched":false,"chainHeight":718729,"syncHeight":715185,"instanceName":"","networkType":"Mainnet","cryptoCode":"BTC","supportedCryptoCodes":["BTC"],"version":"<IP_ADDRESS>"}

Fixed in: https://github.com/Start9Labs/btcpayserver-wrapper/pull/9/commits/d6c0775848e9fcf151d85fe4c0c9fd0cc7f65c33


testing invalid config

Codecov Report

Merging #32 (e2b9e05) into main (4bfb2fd) will increase coverage by 4.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
+ Coverage   67.89%   71.95%   +4.05%
==========================================
  Files           4        4
  Lines         271      271
==========================================
+ Hits          184      195      +11
+ Misses         47       42       -5
+ Partials       40       34       -6

Impacted Files
Coverage Δ

config/config.go
83.78% <0.00%> (+29.72%)
:arrow_up:

Continue to review full report at Codecov.

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


net: cannot send multicast using ListenPacket("udp4", "") on Windows 10 (1909)

What version of Go are you using (go version)?

go version go1.14.1 windows/amd64

Does this issue reproduce with the latest release?
Yes, with 1.14 and 1.14.1 ( never tested earlier )
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
set GO111MODULE=
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\ADFilimonov\AppData\Local\go-build
set GOENV=C:\Users\ADFilimonov\AppData\Roaming\go\env
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\ADFilimonov\go
set GOPRIVATE=
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=c:\go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=c:\go\pkg\tool\windows_amd64
set GCCGO=gccgo
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=C:\Users\ADFILI~1\AppData\Local\Temp\go-build411155850=/tmp/go-build -gno-record-gcc-switches

What did you do?
Single file in directory: mctest_test.go :
package mctest

import (
	"net"
	"testing"
	"time"
)

var (
	bulbMulticastAddr *net.UDPAddr   = &net.UDPAddr{IP: net.IPv4(239, 255, 255, 250), Port: 1982}
	bulbSearchRequest []byte         = []byte("M-SEARCH * HTTP/1.1\r\nHOST: <IP_ADDRESS>:1982\r\nMAN:\"ssdp:discover\"\r\nST:wifi_bulb\r\n")
	bulbSearchConn    net.PacketConn = nil
)

func TestMC(t *testing.T) {
	testMulticast(5)
}

func testMulticast(timeout int) error {
	bulbSearchConn, err := net.ListenPacket("udp4", "")
	if err != nil {
		return err
	}

	readDeadlineTime := time.Now().Add(time.Second * time.Duration(timeout))
	err = bulbSearchConn.SetReadDeadline(readDeadlineTime)
	if err != nil {
		return err
	}

	_, err = bulbSearchConn.WriteTo(bulbSearchRequest, bulbMulticastAddr)
	if err != nil {
		return err
	}

	time.Sleep(time.Second * time.Duration(10))
	return nil
}

I run Wireshark on two interfaces ( "Ethernet" and "Adapter for loopback traffic capture" ) with filter udp port 1982.
I run code

CD /D S:\PRJ\doesntmatter\pkg\mctest
go test

I monitor packets wireshark shows.

What did you expect to see?
Expected 2 packets with source = "ip-of-my-ethernet", target "mulicast address", one using Ethernet interface and one using loopback interface
What did you see instead?
Sometimes everything is right ( I see everything as expected ). And sometimes I see
1 packet with source = "<IP_ADDRESS>", target "mulicast address", using loopback interface.
I can not determine, what happens with my envinronment to "switch" this behaviour. After system reboot I can see it working and some time after it stops, ot vice-versa.
SCREENSHOTS
Wireshark setup

Wireshark shows OK

Wireshark shows NOT OK
Sorry I have no screenshot now. It happens periodically each few days. I try add later
Update: Added Screenshot

@mikioh @bradfitz @ianlancetaylor

@alexbrainman

Added screenshot. On today morning, same code fails.

@filimonic sorry I am not network person, so I do not know why your program does not work reliably. I also don't have Wireshark program installed on my computer, so I cannot even try to reproduce your problem.
But I noticed there are some tests in net package (in %GOROOT%\src\net directory), that call ListenPacket
$ git grep -e ListenPacket.*udp4 -n
mockserver_test.go:337:                 return ListenPacket("udp4", "<IP_ADDRESS>:0
mockserver_test.go:344:                 return ListenPacket("udp4", "<IP_ADDRESS>:0
$

All tests in net are obviously succeed on everyone's Windows computers. Maybe you can use net test code and adjust it to make it do what you need?
Alex

@alexbrainman The listed tests are skipped for Windows because the function used (fd *netFD) dup() is not implemented. It seems that there is no relevant test of UDP on Windows.
@filimonic There are several posts about conflicts between RDP and UDP on 1909. It might be the cause of what is identified.

Here are the relevant tests for UDP on Windows. All successful on 1909.
Microsoft Windows [Version 10.0.18363.836]
$GOPATH\golang\go\src>go test -run=".*UDP.*" -v net
=== RUN   TestUDPListener
--- PASS: TestUDPListener (0.01s)
=== RUN   TestDualStackUDPListener
--- PASS: TestDualStackUDPListener (0.01s)
=== RUN   TestUDPConnSpecificMethods
    TestUDPConnSpecificMethods: protoconn_test.go:151: file udp4 <IP_ADDRESS>:50157: not supported by windows
--- PASS: TestUDPConnSpecificMethods (0.01s)
=== RUN   TestUDPServer
--- PASS: TestUDPServer (0.04s)
=== RUN   TestResolveUDPAddr
--- PASS: TestResolveUDPAddr (0.02s)
=== RUN   TestWriteToUDP
--- PASS: TestWriteToUDP (0.00s)
=== RUN   TestUDPConnLocalName
--- PASS: TestUDPConnLocalName (0.00s)
=== RUN   TestUDPConnLocalAndRemoteNames
--- PASS: TestUDPConnLocalAndRemoteNames (0.00s)
=== RUN   TestIPv6LinkLocalUnicastUDP
--- PASS: TestIPv6LinkLocalUnicastUDP (0.00s)
=== RUN   TestUDPZeroBytePayload
--- PASS: TestUDPZeroBytePayload (0.00s)
=== RUN   TestUDPZeroByteBuffer
--- PASS: TestUDPZeroByteBuffer (0.00s)
=== RUN   TestUDPReadSizeError
--- PASS: TestUDPReadSizeError (0.00s)
PASS
Socket statistical information:
(inet6, datagram, default): opened=93 connected=1 listened=0 accepted=0 closed=93 openfailed=0 connectfailed=0 listenfailed=0 acceptfailed=0 closefailed=0
(inet4, datagram, default): opened=76 connected=5 listened=0 accepted=0 closed=76 openfailed=0 connectfailed=0 listenfailed=0 acceptfailed=0 closefailed=0

@alexbrainman The listed tests are skipped for Windows because the function used (fd *netFD) dup() is not implemented. It seems that there is no relevant test of UDP on Windows.

Fair enough. Thank you for explaining.
Alex

I had the same problem, has the problem been solved?


Overflow when instantiating FixedPoint from integer, but not from hex string.

Description
#Given that 0x7b52 is 31570 why is that?

# This is fine
FixedPoint('0x7b52', signed=1, m=14, n=16, overflow='clamp', rounding='convergent', overflow_alert='error', mismatch_alert='warning', implicit_cast_alert='warning', str_base=16)

# This overflows
FixedPoint(31570, signed=1, m=14, n=16, overflow='clamp', rounding='convergent', overflow_alert='error', mismatch_alert='warning', implicit_cast_alert='warning', str_base=16)

Steps to reproduce
# 0.3 in Q7.8 - both with standard python and fixedpoint lib
a = round ( 0.3*(2**8) ) #77
af = FixedPoint(0.3,signed=1,m=7,n=8)

# 1.6 in Q7.8 - both with standard python and fixedpoint lib
b = round ( 1.6*(2**8) ) #410
bf = FixedPoint(1.6,signed=1,m=7,n=8)

a*b

31570

af*b

FixedPoint('0x7b52', signed=1, m=14, n=16, overflow='clamp', rounding='convergent', overflow_alert='error', mismatch_alert='warning', implicit_cast_alert='warning', str_base=16)

0x7b52 is 31570
log2(31570) is 14.9 , hence 15 bits are necessary to represent it, not 14!
The correct behaviour is to have an overflow when 31570 is converted into Q7.8, and it happens correctly, but when the string "0x7b52" no warnings are shown.
Environment

Operating System: Windows
Python version: python 3.8
fixedpoint version (find in the output of pip list): 1.0.1

When you use a string to initialize a FixedPoint object, it distributes the bits on both sides of the binary point, right justified. So
FixedPoint('0x7b52', 1, 14, 16, ...) becomes 0b00000000000000.0111101101010010, as shown below (see Integer and Fractional Bits in the string formatting page for formatting hints):
>>> ab = FixedPoint('0x7b52', signed=1, m=14, n=16, overflow='clamp', rounding='convergent', overflow_alert='error', mismatch_alert='warning', implicit_cast_alert='warning', str_base=16)
>>> f"{ab:#0{ab.m+2}bm}.{ab:0{ab.n}bn}"
'0b00000000000000.0111101101010010'

However, when you assign an integer, it interprets the integer as only integer bits. So you are correct, that the int 31570 requires ceil(log2(31570)) = 15 bits to represent, however, you're only giving it 14 integer bits (with m=14), one of which is negatively weighted, which is why it overflows. If you initialize 31570 without constraining m or n, you can see what's going on:
>>> x = FixedPoint(31570, signed=1)
>>> f"{x:#0{x.m+2}bm}.{x:0{x.n}bn}"
'0b0111101101010010.0'

In both initializations above, the bit pattern (0b0111101101010010 or 0x7b52) is the same, they just end up on different side of the binary point.
So 0.3 * 1.6 (both in fixed point) look like this:
>>> af = FixedPoint(0.3,signed=1,m=7,n=8)
>>> bf = FixedPoint(1.6,signed=1,m=7,n=8)
>>> abf = af * bf
>>> abf
FixedPoint('0xc58154', signed=1, m=21, n=24, overflow='clamp', rounding='convergent', overflow_alert='error', mismatch_alert='warning', implicit_cast_alert='warning', str_base=16)
>>> f"{abf:#0{abf.m+2}bm}.{abf:0{abf.n}bn}"
'0b000000000000000000000.110001011000000101010100'

Also, please note that af * bf is not equal to FixedPoint('0x7b52', signed=1, m=14, n=16, ...):
>>> x = FixedPoint('0x7b52', signed=1, m=14, n=16, overflow='clamp', rounding='convergent', overflow_alert='error', mismatch_alert='warning', implicit_cast_alert='warning', str_base=16)

If desired, `FixedPoint.trim` will remove superfluous/insignificant bits (i.e., trailing or leading zeros):

```python
>>> from fixedpoint import trim
>>> abf_trimmed = trim(abf)  # or abf.trim() if you want to modify abf iteself
>>> abf_trimmed
FixedPoint('0x3da9', signed=1, m=1, n=15, overflow='clamp', rounding='convergent', overflow_alert='error', mismatch_alert='warning', implicit_cast_alert='warning', str_base=16)
>>> f"{abf_trimmed:#0{abf_trimmed.m+2}bm}.{abf_trimmed:0{abf_trimmed.n}bn}"
'0b0.011110110101001'

These int/str initialization behaviors are documented here:

Initializing from an int
Initializing from a str


paths not used for auto import

From https://github.com/Microsoft/vscode/issues/72931

TypeScript Version:  3.5.0-dev.20190430

Search Terms:

paths
auto import

Repo

clone https://github.com/ac566/typescript-auto-import-issue and open in vscode
In VS Code, set "typescript.preferences.importModuleSpecifier": "non-relative"
Open src/app/app1/test.ts
Delete the import for TestType
Try using a quick fix or auto import to add the impact back

Expected behavior:
The import is added as import { TestType } from "@myLib/example_2";
The is expected since the tsconfig configures paths and the import of @myLib/example_2 was previously working properly:
{
    "compilerOptions": {
        "target": "es5",
        "module": "es2015",
        "moduleResolution": "node",
        "baseUrl": ".",
        "paths": {
            "@myLib/*": [
                "./src/lib/src/*/_index.ts"
            ]
        }
    }
}

Actual behavior:
The import is added as import { TestType } from "@myLib/example_2/_index";

I have the issue described below when using auto import in a project configured with compilerOptions.paths. Let me know if is a different problem that warranties a separated issue.
Using vscode 1.34 on windows with provided tsserver
I'm using webpack resolve option to add a path to look for "root" modules:
resolve: {
  modules: [path.resolve(__dirname, './src/common'), 'node_modules'],
}

To match same behavior in vscode, i configure  jsconfig.json compilerOptions.paths as suggested in TypeScript-Handbook:
{
  "compilerOptions": {
    "baseUrl": ".",
    "paths": {
      "*": [
        "*",
        "./src/common/*"
      ]
    }
  }
}

See sample project: https://github.com/blikblum/vscode-autoimport-path
When i try to import from src/modules/main.js i get always relative to baseUrl which is clearly wrong:
import { myService } from "src/common/myservice";
import { otherService } from "src/common/other/otherservice";
import { Home } from "src/home/home";

If i remove "*" from paths (see this branch) things are a little better:
import { myService } from "myservice";
import { otherService } from "other/otherservice";
import { Home } from "src/home/home";

BTW this is the expected import:
import { myService } from "myservice";
import { otherService } from "other/otherservice";
import { Home } from "../home/home";

In this branch, i try to use an empty jsconfig.json and get a technically correct import although not useful to achieve the intended behavior.

I got a same issue

I faced the same issue in my sample project.
I guess this is the bug of compiler/moduleSpecifiers.ts#L260. The second argument of String#substr should be the length of matchedStar, or String#substring should be used.

I got the same issue. Based on this stackoverflow post it is the same for both tsconfig.json and jsconfig.json.
Sadly this makes VS Code unusable for our project, baseUrl workaround is not possible since we have more paths and imports inserted by VS Code are not working in the running app.

In my case just by using in my settings
"typescript.preferences.importModuleSpecifier": "non-relative",

and then setting in my tsconfig compilerOptions.paths my root as the last options I was able to import files using path correctly
{
  "compilerOptions": {
    "baseUrl": "./",
    "paths": {
      "@model/*": ["src/model/*"],
      "@route/*": ["src/route/*"],
      "@constant/*": ["src/constant/*"],
      "@/*": ["src/*"]
    }
  },
}

This creates
import { JWT_CONFIG } from "@constant/jwt";

instead of
import { JWT_CONFIG } from "./constant/jwt";

or
import { JWT_CONFIG } from "@/constant/jwt";

which is what i was looking for.

In my case just by using in my settings
"typescript.preferences.importModuleSpecifier": "non-relative",

and then setting in my tsconfig compilerOptions.paths my root as the last options I was able to import files using path correctly
{
  "compilerOptions": {
    "baseUrl": "./",
    "paths": {
      "@model/*": ["src/model/*"],
      "@route/*": ["src/route/*"],
      "@constant/*": ["src/constant/*"],
      "@/*": ["src/*"]
    }
  },
}

This creates
import { JWT_CONFIG } from "@constant/jwt";

instead of
import { JWT_CONFIG } from "./constant/jwt";

or
import { JWT_CONFIG } from "@/constant/jwt";

which is what i was looking for.

@nautilor - thank you for the tip. I have adjusted my workspace settings and on first glance it seems to be working! If this is solid, then it will be a huge timesaver.
Thank you!

@nautilor - thank you for the tip. I have adjusted my workspace settings and on first glance it seems to be working! If this is solid, then it will be a huge timesaver.
Thank you!

@paustint
Glad to help 😁

@paustint
Glad to help 😁

I found there are some downsides for this approach in some cases.
I am here -> apps/api/src/app/controllers/sf-misc.controller.ts**
I want to import from here -> apps/api/src/app/services/sf-misc.ts
The import resolves as this:
import { buildPackageXml } from 'apps/api/src/app/services/sf-misc';
When it should be:
import { buildPackageXml } from '../services/sf-misc';
But the auto-imports from a relative @appName/.. are working great :) 🤷

I found there are some downsides for this approach in some cases.
I am here -> apps/api/src/app/controllers/sf-misc.controller.ts**
I want to import from here -> apps/api/src/app/services/sf-misc.ts
The import resolves as this:
import { buildPackageXml } from 'apps/api/src/app/services/sf-misc';
When it should be:
import { buildPackageXml } from '../services/sf-misc';
But the auto-imports from a relative @appName/.. are working great :) 🤷

I found there are some downsides for this approach in some cases.
I am here -> apps/api/src/app/controllers/sf-misc.controller.ts
I want to import from here -> apps/api/src/app/services/sf-misc.ts
The import resolves as this:
import { buildPackageXml } from 'apps/api/src/app/services/sf-misc';
When it should be:
import { buildPackageXml } from '../services/sf-misc';
But the auto-imports from a relative @appName/.. are working great :) 🤷

That's because the non-realtive forces Visual Studio Code to use absolute paths instead of realtive ones

I found there are some downsides for this approach in some cases.
I am here -> apps/api/src/app/controllers/sf-misc.controller.ts
I want to import from here -> apps/api/src/app/services/sf-misc.ts
The import resolves as this:
import { buildPackageXml } from 'apps/api/src/app/services/sf-misc';
When it should be:
import { buildPackageXml } from '../services/sf-misc';
But the auto-imports from a relative @appName/.. are working great :) 🤷

That's because the non-realtive forces Visual Studio Code to use absolute paths instead of realtive ones


Otp runner quoting with tabs

Checklist

[x] Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
[x] Any modified or new methods or classes have helpful JavaDoc and code is thoroughly commented
[ ] The description lists all applicable issues this PR seeks to resolve
[x] The description lists any configuration setting(s) that differ from the default settings
[x] All tests and CI builds passing

Description
Adds the ability to properly escape otp config files that have tabs inside them.

Codecov Report

Merging #328 into dev will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                dev     #328      +/-   ##
============================================
+ Coverage     21.38%   21.41%   +0.02%
  Complexity      447      447
============================================
  Files           153      153
  Lines          8370     8373       +3
  Branches       1159     1159
============================================
+ Hits           1790     1793       +3
  Misses         6408     6408
  Partials        172      172

Flag
Coverage Δ
Complexity Δ

#unit_tests
21.41% <100.00%> (+0.02%)
447.00 <1.00> (ø)

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

Impacted Files
Coverage Δ
Complexity Δ

...com/conveyal/datatools/manager/jobs/DeployJob.java
19.91% <100.00%> (+0.49%)
14.00 <1.00> (ø)

Continue to review full report at Codecov.

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

:tada: This PR is included in version 3.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix 2 XAML copy mistakes

@Altlock : Thanks for your contribution! The author, @gewarren, has been notified to review your proposed change.