Runtime errors with IE 11

After testing a recent project on IE 11 we get the following errors in the console. Does PIXI support IE 11 with WebGL? When this error occurs it seems to not show any DisplayObjects which require a texture source like that we source from an atlas.
If you would like to know specifically the source code that all these errors happen feel free to ask for more info, if it helps. It seems there are features that PIXI relies on that IE 11 does not support my best guess.
This was tested with v2.2.3

We used to use float there, but moved to unsigned byte to fix some rounding issues that were happening. This page seems to suggest that it is supported:
https://msdn.microsoft.com/en-us/library/ie/dn302460(v=vs.85).aspx

vertexAttribPointer can specify UNSIGNED_BYTE, BYTE, SHORT and UNSIGNED_SHORT for both normalized and un-normalized formats (all component types, 1 through 4, are supported) in Internet Explorer 11 using WebGL renderer version 0.93 and later.

Are you on the latest ie 11?

I believe its the latest update 11.0.9600.17031 as this was installed with a new Windows 8.1 machine here in the office,

I think that use of UNSIGNED_BYTE is not supported in ie11 a quick Google search I found this thread https://connect.microsoft.com/IE/feedback/details/814134/ie11-webgl-vertexattribpointer-only-support-the-type-float

That ticket you linked is closed as "fixed". Seems that version 11.0.9600.17107 is the version with the fix, if I read that correctly it should be released in April.
We can put a fallback in to use FLOAT if in IE 11 but that will cause a regression of all the bugs we fixed by changing it to UNSIGNED_BYTE.

Yeah I understand what you're saying but this will affect ** I believe ** quite a few projects and developers. I think the best solution really is to not use webgl for ie at all especially if the latest version is not supported . To me I read this as its a bit unstable to use webgl with ie.
To solve this for our project I will look into how easy it is to set canvas as the renderer we are currently using auto detect for this so not looked into any alternatives as yet.
Let me know if there's a better option?

Hey man!
IE11 has historically been a bit of a pain with regards to its webGL implementation. The version we tested on when we modified the code worked fine, but I guess it was the most up to date version.
Until we do get this nailed - The good news is that IE actually has the fastest canvas implementation of all the browsers so performance will not be an issue if you flicked to canvas mode.

Going to close this for now, since the preferred workaround is to use canvas renderer if on a lower version of IE 11.

ok sure but to clarify the issue here for others as it will affect anyone wanting to support ie 11. The issue is we have installed the very latest ie 11 this is the version I mention. Version asked to be installed is not even out until April this year is that right?  I can also confirm this because by navigating to the ie download site it says we already have the latest.
So the issue isn't that we are not using the latest its that the latest version of ie 11 doesn't work or hasn't been tested with webgl and pixi. So the only solution then is if ie just use canvas really :(

11.0.9600.17501 is installed on my Win 7 machine right now, so I'm not sure why you think you have the latest. 11.0.15 (v11.0.9600.17501) was released on 9 December 2014; 50 days ago.

Yes we've done this and going to the ie website says we have the latest version already but maybe this version as you've got is not supported for the latest operating system Windows 8.1 with 64 bit support.
Enabling only canvas for ie via manually constructing the canvas renderer worked fine as Mat suggested. This was very easy to implement with the heavily utilised if ie javascript check ;) also I agree with Mat the canvas implementation for ie is very performant!!

On my windows 8.1 x64 machine at home I am running 11.0.9600.17498. After checking for and installing update via Windows Update it updated to 11.0.9600.17501.

Was curious about this so just checked, and on my Win 7 x64 install IE is 11.0.9600.17501 by default. It updated itself automatically via Windows Update, but this is definitely the latest "in the wild" version.

Anyway this is getting a bit pointless we have installed the very latest windows update today and still we have this version I am not 100% sure why but this is what we are getting.
We cant even manually download the browser because it says I have the latest being ie 11 and I cant update to this version you have because the windows updates are not updating anyway the solution for me is to not bother with webgl on ie as we cant seem to install a version of ie which works with ie and pixi.js. I think you might be willing to accept that maybe this is not working for us.
i wish it was as simple as you say to just update, its not!

I would have thought it would it would be somewhat trivial for Pixi to detect if the current browser supports UNSIGNED_BYTE or not - and if not to force it to use the canvas renderer (as it's blatantly going to screw up if it tries to use WebGL) - should be part of the auto detect. Doesn't even need to be IE specific, as it could manifest anywhere.

Good suggestion and points too @photostorm would that just be an if null check right?
I think the issue is many html5 developers are not testing as many device and browser configurations as we are in this build. Personally I think this is a real issue With features that are not supported properly in webgl but are required by pixi. Maybe there's a way to add a handler so if rendering fails for any case then the renderer can be changed to canvas.
As I have been told that  ie makes a majority of windows users and windows is still the most popular os too :)

great points guys - we will certainly make it part of the autodetect for v3

would that just be an if null check right?

I don't think so, gl.UNSIGNED_BYTE exists in IE 11 WebGL but is not supported IIRC. That is why I mentioned it would have to be a user-agent string check.


Add support for resources to meeting rooms

We should be able to associate one or more resources ("projector", "telepresence", "whiteboard", etc) with each meeting room.
These resources will be represented with an array of strings in the JSON documents received and returned by the API.

In the DB schema, we will probably want a resource table with id and name, and an association table that allows us to specify which resources are associated with which rooms.

closed by https://github.com/zednis/meeting-scheduler-api/pull/11

Forgot to add creation and association of resources to rooms during room creation.


array_key_exists() expects parameter 2 to be array, null given

Hi, I'm trying to implement anti-spam using your script, everything looks fine (it blocks something..) but I get many errors in error.log :/
PHP Warning: array_key_exists() expects parameter 2 to be array, null given in /nfsmnt/_www/lib/Blacklist.php on line 204
I'm calling it like this:
require_once('../lib/Blacklist.php');
$client = new Thasmo\ProjectHoneypot\Blacklist($_SERVER['REMOTE_ADDR'], $data->key);
$result = $client->getResult();
is there something that I do wrong?

I'll have a look at it later.

@morvy, do you maybe call $client->getResult(); with an argument, like $client->getResult('activity'); or similar? I've released a new version which fixes the error, which happens if you call getResult with an argument while the HTTP:BL API's result doesn't indicate a threat.

