MixinAttribute already defined?

Using Visual Studio 2022 (17.3.1). Any help is greatly appreciated. Aiming at creating some pull requests, but this one might need your expertise! Thanks :)

Create a new WPF project. Target .NET 6.
Add a reference to PartialMixin 1.0.55
Replace MainWindow.xaml.cs with the code below.

First (minor) problem
Visual Studio is unable to resolve the MixinAttribute without manually adding the namespace. Weird!

Second problem
The compiler complains about duplicate attributes and stuff. Once you get this error, the project is forever crippled! No matter what you do, it will produce this compilation error. Amazing stuff... Maybe the guid seen in the VSGeneratedDocuments folder has something to do with this?

Build started...
1>------ Build started: Project: MixinError, Configuration: Debug Any CPU ------
1>C:\Foo\PartialMixins\PartialMixins.PartialMixin\MixinAttribute.cs(5,25,5,39): error CS0101: The namespace 'Mixin' already contains a definition for 'MixinAttribute'
1>C:\Foo\MainWindow.xaml.cs(14,2,14,20): error CS0121: The call is ambiguous between the following methods or properties: 'MixinAttribute.MixinAttribute(Type)' and 'MixinAttribute.MixinAttribute(Type)'
1>C:\Foo\MainWindow.xaml.cs(15,2,15,20): error CS0121: The call is ambiguous between the following methods or properties: 'MixinAttribute.MixinAttribute(Type)' and 'MixinAttribute.MixinAttribute(Type)'
1>C:\Foo\PartialMixins\PartialMixins.PartialMixin\MixinAttribute.cs(4,6,4,27): error CS0579: Duplicate 'System.AttributeUsage' attribute
1>C:\Foo\PartialMixins\PartialMixins.PartialMixin\MixinAttribute.cs(7,16,7,30): error CS0111: Type 'MixinAttribute' already defines a member called 'MixinAttribute' with the same parameter types
1>Done building project "MixinError_znfwmrwh_wpftmp.csproj" -- FAILED.
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========

MainWindow.xaml.cs
using System.Windows;
using Mixin;

namespace MixinError;

public partial class MainWindow : Window
{
    public MainWindow()
    {
        InitializeComponent();
    }
}

// NOTE: Two attributes! Is this somehow related to the compilation error?
[Mixin(typeof(Bar))]
[Mixin(typeof(Baz))]
public partial class Foo
{
}

class Bar
{
    public string TheBar { get; set; }
}

class Baz
{
    public string TheBaz { get; set; }
}

The two Attributes should not be a problem. The generator should only emit one Mixin attirbute ever.
Just to be sure, the screenshot does highligt the generate class option. You haven't used this and generated an additional class.
It may be that in the beginning no code is emmited and no Attribute exists. It will be emitted on frist run, and then the corresponding option to import the namespace should exists. So the error (Class not found) should vanisch after the firts build.
If that is not the case maybe share the sample. I wasnt able to reproduce it yet…
And there are not that many steps I could screw up…

Thanks for your quick response. No, I did not generate a class. In fact, it is 100% reproducible in any new WPF project.
Here's a sample project: SimpleMixinTest.zip
Rebuild, boom!
I'm wondering if the WPF project type itself suffers from some really weird analyzer issue. I.e. some conflict with its XAML source generator?

I can confirm that the error only occurs when a .xaml file is included in the project.

https://github.com/dotnet/wpf/issues/6792
Adding this in the project file solves the problem!!
<Target Name="RemoveDuplicateAnalyzers" BeforeTargets="CoreCompile">
    <!-- Work around https://github.com/dotnet/wpf/issues/6792 -->
    <ItemGroup>
         <FilteredAnalyzer Include="@(Analyzer->Distinct())" />
             <Analyzer Remove="@(Analyzer)" />
             <Analyzer Include="@(FilteredAnalyzer)" />
    </ItemGroup>
</Target>

Ah Thank you, I did not have the affectede DotNet 6 version…


Add pywebview DLLs from version 4.2

What does this PR do?
This PR updates the DLLs added in the pywebview package, which were changed in version 4.2.
The directories were modified as follows:

lib/x86 => lib/runtimes/win-x86/native
lib/x64 => lib/runtimes/win-x64/native
lib/arm64 => lib/runtimes/win-arm64/native

For more info check the directory change on pywebview
Why was it initiated? Any relevant Issues?
I initiated this to make nuitka compatible with pywebview >= 4.2
PR Checklist

[x] Update nuitka/plugins/standard/standard.nuitka-package.config.yml with these changes

Ok, since I have a hotfix coming, I will include this one there directly, so it will be on develop now, but hit main and then develop again through git flow normally.

@kayhayen If you have any issues you'd like me to resolve, please let me know. I'm enthusiastic about contributing to open-source projects and eager to expand my skills.

@sudo-hannes that's great, there are some easy ones behind this tag: https://github.com/Nuitka/Nuitka/issues?q=is%3Aopen+is%3Aissue+label%3A"help+wanted"
And just generally, if you wanted to help with optimization, you could look at a recent commit on the hotfix/1.7.6 branch that added support for input built-in. There are some builtins to add. If you tried to add e.g. enumerate that would be very helpful and a largely isolated activity, i.e. can be recognized from existing stuff how to do it.


problem with nonascii codec into pipe

$ falco -V
falco version: 0.6.2
$ falco -n apple -i 4000 | more
check_item_in_database: 'ascii' codec can't encode character u'\xe9' in position 907334: ordinal not in range(128)

fixed in commit 23902bfd213738bb6dc41890a7fee92aa7db9365
closed


Configuring pino

The log ingestion service I am using have certain requirements that require me to configure pino in a specific way. Is there any way to achieve this?
Great library by the way.

I think best would be to provide a way to pass a pino instance, that would already be configured with transports/serializers/redaction settings/etc. Something like next-logger.config.js where we can export a custom logger instance:
// next-logger.config.js
const pino = require('pino');

const logger = pino({
  messageKey: 'message',
  level: process.NODE_ENV === 'production' ? 'info' : 'trace',
  enabled: !(process.env.NO_LOG === 'true'),
});

module.exports = {
	logger,
};

I've got a beta version of this, with config loading - if you wouldn't mind trying it out. I'm especially interested to see how the config loading copes with your Yarn workspaces, @rohit-gohri!
npm install "https://github.com/atkinchris/next-logger.git#v3.0.0-beta.1"

Create a file at the root of your project called next-logger.config.js (or feasibly anything else that cosmiconfig can read from). Make this a CommonJS module, and export a field called logger. If this is a Pino instance, it will be used instead of the default instance.
For an example, here's the config I've been using this evening on one of my projects that consumes this library.
// next-logger.config.js
const pino = require('pino')

const logger = pino({
  messageKey: 'message',
  timestamp() {
    return `,"@timestamp":"${new Date(Date.now()).toISOString()}"`
  },
  formatters: {
    level(label) {
      return { level: String(label).toUpperCase() }
    },
  },
})

module.exports = {
  logger,
}

Nice! That's works perfectly. :) That way I can require that config for my application logger as well.

I somehow missed the original beta release, tested this now. Working great in yarn workspace based monorepo too! 🥳

Thanks for confirming @karl-run & @rohit-gohri. We've also been using the beta in a couple of production applications since October.
I've formally released this now as a new major version. The existing behaviour is captured in the provided default logger, so there shouldn't be breaking changes.


how to get querystring name=value from message ?

is there methods to from message the query string all (name,value) ?
something like:
valName = message.QueryString["name"];
valAdrs  = message.QueryString["adrs"];

I'm not sure what you mean. Can you elaborate?

querySTring is a part of url after '?' means:
//localhost/index.htm?name=name1&adrs=adrs1
querySTring={name=name1&adrs=adrs1}
queryString["name"]=name1
...
I need it juest for info

I know what a query string is. I just don't understand which query string you want to read off.
The query string when making the websocket connection?
new WebSocket('ws://example.com/endpoint?name=foo&age=25')

Or of the page your browser is on?

yes to the :
<<The query string when making the websocket connection?>>
the second:
<<Or of the page your browser is on?>>
I never used, and I don't know how to handle it.
On Fri, Mar 4, 2016 at 9:01 PM, Jason Staten<EMAIL_ADDRESS>wrote:

I know what a query string is. I just don't understand which query string
you want to read off.
The query string when making the websocket connection?
new WebSocket('ws://example.com/endpoint?name=foo&age=25')
Or of the page your browser is on?
—
Reply to this email directly or view it on GitHub
https://github.com/statianzo/Fleck/issues/162#issuecomment-192411560.

Look at socket.ConnectionInfo.Path. You'll have to parse out the parameters on your own.


Fix failed command execution message

Description
Failed to execute 'mvn clean verify'
it is not failed to execute
it is that this command was executed and  failed
Acceptance Criteria

the message says exactly what happened – e.g. : Command * failed – execution of the following commands skipped: *

@landpro Do you want to fix this?

Hi There, was wondering if there is any specific reason that we have used '"github.com/codeskyblue/go-sh"' package instead of the already provided "os/exec" package.

Hello @akhilbojedla,
Yes, you are right. Adding exit code is a good idea.


HiFiGAN not found

python whisperconvert_exp.py
Traceback (most recent call last):
File "whisperconvert_exp.py", line 9, in
import utils
File "/home/notebook/code/personal/ConsistencyVC-voive-conversion/utils.py", line 14, in
import hifigan
ModuleNotFoundError: No module named 'hifigan'

use FreeVC's hifigan

Actually there is no problem to just delete it. Because ConsistencyVC don't use hifigan directly. I will change the code maybe three hours later.

I revised it, hope there is no bug this time


Can you add support for other field types for vertexes?

this didn't work out for me
CREATE (n:Person {name: 'top_node', attrib: {myclass: 'id'}})
neither a list of attributes. CREATE (n:Person {name: 'Joe', attrib: [{myclass: 'id'}]})

Can you explain in more detail what doesn't work out for you? I do get expected results with your queries:
SELECT * FROM ag_catalog.cypher('test_graph', $$CREATE (n:Person {name: 'top_node', attrib: {myclass: 'id'}}) return n$$) as (n agtype);

                                                          n
---------------------------------------------------------------------------------------------------------------------
 {"id":<PHONE_NUMBER>31970, "label": "Person", "properties": {"name": "top_node", "attrib": {"myclass": "id"}}}::vertex
(1 row)

SELECT * FROM ag_catalog.cypher('test_graph', $$CREATE (n:person {name: 'Joe', attrib: [{myclass: 'id'}]}) return n$$) as (n agtype);
                                                        n
------------------------------------------------------------------------------------------------------------------
 {"id":<PHONE_NUMBER>31971, "label": "person", "properties": {"name": "Joe", "attrib": [{"myclass": "id"}]}}::vertex
(1 row)

SELECT * FROM ag_catalog.cypher('test_graph', $$MATCH (n:person) return n.name, n.attrib$$) as (name agtype, attrib agtype);
    name    |       attrib
------------+---------------------
 "top_node" | {"myclass": "id"}
 "Joe"      | [{"myclass": "id"}]
(2 rows)

def node_tree(parentid , element):
    if parentid:
       x = {key:value for (key,value) in element.attrib.items()}
       print(x)
    for child in element.iterchildren():
         node_tree(element, child)

x = dom_tree.getroot()
node_tree(None, x)

