Cannot Train with Depth Maps without Chamfer Distance

Hi, thanks for your released code. I'm  interested in the your methods that training with depth map. So I design a tool problem where I only train with depth map but I find points become NaN. Do you have any suggestions? Does this mean the depth maps cannot help training?
from p2i_utils import ComputeDepthMaps
compute_depth_maps = ComputeDepthMaps(projection="perspective", eyepos_scale=1.0, image_size=224).float()

def get_depth_render(points, requires_grad=True):
    depth_list = []
    for view_id in range(8):
        _depth = compute_depth_maps(points, view_id=view_id)
        depth_list.append(_depth)
    if requires_grad:
        return torch.cat(depth_list, dim=1)
    else:
        return torch.cat(depth_list, dim=1).detach()

trg_points = load_from_file()
input_points = torch.full([1, 2500, 3], 0.0, device=device, requires_grad=True)
optimizer = torch.optim.SGD([input_points ], lr=1.0, momentum=0.9)
Niter = 2000
loss_list = []

for i in range(Niter):
    optimizer.zero_grad()
    gt_depth = get_depth_render(trg_points, requires_grad=False)
    pred_depth = get_depth_render(input_points , requires_grad=True)
    loss_depth = torch.nn.L1Loss()(pred_depth, gt_depth)
    loss = loss_depth
    loss_list.append(loss.detach().cpu().item())
    loss.backward()
    optimizer.step()

Hi, thanks for your interest! @Wi-sc I think Chamfer Distance is not the reason for NaN in your experiments.
In our method, the supervision of depth maps can help optimize the point encoder and generator networks. However, in your case, you want depth maps to directly optimize the 3D Points themselves. It could be an interesting application of our point renderer.
Could you decrease the learning rate? lr =1.0 might be too large.

@AlphaPav Thanks for your reply. I have changed the learning rate into 0.01. But it also outputs NaN. And actually I also tried to try an auto-encoder network. But l1 distance between depth maps cannot regularize the training. I think this is not reasonable because Render4Completion actually do the experiments. Have you test the modified depth renderer under this setting?

@AlphaPav Hi, I still cannot train a simple encoder-decoder network under supervision of depth maps.

Hi @Wi-sc , in my opinion, using depth maps as the only supervision without CD/EMD should not work.
Just imagine when only one pred point appears in the pred_depth and one gt point appears in the gt_depth, and there is a distance between the gt point and the pred point. When calculating the L1 loss between such two depth maps pixel-wisely, since all pixels nearby the pred point mostly see nothing but close-to-zero values in gt_depth (because the two points have a distance), then the gradients back-propagated to the pred_depth pixels can hardly reflect directional information concerning the location of the gt point. Therefore, the pixel-wise L1 loss will hardly make the two points closer.
Furthermore, in our p2i code, we implement the rendering with only a limited range around points for rendering efficiency: pixels far away from the rendered point are directly set to an arbitrary value. Under our implementation, it becomes more impossible that a L1 loss between depth-maps could ever work.


Redeploy not working in Rancher 1.3

Not sure if it's an API updated, but I'm getting the following output on a redeploy command.
time="2017-01-10T21:57:24Z" level=fatal msg="Failed to read project: yaml: unmarshal errors:\n  line 1: cannot unmarshal !!str `2` into map[string]interface {}"

I did an in-service upgrade to the Cowbell service itself, thinking it might have stale API keys, but it's still not working. Here's the full container log:
1/10/2017 5:30:46 PMINFO[0000] Listening on :8080 with token SOME_TOKEN_HERE
1/10/2017 5:31:22 PMINFO[0035] Running in /tmp/compose-051362227 command /usr/bin/rancher-compose [rancher-compose -p Notify up -d --force-upgrade -c -p PAID-Dialer]
1/10/2017 5:31:22 PMtime="2017-01-10T23:31:22Z" level=fatal msg="Failed to read project: yaml: unmarshal errors:\n  line 1: cannot unmarshal !!str `2` into map[string]interface {}"

The {docker|rancher}-compose.yml files that live in /tmp/compose-#### are docker-compose v2. Perhaps that's the issue?


Detect fediverse:creator tag

https://blog.joinmastodon.org/2024/07/highlighting-journalism-on-mastodon/
I'm not sure how often this would come up, as I'm still a little unsure of why one would use
 instead of a rel="me" tag or vice-versa. But, I imagine news sites will likely lean towards the fediverse:creator tag and it'd be cool if that showed up in streetpass

Just stumbled into this in a practical case, The Verge is indeed using these. Streetpass caught the global Verge account but not the article author's.
Specific link is https://www.theverge.com/24206037/casio-ws-b1000-review-smartwatch-wearable

Thank you for the interest in this feature! It’s coming around the end of the month once I have some free time :)

Apologies on the timeline. When I said end of the month I actually mean end of August, not July😄


notepad added to tools

It is automatically included on every Windows PC and you can do a lot with it.

Great :)


Support async service implementations

Background
As part of the Kubernetes Agent work, we want to start using async service implementations (not just the async client in Server).
There are two things we need to do to support this:

Change how service contracts are registered
Add a command-line flag/environment variable to enable the AsyncHalibutFeature on the Halibut Runtime.

Results
Enabling the AsyncHalibutFeature means that the Halibut Runtime, when handling the incoming message, will choose to use the async path, where it will try and execute service contract methods with an Async suffix
I've also updated how services are registered with the AutofacTypeRegistry. Previously, we used implemented interface names on the types implementing the [Service] attribute and they resolved the service from the Autofac container by name. However, for async services, the contract being sent over the wire is the synchronous contract, when our async service implementation doesn't implement. Thus it won't be registered with the correct name. To fix this, I've changed the ServiceAttribute to take the ServiceContractType, forcing the developer to specific the type that is the contract.
An example async service + contract might be something like this:
public interface IScriptServiceV3Alpha
{
	ScriptStatusResponseV3Alpha StartScript(StartScriptCommandV3Alpha command);
}

public interface IAsyncScriptServiceV3Alpha
{
	Task<ScriptStatusResponseV3Alpha> StartScriptAsync(StartScriptCommandV3Alpha command, CancellationToken cancellationToken);
}

[Service(typeof(IScriptServiceV3Alpha))]
public class ScriptServiceV3Alpha : IAsyncScriptServiceV3Alpha
{
    public Task<ScriptStatusResponseV3Alpha> StartScriptAsync(StartScriptCommandV3Alpha command, CancellationToken cancellationToken)
	{
		// implementation here
	}
}

I'm happy to take feedback on this approach. It feels like making the contract more explicit is a good improvement.
TODO: Fix tests
How to review this PR
Quality :heavy_check_mark:

Pre-requisites

[ ] I have read How we use GitHub Issues for help deciding when and where it's appropriate to make an issue.
[ ] I have considered informing or consulting the right people, according to the ownership map.
[ ] I have considered appropriate testing for my change.

This looks pretty good but my one concern is that, in the future when synchronous contracts are no longer required or used, will we continue to have to implement them just to use the name in the [Service(typeof(...))] Attribute? It feels like the Async contract should be able to "Just work" without needing to know about the synchronous equivalent.
We can probably kick that can down the road if you think it's not worth worrying about now but just a thought.

This looks pretty good but my one concern is that, in the future when synchronous contracts are no longer required or used, will we continue to have to implement them just to use the name in the [Service(typeof(...))] Attribute? It feels like the Async contract should be able to "Just work" without needing to know about the synchronous equivalent.
We can probably kick that can down the road if you think it's not worth worrying about now but just a thought.

Right now there is some limitations where you can't have Task<T> in the contract definition. Maybe that's something that should change, but I don't know what the possible side-affect that would be.
I thought this might be the best approach while we still are mixing async & async services, but definitely happy to have feedback


moving to maintained action and splitting up linux github actions yml and

The existing Github Actions scripts make use of upload-release-asset which is now no longer maintained and suggests using action-gh-release instead.
So I think it might be time to change the GitHub Action used to upload the compressed files to the release.
Additionally as mentioned in the call today the combined action for Ubuntu and Debian upload was getting a bit large so I've split those two into separate yml files which.
The github actions triggered by a test release on my fork all worked as the failing option Ubuntu 21.04 and Cubit 2021.4 has been removed from the matrix.
https://github.com/shimwell/Cubit-plugin/actions

Is the combination of Ubuntu 21.04 and Cubit 2021.4 known to fail before now?


document standard configs

Need examples and explanation of the five standard config types. This will inform users of ziti CLI and ZEDS could link back to this new documentation. ZED services are currently a dead end for the user if they don't have prior knowledge how to compose a standard config. We should prominently feature the two most common standard config types with multiple examples: host.v1, intercept.v1 and list the most relevant variables that apply to each e.g. $dst_hostname, $tunneler_id.name.

This doc has some examples with variables https://github.com/netfoundry/MEC/tree/master/Azure/PublicMEC#readme

Can you reopen the issue?
The current documented standard configs are host.v1 and host.v2.
The documentation for intercept.v1 is still missing?
https://openziti.io/docs/reference/config-types/


No "exports" main defined in package.json

Hey! I'm trying to use altcha-lib in a NestJS backend and when I import it, I get this error:
Error: No "exports" main defined in node_modules/altcha-lib/package-json.
Could you please add the exports field to the package.json? Thank you!

Hi, thanks for reporting. The problem was caused by the missing commonjs build, which is now added in the version 0.1.2.


Training Large-Scale Tesseract OCR Model for Uchen Script

Description
Train Tesseract OCR model on multiple large-scale Uchen script datasets (~4M images) for improved Tibetan text recognition.
Implementation:

Data preprocessing and format conversion
Training file generation
Model training and optimisation

Subtasks:

[x] Consolidate all datasets and verify completeness
[x] Convert text files to ground truth format
[x] Configure Makefile parameters
[x] 	Generate box and lstmf files
[ ] 	Execute and monitor training process

Completion Criteria:

Successfully train a Tesseract model for Uchen script recognition
Card Reviewer:

[x] @ta4tsering

@tenzinyonten can u elaborate the step number 4

@kaldan007 Expanding on the task card - here's the detailed LSTM workflow in Tesseract 5.0+:
For data preparation:

Input: line-segmented Uchen script images
Ground truth: Name files as .gt.txt (e.g., line_01.gt.txt), ensure exact text matching, and use UTF-8 encoding
Tesseract 5 internally handles: image normalization, feature extraction, character encoding, and line geometry

