Can not work on offline env.

Sure, the AstroWeather integration is cloud polling

The problem is the Weather data has been downloaded. But the ui can not display if offline. report css is not the function.


SITL Gazebo unable to echo Gazebo Topics

I have the latest master cloned. When I run the basic IRIS SITL using:
make posix_sitl_default gazebo
After the model loads and PX4 shell opens, I did:
gz topic -l
I get the following

I just cannot echo any iris related topics using:
gz topic -e /gazebo/default/iris/$NAME
It returns nothing. I'm developing a Tiltrotor Tricopter model for SITL. I need to view what's going into the motor_speed topics to debug my code.
I keep getting these warnings but I think this shouldn't cause the topics to not publish anything.

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 30 days. Feel free to reopen this issue if you deem it appropriate.
(This is an automated comment from GitMate.io.)

Hi, did you ever find out more about this? Thanks

@mchen0037 https://github.com/PX4/sitl_gazebo/issues/102


go/types: maintain "owner" information for interface methods and struct fields

Hi Robert,

in our indexer for Go, when we load (from export data) a package that defines types such
as the following:

    type T struct { X int }
    type U T
    type S U

there is nothing in the go/types API that allows the client to determine that although
the *types.Var for X is a member of S, T and U, it was originally declared beneath T.
(Underlying() returns the same struct type for S, T and U.)

An analogous situation holds for interface methods:

   type Closer interface { func Close() error }
   type WriterCloser interface { Writer; Closer }

i.e. there is no record of the fact that the Close() method was originally declared
beneath Closer, not WriterCloser.

The indexer needs this information because it must invent a globally unique stable name
for T.X that relates its definition in this package to its uses in other packages.
Right now, it picks---consistently but arbitrarily---the lexicographically least name
from the set {S.X, T.X, U.X}, i.e. S.X.  So long as all invocations of go/types use
export data that includes S, then all will use the same least name S.X.  However, if the
package was loaded from partial export data that does not include the least name, the
result is a dangling link.

What the indexer needs is that go/types expose some information about the field X that
allows us to come up with a stable name for it each time we encounter it across
different invocations of the type checker.  Associating X with its "true"
owner (T.X) is the most logical approach and will doubtless have benefits for other
applications, but any stable name could work (e.g. the least name, or some fingerprint).

So: how should we do this?

@adonovan Is this still of interest? If so, please comment on this issue and mark for 1.11, "early in cycle" and I'll try to address it. Thanks.

I just ran into this limitation, too, when working on a tool to surface “// Deprecated: ” comments. (See Google-internal cl/478490109 for more context.)
My code works by iterating through info.Uses, constructing a stable name, and then matching said name against an index of known-deprecated symbols (which is produced by using ast.Inspect).
Currently, I can’t surface deprecation warnings for struct fields (only for the struct type itself, or methods on it) because of this limitation.
@griesemer Would you still be interested in looking into this? Timing-wise, the 1.21 cycle should start soon, so perhaps this is an opportune moment. Thank you!

@stapelberg Marking for 1.21 so it's on the radar as we may look into some changes in this space. But exposing the information would be an API change which would require a (if small) proposal.

Thank you! Let me know if there’s anything I can help with, happy to try out a pending change or similar :)

@adonovan @findleyr for feedback in whether this is something we need to prioritize ("early in cycle"). Moving to 1.22 for now.

I have learned to live with this design, so don't keep this open on my account.

FWIW, now that the export and import of type information preserves precise positions, you could use the position of the field to resolve the ambiguity: the named struct type whose position most closely precedes the field position is its "owner".

Just to be clear, we could easily maintain an owner link but at least for go/types this would require an API change if it needs to be accessible externally. That would require a proposal. For now put on Backlog. Feel free to "reactivate" (and send out a proposal) if this continues to be important. Also, feel free to close if no need.

We managed to work around this issue as well: we now look at the AST, and when we find a KeyValueExpr, we go back up the stack to the containing CompositeLit to extract its type.
I’ll close this issue, but please report back in case anyone has a use-case where this limitation cannot be worked around.

https://github.com/golang/go/issues/60831

I ran into this as well, and ended up using a go/ast based solution, specifically because of this limitation. I think it would be a useful addition to go/types, or at a minimum it would be helpful if go/types documented it as a non-feature.


Possible bug found in TCP receive

Possible bug in master branch at src/Hans/Tcp/RecvWindow.hs:77:
There is:
case trimSeg (fromTcpSeqNum (segEnd x - segStart y)) x of

Shouldn't it be:
case trimSeg (fromTcpSeqNum (segEnd x - segStart y)) y of

?
And later:
Just x' -> [x', y]

Shouldn't it be:
Just y' -> [x, y']

?

This is intended. The idea is that you're trimming the segment x to remove the overlap with y. So, it trims from the point where they overlap, shrinking it down so that y doesn't have to be modified. This is also why y is returned unchanged.

Sure, but if you have two overlapping segments: former x and latter y, to remove overlap you would want to trim back of x or front of y, not front of x.

Also, if y is fully contained in x, trimSeg ... y would return Nothing, and you want to handle this case by returning [x], not throwing an error.

You're right, it should be trimming from the front of y. I'll push a fix, thanks for the report!


Cleanup CodeCov tests output spam

Verification

[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

Provide a detailed description of the proposed feature
CodeCov's coverage reporter outputs a huge string multiple times each test run:
https://github.com/Homebrew/brew/actions/runs/4194693982/jobs/7273227048#step:8:227
This information is not useful to us and makes it harder to quickly pass failing tests output.
What is the motivation for the feature?
Make GitHub Actions logs more readable.
How will the feature be relevant to at least 90% of Homebrew users?
Saving maintainer time (which is the project's most precious resource).
What alternatives to the feature have been considered?
Getting over it.

https://github.com/codeclimate-community/simplecov_json_formatter/blob/a03461117e6a837ca68c08fba98852819664f66a/lib/simplecov_json_formatter.rb#L30-L33

@issyl0 That was my first suspect too but it has a "JSON" prefix. Think it's SimpleCov::Formatter::CoberturaFormatter.

Ah, it's also in the HTML formatter: https://github.com/simplecov-ruby/simplecov-html/blob/5bc5798aefed312abd189baed8a97bf1febfa8d7/lib/simplecov-html.rb#L34-L36.


Add tests during when there is a change to/from DST.

Codecov Report

Merging #119 into master will increase coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #119      +/-   ##
==========================================
+ Coverage   74.62%   74.84%   +0.21%
==========================================
  Files          10       10
  Lines         938      938
  Branches      157      157
==========================================
+ Hits          700      702       +2
+ Misses         90       88       -2
  Partials      148      148

Impacted Files
Coverage Δ

src/schedule.c
69.83% <0%> (+1.11%)
:arrow_up:

Continue to review full report at Codecov.

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


Security Day Workshop 1 Preparation - DevOps and Vulnerability Management

Describe the Issue
Nick and I were invited to share our experience of SAST/DAST (Static/Dynamic Application Security Tool) in the Security Day Workshop 1 on May 3rd.  As a security analyst, I would like to  create powerpoint slide deck, clone the github repo and trigger the github action or openshift tekton CI/CD automation pipeline.
Additional Context

[ ] Demo the best practice
[ ] Demo github action to deplop app in KLAB or (local run k8s cluster) and run the security tools in the pipeline
[ ] Demo tekton pipeline to deploy app in KLAB or (local run k8s cluster )and run the security tools in the pipeline
[ ] Security tools : OWASP ZAP, Sonarqube
[ ] Sample Apps : Maven, Flask Web
[ ] Automation : Kustomize, Helm Chart, Tekton, Github Action
[ ] Powerpoint Slides

Acceptance Criteria

[ ] Working Sample Apps
[ ] Relevant Security Vulnerability Detected and Reported
[ ] Presentation Deck pass Grand Mom test

A set of pre-defined requirement that need to be met in order to mark the user story as “done”.
Extra Template Info
Definition of Ready
These set of conditions will need to be met in order to bring a ticket into the sprint and start work. Protects the team from unclear requirements.
Issues (Task/Story/Spike) aligned to an EPIC and linked appropriately.
Assigned to the appropriate CPF MEMBER (and is not left blank in ZenHub) at the start of sprint (1st day of the sprint)
Acceptance has been defined for the issue and has been reviewed with CPF team and approved by the necessary approver.
Has been sized and estimated by the delivery team.
Detailed breakdown of the steps required to complete the story in the additional context
Any additional specifications have been documented, reviewed with CPF and approved by the necessary approver.
Acceptance Criteria
A set of pre-defined requirement that need to be met in order to mark the user story as “done”.
It should be testable with no room for interpretation
It should be either “pass” or “fail”
It should be clear enough for business stake holders to understand
As part of the user story, it should be written from the user perspective
A well written acceptance criteria is great for

managing expectations
defining scope
reducing ambiguity
establishing testing criteria for QA
Given (Pre-condition)
When (Action)
Then (Outcome)

Definition of Done
These set of conditions are met for work to be considered as complete on an issue.
All acceptance criteria(s) have been validated.
All the necessary documentation for the issue has been uploaded to Teams.
Functionality has been tested when applicable.
Functionality has been demonstrated to the relevant stakeholders (where applicable).
Story has been scheduled for a Sprint Demo/community update and impacted teams invited to the Demo.
Stories accepted by PO and documented for potential sharing with impacted users.

Documented = Captured in Community Update (CU) deck
Release notes = aspirational goal of providing a bulleted list of features and changes that our users can touch. Was going to store them in our Cloud Pathfinder repo. Can also display to users in the CU

issue # 2440 and 2470 are related

All preparation completed.   Video Clips created.   Local Backup saved in addition to team drive.  Closed


fixed key vs. path typo in compute_cache

Lame, now I'll have a conflict since I'm fixing everything in one shot!

But looks good


Null safety

We have concrete plan to migrate dumble to null safety. It hopefully will arive on pub during june 21.

Null safty was introduced with version 0.8.0


Another incorrect matching case

local u = (require "stepets.utf8"):init()

local subj = "ab.?"
local patt = "%?"
local repl = "123"

print(      u.gsub(subj, patt, repl) ) --> 123a123b123.123?123     5
print( string.gsub(subj, patt, repl) ) --> ab.123  1

Thanks again!
Fixed
https://github.com/Stepets/utf8.lua/blob/681e935f244b2d3fe6ad9b5103a1059c9e038749/test/test.lua#L184-L185
https://github.com/Stepets/utf8.lua/blob/681e935f244b2d3fe6ad9b5103a1059c9e038749/test/charclass_compiletime.lua#L113-L116
P.S. Do you mind sharing your regex archive? :)

Do you mind sharing your regex archive? :)

I don't have a separate "regex archive". I have Far Manager application (you probably heard about it) and a bunch of various Lua plugins and macros installed (most of them use pattern matching).
Then I take your library and replace Far Manager's built-in library with it.
Then the things stop work and I investigate why.


Make sure that case-insensitive column names are unique