returns
{}
{'charset': 'UTF-8'}
{}
{}
{'class': 'menu'}
{'class': 'itm'}
{'class': 'itm'}
{'class': 'itm'}
{}
{'id': 'idone', 'class': 'classone'}
{}
{'class': 'listing'}
{'class': 'item1'}
{'class': 'item2'}
{'class': 'item3'}
{'id': 'innerone'}
{}
{'id': 'innertwo'}
{}
{'id': 'second', 'class': 'below'}
{'class': 'inner'}
{}
{}
{}

however i have a syntax error
SyntaxError: syntax error at or near "'lang: en'"
LINE 1: ...pher('text_test', $$ CREATE (t:node {tag: 'html', 'lang: en'...
                                                             ^

I'm using https://github.com/rhizome-ai/apache-age-python

It's difficult to analyze your issue without a complete minimal reproducible example, but I think your issue might be more related to https://github.com/rhizome-ai/apache-age-python than to this repository.
Please provide the complete code of an example (as minimal as possible) to reproduce your issue.

@afidegnum
It would be great if you submit self-contained code snippets to see your problem.
Creating code from scratch, to try to replicate what you might be doing is time consuming and we may be missing your issues as we have seen a couple of times. IE we didn't catch what the problem with your code is.

Is this still an issue? Or, can this ticket be closed?

Sure, the ticket can be closed.


[🐞] Serialization does not always reuse strings from DOM

Which component is affected?
Qwik Runtime
Describe the bug
Qwik serialization only sometimes reuses strings from DOM.
Reproduction
https://stackblitz.com/edit/qwik-starter-qe2ajh?file=package.json,src%2Froutes%2Findex.tsx
Steps to reproduce

https://stackblitz.com/edit/qwik-starter-qe2ajh?file=README.md,src%2Froutes%2Findex.tsx
Click test button
Notice that the Qwik/JSON has Spanish even though Manu was correctly reused. I would expect Spanish to also back refer to the DOM.

System Info
na

Additional Information
No response

Spanish is never rendered in the HTML as a text node

What do you mean? It is right here:

Ohh I see Spanish is rendered as Manu/Spanis with no id. How do you know if the ID needs to be placed on the HTML or not? Is it because it is not a signal?

Ohh I think I understand. Not a signal, therefore there is no way to know that we may need this ID, hence we can't save it.
Wondering if we should close this issue, or is it something that we could fix... I think I am just going to close it for now.


Problem with RecyclerView not rendering the progress.

So.. I'm using CardsLib with a CardRecyclerViewList and I'm having problems with the rendering of the ProgressBar.
The only elements that get rendered are the elements that are visible on the first screen. But as I scroll down all of the remaining cards only show a grey background.

Same here.Any solution yet?

same here. Still have errors with listview too.

I update latest version (1.2.0) and fix some problem about this view when using on adapter view. But it still have some bug, I will fix it later.

@akexorcist it worked! At least for me. Once I updated to 1.2 it rendered fine. Thanks for the update!! :+1:

@akexorcist I found the setProgress() will not draw properly in ListView when the convertView is null in getView(). it will always  draw whole progress bar even the getProgress return a value which is not equal to rcMax. Looking forward your fix. Thanks for your effort!!!

i found same problem in header list, not renderer data

Same here.Any solution yet?

I update latest library version. Update your code to using version 2.0.2 and check it again. I've already fix this render problem.

Thank akexorcist! It works perfectly


express deprecated req.host: Use req.hostname instead

Describe the bug
The latest volto starts with the warning express deprecated req.host: Use req.hostname instead build/server.js:62931:70
Software:

Volto Version: [16.0.0-alpha.11]
Node: v14.17.1

Fixed in #3525


integrate tailwind

Feature
Description
Integrate tailwind and stimulus
Fixes #
Checklist

[ ] My code follows the style guidelines of this project
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] The build is passing

Padding is different on header and footer. Going to merge for now and address styles in another PR.


Typo in Node message

In the Node message:
 1114  repeated Delegation delagations = 14;

This has been fixed.


Chrisda to Main

Docs Build status updates of commit 5e7b464:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsTeamsUpgradeStatus.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Get-CsUserCallForwardingSettings.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Get-CsUserDelegates.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Get-CsUserTeamMembers.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsCallQueue.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsTeamsMeetingBroadcastPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsTeamsMeetingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsTeamsMobilityPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsCallQueue.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsConferencingConfiguration.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsTeamsEducationAssignmentsAppPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsUserCallForwardingSettings.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsUserDelegates.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsUserTeamMembers.md
:white_check_mark:Succeeded
View (skype-ps)

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


#94 changes the colur of a bold text or Updated styles.css

### The problem is  :
I found that when we are using dark mode we get the text disapear after applying dark mode. so i just changed the text colour.
Before

vs
After

@Piyushsahu99 one of your ss is not visible upload again


njit version of workloads execute for smaller inputs than the kernel version

njit versions of all workloads fail after the execution of a smaller input size than the kernel versions. For instance, the DBSCAN workload is setup to execute a series of input sizes from 1024 to 16384 with each step incrementing the input size by 2X (i.e., 1024, 2048 ... , 16384). In such a setup, the kernel version runs for all input sizes. In contrast, the njit version fails after input size 4096.
The failure is not due to an OpenCL out-of-memory error. Rather, the process is killed by the OS. To me, this could be due to excessive kernel instances in the queue. The execution of the kernel in the kernel version is likely synchronous. Whereas, njit version could likely be launching several kernels asynchronously leading to excessive kernel launches.

This has been resolved.

This has been resolved.


runtime: optimize for special case where two-return-value map access discards the value

A pattern that I'm coming up on reasonably frequently in my code is the following:
if _, ok := myMap[key]; ok {
    // do some stuff
}

It seems to me like there would be some (albeit small) gains to be had in the compiler detecting this particular case of discarding the value result of map-lookup in favor of only returning the boolean, and avoid doing the pointer arithmetic necessary to find that value.

It's already optimized for in src/cmd/compile/internal/gc/walk.go:
                // don't generate a = *var if a is _
                if !isblank(a) {
                        var_ := temp(Ptrto(t.Type))
                        var_.Typecheck = 1
                        n.List.N = var_
                        walkexpr(&n, init)
                        *init = list(*init, n)
                        n = Nod(OAS, a, Nod(OIND, var_, nil))
                }

And the "pointer arithmetic necessary to find that value" has already been done at the point it's determined whether it's in the map or not.
There is perhaps an addition or multiplication that could be removed, but it's so minor compared to the work already done up to that point. And I imagine plumbing down a "skip this one addition and multiple" would be heavier than just doing it.

mapaccess with two return values will call runtime.mapaccess2
and its "fast" alternatives, they don't actually copy the value
out, they return an unsafe.Pointer for the value, and as Brad
mentions, the compiler won't copy the object if it's assigned
to _.
Adding another map access routine to runtime that only returns
the ok value won't save much because once the entry is found,
it's trivial to return it's address. And not to mention the code
duplication if we optimize for this case.


Failed to find SST files during compaction

What type of bug is this?
Data corruption
What subsystems are affected?
Datanode
Description
When using S3 as storage, datanodes complains failed to compact region since it cannot find some input files.
Failed to compact region:<PHONE_NUMBER>000(1375, 0) err=0: OpenDAL operator failed, at greptimedb/src/mito2/src/sst/parquet/reader.rs:126:14
1: NotFound (persistent) at  => File not found: data/[...]/public/1375/1375_0000000000/62a9180f-ff63-4137-9d5e-6eae7ae5e178.parquet

The missing file is created by another compaction which may still be influenced by this EntityTooSmall issue. But in normal control flow, if the manifest file has been successfully updated, the SST file must have already been uploaded to S3 before manifest update. So the problem is:

GreptimeDB believes the SST file has been uploaded, and no error logs about this SST file.
Manifest file has been updated.
No sign about this SST file has ever been deleted.
No version of this SST file is listed on S3 console.

In summary, this missing SST file might never have been uploaded successfully ever, while for some reason datanode mistakenly thought it was done and updated the manifest.

We can list all incomplete multipart uploads via awscli, and this missing file is marked as "incomplete":
{
  "UploadId": "[...]",
  "Key": "cluster-prod1-2/data/[...]/public/1205/1205_0000000000/0ad457b9-9f21-4593-9d1f-dd1b968e1813.parquet",
  "Initiated": "2023-11-08T12:47:34+00:00",
  "StorageClass": "STANDARD",
  "Owner": {
    "DisplayName": "[...]",
    "ID": "[...]"
  },
  "Initiator": {
    "ID": "[...]",
    "DisplayName": "[...]"
  }
}

Added a flag to ignore those files.
https://github.com/GreptimeTeam/greptimedb/pull/2745
We can enable this to skip incorrect manifests and remove it after all manifests are fixed.

As per AWS ticket response:

由内部工具，我可以看到您在 Nov 08 12:47:33 建立了 Multipar upload，并上传了约 81 个片，每个片约 4194304 （最后一个 29508）。

Total bytes uploaded to S3 was 4194304*80+29508=335,573,828, which is the same as file size in manifest:
        "files_to_add": [
          {
            "region_id":<PHONE_NUMBER>680,
            "file_id": "0ad457b9-9f21-4593-9d1f-dd1b968e1813",
            "time_range": [
              {
                "value":<PHONE_NUMBER>950,
                "unit": "Millisecond"
              },
              {
                "value":<PHONE_NUMBER>950,
                "unit": "Millisecond"
              }
            ],
            "level": 1,
            "file_size": 335573828
          }
        ],

We now suspect the following causes:

datanode did not invoke "complete multipart upload", which is wrapped inside opendal's writer's shutdown method.
datanode invoked "complete multipart upload", which actually failed but datanode falsely considered it a success and proceeded to update the manifest.

Another file not found error https://github.com/GreptimeTeam/greptimedb/issues/3633. But it was caused by a bug.

Looks like not happen again.


Initial value is always null when using localStorage

When the chrome extension API isn't available and the library falls back to using localStorage, the initial value ends up being null rather than the expected value.
I'm using the storage hook in React like so:
const [foo, setFoo] = useStorage("foo", "bar")
When using the browser's localStorage (for example, when testing components outside of the context of an extension), the initial value of foo always ends up being null instead of the expected "bar"
This seems to have cropped up in the latest version (v0.6.0) - it was not happening in v0.5.1 which I was using previously. I think the issue is in the parseValue function here:
https://github.com/PlasmoHQ/storage/blob/a275b47f0314fda8b87cf4c499b1970adb8aa757/src/index.ts#L172-L181
When using chrome extension API's sync or local storage, if a value doesn't exist, it returns undefined - so this would work fine. But if we're using the browser's localStorage, if a value doesn't exist, it returns null and so this function returns null instead of the expected undefined
I was initially thinking just adding a null check in line 174: && rawValue !== null might work, but the problem with that is that then the value would be reset if we later set the value to null (and we want it to stay null)

I think the edge case we want to resolve here is that, if we are in localStorage fallback mode and that the returned value is null, we return undefined instead? It's quite bizzare that localStorage have default value to not undefined tho..... I would need to experiment with this a bit further

We can also just fallback to the original value parsing for localStorage used in the previous version as well


commenting out some code to ensure some pytest fails

what happens when some of my pytests fail?

wow, it actually tells me and shows me that something failed

awesome, so now I have fixed the fails by returning the commented out code and the checks have passed and I am getting the green light to merge this branch.
I also got emails that my tests had failed, added bonus feature :)


[UI] - Broken dashboard with rc1

After installing a fresh epinio 1.0.0-rc1, I got this homepage in the dashboard:

Step to reproduce:

Create k3d cluster
Deploy epinio with:
helm install --devel epinio -n epinio --create-namespace epinio/epinio --set global.domain=${DOMAIN}

I believe I saw it yesterday when I forgot to use EXCLUDES_PKG=rancher-components variable before building my local dev env.

The next build (containing https://github.com/epinio/ui/issues/138) should resolve it

Depends on https://github.com/epinio/helm-charts/pull/235, but i've tested with below and get the expected result
kubectl set image -n epinio deployment/epinio-ui epinio-ui=ghcr.io/epinio/epinio-ui:v1.0.0-0.0.4

Sorry, I forgot to update this one.
Tested successfully with epinio-ui:v1.0.0-0.0.4 and all good in rc2 😄

Sorry, I forgot to update this one.
Tested successfully with epinio-ui:v1.0.0-0.0.4 and all good in rc2 😄


Update Envoy to v1.10.0

Fixes #998 by updating all the example deployments to use Envoy v1.10.0. It also updates the Makefile to have the local task use the same version.
Signed-off-by: Steve Sloka<EMAIL_ADDRESS>

Thank you. Please mark this for the 0.13 milestone.


Add automated E2E code coverage for client applications

Description Of Changes
Experimenting with using istanbul, @cypress/code-coverage, and codecov together to pull automated coverage reports for the client codebases 👍
Code Changes

[X] Add istanbul plugin to admin-ui and privacy-center projects, but only enable it in "test" builds used for Cypress (ie. cy:start)
[X] Use Cypress' code-coverage plugin to publish coverage reports while running specs
[X] Update GitHub CI (frontend_checks.yml) to collect coverage reports and upload to CodeCov
[X] Modify CodeCov actions to tag pytest coverage with a "backend" flag
[X] Modify CodeCov configuration with separate coverage targets for "backend" vs "admin-ui" vs "privacy-center"

Steps to Confirm

[X] View the CodeCov checks attached to this PR:

Pre-Merge Checklist

[X] All CI Pipelines Succeeded
Documentation:

[X] documentation complete, PR opened in fidesdocs
[X] documentation issue created in fidesdocs

[X] Issue Requirements are Met
[X] Relevant Follow-Up Issues Created
[X] Update CHANGELOG.md
[X] For API changes, the Postman collection has been updated

Do you know if there's a way to also instrument our unit tests and combine coverage with cypress?

Amazing! A few comments:

Do you know if there's a way to also instrument our unit tests and combine coverage with cypress? We have some unit tests—not a lot, but some that cover nitty gritty logic of helper funcs—which would be nice to have covered as well. Though again, there aren't that many (more so in fides.js) so I don't think this is a high priority

Yes, that's possible. For the pytest coverage we upload multiple reports to codecov and it aggregates it all together - although I find that kinda hard to "grok". A simpler way would be to run the unit tests & cypress tests from the same job so that the coverage report would aggregate locally on the GitHub runner and then upload together.

I wonder if this can work to cover fides.js at all, given that its tests are in a different folder....

I tried really briefly to do this. It was easy enough to add rollup-plugin-istanbul to fides.js, but had a couple issues:

it immediately explodes the bundle size and our build pipeline was very upset with me
coverage reports didn't work when I ran Cypress tests from /clients/privacy-center; I have to assume it was confused by directories saying /clients/fides-js or something

Overall I punted on it pretty quickly.

I never added tests to the Regulations page (I know, I know! I'm a monster, but time was short and I'll do it as part of https://ethyca.atlassian.net/browse/PROD-1563 😄 ). it uses many of the same components as the Locations page, but I noticed that regulations.tsx doesn't show up in the code coverage report at all, probably because no cypress test visits it. So just flagging that our % may be inflated if there are parts of the app that aren't covered, but are also never visited by cypress, it looks like they just won't be included, as opposed to lowering our %

Interesting... I'd have hoped that coverage reports should include all files, especially those that aren't exercised by tests. I wonder if this might be caused less by the lack of a Cypress test and more if cy:start doesn't force those files to actually get built.

overall, great increment! :shipit:

😄

Just pushed an update that'll now pull all the source files into the coverage report, and fully untested files show up as semi-transparent entries with 0% coverage:

However, it's still misleading - Istanbul doesn't include these untested files in the total coverage statistics, which you can basically see in that screenshot above... and this seems like an open issue with Istanbul in general: https://github.com/istanbuljs/nyc/issues/1415
...overall I hesitate to merge something in that is actively misleading because it doesn't include untested files...


ROS support for robotiq epick vacuum gripper

Are there ROS packages available for controlling robotiq epick vacuum gripper?

I am planning to create a ROS package for the epick vacuum gripper and contribute to the ROS industrial community. So, before starting to work on writing the package, I just want to know , are there anyother people working on this issue, we can collaborate if needed

Hi, I'm sorry for not replying earlier, but I've already done one and I'm planning to do a PR soon about this. So, I developed them less than a month ago and it should be then considered as an "alpha release". As I haven't made a Pull Request to this repo they should not be considered official at this time, but they should work and I'd be glad to provide you assistance if you need it. You can find this new package, named "robotiq_vacuum_grippers" (as per Robotiq's request) on my personal Github for now : https://github.com/jproberge/robotiq  see the package robotiq_vacuum_grippers_control in this stack.
I also recorded a short video in case you would like to check out the package as well as the EPick "in action": https://www.dropbox.com/s/8o2fn0ek5x9nobl/20190810_020640.mp4?dl=0 .
This package stack should work perfectly well on ROS Kinetic or ROS Melodic (I personally use Melodic). The procedure would be exactly the same as if you were to install our package stack from our official Github Repo. The only difference here is that you will have to use my own fork for now since, as I've said, it's not an official release yet.
Finally here's a quick start guide to make your gripper work with ROS, which is also a big part of what I'm showing in the video:

Power up the vacuum gripper and connect it to your computer

Make sure you have the right permissions to read and write your EPick. As noted on the wiki page, this is needed to be done every time by typing, e.g.: "sudo chmod 666 /dev/ttyUSBO" without quotes, OR a single time if you just add your Linux user name to the "dialout" group by typing "sudo usermod -a -G dialout $USER". You just need to execute only one of the aforementioned commands depending on what you prefer the most (having to always give the permission when the gripper is connected or always having the proper rights by default).

Start ros by typing "roscore" in a terminal

Once your catkin workspace containing our ros package stack is compiled, you are now ready to run the sensor node. This is done by simply typing "rosrun robotiq_vacuum_grippers_control RobotiqVacuumGrippersRtuNode.py /dev/ttyUSB0"  without quotes (assuming ttyUSB0 is your gripper, which should be the case if you don't have another UART device plugged in). See the video if needed.

That's it! Now the gripper is listening to your commands! We provide a minimalist interface if you want to see some options for using your gripper (which I use in the video). You can run the interface by typing, in another terminal, "rosrun robotiq_vacuum_grippers_control RobotiqVacuumGrippersSimpleController.py". With this minimalist interface, you will be able to send and read commands to/from your gripper (video). You can also try "rosrun robotiq_vacuum_grippers_control RobotiqVacuumGrippersStatusListener.py" if you want to see that values of some of the input registers.

Best regards,
Jean-Philippe Roberge

Hello,
Thanks for sharing the code. I was testing the code. I am getting the following error , incase if I am not sending any command for sometime,

output.append((response.getRegister(i) & 0xFF00) >> 8)

AttributeError: 'ModbusIOException' object has no attribute 'getRegister'

Thanks @gokul-gokz  for the additional information.
I've actually went to debug this problem at a customer's office this week and I've talked with some engineers from Robotiq about the issue as well. I will need to fix the problem, because there indeed seems to be a communication error that can happen once in a while.
To temporarily solve the problem, I've made a patch for the customer which basically involved line 110 to 123 of the file baseRobotiqVacuumGrippers.py.  These are the problematic lines where a request will be sent to the vacuum gripper to send its status. Sometimes, it seems the gripper doesn't send back its status after it's been requested. So, a non-elegant but working patch for now (until a make a permanent fix) can simply be to modify those lines such that the process will continue to request the status from the gripper until the gripper sends back a response. This is clearly a patch, I will make sure to collaborate with Robotiq engineers to make a proper solution soon. This is just in the meantime, if you have to make this work.
I'll make a pull request this week and reference this issue. Also, you are right: after the gripper has been activated, you should first press "c" (Release) in order to grip again. This will be changed too. Thanks for your comments, you are helping making this package better.

Hi @jproberge
Do you fix this problem for your customer?   _File "/home/li2jq/catkin_ws/src/robotiq-kinetic-devel/robotiq_modbus_rtu/src/robotiq_modbus_rtu/comModbusRtu.py", line 95, in getStatus
output.append((response.getRegister(i) & 0xFF00) >> 8)
AttributeError: 'ModbusIOException' object has no attribute 'getRegister'
Also from your last comment, you made a patch to temporarily solve this problem. Could you provide the detailed instructions about this patch? Like where to add/change/delete the code. Thank you!
Best regards
Jiqing Li

Hi, I'm sorry for not replying earlier, but I've already done one and I'm planning to do a PR soon about this. So, I developed them less than a month ago and it should be then considered as an "alpha release". As I haven't made a Pull Request to this repo they should not be considered official at this time, but they should work and I'd be glad to provide you assistance if you need it. You can find this new package, named "robotiq_vacuum_grippers" (as per Robotiq's request) on my personal Github for now : https://github.com/jproberge/robotiq see the package robotiq_vacuum_grippers_control in this stack.
I also recorded a short video in case you would like to check out the package as well as the EPick "in action": https://www.dropbox.com/s/8o2fn0ek5x9nobl/20190810_020640.mp4?dl=0 .
This package stack should work perfectly well on ROS Kinetic or ROS Melodic (I personally use Melodic). The procedure would be exactly the same as if you were to install our package stack from our official Github Repo. The only difference here is that you will have to use my own fork for now since, as I've said, it's not an official release yet.
Finally here's a quick start guide to make your gripper work with ROS, which is also a big part of what I'm showing in the video:
1. Power up the vacuum gripper and connect it to your computer

2. Make sure you have the right permissions to read and write your EPick. As noted on the wiki page, this is needed to be done every time by typing, e.g.: "sudo chmod 666 /dev/ttyUSBO" without quotes, OR a single time if you just add your Linux user name to the "dialout" group by typing "sudo usermod -a -G dialout $USER". You just need to execute only one of the aforementioned commands depending on what you prefer the most (having to always give the permission when the gripper is connected or always having the proper rights by default).

3. Start ros by typing "roscore" in a terminal

4.  This is done by simply typing "rosrun robotiq_vacuum_grippers_control RobotiqVacuumGrippersRtuNode.py /dev/ttyUSB0"  without quotes (assuming ttyUSB0 is your gripper, which should be the case if you don't have another UART device plugged in). See the video if needed.

5. That's it! Now the gripper is listening to your commands! We provide a minimalist interface if you want to see some options for using your gripper (which I use in the video). You can run the interface by typing, in another terminal, "rosrun robotiq_vacuum_grippers_control RobotiqVacuumGrippersSimpleController.py". With this minimalist interface, you will be able to send and read commands to/from your gripper (video). You can also try "rosrun robotiq_vacuum_grippers_control RobotiqVacuumGrippersStatusListener.py" if you want to see that values of some of the input registers.

Best regards,
Jean-Philippe Roberge

hello，i have followed your steps, after i run command 'rosrun robotiq_vacuum_grippers_control RobotiqVacuumGrippersRtuNode.py /dev/ttyUSB0', the light turns to green, then ,i run "rosrun robotiq_vacuum_grippers_control RobotiqVacuumGrippersSimpleController.py", whatever command i use ,the epick didn't work. but i can use ur screen to control it. could you tell me why?

i also want to ask that why my epick's light turns to green rather than blue after i run 'rosrun robotiq_vacuum_grippers_control RobotiqVacuumGrippersRtuNode.py /dev/ttyUSB0', so i can't control my epick and also could not receive the status from the epick even though i have controlled it by ur.


Fixed docs misspelling: netowrk -> network

PLEASE NOTE THAT THE TORCHAUDIO REPOSITORY IS NO LONGER ACTIVELY MONITORED. You may not get a response. For open discussions, visit https://discuss.pytorch.org/.

lgtm


Dynamically created PDF can Not be exported

In the demo sample (BasicPdfViewerExample) the file (/pdf/example.pdf) is loaded by the class loader (getClass().getResourceAsStream(..)).
And the example works! :-)
If I create a pdf dynamically (via class PdfCreator uses the Java library pdfbox) and hand it over to the PdfViewer, the file can not be downloaded!
Any help is appreciated! Thanks!
PdfViewer_Issue.zip
(Technically spoken: Whether the solution works depends on the implementation of the InputStream.)

Hey, this repository is for the Directory system itself. I'll close this one, but you might want to report here: https://github.com/vaadin-component-factory/vcf-pdf-viewer-flow/issues


Chore: use a single package for all Feature structs

Also replaces golang.org/x/exp/slices with slices from the Go 1.21 stdlib

small spin-off from #3046

Where is the tracking issue for this? Why are we doing this? I would like some explanation - no PRs without discussion.

Where is the tracking issue for this?

Just a refactoring to avoid code duplication. Instead of having a Features struct defined in each package, and the Contains method repeated, it's now implemented just once (and re-exported using generics)


ControlPoint.start(target) not discovering the devices on Android Q

I am trying to discover my target device from my android application. It worked so far. Now I am trying this on Android Q and my target is not getting discovered.
The same code discovers my target device on lower versions of android (P, O). No errors thrown.
I can see the search packet being broadcasted.
My target device does get listed if I remove the ST: parameter from my search on Anroid Q. I mean ssdp:all lists it. When I add the target parameter to the start() method, it doesn't list my target device. What could be going wrong ?
Any help is appreciated.

Solved this issue.
This is because of the Multicast packets filtered out by the Android platform - https://developer.android.com/reference/android/net/wifi/WifiManager.MulticastLock
Since the SSDP Notify multicast packets were getting filtered out, the socket was not receiving any advertisements and hence not discovering any devices.
Acquire the Multicast lock and this issue will resolve.


Core dump after end of phase 2.

Phase 2 took 272.88 sec
Wrote plot header with 268 bytes
terminate called after throwing an instance of 'std::runtime_error'
what(): thread failed with: fopen() failed
Aborted (core dumped)
Any ideas? I am running the latest version downloaded today. I looked around the Chia-plotter folder but I dont see a core file.
./chia_plot -n 1 -r 48 -u 256 -t /mnt/raid16/ -2 /mnt/ramdisk/ -d /mnt/raid8/
raid16 is a 16 way raid 0 10k sas
raid 8 is an 8 way raid 0 10k sas
ramdisk is a 110gb tmpfs ramdisk
Dual Xeon 2690v3 with 24 core 48 threads and 256 gb of ram.
Also tried bumping up my ramdisk to 150gb and 180gb, no change.. still core dumped same message
Lowered the thread count to 46 to give a couple for OS.. still dumped.
I know this system is solid. Its been plotting successfully for weeks using the chia plotter under heavy load.

mine to..

just do on cd chia-plotter/
git reset --hard 16f9793b78b60eb9a4e84ff8048998c3ac62a0e5
and rebuild..
git submodule update --init
./make_devel.sh
./build/chia_plot --help

git show

if show ommit 16f9793b78b60eb9a4e84ff8048998c3ac62a0e5 (HEAD -> master) good to go. update is out soon...

Same here. I got Segmentation fault (core dump). Stopped at [P1] Table 4. Newly installed ubuntu, so new git clone of repo.

Phase 2 took 272.88 sec
Wrote plot header with 268 bytes
terminate called after throwing an instance of 'std::runtime_error'
what(): thread failed with: fopen() failed
Aborted (core dumped)
Any ideas? I am running the latest version downloaded today. I looked around the Chia-plotter folder but I dont see a core file.
./chia_plot -n 1 -r 48 -u 256 -t /mnt/raid16/ -2 /mnt/ramdisk/ -d /mnt/raid8/
raid16 is a 16 way raid 0 10k sas
raid 8 is an 8 way raid 0 10k sas
ramdisk is a 110gb tmpfs ramdisk
Dual Xeon 2690v3 with 24 core 48 threads and 256 gb of ram.
Also tried bumping up my ramdisk to 150gb and 180gb, no change.. still core dumped same message
Lowered the thread count to 46 to give a couple for OS.. still dumped.
I know this system is solid. Its been plotting successfully for weeks using the chia plotter under heavy load.

Why you put "-u 256"? if not mistaken, -u is for bucket number and the argument suppose to be 7 for 128 or 8 for 256.

Same issue

Phase 2 took 272.88 sec
Wrote plot header with 268 bytes
terminate called after throwing an instance of 'std::runtime_error'
what(): thread failed with: fopen() failed
Aborted (core dumped)
Any ideas? I am running the latest version downloaded today. I looked around the Chia-plotter folder but I dont see a core file.
./chia_plot -n 1 -r 48 -u 256 -t /mnt/raid16/ -2 /mnt/ramdisk/ -d /mnt/raid8/
raid16 is a 16 way raid 0 10k sas
raid 8 is an 8 way raid 0 10k sas
ramdisk is a 110gb tmpfs ramdisk
Dual Xeon 2690v3 with 24 core 48 threads and 256 gb of ram.
Also tried bumping up my ramdisk to 150gb and 180gb, no change.. still core dumped same message
Lowered the thread count to 46 to give a couple for OS.. still dumped.
I know this system is solid. Its been plotting successfully for weeks using the chia plotter under heavy load.

Why you put "-u 256"? if not mistaken, -u is for bucket number and the argument suppose to be 7 for 128 or 8 for 256.
I might be wrong, but you can try to remove "-u 256" option let it run using default value.

This argument was changed so you can use either log notation (7, 8 etc) or standard notation (128, 256 etc). But yes. It isn't needed anyway because the default is 256.

just do on cd chia-plotter/
git reset --hard 16f9793b78b60eb9a4e84ff8048998c3ac62a0e5
and rebuild..
git submodule update --init
./make_devel.sh
./build/chia_plot --help

git show

if show commit 16f9793 (HEAD -> master) good to go. update is out soon...

Did not work. Got a new error following the above suggestion.
terminate called after throwing an instance of 'std::runtime_error'
what():  thread failed with: input not sorted
Aborted (core dumped)

you just need to comeback the update. run on the day 13...

Ok try latest master, I might have miscalculated something in 89819ce

Well, I have the same question. My dram is 3200.But when I close the docp( xmp) and set dram is 2666( default), I figured it, now it work well.Try it.

error when trying to plot many in parallel
terminate called after throwing an instance of 'std::runtime_error'
what():  thread failed with: thread failed with: fwrite() failed
error when typing chia_plot again
Cannot open at least 294 files, please raise maximum open file limit in OS.
Attempted fix:
sudo sysctl -w fs.file-max=999999999999999999


Can't see minutes on clock [android]

As the title says can't see the minute clock on bromite(chromium fork & possibly same result with chrome).
Device: Nokia 8.1 (a11)
Browser: Bromite (Version 101.0.4951.53)
Screen resolution: 1080 x 2280 pixels, 19:9 ratio
 preview👀

I tried reproducing the issue but it works just fine, could you please check if its working fine on other browsers, I'm using chrome and it looks like this for me:

I tried reproducing the issue but it works just fine, could you please check if its working fine on other browsers, I'm using chrome and it looks like this for me:

hm chrome does supports the clock perfectly but other chromium bowsers seems to have this bug..tried on firefox, kiwi (android chrome with extensions) & original chromium..all seems to have similar issue.maybe its related to the screen resolution as chrome handles screen resolution differently than others..maybe tweaking some pixel values on media query will fix the issue..i highly suggest you to use firefox for your dev  environment

I tried reproducing the issue but it works just fine, could you please check if its working fine on other browsers, I'm using chrome and it looks like this for me:

hm chrome does supports the clock perfectly but other chromium bowsers seems to have this bug..tried on firefox, kiwi (android chrome with extensions) & original chromium..all seems to have similar issue.maybe its related to the screen resolution as chrome handles screen resolution differently than others..maybe tweaking some pixel values on media query will fix the issue..i highly suggest you to use firefox for your dev environment

thanks for testing it, I will try to look into this and fix it if possible , I'm thinking of using firefox for dev actually, thanks for the suggestion :)

@WahidIslamLinad I tried testing it myself, I even tested i on chromium but it looks just fine for me

Can you test once again and let me know if it worked? thanks.

hm i tested it now it works fine on all platform ❤
now only left to do is maybe add settings to the page & save them to local storage 🤔 ..Alright then so as the bug fixed maybe we should close this issue, btw thnx for quick response ❤

Can you test once again and let me know if it worked? thanks.

hm i tested it now it works fine on all platform ❤ now only left to do is maybe add settings to the page & save them to local storage 🤔 ..Alright then so as the bug fixed maybe we should close this issue, btw thnx for quick response ❤

yes I'm planning to add more features soon, local settings is one of them :)
thanks for pointing out the issue.


$this->_req tweaks

This is mostly some tweaks to the http req class to add a compare function (post and get) as well as a request style function which looks at get then post.
Implemented and cleaned up some existing uses, mostly in the admin directory, of the above
Made some changes to the action controller so it should be even more robust, or I broke it, its one of the two.
Found a few bugs along the way and fixed those as well as some spelling and misc formatting just cuz

I missed that one... well, let's not waste more time! :D


Update MESSAGEix-Materials with new steel technologies

This PR updates MESSAGEix-Materials to version 1.1.0. The model changes are documented in doc/material/v1.1.0.rst:
The PR also adds tests to the MESSAGEix-Materials build on bare RES. Solving is not possible at the moment and can be added with a follow-up PR.
How to review
The PR is rebased on the most recent version of the main branch. The build, solve, and report functionalities are tested and work without problems. Reviewers can:

Check the code structure
Test the build-solve again
Compare input gdx file with an existing model/scenario version to see if any model functionalities are missing after rebase
Provide feedback on how to handle the versioning with the materials module

PR checklist

[x] Continuous integration checks all ✅
[x] Update doc/whatsnew. ✅
[x] Update data/material/material.tar.gz

I understand for the versioning, we can create a separate "What's New" section in doc/material/index.rst and describe the changes that were made there without specifying a model version for materials separately.

I understand for the versioning, we can create a separate "What's New" section in doc/material/index.rst and describe the changes that were made there without specifying a model version for materials separately.

You could do that, but I think it's preferable to keep the change notes in one place. So please consider creating a new subsection in doc/whatsnew.rst. This could be a list of indented bullet points as for tools-costs, but it could also be its own subsection of the v2024... heading (see e.g. here for how to do that).

You could do that, but I think it's preferable to keep the change notes in one place. So please consider creating a new subsection in doc/whatsnew.rst. This could be a list of indented bullet points as for tools-costs, but it could also be its own subsection of the v2024... heading (see e.g. here for how to do that).

To recap the rationale from this Slack message:

In general we treat the message-ix-models main branch as always-ready-to-be-released, and release frequently (at least, more frequently than the lower-level packages in the stack).
OTOH, major milestones for particular model variants (MESSAGEix-Water, -Materials, -Transport) come less frequently, often coinciding with particular publications.
This will mean that a milestone for one of these variants may contain changes scattered across multiple message-ix-models releases.
For readers (e.g. of those publications) who are more concerned with the methods and data in those variants, or the underlying analyses, it may be difficult to skim a lot of mostly-unrelated content in doc/whatsnew to get a clear picture of what differs between, e.g. MESSAGEix-Materials 1.0.0 and MESSAGEix-Materials 1.1.0.

So my suggestion was:

Still keep at least a link/1 sentence for each PR in the main changelog at doc/whatsnew.rst.
For variants that want to collect and designate changes (and in particular to give them their own designations like "MESSAGEix-Materials 1.1.0"), do this in a module-specific location that is cross-linked with the main changelog.
This can also be the place for more expansive methodological discussion and explanation that would be awkward in the main changelog, reducing its usefulness as a skimmable list of changes in each release.

@macflo8 @GamzeUnlu95 per discussion at today's MESSAGE meeting, can I suggest to include (as part of this currently active PR) something like the following on the page doc/material/index.rst —perhaps in the section "Code reference"?
.. note::
   See also :pull:`130`/the archived branch `materials-migrate <https://github.com/iiasa/message-ix-models/tree/migrate-materials>`_ for a distinct version of :mod:`.material`.
   That earlier PR was superseded by :pull:`188`, but may contain usable code or data related to:

   - A
   - B
   - C

This will help reader know that the branch is available, and decide whether they should look at it.

I have rewritten the initial commit history of this PR to remove some unnecessary files. I have also refactored the build functions a bit to make it testable. I made a make_spec() function, borrowed from the transport module and adjusted for materials, that is used for running and testing the MESSAGEix-Materials build. Almost all changes are within the material directories except for 2 small changes to make the new tests pass.
The version documentation for MESSAGEix-Materials 1.1.0 is in a separate file in doc/material. The style is based on the pandas-dev whatsnew docs.
@khaeru The reference to #130 has also been added.
For some reason that I could not trace, one test is failing for the Ubuntu and Windows Python 3.12 setup.
Besides that I think the PR is ready to be merged if @khaeru / @glatterf42 agree.

@measrainsey and I were able to resolve the issue with the failing test in the Python 3.12 setups. There were some odd entries in the relation set B.yaml that look like line counters. Apparently the file is parsed differently with Python 3.12 if these line counters are there. After removing them (see commit here), all tests pass. I am not sure if these line counters are needed. Maybe @khaeru can confirm?

Wow, great catch! I think that is mea culpa—they look like scrollback counters from Tmux on my system. I guess they were originally included when I prepared the contents of those files. They are not needed.
Thanks also for the comments above and work done cleaning up this PR. I am in a meeting all week and will likely be able to review/approve on Monday.

Thanks for the review and all the suggestions, @glatterf42! I will add a commit that extends the type hints of the material code base. I have adopted your suggestions in the docs and the built html looks good to me now.
I would like to add more tests. However since that requires quite a lot of extra work (see responses to the testing-related suggestions for more details), I would prefer to do that in a follow-up PR. As you say, this PR is already quite big.
Regarding the potential use of GitHub Projects: I think it would be indeed a useful tool, that would improve managing our future developments for MESSAGEix-Materials. But I can just speak for myself as one person that works on MESSAGEix-Materials. We should probably also discuss this with @GamzeUnlu95 and @yiyi1991.

Thanks a lot for helping with and reviewing this PR @glatterf42 & @khaeru. I have added a few more commits that address the few minor suggestions that were remaining. I will go ahead and merge now.


Possible fake data from Qwant API

Looks like all URLs from Qwant are fake for me:
Qwant results
1   http://ef.bf/teheuhi
2   http://cez.lr/di
3   http://veoca.la/gured
4   http://vamibif.th/unror
5   http://hicakda.hk/izwe
6   http://nofusive.mr/aco
7   http://ismuaka.pe/welagses
8   http://luho.mg/desler

Can you check if it is not only my problem?

I've realized it is not yet available in my country. So, fake results are deliberately given to me, and I have to use a foreign proxy. Sorry to bother you.


remove checksum-single task from git wh pipeline

use tag 20201214.161340 to test changes
successful build from pipeline

How to test:

check out branch

$ git checkout bn/cnx#1234/webhost-git-pipeline

set up local env file (output-producer-service/bakery/env/local.json)

{
  "ENV_NAME": "local",
  "COPS_TARGET": "http://backend/api",
  "COPS_ARTIFACTS_S3_BUCKET": "",
  "COPS_CLOUDFRONT_URL": "",
  "DIST_ARCHIVE_FEED_FILE_URL": "https://ce-rap-brenda.s3.us-east-1.amazonaws.com/archive-approved-books-list.json",
  "DIST_GIT_FEED_FILE_URL": "https://ce-rap-brenda.s3.us-east-1.amazonaws.com/git-approved-books-list.json",
  "DIST_S3_BUCKET": "ce-rap-brenda",
  "DIST_QUEUE_STATE_S3_BUCKET": "ce-rap-ripal-dev2",
  "DIST_ARCHIVE_QUEUE_FILENAME": "archive-queue.json",
  "DIST_GIT_QUEUE_FILENAME": "git-queue.json",
  "PIPELINE_TICK_INTERVAL": "45m",
  "MAX_BOOKS_PER_TICK": "2",
  "GDOC_GOOGLE_FOLDER_ID": "",
  "GDOC_QUEUE_FILENAME": "",
  "GDOC_FEED_FILE_URL": ""
}

build distribution pipeline, use tag 20201201.172633 and local env file

./bakery/build pipeline distribution local -o distribution.1234.yml

build git webhosting pipeline, use tag 20201201.172633 and local env file

./bakery/build pipeline git-distribution local -o git-distribution.1234.yml

(Must have local concourse up)

set distribution pipeline

fly -t cops-dev sp -p archive-back -c distribution.1234.yml

set git webhosting pipeline

fly -t cops-dev sp -p git-back -c git-distribution.1234.yml

unpause distribution pipeline

fly -t cops-dev unpause-pipeline -p archive-back

set git webhosting pipeline

fly -t cops-dev unpause-pipeline -p git-back

When you unpause these pipeplines:

pipeline git-back will pick up what is on the git-approved-books-list.json

[
    {
        "name": "Precalculus",
        "style": "precalculus",
        "version": "0.0.0",
        "slug": "precalculus",
        "uuid": "fd53eae1-fa23-47c7-bb1b-972349835c3c",
        "repo": "ce-git-storage-spike"
    }
]

pipeline archive back will pick up what is on the archive-approved-books-list.json

[
    {
        "name": "Precalculus",
        "collection_id": "col11667",
        "style": "precalculus",
        "version": "1.32.1",
        "server": "cnx.org",
        "uuid": "fd53eae1-fa23-47c7-bb1b-972349835c3c"
    }
]

The expectation is for both pipelines to succeed and dump artifacts into the ce-rap-brenda bucket.


[WIP] Output eval logging

Evaluating with: output-logger-test-dsQsKa
Confirmed uri s3://mosaicml-internal-checkpoints-test/test_icl_output_logger_7b contains all the data. Samples:
context_tok	continuation_tok_target	continuation_tok_pred	correct	benchmark	question_tok	correct_choice	selected_choice
b'AMERICAN HISTORY: Name that completed an 1856 Republican slogan, Free soil, free speech, free men and...\\nAnswer: Fremont\\nAMERICAN HISTORY: After this waterway opened in 1825, freight rates between Buffalo & NYC were cut by more than 90%\\nAnswer: Erie Canal\\nAMERICAN HISTORY: In 1949 Henry H. Arnold became the first general of this branch of the armed forces\\nAnswer: Air Force\\nAMERICAN HISTORY: After his 1859 hanging, Civil War troops sang of this abolitionists body mouldering in the grave\\nAnswer: John Brown\\nAMERICAN HISTORY: On April 24, 1980 an attempt to rescue 53 hostages in this capital city was called off\\nAnswer: Tehran\\nAMERICAN HISTORY: Witchcraft trials held in this town in 1692 led to the hangings of 19 people\\nAnswer: Salem\\nAMERICAN HISTORY: On March 10, 1977, this Cesar Chavez labor group signed an agreement with the Teamsters union\\nAnswer: United Farm Workers\\nAMERICAN HISTORY: On April 2, 1917 President Wilson told Congress, The world must be made safe for this\\nAnswer: democracy\\nAMERICAN HISTORY: He shook up society in the 1960s & later published Revolution for the Hell of It & Steal This Book\\nAnswer: Abbie Hoffman\\nAMERICAN HISTORY: In searching for the 7 Cities of Cibola, he & his men traveled as far north as present-day Lyons, Kansas\\nAnswer: Francisco Vazquez de Coronado\\nAMERICAN HISTORY: On May 29, 1765 Patrick Henrys Stamp Act protest was interrupted with this one word\\nAnswer:'	b' Treason!'	b' Noason\\n'	False	jeopardy/10-shot/american_history_InContextLearningLMAccuracy
b'AMERICAN HISTORY: Hostilities in the Spanish-American War ended when Manila surrendered to General Merritt & this admiral\\nAnswer: George Dewey\\nAMERICAN HISTORY: WWI sales of Liberty these, whose posters showed the statue, raised more than $15 billion\\nAnswer: bonds\\nAMERICAN HISTORY: In 1949 the U.S. joined this Western defense pact; our 1st ever peacetime alliance\\nAnswer: NATO\\nAMERICAN HISTORY: In 1843 Daniel Webster resigned as this presidents Secretary of State\\nAnswer: John Tyler\\nAMERICAN HISTORY: He was promoted to captain for burning the captured U.S. frigate Philadelphia in Tripoli Harbor\\nAnswer: Stephen Decatur\\nAMERICAN HISTORY: In 1798 Congress passed this collection of bills to control domestic dissent & conspiracy against the federal govt.\\nAnswer: the Alien & Sedition Acts\\nAMERICAN HISTORY: The British party of this name aimed to limit the kings powers; the U.S. party hoped to limit King Andrew Jackson\\nAnswer: the Whigs\\nAMERICAN HISTORY: He shook up society in the 1960s & later published Revolution for the Hell of It & Steal This Book\\nAnswer: Abbie Hoffman\\nAMERICAN HISTORY: On January 21, 1977 this president pardoned several thousand Vietnam War draft evaders\\nAnswer: Jimmy Carter\\nAMERICAN HISTORY: In May of 1780, Gen. Benjamin Lincoln was forced to surrender this South Carolina port to the British\\nAnswer: Charleston\\nAMERICAN HISTORY: This political party founded around 1789 stood for a strong central government\\nAnswer:'	b' the Federalists'	b' Federal Federalists'	False	jeopardy/10-shot/american_history_InContextLearningLMAccuracy

@bmosaicml btw, you can use github draft PRs to signify that your PR is wip

Closing, as @maxisawesome is doing this.


Switch read-only and write-only behaviour in callbacks and webhooks

Currently, RapiDoc hides read-only fields in requests and hides write-only fields in responses.
However, this behavior is the wrong way round for callbacks and webhooks, since the request is a "read" and the response is a "write". An obvious example of how this is the wrong way round would be an object with an ID field getting POSTed to a webhook:
{
    "id": 123,
    ...
}

Under the current behaviour, the ID field is hidden in the docs for requests that gets sent to a webhook or callback. Of course we would actually send the ID field.
Redoc had the same issue: https://github.com/Redocly/redoc/issues/1720. It looks like they solved it by correcting the behavior and adding a new setting to switch back to the old behavior if desired.
If you can think of any workarounds for this in the meantime, then please do share! Looking to add a webhook to my API, and would prefer not to have an awkward comment in the description that says "the ID will be there, even though the docs don't mention it" 😄

PS - Related Issue https://github.com/swagger-api/swagger-ui/issues/6294

No problem. IMO, rapidoc is the best solution out there over redoc and swagger UI - keep up the great work.


revert(treeshaking): wanted a breaking change

Reverts edx/paragon#479

:tada: This PR is included in version 6.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Coverage decreased (-0.07%) to 87.893% when pulling ac11833bb920f04585746d672c8dbf8a37d256dc on revert-479-djoy/babelize into fb5f9a3515d00ca2b960f71f337b4bd2f67d0b2b on master.


features of app

3 mode of pomo:

[x]  pomodoro
[x] short break
[x] long break

handle events

[x] handle onclick START btn:

[x] countdown
[x] active progress bar
[x] button next mode

setting task

[x] handle add task
[x] show task list
[x] show inf of all tasks
[ ] mark current task in left side when click
[x] create box for update
[x] save list task and setting in cache
[x]  edit a specific task

name
quantity
num task done
notes

[x] add a task with quantity and notes

config tasks

[ ] clear finished tasks
[ ] clear all tasks
[x] update time finished at
[x] update target and heading target
[ ] save as template
[ ] add from templates
[ ]  move task position and save pos

others

[x] responsive

[x] page login

[x] page signup

[ ] popup report

[ ] popup settings

[x] add keyframe/ animation

Login

[ ] with google
[ ] email/password
[ ] facebook
[ ] forgot password

Signup

[ ] with google
[ ] with email
[ ] facebook

Loading status

[ ] status with checkIcon
[x]  create progress bar with flex width

sounds

[ ] sound start countdown
[ ] sound finished an interval

notification

[ ] notify:

tittle page

[ ] change tittle and fav depend on pages/mode

switch mode or reload page

[ ] if counting : show popup and confirm

chăm vậy bé 💯

chăm vậy bé 💯
cảm ơn anh đã khen 😛

Cải tiến app

[ ] hiển thị trạng thái online/ offline / giao tiếp thời gian thực

van code a em :)


Increase field height to fit multi line placeholder

Which version of the WSTagsField are you using?
E.g. 5.4.0
On which platform does the issue happen?
E.g. iOS 14.4
Are you using Carthage?
No
Are you using Cocoapods?
No
I installed via SPM
Which version of Xcode are you using?
Version 12.4 (12D4e)
What did you do?
Set a rather long placeholder.
tagField.placeholder = "Tags make it easier to get back in the future. You can add multiple tags separated by space"

What did you expect to happen?
TagsField to increase in height and show the full placeholder in multiple lines
What happened instead?
The placeholder was truncated at tail as shown in the image below.

I tried to dig in, fix it myself and send a PR. But sadly, there are a lot of things which I can't figure out. I got to enumerateItemRects function and I saw that Constants.STANDARD_ROW_HEIGHT was used in calculating the height. Other than that, I can't figure out much of what is happening there. I am not sure how to change that function and make it also work based the length of the placeholder.
Even if you can't find time to fix it, kindly point me to a reference on how to fix this, I'll do what I can and send a PR.
Thanks for the awesome component!

@imthath-m Did you tried changing the numberOfLines to 0 in the placeholder label?

I can not find any placeholder label. Kindly let me know where to look.
I tried to fix this by calculating the attributed placeholder height and applying it to the text field. I've created a PR for those changes. But I'm not able to get the placeholder string to wrap across lines.


Laravel Sanctum (SPA) - Broadcasting - Private Channel + Presence Channel Not Working?

Hi, I am using Laravel -v7.x and Vue CLI two different folders for Frontend and Backend. And using Laravel Sanctum (SPA) Authentication. I want to create real-time Notifications when some Events happen in my backend. and I already configured successfully the Websockets and the public Broadcasting working fine. But when I'm trying to using PrivateChannel then backend WebSockets can run the event on the WebSockets panel I see but could not get response on the console. Can you please help me? Below I share my code.
Vue CLI
#main.js

<script>
import Request from "./apis/Request"
import Echo from 'laravel-echo';
window.Pusher = require('pusher-js');

window.Echo = new Echo({
    broadcaster: 'pusher',
    key: process.env.VUE_APP_WEBSOCKETS_KEY,
    wsHost: process.env.VUE_APP_WEBSOCKETS_SERVER,
    encrypted: true,
    cluster: 'ap1',
    wsPort: 6001,
    forceTLS: false,
    disableStats: true,
    authorizer: (channel, options) => {
        console.log("OPTIONS>>", options);
        return {
            authorize: (socketId, callback) => {
                Request.POST_REQ('/broadcasting/auth', {
                    socket_id: socketId,
                    channel_name: channel.name
                })
                .then(response => {
                    callback(false, response.data);
                    console.log("RESPONSE>>", response);
                })
                .catch(error => {
                    callback(true, error);
                });
            }
        };
    },
})
</script>

#App.vue

<script>
import { mapGetters } from "vuex";
export default {
    computed: {
    ...mapGetters(["user"])
  },
    async mounted() {
        await window.Echo.private(`hello.user.${this.user.id}`).listen(
      "Hello",
      () => {
        console.log("OK");
      }
    );
    },
}
</script>

#vue env
#.env
VUE_APP_WEBSOCKETS_KEY = local
VUE_APP_WEBSOCKETS_SERVER = localhost

#Laravel
#Hello Event
#Hello.php

<?php

namespace App\Events;

use Illuminate\Broadcasting\Channel;
use Illuminate\Broadcasting\InteractsWithSockets;
use Illuminate\Broadcasting\PresenceChannel;
use Illuminate\Broadcasting\PrivateChannel;
use Illuminate\Contracts\Broadcasting\ShouldBroadcast;
use Illuminate\Foundation\Events\Dispatchable;
use Illuminate\Queue\SerializesModels;

class Hello implements ShouldBroadcast
{
    use Dispatchable, InteractsWithSockets, SerializesModels;

    public function __construct()
    {
        //
    }

    public function broadcastWith()
    {
        return [
            'hello' => 'Hello world'
        ];
    }

    public function broadcastOn()
    {
        return new PrivateChannel('hello.user.1');
    }
}

#BroadcastServiceProvider.php

<?php

namespace App\Providers;
use Illuminate\Support\Facades\Broadcast;
use Illuminate\Support\ServiceProvider;

class BroadcastServiceProvider extends ServiceProvider
{
    public function boot()
    {
        // Broadcast::routes();
        Broadcast::routes(['middleware' => ['auth:sanctum']]);

        require base_path('routes/channels.php');
    }
}

#channels.php

<?php

use Illuminate\Support\Facades\Broadcast;

Broadcast::channel('hello.user.{id}', function ($user, $id) {
    return (int) $user->id === (int) $id;
});

#laravel env
#.env
PUSHER_APP_ID=local
PUSHER_APP_KEY=local
PUSHER_APP_SECRET=local
PUSHER_APP_CLUSTER=ap1

#broadcasting.php

<?php

return [

'default' => env('BROADCAST_DRIVER', 'pusher'),

'connections' => [

        'pusher' => [
            'driver' => 'pusher',
            'key' => env('PUSHER_APP_KEY'),
            'secret' => env('PUSHER_APP_SECRET'),
            'app_id' => env('PUSHER_APP_ID'),
            'options' => [
                'cluster' => env('PUSHER_APP_CLUSTER'),
                'useTLS' => true,
                'encrypted' => true,
                'host' => 'localhost',
                'port' => 6001,
                'scheme' => 'http'
            ],
        ],
    ],
];

#I run the event using tinker
php artisan tinker
event(new App\Events\Hello())

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

@driesvints Thanks for the suggestions


rustkrazy: inbound traffic causes errors, ppp2tun panics

might be MTU related

Will fix this when making the connection more robust and self reconnecting.


feat(android): font variation settings

Summary
This PR adds font variation settings feature for variable fonts in react native. The fontVariationSettings property provides low-level control over variable font characteristics, by specifying the four letter axis names of the characteristics you want to vary, along with their values.
Registered axes are the most commonly encountered — common enough that the authors of the specification felt they were worth standardizing. Note that this doesn't mean that the author has to include all of these in their font.
Custom axes can be anything the font designer wants to vary in their font, for example ascender or descender heights, the size of serifs, or anything else they can imagine. Any axis can be used as long as it is given a unique 4-character axis. Some will end up becoming more common, and may even become registered over time.
Example
<Text
  style={{
    fontFamily: 'MaterialSymbolsSharp',
    fontVariationSettings: "'wght' 100, 'FILL' 0",
  }}>
  {String.fromCodePoint(59576)}
</Text>

Font characteristics set using fontVariationSettings will always override those set using the corresponding basic font properties, e.g. fontWeight regardless of the order.
References
Android Documentation
iOS Documentation
MDN Documentation
Specification
Screenshots

Paper
Fabric

Changelog
[ANDROID] [ADDED] - Add fontVariationSettings support for variable font in Android
Test Plan

[x] Ensure functionality on Paper (old architecture)
[x] Ensure functionality on Fabric (new architecture)
[x] Test custom axes for specific font families
[x] Verify that registered axes ('wght', 'wdth', 'slnt', 'ital', 'opsz') work if supported by the font

Not sure why the bot still says changelog is missing :)
https://regex101.com/r/hcLY01/1