Training files are automatically generated:

Box files (line-level, not character-level):
Analyzes image regions
Maps line coordinates and text content
Handles alignment information
LSTMF (LSTM Feature) files - crucial for efficient training:
Optimization: Convert raw images and alignments into a binary format to avoid redundant computations.
Integration: Combine preprocessed image features, box data, and ground truth into a compact package.

LSTMF files serve as preprocessed and optimized representations of training data, integrating image features, text alignments, and ground truth into a compact binary format. This ensures faster training by reducing redundant computations.
Makefile config needed:
START_MODEL= (blank for fresh training).
#If START_MODEL is blank, training starts from scratch. Otherwise, specify a pre-trained model (e.g., 'eng' or 'tib') to fine-tune on your dataset.
MODEL_NAME=tib
DATA_DIR=/path/to/training/data
TESSDATA=/path/to/tessdata
MAX_ITERATIONS=2000000
LEARNING_RATE=0.001
LANG_TYPE=Indic
 #Set LANG_TYPE=Indic for scripts like Tibetan that use complex ligatures and require advanced shaping logic during recognition.

Training Flow:
Input: Line Images + .gt.txt.
Internal Processing: Tesseract handles image normalization, feature extraction, and alignment preparation.
LSTM Training: The preprocessed data is fed into the LSTM network for recognition training.
Evaluation: The model is iteratively tested and refined.

the suggested data sample for parallel training is:
30k,300k,3mill


DSC confirmation layout for Github

Date: Nov 17, 2021
Order ID: 37500885
Shipping and billing information

Shipping address

Billing address

Mike Miller  University of Neverland	 100 N Testroad D-1234  Room 233 Extra Building  New York NY 012345  United States  Phone: 867-5309<EMAIL_ADDRESS>Fedex: 12345670

Susan Meyer  University of Neverland	 100 N Testroad D-1234 R 567 Fault Building New York NY 012345  United States  Phone: 867-5309<EMAIL_ADDRESS>Credit Card

Stocks ordered

Item
Quantity

Strain
1

Plasmid
2

Strain information

ID
Descriptor
Name(s)
Systematic Name
Phenotype

DBS0236180
talA-
tal A null, talin null
GG1666
aberrant cytokinesis, decreased cell-cell, adhesion, decreased growth rate, decreased phagocytosis, increased cell-substrate adhesion

Strain storage

ID
Name
Stored as
Location
No. of vials
Color
Storage Comments
Storage Date

DBS0236180
talA-
axenic
1-34(76-78)
7
Pink
HG1666, 2 in A
4/17/06

DBS0236180
talA-
spores
1-35(21)
2
pink
hg1666, in Tank A and Tank C
5/4/06

Plasmid information

ID
Name
Plasmid Keywords

DBP0000546
pDM353
GFP marker, neomycin resistance, expression vector, extrachromosomal vector, gateway vector, act6 promoter, act15 promoter

Plasmid storage

ID
Name
Stored as
Location
No. of vials
Color
Storage Date

DBP0000546
pDM353
in DB3.1
17(21-22)
2
Blue
5/12/09

DBP0000546
pDM353
DNA
20-[6E,7A]
1
MC vial

DBP0000546
pDM353
DNA
13-16D
1
MC vial

Adding some updated columns to sections forthe DSC order confirmation that will appear in Github as an issue
Just creating an order confirmation in Github is needed

Great thank you, make it much easier to implement.

Great thank you, make it much easier to implement.

In short, i have to modify this code
https://github.com/dictyBase/event-messenger/blob/develop/assets/issue.tmpl


[Feature]  Derive valid checksum last word when restoring seed.

For people generating their own entropy by pulling words out of a hat.

That button would only highlight when all but the last word has been entered.

There is a selection of words that could serve as the last word, since the checksum is only the last few bytes of the final word.
I have considered adding this feature before though, so it's good to have it listed here.

Duplicate https://github.com/sparrowwallet/sparrow/issues/156


Feature: Tab key behavior

Implement tab key behavior.
The program already supports that the user sets tab width.
When pressing tab, the cursor should move to the nearest tab stop counting from the beginning of the line.
Now, the cursor always moves the number of spaces set in the tab width.

Tab key behavior fixed. The tab stops are now calculated from the beginning of the line.


Update gql dependencies

Hi, I'm getting an error when updating graphql package alongside with artemis.
Because artemis 6.12.3-beta.1 depends on gql_http_link ^0.2.9 and no versions of artemis match >6.12.3-beta.1 <7.0.0, artemis ^6.12.3-beta.1 requires gql_http_link ^0.2.9.

I might submit a PR. Thanks!

@vasilich6107 do you also update dependencies inside the example folder?

you can also use dependacy_overrides

Updated deps in #221


Add mesh renderer instancing support GLESv3, Dropping GLESv2 support

Continue #1242
Describe your changes
Issue ticket number and link
Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] I have checked readme and add important infos to this PR (if it needed).
[ ] I have added/adapted some tests too.

Because this feature require GLESv3, and we decide to drop GLESv2 support, so I mark this PR to 1.1.x
once 1.0.x branch be created, when this PR ready, can be merged into dev

I'm NOT sure does glsl-optimizer support translate modern gl shader to metal at runtime

hmm that's gonna suck 😕

according to this blog post I read it seams that GLES V2 isn't really that popular anymore according to this quote

The takeaway is that 88% of active devices support GLES 3+. Only 12% are limited to GLES 2. If you have limited time and resources, dropping support for GLES v2 might make sense. You may find older devices driving a disproportionate number of support requests.

We need many opinions from people at this point. @solan-solan do you think dropping GLES v2 is a bad idea? we're dropping it since it's complicated to maintain a version that 12% of devices support, and keep in mind that the 12% will even drop lower in the next years.

