Various --from-dash improvements

Fixes #4401 #4352 #4043 #3825
What this PR solves / how to test:
This improves wrangler init --from-dash to:

Always use JavaScript, rather than prompting for TypeScript (the API only returns JavaScript files right now)
Support Custom Domains & workers.dev property setting
Hide standard usage model if set

Author has addressed the following:

Tests

[x] Included
[ ] Not necessary because:

Changeset (Changeset guidelines)

[ ] Included
[ ] Not necessary because:

Associated docs

[ ] Issue(s)/PR(s):
[ ] Not necessary because:

Note for PR author:
We want to celebrate and highlight awesome PR review! If you think this PR received a particularly high-caliber review, please assign it the label highlight pr review so future reviewers can take inspiration and learn from it.

A rebase should fix the yarn e2e tests

@zebp Could you have a look and see if the updates here integrating https://github.com/cloudflare/workers-sdk/pull/4521 are valid?

@penalosa I'm still seeing the behavior from #4352 in this pre-release.
To repro:

I created a new worker from the dash.
I installed the prerelease here and ran the command in the develop with Wrangler CLI guide
I'm still prompted for TS/JS (which shouldn't happen)
I still get a src/worker.ts file created even when choosing "no" to TS.

@admah That'll probably happen until C3 is released, this this coe path bounces back and forth between the two. Does it work now in the deployed version?


Warning while uploading image via Media > Add New

Reported by @stbeli
Warning while uploading image via Media > Add New:
Warning: Error while sending QUERY packet. PID=28098 in /home/ucdev/web/wp-stateless.update.care/public_shtml/wp-includes/wp-db.php on line 1877
Server Configuration:
PHP: 7.0.19-1~dotdeb+8.1;
Web Server: Apache / nginx
WordPress: 4.8.1

I was not able to recreate this.

@ewsopp Please contact with @stbeli if he still has the issue.
Seems like he needs to increase max_allowed_packet in mysql configuration file (my.ini).
Related topic on wp.org.
https://wordpress.org/support/topic/e_warning-error-while-sending-query-packet-pid18213-stack-trace-lled-at-ho/

Not sure what resolved the issue, but @stbeli is reporting all issues resolved on his end.


kerning in man file

bad config file can be closed

@StrictAvacado not a big deal at all, but it's good GitHub practice to leave some history, I'd advise against editing the original description, and just leaving a note in a comment.


fix(activeHref): set active href in constructor, not mount

Currently InteriorLeftNav wipes out initial state on componentDidMount - reset to let the initial state declaration accept window.location.pathname if nothing else is passed in

:+1: looks good to me


Player-InvitePlayer-PlayTogether

Story: As a logged-in user who have already been in a game room, he/she can invite other logged-in users to player together.
Acceptance Criteria:

[x] In the “waiting area” page, a progress bar is shown to all users who have already joined the game room, which tells what percentage of players have been in the room.
[x] In the waiting page, there is a “invite online players” area which lists pairs of online friend’s username and a “invite” button.
[x] By clicking on the “invite” button, the users (both owner of the game room and other players) who have already been in a game room, they can send invitations to online friends to enter this game room.
[x] A logged-in user who receives invitation to a game room will see the invitation in the “Notification” page.
[x] There is a “accept” button and a “ignore” button near each invitation in the “Notification” page.
[x] If the user clicks “accept” to accept the invitation, he will be redirected to the “waiting area” page.  This invitation request is processed in the backend.
[x] After a user accepts invitation and join the game room, the progress bar is updated.
[x] If the user clicks “ignore” button, he will stay in the notification page.  This invitation request is processed in the backend.
[x] When the game room is full of players, they cannot invite other logged-in users.
[x] When the game room is full of players, the progress bar is updated as 100 percent.
[x] There is a “start” button shown in the “waiting” page for the room owner. The game room owner can click “start” button to start the game.

Development Tasks:

[x] sopra-fs23-group-19/sopra-fs23-group-19-client#32
[x] sopra-fs23-group-19/sopra-fs23-group-19-client#33
[x] sopra-fs23-group-19/sopra-fs23-group-19-server#53
[x] sopra-fs23-group-19/sopra-fs23-group-19-client#34
[x] sopra-fs23-group-19/sopra-fs23-group-19-server#55
[x] sopra-fs23-group-19/sopra-fs23-group-19-server#56
[x] sopra-fs23-group-19/sopra-fs23-group-19-server#57

add a list of friends on the waiting page to invite friends to join the game: https://github.com/sopra-fs23-group-19/sopra-fs23-group-19-client/commit/b604db57a33f3dcec2358ae8342f115c03f81b45

https://github.com/sopra-fs23-group-19/sopra-fs23-group-19-client/commit/7f5768b7af7f2c9004e1c1378da4fa0d2260e023


Unsure how to throw my own errors

I'm new to both GraphQL and Spring boot, so bear with me please.
I'd like to throw my own errors, instead of getting the
{
  "data": {
    "workflows": {
      "delete": null
    }
  },
  "errors": [
    {
      "message": "Internal Server Error(s) while executing query"
    }
  ]
}

I'm using the graphql-java-tools support and letting the starter auto configure everything. Just have the schema.graphqls in classpath and some Spring beans, that implement com.coxautodev.graphql.tools.GraphQLResolver.
I initially tried to code my resolver methods like so
public String resolverMethod(String key) throws Exception {
    ...
    if (...some validation...) {
        ... some post validation ...
    } else {
        throw new graphql.GraphQLException("Validation failed");
    }
    ...
}

but in the end I get same error as before, and the Exception only appears in my server logs.
I read somewhere that it's the graphql servlet hiding the exception, but I don't know how to modify the servlet, since it's automatically created and configured by the Spring boot starter.
Long story short, I wanna know if I can return my own errors, while still using the Spring boot starter, or will I need to stop using it and configure everything manually from scratch?

You can override the default implementation of error handling by creating your own error handler. For example:
public class CodedGraphQLErrorHandler extends DefaultGraphQLErrorHandler {

    @Override
    protected List<GraphQLError> filterGraphQLErrors(List<GraphQLError> errors) {
        return errors.stream()
                .map(this::codedError)
                .collect(Collectors.toList());
    }

    private GraphQLError codedError(GraphQLError error) {
        if (error.getErrorType() == ErrorType.DataFetchingException) {
            ExceptionWhileDataFetching exceptionWhileDataFetching = (ExceptionWhileDataFetching) error;
            Throwable exception = exceptionWhileDataFetching.getException();
            if (exception instanceof AccessDeniedException) {
                return new CodedGraphQLError(HttpStatus.FORBIDDEN, exception.getMessage());
            } else if (exception instanceof BadCredentialsException || exception instanceof PresetException) {
                return new CodedGraphQLError(HttpStatus.BAD_REQUEST, exception.getMessage());
            } else if (exception instanceof AuthenticationException) {
                return new CodedGraphQLError(HttpStatus.UNAUTHORIZED, exception.getMessage());
            }
        }
        return new CodedGraphQLError(HttpStatus.INTERNAL_SERVER_ERROR, error.getMessage());
    }
}

public class CodedGraphQLError extends GenericGraphQLError {

    private HttpStatus httpStatus;

    public CodedGraphQLError(HttpStatus httpStatus, String message) {
        super(message);

        this.httpStatus = httpStatus;
    }

    public int getCode() {
        return httpStatus.value();
    }

    public String getReason() {
        return httpStatus.getReasonPhrase();
    }
}

And then create your own instance of the GraphQLServlet like this
@Configuration
public class GraphQLConfig {

    @Bean
    public GraphQLServlet graphQLServlet(GraphQLSchemaProvider schemaProvider, ExecutionStrategyProvider executionStrategyProvider) {
        return new SimpleGraphQLServlet(schemaProvider, executionStrategyProvider, null, null, new CodedGraphQLErrorHandler());
    }
}

Judging by your code, this means I'll also need to build the GraphQLSchemaProvider and ExecutionStrategyProvider somehow, so that I can pass it to the servlet. Correct?

No, not necessarily because in the autoconfigurer those are created if they are missing. So you don't have to instantiate those, you can assume they are created.

Yup. That did it. Major thanks :)

Just to clarify, the graphql autoconfiguration will wire in a GraphQLErrorHandler you provide to the context, so you don't need to create the GraphQLServlet yourself, just mark your GraphQLErrorHandler with @Component, or create a bean method that builds it.  https://github.com/graphql-java/graphql-spring-boot/blob/master/graphql-spring-boot-autoconfigure/src/main/java/com/oembedler/moon/graphql/boot/GraphQLWebAutoConfiguration.java#L79

@apottere Didn't realize that, nice

How do I return multiple validation errors for more that one field?
Should I create a custom exception that takes an array of messages and then process such an exception in my custom GraphQLErrorHandler?


Fix Date Mutation in checkPreset Function

This pull request addresses a critical issue in the checkPreset function where the original values of range.from and range.to were being unintentionally mutated. Previously, the function directly modified these Date objects by calling setHours(0, 0, 0, 0) on them. This behavior was leading to side effects, as the range object's from and to values were altered globally, affecting other parts of the application relying on these values.
Changes Made:
Modified the checkPreset function to prevent mutation of the original range.from and range.to values.
Implemented creation of new Date objects (normalizedRangeFrom and normalizedRangeTo) based on the existing range.from and range.to before setting their hours to midnight. This ensures that the original range object remains unchanged after the function's execution.
Impact:
The updates ensure that the checkPreset function now performs its intended check without causing unintended side effects in other parts of the application.
Improves the reliability of date comparisons in the function, ensuring that the preset selection logic is more robust and error-free.

Thanks!


chore: add basic issue templates

Description
Adds basic issue templates.
Screenshots:
Template chooser

Issue form

@apledger / @migueloller Can you add the following labels to this repo?

[ ] runtime
[ ] cli
[ ] examples
[ ] catalyst

TIA 🙏

@apledger / @migueloller Can you add the following labels to this repo?

[x]  runtime
[x]  cli
[x]  examples
[x]  catalyst

TIA 🙏

Done!

Thanks @migueloller! I don't have write access, do you mind merging this one for me? 🙂


Cyoa congress

Notes:

there will be hound errors. I satisfied rubocop on every file that I touched. I didn't want to go through and sanitize the worst of the autogenerated files... my apologies.
This is a combination rails + angular app. So it runs as a SPA on the frontend. The cucumber tests are therefore all run in database truncation mode.
run cucumber and rspec the normal ways (just cucumber and rspec suffice on my machine).
includes full omniauth login support.

