WIP : Subsurface scattering

This PR is a work in progress
It will :

add a subsurface scattering post process
enable to render the scene directly on a multiple render target, with additionnal RT being used to store normals, depth, light info etc..

TODO :

Rename lots of stuff and cleanup code
Add render to MRT in every shader
Add API to add fallback for ShaderMaterial and propose convention for variable names
WebGL 1 disable and WebGL 1 + drawbuffers check
reorganize RTs for more efficient storing and texture fetches
add a simple API to mark materials with subsurface flag
adding uniforms for diffusion profiles etc
PG example
Doc

Could you reopen the PR as a draft ? might prevent early merge ?

Also we probably should rename HIGH_DEFINITION_PIPELINE to DEFERED_PIPELINE to be future looking and allow it to default to half float vs float .
From the shader, I guess this would not work in Webgl1 which I do not mind but might be a good idea to emit a warning/error in this case ?
I ll look more in detail once reopened as a draft but the big picture looks overall good :-)

Review has been done offline with more ideas to prevent changing all the materials.

2 PG are available for testing :
Neutral diffusion profile on stanford dragon (kinda jade material) :
http://localhost:1338/Playground/index-local.html#GTQKYK
Skin diffusion profile on Lee Perry Smith head
http://localhost:1338/Playground/index-local.html#W7DYG2
I'm still not removing draft status because there is still some minor bugs remaining :

Material becomes blue when metallic is set to 1 and SSS is enabled
There is a ghost frame when activating SSS, while the shader is compiling
WebGL 1 needs to be disabled gracefully

@sebavan we're in touch about that offline tomorrow :)

Comments might seem short but did the review in pair programming with @CraigFeldspar  :-) and this is going to be great !!!!

Actually, last point, is it possible to split the prepass renderer and the subsurface specific infos ???
It feels weird to set the metersPerUnit on the prepass which could be generic for deffered ?

Actually, last point, is it possible to split the prepass renderer and the subsurface specific infos ???
It feels weird to set the metersPerUnit on the prepass which could be generic for defered ?
It might not be so no worry we ll check if we could make it more generic later on.

You're totally right, as this will be the renderer for several effect.
What about prePassRender.xxxConfiguration ?
So we could have subSurfaceConfiguration, ssaoConfiguration etc...

I also fixed a bug on postprocesses that caused the playground to infinite loop on reload

@CraigFeldspar as soon as the build is green we can merge it !!!

Helll Yeah !!!!!


Reproduce State of CA Current Report figures

Hi @evsatt,
Is the latest version of the report here?

Weber et al (2021) State of the California Current 2019–2020: Back to the Future With Marine Heatwaves? Front. Mar. Sci.

And specific to the figures:

How would you prioritize production of these figures online?
What interactivity would you like to see with them?
Can all the data sources be identified in the database?
Are there any existing code or instructions available for creating these?

Will use this issue as parent to individual issues per figure.

Map as snapshot of time series summary; play as movie
https://shiny.marinebon.app/seascapes/

Supplementary Figures
Figure S1. Marine heatwave duration

Supplementary Figure 1. Duration of individual large marine heatwaves vs maximum area of each heatwave, from January 1982 to October 2020. Background color indicates the number of heatwaves (out of 208), whereas outliers are marked with numbers indicating the year the heatwave was initiated. Methods for tracking and classifying heatwaves are described in Thompson et al. 2019 and https://www.integratedecosystemassessment.noaa.gov/regions/california-current/cc-projects- blobtracker.
Figure S2.

Supplementary Figure 2. Sea level pressure and wind speed cubed anomalies in the North Pacific Ocean for fall (September-November) 2019, winter (December-February) 2020, spring (March- May) 2020, and summer (June-August) 2020. Labeled contours are for sea level pressure at 25 m3s3 and shaded contours are wind speed cubed at 1hPa intervals. Climatology period is 1980- 2010 for both data sets. Wind NCEP/NCAR Reanalysis data from http://www.esrl.noaa.gov and U.S. Navy Fleet Numerical Meteorology and Oceanography Center (FNMOC) pressure data from https://coastwatch.pfeg.noaa.gov/erddap/griddap/erdlasFnPres6.
Figure S3.

Supplementary Figure 3. Time series plots of local physical and biological anomalies (monthly climatology for the same period removed) from 1996–present at NH-25 (Latitude: 44.6517 N Longitude: 124.65 W; top panel) NH-5 (Latitude: 44.6517 N Longitude: 124.1770 W; lower five panels) along the Newport Hydrographic Line. Temperature are from 150 m and 50 at NH-25 and NH-5 respectively, Oxygen data are from near bottom (50 m; blue line indicates 1.4 ml/L with values <1.4ml/L considered hypoxic), NO2 + NO3 are from the surface, and copepod biomass anomalies are integrated over the upper 60 m. All data except oxygen data were smoothed with a 3-month running mean to remove high frequency variability.
Figure S4.

Supplementary Figure 4. Recent hydrographic observations at station TH02 (mid-shelf) along the Trinidad Head Line in context of seasonal climatology. Panels from top to bottom show A) temperature at 15 m, B) temperature at 65 m, C) salinity at 15 m, D) salinity at 65 m, E) mean (uncalibrated) chlorophyll concentration from 2-30 m, F) dissolved oxygen concentration at 65 m. Grey points indicate individual cruise observations. Thin black line indicates climatology derived from GAM of environmental parameter on day-of-year; light grey ribbon about this mean indicates range of mean residual around this climatology (based on GAM of absolute residuals on day of year). Dark grey line indicates observations during 2019. Black line indicates observations during 2020. Break between March and June 2020 indicates missed sampling due to COVID-19 restrictions.
Figure S5.

Supplementary Figure 5. Hovmoller diagrams of temperature (°C) and salinity (PSU) anomalies at 10 m from CUGN gliders on CalCOFI lines 66.7, 80.0, and 90.0. The climatology period is 2007-2020.
Figure S6.

Supplementary Figure 6. Anomalies of hydrographic properties at the σ=26.4 isopycnal (open diamonds) averaged for each cruise over the standard CalCOFI stations. Shown are anomalies of A: isopycnal depth, B: spiciness, C: oxygen, D: concentrations of nitrate and E: N*. The solid blue line connects annual averages. The climatology period is 1984-2013.
Figure S7.

Supplementary Figure 7. Cruise averages of mixed layer properties for the 66 standard CalCOFI stations, 1984 to the summer of 2020. A: the log10 of chlorophyll-a, B: the cube root of nitrate, and C: nitracline depth. Whiskers indicate the 95% confidence intervals for the means. Red solid lines represent annual averages, grey horizontal lines the climatological mean, which is zero in the case of anomalies. Anomalies are based on the period 1984-2013.
Figure S8.

Supplementary Figure 8. a) Integrated primary production from the Vertical Generalized Production Model of Behrenfeld and Falkowski (1997a, 1997b) with data from VIIRS sensor for the week of 1-6 December 2019. b) Bar graph of net primary production at 30% surface irradiance (~ 18-25 m), integrated primary production from VGPM model andrate of respiration of the microbal community with the rate of respiration of procaryotes. c) spatial distribution of net primary production at 30% light. d) spatial distribution of the rate of respiration of the microbal community at 30 % of surface irradiance.* http://sites.science.oregonstate.edu/ocean.productivity/vgpm.model.php
Figure S9.

Supplementary Figure 9. Abundance anomalies (mean 2007-2019) for the main grazing groups (Copepoda, Euphausiacea, Appendicularia, Thaliacea; Line 100 and Line 103, Station 30). Data were converted to logarithms previous estimation of the anomalies, removing the seasonal means.
Figure S10.

Supplementary Figure 10. Zooplankton volume off north Baja California (Lines 100-117). Small volume a) is the displacement volume after removing large organism (>5 ml) and Total volume b) includes the complete zooplankton sample. The anomalies were estimated removing the seasonal means (2007-2019). The lower panel c) shows the composition of the main large organisms removed. The colors indicated the geometric means of red crabs, salps, pyrosomes, heteropods, the pteropod Corolla spectabilis, medusae, and ctenophores.
Figure S11.

Supplementary Figure 11. Sampling area in orange (a) and size range (b) for albacore tuna. Mean lengths (black diamond) and boxplots of length distributions for market squid (dark grey) and forage fishes (light grey) are depicted in panel (c). Fish prey size were quantified as standard lengths (SL), while market squid size was quantified as mantle length (ML). Annual variability from the time series mean proportional abundance of prey (“mean”, and "sd") are given as a heat map of z-scores in units of standard deviations (“sd”) in panel (d). Non CPS, groundfish, or market squid prey were included in an "other" group.
Figure S2.

Supplementary Figure 12. Whale sightings during CalCOFI cruises in 2019–2020. CalCOFI stations are represented by black dots and the ship’s trackline is represented as a solid black line between stations.

Hey @bbest Here is a mockup of the things that I am thinking are important in the app:
https://docs.google.com/presentation/d/1X_09_XEpOzmv8XVyZZclnxparnBfZ_go/edit#slide=id.gf42739c32a_0_57
Please feel free to edit and provide feedback
I think we captured the main ways to visualize the data including:
-Space
-time
-depth
-Space, time
-Depth, space
-Depth, space, and time

Another thought for the video component (not including the current/particles) but more broadly as a concept: https://faculty.washington.edu/pmacc/LO/p5_PS_speed_top.html


Automatically build NLP domain

It shouldn't be too hard to automate building the NLP domain after you're adding a new OcrTable record. Either of these approaches should work without much effort:

Just call ##class(dc.ocr.OcrNLP).%Build() after you've added a new record. This will build the whole domain from scratch, so for anything but a demo you'd want to look at ways to just pick the modified/new records

Rather than work with a domain that loads from a table, remove the <table> data source, make sure to set/keep allowCustomUpdates="true" and push the text in directly by using $SYSTEM.iKnow.IndexString() as soon as it's OCR'ed

I tried to use it:
//save the results into database to use text analytics - nlp
Set ocrTable = ##class(dc.ocr.OcrTable).%New()
Set ocrTable.FileName = file.Filename
Set ocrTable.OcrText = pResponse.StringValue
Set tSC = ocrTable.%Save()
// re-index NLP to the new extracted text
Do ##class(dc.ocr.OcrNLP).%Build()
But I got:
ERROR #5002: ObjectScript error: <INVALID OREF>zOnProcessInput+35^dc.ocr.OcrService.1

are you sure that error is for the line with %Build()? it's a simple class method, so you should not have any OREFs that could possibly be INVALID.

I right, but I tried the 2 alternatives, and the oref error goes out, but another error appears, and the error is not printed in the log, I don't have idea what is it.