This is my current function call:
$ip = $_SERVER['REMOTE_ADDR'];
if (!empty($data->key) && filter_var($ip, FILTER_VALIDATE_IP)) {
$client = new Thasmo\ProjectHoneypot\Blacklist($ip, $data->key);
$result = $client->getResult();
if(is_array($result)) {
I'm thinking if it's a problem related to wrong IP address (IPv4 vs IPv6 ?) because I'm also getting Fatal errors due to my provider as he's crawling my pages within his local network so his IP address (<IP_ADDRESS>) throws Invalid IP address error (but this is different story, not important and irrelevant for this issue)

IPv4 addresses won't work because the HTTP:BL API doesn't support them. Can you provide an IP address on which the error occurs so it's reproducable for me? Thanks a lot!

Yes, IPv6 cannot work, so I'm filtering only IPv4 (added filter_var option for that), but I receive only that one error regarding getResult();
Invalid IP Address is caused by <IP_ADDRESS> and <IP_ADDRESS> .. I'll try to also filter private and reserved IPs before calling your magic

You can use a try/catch block to handle the invalid IP addresses:
try {
	$client = new Blacklist('<IP_ADDRESS>', 'mykey');
} catch (Exception $exception) {
	print $exception->getMessage();
}

Does this solve your problems?

Well I don't need to catch errors, I'm just skipping submits if there is a threat detected. if the IP cannot be processed, I won't pass it to Blacklist at all (filter_val). I need to wait for a few hours so the website gets some visits to test those recent changes. But as I wrote, array_key_exists error is not related to Invalid IP Address .. I'll wait for new error and will let you know

Sure, I just meant you can rely on the Blacklist class to detect invalid IP addresses and don't need to provide your own solution with filter_var:
if (!empty($data->key) {
	$ip = $_SERVER['REMOTE_ADDR'];

	try {
		$client = new Thasmo\ProjectHoneypot\Blacklist($ip, $data->key);
		$result = $client->getResult();
		if (is_array($result)) {
			// BLOCK
		}
	} catch (Exception $e) {}
}

update: I'm not sure if getResult is directly connected to isSpammer, but I realised that I was also calling isSpammer in other part of page and this was causing the problem with array_key_exists. I didn't change my code only updated yours and it solved those errors, so together with try/catch part, my errorlog is clean again ;) thanks

Awesome, thanks for reporting! Feel free to open a new issue if you need help. Thanks!


Improve logging on explorer, tracker, scheduler, etc...

This patch tries its best to unify logging infrastructure and properly propagate verbose level set from command line. Specifically:

color.py and log.py has been merged to log.py
tar with -v always prints verbose messages to stderr and curl without -s always print progress bar to stderr. runSyncProcess will always print out stderr from subprocess because we use SEVERE level. In this PR, we change to FINE level.
Bug fix. I didn't review some PRs carefully so that my previous PRs that fixed some logging problems were reverted.
Various tiny fixes including word usage, making log look better.

More commits may be pushed later.
cc: @taishi8117 @maosongfu @kramasamy
resolves: #1194 #1129

@maosongfu I have question. Do we need to add -v to tar? Is it necessary to list the files in tar file? Similarly, if -s is not provided to curl, curl will show progress bar. Is progress bar necessary?

@maosongfu never mind. I now think those messages are necessary.

👍

@kramasamy btw, just as a reminder, please do not merge any other Python PR before this one.

@objmagic - sure.

👍


bskyweb: run-dev-bskyweb in debug mode

Small, low-impact, but when doing local dev probably want to load assets fresh off the disk every time.

Just double checked we have a log warning about this because I've torn my hair out trying to figure out why a file doesn't seem to be updating :+1:


Codec utf-16 synonyms do not work in read_csv with c engine

Code Sample, a copy-pastable example if possible
import pandas

path = "test.csv"
pandas.DataFrame({"A": [0,1], "B": [2,3]}).to_csv(
    path, encoding="utf-16")

for encoding in ["utf-16","utf_16","UTF_16","UTF-16"]:
    try:
        pandas.io.parsers.read_csv(
            path,
            engine='c',
            encoding=encoding)
        print(encoding, "succeeded in c-pandas")
    except UnicodeDecodeError:
        print(encoding, "failed in c-pandas")

    try:
        pandas.io.parsers.read_csv(
            path,
            engine='python',
            encoding=encoding)
        print(encoding, "succeeded in pandas")
    except UnicodeDecodeError:
        print(encoding, "failed in pandas")

    try:
        with open(path, encoding=encoding) as file:
            file.read()
        print(encoding, "succeeded in open")
    except UnicodeDecodeError:
        print(encoding, "failed in open")

Expected Output
utf-16 succeeded in c-pandas
utf-16 succeeded in pandas
utf-16 succeeded in open
utf_16 succeeded in c-pandas
utf_16 succeeded in pandas
utf_16 succeeded in open
UTF_16 succeeded in c-pandas
UTF_16 succeeded in pandas
UTF_16 succeeded in open
UTF-16 succeeded in c-pandas
UTF-16 succeeded in pandas
UTF-16 succeeded in open

Actual Output
utf-16 succeeded in c-pandas
utf-16 succeeded in pandas
utf-16 succeeded in open
utf_16 failed in c-pandas
utf_16 succeeded in pandas
utf_16 succeeded in open
UTF_16 failed in c-pandas
UTF_16 succeeded in pandas
UTF_16 succeeded in open
UTF-16 failed in c-pandas
UTF-16 succeeded in pandas
UTF-16 succeeded in open

output of pd.show_versions()
INSTALLED VERSIONS
commit: None
python: 3.5.1.final.0
python-bits: 64
OS: Linux
OS-release: 4.6.2-1-ARCH
machine: x86_64
processor:
byteorder: little
LC_ALL: None
LANG: en_GB.UTF-8
pandas: 0.18.1
nose: 1.3.7
pip: 8.1.1
setuptools: 23.0.0
Cython: None
numpy: 1.11.0
scipy: 0.17.1
statsmodels: None
xarray: None
IPython: 4.2.0
sphinx: None
patsy: None
dateutil: 2.4.2
pytz: 2016.4
blosc: None
bottleneck: None
tables: None
numexpr: None
matplotlib: 1.5.1
openpyxl: None
xlrd: None
xlwt: None
xlsxwriter: None
lxml: None
bs4: 4.4.1
html5lib: None
httplib2: 0.9.2
apiclient: None
sqlalchemy: None
pymysql: None
psycopg2: None
jinja2: 2.8
boto: None
pandas_datareader: None

Without error handling to run through all test cases, the traceback is:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/gereon/python/bugreport/test.py", line 11, in <module>
    engine='c',
  File "/usr/lib/python3.5/site-packages/pandas/io/parsers.py", line 562, in parser_f
    return _read(filepath_or_buffer, kwds)
  File "/usr/lib/python3.5/site-packages/pandas/io/parsers.py", line 325, in _read
    return parser.read()
  File "/usr/lib/python3.5/site-packages/pandas/io/parsers.py", line 815, in read
    ret = self._engine.read(nrows)
  File "/usr/lib/python3.5/site-packages/pandas/io/parsers.py", line 1314, in read
    data = self._reader.read(nrows)
  File "pandas/parser.pyx", line 805, in pandas.parser.TextReader.read (pandas/parser.c:8748)
  File "pandas/parser.pyx", line 827, in pandas.parser.TextReader._read_low_memory (pandas/parser.c:9003)
  File "pandas/parser.pyx", line 904, in pandas.parser.TextReader._read_rows (pandas/parser.c:10022)
  File "pandas/parser.pyx", line 1011, in pandas.parser.TextReader._convert_column_data (pandas/parser.c:11397)
  File "pandas/parser.pyx", line 1071, in pandas.parser.TextReader._convert_tokens (pandas/parser.c:12302)
  File "pandas/parser.pyx", line 1157, in pandas.parser.TextReader._convert_with_dtype (pandas/parser.c:13740)
  File "pandas/parser.pyx", line 1173, in pandas.parser.TextReader._string_convert (pandas/parser.c:13950)
  File "pandas/parser.pyx", line 1460, in pandas.parser._string_box_decode (pandas/parser.c:19767)
UnicodeDecodeError: 'utf-16-le' codec can't decode byte 0x41 in position 0: truncated data

In case it is relevant:
7.2.3. Standard Encodings
[…]Notice that spelling alternatives that only differ in case or use a hyphen instead of an underscore are also valid aliases; therefore, e.g. 'utf-8' is a valid alias for the 'utf_8' codec.
CPython implementation detail: Some common encodings can bypass the codecs lookup machinery […]: […] utf-16 […]

This also holds if the writing encoding is taken to be utf_16:
>>> pandas.DataFrame({"A": [0,1], "B": [2,3]}).to_csv(
        path, encoding="utf_16")
[…]
utf_16 failed in c-pandas
utf_16 succeeded in pandas
utf_16 succeeded in open

yeah this has a string definition of utf-16. Should be easy to make more general.
want to take a crack at it?


fix error in notifcation log documentation

notifications = application.notification_log.select(start=1, limit=2) will call an SQL transaction which uses LIMIT, which will only retrieve 2 records instead of 3 stated in the documentation.

Thank you very much @tohhengxing!!


Security implications of storing OAuth client ID in config.json?

Hi there, thanks for the great plugin!
I'm wondering about whether there are any security implications to storing the client ID in config.json, since I believe this info is accessible to anyone including unauthenticated users via the js source. I know that it's not totally secret info, but I have seen some people saying it's something better kept private, for example on ouath.com they say:

Even though it’s public, it’s best that it isn’t guessable by third parties

I wonder if it's worth using something like sanity-studio-secrets to store the client ID in the data set instead?
I also see that while it isn't documented, the personalAccessToken option is supported in the config and I would have thought that's a security risk since that is secret.

Thanks for the kind remarks!
I've had great ideas when I started working on this plugin but the primary reason why it didn't push through is because of this very concern you're asking me right now. To be honest, and I agree that it's better to keep the OAuth Client Id as a secret and not publicly accessible, say for example it being readable if your studio repo is set to public. But even then if not and we're going to use sanity-studio-secrets as per your suggestion, you can actually be able to read the value of it if your dataset is to public given the right endpoint and api parameters via GROQ to query such value.
At the time when I created this, I didn't have much knowledge so why I opted with using the  config json file. Knut also suggested the same thing to use sanity-studio-secrets but since we're not really specifically storing tokens, and only once you grant the plugin access to your account should it work, figures it's not much of a security risk but that has to say that's just as far as I know. I'd be happy to make changes should I learn more about the implications of how it is right now. Also, I'd be happy to accept contributions to improve the current behavior.
As for personalAccessToken, I never really pushed through implementing it! And I don't think I'll ever will for the reasons that if I'm not mistaken and as I explained earlier, in order to keep that token fully secure,  you have to make sure both your git repo and Sanity dataset is set to private. Thinking about the odds it'll not be followed and tokens possibly leaked (which means full access to anyone's Netlify account), I don't think it's worth it. Better be safe than sorry!


laravel 9 test

--coverage option added to laravel 9 runinng test, but it dont report any thing from codes inside modules!
it run tests inside modules becuse I addes this to phpunite.xml:

./Modules//Tests/Unit

./Modules//Tests/Feature

but do not report covarage

in phpunite.xml i add:

./app-
./Modules//Http
./Modules//Models/

but it dont show coverge of code in module folders

@hooman-mirghasemi  I struggled with this for a while and researched all the posts referring to code coverage and modules.
I could only get code coverage to work in src or app folders. You can test this by simply adding a src folder and putting an example file then adding /src to phpunit.xml
I ended up moving my modules to a src folder by doing the following
edit config/modules.php
'paths' => [
  'modules' => base_path('src')
  ...
  ]

edit composer.json
under "autoload"
     "psr-4": {
            "App\\": "app/",
            "Modules\\": "src/",
            ...
       }

My phpunit.xml is as follows
<testsuites>
    <testsuite name="Unit">
         <directory suffix="Test.php">./tests/Unit</directory>
     </testsuite>
     <testsuite name="Feature">
         <directory suffix="Test.php">./tests/Feature</directory>
     </testsuite>
      <testsuite name="Modules">
         <directory suffix="Test.php">./src/**/Tests/Feature</directory>
         <directory suffix="Test.php">./src/**/Tests/Unit</directory>
     </testsuite>
 </testsuites>
<coverage processUncoveredFiles="true">
     <include>
         <directory suffix=".php">./src</directory>
     </include>
      <exclude>
         <directory suffix=".php">./src/**/Routes</directory>
         <directory suffix=".php">./src/**/Resources</directory>
         <directory suffix=".php">./src/**/Tests</directory>
         <directory suffix=".php">./src/**/Config</directory>
         <directory suffix=".php">./src/**/Infrastructure</directory>
     </exclude>
 </coverage>

@hooman-mirghasemi @ronfuller
Appreciate this is an old issue but I ran into this yesterday and found this issue when searching, so just adding this for future reference in case anyone else comes across this.
I'm using pcov for code coverage and my issue was that the pcov directory was set to /app. This will be the default for Laravel projects unless it's explicitly configured.
So, code coverage should be run with the following command (as an example):
php -d pcov.directory="/Users/bradsi/code/my-project" ./vendor/bin/pest --coverage

This will set the coverage directory to the root of the project, instead of /app, and will cover your /modules folder.
However, in my case this pcov.directory setting was not applied. I have had this working before, so maybe there is something else in my local environment that was preventing it from running. If this happens to you, try setting the pcov.directory variable explicitly in the php.ini file.

Thanks @ronfuller, that worked for me.
Just wondering why only those folders works for coverage...


OPTIONS response

Automatic response for OPTION method requests.

This looks good.
Gonna check it out with some test cases and then merge.


Users cancelling tickets

Feature request for when a Parent is unable to attend an event, that it is possible for the parent to delete the Ticket for the Event.
Ideally, this would be done through links in the confirmation email which you receive when you book a ticket. A line at the bottom which would state the Following :
"If you are unable to attend this event, please click this link to Cancel your booking"
Clicking the link would bring you to a page which would say:
"Thank you for confirming you will not be in attendance; your ticket has now been canceled."

@johnbrett For more information on creating test data, see this doc. The main (readme file is here)[https://github.com/CoderDojo/community-platform/blob/master/README.md].

[Cancel backend logic is here](https://github.com/CoderDojo/cp-events-- service/blob/cc15412e8e8bd3be4e8b91c69ea03a3f6198bb34/lib/cancel-session.js)
You'll probably need a frontend controller like this invite one for the new cancel link page
Here is where an application is cancelled if the champion is doing it through the manage event page.

Let me know if you have any questions, usually around on the Gitter!

@tangentfairy Thanks for the extra info. I'm up and running with the platform and able to manage dojos and events locally but emails don't seem to be sending. Should they work out of the box or do I require any extra config/setup?

I've sent you a direct email with access to our mailtrap.io account!
On 30 January 2016 at 10:48, John Brett<EMAIL_ADDRESS>wrote:

@tangentfairy https://github.com/tangentfairy Thanks for the extra
info. I'm up and running with the platform and able to manage dojos and
events locally but emails don't seem to be sending. Should they work out of
the box or do I require any extra config/setup?
—
Reply to this email directly or view it on GitHub
https://github.com/CoderDojo/community-platform/issues/876#issuecomment-177145469
.

--
Ursula Clarke Everett
Technical Lead @ CoderDojo Foundation
S: tangentfairy | T: @CoderDojo | @tangentfairy
--
Volunteer at your local Dojo and inspire the next generation of digital
creators!

Pledge to start a Dojo https://coderdojo.com/pledge-a-dojo/
How to volunteer at your local CoderDojo
https://coderdojo.com/get-involved/volunteer/
Read the CoderDojo Foundation Strategy
https://coderdojo.com/news/2015/08/07/the-coderdojo-foundation-strategy-20152016/

--
Hello World Foundation operating as the CoderDojo Foundation is a company
limited by guarantee not having a share capital with charitable status, CHY
20812. Registered in Dublin, Ireland, No 524255. Registered Office:
Dogpatch Labs, Unit 1, CHQ Building, Custom House Quay, Dublin 1, Ireland.

Perfect - thanks for the quick reply :)

This is live now, thanks very much @johnbrett

Awesome! Glad to help!
On Tue, 9 Feb 2016, 17:53 Ursula Clarke Everett<EMAIL_ADDRESS>wrote:

Closed #876 https://github.com/CoderDojo/community-platform/issues/876.
—
Reply to this email directly or view it on GitHub
https://github.com/CoderDojo/community-platform/issues/876#event-544176825
.


Can we make it sleep?

I got a last minute idea for a hike I'm going on this coming weekend.
I have one of these ESP32-Cams and a cute little solar powered battery setup that is perfect for it. The place I'm going hiking is somewhere we go back to every year, and I was thinking of throwing up the camera and just let it sit for a year taking photos once every hour or so.
Think we can make it deep sleep and timer wakeup every x minutes, do a loop and take a photo and go back to sleep?

The light sleep issues are due to the camera driver.
Wrote a new sleep implementation using deep sleep. This only works for longer intervals between captures, > 10sec. Once an hour should be okay. It is now available on master(6795dbd8d2a3a11cf1a3c045c6f50755d85661f6).
I must warn tho that this firmware was born a couple of days ago. So don't expect it to be production ready or well tested. I have low expectations of it running flawlessly for a year without maintenance.
Also are you able to get any decent quality images from your ESP32-CAM of a landscape? mine seem to have a fixed focus of about 1 meter away. Things further away are rather vague.

I noticed you just started this thing out. My browsing around turned up a bunch of issues with resuming from sleep conflicting with the LED pin and others. I'll give it a go this afternoon and see what we can come up with.
I think you may have a misaligned camera. I don't have any issues with focus at mid-distance.

How does the interval and the sleep interplay here? I haven't gone through the code yet to see how you're sleeping and setting sleep, or if picture taking is a wake event.  Do I need to set interval as X and Sleep as X - 5seconds or something reasonable so that it wakes up just before the picture time?
Or is taking a picture a wake event so we only need to set the photo as 1 hr. or whatever greater than 10sec.

Just set the interval in the configuration file to '3600000'. The firmware will automatically go to sleep if the interval is big enough.

great!
So I got it loaded up but apparently there's something wonky here. Here's a serial debug from boot:
rst:0x1 (POWERON_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:1
load:0x3fff0018,len:4
load:0x3fff001c,len:928
ho 0 tail 12 room 4
load:0x40078000,len:8424
ho 0 tail 12 room 4
load:0x40080400,len:5868
entry 0x4008069c
�
Mounting SD card... Done
E (2495) sdmmc_req: sdmmc_host_wait_for_event returned 0x107
E (2496) sdmmc_cmd: sdmmc_read_sectors_dma: sdmmc_send_cmd returned 0x107
E (2496) diskio_sdmmc: sdmmc_read_blocks failed (263)
No config found, using defaults.
Failed to determine current date/time
E (3508) sdmmc_req: sdmmc_host_wait_for_event returned 0x107
E (3508) sdmmc_cmd: sdmmc_read_sectors_dma: sdmmc_send_cmd returned 0x107
E (3509) diskio_sdmmc: sdmmc_read_blocks failed (263)
Error reading directory /sdcard/
I've got camera.cfg in root of a fat32 4gb card. Seems to be mounting? I've got wifi and gnss //'d out in the config.h

You can try enabling WITH_SD_4BIT in config.h. else try a different SD card.

I did some power usage measurements. And the board still uses ~4mA @ 3.3V when in deep sleep (that is after commit c65ed3d3b02fac87e127f52215c2412c1f388618). This is almost all used by the camera module. At least removing the camera while in deep sleep reduces power consumption to about 0.4 mA.

Interesting.. my tests show in the neighborhood of 25mA in deep sleep with a peak at about 160mA at photo time.
I'll flash up c65ed3d and test again.

Do you measure at the 5 Volt? I removed the regulator and measure directly at the 3v3.

Measuring at the 5v off of USB since that will be real life usage when it's all complete and drawing off of the solar charger/battery.
New readings are 12mA Sleep and climbs to max of ~160mA for a second when active as the wake processes happens.
I switched SD cards and it works better, but so far I can get about 5 sleep cycles out of it before it locks up at:
Next image at: Thu Jan  1 00:05:38 1970
E (3483) sdmmc_req: sdmmc_host_wait_for_event returned 0x107
E (3483) sdmmc_cmd: sdmmc_read_sectors_dma: sdmmc_send_cmd returned 0x107
E (3484) diskio_sdmmc: sdmmc_read_blocks failed (263)
Error reading directory /sdcard/
I'll try the WITH_SD_4BIT

the WITH_SD_4BIT produces the flash constantly on at low when sleeping.
Also I've got the Flash set to off in the .cfg ( enable_flash = false ) and the flash still fires on photo. The SD seems more reliable now though..
Here's the serial dump of the config I have set
Mounting SD card... Done
Loading config...

'interval' => '50000'
'ssid' => ''
'password' => ''
'ntp_server' => 'pool.ntp.org'
'timezone' => 'GMT0'
'enable_busy_led' => 'true'
'enable_flash' => 'false'
'rotation' => '0'
'framesize' => '1600x1200'
'quality' => '10'
'contrast' => '0'
'brightness' => '0'
'saturation' => '0'
'colorbar' => 'false'
'hmirror' => 'false'
'vflip' => 'false'
'awb' => 'true'
'awb_gain' => 'true'
'wb_mode' => 'auto'
'agc' => 'true'
'agc_gain' => '1'
'gainceiling' => '2'
'aec' => 'true'
'aec2' => 'true'
'aec_value' => '51'
'ae_level' => '0'
'dcw' => 'true'
'bpc' => 'false'
'wpc' => 'true'
'raw_gma' => 'true'
'lenc' => 'true'
'special_effect' => 'none'
Config loaded.

Yes, the flash will light up when using 4-bit SD Card communication. That's because the flash led is connected to one of the data lines of the SD card. That is the reason that by default 1-bit SD card communication is used. Only way to disable flash LED with 4-bit SD card communication is to cut the trace to the LED, as described in this video https://www.youtube.com/watch?v=1ojUXSgMBVY

Since this thing is going to be installed out in nature, near other campers I probably don't want a flash going off once an hour.... I'll look into it

There seems to be a magic sequence to make everything happy. If I eject the card, hit reset and push it back in right away things seem to work. I'm going to leave it at that.
Before it gets installed in a tree, I'll leave it running for a few hours to make sure everything is as it should be. I've settled on a photo every 30 minutes, mostly because i'm not able to bring a laptop to modify the cfg file. Waiting an hour between photos to test it seems a bit much. I've got plenty of SD card space... and hopefully enough power over time.
My math works like this: 6000mAh battery.
A way over estimated average of a constant 20mAh draw. (It's probably more like 15 with booting up 2x an hour for 5 seconds from a 10-12mAh sleep draw) = around 12-14 days of battery power
The solar panel in full sun can provide ~3-400mAh of power. So even if I get just a few hours a sun a week this should be plenty to keep it going indefinitely.
Here's the real problem.. if it ever runs out of power and dies, then reboots the device probably won't ever come back up. I'm relying on the draw being small enough that there is always enough power left in it to do the task and keep it sleeping.
I'll send pix of it installed and running.

Cool, I'd love to see some images!

So I completely failed to bring my phone to take photos of us installing LOL.. the install area was too far away to go back and
get it. I did though get a photo from the other side of the lake.   The camera is installed on the other side of the lake. its in the first tree that the bleached white log is pointed at.

Thanks for the images. Certainly a nice place to make a time-lapse.


Image Similarity exported coreml spec includes two useless layer

Coreml model spec for image similarity has two useless layer:
softmax and fc1, which doesn't effect the result but should be removed.

@nickjong @jakesabathia2 Is this a 6.0 ship blocker?

@jakesabathia2 Is this a regression from 5.8?

The perf I believe isn't affected.
But 5.8 didn't have those two useless layers.

@jakesabathia2 Actually that makes sense now. Now, we are simply just using the coreml-ml model as is and it has those 2 layers. We should remove them.

Here is s sample output of model exported via TC 5.8 and 6.0. The interfaces are the same but the 2 layers does add some space to the .mlmodel but CoreML optimizes unused layers so it won't have impact on the app.

@nickjong @syoutsey Clearing this out of 6.0.

@TobyRoseman, I would like to work on this issue. Thank you!

@kritikadusad - ok, great. I've assigned this issue to you. Let me know if you have any questions.


Build failed: UserDataProfilesManifestSync > first time sync - incoming from server (no state)

Build: https://dev.azure.com/monacotools/a6d41577-0fa3-498e-af22-257312ff0545/_build/results?buildId=258318
Changes: https://github.com/Microsoft/vscode/compare/451333c...63a3873
Note that this is a unit test failure, so flakes are surprising.
  1) UserDataProfilesManifestSync
       first time sync - incoming from server (no state):
     Error: Timeout of 2000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (vs/platform/userDataSync/test/common/userDataProfilesManifestSync.test)

/duplicate
https://github.com/microsoft/vscode/issues/207468

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


docs: Make getting-started guide RBAC aware

Currently the getting started guide only creates a ClusterRole and a ClusterRoleBinding for the Prometheus operator but not for the Prometheus pods. This PR adds the steps to do so for the Prometheus pods as well.
Tested with tectonic-installer and minikube cluster.
Related to #290

@brancz I have updated RBAC related apiVersions to apiVersion: rbac.authorization.k8s.io/v1beta1, otherwise kubectl complains. What do you think?

@mxinden I think we should be good with upgrading the apiVersion for the RBAC resources, master should represent latest, which is 1.6 clusters for us now.

lgtmog

Test this please.


Toolbar GUI remains up when exiting map view

as the title says the GUI remains up when exiting map view

Fixed in https://github.com/StrikeForceZero/KSP-OrbitPOInts/releases/tag/v<IP_ADDRESS>


Enable basic matrix concatenation via the API

Hey let's merge it! Thanks @l5han, this was a ton of work!


TabBar horizontal layout fails when combined with ResponsiveVisibility bead

A TabBar cannot layout horizontally when a ResponsiveVisibility bead is used.
Steps to Reproduce

Create a TabBar with a ResponsiveVisibility bead

<j:TabBar width="100%"
    itemRenderer="renderers.TabBarHorizontalIconItemRenderer"
    dataProvider="{someDataProvider}">

    <j:beads>
        <j:ResponsiveVisibility phoneVisible="false"
            tabletVisible="true"
            desktopVisible="true"
            wideScreenVisible="true" />
    </j:beads>

</j:TabBar>

Expected Results
The TabBar displays horizontally and is hidden in phone sizes.
Actual Results
The TabBar displays vertically in tablet, desktop, and widescreen sizes.
Workaround
A suitable workaround would be to add the ResponsiveVisibility bead to another container surrounding the TabBar.

I can't remember why I set the display to block instead to flex maybe there was some issue at the time I stated Jewel but I don't see a problem now, so this should work always.


Added project list context menu.

Added context menu to project list on startup with option:

'Show In Explorer'

Boot startup window maximized (seemed to be correct size but offset weirdly on Windows).

Hey sorry, I only noticed after i merged but its merged into Main branch.
Main isn't currently being actively developed, Development is.

All good. My bad. I'll work off Dev and reimplement the changes.


didSelectDate not called ever

IMPORTANT - BEFORE CREATING YOUR ISSUE PLEASE READ THE FOLLOWING:

7.1.4
yes

I have this weird issue where the didSelectDate is not called, but the willDisplay is also not called. The calendar does show properly tough. I also set the delegate and datasource correctly.

  fileprivate let calendarView: JTAppleCalendarView = {
        $0.translatesAutoresizingMaskIntoConstraints = false
        $0.scrollDirection = .horizontal
        $0.showsVerticalScrollIndicator = false
        $0.showsHorizontalScrollIndicator = false
        $0.isPagingEnabled = true
        $0.backgroundColor = .white
        $0.minimumLineSpacing = 0
        $0.minimumInteritemSpacing = 0
        $0.register(CalendarCell.self)

        return $0
    }(JTAppleCalendarView(frame: .zero))

calendarView.calendarDataSource = dataSource
calendarView.calendarDelegate = dataSource

extension CalendarDataSource: JTAppleCalendarViewDataSource {
    func configureCalendar(_ calendar: JTAppleCalendarView) -> ConfigurationParameters {
        return ConfigurationParameters(startDate: dateFormatter.date(from: "2017 01 01")!, endDate: dateFormatter.date(from: "2017 12 31")!)
    }
}

extension CalendarDataSource: JTAppleCalendarViewDelegate {
    func calendar(_ calendar: JTAppleCalendarView, cellForItemAt date: Date, cellState: CellState, indexPath: IndexPath) -> JTAppleCell {
        let calendarCell = calendar.dequeueReusableCell(forIndexPath: indexPath) as CalendarCell
        self.calendar(calendar, willDisplay: calendarCell, forItemAt: date, cellState: cellState, indexPath: indexPath)
        return calendarCell
    }

    func calendar(_ calendar: JTAppleCalendarView, didSelectDate date: Date, cell: JTAppleCell?, cellState: CellState) {
        guard let calendarCell = cell as? CalendarCell else { return }

        calendarCell.selectedView.isHidden = false
    }

    func calendar(_ calendar: JTAppleCalendarView, didDeselectDate date: Date, cell: JTAppleCell?, cellState: CellState) {
        guard let calendarCell = cell as? CalendarCell else { return }

        calendarCell.selectedView.isHidden = true
    }

    func calendar(_ calendar: JTAppleCalendarView, willDisplay cell: JTAppleCell, forItemAt date: Date, cellState: CellState, indexPath: IndexPath) {
        guard let calendarCell = cell as? CalendarCell else { return }

        calendarCell.laDate.text = cellState.text
    }
}

I see that you have a return statement in that function if the cell is nil. Can you verify that the possibly the function is returning on that line of code?
    func calendar(_ calendar: JTAppleCalendarView, didSelectDate date: Date, cell: JTAppleCell?, cellState: CellState) {
        guard let calendarCell = cell as? CalendarCell else { return }

        calendarCell.selectedView.isHidden = false
    }

Also, will display cell is called only on special instances.
Can you let me know what you are trying to do with that function?

Hey, thank you for such a quick reply :) I implemented willDisplayCell accordingly to #553.
didSelectDate is not called at all, it's not getting called. The only thing I was trying to do is show an overlay view on the cell once the date is tapped. I tough the view was not letting the touches pass, so I even removed the selectedView but that didn't help either. I'm doing all of this in code, no .storyboards could that be a problem?

@diogoAntunes no. this lib works with both storyboards and code.
one thing though, can you remove this? Just a wild guess.
        $0.translatesAutoresizingMaskIntoConstraints = false

There should be no problems with your code.
If possible can you send a sample app that replicates this problem? Just an empty sample app with no company code in it.

@patchthecode Thank you so much for all your help, specially on a Sunday :) while I was making the sample project for you, I couldn't get it not to work so I turned to my project and eventually found the issue, I was presenting the calendar modally and I added a tap gesture to view of the ViewController so it was dismissing on tap's outside the calendar view, the problem with this was that the calendar view taps were being dismissed :)

Awesome man. Glad you figured it out.
Have a good one 🍻


fix(container): add alignment to td

The website touts Container as 'A layout component that centers all the email content.'
This can be kind of misleading, especially if you're using the email dev preview, because this is not what you'll see if you just try to wrap your content in this component. This is an attempt to address that.

We may need a way to pass this a props since you don't always need to be centered(?)

The Container component is not meant to center the contents always.
This could be a documentation change instead, along with a component change that would center the element based on props. Centering could also be done with other components though so this might not be necessary, I think it'd certainly be helpful since it isn't possible to directly change the td's props, but if we were to address every use case with the props the components would get too complicated.
On the docs though, what do you think would make the docs clearer than what we have currently?


Initial commit of 'promtool check-metrics', promlint package

Review appreciated.  There are obviously quite a few possible checks that could be done, but I wanted to at least lay something down for future work.
Fixes #1953 .

@brian-brazil Updated.

The definition of kelvins comes from the BIPM itself though?
http://www.bipm.org/en/publications/si-brochure/kelvin.html

@mdlayher: Very pleased to see this linter being added. 👍 to intent.

It sounds like the general agreement on IRC is to allow both:

metres and meters
kelvin and kelvins

I'll be back tomorrow to implement these changes and address more feedback.

Dang, should have read this comment first :) But nice, agreed. And great job doing this!

Updated usage string with short description and examples, as discussed on IRC.

One open question:
$ curl -s http://localhost:9100/metrics | promtool check-metrics
node_entropy_available_bits: use base unit "bytes" instead of "bits"

It would seem there are indeed places where it makes sense to allow "bits" instead of "bytes".  Thoughts?

I'd say let's remove the bits->bytes check for now and merge this. We can
always add it later.
On Wed, Apr 12, 2017 at 6:44 PM Matt Layher<EMAIL_ADDRESS>wrote:

One open question:
$ curl -s http://localhost:9100/metrics | promtool check-metrics
node_entropy_available_bits: use base unit "bytes" instead of "bits"
It would seem there are indeed places where it makes sense to allow "bits"
instead of "bytes". Thoughts?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/pull/2605#issuecomment-293716719,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAANaFsZCT0SWp7-835OkzbLqSt4-45Tks5rvUXNgaJpZM4M5aaq
.

The bits is important for network stuff, as otherwise we'll end up with a mix of bits vs bytes there.

Ran this on a couple targets at SoundCloud, looks good! It missed a metric where someone used ms to signify milliseconds, but I think there's a limit to what you can realistically check.

I plan on adding additional checks in the future. I don't think it would be unreasonable to make a list of common abbreviations and have it recognize them and suggest the full unit name.

Yeah, let's not aim to get everything in the first version (_total on non-counters, and _count on non-summaries are other ones to look out for)

This wouldbe great to merge before the 1.6 release.
On Thu, Apr 13, 2017, 11:41 Brian Brazil<EMAIL_ADDRESS>wrote:

Yeah, let's not aim to get everything in the first version (_total on
non-counters, and _count on non-summaries are other ones to look out for)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/pull/2605#issuecomment-293915136,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAANaHT5swkzOtpJHv5XnJ4IrGXjjOv1ks5rvjQAgaJpZM4M5aaq
.

