Add ChaCha related ciphers to default provider.

Checklist

[x] documentation is added or updated
[x] tests are added or updated

Regarding all the -1 returns, it is specified that the dispatched functions that return a status indicator shall return 0 or 1.  See "RETURN VALUES" here.

Well it looks like at least one of these -1 values are needed since 80-test_sslcorrupt.t is now failing.
Should be fun to figure out which one is actually needed..

So the -1 values are needed otherwise the corrupt test fails..

Could it be that there's a condition that isn't coded quite right?

Found the issue.  EVP_Cipher() is not coded right.

Have a look to see if a merge with #10125 fixes the failures.

Reworked so that non dispatch related functions are  seperated into *_hw.c files.

ping

I notice that PR doesnt build :)
I would prefer to do this in a separate pass - There will still be quite a bit of cleanup to do after all the ciphers are moved across.

Thanks. Merged to master.


AM/PM option

I love military time and all, but could I have an option for the time to be displayed in AM and PM? (with the text after the number)
Thank you moh,
you are awesome!
Fox

Sounds gud 🚀


Add rule "missing_stop_name" when location_type is in (0, 1, 2)

What problem in GTFS datasets does this new rule address? Please describe.
stop_name in stops.txt is conditionally required when location_type is 0, 1 or 2 (see reference). Currently, empty stop_name values are not reported for stops with these location_type values
Describe the new validation rule
for all stops in stops.txt where location_type is in (0,1,2), stop_name must not be empty
Error vs warning
error

Hello @rschoening, thanks for opening this issue!
We already have issue #1078 open for this problem. I am tagging this one as duplicate and closing it.
See you around 😊


Don't add trailing slash to routes

When defining custom routes with routes a trailing slash is added.
class TestView(FlaskView):
    @route('/foo', methods=['POST'])
    def foo(self):
        return 'foo'

This adds a trailing slash to the making the route: /test/foo/ causing a
redirect when doing a post to /test/foo.

Just set Content-typ with application/json, it works for API Web Application.


Email Validation Error

Every instance of a form throws up a error
The Email field must contain a valid email address.
and it comes down to this validation rule: valid_email
Anyone else having this problem?

Are you referring to the Contact form? I can not replicate this issue when entering a valid email address.

Since I can not replicate, I am closing this issue. Please reopen if you can provide more specific information on how to reproduce.


Date objects serialized by toRecords fail instanceof Date check

Have you tried latest version of polars?

yes

What version of polars are you using?
Observed on 0.10, still present on main branch
What operating system are you using polars on?
Mac / Linux
What node version are you using
Node 20
Describe your bug.
When .toRecords() returns a Date object, it fails instanceof Date checks
What are the steps to reproduce the behavior?
Extended the existing test with a demonstration:
https://github.com/davidgovea/nodejs-polars/pull/3/files
What is the actual behavior?
See test run here: https://github.com/davidgovea/nodejs-polars/actions/runs/9680675801/job/26709383256
serializedDate.constructor.name
=> 'Date' ✅

serializedDate instanceof Date
=> false ❌

What is the expected behavior?
x instanceof Date should be true.

Have not taken a look under the hood yet, but I am happy to do so when I get a free moment. For now, I'm just paving over the issue with an additional new Date(polarsToRecordsDate) wrapper in my code

toRecords returns a JSON array: [{"date":"2024-07-07T00:00:00.000Z"}]
Why would you expect actual[0].date to be an instance of a Date?
I think you need to convert it to Date to be a Date else it will be a string/object just like everything else in JSON.

Hey @Bidek56, thanks for taking a look!
I'd be fine with that being the case (everything serialized to ISO date string), but today it seems like we're in a strange middleground.
If it were as you suggest, then the constructor.name would be String, not Date.
Additionally, the object has all of the Date methods available.

If it quacks like a Date, I'd assume it should pass instanceof Date 😆
But yeah making toRecords return standard JSON would make sense too.

If you are looking for schema, I would suggest using df.toDataResource() which starts with df.toRecords() then it adds field type to be:
{"data":[{"date":"2024-07-08T00:00:00.000Z"}],"schema":{"fields":[{"name":"date","type":"string"}]}}

@davidgovea this seems like a bug in jest
Using alternative test runners, it all passes
> node example.js
const pl = require("nodejs-polars");
const { test } = require("node:test");
const assert = require('node:assert/strict');
test("toRecords:date", () => {
	const df = pl
		.DataFrame({
			date: [new Date()],
		})
		.withColumn(pl.col("date").cast(pl.Date).alias("date"));

	const serializedDate = df.toRecords()[0].date;
	assert.strictEqual(serializedDate instanceof Date, true);
});

> bun test example.js
const pl = require("nodejs-polars");

test("toRecords:date", () => {
	const df = pl
		.DataFrame({
			date: [new Date()],
		})
		.withColumn(pl.col("date").cast(pl.Date).alias("date"));

	const serializedDate = df.toRecords()[0].date;
  console.log(serializedDate)
	expect(serializedDate instanceof Date).toEqual(true);
});

additionally, using various versions of node & bun directly return the same result
Welcome to Node.js v22.1.0.
Type ".help" for more information.
> const pl = require("nodejs-polars");
undefined
> const df = pl.DataFrame({
...     date: [new Date("2020-01-01"), new Date("2020-01-02")],
... });
undefined
> df.toRecords()[0].date instanceof Date

Wow that's a weird one! 🤦 Thanks @universalmind303
Realizing that I only observed this in jest tests as well, not in production. What is it doing under the hood?!
Again really appreciate the response here


Template: Bulldog Magazine Article

From vg-macktrucks-com-rd created by Gonzalo-CG: Netcentric/vg-macktrucks-com-rd#132
Business Value
As a content author, I want a quick way to create new articles from a template, so I don't have to add every component used on an article manually.
Description
A new "Bulldog Magazine Article" template will be available, which contains the basic structure and blocks that all Bulldog Magazine blog articles share.
The scope of this issue is to create the template, every block contained in this template has a separate issue for its development.
Acceptance Criteria

[x] Magazine Article template is available for authors to use and is added to the AEM Sidekick library, which contains the basic structure and blocks that all Volvo Magazine Articles share.

[x] To set the magazine article title, authors can type standalone text and format it as an H2 if the Magazine Hero block has a title defined, or as an H1 if the Magazine Hero block doesn't have a title defined.

[x] The Magazine Article template contains all of the following elements (in a general order of priority):

[x] aemsites/vg-macktrucks-com-old#766 (priority 1) Reflects the BE work needed. FE search will be implemented following https://github.com/hlxsites/vg-macktrucks-com-old/issues/913 otherwise the template can be created without search in the meantime.

[x]  aemsites/vg-macktrucks-com-old#746 (priority 2) Done

[x] aemsites/vg-macktrucks-com-old#749 (priority 2) Done

[x]  aemsites/vg-macktrucks-com-old#750 (priority 2) Done

[x] aemsites/vg-macktrucks-com#71 (priority 3) _Dependent on this ticket's completion.

[x]  aemsites/vg-macktrucks-com-old#748 (priority 3)Done

[x] aemsites/vg-macktrucks-com-old#767 (priority 3) Done

[x] To add the attribution and social block at the top of the page, a new variant of the v2 social block, named "attribution," will be created. The date and author information will be automatically pulled from the page's metadata.

Notes

No subscribe block needs to be added as aemsites/vg-macktrucks-com#60  is still in backlog

Figma
Magazine Article - Figma.
Desktop:

Mobile:

The template can be found here, and the sub-navigation used within the template is defined here.


The output pdf file is not highlighted

I'm using python3.7 and my os is windows 10.
I built from the source code.
I tried different styles but none of them were highlighted.

There were no errors or warnings in the cmd.
I guess it doesn't work on the windows platform?

it happened to me too, mine is macOS, got no highlited


feat: set labels and annotations with user configmap

Motivation
Allows users to set labels and annotations for ServingRuntime pods via the model-serving-config ConfigMap. This can be used for setting things like network policy for the runtime pods, which was requested by a user.
Modifications

Sets SR deployment PodSpec labels and annotations from the user ConfigMap
Moves default label network-policy from hard-coded value to default ConfigMap

@anhuong you'll also remove the hardcoded component: model-mesh label right?

/lgtm


SslRootOrIntermediate mandatory at 3.3.1 ?

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request

Module Version

3.3.1

Affected Resource(s)

ArcGIS_Server
ArcGIS_Portal (?)

Configuration Files

ArcGIS_Enterprise_HA.txt
Expected Behavior
Configuration should work.
Actual Behavior
Configuration "ArcGISServer" is failing to "dot sourcing" with the following error:
ArcGIS Server : A parameter cannot be found that matches parameter name 'SslRootOrIntermediate'

If I apply my configuration with version 3.3.0, it works.
According to the Wiki, this parameter is optional.
Steps to Reproduce
Run Invoke-ArcGISConfiguration with the provided config file
Thanks !

Many thanks for your reply @cameronkroeker.
Indeed, I forgot to restart powershell once again...
Apologize for the noise !


Add release note for <IP_ADDRESS>

Signed-off-by: Vijayan Balasubramanian<EMAIL_ADDRESS>Description

Fix invalid url
Add release note for <IP_ADDRESS>

Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #90 (1912c67) into main (6a3fdca) will decrease coverage by 0.21%.
The diff coverage is n/a.

:exclamation: Current head 1912c67 differs from pull request most recent head c07610a. Consider uploading reports for the commit c07610a to get more accurate results

@@             Coverage Diff              @@
##               main      #90      +/-   ##
============================================
- Coverage     82.70%   82.48%   -0.22%
+ Complexity      475      474       -1
============================================
  Files            69       69
  Lines          1821     1821
  Branches        178      178
============================================
- Hits           1506     1502       -4
- Misses          253      257       +4
  Partials         62       62

Impacted Files
Coverage Δ

