Element names containing spaces and functional characters can not be reached

Again questions from a newbie user:
when I try to reach an element in a json message containing an functional character e.g. '-' in 'E-Total' with bracket notation:
{
 "fromTopics": ["solar/inverter/2000144729","solar/inverter/2000144729 "],
 "toTopic": "openWB/set/evu/WhExported",
 "emitType": "combineLatest",
 "template": {"$eval": "(messages[0].values['E-Total']+messages[1].values['E-Total'])*-0.9906542-80*1000"}
   },

in following object solar/inverter/2000144729

 {"sn":2000144729,"time":1667563080,"values":{"Upv-Ist":381,"Upv-Soll":382,"Iac-Ist":392,"Uac":230,"Fac":49.989998882636428,"Pac":90,"Zac":0.44800002127885818,"Riso":3000,"Ipv":307,"Aktives Team":1,"E-Total":41159.525954972487,"h-Total":68438.1999388286,"h-On":70897.2655337967,"Netz-Ein":24466,"Seriennummer":2000144729,"Status":"Mpp","Teamfunktion":0,"Fehler":"-------"}}}

the result is null
When I try $eval: values['E-Total'] in jsone-e playground it works:

When I use this bracket notation for normal element names e.g. 'Seriennummer' which is one of the last elements:

works fine.
-When I use .values['E-Total'] in a map e.g.
  {
     "fromTopic": "solar/inverter/1000123",
      "toTopic": "openWB/set/evu/Status",
      "emitType": "map",
      "wrapper":"test",
       "template": {"$eval": "test.values['E-Total']"}
    },

the result is null

So is there a special notation to be used or is that an feature which can be solved?
And a second question I have: is it possible to have LF CR in result message e.g.
I have created:
obislog "/ 1-0:1.8.1*255(96435.65119118353) 1-0:2.8.1*255(111062.242362) 1-0:21.7.0*255(148*W)!"

in my resultfile with using  mosquitto_sub -t "obislog" -h localhost > file.txt every 10 secs in a ramdisk via crontab:
"/ 1-0:1.8.1*255(96435.65119118353) 1-0:2.8.1*255(111062.242362) 1-0:21.7.0*255(148*W)!"

I would need is:
"/<crlf>
1-0:1.8.1*255(96435.65119118353)<crlf>
1-0:2.8.1*255(111062.242362)<crlf>
1-0:21.7.0*255(148*W)<crlf>
!<crlf>
"

Is that possible with mqtt-transformer ?
Thanks in advance
Christian

hmmm interesting problems...
As I started to dig into the possible problems, I found out that I move all the --s lower to _-s in every input object key name. I can't really tell why we doing this, but at some point there was a reason. Can you confirm that the map below works?
{
     "fromTopic": "solar/inverter/1000123",
      "toTopic": "openWB/set/evu/Status",
      "emitType": "map",
      "wrapper":"test",
       "template": {"$eval": "test.values['E_Total']"}
    },

As the second part I have honestly no idea. In theory json-e can handle \n and \r. In theory it should work. I would not be really surprised if end to end it would broke at some point, but;
const jsone = require('json-e');
const inStr2 = {"values":{"E-Total": "--\n\n--"}}
const template2 = {"$eval": "values['E-Total']"}
console.log(jsone(template2, inStr2))

This code prints 1 empty line as it supposed to. I think if you add \r\n to the json and do the concatenations, it will probably work...

Okay, part one is solved: underscore '_' for minus '-' works fine. Also space   works in object key name.
So I have connected my devices yasdi2mqtt via mqtt-transforme to openWB with the available high resolution with all necessary corrections. That was the last peace of this puzzle.
Regarding \r\n I have to do some additional research....
Meanwhile for my problem, a file containing OBIS formated text tob bring D0 data to Solarview,
 mosquitto_sub -t "obislog" -h localhost >> /var/www/html/openWB/ramdisk/obis/obis.txt and
sed -i 's/\s\+/\n/g' /var/www/html/openWB/ramdisk/obis/obis.txt hopefully help - seems to work
Thanks a lot for the support.

I have som ideas, what would be nice to have:

global constants e.g. for correction in my case for the energy counters: CorrAdd and CorrFac so corrections might be done on one place in conf - file and considered in many topics

global variables to use an input in several topics. (at the moment variable are defined for every topic - could be the same vriablename for all . right?)

Initialisation of topics with combineLatest: In my case the converters are enot sending topics when the sun is down.  combineLatest waits for at least one of each input topic before calculation. At the moment at systemstart I send a mosquitto_pub with initialisation values (here zeros). So it would be nice to be able to define a imitialsation for global variables and topics whitch are used in combineLatest topics.

-Absotutely great would be a time based interpolation: e.g. a energycounter topic is provided (CtrEgy [kWh] resolution 1 kWh) and a power topic (Pac [W]) and needed is the energy in a higher resolution (EgyHres [kWh]  resolution 0.0001 kWh).
CtrEgy, Pac are read and the time bewreen the messages time [s]
the constant Factor can be define in the topic or obove in init values
If CtrEgy == CtrEgy n-1
then
EgyHres == EgyHres n-1 + Pac*time*Factor            // Factor =  (1/(3600*1000)) // [Ws ->kWh]
else
EgyHres ==  CtrEgy

What do you think?

I'm not sure I understand some of these...
global constants
Something like this?
        {
            "constant": "secToHour",
            "value": 3600,
        },
        {
            "fromTopic": "tele/sp-desk/STATE",
            "toTopic": "transformed/sp-desk-state",
            "emitType": "map",
            "template": {"uptimeHrs": {"$eval": "UptimeSec / constants.secToHour"}}
        },

Probably this is easy to implement, but than the constants will be a reserved toplevel key. Another way would be to add the useConstants as part of the definitions like;
        {
            "constant": "secToHour",
            "value": 3600,
        },
        {
            "fromTopic": "tele/sp-desk/STATE",
            "toTopic": "transformed/sp-desk-state",
            "useConstants": {"renamed_secToHour": "secToHour"}
            "emitType": "map",
            "template": {"uptimeHrs": {"$eval": "UptimeSec / renamed_secToHour"}}
        },

I think the second is versatile enough to cover a lot of usecases, also not really hard to implement.
global variables
I think we can do this, with the current zip/combineLatest implementations, or I don't get the usecase.
default values for combineLatest
      {
            "fromTopics": ["tele/sp-desk/STATE", "tele/wemos-dev/STATE"],
            "toTopic": "transformed/combineLatest",
            "emitType": "combineLatest",
           "defaultValues": [null, {UptimeSec: 0}]
            "template": {"uptimeDesk": "${messages[0].UptimeSec}", "uptimeWemos": "${messages[1].UptimeSec}", "sum": {"$eval": "messages[0].UptimeSec + messages[1].UptimeSec"}}
        },

and in this case, if the first topic gets a message, it will run (like the second would had the given message already), but if the second topic would get messages it would do nothing (like now).
I think this is doable and easy.
time based interpolation
        {
            "fromTopic": "tele/sp-desk/STATE",
            "toTopic": "transformed/sp-desk-state",
            "useConstants": {"renamed_secToHour": "secToHour"}
            "useMetrics": {"ms": "sincePrevMsg", "avgTime": "avgTimeBetweenMessages", "sumTime": "sumObservedTime", "msgCount": "msgCount"}
            "emitType": "map",
            "template": {"estimatedMessagePerHour": {"$eval": "renamed_secToHour * 1000 / avgTime"}}
        },

If I would start sth like this, I would add multiple metrics, mostly the ones described above. This is what you thinked about?
I think this is also not really hard to add.
implement and reimplement
If I would add these features, I would surely rewrite the whole thing to TS. Which would add some more time to the implementation time, but I think the all things I described here could be done in an afternoon.

global constants
on the first sight I thougt, your first proposal would be sufficient. - But the longer I thinkl about it your second proposal is much better:
        {
            "constant": "secToHour",
            "value": 3600,
        },
        {
            "fromTopic": "tele/sp-desk/STATE",
            "toTopic": "transformed/sp-desk-state",
            "useConstants": {"renamed_secToHour": "secToHour"}
            "emitType": "map",
            "template": {"uptimeHrs": {"$eval": "UptimeSec / renamed_secToHour"}}
        },

global variables
e.g. at the following "wrapper":"energy_pv_sum" the name ist only used inside the topic, right?
    {
    "fromTopic": "hres",
      "toTopic": "calc/out/energy/pv_sum",
      "emitType": "map",
      "wrapper":"energy_pv_sum",
       "template": {"$eval": "energy_pv_sum/3600"}
    },

or is it possible to use "energy_pv_sum" in an other topic?
default values for combineLatest - looks fine !
time based interpolation
yes , might be good
Just as ispiration:
My workaroud at the moment ist to collect with mosqutto_sub topis in ram disk files every 5min @ crontab:
hist=$( tail -n 1 /var/www/html/openWB/ramdisk/obis/KT0.txt ) && echo $hist > /var/www/html/openWB/ramdisk/obis/KT0.txt && mosquitto_sub -t solarview/WR0/KT0_corr -W 299 >> /var/www/html/openWB/ramdisk/obis/KT0.txt
hist=$( tail -n 1 /var/www/html/openWB/ramdisk/obis/PAC.txt ) && echo $hist > /var/www/html/openWB/ramdisk/obis/PAC.txt && mosquitto_sub -t solarview/WR0/PAC -W 299 >> /var/www/html/openWB/ramdisk/obis/PAC.txt
#

and do the interpolation in a sript running in the background.
after the second increment at every increment of the low resolution counter a daption is done.
#!/bin/sh

#All calculation here has to be done with integer variables

#initialisation
fac_phys=3600000 #1 kWh = 1000 Wh * 3600Ws
add_round=$((-fac_phys/2)) #dependend to cut method of input counter: if rounded : (- 1*resolution / 2) (here -0.5 kWh = 180000W)s // if input counter is cut ($

KT0old=$(tail -n 1 /var/www/html/openWB/ramdisk/obis/KT0.txt) #input from file which collects mosquitto_sub (resolution 1  [kWh])
KT0old=$(((KT0old) *fac_phys + add_round)) # -1800000 Ws = 0.5 kWh due to input counter is rounded 0.5...1.5 = 1 kWh
hres=$((KT0old))
ctr=0       #counter delay for mqtt topc (
corr=10000   #correction factor [1/10000 = 0.01%] e.g. +5% -> 10500
sync=false #trigger for adaption @synchronisation
start=true #First sync shall not be considered in adaption / adaption shall be done from first  complete cycle on
diff=0 #difference Ws

#calculatiom in 1 s recurrance /therefore script runs in background nohup <filemame> &
while true ;do
PAC=$(tail -n 1 /var/www/html/openWB/ramdisk/obis/PAC.txt) #input from file which collects mosquitto_sub (integer resolution 1 [W])
KT0=$(tail -n 1 /var/www/html/openWB/ramdisk/obis/KT0.txt) #input from file which collects mosquitto_sub (integer resolution 1 [kWh])
if [ $KT0 -ne 0 ]; then KT0=$((KT0 * fac_phys + add_round)) # -1800000 Ws = 0.5 kWh due to input counter is rounded 0.5...1.5 = 1 kWh
fi
# snchonisation request
if [ $KT0 -ge $((KT0old + fac_phys)) ];then diff=$((hres - KT0)) &&  hres=$((KT0)) && startold=$start && start=false  && sync=true
#counter works
else hres=$((hres + PAC * corr / 10000)) #PAC [W]* 1 s -> [Ws)
fi

