prioritize_critical_css loads css twice

Version: <IP_ADDRESS>-0
Configuration:
pagespeed on;
pagespeed FileCachePath /dev/shm/ngx_pagespeed_cache;
pagespeed Statistics on;
pagespeed StatisticsLogging on;
pagespeed LogDir /var/log/pagespeed;
pagespeed MemcachedServers "<IP_ADDRESS>:11211";
pagespeed EnableCachePurge on;
pagespeed PreserveUrlRelativity on;
pagespeed InPlaceResourceOptimization off;
pagespeed InPlaceSMaxAgeSec 604800;
pagespeed ModifyCachingHeaders off;
pagespeed FetchHttps enable;
pagespeed CssInlineMaxBytes 4096;
pagespeed JsInlineMaxBytes 16384;
pagespeed ImageRecompressionQuality 70;
pagespeed WebpRecompressionQuality 70;
pagespeed WebpRecompressionQualityForSmallScreens 60;
pagespeed RewriteLevel CoreFilters;
pagespeed EnableFilters prioritize_critical_css,inline_javascript,resize_images,resize_mobile_images,remove_comments,collapse_whitespace,insert_dns_prefetch,hint_preload_subresources,recompress_images;

I noticed the CSS files were loaded twice, which means the prioritize_critical_css did not remove the original  tags.

but the 2 times css files are loaded are rewrited by pagespeed? aka have url like full-url-css-file.pagespeed.some-hash.css

Hmm No.

Could it be that you are observing browser prefetching of the original css?

I'm looking at the source code. Also Inspector indicates duplicate CSS rules exist.