Any more thoughts on bits vs bytes? I'll leave it up to the maintainers to find consensus, but I am leaning toward removing that check and perhaps making it stricter at some point later on if it makes sense, since not everything can be cleanly expressed in bytes. (See the entropy metric above)

I'd warn on bits. Entropy is a rare exception where it's not really plausible that someone would compare it with something in bytes, whereas that is a problem between network people (who use bits) and everyone else (who use bytes) which causes problems.

I'm against any rules which can't be enforced in 100% of all cases. Similar
to the corresponding go tools, this check should be usable in CI.
I'm not convinced on forcing all network operators to use bytes either, as
bits are their unit of choice. It's not common to compare network IO
metrics to things like disk space either.
I see that it will take some time to come to a consensus here and would
merge the first version without the bytes enforcement, which doesn't mean
we can't add it later.
On Thu, Apr 13, 2017, 12:36 Brian Brazil<EMAIL_ADDRESS>wrote:

I'd warn on bits. Entropy is a rare exception where it's not really
plausible that someone would compare it with something in bytes, whereas
that is a problem between network people (who use bits) and everyone else
(who use bytes) which causes problems.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/pull/2605#issuecomment-293931340,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAANaD_yilOAWAgr4tXSh2qLm3q2AXAsks5rvkDkgaJpZM4M5aaq
.