#output is limited  to (KT0old + max-1hex)until KT0 is incremented
if [ $hres -le $((KT0old + fac_phys*999/1000)) ];then hresout=$hres
else hresout=$((KT0old + fac_phys*999/1000))
fi

#calculation of adaption
if [ $sync = true ] && [ $start = false ] && [startold = false ];then diff=$(((diff + fac_phys) * 10000 / corr - fac_phys)) && corr=$(((fac_phys -  diff ) * 10$
else sync=false
fi

# mqtt output every 5s
if [ $ctr -ge 5 ]; then mosquitto_pub -t hres -m $hresout && mosquitto_pub -t calc/hres/corr -m $corr && mosquitto_pub -t calc/hres/diff -m $diff && ctr=0
fi

if [ $KT0 -ne 0 ];then KT0old=$KT0
fi
ctr=$((ctr +1))
sleep 1
done
#

#19 Added "global constants" and "default values for combineLatest". Could broke other stuffs, bcs I moved all the codebase to TS. I added some tests, so the bare minimum is covered (about 60%, but I don't believe in coverage numbers).
"global variables" most of the times could be done if you output them to other/random topics, and parse them back with combineLatest
time based interpolation, or metrics or whatever it will be, coming in the next updates, I also want to add output types/web-hooks

I would like to test "global constants" and "default values for combineLatest" - when I use sha-404a9ef the old config does not work longer.
Can you provide the expected syntax for define constants , use constant and default values. - I guess the line "useConstants": {"renamed_secToHour": "secToHour"} has to be set always ibn combineLatest.
Regarding global variables I agree absolutely. - Can be done with combineLatest
Regarding the time based thing I think I could be done after abstraction in a few lines code.
only the sum of time*Input and a possible reset of the counter @ triggervalue n <> triggervalue n-1

The calculation of adaption is not necessary. Factors for physical correction can be multiplied in the output line.
Also the break of sum > trigger + 1 increment - resolution (e.g. 1.999 as long trigger = 1 if the sum is too fast)

My solution above works fine but is unnecessarily complicated regarding the adaption of the correction factor.
Due to using bash script the calculation is done in integer I used [Ws] what leads to big numbers to keep the accuracy.
Saving the mqtt topics may not be the best way. But the best I was able to do.

when I use sha-404a9ef the old config does not work longer

I have a test here which was intended to test backward compatibility, and also the parser should telling you which part of the config is not right. Can you provide more information? What happens when you try to start the latest on your prev working config?

Can you provide the expected syntax for define constants , use constant and default values.

We have a test for that too so the syntax is;
[
            {
                emitType: "constant",
                name: "a",
                value: 1,
            },
            {
                emitType: "constant",
                name: "b",
                value: 4,
            },
        {
            emitType: "map",
            fromTopics: ["t"],
            toTopicTemplate: "out",
            template: {"message": "I ${k.i}, ${f}, ${g}"},
            useConstants: {"f": "a", "g": "b"}
        }
]

@SMCinc Did you have any time to test the modifications?

Hi, yes I did.
witth

{"transforms":[
   {
   "emitType": "constant",
   "name": "FAC_PV",
   "value": "0.99065657"
  },
   {
 "fromTopics": ["shellies/shellyem3/emeter/0/power","shellies/shellyem3/emeter/1/power","shellies/shellyem3/emeter/2/power","solarview/WR0/PAC"],
 "toTopic": "calc/out/power/sum",
 "emitType": "zipLast",
 "template": {"$eval": "floor(0.5+(messages[0]+messages[1]+messages[2])*1.02103678113188-messages[3]*${renamed})"},
 "useConstants": {"renamed": "FAC_PV"}
  },
.....

I get the following message:
pi@raspberrypi:/otp/mqtt-transformer $ sudo docker-compose up
Creating network "mqtt-transformer_default" with the default driver
Creating mqtt-transformer_mqtt-transformer_1 ... done
Attaching to mqtt-transformer_mqtt-transformer_1
mqtt-transformer_1  | App started
mqtt-transformer_1  | MQTT(0): Connected to mqtt://broker:1883
mqtt-transformer_1  | TypeError: Cannot create property 'renamed' on number '0'
mqtt-transformer_1  |     at /home/node/app/build/handleMessage.js:41:37
mqtt-transformer_1  |     at Array.forEach (<anonymous>)
mqtt-transformer_1  |     at /home/node/app/build/handleMessage.js:38:44
mqtt-transformer_1  |     at Array.forEach (<anonymous>)
mqtt-transformer_1  |     at handleMessage (/home/node/app/build/handleMessage.js:30:13)
mqtt-transformer_1  |     at globalHandler (/home/node/app/build/app.js:45:39)
mqtt-transformer_1  |     at MqttClient.<anonymous> (/home/node/app/build/io/mqtt.js:37:9)
mqtt-transformer_1  |     at MqttClient.emit (node:events:390:28)
mqtt-transformer_1  |     at MqttClient._handlePublish (/home/node/app/node_modules/mqtt/lib/client.js:1547:12)
mqtt-transformer_1  |     at MqttClient._handlePacket (/home/node/app/node_modules/mqtt/lib/client.js:535:12)
mqtt-transformer_mqtt-transformer_1 exited with code 0

What is wrong here?

I see, this is not so trivial, and this will be hacky as hell. Your messages are constants and not json objects. So the "correct" way is;
{
  "fromTopics": ["shellies/shellyem3/emeter/0/power","shellies/shellyem3/emeter/1/power","shellies/shellyem3/emeter/2/power","solarview/WR0/PAC"],
  "wrapper": "value"
   "toTopic": "calc/out/power/sum",
   "emitType": "zipLast",
   "template": {"$eval": "floor(0.5+(messages[0].value+messages[1].value+messages[2].value)*1.02103678113188-messages[3].value*messages[3].renamed)"},
   "useConstants": {"renamed": "FAC_PV"}
  }

I will try to come up with a more natural feel to this, but until then this should probably work.

With the latest version (that I just merged), your original rule should work too!


Add formatter to qc report

This PR adds a formatter to the qc report for a better appearance and some linkouts to cov-lineages and covid genes at ensembl. There are also some adjustments made in generate_virologist_output.py. I also updated rust-bio-tools to the latest version. Should be updated to 0.20.3 when it is released after https://github.com/rust-bio/rust-bio-tools/pull/142 is merged to have the table from the csv-report at an optimal size .

@johanneskoester I think everything is working now. The CI artifacts from the qc_report look good to me. The formatting is still now working because rbt needs a new release containing https://github.com/rust-bio/rust-bio-tools/pull/142.


feat(facetquery): maxFacetHits

Coverage decreased (-0.01%) to 30.341% when pulling 9dd10e5b3b52cbfd4ef3b1952b4f98f2ba675adb on feat/maxFacetHits into e395fc540160b4d246db97679bdb94be10a67080 on master.


fix: Don't generate 'service' resource when no ports are exposed

See internal ticket APPX-64

Thanks @mhutter !


[stable/redmine] Lint chart + standardisation

Signed-off-by: juan131<EMAIL_ADDRESS>Is this a new chart
No
Description of the change
Change chart YAML(s) to make it follow our styling standards
Checklist

[x] Chart version bumped in Chart.yaml according to semver.
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/hold

/assign @sameersbn

/retest

/hold cancel

/ok-to-test
/lgtm


support creation of deb package

@pakmarkthub please see @e-ago seminal work in #32

Merged the PR.
Closing this issue.


Sidebar styling

Plus some minor copyedits, auto-whitespace-removal, and an upated Gemfile.lock

Thanks a lot, @Scotchester !
:racehorse: :bicyclist: :candy:


3.0 a good time for a move?

I'm sure there have been discussions about this I'm missed, or have even happened offline.
Probably no better time than 3.0 to move the repo, to github.com/pixijs/pixi, and change npm package name to 'pixi', (I can give you the name). It would be a clean switch, major version, breaking changes, etc. etc.

/agree
@GoodBoyDigital Was waiting to hear back from someone at GitHub to ensure we stayed on some "top something something" lists after the move happened.

It took me approx 10 minutes to migrate from v2 to v3.
And this was mostly warnings about Container refactor and Text's settext.
Good work guys!

To be clear @drkibitz is talking about moving the repository to the pixijs organization and out of GoodBoy's personal project space.

I see, Well then!

@englercj
I added you as a package owner to:
https://www.npmjs.com/package/pixi
and deprecated the project:
https://github.com/drkibitz/node-pixi

Here's some of my thoughts...

After assurance from Github that everything will be ok, move "GoodBoyDigital/pixi.js" to "pixijs/pixi"

Github will take care of all redirects etc. from the old location at GoodBoyDigital/pixi.js, as long as you move the repo. Do not fork it, or create a new one and push to it.
Moving the repo and relying on Github redirects means that GoodBoyDigital/pixi.js can no longer exist, otherwise it would conflict with and prevent the Github managed redirections to pixijs/pixi.

Publish package to https://www.npmjs.com/package/pixii

This package can be started out at 3.0
The pixi.js package can remain v2, but can still have a v3 of which would only contain a README to point to new package. If you want to be really nice, it can even depend on the new package, export it, and throw a warning to say the same thing as the README.

Make some major pushes to all community outlets possible, about the move along with the new version, and the reasons behind the move and the architecture.

My reason behind the move would be that it is part of putting our best foot forward while moving toward a modular and pluggable architecture. Having everything in a centralized Github organization (namespace) helps mitigate confusion, and simplifies source access management. Naming the library pixi is just a simplification of the current name, saves a few keystrokes, and matches the name of the global object that the library exposes. The organization is already called "pixijs", the core will probably be called "pixi-core", so "pixi" just makes sense for a package that glues things together for you. The reason for this architecture is something that we've already had long discussions over that I'm not going to summarize here.

@GoodBoyDigital To help, thought I'd reference a project that recently changed it's name and org:
https://github.com/nwjs/nw.js
As the README says, it was previously known as node-webkit, and was under the author's (Roger Wang) user organization. In that location, the project grew exponentially, and now under the new organization and name, it continues to grow.
Pinging @rogerwang, the author of NW.js (aka node-webkit), to see if he would like to share his experience in making a major move like this, why, and how things went from the perspective of a project steward.

Here is the Github redirect for this project in action ;)
https://github.com/rogerwang/node-webkit

@drkibitz To be clear, the thing stopping us from moving right now is not the redirect stuff but that the pixi repo is listed on some "top X" lists on github based on traffic and what not. Those stats do not move with the repo and will be lost. We are working with GitHub to get those back in place if we move.

Going to close this since we should be moving as soon as we clear up some details with GitHub, thanks all!

Yes, 6to5 recently changed to babel too, I think without too much undue stress


bend_straight_bend cell defective when direction is 'cw' (missing test?)

Describe the bug
The bend_straight_bend cell does not work when called with 'cw' direction.
To Reproduce
from gdsfactory.components import bend_straight_bend

c = bend_straight_bend(direction="cw")
c.show()

Screenshot:

Expected behavior
What would you like to happen?
Like this, but mirrored:

Suggested fix
The code for the bend_straight_bend makes sense to me, so this must be an underlying bug either in the mirror or the connect methods.
    c = Component()
    b = bend_euler(
        angle=angle,
        p=p,
        with_arc_floorplan=with_arc_floorplan,
        npoints=npoints,
        direction=direction,
        cross_section=cross_section,
    )
    b1 = c.add_ref(b)
    b2 = c.add_ref(b)
    s = c << straight(length=straight_length, cross_section=cross_section)
    s.connect("o1", b1.ports["o2"])
    b2.mirror()
    b2.connect("o1", s.ports["o2"])
    c.add_port("o1", port=b1.ports["o1"])
    c.add_port("o2", port=b2.ports["o2"])
    return c

fixed in #2390
this seems to be a very old cell. You could also look into routing functions


Highlighting of uppercase (constant) variable name.

I hope I'm in the right location for this report.

I'm working on a syntax theme.
As an uppercase variable declaration gets .variable.other.constant, I expect an uppercase variable used to call a function on to also get .variable.other.constant, but actually I get .variable.other.class in the above example.
I understand the first uppercase letter and the function call on it marks it as a class, and it would be (kind of) right to do so. But given the all uppercase would it be an option to mark it as a constant as well?

Yes I agree. I'll take a look at this, probably at the end of the week. Thanks.

Fixed in 2.34.0


DRUPAL8_REVERSE_PROXY_ADDRESSES not working?

added docker environments: DRUPAL8_REVERSE_PROXY_ADDRESSES:<IP_ADDRESS>/0
The container has this env defined.
But $_SERVER[‘DRUPAL8_REVERSE_PROXY_ADDRESSES’] in wodby.settings.php gets null.

Solved in #69


Getting "you are not logged in" error in facebook .

This lib have a dependencies of inAppBrowser Plugin, you can't test on a browser, just on emulators or devices, see the README description of the project.

Is there an issue?  Original poster gave us no details.

#matheusrocha89
I am using it on my android device.
#nraboy
When I am logging in fb , I am getting this error "you are not logged in"
N my page just stop there

Still not really any information to go off of.  We can't help you without all the details.  Are you getting through the entire flow?  Do you get that message somewhere during the flow?  Does this error show up in the logs or the UI? Etc.
My guess, without the details, is that you're getting this error when trying to use the Facebook API, not oauth sign in.
Regards,

Yes I am getting this error during the flow .
When I press the log in button then a new error page opens up and saying ' you are not logged in'.

I have checked my fb api and set my callback url as " http://localhost/callback".
Please guide.

This page happens before prompting you with an opportunity to enter credentials?

Yes
It does.

Sounds like you haven't configured something correctly in the Facebook developer dashboard.  I encourage you to read through their documentation.

You get this error because you didn't set the oauth callback correctly.  I had this problem the other day.

Hi
Can someone help me ? :)
I have a similar problem, like  AtulaGuleria  had. When I pressed login btn, I got error "Not logged in: You are not logged in. Please login and try again"

I am trying to test it with Phonegap Developer App on iOS device (also tried with android (the same result)).
I have investigated this issue, and looks like it related to FB app configuration, specifically "Valid OAuth redirects url field" in Client OAuth settings? But I am not sure what should be there if I am using mobile APP(not web version).

Also I have added iOS platform to FB APP, I got a bundle ID in config.xml file. Is it correct? Or how I have to configure iOS platform? What I missed?

And here is a code example in Angular which I was using:

PS: I have tried the same with vkontakte, and it works for me. THANKS!
Please ask if I was missing something? I will provide any extra information.


Is this project maintained?

The last merge was over a year ago and there are 21+ PRs waiting, some which address critical issues. This project kind of looks abandoned, but there are lots of new issues and PRs out there, so there's community interest.
If @orzih or another maintainer can let us know if this project is a going concern, that would be amazing. The project has a ton of potential and I think lots of us are eager to leverage it. I'd be happy to contribute as much as I can, but I'm not able to maintain my own fork right now.
Thank you for the work you've done so far!

Hi, I'm also interested in the answer to #142

+1

I've also wandered by looking for a reasonable PDF export plugin and am wary of a package that hasn't had a release in 2 years. FWIW, the code is licensed under the permissive MIT licence, so anyone can fork it. mkdocs-with-pdf is presumably now occupied on PyPI, so one might want a new name, but corralling a few PR contributions to point to a new project (perhaps under an organisation, rather than individual) would breathe new life into this project.


Change feature type when multivariates are introduced / removed

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?

Changes
When a feature changes to either having or not having multivariate_options, we need to update the type to either MULTIVARIATE or STANDARD. This will now introduce a put to the feature when we detect this change.
How did you test this code?
Added first option and saved, removed last option and saved.

Closed as we are doing on the BE now


Considering pointing to canonical MIT license to allow license validation check to pass

We have a Gradle plugin that validates the licenses used in our dependency graph: https://github.com/cashapp/licensee
And using this SDK triggers the following failure:
> Task :app:licenseeInternalDebug FAILED
com.getpinwheel:pinwheel-android:2.3.5
 - ERROR: Unknown license URL 'https://github.com/underdog-tech/pinwheel-android-sdk/blob/main/LICENSE' is NOT allowed

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:licenseeInternalDebug'.
> Artifacts failed validation. See output above.

According to these related issues,
https://github.com/airbnb/lottie-android/issues/527
https://github.com/square/kotlinpoet/issues/1077
it appears to be because the POM points to a custom license file/URL: https://search.maven.org/artifact/com.getpinwheel/pinwheel-android/2.3.5/aar
Here's an example of a POM that uses the suggested template: https://search.maven.org/artifact/com.squareup/kotlinpoet/1.8.0/jar
More info here:
https://maven.apache.org/pom.html#Licenses
https://spdx.org/licenses/MIT.html

@jrodbx release 2.3.6 has the updated license fields. Thanks for calling it out, and please let me know if anything else comes up.


Added supported repositories

Cool, thanks :)

Oops! I think I have broken the markdown. Give me few minutes to fix it

Already fixed :)

