Local File Link Not Highlighted

VS Code Version: 1.89.1
OS Version: Windows 11

Steps to Reproduce:

Create a .log file
Type a local file link into it

The built-in link detector for code editors is built to only detect file://, http:// or https:// links
The output editors also detect paths (/something/...) or (C:\something...) but we don't turn this on for code editors because of the very high false positive rate in regular programs.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

@alexdima What is the difference between a output editor and a code editor?

I guess the output editor is pretty much one-way, but in the code editor a user could be typing or editing a file path.
You probably wouldn't want it to run any checks on the link while typing, especially if it was doing something like checking if a file existed over a mapped network drive.

I guess the output editor is pretty much one-way, but in the code editor a user could be typing or editing a file path.
You probably wouldn't want it to run any checks on the link while typing, especially if it was doing something like checking if a file existed over a mapped

Is a .txt file an output editor? It should be highlighted if I it is, correct?

The output editor is the one at the bottom:

cc @bpasero


.not.toThrowErrorMatchingInlineSnapshot() doesn't work as expected

Describe the bug
I created a failing test using .not.toThrowErrorMatchingInlineSnapshot() to demonstrate to my colleague the error that they have to address.
However, when they fixed the error - the test still doesn't pass, because vitest returns Error: snapshot function didn't throw for this test.
I expect .not.toThrowErrorMatchingInlineSnapshot() to assert that error was either not thrown, or it was thrown but didn't match the snapshot.
Current behaviour is more like expect().toTrowError.not.matchingInlineSnapshot(), hope this makes sense!
Reproduction
https://stackblitz.com/edit/vitest-dev-vitest-cmmyai?file=test/basic.test.ts
// passes
it("throws", () => {
  expect(() => {
    throw "Bla-bla";
  }).toThrowErrorMatchingInlineSnapshot('"Bla-bla"');
});

// fails with `Error: snapshot function didn't throw`
it("doesn't throw", () => {
  expect(() => {}).not.toThrowErrorMatchingInlineSnapshot('"Bla-bla"');
});

System Info
System:
    OS: Linux 5.15 Ubuntu 22.04.2 LTS 22.04.2 LTS (Jammy Jellyfish)
    CPU: (20) x64 12th Gen Intel(R) Core(TM) i9-12900H
    Memory: 23.12 GB / 31.19 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 18.13.0 - ~/.nvm/versions/node/v18.13.0/bin/node
    npm: 8.19.3 - ~/.nvm/versions/node/v18.13.0/bin/npm
  Browsers:
    Chrome: 111.0.5563.110
  npmPackages:
    @vitejs/plugin-vue: ^3.1.2 => 3.2.0
    @vitejs/plugin-vue-jsx: ^1.3.9 => 1.3.10
    @vitest/coverage-c8: ^0.29.2 => 0.29.2
    vite: ^3.1.8 => 3.2.5
    vitest: ^0.29.2 => 0.29.2

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

After rereading this, I am not sure it makes sense to support "not" operator. What string should it generate then? Just anything except the current value?


WebMedia Position Placement Incorrect with Youtube Music

Not exactly sure why but youtube music seems to return the position with a newline in front. This both moves the time out of it's correct position and causes the substitution for the leading zero to not work.

Thank you for the PR.

Thank you for the PR.


A minor bug in tutorial model

On line 102 of tutorial_models.py https://github.com/tensorflow/cleverhans/blob/master/cleverhans_tutorials/tutorial_models.py
I think it should be output_shape[0] = batch_size

Good catch, this does not seem to affect the code given that input and output shape variables are not used in the ReLU that follows the Conv2D layer. Would you mind looking through the file and making a PR?

No problem :)

Closed by #360