I updated some stuff regarding the edit text for paper but unfortunetaly it's not working. What am I missing here? @NickGerleman

I updated some stuff regarding the edit text for paper but unfortunetaly it's not working. What am I missing here? @NickGerleman

Hard to tell without debugging. Does prop setter for ShadowNode get hit?

I debugged it last week with a debugger and it was being hit. I will do some more testing.

I am gonna open another PR hopefully or fix the current one.


feat: native assets max swaps

Description

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #5961   👈

This stack of pull requests is managed by Graphite.


Feature documentation: Implement the eth_accounts RPC

Implement the eth_accounts RPC in web3signer. This should return all Ethereum accounts associated with the secp256k1 keys that are loaded in web3signer.
See: https://github.com/ConsenSys/web3signer/issues/765
We can probably reuse the Ethsigner documentation.

fixed by https://github.com/Consensys/doc.web3signer/pull/173


Lack of all causes easily avoidable warnings

PyInstaller's warnings (found at./build/*/warn-*.txt where * is the name of your packaged script) :
missing module named pyglet.window.BaseWindow - imported by pyglet.window (top-level), pyglet.window.headless (top-level), pyglet.window.cocoa (top-level), pyglet.window.win32 (top-level), pyglet.window.xlib (top-level)

Inspecting the code made it clear that this issue was caused by the lack of __all__s in the aforementionned packages' __init__.pys