Thanks!


feat: move faucet to trpc server

Description
Move the faucet logic to trpc server instead of on the page.
Blockers
Not tested yet!

@ctrlc03 since this would change your PR, so it's better be merged after you review it

@kittybest let's close this as we don't plan to use the faucet?


Support previous visual studio versions

Add support to locate legacy versions of visual studio
vswhere has support for this via -legacy

Older installs do not support anywhere close to the amount of information these cmdlets are meant to expose. See issue #12. If you want to find older versions, use https://github.com/Microosft/vswhere. These cmdlets are for Visual Studio 2017 and newer because that what the query API on which they are built are bound to.


[feature] Generate ormbfile.yaml for users

Is this a BUG REPORT or FEATURE REQUEST?:
/kind feature
What happened:
For these users using Python SDK, they may not want to write such a configuration file. We may need to:

Make it optional or
Support some parameters to generate it or
Analyze the parameters from the model automatically.

What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:

@simon-cj Did we generate it in model registry?

If upload model by dashboard, we will generate it, but if upload model by ormb command, we won't generate the ormbfile.yaml

Yeah, I think it is what we want. Then I am going to close it.
/close


feat: convert root-layout to full css grid

Description
Related to other PR in dialtone vue https://github.com/dialpad/dialtone-vue/pull/1253. This is the implementation on the dialtone side. Root layout converted to use pure CSS grid and get rid of the grid / flexbox hybrid we had previously. This should also fix the remaining issues on the docsite with root layout.
Fairly big change so please check for any weird edge cases you can think of.
Pull Request Checklist

[x] Ask the contributors if a similar effort is already in process or has been solved.
[x] Review the contribution guidelines.
[x] Use staging as your pull request's base branch. (All PRs using production as its base branch will be declined).
[x] Ensure all gulp scripts successfully compile.
[x] Update, remove, or extend all affected documentation.
[x] Ensure no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).

Obligatory GIF (super important!)

The TOC area is broken while resizing the screen

The TOC area is broken while resizing the screen

But seems like on dialpad.design is broken too, maybe just missing the correct grid-area?

Oh, that has actually been broken forever haha, not 100% sure what the best solution is here, should the TOC move to the top like it does in mobile mode?

merging as this was a preexisting issue. however we should look at it.


Remove CONTROLLER_DEVELOPMENT_MODE in favor of specific settings

Problem Statement
Using a one-size-fits-all dev mode approach violates one of the 12-factor principles https://12factor.net/dev-prod-parity and we're seeing it cause real-world problems.
Proposed Solution

Get rid of the CONTROLLER_DEVELOPMENT_MODE setting in favor of specific settings for specific behaviors.
Determine the minimal set of those "specific behaviors" that must be different on CI than in production and only enable those.

Additional Information
I speculate that allowing non-allowlisted image tags could move from a runtime knob to e.g. an annotation (or some other kind of config as code parameter).
Acceptance Criteria

[ ] CONTROLLER_DEVELOPMENT_MODE does not exist anymore
[ ] CI has anonymous reports disabled and everything else in dev mode enabled unless there's a well understood reason for that particular bit to be non-default

Adding this tentatively to 1.5.

@pmalek this issue mentions:

we're seeing it cause real-world problems.

Can you update the description with the "real-world problems" we're hitting please?

I knew that this non-specific wording would cause confusion. Apologies 😁
What I meant by real-world problems was:

untestable parts of the KGO logic leading to regressions because the code under test ran through different code paths
need to use DEVELOPMENT_MODE as a workaround in production to bypass KIC/Gateway image version tag validations.


make ssh port configurable

Hey there,
it'd be great if the SSH port was configurable for docker-machine create.
Something like --digitalocean-ssh-port 2222.
Thank you!

Thank you :heart:


Add alerts on container nodes

@miq-bot providers/container, enhancement

@miq-bot add_label providers/containers, enhancement

@simon3z please review, this was not mentioned but is also required for next week goal

@moolitayer I assume we'll have more of these depending on what entities we can attach alerts to. Do we support nodes at this point?

@moolitayer I assume we'll have more of these depending on what entities we can attach alerts to. Do we support nodes at this point?