[#294] Add Chucker library

https://github.com/nimblehq/android-templates/issues/294
What happened 👀
Add Chucker library to

Template
Compose sample
XML sample

Insight 📝

Add Chucker library to Template, Compose sample, and XML sample
Use Hilt to provide Chucker interceptor
We will only apply Chucker for debug builds

Proof Of Work 📹
Template
https://user-images.githubusercontent.com/13492460/198258179-1db02efa-e1d9-4912-9346-68201286aa42.mp4
Compose sample
https://user-images.githubusercontent.com/13492460/198258852-0834b4eb-d992-4393-8174-4847c9ecbd53.mp4
XML sample
https://user-images.githubusercontent.com/13492460/198258920-de6f4918-0749-41da-ba59-c1c2c333254c.mp4

@chornerman Nice 👍
However, the CI is failing for some reason... 💭

@chornerman Nice 👍
However, the CI is failing for some reason... 💭

@Tuubz Look good now 🚀


Added Střední škola André Citroëna Boskovice

@JonanekDev Thank you for your patience while waiting for a review and processing your request. Pull request merged. Thank you!


Correcting documentation of position-encoding

Corrected some confusing spelling mistakes, in the position-encoding notebook, in community/en/

@jdduke

Thanks for the contribution but this community/ directory has been deprecated.  We're no longer accepting pull requests here.
Thanks


fix MHC failure on node count reaching supported maximum

Which issue this PR addresses:
Fixes https://issues.redhat.com/browse/ARO-1931
What this PR does / why we need it:
PR provides a fix by implementing a check on nodes count by setting up a watch on Node API in ARO's Machine Health Check Controller.
Existing MHC CRs are deleted if the node count reaches the support maximum value, i.e. 62. Otherwise recreated.
Test plan for issue:

[ ] Create a test cluster
[ ] Change the maxSupportedNodes value from 62 to 6(current count of nodes) in MHC Controller
[ ] Run the local operator
[ ] Check MHC CRs existence

Is there any documentation that needs to be updated for this PR?
N/A

@microsoft-github-policy-service agree [company="redhat"]

@microsoft-github-policy-service agree company="redhat"


Extend Reference functionality

Motivation
Using the sdk I've noticed some code that we end up repeating pretty often around References, like having to create a new reference to request a language variant of a content item that I already have, or checking agaisnt Guid.Empty when I want to filter if the items are in the default language/workflow...
Proposed solution
#267 would be a pre-requisite

[ ] Add a static Default method to Reference class (will return a Reference instance with Guid.Empty id and default codename)
[ ] Implement IEquatable to Reference class
[ ] Add Implicit conversor from sdk models (ContentItemModel, LanguageVariantModel...) to Reference

Additional context
I listed #267 as pre-requisite so we can handle better the compairsons and just throw an Exception if references can't be compared (g.e. one having just codename and the second one just the id)

@pokornyd I can prepare a PR with the #267 changes and whatever of the 3 proposed changes are approved. Just let me know in which ones you see value

hello Diego, pardon the late reply. I'll check with wider team which of your proposed solutions is preferred and get back to you soon, thanks for your patience.

I'd say the best approach is indeed to have a static method to create a default reference, e.g.:
    /// <summary>
    /// Creates the reference by empty guid (for entities with default ID).
    /// </summary>
    public static Reference ByDefaultId() => new() { Id = Guid.Empty };

I'm not sure a similar implementation for default codename is necessary. IDs are immutable, whereas codenames can be modified. all default entities come with an empty ID if I'm not mistaken, so I think above method should be sufficient.
do you have a use case for default codename as well?


styling different pages

In your screencast you mentioned that the html files for the navigation wouldn't include head, div and body tags..so how would be able to style these different pages?

Probably easiest to have a single CSS file in the root page, then use the various CSS classes in the different pages.

That makes sense. The thing is, you mentioned that these .html files are 'partial' and that they shouldn't have html tags such as body & head. If that's the case how can I add more content to these partial .html files, such as divs with images, text etc...sorry if I'm misunderstanding!

You can add divs and nested elements, that should work just fine I think. It's just that there can only be one <head> and <body>, these cannot be nested.

Ok thanks! I'll give a try and let you know how it goes :)
On Wed, Oct 11, 2017 at 4:43 PM, Curran Kelleher<EMAIL_ADDRESS>wrote:

You can add divs and nested elements, that should work just fine I think.
It's just that there can only be one  and , these cannot be
nested.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/curran/screencasts/issues/24#issuecomment-335854507,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANddKUNPjYzOMwKoFqvgTg0AUIX6Wspeks5srOIZgaJpZM4P0p1i
.


fix: bump spectral dependencies

PR summary
This commit bumps our spectral-related dependencies to the most recent versions.
This change also required some adjustments to several unit tests because the ordering of the errors returned by spectral has changed slightly.
PR Checklist
General checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[x] Dependencies have been updated as needed
[ ] .secrets.baseline updated as needed?

Checklist for adding a new validation rule:

[ ] Added new validation rule definition (packages/ruleset/src/rules/*.js, index.js)
[ ] If necessary, added new validation rule implementation (packages/ruleset/src/functions/*.js, updated index.js)
[ ] Added new rule to default configuration (packages/ruleset/src/ibm-oas.js)
[ ] Added tests for new rule (packages/ruleset/test/*.test.js)
[ ] Added docs for new rule (docs/ibm-cloud-rules.md)

:tada: This PR is included in version 1.17.1 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.18.1 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.4.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Allow forcing read from a read-only replica either per request or as a client setting

Is your feature request related to a problem? Please describe.
When configuring a Cosmos account with read-only replicas, it assigns RU to those replica's and you pay for them. It would be good to be able to force read to a replica in order to balance those RU's between the master and the replicas.
Describe the solution you'd like
A setting on the QueryRequestOptions/ItemRequestOptions to force the request to go to a read-only replica, or a client setting that forces all reads to replicas.
Describe alternatives you've considered
Doing it automatically internally when the primary returns 429 is not ideal because it leads to overloading the primary database and creating fights conditions with the write workloads before moving over to a replica.
Additional context
This feature may already exist but I cannot find any information about it?

@jahmai Cosmos SDK already leverages all replica's. Please check the Diagnostics which reflect which replica was used.
Please re-open with more context/scenario with related diagnostics string to further troubleshoot.

@kirankumarkolli Yes, I am aware of that. The feature request is to force reading from replicas instead of letting the SDK do it itself. The rationale being there are some read operations that I never want to consume to RU of the primary. Does that make sense? Is that something the SDK can currently do?

@jahmai are you referring to geo-replicated replicas?
For example:
Application location is EastUS
Primary CosmosDB location is EastUS, and geo-replicated to WestUS
The application can decide to take the additional latency hit to go to the WestUS for certain read operations.
If you are referring to the local replicas what's the reason for needing to force to read from the replicas?

@j82w I am referring to geo-replicated read replicas. There are some operations (like reports) where time is not of the essence but tend to drain a large amount of RUs to complete, which would be ideal to force to read from WestUS instead of competing for more critical operations on EastUS.
I know that if you have multi-write regions enabled you can nominate which region you prefer to send all operations to, but I would like to do the same thing for read-only replicas in other regions for certain operations only. Per request is ideal, but I could just also create a while new client instance with the preferred setting. Is this something already built-in?

The best way to do this is with multiple client instances with preferred region settings. Then the application can decide which client to use.

@j82w Yes I think that's the way it should be done, with CosmosClientOptions.ApplicationPreferredRegions. I will close this ticket and re-open if it doesn't work the way I expect. Thanks!


Bug/auth fixing the environmental variable bug  -- nasir ali

Naming Rules
Name your PR like this: ISSUENUMBER-TITLE-YOURNAME
Description
the env variables have to be accessed by in server only not through react component to make it more secure and safe
Related to
Make sure you include the issue number with a hash sign # so Github can link this PR to the right issue, like this:
Fixes #48
Checklist:

[X] My code follows the style guidelines of this project
[X] I have carefully reviewed my own code
[X] I have commented my code
[ ] I have updated any documentation

❤


Fill in stub files

Detailed Description
Added a lot of stub files based off  the general plan for data pipes, most are empty at the moment though, but do need to be added sometime!
Context

Possible Implementation

Hi @jacobbieker is this still open? Thanks!


ci: Copied recent protobuf changes from hedera-services

ci: Copied recent protobuf changes from hedera-services

Note, this automation should exclude the block folder (really, anything outside services.  Other teams are primary for block streams and other non-services proto files.
For block stream, in particular, the hedera-protobuf repo is primary, and copies should go from protobufs to services.

Note, this automation should exclude the block folder (really, anything outside services). Other teams are primary for block streams and other non-services proto files.
For block stream, in particular, the hedera-protobuf repo is primary, and copies should go from protobufs to services.

I agree with this @mishomihov00, the changes coming from services don't meet the needs of all teams using the protobufs repo. Let's plan to target the specific items that services needs as @jsync-swirlds suggests


Basic Banking System Project

🛠 Issue (Number)

Closes #1586
Description
Hey @TusharKesarwani, I have added the Basic Banking System Project.
Type of change

[ ] 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 not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] I have made this from my own
[x] I have taken help from some online resourses
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] The title of my pull request is a short description of the requested changes.

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

https://github.com/TusharKesarwani/Front-End-Projects/issues/1498


Implement delete line action for iterm

This PR implements delete_line for iterm. Although, I wonder if this should be in generic_terminal.py? I think most terminals have the same keybindings.

That shortcut is actually implemented by readline I think. So ctrl-u could be used by any program using that library, for example bash, zsh, Python REPL, Ruby REPL, fzf.
Given that I might recommend the following instead of this PR:

Making /tags/terminal/readline.py file that defines a tag and implements this action (and potentially others). You'd then add the readline tag to the iterm.talon file.
Change the implementation of this action; ctrl-u deletes backwards from the current position in readline. You'd probably implement it as ctrl-e (go to end of line), ctrl-u (delete backwards from cursor).

I see you force pushed, but FYI no code got updated.

With the latest push, I've created the readline tag, implemented just the delete_line action, and added the tag only to iterm.
I can move more actions into the tag and add it to more applications, but with limited time and limited typing ability, I'd like to do that in follow up PRs instead of this one if that's ok.


Textline Box Files Tesseract 4.0 bad wording?

https://github.com/tesseract-ocr/tesseract/wiki/TrainingTesseract-4.00 states that:

the required format is still the tiff/box file pair, except that the boxes only need to cover a textline instead of individual characters.

https://github.com/tesseract-ocr/tesseract/wiki/Making-Box-Files---4.0 states pretty the same:

The required format for LSTM 4.0alpha is still the tiff/box file pair, except that the boxes only need to cover a textline instead of individual characters. 'Newline' boxes with tab as the character must be inserted between textlines to indicate the end-of-line.

But example box file has individual character bboxes:
T 112 4663 140 4696 0
e 140 4662 160 4686 0
s 163 4662 179 4686 0
s 182 4661 198 4686 0
e 200 4661 220 4685 0
r 221 4662 238 4685 0
a 239 4661 260 4685 0
c 261 4661 281 4685 0
t 281 4661 296 4691 0
  296 4661 311 4696 0
O 311 4661 344 4696 0
C 347 4661 377 4696 0
R 378 4661 414 4695 0
     414 4694 415 4695 0

Does it means that we need to create character bboxes AND textline bboxes?
Thus, suggested wording needs to be something like:
"the required format is still the tiff/box file pair, except that the boxes need to cover a textline in addition to individual characters."?
Or example box file is wrong?

The lstm training does not really need individual char coordinates.
For each char, you can give coordinates of its entire line.

Or example box file is wrong?

Well, it not wrong. Tesseract will accept it.

Multiple formats of box files are accepted by Tesseract4 for LSTM training, though they are different from the one used by Tesseract 3.

text2image generated box file using font files and training text

I 114 4655 120 4691 0
n 127 4655 150 4682 0
f 152 4655 169 4692 0
o 168 4654 193 4682 0
r 197 4654 213 4681 0
m 214 4654 250 4681 0
a 255 4654 280 4681 0
t 282 4654 295 4689 0
i 298 4654 304 4690 0
o 308 4654 333 4681 0
n 337 4654 360 4681 0
  360 4653 378 4691 0
G 378 4653 413 4691 0
r 418 4653 434 4680 0
o 434 4653 459 4680 0
u 463 4653 486 4679 0
p 491 4643 515 4680 0
s 517 4653 540 4680 0
  540 4653 555 4690 0

Generated by tesseract using lstmbox config from image files - each char uses coordinates of its entire line.

I 114 4640 1912 4692 0
n 114 4640 1912 4692 0
f 114 4640 1912 4692 0
o 114 4640 1912 4692 0
r 114 4640 1912 4692 0
m 114 4640 1912 4692 0
a 114 4640 1912 4692 0
t 114 4640 1912 4692 0
i 114 4640 1912 4692 0
o 114 4640 1912 4692 0
n 114 4640 1912 4692 0
  114 4640 1912 4692 0
G 114 4640 1912 4692 0
r 114 4640 1912 4692 0
o 114 4640 1912 4692 0
u 114 4640 1912 4692 0
p 114 4640 1912 4692 0
s 114 4640 1912 4692 0
  114 4640 1912 4692 0

Generated by tesseract using wordstrbox config from image files - Uses Wordstr and text for whole line

WordStr 114 4640 1907 4692 0 #Information Groups for public OPTIONAL, jaundice Proterozoic Have LOCATION
	 1908 4640 1912 4692 0
WordStr 112 4544 2015 4592 0 #mixed, Male By TEXT Cove... ¥ INSTABILITY About WERE Crimson THAT HOPKINS
	 2016 4544 2020 4592 0

Please note that box files generated using makebox config file are OK for Tesseract3 but not for Tesseract4 LSTM training.
I 114 4654 120 4691 0
n 127 4654 150 4682 0
f 152 4654 169 4692 0
o 168 4654 193 4682 0
r 197 4654 213 4682 0
m 214 4654 250 4682 0
a 230 4653 270 4692 0
t 255 4654 280 4682 0
i 282 4653 304 4691 0
o 308 4653 333 4681 0
n 337 4653 360 4681 0
G 378 4653 413 4691 0
r 395 4643 435 4691 0
o 418 4653 434 4681 0
u 434 4653 459 4681 0
p 463 4653 486 4680 0
s 491 4643 540 4681 0

Attached zip file has a sample tif file and the different types of box files for it so that it is easy to see the additional line with TAB using to mark EOL in the box files for Tesseract4.
boxfiles.zip

WordStr box files
create phototest-wordstr.box from phototest.tif
cd tesseract/test/testing
tesseract phototest.tif phototest-wordstr  -l eng --psm 6 wordstrbox

Review and edit phototest-wordstr.box to have the correct text for each line
Save as phototest.box
Use phototest.tif and phototest.box to create phototest.lstmf files
In case a groundtruth text file is available for the image, you can try to automate the edit process.
Remove the OCRed text from the box file
Delete any blank lines from ground truth file
Add blank line after every textline in ground truth file
paste both files together
sed -i -e 's/\([0-9] \#\).*$/\1/g'  phototest-wordstr.box
sed '/^$/d' phototest.gold.txt > phototest-gt.txt
sed -i -e 's/$/\n/g' phototest-gt.txt
paste --delimiters="\0"  phototest-wordstr.box  phototest-gt.txt > phototest.box

Review phototest.box to make sure that the lines match.

@Shreeshrii many thanks for great and detailed explanation!
I''ll put some changes into wiki to clarify this question.

The lstmbox and wordstrbox options have been added recently. Please try them out with your image files.
Thank you for changing the wiki to clarify this.

有这个的说明吗？

@Shreeshrii hey i need small help as i have to train tesseract on my documents.
I have already read some training and i have steps that i can perform.

!tesseract "document.png" "document" -l eng --psm 11 wordstrbox      it will give me line lavel box
correct ocr. copy image file and box file in tessdata  folder

src/training/tesstrain.sh --fonts_dir /usr/share/fonts --lang eng --linedata_only
--noextract_font_properties --langdata_dir ../langdata
--tessdata_dir ./tessdata --output_dir ~/tesstutorial/engtrain

so this is right thing or i have to do anything else?

It will also finetune detection right?

can i finetune on word level like linedata_only?

i am using tesseract 5 and tessedit_do_invert=0 but it still slow. so which steps i can perform at time training will improve time.

@kbrajwani,
Please use the forum for asking questions.

Thank!


Fix pluralisation of OS

The correct pluralisation of OS is OSes.

do you have a source? quick google search result: https://www.reddit.com/r/asklinguistics/comments/hj66gp/what_is_the_plural_from_os_operating_system_is_it/


How can I get selected rows or columns?

Hi,
I am trying to add an option to delete multiple columns or rows at the same time and was wondering how I would get the column selection. In the storybook I found onSelectedColumnsChange, but it seems to not be supported anymore? I also found onGridSelectionChange, but could not figure out how to actually get the info of which columns/rows are selected. It breaks for me, like mentioned in this ticket https://github.com/glideapps/glide-data-grid/issues/515. Unfortunately I do not understand what is meant by

if you set onGridSelectionChange you should also be passing a gridSelection value.

Any help would be appreciated :-)

onGridSelectionChange returns an object with
{
  current: {
    cell: [number, number],
    range: { x: number, y: number, width: number, height: number },
    rangeStack: { x: number, y: number, width: number, height: number }
  },
  rows: CompactSelection,
  columns: CompactSelection
}

Would dig into source code to be more specific.

Ok, thanks. I will have a look.

//Import CompactSelection
import DataEditor, { CompactSelection } from '@glideapps/glide-data-grid';
//Initialize an initial State
const initialSelection = {
columns: CompactSelection.empty(),
rows: CompactSelection.empty(),
};
//create a useState to manage the selection
const [gridSelection, setGridSelection] = React.useState(initialSelection);
//Pass them as props in DataEditor
<DataEditor
{...cellProps}
gridSelection={gridSelection}
onGridSelectionChange={setGridSelection}
/>


Description of Get-MailboxAuditBypassAssociation is misleading

Get-MailboxAuditBypassAssociation actually returns all objects ( those with Audit bypass Enabled and Disabled )
you will need to run something like this to get a list of bypass
Get-MailboxAuditBypassAssociation  -ResultSize unlimited | ? {$_.auditbypassenabled -eq $true}

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

ID: 76a3f050-9e32-afed-3ae0-23fd5c3c8396
Version Independent ID: b0688e71-ccb0-835d-1730-681dece1f699
Content: Get-MailboxAuditBypassAssociation (exchange)
Content Source: exchange/exchange-ps/exchange/Get-MailboxAuditBypassAssociation.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@akhtarmughal-9, confirmed. I updated the topic.
Thanks for the feedback.
Closing


Update Google_MediaPipe2.py

shoulder

shoulder angle

shoulder

Great! Refer to the following link.
Also released an app for Android that includes neck, shoulder, and hip joints.
https://azimuthkorea.blogspot.com/2024/12/poseinsight_11.html


[Solved] A lot of errors

After I add your library to Xcode via cocoa pods, a lot of error immediately happens:
1, Can not find explicit declaration of interface NSObject;
after some search around, this one can be solved by via adding "import "UIKit/UIKit.h" to the .h file generated by photobuf compiler;
2, Your library is NOT built for automatic reference counting!!!
Sorry, this issue can be solved by adding compile flag "-fno-objc-arc" to *.m files generated by the protobuf compiler. In Xcode, select the main project/targets/build phases/compile sources/select the m files/double click under compiler flag/add "-fno-objc-arc" to the popped window.

There is nothing related UI. So should not include UIKit.h
I have no plan to make generate ARC code. ARC code and none ARC code can work together.

On Mar 8, 2015, at 15:45, Qiang Zhang<EMAIL_ADDRESS>wrote:
After I add your library to Xcode via cocoa pods, a lot of error immediately happens:
1, Can not find explicit declaration of interface NSObject;
after some search around, this one can be solved by via adding "import "UIKit/UIKit.h" to the .h file generated by photobuf compiler;
2, Your library is NOT built for automatic reference counting!!!
—
Reply to this email directly or view it on GitHub.

@mingchen that's wired, if I don't use "#import "UIKit/UIKit.h", then there will be errors as I mentioned above.
ARC issue is now solved.

Thank you for non-arc setting. I also update README to note generated code  is non-ARC.

@mingchen you are more than welcomed.


Where is the code?

The repository seems inactive, and there has been no response on Discord either.

Thanks for your comments, and sorry for the late reply. We've been discussing our release plan and further todo items for the past few days. Here is a brief of our plan and motivations:

Refactoring and cleaning code first: since it is now a rather big project integrating toolkits, backend, and frontend, we hope the released version can be a somewhat organized and easy-to-contribute structure.
We have come up with some interesting and useful stuff to add to the package to benefit researchers and developers.
Demo updates: bug fixing and potential new features

We genuinely appreciate all the interest in implementing these usable agents, and that's why we built this project at the very beginning. Thus, we will definitely push toward the release plan.  It is estimated to take 3-4 weeks to refactor and add new features.  Our commitment is to provide clarity on the implementation details of our agents.
Though we can't guarantee the exact release date, we will try to answer every single question about our agent implementations and demos on github issues (kindly see #3), discord, and slack (again, sorry for the delayed reply).  In addition, if certain independent components of the code are ready before others, we will release them in sequence.


Delete consumed message from the queue in the SQS Consume task

Expected Behavior
There is a bug in the io.kestra.plugin.aws.sqs.Consume task. In SQS, the consumed message must be deleted from the queue after being consumed. Atm, this is not happening, leading to a potential of a never-ending loop, or issues that sometimes a new message doesn't trigger an execution when used with SQS trigger task.
you can reproduce it using:
https://github.com/anna-geller/kestra-terraform-examples/blob/main/aws_sqs_tf/sqs.tf
When two messages are sent, i.e., 2 tasks - each sending an SQS message, I would expect that each of these messages would trigger an execution of the flow that has an SQS trigger:
id: sqsPublishMessage
namespace: prod
tasks:
  - id: generateMessage
    type: io.kestra.core.tasks.debugs.Return
    format: "Hi from a Kestra task {{task.id}}"
  - id: publishMessage
    type: io.kestra.plugin.aws.sqs.Publish
    accessKeyId: "{{envs.aws_access_key_id}}"
    secretKeyId: "{{envs.aws_secret_access_key}}"
    region: eu-central-1
    queueUrl: https://sqs.eu-central-1.amazonaws.com/338306982838/kestra
    from:
      data: |
        {{outputs.generateMessage.value}}
  - id: publishMessage2
    type: io.kestra.plugin.aws.sqs.Publish
    accessKeyId: "{{envs.aws_access_key_id}}"
    secretKeyId: "{{envs.aws_secret_access_key}}"
    region: eu-central-1
    queueUrl: https://sqs.eu-central-1.amazonaws.com/338306982838/kestra
    from:
      data: This is a new message to check again

and reacting to those messages:
id: sqsReactToMessage
namespace: prod
tasks:
- id: consumeMessage
  type: io.kestra.plugin.aws.sqs.Consume
  accessKeyId: "{{envs.aws_access_key_id}}"
  secretKeyId: "{{envs.aws_secret_access_key}}"
  region: eu-central-1
  queueUrl: https://sqs.eu-central-1.amazonaws.com/338306982838/kestra
  maxRecords: 1
- id: print
  type: io.kestra.core.tasks.scripts.Bash
  commands:
  - "echo received message stored in {{outputs.consumeMessage.uri}}"
triggers:
- id: sqs
  type: io.kestra.plugin.aws.sqs.Trigger
  description: "to activate it, set disabled to false"
  disabled: false
  queueUrl: https://sqs.eu-central-1.amazonaws.com/338306982838/kestra
  accessKeyId: "{{envs.aws_access_key_id}}"
  secretKeyId: "{{envs.aws_secret_access_key}}"
  region: eu-central-1
  maxRecords: 1

all messages remain in the Queue - they should be deleted as soon as they have been consumed

Actual Behaviour
Apart from that, it would be great if the output would be stored as a variable rather than as a file in internal storage. The message is a small piece of text/metadata so it makes sense to make it available as a variable without the ion file's overhead in internal storage. The UX would benefit from that by making it possible to view the message output and pass it to downstream tasks.
So instead of: {{outputs.consumeMessage.uri}}
Having: {{outputs.consumeMessage.value}} would be beneficial

Steps To Reproduce
No response
Environment Information

Kestra Version: develop-full docker image

Example flow
No response

Apart from that, it would be great if the output would be stored as a variable rather than as a file in internal storage. The message is a small piece of text/metadata so it makes sense to make it available as a variable without the ion file's overhead in internal storage. The UX would benefit from that by making it possible to view the message output and pass it to downstream tasks.

We usually not provide this functionality for message broker, consume and trigger task will consume a list of messages and messages in a broker can be of any size so we strongly encourage user to store them inside our internal storage.
You can open an enhancement request for that, we may revisit this but it would need to add task properties to choose between STORE or FETCH (the same concept we have in a lot of database query-based tasks).


[7354] Implement ITT reform on Register API - part 2

Context
7354-implement-itt-reform-on-register-api
As part of the ITT reform introduce an inclusion validation for trainee training route
Changes proposed in this pull request

Add an inclusion validation for training_route dependant on the ITT reform year

Guidance to review
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

@Reyko I've just noticed that we have this ticket in the backlog to go through the code and do more renaming. So I guess those suggestions are out of scope for this PR.


Add dev environment for argocd plugin

Description:
Created a dev environment for argocd plugin.
Screenshots:
Example application:

How to test:

run yarn install
run yarn start:backstage to start the example application.

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

/cc @BethGriggs @nickboldt


Replicate an entire RUP to a different RAN

There are many areas where there are multiple overlapping agreements for exactly the same area = same pastures, plant communities, monitoring areas, additional requirements and issues. Often the only thing different in these agreements is the client information, usage and schedule (i.e. info pulled from FTA and entered by AH). It would be considerable time savings to staff to be able to export the info they have entered into a RUP to a different RAN. The luxury edition would be being able to select which content they want to copy over. The base model would be just export/import everything currently available and they can delete/add as needed.

@LisaMoore1 What about statuses and status histories? I assume you wouldn't want to copy that over. And what if there's already an RUP in progress, would the copy overwrite it?

@calebissharp only the data (this is basically a glorified cut and paste). I'll ask my SME group what they think is best re. overwrite.  Should have something for you by Tues if that works for you.


error msg when openin up windows wallet

my wallet was working and connected to the network i did test sends of monyx to my wallet they went through fine now i restarted my pc and i get this error message, when i bypass and push ignore the wallet will open i see my tokens but the wallet does not connect to the network please help

debug1.txt

@OgRastaman as one of possible solutions you can try to delete everything in the wallet folder EXCEPT wallet.dat file and start the wallet again.

thank you very much i deleted the files except for the wallet.dat file and i was able to restart my wallet and connection was established in 5 min thank you for the quick response


Change domain in ingress

Motivation
"crypto" subdomain wasn't the best. We're routing based on domains not paths thus updating to:
signup.stage-crypto.worldcoin.dev, other services will be available at hubble.stage-crypto.worldcoin.dev and mint.stage-crypto.worldcoin.dev
Solution
Updating ingress

Codecov Report

Merging #92 (7f21f42) into main (f8feea8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #92   +/-   ##
=======================================
  Coverage   38.69%   38.69%
=======================================
  Files          14       14
  Lines        4660     4660
=======================================
  Hits         1803     1803
  Misses       2857     2857

Flag
Coverage Δ

test
38.69% <ø> (ø)

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

Continue to review full report at Codecov.

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

Closing this one in favor of #93


change yoyo servo mount to torque servo mount

Torque servos are supposed to be mounted in a mount specific to torque servos. This should be changed to preserve the integrity of our wonderful servo friend

This has already been done for both Claw servos.


JSPM: Chartist is not defined

Hey Globegitter,
Great work over here. However I can't seem to use your plugin.
I am currently using Aurelia.IO and using JSPM as the package manager. It uses SystemJS to include the modules.
I have Chartist installed and I tried to install the chartist-plugin-tooltip. When I load the plugin, it says Chartist is not defined in shim.min.js.
I added the shim package override and it does not seem to work as well
<EMAIL_ADDRESS>{
  "main": "dist/chartist-plugin-tooltip",
  "registry": "github",
  "shim": {
    "dist/chartist-plugin-tooltip": {
      "deps": ["chartist"]
    }
  }
}

Any idea if I'm doing anything wrong over here?
Would really want to use your plugin for my project.
Thanks!
ZenVisuals

@zenvisuals Unfortunately have no experience with JSPM whatsoever. Would love to help!

@Globegitter Just realised I pulled the wrong or outdated repo from JSPM, the version I pulled from didn't had reference to Chartist during AMD loading as you can see from here:
if (typeof define === 'function' && define.amd) {
    // AMD. Register as an anonymous module.
    define([], function () {
      return (root.returnExportsGlobal = factory());
    });
  } else if (typeof exports === 'object') {
    // Node. Does not work with strict CommonJS, but
    // only CommonJS-like enviroments that support module.exports,
    // like Node.
    module.exports = factory();
  } else {
    root['Chartist.plugins.tooltips'] = factory();
  }

I referred to this issue: https://github.com/gionkunz/chartist-plugin-accessibility/issues/1
and figured out the solution but realised that the newer version of this repo had the change already.
Thanks for your message anyway :+1:


API not available?

@crubier This works for me. Are you able to provide the referer URL from Chrome/Firefox developer console?

Ok got it, I was using Brave Browser with Shield on. This cause some Auth problems with some websites. Can close, sorry

@crubier No worrries. I'm going to leave this open while I investigste this further. You're not the first to have this problem. I'll update this issue when I have some more information. Thank you for taking the time to post it too.

OK, so Brave blocks cross-site cookies but www.jsonschema.net uses an API hosted on https://json.systems and so the cookie is blocked. Both the frontend and backend are developed by me, it's just that they're quite separate (hence the differen domains).

This should now work in Brave. I tested with Version 1.11.104 Chromium: 84.0.4147.105 (Official Build) unknown (64-bit).


Alexa Sound Library files for AVS - File sizes are pretty much the same, docs state something different

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:

I have noticed the following statement in the documentation :

you will notice folders filled with audio clips of various size/quality - if your shipping device has limited memory, you might want to choose the "sml" folder to save space.

The thing is checking the files themselves it does not reflect such statement, so for that, are the files wrong or the docs are?
What is the expected behavior?

Consistency between what is stated in the documentation and the actual files. Just clarifying if the files should actually differ in quality and size (all of them) or not.
What behavior are you observing?

I have checked a few files with different tools avprobe and sox I have not noticed a real difference.
I have noticed a file size change only in one file *_ui_wakesound.wav. What about the rest?
$ sox --i ful/ful_ui_wakesound.wav

Input File     : 'ful/ful_ui_wakesound.wav'
Channels       : 2
Sample Rate    : 48000
Precision      : 16-bit
Duration       : 00:00:00.65 = 31065 samples ~ 48.5391 CDDA sectors
File Size      : 124k
Bit Rate       : 1.54M
Sample Encoding: 16-bit Signed Integer PCM

pi@raspberrypi:~/Documents/Alexa Sound Library for AVS $ sox --i med/med_ui_wakesound.wav

Input File     : 'med/med_ui_wakesound.wav'
Channels       : 2
Sample Rate    : 48000
Precision      : 16-bit
Duration       : 00:00:00.65 = 31065 samples ~ 48.5391 CDDA sectors
File Size      : 124k
Bit Rate       : 1.54M
Sample Encoding: 16-bit Signed Integer PCM

pi@raspberrypi:~/Documents/Alexa Sound Library for AVS $ sox --i sml/sml_ui_wakesound.wav

Input File     : 'sml/sml_ui_wakesound.wav'
Channels       : 2
Sample Rate    : 48000
Precision      : 16-bit
Duration       : 00:00:00.47 = 22653 samples ~ 35.3953 CDDA sectors
File Size      : 90.7k
Bit Rate       : 1.54M
Sample Encoding: 16-bit Signed Integer PCM

The same output for avprobe and file.
$ file ful/ful_ui_wakesound.wav med/med_ui_wakesound.wav sml/sml_ui_wakesound.wav
ful/ful_ui_wakesound.wav: RIFF (little-endian) data, WAVE audio, Microsoft PCM, 16 bit, stereo 48000 Hz
med/med_ui_wakesound.wav: RIFF (little-endian) data, WAVE audio, Microsoft PCM, 16 bit, stereo 48000 Hz
sml/sml_ui_wakesound.wav: RIFF (little-endian) data, WAVE audio, Microsoft PCM, 16 bit, stereo 48000 Hz

Provide the steps to reproduce the issue, if applicable:

Tell us about your environment:

What version of the AVS Device SDK are you using?
  <1.11>

Tell us what hardware you're using:

[ ] Desktop / Laptop
[X] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[ ] Linux
[ ] MacOS
[X] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

Thank you very much in advanced

Hello @kjkh
No. I am not having any problems caused by that. In any case I can use ffmpeg to convert it. I just wanted to bring this to AVS team's attention. Thank you for the support. I can help fixing the documentation and/or the files.


diagnostic-channel-publisher to support "pg-copy-streams"

When executing a pg_copy_stream:
        const pool = new Pool(config.database_config);
        const copyFrom = require('pg-copy-streams').from;
        const { Readable } = require('stream');
        const createCsvWriter = require('csv-writer').createArrayCsvStringifier
        [...]
        var target = db.query(copyFrom(`COPY temptable (a,b,c) FROM STDIN DELIMITER ';' CSV HEADER':''}`));
        var source = Readable.from(await csvWriter.stringifyRecords(table_data));
        source.pipe(target);

The target variable is not of type Promise it is of type CopyStreamQuery extending Writeable. This of course ends in the error:
TypeError: queryResult.then is not a function
    at Client.query (D:\home\site\wwwroot\node_modules\diagnostic-channel-publishers\dist\src\pg.pub.js:245:18)

versions:
    "applicationinsights": "^1.7.5",
    "pg": "^8.2.1",
    "csv-writer": "^1.6.0",
    "pg-copy-streams": "^5.1.0"

"applicationinsights" pull dependency to:
        "diagnostic-channel": "0.2.0",
        "diagnostic-channel-publishers": "^0.3.4"

This seems like a dupe of #67. I'll look into this today


Selection time frame

Hi all - We should make the time frame of all the selection graphs and the plot of s estimates focused on current trends.  My suggestion would be to have a four-month cut-off for the data (i.e., include only data within four months of the most recent data), so this updates automatically with each update.  Thanks, Sally

time frame is updated :
as.Date(VirusSeq_release)-days(120)


WIP: read/write epochs

WIP: Reading epochs in Matlab works, writing function and comparison with MNE-Python to follow.

Hi, I'm struggling to read MNE epochs in fif format in matlab using ft_read_data()
I replaced the contents of external/mne in fieldtrip with the latest version of this repo...
`Reference to non-existent field 'FIFFB_MNE_EPOCHS'.
Error in fiff_read_epochs (line 43)
ep = fiff_dir_tree_find(meas, FIFF.FIFFB_MNE_EPOCHS);
Error in ft_read_header (line 1697)
epochs = fiff_read_epochs(filename);
Error in ft_read_data (line 200)
hdr = ft_read_header(filename, 'headerformat',
headerformat, 'chanindx', chanindx, 'checkmaxfilter',
checkmaxfilter);`
Do you have any idea what could have gone wrong?

with what version of mne did you save the file?

Sorry @agramfort, I actually had another fieldtrip version in my path. All solved now/deleted my comment. :)
FWIW, I don't think the latest fieldtrip version inlcudes your latest commits to this repo, I had to manually replace fieldtrip/external/mne (maybe @robertoostenveld will be interested?)

I think it does. Please close if it the case.


[BUG] GitHub Action fails, despite valid title and description

Describe the bug
The GitHub Action "Check title and description" fails, for example in this PR, even though the title and description both fulfill the requirements
Where
PRs
Additional context
I've already asked around and it seems that we are running an older version of the action: mristin/opinionated-commit-message#138
I therefore suggest updating the CI.

Yes, this still applies. The newest version is v3.1.1 and we're running v3.0.0.

I've created a PR (#496) that updates the version.


Please provide BibTeX citation information

Same here. Do you have a technical report summarizing this work? Thanks.

It should be released in one month, please wait for a moment, thanks!


Unable to find start pattern

Hi there. I wanna report this issue.
pi@raspberrypi ~ $ python
Python 2.7.3 (default, Mar 18 2014, 05:13:23)
[GCC 4.6.3] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> from pytube import YouTube
>>> from pprint import pprint
>>> yt = YouTube("http://www.youtube.com/watch?v=Ik-RsDGPI5Y")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python2.7/dist-packages/pytube/api.py", line 66, in __init__
    self.from_url(url)
  File "/usr/local/lib/python2.7/dist-packages/pytube/api.py", line 158, in from_url
    video_data = self.get_video_data()
  File "/usr/local/lib/python2.7/dist-packages/pytube/api.py", line 269, in get_video_data
    json_object = self._get_json_data(html)
  File "/usr/local/lib/python2.7/dist-packages/pytube/api.py", line 317, in _get_json_data
    raise PytubeError("Unable to find start pattern.")
pytube.exceptions.PytubeError: Unable to find start pattern.
>>>

This happened with others videos too.
Cheers.

Please don't consider this issue. Long time ago, I put this line in my host:
<IP_ADDRESS>	www.youtube.com

Close this issue
Cheers.

No youtube browsing at work! :D

The truth is I have a raspberry pi in my house and through port forwarding I share internet at work. But I do not want to occupy all the bandwidth, so I put the Youtube line in host. hahaa all for the damned firewall.

:tanabata_tree:


Optimize ConditionalRequest to require only process id + logical time

Currently the ConditionalRequest requires a full VectorTime, which scales with the number of represented processes. From my understanding the conditional request should depend on the logical time of a single process only, so that ConditionalRequest could require only a String + Long - independently from the number of involved processes.
If transported via an http etag (or some other header), today the http header would be unnecessarily burdened.  This would be a real issue if the server configured header length gets exceeded, if not it's just inefficient.
Not sure if the optimization is worth it, depends on the use case.

ConditionalRequest actually requires the full vector time as it should work across locations (replication endpoints).

Nevermind my previous (meanwhile deleted) comment. +1 on the proposed change.

Discussed this with Martin:
There might be use cases where a conditional request depends on two events from different logs - so that more than only a single / the local sequence number would be needed.
Even if the application / conditional request depends only on the local sequence number, this is still possible with the current api: a VectorTime containing only the local processId + sequence number could be used.
Therefore the proposed change should not be done.


Provide callback for when LSP is setup

When settings up a server, sometimes one also wants to run some lua to update their neovim environment. This can be supported by allowing the config for a language server to be a table, or a function which returns a table, and is passed some context. This would provided much greater flexibility to the loading of servers.

Could you provide a concrete example which specific language servers are you using and what are you trying to achieve?

Sure. This is the functionality i would like to be able to use:
servers = {
  rust_analyzer = {
    settings = {
      ['rust-analyzer'] = {
        checkOnSave = {
          command = 'clippy',
        },
      }
    }
  },
  lua_ls = function()
    require('neodev').setup {}
    return {}
  end,
}

Thanks for the example, it is always good to have a concrete use case at hand. I am trying to keep number of available options at minimum, trying to see if it is possible to use existing options first. For example many use cases can be achieved using existing lspconfig hooks like on_new_config.
In the case of neodev, it mentions in the README it needs to be setup before lspconfig is loaded.
I am not quite sure if it would even work the callback setup, because it might be too late since lazy-lsp would require the lspconfig internally by then.
My suggestion would be to try to configure it like:
require('neodev').setup { ... }
require('lazy-lsp').setup { ... }

I am going to close the issue for now, we can revisit it in the future if a more compelling use case or reason comes up.


[Kotlin] Add an operator overload to apply a qualifier to a column.

This gets us close to natural SQL syntax for qualifiers.

Coverage remained the same at 100.0% when pulling a3a9cf731d9e20c3285d7e048b1537170f41d4d5 on jeffgbutler:better-column-qualifier into 7e908201d8fbedf11dd8e4678bb20d5312054841 on mybatis:master.


Make Winograd RxS f2x3 non-tunable for n_groups == 1

Disables auto-tuning for ConvBinWinogradRxSf2x3 for non-group convolutions. Why:

Auto-tuning for n_groups == 1 is useless but costs some wall time (host time is wasted for reading perf-db)
This above spoiling the performance of the results of "Immediate Mode Fallback improvements using GetWti() #410".

[Note] The intermediate implementation of GetWti() for ConvBinWinogradRxSf2x3 and GEMM can be found at the getwti-wino2111 branch.

Implementation details:

The new non-tunable solver, ConvBinWinogradRxSf2x3g1, is introduced. It handles non-group convolutions.
Existing ConvBinWinogradRxSf2x3 solver made applicable for group convolutions only.

This ensures backward compatibility of perf-db.

High priority as this blocks Initial Network Time Improvements work.
Find-db needs to be updated to avoid perf drops in Fast/Hybrid Find modes. Perf-db can be left as is (there will be some unused records, but this is almost harmless and can wait until next tuning round).

@daniellowell All CI tests passed, ready for merging.

Ready for merging.


Calculate the background color from image

Example:

refer to
https://www.npmjs.com/package/fast-average-color


chore(4.4): release 1.7.1

:robot: I have created a release beep boop
1.7.1 (2022-05-09)
Bug Fixes

pass spanner emulator env (1565347)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/GluuFederation/cloud-native-edition/releases/tag/v1.7.1 :sunflower:


Automatically detect Location based on Descriptor for a problem

The documentation for Problem.Descriptor states that if Location is unset, we will use the location of the descriptor to set it automatically.
However, I don't believe we're currently doing this.

Pretty sure this is working; the tests definitely use the location line values that I expect.


Add missing country code from ISO_3166-1 Alpha 2 specification

Your implementation misses some countries specified in the ISO-3166-1 Alpha 2 document.
Cf. Support request _00D24JOOb._500241wtIaP

Hello @MaximeGaudin,
The two codes can't be considered as real ISO_3166-1 Alpha 2 compliant ones.
They are called "Reserved" ones (and these two are "Exceptionnal Reserved" categorized ones), which means they can be used for specific and/or limited uses.
They are not unknown and are recognized by ISO_3166-1 Alpha 2, but not qualified to be a part of it.
We can't accept this add, because of this, and because we're not supporting this code actually on our API.


Add support for cross-modality metadata

Do we have an idea of how cross-modal metadata (e.g. timings) will be stored? Will they be dumped in an extra directory, or do we want to try and store this information in a standardised way?

From my quick look through the docs, this stands out as a large potential gap of the standard. We have found that data is commonly recorded from 2 or even 4-5 independent acquisition systems that need to be aligned. These different streams can have different sampling rates, starting times, and can drift between one another. All three of these cases really need to be accounted for because each one comes up often. Here is how we handle this when converting to NWB. I would guess that since you are planning on storing the original raw data files you would use a different solution.

Thanks a lot @bendichter, this is definitely something we should think about sooner rather than later. That is a useful link you sent, I will send that to some researchers as the alignment is always very tricky. It will be good to leverage existing tools for this as much as possible. @niksirbi @adamltyson maybe we can run a survey to see how people are storing / dealing with their TTL pulses etc for their behavioural / ephys alignment?

@JoeZiminski sounds good. This isn't my area of expertise, but my feeling has always been that it will involve creating some metadata format. For downstream derivative data, then I think NWB is probably the way, but that doesn't help if we want to support raw data formats.

I suppose another possible solution would be to have sidecar files for each secondary stream to hold timing information. You'd need to communicate what stream is acting as the primary clock, and either a starting time wrt that clock (in the case where you only record offset) or a full timestamp array wrt that clock (e.g. as a .npy file).

Will close in favour of #30 as discussion will envelop all forms of metadata


Update font-awesome-rails from <IP_ADDRESS> to <IP_ADDRESS>

Coverage remained the same at 83.424% when pulling c8589a083b980911882ff5220beaf2b810143c4b on update-gems into 0f9378006b08ad3dc5f50207284762dbf2e3725c on master.

Coverage remained the same at 83.424% when pulling 418daa6c593afe1d98cf54c0dc9ed46689bccbdd on update-gems into 0f9378006b08ad3dc5f50207284762dbf2e3725c on master.


Ablation on LORA

Have you guys tested finetuning the whole llama decoder for the finetuning stage instead of using LoRA? Curious what findings or insights y'all might have there, since I didn't see it included in the paper.

We haven't done that yet. You can use the script train_it_wo_lora.sh to finetuning the whole LLM. Currently, we are working hard on the next version. Once we have done, we will release the checkpoint. Stay Tuned.

Hi, we have release the finetuned checkpoint for your reference!


Invalid Sentry DSN when using a public key

Hi,
Perhaps i do't correctly understand the notion of public DSN. When I try to use Sentry with public I got this exception:
    self._client = raven.Client(CrashReport.DSN, release=__version__)
  File "/site-packages/raven/base.py", line 139, in __init__
    self.set_dsn(dsn, **options)
  File "site-packages/raven/base.py", line 195, in set_dsn
    dsn_config = raven.load(dsn, transport_registry=self._registry)
  File "site-packages/raven/conf/__init__.py", line 54, in load
    scope_extras = transport_registry.compute_scope(url, scope)
  File "site-packages/raven/transport/registry.py", line 73, in compute_scope
    return transport.compute_scope(url, scope)
  File "python3.4/site-packages/raven/transport/base.py", line 57, in compute_scope
    raise ValueError('Invalid Sentry DSN: %r' % url.geturl())
ValueError: Invalid Sentry DSN<EMAIL_ADDRESS>

Is it an issue?
Can we distribute a desktop application including sentry private key without security issue (i don't understand what are the exact access level of an API key)?

Thanks for the help

Unless you're a browser-based implementation you should use the full DSN.
The security implication is only that someone can send events on your behalf, which your only resolve is to revoke the API key and ship a new one. There's no other access given by default with client keys.

Not wanting to reopen this - but have a question:
Could we not implement similar auth that we have in client JS to the server / app implementations?
Our use case is having an open source application and we are trying our best not to have secret config anywhere.

There is no alternative that is secure. You just need to expose the secret and be willing to revoke it upon abuse.

@dcramer I'd like to clarify this statement:

The security implication is only that someone can send events on your behalf.

Does this mean that events can be sent only to Sentry, or that the API key can be used to make Sentry send events to somewhere else?
Thanks for your time.

It’s only for sending events into Sentry.

I would also like to see the ability to use the public DSN with raven. I am deploying Python code to clients and would prefer to not have the full DSN in an untrusted environment. This would be conceptually the same as how I don't want the private DSN in the web bundle I send to users.

Heads up we are removing the secret key (its already in progress) from all SDKs. There's not enough security provided by it, and given the nature of us supporting non-server-to-server communication these days its just not worth it.

If someone abuses the secret (whether it's the full DSN or the public version) — you have to revoke it. This will require all of the consumers to be updated. Common sense here, in terms of security, is that DSN must be disposable.

@dcramer ok, can we get a blog post on that? I don't fully understand the reasoning and I am very interested in the subject.

@ermik it simply provides no additional security. You can have a DSN with only a public key, which is still very hard to brute force, and use that only for server-to-server communication. You can then generate a separate DSN to ship with client software, and as you mentioned, revoke when abused (or just block IPs using our tools).


clean up documentation for release

Description
Documentation clean up in preparation for release. Also, removing unused properties of resources.
Issues Resolved
closes #346
Contribution Check List

[ ] All tests pass.
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable

          1 Warning

:warning:
This Pull Request is probably missing tests.

  Generated by :no_entry_sign: Danger


ginac: 1.7.5 -> 1.7.6

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/ginac/versions.
meta.description for ginac is: '"GiNaC is Not a CAS"'.

Checks done (click to expand)

built on NixOS
/bin/sh passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 1.7.6 with grep in /nix/store/c2cikwhqldy2hv5cqzi55kq5vnz91k8l-ginac-1.7.6
directory tree listing: https://gist.github.com/3469550db1488a32a7de3212d73a2294
du listing: https://gist.github.com/654e2d90e7bf5af1396e6d3582e56323

Rebuild report (if merged into master) (click to expand)

7 total rebuild path(s)
2 package rebuild(s)
2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
ginac
i-score

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/c2cikwhqldy2hv5cqzi55kq5vnz91k8l-ginac-1.7.6 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A ginac https://github.com/r-ryantm/nixpkgs/archive/1d663a6b2e2d67b910a0a374958c1be5d699e38e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/c2cikwhqldy2hv5cqzi55kq5vnz91k8l-ginac-1.7.6
ls -la /nix/store/c2cikwhqldy2hv5cqzi55kq5vnz91k8l-ginac-1.7.6/bin

cc @lovek323 for testing.

This pull request has been mentioned on Nix community. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/12


Expose count_workflow_executions on the temporal client

Summary
This change exposes a count_workflow_executions method on the Temporal client so that users can invoke it using code such as:
client = Temporal::Client.new(config)
result = client.count_workflow_executions(namespace: 'ruby-samples', query: 'WorkflowType="TestingWorkflow"')

Test Plan
Unit test:
bundle exec rspec spec/unit/lib/temporal/client_spec.rb:1106

@DeRauk - Addressed your feedback to return the count value directly instead of as a wrapped object.

@DeRauk - it looks like the GitHub actions for this project sets up a temporal-server instance which does not use advanced visibility.
Ultimately, I think it might just be worth removing the example spec for count_workflow_executions since the spec relies on a visibility backend store, and it is also reliant on timing since the data push to the visibility store happens async. The spec would've been nice to have, but I don't believe it's critical, so I've removed it.

Thank you!


Optimize input to dask.delayed

[x] Closes #7587
[ ] Tests added / passed
[x] Passes black dask / flake8 dask / isort dask

I am not sure if this is a good idea or not, but it seems like it wouldn't do any harm.

@jrbourbeau can you give me a :+1: or a :-1: on whether this is a good idea?

This might be an issue, we've be trying to move all the optimizations passes to the scheduler, this would for the optimization to happen client side, correct ?
@madsbk would you have time to comment here ?

This might be an issue, we've be trying to move all the optimizations passes to the scheduler, this would for the optimization to happen client side, correct ?

Yes, ideally we want to avoid materializing high level graphs on the client, which this kind of optimization might do.
What about moving the graph optimization to optimize(): https://github.com/dask/dask/blob/main/dask/delayed.py#L466?
Isn't optimize() called in this case?

Yeah actually we use to do it this way and then moved all optimization the local and distributed scheduler. In general optimization can be kind of expensive so it is good to delay it as much possible
Plus it's always possible keys that would have been pruned in an optimization may get used later as part of a later computation. Keeping them in the graph avoids having to merge 2 graphs together later by just having everything in one big graph

What's the right place for this change, then?
I've noticed scheduler memory grows substantially when large graphs are received.  Does deferring optimization exacerbate this?
If I do a visualize(..., optimize_graph=True), will I always get the same optimization as on the distributed scheduler?
I ask these questions, since I'm on a mission to make my graphs smaller, and I had assumed I was shipping the optimized version to the scheduler and should be concentrating on the optimized view.

Yeah sorry @chrisroat I didn't understand the vision fully until I went to that talk (link to recording). I'm going to close this PR now, since this is not the recommended approach.

Not sure what all functionality is of interest, Chris, but Genevieve started working on an HLG implementation of map_overlap ( https://github.com/dask/community/issues/159 ), which may be of interest

Not sure what all functionality is of interest, Chris, but Genevieve started working on an HLG implementation of map_overlap ( https://github.com/dask/dask/pull/7595 ), which may be of interest

Thanks for the the information on the ongoing work and the video, @jakirkham @jsignell.  It's helping me get my head around what is coming.
The functionality of interest is avoiding the non-optimization that occurs when using delayed objects.  The reason for this PR was because using a delayed object seems to block optimization of the graph prior to the object - see the linked issue (#7587).
What you all indicate is this optimization this can be re-established in the HLG code somewhere?

Right, my understanding is that the objects passed into a delayed function would have a __dask_optimize__ method on them and the scheduler would call that method before materializing the low-level task graph. But @jakirkham feel free to correct me :)


[BUG] OpenSearch-Dashboards integTest does not uninstall OpenSearch after completion

Hi,
When running the OSD integTest through Python, both OS and OSD will be installed.
But when completing the test or failed the test, only OSD will be uninstalled.
The logic here should be finetuned, this is only for DEB/RPM, not for TAR due to how uninstall function is being written:
https://github.com/opensearch-project/opensearch-build/blob/main/src/test_workflow/integ_test/distribution_rpm.py#L56
https://github.com/opensearch-project/opensearch-build/blob/main/src/test_workflow/integ_test/distribution_deb.py#L54
Thanks.

We might able to retrieve the above service list and run the uninstall based on that.
In src/test_workflow/test_cluster.py

[Triage] We have enough information here. @Divyaasm Please help take a look here. Thanks.


Add support for 2-pass ledgers in the application logic

This PR injects the second-pass ledger at the end of the fee-payer's segment, to ensure that it's ready and waiting for the remaining account updates in a zkApp command.
This ensures that the relationship between the initial global_state.second_pass_ledger and the local_state.ledger is asserted in the snark.
Checklist:

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me


Add git branch to version package

Signed-off-by: Iaroslav Ciupin<EMAIL_ADDRESS>Read then delete this section
- Make sure to use a concise title for the pull-request.
- Use #patch, #minor or #major in the pull-request title to bump the corresponding version. Otherwise, the patch version
will be bumped. More details
TL;DR
Please replace this text with a description of what this PR accomplishes.
Type

[ ] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[ ] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
Remove the 'fixes' keyword if there will be multiple PRs to fix the linked issue
fixes https://github.com/flyteorg/flyte/issues/
Follow-up issue
NA
OR
https://github.com/flyteorg/flyte/issues/

LGTM
@eapolinario @hamersaw @EngHabu can one of you approve for open source?


Support resource requests for staging jobs

Fix #2286
Ref https://github.com/epinio/helm-charts/pull/439
feat:  staging job resource request support

Testing that the new facility operates as desired:

Install E with --set server.stagingResourceRequests.cpu=100m --set server.stagingResourceRequests.memory=100M
Use kubectl get pod -o yaml ....  to look at the YAML of the epinio-server pod and check that the STAGING_RESOURCE_... environment variables are set to the requested values.
Push an application and inspect the stage job pod to also have these values

Example server data
    - name: STAGING_RESOURCE_CPU
      value: 100m
    - name: STAGING_RESOURCE_MEMORY
      value: 100M

Example job data
spec:
  containers:
    - [...]
      resources:
        requests:
          cpu: 100m
          memory: 100M

Verified in epinio v1.9.0-rc3 according to the report above - all good


fix: update time range select tooltip

SUMMARY
Updates time range select tooltip in Explore to provide more clear examples of smart date parsing
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before

After

TEST PLAN
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Side-note, eventually it would be great to show the user what their string evaluates to, or whether it's actually working.


[Suggestion]: Allow Volume Change in Config File

General Info

[ ] I would like to propose new content
[X] I would like to propose changes
[ ] I would like to propose another thing

Suggestion Description
Currently, there isn't a great way to lower the volume for Daily Dad without turning the master volume down. Is it possible we could have a volume setting in the config to lower it a bit? (It's quite a bit louder than everything else in the game right now for me)
Thank you! I love you <3
Darksosoto
MC: 1.18.2
Forge: 40.1.71
DailyDad: 1.3.2

I've looked into it but the Text2Speech library mojang made/uses doesn't have any way of changing the volume

That is sad. Feelsbad


Cannot establish a connection to remote DB

Driver version or jar name
6.4.0.jre8, 6.5.3.jre8-preview are tested
SQL Server version
10.50.4000.0
Client operating system
Ubuntu 18.04
Java/JVM version
java version "1.8.0_171"
Java(TM) SE Runtime Environment (build 1.8.0_171-b11)
openjdk version "1.8.0_171"
OpenJDK Runtime Environment (build 1.8.0_171-8u171-b11-0ubuntu<IP_ADDRESS>-b11)
java version "10.0.1" 2018-04-17
Java(TM) SE Runtime Environment 18.3 (build 10.0.1+10)
Table schema
Any
Problem description
Cannot use JDBC driver with any app on this system
com.microsoft.sqlserver.jdbc.SQLServerException: The driver could not establish a secure connection to SQL Server by using Secure Sockets Layer (SSL) encryption. Error: "SQL Server did not return a response. The connection has been closed. ClientConnectionId:d576e930-9065-4896-90d9-d29c70fe081e".
	at com.microsoft.sqlserver.jdbc.SQLServerConnection.terminate(SQLServerConnection.java:2670)
	at com.microsoft.sqlserver.jdbc.TDSChannel.enableSSL(IOBuffer.java:1837)
	at com.microsoft.sqlserver.jdbc.SQLServerConnection.connectHelper(SQLServerConnection.java:2257)
	at com.microsoft.sqlserver.jdbc.SQLServerConnection.login(SQLServerConnection.java:1921)
	at com.microsoft.sqlserver.jdbc.SQLServerConnection.connectInternal(SQLServerConnection.java:1762)
	at com.microsoft.sqlserver.jdbc.SQLServerConnection.connect(SQLServerConnection.java:1077)
	at com.microsoft.sqlserver.jdbc.SQLServerDriver.connect(SQLServerDriver.java:623)
	at java.sql.DriverManager.getConnection(DriverManager.java:664)
	at java.sql.DriverManager.getConnection(DriverManager.java:270)
	at connectURL.main(connectURL.java:18)
Caused by: java.io.IOException: SQL Server did not return a response. The connection has been closed. ClientConnectionId:d576e930-9065-4896-90d9-d29c70fe081e
	at com.microsoft.sqlserver.jdbc.TDSChannel$SSLHandshakeInputStream.ensureSSLPayload(IOBuffer.java:780)
	at com.microsoft.sqlserver.jdbc.TDSChannel$SSLHandshakeInputStream.readInternal(IOBuffer.java:836)
	at com.microsoft.sqlserver.jdbc.TDSChannel$SSLHandshakeInputStream.read(IOBuffer.java:827)
	at com.microsoft.sqlserver.jdbc.TDSChannel$ProxyInputStream.readInternal(IOBuffer.java:1009)
	at com.microsoft.sqlserver.jdbc.TDSChannel$ProxyInputStream.read(IOBuffer.java:997)
	at sun.security.ssl.InputRecord.readFully(InputRecord.java:465)
	at sun.security.ssl.InputRecord.read(InputRecord.java:503)
	at sun.security.ssl.SSLSocketImpl.readRecord(SSLSocketImpl.java:983)
	at sun.security.ssl.SSLSocketImpl.performInitialHandshake(SSLSocketImpl.java:1385)
	at sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:1413)
	at sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:1397)
	at com.microsoft.sqlserver.jdbc.TDSChannel.enableSSL(IOBuffer.java:1767)
	... 8 more

Expected behavior and actual behavior
Should be able to connect to the database (on Ubuntu 16.04 it was ok)
Repro code
import java.sql.*;

public class connectURL {

   public static void main(String[] args) {

      // Create a variable for the connection string.
      String connectionUrl = "jdbc:sqlserver://***.***.**.**:****;user=*******;password=**********";

      // Declare the JDBC objects.
      Connection con = null;
      Statement stmt = null;
      ResultSet rs = null;

      try {
         // Establish the connection.
         Class.forName("com.microsoft.sqlserver.jdbc.SQLServerDriver");
         con = DriverManager.getConnection(connectionUrl);

         // Create and execute an SQL statement that returns some data.
         String SQL = "SELECT TOP 10 * FROM Person.Contact";
         stmt = con.createStatement();
         rs = stmt.executeQuery(SQL);

         // Iterate through the data in the result set and display it.
         while (rs.next()) {
            System.out.println(rs.getString(4) + " " + rs.getString(6));
         }
      }

      // Handle any errors that may have occurred.
      catch (Exception e) {
         e.printStackTrace();
      }
      finally {
         if (rs != null) try { rs.close(); } catch(Exception e) {}
         if (stmt != null) try { stmt.close(); } catch(Exception e) {}
         if (con != null) try { con.close(); } catch(Exception e) {}
      }
   }
}

Hi @olegbonar, the team is able to use the driver/run tests with Ubuntu18.04. Can you ping the SQL server you want to connect to, and share more information regarding your environment.

@rene-ye, hello, thanks for your reply! I'm using openvpn client to be able to connect to the server and i definitely can ping the server while it's turned on. Also there is no problem connecting using IntelliJ IDEA plugin "Database Tools and SQL" (Version: 181.5087.20) with both ms jdbc driver and jtds driver. By the way i have no trouble using jtds with any app on my laptop, it just works.

Hi @olegbonar, I was successfully able to connect to a database using a fresh copy of Ubuntu18.04 (from their website), java 10, and the sample code provided. Please ensure your connection string follows the format:  "jdbc:sqlserver://databaseURL:port;database=DatabaseName;user=Username;password=Password;". If you are specifying your database target the way jtds does, with databaseURL/databaseName, the driver will not work (I noticed you don't have this property in the code above).

@rene-ye, I edited the connection string exactly like this: jdbc:sqlserver://databaseURL:port;database=DatabaseName;user=Username;password=Password; and tried #414 solution, both did not work, still trying to find what is wrong.

Hi @olegbonar
Can you check this link for troubleshooting SSL Error if it helps?
If nothing works, please enable tracing and provide us trace logs for deeper investigation as mentioned in the link above.

We have a similar issue, the error and the stacktrace are the same.
We're having this issue with JDK8u171, we were not having it with u162.
The error seems to be related to a change in u171:
3DES Cipher Suites Disabled
To improve the strength of SSL/TLS connections, 3DES cipher suites have been disabled in SSL/TLS connections in the JDK via the jdk.tls.disabledAlgorithms Security Property.
(http://www.oracle.com/technetwork/java/javase/8u171-relnotes-4308888.html)
After changing java.security and removing 3DES_EDE_CBC from jdk.certpath.disabledAlgorithms everything works fine. I'm not sure what needs to be changed in SQLServer config to disable 3DES as an SSL scheme though.

@Athou, thanks for your reply, changing the jdk.tls.disabledAlgorithms property helped me.

Commenting how to disable 3DES cipher suites on sqlserver 2003/2005 for others incase they run in to a similar situation where it was not feasible to change the java security config everywhere.
this article lists the ways to disable it. Specifically the section on Triple DES 168, setting DWORD to 0x0.

Hi, I was facing the same issue when using IBM JDK 8 since it was using TLSv1 connecting to a MS SQL Server installed on Win2019 which was enforcing TLSv1.2. I solved the problem by setting this property at JVM level:
-Dcom.ibm.jsse2.overrideDefaultTLS=true. The SQL jdbc driver version used was v12.2 jre8.


Expose assume_sorted as a configurable Picard parameter

While the default pipeline flow should produce sorted input, the assume_sorted parameter should not be hardcoded as true - the modular system allows for the Picard stage to be run independently, and we cannot guarantee that unsorted input will never be encountered.

This is now a configurable input under the [large_cohort.cram_qc][assume_sorted] parameter (the default is true, but can be changed)


Synchronization with high number of labels with long strings

Issue #197:
Switching from GET to POST in fetch calls allows the handling of long labels or large data efficiently. Unlike GET, which has URL length limits, POST carries data in the request body, preventing data truncation and ensuring accurate data transmission. This change resolves issues with long labels in URL queries, eliminating risks tied to URL string parameter limits.
Description of changes:

Updated fetch functionality for templated queries to use the POST method.
Resolved linter and pre-commit issues.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@michaelnchin PR is ready for review.

Thanks @cubeddu ! Requesting a couple more changes per testing of the latest revision rebased on main.
With IAM Auth enabled DB:
Gremlin/OC/SPARQL connection sync queries are failing immediately with AccessDeniedExpection - requests are likely not being SigV4 signed correctly.
{
    "error": {
        "status": 500,
        "message": "\n{\n  \"detailedMessage\": \"Missing Authentication Token\",\n  \"code\": \"AccessDeniedException\",\n  \"requestId\": \"--\"\n}"
    }
}

With non-IAM Auth DB:

On both Gremlin and OC connection sync, the initial summary?mode=detailed endpoint request fails with BadRequestException.

{
    "error": {
        "status": 500,
        "message": "\n{\n  \"detailedMessage\": \"Bad route: /pg/statistics/summary\",\n  \"code\": \"BadRequestException\",\n  \"requestId\": \"--\"\n}"
    }
}

Gremlin sync is able to succeed via the fallback schema retrieval queries, but OC schema queries are terminated with different 500 error.

{
    "error": {
        "status": 500,
        "message": "\n{\n  \"detailedMessage\": \"Missing 'query' parameter for the request\",\n  \"code\": \"MissingParameterException\",\n  \"requestId\": \"--\"\n}"
    }
}

@michaelnchin @charlesivie changes are up, I addressed the issue and converted the signing of the headers to a middleware.


Clarification needed on "conversion is not necessarily lossless"

From the readme.md:
The conversion is not necessarily lossless but it is very convenient.

Is it possible to elaborate on that? In which cases is it losing data?

Nobody has compiled such a list, to my knowledge. Things that pop into mind are:

XML comments do not survive XML->JS->XML conversion.
CDATA content is converted to plain text content if it is the only child of an element (setting dependent).

There are likely more such aspects and, potentially, places where data is ignored or converted undesirably. However, to my knowledge, nobody has done any investigation of the exact limitations of the library. I would be interested to hear of any experiences you may have on this topic, so it can be better documented!

The one I was mainly concerned was the order of children nodes.
Eg, the readme uses the example:
console.log(document.MyRootElement.ElementX[1].toString());

This means that the objects are represented as:
MyRootElement = {
  ElementX: [ /* ... */ ]
}

