git lfs file locking support?

Hi,
First, this is a great product. I was able to set up two instance of server within 1 hour is. It is a great experience compared to a failed attempt to set-up two instances of GitLab. The GitBlit is also so simple to admin!
But...
Can we have git-lfs lock support? Currently it is not supported, I think (v1.8.0). I am getting something like:
D:\x\lfs\lfs>git lfs lock kanji.uni
Lock failed: Repository or object not found: http://---------------:8090/r/lfs.git/info/lfs/locks
Check that it exists and that you have proper access to it
Can we have it done?
I know that locking is not a well viewed for source code edition and I agree with that. But I do currently try to extend the intended use of GIT to store the mechanical drawings and etc and those are non-mergable. So ideally I would be glad to be able to lock files for editing and ensure, that anyone who:

checks them out is getting the "read-only-files";
attempts to push them is stopped;
can inspect the lock status through web interface.

Without this I will be, sadly, forced to start admining the GitLab what would be a pain.
Best regards, Tomasz Sztejka.

You are running version 1.8.0, not 1.9.1?
This is a good request. I'm afraid I won't get around  to implement it soon.

@TomaszSzt,
You want to control a remote repo down to individual files, via some locking mechanism. Wouldn't it be better to use a document-management application which would give you these fine-grained controls, instead of git or gitblit? These document software would allow you to "reserve" a document so that nobody else can work on it until you "release" the document and clear the lock.
Another point: you want a centralized repo to control individual files? Just throw out git (or gitblit) and use a centralized revision control software instead like CVS, Subversion, Rational ClearCase, etc. I'll be interested to know if GitLab can do this kind of central control of individual files.
In short, I think a distributed version-control software like git is the wrong tool for your purpose, because you want a centralized control of the files via some locking mechanism, even at the local repo that people have cloned which is impossible. This is quite against the distributed nature of git!

While it is debatable in how far locking for LFS makes sense or not, I guess Tomasz simply refers to LFS locking as it is part of Git LFS 2.0.

Thanks for quick response.
Yes, I am talking about LFS locking.
@chirontt I am aware of all what You said. The company I work for is now using Autodesk Vault (something live CVS) for such a purpose and I can tell You it is a complete mess. The critical thing about version control system is not only versioning of "files" or "repositories" but also capturing the "dependencies" between files, so that if You take out file A in version 1.0 from project P then You can say (and track, monitor, inspect) that this file needs file B in version 3.4 from project X. You know, something like: "a user guide version 5 should be provided with printed circuit board version 7".
And I'm talking about cyclic-criss-crossed dependencies here, so even GIT is shedding tears here - I needed to write own tools to track it because sub-git dies in such case.
What is good about GIT it is that in simplest case it captures the "snapshot" of a project, so at least You don't have to worry about dependencies inside a project. This is a great plus for users.
The place I work for is understaffed and I was hoping to get a solution which is extremely simple to admin. Believe me,  GitBlit is best - no database, no external dependencies, single folder to backup, single excellently documented configuration file to read and modify. No SSH is necessary so I could remove one of potential breaking entry. Nothing except JRE8 is needed. I was able to move server between Linux and Windows without any problems.
Gitblit has a decent web GUI, so no external application is needed to inspect repositories and download them if "read-only" work needs to be done. That way I can use it as a kind of poor man "document distribution center".
And it can run in two copies on the same server serving different content at different ports. This is also important so my team members can train on "testing server" without risking breaking of "production server".
Quality is also good, there a logs and etc. The only bug I have found up to now is when file contains ()[]{} in a name, but it might be GIT problem and can be ruled out by simply not using such names.
So I was thinking: Since it works well for source code, shouldn't it work at least equally good for binary, non-mergable files?
Git works well. As long as I work with a tiny team it is not a problem without file locks. So I came to think: since git-lfs has locks, why not try to use it? It looks good for me and should suit my needs and it will be a single server, single tool chain, single resource and what is important: single training for my non-programmers team members. I am strongly against using two tools - one to server non-mergable documents, and another to serve mergable. Especially often because the ODT or DOC/PDF specs are part of software specs.
@flaix I'm against "update,update,update!" policy. If something is working, why try to break it? I did inspected 1.9 release notes tough and did not found anything what would make the risk of upgrading reasonable.
I do understand that You are not having a free time to spare. Its fine, I also do refrain from coding in my free time, we all have lifes to live.
I did try to inspect source code for GitBlit, but I'm totally green when it comes to jetty, json and that level of techniques. There are also far to less comments (esp. generic ones, like overview.html, package.html and etc) for me to quickly guess where should I look for it. I am also not very fluent in Git administration and still learning.
I think about two possibilities:

Adding this functionality to GitBlit by my self. I think it is doable, providing You will be so kind, and point me to some starting points in Your code.
Setting up git-lfs as a "side channel" using separate server. This is however a total mystery for me and I have no blind idea if it is even possible?

Best regards
Tomasz Sztejka.
P.S. The GitBlit is "mature" now, right? Maybe it is a good time when someone who knows the code should write a kind of "guide", "overview" or something like that and put it in sources? Like, this package does that, this is read by Jetty to do that, this is..... It would really, really x100, help people like me to add fixes. It would let us figure out a proper source file within few hours instead of few days.

I would be happy if I had a "guide", or well documented code and architecture. But there is no such thing. I would also be happy, if the code were well covered with unit tests, so that refactoring it would be less risky. But again, no one really wants to tackle that task. So I'm left with doing little improvements, step by step.


##Peer to peer: Bertrand's Concert Project

You have done a great job thus far.
while your header looks nice it would be good to add the header above the navbar in order to make this design similar to the original.

Thanks @butlermuwo  the second point was solved with the recent commit I made.


Prettier printing for OptionallyStaticUnitRange?

julia> using PaddedMatrices

julia> @StrideArray rand(4,5)
4×5 StrideMatrix{Tuple{StaticInt{4}, StaticInt{5}}, (true, true), Float64, 1, 0, (1, 2), Tuple{StaticInt{8}, StaticInt{32}}, Tuple{StaticInt{1}, StaticInt{1}}, PaddedMatrices.MemoryBuffer{20, Float64}} with indices StaticInt{1}():StaticInt{1}():StaticInt{4}()×StaticInt{1}():StaticInt{1}():StaticInt{5}():
 0.301859  0.54085   0.222296  0.599135   0.767651
 0.336525  0.521614  0.120895  0.891117   0.58328
 0.469067  0.580745  0.745368  0.0832662  0.0853315
 0.489557  0.187212  0.228955  0.261394   0.632375

julia> StrideArray(rand(4,5))
4×5 StrideMatrix{Tuple{Int64, Int64}, (true, true), Float64, 1, 0, (1, 2), Tuple{StaticInt{8}, Int64}, Tuple{StaticInt{1}, StaticInt{1}}, Matrix{Float64}} with indices StaticInt{1}():StaticInt{1}():4×StaticInt{1}():StaticInt{1}():5:
 0.892622  0.862319   0.130142  0.308712   0.196662
 0.600617  0.0845287  0.752346  0.612079   0.534358
 0.545485  0.329272   0.305997  0.922645   0.650952
 0.618598  0.991797   0.60527   0.0707488  0.247227

The
with indices StaticInt{1}():StaticInt{1}():StaticInt{4}()×StaticInt{1}():StaticInt{1}():StaticInt{5}()
with indices StaticInt{1}():StaticInt{1}():4×StaticInt{1}():StaticInt{1}():5

Could be a lot prettier. Of course, I should probably make the rest of the StrideArray type print prettier, too.

Do you think it should be printed with normal integers or have a concise symbol standing in for StaticInt?

I think StaticInt{1}()s for the step could be omitted, so that we have
StaticInt{1}():StaticInt{4}()×StaticInt{1}():StaticInt{5}()

instead of
StaticInt{1}():StaticInt{1}():StaticInt{4}()×StaticInt{1}():StaticInt{1}():StaticInt{5}()

You could also still copy/paste one of these ranges and recover the OptionallyStaticUnitRange.
But if we prioritize pretty printing, we could also pick something more concise. Like just Static(1):Static(4).


Updated all dependencies to the latest versions

@alexey-temnikov @rajeiyer  can you please review this PR

@alexey-temnikov @rajeiyer can you please review this PR

@vishalsingh2972 , Can you please resolve conflicts  in jdbc/pom.xml and send the pull request?

@vishalsingh2972 , Can you please resolve conflicts in jdbc/pom.xml and send the pull request?


The module ... was compiled against a different Node.js version

I'm using Node version 12.1.0 and trying to compile for electron. I have the error message
The module '[project path]\node_modules\sharp\build\Release\sharp.node'
was compiled against a different Node.js version using
NODE_MODULE_VERSION 72. This version of Node.js requires
NODE_MODULE_VERSION 70. Please try re-compiling or re-installing
the module (for instance, using `npm rebuild` or `npm install`).

I noticed that version 70 isn't listed on the releases page. I looked at the releases for this package and the electron release is titled sharp-v0.22.1-electron-v70-win32-x64.tar.gz but the non electron release is titled sharp-v0.22.1-node-v72-win32-x64.tar.gz.

Did you use electron-rebuild?

that did it!

I'm not using electron. I need sharp for cropping images in my node js app. I had installed sharp as a local module and it was working fine but when i removed local modules in favor of global modules, i encountered the error below
0|index  | Something went wrong installing the "sharp" module
0|index  | The module '/home/ubuntu/.nvm/versions/node/v12.2.0/lib/node_modules/sharp/build/Release/sharp.node'
0|index  | was compiled against a different Node.js version using
0|index  | NODE_MODULE_VERSION 72. This version of Node.js requires
0|index  | NODE_MODULE_VERSION 57. Please try re-compiling or re-installing
0|index  | the module (for instance, using `npm rebuild` or `npm install`).
0|index  | - Ensure the version of Node.js used at install time matches that used at runtime
0|index  | - Consult the installation documentation at https://sharp.pixelplumbing.com/en/stable/install/
0|index  | - Search for this error at https://github.com/lovell/sharp/issues

I'm running across the same thing. Node 12.2.0, latest version of Sharp. Reinstalling and rebuilding haven't had an effect. Not sure what I'm doing wrong.


Submission form/assets page likely still lack feedback about moderation

Given the slight mess (pun intended) made by @RandomShaper and @ndee with their recent submissions and edits (MessDeform and Asset Dropper), I assume that the submission form/assets page are still not explicit enough about assets being properly submitted and awaiting moderation: https://godotengine.org/asset-library/asset/edit?page=0&status[accepted]=1&status[rejected]=1&filter=
It would be nice to hear back from you guys about what could be improved usability wise to make things clearer.