Good start! The code is clean and easy to follow. Is this still a work in progress? There seems to be a lot of missing functionality (both from the plan document cyoa-plan.txt as well as the original description of this particular CYOA - https://github.com/jmmastey/level_up_exercises/tree/master/opposite_of_progress).
One question I have is what made you go with Angular for this particular app? The actual implementation could just as easily have been made using straight Rails and a little bit of Ajax without imposing the additional dependencies of Angular, Bower, Restangular, etc. Using the right tools for the job is a big part of the design process and I'm not sure Angular is adding anything here.
The other question is why are there no access controls on voting? Users can log in but there is no difference in app functionality if the user is logged in or out. I should be able to do something more if I login to the site.
Finally, make sure you go back and flesh out the testing for the app. Most of the happy path is covered but there's zero sad or error path testing.

hey @jmtai - thanks for the really helpful comments. The story is that I started this, got to a "finishing" point, then switched over to find me a sandwich, the replacement project that dan and blake have been working on. So you're right that there could be more functionality.
I chose angular because 1) I really like it and 2) I wanted more practice integrating angular and rails in sustainable, maintainable ways. This was helpful for figuring that out.
I will go back and follow your suggestions if time permits.


The new codes do not get saved to KeePass2Android

Hi!

I click + to create a new entry,
take a photo of the QR code from the app,
enter the name and click the Save icon
unlock the database with KeePass2Android
perform a search, and
select an entry

at which point nothing really happens. I go back to the Fidelity app and the entry is there and the list is on the same screen (looks like a bug).
If I exit and reopen the app, the new entry is there.
But, if I look into the KeePass database, there is no additional record in the previously selected entry, where the code is stored.
I already have several such records and would really like to make sure that the codes are kept in my KeePass database. I don't always carry the original card or paper and, besides, I would want to know that I have a safe copy.
Is there a way to confirm this?
If the records are not linked, would it be possible to have an option to link a code to an existing entry in KeePass?
Thanks again, I find this an indispensable app. It is already replacing 7-8 cards and printouts of different sizes and significantly reduces time and stress levels before and at the scanners.

I fixed the issue where you saw old screens under new ones.
In relation to the issue concerning "search and edit", unfortunately you need to create a new entry and merge it with an existing one. The keepass2android plugin interface does not seem to support "adding fields to existing entries" only "create new entries".

Thank you for the work that you're doing and the clarifications here.
It seems, however, that some entries were not created in KeePass but are shown in the app. I thought only the entries found in a KeePass database would be displayed.
If I search for "fidelity", I only find two entries in KeePass. But the Fidelity app shows five. Is there any way to re-sync?
Cheers

Since I don't get any information back when the "create task completes" (user left without doing anything , user added entry,...) I by default store them in the app.
The alternative would be that it only stores them when searching/querying so that after adding you would need to search it to be added to the list.
The best approach would be to request on the keepass2android git a way for plugins to edit entries and to return success/failure information


[BUG] Return code is 0 when encounter errors.

E.g.
$ coscli cp big_files.tar.gz cos://bucket/big_files.tar.gz
FinishWithError: Total num: 1, size: 549,304,363,924 Byte (511.58 GB). Error num: 1. OK num: 0, Transfer size: 0 Byte (0.00 B).
Some file upload failed, please check the detailed information in dir /path/to/wd/coscli_output/20240903_063837.

AvgSpeed: 0.00 B/s

cost 0.012000(s)
$ echo $?
0

error.report

Upload big_files.tar.gz to cos://bucket/big_files.tar.gz failed: Too many parts, out of 10000

When an error occurred should return a not 0 exit code. Reference: https://en.wikipedia.org/wiki/Exit_status

Hello, the coscli process returns 0 if no errors are reported. Failure to transfer an individual file does not affect the final result.


Add autoyast support, the SUSE/openSUSE preseed system

I have succesfully installed mr provisioner (using the Ansible recipes) with Autoyast support using only this additional patch that corrects linting issues : https://github.com/BaptisteGerondeau/mr-provisioner/commit/3dc689c2035255945f7c2380ad51a357461873a7

@ggardet thanks for your patch! can you please fix the linting issues as per @BaptisteGerondeau 's recommendation?

After some investigation into the travis fail, I've found the reason. Here is the linting command:
prettier --no-semi --write --trailing-comma es5 --single-quote -l 'src//*.js' 'src//*.jsx'
This will change the files in place.
The version of prettier you use is paramount: you need to use the one from package.json, at the moment 1.5.2. If you use the latest one, 1.15.3, although the command goes through (after having used write), npm run lint will always return an error.
https://github.com/BaptisteGerondeau/mr-provisioner/commit/c6a8a6b19f7ab15de09d661b56746d7af724032c
Furthermore, could you please add a tidbit about this patch in the Changelog ?

Done. All check passes.

Tested in a production environment and everything works !
(note: I did not need the nameserver option since it is provided by kea in the lab's case)


Update desktopwindowxamlsource.md

Remove the restriction around only being able to have one DesktopWindowXamlSource per thread as it is not correct

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

#sign-off


Add new Feature getExternalIp.py

Scripts print out the external IP-Address of the current machine

Thanks for contributing to making this project better.😊I hope you liked the project.


WS method addListener: formatted query string not working

When I hardcode the address, it works
    const query = "tm.event='Tx' AND ethereum_tx.recipient='0xfaC60Ac942b8Ac6a2BC2470D81124C34e8719d88'";

    const stream = this.tendermintWs.listen({
      id: 0,
      jsonrpc: '2.0',
      method: 'subscribe',
      params: { query }
    });

    stream.addListener({
      next: data => this.onEventTendermintData(data),
      error: err => console.error(err),
      complete: () => console.log('completed'),
    });

But not working if the address is formatted in query
   const query = `tm.event='Tx' AND ethereum_tx.recipient='${address}'`;

Did you debug the content of query and address? The code looks correct.

Have you tried double quotes for
const query = "tm.event='Tx' AND ethereum_tx.recipient='${address}'";


View data for more than 2 weeks ago

First of all thank you for the work in the app.
I've been using ackee for the last month and realized that the UI doesn't have a way to see "daily" data more than 2 weeks back. Is it possible that's the case or maybe i've been missing an hidden option in the interface?

I can't confirm myself but, if you go on the insights tab on the top of the page and select the metric you want to know, you will see a small floating bubble fixed on the bottom of the page, there you can select older data. 👍
I do believe that it should have a small arrow like this, would facilitate tho:

Interesting,
I cannot see this arrow on demo as well

oh no, it's not on the demo, i just put there using the elements to show how i think it should be done, adding the svg element that's all what I did, no such thing was implemented but I hope it will, sorry for the mix up. 😅
Just for clarifying:

Then, select any stat you want:

Voilá!

It's indeed not possible right now to see previous weeks. You can only group by day, month or year. Going back in time is a feature that has been planned size the beginning, but it doesn't have a high priority.
@Go-Merk Thanks for the suggestion!
Tip: You can use the API to get more data if you really want to :)
Please follow this issue if you want to stay up-to-date: https://github.com/electerious/Ackee/issues/39


Math formulas

Is there a plan to provide the rendering function for math formulas?
FYI: https://github.com/karser/QDSEquations

Well, if you somehow standardize the formula format in the response of the neural network and set it to wrap the formulas in special quotes, then you can port this component to FMX

Porbabaly we can use LaTex:

Tampermonkey plugin for chrome:
https://chrome.google.com/webstore/detail/tampermonkey/dhdgffkkebhmkfjojejmpbldmpobfkfo

can you throw off the document or links to resources where you can find a similar standard for recording and displaying such formulas?

can you share the document or links to resources where you can find a similar standard for recording and displaying such formulas?

https://en.wikipedia.org/wiki/LaTeX
https://github.com/germandzmitry/TMComplex

For now, I installed the Tampermonkey plugin and the 456049-openai-chatgpt-latex-auto-render-with-mathjax-v2 plugin script for my Chrome browser. That's javascripts things. I have no very clear idea of how to implement it in your project yet.

GPT4 has the ability to read images with math formulas. Maybe we can give it a test later.

I would like to close this issue now.


Error: Cannot read property 'lat' of undefined

Code:
<link rel="stylesheet" href="/js/leaflet/leaflet.css" />
<script src="/js/leaflet/leaflet-src.js"></script>
<script<EMAIL_ADDRESS>
var map = L.map('leafletmap', {
        center: [53.38877, 85.0988],
        zoom: 5,
    });

let key = 'sk.eyJ1Ijoic2VyZ2V5amFjb2JpIifjrut8mNrMDRwZ2IyeTJxdTUzZ3E5OGl6dXVvNWgifQ.h6YnuwoH9qFHjrEY5EFsBg';

var layerSatellite = L.tileLayer('https://server.arcgisonline.com/ArcGIS/rest/services/World_Imagery/MapServer/tile/{z}/{y}/{x}?blankTile=false', {
        maxZoom: 10,
        id: 'mapbox.satellite',
    }).addTo(map);

var url = 'https://api.mapbox.com/v4/sergeyjwee.3j31gq5r/' + '{z}/{x}/{y}.vector.pbf?access_token=' + key;

L.vectorGrid.protobuf(url, {
        // rendererFactory: L.canvas.tile,

        vectorTileLayerStyles: {
            'firedata-7i4kge': {
                weight: 1,
                color: '#ff0000',
                fillOpacity: 1,
                fill: true,
            }
        },

        subdomains: "abcd",
        interactive: true,	// Make sure that this VectorGrid fires mouse/pointer events
        getFeatureId: function(f) {
            console.log(f);
            return f.properties.id;
        }
"pk.eyJ1Ijoic2VyZ2V5amFjb2JpIiwiYSI6ImjFc4iDBrc2pmNXpxdWEifQ.ylwO6LsTScejbc-SlrDk2w"
    }).on('click', function(e) {

        L.popup()
            .setContent('проверка')
            .setLatLng(e.latlng)
            .openOn(map);

    }).addTo(map);

Error in chrome:
Projection.SphericalMercator.js:24

[Uncaught TypeError: Cannot read property 'lat' of undefined
    at Object.project (Projection.SphericalMercator.js:24)
    at Object.latLngToPoint (CRS.js:28)
    at NewClass.project (Map.js:992)
    at NewClass.latLngToLayerPoint (Map.js:1014)
    at NewClass.latLngToContainerPoint (Map.js:1071)
    at NewClass._fireDOMEvent (Map.js:1433)
    at NewClass._handleDOMEvent (Map.js:1396)
    at HTMLDivElement.handler (DomEvent.js:79)](url)

I got no bind onlick action. what to do with it?

what to do with it?

Remove all unnecessary parts from your code.
Leave only minimum to reproduce core issue.
Setup some sandbox (like jsfiddle) to show the issue.