.../java/org/opensearch/knn/index/util/KNNEngine.java
89.47% <0.00%> (-5.27%)
:arrow_down:

...n/java/org/opensearch/knn/index/KNNIndexCache.java
92.38% <0.00%> (-2.86%)
: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 6a3fdca...c07610a. Read the comment docs.


feat(netlify-cms-widget-relation): use react-select and add support for multiple entries

Summary
Closes #1061
Related to #1928 but this only focuses on the existing relation widget. The behavior should be the same as the old one. Before I continue on this I'd like some confirmation if this can "live" beside the other PR, did this pretty quickly to play around a bit.
Test plan
So far just manual testing – should probably add some unit tests

Deploy preview for netlify-cms-www ready!
Built with commit 87ff3dcfec04e2296c1b2dc34d867b78332c391a
https://deploy-preview-1936--netlify-cms-www.netlify.com

Deploy preview for cms-demo ready!
Built with commit 87ff3dcfec04e2296c1b2dc34d867b78332c391a
https://deploy-preview-1936--cms-demo.netlify.com

Horray! 🎉
was going to do something like this today, you just saved me a bunch of effort (:

Going to hold onto this until we finish implementing tests for the widgets, working on it as we speak.

I left some off-band review here: https://github.com/alexandernanberg/netlify-cms/pull/1
it's mostly compatibility with the code I implemented earlier in #1928
thumbs-up for testing it, especially with something like react-testing-library that allows the changing of implementation

That's exactly what we're using 👍

Just ran into a use case where I needed to use multiple select in the relationship component. So happy to see this PR open. Awesome work @alexandernanberg.
Looking forward to seeing this merged.

No worries! Huh weird - I'll have a look asap (probably in a day or two)

@alexandernanberg  any reason why you omitted the relation widget's fieldsmetadata in this PR?

@barthc I don't see any mention of fieldsmetadata?

I guess the relation widget should be updated to work like the select widget (with the changes in #2054)?
Though I'm not sure that I'm 100% onboard with the changes in that PR in retrospect, kind of makes sense to keep the same type always to me. Uncertain if it will break some Gatsby configs because you expect to receive an empty array instead of null 🤔

Also the bug seems to have gone away... not sure why ¯_(ツ)_/¯

Good point on null vs. string in the other PR. Strictness of GraphQL is still something we (as a community) have to figure out.
It looks like the preview isn't working here, but it is working on master. There was always an issue where the preview sometimes didn't show initially, but you can test by typing "json" into the field and selecting the "This is a JSON front matter post" entry.
You'll notice on master it loads the preview, but in this PR updating the value still doesn't show the preview.

Ah, it's the thing @barthc was talking about, figured I was missing something when he said that 🙄
The original onChange call passes in an object for fieldsMetadata: https://github.com/netlify/netlify-cms/pull/1936/files#diff-1c6e2a499213935b7c1b04dbb4e1ca1cL110
The new onChange will need to do the same, as this provides the connection between the selection value and the related entry.

I think the metadata should work like before now. But I found some other bugs that I've been unable to solve... if you search for eg. "YAML" and select that post, then try to select another one without searching the selectedValue will be null. That's because queryHits.get just returns the hit for "YAML".
Not sure how we'd handle this 🤔

Gotcha - the previews are definitely fixed, thanks for that 👍👍
@barthc would you be up for reviewing this and weighing in on the queryHits.get question?

A couple of things. The kitchen sinkrelation widget preview doesn't  work on initial load here.

That's because queryHits.get just returns the hit for "YAML".
Not sure how we'd handle this

I suggest you store the initial defaultOptions  and  concat them here. The idea is to always make the defaultOptions avalaible for selection inside the getSelectedValue function.

@alexandernanberg if you don't mind, I would like to push a commit, that would address my comments.

@barthc That would b great since I don't have time atm. You should have push access to my fork now!
It would be nice it we could solve the intial preview without updating the state imo, but that might not be possible? 🤔

@alexandernanberg  @erquhart  review?

I'm thinking since we have all the "options/hits" already loaded in allOptions, does it still make sense to query them?

Yes we have all hits, but we still need to select the items that matches the query using the fuzzy module. The best we can do is cache results for backend listAllEntries which should improve performance and help fix #1357 but that will be handled in a separate PR.

Does this mean this should now be available for everyone use? Is there an example of code for the config.yml?

@sanscheese you would need to add the multiple options to the widget multiple: true  just like you would do for select widget.
@erquhart we need to merge this #2112 to update the doc.

@barthc Hmm. I tried that. It doesn't look like the field is still using react-autosuggestion rather than react-select for me. (So I can type to find a tag)
This is the field I've setup in my config.yml
- label: "Tags"
    name: "tags"
    widget: "relation"
    collection: "tags"
    displayFields: ["title"]
    searchFields: ["title"]
    valueField: "title"
    multiple: true

My admin/index.html file is using<EMAIL_ADDRESS>I can see the new multi-relation using the dev-test admin in this netlify-cms repo but have noticed it's config.yml is using widget: 'relationKitchenSinkPost which made me think is still not release for full use yet?

@erquhart  is the new relation widget released yet?

No, releasing today.

This is amazing timing for me! Congrats, and great work on this! 😀

Update: this was released in 2.5.0 🎉

Update: this was released in 2.5.0 🎉

I must say: It looks and works very well! Great work!


Sec web socket protocol

Codecov Report
Patch coverage: 75.00% and project coverage change: -0.70% :warning:

Comparison is base (608a7f8) 81.94% compared to head (3d787ad) 81.25%.

:exclamation: Current head 3d787ad differs from pull request most recent head bbfc3c9. Consider uploading reports for the commit bbfc3c9 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
- Coverage   81.94%   81.25%   -0.70%
==========================================
  Files          13       13
  Lines         770      779       +9
==========================================
+ Hits          631      633       +2
- Misses         89       92       +3
- Partials       50       54       +4

Files Changed
Coverage Δ

config.go
100.00% <ø> (ø)

server_handshake.go
61.11% <33.33%> (-5.56%)
:arrow_down:

client.go
77.58% <100.00%> (+1.06%)
:arrow_up:

client_options.go
84.61% <100.00%> (+2.79%)
:arrow_up:

... and 1 file with indirect coverage changes

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


A11y: tabbing through page gets stuck in the editor

Eventually, I get stuck in the code editor and can't tab out.

Not sure how if its really possible to fix this one. Same issue on all the pages I've found with CodeMirror.
It doesn't break voice over on OS X.

This isn't an issue anymore for any first time users. (An overlay is shown on editors).
AFAIK, this doesn't break a11y readers, so I'm going to close it without a reasonable fix. Generic problem with most editor sites, eg. jsfiddle.net


Update Installation for IPython 3.0

Will fix issues like https://github.com/quantopian/qgrid/issues/16.

Any idea how to install this for ipython 3.0?
I tried pip install git+https://github.com/quantopian/qgrid but I get the following error -
python UnknownExtra: ipython 3.0.0 has no such extra feature 'notebook'

I'm also seeing this issue. Any work arounds?

@chausler @signalseeker I spent some time looking into this and haven't been able to reproduce the issue.  How did you install IPython 3.0? I attempted to reproduce it by installing IPython 3.0 like this:
pip install "ipython[all]"==3.0.0

Running pip install git+https://github.com/quantopian/qgrid completes successfully for me after that.  From the error message it sounds like the installation is getting tripped up by the ipython[notebook] requirement in our setup.py file, which seems strange to me because IPython 3.0.0 definitely has the notebook "extra".

I believe we can close this.  Installation steps work with IPython 4.1.2 at this point.


Make some improvements

Do warmup for some samples for benchmark purpose
Do not run semantic_segmentation sample immediately since which
is not user friendly for running CPU backend or NHWC layout

@BruceDai, @ibelem, PTAL, thanks!

Close it since I just merged this change to https://github.com/webmachinelearning/webnn-samples/pull/141.


Remove extra for loop in clients/horizon/client.go

This PR stops the infinite loop documented in issue https://github.com/stellar/go/issues/895. When run this way the request is still sent 10 times instead of once; I'm not sure why. Way better than an infinite loop though.

Ok, it does need the loop, just after the GET instead of before.
Something is wrong with my go formatting; my IDE doesn't like the huge repo apparently and chokes when it tries to run gofmt. It's been a long day and I don't see what I formatted wrong so I'm done for now.

Thanks for the PR! If your IDE is struggling, would you mind running gofmt on the command line and seeing if you can fix the build failure?

@ire-and-curses I realized it might have been choking because I had the whole monorepo open at once. I'll retry with just the horizon folder open.

@ire-and-curses fixed...it was a single space

Even though I need some time to debug #895 I'm sure your fix is not a valid solution. The outer for is crucial to restart the stream when it was closed to support continuous streaming.


Feature Suggestion

Where ever the Helium L1 Address is referenced - call it Helium (Legacy)

Will be fixed by #441


Broken link in Service Mesh with Istio module

On the page at https://eksworkshop.com/servicemesh_with_istio/visualize/
there is a broken curl reference
curl -LO https://eksworkshop.com/servicemesh/deploy.files/istio-telemetry.yaml
which results in an AccessDenied error.
The correct curl command should be:
curl -LO https://eksworkshop.com/servicemesh_with_istio/deploy.files/istio-telemetry.yaml

Further to the above, I don't think this manifest is actually required for the use case being demonstrated i.e. it is possible to use Prometheus and Grafana to verify that traffic is being split equally between two versions WITHOUT applying this manifest.
This manifest appears to be an (old) version of an example of how to add new metrics. However it has been superseded by other manifests in samples/bookinfo/telemetry.

Hello,
Istio has been updated to 1.4.5 #667
Closing this issue


C_CPP Documentation on C++ versions.

Description 📜
This is the documentation of a comparison between C++14 and C++17 and other versions of C++.
Domain of contribution: C/CPP
Format: Documentation
Language: C++
Fixes #2711

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[x] C/CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] Java#Script
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

Add your content in markdown file format, can't accept python notebooks sorry.

Add your content in markdown file format, can't accept python notebooks sorry.

Sir, I have added the .md file now. Please check.


[back end] user profile picture save to database

when user selects a profile picture, save to their user model

Done! Just as an explanation, the picture itself is not being saved, as that would have required a lot more. What's being saved is the name of the avatar, and once the user is created or they log in, the avatar utils function handleAvatar will compare the short name to a set name and return the image that is available within the src folder. This is a much easier solution, in my opinion.


RIDs for .NET 6 iOS and Android are missing

It would be nice if you could add the RIDs for iOS and Android as well.

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

ID: 009616d0-6c93-a2de-82fe-4f7168817f18
Version Independent ID: 4c2c5549-0151-516d-91d0-9b017c74e787
Content: .NET Runtime Identifier (RID) catalog
Content Source: docs/core/rid-catalog.md
Product: dotnet-fundamentals
GitHub Login: @tdykstra
Microsoft Alias: dotnetcontent

@wfurt should we add sections for iOS and Android?

It would make sense to me since they are in the RID graph.
https://github.com/dotnet/runtime/blob/main/src/libraries/Microsoft.NETCore.Platforms/src/runtimeGroups.props


Payload cloud does not auto-populate PAYLOAD_PUBLIC_SERVER_URL env

Link to reproduction
cloud
Describe the Bug
When creating a new payload cloud instance, I would expect payload to automatically populate the PAYLOAD_PUBLIC_SERVER_URL env variable, just like the resend var for example. After all, payload even logs the following in the deploy logs:
(payload): App URL: https://lavis-detailing.payloadcms.app

To Reproduce
env var is not added, has to be added manually.
Payload Version
2.0.13
Adapters and Plugins
cloud

Right now, 2 conditions must be true for Payload Cloud to automatically populate this env var:

The project must have been made from one of the templates
The project must have an .env.example file containing PAYLOAD_PUBLIC_SERVER_URL

Can you verify that your project satisfies both of these?

Closing due to inactivity.


Problem with number of artists

Number of matched artists in version 1.2 of LastFM dataset is 10,180 (number of rows in the file) not 11,180 as listed in the Readme

Thanks Giulio. issue closed


[2673] Fix "mark_as_review" text

Context
Fix mark_as_review text
Changes proposed in this pull request

Remove legend text by making it nil

Guidance to review
First check

On the homepage, click on: View draft trainees imported from Apply
Select a trainee and click on Trainee data
Scroll down and look at the checkbox, it should match the prototype and there should be no legend text above it.

Second check

On the homepage, click on: View draft trainees imported from Apply
Select a trainee and click on Course details
Look at the checkbox, it should match the prototype and there should be no legend text above it.

 Bug - Marked as review text is not correct


C64 cart preset

The wish is fee, even if I know this is a total corner case and realise the consequential priority;
I guess you have a mechanism for parsing/handling the different filetypes, when selecting the file types. So if I select C64 PRG it will have a routine that imports knowing the file type structure of that file type which is slightly different than when selecting c64 BIN.
I would very much like to see support for c64 cartridges (as that seems to be what I have used the program for, given that no other program on that market can handle the size of these 6502 binaries).
This would means setting an ".addrs" to $8000 on every $2000 bytes of the file. Adding a note that also denotes the location would be a clear bonus.
(There is also a CRT format with a header but parsing the binary option is likely easier).

The trick is to avoid hard-coding it into the executable.  Adding a feature to SystemDefs.json that drops an address region every N bytes might be sufficient, possibly with an initial offset for the file header, though it looks like CRT headers aren't fixed-length.  Then we'd just add a new "C64 cartridge" entry to SystemDefs.
This might be better served by an existing item on the to-do wiki:

Add "file format recognizers". Simplest form looks at filename and bits of files, and sets default values for things like address and entry-point status flags. Could pull in optional symbol files and extension scripts.

Could implement as one or several extension scripts. Possibly platform-specific.

Instead of trying to define everything in SystemDefs.json, SourceGen would run a script that parses the file contents and generates project elements (address regions, notes, comments, project symbols, etc).
On a related note, marking the header as non-addressable actually works out pretty well... go to https://6502disassembly.com/nes-smb/SuperMarioBros.html#SymStart and scroll up to see what it looks like for an iNES header.  (I reworked SMB's addresses after releasing v1.8.0-beta1.  FWIW, Metroid is closer to what you're doing.)

Moved to "TO DO" wiki.


Fontello website is changing mapping of contents

As we noticed fontello changed mapping of contents in classes. For example previously content=”e807” associated to class .si-employee:before and now content=”e807” associated to class .si-time-location:before.
Please guide us how frequently you make changes in fontello libraries and also give us link where you wrote that type of changes? As if you will change core libraries this will affect us a lot.

Dupe of #724/#725


Upgrade service loadbalancer to haproxy 1.6

Tracking bug to test and bump haproxy in service loadbalancer to 1.6, now that it's official (related https://github.com/kubernetes/contrib/pull/120, http://marc.info/?l=haproxy&m=144476286029747)

@bprashanth this can be closed (haproxy is using 1.6)


WIP: Add active-sprint strategy

Resolves #17. A yellow warning appears for each issues that the user attempts to commit, but does not stop the commit from being made.

Tests? They would help me understand your implementation on why you are extracting the start and end date the way you are.

Closing, since I don't have the time left to work on this.


Optimize cross

d3.cross generates cartesian products recursively, at each level of recursion first iterating through the items in the current array and then for each such item recursing to generate the tail of the cartesian product.
This can be sped up by flipping the generator around: first recurse, and for each result iterate through the current array. This avoids generating the intermediate recursion values n times in each recursion step, which should be faster, especially when creating cartesian products of more than two arrays.
A small benchmark illustrates this (here be dragons):
const { cross } = require("d3-array");

for (let i = 0; i < 100000; i++) {
  cross([1, 2, 3, 4], ["a", "b", "c", "d"], [true, false, null, NaN]);
}

With my development machine, Node.js v11.1 and the master version of cross the benchmark takes about 2.47 seconds:
$ time node benchmark.js
real	0m2.470s
user	0m2.418s
sys	0m0.067s

With the modified version the benchmark takes about 1.69 seconds:
$ time node benchmark.js
real	0m1.689s
user	0m1.658s
sys	0m0.072s

This implementation keeps the same order of the results as the master version, so e.g. cross([1, 2], ["a", "b"]) returns [[1, "a"], [1, "b"], [2, "a"], [2, "b"]]. If the order is not important (e.g. cross([1, 2], ["a", "b"]) is allowed to return [[1, "a"], [2, "a"], [1, "a"], [2, "b"]]) then the code can be slightly simplified by changing the beginning of product to something like this:
function* product(head, ...tail) {
  if (tail.length) {
    for (const t of product(...tail)) {
      for (const h of head) {
        yield [h, ...t];
...

Excellent, thanks!


[feature]: Update to new Transaction Insights version

From MetaMask Snaps team:

A new version of transaction insights will soon be implemented altering how insights are fetched. You may need to update your Snap logic; please review the detailed discussion.

closed by https://github.com/filecoin-project/filsnap/releases/tag/filsnap-v1.3.0


Add all contributors to ml5-website-2 repo

@all-contributors please add @shiffman for content code

@all-contributors please add @wenqili for content code infra maintenance ideas

@all-contributors please add @garym140 for design

@all-contributors please add @nicoleflloyd for design

@all-contributors please add @AbolTaabol for design

@all-contributors please add @joeyklee for design code maintenance

@all-contributors please add @hx2A for content maintenance ideas

Looks like this has been addressed!


Add a Gitter chat badge to README.md

Mindsers/configfile now has a Chat Room on Gitter
@Mindsers has just created a chat room. You can visit it here: https://gitter.im/mindsers/configfile.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Duplicate of #6


bug: Version 7.25.0 Mutation of posts not working in scaffolded app

Provide environment information
System:
OS: macOS 14.1.2
CPU: (12) arm64 Apple M2 Max
Memory: 50.76 GB / 96.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 20.9.0 - ~/.nvm/versions/node/v20.9.0/bin/node
npm: 10.1.0 - ~/.nvm/versions/node/v20.9.0/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
version: 7.25.0
Describe the bug
Mutation of posts in the scaffolded app is not working when selecting prisma during the setup process. createPost.mutate({ name }); in create-post.tsx doesn't even trigger a network call.
Reproduction repo
Scaffolded app. No code change.
To reproduce
Initialize the T3 app with npm create t3-app@latest.  Run npm run dev. Go to the website and try to create a new post.
Additional information
No response

The issue was not present anymore


Default to multiprocesses instead of single process

Resolving Issue #332 .
This configuration change in JobQueueCluster changes the default processes per job so that they follow the same heuristic as dask (proc=cores for cores <= 4, otherwise it's the smallest factor greater than the square root of cores).
PR also includes modified documentation (though it doesn't explain the heuristic -- perhaps that should be done on dask's docs and we link it?).

Note that there is a pending rewrite of this project in
https://github.com/dask/dask-jobqueue/pull/307 .  It might be a good idea
to wait for that;
On Sun, Sep 8, 2019 at 6:43 AM Jenni Rinker<EMAIL_ADDRESS>wrote:

This configuration change in JobQueueCluster changes the default processes
per job so that they follow the same heuristic as dask (proc=cores for
cores <= 4, otherwise it's the smallest factor greater than the square root
of cores).
PR also includes modified documentation (though it doesn't explain the
heuristic -- perhaps that should be done on dask's docs and we link it?).
You can view, comment on, or merge this pull request online at:
https://github.com/dask/dask-jobqueue/pull/334
Commit Summary

adding heuristic for processes=None
fixing tests

File Changes

M dask_jobqueue/core.py
https://github.com/dask/dask-jobqueue/pull/334/files#diff-0 (21)
M dask_jobqueue/jobqueue.yaml
https://github.com/dask/dask-jobqueue/pull/334/files#diff-1 (7)
M dask_jobqueue/tests/test_jobqueue_core.py
https://github.com/dask/dask-jobqueue/pull/334/files#diff-2 (2)
M docs/source/configuration-setup.rst
https://github.com/dask/dask-jobqueue/pull/334/files#diff-3 (6)

Patch Links:

https://github.com/dask/dask-jobqueue/pull/334.patch
https://github.com/dask/dask-jobqueue/pull/334.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask-jobqueue/pull/334?email_source=notifications&email_token=AACKZTHEZGMCX6OD4F2KB53QIT6PZA5CNFSM4IUTPVIKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HKALB3A,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACKZTEDUDCLONOXJT5LYGDQIT6PZANCNFSM4IUTPVIA
.

Ack, I already wrote and pushed the new test. Well, that's ok if this never gets merged, it was a good test-run for me on contributing.

Well, that's ok if this never gets merged, it was a good test-run for me on contributing.

Thanks a lot for the PR @jennirinker! I am confident that this will get merged since this is something we want to do as #332 indicates.
I agree with @mrocklin that this is a good idea to first merge #307. Once #307 is merged this PR may need to be tweaked a bit but I expect that it should not be too much work.

@jennirinker side-comment: Please use "Fix #issueNumber" in your PR description, this way the associated issue gets closed automatically when the PR is merged. For more details, look at this.
Using anything more fancy e.g. "resolving issue #issueNumber" does not work unfortunately. I updated your PR description.

@lesteve Ah, thanks. I was close -- "resolve" instead of "resolving" and I would've had it. Next time.

No worries, this is the kind of caveats that trip everyone up once in a while ..


Register username

seems that field name in register.html default is name, as in account validation rules, when login attribute is LOGIN_USERNAME field name is username. So do not match and no error returned by validation when field is empty.
Try to replace name by username in register.html, and all is ok.
Dit i miss something ?
Thanks

To clarify on this.
The field with the attribute name="name" in the register.htm file for the Account component has nothing to do with managing the login and authentication of a user. This field simply allows users to designate their own alias or leave it empty to be "anonymous".
As per the readme.md file:

By default the User plugin will use the email address as the login name. To switch to using a user defined login name, navigate to the backend under System > Users > User Settings and change the Login attribute under the Sign in tab to be Username. Then simply ask for a username upon registration by adding the username field:

<form data-request="onRegister">
    Full Name
    <input name="name" type="text" placeholder="Enter your full name">

    Email
    <input name="email" type="email" placeholder="Enter your email">

    Username
    <input name="username" placeholder="Pick a login name">  // NEW FIELD FOR USERNAME

    Password
    <input name="password" type="password" placeholder="Choose a password">

    <button type="submit">Register</button>
</form>

We could probably add the field to the default component markup when that mode is in use however @JBonnyDev. Would you be willing to make that a PR?

@LukeTowers Yeah I can take a look at it.


Hitting Enter on Search

When hitting enter on the Semantic UI search, I am being redirected from http://localhost:3030 to http://localhost:3030/?. Is there a way I can choose where the user is being redirected after they inserted some input in the search bar and hit enter?

Hi @MarcusEFC, Enter uses preventDefault() to stop it from happening, and I couldn’t reproduce.
Could you please fork this JSFiddle and create a test case?

Closing the issue until there’s a test case.


Add support for saving mhlo level IRs in python artifacts

In the past, the result of iree-import-tf was mhlo IRs. Now they are all lowered to linalg/std/tensor ops. It's would be very nice if we can save mhlo IRs to the artifact dir. It's easier to debug and spot missing support if the issue happens at MHLO level.

We already save a couple of temp files here. Easy enough to add another
flag:
https://github.com/google/iree/blob/5fea3963b934d2907c2eead3f637097d3f0ad8e1/integrations/tensorflow/iree_tf_compiler/iree-import-tf-main.cpp#L227
You could just split the main tf pipeline here:
https://github.com/google/iree/blob/5fea3963b934d2907c2eead3f637097d3f0ad8e1/integrations/tensorflow/iree_tf_compiler/TF/Passes.cpp#L108
and do the MHLO lowering in the main() function, saving a temp fine in
between.
Note that it is not pure MHLO: some things are lowered directly from tf and
we will be lowering some things to tosa in the near future. But there will
likely always be this "mid level ir" split and dumping it as a temp file
there makes sense to me. Maybe call it -save-temp-mid-level ?
They can be plumbed through the python API by following the example of one
of the other save-temp flags:
https://github.com/google/iree/blob/5fea3963b934d2907c2eead3f637097d3f0ad8e1/bindings/python/iree/compiler/tf.py#L152
Looks like we aren't testing those:
https://github.com/google/iree/blob/5fea3963b934d2907c2eead3f637097d3f0ad8e1/bindings/python/tests/compiler_tf_test.py#L46
But could be with a test like this:
https://github.com/google/iree/blob/5fea3963b934d2907c2eead3f637097d3f0ad8e1/bindings/python/tests/compiler_core_test.py#L175
On Wed, Jul 7, 2021, 8:00 PM Han-Chung Wang @.***>
wrote:

In the past, the result of iree-import-tf was mhlo IRs. Now they are all
lowered to linalg/std/tensor ops. It's would be very nice if we can save
mhlo IRs to the artifact dir. It's easier to debug and spot missing support
if the issue happens at MHLO level.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/iree/issues/6412, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADYVAHHBTUFCN33NE5PFHLTWUIGBANCNFSM4772N37A
.

Also, sorry I hadn't thought of adding this. I use -print-ir-after on
iree-import-tf pretty extensively but it sounds like you have another flow,
which is fine.
On Wed, Jul 7, 2021, 8:15 PM Stella Laurenzo @.***>
wrote:

We already save a couple of temp files here. Easy enough to add another
flag:
https://github.com/google/iree/blob/5fea3963b934d2907c2eead3f637097d3f0ad8e1/integrations/tensorflow/iree_tf_compiler/iree-import-tf-main.cpp#L227
You could just split the main tf pipeline here:
https://github.com/google/iree/blob/5fea3963b934d2907c2eead3f637097d3f0ad8e1/integrations/tensorflow/iree_tf_compiler/TF/Passes.cpp#L108
and do the MHLO lowering in the main() function, saving a temp fine in
between.
Note that it is not pure MHLO: some things are lowered directly from tf
and we will be lowering some things to tosa in the near future. But there
will likely always be this "mid level ir" split and dumping it as a temp
file there makes sense to me. Maybe call it -save-temp-mid-level ?
They can be plumbed through the python API by following the example of one
of the other save-temp flags:
https://github.com/google/iree/blob/5fea3963b934d2907c2eead3f637097d3f0ad8e1/bindings/python/iree/compiler/tf.py#L152
Looks like we aren't testing those:
https://github.com/google/iree/blob/5fea3963b934d2907c2eead3f637097d3f0ad8e1/bindings/python/tests/compiler_tf_test.py#L46
But could be with a test like this:
https://github.com/google/iree/blob/5fea3963b934d2907c2eead3f637097d3f0ad8e1/bindings/python/tests/compiler_core_test.py#L175
On Wed, Jul 7, 2021, 8:00 PM Han-Chung Wang @.***>
wrote:

In the past, the result of iree-import-tf was mhlo IRs. Now they are all
lowered to linalg/std/tensor ops. It's would be very nice if we can save
mhlo IRs to the artifact dir. It's easier to debug and spot missing support
if the issue happens at MHLO level.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/iree/issues/6412, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADYVAHHBTUFCN33NE5PFHLTWUIGBANCNFSM4772N37A
.

Thanks for the detailed pointers and explanations! I will add the feature.
Yes, I also use -print-ir-after with iree-import-tf. However, I found that it is not convenience to get the mid level IR through python test target. I can run iree-import-tf with the saved model artifacts (i.e. the pb file) to get the IR. After doing it couple times, I think it's better to dump the mid level IR to the artifacts as well.


Class method can‘t recognize class variable.

Environment data

Language Server version: 2021.3.1
OS and version: Mac OS 10.15.7
Python version (& distribution if applicable, e.g. Anaconda): Python3.8.5

Expected behaviour
example for my code:

I can jump to NAME from cls.NAME, I also can jump to NAME from self.NAME, and all NAME will highlight.
Actual behaviour
I can't jump to NAME from cls.NAME, bug I can jump to NAME from self.NAME, and cls.NAME not highlight.
class Hello:

    NAME = 'hello'

    @classmethod
    def get_cls_name(cls):
        return cls.NAME

    def get_self_name(self):
        return self.NAME

I update the pylance to 2021.3.2, this problem also exists.

Thanks for the bug report. This will be fixed in the next release. It occurs specifically when accessing a class member from the cls parameter in a classmethod.

This issue has been fixed in version 2021.3.3, which we've just released. You can find the changelog here: https://github.com/microsoft/pylance-release/blob/main/CHANGELOG.md#202133-24-march-2021


convert to big number, serialization issue

fixes https://github.com/AugurProject/augur/issues/3702
this was a blocker for working on other task, just fixing it.

combining on integration PR.


Introduce semantic release strategy

With automatic npm publish and github tags creation.
Also make sure all commits will not be done before tests and coverage won't be verified.
Next npm libraries shall be used:

semantic-release
commitizen
ghooks

Also for commitizen I'm going to use cz-conventional-changelog in order to follow right format for commit messages for semantic-relese'ing.


Issues due to mixing pdfbox versions

Issues like the following appear due to mixing pdfbox versions:
[main] WARN org.apache.fop.apps.FOUserAgent -  COMMONS-LOGGING Unable to load font file: file:/C:/WINDOWS/FONTS/DavidCLM-Bold.otf. Reason: java.lang.RuntimeException: java.lang.NoSuchMethodError: 'java.util.List org.apache.fontbox.cff.CFFParser.parse(byte[])'
MustangProject is using pdfbox 3.x, whereas FOP 2.10 (as used by MustangProject) is still using pdfbox 2.x.
pdfbox 3.x does not contain the signature "CFFParser.parse(byte[])" anymore.

Nothing to be fixed, just important to be aware of. It seems that future versions of FOP are going to use pdfbox 3.x as well.


Add unit of measurement to Curtain Illuminance

Add unit of measurement = "Level"(same as Home Assistant default switchbot integration), then Home Assistant will show Illuminance History as graph.

I will add this to the v7 release


Button card error when entity is not set

Works great in the browser and in the iOS app.

Log:
0417 20:35:06 [Error] :  Caut Flutter runtime error: NoSuchMethodError: The setter 'overrideName=' was called on null.
Receiver: null
Tried calling: overrideName=
0417 20:35:07 [Debug] :  [Received] <== event.state_changed: switch.water_heater
0417 20:35:07 [Debug] :  [Received] <== event.state_changed: binary_sensor.living_room_tv_sensor
0417 20:35:07 [Error] :  Caut Flutter runtime error: NoSuchMethodError: The setter 'overrideName=' was called on null.
Receiver: null
Tried calling: overrideName=

Thanks for the report. Could you please show me the config of that view?

Never mind I found an issue

@estevez-dev this is a button triggering a scene. if you still need the configuration let me know.


Autonat support

An implementation of Autonat compatible with https://github.com/libp2p/specs/tree/master/autonat and https://github.com/libp2p/go-libp2p-autonat is required.
See also: https://docs.libp2p.io/concepts/nat/

backlog grooming: nearly done, but needs some time spent on it.


[Feature Request] Extend Contract logic with COPY and/or RENAME

Renaming assay identifier should also rename parent folder.
Moving files, not stored in model to another assay might require to COPY a file from a to b.

@HLWeil @kMutagene @omaus

Yup rename might be necessary to rename the folders of assays/studies. Not sure whether there would still be a need for copy then.

https://github.com/nfdi4plants/ARCCommander/issues/212

Rename specifically could be very important to handle soon:
https://github.com/nfdi4plants/ARCitect/issues/127


Wheels not published for each platform

Travis builds wheel files using the manylinux docker image. See an upstream example and ours.
This issue is to build and publish a wheel file for different platforms (and architectures?)

For reference this is what numpy uploads to pypi:
numpy-1.17.1-cp35-cp35m-macosx_10_9_x86_64.whl
numpy-1.17.1-cp35-cp35m-manylinux1_i686.whl
numpy-1.17.1-cp35-cp35m-manylinux1_x86_64.whl
numpy-1.17.1-cp35-cp35m-win32.whl
numpy-1.17.1-cp35-cp35m-win_amd64.whl
numpy-1.17.1-cp36-cp36m-macosx_10_9_x86_64.whl
numpy-1.17.1-cp36-cp36m-manylinux1_i686.whl
numpy-1.17.1-cp36-cp36m-manylinux1_x86_64.whl
numpy-1.17.1-cp36-cp36m-win32.whl
numpy-1.17.1-cp36-cp36m-win_amd64.whl
numpy-1.17.1-cp37-cp37m-macosx_10_9_x86_64.whl
numpy-1.17.1-cp37-cp37m-manylinux1_i686.whl
numpy-1.17.1-cp37-cp37m-manylinux1_x86_64.whl
numpy-1.17.1-cp37-cp37m-win32.whl
numpy-1.17.1-cp37-cp37m-win_amd64.whl
numpy-1.17.1.zip

This could help? https://github.com/joerick/cibuildwheel

Needs to be automatically published to pypi e.g. the macos binary wheel

with release 0.13.0 we publish wheels for Linux, Mac, Windows for Python 3.6, 3.7, 3.8.
That should be enough for now.


Crash when decrypting base64 encoded secret

Thread 1: Fatal error: 'try!' expression unexpectedly raised an error: CryptoKit.CryptoError(identifier: "EVP_CipherFinal_ex", reason: "Failed finishing cipher.: error:00000000:lib(0):func(0):reason(0).")
This is my library to encrypt/decrypt passwords for remote nodes on my CI:
https://github.com/Einstore/Speedster/blob/master/Sources/SpeedsterApi/Libs/Secrets.swift
the tests for it here:
https://github.com/Einstore/Speedster/blob/master/Tests/SpeedsterApiTests/SecretsTests.swift
It only crashes if the secret has been decoded from base64 string ...

Closing as Swift Crypto is now used instead of this library


how parse 100G sql file to csv files?

how parse 100G sql file to csv files?
thanks

@hktalent, what exactly do you want to parse? The question is too broad. What do you want to accomplish by that?
Note that the parser, as stated multiple times, is permissive. If you want to parse for syntax validation, then this isn't the place to.
In the end, the parser doesn't exporting CSV as long as I'm aware. Check the README.md, there is an explanation of everything that you can or cannot do.
@alamb, this could be closed IMHO.

sqlparser isn't a streaming parser (aka it will parse the entire tree into memory).
If you have 100GB of SQL with data, like INSERT INTO .... VALUES ... one thing you could do would be to "tokenize" the sql outside of sqlparser (aka split on \n) and then parse each line / statement individually

@alamb @AugustoFKL
Thanks, I have written a simple parser myself, and the test passes


displaying issues

Problem with displaying... Scrollbars in content . see attached screenshot
after restarting brackets it works the first time. 2nd fails like shown.

Hey,
what version of brackets and operating system are you using ?

Release 1.7 Build 1.7.0-16898 (release b0a363b71)
Time of Build: Wed Jun 08 2016 12:27:56 GMT+0100
on windows 10 Enterprise

Still no scrollbars
Release 1.9 build 1.9.0-17312 (release 189f6d39a)


  Will need to debug

The encoding problem or something else entirely like a random byte thrown in there that doesnt make sense?

 Will need to debug

The encoding problem or something else entirely like a random byte thrown in there that doesnt make sense?
I can try to look at it in Ghidra later if you want.

I think there were 2 errors. One for decoding and there ended being another that I don't remember what it was (I replaced both with data8 in the schemas for now)

Ok thanks god the whole entry is fixed length I guess. Might check what is up later

t_place:

1 - (size = 8 bytes), 0x2710 - Looks like a scenario flag, but could also be an ID (it doesn't seem unique) but is not read as a scena flag anywhere I could find)
2 - (size = 8 bytes, data pointed to size = until first 0 is reached) 0x3F84 - string that usually is 5 bytes long and ends with a 0.
3 - (size = 8 bytes, data pointed to size = until next multiple of 4 bytes is reached) 0x3F8A - another string (points right after the previous string null terminator)
(note: the whole sequence of strings 2 + 3 seems to need to be aligned so there might be padding 0s to ensure the next data starts at the beginning of a 4 bytes word)
4 - (size = 8 bytes, data pointed to size = 2 * (value of (5)) - 0x3F8C - the next pointer points to a sequence of 2 bytes words (written as 4 bytes)
5 - (size = 8 bytes) 0x01 - the number of 2 bytes words pointed to by 4
6 - (size = 8 bytes, 0x3F8E - data pointed to size = 2 * (value of (7)) another pointer to a number of 2 bytes words
7 - (size = 8 bytes) 0x01 - the number of 2 bytes words pointed to by 6
8 - (size = 4 bytes) 0x3F800000 - a float
9 - (size = 8 bytes, data pointed to size = until 0 is reached) 0x3F90 - a pointer to a latin-1 (if you want) encoded array of bytes (likely should only take the values "A" and "D") terminating at 0

10 - (size 8 bytes, data pointed to size = until 0 is reached) 0x3F91 - UTF 8 encoded text


ViewModel

For now I'm finished with the ViewModel logic. Note to self - when you finish with persistence, find a way to separate ListViewModel and AddViewModel. You will need a second one for the detailed view.


Bookstack fails to send email through relay

Attempted Debugging

[X] I have read the debugging page

Searched GitHub Issues

[X] I have searched GitHub for the issue.

Describe the Scenario
When sending a test email to an open, internal relay with no encryption, email fails to send.

Exact BookStack Version
v24.10.3
Log Content
Nothing pertaining to email failure in any log file.   APP_DEBUG=true does not display errors.
/var/lib/nginx/logs
/app/www/storage/logs
/app/www/vendor/aws/aws-sdk-php/src/data/logs
Hosting Environment
Kubernetes v1.31.2+k3s1 using the image provided by linuxserver linuxserver/bookstack:latest
Container is able to resolve the mail relay.
There is no authentication or encryption required (or enabled) to use the relay.  Strictly old-timey port 25 only.
I'm at a lost since this is about as simple as an email relay can get.
Details

apiVersion: apps/v1
kind: Deployment
metadata:
  name: bookstack-tech
  namespace: bookstack-tech
spec:
  replicas: 1
  selector:
    matchLabels:
      app: bookstack-tech
  strategy:
    type: Recreate
  template:
    metadata:
      labels:
        app: bookstack-tech
    spec:
      containers:

      - name: bookstack-tech
        image: linuxserver/bookstack:latest
        imagePullPolicy: Always
        env:
        - name: TZ
          value: America/Chicago
        - name: PUID
          value: "1000"
        - name: PGID
          value: "1000"
        - name: APP_URL
          value: "XXXXXXXX"
        - name: APP_KEY
          value: "XXXXXXXX"
        - name: DB_HOST
          value: "XXXXXXXX"
        - name: DB_PORT
          value: "3306"
        - name: DB_USERNAME
          value: "XXXXXXXX"
        - name: DB_PASSWORD
          value: "XXXXXXXX"
        - name: DB_DATABASE
          value: "XXXXXXXX"
        - name: MAIL_DRIVER
          value: "smtp"
        - name: MAIL_FROM_NAME
          value: "STUFF Admin"
        - name: MAIL_FROM
          value: "XXXXXXXXX"
        - name: MAIL_HOST
          value: 'mail.XXXX.XXX'
        - name: MAIL_PORT
          value: '25'
#        - name: APP_DEBUG
#          value: "false"
        ports:
          - containerPort: 6875
            protocol: TCP

        volumeMounts:
        - name: bookstack-tech-claim0
          mountPath: "/config"

      volumes:
        - name: bookstack-tech-claim0
          persistentVolumeClaim:
            claimName: bookstack-tech-claim0

I also tried editing the values in .env directly within the container, no change in behavior.   I will include them in the spirt of being thorough.
# Mail system to use
# Can be 'smtp' or 'sendmail'
MAIL_DRIVER=smtp

# Mail sender details
MAIL_FROM_NAME="STUFF Admin"
<EMAIL_ADDRESS>
# SMTP mail options
# These settings can be checked using the "Send a Test Email"
# feature found in the "Settings > Maintenance" area of the system.
# For more detailed documentation on mail options, refer to:
# https://www.bookstackapp.com/docs/admin/email-webhooks/#email-configuration
MAIL_HOST=mail.XXXX.com
MAIL_PORT=25
MAIL_USERNAME=null
MAIL_PASSWORD=null
MAIL_ENCRYPTION=null
MAIL_VERIFY_SSL=false

I'll try to fix the formatting, `` code didn't work well.

Hi @KCKgithub,
A couple of notes on this:

Even on port 25 with TLS disabled, The mail system in BookStack will attempt to upgrade and use TLS if advertised by the mail system. If TLS is announced as supported by the mail server, but it cannot handle it, then things will fail.

I also tried editing the values in .env directly

Keep in mind that the .env values may still be overidden by the container env options, since they take priority, so settings may not have been acting as you expect. If you want to see which settings BookStack is probably picking up, you can run the following against your instance: php artisan tinker --execute="dd(config('mail.mailers.smtp'))"

Hi @KCKgithub, A couple of notes on this:

Even on port 25 with TLS disabled, The mail system in BookStack will attempt to upgrade and use TLS if advertised by the mail system. If TLS is announced as supported by the mail server, but it cannot handle it, then things will fail.

I also tried editing the values in .env directly

Keep in mind that the .env values may still be overidden by the container env options, since they take priority, so settings may not have been acting as you expect. If you want to see which settings BookStack is probably picking up, you can run the following against your instance: php artisan tinker --execute="dd(config('mail.mailers.smtp'))"

Thanks Dan!
This got me where I needed to get.
All working now.
For anyone that lands here through search.
If you are using postfix as a unencrypted email relay (port 25) you need to ensure that it is explicitly disabled in main.cf:
smtpd_tls_security_level = none
This will allow Bookstack to send emails using an internal relay and still allow Postfix to use TLS as a client to an email server clearing house (gmail.com in my case).
My Postfix is running in compatibility mode 3.6 FWIW.
php /app/www/artisan tinker --execute="dd(config('mail.mailers.smtp'))"
array:10 [
"transport" => "smtp"
"scheme" => null
"host" => "mail.yourdomain.com"
"port" => "25"
"username" => null
"password" => null
"verify_peer" => false
"timeout" => null
"local_domain" => null
"tls_required" => false
]
Thanks again!
KCK

Good to hear things are working! I'll therefore close this off.


Adding click command breaks pygments

Expected Behavior
It should take in one parameter- the URL, then format and highlight code given from github raw code url.
from pygments import highlight
from pygments.lexers import get_lexer_by_name
from pygments.formatters import terminal
import requests
import click

# command and argument cause the problem
@click.command()
@click.argument("url")
def ghs(url):
    s = requests.Session()

    s.headers["user-agent"] = "<my user agent>"
    s.headers["cookie"] = "<my user cookie>"

    r = s.get(url)

    code = r.text

    lexer = get_lexer_by_name(r.url.split('.')[-1], stripall=True)
    formatter = terminal.TerminalFormatter()
    result = highlight(code, lexer, formatter)
    print(result)

Actual Behavior
None of the formatting and highlighting are working, and instead
it prints the exact characters and doesn't change any color.
It works fine without click command and click argument.

←[34mfrom←[39;49;00m ←[04m←[36mpygments←[39;49;00m←[04m←[36m.←[39;49;00m←[04m←[36mformatters←[39;49;00m ←[34mimport←[39;49;00m terminal
←[34mimport←[39;49;00m ←[04m←[36mrequests←[39;49;00m

s = requests.Session()
s.headers[←[33m"←[39;49;00m←[33muser-agent←[39;49;00m←[33m"←[39;49;00m] = ←[33m"←[39;49;00m←[33m<YOUR USER AGENT>←[39;49;00m←[33m"←[39;49;00m
s.headers[←[33m"←[39;49;00m←[33mcookie←[39;49;00m←[33m"←[39;49;00m] = ←[33m"←[39;49;00m←[33m<YOUR COOKIE>←[39;49;00m←[33m"←[39;49;00m

r = s.get(←[33m"←[39;49;00m←[33mhttps://raw.githubusercontent.com/Lvyn/neco-net-compiler/cb80bab29cf8d8f8546d58dfdac016d94293138a/tests/benchmarks/ns_nospy/dolev_yao.py←[39;49;00m←[33m
"←[39;49;00m)

code = r.text

lexer = get_lexer_by_name(r.url.split(←[33m'←[39;49;00m←[33m.←[39;49;00m←[33m'←[39;49;00m)[-←[34m1←[39;49;00m], stripall=←[34mTrue←[39;49;00m)
formatter = terminal.TerminalFormatter()
result = highlight(code, lexer, formatter)
←[36mprint←[39;49;00m(result)

Environment

Python version: 3.9.1
Click version: 7.1.2

I can't reproduce your issue. I copied your code, and used it to print out Click's setup.py file in color.
Are you on Windows? You need to install colorama to get color on Windows, and that's not specific to Click. Also, when using Click (on any platform) you should use click.echo instead of print, which will automatically use colorama if it's installed as well as handle some other things that print doesn't.

I can't reproduce your issue. I copied your code, and used it to print out Click's setup.py file in color.
Are you on Windows? You need to install colorama to get color on Windows, and that's not specific to Click. Also, when using Click (on any platform) you should use click.echo instead of print, which will automatically use colorama if it's installed as well as handle some other things that print doesn't.

Thank you, that fixed it.

Thank you, that fixed it.


Exception on Windows shutdown

Getting the same thing on every shutdown right now

HookTask probably needs a CancellationToken when being disposed.


Conformity to ROS IMU message

Hi,
Does SbgImuData message have the same coordinate system as the sensor_msgs/Imu or is it different?
Thanks

Hi @ashnarayan13,
I believe the measurements are made within a NED reference frame.
Cheers


add unidling APIs

These are the current APIs used by the openshift-controller-manager, oc, and the proxier.

this is promoting existing code.  @squeed confirmed that these are supported.


RFC: stop using the _async terminology

Between the _async and _future suffixes, there was quite a zoo of
possibilities:

Scope::spawn
Scope::spawn_future
::spawn_async
::spawn_async_future

After this PR, there are now two orthogonal concepts. First, what
scope are you spawning the task into? If you use the free-standing
functions, like rayon::spawn(), or methods on a thread-pool, like
pool.spawn_future(), that is the "static" or "global" scope, and
hence no stack references are allowed. Otherwise, if you create a scope
using scope(), then when you do scope.spawn() it is spawned into
that scope.
Second, are you spawning a task for side-effects (spawn()) or to
produce a result (spawn_future()).
NB: Everything but Scope::spawn() remains "unstable" in this PR.

I also considered adding a synthetic "global scope", so that you would do something like rayon::GLOBAL_SCOPE.spawn(), but it just looks ugly.

I suppose rayon::global_scope().spawn(...) is not so ugly. And, to be clear, this idea of a global scope is actually something that exists in the implementation: the futures code for example uses the Registry as a kind of "global scope" (i.e., it implements the FutureScope trait). Hmm, I sort of like that.
One could also removing the free functions and giving a way to get a handle on the "global thread-pool" (probably an Arc<ThreadPool>). Then you would do rayon::global_pool().spawn() and hence there would be two spawn methods, one on thread-pools and one on scopes.

One could also removing the free functions and giving a way to get a handle on the "global thread-pool" (probably an Arc). Then you would do rayon::global_pool().spawn() and hence there would be two spawn methods, one on thread-pools and one on scopes.

I like the idea of a global_pool(), but note that this is not exactly the same as the free functions.  Those operate on the current pool (Registry::current()) which is still valuable in itself.  I think that's a good generalized model -- use free functions for the current pool, ThreadPool methods for a targeted pool.

@cuviper

I think that's a good generalized model -- use free functions for the current pool, ThreadPool methods for a targeted pool.

Yes. I realized this at some point. =)

The only thing not clear to me -- already underdocumented -- is the lifetime of Scope::spawn_future.

Yeah, we need docs here.

I guess the returned future must either be completed or dropped before the scope returns, but it would be nice to see this spelled out.

Probably this is accurate, I'm not 100% sure what you mean. The answer is that the future must have finished executing before the scope is dropped; but the returned future can escape the scope, if its types don't include any references.

@cuviper so I'm inclined to merge this PR, and leave the docs of future for another PR. In particular, I want to open a PR that moves the future support out of core (which is blocked on this one), and we can start thinking about docs then. Seem ok?

@cuviper ok, fixed the docs I think (I tested and the links seem to resolve too...)

The current nightly is broken for CI, but locally my slightly older nightly tested fine.
👍


Fix 406 from tika-server when calling tika.parse1('text'...)

responseMimeType in tika.parse1 was changed to text/plain
after the headers were already set, resulting in
applicaiton/json being sent to tika-server which then
returned a 406.  This commit fixes the issue and also
does some white-space cleanup.
Signed-off-by: Jason Dossett<EMAIL_ADDRESS>

Thanks so much for patching.  Sorry about all the whitespace removals.

anytime thanks for the PR @jdossett !


feat: Add overlay to scanner

What

This PR adds an option to add an overlay to the scanner which is only visible when the scanner has started.
Reopening of https://github.com/juliansteenbakker/mobile_scanner/pull/586 due to restructuring of fork.

LGTM ! Thanks for your contribution! I'm gonna change it a littlebit so that it doesn't use the stack when no overlay is given.


What is EnableConnectionPool?

        /// <summary>
        ///     Should we enable experimental server connection pool?
        ///     Defaults to true.
        /// </summary>
        public bool EnableConnectionPool { get; set; } = true;

Is this setting up connections to the remote proxy servers? How many will it set up? Where can I find some more info on this, as trying to trace back the code is getting me confused.

When you enable connection pooling, instead of creating a new TCP connection to server for each client TCP connection, we check if a server connection is available in our cached pool. If it is available in our pool, created from earlier requests to the same server, we will reuse that connection. There is also a ConnectionTimeOutSeconds parameter, which determine the eviction time for inactive server connections. In that way, we reduce the TCP connection establishment cost, both the wall clock time and CPU cycles. It was experimental when it was implemented years ago, now that it is stable and is enabled by default, we will update the docs.

Thanks for the info. I notice that the number in the pool seems to be higher than the total number of proxies available. Does it connect multiple times to the same server? There seems to be about a 3:1 ratio in my case

It depends. If you had 3 concurrent client connections to the same server at some point, it will stay in the pool even after client connections has dropped, until its evicted by timeout. Not sure what you mean by total proxies available.


Disable log output?

I'm making an assumption here that this output is from this plugin - if it is, would it be possible to disable it? Or at least put it behind a debug flag etc?
XoFixCommand -> is_enabled?
XoFixCommand -> project_root -> None
XoFixCommand -> xo_start_dir /home/*********/.config/sublime-text-3/Packages/User
XoFixCommand -> xo_path None
XoFixCommand -> xo_path -> False

FWIW, the debugging here is made via plain print(), you could and should instead use
import logging
logger = logging.getLogger("SublimeLinter.plugin.xo")
....
# use it
logger.info("lirum")
logger.warn("larum")

This way it can be enabled/disabled with SublimeLinter's own debug switch.

// @LuKks


[Import API] Quota checker tries to subtract nil value when UserDataSize fail

The quota checker uses the method raise_if_over_storage_quota in order to check if the user is over storage quota:
def raise_if_over_storage_quota(requested_quota: 0, available_quota: 0, user_id: nil)
    quota_overage = requested_quota - available_quota

    if quota_overage > 0
        report_over_quota(user_id, quota_overage: quota_overage) if user_id

        raise StorageQuotaExceededError.new
    end
end

However, the parameter available_quota could be nil if some error occurred while the importer is getting the user available size through the method db_size_in_bytes raising the error TypeError: nil can't be coerced into Float.
We should check if the parameters are not nil or managing the errors occurred while the function db_size_in_bytes was being executed.

Fixed in #15512


switch does not honor .gitignore

When switching branches, gitless does not honor .gitignore and will stash ignored files and directories.  This is true of the .gitignore file itself.   Each branch will have it's own version of .gitignore.   Is this a 'feature' or an issue?   Should ignored entries not be ignored?   This seems counterintuitive.

I was able to fix this on my own by changing line 328 of core.py from '--all' to '--include-untracked' which seems like more logical behavior to me.
"If the --include-untracked option is used, all untracked files are also stashed and then cleaned up with git clean, leaving the working directory in a very clean state. If the --all option is used instead then the ignored files are stashed and cleaned in addition to the untracked files."

The default behavior is to keep branches independent from each other so ignored files are stashed when you switch. If you'd like the changes in the current branch to follow you to the destination branch you can use the -mo/--move-over flag of gl switch

I don't see the --move-over option as a suitable solution.  Ignored files should exist outside of git and should be ignored.   Having to use --move-over eliminates much of the benefit of gitless.

Can you tell me more about your use case? What kind of files are you ignoring and why do you want them to follow you as you switch branches?

I tend to keep notes and config files that are only of interest to me in my directories and I list these in .gitignore.  My big issue with gitless in switching branches is that it also stashes .gitignore. Which seems counter intuitive as it's a git control file.  I guess I could duplicate .gitignore per branch but that isn't a great solution.  It's a simple change (replacing --all with --include-untracked) and thanks to git/gitless I can maintain my own local branch of gitless so it's not a big deal.
Other than this issue, gitless has solved 90% of my frustration in using git.


Penny: links to products gone

Links to penny product pages are gone. Instead index.html on heissepreise server is referenced

Regression of #55


[Application] Implement Organization-Verification PingPong

To verify that an applicant actually works as a volunteer, we need to implement an email-based verification with the organization.
Each organization that the user has entered during the application process should be contacted via mail with a verification link.
Clicking on this link, the organization can Verify that the applicant has worked as a volunteer in their organization.
Once all organizations have verified the applicant's work, the application is shown more prominently in the administration frontend.

[ ] sending e-mail after creating the application
[ ] a view to verify an application for an "organization" employee
[ ] split organization information from application in the database
[ ] reflect the "organization" verification state in administration frontend
[ ] inform the applicant, whether the application was not successful (maybe allow the region manager, etc. to decide whether an email should be sent)


Add cla command

Hey,
Adding a cla command to bot so that we can ping users that forgot about it more conveniently. I am not really sure if that's everything we have to do in order for the bot to use it - but just wanted to give it a go.
CC: @mkonicek
PS. Wording probably needs some loving :)

Shipping as per @mkonicek comment on Facebook group https://www.facebook.com/groups/reactnativeoss/permalink/1544673329162629/?comment_id=1545417385754890&reply_comment_id=1545607015735927&comment_tracking={"tn"%3A"R4"}

@facebook-github-bot shipit

Closed in https://github.com/facebook/react-native/commit/9bae30f8b519763c212e7738391be52dd2e20981


Failing to install inputstreamhelper with the latest Kodi 19

With the latest nightly for Kodi 19 win64 installing inputstreamhelper fails.

Okay, this comes as a total surprise for me.
Apparently on the 1st of March https://github.com/xbmc/xbmc/pull/17188 was merged and this broke all addons with <import addon="xbmc.python" version="2.25.0"/> in addon.xml
I didn't receive any communication from Kodi Team regarding this breaking change. As far as I know the relevant Kodi wiki pages are not updated to inform add-on developers about this breaking change:
https://kodi.wiki/view/General_information_about_migration_to_Python_3
https://kodi.wiki/view/Addon.xml
It seems we have to submit our add-on twice now, one time for Kodi 18 (and older) and one time for Kodi 19. I'll investigate what the best practice is to do so.

I think it's best to create a branch for <18 and make the necessary change to master. Since v0.4.4 is currently being released, it would be great if this was done quickly.

I won't add a change to master branch that breaks InputStream Helper for the currently released and most widely used Kodi 18 version.
Kodi 19 is still in Alpha and practically useless since https://github.com/xbmc/xbmc/pull/17188 has been merged and all Python3 compatible scripts/plugins were removed.
There is also no release date for Kodi 19 communicated, so there is no need to hurry.
Submitting our single Py2/Py3 compatible codebase twice makes no sense to me. I'll wait for Team Kodi to come up with a better solution.
v0.4.4 will be submitted once just like I did for the previous InputStream Helper versions.

You could just mark the new branch as default, but that's fine. I actually thought Kodi 19 was closer to release.

@horstle The problem is that we need to offer 2 ZIP files (pre v19 and post v19) for people to troubleshoot and maintain two branches. Where the only difference is one version in the addon.xml.

I totally agree, that having only one branch would be much less hassle, but from what I read it sounded like there was no alternative. I guess waiting is the best thing to do right now.
By the way does anybody know how it shows up in kodi, when an addon is submitted twice to different repos, but with the same version number? Will the user see the same entry twice? In that case on Matrix there would always be two entries, where one fails to install.

@horstle Entries submitted in any branch prior to Matrix will not be build for Matrix. So you will not have the same add-on version twice.

So I guess there is no other option, but to submit twice. Of course sticking with only one branch would still be much easier.

We will be sticking to one master branch in this project, which will work with v18 and older for now.
But to submit to xbmc/repo-plugins we may have to submit twice.
We could have a make target do this work for us: create 2 branches (krypton and matrix) and the PRs
And for releases we could create 2 ZIP files as well, but it is all so pointless...

No fixes available?
KodiSetup-20200226-cde760b9-master-x64 is the latest version that hasnt broke the addons.
All versions after that does not work.

No fixes available?
KodiSetup-20200226-cde760b9-master-x64 is the latest version that hasnt broke the addons.
All versions after that does not work.

Temporary you could use this slightly modified inputstreamhelper with the latest Kodi 19 Matrix just remember to install your add-ons from zip files and to install them after installing the modified inputstreamhelper, and in each one of your add-ons zip files open addon.xml and replace
<import addon="xbmc.python" version="2.25.0"/>
with
<import addon="xbmc.python" version="3.0.0"/>
script.module.inputstreamhelper-0.4.4.py3.zip
and here are plugin.video.retrospect-5.1.8 and plugin.video.retrospect-5.1.9 modified so you can install them on the latest Kodi 19 Matrix
plugin.video.retrospect-5.1.8.py3.zip
plugin.video.retrospect-5.1.9.py3.zip

Thank you so much! :)

script.module.inputstreamhelper-0.4.4.py3.zip

This is a misleading name because most versions work for both py2 and py3 anyway. Please indicate that they are customized to run on Matrix instead: script.module.inputstreamhelper-0.4.4-matrix.zip

@dagwieer I understand inputstreamhelpe work for both py2 and py3, I just named them like the developer of plugin.video.youtube named his version for matrix https://github.com/jdf76/plugin.video.youtube/releases/tag/6.6.0, and modified inputstreamhelpe for my personal use as I said before here https://github.com/CastagnaIT/plugin.video.netflix/issues/506#issuecomment-596341292.

Once https://github.com/emilsvennesson/script.module.inputstreamhelper/pull/293 gets merged, I could do a Matrix release via Github.

A matrix compatible zip package is available at https://github.com/emilsvennesson/script.module.inputstreamhelper/releases/download/v0.4.4/script.module.inputstreamhelper-0.4.4+matrix.1.zip

@mediaministe Shouldn't inputstreamhelper be on the Kodi Matrix repo same as it is on the Kodi Leia repo so the other add-ons that depend on inputstreamhelper install it while installing the add-ons on Kodi 19 Matrix (for example the Netflix add-on and Amazon VOD add-on)?

Many thanks for all the developers for submitting inputstreamhelper to the Kodi Matrix repo.

There have been changes in the Kodi 19 python API
the addon doesn't work anymore
https://github.com/xbmc/xbmc/pull/17456
https://github.com/xbmc/xbmc/pull/17494
please check thanks

It works very well on my end.
Using KodiSetup-20200318-809e36a2-master-x64

I'm working on it.

@CastagnaIT Can you please test https://github.com/emilsvennesson/script.module.inputstreamhelper/pull/298

yes now seem all working as expected

You need to remove line1, line2 ect in xbmcgui.Dialog(). otherwise it will fail during install.
https://forum.kodi.tv/showthread.php?tid=344263&pid=2933596#pid2933596

Is there a chance to get an updated version into the Kodi repo? Version 0.4.4 seems not to work with latest Matrix anymore.
I think @CastagnaIT and @Essam311 already provide a patched version here for the ntetflix plugin - maybe you could create a PR?

Thank you so much for submitting script.module.inputstreamhelper-0.4.5+matrix.1 to the Kodi Matrix repo.


Jitter at animation end

I've just updated our code to utilise the new Motion framework instead of the old one built in to Material.
Using the Motion.rotation(angle:) function on a FabButton (as primary button of a Material Menu) works fine in terms of animating the button. However, we're noticing a flash of the initial animation state at the end of the rotation (GIF recording)
menu.views.first?.animate(animation: Motion.rotation(angle: -90))
Any ideas what might cause this?

Can you try using motion for this?
menu.views.first?.motion(.rotationAngle(-90))

Thanks! That fixed the issue.

Awesome :)


🐛 BUG: wrangler publishes to workers.dev if there's no top level route, but does have one in an environment

What version of Wrangler are you using?
0.0.23
What operating system are you using?
Mac
Describe the Bug
Given this wrangler.toml
name = "worker-name"
main = "src/index.js"

[env.production]
routes = ["https://some=domain.com/*"]

and run the command wrangler publish --env production, wrangler will attempt to publish to both worker-name.threepointone.workers.dev and https://some=domain.com/.
I investigated it a little, and noticed that this section https://github.com/cloudflare/wrangler2/blob/82ec7c2c65b1515cf081420499091cd0878fed8d/packages/wrangler/src/config/validation.ts#L101-L105 marks workers_dev as true (since there aren't any routes there), and then passes topLevelEnv to the next call at https://github.com/cloudflare/wrangler2/blob/82ec7c2c65b1515cf081420499091cd0878fed8d/packages/wrangler/src/config/validation.ts#L114-L121 which assumes that the top level value doesn't get overridden by the defaultValue passed at https://github.com/cloudflare/wrangler2/blob/82ec7c2c65b1515cf081420499091cd0878fed8d/packages/wrangler/src/config/validation.ts#L531. I'm not sure what a fix for this is, hence filing the issue. @petebacondarwin do you have any ideas?

Also question, is there a scenario where folks will want to publish to both workers.dev and a route? I ask this, because I'm not actually sure how to publish to a plain route on Workers 🤪. I just setup a CNAME to proxy from the workers.dev deploy to my domain's route.

Ok, we're already setup ti publish to both if workers_dev = true and if routes are defined, so I guess there's no cod changes to make for the latter scenario.

Let's make workers_dev default to undefined in the config, and then move the logic for the "actual" default into the publish command.

Ok good shout. I'll send a PR.


Disabling pylance extension causes an error notification

Open a notebook
disable pylance
reload vs code
:bug: error notification: Could not get references provider from language server, Pylance prerelease required.

If relevant, python.languageServer is set to default, which claims that it can fallback if Pylance is uninstalled.

@rebornix looks like this is code related to the executionAnalysis stuff.
We seem to have a hard dependency on Pylance for that even if not using Pyhton kernels.
Perhaps we might want to check if the notebook being used in a Pythonnotebook or not.
You can use isPythonNotebook helper.

Good catch!


force:apex:execute does not exit with non-zero exit code when script does not compile

Hi,
Today I noticed that force:apex:execute does not exit with a non-zero exit code when the script that is supplied does not compile. I did not experiment further, but it would be good if it also exits with non-zero in case of execution exceptions, etc. The command does exit with a non-zero exit code when the file does not exist.
Cheers, Oscar
Steps To Reproduce:
Open a terminal/dos box and execute the following commands
C:\Users\Oscar\tmp>echo "i do not compile" > foo.apex
C:\Users\Oscar\tmp>sfdx force:apex:execute -u dx -f foo.apex
ERROR:  Compilation failed.
!    ERROR: Line: 1, Column: 1
!    Unrecognized symbol '"', which is not a valid Apex identifier.
ERROR:  Execution failed.
C:\Users\Oscar\tmp>echo %ERRORLEVEL%
0
Expected result
The process should have exited with a non-zero exit code.
Actual result
The process exits with a 0 exit code, which is inconsistent with general accepted behavior of command line tools.
Additional information
I'm running this on Windows 10 Pro
Version info:
C:\Users\Oscar\tmp>sfdx -v
sfdx-cli/7.1.4-79f97a7df8 win32-x64 node-v10.15.3

@ntotten, this looks like an error on the apex:execute command. Can you take a look

Yup, this looks like a bug. I added this to our backlog.

it seems that current version sfdx-cli/7.75.0 darwin-x64 node-v12.18.3 still has the issue.
This complicates CI/CD implementations where it is expected that pipeline should fail in case of error... Does anyone has maybe a workaround for that?

A workaround is to dump to json file and then parse the result flag (not the status code which is also always 0 in the json).
Then if the result is not true you can also output the compilation error or the full logs.
Example, if you have a bunch of apex files that are doing setup/integration tests/whatever, in the CI/CD pipeline you can do:
#!/bin/bash
set -e

cd integration_tests

for f in ./*.apex; do
    echo "Running test $f"
    # substitute environment variables in the apex code
    envsubst < "$f" > "$f.tmp" && mv "$f.tmp" "$f"
    # execute the apex code, capture the output to file
    sfdx force:apex:execute -f "$f" --json > out.txt
    # check for apex compilation success
    COMPILE_SUCCESS=`cat out.txt | jq -r .result.compiled`
    if [ $COMPILE_SUCCESS != "true" ]
    then
        # The result json contains a terse error description without too much noise
        cat out.txt | jq -r .result
        exit 1
    fi
    # check for apex execution success
    EXECUTION_SUCCESS=`cat out.txt | jq -r .result.success`
    if [ $EXECUTION_SUCCESS != "true" ]
    then
        # Returning just the logs value will mean it renders correctly in terminal without escaped characters
        cat out.txt | jq -r .result.logs
        exit 1
    fi

    rm -f out.txt
done

Requires jq to parse the json, the envsubst thing is not a requirement but useful in CI/CD environments

If you don't have jq installed, here's an adaptation of @jamesweakley's script above that uses the built-in grep command instead.  You could get fancier with this and grep just the first 2 lines of the output to ensure you're really getting the actual status, but the chances of the exact lines "Compiled successfully." and "Executed successfully." being elsewhere in the file are pretty slim.
#!/bin/bash
set -e

cd integration_tests

for f in ./*.apex; do
    echo "Running test $f"
    # substitute environment variables in the apex code
    envsubst < "$f" > "$f.tmp" && mv "$f.tmp" "$f"
    # execute the apex code, capture the output to file
    sfdx force:apex:execute -f "$f"  > out.txt

    # check for apex compilation success - should return "1" if the line was found
    COMPILE_CHECK=`grep -c -x 'Compiled successfully.' out.txt`

    if [ $COMPILE_CHECK -eq 0 ]
    then
        cat out.txt
        echo "******FAILED to compile apex script $f. ***********"
        exit 1
    fi
    # check for apex execution success - should return "1" if the line was found
    EXECUTION_CHECK=`grep -c -x 'Executed successfully.' out.txt`

    if [ $EXECUTION_CHECK -eq 0 ]
    then
        cat out.txt | grep 'Exception'
        echo "*****************Failed while running apex script $f.  See errors above*************"
        exit 1
    fi

    rm -f out.txt
done

Two years later we discovered a potentially dangerous build situation where sfdx apex execute to schedule some reoccurring tasks was silently failing because it's not reporting the error back

This was fixed in sfdx v7.190.0, see release notes:
https://github.com/forcedotcom/cli/tree/main/releasenotes/sfdx#71900-mar-2-2023


#393 update Maven version to 3.8.1

Build locally according to guideline and tested with a local project. OS is Linux.
Is this already enough to verify this is working?
closes #393

Given 3.8.1 creates some incompatibilities, I'd really like to get the time to work on supporting multiple maven versions as indicated by #384.

Given 3.8.1 creates some incompatibilities, I'd really like to get the time to work on supporting multiple maven versions as indicated by #143 .

Currently, Maven does not release that often. But in general mvnd might require to become more reponsive to release, e.g in case of bugfix releases or Maven 4. So yes, supporting multiple versions might be necessary but I guess this won't be that easy to do, right? And I guess requires more time.
In this regard, what happens with this pull request and by that the Maven 3.8.1 support?