(It's not just @RandomShaper and @ndee of course, basically all submissions and edits to the asset lib have been made two or three times in a row before the submitter gave up or understood that the submission was being moderated).

I can't quite remember the exact flow but probably it lacked a message clear and visible enough about submission success.
Today (I hope) I will update with another release so I'll pay good attention and report my observations.

I think the update submission flow could benefit from a few changes (probably also applicable to the submission of a new asset, but it's not fresh enough in my head to be sure).
I think it's confusing that after submitting the edited asset you are able to revise while it's already been submitted for review. Maybe it would be better to let the user confirm the edit and then in a separate screen tell the user about the status.  This screen could have a link to the list of edits with a text letting the user known he can still revise it and also making him aware of the edit history at the same time.
Also the verb "revise" is misleading for people whose first language is not English. For me it was assumed to mean 'submit for review' more than 'take me back to the form again'.
As a last thing, the actions available would be better located after the difference report rather than on top of it to get a more understandable flow.
Hope all this makes sense.

Another example from @Geequlim that we're not giving enough feedback:
https://godotengine.org/asset-library/asset/edit/74
https://godotengine.org/asset-library/asset/edit/75
https://godotengine.org/asset-library/asset/edit/76

The first and second edit is that I didn't fill all the informations but I clicked submit button.

@Geequlim Well, there is that greenish "Revise" button, that has to be renamed...

Yet another duplicate:

https://godotengine.org/asset-library/asset/edit/82
https://godotengine.org/asset-library/asset/edit/83

I'm growing tired of those, we really need to improve the usability here.

Had issues when submitting an asset yesterday. I left the fields for thumbnails of preview slots I used empty. When I clicked Submit, I got an error message due to the missing thumbnails.
I assumed this means it didn't submit, since there's no point in forwarding a submission to humans for checking if the machine already knows something's wrong. So I submitted again, with thumbnails, unintentionally creating 2 submissions.
What I would expect is this:

If I click submit and there's an error, nothing is submitted and I'm looped back to /asset/submit with the fields filled as I left them and a prominent error message on top (or per AJAX without leaving the page at all)
If I submit and everything's right, I get a page that clearly explains that the asset was submitted for approval and a link to the "revise submission" page

Ah right, thumbnails are submitted seperatelly. @eska014 Maybe you can make another issue for this, as I think it is fairly important? :smile:

You might like to read https://github.com/godotengine/asset-library/commit/48da8fa29535e1311e1ea1196912b784f4bd0843#diff-9c79f95cd754be44dc6c0a985a42c49aR59 and think if it is enough. :smile:

We're not there yet... :(

https://godotengine.org/asset-library/asset/edit/123
https://godotengine.org/asset-library/asset/edit/124
https://godotengine.org/asset-library/asset/edit/125

Please save form inputs during errors,  cause last time i was uploading a plugin there was an error and i had to start filling all over again.

@funabab Can you open another issue about this?

With #68 being fixed which should prevent some "false positives" about submissions errors, it should be better already. Let's see how it goes practically before closing this general issue.

So.... seems like that's the last step before 1.0 :tada:

Before beta you mean ;)

Let's assume it's good, and we'll open a new issue or reopen if need be.


[docker] prune docker images once a day

Motivation
So we can keep our automated releases/testing in check, delete any release older than 90 days, and any other image older than 14 days (usually testing repos).
Have you read the [Contributing Guidelines on pull requests]
yes
Test Plan
Already ran the script locally.
 It's age is 8 -- deleting
test-1_265e7a2c not release. It's age is 8 -- deleting
test-1_867905c1 not release. It's age is 8 -- deleting
test-1_7dca5019 not release. It's age is 9 -- deleting
test-1_4c35f08d not release. It's age is 9 -- deleting
test-1_ffe3bc71 not release. It's age is 9 -- deleting
libra/validator_tcb:release-0.19_79e9afb2 is new, leaving alone.
test-1_e89e70b3 not release. It's age is 9 -- deleting
libra/validator_tcb:release-0.19_ba12e695 is new, leaving alone.
test-1_fc2d0453 not release. It's age is 9 -- deleting
test-1_d7cac24f not release. It's age is 11 -- deleting
test-1_bff6cd89 not release. It's age is 11 -- deleting
test-1_24e72d69 not release. It's age is 11 -- deleting
test-1_dc451e8d not release. It's age is 11 -- deleting
test-1_91bd655b not release. It's age is 11 -- deleting
testflow1_97f49d24 not release. It's age is 11 -- deleting
testflow1_1057f73b not release. It's age is 11 -- deleting
testflow1_bd5556e1 not release. It's age is 11 -- deleting
testflow1_b56e4f15 not release. It's age is 11 -- deleting
testflow1_6ed89f2b not release. It's age is 11 -- deleting
testflow1_fc50a578 not release. It's age is 11 -- deleting
testflow1_8035243d not release. It's age is 11 -- deleting
testflow1_73d9eb7c not release. It's age is 11 -- deleting
premainnet-0.19_6957709d not release. It's age is 15 -- deleting
testflow1_d641f942 not release. It's age is 15 -- deleting
premainnet-0.19_f6c1d193 not release. It's age is 15 -- deleting
testflow1_c970629b not release. It's age is 22 -- deleting
testflow1_67588d65 not release. It's age is 23 -- deleting
NOW<PHONE_NUMBER>
NOW_DAYS 18496
libra/build_environment:latest is new, leaving alone.
libra/build_environment:circleci-latest is new, leaving alone.

Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

/land


v2.0.0 'ValidationException' object has no attribute 'errors'

Bug Description
When a validation fails, instead of logging the error it will raise an exception about missing attribute 'errors' on ValidationException
Likely culprit: https://github.com/thatmattlove/hyperglass/blob/be09dbc476fa9e35078fd6a648d6756a9c34b0b9/hyperglass/api/error_handlers.py#L51
Expected behavior
No clue this is on a built-in directive - it should be just working. The ValidationError is already a bug in itself.
Steps to Reproduce

Build current v2.0.0
Run AS_PATH or Community built-in directive against FRR
See errors

Local Configurations
Kept everything defaults apart from adding a FRR instance in devices.yaml
Logs
[CRITICAL] 20240403 13:51:29 | hyperglass.api.error_handlers:21 | default_handler → POST /api/query 'ValidationException' object has no attribute 'errors'
[CRITICAL] 20240403 13:51:29 | hyperglass.api.error_handlers:21 | default_handler → POST /api/query 'ValidationException' object has no attribute 'errors'

Possible Solution
https://github.com/thatmattlove/hyperglass/blob/be09dbc476fa9e35078fd6a648d6756a9c34b0b9/hyperglass/api/error_handlers.py#L51 raises the exception - unclear why it's even caused since * from the directive should allow for any input.
Environment
Server
v2.0.0 on Docker using the docker-compose yaml in that branch
Client

OS & Version:
Browser:

Smartphone Details (if applicable)

Device:
OS:
Browser:

Hello,
I've just pushed dd2ce83 which should fix this. I don't have time to do a reproduction of the issue and fully test it, though. I'll leave this issue open so you can test it out and report back.


Initialize form values

Hi! Thanks for making this awesome library!
I have a question, how do you initialize form values? Is there such a feature?

Hi @bobysf12,
we are using standard react input default value here.
https://codesandbox.io/s/l3mxpvmm9
More examples can be found here:
https://github.com/bluebill1049/react-hook-form/tree/master/examples
and react doc for defaultValue can be found here:
https://reactjs.org/docs/uncontrolled-components.html
cheers
bill

Thanks man!

no worries man! if you find it useful please leave a star to support this repo 🙏if you already did, then thank you very much ❤ happy coding feel free to ask questions here too :)


No symbols found in document for custom extension for YAML files

We are writing a custom text editor extension using the LSP example as the baseline. We are set up to activate on YAML files:
"activationEvents": [
    "onLanguage:yaml"
  ],

In connection.onInitialize, we've set up the capability:
  result.capabilities.documentSymbolProvider = {
    label: 'Solution'
  };

And we have implemented onDocumentSymbol:
connection.onDocumentSymbol((params): DocumentSymbol[] => {
  logger.info('onDocumentSymbol called');
  const document = documents.get(params.textDocument.uri);
  return getDocumentSymbols(document);
});

I can see from the LSP server tracing that it is receiving a document symbol but the outline says that no symbols were found.

We do get diagnostics and completions as expected, just not the outline. Are we missing something to get the outline to show?

I figured out that the code that is building the document symbols was throwing an error which caused the outline view to not populate. However this error is not appearing in any output or console view. Is there a way to get those errors to show?

Were did the error happen. On the server or on the client?

Were did the error happen. On the server or on the client?

On the server.

I changed my code of the test server to throw in the document symbol handler and here is what I see

plus

Any additional info

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!
Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!


Support transform request properly on upload

I just realize we can set request body by call uppy.setMeta(), so I grab the request body come out from requester.transformRequestParams() and stick it in, now is properly transformed before upload.
~But unfortunately I don't think we can transform request method or query params, that just doesn't seems possible (or may be too expensive?). If I try set method or endpoint in XHR plugin config, nothing will happen after called uppy.upload() instead of actually starting sending files to the server. I haven't try swap XHR plugin before upload, that's kinda dumb tho.~
Nvm I literally found my way to do that part. Fixing right now.

nice.
other than that, there is a warning and I think its related to uppy. can you check this too.

Okay, I'm checking it

Alright, now will be properly transformed on upload 👌
And I don't see anything on upload, maybe is from your local dev env?

Check the demo, it its not there too maybe its from one of my chrome extension, I don't know. I'll check it later.

Yea I still don't see it.. maybe

found the reason. nvm :)

merged the demo, and I figured from your ss that the backend of the demo was legacy code :) updated that as well.


Sample/Instructions for Spring REST APIs with AAD Spring Boot Starter

Hi there,
Currently we don't have enough docs on how to use AAD Spring Boot Starter with REST APIs - only web applications.
@cljung did an awesome job creating the following blog post:

http://www.redbaronofazure.com/?p=7607
And also documenting his finds at:
https://github.com/cljung/java-rest-api

I believe we could incorporate his findings in the official Microsoft docs (if @cljung agrees, of couse)

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

ID: 2cf5423d-6c4c-26a8-89bb-7686794716d0
Version Independent ID: 998d0369-045c-d32c-d07e-ff7d4352e878
Content: How to use the Spring Boot Starter for Azure Active Directory
Content Source: java/spring-framework/configure-spring-boot-starter-java-app-with-azure-active-directory.md
Service: active-directory
GitHub Login: @bmitchell287
Microsoft Alias: brendm

