simple proposal for buffer_id in rawio

A very simple proposal for spikeglxrawio
@h-mayorquin
@zm711

In your example the idea would be that we move the sync channel into a new stream right? So it will keep the buffer but you'll assign a new stream ?

Hey, Sam, what I had in mind last time we discussed was something like this that I think should be less concentrate the disruption in one PR and then we can work piece by piece on the formats:
https://github.com/NeuralEnsemble/python-neo/pull/1545

In your example the idea would be that we move the sync channel into a new stream right? So it will keep the buffer but you'll assign a new stream ?

In this PR the idea to have no change IOs. Only adding this buffer_id stuff in a very neutral way.
The next PR will have some changes for some IOs (plexon, spikeglx, ced) to split actual stream in more streams but with the same buffer.

Hey, Sam, what I had in mind last time we discussed was something like this that I think should be less concentrate the disruption in one PR and then we can work piece by piece on the formats:

Your proposal is cool and easy but I think I prefer to be brave and explicitly add this features everywhere as a necessary implementation for rawios.

Good. We covered in the discussion today:
The buffer will be undefined if:

The stream is divided across multiple files. Example intan's one-file-per-channel
There is an API between neo and the data which obfuscates the data layout. Example plexon2

We decide between two approaches:

Have a strict version where stream and buffer are orthogonal concepts.
Have a "nested-unless-edge-case" approach where the stream headers contain a reference fo their buffer unless the buffer is hard to define (cases 1 and 2 above).

I am weary of not going with the strict apporach but this will faciliate some work on Sam and I could not find a concrete example where this could cause problems. In such a case "practicallity should beat purity" and we will stick to this.

Good. We covered in the discussion today:
The buffer will be undefined if:

I think what I would add to this  so we don't forget that we didn't completely settle on a rigid buffer definition so that we have some flexibility as we implement it. Buffers ought to have same dtype, sampling rate, but something like same file vs same memmap was left in the air

plexon 1 for example is one file, but multiple memmaps.
Sam mentioned thinking about intan header-attached which is one file, one memmap but with the block structure doesn't quite fit with what he was imagining for a buffer (when you hopped off @h-mayorquin).

Thanks for summary of the discussion @h-mayorquin! I think keeping our documentation up will help us when we go back and think about our choices!

Thanks for this summary of internal discussion @h-mayorquin  and @zm711.

@zm711 @h-mayorquin this is ready on my side.

Looks like raw binary signal wasn't done quite right (tests failing). Want to fix that? I'll look over this in parallel.

I will take a look once the tests are passing.

Now plexon2 tests are failing.  He has another PR that we need to review (but I need to check if it is plexon1 or plexon2). Do we want to get his last PR done and then you can finalize this or do you want to finish this first. There are slightly organizational changes occurring that I think will keep leading to test failures. Up to you both from my perspective.

I think with the latest boost to plexon2 once you fix conflicts and merge from main I hope the plexon2 won't give us any troubles.

@zm711 : it would be nice to merge this soon.
So I can continue working on #1513.

scanning through this it looks good. Once tests pass I'll do a final read through, so let's target to merge this by end of weekend. We can ping @h-mayorquin for a read through too, hopefully today if he has time?

If Sam doesn't respond I'll package up your edits into a commit and push them, then once final tests pass we can merge it.
Definitely down for another call at some point to discuss your desire for decoupling more !

merci les amis


Change call to action text for components.

Suggestion: change the text of this button

To "Browse Components".
And in general, where the word "Consult" is used, needs to be rethought.
Scott

Consult simply, isn't used like that. You can say something like, Please consult with your layer. But, you wouldn't say, "consult lawyer". It's hard to explain. 🤷
Scott

Ok, will change to "Browse components", anyone has other ideas comment below

Fair with "Browse components" if it is correct.


Update push UI to match spec

@hanningni please review

@lamagnifica the UI change earlier is adding that "Notifications Enabled" section

:+1:


Correct typo: add missing "n" in "isn't"

Fix a minor typo: "is't" > "isn't".

#sign-off

Thanks, @cw-sexton!


Always record plugin errors that fail the build

Even do this for invalid configuration or missing plugins.

Coverage increased (+0.005%) to 89.944% when pulling 1062278f60af6462ad13f37c1c1aae3b799a884d on nonexistent-plugin-error-message into 8b481ff4583bff6c1646a2f826de7414aefa136c on master.


New device: tuya wifi energy consumption meter

Adds support for Tuya Wifi Energy Consumption Meter. Box labelled as V-WIFI-DL02-ES,
Tuya shows device as "双路电流互感计量器（WiFi版）"

completed modify device name and entity name to follow Home Assistant naming guidelines.
thanks.

Has anyone managed to connect it to Home Assistant?

I have this device and have not been able to get it to connect.   The log file shows:
2024-03-20 14:58:22.721 WARNING (SyncWorker_1) [custom_components.tuya_local.device] Test protocol error 914: Check device key or version
2024-03-20 14:58:37.072 WARNING (SyncWorker_10) [custom_components.tuya_local.device] Test protocol error 914: Check device key or version
2024-03-20 14:58:41.391 WARNING (MainThread) [custom_components.tuya_local.config_flow] Device matches None with quality of 0%. DPS: {"updated_at":<PHONE_NUMBER>.4453688, "101": "request", "102": "idle", "103": "monitor", "105": 0, "106": 0, "107": 1243, "110": "normal", "111": 500, "113": "monitor", "115": 0, "116": 0, "117": 1242, "120": "normal", "121": 500, "123": 321, "124": "cloud_net"}

It seems like the best match for this device is erz04c_energy_meter

Thank you for that.   I'm not sure how to tell tuya-local to use that device profile instead.    I had chosen to install tuya-local because my specific device, "V-WIFI-DL02-ES energy consumption clamp meter"  was itself on the compatibility list.     Nevertheless I am encouraged by your prompt reply.  Thank you.

OK I know this is going to break all the rule of how github is supposed to work, but I just want to help the next person coming along this thread.      I was able to get my "V-WIFI-DL02-ES" to work in Home Assistant with TUYA-LOCAL in the following way:
In the folder "homeassistant/config/custom_components/tuya_local/devices", I modified my local copy of: "tuya_energy_meter.yaml" to the following to match the availability of the datapoints (DPS) in the comment above.
When adding the new device in Home Assistant, I needed to specify "protocol: 3.5", as "AUTO" and 3.3, 3.4 did not work.
Here is my modified and working version of tuya_energy_meter.yaml:
name: Dual energy meter
products:

id: 79a7z01v3n35kytb
name: V-WIFI-DL02-ES
primary_entity:
entity: sensor
name: Total energy
class: energy
dps:

id: 123
name: sensor
type: integer
unit: kWh
class: total_increasing
mapping:

scale: 1000

id: 103
name: Device state A
type: string
mapping:

dps_val: close
value: Close
default: true
dps_val: monitor
value: Monitor
dps_val: working
value: Working
dps_val: warning
value: Warning

id: 113
name: Device state B
type: string
mapping:

dps_val: close
value: Close
default: true
dps_val: monitor
value: Monitor
dps_val: working
value: Working
dps_val: warning
value: Warning

id: 110
name: Power Type A
type: string
mapping:

dps_val: normal
value: Normal
default: true
dps_val: warn
value: Warn

id: 120
name: Power Type B
type: string
mapping:

dps_val: normal
value: Normal
default: true
dps_val: warn
value: Warn

id: 101
name: Sync request
type: string
mapping:

dps_val: idle
value: Idle
default: true
dps_val: request
value: Request

id: 124
name: Net state
type: string
mapping:

dps_val: cloud_net
value: Cloud
default: true
dps_val: local_net
value: Local
dps_val: no_net
value: Offline
secondary_entities:

entity: sensor
name: Power A
category: diagnostic
class: power
dps:

id: 105
name: sensor
type: integer
unit: W
force: true
class: measurement
mapping:

scale: 10

entity: sensor
name: Current A
category: diagnostic
class: current
dps:

id: 106
name: sensor
type: integer
unit: A
class: measurement
force: true
mapping:

scale: 1000

entity: sensor
name: Voltage A
category: diagnostic
class: voltage
dps:

id: 107
name: sensor
type: integer
unit: V
class: measurement
force: true
mapping:

scale: 10

entity: sensor
name: Power B
category: diagnostic
class: power
dps:

id: 115
name: sensor
type: integer
unit: W
force: true
class: measurement
mapping:

scale: 10

entity: sensor
name: Current B
category: diagnostic
class: current
dps:

id: 116
name: sensor
type: integer
unit: A
class: measurement
force: true
mapping:

scale: 1000

entity: sensor
name: Voltage B
category: diagnostic
class: voltage
dps:

id: 117
name: sensor
type: integer
unit: V
class: measurement
force: true
mapping:

scale: 10

entity: button
name: Real-time updates
category: config
dps:

id: 102
name: button
type: string
mapping:

dps_val: idle
value: false
default: true
dps_val: ok
value: true
optional: true

entity: number
name: Alarm power setting A
category: config
dps:

id: 111
name: value
type: integer
unit: W
range:
min: 200
max: 500000
mapping:

step: 100

entity: number
name: Alarm power setting B
category: config
dps:

id: 121
name: value
type: integer
unit: W
range:
min: 200
max: 500000
mapping:

step: 100

Did you try erz04c_energy_meter, which is already a 100% match for the log message you posted above?
And what did you change?  It is too difficult to figure out what might need changing when you just post the whole config like that.

I had tried the erz04_c_energy_meter.     It failed to work.   But your suggestion to look at that helped me make the new YAML that worked.
Only when I wrote the YAML to completely match the DPS (and remove all non-matching) did it finally work.    I'm sorry I dont have the bandwidth in life to learn github; I do understand it has a way to show what changed and what stayed the same.   I sure am thankful for your efforts though.   Your hint to me helped make my useless energy meter work on my system.


Fix Null-Safety error

Fix for when using enableAudioVolumeIndication, null-safety errors are observed at:
https://github.com/AgoraIO/Agora-Flutter-SDK/blob/1b44bcdb534a8acac94daaf1347608b515c9a962/lib/src/events.dart#L1570

@fareesh Thanks for your contribution, this bug had fixed in our dev branch, we will close the PR to avoid conflict, thank you for your contribution again.


UI Theme Heading and Margins

Heading
Margins
Replace home icon
Sidebar border
Font family
Container max-width

Will merge this to be able to review it among all of us.


Unexpected overlapping_sprites behavior

I'm running into some unexpected behavior with overlapping_sprites, and set up a minimal repro here: https://github.com/paulstraw/bzpd-overlapping-sprites/blob/main/src/lib.rs#L106-L108. Specifically:

It panics if no sprites overlap (or if the sprite has no collide rect set).
It leaks memory when other sprites do overlap.

Yup. Thanks! According to documentation

The caller is responsible for freeing the returned array

we have to dealloc it, not just borrow.
So, also need to check and fix same for

[ ] checkCollisions
[ ] moveWithCollisions
[ ] querySpritesAtPoint
[ ] querySpritesInRect
[ ] querySpritesAlongLine
[ ] querySpriteInfoAlongLine
[ ] overlappingSprites
[ ] allOverlappingSprites

Fixed in #413


