 If you do go run main.go it will only load main.go and ignore anything in other files.
To run it properly, try building it first (which will include all files) with: go build -o demo and then run the newly created demo binary: ./demo.
Hope this helps.

or you can try go run *.go which will also select all .go files.

SyntaxError: missing ) after argument list[Learn More]  semantic.min.js:16:3023
console showing this.

it asks for go cam permission that i give. but nothing shows up in video player so i checked console which was showing following error.
SyntaxError: missing ) after argument list[Learn More]
semantic.min.js:16:3023

replacing following
video.src = window.URL.createObjectURL(stream);
with video.srcObject = stream;
seems to work in firefox.

Strange, so I guess that API isn’t stable yet?!
On Mon, 25 Feb 2019 at 12:09, najamsk<EMAIL_ADDRESS>wrote:

replacing following
video.src = window.URL.createObjectURL(stream);
with video.srcObject = stream;
seems to work in firefox.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/machinebox/webFaceID/issues/1#issuecomment-466988375,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGNG-6SHG3IbCRoTAkJtLPfT1EO-Hi0ks5vQ9JrgaJpZM4bPJLa
.

nope. i tried to ask co-worker to show their face on cam. since didnt train model on their face. i was expecting some error message but instead the camera icon truns into loading and keep doing loading stuff. i check console it was saying button.transition is undefined. so to solve that i change more code by showing text message. maybe run demos again in cross browsers so people can actually see them working and hop into stuff. Having said that i still enjoy browsing through demos and articles on medium.

Yeah it’s annoying that it doesn’t work in your browser. I’m sorry about
that.
They were all tested in Chrome, and are just examples so are unlikely to
get cross-browser QA testing etc like we do with our production services.
Good job for making it work though - skillz.
Mat
On Mon, 25 Feb 2019 at 12:23, najamsk<EMAIL_ADDRESS>wrote:

nope. i tried to ask co-worker to show their face on cam. since didnt
train model on their face. i was expecting some error message but instead
the camera icon truns into loading and keep doing loading stuff. i check
console it was saying button.transition is undefined. so to solve that i
change more code by showing text message. maybe run demos again in cross
browsers so people can actually see them working and hop into stuff. Having
said that i still enjoy browsing through demos and articles on medium.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/machinebox/webFaceID/issues/1#issuecomment-466992194,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGNG1w_srZNdzUMMPfYoRtwUdsuKPwQks5vQ9XLgaJpZM4bPJLa
.

i check on my co-workers machines using chrome still same errors. I think underlaying apis for html5 stuff got changed and they require minor fixes to work as expected.

Hmm, in that case, do you fancy submitting a PR with your fixes? We’d love
to improve the experience for others if we can.
On Mon, 25 Feb 2019 at 12:27, najamsk<EMAIL_ADDRESS>wrote:

i check on my co-workers machines using chrome still same errors. I think
underlaying apis for html5 stuff got changed and they require minor fixes
to work as expected.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/machinebox/webFaceID/issues/1#issuecomment-466993078,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGNG6-ozDntZ5J_AbWWVK-oIsIRMsZWks5vQ9angaJpZM4bPJLa
.

one more thing. is there any forum or channel where devs can ask your team questions or report issues. asking on twitter is not that great and engaging founder for small code stuff is like wasting your time. thanks for your time, responses and your efforts.

my pull request will be broken as its not working in chrome. but if i make it work in chrome sure.

Hi @najamsk - can you join the Machine Box slack here: https://machineboxslack.herokuapp.com/

Hi @najamsk - can you join the Machine Box slack here: https://machineboxslack.herokuapp.com/

joined the community slack, thanks for sharing. let me create a pull request.

Hmm, in that case, do you fancy submitting a PR with your fixes? We’d love to improve the experience for others if we can.
…
On Mon, 25 Feb 2019 at 12:27, najamsk @.***> wrote: i check on my co-workers machines using chrome still same errors. I think underlaying apis for html5 stuff got changed and they require minor fixes to work as expected. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AAGNG6-ozDntZ5J_AbWWVK-oIsIRMsZWks5vQ9angaJpZM4bPJLa .

pull request created. I have never contributed in open source stuff and don't know about creating pull request. by clicking few buttons i manage to create pull request that I hope somewhat close to your expectations.

you can find my pull request at https://github.com/machinebox/webFaceID/pull/2


Doc references should be updated

We used to have the elemental docs at https://rancher.github.io/elemental [1].
The docs have been migrated to https://elemental.docs.rancher.com/ [2].
While there is a redirect from [1] to [2], links meant for specific documentation section are forward to the main page of the new docs.

I checked all links and all good now, thx


ValueError: Amount 0.65 is not a multiple of the quantum size 1e-09

I'm trying to place an order for ETH-USD pair with amount of 0.65 ETH, but it shows this error message:
ValueError: Amount 0.65 is not a multiple of the quantum size 1e-09
I can't understand what this means? would you please clarify & show me a solution?
FYI, I get this amount based on some calculations of my open position & market data. So I need a way to open or close a position with exactly the amount returned by the calculations, not even 0.1% more or less.

I got the same error. Have you found a way to solve this problem?

Amount 0.03 is not a multiple of the quantum size 1e-09 is my error
So computers can't count when adding or subtracting numbers are less than one and have decimal places 😁. On the python interpreter prompt type the following.
>>> 1.2 - 1.0
0.19999999999999996

So my work-around is to convert my figures into int then do my calculation and convert back to float. I do this by having a constant J which is a large arbitrary number
J =<PHONE_NUMBER>0
Then my math includes J to convert to and from float to int
    totalSize += int(orderSize * J) / J
    totalCash += (int(orderSize * J) * orderPrice) / J

Basically get rid of the decimals by multiplying, do the math then divide to bring the result back to what you need. The error doesn't show because the string seems to be truncated in the code.


create XSLT Transform endpoint

This endpoint should transform (xml) content via xslt style sheets. These style sheets are saved in the FREME database.

Create style sheet management endpoints: create, update, delete, get

use a (descriptive) name as id
test it

Create transformation endpoint

use the java lib Saxon-HE as XSLT processor
check, if newest Saxon-HE supports html input/parsing:

use [validator.nu]: nu.validator.htmlparser.tools.HTML2XML from this lib
otherwise use this Saxon-HE version

test stylesheet, test input: xml-to-rdf-tutorial --> TEI P5 document (load a sample)
write tests

document all new functionality

add endpoints to the api-doc
create a knowledge-base/tutorial (?) article

@fsasaki anything to add?

Looks good, thanks!

Looks good! Please name the new basic service "xslt-converter". The API endpoints should be /toolbox/xslt-converter/documents/{converter-name}, /toolbox/xslt-converter/manage, ...

The transformation endpoint is implemented. Please try:
curl -X POST -H "Content-Type: text/xml" -d '
<TEI xmlns="http://www.tei-c.org/ns/1.0" xmlns:exist="http://exist.sourceforge.net/NS/exist" version="5.0">
<teiHeader xmlns="http://www.tei-c.org/ns/1.0">
<fileDesc> <titleStmt>
<title>From XML to RDF step by step: Approaches for Leveraging XML Workflows with Linked Data</title>
</titleStmt><publicationStmt>

Published for XML Prague
</publicationStmt>
<sourceDesc>
Made by the FREME project and collaborators

</sourceDesc></fileDesc></teiHeader><text>

We very much welcome you in the city of Prague, a home of XML!

</text></TEI>' "http://api-dev.freme-project.eu/current/toolbox/xslt-converter/documents/tei2temp-html"

Note: for any curl call the Content-Type header has to be set. Otherwise Curl defaults to application/x-www-form-urlencoded which causes url encoding of the body.
 Again, use Content-Type: text/html. And a valid token.
If content-type=text/html, nu.validator.htmlparser.sax.HtmlParser will be used to parse the data. But I did not test any html input. @fsasaki do you have examples I could try?

Great! One question: you wrote

This call uploads the tei2temp-html stylesheet. Again, use Content-Type: text/html. And a valid token.

Why do you use the content type text/html? The stylesheet also has an XML content type.
On

If content-type=text/html, nu.validator.htmlparser.sax.HtmlParser will be used to parse the data.

This would not be needed for the stylesheet upload but the HTML input.
I will provide the other stuff later.
Thanks again!

Why do you use the content type text/html?

Oh, my mistake, I meant text/xml.

This would not be needed for the stylesheet upload but the HTML input.

For sure, I meant the transformation functionality.

Thanks. Attached is a stylesheet that could be used to process HTML input with XML output.
html5-preprocessing.txt
Also attached is a build file that shows how the saxon xslt processor is used, see the target "preprocess-html5", to process HTML5 input. Does that help?
html5-preprocessing.txt

@fsasaki thanks! But it looks like the first file was attached twice.

ah, sorry! Here is the second file, the built file.
build.txt

One more question: how could one submit a parameter? See
http://www.saxonica.com/documentation/#!using-xsl/commandline
2016-07-08 11:00 GMT+02:00 ArneBinder<EMAIL_ADDRESS>
@fsasaki https://github.com/fsasaki thanks! But it looks like the first
file was attached twice.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/freme-project/basic-services/issues/51#issuecomment-231310336,
or mute the thread
https://github.com/notifications/unsubscribe/ABH5ApHQ4557-v-GuNXaZvoC-PtOMibJks5qThGugaJpZM4I0KQz
.

check html input

@fsasaki I tried html (with the identity-transformation stylesheet you uploaded) and it works as expected:
curl -X POST -H "Content-Type: text/html" -d '<!doctype html>

  <meta charset="utf-8">

  <title>The HTML5 Herald</title>
  <meta name="description" content="The HTML5 Herald">
  <meta name="author" content="SitePoint">

hello world

' "http://api-dev.freme-project.eu/current/toolbox/xslt-converter/documents/identity-transformation"

result:
<?xml version="1.0" encoding="UTF-8"?>

        <meta charset="utf-8"/>
        <title>The HTML5 Herald</title>
        <meta name="description" content="The HTML5 Herald"/>
        <meta name="author" content="SitePoint"/>

hello world

Furthermore, if you set the header Accept: text/html you get:

        <meta http-equiv="Content-Type" content="text/html; charset=UTF-8"></meta>
        <meta charset="utf-8"></meta>
        <title>The HTML5 Herald</title>
        <meta name="description" content="The HTML5 Herald"></meta>
        <meta name="author" content="SitePoint"></meta>