There's a reference to L.VectorGrid there - make sure whether the apparent bug is in core leaflet or in VectorGrid.

I'm getting this error Cannot read property 'lat' on adding point mvt layer as an overlay. Polygon and polyline are working fine

Got the same issue here with pbf coming from geoserver vector tiles. My layer consists of land parcels polygon, electricity network line, and distribution transformer poles as points. The polygon and line composition is working fine. Once i add point layer, the browser logs Cannot read property 'lat' of undefined.
Probing clickedLayer value from the scope of Leaflet.VectorGrid without firing Leaflet.GridLayer events: this._fireEvent([clickedLayer], e); and this._handleMouseHover(e, point); show that clickedLayer has correct properties and all chrome errors are gone.
	_onClick: function (e) {
		var point = this._map.mouseEventToLayerPoint(e).subtract(this.getOffset()), layer, clickedLayer;

		for (var id in this._layers) {
			layer = this._layers[id];
			if (layer.options.interactive && layer._containsPoint(point) && !this._map._draggableMoved(layer)) {
				clickedLayer = layer;
			}
		}
		if (clickedLayer)  {
			L.DomEvent.fakeStop(e);
			console.log(clickedLayer);
			//this._fireEvent([clickedLayer], e);
		}
	},

	_onMouseMove: function (e) {

		if (!this._map || this._map.dragging.moving() || this._map._animatingZoom) { return; }
		var point = this._map.mouseEventToLayerPoint(e).subtract(this.getOffset());
		//this._handleMouseHover(e, point);
	},

Also please take a look at this 236 issue. I found weird number from Leaflet.VectorGrid getOffset() at fractional zoom. Looks like the number has been incorrectly multiplied by the scaling.

Well, I just found the solution. This code works for me (See original answer). Hope it helps others.
_createLayer: function(feat, pxPerExtent, layerStyle) {
    var layer;
    switch (feat.type) {
    case 1:
        layer = new PointSymbolizer(feat, pxPerExtent);
        // [YB 2019-10-23: prevent leaflet from treating these canvas points as real markers]
        layer.getLatLng = null;
        break;
    case 2:
        layer = new LineSymbolizer(feat, pxPerExtent);
        break;
    case 3:
        layer = new FillSymbolizer(feat, pxPerExtent);
        break;
    }

    if (this.options.interactive) {
        layer.addEventParent(this);
    }

    return layer;
}

Please forward that to the Leaflet.Vectorgrid repo, propose a PR.

Well, I just found the solution. This code works for me (See original answer). Hope it helps others.
_createLayer: function(feat, pxPerExtent, layerStyle) {
    var layer;
    switch (feat.type) {
    case 1:
        layer = new PointSymbolizer(feat, pxPerExtent);
        // [YB 2019-10-23: prevent leaflet from treating these canvas points as real markers]
        layer.getLatLng = null;
        break;
    case 2:
        layer = new LineSymbolizer(feat, pxPerExtent);
        break;
    case 3:
        layer = new FillSymbolizer(feat, pxPerExtent);
        break;
    }

    if (this.options.interactive) {
        layer.addEventParent(this);
    }

    return layer;
}

where did you put this? which source code and on what line?

Ensure that the data the script is consuming exists ie do null-checks on all coordinate pairs and other necessary data

This does not seem to be fixed yet. Any ETA on the fix?
Adding the following simple check to src/map/Map.js works:
const isMarker = target.getLatLng && **target._latlng** && (!target._radius || target._radius <= 10);


need help: how can i change the default font

like the title said, i want to change the base font for my select title to keep consistancy, can i do that ? qnd how ? thank you

Use CSS. https://css-tricks.com/almanac/properties/f/font-family/


Error: No named parameter with the name 'timeInSecForIos'.  for building Android application

lib/main.dart:111:17: Error: No named parameter with the name 'timeInSecForIos'.
timeInSecForIos: 1,
^^^^^^^^^^^^^^^
../../../flutter_sdk/flutter/.pub-cache/hosted/pub.flutter-io.cn/fluttertoast-7.0.2/lib/fluttertoast.dart:18:23: Context: Found this candidate, but the arguments don't match.
static Future showToast(
^^^^^^^^^
FAILURE: Build failed with an exception.

change timeInSecForIos to  timeInSecForIosWeb


Provide option to prevent unorphan when last & 2nd to last words are long

Currently working on a site with headlines that end with stuff like "immunotherapy treatment".  Because both words are relatively long, they're overflowing the container, especially on mobile.
I've used jQuery widowFix in the past, which has an option to no-op if the words are longer than X. However, widowFix does not handle elements with children elements reliably, so not a good option.
Any chance of adding a similar option to unorphan?

How do you think this should work? unorphan('.selector', { maxLength: 9 }) ? PR's welcome!

unorphan('.selector', { prevLastWordLimit: 8, lastWordLimit: 9 }) would be great.
Maybe on line 44, something like this
var words = n.nodeValue.split(' ')
if ( (words.pop().length <= lastWordLimit) && (words.pop().length <= prevLastWordLimit) ) {
  n.nodeValue = text.replace(/\s+([^\s]+)\s*$/, nbsp + '$1')
}

and I guess you'd need to test to see if the word limit options were set or maybe set them to something gratuitously high, like 100?

I think it's better to just do maxCombinedLength: 18 to check if the last two words will have a short-enough length (where 18 is same as 8 + 9 in your example, with 1 space).

Sounds good, forked & seeing if works...

Hey guys, did this end up working?


[Console-IoT] Add meaningful id to address detail module components

Type of change

Refactoring

Description

Adding meaningful id to components for address detail module
Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

Can one of the admins verify this patch?

Resolved all comments. Looks good to merge


Mod: Applied Energistics 2 + Addons

This is a collection issue for AE2 problems.

[x] Cables don't connect to blocks
[x] Controller, CPUs and others don't "glow"
[ ] Doesn't differentiate between different terminal types, buses, interfaces or similar. All are currently rendered as some dummy terminal, including those structural supports.

This is a collection issue for AE2 problems.

[x]  Cables don't connect to blocks
[x]  Controller, CPUs and others don't "glow"
[ ]  Doesn't differentiate between different terminal types, buses, interfaces or similar. All are currently rendered as some dummy terminal, including those structural supports.

for me, the certus quartz ore is also broken

Ahh, I guess I haven't seed that as GTNH uses its own ores for everything.


Sendmail

https://sendgrid.com/docs/Integrate/Mail_Servers/sendmail.html
Setting up sendmail on centos6 was exactly as described per your docs.
However, authentication errors pursued, which prevented connections until I installed cyrus-sasl-plain.
yum install cyrus-sasl-plain
errors in /var/log/maillog looked like this:
STARTTLS=client, relay=smtp.sendgrid.net., version=TLSv1/SSLv3, verify=FAIL, cipher=AES128-GCM-SHA256, bits=128/128
AUTH=client, available mechanisms do not fulfill requirements
AUTH=client, relay=smtp.sendgrid.net., temporary failure, connection abort

I suggest adding this check as a suggestion for folks.

I have added a note here https://github.com/sendgrid/docs/pull/2329
Thank you!


Multi-Sheet Characters (Experimental Branch)

This is my pull request #12517, but on the actual correct branch this time.
Copy-pasted from the previous iteration:
This adds a new field to the character editor that lets you reference other sprite sheets for certain animations. One of the things this is useful for is if your character is too big (or just has fully animated voicelines) and is forced to use multiple sprite sheets (unless you downgraded to Adobe Animate 2018 just for texture atlases for some reason).


Map Pin Points from "leaflet-map" Module Not Loading Properly on Android Mobile App

Describe the bug
In the Android Mobile App, the map pin points from the "leaflet-map" module are failing to load as expected. Instead of showing the correct icon, a default missing picture icon is displayed.
To Reproduce
Steps to reproduce the behavior:

Install the "Location display and search" Pack.
Populate [EnglandLibraries] table with data
Use the Mobile App Builder to make a native Android App
Install/Open App on mobile device
See error

Expected behavior
The expected behavior is that the map pin points from the "leaflet-map" module should load properly, displaying the correct icons instead of a missing picture icon.
Screenshots
Attached to this report is a screenshot illustrating the issue, where the map pin points are shown as missing picture icons.
Additional context
This issue appears to have surfaced sometime after the Saltcorn 0.8.6-beta.2 version. I have updated to the latest version and ensured all modules are updated as well, yet the problem persists. This significantly impacts the user experience and functionality of the map features in the Android Mobile App. I would greatly appreciate it if you could investigate and resolve this issue. Thank you.

Can you try it again with v0.8.7-beta.1

@christianhugoch Yes I can confirm, as of v0.8.7-beta.1 map pin point icons are working again!!!
Thank you for looking into this and fixing it.


sitemap addition

addition of sitemap for SEO

@SahityaRoy

I have already worked on this type of issue that I added a sitemap section in a footer I can do this work better, Assign me this issue under GGSSoC'23.


Wildcard upload requires confirmation

Conan will ask for confirmation (yes/no) if the upload reference is a pattern.
The packager should probably pass --confirm to conan upload since it's purpose is mostly for CI/automation. Without --confirm my CI job creates an infinite large log when it hangs at:
Are you sure you want to upload '...'? (yes/no): ERROR:  is not a valid answer
Are you sure you want to upload '...'? (yes/no): ERROR:  is not a valid answer
...

Fixed in 0.4.31


Seed for mutant generation

Is your feature request related to a problem? Please describe.
I'd like to add the Stryker to our team CI pipeline, but I'm not doing it, because the Stryker generates the mutators in some random way for each run, so the results are not consistent. Sometimes the tested code passes, sometimes doesn't.
Describe the solution you'd like
I'd like to be able to add some "seed" to the configuration file, which would ensure all runs (on the same piece of code) will generate the same mutants consistently.
Describe alternatives you've considered
Additional context
I'm aware that this will cause there can be a potential bug in the tested code which will not be detected because all runs will have the same mutants, so some mutants regularly wouldn't be generated. But many of them would be and much of the value would be added to the quality of the tests.

@VaclavObornik its most likely a bug. Mutation tests should be consistent unless you add some random things. It is (i think) caused because of the number of resources that are used and some memory leaking. I believe most of it will be solved by Nico in mutation switching tho.

Hi @VaclavObornik , as @kmdrGroch stated, Stryker is not doing anything random. Can you explain what appears to be random in the results?


Feature Request - which_result in gdf_from_places

Hey Guys!
I need to query a lot of cities, so I'm using the gdf_from_places function, but as you could expect I don't always want the first returned result, so it would be needed to add the which_result (available in gdf_from_place) to the gdf_from_places function as well.
I have implemented this feature locally, just checking to see if this is something it would be wanted to be implemented upstream.
If so, there is a project decision to be made:
Do we restrict the same which_result for all places (this way we just need to add an optional parameter in the function), or we use a specific parameter for each of the places (which is what I used in my case) that must be passed together in the dictionary, or maybe another idea?

A good option would be to add which_results as an optional argument as a list with the same length as the queries argument, then zip them in gdf_from_places's for loop to pass them both into the gdf_from_place call one at a time.

That’s a great idea. I can make a PR today for it. Is that good for you?

Sure, thanks.

Pull Request addressing this feature opened at #301


Pull changes from upstream

Coverage remained the same at 91.607% when pulling ac1fdc69f76776fd48cb1f8e7b79f6df0d548f15 on jjafuller:master into 26eb4b233668fd7aa5c405623d787f6dea05ccc7 on moov-io:master.

I meant for this to go in the other direction. Sorry about the clutter.


Python error when exiting idf_monitor (IDFGH-10965)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Describe: v5.1.1-1-gd3c99ed3b8
Pressing Ctrl + ] to close idf_monitor results in:
Exception in thread Thread-2 (_run_outer):
Traceback (most recent call last):
  File "/usr/lib/python3.11/threading.py", line 1038, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.11/threading.py", line 975, in run
    self._target(*self._args, **self._kwargs)
  File "/tools/python_env/idf5.1_py3.11_env/lib/python3.11/site-packages/esp_idf_monitor/base/stoppable_thread.py", line 47, in _run_outer
    self.run()
  File "/tools/python_env/idf5.1_py3.11_env/lib/python3.11/site-packages/esp_idf_monitor/base/serial_reader.py", line 100, in run
    self.close_serial()
  File "/tools/python_env/idf5.1_py3.11_env/lib/python3.11/site-packages/esp_idf_monitor/base/serial_reader.py", line 105, in close_serial
    self.set_closing_wait(delay_sec=1)
  File "/tools/python_env/idf5.1_py3.11_env/lib/python3.11/site-packages/esp_idf_monitor/base/serial_reader.py", line 128, in set_closing_wait
    fcntl.ioctl(self.serial.fd, termios.TIOCSSERIAL, buf)