Closing as per the recent merge. Thanks!


IPROTO-146 Adapters for java.time classes

https://issues.redhat.com/browse/IPROTO-146

Rebased

Thanks @tristantarrant


Split initial setup and day-to-day startup into separate scripts

Will do soon.

You should keep it all in one script just rename it from start.sh to seds.sh


Use only Spring, no Java EE!

Hi
@martin-g Thanks for this got me in the right direction meant to close this out some time ago. As I was trying to use JEE.
@UlrichKnaack Thanks for interest if you need this, please feel free to create a new PR so that I can create a new branch.


UUID is not correctly overridden

Version
1.25.0
What happened?
uuid is only overridden for the primary keys, but not for the foreign keys.
Relevant log output
type Event struct {
	ID          uuid.UUID
	OrganizerID pgtype.UUID
	Name        string
	Description string
	Status      string
	ImgUrl      string
	Url         string
	CreatedAt   pgtype.Timestamptz
	UpdatedAt   pgtype.Timestamptz
}

type Organizer struct {
	ID        uuid.UUID
	Name      string
	ImgUrl    string
	Url       string
	CreatedAt pgtype.Timestamptz
	UpdatedAt pgtype.Timestamptz
}

Database schema
CREATE TABLE organizer
(
    id         uuid                     DEFAULT gen_random_uuid() PRIMARY KEY,
    name       VARCHAR(100)                           NOT NULL,
    img_url    TEXT                                   NOT NULL,
    url        TEXT                                   NOT NULL,
    created_at TIMESTAMP with time zone DEFAULT now() NOT NULL,
    updated_at TIMESTAMP with time zone DEFAULT now() NOT NULL
);