@bdeboe I resolved, I was with errors in the status calls only, not related to iKnow, thanks your tip. I choose alternative 2 to your tip


questions about this library

Hi,
I have 4 questions basically(i am new to this library and would like to make a decision whether to use this library or some other library. Hopefully someone can answer my questions)

Does this library support websockets?

This is what I want to accomplish in my application. Basically users(from a mobile device - Android / iOS) will send an audio stream to the server, the audio stream is then passed on to my application where it is transcribed and the text is sent back to the client.
Can I accomplish this using this library? Or is it an overkill?

Does this library have necessary API to integrate into a C++ application?

How many requests can this library handle? in thousands? Should I consider Proxygen? Any suggestions?

Thanks in advance for your responses.

Please see Simple-WebSocket-Server. The WebSocket implementation is quite different from a Web Server implementation, so I decided to keep the projects separate.
Your task here sounds like it should be implemented using WebSocket
Yes, this is the main reason why I created these libraries, and that people should not need to learn all the nuts and bolts of (Boost.)Asio in order to create such services using C++.
The overhead of the libraries are very low. Just remember to do a release build (-O3) if you want serious speed.

Finally, Asio, and networking in general, is somewhat complex, and I do not think that someone has made the tests run successfully using MSVC yet. You should be fine though if you use GCC or Clang on a Unix-like system.

Hi eidheim,
thank you for your answers. i will look into the websocket server as you suggested.
thanks

@eidheim is there an example anywhere of using Simple-Web-Server to upgrade a connection to WS using Simple-WebSocker-Server? That would be slick. Actually a client example would be slick too. I have use cases for receiving streaming data and re-streaming data back out to an endpoint.

@SethHamilton you can have a look at https://github.com/eidheim/desktop-stream that uses upgrade in order to have a web server and a websocket server share the same port.
I have never considered supporting upgrade on the client side, since the websocket upgrade is always initiated by the client if I'm not mistaken.


Characteristic.WriteAsync not hitting connected ble device for Android