hello world

Very nice! It works with other stylesheets too, see
http://api-dev.freme-project.eu/current/toolbox/xslt-converter/manage
I did not yet try the submission of parameters, see my previous question in
this issue.
2016-07-13 9:17 GMT+02:00 ArneBinder<EMAIL_ADDRESS>
check html input
@fsasaki https://github.com/fsasaki I tried html (with the
identity-transformation stylesheet you uploaded) and it works as expected:
curl -X POST -H "Content-Type: text/html" -d '<!doctype html>

The HTML5 Herald

hello world

' "http://api-dev.freme-project.eu/current/toolbox/xslt-converter/documents/identity-transformation"
result:

The HTML5 Herald

hello world

Furthermore, if you set the header Accept: text/html you get:

The HTML5 Herald

hello world

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/freme-project/basic-services/issues/51#issuecomment-232275592,
or mute the thread
https://github.com/notifications/unsubscribe/ABH5Apx8PSqiu9qfBz9XkkYmbDUHwdeKks5qVJEHgaJpZM4I0KQz
.

submission of parameters

What kind of parameters you want to submit? Paramters like this:

A param takes the form name=value, name being the name of the parameter, and value the value of the parameter. These parameters are accessible within the stylesheet as normal variables, using the $name syntax, provided they are declared using a top-level xsl:param element.

This isnt yet implemented. Should I do this?

Yes, parameters as name value pairs, that would be great to have as a
feature.
2016-07-13 10:02 GMT+02:00 ArneBinder<EMAIL_ADDRESS>
submission of parameters
What kind of parameters you want to submit? Paramters like this:
A param takes the form name=value, name being the name of the parameter,
and value the value of the parameter. These parameters are accessible
within the stylesheet as normal variables, using the $name syntax, provided
they are declared using a top-level xsl:param element.
This isnt yet implemented. Should I do this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/freme-project/basic-services/issues/51#issuecomment-232284962,
or mute the thread
https://github.com/notifications/unsubscribe/ABH5AjGuP-8cwGShtXiliWwfXmiBV2j8ks5qVJuLgaJpZM4I0KQz
.

Yes, parameters as name value pairs, that would be great to have as a feature.

Ok, I will implement this.
Added the xlt-converter endpoints to the api doc.

I created
http://api-dev.freme-project.eu/current/toolbox/xslt-converter/documents/xslt-with-param
if there is no external parameter given, the output is

set internally

With the attached curl request I can process a well-formed (X)HTML document. But if I set the content type to text/html, I get the error

{
"exception": "eu.freme.common.exception.FREMEHttpException",
"path": "/toolbox/xslt-converter/documents/html-to-xliff20",
"message": "The XML parser reported one validation error",
"error": "Internal Server Error",
"status": 500,
"timestamp":<PHONE_NUMBER>526
}

Any idea why this is the case?
xslt-curl-request-with-html-input.txt

I created
http://api-dev.freme-project.eu/current/toolbox/xslt-converter/manage/xslt-with-param
if there is no external parameter given, the output is

set internally

@fsasaki does this work for you as expected when you pass some value to myparam? At the moment it is assumed that parameters are atomic values, is this sufficient?

With the attached curl request I can process a well-formed (X)HTML document. But if I set the content type to text/html, I get the error

I will examine what's wrong here.

With the attached curl request I can process a well-formed (X)HTML document. But if I set the content type to text/html, I get the error

Two problem:

the parser needs <!DOCTYPE html> in the input
the transformer has a problem with this: fn:parse-xml(//*[local-name()='script' and @id='orgdoc']) in the stylesheet

The parameter works, see this request.
curl -X POST -H "Content-Type: text/xml" -d '' "
http://api-dev.freme-project.eu/current/toolbox/xslt-converter/documents/xslt-with-param?myparam=set
externally"
2016-07-18 19:28 GMT+02:00 ArneBinder<EMAIL_ADDRESS>
I created
http://api-dev.freme-project.eu/current/toolbox/xslt-converter/manage/xslt-with-param
if there is no external parameter given, the output is
set internally
@fsasaki https://github.com/fsasaki does this work for you as expected
when you pass some value to myparam? At the moment it is assumed that
parameters are atomic values, is this sufficient?
With the attached curl request I can process a well-formed (X)HTML
document. But if I set the content type to text/html, I get the error
I will examine what's wrong here.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/freme-project/basic-services/issues/51#issuecomment-233398353,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABH5AvLKc5iyZujynPjMWOpf88NlZK-Mks5qW7ergaJpZM4I0KQz
.

Maybe we should open a separate issue on this?
On the second problem: I update the stylesheet to say
fn:parse-xml(//*[local-name()='script']), it works now with text/xml, but
still not with text/html even with the DOCTYPE html statement. See attached
request.
2016-07-18 19:45 GMT+02:00 ArneBinder<EMAIL_ADDRESS>
With the attached curl request I can process a well-formed (X)HTML
document. But if I set the content type to text/html, I get the error
Two problem:

the parser needs  in the input
the transformer has a problem with this: fn:parse-xml(//*[local-name()='script'
and @id='orgdoc']) in the stylesheet

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/freme-project/basic-services/issues/51#issuecomment-233403270,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABH5AgM_VF0THAj3VWHPwiM95SFS8j4_ks5qW7vDgaJpZM4I0KQz
.

curl -X POST -H "Content-Type: text/html" -d '

@@@

We very much welcome you in the city of Prag
            Praguehome of XML!

' "http://api-dev.freme-project.eu/current/toolbox/xslt-converter/documents/html-to-xliff20"

Maybe we should open a separate issue on this?

I opened https://github.com/freme-project/basic-services/issues/65

@ArneBinder please close this issue and open new issues for tasks that are still open from this issue.

Implemented management and transformation tests. The open documentation task is moved to https://github.com/freme-project/Documentation/issues/272.


Fix Bottom Sheet linear animations

Boy, this was a hard one :sweat_smile:  This PR eases the enter and exit transitions for bottom sheets, while also preserving regular dragging behavior.
TODO:

[x] showModalBottomSheet
[x] showBottomSheet

Same comment here as on the other one - this has had no activity in some time now, needs tests, and needs to be synchronized up to HEAD.  Please feel free to re-open once some of that has been addressed.


Demuxer status

Hi guys.
This repo is very important initiative given the state of the web media capabilities ))
I have got a question. What's the status of demuxing functionality? What's missing? Maybe I can help,because I an in search for a fast and minimal front end API for demuxing.
Thanks.

I have left this up as a proof-of-concept but no immediate plans to revive it until WebCodecs API reaches wider browser support.
See mp4-muxer which is more actively maintained.


vscode not adhering to my syntax highlighting in settings.json unless I open the file and edit one of the colors

VSCode Version:
OS Version:

Steps to Reproduce:

Open any rermote project
Syntax coloring is the default, all other settings appear to be ok
edit "editor.background": "#000000", by removing a zero, then saving (screen turns red) then re-adding the zero and saving
syntax highlighting is enabled

Does this issue occur when all extensions are disabled?: Yes/No
Not locally, but with only remote ssh plugin enabled, it occurs when I connect to the remote host with all other plugins disabled.  It does this on multiple AWS server/project roots.

What theme do you have configured? Is it set in the user settings or the (remote) workspace settings?
(maybe a duplicate of #101433)

Default Dark+ how do I tell where it's set? It isn't in my settings.json, but when I tab through preferences "Workbench: Color Theme", it shows up on
It does not say "Modified in..." on any of the tabs.
"Color customization" says "modified in workspace"

I tried changing the workspace config to "Visual Studio Dark" so that it weas different from "User", but that didn't help.

You use the default theme (otherwise it would have a blue bar next to the settings).

`edit "editor.background": "#000000", by removing a zero, then saving (screen turns red) then re-adding the zero and saving
Where did you do this? User settings or workspace settings?

To open the user settings JSON, run Preferences: Open User Settings (JSON)
To open the workspace settings JSON, run Preferences: Open Workspace Settings (JSON)

only settings i ever usually edit are in workspace .vscode/settings.json

Ok, that's the workspace settings.
The bug that you are seeing is https://github.com/microsoft/vscode/issues/101433. It's fixed in the latest insiders.

I installed 1.47.0-insider and the problem did not go away. I even uninstalled and reinstalled everything clean.

Ok, sorry for that, the commit got delayed, bit is in now. Please try with Monday's update.


auro-popover: popover content overflows border

Describe the bug
Popover used within a parent that has 'nowrap' set causes popover content to overflow popover borders.
To Reproduce
Steps to reproduce the behavior:

Insert latest popover version with a large chunk of text as content into a parent element that has 'no wrap' set on it.
On hover - content Text overflows the popover background and border on the right side.

Expected behavior
Popover box is large enough to contain the content text OR Popover box keeps content from being larger than box.
Screenshots

Desktop (please complete the following information):

OS: macOS Big Sur 11.2.3
Browser: Chrome  91.0.4472.114
Popover 3.1.1

Additional context
Problem seems to resolve when percentage based max- width for popover are removed OR when the nowrap on the parent element is removed.

The issue is that in a previous release we added a max-width to the popover content. However, we did not add any CSS rules that would force the slot content to render with the same max-width.
Anything that has a rendering baseline larger than the popover's content max-width will fail. This means that long strings with the rule 'whitespace: nowrap;', images with a width that is wider, tables with wide/many columns, etc. will overflow the popover content.
My initial thought is that the slot content CSS should have the same max-width or `width: 100%' rule(s).
.popover {
  ::slotted {
    width: 100%;
  }
}

Doing so should make the slotted content render correctly for clipping/whitespace wrapping and prevent the overflow demonstrated in the original ticket.


Nav2 and Battery issues

I am trying to run Nav2. I get the error below. How to fix this error? The robot does not show up in Rviz and I cannot connect it to the controller.
...
controller_server-3] [INFO] [1684982046.230401850] [local_costmap.local_costmap]: Timed out waiting for transform from base_link to odom to become available, tf error: Could not find a connection between 'odom' and 'base_link' because they are not part of the same tree.Tf has two or more unconnected trees.
[sync_slam_toolbox_node-2] [INFO] [1684982046.334276758] [slam_toolbox]: Message Filter dropping message: frame 'rplidar_link' at time<PHONE_NUMBER>.200 for reason 'discarding message because the queue is full'
[sync_slam_toolbox_node-2] [INFO] [1684982046.458095791] [slam_toolbox]: Message Filter dropping message: frame 'rplidar_link' at time<PHONE_NUMBER>.335 for reason 'discarding message because the queue is full'
[sync_slam_toolbox_node-2] [INFO] [1684982046.592352981] [slam_toolbox]: Message Filter dropping message: frame 'rplidar_link' at time<PHONE_NUMBER>.458 for reason 'discarding message because the queue is full'