PermissionError: [Errno 1] Operation not permitted

Done

Hi, I am sorry for the issue. We are aware of it and we currently have a PR under review that implements the fix.
Possible workarounds are:

Downgrading esp-idf-monitor to a version lower than 1.1.3 - pip install esp-idf-monitor==1.1.2
Running monitor as a root - I would recommend doing this just in case  you really need a newer version that fixes issues on new kernels (e.g. waiting for 30 seconds before the serial port gets closed)

Just make sure the fix will be available in release/v5.1.

No worries, it will be definitely available in ESP-IDF 5.1. As the monitor is now a separate package we can release it without needing to wait for ESP-IDF. I will let you know once the new version with the fix is available.

This can be resolved by running
pip install "esp-idf-monitor>=1.2.0"

in active ESP-IDF environments.
Or just re-run the install and export scripts again.

Hi there,
Since I had a similar issue on version 5.1.2, can the issue be resolved by running the lower version like pip install esp-idf-monitor==1.1.2? Has this issue been fully solved in version >=5.1?

Hi @4444monkey this issue should be resolved completely in the latest versions of esp-idf-monitor (included in IDF 5.1). You can make sure that you have the latest version by running install and export scripts. If you still see the issue, please provide more details and open a new issue in esp-idf-monitor repository and we will take a look. Installing the mentioned lower version was just a workaround and should not be needed anymore.


@required annotation in data class constructor

I like using data classes instead of companions for my insert functions to help denote which fields are required, but it looks like nullability is the only determining factor when deciding whether to generate a @required annotation. Would it be possible to also check if the column is rowid or alias (#112), AUTOINCREMENT or has a default value, similar to the logic in GeneratedColumn.required?

I feel like the best approach here is to make companions easier to use. Data classes are designed to hold a full row with all data, which is the reason we only omit the required annotation on nullable columns.
I see how data classes provide better hints on which fields are required, but the concept of using null to indicate absent values will only get worse in future Dart versions that have NNBD. By then, we'd have to explicitly make columns that are optional for inserts nullable, which would give the wrong nullability for select statements.

Would it be possible to also check if the column is rowid or alias (#112), AUTOINCREMENT or has a default value, similar to the logic in GeneratedColumn.required?

Better guidance on which fields are required during insert is the only use case for this, right? How about an additional constructor on the companion class (say MyTableCompanion.insert) where values which need an explicit value during inserts are marked @required? We also wouldn't have to use Value-wrappers in that case:
class Things extends Table {
  IntColumn get id => integer().autoIncrement()();
  IntColumn get foo => integer().withDefault(const Constant(42))();
  TextColumn get bar => text().nullable()();
  TextColumn get baz => text();
}

// generated constructor can now have this signature:
ThingsCompanion.insert({
  @required String baz,
  @required this.id = const Value.absent(),
  // all other values would be formal initializers to Value.absent() as well
}): this.baz = Value(baz);

Yeah good point, the signature of a row can be different for reading vs writing so it definitely makes sense to keep that separate with data classes and companions.

Better guidance on which fields are required during insert is the only use case for this, right?

Yeah exactly - I'd just like some way to statically determine what the minimum required fields are to not violate any constraints. The Companion.insert constructor could work, although in your example I'd expect it to look something like:
ThingsCompanion.insert({
  this.id = const Value.absent(),
  this.foo = const Value.absent(), // maybe even Value(42)? Probably best to let the database do this though
  this.bar = const Value.absent(),
  @required String baz,
}): this.baz = Value(baz);

For me, the main downside of this approach is that it's still the same ThingCompanion type, so you wouldn't be able to take advantage of it when writing something like:
 Future<int> insertThing(ThingCompanion insert) {
    // hopefully the caller used ThingCompanion.insert()!
  }

 Future<int> updateThing(int thingId, ThingCompanion updates) {
    // ...
  }

For me, the main downside of this approach is that it's still the same ThingCompanion type, so you wouldn't be able to take advantage of it [...]

It's true that we can't verify this statically, but we would fail early and provide help by throwing an InvalidDataException, so users are very likely to catch this during development. To get proper static warnings, we'd have to introduce another type just for inserts. IMHO the additional generated code is not worth the benefit here. For almost all cases, it's clear which methods perform inserts vs updates, so it's intuitive to just use the insert constructor when calling these methods.

I think generating an additional insert constructor is fine, it's fairly straightforward to generate and it won't be a lot of code. For people who really want less generated code, I think we can make some less-used features (like the builtin json serializazion or overridden equals/hashCode/toString in data classes) opt-out instead.

I'd probably write a custom insert statement and use the sqlparser to generate a method with exact arguments to achieve more strict inserts

Speaking of that, we can't parse insert statements yet. I hope I get to do that soon, they could really help verify that all required columns are present.

We can now parse insert statements and generate appropriate implementations. We also generate a named "insert" constructor for each companion.


Alignment of boons and advantages doesnt quit ematch in the centre[BUG]

SO looks like the heading are in the centre but the dots are left justified on the 2nd half of the screen

disciplines is messy as well seems to be something to do with flexrows, may have to change to columns to have some control over width


Minecraft launcher crashes while running malmo parallel in docker container

When I try to run multiple agents in multiple Minecraft environment with python threading in docker container, the Minecraft launchers are crashed and exited. The same code has no problem in normal desktop system(non-container) and Python_Examples(no threading) in Malmo are working in docker environment. Do you have any idea why it happens?
It seems threading makes problem...
I installed everything from docker ubuntu:14.04 container (anaconda2, tensorflow, malmo)
and I run the docker as follows to use GUI,
xhost + ; docker run -it -p 8888:8888 -p 6006:6006 -p 8000:8000 -v `pwd`:/home/sharedfolder --privileged --device=/dev/snd:/dev/snd -v /tmp/.X11-unix:/tmp/.X11-unix:ro -e DISPLAY=unix$DISPLAY <container name>

I got following error messages from Minecraft launcher,
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGFPE (0x8) at pc=0x00007f2a8b48ce88, pid=3444, tid=0x00007f2ac7528700
#
# JRE version: Java(TM) SE Runtime Environment (8.0_131-b11) (build 1.8.0_131-b11)
# Java VM: Java HotSpot(TM) 64-Bit Server VM (25.131-b11 mixed mode linux-amd64 compressed oops)
# Problematic frame:
# C  [swrast_dri.so+0x272e88]
#
# Failed to write core dump. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# An error report file with more information is saved as:
# /root/Malmo-0.21.0/Minecraft1/run/hs_err_pid3444.log
#
# If you would like to submit a bug report, please visit:
#   http://bugreport.java.com/bugreport/crash.jsp
# The crash happened outside the Java Virtual Machine in native code.
# See problematic frame for where to report the bug.
#
                                                                                :runClient FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':runClient'.
> Process 'command '/usr/lib/jvm/java-8-oracle/bin/java'' finished with non-zero exit value 134

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Total time: 36.853 secs

Possibly out of memory?


Send Document - Payload structure - Attachments

http://gpconnect-messaging.netlify.com/senddocument_payload.html
It is not clear from the diagram how additional attachments (e.g. ECG) are recorded in the message structure.

Details to be added when addressing #40

Changes made with #40


add subdomain to password reset url

I rule and have completed some work on Case Manager that's ready for review!
minor bug when testing password reset in new env, with an easy fix - add subdomain arg to reset url
This pull request makes the following changes:

add subdomain arg to password reset url

no view changes, )though the password reset screen is ugmo)
It relates to the following issue #s:

Bumps #1817

For reviewer:

Adjust the title to explain what it does for the notification email to the listserv.
Tag this PR:

feature if it contains a feature, fix, or similar. This is anything that contains a user-facing fix in some way, such as frontend changes, alterations to backend behavior, or bug fixes.
dependencies if it contains library upgrades or similar. This is anything that upgrades any dependency, such as a Gemfile update or npm package upgrade.

If it contains neither, no need to tag this PR.

yolo merge


SqlException Transaction (Process ID 66) was deadlocked on lock resources with another process and has been chosen as the deadlock victim. Rerun the transaction.