CREATE TABLE event
(
    id           uuid                     DEFAULT gen_random_uuid() PRIMARY KEY,
    organizer_id uuid REFERENCES organizer (id),
    name         VARCHAR(100)                           NOT NULL,
    description  TEXT                                   NOT NULL,
    status       VARCHAR(100)                           NOT NULL,
    img_url      TEXT                                   NOT NULL,
    url          TEXT                                   NOT NULL,
    created_at   TIMESTAMP with time zone DEFAULT now() NOT NULL,
    updated_at   TIMESTAMP with time zone DEFAULT now() NOT NULL
);

SQL queries
-- name: GetEventById :one
SELECT * FROM event WHERE id = $1;

-- name: GetOrganizerById :many
SELECT * FROM organizer WHERE id = $1;

-- name: ListEventsByOrganizer :many
SELECT * FROM event WHERE organizer_id = $1;

Configuration
version: "2"
sql:
  - engine: "postgresql"
    queries: "db/queries.sql"
    schema: "db/schema.sql"
    gen:
      go:
        package: "repository"
        out: "internal/repository"
        sql_package: "pgx/v5"
        # https://github.com/sqlc-dev/sqlc/issues/3005
        overrides:
          - db_type: "uuid"
            go_type:
              import: "github.com/google/uuid"
              type: "UUID"