Try this shader conver tool (compiled from https://github.com/microsoft/ShaderConductor), seems can write shader by HLSL, then use it convert to OpenGL, GLES, Metal shader)
ShaderConductor-f18f7b5_2.zip

@DelinWorks

do you think dropping GLES v2 is a bad idea?

I really do not know. If you sure that it would be better. But it is needed to fix material also? There is some parsing of shaders now when attributes are saved to internal array. Do you propose to reject gles 2.0 shaders at all? There's no opportunity to leave existing shaders as is and extend material class that it can process gles 3.0 and so on? Version tag could be related from some variable.
And such question. Is instancing feature really depend on gles v3.0? I launched some example with gles v1.0 and it worked.  Just 'gl_InstanceID' could not be used

Can be cloased, it's already merged into latest dev branch


File name / command ambiguity

Currently (0.7.7) filenames are preferred over subcommands and this leads to name clashes. For instance, having a shell script named build prevents the subcommand crystal build from being called. Instead, Crystal will try to compile the shell script.

+1. I think this should be the other way, i.e. subcommand > files. And if we really want to compile and run a crystal file called build, crystal ./build.

Is it like git? (for example, git pull calls git-pull shell script)

The fix is actually quite simple, but I don't have everything set up to compile and run the tests.
https://github.com/manastech/crystal/blob/master/src/compiler/crystal/command.cr
@MakeNowJust no, there are none of these shortcuts in place as far as I can tell.

@beno I understand your sense.

Fixed by #1414


Add various declarations for the CPython C API

Add a bunch of declarations which are used by SageMath.

please also copy the short documentation text from the CPython docs.

I will do that but personally I think it's rather pointless to duplicate CPython docs.

personally I think it's rather pointless to duplicate CPython docs.

I know what you mean, but wouldn't call it "pointless". There should at least be a hint about borrowed references for each "risky" or "special" function, and since we have the tradition of keeping the documentation next to the function declaration, I don't see why we should stop doing that. I think it's good to ship at least some minimal documentation with the API declarations.

The problem with the dict issue is that you're using dict really for two different things: the builtin dict on the one hand and the cdef class dict on the other hand. I've been bitten by this in the past: if you do
from cpython.type cimport *

def foo(x):
    if type(x) ...

then the call type(x) is no longer optimized by Cython.
Maybe we should be using different names for these builtin cdef classes, for example typeobj or typesubclass (ugly but clear) instead of type. It's just an idea...

I removed the tricky __builtin__.dict part (maybe to be reconsidered later) and fixed your other remarks.

Thanks Jeroen!


Add functionality to disable folders

Basically copied it from the Controller class to the GUI class.
The only difference is that I made the folder autoclose when disabling it.
At the moment I use this external function to do the same, but I think it makes sense to have this functionality as part of the GUI class instead:
function setFolderEnabled(folder, enabled) {
    const folderElement = folder.domElement;
    folderElement.style.pointerEvents = enabled ? 'auto' : 'none';
    folderElement.style.opacity = enabled ? '1.0' : '0.5';

    if(!enabled) {
        folder.close();
    }

    return folder;
}

Hi there, thanks for this. So we just crossed over 30kb un-gzipped and I'm starting to get a little self-conscious about file size. I'm already a little bothered by the fact that Controller and GUI duplicate code for show/hide.
If we were to do this, I'd want an elegant way for these classes to share code and styles. At one point I tried making them both inherit from some base class to share fields like domElement and hide() / show() but it made the code a little too clever / hard to read. Maybe someone could do a better job of that than me.

Hi @georgealways!
I added a new commit, which implements a simple function based implementation sharing between Controller.js & GUI.js for the show & disable functionality.
I opted for this simple approach instead of an inheritance based one because I don't think using a base class is useful for these classes.
About sharing the styles: I don't really know how the scss syntax works :|
I can take a look later at how the styles could be shared.

@georgealways
I added a new commit where I renamed hover.scss to shared.scss.
I added a new mixin called 'disabled' and now use it instead of the copy-pasted approach.

Are you able to see if these new mixed-in methods appear correctly on the API page?

These images are from the API.md previewed in VSCode. Both 'enable' and 'disable' are there for GUI and Controller too

Thanks again for looking at that! Just had a chance to try this PR myself, overall looking great.
Thoughts on the functionality itself:

I don't think disable should automatically close a GUI. Let's leave that up to the dev.
The entire GUI shouldn't be transparent if the root is disabled: only the text and controllers should dim. I can lend a hand here if that gets hairy.

I like your change on hover => shared.scss

I think @disabled should be renamed to @can-disable to match the hover functions (but maybe they won't share too much CSS if disabled folders and controllers are styled differently).

On utils/guiUtils.js I have some thoughts around naming mostly:

The entire folder is basically GUI utils, so I wonder if this could be more descriptive.
disableGUI/showGUI — Could be disableItem to be more accurate, or maybe just disable/show?
The JSDoc you've done here is great, but it might be a little above and beyond since these functions aren't exposed. Then it might feel a bit better to call the gui parameter something like item.

This is petty on my part, but internally, I wish the helper methods were (enable/show) or (disable/hide). The fact that there's one "positive" and one "negative" bugs me. I think you're following my current pattern though, so that one's on me :)
Let me know what you think. Appreciate your help! -g

I don't think disable should automatically close a GUI. Let's leave that up to the dev

For me a 'disabled' folder means the user can't open it -> the user can't interact with any of it's fields. That's why I added an automatic close to the disable method. Maybe converting it to a parameter named 'closeOnDisable' with a default value of false is a good compromise?

The entire GUI shouldn't be transparent if the root is disabled: only the text and controllers should dim. I can lend a hand here if that gets hairy

In this case yes, I might need help with this!

I think @disabled should be renamed to @can-disable to match the hover functions

Makes sense! Renamed the mixin to '@can-disable'

it might feel a bit better to call the gui parameter something like item

This makes sense to me! So then

Could be disableItem to be more accurate

'disableItem' and 'showItem' names should also make sense to me! Also

The entire folder is basically GUI utils, so I wonder if this could be more descriptive

maybe 'itemUtils'? Because now it should contain 2 functions ('disableItem' and 'showItem')

The JSDoc you've done here is great, but it might be a little above and beyond since these functions aren't exposed

Yes, I left them there so that I could use autocompletion & typing checks, removing it makes sense because these are just internal functions.

Thanks so much! Looking awesome.

I don't think disable should automatically close a GUI. Let's leave that up to the dev

For me a 'disabled' folder means the user can't open it -> the user can't interact with any of it's fields. That's why I added an automatic close to the disable method. Maybe converting it to a parameter named 'closeOnDisable' with a default value of false is a good compromise?

Well we have the chaining: gui.disable().close() feels better than gui.disable( true, false ). And I think it's more conventional to have the controls still visible while disabled.
I also feel like folders should still be foldable/unfoldable in their disabled state... what if GUI.disable() just loops through its children and calls .disable() on each controller? That would also fix the issue of the semi-transparent root. But if you wanted disabled controllers to stay disabled when you enable the parent folder (🙄) you'd be out of luck. Let me know what you think of that idea.
Otherwise, I'll take care of semi-transparent root. And I'll take care of show/hide rename for sure.

Wanted to add, that this is also a feature request from me.
I have some toggles, that enable or disable features. So the user doesn't get confused about toggling things which are disabled, I .disable() elements. I would love to have the same for entire folders, as right now I just disable all child elements and the associated code doesn't look very elegant.
@Degubi Big thanks for taking on this challenge.

Well we have the chaining: gui.disable().close() feels better than gui.disable( true, false )

Yep, the chaining solution definitely looks better!

But if you wanted disabled controllers to stay disabled when you enable the parent folder (🙄) you'd be out of luck

Yeah, this is why I decided to make disabled folders not openable/closable in the first place :|
(Which is why i also autoclose folders when disabling them)
Tbh, I don't really know anymore what would be the best solution here...
If i wanted to disable all controllers in a folder I could just loop through all of its child controllers and disable them.
For me disabling a folder means disabling the whole 'group', which means I can't even access any of its elements.

Yea, I'm having some trouble with it myself. I took a crack at it in this branch: https://github.com/georgealways/lil-gui/compare/dev...disable-folders
I think it works / looks as intended, but you can still tab into inputs inside a disabled folder. That's what the $disable property of controllers is for: marking the actual form element with a disabled HTML attribute.
There's no way to add that attribute to the container element and have it filter down unfortunately. GUI could try to apply that attribute to children on disable, but then we run into a dual state issue: how to remember which controllers weren't enabled when re-enabling the folder?
Stepping back, this feels like a lot of complexity when gui.controllers.forEach( c => c.disable() ) exists, but I'll keep thinking on it.


Watermark a Video

Hi
I was looking at the following link for adding a watermark
https://github.com/bramp/ffmpeg-cli-wrapper/issues/76
Specifically the code
`FFmpegBuilder builder = new FFmpegBuilder()
.setInput(infile)
.addInput(sampleWatermarkImg)
.overrideOutputFiles(true)
.addOutput(destFile)
.setVideoCodec(codec)
.addExtraArgs(extraArgs)
.setVideoFrameRate(24, 1)
.setVideoResolution( iwidth , newHeight)
.setComplexVideoFilter("overlay=main_w-overlay_w-10:main_h-overlay_h-10")
.setStrict(FFmpegBuilder.Strict.EXPERIMENTAL)
.done();
log.debug( "Running ffmpeg as " + builder.getExecString());`
However with the current version of the code - the method setComplexVideoFilter doesnt exist
Which method should I be using?
Thank you
Damien

In case of anyone else struggled with the same issue:
You should use .setComplexFilter() not .setComplexVideoFilter() and use it before .addOutput() !!


Make sure global version works.

global.js should be usable via <script> tag.

Solved in v10.0.1.


loudflare

Assuming you were intending to say "Cloudflare", they are currently hosting the DNS for the sheetjs.com domain.  If there is a particular resource you would like to access, let us know and we can try to find a non-CF approach


io.circe.generic.extras.auto._

Hi,
I migrated to the new akka-http-json version, and now looks like that with the new import
import de.heikoseeberger.akkahttpcirce.FailFastCirceSupport._ and import io.circe.generic.extras.auto._ my code is not compiling and I am getting:
could not find implicit value for parameter encoder
however when I replaced it with import io.circe.generic.auto._ , the code compiles.
Any know issues ?
thanks,

Never used ...extras. Why would you want to do that?

well, simply because it have extra stuff :-)
for example
implicit val customConfig: Configuration = Configuration.default.withSnakeCaseKeys which allow to control how the json keys would look like

@mazorigal not sure if that might help, but this is how I'm getting snake case:
A trait to be mixed in your routes:
https://github.com/prascuna/scala-pills/blob/master/04-sample-rest-service/src/main/scala/scalapills/sample/routes/JsonSupport.scala
and your Configuration inside your DTOs package object
https://github.com/prascuna/scala-pills/blob/master/04-sample-rest-service/src/main/scala/scalapills/sample/models/package.scala
I don't manually import FailFastCirceSupport._

Is this still an issue?

@prascuna Thanks,
I would try it.


Mitaka nfp 10 may 2016 description changes multi vendor

This PR includes description changes and multiple service vendor support ,made network_functions description size to 1024

Ran ATF .


UndefVarError when referring to struct in closure dynamo

Basically I can write:
module ClosureScratch

using IRTools
using IRTools: IR, @dynamo

mutable struct Counter
    count::Int
end

@dynamo function (c::Counter)(m...)
    ir = IR(m...)
    for (v, st) in ir
        c.count += 1
    end
    return ir
end

function foo(x::Int)
    y = x + 5
    return y
end

c = Counter(1)

c() do
    foo(5)
end

println(@code_ir c foo(5))
println(c)

end #module

and this will work, you can refer to c inside @dynamo.
But if I define the dynamo elsewhere in the library, I run into issues...
ERROR: LoadError: Error compiling @dynamo Main.TraceTransform.Jaynes.Trace on (Main.TraceTransform.var"#1#2",):
UndefVarError: tr not defined

where I've defined a dynamo inside my library as
@dynamo function (tr::Trace)(m...)
    ir = IR(m...)
    println(tr)
    ir == nothing && return
    recurse!(ir)
    return ir
end

The first example seems like it must be an IRTools bug. @dynamos don't have values available (since they run at compile time), so the object c should refer to the type of c that you passed in, not the actual counter object (just like the other arguments to the dynamo do).
I guess in @code_ir we're wrongly making the self argument available, whereas during actual compilation we ignore the self argument entirely and should instead expose tr = typeof(tr).
We could fix these but I suspect you don't actually want that fix, based on the examples you've shown, so we might need to just discuss the use case a bit.

Okay cool! It's very likely I'm trying to implement something in the wrong way. Basically, I've been trying to use a struct to control the IR transformation. Idiomatically, that would mean I'm writing something like
@dynamo function (trace::Trace)(m...)
    ir = IR(m...)
    ir == nothing && return
    recurse!(ir)
    return ir
end

function (tr::Trace)(call::typeof(rand), dist::T) where T <: Distribution
    result = call(dist)
    score = logpdf(dist, result)
    record!(tr, _address goes here_, dist, result, score)
    return result
end

but here the dispatch on typeof(rand) and dist only has local information available from the right hand side of that particular IR assignment so I can't do anything clever with address goes here.
I would really like to be able to fill that address goes here with the symbol (assuming that it exists) from the left hand side. So I started trying a few things ... I started by using @code_lowered to grab slotnames from the program and then pairing them with IR variable names. But this doesn't work with dispatch because dispatch only knows about the right hand side call. Then, I started messing around in the dynamo - to pass that sort of address information (which I pre-compute and store in the Trace object which I'm using) I'm trying to access the field inside the dynamo.
I get the feeling there's a more elegant way to do this without the dynamo but I'm curious to hear your thoughts.


ToDo App using Kotlin [Android Development]

Prerequisites

[X] I am a GSSoC-22 Participant
[X] I checked to make sure that this issue has not already been filed

Description
I would like to work on Todo application which is a task manager application using Kotlin. I will use Room Database for storing tasks with SQLite under Android Development tech stack.
Tech Stack
Others (Mention in Description)

@Phani-22 Go for it. Good Luck 👍

closed with #425


Add file name expansion when running a shell command

Please describe the problem you're trying to solve
Sometimes user might want to open a file with a non-default program which is neither listed in the open interactively list.
For example there's a PDF file, the default program in use should be a PDF viewer. The user might want to open it with nvim (I know PDF has binary data, just giving an example here), then the user had to type :nvim filename.
But the file name can be long, or can be in non-English language, and switching IME is required to input the file name, let alone that there's no auto completion for entering shell commands in yazi.
Would you be willing to contribute this feature?

[ ] Yes, I'll give it a shot

Describe the solution you'd like
This would be more convenient if the filename can be expanded, like :nvim $F, and $F will be expanded to filename when the command is actually run. And which file's name to expand? The file under the cursor of course. Also it can be expanded to multiple file names if several files are selected.
Additional context
No response