Hi all
We are using durable functions in a stand alone process (not azure functions) in kubernetes cluster.
Microsoft.DurableTask.SqlServer.AzureFunctions nuget version 1.1.1
Sometimes when calling ReadEntityStateAsync we get an exception:
Transaction (Process ID 63) was deadlocked on lock resources with another process and has been chosen as the deadlock victim. Rerun the transaction.
It happens once every few thousands of orchestrations.
The full call stack:
Microsoft.VideoIndexer.Telemetry.ViException: Transaction (Process ID 63) was deadlocked on lock resources with another process and has been chosen as the deadlock victim. Rerun the transaction.
 ---> Microsoft.Data.SqlClient.SqlException (0x80131904): Transaction (Process ID 63) was deadlocked on lock resources with another process and has been chosen as the deadlock victim. Rerun the transaction.
   at Microsoft.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at Microsoft.Data.SqlClient.SqlInternalConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at Microsoft.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
   at Microsoft.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
   at Microsoft.Data.SqlClient.SqlDataReader.TryHasMoreRows(Boolean& moreRows)
   at Microsoft.Data.SqlClient.SqlDataReader.TryReadInternal(Boolean setTimeout, Boolean& more)
   at Microsoft.Data.SqlClient.SqlDataReader.Read()
   at DurableTask.SqlServer.SqlOrchestrationService.GetOrchestrationStateAsync(String instanceId, String executionId, CancellationToken cancellationToken) in /_/src/DurableTask.SqlServer/SqlOrchestrationService.cs:line 614
   at DurableTask.SqlServer.AzureFunctions.SqlDurabilityProvider.RetrieveSerializedEntityState(EntityId entityId, JsonSerializerSettings serializierSettings) in /_/src/DurableTask.SqlServer.AzureFunctions/SqlDurabilityProvider.cs:line 103
   at Microsoft.Azure.WebJobs.Extensions.DurableTask.DurableClient.ReadEntityStateAsync[T](DurabilityProvider provider, EntityId entityId) in D:\a\_work\1\s\src\WebJobs.Extensions.DurableTask\ContextImplementations\DurableClient.cs:line 566
   at Microsoft.VideoIndexer.IndexingService.Common.ActivityRunner.RunActivity[TContext](Func`3 function, TContext context, IDurableEntityClient entityClient, Boolean throwExceptions, String callingMethod) in C:\__w\1\s\Src\Microsoft.VideoIndexer.IndexingService.Common\ActivityRunner.cs:line 121
ClientConnectionId:dba86c5b-4d3b-4f29-89b9-d169ac0d9362
Error Number:1205,State:51,Class:13
   --- End of inner exception stack trace ---

Kind ping
@cgillum

We should be automatically retrying deadlocks, so I wonder if those retries are being exhausted. Unfortunately, it's hard to fix these without having a reliable repro that can be used for testing.
Are you comfortable making changes directly to your database? One thing you can try is changing the deadlock priority of the dt.QuerySingleOrchestration stored procedure, adding a SET DEADLOCK_PRIORITY HIGH to the top of the procedure to make it more likely to survive deadlocks.

It doesn't fix the deadlock, but it makes it less likely to impact the ReadEntityStateAsync. Instead, the other deadlock participant will get rolled back. Assuming the other deadlock participant is less important or is a background task that can be automatically retried, this could improve the overall availability of your app. However, it could also just move the problem elsewhere.
If you're able to find a way to reproduce this consistently, I could use that reproducer to find a more reliable/permanent fix.

Hi @cgillum
I manually applied your change to the stored procedure and it seems to help. After the fix, I wasn't able to reproduce the error.
Is there a way to apply it programmatically when the process starts?
Currently, we are letting the SDK create the DB with the flag task.StorageProvider["createDatabaseIfNotExists"] = true;


No pip executable for Python 3.11

Describe the bug
When switching to Python 3.11 with rtx use -g<EMAIL_ADDRESS>or rtx use -g python@latest, there is no pip executable set up for it.
To Reproduce

rtx use -g<EMAIL_ADDRESS>type pip

Expected behavior
The output from type pip should point to a pip executable in ~/.local/share/rtx/installs/python/3.11.3/bin like it does for Python 2.x and 3.10.x.
rtx doctor output
rtx version:
  1.29.7 linux-x64 (unknown 2023-05-19)

build:
  Target: x86_64-unknown-linux-gnu
  Features:
  Built: Fri, 19 May 2023 20:51:50 +0000
  Rust Version: rustc 1.69.0 (84c898d65 2023-04-16) (Arch Linux rust 1:1.69.0-3)
  Profile: release

shell:
  /bin/zsh
  zsh 5.9 (x86_64-pc-linux-gnu)

rtx environment variables:
  RTX_SHELL=zsh

settings:
  {"experimental": "true", "missing_runtime_behavior": "warn", "always_keep_download": "false", "always_keep_install": "false", "legacy_version_file": "true", "plugin_autoupdate_last_check_duration": "10080", "trusted_config_paths": "[]", "verbose": "false", "asdf_compat": "false", "jobs": "4", "disable_default_shorthands": "false", "log_level": "INFO", "raw": "false"}

config files:
  /home/sharparam/.config/rtx/config.toml

plugins:
  lua      https://github.com/Stratus3D/asdf-lua.git#6d4e66a
  node     (core)
  python   (core)
  ruby     https://github.com/rtx-plugins/rtx-ruby.git#ff2640f

toolset:
  node@lts, python@latest python@2, ruby@latest, lua@latest <EMAIL_ADDRESS>
No problems found

Additional context
When using rtx with just the latest Python, it ends up working anyway because it falls back to the system pip install (if there is one that is) which seems to locate the proper paths.
However, if setting multiple Python versions, such as:
rtx use -g python@latest python@2

So that one can run Python 2 stuff with python2, plain pip will use the Python 2 install of pip instead of Python 3.11.
All of this is with the "core" built-in Python in rtx.
As a workaround, one can invoke pip with python -m pip instead, but it's nicer to be able to just run plain pip.

it's working for me:
rtx  main ❯ rtx use -g<EMAIL_ADDRESS>Downloading Python-3.11.3.tar.xz...
-> https://www.python.org/ftp/python/3.11.3/Python-3.11.3.tar.xz
Installing Python-3.11.3...
Installed Python-3.11.3 to /Users/jdx/.rtx/installs/python/3.11.3
[notice] A new release of pip available: 22.3.1 -> 23.1.2
[notice] To update, run: /Users/jdx/.rtx/installs/python/3.11.3/bin/python3.11 -m pip install --upgrade pip
rtx<EMAIL_ADDRESS>Successfully installed MarkupSafe-2.1.2 PyYAML-6.0 ansible-7.5.0 a ✓ 2m
rtx  main ❯ ls ~/.rtx/installs/python/3/bin/
2to3@               ansible-galaxy*     idle3.11*           python-config@
2to3-3.11*          ansible-inventory*  pip@                python3@
ansible*            ansible-playbook*   pip3*               python3-config@
ansible-community*  ansible-pull*       pip3.11*            python3.11*
ansible-config*     ansible-test*       pydoc@              python3.11-config*
ansible-connection* ansible-vault*      pydoc3@             python3.11-gdb.py*
ansible-console*    idle@               pydoc3.11*
ansible-doc*        idle3@              python@
rtx  main [!] ❯ rtx x -- which python
/Users/jdx/.rtx/installs/python/3.11.3/bin/python
rtx  main [!] ❯ rtx x -- which pip
/Users/jdx/.rtx/installs/python/3.11.3/bin/pip
rtx  main [!] ❯ rtx x -- pip --version
pip 22.3.1 from /Users/jdx/.rtx/installs/python/3.11.3/lib/python3.11/site-packages/pip (python 3.11)

what other bins do you see?

Running the same command sequence as you (with install path adjusted):
~
❯ rtx use -g<EMAIL_ADDRESS>
~
❯ ls ~/.local/share/rtx/installs/python/3/bin
'Tipz:' sl ~/.local/share/rtx/installs/python/3/bin
2to3       idle   idle3.11  pydoc3     python   python3.11         python3.11-gdb.py  python-config
2to3-3.11  idle3  pydoc     pydoc3.11  python3  python3.11-config  python3-config

~
❯ rtx x -- which python
/home/sharparam/.local/share/rtx/installs/python/3.11.3/bin/python

~
❯ rtx x -- which pip
/usr/bin/pip

~
❯ rtx x -- pip --version
pip 23.1.2 from /home/sharparam/.local/lib/python3.11/site-packages/pip (python 3.11)

If it matters, I'm using rtx from the AUR. (rtx version outputs 1.29.7 linux-x64 (unknown 2023-05-19))

Update: I must've had some stuff leftover from my pyenv setup.
After deleting all python folders in ~/.local/lib and then doing a clean install of Python 3.11 with rtx, the pip scripts installed as expected.
So it was just leftover garbage on my end, sorry ^^


Benefit Accounts

Hello,
Can it be possible to include a section that states that all the benefit accounts, such as Azure credits, Office 365 Developer and VSTS, are one time benefits that cannot be recovered after activated? So, for example if a VS subscription is assigned to user  "A" via Volume Licensing, and this subscription is reassigned to user "B". In the event that the "A" user activated all the benefit accounts, user "B" will not have those benefits available once he accesses his subscription?

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

ID: 58c83add-3638-8dd7-6f4a-3537aa6adff6
Version Independent ID: fb37fe41-88ed-e7e1-f701-456625e81adb
Content: Visual Studio Subscriptions - Subscriber Benefits
Content Source: subscriptions/subscriber-benefits.md
Product: vs-subscription
GitHub Login: @evanwindom
Microsoft Alias: jaunger

@Nathan-Tarin thanks for your feedback! @evanwindom FYI


Add script for converting plain USDs into OmniGibson object format

This would help things like #991.

Hello,
Do you have provided  a script that rearranges an arbitrary object into the appropriate hierarchy?
For example, usd in https://github.com/clvrai/furniture-bench/tree/main/furniture_bench/assets/furniture/mesh/square_table
I have tried using the method in #991  but still no success.
I hope to get your support and help. Thanks again.

@johnbhlm @cgokmen Any updates about the issue?

We are still actively working on this. Currently no ETA but we should be able to ship it this year. Will keep you updated.

@cremebrule fyi

This script would be very helpful. Is there any progress? @cgokmen

Hi @jinkun-hao ,
Apologies for the delay! We've integrated this feature into our og-develop branch, and is now avaiable as the import_custom_object.py example script!


how to limit the maximum input length?

You could check the size of the contents in onChange if the ContentState has changed, and slice off the contents that go beyond the limit.

to solve the problem, as follows:
onChange(editorState) {
    const contentState = editorState.getCurrentContent();
    if (contentState.getPlainText().length > this.props.maxLength) {
        editorState = EditorState.undo(editorState);
    }
    this.setState({editorState});
}

Well, I don't know if I would recommend using undo for this, since it may jump you back to a previous boundary state that you don't want.
Ideally, you'd be able to do:
onChange(editorState) {
  const contentState = editorState.getCurrentContent();
  const oldContent = this.state.editorState.getCurrentContent();
  const {maxLength} = this.props;
  if (contentState === oldContent || contentState.getPlainText().length <= maxLength) {
      this.setState({editorState});
  }
}

But I think beforeInput native rendering might cause trouble for this, so for now, to be really safe, you'd also want to implement handleBeforeInput:
handleBeforeInput(chars) {
  const totalLength = this.state.editorState.getCurrentContent().getPlainText().length + chars;
  return totalLength > this.props.maxLength;
}

By returning true, we prevent the beforeInput event from rendering the new character or from executing onChange.

chars type is String chars.length  ?

That's correct, thanks. Fixed my comment.

handleBeforeInput(chars)
when chars will be greater than an length?

Sorry, I don't understand the question.

@AnasRezk to prevent focus move to end, use following solution. This also doesn't jump as mentioned from @hellendag
onChange = editorState => {
        const contentState = editorState.getCurrentContent();
        const oldContent = this.state.editorState.getCurrentContent();
        if(contentState === oldContent || contentState.getPlainText().length <= 25) {
            this.setState({ editorState });
        } else {
            const editorState = EditorState.undo(
                EditorState.push(
                    this.state.editorState,
                    ContentState.createFromText(oldContent.getPlainText()),
                    'delete-character'
                )
            );
            this.setState({ editorState });
        }
    }

here some typescript solution:
getStateWithMaxLengthEnsured(oldState: Draft.EditorState, newState: Draft.EditorState, maxLength: number): Draft.EditorState {
        const contentState = newState.getCurrentContent();
        const oldContent = oldState.getCurrentContent();
        if (contentState === oldContent || contentState.getPlainText().length <= maxLength) {
            return newState;
        } else {
            return Draft.EditorState.undo(
                Draft.EditorState.push(
                    oldState,
                    Draft.ContentState.createFromText(oldContent.getPlainText()),
                    'delete-character'
                )
            );
        }
    }

When pasting text:
function handlePastedText(text: string, _: string, state: EditorState): DraftHandleValue {
    const overflowChars = text.length + state.getCurrentContent().getPlainText().length - MAX_LENGTH;

    if (overflowChars > 0) {
      if (text.length - overflowChars > 0) {
        const newContent = Modifier.insertText(
          state.getCurrentContent(),
          state.getSelection(),
          text.substring(0, text.length - overflowChars)
        );
        setEditorState(EditorState.push(state, newContent, 'insert-characters'));
      }
      return 'handled';
    } else {
      return 'not-handled';
    }
  }

Emm, I found I cannot intercept the onCompositionEnd event,
el?.addEventListener("compositionend", handler);

e.stopPropagation()
e.preventDefault()
//...
setEditorState(newEditorState)

the code above is useless.Becase the source code still invoke the onCompositionEnd event after I invoke the custom event,so what can I do to intercept the onCompositionEnd like handleBeforeInput returned either handled or not-handled.
like the code below:
 handleBeforeInput={(chars) => {
          const totalLength = chars.length +editorState.getCurrentContent().getPlainText().length;
          const flag = totalLength > max ? "handled" : "not-handled"
          return flag
}}
```


Adding Fibonacci Search in C++

🚀 Feature
Adding Fibonacci Search under Search Section in C++.
Have you read the Contributing Guidelines on Pull Requests?
YES
Motivation
I want to contribute to this feature as a part of  GSSoC 2021.
Pitch
This feature will add one more Searching algorithm to the project which is for sorted arrays.

If want to work on this issue.

@maneprajakta Go ahead with C++.

@vgb-codes  Sure


Colors are very very different

Hi, could you elaborate a bit more? As vscode themes are pretty irregular, most of the themes lack some of the highlighting groups, djanho will try to fallback those missing groups to other existing groups, but some themes might have so few defined color tokens that there is a difference. Also some themes have a string of groups separated by commas at their tokenColors fields, djanho currently doesn't handle those correctly.


Docker compose failing when building

This seems like a nice project but there are some issues with it.
After cloning the project: git clone https://github.com/artiz/beobachter.git and going to the project directory, I tried running:  docker-compose -f docker-compose.yml up --build.
These are the issues that I can see:
Backend:
beo_backend            | bash: app/run.sh: No such file or directory
beo_backend exited with code 127

Nginx:
fast_treat-nginx-1     | 2023/01/11 14:40:15 [emerg] 1#1: host not found in upstream "backend:8888" in /etc/nginx/conf.d/default.conf:11
fast_treat-nginx-1     | nginx: [emerg] host not found in upstream "backend:8888" in /etc/nginx/conf.d/default.conf:11

Celery worker:
fast_treat-worker-1    | Usage: celery [OPTIONS] COMMAND [ARGS]...
fast_treat-worker-1    | Try 'celery --help' for help.
fast_treat-worker-1    |
fast_treat-worker-1    | Error: Invalid value for '-A' / '--app':
fast_treat-worker-1    | Unable to load celery application.
fast_treat-worker-1    | While trying to load the module app.tasks the following error occurred:
fast_treat-worker-1    | Traceback (most recent call last):
fast_treat-worker-1    |   File "/usr/local/lib/python3.10/site-packages/celery/bin/celery.py", line 53, in convert
fast_treat-worker-1    |     return find_app(value)
fast_treat-worker-1    |   File "/usr/local/lib/python3.10/site-packages/celery/app/utils.py", line 384, in find_app
fast_treat-worker-1    |     sym = symbol_by_name(app, imp=imp)
fast_treat-worker-1    |   File "/usr/local/lib/python3.10/site-packages/kombu/utils/imports.py", line 56, in symbol_by_name
fast_treat-worker-1    |     module = imp(module_name, package=package, **kwargs)
fast_treat-worker-1    |   File "/usr/local/lib/python3.10/site-packages/celery/utils/imports.py", line 100, in import_from_cwd
fast_treat-worker-1    |     return imp(module, package=package)
fast_treat-worker-1    |   File "/usr/local/lib/python3.10/importlib/__init__.py", line 126, in import_module
fast_treat-worker-1    |     return _bootstrap._gcd_import(name[level:], package, level)
fast_treat-worker-1    |   File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
fast_treat-worker-1    |   File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
fast_treat-worker-1    |   File "<frozen importlib._bootstrap>", line 1004, in _find_and_load_unlocked
fast_treat-worker-1    | ModuleNotFoundError: No module named 'app'

Frontend:
beo_frontend           | ERROR in src/components/dropdown/Menu.tsx:83:17
beo_frontend           | TS2345: Argument of type '{ toggle: boolean; pointerDown: boolean; ignoreMouse: boolean; }' is not assignable to parameter of type 'Props'.
beo_frontend           |   Object literal may only specify known properties, and 'pointerDown' does not exist in type 'Props'.
beo_frontend           |     81 |             useClick(context, {
beo_frontend           |     82 |                 toggle: !nested,
beo_frontend           |   > 83 |                 pointerDown: true,
beo_frontend           |        |                 ^^^^^^^^^^^^^^^^^
beo_frontend           |     84 |                 ignoreMouse: nested,
beo_frontend           |     85 |             }),
beo_frontend           |     86 |             useRole(context, { role: "menu" }),
beo_frontend           |
beo_frontend           | ERROR in src/components/dropdown/Menu.tsx:204:29
beo_frontend           | TS2322: Type '{ children: Element; context: FloatingContext<HTMLButtonElement>; preventTabbing: true; modal: boolean; order: ("content" | "reference")[]; }' is not assignable to type 'IntrinsicAttributes & Props<HTMLButtonElement>'.
beo_frontend           |   Property 'preventTabbing' does not exist on type 'IntrinsicAttributes & Props<HTMLButtonElement>'.
beo_frontend           |     202 |                         <FloatingFocusManager
beo_frontend           |     203 |                             context={context}
beo_frontend           |   > 204 |                             preventTabbing
beo_frontend           |         |                             ^^^^^^^^^^^^^^
beo_frontend           |     205 |                             modal={!nested}
beo_frontend           |     206 |                             // Touch-based screen readers will be able to navigate back to the
beo_frontend           |     207 |                             // reference and click it to dismiss the menu without clicking an item.
beo_frontend           |
beo_frontend           | ERROR in src/core/api/client.ts:74:9
beo_frontend           | TS2322: Type 'T' is not assignable to type 'T & { detail?: string | undefined; }'.
beo_frontend           |   Type 'T' is not assignable to type '{ detail?: string | undefined; }'.
beo_frontend           |     72 |         super();
beo_frontend           |     73 |
beo_frontend           |   > 74 |         this.data = d;
beo_frontend           |        |         ^^^^^^^^^
beo_frontend           |     75 |     }
beo_frontend           |     76 | }
beo_frontend           |     77 |
beo_frontend           |
beo_frontend           | ERROR in src/core/api/client.ts:204:13
beo_frontend           | TS2322: Type 'T' is not assignable to type 'T & { detail?: string | undefined; }'.
beo_frontend           |   Type 'T' is not assignable to type '{ detail?: string | undefined; }'.
beo_frontend           |     202 |
beo_frontend           |     203 |         if (result?.headers?.get("content-type")?.includes("json")) {
beo_frontend           |   > 204 |             result.data = (await result.json()) as T;
beo_frontend           |         |             ^^^^^^^^^^^
beo_frontend           |     205 |         }
beo_frontend           |     206 |
beo_frontend           |     207 |         let ex: Error | undefined;