@allantargino thanks for the input! We'll take this suggestion into consideration in our planning.

thank you for your feedback and I've noted it down in our backlog. we are lacking reference guides for most of our Spring modules and we will improve it over time.

Thank you folks!

Closing this issue since we're tracking this in our internal backlog.


Issues with junk after document for anthropic.

FAILED tests/anthropic/test_simple.py::test_list - xml.parsers.expat.ExpatError: junk after document element: line 2, column 0
FAILED tests/anthropic/test_simple.py::test_nested_list - xml.parsers.expat.ExpatError: junk after document element: line 2, column 0
When you run the tests, we get some errors.

I ran into this running the example on the blog: https://python.useinstructor.com/blog/2024/03/20/announcing-anthropic-support/

https://github.com/jxnl/instructor/pull/531


Added Bauman Moscow State Technical University

@bmstuniversity Unfortunately your request will not be processed further as JetBrains does not conduct business with persons or organisations individually targeted or incorporated in territories subject to applicable EU, U.N. and/or U.S. sanctions and/or certain export control restrictions due to legal, contractual and commercial related risks and reasons.
JetBrains complies with all relevant regulations, including applicable sanctions and export control laws.
We appreciate your understanding and apologise for any inconvenience caused.

@philipto @bmstuniversity
This is obviously a fraud. The official domain name of Bauman Moscow State Technical University is bmstu.ru. According to whios information, the registrant information of bmstuniversity.com is filled in Hubei, China.

Dear @philipto ,
Thank you for your response. We understand the limitations and rules that you are following.
@kawa11 , we kindly request you to recheck and update your whois information. The domain in question was registered through the provider reg.ru and has no connection to Hubei Province, China. Our website serves as a platform for international students, offering various programs. You can find more information about these programs on our website at https://bmstuniversity.com/international-programs. As you may have noticed, the domain bmstu.ru is intended for Russian students. Additionally, we have other international domains like bmstu.net and bmstu.cloud. Some of these domains have also been added to your repository.
If you have any further questions or concerns, please feel freeto contact us at any of the provided contact addresses. We appreciate your understanding.
Best regards


custom training error

When I train using custom dataset.
I got the "AssertionError: Caught AssertionError in DataLoader worker process 1.", "AssertionError: empty batch"
custom dataset:

samples from scannetv2
4 classes
1 instance/scene
no background(only instance)

command:

!python ./tools/train.py ./configs/softgroup_sample3D_3_c4_train.yaml

config:
model:
channels: 32
num_blocks: 7
semantic_classes: 4
instance_classes: 4
sem2ins_classes: []
semantic_only: False
ignore_label: -100
grouping_cfg:
score_thr: 0.2
radius: 0.04
mean_active: 300
class_numpoint_mean: [2303., 3166., 8331., 3948.]
npoint_thr: 0.05  # absolute if class_numpoint == -1, relative if class_numpoint != -1
ignore_classes: [0, 1]
instance_voxel_cfg:
scale: 50
spatial_shape: 20
train_cfg:
max_proposal_num: 200
pos_iou_thr: 0.5
test_cfg:
x4_split: False
cls_score_thr: 0.001
mask_score_thr: -0.5
min_npoint: 100
fixed_modules: ['input_conv', 'unet', 'output_layer', 'semantic_linear', 'offset_linear']
data:
train:
type: 'sample3D_3_c4'
data_root: '/content/drive/MyDrive/data/sample3D_3_c4'
prefix: 'train'
suffix: '_inst_nostuff.pth'
training: True
repeat: 4
voxel_cfg:
scale: 50
spatial_shape: [128, 512]
max_npoint: 250000
min_npoint: 5000
test:
type: 'sample3D_3_c4'
data_root: '/content/drive/MyDrive/data/sample3D_3_c4'
prefix: 'val'
suffix: '_inst_nostuff.pth'
training: False
voxel_cfg:
scale: 50
spatial_shape: [128, 512]
max_npoint: 250000
min_npoint: 5000
dataloader:
train:
batch_size: 4
num_workers: 4
test:
batch_size: 1
num_workers: 1
optimizer:
type: 'Adam'
lr: 0.001
save_cfg:
semantic: True
offset: True
instance: True
fp16: False
epochs: 64
step_epoch: 25
save_freq: 4
pretrain: './hais_ckpt_spconv2.pth'
work_dir: ''
error log:
Traceback (most recent call last):
File "./tools/train.py", line 236, in
main()
File "./tools/train.py", line 229, in main
train(epoch, model, optimizer, scaler, train_loader, cfg, logger, writer)
File "./tools/train.py", line 57, in train
for i, batch in enumerate(train_loader, start=1):
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/dataloader.py", line 530, in next
data = self._next_data()
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/dataloader.py", line 1224, in _next_data
return self._process_data(data)
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/dataloader.py", line 1250, in _process_data
data.reraise()
File "/usr/local/lib/python3.7/dist-packages/torch/_utils.py", line 457, in reraise
raise exception
AssertionError: Caught AssertionError in DataLoader worker process 1.
Original Traceback (most recent call last):
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/_utils/worker.py", line 287, in _worker_loop
data = fetcher.fetch(index)
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/_utils/fetch.py", line 52, in fetch
return self.collate_fn(data)
File "/content/drive/MyDrive/SoftGroup/softgroup/data/custom.py", line 222, in collate_fn
assert batch_id > 0, 'empty batch'
AssertionError: empty batch
Looks like there are no data in batch at custom.py -> def collate_fn(self, batch):
Hopefully you can help me.

I will add guidelines for custom datasets very soon.

Thank you very much and I'm looking forward to it.
I am confused about class label particularly.
(sem_labels in .pth file, labels in val_gt .txt file, ignore_label/ignore_classes in config file)
And if possible, it would be nice if there was a way to train by including the intensity value of the point cloud.
Have a nice weekend.

May I ask how you solved this problem？


Integrated Marshal for client send.

Need to pull from origin/client-dev

Build fails. Why?

See
make[1]: Entering directory `/home/travis/build/gwAdvNet2015/gw-kv-store/client'
make[1]: *** No rule to make target `test'.  Stop.

You're going to want to add a test target (you don't need to actually write tests, you can just echo "No tests here" for now) into client/Makefile

Made to wrong branch. Superseded by #46


Added alert rule to check if zinc instance is down

Description
This commit adds a basic Prometheus alert rule that will trigger
if a zinc instance is down.

Having an Alert rule in the zinc charm will also facilitate its use for testing other charms.

Oh nice. It's on my list to get this integrated with ingress at some point, too.


Make XML comment examples AoT compatible

Use the JSON source generator with net8.0 to make XML comments' example generation compatible with native AoT.
Relates to #2985.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.55%. Comparing base (f85abd4) to head (ffc96c8).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2986      +/-   ##
==========================================
+ Coverage   90.51%   90.55%   +0.03%
==========================================
  Files          74       74
  Lines        2890     2900      +10
  Branches      455      455
==========================================
+ Hits         2616     2626      +10
  Misses        274      274

Flag
Coverage Δ

Linux
90.55% <100.00%> (+0.03%)
:arrow_up:

Windows
?

macOS
90.55% <100.00%> (+0.03%)
:arrow_up:

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

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

Closing as doesn't appear to be appetite to fix.


[Bug] 'Apply Filter' button missing on mobile - P1

STEPS TO REPRODUCE:
On mobile device, the 'Apply Filter' button is hidden and the user is not able to scroll down to see it.
Site location: https://sites-public-tj3gg4i5eq-uc.a.run.app/listings?page=1
EXPECTED RESULTS:
On mobile device, user should be able to see / click 'Apply Filter' button.
OBSERVED RESULTS: Experienced during user testing on resident's mobile device.
 Is this on a real device or on Chrome emulated as a mobile device?

I was not able to recreate on Chrome emulated, so it may be mobile-device specific. Acquiring device specifications from Grady who submitted the bug for review. Please standby.

Confirmed this was resident reported, using an iPhone / iOS system and the button was not visible. The user was unable to scroll down to it. Could this possibly be a iOS system issue resolvable by a software upgrade / reset? If so, I can suggest this to Grady to share with the resident.

I don't know how this could be the case - do you think the resident just wasn't able to scroll it or was trying to scroll on the side instead of inside the filter modal? I've tried reproducing this but can't.

Since we cannot reproduce, will close out. Spoke to Grady and he will keep an eye out for any similar feedback, assuming it was user error. Thanks!


add link to clueless logo in footer

🚀 Feature Request
Is your feature request related to a problem? Please describe.
At present, the clueless logo in footer isn't taking anywhere.
Describe the solution you'd like
I would prefer that logo in the footer redirect to the home page.

@Debajyoti14

@saimaheshtaduri assigned you


 Add a way to access the regular resource pack and mod menu in the UI

The problem with the current UI is that you can't config your mods or properly config your resource packs (Its difficult to put one over the other so maybe keep it the same on world creation but in the actual settings give us the option to change things)
Also sodium options get removed from the settings when this mod is active so there should be a way to access them (as it also makes it hard to enable shaders)

There is already a way to access the "regular" resource pack menu, shift-click the resource pack selector in the Graphics section or on the Load/Create World screen.
About the mod menu, not if you are referring to the fact that it is not compatible with most configuration screens, but the Mods screen can be accessed through the Game Options section
Regarding the sodium options, this is a compatibility issue, I will probably focus on this in future versions


Add bookmarks to self hosted Cherry server on Android phone

Is there a way to add a bookmark from my android phone?
Thanks
Christian

For me, it doesn't work even after and syncing the PC with the phone to add the bookmark, but Kiwi Browser lets you install extensions like the Cherry one.


Upgrade V8 binaries for <IP_ADDRESS> version working

Use existing workflows to build latest V8 artifacts as well as code changes to ensure v8go continues to work.
Includes:

Update github workflow to use go 1.8 & 1.9
Update github workflow to use macos-latest
Update github build workflow to use ubuntu 22.04
Add gitignore for jetbrains and .gclient_previous files
Switch cgo build to C++17 and enable sandbox at build time
Update test with update to date error message
Remove no longer supported build flag.
Move initialization to v8go.go and include flag set to avoid flag freezing
Reorder initialization so allocator is initialized after v8 (required by latest v8)
Commits are kept separate with merges to certify provenance of all library objects built using existing rogchap/v8go mechanisms.

This is reopen of the previous PR using a different branch name so github action automation doesn't accidentally close or overwrite.

@fizx, can you review this? It's basically trivial changes other than the v8 upgrade.

https://github.com/rogchap/v8go#v8-dependency says

Due to security concerns of binary blobs hiding malicious code, the V8 binary is built via CI ONLY.

