Error in sending message

I saw this issue on solved ones but I tries using the latest commit, the commit on the solved thread and other solutions but none worked.
ERR! getUserInfo { __ar: 1,
ERR! getUserInfo   __sf: 'j0',
ERR! getUserInfo   error: 1357004,
ERR! getUserInfo   errorSummary: 'Sorry, something went wrong',
ERR! getUserInfo   errorDescription: 'Please try closing and re-opening your browser window.',
ERR! getUserInfo   payload: null,
ERR! getUserInfo   bootloadable: {},
ERR! getUserInfo   ixData: {},
ERR! getUserInfo   lid: '6443046541094488234' }

Hey @Yeonin, thanks for reporting this. Are you able to send any messages or do you immediately get this error?
Could you try with a different account to see?
Could you log into the account you're using and check that it doesn't ask you to verify anything? We've observed weird responses when running bots on machines that are in totally different places than where the account usual logins from, because FB thinks it's someone who took control of your account.
One last thing that would help: is there any way you could give us access to the account that's having that issue? We can't reproduce it with our accounts, it's likely that this only happens in your region. If you can't share the bot's account with us, maybe try making a new account and reproducing the issue with that one and sharing that one with us?
Also this might be due to sending repeated messages to a bunch of people or non-friends. (that's why trying with a different account helps).

There's an error immidiately after trying to send. And I receive a message
that my account was logged in from a different device. I was able to
receive message details like body and threadid but isnt able to send
anything. I am using the same PC that I regularly use for facebook.
On Jul 18, 2017 2:48 AM, "Benjamin San Souci"<EMAIL_ADDRESS>wrote:

Hey @Yeonin https://github.com/yeonin, thanks for reporting this. Are
you able to send any messages or do you immediately get this error?
Could you try with a different account to see?
Could you log into the account you're using and check that it doesn't ask
you to verify anything? We've observed weird responses when running bots on
machines that are in totally different places than where the account usual
logins from, because FB thinks it's someone who took control of your
account.
One last thing that would help: is there any way you could give us access
to the account that's having that issue? We can't reproduce it with our
accounts, it's likely that this only happens in your region. If you can't
share the bot's account with us, maybe try making a new account and
reproducing the issue with that one and sharing that one with us?
Also this might be due to sending repeated messages to a bunch of people
or non-friends. (that's why trying with a different account helps).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Schmavery/facebook-chat-api/issues/500#issuecomment-315846170,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APxN2z8oVPHkbqGjTO5Ip66HflREe2BVks5sO6x9gaJpZM4OZEjj
.

@Yeonin it would help to have more details.  Can we see some code that produces this issue? Can you include the contents of the error you just mentioned?  Have you tried logging into this specific account in the browser?  If you're able to go through @bsansouci 's steps one by one and let us know the results, that would be really helpful.

I have the same error when sending a message

This is one of the most generic errors and unfortunately without more information, it is basically impossible to diagnose.  Feel free to reopen when applicable.


Default doc mapping mode

Here the docs say that the mode attribute by default takes the lenient value:

However, here it says that mode is dynamic by default:

Is it a mismatch or am I missing something?

Thanks for reporting. A PR fix is on the way.
The default mode is the dynamic mode.


Show log and keep running on export failure

Which problem is this PR solving?
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #147
Short description of the change
Catch error on export failure and write a log.
How Has This Been Tested?
I confirmed that the collector in my sample code emited a warning log below by executing curl localhost:8080 3 times in a row.
$ dart run bin/server.dart
uri: http://<IP_ADDRESS>:4318/v1/traces
ok
ok
ok
WARNING: 2024-02-19 16:38:23.374773: Failed to export 3 spans.

import 'dart:io';

import 'package:logging/logging.dart';

import 'package:opentelemetry/sdk.dart' as otel_sdk;
import 'package:opentelemetry/api.dart';

Future<void> main(List<String> args) async {
  final uri = Uri.parse('http://<IP_ADDRESS>:4318/v1/traces');
  print('uri: ${uri}');
  final exporter = otel_sdk.CollectorExporter(uri);
  final processor = otel_sdk.BatchSpanProcessor(exporter);
  // Send spans to a collector every 5 seconds
  final provider = otel_sdk.TracerProviderBase(processors: [processor]);

  registerGlobalTracerProvider(provider);

  Logger.root.level = Level.ALL;
  Logger.root.onRecord.listen((record) {
    print('${record.level.name}: ${record.time}: ${record.message}');
  });

  var server = await HttpServer.bind(
    InternetAddress.anyIPv4,
    8080,
  );

  await for (HttpRequest request in server) {
    final tracer = globalTracerProvider.getTracer('instrumentation-name');
    final span = tracer.startSpan('doingWork');
    request.response.write('ok');
    print('ok');
    span.end();
    await request.response.close();
  }
}

Checklist:

[x] Unit tests have been added
[ ] Documentation has been updated

Security Insights
No security relevant content was detected by automated scans.
Action Items

Review PR for security impact; comment "security review required" if needed or unsure
Verify aviary.yaml coverage of security relevant code

Questions or Comments? Reach out on Slack: #support-infosec.

@tapih Would you please mind adding a unit test for these changes?

@tylerrinnan-wf
I added a unit test for export failure😀 Could you please take a look at https://github.com/Workiva/opentelemetry-dart/pull/148/commits/26320cbdef8e6e752fc6cd553b9a077a4d75f131 ?

RM +1


extract authz into response.bearer_token

assert(request.headers.has('authorization'));
const authorization = request.headers.get('authorization');
assert(authorization.startsWith('Bearer '));
const bearer_token = authorization.slice(7);

x hmac token
x hmac signature
also jwt


Installation failed

`composer require zizaco/entrust:5.2.x-dev
You are running composer with xdebug enabled. This has a major impact on runtime performance. See https://getcomposer.org/xdebug
./composer.json has been updated
{
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.
Problem 1
- Conclusion: don't install laravel/framework v5.2.37
- Conclusion: remove laravel/framework v5.2.36
- Conclusion: don't install laravel/framework v5.2.36
- don't install laravel/framework v5.2.36|remove illuminate/support v5.2.37
- don't install illuminate/support v5.2.37|don't install laravel/framework v5.2.36
- don't install illuminate/support v5.2.37|don't install laravel/framework v5.2.36
- Installation request for laravel/framework (locked at v5.2.36, required as 5.2.*) -> satisfiable by laravel/framework[v5.2.36].
{
- Installation request for illuminate/support (locked at v5.2.37) -> satisfiable by illuminate/support[v5.2.37], laravel/framework[v5.2.37].
Installation failed, reverting ./composer.json to its original content.`

Consider posting your composer.json file

You should try this, besides entrust is poorly maintained.


Product upgrades

After i selected an upgrade for a product i can only change the upgrade product but how i disable it?

Please do not use issues tracker for question like this.

Use click on product title while holding ctrl key on keyboard.


fix(versions): update Activiti/activiti-cloud-runtime-bundle-service versions into develop

UpdateBot pushed maven dependency: org.activiti.dependencies:activiti-dependencies to: 7.1.109

UpdateBot commands:
updatebot push-version --kind maven org.activiti.dependencies:activiti-dependencies 7.1.109 org.activiti.api:activiti-api-dependencies 7.1.54 org.activiti.core.common:activiti-core-common-dependencies 7.1.61 org.activiti.build:activiti-parent 7.1.28


Reactions flipped and dropped from enzyme constraint

Hello,
Nice package, thank you very much!
There might be an issue with how you split your reactions that causes reversible parts of reactions to drop out from the enzyme constraint.
I was looking at the bounds of the enzyme-constraint reactions and noticed that cobrapy and optlang bounds are occasionally different. It seems like the reversible reactions get flipped once more when they go into optlang and then are no longer part of the enzyme constraint, so the model can use them in the reverse direction "for free". It's possible that I missed something but you may want to check.
import sys
sys.path.append('../../../../ECMpy/code/')
from cobrapy_ec_model_function import *
json_model_path="../../../../ECMpy/model/iML1515_irr_enz_constraint_adj_round2.json"
enz_model=get_enzyme_constraint_model(json_model_path)

# Look for reactions that are part of the enzyme constraint, but have contradictory optlang and cobrapy bounds
enzyme_constraint = [c for c in enz_model.constraints if c.ub>0][0]
reacs = []
for v in enzyme_constraint.variables:
    cp_ub = float(enz_model.reactions.get_by_id(v.name).upper_bound)
    cp_lb = float(enz_model.reactions.get_by_id(v.name).lower_bound)
    if v.ub != cp_ub or v.lb != cp_lb:
        reacs.append(v.name)
        print("Variable: "+v.name)
        print("optlang bounds: "+str(v))
        print("cobrapy bounds: "+str(cp_lb)+" <= "+str(v.name)+" <= "+str(cp_ub))
        if len(reacs) >= 3:
            break

print("=== variable lookup in enzyme constraint ===")

# Look if these reactions exist somewhere in the model under a different name
for r in reacs:
    for v in enz_model.variables:
        if r in str(v):
            print(("" if v in enzyme_constraint.variables else "NOT ")+"in enzyme constraint: "+str(v))

produces
Variable: SHK3Dr_reverse
optlang bounds: 0 <= SHK3Dr_reverse <= 0.0
cobrapy bounds: -1000.0 <= SHK3Dr_reverse <= 0.0
Variable: ACOAD2f
optlang bounds: 0 <= ACOAD2f <= 0.0
cobrapy bounds: -1000.0 <= ACOAD2f <= 0.0
Variable: ACACT4r_num2
optlang bounds: 0 <= ACACT4r_num2 <= 0.0
cobrapy bounds: -1000.0 <= ACACT4r_num2 <= 0.0
=== variable lookup in enzyme constraint ===
NOT in enzyme constraint: 0 <= SHK3Dr_reverse_d5c8f <= -0.0
in enzyme constraint: 0 <= SHK3Dr_reverse <= 0.0
NOT in enzyme constraint: 0 <= SHK3Dr_reverse_reverse_f4102 <= 1000.0
in enzyme constraint: 0 <= ACOAD2f <= 0.0
NOT in enzyme constraint: 0 <= ACOAD2f_reverse_6e942 <= 1000.0
in enzyme constraint: 0 <= ACACT4r_num2 <= 0.0
NOT in enzyme constraint: 0 <= ACACT4r_num2_reverse_47058 <= 1000.0

It might be easier to avoid the issues with cobrapy and optlang altogether if you express the enzyme constraint through pseudo-reactions and pseudo-variables. That way, everything is clean in the cobrapy and optlang will figure out the necessary details in the back.
The steps after splittling the reactions would be:

Add a new metabolite (enzyme pool)
Drain the new enzyme-pool metabolite in all enzyme-driven reactions, use your coefficients for that stoichiometry
Add a pseudo reaction to produce/control the amount of available enzyme and bound that reaction with your enzyme-constraint-bound

Q1. There are occasional issues related to the disparities between cobrapy and optlang boundaries. When converting a cobrapy-formatted model to an ECMpy-formatted enzyme-constrained model, the first step involves splitting reversible reactions into two irreversible reactions. In this process, reactions with a flux range of (-1000, 1000) are considered reversible, while reactions with a flux range of (0, 1000) are treated as irreversible. However, due to the presence of reactions with a flux range of (-1000, 0) in the model, we mistakenly overlooked them, resulting in the aforementioned problem. Therefore, we have made corrections to the convert_to_irreversible(model) function, and after the code modification, this issue should no longer occur.
Q2: 2. Regarding the issue of random characters appearing in reaction names in variables: During the cobrapy calculation process, reversible reactions are split into two irreversible reactions for computation. In this splitting process, random characters (e.g., reverse_f4102) are automatically added after the reverse reaction to prevent naming conflicts. Since ECMpy models undergo the splitting of reversible reactions, the flux range for reverse reactions is (0, 0), and they are excluded from enzyme constraints. For metabolic network models, the same issue of adding random characters to reaction names can arise. Additionally, we experimented with the enzyme constraint formulation you suggested (pseudo-reactions and pseudo-variables). However, enzyme-constrained models generated using this approach also exhibit the problem of random characters in reaction names during computation. Therefore, this issue is inherent to the cobrapy model calculation mechanism rather than a problem with how you have modeled it.

The reverse reactions with random character sequences do not pose a problem. Since you split reactions into forward and reverse parts, most of the time, these reverse parts have boundaries (0,0) and are simply irrelevant. The main issue was (or is, I haven't checked, again), that there reactions whose reverse part should be limited by enzyme constraint but isn't.
The main reason I advocate for an implementation with pseudo metabolites and reactions is robustness and the improved universality and compatibility with plenty of other tools e.g., when you export sbml, when you use visualization like Escher or CNApy or compute intervention strategies with StrainDesign. The constraint-implementation that is currently used, is unique to Optlang and limited to vanilla cobrapy functions. If Optlang changes their syntax your package will break. If cobrapy uses another backend than Optlang, your package will break. You can see that it's even sensitive to how you split your reactions into forward and reverse. Hence my suggestion

Optlang constraints are conceptually and mathematically nice, but actually impractical

I checked your changes and it solved the issue! Well done and thanks :)
In case you're considering the pseudo reaction thing I encouraged you to do, take a look at pull request https://github.com/tibbdc/ECMpy/pull/5


ccrewrite adds invalid source reference to PDB file

The source for the project is at https://tomstoolbox.codeplex.com/SourceControl/latest (Feature/Default branch, Desktop module)
When "Perform Runtime Contract Checking (Preconditions)" is on, an invalid source file reference is added to the .pdb file.
The class is TomsToolbox.Desktop.DependencyObjectExtensions.DependencyPropertyEventWrapper`1, which implements TomsToolbox.Core.INotifyChanged.
Without the rewriter the pdb contains a link to the correct source file, but when enabling the rewriter the pdb contains an additional link to the source file of TomsToolbox.Core.INotifyChanged, which is in a different project!
Now the NuGet package manager can't find this
source file, since it's from another project, and the NuGet symbol package is considered invalid.

This is by design. Additional source code file comes from a contract for INotifyChanged interface (NotifyChangedContract class) which is being injected into Desktop assembly by ccrewrite.
This allows you for example to step into the contract defined for an interface method from the implementing type during debugging.
You can specify symbol package source files manually or submit a feature request to nuget project to make nuget read pdb files for list of source files needed.

Not a bug here, the problem is on NuGet side.


Automatic routing from swagger api doc

Hi,
I was wondering if the library offers any way to parse x-swagger-router-controller and operationId nodes to achieve automatic routing to the controller method when a path is hit.

This library doesn't provide any routing functionality.

Happy it doesn't - I think documentation should be independent from the controller.


[Fortune - Recording Oracle] Fix production build command

Bug description
Fix production build command, since yarn start:prod command return error: Error: Cannot find module '/opt/render/project/src/packages/apps/fortune/recording-oracle/dist/main'

It was fixed with https://github.com/humanprotocol/human-protocol/pull/2539


Kafka AdminClient Wrapper?

Would there be any interest in some thin wrapper around the AdminClient? I'm working on some functions in a project at work and could make them nice and maybe create a PR. Any thoughts?

Yes please!

Ok. Will take some days. Will create an initial PR sometime next week.


[1.12] Bump Test Utils to get pods cleaned up by marathon.purge

High-level description
This is a backport of https://github.com/dcos/dcos/pull/3833.
The description of that PR is:
This bumps dcos-test-utils for a better cleanup of the Marathon state between tests in the DC/OS Integration test suite.

Specifically, this brings this patch: https://github.com/dcos/dcos-test-utils/pull/69/files

To be sure<EMAIL_ADDRESS>
As this was
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS_OSS- Foo the Bar so it stops Bazzing.

Related tickets (optional)
Other tickets related to this change:

DCOS_OSS- Foo the Bar so it stops Bazzing.

Checklist for all PRs

[ ] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change:
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[ ] Read the DC/OS contributing guidelines
[ ] Followed relevant code rules Rules for Packages and Systemd

@mesosphere-mergebot bump-ee

This addresses instability, fast-merging.

@mesosphere-mergebot bump-ee


fix: strict dependencies for textproto indexer

CI died trying to download files, merging anyways
"""
(17:09:08) ERROR: An error occurred during the fetch of repository 'com_googlesource_code_re2':
| java.io.IOException: Error downloading [https://github.com/google/re2/archive/2019-01-01.zip] to /var/lib/buildkite-agent/.cache/bazel/_bazel_buildkite-agent/ec321eb2cc2d0f8f91b676b6d4c66c29/external/com_googlesource_code_re2/2019-01-01.zip: Read timed out"""

CI died trying to download files, merging anyways
"""
(17:09:08) ERROR: An error occurred during the fetch of repository 'com_googlesource_code_re2':
  | java.io.IOException: Error downloading [https://github.com/google/re2/archive/2019-01-01.zip] to /var/lib/buildkite-agent/.cache/bazel/_bazel_buildkite-agent/ec321eb2cc2d0f8f91b676b6d4c66c29/external/com_googlesource_code_re2/2019-01-01.zip: Read timed out"""

Generally when this happens you can just retry the build rather than "force pushing".


EPI datasets, not getting published results

Hello,
Thank you for posting the GUE+ datasets. Unfortunately, I am not getting the same results as your published results when I run DNABERT2 on them. I believe in the supplemental, you state that you reported MCC for the EPI dataset.  (also, I am not 100% sure of the mapping between your reported EPI datasets of 0-1 and the listed datasets in the GUE+ data (GM12878, HUVEC, HeLa-S3, IMR90, K562, NKEK)
Our results:
DNABERT2 on GM12878
acc: 0.6845, f1: 0.6806000467204687, mcc: 0.3622498911964387
you reported MCC: 61.91 for dataset 0
DNABERT2 on HeLa-S3
acc: 0.8995, f1: 0.8994577471182263, mcc: 0.7997480886818011
you reported MCC: 79.19 for dataset 1
DNABERT2 on K562
acc: 0.7145, f1: 0.7066583167889823, mcc: 0.43058726256329627
you reported MCC: 83.50 for dataset
Can you share the hyperparameters that you used for these results, or is it possible that they are accuracy results and not mcc?
Thank you for your help.
LeAnn

The reported results are mcc. You can see the following figure for some of our validation results during training.

For the hyperparameters, we used 2e-5 lr and 32 batch sizes.

Thank you for your response. I will try again with these hyperparameters

Can you clarify which datasets correspond with 0, 1, 2, 3, 4, 5?


Closed #37

Codecov Report

Merging #38 (b0d3536) into master (e895d06) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #38      +/-   ##
==========================================
- Coverage   81.66%   81.61%   -0.05%
==========================================
  Files          26       26
  Lines        1298     1300       +2
==========================================
+ Hits         1060     1061       +1
- Misses        238      239       +1

Flag
Coverage Δ

unittests
81.61% <100.00%> (-0.05%)
:arrow_down:

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

Impacted Files
Coverage Δ

indexpy/routing/routes.py
82.15% <100.00%> (-0.31%)
:arrow_down:

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 cc8538a...b0d3536. Read the comment docs.


Docker compose version warning on -desktop release

Since a few updates ago, I am getting this warning in the terminal every time any command is used. I think it's because the installed version doesn't meet the SemVer standard in this case. Is there any way I can avoid this message?

REWARD INFO
INFO    reward version: 0.6.13+202410181344
INFO    GOOS: linux
INFO    GOARCH: amd64
DOCKER INFO
Docker Compose version v2.29.7-desktop.1
Client:
Version:           27.3.1
API version:       1.47
Go version:        go1.22.7
Git commit:        ce12230
Built:             Fri Sep 20 11:39:44 2024
OS/Arch:           linux/amd64
Context:           default
Server: Docker Desktop  ()
Engine:
Version:          27.3.1
API version:      1.47 (minimum version 1.24)
Go version:       go1.22.7
Git commit:       41ca978
Built:            Fri Sep 20 11:41:11 2024
OS/Arch:          linux/amd64
Experimental:     false
containerd:
Version:          1.7.21
GitCommit:        472731909fa34bd7bc9c087e4c27943f9835f111
runc:
Version:          1.1.13
GitCommit:        v1.1.13-0-g58aa920
docker-init:
Version:          0.19.0
GitCommit:        de40ad0

Fixed in v0.6.14


Add GitHub Binary Release Pipeline for RustPython

Add Release Pipeline for RustPython. Pipeline is currently configured to cut a new release on every merge to main.
Sample Release: https://github.com/theshubhamp/RustPython/releases/tag/2024-12-07-gh-rel-rustpython-11

Sample Build: https://github.com/theshubhamp/RustPython/actions/runs/12212320619

Validated to run on Linux / Fedora:

Issue #5440

Thanks for your review @youknowone! I have pushed some new changes. PTAL

@youknowone could you re-review when you get some time? Thanks!

@youknowone thank you for the review. Could you recheck ?


SO_REUSEADDR may be a typo

The README.md says,

KeyDB currently relies on SO_REUSEADDR's load balancing behavior which is available only in Linux

I think it should be SO_REUSEPORT here.

You are correct. That is a typo.

Fixed.


BC-34 # deploy --only flag for selective deployment

Adds

BC-34: deploy --only flag for selective deployment (#70, @jokeyrhyme), e.g.

bm bmp deploy --only answerSpace.* only the top-level

bm bmp deploy --only interactions/**/*.html only interactions related to HTML files

bm bmp deploy --only interactions/name/**/* only "name" interaction

note: this will still deploy a whole resource at a minimum, not the part related to the specified files

note: if you get errors, check that your wildcards actually match expected files

Code Review Notes

quite a bit of refactoring here, so it's recommended that you review commit-by-commit

Looks Good, +1, Is it worth having to get someone else just a quick look at it. Just because it a big change and I might have missed something...

lgtm too, tests seem to cover all the bases

Thanks, @ashishtilara and @simonmarklar  :)


Refine words and add more social media links in README

@PragmaTwice I'm thinking of hosting all contact info in https://kvrocks.apache.org/community/ only and set links from the README on demand.

@PragmaTwice I'm thinking of hosting all contact info in https://kvrocks.apache.org/community/ only and set links from the README on demand.

Great.


Update and add new Ravn logo

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes: #177
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.

Update to use new logo for RAVN Alaska
Add RAVN Alaska logomark

Type of change:

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[ ] My update follows the CONTRIBUTING guidelines of this project
[ ] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


E,R,T,P control

i actually made and extension to play in the privete server..and there is an issue when i use the ERTP control script.
-when i'm using the script in my extension :

-and when i'm not :

Because ERTP control send direct the key with WebSocket directly.

@pabloalber84 and how to fix that???

privete server??? whaat????

@ArixProject  yes...extension of agarioV786 for privete server..cz the original agario is so laggy and ugly(i don't like it)...would you like to help me plz??

for f**k sake ..no one helps???

@FantasyIsBae

Use older client


Cannot view entire value in "Message Custom Properties"

Sometimes we dead letter messages ourselves due to an exception.  We include the exception detail information in the dead letter message as a custom property but we can't view it in the "Message Custom Properties" section after peeking.  This would be a great enhancement for our troubleshooting efforts using SBE.

Hi Tom
This sounds more like a bug than an improvement :) The codebase of the control used to show deadletter messages should be the same of the control used to show normal messages. I'll check when I have some time. I'm pretty busy at the moment, I'll do my best to look at it... or maybe @SeanFeldman can fix that :)
Ciao
Paolo

@tdoman are you using
BrokeredMessage.DeadLetterAsync(string deadLetterReason, string deadLetterErrorDescription)
or
BrokeredMessage.DeadLetterAsync(IDictionary<string, object> propertiesToModify)
Let me confirm if I understand what you do. Assuming you're using the (string, string) version, you don't see deadLetterReason customer header with deadLetterErrorDescription message? If that's not the case, please elaborate on steps to reproduce. Thank you.

@tdoman I quickly ran the test with both API calls, and both work fine with the latest (3.4.0) SBE.
Using BrokeredMessage.DeadLetterAsync(IDictionary<string, object> propertiesToModify)

Using BrokeredMessage.DeadLetterAsync(string deadLetterReason, string deadLetterErrorDescription)

@SeanFeldman Yep, that's exactly what we're doing but I should have been clearer.  When we get an exception, we are passing ex.ToString() in as the DeadLetterErrorDescription.  So, at the very least, we'd like to be able to right-click and copy the value data to an editor or, at best, have the ability to view the entire custom property value within the SBE.

@tdoman got it. You can already do that. Select the message, double click it for "Edit and Resubmit" (not quite accurate, but still), select the row/column you need to copy, ctrl-c et voila! Attached a small visualization. Is this what you wanted?

@SeanFeldman yep, that's it, fantastic!  that's what we need to inspect those stack traces we put in there (hopefully they never exceed the max message size) AND I didn't realize that "edit and resubmit" feature was even in the SBE.  that's useful in it's own right, thanks a lot, i appreciate the prompt and helpful feedback!

@tdoman :)
One word of caution with exceptions and headers - you only can use 64K for headers, not the full message size. As long as you guessed against that, you should be good.
@paolosalvatori good to close this issue

Hi Tom
I used to be in the BizTalk Customer Advisory Team so I tried to implement the Message Repair and Resubmit pattern that was available in BizTalk ESB. That's why you can right click a single message, edit and resubmit it. As you probably realized by right clicking a message, you can also save the message to a text file in JSON format, including custom properties. Last but not least, you can also select multiple messages and resubmit them in a batch mode or save them to a text file as a JSON array. Hope this helps :)
Ciao
Paolo


Replace uncle with ommer in spec

ommer is the preferred terminology.

Might be worth putting up an EthCatHerders bounty for this? cc: @poojaranjan

To share with the ECH engineering team or put out a bounty, it will be helpful to get details of the task with link(s) where change is requested.

Not sure if there were other locations, but I added a list of places inside the Python spec itself.
Think it's worth a bounty?

Should take 10 minutes or less, not sure it's worth a bounty.

Does this method name change as well?
https://github.com/ethereum/execution-specs/blob/c5b6721123267e214e97f6c038ddbe7c60894866/src/ethereum_spec_tools/sync.py#L325


getXSSFColor returns null

I'm not sure how to reproduce this, but I have a spreadsheet I'm reading in and adding data to.
When attempting to do any formatting to the cells, I'm getting an error due to getXSSFColor returning null in the setColor call of the cloneFont function (around line 2994).
A workaround for me is simply to check that a colour is returned first and use a default if not. Happy to put this into a pull request if you're ok with it - I'm afraid I've no idea what causes the issue in the first place.
var color = isXmlFormat( arguments.workbook )? arguments.fontToClone.getXSSFColor(): arguments.fontToClone.getColor();
newFont.setColor( isDefined('color') ? color : fontToClone.DEFAULT_FONT_COLOR );

Shame we can't identify the cause, Gary, but I've implemented a similar fix. Can you check it with the latest develop branch?

Thanks yes, that seems to work for me... though the dev version seems significantly different to the version I'm running so I must be on an old version!

Yes, I assumed you were on quite an old version from the line number you gave. The version property should be right at the top of Spreadsheet.cfc
I'll knock out a new release which includes this fix in the next few days all being well.


Bug fix to ensure shift, control, alt, and caps lock keys do not affe…

…ct autocomplete.
ASCODCP-300

Hi Dylan, Greg,
Could you review my change to ignore the Shift key in autocomplete (as well as Ctrl, Alt, and Caps Lock) ?
Thanks,
Mike

Does what it needs to by inspection, pulling it now to test.

Works, I'm merging


Centralize Public properties

While we should still use pick, we should set these in the config (with typesafety!) and have a method to convertToPublicEntity or something
https://github.com/plutomi/plutomi/blob/main/Controllers/PublicInfo/getOpening.ts

no longer relevant


[Bug]: Margin Issue in Navbar.

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
There is issue with margin and padding in navbar.
Add Screenshots / Video Clip
Before

After(Inspect Mode)

What browsers are you seeing the problem on?
Google Chrome
Agreements

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC contributor
[X] I'm a Hacktoberfest contributor
[X] I want to work on this issue [Optional]
[X] I'm willing to provide further clarification or assistance if needed.

@Tejashri-Taral Can you please assign me this under GSSoC 24 Ext?


Union conflict?

used release 1.0 files +GCC 12.1.0 x86_64-pc-linux-gnu+ vscode on EndeavourOS Linux x86_64:
Trying to include this lib from folder in project with cmake via:
include_directories(${PROJECT_NAME} INTERFACE include)
link_directories(${PROJECT_NAME} PUBLIC include)
And including it in main.cpp with #include <mutil.h> brings up this on build

am I doing something wrong?

something like this, i guess they dont like the b letter being repeated

Fixed in commit 6cf8e3ace40ddd9864cbcfd8c06329482b107091.
Caused by the union member conflict, and did not arise while compiling with MSVC.


build TT files during compile phase

Currently, TT files used by ncm-metaconfig services are built during the test phase which is not compliant with what the test file is supposed to do... it should not produce anything required by other phases, just validate what has been done in the previous phases (mainly compile).
We need to improve the build tools to process the TT files during the compile phase.

Not going to happen for 15.4, might make it for 15.6.

This was the cause of quattor/release#95.

Fix this year, is annoying but not high priority.


Cards

Cards can be added to your board to track the progress of issues and pull requests. You can also add note cards, like this one!

Closed

@dependabot merge


[BUG] Badly-constructed GET request fired at an useForm / useModalForm with no need (and gets 404/500)

Describe the bug
When using useForm or useModalForm on resource A in the Show context of resource B - a GET request is being performed to resource A with the ID of shown resource B.
To Reproduce
Steps to reproduce the behavior:

have two resources Person and Car
Create a ShowPerson page with useModalForm to create a Car for that Person:  (note that all usage of the hook results is commented out, and this still happens)

export const PersonShow = () => {
  const { queryResult } = useShow();
  const { data, isLoading } = queryResult;
  const record: any = data?.data;
  const { list } = useNavigation();

  const { modalProps, formProps, show } = useModalForm({
    action: 'create',
    resource: 'cars',
    defaultFormValues: { person_id: record?.id },
  });

  return (
    <>
      <Show isLoading={isLoading}>
        <Title level={5}>Person Name</Title>
        <Text>{record?.person.name}</Text>

        <Title level={5}>ID</Title>
        <Text>
          <Tag>{record?.id}</Tag>
        </Text>

        <Title level={5}>Actions</Title>
        <ButtonToolbar>
          <DeleteButton
            onSuccess={() => {
              list('persons');
            }}
          />
          {/* <Button onClick={() => show()}>Create Car</Button> */}
        </ButtonToolbar>
      </Show>
      {/* <Modal {...modalProps}>
        <Form {...formProps} layout="vertical">
          <Form.Item label="Person ID" name="person_id">
            <Input disabled/>
          </Form.Item>
          <Form.Item label="Car Model" name="model">
            <Input/>
          </Form.Item>
        </Form>
      </Modal> */}
    </>
  );
};

go to that Show page
See 404 popup (and 404 in browser network tab) - the request is done to /api/cars/the_person_id

Expected behavior
No request should be made for that hook...since this is a create form (with no useOptions etc...")
At most the request should be done to /api/persons/the_person_id
Desktop:

OS: MacOS 11.6 (Big Sur)
Browser: chrome
Version 95.0.4638.69
Used dataProvider refine-simple-rest

Hey @banuni ,
I have good and bad news for you.
Good;
We identified the source of the issue.
Bad,
The problem has multiple possible solutions. We should discuss this with the core team. Although I cannot give an exact date, I think it will be resolved within this week

Amazing! For now this doesn't block me since the only  sode effect is a popup of a 404 on the UI. Thanks for the response :)

Hey @banuni ,
We've released 2.2.2. Could you approve this issue is fixed?

Hey @omeraplak , sorry for the delayed response, it is indeed fixed! thank you!!!


all function smoothly, but i got 1 problem. password did't update into database, but username,email, and active is update normally. please help me.

this is my problem,i think maybe some error in hashed code in index.php or password.php

i upload it on server online. anything else work fine, email also work smoothly. Thanks.

Hi what version of php are you running this on?
The functions for hashing only work on php 5.3 and up. You really should not be using anything less then 5.5 now the rest are deprecated and no longer supported by php.


Make sure that the user specified path additions take precedence over things in the user's path by default

Seen on php: "path_additions": ["/usr/local/php5/bin"],
Yet, the php which was already in my path took precedence over that one. So I had to fully specify the php path elsewhere:
"binary": "/usr/local/php5/bin/php",

fixed by #10


Feature to provide custom error messages

Would be nice if there's a way to provide custom error messages when validating structs - https://github.com/asaskevich/govalidator#validatestruct-2

I implemented custom error message by forking this code. Please take a look at my github account.
https://github.com/chatwithrajaram/govalidator
You can configure custom error message with below format.
validationTab~custom error messge
Example: required~username is manatory
If you don't specify custom error message, it will fall back to default behaviour

Hey @chatwithrajaram, create a PR from your fork, is more easy to review the code and all that.

I created pull request. Please take a look.
On Thu, Mar 31, 2016 at 7:43 PM, Waltton Santana<EMAIL_ADDRESS>wrote:

Hey @chatwithrajaram https://github.com/chatwithrajaram, create a PR
from your fork, is more easy to review the code and all that.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/asaskevich/govalidator/issues/104#issuecomment-203956910

--
Best Regards,
Rajaram Ganjikunta

What is the status of this issue?

I hope that this issue was resolved with one of merged PRs. You can reopen issue if not.


Add project logo to header

Also removes the rendering of the header submenu if empty (which it is, atm), because it causes some problems on the smallest viewport size.

I only see trento-agent.png used - what are all the other icons for that you added?

According to our guidelines SUSE products should not have any icons/illustration

As we do not create product brands, we don’t apply logos for commercial products. We label them with their names, written in our corporate font Poppins and without any additional picture mark. As with any rule, there is one exception: SUSE Rancher

Yes, the logo can be there!
BTW maybe this could be a good moment to also add it as the websites favicon ?

I only see trento-agent.png used - what are all the other icons for that you added?

hmm, I added those for potential future usage.


multi: accept memo note when opening channel

Change Description
Fixes #7639 by allowing a note ("memo") to go along with channel creation when using the openchannel CLI/RPC/JSON command. We accept a string with max length of 500 (command is rejected if length is exceeded) and persist it in a hex-encoded format as part of the TLV stream. When channels are queried, we retrieve the value and decode it before passing the info to the caller. We also add a new itest to verify both the happy and unhappy path.
Steps to Test
When using openchannel through CLI/RPC/JSON, pass in a memo note using the memo argument. If length is valid (<= 500), it should persist and be retrievable in listchannels call. If length is invalid, the command should be rejected with an appropriate error message.
Pull Request Checklist
Testing

[ ] Your PR passes all CI checks.
[X] Tests covering the positive and negative (error paths) are included.
[X] Bug fixes contain tests triggering the bug to prevent regressions.

Code Style and Documentation

[X] The change obeys the Code Documentation and Commenting guidelines, and lines wrap at 80.
[X] Commits follow the Ideal Git Commit Structure.
[X] Any new logging statements use an appropriate subsystem and logging level.
[X]  There is a change description in the release notes, or [skip ci] in the commit message for small changes.

📝 Please see our  Contribution Guidelines for further guidance.

Looks like TestFetchHistoricalChannel is failing inside db_test.go and I'm not quite sure why. It fails when deep-comparing channel with histChannel:
https://github.com/shaurya947/lnd/blob/cd2bea92db0dc1b45b84d69d15e65ffba8d674eb/channeldb/db_test.go#L693
The test seems to pass if I attach a Memo here:
https://github.com/shaurya947/lnd/blob/cd2bea92db0dc1b45b84d69d15e65ffba8d674eb/channeldb/channel_test.go#L313
But that shouldn't be required right? Am I doing something wrong with the TLV stream which messes up (de)serialization when the Memo field is not present?

Hey @guggero I updated the PR to address all your feedback:

Ditch hex encoding
Combine dbchannel changes into a single commit
Combine lnrpc changes into a single commit
Reuse existing itest

Also thanks so much for helping with the unit test failure. Your suggestion seems to have worked :slightly_smiling_face:

Hey @guggero addressed all your feedback as well as fixed a silly mistake I made in the itest (used ht.Alice/ht.Bob instead of the alice and bob arguments)

Hey @guggero the windows itest check seems to fail because of a process lock on a file it seems? Is that expected?
Also any suggestions as to whom I could tag for a second review?

Thanks for the review @positiveblue and @ffranr ! :slightly_smiling_face:
I fixed the one commit message and created a constant for the magic number (along with some comments). I will defer the PendingChannels change for a follow-up PR if that's ok. I've made a branch for it, I just want to make sure I can handle all the different pending states properly and not cause any more inadvertent regressions in this PR.

Thank you folks! My first PR :smiling_face_with_tear:


201 (Old 201)

In gitlab by @A-Lawrence on Aug 31, 2015, 01:09
Unable to generate correct URLs via artisan in dev/staging areas.
Need to wait on Laravel fix.

In gitlab by @A-Lawrence on Aug 31, 2015, 01:15
Seems to be working properly, now.
ALawrence66 - 2015-01-18T19:28:54.487263+00:00


Explicit python version in FindNanopb.cmake

CMake will use python2 binary instead of just python if the former
is found.

This fixes a issue in my system with both python2 and python3 binaries installed and /usr/bin/python link pointing to python3.

Hmm, with pull request #169 I think nanopb should work with python3
also now. I guess you don't have protobuf installed for python3?

You're right. My distribution only have the python2 protobuf version in the official repositories. I didn't find the python3 version and blamed Google too fast.
Sorry for bothering you.

No problem.
This might actually be a real issue, not sure if the CMake rules
should check for python3-protobuf and try python2 if not found.
Though if python3 is the default python for system, it makes sense
to expect that python3-protobuf would be the default protobuf
library.


Migrating to new _meta api for Django 1.10

The model _meta API has changed as of Django 1.10 :
https://docs.djangoproject.com/en/1.10/ref/models/meta/#migrating-from-the-old-api

Awesome, LGTM, thanks!


Fix camera's fadeIn/fadeOut effects

This PR

Fixes a bug

If any object is using the Light2D pipeline (or any other I think), it will not be affected by camera effects.
Set the TextureTintPipeline pipeline before rendering camera effects.

👍


Cannot run on Android 13

I am trying to run the app on Android 13. But getting this error when build.
Error:
Unable to load class 'org.gradle.api.publication.maven.internal.MavenPomMetaInfoProvider'.
This is an unexpected error. Please file a bug containing the idea.log file.

It is android studio error. You can search it.


Update integration tests to async tests with #[tokio::test]

Closes: #172

[x] Referenced an issue explaining the need for the change
[ ] Updated all relevant documentation in docs
[ ] Updated all code comments where relevant
[ ] Wrote tests
[ ] Updated CHANGES.md

Codecov Report

Merging #173 into master will not change coverage by %.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #173   +/-   ##
======================================
  Coverage    38.7%   38.7%
======================================
  Files          91      91
  Lines        3309    3309
  Branches      529     529
======================================
  Hits         1281    1281
  Misses       1689    1689
  Partials      339     339

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 00873bd...129435f. Read the comment docs.


concurrency will cause issues when writing

There needs to be a lock file to make sure that only one stately process operates on the state file at a time.

:tada: This issue has been resolved in version 0.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


react/jsx-indent bug with component "extends" syntax

I'm getting a strange indent error only when using the extends component syntax. Stateless components don't get this error:
Expected indentation of 10 space characters but found 8.
class CreateArticle extends Component {
  state = {};

  handleContextRef = contextRef => this.setState({ contextRef });

  render() {
    const { contextRef } = this.state;

    return (
      <div ref={this.handleContextRef}>
        <Shelf title="Create new article" />
        <Form                                 <--- Offending line
          size="large"
          onSubmit={e => {
            e.preventDefault();
          }}
        >
         ...
       </Form>
     </div>
    )
  }
}

eslint-plugin-react version: 7.5.1

I had the same issue, my problem was I was using babel-eslint ^8 (which uses the babylon parser - babel-7).
Downgrading babel-eslint to 7.2.3 fixed this issue for me.

@SimeonC OMG thank-you so much! 🚀

Sounds like this is an incompatibility with babel 7 (which is not released yet); I’ll leave this open until it is so we don’t forget.

I'm pretty confident these issues are all resolved; please file a new issue if not.


Removed {rlang} from derive_basetype_records

issue #2596
Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the main branch until you have checked off each task.

[x] Place Closes #<insert_issue_number> into the beginning of your Pull Request Title (Use Edit button in top-right if you need to update)
[x] Code is formatted according to the tidyverse style guide. Run styler::style_file() to style R and Rmd files
[ ] Updated relevant unit tests or have written new unit tests, which should consider realistic data scenarios and edge cases, e.g. empty datasets, errors, boundary cases etc. - See Unit Test Guide
[x] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[ ] Review the Cheat Sheet. Make any required updates to it by editing the file inst/cheatsheet/admiral_cheatsheet.pptx and re-upload a PDF and a PNG version of it to the same folder. (The PNG version can be created by taking a screenshot of the PDF version.)
[ ] Update to all relevant roxygen headers and examples, including keywords and families. Refer to the categorization of functions to tag appropriate keyword/family.
[ ] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[ ] Address any updates needed for vignettes and/or templates
[ ] Update NEWS.md under the header # admiral (development version) if the changes pertain to a user-facing function (i.e. it has an @export tag) or documentation aimed at users (rather than developers). A Developer Notes section is available in NEWS.md for tracking developer-facing issues.
[ ] Build admiral site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the "Reference" page.
[ ] Address or fix all lintr warnings and errors - lintr::lint_package()
[ ] Run R CMD check locally and address all errors and warnings - devtools::check()
[ ] Link the issue in the Development Section on the right hand side.
[x] Address all merge conflicts and resolve appropriately
[x] Pat yourself on the back for a job well done! Much love to your accomplishment!

Hi @Brijeshthummar02 thanks for jumping on this! :)
Can you create the related .Rd file, update the Pull Request title to start with "Closes #2596" as well as link the issue on the side.
Thanks again!

Hi @Brijeshthummar02 are you able to finish this PR?  It just needs the .Rd file made from the update

@bms63 i've my exams ongoing currently of university, will be doing it after 25th dec. you may also assign someone who wants to help me doing it before 25.

All good!  We have this done in another PR - https://github.com/pharmaverse/admiral/pull/2601
Plenty of issues to come back to once your are done with your exams!  Good luck!


Update sbt-sonatype to 3.9.7

Updates org.xerial.sbt:sbt-sonatype from 3.9.6 to 3.9.7.
GitHub Release Notes - Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.xerial.sbt", artifactId = "sbt-sonatype" } ]

labels: sbt-plugin-update, semver-patch

Codecov Report

Merging #137 (3e575e0) into master (cf99903) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #137   +/-   ##
=======================================
  Coverage   70.00%   70.00%
=======================================
  Files           4        4
  Lines          80       80
  Branches        4        4
=======================================
  Hits           56       56
  Misses         24       24

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 cf99903...3e575e0. Read the comment docs.


Included SCSS for importing

Hi There,
Could I kindly ask you to include an scss file as well so the reset rules can be included? I'd like to use the package through composer so I don't have the luxury of renaming/changing things :)

Hi ! No problem, I'll do that by tomorrow ;)

Thanks so much, I am in your debt :)


fail when i ran get-credentials 'az aks get-credentials --resource-group testResourceGroup --name myAKSCluster'

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az aks get-credentials
Errors:
'clusters'
Traceback (most recent call last):
python3.8/site-packages/knack/cli.py, ln 215, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 625, in execute
    raise ex
cli/core/commands/__init__.py, ln 689, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 682, in _run_job
    six.reraise(*sys.exc_info())
...
cli/command_modules/acs/custom.py, ln 1181, in merge_kubernetes_configurations
    _handle_merge(existing, addition, 'clusters', replace)
cli/command_modules/acs/custom.py, ln 1116, in _handle_merge
    if existing[key] is None:
KeyError: 'clusters'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az aks get-credentials --resource-group {} --name {}

Expected Behavior
Environment Summary
macOS-10.15.4-x86_64-i386-64bit
Python 3.8.2
Installer: HOMEBREW

azure-cli 2.3.1 *

Additional Context

aks

Hello! Please try again with the latest Azure CLI version. Feel free to reopen an issue if it still doesn't work.
https://docs.microsoft.com/en-us/cli/azure/install-azure-cli?view=azure-cli-latest


nsceventsandroid_sprint5_226_removesignup

Resolves #226
This PR removes the signup link from the navigation menu.

I checked out he branch and the bug changed is fixed. I don't see the sign up page in the navbar from my end. Code looks good.

I cloned it to my local device and it looks good!


chore(deps): update docker docker tag to v23

This PR contains the following updates:

Package
Update
Change

docker
major
20.10.23-dind -> 23.0.5-dind

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.

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

This PR has been generated by Renovate Bot.

lgtm

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


interfaces.php fix sunbetv6 var name typo

@sbeaver-netgate please look and commit if you like, before your next round of edits makes conflicts.

@sbeaver-netgate incorporated the changes manually.


interface with multiple struct

Hi Gophers
how can i use interface with diffrent-diffrent strcut?
I just want to use interface like struct with field name.So can we use interface as dynamic struct type.
i was used ff := getdata.(SecondType)  fmt.Println(ff.Address) but it is not dynamic.
Like
package main
import (
"fmt"
)
type FirstType struct {
Name string
Email string
}
type SecondType struct {
Address string
Points float64
}
func main() {
UseOfInterface(SecondType{Address: "Raj. india", Points: 4.4})
UseOfInterface(FirstType{Name: "Manish Jangid", Email<EMAIL_ADDRESS>}
func UseOfInterface(getdata interface{}) {
?
?
?
?
}

Hi @tgg-manish .
This appears to be a question rather than a Go bug. We do not use the issue tracker for general discussions or questions like these. The Github bug tracker is only used for tracking bugs and proposals going through the Proposal Process.
You might want to take a look at Questions wiki page; it has a list of good places for asking questions. Thanks.
Closing since this is not a bug.


Fixes #1405 remove deprecated alerts

… alerts

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

ok to test

I like that we're removing these from getting turned on in new installs. I don't recall, did we discuss having tasks to explicitly remove these if they exist on the filesystem, so that we take them out of existing installs?

Yes but that was out of scope for this card as I understood it

Should I add the tasks so we only make one pass?

Nah, having a follow up PR is good enough. Out of scope sounds right.

test this please

This will need to be rebased after #1506 is merged.


Cleanup saveservice

Description
Cleanup code in SaveService.
Motivation and Context
I always like to look at the warnings in the IDE to see, where the code has problems. If there are too many warnings, I tend to oversee the real dangerous ones. So this pull request tries to get rid of the ones that are easy to fix and hopefully clearer to read.
How Has This Been Tested?
Ran the unit tests.
Screenshots (if appropriate):
Types of changes

Code cleanup / refactoring

Checklist:

[x] My code follows the code style of this project.

+1
Thanks


Integration with flowtype

It would be really nice to be able to have static type checking with Flow while using baobab.
I didn't push Flow that far though, so I don't know if it's possible at the moment. I could give a try if you are interested :)

I don't know Flow enough to do this myself but I am sure it is possible for a library to come with some kind of flow definition files for external consomption with flow.

I could give a try if you are interested :)

Don't hesitate to do so. Just know that I probably won't be able to know if what you did is good :).

@ThomasCrevoisier did you make any progress on this issue ?

Sorry about giving no news... I looked at it and I don't think the lib is well-suited for a typechecker as we access properties dynamically via strings.
Of course we can mimic TypeScript interfaces, but using any everywhere is not satisfying IMHO. That's why I didn't push this further, but we can reproduce what has been done on TypeScript interfaces if it's needed :)

I've prepared a basic typing, enough for my needs. If anyone would test and/or finish it, I'd polish it and make a PR for flow-typed.
declare class Baobab extends events$EventEmitter {
  apply(updater: (any) => any): any;
  clone(): any;
  concat(path: BaobabPath, value: any[]): any;
  deepClone(): any;
  // deepMerge()
  exists(path: BaobabPath): boolean;
  get(path: BaobabPath): any;
  merge(path: BaobabPath, value: any): any;
  // pop
  // project
  push(path: BaobabPath, value: any): any;
  // serialize
  set(path: BaobabPath, value: any): any;
  // shift
  // splice
  unset(path: BaobabPath): void;
  // unshift
  // update()

  static dynamicNode: (...definition: (BaobabPath | (...paths: any[]) => any)[]) => BaobabMonkeyDefinition;
  static monkey: (...definition: (BaobabPath | (...paths: any[]) => any)[]) => BaobabMonkeyDefinition;

  commit(): Baobab;
  constructor(initialData?: {}, options?: BaobabOptions): void;
  getMonkey(path: BaobabPath): null | BaobabMonkey;
  release(): void;
  select(): BaobabCursor;
  select(path: BaobabPath): BaobabCursor;
  toJSON(): any;
  toString(): string;
  validate(affectedPaths?: string[][]): boolean;
  watch(mapping: {[string]: BaobabPath}): BaobabWatcher;
}

declare class BaobabCursor extends events$EventEmitter {}

declare class BaobabMonkey {}

declare class BaobabMonkeyDefinition {}

declare type BaobabOptions = {
 asynchronous?: boolean;
 autoCommit?: boolean;
 immutable?: boolean;
 persistent?: boolean;
 pure?: boolean;
 validate?: (prevState: {}, nextState: {}, affectedPaths: string[][]) => void;
 validationBehavior?: 'notify' | 'rollback';
};

declare type BaobabPath = (number | string | {[string]: number | string})[];

declare class BaobabWatcher extends events$EventEmitter {
  get(): any;
  getCursors(): {[string]: BaobabCursor};
  getWatchedPaths(): string[][];
  refresh(mapping: {[string]: BaobabPath}): void;
  release(): void;
}

declare module 'baobab' {
  declare module.exports: typeof Baobab;
}


Quarantined but no errors listed

Hello. Our zendesk user provisioning is listed as "Quarantined due to a high number of errors. Please view the audit logs below to assess the errors and remediate them. Sync interval reduced to once per 24 hours."
But when looking at the audit logs, there are nothing but "success".
Something works because testing works, and the roles from zendesk are provisioned.
I have a test user which is not provisioned down to our zendesk.
Activity ID: 2020-03-24 08:23:09
35010c81-e7eb-4d3f-a065-b31efa82d41e
Synchronization Job ID:
zendesk.26c323f158c24794b1237344c86ae9fa.49311620-7c14-49ad-9d0c-668dfe95f013

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: fb82bd89-1782-c13e-81ca-33eb42187a39
Version Independent ID: 2ca54f47-bf69-f001-3c5d-551e89918c18
Content: Problem configuring user provisioning to an Azure AD Gallery app
Content Source: articles/active-directory/app-provisioning/application-provisioning-config-problem.md
Service: active-directory
Sub-service: app-provisioning
GitHub Login: @msmimart
Microsoft Alias: mimart

@Niclas1976 Thank you for your feedback . We will investigate and update this thread

I would recommend creating a support case if this persists and the quarantine message in the progress bar is not helpful. #please-close


Client command to add a repository to the configuration

This would make it easier to add repositories managed by others.
> amber repository add <URL>
<URL> could be a GitHub URL or something like file://path/to/repos. Modify .repositories and add a new entry. Repository types can check whether they support the URL with a pattern.

Tested and verified.

Still need to document this.


Increment version for synapse releases

Increment version for synapse releases

Can one of the admins verify this patch?

The branch is outdated and has conflicts. Closing.


Error: Invalid prop fontWeight supplied to View

Error message:
Failed prop type: Invalid props.style key fontWeight supplied to View.
Bad object: {
"fontWeight": "bold",
"fontSize": 20,
"marginBottom": 10,
"marginTop": 20
}
React native version: 0.55.3

Related to #150


Update specs2-cats, specs2-core to 4.10.0

Updates

org.specs2:specs2-cats
org.specs2:specs2-core

from 4.6.0 to 4.10.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.specs2" } ]

labels: library-update, semver-minor

Superseded by #115.

Superseded by #115.


Default fetch request headers differ between Cloudflare Workers and Miniflare

When making a fetch request to an external service from a Worker, certain request headers are populated by default. The Workers runtime and miniflare send different default headers, meaning that when calling some external APIs (e.g. Github's which requires a User-Agent header for all requests!) code that works in miniflare will fail when running on the Workers runtime.
To validate this, I set up a simple echo service running on https://http-ua-proof-of-concept.dancowell.workers.dev/ which will return a structured JSON object containing request data.
Executing the following worker in each context results in different output:
export async function handleRequest(request: Request, env: Bindings) {
  const response = await fetch(
    "https://http-ua-proof-of-concept.dancowell.workers.dev/"
  );

  const body = await response.text();

  return new Response(body);
}

const worker: ExportedHandler<Bindings> = { fetch: handleRequest };

export default worker;

Request from miniflare
{
  "method": "GET",
  "url": "https://http-ua-proof-of-concept.dancowell.workers.dev/",
  "body": "",
  "headers": {
    "accept": "*/*",
    "accept-encoding": "gzip",
    "accept-language": "*",
    "cf-connecting-ip": "<IP_ADDRESS>",
    "cf-ipcountry": "JP",
    "cf-ray": "6c6fcc948c918d01",
    "cf-visitor": "{\"scheme\":\"https\"}",
    "connection": "Keep-Alive",
    "host": "http-ua-proof-of-concept.dancowell.workers.dev",
    "sec-fetch-mode": "cors",
    "user-agent": "undici",
    "x-forwarded-proto": "https",
    "x-real-ip": "<IP_ADDRESS>"
  }
}

Request from Cloudflare
{
  "method": "GET",
  "url": "https://http-ua-proof-of-concept.dancowell.workers.dev/",
  "body": "",
  "headers": {
    "accept-encoding": "gzip",
    "cf-connecting-ip": "<IP_ADDRESS>",
    "cf-ipcountry": "US",
    "cf-ray": "6c6fdbd42d216318",
    "cf-visitor": "{\"scheme\":\"https\"}",
    "connection": "Keep-Alive",
    "host": "http-ua-proof-of-concept.dancowell.workers.dev",
    "x-forwarded-proto": "https",
    "x-real-ip": "<IP_ADDRESS>"
  }
}

Based on this output, miniflare sends 4 additional headers by default: accept, accept-language, sec-fetch-mode, user-agent
I would expect that miniflare's default request headers should match that of the workers runtime.

If you'll permit me to speculate for a minute here, I'm guessing that the desire to use undici comes from that very same spec-compliance - it doesn't make sense for the Miniflare project to maintain its own fetch implementation with all the classes that come along with that.
Forking undici brings on that very same maintenance burden, with extra steps (since you also have to incorporate upstream changes into your fork.) Therefore it's not really a great option.
This might be a terrible idea, but what if Miniflare ran its own lightweight proxy endpoint that stripped the offending headers from outbound requests before forwarding to the intended destination, and in this shim fetch function rewrote the URI passed into fetch to send it via that proxy?
You would keep the benefits of having a spec-compatible internal API, while also being compliant with the CF implementation. There would be a small maintenance cost for the proxy, but I imagine it would probably be substantially lower compared to maintaining a fork, particularly considering the very specific transform required.

Modifying the global dispatcher smells really bad to me. If anything else in Miniflare uses undici in the future (even indirectly) it would be quite difficult without context of this issue to understand why those requests are getting mangled.


fix Fix the bugs of data distribution calculation

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.54 seconds
load time: 478 seconds
storage size:<PHONE_NUMBER>9 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230107094038_clickbench_pr_75328.html


enable updating transitive dependencies with solver

Previously if updating a transitive dependency, it was just pinned and no conflict resolution was performed.
This PR fixes that by elevating the check for using the dependency solver up one function.  This enables a transitive dependency to be fixed by updating the top-level package.  The solver has always had this capability, but it was bypassed in the transitive case.
The following other changes were performed:

Add test helper DependencySolverEnvironment to make unit testing the dependency solver easier; fewer magic strings.
Rename ResolveDependencyConflictsNew => ResolveDependencyConflicts
Rename ResolveDependencyConflictsOld => ResolveDependencyConflictsWithBruteForce

@brettfo should this smoke test be merged too? https://github.com/dependabot/smoke-tests/pull/219


Please add the possible configuration paramters

Hi!
Would be nice to have the possible configuration parameters documented, to not have to dive into the code and find somewhere that :kms and :nms are possible as unit.
Regarding the documentation I expected :kilometers should be set.
Best regards,
TheWudu

This is documented in Readme.md presently, so this seems complete.

Yes it's there now.


Issue with Camera plugin in release build

FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring root project 'camera'.

SDK location not found. Define location with sdk.dir in the local.properties file or with an ANDROID_HOME environment variable.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 2s
The plugin camera could not be built due to the issue above.

hi @RohithKilaparthi
Can you please provide your flutter doctor -v your flutter run --release  --verbose and a complete reproducible minimal code sample
Thank you

We are also experiencing a similar issue with our project. This happened sometime after 13th of October (When we had our last successful deploy). One fix that we have noticed, is to set the version of all flutter plugins to a version before the October 14th release (where compileSdkVersion and targetSdkVersion got bumped from 28 to 29). IOS builds also work, so this is an Android issue. So in our case, we set the version of these plugins back to a version that we knew worked:

path_provider: ^1.6.11 -> 1.6.11
url_launcher: ^5.5.0 -> 5.5.0
file_picker: ^2.0.1+2 -> 2.0.1+2
image_picker: ^0.6.7+11 -> 0.6.7+11
camera: ^0.5.8+7 -> 0.5.8+7
device: ^0.4.2+8 -> 0.4.2+8

This solved our issue, but we would like to still use the latest version of the packages that we use.
We have also tried to use the latest version before the 14th of October update for all these packages, that also worked.
We have also tried to update compileSdkVersion and targetSdkVersion to 29 in the file android/app/build.gradle for our app, without luck
Error message
Launching lib\main.dart on SM G975F in debug mode...
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:processDebugResources'.

A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
Android resource linking failed
C:\Users\gaute.haugen\Documents\GithubProjects\flutter_appframe\appframe_filter\example\build\file_picker\intermediates\library_manifest\debug\AndroidManifest.xml:9:5-15:15: AAPT: error: unexpected element  found in .

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 28s
The built failed likely due to AndroidX incompatibilities in a plugin. The tool is about to try using Jetfier to solve the incompatibility.
√ android\settings_aar.gradle created successfully.
Building plugin camera...
FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring root project 'camera'.

SDK location not found. Define location with sdk.dir in the local.properties file or with an ANDROID_HOME environment variable.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 2s
Exception: The plugin camera could not be built due to the issue above.
Exited (sigterm)

flutter doctor -v
[√] Flutter (Channel stable, 1.22.1, on Microsoft Windows [Version 10.0.19041.572], locale en-US)
    • Flutter version 1.22.1 at C:\flutter
    • Framework revision f30b7f4db9 (2 weeks ago), 2020-10-08 10:06:30 -0700
    • Engine revision 75bef9f6c8
    • Dart version 2.10.1
[√] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at C:\Users\gaute.haugen\AppData\Local\Android\sdk
• Platform android-30, build-tools 29.0.2
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
• All Android licenses accepted.
[√] Android Studio (version 4.0)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin version 46.0.2
• Dart plugin version 193.7361
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
[√] VS Code (version 1.50.1)
• VS Code at C:\Users\gaute.haugen\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.15.0
[√] Connected device (1 available)
• SM G975F (mobile) •  • android-arm64 • Android 10 (API 29)
• No issues found!

build.gradle

def localProperties = new Properties()
def localPropertiesFile = rootProject.file('local.properties')
if (localPropertiesFile.exists()) {
    localPropertiesFile.withReader('UTF-8') { reader ->
        localProperties.load(reader)
    }
}
def flutterRoot = localProperties.getProperty('flutter.sdk')
if (flutterRoot == null) {
throw new GradleException("Flutter SDK not found. Define location with flutter.sdk in the local.properties file.")
}
def flutterVersionCode = localProperties.getProperty('flutter.versionCode')
if (flutterVersionCode == null) {
flutterVersionCode = '1'
}
def flutterVersionName = localProperties.getProperty('flutter.versionName')
if (flutterVersionName == null) {
flutterVersionName = '1.0'
}
apply plugin: 'com.android.application'
apply plugin: 'kotlin-android'
apply from: "$flutterRoot/packages/flutter_tools/gradle/flutter.gradle"
def keystoreProperties = new Properties()
def keystorePropertiesFile = rootProject.file('key.properties')
if (keystorePropertiesFile.exists()) {
keystoreProperties.load(new FileInputStream(keystorePropertiesFile))
}
android {
compileSdkVersion 29
sourceSets {
    main.java.srcDirs += 'src/main/kotlin'
}

lintOptions {
    disable 'InvalidPackage'
}

defaultConfig {
    applicationId "SomeUniqueName"
    minSdkVersion 21
    targetSdkVersion 29
    versionCode flutterVersionCode.toInteger()
    versionName flutterVersionName
    testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
}

signingConfigs {
   release {
       keyAlias keystoreProperties['keyAlias']
       keyPassword keystoreProperties['keyPassword']
       storeFile keystoreProperties['storeFile'] ? file(keystoreProperties['storeFile']) : null
       storePassword keystoreProperties['storePassword']
   }

}
buildTypes {
    release {
        signingConfig signingConfigs.release
    }
}

}
flutter {
source '../..'
}
dependencies {
implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
testImplementation 'junit:junit:4.12'
androidTestImplementation 'androidx.test:runner:1.1.1'
androidTestImplementation 'androidx.test.espresso:espresso-core:3.1.1'
}

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Move MNISTReader to use new api

@dayo05 -- I'll go through this PR and try it out today. Looking at it, it seems like a good idea to accept it.
I would also like to draw your attention to our examples repo: https://github.com/dotnet/TorchSharpExamples, and suggest that you take a look at the C# and F# examples there. If you want to add this API to those data loaders, that would be great, and perhaps also add a tutorial on how to use the data reader APIs. If you're not comfortable with F#, I can translate from C# later.

@dayo05 -- this is a long weekend in the US, so I'll take a look at the PR changes on Tuesday.

@dayo05 -- would you be okay with me making some changes to your PR and then sending them your way?

@dayo05 -- would you be okay with me making some changes to your PR and then sending them your way?

I just sent you a PR in your fork. One thing I'm wondering about, is that the DataLoader always throws away the tensors that it has built up. That doesn't seem like it's necessary when 'shuffle=false'.

@dayo05 -- would you be okay with me making some changes to your PR and then sending them your way?

I just sent you a PR in your fork. One thing I'm wondering about, is that the DataLoader always throws away the tensors that it has built up. That doesn't seem like it's necessary when 'shuffle=false'.

But I think this makes same between shuffled datasets and non-shuffled datasets

But I think this makes same between shuffled datasets and non-shuffled datasets

No, I don't think so. If shuffle=true, then the batches will be different in each epoch, so it makes sense to dispose each batch as soon as possible, to relieve memory pressure. On the other hand, when shuffling is turned off, then the batches are the same every epoch, and there's no need to reconstruct them. It is more efficient (but use more memory) to keep the batched data around for the next epoch.

@NiklasGustafsson bump


Angular2 version not working in FF, source maps, runtime errors

Hi. I'm trying to choose a workflow for TypeScript + angular project. Looks like JSPM with your plugin-typescript has the most straightforward workflow to me. So i checked the example (this repo).
The angular2 branch doesn't work in FireFox.
The master branch does work in FF but it takes almost 10 seconds to load it which is not acceptable. Do you know why? Is it a problem with JSPM/SystemJS loader? Have you noticed? It's quite fast to load it in Chrome.
Source maps:
When using console.log, it works fine, awesome!
getting available for 0 - hand.ts:31
But when trying a runtime error:
angular2.dev.js:9842 ReferenceError: foo is not defined in [availableCards in <li class="hand-card" *for="#card of availableCards">]
Are these issues somehow solvable?
BTW thanks for your work.

Apologies for not getting back to you before, I think the angular2 example is now working.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 2019 ms

Resolved: Bofrost.at performance has improved in f77dd0f after 6 minutes.


Compiled but APP stop just before the app start on Android

I created the empty project to test admob working or not. I just put react-native-admob and react-native
"react": "16.6.0-alpha.8af6728",
"react-native": "0.57.3",
"react-native-admob": "^2.0.0-beta.5"

ext {
        buildToolsVersion = "27.0.3"
        minSdkVersion = 16
        compileSdkVersion = 27
        targetSdkVersion = 26
        supportLibVersion = "27.1.1"
    }

I even tried
subprojects { subproject ->
    afterEvaluate{
        if((subproject.plugins.hasPlugin('android') || subproject.plugins.hasPlugin('android-library'))) {
            android {
                compileSdkVersion rootProject.ext.compileSdkVersion
                buildToolsVersion rootProject.ext.buildToolsVersion
            }
        }
    }
}

But still remain without any luck.

Did you try putting your admob app ID in AndroidManifest.xml?

There is no documentation of putting admob app ID in AndroidManifest.xml in this package.

Well its same result.

any luck with this issue?

My Android Device:
Android 8.1
//==============================================================
My Project Specification:
"react": "16.6.1",
"react-native": "0.57.5",
"react-native-admob": "^2.0.0-beta.5"
//==============================================================
My Project's Android Specs:(MyProject/android/build.gradle)
ext {
buildToolsVersion = "27.0.3"
minSdkVersion = 16
compileSdkVersion = 27
targetSdkVersion = 26
supportLibVersion = "27.1.1"
}
//==============================================================
Admob's Android Specs:(react-native-admob/android/src/build.gradle)
compileSdkVersion 23
buildToolsVersion "23.0.1"
defaultConfig {
minSdkVersion 16
targetSdkVersion 22
}
dependencies {
compile 'com.facebook.react:react-native:+'
compile 'com.google.android.gms:play-services-ads:16.0.0'
}
Its Running Good But didnt test it with real ads yet

For Now I am Closing It..


Fix Style/UnneededPercentQ condition

%q(foo #{'bar'} baz) should not be warned because auto-correcting it
to "foo #{'bar'} baz" is wrong.

Looks correct to me. Please update CHANGELOG.md too.

Sure. Added a CHANGELOG entry.

I have to resolve conflicts of CHANGELOG.md several times :weary:


Select only one option [feature]

hey there ,
the package is all good , thanks for that,
i was hoping if you can add one more feature that would be great help
--------------The Feature----------------
the user can select the multiple options right
can you add something like the user can select only one option
when he tries to select other option then other option should be automatically deselected
---basically user can select only one option-------

@TheAlphamerc Do you have any plans to work more on this package ? i may this feature later, so i prefer to make a pull request for this package instead of making something similar from scratch. What do you think ?

Currently, I have no plan for it. I will appreciate it if you can contribute to this package.

hey there ,
the package is all good , thanks for that,
i was hoping if you can add one more feature that would be great help
--------------The Feature----------------
the user can select the multiple options right
can you add something like the user can select only one option
when he tries to select other option then other option should be automatically deselected
---basically user can select only one option-------

Hi @Madanraj1 , The feature you are asked for is introduced in package version 0.0.8.


Add Kodi snap for stable release.

Add a snap for Kodi for the last stable tag "16.1-Jarvis"

[ ]   File: kodi-stable/README.md:L1-30
 Do you know if they were discussed on<EMAIL_ADDRESS>It looks like we might need another interface or two.
 I dont think so, maybe is a new issue and I havent reported it
 I think it'd be good to at least bring it up there and see if others can work on resolving them.
[ ]   General Comment
 In Travis this fails to build with

checking mpeg2dec/mpeg2.h usability... no
checking mpeg2dec/mpeg2.h presence... no
checking for mpeg2dec/mpeg2.h... no
configure: error: Could not find a required library. Please see the README for your platform.
Command '['/bin/sh', '/tmp/tmpudwgxd8_', './configure', '--prefix=', '--prefix=/']' returned non-zero exit status 1```
- <a href='https://github.com/dholbach'><img border=0 src='https://avatars.githubusercontent.com/u/1346979?v=3' height=16 width=16'></a> On my machine locally it fails with

checking for LIBDCADEC... no
configure: error: libdcadec not found
Command '['/bin/sh', '/tmp/tmpwwmf3jxr', './configure', '--prefix=', '--prefix=/']' returned non-zero exit status 1
- <a href='https://github.com/dholbach'><img border=0 src='https://avatars.githubusercontent.com/u/1346979?v=3' height=16 width=16'></a> Can we somehow adjust where ffmpeg comes from? This is how the build fails (both using 'snapcraft' or 'snapcraft cleanbuild'): http://paste.ubuntu.com/21252125/
- <a href='https://github.com/dholbach'><img border=0 src='https://avatars.githubusercontent.com/u/1346979?v=3' height=16 width=16'></a> :+1:<a href='#crh-update-none'></a>
- <a href='https://github.com/dgmvecuador'><img border=0 src='https://avatars.githubusercontent.com/u/406504?v=3' height=16 width=16'></a> Kodi developers usually recommend to use version of ffmpeg provided inside the code, so we should use that one.  They update ffmpeg each major release.
- <a href='https://github.com/dholbach'><img border=0 src='https://avatars.githubusercontent.com/u/1346979?v=3' height=16 width=16'></a> Ok, let's pick whatever works. Can you apply the patch I specified earlier? That way you should see in the next Travis build how it's currently failing. Currently it's missing some build-packages.
- <a href='https://github.com/dgmvecuador'><img border=0 src='https://avatars.githubusercontent.com/u/406504?v=3' height=16 width=16'></a> I have applied your patch and other changes. Now the build is successful in Travis
- [ ] <a href='#crh-comment-Pull 43e19fe8398103a3994e25cc56e52622e9405ccb kodi-stable/snapcraft.yaml 49'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60> <b><a href='https://github.com/ubuntu/snappy-playpen/pull/197#discussion_r72583865'>File: kodi-stable/snapcraft.yaml:L1-209</a></b>
- <a href='https://github.com/dholbach'><img border=0 src='https://avatars.githubusercontent.com/u/1346979?v=3' height=16 width=16'></a> Can you apply this? http://paste.ubuntu.com/21252024/
It add's a few `build-packages`.
- [x] <a href='#crh-comment-Pull abebc4a09f34fd3c26b67a0859efc2342aaf69dc kodi-stable/setup/gui/kodi.desktop 14'></a> <img src='http://www.codereviewhub.com/site/github-completed.png' height=16 width=60> <b><a href='https://github.com/ubuntu/snappy-playpen/pull/197#discussion_r72477168'>File: kodi-stable/setup/gui/kodi.desktop:L1-23</a></b>
- <a href='https://github.com/dholbach'><img border=0 src='https://avatars.githubusercontent.com/u/1346979?v=3' height=16 width=16'></a> Do these actions work as expected?
- <a href='https://github.com/dgmvecuador'><img border=0 src='https://avatars.githubusercontent.com/u/406504?v=3' height=16 width=16'></a> Yes, they work as expected
![kodi_desktop_snap](https://cloud.githubusercontent.com/assets/406504/17200199/e016032c-5448-11e6-852f-acd7bef7fba1.png)

<a href='https://www.codereviewhub.com/ubuntu/snappy-playpen/pull/197?mark_as_completed=1'><img src='http://www.codereviewhub.com/site/github-mark-as-completed.png' height=26></a> <a href='https://www.codereviewhub.com/ubuntu/snappy-playpen/pull/197?approve=1'><img src='http://www.codereviewhub.com/site/github-approve.png' height=26></a> <a href='https://github.com/ubuntu/snappy-playpen/pull/197'><img src='http://www.codereviewhub.com/site/github-refresh.png' height=26></a>
<a href='#crh-end'></a>

This pull request is now managed with CodeReviewHub. Refresh.

In Travis this fails to build with ```
checking mpeg2dec/mpeg2.h usability... no
checking mpeg2dec/mpeg2.h presence... no
checking for mpeg2dec/mpeg2.h... no
configure: error: Could not find a required library. Please see the README for your platform.
Command '['/bin/sh', '/tmp/tmpudwgxd8_', './configure', '--prefix=', '--prefix=/']' returned non-zero exit status 1

On my machine locally it fails with
checking for LIBDCADEC... no
configure: error: libdcadec not found
Command '['/bin/sh', '/tmp/tmpwwmf3jxr', './configure', '--prefix=', '--prefix=/']' returned non-zero exit status 1

Can we somehow adjust where ffmpeg comes from? This is how the build fails (both using 'snapcraft' or 'snapcraft cleanbuild'): http://paste.ubuntu.com/21252125/

:+1:

Kodi developers usually recommend to use version of ffmpeg provided inside the code, so we should use that one.  They update ffmpeg each major release.

Ok, let's pick whatever works. Can you apply the patch I specified earlier? That way you should see in the next Travis build how it's currently failing. Currently it's missing some build-packages.

I have applied your patch and other changes. Now the build is successful in Travis

Great work!
When I try to run it, I see this though:
daniel@daydream:~/dev/snappy/snappy-playpen/kodi-stable$ kodi
libGL error: unable to load driver: i965_dri.so
libGL error: driver pointer missing
libGL error: failed to load driver: i965
libGL error: unable to load driver: swrast_dri.so
libGL error: failed to load driver: swrast
And I get a black screen. Anything I'm missing?

Its a problem with opengl libs, there are multiple bugs reported like https://bugs.launchpad.net/snappy/+bug/1574851
As workaround, you can copy your driver libs to the folder "/var/lib/snapd/lib/gl/" in my case, I copied all the content of "/usr/lib/nvidia-xxx". I dont know about your graphic card, but it seems to work with intel without problems.

I'm on intel too (i915).

You should copy all the libs from Intel, "i965_dri.so" and the dependencies to "/var/lib/snapd/lib/gl/", maybe check the deb package from the repository.

Hi, did you solve the problem in your environment??

Sorry for the long wait. This makes it work for me: http://paste.ubuntu.com/22795846/

Brilliant work! :-D


TreeDataGrid support planned?

Hi. Do you plan to add styles/templates for TreeDataGrid? As for now seems you do not support this control,
<sty:FluentAvaloniaTheme />

is not enouth to work with TreeDataGrid

At this time, I have no plans to add the styles for TreeDataGrid as it's A) outside the main Avalonia library, and B) considered a "premium" control with support only for Avalonia's paying customers.
However, it shouldn't be too hard to add the styles yourself. They should mostly work, you'll just have to update/change some of the brush resources to the Fluent v2 naming.


Can Chrome support aframe-ar.js?

Could you please specify in what context, such as intended OS for development?
As per the README

Browser Support
Demo tested on the following browser setups:

Desktop Chrome with webcam and 2 tabs (one for Hero, one for result) (works!)
Android native 4.4.2 (doesn't work, doesn't ask for permission to use camera. I see white background and text)
Android native 5.0 (doesn't work, doesn't ask for permission, I see white background and text)
Chrome on Android 4.4.2 (works!)
Chrome on Android 5.0 (doesn't work, asks for permission, I see black background, text and a chart)
Safari and Chrome on iOS < 11 (doesn't work, doesn't ask for permission, I see white background and text)
Microsoft Edge on Windows 10 (Chrome on Google Pixel phone to view hologram)

Credits: @HelloDeadline, @sorianog

Additionally, do you intend to use aframe-ar.js with any framework? Frameworks like Angular2+ and React have had issues that are being actively discussed and worked on in this thread. My coworkers and I have been working on documenting an Angular solution based on the very useful linked React thread. Most of our issues with Angular have been in Chrome, so if you're asking from a bad experience with Angular or another framework then I can give you a more detailed write up.

@michael-small Hi, can you share with us the angular doc to run ar.js app on the thread linked by you?

@VinceMar please detail your issue. Also, @michael-small have answered you. If you have more doubts, open a more detailed issue, thanks

@michael-small Hi, can you share with us the angular doc to run ar.js app on the thread linked by you?

To be honest I misspoke a bit because technically my team's Angular doc isn't quite ready. However, I will give a quick and dirty overview of our Angular advice. When our doc is fully fleshed out we will link it here and in the implementations thread. We are figuring this out and optimizing as we go, so this is no be-all-end-all solution and would gladly take input.
Apologies for the length and formatting, I was a bit rushed.
Solution # 1 (simpler)
Before anything with AFrame syntax can be used in Angular, CUSTOM_ELEMENTS_SCHEMA must be set in the NgModule directive and the two AR.js related script tags must be set in index.html. See this comment by @jsebrech on how to do that from the start on an Angular project with AR.js to the first run of it. Note: the scripts are outdated so make sure to use more current ones.
The example app.component.html from the link above should work out of the box on any supported browser with a Hiro marker. That said, our team uses barcode markers that show a-image's. My team's big issues with Angular and AR.js was with barcodes working in Chrome. Barcodes showed up using example code like below in all browsers except Chrome. More on that below.
This should be all that is necessary for Angular and AR.js to get along, in Chrome, on supported OS's, using barcode markers.
Our component is app.component with the selector app-root, so the HTML of index.html and app.component.html should look like this:
index.html
<!doctype html>
<html lang="en">
<head>
  <meta charset="utf-8">
  <title>ArSimulation</title>
  <base href="/">

  <meta name="viewport" content="width=device-width,initial-scale=1,user-scalable=no,minimum-scale=1,maximum-scale=1"/> <!--https://github.com/jeromeetienne/AR.js/issues/541#issue-457792891-->
  <link rel="icon" type="image/x-icon" href="favicon.ico">
  <link rel="manifest" href="manifest.webmanifest">
  <meta name="theme-color" content="#1976d2">
  <link href="https://fonts.googleapis.com/css?family=Roboto:300,400,500" rel="stylesheet">
  <link href="https://fonts.googleapis.com/icon?family=Material+Icons" rel="stylesheet">

  <!-- AR.js related Script Imports -->
  <script src="https://aframe.io/releases/0.9.1/aframe.min.js"></script>
  <script src="https://cdn.rawgit.com/jeromeetienne/AR.js/1.7.2/aframe/build/aframe-ar.js"></script>

</head>

<body class="mdc-typography">

  <!-- Component selector on app.component -->
  <app-root></app-root>

  <noscript>Please enable JavaScript to continue using this application.</noscript>
</body>
</html>

app.component.html
<a-scene embedded arjs='sourceType: webcam; debugUIEnabled: false; detectionMode: mono_and_matrix; matrixCodeType: 3x3;'>
	<a-marker type='barcode' value='2'>
                <!-- Replace with your own image from the assets folder or linked from an HTTPS source  -->
		<a-image src="assets/image.png" position="0 0 0" rotation="-90 0 0" scale="1 1 1"></a-image>
	</a-marker>
	<a-entity camera></a-entity>
</a-scene>

With this configuration, barcode images seemingly should work by the syntax conventions of AR.js, but is a little more complicated. As @fega mentions but modified from React to Angular, for AR.js to work in an Angular component there needs to be a delay in the component being instantiated (in Chrome, other browsers like Firefox or Edge didn't need this solution).
This can be achieved by just a few lines in the component's typescript file and html file.
app.component.ts
export class AppComponent implements OnInit {

  //tied to the *ngIf="showContent" in the component's html
  public showContent: boolean = false;

  public ngOnInit() {
    //This delays the content of the component from showing until aframe and aframe-ar.js load
    //NOTE: This arbitrary 1 second wait may be too long, or worse, too short. See end for more info.

    setTimeout(()=>this.showContent=true, 1000);

    //Thanks to Valikhan Akhmedov, Oct 4 '16 at 12:22 @ https://stackoverflow.com/a/39852249/8273171
    //This move was inspired by kawalt's react-aframe-ar-test's index.js @ https://github.com/kalwalt/react-aframe-ar-test/blob/master/src/index.js
    //which was inspired by fega @ https://github.com/jeromeetienne/AR.js/issues/493#issuecomment-490303718
  }

app.component.html
<!-- Notice the *ngIf in the a-scene. The component will not show up until showContent is set true in ngOnInit -->

<a-scene embedded arjs='sourceType: webcam; debugUIEnabled: false; detectionMode:
mono_and_matrix; matrixCodeType: 3x3;' *ngIf="showContent">
	<a-marker type='barcode' value='2'>
		<a-image src="assets/image.png" position="0 0 0" rotation="-90 0 0" scale="1 1 1"></a-image>
	</a-marker>
	<a-entity camera></a-entity>
</a-scene>

And for our team implementing AR.js with Angular, that was all we needed.
Solution # 2 (refined Solution # 1)
However, there are a few caveats. As I mentioned about the showContent 1 second delay, delays with a static time aren't reliable as that delay is either too much or possibly not enough due to potential network or async issues.
Our solution is to dynamically load aframe and aframe-ar.js using DynamicScriptLoaderService.ts by Zain Zafar in his article "Angular: Load External JavaScript File Dynamically". First of all, remove the two script tags related to AR.js from index.html. Then add DynamicScriptLoaderService.ts to your project by making it a provider in app.module.ts's @NgModule and  import the service in app.component.ts. Our ScriptStore is:
export const ScriptStore: Scripts[] = [
  { name: 'aframe', src: "https://aframe.io/releases/0.9.1/aframe.min.js" },
  { name: 'arjs', src: "https://cdn.rawgit.com/jeromeetienne/AR.js/1.7.2/aframe/build/aframe-ar.js" }
];

Back in your app.component.ts, add the following method loadScripts() and call it on ngOnInit():
app.component.ts
  ngOnInit() {
    this.loadScripts();
  }

  private loadScripts() {
    this.dynamicScriptLoader.load('aframe','arjs').then(data => {
      this.showContent=true;
    }).catch(error => console.log(error));
  }

We're almost there. The last needed bit is to add an event listener in ngOnInit.
Once everything is said and done, your app.component.ts should look like this:
app.component.ts
import { Component, OnInit } from '@angular/core';
import { DynamicScriptLoaderService } from '.../path_to_service/DynamicScriptLoaderService'

@Component({
  selector: 'app-root',
  templateUrl: './app.component.html',
  styleUrls: ['./app.component.scss']
})

export class AppComponent implements OnInit {

  public showContent: boolean = false;

  constructor(private dynamicScriptLoader: DynamicScriptLoaderService) {}

  ngOnInit() {
    this.loadScripts();

    window.addEventListener('load', (event) => {  //Thanks to this article: https://www.kirupa.com/html5/running_your_code_at_the_right_time.htm
    });
  }

  private loadScripts() {
    this.dynamicScriptLoader.load('aframe','arjs').then(data => {
      this.showContent=true;
    }).catch(error => console.log(error));
  }
}

Now AR.js and Aframe will be loaded in dynamically when it is ready.
This should be all that is necessary for Angular and AR.js to get along, in Chrome, on supported OS's, using barcode markers. As I said earlier, my team intends to refine and condense this info into a proper documentation document at a later date when we have more figured out.
PS: If you want to set markers's properties using our setup, check out this issue by @HeinPauwelyn. Our team uses the typescript solution inside of the window.addEventListener.