Right we will have more, see response to your code comment. We will support nodes after this pr (the actual collection bringing in miq_alert_statuses on nodes as well as code allowing to define miq_alerts on them is in https://github.com/ManageIQ/manageiq/pull/12773)

@simon3z This is only to add the link now after suggested changes PTAL

@moolitayer I have the feeling we don't want to lose the alerts statuses when a resource disappears (:dependent => :destroy). Isn't it also containing all the history (I suppose that if you drop this you will also drop all the life-cycle history of actions related to the status).

@simon3z note this is how miq_alerts_statuses are attached to all other objects in the system.
@blomquisg can you please point me to somewhere where we purge data that is not dependent destroy?

@simon3z note this is how miq_alerts_statuses are attached to all other objects in the system.

@moolitayer if we end up dropping the history when entities disappear (not suggesting to go that way, just saying) then we need it (at the very least) to be very well documented and understood by the end user.

@simon3z Created https://github.com/ManageIQ/manageiq/pull/13430 to purge the alert statuses, we could remove the dependent destroy after that merges.
We should discuss what should show up on the screens at that point.
Are you ok with merging this one now?

LGTM :+1: cc @chessbyte
Assigning to @blomquisg for review/merge
@miq-bot assign blomquisg


Convert Keys to integer values for state/other APIs?

I'm making an filesystem API, and SlotMap would be PERFECT for holding file handles and such.
However, I need to interface with non-Rust at some point, and pass some sort of file descriptor to it. This can be any sort of 64-bit integer - it doesn't need to be sequential or represent an actual index - but I don't see any way to do this with a SlotMap without like, leaking the Key and using a pointer to it as the fd, which is very unsafe and something I would very much not like to do.
A different way to explain it is that I want some sort of value that represents a key, which I can hand off to some API that doesn't know what a slotmap is or even what Rust is, and then said API can hand me back that value later, unchanged, which I can once again use as a key for the slotmap.

See as_ffi.

Thanks!


Bluetooth: host: foreach bond only SMP|BREDR enabled

Bluetooth: host: foreach bond only SMP|BREDR enabled

@joerchan can you take another look please?

Could you add some more explanation in the commit message, i.e. what exactly was broken and why the change you're doing is the right one?

Ok, I will add the details of the breakage log to the commit

@joerchan and @jhedberg please take another look

I think || IS_ENABLED(CONFIG_BT_BREDR) should be removed. It won't compile if it is possible to disable CONFIG_BT_SMP

Done.

Thanks @anchao


[Issue] CompareExchangeHashesh.ps1 Stuck on "Comparison Complete. Writing Results

Edit: Ran all updates on 2012 R2 server and used latest version of script. Came back clean
Version <IP_ADDRESS>6
PS > .\CompareExchangeHashes.ps1
[03/10/2021 08:22:35] Started...
Found exchange version: 15.1.2106.2
Loaded baseline for 15.1.1466.3, hashes number 14591
Loaded baseline for 15.1.2106.2, hashes number 15237
Loaded baseline for 15.1.2106.13, hashes number 15237
Loaded baseline for 15.1.1531.3, hashes number 14653
Loaded baseline for 15.1.1591.10, hashes number 14653
BaselineData - 15.1.1591.10 15.1.1531.3 15.1.2106.2 15.1.1466.3 15.1.2106.13 15.1.2106.2
Processing 3 directories in parallel
Processing 3 directories in parallel
Processing 3 directories in parallel
Processing 3 directories in parallel
Processing 3 directories in parallel
Processing 3 directories in parallel
Processing 4 directories in parallel
Processing 4 directories in parallel
Processing 4 directories in parallel
Processing 4 directories in parallel
Comparison complete. Writing results.
Exporting  objects to results
Exported results to C:\users<user>\Downloads\15.1.2106.2_result.csv
[03/10/2021 08:44:59] Done.
Version <IP_ADDRESS>0
PS C:\users<user>\Downloads> .\CompareExchangeHashes.ps1
[03/09/2021 17:43:35] Started...
Found exchange version: 15.1.2106.2
BaselineData -  15.1.2106.2
Processing 3 directories in parallel
Processing 3 directories in parallel
Processing 3 directories in parallel
Processing 3 directories in parallel
Processing 3 directories in parallel
Processing 3 directories in parallel
Processing 4 directories in parallel
Processing 4 directories in parallel
Processing 4 directories in parallel
Processing 4 directories in parallel
Comparison complete. Writing results.

This is because it uses a inefficient way of adding to array.

Thank you!

This is fixed.


Add README to glance operator

This change just adds a simple README where the basic operator informations are provided.
Signed-off-by: Francesco Pantano<EMAIL_ADDRESS>

Adding a simple README here that was missing: @fultonj there's also a description of  the CephBackend and the related parameters.


location to use variables outside the vision step - custom SikuliX code

how to use location to use variables outside the vision step

y = y + 60
click (x,y)
vision_step('x = "' + x + '"')
vision_step('y = "' + y + '"')
vision dragDrop(Location(x,y), Location(500,250))
where could be the prob.  x and y goes exactly the same.  Is there any .value?

If relative coordinates is needed, I showed you an example link from the other issue -
https://github.com/kelaberetiv/TagUI/issues/535#issuecomment-522890904
https://github.com/kelaberetiv/TagUI/issues/537#issuecomment-523069703
So there is no need to use custom SikuliX commands by using vision step.
The above code is SikuliX Python code. I think your error is making x and y a string when it should be numbers. You can try something like below or explore more on SikuliX documentation. But there is no need to use this method because you can do it directly using above examples.
vision_step('x = ' + x)
vision_step('y = ' + y)

Gotcha! worked!!

Am sorry... Actually this didnt work.  Line 24 vision_step is not getting the values into the variables.  What could be the problem.
I am yet to master the knack.  Pls. suggest.

For drag and drop you don't have to use vision step. You can do a mouse down -> hover to some x,y coordinates, then mouse up. Make sure to have the cutting edge release (check setup section).
If you still want to use above logic, once variables go to SikuliX, you can reference directly with Location(sd1_x, sd1_y) don't have to use ` symbol to replace variables anymore. That is only used in TagUI because TagUI a lot of steps assume text as text and not variable names.

dragDrop is working nice if I give Location(100,200) - some location.  Would like to know only how to use variables.

In the image - I removed that back tick and still not substituting - LIne 17.
Advice?

I am able to change to hover and even the coordinates are coming - still some where there is challenge.  Why does it say cannot find a location?

Something strange is happening - if I hardcode, it works and if it is getting substituted from variable - it says cannot find - is this a bug?

Your screen shot shows that for the second Location you are using using backticks `

if I dont use back ticks - in the previous snapshot you can see that it substitutes the right values.  We have to use or just leave it?

Use backticks for variables in TagUI steps. For vision step in SikuliX, don't use backticks.

Also, make sure you are using the latest cutting edge version -
https://github.com/kelaberetiv/TagUI#set-up

Adding on, below is an example using the cutting edge version (typing tagui command with no parameters will show v5.5).
script
hover (100,200)
x = 100
y = 200
hover (`x`,`y`)

result
START - automation started - Thu Sep 05 2019 20:57:21 GMT+0800 (+08)

hover (100,200)
hover (100,200)

FINISH - automation finished - 14.8s


Can we use it for personal use only?

Like i have few websites where i want to keep and an eye like amazon or few real estate builer website.
Can i remove the twitter section?

Yes the tweeting should be optional. Did you try removing the Twitter config?

No havent yet. I still need to see from where to remove twitter config

In your diffengine directory you should see a config.yaml. remove the stanzas marked with twitter and give it a try.


Mingw32: _vsnprintf_s does not exist on msvcrt.dll on Windows 2003

Hello,
I use mbedtls on Windows 2003. I get the following error: The procedure entry point _vsnprintf_s could not be located in the dynamic link library msvcrt.dll. It is used because _TRUNCATE is defined in
/usr/i686-w64-mingw32/include/_mingw.h. As a workaround I use the attached patch, but is there a better solution?
Cheers,
Thomas
0001-Remove-calls-to-_vsnprintf_s-as-it-is-not-part-of-ms.txt

ARM Internal Ref: IOTSSL-551

Hi @sithglan.
Although the file msvcrt.dll is pre-installed on Windows, it's actually part of Visual Studio, and I believe the name is a shortening of 'Microsoft Visual C Runtime'. You may have an older version on Windows Server 2003, but Microsoft do provide newer versions that can be installed. See here for the Microsoft Visual C++ 2010 SP1 Redistributable Package (x64) which contains a newer version of msvcrt.dll, that according to MSDN here should contain a version of the runtime with the correct function. If you search around you should be able to find a combination of versions (Visual Studio 2010/2012/2015 etc) that suit your platform (Windows Server 2003) and architecture (e.g. i386 vs AMD64).
Please note, I haven't tested this out, so may have missed an issue.
Because the dependency - msvcrt.dll can be updated on platforms as old as Windows XP and Windows Server 2003, I don't think there should be a need to change mbed TLS, so I'm closing this defect. If you have any problems with changing msvrtldll, please let us know and we can revisit this.
Hope this helps.

Hi @sithglan.
Although the file msvcrt.dll is pre-installed on Windows, it's actually part of Visual Studio, and I believe the name is a shortening of 'Microsoft Visual C Runtime'. You may have an older version on Windows Server 2003, but Microsoft do provide newer versions that can be installed. See here for the Microsoft Visual C++ 2010 SP1 Redistributable Package (x64) which contains a newer version of msvcrt.dll, that according to MSDN here should contain a version of the runtime with the correct function. If you search around you should be able to find a combination of versions (Visual Studio 2010/2012/2015 etc) that suit your platform (Windows Server 2003) and architecture (e.g. i386 vs AMD64).
Please note, I haven't tested this out, so may have missed an issue.
Because the dependency - msvcrt.dll can be updated on platforms as old as Windows XP and Windows Server 2003, I don't think there should be a need to change mbed TLS, so I'm closing this defect. If you have any problems with changing msvrtldll, please let us know and we can revisit this.
Hope this helps.

Hello sbutcher,
for me this is not an option. Because what you tell me, that in order to use a 256 KB programm, I need to download a 40 MByte runtime in order to use it. Also it requires Administrator access which I may not have. So that is not really an option. So mbedtls runs on an unpatched Windows 2003/XP with one exception: This function. So if you don't care about this platform, this is fine for me. I'll continue to use the out of tree patch I already use since I submitted the patch but I really think this is the wrong way of doing this. This is why I submitted the error and patch.
Cheers,
Thomas

Hello sbutcher,
for me this is not an option. Because what you tell me, that in order to use a 256 KB programm, I need to download a 40 MByte runtime in order to use it. Also it requires Administrator access which I may not have. So that is not really an option. So mbedtls runs on an unpatched Windows 2003/XP with one exception: This function. So if you don't care about this platform, this is fine for me. I'll continue to use the out of tree patch I already use since I submitted the patch but I really think this is the wrong way of doing this. This is why I submitted the error and patch.
Cheers,
Thomas

Hi @sithglan,
I can appreciate your position. If you look at the issues here on github and on the forum, you'll see this isn't the only problem we have related to versions of Visual Studio earlier than 2010, (which because mingw uses the msvcrt.dll runtime I'll class this as).
I'll reopen the bug for the moment, while we decide what we want to do with these older Microsoft platforms, and if we can support them. If we do, I may come back to you and ask you to sign a CLA to use your patch.
As a final point - to make use of the msvcrt.dll in the .msi I gave above - you may not need to install the .msi itself as administrator, but simply need to extract the .dll (and any dependencies) out of the .msi, and copy them locally with your application code.

Hi @sithglan,
I can appreciate your position. If you look at the issues here on github and on the forum, you'll see this isn't the only problem we have related to versions of Visual Studio earlier than 2010, (which because mingw uses the msvcrt.dll runtime I'll class this as).
I'll reopen the bug for the moment, while we decide what we want to do with these older Microsoft platforms, and if we can support them. If we do, I may come back to you and ask you to sign a CLA to use your patch.
As a final point - to make use of the msvcrt.dll in the .msi I gave above - you may not need to install the .msi itself as administrator, but simply need to extract the .dll (and any dependencies) out of the .msi, and copy them locally with your application code.

Hello @sbutcher-arm,
thank you for considering. My patch is trivial but I'm happy to sign a CLA if necessary. To explain my usecase a little bit more. I provide one executable to my users. They download an run it. I want to keep it as easy as possible. So distributing any additional files is not an option.
Cheers,
Thomas

Hello @sbutcher-arm,
thank you for considering. My patch is trivial but I'm happy to sign a CLA if necessary. To explain my usecase a little bit more. I provide one executable to my users. They download an run it. I want to keep it as easy as possible. So distributing any additional files is not an option.
Cheers,
Thomas

I had a quick look into this, in case it was related to a PR I have open about Visual Studio support (PR https://github.com/ARMmbed/mbedtls/pull/353). It isn't, but now I've worked out what the problem is I hope you don't mind if I add my opinion!
On Windows, the "normal" compiler is Visual Studio, which comes with a CRT (=libc) in the form of a DLL and a static library. It's up to the application developer to choose whether to link to it dynamically or statically. If you link dynamically, the correct DLL version needs to be present on the customer's machine, either by installing the "Microsoft Visual C++ 20XX Redistributable" along with your application, or just shipping the DLL. That's why many (most?) applications link statically, which slightly increases binary sizes but the resulting exes are universal and have no CRT dependency.
What's different with MinGW is that it doesn't offer a way to statically link to the CRT (because it's "not free enough"). So instead it tries to link to the version-independent functions from the msvcrt.dll that ships with Windows, which doesn't contain _vsnprintf_s on Windows XP.
So, the options are: (1) tell mbedTLS users to use Visual Studio instead; (2) tell users you don't support Windows XP with MinGW (an odd combination of an out-of-support OS with an unusual compiler); or (3) don't call _vsnprintf_s on MinGW.
Getting rid of the calls to _vsnprintf_s will cause compiler warnings on Visual Studio, so a better patch would be to blacklist MinGW like this:
--- a/library/platform.c
+++ b/library/platform.c
@@ -74,16 +74,12 @@ int mbedtls_platform_win32_snprintf( char *s, size_t n, const char *fmt, ... )
		 return( -1 );

	 va_start( argp, fmt );
-#if defined(_TRUNCATE)
+#if defined(_TRUNCATE) && !defined(__MINGW32__)
     ret = _vsnprintf_s( s, n, _TRUNCATE, fmt, argp );
 #else
	 ret = _vsnprintf( s, n, fmt, argp );
	 if( ret < 0 || (size_t) ret == n )
	 {

I had a quick look into this, in case it was related to a PR I have open about Visual Studio support (PR https://github.com/ARMmbed/mbedtls/pull/353). It isn't, but now I've worked out what the problem is I hope you don't mind if I add my opinion!
On Windows, the "normal" compiler is Visual Studio, which comes with a CRT (=libc) in the form of a DLL and a static library. It's up to the application developer to choose whether to link to it dynamically or statically. If you link dynamically, the correct DLL version needs to be present on the customer's machine, either by installing the "Microsoft Visual C++ 20XX Redistributable" along with your application, or just shipping the DLL. That's why many (most?) applications link statically, which slightly increases binary sizes but the resulting exes are universal and have no CRT dependency.
What's different with MinGW is that it doesn't offer a way to statically link to the CRT (because it's "not free enough"). So instead it tries to link to the version-independent functions from the msvcrt.dll that ships with Windows, which doesn't contain _vsnprintf_s on Windows XP.
So, the options are: (1) tell mbedTLS users to use Visual Studio instead; (2) tell users you don't support Windows XP with MinGW (an odd combination of an out-of-support OS with an unusual compiler); or (3) don't call _vsnprintf_s on MinGW.
Getting rid of the calls to _vsnprintf_s will cause compiler warnings on Visual Studio, so a better patch would be to blacklist MinGW like this:
--- a/library/platform.c
+++ b/library/platform.c
@@ -74,16 +74,12 @@ int mbedtls_platform_win32_snprintf( char *s, size_t n, const char *fmt, ... )
		 return( -1 );

	 va_start( argp, fmt );
-#if defined(_TRUNCATE)
+#if defined(_TRUNCATE) && !defined(__MINGW32__)
     ret = _vsnprintf_s( s, n, _TRUNCATE, fmt, argp );
 #else
	 ret = _vsnprintf( s, n, fmt, argp );
	 if( ret < 0 || (size_t) ret == n )
	 {

Hello @NWilson,
thank you for getting to the bottom of this. I think you're fix is the way to go. I produced a small patch which fixes both occurances in the source code and tested it on Windows 2003.
Cheers,
Thomas
0001-Blacklist-_vsnprintf_s-on-MINGW-to-main-downward-com.txt

Hello @NWilson,
thank you for getting to the bottom of this. I think you're fix is the way to go. I produced a small patch which fixes both occurances in the source code and tested it on Windows 2003.
Cheers,
Thomas
0001-Blacklist-_vsnprintf_s-on-MINGW-to-main-downward-com.txt

Hi @sithglan,
If you'd still be happy to sign or agree to a Contributor's License Agreement (CLA), I think we should accept your patch and close this, by now, quite old defect. Unfortunately, the alternative is probably to close the issue unfixed as it's unlikely anyone else will submit another fix, and it's not something the team can spend time on.
You can find an agreement to sign here, which can be signed and returned to us, or you could create an mbed account and accept a slightly different agreement here with a click through if this is a personal contribution.
Apologies for the long, long delay on resolving this, which is my fault for prematurely closing the issue.

Hello Simon,

Simon Butcher  [2016-06-01 20:31]:

If you'd still be happy to sign or agree to a Contributor's License
Agreement (CLA), I think we should accept your patch and close this,
by now, quite old defect. Unfortunately, the alternative is probably
to close the issue unfixed as it's unlikely anyone else will submit
another fix, and it's not something the team can spend time on.

find the signed CLA in separate post. Find attached the two patches to
make mbedtls work. Also note that I incorporated Nicholas Wilson
suggestion.
Cheers,
Thomas
From a076a0d8cec28ed14e6fb5c387a357fa5c6dc597 Mon Sep 17 00:00:00 2001
From: Thomas Glanzmann<EMAIL_ADDRESS>Date: Fri, 1 Jan 2016 13:29:29 +0100
Subject: [PATCH] Blacklist _vsnprintf_s on MINGW to maintain downward
compatibility for Windows 2003/XP

library/debug.c    | 2 +-
library/platform.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/library/debug.c b/library/debug.c
index 4752ab1..afa21cb 100644
--- a/library/debug.c
+++ b/library/debug.c
@@ -90,7 +90,7 @@ void mbedtls_debug_print_msg( const mbedtls_ssl_context *ssl, int level,
 va_start( argp, format );

#if defined(_WIN32)
-#if defined(_TRUNCATE)
+#if defined(_TRUNCATE) && !defined(MINGW32)
ret = _vsnprintf_s( str, DEBUG_BUF_SIZE, _TRUNCATE, format, argp );
#else
ret = _vsnprintf( str, DEBUG_BUF_SIZE, format, argp );
diff --git a/library/platform.c b/library/platform.c
index d634c62..e90dc3c 100644
--- a/library/platform.c
+++ b/library/platform.c
@@ -74,7 +74,7 @@ int mbedtls_platform_win32_snprintf( char *s, size_t n, const char *fmt, ... )
return( -1 );
 va_start( argp, fmt );

-#if defined(_TRUNCATE)
+#if defined(_TRUNCATE) && !defined(MINGW32)
ret = _vsnprintf_s( s, n, _TRUNCATE, fmt, argp );
#else
ret = _vsnprintf( s, n, fmt, argp );
2.1.4
From a56ea243895426baa2eb76bddbbea604a076ab16 Mon Sep 17 00:00:00 2001
From: Thomas Glanzmann<EMAIL_ADDRESS>Date: Thu, 7 Jan 2016 11:14:59 +0100
Subject: [PATCH] Unbreak make install for mingw32

Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Makefile b/Makefile
index 0950e6b..a41f52d 100644
--- a/Makefile
+++ b/Makefile
@@ -20,7 +20,7 @@ tests: lib
$(MAKE) -C tests
ifndef WINDOWS
-install: no_test
+install: lib
mkdir -p $(DESTDIR)/include/mbedtls
cp -r include/mbedtls $(DESTDIR)/include
--
2.1.4

Hi @sithglan I have made a PR in https://github.com/ARMmbed/mbedtls/pull/1079 according to your patch, but since I don't have the right environment, I would be happy if you could confirm


Is PersonListPanel.fxml still used?

It seems to have been dropped as a dependency from MainWindow.java.

No


Integrate Cloudinary to Netlify CMS

Closes #395
Description
Integrate Cloudinary to Netlify CMS so content admins/editors can upload images from the admin UI.
Current Tasks

[x] update netlify.toml
[x] register custom media library - docs
[x] customize configs/options
[x] update related image/UI component
[ ] ?? test htmrTransformer

🤯 literally plug and play!! still have to configure options etc, just impressed with the good DX.

Anyone has hints/example on how to test the htmr-transformers functions?

Please mark it as "Ready to Review" @ekafyi


Add Asus ROG Strix G513IM

Description of changes
Added a new profile for the Asus ROG Strix G513IM laptop.

[x] Tested the changes in your own NixOS Configuration
[x] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

@mergify queue


[Hot Fix] Fix flaky ZeppelinIT build

What is this PR for?
Most of the builds are failing for ZeppelinIT at testSparkInterpreterDependencyLoading:243, as the previous line's testDepRemoveBtn.click() is taking it to a different page.
What type of PR is it?
[Hot Fix]
Todos

[x] - Change all add/remove dependencies/property's div to button
[x] - make relevant change in selenium test case.

What is the Jira issue?
ZEPPELIN-950
How should this be tested?
CI should pass
Questions:

Does the licenses files need update? N/A
Is there breaking changes for older versions? N/A
Does this needs documentation? N/A

Tried it 4 different times

https://travis-ci.org/apache/incubator-zeppelin/builds/135110386
https://travis-ci.org/apache/incubator-zeppelin/builds/135119542
https://travis-ci.org/apache/incubator-zeppelin/builds/135124485
https://travis-ci.org/apache/incubator-zeppelin/builds/135128424

Every time *.7 passes, but fails for either spark download or npm install.

thanks! we should probably look into download/network errors too.
LGTM

Sure, I'll try to figure something out for npm related network error(s).
I'll merge this if no more discussion.


Create GWN model in message passing way

Such as pytorch_geometric's MessagePassing. Thx!

I made a toy GWNet with message passing diffuse convolution layer. If someone is interested, I can put it here, this project or else.


Added Cloud Native Security File

Describe your changes
Created a file cloud-native-security.md to localize the page: glossary/content/ja into japanese
Related issue number or link (ex: resolves #issue-number)
#2112
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your git-config by running git config --global alias.ci "commit -s".

Hi @naonishijima
Please guide @aryasoni98 to use the right dev branch for Japanese localization. :)

Please review my PR @caniszczyk @seokho-son @jihoon-seo @CathPag @JasonMorgan @iamNoah1

Hi @aryasoni98.
Please check my comment https://github.com/cncf/glossary/pull/2126/#issuecomment-1589160231 .
We can not PR to main branch. Localization Japanese branch is dev-ja.

Hi @aryasoni98. Please check my comment #2126. We can not PR to the main branch. Localization Japanese branch is dev-ja.

I just make another PR #2140 please check it. So, should I close this PR


Experimental/solo node consensus

Problem
Fix #179
Solution
Copy these scripts into image.
Checklist

[x] I have ran the tests locally.
[x] I branched off of the experimental/solo-node-consensus branch and not master.
[x] I did not change the VERSION file.

Thank you @Ashuaidehao


ZMQ sends and threads conflict

ZMQ doesn't seem to like threads (I saw some stuff about that in the documentation but can't find it right now).
Right now, all threads share the same context and thus uses the same socket to communicate with the server which
might be an issue. Or maybe ZMQ sockets don't like bursts of messages.
The error is the following:
[1,0]<stderr>: [quo-vadis error at (qvi-rmi.cc::qvi_zerr_msg::101)] zmq_msg_send() failed with errno=156384763 (Unknown error 156384763)
[1,0]<stderr>: [quo-vadis error at (qvi-rmi.cc::qvi_zerr_msg::101)] zmq_msg_send() truncated with errno=156384763 (Unknown error 156384763)

Adding some delay in the example (with a call to sleep) fixes the issues, adding  a lock doesn't fix anything so I will investigtate the burst of messages.  I already tried option for ZMQ sockets but without positive results.
See PR https://github.com/hpc/quo-vadis/pull/163

I have a fix doing it a brute-force way, but let me see if I can come up with a nicer solution.

And what is the current fix? I'm curious.

Having a context mutex managed by a lock_guard at the interface boundary. A little heavy handed, so I think I can do better.

But I think I tried this and it didn't work.

I'm not sure how you implemented it, but mine seems to do the trick.

struct qv_context_s {
    qvi_rmi_client_t *rmi = nullptr;
    qvi_zgroup_t *zgroup = nullptr;
    qvi_bind_stack_t *bind_stack = nullptr;
    pthread_mutex_t lock;

Ok, I'm puzzled.

I guess it has to do with when/where you do lock/unlock

I put the lock/unlock phase around the call to qv_bind_push in qv_thread_routine

@GuillaumeMercier your issues should be fix by #164. I've also pushed a new branch named thread-bug-work that has the fixes to build your code. When you are ready, please issue a pull request so I can merge your work into master.

@GuillaumeMercier things look much better now since merging #189. For what it's worth, there were other issues outside the RMI and threading impacting this one (e.g., unreliable binding after split). One particularly nasty bug to track was stale task data being passed to hwloc. Now these data are gathered at the time the call is performed. And the relevant task details update appropriately when threads are introduced. Closing for now, but please re-open if needed.

@samuelkgutierrez : so, did you get rid of the lock?

I'll update my repo and make tests asap.

Yes, I got rid of the lock. Let me push a minor update before you test.

No sweat, I won't make tests today, but maybe tomorrow.

The minor change is in. Thank you for testing! Fingers crossed.


[Bug] TC-CNET-4.13 Step2 provided command used a wrong parameter.

Describe the bug
Step 2: TH readsNetworksattribute from theDUT and saves thenumber of entries as 'NumNetworks'
./chip-tool network-commissioning read networks 1 0
It should be
./chip-tool networkcommissioning  read networks 1 0
Steps to reproduce the behavior
No response
Expected behavior
No response
Log files
No response
PICS file
No response
Screenshots
No response
Environment
No response
Additional Information
No response

@slidesoon Are you referring to the verification steps spreadsheet or the TH GUI?

TH GUI

@slidesoon Are you referring to the verification steps spreadsheet or the TH GUI?

@raul-marquez-csa It seems a script issue, could you please take a look?


Undefined variable in ansible export

Type: Bug or PEBKAC
OKIT version: 0.14.1
Component: Ansible generator
Severity: medium, workaround possible
Description:

To reproduce: I "draw" a rudimentary configuration with only one VCN and the bastion fragment. Only the bastion's subnet CIDR needs to be changed. Then I export the content to Ansible.
Problem: In okit-ansible.zip/main.yml the task named "Create Security List okit-bastion-sl" tries to use the variable Okit_Vcn001_default_security_list_id and fails.
Workaround: This variable should have been set in set_fact just after "Create Virtual Cloud Network okit-vcn001" but is missing there. It can manually be added in main.yml

Attached files:

okit.json.zip
okit-ansible.zip

Variable will be added in the 0.15.0 release.
Just as an FYI we will be upgrading to the Ansible Collection version of the in the New Year


Correct double date in Posts

This solves #746
A few posts had the date hard written in the md file, when Jekyll is building automatically a date header based on the date in the title.
As a result, those posts were displaying the date twice.
I removed the hard written date.

Tested ok on the following platforms:
| Edge | IE 11 | Chrome | Firefox | Opera | Vivaldi
-- | -- | -- | -- | -- | -- | --
Windows 7 | - | OK | OK | OK | OK | OK
Windows 10 | OK | OK | OK | OK | OK | OK
Ubuntu 18.04 | - | - | OK | OK | OK | OK
CentOS 7 | - | - | - | OK | OK | OK

Before:

After:


Won't draw small images on all lines.

I am using this library to interface with 2 of the Sure Electronics 24x16 displays. I made the necessary changes in the header file to set up the library to work with my display, and most of the time everything works fine. There is just an issue with drawing images of height less than 4.
I have found that if I try to draw an image with a height of less than 4, the image will not get drawn at certain y values.
I tested with a single pixel image...
char IMG_PIXEL[] = {0b0001};
#define IMG_PIXEL_WIDTH 1
#define IMG_PIXEL_HEIGHT 1
When height is set to 1, it will only draw when y is 0, 4, 8, and 12.
When height is set to 2, it will only draw when y is 0, 1, 4, 5, 8, 9, 12, and 13.
When height is set to 3, it will only draw when y is 0, 1, 2, 4, 5, 6, 8, 9, 10, 12, 13, and 14.
When height is set to 4 or more, it will draw on any line.

Hi Jzooor,
Thanks to your wonderfully detailed debug information, I found the problem.
With reference to HT1632.cpp Line 231 and Line
241https://github.com/gauravmm/HT1632-for-Arduino/blob/master/Arduino/HT1632/HT1632.cpp#L231-241
(which
are identical), the following expression is incorrect:
(height-loc_y >= 4)
The corrected expression is:
(height-j >= 4)
The fix is to substitute the first expression with the second expression in
Line 231 and Line 241 of HT1632.cpp. As I have no test hardware at the
moment, could you please verify that this fix works as designed? If it does
not work properly, please notify me so that I can invest more time into
understanding the bug.
Thanks for your help!
Regards,
Gaurav
On Wed, Oct 26, 2011 at 2:32 AM, Jzooor <
<EMAIL_ADDRESS>
I am using this library to interface with 2 of the Sure Electronics 24x16
displays. I made the necessary changes in the header file to set up the
library to work with my display, and most of the time everything works fine.
There is just an issue with drawing images of height less than 4.
I have found that if I try to draw an image with a height of less than 4,
the image will not get drawn at certain y values.
I tested with a single pixel image...
char IMG_PIXEL[] = {0b0001};
#define IMG_PIXEL_WIDTH 1
#define IMG_PIXEL_HEIGHT 1
When height is set to 1, it will only draw when y is 0, 4, 8, and 12.
When height is set to 2, it will only draw when y is 0, 1, 4, 5, 8, 9, 12,
and 13.
When height is set to 3, it will only draw when y is 0, 1, 2, 4, 5, 6, 8,
9, 10, 12, 13, and 14.
When height is set to 4 or more, it will draw on any line.
--
Reply to this email directly or view it on GitHub:
https://github.com/gauravmm/HT1632-for-Arduino/issues/6

--
Regards,
Manek

I tried out the changes you suggested, there was no change in the behavior.

I solved this issue replacing line 241:
this:
(height-loc_y >= 4)
with:
(height-loc_y >= 0)
I don't know if it's the right way to fix but works =)


return None using demo

I got None when I use the following demo. I try to changed different network and also tried to use proxy, but all failed. I wonder why.
from duckduckgo_search import ddg

keywords = 'Bella Ciao'
results = ddg(keywords, region='wt-wt', safesearch='Off', time='y')
print(results)

2.9.5

Please run this and show the result
import logging
from duckduckgo_search import ddg

logging.basicConfig(level=logging.DEBUG)

keywords = 'Bella Ciao'
results = ddg(keywords, region='wt-wt', safesearch='Off', time='y')
print(results)

DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): duckduckgo.com:443
DEBUG:urllib3.connectionpool:https://duckduckgo.com:443 "POST / HTTP/1.1" 200 None
ERROR:duckduckgo_search.utils:
Traceback (most recent call last):
  File "C:\Users\86132\AppData\Roaming\Python\Python310\site-packages\duckduckgo_search\utils.py", line 51, in _get_vqd
    vqd_index_start = resp.content.index(b"vqd='") + 5
ValueError: subsection not found
WARNING:duckduckgo_search.utils:keywords=Bella Ciao. _get_vqd() is None, refreshing SESSION
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): duckduckgo.com:443
DEBUG:urllib3.connectionpool:https://duckduckgo.com:443 "POST / HTTP/1.1" 200 None
ERROR:duckduckgo_search.utils:
Traceback (most recent call last):
  File "C:\Users\86132\AppData\Roaming\Python\Python310\site-packages\duckduckgo_search\utils.py", line 51, in _get_vqd
    vqd_index_start = resp.content.index(b"vqd='") + 5
ValueError: subsection not found
WARNING:duckduckgo_search.utils:keywords=Bella Ciao. _get_vqd() is None, refreshing SESSION

Do you also return None when you test?

I checked the code, you have an old version.
It should be like this: https://github.com/deedy5/duckduckgo_search/blob/90a2cd7f5951b4dc5373753d65343b75a75293ba/duckduckgo_search/utils.py#LL51C13-L51C17
Reinstall the package.
pip install -I duckduckgo_search

Thank a lot! The problem may be that I have several different Pythons and the version I used was older.

I am having the same issue as OP, but the command you gave to reinstall did not fix the issue. I have version 2.9.5 and am on python 3.10

https://github.com/deedy5/duckduckgo_search/issues/61#issuecomment-1548065763

DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): duckduckgo.com:443
INFO:duckduckgo_search.utils:_get_vqd() keywords=Bella Ciao ConnectTimeout HTTPSConnectionPool(host='duckduckgo.com', port=443): Max retries exceeded with url: / (Caused by ConnectTimeoutError(<urllib3.connection.HTTPSConnection object at 0x7f33e6681810>, 'Connection to duckduckgo.com timed out. (connect timeout=10)'))
WARNING:duckduckgo_search.utils:keywords=Bella Ciao. _get_vqd() is None, refreshing SESSION
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): duckduckgo.com:443
INFO:duckduckgo_search.utils:_get_vqd() keywords=Bella Ciao ConnectTimeout HTTPSConnectionPool(host='duckduckgo.com', port=443): Max retries exceeded with url: / (Caused by ConnectTimeoutError(<urllib3.connection.HTTPSConnection object at 0x7f33e6682aa0>, 'Connection to duckduckgo.com timed out. (connect timeout=10)'))
WARNING:duckduckgo_search.utils:keywords=Bella Ciao. _get_vqd() is None, refreshing SESSION
None
```DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): duckduckgo.com:443
INFO:duckduckgo_search.utils:_get_vqd() keywords=Bella Ciao ConnectTimeout HTTPSConnectionPool(host='duckduckgo.com', port=443): Max retries exceeded with url: / (Caused by ConnectTimeoutError(<urllib3.connection.HTTPSConnection object at 0x7f33e6681810>, 'Connection to duckduckgo.com timed out. (connect timeout=10)'))
WARNING:duckduckgo_search.utils:keywords=Bella Ciao. _get_vqd() is None, refreshing SESSION
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): duckduckgo.com:443
INFO:duckduckgo_search.utils:_get_vqd() keywords=Bella Ciao ConnectTimeout HTTPSConnectionPool(host='duckduckgo.com', port=443): Max retries exceeded with url: / (Caused by ConnectTimeoutError(<urllib3.connection.HTTPSConnection object at 0x7f33e6682aa0>, 'Connection to duckduckgo.com timed out. (connect timeout=10)'))
WARNING:duckduckgo_search.utils:keywords=Bella Ciao. _get_vqd() is None, refreshing SESSION
None