Could you instead open a PR against the #357 branch with the changes needed to fix the build, but without committing the actual V8 build. I can then merge this PR into that branch and trigger the V8 Build workflow (as documented in https://github.com/rogchap/v8go#upgrading-the-v8-binaries) to merge in the V8 binaries.

Can do.
FYI: this patch already followed that pattern to address this concern: the v8 build commits within this patch are were generated by Github action CI and are maintained as separate commits to guarantee hash provenance. (You'll need to look at the individual commits of this PR to actually see this-- each is a GH generated PR with merge commit to guarantee hash history)

Sorry, my mistake about thinking the build wasn't CI generated.  Github Actions seems to now attribute the automated commits to a person (e.g. https://github.com/rogchap/v8go/pull/357/commits/208b63ea0bd4c82f904e0029bafa32a4c0197f0e is attributed to me) which made it look like a local build.

Sorry, my mistake about thinking the build wasn't CI generated. Github Actions seems to now attribute the automated commits to a person (e.g. 208b63e is attributed to me) which made it look like a local build.

I've kept as is.
If you'd like, I can back out the v8 builds but then someone else just has to do them again :D
I've updated the changelog. Let me know if you are happy with this.

This branch is out-of-date with the base branch

Looks like the branch protection requires branches to be up to date before merging.  I also don't have permission to push a merge of master into this branch to fix it myself for merging.  @jacques-n could you get this branch up-to-date so it can be merged?

Updated


Now validates that a session is still valid before reconnecting a lost websocket connection

@michaelkantor would the challenge event be emitted in this case? It seems like that would transparently restore a new session, and let things proceed as usual?

Thats exactly how this works.  The reauthentication is entirely transparent to the user, and should involve no additional code from you.  The only possible changes developers might be required to make is going from client.once('challenge', f) to client.on('challenge', f) (if anyone was using once).


How can we do detection with our own dataset without mask and seg

when we use our own dataset to train the DetectoRS, the error occured. our dataset have not mask, and we only want to do detectio, how can we do that?

Please checkout the mmdetv2 branch which supports detectors that do not output masks.

Please checkout the mmdetv2 branch which supports detectors that do not output masks.
Thank you for your reply, can you tell us how to get mmdev2? we can not find it.

when we use mmdev2 to do detection task, the error occured, can you tell me how to solve it?
the error is as follows:
Traceback (most recent call last):
File "/home/lgl/PycharmProjects/DetectoRS-mmdetv2/train_crossing.py", line 148, in
main()
File "/home/lgl/PycharmProjects/DetectoRS-mmdetv2/train_crossing.py", line 144, in main
meta=meta)
File "/home/lgl/PycharmProjects/DetectoRS-mmdetv2/mmdet/apis/train.py", line 116, in train_detector
seed=cfg.seed) for ds in dataset
File "/home/lgl/PycharmProjects/DetectoRS-mmdetv2/mmdet/apis/train.py", line 116, in
seed=cfg.seed) for ds in dataset
File "/home/lgl/anaconda3/envs/DetectoRS/lib/python3.6/site-packages/mmcv/utils/config.py", line 324, in getattr
return getattr(self._cfg_dict, name)
File "/home/lgl/anaconda3/envs/DetectoRS/lib/python3.6/site-packages/mmcv/utils/config.py", line 36, in getattr
raise ex
AttributeError: 'ConfigDict' object has no attribute 'gpu_ids'


404 Error ehwn i try to connect matrix-registration on port 5000

How did you install matrix-registration?
docker
What python version are you running?

What version of matrix-registration are you running?
0.9.2-dev2
Your config.yml
server_location: "http://localhost:8008"
server_name: "matrix.."
registration_shared_secret: "---" # see your synapse's homeserver.yaml
admin_api_shared_secret: "---" # to generate tokens via the web api
base_url: "/register" # e.g. "/element" for https://example.tld/element/register
client_redirect: "https://app.element.io/#/login"
client_logo: "static/images/element-logo.png" # use '{cwd}' for current working directory
db: "sqlite:////data/db.sqlite3"
host: "<IP_ADDRESS>"
port: 5000
rate_limit: ["100 per day", "10 per minute"]
allow_cors: false
ip_logging: false
logging:
disable_existing_loggers: false
version: 1
root:
level: DEBUG
handlers: [console, file]
formatters:
brief:
format: '%(name)s - %(levelname)s - %(message)s'
precise:
format: '%(asctime)s - %(name)s - %(levelname)s - %(message)s'
handlers:
console:
class: logging.StreamHandler
level: INFO
formatter: brief
stream: ext://sys.stdout
file:
class: logging.handlers.RotatingFileHandler
formatter: precise
level: INFO
filename: /data/m_reg.log
maxBytes: 10485760 # 10MB
backupCount: 3
encoding: utf8
password requirements
password:
min_length: 8
username requirements
username:
validation_regex: [] #list of regexes that the selected username must match.        Example: '[a-zA-Z].[a-zA-Z]'
invalidation_regex: [] #list of regexes that the selected username must NOT match.  Example: '(admin|support)'
Your error log
waitress - INFO - Serving on http://<IP_ADDRESS>:5000

Area of your issue?
installation
What happened
When i creat a container with config, there no errors in logs, but when i try to connect internal to my server to check registration page it shows:Not Found
The requested URL was not found on the server. If you entered the URL manually please check your spelling and try again.
Steps to reproduce
No response
cant execute to console of container. in logs - 2022-09-01 08:13:02,279 - waitress - INFO - Serving on http://<IP_ADDRESS>:5000/
But in web interface - Not Found
The requested URL was not found on the server. If you entered the URL manually please check your spelling and try again.

sounds like you aren't launching the docker container with that port exposed to the host.

sounds like you aren't launching the docker container with that port exposed to the host.

Sounds you can't tell application layer 404 error code from network layer connection error.

Closing due to lack of activity, if your issue is still relevant feel free to reopen

Yes, it does reoccur, why do you think I'm wasting time talking to you?
No logs, no verbose mode, no tests. Here, mr. docker, expose yourself to this:
# curl localhost:5000
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">
<title>404 Not Found</title>

Not Found

The requested URL was not found on the server. If you entered the URL manually please check your spelling and try again.

sounds like you aren't launching the docker container with that port exposed to the host.

Sounds you can't tell application layer 404 error code from network layer connection error.

You were very rude to me when I was trying to help you, so good luck troubleshooting this obviously user-error problem. You can still get a 404 if you're serving something else on your hosts port 5000.


Support django 2.2

https://github.com/AltSchool/dynamic-rest/releases/tag/v1.9.5 now supports this. U can close this

Included in v1.9.5


[REST] /media/upload media/get

There should be a way to upload media (image, video, audio) and retrieve it.

done


CircleShape creation support

This would be my choice of feature too :+1:

Sorry, this is not going to happen. VisEditor is deprecated now. Some more info.


ibmcloud sl vs upgrade does not have --flavor option

VSIs ordered in flavor cannot be upgraded with --cpu or --memory options but ibmcloud cli does not have option to specify flavor.
NAME:
  upgrade - Upgrade a virtual server instance

USAGE:
  ibmcloud sl vs upgrade IDENTIFIER [OPTIONS]
    Note: Classic infrastructure service automatically reboots the instance once upgrade request is
    placed. The instance is halted until the upgrade transaction is completed.
    However for Network, no reboot is required.

EXAMPLE:
   ibmcloud sl vs upgrade 12345678 -c 8 -m 8192 --network 1000
   This commands upgrades virtual server instance with ID 12345678 and set number of CPU cores to 8, memory to 8192M, network port speed to 1000 Mbps.

OPTIONS:
  -c value, --cpu value     Number of CPU cores
  --private                 CPU core will be on a dedicated host server
  -m value, --memory value  Memory in megabytes
  --network value           Network port speed in Mbps
  -f, --force               Force operation without confirmation
  --output value            Specify output format, only JSON is supported now.

reference:
https://softlayer-api-python-client.readthedocs.io/en/latest/cli/vs/#vs-upgrade

The latest version 2.6.0 should have this option (as --flavor)
$ ibmcloud.exe  sl vs upgrade --help
NAME:
  upgrade - Upgrade a virtual server instance

USAGE:
  C:\Program Files\IBM\Cloud\bin\ibmcloud.exe sl vs upgrade IDENTIFIER [OPTIONS]
    Note: Classic infrastructure service automatically reboots the instance once upgrade request is
    placed. The instance is halted until the upgrade transaction is completed.
    However for Network, no reboot is required.

EXAMPLE:
   C:\Program Files\IBM\Cloud\bin\ibmcloud.exe sl vs upgrade 12345678 -c 8 -m 8192 --network 1000
   This commands upgrades virtual server instance with ID 12345678 and set number of CPU cores to 8, memory to 8192M, network port speed to 1000 Mbps.

OPTIONS:
  -c value, --cpu value     Number of CPU cores
  --private                 CPU core will be on a dedicated host server
  -m value, --memory value  Memory in megabytes
  --network value           Network port speed in Mbps
  --flavor value            Flavor key name
  -f, --force               Force operation without confirmation
  --output value            Specify output format, only JSON is supported now.


Support games from chess.com/live + More error catching

Support for using the extension on live chess, i.e. URL such as https://www.chess.com/live#g=X
Catch error in getting a response from lichess.org.
Catch error in finding a valid sharable PGN.

Also added support for using the extension on live chess, i.e. URL such as https://www.chess.com/live#g=X

@WojtekTB  - The issue mentioned here is fixed in this PR.


Can't compile since file name changed from CScope_guard.cpp to Scope_guard.cpp