Steps to reproduce
I am unable to use the Characteristic.WriteAsync method for Android Devices. I am able to connect to a device and call GetServicesAsync(). However, the WriteAsync command does not appear to reach my device. No exception is thrown, my device just does not respond. The same code works as expected in iOS.
code snippet:
connectedDevice = await adapter.ConnectToKnownDeviceAsync(uuid, ConnectParameters.None, cancellationToken);
var services = await connectedDevice.GetServicesAsync();
var commandService = services.First(x => x.Id == commandBLEServiceId);
    if (commandService != null)
    {
        var availableCharacteristics = await commandService.GetCharacteristicsAsync();
        var writeCharacteristic = availableCharacteristics.First(x => x.Id == writeCharacteristicId);
        status = await writeCharacteristic.WriteAsync(Encoding.ASCII.GetBytes(msg));

AndroidManifest.xml:

Expected behavior
WriteAsync command should hit the connected ble device
Actual behavior
The command does not appear to hit the connected ble device
Crashlog
N/A
Configuration
2.1.1
Platform:
Android
Device:
iPhone XS Max

Hi @gardo7,
I'm the friendly issue checker.
Thanks for using the issue template :star2:
I appreciate it very much. I'm sure, the maintainers of this repository will answer, soon.

Resolved -- Issue was the size of the message, fixed by requesting a larger MTU size: IDevice.RequestMtuAsync(int mtuSize)


Make it work without Git

https://github.com/box-project/box2/issues/171

@paulovitorbal there is actually several tests where the PHARs are being built without a Git repository to be present. I also tested it manually and it works fine. The only cases where it will fail is if you attempt to use git related replacement values: https://github.com/humbug/box/blob/master/doc/configuration.md#replaceable-placeholders


Add support for building and hosting mdBook content

Motivation
One of the best aspects of docs.rs is that it encourages developers to create high-quality API-focused documentation by removing roadblocks around building and hosting the generated content.
I think it would be wonderful to extend this concept to building and hosting longer-form documentation produced by mdBook. This will encourage developers to produce this style of content as a complement to the rustdoc-generated documentation.
Guide-level explanation

Add this to your Cargo.toml:
[package.metadata.docs.rs]
book = "some-subdir"

Push your code to crates.io

After a bit, visit https://docs.rs/book/crate-name/x.y.z

Drawbacks

Increased load on the build machines.

May be negligible compared to building dependencies

Unresolved questions

How do we make access to the book "obvious"?

Suggestion: Make it another tab in the top bar
Suggestion: Make it another button next to 'Documentation'

How do we handle different themes between rustdoc and mdBook?

Suggestion: Don't worry about it, allow them to differ
Suggestion: Add "rustdoc dark theme" and "rustdoc light theme" to mdBook, only enable those themes + ayu (the common themes) when generating the book

How frequently can books be built?

Suggestion: Only when crates are pushed

How does a user access the book content?

Suggestion: https://docs.rs/book/crate-name/x.y.z
Suggestion: https://docs.rs/crate-name/book/x.y.z
Suggestion: https://docs.rs/crate-name/x.y.z/book-name (where book-name is the name of the book, not a hard-coded string)

How does the builder access the book source?
Do we support multiple versions of mdBook?

Suggestion: Only support the latest version

Do we support mdBook plugins?
Do we support multiple books per crate?
How do we bind the JS events for the theme change? Should we limit it to the book or apply it to the rest of docs.rs too?

Future possibilities

Improve the UX on docs.rs to make the integration better

Please feel free to edit this summary comment!

How do we handle different themes between rustdoc and mdBook?

Don't worry about it, allow them to differ

This is actually my biggest worry: it's very important to keep a unified UX. I guess it can be simply fixed by adding "rustdoc dark theme" and "rustdoc light theme" in mdbook and then allow enable rustdoc dark, ayu and rustdoc light themes when generating the mdbook.

My first thought on the URL would be to have it like https://docs.rs/crate-name/x.y.z/book-name (I'm not sure if mdBook has some kind of book-name, that could be the subdir name if not). That allows for compatibility with multiple-books, but does bring up questions about what to do if the book-name conflicts with one of the targets (especially if we start building documentation for binary and example targets too).

Do we support multiple books per crate?

Handling multiple books sounds hard and I think hosting your own site where you can control the styles makes sense at that point.

How do we bind the JS events for the theme change? Should we limit it to the book or apply it to the rest of docs.rs too?

This should hook in to the rest of the themes I think, the same way rustdoc themes work now. If mdbook has a theme docs.rs doesn't recognize, make no change.

Do we support mdBook plugins?

I think for an MVP we can just have no plugins and add them later if people want them.

See also https://github.com/rust-lang/rust/issues/66249


Read/Write Bandwidth Chart shows "No Data Available"  in Server Dashboard

Description
I have installed lustre 2.9.0 on bare metal.
MGS/MDS :  0c-c4-7a-15-e1-9c (hostname)   mgt:  /dev/sdb2  mdt: /dev/sdb1
OSS0: 00-25-90-c0-f7-c8   ost0:  /dev/sdb
OSS1: 00-25-90-c0-f7-88   ost1:  /dev/sdb
IML 4.0.0 Preview 2  installed on host: pxe-15-253
I have added 3 of the lustre servers into IML, detect lustre file system and everything looks good.
Read/Write Bandwidth Chart shows "No Data Available" In Server Dashboard,

Repro

add repo as shown in https://github.com/intel-hpdd/intel-manager-for-lustre/wiki/Installing-IML-Preview1-on-a-cluster
download and install IML  4.0.0 Preview 2  with command:  ./install --no-repo-check  --no-dbspace-check
login IML
add lustre servers to IML in Configuration->Servers  as "Monitored Storage Server"
click "Detect File Systems" and import lustre file system into "File Systems"

Version
IML 4.0.0 Preview 2
CentOS 7.3.1611  3.10.0-514.2.2.el7.x86_64
Please help.
Thanks!

Can you reproduce this using Lustre 2.10?  That is the version that IML is currently targeting and being tested with the preview releases.

Seems to be working in the latest lustre:

It seems to be working in Lustre 2.10:

I was looking at the wrong server. The MDT server does have the bug:

In my case, every server has the same bug.

There is an Error Occurred when I click on the "Chart Icon" at the right side of  "File Systems -> lustre -> OSTs (MDTs/MGTs)-> lustre-XXX0000x"

Message:

[$interpolate:interr] Can't interpolate: configure/server/{{targetDashboard.target.active_host | extractApi}}
Error: could not extract api id from undefined
http://errors.angularjs.org/1.5.11/$interpolate/interr?p0=configure%2Fserver%2F{{targetDashboard.target.active_host | extractApi}}&p1=Error%3A could not extract api id from undefined

Client Stack Trace:

Error: [$interpolate:interr] Can't interpolate: configure/server/{{targetDashboard.target.active_host | extractApi}}
Error: could not extract api id from undefined
http://errors.angularjs.org/1.5.11/$interpolate/interr?p0=configure%2Fserver%2F{{targetDashboard.target.active_host | extractApi}}&p1=Error%3A could not extract api id from undefined
at https://<IP_ADDRESS>/gui/main.99de3e4adcf92226aeb5.js:12:374
at Function.ci.interr (https://<IP_ADDRESS>/gui/main.99de3e4adcf92226aeb5.js:12:113844)
at l.exp (https://<IP_ADDRESS>/gui/main.99de3e4adcf92226aeb5.js:12:53836)
at Object.pre (https://<IP_ADDRESS>/gui/main.99de3e4adcf92226aeb5.js:12:39867)
at https://<IP_ADDRESS>/gui/main.99de3e4adcf92226aeb5.js:12:5636
at Mt (https://<IP_ADDRESS>/gui/main.99de3e4adcf92226aeb5.js:12:40610)
at p (https://<IP_ADDRESS>/gui/main.99de3e4adcf92226aeb5.js:12:32827)
at a (https://<IP_ADDRESS>/gui/main.99de3e4adcf92226aeb5.js:12:28666)
at https://<IP_ADDRESS>/gui/main.99de3e4adcf92226aeb5.js:12:28197
at https://<IP_ADDRESS>/gui/main.99de3e4adcf92226aeb5.js:12:30967

@brianjmurrell Which IML version is being tested and works well for lustre 2.9.0 ？
Thanks a lot.

Hi @jiamliang, there is currently no version of IML that is tested with Lustre 2.9.
That said, we are looking at updating the way we interact with Lustre; which may open the door to supporting versions prior to 2.10.
Thanks for reporting that stacktrace btw, should have that fixed shortly.

@jgrund Thanks a lot.


Remove arrows

it seems like it isn't easy to remove the trailing icon in ExpansionPanel.There is an open issue to fix this on the Flutter repository (https://github.com/flutter/flutter/issues/108973). or alternatively to modify a local version of Flutter that is not clean at all. @CurtlyCritchlow What do you think?

We'll have to leave it as is or build our own widget.

I suggest leaving it as it is for now...


jobs_runs_wait_for_completion doesn't retry correctly

Expectation / Proposal
I may be misunderstanding something, but I have a custom flow that calls jobs_runs_wait_for_completion under the hood. I am calling my custom flow with additional options like this:
custom_flow.with_options(retries=1)

However, when the underlying Databricks job fails, I see it acknowledged by Prefect but I don't see the Databricks job ever actually rerun.
Am I misunderstanding something basic about how to use jobs_runs_wait_for_completion in combination with with_options(retries=) to get automatic retries of the called Databricks job?
Traceback / Example
17:37:34.392 | INFO    | Flow run 'amorphous-sawfly' - Received non-final state 'AwaitingRetry' when proposing final state 'Failed' and will attempt to run again...
17:37:36.726 | INFO    | Flow run 'encouraging-bustard' - Finished in state Completed()
17:37:36.860 | ERROR   | Flow run 'amorphous-sawfly' - Encountered exception during execution:
Traceback (most recent call last):
  File "/Users/nchammas/Library/Caches/pypoetry/virtualenvs/ozawa-prefect-LtscNPhu-py3.9/lib/python3.9/site-packages/prefect/engine.py", line 833, in orchestrate_flow_run
    result = await flow_call.aresult()
  File "/Users/nchammas/Library/Caches/pypoetry/virtualenvs/ozawa-prefect-LtscNPhu-py3.9/lib/python3.9/site-packages/prefect/_internal/concurrency/calls.py", line 292, in aresult
    return await asyncio.wrap_future(self.future)
  File "/Users/nchammas/Library/Caches/pypoetry/virtualenvs/ozawa-prefect-LtscNPhu-py3.9/lib/python3.9/site-packages/prefect/_internal/concurrency/calls.py", line 316, in _run_sync
    result = self.fn(*self.args, **self.kwargs)
  File "/Users/nchammas/dev/spx/ozawa/prefect2/ozawa_prefect/flows/databricks/__init__.py", line 66, in run_job_or_fail
    raise Exception(
Exception: {'result_state': 'FAILED', 'run_page_url': 'https://...cloud.databricks.com/?o=xxx#job/126796463608738/run/28379095', 'notebook_output': {}, 'error': 'ArithmeticException: / by zero'}
17:37:37.000 | ERROR   | Flow run 'amorphous-sawfly' - Finished in state Failed("Flow run encountered an exception. Exception: {'result_state': 'FAILED', 'run_page_url': 'https://...cloud.databricks.com/?o=xxx#job/126796463608738/run/28379095', 'notebook_output': {}, 'error': 'ArithmeticException: / by zero'}")

[x] I would like to help contribute a pull request to resolve this!

I migrated to jobs_runs_submit_by_id_and_wait_for_completion and don't see this issue with retries anymore, so I'm not sure what's going on. But we were perhaps misunderstanding or misusing jobs_runs_wait_for_completion somehow.


Add CSS listbox size parameters

This PR attempts to make the default CSS more universal with regard to size. It does a few things:

Make the listbox size explicitly configurable with custom CSS properties. This includes the width and the height, but also a toggle for forcing the scaling of the listbox to the left. This should make the default CSS easily usable on pages with a wide range of search box locations. The demo page is updated to present this.
Remove the @media sections. They were very opinionated, the current approach is more flexible. The viewport width ranges didn't match each other, so there were actually 3 width ranges, each of them with a different set of CSS attributes. The result was that  only in the wide viewport the listbox was flexible and good looking, and narrowing it caused glitches.
Move .tsmb-form--slash::after sections closer to each other, so it's easier to follow the slash icon behavior
Clean up white spaces

I've committed the clean ups to the demo in https://github.com/jquery/typesense-minibar/commit/21eaf972c563a959307ee947ba59dee0a6167e46 as a start.

@CodeSandwich Thanks for clarifying, I missed the interaction between min-width and width. I think I made some local changes before propertly testing that I overly confident in would not affect the outcome. I'll have another look!

@CodeSandwich I've narrowed down where it goes wrong in the demo. I modified the patch to limit the margin-left and right: 0 only to demo-right-form, so as to keep the first row as a a themed version of the default left-aligned mode.
In that mode, the listbox starts to overflow the document at certain widths, creating a scrollbar:

Debugging it further, I realize this is specific to the --tsmb-size-listbox-width override. With the default value of min(30rem,60vh) it indeed doesn't cause an overflow in the demo. And more specifically, it is due to the minimum value of 50rem. This makes sense indeed as 30rem is approximately the width of the logo (10rem) + search input (20rem), and when right-aligned, it can go under both. I didn't understand at first where those values came from. It makes sense to me now.
I've updated my branch to move the width and listbox-width override to the demo-right-form class as well.
The remaining question then is:

Why 30rem
Why 60vh

I believe 30rem, is the closest round number to match the previous default of 500px. Is that right? This is a bit wider than the default input width of 20rem, so there is a chance of overflow there, but seems to work out in practice since 60vh is likely (or guruanteed?) to kick in by that point and trim it down, until eventually it matches the width of the input field (20rem), after which both input and listbox shrink together on narrow viewports.
Does that sound about right?
I've uploaded my amendment at https://github.com/jquery/typesense-minibar/commit/7b47cbc73dbdc66231afb683544447ed1978bebd.
Regarding the new CSS variables. The motivation for the variables is currently to create a stable way to set values that are (or likely will) be used in multiple places, or otherwise need to apply via a rule or selector that isn't selectable through public and stable selectors. For example, the theme colors and font sizes are used in various ways that I think are more future proof this way.
The Styles API shows which selectors are considered "stable", and that includes .tsmb-form [role=listbox]. Can you share a few words around why you'd prefer to set max-height and width via a variable rather than directly? For example, do you foresee re-use of variables on another element or in another calculation at some point?

Thank you for looking so deeply into it! Before diving deeper into your branch, have you tried this PR in its vanilla form, as in checking out commit https://github.com/jquery/typesense-minibar/pull/2/commits/ce54138b9a2ef2b07768e89bff2786bdd435b469 and seeing if things are working as expected? This could be a good starting point, it would be easier to pinpoint the bugs and the unwanted behaviors, because this commit doesn't have layers of patches in the demo yet.

The Styles API shows which selectors are considered "stable", and that includes .tsmb-form [role=listbox]. Can you share a few words around why you'd prefer to set max-height and width via a variable rather than directly? For example, do you foresee re-use of variables on another element or in another calculation at some point?

Sorry, I haven't noticed the API-Style.md file, I've updated it and pushed to my branch. I would prefer max-height and width as variables to make them stable and explicitly exposed. These are the parameters that the person putting minibar on their page probably should customize, should be able to do that without going too deeply into minibar's default CSS file, and shouldn't break when upgrading minibar.

Could you explain what is your goal of these changes?

I've uploaded my amendment at https://github.com/jquery/typesense-minibar/commit/7b47cbc73dbdc66231afb683544447ed1978bebd.

It seems to drop the CSS variable for width, but the demo page seems to be looking good.

@CodeSandwich I did test the patch as-is, but it doesn't give me the level of confidence I'd like as the themed mode set both bars to be right-aligned. This left it unclear how the left-aligned case would behave, with uncertainty/ambiguity over the width override (is the value only appropiate for right-aligned? do we require users to calculate a different override for left? or would the default work as-is in a flebox context for left-aligned mode and without overflow bugs?).
Those questions reflect the modifications I made locally.

I would prefer max-height and width as variables […]. These are the parameters that the person putting minibar on their page probably should customize, […]

Thanks for mentioning that. The reason I highlighted (promise stability, reduce selector complexity and/or permit re-use) don't cover this, but I like the idea of recommending them. We could evolve that later into a short-list of recommended styles to consider setting, but that's for later. The reason I'm leaning in that direction is that I don't see a way that these could become anything other than explicitily setting 1 specific CSS propertly on 1 specific stable selector. I don't want to discourage setting individual CSS styles, and would likely decline feature requests that add variables for properties we don't set by default. But for built-in ones, especially if it's likely that each user should think about setting it, I think that makes for a fine addition. Let's roll with it!

Thanks for the de-facto merge! And a separate thank you for all the effort you put into this PR :tada:

@CodeSandwich You're welcome. I want to clarify that your patch wasn't wrong. It just took me a few tries to understand everything that went into it. We could draw the themed/unthemed line differently and also demo/test the custom width behaviour both left and right aligned (as you did originally).
I ended up with a slightly differently opinionated outcome, but this was more the result of me working with it to understand it, than disagreeing on anything in particular. I hope that makes sense!


Pseudocode

Is there a way to properly highlight such pseudocode in a single
 element?

.button {
	background-image: url(button-with-fake-text.png);
}

Currently it looks like this

I guess it could be achieved by putting


Update Set-AzureADMSPrivilegedRoleSetting.md

Single quotes ' is invalid JSON, which breaks the role setting. Instead, escape double quotes inside of a string using the tick character `

Thanks @Virgil-Bulens for your contribution.
@CelesteDG , this is good to go.


/etc/hosts not updated when the host name is changed via web UI

Steps to reproduce: The host name was changed to test-raspi. The Raspberry Pi was rebooted:
# hostname
test-raspi
# ping test-raspi
ping: bad address 'test-raspi'
# cat /etc/hosts
<IP_ADDRESS>       localhost
<IP_ADDRESS>       homematic-raspi

Other applications cannot resolve the name either (mdzio/ccu-historian#90).

Thanks for the hint. Please note, however, that changing /etc/hosts is simply not supported yet. It is also questionable if this is really required.
In your example the ping should normally work if your DNS would work if expected. Please check the content of /etc/resolv.conf in which the domain and nameserver setting should normally match what your DNS (e.g. fritzbox) expects. If this is correctly setup, then a simple ping test-raspi should work if your DNS (e.g. FritzBox) is also using that name.

I ran some more tests: It works as expected, if not a fixed IP address is configured in RaspberryMatic.
/etc/resolv.conf has now the expected content. For all I care, this issue can be closed. Workarounds exist for the other case.

Content of /etc/resolv.conf with a fixed IP address:
nameserver <IP_ADDRESS>
nameserver <IP_ADDRESS>

The router now shows the correct name and ip address. The error characteristics are very diffuse. I am still waiting for further feedback from other users.


Providing DHCP zones for on-premise usage from Azure.

Am I correct in reading this as the DHCP server role is not supported on Azure VMs even if it's only providing on-prem DHCP ranges via a DHCP relay forwarding requests to it's Azure hosted NIC?
If so, what is the supported mechanism for removing on-prem DHCP servers while still maintaining integration with DNS zones stored in Active Directory servers running in Azure?

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 2b703446-c838-fd33-09fe-1df826399931
Version Independent ID: ff85d0c9-ecb0-987b-7b4c-fa252a93c55b
Content: Microsoft server software support for Microsoft Azure Virtual Machines - Virtual Machines
Content Source: support/azure/virtual-machines/server-software-support.md
Service: virtual-machines
Sub-service: vm-support-statements
GitHub Login: @genlin
Microsoft Alias: genli

@ChrisParrBIE, Thanks for submitting the issue. For technical issues/questions, please contact our support, or ask Azure community support.

Hi @genlin ,
I was mostly after clarity in the documentation as https://learn.microsoft.com/en-us/azure/virtual-network/virtual-networks-faq#what-protocols-can-i-use-within-vnets states "A possible scenario is configuring DHCP relay from devices on-premises to an Azure VM running a DHCP server. Customer is responsible for configuring on-premise devices (for example, router configuration) to create this DHCP relay traffic to the VM's IP in Azure." which combined with the statement in https://learn.microsoft.com/en-US/troubleshoot/azure/virtual-machines/server-software-support that DHCP isn't a supported role if the IP listening for DHCP requests is an Azure one, that seems to be saying that you're fine running DHCP in Azure, but only if you don't use Windows server do to it. This doesn't really feel like the intended meaning!

@ChrisParrBIE, I think we have to keep the DHCP server on local network for on-premises devices.


Tasks view

Hi, may be add some simple tasks view?
I'm about: https://www.elastic.co/guide/en/elasticsearch/reference/current/tasks.html
I'm do reindex some time with new api and it's hurts to watch several tasks in console =_=''

P.S.
Thanks for you job, i did use kopf before, and now use cerebro, they are awesome.

May I ask if it would be possible to provide the application.conf,
and more specifically, the auth directive either fully commented out or in a line ?
For example, like this:
#auth =
Example of LDAP authentication
#type: ldap
#settings: {
#url = "ldap://host:port"
#base-dn = "ou=active,ou=Employee"
#method  = "simple"
#user-domain = "domain.com"
#}
Example of simple username/password authentication
#type: basic
#settings: {
#username = "admin"
#password = "1234"
#}
#}
or
make it an one-liner ?
The reason for asking this is because seems to be easy to implement, and most importantly,
we can adjust the configuration file using ansible in a reliable way (instead of commenting out a }).

Bumping this issue from 2016. Love Cerebro for its easy usage, but task management is a crucial part missing for us!
Any plans to add this anytime soon?


Add Deleted Index to Daily Operation Manager

Coverage increased (+0.003%) to 65.455% when pulling 844a911ec95123280a98371091b76159ca9c9b07 on jackyrusly:dev into bb5023e0b830c6840f068c8eccdb063ed2ebc932 on danliris:dev.

Coverage increased (+0.05%) to 65.503% when pulling 877e20d567964fdc89207dd498192f7774c16908 on jackyrusly:dev into bb5023e0b830c6840f068c8eccdb063ed2ebc932 on danliris:dev.


click event's interface should be changed to PointerEvent

⚙ Compilation target
esnext
⚙ Library

Missing / Incorrect Definition
In the GlobalEventHandlersEventMap interface the field "click: MouseEvent" should be changed to "click": PointerEvent
Sample Code
-

Documentation Link
https://w3c.github.io/uievents/#event-type-click

According to https://developer.mozilla.org/en-US/docs/Web/API/Element/click_event, this seems to be Chrome only, at least for now.

MDN hasn't been updated yet, but both Webkit and Gecko already support the PointerEvent.


Foundation: tweak address resolution on Windows

Rewrite the host address resolution to be easier to follow.  The old
path obfuscated pointers to the point where the invalid memory access
could not be found (probably due to the incorrect handling with the
unsafe pointer).  This now passes the Host tests.

@swift-ci please test


[WIP] Implementing new react game-play UI.

New UI is still build using React, but more clear and with improved structure.
Creating new components & porting some old components.

Making wireframe UI and logic.
New structure with some clean up.

Gonna close this PR since it's gotten too big.
To keep in spirit with our iterative process, we should be shipping parts of the UI every 2 days tops; it's basic React.
This has been in wip for over a week and balooned into something with no end in sight. Additionally, part of the qualifications of being able to ship PR is that is done fast enough to demonstrate our ability to support it in production, which this PR isn't meeting.
It's cool to use this code in future PRs, but we will not be able to integrate PRs which take this long.


nomsfs: general cleanup

added a README (merged with Mike's)
cleaned up Ref handling
automatic unmounting
signal handling to unmount on exit

@aboodman good to go?

lgtm w/ nits


Add InteractiveViewer Widget to the Widget Catalog

InteractiveViewer Widget needs to be added to the Widget Catalog. Read How to contribute to the Widget Catalog in the README for detailed steps. Make sure to include all the arguments(name, implementation, description, codeString, link and category). Refer to any widget in widgetList.dart and its own *widget*.dart file for better understanding. Showcase all properties of the widget. (Refer to the official docs in case of any doubts)
Please attach screen recording of the same once done in the PR.

pls assign this to me.


How to get the midpoint hit criterion for the MPII?

All I konw is how to get the percision and recall. But I don't know how to get the midpoint hit, and they are different.

hi,
we have used the official evaluation toolkit from the MPII auhtors (matlab) here. you can also have a closer look at their paper to understand it better.
http://human-pose.mpi-inf.mpg.de/#evaluation


Update to Java 17

We currently use Java version 11 for our project, but Java 17 is the latest LTS version; we would like to update it to take advantage of its new features and performance improvements.
Updating to Java 17 will require changes to our build process and potentially some code changes. However, the benefits of upgrading will be worth the effort.
We should also ensure that our project dependencies are compatible with Java 17 before switching.
Benefits:

Improved performance and security
Access to new features and functionality
Staying up to date with the latest technology

@otaviojava, I have a question: once we do up to date this plugin to Java 17 as you're recommending, this plugin cannot be used with maven projects that it's using previous Java versions, am I right?
It would be not a good idea maybe, because according to JRebel Developer Report 2022, companies with over 100 employees are using Java 8 yet, while in companies with under 100 employees are increasing the utilization of Java 11 version and above.
What do you think about it?

Hello ! Giving my contribution to the discussion. Surely Java 17 will be a new pattern between the companies but will take some time until all migrations are done. Many companies use Java 11 (other still are in Java 8 with their legacy apps). But i think we need to embrace those versions in this project.
One suggestion/doubt: Could the "java17 mode" be a flag like "--java17" in the CLI ? At least until most of the companies/users migrate to this version.

@midhatdrops, thanks for your contribution!!!

One suggestion/doubt: Could the "java17 mode" be a flag like "--java17" in the CLI ? At least until most of the companies/users migrate to this version.

IMHO, that suggestion would work if it's a CLI application, but it's not. It is a maven plugin then it depends on the Java version used by the maven execution.
I think that we could be improving the CI process to make sure that this plugin is ready to build with newer Java versions like @otaviojava has proposed.

Hello! My opinion is that to spread the plugin use, it should be compatible with the last LTS version.
New projects, teaching classes are some examples that will not be using older JDKs.

@brunobaiano thanks so much for your opinion!
Yeah! Today, the plugin is built with Java 11... in theory, it should work properly with the latest LTS Java version, that is Java 17.
With that, we'll be free to produce new projects, teach classes, live coding, blog posts, and/or other content using the plugin with the latest LTS Java versions;
I think that we could add new CI runners in order to make sure that the plugin could be compiled with the latest LTS Java versions, at this time, Java 17.

I think it's a great idea to at least ensure that things work as expected with Java 17.

We already have a workflow defined for java 11, so we just need to create a new one with java 17, correct?
Also, we can keep this new workflow running with the same triggers as the java 11 version (for changes in the main branch & pull requests)
What do you guys think?

@viniciusbrg, that's my idea too!
Well, I think we got some options:

Create a new GitHub Action for Java 17

Pros: we'll separate the CI processes for each Java LTS version;
Cons: more files to maintain;

Rename the current one java-11.yml to something like ci.yml for example and make the build job use a matrix strategy with Java LTS versions, it looks like below:
jobs:
  build:
    strategy:
      matrix:
        java: [ 11, 17 ] /# we will just need to put the next new LTS version here

Pros: less files to maintain;
Cons: more complexity to handle with;

Friends, what do you think?

I like two different files; we need to handle two files, but if this increases the CI experience, that is the goal.

Two different files seem better, have more complexity to deal, but you'll have more granularity for each jdk version. For example, if you use some SAST tool or something like that, it does not have compatibility yet to newers LTS versions.

I also like this approach with two different files. Can I work on an implementation for this?
The matrix strategy that Maximilliam suggested would be good if we had many different versions to build, but since that's not the case here, I think two files will work just fine for us.

Friends, I think that we got the point! I'll create another issue for that task:

[ ] Create a new GitHub Action for Java 17

@otaviojava, do you think that we could close this issue when the new related issue gets closed or we could close it by now?

sure


CMD HANDLER WITH HOSTING NOT WORKING.

so basically when I made a discord bot using aoi.js WITH the command handler in the docs. I tried hosting on repl.it 24/7 HACKER plan, it turns on for some time then it turns off after 1 minute.. this happened even when I tried with repl.it + uptime robot same result, even when I tried using a Paid VPS same result.. now I'm pretty sure this is the cause of the command handler as I host my other bot in the repl.it 24/7 hacker plan the same code WITHOUT command handler. IT. WORKED. please fix 🙏
More:

Aoi.js version: ^1.0.6
Node.js version: v12
Operating system: Linux

i don't think this has anything to do with aoijs?


Switch the ADLINK logo to a PNG file.

For some reason Firefox did not want to always render the previous
SVG, so with the blessing of ADLINK switch it to a PNG instead.
Signed-off-by: Chris Lalancette<EMAIL_ADDRESS>

I'm going to go ahead with this one just to get it in and deployed.  There shouldn't be much in here to object to.


Update icons to support Font Awesome 4.3

Whew, this took a while. :stuck_out_tongue_closed_eyes: Let me know if you need any changes, or if there's another part of the codebase I need to update in addition to this.
Closes #1642
Notes:

The Font Awesome attribution/license banner was shortened here: https://github.com/FortAwesome/Font-Awesome/commit/6fbbefc89fa530c55a33bb5277d8b56ef27b3015
signal and wifi are now two separate icons
Added the Gender and Payment Options categories
The i.icon.plane was repeated so I removed the second entry
Font Awesome reference can be found here and here

Thanks, having done all the original icons I know how much busy work this is.
Will have to update the docs as well, will add commits to this PR

Icons btw are sorted by hexcode to help make sure there are no duplicates, and to make it easier to see which icons were added last (highest hexcode values)

Added docs https://github.com/Semantic-Org/Semantic-UI-Docs/commit/5f4a6c76464bab157d44701f5b45c8f8724a2c0c

Available in next

Awesome! :+1:

Icons btw are sorted by hexcode to help make sure there are no duplicates, and to make it easier to see which icons were added last (highest hexcode values)

Ah, gotcha. I'll keep this in mind for the next Font Awesome update.


Problem with setup

Describe the bug
I upload an SVG that I want to optimize and after downloading an extra line appears.
To Reproduce
I upload an SVG that I want to optimize and after downloading an extra line appears.

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Be the same SVG
Screenshots
good one

bad one

Desktop (please complete the following information):

SVGO Version [e.g. 3.0.0]
NodeJs Version [e.g 14.0.4]
OS: [e.g. iOS]

Additional context
Help setings

The two SVGs you uploaded don't seem to represent the "input" and "output" from SVGO, so it's hard to find the problem.

I upload an SVG that I want to optimize and after downloading an extra line appears.

SVGO is a local tool. Without clear steps to reproduce, we also can't be certain if the issues lies with us, or something the online tool is doing. Some online tools are still using an outdated version of SVGO as well.
Can you clarify what online tool you used to optimize SVG?

SVGO is a local

@mariannekita Could you clarify how you optimized the vector, and share the input/output?
If you're unable to, I can close the issue for now, and you're welcome to create another one once you can provide steps to reproduce.


How to: Certificate rotations

How should certs be rotated with YARP without having to stop the service? Do we detect that cert store or files on disk have changed?

Yes both. Also how to associate the right certs with either routes (incoming) or clusters/destinations (outgoing).

Incoming cert config is handled by the server, it's not a YARP concern. Barry is working on rotation guidance there.
For outgoing certs, see https://github.com/microsoft/reverse-proxy/issues/989#issuecomment-839366901.

Duplicate of #989


Update readme

Summary by CodeRabbit

Documentation

Updated the README.md to reflect the latest supported versions for ASP.NET Core and the GraphQL library.
Added detailed descriptions for middleware configuration, authorization, user context, and GraphQL request handling options.
Emphasized CSRF protection and provided configuration examples for CORS and response compression.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 92.766%

Totals

Change from base Build<PHONE_NUMBER>3:
-0.02%

Covered Lines:
2000

Relevant Lines:
2107

💛  - Coveralls


"ports" property missing under main container in new version 2.0.1

Details
This is primarily a question to know ensure that the change below was intentional and hear the reasoning, not necessarily a bug.
I upgraded my Helm chart from using version 1.5.1 to 2.0.1 and compared the rendered result.
Previously I automatically got a ports property under the main container in the rendered result, like this:
    spec:
...
      containers:
        - name: ...
...
          ports:
            - name: http
              containerPort: 8080
              protocol: TCP

... in the new version I have to add it myself manually in my values-file, like this:
controllers:
  main:
    containers:
      main:
...
        ports:
          - containerPort: 8080
            name: http
            protocol: TCP

... otherwise it is not rendered. Was this change on purpose, and what is the reason?
What did you expect to happen:
I expected the new version to render the same as the old version for ports, but I'm no charting expert, so there is probably a reason for the change, that I am not aware of.
Anything else you would like to add:
No.
Additional Information:
No.

You are correct that this is a change that I forgot to put in the release notes. The container ports section relied heavily on some magic templating that was "impossible" to recreate with the deprecation of primary services.
Based on what I could read here, there is actually not much benefit to setting them:
https://matthewpalmer.net/kubernetes-app-developer/articles/kubernetes-ports-targetport-nodeport-service.html

This array, defined in pod.spec.containers[].ports, provides a list of ports that get exposed by the container. You don’t really need to specify this list—even if it’s empty, as long as your containers are listening on the port, they’ll still be available for network access. This just provides some extra information to Kubernetes.

The only use case I've found for explicitly adding container ports has been with custom probes. In the end I've just resorted to configuring them with the port number instead of the named port.

@bjw-s : Thanks for the explanation - this was also my suspicion based on another link I found, that said something similar. Just wanted to make sure this was intentional, and that you agreed 🙂


Get old courseroad to export things properly

All that's left to do for this is to finish mapping the majors over. I'll do it over lunch tomorrow. See https://npfoss.scripts.mit.edu/courseroad/#npfoss for the current experimental version.

done in dannybd/courseroad#36


Create a workload identity credential for DAC auth flow

Related to https://github.com/Azure/azure-sdk-for-net/issues/27159.
Intro:
Azure AD workload identity federation is a new capability. It applies to both application identities and managed identities in Azure AD. The k8s team has built additional components around this that help customers use this capability in their k8s deployments. It's essential for the Azure Identity SDK to support this pattern so customers can easily benefit from this capability.

Allow customers to seamlessly move their code, which uses DefaultAzureCredential, to also run in a k8s cluster.
Do this in a way that isn't confusing to customers. They could either use application identities or managed identities. Telling a customer to pass a managed identity when they're using an application identity is going to be very confusing.
Provide predictability on the credential. For example, a customer may want to exclude ManagedIdentityCredential, so they know the request isn't going through the compute endpoint like IMDS.

Proposal:

Add a new credential type named WorkloadIdentityCredential. This new credential type will become part of the DefaultAzureCredential auth flow.

The WorkloadIdentityCredential will come into play if the necessary environment variables are populated: (AZURE_FEDERATED_TOKEN_FILE, AZURE_CLIENT_ID, AZURE_TENANT_ID). These are set up by the k8s platform based on the assertions on a service account.

Allow an optional client ID parameter to be passed into DefaultAzureCredential, which can be used by the customers to override the AZURE_CLIENT_ID in the event they have more than one identity for which they intend to use this flow.

With these changes, customers can do the following:

Use DefaultAzureCredential as their main guidance to their developers.
Recommend excluding the necessary credential types to get predictability in their code execution.
Pass explicit client IDs for which they want to get an Azure AD token.

Ideally with these changes, we'd also remove the current implementation where ManagedIdentityCredential is overloaded to work with these env. Doing this is ideal to avoid the ensuing confusion for customers.

Fixed in https://github.com/Azure/azure-sdk-for-net/pull/34280


TypeError: iteration over a 0-d tensor

when training the model this error occurred in the first iteration
Epoch: 01 learning rate[0.0001]
/usr/local/lib/python3.7/dist-packages/torch/nn/functional.py:718: UserWarning: Named tensors and all their associated APIs are an experimental feature and subject to change. Please do not use them for anything important until they are released as stable. (Triggered internally at  /pytorch/c10/core/TensorImpl.h:1156.)
  return torch.max_pool2d(input, kernel_size, stride, padding, dilation, ceil_mode)
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-16-6360cfd97324> in <module>()
     15     start_time = time.time()
     16
---> 17     train_loss,outputs = train(model,  criterion, optimizer, scheduler, train_loader)
     18     valid_loss = evaluate(model, criterion, val_loader)
     19

/usr/local/lib/python3.7/dist-packages/torch/_tensor.py in __iter__(self)
    605         # See gh-54457
    606         if self.dim() == 0:
--> 607             raise TypeError('iteration over a 0-d tensor')
    608         if torch._C._get_tracing_state():
    609             warnings.warn('Iterating over a tensor might cause the trace to be incorrect. '

TypeError: iteration over a 0-d tensor

The issue is maybe with your data.

For me, it's working fine. I have also updated the requirement.txt file. Can you check now?

It is working fine for me. I have also updated the requirement.txt, can you check now?

same thing I even checked if the data loaders are empty and they are not

Replace train function call
train_loss,outputs = train(model,  criterion, optimizer, scheduler, train_loader)
with
train_loss = train(model,  criterion, optimizer, scheduler, train_loader)
As train function returns only single value, not typle.

I checked it is correct in the Google colab but this error exists in Notebook. I'll change it.
Thanks for pointing out.


Example use regarding how to read and save images from E57 files

Hi,
I am adding this to read/save panoramic images from E57 files.
Also, I think it could be convenient to create a "example.cpp" that shows how to use the class/functions.
Let me know if you have any questions.

Thank you @bigeyesung.
Some initial things to fix:

The PR is not passing CI. When you set up the build for the example executable in cmake, you need to make it use C++11.
I would prefer putting the example (and its cmake build instructions) in a new folder - examples - and changing the name: examples/src/ImageExample.cpp or something. That would make it easier to add other examples in the future.
The example should not be built by default. If the developer is including E57Format as a submodule it should not be built. If it is being built on its own, it should be an option E57_BUILD_EXAMPLES (default to false).
You need to run clang-format on the code for proper formatting (see README -> Contributing).

Thank you @bigeyesung.
Some initial things to fix:

The PR is not passing CI. When you set up the build for the example executable in cmake, you need to make it use C++11.
I would prefer putting the example (and its cmake build instructions) in a new folder - examples - and changing the name: examples/src/ImageExample.cpp or something. That would make it easier to add other examples in the future.
The example should not be built by default. If the developer is including E57Format as a submodule it should not be built. If it is being built on its own, it should be an option E57_BUILD_EXAMPLES (default to false).
You need to run clang-format on the code for proper formatting (see README -> Contributing).

@asmaloney, thanks for the feedback, I am working on it.
I have one question in terms of #4 "clang-format verification". I am trying to make my "ReadImages.cpp" to run it automatically, not sure how to do it. I traced code to cmake/Modules/ClangFormat.cmake,
if ( E57_CLANG_FORMAT_EXE )        get_target_property( e57_sources ${PROJECT_NAME} SOURCES )
Currently it checks include and src folders. Not sure how to make it to scan other folders (E.g. examples dir for this case).
I tried using filters and checked cmake doc. Any hints is appreciated. thx
  list( FILTER e57_sources INCLUDE REGEX ".*/examples/.*" )
https://cmake.org/cmake/help/latest/manual/cmake-properties.7.html#target-properties

I have one question in terms of #4 "clang-format verification"

So everything is done in terms of targets to keep as much as possible out of the global scope. In the clang-format file, I'm looking at the E57Format project, grabbing the sources files, filtering some out, and running clang-format on the list of files.
Since this file is included after your new example, you should be able to mimic this using your target.
So before add_custom_target, try grabbing the source files from your target using get_target_property with your target name(ReadImages), then adding them to the e57_sources list. (All wrapped in if(E57_BUILD_EXAMPLES) of course.)

Seems to have been abandoned. If not, please split into two PRs - one for the change to the lib, one for the example - and apply all the changes above (including formatting).


ML: VGG16 Model #1 (24 Layers): Configuration#3: Training model and store the results in the directory for 30 and 50 epochs

Model Architect:

4Conv(Conv+Conv+MaxPool+Dropout)
4Dense(Dense+Dropout)

Put the training on the Grid by using jetscape-ml-tensorflow-cnn.sh
save the model and the evaluation data in the power point slides
update the diagrams data on the power point slides

30 epochs
839936/840000 [============================>.] - ETA: 0s - loss: 4.5717 - accuracy: 0.9317
Epoch 00012: val_accuracy did not improve from 0.84683

840000/840000 [==============================] - 308s 367us/sample - loss: 4.5717 - accuracy: 0.9317 - val_loss: 4.5195 - val_accuracy: 0.8195
Epoch 00012: early stopping

50 epochs
Epoch 31/50
839936/840000 [============================>.] - ETA: 0s - loss: 0.6477 - accuracy: 0.9345
Epoch 00031: val_accuracy did not improve from 0.94070
840000/840000 [==============================] - 316s 376us/sample - loss: 0.6477 - accuracy: 0.9345 - val_loss: 0.6503 - val_accuracy: 0.9238
Epoch 00031: early stopping

30 epochs
Epoch 12/50
839936/840000 [============================>.] - ETA: 0s - loss: 4.5717 - accuracy: 0.9317
Epoch 00012: val_accuracy did not improve from 0.84683
840000/840000 [==============================] - 308s 367us/sample - loss: 4.5717 - accuracy: 0.9317 - val_loss: 4.5195 - val_accuracy: 0.8195
Epoch 00012: early stopping

port to excel for overleaf


multistore_file raises errno.EDEADLK when trying to open credential store

multistore_file claims to be process and thread safe, but if multiple threads try to refresh credentials simultaneously, it can result in an exception like the stack trace at the bottom of this issue.
multistore_file catches errno.ENOSYS and errno.ENOLCK already.  The simple fix would be to add EDEADLK to this list, since multiple threads opening the file can definitely result in this error.  Additionally, it might be useful to add an actual Python lock to protect this access, which would help a lot when threads (or, optionally, child processes) contend for this resource on startup.
Encountered exception while copying:
Traceback (most recent call last):
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/gslib/command.py", line 1707, in PerformTask
    results = task.func(cls, task.args, thread_state=self.thread_gsutil_api)
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/gslib/copy_helper.py", line 1796, in _PerformSlicedDownloadObjectToFile
    end_byte=args.end_byte))
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/gslib/copy_helper.py", line 2185, in _DownloadObjectToFileResumable
    digesters=digesters, progress_callback=progress_callback)
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/gslib/cloud_api_delegator.py", line 225, in GetObjectMedia
    serialization_data=serialization_data, digesters=digesters)
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/gslib/gcs_json_api.py", line 694, in GetObjectMedia
    serialization_data=serialization_data)
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/gslib/gcs_json_api.py", line 716, in _PerformResumableDownload
    end_byte=end_byte, serialization_data=serialization_data)
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/gslib/gcs_json_api.py", line 772, in _PerformDownload
    use_chunks=False)
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/third_party/apitools/apitools/base/py/transfer.py", line 459, in GetRange
    additional_headers=additional_headers)
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/third_party/apitools/apitools/base/py/transfer.py", line 396, in __GetChunk
    retries=self.num_retries)
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/third_party/apitools/apitools/base/py/http_wrapper.py", line 325, in MakeRequest
    http, http_request, e, retry, max_retry_wait))
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/third_party/apitools/apitools/base/py/http_wrapper.py", line 316, in MakeRequest
    check_response_func=check_response_func)
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/third_party/apitools/apitools/base/py/http_wrapper.py", line 365, in _MakeRequestNoRetry
    redirections=redirections, connection_type=connection_type)
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/gslib/gcs_json_media.py", line 377, in NewRequest
    connection_type=connection_type)
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/third_party/oauth2client/oauth2client/util.py", line 137, in positional_wrapper
    return wrapped(*args, **kwargs)
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/third_party/oauth2client/oauth2client/client.py", line 541, in new_request
    self._refresh(request_orig)
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/third_party/apitools/apitools/base/py/credentials_lib.py", line 308, in _refresh
    oauth2client.client.OAuth2Credentials._refresh(self, do_request)
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/third_party/oauth2client/oauth2client/client.py", line 771, in _refresh
    self.store.acquire_lock()
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/third_party/oauth2client/oauth2client/multistore_file.py", line 224, in acquire_lock
    self._multistore._lock()
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/third_party/oauth2client/oauth2client/multistore_file.py", line 284, in _lock
    self._file.open_and_lock()
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/third_party/oauth2client/oauth2client/locked_file.py", line 374, in open_and_lock
    self._opener.open_and_lock(timeout, delay)
  File "/usr/local/share/google/google-cloud-sdk/platform/gsutil/third_party/oauth2client/oauth2client/locked_file.py", line 209, in open_and_lock
    fcntl.lockf(self._fh.fileno(), fcntl.LOCK_EX)