In addition to this error, I have some issues with the battery. The robot had a red light, and it did not want to work (e.g., navigate or show in Rviz). I cannot recall if I had the same error as above when I had this firt battery issue. To solve this battery issue, I changed the battery. Initially, this battery was charging. It went red again, after some days. I changed the battery again (third time), but the robot does not want to charge now. The screen keeps showing 0% for the battery (while in the docking station for hours).  When I try to echo the topic for the battery it returns nothing. After some hours the red light is red.
How to fix this issue?  are they related?

@nidia-bucarelli The issues are likely related. It looks like the RPi4 and Create 3 are not communicating.
Can you specify what ROS2 version you are using, as well as whether you are using simple discovery or the discovery server?

I am using ROS2 Galactic.
I am not sure about the discovery. How do I check for that?

If you haven't gone through these instructions then we can assume you are on simple discovery. Have you connected your Create 3 to your Wi-Fi network? Also what is the output of ros2 topic list when called from the RPi4?

@nidia-bucarelli Check that the RPi is powering on inside the robot. There should be one red and one yellow LED on near the USB-C connector.
If you have a micro HDMI cable you can connect it to one of the HDMI ports on the Raspberry Pi. Then connect the other end to a monitor. Power on the RPi after connecting these cables and with a USB keyboard you should be able to log in.

I think it is not powering on inside the robot. I don't see a light near the USB connector.

As per the micro HDMI cable, I cannot see the port. Should I remove the case/cover of the robot? Or should I use the HDMI ports on the edges (on the cover)?

I would recommend removing the top plate and standoffs so that you can look under the PCBA. Instructions are here.
The four ports on the PCBA are USB-C ports, and I don't believe you can connect a monitor to the RPi4 through that (though I haven't tested). The HDMI ports are labelled in this image:

I was able to remove the PCBA. I unplugged all cables, except the cable that gives power (that comes from the base). The base is ON. But the pi4 is not ON.
I tried to change the power supply for the Pi4 using another source of power, and it worked.
any suggestions?

@nidia-bucarelli Can you check that the Create 3 Adapter board is fully inserted into the Create 3, and that the USB-C cable is connected properly? If neither of those is an issue, can you try using a different USB-C to USB-C cable if you have one?

I checked the connection, and It still does not want to stay on. It only last 2-3 seconds.  I tested a different USB-C cable (to connect from the pi to the adapter). I also changed the adapter using the one from another robot I have.
Also, in case this is important.... the base is only ON when it is in the docking station. If I remove the base from the docking station, the base goes off.  The battery is also new. The one I had before lasted 2 days, before the robot going to red light.

It sounds like your battery has been fully discharged and is unable to charge again. Can you try the following:

Remove the battery for 15 minutes
Remove the adapter board
Insert the battery back in and place the robot back on the dock.
Allow it to charge fully with the adapter board removed.
Reconnect the adapter board once it is charged.

My p4 is ON now.
But the LED screen always shows that the battery is 0%.  But, the robot is not docked and is still ON. For example, it's been 2 hours or so, showing 0% and still ON (without charging).

When I do:
ros2 topic echo /diagnostics_agg
I get:

level: "\x02"
name: '/Turtlebot4/Create3/Battery/turtlebot4_diagnostics: Battery Percentage'
message: Critical
hardware_id: Turtlebot4
values:

key: Battery Percentage
value: '0.00'

When I do:
ros2 topic echo /battery_state
I don't get anything.

@nidia-bucarelli The comms LED is off which indicates that the RPi4 cannot communicate with the Create 3. Has the Create 3 been connected to Wi-Fi? Can you check the webserver to make sure that there is an IP address showing in the Connect page?

Yes. It has been connected to Wi-Fi. I also checked the webserver and there is an IP address.
These are the topics that my computer can see:
I can see my topics from my computer as well.

/base/battery_state
/base/cmd_audio
/base/cmd_lightring
/base/cmd_vel
/base/dock
/base/hazard_detection
/base/imu
/base/interface_buttons
/base/ir_intensity
/base/ir_opcode
/base/kidnap_status
/base/mouse
/base/odom
/base/robot_state/transition_event
/base/slip_status
/base/static_transform/transition_event
/base/stop_status
/base/system_monitor/transition_event
/base/tf
/base/tf_static
/base/wheel_status
/base/wheel_ticks
/base/wheel_vels
/battery_state
/cmd_vel
/color/preview/camera_info
/color/preview/image
/diagnostics
/diagnostics_agg
/diagnostics_toplevel_state
/dock
/hazard_detection
/hmi/buttons
/hmi/display
/hmi/display/message
/hmi/led
/imu
/interface_buttons
/ip
/joint_states
/joy
/joy/set_feedback
/mouse
/parameter_events
/robot_description
/rosout
/scan
/stereo/camera_info
/stereo/depth
/tf
/tf_static
/wheel_status

These are the topics i can see from the Pi4:
/base/interface_buttons
/base/robot_state/transition_event
/base/static_transform/transition_event
/base/system_monitor/transition_event
/parameter_events
/rosout

It looks like you have namespaced the Create 3 to base. Namespacing isn't supported in Galactic, so please change it back to an empty string in the webserver.

It is working now. Thanks much!


Install jest and react-testing library and create first jest tests

Resolves #450.
First in a series of adding basic test coverage. Started with snapshot tests since they're easy to set up.
Setting up jest to play nice with webpack was kind of a pain though, so hopefully this paves the way for devs to write more tests!
Next steps are to figure out how to incorporate the tests into github's CI checks and write some reusable mocking utilities, but those are out of scope of this PR.

LGTM Very nice start to testing!! Can you document/link to this in our testing wiki so people can use it as a template? https://github.com/waterthetrees/waterthetrees/wiki/Testing Thanks!

I was thinking of including a readme. Should I add to the wiki instead? or both?
Also, I took this PR into draft mode since I need to fix a bug. Snapshots break each time due to mui components adding a randomly generated identifier to their css classes 🙄

I was forced to upgrade @mui/material a few minor versions in order to get the jest snapshots to be somewhat stable. This probably means we'll want to test on dev.waterthetrees.com for a while after we merge to snuff out any unintended changes I might miss.

The action for wiping the node_modules doesn't run every time a merge goes to main. There's a special action for that.
Oh wait, that's right. @mui/material is pinned with "^" to the latest minor version, so it should already have been updated. I'm a little confused, but I guess I haven't updated my dependencies in a while or something. Checking @mui/material's github, it looks like there have been quite a few releases in the last 2 months, so that might check out...

In any case, I can't repro the instability that I was experiencing before, so I think this should be ready to merge. If anyone runs into any trouble with the snapshot tests complaining right after merging, it should be fixable by wiping node_modules and npm install.
I'll update the testing wiki for some more info on snapshot tests sometime later. Tldr; we should expect them to be somewhat brittle compared to other types of tests, but it's really easy to update the snapshots. I'll include how to look through the diffs and the updateSnapshot command.

What should the testing structure be? Should we have all the tests in the tests folder and then do we also need to add a snap or is that created automatically?

Yeah, I was going to dump all tests there in that tests folder and group into subfolders as we add more tests. Didn't really give it much thought.
The snap files are created automatically by running npm tests. I'll include these in the docs and more importantly how to update the tests if the tests are broken. I thought I did this already...

What should go in fileMock vs styleMock?
Ooh, I wasn't familiar with this separation, but that's definitely what I want to tackle next in terms of making the mocking functions reusable and easy to find. I'll need to work on this

how are the test scripts different? test:coverage vs test?
test:coverage should show the coverage of the front-end code by % as calculated by Jest. But it looks like npm test basically does the same thing currently

I'm updating the testing documentation for front-end now!
Other than that, the other action item is to make the mocking logic reusable and organize it into fileMock and styleMock
And after that, I want to hit a couple integration tests before I move on to working on other things.

What should go in fileMock vs styleMock?

D'oh, right, these files were in my PR but I completely forgot. I'm 90% sure we should not touch either of these files. These exist for the sole purpose to make sure styles (any .css, etc imports) and files (any .jpg, etc imports) don't break the tests.
See: https://jestjs.io/docs/webpack#handling-static-assets


add format skills into method instead

if isinstance(List(str)):
format_skills()

logger.warning about what you've passed in map_skills
update documentation to reflect the different types map_skills takes now

@india-kerle I think this can be closed?

yup, closing now!


Read version number from package.json

When you run the token-engine, it prints out the package version you're using (src/utils/title.ts const version), read that version from the package.json file to keep it updated

Bug: the script shows the version of the package.json who is running the package, not of the package itself.

Solved in #52


sort requirements on info body response

@segiddins merging this will actually break some of the info checksums. May I release a new version for this?

Should be fine?

I think will be fine, but I tought was better to tell you because of compact_index_client...

Nah, the client doesn't care about the order.


Remove last remnant of KokkosKernels left from #29

Codecov Report

Merging #31 (6f9696e) into master (487020f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #31   +/-   ##
=======================================
  Coverage   92.86%   92.86%
=======================================
  Files          43       43
  Lines        4079     4079
=======================================
  Hits         3788     3788
  Misses        291      291

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 487020f...6f9696e. Read the comment docs.


Remove layout_array and telescope positions warning