But that the following:
<MyRootElement>
  <ElementX/>
  <NotElementX/>
  <ElementX/>
</MyRootElement>

Looses the ordering.

Yes, indeed - that is also a good example of this factor.

Closing this. Please reopen if you have further questions or requests on the topic.

@sandersaares

CDATA content is converted to plain text content if it is the only child of an element (setting dependent).

Is this something that is configurable? I need to retain the CDATA code within an element even if it's the only child.


Updated 'toY' to 'fromY'

toY was not appropriate as per the description attribute
The appropriate toY is repeated in line 23

thanks @nimishjn


Add item pubDate to rss.xml based on publish_date metadata.

v0.20.0 introduced an optional publish_date metadata field. I think it would make sense if this was used to add a  element for each item in the rss.xml feed, if provided. What do you think?

Yeah, that makes sense to me! If you wanna make a PR, I'd take a look at it. 👍

Awesome, thanks!


Paused monitors included in uptime percentage on status page

⚠ Please verify that this question has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
I'm not sure if this is a bug or if this is the intended behavior, so I wanted to ask the community. I have paused 2 of the 64 monitors. All 64 monitors are included in the primary status page.
My initial reaction is the two paused monitors should not be included when determining if the service is partially degraded. However, I wanted to get the community's feedback.
📝 Error Message(s) or Log
No response
🐻 Uptime-Kuma Version
1.23.11
💻 Operating System and Arch
Ubuntu 22.04
🌐 Browser
Microsoft Edge 122.0.2365.66
🖥 Deployment Environment