IOError: [Errno 35] Resource deadlock avoided

Thanks for the report, we would definitely be receptive to an implementation of your suggested fixes, and today I learned that Python's multiprocessing module contains a Lock class. :-)

The file-locking notion is still useful for dealing with multiple applications executing simultaneously.
Using a multiprocessing lock is feasible, though it would probably need to come from the calling application.  A thread lock would be easy, though.  This pattern worked well elsewhere in gsutil:
multistore_lock = threading.CreateLock()
def UseProcessSafeLock(multiprocessing_lock):
global multistore_lock
multistore_lock = multiprocessing_lock


CUDA out of memory

Hi @danielism97 I have RTX2060 6gb vram but it needs more vram to execute. How can I use small length sequences to get inference on lower memory gpu ?

Hi Muhammad,
The model does require quite a bit of memory. For example, to interpolate at a resolution of 960x540, around 9GB is needed on a GPU. This is indeed a limitation of the model.
However, the interpolate_yuv.py script provides options for doing block-wise interpolation, i.e. the frames can be divided into small blocks for inference, and the interpolated blocks are then aggregated to form a single frame. Here is an example:
python interpolate_yuv.py \
--net STMFNet \
--checkpoint <path to pre-trained model (.pth file)> \
--yuv_path <path to input YUV file> \
--size <spatial size of input YUV file, e.g. 1920x1080> \
--out_fps <output FPS, e.g. 60> \
--out_dir <desired output dir> \
--patch_size 256 \
--overlap 3 \
--batch_size 4

