InteractiveMarker Feedback has Timestamp 0

I have an InteractiveMarker (Arrow) and use its Timestamp header.stamp for further processing.
Since indigo it is constantly set to 0 (both sec & nsec)
In groovy (I could check on hydro soon) the timestamp is updating well.
Moved this issue from https://github.com/ros-visualization/rviz/issues/792

I'm going to close this since there hasn't been any updates. @orzechow feel free to comment here if you find time. I can always reopen the issue.

We are seeing a similar issue in Kinetic, where IMs that work fine, but will report a timestamp of 0 after a setPose() call to the IM server with a pose in a new frame from the frame used in the original inert() call to set up the marker.  If you then setPose() back to the original frame used when creating the marker, the timestamps start being non-zero again.  Seems related to this issue, as the user may have been calling setPose() early on with maybe a different frame from what was used during IM creation.  I can create a new issue if desired.

Could you provide some code that triggers this behaviour?

Not really easy because its a complicated piece of code that this is embedded in.  By the time I write a standalone, anyone should do it by modifying a tutorial, so I'll leave those instructions and move on for the day because this has already cost us a lot of time to track down.

Make an IM client that sets up an IM by calling insert() with some frame that isn't /world.
Make a processFeedback callback for the IM that prints out the feedback header.
make some event (let's say right button up) in the processFeedback that calls setPose() on the IM with the "/world" frame.
Notice that the header stamp is now 0.

There are some related issues we've seen with non-"world" frames, but I won't bring those up now.

Actually, let me mention the related issues.  So, if you set up an IM with let's saw frame "left_plam", you can call setPose() on it all day with poses in the left_palm frame and it works fine.  If you then call setPose() but if a pose and "/world" frame, the IM still works correctly but the feedback header timestamp is 0.  If you call setPose() with any frame other than "left_palm" or "world" then the IM goes "crazy" and the processFeedback always shows "/world" in the header that's returned regardless of the frame.  It's like anytime the frame does not match the startup frame it assumes it must be in /world, which leads to very weird behavior.  The way we are going to get around this is to get the IM, delete it, then readd it in whatever frame we want to change to, but it seems like that is overkill given the setPose() should be able to handle that.  If not, then that needs to be better documented.

I'm starting to this this may be an issue in the RViz side of IMs.

I vaguely remember that I wrote those lines of code, but not in detail all the considerations that went into it. It does seems that this behavior is intentional though. What would the correct behavior be in your opinion?

In my opinion, the time stamp shouldn't be 0, it should be
ros::Time::now().  I'm unsure why it matters that rviz is time synced with
the IM server, I can't see an instance where you would have RViz and IMs
running on different machines since IMs are inherently grounded in Rviz
itself.
On Tue, Nov 27, 2018 at 6:21 AM David Gossow<EMAIL_ADDRESS>wrote:

I vaguely remember that I wrote those lines of code, but not in detail all
the considerations that went into it. It does seems that this behavior is
intentional though. What would the correct behavior be in your opinion?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ros-visualization/interactive_markers/issues/17#issuecomment-442039339,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADrfxml_v1rZeYnWNHwg1xbR7kPLbgnYks5uzS5ggaJpZM4CNUyT
.

In a typical scenario, the IM server runs on the robot while RViz runs on a remote machine. This is not an issue when only viewing stuff in RViz, but when RViz sends feedback the different time domains on the two machines start to matter.
Perhaps a way to solve this would be to assign Time::now() inside the IM server before passing feedback on via a callback. Of course this is slightly intrusive and could as well be done in the user code, so perhaps the current behavior should just be better documented?

In a typical scenario, the IM server runs on the robot while RViz runs on a remote machine. This is not an issue when only viewing stuff in RViz, but when RViz sends feedback the different time domains on the two machines start to matter.
Perhaps a way to solve this would be to assign Time::now() inside the IM server before passing feedback on via a callback. Of course this is slightly intrusive and could as well be done in the user code, so perhaps the current behavior should just be better documented?


Parser: Should be able to use arrow functions on anonymous functions

x Is new feature

Description of suggestion or shortcoming:

//This works
a = {|f| normal(f,f)}
b = a -> {|p| p(3)}

//This doesn't
a = {|f| normal(f,f)} -> {|p| p(3)}

I wrote most of these tests, in parse_tests.ts, to try to better understand the error. Didn't get far though.
  describe("variables", () => {
    testParse("x = 1", "(Program (LetStatement :x (Block 1)))");
    testParse("x", "(Program :x)");
    testParse("x = 1; x", "(Program (LetStatement :x (Block 1)) :x)");
    //succeeds
    testParse(
      "x = 1 -> {|e| e}",
      "(Program (LetStatement :x (Block (Pipe 1 (Lambda :e :e)))))"
    );

    //succeeds
    testParse(
      "{|e| e} -> {|e| e}",
      "(Program (Pipe (Lambda :e :e) (Lambda :e :e)))"
    );
    //succeeds
    testParse("a = {|r| r}", "(Program (LetStatement :a (Lambda :r :r)))");

    //fails
    testParse(
      "a = {|r| r} -> {|fn| fn(3)}",
      "(Program (LetStatement :a (Block (Pipe (Lambda :e :e) (Lambda :e :e)))))"
    );
    testParse(
      "{|r| r} -> {|fn| fn(3)}",
      "(Program (Pipe (Lambda :r :r) (Lambda :fn (Call :fn 3))))"
    );
  });


1_Filter_otu_table.R:

Filter OTUs present in only one sample from relative abundance table

We haven't fixed this issue.  Right now, the code calculates read sum, but it does not calculate what we need, that is taxa sum
‘‘‘
Remove OTUs that appear only in 1 sample (using presence/absence)
any(taxa_sums(phyloseq.rel) == 1)
otu_table(prune_taxa(taxa_sums(phyloseq.rel) <= 1, phyloseq.rel))
phyloseq.rel_OTU2 <- prune_taxa(taxa_sums(phyloseq.rel) > 1, phyloseq.rel)
‘‘‘


ExternalInterfaceAttribute

In Bridge 15, interfaces implementation is chnaged to support all possible .NET cases
In 1.14, some cases are not supported, for example
public interface I {
    void Foo ();
}

public class A : I {
    public void Foo () {
        Console.WriteLine("A");
    }
}

public class B : A, I {
    void I.Foo () {
        Console.WriteLine("B");
    }
}

If create instance of B class and assign to I type variable then during compilation time we don't know which Foo method invocation should be generated (should be B.Foo which has foo$1 name in javascript because foo name is already reserved by A.Foo method).
I i = new B();
i.Foo(); // should be translated to i.foo$1() but Bridge doesn't what exactly instance is assigned to `i` variable

Bridge 15 introduces interface member alias to avoid such collisions. For each interface member implementation will be generated alias like "I$foo" which will be referenced to implementation
I i = new B();
i.Foo(); // translated to `i.I$foo()`

If interface is external (created without Bridge type system) and its implementation is also external (no generated aliases) then such interface must marked by ExternalInterfaceAttribute to instaruct Bridge that aliases are absent. In this, case Bridge will generate special code to get member in javascript (Bridge will check alias and direct name)
i[Bridge.geti(i, "I$foo", "foo")]()

http://forums.bridge.net/forum/bridge-net-pro/bugs/2719-members-on-external-interfaces-are-not-correctly-specified-in-the-javascript

Implemented with #1025 issue


Grades: Course grades not working in edX

We recently created this course to test the automatic certificate generation that was done in https://github.com/mitodl/mitxonline/pull/722. Later on, we noticed that the grades are always 0 when we hit the grades API from edX. This is not the problem with MITxOnline since the grades within edX are 0 in the progress tab. However, overriding grades through gradebook works fine.
Steps to Reproduce

Enroll in this course and go through the outline while attempting some basic problems
Go to Progress tab and notice the grades will not be reflected in the progress (It will always show 0)

Expected Behavior

The grades in the Progress tab should work

Actual Behavior

Grades don't work, When you go through some units in a course and give the right answers the progress isn't reflected in Progress tab and hence no grades are returned to the Grades API in MITxOnline.

Posting Update:
While looking through this issue I was able to identify a few things:

We have persistent grades enabled for all courses in RC
Persistent grades are now enabled by default in edX
With persistent grades enabled, edX starts a task recalculate_subsection_grade_v3 which calculates the grade and saves to the DB.

In our case, we had the persistent grades enabled for all the courses on RC. We also added course level grades enable for a course and I created another entry here. Although, with persistent grades enabled we shouldn't need to add course level enable flags.
While looking around, i could see the grades update task being called but the grades in progress weren't updating.
But as of now, the grades are back to working i suspect that the grade update tasks might have been pushed to the queue which might have other tasks running at that moment. But currently, i can see the grades working without any code or other configuration changes except adding an entry for my Certificate Test Course in here.
Some good reads that also helped were:

Persistent Course Grade (Atlassian Doc)
Discussion on persistent grades on edX discussions

Closing this issue since it's working now. We might reopen if it stops working again and circle back to this comment.


Can not view the chart inline in jupyter notebook

Hi,
I opened your 'maps' notebook, I can't view the plot inline in this notebook. I just got blank output for your info please.
I attached the screen shot
Thanks

Hi - this is a github issue. Github's notebook viewer does not allow execution of javascript within notebooks for security reasons, and Altair charts are rendered with Javascript. There's nothing we can really do about that on our end. I'd recommend either downloading the notebooks and viewing them with JupyterLab locally, or using a service like http://nbviewer.jupyter.org which does not remove javascript artifacts from its renderings.

I have the same issue with jupyter notebook too.
I have created a new thred about displaying Altair chart inline in jupyter notebook.
Still conversation is going on there?
Just I get a blank output while I render the chunk of code.

Yes, I saw that thread. It's a different issue than the one you raise here.

In that case it seems to be something about your local browser's security configuration. I'm not sure how to fix it given the information you provided.

Yes, I saw that thread. It's a different issue than the one you raise here.

I do understand, I was under impression that the issue that I am facing might be same as with this too.
Thank you for the reply.
However I can work with jupyter lab with mimetype renderer. Good luck


Updating to the latest avro and addition of decimal logical schema

This is related to this PR: https://github.com/premsantosh/avro/pull/1

The change looks good.


[UTM] Drag & drop VMs

Copy VMs from another app.

How would this work ?

This has been implemented a while ago.


sickbeard: migrate to Homebrew

Deleting sickbeard from cask as part of deduplication process . See context here.
Please note that this PR is created to continue case-by-case discussions from thread linked above and if decision is made not to remove the cask, PR will be closed.

Thank you for the contribution. It needed some fixes, so they were made in b8feea7b9056689a081f781a0f0dba523c965eb5. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


Improve readme

Remove godoc badge
Show output
Show local + docker usage together
Fix local install instructions (go install)
Add contributing section
Install before usage?
Explain docker volume
Exit code 0 and stderr+stdout behaviour
Ordering
Show help output? or include details of output formats

Others from https://gist.github.com/alcortesm/f45a5f6b5e1543be3f1f88dde43cdecd
Thanks @rossmcf and @alcortesm

Please also do explain what algorithm(s) are used for searching and identifying the licenses. It's quite important when evaluating various "license detectors", as licenses can be hidden in a lots of non-standard places, such as file headers, etc; also subdirectories can have different licenses, also I'd like to know if the tool detects non-standard licenses (e.g. commercial ones) to warn me about them, etc, etc. Otherwise it's "the tool detected something, but I don't know if/to what extent I can rely on that". For example, go-license-detector has a rather nice explanation, from which I can see that it seems to only focus on LICENSE file, ignoring any subdirectories IIUC. This clearly communicates to me its features and its limitations.


Profile header sizing is incorrect after editing bio

If your bio increases in length after editing it then the header will grow in size but the tabview below stays full height, resulting in clipped content within. Might be a GeometryReader problem.
We ideally want a sticky header that collapses to a minimal version on scroll to preserve screen space. Perhaps we can work around the issue by implementing that.

https://exyte.com/blog/swiftui-tutorial-scaling-header

@justinabrahms found an additional wonky behaviour:

note for later: display jitter bug when you have a bio, then click into a follower then go back.

It's not just editing the bio, seemingly having one causes strange layout jitter.


Hello, @zzw922cn, when I run speechvalley/main/madarian_train.py, error as follow , how can I slove it?

NotADirectoryError: [Errno 20] Not a directory: '/home/data/data_thchs30/cha/train/B6_286.wav.trn'

@KyrieHee, hi, have you successed to train madarian model yet? could you share your script for prepreocess dataset?


SyntaxError in Python 3

The function codecs.span_context_to_string has the following line in it:
parent_id = parent_id or 0L
This is invalid syntax in Python 3 as per PEP 237, as can be seen here:
SyntaxError: invalid syntax  Traceback (most recent call last):    File "/usr/local/lib/python3.6/site-packages/jaeger_client/__init__.py", line 32, in <module>      from .tracer import Tracer  # noqa    File "/usr/local/lib/python3.6/site-packages/jaeger_client/tracer.py", line 34, in <module>      from .codecs import TextCodec, ZipkinCodec, ZipkinSpanFormat, BinaryCodec    File "/usr/local/lib/python3.6/site-packages/jaeger_client/codecs.py", line 140      parent_id = parent_id or 0L                                ^  SyntaxError: invalid syntax
Instead of 0L in should just be 0. Not sure how to make in compatible with both 2 and 3 though.

Okay, it seems there are various Python 3 incompatibilities in here. My guess is that this is because of Thrift? From what I can see Thrift is Python 3 compatible from version 0.10.0.
What would be needed to get jaeger_client to be Python 3 compatible?

the setup.py does not pin thrift to 0.9, and if I look at the latest build from master, I can see that it is in fact thrift 0.10 that is installed. The crossdock test includes an end-to-end test where the client communicates with jaeger-agent, so I think it is safe to assume using thrift-0.10 is fine and compatible with Jaeger backend (even if jaeger-agent is using 0.9)

@drzero42 @yurishkuro I think I have fixed most of the incompatibilities in #43 - feel free to take a look.

$ python --version
Python 3.6.1
pip freeze | grep jaeger
jaeger-client==2.1.0
$ pip freeze | grep thrift
thrift==0.10.0
And I still experiencing the error:
python3.6/site-packages/jaeger_client/codecs.py", line 88 parent_id = parent_id or 0L

this was fixed in #57, but we haven't released a new version since the work is not finished.
There doesn't seem to be any new information in this ticket, so I am going to close it as a dupe of #59.


Missing "montecarlo.jl" script referenced by docs (and other minor issues)

Thanks @davidanthoff and co. for putting this package together. Very cool.
I've been playing around with it today and noticed a few niggles that are probably due to the migration over to Julia >1.0.0.
For example, the "Getting Started" document mentions a montecarlo.jl file for setting the number of Monte Carlo runs. This file no longer appears anywhere in the repo. To be fair, I don't think it is needed anymore since users can just run (say) do_monte_carlo_runs(1000) directly after loading the src/mcs.jl script. OTOH, it would probably be helpful to let new users know that they need to run this last line in order to actually implement the stochastic version of the model.
FWIW, the link to the documentation on the package README also appears to be broken, although these are all viewable in the docs/src directory.
Another minor issue is that new Julia users should probably be advised that they need to load the Pkg package (i.e. using Pkg) prior to running commands like Pkg.add(...). I also had to load the Dates package for date times to work, which I don't think was a requirement with older versions of Julia.
I can fold these suggestions into a minor PR if you want, but they should be pretty quick to implement on your side too.

I'll assign this to @lrennels, who introduced MC changes.

@grantmcdermott Thank you so much for pointing these out.  A few of these edits I have fixed but won't be integrated into the built docs until our new version is tagged sometime this week, I'll take a look at your other suggestions now so that they get folded in to the next tag (which should occur sometime next week, I believe).  Don't hesitate to drop any other suggestions here, as I'm currently working on making sure all of the documentation is fixed up post-merge and also working on a set of more in-depth tutorials.

To address:

@grantmcdermott also if you are new to Julia 1.0, note that the use of ] to enter Pkg REPL-mode is quite handy I've found, you can read more here:https://docs.julialang.org/en/v1/stdlib/Pkg/index.html