download the lastest code.
and compile, getting following error.
wanpengqian@dev:~/threadpool/ThreadPool$ g++ -std=c++17 tutorial/example.cpp src/* ../lib/src/Scope_guard.cpp
In file included from src/../../lib/header/thread/CWait_bounded_queue.hpp:6:0,
                 from src/CThreadPool.cpp:6:
src/../../lib/header/thread/CLock_bounded_queue.hpp:10:9: fatal error: ../tool/CScopeGuard.hpp: No such file or directory
 #include"../tool/CScopeGuard.hpp"
         ^~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
In file included from src/../../lib/header/thread/CWait_bounded_queue.hpp:6:0,
                 from src/IThreadPoolItemExecutor.cpp:6:
src/../../lib/header/thread/CLock_bounded_queue.hpp:10:9: fatal error: ../tool/CScopeGuard.hpp: No such file or directory
 #include"../tool/CScopeGuard.hpp"
         ^~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

try to rename Scope_guard.cpp to CScope_guard.cpp didn't get any better.
wanpengqian@dev:~/threadpool/ThreadPool$ g++ -std=c++17 tutorial/example.cpp src/* ../lib/src/CScope_guard.cpp
In file included from src/../../lib/header/thread/CWait_bounded_queue.hpp:6:0,
                 from src/CThreadPool.cpp:6:
src/../../lib/header/thread/CLock_bounded_queue.hpp:10:9: fatal error: ../tool/CScopeGuard.hpp: No such file or directory
 #include"../tool/CScopeGuard.hpp"
         ^~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
In file included from src/../../lib/header/thread/CWait_bounded_queue.hpp:6:0,
                 from src/IThreadPoolItemExecutor.cpp:6:
src/../../lib/header/thread/CLock_bounded_queue.hpp:10:9: fatal error: ../tool/CScopeGuard.hpp: No such file or directory
 #include"../tool/CScopeGuard.hpp"
         ^~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

I am trying to compile the example. thanks.

It should be fixed by
fix incorrect class name and fix incorrect class name.
Thank you.


Wildcard rules behavioural differences in 5.4

Laravel Version: 5.4.6
PHP Version: 7.0.8
Database Driver & Version: n/a

Description:
I previously enjoyed (5.3) doing the following with multiple, predefined array inputs. In this example, there are x emails[] fields but only the first email should be required.
return [
    'emails.*' => 'distinct|email', // all fields should be distinct and a valid email
    'emails.0' => 'required', // but only the first one is required
];

I'd then collect($request->emails), filter out the nulls and do something. However, in 5.4 this results in:
ErrorException in ValidationRuleParser.php line 154:
array_merge(): Argument #1 is not an array

Steps To Reproduce:
As a full example:

Generate x array inputs.

<form action="/share" method="post">
    @foreach (range(1, 3) as $n)
        <div class="form-group">
            <label for="email.{{ $n }}">
                Email #{{ $n }}
            </label>
            <input type="text" name="emails[]" id="email.{{ $n }}" class="form-control">
        </div>
    @endforeach

    <button type="submit" class="btn btn-primary">Send</button>
    {{ csrf_field() }}
</form>

Create form request, use controller validate method or manually validate with:

return [
    'emails.*' => 'distinct|email',
    'emails.0' => 'required',
];

In 5.3.*, this worked perfectly. My main question is whether this is an intended change. I know the ValidationRuleParser was added in 5.4, so not sure if this very niche requirement was ignored due to it feeling a bit hacky anyway.

Thank you, a PR was created that should handle the issue.

Much appreciated, thanks!

@themsaid in Laravel 5.2 I was doing:
'x.0.y' => 'in:0',
'x.*.y' => 'exists:table,id',

{"x":{"0":{"y":"0"},"1":{"y":"1"}}}

Where the first should be a value of 0 and all subsequent should be a valid ID in the database. In Laravel 5.5 this is no longer working because ValidationRuleParser is combining the rules:
'x.0.y' => [ 'in:0', 'exists:table,id', ],
'x.*.y' => [ ''exists:table,id', ]

Is this expected functionality?

@bytestream I'd say that is 'expected' functionality given that * applies to all indexes in the array.
I can see it's not the functionality you're after though.
If all of the items in x are the same enough to have identical rules for the majority case, why is the first item in x different? Should it even be a part of that array?


Scrap bugs rarity

In response to #6
I did just the bugs for a start, if it's accepted I'll try the other ones.
To scrap the rarity we need to go on every bug page so it takes longer to scrap.
The PR for sure needs a bit of rework, it was to start the thing. So please comment :)

Thanks for taking this on. We might want to make the scraping a little more efficient. This is going to make it hit 100s of pages (if we enable it for fish as well). If we do this we will probably want to run multiple pages at once (this work can be done in parallel). I think we should rework this to do that before I merge in. Let me know if you need any help!

Thanks for taking this on. We might want to make the scraping a little more efficient. This is going to make it hit 100s of pages (if we enable it for fish as well). If we do this we will probably want to run multiple pages at once (this work can be done in parallel). I think we should rework this to do that before I merge in. Let me know if you need any help!

I changed it to run in parallel, but loading 80+ tabs in parallel take some times :joy:
So I changed it to run by chunks, 4 chunks of the bugs array in parallel.
I timed :

45s for 4 parallel chunks
more than 50s for 2 chunks
around 52s for 3 chunks
around 46s for the sequential

My machine is not a beast and I'm not equipped with fiber.
What do you think ?


Add EUPL (European Union Public License)

Sorry, kotlin support should be in another pull request.
Should I remove it?

It is ok. Thank you and sorry for late reply.


Customer feedback | az storage queue | No output example in documentation

Describe the bug
This GitHub issue is being copied from azure-docs-cli 4475 for engineering team awareness.
Suggestion: Until we have an output property for our autogenerated reference content, consider using the long-summary property to return possible command output.  Please see @dcaro for additional feedback / ideas.

Type of issue
Missing information
Feedback
Articles like this shows all inputs but does not provide any example output, making it hard for developers to understand which information is provided by each command.
Page URL
https://learn.microsoft.com/en-us/cli/azure/storage/queue?view=azure-cli-latest
Content source URL
https://github.com/MicrosoftDocs/azure-docs-cli/blob/main/latest/docs-ref-autogen/storage/queue.yml

Related command
az storage queue
Errors
n/a
Issue script & Debug output
n/a
Expected behavior
n/a
Environment Summary
n/a
Additional context
No response

Thank you for opening this issue, we will look into it.

This will be a long term effort to add output examples as currently only some tutorials have sample output. Looking into using swagger to get a sample output.


Support the construction of SortedDawg from a BufferedInputStream

If the collection is already sorted, don't require it to be loaded into memory as a list.

Merging with #38, for PlainTextSerializer


Don't use "char[1] at end of struct" trick (PR 15707)

Re-creation of #15707.  See that for commentary, discussion, etc.

I still remain unconvinced that this change is beneficial, "chuminess" or not.

Closing.


Add progress dialog to readme

thanks!

Thank you @Vojtech!!
On Wed, Mar 28, 2018 at 3:25 PM, Vojtech Novak<EMAIL_ADDRESS>wrote:

thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aakashns/react-native-dialogs/pull/79#issuecomment-377058165,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGE8iUa0ka-PLl0kZM3NfgWPE_4YQPlKks5tjA3qgaJpZM4S_c-m
.


Hybrid SPA sample update

Purpose

Updated hybrid SPA  to the following:

Updated Config file to support Optional claims and hybrid SPA URL
Updated sample.json to include Optional claims and hybrid SPA URL
Updated readme to include to topology image

Thanks for the feedback!
Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[x] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone https://github.com/Azure-Samples/ms-identity-javascript-react-tutorial.git
cd  .\ms-identity-javascript-react-tutorial\6-AdvancedScenarios\4-sign-in-hybrid\App
git checkout update-hybridSpa-sample
npm install
npm start

Test the code

What to Check
Verify that the following are valid

...

Other Information

update topology for two calls to Graph and remove one AAD icon


Add a Dark mode & Light mode

I want to create a dark UI for this website and add a toggle button at the navbar to toggle between light mode and dark mode.
Please assign me under JWOC 2024, if approved.

Hi @Priyanshi662 , Currently there are few major problem in UI of this project like...

There is no fixed navbar....

Footer is not right....

Also few game are repeating and few are out of their div.....
Should I wait till navbar updated then I can add there a toggle button properly...or should I add anywhere....or give me a hard tag and I will fix all UI related problem then add dark & light mode !!
What should I do ??!!

@Surajit0573   #227   needs to be completed first, then you can work on the issue.

@Surajit0573 please start working on this issue


Make max_subcompactions dynamically changeable

What is changed and how it works?

Issue Number: Close #13145
What's Changed:

Make max_subcompactions dynamically changeable

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

No code

Side effects

Performance regression

None

Release note
Make max_subcompactions dynamically changeable

/run-tests

/merge


querySelector(:nth-child) fails on second call

This can be reproduced by querying for nth-child with a descendant, where the descendant doesn't exist for the first matching nth-child.
This should alert "http://fiddle.jshell.net/1, http://fiddle.jshell.net/1", but actually alerts "http://fiddle.jshell.net/1, null" on Phantom:
http://jsfiddle.net/GKb2g/5/
Probably something not being cleared up after the query.

Just came across this bug. I'm using 1.9.8.

In 1.9.8, querying something like "ul > li:nth-child(1), ul > li:nth-child(2)" returns only the first child.

Also cam across this bug: .dashboard_item .title:nth-of-type(1) returns more than one element.

I can confirm PhantomJS 2.0.0 doesn't contain this bug anymore, and this can be marked as resolved.

But it is occurring in PhantomJS 2.1.1.


property translated to undefined after generate

Hi,
first of all - thank you a lot for this cool JS Framework.
I've a little question regarding escodegen + nashorn - when I run the code you can see here:
https://github.com/klopfdreh/wicket/blob/5ca8af738b426b7947dea2d9ea7c305b299cb77a/wicket-core/src/main/java/org/apache/wicket/resource/NashornResource.js
and modify the AST - unshift an abort IfStatement into in a body of a loop e.g. WhileStatement the AST looks ok, but if I use generate the function invocation is going to be translated to undefined:
Actual output:
x = 0;
while (x < 1) {
    if (nashornResourceReferenceScriptExecutionThread.undefined()) {
        print('NashornResource - Abort thread execution and skip all loop / function calls');
        break;
    }
    print('while' + x);
    x++;
}

Expected output:
x = 0;
while (x < 1) {
    if (nashornResourceReferenceScriptExecutionThread.isInterrupted()) {
        print('NashornResource - Abort thread execution and skip all loop / function calls');
        break;
    }
    print('while' + x);
    x++;
}

Thanks in advance!

The "property" field of MemberExpressions is an Expression, not a string.

Thank you a lot! :+1:

May I ask you how to modify it to fit my requirement?

I think you want to use an Identifier node.

That's working like a charm! Thanks!

@michaelficarra - the reason why this happened was that the Oracle nashorn parser translated that into a MemberExpression what is wrong. I use esprima and everthing is working, now. Just wanted to mention that:
https://raw.githubusercontent.com/wicketstuff/core/574f223c7ec887846a160ad722c1ebf702ad7e10/nashorn-parent/nashorn/src/main/resources/org/wicketstuff/nashorn/resource/NashornResource.js


In VSCode, first plot does not appear, only after creating a second plot

Setup
Version: 1.93.0 (system setup)
Commit: 4849ca9bdf9666755eb463db297b69e5385090e3
Date: 2024-09-04T13:02:38.431Z
Electron: 30.4.0
ElectronBuildId: 10073054
Chromium: 124.0.6367.243
Node.js: 20.15.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22631
R 4.4.1
Problem
I have following code:
library(tinyplot)
aq = transform(
  airquality,
  Month = factor(Month, labels = month.abb[unique(Month)])
)
aq$grp <- sample.int(3, nrow(aq), replace = TRUE)

tinyplot(
  Temp ~ Day | Month + grp,
  data = aq,
  pch = 16,
  cex = 2,
  frame = FALSE,
  grid = TRUE,
  facet.args = list(bg = "grey90")
)

tinyplot(
  y = aq$Temp,
  x = aq$Day,
  by = interaction(aq$Month, aq$grp),
  data = aq,
  pch = 16,
  cex = 2,
  frame = FALSE,
  grid = TRUE
)

tinyplot(
  Temp ~ Day | Month,
  facet = ~ grp,
  data = aq,
  pch = 16,
  cex = 2,
  frame = FALSE,
  grid = TRUE,
  facet.args = list(bg = "grey90")
)

When I run this code in VSCode, the first plot does not open the plot-view. After creating the second plot, the second one is shown in the plot view and both plots are accessible in the plot-history. This happens after initial start of VSCode.
Not sure if this is a tinyplot issue or VSCode issue - in RStudio, everything works fine.

Hmmm, I can't replicate. Everything works out the gate for me in my local VS Code session.
Are you using httpgd or the native plot viewer?

Edit: To help narrow down, can you also confirm that vanilla plot(1:10) doesn't produce the same behaviour? Thanks.

Yes, same behaviour. I'm using httpgd. Maybe it's not related to the pkg?

Interestingly, this doesn't happen with ggplot, where the plot immediately appears.

Results are even worse for Positron (https://github.com/posit-dev/positron/issues/4611), but I guess that's likely to be a Positron issue.

Yes, same behaviour. I'm using httpgd. Maybe it's not related to the pkg?

Thanks, that's helpful. In that case, I think we can rule out tinyplot, specifically, being the problem.
My guess is that this is a VS Code issue. I haven't upgraded to the latest release that you are using, so that may well be the problem. However, before closing and passing to an upstream source, do you mind running the following code—one line at a time—in a pure terminal environment (R console / radian / nvim.R / ESS / etc):
library(httpgd)
hgd()
hgd_browse()
plot(1:10)
dev.off()

And then trying the same thing with tinyplot:
library(httpgd)
hgd()
hgd_browse()
library(tinyplot)
plt(1:10)
dev.off()

If both of those work (like they do on my system), then I think we should probably close here and raise this issue at the vscode-R extension repo.

Yes, both work fine. I'll close then.


Introduce passive code checks

High-level description
Introduce passive code checks implement an optional static type checker for Python.
Corresponding DC/OS tickets (required)

D2IQ-64328 Introduce passive code checks.

Related tickets (optional)

D2IQ-ID JIRA title / short description.

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain.
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot request-review @jongiddy

@mesosphere-mergebot changelog-not-required change to unreleased code


Windows support

Beaver not running in windows environment for the following reasons

lack of fcntl library. (can redirect to win32api http://code.activestate.com/recipes/65203/)
difference in os.fork implementation between Linux and Windows (open file descriptor not shared with child process)
Process target not pickleable.

I can pitch it and submit a fix. Please let me know.

Go for it! I don't have a windows machine (fortunately?) so I can't test, but it might be good to see if the package installs/runs on windows somehow (there are CI services for windows out there like appveyor).

Hi,
One of my colleague has a patch. Let me know on how to submit.
On Mon, Jan 12, 2015 at 9:11 PM, Jose Diaz-Gonzalez <
<EMAIL_ADDRESS>wrote:

Go for it! I don't have a windows machine (fortunately?) so I can't test,
but it might be good to see if the package installs/runs on windows somehow
(there are CI services for windows out there like appveyor).
—
Reply to this email directly or view it on GitHub
https://github.com/josegonzalez/python-beaver/issues/284#issuecomment-69588739
.

Pull request is fine! :)

Hi,
changes are ready for your review. Pull request initiated from https://github.com/rajmarndi/python-beaver. Please have a look and let me know in you need any modifications.

Hi ,
Can you complete review of the code at https://github.com/rajmarndi/python-beaver ?
regards
Raj

Can josegonzalez, can you do the review or let others know who can do the review?

Closing due to inactivity. PRs are accepted - Feel free to reopen!


List transaction on personCard click

Summary:
When the person card is clicked, it will list all the transactions associated with the person. Not really sure this is the best way to do it cause I need to pass down the command executor.

Codecov Report

Merging #100 (6caa121) into master (a13b4a3) will decrease coverage by 0.27%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #100      +/-   ##
============================================
- Coverage     56.44%   56.17%   -0.28%
  Complexity      534      534
============================================
  Files           103      103
  Lines          2032     2042      +10
  Branches        203      203
============================================
  Hits           1147     1147
- Misses          817      827      +10
  Partials         68       68

Impacted Files
Coverage Δ

src/main/java/seedu/address/ui/MainWindow.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/address/ui/PersonListPanel.java
0.00% <0.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

LGTM!!!


[DUP] sub-0082_ses-01_task-social_acq-mb8_run-01

TODO

[ ] delete duplicate and delete bold altogether

scannotes
PVC: We start PVC task multiple times (P), the cursor kep diappering and the code was freezing midway. After restarting everything (besides getting the lab jack error multiple times) and checking connections, I notived the anchor for cable from stim PC to TP link was broken and connection was loose. Since we're in the middle of align vids, I'm not sure if the problem is resolved but will check before the next participant.

[ ] Need to remove from fmriprep as well


Feature:  Display variable from ipython interpreter

Spyder IDE has a very nice variable display tab :
it just dispplay into a tab , variable from the Ipython interpreter.
very handy for debugging,

https://stackoverflow.com/questions/64409231/is-there-any-wayby-setting-or-extension-to-view-and-use-variables-in-vscode-ot#_=_

The stack overflow answer points out the Variables view which appears when using the interactive window, that is the closest thing we have in vscode


Windows Build broken

We currently have a broken windows build. It will be fixed within the next hours.
Reason is the underlying binary (gefyra-json) broke.
We added tests for gefyra-ext to make sure this does not happen again.

Fixed in 1.2.10.


[Feature] Support regex rewrite for route

The New Feature
Describe the new feature you want to support clearly.
Your scenes
Describe your use scenes (why need this feature).
Your advice
Describe the advice or solution of this new feature.
Environment

MOSN Version

I'm working in this issue.

In Envoy , if both prefix_rewrite and regex_rewrite are configured, the envoy will exit and return Cannot specify both prefix_rewrite and regex_rewrite.
So i do the some thing in Mosn,
routes_impl.go#NewRouters
for _, route := range vhConfig.Routers {
			if len(route.Route.RegexRewrite.Pattern) > 1 && len(route.Route.PrefixRewrite) > 1 {
				return nil, errors.New("Cannot specify both prefix_rewrite and regex_rewrite")
			}
		}

The reason It's not working  is that mosn ignored this error
routes_manager.go#AddOrUpdateRouters
// adds new router
		// if a routerConfig with no routes, it is a valid config
		// we ignore the error when we addsd a new router
		// becasue we may stored a nil routers, which is used in istio "RDS" mode
		routers, _ := NewRouters(routerConfig)
		rm.routersWrapperMap.Store(routerConfig.RouterConfigName, &RoutersWrapper{
			routers:       routers,
			routersConfig: routerConfig,
		})

So, just ignore this error and use prefix_rewrite or regex_rewrite config?

@liangyuanpeng If both are configured, we can use prefix_rewrite by default first.
In addition host_rewrite is not supported and you can also contribute if you are interested.

@wangfakang
Cool, I am interested in trying it out


#3111 change external login page

Description
change external login page logo
Related Issue : #3111

How Has This Been Tested?

generate clientId and clientSecret
URL : /api/oauth/generate
Param : clientName(clientName), redirectUri(Uri you want to redirect)
Header : Authorization (Bearer {})
Response(ex) : {
"clientId": "naU8OH",
"clientName": "test",
"basicHeader": "bmFVOE9IOlBuNWFwWERZaTc=",
"clientSecret": "Pn5apXDYi7"
}
test login page
{domain}/oauth/authorize?response_type=code&client_id=naU8OH&redirect_uri=https://www.google.com/&scope=read

the logo path is "/assets/images/oauth/{clientName}.svg"
the external app name is {clientName}
ex: Go to {clientName}

Need additional checks?
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

deploy to 2

deploy to 2

deploy to 2

deploy to 2

deploy to 2

deploy to 2


[WIP] fix failed precompile in integration-tests

Suppose to use https://github.com/scroll-tech/zkevm-circuits/pull/301


n_best support

Currently translate.lua supports the 'n_best' option which is useful.
Would be helpful to have this option in CTranslate as well.

I added it as a feature request: http://forum.opennmt.net/t/support-n-best-option-for-ctranslate/135/1


feat!: change backend width to 4

This is a breaking change because it will change the acir opcode compiled by Noir. If they are not updated, proving and verification will fail.

Benchmark results
Metrics with a significant change:

note_history_trial_decrypting_time_in_ms (5): 153 (+77%)
note_trial_decrypting_time_in_ms (32): 81.3 (+109%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit ee176d23 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
676
676
676

l1_rollup_calldata_gas
6,448
6,436
6,460

l1_rollup_execution_gas
582,685
582,673
582,697

l2_block_processing_time_in_ms
1,367 (+3%)
5,228 (+9%)
9,700 (+8%)

note_successful_decrypting_time_in_ms
186 (-2%)
561 (+8%)
993 (+2%)

note_trial_decrypting_time_in_ms
92.4 (-5%)
:warning: 81.3 (+109%)
126 (+21%)

l2_block_building_time_in_ms
17,939
67,987 (+1%)
135,542 (+1%)

l2_block_rollup_simulation_time_in_ms
8,129 (+1%)
29,134 (+3%)
57,520 (+2%)

l2_block_public_tx_process_time_in_ms
9,790
38,795
77,922 (+1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
13,999 (+1%)
26,741

note_history_successful_decrypting_time_in_ms
1,422 (+15%)
2,541 (+4%)

note_history_trial_decrypting_time_in_ms
:warning: 153 (+77%)
139 (-11%)

node_database_size_in_bytes
18,669,648
35,061,840

pxe_database_size_in_bytes
29,859
59,414

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
279 (-1%)
44,366
28,245

private-kernel-ordering
216
52,869
14,327

base-parity
1,896 (+3%)
128
311

base-rollup
715 (+1%)
165,788
925

root-parity
1,507 (-5%)
1,244
311

root-rollup
51.6
4,487
725

private-kernel-inner
645
73,772
28,245

public-kernel-app-logic
429
35,260
28,215

public-kernel-tail
172 (-1%)
40,926
28,215

merge-rollup
8.38
2,696
925

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.2 (+2%)
16.0
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.6
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.593 (+2%)
0.496
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
47.3 (+3%)
73.6 (+2%)
236 (+1%)
443 (-1%)
922 (+6%)
1,811 (+5%)
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
96.0
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.485 (+3%)
0.454 (+2%)
0.430 (+1%)
0.415 (-1%)
0.439 (+6%)
0.433 (+5%)
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
55.8 (+4%)
110 (+3%)
349 (+4%)
659
1,450 (+10%)
2,790 (+6%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
104
207
691
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.498 (+4%)
0.498 (+3%)
0.475 (+4%)
0.455
0.504 (+11%)
0.484 (+7%)
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
62.4 (+2%)

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
109

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.547 (+2%)

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes

tx_size_in_bytes
22,013

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes

tx_pxe_processing_time_ms
3,257 (+2%)
1,758 (+1%)

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
11.6 (-17%)
1,208

@guipublic We restrict this increase when serializing ACIR to a constraint buffer

We likely want to update the default width to match this as well.

Looks like there's a magic variable which needs to be updated somewhere.

I think that deployer address exists in a couple of different files.

I think that deployer address exists in a couple of different files.

I was always changing it just there

@guipublic looks like it needs to be updated again after a master merge

Ah yeah, it was just master breaking the PR again. Nevermind.


[firestore-send-email] Unable to send base64 encoded attachment

[READ] Step 1: Are you in the right place?
Issues filed here should be about bugs for a specific extension in this repository.
If you have a general question, need help debugging, or fall into some
other category use one of these other channels:

For general technical questions, post a question on StackOverflow
with the firebase tag.
For general Firebase discussion, use the firebase-talk
google group.
To file a bug against the Firebase Extensions platform, or for an issue affecting multiple extensions, please reach out to
Firebase support directly.

[REQUIRED] Step 2: Describe your configuration
// ? send the email notification
      return await db.collection('mail').add({
        to:email,
        template: {
          name: 'template',
          data: {
            ...
          },
          attachments: [
            {
              filename: `Invoice-${invoice.invoice_number}.pdf`,
              content: pdf.toString('base64'),
              encoding: 'base64',
              contentType: 'application/pdf'
            }
          ]
        }
      })

Extension name: firestore-send-email
Extension version: 0.1.17
Configuration values (redact info where appropriate):

<EMAIL_ADDRESS><EMAIL_ADDRESS>LOCATION=us-central1
MAIL_COLLECTION=mail
SMTP_CONNECTION_URI=smtps://email_address@smtp.gmail.com:465
SMTP_PASSWORD=password
TEMPLATES_COLLECTION=mail_templates
USERS_COLLECTION=users

[REQUIRED] Step 3: Describe the problem
The extension does not process a base64 encoded attachment. The email is successfully send, however, the email does not have a file attachment.
Steps to reproduce:
Create a record in the mail collection

Expected result
The email should have a PDF attachment.
Actual result
There is no attachment in the received email.

Hi @atanaskanchev.
Could you possibly be missing a content type definition?
export interface Attachment {
  filename?: string;
  content?: string;
  path?: string;
  encoding?: string;
  raw?: string;
  href?: string;
  httpHeaders?: any;
  contentDisposition?: string;
  contentType?: string;
  headers?: any;
  cid?: string;
}

Hi @atanaskanchev.
Could you possibly be missing a content type definition?
export interface Attachment {
  filename?: string;
  content?: string;
  path?: string;
  encoding?: string;
  raw?: string;
  href?: string;
  httpHeaders?: any;
  contentDisposition?: string;
  contentType?: string;
  headers?: any;
  cid?: string;
}

Hi @dackers86  the contentType is set to application/pdf

I've just notice some strange warning in the execution logs:
 {"location":"us-central1","mailCollection":"mail","smtpConnectionUri":"<omitted>","smtpPassword":"<omitted>","defaultFrom":"xxx","defaultReplyTo":"xxx","usersCollection":"users","templatesCollection":"mail_templates","testing":false,"severity":"INFO","message":"Started execution of extension with configuration"} {"severity":"INFO","message":"Completed execution of extension"}

{"severity":"WARNING","message":"message 'undefined' is not a valid object - please add as an object or firestore map, otherwise you may experience unexpected results."}

That warning message should be ok as it is just stating a message object has not been included.
This may be an edge case, but the attachments are expected to be included in the template that you have included as part of your document.
This should be accessed through a name and any relevant data. For example:
 template?: {
    name: string;
    data?: { [key: string]: any };
  };
``

The template looks like

and it is set in the mail doc as

@atanaskanchev
Below is a basic example of how to add a base64 attachment through a template:
/** create an example template */
await firestore()
.collection("templates")
.doc("base64_example")
.set({
  html: "testing",
  subject: "testing subject",
  attachments: [
    {
      filename: `ImportantDocument.pdf`,
      content: "THISISAB64STRING",
      encoding: "base64",
    },
  ],
});