This closes #21.
When there are metadata entries with the same case insensitive string, then there is an error when trying to create a new column for that metadata entry in the database table, because a column with that case insensitive name already exists.
 <Record type="HKQuantityTypeIdentifierDietaryCholesterol" sourceName="MyFitnessPal" sourceVersion="35120" unit="mg" creationDate="2021-07-04 20:55:27 +0200" startDate="2021-07-04 20:55:00 +0200" endDate="2021-07-04 20:55:00 +0200" value="124">
  <MetadataEntry key="meal" value="Dinner"/>
  <MetadataEntry key="Meal" value="Dinner"/>
 </Record>

The code added in this PR checks if a key already exists in a record and if so adds a number at its end. The resulting column names look like the example below then. Interestingly, the column names viewed with Datasette are not case insensitive.
startDate, endDate, value, unit, sourceName, sourceVersion, creationDate, metadata_meal, metadata_Meal_2, metadata_Mahlzeit

Did not see that #13 fixes the same issue in a similar way. You can decide which one to merge ;)

@simonw please merge this request


Question about generating custom keysims

Hi,
I have the following settings in my xkbmap:
clear mod1
keycode 108 = Mode_switch
keycode 64 = Super_L
keysym e = e E EuroSign
keysym c = c C cent
keysym w = w W adiaeresis Adiaeresis
keysym o = o O odiaeresis Odiaeresis
keysym u = u U udiaeresis Udiaeresis
keysym r = r R ssharp
keysym q = q Q acircumflex Acircumflex
keysym a = a A abreve Abreve
keysym s = s S U0219 U0218
keysym t = t T U021B U021A
keysym i = i I U00EE U00CE

and I've tried using keyd to generate the same:
rightalt = layer(dia)

[dia]
r = ß
u = ü
o = ö
e = €
...etc...

As I understand I cannot use r = G-r and so on because keyd resets all the xkbmap mappings. is it possible to generate these keysims with keyd? perhaps via a macro calling xdotool or something like that?
Thanks.

The config file does not support unicode so you can't do u = ü. Even if this was valid, it would have no meaning since keyd operates at the level of the keycode.  Symbols are handled by the display server (see #31 and #30). My advice would be to use compose key macros to obtain the result you want.
E.G