Runtime: Docker 25.0.3
Database: sqlite
Filesystem used to store the database on: ext4 on SSD
number of monitors: 64

For the current behaviour, I would classify this is intended behaviour.
If monitors are paused this means we cannot determine their status => cannot say that everything is up.
https://github.com/louislam/uptime-kuma/blob/574705a4c3041d65b752bbcffd3fb8324c74d832/src/pages/StatusPage.vue#L529-L555

Thank you, CommanderStorm. I'm not sure I agree though.
A better status message would be "Good (2 monitors paused)" with the logic added to the if statement on 541
if (beat.status === PAUSED) {
   return STATUS_PAGE_WITH_PAUSED;
} else if ...
...

hate stale bots, this should not have been closed..

another use case in addition to will-knoll's; i want to be able to list paused, offline servers on the status page as backup capacity
perhaps a better way to go about this is to check if a monitor has a tag with the value of non-critical or similar, and if so, exclude it from this total server status
maintainer pls reopen

hate stale bots

Sorry, but this is the only way we can deal with the level of support help-requests (other requests don't have this automation) we get.

I have stated my opinion above that the current behavior not a bug and intended behavior.
Though, I can see that users might find the current behavior mildly confusing.
=> If someone would provide a PR changing this to be better communicated (... (2 monitors paused)), this would get reviewed and merged if good.
Our contribtuion guide is here: https://github.com/louislam/uptime-kuma/blob/c919d2c990beba0d5d91b28f436b43296b6992b0/CONTRIBUTING.md
The rest of your comment (listing offline servers as backup capacity, tag based status exclusion) is off-topic.
Listing offline servers as backup capacity can be achived by adding a tag to them.

yeah i have the monitors tagged, just wish they wouldn't make the service appear "degraded" when i list them on the status page >m<


Novelupdates Double Chapters

Source information
Novelupdates
Steps to reproduce

Save a novel from novelupdates
Update through updates tab
Check the chapter list of the novel
Done

Expected behavior
Chapters will not double if the library is updated
Actual behavior
When Updating the library, chapters from  the novles of novelupdates will double, sometimes it Doesn’t happen.
LNReader version
2.0.0
Android version
Android 14
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 1.1.19.
[X] I have updated all installed extensions.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Duplicate of #1185


Possible to generate from a serialized file instead of talking with XRM api.

I assume that XrmContext retrieve all the metadata from API. I would like to be able to extract that separately and serialize it to a file format.

What is the actual retrieve calls that XrmContext calls?

and then i would like to call XrmContext --file myserializeddata.xml to generate the context.

How easy would this be to add?

The reason is that i would from within a plugin in an solution do the extract and then post the serialized content to an external service that generates the dll and publish as a nuget - and i dont need to give CRM credentials to the external service that generates the dll.
@magesoe  - is it possible? :)