Every initialization of ArraySimulator or ShowerSimulator results in a couple of lines of warnings related to layout_array and telescope_positions. Very likely this is not useful / relevant.
e.g.,
WARNING::layout_array(l578)::_getCrsLocal::crs_local cannot be built because center lon and lat are missing
WARNING::layout_array(l590)::_getCrsUtm::crs_utm cannot be built because center lon and lat are missing
WARNING::telescope_position(l612)::convertAll::crsLocal is None - conversions will be impacted
WARNING::telescope_position(l614)::convertAll::crsUtm is None - conversions will be impacted
WARNING::telescope_position(l650)::convertAll::Warning: telescope height might be incorrect du to incomplete CORSIKA observation (None)or sphere centre information (None)
INFO::layout_array(l298)::readTelescopeListFile::Reading telescope list from /afs/ifh.de/group/cta/scratch/maierg/CTA/MC-Pipeline/gammasim-tools/data/layout/telescope_positions-North-Prod5.ecsv
INFO::corsika_config(l499)::_setOutputFileAndDirectory::Creating directory /afs/ifh.de/group/cta/scratch/maierg/CTA/MC-Pipeline/gammasim-tools/simtools-output/test-production/corsika, if needed.
WARNING::layout_array(l578)::_getCrsLocal::crs_local cannot be built because center lon and lat are missing
WARNING::layout_array(l590)::_getCrsUtm::crs_utm cannot be built because center lon and lat are missing
WARNING::telescope_position(l612)::convertAll::crsLocal is None - conversions will be impacted
WARNING::telescope_position(l614)::convertAll::crsUtm is None - conversions will be impacted
WARNING::telescope_position(l650)::convertAll::Warning: telescope height might be incorrect du to incomplete CORSIKA observation (None)or sphere centre information (None)
INFO::layout_array(l298)::readTelescopeListFile::Reading telescope list from /afs/ifh.de/group/cta/scratch/maierg/CTA/MC-Pipeline/gammasim-tools/data/layout/telescope_positions-North-Prod5.ecsv

Does #323 mean this is resolved and can be closed?

Fixed with #323


Template doesn't include link to line (/line/XYZ) for 'View'

Template doesn't include link to line (/line/XYZ) for 'View'.

Originally posted by @bhajneet in https://github.com/ShabadOS/database-viewer/pull/57#issuecomment-502706211

Would you prefer a redirect from that URL to the new one? Or can just modify it


run cargo diet to reduce crate size by 85%

I noticed the crate is pretty big and took a quick look, here
is the outcome of slimming it down.
➜  termscp git:(main) cargo diet
┌───────────────────────────────────────────┬─────────────┐
│ File                                      │ Size (Byte) │
├───────────────────────────────────────────┼─────────────┤
│ codecov.yml                               │          96 │
│ .github/ISSUE_TEMPLATE/feature_request.md │         203 │
│ dist/deb.sh                               │         210 │
│ .github/workflows/macos.yml               │         319 │
│ .github/workflows/windows.yml             │         321 │
│ dist/rpm.sh                               │         336 │
│ dist/build/README.md                      │         345 │
│ .gitignore                                │         356 │
│ dist/build/x86_64/Dockerfile              │         509 │
│ dist/pkgs/arch/.SRCINFO                   │         511 │
│ .github/workflows/aur-pub.yml             │         570 │
│ dist/pkgs/arch/PKGBUILD                   │         580 │
│ .github/ISSUE_TEMPLATE/bug_report.md      │         598 │
│ dist/build/x86_64_archlinux/Dockerfile    │         905 │
│ .github/workflows/linux.yml               │        1013 │
│ .github/PULL_REQUEST_TEMPLATE.md          │        1093 │
│ dist/build/deploy.sh                      │        1291 │
│ docs/drawio/UI.drawio                     │        1993 │
│ CODE_OF_CONDUCT.md                        │        3368 │
│ CONTRIBUTING.md                           │       10756 │
│ assets/images/bookmarks.gif               │      298453 │
│ assets/images/auth.gif                    │      321769 │
│ assets/images/explorer.gif                │      650583 │
│ assets/images/config.gif                  │      705780 │
│ assets/images/text-editor.gif             │     1898000 │
└───────────────────────────────────────────┴─────────────┘
Saved 85% or 3.9 MB in 25 files (of 4.6 MB and 75 files in entire crate)

Please let me know if you would like some other files to be included
or whatever else is needed to make this PR mergeable.
Thanks :).

On another note, when skimming the README I saw the passage about storing encrypted secrets locally.
Maybe the keyring would be something you are interested in to make improvements in that realm.

On another note, when skimming the README I saw the passage about storing encrypted secrets locally.
Maybe the keyring would be something you are interested in to make improvements in that realm.

Thank you for your pull request, this is great. I must admit my ignorance, I didn't know that cargo included the all repository, I thought it was already smart enough to include only the necessary stuff.
I merged it; if you have any other suggestion/issue, don't hesitate to report to me.
Thank you also for letting me know about "keyring", I'm going to add it to my todo list.

Thank you for your pull request, this is great. I must admit my ignorance, I didn't know that cargo included the all repository, I thought it was already smart enough to include only the necessary stuff.
I merged it; if you have any other suggestion/issue, don't hesitate to report to me.
Thank you also for letting me know about "keyring", I'm going to add it to my todo list.


Add template and example for uv3d calcs on azure batch

It is possible that the basic centos7 template works for this as well. However this issue is to confirm if a template is needed and create one. Finally update the docs to show how to submit the batch job on Azure
Depends upon https://github.com/CADWRDeltaModeling/BayDeltaSCHISM/issues/36

@dwr-psandhu It should. In this case, we only need interpolate_variable that works on a template.

@kjnam Then i'll check in an example in docs for doing so using existing template.


Compile failure on Windows