Playground URL
No response
What operating system are you using?
macOS
What database engines are you using?
PostgreSQL
What type of code are you generating?
Go

It is quite blocking, is there anyone that can help?
@kyleconroy @andrewmbenton @victoraugustolls

Taking a look and can hopefully have a fix up soon.

It's a bit confusing, but the issue is that your foreign key field can be null so you'll need an additional override for nullable uuids.

Note that a single db_type override configuration applies to either nullable or non-nullable columns, but not both. If you want the same Go type to override in both cases, you’ll need to configure two overrides.

Here's a playground link showing how to make it work: https://play.sqlc.dev/p/1f1d7940f4143a4635f7a8afc805416449ed7c7daf65d93b4f39989d64d24976

This did resolve my issue too however, I have a few questions. When using uuid.UUID over pgtype.UUID, how does inserting an actual NULL value in that column work. Does a zero value uuid.UUID map to a NULL value when writing to the database or is it otherwise possible to use a pointer to a uuid.UUID to distinguish between zero value uuid.UUIDs from a NULL value.


Prep for fastapi bump to ^0.89.1

Describe what the PR does:
Per https://github.com/tiangolo/fastapi/issues/5859#issuecomment-1377632161, fastapi ^0.89.1 has a new feature to determine a route's response type via method signature. This will currently fail in our codebase. This PR fixes the issue so that the bump can proceed.
Does this fix a specific issue?
N/A
Checklist:

[ ] Confirm that one or more new tests are written for the new functionality.
[x] Run tests and ensure everything passes (with 100% test coverage).
[ ] Update README.md with any new documentation.

Looks like we'll need to adjust the bump PR directly.


[Question] how to resume vim option of path

When I want to use gf to goto filename under cursor, it can't take me to the file which is in then same directory of current file.
And I found path option don't inclue . which is necessary to find files in the same directory.
" set path?
  path=~/.config/nvim,~/.vim,~/.vim/pack/jetpack/opt/nerdtree,/usr/share/vim/vimfiles,~/.vim/pack/jetpack/opt/_,/usr/share/vim/vim82,~/.vim/pack/jetpack/opt/_/after,/usr/share/vim/vimfiles/after,~/.vim/after

In the image below, neither startify nor startify.vim work properly.

The problem is I even can't set path& in ~/.vim/plugin/a.vim!

Try autocmd VimEnter * let &path .= ','.getcwd(). Cheers!

Thx, this works for my case:
autocmd VimEnter * let &path .= ',.'

May i ask how jetpack change the path? Because I didn't see anywhere path was changed in the source code

I surely say vim-jetpack does not change path. For the proof, we can see that path contains . using the following example.
call jetpack#begin()
call jetpack#add('michaelb/do-nothing.vim')
call jetpack#end()
echomsg &path

Would you mind to show the minimal settings to reproduce your problem?

Thanks for your information! It was caused by vim-scriptease :
https://github.com/tpope/vim-scriptease/blob/710fe7c2f3c636fae0b5cac041ff8da028a0777f/plugin/scriptease.vim#L117-L129
which calls this:
https://github.com/tpope/vim-scriptease/blob/710fe7c2f3c636fae0b5cac041ff8da028a0777f/autoload/scriptease.vim#L788-L790

Ah! It's nice to inspect the reason. Tim's plugins are mostly fantastic, and however they oftenly breaks the Vim's default behaviour like this. Anyway, cheers!


MSTest framework is not localized

Describe the bug
Change culture using for example DOTNET_CLI_UI_LANGUAGE and then run some failing assertion (e.g. Assert.Equal(0, 1)) through dotnet test. The UI will display assertion failure in English.
Steps To Reproduce

Change culture using for example DOTNET_CLI_UI_LANGUAGE="fr-fr"
Add some failing assertion (e.g. Assert.Equal(0, 1))
Run test (e.g. dotnet test.)
The UI will display assertion failure in English and not in the culture specified earlier

AB#1684580

Fixed by #1397


www-authenticate header?

We have a client who is using the Java proxy in a portal 10.4.1 (federated with Server) implementation.
They have created a simple app with the Developer Edition of Web AppBuilder which has been registered with the Portal, AppID included, setup the Java Proxy with the secret, the app is accessing Web Map & content in the portal  that has been shared to Everyone (so to not breach the terms of use of App Logins).  The proxy is behaving as expected, and requests via a browser work as expected.
The app is being hosted in a separate domain to the portal - hosted in Pega.  The app communicates to the internal portal through Netscaler and this is where the proxy then fails to work.
Citrix ruled that they need the requests from the proxy to include the WWW-Authenticate header.   I have checked on my own deployment and can see that this indeed is missing when making requests through the proxy (screenshot is actually the .NET proxy)
Q1. Any particular security reason why we do not include this header?
Q2. Any ideas on what would need to be modified in the proxy to enable this?
Q3. Should the header be: WWW-Authenticate: Basic

Q1. Any particular security reason why we do not include this header?