@pksorensen the flag is technically there (save / load respectively), but save doesn't seem to work. It's done for XRMDT though, so it should be possible.
For future reference:
Parameter added to GenerateCSharpContext extraArguments: "save", "test.tmp"
Error dump:
(2021-05-31 14:39:08Z) [Info]: Retrieving CRM version...Done!

(2021-05-31 14:39:08Z) [Error]: Unable to generate context file: Encountered unexpected namespace 'http://schemas.microsoft.com/xrm/2011/Metadata'. The namespace must be empty.

(2021-05-31 14:39:08Z) [Error]: Parameter name: ns
System.Exception: DG XrmContext failed
at DG.Daxif.Common.Utility.postProcess(Int32 code, String es, String os, ConsoleLogger log, String proc) in C:\git\Daxif\src\Delegate.Daxif\Common\Utility.fs:line 173
at DG.Daxif.Modules.Solution.Main.updateCustomServiceContext(Environment env, String outputDirectory, String pathToExe, LogLevel log, FSharpList1 solutions, FSharpList1 entities, FSharpList`1 extraArgs) in C:\git\Daxif\src\Delegate.Daxif\Modules\Solution\Main.fs:line 312
at <StartupCode$FSI_0007>.$FSI_0007.main@() in C:\git\DG\DemoProject\XrmOrg\XrmOrg.XrmSolution\Tools\Daxif\GenerateCSharpContext.fsx:line 11
Stopped due to error

@Xephyz also played around some stuff related to this.


Configure IP address ranges

Hi,
at one of my client we identified some impossible travel etc. We analyzed the used RemoteIP of the user. The issue is: users are using the client user account, but not using a client-managed devices. These devices are managed by third-party, These third-party devices are using SDP (Software defined Parameter like ZScaler, AppGate, what ever). The result is that the user is switching its RemoteIP across country borders some time. In addition using Mobilephone is adding another source of IP addresses with other locations.
We can't get it sorted for the client.
If we use the following Advanced Hunting:
AlertEvidence
| where Title contains "Impossible travel"
| where isnotempty( RemoteIP)
| summarize RemoteIPcount = count() by  RemoteIP
we  get a list of more then 200 IP-addresses per week, which are count only once.

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

ID: a4e03e45-c169-67b4-b0f1-03597040bf0d
Version Independent ID: 197ab35b-a26a-c186-8949-d1828303ca9f
Content: Detect suspicious user activity with UEBA - Microsoft Defender for Cloud Apps
Content Source: CloudAppSecurityDocs/tutorial-suspicious-activity.md
Service: defender-for-cloud-apps
GitHub Login: @batamig
Microsoft Alias: bagol

Hi @harald101010, thanks for writing into docs! I'm tagging an SME to help advise.
@assafyatziv, can you take a look?

Hi @harald101010
"Impossible travel activity" detection was switched to a new data type called "behavior", and was disabled by default on the 28th of May. One of the reasons is such cases, where although we do use suppressions such as prevent the alert trigger on common ISP, there are use cases where such alerts won't be suppressed for some reasons.
We recommend stop using this detection as alert, and if you say the outcome valuable, consider creating advanced hunting custom detection rules that trigger alerts only in certain use cases, such as ISP does not equal a closed list of ISPs.
More information about behaviors can be found in the documentation: https://learn.microsoft.com/en-us/defender-cloud-apps/behaviors or in this blog: https://aka.ms/mda/behaviors

Thank you. this is exact the same, what we have planned to do our self at the customer: ignore the alerts but still use the information as additional input for further analyses. Indeed, i can think about use cases too, which requires not to have an impossible travel e.g. for high secure areas.

Hi @harald101010 - please let me know if this answers your question and we can close this issue?

Hi @harald101010,
I'm going to close this item now as it seems to have been resolved. If that's not the case, please feel free to re-open the issue and/or continue commenting. Thank you for contributing to docs!
#please-close


Refactor PageHeader component

The PageHeader component was a little difficult to work with previously, due to its inheritance of multiple unrelated style components. This PR introduces styles specifically for the PageHeader that more closely match the design. Its behavior at smaller viewport widths is more consistent as well.
I also decided to change the classname of our page's tabbed navigation, which was previously called page-header. I think this will help clarify the roles of the elements.

Reviewer: @pierlo-upitup

LGTM although it's quite a lot of new CSS!

@pierlo-upitup That's true, but I do think this is better — please let me know if you're opposed!
The classes that we applied to this previously got the PageHeader to match the designs most of the way, but not fully, and there were a ton of classes that were added to get some basic properties (notably the 5+ container-pod variations just to get the grey divider). I also think the media-object classes are being abused in general and I was really delighted to remove them, but I'm happy to rethink this if you're opposed to this solution!

🚢


Icon request: .csslintrc

http://csslint.net
Not sure if they have a logo we could use, but perhaps at the very least a gear icon would be good

Done!


update ParseResourceURL function to support legacy URLs

/assign @bowei

cc/ @yazanri

/ok-to-test

Is it possible to rebase only this commit onto master instead of rgraph-unstable?

yes, created new PR https://github.com/GoogleCloudPlatform/k8s-cloud-provider/pull/151


[release-4.15] OCPBUGS-29586: use 2040 for apiserver svc in IBM provider

This is a manual cherry-pick of https://github.com/openshift/hypershift/pull/3572

/retest

/lgtm

/approve


Adds numArguments and  isInlined for FunctionDecl

Just a single nit.  Feel free to weigh in and replace this with whatever you prefer.  My only request is that it not use abbreviations like num.  C++isms cannot be allowed to breed.

⛵


feat: add support for synchronous generator (max_q_size=0)

This request adds support for max_q_size=0 to fit_generator.
When max_q_size=0, this will prevent the generator from being run in parallel to the model.
This might be useful when the generator actually depends on the current state of the model itself.
An example is when the generator needs to do a forward pass of the model to only yield difficult batches (e.g. hard negative triplet sampling strategy in FaceNet paper).

An example is when the generator needs to do a forward pass of the model to only yield difficult batches

Wouldn't you better off writing your own loop using model.train_on_batch and model.test_on_batch in these cases?

Thanks @fchollet for the feedback.
Yes, I could write my own fit_generator loop.
However, I thought it would be nice to allow both synchronous (max_q_size = 0) and asynchronous (max_q_size > 0) behaviors in the same place; because, even in the case where the generator depends on the model, one might want to allow the use of a slightly outdated model for batch generation, for faster (yet less robust) training.
Also, still in the special case where the generator depends on the model, I wanted to highlight that this new feature (max_q_size=0) has the side-effect of making the training process deterministic (but slower), which is not the case right now (because batch generation is done in parallel in an independent thread). See my old question on Google Groups.
What do you think?

Closing outdated PR. If you still care about the content of the PR, please submit a new PR to master, updated for the Keras 2.0 API.


Add type for jsc.throws

Updated!

I think that type of ctor parameter should be ErrorConstructor rather than typeof Error. I would also adjust the return type of first parameter f to be any rather than void. Since this implies that jsverify code does not care what the function returns. This should also allow for cases when functions returns something other than void.

I think that type of ctor parameter should be ErrorConstructor rather than typeof Error. I

I'm ~80% sure that ErrorConstructor and typeof Error are the same type, but I'm happy to change it if you want. That said, it won't solve the need to cast RangeError under Node; here's a playground link showing that.

I would also adjust the return type of first parameter f to be any rather than void. Since this implies that jsverify code does not care what the function returns. Tis should also allow for cases when functions returns something other than void.

void in Typescript is a bit of a special snowflake, and I'm pretty sure it's actually what we want here. As a return type, void works basically like a top type from a subtyping/assignability perspective, but then it works almost like unknown when the caller tries to do anything with the return value. So using void (or, I guess, unknown) instead of any here is actually the standard way to imply that jsverify doesn't care about the return value. See docs.

under Node

Yea, sorry, my bad. I noticed that node definitions define more things which are not defined in TS definitions for Error and ErrorConstructor
Sorry, my bad about void. In my mind it was backwards. That it should be set to any when return type is ignored.

Is anything else needed to get this merged? There are now tests, and I believe the types are all correct.

lib/string.js: line 22, col 5, 'char' is defined but never used.

The CI fails for a reason

... but seems unrelated to this PR...


Xcode 15: The request of a upload task should not contain a body or a body stream

Xcode 15 gives the below warning when running Atlantis, line 325 in _swizzleURLSessionUploadFromDataWithCompleteHandlerSelector
The request of a upload task should not contain a body or a body stream, use `upload(for:fromFile:)`, `upload(for:from:)`, or supply the body stream through the `urlSession(_:needNewBodyStreamForTask:)` delegate method.

Thanks. Let me investigate it with Xcode 15 Beta

I'm unable to reproduce this issue:

Xcode 15 Beta 7
iOS 17 Beta
Use URLSession.shared.uploadTask(with: request, from: data)
the latest Atlantis version.

and I don't get the warning.

Just wondering:

What iOS version you're using with Xcode 15?
When you get the warning, can you help me to trace back (by setting the Breakpoint) what the URLSession's method is called?

thanks in advance 👍

I've got the same issue with lates Xcode 15 RC and Atlantis installed as SPM

I've tracked it down to NewRelic. When I disable NewRelic from starting, I do not get this error.
So Xcode 15 + NewRelic + Atlantis causes this to appear.
Your original stack trace request

that should be the case I've got new relic too in the project but I can't disable it

that should be the case I've got new relic too in the project but I can't disable it

I cannot disable it either, however, in Production since one would not have Atlantis enabled, this is less of a concern, at least to me. It would still be nice to not have this warning during development.

yeah agree on that. it could stay open but with less priority @NghiaTranUIT

Thanks. Let me investigate Atlantis and New Relic 👍

@yjdv @brandonskane it seems this warning is not from Atlantis. It's from the NewRelic (See the screenshot)
How to reproduce

Setup Xcode 15 + iOS 16
Create new empty iOS project
Add New Relic library -> Get the same warning even though there is no Atlantis framework here.