Formatting issue with complex vertical merging

I have a need for a very specific table, which logically seems like it should work but in practice does not yield the result I would expect.
The table I want is shaped like this:

Notice how the cells in columns 4 and 5 are merged, compared to columns 2 and 3.
Logically, we would make this shape with 4 rows and then vertically merge the necessary cells. Doing so in Obsidian with this plugin looks like this:
| Col 1 | Col 2 | Col 3 | Col 4 | Col 5 | Col 6 |
| ----- | ----- | ----- | ----- | ----- | ----- |
|       |       |       |       |       |       |
| ^     | ^     | ^     |       |       | ^     |
|       | ^     | ^     | ^     | ^     |       |
| ^     | ^     | ^     |       |       | ^     |

Right away, we can tell that the table looks different from what I'm after. If I begin filling in cells that I need, it starts to look correct:
| Col 1 | Col 2 | Col 3 | Col 4 | Col 5 | Col 6 |
| ----- | ----- | ----- | ----- | ----- | ----- |
| X     | X     | X     |       |       | X     |
| ^     | ^     | ^     | X     | X     | ^     |
| X     | ^     | ^     | ^     | ^     | X     |
| ^     | ^     | ^     |       |       | ^     |

But as soon as I fill certain cells, such as in column 4 or 5 on row 1, it breaks down again:
| Col 1 | Col 2 | Col 3 | Col 4 | Col 5 | Col 6 |
| ----- | ----- | ----- | ----- | ----- | ----- |
| X     | X     | X     |       | X     | X     |
| ^     | ^     | ^     | X     | X     | ^     |
| X     | ^     | ^     | ^     | ^     | X     |
| ^     | ^     | ^     |       |       | ^     |

I suspect this has to do with how the cells are anchored/stretched, but I do not know if this is an Obsidian quirk or the plugin itself. I am not sure what can be done about it, or if the behavior is simply expected and what I am doing should fail. A workaround might be to use HTML to create nested tables where needed, but I would prefer to keep my documents without HTML.
Thanks for any help or suggestions!

From what I suspect this is more of a css problem than a problem with this plugin itself. I will try to come up with a fix soon

I appreciate it! No rush at all. Your plugin is great!

Should be fixed in 1.2.5 !


[Bug] Freezes on step 3 of install npm install

Describe the bug
I followed the Installation steps:

git clone https://github.com/onlook-dev/studio.git
cd app
npm install

Keeps freezing here. Tried multiple times. Even deleting and recloning...
PS me> git clone https://github.com/onlook-dev/studio.git
Cloning into 'studio'...
remote: Enumerating objects: 961, done.
remote: Counting objects: 100% (274/274), done.
remote: Compressing objects: 100% (181/181), done.
Rremote: Total 961 (delta 137), reused 104 (delta 92), pack-reused 687eceiving objects:  98%

Receiving objects: 100% (961/961), 12.05 MiB | 29.10 MiB/s, done.
Resolving deltas: 100% (343/343), done.
PS me> cd .\studio\app\
PS me> npm install
[..................] / idealTree:app: sill idealTree buildDeps

Thoughts on this one?

Hey @SeaDude ,
How long did you let it run? I tested it with another machine and since its downloading electron it could take a long time.
Can you try running with npm install --verbose and share the logs?
https://stackoverflow.com/questions/37038134/install-electron-on-node-for-windows-hangs

Closing since there's a potential solution. Please re-open if this issue still exists :)


Preview opening an empty browser window

With the latest version of Sublime and the plugin preview is opening an empty page. I can provide details if needed.
Regards,
Marcin

Unfortunately, this is not an issue I'm seeing.  I imagine this was a user configuration issue. I imagine you got this working as it's been over a year. Sorry, someone didn't get to this issue sooner.  I'm going to close this, but open again if you are still having issues.


feat(pixels): add my new pixel

Checklist

[x] I ran npm test locally and it passed without errors.

[x] I only edited the _data/pixels.json file.

[x] I entered the username in the pixels.json that I'm also using to create this pull request.
[x] I acknowledge that all my contributions will be made under the project's license.

I have made  changes as per guidelines, tested and pushed on to my branch


[ENHANCEMENT] move code to test-support directory

This looks like a great project! I'm excited to try this out.
Something that I noticed was that all the code is in the addon directory. It is not really needed outside of  testing. Because of that, I might recommend moving the code into the test-support directory so as not to bloat the production build of your hosting app.
I was unaware of this feature until @bcardarella pointed it out in an addon that I am heavily involved with. I also wrote a really quick blog post (mainly for myself) about the feature.
Again, looking forward to trying out this addon. Thanks!

Thanks for you suggestions, I'll move the code to test-support. Can you point me to some documentation about that folder? I couldn't find good documentation on how to build an ember-cli addon focused on creating testing helpers.
I'm really glad you liked the project and I really appreciate your feedback!

There isn't much documentation yet. Projects are the best documentation we have right now. You can checkout the addon I referenced above, the blog post I wrote, or this example. That is where I stole my inspiration from. If you'd like to pair up sometime, let me know.


fix(deps): :arrow_up: update dependency aws-cdk to ^2.116.1

This PR contains the following updates:

Package
Change
Age
Adoption
Passing
Confidence

aws-cdk (source)
^2.116.0 -> ^2.116.1

Release Notes

aws/aws-cdk (aws-cdk)
v2.116.1
Compare Source
Bug Fixes