/** send an email using the new template*/
await firestore()
.collection("mails")
.add({
  to<EMAIL_ADDRESS>  template: {
    name: "base64_example",
    data: {},
  },
});

Is this the type of solution you are looking for?

Hi @dackers86 thanks for the suggestion, but my goal is to use the template doc to set the html body and dynamically generate one or more base64 attachments. Adding the attachments to the template doc defeats its purpose as a template. I suppose can generate a new template per email, but ideally the template should not handle the attachments

@atanaskanchev I think I understand, does this example meet your requirements?
  await firestore().collection("templates").doc("base64_example").set({
    html: "{{html_data}}",
    subject: "{{subject_data}}",
  });

  await firestore()
    .collection("mail")
    .add({
      to<EMAIL_ADDRESS>      message: {
        attachments: [
          {
            filename: `ImportantDocument.pdf`,
            content: "",
            encoding: "base64",
          },
        ],
      },
      template: {
        name: "base64_example",
        data: { html_data: "This is html", subject_data: "This is a subject" },
      },
    });

@dackers86 this works perfectly, thanks for your help!

This should really be in the documentation, it was driving me crazy until I found this issue.


Too long list of currency conversions

The send confirmation dialog shows a too long list of different currency conversions.
Is this really needed?

Maybe limit it to NAV + the selected currency?