Sorry, my English is bad. Is it possible to make this function as an additional item "Open with" when choosing to open interactively? That is, we press the key to open interactively, and among the list of things we can open with, the command open with, and when we select this item, we need to enter the name of the program with which to open? That would be very cool! Thak you!

This would be more convenient if the filename can be expanded, like :nvim $F, and $F will be expanded to filename when the command is actually run

It's already supported, you can use $0, $1, $2 etc. variables in the shell, see https://yazi-rs.github.io/docs/configuration/keymap#manager.shell

Also it can be expanded to multiple file names if several files are selected.

Likewise you can use $@ to represent multiple files.

Is it possible to make this function as an additional item "Open with" when choosing to open interactively?

The items in "Open with" come from user-configured opener rules, and they also support variables like $0, $1, $@, see https://yazi-rs.github.io/docs/configuration/yazi#opener

Oh I didn't know this. Thanks!


Unable to learn codes

Hello,
I am unable to learn codes. Could you please help me?
Hardware:
Raspberry PI
2 Broadlink RM Mini 3
Here is my config.json:

{
"platform": "BroadlinkRM",
"name": "Broadlink RM",
"hideWelcomeMessage": true,
"hideScanFrequencyButton": true,
"hideLearnButton": false,
"hosts": [
{
"address": "<IP_ADDRESS>",
"mac": "24:df:a7:b1:f7:d6",
"isRFSupported": false
},
{
"address": "<IP_ADDRESS>",
"mac": "24:df:a7:b1:ef:c2",
"isRFSupported": false
}
],
"accessories": [
{
"accessory": "Broadlink RM Learner",
"name": "Learn Salón",
"type": "learn-code",
"host": "<IP_ADDRESS>"
},
{
"accessory": "Broadlink RM Learner",
"name": "Learn Dormitorio",
"type": "learn-code",
"host": "<IP_ADDRESS>"
}
]
}

Log output:

Jan 30 14:22:13 raspberrypi homebridge[13402]: [30/01/2020, 14:22:13] [Broadlink RM] Learn Salón getSwitchState: undefined
Jan 30 14:22:13 raspberrypi homebridge[13402]: [30/01/2020, 14:22:13] [Broadlink RM] Learn Dormitorio getSwitchState: undefined
Jan 30 14:22:17 raspberrypi homebridge[13402]: [30/01/2020, 14:22:17] [Broadlink RM] [INFO] Learn Code initializing (<IP_ADDRESS>)
Jan 30 14:22:17 raspberrypi homebridge[13402]: [30/01/2020, 14:22:17] [Broadlink RM] [ERROR] Learn Code (Couldn't learn code, device not found)
Jan 30 14:23:42 raspberrypi homebridge[13402]: [30/01/2020, 14:23:42] [Broadlink RM] Learn Salón getSwitchState: undefined
Jan 30 14:23:42 raspberrypi homebridge[13402]: [30/01/2020, 14:23:42] [Broadlink RM] Learn Dormitorio getSwitchState: undefined

Thanks in advance,
Best regards

Looks like the devices are discovered but for some reason the learn-code is not working.

Feb 01 11:05:08 raspberrypi homebridge[13945]: [01/02/2020, 11:05:08] [Broadlink RM] Broadlink RM device at <IP_ADDRESS> () is no longer reachable. (<IP_ADDRESS>, Request timed out)
Feb 01 11:05:16 raspberrypi homebridge[13945]: [01/02/2020, 11:05:16] [Broadlink RM] Broadlink RM device at <IP_ADDRESS> () has been re-discovered.

So are you still getting

... [Broadlink RM] [ERROR] Learn Code (Couldn't learn code, device not found)

even after

Broadlink RM device at <IP_ADDRESS> () has been re-discovered

? That seems pretty weird, and if so, I am afraid I'm not able to help — this vastly exceeds my knowledge of the software :(
Just one idea, most probably wrong and misleading, but well, you were warned :) Perhaps there might be some mess caused by the two separate devices? Myself,. so far, I have always used only one. In your situation, I would probably try to temporarily remove one of the devices and check with only one to see whether it works all right or not.


Connection error with 0.20.1

Describe the bug
I experienced issues when upgrading from 0.19.4 to 0.20.1.
In particular datasets.load_dataset(<repo_name>, data_files=) throws the following exception in 0.20.1, but works fine when running exactly the same code in 0.19.4
ConnectionError: Couldn't reach https://huggingface.co/datasets/.../README.md (ConnectionError('Unauthorized for URL https://huggingface.co/datasets/.../README.md. Please use the parameter `token=True` after logging in with `huggingface-cli login`'))

Setting token=True, I get:
ConnectionError: Couldn't reach https://huggingface.co/datasets/.../README.md (error 401)

Reproduction
datasets.load_dataset(<repo_name>, data_files=)
Logs
ConnectionError: Couldn't reach https://huggingface.co/datasets/.../README.md (ConnectionError('Unauthorized for URL https://huggingface.co/datasets/.../README.md. Please use the parameter `token=True` after logging in with `huggingface-cli login`'))

System info
- huggingface_hub version: 0.20.1
- Platform: Linux-6.2.0-39-generic-x86_64-with-glibc2.35
- Python version: 3.10.12
- Running in iPython ?: No
- Running in notebook ?: No
- Running in Google Colab ?: No
- Token path ?: /home/<machinename>/.cache/huggingface/token
- Has saved token ?: True
- Who am I ?: <username>
- Configured git credential helpers: store
- FastAI: N/A
- Tensorflow: N/A
- Torch: 2.1.1
- Jinja2: 3.1.2
- Graphviz: N/A
- Pydot: N/A
- Pillow: 10.1.0
- hf_transfer: N/A
- gradio: N/A
- tensorboard: <IP_ADDRESS>
- numpy: 1.26.2
- pydantic: N/A
- aiohttp: 3.9.0
- ENDPOINT: https://huggingface.co
- HF_HUB_CACHE: /home/<machinename>/.cache/huggingface/hub
- HF_ASSETS_CACHE: /home/<machinename>/.cache/huggingface/assets
- HF_TOKEN_PATH: /home/<machinename>/.cache/huggingface/token
- HF_HUB_OFFLINE: False
- HF_HUB_DISABLE_TELEMETRY: False
- HF_HUB_DISABLE_PROGRESS_BARS: None
- HF_HUB_DISABLE_SYMLINKS_WARNING: False
- HF_HUB_DISABLE_EXPERIMENTAL_WARNING: False
- HF_HUB_DISABLE_IMPLICIT_TOKEN: False
- HF_HUB_ENABLE_HF_TRANSFER: False
- HF_HUB_ETAG_TIMEOUT: 10
- HF_HUB_DOWNLOAD_TIMEOUT: 10

Hi @alex-hh, thanks for reporting this issue. Just to be sure, which version of datasets are you using? Also, you could copy-paste the full stacktrace and the snippet of code that leads to this error? If you don't want to show which repo dataset you are trying to access, can you still report if it's a public, gated or private repo? Thank you in advance.

Hi @Wauplin thanks for the response.
I'm using datasets 2.15.0. And accessing a private repository. When I try to pull the same file from a public repository with a similar data structure, I don't get the error with either huggingface_hub library version.
When using 0.19.4 I get this warning:
/home/instadeep56uk/alfred/venv/lib/python3.10/site-packages/huggingface_hub/repocard.py:105: UserWarning: Repo card metadata block was not found. Setting CardData to empty.
  warnings.warn("Repo card metadata block was not found. Setting CardData to empty.")

When using 0.20.1 I just get the error message above.

raceback (most recent call last):
File "/home/alexhh/alfred/test_hub.py", line 19, in
load_gym_dataset(DMS_name)
File "/home/alexhh/alfred/test_hub.py", line 7, in load_gym_dataset
return datasets.load_dataset(
File "/home/alexhh/alfred/venv/lib/python3.10/site-packages/datasets/load.py", line 2128, in load_dataset
builder_instance = load_dataset_builder(
File "/home/alexhh/alfred/venv/lib/python3.10/site-packages/datasets/load.py", line 1814, in load_dataset_builder
dataset_module = dataset_module_factory(
File "/home/alexhh/alfred/venv/lib/python3.10/site-packages/datasets/load.py", line 1511, in dataset_module_factory
raise e1 from None
File "/home/alexhh/alfred/venv/lib/python3.10/site-packages/datasets/load.py", line 1495, in dataset_module_factory
).get_module()
File "/home/alexhh/alfred/venv/lib/python3.10/site-packages/datasets/load.py", line 1030, in get_module
dataset_readme_path = cached_path(
File "/home/alexhh/alfred/venv/lib/python3.10/site-packages/datasets/utils/file_utils.py", line 182, in cached_path
output_path = get_from_cache(
File "/home/alexhh/alfred/venv/lib/python3.10/site-packages/datasets/utils/file_utils.py", line 599, in get_from_cache
raise ConnectionError(f"Couldn't reach {url} ({repr(head_error)})")
ConnectionError: Couldn't reach https://huggingface.co/datasets/<repo_name>/resolve/819fa85d06b47235b8a66a288eef7803e538bb5d/README.md (ConnectionError('Unauthorized for URL https://huggingface.co/datasets/<repo_name>/resolve/819fa85d06b47235b8a66a288eef7803e538bb5d/README.md. Please use the parameter token=True after logging in with huggingface-cli login'))

Thanks! Will have a look and get back to you :)

Ping @lhoestq would you know when such an error is raised?
As far as I understand it looks like an authentication issue but I don't see what changes in huggingface_hub that could have produced this issue. We started to -slowly- deprecate HfFolder.get_token but the code implementation did not change at all. Also the bug fix pushed to datasets (but not released yet) to comply with 0.20.1 was only fixing warning messages, not the behavior itself. So maybe it's completely something else and I don't see it.
For the record I've tried to reproduce the issue with no success. @alex-hh maybe you could share what the format structure of the dataset is? Or even better, create a reproducible dataset example with dummy data for which you get this error when the dataset is in private but not when it's public. That would help a lot solving the idea.

datasets uses HfFolder.get_token in get_authentication_headers_for_url()
Can you try logging in again @alex-hh ?
huggingface-cli login

Thanks both for investigating. Logging back in appears to have resolved the issue (now confused about what was going wrong yesterday as thought I had tried this). Thanks!

Glad to know your issue has been resolved! :tada:
I'm closing this issue then :)


Incorrect documentation for Ecto.Changeset.cast/3

The documentation for Ecto.Changeset.cast/3 says

The given data may be either a changeset, a struct or a {data, types} tuple. The second argument is a map of params that are cast according to the type information from data. params is a map with string keys or a map with atom keys containing potentially unsafe data.

However, the function doesn't accept struct as input.  It should be schema instead.

iex> defmodule Post do
...    defstruct [:title, :body]
...  end
iex> Ecto.Changeset.cast %Post{}, %{title: "hello", body: ""}, [:title]
** (UndefinedFunctionError) function Post.__changeset__/0 is undefined or private
           Post.__changeset__()
    (ecto) lib/ecto/changeset.ex:422: Ecto.Changeset.do_cast/4

The offending code is https://github.com/elixir-ecto/ecto/blob/v2.1.4/lib/ecto/changeset.ex#L422
When a struct is given, the function expects the struct's module to implement a __changeset__ function which is defined when using Ecto.Schema.
Can I submit a PR for the docs?

Yes, I think the corrected text should read "a struct of an ecto schema" instead of "a struct"

It looks this is more complicated than a simple text change here. I noticed in several places that Ecto documentation is using the term struct when it is actually a module that uses schema macro.  Technically having struct in the docs is incorrect and is misleading. For eg., Repo.insert/2 says

Inserts a struct or a changeset.

It cannot insert a struct. Struct has no information about the table name to store data.
However, having to replace struct everywhere with  struct created by schema macro  is too verbose and repetitive. Other option is to use schema but again it's technically incorrect, though it is short. It can be ecto schema module, though. Any thoughts?

I would say struct is fine because in the context it makes sense to be a struct backed by a schema. But being clearer is always welcome, so we can also say schema struct. I would agree that calling it schema is confusing.

PR please? :)

Closing in favor of the PR.


So here it is...

Hi, @SimonDanisch, @konsim83,  @furio
so I started his package from the content of TriangleRaw,
at least tests are running.
First obstacle:
We cannot have a struct TriangulateIO.TriangulateIO. So naming the module after
the datastructure it exports was  not my best idea.
Now it is TriangulateIO.JLTriangulateIO  (there is also CTriangulateIO, so some consistency here).
We could keep this or otherwise perhaps we would need a different name of the package...

I think the name of the structure is fine. Why we don-t drop the IO from the package name?

I think the name of the structure is fine. Why we don-t drop the IO from the package name?
Exactly, this also came to my mind when sleeping over it... So we just would take the
method as the name...
Maybe this evening I can fix this and work on the other stuff, need to give my lecture now.

Ok. "Triangulate.jl" it is now.  The struct name is back to TriangulateIO as initially intended.
Travis works for the initial setup.

@SimonDanisch: I set up TriangleBuilder with the BinaryBuilder wizard and got stuck now on travis...
May be you can have a look at the error log...
All cross-compilations were ok on my laptop.

Can you point me to the log?

The last one is
https://travis-ci.org/JuliaGeometry/TriangleBuilder/jobs/624370595
I'm getting some clues anyway. I did the wizard on my Laptop on 1.2 with BinaryBuilder v0.1.4. On travis I get  the build_tarballs  running only on Julia 1.3 with a much newer version. Last try was without the "prefix" stuff as suggested in https://julialang.org/blog/2019/11/artifacts  .
Will download 1.3  and try to run this locally. But the question is anyway: will the new artifact system work with Julia versions <1.3  if it comes to installing packages ?
If yes,   the stuff from TriangleBuilder might be moved to Yggdrasil anway?
First I could try a manual release with the artifacts I have on my laptop...

Tests go through on linux, win, mac, and windows works in my VM. Documention build fails, though.
See the job log https://travis-ci.org/JuliaGeometry/Triangulate.jl/jobs/624517081
I started the registration process, the mandatory waiting period should be over on Sunday evening.

That's great to hear :)

Finally builds for win+linux+osx+ documentation are green. What was missing was a build step in the documenter section of .travis.yml.
... waiting for the registrator now...

Registration done. v0.2.0 should work...


Bluetooth Headsets No Mic

Airpods pro connected
Microphone input detected in system preferences
No input
Google meets detects issue with microphone when it is on airpods pro

Can confirm: No mice with FreeBuds 3
Also sound preferances become very laggy when using headset-mode instead (Codec: SCO) of only speaker (AAC)

@suhrmann
I have the same exact behavior. I bought an external mini USB microphone for the time being. A very odd issue to be sure.
I'm thinking of getting a reddit thread going. This issue is relatively common. Perhaps tyler n has some insight.
Also, adding to this issue.
The microphone no longer works in Windows when booting through opencore boot picker. I will test just from windows UEFI now

No working mic in Windows might be caused by faulty ACPI

I've investigated a little and there seems to be some interaction between Apple ALC and dual-booting. Someone had mentioned that when they boot into windows, hit restart (NOT SHUTDOWN) and then boot into MacOS their mic works, but it breaks when going from MacOS after a shutdown to windows. That hasn't given me any success but it helps inform the way to solve this I guess

Maybe this issue is related to troubleshooting of IntelBluetoothFirmware via restart:
https://openintelwireless.github.io/IntelBluetoothFirmware/Troubleshooting.html#identifying-issues

This kernel extension is only a firmware uploader and it is not needed when you directly reboot from other operating systems that natively support your Wi-Fi card's Bluetooth.
[...]
3.

If your Bluetooth can discover devices, then it's an issue with your Kext injection.
If Bluetooth does not work at all, then it's either a hardware issue (your motherboard leaves your card in a bad state) or a USB mapping issue.

Bad news: [Bluetooth mic not working #176 of OpenIntelWireless/IntelBluetoothFirmware](https://github.com/OpenIntelWireless/IntelBluetoothFirmware/issues/176] got label wontfix

@suhrmann Agh that's super annoying. I bought a USB mini microphone from amazon that I'm going to use in the meantime. I read through the thread and it seemed to be a possible issue with either AppleALC or Bluetooth - if we can collect some persuasive/solid evidence that it's Bluetooth there's a possibility they will try to remedy the issue. It's a huge hole in their software that's clearly being experienced by a lot of users.

Just took a half-assed stab at debugging. Here are my notes:

Tried looking at logs. There are none being reported. Likely because we are on release and not debug version. Need to swap.
Placed all USB injection above all Bluetooth injection. No change in behavior.
Swapped Firmware and Regular Kext positions. No change in behavior.

Next steps to take:

Replace with debug version of kext - see if logs are printing out.
Try their recommended debugging steps to see if kext injection is working as expected. However, I know @suhrmann doesn't even have a windows partition to boot onto, so if the kext works, it's because of proper kext injection, not incidental firmware loading from Windows.

Long long shots:

Reddit post
Contacting people who've also had the issue and asking them if they have found a solution to it.


Trying to fix issues with build script

Addresses https://github.com/elm-lang/elm-platform/issues/64.
Corresponding changes would apply to BuildFromSource.hs.

I tried this and it didn't work for me (same system as in #63), with the much shorter error,
Downloading the latest package list from hackage.haskell.org
Skipping download: Local and remote files match.
Writing a default package environment file to
/opt/elm-platform/versions/0.15.1/cabal.sandbox.config
Using an existing sandbox located at
/opt/elm-platform/versions/0.15.1/.cabal-sandbox
Cloning into 'elm-compiler'...
remote: Counting objects: 20457, done.
remote: Total 20457 (delta 0), reused 0 (delta 0), pack-reused 20457
Receiving objects: 100% (20457/20457), 6.12 MiB | 0 bytes/s, done.
Resolving deltas: 100% (11942/11942), done.
Checking connectivity... done.
Writing a default package environment file to
/opt/elm-platform/versions/0.15.1/elm-compiler/cabal.sandbox.config
Using an existing sandbox located at
/opt/elm-platform/versions/0.15.1/.cabal-sandbox
Resolving dependencies...
cabal: Could not resolve dependencies:
trying: elm-compiler-0.15.1 (user goal)
next goal: base (dependency of elm-compiler-0.15.1)
rejecting: base-<IP_ADDRESS>/installed-5ef..., <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS> (global
constraint requires ==<IP_ADDRESS>)
rejecting: base-<IP_ADDRESS> (global constraint requires installed instance)
rejecting: base-<IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>,
<IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>,
<IP_ADDRESS> (global constraint requires ==<IP_ADDRESS>)
Dependency tree exhaustively searched.

Note: when using a sandbox, all packages are required to have consistent
dependencies. Try reinstalling/unregistering the offending packages or
recreating the sandbox.

The "existing sandbox" at /opt/elm-platform/versions/0.15.1/.cabal-sandbox did not exist prior to running build.sh, and yes, that's the patched version from the PR.

I can copy over the last part of this comment: https://github.com/elm-lang/elm-platform/issues/61#issuecomment-135059357:
"Anyway ... Why is your build still failing? I guess all your new attempts were with GHC 7.10.2? The freezed cabal versions Evan has put into his cabal.config file (the one you downloaded from elm-platform and which the build.sh script is copying into all subdirectories) work only with something from the GHC 7.8 branch. He has said somewhere in another GitHub issue that he is for the moment willing to tell all people that they have to use that version of GHC.
If you want to use a newer GHC, you can replace the cabal.config from https://github.com/elm-lang/elm-platform/blob/master/versions/0.15.1/cabal.config by this one: https://www.stackage.org/lts-3.2/cabal.config. With that file (put into your directory versions/0.15.1 before you call build.sh) building everything with GHC 7.10.2 should work. (It does on my machine.)"

Thanks, but I'm on GHC 7.8.4.

Well, same problem. GHC 7.8.4 uses base-<IP_ADDRESS>, but Evan's cabal.config says that base has to be exactly <IP_ADDRESS>. Evan is fixing a specific compiler from the GHC 7.8 branch.
So, to apply my suggestion above, replace cabal.config not by https://www.stackage.org/lts-3.2/cabal.config, but by something compatible with GHC 7.8.4. Possibly https://www.stackage.org/lts-2.22/cabal.config.

This is such a nightmare. So each version of GHC corresponds to a specific version "double patch" of base? Is this documented anywhere?
Has anyone in Haskell ever made a binary that can be built with 7.6 and 7.8 and 7.10 reliably?

Based on the fact that cabal.config cannot be used across different versions of GHC, I think this whole enterprise is doomed. If it's possible, cool, but I can't lead the way anymore. I'm going to get rid of the experiment as a failure.
Thanks for taking a look at this everyone.

Sorry, I just got pretty upset at everything not working every time and wanted to give up on it. I'm in year 3+ of "cabal does not work, I can't build elm-*" so I don't have much hope left. I need to get over that, and I think this is an improvement and maybe can get us to a thing that works.


Ensure statistic defaults in parquet writers are in sync, and note the intitial difference in compression's default setting.

Which issue does this PR close?
Closes #11367
Rationale for this change
Final step to ensure that all default configuration settings, between the parquet session options and the arrow writer options, remain in alignment.
What changes are included in this PR?
Doc that the compression defaults are intentionally different.
Make the statistics_enabled defaults match.
Fix the bloom filter tests.
Are these changes tested?
Yes.
Are there any user-facing changes?
No.

The clippy failure can likely be resolved by updating from main


fix: using rollupOptions instead of esbuildOptions

Description

Additional context

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[ ] Read the Contributing Guidelines.
[ ] Read the Pull Request Guidelines and follow the PR Title Convention.
[ ] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

Current dependencies on/for this PR:

feat_migrate_optimizer_to_rollup

PR #3

PR #4

PR #5

PR #6

PR #7

PR #8

PR #9

PR #10

Current dependencies on/for this PR:

feat_migrate_optimizer_to_rollup

PR #3

PR #4

PR #5

PR #6

PR #7

PR #8

PR #9

PR #10


Cant determine where the auctions are stored?

Can you please provide insight on where the auctions that are created are stored? Is it in some sort of data store or are they coming directly from the blockchain?

Auctions are contracts which live on the blockchain. The auction data retrieved and displayed on ergo auctions is all from the blockchain, including token name, description,image link, royalty percentage, everything

@HazeyOneKenobi Thank you for following up on this.
That makes perfect sense! So how/where would I need to change the code to make it pull from my address instead of the current address when i run the code, which shows the users from the real website?

Lastly, where is the portion of code that creates the contract?


Remove inner declare

Fixes the template so that $params and $path are not using declare

Thanks, could please update snapshot by running npm run test -- --run -u under root dir.

Thanks, could please update snapshot by running npm run test -- --run -u under root dir.

Done. Also added react-dom to e2e since I couldn't run pnpm i without it.

Merged in https://github.com/yesmeck/remix-routes/commit/7cff5745fe291e1d83bcfcef352fd76f1089a053. However, adding react-dom to dependencies breaks test which I need some time to investigate. Thank you.

Merged in 7cff574. However, adding react-dom to dependencies breaks test which I need some time to investigate. Thank you.

Anything I can do to help move it along?

Merged in 7cff574. However, adding react-dom to dependencies breaks test which I need some time to investigate. Thank you.

Anything I can do to help move it along?

I have no idea why changing dependencies breaks tests; you can look into it if you're interested.

@yesmeck Most likely a minor version that has some breaking change. Me installing react dom generated a new lockfile, bumping all the packages to newest minor version. If you lock all the dependencies to patches it works:

@yesmeck This is the offending release: https://github.com/remix-run/remix/blob/remix%401.14.1/packages/remix-dev/CHANGELOG.md#patch-changes-1. Does that ring a bell for you?


Research Web Components

Pro's and Cons
...

critisism

write about it in your design rationale

What discussions are there about ..?
Pro's and cons
The opinion of the industry


Merge prTesting_201904231037 branch to master branch

Docs Build status updates of commit 09744d6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Bug with Hash

There's this bug I've encountered which I'm sure is because of my code. The bug can be reproduced as follows:
Type this in your browser 'http://mozpacers.org/#contact_form' and press enter.

You'll see that it opens the interface that is meant to display more events or more information about events. This is true for any other hash you type as well. The problem here is that, I wrote code which scans what the hash is after opening this interface, so it should be fairly simple to correct.
Another part of this bug is when you're on the website itself, try to add the hash '#future-more' to it and press enter. It should lead to the interface, but it doesn't.

Anyone who is interested in picking this, let us know.

@CuriousLearner Check if this is still an issue?

Seems resolved. Closing this now.


Feature/tettra integration

This PR works with https://github.com/ombulabs/ombulabs.com/pull/200
The search.json needs to be added before the other PR will work. This search.json file and plugin (simple-jekyll-search) will also be used in the search bar PR.
https://www.pivotaltracker.com/story/show/172560740

I'm merging this since there's nothing to do manual QA.


[Focus Mode] Modal Redesign

Details
We redesign the Focus mode notification modal.
Fixed Issues
$ https://github.com/Expensify/App/issues/32610
PROPOSAL: https://github.com/Expensify/App/issues/32610#issuecomment-1852495529
Tests

Precondition: use an account you haven't logged in with yet today, or create a new account

Login
Popup appears
Verify that the correct copy is displayed
Verify that the text link works
Verify that the Modal has the required illustration

[x] Verify that no errors appear in the JS console

Offline tests
Cannot login when offline
QA Steps
Same as Tests Step

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message: https://expensify.slack.com/archives/C01GTK53T8Q/p1707353348179649

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native
Spanish:

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

Let me know when you got screenshots for me to look at :)

@neonbhai let me know if you need any help : )

Any updates here?

@neonbhai how's it looking? do you need any help?

You should use pink800 for the background color of the illustration and appBackground for the modal background color (is that what you're using now?)  cc @Expensify/design

Sorry for the late feedback, but @dubielzyk-expensify do you think we should use our h1 styling for the headline here:

I don't feel strongly, but that might feel more consistent with other places where we have illustration + headline + text?

That's a good call out Shawn—I'm for it but don't feel SO strongly as to fight about it haha. I think including the H1 style will make this modal feel a bit more promo/marketing-y which in this situation is a good thing IMO.

Agreed. Let's make it h1:

Love it <3

Looks great to me! 🙌 One point though: @shawnborton Is the spacing between the headline and paragraph too big? It looks like we mostly use 8px spacing between h1 and paragraph, but here it looks bigger

Hmm yeah, it does feel a little large. I like using either 8px or 12px (or whatever it is that we typically use 🤣 )

@neonbhai Would be great if you could look at our other screens and replicate the spacing. You can find a similar spacing if you go to Settings > About :)

I think those centered modals are a different kind of component than the #focus mode modal.
I think what you have for the #focus mode modal with 8px looks pretty good. Curious what others think too though.

@shawnborton Agree! I think it's looking good.

@neonbhai could you please merge latest main because it can cause TS errors

@neonbhai conflicts

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/luacmartins in version: 1.4.46-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/roryabraham in version: 1.4.46-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Use Docker image instead of composite action

Context
Currently this action uses another action to install steamcmd, when a docker image could theoretically be used instead.
Suggested solution
Would like to see an approach like in https://github.com/game-ci/steam-deploy/pull/46
Considered alternatives
https://github.com/game-ci/steam-deploy/pull/46 could be good, but we haven't heard back from the person who raised the PR, and we have heard of some issues during testing.
We may want someone else to lead this change.
Additional details
I think completing this change would require version 2.0.0 for this action.

Fixed by https://github.com/game-ci/steam-deploy/pull/46


ocaml-ng.ocamlPackages_5_0: 5.0.0-rc1 -> 5.0.0

Description of changes

https://github.com/ocaml/opam-repository/pull/22687
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Also you may want to update the ocaml-ng.ocamlPackages_latest attribute.

Also you may want to update the ocaml-ng.ocamlPackages_latest attribute.

Are we OK using it as the latest already? If so I'll gladly do it

I guess this is what is to be expected from the meaning of “latest”.

Yes, I was conflating it with ocamlPackages

Updated the PR

Thanks!


Update sbt version and add Play 2.5 support

Needed some swagger in my APIs I am migrating to Play 2.5.
Tests pass, and my swagger docs are working again.

thanks! what about  https://github.com/iheartradio/play-swagger/issues/56?

Fixes #56

This might break compatibility with play 2.4. Do you happen to also have a play2.4 app that you can test with?

I will give that a try right now.

Indeed unfortunately it does break compatibility with 2.4... Not sure how to resolve that.

Get the same error as I got in #56

Thanks, I am going to create a temporary release just for 2.5, do you mind remove the changes you made in version.sbt and retarget the PR to the play-2.5 branch?

Superseded by #58


google global: better empty tracker handling

i'm using the following code:
use Rack::Tracker do
  handler :google_global, trackers: [ { id: ENV["GA_ID"] } ], anonymize_ip: true
end

which throws up the following way if GA_ID is empty:
NoMethodError:
  undefined method `[]' for nil:NilClass
# ./lib/rack/tracker/google_global/template/google_global.erb:2:in `block in singleton class'
# ./lib/rack/tracker/google_global/template/google_global.erb:-2:in `instance_eval'
# ./lib/rack/tracker/google_global/template/google_global.erb:-2:in `singleton class'
# ./lib/rack/tracker/google_global/template/google_global.erb:-5:in `__tilt_70324565778440'
# ./lib/rack/tracker/handler.rb:37:in `render'
# ./lib/rack/tracker/handler.rb:50:in `block in inject'
# ./lib/rack/tracker/handler.rb:49:in `inject'
# ./lib/rack/tracker.rb:67:in `block in inject'
# ./lib/rack/tracker.rb:100:in `each'
# ./lib/rack/tracker.rb:100:in `each'
# ./lib/rack/tracker.rb:66:in `inject'
# ./lib/rack/tracker.rb:54:in `block in call'
# ./lib/rack/tracker.rb:54:in `call'
# ./spec/integration/google_global_integration_spec.rb:36:in `block (3 levels) in <top (required)>'

this patch properly handles the case of empty trackers in the templates and deals with a few other cases which should be considered "empty". also caches collected trackers in an ivar to save a few iterations and cleans up the integration test a little.

thanks for the PR! :+1:

ping @glaszig :) is this still relevant?

just added the warning.

still a blocker? needs improvements? shall i squash the commits?

thanks for the reminder! :)