core: core constructs fail with Error: Cannot find module '../dist/core/<file>.generated' (#28467) (e8be128), closes #28251 #28465

Alpha modules (2.116.1-alpha.0)
v2.116.0
Compare Source
Features

apigateway: validate integrationHttpMethod with non-MOCK integration types (#28316) (93cb6e4)
ec2: support creating key pairs (#28138) (6b3caff), closes #5252
ecr: Add emptyOnDelete CloudFormation property to Repository L2 construct (#28233) (a175da8), closes #24572 #28196
ecr: tag pattern list for lifecycle policy (#28432) (aa075cd)
ecs: log retention for FireLensLogDriver (#28354) (52a5579), closes /github.com/aws/aws-cdk/blob/db22b85c9b2a853aa2f830c182a340f0bcf95d1a/packages/aws-cdk-lib/aws-ecs/lib/base/task-definition.ts#L816 /github.com/aws/aws-cdk/blob/db22b85c9b2a853aa2f830c182a340f0bcf95d1a/packages/aws-cdk-lib/aws-ecs/lib/firelens-log-router.ts#L170 #28258
ecs: Support specifying revision of task definition (#27036) (de0d77b), closes #26983
events: add multiple event bus policies on a single event bus (#27340) (4dde502), closes #24671
rds: support aurora mysql 3.05.1 (#28370) (986db38)
ses: add scalingMode property to dedicated ip pool (#28392) (c687778), closes #27418
sns: delivery status logging configuration (#28433) (36bb696), closes #21971
stepfunctions: CustomState addCatch (#28422) (cf923bc), closes #25798
stepfunctions: support Map ItemProcessor (#27913) (ac41730), closes #27878
stepfunctions-tasks: EMR createCluster command support OnDemandSpecification (#27791) (73a5e74), closes #27761
implement code generated handler framework (#28251) (1a9c30e), closes #27303
update AWS Service Spec (#28407) (fcb701c)

Bug Fixes

core: messages from annotations.ts can show up as [object Object] (#28414) (3e6f10d)
custom-resources: cannot set logging for state machine generated in CompleteHandler (#27310) (bfeef00), closes #27283
ecs-patterns: taskDefinition ignored by queueProcessingFargateService (#28220) (5d6b8ee), closes #27360
eks: neuron plugin not added with addNodegroupCapacity() (#27909) (b87212b)
events-targets: imported sqs queue cannot be used as a rule dlq (#28165) (#28285) (588b106)
iam: importedRoleStackSafeDefaultPolicyName feature flag results in excessively long IAM policy names (#27548) (4f88db6), closes #27409 #24441
logs: adding a resource policy statement with AnyPrincipal fails (#27787) (0b2ba1c), closes #27783
s3-notifications: multiple notifications doesn't work (#28132) (37be7b9)
secretsmanager: cannot set hourly rotation (#28303) (09cb003), closes #28261
secretsmanager: hosted rotation with fromSecretNameV2() does not create correct iam policy (#28379) (4ab5bb2), closes #28308

Alpha modules (2.116.0-alpha.0)
Features

scheduler: flexible time windows (#28098) (6554e48)
scheduler-targets: add CodePipeline as target for scheduler (#27799) (8c44f32), closes #27449

Bug Fixes

integ-tests: apply correct IAM policy to waiterProvider (#28424) (c488035), closes 40aws-cdk/integ-tests-alpha/lib/assertions/sdk.ts#L136 40aws-cdk/integ-tests-alpha/lib/assertions/sdk.ts#L247 #27865
lambda-python-alpha: pipenv lock -r is no longer supported (#28317) (f85f486), closes #28015 /github.com/pypa/pipenv/blob/main/CHANGELOG.md#2022813-2022-08-13 #28015

v2.115.0
Compare Source
Features

eks: support Bottlerocket Nvidia AMIs (#28287) (6aa1b1b), closes #28241
rds: support RDS for MariaDB 10.11.6, 10.6.16, 10.5.23 and 10.4.32 (#28345) (0347a19)
rds: support RDS for SQL Server 16.00.4095.4.v1 (#28274) (7a22501)
stepfunctions-tasks: runtime role in EmrAddStep  (#27736) (314fbfa), closes #27691
stepfunctions-tasks: support for the Step Functions optimized integration for Bedrock InvokeModel API (#28276) (f3dafa4)
update AWS Service Spec (#28328) (15c7bb2)
ROADMAP: updates to public roadmap 🚀  (#28302) (fe30921)

Bug Fixes

rds: publiclyAccessible=false set on an instance is ignored when cluster is placed in a public subnet (#28038) (569593c), closes #28037 #28037
route53: delete old NS records on cross-account delegation renaming (#21249) (#27523) (448e2bd), closes #25285
stepfunctions-tasks: missing tags & perms for emr cluster creation (#28327) (14e5e50)
stepfunctions-tasks: state machine role is missing sagemaker:AddTags permission for SageMakerCreateTransformJob task (#27264) (8ce0283), closes #26012

Alpha modules (2.115.0-alpha.0)
⚠ BREAKING CHANGES TO EXPERIMENTAL FEATURES

scheduler: The typos in the Schedule and Group construct method names have been fixed, changing metricSentToDLQTrunacted to metricSentToDLQTruncated and metricAllSentToDLQTrunacted to metricAllSentToDLQTruncated.
redshift: Further updates of the Redshift table will fail for existing tables, if the table name is changed. Therefore, changing the table name for existing Redshift tables have been disabled.

Features

appconfig-alpha: add deploy method to configuration constructs (#28269) (c723ef9)
cloud9-alpha: support image ids for Amazon Linux 2023 and Ubuntu 22.04 (#28346) (93681e0), closes /docs.aws.amazon.com/ja_jp/AWSCloudFormation/latest/UserGuide/aws-resource-cloud9-environmentec2.html#cfn-cloud9-environmentec2
scheduler: start and end time for schedule construct (#28306) (0b4ab1d), closes /github.com/aws/aws-cdk/pull/26819#discussion_r1301532299

Bug Fixes

appconfig-alpha: extensions always create cdk diff (#28264) (2075559), closes #27676
redshift: tables were dropped on table name change (#24308) (7ac237b), closes #24246
scheduler: typo in metricSentToDLQ... methods (#28307) (8b91e10)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: package-lock.json
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@angular/core
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@angular/animations
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR!
npm ERR! For a full report see:
npm ERR! /tmp/renovate/cache/others/npm/_logs/2023-12-25T18_28_59_698Z-eresolve-report.txt

npm ERR! A complete log of this run can be found in: /tmp/renovate/cache/others/npm/_logs/2023-12-25T18_28_59_698Z-debug-0.log


Showcase Section - Properly display new additions

Hi,
I was just thinking that it might be cool to more prominently display new additions to the showcase section.
Is there a way we could automate that without using a dedicated tag where we wouldn't have to open a PR to remove it, once a given time has passed.
Why would we need that?
Right now, you can't really tell what projects were recently added if you're not checking out the PRs on GitHub.
I believe it would a lot of value and would be just a cool feature.
Cheers,
Nicolas

I think this is a great suggestion. If we want to do this and assuming that people add projects at the end of the array (which is usually the case) we would need to look at the last x entries and list them with a dedicated category tag. That's all.

nice poll, I mean PR: #749


White header flashes on page loading

The page header component has a white default background. This background makes a "flash" during the page loading, so it will be good to remove it.

Fixed in v0.1.24.


Need better documentation

Issue description
Need better documentation
Steps to reproduce and a minimal demo of the problem
Not possible to find good articles, documentation how to use in Angular apps other than how to add agm to a template.  It would be benefitial to add examples how to draw location tracking line as user moves etc.

Document please


[Feature Request] Custom clean up schedule (rate)

Would you be willing to accept a pull request to parameterize the schedule for invoking the Lambda function in your CloudFormation template that is currently hard coded to rate(1 hour)? (Template.yaml line 41)
The proposed change would be to add an optional parameter with the default of the current value (rate(1 hour)).
This would allow us to invoke it much less frequently for some scenarios where changes are less frequent.

@afittz sure, that's a good idea, go ahead!

3 years later - A custom schedule (rate) would be really great.


[Security] Rebuild on Rust >= 1.58.1 to address CVE-2022-21658

Rebuild and re-release on rust >= 1.58.1
https://blog.rust-lang.org/2022/01/20/cve-2022-21658.html
This CVE is only relevant for privileged contexts.
If you are running wash in a privileged context and want to be sure to avoid this vulnerability, a work-around is to upgrade to the latest rust ('rustup') and rebuild wasah from source with cargo build --release

Fixed in wash 0.9.1


Another malformed URL exception

Describe the bug
This exception is thrown on malformed file URL:
type 'Null' is not a subtype of type 'String'
#0      CloudStoragePublicEndpoint.register.<anonymous closure> (package:serverpod/src/cloud_storage/public_endpoint.dart:119:61)
public_endpoint.dart:119
#1      EndpointDispatch.handleUriCall (package:serverpod/src/server/endpoint_dispatch.dart:128:37)
endpoint_dispatch.dart:128
<asynchronous suspension>
#2      Server._handleRequest (package:serverpod/src/server/server.dart:303:18)
server.dart:303
<asynchronous suspension>

The problematic line uses a dynamic value (which is Null in a malformed URL) and passes it as an arg to a String-typed parameter:
            return file(session as MethodCallSession, params['path']);

I'll copy here the comment I made in #2398:

There is a more general pattern here though: this is about the third issue I have filed for the handling of malformed HTTP requests. One of them was for URLs that were not formatted correctly to be interpreted as an endpoint call (also triggered by rogue crawler requests). This one and one other (IIRC) is about malformed UTF8 in the request.
You really need to stress-test your API server, webserver, and web socket upgrade handler by fuzzing them, so that unexpected exceptions are not thrown. This sort of issue does not instill confidence in Serverpod (even though Dart is a pretty safe language in general).

Basically, you need to assume that every part of every request could be wrong or invalid. There aren't enough validity checks being run right now.

This issue was fixed here: #2676
Specifically this is the code that now validates all parameters passed to endpoints:
https://github.com/serverpod/serverpod/blob/aeba3a8d126a2f4eb814bbb772ae71f9a36fa237/packages/serverpod/lib/src/server/endpoint_parameter_helper.dart#L8-L32


Deprecate registration of the span names for zpages.

Codecov Report

Merging #1589 into master will increase coverage by 0.02%.
The diff coverage is 75%.

@@             Coverage Diff              @@
##             master    #1589      +/-   ##
============================================
+ Coverage     82.16%   82.18%   +0.02%
  Complexity     1758     1758
============================================
  Files           261      261
  Lines          8107     8102       -5
  Branches        772      772
============================================
- Hits           6661     6659       -2
+ Misses         1211     1208       -3
  Partials        235      235

Impacted Files
Coverage Δ
Complexity Δ

.../main/java/io/opencensus/trace/EndSpanOptions.java
100% <ø> (ø)
3 <0> (ø)
:arrow_down:

...us/implcore/trace/export/SampledSpanStoreImpl.java
92.85% <ø> (ø)
3 <0> (ø)
:arrow_down:

...re/trace/export/InProcessSampledSpanStoreImpl.java
94.04% <ø> (ø)
19 <0> (ø)
:arrow_down:

...a/io/opencensus/trace/export/SampledSpanStore.java
92.53% <ø> (ø)
2 <0> (ø)
:arrow_down:

...r/stats/stackdriver/StackdriverExporterWorker.java
72.03% <100%> (-0.7%)
19 <0> (ø)

...race/stackdriver/StackdriverV2ExporterHandler.java
88.73% <100%> (+0.05%)
40 <1> (ø)
:arrow_down:

...ter/stats/prometheus/PrometheusStatsCollector.java
70.27% <100%> (-0.79%)
9 <0> (ø)

...io/opencensus/contrib/zpages/ZPageHttpHandler.java
45.16% <25%> (+7.66%)
5 <0> (ø)
:arrow_down:

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 eca2d29...261d5fe. Read the comment docs.

SampledSpanStore#registerSpanNamesForCollection says it's deprecated and we should use EndSpanOptions#getSampleToLocalSpanStore(). But EndSpanOptions#getSampleToLocalSpanStore() is marked @Experimental and says we should use SampledSpanStore#registerSpanNamesForCollection.
Which is accurate?


DOCS-6373: Documentation enhancements for slice, indexOf, and indexWhere functions (in DW Epic)

slice
Both drop and take describe what happens when a client provides numeric input that is outside of the bounds of the array. slice doesn't have this in its documentation.
Here's a draft I've put together for a blog I'm writing that describes this functionality. Assuming  arr = [1,2,3,4,5]:

You may ask yourself what the rules are if you specify indexes that are out-of-bounds. For example, what happens if we take the arr defined above, but try to slice up to index 20?
slice(arr, 1, 20)
// Returns: [2,3,4,5]

So slice doesn't choke on that. Instead, it exhibits the same behavior as take.
What happens if we take the arr defined above, but try to begin the slice with a negative index?
slice(arr, -5, 3)
// Returns: [1,2,3]

So slice won't choke on that, either, exhibiting the same behavior as drop.
Finally, what happens if we completely mix up the inputs, so that the first index we provide is greater than the second index?
slice(arr, 10, 1)
// Returns: []

In this case, we get an empty Array.

indexOf and indexWhere
The documentation of these functions does not disclose what the client should expect if there is no match at all. A brief test showed that if there is no match found, both indexOf and indexWhere will return -1.

Sorry for the delay, we have added a Jira ticket to take up this Issue. Thanks and please feel free to provide any and all suggestions to us!

https://www.mulesoft.org/jira/browse/DOCS-6373

Added to DW Q3 Epic: https://www.mulesoft.org/jira/browse/DOCS-8704

Jira is closed now: https://www.mulesoft.org/jira/browse/DOCS-6373


Started getting Failed to activate package named 'fixmyjs' Error: Cannot find module 'fixmyjs'

Although this used to work for me (and be awesome), it has stopped working recently. I tried uninstalling it and reinstalling it, but no dice. I globally installed fixmyjs via npm, and although I can not run it on the command line, I still can't use the Atom version. I'm not sure if something else I installed messed it up, or what.
Failed to activate package named 'fixmyjs' Error: Cannot find module 'fixmyjs'
  at Module._resolveFilename (module.js:344:15)
  at Function.Module._resolveFilename (/Applications/Atom.app/Contents/Resources/app/src/module-cache.js:383:52)
  at Function.Module._load (module.js:286:25)
  at Module.require (module.js:373:17)
  at require (module.js:392:17)
  at Object.<anonymous> (/Users/gears/.atom/packages/fixmyjs/index.js:2:15)
  at Module._compile (module.js:468:26)
  at Object.Module._extensions..js (module.js:486:10)
  at Module.load (/Applications/Atom.app/Contents/Resources/app/node_modules/coffee-script/lib/coffee-script/register.js:45:36)
  at Function.Module._load (module.js:318:12)
  at Module.require (module.js:373:17)
  at require (module.js:392:17)
  at Package.module.exports.Package.requireMainModule (/Applications/Atom.app/Contents/Resources/app/src/package.js:623:34)
  at Package.module.exports.Package.activateConfig (/Applications/Atom.app/Contents/Resources/app/src/package.js:250:12)
  at Package.module.exports.Package.activateNow (/Applications/Atom.app/Contents/Resources/app/src/package.js:232:14)
  at /Applications/Atom.app/Contents/Resources/app/src/package.js:682:29
  at Emitter.module.exports.Emitter.emit (/Applications/Atom.app/Contents/Resources/app/node_modules/event-kit/lib/emitter.js:82:11)
  at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app/src/command-registry.js:224:20)
  at /Applications/Atom.app/Contents/Resources/app/src/command-registry.js:3:61
  at CommandPaletteView.module.exports.CommandPaletteView.confirmed (/Applications/Atom.app/Contents/Resources/app/node_modules/command-palette/lib/command-palette-view.js:120:32)
  at CommandPaletteView.module.exports.SelectListView.confirmSelection (/Applications/Atom.app/Contents/Resources/app/node_modules/go-to-line/node_modules/atom-space-pen-views/lib/select-list-view.js:336:21)
  at space-pen-li.<anonymous> (/Applications/Atom.app/Contents/Resources/app/node_modules/go-to-line/node_modules/atom-space-pen-views/lib/select-list-view.js:140:19)
  at HTMLOListElement.jQuery.event.dispatch (/Applications/Atom.app/Contents/Resources/app/node_modules/archive-view/node_modules/atom-space-pen-views/node_modules/space-pen/vendor/jquery.js:4676:9)
  at HTMLOListElement.elemData.handle (/Applications/Atom.app/Contents/Resources/app/node_modules/archive-view/node_modules/atom-space-pen-views/node_modules/space-pen/vendor/jquery.js:4360:46)

Looks like an npm install failure where the fixmyjs dependency doesn't get installed. I can reproduce this too. But if I go to ~/.atom/packages/fixmyjs and run npm install manually it works.
Open a ticket on https://github.com/atom/atom as this isn't really specific to this package.

@sindresorhus Can you please point me to the correct URL in windows? I am just getting a file atom-1.0.2-delta.nupkg.

Perhaps mention this work around in the Install section of the README ;)

@sindresorhus - thanks that worked.

FYI I'm still having this issue with the latest stable atom (v1.6.0).


proposal: Pass CoroutineExceptionHandler

Currently, it is not possible to catch exceptions during .runSuspend.
https://github.com/davidepianca98/KMQTT/blob/fcef2ca6d84550235f3e62ad5074af419d6238f2/kmqtt-client/src/commonMain/kotlin/MQTTClient.kt#L429-L433
My suggestion is to be able to pass CoroutineExceptionHandler.

Hello, thanks for the suggestion. Would you be able to open a pull request with the change? Otherwise I'll make it as soon as I can but it might take a while.

Implemented here https://github.com/davidepianca98/KMQTT/commit/9fa3d003d2b0d1f74f448966e211684aee2a5789


Use gshowsPrecWithoutRecordSyntax for some data types

This commit adds a generics-deriving-based function that derives a showsPrec for record types that renders as if the fields were plain constructor arguments. This makes printed exceptions, traces, etc much more legible -- so much so that I've been repeatedly patching these instances manually whenever debugging the consensus tests. I decided to open this PR in case any of you agree it's worthwhile.
See the commit message for some more context.
I defined this function in ouroboros-network because that was the deepest package I wanted to use it in. It feels generic enough to be deeper, but I'm unsure where else that might be.

@nfrisby it could go in cardano-prelude which would make it available to the fill stack of libraries execpt ouroboros-network.

@erikd Thanks for the suggestion, but I'm unable to connect the dots myself. Why can't ouroboros-network depend on cardano-prelude? Since it can't, what would I do for the places I'm using it in ouroboros-network? Thanks.

I'm unsure as to whether there's a particular reason that ouroboros-network shouldn't depend on cardano-prelude. Maybe @dcoutts or @coot know if there's a good reason to avoid this?
I'm pretty sure ouroboros-network will have to depend on cardano-prelude pretty soon anyways in order to utilize Edsko's utilities for facilitating normal form assertion tests..

ouroboros-network already depends on cardano-prelude

I opened input-output-hk/cardano-prelude#89 to define the generic function in cardano-prelude. Then I'll update this PR to use it.
(What is the process for updating this repo's tag for cardano-prelude? Just open a PR? Notify anyone? Thanks.)

I opened input-output-hk/cardano-prelude#89 to define the generic function in cardano-prelude. Then I'll update this PR to use it.
(What is the process for updating this repo's tag for cardano-prelude? Just open a PR? Notify anyone? Thanks.)

In both stack.yaml and cabal.project, change the commit hash for cardano-prelude so that it points to a commit hash (on master) that is after merging the PR (HEAD should be fine). The same has to be done for nix using the ./nix/regenerate.sh script, but that requires a nix installation. If you don't have that available, CI (buildkite) will fail and include a patch to fix the nix configuration.
There's also https://github.com/input-output-hk/cardano-repo-tool to help with updating stack.yaml and cabal.project, but that may be overkill for updating a single dependency.

I've finally gotten around to updating this PR after waiting for the generic stuff to be in the pinned version of cardano-prelude.

I forced pushed an up-to-date set of changes that just uses quiet. It was already in stack.yaml.

Looks like I've just got the touch! That CI failure is the first CI failure I'm aware of that's due to Issue #2047 (which I had only found locally, before this). I've confirmed that the new commit on PR #2048  fixes this particular QC counterexample.
prop_simple_real_pbft_convergence ProduceEBBs (SecurityParam 9) TestConfig
            { numCoreNodes = NumCoreNodes 5
            , numSlots     = NumSlots 91
            , nodeJoinPlan = NodeJoinPlan (Map.fromList [(CoreNodeId 0,SlotNo {unSlotNo = 0}),(CoreNodeId 1,SlotNo {unSlotNo = 2}),(CoreNodeId 2,SlotNo {unSlotNo = 8}),(CoreNodeId 3,SlotNo {unSlotNo = 8}),(CoreNodeId 4,SlotNo {unSlotNo = 8})])
            , nodeRestarts = noRestarts
            , nodeTopology = meshNodeTopology (NumCoreNodes 5)
            , slotLength   = slotLengthFromSec 1
            , initSeed     = Seed<PHONE_NUMBER>431640712,11973858248747496943,11215859779704886592,15702589283301240594,17902124897060132005)
            }

So now PR #2048 is blocking this PR.

I still don't understand what purpose Quiet serves here at all. Why not just
newtype ByronHash = ByronHash { unByronHash ::CC.HeaderHash }
   deriving newtype (Show, Read)

Done?

I still don't understand what purpose Quiet serves here at all. Why not just
newtype ByronHash = ByronHash { unByronHash ::CC.HeaderHash }
   deriving newtype (Show, Read)

Done?

Quiet will still include ByronHash  in the output, whereas the newtype derived output won't.

I still don't understand what purpose Quiet serves here at all.

To abstract a bit from Thomas' answer: we greedily both want more legible output from QuickCheck (etc) (and Tracers!) and also the ability to paste that output as code. That's the high-level motivation.
Erik formerly pointed out the most reliable option for "pasting as code" is to invoke read. I haven't adopted that myself, since I (personally) want my typos at compile-time even for tests. But it makes sense and would at least be preferable compared to having to do a few imports and then prefix each occurrence of fromList in the copy-pasted string.

@nfrisby Still relevant?

Yes, still relevant. But minor and bitrot -- it has suffered from its low priority.
And it would technically be changing master -- eg I imagine it might change the format of some log messages. So I'm just going to Close it for now. I'll open a smaller-easier-to-merge PR every time I come across a non-quiet Show instance in the future.


Refactor infra-networking into a module and per-environment projects

Why I am making this change
We wanted to refactor the terraform projects, deployable via Makefile commands, into modules and per-environment projects so that we can just use terraform apply everywhere and not have to wrap it.
We've started with infra-networking as it's the foundation of all of the others.
What approach I took
We've created a generic observe-dev bucket, with <projectname>-dev statefile, and the same naming convention for the statefiles in other environments, to avoid blatting the existing ones when we apply this.
There is still a question mark over how we spin up dev stacks, whether people have to change these bucket and tfstate values for themselves, but that's a question for another time.
Paired with @matthewcullum-gds.

The terraform state mv was tedious, but we did it! It involved commands like:
aws-vault exec gds-prometheus-staging -- terraform state mv -state=../infra-networking/infra-networking-staging-project-state.json -state-out=./infra-networking-staging-module-state.json aws_eip.nat module.infra-networking.module.vpc.aws_eip.nat


PSA: Use extensions.experimental.affinity to speed up extension

VSCode allows extensions to run in their own thread. This improves productivity, and also is more important as of recent PRs.
Add to settings.json:
"extensions.experimental.affinity": {
    "vscodevim.vim": 1,
    "asvetliakov.vscode-neovim": 1
},

Next on my PR list is to make the extension automatically detect if this is set, and prompt if not.

I'm pretty sure that the "vscodevim.vim" field is for setting the affinity for the VSCodeVim extension, so if a prompt is added to this extension, it should only add the "asvetliakov.vscode-neovim" field.
The affinity setting can be verified by running "Developer: Show Running Extensions" in the Command Palette ("local process #"):

You are correct, I did know that but didn't think it was an issue. I didn't think it would hurt to include both as they both benefit from this setting and then you don't have duplicates in settings.json.
However, I can see how this can be confusing, and will delete that line when I have the chance.

in the window render log for vscode there are these lines:
2022-11-07 21:02:03.310 [info] Ignoring configured affinity for 'asvetliakov.vscode-neovim' because the extension is unknown or cannot execute for extension host kind: LocalWebWorker.
does anyone else have this?

I've found one issue when enabled this option. Ctrl+Backspace which usually removes the whole word in insert mode now removes only 1 character. Another thing is that "keyboard spam" is creating a noticeable cursor lag (might be just my laptop). Not sure if the cursor issue is solvable but word removal could be

I've found one issue when enabled this option. Ctrl+Backspace which usually removes the whole word in insert mode now removes only 1 character. Another thing is that "keyboard spam" is creating a noticeable cursor lag (might be just my laptop). Not sure if the cursor issue is solvable but word removal could be

Strange, I have no idea how the setting would cause these issues. Are you sure it’s not caused by the recent plugin updates? That is more likely

Small update, I've updated extension and both issues are fixed. Had couple system reboots since then, it was probably something on my part. Keyboard spam is not noticable anymore and ctrl+Backspace works as expected with extension running as a separate process.

I am having this issue (Ignoring configured affinity for 'asvetliakov.vscode-neovim' because the extension is unknown or cannot execute for extension host kind: LocalWebWorker.) and a very noticeable lag that seemingly starts when I alt-tab to another window, then back to VSCode, and it feels like it just gets worse over time after that. I end up having to ctrl palette -> Restart Extension Host / Reload Window every few minutes.

Details
VSCode
Version: 1.75.0-insider
Commit: 6ed4d436a9b6ae131732f4e2a723868231f4473a
Date: 2022-12-23T05:28:27.642Z
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 6.0.12-76060006-generic
Sandboxed: Yes

VSCode Neovim  v0.0.95

@laoshaw the plugin id is asvetliakov.vscode-neovim , not vscodevim.vim .


Download complete message shows 'Connection error'

Describe the bug:
Clicking 'download complete message' brings up a 'connection error' toast, but after going back to the folder and returning to the message, the complete message is shown.
Environment (please complete the following information):
K-9 Mail version: 5.725-SNAPSHOT
Android version: 7.0
Device: Android emulator
Account type: IMAP

Works for me. Can you provide a debug log?


Feature request: add JSDependency.minified()

One of the most common problems with SJS at the moment is its handling of minified JavaScript dependencies.  When you want to use an existing JavaScript library, you generally want to use the "raw" form during development, so you can trace into it during debugging (in, eg, the Chrome debugger), but the "minified" form during production, for efficiency.  Most JS libraries ship both forms, and it is assumed in the JS world that you use them this way.
But we have no elegant way of representing that in build.sbt, so folks often wind up writing duplicate jsDependencies trees for development and production.  Worse, there is no good way for a facade library such as scalajs-jquery to declare both forms such that the consuming application can use them, requiring considerable hackery to work around.  This is inconvenient for anyone who really delves into the JS during debugging, and I expect it to be an adoption hurdle for folks coming from the JS side of things -- they generally care about this a lot.
So I propose that we should add a new clause to JSDependency, named minified(). This would be parallel to dependsOn() and commonJSName(), and look something like this in practice:
jsDependencies += "org.webjars" % "jquery" % "2.1.3" / "jquery.js" minified "jquery.min.js"

This "minified" name would get carried through the jsDependencies pipeline, including from library to application, but would be largely invisible to most of the code; in particular, the "raw" name would always be used for dependency calculation.
Along with this, we would add a Setting named something like useMinifiedJS. The only place where minified would matter is in packageJsDependencies -- if useMinifiedJS is set to true, it would use the minified path instead of the "raw" one.
I'm sure that there are some subtleties I'm missing, but in principle this seems right: it ties the minified version closely to the "raw" one, in such a way that the dependency code can completely ignore it, removing the need for the sort of completely alternate jsDependencies trees that we see in, eg, @ochrons' tutorial. This should make things easier to reason about, and reduce bugs. It would allow libraries to declare both forms easily. And it should be straightforward enough to add useMinifiedJS to folks' Dev vs. Prod pathways in their build.sbt files, the same way they choose fastJSOpt vs. fullJSOpt.

I think that makes sense. It is also plausible to implement this in a backwards compatible manner, as the fallback if no minified exists, should be the normal version anyway.
The dependency tree and linearization would in any case be calculated based on the non-minified versions. I hope this is acceptable.
@sjrd opinions?

Sounds good and well thought-of. :-)

The dependency tree and linearization would in any case be calculated based on the non-minified versions. I hope this is acceptable.

I can't think of any cases where that isn't correct.  (Indeed, I would worry about any library where it wasn't...)

Actually there is one minor point I'm not so sure of: Should we have two different packageJSDependencies tasks (just like we have fastOptJS and fullOptJS), or should we make it depend on another setting?
The thing which is ugly with the setting, is that the file contents of <name>-jsdeps.js depend on a setting (which is IIUC not optimal).
Another option would be to change the artifact path to <name>-jsdeps.min.js if we are in useMinifiedJS mode...
WDYT?

Or simply have packageJSDependencies systematically produce both <name>-jsdeps.js and <name>-jsdeps.min.js.

But I think having two tasks is better.

Hmm.  I don't think I have a passionate opinion here.  Having a second task adds slightly to the boilerplate, but only to a constant degree, and having a separate -jsdeps.min.js means that, eg, my .html file goes from saying:
    <script src="@routes.Assets.at("querkiclient-jsdeps.js")"></script>

    @if(isProd) {
      <script src="@routes.Assets.at("querkiclient-opt.js")" type="text/javascript"></script>
    } else {
      <script src="@routes.Assets.at("querkiclient-fastopt.js")" type="text/javascript"></script>
    }

to
    @if(isProd) {
      <script src="@routes.Assets.at("querkiclient-jsdeps.min.js")"></script>
      <script src="@routes.Assets.at("querkiclient-opt.js")" type="text/javascript"></script>
    } else {
      <script src="@routes.Assets.at("querkiclient-jsdeps.js")"></script>
      <script src="@routes.Assets.at("querkiclient-fastopt.js")" type="text/javascript"></script>
    }

So long as the generated launcher does the right thing (for those who are using it), it seems likely to be fine...

Did you decide not to add the "useMinifiedJS" option?

@gzmo did the work, but yeah - the general sense was that using the minified version in fullOpt is essentially always correct, so it's automatic, IIRC...

There are two files generated: xyz-jsdeps.js and xyz-jsdeps.min.js. The latter may be generated only by fullOptJS by default, IIRC. Just include the one you want in your HTML.


Unknown exception in the log

The problem
I do not experienced any issues - just this log entry
What version of this integration (ha-nest-protect) has the issue?
0.4.0b5
What version of Home Assistant Core has the issue?
2024.9.1
Device / Model
Nest Protect (Battery) Firmware: 3.5rc3
Diagnostics information
No response
Home Assistant log
Logs
Traceback (most recent call last):
  File "/config/custom_components/nest_protect/__init__.py", line 208, in _async_subscribe_for_data
    dict(b, **buckets.get(b.object_key, {})) for b in [data.updated_buckets]
                          ^^^^^^^^^^^^
AttributeError: 'list' object has no attribute 'object_key'

Additional information
No response

Duplicate of #347


Resolved merge conflicts, among other things

The camera was already commented out, you're gonna need to upload another PR just for autonomous


VRChat：Crashes when moving to a world that plays video.

It's just like the title says.
It crashes in autoplay worlds, and it also crashes if I enter an instance of a world with a video player where someone else is playing a video.
(Cozy Apartment, for example).

As it is a free game i can look into this and debug why it crashes. Thanks for the report.

I can confirm that it crashes with the current reshade build.

Fixed with the latest v0.9.7-pre release version.


memory leak when running kwok nodes/pods multi hrs/days

How to use it?

[ ] kwok
[ ] kwokctl --runtime=docker (default runtime)
[ ] kwokctl --runtime=binary
[ ] kwokctl --runtime=nerdctl
[ ] kwokctl --runtime=kind

What happened?
Run kwok in cluster and monitor memory usage and found memory is keep increasing after multi hrs:

kwok running on node: AWS EC2 c6i.8xlarge (64 GB memory)
simulate 2000 fake nodes and 20k fake pods
monitor memory usage from cloudwatch and top
found memory usage keep increasing
at the beginning of fake nodes/pods running: kwok memory usage is 0.8%, after overnight changed to 11.2%, after 2 days change to 28.2%

What did you expect to happen?
memory shouldn't keep increasing when fake nodes/pods running
How can we reproduce it (as minimally and precisely as possible)?
repro steps included in "What happened";
yaml for nodes:
cat kwok-node.yaml
apiVersion: v1
kind: Node
metadata:
  annotations:
    node.alpha.kubernetes.io/ttl: "0"
    kwok.x-k8s.io/node: fake
  labels:
    beta.kubernetes.io/arch: amd64
    beta.kubernetes.io/os: linux
    kubernetes.io/arch: amd64
    kubernetes.io/hostname: kwok-node-0
    kubernetes.io/os: linux
    kubernetes.io/role: agent
    node-role.kubernetes.io/agent: ""
    type: kwok
  name: kwok-node-0
spec:
  taints: # Avoid scheduling actual running pods to fake Node
  - effect: NoSchedule
    key: kwok.x-k8s.io/node
    value: fake
status:
  allocatable:
    cpu: 32
    memory: 256Gi
    pods: 110
  capacity:
    cpu: 32
    memory: 256Gi
    pods: 110
  nodeInfo:
    architecture: amd64
    bootID: ""
    containerRuntimeVersion: ""
    kernelVersion: ""
    kubeProxyVersion: fake
    kubeletVersion: fake
    machineID: ""
    operatingSystem: linux
    osImage: ""
    systemUUID: ""
  phase: Running

yaml for pods:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: fake-pod-${DEPI}
  namespace: fake-pod
spec:
  replicas: ${REP}
  selector:
    matchLabels:
      app: fake-pod-${DEPI}
  template:
    metadata:
      labels:
        app: fake-pod-${DEPI}
    spec:
      affinity:
        nodeAffinity:
          requiredDuringSchedulingIgnoredDuringExecution:
            nodeSelectorTerms:
            - matchExpressions:
              - key: type
                operator: In
                values:
                - kwok
      # A taints was added to an automatically created Node.
      # You can remove taints of Node or add this tolerations.
      tolerations:
      - key: "kwok.x-k8s.io/node"
        operator: "Exists"
        effect: "NoSchedule"
      containers:
      - name: fake-container
        image: fake-image

Anything else we need to know?
No response
Kwok version

kwok-controller:
    Container ID:  containerd://ba8bfdfa5cd0181f845a901984a1498a0b701f622540e4e03357e1cd348e03b0
    Image:         registry.k8s.io/kwok/kwok:v0.6.0

OS version

cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04.5 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.5 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy

Good catch, thank you!


Updates for July 2020 course

Let me know if the schedule looks good to you, @twiecki
Removed torch from environment, since we won't be using it.

Small changes, assigned MCMC basics to me and gradient-based to you if that's OK.


feat: ctrl + o to insert a line break

I’v been using oatmeal for the past several days and have found it  to be quite beneficial. However, I believe that the chat functionality could be improved  by allowing the insertion of line breaks.
My intention was to configure a key binding using Shift + Enter to insert a line break, but unfortunately, the application does not seem to recognize this.
Additionally, I've tried setting Ctrl + M and Ctrl + Enter as potential key bindings, but these too aren't detected as intended. Instead, the application appeared to recognize them as a simple Enter key press, which results in the message being sent.(Possibly, this could be due to the specifications of tui-textarea...?)
Thank you for considering this enhancement.

Ah this is a nice addition, thanks!
Yes the issue is around tui-textarea which doesn't support nice multi line inputs yet. It's being tracked in https://github.com/dustinblackman/oatmeal/issues/16 and https://github.com/rhysd/tui-textarea/issues/5.

Thank you for reviewing and accepting my suggestion!
Also, thank you for releasing new version so quickly!
I'm so happy that I can use the new version so soon.


yarn build fails

The current installation sequence throws the following error when running yarn build:
$ python -m build
/usr/local/anaconda3/envs/your-package-name/bin/python: No module named build
error Command failed with exit code 1.

If you encounter this error, simply run the following:

Handle by commit 7a53eb1


Not able to run web tests via Zebrunner's launcher on SauceLabs selenium grid

Description
An attempt to launch web tests on SauceLabs selenium grid fails with the following error:
org.openqa.selenium.WebDriverException: Unable to parse remote response:
Misconfigured -- Mixed Capabilities Error.
W3C keys (platformName/browserVersion) were detected alongside JWP keys (platform/version).
To fix this, replace all JWP keys with W3C keys.
The following desired capabilities were received:
{'acceptInsecureCerts': True,
 'acceptSslCerts': True,
 'browserName': 'chrome',
 'carinaTestRunId': 'XXXXXXXX-XXXX-XXXX-XXXX-XXXXb63bce55',
 'chrome.switches': ['--start-maximized',
                     '--ignore-ssl-errors'],
 'enableVideo': True,
 'goog:chromeOptions': {'args': ['test-type',
                                 '--lang=en_US'],
                        'extensions': [],
                        'prefs': {'intl.accept_languages': 'en_US'}},
 'name': 'default',
 'platformName': '*',
 'provider': 'SAUCELABS',
 'takesScreenshot': False,
 'version': ''}

Steps to reproduce

Obtain credentials to  SauceLabs account. You can either register a new account or contact me to get my credentials.
Register new workspace in Zebrunner.
Configure SauceLabs integration in Zebrunner with credentials from point 1.
Navigate to 'test run' page in Zebrunner
Open Launcher popup
Select 'Carina WEB' suite and launch it.

Actual results: all the tests from the suit fail with the error depicted above.
Expected results: test suit executed successfully
Notes
I've made a trivial attempt to fix the issue. My changes can be found here - https://github.com/qaprosoft/carina/commit/746c841f05c29d451f3b537a558f6f2e9b24c131

@DenisPitsul2020, let's reorganize AbstractCapabilities.java logic in such way:

capabilities.platformName has higher priority. If it is available we shouldn't set just platform

capabilities.setPlatform(Platform.extractFromSysProperty(platform));

capabilities.browserVersion has higher priority. If it is available we shouldn't set just version

        // Selenium 3.4 doesn't support '*'. Only explicit or empty browser version should be provided
        String browserVersion = Configuration.get(Parameter.BROWSER_VERSION);
        if ("*".equalsIgnoreCase(browserVersion)) {
            browserVersion = "";
        }
        capabilities.setVersion(browserVersion);

do not setVersion if it is empty

Resolved: https://github.com/qaprosoft/carina/pull/1297,
you can test it on 7.0.3-SNAPSHOT

closing the ticket to start QA validation


Anonymize cmd

Related to issue #4982 .
@agramfort I wonder if we should overwrite a file by default or rather implement some warning with y/n question for the user (+ extra option -f force to avoid that).

I think we should stick to the historical API which asks that we use the
--file option.
mne anonymize --file foobar_raw.fif
now it's the question of --overwrite and the use of prefix if overwrite is
False
I don't think we should overwrite by default also.

Thanks, fixed

BTW I know that it might be a subject for another PR, but since you asked me about changing input fif from argument to option, I noticed some inconsistency in command tools:
$mne show_info --help
Usage: mne show_info <file>

Show measurement info from .fif file.

$mne browse_raw --help
Usage: mne browse_raw [options]

Browse raw data.

Options:
  --version             show program's version number and exit
  -h, --help            show this help message and exit
  --raw=FILE            Input raw FIF file

$mne anonymize --help
Usage: mne anonymize [options]

Anonymize .fif file.

Options:
  --version             show program's version number and exit
  -h, --help            show this help message and exit
  -f FILE, --file=FILE  the file to modify.

Each command takes input fif in a different way...

you can anonymize more than raw files technically. Any data container with info could be anonymized.

Well, but it doesn't explain why in $mne show_info it goes with argument and not with -f option. But anyway, that's just nitpicking remark.

I guess I wanted to stick to old API so little code would need to be
changed to use mne-python.

Okay, got it.

Closing this for now since it seems like we ultimately need a different solution. Thanks for working on this though @dokato


⚠ Bofrost.fr has degraded performance

In b81d99e, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1219 ms

Resolved: Bofrost.fr performance has improved in 4c2083e after 8 minutes.


Issue with loading dataset

After cloning the package, when I run make train I get an error. Following is the trace:
Downloading and preparing dataset spider/spider to /transformers_cache/spider/spider/1.0.0/3c571ccbabdc104a8d1f14edff4ce10d09d7724b0c3665fc42bec1ed51c84bf3...
Traceback (most recent call last):
File "seq2seq/run_seq2seq.py", line 270, in
main()
File "seq2seq/run_seq2seq.py", line 148, in main
tokenizer=tokenizer,
File "/app/seq2seq/utils/dataset_loader.py", line 89, in load_dataset
dataset_dict=_spider_dataset_dict(),
File "/app/seq2seq/utils/dataset_loader.py", line 45, in
path=data_args.dataset_paths["spider"], cache_dir=model_args.cache_dir
File "/opt/conda/lib/python3.7/site-packages/datasets/load.py", line 1699, in load_dataset
use_auth_token=use_auth_token,
File "/opt/conda/lib/python3.7/site-packages/datasets/builder.py", line 596, in download_and_prepare
dl_manager=dl_manager, verify_infos=verify_infos, **download_and_prepare_kwargs
File "/opt/conda/lib/python3.7/site-packages/datasets/builder.py", line 690, in _download_and_prepare
) from None
OSError: Cannot find data file.
Original error:
[Errno 20] Not a directory: '/transformers_cache/downloads/2328a961663d7b95ae1aa7a8f378034cc12f525cf9ce537792e7b0856332da67/spider/train_spider.json'
Do I need to do any extra processing to get Spider dataset?

Hi @PramuPerera,
No additional processing should be required. Perhaps the download failed somehow. Can you just run this again? Also, what platform are you on?

Hey @tscholak ,
I am also facing the same issue. Builder is trying to find this file /transformers_cache/downloads/e802648de711b5135807858200be8df74dc610c8606926d2c47de192c44790e9/cosql_dataset/sql_state_tracking/cosql_train.json but there is just this /transformers_cache/downloads/e802648de711b5135807858200be8df74dc610c8606926d2c47de192c44790e9 no files is created inside this.
and I see .lock file is created in the same directory.

And I tried deleting python lock files to see if it helps to download but no luck.
I am using aws-ec2(Deep Learning Base AMI (Ubuntu 16.04) Version 21.0)  machine as a host if it helps.
kindly provide if any solution comes to your mind, otherwise tomorrow I am gonna debug little bit details to nail down the issue.
Thanks,
Raghav

Hello! I am actually getting the same issue at the moment, even if before it worked just fine.
BR,
Elena

Hello! I am actually getting the same issue at the moment, even if before it worked just fine.
BR,
Elena

A temporary work around is to manually download spider.zip from the google drive and change the _URL in spider/spider.py.

I was able to reproduce the issue, but I have not yet had the time to investigate the cause of the error.

The issue was upstream and has been recently fixed, https://github.com/huggingface/datasets/issues/3786

I will update the image to include that fix.

I have merged a fix. It should work again.
I recommend removing the download cache, so that the zip file is downloaded again.
The issue was caused by a bug in huggingface's datasets library that caused the code to download the no-virus-check page instead of the zip file from the google drive.

Hi everyone,
I faced the same problem, it occurs due to the Google Drive virus check message.
Reference:
Fix: Bypass Virus Checks in Google Drive Links (CNN-DM dataset) #3785
To fix is simple just include  &confirm=t at the URL end.
File: picard/seq2seq/datasets/spider/spider.py
Near to Line 45
From:
_URL = "https://drive.google.com/uc?export=download&id=1_AckYkinAnhqmRQtGsQgUKAnTHxxX5J0
To
_URL = "https://drive.google.com/uc?export=download&id=1_AckYkinAnhqmRQtGsQgUKAnTHxxX5J0&confirm=t"

Hi @Marchanjo,
Thanks for sharing! With the fixes I pushed recently, the change you're proposing shouldn't be necessary anymore. Did you still run into problems with the latest image?

Hi @tscholak, thank you!
I did not test the latest image yet.

This issue has been fixed. If it still doesn't work for you, please pull the latest docker image(s) and delete the datasets cache in your $HF_HOME where huggingface saves its temporary files.


How to create eos wallet using eosjs

I want to perform
cleos create wallet -n 'hello'
using eosjs is this possible

eosjs does not implement a wallet..  It is only the API..


Iterating over notes to detect build ID

This PR changes the build ID detection to iterate over all ELF notes in a binary in order to find those of type NT_GNU_BUILD_ID.

This PR finds the build ID similar to how readelf does, like this.
We can now remove read_build_id_from_section_name, as its functionality is a subset of this patch's.

OK, I squashed my fixup commits and addressed your comments. Thanks!


publish crate on crates.io

did you consider publishing this as a crate like e.g. tree-sitter-swift
this is required so rust projects depending on this can be published on crates.io

did you already publish it?
because I can't find it

See https://crates.io/crates/tree-sitter-query


Manifest issue with polus-s3import-plugin/plugin.json

The plugin polus-s3import-plugin/plugin.json is missing fields: ('author',)

@Nicholas-Schaub There is no plugin with this name. Which plugin do you mean?

This is an old plugin that directly pulled in data from an S3 bucket. It was removed due to security concerns.


Home Page Movie Filter

Hey 👋, I am a Gssoc'23 contributor
Feature Request
I would like to contribute a card movie filter on the home page of screen which will help users to find out different genres of movie interactively. a filter is already there But I think that is not highly eye picking.
Screenshot

Kindly assign it to me under GSSOC'23
I will surely complete it ASAP

Thank You

I'd like to work upon this issue if this hasn't been done yet.Pls assign it to me

Hey @bhoomikhurana 👋 Thanks for your time But I am working on it ...

Home page is not loading !!
I don't know why it is happening I am trying to add feature through Github code spaces but What I found is the cards or different movie sections are properly loading. It is taking more time then expected.


Deploying from Travis

I'm attempting to deploy a site to Flynn from Travis, and I'm running in to a couple of showstoppers.

When attempting to install the CLI, there's a permission denied trying to write to /user/local/bin
Having renamed the filename to .flynn, trying to cluser add my cluster I'm getting an unknown cluster error

I've tried looking around, and I can't seem to find anyone deploying to Flynn from Travis or a working example. Here's my travis stuff:
language: node_js
node_js:
  - 'v5'

#########################
## Deploy
#########################
before_deploy: 'L=.flynn && curl -sSL -A "`uname -sp`" https://dl.flynn.io/cli | zcat >$L && chmod +x $L'

deploy:
  skip_cleanup: true
  provider: script
  script: sh ./.deploy.sh
  on:
    branch: master

#!/bin/bash

#########################
## Adapted from https://gist.github.com/domenic/ec8b0fc8ab45f39403dd
#########################
set -e # exit with nonzero exit code if anything fails

# Initialize Flynn
./.flynn cluster add -p ${FLYNN_CLUSTER} default ${FLYNN_URL} ${FLYNN_TOKEN}
./.flynn create ${FLYNN_NAME}

# Configure Git
git config user.name "Travis CI"
git config user.email<EMAIL_ADDRESS>
# Commit all the things into the repo
git init
git add .
git commit -m ":shipit: Deploy to Flynn"

# Force push to gh-pages
git push flynn master

In addition to that, if I get all the way down to just adding the remote and trying to push to Flynn, I can't because Travis fails at the cert validation that Flynn needs and self-generates

I'd like to hook in on this one with a quite general question: If I use a CI server (be it Travis), which ultimately builds the artefacts, wouldn't it be the best way that the CI deploys the artefacts on Flynn? I.e. push the (fat) jar to Flynn and not building it again?

Are the Travis builds public? Can you link to an example failure?

it could be very useful if someone writes an article on using any CI tool (Travis or otherwise) with Flynn.

@hadifarnoud Well the purpose of a CI is to have ongoing tests on new commits and pull-requests. This is not really affected how you deploy, however, since both CI and Build-Pack Style deployment require you to have a completely automated build, one leads to the other.
The build-pack (Flynn) needs to compile the source to generate a slug / container.
The CI needs to compile the source to execute the tests.
This basically means that the CI is invoking the same build and test tool in your repository as later the build-pack is. This works and we currently use it that way. Some CI status badges are in the repositories so we can easily track if the build is alright.
Since our build for the build pack also executes the tests (and aborts if the test fail) this is some double work going on.
Now, this is off-course just the Unit-Testing and limited integration testing. Having automated Integration Tests spanning multiple services deployed on Flynn would be another story. For now, we just go for a separate staging Flynn cluster and do tests manually.
If you or someone else has more thoughts on this I would be really interested.

I will write a small article tonight about how we handle CI/CD here with Jenkins and the CD towards Flynn.

Here we go: https://philiplb.de/flynn/2016/10/04/flynn-cd/

Thanks @philiplb!
https://twitter.com/FlynnScale/status/783428265977454592

Ou, nice, thank you. :)


Shinylive Python Question: Importing from subfolders

Curious if it's possible to import modules from local directory-based packages or folders. I feel like I was able to do this a couple of versions back, but I'm unable to now (it's also possible I'm just incorrectly remembering using regular py-shiny as well.
With larger apps/utilities, folders really help for organizing code (i.e., right now I have a utility that contains 28 other module files in the same directory as app.py), but I'm unable to figure out how to successfully import them (I've tried absolute imports, relative imports, regular folders, package folders, etc).
Any direction is helpful. Thanks!
(btw - really enjoying Shinylive)

Hi, can you provide a simple example app?

Sure thing. I will try to get something to you tomorrow.

My apologies for not providing an example app yet. I did discover the issue, and figure out why I felt like I "remembered" that it use to work.
On personal projects I use a MacBook, and importing from modules organized into folders works fine in Shinylive.
At work I use a Windows laptop, and that is where importing from folders runs into issues.
The crux of it is this:

On MacOS, since it's unix based, the path for the module in the shinylive app.js file is formatted with "/" between directory levels (i.e., module_dir/sub_dir/tool.py
On Windows, the path format in app.js is formatted with an escaped backslash (i.e., module_dir\\sub_dir\\tool.py)

When I change the "\" to a "/" in app.js after exporting the app, importing from modules in folders works (where before the browser error message says that it the module doesn't exist).
I didn't dig into your repo to see if the path is being generated in Python or something else like Javascript, but it seems like it's the kind of behavior created by pathlib.Path (i.e., because we develop on Windows and run on Linux servers we often use PurePosixPath on paths in our internal tooling where the a raw path string is passed directly to the server to be executed (since regular Path would send a Windows formatted path string).
The crude and kind of ugly tool I hurriedly threw together today as a temporary workaround is just a script that finds those locations in app.js and "fixes" the path to be a unix format. When I export the shinylive app, then run this script, my app with lots of modules nested in subfolders works.
import re
from pathlib import Path

filepath = Path(__file__).resolve().parent / "staging" / "app-name" / "app.json"

with open(filepath) as f:
    text = f.read()

def fix_path_slashes(match_obj):
    return re.sub(r"\\\\", "/", match_obj.group(1))

matches = re.findall(r'("name": "\S*.py", "content":)', text)
matches = [x for x in matches if "\\" in x]
if matches:
    print("FIXING:")
    for m in matches:
        print(">", m.replace('"name": "', "").replace('", "content":', ""))
    text = re.sub(r'("name": "\S*.py", "content":)', fix_path_slashes, text)

    with open(filepath, "w") as f:
        f.write(text)


JSON value 'user_session=abcdefg; path=/; expires=Thu, 1 Jan 2030 00:00:00 -0000; secure; HttpOnly' of type NSString cannot be converted to NSDictionary

JSON value 'user_session=abcdefg; path=/; expires=Thu, 1 Jan 2030 00:00:00 -0000; secure; HttpOnly' of type NSString cannot be converted to NSDictionary
+[RCTConvert NSDictionary:]
RCTConvert.m:55
__41-[RCTModuleMethod processMethodSignature]_block_invoke.135
-[RCTModuleMethod invokeWithBridge:module:arguments:]
facebook::react::invokeInner(RCTBridge*, RCTModuleData*, unsigned int, folly::dynamic const&)
facebook::react::RCTNativeModule::invoke(unsigned int, folly::dynamic&&, int)::$_0::operator()() const
invocation function for block in facebook::react::RCTNativeModule::invoke(unsigned int, folly::dynamic&&, int)
_dispatch_call_block_and_release
_dispatch_client_callout
_dispatch_lane_serial_drain
_dispatch_lane_invoke
_dispatch_workloop_worker_thread
_pthread_wqthread
start_wqthread

I followed your example and got this error
CookieManager.setFromResponse(
'http://localhost',
'user_session=abcdefg; path=/; expires=Thu, 1 Jan 2030 00:00:00 -0000; secure; HttpOnly')
.then((res) => {
alert(res)
// res will be true or false depending on success.
console.log('CookieManager.setFromResponse =>', res);
});

https://github.com/joeferraro/react-native-cookies/issues/15

@UZIhuhuhu Did u find out how todo this?

I have forked the repo here: https://github.com/safaiyeh/react-native-cookie-store
We can create issues and PRs there to continue the development of the project.


Computationally expensive

Hello.
kcptun is highly expensive in terms of cpu usage. How can I optimize it? I just need to forward a large number of tcp packets to another server through a udp tunnel and I don't care about encryption (so already set encryption to none). Any suggestions?

you can also disable FEC by setting -ps 0

@xtaci Should I also set -ds 0 ?

I did a quick test, it practically makes no difference, the test results are basically the same (with marginal differences), roughly 40Mb/s at 100% CPU(crappy vps), with -ps 0 or default.
Can confirm in log the parameter is set correctly:
Sep 18 21:51:56 Zeta client[5418]: 2022/09/18 13:51:56 datashard: 10 parityshard: 0

use hardware accelerated encryption algorithm, eg: aes

I’m using none in both tests.


Conflict between use of Security Defaults and Backup Administrator Accounts

Documentation does not make it clear that Backup Administrator Accounts can NOT be used when Security Defaults are enabled.
The section on Backup Administrator Accounts plus the linked page here - https://docs.microsoft.com/en-us/azure/active-directory/roles/security-emergency-access - suggest that MFA should be disabled for backup account.
Please amend this page to highlight that the advice around configuring backup accounts cannot be applied when using Security Defaults.
If I have misunderstood the situation and MFA can indeed be disabled on individual accounts when Security Defaults are in effect, please advise how this can be done.
Perhaps we can use backup accounts with Security Default enabled, but would have to accept that when they are used we will be prompted to configure MFA for those accounts.

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

ID: cfac0ecf-e5a9-c5ff-f8c2-ce27789708f5
Version Independent ID: 0f5e0cc9-a54f-a5c8-192d-d60a6a4db3ff
Content: Azure Active Directory security defaults
Content Source: articles/active-directory/fundamentals/concept-fundamentals-security-defaults.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@danwatford
Thanks for your feedback! We will investigate and update as appropriate.

@danwatford
Thank you for your detailed post!
When it comes to Security Defaults and Backup Admin Accounts (i.e. Global Admin), MFA can't be "disabled" for these accounts, but if you plan to leverage Conditional Access Policies then you'll need to disable Security Defaults first. Additionally, as mentioned within the - Manage emergency access accounts doc that you linked, it's recommended to "exclude" one Admin account from the CA policy to prevent being locked out of your tenant.
Can you share where you saw the suggestion(s) that MFA should be disabled for backup accounts, so I can gain a better understanding of your issue?
If you have any other questions, please let me know.
Thank you for your time and patience throughout this issue.

Hi @JamesTran-MSFT ,
I think we are all in agreement on the limitations of Security Defaults with regards to Backup Admin Accounts and MFA. I think the problem is that for a new person to the subject the documentation as currently presented is potentially misleading. I think this is evidenced by the previous issues raised in this repository on the same topic.
But first, to answer your question, this page - https://docs.microsoft.com/en-us/azure/active-directory/roles/security-emergency-access - has a section titled Exclude at least one account from phone-based multi-factor authentication. That page is linked to from the page this issue (#91538) relates to, Security defaults in Azure AD.
Putting the issue another way with a scenario, an administrator of a small organisation with a Microsoft 365 subscription may be pointed to Security Defaults as a good starting point to improve their security posture, per promotion from MS blogs and from documentation. This administrator will likely have no prior knowledge of Conditional Access and won't have purchased any AAD premium licenses to unlock that functionality.
While reading documentation page, Security defaults in Azure AD, the administrator reads the section, Who's it for?, and concludes that their organisation is a good fit for Security Defaults and that Conditional Access does not apply.
Reading further in the same document, we reach the section on Backup Administrator Accounts. This subsection is itself contained within subsection Deployment Considerations, so perhaps the authors thought this would indicate a different context for the reader. However when reading the documentation page these section changes can be quite subtle and will likely be missed on long pages.   Please remember this is a section titled Backup Administrator Accounts on a page titled Security Defaults in Azure AD where I think the reader think they are being advised to used backup administrator accounts at the same time as using Security Defaults.
The opening line of the Backup Administrator Accounts section is:

Every organization should have at least two backup administrator accounts configured

This tells the reader that they should be using backup accounts even if they are using Security Defaults. The document even provides the steps for creating the backup accounts.
The confusion sets in when the section on backup accounts directs the user to page Manage emergency accounts in Azure AD, and on that page the user is advised to disable MFA for backup accounts. This is the conflict in documentation.
I appreciate this has been a long and rambling post, so thank you for sticking with me this far! :)
Question 1:
Does Microsoft still advise every organisation have at least two backup administrator accounts WHILE ALSO using Security Defaults?
Question 2:
If the above answer is 'yes', then please can the documentation be amended to highlight that Security Defaults will cause MFA to be applied to those accounts and how this will impact the use of those accounts. In this case I would expect the user of the backup account to be directed to configure MFA on their mobile default at first use. This may then necessitate the creation of new backup accounts and deletion of the used backup account in order to ensure backup accounts exist which are not tied to any MFA device.
Question 3:
If the answer to question 1 is 'no', then to avoid confusion for the new administrator I would suggest making changes to the Security Defaults in Azure AD page similar to:

In section Who's it for? have a bullet point stating 'Organisations that do not use Backup Administrator Accounts'.
In section Backup administrator accounts remove the text after the first two paragraphs, replacing with text such as 'Backup administrator accounts cannot be used with Security Defaults. If you wish to use backup administrator accounts you should use Conditional Access policies.'

Author here...
Yes there are differences between emergency access accounts and what I called backup administrator accounts in the security defaults doc. Security defaults does not allow you to do exclusions but we link to the emergency access accounts doc if they want lots of extra detail for the future.
We are recommending people using security defaults add extra admin accounts because we see lots of instances of organizations only having one account with global administrator rights and getting locked out. These cases take a long time to resolve.
If you want to chat more feed free to ping me via email in my profile.
#reassign:@MicrosoftGuyJFlo
#please-close

Hi @MicrosoftGuyJFlo ,
Answering here rather than by email as I think a lot of people have been confused by this topic, evidenced by the number of other issues raised previously.
I think it is a good idea to draw a distinction between Backup Administrator Accounts and the 'full-blown' Emergency Access Accounts as described on the linked-to page.
Perhaps the confusion can be mitigated another way. On the section titled Backup Administrator Accounts, add a note highlighting that the treatment of these is slightly different to that of Emergency Access Accounts in that MFA cannot be disabled when Security Defaults are enabled. Explicitly stating this fact rather than the reader to put pieces of documentation together will save the new administrator a lot of time and confusion.
Another useful addition would be an explanation of what the user experience will be when someone needs to use a Backup Administrator Account. Presumably they will need to configure an MFA device when they first attempt to use the account. This will preclude anyone else from using the same backup account, so the recommendation should then be that a new backup account is created as a standby.

Heya @MicrosoftGuyJFlo
I am one of these "new administrators" and had exactly the confusion that @danwatford described.
I'm planning on creating a new global administrator with a preset password that I will store in a bank safe. Reading about the Security Defaults made me unsure how and when the MFA policy kicks in because - as I just created the Microsoft 365 tenant today - I haven't gotten any prompts to enable MFA on any account yet.
So, the proposed documentation changes by @danwatford would have saved me a lot of headaches.
By extension, wouldn't there be a use case for a flag or role "Emergency Admin" to be assigned to these accounts which could be the only exception to the Security Defaults' MFA policy? Let's say that an organization uses its emergency account at some point. I would argue that it's quite likely that the administrator that just "burned" the emergency account might forget to add a new, fresh global administrator in the heat of the moment.


fix some problems with rans3p with new pybind11/cArgumentsDict

This fixes some issues:

missing import of cArgumentsDict in RANS3PSed
missing definition of keyword arguments in SedimentClosure.cpp
several instances where 'arg' was used instead of 'argsDict'

@acatwithacomputer you may also need to update some of the input files for the sediment tests to 1) ensure that floating point numbers are used for floating point parameters,  otherwise you'll get an error that the parameter is not available (e.g. use rho_s = 1.0, not 1, etc.) 2) We got rid of the RDLS3P class because it was redundant, so you may need get errors trying to import RDLS3P--just change it to RDLS.
/cc @JohanMabille
Mandatory Checklist
Please ensure that the following criteria are met:

[ ] Title of pull request describes the changes/features
[ ] Request at least 2 reviewers
[ ] If new files are being added, the files are no larger than 100kB. Post the file sizes.
[ ] Code coverage did not decrease. If this is a bug fix, a test should cover that bug fix. If a new feature is added, a test should be made to cover that feature.
[ ] New features have appropriate documentation strings (readable by sphinx)
[ ] Contributor has read and agreed with CONTRIBUTING.md and has added themselves to CONTRIBUTORS.md

As a general rule of thumb, try to follow PEP8 guidelines.
Description

Arf sorry for missing it during the migration.


rustfmt causes internal error when add comment between conditions

source:
fn main() {
    if a
        // panic!
        && b
    {}
}

playground:
https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=856d116c793b74e6809441b4ccf7af68
error message:

This works now, I guess the issue can be closed


[PORT] Add exception to missing predicate condition

https://github.com/microsoft/botbuilder-dotnet/pull/4678

this issue doesn't not apply to js version of IfCondition, closing this.


express mutually exclusive features

I don't really understand what --group-features is doing just from the readme or help output so it's possible I'm using it wrong and that this is possible already.
I want to provide an optional TLS backend openssl or rustls but they emit conflicting code so using --feature-powerset is not working out yet.
TIA

I don't think there is currently a way to do this with a single command. (--group-features treats the specified features as if it were a single feature.)
If you are okay with splitting it into multiple commands, I think you can express this by combining --skip(--exclude-features) and --features.
# All combinations that contain openssl and do not contain rustls.
cargo hack build --feature-powerset --features openssl --skip rustls
# All combinations that contain rustls and do not openssl.
cargo hack build --feature-powerset --features rustls --skip openssl
# All combinations that do not contain both openssl and rustls.
cargo hack build --feature-powerset --skip openssl,rustls

https://github.com/taiki-e/cargo-hack/issues/132#issuecomment-947800980

If it is not possible, consider adding a compile error to detect this scenario.

Btw, #115 is a feature request to check that a proper compilation error occurs in such cases.


feat: 🎸 adds initial Jenkinsfile, for transitioning to jenkins artifact builds

Description
adds initial Jenkinsfile, for transitioning to jenkins artifact builds
Breaking Changes
NA
JIRA Link
NO
Checklist

[ ] Updated the Readme.md (if required) ?

the authenticate commit step needs your signing key set on an org level variable MIDDLEWARE_ALLOWED_SIGNERS. @adamdossa could probably add it for you if you shared your public key with him, or I can rebase this with my signature.
I restarted test 3, which seemed to fail due to a coverage upload endpoint not being available when you opened this.

/fast-forward

seems like its not blocking so I merged it in

:tada: This PR is included in version 26.2.0-alpha.2 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 26.2.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 26.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 26.2.0 :tada:
The release is available on:

npm package (@release-v26 dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[feature] SAML NameID Mapping

I'm trying to log in to Aliyun using Casdoor, but Aliyun requires that the NameID field in the SAML response matches the Aliyun RAM account (an email account), but the value of the NameID that Casdoor passes is the Casdoor username not the email address, and I can't find a place where I can configure this option.
Is it possible to add a feature for configuring application-specific NameID fetching or mapping?

@Oniokey can you make a PR?

@Oniokey can you make a PR?

I'm not sure at what point Casdoor processes the SAML response, can you give me the exact location?

SAML response generate function in
https://github.com/casdoor/casdoor/blob/3718d2dc04b775fecf70db6e8c858d5832f094d4/object/saml_idp.go#L40
and
https://github.com/casdoor/casdoor/blob/3718d2dc04b775fecf70db6e8c858d5832f094d4/object/saml_idp.go#L269

@Oniokey can you make a PR?

I'm not sure at what point Casdoor processes the SAML response, can you give me the exact location?
for example，at which .go file

SAML response generate function in
https://github.com/casdoor/casdoor/blob/3718d2dc04b775fecf70db6e8c858d5832f094d4/object/saml_idp.go#L40
and
https://github.com/casdoor/casdoor/blob/3718d2dc04b775fecf70db6e8c858d5832f094d4/object/saml_idp.go#L269

@Oniokey can you make a PR?

I'm not sure at what point Casdoor processes the SAML response, can you give me the exact location?
for example，at which .go file

SAML response generate function in
https://github.com/casdoor/casdoor/blob/3718d2dc04b775fecf70db6e8c858d5832f094d4/object/saml_idp.go#L40
and
https://github.com/casdoor/casdoor/blob/3718d2dc04b775fecf70db6e8c858d5832f094d4/object/saml_idp.go#L389

ok，I get it.

@hsluoyz This feature is potentially a breaking change because it requires permanently storing a configuration for determining the style of NameID, meaning that it requires adding a field to the application table (similar to SamlNameIdMode), which involves updating of the stock user.
Any good advice on this, or is this level of breaking change acceptable?

add a switch to determine whether use NameID is a good idea we can set it to be false by default,  why it will be a breaking change, will it change something when we turn it off? and it seem that this is the proprietary field of aliyun, maybe we can check the prefix of destination  to determine whether use NameID.

I need to change the table structure to add a configuration item（use default or the email as NameID）

I need to change the table structure to add a configuration item（use default or the email as NameID）

imho，this is acceptable,  yoi can try to make a pr first

I need to change the table structure to add a configuration item（use default or the email as NameID）

imho，this is acceptable, yoi can try to make a pr first

ok

@Oniokey don't add a new boolean field to application. There are already so many SAML bool fields in it. Better with a SAML field table structure as the property of application to contain all SAML related stuff

@Oniokey don't add a new boolean field to application. There are already so many SAML bool fields in it. Better with a SAML field table structure as the property of application to contain all SAML related stuff

it's not a new field in SAML Response, it's a already exist field called NameID, what I need is to change the content of it


why tune n here?isn't n the batch size?

here is a segment of wavenet_modules:
     l_old = int(round(l / dilation_factor))
     n_old = int(round(n * dilation_factor))
        l = math.ceil(l * init_dilation / dilation)
        n = math.ceil(n * dilation / init_dilation)
i can't figure out why the batch size changed,
i think by this way,the net would be sensitive to the order of the data
can someone help with this? thanks

Have you figured out? Can you explain why the n is changed/recalculate


Create PR a directly from in-place edits

Description
Related to https://github.com/refined-github/refined-github/issues/6756, i would like to get a PR directly done when doing an in-place edit, instead of showing me the commit and needing to create the PR by hand.
Example URLs
https://github.com/refined-github/refined-github/edit/main/readme.md

After editing a file o a new repo you’re immediately shown a form to create a PR, where you can confirm with one click. I don’t think it’s worth optimizing this step because most people should add more information to the PR body.

For single commit PRs, the PR body IS mostly equal to the commit body, that's why i think they can be done directly.


commit d490f19 - mouse never release

Hyprland Version
Hyprland, built from branch main at commit c55fe27e90017860a2c494a7f5404e53eba94941 dirty (Revert keybinds: ignore mods on release of special binds).
Bug or Regression?
Bug
Description
Mouse clicks never get released.
Reverting commit d490f19 fixes the issue.
How to reproduce
...
Crash reports, logs, images, videos
No response

Can confirm, same issue here.

あ！ちょっと過失した、すまん


Feat/lesq 601/subject unit listing cohort [LESQ-601]

Description
Music year: 2010

update mvs for unit_listing and subject_listing
add filter to the queries to limit to non-legacy results

How to test
I tested that there is no contamination by publishing legacy programmes locally.

Go to https://deploy-preview-2227--oak-web-application.netlify.thenational.academy
Go to the subject listing page, then the unit listing page for legacy and non legacy
You should see no difference

Checklist

[ ]  Owa should look and behave the same as it does now with regards to the new tag on content
[ ]  update GQL query to filter out legacy data

:tada: This PR is included in version 1.340.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


feat: adjustable rule_group interval

This allows for more expensive SLO recording rules by letting users
create a custom Prometheus rule_group.interval, as rule group default or per type of rule.
https://prometheus.io/docs/prometheus/latest/configuration/recording_rules/#rule_group
Now instead of all rule groups using the default global evaluation
interval, a custom interval can be set on all three types of recording rules
for an SLO.
If no interval is set then the global default will be assumed,
matching current Sloth behavior.
Resolves: #367
Edit: I didn't touch the kubernetes operator side and probably need adjustments there

@slok when you have a chance could you take a look and let me know what you think?

looks like an amazing feature, cc @slok


Extract reads between non-consecutive bins

The examples (https://github.com/theaidenlab/straw/wiki/CPP#running) show how to extract reads between two consecutive regions on the chromosome. If I have two sets of bins which are not consecutive, and I am only interested to extract reads between these two sets of bins, is there a fast way to use straw to do that? Thanks.

I am not sure I understand the question.  You can extract reads that are
off diagonal between different bins, e.g.:
straw KR HIC001.hic 1:1000000:7400000 1:2000000:8400000 BP 25000
On Thu, May 3, 2018 at 5:25 PM, wgmao<EMAIL_ADDRESS>wrote:

The examples (https://github.com/theaidenlab/straw/wiki/CPP#running) show
how to extract reads between two consecutive regions on the chromosome. If
I have two sets of bins which are not consecutive, and I am only interested
to extract reads between these two sets of bins, is there a fast way to use
straw to do that? Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/theaidenlab/straw/issues/24, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABWiW_uEW1gZX-0bhc3A-7FamiP2NsA4ks5tu3XXgaJpZM4Txzut
.

--
Neva Cherniavsky Durand, Ph.D.
Staff Scientist, Aiden Lab
www.aidenlab.org

Sorry for the confusion. For example, I have two sets of bins:
Set1:
1:1000000:2000000
1:6000000:7400000
Set2:
1:2000000:3000000
1:7000000:8400000
If I would like to extract the reads between these two sets, I have to query four times.
straw KR HIC001.hic 1:1000000:2000000 1:2000000:3000000 BP 25000
straw KR HIC001.hic 1:1000000:2000000 1:7000000:8400000 BP 25000
straw KR HIC001.hic 1:6000000:7400000 1:2000000:3000000 BP 25000
straw KR HIC001.hic 1:6000000:7400000 1:7000000:8400000 BP 25000
If I only query once:
straw KR HIC001.hic 1:1000000:7400000 1:2000000:8400000 BP 25000
I have to filter a lot of regions which don't overlap with these two sets.
Querying four times externally is one solution but it can be painful  if sets are really large. I just wonder if there is an internal way to extract reads given two sets of non-overlapping bins.


Fix runtime.txt generation when using pipenv

The sentence in the README explaining how runtime.txt is automatically generated when using pipenv is not correct as it is referring to python-version which is not very clear (where is this field?) and not correct (it should be either python_version or python_full_version)

@Thomas-Mollard thanks for fixing this


Callable/ Module fields are seen as unbound modules

System information
Irrelevant.
Problem you have encountered:
I'm new to flax, but when building composite modules I wasn't able to call field modules directly without getting unbound errors. I was able to get around this by defining proxy functions, however, this becomes more and more unreadable as I add callable/ module fields.
What you expected to happen:
It would be nice to be able to call myModule.apply(..., method=myModule.field) instead of myModule.apply(..., method=myModule.proxy_to_field). This is more readable and intuitive. At the moment, the design does not allow me to easily use the builder design pattern (or other factory patterns) to populate Module callables for a larger composite Module. The workaround requries me to either predefine proxy methods for all fields or write a possibly conflicting meta-class or something else.
Logs, error messages, etc:
CallCompactUnboundModuleError             Traceback (most recent call last)
Cell In[21], line 1
----> 1 mya.apply(v, jnp.zeros(3), method=mya.b)

    [... skipping hidden 5 frame]

File ~/anaconda3/envs/jax/lib/python3.9/site-packages/flax/linen/module.py:953, in Module._call_wrapped_method(self, fun, args, kwargs)
    951 if is_compact_method:
    952   if self.scope is None:
--> 953     raise errors.CallCompactUnboundModuleError()
    954   is_recurrent = self._state.in_compact_method
    955   self._state.in_compact_method = True

CallCompactUnboundModuleError: Can't call compact methods on unbound modules (https://flax.readthedocs.io/en/latest/api_reference/flax.errors.html#flax.errors.CallCompactUnboundModuleError)

Steps to reproduce:
import flax
import flax.linen as nn

import jax
import jax.numpy as jnp

class A(nn.Module):
    b: nn.Module

    def do(self, x):
        # Proxy to call `b`.
        return self.b(x)

    @nn.compact
    def __call__(self, x):
        return self.b(x)

myA = A(nn.Dense(3))
v = myA.init(jax.random.PRNGKey(0), jnp.zeros(3))

# call myA.b through the proxy: Works
myA.apply(v, jnp.zeros(3), method=myA.do)

# call myA.b directly: Doesn't work
myA.apply(v, jnp.zeros(3), method=myA.b)
>> CallCompactUnboundModuleError             Traceback (most recent call last)
>> Cell In[21], line 1
>> ----> 1 mya.apply(v, jnp.zeros(3), method=mya.b)

I wanted to share another example of a hacky way to solve my problem, simply by generating functions that assign methods for each module field.
def module_methods(m: nn.Module):
    for field in fields(m):
        if field.type == nn.Module:
            name = field.name

            def fun(self, *a, **k):
                return getattr(self, name)(*a, **k)

            setattr(m, f'{name.lstrip("_")}', fun)  # Assign generated method to the public module field
    return m

@module_methods
class A(nn.Module):
    _b: nn.Module  # Make module field private with '_'

    def do(self, x):
        return self.b(x)

    @nn.compact
    def __call__(self, x):
        return self.b(x)

Implemented in this way, the code below does not generate errors.
# call myA.b through the proxy: Works
myA.apply(v, jnp.zeros(3), method=myA.do)

# call myA.b directly: Now it works because the decorator assigned a method to the `b` field.
myA.apply(v, jnp.zeros(3), method=myA.b)

Hey @joeryjoery, one very simple way to solve this is just by passing lambda to method that calls b:
myA.apply(v, jnp.zeros(3), method=lambda m, *args: m.b(*args))

That said, in #3281 I am improving method's string support such that you would be able to just do this:
myA.apply(v, jnp.zeros(3), method="b")

Hey @joeryjoery, one very simple way to solve this is just by passing lambda to method that calls b:
myA.apply(v, jnp.zeros(3), method=lambda m, x: m.b(x))

That said, in #3281 I am improving method's string support such that you would be able to just do this:
myA.apply(v, jnp.zeros(3), method="b")

Ha that is amazing, great! I'll use the lambda method for now then until the PR is merged, you can also close this issue afterwards.
On a small sidenote, does that mean that #3281 only allow's the string method="b" or also a direct reference method=myA.b? I tend to use direct references over strings at the moment because of typos in strings (I can of course use myA.b.__name__ but I don't like to use direct access of dunders).

You can use both strings or functions, this is already supported :)
The PR only improves string support for sub-modules to enable this use case.


announce deprecation of CmiBool

Original author: Ramprasad Venkataraman
Original issue: https://charm.cs.illinois.edu/redmine/issues/191

No body.

Original date: 2013-06-03 22:47:58

Mentioned directly to namd-ppl list.

Original date: 2013-06-03 22:51:15

And to the charm mailing list.


llama: Add CI to verify all vendored changes have patches

With the new vendoring model we want to make sure we don't accidentally merge changes in the vendored code without having those changes covered by a patch that applies cleanly on the current baseline.

CI verified:

Bad commit (didn't include a patch) so CI failed - https://github.com/ollama/ollama/actions/runs/11130805795/job/30930965173

Good commit (with a patch, so it applies cleanly) CI passes - https://github.com/ollama/ollama/actions/runs/11130919153/job/30931340533?pr=7066

CI will fail since the branch isn't clean right now.


Switch to configV1 ClusterVersions to fetch the version of the cluster

The reason we used the equivalent of oc get OpenShiftControllerManager/cluster -o jsonpath='{.status.version}'
was that it was a very simple way to get the current version of the
cluster. Since certain deployed OCP clusters do not seem to have the
.status.version, we take the more complex route of parsing the status history
of oc get clusterversion/version -o yaml. These fields are always
present in all clusters and it is the same approach the UI is
implementing currently [1][2].
Furthermore the clusterVersionStatus docs at [3] specify the following:
// desired is the version that the cluster is reconciling towards.
// If the cluster is not yet fully initialized desired will be set
// with the information available, which may be an image or a tag.
// +kubebuilder:validation:Required
// +required
Desired Release `json:"desired"`

// history contains a list of the most recent versions applied to the cluster.
// This value may be empty during cluster startup, and then will be updated
// when a new update is being applied. The newest update is first in the
// list and it is ordered by recency. Updates in the history have state
// Completed if the rollout completed - if an update was failing or halfway
// applied the state will be Partial. Only a limited amount of update history
// is preserved.
// +optional

So what we do in this patch is to check the history and return the first
version in the history which has its state as completed. Should we not
find any (as it might happen during cluster initialization), we fall
back on returning the ClusterVersion.Status.Desired. This is only for
the les slikely case where the cluster still has not version history, in
such cases (and only in such cases), it seems to be okay to just fall
back to that.
Tested both on a ROSA 4.12.x deployed via ClusterBot (which did show the
issue about missing OpenShiftControllerManager/cluster.status.version
and on a stock 4.13 deployed via install-config.
[1] https://github.com/openshift/console/blob/master/frontend/public/module/k8s/cluster-settings.ts#L239
[2] https://github.com/openshift/console/blob/master/frontend/public/module/k8s/cluster-settings.ts#L247
[3] https://pkg.go.dev/github.com/openshift/api/config/v1#ClusterVersionStatus

/lgtm


Performance helpers for fonts and things

Example:

do the delayed font load trick instead of the one that google fonts gives you (which is blocking)

This is also automatically done with the new head function.


Has windows line endings. This is a bash script.

This is a bash script with Windows line endings. Why?
➜  helm secrets version
/usr/bin/env: ‘bash\r’: No such file or directory
Error: plugin "secrets" exited with error

I was having this problem in Ubuntu shell on Windows 10 but I found a workaround.  I tried to install it - it failed.  I then ran dos2unix on ~/.helm/plugins/secrets/install-binary.sh and ran:
helm plugin update secrets
and it worked.


Feat: Multilingual support in image

Describe the feature
THIS IS FOR FUTURE IMPLEMENTATION ONCE BASIC SETUP OF AI IMAGE GENERATION IS COMPLETE
💡 Idea is to have multilingual support for images
gityzer.vercel.app?<github_username>&update=14&style=dark&theme=dark&color=white&lang=en
gityzer.vercel.app?<github_username>&update=14&style=dark&theme=dark&color=white&lang=hin
gityzer.vercel.app?<github_username>&update=14&style=dark&theme=dark&color=white&lang=marathi
this will determine the language of the text on the image making it more customizable for the user
This task will be a level 3 task
Add ScreenShots
N/A
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm GSSOC'24 Extd registered Contributor
[X] I'm Hacktoberfest registered Contributor
[X] I want to work on this issue

Can I work on this issue @vansh-codes

Its not supposed to be implemented now 😅
I will ping you here when its has to be implemented

okh @vansh-codes