@aguycalled Does this look ok?

NOTE: my selected currency in the wallet is "PHP" so it shows the value in BTC + Your selected display unit.

Looks great if you ask me.

Or should it be Total Amount XXX NAV
(=XXX BTC or XXX DISPLAY UNIT)

mmm, what about
Total Amount XXX DISPLAY UNIT
(=XXX BTC or XXX NAV if DISPLAY UNIT != NAV)

mmm, what about
Total Amount XXX DISPLAY UNIT
(=XXX BTC or XXX NAV if DISPLAY UNIT != NAV)

Ok, I like that idea

Display unit: NAV

Display unit: BTC

Display unit: PHP or Any other


Use this->GetCallStack instead of GSamplingProfiler->GetCallStack

GSamplingProfiler is currently a global inside the capture class. Within the SamplingProfiler class, this global should not be used, but instead this, as it is a usual class.

GSamplingProfiler is currently a global inside the capture class. Within the SamplingProfiler class, this global should not be used, but instead this, as it is a usual class.

Actually question - is there only one instance of SamplingProfiler? I think we had something with multiple instances for selection tab - was it SamplingReport?

GSamplingProfiler is currently a global inside the capture class. Within the SamplingProfiler class, this global should not be used, but instead this, as it is a usual class.

Actually question - is there only one instance of SamplingProfiler? I think we had something with multiple instances for selection tab - was it SamplingReport?

You are totally right. I wasn't aware of this right now. So we probably need the underlying global data first. Will have a look into this.

As discussed offline. Larger refactoring is needed. So I'll close this one.


built tensorrt_llm-0.14.0.dev2024092401-cp310-cp310-linux_aarch64.whl on Jetson AGX Orin Developer Kit 32gb

System Info
Here is the environment in which  tensorrt_llm-0.14.0.dev2024092401-cp310-cp310-linux_aarch64.whl  was built.
Now I've got a problem in that pynvml does not work much on Jetson. So if anyone has a tensorrt_llm profiler.py that's works on Jetson or is more conversant with and could modify it I'll be able to test this. If not I'll try to make a version of profiler.py that works without pynvml.
If you would like a copy of the wheel let me know where to upload.
It would not build when I set  --extra-cmake-vars "ENABLE_MULTI_DEVICE=0" because of undefined references to ompi_mpi_* and MPI_* when linking to  libtensorrt_llm.so.
Then tried with following command line. and computer os configuration.
export LD_LIBRARY_PATH=/usr/lib/aarch64-linux-gnu/openmpi/lib:$LD_LIBRARY_PATH
python3 ./scripts/build_wheel.py --cuda_architectures "native"
--nccl_root "/usr/lib/aarch64-linux-gnu"
--extra-cmake-vars "USE_CUDNN=1;USE_CUSPARSELT=1"
--python_bindings

JetPack 6.1 Jetson Linux 36.4 Linux Kernel 5.15 and Ubuntu 22.04 based root file system.
CUDA 12.6
Python 3.10.12
torch-2.4.0a0%2Bc469d14-cp310-cp310-linux_aarch64.whl  Pytorch compiled on Jetson 36.3 os with same versions cuda cudnn.
nv-tensorrt-local-tegra-repo-ubuntu2204-10.4.0-cuda-12.6_1.0-1_arm64.deb
cuDNN 9.4.0-1
libnccl2, libnccl-dev 23.4-1+cuda12.6 arm64
Added these ubuntu packages libopenmpi-dev libopenmpi3 libcaf-openmpi-3 openmpi-bin openmpi-common
Editited build_wheel.py to change job_count=8