I'm sorry but I got the same problem in v3.2.0. I got None when I using demo code. I've had success getting the result, but most of the time I've returned None when I use DDGS().text()

Something has changed in the api again, I'm trying to figure it out. After a certain number of successful responses api starts to give responses with status code 202, after which the status code changes to 418 with ip blocking for 60 seconds.
This only applies to DDGS().text().

Can confirm I am seeing the same with 3.2.0. Examining the debug logs, every request is met with a 202 response.

I confirm that I faced the same problems when using v3.2.0, DDGS().text() returns nothing.

I have the same problem with V3.3.0 using the provided example.
If the keywords value is 'Bella Ciao' it produce an error,
If the keywords value is 'Bella Ciao filetype:pdf' there's no error
from duckduckgo_search import DDGS

ddgs = DDGS()

# keywords = 'Bella Ciao'
keywords = 'Bella Ciao filetype:pdf'

ddgs_text_gen = ddgs.text(keywords, region='wt-wt', safesearch='Off', timelimit='y')
for r in ddgs_text_gen:
	print(r)

I can't reproduce the problem right now. Everything works.


Update kubeadm reference for v1.32

This PR updates the reference docs for kubeadm in v1.32.

Description

Issue

Closes: #

/approve


Track androidx.paging:paging-*:3.2.0-alpha05 and androidx.paging:paging-compose:1.0.0-alpha19