@jrising I'm working on these in #163

Excellent, thanks @lrennels! I'm still catching up to some of the changes in Julia 1.0, so that ] tip for entering the Pkg REPL-mode is much appreciated.
Please feel free to close this thread as resolved whenever you want, or as soon as the new PR is complete.
P.S. I've been comparing the Excel @RISK version of PAGE09 (generously provided to me by Chris Hope for an older project). The speed gains from this Julia / MIMI version are huge... at least an order of magnitude faster. Open source FTW.


Unsound GPU driver version scheme assumption in StringToDriverVersion

Tensorflow's cuda_diagnostics.cc here assumes that the GPU driver version will match %d.%d.%d but that doesn't seem to be a safe assumption.
2017-05-04 09:48:39.543664: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:189] libcuda reported version is: Invalid argument: expected %d.%d or %d.%d.%d form for driver version; got "3<IP_ADDRESS>"

In FindDsoVersion(), TF is parsing the name of the loaded libcuda library to get the version number, and the current Nvidia web driver library is:
/Library/Frameworks/CUDA.framework/Versions/A/Libraries/libcuda_3<IP_ADDRESS>_mercury.dylib

Would you care to send a PR?


Rework cube hints & add item location/zone switch

Change the interaction such that barren hints factor their goddess cubes in as well and sots hints pointing to goddess chests display the zone of the associated cube instead of the chest. Also adds an option for modifying the sots hint to say it involves a cube.

This is somewhat off-topic for the rest of this PR but it is just an option being added so it shouldn't conflict with anything else.

In general you shouldn't make 2 completely separate changes in the same PR, opening a new one is usually better or we end up with gui-redesign again

My main concern was that I wanted to make the zone item hints compatible with new cube handling, and I was unsure if I made a new branch based off of this one if that meant we'd just end up with all the same commits again, plus the new one


Want to send Emails to Multiple People

As we have option in Alert (Classic) to provide the multiple Emails by semicolon separated email ID's same option is not there in Latest Alert.
RBAC is there but we want only Few People to send the mails, As in bigger Organizations we don't have much control  over Access for creating groups.
If we will give some group name also then unnecessary people will get the alert as they have inherited access on resource group.

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

ID: b3eff5cd-b885-2e94-2ea6-3f60154123df
Version Independent ID: ec1862a7-fa36-1ac4-fe68-011ed39c16f3
Content: Overview of alerting and notification monitoring in Azure
Content Source: articles/azure-monitor/platform/alerts-overview.md
Service: azure-monitor
Sub-service: alerts
GitHub Login: @rboucher
Microsoft Alias: robb

@MrLokeshV Thanks for the question. Would it be possible for you to create a new group and add all those users who you want to send the alerts?

@rboucher Would you have any other suggestions for this scenario?

@SadiqhAhmed-MSFT As i mentioned it is difficult to get the access for group creation. For one time if we will get the group created also then if every time if we wanted to add or remove any member we have to take permission again and again, So it will be better if we have one option to add multiple mails id's as we have before in classic alert.

@MrLokeshV Is there group we are talking about an "Action Group" or some other type of distribution or RBAC group? Using action groups, you can send emails to multiple people.  Each one is just on a different line vs. separated by a semi-colon. You can send it up to 1000 people per https://docs.microsoft.com/en-us/azure/azure-monitor/service-limits.   Let me know if I've missed something.

@MrLokeshV  Can you clarify and let me know if the above solves your problem?

We will close this issue now.  Let us know if you this didn't answer your question. @please-close


OpenStack boot test is flaky

The OpenStack boot test is flaky and doesn't output enough information to investigate the issue. For example:
=== RUN   TestImages
=== RUN   TestImages/fedora_32-x86_64-openstack-boot.json
=== RUN   TestImages/fedora_32-x86_64-openstack-boot.json/image_info
=== RUN   TestImages/fedora_32-x86_64-openstack-boot.json/boot
    main_test.go:192: ssh test failure, 20 attempts were made
--- FAIL: TestImages (516.02s)
    --- FAIL: TestImages/fedora_32-x86_64-openstack-boot.json (515.95s)
        --- PASS: TestImages/fedora_32-x86_64-openstack-boot.json/image_info (25.59s)
        --- FAIL: TestImages/fedora_32-x86_64-openstack-boot.json/boot (266.80s)
FAIL

@atodorov do you mind taking a look at this one?

FTR none of the boot tests provides any information in case ssh attempts fail so it could be a multitude of issues here.

@atodorov do you mind taking a look at this one?

Do you mind providing some more info:

Which PR/branch it failed
which jobs
How often/since when did you start seeing this issue
Which commit/osbuild-composer version is being tested

Do you mind providing some more info:

Sorry for not answering earlier. I did see this a couple of times, but it seems to have been temporary. I'm closing this issue, as I haven't seen this in any of the recent PRs.