I'm not convinced on forcing all network operators to use bytes either, as
bits are their unit of choice. It's not common to compare network IO
metrics to things like disk space either.

I've done that. I've also wanted to compare network traffic as reported by a switch to http network traffic reported by an application, and been quite confused due to the units difference.

Even tho most network operators talk about bits, SNMP always says "octests", so I think warning bits -> bytes is fine.  I agree with @brian-brazil that being able to compare cross-domain via bytes is worth the hassle of having to * 8 in graphs where people would prefer to see bits.

An addition to my comment about bits/bytes.  @grobie pointed out that there are some use cases for bits that don't have a good natural requirement for bytes.  His example is node_entropy_available_bits, which would be somewhat funny expressed as a byte value.  Sure, you could have something like 452.125 bytes for 3617, but it's a little funky.
There are things that we may want to express as "warning", not "error".

My goal with these tools is to get them into a similar state as golint and
that people use them in all their CI pipelines to abort on violations. I'm
concerned that enforcing bits -> bytes isn't a rule which makes sense in
100% of all cases and will hinder adoption.
I haven't had considered the option to have different problem levels. The
go tools don't have such "soft" issues, but no objections to include it as
a non-critical (won't cause exit code to be != 0) suggestion.
On Thu, Apr 13, 2017 at 4:27 PM Ben Kochie<EMAIL_ADDRESS>wrote:

An addition to my comment about bits/bytes. @grobie
https://github.com/grobie pointed out that there are some use cases for
bits that don't have a good natural requirement for bytes. His example is
node_entropy_available_bits, which would be somewhat funny expressed as a
byte value. Sure, you could have something like 452.125 bytes for 3617,
but it's a little funky.
There are things that we may want to express as "warning", not "error".
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/pull/2605#issuecomment-293998666,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAANaK_NgqBZmoruA6d3jFFejVcjT09Mks5rvnc_gaJpZM4M5aaq
.

Yeah, I think allowing both bits and bytes makes sense in that case.
@grobie Note that golint is explicitly not a tool that you can fail CI pipelines on, from the README: "The suggestions made by golint are exactly that: suggestions. Golint is not perfect, and has both false positives and false negatives. Do not treat its output as a gold standard. We will not be adding pragmas or other knobs to suppress specific warnings, so do not expect or require code to be completely "lint-free". In short, this tool is not, and will never be, trustworthy enough for its suggestions to be enforced automatically, for example as part of a build process."

In reality, thousands of projects including us use go vet and go lint in
exactly these ways and abort their builds.
On Thu, Apr 13, 2017 at 4:38 PM Julius Volz<EMAIL_ADDRESS>wrote:

Yeah, I think allowing both bits and bytes makes sense in that case.
@grobie https://github.com/grobie Note that golint is explicitly not a
tool that you can fail CI pipelines on, from the README: "The suggestions
made by golint are exactly that: suggestions. Golint is not perfect, and
has both false positives and false negatives. Do not treat its output as a
gold standard. We will not be adding pragmas or other knobs to suppress
specific warnings, so do not expect or require code to be completely
"lint-free". In short, this tool is not, and will never be, trustworthy
enough for its suggestions to be enforced automatically, for example as
part of a build process."
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/pull/2605#issuecomment-294001448,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAANaIa_O8L3PT1ecNAg3yWVwgpQ_Eccks5rvnmbgaJpZM4M5aaq
.

They might, and I've seen it cause problems in the past.
Anyways, that doesn't really matter for the topic at hand here. I agree with your overall sentiment about promtool.

Guilty of using golint in CI here.
As for this issue, what do you all think about:

removing the check for the time being
opening a new issue to continue the discussion
merging the base of promtool check-metrics as-is

Would love to hear your thoughts since it doesn't seem like there is a strong consensus one way or another.

That's what I suggested above, so :+1: obviously.
On Thu, Apr 13, 2017, 17:18 Matt Layher<EMAIL_ADDRESS>wrote:

Guilty of using golint in CI here.
As for this issue, what do you all think about:

removing the check for the time being
opening a new issue to continue the discussion
merging the base of promtool check-metrics as-is

Would love to hear your thoughts since it doesn't seem like there is a
strong consensus one way or another.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/pull/2605#issuecomment-294010771,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAANaE59VfJ4eNzJXkJa20380xduACXfks5rvoMhgaJpZM4M5aaq
.

SGTM too.

Bits to bytes error removed for initial commit, per @grobie , @SuperQ, @beorn7 , and @juliusv .
Please take a look.

:+1: again, LGTM. I'll leave it to the repo maintainers to press merge.
On Thu, Apr 13, 2017 at 5:51 PM Matt Layher<EMAIL_ADDRESS>wrote:

Bits to bytes error removed for initial commit, per @grobie
https://github.com/grobie , @SuperQ https://github.com/SuperQ, @beorn7
https://github.com/beorn7 , and @juliusv https://github.com/juliusv .
Please take a look.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/pull/2605#issuecomment-294018197,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAANaIjl1DcLeztftMp0_pXEjTpk_6S6ks5rvorUgaJpZM4M5aaq
.

If the choice is between allowing a mix of bit and bytes, and having an exception for entropy I say we switch entropy to bytes.

The entropy example was just one, there are likely many other cases.

The discussion about a bits to bytes rule has been moved to https://github.com/prometheus/prometheus/issues/2617.

Was the addition of sort.SliceStable intentional? As this is a go1.8 library function. As a result, promlint can't be installed on any system with older go versions. e.g. go get github.com/prometheus/prometheus/cmd/promtool started failing for me in various environments after this got merged.

Yes, we decided to require go1.8 to build Prometheus. Promtool is not the
only place we started using the new functions in the sort package. Please
note that building and installing go programs are two different things. You
only need go1.8 to be installed on a single build machine, which could even
be your laptop. The produced binary can then be shipped to all other
machines. Also note that we provide already prebuilt binaries, just
download them from https://prometheus.io/download/.
On Thu, Apr 20, 2017 at 3:05 PM James Ravn<EMAIL_ADDRESS>wrote:

Was the addition of sort.SliceStable intentional? As this is a go1.8
library function. As a result, promlint can't be installed on any system
with older go versions. e.g. go get
github.com/prometheus/prometheus/cmd/promtool started failing for me in
various environments after this got merged.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/pull/2605#issuecomment-295685554,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAANaEdFlnaUilLXWux52N_alzWJqMo8ks5rxzv2gaJpZM4M5aaq
.

@grobie Okay, thanks :).