Lint and test passing for wrong project

Expected Behavior
Just tried using this template for my TS project and when I click the Lint&test passing button, it should refer to my project.
Actual Behavior
The button points to: https://github.com/metachris/micropython-ctl/actions?query=workflow%3A"Build+and+test"
Steps to Reproduce the Problem

Create new Github project with this template.
Press button on github.

Specifications

Version: 0.4.2
Platform: Github

Thanks for the report! You can fix this by changing the url in your readme.
Will fix it in this too tonight

Fixed in https://github.com/metachris/typescript-boilerplate/commit/d5daaf73a6b8ad44d0da3bc7915c3ecf4eb9138b


[SPARK-7451][YARN] Preemption of executors is counted as failure causing Spark job to fail

Added a check to handle container exit status for the preemption scenario, log an INFO message in such cases and move on.
@andrewor14

Can one of the admins verify this patch?

Note that the AllocateResponse also contains PreemptionMessage which is list of containers RM is asking for back.  AM could potentially use this to decide which containers to kill.  That can be future enhancement though.
I think this looks good other then @sryza  comments.

Thanks all for reviewing !
@sryza Addressed your comment in the latest patch.
@tgravescs Yes, one use case on top of my mind is try to avoid preempting containers
which has RDD's cached. However, I believe the current implementation in the scheduler
doesn't take inputs from AM for preemption and that the list of containers in PreemptionMessage
is more of a notification to AM, no ?