The last three arguments specify how the block-wise evaluation is performed. You can experiment with these figures until it can run on your GPU. However, please note that doing interpolation this way can produce blocking artefacts near the block edges.
Thanks.

I tried using those last 3 arguments and I keep getting the exact same error unfortunately.
"RuntimeError: CUDA out of memory. Tried to allocate 1.05 GiB (GPU 0; 11.00 GiB total capacity; 8.26 GiB already allocated; 0 bytes free; 8.96 GiB reserved in total by PyTorch)"

Hi,
Apologies! I didn't enable it properly previously. I just updated the code. Now it should work.
Thanks.

Should be noted that an overlap of 3 throws a fatal error. I raised it to 4 and it seems to be working..


feature: rust https://github.com/rustformers backend

Is your feature request related to a problem? Please describe.
A rust backend can be a robust alternative, and rustformers supports also other models
Describe the solution you'd like
Have a rust backend to include support to models supported by rustformers
Describe alternatives you've considered

Additional context
For reference:
https://github.com/huggingface/candle
https://github.com/hyperium/tonic
https://capnproto.org/
@lu-zero promised to give some help here =)

Hi, guys. I am interested in this one. May I ask you guys the processing of this? I remember that we have an issue mentioned we need to have an example of building an external backend for localAI. I believe if we finish the example first and it would be more easy for us to do this. We need a template of external backend service.
@mudler May you please split the issue to a set of issues? We can pick up one by one. And it can also make sure we can do everything in the way that we want to.