description_for_human in ai-plugin.json file generated by Kiota contains speciall characters  \u0026apos;

What are you generating using Kiota, clients or plugins?
API Client/SDK
In what context or format are you using Kiota?
Windows executable
Client library/SDK language
None
Describe the bug
Kiota generate a plugin, in description_for_human of ai-plugin.json file, ' will be converted to unicode \u0026apos; which is not necessary.

original text:
This is a sample Pet Store Server based on the OpenAPI 3.0 specification.  You can find out more about\nSwagger at [https://swagger.io](https://swagger.io). In the third iteration of the pet store, we've switched to the design first approach!\nYou can now help us improve the API whether it's by making changes to the definition itself or to the code.\nThat way, with time, we can improve the API in general, and expose some of the new features in OAS3.\n\n_If you're looking for the Swagger 2.0/OAS 2.0 version of Petstore, then click [here](https://editor.swagger.io/?url=https://petstore.swagger.io/v2/swagger.yaml). Alternatively, you can load via the `Edit > Load Petstore OAS 2.0` menu option!_\n\nSome useful links:\n- [The Pet Store repository](https://github.com/swagger-api/swagger-petstore)\n- [The source API definition for the Pet Store](https://github.com/swagger-api/swagger-petstore/blob/master/src/main/resources/openapi.yaml)

Expected behavior
' does not need to convert to \u0026apos;
How to reproduce

Load pet store OpenAPI spec file
Run generate -> generate a plugin
In the generated ai-plugin.json file, you can find special characters \u0026apos; inside description_for_human

Open API description file
https://raw.githubusercontent.com/swagger-api/swagger-petstore/master/src/main/resources/openapi.yaml
Kiota Version
1.18.24082901
Latest Kiota version known to work for scenario above?(Not required)
No response
Known Workarounds
No response
Configuration
No response
Debug output
Click to expand log
```

</details>

### Other information

_No response_

This is caused by the call to CleanupXMLString at https://github.com/microsoft/kiota/blob/f96fd861a19a5ebbccd9707c4c1c31dc27e4cedf/src/Kiota.Builder/Plugins/PluginsGenerationService.cs#L141


Background notifications not triggering in iOS (neither simulator not device)

Hi there, I can't seem to get the local notifications trigger on background; in Android it all works as intended, but so far I had no luck in iOS.
I've configured the plugin as per the instructions. The requireis placed in the app.module.tsand this is the code that triggers the notification:
This is called on the ngOnInit(obviously using nativescript-angular):
LocalNotifications.hasPermission().then(
            function (granted) {
                console.log(">>>> Permission granted " + granted);
                if (!granted)
                    LocalNotifications.requestPermission().then(
                        (grt) => console.log('>>>> Permission granted ' + grt)
                    )
            }
        )

        LocalNotifications.addOnMessageReceivedCallback(notificationData => {
            if (notificationData.id == 2)
                this.showPanicOnMap(notificationData.title);
            else if (notificationData.id == 3)
                this.isOnDangerZone(); // This performs some checks an then calls LocalNotifications.schedule()
        }

    // Then I call firebase.init()
    firebase.init({
            onMessageReceivedCallback: (message) => {
            // Logic to handle the notification.}
        })
            .then(() => console.info(">>>>> Firebase initialized"))
            .catch(err => console.info(">>>>> Firebase init error: " + err));
  );

The firebase notification does come through, because I can see it in the console; I also checked in the debugger and it is handled correctly.
Here are the Nativescript version specs:

Nativescript 5.0.2
tns-core-modules 4.2.1
tns-ios 4.2.0

Plugin is in the latest version.

Nevermind, it was a misconfiguration on Firebase console.


Modifying an Existing Token

I'm submitting a ...
[ ] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary

Modifying an Existing Token adding websites to token

https://github.com/solana-labs/token-list/pull/19776/commits/8ed9970708f2e95f5fcbb09c87a8843af3e441cf

website : https://www.thedriftinghobo.com/ ,
Twitter: https://twitter.com/DriftingHoboCo ,
Instagram: https://www.instagram.com/driftinghoboco/

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

please create a PR for that commit and link it here

duplicate token: token address AwrdDfbE7CP69kpiEctdtE2YJvr42kKw3rXiCYgFJp85 is already used
explanation / how to resolve: https://github.com/solana-labs/token-list#duplicate-token

I am just trying to update the information of a that token Already
created... how can I add a website at this point?
On Wed, Feb 23, 2022, 10:25 PM keone @.***> wrote:

duplicate token: token address
AwrdDfbE7CP69kpiEctdtE2YJvr42kKw3rXiCYgFJp85 is already used
explanation / how to resolve:
https://github.com/solana-labs/token-list#duplicate-token
—
Reply to this email directly, view it on GitHub
https://github.com/solana-labs/token-list/issues/19778#issuecomment-1049457934,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXSEQVCBTWPUG3YEE2EBF23U4WQMPANCNFSM5PC2Q36Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

on your local checkout (assuming that origin is git@github.com:DriftingHobo/token-list.git)
git remote add pub-origin git@github.com:solana-labs/token-list.git
git fetch pub-origin main
git rebase pub-origin/main
git push origin main -f

Not sure I follow... but I'll check it out.... then where can I add the
sites?
I appreciate your help.
On Wed, Feb 23, 2022, 11:06 PM keone @.***> wrote:

on your local checkout (assuming that origin is @.***:
DriftingHobo/token-list.git)
git remote add pub-origin @.***:solana-labs/token-list.git
git fetch pub-origin main
git rebase pub-origin/main
git push origin main -f
—
Reply to this email directly, view it on GitHub
https://github.com/solana-labs/token-list/issues/19778#issuecomment-1049474135,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXSEQVCCL7GKEHTYSWVIBXTU4WVF5ANCNFSM5PC2Q36Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


turnTowards not working on "up-left", "down-right" etc.

firstly, thanks for last commit
turnTowards with normal directions works now "up" "left" etc
I am working on isometric map and cross directions not working like this.gridEngine.turnTowards("player", "up-left");
Can you help me

Sure, I commented on the other bug: https://github.com/Annoraaq/grid-engine/issues/319#issuecomment-1312737266

ok I close this issue, I follow the otherone


CircuiTikZ in Manim automatically fills some unknown area

My problem
I was trying to use CircuiTikZ in LATEX with manim. For that, I added the CircuiTikZ package to tex_template.tex  and tried to run the code for a simple battery and resistor, although they appeared on the screen but they don't look good. Manim automatically fills some unknown area in the figure. I mean that it looks like the image provided and even the code does not produce any error.
Code:
textext = r"""\begin{circuitikz} \draw
(0,0) to[battery] (0,4)
	   to[resistor] (4,4) -- (4,0) -- (0,0)
;
\end{circuitikz}
"""

class MyCkt(Scene):
    def construct(self):
        t = Tex(textext)
        self.play(Write(t, run_time=5))
        self.wait()

Wrong display or Error traceback:

This same line of LATEX code works fine on the latex console and produces the required output as needed but the manim does not do so.
Additional information
I tried inverting the colours but the problem persists.

Open file manimlib/animation/creation.py and add the following codes below line 85
`
class DrawBorder(Animation):
CONFIG = {
"run_time": 2,
"stroke_width": 2,
"stroke_color": None,
"rate_func": smooth,
}
def __init__(self, vmobject, **kwargs):
    if not isinstance(vmobject, VMobject):
        raise Exception("DrawBorder only works for VMobjects")
    self.reached_halfway_point_before = False
    Animation.__init__(self, vmobject, **kwargs)

def update_submobject(self, submobject, starting_submobject, alpha):
    submobject.pointwise_become_partial(
        starting_submobject, 0, min(2 * alpha, 1)
    )
    if self.stroke_color:
        color = self.stroke_color
    elif starting_submobject.stroke_width > 0:
        color = starting_submobject.get_stroke_color()
    else:
        color = starting_submobject.get_color()
    submobject.set_stroke(color, width=self.stroke_width)
    submobject.set_fill(opacity=0)

`
and use DrawBorder(t, run_time=5) instead of Write(t, run_time=5)

I thought you were using the latest version of Manim.
But I am using the version 3 Feb 2019.
I will try to solve the problem later.


Second part of corrections

Pull Request Test Coverage Report for Build 18

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+10.6%) to 95.536%

Totals

Change from base Build 16:
10.6%

Covered Lines:
107

Relevant Lines:
112

💛  - Coveralls


Why does this library prevent . in the domain name?

Not allowing . in the domain name breaks the cookies for use with non-complient browsers when trying to use subdomains ie safari and edge
Feels like this is a bit opinionated, is there a reason why it's necessary to have this in this library?
https://github.com/ardatan/whatwg-node/blob/64e4be67a4516dd32fc479bc3d2190faa82edf2e/packages/cookie-store/src/CookieStore.ts#L57

It is part of the spec;
https://github.com/WICG/cookie-store/blob/main/index.bs#L1143


Calendar showing previous month for current page, but current page in stack has the correct month.

This just started happening in the past week, the calendar is showing the previous month.  You can see in the stack below that the current page is 2017-01-01 06:00:00 utc, but it is showing the prior month.  I have tried setting the current page in view did load, but that does not appear to work.

A brief bug description = showing previous month
Stack trace.

Integration method = cocoapods)
Full steps to reproduce.
Device modal and iOS version=  iPhone 7 10.0
Xcode version = Xcode 8.2.1
FSCalendar version =  FSCalenda 2.7.1
Appears to work fine in demo project

This issue is from local timezone and UTC time issue.
FSCalendar.currentPage gets current local date and time.
But it returns as UTC time. So If you convert currentPage for perfect operation.

@dapeng313 Thanks.  I can see how that.  Do you have a suggestion on how to handle that conversion?

Hi @mcdolejt  i having same problem. please help me


Factorize client and server tests

It would be nice to have a way to write tests that work for both peers.
The problem is tests would have to be both loose (what they accept) and exact (what they send) for each event.

Idea: connect a client software and a server software together, and only have checks on what they send.
But I am afraid that would be even less readable.


prefix= filtering is case insensitive

The implementation of endpoints that have the prefix= query parameter treats the prefixes in a case insensitive manner, for example conflating FMA and fma. The expected behavior is for the restriction to fma to only return matches against the lower case form.

This is false. The issue is cause by trying to use the set of all prefixes as a way to do negative queries, but that means that prefixes that expand to a common namespace also have to be excluded.
For example.
FMA: -> http://purl.org/sig/ont/fma/fma
fma: -> http://purl.org/sig/ont/fma/
Means that if you want to exclude FMA you also have to exclude fma, and any other prefixes that are shorter, which is unfortunate. I will submit another ticket about adding exclude_prefix support.


Add possibility to add systemd additionnal configuration settings.

Description
Add the possibility to add systemd addtionnal configuration settings using attributes.
Issues Resolved
None, basic improvement.
Check List

[] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

Since this PR is relatively old and the last commit did not pass all tests, I'm closing the PR. If an update is warranted, please rebase with the current version and reopen or resubmit a new PR.


Add device configuration for matter-fan

The matter-fan driver uses airCondtionerFanMode and its label is "Air Condtioner Fan Mode".
This is for changing a label to "Fan Mode"  from "Air Conditioner Fan Mode"

I believe this will solve the issue in the future when we can use this: https://github.ecodesamsung.com/iot/SmartThingsCapabilities/pull/351
@HunsupJung could we wait for the new capability to use here, or do we need to make this change now?

We can use Fan Mode capability that you made. Thank you 👍

@ctowns  When should we change the airConditionFanMode to the newly created Fan Mode capability?

@ctowns
Can we use this PR until creating new Fan Mode capability?


Feature request: add xterm-style select/copy/paste options

Description of the new feature/enhancement
WSL + Ubuntu  (+ an X server) is a light and awesome replacement to running Ubuntu inside a virtual machine (e.g. VirtualBox). Linux users also need a terminal that will mimic what they have been using for years (25 years in my case) in the way of selecting, copying and pasting text
I'm currently using PuTTY and WSLtty to achieve this goal and I thought I would give the Windows terminal a try, because it looks quite promising. Unfortunately:

there is no GUI yet for easily changing the options/settings. I understand this is a work in progress and I could edit the required json file
there does not seem to be a way (yet) to mimic all the usual (and I'd say de facto standard for some people, after so many years) xterm select/copy/paste behavior. I have checked the documentation and read the #1332 #524 and #1496 issues that seemed to address this problem, and were possibly closed a bit hastily

The PuTTY Selection settings screenshot below shows the xterm setting I'm using. It would be useful for a lot of people if there was a way to easily activate xterm-style behavior in the Windows terminal. Maybe this is already possible, but I have checked several sections of the Terminal documentation (tips and tricks, tutorials, Customize settings, ...) without any success

The WSLtty Selection and Mouse settings screenshots show similar settings

Try setting "experimental.input.forceVT": true (this should be added to the docs as you are right it isn't shown).

Also note middle paste is not yet available. See https://github.com/microsoft/terminal/issues/1553.

I have changed the default settings.json to look like this
[...]
    // If enabled, selections are automatically copied to your clipboard.
    "copyOnSelect": false,
    "experimental.input.forceVT": true,

    // If enabled, formatted data is also copied to your clipboard
    "copyFormatting": false,
[...]

and exited then opened a new Terminal, but the behavior seems very much like the default behavior

select text with left mouse button: copy
middle button: does nothing (should paste)
DOUBLE-click with right mouse button: paste
subsequent SINGLE-clicks with right mouse button: paste again (should extend or show context menu)

Oh, I just saw your extra note about No middle button support !
Is "experimental.input.forceVT": true just the same as "copyOnSelect": false for now? Or am I missing some subtle things?
I'm using Terminal version 1.2.2381.0 straight out of the Store, not the Preview

Yeah it isn't related in this case. When win32 input mode was added, the option helped alleviate the issue as a workaround. Just subscribe and upvote the issue above. It's known but not the priority (at least not until enough people upvote and ask on Twitter to bump priority of that issue.) Of course anyone is welcome to submit a PR resolving the issue.

Done (I think). I clicked on the "thumb-up"
Can my issue remain open till this is resolved and documented, if other users have the same request?
I had a hard time finding older (and unfortunately closed) issues describing what I need. It seems that we (Linux users) are not using exactly the same keywords or feature description as the Terminal developers

just search xterm-style or middle mouse. You'll find plenty closed issues. This is a dupe of the issue above and it's tracked there so this issue will be closed by a WT team member.

("experimental.input.forceVT": true should only be used if you're having keyboard input issues that impact text-mode applications; if you are, we'd like to know what they are because otherwise we'll never be able to address them.)

I removed that recommendation quickly once I realized it was different issue but unfortunately it was followed before I removed it.

Well, well, I still believe that this issue or a similar one should stay open till the Feature request in the title is resolved, unless of course somebody argues convincingly that the feature is unreasonable or undoable

I have searched the issues with is:issue xterm in:title  and is:issue middle in:title, and even is:open paste in:title  (and putty, wsltty, mintty) and all the relevant issues are closed! I may be wrong, but when there is an issue or a feature request and you don't find it in the open issues, it means that either nobody as reported it (and it is your duty to do it), or it has been solved (possibly in a more recent or beta version). You should not have to look for a closed issue in this particular case (I did, but it took me some time)
this is indeed very similar to Feature Request: Mouse/Touch/Pointer Bindings #1553 that you mentioned, but unfortunately the wording of the title is quite different! I'd say that my issue is a functionality request from the users' point of view, whereas the other issue if an implementation request from the developers' point of view, that can be used to fulfill my request
I've had a look at #1553. It mentions We definitely need a spec at the beginning, but xterm is only mentioned when citing my issue and the closed #7624. Well, there you have part of the spec : make it possible to reproduce xterm behavior! Besides the screenshots I have included above show all the different settings that similar terminal programs have, and which settings are used to reproduce xterm behavior :-) That's something that can probably go to the specs

The spec is needed to actually implement the behavior. Take a look at https://github.com/microsoft/terminal/tree/master/doc/specs. You'll start to see why, especially with some of the bigger features, why a spec is needed for this feature.

Impressive! I'm completely out of my depth there

Thanks for all the feedback. Here--I'll merge a bit of this discussion into the body of #1553 so that it comes up when you search for it. I appreciate that some of our workitems are developer-focused, but that's unfortunately the best way to make sure the dev team understands the scope of the problem :smile:

/dup #1553
I renamed the issue as well


docs: small fix to getting started

small suggestion to fix the code block from "Getting Started" section, AFAIK await can only be used inside an asynchronous function.

Can we merge to dev first?

Can we merge to dev first?

Absolutely. Apologies, one of my first so wasn't aware on etiquette for the repo yet.


refactor(interval): :recycle: declare and reassign dynamic notes in record

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
903

Relevant Lines:
903

💛  - Coveralls


Revise integration stage condition.

This PR revises the integration stage condition to rely more on the SetDevVersion variable.

/check-enforcer override


Update launch.json schema to add "qt"

This controls whether the debugger monkey-patches PyQt/PySide to detect and trace Qt threads (https://github.com/microsoft/debugpy/pull/308).
Valid values are:

"auto"
"none"
"pyside"
"pyside2"
"pyqt4"
"pyqt5"

The default is "auto", which tries to auto-detect this by import probing - this is the same as ptvsd 4 behavior.
Since that heuristic may be wrong if several different packages are installed side-by-side, the ability to specify the desired one explicitly is also provided.
"none" is provided, to give the users the ability to opt out of monkey-patching in case it breaks something.

/cc @luabud for the defaults and text.

This doesn't seem to support PyQt6 - the only way I could get breakpoints to trigger in a thread's run method was to manually add debugpy.debug_this_thread(). "pyqt6" isn't a valid value for the "qt" field; when setting "qt": "pyqt6" in my launch configs I get the error:
couldn't handle #1 request "launch" from Adapter:
             Invalid message: "qt" must be one of: ['none', 'auto', 'pyside', 'pyside2', 'pyqt4', 'pyqt5']```

Due to limited resources and lack of general interest, we are unfortunately closing this issue.


~Get Free V Bucks Codes Fortnite V Bucks Generator 2024 [ mJAo]

★Get Free V Bucks Codes Fortnite V Bucks Generator 2024 [ mJAo]
Are you tired of spending your hard-earned money on in-game currency for Fortnite? Look no further, because we have the ultimate solution for you! Introducing the Free V Bucks generator that will revolutionize the way you play the game.
🔴📺📱👉here get Free V Bucks
🔴📺📱👉here get Free V Bucks
Imagine never having to worry about buying V Bucks again. With this incredible generator, you can have unlimited access to V Bucks without spending a dime. That's right - no more emptying your wallet for virtual currency. Say goodbye to expensive in-game purchases and hello to endless possibilities in Fortnite.
But wait, it gets even better. Not only will you have access to unlimited V Bucks, but you will also unlock exclusive skins, emotes, and other in-game items that will make you stand out from the crowd. Imagine having the coolest gear without ever having to spend a penny. It's like a dream come true for any Fortnite player.
And the best part? This generator is completely safe and secure. You don't have to worry about getting banned or compromising your account. Our team of experts has developed a foolproof system that guarantees your safety while using the generator. You can generate V Bucks with peace of mind, knowing that your account is in good hands.
But you must act fast, as this offer won't last forever. Take advantage of this limited time opportunity to get Free V Bucks and elevate your Fortnite experience to a whole new level. Don't miss out on this once-in-a-lifetime chance to enhance your gaming experience without breaking the bank.
So what are you waiting for? to access the Free V Bucks generator and start enjoying all the benefits it has to offer. Say goodbye to expensive in-game purchases and hello to a world of endless possibilities in Fortnite. Don't miss out on this amazing opportunity - your gaming experience will never be the same again!"


Add the user, fill in the mailbox, the database already exists in the mailbox, the page does not have any tips

In GitLab by @luotian-github on Sep 14, 2017, 15:47
fix bug #188

In GitLab by @wknet123 on Nov 8, 2019, 07:36
Fixed with PR #303

In GitLab by @sokril on Nov 8, 2019, 08:10
closed

In GitLab by @sokril on Nov 8, 2019, 08:10
assigned to @wknet123


Stable/3.2.x

I am here to learn

Closing. Assuming a mistake.

See the docs for details on the repo and stable vs main branch https://docs.djangoproject.com/en/dev/internals/git/#stable-branches


Add a link to navigate to former citations

Modifications to do on Documentation page:

Add link to get to previous citations
Remove command line usage from Documentation

closed by PR #51


[x] generate JUnit reports and publish them as artifacts

Adds a couple new features: JUnit reporting, and printing out failing test results at the end.
Once this lands, every ci-test run will have two new artifacts associated with it: codegen-unit-test-results and unit-test-results. These artifacts will contain JUnit XML files with the status of each test. They are not presented in a user-visible format yet -- solving that will require some work to handle GitHub's security model. See this section for an example of some of the work that will need to be done.

LSR dependency change summary:
target packages:
  M anyhow 1.0.40 (direct third-party, crates.io)
    * version upgraded from 1.0.39
    * (unchanged features: default, std)
  M serde 1.0.125 (direct third-party, crates.io)
    * version upgraded from 1.0.124
    * (unchanged features: default, derive, rc, serde_derive, std)

host packages:
  M serde_derive 1.0.125 (transitive third-party, crates.io)
    * version upgraded from 1.0.124
    * (unchanged features: default)

LEC dependency change summary:
target packages:
  M anyhow 1.0.40 (direct third-party, crates.io)
    * version upgraded from 1.0.39
    * (unchanged features: default, std)
  M serde 1.0.125 (direct third-party, crates.io)
    * version upgraded from 1.0.124
    * (unchanged features: default, derive, rc, serde_derive, std)

host packages:
  M serde_derive 1.0.125 (transitive third-party, crates.io)
    * version upgraded from 1.0.124
    * (unchanged features: default)

release binaries dependency change summary:
target packages:
  M anyhow 1.0.40 (direct third-party, crates.io)
    * version upgraded from 1.0.39
    * (unchanged features: default, std)
  M serde 1.0.125 (direct third-party, crates.io)
    * version upgraded from 1.0.124
    * (unchanged features: alloc, default, derive, rc, serde_derive, std)

host packages:
  M anyhow 1.0.40 (transitive third-party, crates.io)
    * version upgraded from 1.0.39
    * (unchanged features: default, std)
  M serde_derive 1.0.125 (transitive third-party, crates.io)
    * version upgraded from 1.0.124
    * (unchanged features: default)

/land

@sunshowers :exclamation: This PR is still missing approvals, unable to queue for landing

LEC dependency change summary:
target packages:
  M anyhow 1.0.40 (direct third-party, crates.io)
    * version upgraded from 1.0.39
    * (unchanged features: default, std)
  M serde 1.0.125 (direct third-party, crates.io)
    * version upgraded from 1.0.124
    * (unchanged features: default, derive, rc, serde_derive, std)

host packages:
  M serde_derive 1.0.125 (transitive third-party, crates.io)
    * version upgraded from 1.0.124
    * (unchanged features: default)

release binaries dependency change summary:
target packages:
  M anyhow 1.0.40 (direct third-party, crates.io)
    * version upgraded from 1.0.39
    * (unchanged features: default, std)
  M serde 1.0.125 (direct third-party, crates.io)
    * version upgraded from 1.0.124
    * (unchanged features: alloc, default, derive, rc, serde_derive, std)

host packages:
  M anyhow 1.0.40 (transitive third-party, crates.io)
    * version upgraded from 1.0.39
    * (unchanged features: default, std)
  M serde_derive 1.0.125 (transitive third-party, crates.io)
    * version upgraded from 1.0.124
    * (unchanged features: default)

LSR dependency change summary:
target packages:
  M anyhow 1.0.40 (direct third-party, crates.io)
    * version upgraded from 1.0.39
    * (unchanged features: default, std)
  M serde 1.0.125 (direct third-party, crates.io)
    * version upgraded from 1.0.124
    * (unchanged features: default, derive, rc, serde_derive, std)

host packages:
  M serde_derive 1.0.125 (transitive third-party, crates.io)
    * version upgraded from 1.0.124
    * (unchanged features: default)

/land

/land p=high

@sunshowers :bulb: This PR is already queued for landing


RuntimeError: torch.nn.functional.binary_cross_entropy and torch.nn.BCELoss are unsafe to autocast

File "/export/software/anacondamini/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "/export/segment/transfiner/detectron2/modeling/roi_heads/mask_head.py", line 537, in forward
x, x_uncertain, x_bo, x_hr, x_hr_l, x_hr_ll, x_c, x_p2_s, encoder, instances, self.vis_period)
File "/export/segment/transfiner/detectron2/modeling/roi_heads/mask_head.py", line 296, in mask_rcnn_loss
pred_mask_logits_uncertain, gt_masks_uncertain, pred_mask_logits_uncertain.shape[0]) + F.binary_cross_entropy(pred_mask_logits_uncertain, gt_masks_uncertain, reduction="mean")
File "/export/software/anacondamini/lib/python3.7/site-packages/torch/nn/functional.py", line 2526, in binary_cross_entropy
input, target, weight, reduction_enum)
RuntimeError: torch.nn.functional.binary_cross_entropy and torch.nn.BCELoss are unsafe to autocast.
Many models use a sigmoid layer right before the binary cross entropy layer.
In this case, combine the two layers using torch.nn.functional.binary_cross_entropy_with_logits
or torch.nn.BCEWithLogitsLoss.  binary_cross_entropy_with_logits and BCEWithLogits are
safe to autocast.