Error in bcbio when running GATK on hg38

I have met an error in running GATK RealignerTargetCreator using hg38 as the reference genome.
This is the error from GATK::
CalledProcessError: Command 'java -Xms250m -Xmx1750m -XX:+UseSerialGC -Djava.io.tmpdir=/media/storage3/hodl/180/bcbio/project2/work/tx/tmpNXS7Z2 -jar /home/hodl/local/share/bcbio/toolplus/gatk/3.5-0-g36282e4/GenomeAnalysisTK.jar -T RealignerTargetCreator -I /media/storage3/hodl/180/bcbio/project2/work/bamprep/N180_WBC/chr1/N180_WBC-sort-chr1_0_16125330-prep-prealign.bam -R /home/hodl/local/share/bcbio/genomes/Hsapiens/hg38/seq/hg38.fa -o /media/storage3/hodl/180/bcbio/project2/work/bamprep/N180_WBC/chr1/tx/tmp_HB4vW/N180_WBC-sort-chr1_0_16125330-prep-prealign-realign.intervals -l INFO -L chr1:1-16125330 --interval_set_rule INTERSECTION --known /home/hodl/local/share/bcbio/genomes/Hsapiens/hg38/variation/Mills_and_1000G_gold_standard.indels.vcf.gz -U LENIENT_VCF_PROCESSING --read_filter BadCigar --read_filter NotPrimaryAlignment
INFO  22:18:13,525 HelpFormatter - --------------------------------------------------------------------------------
INFO  22:18:13,529 HelpFormatter - The Genome Analysis Toolkit (GATK) v3.5-0-g36282e4, Compiled 2015/11/25 04:03:56
INFO  22:18:13,529 HelpFormatter - Copyright (c) 2010 The Broad Institute
INFO  22:18:13,529 HelpFormatter - For support and documentation go to http://www.broadinstitute.org/gatk
INFO  22:18:13,538 HelpFormatter - Program Args: -T RealignerTargetCreator -I /media/storage3/hodl/180/bcbio/project2/work/bamprep/N180_WBC/chr1/N180_WBC-sort-chr1_0_16125330-prep-prealign.bam -R /home/hodl/local/share/bcbio/genomes/Hsapiens/hg38/seq/hg38.fa -o /media/storage3/hodl/180/bcbio/project2/work/bamprep/N180_WBC/chr1/tx/tmp_HB4vW/N180_WBC-sort-chr1_0_16125330-prep-prealign-realign.intervals -l INFO -L chr1:1-16125330 --interval_set_rule INTERSECTION --known /home/hodl/local/share/bcbio/genomes/Hsapiens/hg38/variation/Mills_and_1000G_gold_standard.indels.vcf.gz -U LENIENT_VCF_PROCESSING --read_filter BadCigar --read_filter NotPrimaryAlignment
INFO  22:18:13,546 HelpFormatter - Executing as hodl@administrator-ProLiant-DL385-G7 on Linux 3.2.0-24-generic amd64; OpenJDK 64-Bit Server VM 1.7.0_03-b147.
INFO  22:18:13,546 HelpFormatter - Date/Time: 2016/04/15 22:18:13
INFO  22:18:13,547 HelpFormatter - --------------------------------------------------------------------------------
INFO  22:18:13,547 HelpFormatter - --------------------------------------------------------------------------------
INFO  22:18:13,719 GenomeAnalysisEngine - Strictness is SILENT
INFO  22:18:14,524 GenomeAnalysisEngine - Downsampling Settings: Method: BY_SAMPLE, Target Coverage: 1000
INFO  22:18:14,533 SAMDataSource$SAMReaders - Initializing SAMRecords in serial
INFO  22:18:14,873 SAMDataSource$SAMReaders - Done initializing BAM readers: total time 0.33
WARN  22:18:17,732 RestStorageService - Error Response: PUT '/zWbxlBA4SGGrU8I5RMMpCxPHGgnM0e8K.report.xml.gz' -- ResponseCode: 403, ResponseStatus: Forbidden, Request Headers: [Content-Length: 1014, Content-MD5: hRi0NswhpyEXs8spE8tk1Q==, Content-Type: application/octet-stream, x-amz-meta-md5-hash: 8518b436cc21a72117b3cb2913cb64d5, Date: Fri, 15 Apr 2016 14:18:15 GMT, Authorization: AWS AKIAI22FBBJ37D5X62OQ:J7Md4tcIXFjgYfd89VeFMujyD/Q=, User-Agent: JetS3t/0.8.1 (Linux/3.2.0-24-generic; amd64; en; JVM 1.7.0_03), Host: broad.gsa.gatk.run.reports.s3.amazonaws.com, Expect: 100-continue], Response Headers: [x-amz-request-id: 70362A01F5DEBA02, x-amz-id-2: DzP1jNTV7GWtC88p0Xu47rMuLKpc+2cfY0BWnBMMredwoSsdLxD2/iUiC6tUuIb8, Content-Type: application/xml, Transfer-Encoding: chunked, Date: Fri, 15 Apr 2016 14:49:27 GMT, Connection: close, Server: AmazonS3]
WARN  22:18:18,749 RestStorageService - Adjusted time offset in response to RequestTimeTooSkewed error. Local machine and S3 server disagree on the time by approximately 1869 seconds. Retrying connection.
INFO  22:18:20,135 GATKRunReport - Uploaded run statistics report to AWS S3
ERROR ------------------------------------------------------------------------------------------
ERROR A BAM/CRAM ERROR has occurred (version 3.5-0-g36282e4):
ERROR
ERROR This means that there is something wrong with the BAM/CRAM file(s) you provided.
ERROR The error message below tells you what is the problem.
ERROR
ERROR Visit our website and forum for extensive documentation and answers to
ERROR commonly asked questions http://www.broadinstitute.org/gatk
ERROR
ERROR Please do NOT post this error to the GATK forum until you have followed these instructions:
ERROR - Make sure that your BAM file is well-formed by running Picard's validator on it
ERROR (see http://picard.sourceforge.net/command-line-overview.shtml#ValidateSamFile for details)
ERROR - Ensure that your BAM index is not corrupted: delete the current one and regenerate it with 'samtools index'
ERROR - Ensure that your CRAM index is not corrupted: delete the current one and regenerate it with
ERROR 'java -jar cramtools-3.0.jar index --bam-style-index --input-file  --reference-fasta-file '
ERROR (see https://github.com/enasequence/cramtools/tree/v3.0 for details)
ERROR
ERROR MESSAGE: Premature end of file
ERROR ------------------------------------------------------------------------------------------
' returned non-zero exit status 1
I have also run Picard's ValidateSamFile, which has produced this output:
HISTOGRAM    java.lang.String
Error Type      Count
ERROR:MATE_CIGAR_STRING_INVALID_PRESENCE        1986
ERROR:MATE_NOT_FOUND    158932
I have previously run the same samples on bcbio using GrCh37 reference genome successfully, so I don't understand why this is happening. I would really appreciate if someone can advise! Thanks!

Sorry about the problems. GATK thinks something is wrong with the input BAM file (or it's bai index):
/media/storage3/hodl/180/bcbio/project2/work/bamprep/N180_WBC/chr1/N180_WBC-sort-chr1_0_16125330-prep-prealign.bam

If ValidateSamFile passes on that file and it otherwise looks okay, I don't have any great suggestions. You could try removing the file and it's index and letting bcbio regenerate it to ensure that it isn't any kind of filesystem error.
If that doesn't fix it, I'd suggest setting turning off recalibration and realignment:
algorithm:
   recalibrate: false
   realign: false

to avoid these steps. They don't provide a lot of improvements in variant detection:
http://bcb.io/2013/10/21/updated-comparison-of-variant-detection-methods-ensemble-freebayes-and-minimal-bam-preparation-pipelines/
and I'm not sure how well supported they are with build 38 (although I don't see anything specifically about build 38 that would cause the error here). Hope this helps.

Hi
I am closing this because it seems an old issue. Come back if you find other issues  or want to continue with this one.
cheers


Handle Special Data Groups

SDGs are a way to put user-defined information into the ODX database. odxtools currently completely ignores them...
Andreas Lauser<EMAIL_ADDRESS>on behalf of MBition GmbH.
Provider Information

PR #99 should resolve this issue.

correct


Consider a CLI option for separating package name and project directory

I am looking at create-react-app --help and it doesn't seem to mention any existing functionality to achieve creating a project with different package name and project directory path in one command. I'd like to gauge whether there is any interest in accepting this feature as a pull request.
As an example of a CLI tool that provides this, consider dotnet:
dotnet new webapi -n MyProject -o be
The above creates a .NET Core project called MyProject, but places it in a be directory. I'd like to do the same for CRA. I suggest the following should be possible:
create-react-app fe -n my-project
Please let me know if you can see this as being a good fit for CRA - in a sense this can be seen as a deviation from zero configuration promise CRA holds.

I'm not sure if we will add a flag for this. This should be accomplish-able via:
create-react-app my-project && mv my-project fe


Qt Votingdialog

I fixed some issues with pie chart and added the ability to vote and create polls. I only tested it on arch linux. And please note that the creation of polls is untested! So it would be great if someone can test the Qt voting dialog, especially the creation of polls.

Great work! A could of notes while reviewing:

The dialog took maybe 30 seconds to show up. Before that the UI was locked. Would it be possible to add a loading indicator?
There's no way to close the window for me. Same with the poll results.
Can we merge all the filters into one? The filter would then check all the fields that the 4 separate filters check now. Yes, you would get some irrelevant results, but that shouldn't matter too much. Think of how Spotlight on OSX works.
Maybe for the future, but can we compact the layout a bit? I'd like to see the pie chart always visible to the right of the polls. Some fields (Share type, answers and url) can possibly be removed from the grid and placed under the pie chart as details.

Tried your branch and I like it. Build failed first due to missing library but then got it right. Opening (and refreshing) the voting form takes a few seconds. I can't confirm the closing issue. I can close the window just fine with x button on window bar. The URL (URI) should be clickable. And it even works with your other branch merged.

@tomasbrod On my installation I have no window buttons :(

The black widgets might be the same issue as I had in #217 which was fixed for me with e8cc55f57d942a52de36be722ba2fe64142fabbc. Similar issue here?

The problem with the black checkboxes is that they are images. We cannot simply change the colors. So I think we either design our own images for checked/unchecked and selected/unselected or we are stuck with what the systems color scheme provides. This should only be an issue with dark color schemes since they expect a dark background and we chose a light background picture.

@skcin Aren't we using standard checkboxes? If no then we should :)
Also, regarding the missing window buttons it's probably because we derive from QDialog, see #220. Switch to QWindow?

Yes, we are using standard checkboxes. I believe that's why you see black checkboxes. All the stylesheets I added fix the missbehaviour of dark color schemes with a light background image. I think we could also remove the background image and almost all the stylesheets and have a standard (system specific) qt look.
Regarding the missing window buttons, I am not that familiar with tiling window managers but maybe it is the intended behaviour to start in full screen? QDialog instead of QMainWIndow doesn't sound right ;) So as it breaks something else I think we should change it back.

@skcin You're right, I've reverted the QDialog change. Though the voting dialogs show the same problem. There are no Window controls in Cinnamon and no close button so there's no apparent way to close them. Maybe we should change them to QWindow instead of QDialog?

@denravonska Maybe that's also intended behaviour?
found this discussion:
https://discussion.enpass.io/index.php?/topic/1214-no-close-button-for-the-settings-window-on-cinnamon/#comment-3675

@skcin Yes, for some WMs. I guess we could use dialogs but then we have to add an OK/Close button.

Great work, @skcin!
@gridcoin We need your input on what to do with the new Qt requirement from QChart. It will require Qt 5.7 which is a little high IMO. For build requirements this is probably a non-issue but the runtime requirement is worse, so we could either:

replace the chart with something else
bundle QChart in some way
ship the binary statically

We believe the third option is what other wallets do.

After "load history" most (or all) polls are triduplicated with ";burnamount" and ";recipient" appended to poll name.

@skcin Ready for merge?

@denravonska I think so. We should get people to test it.


ENH: Bump for ITK v5.4rc02 pre-release candidate

Bump for ITK v5.4rc02 and release v0.2.1

git checkout v5.4rc02

error: Your local changes to the following files would be overwritten by checkout:
126
scripts/windows-build-tarball.ps1

This could be fixed with git checkout --ours v5.4rc02, but why are we not using the remote module GitHub Action?

why are we not using the remote module GitHub Action?

Tensorstore requires NASM and Perl which may require extra runner setup steps and thus don't seem to be compatible with our reusable workflow. Would be good to investigate further how we can support complex build prerequisites in the future.


chore: Use built in caching from setup-node action

What does this change?
This change updates the ci workflow to use caching implemented by guardian/actions-setup-node now that it's available. It also swaps out bahmutov/npm-install for yarn install --frozen-lockfile as we no longer need the caching implemented by that action.

These docs imply that we need to explicitly enable caching? Is there something else going on under the hood that I'm missing here?

Nope, I've just missed a bit of copying-and-pasting 🤦  One second...

Nope, I've just missed a bit of copying-and-pasting 🤦 One second...

Thanks for adding this 👍 🚢


Cells and content in table style subform misaligned

Attach (recommended) or Link to PDF file here:
https://planeacion.uniandes.edu.co/images/Formatos/Juridica/Solicitud_para_legalizacion_elaboracion_de_un_contrato.pdf
Configuration:

Web browser and its version: 91.0a1 (2021-07-01) (64-bit)
Operating system and its version: Windows 10

Steps to reproduce the problem:

enableXfa = true
Open form

What is the expected behavior? (add screenshot)
Text centered in cells

What went wrong? (add screenshot)
Text not centered in cells

The cells are now aligned, but the IMPUESTO DE TIMBRE: text is placed behind A CARGO DE UNIANDES .

About the second column, there is a subform with a tb (top-to-bottom) layout which contains an exclGroup which itself contains 2 fields. The top subform has a para element where vAlign is "middle".
In playing with the vAlign attribute in Acrobat I can guess that it is taken into account to place the two fields in the middle of its parent.


Created dependency-review.yml

This will restrict any vulnerable dependencies from entering the module.

Merging for the above reasons...


[DUOS-302][risk=no] Fix Action Badge Images

Addresses
Maintenance for https://broadworkbench.atlassian.net/browse/DUOS-302
Fixes the current broken workflow action badge links in the README
Before

After

Have you read Terra's Contributing Guide lately? If not, do that first.
I, the developer opening this PR, do solemnly pinky swear that:

[ ] PR is labeled with a Jira ticket number and includes a link to the ticket
[ ] PR is labeled with a security risk modifier [no, low, medium, high]
[ ] PR describes scope of changes

In all cases:

[ ] Get a minimum of one thumbs worth of review, preferably 2 if enough team members are available
[ ] Get PO sign-off for all non-trivial UI or workflow changes
[ ] Verify all tests go green
[ ] Squash and merge; you can delete your branch after this
[ ] Test this change deployed correctly and works on dev environment after deployment

Thank you @JVThomas!

Thank you @JVThomas!


Push to gh-pages without using maven's site-deploy

This PR pushes the site to github without using maven's site-deploy, which dramatically increases the speed of deployment.  Travis CI no longer times out when deploying the site to github.

For non-SNAPSHOT releases, we currently do not deploy to maven via the script.  Do we want to keep it that way, or should I try adding that in?

Lets create a separate issue for handling maven for non-SNAPSHOT releases.


Added default volumes and JWT environment variables

Type of change :

[x] Bug fix

I have worked on providing the default volumes for each of OM services and also added the. JWT configurations in docker-compose file

IMO we should remove the created volumes on metadata docker --clean
@akash-jain-10 @Vj-L @harshach

@Vj-L can you run make py_format

Closing this PR since everything is taken care in other PR


Values not passed to second instance of aliased sub sub chart

Output of helm version:
version.BuildInfo{Version:"v3.7.1", GitCommit:"1d11fcb5d3f3bf00dbe6fe31b8412839a96b3dc4", GitTreeState:"clean", GoVersion:"go1.17.2"}

Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.13", GitCommit:"2444b3347a2c45eb965b182fb836e1f51dc61b70", GitTreeState:"clean", BuildDate:"2021-11-17T13:05:33Z", GoVersion:"go1.15.15", Compiler:"gc", Platform:"darwin/amd64"}

Problem
I came across this bug when trying to create multiple instances of a of dependency that in turn had aliased dependencies. Can be reproduced with the following chart.
apiVersion: v2
name: parent
description: parent
type: application
version: 0.0.1
dependencies:
- name: sub-chart
  version: 0.0.1
  alias: sub1
- name: sub-chart
  version: 0.0.1
  alias: sub2

apiVersion: v2
name: sub-chart
description: sub-chart
type: application
version: 0.0.1
dependencies:
- name: sub-sub-chart
  version: 0.0.1
  alias: sub-sub1

In sub-sub-chart we have a template like this.
name: {{ required "name is required" .Values.name }}

Using this values file for the parent chart.
sub1:
  sub-sub1:
    name: foo
sub2:
  sub-sub1:
    name: baz

Testing it out gives this error
helm template test ./subsubalias
Error: execution error at (parent/charts/sub2/charts/sub-sub-chart/templates/test.yaml:1:9): name is required

Values are not getting passed to sub2.sub-sub1.
From my limited understanding of the code base, it seems the bug comes from the fact that a requirement with an alias has its name overwritten to the alias value at some point. And if that requirement is used again it cannot be matched against an existing chart by name. See linked PR with chart example and proposed code fix: https://github.com/helm/helm/pull/10381

Charts are not nested. When Helm loads the dependency tree, the dependency tree is flattened to a single level. In addition charts that are included twice in the dependency tree are only included once. That is, if two dependencies depend on the same chart (for example, a chart wordpress and chart psql_admin both rely on postgresql), it will only be included once.
Note that this is just how the system works as it is today. The dependency subsystem hasn't been worked on for a long, long time.

I kind of understand how that dependency loading works. But I just wanted to fix this specific issue that things break if there are aliases further down the dependency tree, when the dependencies are later flattened. This is I think is due to the fact that dependencies will have the name rewritten to the alias at some point.

Related https://github.com/helm/helm/issues/9150


Added useful function matches

I remembered using something like this is in some language and tried to implement it, the comments i wrote are cargo doc ready,
sorry for the ton of commits im used to git on terminal but i tried with vscode integrated ui.
Hope its useful, create some issues in the future and i will fix them in order to contrib

I would probably move this into the class itself as a instance method rather than a single function to urge people to re-use the instance and not re-compile the pattern every time (the Regex lib's compile is really rather heavy and slow)
Would also like it if you would mind deleting the binary, cache and lock files that I think your VSC has sent with the pr. 😄

In the last pull request its fixed all the crap that vscode sent. Sounds like a good idea move it to the class but also would be great to have into the library powerfull public functions to allow easier scriping.

Seems reasonable, might worth having a seperate file for all the public functions vs all in one file but that can be done later when I get around to reorganising everything.


make Rust code in documentation executable

like how Rust code in Rust book works

rust code is executable now on website. However, most of them are snippets, which takes a lot of work to make them actually executable e.g. providing sufficient input. I will open another issue if necessary.


Updated known issues with  'ERROR: Invalid value specified for property 'web' of resource 'Application'.'

Hi,
I was just following the readme and mostly things were smooth until I encountered the below error: 'ERROR: Invalid value specified for property 'web' of resource 'Application'.'

It took a while to figure out that I just need to wait.
After a while, it just worked fine. Updated the known issues with this details.

@microsoft-github-policy-service agree

On Feb 13, 2024, at 10:27 AM, microsoft-github-policy-service[bot] @.***> wrote:
@microsoft-github-policy-service agree

Thanks for the contribution @sagarvadodaria, this should help others that run into the same issue.
Also wanted to let you know that we're planning to update these files in the near future.


publishAndFetchRestricted example getRoot instead of returning the root from publishall

Hello, I want to use getRoot function instead of getting root form parameter function.
Like this change publishAndFetchRestricted.js example
 const root = Mam.getRoot(mamState)
    // Output syncronously once fetch is completed
    const result = await Mam.fetch(root, mode, secretKey)

fetching function will look like this
publishAll()
  .then(async () => {

    // Output asyncronously using "logData" callback function
    await Mam.fetch(root, mode, secretKey, logData)
    const root = Mam.getRoot(mamState)
    // Output syncronously once fetch is completed
    const result = await Mam.fetch(root, mode, secretKey)
    result.messages.forEach(message => console.log('Fetched and parsed', JSON.parse(trytesToAscii(message)), '\n'))

    console.log(`Verify with MAM Explorer:\n${mamExplorerLink}${root}\n`);
  })

But it's not working could anyone tell me why? Thanks in advance.

Dear Yehia,
looking at your code. I see already a pure Javascript issue there:
  publishAll()
    .then(async () => {
    // Output asyncronously using "logData" callback function
    await Mam.fetch(root, mode, secretKey, logData)
    const root = Mam.getRoot(mamState)

you are using root in the fetch but it is not defined.
you are defining a "root"  variable after usage.

I'm not quite sure what you are up for, could you provide a "working" preseudo-code?
Please also note whenever you want to fetch, you need already a root variable :)
hope this helps

Oh, I am so sorry its a silly mistake. I made a lot of trials and it ends up sending code with a bug. But even when I write the root variable right.
publishAll()
  .then(async () => {

    // Output asyncronously using "logData" callback function
    const root = Mam.getRoot(mamState)
    await Mam.fetch(root, mode, secretKey, logData)
    // Output syncronously once fetch is completed
    const result = await Mam.fetch(root, mode, secretKey)
    result.messages.forEach(message => console.log('Fetched and parsed', JSON.parse(trytesToAscii(message)), '\n'))

    console.log(`Verify with MAM Explorer:\n${mamExplorerLink}${root}\n`);
  })

the output will be  like that without any fetched data
Published { message: 'Message from Alice',
  timestamp: '11/30/2019, 10:52:02 PM' }

Published { message: 'Message from Bob',
  timestamp: '11/30/2019, 10:52:05 PM' }

Published { message: 'Message from Charlie',
  timestamp: '11/30/2019, 10:52:07 PM' }

the root is HZGS9VNRJWXG9GAWGJKJBBOUGNIKUHPPGMLRZDFSVYSQKNMVKGIKMFFCWJOLYIZKYK9XSJPEPYNLJTIEB
Verify with MAM Explorer:
https://mam-explorer.firebaseapp.com/?provider=https%3A%2F%2Fnodes.devnet.iota.org&mode=restricted&key=PASSWORDYEHIAAAAAAA99999999999999999999999999999999999999999999999999999999999999&root=HZGS9VNRJWXG9GAWGJKJBBOUGNIKUHPPGMLRZDFSVYSQKNMVKGIKMFFCWJOLYIZKYK9XSJPEPYNLJTIEB

You can find all my code here: https://pastebin.com/gY0J4JnK

the mamState is just a Javascript object literal that has the root property accessible directly :
mamState.root
You don't to use any API for that.
However be advised that if you haven't called Mam.create
  // some code for Channel generation/initialization
  const fullState = Mam.create(emptyState, trytes);

Only fullState has root defined, you can also console.log the state and inspect what you want from it.
Good luck

Okay Thank you @mtrabelsi


Actions: Complete version tags in workflow comments

Summary
This pr adds the complete version tags in our workflows and bumps two action version which were stuck pinned to commits in branches.
While fixing https://github.com/stacklok/minder/issues/2300 I noticed that the versions in the workflow tags in our workflows are not complete and this is preventing dependabot from updating them when bumping versions. This makes it hard to know  which revisions we're running.
Fixes #2300
Change Type

[ ] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[x] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[ ] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 48.341% (-0.01%) from 48.353%
when pulling c710e78f1bd114948405f1be013793c8d74c45d3 on puerco:versions
into c60a007b4e3edce17c88fadfc10722bad0e023ee on stacklok:main.

While fixing #2300 I noticed that the versions in the workflow tags in our workflows are not complete and this is preventing dependabot from updating them when bumping versions. This makes it hard to know which revisions we're running.

Is this pinning something we did with https://github.com/stacklok/frizbee?  If so, we might need to modify that, as well.

@evankanderson ah let me merge this and I'll running it and see what happens

I ran frizbee it and worked perfectly, It caught one that was referenced with a tag and I missed ( https://github.com/stacklok/minder/pull/2975 )

We should have a frizbee rule remediating on Minder.  We should figure out why that didn't fire on Monday.


Document limitations of class_name

Make it clear that PaperTrail doesn't currently support relative class_names for modules - an absolute module path is required. (See #558)

Thanks Toby!


Fix issue when PR from non-maintainers fails sonar-pr

Fixes sonar-pr error in https://github.com/hyperledger/iroha/pull/631

Codecov Report

Merging #634 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #634   +/-   ##
========================================
  Coverage    44.66%   44.66%
========================================
  Files          300      300
  Lines        11819    11819
  Branches      6287     6287
========================================
  Hits          5279     5279
  Misses         773      773
  Partials      5767     5767

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 31888c5...1506365. Read the comment docs.

Codecov Report

Merging #634 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #634   +/-   ##
========================================
  Coverage    44.66%   44.66%
========================================
  Files          300      300
  Lines        11819    11819
  Branches      6287     6287
========================================
  Hits          5279     5279
  Misses         773      773
  Partials      5767     5767

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 31888c5...b18969b. Read the comment docs.


Misunderstanding when using typescript

Hi, @andrewcourtice
I am tring to use your state manager and am stuck with some misunderstanding
(I am not very proficient in typescript)
Could you explain why I see this type error?

codebase is here

Hi @levchak0910 - the reason you are seeing the above error is because user.value could be null based on the type you have specified for your state structure.
To resolve this you would need to do a null check before accessing the username property of the user either in the computed of your component or in the user getter itself.
Hopefully that helps :)

Type checking I have by checkAuthorization(user) which return me that user is not null
I just can't figure out why the place of destructuring affects the type checking
(In the right part types are ok)

It seems like typescript is not sure, if user.value will be of type User after running setup (in setup function's scope, even after user defined guard). Because setup could be async and when it run, user.value could be another type (null) - that why error on left side
At least that's how I understood
Related typescript issues: 1, 2, main thread


fixed typo at the bottom of the doc

@midesa : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#please-close this article has been renamed and the issue was resolved.


Setup buildbot for OpenBLAS

Hi all,
I plan to setup buildbot for OpenBLAS. Dose anybody have some experiences?
Just go through the buildbot manual. It looks like I need to buy a VPS to run build-master. Then, I setup the build-slaver on OpenBLAS dev machines.

Just install build master.
http://<IP_ADDRESS>/ or http://build.openblas.net/ (wait DNS to update)

Sorry to be slow to get back to you.  I have a lot of buildbot experience; let me know if I can help.

@matthew-brett ,  Do you know the reason for the following error? unstable network?
[Failure instance: Traceback (failure with no frames): <class 'twisted.internet.error.ConnectionLost'>: Connection to the other side was lost in a non-clean fashion.

I don't think I've seen these, at least, not as consistently as you have.  Are the the slave machines on the same network as the master?
For my setup, I run no builds on the master machine - do you have the same?

@matthew-brett , the master and slave are not at same network.

@stefanv @yarikoptic - have you seen this error on your setups?
@xianyi - I assume there was no reason the buildbot slaves could have restarted during the build?
What versions of masters / slaves do you have?

The slaves are in China. The master is a VPS in US. I don't think the slave has restarted during the build.

Could this be relevant? http://trac.buildbot.net/ticket/2147
Is it possible the master has got overloaded?
Which buildslave / buildmaster versions are you running?

build master
  buildbot --version
Buildbot version: 0.8.12
Twisted version: 13.2.0

build slave
buildslave --version
Buildslave version: 0.8.8
Twisted version: 13.2.0

I just login to the slave. Although the twisted connection is closed, the test scripts (docker) are still running.

Maybe worth upgrading the buildslaves to latest?

@xianyi would it be possible to set up clang scan-build (easy to set up with ubuntu/debian and "official" clang-3.7 clang-3.8). it is quite reveling regarding uninitialized variables

@brada4 , OK.
I will try to add clang

@brada4 , I added a clang build step at quick-xxx-linux builder. You can check the output http://build.openblas.net/builders/quick-core2-linux/builds/43/steps/shell_1/logs/stdio
Meanwhile, on OS X, the default compiler is clang.

I did not mean alias gcc=clang, i meant alias make='scan-build-3.8 make'
e.g. within limits of my understanding
Index: kernel/generic/trsm_ltcopy_8.c
===================================================================
--- kernel/generic/trsm_ltcopy_8.c      (revision 6263)
+++ kernel/generic/trsm_ltcopy_8.c      (working copy)
@@ -798,7 +798,7 @@
        *(b +  1) = data02;
        *(b +  2) = data03;
        *(b +  3) = data04;
-       *(b +  4) = data05;
+       // *(b +  4) = data05;
       }

       if (ii < jj) {
Index: kernel/generic/zgemm_tcopy_4.c
===================================================================
--- kernel/generic/zgemm_tcopy_4.c      (revision 6263)
+++ kernel/generic/zgemm_tcopy_4.c      (working copy)
@@ -293,8 +293,8 @@

        aoffset1 += 8;
        aoffset2 += 8;
-       aoffset3 += 8;
-       aoffset4 += 8;
+       // aoffset3 += 8;
+       // aoffset4 += 8;

        boffset1 += m * 8;
        i --;
Index: kernel/generic/zneg_tcopy_4.c
===================================================================
--- kernel/generic/zneg_tcopy_4.c       (revision 6263)
+++ kernel/generic/zneg_tcopy_4.c       (working copy)
@@ -293,8 +293,8 @@

        aoffset1 += 8;
        aoffset2 += 8;
-       aoffset3 += 8;
-       aoffset4 += 8;
+       // aoffset3 += 8;
+       // aoffset4 += 8;

        boffset1 += m * 8;
        i --;
Index: .
===================================================================
--- .   (revision 6263)
+++ .   (working copy)

Oh, I see.
However, when I ran scan-build-3.8 make, I met the following error.
make[1]: /usr/share/clang/scan-ar: Command not found
make[1]: *** [libs] Error 127
make[1]: Leaving directory `/home/traits/project/OpenBLAS_tmp/interface'
make: *** [libs] Error 1
scan-build: 48 bugs found.
scan-build: Run 'scan-view /tmp/scan-build-2016-03-31-034631-41225-1' to examine bug reports.

ln -sf `which ar` /usr/share/clang/scan-ar
ln -sf `which ranlib` /usr/share/clang/scan-ranlib

@xianyi is buildbot setup still an open issue ?


Clarify what the default optimizer for "finish" is in scipy.optimize.brute's documentation

It appears that the current documentation doesn't make it clear enough that Scipy uses a default optimizer (apparently the Nelder-Mead optimizer) for its "finish" argument.  The documentation should especially make it clear that this default "finishing" optimizer can go well beyond the edges of the grid under some circumstances.
See also #11445.

Withdrawn.


Fixes to not show window promt of saveing scene when choosing same scene

fixes #320

SonarQube analysis reported 3 issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AncientFungus:LGEPR: Cppcheck cannot find all the include files (use --check-config for details)
 SavePromptWindow.cpp#L45: Function parameter 'newTitle' should be passed by reference.
 Resources.hpp#L54: Unused private function: 'ResourceList::GetInstance'


Correct Login/Register Flow

Correct Lojista flow onboarding on "É simples" screen, according to design.
https://www.figma.com/file/Ny3rRkAQbvPJ5zSOYkIyLj/NAFILA?node-id=4285%3A809
Review footers, spacing to the top, arrow placements, font sizes and weights and background images placements.

Remove "nafila. Sem filas. Bem-Vindo!" from mobile screens.


umurmur can't start without specifying config file manually

Error in config file /usr/local/etc/umurmur.conf line 0: file I/O error

if I'm run with -c /usr/local/etc/umurmur.conf it starts normally
OS: FreeBSD 13.2-RELEASE
umurmur version: 0.2.17_2 (from FreeBSD repositories)

Default config file location is '/usr/local/etc/umurmur.conf' and umurmur can't find any file there.

Oh, FreeBSD version specify file in /usr/local/etc/umurmur/umurmur.conf lol


Disable "SetTensor/CopyTensor" console logging.

OS Platform, distribution and device
ArchLinux 6.1.32-1-lts - AMD Radeon RX 6700 XT - gfx1031
Issue description
I'm training an LSTM neural network using CuDNN, training works perfectly but I cannot be able to disable some console logging about SetTensor/CopyTensor. This is quite annoying because the logs are printed during the entire training process surrounding Tensorflow logs. This doesn't happen if the same code run on CPU.
Logging example:
...
SetTensor
real descritor: 1, 1, 64
flat descritor: 64
SetTensor
real descritor: 1, 1, 64
flat descritor: 64
CopyTensor
src real descriptor: 1, 1, 64
src flat descriptor: 64
dst real descriptor: 1, 1, 64
dst flat descriptor: 64
...

I also tried to disable logging using the MIOPEN_ENABLE_LOGGING, MIOPEN_ENABLE_LOGGING_CMD, MIOPEN_LOG_LEVEL, and TF_CPP_MAX_VLOG_LEVEL without success.
self.rocm_logging_level = -1  # -1 disable logging
os.environ['MIOPEN_ENABLE_LOGGING'] = f'{1 if self.rocm_logging_level > 0 else 0}'
os.environ['MIOPEN_ENABLE_LOGGING_CMD'] = f'{1 if self.rocm_logging_level > 0 else 0}'
os.environ['MIOPEN_LOG_LEVEL'] = f'{self.rocm_logging_level}'
os.environ['TF_CPP_MAX_VLOG_LEVEL'] = f'{self.rocm_logging_level}'

Here are the values of the environment variables above at running time:
MIOPEN_ENABLE_LOGGING = '0'
MIOPEN_ENABLE_LOGGING_CMD = '0'
MIOPEN_LOG_LEVEL = '-1'
TF_CPP_MAX_VLOG_LEVEL = '-1'

I tried to train another model using InceptionResNetV2 and the same issues happens.
Also, this happens even using the model.predict() method if using the GPU.
Probably this is an issue related to the AMD Radeon RX 6700 XT or some mine misconfiguration.

I found the source of the problem in the MIOpen repository. If it is of interest to anyone, I have opened a new issue on the MIOpen project https://github.com/ROCmSoftwarePlatform/MIOpen/issues/2247 in order to close this issue not directly related to this ROCm repository.


cant import tensorflow.contrib.tensorrt

is there any way to import contrib.tensorrt in tensorflow 2.2.0?

@glennford49
Please refer to this comment and downgrade to 1.x version to use tensorflow contrib. and let us know if we can move this issue to resolved status.

I've already fix this... tensorrt has been moved to core in version 2.x

@glennford49
Please confirm if we may move this issue resolved status.


empty function

Hi, I want to know if it were possible that you add compatibility for the "empty" depending on the conditions
Example:
{if !empty($var)} // {if empty($var)}

Do you mean a full analog of the php empty() function as it is described here - http://php.net/manual/en/function.empty.php ?
Well, probably it would be a good idea.

Yes, I meant this php function.

There is a PHPJS version of empty
/**
 * PHP Empty
 * @param mixedVar
 * @returns {boolean}
 */
function empty (mixedVar) {
	var undef;
	var key;
	var i;
	var len;
	var emptyValues = [undef, null, false, 0, '', '0'];

	for (i = 0, len = emptyValues.length; i < len; i++) {
		if (mixedVar === emptyValues[i]) {
			return true
		}
	}

	if (typeof mixedVar === 'object') {
		for (key in mixedVar) {
			if (mixedVar.hasOwnProperty(key)) {
				return false
			}
		}
		return true
	}

	return false
}

Thank you :smiley:

Actually there is no need for a special function like this. You can use ! operator whenever you need to check if a variable is 'empty'. It works just the same.


Implement CheckAll for OnyxCheckBoxGroup

Why?
Update the implementation of the "Check all" functionality of the checkbox group to match the current Figma design.
Design
Figma link
Acceptance criteria

[x] The implementation of the "Check all" is updated

Definition of Done

[x] The following component requirements are implemented:

[x] skeleton
[x] density
[x] example usage added to apps/alpha-test-app/src/views/HomeView.vue

[x] covered by component tests
[ ] screenshot tests are updated
[x] follow-up tickets were created if necessary
[ ] updated version + documentation is deployed
[x] Storybook can show the feature
[ ] Set task to In Approval

Approval

Storybook
Deployed Docs
[ ] approved by designer @jannick-ux
[ ] approved by dev

I need more info on the behavior of the spacings of the "check all" border in horizontal layout in terms of densities


Add developers guide

I've created a new page for the "Developer guide" which is the guide for developers using grobid. I've renamed the old "Developer guide" as "Notes for grobid developers", where we can write procedures that usually tends to be forgotten.
Meanwhile I've also added some documentation about #410, but ideally we could copy/add the many responses to questions that are recurrently asked.

Coverage remained the same at 38.434% when pulling 7e801287028f7f410196527f40e3cf1b0cd658f9 on add-developers-guide into 8f500fe8a8aab9aa68eab97bb2cc28a40eae22a6 on master.


feature: Allow to check statistics of a given torrent

This is a feature request.

Allow to check statistics of a given torrent

I've created this pull request:

https://github.com/mpetazzoni/ttorrent/pull/246

Which content is:

Added to TorrentStatistic: percentage downladed, total bytes
Added to SimpleClient: get statistics of a given torrent
Added to SimpleClient, CommunicationManager: has stop

Please @mpetazzoni  let me know if you could review and if I have to change anything.
Thanks!

Thank you! I've commented on the pull request. Do there a little fix please and I'll merge it

@Dead-off please could you submit your review comments again?
I do not see them in the pull request. Thanks!

done

Thanks! you can solve this feature request :)


I can't get javafx to run on my computer

I followed the getting started with eclipse to the best of my ability. however I keep getting this error when I try to run the gui class.
Error: Could not find or load main class Files\Java\javafx-sdk-11.0.2\lib
Caused by: java.lang.ClassNotFoundException: Files\Java\javafx-sdk-11.0.2\lib
any help would be very much appreciated! thanks

When you defining path to javafx, use quotes "" (at start and at end of path), because you have space in the name (Program Files). I think in manual for windows is quotes too.

Wow It worked! Such a simple fix! Thank you so much!!! ...Yeah I overlooked the fact that I was looking at the tab for Linux/Mac on the instructions and not Windows. lol I can't believe I wasted hours on this haha. Anyways thanks again!!!


Display 'No matching feature flags found' message when no flags found from search

This PR displays the message "No matching feature flags found" on the Feature Flags Root Section Card Table instead of "No feature flags are defined. Create a new feature flag." when no matching feature flags are found from the search.

@Yagnik56 @VivekFitkariwala Please review and approve this PR.

OK, I will first resolve conflicts.

@VivekFitkariwala I resolved all your comments. Please feel free to approve this PR.


[ENH] Add flexible number of agents feature for training

Currently this implementation supports only 3 agents during training, with this enhancement it can support any number of agents decided by user during training

Issue resolved with PR #7


workshop 01: add README

Here we are planning on a workshop to make our audience familiar with
the wonderful world of Quantum Computing! Let's add a README to mark
its beginning!

seems good!


Type error with infinite query and select

What version of kubb is running?
2.1.5
What platform is your computer?
Fedora Linux
What version of external packages are you using(@tanstack-query, MSW, React, Vue, ...)
"@tanstack/react-query": "^5.17.0"
What steps can reproduce the bug?
Minimal repro: https://github.com/la55u/kubb-ts-bug

run npm run generate
inspect the types in main.tsx

How often does this bug happen?
Every time
What is the expected behavior?
AFAIK the type A should be UserPlaceResponse[] same as the content type in the selector. Right now there is two nested pages keys but the data itself has a single pages key when you console.log it.
Swagger/OpenAPI file?
https://github.com/la55u/kubb-ts-bug/blob/dev/openapi.yaml
Additional information

I'm using the select option to transform the response returned by our API since it's not a flat array, the data itself is behind a content key and I want to extract that array to the pages array of react-query.

fixed in 2.1.6


Improve colors

improved contrast for "muted" text (inside link cards under "News" or "Learn more"
used light grayscale instead of white for main text to make it easier on the eye
improved link color visibility

Oops, very sorry, I meant to merge this into my own fork but wasn't paying attention... I'm don't think I can close this PR from what I can see in the UI.
Please close this one, I will keep merging my changes with the other PR (when I'm paying enough attention...).


executeFormAction type issue

Type '((...opts: any) => Promise) | ((...opts: any) => Promise) | ((...opts: any) => Promise) | ((...opts: any) => Promise) | ((...opts: any) => Promise<...>)' is not assignable
to type 'string | ((formData: FormData) => void | Promise) | undefined'.
Type '(...opts: any) => Promise' is not assignable to type 'string | ((formData: FormData) => void | Promise) | undefined'.
Type '(...opts: any) => Promise' is not assignable to type '(formData: FormData) => void | Promise'.
Type 'Promise' is not assignable to type 'void | Promise'.
Type 'Promise' is not assignable to type 'Promise'.
Type 'null' is not assignable to type 'void'. [2322]

Hi, can you share more code to help repro?

I have this issue as well after an update from 0.2.2 to 0.2.3
<form action={executeFormAction} className="flex w-full items-end gap-4">

const { isPending, executeFormAction, isError, error } =
    useServerAction(createOrganization);

export const createOrganization = authenticatedProcedure
  .createServerAction()
  .input(
    z.object({
      name: z.string().min(1).max(255),
    }),
    {
      type: "formData",
    },
  )
  .handler(async ({ input, ctx }) => {
    const { name } = input;

    await x.db.organization.create({
      data: {
        name,
        userId: ctx.user.id,
      },
    });

    return redirect("/onboarding");
  });

After further debugging, the issue actually happens for me when updating @types/react from 18.3.5 to 18.3.10

I have the same issue...

And my @types/react version is 18.3.10

Did you find a solve for this besides not updating the types?

Sorry, I for for to close the issue. It was resolver after updating the
types.
Thank you for help!
Victor Novais
Em qui., 31 de out. de 2024 às 19:40, Chris Watts @.***>
escreveu:

Did you find a solve for this besides not updating the types?
—
Reply to this email directly, view it on GitHub
https://github.com/IdoPesok/zsa/issues/217#issuecomment-2450971626, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AU5V3A7PJRVXXNKQZHP47WTZ6KWVNAVCNFSM6AAAAABO7EJYL2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJQHE3TCNRSGY
.
You are receiving this because you commented.Message ID:
@.***>


[knife-ec2-547] Update config_source to support using knife classes without requiring merge_config

Description
This moves initialization of @original_config out of
merge_configs and into the constructor.
A common access pattern running knife commands directly in code is:
cmd = KnifeCommand.new
cmd.config[:something] = value
cmd.run

This bypasses the Knife.run class method, which does extra config
initialization - including merge_configs.
With this change,  config_source will now return nil as the
source instead of exploding when merge_configs is bypassed.
Signed-off-by: Marc A. Paradise<EMAIL_ADDRESS>
Related Issue

Fixes https://github.com/chef/knife-ec2/issues/567
Replaces https://github.com/chef/chef/pull/8494

Types of changes

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

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] All commits have been signed-off for the Developer Certificate of Origin.

We can do this, but the client code bypassing merge_configs is almost certainly a pile of bugs waiting to be found tho...


🎯 AI-Driven Media Investment Plan Across Channels for E-commerce 🚀 BUDGET MASTER APPLICATION

Project Name
🎯 AI-Driven Media Investment Plan Across Channels for E-commerce 🚀 BUDGET MASTER APPLICATION
Description
Hackathon Project Summary: AI-Driven Media Investment Plan 🚀💼
Problem: E-commerce businesses face challenges in optimizing marketing budgets across multiple paid media channels, often resulting in inefficient allocation and reduced ROI. Traditional methods lack data-driven insights into customer journeys and ad performance, limiting their effectiveness. 📉
Solution: The BUDGET MASTER APPLICATION leverages AI-driven, Retrieval-Augmented Generation (RAG) LLM chatbot to intelligently optimize a $200,000 marketing budget. By analyzing customer journey data and ad spend insights, combined with machine learning techniques, the application reallocates budgets across channels to maximize conversions and boost overall marketing efficiency. 💡
Key Tools:

AzureChatOpenAI 🤖: Provides real-time AI insights and generates intelligent responses.
LangChain & Chainlit 🛠: Enables document retrieval and a conversational interface for interactive user queries.
Machine Learning (Random Forest & Gradient Boost) 📊: Powers predictive analysis and budget reallocation strategies.
Python REPL & Vector Store 🧰: Handles data processing, document search, and deep analysis for decision-making.

This solution empowers marketers with data-driven decision-making tools, enabling better budget management, improved conversion rates, and a higher return on investment. 📈
Technology & Languages

[ ] JavaScript
[ ] Java
[ ] .NET
[X] Python
[X] AI Studio
[X] AI Search
[ ] PostgreSQL
[ ] Cosmos DB
[ ] Azure SQL

Project Repository URL
https://github.com/karthik-keerthi/BUDGET-MASTER---ECOMMERCE
Deployed Endpoint URL
No response
Project Video
https://www.popai.pro/share.html?shareKey=51436dd50214b0bb877c20f5b09c519a025e6cd19e568d00fda8ce24ce954155&utm_source=presentationsharepage
Team Members
https://github.com/karthik-keerthi

Hello @karthik-keerthi, thank you for participating in RAG Hack!
The team is working hard to distribute badges. Please have each team member fill out this form:
aka.ms/raghack/badge-dist
Thank you!


Pr/unity 2022 compatibility

Unity 2022.3 LTS & Android 12L Compatibility Updates
Changes
Unity Downgrade (2022.3 LTS)

Migrated project from Unity 6000 to Unity 2022.3 LTS
Updated project settings for 2022.3 compatibility
Modified native Java implementations:

DisplayCaptureManager
DisplayCaptureNotificationService
BarcodeReader

Android 12L Support

Modified gradle dependencies for Android API level 32 compatibility
Downgraded AndroidX dependencies:

androidx.appcompat:appcompat:1.5.1
androidx.core:core:1.8.0
androidx.activity:activity:1.5.1
androidx.annotation:annotation-experimental:1.2.0

Meta XR Improvements

Addressed warnings from Meta XR project validation manager
Optimized project settings according to Meta's recommended guidelines
Enhanced compatibility with Quest platform requirements

Purpose

Ensures broader compatibility by supporting the stable Unity 2022.3 LTS release
Maintains compatibility with Android 12L (API 32) for Horizon Store submission
Prevents dependency conflicts with newer Android API requirements
Improves Meta Quest platform compliance

Testing

Verified successful builds in Unity 2022.3
Confirmed display capture functionality on Quest 3 with Android 12L
Validated barcode scanning capabilities for QR codes
Tested notification service behavior
Passed Meta XR validation checks

Wow, super comprehensive.  I will merge this ASAP. Thank you so much for the time spent on this <3

It's my pleasure! Your work is just amazing. This repository has opened doors for a lot of prototyping work that would have otherwise been laying dormant until Meta eventually brings this feature as a building block.
I hope this will not be my last contribution to your projects. Keep up the great work!

Just wanted to leave a confirmation here that it (only tested display capture) works on Unity 2022.3.38f1, just had to set the Target API level to 34 👍 Thanks!

so sorry i haven't accepted this yet. super busy but i will have time over christmas hopefully. just updating you so you don't feel your contribution is unappreciated


Support ics-29-fees

A new MsgPayPacketFee was introduce to attach fees to an IBC packet. Both messages are normally bundled in a single TX to incentivise the right IBC package. See this nice article for the full workflow.
I think that we could fully create and handle the MsgPayPacketFee for "fee enabled channels" in wasmd without exposing any internals to the contracts. The RecvFee, AckFee and TimeoutFee coin values could be added to our IBCMsg messages for this.

Looking at how this is implemented as separate messages, I think we could just add support for the two message types MsgPayPacketFee and MsgPayPacketFeeAsync as new enum cases to IbcMsg. Since this is #[non_exhaustive] we can add message types in a non-breaking way.

I have two questions here, pretty independent of the implementation:

How would the contract know which IBC fee is appropriate?
If the contract say it pays X foocoin, how should we split this in the fee components by default?

Do I understand correctly that the fee payer must laways be the contract itself? I.e. refund_address can be set automatically by wasmd to the contract and we only support the Fee fee field?

what we need to fill is MsgPayPacketFee and there signer will just be the contract.

That is correct. For the contract developer though we can have a nicer interface.

Contract developers will have a hard time to come up with some reasonable IBC fees. I had talked with the IBC-Go team on private slack about this in the past and they opened https://github.com/cosmos/ibc-go/issues/3336 to provide some data on chain. We can then have a query for contract devs. Alternatively some people may want to maintain price oracle contracts that could be queried.

We can then have a query for contract devs. Alternatively some people may want to maintain price oracle contracts that could be queried.

That sounds like a reasonable path. In this case the contract can query a struct with all 3 fields set right away, decide if it wants to change values and send the price info along with the packet. Merging the price info in a single amount (i.e. losing information) and then having to split it back in wasmd does not seem to be very helpful.

Getting the data from chain directly would be limited to contract executions with a stable gas consumption. If a relayer usually accepted a certain price and the next packet now requires 10x the gas, the price of the past packets will be too low. In general, paying per packet assumes constant or limited relaying cost.

IBC fee support is added in https://github.com/CosmWasm/cosmwasm/pull/2196 and will be shipped as part of CosmWasm 2.2.


fix/Push docker after merging to the main branch

Update CI for docker push job. Push docker image to Docker Hub after pushing to main branch.

CI successfully updated 👍. Closing this issue (PR #14)


Cant establish connection to newer k8s versions with ansible-runner

Environment

OS: Ubuntu 22.04
Micrkok8s
Kubernetes/K3s: v1.25.5
AWX Operator: 1.1.4

Description
First thank you for your great work. This repo is a great resource and while looking for a way to get daily awx backups i found your solution using ansible-runner and was very happy. I know with using microk8s its not exactly the scope of the repo but i think my problem is more a general k8s one.
I followed you backup guide and created pvc/pv via kustomize and a service account. With the newer k8s releases the token for accessing the service account is not created anymore by default. I extended the service account creation to create a permanent token again but sadly the token is not working.
On my test system running thill the older k8s version everything is working great by with the newer version I think something is missing.
Anybody solved this problem and found a solution?
My extended serviceaccount  file:
---
apiVersion: v1
kind: ServiceAccount
metadata:
  name: awx-backup

---
apiVersion: v1
kind: Secret
metadata:
  name: awx-backup-secret
  annotations:
    kubernetes.io/service-account.name: awx-backup
type: kubernetes.io/service-account-token

---
apiVersion: rbac.authorization.k8s.io/v1
kind: Role
metadata:
  name: awx-backup
rules:
  - apiGroups:
      - awx.ansible.com
    resources:
      - awxbackups
      - awxrestores
    verbs:
      - "*"

---
kind: RoleBinding
apiVersion: rbac.authorization.k8s.io/v1
metadata:
  name: awx-backup
subjects:
  - kind: ServiceAccount
    name: awx-backup
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: Role
  name: awx-backup

@paulkitt
Hi, thanks for using my gude.
Your issue can't be reproduced on my side. I've created SA with perment token (by using YAML file you provided), obtain token, modify envvars, and invoke playbook using ansible-runner. Everything works fine,
Depending on whether the error is that K8S_AUTH_HOST cannot be connected from inside the container, or whether the error is that you do not have sufficient permissions, there are different things to investigate.
Please do not omit any fields in the issue template, it's important for others to understand your situation.
How did you obtain the token? Could you provide the logs from playbook?

Thank you for your fast reply. Ok great that the problem is more specific and not easily reproduce able. I will investigate the problem further and check connectivity from the runner container


 It isn't immediately obvious to me why the capitalization doesn't seem to matter to the first test but does to the third. Changing the IMG to img gets this code passing, but that isn't obvious from the error message of "Your image element's alt attribute should not be empty."

I'm honestly not sure why that regex in the test is even necessary. We already query the DOM to assert that the alt attribute exists and is not empty.

I am wondering how should I tackle this bug. I can edit the third hint add-images-to-your-website, but I am wondering if anything should be done to regex expression, as nhcarrigan said in

I'm honestly not sure why that regex in the test is even necessary

@Sboonny
An issue with the help wanted or first timers only label is open for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors.
If you would like to contribute, and have not read the contributors docs, please do so here: https://contribute.freecodecamp.org/#/
If you have any issues with contributing, be sure to join us on the contributors channel, or on the contributors sub-forum

I am in favour of allowing valid HTML to pass. That is, change the test to be case-insensitive.

Interesting, I didn't know that HTML is case insensitive. I thought because the challenges didn't accept <P> as an answer, that HTML is case sensitive 😅https://stackoverflow.com/questions/4351857/is-html-case-sensitive

Hello, can I work on this issue?

You can send pull request, when they add label as first-time-only, help-wanted, or they comment that they will accept pull requests for this issue. On my part, I won't submit pull request when they open it, if you planning to tackle this issue

What @ShaunSHamilton said:

I am in favour of allowing valid HTML to pass. That is, change the test to be case-insensitive.

@zidanehammouda the issue is up for grab, if you still interested


New provider: City of Austin Utilities

This PR related to issue https://github.com/tronikos/opower/issues/63
Adding support of City of Austin Utilities provider as well as supporting DSS portal type for Opower for this
Tested locally, demo script working fine, run all precommit checks. Not sure how I could test it with HA integration.
One note: I added ability to fetch user accounts from server. Not sure if user could have multiple accounts (it's one in my case) so I hardcoded it to use first every time. In case of some problems in future with this we could think about adding additional user accounts support.

I am also on COA Utilities and would be willing to help with testing. I have no real Python experience but was able to get an opower development environment installed.

Hello @tronikos Thank you for your review. Unfortunately it looks impossible to avoid changes at opower.py file:

opower.py has hardcoded /ei/edge/apis/ paths. For COA provider it's /webcenter/edge/apis/ (see my posts at https://github.com/tronikos/opower/issues/63)
All REST API endpoints for DSS require header Opower-Selected-Entities with urn:session:account: information. They will return 401 in other cases.
Subdomain starts dss- for all cases (See Oracle documentation here: https://docs.oracle.com/en/industries/energy-water/digital-self-service/transactions-seamless/Content/Transactions/SeamlessIntegration/Introduction_Seamless.htm)

I would like to avoid passing my credentials because that portal have a lot of personal and financial information (name, address, bank account) but I could run and show results for every test that is required to be sure that this solution is working. Alternatively, we could make a call in Discord and share screen to make sure that it's working properly.

Hi @atx32, you can pull my branch and run the following command:
python src/demo.py --utility coautilities --username your_username  --password your_password --aggregate_type day

Please check if it's working for you as well. It's possible that you may have a different combination of accounts, and the behavior could be different. Do you have a gas account or is it electricity only for this portal? My version is electricity only."

I just have an electric account. It may be good to find users who have other services on the same account such as water or trash to make sure there is no issue. This is what I get for the above command (with any sort of personal IDs in the response deleted):
Current bill forecast: Forecast(account=Account(customer=Customer(uuid='(deleted)'), uuid='(deleted)', utility_account_id='(deleted)', meter_type=<MeterType.ELEC: 'ELEC'>, read_resolution=None), start_date=datetime.date(2024, 1, 2), end_date=datetime.date(2024, 2, 1), current_date=datetime.date(2024, 1, 10), unit_of_measure=<UnitOfMeasure.KWH: 'KWH'>, usage_to_date=143.0, cost_to_date=15.0, forecasted_usage=725.0, forecasted_cost=78.0, typical_usage=632.0, typical_cost=65.0)
Getting historical data: account= Account(customer=Customer(uuid='(deleted)'), uuid='(deleted)', utility_account_id='(deleted)', meter_type=<MeterType.ELEC: 'ELEC'>, read_resolution=<ReadResolution.QUARTER_HOUR: 'QUARTER_HOUR'>) aggregate_type= day start_date= 2024-01-02 23:32:58.470564 end_date= 2024-01-09 23:32:58.470583
start_time      end_time        consumption     provided_cost   start_minus_prev_end    end_minus_prev_end
2024-01-02 00:00:00-06:00       2024-01-03 00:00:00-06:00       23.1624 0.946878912     None    None
2024-01-03 00:00:00-06:00       2024-01-04 00:00:00-06:00       22.5942 0.923650896     0:00:00 1 day, 0:00:00
2024-01-04 00:00:00-06:00       2024-01-05 00:00:00-06:00       23.2704 0.951293952     0:00:00 1 day, 0:00:00
2024-01-05 00:00:00-06:00       2024-01-06 00:00:00-06:00       24.6132 1.006187616     0:00:00 1 day, 0:00:00
2024-01-06 00:00:00-06:00       2024-01-07 00:00:00-06:00       22.3026 0.911730288     0:00:00 1 day, 0:00:00
2024-01-07 00:00:00-06:00       2024-01-08 00:00:00-06:00       26.6346 1.088822448     0:00:00 1 day, 0:00:00
I also tried aggregate_type=hour and for the historical data get the following (only copied the first few lines):
Getting historical data: account= Account(customer=Customer(uuid='(deleted)'), uuid='(deleted)', utility_account_id='(deleted)', meter_type=<MeterType.ELEC: 'ELEC'>, read_resolution=<ReadResolution.QUARTER_HOUR: 'QUARTER_HOUR'>) aggregate_type= hour start_date= 2024-01-02 23:48:26.760628 end_date= 2024-01-09 23:48:26.760647
start_time      end_time        consumption     provided_cost   start_minus_prev_end    end_minus_prev_end
2024-01-02 00:00:00-06:00       2024-01-02 01:00:00-06:00       0.798   0.03262224      None    None
2024-01-02 01:00:00-06:00       2024-01-02 02:00:00-06:00       0.8508  0.034780704     0:00:00 1:00:00
2024-01-02 02:00:00-06:00       2024-01-02 03:00:00-06:00       0.8154  0.033333552     0:00:00 1:00:00
2024-01-02 03:00:00-06:00       2024-01-02 04:00:00-06:00       0.7032  0.028746816     0:00:00 1:00:00
2024-01-02 04:00:00-06:00       2024-01-02 05:00:00-06:00       0.7278  0.029752464     0:00:00 1:00:00
2024-01-02 05:00:00-06:00       2024-01-02 06:00:00-06:00       0.732   0.02992416      0:00:00 1:00:00
2024-01-02 06:00:00-06:00       2024-01-02 07:00:00-06:00       0.753   0.03078264      0:00:00 1:00:00
2024-01-02 07:00:00-06:00       2024-01-02 08:00:00-06:00       0.9072  0.037086336     0:00:00 1:00:00
2024-01-02 08:00:00-06:00       2024-01-02 09:00:00-06:00       0.7686  0.031420368     0:00:00 1:00:00
2024-01-02 09:00:00-06:00       2024-01-02 10:00:00-06:00       1.146   0.04684848      0:00:00 1:00:00
2024-01-02 10:00:00-06:00       2024-01-02 11:00:00-06:00       1.1718  0.047903184     0:00:00 1:00:00
No issues for me so far.

Once you address my comments and I release a new version, could you please:

bump the version on the Home Assistant side
update documentation at https://www.home-assistant.io/integrations/opower/ to mention support for this utility
create a virtual integration with icon and documentation

I released a new version. Thanks! Please bump the version on the Home Assistant side, update documentation, and add a virtual integration.

@atx32 New version of HA includes this provider, you could try to use it. My integration is working good for now: https://www.home-assistant.io/integrations/coautilities/

@atx32 New version of HA includes this provider, you could try to use it. My integration is working good for now: https://www.home-assistant.io/integrations/coautilities/

I already updated HA and added the integration. Working good for me too. Thanks for putting in the time on this.


Carthage iOS framework issue

I've followed the guidelines for using Carthage when creating a custom iOS app but encounter the following error when trying to generate .framework files.
$ carthage update --platform iOS --use-submodules --no-use-binaries
*** Fetching spark-sdk-ios
*** Checking out spark-sdk-ios at "appstore_tinker_v1.0"
*** xcodebuild output can be found in /var/folders/1d/yjtzzlj10b9g7109rsshw2tr0000gn/T/carthage-xcodebuild.QRmzUF.log
*** Skipped building spark-sdk-ios due to the error:
Dependency "spark-sdk-ios" has no shared framework schemes for any of the platforms: iOS
If you believe this to be an error, please file an issue with the maintainers at https://github.com/spark/spark-sdk-ios/issues/new

I am having the same issue.  Looking at the project I am not sure how Carthage would work since there is not a top level XCode Project file with shared schemes.  I see XCode Project files in the example folder, but not sure how Carthage would know to look into that folder.  Maybe I am missing something.  Any help with getting this to work with Carthage would be greatly appreciated.

oh, carthage seems to pull the wrong commit because of the git tag label. Let me see if I can fix that quickly

I specified "master" in my Cartfile or spark-sdk-ios and that seems to be working.  Not sure why it would use that other label, maybe it had a weird conflict with some of the other dependencies in my Cartfile.

Oh, glad to hear it works now. Carthage does seem to have sensitivity to tag names in Cartfile, I'll check if that applies in the general case and try to fix it


weechat-unwrapped: 4.3.5 -> 4.3.6

Description of changes
https://github.com/weechat/weechat/releases/tag/v4.3.6
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 334885 run on aarch64-darwin 1

9 packages built:

weechat
weechat-unwrapped
weechat-unwrapped.guile
weechat-unwrapped.lua
weechat-unwrapped.man
weechat-unwrapped.perl
weechat-unwrapped.python
weechat-unwrapped.ruby
weechat-unwrapped.tcl

Result of nixpkgs-review pr 334885 run on x86_64-linux 1

10 packages built:

weechat
weechat-unwrapped
weechat-unwrapped.guile
weechat-unwrapped.lua
weechat-unwrapped.man
weechat-unwrapped.perl
weechat-unwrapped.php
weechat-unwrapped.python
weechat-unwrapped.ruby
weechat-unwrapped.tcl

Thanks for merging this, @fpletz, very helpful and much appreciated 🙂


Update AFNetworking to 3.1

https://github.com/matrix-org/matrix-ios-sdk/issues/113
I try to keep the same behaviour we had with the previous AFNetworking version. Specifically for media upload when the app goes in background.
I tested SSL in different situations but I failed to enter in this code https://github.com/matrix-org/matrix-ios-sdk/blob/1feaa3ddc4c1a49f9fb7438ac202721024dbc732/MatrixSDK/Utils/MXHTTPClient.m#L455-L471.
As I cannot understand how we can enter in this else block, I have removed it.
There is an API break in MXRestClient with the introduction of NSProgress.

sgtm


SQL with gMSA breaks SSL implementation

After creating a SQL Docker image, the SSL does no longer work to import certain certificates, or create new self signed certificates.
We tested on different Server 2019 versions updated to the last update:

1809: same error
1903: works (we have no clue why it does work here)
1909: same error
2004: same error
20H2: same error

The error we received is the following:
New-SelfSignedCertificate : CertEnroll::CX509Enrollment::_CreateRequest: The remote procedure call failed. 0x800706be (WIN32: 1726 RPC_S_CALL_FAILED)
At C:\start.ps1:72 char:1
+ New-SelfSignedCertificate -KeyLength 2048 -SmimeCapabilities -KeyAlgo ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [New-SelfSignedCertificate], Exception
    + FullyQualifiedErrorId : System.Exception,Microsoft.CertificateServices.Commands.NewSelfSignedCertificateCommand

We used the dockerfile from Microsoft.
We adjusted the start.ps1 to not loop, but instead just run a new-selfsignedcertificate command:
New-SelfSignedCertificate -KeyLength 2048 -SmimeCapabilities -KeyAlgorithm RSA -DnsName "test.test.com" -Type SSLServerAuthentication -NotAfter (Get-Date).AddYears(40)
The issue started with Business Central containers which was also reported here on the navcontainerhelper  github. We worked a bit to more pin down the issue. This is the furthest / cleanest repro point we got.
If you do not enabled gMSA, the issue is not there. However in our scenario we need to use gMSA.
For completeness, I will include the files we used in DevOps for multiple server run. We also did this manually, so it is not a DevOps issue.
github_report.zip

acknowledging; looking which SMEs in side who can help triage and root cause...

We ran the following docker file with the same results as above:
FROM %%sourceimage%%

SHELL ["powershell", "-Command", "$ErrorActionPreference = 'Stop';"]

COPY start.ps1 /
WORKDIR /

RUN \
  Write-Host 'Trying to create certificate'; \
  New-SelfSignedCertificate -KeyLength 2048 -SmimeCapabilities -KeyAlgorithm RSA -DnsName 'test.test.com' -Type SSLServerAuthentication -NotAfter (Get-Date).AddYears(40);

RUN \
  Write-Host 'Trying to create certificate'; \
  New-SelfSignedCertificate -KeyLength 2048 -SmimeCapabilities -KeyAlgorithm RSA -DnsName 'test.test.com' -Type SSLServerAuthentication -NotAfter (Get-Date).AddYears(40);

CMD powershell -Command .\start.ps1

the start.ps1 is as follows:
Write-Host "Trying to create certificate (1)"
New-SelfSignedCertificate -KeyLength 2048 -SmimeCapabilities -KeyAlgorithm RSA -DnsName "test.test.com" -Type SSLServerAuthentication -NotAfter (Get-Date).AddYears(40)

Write-Host "Trying to create certificate (2)"
New-SelfSignedCertificate -KeyLength 2048 -SmimeCapabilities -KeyAlgorithm RSA -DnsName "test.test.com" -Type SSLServerAuthentication -NotAfter (Get-Date).AddYears(40)

it appears as soon as the build step creates a certificate, the start.ps1 can no longer create certificates.
If we place the 2 tryouts in the dockerfile in comment, the start.ps1 works as expected.
so presumably, SQL Install creates a certificate in the install process which results in the same problem.

Any updates?

We are having the same problem, currently blocking upgrade for our customers.
Kind regards

We too :(.
@weijuans-msft , would you have any feedback on this?

The SQL container image is not an officially supported image. However I understand this is not SQL specific. We are looking at repro'ing in house and investigate. If there is any other info such as logs etc, please do share.

The most straight forward example for recreating the issue is the example of @joachimcarrein
=> https://github.com/microsoft/Windows-Containers/issues/124#issuecomment-870525577
It's based on windows core (no sql involved) and can be repro'd very easily.

@joachimcarrein, can you help me understand the behavior of this issue:

is this gMSA v1 or v2?
Just curious why you need to create same certs twice? Does the failing application have write access to HKLM\My store? Is a new certificate added each time. Or does it do name matching and overwrite?
Do you mean the New-SelfSignedCertificate cmdlet work in docker file but not in start.ps1?
What happens if you remove the lines to create cert in docker file?

Hi @bingbing8

No clue if it is v1 or v2. This is how we create the gmsa with PowerShell:
New-ADServiceAccount -Name $GmsaAccount
-DNSHostName $ServerFQDN                   -PrincipalsAllowedToRetrieveManagedPassword "Domain Controllers", "domain admins", "CN=Container Hosts,CN=Builtin, DC=domain, DC=local"
-KerberosEncryptionType RC4, AES128, AES256`
We do not need the same certificate twice. This is just a reproduction scenario that is easy to test and gives the exact same error we had. In our scenario, we do customisations on an Microsoft pre built image and do need a self signed certificate. Our suspision is that in the Microsoft pre built image their is already a certificate created (maybe even temporary).
No. It doesn't work within docker or start.ps1 as soon as the command was run in the docker file.
This is just repro scenario, see point 2, we have no control over the Microsoft pre built image
We have explicitly tested on the same host OS as container image version. No differences there
We tested both

@joachimcarrein by gMSAv2, we meant "gMSA with non-domain joined host" :), see more here in this blog: https://techcommunity.microsoft.com/t5/containers/announcing-active-directory-identity-improvement-on-aks-on-azure/ba-p/2374198.
I believe you are using the current version of gMSA which requires a domain joined host, sometimes referred as gMSA v1 internally.

As described in the issue, for some reason it did work on 1903. So internally we have rebuild all our platforms to 1903 and migrated all dockers to work around the issue. On other locations where we were currently unable to upgrade the platform, we halted all upgrades and kept everything as it is for continuity.
Very recently we did a build on an 1809 machine, where we noticed that the base image was changed (windows update) and apparently also don't run into the issue anymore.

@tommyreinsmsft any update on this? This is impacting our ability to upgrade to newer versions of the container. We are running on 2019 LTSC hosts that are domain joined. We cannot remove gMSA.

For anyone stumbling on this issue, I've found a workaround. You'll want to generate the self signed certificate during your docker build, not at runtime. The key here is that the the hostname on the certificate must match the hostname of the container at runtime. I'm running on ECS w/ EC2 so the hostname is set for me to be the name of the gMSA. You can also override hostname in the docker run command as documented here.
For example, look at the credential spec to get the MachineAccountName value:
{
  "DomainJoinConfig": {
    "Sid": "S-1-234",
    "MachineAccountName": "my-gmsa",
    "Guid": "1234",
    ....
  },
  ...
}

Take that and generate the self signed cert at build like so in your Dockerfile:
RUN New-SelfSignedCertificate -Subject "my-gmsa" -DnsName "my-gmsa" -CertStoreLocation "Cert:\LocalMachine\My" -NotAfter (Get-Date).AddYears(10)


Revert "Feature/issue 2401 alg solver adjoint"

Reverts stan-dev/math#2421 it looks like we are having some upstream issues. I thought I sorted out the arena_allocator problem but I think there's another place we need a fix for that. I'll open up a revert revert with a fix tmrw

Ty! I'll have a fix for this up tmrw

Great! Thanks!


KL Divergence for Latent SDEs

A continuation of https://github.com/patrick-kidger/diffrax/pull/402 with the new 0.6.0 lineax changes

Okay, I really like the example here!
I'm afraid this might still take a bit more iteration to get to something clean enough to merge, though -- see my comments. :)

Happy to iterate on cleaning it, I think the biggest question is the design one (on solvers, terms, and how to represent the problem in diffrax). Once that is resolved, I can iterate quickly to get the rest in :)

Ok, I took the feedback from Andraz's Langevin PR regarding terms and incorporated it into this PR. I think it made things simpler and more in line with the diffrax philosophy, let me know what you think. Basically, now like Langevin, there's just a function that accepts multi term and returns a multi term of private terms that can be consumed by any solver.


CircuitPython remount in boot.py makes drive readonly to PC.

I have this boot.py per the instructions and now the device is mounted read only to my pc.  I think it changes the drive from writable via circuitpython and read only to the computer.  You may wish to add notes on how to remove that boot.py from the repl so that people can code with files on the drive again.

https://docs.circuitpython.org/en/latest/shared-bindings/storage/index.html
https://learn.adafruit.com/circuitpython-essentials/circuitpython-storage

import storage

storage.remount("/")

I'll add documentation to the circuitpython section with the commands you provided, as well as instructions how to delete the boot.py

import os
os.listdir()
os.remove("boot.py")

I added more documentation in #96. Closing this for now, if you have feedback please provide it in the PR. I'll be merging it after I update some other documentation sections on new features.


add send_typing

add send_typing method
update evilbot

work on auxin? groups?

works on groups, doing this with auxin would require the simulate dance, which i'm hoping will be simpler soon

i tried adding a version for auxin, but i don't know why it doesn't work. can be tested with evilbot


build.prop editing disabled

Hello, my propsconf.log shows build.prop editing disabled but I can't figure out why. No conflicting modules are listed. I have tried disabling modules but this has made no difference. I am using a project treble GSI rom, running Android 8.1.

And my propslogs.
propslogs.tar.gz

Yup. Somethingis is definitely wrong. Must be something with the logic in checking for the stock prop values.
Will take a look when I get an opportunity.

Great, thanks.

So... I finally had some peace and quiet to sit down and look at this. Started looking through the code but couldn't find anything. Hm.... I went back to this issue and realised I was a bit sloppy reading the bug description the first time around.
The log is supposed to say that build.prop editing is disabled if you haven't actually enabled it.
@darren3 Did you even run the props script and enable "Improved hiding"? Because if you didn't, there's no bug... 😆 Everything in your logs and settings look exactly as they should.

Hi, I hadn't enabled Improved hiding. Thanks for having a look.


Add container logs for Read Only user

@hibooboo2

ACTWFBWDWTWFBTTCTI*  https://github.com/rancher/rancher/issues/1990

Adds code that will fix but we don't want to say "Fixes" because then that closes the issue


add macOS 10.14 Mojave to Azure Pipelines CI

[X] Closes  #719
[X] I am familiar with the contributing guidelines.
[x] Fully tested. Added and/or modified tests to ensure correct behavior for all reasonable inputs. Tests (usually) must pass on the TravisCI and Appveyor testing services.
[ ] Updates entries to docs/sphinx/source/api.rst for API changes.
[ ] Adds description and name entries in the appropriate docs/sphinx/source/whatsnew file for all changes.
[x] Code quality and style is sufficient. Passes LGTM and SticklerCI checks.
[ ] New code is fully documented. Includes sphinx/numpydoc compliant docstrings and comments in the code where necessary.
[ ] Pull request is nearly complete and ready for detailed review.

Adds Azure test functionality for the macOS 10.14 Mojave system. This implementation was simple (for now) and replicates the standard Linux job.
I wanted to ask you @wholmgren if we should be mindful of increasing the time duration of Azure Pipelines with another three test cycles. This may lengthen each iteration a contributor must wait after a change so I think it's worth considering.
If it is a concern, would it a better use of time be to only test the "bare_linux" version and not the 'Test_conda_linux'? I would argue that the "bare_linux" is the least stable from an infrastructure POV. I'd also guess, and I could be completely wrong, that most people that choose to run on Linux aren't going the Anaconda route.
If that's not the case and we want to be sure about Anaconda installs would we then add a 'Test_conda_macOS' job? If so, we'd be increasing the CI time by 66%. =S

I'm a little reluctant to drop the conda_linux configuration because we previously ran into an issue that was specific to how continuum compiled scipy for linux. I do use conda environments on linux because it's an easy way for me to manage custom python environments without root access.
Is a conda_mac configuration more representative of end user environments than a bare_mac configuration? I don't think we need both.

can we spread the load over several CI's from different providers, some travis, some azure, and some appveyor or circle? what are their free offerings? Does azure offer free concurrency for oss? I'll try to answer these if useful

I've thought it would be easier to maintain the CI if we reduce to just 1 provider. They all have their quirks, but lately Azure has been my favorite. In this PR, Azure completes 12 configurations across 3 platforms in roughly the time it takes Travis to complete 4 configurations on 1 platform.

I'm a little reluctant to drop the conda_linux configuration because we previously ran into an issue that was specific to how continuum compiled scipy for linux. I do use conda environments on linux because it's an easy way for me to manage custom python environments without root access.

I'm not familiar with the severity of that issue you had before but if it was a "one-off" occurance I would advocate for a quicker CI experience than covering than for trying to cover all avenues of usability all the time. Having to wait an extended amount of time for the CI through every iteration of a PR could scare off new contributors and a drag on the frequent contributors. We can always spin up a wider scoped CI on our own branch if an issue comes up with another build that's in heavy use.
You've earned plenty of wisdom in this area so I'd defer to you from a maintainers perspective. The rest of the team may have a strong opinion one way or another as well.

Is a conda_mac configuration more representative of end user environments than a bare_mac configuration? I don't think we need both.

This may be a good question for a survey. I personally use Homebrew on Mac which shields the user from using the system Python and doesn't require sudo. I feel like conda is another layer of abstraction that's not necessary for *nix systems but worthwhile on Windows.

I don't have a strong preference between bare_mac, conda_mac, or both.
I'm not that concerned about total CI speed so long as results begin to be returned quickly. The vast majority of errors are caught within a single platform's bare (~1 min) or a single platform's conda (~4 min) configurations, and then the total build is immediately marked failed. That's faster than any of the Travis results (~6 minutes). LGTM is always the slowest (typically 30-60 minutes).

I don't have a strong preference between bare_mac, conda_mac, or both.

Let's just keep it bare_mac for now. If we hear that everyone is using conda on mac and/or there's a significant difference in bug occurrence then we can add it in.

I'm not that concerned about total CI speed so long as results begin to be returned quickly.
I wasn't aware of the "fail fast" mode. From that perspective, adding tests shouldn't be as big of a deal especially with LGTM making us wait significantly longer as well.

Thanks for the feedback!


Allow PyPDFLoader to restrict page range loaded

When loading PDFs, I've needed to use manual workarounds to restrict page sections loaded. This is either to exclude pages that are confusing the retriever, such as tables of contents or references or to only include relevant sections of books. This PR replicates my data-cleaning workarounds.
Who can review?
@eyurtsev
@hwchase17

@mvanwyk Hi , could you, please, resolve the merging issues and address the last comments (if needed)? After that, ping me and I push this PR for the review. Thanks!

Closing because the PR wouldn't line up with the current directory structure of the library (would need to be in /libs/langchain/langchain instead of /langchain). Feel free to reopen against the current head if it's still relevant!