Yeah I'm sure it would require more plumbing hence a future enhancement if we decide its useful.

LGTM

Looks like we never ran tests for this. Jenkins test this please

Merged build triggered.

Merged build started.

Test build #32272 has started for   PR 5993 at commit 90900cf.

LGTM2

Test build #32272 has finished for   PR 5993 at commit 90900cf.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32272/
Test PASSed.

Committing this

Thanks for committing this, Sandy !


Facebook share button is broken

The 'Share to Facebook' button is broken - does Code for Germany have access to the Facebook app admin settings?

Actually, I would suggest removing the Share via Facebook/Twitter/... functionality at all. Code for Germany has no access to the Facebook app settings.

I think you can make a pretty convincing case that the social shares were super-effective for spreading awareness of the game - see the 'You Don't Know Africa' experience, which is using the CTH codebase: http://www.davidbauer.ch/2013/12/01/youdontknowafrica-learnings-from-an-accidental-worldwide-internet-hit/
but I dunno if Facebook shares of game results would still as "viral" today as they were 10 years ago.
Facebook sharing has changed a lot since 2013 and maybe we're better off using the newer, simpler sharing setup that doesn't need a facebook "app id" and doesn't load any javascript from FB. That means we can't post things like "I solved the game in 1 minute and 19 seconds" but instead the button would just share the site, just like if someone copy/pasted the link into Facebook themselves.
If we really do want to restore the personalized "I solved the game in 1:23 time" message, we can register for a new app.
Personally, I'm fine if we don't have the personalized "I solved the game in 1:23 time" message, but I do think we should keep some kind of social share button, especially for Facebook, so we should at a minimum change the current FB share button to be the generic Facebook sharer.php link

Fixed via #333.


regression: java symbols no longer displayed as a tree

With vscode-java/jdt.ls, a textDocument/documentSymbol call returns something like:
[Trace - 12:48:07] Received response 'textDocument/documentSymbol - (33)' in 6ms.
Result: [
    {
        "name": "main(String[])",
        "kind": 6,
        "location": {
            "uri": "file:///Users/fbricon/Desktop/Standalone.java",
            "range": {
                "start": {
                    "line": 12,
                    "character": 20
                },
                "end": {
                    "line": 12,
                    "character": 24
                }
            }
        },
        "containerName": "Standalone"
    },
    {
        "name": "Standalone",
        "kind": 5,
        "location": {
            "uri": "file:///Users/fbricon/Desktop/Standalone.java",
            "range": {
                "start": {
                    "line": 10,
                    "character": 13
                },
                "end": {
                    "line": 10,
                    "character": 23
                }
            }
        },
        "containerName": "Standalone.java"
    }
]

vscode-code-outline displays the main method at the root of the tree, same level as Standalone, instead of nesting it under Standalone.  My guess is the latest algorithm gets confused by the root node having a containerName (Standalone.java) instead of a blank value.
This used to work a while ago:

This library relies on location rather than containerName as there was no way to have it work correctly based on container names (there can be multiple objects with the same name).


Support GA4 tag

Added support for Google Analytics 4 tags.
https://github.com/gohugoio/hugo/blob/master/tpl/tplimpl/embedded/templates/google_analytics.html
https://gohugo.io/templates/internal/#use-the-google-analytics-template

@nodejh Can this get merged?

LGTM


Fix proto Makefile

make proto was not executed in CI and this mistake was overseen.

Codecov Report

Merging #101 (6e99bca) into master (f5e7c1a) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 6e99bca differs from pull request most recent head 24684d6. Consider uploading reports for the commit 24684d6 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##           master     #101   +/-   ##
=======================================
  Coverage   70.51%   70.51%
=======================================
  Files          10       10
  Lines        1126     1126
=======================================
  Hits          794      794
  Misses        211      211
  Partials      121      121

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Question]: Set/Change reranker model?

Describe your problem
In the system architecture diagram I see that the system uses reranking and I see there is some code regarding reranker models but in the UI I don't see anywhere to set a reranker model.. The 0.7.0 release notes also say BCE reranker models are supported.
I see in this issue rerank was still in the process of being implemented. Is this still the case?
Is there any way to change or set the reranker model or is that feature still yet to be implemented?


Feature Support auto create range partition for NULL value

Proposed changes
Issue Number: close #xxx
We always put NULL value in the least partition. so if meet one, auto partition will also create the MIN_VALUE partition of it.


enhanced string interpolations

Sorry for the giant code drop... unfortunately it's almost a complete re-write.
I started by just trying to fix multi-line comments but ended up adding a whole bunch more:

booleans
number literals, including supported suffixes
syntax highlighting inside string interpolations

functions
variables
literals

support for heredoc strings

embedded syntax hightlighting based on token suffix (<<CONFIG_JSON, <<CONFIG_YAML, ...)
string interpolation highlighting inside the embedded language (this one is kind of magical)

Any chance we could look at merging this?

Merged and published.


Remove the use of reflect.DeepEqual from a couple of places

Let's close this for now and revisit this after the upcoming work is merged in


feat: update an address of a given customer

Codecov Report

Merging #95 (9273e6f) into master (077b0c9) will decrease coverage by 0.10%.
The diff coverage is 94.44%.

@@            Coverage Diff             @@
##           master      #95      +/-   ##
==========================================
- Coverage   96.43%   96.33%   -0.11%
==========================================
  Files           6        6
  Lines         365      382      +17
==========================================
+ Hits          352      368      +16
- Misses         13       14       +1

Impacted Files
Coverage Δ

service/routes.py
97.50% <94.11%> (-0.58%)
:arrow_down:

service/models.py
99.33% <100.00%> (+<0.01%)
:arrow_up:

Continue to review full report at Codecov.

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


[Docs]: Clarify the schedule in auto update Appsmith

Is there an existing issue for this?

[X] I have searched the existing issues

Documentation Link
https://docs.appsmith.com/getting-started/setup/instance-management/update-appsmith#automatic-update
Discord/slack/intercom Link
https://theappsmith.slack.com/archives/CGBPVEJ5C/p1685456084561999
Describe the problem and improvement.
We need to clarify that the cron job is set to check for updates at so and so time by default. Else users expect the update to start automatically after they set it up

Closed as part of #1321


How to get the muscle activation signal like Figure.10 showed in paper?

Hi myosuite team,
Thanks for your contribution.
I had some issues getting the muscle signal using myosuite (eg, force, fatigue, and velocity, among other things). Through the model attribute I haven't find anything related. Can you show how you get paper 'MYOSUITE A CONTACT-RICH SIMULATION SUITE FOR MUSCULOSKELETAL MOTOR CONTROL' Figure.10?
I hope to know how you get the muscle signal and how to find the muscle activations along a given pose trajectory over time.
Thanks for your patience

Hi @Norweig1an thanks for your interest in MyoSuite!
Are you trying to get info about sarcopenia or fatigue? or, are you trying to replicate that figure or just to capture the info presented?
Typically we get the information about the state of the simulation from  env.sim.data . Below a script to compare this information from both a normal and a model with sarcopenia. Hope it helps!
import numpy as np
import myosuite, gym
env = gym.make('myoElbowPose1D6MFixed-v0')
print("Muscle Force Gain",env.env.sim.model.actuator_gainprm[:,2])
env.reset()
data = []
#to start from approx the same state
for _ in range(100): next_o, r, done, ifo = env.step(np.zeros(6,))
for _ in range(1000):
    a = np.array([0,0,0,.1,.1,.1])
    next_o, r, done, ifo = env.step(a)
    data.append({"action":a.copy(),
                "target":env.env.target_jnt_value.copy(),
                "jpos":env.sim.data.qpos.copy(),
                "mlen":env.sim.data.actuator_length.copy(),
                "act":env.sim.data.act.copy(),
                "mForce":env.sim.data.actuator_force.copy().squeeze(),
                "mForce_gain":env.sim.model.actuator_gainprm[:,2].copy()})
env.close()
env = gym.make('myoSarcElbowPose1D6MFixed-v0')
print("Muscle Force Gain",env.env.sim.model.actuator_gainprm[:,2])
env.reset()
dataS = []
#to start from approx the same state
for _ in range(100): next_o, r, done, ifo = env.step(np.zeros(6,))
for _ in range(1000):
    a = np.array([0,0,0,.1,.1,.1])
    next_o, r, done, ifo = env.step(a)
    dataS.append({"action":a.copy(),
                "target":env.env.target_jnt_value.copy(),
                "jpos":env.sim.data.qpos.copy(),
                "mlen":env.sim.data.actuator_length.copy(),
                "act":env.sim.data.act.copy(),
                "mForce":env.sim.data.actuator_force.copy().squeeze(),
                "mForce_gain":env.sim.model.actuator_gainprm[:,2].copy()})
env.close()
import matplotlib.pyplot as plt
plt.plot([v['jpos'] for v in data],label='Normal'),
plt.plot([v['jpos'] for v in dataS],label='Sarcopenia'),
plt.ylabel('Elbow angle')
plt.xlabel('Samples')
plt.legend()
plt.show()


MINOR: Add Timed wait to SslTransportLayerTest.testNetworkThreadTimeRecorded