how to solve it?thanks

you can refer to the solution here. or update your pytorch version to 1.7.1.


Settings Dialog

[x] Add a shortcut Ctrl+Alt+S

[x] Disable all wizards ?
[x] Before uploading a new path, check if it exists first, you should use the version to run as well.
[ ] Create folder by version
[ ] Copy version, and absolute path
[ ] Show menu items, hides when the screen width is small, it does not fit well with responsive.
[x] Allow change of the default download folder of the esptool.py bundle, instead of default tmp dir

[x] Send event with the new esptool.py path

[x] Create a ComboBox instead of TextField

[x] Save the states in repo for fill the ComboBox

[ ] Troubleshooting https://github.com/Fazecast/jSerialComm?tab=readme-ov-file#troubleshooting

[ ] allow change of the default download folder of the flash device read
[ ] Disable notificacion bell
[ ] export settings to a file ?
[ ] import settings to a file ?
[ ] Check latest version of esptool.py

Fixme please


open menu controller on full screen above navigation controller

1st Screenshot for ios and 2nd for android  but i have to implement menu display  like android screen shot in ios which menu bar shown above the navigation bar

got solution at time of addSubiew we add it to main window
change this line  self.view.addSubview(menuVC.view) to
UIApplication.shared.delegate.window?.addSubview(menuVC.view)