Release Notes
androidx.paging:paging-*:3.2.0-alpha05
androidx.paging:paging-compose:1.0.0-alpha19

3.2.0-alpha05-0.2.1 has been released.


Fix multiple select form storage

The code
input.attr('multiple')

returns an empty string (which will be interpreted as false) in this case
<select multiple></select>

Thanks


"expire" notification was lost when waiting on a directory recursively, and there are more than one child expires at the same time

it's etcd 0.4.6, here is the steps:

tcpdump on  port 4001

sudo tcpdump -i lo port 4001 -s0 -w tcpdump.cap

execute the following script on another terminal

curl -XPUT http://<IP_ADDRESS>:4001/v2/keys/dir/key1 -d value=value1 -d ttl=5;
curl -XPUT http://<IP_ADDRESS>:4001/v2/keys/dir/key2 -d value=value2 -d ttl=5;
curl "http://<IP_ADDRESS>:4001/v2/keys/dir?wait=true&recursive=true&stream=true";

after 5 seconds, the watcher will receive only one 'expire' notification not two. the captured 'tcpdump.cap' is for details.

watch stream is pretty racy right now. in v3, we support watch stream. there is an ongoing fix: https://github.com/coreos/etcd/pull/2879. we are not sure it still worth the effort though. so probably we will not fix this for v2.