prioritize_critical_css works removing the css file link, change it by a css snipet (those used above the fold) and then puy a javascript snipet at the bottom of the page wirh an onload event to load the rewrited css files.
You are viewing 2 css files both w/o any rewrite by pagespeed?
I ask for that because maybe you have a preload hint in the http headers, not in the code. In this case, the browser load these files 2 times, 1 not rewrited by pagespeed (cause pagespeed don´t rewrite link headers) and 1 more time because the css (rewrited by pagespeed) is loaded in the code.
These 2 files are different because in the header link is the original (https://yoursite.com/some-path/file.css) and in the html code  the file is loaded with a different url ( https://yoursite.com/some-path/file.css.ce.pagespeed.HASHCHAIN.css).
Can you share a url to test?

https://winterco.org
^^
I'm pretty sure I see the original stylesheet links.

Is there some form is html caching going on?
I'm curious what happens when cloudflare is taken out of the equation.

Well... you have allmost 2 big things to look in it:
The site has headers to preload some things, so when the html is rewrited by pagespeed you have 2 diferent files with the same content, 1 loaded by the header and other loaded by the html code. These preload headers can come from Cloudflare.

When the 2 files are the same, is loaded only 1 time but in chrome dev tools network tab in the initiator column you can see is loaded by "other"

The other thing to look is Cloudflare. Cloudflare is a proxy-cache approach to do CDN work. This mean that the only User Agent that hit the real server is the UA used by Cloudflare, so some optimizations from pagespeed are missed because it are UA dependent.
You need to configure Cloudflare in a way that the real UA from the final user hit the real server and headers from the real server go to the final user.
For example, in the html request (hhtps://winterco.org) pagespeed set a cache-control header max-age=0, no-cache, this header forbid to store the html in any cache, why? cause the html can be, or not, rewrited by pagespeed.
As a first step to make it work, you can put Cloudflare in by-pass mode, so the request from final users can to to the real server, once you got the server with pagespeed working well, the you enable Clodflare and face the issues that introduces.
If I remember, Cloudflare can be set to respect original headers.

Is there some form is html caching going on?
I'm curious what happens when cloudflare is taken out of the equation.

Yes, some html cache is done. The html is served with cache-control: max-age=3600, must-revalidate

Negative. No cloudflare this time:
root@localhost ~ # curl -vv --insecure --resolve "winterco.org:443:<IP_ADDRESS>" https://winterco.org
* Expire in 0 ms for 6 (transfer 0x55d944151f50)
* Added winterco.org:443:<IP_ADDRESS> to DNS cache
* Hostname winterco.org was found in DNS cache
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Expire in 200 ms for 4 (transfer 0x55d944151f50)
* Connected to winterco.org (<IP_ADDRESS>) port 443 (#0)
...(garbage)...
> GET / HTTP/2
> Host: winterco.org
> User-Agent: curl/7.64.0
> Accept: */*
>
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
* old SSL session ID is stale, removing
* Connection state changed (MAX_CONCURRENT_STREAMS == 128)!
< HTTP/2 200
< server: nginx
< date: Thu, 16 Jul 2020 09:40:51 GMT
< content-type: text/html; charset=UTF-8
< cache-control: max-age=3600, must-revalidate
< vary: Accept-Encoding, Cookie
< hummingbird-cache: Served
< x-content-type-options: nosniff
< x-xss-protection: 1; mode=block;
< x-powered-by: ASP.NET
< x-aspnet-version: 2.0.50727
< x-aspnetmvc-version: 2.0
< x-recruiting: Like hacking? Join us: winterco.org/careers
< x-page-speed: <IP_ADDRESS>-0
< link: </wp-content/themes/creativeily-winterco-mod/style.css?ver=5.4.2>; rel=preload; as=style; nopush
< link: </wp-content/uploads/bws-custom-code/bws-custom-code.css?ver=5.4.2>; rel=preload; as=style; nopush
< link: </wp-content/themes/creativeily-winterco-mod/assets/js/creativeily.js?ver=5.4.2>; rel=preload; as=script; nopush
< link: </wp-content/themes/creativeily-winterco-mod/assets/js/accessibility.js?ver=5.4.2>; rel=preload; as=script; nopush
< link: </wp-includes/js/wp-embed.min.js?ver=5.4.2>; rel=preload; as=script; nopush
<
<!DOCTYPE html>

<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1">
<link rel="profile" href="http://gmpg.org/xfn/11">
<title>凛冬华人联盟 – Winter Coalition</title>
<link rel='dns-prefetch' href='//www.googletagmanager.com'/>
<link rel='dns-prefetch' href='//fonts.googleapis.com'/>
<link rel='dns-prefetch' href='//cdn.jsdelivr.net'/>
<link rel='dns-prefetch' href='//recaptcha.net'/>
<link rel='dns-prefetch' href='//fonts.gstatic.com'/>
<link rel='dns-prefetch' href='//ajax.googleapis.com'/>
<link rel='dns-prefetch' href='//google-analytics.com'/>
<link rel='dns-prefetch' href='//www.google-analytics.com'/>
<link rel='dns-prefetch' href='//ssl.google-analytics.com'/>
<link rel="alternate" type="application/rss+xml" title="凛冬华人联盟 » Feed" href="https://winterco.org/feed/"/>
<link rel="alternate" type="application/rss+xml" title="凛冬华人联盟 » 评论Feed" href="https://winterco.org/comments/feed/"/>
<link rel='stylesheet' id='wp-block-library-css' href='https://cdn.jsdelivr.net/gh/WordPress/WordPress@5.4.2/wp-includes/css/dist/block-library/style.min.css?ver=5.4.2' type='text/css' media='all'/>
<link rel='stylesheet' id='comment_styles-css' href='https://cdn.jsdelivr.net/wp/plugins/wp-discourse/tags/2.0.6/lib/../css/comments.min.css?ver=1594583367' type='text/css' media='all'/>
<link rel='stylesheet' id='creativeily-google-fonts-css' href='https://fonts.googleapis.com/css2?family=Roboto%3Aital%2Cwght%400%2C400%3B0%2C500%3B0%2C700%3B0%2C900%3B1%2C400%3B1%2C500&display=swap&ver=5.4.2' type='text/css' media='all'/>
<link rel='stylesheet' id='dashicons-css' href='https://cdn.jsdelivr.net/gh/WordPress/WordPress@5.4.2/wp-includes/css/dashicons.min.css?ver=5.4.2' type='text/css' media='all'/>
<link rel='stylesheet' id='creativeily-style-css' href='https://winterco.org/wp-content/themes/creativeily-winterco-mod/style.css?ver=5.4.2' type='text/css' media='all'/>
<link rel='stylesheet' id='bws-custom-style-css' href='https://winterco.org/wp-content/uploads/bws-custom-code/bws-custom-code.css?ver=5.4.2' type='text/css' media='all'/>

<link rel='https://api.w.org/' href='https://winterco.org/wp-json/'/>
<link rel="EditURI" type="application/rsd+xml" title="RSD" href="https://winterco.org/xmlrpc.php?rsd"/>
<link rel="wlwmanifest" type="application/wlwmanifest+xml" href="https://winterco.org/wp-includes/wlwmanifest.xml"/>
<meta name="generator" content="WordPress 5.4.2"/>
<meta name="generator" content="Site Kit by Google 1.11.1"/>

<link rel="icon" href="https://winterco.org/wp-content/uploads/2020/07/cropped-11355daf357ecd7cd74ad2c64b3de3e4b15ba35c-32x32.png" sizes="32x32"/>
<link rel="icon" href="https://winterco.org/wp-content/uploads/2020/07/cropped-11355daf357ecd7cd74ad2c64b3de3e4b15ba35c-192x192.png" sizes="192x192"/>
<link rel="apple-touch-icon" href="https://winterco.org/wp-content/uploads/2020/07/cropped-11355daf357ecd7cd74ad2c64b3de3e4b15ba35c-180x180.png"/>
<meta name="msapplication-TileImage" content="https://winterco.org/wp-content/uploads/2020/07/cropped-11355daf357ecd7cd74ad2c64b3de3e4b15ba35c-270x270.png"/>

Skip to content

...(garbage)...

<link rel='stylesheet' id='wp-block-library-css' href='https://cdn.jsdelivr.net/gh/WordPress/WordPress@5.4.2/wp-includes/css/dist/block-library/style.min.css?ver=5.4.2' type='text/css' media='all'/><link rel='stylesheet' id='comment_styles-css' href='https://cdn.jsdelivr.net/wp/plugins/wp-discourse/tags/2.0.6/lib/../css/comments.min.css?ver=1594583367' type='text/css' media='all'/><link rel='stylesheet' id='creativeily-google-fonts-css' href='https://fonts.googleapis.com/css2?family=Roboto%3Aital%2Cwght%400%2C400%3B0%2C500%3B0%2C700%3B0%2C900%3B1%2C400%3B1%2C500&display=swap&ver=5.4.2' type='text/css' media='all'/><link rel='stylesheet' id='dashicons-css' href='https://cdn.jsdelivr.net/gh/WordPress/WordPress@5.4.2/wp-includes/css/dashicons.min.css?ver=5.4.2' type='text/css' media='all'/><link rel='stylesheet' id='creativeily-style-css' href='https://winterco.org/wp-content/themes/creativeily-winterco-mod/style.css?ver=5.4.2' type='text/css' media='all'/><link rel='stylesheet' id='bws-custom-style-css' href='https://winterco.org/wp-content/uploads/bws-custom-code/bws-custom-code.css?ver=5.4.2' type='text/css' media='all'/>

* Connection #0 to host winterco.org left intact

The Cache-Control on the header is likely provided by Hummingbird, a plugin of WordPress and it does not interfere with PageSpeed.
These Link header are provided by hint_preload_subresources I believe.

Note that there is two set of <link>, one in  and the other in . Also  got repeated. DevTools says:

though I have totally no idea why they are all tagged as
    <link rel="stylesheet" href="small.css">

      Hello, world!

And the resource small.css is like this:
  .yellow {background-color: yellow;}
  .big { font-size: 8em; }
  .bold { font-weight: bold; }

We expect PageSpeed to rewrite it into:

      Hello, world!

<link rel="stylesheet" href="small.css">

However, now we are getting this instead:

    <link rel="stylesheet" href="small.css">

      Hello, world!

<link rel="stylesheet" href="small.css">

As far I can see your site is working, pagespeed does their job, css files are loaded once, css abobe the fold is inlined in the html....
And yes, some css rules are set twice, 1 time in the inlined css and 1 more time in the css files.

And using a browser, so Cloudflare is here.

So why is it link css files multiple times? I understand that it extracts a few rules in


x/tools/gopls: TestSimultaneousEdits flaking on Kokoro with /gopath/pkg/mod: directory not empty

Just observed the following flake:
https://source.cloud.google.com/results/invocations/747b77b4-86df-4738-8ebf-9c38009a1fc0/targets/golang%2Ftools%2Fgopls-legacy%2Fpresubmit-114/log
It looks like we may have a race during cleanup of the shared directory.

I believe this should be resolved now. Tentatively closing.


fix(stream): Update "IterableReadableStream" pull method to enqueue nullish data (e.g. empty string)

This PR is to address the issue our team identified while we were integrating BaseLLM with our own AI to enable the streaming capability. The issue we faced was, after we implemented *_streamResponseChunks generator function, we noticed it did not terminated in many cases.
After investigation, we found out that our response stream would end by yielding empty string, and this behavior, paired with IterableReadableStream's pull method, which only enqueue's value if it is not nullish, will hang the generator. Hence preventing it from terminating successfully.

Phenomenal catch - I think this makes more sense to check for !== undefined though?
I'll update.

Huge fix, thanks!


Add SchNetPack Interface

To allow future comparisons between neural networks we need an interface between SchNetPack and Q|R.
SchNetPack: https://github.com/atomistic-machine-learning/schnetpack
Documentation: https://schnetpack.readthedocs.io/en/stable/

[ ] schnetpack is only written in Python 3. This is blocking us at the moment.

[ ] add schnetpack_qr.py interface for ASE type calculator.

[ ] add unit tests to ensure energy and gradients are correct

[ ] add schentpack to phil in qr.py

[ ] add different models into quantum.methods keyword.

stale


FaloopIntegration no longer working

You get connected but no S-ranks are ever shown. Did the API change?

Hmmm, I recently got back into FFXIV and the plugin certainly doesn't seem to be working.
I'll look into it tomorrow.

Fixed in 6.57.1. https://github.com/SlashNephy/Divination/commit/6dab5afb05c45510c7cf943c653f90a85d1760ae


GitLabCommitCollector needs performance improvement

Due to the nature of GitLab project searching, finding the correct project id to gather commit metrics is extremely time consuming and has the potential to make the prometheus scraper timeout (default scraper timeout value is 10 seconds).

When I originally wrote the collect it had a map to cache project ids.  I'm not sure why I removed it but I did.  The would help as you'd only have to do the search 1x.

When I originally wrote the collect it had a map to cache project ids.  I'm not sure why I removed it but I did.  The would help as you'd only have to do the search 1x.

@themoosman I seem to remember that being in the code as well.  Okay, I will work on adding it back in.

Testing made easy:

In the pelorus dir create app.py

cat <<EOF > app.py
#!/usr/bin/env python3
print("Hello World")
EOF

chmod +x app.py

Create namespace for build and run the build, annotate with existing gitlab url and hash

export NS=buildtest
export BUILD_NAME=gitlab-binary-build

oc create namespace "${NS}"
oc config set-context --current --namespace "${NS}"

oc new-build python --name="${BUILD_NAME}" --binary=true
oc label bc "${BUILD_NAME}" app.kubernetes.io/name="${BUILD_NAME}"

oc start-build bc/"${BUILD_NAME}" --from-file=./app.py --follow

oc annotate build "${BUILD_NAME}"-1 --overwrite \
io.openshift.build.commit.id=b807fd8e1b2bd1755eca14960e5352fe89ff466e \
io.openshift.build.source-location=https://gitlab.com/mpryc/pelorus-gitlab

Run the gitlab collector with the given build/namespace

export GIT_PROVIDER=gitlab
export NAMESPACES="${NS}"
python exporters/committime/app.py

Confirm the timestamp was correctly "collected"

07-21-2022 13:19:56 INFO     Collected commit_timestamp{ namespace=buildtest, app=gitlab-binary-build, commit=b807fd8e1b2bd1755eca14960e5352fe89ff466e, image_sha=sha256:cc4e6a5c159edb6477d7b07f8a82a1708201a4a218151fd1f1616d7a2daeafaa }<PHONE_NUMBER>.0


Sparkfun Artemis and CS320 pyranometer

Hi All, I've come across a sensor that communicates through SDI-12 and would be great to use a microcontroller to log the data.
Has anyone got experience using the CS320 pyranometer (Campbell Scientific) with the SDI-12 library?
Also, any experience with the SDI-12 library and the Sparkfun Artemis? @jerabaul29
Any feedback much appreciated

I had a look, it seems like a bit of "board-specific definitions" is needed to make things work:
https://github.com/EnviroDIY/Arduino-SDI-12/blob/fd9699b390edeac3a8681e2a6d4fe2ba8b1f9a51/src/SDI12_boards.h#L389-L391
If we could get some help here it would be really great. Any hope @Wenn0101 that someone at SF could help us on that? We are quite big buyers of the AGTs for geophysical measurements (and we intend to use AGTs instead of Campbell systems to log / transmit data about these sensors in the future), and we are trying to push more people around us to go for these, if you can help us get this SDI-12 stuff to work this is more customers from Met organizations around the world :) .
Just putting the datasheet link: https://s.campbellsci.com/documents/us/manuals/cs320.pdf .

I'm just wondering if it also needs SDI12Timer:: class instances for that board in SDI12_boards.cpp
I've been looking at how to do this on  Cortex-M4 SAMD51 (Seeed Wio Terminal). The registers/implementation for the SAMD51  TimerCounter are different than that of the SAMD21.
For a fast processor, and since the whole approach is port bit-banging, the ESP32/ESP8266 has perhaps a generic approach.
From SDI12.h:

@note  The ESP32 and ESP8266 are fast enough processors that they can take the
time to read the core 'micros()' function still complete the other processing needed
on the serial bits.  All of the other processors using the Arduino core also have the
micros function, but the rest are not fast enough to waste the processor cycles to
use the micros function and must use the faster assembly macros to read the
processor timer directly.

SDI12_boards.cpp:  ESP32/8266 approach is pretty simple

void         SDI12Timer::configSDI12TimerPrescale(void) {}
void         SDI12Timer::resetSDI12TimerPrescale(void) {}
sdi12timer_t SDI12Timer::SDI12TimerRead(void) {
// Its a one microsecond clock but we want 64uS ticks so divide by 64 i.e. right shift 6
return ((sdi12timer_t)(micros() >> 6));
}

Aaah, that would be very convenient if KISS bit banging with the micros function would be enough :) .
The Artemis runs at 48MHz with a boost mode (that works the same, just more power draw) at 96MHz. Any idea if that would be enough? :) .

Any idea of a cheap SDI-12 device we could use just to test the communications? :) .

Yes the KISS is a good baseline.  I would guess 48MHz should have a fast enough cycle time,... but its all in the trying it out. Thanks to whoever did the ESP32 for thinking it out .....
I'm trying it with a SAMD51, 120MHz (https://wiki.seeedstudio.com/Wio-Terminal-Getting-Started/)
and using the front right hand port with a Seeed connector, that maps to  pin A1
For generic test I'm pressing an old 16bit version of https://vegetronix.com/Products/SDI-12-Sensor-Translator/ I've set it to address 1 in the past when testing with EnviroDIY Modular sensors.
However it needs 5.5V.  So I've got a booster from the WioT 3.3V to +12V.
Starting with the i_SDI_12_interface.cpp, and modified similar to ESP32. It has compiled, downloaded and emitting SDI-12 protocol type characters, but the timing is  off.

Maybe I should switch to a repeated a_wild_card.ino

welll... so a quick report, but it doesn't look good.
the CortexM devices have peripheral routing by the manufacturer. The ESP32 is RISC ~ which is the KISS principal, and the persumably delay() works, perhaps it has simple port perhipherals.
I guess you need to check with the Artemis how it does it.
For the SAMD51 my research on simple timing and bit-banging a port isn't encouraging and not what I expect.
Going back to basics, I set up a simple bit toggle, to generate a square wave, controlling the low and the high periods using a delay.
There are various sources of the timing that are available for the SAMD51 .platformio\packages\framework-arduino-samd-seeed
Using a standard;
delay(1) ms  squ wave low is 949uS & high is 1047uS as measured by the Salae logic analyzer running at 100Mhz - I consider that pretty bad accuracy but its seems to get wors
For a custom delay_usec() to better control registers accessed, at the 1200baud bit rate of 866uS
delay_usec(866) low 813.4uS & high 922.2uS total time 1735uS
however at the SDI-12 bit sampling rate of 64uS
delay_usec(64) low 11.2uS & high 120.7uS   ~ total 132.8uS  - very skewed. Not good.

Hi @neilh10 ,
Many thanks for  your comments. Ok, interesting to hear that things do not look too good. We will discuss with @jvoermans if we try to find an alternative or go for SDI-12 interfaced sensors, we will let you know.

There are many timers on the Sparkfun Artemis boards, so an option too is to go for the more involved "timer solution". Just for reference (in addition to the datasheet):
https://forum.sparkfun.com/viewtopic.php?t=52254
https://forum.sparkfun.com/viewtopic.php?t=51868
@neilh10 if you would be interested to look into this we could ship to you an Artemis RedBoard ( https://www.sparkfun.com/products/15444 ), which is a not too expensive board that uses the same processor as the whole Artemis family. Of course no pressure whatsoever though, we perfectly understand if you are not interested in supporting a multitude of boards and architectures :) .

@jerabaul29 I checked with the PlatformIO environment and it doesn't support the Sparkfun Artemis board. I have been using the https://platformio.org/ IDE,  (using Visual Code) which when it supports a target is a very easy turn-up environment and supports a number of targets.
All I could really do would be to test the ability to generate a square wave from the internal delay loops. The target is the 1200Baud x 13 internal sampling or 64uS square wave. If that isn't accurate, within the SDI-12 spec of 10%, (say measured target of 5%) then the SDI-12.cpp isn't likely to work.
The multitude of timers, technically fantastic, often require a detailed driving knowledge.
There is a test setup https://github.com/addreas/platform-apollo3blue, I could try later and see if it goes in.
I am using ModualrSensors with the SDI12, with target deployments on https://monitormywatershed.org/ so that is mostly what I'm targeted on. The Artemis Apollo3 board doesn't seem a good fit with simple field deployment with no battery.
I'm still puzzling over the results I got with the SAMD51, so might try it on the similar SAMD12, which is supported.
Unfortunately this is the story with embedded microcontrollers, very fractured environemtn.
The other option for an accurate wave generation is to use the DMA and investigating how the interface to tri-colour leds work, They require very accurate edges to be generated.

Yeah, platformio and the Artemis is a bit of a complicated story, but there is a good platformio support now - though this is community contributed and requires manual install:
https://github.com/nigelb/platform-apollo3blue
Ok, thanks. Will try to get my hand on a SDI-12 device and test. Really sad / painful with the "fractured" of the specs, library, etc, indeed as you say.

Super nice website by the way :) .

We rather use our systems for measurements of sea ice cover in the Arctic and Antarctic, so this is why we use these MCUs - since there is a development board version with iridium and GPS all on the same board that we like quite well ( https://www.sparkfun.com/products/16469 ). The nice thing with these Artemis boards is that they are based on these Ambiq processors that use around 0.5mA at 3.3V when running at full 48MHz with everything on :) .

Gosh thanks for the insights, and link to  nigelb/platform-apollo3blue.  I'll see if it goes in as easy as it suggests.
My career is with embedded MCUs, first with Motorola D2 6800 kit with a teletype interface, then intel 8048 (RIP silicon souls).
The 0.5mA is very nice as is the integration with Artemis Global tracker.
Looking at the Apollo3, for me its lacking in UART and integrated USB (Host open source is still a mirage).   Apollo4 seems to include 4 UARTs and a USB 2. .
Also interested in your blog https://jerabaul29.github.io/jekyll/update/2020/12/20/using-sparkfun-artemis.html on Arduino mbed version.
monitormywatershed.org is a project of the Stroud Water Research center, and they have enviroDIY.org to encourage users and a mega1284 board Mayfly with the SDI-12. I'm a user, and have my own fork of the software https://github.com/neilh10/ModularSensors

Nice! I (and @jvoermans ) are working in Geosciences (in opposite parts of the world), measuring sea ice dynamics in the Arctic and Antarctic. We are developing our own instrumentation (which we open source when its mature, but working on a new "v2021" that is not open yet, still too early phase), since it can cut costs by a factor 10 to 15 compared with commercial solutions, and it makes a big difference for the science we work on if we have 1 or 20 measurement points...
The ironic part is that in the project I work with now, I actually decided to stick to the core v1 which is a "simple C++ main program running" and not using the RTOS, since a bunch of stuff was not really stable when I started the project back then. But I think that the MbedOS based arduino core is getting much better these days. This is the nice thing with the SF Artemis platform, there is an actual company behind and a relatively large users community, so that there is tooling examples etc made available.

@jerabaul29   I followed the  https://github.com/nigelb/platform-apollo3blue
Just wondering Is that the environment you've tried at all.?
Its almost compiled, except for finding interrupts() nointerrupts()
I've tried the ARTEMIS_THING_PLUS as I've used the feather footprint before.
 If you have any issues, please report these directly on the platformio repo :) .
Another thing: you can choose either to use the core v1 or v2 with platformio. The people I know all use the core v1, no idea how stable the core v2 (the one with RTOS) is. If you want to discuss this further, may you open an issue at nigelb's repo (feel free to tag me there if you want :) ).


develop State Estimation message passing for Valkyrie

@gizatt : I've summarised the messaging we are looking to put in place here:
https://github.com/openhumanoids/oh-distro/wiki/State-Estimation-Messaging
I'll try to run your translator while manually moving the robot on the winch to get a log to patch all this up.
This is also related ( @raluca-scona ):
https://github.com/openhumanoids/main-distro/issues/2285

Few questions:

Are NASA_CORE_ROBOT_STATE and CORE_ROBOT_STATE the same? If so, is there a reason to keep NASA_CORE_ROBOT_STATE around?
Are FORCE_TORQUE and NASA_FORCE_TORQUE the same? (same followup)
Should the val driver generate POSE_BDI, even if it's just a (bad) guess?
How is MICROSTRAIN_INS populated? (Pick a particular INS arbitrarily?)

In order:

I was going to get rid of CORE_ROBOT_STATE as all other signals from NASA are called NASA_*****. But I could aso just leave out the NASA_ instead. What's your vote?
Same response
POSE_BDI is currently its needed by Pronto (long story). Actually its just the orientation that's needed. I really need to do this. But I can assemble that from the orientation estimate that is in the pelvis IMU message.
For now lets publish the orientation from pelvisRearImu as POSE_PELVISREARIMU
No, lets publish all available INS sensors for now and fix it up in the cfg file: NASA_INS_pelvisRearImu, NASA_INS_torsoRightImu
Perhaps we should go by NASA_IMU_ as well

Mode 4 has some inaccuracies - libreoffice isn't the best for that. I'll revise this and remove some cruft

Keeping them all NASA_ prepended is fine with me / makes sense... as long as we don't eventually end up requiring other robot drivers (like the Atlas driver) to publish such-named messages to. It makes sense to publish those-named messages from the Val driver to indicate that they're Valkyrie-specific (unless one day we support a new NASA robot). Are you envisioning these channels being Val-specific?

The rest sounds good.

as long as we don't eventually end up requiring other robot drivers (like the Atlas
driver) to publish such-named messages to

The Atlas driver publishes only messages beginning ATLAS_**** so its clear what comes from the robot.
We could use VAL_*, I chose NASA to distingush from confusing with any signals from IHMC. VAL would make more sense.

Agreed on using VAL_*

Ok, Lets go with:

VAL_CORE_ROBOT_STATE  ... joint_states, does not include multisense joint currently
VAL_IMU_pelvisRearImu   .... i.e. renaming the INS to IMU
VAL_FORCE_TORQUE
VAL_COMMAND ... NASA's diagnostic feedback of commands
ROBOT_COMMAND ... Drake's control commands to the robot

SGTM. To be complete, message types, for now, are:

VAL_CORE_ROBOT_STATE: pronto::joint_states_t
VAL_IMU_*: mav::ins_t
VAL_FORCE_TORQUE: drc or pronto::six_axis_force_torque_array_t
VAL_COMMAND: this is currently a pronto::joint_angles_t... which is perfect for this message in everything except the message type's name. I don't feel a huge need to change this unless you want it to be?
ROBOT_COMMAND: currently drc::atlas_command_t, TBD as controller comes online.
Are these good?

Don't use pronto types in the translator - use the identical drc type. (I'd like to make a drc::ins_t message for that reason.). Pronto lcmtypes should be restricted to Pronto.
Where an LCM message type's contents are equivalent, the lcm pub/sub knows no difference - so pronto::joint_angles_t can be transmitted and drc::joint_angles_t received.
VAL_COMMAND contains the a ROS joint_state msg - hence the use of a joint message. Maybe this should be VAL_COMMAND_FEEDBACK

I think this is resolved and summarised here:
https://github.com/openhumanoids/oh-distro/wiki/State-Estimation-Messaging
which I will update later


Design more states for card image viewer pagination

what does it look like when you are in the middle?
what does it look like when you are at the end?
what does it look like if there are only 1 or 2 cards?
how does it work with a large number of card? (i.e. Killen cards)
document the logic (e.g. current card is the only fully-opaque)

@jkotin @clmahoney Here are the designs for this issue:
(zpl.io/2vk5m7n)
Please read the text on the left of each row.

What you see is the logic of navigating through the cards and the small card preview layout logic.
"Page" numbers and ranges have also been added to help the user know on which card they are as well as the total number of cards to help them anticipate the number of cards through which they would be scrolling.

Thank you Gissoo!  This looks fantastic.
Questions:

On non-touch screens is it possible to scroll through the previews?  I can imagine users wanting to get quickly to a particular card or date, or to get a quick survey of the cards.

Is it possible and/or advisable to have the date ranges of the card with the large card display?  What about having the ranges under each preview image?

What happens if a user clicks on the image of the large card display?  Does the card image get bigger?  Can users right click and save card images as jpgs?

For the fewer cards scenario, will there just be fewer cards in the preview?  What if there's only one card side?

@jkotin, below are the answers:

Yes, it is possible to scroll through the previews on non-touch screens, thanks for catching that, I've reflected it in the revised version that is accessible through the same link

We could add this to the wishlist.

Enlarging/deep zooming the large card display is not in scope right now.

Yes, there will be fewer cards in the preview. When there is only one card side it will be just like the screen on the bottom left (on the Zeplin link), with only one card preview present. (Highlighted in red, below)

Thank you. Yes, we should add including dates to the wish list.  Often the date of the card is not clear from the card itself.

@jkotin Please let me know if you have any other questions or comments so I can address them, if not, let's close the issue.


AUI-1885 Allows adding multiple fields on the same column

This lgtm, I just found a couple of issues that need fixing. I updated some of these myself, but not the ones I commented about, so I'm sending you a pull request so you can add those last changes on top of what I already did, ok @thiago-rocha? I'll send you the pr soon.

Just started reviewing :) :octocat: Sent from GH.


bug select day

https://github.com/user-attachments/assets/6c78953c-7e51-41a6-8d01-733ca51986c0
I encountered this bug after fixing the bug in version 0.3.7

help me please @AndriiVivcharenko

Please fix this for me.

Hi, @noppon0031 !
Sorry for the delayed response. Please test this with the release/0.3.8 branch. This issue should be fixed now
flutter_customizable_calendar:
  git:
    url: git@github.com:Radency/flutter_customizable_calendar.git
    ref: release/0.3.8

Thanks

@AndriiVivcharenko Thanks, this has been fixed.

@AndriiVivcharenko When will you publish version 0.3.8 to pub.dev?


STORM-3696 don't ignore InterruptedException

What is the purpose of the change
It's not safe to assume InterruptedException is ignorable.
How was the change tested
Rebuilt storm-client

Good catch!

squashed


Privacy Center not notifying user of failed request

Bug Description
I put in a malformed request (bad phone number, I saw a 422 in the dev console) but no notification was given of failure in the UI
Steps to Reproduce

nox -s "fides_env(test)"
Go to localhost:3001 and submit a private request with something malformed (might need to mess about in the console to get it to send)
Verify that with a 422 error response, no exception/warning is shown to the user

Expected behavior
As a user, they should be notified if the Privacy Request does not succeed for no reason. In this case, it seems we're relying entirely on front-end validation as opposed to also checking the backend response.
Environment

Version: 2.8.0 + (dev version)

Additional context
This came about due to another bug I was fixing, and theoretically the frontend validation would probably catch this but it still made debugging more difficult at first because I didn't know it had failed.

heads up @rsilvery, not sure how/when you want to slot this in. Not major, but also potentially dangerous if people think their privacy requests are getting submitted when they actually aren't?

@mfbrown to ask @ThomasLaPiana  for more info


FAB menu is missing in the welcome page

After https://github.com/apache/incubator-superset/pull/8289 FAB menu is missing in the welcome page in superset. However it is present in other pages
Expected results
git checkout 54d9154a14c2b62668c7386903af17f6907874c9

what you expected to happen.
Actual results
git checkout 300c4ecb0f6798e5901dcb88a034c53e708ff0b4

what actually happens.
How to reproduce the bug
Go to the http://localhost:8088/superset/welcome
Environment
(please complete the following information):

superset version: master

needed to run npm run build to resolve this.
False alarm


Share button isn't working on https://carbon.now.sh/

when you hit "tweet image" button, nothing happens.

Hmm, It's working just fine on my end. Are you seeing any errors in your console?

Yeh, seems like the URL it's a bad request here.

I can confirm the same. 😟

429 is a rate limit status code ☹ . We will post an update as soon as we have one.

Try now! (:

Closing if this is resolved. Feel free to reopen if the problem persists

sorry my delay of the feedback. it's working fine! thanks!


vscode not loading file change or new file in time

when opening a folder in vscode, the changed file are not updated in time. And when creating new file outside of vscode, the new file do not show in explorer and can't be searched by CTRL+P. This happens more when the folder is in a remote server and contains hundreds of files. Is there anyway to make vscode load in time? Thanks.

Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: http://code.visualstudio.com/Download#insiders

@caiqi do you have some more specific repro steps?

I have a directory on a remote server where I save models and write logs into it. Sometimes, the new created models are not shown in vscode explorer and the logs are old. However, when I click another file and then click the old logs , it gets updated.

@caiqi
VS Code is using different file watching strategies depending on your OS and your setup.
First thing to check:

[ ] I verified that the issue still reproduces in our insiders version (https://code.visualstudio.com/insiders/)
[ ] I verified that this issue is happening outside of a node_modules folder. Our file watcher ignores this folder!

To find out more about this issue, please first answer this:

[ ] What OS are you on
[ ] Does this happen from an empty workspace, a single-folder workspace or a multi-root workspace?
[ ] Do you have files.useExperimentalFileWatcher: true configured in settings?
[ ] What is your configured files.watcherExclude setting?

Then please answer the following depending on your setup:

[ ] It reproduces with any folder or workspace you open
[ ] The files/folders you open are on your local disk and not on a network share nor a UNC drive
[ ] The files/folders you open are not behind a symbolic link
[ ] The folder(s) opened use the exact same casing as they are on disk
[ ] It reproduces when I make a change to the file from another VS Code window (either from the editor or the file explorer)

Then please attach the output of our development console

run code --verbose from the command line
perform the action that causes you to not see the file change in VS Code
attach the output from the console (Help > Toggle Developer Tools > Console)

Thanks for taking the time to fill out this questionnaire.


Fix word2vec excepction when words contain chinese.

When word2vec deal with Chinese language, it will break out below exception.
Traceback (most recent call last):
  File "word2vec_tf.py", line 58, in <module>
    main()
  File "word2vec_tf.py", line 47, in main
    model = Word2Vec(options, session)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/models/embedding/word2vec.py", line 166, in __init__
    self.save_vocab()
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/models/embedding/word2vec.py", line 382, in save_vocab
    opts.vocab_counts[i]))
UnicodeEncodeError: 'ascii' codec can't encode character u'\u90fd' in position 0: ordinal not in range(128)

I fix this exception by adding encode('utf-8') , and then it successful run word2vec model.

Can one of the admins verify this patch?

I signed it!

@tensorflow-jenkins test this please


Marshaler does not add quotes for keywords used as struct key

What version of CUE are you using (cue version)?

$ cue version
cue version v0.4.3-beta.1 darwin/arm64

Does this issue reproduce with the latest release?
reproduce in cue version v0.4.2 darwin/arm64
What did you do?

$ echo '{"mapping":{"null":0,"true":1,"false":2}}' > test.json && cue import test.json -o -
What did you expect to see?
foobar: {
        "null":  0 // add quotes for keywords used as struct key
        "true":  1
        "false": 2
}

What did you see instead?
foobar: {
        null:  0
        true:  1
        false: 2
}

This is not technically incorrect, as cue allows keywords in field positions. But for value keywords this is arguably  undesirable, in case we ever want to allow maps of those values.


pidusage.stat pidusage.stat is not a function

Hi,
Thank you for this great tool 👍
I was testing the current master commit
The current commit: 1ddc160b337eb0c9e9daf1f00b726aa095bfc1f7
"hapijs-status-monitor": {
      "version": "git+https://github.com/ziyasal/hapijs-status-monitor.git#1ddc160b337eb0c9e9daf1f00b726aa095bfc1f7",
      "requires": {
        "pidusage": "2.0.6",
        "socket.io": "2.1.0"
      }
},

And it looks like it fails on:
/node_modules/hapijs-status-monitor/src/helpers/gather-os-metrics.js:16
  pidusage.stat(process.pid, (err, stat) => {
           ^
TypeError: pidusage.stat is not a function
    at module.exports (/Users/wessel/Development/VitalTracker/reporting-services/node_modules/hapijs-status-monitor/src/helpers/gather-os-metrics.js:16:12)
    at Timeout.setInterval [as _onTimeout] (/Users/wessel/Development/VitalTracker/reporting-services/node_modules/hapijs-status-monitor/src/helpers/socket-io-init.js:24:42)
    at ontimeout (timers.js:458:11)
    at Timer.unrefdHandle (timers.js:570:5)

I think it's a code breaking change in the pidusage from 1.x to 2.x that no longer makes the pidusage.stat available
Regards,

added a pull request, enjoy


feat: debug_traceCall

Motivation
Minimal implementation of #1801 debug_traceCall.
Solution
Similar implementation to debug_traceTransaction.
stateoverrides and blockoverrides trace options were not implemented.
PR Checklist

[x] Added Documentation
[x] Updated the changelog

Smol merge conflict lgtm otherwise

sick work mate, welcome :)


Add wiliwili v1.0.1

Add wiliwili v1.0.1

Relates to #11447

[*] I have read the Contributing Guide.

/verify


chore: merging in gotenberg repo 8.9.1

Hi @ZachAtOneBrief,
I brought in the changes from the gotenberg repo (their main branch to our main branch).  This PR is attempting to merge our main to main-bc.


Performance info

Have you read the FAQ and checked for duplicate open issues?
yes
What version of Shaka Player are you using?
2.5.10
Please ask your question
Where can find performance info about the player such as:
Download bit rate (min, max)
Time to start the video (video startup time)
No of stalls
etc

There's a method on player, called getStats. It has the (current) stream bandwidth, the number of dropped frames, etc. You can determine the start time and the number of stalls by examining the stateHistory (which lists the timestamps of each change in state, e.g. between playing, paused, buffering, etc).

Thank you so much for the info.

Okay. It sounds like everything is settled then.
If you have any further questions about performance issues, you can re-open this.


How do I save an optimizer state of trained model?

I am playing with the mnist_vae example and can't figure out how to properly save/load weights of the trained model.
enc_init_rng, dec_init_rng = random.split(random.PRNGKey(2))
_, init_encoder_params = encoder_init(enc_init_rng, (batch_size, 28 * 28))
_, init_decoder_params = decoder_init(dec_init_rng, (batch_size, 10))
init_params = init_encoder_params, init_decoder_params

opt_init, opt_update, get_params = optimizers.momentum(step_size, mass=0.9)
opt_state = opt_init(init_params)

after that, I train the model using opt_update and want to save it. However, I haven't found any function to save the optimizer state to the disk.
I tried to save parameters and initialize opt_state with them, but not all the information conserves, and the result opt_state_1 is not the original opt_state.
weights=get_params(opt_state)
jnp.save(file, weights)
weights = jnp.load(file,allow_pickle=True)
opt_state_1 = opt_init(init_params)

How do I properly save the model I trained?

Hi, did you manage to get a solution for this? I'm also encountering a similar issue and would appreciate some help on this.

I ended up following the instructions listed here and managed to get something that looks correct:
https://stackoverflow.com/questions/64550792/how-do-i-save-an-optimizer-state-of-jax-trained-model

Have you considered using the Flax or Haiku libraries for writing models. You will likely find more examples for how to do things with neural networks in those libraries support channels.

I'm going to close this issue – I think referring to Flax or Haiku for this sort of problem is probably the best solution.
If there are further questions along these lines, please feel free to open a new discussion about the specific issue you're having. Thanks!


HtmlFileInput how to upload a directory

Hi
What should i do
<input type="file" id="dir" name="file"  webkitdirectory mozdirectory />

In the best case provide a PR ;-)
This is not supported at the moment, will have a look.
But if you like to help it will be great if you can provide testcases like we have in com.gargoylesoftware.htmlunit.html.HtmlFileInput2Test
com.gargoylesoftware.htmlunit.html.HtmlFileInputTest.

Have added some support for this, please wait for the next snapshot and check

released in between, will close this.


fix(release): update pnpm-lock.yaml when release

Thanks, but it's better to let pnpm update the lockfile. See 7a5c7844


Is your script supposed to work with another kernel and set of patches ?

I've recompiled the kernel because I wanted to enable kvm. The commands that I have issued are :
git clone https://github.com/OE4T/linux-tegra-4.9.git
cd linux-tegra-4.9
git checkout oe4t-patches-l4t-r32.7.3
cp /proc/config.gz .
gunzip config.gz
mv config .config
make menuconfig —> (enable kvm)
make -j4
make -j4 modules_install
make -j4 Image
mv /boot/Image-kvm Image2-kvm
cp arch/arm64/boot/Image /boot/Image

the dtb file to use is called : tegra210-p3448-0000-p3449-0000-a02.dtb
and the extlinux.conf is something like this :

TIMEOUT 30
DEFAULT primary

MENU TITLE L4T boot options

LABEL primary
MENU LABEL primary kernel
LINUX /boot/Image
INITRD /boot/initrd
FDT /boot/tegra210-p3448-0000-p3449-0000-a02.dtb
APPEND ${cbootargs} quiet root=/dev/mmcblk0p1 rw rootwait rootfstype=ext4 console=ttyS0,115200n8 console=tty0 fbcon=map:0 net.ifnames=0

Unfortunately Xfce freezes a second before to be fully loaded and I suspect that the cause is your script. Is it supposed to work on the scenario that I have explained ? Yes,I've removed your script,giving these commands :

systemctl disable automagic-fan
mv /usr/local/bin/automagic-fan /usr/local/bin/automagic-fan_
mv /lib/systemd/system/automagic-fan.service /lib/systemd/system/automagic-fan.service_
mv /etc/automagic-fan /etc/automagic-fan_

but the problem still there. xfce won't fully load.  I would like to know your opinion.

if you disabled the service then it is not running (i.e. does nothing and didn't cause your crash). If your pc still crashes after disabling the fan script then it is not the cause. Your custom kernel stuff sounds much more likely to cause that, so if I were you I'd look into that, or according driver issues.
The worst thing a python script can do is have an exception, which just stops the script and nothing more.


Query an array value

For express, when querying an array as value[]=a&value[]=b, you get value: ['a', 'b'].
It helps distinguish between an array and a string when querying an array of lenght 1.
That is,  value=a and value[]=a results in value: 'a' and value: ['a'].
But for polka,  it doesnot provide a direct way distinguishing them.
I have to check the type of the value by myself.

Hey there,
Not sure which version you're using, but stable uses the native querystring.parse which ignores the key's [] and only converts to array on repeat values. The polka/url@next (which polka@next uses) emulates this behavior too.
This is the common, safe approach for most platforms. Node.js does it by default, as does new URL and new URLSearchParams on all browsers and all Node.js versions. The [] is part of the key name – values (and value types) are determined by the values themselves.
If you need to copy Express' approach exactly, you can easily add that. Express uses qs to create its req.query object. You can copy the same behavior wherever you need it or as a custom middleware:
const { parse } = require('qs');

app.get('/foo/bar', (req, res) => {
  let query = {};
  if (req.search && req.search.length > 1) {
    query = parse(req.search.substring(1));
  }
  // ... use 'query' var
})

Hope that helps!

I was using polka@next.
But it does not emulates this behavior.
In my route /q, I have
console.log(req.search, req.query)

And when I visit /my-server-path/q?value[]=test, I see
?value[]=test { 'value[]': 'test'}

BTW, I enables bodyParser.json() and bodyParser.raw() for the server.

I know, that's what I'm saying. By default, the majority of parsers (including browsers) will treat the [] as part of the key name. This is why Polka continues to do the same... It's the more generalized behavior.
The behavior you're expecting is custom. If you want that behavior too you must opt into it, as shown above.

If it were me, I would just send URLs like ?value=a&value=b. That's always parsed as an array since value repeats: { value: ['a', 'b'] }. It is true that only sending one value will be parsed as a non-array ({ value: 'a' }) but that's easily fixed within your server handler:
const values = [].concat(req.query.value || []);
//=> Will always be an array

Oh, I misunderstood you.
Thank you for your instructions.


emdash to colon

I think you need a colon for this list rather than emdash

I feel like emdash also works, sort of like introducing?  But I am not an expert with English. I can go with either one.


[Question ]  EntityFramework Core Supported?

Hello!
Great library!! I love how you have structured your code - very modular and helpful with the interceptors
My primary use-case is for Entity Framework Core & MSSQL
I saw that you had: EntityFrameworkRepository and also saw on your Readme.md that:

Coming soon: EntityFramework Core and OData

Question
1.) Is Entity Framework Core (latest net7) supported
*  Partially?
*  Fully?
*  None?
2.) When I saw EntityFrameworkRepository I was confused to if this meant EfCore or EntityFramework  - Can you clarify please?

Thanks all!

Hello!
The current version supports EF Core (v6). Unfortunately the documentation not up to date.
I am currently working on v7 of the library which will include Unit-of-Work support for every storage including EF Core v7, additional samples and (hopefully) better documentation. I hope to finish this up over the weekend.
But to answer your questions:

EF Core 6 is fully supported in the v6 branch of the library and EF Core 7 will be in v7 of the library.
The EntityFrameworkRepository actually means EF Core. I'll fix the naming in v7 to avaid this kind of confusion in the future. Thanks for pointing this out.

I hope I could clarify things up a bit. :-) If you have any other questions, ideas or recommendations let me know.
Cheers, Matt

Thanks for this! This clears things up alot.
I actually am also creating a UnitOfWork implementaton for this - I can create a pull request also when I get it implemented and possibly add features, or use features from your Implementation.

EF Core 6 is fully supported in the v6 branch of the library and EF Core 7 will be in v7 of the library.

Sounds good, I believe I am using Version 7 of your library (most recent)
Yes this clears things up alot!

Off-topic: I have been involved in crypto since 2016/17, and launched a few coins professionally for my employers - I saw you are also!! Very cool!! Few points I have to give you a hard time 😄  =>

Cardano is AWESOME except:

Haskell is gross
The team moves sooo slow - not sure if its realistic for them to deliver a functioning chain
The coin is large enough in $$ (trading volume) that it will never go away
The community is so engrossed in 'voting' that they never get anything done that can help 'real enterprise chains'
The language is so gross - that its painful to build a functioning application (Algorand also had this issue)

Bitcoin Cash is the real king 🙃

Also - I am surprised your not a big supporter of Neo! Why Cardano in comparison to Neo? The Neo tech v3 has more features than IPFS and very good smart contracts, good tech -- its China - Fair  Point - but the tech is amazing

Very cool to meet you

Hi Jeff,
I published v7.0 yesterday. If you like you can have a look at the UoW implementation for EFCore 7.
I needed to jump through some hoops, because I allow to have multiple repositories with different configurations in a single application. But I am fine with how it turned out.

Off-topic:

Cardano is AWESOME

I agree that it is. 😉
I too agree that Haskell/Plutus is painful to work with. But Haskell fits the academic approach and the formal verfication of code. This discussion is widly held within the communitiy. Maybe we see other langages for smart contracts in the future. Would be cool to write them in C#.
I really hope it will stay for a long time. Not only because I invested in it, but because I really believe this is right way to appoach projects of this magnitude.
I think the self convernance of the chain is a major step to true decentralization. But I agree that Cardano is not seen as an "enterprise chain". I hope this will happen over time.
I agree that the language is gross, but maybe it leads to evelopers coding more carfully. There are a lot projects already running their contracts and a lot projects in the making. But this needs to be monitored for sure.
After last weeks FTX armageddon I am more certain that Cardano has a very strong selling point in decentrelization and escpecially their liquid staking. As you know: Not your keys, not your coins.😉

Bitcoin is the real Slim Shady 😄

I hope you haven't lost anything in the FTX debacle.

Cool to meet you too
Matt


Landscape + 3-button navigation results in cut off comments

Seems like we have to specify the inset paddings on the comments screen in a different way.
Related to #73

Oh yeah this doesn't look good at all. Seems the main screen and settings are working find and it is just the comments screen that needs to be fixed from what I can tell. Will look into this.


"ref" callback and "componentDidMount" not being triggered on initial render

Description
ref callback and componentDidMount life-cycle hook is not being triggered until after manually triggering an additional re-render with setState.
Also asked on SO, but not getting any responses: http://stackoverflow.com/questions/43009993/react-native-initial-lifecycle-hooks-not-being-called/43010267#43010267
Reproduction Steps and Sample Code
"constructor", "componentWillMount", and "render" are called immediately upon launching the application, as expected. However, "ref", and "componentDidMount" are not called until manually triggering a re-render via this.setState() after the 5 second delay.
import React, { Component } from 'react';
import {
  AppRegistry,
  StyleSheet,
  Text,
  View
} from 'react-native';

class TestText extends Component {

    constructor(props, context) {
        super(props, context);
        console.warn('constructor');
        this.textRef = null;
    }

    componentWillMount() {
        console.warn('willMount');
    }

    componentDidMount() {
        console.warn('didMount');
    }

    render() {
        console.warn('render');
        return (
            <View>
                <Text
                    ref={(me) => {
                        console.warn('ref: ' + (me ? 'not null' : 'null'));
                        this.textRef = me;
                    }}>
                    {'TEST'}
                </Text>
            </View>
        );
    }

}

export default class AwesomeProject extends Component {

  constructor(props, context) {
      super(props, context);
      this.state = {
        test: 0
      };
      setTimeout(() => {
        this.setState({
            test: 1
        });
      }, 5000)
  }

  render() {
    return (
      <View style={styles.container}>
        <TestText />
      </View>
    );
  }

}

const styles = StyleSheet.create({
  container: {
    flex: 1
  }
});

AppRegistry.registerComponent('AwesomeProject', () => AwesomeProject);

Solution
Expected behavior: Shouldn't "ref" and "componentDidMount" be called immediately after rendering. I shouldn't have to setState to get those hooks to run.
Additional Information

React Native version: 0.42.3
Platform: Android (maybe iOS as well)
Development Operating System: Windows
Dev tools: compileSdkVersion = 23, buildToolsVersion = 23.0.1

same here ++

Hi there! This issue is being closed because it has been inactive for a while. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. Either way, we're automatically closing issues after a period of inactivity. Please do not take it personally!
If you think this issue should definitely remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release candidate? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.

If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contribution guide, and feel free to hop into #react-native if you need help planning your contribution.

Even I am facing this issue.
Here is the test code:
import React, { Component } from 'react';
import {
  Button,
  Container,
  Content,
  Form,
  Item,
  Input,
  Label,
  Text
} from 'native-base';

export default class TestComponent extends Component {
  constructor(props) {
    super(props);
    this.refs = {};
  }

  componentDidMount() {
    console.log('=====>>>>>> 2. In componentDidMount() ....');
    this.refs.detailsField.focus();
  }

  render() {
    const { navigate } = this.props.navigation;
    return (
      <Container>
        <Content>
          <Form>
            <Item floatingLabel last>
              Details
              <Input
                ref={c => {
                  this.refs.detailsField = c;
                  console.log('=====>>>>>> 1 ....');
                  console.log(this.refs.detailsField);
                }}
                multiline={true}
                numberOfLines={10}
                style={{ height: 200, textAlignVertical: 'top' }}
                onChangeText={text => this.setState({ details: text })}
              />
            </Item>
          </Form>
        </Content>
      </Container>
    );
  }
}

ref callback is not getting called on rendering of Inpput component. Ideally we should see below console logs
=====>>>>>> 1 ....
=====>>>>>> 2. In componentDidMount() ....

But on console I am only seeing
=====>>>>>> 2. In componentDidMount() ....

Also seeing below warning / error message from React-Native (as I am calling this.refs.detailsField.focus() in componentDidMount()
Possible Unhandled Promise Rejection (id: 0):
TypeError: undefined is not an object (evaluating 'this.refs.detailsField.focus')
...
....

Looks like ref callback on Input field is not getting called at all.
Is my understanding wrong about ref or is it a bug ?
Please try this test case.

@gauravdhiman did you find the solution? I have the same problem, ref doesn't get called at all.

@sourcesoft Yes, found the solution. I am using Native Base and in my case the culprit was floatingLabel on <Item floatingLabel last> component. If I remove floatingLabel, ref works fine, but with floatingLabel I need to use getRef as explained here.
Here is the code that is working for me. Note the getRef in second input field.
<Item floatingLabel last>
    Address
    <Input
    onFocus={() => {
        // move to next field so that user an not modify this field manually
        this._detailsField._root.focus();
        // show screen to select an address
        navigate('SearchAddressScreen', {
        onSelectAddress: address => this.saveAddress(address)
        });
    }}
    value={this.state.address.text}
    />
</Item>
<Item floatingLabel last>
    Details
    <Input
    getRef={e => (this._detailsField = e)}
    multiline={true}
    numberOfLines={10}
    style={{ height: 200, textAlignVertical: 'top' }}
    onChangeText={text => this.setState({ details: text })}
    />
</Item>

If you are using some navigators library( like react-navigation in my case) this issue is still here. Maybe they delay the mount of the screen.
My actual solution is using something like InteractionManager.runAfterInteractions(() =>functionThatUseTheRef).
Always a workaround, but better than arbitrary setTimeout()


updates set up documentation on how to start the thin server

I found that bundle exec rails s now throws an error when trying to start this application locally.  However, these updated instructions do start the local server successfully.

Think Fernando updated it. Probably just need to update the docs.

On Nov 28, 2015, at 5:35 PM, Nell Shamrell-Harrington<EMAIL_ADDRESS>wrote:
I found that bundle exec rails s now throws an error when trying to start this application locally. However, these updated instructions do start the local server successfully.
You can view, comment on, or merge this pull request online at:
https://github.com/OperationCode/operationcode/pull/108
Commit Summary
updates set up documentation on how to start the thin server
File Changes
M CONTRIBUTING.md (4)
Patch Links:
https://github.com/OperationCode/operationcode/pull/108.patch
https://github.com/OperationCode/operationcode/pull/108.diff
—
Reply to this email directly or view it on GitHub.

Yep!  Added to the CONTRIBUTING.md :)

Added to the CONTRIBUTING.md here https://github.com/OperationCode/operationcode/pull/109

😃

On Dec 6, 2015, at 3:01 PM, Nell Shamrell-Harrington<EMAIL_ADDRESS>wrote:
Yep! Added to the CONTRIBUTING.md :)
—
Reply to this email directly or view it on GitHub.

👌

On Dec 6, 2015, at 3:08 PM, Nell Shamrell-Harrington<EMAIL_ADDRESS>wrote:
Closed #108.
—
Reply to this email directly or view it on GitHub.


Use strongly typed NumberFormat to discriminated custom and predefined formats

This code adds the ability to specify explicitly whether a NumberFormat is a custom format, or a predefined one. The advantage of this is twofold. Firstly, if you actually want to specify a custom format such as, for example, "mm-dd-yy" then this is impossible with the current code. It would get auto-converted to predefined format 14, which will not necessarily be displayed as "mm-dd-yy" by Excel (being locale dependent).
Secondly, if you know you do want to specify a predefined format like "Short Date", it's not at all obvious that the way to do so is to pass the string "mm-dd-yy" as the NumberFormat.
To address this, I've changed Style.NumberFormat to be a new NumberFormat struct rather than a string. That way you can specify exactly what you want, NumberFormat.Custom("mm-dd-yy") or NumberFormat.Predefined(PredefinedNumberFormat.ShortDate) and get the correct result.
For backwards compatibility, I've included implicit cast operators from string, so you can assign a string directly, the existing code style.NumberFormat = "mm-dd-yy" will still work. To maintain exact compatibility I've had the implicit operator check if it was one of these old hard-coded formats and replace it by the predefined one, so if you do style.NumberFormat = "mm-dd-yy" what you actually get is Predefined Format 14, just like you used to.
Depending on your views on consistency vs. breaking changes, you may prefer to remove this so that style.NumberFormat = "mm-dd-yy" gives you a custom format instead, the same as NumberFormat.Custom("mm-dd-yy") does, but I thought the safest option would be to keep compatible behaviour.

First of all, thanks for the PR!
You are right that it is currently not possible to specify a custom format that would match a predefined format.
Changing NumberFormat from a string to a struct would be a breaking change that I would like to avoid. It might be source compatible with the implicit operator, but I don't think it would be binary compatible. The use of implicit operators is something I would generally avoid as well.
Perhaps another approach would be to add a boolean property to Style, e.g. bool PreferPredefinedNumberFormats that would default to true. When it is false it would always treat NumberFormat as a custom format.

Yes, the PR as written would not be binary compatible, if that's important to you another approach will be required.
One of the goals I had for this was to make the distinction between custom and predefined formats clear, and to make predefined formats obvious and discoverable. My own experience with trying to format dates as Short Time was quite bad, every attempt ended up with the cell having a custom format and not behaving the same as a cell I manually set to have short date format in Excel, and I didn't understand why. Eventually comparing the xml files inside the xlsx produced by Excel and by SpreadCheetah led me to the existence of predefined formats. From there, I had to inspect the source of SpreadCheetah to discover how to set the format of the cell to a predefined format - what magic string to use.
Some possible suggestions, keeping binary compatibility:

Deprecate string? Style.NumberFormat and add a new NumberFormat? Style.Format using the class in this PR. Style.NumberFormat setter simply sets the .Format in the same way as the implicit cast I wrote does. Remove the implicit operators (your preference).
Keep string? Style.NumberFormat and add a new PredefinedFormat Style.PredefinedFormat that is an enum, with a default value of PredefinedFormat.Custom. If set to anything other than PredefinedFormat.Custom, Style.NumberFormat is ignored and the predefined format used instead. The setter of Style.NumberFormat implements the logic in the implicit cast of setting either a Predefined Format or a Custom Format as appropriate. Discard the NumberFormat class from this PR.

While you are deciding, it's probably also worth considering a change in terminology, from "Predefined Formats" to "Standard Formats". "Predefined" sounds like it's a convenient list of frequently used formats, provided by SpreadCheetah. "Standard" more implies that these are formats built into Excel that can be chosen instead of defining a custom format.
Anyway, I'm happy to modify this PR in any of these directions, but if you decide you prefer the existing approach to any of this, just let me know and I'll drop it.

Thanks for your input. I do now understand the struggle you were facing, and I also agree it is worth making a change to avoid others from facing the same struggle.
The option of deprecating string? Style.NumberFormat and adding NumberFormat? Style.Format sounds like the best approach to me. So if you can modify the PR it would be greatly appreciated!
I also agree with the terminology change you suggest. I think consistency is important, so any code, comment, or test should all use the same terminology.

I've implemented the change as discussed, including removal of the implicit operators. I agree that consistency is important, I think I have found everywhere that refers to "predefined" and made it refer to "standard" instead. Including NumberFormats.GetPredefinedStandardNumberFormatId, on the grounds that it was internal and therefore shouldn't affect backwards compatibility.
I wasn't sure whether to mark the NumberFormats class itself as deprecated or not. On the one hand, people should ideally not use constants like NumberFormats.Fraction for anything (instead using NumberFormat.Standard(StandardNumberFormat.Fraction), but there is also the NumberFormats.DateTimeSortable which is not a standard format, and is therefore not deprecated. Would you prefer the whole class deprecated, or each individual constant (except DateTimeSortable) deprecated, or leave it as it is?

Looks good now, thanks!

Great, thank you!


[WIP] feat(update tokens): update current tokens

Update current tokens

@daragh-king-genesys I've added a ticket to the GDS backlog to look at reflecting these changes on the typography page in the web component library. I've also verified that the changes are reflected in the Spark library and Figma
https://inindca.atlassian.net/browse/GDS-2292


Update hdinsight-apache-spark-use-with-data-lake-store.md

I believe there is a typo here.

Hi @eloldag, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@eloldag : Thanks for your contribution to the Azure documentation! The author, @nitinme, has been notified to review your proposed change.

#sign-off


Wrong sequence used in case of inherited default value

OrientDB Version: community-2.2.29
Java Version: OpenJDK 1.8.0._131
OS: Ubuntu 16.10
Expected behavior
After running the following script:
create sequence parent_seq type ordered start 1023 increment 1
create sequence child_seq type ordered start 0 increment 1

create class parent extends v
create property parent.parent_id string (notnull true, mandatory true)
alter property parent.parent_id default 'sequence("parent_seq").next()'

create class child extends parent
create property child.child_id string

create vertex child set child_id=sequence("child_seq").next()
select parent_id, child_id from child

The two sequences should have been be used:
+----+---------+--------+
|#   |parent_id|child_id|
+----+---------+--------+
|0   |1024     |1       |
+----+---------+--------+

Actual behavior
...but the output shows that only parent_seq has been used, twice:
+----+---------+--------+
|#   |parent_id|child_id|
+----+---------+--------+
|0   |1024     |1025    |
+----+---------+--------+

Steps to reproduce
Run the script specified above.

Hi @bordig-f
Thank you for reporting, I'll check it asap
Thanks
Luigi

Hi @bordig-f
I just pushed a fix to 2.2.x branch, it will be released with 2.2.31
Thanks
Luigi

Thank you Luigi =)


remove deprecated version method

deprecated for long enough now I'd hope ...
@gmmeyer

I'd think master is 4.0 anyway, so merge now ?

added deprecation in https://github.com/DataDog/dogstatsd-ruby/pull/77

You're right!


ACL - audit CAT validation

Problem
Side-discussion regarding ACL validation in
https://github.com/CHIP-Specifications/chip-test-plans/blob/master/src/cluster/AccessControl.adoc#tc-acl-2-4-acl-attribute resulted in us wondering about how subjects are being validated.
Specifically we are testing now for some invalid values like 0xFFFF_FFFF_FFFF_FFFF, 0xFFFF_FFFF_FFFF_0000 and 0xFFFF_FFFD_0000_0000 however additional review would be required.
Proposed Solution
@tcarmelveilleux volunteered to be assigned an Audit ACL validation of CATs so here is the issue for that.

For collisions of CAT versions in subjects, they are independant subjects. We already validate for NOCs that there are no collisions of CAT identifier versions. However, for ACL enforcement, I don't think we should internally audit for complex "possibly ugly" policies. If there are multiple versions of a single CAT identifier present in Subjects of an ACL, it means an admin decided to have it there. The ACL privilege algorithm tries to match the highest privielege, skipping all non-matching. Therefore, the smaller version of a CAT identifier will eventually match, if a CAT of the incoming subject has that version or later. This is OK and we should not error-out on it. Otherwise we need complex  new code in ACL cluster to guard against what amounts to just a "weird" (but not illegal) configuration. In Access control enforcement, all subjects are considered independently.
I wouild argue that after finding version 0 CATs are properly disallowed, and that CAT collisions are disallowed in NOCs, we don't need to audit the subjects of ACL entries for CATs beyond that.

I wouild argue that after finding version 0 CATs are properly disallowed, and that CAT collisions are disallowed in NOCs, we don't need to audit the subjects of ACL entries for CATs beyond that.

I agree.


Ignore case for Color names

When loading a color fromName, it shouldn't be case-sensitive

Fixed in commit https://github.com/Sygmei/ObEngine/commit/7ff6b0b6b119965e8d318bfa8a49a2c2dd2c6691


Group Functions sometimes don't work due to captchas

I use Group Functions such as setRank and it says "Invalid permissions". I go and rank someone a few times to farm some captchas, then it works! This means, captchas a preventing the bot and this is a bug needing to be fixed. You could possibly use https://www.npmjs.com/package/deathbycaptcha to solve the captchas and the issue can be resolved. This may also have to do with the Group Events not working, as Roblox is adding captchas EVERYWHERE.

It's not a bug and it can't be fixed without me breaking the bank.

'npm' is not recognized as an internal or external command,
operable program or batch file.

wait huh... setting player ranks result in captchas?  I thought only group joins did


i don't understand how to install or use it?

See #4.

See also https://github.com/kkuchta/css-only-chat/blob/master/INSTALL.md


Documentation Improvements

Summary:
This PR makes a ton of documentation improvements
Type (select 1):

[ ] Bug fix
[ ] Feature implementation
[x] Documentation improvement
[ ] Testing improvement
[ ] Test added to report bug (GitHub issue #--- @---)
[ ] Something not listed here

Is this a breaking change? (select 1):

[ ] 🚨 YES 🚨
[x] No
[ ] I'm not sure

Is this ready to be merged into Dynamoose? (select 1):

[x] Yes
[ ] No

Are all the tests currently passing on this PR? (select 1):

[x] Yes
[ ] No

Other:

[x] I have searched through the GitHub pull requests to ensure this PR has not already been submitted
[x] I have updated the Dynamoose documentation (if required) given the changes I made
[x] I have added/updated the Dynamoose test cases (if required) given the changes I made
[x] I have ensured the following commands are successful from the root of the project directory

[x] npm test
[x] eslint .

[x] I agree that all changes made in this pull request may be distributed and are made available in accordance with the Dynamoose license
[x] All of my commits and commit messages are detailed, explain what changes were made, and are easy to follow and understand
[x] I have filled out all fields above

Pull Request Test Coverage Report for Build 882

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 85.582%

Totals

Change from base Build 881:
0.0%

Covered Lines:
2212

Relevant Lines:
2510

💛  - Coveralls

Pull Request Test Coverage Report for Build 882

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 85.582%

Totals

Change from base Build 881:
0.0%

Covered Lines:
2212

Relevant Lines:
2510

💛  - Coveralls

Pull Request Test Coverage Report for Build 882

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 85.582%

Totals

Change from base Build 881:
0.0%

Covered Lines:
2212

Relevant Lines:
2510

💛  - Coveralls


`React/RCTBridgeModule.h' file not found - RN 0.61.2 for iOS build

Hi,
Trying to upgrade RN from 0.60.3 to 0.61.2 and getting the below error when compiling for iOS
React/RCTBridgeModule.h' file not found
Has anyone successfully been able to upgrade to RN 0.61+?

hi,me too

RN -->>>  0.62.2

Any solution? Facing same issue

Use #import "RCTBridgeModule.h" @dijanard


Change Action type

Make it consistent with ActionType

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

@cgvedant: Can you accept the contribution agreement?

!import


remove spaces between switches in pricing calculator

Description

The switches should be next to each other.

Will be fixed once the tooltips are enabled again in https://github.com/threefoldtech/tfgrid-sdk-ts/issues/2000

verified on devnet
version : 9d076f1
spaces removed between switches
tool tips are now working fine

Verified:

Devnet
7b55dd4


RELEASE: registry eth

Codecov Report

Merging #300 (a95be27) into main (30b4a1c) will increase coverage by 0.63%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             main     #300      +/-   ##
==========================================
+ Coverage   41.59%   42.23%   +0.63%
==========================================
  Files          20       22       +2
  Lines        1613     1629      +16
==========================================
+ Hits          671      688      +17
+ Misses        942      941       -1

Impacted Files
Coverage Δ

src/web3_utils.py
61.70% <33.33%> (+1.48%)
:arrow_up:

src/data_classes/contract.py
100.00% <100.00%> (ø)

src/settings/registry_settings.py
100.00% <100.00%> (ø)


Add zig

Simply, I have added zig file that outputs "Hello, world!" in the terminal.

You don't need to have , in the hello world sentence just like other languages.

I have seen that in the assembly one

But ok, i will change it


Rerun tests that run in Scala PR integration

I don't know what else to do. The tests are correct and pass on other
systems.
See failing tests:
https://scala-ci.typesafe.com/job/scala-2.11.x-integrate-ide/2200/console
https://scala-ci.typesafe.com/job/scala-2.11.x-integrate-ide/2195/console
Fix scala/scala-dev#96

build failure unrelated.


There is no time-out when Paragon App CAN not find Paragon unit

Open App > Hit 'settings' on Blue tooth setting pop-up > Do not turn on Blue tooth > Open Paragon App again > Hit ADD PRODUCT > Hit Paragon
There is no time-out when Paragon App CAN not find Paragon unit.
'Searching for your Paragon..' page shows endless.

@Zenah we added a back button to the screen which will allow the user to cancel when they want.


swift2 branch - Will not build for iOS 7 Xcode 7 beta 4

I get lots of errors when building with XCode 7 beta 4 for iOS 7.  Is iOS 7 no longer supported?

@matthewcarlreetz, right. We have to support dynamic frameworks, which only work on 8.x. This is also a requirement to use CocoaPods, see #50.


Clusterstaticentries don't get created

I am having some problems with clusterstaticentries not being created. I have SPIRE deployed with the helm-charts. I am using the following version:
spire-server: 1.11.0
spire-controller-manager: 0.6.0
spire-crd (helm): 0.5.0
spire (helm): 0.24.0
I am using the following helm values:
global:
  spire:
    clusterName: playground
    trustDomain: spire.internal.mattiasgees.be
    jwtIssuer: https://oidc-discovery.${domain}
    recommendations:
      enabled: false
    namespaces:
      create: false
    strictMode: true
    telemetry:
      prometheus:
        enabled: true
        podMonitor:
          enabled: true

spire-server:
  logLevel: debug
  nodeAttestor:
    joinToken:
      enabled: true
  unsupportedBuiltInPlugins:
    nodeAttestor:
      x509pop:
        plugin_data:
          ca_bundle_path: "/run/spire/ca-bundle/ca.crt"
  extraVolumes:
  - name: ca-bundle
    secret:
      secretName: laptop-tls
  extraVolumeMounts:
  - name: ca-bundle
    mountPath: /run/spire/ca-bundle
    readOnly: true
  caSubject:
    country: UK
    organization: My Org
    commonName: spire.internal.mattiasgees.be
  controllerManager:
    enabled: true
    identities:
      clusterSPIFFEIDs:
        default:
          autoPopulateDNSNames: true
          workloadSelectorTemplates:
          - "k8s:pod-name:{{ .PodMeta.Name }}"
          - "k8s:ns:{{ .PodMeta.Namespace }}"
          - "k8s:node-name:{{ .NodeMeta.Name }}"
  service:
    type: LoadBalancer

I am joining an external host through the x509pop and that works without any problem. I can see it showing up in the SPIRE server as spiffe://spire.internal.mattiasgees.be/spire/agent/x509pop/<fingerprint>
The Kubernetes auto-discovery with ClusterSPIFFEIds works correctly, but when I add a clusterStaticEntry for a workload running on that x509pop, it doesn't get created in the SPIRE server. The clusterStaticEntry looks the following:
apiVersion: spire.spiffe.io/v1alpha1
kind: ClusterStaticEntry
metadata:
  name: company-laptop-workload
spec:
  spiffeID: "spiffe://spire.internal.mattiasgees.be/static/macbook/company/workload"
  parentID: "spiffe://spire.internal.mattiasgees.be/spire/agent/x509pop/<fingerprint>"
  selectors:
    - unix:uid:505

In the SPIRE controller manager logs, I can see that the reconciliation is starting, but nothing else. It doesn't show any errors or further output.
2024-11-01T07:40:11Z	DEBUG	Triggering reconciliation	{"controller": "clusterstaticentry", "controllerGroup": "spire.spiffe.io", "controllerKind": "ClusterStaticEntry", "ClusterStaticEntry": {"name":"company-laptop-workload"}, "namespace": "", "name": "company-laptop-workload", "reconcileID": "423258f0-13fe-49c8-afb5-274807f2c846"}

Can you provide the configuration that is ultimately passed into the controller?

The chart by default does not watch all entries, but instead only watches ones with a generated className.
There's 4 ways to go from here.

If you create the static entries through the chart values, it will "just work" (ex: https://github.com/spiffe/helm-charts-hardened/blob/main/charts/spire/charts/spire-server/values.yaml#L670)
You can add the className setting to the static entry. can see one of the loaded clusterspiffeid's for the value.
You can do 2, along with specify the exact className you want the chart to use: https://github.com/spiffe/helm-charts-hardened/blob/main/charts/spire/charts/spire-server/values.yaml#L510-L511
You can have your instance honor unclassed objects too: https://github.com/spiffe/helm-charts-hardened/blob/main/charts/spire/charts/spire-server/values.yaml#L512-L513

Thanks, @kfox1111. That fixed it. I have explicitly added the classname (option 2), which now registers the workloads. I was using the docs of the CRDs and in there it didn't say anything on the classnames https://github.com/spiffe/spire-controller-manager/blob/main/docs/clusterstaticentry-crd.md

Ah. Yeah, we should add an entry for that in the document.


Can I use fp8 only when the code runs to the fp8 branch?

I am using deepspeed. ops. adam FusedAdam cannot go to the fp8 branch

deepspeed zero stage2

Hi @forevergj , thanks for your attention to our work!
The reason is that master weight, weight, weight gradient and optimizer states are all tensors with scaling factors. FusedAdam did not support the computation on scaling tensors.

Thank you for your reply！Where do the benefits of using msamp for training acceleration come from?

@forevergj
MS-AMP applies low-bit data formats on master weight, weight, weight gradient and optimizer states. It saves the GPU memory to allow larger batch size for faster training speed.

May I ask how to reproduce the results in the paper？ I used two H800GPUs.
msamp: samples per second: 8.130 | TFLOPs: 356.36
bf16：samples per second: 8.8| TFLOPs: 356.36
I run msamp and bf16 on https://github.com/Azure/MS-AMP-Examples GPT 6.7B, msamp has decreased performance compared to bf16.with this：
`#!/bin/bash
Runs the "6.7b" parameter model
GPT-13B: 32 layers, 4096 hidden size, 32 attention heads
set -e
USAGE="usage: bash pretrain_6b7_megatron.sh [bf16|te|msamp]"
export CUDA_VISIBLE_DEVICES=6,7
if [ "$#" -ne 1 ]; then
echo $USAGE
exit 1
fi
FP_TYPE=$1
export CUDA_DEVICE_MAX_CONNECTIONS=1
GPUS_PER_NODE=2
Change for multinode config
MASTER_ADDR=localhost
MASTER_PORT=6000
NNODES=1
NODE_RANK=0
WORLD_SIZE=$(($GPUS_PER_NODE*$NNODES))
VOCAB_FILE=$PWD/data/gpt2-vocab.json
MERGE_FILE=$PWD/data/gpt2-merges.txt
DATA_PATH=$PWD/data/wikipedia_text_document
DISTRIBUTED_ARGS="
--nproc_per_node $GPUS_PER_NODE
--nnodes $NNODES
--node_rank $NODE_RANK
--master_addr $MASTER_ADDR
--master_port $MASTER_PORT
"
GPT_ARGS="
--tensor-model-parallel-size 1
--pipeline-model-parallel-size 2
--distributed-backend nccl
--no-query-key-layer-scaling
--seed 43
--num-layers 32
--hidden-size 4096
--num-attention-heads 32
--seq-length 2048
--max-position-embeddings 2048
--train-samples 48828125
--lr-decay-samples 43945312
--lr-warmup-samples 2048000
--lr 3.0e-4
--min-lr 3.0e-5
--lr-decay-style cosine
--micro-batch-size 1
--global-batch-size 2048
--clip-grad 1.0
--weight-decay 0.1
--attention-dropout 0.0
--hidden-dropout 0.0
--optimizer adam
--adam-beta1 0.9
--adam-beta2 0.95
--init-method-std 0.0099
--num-workers 1
--bf16
--sequence-parallel
--use-flash-attn
--no-gradient-accumulation-fusion
--use-distributed-optimizer
"
DATA_ARGS="
--data-path $DATA_PATH
--vocab-file $VOCAB_FILE
--merge-file $MERGE_FILE
--data-impl mmap
--split 949,50,1
"
OUTPUT_ARGS="
--log-interval 1
--save-interval 1000
--eval-interval 500
--eval-iters 7
"
if [ "$FP_TYPE" = "bf16" ]; then
CHECKPOINT_PATH=$PWD/checkpoints/gpt_6b7_bf16
torchrun $DISTRIBUTED_ARGS ../third_party/Megatron-LM/pretrain_gpt.py
$GPT_ARGS
$DATA_ARGS
$OUTPUT_ARGS
--save $CHECKPOINT_PATH
--load $CHECKPOINT_PATH
elif [ "$FP_TYPE" = "te" ]; then
CHECKPOINT_PATH=$PWD/checkpoints/gpt_6b7_te
torchrun $DISTRIBUTED_ARGS ../third_party/Megatron-LM/pretrain_gpt.py
$GPT_ARGS
$DATA_ARGS
$OUTPUT_ARGS
--save $CHECKPOINT_PATH
--load $CHECKPOINT_PATH
--fp8-hybrid
--transformer-impl transformer_engine
elif [ "$FP_TYPE" = "msamp" ]; then
CHECKPOINT_PATH=$PWD/checkpoints/gpt_6b7_msamp
torchrun $DISTRIBUTED_ARGS ../third_party/Megatron-LM/pretrain_gpt.py
$GPT_ARGS
$DATA_ARGS
$OUTPUT_ARGS
--save $CHECKPOINT_PATH
--load $CHECKPOINT_PATH
--fp8-hybrid
--transformer-impl transformer_engine
--msamp
else
echo $USAGE
exit 1
fi
`

MS-AMP applies low-bit data formats on master weight, weight, weight gradient and optimizer states. It saves the GPU memory to allow larger batch size for faster training speed.

1.Does it mean that the fp8 operation of msamp only occurs after backpropagation when the optimizer updates the weights. However, forward propagation does not involve fp8 operations? 2.Is the main optimization focus on gradient communication time?

The forward propagation and backward propagation both involve FP8.
No. The acceleration benefits from FP8 matrix multiplication in the linear layers. MS-AMP reduces the GPU memory by low-precision data formats. It allows larger batch size for the further acceleration.

Is it similar to a transformer engine? What is the difference between msamp and transformer engine？

 Is it similar to a transformer engine? What is the difference between msamp and transformer engine？  Does the image in the MSAMP paper mean that the conversion of FP8 is executed in two places. They are before the all-gather gradient and before executing Linear. This means that the output of Linear is high-precision？

MS-AMP can be combined with TransformerEngine. The difference is that MS-AMP applies FP8 data format on master weight, weight, weight gradient, optimizer states, and the communication of gradient reduction.

Regarding to your second question, the output of LayerNorm is a high-precision tensor (FP16 or BF16), which is converted to an FP8 tensor with a scaling factor, as the input of the all-gather operation. The output of the all-gather operation is an FP8 tensor with a scaling factor, that is the input of the FP8 GEMM (YA). The output of the FP8 GEMM (YA) is a high-precision tensor (FP16 or BF16). Z is also a high-precision tensor, being converted to an FP8 tensor with a scaling factor.

Thank you for your reply. It's very clear


CI/CD: avoid building the inclavared rpm package in the pr tests and nightly tests

Signed-off-by: Yilin Li<EMAIL_ADDRESS>

Test results: https://github.com/hustliyilin/inclavare-containers/runs/2580768864?check_suite_focus=true#step:10:6


Tab bars scroll buttons, double-click creates new empty tab

Godot version
v4.0.beta16.official [518b9e580]
System information
Windows 10 22H2, Compatibility, Geforce GTX 550 Ti
Issue description
When the tab bars scroll buttons are double-clicked, then they create a new empty tab.
Expected: Scroll the tab bar twice.
Steps to reproduce

open the attached project: issue, next or prev tab buttons, double click adds empty scene.zip

The right side of the tab bar shows two buttons: scroll tabs left and scroll tabs right
when there are more tabs than fit on the tab bar.
When Godot is maximized on a display with a resolution of 1920x1080,
then the tab bar shows the tabs, from: scene_name - Copy (10) to: scene_name - Copy (15)
The project has 16 tabs, hopefully, that's enough to fill the tab bar on larger resolutions.
But if the person that tests this has a large enough resolution to show all tabs,
then the tester needs to either:
unmaximize Godot and possibly shrink Godot's width, until the tabs don't fit
or create and save additional scenes to overfill the tab bar

to the right of the last scene tab: scene_name - Copy (15),
click on the button to scroll the tab bar to the left

now the tab bar shows the tabs from: 9 to: 14

click the scroll tab bar left button twice within 1 second

Expected
The tab bar should scroll to the left twice, and show the tabs from: 7 to 12
Observed

A new [empty] scene tab was added to the right of scene 15,
the tab bar scrolled to the rightmost position,
and it now shows the tabs from 11 to 15 followed by an [empty] tab.
The same thing happens when the button: scroll tab bar to the right, is pressed twice within 1 second.
This means that it isn't possible to scroll the tab bar quickly, by pressing the scroll tab bar buttons quickly.
A possible cause
Double-clicking on an empty area of the tab bar, to the right of the rightmost tab, creates a new [empty] tab.
If the scroll tab bar buttons don't have a specific double-click action,
then maybe the double-click event is picked up by the tab bar, behind the scroll tab bar buttons.
This is possibly confirmed by the fact that the double-click action,
also happens when the scroll tab bar buttons are disabled.
The scroll tab bar right button is disabled when the rightmost tab is focused.
The scroll tab bar left button is disabled when the leftmost tab is focused.
Minimal reproduction project
issue, next or prev tab buttons, double click adds empty scene.zip

double-click creates new empty tab

This is intentional behavior, but it was discussed to remove it as it's obscure and rarely used.

To add my two cents, I "use" this feature all the time, but always accidentally when I'm just trying to navigate through tabs, which has caused me to compile my own version of the editor with this feature specifically removed. Especially since the "add new" tab button is right next to the arrow buttons anyways, I'd love to see this removed officially.

Ah, so that's what I've been experiencing all this time. I was wondering where all those new tabs were coming from. I also thought it was just incredibly buggy.
The whole scrolling arrows thing is terrible UX in the first place, and personally I'm making a plugin to replace the scene tabs entirely. But yeah this should be removed asap.

The ability to change scenes by scrolling the mouse wheel on the list of tabs was removed in 4.1 by https://github.com/godotengine/godot/pull/70800.

I am honestly completely confused by removing the ability to change scenes by scrolling and not even making it an option. If I have to use the two arrow buttons to scroll but I also have to take my time and carefully make sure I don’t click either of them too quickly to count as a double press so I don’t unintentionally make a new scene instead, I’m going to completely stop using the official builds and patch that back in myself. I consider that a usability nightmare.

I am honestly completely confused by removing the ability to change scenes by scrolling and not even making it an option. If I have to use the two arrow buttons to scroll but I also have to take my time and carefully make sure I don’t click either of them too quickly to count as a double press so I don’t unintentionally make a new scene instead, I’m going to completely stop using the official builds and patch that back in myself. I consider that a usability nightmare.

Unfortunately, options that are disabled by default suffer from poor discoverability. They often end up being dead weight in terms of maintenance, as we have to maintain something that will not be used often (and may be buggy for several months if not years).

Fixed by #79852


cudnn_status_mapping error

when I run your code with my own data. I encounter this problem"CUDNN_STATUS_MAPPING_ERROR"，my pytorch version is 1.5,cuda 10.2. not the same as yours.

https://github.com/pytorch/pytorch/issues/33134
I think it is due to the mismatch between your GPU and pytorch&CUDA version. Your GPU might not be compatible with the latest pytorch version. Please downgrade CUDA and pytorch according to your GPU edition.


feat: add Button

closes #656
Description

The button is centered both horizontally and vertically
The appropriate meta tags are included
There is no unnecessary code present
CSS variables are utilized for the color palette

@kuvamdazeus @Spyware007
I am tagging because there is no option to request a review :)


Error Calling NVAPI Function

I'm also getting this error. The exe will still launch, but the configuration section is completely blank. If saved, the game will fail to launch after making changes.

I get the same Issue, but after playing the game, exiting and reopening I got the settings area, then when restarting the game i got a prompt to update the exe oddly...

recently buy the game and installed this mod to play with my ds4 but i've been getting this message too, any news or an update on this?

I'm getting this error too. Did anyone manage to find a fix?


Copyright, PR/Issue template, Code of conduit and contributing.md

Description

Add Nordix Foundation Copyright header
PR/Issue template, Code of conduit and contributing

The contributing guide will be written later based on the Meridio one.

Issue link
/
Checklist

Purpose

[ ] Bug fix
[ ] New functionality
[x] Documentation
[ ] Refactoring
[ ] CI

Test

[ ] Unit test
[ ] E2E Test
[ ] Tested manually

Introduce a breaking change

[ ] Yes (description required)
[x] No

/reverify


Add now dev to stable

Codecov Report

Merging #2052 into master will decrease coverage by 0.32%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #2052      +/-   ##
=========================================
- Coverage    6.51%   6.18%   -0.33%
=========================================
  Files          90      86       -4
  Lines        4144    4072      -72
=========================================
- Hits          270     252      -18
+ Misses       3874    3820      -54

Impacted Files
Coverage Δ

src/index.js
0% <0%> (ø)
:arrow_up:

src/util/get-files.js
96.89% <0%> (-3.11%)
:arrow_down:

src/util/read-metadata.js
100% <0%> (ø)
:arrow_up:

src/util/get-project-name.js
100% <0%> (ø)
:arrow_up:

src/util/hash.js
100% <0%> (ø)
:arrow_up:

src/util/error.js
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


listen on ipv4 addresses by default

listen on :: if the listener IPFamily is IPv6 or DualStack
Set ipv4_compat to true if IPFamily is DualStack

will this make ipv6 test failed without additional IPFamily in EnvoyProxy?

will this make ipv6 test failed without additional IPFamily in EnvoyProxy?
Yes we'll need to explicitly set IPFamily in EnvoyProxy for the ipv6 test


Web component: Dotcom shell - Card with images are not operable.

Description
Issue:
Dotcom shell - Card with images are not operable.
For screen reader, cards are not identified as links.
Component(s) impacted

Browser
Chrome, Firefox
Carbon for IBM.com version
1.28
Severity
Severity 3 = The problem is visible or noticeable to users but does not impede the usability or functionality. Affects minor functionality, has a workaround.
Application/website
Web component
Package
@carbon/ibmdotcom-web-components
CodeSandbox example
NA
Steps to reproduce the issue (if applicable)
No response
Release date (if applicable)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate issues

@kennylam Here is the dotcom shell Card with images bug we talked about in the meeting earlier today.  If you could take a deeper look to be sure that this issue to confirm that this should not be fixed in an rc, that would be great.  If our suspicions are correct, then remove yourself as the assignee and move the issue into the Pending scheduling pipeline.


nuttx/libc:Add _dl_find_object and dl_iterate_phdr function.

Add _dl_find_object() function, because when cxx_exception configuration is enabled, a link error occurs and that function cannot be found.
ld: /usr/lib/gcc/x86_64-linux-gnu/13/libgcc_eh.a(unwind-dw2-fde-dip.o): in function _Unwind_Find_FDE': (.text+0x250c): undefined reference to _dl_find_object'
Summary
Impact
Testing

[Experimental Bot, please feedback here]
Let's break down if this Pull Request (PR) meets the NuttX requirements.
Summary: The PR seems incomplete based on the provided description.

Why is this change necessary?  The description mentions a link error related to cxx_exception configuration and  _dl_find_object. This gives a hint, but it needs more clarity:

Is cxx_exception a new configuration? If so, why was it added?
What causes the link error when this configuration is enabled?
Why is  _dl_find_object the specific solution?

What is being changed?  State explicitly that the change is the addition of the _dl_find_object function. Specify where it's being added (which file/module).
How does it work? Briefly explain the function's purpose and how it resolves the link error.

Impact: This section needs more details.

New feature?  Is cxx_exception (assuming this is the case) a new user-facing feature?
Impact on... The PR description currently answers "NO" to all impact questions without elaboration. Given the nature of the change (handling link errors, potentially related to exception handling), there might be impacts on:

Build: Does this change the build process in any way (new dependencies, configuration options, etc.)?
Hardware/Architecture: Does this change affect specific architectures or boards?
Compatibility: Does this affect compatibility with existing codebases that might be using NuttX?
Security:  Exception handling mechanisms can sometimes have security implications. While this might not be the case here, it's worth considering.

Testing: The testing section needs significant improvement.

Provide context:

What specific "local setup" was used for testing (OS, CPU architecture, compiler, NuttX version)?
Which target architectures and boards were tested?

Show relevant logs:

The current PR description has placeholders for logs. You must include the actual logs from before and after the change to demonstrate the issue and its resolution.
Focus on the relevant parts of the logs that highlight the error and how your change fixes it.

In summary, this PR needs more work before it can be considered complete.
Recommendations:

Expand the Summary: Provide a clear and concise explanation of the problem, the proposed solution, and the reasoning behind it.
Detail the Impact: Thoroughly analyze the potential impact of this change on different aspects of NuttX.
Provide Concrete Testing Evidence: Include detailed information about your testing environment and the relevant logs that demonstrate the problem and the effectiveness of your solution.