'common' not available error notification

@wyatt-howe @lapets
I keep getting an error like this below:

When I manually setup a 'common' folder. I receive another error about missing the 'Cargo.toml' file in the 'crypto' folder. And then another one and another one...

I just would like to try the demo of this private-id-web solution to see if I can work with it and use it to match certain datasets together via this solution. Could you perhaps elaborate on the steps to take since the README file is not sufficient for me.

Thanks for your interest in our adaptation of Private-ID!  It seems we had the submodule pointed at the wrong commit.  This is now fixed.  Since two years ago, Rust has lost support for the wasm versions we had initially used, but we updated this as well, and everything seems to build as it should.
I updated the README with some more notes on building.  Particularly, note that to properly run this between two parties in the browser you will need a setup Firebase or the like in the IO module.
Give it another try and see if that solves the issue!

Also, depending on how large your datasets are, you may prefer to use the original Private-ID implementing for scale.
Although with some amount of engineering effort, this could scale to millions of rows even in the browser.

@wyatt-howe
Thank you for the updates, I really appreciate it.
I am a couple of steps further now. I have two questions:

Now, it seems the path cannot be found. I tried several statements already with a variety of paths. But it does not seem to get the path properly though. What do I do wrong?
You mention Firebase, but what specific part (URL) of Firebase do I need to apply to? Since Firebase consists of multiple 'solutions'.