@rajinisivaram  SslTransportLayerTest.testNetworkThreadTimeRecorded test is consistently getting stuck in infinite while loop (check below stack trace) on my machine. looks like some timing issues.  spent some time to debug the issue, but no success.
For now, i changed the while loop to timed wait loop. Pls take a look.
"Test worker" #12 prio=5 os_prio=31 tid=0x00007f7fecbb9000 nid=0x5a03 runnable [0x00007000015d2000]
   java.lang.Thread.State: RUNNABLE
        at sun.nio.ch.KQueueArrayWrapper.kevent0(Native Method)
        at sun.nio.ch.KQueueArrayWrapper.poll(KQueueArrayWrapper.java:198)
        at sun.nio.ch.KQueueSelectorImpl.doSelect(KQueueSelectorImpl.java:117)
        at sun.nio.ch.SelectorImpl.lockAndDoSelect(SelectorImpl.java:86)
        - locked <0x00000007969a1b98> (a sun.nio.ch.Util$2)
        - locked <0x00000007969a1b88> (a java.util.Collections$UnmodifiableSet)
        - locked <0x00000007969a1a68> (a sun.nio.ch.KQueueSelectorImpl)
        at sun.nio.ch.SelectorImpl.select(SelectorImpl.java:97)
        at org.apache.kafka.common.network.Selector.select(Selector.java:684)
        at org.apache.kafka.common.network.Selector.poll(Selector.java:408)
        at org.apache.kafka.common.network.SslTransportLayerTest.testNetworkThreadTimeRecorded(SslTransportLayerTest.java:665)

retest this please.

Test failures are unrelated, merging to trunk.


docs(linux): dedupe installation instructions and update conf desc

Added some more detail about the compatibility of the agent with the listed Linux distros
Deduped the install/upgrade instructions since it's safe for people in either camp to fully follow along with the appropriate instructions for their OS distribution type
Added some header notes in the relevant sections to cover our bases in terms of supporting these 3 end-users / scenarios

User doing a new / fresh install of the Linux agent - notes aren't applicable to them, they can & should run every command in the instructions
User upgrading from a older version of logdna-agent-v2 - they can skip the install logdna-agent command if they've ever installed the Linux package for the current agent OR the legacy agent and just update their package. Running through these steps will also ensure that they're pulling the latest version from the correct source repository.
User migrating from the legacy client logdna-agent - similar to above, they can skip the install logdna-agent step if they've ever installed the Linux package and just update the agent. The most recent release of logdna-agent-v2 automates the teardown and removal process of the legacy client as well.

Fleshed out the description/explanation of the logdna-agent service managed by systemd & updated the usage instructions to reflect the standard configuration process for the agent

The default filepath for the agent's configuration file is /etc/logdna.conf and its expected to be plain text file which stores environment variables as key-value pairs to be passed into the agent at runtime

@tanberry I made most of the changes as suggested but I think referring to the legacy agent logdna-agent and the Rust agent logdna-agent-v2 is probably the least confusing way to distinguish the agent for our users. There are inconsistencies in your comments about how we should be referring to the distinct agents so I left them the same for now.


Question: how to ensure the fairness when a single consumer subscribe multiple topics?

When a single consumer subscribe multiple topics, if there are too many messages in one topic, the consumer will keep reading from same topic for a long time. How to ensure other topics have a fair chance to be subscibed?

the consumer doesn't offer any guarantees here. you could try fiddling with fetch.message.max.bytes, queued.max.messages.kbytes or some other consumer config to limit batching of messages read from the broker (messages in batches will be from the same partition and delivered to the application one after another). I don't know off the top of my head the strategy used to decide on where messages are consumed from, but I have a feeling the above would be effective, and the behavior unlikely to change (but with that said, not guaranteed).
also, you could just use multiple consumer instances, if you don't have too many topics.


Support packageManager field in package.json and provide the requested version automatically

Is your feature request related to a problem? Please describe:
After updating pnpm and the engine restriction in a repository, it may no longer be compatible with webcontainers version of pnpm
see https://github.com/stackblitz/webcontainer-core/issues/861
Describe the solution you'd like:
Support the packageManager field in package.json similarly to corepack.
Describe alternatives you've considered:
Use a more relaxed engine constraint and live with the fact that webcontainers use a different version of pnpm
Additional context:
https://nodejs.org/api/packages.html#packagemanager
https://nodejs.org/api/corepack.html
Running corepack inside webcontainer might not be feasible, but you can do it without by simply parsing packageManager and fetching the version from npm registry as well:
https://github.com/sveltejs/vite-plugin-svelte/blob/74fd16db25d0be6c624e3d7e77b170ff7afb34ed/.github/workflows/ci.yml#L46-L48
request extracted from this comment: https://github.com/stackblitz/webcontainer-core/issues/861#issuecomment-1285158343

Thanks for the report!
Yea this is currently a limitations as we only support one version of pnpm right now. I'd be ideal if we supported the packageManager field indeed. I ll take this with me and discuss this with the team.

My suggestion for the time being would be to not use strict engine checks.

fyi,
pnpm@8 has been released today https://github.com/pnpm/pnpm/releases/tag/v8.0.0
projects using pnpm8 are going to use lockfile format v6 by default which is not supported by the current pnpm version deployed on stackblitz webcontainers
~/projects/vitejs-vite-njsrpa 1m 9s
❯ pnpm -v
7.13.6

Thanks for putting this on our radar. I have seen the changes in pnpm and we definitely need to update to the latest version so both versions of the lock file are supported and that newer projects don't break.

If you can't support 8 right away, going up to 7.24 might help to support the new lockfile format at least.

@Shinigami92 Thanks for the info. The plan is to update soon-ish and I will let you know once it's on prod.


__PROC__ is not implemented

__PROC__ compiles as a reference to the current proc, which will pair well with nameof().


[>=8.0.1] Vue warns that onUnmounted is called when there is no active component instance to be associated with

Environment
- Operating System: Linux
- Node Version:     v18.18.0
- Nuxt Version:     3.10.0
- CLI Version:      3.10.0
- Nitro Version:    2.8.1
- Package Manager<EMAIL_ADDRESS>- Builder:          -
- User Config:      devtools, modules, i18n
- Runtime Modules<EMAIL_ADDRESS>- Build Modules:    -

Reproduction
https://stackblitz.com/edit/bobbiegoede-nuxt-i18n-starter-esldjk
Describe the bug
Vue warns that onUnmounted is called when there is no active component instance to be associated with.
Run repro and observe console.
Additional context
The warning emits from useLocaleHead specifically this line. Even if it unused.
Logs
client.mjs:29 [Vue warn]: onUnmounted is called when there is no active component instance to be associated with. Lifecycle injection APIs can only be used during execution of setup(). If you are using async setup(), make sure to register lifecycle hooks before the first await statement.
warn2 @ chunk-RAKT47ZN.js?v=91e5b74c:1449
injectHook @ chunk-RAKT47ZN.js?v=91e5b74c:4068
(anonymous) @ chunk-RAKT47ZN.js?v=91e5b74c:4075
useLocaleHead @ index.mjs?v=91e5b74c:112
injectNuxtHelpers @ utils.mjs?v=91e5b74c:254
setup @ i18n.mjs?v=91e5b74c:315
await in setup (async)
(anonymous) @ client.mjs:29

I have the same problem. I updated to 8.0.1 and now I get warnings.
[Vue warn]: onUnmounted is called when there is no active component instance to be associated with. Lifecycle injection APIs can only be used during execution of setup(). If you are using async setup(), make sure to register lifecycle hooks before the first await statement.

warn$1	@	chunk-UKZM2QLG.js?v=149077c0:1502
injectHook	@	chunk-UKZM2QLG.js?v=149077c0:4311
(anonymous)	@	chunk-UKZM2QLG.js?v=149077c0:4318
useLocaleHead	@	index.mjs?v=149077c0:113
injectNuxtHelpers	@	utils.mjs?v=149077c0:256
setup	@	i18n.mjs?v=149077c0:316
await in setup (async)
(anonymous)	@	client.mjs?t=1706848129076:42
executeAsync	@	index.mjs?v=149077c0:111
setup	@	client.mjs?t=1706848129076:42
(anonymous)	@	nuxt.js?v=149077c0:111
fn	@	nuxt.js?v=149077c0:182
runWithContext	@	chunk-UKZM2QLG.js?v=149077c0:5407
callWithNuxt	@	nuxt.js?v=149077c0:187
(anonymous)	@	nuxt.js?v=149077c0:32
run	@	chunk-UKZM2QLG.js?v=149077c0:312
runWithContext	@	nuxt.js?v=149077c0:32
applyPlugin	@	nuxt.js?v=149077c0:111
executePlugin	@	nuxt.js?v=149077c0:130
applyPlugins	@	nuxt.js?v=149077c0:155
await in applyPlugins (async)
initApp	@	entry.js:48
(anonymous)	@	entry.js:65

% yarn nuxi info
------------------------------
- Operating System: Darwin
- Node Version:     v18.18.0
- Nuxt Version:     3.10.0
- CLI Version:      3.10.0
- Nitro Version:    2.8.1
- Package Manager<EMAIL_ADDRESS>- Builder:          -
- User Config:      ssr, devtools, srcDir, modules, imports, runtimeConfig, app, vuetify, vite, nitro
- Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>- Build Modules:    -
------------------------------

After upgrading to Nuxt 3.10.1 and upgrading i18n from 8.0.1 to 8.0.2, this has now started appearing for us.
We only use useLocaleHead in 1 place, a composable that is called in scrtip setup.
I think this change introduced a regression


/init in initramfs is broken

/init cause kernel panic, rewriting. Consider not installing to your system

Fixed in 1.3.2


fix(footer): language selector styles

Related Ticket(s)
#7329
Description
adjusted icons slightly to the right in language selector for the footer to align with styles

Changelog
New

adjust buttons slightly to the right

@RichKummer just doublechecking, this diff looks like was intentional based on the PR description but wanted to make sure before we push this through?

@RichKummer just doublechecking, this diff looks like was intentional based on the PR description but wanted to make sure before we push this through?

Yup, that's intentional - both icons shifted for this PR


HIDPI Screen issues

When on HIDPI resolution i get this non HIDPI works good

If I remove SDL_WINDOW_ALLOW_HIGHDPI in WispWindows.cpp line 290 all is ok

Waiting for the PR :)


Reformat documentation

Currently the documentation is a single, long README file.
This is great for searching it using Ctrl + F, but it makes it feel more overwhelming to read and makes some third party websites that attempt to render the readme really slow.
Split up most parts of the documentation (except for a quick getting started guide) into smaller files inside a new docs/ directory

I can take care of this!