OK, will check it

I am using WSL2 under Windows 11 on my workstation, just run proejcvt build from scratch and all images were build correctly, could you check  docker/docker-compose version on your side? Docker-compose v2 required
I have following:
➜  docker -v
Docker version 20.10.21, build baeda1f
➜  docker-compose -v
Docker Compose version v2.13.0

Just fixed frontend build, please check latest master.
Concerning backend - please run backend Docker image build to get error details:
 docker build backend --progress=plain --no-cache

@alexandru-grajdeanu any news?


fix: enable activating remove button with keyboard

In this PR:

Pressing "enter" when focusing the widget removal transclude slot now correctly triggers a click event on the transcluded button

PR considerations checklist:

[x] Updates CHANGELOG.md to reflect this PR's change.
[x] This Contribution is under the terms of Individual Contributor License Agreements (and also Corporate Contributor License Agreements to the extent applicable) appearing in the Apereo CLA roster.

@thevoiceofzeke , does this work for compact widget mode?


Use role overwrites for new help channel system

Copying from the main PR #786:
Are per-member overwrites scalable? Discord may have a limit. Furthermore, there is supposedly client performance degradation with a high amount of overwrites. If this becomes an issue, an overwrite for a role can be used instead. The role would be added/removed from users as needed.
Not 100% decided if it should be switched to a role. Given a total channel limit of 32, there's going to be at most a bit over 32 simultaneous overwrites. That should be fine performance wise. I prefer user overwrites since they're "stealthier" than a new role, but a role would technically be a more robust solution. For now, user overwrites will stay and I'll open an issue to switch to roles.