we have https://github.com/go-skynet/LocalAI/issues/1057 already to track documentation on how to create LocalAI backends

Here is the equivalent tutorial but using tonic.

Noticed and thanks.

Here is an idea of the choosing of the default backend of rust #1219


Fix Issue with avahi callback

Store elements of Added Services in order to make a clean remove later on possible. Closes #8

Thanks @Kr0llx for the PR and the contribution. This looks great :)
Would it be possible to have a look at the failing tests and adjust them to the changed functions? Sadly it is not possible to mock the Avahi package, so testing is a bit limited.

Yes. I updated the test accordingly.

coverage: 90.154% (-0.7%) from 90.816%
when pulling 4b9de8e85cad447cd62b0ea5d0ff4569cc570b72 on Kr0llx:bugfix-mdns-avahi-callback-8
into 073f3ca03af0e9e15d41a03f99127ac79a4c9769 on enbility:dev.


Fixed-point scalar types

@jschall once proposed to add those to the set of native DSDL types:

Can we use a floating point standard that matches IEEE 754 for 16, 32, 64 and 128-bit floats, but also provides for arbitrary bit lengths of floating point fields? I've got a spot where I'd like to use a 24-bit float, but there isn't really any support for it.
Actually what I really want to use here is fixed point Q16.8 - because I want uniform precision for this field. This keeps coming up, again and again and again, and I'd really like built-in support for it.
I can't make it a 32-bit float without spilling into another CAN frame

To which I said:

while I do like the idea in general, it seems like too much work right now, and besides there are questions on how to represent fixed-point numbers in generated DSDL classes. In Python there is support for fixed-point arithmetics in the standard library; but what about libuavcan. We'll have to either cast fixed-point numbers to floats, which pretty much defeats the purpose, or we'll need to code up custom fixed-point arithmetic classes (I know that many modern MCU support fixed-point in hardware, but the solution has to be generic). Overall I'm not sure if this feature is worth the extra complexity, at least right now.

This issue is to attract relevant discussion and feedback.
In this thread over there there was also a related discussion on how to represent the elements of rotation quaternions in the most efficient and usable way; fixed-point types could be applied there greatly: https://github.com/UAVCAN/dsdl/issues/34

It would also be possible (and quite low effort) to do some experimentation with this without protocol support if we created DSDL definitions similar to the following
# Q16_8
uint16 integer_bits
uint8 fractional_bits

If it was found out that having these as native types would improve things a lot we could then upgrade them to first class citizens of dsdl and deprecate the experimentation types. This would leave the problem of using the fixed point types up to do application writer until native support was implemented.