HelixBootstrapTool changes to support InstanceConfig-To-PropertyStore migration

Ambry will eventually migrate datanode info (i.e. disk, replica) from InstanceConfig to PropertyStore in Helix. This PR makes several changes in HelixBootstrapUpgradeTool to support both. User should be able to specify which DataNodeConfigSource type to use and the tool will update Helix accordingly.
(Tests to be added)

Codecov Report

Merging #1739 (7e61959) into master (31649db) will decrease coverage by 58.48%.
The diff coverage is 2.32%.

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


Add user on login

Changes:

User is created on login rather than at AboutMe creation (changes to loginServlet)
If a user attempts to access a non-existent user page, it redirects to an error page
Some fixup of error pages and navigation
Fixes issue #43

While you made some changes in the AboutMeServlet's doGet, I'm not sure we'll even use the doGet function anymore since the user page is SSRed, unless we plan to display the AboutMe elsewhere, where we can use doGet. Although, we can keep the doGet, as long as we make sure that any added behavior/code is also added in the UsersServlet.

I will see if using the doGet method from master works with my code and update - if it does, I will check with @acolwell if it is better to remove this method or not - I feel like it's a little strange to have a doPost without a doGet, but I also think it is strange not to need the doGet... I am wondering about best coding practices

doGet is updated and it works identically, so I'm going to go ahead and merge. Thank you!


How readonly txn.Reset and txn.Renew use?

I create a read-only txn in a goroutine, then this txn will called in another goroutine frequently.
The scene is:

four writer goroutines write to the db, then four reader goroutines read this db(this four readers don't share txn.)
each time, a reader read some data from db (such as 20 key one time)，if db has not enough data, return, after 50ms, read again.

so I call txn.Reset after I create txn in a goroutine, before read call txn.Renew, after read call txn.Reset again. But some time, txn.Renew return error:

mdb_cursor_renew: invalid argument

So

How to use these two funcs?
Need I call cursor.Renew after txn.Renew?

So are you using a cursor in your read transactions? I'm not immediately sure why mdb_cursor_renew is in that error message.
The order of txn operations sounds fine... Begin, Reset, Renew, Get, Reset, Renew, Get, Reset, Renew, ..., Abort. If cursors are not involved I believe that should work.
When you are using cursors you should make sure to call txn.Renew() before calling cursor.Renew(txn) -- make sure the transaction you are giving the cursor is "open".  If you call those methods in reverse order you will probably get an error like the one you provided.

@bmatsuo after txn.Reset and before txn.Renew, may GC will free this txn?

@zwb-ict no, the GC will not free the txn. As far as I know the primary benefit of reseting/renewing transactions is to reduce allocations and bookkeeping. It wouldn't be very useful if the GC destroyed the transaction between calls.
You should also make sure that txn.Abort() is called when you are done with the Txn object because the C library needs to clean up the transaction and do its final bookkeeping.

@bmatsuo this caused by open the same env in multi goroutines.


Mangasee source broken

The Mangasee source is broken.
From the logs:
"Failed to get available sources: err: Error: Command failed with exit code 1: mangal inline --source Mangasee --query xxxx -j\nX /config/.config/mangal/sources/Mangasee.lua:46: attempt to index a non-table object(nil) with key 'page'\nstack traceback:\n\t/config/.config/mangal/sources/Mangasee.lua:46: in main chunk\n\t[G]: ?"}

It seems like headless chrome cannot start when you select the mangasee. But I couldn't reproduce this. https://github.com/oae/kaizoku/blob/5713b206a0dbfb329ef3ac8466eca4a2c93b05a5/docker/mangal/sources/Mangasee.lua#L46

There is a bunch of missing dependencies for headless chrome to work in the docker container.
I installed these that I found in a random list somwhere saying they are needed dependencies for headless chrome: ca-certificates fonts-liberation libappindicator3-1 libasound2 libatk-bridge2.0-0 libatk1.0-0 libc6 libcairo2 libcups2 libdbus-1-3 libexpat1 libfontconfig1 libgbm1 libgcc1 libglib2.0-0 libgtk-3-0 libnspr4 libnss3 libpango-1.0-0 libpangocairo-1.0-0 libstdc++6 libx11-6 libx11-xcb1 libxcb1 libxcomposite1 libxcursor1 libxdamage1 libxext6 libxfixes3 libxi6 libxrandr2 libxrender1 libxss1 libxtst6 lsb-release wget xdg-utils
Mangasee now works as it should.


Allow null values to be passed into Spanner DML

Update other areas where we use DML/queries with spanner to pass in types

Update all operators to properly pass in types to parameterize()


Search service memory issue

Search service suffers from memory issues.
The process reaches 12GB of memory consumption before GC collection is initiated, on machines with 14GB of memory.
This causes IIS to stop functioning, and the service is shutdown.

This issue is on-going.
https://icm.ad.msft.net/imp/v3/incidents/details/36045155/home
https://icm.ad.msft.net/imp/v3/incidents/details/36087582/home
https://icm.ad.msft.net/imp/v3/incidents/details/36112252/home
https://icm.ad.msft.net/imp/v3/incidents/details/36112253/home
The last case (93%) occurred on a prod USSC staging slot. This indicates an issue with index reload (given that staging slots get no traffic).


Automated cherry pick of #1408: init model

Cherry pick of #1408 on release/2.11.
#1408: init model

/lgtm
/approve

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@8dc6bd1). Click here to learn what that means.
The diff coverage is 3.68%.

@@              Coverage Diff               @@
##             release/2.11   #1966   +/-   ##
==============================================
  Coverage                ?    7.4%
==============================================
  Files                   ?     432
  Lines                   ?   66382
  Branches                ?       0
==============================================
  Hits                    ?    4917
  Misses                  ?   61051
  Partials                ?     414

Flag
Coverage Δ

#aFlag
7.4% <3.68%> (?)

Impacted Files
Coverage Δ

pkg/util/aws/bucket.go
0% <ø> (ø)

pkg/util/ucloud/ufile.go
0% <ø> (ø)

pkg/util/aws/networkinterface.go
0% <0%> (ø)

pkg/util/aws/dbinstance_database.go
0% <0%> (ø)

pkg/compute/models/cloudregions.go
2.15% <0%> (ø)

pkg/compute/models/cloudproviders.go
1.08% <0%> (ø)

pkg/util/aws/dbinstance_snapshot.go
0% <0%> (ø)

pkg/util/aws/aws.go
0% <0%> (ø)

pkg/util/aws/dbinstance.go
0% <0%> (ø)

pkg/util/aws/region.go
0% <0%> (ø)

... and 13 more

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 8dc6bd1...ad45ac9. Read the comment docs.


Update virtual-machine-recs.md

Increased the minimum CPU to 12 vCPU and replaced NV6 (retired) with NV12s_v3 which is the closest replacement of NV6.

#assign: @robinharwood, @Xelu86
@robinharwood @Xelu86
Can you review the proposed changes?
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Updates constructors to be const.