tensorrt-llm-compilation.txt
python_packages.txt
Who can help?
No response
Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
successful build_wheel.py on Nvidia Jetson Orin AGX Developer kit
pynvml doesn't work on nvidia jetson
Expected behavior
profiler.py will need to be modified to get tensorrt_llm operational on Jetson.
actual behavior
import tensorrt_llm
Traceback (most recent call last):
File "", line 1, in
File "/home/scott/.local/lib/python3.10/site-packages/tensorrt_llm/init.py", line 35, in
import tensorrt_llm.runtime as runtime
File "/home/scott/.local/lib/python3.10/site-packages/tensorrt_llm/runtime/init.py", line 22, in
from .model_runner import ModelRunner
File "/home/scott/.local/lib/python3.10/site-packages/tensorrt_llm/runtime/model_runner.py", line 30, in
from ..builder import Engine, EngineConfig, get_engine_version
File "/home/scott/.local/lib/python3.10/site-packages/tensorrt_llm/builder.py", line 30, in
from .auto_parallel import auto_parallel
File "/home/scott/.local/lib/python3.10/site-packages/tensorrt_llm/auto_parallel/init.py", line 1, in
from .auto_parallel import auto_parallel
File "/home/scott/.local/lib/python3.10/site-packages/tensorrt_llm/auto_parallel/auto_parallel.py", line 14, in
from .config import AutoParallelConfig
File "/home/scott/.local/lib/python3.10/site-packages/tensorrt_llm/auto_parallel/config.py", line 9, in
from .cluster_info import ClusterInfo, cluster_infos
File "/home/scott/.local/lib/python3.10/site-packages/tensorrt_llm/auto_parallel/cluster_info.py", line 12, in
from tensorrt_llm.profiler import PyNVMLContext, _device_get_memory_info_fn
ImportError: cannot import name '_device_get_memory_info_fn' from 'tensorrt_llm.profiler' (/home/scott/.local/lib/python3.10/site-packages/tensorrt_llm/profiler.py)
additional notes
import pynvml
print(pynvml.version)
Traceback (most recent call last):
File "", line 1, in
AttributeError: module 'pynvml' has no attribute 'version'

There are two pynvml based packages.

pip install pynvml  that I uninstalled and kept the following one
pip install nvidia-ml-py  that installs pynvml.py and is the correct package .

I'll work with Tensorrt_llm more tomorrow but with this slightly modified profiler.py I got rid of the instant  errors.
profiler.py.txt

If I can figure out the method to create the many cpp/tensorrt_llm/kernels/decoderMaskedMultiheadAttention/cubin/*.cubin.cpp and other cubin.cpp files for sm87 will do so to see if Tensorrt-LLM will work on Jetson Orin.
If anyone would point me in the right direction I'll attempt this.
Tried branch dev-sm87-trt101 and
https://github.com/sunnyqgg/TensorRT-LLM but it wouldn't build in my Jetson os configuration.


fix: Pad milliseconds with three zeroes

Hi!
This fork appears to have inherited a bug from upstream. As this seems to be the best-maintained fork of glossy, I'm submitting this PR to fix this issue but also in hopes that you might be interested in publishing this to NPM for others to use :grinning:
The bug is that milliseconds are padded only with two zeroes, e.g.
<131>1 2021-09-27T15:52:35.35-07:00 localhost test-graylog-syslog 103065 - - error: Test message

Some logging platforms (like Graylog, not sure about logdna) parse 15:52:35.35 as 15:52:35.350 instead of 15:52:35.035, so this fixes that.
Here's a related issue with more details: https://github.com/winstonjs/winston-syslog/issues/139
Thanks!

I also just ran across this in the spec in case there's any doubt about this being a bug:

The TIMESTAMP described in Section 6.2.3 supports fractional seconds.
This provides grounds for a very common coding error, where leading
zeros are removed from the fractional seconds.  For example, the
TIMESTAMP "2003-10-11T22:13:14.003" may be erroneously written as
"2003-10-11T22:13:14.3".  This would indicate 300 milliseconds
instead of the 3 milliseconds actually meant.

(https://datatracker.ietf.org/doc/html/rfc5424#appendix-A.4)
Thanks!


Feature: support static DNS with dynamic IP

Added the support of static DNS configuration with dynamic IP interface as
long as specified interface has InterfaceIP.AUTO_DNS set to False.
When choosing interface to storing DNS configuration, the interface with
static address and gateway is preferred over dynamic interface with
InterfaceIP.AUTO_DNS set to False.
Added Unit test cases and integration test cases.

Coverage remained the same at ?% when pulling e1ad09f8b4b7a1547b6536269ce678bd9f17b874 on cathay4t:static_dns_dyn_ip into 4d05b7ee88802f65effcd399b4cfdff0d67d6771 on nmstate:master.

Waiting on PR #970


3200 / kwh meter - esphome config ?

Hi there,
I have a meter that has a 3200 / kwh pulse rate rather than a 1000 /kwh pulse rate ..how would I change the config to allow for this ..I have tried to work it out but have not found anything that makes sense
if you could let me know where and how I need to change my config that woul be amazing
cheers
Richard

Try version 1.2.0, we have improved the way of configuring the imp/kWh pulse rate. From now on you fill this value at the top under substitutions 😉


MicrobitLog HTML updates

build tooling to minimise HTML and update C++
iframe-based reload
fix injection issue with innerHTML (but the comment is still problematic, see #108)
some styling
button and title case fixes

Steps towards #110 but no "online" mode yet. I'll update this draft PR.
For the moment the button styling is in the offline version, but we only have 30 bytes left, so we'll need to move it to the online version in order to have an online version at all.
From the list of style changes on #110, I've not done "In the table: headers should be center aligned, data right aligned" as I'm not sure I can tell the difference. I could assume the first row should be th? Not sure we've got the bytes either.

If you test with something like:
    for (int i = 0; i < 100; ++i) {
        uBit.log.beginRow();
        uBit.log.logData("index", i);
        uBit.log.logData("a.x", uBit.accelerometer.getX());
        uBit.log.endRow();
        uBit.sleep(1000);
    }

then you can see reloads working as you disconnect/reconnect the micro:bit.

Misc notes on the JavaScript for me to follow up on:

I don't trust the comment approach especially as it seems to contain binary data. I found I had to add charset for it to work reliably but still not confident.
Discussion on #108
The code to copy assumes that we can create TSV from CSV by replacing commas with tabs. Does the CSV writing allow escaped commas to be written out (e.g. quoted) ? If so it'll break them. CSV formats are a bit of a nightmare when it comes to escaping/quoting and consistency between tools.

I understand concerns about the trailer, but the content is controlled. All data is not uncontrolled binary data. The only things we'll see in there are text or binary bytes with a value of 0xFF (unused data) or 0x00 (invalidated data). if it were uncontrolled, I would completely agree with you.
There is no escaping of characters at all, as I didn't want to use that much flash for the JS code. Again, I was going to filter out commas, and double quotes from keys and values...

I went through this with @jaustin earlier and I've implemented all his feedback. @finneyj I think this is ready to review.
The dl.css and dl.js files for the online mode are now on microbit.org.
Tested on Chrome/Safari/FireFox on Mac and Windows 10 Chrome/Edge/IE11 for basic functionality.
I'll look at a log space indicator as a separate PR.
I'll also look more carefully tomorrow at the C++ changes intended to address the comment escape issue and retest the iframe reload on Windows.

Looks great @microbit-matt-hillsdon!

Just tested locally, and it works a charm. I really like the auto page refresh feature when the microbit remounts! Very tasty indeed.
I'm happy to merge as soon as you sign off on the minor comments above. Thanks @microbit-matt-hillsdon!

I think we've got all the input on this now Matt, so let's merge.
Thanks @microbit-matt-hillsdon!


Supporting for stLFR reads

Will EMA support mapping reads from the stLFR (http://dx.doi.org/10.1101/324392), where the uncorrected barcode is at the 3'-end of read2.

Hi @YingYa,
This looks like it should be easy to support during preprocessing. Are there any unique properties of this data type compared to 10x? e.g. what is the barcode length? read length? avg. reads per barcode? If these are at least comparable to 10x then we can likely just add a preprocessing flag for this data type and run EMA as is.

Hi @arshajii ,
There are some properties of the stLFR data:

The raw data were sequenced by BGISEQ-500 with read length 100 (read1) + 100 (read2) + 54 (read2).
The whole barcode is combined by three sub-barcodes (B1, B2, B3) randomly, which is located among the tail 54bp of read2.
There are 1536 sub-barcodes with fixed length of 10bp. Go to 'https://www.biorxiv.org/highwire/filestream/99369/field_highwire_adjunct_files/1/324392-2.xlsx' for more detail.
The tail 54bp of read2 contain three sub-barcodes (B1, B2, B3)  and two other primers (P1, P2). And 54 (position from 101 to 154) = B1 (10bp) + P1 (6bp) + B2 (10bp) + P2 (18bp) + B3 (10bp).
1 mismatch maybe allow for each sub-barcode when processing the data.
avg. reads per barcode would be 40~50 in 100Gb data.

Hi @arshajii ,
What does each parameter mean and I want to create a new PlatformProfile profile in 'src/techs.c' for stLFR?
Thanks


🚀 Feature: Add a --privacy flag for publishConfig > access

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
Publish configs were added in #465 -> #466. It should be that if you initialize a new repository and add the requisite tokens, publishing should work immediately. But, per https://twitter.com/malloc007/status/1693623834509824120, the lack of access: true means packages might not immediately publish well.
Let's add a --privacy flag that defaults to "public". That way users have control over whether the package is public or private, and it's surfaced in the docs.
Additional Info
Thanks @ryota-murakami for speaking up! ❤

@JoshuaKGoldberg Recently, I've gotten used to npm publish and release-it, and I've come to think that this is not an issue at all.
Can I close it?

Sure, works for me!

Ah, I actually hit this today:

I created a new package
I added GitHub and npm tokens
Error: Can't generate provenance for new or private package, you must set access to public.

I'll add this feature in after all. 😄


Fix resendToReportStream migration

Related Issue or Background Info

Per the comment removed in this PR: moving to the queued uploader breaks the resend logic. This is the fix.

Changes Proposed

Drive TestEvents to the queue on request instead of uploading directly to ReportStream

Checklist for Author and Reviewer
UI

[x] Any changes to the UI/UX are approved by design
[x] Any new or updated content (e.g. error messages) are approved by design

Testing

[x] Includes a summary of what a code reviewer should verify

Changes are Backwards Compatible

[x] Database changes are submitted as a separate PR

[x] Any new tables that do not contain PII are accompanied by a GRANT SELECT to the no-PHI user
[x] Any changes to tables that have custom no-PHI views are accompanied by changes to those views
(including re-granting permission to the no-PHI user if need be)
[x] Liquibase rollback has been tested locally using ./gradlew liquibaseRollbackSQL or liquibaseRollback
[x] Each new changeset has a corresponding tag

[x] GraphQL schema changes are backward compatible with older version of the front-end

Security

[x] Changes with security implications have been approved by a security engineer (changes to  authentication, encryption, handling of PII, etc.)
[x] Any dependencies introduced have been vetted and discussed

Cloud

[x] DevOps team has been notified if PR requires ops support
[x] If there are changes that cannot be tested locally, this has been deployed to our Azure test, dev, or pentest environment for verification

For future readers... I wrote "migration" but 100% meant "mutation" 🤦