A little Review/Questions/Suggestion etc

What I really liked about zuix:

I love this!!
I don't know what is wrong with everyone else, why do i need npm, and a ton load of npm commands and a gazillion dependencies just to run your javascript library.
Please keep it this way.
I love the following:

but can I have pages without .html for example /api/v1/generateContent/100/200 ?

is this possible:

and then use something like an onclick
onClickHandler(){
  someVariable = "/api/v1/etc/etc";
}

I imagine with the help of lazy loading, it would be really easy to create a wiki or documentation without using crazy things like static site generators. This would be a killer static website making framework!! If it is already possible, may be some example would be nice. I know about the Hacker News thing. But would love a much more bare bones stripped down version.
So far, I liked the framework because it does makes things simpler!!! Thanks for this!!

but can I have pages without .html for example /api/v1/generateContent/100/200 ?

Yes by using the cext parameter in the loading options. This parameter allow you set a custom extension that in your case will be just empty string ''.
Full options list is described here: https://genielabs.github.io/zuix/#/api#ZUIX_API--ContextOptions .
Example
var my_options = {
    cext: ''
}

And yes you can definitely create static sites without need of going through a server side generation process nor by using external tools.
As an example please have a look at this pure client-side javascript content generation supporting markdown as well (i.e. like Jekyll):
https://github.com/genielabs/HomeGenie/tree/gh-pages
which is published at:
https://genielabs.github.io/HomeGenie/
Perhaps I'll publish a boiler plate for such kind of sites. I've just got too much stuff in my head right now :D that I don't know what do do first. I wish more people would get involved in this project and contribute.
Thank you for liking this. :)

I'm not sure whether I am posting in the right section. I liked Zuix.js from the initial look. I am not a professional developer though I would love the idea of having single file creating superb Modular application. I'm not sure whether it is my browser problem, but Github docs page didn't load in Chrome and Safari on my iPad as well as iPhone. I was trying to just go through the API to see what is posssible with Zuix.
Additionally, I would like this to be one of the popular frameworks. Probably you need little time and effort to market this one. Just my view!
Love it!

So it's been a while and there have been lot of progress.
Have a look to the new website
https://genielabs.github.io/zuix

That's some nice work!!!


Add ":starterscript" command

The ":cs" command is useful for a variety of reasons. If I need to display a dependent message during an in-game event, I can use the command to do this easily. It is restricted in the way that I need to rerun this command each time a player joins, and I've had to create some workarounds due to this limitation. While large scripts are not usually ran with :cs, I believe adding a :starterscript command would make things much easier.

We should probably also implement a mechanism to keep track of starterscripts added using the :starterscript command, plus a way to remove those starterscripts using another command.
I'll work on this in a separate PR.


#853 Adding Inheritance in CPP

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes #853

Type of change 📝

[x] Documentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] C/CPP

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

@ShikhaSwati60 Where is the Changed file...
Make sure you make PR from the same brancg where you made changes


[Bug]: theme translations lost when pushing theme with Shopify GitHub integration

Please confirm that you have:

[X] Searched existing issues to see if your issue is a duplicate. (If you’ve found a duplicate issue, feel free to add additional information in a comment on it.)
[X] Reproduced the issue in the latest CLI version.

In which of these areas are you experiencing a problem?
Theme
Expected behavior
Theme translations should be preserved.
Actual behavior
Theme translations are lost. It seems translations and Shopify github integration can't coexist right now. There was a similar issue: https://github.com/Shopify/cli/issues/1562 which seems to show we can manually duplicate a translated theme in Shopify backoffice to preserve the "translation link", and then push changes to that duplicated theme (works, but still not ideal and confusing).
But this is incompatible with github workflow which uploads a new theme every time we may need to add a new branch. I find this very problematic and haven't found any workaround.
Verbose output
NA

Reproduction steps

Translate theme (such as templates) in Shopify Translate & Adapt
shopify theme pull to work locally
Publish the local theme to a git repository
Add theme to store with "Connect from GitHub"