It would be great if you could help me.

In addition to #2: would a general web key be sufficient? --> https://firebase.google.com/docs/projects/api-keys

Certainly, I'm happy to help.
For (1) you might need to use copy rather than cp.  You're the first to try this on Windows.  You may also want to change the same command in package.json.
For (2), I'm referring to the Firebase project configuration which should be set as the value of the firebaseConfig JSON object in io.js here.  (We have it generated from an encrypted string, but you can just paste it in.)
Let me know if you're still experiencing difficulties.

OK, so I tried once more.
(1) I needed to manually copy the appropriate files into the folder since I could not find how to use 'cp'. 'copy' did not work either.
(2) It seems I cannot use 'cd src; cargo run --bin private-id-test' in one line. First, I needed to confirm 'cd src' and then, I needed to confirm 'cargo run --bin private-id-test'
This lead to the following result with version errors with zeroize:

(3) Regarding Firebase. I set this up and added the credentials to the io.js file. Do I need to host the Firebase app or may I use this demo locally, correct?

(1)  Good to know.  I have only ever built the library on macOS.
(2) Hmm, I'll have to look into the error with private-id-test, but you actually do not need to compile the test suite to run the demo.  The web/ directory should already be good to go now that you have copied over the .wasm and .js files.
(3) Correct, you can run everything locally, such as by running python -m http.server 3000 in the web/ directory and then pointing your browser to http://localhost:3000/.  That's all!  You can simulate two users by opening two tabs.  Hope this helps.

@wyatt-howe
(1) It would be great if you could take a look at private-id-test since it still does not work, unfortunately.
(2) It seems to be very hard to use python with Nodejs. I tried everything mentioned here and restarted multiple times as well but even with python installed, it does not recognize it when I tried python -m http.server 3000 in the web directory.
I managed to set this up via this solution though (Github repo).
Now, I see this:

How may I try test CSVs and use the solution? And how may I try things that are mentioned here? I.e. PS3I, Private Join and Compute etc. Is this the exact same set up as described for the FB solution? Could you elaborate, please?

@wyatt-howe
I tried the FB solution statements but this does more work, unfortunately. Could you maybe describe how to use an example json dataset to use in the test setup?

@wyatt-howe
Would be great if you could help me with a test drive..? :)

@wyatt-howe
Let me know.

@wyatt-howe
@lapets
I tried the Facebooks solution statements but this did not work, unfortunately. Could you maybe describe how to use an example dataset to use in the test setup? A description of some steps would really help me since now I am still not able to use this though.
Appreciate it, thank you in advance.


attributes that are lists of strings with a single member don't survive a round-trip

What happened:
If I create a Dataset with an attr that is a list of strings with only a single member, write it to netcdf and open it again, the attr is a single string instead of a list of strings.
What you expected to happen:
Writing the Dataset to netcdf and opening it should not change the attrs.
Minimal Complete Verifiable Example:
import xarray as xr

ds = xr.Dataset(attrs={"foo": ["bar"]},)

ds.to_netcdf("ds.nc", engine="netcdf4")
rd = xr.open_dataset("ds.nc", engine="netcdf4")

assert ds.attrs["foo"] == rd.attrs["foo"]

Anything else we need to know?:
This was implemented in PR2045, including the test test_setncattr_string. However, there assert_array_equal is used to compare the attrs after a roundtrip (line 1454 ), which unfortunately compares element-wise if one of the elements is a scalar. Changing the
test like this:
-                assert_array_equal(one_element_list_of_strings, totest.attrs["bar"])
+                assert one_element_list_of_strings == totest.attrs["bar"]

corrects it (the test then fails currently).
Environment:
Output of xr.show_versions()
INSTALLED VERSIONS
commit: f52a95cbe694336fe47bc5a42c713bee8ad74d64
python: 3.8.6 (default, Sep 25 2020, 09:36:53)
[GCC 10.2.0]
python-bits: 64
OS: Linux
OS-release: 5.8.0-36-generic
machine: x86_64
processor: x86_64
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8
LOCALE: en_US.UTF-8
libhdf5: 1.12.0
libnetcdf: 4.7.4
xarray: 0.16.3.dev75+gf52a95cb
pandas: 1.2.0
numpy: 1.19.5
scipy: None
netCDF4: <IP_ADDRESS>
pydap: None
h5netcdf: None
h5py: None
Nio: None
zarr: None
cftime: 1.3.0
nc_time_axis: None
PseudoNetCDF: None
rasterio: None
cfgrib: None
iris: None
bottleneck: None
dask: 2020.12.0
distributed: 2020.12.0
matplotlib: None
cartopy: None
seaborn: None
numbagg: None
pint: None
setuptools: 51.1.2
pip: 20.3.3
conda: None
pytest: 6.2.1
IPython: None
sphinx: None

There is an explicit condition for the shape to be > 1
https://github.com/pydata/xarray/blob/f52a95cbe694336fe47bc5a42c713bee8ad74d64/xarray/backends/netCDF4_.py#L275
However, even when setting the condition to > 0 it is saved as a string and not as a list of strings - maybe check if this can be achieved with netCDF4 at all.

There is an explicit condition for the shape to be > 1
https://github.com/pydata/xarray/blob/f52a95cbe694336fe47bc5a42c713bee8ad74d64/xarray/backends/netCDF4_.py#L275
However, even when setting the condition to > 0 it is saved as a string and not as a list of strings - maybe check if this can be achieved with netCDF4 at all.

Actually, this is a problem which is specific to the netcdf4 backend, using h5netcdf, this works:
import xarray as xr

ds = xr.Dataset(attrs={"foo": ["bar"]},)

ds.to_netcdf("ds.nc", engine="h5netcdf")
rd = xr.open_dataset("ds.nc", engine="h5netcdf")

assert ds.attrs["foo"] == rd.attrs["foo"]

and runs without problems.
As soon as I involve netcdf4 in reading or writing, this fails.
I don't know enough about netcdf's on-disk format to really debug what is going on. The ncdump output looks identical to me for both engines.

Actually, this is a problem which is specific to the netcdf4 backend, using h5netcdf, this works:
import xarray as xr

ds = xr.Dataset(attrs={"foo": ["bar"]},)

ds.to_netcdf("ds.nc", engine="h5netcdf")
rd = xr.open_dataset("ds.nc", engine="h5netcdf")

assert ds.attrs["foo"] == rd.attrs["foo"]

and runs without problems.
As soon as I involve netcdf4 in reading or writing, this fails.
I don't know enough about netcdf's on-disk format to really debug what is going on. The ncdump output looks identical to me for both engines.

@mikapfl That's by design in netCDF4:
https://github.com/Unidata/netcdf4-python/blob/bcbe3a0c62e6aa012b7afe7c9b2cfc9b2abc5424/src/netCDF4/_netCDF4.pyx#L1478-L1482

@mikapfl That's by design in netCDF4:
https://github.com/Unidata/netcdf4-python/blob/bcbe3a0c62e6aa012b7afe7c9b2cfc9b2abc5424/src/netCDF4/_netCDF4.pyx#L1478-L1482

@kmuehlbauer thanks for finding that out!
Now I'm wondering if that should be documented or some other solution; certainly, for me it was surprising, but maybe also my expectations were just wrong.

@kmuehlbauer thanks for finding that out!
Now I'm wondering if that should be documented or some other solution; certainly, for me it was surprising, but maybe also my expectations were just wrong.

@mikapfl This was added with https://github.com/Unidata/netcdf4-python/pull/597. But I can't say anything profound why the single element is extracted from the list.

@mikapfl This was added with https://github.com/Unidata/netcdf4-python/pull/597. But I can't say anything profound why the single element is extracted from the list.

Could add a note to the docstring...

Could add a note to the docstring...

h5netcdf aligned with netCDF4 in https://github.com/h5netcdf/h5netcdf/issues/116.

As this recently surfaced in #9699, it would be great to at least mention this in the docs. Those are the locations where attrs are mentioned.

[ ] https://docs.xarray.dev/en/latest/user-guide/data-structures.html
[ ] https://docs.xarray.dev/en/latest/getting-started-guide/faq.html#what-is-your-approach-to-metadata
[ ] https://docs.xarray.dev/en/latest/generated/xarray.Variable.html#xarray.Variable
[ ] https://docs.xarray.dev/en/latest/generated/xarray.DataArray.html#xarray.DataArray
[ ] https://docs.xarray.dev/en/latest/generated/xarray.Dataset.html#xarray.Dataset

Should we add this to the data structures or the faq section. We could then link to there from the three API docstrings.


Documentation for postgresql citus support

This experimental support was added under https://github.com/IBM/FHIR/issues/3437
Now we should document what that consists of and how to use it.  Probably at least 3 locations:

Updates to the schema design documentation at https://github.com/IBM/FHIR/blob/main/fhir-persistence-schema/docs/SchemaDesign.md
Updates to the schema tool usage guide at https://github.com/IBM/FHIR/blob/main/fhir-persistence-schema/docs/SchemaToolUsageGuide.md
Updates to the User's guide with what feature is and how to configure it
more?

I now see that #3683 does include some updates to the SchemaToolUsageGuide (number 2 in the description), its possible that this part is done (or at least started)

didn't see any mention in the user's guide

Citus now has its own section in the user's guide under <IP_ADDRESS> Supported databases.  Future enhancements can come as necessary.


BeanExplanationService -> UserBeansExplanationService

https://github.com/jabrena/spring-boot-user-beans/pull/91