Could not run [open -a Safari on windows

how to open with chrome or safari in windows?

You need to set the command to open a browser in ~/.sbt/gilt/sbt-dependency-graph-sugar-cmd. See README.


Better logging systems

Set up S3 buckets to collect logs
Stream system logs to S3 instead of storing them on the EC2 instance
Set up lifecycle policies on S3 to clear out logs periodically. Any suggestions for how long logs should be stored?

cc/ @hotosm/tech

How did we deal with system logs earlier?


Add "Parse CSR"

Operation to parse Certificate Signing Requests for basic information:

CommonName
Locality
State
Country
EmailAddress
Subject Alternative Names

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @arnydo, this looks great but I'm having trouble getting any example CSRs to parse correctly. Could you take a look and include some tests please?

Ah! It looks like it is expecting specific extensions to be present (subjectAltNames, etc). Ill work on adding some conditions...

Hi @arnydo, are you still intending on working on this?

Hi @n1474335 I haven't had a chance to dig into this and if others don't seem to have a need for it then we can drop it.

Closed as stale


Introduce Key

Partially addresses #398
The new Key replaces UniqueKey and has two type parameters: Key[KT, T]

KT identifies whether the table has a Simple or Composite key
T is the type of key values. In the Composite case, it is actually (T', T")

KT is just a phantom type to make sure a key is "correct by construction". This changes ripples through the whole API, notably Table is now indexed by these two types to make sure all APIs relying on keys receive correct values.
This is still a WIP as I want to tackle Query in here too, because now we can enforce the query* APIs can only be called when the table has a composite key at compile time, which I think is great.

❤ 0c4d3f0b98140b946c7a570ee074b6d3d1edb470
Why calling it IsSimpleKey instead of SimpleKey?

Naming, right 😆 SimpleKey is shorter 👍 I'm not sure about whether or not we should give access to it or keep it private. There's only a few simple types, but people might create types that map to them, e.g. UUID -> String. We can't currently represent that because DynamoFormat.xmap loses that information, but it would be to address that I think.


WIP: mockingbird v0.0.1 release

fixes #8

Summary by CodeRabbit
Release Notes

New Features

Introduced a new LICENSE file specifying the MIT License terms.
Added documentation for using the mockingbird-cljs NPM package, including installation and usage examples.
New nav-link function created for modular navigation links in the navbar.
Enhanced message component to display the current date.

Improvements

Updated README.md for clarity on npm usage and prerequisites.
Restructured shadow-cljs.edn to include a new build configuration.
Improved error handling in various components and functions.

Bug Fixes

Adjusted installation instructions in README.md to include missing dependencies.

Chores

Cleaned up package.json by removing duplicate dependencies and updating scripts.
Removed outdated webpack.config.js and docs/Roadmap.md files.

@coderabbitai configuration

@coderabbitai review

@coderabbitai show config

🎉


ignore payload address type if derivation strategy is BIP45

Looks great. Thanks for this. This one line fix will help a lot of users.

Has this been merged and deployed? I'm still having the issue.

I was able to get the recovery tool to find the BTG Multi-Signature addresses.  The recovery tool wants me to send the coins to a new destination.  I tried.  It failed.  Seems that the raw transaction sent to https://btgexplorer.com/api/tx/send came back with
"66: min relay fee not met. Code:-26"
Thus, it's good the coins are there but I need to get more control over the transaction fee to send them on.  Any guidance?  It's really my first time using the recovery tool.

PR #107 addresses this and needs to be merged.

bump

morrisonbrett, does that pull request do anything with BCH (or BCC? I don't know which is proper anymore) mining fees? See my dilemma in the issues, basically I think my transaction is in limbo due to an absurdly low transaction fee. :/

@shawnp0wers, #107 only changes the miners fee for BTG: https://github.com/bitpay/copay-recovery/compare/master...robert-claypool:master


_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m 27s
master passed

+1 :green_heart:
compile
0m 23s
master passed

+1 :green_heart:
checkstyle
0m 14s
master passed

+1 :green_heart:
spotless
0m 56s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
0m 27s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 39s
the patch passed

+1 :green_heart:
compile
0m 19s
the patch passed

+1 :green_heart:
javac
0m 19s
hbase-hadoop-compat generated 0 new + 2 unchanged - 51 fixed = 2 total (was 53)

+1 :green_heart:
checkstyle
0m  7s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
14m  6s
Patch does not cause any errors with Hadoop 3.1.2 3.2.2 3.3.1.

+1 :green_heart:
spotless
0m 44s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
0m 27s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 10s
The patch does not generate ASF License warnings.

31m 16s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4648/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4648

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 3e2592dbccc1 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / b1706a8a12

Default Java
AdoptOpenJDK-1.8.0_282-b08

Max. process+thread count
64 (vs. ulimit of 30000)

modules
C: hbase-hadoop-compat U: hbase-hadoop-compat

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4648/1/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Fixed #354 - Add standalone Path and filer-buffer modules

r+ with travis.  @gideonthomas can you r?

nit: I don't think we need to prefix buffer as filer-buffer if we're not doing it for path. Either way, r+


AMQP-602: Add Exchange internal option

JIRA: https://jira.spring.io/browse/AMQP-602

Add internal property to the Exchange abstraction
Use the proper Channel.exchangeDeclare() method to address an internal option too.
Expose internal attribute to the XML components for Exchange
Add Documentation and tests

Polishing here: https://github.com/garyrussell/spring-amqp/commit/f9c09caaa681f7a0cc59f17d6aaea50d7b94ef6a

Squashed, rebased, merged as 7f8d520997d6f4c02809824d8e2cdf8a55a45abd


Produce IntelliSense comments from winmd custom attributes

Fixes #804
Description
Implements proposal to leverage any attribute named HelpStringAttribute (in any namespace).
This PR will produce comments in the projected code of the form /// <summary>comment</summary>, where comment is a string stored in a custom attribute named HelpStringAttribute.
NYI

cppwinrt has some text-replacement logic (^, %, @) which the doc strings need to be exempted from.
XML comments seem to work in VS but they're not officially supported (feature request filed)
test which should check that when building with /doc, the intellisense XML produced has the right entries.

Demo
Assume this IDL:
namespace Windows.Foundation.Metadata
{
    [attributeusage(target_runtimeclass, target_interface, target_struct, target_enum, target_delegate, target_field, target_property, target_method, target_event)]
    [attributename("helpstring")]
    attribute HelpStringAttribute {
      String Content;
    }
}

namespace test_component
{
    [helpstring("this is an interface")]
    interface InterfaceWithDoc
    {
      [helpstring("this is a function with docs")]
      void f();

      [helpstring("this is a property")]
      Int32 InterfacePropertyWithDoc;
    };

    [helpstring("this is an enum")]
    enum EnumWithDoc
    {
      Dog,
      Cat,
      [helpstring("this is a gecko")] // xmldoc in enum values aren't properly displayed by VS yet
      Gecko
    };

    [helpstring("this is a struct")]
    struct StructWithDoc
    {
      [helpstring("this is the value")]
      Int32 val;
    };

    [helpstring("this is a class")]
    runtimeclass ClassWithDoc
    {
      [helpstring("this is a property")]
      Int32 SomeValue;

      [helpstring("this is a method")]
      void g();
    }

With this you get these kind of suggestions:
on an interface

on a method in an interface

on a property in an interface

on enums

on enum values

on structs

on struct fields

on classes

on a method in a class

on a property in a class

This is very cool. Just want to set expectations. My main reservation is that this is a feature that has zero adoption currently. We added a few features over the years that were meant to be used by various key customers and gain adoption but ended up not being used at all. Such features add considerable liability to the code base and supportability of cppwinrt. Let's see what we can do about getting more support for this idea before committing this change. Happy to discuss further.

thanks @kennykerr! We discussed this in the api review meeting and folks seemed supportive. Would that count as enough, or is there something else you'd like to see to make this official? :)

@AArnott raised some questions about scalability.
There's also the question of who or how we are going to get doc comments into the IDL files for APIs and when that will happen.
This would also have to go through @sotteson1 as he ships out platform metadata.
We'd also have to work with DevDiv to improve the visualization as we should not be showing tooltips that refer to the implementation type within C++/WinRT.
Finally, we are moving away from any tools that are specific to Visual Studio as we don't have the resources to manage that. I would rather see a solution to this problem that is independent of the cppwinrt compiler itself. Perhaps a Visual Studio or VS Code extension that offers this capability without having to inject code into the already very large headers.

Closing the PR for now. Feel free to continue the discussion on the issue: https://github.com/microsoft/cppwinrt/issues/804


SAK-EFK Filebeat config example in README.md seems not to be correct

Regular expressions in SAK-EFK module for Filebeat config example seems not to be valid.
The regular expressions used in https://github.com/provectus/sak-efk/blob/master/README.md are:
"message" = "(?i)kube-probe/1.18+"
"message" = "(?i)ELB-HealthChecker/2.0"
According to Filebeat docs Golang flavor is used (https://pkg.go.dev/regexp/syntax?utm_source=godoc) so "." and "+" should be escaped like this:
"message" = "(?i)kube-probe/1.18+"
"message" = "(?i)ELB-HealthChecker/2.0"

"message" = "(?i)kube-probe/1.18+"
"message" = "(?i)ELB-HealthChecker/2.0"


Running org.scalatest.tools.Runner with an invalid Reporter shows exception but then hangs

I have a custom reporter that I provide to scalafmt using the CLI (no SBT or other build tool involved). This is a bit problematic because if I screw something up in CI, my CI just hangs. It would be ideal if the Runner would exit with a non-zero code rather than hanging.

@jackkoenig Sorry for replying late on this, I am curious what is causing the hang, is it still possible to get the stacktrace of the last error when it hangs?
Cheers.

No worries and thanks for your response @cheeseng! I should have included it before, here's the stack trace:
An exception or error caused a run to abort. This may have been caused by a problematic custom reporter.
java.lang.IllegalArgumentException: Couldn't load a Reporter class: "my.custom.ReporterButMisspelt".
Problem arg: -r... my.custom.ReporterButMisspelt
        at org.scalatest.tools.ReporterFactory.getCustomReporter(ReporterFactory.scala:39)
        at org.scalatest.tools.ReporterFactory.createCustomReporter(ReporterFactory.scala:213)
        at org.scalatest.tools.ReporterFactory.getReporterFromConfiguration(ReporterFactory.scala:246)
        at org.scalatest.tools.ReporterFactory.$anonfun$createReportersFromConfigurations$1(ReporterFactory.scala:254)
        at scala.collection.TraversableLike.$anonfun$map$1(TraversableLike.scala:286)
        at scala.collection.Iterator.foreach(Iterator.scala:943)
        at scala.collection.Iterator.foreach$(Iterator.scala:943)
        at scala.collection.AbstractIterator.foreach(Iterator.scala:1431)
        at scala.collection.IterableLike.foreach(IterableLike.scala:74)
        at scala.collection.IterableLike.foreach$(IterableLike.scala:73)
        at org.scalatest.tools.ReporterConfigurations.foreach(ReporterConfiguration.scala:42)
        at scala.collection.TraversableLike.map(TraversableLike.scala:286)
        at scala.collection.TraversableLike.map$(TraversableLike.scala:279)
        at org.scalatest.tools.ReporterConfigurations.map(ReporterConfiguration.scala:42)
        at org.scalatest.tools.ReporterFactory.createReportersFromConfigurations(ReporterFactory.scala:253)
        at org.scalatest.tools.ReporterFactory.getDispatchReporter(ReporterFactory.scala:257)
        at org.scalatest.tools.Runner$.withClassLoaderAndDispatchReporter(Runner.scala:1480)
        at org.scalatest.tools.Runner$.runOptionallyWithPassFailReporter(Runner.scala:971)
        at org.scalatest.tools.Runner$.main(Runner.scala:775)
        at org.scalatest.tools.Runner.main(Runner.scala)
Caused by: java.lang.ClassNotFoundException: <my.custom.ReporterButMisspelt>
        at java.base/java.net.URLClassLoader.findClass(URLClassLoader.java:471)
        at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:589)
        at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:522)
        at org.scalatest.tools.ReporterFactory.getCustomReporter(ReporterFactory.scala:29)
        ... 19 more

This is using scalatest 3.2.6 but the same issue manifests with 3.2.10.

@jackkoenig Fyi I submitted the following PR for this:
https://github.com/scalatest/scalatest/pull/2090

Thank you!

@jackkoenig Fyi this should be fixed now in 3.2.11, do you mind to verify and close this ticket if it works?

I can confirm this is indeed fixed. Thank you!


Update JSON Tutorial Code for Python 3

Thanks for writing such a helpful tutorial!
The existing code snippets which demonstrate the Transformer class rely
on tuple parameter unpacking, which was removed as of PEP-3113, which
landed in Python 3.0.
https://www.python.org/dev/peps/pep-3113/
The class methods don't make use of any internal data, so they can be
marked with @staticmethod.
The updated versions of these code snippets continue to work as expected
in Python 2.7.16, noting the difference in how unicode strings are
marked.
The code snippets copy and paste cleanly into the Python repl.

Thank you for the update.
Regarding the tuple unpacking, Lark provides a utility for that  @v_args(inline=True).
I think it's more readable, so it'd be great If you could use that instead.

Updated examples to use the @v_args decorator.
Updated the benchmark example at the bottom.
Verified copy-paste in both the Python 3.7.3 and 2.7.16 repls.

Does it mean that rename v_args(inline=True) and use @inline decorator instead?

@erezsh What do you think of the changes?
The TreeToJson transformer could be made more brief, by doing the following:
null = None
true = True
false = False

However, I am not sure if this helps or hinders understanding.

The changes are good, but the code is a little verbose. After all, this is supposed to be a tutorial for beginners, so being clear and simple is very important, especially in the code. I'll have a look at it soon.


why do you use RethinkDB

Hi, I am reading your project and I love this project
I have a question, why do you choose RethinkDB in your project instead of other NoSQL like Redis, Cassandra , are there any special features that others dont have it. Can you share me it. I am wondering when I should choose RethinkDB instead of your project instead of other NoSQL like Redis, Cassandra

Hello, sorry about the late reply. I used Rethink because it was a close
open source alternative to MongoDB. When I established my company for a
product I had been developing in Mongo, I realized they went to a different
license model than they used to have years ago. I know a lot of other
people have gone to Cassandra, but I felt a little more at home with
Rethink primarily because of the ability to easily store JSON data. I was
on a little bit of a time crunch, too, so I didn't really get super far
into Cassandra.
On Tue, Jan 31, 2023 at 1:43 AM Mary Lilian @.***>
wrote:

Hi, I am reading your project and I love this project
I have a question, why do you choose RethinkDB in your project instead of
other NoSQL like Redis, Cassandra , are there any special features that
others dont have it. Can you share me it. I am wondering when I should
choose RethinkDB instead of your project instead of other NoSQL like Redis,
Cassandra
—
Reply to this email directly, view it on GitHub
https://github.com/rpsirois/express-session-rethinkdb-esm/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKOABYPJMCO4IY2ZDLX533WVDGBXANCNFSM6AAAAAAUMEONXY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Robert Sirois
https://universesgames.com https://www.universesgames.com


python310Packages.elastic-apm: 6.13.2 -> 6.14.0

Diff: https://github.com/elastic/apm-agent-python/compare/refs/tags/v6.13.2...v6.14.0
Changelog: https://github.com/elastic/apm-agent-python/releases/tag/v6.14.0
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 213927 run on x86_64-linux 1

2 packages built:

python310Packages.elastic-apm
python311Packages.elastic-apm


Some getting started doc for AWS

This pull request is old but I think still valid - maybe @tobiasmcnulty and/or @copelco would like to review?


Update workflows.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (e366320) 91.19% compared to head (d89589d) 91.19%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #111   +/-   ##
=======================================
  Coverage   91.19%   91.19%
=======================================
  Files           4        4
  Lines         602      602
  Branches       45       45
=======================================
  Hits          549      549
  Misses         53       53

Flag
Coverage Δ

unittests
91.19% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


manual linting with flake8

Codecov Report

Merging #42 (a3c1681) into main (94a9491) will increase coverage by 0.12%.
The diff coverage is 77.04%.

@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
+ Coverage   77.64%   77.77%   +0.12%
==========================================
  Files          14       14
  Lines        1812     1795      -17
  Branches      233      230       -3
==========================================
- Hits         1407     1396      -11
+ Misses        320      314       -6
  Partials       85       85

Impacted Files
Coverage Δ

src/mud_examples/parsers.py
100.00% <ø> (ø)

src/mud_examples/runner.py
80.80% <ø> (ø)

src/mud_examples/utils.py
94.28% <ø> (ø)

src/mud_examples/plotting.py
46.66% <7.14%> (+1.66%)
:arrow_up:

src/mud_examples/linear/lin.py
80.30% <95.45%> (-0.09%)
:arrow_down:

src/mud_examples/linear/models.py
64.86% <100.00%> (ø)

src/mud_examples/monomial.py
95.60% <100.00%> (-0.10%)
:arrow_down:

src/mud_examples/ode.py
79.41% <100.00%> (ø)

src/mud_examples/pde.py
73.17% <100.00%> (-0.64%)
:arrow_down:

src/mud_examples/poisson.py
84.75% <100.00%> (ø)

Continue to review full report at Codecov.

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


im sure this has been done before : Custom List not working

I did the custom list thing to remove the bloating (Selected Minecraft, Spotify, and Gaming Services to not be removed) and restarted. All of the bloat was back from my previous attempts to remove it was back. Now when I went to start Minecraft with the launcher, it wouldn't start because the gaming service would just close itself when started. I luckily did a restore point and got back to before I used this.
edit : I have figured out that its not just the Minecraft Launcher thats the issue, Its the entire Microsoft Logging In process. I can't log in on my microsoft account, it will say I am logged in but signing in on the Minecraft Launcher, it will say its missing a file needed to sign in or something.

Currently having the same problem. I looked through everything in the list and unchecked some programs. Spotify and such were not removed, but Windows Store and Minecraft were even though I unchecked them.
The custom checklist not only does not work, but has undefined behaviour and now i don't even know what works or not.

Hm..... I have a clue, now, why it might not work. Package names changed, probably. I'm betting Microsoft Store and Minecraft had some kind of AppX update that renamed their package. I have no clue how to fix it, though. I don't know the new package names.

Hm..... I have a clue, now, why it might not work. Package names changed, probably. I'm betting Microsoft Store and Minecraft had some kind of AppX update that renamed their package. I have no clue how to fix it, though. I don't know the new package names.

I did a fresh install of windows 10 and I got it working great now I just gotta do everything I prev did for the past year

I bet the new launcher starts some process that the script doesn't recognise. The new launcher is flimsy, I haven't even installed it yet, despite being a huge Minecraft fan.


Attempting to control Daikin AC's via SmartIR and GlobalCache IP2IR

Home Assistant version  2024.6.3
SmartIR version  1.17.16
SmartIR configuration only one of 8 devices shown
climate:

platform: smartir
name: l1_rumpus_ac
unique_id: l1_rumpus_aircon
device_code: 1101
controller_data: remote.L1 Rumpus AC        #this is the global cache device

the https://www.home-assistant.io/integrations/itach device configuration appears to be working) - only one of 8 devices shown here.  "Note: Global Cache devices expect data in their own format of “sendir…”. This integration converts hex code to Global Cache IR form."  this means that SmartIR needs to send code to the "remote" in HEX format (which I think it does)
remote:

platform: itach
host: <IP_ADDRESS>
devices:

name: L1 Rumpus AC
modaddr: 1
connaddr: 1
ir_count: 3

Describe the bug
the "thermostat" card appears to be attempting to send the message via smartir but something is not working between smartir and global cache..The global cache device is functioning.  (meaning I can force it to operate via the "remote: send command" service)
Debug log
This error originated from a custom integration.
Logger: custom_components.smartir.climate
Source: custom_components/smartir/climate.py:591
integration: smartir (documentation, issues)
First occurred: 10:36:13 (3 occurrences)
Last logged: 10:36:16
not a valid value for dictionary value @ data['entity_id']
Traceback (most recent call last):
File "/config/custom_components/smartir/climate.py", line 591, in _send_command
await self._controller.send(
File "/config/custom_components/smartir/controller.py", line 117, in send
await self.hass.services.async_call("remote", "send_command", service_data)
File "/usr/src/homeassistant/homeassistant/core.py", line 2706, in async_call
processed_data: dict[str, Any] = handler.schema(service_data)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/voluptuous/schema_builder.py", line 272, in call
return self._compiled([], data)
^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/voluptuous/validators.py", line 229, in _run
return self._exec(self._compiled, value, path)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/voluptuous/validators.py", line 355, in _exec
raise e if self.msg is None else AllInvalid(self.msg, path=path)
File "/usr/local/lib/python3.12/site-packages/voluptuous/validators.py", line 353, in _exec
v = func(path, v)
^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/voluptuous/schema_builder.py", line 818, in validate_callable
return schema(data)
^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/voluptuous/schema_builder.py", line 272, in call
return self._compiled([], data)
^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/voluptuous/schema_builder.py", line 595, in validate_dict
return base_validate(path, iteritems(data), out)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/voluptuous/schema_builder.py", line 433, in validate_mapping
raise er.MultipleInvalid(errors)
voluptuous.error.MultipleInvalid: not a valid value for dictionary value @ data['entity_id']

Hello,
I would bet, that you gave error in your remote name. Did you consider using quotes, when configuring devices with spaces in the names? This is the standard and required approach in the HA YAML configuration.
controller_data: "remote.L1 Rumpus AC"
Please post you config again formatted as a code, as quotes could be removed by Github.
Regarding you finding, I am sorry, but this is outside of the control of this integration. I you look into the code, you will see this:
        service_data = {
            ATTR_ENTITY_ID: self._controller_data,
            "command": commands,
            "delay_secs": self._delay,
        }
        await self.hass.services.async_call("remote", "send_command", service_data)

So in fact this integration execute standard HA service call "send_command" with "command" as provided in the data device json file and delay set to configured delay parameter. Nothing more, nothing less. I do not have any "Global Cache device" to test with, but as stated above - if you are able to call service send_command directly over your IR remote entity, this integration shall work if you provide correct name of the remote entity.

thank you very much and you were correct - it was the naming convention
that I had used - I now get a clean start up as follows...
"2024-06-23 11:36:21.467 INFO (MainThread)
[homeassistant.components.remote] Setting up itach.remote
2024-06-23 11:36:21.467 INFO (MainThread) [homeassistant.components.remote]
Setting up itach.remote
2024-06-23 11:36:21.467 INFO (MainThread) [homeassistant.components.remote]
Setting up itach.remote
2024-06-23 11:36:21.467 INFO (SyncWorker_7) [pyitachip2ir] tryBeacon:
2024-06-23 11:36:21.467 INFO (SyncWorker_7) [pyitachip2ir] tryConnect:
<IP_ADDRESS>:4998
2024-06-23 11:36:21.468 INFO (SyncWorker_1) [pyitachip2ir] tryBeacon:
2024-06-23 11:36:21.468 INFO (SyncWorker_1) [pyitachip2ir] tryConnect:
<IP_ADDRESS>:4998
2024-06-23 11:36:21.469 INFO (SyncWorker_5) [pyitachip2ir] tryBeacon:
2024-06-23 11:36:21.469 INFO (SyncWorker_5) [pyitachip2ir] tryConnect:
<IP_ADDRESS>:4998
2024-06-23 11:36:21.469 INFO (SyncWorker_7) [pyitachip2ir] checkConnect:
connected
2024-06-23 11:36:21.470 INFO (SyncWorker_5) [pyitachip2ir] checkConnect:
connected
2024-06-23 11:36:21.470 INFO (SyncWorker_1) [pyitachip2ir] checkConnect:
connected"
SmartIR/iTach seems happy to process commands in this format (which I
assume is HEX):
"0000 006D 0000 0022 00AC 00AC 0015 0040 0015 0040 0015 0040 0015 0015
0015"
However, SmartIR is generating commands in Base64 (from json file 1101)
which are being rejected....
2024-06-23 11:37:39.912 INFO (SyncWorker_14) [pyitachip2ir] Unknown
command:b64:JgBQAg0PDg4OEAsQDRAMAANEdDgOKw4PDhAMDw0uDA8MEA0RDA8NLA0PDiwNLA4PDSwOKw4rDSwOLA4ODQ8NLA0QDRAMDw8PDBEMDwwQDREMDw0QDCwNEQwsDg8NEAwQDC0NLA4PDQ4NEQwPDg8MEQwPDhAMEAwQDQ8NEAwPDhAMEA0ODiwOKw0sDg8NLQ0QDCwNLA0ABH5yOg4sDQ4NEA0RDCwODwwQDBENDw0sDQ4PKw4rDRAPKg4rDisOLA4rDREMDw0tDA8NEA4PDQ8NDw0RDA8MEA0RDBAMEAwsDhAMDw4PDBEMLA0QDQ8NLQ0ODRENKw0QDSwODwwQDg4NLA4PDSsOLA4ODREMEAwtDBAOKw4rDw8MLA0sDgAEfnI5DS0ODwwPDRANLA0QDQ8MEA0QDSwNDw0uCy0MEQ0rDisNLQ0sDisNEA4ODS0NDg4QDBAODwwQDBANDw0PDRAODg0QDBANDw0QDBEMDw0RDBAMLA4PDRAMLA4QCxANLA0QDQ8ODwwQDSwNEA4rDg8MEAwQDRANDwwRDBANDwwRDQ8NEAwQDBANEAwQDCwNEQwsDRAMEQwPDRANDw0PDREMEAwPDREMDw4QDA8NDwwRDQ8MEQwtDisODw4PCxAOEAwPDRAMEA0PDg8MEA4rDisOEAwPDQ8NEA0QDBAMEA4PDBAMEQwRCxANDw0QDBEMEAwPDS4MDw0QDBANEAwQDSwNLA4PDBAODwwRCxANEA0QDA8NEA0PDRELEA0PDRELEA0PDREMLA0QDCwNEA8ODCwODwwADQUAAAAAAAAAAA==
I tried converting one command in the climate codes file to HEX and the
command passed through the global cache device but it caused my
homeassistant to reboot!  I tried it several times and am at a loss to
understand what is going on.  The same thing happens when I use the
remote.send_command service call.  Any ideas?  It looks as though SmartIR
is working fine but I need it to generate HEX commands.  (I suspect that
means a custom climate codes file will be required - please confirm that
this is the case).
In summary, I have 2 new issues!
regards
On Sat, 22 Jun 2024 at 04:30, litinoveweedle @.***>
wrote:

Hello,
I would bet, that you gave error in your remote name. Did you consider
using quotes, when configuring devices with spaces in the names? This is
the standard and required approach in the HA YAML configuration.
controller_data: "remote.L1 Rumpus AC"
Please post you config again formatted as a code, as quotes could be
removed by Github.
Regarding you finding, I am sorry, but this is outside of the control of
this integration. I you look into the code, you will see this:
    service_data = {
        ATTR_ENTITY_ID: self._controller_data,
        "command": commands,
        "delay_secs": self._delay,
    }
    await self.hass.services.async_call("remote", "send_command", service_data)

So in fact this integration execute standard HA service call
"send_command" with "command" as provided in the data device json file and
delay set to configured delay parameter. Nothing more, nothing less. I do
not have any "Global Cache device" to test with, but as stated above - if
you are able to call service send_command directly over your IR remote
entity, this integration shall work if you provide correct name of the
remote entity.
—
Reply to this email directly, view it on GitHub
https://github.com/litinoveweedle/SmartIR/issues/38#issuecomment-2183259429,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQEIO6SDSHURI2VUPMI4XADZIRWODAVCNFSM6AAAAABJNBNRGOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBTGI2TSNBSHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Michael Brooks

Hello, sorry to hear about your troubles.
SmartIR only send what is recorded in the json files. So the format of the command has to be correct for your remote type and content of the command has to correspond to the valid command for your AC device. There is nothing SmartIR is doing with the command except passing it to the HA remote send_command service. So if your commands cause HA reboot you need to open issue for HA remote integration here. And you need to do the same if you are unable to record valid commands using HA remote integration learn_command service. This is outside of the SmartIR scope and I can't support it.

thank you for the response.  What you said is what I had concluded.  All  I
need to do now is source the correct codes that will work with
remote.send_command and create a custom_codes json file in smartir for my
situation.
regards
On Sun, 23 Jun 2024 at 18:53, litinoveweedle @.***>
wrote:

Hello, sorry to hear about your troubles.
SmartIR only send what is recorded in the json files. So the format of the
command has to be correct for your remote type and content of the command
has to correspond to the valid command for your AC device. There is nothing
SmartIR is doing with the command except passing it to the HA remote
send_command service. So if your commands cause HA reboot you need to open
issue for HA remote integration here
https://github.com/home-assistant/core/issues. And you need to do the
same if you are unable to record valid commands using HA remote integration
learn_command service. This is outside of the SmartIR scope and I can't
support it.
—
Reply to this email directly, view it on GitHub
https://github.com/litinoveweedle/SmartIR/issues/38#issuecomment-2184907428,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQEIO6RDYDKNHNMPNLFPATDZI2EHHAVCNFSM6AAAAABJNBNRGOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBUHEYDONBSHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Michael Brooks


Translation of UI.

Here are the changes to display the user interface in English and French.

Testing this out, but @foxbike -- you need to sign your commits for DCO (Developer Certificate of Origin).  See the "Details" link above, beside the "DCO" check.

Looks good and ran nicely.  Looks like there are still some things needed:

add a way to get to the landing point in the desired language -- perhaps: http://localhost:4200/en and http://localhost:4200/fr
a way for a specific deployment to pick the default language when just hitting the home page http://localhost:4200
a language selector on the front page
getting the language selector to work from the footer -- at least it didn't work for me.

Great work!

@swcurran definitely those are updates that we're aware need implementation, but we'd prefer to have them addresses in separate tickets to keep PR complexity and volume of changes on merge down since we'll already going to have merge conflicts.


[Serialization] Improve extensions of nested types with the same name

Previously looking up an extension would result in all extensions for types with the same name (nested or not) being deserialized; this could even bring in base types that had not been deserialized yet. Add in a string to distinguish an extension's base type; in the top-level case this is just a module name, but for nested types it's a full mangled name.
This is a little heavier than I'd like it to be, since it means we mangle names and then throw them away, and since it means there's a whole bunch of extra string data in the module just for uniquely identifying a declaration. But it's correct, and does less work than before, and fixes a circularity issue with a nested type A.B.A that apparently used to work.
SR-3915

This looks like it needs a rebase? Excellent work though :)

Yeah, it's "just" the module format version but I'm taking the opportunity to retest locally. Thanks!

@swift-ci Please test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - aabe0ded355e70964241a28819b00dc2af35c144
Test requested by - @jrose-apple

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - aabe0ded355e70964241a28819b00dc2af35c144
Test requested by - @jrose-apple

@swift-ci Please smoke test


"My Wishlist" Coming Under profile icon is not redirecting to Wishlist Page

Describe the bug
Under the Profile Icon the My Wishlist Button is not linking to the Wishlist Page Present
To Reproduce
Steps to reproduce the behavior:

Go to Profile Icon on top right corner after logging in and click on it
Go down to the third bar "My Wishlist Icon" and select it.
Not redirecting to the "My Wishlist" Page"

Expected behavior
It has to redirect to the "My Wishlist Page" but isn't doing it
Screenshots

Please assign the above issue to me.

@aditya-chari25 go ahead

@MAVRICK-1 please check this pull request related to this issue https://github.com/MAVRICK-1/Nest-Ondc/pull/94 .

Issue closed with pr https://github.com/MAVRICK-1/Nest-Ondc/pull/94


How to convert or export xps format from word

How to convert or export xps format from word

No, NPOI doesn't have any support for xps format.


Live observation fixes

What was solved?

Localization changes;
Showing LO indicator for CV engagement;
Showing LO indicator for restored CV engagement;
Showing LO indicator for Chat engagement;
Showing LO indicator based on Site configurations;
Fixed link buttons height.

⚠ Tests will be added in the next PRs
Release notes:

[x] Feature
[ ] Ignore
[ ] Release notes (Is it clear from the description here?)
[ ] Migration guide (If changes are needed for integrator already using the SDK - what needs to be communicated? Add underneath please)

Additional info:

[ ] Tests fixed, added? Unit, acceptance, snapshots?
[ ] Logging necessary for future troubleshooting of customer issues added?

Screenshots:

!squash


Error when using self signed certs

Doing a art config using my companies internal artifactory URL I get the error message below.
It clearly doesn't like the self signed company cert, however I do have the certs imported into my Mac keychain and in the shell in a cert bundle that fixes things like curl and other tools but doesn't seem to work with Golang binaries. Is there a simple fix to expose a cert bundle to Golang tools or is this a larger issue?
art version 1.3.0 (via Homebrew)
Mac OS X: 10.10.5
Thanks!
Password: panic: Get https://redacted/api/security/encryptedPassword: x509: certificate signed by unknown authority

goroutine 1 [running]:
github.com/JFrogDev/artifactory-cli-go/utils.CheckError(0xa6d9f0, 0xc820471f80)
        /private/tmp/artifactory-cli-go20160106-50997-aa066h/artifactory-cli-go-1.3.0/src/github.com/JFrogDev/artifactory-cli-go/utils/utils.go:25 +0x4b
github.com/JFrogDev/artifactory-cli-go/utils.Send(0x459968, 0x3, 0xc820016b00, 0x39, 0x0, 0x0, 0x0, 0x0, 0xc82000b720, 0x1b, ...)
        /private/tmp/artifactory-cli-go20160106-50997-aa066h/artifactory-cli-go-1.3.0/src/github.com/JFrogDev/artifactory-cli-go/utils/ioUtils.go:147 +0x38c
github.com/JFrogDev/artifactory-cli-go/utils.SendGet(0xc820016b00, 0x39, 0x0, 0xc82000b720, 0x1b, 0xc820011400, 0x7, 0xc820011420, 0x8, 0x0, ...)
        /private/tmp/artifactory-cli-go20160106-50997-aa066h/artifactory-cli-go-1.3.0/src/github.com/JFrogDev/artifactory-cli-go/utils/ioUtils.go:116 +0x9c
github.com/JFrogDev/artifactory-cli-go/utils.GetEncryptedPasswordFromArtifactory(0xc82000f270, 0xc8200c2fa0, 0x0, 0x0)
        /private/tmp/artifactory-cli-go20160106-50997-aa066h/artifactory-cli-go-1.3.0/src/github.com/JFrogDev/artifactory-cli-go/utils/artifactoryUtils.go:71 +0xbb
github.com/JFrogDev/artifactory-cli-go/commands.encryptPassword(0xc82000f270, 0xc82000b700)
        /private/tmp/artifactory-cli-go20160106-50997-aa066h/artifactory-cli-go-1.3.0/src/github.com/JFrogDev/artifactory-cli-go/commands/config.go:86 +0x39
github.com/JFrogDev/artifactory-cli-go/commands.Config(0xc82000f270, 0x450101)
        /private/tmp/artifactory-cli-go20160106-50997-aa066h/artifactory-cli-go-1.3.0/src/github.com/JFrogDev/artifactory-cli-go/commands/config.go:30 +0x30f
main.config(0xc8200970e0)
        /private/tmp/artifactory-cli-go20160106-50997-aa066h/artifactory-cli-go-1.3.0/src/github.com/JFrogDev/artifactory-cli-go/art/main.go:268 +0x4b1
main.main.func1(0xc8200970e0)
        /private/tmp/artifactory-cli-go20160106-50997-aa066h/artifactory-cli-go-1.3.0/src/github.com/JFrogDev/artifactory-cli-go/art/main.go:30 +0x21
github.com/JFrogDev/artifactory-cli-go/Godeps/_workspace/src/github.com/codegangsta/cli.Command.Run(0x45b4f0, 0x6, 0x0, 0x0, 0xc820010ea0, 0x1, 0x1, 0x45b4f0, 0x6, 0x0, ...)
        /private/tmp/artifactory-cli-go20160106-50997-aa066h/artifactory-cli-go-1.3.0/src/github.com/JFrogDev/artifactory-cli-go/Godeps/_workspace/src/github.com/codegangsta/cli/com
mand.go:137 +0x107e
github.com/JFrogDev/artifactory-cli-go/Godeps/_workspace/src/github.com/codegangsta/cli.(*App).Run(0xc820096ea0, 0xc82000a5e0, 0x2, 0x2, 0x0, 0x0)
        /private/tmp/artifactory-cli-go20160106-50997-aa066h/artifactory-cli-go-1.3.0/src/github.com/JFrogDev/artifactory-cli-go/Godeps/_workspace/src/github.com/codegangsta/cli/app
.go:176 +0xfff
main.main()
        /private/tmp/artifactory-cli-go20160106-50997-aa066h/artifactory-cli-go-1.3.0/src/github.com/JFrogDev/artifactory-cli-go/art/main.go:53 +0x43d

goroutine 17 [syscall, locked to thread]:
runtime.goexit()
        /usr/local/Cellar/go/1.5.2/libexec/src/runtime/asm_amd64.s:1721 +0x1

https://github.com/golang/go/issues/13335 raises the issue of making crypto/x509.systemRootsPool a publicly accessible method in the https://golang.org/pkg/crypto/x509/ package.

I have a working version here https://github.com/jackspirou/artifactory-cli-go, but it uses the vendor directory instead of Godeps.  The vendor tool I use is https://github.com/govend/govend.

Thanks @kholloway and @jackspirou.
We will address this issue soon.

Hi @kholloway and @jackspirou,
Since The Artifactory CLI is deprecated and replaced with the new JFrog CLI.
JFrog CLI is an evolution of the old tool.
JFrog CLI added support for  self sign certificates .
We recommend migrating to  JFrog CLI.


startOfWeek should accept a Weekday parameter

Some areas of the World have their weeks starting on Sundays, some other on Mondays. Google Calendar even has an option to make them start on Saturday haha.
Anyway, I think the startOfWeek function should accept a Time.Weekday parameter to make it more configurable and locale-agnostic.
Something along this line:
startOfWeek : Zone -> Weekday -> Posix -> Posix

What do you think? I can work on a PR if you agree.
Edit: if accepted, this evolution should also be ported to other functions like endOfWeek, weekdayFromInt and friends.

The current implementation allows you to set the start of the week. The function signature is exactly as you suggested.
For the weekdayFromInt function I was just following the JavaScript standards. I'd recommend just defining your own function if it is different.

Wow. This is were I realize I'm having a tough startOfWeek myself... Sorry for the noise.


Download is very slow

How long is the download taking for you. Does it take a long time to start downloading after you've hit the "download" button? What's the size of the file that downloaded?

How long is the download taking for you. Does it take a long time to start downloading after you've hit the "download" button? What's the size of the file that downloaded?

It took me about 5 minutes to load before it actually started downloading and it was 300 kb +
That does not include the time to start up the server

It took me about 5 minutes to load before it actually started downloading and it was 300 kb +
That does not include the time to start up the server

Okay... I've made some changes to the code, would you mind trying again?

Okay... I've made some changes to the code, would you mind trying again?

OK!

OK!


[WIP] build: add commitlint

Specification for commit messages can be found here: https://www.conventionalcommits.org/en/v1.0.0/#specification

@sneridagh I'd like to start using semantic versioning changelog entries from Volto 4.0.0 on. Let's talk about this tomorrow...

Superseded by: https://github.com/plone/volto/issues/3953


[BEAM-59] Beam FileSystem: ResourceId and its local and gcs implementation.

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @dhalperi @davorbonaci
CC: @jkff

PTAL
Verified locally that followings passed:
mvn clean verify
mvn findbugs:check
mvn checkstyle:checkstyle

please retest this

retest this please

Coverage increased (+0.007%) to 69.698% when pulling 57a9463fb5fa945de692dbc4d04817f01d895504 on peihe:file-system-local-gs-resolve into c0910fdf4939a4db52b4e7c49e3593b6da31694a on apache:master.

@dhalperi
for the question that whether to expose path.
as you said, that is the implementation details.
We can have HadoopResourceId that built on top of hadoop path, and UriResourceId that built on top of URI. And, we don't want to expose them as well.

also, added tests for GetScheme().

retest this please

Coverage decreased (-0.01%) to 69.68% when pulling 551adcbb1513f745abe9ffdac6fb58dc10668c9d on peihe:file-system-local-gs-resolve into c0910fdf4939a4db52b4e7c49e3593b6da31694a on apache:master.

PTAL
Tests looks fine to me: Jenkins passed, travis half passed, the other exceeded the time limit.

LGTM ; self-merge when green.

retest this please

Changes Unknown when pulling 4cbb1676812d80eda46d58944233efc1bf041533 on peihe:file-system-local-gs-resolve into ** on apache:master**.


Internet Explorer support

Thanks for the plugin, it's a big help. Any chance of support for Internet Explorer in the future? I work on a lot of enterprise projects where IE support and testing is still required.

I thought about it a bit but given that I personally don’t do IE test
automation I wanted the plugin to gather some traction to justify the
effort. Now that you it has been requested by you it’s probably time to
look into providing support for IE. And end to end testing it at AppVeyor
will be an interesting challenge.
On Thu, 21 Dec 2017 at 17:50, Dennis Rausch<EMAIL_ADDRESS>wrote:

Thanks for the plugin, it's a big help. Any chance of support for Internet
Explorer in the future? I work on a lot of enterprise projects where IE
support and testing is still required.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/energizedwork/webdriver-binaries-gradle-plugin/issues/7,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMii9YkvnvsE7Qp6WpOSmf_IXJbPswHks5tCppVgaJpZM4RKJkx
.


How about having the mentioned commit also be applied to the MSVC build?

https://github.com/DiscordMessenger/dm/commit/e7ca531b25ee990f1eaff6c4d60c11d0807412eb

It's not necessary, because MSVC build only supports down to Windows XP SP2 which implements I/O completion ports completely.


Is help intended to go to stderr?

$ drive pull -h | grep prompt
Usage of drive pull:
  -exclude-ops="": exclude operations
...
*** all this is stderr output ***
...
  -skip-mime="": skip elements with mimeTypes derived from these extensison
$ drive pull -h 2>&1 | grep prompt
  -no-prompt=false: shows no prompt before applying the pull action

Hello again @utgarda. Thank you for reporting this issue as I can see from a continuation of one you made earlier on.
The flag parsing output is actually done by an external package command and not actually by drive. I made a fork from github.com/rakyll/command now at github.com/odeke-em/command.
In regards to your question: since flag -h it is in an error-ed state, yes help output should go to stderr, as opposed to drive help pull which is just a normal functioning mode and a subcommand of a flag.

Hi @odeke-em ,
but drive help tells us to use commands like drive pull -h:
$ drive help
Name
        copy - copy remote paths to a destination
Name
        move - move files/folders
Description
        Moves files/folders between folders
* For usage flags: `drive move -h`
...
* For usage flags: `drive pull -h`

so I'm not sure if it's ok or not. Anyhow, I'm pretty sure it's not urgent. Unfortunately, I'm quite far from Go development, so all I can offer is these reports.

Oh yes it asks you to run drive pull -h but only because it is known that the parse will spew out whatever options. The list of options may grow or shrink but -h directly to the parser will give the list of options defined.
Thank you for the offer, no worries your input is quite useful. This is an interesting case because there doesn't seem to be a standard of sorts -- a bunch of commandline options send -h output to stderr, others send it to stdout. go seems to mostly treat -h output as help mode and -h sends output to stderr as well.
Otherwise, if we agreed on getting this to stdout, nothing would change in drive, it would however in the command parser which is an external package as I had mentioned earlier. It shouldn't be too hard to replace stderr with stdout.

Alrighty then, now we know it :)

Would you mind opening this issue on my fork of command? This way I won't forget as am interested in us solving this issue. Thank you.

Hello @utgarda, thank you for reporting this issue. I fixed it in my fork of command with PR https://github.com/odeke-em/command/pull/2.


Enabling Animation

How do you enable the animation facility?
Testing this repo in MyBinder ([example])(https://mybinder.org/v2/gh/uclmr/egal/master), which uses the Dockerfile to build a container in which to try out the egal extension, I can edit SVG files but nor enable the animation steps?
I also note from the example screenshots in the repo README that the first two examples do not show the animation step buttons / frame counters as being enabled?

Hi, I think that's just because the docker file is out-of-date. The two screen shots are from an earlier version. Have you tried installing this as per the instructions?


Build as an overlay on Windows/Linux

Updated repos file to build as an overlay
Upated build instructions
Created a Linux build workflow

Closes #51
Closes #53
Closes #55

Ugh building in CI has been one of the most frustrating topics...I've been testing over in my private fork for over a week (with all passes yesterday) and all I did differently was submit a PR. I'll keep working on it I guess.
https://github.com/theseankelly/ros2_dotnet/actions

@theseankelly is there anything I can help with?

@esteve thanks, but I don't think so -- in theory it should be straightforward. I think GitHub Actions is inserting some entropy just to troll me.

Wow, talk about poor timing.
The Windows runner for GitHub actions was refreshed very recently, which updated the default system Python to 3.7.7. It looks like the ros-tooling/setup-ros action hardcodes Python 3.7.6, which is now broken.
I filed a bug on the ros-tooling/setup-ros action, but its easily worked around by configuring python 3.7.6 prior to invoking the ros-tooling/setup-ros action.
Looks like the Build (Desktop) job has made it farther this time -- fingers crossed.

@theseankelly yay, the desktop runner passed! Thanks

@theseankelly is this ready for review or you'd rather way for the fix in ros-tooling/setup-ros

Let's go ahead with the review as-is.


Implement triangle wave

I think this can be the last type of wave generation for now.

Implemented 2016-12-05


Named domains aren't restored

If you have a named domain, say via the following config:
config.ssh_domains = {
		{
			name = "rpi5",
			remote_address = "<IP_ADDRESS>",
		},
	}

Then the domain wont be saved, as it is currently unsupported as it reaches the else branch in pane_tree.lua

I'll experiment a bit, but I don't think this bug will be terribly difficult to fix.
We could check if domain ~= "local" first, and if so, save the domain key.
Alternatively, we can make the domain key non-optional and always save it.

I can confirm that https://github.com/MLFlexer/resurrect.wezterm/pull/45 saves and restores named ssh domains


Suggest changing file permissions on OSX

Inform users who don't want to (because you probably shouldn't) use sudo to install node modules globally how to update their file permissions.

Thanks for the PR @gnomeontherun!


pass --verbose to cargo when VERBOSE env is set

and also show that cargo's version after rustc's one
Closes #1216

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @huonw (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see CONTRIBUTING.md for more information.

@bor: r+ 282ff5c thanks!

@bors  r+ 282ff5c
bot name typo* :)

@bors: r+ 282ff5c
oops!

:hourglass: Testing commit 282ff5c with merge 464d8f4...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-32, cargo-win-64


Add override option for providing context when checking a feature toggle

Is your feature request related to a problem? Please describe.
The programming pattern of using the global HttpContext is not a recommended pattern, nor is it natively supported by dotnet core.
A simple way to bypass this issue would be to pass in context with every request to check if a flag is enabled, but this is not available.
Describe the solution you'd like
Adding a simple override for passing in the context on IsEnabled
public bool IsEnabled(string toggleName, UnleashContext context, bool defaultSetting)
        {
            return CheckIsEnabled(toggleName, context, defaultSetting);
        }

Describe alternatives you've considered

Force the HttpContext to be available, but it goes against the reason why it was removed in the first place
https://stackoverflow.com/questions/38571032/how-to-get-httpcontext-current-in-asp-net-core
Make the CheckIsEnabled function public. This would work, but it is a naming inconsistency with the IsEnabled

Additional context
I will have authentication information in my controller of the request, but I currently can't access the HttpContext without some workaround. I was trying to figure out exactly what my ContextProvider will look like and since we don't have things like local cache or session information, but we have the userid at the time of the request, this seems like the best way that I can think of.
If there is a better way to do this with a dotnetcore2.0 application, I would love to know.

Great input, @andywgarcia . Please have a look at the referenced commit and confirm that it solves your issue.

@gnab Yup this solves my problem and exactly what I would have done in a fork. Thank you very much!

@gnab Looking a bit more at the documentation in the readme, I am sure that could use an update as well for the section about using the context provider

@andywgarcia Indeed. The README needs to be revised, but we'll deal with that separately. The changes have been merged now and will be included in the next release.

Thanks for the quick turnaround. Is there an ETA on the next release?

@gnab Should there be an additional method as well with the default flag set to false so the option of IsEnabled(string toggleName, UnleashContext context) is available as well?

This has now been released in the new Unleash.Client 1.4.0 NuGet package.


x/mobile: Window always 800x800px

I can't seem to resize the viewport with gomobile.  It's cumbersome to manually drag the corner of the viewport to match the screen size of an iPhone, so I'm hoping to do it programmatically, but the events seem to be getting overruled.
package main

import (
	_ "image/jpeg"

	"github.com/kr/pretty"

	"golang.org/x/mobile/app"
	"golang.org/x/mobile/event/size"
	"golang.org/x/mobile/geom"
)

func main() {
	pxPerPt := float32(163.0 / 72.0)
	defaultSizeEvent := size.Event{WidthPx: 400, HeightPx: 800, WidthPt: geom.Pt(float32(400) / pxPerPt), HeightPt: geom.Pt(float32(800) / pxPerPt), PixelsPerPt: pxPerPt, Orientation: size.OrientationPortrait}

	app.Main(func(a app.App) {
		a.Send(defaultSizeEvent)
		for e := range a.Events() {
			pretty.Println(e)
		}
	})
}

I ran gomobile build and executed the binary and expected to see a viewport 400px wide by 800px tall.  Instead, I always get the default 800x800px viewport.  I do not have a device plugged in.  Debugging the Events() channel, it looks like 3 Size Events are automatically run after my Size event is processed:
size.Event{WidthPx:400, HeightPx:800, WidthPt:176.68711853027344, HeightPt:353.3742370605469, PixelsPerPt:2.2638888359069824, Orientation:1}
size.Event{WidthPx:800, HeightPx:800, WidthPt:260.63348388671875, HeightPt:260.63348388671875, PixelsPerPt:3.069444417953491, Orientation:0}
size.Event{WidthPx:800, HeightPx:800, WidthPt:260.63348388671875, HeightPt:260.63348388671875, PixelsPerPt:3.069444417953491, Orientation:0}
size.Event{WidthPx:800, HeightPx:800, WidthPt:260.63348388671875, HeightPt:260.63348388671875, PixelsPerPt:3.069444417953491, Orientation:0}
lifecycle.Event{
    From:        0x0,
    To:          0x1,
    DrawContext: &gl.context{
        cptr:          0x0,
        debug:         0,
        workAvailable: 0xc820062240,
        work:          0xc8200622a0,
        retvalue:      0xc820062300,
        cargs:         {
            {},
            {},
            {},
        },
        parg: {
            (*gl._Ctype_char)(nil),
            (*gl._Ctype_char)(nil),
            (*gl._Ctype_char)(nil),
        },
    },
}
lifecycle.Event{
    From:        0x1,
    To:          0x2,
    DrawContext: &gl.context{
        cptr:          0x0,
        debug:         0,
        workAvailable: 0xc820062240,
        work:          0xc8200622a0,
        retvalue:      0xc820062300,
        cargs:         {
            {},
            {},
            {},
        },
        parg: {
            (*gl._Ctype_char)(nil),
            (*gl._Ctype_char)(nil),
            (*gl._Ctype_char)(nil),
        },
    },
}
lifecycle.Event{
    From:        0x2,
    To:          0x3,
    DrawContext: &gl.context{
        cptr:          0x0,
        debug:         0,
        workAvailable: 0xc820062240,
        work:          0xc8200622a0,
        retvalue:      0xc820062300,
        cargs:         {
            {},
            {},
            {},
        },
        parg: {
            (*gl._Ctype_char)(nil),
            (*gl._Ctype_char)(nil),
            (*gl._Ctype_char)(nil),
        },
    },
}
lifecycle.Event{
    From:        0x3,
    To:          0x2,
    DrawContext: &gl.context{
        cptr:          0x0,
        debug:         0,
        workAvailable: 0xc820062240,
        work:          0xc8200622a0,
        retvalue:      0xc820062300,
        cargs:         {
            {},
            {},
            {},
        },
        parg: {
            (*gl._Ctype_char)(nil),
            (*gl._Ctype_char)(nil),
            (*gl._Ctype_char)(nil),
        },
    },
}

I also tried setting the viewport using the Viewport() function from https://godoc.org/golang.org/x/mobile/gl#Context, but that did not work either.  It appears that an 800x800px viewport is somehow registered into the startup process and there doesn't appear to be an easy way to change that, or am I missing something?
go version go1.6.2 darwin/amd64

GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/ever/dev/go/"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GO15VENDOREXPERIMENT="1"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fno-common"
CXX="clang++"
CGO_ENABLED="1"

By viewport, I assume you're referring to the actual window size. Sending a size.Event will not affect the window size, nor does glViewport. Currently, the window size hints are hardcoded.

Is there a reason it's hard coded, or is this a good candidate for a patch? I'm interested in developing with GoMobile, but without having a way to adjust the window size to fit common device sizes makes testing difficult.

Note that we generally do not use the issue tracker for discussions.  See https://golang.org/wiki/Questions .


script uses standard "shutdown" method - not the "shutdown only" function.

If a windows update is pending, this isn't helping against that.
Not necessarily a problem, but should at least be mentioned in the readme.

You are right, will add it, thanks for the info.


Upgrade flutter_platform_widgets

On upgrading to flutter 3.7.0

CupertinoListTile package was discontinued
Some APIs from the flutter framework were deprecated

Thanks a lot! Why is CupertinoListTile being removed?

Thanks @robert-virkus
A similar widget was added to the core cupertino library in Flutter 3.4.

The CupertinoListTile is a Cupertino equivalent of Material ListTile. It comes in two forms, an old-fashioned edge-to-edge variant known from iOS Settings app and in a new, "Inset Grouped" form, known from either iOS Notes or Reminders app.
-- Read more

Thanks, this makes (of course sense)!


Hello http+json->object->load[method]-call[method+object]->stringify[object]->goodbye json->http

hi,
how's it all going?
i was wondering how the following can be done using asp .NET 5?

Hello http
Hello json
Hello dynamic object (are inferred classes possible from json? or do i need to provide a generic one?)
Hello dynamically loaded method -- retrieved from memory||disc||dll||cs
asyncMethod(object) --> result
ReturnViaHttp(JSON.stringify(result)) // goodbye json

any links/docs/samples that could
help me implement this would be great
thanks
i don't know or particularly like python, but i'm working thru the various languages, and doing the same task (as above) it took me half a day to write the following in python, and in powershell ~ https://github.com/aspnet/Home/issues/770 and vert.x https://github.com/aspnet/Home/issues/787 these three posts are all the same issue just stated in different ways
http+json->object->load[method]-call[method+object]->stringify[object]->goodbye json->http
import json
import urllib
import cgi
import time
from os import curdir
from os.path import join as pjoin
from importlib import import_module
from http.server import BaseHTTPRequestHandler, HTTPServer
import imp
import urllib.request
import sys
hd=__file__.replace("2/1/Ax/250/run.py","")

NUT={}

def getf(rx):
    if rx in NUT.keys():return NUT[rx]
    mod = imp.load_source('bar',hd+'2/yy/Ax/'+(rx[1:])+'/y.py')
    NUT[rx]=mod.task
    return NUT[rx]

def next(o):
    rn=o.get('rn',-1)
    rn+=1
    o['rn']=rn
    if rn>0:o['e']=o.get('rv','')
    print('####'+str(rn))
    if type(o.get('rx')) is not list: return o
    RX=o.get('rx')
    if rn>=len(RX):return o
    try:rx=RX[rn]
    except Exception:return o

    if rx:
        if rx[0]!='y':return o

        f=getf(rx)
        f(o)
        return next(o)
    return o

class web1(BaseHTTPRequestHandler):
    def do_GET(self):
            self.request.sendall((json.dumps({'path':self.path,'d':int(time.time()*1000),'a':1,'b':2,'c':[1,2,3]}).encode("utf-8")))
    def do_POST(self):
        try:
            if 'content-length' in self.headers:
                length= int(self.headers['content-length'])
                data = self.rfile.read(length).decode("utf-8")
                o2 = json.loads(data)
                rc=o2.get('rc',0)
                if rc:
                    self.request.sendall(json.dumps(o2).encode("utf-8"));
                    pass
                rx=o2.get('rx',0)
                if rx:
                    next(o2)
                    self.request.sendall(json.dumps(o2).encode("utf-8"));
                    pass
            else:
                print("No data")
                self.send_response(200)
                self.send_header("Content-Type", "text/json; charset=UTF-8")
                self.end_headers();
                self.wfile.write("No data");
        except Exception as excp:
            print(excp)
            self.send_response(200)
            self.send_header("Content-Type", "text/json; charset=UTF-8")
            self.end_headers();
            self.wfile.write(str(excp));

http = HTTPServer(('localhost',1425), web1)
print('Python listening on http://<IP_ADDRESS>:1425/ \n'+sys.version)
http.serve_forever()

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain spinthemoose.com has exceeded the max emails per hour (235/200 (117%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from o10.sgmail.github.com ([<IP_ADDRESS>]:28561)
by cp1164.blacksun.ca with esmtps (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128)
(Exim 4.89_1)
(envelope-from<EMAIL_ADDRESS>id 1eVkHv-0001Q5-V7
for<EMAIL_ADDRESS>Sun, 31 Dec 2017 14:32:56 -0600
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=github.com;
h=from:reply-to:to:cc:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe;
s=s20150108; bh=1RUU43qW1UnajpBUIRQfhGDyvPI=; b=mLwp6aTD+wjge/RL
448fXr0IG8IUbJx5Ajc9qj3MlHM4DPXr8lRmhQRh7WecVpDi3NYSxY/COmNg7DtP
hooC3g5tR0PrI835vUUdnDn80UIKt48O3c6xe4d6mROg5s+tndaeRwhN6KUhjpF+
L5PeXdFvrazD7nwuHRJ/0qEQj8c=
Received: by filter0349p1las1.sendgrid.net with SMTP id filter0349p1las1-17898-5A494962-25
2017-12-31 20:32:34.597679505 +0000 UTC
Received: from github-smtp2a-ext-cp1-prd.iad.github.net (github-smtp2a-ext-cp1-prd.iad.github.net [<IP_ADDRESS>])
by ismtpd0013p1iad2.sendgrid.net (SG) with ESMTP id Vusix2orSoywfmXgmE4Daw
for<EMAIL_ADDRESS>Sun, 31 Dec 2017 20:32:34.609 +0000 (UTC)
Date: Sun, 31 Dec 2017 20:32:34 +0000 (UTC)
From: "ASP.NET Hello Bot"<EMAIL_ADDRESS>Reply-To: aspnet/Home<EMAIL_ADDRESS>To: aspnet/Home<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [aspnet/Home] Hello
http+json->object->load[method]-call[method+object]->stringify[object]->goodbye
json->http (#818)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5a4949627913c_23663fb62e086f28105952a";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: aspnet-hello
X-GitHub-Recipient: davidalpert
X-GitHub-Reason: subscribed
List-ID: aspnet/Home <Home.aspnet.github.com>
List-Archive: https://github.com/aspnet/Home
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AACpRDwUE7MuX7jY6daRLeTGq8pPkr8Cks5tF-9igaJpZM4FpJvU
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-SG-EID: rlj3t9ahcX0BZUzAOlajQ3VmBXo43xFJc4M1ImIDJhEJ/8TpcMQx1M61zILvEYYvtLoIA85nbwjhiL
NWJBx4VsHrMUiUEJB6j4Cw12xWcq6ULzP++YOxfRz1qTjs/Xbt7DlUBtgPYeK9al9RCf4lJdcorcCm
eAcnoYiw/0LSBKU4MPLVaYjMs5iifAymAeqoPi84DukT9vjcU4jgNQnPREdflpVfOmz44n7RXZd81b
k=
----==_mimepart_5a4949627913c_23663fb62e086f28105952a
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #818.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/aspnet/Home/issues/818#event-1405408732
----==_mimepart_5a4949627913c_23663fb62e086f28105952a
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #818.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

----==_mimepart_5a4949627913c_23663fb62e086f28105952a--

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain spinthemoose.com has exceeded the max emails per hour (234/200 (117%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from o5.sgmail.github.com ([<IP_ADDRESS>]:12598)
by cp1164.blacksun.ca with esmtps (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128)
(Exim 4.89_1)
(envelope-from<EMAIL_ADDRESS>id 1eVkHv-0001Q3-E9
for<EMAIL_ADDRESS>Sun, 31 Dec 2017 14:32:55 -0600
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=github.com;
h=from:reply-to:to:cc:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe;
s=s20150108; bh=cWdkoBKQIH03BtBOW4I7Z2rB/9E=; b=jSJcdAP8QfsHAtAh
tkR+sROgBrjwv4CK/PowNM+K4pJIcG99Brtpt3MhLDbfGohElGs+zvl6R9LSqfN0
FFjo3ikR8yus4JZNfTRjSBHuuN7btTL2PRW/MZtr1CgCsYM7OWllVGSW2DqNDNaX
Gdgbl7xdikMjbi4qKBgvcHbchA8=
Received: by filter0105p1las1.sendgrid.net with SMTP id filter0105p1las1-22905-5A494961-18
2017-12-31 20:32:33.769272198 +0000 UTC
Received: from github-smtp2a-ext-cp1-prd.iad.github.net (github-smtp2a-ext-cp1-prd.iad.github.net [<IP_ADDRESS>])
by ismtpd0015p1iad2.sendgrid.net (SG) with ESMTP id AgtDbn3qRcCNmNqO-4GjaA
for<EMAIL_ADDRESS>Sun, 31 Dec 2017 20:32:33.677 +0000 (UTC)
Date: Sun, 31 Dec 2017 20:32:33 +0000 (UTC)
From: "ASP.NET Hello Bot"<EMAIL_ADDRESS>Reply-To: aspnet/Home<EMAIL_ADDRESS>To: aspnet/Home<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [aspnet/Home] Hello
http+json->object->load[method]-call[method+object]->stringify[object]->goodbye
json->http (#818)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5a494961a4432_5d23fc381302f304470b4";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: aspnet-hello
X-GitHub-Recipient: davidalpert
X-GitHub-Reason: subscribed
List-ID: aspnet/Home <Home.aspnet.github.com>
List-Archive: https://github.com/aspnet/Home
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AACpRFiPbWD8vCTEVXQjcxVxEFoatJ_aks5tF-9hgaJpZM4FpJvU
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-SG-EID: rlj3t9ahcX0BZUzAOlajQ3VmBXo43xFJc4M1ImIDJhFMkFhhXCFLsGxwVf6zlF5OFieRcQ3mcGJ62J
J+2crInmz+Bek/Yhtlu3AIR5aqHG/aU32NfwCLmeeGb9GEBNNhlHjnrb898QrNwQR7udnO9n8frdmX
QXXrV+7HFKMluZleA72t0t0EvJv/rPIBO927ctOIQM82Bpg1KzXG8o79o308ZZ/pPcYFjobttWevCe
s=
----==_mimepart_5a494961a4432_5d23fc381302f304470b4
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/aspnet/Home/issues/818#issuecomment-354622624
----==_mimepart_5a494961a4432_5d23fc381302f304470b4
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: quoted-printable
This issue is being closed because it has not been updated in 3 months.<=
/p>
We apologize if this causes any inconvenience. We ask that if you are st=
ill encountering this issue, please log a new issue with updated informatio=
n and we will investigate.
&mda=
sh;You are receiving this because you are subscribed to this thread.<=
br />Reply to this email directly, view it on GitHub, or mute the thread.

=
----==_mimepart_5a494961a4432_5d23fc381302f304470b4--


Fix one more missing corepack issue

Missed this one in #777.

Also probably need a changeset for this since I touched the utils


APNS bundleId is deprecated, use topic instead

With the new release I saw this in my logs: APNS bundleId is deprecated, use topic instead. However I could not find anything about how to use topic in my configuration in the docs yet. I would love to see this covered as I will probably not be the only one with this issue.

Closing as duplicate of #83
As the warning says, just pass topic instead of bundleId in the adapters configuration.