Okay, it's time to bump this issue and increase the priority of it. We've now got a couple of days worth of experience with the current system based on permission "overrides" (see note 1) and it's not working well in ways we did not anticipate.
For those not familiar with the current mechanism, we use permission overrides to stop members from claiming a second help channel within 15 minutes of claiming their first (see note 2). We do that by adding a member-specific permission override to the Help: Available category that revokes their "send messages" permissions. This should stop them from claiming another channel as they cannot send messages to the available help channels.
An essential part of this system is that the available help channels synchronize their permissions with the category, both to make sure that expired overrides are removed and new overrides are added to the channel. Unfortunately, despite our best efforts to ensure that they do (see e.g., #873), channels stop synchronizing their permissions from time to time or, weirdly, seem to receive permission overrides from some cache that in reality no longer exist in the category.
That's why we want to switch to a role-based system: It means that the category permissions stay consistent over time, hopefully preventing channels from going out of sync. Obviously this will now rely on being able to consistently assign and remove roles from members, but let's hope that this will work as expected.

Note 1: Permission overrides are actually called permission overwrites but @lemonsaurus has his own ideas about that.

How would it deal with users which leave the guild? The roles do perist in the database but there would not be a way to know if the user's role should be re-added. Since the mapping of channels to users does not currently persist, it will not be possible to determine if the user's help channel went dormant. Should it just drop the role until some persistence is implemented?

I think it's fine for now to not persist the roles. It's a role that a user should have for a maximum of 15 minutes at a time anyway and if people try to abuse the non-persistence by leaving/joining to claim multiple channels (I don't think that will happen often), we can moderate them.
Still, persistence has come up a couple of times now for different elements of the help channel system; we should evaluate in due time if and how we're going to tackle that. Once such persistence gets implemented, we could add this to the considerations.

I've added a role (Help Cooldown -<PHONE_NUMBER>25421825) and set the appropriate permissions on the category.

Just an update: I've not made too much progress yet since it turned out to be a busy week for me. I am still planning on working on it once I am not so busy.


Add tryMap eventual for fallible mapper functions

This adds a fallible version of map that allows custom error handling (e.g. logging) and will ignore failed calls.
A good use case for this is a timer that periodically fetches data and simply skips over failed attempts by retrying with the next tick:
timer(60_000).tryMap(
  () => fetchData(),
  { onError: logger.warn(`Failed to fetch data, will try again in ~60 seconds`) }
)

@That3Percent I agree that would be a better direction. When I need something like this the next time, I'll think about a more general approach to it.

@That3Percent I agree that would be a better direction. When I need something like this the next time, I'll think about a more general approach to it.


Implement Cluster Lens Simulations

The following classes are proposed to implement group/cluster lens simulations within the current SLSIM design. This is only a draft and I really appreciate any comment and contribution.

ClusterPipeline:

Inherits: -
Purpose: Generate a sample of group-to-cluster scale halo parameters (probably NFW) and a sample of  parameters describing the subhalos of each halo (thinking on SIE, ELP or dPIE).
Implementation: Obtain the mass from a halo mass function, obtain a concentration parameter from a mass-concentration relation, and also an ellipticity. For the subhalo distribution it may use a model (e.g. Oguri's work) or query from a group/cluster catalog (e.g. redMaPPer), assign them to a halo with some criteria (e.g. richness), and fill other information with SkyPyPipeline. Maybe use synth_cluster for all this (I haven't tested it yet).

LensClusters (or maybe a more clear name):

Inherits: DeflectorBase
Purpose: Handle the parameters of group/cluster dark matter halos and their subhaloes.
Implementation: This class would use the output of ClusterPipeline to build a catalog of individual dark matter halos, encapsulating properties like mass, concentration, number of subhalos, etc. Also may assign the velocity dispersion of the subhalos in the same way as in EllipticalLensGalaxies and fill any missing data.

ClusterLens:

Inherits:  LensedSystemBase
Purpose: Combine the parameters of the halo, subhalos, and source into a single lensing system and generate a Lenstronomy model. Consider environmental effects in the same way as in Lens. This should also include the surface brightness of every subhalo.
Implementation: This class would act as a container for a complete lensing system, including a main halo, its subhalos, and the background source. It should handle the combined lensing effect of all these components.

ClusterLensPop:

Inherits:  LensedPopulationBase
Purpose: Contain and manage a population of halos, subhalos, and sources. Be able to generate lensed images that meet some criteria.
Implementation: This class would generate random combinations of halos, subhalos, and sources to create instances of ClusterLens. It should include methods for testing the validity of the generated lens systems.

Anyone interested can follow the progress in this fork

Hi @furcelay, thank you very much for this initiative! Looking forward to seeing your progress on this promising initiative.

Thank you very much @furcelay for this very detailed and excellent description of your planned work!
This is quite a substantial project and there are certain things we (@nkhadka21 and myself) will probably have to do/help on the design to extend and allow of the complexity of the models (for example, having the possibilities for multiple sources associated with one lens. These things are all doable. I highly recommend you in small steps with small-ish pull requests such that we can sort out step-by-step how these features can be implemented and made available consistently through SLSim.
@nkhadka21 what we might start with on our side is having the skeleton for the elliptical NFW profile and the Hernquist light (as also required for Katsuya's model), and then have a Deflector class with multiple sets of such (perhaps even literally as a list of 'single' deflectors


Reset image is not fit in canvas.

When I click to sign button then clear image then click to reset button.
So signature image is not come exact fit.

What browser are you using?
Everything is confirmed to work fine in:

Google Chrome
Firefox
IE11
Edge

Patches are welcome.

I can confirm the above behavior. It seems to happen on high DPI devices. Using Chrome, my Android phone, iPad, & iMac all show the problem. However, an old 1080p Windows 7 machine works fine.
The signature_pad README makes mention on this. I'm currently tinkering with the directive source & will report back if I make any progress.


ROUND | Switch Like Home bug

While using the Round theme along with  Switch Like Home, it seems to displace and not fit correctly after entering in a game library page and coming back to home:
https://streamable.com/hk2che

Will be treated as a bug to attempt better compatibility with other themes. Will be fixed and (hopefully) updated in the theme database by the end of the day.

Never mind, my fix doesn't appear to be working. Going to keep investigating.


Fix org-static-blog-get-body logic

As far as I understood, the logic should find where the postamble div is, assuming it is immediately after content div, then skip the comments div if any.
The fix is to correct this logic, without it when comments are enabled, it will make the index page look wrong when org-static-blog-use-preview nil.

To be honest, I don't understand the issue. What exactly is rendered incorrectly on the index page?

In my case, it happens when (setq org-static-blog-post-comments "

") and org-static-blog-use-preview nil.
The post output would look like roughly this:

...

Without the fix, in the index page due to the org-static-blog-get-body logic, from the second post it would look like a unclosed
 hence breaking the layout:

 <--- not closed

Thank you for the clarification. It seems that the post-extraction logic was not updated when we introduced the contents div.
Actually, we should probably move the comments-div out of contents, but that would potentially breaks existing CSS rules. I'd rather not do that without a good reason.
Alternatively, we could extract the entire contents, and scrape out the comments div afterwards manually. That's a bit more effort, but probably the better solution. This could be done as an additional step at the end of org-static-blog-get-body.
Would you like to take a stab at that?

I do notice that the content divs themselves are not included in the index page. In the change you are proposing, would you want these content divs also be part of the page or?

Okay, I mean I refactored "org-static-blog-get-body"  a bit with a two steps process. I am not sure if this is the best way approaching this. But I don't know idomatic way of processing it in emacs neither...

I am confused. I think my previous comment was mistaken.
I had somehow assumed that the extracted content was supposed to include the
, but looking at it now that doesn't seem to be the case. If that's true, your original commit was actually completely correct, I think. Right?
Although to avoid further confusion, we should probably change the function name org-static-blog-get-body to org-static-blog-get-content, and add a comment explaining that (search-backward "
" nil t) is optional because comments are optional.
I am terribly sorry for the confusion I caused.

Okay. Don't worry, I am the beneficiary of your work rather, nothing to complain here from me.
Change org-static-blog-get-body could indeed resolve some confusion, unless backward compatibility is an issue (affecting people who redefined the function name).
Let me know, I could restore the original code and add more comments, and perhaps change function name too.

If you'd like to do that, I'd be grateful!

Hi @bastibe,
I am finally back to this. Please have a look now!

Note that org-static-blog-get-post-content could be a breaking change to people actually customize the function, but maybe flagging it in the commit message is sufficient for people to find out!

Thank you for getting back to this!
The pull request looks good to me. I'll happily merge it when you think it's ready.

Yes, let's do it!


Fix handling of unknown types

When the dialect encounters an unknown type it tries to instantiate
NULLTYPE which is not callable. This removes that and correctly passes
it as a sentinel value.
Essentially an updated version of #31

The tests passed, build failure appears to be because of a missing dependency in the CI environment.

I just noticed #84, rebased with master.

Thanks for the contribution and for updating the PR! I just have one request (shared with https://github.com/cockroachdb/cockroachdb-python/pull/31#issuecomment-321332445): can you please add a test that actually executes this line?

I've added a test, and confirmed that is was failing before the change I made.


Improving error message when users pass layers_to_transform and layers_pattern

Hi @BenjaminBossan Ive reopened with another PR.
This tries to add some verbose message when users pass layers_to_transform and layers_pattern addressing point 3 in https://github.com/huggingface/peft/issues/2155.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Deprecate support for IO In FFI declarations

When doing FFI, there's always a confusion about whether to use IO or whether to use Java. The support for IO was added for convenience but it looks like it is hurting more than it is helping.
@mandubian says on Gitter:

I tend to like to see java someFunction in my code
it shows where I'm in java world

Which is a very strong point. The whole point of the FFI is to clearly separate functions that are imported from the Java world and having IO blurs that barrier.
This change will be quite large since it requires us to refactor all the existing FFI declarations that involve IO, which shouldn't be many.
The best way to start is with a deprecation warning that is emitted by the compiler when it detects IO. In Eta v1.0 (or even earlier), we can remove all support for IO entirely in FFI declarations.
We need to think through other consequences of this decision. Another thing to think about is whether functions on immutable data types like java.lang.String should be allowed to have pure return types or whether they must also have a Java monad return type.
Thoughts on this?

I don't really get the problem, but I don't care very much about Java vs IO.
I think pure Java things should be able to be typed as normal Haskell values.

@puffnfresh it's just that when you try to write a FFI to a Java lib, you don't know whether you should build API with Java or IO monad or both... I experimented exactly that!
Personally I would prefer all automatically built FFI to be only using Java monads...
Then I can call java myJavaMonadAPI from Haskell and it clearly shows where java is called in Haskell code.
... Or the other solution in the java FFI builder would be to always build a Java Monad API and the corresponding IO version using only Haskell types...
For the types, it's also not so trivial between Haskell types, Eta/Java types and JavaConverters...

We'll defer this question until after the FFI generator comes out - it's an active work item right now. I have a feeling this won't matter much when all the bindings are more or less generated for you.
At that point, IO can be used by people who are doing fine-grained imports - just a handful of methods from a Java library.
I want to make sure there's a balance between flexibility of library authors being able to define their own type signatures which the FFI gen won't be able to predict uniformly, and normal users to be able to generate a "standard" set of bindings.
A practical example of this is the signature for the Spark web framework's request and response handlers. They return/take Object when it should really be a generic type. The flexibility of this approach allows you to add stronger typing to a weakly typed Java API. You can even add phantom types into in the mix.

So I guess a compromise to this issue is to make the FFI generator generate Java monadic actions by default and add some overrides in case you want to generate IO for whatever reason (you're importing a library full of static, effectful methods).


Whatsapp Sharing Failed! Please try again

Hi Authors,
Thanks a ton for the script.
I've implemented the script and followed the installation steps mentioned by you. Additionally, I've reviewed the FAQ.
I am unable to find a solution, hence reaching out to you.
Link -http://m.flickstime.com/Ahmedabad/movie-shows-in-theater-details/mvid/557/movie/Ek-Paheli-Leela-
I've installed the button in the link mentioned above. When i try to click the button it shows - sharing failed. Try after some time! I am not sure what's the issue? Can you help?
Thanks a ton in advance
Boni

The create() method doesn't run in your example. So the a tag isn't replaced with an iframe and the data attributes are not added to the url scheme. I guess the code doesn't fire because of the massive amount of js-errors on your page. (Check the console, I've counted 71 on Firefox)
As a hotfix you can try to run theWaShBtn.crBtn(); after the page is loaded, but you should probably fix all the other errors.

facing same issue .......... in this regards could you please help me

Noone can help you without code to look at.
Read this first: https://github.com/kriskbx/whatsapp-sharing/blob/master/ISSUES.md

I also get this issue.
No console errors, if I type WASHAREBTN in console, it seems to be working
From my PC I get:
WASHAREBTN {isIos: false, cntLdd: function, addStyling: function, setButtonAttributes: function, setIframeAttributes: function…}
However, when I click it on my mobile, on Android I get a sharing failed message, while on iOs it opens Whatsapp, I select a recipient, but no link is shared.
My whatsapp link is like this:
Whatsapp
In my footer I have:

Well, @akmur you removed all the css-classes and applied your own... The orignal classes are needed for the JavaScript part to select the button elements. This is the original code you must use: Share
You also removed style="display:none" which ensures that the button won't be displayed by default. On a device that supports WhatsApp the JavaScript will display it again.
As I said here - and this is important though - the library was not made for heavy modifications. If you want to use your own styling feel free to research/fork/clone the code and build your own button upon it.

Many thanks for your reply, I thought classes were just for styling purpose it seems to work now


Irregular whitespace in task terminal output on Windows

While testing #76016 on Windows.
task.json used:
{
    // See https://go.microsoft.com/fwlink/?LinkId=733558
    // for the documentation about the tasks.json format
    "version": "2.0.0",
    "tasks": [
        {
			"label": "One",
			"type": "shell",
			"command": "echo 1",
			"problemMatcher": [],
			"dependsOrder": "sequence",
			"dependsOn":[
				"Two",
				"Three"
			]
		},
		{
			"label": "Two",
			"type": "shell",
			"command": "echo 2",
			"problemMatcher": []
		},
		{
			"label": "Three",
			"type": "shell",
			"command": "echo 3",
			"problemMatcher": [],
			"dependsOrder": "sequence",
			"dependsOn": [
				"Four",
				"Five"
			]
		},
		{
			"label": "Four",
			"type": "shell",
			"command": "echo 4",
			"problemMatcher": []
		},
		{
			"label": "Five",
			"type": "shell",
			"command": "echo 5",
			"problemMatcher": []
		}
    ]
}

This generates the following output with irregular whitespace inbetween:

@kieferrm what version of Windows are you using?

Microsoft Windows [Version 10.0.17763.107]

This is the good old winpty bug. Duplicate of #57803


panic:cannot intersect: ^2 && ^3.7 when using python

I wanted to run an old python 2 script and tea seemed perfect for the job since I don't want to fuzz around with managing python versions. However, when I run tea python^2 I get that error (if I do tea python^2.7 I get panic:cannot intersect: ^2.7 && ^3.7. tea python just brings up python 3.11.
I assume this is a bug because tea node^16 for example works fine. It's also suspicious that it says it's not compatible with 3.7 since the last minor of 2 was 2.7 and 3 is up to 3.11.
Is it just that it's not in the registry or something?
I'm on an M1 mac btw.

This is because our pip pkg is considered a “companion” to python but we don't have a pip that can support python < 3.7.
An easy fix for now would be to stop pip having that requirement.
After that the problem is we don’t have a python 2 pkg. We tried but it was quite involved. I'll give it another go today.

Solved with a hack.


AuthenticationProvider.createSession: Spec what to do when the authentication fails

Testing #117299
AuthenticationProvider.createSession should spec what needs to be returned when the login fails (rejected promise).
Also

mention that a successful login should trigger the onDidChangeSessions event.
mention what should happen when a session already exists for the given scopes . Looking at the Github AuthProvider implementation, I think it authorize again and replace the existing session. The event is remove and add in that case. Unclear to me if changed could also be ok to be used? I created #117414 to clarify what changed should be used for (vs remove/add)

I think AuthenticationProvider.createSession should still spec what needs to be returned when the login fails (rejected promise).


Warnings in actions log

Describe the bug
Warnings are displayed in actions log
To Reproduce
Run the action and observe the log
Expected behavior
No errors or warnings
Screenshots

Desktop (please complete the following information):
Not applicable
Smartphone (please complete the following information):
Not applicable
Additional context

Thanks for your report. Fixed in v1.0.2
Now you should use v1.0.2
- name: Create a Branch
  uses<EMAIL_ADDRESS>


Add support HTTP status code, bytes written and potentially other data (?) to the middleware package

Currently there's no way to log the status code or bytes written to the client using the middleware. Both of these can be useful when trying to debug issues or when just generally looking at logs to pickup on patterns of requests. It would be nice there was a way to extract this data from the response writer so it can be logged after a reply to a client is sent.

Seems like a bit of an oversight, eh? 😬

The proposed fix in #12 means breaking the API, so the MR suggests merging to the v2 working branch


htmq: add section on latency and data speeds

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#62
#61  👈
#60
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @dyc3 and the rest of your teammates on  Graphite

Merge activity

May 4, 5:44 PM EDT: @dyc3 started a stack merge that includes this pull request via Graphite.


Ansible roles: Traffic Ops no longer serves static files

I'm submitting a ...

bug report

Traffic Control components affected ...

Ansible roles

Current behavior:

As mentioned in #5587, Traffic Ops Perl included a static web server for files in the /opt/traffic_ops/app/public directory, but Traffic Ops (Golang) does not include this functionality (and TO Perl was removed in #5209). Also, /opt/traffic_ops/public itself was removed in #5800.
The Ansible roles still expect to get the MaxMind GeoIP database, coverage zone file, and other static files from Traffic Ops:

https://github.com/apache/trafficcontrol/blob/b05a5e48b8a7031407de3ffffe3c4da371d02fd8/infrastructure/ansible/roles/traffic_ops/defaults/main.yml#L30
https://github.com/apache/trafficcontrol/blob/eed246c7854f6f9aefab4b65ce6c35846460e8c9/infrastructure/ansible/roles/dataset_loader/defaults/main.yml#L28
https://github.com/apache/trafficcontrol/blob/eed246c7854f6f9aefab4b65ce6c35846460e8c9/infrastructure/ansible/roles/dataset_loader/defaults/main.yml#L30
https://github.com/apache/trafficcontrol/blob/eed246c7854f6f9aefab4b65ce6c35846460e8c9/infrastructure/ansible/roles/dataset_loader/defaults/main.yml#L32
https://github.com/apache/trafficcontrol/blob/eed246c7854f6f9aefab4b65ce6c35846460e8c9/infrastructure/ansible/roles/dataset_loader/defaults/main.yml#L34
https://github.com/apache/trafficcontrol/blob/eed246c7854f6f9aefab4b65ce6c35846460e8c9/infrastructure/ansible/roles/dataset_loader/defaults/main.yml#L346

Also, these parameters are unused by the Go implementation of Traffic Ops:

tm.logourl in the GLOBAL profile: https://github.com/apache/trafficcontrol/blob/b05a5e48b8a7031407de3ffffe3c4da371d02fd8/infrastructure/ansible/roles/dataset_loader/defaults/main.yml#L388-L390
geniso.iso_root_path in cdn.conf (this is a directory where generated ISOs can be downloaded after the POST /isos response and in TO Perl was within /opt/traffic_ops/public): https://github.com/apache/trafficcontrol/blob/b05a5e48b8a7031407de3ffffe3c4da371d02fd8/infrastructure/ansible/roles/traffic_ops/templates/cdn.conf.j2#L18-L20

Expected behavior:

The Ansible roles should reference a new static web server that can host these files.
tm.logourl in the GLOBAL profile and geniso.iso_root_path in cdn.conf should be removed.
we should document the need to set up a static web server to host these files.

Since a blanket webserver is not part of the stock ATC project, there's not a better alternative.

Since a blanket webserver is not part of the stock ATC project

That shouldn't be an issue if we document the need to set one up.

Should the TO API endpoint also be updated to a valid version?

https://github.com/apache/trafficcontrol/blob/eed246c7854f6f9aefab4b65ce6c35846460e8c9/infrastructure/ansible/roles/dataset_loader/defaults/main.yml#L25

IMO only if API updates are necessary to close this Issue. The Issue for updating the Ansible roles to use TO API v4 is #5377.
That said, updating the Ansible roles to use TO API v4 seems like it could be a pretty large task, so maybe we can break #5377 into multiple Issues so we can chip away at it.

The only update that's actually necessary is that the variable default TO_URL should be swapped to localhost to conform with  other fields that have no appropriate default.


feature: Update matplotlib and numpy

note to tomorrow's self: WTF.  numpy update is a bug fix release with no api changes.

This seems to work now following the merge of #472