Operating System
Windows 11
Shopify CLI version (check your project's package.json if you're not sure)
3.49.4
Shell
No response
Node version (run node -v if you're not sure)
No response
What language and version are you using in your application?
No response

Same issue here! Really hope this gets fixed soon!

Still relevant

Same, still relevant!
On Thu, Feb 15, 2024 at 10:42:43, MaxDesign < @.*** > wrote:

Still relevant
—
Reply to this email directly, view it on GitHub (
https://github.com/Shopify/cli/issues/3165#issuecomment-1945710413 ) , or unsubscribe
(
https://github.com/notifications/unsubscribe-auth/ARQXFIMMXAMFVZ2Q5Z2X34DYTXKBHAVCNFSM6AAAAABAFYGUGGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBVG4YTANBRGM
).
You are receiving this because you commented. Message ID: <Shopify/cli/issues/3165/1945710413
@ github. com>

Any updates on this issue?
Driving me crazy (and clients)

Yes, still relevant!!

👋 the team is aware of this issue. in the meantime they shared this workaround to hopefully ease the friction of copying translations via CSV https://community.shopify.com/c/shopify-translate-adapt/why-did-my-theme-update-erase-most-translations/m-p/2016134#M628

@mgmanzella This is still a ongoing issue that creates significant challenges for live stores. Closing a critical issue for "CLI housekeeping" is doesn't make it go away.


Mapmod does not work

I can not get this map mod to work.
https://steamcommunity.com/sharedfiles/filedetails/?id=1544515871
What do I do wrong?
$ arkmanager status @dm
Running command 'status' for instance 'dm'
 Server running:   Yes
 Server PID:   6371
 Server listening:   No
 Server build ID:   5302515
 Server version:   312.52

The Server name is shown and Players 0/0 but no map in the steam Servers Window (Favourites).
This is my config file:
# config environment
arkserverroot="/home/user/ark-deathmatch"                                     # path of your ARK server files (default ~/ARK)

# ARK server options - use ark_<optionname>=<value>
# comment out these values if you want to define them
# inside your GameUserSettings.ini file
serverMap="Deathmatch"                                           # server map (default TheIsland)
#serverMapModId="1544515871"                                               # Uncomment this to specify the Map Mod Id (<fileid> in http://steamcommunity.com/sharedfiles/filedetails/?id=<fileid>)
#ark_TotalConversionMod="496735411"                                 # Uncomment this to specify a total-conversion mod
ark_RCONEnabled="True"                                              # Enable RCON Protocol
ark_RCONPort="32332"                                                # RCON Port
ark_SessionName="MeinsDeathmatch"                                  # if your session name needs special characters please use the .ini instead
ark_Port="7778"                                                     # ARK server port (default 7778)
ark_QueryPort="27015"                                               # ARK query port (default 27015)
#ark_ServerPassword=""                                               # ARK server password, empty: no password required to login
ark_ServerAdminPassword="------"                               # ARK server admin password, KEEP IT SAFE!
ark_MaxPlayers="10"
#ark_GameModIds="1544515871"
#ark_GameModIds="487516323,487516324,487516325"                     # Uncomment to specify additional mods by Mod Id separated by commas
#ark_AltSaveDirectoryName="ScorchedEarth"                           # Uncomment to specify a different save directory name

# ARK server flags - use arkflag_<optionname>=true
#arkflag_OnlyAdminRejoinAsSpectator=true                            # Uncomment to only allow admins to rejoin as spectator
#arkflag_DisableDeathSpectator=true                                 # Uncomment to disable players from becoming spectators when they die

# ARK server options - i.e. for -optname=val, use arkopt_optname=val
#arkopt_StructureDestructionTag=DestroySwampSnowStructures

#arkPreStart="/etc/arkmanager/instances/instance.start"              # Set this to override the start script; defaults to config file name with .cfg replaced by .start
#arkmod_821530042=game
#arkmod_1404697612=game
#arkmod_1544515871=game

Ok I got it.
 Thats the Name of the Map.
And with that the server should start.


Support<EMAIL_ADDRESS>

Design

Angular 2s component interaction is now centered around ViewContainerRef which are logical units of view's, angular2-modal followed.
A default ViewContainerRef is needed for angular2-modal. This is done once (in 99%) and from there on you can forget about ViewContainerRef.
This is 1 new code line, the other option was to force a mandatory element in the root template.
The way dependency injection is used is now very important.
Since a Modal instance is paired with a default view container, every new instance of the modal requires setting a new default view container.
This can be handy for advanced scenarios but for most cases setting the default view container once is what you should do.
If you set it once from in the top level component don’t request the Modal in the providers property of a component meta, if you do so you will get a new instance of Modal.
This is how a setting it once looks:

    @Component({
        selector: 'app',
        providers: [ ...MODAL_PROVIDERS], // list of angular2-modal tokens, Modal included.
        template: `
        <main>
          <router-outlet></router-outlet>
        </main>
      `
    })
    export class App {
        constructor(public modal: Modal, viewContainer: ViewContainerRef) {
            /**
             * A Default view container ref, usually the app root container ref.
             * Has to be set manually until we can find a way to get it automatically.
             */
            modal.defaultViewContainer = viewContainer;
        }
    }

From here on, there's no need to require a new provider for Modal
So don't do this:
        @Component({
            selector: 'myComponent',
            providers: [Modal], // results in a brand new instance of Modal
            template: `<h1>My Component></h1>`
        })
        export class MyComponent {}

Generic type changes from<EMAIL_ADDRESS>
Every ResolveProvider is now ResolvedReflectiveProviders
Every Injector is now ReflectiveInjector

Class changes:
BootstrapModalContainer:

Change selector from bootstrap-modal to modal-container

Modal:

open() and openInside() now get ViewContainerRef instead of ElementRef
openInside() anchor parameter was removed
Added property defaultViewContainer, to be set on initial root app element load.

ModalDialogInstance:

Change method dispose() to destroy()
Does not store a reference to Backdrop and BootstrapContainer ComponentRef
Destroy is now set by the Modal service, DialogRef starts with a noop destroy functionality

ModalAwarePreset:

open method now get 1 optional parameter ViewContainerRef instead of the inside object.
If supplied it modal will render inside that view, otherwise use the defaultViewContainer

Demo:
Sample element is now modal agnostic, use parent component to get ViewContainerRef

Closes #66


How make an HTTP request with curl or http when username and password was set?

Hello, I'm doing some experiment with Dockerfile
and setting HTTP_USERNAME and HTTP_PASSWORD
but I will fail when I call at normally (like docs)
vctqs1@vctqs1:~/Documents/manabie/backend/deployments/import-map-deployers$ http :5000/enviroments
HTTP/1.1 401 Unauthorized
Connection: keep-alive
Content-Length: 0
Date: Wed, 09 Feb 2022 10:52:03 GMT
Keep-Alive: timeout=5
WWW-Authenticate: Basic realm="sofe-deplanifester"
X-Powered-By: Express

sorry I get confused -a username:password will help
http -a username:password :5000/environments


Add additional infrastructure

Resolves #42
Resolves #39

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

Project coverage is 100.00%. Comparing base (02fe5b5) to head (8292ed7).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #43   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9         9
  Lines          521       521
=========================================
  Hits           521       521

Flag
Coverage Δ

macos-latest-py-3.10
100.00% <ø> (ø)

macos-latest-py-3.11
100.00% <ø> (ø)

macos-latest-py-3.12
100.00% <ø> (ø)

macos-latest-py-3.8
100.00% <ø> (ø)

macos-latest-py-3.9
100.00% <ø> (ø)

ubuntu-latest-py-3.10
100.00% <ø> (ø)

ubuntu-latest-py-3.11
100.00% <ø> (ø)

ubuntu-latest-py-3.12
100.00% <ø> (ø)

ubuntu-latest-py-3.8
100.00% <ø> (ø)

ubuntu-latest-py-3.9
100.00% <ø> (ø)

windows-latest-py-3.10
100.00% <ø> (ø)

windows-latest-py-3.11
100.00% <ø> (ø)

windows-latest-py-3.12
100.00% <ø> (ø)

windows-latest-py-3.8
100.00% <ø> (ø)

windows-latest-py-3.9
100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Any potential GA date (or roadmap) on Azure HDInsight 5.1?

Hi @tomqwu , We are working relentlessly to get the GA for HDInsight 5.1, and we should be doing this within the next 2 months, Please do reach out to our support for any further questions. Happy to hear from you and we look forward to continue engaging with you on https://learn.microsoft.com/en-us/answers/tags/168/azure-hdinsight and also please post your ideas on https://feedback.azure.com/d365community/forum/14770d6d-7626-ec11-b6e6-000d3a4f032c


Prevent workflows from eating up GitHub openverse-bot user's API requests

Problem

We have a lot of workflows that run on every push and some on every issue and PR that opens. Several of these could be limited to run in series or cancel in progress runs.
Description

Use the concurrency setting for workflows to manage runs that overlap: https://docs.github.com/en/actions/using-jobs/using-concurrency
Note that not all workflows will need the same configuration so do take care to investigate each one and ensure you understand when and why it runs to be able to develop the correct group name for them.
Implementation

[ ] 🙋 I would be interested in implementing this feature.

After #1668, we have a good concurrency configuration for the workflows and I have not seen any rate-limiting affecting our workflows.


inspectors throws an error but is not handled.

Added error handling on inspector and passed it to the gulp through stream.

@alexeyraspopov @Semigradsky I am using your plugin in my gulp task but keep getting an unhandled error event from jsinspect.

Thanks! New version ready.


Added unit tests for Common.Core

Restructured folders and solution for integration and unit tests
Added unit tests for Common.Core
Updated documentation with link to online code coverage report
InterfaceConverter is excluded from unit tests and code coverage
Initial line coverage is 91% and branch coverage is 75%

Refactoring:

Removed unnecessary default constructors for Address, Company, ISOCountry, Person, Postcode, Province, User and ResultsSet
Added an optional maxDepth parameter to the JsonHelper.ReadAppSettings method with a default value of 2.

Dependences:

Xunit
FluentAssertions
Xunit.SkippableFact

This closes #20

Wrong issue number mentioned in description.
Should be closes #30


Warp in NetworkedNavMeshAgent

Not sure you want Warp for correction on a low frequency...probably should treat that as a drift correction while it's navigating, and if it's stationary within a short distance, leave it alone, like arrival distance where it considers itself having arrived when it's close to the destination.

Good point that if it's stationary it shouldn't be corrected. Just left alone.
As for the Warp, you are right. Technically we would always be off by a tiny amount during corrections. That will cause it to teleport which won't be smooth.
Suggestions of improvements? Interpolate towards the corrected target if it's a certain distance off maybe?

Interpolate toward calculated next position of agent, perhaps, so that if client's agent is running roughly parallel to server's agent it will nudge over toward server's agent's path over time.

That will be a bit difficult tho. Considering we don't know of the position. We get a state update containing the position and destination. Then it's all client sided movement. If per say we drift away.  Now we know where we are on the server. But we can't really interpolate unless we send correction very often at which point we have created a clunkier version of the NetworkedTransform.
To me the solution would be to continue with Warp but only if the drift is severe enough.

We can't get server agent's next positon and send that to client to determine variance?

https://github.com/TwoTenPvP/MLAPI/commit/c46f2684708cb68b68dd56b0f3af514acac82f76
A full warp still happens during the "StateUpdates" I.E everytime we switch destination.

Oh whoops - can't do a full warp when changing destinations because there are such things as moving targets.  Repeatedly setting a new destination to the target's new location is how chase works.  You have to leave that in our hands as to how we decide to handle the slop based on how our game works.  Document that it can create slop and point us to the adjustable parameters to tighten or loosen as we see fit.

https://github.com/TwoTenPvP/MLAPI/commit/5d8c1ece0b65479cf13bee1778fe92217982f230


Exported model does not comply with the JSON scheme

Describe the bug:
Exporting the model may lead to vertices in the JSON model that contain decimals, when importing the model it expects an integer and will lead to an error message.
Expected behaviour:
Either the vertices must be exported as integer (no decimals), or the vertices must change of type in the schema so it is able to import vertices as a number.
Environment:

Version: 2.2.0
Platform: Web App and Desktop App
OS: Only tested on Windows, but looking at the code, it should apply to all: MacOS / Windows / Linux
Browser: Chrome, Firefox, Safari

To Reproduce:
One of my co-workers managed to export a model that had decimals in the vertices values using the same setup as me (Desktop App, Windows machine, version 2.2.0). It is still unknown what action led to these decimals, I have tried many things but have not found how to reproduce it.
Any additional context, screenshots, etc:
The model contains vertices values where the 'x' or 'y' contains a decimal:
"vertices": [
    {
        "x": 739.5,
        "y": 640
    }
]

The error message after importing the model:

Thanks for finding this @mi0x , and it is a bad thing to have happen
It is just a warning, if I understand correctly, so can be ignored - but it is still a high priority blocker bug
I agree that either the schema should be changed or the save functionality changed, not sure which
I think it can be repeated by turning the snap grid off when editing the diagram


Consider syntactic variants of multiple subclass assertions in gist style guide

The following are syntactically/stylistically different, semantically identical variants:
<owl:Class rdf:about="&gist;PhysicalIdentifiableItem">
        <rdfs:subClassOf>
                <owl:Restriction>
                        <owl:onProperty rdf:resource="&gist;hasMagnitude"/>
                        <owl:someValuesFrom rdf:resource="&gist;Volume"/>
                </owl:Restriction>
        </rdfs:subClassOf>
        <rdfs:subClassOf>
                <owl:Restriction>
                        <owl:onProperty rdf:resource="&gist;hasMagnitude"/>
                        <owl:someValuesFrom rdf:resource="&gist;Weight"/>
                </owl:Restriction>
        </rdfs:subClassOf>
<owl:Class>

<owl:Class rdf:about="&gist;PhysicalIdentifiableItem">
        <rdfs:subClassOf>
                <owl:Class>
                        <owl:intersectionOf rdf:parseType="Collection">
                                <owl:Restriction>
                                        <owl:onProperty rdf:resource="&gist;hasMagnitude"/>
                                        <owl:someValuesFrom rdf:resource="&gist;Volume"/>
                                </owl:Restriction>
                                <owl:Restriction>
                                        <owl:onProperty rdf:resource="&gist;hasMagnitude"/>
                                        <owl:someValuesFrom rdf:resource="&gist;Weight"/>
                                </owl:Restriction>
                        </owl:intersectionOf>
                </owl:Class>
        </rdfs:subClassOf>
</owl:Class>

Adopt one of these styles and document in style guide.
(I personally prefer the second variant, but consensus is required.)

This may be a tool thing. Hardly anybody will type in such axioms. They will use a tool (e.g. PRotege) which will decide how to write the triples. A different tool may make a different choice. Two version of 100% correct OWL. Many people creating ontologies will never need to look at nor know about such differences.
Does the same difference also show up in Turtle syntax? I'm guessing yes.
There is a potential argument for post processor to make sure the same conventions is used across the board.  Ideally the rdf serializer would do that, but my guess is that it does not.  Rather it will faithfully represent triples in one style or the other, depending on what the input is.

@rjyounes
We are now using Turtle, so if we still care about this distinction, the two styles should be redone.  With luck, the serializer will render them both the same and this becomes a non-issue, so to speak.
My guess is the serializer will not have them be the same. This is something that is so minor that I don't think we should have a preferred style. They both mean exactly the same thing.


ESP32-H2 support

Add esp32h2 support to the Direct Boot example.
To build an example:

For esp32c3: cmake -B build -D target=esp32c3 -G Ninja .
For esp32h2: cmake -B build -D target=esp32h3 -G Ninja .
cmake --build build

Ah no, too early. There are still problem with hello_world example.
For some reason, the cpu restarting because of the RTC WDT Reset.

Now everything works.
LED blinks, Hello, world! appears on the UART0 (TX pin) of the H2 board.
I have not squashed the commits (for easier review process). If I need to squash them before merge, please tell me and I will do that without any problem.
Thanks.

@igrr Thank you so much for the review! :pray:
I have fixed all the nitpicks.
If you have anything that in your opinion I can make better feel free to share.

@roma-jam I forgot one thing in the previous review, please also take a look and update the CI workflow:

https://github.com/espressif/esp32c3-direct-boot-example/blob/main/.github/workflows/build.yml
https://github.com/espressif/esp32c3-direct-boot-example/actions/runs/6253664164/job/17002678017?pr=13

@igrr No problem, I think that I can fix that.
But it seems that I am not able to trigger Actions, so I have changed the build.yml file and hope that it will work without any problem. Also, I have checked YML syntax with some online verification tool but I have not installed anything to verify it locally.
Probably, it is better do install any tool to be able to run build actions locally, isn't it?

@roma-jam Looks good to me to be merged, could you please clean up the history?

@roma-jam Looks good to me to be merged, could you please clean up the history?

@igrr ,
Sure, no problem!

@igrr
Thanks for giving me the chance to help in such an interesting project.
Is there anything else you want to check/verify? Or I can merge pull request?
Thanks :pray:

Yes, please go ahea


APT whitelist request for libiberty-dev

How can I move this forward? libiberty-dev is required for kcov's --verify flag. The kcov in apt repositories doesn't work for my need.

+1