Hell yeah. Thank you @nathansizemore - this is great. I'll tag a release.

The crate is great, thank you!
This would be beneficial for my use of wanting to keep WGS84 refs around as statics and would allow me to remove using lazy_static crate for those

v0.4.2 on crates now, thanks again @nathansizemore - this is great


Tabs without titles

Coverage increased (+0.1%) to 66.569% when pulling 0e0c5813d201a7f9ab0e98bdf43402227bb364c1 on feature/tabs-without-titles into dbad9bb33df368ea0ed580df4c989668665f7065 on development.

@yombunker Check out the latest changes.
Now the shifting behavior on titleless tab is allowed, but I feel it needs something as it looks quirky. Maybe the active icon should be lifted up a little bit? I already tried that with the titles on shifting titleless items set to "invisible" instead of a "gone" visibility, but that looked even weirder. Maybe the optimal is somewhere in the middle.
Let me know if you have suggestions what the selected shifting & titleless items should look like.

@roughike i'll try it in a few hours to see how it looks, but, if it looks weird then i guess will be better to disable it as you had it, cause, then we will get more issues open of people that want it and it looks weird :)

Coverage decreased (-1.1%) to 65.296% when pulling 649cb8c320956976818f7e2fb7e1751e4df99d1d on feature/tabs-without-titles into dbad9bb33df368ea0ed580df4c989668665f7065 on development.

I added some simple zoom animation for shifting titleless tabs, which looks at least better than the first alternative.

Coverage decreased (-1.7%) to 64.739% when pulling d8d9c569ce32c89df340566f8d097930e152e1eb on feature/tabs-without-titles into dbad9bb33df368ea0ed580df4c989668665f7065 on development.

@roughike you are right, looks weird :P let's better prevent it. You made a right call by not allowing it in the first place.

I'm betting that if we disable shifting titleless tabs, people will probably complain more than just simply allowing it. But this is hard to decide.

@roughike not saying remove shifting, but, don't allow shifting and icons only, kind of what you did in the first place. But, go to sleep :)

I was talking about the same thing 💩
Maybe we could track down all the open and closed issues about this, make a gif on the shifting + iconOnly tab behavior and have a vote:

Should we keep it as is?
Should we change it? How?
Should we just disable shifting + iconsOnly tabs alltogether

Then just @mention all those people that originally requested this feature or commented on those feature requests in this PR / a separate issue. #373 is a good starting point, but I bet there's a lot of these closed and open.

I think the current way of shifting & iconsOnly is just fine. Let's allow it, people will create issues if they want some other behavior.

If there way to describe tab behaviour for every tab instead of for whole tab bar layout?
For example I want to have one titless element in my bar.

@dimorinny maybe this will be available, i need to talk with @roughike

I don't see reason why not.
I'll make it available, since it's a minor change.

@roughike thanks a lot. When are your planned to release this changes?
If there is something I can do to help you, let me know.

I'd say we could make a release pretty soon, as we already have a lot of stuff lined up.

How do in bar only icons(without title)?

@Defaault please look at the readme in the main page, is the titleless feature

Must i write something in class?

Or exist some function

@yombunker Thanks! I had seen

@yombunker There's no such feature in the README...


Add Line chat in-app to detection

Github issue: https://github.com/WhichBrowser/Parser-PHP/issues/263

Coverage remained the same at 100.0% when pulling e97fdf99aa591c374f5fb6d18cafd645c2ed5476 on ayumi-cloud:line into 4899110cd2f87b01e04ced62dbb9dec541031dee on WhichBrowser:master.


Allow formatting while minimized

Again, VS Code probably doesn't allow this, but it would be great to be able to run the formatter in the background without it coming to focus for every file.

Thats currently not possible


Jdm/json backport

lol, okay, yes.  :+1:
merge this, then tag and release.
i looked into the mixlib-shellout thing and we're only pinning ~> 1.2 while rubygems is pulling in 1.6.1 even though we're on ruby 1.8.7/1.9.2 which is dumb, expected, and nothing we can do about it.  this branch supports 1.8.7/1.9.2 so we can't exclude them like mixlib-shellout 1.6.x does.
bumping rubygems in the travis config might let the tests run on the older versions of ruby.


How to run examples? cimplot cannot be accessed

Hi,
I tried running the examples but cannot compile.
rustc 1.66.0 (69f9c33d7 2022-12-12)
I cloned your repo, cd in implot-examples, and at first clance, line_plot is not present.
error: no example target named "line_plots". so I guess the readme might be up little outdated,
ls the dir and i see
d-----        18/02/2023     13:09                examples-shared
d-----        18/02/2023     13:13                implot-glium-demo
d-----        18/02/2023     13:08                implot-wgpu-demo
-a----        18/02/2023     13:08            762 README.md

So i try cargo run --example implot-glium-demo, but again error: no example target named "implot-glium-demo".
might be me not knowing how to use the --example flag. So I cd in implot-glium-demo and run cargo run.
It compiles until
error: failed to run custom build command for `implot-sys v0.6.0  [REDACTED]\implot-rs\implot-sys)`

Caused by:
  process didn't exit successfully: ` [REDACTED]\implot-rs\implot-examples\implot-glium-demo\target\debug\build\implot-sys-669e58a21423fafe\build-script-build` (exit code: 101)
  --- stderr
  thread 'main' panicked at 'Can't access third-party/cimplot/cimplot.cpp. Did you forget to fetch git submodules?', [REDACTED]\implot-rs\implot-sys\build.rs:21:13
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...

what am i missing?

hum... implot-sys\third-party\cimplot is empty, maybe i just need to install this

Ok I got it to work,
You need to clone cimplot and imgui (ithin cimplot) with the correct commit. maybe it should be mentionned in the readme. I'll give the library a try to see if it fits my need, in which case i'll try and do a PR. Maybe there's a git command to follow links and clone related repos, i'm not sure

Ok I got it to work, You need to clone cimplot and imgui (ithin cimplot) with the correct commit. maybe it should be mentionned in the readme. I'll give the library a try to see if it fits my need, in which case i'll try and do a PR. Maybe there's a git command to follow links and clone related repos, i'm not sure

Hey, Will. Do you mind giving more details on how you managed to make it work? I'm struggling in here

Basically, in GitHub, when you go to the dir for cimplot and within it for imgui you see a link to the exact commit that was used.
So clone implot the on your local machine go to the cimplot directory and clone cimplot (make sur NOT to have cimplot/cimplot) you have to checkout to the exact commit using the hash (look online for that) you repeat the same thing for the nested implot folder

This repo uses Git submodules. I.e. it references another repository at the specific commit.
You can run git submodule update --init after cloning the repo and it will automatically clone required version at the correct location.
$ git submodule update --init
Submodule 'implot-sys/third-party/cimplot' (https://github.com/cimgui/cimplot) registered for path 'implot-sys/third-party/cimplot'
Submodule path 'implot-sys/third-party/cimplot': checked out '374317a84121a01868aa5274dbcf1c5a5df96e8c'

Or like this, step-by-step:
$ git submodule status
-374317a84121a01868aa5274dbcf1c5a5df96e8c implot-sys/third-party/cimplot

$ git submodule init
Submodule 'implot-sys/third-party/cimplot' (https://github.com/cimgui/cimplot) registered for path 'implot-sys/third-party/cimplot'

$ git submodule update
Cloning into '/home/user/t/implot-rs/implot-sys/third-party/cimplot'...
Submodule path 'implot-sys/third-party/cimplot': checked out '374317a84121a01868aa5274dbcf1c5a5df96e8c'

$ git submodule status
 374317a84121a01868aa5274dbcf1c5a5df96e8c implot-sys/third-party/cimplot


Update quick start

Codecov Report

Merging #1086 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1086      +/-   ##
==========================================
- Coverage   84.23%   84.19%   -0.05%
==========================================
  Files          76       76
  Lines        2436     2436
  Branches      184      186       +2
==========================================
- Hits         2052     2051       -1
- Misses        384      385       +1

Impacted Files
Coverage Δ

...d/src/main/scala/io/circe/NonEmptySeqDecoder.scala
91.66% <0%> (-8.34%)
:arrow_down:

...re/shared/src/main/scala/io/circe/KeyDecoder.scala
94.59% <0%> (-2.71%)
:arrow_down:

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
89.75% <0%> (ø)
:arrow_up:

...re/shared/src/main/scala/io/circe/JsonNumber.scala
93.75% <0%> (+1.25%)
: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 c2a3fe3...938594e. Read the comment docs.


Implement combat melee dice logic

A player must determine when using a melee attack how many dice are used for attack. Defense dice until the end of the round are Melee skill - Attack dice = Defence dice.

Expand to full pop-up for roll modifiers. Input field vs. Checklist?


Warning and print statement in lightcurve plotting

The source light curve plotting is issuing a warning along with a print statement which should be removed.
Culprit line for the print:
https://github.com/askap-vast/vast-tools/blob/10d8b92e75854e8c5636f28447f9cb2fb638c42d/vasttools/source.py#L523
Not sure where this warning is coming from:
/Users/adam/anaconda3/envs/vast-tools-dev-poetry/lib/python3.8/site-packages/matplotlib/axes/_base.py:2283: UserWarning: Warning: converting a masked element to nan.
  xys = np.asarray(xys)

I've fixed the print statement.
I can reproduce the warning on ada but not on the hub. I haven't got a clue what's causing it though... as far as I can tell there's no masked elements?

I've fixed the print statement.
I can reproduce the warning on ada but not on the hub. I haven't got a clue what's causing it though... as far as I can tell there's no masked elements?

Different matplotlib versions? I don't remember seeing it before the recent changes with the flux support and perhaps the fix of the legend marker? I reckon it's something to do with those.

That's pretty likely. I'll take a look into it.

It's definitely https://github.com/askap-vast/vast-tools/blob/5315aac838e954f9ef2007d529227e068f907807/vasttools/source.py#L597-L604
because the warning is resolved if you replace the nans with numbers. Interestingly, the warning remains if you replace them with np.inf. I did some searching on stack overflow and github and the warning itself seems to be for any non-plotted point rather than masked elements, so that's slightly frustrating as there's no real way to plot dummy points without triggering this warning.
There's ways to add legend elements without plotting them, but they're all more complex than our current implementation. I'm tempted to just add a warning filter to ignore it. What do you think?

It's definitely
https://github.com/askap-vast/vast-tools/blob/5315aac838e954f9ef2007d529227e068f907807/vasttools/source.py#L597-L604
because the warning is resolved if you replace the nans with numbers. Interestingly, the warning remains if you replace them with np.inf. I did some searching on stack overflow and github and the warning itself seems to be for any non-plotted point rather than masked elements, so that's slightly frustrating as there's no real way to plot dummy points without triggering this warning.
There's ways to add legend elements without plotting them, but they're all more complex than our current implementation. I'm tempted to just add a warning filter to ignore it. What do you think?

How complex are we talking? Remind me, the original issue was that the error bar down arrow was not showing in the legend? Was it possible to fetch the legend object and add it manually? While the solution works it also doesn't seem ideal to plot NaN points if it is causing that warning.