Attempting to compile this project on Windows as a SPM dependency yields the following compiler errors. It seems to be in the C++ parts of BoringSSL
In file included from C:\SourceCache\AutoML\desktop_client\shared\automl_core\.build\checkouts\swift-nio-ssl\Sources\CNIOBoringSSL\include/CNIOBoringSSL_base.h:478:
In file included from C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\include\memory:11:
In file included from C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\include\exception:12:
In file included from C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\include\type_traits:11:
C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\include\xstddef:283:22: error: 'auto' return without trailing return type; deduced return types are a C++14 extension
_NODISCARD constexpr auto _Unfancy(_Ptrty _Ptr) noexcept { // converts from a fancy pointer to a plain pointer
                     ^
In file included from C:\SourceCache\AutoML\desktop_client\shared\automl_core\.build\checkouts\swift-nio-ssl\Sources\CNIOBoringSSL\ssl\tls_record.cc:109:
In file included from C:\SourceCache\AutoML\desktop_client\shared\automl_core\.build\checkouts\swift-nio-ssl\Sources\CNIOBoringSSL\include\CNIOBoringSSL_ssl.h:145:
In file included from C:\SourceCache\AutoML\desktop_client\shared\automl_core\.build\checkouts\swift-nio-ssl\Sources\CNIOBoringSSL\include/CNIOBoringSSL_base.h:478:
In file included from C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\include\memory:15:
In file included from C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\include\xmemory:16:
C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\include\xutility:192:15: error: deduced return types are a C++14 extension
    constexpr decltype(auto) operator()(_Args&&...

Yup, this project hasn't been tested on Windows yet. There are two reasons for that: firstly, I can't get SwiftPM working on Windows, and secondly, NIO doesn't support Windows yet, so the dependent projects can hardly do that either.
I'm not going to keep this open: we'll get to this project once we get NIO working on Windows.

@Lukasa the SPM is working on the latest builds from Saleem here https://github.com/compnerd/swift-build. This is somewhat of a need to get this working, please advise if there's any help I can offer to move this along, including modifications to the NIO project

Yup, I'm aware that the latest builds have a working SPM. I appreciate I didn't emphasise it much, but my wording was kinda precise: I personally can't get SPM working. That is to say, I have a Windows machine with the Swift toolchain on it, and SPM hangs indefinitely.
Regarding modifications to the NIO project, @compnerd is actively involved in making changes there. We are expediting reviews of those changes as needed. My recommendation is that you sync up with @compnerd to work out what places still need work.

Ahh ok thanks. I actually had that exact same issue with SPM, I just wiped my entire machine, reinstalled Windows, VS2019, and Swift and it took care of the issues. Saleem seemed to think there was some race condition causing this problem. I realize that a full wipe and reinstall is not ideal, but is at least a solution. I'll sync up with him

Ah, yeah, I think I'd like to try to work out what the actual problem is and fix it. Not least because my Windows machine is my personal machine and I'd really rather not download Flight Simulator 2020 again! 🛩


maintainers: fix alphabetical sorting

Motivation for this change
Not all handles are in alphabetical order.
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Thanks


Add option category to BB processing script

This patch adds an option category to the BB processing script, which separates the options specifically for the script from the other options that end up showing on the list from around LLVM. This significantly improves the readability of the help page, at least when looking for options specifically related to the script (which is essentially all the time).

Note that this is stacked on #90.


Write file with extension .t.sol rather than .sol

As is usually expected in the case of forge tests :)

Thank you for opening an issue, good ser!

Write file with extension .t.sol rather than .sol

Yes! This is a corollary of https://github.com/alexfertel/bulloak/issues/1. Since every .tree file starts with an output filename, .t.sol can be specified there. This way, we don't force a convention onto the user, who can freely choose their pain.
Closing because https://github.com/alexfertel/bulloak/issues/1 solves this!

@jparklev https://github.com/alexfertel/bulloak/issues/1#issuecomment-1673895532

What good service, thank u mr alex


payment method

That's because I removed the edits I had on example.env file

It's just two files that are needed.


High Security Risk - Code Helper (Renderer) wants to use your confidential information

The problem
Current Keychain request dialog box poses significant Azure Security risk because it does not  display the name of initiating extension
Current Keychain request dialog box poses a significant Azure Security risk to any Visual Studio Code user that uses Azure because it does not display the name of the extension that is requested access to the keychain allowing malicious extension writers to gain elevated access to Azure credentials.
The prominent error message is "
When an extension requests keychain access, the name of the exact extension is not displayed prominently in the dialog message.
Example (steps to reproduce)
This problem has been around for many months, if not perhaps even for a few years.
steps

open visual studio code on mac
disable all non Microsoft extensions
add in and enable Microsoft VSCode extension "Azure Pipelines", do not enter any passwords.
log out of azure, and remove any azure cached credentials. (Yes I know that makes this quite an annoying bug to test, but given the high risk, and ease of abuse, ...)

close visual studio code
now open a non development text file somewhere on your local drive that has nothing to do with Azure, e.g. /foo/notes.txt
observe dialog box

Notes

This problem is not limited to only "Azure Pipelines".
the problem will only occur when certain cached credentials are no longer cached. I do not know the full list of what credentials, because Apple's dialog does not describe what information is being requested, just the broad category, "VS Code Azure". (perhaps that is enough, I do not know).

I know that there are technical reasons why this is is a difficult problem to be solved, however, I believe that the risks posed far exceed the level of technical difficult, and therefore justify the effort required to find an appropriate solution.
Some random ideas to get you started

collaborate with Apple to allow for some type of code signing with regards to Keychain requests, so that the name of a calling application can be sufficiently detailed, prefixed or suffixed to determine the exact source of a request. (or something similar)

or
dynamically create a requesting class in javascript that is named according to the extention, that then becomes the code helper, so that the error messages reads something like "Code Helper (Render) + Azure Pipelines extensions wants to use your ...."
etc, etc.

Possible acceptance criteria
I don't claim to know Microsoft's business, but would like to suggest that the acceptance criteria for an emergency fix could be something like

some change that makes it difficult for a malicious extension writer to pretend to be an Azure extension.
reduced the risk of a user accidentally allowing access to Azure Keychain
increases transparency around what extension is making a request
better QA around existing Microsoft extensions so that this pop up dialog never occurs without the user doing a specific activity from a known interface belonging to the extentions, eg. Command pallete.


Bau add frontend translations for demo

Proposed changes

BAU Add translations for frontend demo account
What changed

Added translation's for demo account
Why did it change
New environment demo requires translations. Backend has been configured already.
Checklists

Environment variables or secrets

[x] No environment variables or secrets were added or changed

Hey Latif, not sure if we need this as there is currently no frontend on demo, or is it breaking something?

Not breaking anything - but noticed we add 6 translations per file on the frontend and 7 on the backend so thought it would be worth making it consistent

not currently needed - closing PR.


fixed changing value on changing props of the ReactTelephoneInput

this PR fixes issue #66

@yktv4 Can you please add tests for the PR?

@anilreddykatta added test for changing the value prop and fixed the shouldComponentUpdate callback. is that okay?

@yktv4 will verify and merge. Thanks!

@yktv4 Thanks for the help. Mukesh can you also update npm with the fix, I have a defect on this one for very long time ?

Hi @anilreddykatta . I will do it over the weekend

@mukeshsoni I am sorry for bothering again and again. But can we expect version update on npm very soon?

@anilreddykatta sorry, but i am kind of busy right now on both professional as well personal front. I won't be able to give you a deadline on this one.

@yktv4 I have cut a release 3.5.0 with your changes. Can you verify if they are there?


Make objC InteropTests and MacTests buildable with bazel

Without this, building :InteropTest or :MacTests with bazel fails with missing TestBase.h.

@dennycd  please doublecheck what I'm doing, I'm quite new to objC (and building ObjC with bazel).

Adhoc run of grpc_objc_bazel_test: http://sponge/7a7af2a9-e9c3-412b-8783-a2cdd231c2fe


Only enable module if it has been acked

Fixed in https://github.com/ZeusWPI/OBUS/commit/14fce40f9f92a123d149d8d30110711e50bef9b9


[8.x] Run clearstatcache after deleting file and asserting Storage using exists/missing

PHP by default would cache is_file() and file_exists() output and this would potentially cause an issue when asserting file changes if changes occurred from a different process.

https://www.php.net/manual/en/function.clearstatcache.php
https://github.com/composer/composer/blob/711f436b2431ebf2845ee89a556ffde619212405/src/Composer/Util/Filesystem.php#L125-L126

Great work @crynobone 👍


Temp FIX for Routing to Home page regardless of SignIn

I have fixed the bug, New commit [ec95848] solves the navbar and the landing page.
I apologise for any trouble I have caused.

The deployment on vercel is not the latest version, please look into, the
changes are not being reflected over there, once allow the vercel to deploy
it
[image: Screenshot 2024-05-14 at 17.27.23.png]
[image: Screenshot 2024-05-14 at 17.27.42.png]
On Tue, 14 May 2024 at 16:51, Sayak Bhunia @.***> wrote:

Merged #87 https://github.com/Sayak-Bhunia/mystory/pull/87 into main.
—
Reply to this email directly, view it on GitHub
https://github.com/Sayak-Bhunia/mystory/pull/87#event-12800442944, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A32QI6VWJTHAKVOXCBS642LZCHXSFAVCNFSM6AAAAABHV6RWT2VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSHAYDANBUGI4TINA
.
You are receiving this because you were mentioned.Message ID:
@.***>

This PR is giving the build errors - FAQs page
[image: Screenshot 2024-05-14 at 17.29.59.png]fin
[image: Screenshot 2024-05-14 at 17.30.28.png]


Possible problem with other required fields

I've noticed that if some fields of profile are required but cannot be provided by gateway it will fail to register new account. Also fields like gender if are hidden on gateway it will fail to register new account. Maybe consider allowing then people to fill missing fields on some kind of "stripped" registration form or just simply load default values provided and allow account to be created.

What about just skip these required fields if user is registered via 3rd party... because many 3rd party services does not provide these data (or it is really limited). I think this is quite a high priority ticket - especially for boards where any profile field is required in registration process. @effone @lairdshaw

What about just skip these required fields if user is registered via 3rd party... because many 3rd party services does not provide these data (or it is really limited). I think this is quite a high priority ticket - especially for boards where any profile field is required in registration process. @effone @lairdshaw

This is very awkward condition while there might not be any possible default value against a set required field.
Will explore possibilities to bypass required field check or may end up asking from the user only before creating the account...

This is very awkward condition while there might not be any possible default value against a set required field.
Will explore possibilities to bypass required field check or may end up asking from the user only before creating the account...

Yeah, is there any usable hook?

Yeah, is there any usable hook?

I am afraid there is no hook as such.
But there is one hack available. Admin CP has inbuilt bypass ability. So, if we define IN_ADMINCP the check gets bypassed. Don't know if this is acceptable or not.

I am afraid there is no hook as such.
But there is one hack available. Admin CP has inbuilt bypass ability. So, if we define IN_ADMINCP the check gets bypassed. Don't know if this is acceptable or not.

Like to have opinion of @lairdshaw on this.

Like to have opinion of @lairdshaw on this.

@lairdshaw - what do you think?
@effone - I think it is quite necessary for boards where some profile fields are required. Btw MyFacebook connect from Shade works fine even when you have some fields required during the registration. If you register via MyFacebook connect plugin, the field is empty but registration is done without any issue.

@lairdshaw - what do you think?
@effone - I think it is quite necessary for boards where some profile fields are required. Btw MyFacebook connect from Shade works fine even when you have some fields required during the registration. If you register via MyFacebook connect plugin, the field is empty but registration is done without any issue.

@effone, @Eldenroot, I reckon that ideally we'd do as @effone suggested: present to the user a minimal registration form with the missing required fields.
The IN_ADMINCP hack is another possibility, and here's yet another (untested code; just to get the idea across):
Just before line 384 of inc/plugins/isango.php, insert the following:
// Temporarily set all required profile fields to not
// required, keeping a list of those fields so we can
// restore them afterwards.
$req_fields = array();
global $cache;
$pfcache = $cache->read('profilefields');
if (is_array($pfcache)) {
	foreach ($pfcache as $id => $profilefield) {
		if ($profilefield['required'] == 1) {
			$req_fields[$id] = $profilefield;
			$profilefield['required'] = 0;
		}
	}
}
$cache->write('profilefields, $pfcache);

And then immediately after line 386, to insert this code:
// Restore required profile fields
foreach ($req_fields as $id => $profilefield) {
	$profilefield['required'] = 1;
	$pfcache[$id] = $profilefield;
}
$cache->write('profilefields, $pfcache);

@effone, @Eldenroot, I reckon that ideally we'd do as @effone suggested: present to the user a minimal registration form with the missing required fields.
The IN_ADMINCP hack is another possibility, and here's yet another (untested code; just to get the idea across):
Just before line 384 of inc/plugins/isango.php, insert the following:
// Temporarily set all required profile fields to not
// required, keeping a list of those fields so we can
// restore them afterwards.
$req_fields = array();
global $cache;
$pfcache = $cache->read('profilefields');
if (is_array($pfcache)) {
	foreach ($pfcache as $id => $profilefield) {
		if ($profilefield['required'] == 1) {
			$req_fields[$id] = $profilefield;
			$profilefield['required'] = 0;
		}
	}
}
$cache->write('profilefields, $pfcache);

And then immediately after line 386, to insert this code:
// Restore required profile fields
foreach ($req_fields as $id => $profilefield) {
	$profilefield['required'] = 1;
	$pfcache[$id] = $profilefield;
}
$cache->write('profilefields, $pfcache);

Shade did this with FacebookConnect:
        unset ($data->errors['missing_required_profile_field'],
               $data->errors['bad_profile_field_values'],
               $data->errors['max_limit_reached']);

But I like @lairdshaw 's approach.

Shade did this with FacebookConnect:
        unset ($data->errors['missing_required_profile_field'],
               $data->errors['bad_profile_field_values'],
               $data->errors['max_limit_reached']);

But I like @lairdshaw 's approach.

Yes, Shade's approach is simpler and perhaps more elegant than mine - as well as less prone to accidentally leaving the profilefields cache with incorrect contents.

Yes, Shade's approach is simpler and perhaps more elegant than mine - as well as less prone to accidentally leaving the profilefields cache with incorrect contents.


[5.5][Refactoring] Add @completionHandlerAsync to sync function

Cherry-picks https://github.com/apple/swift/pull/37417

When adding an async alternative, add the @completionHandlerAsync
attribute to the sync function. Check for this attribute in addition to
the name check, ie. convert a call if the callee has either
@completionHandlerAsync or a name that is completion-handler-like name.
The addition of the attribute is currently gated behind the experimental
concurrency flag.
Resolves rdar://77486504

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - b73c4f87080e97afa22edeed1fc066331c0b8aa4

@swift-ci please test Linux platform

@swift-ci please test

@swift-ci please nominate
--- CCC ---
Explanation: Add a "@completionHandlerAsync" attribute to the sync function when performing the "Add Async Alternative" refactoring and experimental concurrency features are enabled with -enable-experimental-concurrency. Have the function conversion check if this attribute is on the callee of a callsite when deciding whether to convert it (in addition to the existing name check).
Adding the attribute also enables a warning when that function is used in an async context (already used for implicitly imported ObjectiveC async functions).
Radar/SR Issue: rdar://77486504
Risk: Very low - only affects the async refactoring and only when experimental concurrency features are enabled.
Testing: Added new test cases to the refactoring test suite


HBASE-24025: Improve performance of move_servers_rsgroup  by using async region move API

HBASE-24025: Improve performance of move_servers_rsgroup  by using async region move API

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 30s
Docker mode activated.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 19s
the patch passed

+1 :green_heart:
compile
1m  7s
the patch passed

+1 :green_heart:
javac
1m  7s
the patch passed

+1 :green_heart:
shadedjars
5m 35s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 39s
hbase-server in the patch failed.

_ Other Tests _

-1 :x:
unit
179m 45s
hbase-server in the patch failed.

206m 12s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1549/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

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

JIRA Issue
HBASE-24025

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 3b077ef5a8f0 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 2846ea4700

Default Java
2020-01-14

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1549/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1549/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1549/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1549/1/testReport/

Max. process+thread count
3364 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1549/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

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

This message was automatically generated.

Related to #1321

Maybe try pushing again @arshadmohammad  to get a new test fun?

Corrected code to address TestRSGroupsAdmin2 test failure, rebased the PR

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 26s
Docker mode activated.

Hmm... the build passed:
03:46:34  +1 overall 03:46:34   03:46:34  | Vote |      Subsystem |  Runtime   | Comment 03:46:34  ============================================================================ 03:46:34  |   0  |        reexec  |   0m 27s   | Docker mode activated.  03:46:34  |  -0  |         yetus  |   0m  3s   | Unprocessed flag(s): --brief-report-file  03:46:34  |      |                |            | --spotbugs-strict-precheck 03:46:34  |      |                |            | --whitespace-eol-ignore-list 03:46:34  |      |                |            | --whitespace-tabs-ignore-list 03:46:34  |      |                |            | --quick-hadoopcheck 03:46:34  |      |                |            | Prechecks  03:46:34  |      |                |            | master Compile Tests  03:46:34  |  +1  |    mvninstall  |   3m 50s   | master passed  03:46:34  |  +1  |       compile  |   0m 56s   | master passed  03:46:34  |  +1  |    shadedjars  |   6m 36s   | branch has no errors when building our  03:46:34  |      |                |            | shaded downstream artifacts. 03:46:34  |  +1  |       javadoc  |   0m 40s   | master passed  03:46:34  |      |                |            | Patch Compile Tests  03:46:34  |  +1  |    mvninstall  |   3m 31s   | the patch passed  03:46:34  |  +1  |       compile  |   0m 56s   | the patch passed  03:46:34  |  +1  |         javac  |   0m 56s   | the patch passed  03:46:34  |  +1  |    shadedjars  |   6m 32s   | patch has no errors when building our  03:46:34  |      |                |            | shaded downstream artifacts. 03:46:34  |  +1  |       javadoc  |   0m 37s   | the patch passed  03:46:34  |      |                |            | Other Tests  03:46:34  |  +1  |          unit  | 135m 18s   | hbase-server in the patch passed.  03:46:34  |      |                | 161m 28s   |  03:46:34


Not-Authorized | Error Reading From XML stream

I'm able to authenticate via Pidgin and connect to the conference chat, but when I run i get an "not-authorized" error followed by a "Error reading XML Stream", which i assume is because i'm not authenticated.... Curious if this is still being used.
I'm getting this error:
root@muffserver:/var/www/wwwshare/wx/nwws-python-client# python nwws2.py config.json INFO     Negotiating TLS INFO     Using SSL version: TLSv1 INFO     Authentication failed: not-authorized INFO     Authentication failed: not-authorized INFO     Authentication failed: not-authorized ERROR    Error reading from XML stream. INFO     Waiting for </stream:stream> from server INFO     Negotiating TLS INFO     Using SSL version: TLSv1 ^CCaught Ctrl+C. Exiting. INFO     Waiting for </stream:stream> from server
I should note i'm in way over my head as i have no clue how xmpp works.

Hi William,
Are you connected to Pidgin at the same time that you're trying to connect with the nwws-python-client ? The National Weather Service doesn't allow multiple simultaneous connections using the same credentials, that is the error you would get if you are already connected.
Jim

Yea I had made sure i closed down Pidgin. Curious if you are still using this? I was reading something about SSL certs being V3 in the sleekxmpp. I'm not sure what "Use old-style SSL" does within Pidgin.

I suspect this issue is related to what came here to report.
With Python 3.7, the connection process fails on SSL cert negotiation.  It goes no further, and TCP connections are continually re-established, hinting at SSL failures.  Seeing that nwws-oi specifically calls out to use old-style SSL, this seems to need to be specified in Sleekxmpp.
The below diff does that.  First True says to retry, use TLS is set to False, use legacy SSL set to true.
`diff --git a/nwws.py b/nwws.py
index b5f301c..23bbe93 100644
--- a/nwws.py
+++ b/nwws.py
@@ -190,7 +190,7 @@ if name == 'main':
xmpp.register_plugin('xep_0199') # XMPP Ping
     # Connect to the XMPP server and start processing XMPP stanzas.

   if xmpp.connect((config['server'], config['port'])):

   if xmpp.connect((config['server'], config['port']), True, False, True):
       # If you do not have the dnspython library installed, you will need
       # to manually specify the name of the server if it does not match
       # the one in the JID. For example, to use Google Talk you would`

Thank you for reporting this. I'm using Python 3.6 on my dev box and using TLS works there. I have updated the code to include options for setting retry, use_tls, and use_ssl so users can set the options appropriately on their machine to get a working connection.


Fetch branches while doing pull

This increases the UX and also is convenient shortcut for fetching.

Please read this comment: https://github.com/Microsoft/vscode/issues/56463#issuecomment-420666392

I tried to find this before making the PR but  my attempt was unsuccessful, thank you.


Demo input css

How can I change the background color of the ion-md-input , it seems that ionic material set the background to black by default and I am unable to change it even I copy the stylesheet from ion-md-input itself.

ion-md-input author here, you should be able to pass a highlight color to the directive.
https://github.com/mhartington/ion-md-input#usage
<ion-md-input placeholder="Name" hightlight-color="energized" type="text"></ion-md-input>

Hi
My default color is white on the background page therefore I need change to style to white background color of the input box and the floating label to dark color.
What I mean is the background color of the input text, at the moment it set to dark #444 in your style.css and I could not change the style unless overwrite your css.
The hightlight-color is just change the underline color of the input text and it did not change the background of the input.

Same issue for me

So, I'm not sure I follow. Are you talking about the background color of the inputs or the text-color of the label? Either of the two can be overwritten with your own css, so I'm not sure this is much of an issue.
http://codepen.io/mhartington/pen/zGNrzV
It just would require you to write your own css to fit your needs.

Hi, actually, including your css, and following the instructions on ion-md-input to use it, I get this

Any way you could put together a codepen demo?
Something else must be happening here that I'm not aware of

Hi ,
I means the background of the input, how can you change it to dark color to white color? Also, the input floating label as well.
I used bower to install to my application and reference the css and js, however I could not find any document how to change the style of those input.
Thanks
Thomas

Codepen or Plunkr please
On Fri, 29 May 2015 at 00:36, Thomas Tran<EMAIL_ADDRESS>wrote:

Hi ,
I means the background of the input, how can you change it to dark color
to white color? Also, the input floating label as well.
I used bower to install to my application and reference the css and js,
however I could not find any document how to change the style of those
input.
Thanks
Thomas
—
Reply to this email directly or view it on GitHub
https://github.com/zachsoft/Ionic-Material/issues/74#issuecomment-106629474
.

Please demonstrate this issue with a codepen

It likes the ionic-material conflict with ion-md-input.

ionic-material overrides the ion-md-input.

Just include both ionic-material and ion-md-input to reproduce the issue.

Any feedback？

Yes, if use ionic-material and ion-md-input together, the background of ion-md-input will change.

.item-md-label input classes conflict. Need custom override to fix.

Hi - I've added a new directive to the ion-md-input.  This now resets the style.  If you want to use without Ionic Material styling just use like this:
My fork here: https://github.com/adskiremote/ion-md-input

Pretty sure this was fixed in my original repo, just this repo was pulling an older version

Hi Mike,
ah sorry didn't realise.  Thought the other was the updated.  Thanks for that as I'll update mine and check :-)

1 add via bower ionic-material and ion-md-input.
2 in bower component ion-md-input open bower.json and modify main like that "main": [
"ion-md-input.js",
"css/ion-md-input.css"
],
3 open ionic-material.css in bower components and comment all row in section /* Directives : MD Label.


Add validation comment to enums

This closes #119

Pull Request Test Coverage Report for Build 119278200

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
190 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-3.1%) to 27.775%

Files with Coverage Reduction
New Missed Lines
%

pkg/zips/env.go
3
0%

pkg/zips/types.go
25
54.76%

pkg/zips/client.go
75
3.96%

pkg/zips/template.go
87
26.32%

Totals

Change from base Build 118699764:
-3.1%

Covered Lines:
1419

Relevant Lines:
5109

💛  - Coveralls

One thing is that Kubebuilder allows validation comments on the type definition. Do we want to put it there instead of on each field using that type? It would stop us “down-casting” in getFields as well.


Several python cleanups

Add some init.py files so we can make importing work.
Move shipnames into shipname.py. (Not having to drag the data.txt around is nice).
Add a basic .pylintrc file, generated with pylint --generate-rcfile > .pylintrc
Ignore all files except cloud_logging and coords, which I've fixed up.
I'll fix up the other files in a later CL.
Add a test.sh script for running linting + tests.

Partially addresses: https://github.com/tensorflow/minigo/issues/35

Good idea. Fixed the makefile.


2 problems when parse xml

here is the xml:http://www.f4l.org.il/site/shop/compareEngine.asp?detail_id=4161674&depart_id=261546&allUrl=1
when it parsed to json here is a part of what I got:
{
"STORE": {
"$": {
"URL": "http://www.f4l.org.il/site/detail/detail/detailDetail.asp?detail_id=4161674",
"DATE": "6/10/2015",
"TIME": "11:51:52",
"NAME": "����� ����-NUTRA GOLD",
"STATUS": "ONLINE",
"ID": ""
},
"PRODUCTS": {
"PRODUCT": [
{
"$": {
"NUM": "1"
},
"PRODUCT_URL": ""http://www.f4l.org.il/site/detail/detail/detailDetail.asp?detail_id=4161712\"",
"name": ""����� ���� ��� ��� ��� 3 �"�"",
"MODEL": """",
"description": """",
"CATALOG_NUMBER": ""NG1111003000"",
"CURRENCY": ""ILS"",
"pricePerUnit": ""89"",
"SHIPMENT_COST": ""0"",
"DELIVERY_TIME": """",
"MANUFACTURER": """",
"WARRANTY": ""0"",
"imageUrl": ""http://www.f4l.org.il/image/users/261546/detail/big/4161712-8149.jpg\"",
"TAX": ""1""
},
as you can see there are additional bracelets in each text and the encoding is bad...
any solution for me?
edit: the first solution was salved by set utf8 to my response.
however the second one (addiotnal bracelets) is not solved

Not sure what you mean by "additional bracelets"

@kalitas you get double quotes "" because your xml contains data betwenn "", so you get same content. its not modify, its not problem with xml2js
Look, this its a part of your xml

        "http://www.f4l.org.il/site/detail/detail/detailDetail.asp?detail_id=4161712"

"נוטרה גולד כלב גזע קטן 3 ק"ג"
""

 ""

"NG1111003000"
"ILS"
"89"
"0"
""
""
"0"

        "http://www.f4l.org.il/image/users/261546/detail/big/4161712-8149.jpg"

    "1"

you can see double quotes (bracelets)?
If you want remove them you can do this before use xml2js
var xml = loadXML('........');

xml = xml.replace(/>\s*\"/ig,'>').replace(/\"\s*</ig,'<'); // remove any double quote `"` envolves your content

var json = parser.parseString(xml);

try it!

@clystian That regex ignores <![CDATA[ ... ]]> which is to be completely ignored by the parser.
Just a heads up, fyi, etc, etc

Hi guys, thanks for the answer I have 7 years experience in iOS and zero in server side :(
Anyway, @clystian that solved the double quoted problem.
1.what about the CDATA: it parsed as ""... ?
2.Im having difficulties with the encoding (which I thought i solved already); the hebrew look like that:
(look in the NAME Tag)
{
"STORE": {
"$": {
"URL": "http://www.f4l.org.il/site/detail/detail/detailDetail.asp?detail_id=4161674",
"DATE": "7/10/2015",
"TIME": "10:40:15",
"NAME": "����� ����-NUTRA GOLD",
"STATUS": "ONLINE",
"ID": ""
},
"PRODUCTS": {
"PRODUCT": [
{
"$": {
"NUM": "1"
},
"PRODUCT_URL": "http://www.f4l.org.il/site/detail/detail/detailDetail.asp?detail_id=4161712",
"name": "����� ���� ��� ��� ��� 3 �"�",
"MODEL": "",
"description": """",
"CATALOG_NUMBER": "NG1111003000",
"CURRENCY": "ILS",
"pricePerUnit": "89",
"SHIPMENT_COST": "0",
"DELIVERY_TIME": "",
"MANUFACTURER": "",
"WARRANTY": "0",
"imageUrl": "http://www.f4l.org.il/image/users/261546/detail/big/4161712-8149.jpg",
"TAX": "1"
},
this is how i get my xml from the server:
function getXmlFromServer(url, callback) {
var req = http.get(url, function(res) {
res.setEncoding('utf8')
var xml = '';
res.on('data', function(chunk) {
xml += chunk;
});
res.on('error', function(e) {
  callback(e, null);
});

res.on('timeout', function(e) {
  callback(e, null);
});

res.on('end', function() {
  nonDoublequetedXML = removeDoubleQuotes(xml);
  parseString(nonDoublequetedXML, {explicitArray:false,tagNameProcessors:[modifyTags]}, function(err, result) {
    fs.writeFile("./xmlresults.json", JSON.stringify(result, null, '\t'));
    callback(null, result);
  });

//  });
});
});
}
I tried so many things but nothing is fixing the problem.
by the way when I look at the XML from the browser (chrome) I see the hebrew fine...

OK found the encoding solution: i used xml += chunk and then encode, while I should do is       newBuffer = Buffer.concat([newBuffer, chunk]) and now encode.
thanks anyway.
if you have any idea why the CDATA parse as  """" please let me know...

Because this is invalid data:
"description": """"

That is two strings missing an operator, so they must be escaped to be valid JSON. Hence the backslashes.

So, I suppose this is solved and had no actual bugs in xml2js, so I'm closing it.


workspaces.yaml has no versioning mechanism

See original issue on GitLab
In GitLab by [Gitlab user @richardmaw-codethink] on Mar 1, 2018, 17:19
!271 was submitted to add a versioning mechanism to workspaces.yaml so that after upgrading buildstream to a new version, it can cope with having older workspaces.yaml files.
Review suggests that the implementation in !271 may be overcomplicated, the changes in the wrong place and not tested at the correct level, so an alternative implementation is required to solve this issue, hence this issue has been created rather than hijacking the merge request.

In GitLab by [Gitlab user @richardmaw-codethink] on Mar 8, 2018, 19:45
mentioned in merge request !312

In GitLab by [Gitlab user @richardmaw-codethink] on Mar 9, 2018, 10:08
mentioned in merge request !313

In GitLab by [Gitlab user @juergbi] on Mar 22, 2018, 15:54
closed via merge request !313


How to install and deploy pinpoint after the jar file is deleted?Where should I find the jar files for installation?

When installing and deploying pinpoint, I need to manually upload the pinpoint-Flink-job - {pinpoint version}. Jar file into the Flik component.But now that the jar file has been deleted, where can I download or find a jar file that conforms to my own pinpoint version?

@Zhaoyang123
Flink component is not required.
Uploading pinpoint-Flink-job in the past was a mistake.
To able inspector aggregation feature, you need to build the flink job. jar file by yourself.


find - regular expression

Issue Type: Bug
\d{6}-\W this criteria not working on this text editor, in other text editor works
text
180702-
180703-180704
VS Code version: Code 1.24.1 (24f62626b222e9a8313213fb64b10d741a326288, 2018-06-13T17:51:32.889Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700 CPU @ 3.60GHz (8 x 3600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwarevideo_decode: enabledvideo_encode: enabledvpx_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.87GB (9.56GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

mssql
ms-
1.4.0

I guess the difference is whether \W matches \n, which it seems like it should because they match in the JS regex engine, and replacing it with a \n matches

we should have a wiki page for both find widget and search widget about the special cases of regex handling.


Staging cannot serve .json files

I have no idea about this, but this is my general pitch, files like:

http://testsite-typescript-41eeb979-7eaa-4c74-9d47 9d182c7b61ab.azurewebsites.net/chunk-map.json
http://testsite-typescript-41eeb979-7eaa-4c74-9d47 9d182c7b61ab.azurewebsites.net/js/examples/en.json
http://testsite-typescript-41eeb979-7eaa-4c74-9d47-9d182c7b61ab.azurewebsites.net/page-data/docs/handbook/release-notes/typescript-3-2.html/page-data.json

Just end up 404ing on the staging, but not on the main site:

http://www.typescriptlang.org/v2/js/examples/en.json
http://www.typescriptlang.org/v2/page-data/docs/handbook/release-notes/typescript-3-2.html/page-data.json

The files are definitely getting shipped, you can see them in the rendered HTML: https://github.com/microsoft/TypeScript-Website/tree/SITE-STAGING/page-data
So, there must be some kind of switch in the settings for staging. This alas, means we can't use staging with v2 for the moment unless you have JS disabled.

This is the same thing on prod - sad face

On the flip side, this means I can test on staging and get the same results?

The v1 website ships with a Web.config which says JSON is OK according to this site - I've just sent a build testing that for the v2 - https://github.com/microsoft/TypeScript-Website/commit/d4916f4fad8f4b8913319023c1c53e0e5c89fd68

Fixed!


Doesn add template entry

It doesnt add template entry in brackets

Hi @mixedd69, can you provide some details about the version of Brackets you're using. That way, I can properly debug.


refactor: Fixed alertContainerSelector path

Hey!
This pull request include a fix because there was an issue where the alert container selector was not found because it simply didn't exist. I assume npmjs made some changes to their website, anyways I changed the selector path to the correct path and tested the extension and everything seems to be work again.
I also tried to follow the commit message syntax by looking at the other commits so I hope I did it correctly :)

Hey! Thank you so much 🙏 I Might've gone for the fix instead of the refactor 😛 But that doesn't really matter 😁 I'll release it later today 🥇


Kill daemon process if running with a different version

Solution/Idea
This pr try to kill an existing daemon process if it is running with different version than expected. It will allow setup to run
required version of daemon.
Testing

Start crc daemon with older version
Try crc setup with this PR (bump the version to something like 2.2.0)
crc setup should kill the existing crc daemon process and start the daemon with correct version.

Also

Start crc daemon with same version
Try crc setup with this PR
crc setup --check-only should pass with using existing daemon process.

This will be useful for upcoming releases where user try to upgrade crc but daemon might be running with older version of crc so that case crc setup should kill that existing process and start daemon process with launchd/task manager.

LGTM 👍🏽

/retest


Fix project export with skeletons

Motivation and context

Fixed #4985, #4990

How has this been tested?

Added REST API test
Checklist

[x] I submit my changes into the develop branch
[x] I have added a description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[x] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

/check

/check


fix CVD_FREE for ctest

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

Project coverage is 90.36%. Comparing base (52c59db) to head (dfa2e9f).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #110   +/-   ##
=======================================
  Coverage   90.36%   90.36%
=======================================
  Files          35       35
  Lines        5322     5322
=======================================
  Hits         4809     4809
  Misses        513      513

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


Clean up code to make it clean for flake8

Need to clean up code for flake8.

@RehabSayed-G
I need you to work and this.


New-DbaAvailabilityGroup - add a test before attempting to add database

Sometimes this awesome command would be too darn fast and the ag wasn't ready to accept a database. This waits until the state is connected before attempting to add.

Good idea. But we should not only wait if we want to add databases within the command, but also wait in any case for the state "connected" before running the final Get-DbaAvailabilityGroup. Because then all other commands in a script that runs New-DbaAvailabilityGroup can rely on a ready-to-go AG. So maybe move the new block up to line 642?

good idea! moved it

Maybe add a warning message, if $ready is still $false? I would not fail, but let the user know that the AG is maybe not yet ready. Then, if adding the databases failes, the user has a hint why adding databases failed.

"One last thing": -not $ready is sufficent, -or $wait -gt 40 not needed.

I know, im just covering alll the bases 😅

sweet, thanks 👍🏼


chmod hack in darwin-build seems to be unnecessary

Hi. Thanks for maintaining this excellent repo. I am not using it directly, but it's guiding my own setup.
I ran into the problem that this is trying to handle: https://github.com/dustinlyons/nixos-config/blob/d9b4eca2a349d10670d65fe117c85c042b10b591/bin/darwin-build#L20-L25
And found this discussion here https://github.com/zhaofengli/nix-homebrew/issues/9#issuecomment-1774684583.  I followed the suggestion about tapping homebrew-bundle in nix, and it works.  This seems like a better approach than what darwin-build is doing, as far as my understanding of the situation goes 🌞.
So, maybe you want to do the same; this is just FYI.

Oh wow, this is huge! Thanks for taking the time to let me know; I hated writing this hack 😂
I've updated the configuration. What an eloquent solution.

Confirmed this works for me as well. Soooo happy I get to remove that incredibly egregious chmod 🙌


Problem: Detailed transaction responses disabled

Solution: Migrate the response to Tendermint 0.19.0 and enable the same

I didn't close this issue directly --- and didn't intend to close it at all! It got closed automatically when I deleted the old branch named alpha-3, which is the branch that this PR was merging into.
@kansi - You should probably still have the only commit in this PR, i.e.  https://github.com/bigchaindb/bigchaindb/pull/2239/commits/908bbe101d4173e0ba279d85d3902e1c732d16dd --- along with the branch kansi:iss/txn-responses. Can you create a new PR based on that?
GitHub also has that commit.

@ttmc This feature needs to be redone and I don't the code in this branch would be of much help. So, I think it is okay to close this one.


http://mibs.snmplabs.com/asn1/IEEE8023-LAG-MIB is out of date

file there is updated "201412180000Z".
Later MIB at http://www.ieee802.org/1/files/public/MIBs/IEEE8023-LAG-MIB-201610120000Z.txt is updated "201610120000Z"
Example significant changes that cause validation failures:
dot3adAggPortActorSystemPriority OBJECT-TYPE
    SYNTAX       Integer32 (0..255)
    MAX-ACCESS   read-write
    STATUS       current
    DESCRIPTION
        "A 2-octet read-write value used to define the priority
        value associated with the Actor's System ID."
    REFERENCE
        "<IP_ADDRESS>.2"
    ::= { dot3adAggPortEntry 2 }

vs.
dot3adAggPortActorSystemPriority OBJECT-TYPE
    SYNTAX       Integer32 (0..65535)
    MAX-ACCESS   read-write
    STATUS       current
    DESCRIPTION
        "A 2-octet read-write value used to define the priority
        value associated with the Actor's System ID."
    REFERENCE
        "<IP_ADDRESS>.2"
    ::= { dot3adAggPortEntry 2 }

Thank you for the heads up!
I just run the mibcopy tool against the entire ieee802 repo. The total of 34 MIBs have been copied. So I hope we are now in-sync.
Let me know if else anything comes up!
Thanks!


TypeError: Cannot read property 'fetch' of undefined

I am trying to get it work with my webpack setup.
Repo: https://github.com/christophmayrhofer/cc-client
I get the following error:
TypeError: Cannot read property 'fetch' of undefined
I tried:
<require from="./app.css!bind" as="styles"></require>
and
<require from="./app.css!module!bind" as="styles"></require>
I get the same error for both.
Would be great if you could have a look at the repo and tell me what's wrong.
Using
import styles from './app.css'
in the js file and the ${styles.xxx} in the html works just fine.

Sorry, but I don't have much time to spare right now. I haven't actually used this with webpack, so it's certainly possible that it needs some changes. PRs welcome.


Separate core snapshot engine from JUnit integration

There are currently a few places where the snapshot engine implementation is tightly coupled to JUnit. I don't think this is necessary and we could try to separate the concerns here and split into

pure reusable snapshot engine w/o JUnit dependency
JUnit5 integration of the engine

This would also allow to integrate with further testing frameworks or to use the engine standalone

Unfortunately we can not completely get rid of JUnit dependencies in the main API package. We'd need to move the @EnableSnapshotTests annotation into a new package which would be a breaking change. So we can only do that with the next major release. (see #17)

Closed with 1.1.0


Check schema for tabularity

Currently, you will get a ValueError if trying to read a schema containing record, list, map or union types. This only happens once actually reading, which could be a problem for dask applications. Could make an up-front check.

Fixed in 61e4c51b09f5200d6e23a370711d79c7d97b7f59


Add "logical fallacy" check to model output

I'd added FallacyChain to LangChain last month (checks for logical fallacies) - link below - thinking I could add similar functionality to natural language model output here - thoughts?
https://python.langchain.com/docs/guides/safety/logical_fallacy_chain

This is very interesting - how would one capture the the degree of logical fallacy?

It could return a dictionary of logical fallacies but I think better for evaluation purposes a % of logical fallacies (with the denominator being sentence count) but returning a float.


Question about "collate"

    def collate(self, batch):
        elem = batch[0]
        if isinstance(elem, Data):
            return Batch.from_data_list(batch, self.follow_batch)
        elif isinstance(elem, torch.Tensor):
            return default_collate(batch)
        elif isinstance(elem, float):
            return torch.tensor(batch, dtype=torch.float)
        elif isinstance(elem, int_classes):
            return torch.tensor(batch)
        elif isinstance(elem, string_classes):
            return batch
        elif isinstance(elem, container_abcs.Mapping):
            return {key: self.collate([d[key] for d in batch]) for key in elem}
        elif isinstance(elem, tuple) and hasattr(elem, '_fields'):
            return type(elem)(*(self.collate(s) for s in zip(*batch)))
        elif isinstance(elem, container_abcs.Sequence):
            return [self.collate(s) for s in zip(*batch)]

Thanks for your great repository!
I have questions about the collate function. Maybe the isinstance(elem, Data) is the most common case, but my requirement is very close to  isinstance(elem, tuple) and hasattr(elem, '_fields'). I don't understand when the _fields attribute comes up. Could you please introduce the logic of using isinstance(elem, tuple) and hasattr(elem, '_fields')?

hatasstr(elem, _fields) is a common way to check for NamedTuples. If this is the case, we recursively apply the collate function on every element of the tuple.

hatasstr(elem, _fields) is a common way to check for NamedTuples. If this is the case, we recursively apply the collate function on every element of the tuple.

@rusty1s
"recursively apply the collate function on every element of the tuple",
OK, that's what I want.
Thanks for your reply.
This issue will be closed later.

@rusty1s
"recursively apply the collate function on every element of the tuple",
OK, that's what I want.
Thanks for your reply.
This issue will be closed later.


Use ics package to parse schedules

The ics package seems useful for parsing calendars (schedules, TimeEdit, etc.).

Done.


Ability to customize choose team event

switch to spec
settings

Thank you sir you are a professional 🧑🎓🧠


About Keep Messages Encrypted

Hello 👋
Is there a way for me to keep messages encrypted?

Hi @sinapsis793,
No, not really. It will involve some Cloud Functions I suppose, and generally more robust solution. I wouldn't rate this lib as a good solution when you want to work with encrypted messages.


name property in pagerduty.CustomFieldArgs is not required

What happened?
Using the newly released 3.8.0 version of the provider, create a simple Custom Field with this code:
const field = new pagerduty.CustomField('test', {datatype: 'string', displayName: 'Test Field'})

Expected Behavior
Either:

There should be a TypeScript compilation error because the name property is not defined in the args parameter to the CustomField constructor
OR... the custom field should be successfully created

Steps to reproduce

Have a PagerDuty account enrolled in the Custom Fields early access program
Set the PAGERDUTY_TOKEN environment variable
Add the line of code above
Run pulumi up

Observe error:
POST API call to https://api.pagerduty.com/customfields/fields failed 400 Bad Request. Code: 2001, Errors: [name: can only contain a-z 0-9 and _], Message: Bad Request

Output of pulumi about
CLI
Version      3.55.0
Go Version   go1.20
Go Compiler  gc
Plugins
NAME       VERSION
nodejs     unknown
pagerduty  3.8.0
Host
OS       darwin
Version  13.2.1
Arch     arm64
This project is written in nodejs: executable='/Users/jedelson/.asdf/shims/node' version='v16.17.0'
Current Stack: justinedelson/pulumi-quickstart/dev
TYPE                        URN
pulumi:pulumi:Stack         urn:pulumi:dev::pulumi-quickstart::pulumi:pulumi:Stack::pulumi-quickstart-dev
pulumi:providers:pagerduty  urn:pulumi:dev::pulumi-quickstart::pulumi:providers:pagerduty::default_3_8_0
Found no pending operations associated with dev
Backend
Name           pulumi.com
URL            https://app.pulumi.com/justinedelson
User           justinedelson
Organizations  justinedelson
Dependencies:
NAME               VERSION
@pulumi/pagerduty  3.8.0
@pulumi/pulumi     3.55.0
@types/node        16.18.13
Pulumi locates its logs in /var/folders/vs/dqgdcsvn7gv642l5f6tpbcw40000gq/T/ by default
Additional context
Part of the issue seems to be that in the absence of the name property being provided in the args, a synthetic value is being generated. If I look at the actual POST call being made to the PagerDuty API, the request payload is this:
{
  "field": {
    "name": "test-88e8a27",
    "display_name": "Test Field",
    "datatype": "string",
    "multi_value": false,
    "fixed_options": false
  }
}

But as the error message produced by the API states, this isn't a valid name value (specifically dashes are not permitted).
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Hey @jedelson-pagerduty. Thanks for bringing this bug to our attention. name should be marked as required and isn't. I'll take a look.

Thanks @iwahbe . Please let me know if I can do anything to help. I tried updating the schema.json and running make build_sdks but it didn't change the type definition for pagerduty.CustomField. It did, however, highlight that there might be a similar issue with the Java SDK (although in the Java SDK it looks like these checks are done at runtime vs. compile time).

@jedelson-pagerduty I appreciate the offer. schema.json is derived from resources.go. We have a technology that does auto-naming for fields called "name", based on the resource name ('test' in your example), as you pointed out in your issue. By default, auto names look like ${RESOURCE_NAME}-1234abc, with the random portion pulling from 0-9, a-f (Hex). We can give a custom auto-naming implementation that will fix this.