@jakc - It's most likely a bug in the proxy. For java proxy, I thought it was fixed as part of #164 but maybe not completely.  We also have open issue #175 about forwarding request headers (with an open PR for DotNet).
"Proxies MUST be completely transparent regarding user agent authentication by origin servers. That is, they must forward the WWW-Authenticate and Authorization headers untouched..." (from http://www.ietf.org/rfc/rfc2617.txt)


3448 ERROR: Assembly x86_Microsoft.VC90.CRT_1fc8b3b9a1e18e3b_9.0.21022.8_none not found

hello, when I run ./build-windows-binary.sh, it shows error:

3447 INFO: Searching for assembly x86_Microsoft.VC90.CRT_1fc8b3b9a1e18e3b_9.0.21022.8_none ...
3448 WARNING: Assembly not found
3448 ERROR: Assembly x86_Microsoft.VC90.CRT_1fc8b3b9a1e18e3b_9.0.21022.8_none not found

how to solve this problem?
anyone can give some advises?

What OS are you using?

Closing due to inactivity.


implement core express

CI found issue in Jest with TS for Node #10. The fix is to updated node on CI to actual latest version (12)


Generate working URLs as output

So now the app displays the checksums it finds in its database. Users can copy/paste them into hoover search and find their documents. But we can already provide a clickable URL to the search results page. Let's define an environment variable, e.g. REUP_MD5_URL="https://hoover.example.org/?q=md5%3A{md5}", which can be used in Python code like url = REUP_MD5_URL.format(md5=value). Also another variable REUP_SHA1_URL.

@mgax Please have a look at https://github.com/hoover/reup/pulls


Change  class to "selectpicker".

It's probably better to split the pull request, if there's javascript code in the page, there are scenario's where you can't just replace the type.

Ok.


MHP-3683: Remove MissionHub references

None of this config happened in the PR but I also renamed the repo, made master and staging the default branches again, and reconfigured Amplify to host the new repo and branches. I also stopped hosting anything related to MissionHub on Amplify since the main domains have all been moved to Wordpress.
I'll try and extract all the React stuff to a new repo in another PR so there are still a few MissionHub strings in the repo until that's completed.
This is probably easier to review by commit. The module rename touched a lot of files.

@TheNoodleMoose Can you review this at some point? Not urgent just wanted to remind you (and maybe email notifications are weird with the new repo name?).


Dropdown button icon not showing up

I copied example from the readme (including both datatemplate and styles https://github.com/RWS/Multiselect-ComboBox#examples) and here is result, can't see the dropdown icon, what to do ??

never mind it was beause the combobox was in a grid and was not allowed enough vertical space. I increased Grid.Row height and the button shows up.


Align lesson count

Currently the lesson count alignment is a little off relative to the "Lessons" text:

I would like to work on this!

All yours @medhavi11 👍

Closed by #85


Feature request: change svg fill to currentColor optionally

It will be nice if script could optionally change svg fill to 'currentColor', so it will be possible to change svg color easy with css styles.

I'm sorry but it's definitely out of svg-sprite's scope to alter the SVG files, at least on a builtin support level. However, you have several options for this:

Prepare your SVG files before processing them with svg-sprite.
Use one of the many available SVG packages out there to pre- or post-process the SVG files.
Use svg-sprite's shape or post-processing support for writing code doing what you desire.

Cheers, Joschi

Yep, thats clear now.
Thanks for expanded answer. Nice day to you.


add writeParams() method

This is untested, because I currently don't have PySCIPOpt installed on my system.  Could you check whether this works and merge?

Codecov Report

Merging #87 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #87   +/-   ##
=======================================
  Coverage   71.23%   71.23%
=======================================
  Files          14       14
  Lines         977      977
=======================================
  Hits          696      696
  Misses        281      281

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 7d7ca49...2af6631. Read the comment docs.


Syntax check shortcoming... +++

First, it's very nice you have brought an IntelliJ plugin for Firebase security rules to existence. Thank You!
The real benefit from such would be in complex rules, highlighting rule syntax problems. Unfortunately, this is where my 2s test already failed:
      allow create: if isSignedIn() +++

The plugin does not mark the (added) +++ as a breach of syntax. Surely, it is...
For being able to truly put one's weight on such a plugin, would it be possible to bring Firebase authors in the loop, to help make it more aligned with the real rules language.

Thanks for testing, issue will be fixed on version: 2.0.0


Bot  crashes after restart using followPath : UnicodeDecodeError: 'utf8' codec can't decode byte 0x9c in position 1: invalid start byte

Expected Behavior
The bot should resume were it was .
Actual Behavior
Crashs
Your FULL config.json (remove your username, password, gmapkey and any other private info)
http://pastie.org/private/g9hdck1e2usx5hviyvtq
Output when issue occurred
(PokemonGo-Bot) administrators-Mac-mini:PokemonGo-Bot administrator$ python pokecli.py --config ./configs/config.###.json
2016-08-21 02:43:59,786 [       cli] [INFO] PokemonGO Bot v1.0
2016-08-21 02:43:59,795 [       cli] [INFO] commit: 6f9ae7dc06c6518d2cc8db13ad3dc2dec167e22a
2016-08-21 02:43:59,804 [       cli] [INFO] Configuration initialized
2016-08-21 02:43:59,804 [pokemongo_bot.health_record.bot_event] [INFO] Health check is enabled. For more information:
2016-08-21 02:43:59,804 [pokemongo_bot.health_record.bot_event] [INFO] https://github.com/PokemonGoF/PokemonGo-Bot/tree/dev#analytics
2016-08-21 02:44:00,220 [requests.packages.urllib3.connectionpool] [INFO] Starting new HTTP connection (1): www.google-analytics.com
2016-08-21 02:44:00,382 [PokemonGoBot] [INFO] [set_start_location] Setting start location.
2016-08-21 02:44:00,383 [PokemonGoBot] [INFO] [login_started] Login procedure started.
2016-08-21 02:44:01,733 [PokemonGoBot] [INFO] [login_successful] Login successful.
2016-08-21 02:44:01,733 [PokemonGoBot] [INFO] Found encrypt.so! Platform: darwin encrypt.so directory: /Users/administrator/Documents/PokemonGo-Bot
2016-08-21 02:44:01,733 [PokemonGoBot] [INFO]
2016-08-21 02:44:02,622 [PokemonGoBot] [INFO] Level: 24 (Next Level: 24255 XP) (Total: 685745 XP)
2016-08-21 02:44:02,623 [PokemonGoBot] [INFO] Pokemon Captured: 2277 | Pokestops Visited: 4573
2016-08-21 02:44:03,773 [PokemonGoBot] [INFO]
2016-08-21 02:44:03,773 [PokemonGoBot] [INFO] --- ###### ---
2016-08-21 02:44:03,843 [PokemonGoBot] [INFO] Pokemon Bag: 185/250
2016-08-21 02:44:03,844 [PokemonGoBot] [INFO] Items: 319/350
2016-08-21 02:44:03,844 [PokemonGoBot] [INFO] Stardust: 287799 | Pokecoins: 0
2016-08-21 02:44:03,845 [PokemonGoBot] [INFO] PokeBalls: 35 | GreatBalls: 34 | UltraBalls: 74 | MasterBalls: 0
2016-08-21 02:44:03,845 [PokemonGoBot] [INFO] RazzBerries: 61 | BlukBerries: 0 | NanabBerries: 0
2016-08-21 02:44:03,845 [PokemonGoBot] [INFO] LuckyEgg: 5 | Incubator: 0 | TroyDisk: 5
2016-08-21 02:44:03,845 [PokemonGoBot] [INFO] Potion: 17 | SuperPotion: 13 | HyperPotion: 26 | MaxPotion: 0
2016-08-21 02:44:03,845 [PokemonGoBot] [INFO] Incense: 8 | IncenseSpicy: 0 | IncenseCool: 0
2016-08-21 02:44:03,845 [PokemonGoBot] [INFO] Revive: 39 | MaxRevive: 0
2016-08-21 02:44:03,846 [PokemonGoBot] [INFO]
2016-08-21 02:44:03,849 [SleepSchedule] [INFO] [next_sleep] Next sleep at 2016-08-21 23:15:18.849715
2016-08-21 02:44:03,850 [RandomPause] [INFO] [next_random_pause] Next random pause at 03:28:32, for a duration of 0:01:23
2016-08-21 02:44:05,193 [PokemonGoBot] [INFO] [x] Coordinates found in passed in location, not geocoding.
2016-08-21 02:44:05,194 [PokemonGoBot] [INFO] [x] Coordinates found in passed in location, not geocoding.
2016-08-21 02:44:05,194 [PokemonGoBot] [INFO] [x] Coordinates found in passed in location, not geocoding.
2016-08-21 02:44:05,375 [       cli] [INFO]
2016-08-21 02:44:05,376 [       cli] [INFO] Ran for 0:00:05
2016-08-21 02:44:05,376 [       cli] [INFO] Total XP Earned: 0  Average: 0.00/h
2016-08-21 02:44:05,376 [       cli] [INFO] Travelled 0.00km
2016-08-21 02:44:05,376 [       cli] [INFO] Visited 0 stops
2016-08-21 02:44:05,376 [       cli] [INFO] Encountered 0 pokemon, 0 caught, 0 released, 0 evolved, 0 never seen before
2016-08-21 02:44:05,377 [       cli] [INFO] Threw 0 pokeballs
2016-08-21 02:44:05,377 [       cli] [INFO] Earned 0 Stardust
2016-08-21 02:44:05,377 [       cli] [INFO] Hatched eggs 0
2016-08-21 02:44:05,377 [       cli] [INFO]
2016-08-21 02:44:05,377 [       cli] [INFO] Highest CP Pokemon:
2016-08-21 02:44:05,377 [       cli] [INFO] Most Perfect Pokemon:
Traceback (most recent call last):
  File "pokecli.py", line 677, in <module>
    main()
  File "pokecli.py", line 105, in main
    tree = TreeConfigBuilder(bot, config.raw_tasks).build()
  File "/Users/administrator/Documents/PokemonGo-Bot/pokemongo_bot/tree_config_builder.py", line 74, in build
    instance = worker(self.bot, task_config)
  File "/Users/administrator/Documents/PokemonGo-Bot/pokemongo_bot/base_task.py", line 20, in __init__
    self.initialize()
  File "/Users/administrator/Documents/PokemonGo-Bot/pokemongo_bot/cell_workers/follow_path.py", line 22, in initialize
    self.ptr = self.find_closest_point_idx(self.points)
  File "/Users/administrator/Documents/PokemonGo-Bot/pokemongo_bot/cell_workers/follow_path.py", line 88, in find_closest_point_idx
    alt = float(point['alt'])
KeyError: 'alt'
2016-08-21 02:44:05,394 [sentry.errors] [ERROR] Sentry responded with an error: 'utf8' codec can't decode byte 0x9c in position 1: invalid start byte (url: https://app.getsentry.com/api/90254/store/)
Traceback (most recent call last):
  File "/Users/administrator/Documents/PokemonGo-Bot/lib/python2.7/site-packages/raven/transport/threaded.py", line 174, in send_sync
    super(ThreadedHTTPTransport, self).send(data, headers)
  File "/Users/administrator/Documents/PokemonGo-Bot/lib/python2.7/site-packages/raven/transport/http.py", line 47, in send
    ca_certs=self.ca_certs,
  File "/Users/administrator/Documents/PokemonGo-Bot/lib/python2.7/site-packages/raven/utils/http.py", line 66, in urlopen
    return opener.open(url, data, timeout)
  File "/Users/administrator/Documents/PokemonGo-Bot/lib/python2.7/site-packages/future/backports/urllib/request.py", line 494, in open
    response = self._open(req, data)
  File "/Users/administrator/Documents/PokemonGo-Bot/lib/python2.7/site-packages/future/backports/urllib/request.py", line 512, in _open
    '_open', req)
  File "/Users/administrator/Documents/PokemonGo-Bot/lib/python2.7/site-packages/future/backports/urllib/request.py", line 466, in _call_chain
    result = func(*args)
  File "/Users/administrator/Documents/PokemonGo-Bot/lib/python2.7/site-packages/raven/utils/http.py", line 46, in https_open
    return self.do_open(ValidHTTPSConnection, req)
  File "/Users/administrator/Documents/PokemonGo-Bot/lib/python2.7/site-packages/future/backports/urllib/request.py", line 1284, in do_open
    h.request(req.get_method(), req.selector, req.data, headers)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 1053, in request
    self._send_request(method, url, body, headers)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 1093, in _send_request
    self.endheaders(body)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 1049, in endheaders
    self._send_output(message_body)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 891, in _send_output
    msg += message_body
  File "/Users/administrator/Documents/PokemonGo-Bot/lib/python2.7/encodings/utf_8.py", line 16, in decode
    return codecs.utf_8_decode(input, errors, True)
UnicodeDecodeError: 'utf8' codec can't decode byte 0x9c in position 1: invalid start byte
2016-08-21 02:44:05,396 [sentry.errors.uncaught] [ERROR] [u"KeyError: 'alt'", u'  File "pokecli.py", line 677, in <module>', u'  File "pokecli.py", line 105, in main', u'  File "pokemongo_bot/tree_config_builder.py", line 74, in build', u'  File "pokemongo_bot/base_task.py", line 20, in __init__', u'  File "pokemongo_bot/cell_workers/follow_path.py", line 22, in initialize', u'  File "pokemongo_bot/cell_workers/follow_path.py", line 88, in find_closest_point_idx']

Steps to Reproduce

Other Information
OS:

Branch:
dev
Git Commit:
6f9ae7dc06c6518d2cc8db13ad3dc2dec167e22a
Python Version:
Python 2.7.10
Any other relevant files/configs (eg: path files)
Does not crash on cc3b4c92b94b9e6b8f23f08eae93189a82d5cfd0
Path file
[
  {"location": "48.892767, 2.387508"},
  {"location": "48.892148, 2.388629"},
  {"location": "48.892607, 2.390936"}
]

PS : I have 2 bots in the same folder, the other one seems to work fine.
Maybe I missed a step to run 2 bots in parallel

I think API responses have been changed and bot cannot decode it.

try replacing
https://github.com/PokemonGoF/PokemonGo-Bot/blob/dev/pokemongo_bot/cell_workers/follow_path.py#L111
with
if 'alt' in point:
    alt = float(point['alt'])
else:
    alt = uniform(self.bot.config.alt_min, self.bot.config.alt_max)

Had to change in find_closest_point_idx too and it works thanks !
Should I pass the alt in the locations? Maybe the doc can but updated on that
Thanks a lot
Nicolas


build.zig: fix invalid reference

computeTestTargets returns reference to a function local scope variable which goes out of scope once the function returns.
Description
When cross compiling the build panics as it accesses garbage memory.
Checklist

[ ] I added tests for my changes and they pass

Thanks !