[dia] o = macro(compose o ") e = macro(compose c =) ...  ```

You will have to set the compose key in X for this to work. This can be done with `setxkbmap -option compose:menu` and will need to be done each time you restart keyd (see below).

You can find a list of default compose sequences in `/usr/share/X11/locale/en_US.UTF-8/Compose` (replace en_US-UTF8 with your locale).  '<Multi_key>' corresponds to the compose keycode.

> because keyd resets all the xkbmap mappings.

keyd doesn't reset anything, but it does create and destroy a virtual evdev device each time you restart it. It seems X doesn't preserve mappings when keyboards are unplugged, but this should be less of a problem once you have a config you are happy with. In the meantime you can just do something like `sudo systemctl restart keyd; sleep 2s; setxkbmap -option compose:menu`

Thank you,  your solution except that I can't produce upper case characters.  What you want is a little more involved but can be achieved
by combining the appropriate features like so:
rightalt = layer(dia)
shift = layer(shift)
rightshift = layer(shift)

[shift:S]

rightalt = layer(shiftedDia)

[dia]

shift = layer(shiftedDia)
rightshift = layer(shiftedDia)

o = macro(compose o /)
...

[shiftedDia]

o = macro(compose O /)
...

This creates a layer called 'shiftedDia' which is accessible by holding down both shift and AltGr (right alt).

This works perfectly, thanks! May I suggest adding one such example to the man page as it's quite a common use case.

Thanks for the feedback. Given the recent influx of layout related questions I intend to update the documentation when I have some time.

There is now first class support for this, so you can do o = ö directly in the config file (provided the xkb option compose:menu is set). See the man page for details.


ER: select path() from (traverse ...)

Currently (Version 2.1.8), there doesn't seem to be a way (or at least
there doesn't seem to be a documented way that works) to convert paths
(in the sense of $path) into a result-set.
By contrast:

"select $path from (traverse ...)" gives string representations of the paths being traversed;
"select expand(shortestPath) from shortestPath(...) ...." gives the path as a result set

Since both:
'select $path from (traverse ...)' and
'select traversedVertex(0,100) from (traverse ...)'
are supported, it would, I think, make sense to support: 'select
path() from (traverse ...)', where path() would be a list of the
vertices and edges in the path, so that one could expand(_) it.
NOTE: One might expect the following query to emit both the vertices
and edges along a path, but it only emits vertices:
select expand(traversedElement)
from (select traversedElement(0,10)
      from (traverse out(), inE() from #11:0 ) )
      where last(traversedElement) = #11:3

Is that a bug?  If traversedElement(_,_) is supposed to return all the
vertices and edges in each path in order, then this ER boils down to a
bug report together with an ER to the effect that there should be a
variant of or enhancement to traversedElement() that would obviate
the need to specify a bound.

Hi @pkoppstein
yes, it makes sense. I'm marking it as an enhancement and as contributor friendly.
For you query, could you please try this:
select expand(traversedElement)
from (select traversedElement(0,10)
      from (traverse outE(), inV() from #11:0 ) )
      where last(traversedElement) = #11:3

@luigidellaquilla - Thanks for reviewing the ER.
It is true that the query you mention returns both vertices and edges, but queries of that sort find too many paths.  In the present case, there is only one path from the start to the end vertex, but:
select $path from (traverse outE(), inV() from #11:0) where traversedElement(-1) = #11:3;
----+------+-------------------------------------------------------------------------------------
#   |@CLASS|$path
----+------+-------------------------------------------------------------------------------------
0   |null  |(#11:0).outE[0](#13:2).inV(#11:1).outE[0](#14:1).inV(#11:2).outE[1](#15:3)
1   |null  |(#11:0).outE[0](#13:2).inV(#11:1).outE[0](#14:1).inV(#11:2).outE[1](#15:3).inV(#11:3)
----+------+-------------------------------------------------------------------------------------

That is, the first result ends in an edge.  Since the query includes the condition traversedElement(-1) = #11:3), this looks like a bug to me.
As I said, this ER might turn out to be a combination of a bug report and ER. If so, the bug seems quite serious to me, and hopefully it will get prioritized accordingly.

Not relevant anymore.


Issues getting started

So I am trying to use MSPypeline through Jupyter Notebook and keep getting errors.
I ran:
!pip install mspypeline

from mspypeline import MSDataFile
from mspypeline.plotting import plot_ion_mobility_spectrum

And received the following warning:
UwerWarning: R_HOME environment variable is not set and location could not be inferred
So then I went into my systems enivronments and fixed it to contain a R_HOME environment with the correct installation directory. But now when I run the last bit, I get the following error:
ImportError                               Traceback (most recent call last)
Cell In[9], line 1
----> 1 from mspypeline import MSDataFile
      2 from mspypeline.plotting import plot_ion_mobility_spectrum

ImportError: cannot import name 'MSDataFile' from 'mspypeline' (C:\Users\STGECC\AppData\Local\anaconda3\lib\site-packages\mspypeline\__init__.py)

I tried reinstalling and updating MSPypeline multiple times and it doesn't seem to be helping. Any help would be great.

Hi @cstgermain4 , so the import worked intially, after you ran the pip install command? What did you do to fix the warning exactly? Could you maybe also have manipulated other environment variables, which point to your python installation?
Did the import work again after you reinstalled but not set the environment variable?
In general I would recommend to not install anything from within a jupyter notebook but testing it plain on the console.
So depending on what you use create a new py venv or conda venv, activate it in the console, install mspypeline, start the python interpreter on the console and test the imports you listed.
Also, since you plan to work from within a notebook you cold set the venv variable in the notebook by import os; os.environ["R_HOME"] = os.path.join("path", "to", "your", "R")

So I've tried running it both in python and in a conda environment. When I try in the python, I am unable to create a local environemnt:

And in Anaconda it will cycle through the following two things until it gives up (collecting package metadata followed by solving environment: failed)

I reinstalled my python and everything to make sure it was all up to date.
When I received the warning:
UwerWarning: R_HOME environment variable is not set and location could not be inferred

I went into my settings-->edit environment variables-->New, then added R_HOME with a file directory to my Rprogram. Then I restarted my jupyter kernal and reran the installation.
I also just ran your suggestion about the import, and maybe I am doing this in the wrong order. I am pretty new to coding, so if I mess something up please feel free to let me know. So in the notebook I ran the following:
import os

os.environ["R_HOME"] = os.path.join(r"C:\Users\STGECC\AppData\Local\Programs\R\R-4.3.1\bin\x64")

python3 -m venv mspypeline

output:
  Cell In[5], line 1
    python3 -m venv mspypeline
               ^
SyntaxError: invalid syntax

I'm open to using a python or anaconda environment instead of Jupyter, I just felt like I got further in the jupyter notebook.

There are multiple issues with different things you wrote about, but they might not be really relevant :D
I am not really sure where you got the name MSDataFile from, maybe you were looking at another library?
examples of how to analyse your data in a custom fashion can be found here: https://mspypeline.readthedocs.io/en/latest/examples.html

it might be simpler to start with the gui first to analyse your data, which is explained here https://mspypeline.readthedocs.io/en/latest/get_started.html#usage-with-gui

I will close this issue for now, hope you had success


getEditText return new instance

You are not able to reference the editText used for entering the input. This means I can't add extra TextWatcher nor can I get the current text input.

Also it is not possible to disable Edittext and solve autofocus problems. Do you think this will be fixed or should I make a fork?
PS: nice lib! thanks!

Hi @mariciv, @pchmn
I've created a PR with a fix to the EditText new instance problem. Please have a look. This is a temporary fix and I would recommend to move the EditText initialisation to the View layer, instead inside of the adapter. This will be, ofc, up to @pchmn ;)

Is there any chance for this gets merged or fixed in some other way? @pchmn

in case you don't wanna wait to PR or Fork the project you can use this trick to get the actual ChipInputEditText
@BindView(R.id.configuration_mails)
ChipsInput editMails;

RecyclerView recyclerView = (RecyclerView) editMails.findViewById(R.id.chips_recycler);
ChipsInputEditText chipsInputEditText = (ChipsInputEditText(recyclerView).getChildAt(recyclerView.getChildCount() - 1);

@TitusCln you are doing dirty things! but I like it ;)

in case you don't wanna wait to PR or Fork the project you can use this trick to get the actual ChipInputEditText
@BindView(R.id.configuration_mails)
ChipsInput editMails;

RecyclerView recyclerView = (RecyclerView) editMails.findViewById(R.id.chips_recycler);
ChipsInputEditText chipsInputEditText = (ChipsInputEditText(recyclerView).getChildAt(recyclerView.getChildCount() - 1);

this is not working, is there any way to add a contact manually after pressing enter key

@pchmn any reason this has not been merged yet? is there any other solution apart from the suggested one above ?

Hi @ajay-nagarajan
You can always pull from the PR I did with a temporary fix.
https://github.com/pchmn/MaterialChipsInput/pull/51


TCGA analysis

"Run" scripts and helper functions for search and analysis of patient survival and primary tumor gene set expression

added files


fix: openid4ci integration test updated to work with latest changes t…

…o openid4ci api.
Signed-off-by: Volodymyr Kubiv<EMAIL_ADDRESS>

Codecov Report
Base: 90.97% // Head: 90.97% // No change to project coverage :thumbsup:

Coverage data is based on head (baf330f) compared to base (2cdfab9).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #94   +/-   ##
=======================================
  Coverage   90.97%   90.97%
=======================================
  Files          32       32
  Lines        1031     1031
=======================================
  Hits          938      938
  Misses         56       56
  Partials       37       37

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add individual stmt dispose when not caching statements

I've included the try/finally when not using the prepared statements cache.
@simolus3 Do you want to use this PR to change the default flag of cachePreparedStatements to false?
I'm not sure what could cause the SQLITE_BUSY error from #2464 in this context.

:sweat_smile: Yes, I've just seen your changes


Error after logging in

I get this message after logging in. I have downloaded the encrypt32 and i have also tried the 64. Maybe that is not part of the problem?

Same issue.

Try to delete the following folder: C:\Users\Username\AppData\Roaming\PokemonGoBot
Restart the apps again, set your logins and the right encrypt32.dll location.
Should solve the issue.

I have deleted the folder. Its giving me a server busy, try again in 10 seconds message. hmm.. waiting..waiting..

I'm getting the same message even after deleting the folder.

I have a similar problem.
Deleted the folder, re-entered all in the info and also getting the error.


Random nodelets "finished cleanly" at launch

Describe the bug
I wonder whether anyone has seen similar behaviors. I am using ROS nodelets as the basic operation units of my program. Usually there are about 10 nodelets loaded to one manager. I have been using this way for a while and have not had problems.
Now I am running into strange problems when I test my ROS program on other computers. Every time I ran roslaunch, All nodelets were initialized successfully (according to log file), but after 10-15 seconds, two of the nodelets would die with "$$$ has finished cleanly" printed to the log. Every time it could be two different nodelets, and it seemed random.
To Reproduce
Steps to reproduce the behavior:
I don't know how to make this into a simple case for others to reproduce. We've tested the same program on different computers, and not all of them suffer from it. The first time we saw this was a couple of months ago on one PC. We couldn't resolve it but decided to move on to another computer since it was only one case and we just assumed something wrong in the settings. In the past week, several of our computers started this problem - some of them have been used to run this program for months. And once it started on one computer, it would not go away, even after we reverse the code to previous versions. It just kept happening every launch.
Expected behavior
The expected behavior is obvious. I understand I am not providing much as I don't know where to look further. I would appreciate any suggestion to try out or get more information to help us diagnose.
Environment information:

OS version: winver
Chocolatey Versions: choco list --local-only

This is the launch file if it helps.

@songlan-hl, Hi, thanks for the reporting. I would recommend to start from enabling more logging to see any obvious problems there. Can you try to do the following?

Create and use rosconsole.config to enable nodelet related DEBUG logging.

# Override nodelet package to output everything
log4j.logger.ros.nodelet=DEBUG

In the launch file, add output="screen" to the nodelet and force everything to stdout to be printed.

You may also try to use ROS extension to do debug from a launch file and see if any issues seen in the debugger.

Hope that helps the debugging.

Not much can be found here:
[ INFO] [1574716365.333913900]: waitForService: Service [/MYPROGRAM_job_manager/load_nodelet] has not been advertised, waiting...
[ INFO] [1574716365.726890200]: waitForService: Service [/MYPROGRAM_job_manager/load_nodelet] is now available.
[ INFO] [1574716365.726612600]: waitForService: Service [/MYPROGRAM_job_manager/load_nodelet] is now available.
[ INFO] [1574716365.726756500]: waitForService: Service [/MYPROGRAM_job_manager/load_nodelet] is now available.
[ INFO] [1574716365.728351800]: waitForService: Service [/MYPROGRAM_job_manager/load_nodelet] is now available.
[ INFO] [1574716365.728490300]: waitForService: Service [/MYPROGRAM_job_manager/load_nodelet] is now available.
[ INFO] [1574716365.728919700]: waitForService: Service [/MYPROGRAM_job_manager/load_nodelet] is now available.
[ INFO] [1574716365.733331900]: waitForService: Service [/MYPROGRAM_job_manager/load_nodelet] is now available.
[ INFO] [1574716365.733579400]: waitForService: Service [/MYPROGRAM_job_manager/load_nodelet] is now available.
[ INFO] [1574716365.734704100]: waitForService: Service [/MYPROGRAM_job_manager/load_nodelet] is now available.
[ INFO] [1574716379.627049200]: Bond broken, exiting
[ INFO] [1574716379.637663900]: Bond broken, exiting
[IOController-6] process has finished cleanly
log file: C:\Users\DR.ros\log\4a4ce291-0fc8-11ea-86d7-982cbc970c9c\IOController-6*.log
[inspector1-9] process has finished cleanly
log file: C:\Users\DR.ros\log\4a4ce291-0fc8-11ea-86d7-982cbc970c9c\inspector1-9*.log

actually there is more in master.log
[rosmaster.master][INFO] 2019-11-26 05:33:21,351: publisherUpdate[/HLZN_job_manager/bond] -> http://DESKTOP-I78E87T:55221/ ['http://DESKTOP-I78E87T:55236/', 'http://DESKTOP-I78E87T:55199/', 'http://DESKTOP-I78E87T:55226/', 'http://DESKTOP-I78E87T:55201/', 'http://DESKTOP-I78E87T:55209/', 'http://DESKTOP-I78E87T:55185/', 'http://DESKTOP-I78E87T:55231/', 'http://DESKTOP-I78E87T:55195/', 'http://DESKTOP-I78E87T:55217/']: sec=8.01, exception=[Errno 10061]
[rosmaster.threadpool][ERROR] 2019-11-26 05:33:21,354: Traceback (most recent call last):
File "C:\opt\ros\melodic\x64\lib\site-packages\rosmaster\threadpool.py", line 218, in run
result = cmd(*args)
File "C:\opt\ros\melodic\x64\lib\site-packages\rosmaster\master_api.py", line 210, in publisher_update_task
ret = xmlrpcapi(api).publisherUpdate('/master', topic, pub_uris)
File "C:\opt\python27amd64\lib\xmlrpclib.py", line 1243, in call
return self.__send(self.__name, args)
File "C:\opt\python27amd64\lib\xmlrpclib.py", line 1602, in __request
verbose=self.__verbose
File "C:\opt\python27amd64\lib\xmlrpclib.py", line 1283, in request
return self.single_request(host, handler, request_body, verbose)
File "C:\opt\python27amd64\lib\xmlrpclib.py", line 1311, in single_request
self.send_content(h, request_body)
File "C:\opt\python27amd64\lib\xmlrpclib.py", line 1459, in send_content
connection.endheaders(request_body)
File "C:\opt\python27amd64\lib\httplib.py", line 1038, in endheaders
self._send_output(message_body)
File "C:\opt\python27amd64\lib\httplib.py", line 882, in _send_output
self.send(msg)
File "C:\opt\python27amd64\lib\httplib.py", line 844, in send
self.connect()
File "C:\opt\python27amd64\lib\httplib.py", line 821, in connect
self.timeout, self.source_address)
File "C:\opt\python27amd64\lib\socket.py", line 575, in create_connection
raise err
error: [Errno 10061]

could the problem be the connection timeout?

@songlan-hl This log looks promising. It looks like the Windows firewall is blocking the connection. Could you try (in a secure environment) disabling the firewall and re-running to see if this reproduces?

I turned off windows defender firewall. And it didn't show any difference in the behaviors.
Is there a way to extend the timeout limit?

I tried adding 11311 to the inbound rule of the firework as well and still no change. It is puzzling as other 8 nodelets apparently were successfully launched. And only 2...

Another thing about my setup: I used more than one publishers to publish to the same topic. It was working fine even though there were messages "ERROR: failed to advertise a service that is already advertised in this node". According to some general ROS forum this message can be ignored. And I didn't have problems on most of the computers before.  On the computer I am debugging now, its ros system was installed recently. Up to this week it was running ok although didn't show these ERROR messages. Now it's having problems....

The Firewall has caused issues in the past, but that isn't it for this bug.
(Debugging nodelet failures is quite difficult, we'll look to see what we can do to help improve this).
A Nodelet is failing to load, which can be caused by a number of reasons:

The nodelet is missing a dependency.
What the nodelet is connecting to is failing.

Debugging nodelets
To check these, it is best to capture the node under a debugger. Sean mentioned vscode debugging. You can isolate the failing nodes into a single roslaunch file, then start the rest of the ros graph minus those nodes. You can then use the VSCode ROS extension to launch the nodelet manager under a debugger.
Dependencies
If a nodelet is missing a dependency it will fail to load at all. This can be difficult to diagnose. Fortunately, the Windows Debugger tools has a feature called gflags.exe. Install the Windows Debugger Tools, then load gflags.exe. In the image file execution options, type nodelet.exe, then press tab. Select Snow Loader Snaps, which will dump every import and export. You can see at runtime what it was attempting to load - and what failed.
Failing after startup
If the nodelet is failing after startup, without good error messages, you'll need to step through the startup of the node itself.
Would you be able to provide repro steps?

Thanks for taking the time. Looking through the possible reasons, I wonder why these would trigger "random" nodelets to stop working? Everytime the two nodeletes (now I have seen some cases failed with one nodelet in the past couple of hours) could be different.

At this point, I don't think the issue is systemic, so I'd like to understand what's happening within the nodelets themselves. Could you provide a repro? (Are they in a public github we can investigate?)

I will try to simply our application into something easier to reproduce.
In the meanwhile, I am tracing backward in ros system logs. The previous connection failure actually happened after the two nodes aborted, so it's not the cause. Around the time of abortion, master.log showed this:

[rosmaster.master][INFO] 2019-11-26 08:39:34,213: publisherUpdate[/HLZN/original_cam_image] -> http://DESKTOP-I78E87T:57101/ []
[rosmaster.master][INFO] 2019-11-26 08:39:34,214: publisherUpdate[/HLZN/original_cam_image] -> http://DESKTOP-I78E87T:57101/ []: sec=0.00, result=[1, '', 0]
[rosmaster.master][INFO] 2019-11-26 08:39:35,740: -PUB [/rosout] /cameraController http://DESKTOP-I78E87T:57060/
[rosmaster.master][INFO] 2019-11-26 08:39:35,740: -PUB [/HLZN_job_manager/bond] /cameraController http://DESKTOP-I78E87T:57060/
[rosmaster.master][INFO] 2019-11-26 08:39:35,742: -SUB [/HLZN_job_manager/bond] /cameraController http://DESKTOP-I78E87T:57060/
[rosmaster.master][INFO] 2019-11-26 08:39:35,743: -SERVICE [/cameraController/get_loggers] /cameraController rosrpc://DESKTOP-I78E87T:57059
[rosmaster.master][INFO] 2019-11-26 08:39:35,744: -SERVICE [/cameraController/set_logger_level] /cameraController rosrpc://DESKTOP-I78E87T:57059

In another run where the cameraController node did not abort, the corresponding message is

[rosmaster.master][INFO] 2019-11-26 08:40:27,532: publisherUpdate[/HLZN/original_cam_image] -> http://DESKTOP-I78E87T:57534/ ['http://DESKTOP-I78E87T:57534/']
[rosmaster.master][INFO] 2019-11-26 08:40:27,532: publisherUpdate[/HLZN/original_cam_image] -> http://DESKTOP-I78E87T:57534/ ['http://DESKTOP-I78E87T:57534/']: sec=0.00, result=[1, '', 0]

where 57534 is related to job manager. I wonder if you have any quick thoughts about this.

Nodelets need to be dropped into the correct path in order to be found. If you are still hitting this problem, please reopen with details about which nodelets have issues


Bugfix notify enabled

Which changes (Bug/Feature):

bugfix; handlers always nil
Special notes for reviewers:
none

thanks to @gosszhang


Constraint selection can be unreasonably slow for time coords

Discovered that doing a date selection for a given year + month based on categorised coordinates, with an operation like month_data = cube.extract(Constraint(year=y, month=m)).
It seems much quicker to do
indices = (cube.coord('year').points == y) & (cube.coord('month').points == m)
month_data = cube[indices]

In short, this is because operation is not vectorised (unlike the above index-calculating solution).
This is worse for time coordinates, as a unit.num2date() is applied individually to each time cell (points + bounds), before calling the discriminator function.
This could definitely be fixed, within the existing API, at least for the non-user-function coordinate selection styles ("equals-point-value" and "in-list-of-values").
A useful addition would be an extension to the extract API to allow user functions to also be called with vector info.
E.G. either a recognised property visible on the function, or a keyword like vectorise_functions=True ?
TODO:
add a simple testcase demonstrating the performance hit

This was done at #4969 🎉


adding agenda for next community call

This change is

Not found:
https://github.com/PowerShell/DscResources/CommunityCalls/DSC Resource Kit Community Call ICS.zip
As has been true for over a year now.
Thanks.
Regards,
Michael B.
From: Gael<EMAIL_ADDRESS>Sent: Sunday, April 7, 2019 10:08 PM
To: PowerShell/DscResources
Cc: Subscribed
Subject: [PowerShell/DscResources] adding agenda for next community call (#495)

You can view, comment on, or merge this pull request online at:
https://github.com/PowerShell/DscResources/pull/495
Commit Summary

adding agenda for next community call

File Changes

A CommunityCalls/2019-05-09/agenda.mdhttps://github.com/PowerShell/DscResources/pull/495/files#diff-0 (35)

Patch Links:

https://github.com/PowerShell/DscResources/pull/495.patch
https://github.com/PowerShell/DscResources/pull/495.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/PowerShell/DscResources/pull/495, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AOe439h88rUUk9xUZUayB5jb7WMmB_kgks5veqSQgaJpZM4chKH4.

Thanks for the heads up @swngdnz will update the link. The ICS has been fixed though, just the link seems wrong.

Here's the correct link

That seems to work. At least I have a meeting for 8-May at 15:00 eastern time.
Thanks.
Regards,
Michael B.
From: Gael<EMAIL_ADDRESS>Sent: Sunday, April 7, 2019 10:13 PM
To: PowerShell/DscResources
Cc: Michael B. Smith; Mention
Subject: Re: [PowerShell/DscResources] adding agenda for next community call (#495)
Here's the correct linkhttps://github.com/PowerShell/DscResources/blob/master/CommunityCalls/DSC Resource Kit Community Call ICS.zip
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/PowerShell/DscResources/pull/495#issuecomment-480659499, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AOe431L-ogYi2Z58RNWxtWKk9s453rhwks5veqW8gaJpZM4chKH4.

Ah, you pointed me to a mistake! Thanks again! I put 9th of May, because of my timezone!

So what is the correct link?
Thanks.
Regards,
Michael B.
From: Gael<EMAIL_ADDRESS>Sent: Sunday, April 7, 2019 10:17 PM
To: PowerShell/DscResources
Cc: Michael B. Smith; Mention
Subject: Re: [PowerShell/DscResources] adding agenda for next community call (#495)
Ah, you pointed me to a mistake! Thanks again! I put 9th of May, because of my timezone!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/PowerShell/DscResources/pull/495#issuecomment-480660084, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AOe436GkONQaB1In3SrLPSFqdwvrm9Hwks5veqawgaJpZM4chKH4.

The link was and is OK. Just the Agenda.md and its parent folder had an error. I'm pushing the correction and will merge it in.

Thank you!
Thanks.
Regards,
Michael B.
From: Gael<EMAIL_ADDRESS>Sent: Sunday, April 7, 2019 10:21 PM
To: PowerShell/DscResources
Cc: Michael B. Smith; Mention
Subject: Re: [PowerShell/DscResources] adding agenda for next community call (#495)
The link was and is OK. Just the Agenda.md and its parent folder had an error. I'm pushing the correction and will merge it in.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/PowerShell/DscResources/pull/495#issuecomment-480660753, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AOe43zt9FIYxRrVfBgFYtby25EXb7wEFks5veqepgaJpZM4chKH4.


Question about dependent fields

Hi,
I'm doing a form with 2 dependent fields, linked together through a m2m relation:
class PaperType(Administrable, models.Model):
    """
    Paper weights and types
    """
    #: Paper weights, in g/m²
    weight = models.PositiveIntegerField(verbose_name=_('weight'), help_text=_('In g/m²'))

    #: Extra
    extra = models.CharField(max_length=255, verbose_name=_('extra'), null=True, blank=True)

class PaperColor(Administrable, models.Model):
    """
    Paper colors
    """
    #: color label
    label = models.CharField(verbose_name=_('label'), max_length=255)

    #: Available paper weights for this color
    available_paper_types = models.ManyToManyField('printflow.PaperType',
                                                   related_name='available_paper_colors',
                                                   blank=True)

Then I've set a relation between 2 ModelSelect2Widget (it is in a ModelForm, I've not set the form fields, but only the widgets:
class PrintJobForm(forms.ModelForm):
    class Meta:
        widgets = {'paper_type': s2_widgets.ModelSelect2Widget(model=models.PaperType,
                                                               search_fields=['extra__icontains',
                                                                              'weight__icontains', ],
                                                               dependent_fields={
                                                                   'paper_color': 'available_paper_colors'},
                                                               max_results=50
                                                               ),
                         'paper_color': s2_widgets.ModelSelect2Widget(model=models.PaperColor,
                                                                search_fields=['label__icontains'],
                                                                dependent_fields={
                                                                    'paper_type': 'available_paper_types'},
                                                                max_results=50
                                                                ),}

I can see in the html the data-select2-dependent-fields="paper_type" attribute in the paper_color field definition, but when I step-by-step debug the ModelSelect2Mixin.filter_queryset() method, no data in passed to the queryset: no GET "paper_type" parameter is send to the view.
Do you think it is possible to enable the dependent_fields in this case ? Did I miss something ?
Thanks for your great django application and for your help  :)

I've got it: I'm creating a form wizard, so the name of the fields are prefixed with the step names...
** Quick fix:  **
I've updated the dependent_fields from paper_type to JOBS-paper_type, and it works.
** Long (better) fix:  **
Detect the name of the wizard step on the data-select2-dependent-fields="paper_type" generation...
Thanks !

@frague59 thanks for reaching out. Good that you made it work, but I believe you just uncovered a bug. Seems the dependent field features doesn't play nice with form prefixes.

@a-ruzin can you check that?

@frague59 Could you post full fix.
Is the problem "you have two forms on a single page with same input names"?

@codingjoe btw, is this relates to https://github.com/applegrew/django-select2/pull/399 ?

@a-ruzin it does seem related to #399, could you review that PR?

I will review. But He suggests a javascript hack (as I do in my first proposal for dependent fields). But it's better to change field names in python (as You insisted). I need some days to investigate the problem and solution.

@a-ruzin
I agree with this last proposal, because using python will work also for the wizards (I hope so...).
The point is to use dthe form_prefix as prefix for the fields, as done in formtools:
https://github.com/django/django-formtools/blob/master/formtools/wizard/views.py, line 405
...
'prefix': self.get_form_prefix(step, form_class),
...

The main problem here seems to be that prefixes are not supported for dependent fields. Which is bad, since everything else does. Prefixes are a common concept in django.forms. I think we should really fix it. I did write the dependent fields feature (nor do I use it). I would really welcome some help here 👍

Would a regex in javascript be enough? I think there have been already similar proposals, something like this might work if the convention to use a dash to separate the prefix is kept:
          var dependentFields = $element.data('select2-dependent-fields')

          var prefix_regex = new RegExp("(.+-).+");
          var prefix_match = prefix_regex.exec($element.attr('name'));
          var prefix = '';
          if (prefix_match.length >= 2) {
            prefix = prefix_match[1]
          }

          if (dependentFields) {
            dependentFields = dependentFields.trim().split(/\s+/)
            $.each(dependentFields, function (i, dependentField) {
              result[dependentField] = $('[name=' + prefix + dependentField + ']', $element.closest('form')).val()
            })
          }

Solving this in python seems more difficult, the form class would have to be modified to pass the prefix to the widget.

@bc500 I would prefer to have the prefix prepended on python using data-select2-dependent-fields. At this point they should already have the correct prefix. This should solve the issue and avoids any issues with dependent fields pointing to anything but the current form. Sadly the prefix is not available in widgets per se. It should be extractable though.

What do you mean by extractable? As far as I understand, there is no visibility on the form object from the widget, as you pointed out.

I added a new in the current pull-request, but essentially. By default Django uses - as a prefix separator. This character is not allowed for class attributes. So it's save to assume that this is the separator even if you don't have access to the form.

It won't work in the cases where people override the two methods that build the prefixes, but hey. There are many ways to break this package ;)

Closed: This issue has been inactive for an extended period of time. Please reopen the issue if it is still relevant.


Needed help.

Hello!
Basically I have some problems with my website.
At first, timestamps are not working correctly.

And second. Song/video doesnt update some times

And finally. When presence doesnt have timestamp, console goes crazy.

This is not a lanyard issue, you've already asked in the discord and received an answer sufficient to solve your issue.
Lanyard doesn't do any processing on the data. Just returns what discord gives us, the start and end timestamp need to be used for calculation, in the same way the discord client does.
In the case of both start and end timestamp existing, you calculate (end-start) and that's the how long in milliseconds that activity will be going on for, and from there you can do all kinds of things, like to get time till end of song take the current date milliseconds and subtract it from the end timestamp.
Closing this, as this is not a support channel for non-lanyard related bugs, this is with your implementation of the math, please try using discord again for this.


Test failure in 0.2.6

Hi there,
I'm running tests as part of AUR package, and it shows one failed test, which I can also reproduce manually 😉
It's mostly FYI, plus a suggestion to perhaps use Github Actions to run tests on main branch?
If you want, this works really well for my other rust project:
name: CI
on:
  push:
    branches: [main]
  pull_request:
    branches: [main]

jobs:
  test:
    name: test
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
      - run: cargo test --locked

  lint:
    name: lint
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
      - run: cargo fmt -- --check
      - run: cargo clippy -- -Dwarnings

I'll deal with it tonight

Hi, thanks for the heads-up! I made a mistake when updating the specification of the test case, and I've now fixed it in v0.2.7. I'll be sure to add in the automated testing Action as soon as possible. Do let me know if there are any further issues with this release.

Thanks! There is an issue of Cargo.lock being not up-to-date with Cargo.toml (it still shows 0.2.6 for example), so building the project with --locked flag (to make sure it uses dependency versions exactly as you wanted them to be) is failing. I think if you just do cargo build or any command, it will update it 👍

I made a PR for you to play with, you can see how it fails now and how after updating Cargo.lock it should hopefully succeed: https://github.com/Gskartwii/teehee/pull/13

Okay, I hope third time's the charm! For the sake of not overwriting Git tag history, I've created yet another release v0.2.8. 😅


Wallet Address Difference Problem

When I log in to apollo.cafe all the como counts, completed grids, and total special objects are all correct. For collections and in the profile it is connected to someone else's account, not mine, maybe because the wallet address in apollo.cafe is different from the one in my cosmo app.

This usually happens when someone does the delete account trick to change IDs, and ends up swapping IDs between accounts. Let me know your Cosmo ID, Polygon address and any previous IDs if this is something you've done, and I can manually fix it up. Feel free to message me on Discord too if you'd like: kairunz


[DO NOT MERGE] Auto PR to update dependencies of istio

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
1c8f6d2791ad3f37c7e8f8123cac328f238cf0c4
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


[W5.7][W13-4] Ong Yan Zhi

Review made for W5.7:
Good encapsulation of Sort Function in UniquePersonList.java instead of the SortCommand.java file.


[HOLD #7535] [$250] Refactor StatePicker to be compatible with Form

With the implementation of our new Form component we need to refactor StatePicker to be Form compatible. Here are the changes that need to be made:

An optional isFormInput prop.
If isFormInput is true, require a inputID prop. Use getInputIDPropTypes to enforce this propType rule.
Add an optional shouldSaveDraft prop. Defaults to false.
Make the value prop optional.
In the input’s onBlur method, call props.onBlur().
In the input’s onChange method (or equivalent e.g. onTextChange, etc), call props.onChange().
Remove the hasError prop.
Add an optional errorText prop. Defaults to an empty string.
Update the error message to display if errorText is truthy.
Update the inline error display style so it is left aligned with the label and input value. See example image for TextInput:

Make sure that props.ref is attached to the appropriate DOM node. This could involve forwarding the ref to a child component. This is important so we can call ref.focus().
Add the input to the test Form stories and make sure that it works. You can check this by running npm run storybook and testing the component in the example form.
Remove any unused code.

There's an example of a refactor to TextInput in this PR.

Triggered auto assignment to @MitchExpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

Upwork Job

Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat (Exported)

Triggered auto assignment to @thienlnam (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

The problem statement seems quite self-explanatory in regards of the changes needed. Can I possibly work on it? Thanks.

@sig5  Do you have questions before we move forward? I would suggest to give it another look and ask questions if any.
Though, I have one :smile:

Add an optional shouldSaveDraft prop. Defaults to false.

@luacmartins Does it need to be implemented as well or just define a prop for now?

Hi @parasharrajat , Please correct me if I am wrong or misunderstanding the problem, But  I can observe, shouldsaveDraft is used to see if the form element needs to save value as the draft.
It is handled here.
https://github.com/Expensify/App/blob/4c48227a265040fca6077e743907583d04b63b47/src/components/Form.js#L144-L167
So adding a propType in the component along with a defaultProp should suffice to trigger it save in Onyx, which can be fetched next time it is mounted.

Yeah, that's correct.
I think we can proceed with your proposal @sig5  if you don't have any questions.
cc: @thienlnam
:ribbon: :eyes: :ribbon:  C+ reviewed

I think I do not have any questions atm :D

Does it need to be implemented as well or just define a prop for now?
@sig5 comment is correct. We have already implemented that and should only need to pass shouldSaveDraft + call onChange in the input as described in the OP.

Sending it! @MitchExpensify could you please hire @sig5 ?

For sure! Can you please make a proposal on the Upwork job @sig5 ?

Done!

Thanks, hired @sig5 !

Hey guys, I am coming across this error whilst adding the StatePicker to the storybook. Seems like a Transpilation error related to Flow. I am not sure how to tackle it.
RR! ./node_modules/react-native/Libraries/Components/UnimplementedViews/UnimplementedView.js 19:47
ERR! Module parse failed: Unexpected token (19:47)
ERR! You may need an appropriate loader to handle this file type, currently, no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
ERR! |  * View component and renders its children.
ERR! |  */
ERR! > class UnimplementedView extends React.Component<$FlowFixMeProps> {
ERR! |   render(): React.Node {
ERR! |     // Workaround require cycle from requireNativeComponent
ERR<EMAIL_ADDRESS>2:0-103 5:42-59 12:17-34
ERR<EMAIL_ADDRESS>ERR!  @ ./node_modules/react-native-picker-select/src/index.js
ERR!  @ ./src/components/Picker/BasePicker/index.js
ERR!  @ ./src/components/Picker/index.js
ERR!  @ ./src/components/StatePicker.js
ERR!  @ ./src/stories/Form.stories.js
ERR!  @ ./src sync ^\.(?:(?:^|\/|(?:(?:(?!(?:^|\/)\.).)*?)\/)(?!\.)(?=.)[^/]*?\.stories\.(js|jsx|ts|tsx))$

This seems like a relevant issue: https://github.com/storybookjs/storybook/issues/10131
Can someone suggest changes on how to integrate Flow support for the storybook? ( Or tag someone? )

@sig5 @parasharrajat Before starting on this one, we need to first merge a PR for Refactor Picker right?

Yes  Statepicker uses Picker Component under the hood, so having those changes done first would be ideal.

Good eye @rushatgabhane. We are already waiting for some issue @sig5 is having.

The issue is also encountered in Picker. The Picker PR should resolve the issue. We have the solution which is adding a devDependency as discussed in slack. In the worst case, we can apply that, Marc has approved it. I will update on the  Picker issue if I found a better solution :D.

Looks like we're holding on https://github.com/Expensify/App/issues/7535 before starting this

Still holding

Same as above

Putting on a monthly until the holding PR is resolved

The holding PR is merged! We can start working on this 🎉

Not overdue, just became ready to work on

@sig5 Let us know when would you be able to submit the PR.

Hi @parasharrajat , i had taken up this issue when I had enough time for it. But since the PR it is dependent on has taken too much time, there are some personal commitments at the moment, i think it will be in everyone's best interest for me to drop this issue as I can not allocate the time needed for this PR.

Ok. Thanks, @sig5 for the updates. Open for new proposals...

Doubling price and creating a new Upwork post because the older one is closed.
New Upwork Job (Invited @parasharrajat for eventual C+ payment)

Hey everyone! Since this is taking too long to be completed externally and I'd like to push this project forward, I'll switch this to internal and work on it myself. Thanks!

@luacmartins is there any chance it can be External again?
I saw this issue a few hours back and I was working on this and I have done all the changes that are mentioned I was testing these changes in the app, here are videos
[Form stories]
https://user-images.githubusercontent.com/35371050/164755723-0c8407b0-afe1-4e9b-9087-5f403dfa5ca7.mov
DebitCardPage Before Changes
https://user-images.githubusercontent.com/35371050/164756039-58e41b4c-756e-4d1b-9299-0948c837f0ec.mov
DebitCardPage After Changes
https://user-images.githubusercontent.com/35371050/164756113-2f24d044-68fd-4136-8b4a-44b6639d7c6a.mov

Hey @jayeshmangwani! Sure, feel free to pick it up and submit a PR. I'll re-add the labels and re-assign the relevant people to this issue. Thanks for the interest and picking this up.
@MitchExpensify Please disregard my comment above about removing this job. We are making it External again and @jayeshmangwani will work on this.

Ok Great. @jayeshmangwani this is urgent so I am hoping that PR will be ready by Monday.

@jayeshmangwani Can you apply to this Upwork Job please? I had to abandon the original job so we'll use this new posting to issue payment (invited you to the new job @parasharrajat)

@jayeshmangwani Can you apply to this Upwork Job please? I had to abandon the original job so we'll use this new posting to issue payment (invited you to the new job @parasharrajat)

Sure I am applying

Ok Great. @jayeshmangwani this is urgent so I am hoping that PR will be ready by Monday. Thanks for taking it up.
:ribbon: :eyes: :ribbon:  C+ reviewed

Sure 👍

PR raised
@parasharrajat

https://www.upwork.com/jobs/~0116994c22a1b60fb6 mind applying here @parasharrajat so I can issue C+ payment?

Paid @jayeshmangwani , thank you very much!

@MitchExpensify Thankyou

@MitchExpensify Done.

Done! Thanks all


Example ble_existing_connection_sniffer.py does not work

The example ble/ble_existing_connection_sniffer.py throws an error if executed:
Traceback (most recent call last):
  File "/home/mkesenheimer/tools/whad/whad-client/examples/ble/ble_existing_connection_sniffer.py", line 20, in <module>
    sniffer.configure(
  File "/home/mkesenheimer/tools/whad/venv/lib/python3.12/site-packages/whad/ble/connector/sniffer.py", line 186, in configure
    self._enable_sniffing()
  File "/home/mkesenheimer/tools/whad/venv/lib/python3.12/site-packages/whad/ble/connector/sniffer.py", line 124, in _enable_sniffing
    self.sniff_active_connection(access_address, crc_init, channel_map, hop_interval, hop_increment)
  File "/home/mkesenheimer/tools/whad/venv/lib/python3.12/site-packages/whad/ble/connector/__init__.py", line 426, in sniff_active_connection
    msg = self.hub.ble.create_sniff_active_conn(
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/mkesenheimer/tools/whad/venv/lib/python3.12/site-packages/whad/hub/ble/__init__.py", line 339, in create_sniff_active_conn
    sniff_connreq.channel_map = channel_map.value
                                ^^^^^^^^^^^^^^^^^
AttributeError: 'bytes' object has no attribute 'value'

channel_map is defined in ble/ble_existing_connection_sniffer.py like so:
channel_map=bytes.fromhex("ffefffff1f")

What is the correct value for channel_map?

Even replacing the above line with
channel_map=0xffefffff1f

does not work, as then the following error occurs:
  File "/home/mkesenheimer/tools/whad/venv/lib/python3.12/site-packages/whad/hub/ble/__init__.py", line 339, in create_sniff_active_conn
    sniff_connreq.channel_map = channel_map.value

AttributeError: 'int' object has no attribute 'value'

Which python type implements the value attribute?

The next error occurs, if the optional arguments crc_init and channel_map are removed from SynchronizedConnection:
Traceback (most recent call last):
  File "/home/mkesenheimer/tools/whad/whad-client/examples/ble/ble_existing_connection_sniffer.py", line 30, in <module>
    for i in sniffer.sniff():
             ^^^^^^^^^^^^^^^
  File "/home/mkesenheimer/tools/whad/venv/lib/python3.12/site-packages/whad/ble/connector/sniffer.py", line 279, in sniff
    if message.hop_increment > 0:
       ^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'hop_increment'

It seems our example was a bit outdated and also that some part of the BLE sniffer has not been tested enough. I've managed to reproduce this behavior, fixed these errors and I created a PR related to this issue.
You can check if these fixes solve your problems by testing the bugfix/issue-74 branch.

Thank you for addressing these issues. However, I don't think that fully fixed the problems.
I still get problems if I run the examples.
With a butterfly device:
$ python ble_existing_connection_sniffer.py uart0
Traceback (most recent call last):
  File "/home/mkesenheimer/tools/ble/whad/whad-client/examples/ble/ble_existing_connection_sniffer.py", line 31, in <module>
    for i in sniffer.sniff():
             ^^^^^^^^^^^^^^^
  File "/home/mkesenheimer/tools/ble/whad/venv/lib/python3.12/site-packages/whad/ble/connector/sniffer.py", line 279, in sniff
    if message.hop_increment > 0:
       ^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'hop_increment'

With a Bluetooth dongle:
$ python ble_existing_connection_sniffer.py hci0
Traceback (most recent call last):
  File "/home/mkesenheimer/tools/ble/whad/whad-client/examples/ble/ble_existing_connection_sniffer.py", line 19, in <module>
    sniffer = Sniffer(dev)
              ^^^^^^^^^^^^
  File "/home/mkesenheimer/tools/ble/whad/venv/lib/python3.12/site-packages/whad/ble/connector/sniffer.py", line 36, in __init__
    raise UnsupportedCapability("Sniff")
whad.exceptions.UnsupportedCapability: UnsupportedCapability(Sniff)

Sorry, I got something wrong with my virtual environment. The commit fixed the issues for me. Thanks!


Update julia set example.

Zoom increase/decrease should now scale depending on the current zoom, and the zoom should also now behave as expected (increased zoom = closer image).

@eggmund nice! thanks! 😀


Liked tracks are not playable

Describe the bug
When I try to play any track in the Liked tracks - nothing happens. Everything else, playlists and podcasts work fine.
To Reproduce
Go to Liked Tracks, try to play any track
Expected behaviour
Playback should start
Log and backtrace
2024-12-01T19:58:23.324372Z  INFO client_request{request=GetUserSavedTracks}: spotify_player::client: Successfully handled the client request, took: 13406ms
2024-12-01T19:58:24.362797Z ERROR client_request{request=Player(StartPlayback(Context(Tracks(TracksId { uri: "tracks:user-liked-tracks", kind: "Liked Tracks" }), Some(Uri("spotify:track:6zU0xyXN0AsFZ57dI3HM0s"))), None))}: spotify_player::client::handlers: Failed to handle client request: `StartPlayback` request for `tracks` context is not supported

Screenshots
If applicable, add screenshots to help explain your problem.
Environment

OS: Windows 11
Application version:  0.20.2

thanks for reporting. Look like the issue is introduced by the refactoring PR. Fix is in #625


C style comments mangled: indent, unindent, format code

Type: Bug
If you have some code like the following:
#include <cstdio>

void SomeFunction()
{
    /*
     * This is a comment.
     */
}

And you indent, unindent, or format the code (might be triggered by line completion or other automatic things like brace completion), then the C style comments are all left aligned at the ident level instead of the '*' alignment.
After: (wrong)
#include <cstdio>

void SomeFunction()
{
    /*
    * This is a comment.
    */
}

VS Code version: Code - Insiders 1.80.0-insider (6545c4c2671127c323182225963fcd732e1fbcc5, 2023-06-15T05:33:30.720Z)
OS version: Windows_NT x64 10.0.22621
Modes: Restricted

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz (12 x 3192)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
63.78GB (29.23GB free)

Process Argv
--crash-reporter-id a5e3b2dc-dfa2-4cce-8c59-ba8196782dc1

Screen Reader
no

VM
0%

Extensions (14)

Extension
Author (truncated)
Version

Bookmarks
ale
13.3.1

gitlens
eam
14.0.0

vscode-codeql
Git
1.8.6

vscode-test-explorer
hbe
2.21.1

vscode-docker
ms-
1.25.1

dotnet-interactive-vscode
ms-
1.0.4313020

python
ms-
2023.10.1

vscode-pylance
ms-
2023.6.20

jupyter
ms-
2023.5.1001582324

jupyter-keymap
ms-
1.1.2

hexeditor
ms-
1.9.11

test-adapter-converter
ms-
0.1.8

vsliveshare
ms-
1.0.5873

vscode-yaml
red
1.13.0

A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythontb:30258533
pythonptprofiler:30281269
vshan820:30294714
vscod805cf:30301675
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30404738
py29gd2263cf:30763247
vsclangdf:30492506
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30618038
pylantcb52:30590116
pyind779:30611226
pythonsymbol12:30651887
showlangstatbar:30737417
pythonms35:30671666
03d35959:30757351
ecj1e332:30687743
pythonfmttext:30716741
pythoncmvfstr:30726892
fixhidewlkth:30724384
showindicator:30766888
pythongtdpath:30726887
bgfeh915:30763804
gsof2:30767485
dh2dc718:30763024
pythonidxpt:30763123
pythondjangots:30763122
pythonnocebcf:30766856
copilotsettingc:30767685

VS Code doesn't come with a C formatter, you have to install an extension that provides a formatter for C files.

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!

There is at least the Tab and Shift-Tab when the three comment lines are highlighted. Doing this messes up the formatting. This is with no formatter installed. I will look for the code formatters, but I thought all I installed was basic C/C++ language support from Microsoft.


updates swarm overlay network diagram

- What I did
updates the diagram for the service on an overlay network to show the containers on the node and the nodes and the vip on the network.
Signed-off-by: Charles Smith<EMAIL_ADDRESS>

@mstanleyjones , @thaJeztah PTAL

LGTM!

LGTM


Begin migrating FIRLoggerService declarations to clients

This is the first step of migrating FIRLoggerService declarations from FirebaseCore to the clients

Need a SpecStaging update to pass travis

cc: @ryanwilson @chliangGoogle @dmandar
Merging now to reduce churn on SpecsStaging. If any additional review feedback, I'll create another PR.


-1 in Dex for NPCs leads to weapon problems

When you set an npc's dex to -1 and then try to edit a weapon in their inventory, you won't be able to
-2 in dex works fine, -1 in other attribute work fine

I've confirmed this issue. Thanks for the heads up.


[MM-15170] Fix admin console urls and system-scheme permission

Summary
Fixes license check for /permissions/system-scheme url and url inconsistencies related to the group links in system console.
Ticket Link
https://mattermost.atlassian.net/browse/MM-15170

@mgdelacroix a conflict needs to be fixed

This should have been fixed in my last PR #2656 , but looks like it became out-of-sync somehow.

This should have been fixed in my last PR #2656 , but looks like it became out-of-sync somehow.

I saw it when trying to merge, we fixed it in the exact same way so I changed the commit name to include only the url fix and rebased on top of your change.


--keep-invalid

There should be the ability to not prune on inventory watch.

watch was updated to only commit. inventory prune was used as an alternative


Modal handleUpdate is protected

From the docs:

If the height of a modal changes while it is open, you should call $('#myModal').data('bs.modal').handleUpdate() to readjust the modal’s position in case a scrollbar appears.

In the source the method is protected:
https://github.com/twbs/bootstrap/blob/045888fa3887f5e65658499da11c8ad737222759/js/dist/modal.js#L394

You can call call _handleUpdate by the following lines :
$('#myModal').modal('_handleUpdate')
// or
$('#myModal').data('bs.modal')._handleUpdate()

We should update the documentation


helm-docs

@karuppiah7890 How would you feel adding this functionality to helm-docs instead?
https://github.com/norwoodj/helm-docs/issues/74

I'll look into, but I can't commit anything. 😅 And thanks for communicating about this! :)

I'll look into, but I can't commit anything. 😅 And thanks for communicating about this! :)

And I would still keep this standalone tool just like that. I'll check how this feature can be added to helm-docs. I'm guessing there's a bit more work, as the current code generates just types and nothing else. In helm-docs, we want description, and I'm assuming probably even things like, if a field is required or not? Especially if there's no default value for it (a rare case though) in the default values.yaml . Another thing is, the current code also doesn't add any extra schema like additionalProperties to mention if unknown properties are allowed or not. Currently it's allowed, as that's the default.
This tool currently just gives the user a head start on writing their schema but not exactly a complete solution. But yeah, I agree that helm-docs capability can help the schema to be more enriched. I haven't found the time and reason to use helm-docs yet, as I don't use helm much now. But I'll check it out and get back into this

And I would still keep this standalone tool just like that. I'll check how this feature can be added to helm-docs. I'm guessing there's a bit more work, as the current code generates just types and nothing else. In helm-docs, we want description, and I'm assuming probably even things like, if a field is required or not? Especially if there's no default value for it (a rare case though) in the default values.yaml . Another thing is, the current code also doesn't add any extra schema like additionalProperties to mention if unknown properties are allowed or not. Currently it's allowed, as that's the default.
This tool currently just gives the user a head start on writing their schema but not exactly a complete solution. But yeah, I agree that helm-docs capability can help the schema to be more enriched. I haven't found the time and reason to use helm-docs yet, as I don't use helm much now. But I'll check it out and get back into this


Define some keywords for the primary hdu

I think now, we do not make any definitions for the primary hdu.
Maybe we should also include at least some header keywords here so a file can be immediately recognized as a file produced according to theses standards here.

Sounds like a good idea. Can you make a proposal?

Might consider just using the keywords that OFWG already recommends:
The OFWG recommends that the following set of keywords be used to for such purposes in each extension:

HDUCLASS - a character string giving a general identifier of data format used
HDUDOC - a character string giving the document (preferably published) that describes the format/classification used
HDUVERS - a character string giving the specific version of the document specified by HDUDOC.
HDUCLASn - an indexed set of character strings giving the classification of data in the extension

Current allowed values here: https://heasarc.gsfc.nasa.gov/docs/heasarc/ofwg/docs/ofwg_recomm/hduclas.html
You'd have to invent a HDUCLASS I guess.

These are used for the actual HDUs already.
So I'd suggest HDUCLASS = OGADF_PRIMARY?

These are used for the actual HDUs already.
So I'd suggest HDUCLASS = OGADF_PRIMARY?

I agree with this.

Maybe we should also include at least some header keywords here so a file can be immediately recognized as a file produced according to theses standards here.

Perhaps we could add a list of the different tables within the DL3 file? Perhaps that is too complex?

Perhaps that is too complex?

Not really, I think.
Is there a something standard for this?
If not, something like this:
HDU0 = OGADF_PRIMARY
HDU1 = EVENTS
HDU2 = RESPONSE/EFF_AREA/POINT-LIKE/AEFF_2D
HDU3 = RESPONSE/EDISP/POINT-LIKE/EDISP_2D

This could be optional but recommended in the standard, since you can always get this information thorugh looking at the individual hdus's

Here is the standard
https://heasarc.gsfc.nasa.gov/docs/heasarc/ofwg/docs/ofwg_recomm/hduclas.html
Note sure what the "_PRIMARY" stands for. "OGADF" for HDUCLASS seems sufficient.

Le 23 mars 2020 à 16:46, Maximilian Nöthe<EMAIL_ADDRESS>a écrit :
Perhaps that is too complex?
Not really, I think.
Is there a something standard for this?
If not, something like this:
HDU0 = OGADF_PRIMARY
HDU1 = EVENTS
HDU2 = RESPONSE/EFF_AREA/POINT-LIKE/AEFF_2D
HDU3 = RESPONSE/EDISP/POINT-LIKE/EDISP_2D
This could be optional but recommended in the standard, since you can always get this information thorugh looking at the individual hdus's
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/open-gamma-ray-astro/gamma-astro-data-formats/issues/141#issuecomment-602684939, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAW2QV4B23WSS3BO3OTOB3LRI574ZANCNFSM4LQOA4SQ.

@jknodlseder this is about the first primary hdu, that is not used in this standard for anything right now.
HDUCLASS is recommended to be GADF in the current standard, so the primary hdu (the image  hdu the FITS standard requires  as first hdu) could have:
HDUCLASS = "GADF"
HDUDOC    = "https://..."
HDUVERS   = "v0.3"
HDUCLAS1 = "PRIMARY"

Going through the format used within the CREF# keywords in Fermi-LAT IRFs (also containing arrays), the way they store them is:
Instead of @MaxNoe proposal:

HDU0 = OGADF_PRIMARY
HDU1 = EVENTS
HDU2 = RESPONSE/EFF_AREA/POINT-LIKE/AEFF_2D
HDU3 = RESPONSE/EDISP/POINT-LIKE/EDISP_2D

It would be:
HDU0 = OGADF_PRIMARY
HDU1 = EVENTS
HDU2 = (RESPONSE,EFF_AREA,POINT-LIKE,AEFF_2D)
HDU3 = (RESPONSE,EDISP,POINT-LIKE,EDISP_2D)

Maybe the initial "(" helps identifying that an array is coming? Anyway, if we can be as consistent as possible with other approaches used in the past, better.

That's actually not standard conform, so we should not use it.
Enclosed in quotes as a string, it would be.

Using HDUCLASS for an empty extension does not make any sense. HDUCLASS is meant to identify content.

Le 24 mars 2020 à 16:19, Maximilian Nöthe<EMAIL_ADDRESS>a écrit :
@jknodlseder https://github.com/jknodlseder this is about the first primary hdu, that is not used in this standard for anything right now.
HDUCLASS is recommended to be GADF in the current standard, so the primary hdu (the image hdu the FITS standard requires  as first hdu) could have:
HDUCLASS = "GADF"
HDUDOC    = "https://..."
HDUVERS   = "v0.3"
HDUCLAS1 = "PRIMARY"
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/open-gamma-ray-astro/gamma-astro-data-formats/issues/141#issuecomment-603300982, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAW2QV5FRXZPAMPR64KULN3RJDFRBANCNFSM4LQOA4SQ.

I agree with Jurgen. You don't need an HDUCLASS in the primary.  Couldn't you just identify the file type by the HDUs present?   There's also the hierarchical grouping standard to group a set of HDUs together, but I think it also leave the primary HDU blank.  https://fits.gsfc.nasa.gov/registry/grouping.html


Pretrained weights do not load correctly

The style transfer loads has the following keys odict_keys(['features.0.weight', 'features.0.bias', 'features.2.weight', 'features.2.bias', 'features.5.weight', 'features.5.bias', 'features.7.weight', 'features.7.bias', 'features.10.weight', 'features.10.bias', 'features.12.weight', 'features.12.bias', 'features.14.weight', 'features.14.bias', 'features.16.weight', 'features.16.bias', 'features.19.weight', 'features.19.bias', 'features.21.weight', 'features.21.bias', 'features.23.weight', 'features.23.bias', 'features.25.weight', 'features.25.bias', 'features.28.weight', 'features.28.bias', 'features.30.weight', 'features.30.bias', 'features.32.weight', 'features.32.bias', 'features.34.weight', 'features.34.bias', 'classifier.0.weight', 'classifier.0.bias', 'classifier.3.weight', 'classifier.3.bias', 'classifier.6.weight', 'classifier.6.bias']).
It does not have the model key to load the model. Please recommend the best way to load the weights.

If the error occurred while loading the model for style transfer. I have updated the code to work correctly now.  Please try again with the new changes.
Let me know if you still face problems.

Thank you.
However, this brings this error on line target_feature = model(style_img).detach() inside style_transfer_utils.py
The size of tensor a (4) must match the size of tensor b (3) at non-singleton dimension 1


[1.19.2 | Suggestion ] Make crates render more items the more you fill them

Right now crates seem to cap at 2-3 rendered Items which seems a little empty for some items. Perhaps a progressive "filling" of the crate the more items are in storage would make it more immersive and also act as an indicator to buyers how much they would be able to buy with just one glance.

I think making it look more organized goes well with it being a way for shop keepers to show of their goods, so rows from the bottom to the top would make sense I think
I appreciate the quick answer! Lovely mod otherwise

functionality added, file uploaded. look for 2.9.0 for 1.19.2


Cache results from resque

This should make the app resilient to lots of people hitting it at the
same time.  This also paves the way to have an offline task insert into
the cache to avoid web request timeouts

@joecannatti if you merge your PR, I'll want to update this to not cache nil
@jlambert121 if I modify fixops-service's database for this app to indicate it requires a memcache, will it just get provisioned automagically?

Whoops, never came back to this, I have code on my machine that adds an error field to the returned struct. Was going to go that route. Will still require some change because I didn't even notice there was caching involved.

I don't have the energy to deal with Travis.  Circle is the CI of record for this for now, and we need these changes merged.


Fix GroverOptimizer's rotation_count

Summary
Fixes #129
GroverOptimizer's rotation_count is set in a wrong way.
It is originally rotation_count = int(np.ceil(algorithm_globals.random.uniform(0, m - 1))), but it is always 1 if m=1.
It should be either 0 or 1. So, I replace it with algorithm_globals.random.integers(0, m).
The order of result.get_counts(qc) has an issue. It is sorted by probability, but some bitstrings may have the same probabilities and it results in a probabilistic behavior of unit tests on different environment (mac and linux).
This PR sorts the counts by bitstring order if qasm_simulator is used. So, the resulting iterator has the same order in any environment.
If statevector_simulator is used, it omits the sort because the results are generated in a sorted order.
Details and comments

LGTM! The code rotation_count = int(np.ceil(algorithm_globals.random.uniform(0, m - 1))) was not working as expected. We needed random integers including 0. However the code did not return 0 since it has ceil.

A question is still left. A function is_good_state in grover_optimizer is used for making Grover object. But, I found that is_good_state is never used. Even if I put assert False in it, the unit tests pass. This is the reason why is_good_state has a wrong signature, which I fixed in this PR, but it did not raise an error in the past.
As a conclusion, we need overhaul GroverOptimizer.

A question is still left. A function is_good_state in grover_optimizer is used for making Grover object. But, I found that is_good_state is never used. Even if I put assert False in it, the unit tests pass. This is the reason why is_good_state has a wrong signature, which I fixed in this PR, but it did not raise an error in the past.

Looking at the coverage output from CI here is_good_state code is still not covered by unit tests - it looks like its still not called at all despite being passed to Grover.

I see. We should add a unit test of is_good_state in the future.

I see. We should add a unit test of is_good_state in the future.

I would have imagined that is_good_state should have been called by Grover during its normal operation. As such I am unsure why coverage shows that its not called - and your comment above indicated its not called since even with parameters mismatching on the signature it should get called, but may go wrong/exception. Does this need further investigation?

Yes, we need further investigation about is_good_state in a separate issue. Since it does not seem to affect the existing unit tests, I think we can merge this PR.


TypeError: Cannot read property 'true' of undefined

I believe    credentials.url = DEFAULT.SECURE_MODES[secure] should be singular SECURE_MODE
getting the following error:
$ node
> var MeaningCloud = require('meaning-cloud');
undefined
> var meaning = MeaningCloud({
...   key: 'xxxxx'
...   });
TypeError: Cannot read property 'true' of undefined
  at module.exports (/Users/duckworth/Repos/hsdnapi/node_modules/meaning-cloud/lib/Meaning.Credentials.coffee:19:42)
  at module.exports (/Users/duckworth/Repos/hsdnapi/node_modules/meaning-cloud/lib/Meaning.coffee:30:19)
  at repl:1:15
  at REPLServer.defaultEval (repl.js:248:27)
  at bound (domain.js:280:14)
  at REPLServer.runBound [as eval] (domain.js:293:12)
  at REPLServer.<anonymous> (repl.js:412:12)
  at emitOne (events.js:82:20)
  at REPLServer.emit (events.js:169:7)
  at REPLServer.Interface._onLine (readline.js:210:10)
  at REPLServer.Interface._line (readline.js:549:8)
  at REPLServer.Interface._ttyWrite (readline.js:826:14)
  at ReadStream.onkeypress (readline.js:105:10)
  at emitTwo (events.js:87:13)
  at ReadStream.emit (events.js:172:7)
  at emitKeys (readline.js:1250:14)
  at [object Generator].next (native)
  at ReadStream.onData (readline.js:918:36)
  at emitOne (events.js:77:13)
  at ReadStream.emit (events.js:169:7)
  at readableAddChunk (_stream_readable.js:146:16)
  at ReadStream.Readable.push (_stream_readable.js:110:10)
  at TTY.onread (net.js:523:20)

Fixed, thanks!


Test Plan
https://github.com/libra/pre-mainnet-genesis/commit/3de84bd899f15cb44f97bf11ff3d1dbd0c64f4ea
https://github.com/libra/pre-mainnet-genesis/commit/51d33888e4b0620c41b560558de963146f6d150c

@bors-libra r=davidiw

:pushpin: Commit d9b9147 has been approved by davidiw

:hourglass: Testing commit d9b9147cd641ba78be61d2c7e32a7210a95e78ea with merge 434e379c99b0f324c54465cc73f217b952edb801...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: davidiw
Pushing 434e379c99b0f324c54465cc73f217b952edb801 to master...


LOC-5087 Fix Strapi not preserving order of items in repeatable components

Task linked: LOC-5087 Fix Strapi not preserving order of items in repeatable components


Error loading plugin "alexanderzobnin-zabbix-datasource" is unsigned

After having updated the plugin grafana-zabbix to version 4.0.1 the following error is logged at Grafana startup:
EROR[10-14|07:30:58] Failed to load plugin                    logger=plugins error="plugin "alexanderzobnin-zabbix-datasource" is unsigned" pluginPath=C:\Users\Public\Dev\grafana-7.2.1\data\plugins\alexanderzobnin-zabbix-app\datasource-zabbix
Grafana 7.2.1 runs on Windows Server. The error comes with version 7.1.5 too (Windows). Please get the plugin signed correctly.

Duplicates #1042


[Bug Report]

Versions:

Minecraft Server Version: git-Purpur-1842 (MC: 1.19.2)*
Plugin Version: 1.11.7

Issue
Holograms. every time you restart the plugin with /crates reload it duplicates them.
Screenshots
https://imgur.com/a/EXLtdi8

What holograms plugin?

Decentholograms

https://ci.crazycrew.us/view/Plugins/job/CrazyCrates/5/


Improve makefile

All comments addressed 😄

Sounds good to me but we might wait until we fix semver about misc and doc branches.


Compatibility with react-native-toast-message

Describe the bug
Hello, I'm aware that native wind does not support each and every library, but since I'm not a native wind pro, I wanted to ask if there might be a workaround for this in particular.
The library in question is the decently popular react-native-toast-message and I would like to use native wind to style the toasts there.
To Reproduce
This is the code I have for the toast config so far:
import { cssInterop } from "nativewind";
import {
	BaseToast as BToast,
	ToastConfig,
} from "react-native-toast-message";
const BaseToast = cssInterop(BToast, {
	className: "style",
	text1ClassName: "text1Style",
	text2ClassName: "text2Style",
	contentContainerClassName: "contentContainerStyle",
});
export const toastConfig: ToastConfig = {
	custom: (props) => (
		<BaseToast
			{...props}
			className="bg-red-400"
		/>
	),
};

Since the toast elements have different style props, I tried using cssInterop to add styling support. Not quite sure what the ideal function is to use here... Weirdly enough, certain props like the font Size on the text1ClassName props seem to work, others, like colors, do not.
Expected behavior
It would be nice if it were possible to style toasts on all properties (like spacing, color, etc.) though nativewind, since it doesn't seem to do so currently.

A workaround that I am currently using (in case someone else is facing a similar issue) is to to use a different tailwind library like twrnc. This works, since it compiles the tailwind classes to style objects at run-time. Having to work with two tailwind packages in conjunction is a little cumbersome, but at least you're not dependent on other packages having a specific implementation.
I might consider opening a pr in the toast message library, but I don't know if it's worth it. Although a little frustrating, since this technically doesn't seem to be a native wind issue, I guess I'll close this issue for now.


[typed] Introduce typed mode ("use types")

@arv, PTAL
I created the typed-mode branch in my fork for now.

Very nice. Very excited to see this starting.

PTAL

LGTM

Do you want me to merge it into google/traceur-compiler/strong-mode or did you want to add more logic to this before landing?

Upstreaming this makes sense, but perhaps we should keep it in a branch separate from strong-mode for now? (like the typed-mode I introduced in my fork.)

I'm fine with that. Maintaining branches have some overhead but we can
merge the strong and typed branches when we have some more logic
implemented.
On Mon, Jun 8, 2015 at 12:59 PM, rossberg-chromium<EMAIL_ADDRESS>
wrote:

Upstreaming this makes sense, but perhaps we should keep it in a branch
separate from strong-mode for now? (like the typed-mode I introduced in my
fork.)
—
Reply to this email directly or view it on GitHub
https://github.com/rossberg-chromium/traceur-soundscript/pull/1#issuecomment-110073288
.

--
erik


not selecting servant

Hello i wantedto test this appon doors node i play on  android 10.
I set up everything and prepared autoskill script.
Then i started the app and it select the node i wanted (40ap door) after that it sit on support screen whitout doing anything at all.
Please note that in some cases when i too deep o  level selction screen the app would tap on "i" instead of the node.
Here are the screen form my app, what am i doing wrong?

Another issue is "img maker not working on select support screen"

Please fill out the bug template:
Describe the bug
A clear and concise description of what the bug is.
Include steps to reproduce the error if required
Screenshots
If applicable, add screenshots to help explain your problem.
If an error occurred, you can find the message by expanding the app's notification.
It is recommended that you atleast add one screenshot of FGO running on your device.
Video
If relevant, include a video of the script running on your device with Debug mode ON (recording functionality is available within the app in More options).
Device Info (please fill atleast 'Device model'):

Device model: [e.g. Xiaomi Redmi 4]
Screen size: [e.g. 1280x720]
Android Version: [e.g. Android 7.1]
RAM: [e.g. 3GB]

What is your phone model?
Please don't ignore the issue template. The questions asked in it matter to us for solving your problems.

Samsung galaxy s10

https://mathewsachin.github.io/Fate-Grand-Automata/SAMSUNG.html

I tried it with both ignore notch on and off before making the post.

And did you activate full screen mode for FGO?

Make sure you’re running F/GO with Full screen mode ON in Samsung’s settings. Blue-bars should be shown in the space which is not used by FGO. If you did this wrong, that area would be black.

Did you try this? Post a screenshot of FGO running on your phone.

Nope how do i do that?

https://www.google.com/search?q=samsung+full+screen+mode&oq=samsung+full+&aqs=chrome.1.69i57j35i39j0l6.3903j0j9&sourceid=chrome&ie=UTF-8

Now the game has 2 blue bands o  the side isnit oklike that? It seems to work now

That's what you want, yes.

Please fill the issue template and follow the instructions the next time you open an issue. Thanks.

ok thx for the fast answer btw with ignore nocht doesnt work you ha e to disable it now. Btw thx
One last question is there ban risk by using this?

ignore nocht doesnt work you ha e to disable it now
Yeah, we have an issue stating this pinned and the website link I gave you also says so.

is there ban risk by using this
Unless you tell DW yourself, you shouldn't get banned.

Ty very much for the infos man.

One more question what about the repeat screen? Will the app auto repeat or go to the quest selection screen?

It will repeat the quest

K ty very much

Thies the app have an autoupdate function if not how do j update it when new versions comes out?

See #135 and you can also click on Watch and then select Releases only. That way you'll get an email every time there's a release.


socket path change due to qemu limitation : Path must be less than 10…

Hi, i've fix a socket path issue ( due to qemu  >104 chars limitation on socket)
https://unix.stackexchange.com/questions/367008/why-is-socket-path-length-limited-to-a-hundred-chars
I've moved socket to <user_home>/.vagrant.d/tmp.
Please, let me know if you have a dedicated process for PR

Or add a env parameter to the Driver's start/initialize function?

Fix in v0.1.7


Fix relative location of the start script

The compilation of a very simple example with a start
script in the main folder was failing (see #399)
because the start script was not found.
Most probably, the reason is that the folder in which
the script resides is not in the PATH. To solve this,
we replace the start string with a relative ./start,
which solves the problem and completes the build and the
run.
This fixes #399

I think travis is failing on this because it is flaky or there is an issue with the infrastructure that the julia build uses. Let's see if restarting it helps.

@giovannipizzi this has been deployed to mybinder.org. Let us know if your repo now works.

Thanks! We (with @ltalirz @yakutovicha) will report back!

@betatim I confirm it works - thanks for the quick fix!
One issue I stumbled over is this: if you make a mistake in the start script (as I did, of course ;-) ), the image will build fine, and when you start the container, the log on mybinder.org just says something like "failed to start".
I debugged it locally using repo2docker but if more people start using this feature, it might make sense to show a bit more info here (I can open an issue if you like)

An issue would be great. The place for it would be https://github.com/jupyterhub/binderhub I think as that takes care of showing (or in this case not showing) logs.

Done! https://github.com/jupyterhub/binderhub/issues/667


Cant login in google calendar

Hi,
I use nativefier 47.1.3 to use my calendar in google.
nativefier -p linux -a x64 --disable-context-menu -n google-calendar https://calendar.google.com/
Everything is also going well and nativefier finishes without reporting any errors.
But when I try to login to google I get an error:
https://postimg.cc/qt7y9qsd
worked fine with previous version.
What am I doing wrong?
5.15.0-33-generic x86_64 * Ubuntu 22.04 * KDE 5.24.4

See https://github.com/nativefier/nativefier/blob/master/CATALOG.md#google-apps


Array is not assignable to array of string literal type in overwritten class method

Bug Report

🔎 Search Terms

array of string literals
array specific string values
TS2416

🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about string literals

⏯ Playground Link

Playground link with relevant code
Codesandbox mirror
💻 Code

type HTTPMethods = "get" | "post";

abstract class API {
  protected abstract allowedMethods2: HTTPMethods[];
}

export class Resource extends API {
  protected allowedMethods2 = ["get", "post"];
}

🙁 Actual behavior

The following error occurs:
Property 'allowedMethods' in type 'Resource' is not assignable to the same property in base type 'API'. Type 'string[]' is not assignable to type 'HTTPMethods[]'. Type 'string' is not assignable to type 'HTTPMethods'.

However, when we do the following:
let allowedMethods3: HTTPMethods[];
allowedMethods3 = ["get", "post"];

Everything works as expected.
🙂 Expected behavior

I would expect TypeScript to not throw an error when implementing an abstract class method. The same issue also occurs in non-abstract classes when overriding initial methods. For replication of the latter, have a look into the CodeSandbox example.
Workaround
For workarounds, const enum could be used.

Duplicate of or strongly related to #3667

As another workaround, if the base declaration was a readonly array, in the subclass you could do protected allowedMethods2 = ["get", "post"] as const;.


Added an explicit System.Security assembly loading

This fixes "Unable to find type" [System.Security.Cryptography.ProtectedData] error.

Hi there, thanks for PR! Have you seen #2?
Do you think that I should add a warning in readme about directly importing psm1 file? Because you're not the first person facing this issue and it's clearly that something has to be done about this.

I've added assembly import and a warning if module was imported without manifest in e1fdbc2. Hope this will be enough. Thanks!


🌐 Add German translation for docs/de/docs/advanced/response-cookies.md

← advanced/additional-responses.md (#10626)
→ advanced/response-headers.md (I am currently translating)
German translation progress

📝 Docs preview for commit 8178014e9519bfec81f31d57894167b1f2dde10f at: https://1f2c2cad.fastapitiangolo.pages.dev

📝 Docs preview for commit d2074e10000b821916d2cbe7d967d947beaa8b2f at: https://6c58125f.fastapitiangolo.pages.dev

📝 Docs preview for commit 35dc0f6762b116358840e5b7f8de707574d0ffd3 at: https://bafe764c.fastapitiangolo.pages.dev

📝 Docs preview for commit c55bd9bdfdaa70f221853dac2aa5e55f9bf830af at: https://cec4a3a3.fastapitiangolo.pages.dev

📝 Docs preview for commit 0ea12fe2d927898e186a85e12a64da2134c6e778 at: https://fa178de1.fastapitiangolo.pages.dev

Awesome @nilslindemann! :rocket:
As you are the champion of German translations and you have done so many, and with so much attention to detail, I'm gonna go ahead and merge this. :tada:
If others see things that could be improved later, we can check that in new PRs. :nerd_face:


[BUG] [2.0] Cannot save database If adding special characters in the database's filepath.

Steps to reproduce:

Create a new database
Choose "File menu -> Save DB"
Add any of the following characters in the file name: "`~!@#$%^&*()=+[]{}\|;:'",<.>/?"
Observe that the database will not be saved, but the above characters will be set in the database's filepath.

TODO:

Filter the database's filepath in the same way when choosing the database's name
Show the error dialog popup when necessary

This should be fixed with the commit: e1016356985131c980aea4d1cdcfb635ec9c41cd