⚠ Bofrost.de has degraded performance

In 94679a1, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 955 ms

Resolved: Bofrost.de performance has improved in 402810d after 47 minutes.


Documentation needs to be more clear

Instructions in the ReadMe are not clear and at times, confusing.

Probably, because I'm using Windows.

Hey @shubhamcodez, Thanks for reporting that!
Where did you find in confusing. I think I can help you out!

I created a Personal Access Token but, what is a secret ?
then, I went to Actions and there is no option to add a new Workflow.
Some problems I think were coz of different OS:
What does cp -r do and what to do after fetching and generating_doc.py

Go to the repo settings -> secrets -> new secret. You will find repo settings on the first page of your repo, right after insights. Add your access token as a secret with the name GIT_TOKEN. We need to use it as a secret because we can not make our access token publically available. Others can misuse it.
I don't think there should be any problem with OS. cp -r will copy things recursively.

ok got it. Thanks a lot


fix yaml deprecations for symfony version 3.1

At least:
Not quoting the scalar "%boekkooi.jquery_validation.additional.required_group% " starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0: 2x

Not quoting the scalar "%boekkooi.jquery_validation.form.form_extension.class%" starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0: 1x

Not quoting the scalar "%boekkooi.jquery_validation.additional.luhn% " starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0: 1x

Not quoting the scalar "%boekkooi.jquery_validation.additional.pattern% " starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0: 1x

Not quoting the scalar "%boekkooi.jquery_validation.additional.accept% " starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0: 1x

Not quoting the scalar "%boekkooi.jquery_validation.additional.one_or_other%" starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0: 1x

Not quoting the scalar "%boekkooi.jquery_validation.additional.ipv6%" starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0: 1x

Not quoting the scalar "%boekkooi.jquery_validation.additional.ipv4%" starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0: 1x

Not quoting the scalar "%boekkooi.jquery_validation.additional.is_boolean% " starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0: 1x

Not quoting the scalar "%boekkooi.jquery_validation.additional.iban% " starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0: 1x

Not quoting the scalar "%boekkooi.jquery_validation.enabled%" starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0: 1x

Not quoting the scalar "%boekkooi.jquery_validation.additional.time% " starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0: 1x

Not quoting the scalar "%boekkooi.jquery_validation.constraint_finder.class%" starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0: 1x

Not quoting the scalar "%boekkooi.jquery_validation.rule_compiler.class%" starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0: 1x

Not quoting the scalar "%boekkooi.jquery_validation.rule_processor.class%" starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0: 1x

Not quoting the scalar "%boekkooi.jquery_validation.form.button_extension.class%" starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0: 1x

Not quoting the scalar "%boekkooi.jquery_validation.twig.jquery_validation.class%" starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0: 1x

Hi @robertfausk, Thanks for reporting this issue do you have a chance to create a PR for this?

Closing this since @lvillanne created the PR :smile:

Thx @lvillanne :)


Update getFileScanRDD shim for recent changes in Spark 3.3.0 [databricks]

Fixes #4423.  Updated the getFileScanRDD shim interface to take the read data schema and metadata columns, and updated the only place this is called to pass the read data schema.  I filed #4426 to track handling the new hidden metadata column functionality added in 3.3.0.

build

build

build

Do you know what the change was for?

Do you know what the change was for?

Yes, details are in #4426 and the linked Spark JIRA.  Spark 3.3 added support for loading "hidden" metadata if specified explicitly.  This PR just gets us building again, but we'll need to properly handle this feature, either by implementing the metadata columns for the GPU or falling back to the CPU if they are detected in the read schema, when we start officially supporting Spark 3.3.0.


CTHUB - Investigate Data BC Address Geo locater for CTHUB

Problem Description
Investigate Data BC Address Geo locater for CTHUB
Solution Needs

Investigate if we can incorporate Data BC Address Geo locater for CTHUB:
https://www2.gov.bc.ca/gov/content/data/geographic-data-services/location-services/geocoder

Timebox
Outcome
Details describing the outcome of the research

Was it successful? What direction should the work go?
Was it unsuccessful? Discuss next steps with team

We met with the engineers on Feb 22, 2024, and discussed BC Stat's Geo Locator, for which our branch has a license. We are currently awaiting information on whether an API is available.

BA will be reaching out to BC Stats again to ask more questions as another teams requirements have changed.


Broken Spanner tests

In which file did you encounter the issue?
Test file: spanner/leaderboard/complete/src/test/java/com/google/codelabs/AppTest.java
Describe the issue
Test Fusion

@averikitsch or @skuruppu Could one of you copy-paste the error from Fusion into this issue? I cannot access the build logs on Fusion.

This problem was fixed in https://github.com/googleapis/google-cloud-java/pull/5952. That PR was merged on 2019-08-01 and this flaky failure occurred on 2019-06-25, so this should not happen anymore. If it does, then please re-open.


Feature: Write a macro to convert a jinja dictionary to yaml

This will make generating yaml so much cleaner!
This is almost it, but it doesn't unnest things that are lists in the way we would expect:
{% macro dict_to_yaml(my_dict, indent=0, result="") %}
{% set ns=namespace(result=result) %}
{% for key, value in my_dict.items() %}
    {% set ns.result= ns.result ~ " " * indent ~  key %}

    {% if value is mapping %} {# if it's a dictionary, recurse #}
    {% set ns.result=dict_to_yaml(value, indent + 2, ns.result ~ ":\n") %}
    {% elif value is iterable and value is not string %} {# janky way to check for a list #}
    {% set ns.result = ns.result ~ ":\n" %}
    {% for item in value %}
        {% set ns.result = ns.result ~ " " * indent ~ "- " ~ item ~ "\n" %}
    {% endfor %}
    {% else %}
    {% set ns.result = ns.result ~ ": " ~ value ~ "\n" %}
    {% endif %}

{% endfor %}

{{ return(ns.result) }}
{% endmacro %}

For anyone who could stumble on it from a Google search: there is a toyaml function in DBT right now to generate yaml.


Moar Compiler Flags

Add some of these compiler warnings:
-Wduplicated-cond
-Wcast-align
-Wcast-qual
-Wunused
-Wmissing-declarations

https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html
https://web.archive.org/web/20111015064908/http://lars-lab.jpl.nasa.gov/JPL_Coding_Standard_C.pdf

Added some more, some of these checks should also get caught by the static analyzer.


Cropped overlay image when user changes zoom

Is this a bug report?
Yes
Have you read the Installation Instructions?
Yes
Environment
react-native: 0.54.0-rc.3
react: 16.0.2
react-native-maps<EMAIL_ADDRESS>
iOS 11.2, iPhone and Simulator, AppleMaps
Steps to Reproduce
Sometimes when I zoom in and zoom out of the map I cannot see rest of the image overlay. I don't know how exactly it happens but I found that when I want to change images (1198×760 px) in one overlay it happens very often. It happens also sometimes without changing images.
Expected Behavior
Overlay in Maps should be visible all the time without any changes based on zoom.
Actual Behavior
VIDEO: https://youtu.be/LJiyA3xJUKg
Cropped detail: User can't see left side of the image

Cropped: User can't see bottom side of the image

Not cropped – Proper behaviour, user can see whole image

There is also weird behaviour when I zoom too much. Whole overlay will dissapear.
Reproducible Demo
class DemoScreen extends React.Component<null, DemoScreenState> {
interval: ?Object = null;
actualImageId: number = 0;
map = null;

state = {
  region: {
    latitude: LATITUDE,
    longitude: LONGITUDE,
    latitudeDelta: LATITUDE_DELTA,
    longitudeDelta: LONGITUDE_DELTA,
  },
  overlay: {
    bounds: [OVERLAY_TOP_LEFT_COORDINATE1, OVERLAY_BOTTOM_RIGHT_COORDINATE1],
    image: Images.radar[this.actualImageId],
  },
};

play = () => {
  this.interval = setInterval(
    () => {
      if (this.actualImageId === 7) {
        this.actualImageId = 0
      }
      this.setState({
        overlay: {
          bounds: [...this.state.overlay.bounds],
          image: Images.radar[this.actualImageId],
        },
      })
      this.actualImageId += 1
    },
    1000,
  )
};

render() {
  return (
      <View>
        <View style={styles.mapContainer}>
          <MapView
            ref={ref => {
              this.map = ref
            }}
            style={styles.map}
            initialRegion={this.state.region}
            showsUserLocation
          >
            <MapView.Overlay
              bounds={this.state.overlay.bounds}
              image={this.state.overlay.image}
              zindex={2}
            />
          </MapView>
        </View>
        <View style={styles.buttons}>
          <Button onPress={this.play}>PLAY</Button>
        </View>
      </View>
    )
  }
}

export default DemoScreen

I am experiencing the same issue with images cropped horizontally.  Maybe it is due to some z-index issue with the map tiles?

Any news about this ? There are still major issues with overlays on iOS (cropping and/or not rendering at all if the map is too much zoomed in), even on latest react-native-maps versions.
react-native: 0.59.3
react: 16.8.3
react-native-maps: 0.23.0

Video : https://www.youtube.com/watch?v=A3wf40oubeA
I tried to dig a bit in the implementation but my Objective-C knowledges are closed to none. Despite,  I think from my tests that the drawMapRect() delegate function in lib/ios/AirMaps/AIRMapOverlayRenderer.m is not called when it should.
Could it be a MapKit issue ? I successfully did a small repro on an empty react project with the MapKit Swift version and it was working great.
Could it come from a lack of implementation of the zoomScale:(MKZoomScale) parameter in the Air component ?
Also, I tried to jiggle with zindex/zIndex but I'm not sure that this props exists ? (Nothing in the  doc, despite some implementation in the Android part, but I could not find any reference in the iOS code)


Add recaptcha v3 to sign up form

What does this pull request do? Explain your changes. (required)

Specific updates (required)

Add recaptcha v3 to sign up form

Does this pull request close any open issues?

Fixes #699

Screenshots (optional):

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

Hey @FullStackTiger this change seems to have broken the API build/tests. Could you take a look at it?

Ah, I will check it @victorges

I created one pr @victorges
https://github.com/livepeer/livepeer-com/runs/4351112133?check_suite_focus=true


Add homebrew download stats

:clipboard: Description

Which service is this badge for: Homebrew
What sort of information should this badge show?: Download stats
Provide an example in plain text e.g: "downloads | 1200/year"

:link: Data
Where can we get the data from?

Is there a public API?: yes
Does the API requires an API key?: No
Link to the API documentation: https://formulae.brew.sh/formula/

:microphone: Motivation
Please explain why this feature should be implemented and how it would be used.

What is the specific use case?

It would be nice to show download stats from homebrew, similar to how we can now show download stats from github releases.

I'm having a bit of deja vu with this request, though can't seem to find any issues where we've discussed it previously.
We wouldn't want to use https://formulae.brew.sh/api/formula.json as that's way too big and takes way too long to retrieve, but i guess we could look at the analytics.install fields in the formula-specific api response (which is what we use for the version badges) to get the raw data.
This would be a relatively easy badge to add, and would recommend anyone interested in implementing take a look at our tutorial as well as the existing homebrew version badge code for reference. Note it might be beneficial to extract the common bits out (like the fetch function) into a separate helpers file or a base class that the existing version badge and future downloads badge could extend

Thanks for the comment! I submitted a PR for this.


Clarify processing of pod resource overrides

By using reflection, instead of per-struct methods (GetMetadata() etc.) for each override struct that we define, it becomes much easier:

to see that the logic is uniform, and is to apply overrides whenever the relevant override struct has a particular field; and
to document any field-specific details of the overriding process, e.g. that Metadata labels and annotations are only copied across when they have non-clashing keys.

Thanks @caseydavenport .  Yes, I think I can see how we can easily add UT to catch any fields that are unhandled, and I'm very happy to follow up with that.  I will merge this PR now and follow up with that extra UT in a separate PR.

The extra UT - and also a fix for PriorityClassName - is at #3626


Wei from Eth return 0 for decimal value

$wei = Wei::fromEth('0.01');
Results:
Web3\ValueObjects\Wei^ {#874
-value: "0"
}

Perhaps could use BC Math functions: https://www.php.net/manual/en/ref.bc.php

The current implementation uses phpseclib/phpseclib's Math classes, which will fall back to the BCMath or GMP functions if they're available, and otherwise use a userland implementation. However, I was able to reproduce this issue, both with BCMath and GMP. I'm currently not able to test the userland implementation as I'm on Windows atm which always has BCMath enabled by default.


BaseModel assumptions

In upgrading the version control app to v2 support, I have found several places where we should remove assumptions about models and feature availability.
In summary, we make a number of unsafe assumptions about custom Nautobot APIs that should be addressed to create friendlier interfaces.

Naive assumption that any model instance passed in supports change logging and have the create/upgraded fields.
https://github.com/nautobot/nautobot/blob/b969cb8f68322411bf62f33ab572e4a2930aa9c7/nautobot/core/views/generic.py#L104
A few ways we could solve this, but regardless, we should add a method to ObjectView that calls get_created_and_last_updated_usernames_for_model instead of this happening directly in get. This would allow a consumer of ObjectView to easily override the method and control the behavior, without the need to override all of get.
Further, the internals of get_created_and_last_updated_usernames_for_model should check to see if the instance passed is actually a ChangeLoggedModel and short circuit.
The type checks in get_changes_for_model are insufficient because changed_object_id must always be a UUID per the ObjectChange model and 3rd party libraries rarely use a UUID pk. `https://github.com/nautobot/nautobot/blob/b969cb8f68322411bf62f33ab572e4a2930aa9c7/nautobot/core/utils/lookup.py#L20

Related? #1169


📝 Update includes in ./docs/zh/docs/advanced/using-request-directly.md

Thanks for the effort! :cake:
This was handled in another PR, so I'll close this one now. :coffee:


Pre-2006 videos show NPC's only moving cardinally when pathing towards target

Video example 1 (source):
https://github.com/user-attachments/assets/26436d0c-5fb3-4275-aa97-3059dc542c9f
Video example 2 (source):
https://github.com/user-attachments/assets/13514441-c284-4cbc-a9cf-679ca3adad00
Looks like it moves on the x axis first, until lined up, or blocked, with its target. Then it moves on the z axis. No diagonal movement! Though interestingly npcs can still be safespotted diagonally, so perhaps it doesnt always switch the axis if blocked. Hmmmmm
Video of npc switching axis when hitting a wall (source):
https://github.com/user-attachments/assets/011848ee-7892-48b5-913a-cda2bf96852b
Diagonal safespot:

Not gonna be easy to pull off with basically 0 information and can't test this in the live game. Imo not worth trying to "fix" without risk of breaking parts of the server.

Let's try getting the server in beta first?

I think it should definitely be implemented eventually. Pretty clear-cut proof that npc pathing was different during, and probably before, 2005. Can't call ourselves authentic with out it
I forgot to mention this in the main post but, in the 3rd video shown, the npc moves in a sort of zigzag pattern since the west wall is shaped like this (camera is facing west btw)

What if it only affected npcs larger than 1 tile? We have no idea. That's just 1 example of not enough information.

Thats a good theory. Went ahead and searched for more videos and found some examples, and I think you're right (see video 1)!! Also looks like the change was pinned down to be around September/October of 2005?

Video 1, April 2005. Skeleton moves diagonally, and right after the scorpion moves cardinally! (source):

https://github.com/user-attachments/assets/8b7e6903-ee34-4e1c-a7db-2b004f49b5be

Video 2, Sept 11, 2005. Big npc with cardinal pathing, at the top of the screen. (source):

https://github.com/user-attachments/assets/554ce1c5-868f-4e64-88ae-b76739ee3ed2

Video 3, Oct 16, 2005. Big npc moving diagonally. (source):

https://github.com/user-attachments/assets/80efafd4-c1ca-4375-aee3-6f003b067412

My theory is that it big npcs were changed when fight caves were released. Its probably the biggest update between September 11 and October 16. The blog post for it dont seem to mention anything though unfortunately

so wats da fix??

Do it without naive pathfinder used is what it sounds like to me. Wandering and escape mode are very familiar to this.

This is probably authentic? idk if we'll ever find a video of this
https://github.com/user-attachments/assets/1644da16-36f7-45d5-b099-61e3914ab4be

Found a video from Aug 9, 2006 with big npc pathing! Perhaps big npc pathing was changed in 2006 with the npc graphical reworks? Video (source):
https://github.com/user-attachments/assets/d1346769-9a3b-4106-9801-b98343fd0913
I think the video from

Video 3, Oct 16, 2005. Big npc moving diagonally.

Was just west movement + hit anim + south movement next tick.

Some more clips.
Sept 21, 2006 (source):
https://github.com/user-attachments/assets/31a1822f-2ccd-4664-829b-fab929a2fe93
Oct 7, 2006 (source):
https://github.com/user-attachments/assets/4094170c-f587-4477-adda-12c6aa4286f3


Overhauled the ParallaxBackground system

Overhauled the ParallaxBackground so that it works properly regardless of Camera Zoom or Camera anchor point.
It builds on my previous ParallaxBackground work: https://github.com/godotengine/godot/pull/12132 and both try to fix the bug I submitted here: https://github.com/godotengine/godot/issues/11555
Essentially, the Parallax Background previously acted weirdly depending on Camera2D's zoom, and whether it was anchored the the center or top left.
I've coded the camera to send more information to the ParallaxBackground, using this information it's able to have a properly behaved Parallax Background behaviour, regardless of the camera's zoom or anchor point. It operates consistently and in line with the other elements on the screen.
I should note, this patch does NOT fix "Ignore Camera Zoom" setting. While that mode behaviour acts fine on a superficial level, it acts weird when you zoom in and out. The reason for this is that the Camera2D only sends screen space coordinates to the ParallaxBackground, and in order to fix it the camera would need to send it's actual real world position.
Example project used for testing:
ParallaxRecreate.zip
Left/Right- Scrolls left and right
Up/Down- Zooms in and out.
PgUp/PgDown - Scrolls up and down (not needed much)

By the way, if this gets accepted, I can look towards backporting it to 2.1. I'll see the response to this first.

Quoting @reduz:

it seems good too, let people complain if it breaks

This seems to break infinite backgrounds though...
E.g., set a mirroring on one of the parallax layers in the demo -- it won't repeat as it did before this PR.

Oh, the documentation wasn't clear about what mirroring was meant to do so I thought it just meant flip the background.
I'll fix this up along with other issues and make another pull request in a week or two. Thanks for bringing it to my attention. If you can point to anything that would better explain its behaviour, let me know. Thanks.

Oh, the documentation wasn't clear about what mirroring was meant to do so I thought it just meant flip the background. I know the code I changed that probably broke this.
I'll fix this up along with other issues and make another pull request in a week or two. Thanks for bringing it to my attention. If you can point to anything that would better explain its behaviour, let me know. Thanks.

For the record, I think "Mirroring" should be renamed "Tiling", more clearly explains what its about.

See #13167 to keep track of the regression. I'll need it fixed in the coming days before I make official beta binaries.


#944 The actions don't work when arrivals list is sorted by route

@aaronbrethorst  While sorting by route, it calls the following method in "OBAStopViewController" class:
   - (OBATableSection*)createDepartureSectionWithTitle:(NSString*)title fromDepartures:(NSArray<OBAArrivalAndDepartureV2*>*)departures {
    NSMutableArray *rows = [[NSMutableArray alloc] init];

    for (OBAArrivalAndDepartureV2* dep in departures) {
        OBADepartureRow *row = [[OBADepartureRow alloc] initWithAction:^(OBABaseRow *blockRow){
            OBAArrivalAndDepartureViewController *vc = [[OBAArrivalAndDepartureViewController alloc] initWithArrivalAndDeparture:dep];
            [self.navigationController pushViewController:vc animated:YES];
        }];
        row.routeName = dep.bestAvailableName;
        row.destination = dep.tripHeadsign.capitalizedString;
        row.statusText = [OBADepartureCellHelpers statusTextForArrivalAndDeparture:dep];
        row.model = dep;
        row.bookmarkExists = [self hasBookmarkForArrivalAndDeparture:dep];
        row.alarmExists = [self hasAlarmForArrivalAndDeparture:dep];
        // Only allow alarms to be created if the time to departure is greater than OBAAlarmIncrementsInMinutes.
        row.alarmCanBeCreated = dep.timeIntervalUntilBestDeparture > OBAAlarmIncrementsInMinutes * 60;

        [row setShowAlertController:^(UIAlertController *alert) {
            [self presentViewController:alert animated:YES completion:nil];
        }];
        [row setToggleBookmarkAction:^{
            [self toggleBookmarkActionForArrivalAndDeparture:dep];
        }];
        [row setToggleAlarmAction:^{
            [self toggleAlarmActionForArrivalAndDeparture:dep];
        }];
        [row setShareAction:^{
            [self shareActionForArrivalAndDeparture:dep atIndexPath:[self indexPathForModel:dep]];
        }];

      OBAUpcomingDeparture *upcoming = [[OBAUpcomingDeparture alloc] initWithDepartureDate:dep.bestArrivalDepartureDate departureStatus:dep.departureStatus arrivalDepartureState:dep.arrivalDepartureState];

        row.upcomingDepartures = @[upcoming];
        [rows addObject:row];
    }

    OBATableSection *section = [[OBATableSection alloc] initWithTitle:title rows:rows];
    section.tag = kStopsSectionTag;
    return section;
}

I've added the following code inside the method:
    row.bookmarkExists = [self hasBookmarkForArrivalAndDeparture:dep];
        row.alarmExists = [self hasAlarmForArrivalAndDeparture:dep];
        // Only allow alarms to be created if the time to departure is greater than OBAAlarmIncrementsInMinutes.
        row.alarmCanBeCreated = dep.timeIntervalUntilBestDeparture > OBAAlarmIncrementsInMinutes * 60;

        [row setShowAlertController:^(UIAlertController *alert) {
            [self presentViewController:alert animated:YES completion:nil];
        }];
        [row setToggleBookmarkAction:^{
            [self toggleBookmarkActionForArrivalAndDeparture:dep];
        }];
        [row setToggleAlarmAction:^{
            [self toggleAlarmActionForArrivalAndDeparture:dep];
        }];
        [row setShareAction:^{
            [self shareActionForArrivalAndDeparture:dep atIndexPath:[self indexPathForModel:dep]];
        }];

@aaronbrethorst Travis CI not successfull. is the issue related to the "Fix Travis #998" issue?

Looks good, but please take a peek at that indentation. The Travis issue is what you identified.

@aaronbrethorst I fixed the xcode indention.I'm getting the the error " Error Domain=IDETestOperationsObserverErrorDomain Code=5 "Early unexpected exit, operation never finished bootstrapping". What I found from the internt, it was due to some library not being linked to the test target correctly . Can you tell which library is not linked to test target?

Commit looks good, but the original commit message was overwritten. Can you edit your commit message? Don't worry about the Travis CI failure.

@aaronbrethorst I've changed the commit message.

Cool, thank you. Please take a peek at the preferred message format for commits for future reference: https://github.com/OneBusAway/onebusaway-iphone/blob/develop/CONTRIBUTING.md#preferred-commit-message-format


Fix International Name Support

Adding members with non-ASCII characters in their names should now be properly supported. This was just a matter of encoding names when we get them and decoding them when we need to output them. This involved a lot of minor changes, but I guess it was better than carelessly setting a new default encoding, as outlined here.
Also, name validation is less strict now; we basically just require the first name to be non-empty, inspired by this blog post.
This hasn't been extensively tested yet besides basic functionality, so I wouldn't be surprised if there's some errors. Let me know if you catch any!

Thanks @dkong1796!
Yeah, I definitely wanna migrate to Python 3 in the future, but figured this wouldn't be so bad to fix in the meantime.

I did grep -r, but I realized that there are a lot of places in the code where it's fine to pass these around without having to encode/decode them. As long as they're encoded in the objects and decoded when they need to be displayed, I think everything should be fine.
Although I did just realize that I may have missed some spots in the rotation/budget templates... gonna take a look at those in a bit!

Yeah, the budget uses arbitrary strings for payee names, cause sometimes you have to write checks to "Pasadena Archery" or whatever.


fix: value fixed decimal

Related to #847

Description
Changed value decimal places to 2
Test plan
Run server with docker compose up, go to button page, download CSV and check value column

On this branch, the CSV export isn't working on many of the buttons.
It seems like it's not working if there's too many transactions to go through.
Able to reproduce?
I'm simply testing a few of the buttons in the local dev@ account.

I was able to reproduce an error but It is related to missing price.
When there is 1 tx without price we stop generating the csv and throw error.
Testing Paybutton:
account: dev@
name: DEV
id: f03981ab-8ded-4aab-93e8-b1d38f9b219b
I received this json :
{ "message": "Missing price for transaction., txId 57f47d4b-6560-11ef-af58-0242ac120002, at<PHONE_NUMBER>" }
If we check getTransactionByPaybuttonId endpoint http://localhost:3000/paybutton/transactions/f03981ab-8ded-4aab-93e8-b1d38f9b219b with the ID of the button mentioned above we can see that tx 57f47d4b-6560-11ef-af58-0242ac120002 is not returning prices

I was able to reproduce an error but It is related to missing price.

So looks like this was related to the bug I introduced in master that would prevent the software from syncing all the prices.
It is fixed now, and the PR seems to work well.
I will approve it because it does what is supposed to do,  but first:

Shouldn't we actually use 8 decimal cases for bitcoin cash? As of now 0.003 BCH =~1USD, so small txs would basically be all shown as 0 USD.

This decimal change is only supposed to be on the 'value', not the 'amount'.

This decimal change is only supposed to be on the 'value', not the 'amount', so all good right?

Ahhh of course, approving in then.
PS: I still think we should at least put the units in the headings, like: amount (USD) and value (XEC) (or value (BCH))
a lot more clarity in exchange for a few characters —the implicit difference that exists here between the words "amount" and "value" is something that we essentially bred inside our own codebase.
And even when used/interpreted by "codebase people", as shown by earlier my misunderstanding, it still can lead to confusion.

Sure we can make a new task for that


[auto-merge] branch-22.06 to branch-22.08 [skip ci] [bot]

auto-merge triggered by github actions on branch-22.06 to create a PR keeping branch-22.08 up-to-date. If this PR is unable to be merged due to conflicts, it will remain open until manually fix.

FAILURE - Unable to auto-merge. Manual operation is required.
{'message': 'Pull Request is not mergeable', 'documentation_url': 'https://docs.github.com/rest/reference/pulls#merge-a-pull-request'}

Please use the following steps to fix the merge conflicts manually:
# Assume upstream is NVIDIA/spark-rapids remote
git fetch upstream branch-22.06 branch-22.08
git checkout -b fix-auto-merge-conflict-5699 upstream/branch-22.08
git merge upstream/branch-22.06
# Fix any merge conflicts caused by this merge
git commit -am "Merge branch-22.06 into branch-22.08"
git push <personal fork> fix-auto-merge-conflict-5699
# Open a PR targets NVIDIA/spark-rapids branch-22.08

IMPORTANT: Before merging this PR, be sure to change the merging strategy to Create a merge commit (repo admin only).
Once this PR is merged, the auto-merge PR should automatically be closed since it contains the same commit hashes

https://github.com/NVIDIA/spark-rapids/pull/5700


-fno-strict-aliasing

Our ATS 9.x builds via Automake generally built with -fno-strict-aliasing. This updates our CMake configuration to do the same. This can avoid hard to debug runtime bugs from optimization.

[approve ci autest fedora debian]

Cherry-picked to v10.0.x


Explain that rehash is now built-in to rbenv

I think this this is worth mentioning to avoid confusion such as
https://github.com/thoughtbot/laptop/pull/391

:+1:

:+1:

This would avoid confusion over "why is this repository marked DEPRECATED with no information about what the replacement is?".

@sstephenson @josh @mislav ?


inconsistent slideshow speed + starting white

          The first image fades in from a white background, also the speed of the animation changes.

(the speed issue seems to be connected to clicking)
https://user-images.githubusercontent.com/66779682/213573528-bfcfffb3-4bdf-49e4-b0c0-2649048cefff.mp4
Originally posted by @NoYavy in https://github.com/gruener-campus-malchow/gcmofficial/pull/18#discussion_r1081924635

Irgendwann gefixt, funzt jetzt jedenfalls

ne

traced the origin to the interval clear, fixin it in a couple days

seems to be fixed please reopen if persists, only tested on 2 machines so far


Add daychange command to padglobal

Resolves #647. Does not mention DST yet, but may not need to as it uses UTC instead of PST.

It's currently **not** Daylight Savings Time in North America! This lasts for 65 more days.
Reset (when dungeons and events expire) is in x time (12:00am PST)
Reset (when dungeons and events expire) is in x time (4:00am PST)

It currently **is** Daylight Savings Time in North America! This lasts for 65 more days.
Reset (when dungeons and events expire) is in x time (1:00am PDT)
Reset (when dungeons and events expire) is in x time (5:00am PDT)

It's currently **not** Daylight Savings Time in North America! This lasts for 65 more days.
Reset (when dungeons and events expire) is in x time (12:00am PST)
Reset (when dungeons and events expire) is in x time (4:00am PST)

It currently **is** Daylight Savings Time in North America! This lasts for 65 more days.
Reset (when dungeons and events expire) is in x time (1:00am PDT)
Reset (when dungeons and events expire) is in x time (5:00am PDT)


Add a scroll to top shortcut !

Currently , there is no scroll to top shortcut . Add it on the above page

I would like to work on this. Kindly assign me if available.

Please go through #8


Autocomplete not working for "set_shader_param"/"get_shader_param"

Godot version
v3.5.beta1.official [b9b23d222]
System information
w10 64
Issue description
Autocompletion doesn't work for "set_shader_param" and not for "get_shader_param" and maybe not for other methods either.
This can lead to thinking "am I wrong?" or "am I typing it wrong?" or "set_shader_param is not a material method?".

Steps to reproduce
write "set_shader_param"
Minimal reproduction project
...

Because $whatever_node.material returns the base Material class, not the ShaderMaterial

Well... how do you have to write it so that it appears in the autocomplete?
I'm trying but I don't see the way.

Workaround is (material as ShaderMaterial)

Ok, I guess this is the "correct" way.


Fix bug when switching between layers.

There are a couple of bugs when switching between layers:
Bug 1

Create new image
Click add new layer
Click Background layer
Click Layer 2

Bug 2

Create new image
Click add new layer
Click checkbox to make Layer 2 hidden
Click checkbox to make Background layer hidden
Click checkbox to make Layer 2 visible

Both result in the layer list being duplicated.  Performing more switching will cause the list to get even more out of sync until you get a native error about accessing protected memory.
This seems to be caused by a change in TreeStore.Clear () in GTK3.  Previously calling this did not trigger TreeView.CursorChanged, but now it does.  This causes us to call Reset () multiple times recursively which wreaks havoc.
The fix is to create an updating_model flag to prevent us from running the update code multiple times from a single click.

Nice catch, thank you for the fix!


Find in home page should point to Registry Search, Also all the links on home page should be correct

Here is the PDF for the QI criteria
Criteria for Quality Improvement Projects (002).pdf

@DevMattM Thanks for converting this, I will add to the landing page.

Duplicate issue #224

#267


Variance is not being reset when new items are being selected

Current behavior

https://jam.dev/c/a3a062be-ce01-4f93-b919-2f2920da3ab6

Will be fixed in https://github.com/hotwax/inventory-count/pull/366

This issue is replicating again in the latest release i.e. v2.0.0. Hence, reopening this issue.
https://jam.dev/c/3637eef9-9e81-42a1-8bd4-910df812b315


sql/parser: Added current_schema keyword support

PR: https://github.com/cockroachdb/cockroach/pull/16921

Regenerated the 1.1 full grammar, which added this keyword.


unable to split the folder id value in jinja file

Hi Team,
I'm doing project factory automation in jinja file, However i faced issue while split the folder name and folder id from folder api. Please find the below steps for more details
step1: I have created a folder using below code and it is successfully created.
resources:

name: {{ properties['foldername'] }}
type: gcp-types/cloudresourcemanager-v2:folders
properties:
{% if properties['isorg'] == "yes" %}
parent: organizations/{{ properties['orgid'] }}
{% else %}
parent: folders/{{ properties['parentfolderid'] }}
{% endif %}
displayName: {{ properties['folderdisplayName'] }}

Step 2: once created i need the folder id to parse the value in parent->id of project.

name: {{ properties['projectid'] }}
type: gcp-types/cloudresourcemanager-v1:projects
properties:
projectId: {{ properties['projectid'] }}
parent:
type: folders
id: $(ref.{{ properties['foldername'] }}.name).split('/')[1]

I'm getting error that above split command, it is not working. Kindly assist me how to split the above value>
Example:
Output of Inline reference value for newly created folder name is : core/123456789
I want to split foldername is core and folderid is 12345678 with above jinja file.

Hi,
I've a similar issue with another resource : iap.projects.brands.identityAwareProxyClients.create (I've created a deploymentmanager.v2beta.typeProvider to use it)
It create  a IAP Client and doesn't return name with the oauth2ClientId, but with relative path: projects//brands//identityAwareProxyClients/
  - name: iap-v1
    type: deploymentmanager.v2beta.typeProvider
    properties:
      descriptorUrl: 'https://iap.googleapis.com/$discovery/rest?version=v1'
      options:
        inputMappings:
          - fieldName: Authorization
            location: HEADER
            value: >
              $.concat("Bearer ", $.googleOauth2AccessToken())

  - name: brand
    action: {{ env["project"] }}/iap-v1:iap.projects.brands.create
    properties:
      parent: projects/{{ env['project_number'] }}
      supportEmail<EMAIL_ADDRESS>      applicationTitle: "Brand"
    metadata:
      dependsOn:
      - iap-v1
      runtimePolicy:
      - CREATE

  - name: iap-client
    action: {{ env["project"] }}/iap-v1:iap.projects.brands.identityAwareProxyClients.create
    properties:
      parent: projects/{{ env['project_number'] }}/brands/{{ env['project_number'] }}
      displayName: "IAP"
    metadata:
      dependsOn:
      - iap-v1
      - brand
      runtimePolicy:
      - CREATE

I want to reference oauth2ClientId in a BackendService, but I don't know how to reference only the last field of the output value
  - name: global-https-backend-service
    type: backend_service.py
    properties:
      protocol: HTTPS
      loadBalancingScheme: EXTERNAL
      sessionAffinity: CLIENT_IP
      backends:
        - group: $(ref.uig.selfLink)
          balancingMode: UTILIZATION
          maxUtilization: 0.8
      healthChecks:
        - $(ref.https-healthcheck.selfLink)
      portName: https
      iap:
        enabled: true
        oauth2ClientId: $(ref.iap-client.name)   # HERE THE ISSUE, I NEED SOMETHING LIKE .SPLIT("/")[-1]
        oauth2ClientSecret: $(ref.iap-client.secret)

Thanks


Animated sprite problem

This is related to issue #29 - Problem with animated sprites.
I'm redesigning http://ieee-qld.org to be responsive, using flexbox as the basis for the redesign. The original website shows how the sprite is supposed to work.
The code below better reflects the way I want to use animated sprites than the code in issue #29. The sprite is supposed to take up the first 150 pixels of the host flex item. Instead it occupies the entire width of the flex item, ie, 20% of the screen width. The rendering error appears in Win7 with Chrome, FF, Opera and IE11.
I have set up a demo of the problem at http://ieee-qld.org/sprite.
<!doctype html>

<meta charset="UTF-8" />
<meta name="viewport" content="width=device-width, initial-scale=1" />
<title>flex problem</title>

IEEE Queensland Section

IEEE is the world's largest
professional association for the
advancement of technology

These type of questions are better suited for coding forums such as StackOverflow. The fact that you have interoperable behavior between all of the browsers means that this is not a flexbox bug that needs to be documented.

@gregwhitworth StackOverflow doesn't always have an answer, especially where sprites are involved. See http://stackoverflow.com/questions/29274049/how-to-get-a-fade-transition-on-a-simple-sprite-carousel.
It seems to me that it is a flexbox problem, where all the browsers have made the same mistake, ie, allowing the sprite to occupy the entire flex item's allocated space, ignoring the width specified for the sprite. I'm looking for a workaround, but the only one I can think of involves not using a sprite.

@gregwhitworth I have created a different version of the sprite animation, taking out all flexbox references. The sprite works perfectly when not in a flexbox context, ergo, the problem must be flexbox related. Please see http://ieee-qld.org/grid.

Ok, a couple things, I highly recommend sticking to the flex shorthand (as noted in the spec) as flex has quite a complex layout algo and it can be hard to get the exact desired result unless you know exactly which dials to turn. Additionally, removing the box class from your sprite fixes this issue in IE 11, Project Spartan, Chrome and Firefox. This is because you had it set to flex grow and flex shrink via the box class having a flex: 1. Basically, when flexing, if you have flex grow your width initially is used as a min-width, if you don't have flex shrink then it would be a max-width, but you were telling the item to be included in the flexing of the items; thus it flexing beyond the 150px and if you resize your browser very small it will get smaller than 150px. I have updated your code to include a distinct no grow, no shrink, and set flex-basis for the sprite: http://jsbin.com/wapotejoko/1/edit
Let me know if you have any questions.

@gregwhitworth Thanks. I see how you've made it work. It's not exactly intuitive, and as you said one needs to know which dials to turn. You've taught me a useful trick not available in any tutorials. Thanks again.

No problem, and I agree it's not the most intuitive, mainly because flex is trying to set both the width or height (depending on flex-direction) but web developers want to initially treat them like regular blocks. So you hit this issue where you're setting a width, but then the browser sees that you want it to flex, so one of those has to fall out, either the width being both the min and max (which would be the case for blocks) or not allow it to flex. Ultimately, I'm glad you got it working. Additionally, I recommend posting these types of questions to stackoverflow and then ping me (twitter: @gregwhitworth | email<EMAIL_ADDRESS>and I can work it out there (also, if I can't get to it right away someone else may be able to help out). Thanks, and have a great weekend!!

Thanks for the explanation. In my view sprites 'break' the flexbox
model. Here's why: To get flexbox to display the sprite one frame at a
time the sprite has to be contained in a space exactly equal to its
width. The other flex items in the flex container obey the proportional
spacing rule, but only for the remaining space. This doesn't achieve the
balanced layout http://www.ieee-qld.org/ I want to give the
item.
I can move the sprite further to the right with CSS, covering part of
the middle flex item, but that disturbs the flexiness of the . I
will experiment to see if I can place the sprite properly in its flex
item using JavaScript. If flexbox treated sprites as just another image
with a fixed width and height, as it has done with the logo in the third
 flex item, there wouldn't be a problem.

Solved it! I created a  wrapper around the sprite. Now
flexbox distributes the three flex items proportionately in the
container and the sprite shows one frame at a time. I've used CSS to
right-justify the sprite in the flex item.

IEEE Queensland

Section

IEEE is the world's largestprofessional
association for theadvancement of technology

 .sprite {width:140px;
               height:130px;
               background-image:url("assets/images/sprite.jpg");
               position:relative;
               left:37%;
               flex: 0 0 140px;
               -webkit-animation: play 30s steps(6) infinite;
               animation: play 30s steps(6) infinite;}

@gregwhitworth You can see the results at http://ieee-qld.org/grid. I haven't had time to make  responsive, but the other parts are, using a combination of flexbox and JavaScript.


[Statistic] Adjust for LLVM r374490

Use appropriate member functions instead of accessing Value directly.

CC @adrian-prantl @compnerd @dcci
I'll need someone to commit this for me.

@swift-ci please test and merge

CC: @shahmishal - I figure that you might want to be aware since you just setup the master-rebase scheme

@compnerd @shahmishal what's the master-rebase scheme? Is there any documentation or forum post I can read?


struct forward declarations  and non-inline method definitions

Hi,
I am moving code from a DXC compiled base to Slang. I ran into the following problems making porting the shader code an issue for code readability.

struct forward declarations not working
non-inline method definitions are not working

The following is valid DXC code that does not compile using Slang:
struct PSInput
{
	float4 color : COLOR;
};

class SDR;

class SDR
{
    float4 do_smth();
};

float4 SDR::do_smth()
{
    return float4(1.0, 0.0, 1.0, 1.0);
}

float4 PSMain(PSInput input) : SV_TARGET
{
    const SDR sdr;
	return sdr.do_smth();//input.color;
}

Is this by design? With a larger shader code base I find it very important to have a good overview of struct interfaces and data members and the implementation details separate.

Hi @chrislu, thank you for your feedback! Slang supports out of order define-use so declarations aren't needed. We agree that such ability would allow codebases to present a cleaner outline, and the feature may not be difficult to add. However we are currently resource limited and most likely are not going to have the cycles to add this support. We hope that the document outline feature in our VSCode extension can help with that.

We totally hear that this is convenient, however for now we're prioritizing keeping our backlog tidy and things we're going to get to, and our priorities for now are hardening the language. So, this one doesn't quite make the cut for now --- if we encounter more users who need this , there's a good possibility down the road that we might reopen, so again, closing more to keep our backlog tidy.
Thanks for filing this bug!


Statefull -> Stateful

@andresnava : Thanks for your contribution to the Azure documentation! The author, @dariagrigoriu, has been notified to review your proposed change.


Adding init script for matchbox

Added a rapid implementation of an init script useful for some systems that does not provide systemd yet.

I don't think this can stand alone, its needs:

Add a section to the deployment.md guide (also mention customization and logging)
Define at least one pre-systemd era setup this has been used on
Init script assumes --address <IP_ADDRESS>:8080 should be used

There is the additional question of how this will be maintained over time, it seems likely to go unsupported.


Datastore Helpers

This change intends to make it easier to write (and maintain) Datastore code using gcloud. It adds three small convenience methods that will save on characters needed to be typed and read.
Before:
dataset = gcloud.datastore

# Create a new task to demo datastore
demo_task = Gcloud::Datastore::Entity.new
demo_task.key = Gcloud::Datastore::Key.new "Task", "datastore-demo"
demo_task["description"] = "Demonstrate Datastore functionality"
demo_task["completed"] = false

# Save the new task
dataset.save demo_task

After:
dataset = gcloud.datastore

# Create a new task to demo datastore
demo_task = dataset.entity "Task", "datastore-demo" do |t|
  t["description"] = "Demonstrate Datastore functionality"
  t["completed"] = false
end

# Save the new task
dataset.save demo_task

Before:
# Create a query for all completed tasks
query = Gcloud::Datastore::Query.new.
  kind("Task").
  where("completed", "=", true)

# Run the query
completed_tasks = dataset.run query

After:
# Create a query for all completed tasks
query = dataset.query("Task").
  where("completed", "=", true)

# Run the query
completed_tasks = dataset.run query

I think these added methods must set a project record for docs to code ratio. Nice work. Everything is perfectly clear.

Well there is almost no implementation to them. They are just shortcuts. All I can do is add documentation! :)


Identify glottolog mapping of Binford B62.

B62 ("Bambote of Lake Tanganyika") is currently mapped to a placeholder glottocode [tempxx04] as we do not know what it should be:
"Note that Terashima (1980), the main source of cultural data for this group, says further research is needed to confirm whether Bambote speak language of a neighbouring group, or their own language. Both the Bambote and their neighbours say the group speaks a distinct language, ""Kimbote."" Rebecca Grollemund said this society should NOT be matched to iso ""tap""; she said ""Bambote is a pygmy language related to D20 languages"""

Wikipedia says that [holo1240] is the best candidate: here.


Bias is ignored while loading models

During loading models, bias is ignored and set to zero due to the third parameter here is true. Is that normal?
https://github.com/noon99jaki/pra/blob/cf02b24a6a50347cf59bbb307941c429a946f8ef/edu/cmu/lti/algorithm/optimization/AModel.java#L43
Thanks!

Yeah, it does seems a bit counter intuitive, but since it is a ranking model the bias term should not matter. Might be an issue if we try to do classification with it.

Yeah, it does seems a bit counter intuitive, but since it is a ranking
model the bias term should not matter. Might be an issue if we try to do
classification with it.
Ni
On Sun, Jun 7, 2020 at 5:53 AM whwang1996<EMAIL_ADDRESS>wrote:

During prediction or test, bias is not read and set to zero because the
third parameter here is true. Is that normal?
https://github.com/noon99jaki/pra/blob/cf02b24a6a50347cf59bbb307941c429a946f8ef/edu/cmu/lti/algorithm/optimization/AModel.java#L43
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/noon99jaki/pra/issues/7, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSZNYYMWWOEIYV4R33WAMTRVOEWBANCNFSM4NXGHUNA
.

I see. So if I want to get the confidence that a triple exists, should I add the bias term to the score? For example, the predictions score of (h,r,t) is 3, and the bias term is -1. So the confidence of (h,r,t) exists is sigmoid(3-1)=0.881. Is that correct?

yeah that work. However, the estimated probability is based on the distribution of the training set which might differ than your test set. So you might anyway need to calibrate your probabilities like in
https://dl.acm.org/doi/abs/10.1145/2623330.2623623


backport to python 3.5.1, without hiredis

in order to install this lib on an embedded system with python 3.5.1 and without hiredis, I had to make those patches.

There is a little bit more than just a create_future (e864842fdb52feaa49718747648dc3cf758b54d4).
I don't really want to bring those wrappers back to master but I want to help you to use aioredis on embeded device.
Currently I see two major issues with this backport:

running tests for Python 3.5.1 — travis-ci now has Python 3.5.6, so we would need some special setup to run tests.
the 3.5.1 support itself — this would mean that we won't be able to use new language features
to implement library features in easy way.

I can imagine this backport living in a separate branch and being synced from master. However this is just a quick thought and I need to think it over because in this case the support (features backporting and releases) can become really hard.

Thank you Alexey for your answer.
In effect, I understand that remaining constrained to the support of 3.5.1 would prevent to use new language features and this is too much.
A separate fork may be a good way to go. I can imagine that It will survive for a shorter period than the main project, i.e. soon or late even peculiar/embedded environments will be upgraded.
Anyway, with those patches I can do what I need in the short/medium term, So take all your time to think it over.
Thanks a lot for your time, Giovanni.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@giovanni-angeli -
Thank you for your submission - however we have decided to deprecate support for Python3.5 moving forward.


Add support for serializing HTML tags (web components) in simple Markdown (.md) files (e.g. remark-html plugin)

What problem will this feature address?
I want to use Fumadocs with some simple Markdown (.md) files that includes native web components (instead of MDX) and I use to use rendering it with markdown-it parser (and including a tag script in some layout) in another technology, but...

When I try to use html tags with Fumadocs (in .md files) this components are Not rendering any more.

Describe the solution you'd like
Just the capacity or posibility to use html tags with simple Markdown (.md) files similar with markdown-it (e.g. remark-html plugin)
Describe alternatives you've considered
Migrate my .md files to MDX and migrate native web components (made with Lit) but it sounds crazy and I want to adopt Fumadocs (now that I know Fumadocs I love it)
Additional context
Project with web components: https://github.com/kaesar/onmind-cui

I'm not planning to add Web Components support, you should use it with JSX or a React component wrapping a web component, or see https://react.dev/reference/react-dom/components#custom-html-elements


Lebovits/refactor etl pipeline

Removing Dockerfile-pg from the PR because it's outside the scope, but I was using this to get around postgis versioning:
FROM postgres:16-bullseye

LABEL maintainer="PostGIS Project - https://postgis.net" \
      org.opencontainers.image.description="PostGIS with PostgreSQL 16" \
      org.opencontainers.image.source="https://github.com/postgis/docker-postgis"

ENV POSTGIS_MAJOR 3

# Install dependencies and PostGIS
RUN apt-get update \
    && apt-get install -y --no-install-recommends \
           gnupg \
           postgresql-common \
           apt-transport-https \
           lsb-release \
           wget \
           ca-certificates \
    && yes | /usr/share/postgresql-common/pgdg/apt.postgresql.org.sh \
    && apt-get update \
    && apt-get install -y --no-install-recommends \
           postgresql-16-postgis-3 \
           postgresql-16-postgis-3-scripts \
           postgresql-client-16 \
    && rm -rf /var/lib/apt/lists/*

RUN mkdir -p /docker-entrypoint-initdb.d

Relevant for #997


[stable/jenkins] helm template - error calling semverCompare: Invalid Semantic Version

Describe the bug
when running the below commands we see the following errors: error calling semverCompare: Invalid Semantic Version
Error: render error in "jenkins/templates/jenkins-master-deployment.yaml": template: jenkins/templates/jenkins-master-deployment.yaml:42:28: executing "jenkins/templates/jenkins-master-deployment.yaml" at <include (print $.Template.BasePath "/config.yaml") .>: error calling include: template: jenkins/templates/config.yaml:139:7: executing "jenkins/templates/config.yaml" at <semverCompare ">=1.20.2" (include "jenkins.kubernetes-version" .)>: error calling semverCompare: Invalid Semantic Version

these are the lines which are causing the error:
## local file path: out/jenkins/templates/config.yaml

{{- if semverCompare ">=1.20.2" (include "jenkins.kubernetes-version" . ) }}
          <directConnection>false</directConnection>
{{- end }}

if i comment them out it works without errors.
Version of Helm and Kubernetes:
$ helm version
Client: &version.Version{SemVer:"v2.15.2", GitCommit:"8dce272473e5f2a7bf58ce79bb5c3691db54c96b", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.15.2", GitCommit:"8dce272473e5f2a7bf58ce79bb5c3691db54c96b", GitTreeState:"clean"}

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.2", GitCommit:"f6278300bebbb750328ac16ee6dd3aa7d3549568", GitTreeState:"clean", BuildDate:"2019-08-05T16:54:35Z", GoVersion:"go1.12.7", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.5", GitCommit:"20c265fef0741dd71a66480e35bd69f18351daea", GitTreeState:"clean", BuildDate:"2019-10-15T19:07:57Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"linux/amd64"}

Which chart:
stable/jenkins
How to reproduce it (as minimally and precisely as possible):
$ helm fetch stable/jenkins --untar --untardir=./out
$ helm template ./out/jenkins --values utils/infrastructure-cluster/helm/jenkins-values.yaml >> out.yaml
Error: render error in "jenkins/templates/jenkins-master-deployment.yaml": template: jenkins/templates/jenkins-master-deployment.yaml:42:28: executing "jenkins/templates/jenkins-master-deployment.yaml" at <include (print $.Template.BasePath "/config.yaml") .>: error calling include: template: jenkins/templates/config.yaml:139:7: executing "jenkins/templates/config.yaml" at <semverCompare ">=1.20.2" (include "jenkins.kubernetes-version" .)>: error calling semverCompare: Invalid Semantic Version

anyone?

Looks like this commit introduced the regression. It's preventing me from using using enabled JCasC. I'll post a minimal reproducible example later.
https://github.com/helm/charts/commit/aa409f8a5257d1851aea363d5f6f5fdcf8777ad3

cc @bergemalm - have a look.

@HerrmannHinz Maybe you're using latest as version for the kubernetes plugin?
Anyway, they decided to keep the new direct connection setting for the kubernetes plugin to false instead of true so I made a new pr removing what was added to fix that issue here:
https://github.com/helm/charts/pull/18490
Still waiting for response on that one.

@bergemalm this is my values file:
master:
  image: "jenkins/jenkins"
  tag: "2.202-alpine"
  imagePullPolicy: "IfNotPresent"
  component: "jenkins"
  useSecurity: false
  resources:
    requests:
      cpu: "1000m"
      memory: "4Gi"
    limits:
      cpu: "1000m"
      memory: "4Gi"
  initContainerEnv:
    - name: JENKINS_INCREMENTALS_REPO_MIRROR
      value: https://repo.jenkins-ci.org/incrementals
  containerEnv:
    - name: JENKINS_SLAVE_AGENT_PORT
      value: "50000"
    - name: JENKINS_INCREMENTALS_REPO_MIRROR
      value: https://repo.jenkins-ci.org/incrementals
    # - name: CASC_JENKINS_CONFIG
    #   value: /var/jenkins_casc/casc_configs
  # set timezone and stuff / change default Content Security Policy
  javaOpts: >
    -Dorg.apache.commons.jelly.tags.fmt.timeZone=Europe/Berlin
    -Duser.timezone=Europe/Berlin
    -Dhudson.slaves.ChannelPinger.pingIntervalSeconds=60
    -Dhudson.model.DirectoryBrowserSupport.CSP=""

# Set min/max heap here if needed with:
# JavaOpts: "-Xms512m -Xmx512m"
# Optionally configure a JMX port
# requires additional JavaOpts, ie
# JavaOpts: >
#   -Dcom.sun.management.jmxremote.port=4000
#   -Dcom.sun.management.jmxremote.authenticsate=false
#   -Dcom.sun.management.jmxremote.ssl=false
# JMXPort: 4000
# List of plugins to be install during Jenkins master start
  installPlugins:
      - git:3.12.1
      - slack:2.34
      - configuration-as-code:1.32

  servicePort: 8080
  serviceType: ClusterIP
  serviceAnnotations: {}
  containerPort: 8080
  slaveListenerPort: 50000

  ingress:
    hostName: jenkins.infrastructure.caas.xxxx.xxxxx.io
    enabled: true
    annotations:
      zalando.org/aws-load-balancer-scheme: internal

persistence:
  enabled: false
  ## A manually managed Persistent Volume and Claim
  ## Requires Persistence.Enabled: true
  ## If defined, PVC must be created manually before volume will be bound
  existingClaim: jenkins-non-prod

  ## mount in the casc yaml from a configmap
  # volumes:
  # - name: casc-config
  #   configMap:
  #     name: jenkins-casc-config
  # mounts:
  # - name: casc-config
  #   mountPath: /var/jenkins_casc/casc_configs
  #   readOnly: true

no kubernetes plugin at all. oO
the idea was to install all other necessary plugins via JCasc.
i'll keep an eye on your PR. thanks for the heads up.

https://github.com/helm/charts/pull/18490 has been merged. Should solve this issue.
@HerrmannHinz

@bergemalm  works now - thanks!


Add an API endpoint to get static search filter definitions

Purpose
We're refactoring our course search to make it more modular. One key part of this is to allow components to use search/autosuggest based on filters without making full course search requests first.
Proposal

[x] rework filter definitions to add a method that returns the static definition for a filter
[x] all filter definitions have a base_path property
[x] add an API route with the static filter definitions

While doing this, we noticed a missing trailing slack on the bootstrap-elasticsearch API url, and decided to add it.

@sampaccoud done! I ended up just adding cache to the filter definitions view


WSO2 APIM do not validate the "Origin" header in WS connections

Description:
The Origin header prevents another website from exposing the websocket-service to their users. However, this is not validated by WSO2 API Manager 3.2.0 WS connections (in REST connections, Access-Control-Allow-Origin header is validated)
Steps to reproduce:
Currently not available in the WSO2 API Manager 3.2.0 product
Affected Product Version:
WSO2 API Manager 3.2.0
Environment details (with versions):

OS: Any
Client: <Censored>
Env (Docker/K8s): Any

Optional Fields
Related Issues:

APIM Internal ticket: https://support-internal.wso2.com/jira/browse/APIMINTERNAL-1659

Suggested Labels:
APIM320, security, patch, critical, P2
Suggested Assignees:
N/A

APIM Patch issue: https://support-internal.wso2.com/jira/browse/APIMINTERNAL-1670
APIM GIT issue: https://github.com/wso2/product-apim/issues/12414
APIM Internal Issue: https://support-internal.wso2.com/jira/browse/APIMINTERNAL-1659
CS ticket: <Censored>


How to change charset in main function, when insert shp

I am developing a nifi processor that uses geowave to insert shp data into an accumulo. When I insert shp using processor, fields are garbled.

I need to create a nar file using "mvn clean install" when I create an nifi processor. Is there any way to specify the GEOWAVE_CHAR_SET of the StringUtils class in the main function?

the best practice would be to run each JVM that needs to access geowave (such as NiFi) with the Java commandline option -Dgeowave.charset=<my charset> . In lieu of that you can programmatically set it as early as possible in the JVM startup using System.setProperty("geowave.charset","<my charset>"); but you just really should make sure it happens first thing or there could be a race condition within GeoWave. This is why its simply best practice to use a commandline arg.

and jvm of the geoserver also to configured "-Dgeowave.charset="

re: Thank you!! so just set -Dgeowave.charset= before mvn clean install??

not~ only when you execute "java -cp ...." add -Dgeowave.charset=charset
java -Dgeowave.charset=charset   -cp ....

Then I need to create a nar file in a different way (java-cp ....) than mvn clean install. That case, I can use -Dgeowave.charset = charset?

nope~ after you compile,
java -Dgeowave.charset=charset -cp $GEOWAVE_HOME/deploy/target/geowave-deploy-1.0.0-SNAPSHOT-tools.jar org.locationtech.geowave.core.cli.GeoWaveMain

OK thanks!! I will try it

I still have a problem. I set charset with -Dgeowave.charset=charset, but fields are still garbled. Do I set collectly?

Otherwise how I set this code in commandline?

Why? EPSG is coordinate, not charset And How and where to add '-Dgeowave.charset=EPSG:2097' in geoserver? Please let me know :)
then What about in java? just only need to type in 'System.setProperty("geowave.charset","");' you said before? that is all for changing Charset?

sorry, i spell error, alias geowave="java -Dgeowave.charset=cp949 -cp /usr/local/.....GeoWaveMain"

I haven't seen any activity on this in several months.  Thanks for the assist @scially. I assume this is resolved now?


Whitelist integrations

Init now creates a .sentinel.json file which contains some project specific config which is used by the cli.
Allow the user to specify which integrations they are using so that we only start containers that will be used.
e.g. Snyk package scanning may only require "node", "docker", "snyk.io" integrations
Removes the BYO container arguments (--yaml) and move that config into the new .sentinel.json file.
This simplifies forwarding arguments to the cucumber/docker-compose cli tools.

"Allow the user to specify which integrations they are using so that we only start containers that will be used."
YASS FINALLY! Favorite feature from this PR.
I will have a go and update the readme doc accordingly :)


image-sets: fixes image-sets-view updated_at.

image-sets-view item updated_at should correspond to the latest image updated_at.
Fixes THEEDGE-1553
Type of change
What is it?

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[X] Tests update
[ ] Refactor

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] I run make pre-commit to check fmt/vet/lint/test-no-fdo

Codecov Report
Base: 44.98% // Head: 44.97% // Decreases project coverage by -0.00% :warning:

Coverage data is based on head (fe60f70) compared to base (21a1357).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1751      +/-   ##
==========================================
- Coverage   44.98%   44.97%   -0.01%
==========================================
  Files          70       70
  Lines        9811     9810       -1
==========================================
- Hits         4413     4412       -1
  Misses       4967     4967
  Partials      431      431

Impacted Files
Coverage Δ

pkg/services/imagesets.go
53.19% <100.00%> (+0.73%)
:arrow_up:

pkg/services/devices.go
50.10% <0.00%> (-0.33%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Post_PR657: Add workaround comment

Add a comment following workaround rules for tracking the hotfix (PR #657) of issue #659

@zjing14 Please let me know if you don't have time for this, and I will do.


No filter in multiple select

  Supported languages
  <ng-select [options]="locales"
             [multiple]="true"
             (selected)="langSelectedPushToForm($event)"
             (deselected)="langDeselectedPopFromForm($event)"
             placeholder="Choose the languages">
  </ng-select>

The above code produce a multiple select but no filter:

From the documentation/examples I thought it was possible: there is a point "No filter multiple select" which explains how to hide the filter on multiple select, so I derived that by default multiple selects have the filter. Is there something I am missing?

I proposed the following modification to solve this issue: #164

Your screenshot shows a cursor before the 'Choose the languages' placeholder. This is the filter for the multi-select. Just start typing and the list will be filtered.
The multi-select filter is located inline after the last selected option. The single-select filter is located underneath the currently selected option (so that you can still see what the currently selected option is).


Builder review request

@dtolnay I think I got it! Let me know what you think. Thanks again!

Good feedback, I definitely thought about those things but I got lazy. I'll close it here instead of addressing them as it's not proc-macro specific which was the original goal. Thanks!


🛠 Tooling: Migration test failures on main from version mismatch

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
#1803 added the "description" line in package.json to script/__snapshots__/migrate-test-e2e.ts.snap. Unfortunately, that also added in the "version" line, which is failing end-to-end tests.
Example from https://github.com/JoshuaKGoldberg/create-typescript-app/actions/runs/12498490322/job/34890238040?pr=1810:
@@ -1,11 +1,11 @@
  "--- a/package.json
  +++ b/package.json
  @@ ... @@
   {
   	"name": "create-typescript-app",
-  	"version": "1.78.0",
+  	"version": "1.79.0",
  -	"description": "Quickstart-friendly TypeScript template with comprehensive, configurable, opinionated tooling. ❤🔥",
  +	"description": "A very lovely package. Hooray!",

Additional Info
~For now, I think make the description-defaulting logic added in #1804 just ignore this repo...~ Ah, but that's not yet known at the time... hmm..
💖

Well, for now I fixed this in c047820e2. Hoping to do away with this system soon anyway.


Windows10 version

Minecraft win 10 version generates a mcworld file which when decompressed with winrar give us level.dat and other stuff. level.dat can't be opened by your tool, it fails to open.
Here is my world for example if u wish to try:
http://www.mediafire.com/download/k6uy4fjvje89mcd/Our+World.mcworld

Or alternately you can play the multi-platform Java version of minecraft which IS supported by this tool.

I agree with @TheWebs I would love to see this application support the "pocket edition" saves

Bedrock Edition is not supported, as the format is different. Maybe you could look for a different tool that can modify Bedrock Edition


Use kueue.x-k8s.io/resource-in-use finalizer for Topology

What happened:
Topology can be deleted easily at any time, even when in use.
What you expected to happen:
There should be some guard before accidental deletion, such as we have for ResourceFlavor - the kueue.x-k8s.io/resource-in-use finalizer.
How to reproduce it (as minimally and precisely as possible):
Create the structure: LQ, CQ, RF, Topology.
Now, the Topology can be deleted.
Anything else we need to know?:
Here is the analogous code handing the finalizer for ResourceFlavors: https://github.com/kubernetes-sigs/kueue/blob/e544dc8a447abfafb471886f9390af0a303b24d5/pkg/controller/core/resourceflavor_controller.go#L83-L108

cc @PBundyra @mbobrovskyi @tenzen-y

cc @mwysokin

/assign


test(blockifier): storage changes after inner revert

This change is

#2910  👈 (View in Graphite)
main

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


Automatically fetch latest Fuel deps when initializing manifest

Related: #601
Context
Running forc init creates a default Cargo manifest file that includes some Fuel dependencies for testing purposes.
https://github.com/FuelLabs/sway/blob/7b51e653177bdc58bed454447c8dc43d9a4147f6/forc/src/utils/defaults.rs#L35-L42
Unfortunately, this 1) must be manually updated and 2) cannot be updated until a new version of the SDK is released, which first requires a new version of certain sway-* crates to be released, i.e. a circular dependency.
Proposal

If online, the latest tag of the Fuel dependencies is fetched from their respective repositories and used.
If offline, a default Cargo manifest file is generated with whatever hard-coded Fuel dependency versions are in defaults.rs.

Considerations
The latest version of Fuel dependencies is not guaranteed to be compatible with the version of forc that happens to be used; indeed, if a user downloads an older version of forc then it's almost certain to not be compatible with the latest version of fuel-core.
This might be indicative that what generates the Cargo manifest file shouldn't in fact be forc, but should maybe be a separate binary installed as a plugin that's distributed via fuels-rs for example.

cc @mitchmindtree @digorithm

Tricky issue. Blindly pulling the latest tags will definitely break things on a user's project, especially for now that we're pushing lots of breaking changes. Plus the considerations you mentioned.

This might be indicative that what generates the Cargo manifest file shouldn't in fact be forc, but should maybe be a separate binary installed as a plugin that's distributed via fuels-rs for example.

I like this idea, though we have to be careful when introducing more complexity to the toolchain and to the users 🤔

Circular dependencies between Sway and fuels-rs have been alleviated, and fuels-rs will export everything as a single crate so this is less relevant.


latest geospatial data product for renewables doesn't contain any solar farms

I checked the file and it only contains nan and wind_turbines for the category. I'd expect it to have all detected solar farms to-date that are reconfirmed by the model according to this description from the data readme

For example, 2023-01.geojson contains wind turbines and solar farms that we believe are present as of January 2023. Replace YYYY-MM with latest to get the latest data.
https://github.com/allenai/satlas/blob/main/GeospatialDataProducts.md
import requests
import geopandas as gpd
import json
solar_farm_detections_url = 'https://pub-956f3eb0f5974f37b9228e0a62f449bf.r2.dev/outputs/renewable/latest.geojson'
response = requests.get(solar_farm_detections_url)
geojson_data = response.json()
gdf = gpd.GeoDataFrame.from_features(geojson_data['features'])
gdf['category'].unique()

array(['wind_turbine', nan], dtype=object)

It looks like the category property was missing from the GeoJSON for solar farms, thus leading to the nan that you saw.
The files have been fixed now (at least in the latest files including renewable/latest.geojson).

the category is now non-nan but the score is nan
solar_farms['score'].unique()
array([nan])

Score is currently only available for marine infrastructure.

Got it, why is this the case? Just curious. Are the scores not informative for solar farms because the results are not as robust as marine infra?

The scores for marine infrastructure were only added recently. The property is not documented yet at https://github.com/allenai/satlas/blob/main/GeospatialDataProducts.md. We do plan to add scores to the other outputs soon.


NonemptyString (doesn't really need a Pattern though; a refinement should be plenty. no Arbitrary construction issues here)

Closed with #158


Support configuration from gradle properties

Codecov Report

Merging #7 into master will decrease coverage by 1.3%.
The diff coverage is 56.43%.

@@             Coverage Diff              @@
##             master       #7      +/-   ##
============================================
- Coverage     68.92%   67.61%   -1.31%
- Complexity      108      128      +20
============================================
  Files            25       26       +1
  Lines           399      457      +58
  Branches         27       33       +6
============================================
+ Hits            275      309      +34
- Misses          115      138      +23
- Partials          9       10       +1

Impacted Files
Coverage Δ
Complexity Δ

...c/main/java/com/monjam/core/command/DbMigrate.java
100% <ø> (ø)
10 <0> (ø)
:arrow_down:

...va/com/monjam/core/history/DbMigrationHistory.java
100% <ø> (ø)
6 <0> (ø)
:arrow_down:

...jam-core/src/main/java/com/monjam/core/Monjam.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ore/src/main/java/com/monjam/core/api/Context.java
91.66% <ø> (ø)
5 <0> (ø)
:arrow_down:

...com/monjam/core/resolve/JavaMigrationResolver.java
92% <ø> (ø)
5 <0> (ø)
:arrow_down:

.../main/java/com/monjam/core/command/DbRollback.java
92.59% <ø> (ø)
9 <0> (ø)
:arrow_down:

...c/main/java/com/monjam/gradle/MonjamExtension.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ava/com/monjam/gradle/task/AbstractMonjamTask.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...src/main/java/com/monjam/core/command/Command.java
85.18% <60%> (-6.82%)
6 <0> (ø)

...a/com/monjam/core/configuration/Configuration.java
62.16% <62.16%> (ø)
14 <14> (?)

... and 3 more

Continue to review full report at Codecov.

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


pytest rerun is not working properly

https://github.com/ansys/pyfluent/actions/runs/9360562203/job/25766147026 - to investigate
probably specific to nightly run

The primary issue is due to stale containers - code merged to detect such incidents.


red pen skill-development/introduction.md

Used to do this sort of thing in high school and college. It was kinda fun.

These are great, thanks!


multiple dice roller app

This app allows you to select up to 8 different dice to roll at the same time. Tap on one on the spaces to select the dice (d4 -> d6 -> d8 -> d10 -> d12 -> d20 -> d% -> none), and then shake the watch or press BTN to roll.

Great - thanks!


BatchHttpRequest and google-auth

Hi,
I'm using this client library with google-auth to make calls to several Google APIs (Drive mainly), it works fine however it seems that when using a Batch request it doesn't use the credentials of each request in the batch, only the credentials from the first one.
Which means that when I impersonate two different users in two different requests in the same batch, both requests are made impersonating the first user.
For example here's how I proceed to list the files of two users:
creds_one = google.oauth2.service_account.Credentials.from_service_account_file(path_to_json, scopes=scopes<EMAIL_ADDRESS>creds_two = google.oauth2.service_account.Credentials.from_service_account_file(path_to_json, scopes=scopes<EMAIL_ADDRESS>
drive_service_one = build('drive', 'v3', credentials=creds_one)
drive_service_two = build('drive', 'v3', credentials=creds_two)

_batch = BatchHttpRequest()
_batch.add(request=drive_service_one.files().list(), request_id='u1')
_batch.add(request=drive_service_two.files().list(), request_id='u2')

_batch.execute()

What I observe in this case is that both requests return the list of files from the first user.
Am I missing something here ?
Thanks

@sbrieuc is this perhaps a duplicate of #350?

@jonparrott Well it doesn't look like it.
To me issue #350 relates to credentials having no valid access token and using oauth2client. From what I can see in my case the issue is specific with using credentials from google-auth library. With oauth2client credentials it works perfectly.
Looking a bit more in the code, when building a service from a credentials object there are two cases:
if HAS_GOOGLE_AUTH and isinstance(
            credentials, google.auth.credentials.Credentials):
        return google_auth_httplib2.AuthorizedHttp(credentials,
                                                   http=build_http())
else:
        return credentials.authorize(build_http())

1/ If the Credentials object is from oauth2client library, then the authorized http object returned is an Http() instance from httplib2 where the request method has been overridden and on which a credentials attribute is set with the actual credentials provided
def authorize(self, http):
        ...
        # The closure that will replace 'httplib2.Http.request'.
        def new_request(uri, method='GET', body=None, headers=None,
                        redirections=httplib2.DEFAULT_MAX_REDIRECTS,
                        connection_type=None):
            ...

        # Replace the request method with our own closure.
        http.request = new_request

        # Set credentials as a property of the request method.
        setattr(http.request, 'credentials', self)

        return http

2/ If the Credentials object however is from google-auth library, then the authorized http object returned is an instance of AuthorizedHttp. This object has its own request method which does not have a credentials attribute
class AuthorizedHttp(object):
    def __init__(self, credentials, http=None,
                 refresh_status_codes=transport.DEFAULT_REFRESH_STATUS_CODES,
                 max_refresh_attempts=transport.DEFAULT_MAX_REFRESH_ATTEMPTS):

        if http is None:
            http = _make_default_http()

        self.http = http
        self.credentials = credentials
        self._refresh_status_codes = refresh_status_codes
        self._max_refresh_attempts = max_refresh_attempts
        # Request instance used by internal methods (for example,
        # credentials.refresh).
        self._request = Request(self.http)

    def request(self, uri, method='GET', body=None, headers=None,
                **kwargs):
        ...

Now, when making a regular request, this works fine, as the request method from AuthorizedHttp takes care of refreshing and applying the credentials to the headers.
However In BatchHttpRequest, when serializing requests it works a bit differently from what I can see. The request method from the http object is expected to have a credentials attribute, and only in that case the credentials are applied to the headers.
if request.http is not None and hasattr(request.http.request,
        'credentials'):
      request.http.request.credentials.apply(headers)

This is where the issue comes up, as with google-auth the AuthorizedHttp object request method does not have this credentials attribute. Therefore the credentials cannot be applied to the headers and it falls back on using the credentials from the first request in the batch as we can see in the execute() method of BatchHttpRequest:
# If http is not supplied use the first valid one given in the requests.
    if http is None:
      for request_id in self._order:
        request = self._requests[request_id]
        if request is not None:
          http = request.http
          break

This is as far as I understand what's going on. Maybe I missed something but it seems to me that google-auth credentials are not fully supported in the case of batch request.

Thanks, @sbrieuc. This gives me a good idea of what's going on. It seems to me that google_auth_httplib2 should be modified to support a per-request credentials. I'll look into that.

@sbrieuc can you verify that https://github.com/google/google-api-python-client/pull/376 resolves this issue?

@jonparrott I made my comments in #376 but it looks like it will solve the issue.


net: set FD_CLOEXEC on RktLock prior to fork/exec

When launching metadata service and net plugins,
set the close-on-exec flag so the child processes
don't end up holding the lock.

Instead of double-switching the FD_CLOEXEC flag on RKT_LOCK_FD with WithCloExecRktLock everytime Rocket needs to exec something (metadata server, network plugins), can we keep the flag FD_CLOEXEC set by default and reset FD_CLOEXEC only when executing stage1 and systemd-nspawn?
In this way, the code switching the RKT_LOCK_FD will be near the execution of stage1/systemd-nspawn rather than near things it is not related to (metadata server and network plugins).

@alban I think it's a good idea. @vcaputo ?

yeah sgtm!
On Tue, Mar 17, 2015 at 7:36 AM, Alban<EMAIL_ADDRESS>wrote:

Instead of double-switching the FD_CLOEXEC flag on RKT_LOCK_FD with
WithCloExecRktLock everytime Rocket needs to exec something (metadata
server, network plugins), can we keep the flag FD_CLOEXEC set by default
and reset FD_CLOEXEC only when executing stage1 and systemd-nspawn?
In this way, the code switching the RKT_LOCK_FD will be near the execution
of stage1/systemd-nspawn rather than near things it is not related to
(metadata server and network plugins).
—
Reply to this email directly or view it on GitHub
https://github.com/coreos/rocket/pull/628#issuecomment-82387254.

">

@eyakubovich yeah that sounds better to me as well

@vcaputo Should pkg/lock just use golang's Open to get FD_CLOEXEC and then also clear it stage1/init main? Or would you prefer to keep pkg/lock unchanged?

@eyakubovich pkg/lock can set the FD_CLOEXEC, do you see any caveats?  Requiring all callers to explicitly clear FD_CLOEXEC when they want lock files to cross exec boundaries seems reasonable.

lgtm, seems like a bunch of hoop-jumping, golang....


Add binery_encoding generator

Prerequisite feature for https://github.com/dwavesystems/dimod/issues/959

Codecov Report

Merging #964 (5ad5971) into main (1149765) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #964      +/-   ##
==========================================
+ Coverage   84.01%   84.06%   +0.04%
==========================================
  Files          80       81       +1
  Lines        6883     6901      +18
==========================================
+ Hits         5783     5801      +18
  Misses       1100     1100

Impacted Files
Coverage Δ

dimod/generators/__init__.py
100.00% <100.00%> (ø)

dimod/generators/integer.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Added grid block

Besides the name of the branch, this pull request contains [all grid modules from issue #30] (https://github.com/pyturf/pyturf/issues/30)
Please make sure that:

[x] You have read CONTRIBUTING.md
[x] You Include a link to the issue this pull request is associated to.
[x] The linter has been run at the root level of the project: black .
[x] All tests are passing: python -m pytest --verbose --cov=./
[x] Your branch is up to date with master: git rebase upstream/master

For new modules, also make sure that:

[x] The new module function is exported in turf/<your-module>/__init__.py and turf/__init__.py.
[x] The new module has been added under the Available Modules section on the README.

Codecov Report

Merging #45 (d80297b) into master (5adc781) will increase coverage by 0.01%.
The diff coverage is 97.78%.

@@            Coverage Diff             @@
##           master      #45      +/-   ##
==========================================
+ Coverage   97.69%   97.70%   +0.01%
==========================================
  Files         108      123      +15
  Lines        2471     2876     +405
==========================================
+ Hits         2414     2810     +396
- Misses         57       66       +9

Flag
Coverage Δ

unittests
97.70% <97.78%> (+0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

turf/hex_grid/_hex_grid.py
94.11% <94.11%> (ø)

turf/square_grid/tests/test_square_grid.py
97.05% <97.05%> (ø)

turf/point_grid/_point_grid.py
97.14% <97.14%> (ø)

turf/rectangle_grid/_rectangle_grid.py
97.14% <97.14%> (ø)

turf/triangle_grid/_triangle_grid.py
97.95% <97.95%> (ø)

turf/__init__.py
100.00% <100.00%> (ø)

turf/distance/_distance.py
100.00% <100.00%> (ø)

turf/hex_grid/__init__.py
100.00% <100.00%> (ø)

turf/hex_grid/tests/test_hex_grid.py
100.00% <100.00%> (ø)

turf/point_grid/__init__.py
100.00% <100.00%> (ø)

... and 22 more

Continue to review full report at Codecov.

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


Django 2.0 compatibility

Tabbed admin isn't working with Django 2.0. Probably because TabbedAdmin uses django.forms.Media.add_css() and django.forms.Media.add_js() functions, which do not exist in Django 2.0.
Question - are there any plans on making TabbedAdmin compatible with Django 2.0?

Sent up a pull request to fix this issue. Let me know if you need me to do any other work or testing on it.


Fuzz test not recognized with quickstart with cmake

Hello! I am following the Quickstart with CMake guide provided here: https://github.com/google/fuzztest/blob/main/doc/quickstart-cmake.md and am having some trouble getting the simple fuzz test to be recognized.
My CMakeLists.txt looks as follows (pretty much copy paste from the quickstart guide, except for some minor directory changes since the quickstart guide specifies to create a directory called “first_fuzz”):
cmake_minimum_required(VERSION 3.19)
project(first_fuzz_project)
# GoogleTest requires at least C++17
set(CMAKE_CXX_STANDARD 17)

add_subdirectory(fuzztest)

enable_testing()

include(GoogleTest)

fuzztest_setup_fuzzing_flags()

add_executable(
 first_fuzz_test
 first_fuzz/first_fuzz_test.cc
)

# If you have other dependencies than FuzzTest, link them:
# target_link_libraries(first_fuzz_test PRIVATE other_deps)

link_fuzztest(first_fuzz_test)
gtest_discover_tests(first_fuzz_test)

In the new directory (first_fuzz) that I created, I added the test first_fuzz_test.cc file with the content specified in the quickstart guide:
 #include "fuzztest/fuzztest.h"
#include "gtest/gtest.h"

TEST(MyTestSuite, OnePlustTwoIsTwoPlusOne) {
 EXPECT_EQ(1 + 2, 2 + 1);
}

void IntegerAdditionCommutes(int a, int b) {
 EXPECT_EQ(a + b, b + a);
}
FUZZ_TEST(MyTestSuite, IntegerAdditionCommutes);

I then built the program with the specified cmake flags:
$ CC=clang CXX=clang++ cmake -DCMAKE_BUILD_TYPE=RelWithDebug -DFUZZTEST_FUZZING_MODE=on ..
$ cmake --build .

The build succeeds without any errors and I am able to run the unit tests without any issue. When trying to run the fuzz test, the binary reports containing coverage but there are no tests that are recognized:
$ ./first_fuzz_test --fuzz=MyTestSuite.IntegerAdditionCommutes

[.] Sanitizer coverage enabled. Counter map size: 35469, Cmp map size: 262144
No FUZZ_TEST matches the name: MyTestSuite.IntegerAdditionCommutes
Valid tests:
Just for documentation purposes, I am using Apple clang++ version 14.0.3 on a 2019 Intel Macbook Pro with cmake version 3.26.4.
Is there something missing in the quickstart cmake guide that I am forgetting? Any additional guidance would be helpful, thanks in advance!

I suspect #179 may be related to this issue.

I have the same Problem but I have no mac. So maybe it is not related to #179 ?
I was today so happy that I got the compilation and everything running without too much fuzz and now this happened :(


Minimum Remnant Items Spawned

Not sure if this is intentional, but I've been trying to change the spawn rates of items on early moons to be considerably rare, since the danger level of these moons is supposed to be pretty low. However, the config for minimum items spawned on a moon cant be set lower than 1, guaranteeing a dead body to spawn on each level.

To change the spawn rates relatively to the danger of the moons, its best to tweak these two config features:

Remnant items spawn modifier (Section Spawning): this is a modifier relative to the threat level of the moon, lowering this number decreases the amount of remnant items spawning on moons. The higher the risk level, the more influence the modifier will have.
Body spawn modifier per moon risk level (Section GeneralBody): This works the same way but for bodies.

Do know that a moon like Experimentation has a high level of threat and for example Vow has a very low one.
Is this what you meant?

Basically, im just trying to set this value to 0. I'm not sure if its a limitation in the code but it keeps changing to 1 every time I launch to game and check back.

It is a set limitation that I put in, I basically clamp the number between 1 and 50 to prevent it crashing the game.
I would recommend you reducing the config features that I mention to a number like 0.05 or lower.
This means that there is a chance that no items or bodies will spawn on a level.


binary/inline tensor

To remove the dependency on SD card for model parameters, we could compile weight parameter into code and flash onto the chip. We can make those parameters stored in read only flash area and create tensors using const array instead of  idxImporter. Any Ideas ?

I think we should still maintain the importer layer.
i.e. something like this, or have an importer factory or something.
class ImporterBase{
 public:
   virtual some_read_write_interface(...)
};
class ImporterIdxSD : public ImporterBase ...
class ImporterIdxConst: public ImporterBase ...

That is cool and it will minimize the complexity of idxloader now.  Just we have to consider whether we support idx format or not ? In order to minimize the code size, i would prefer to store the const weight pointer purely. How do you think ?

Performance is a major consideration for the binary/inline tensor class. It would be nice if it adapts target devices' native format.
I am trying to think of the reason for keeping the importer here. @mbartling, my guess is we can do more runtime trick this way?
Keep in mind that every tensor also inherits an init and de-init hook from the tensor-base class.

I would still sacrifice the slight read time and have the FormatReaders separate from the PlaceImporters.
For example ImporterSD<IdxReader> myReader(); or ImporterSD<WAVReader>, or ImporterSdRam<ConstReader>.
Block memory reads are way more expensive than adding one more read pointer function call, so having the additional configurability greatly simplifies the code base without any real cost. Additionally, it makes adding Exporters trivial.
@Knight-X what do you think

I like the idea of having the FormatReaders and PlaceImporters separated. We might be able to make universal tensor importers using polymorphism, Importer(Tensor* t, FmtReader* pipe)
@mbartling Can you clarify what you meant by "Block memory"? Block in the sense of a chunk of memory or busy I/O?

wave files, think of it as bmp for sound.

@neil-tan Block memory as grab adjacent bytes. It's partially file system dependent for performance

To give it a bit of context:

The Tensor class initialization

Heap allocation

Different types of tensors can be derived from the base Tensor, for example, the SDTensor.

The importers are here.

I think that clean importer interface is nice, but we need to consider whether it could reduce the sram memory usage in binary tensor. Should we just consider the interface of making binary tensor first ? Then, we could just make importer interface clean and decide what type of tensor should be supported.

Hey, just spoke with Marcus, looks like the bottomline is having the tensor-data-void-pointer be static-const. This should allow us to keep the model parameters on the flash instead of heap. Creating a new Tensor class is probably the easiest way. It will be read-only. Don't mind having the importer here as long as the interface makes sense and keeps the data in flash.

the bottomline is having the tensor-data-void-pointer be static-const.

Just to clarify, the important part is having the actual array being declared const and not just the pointer.

I have drafted binary tensor, and i will write test case which is similar to sdtensor test for it. Therefore, I need some feature to transform weights to const pointer in c++, which will be in codegen.

uTensor is a amazing job.  I want to use it to speed up my model. But I got some trouble.
How can I initialize Tensor with a vector or other something, not idx file？Just like this for
"simple_mnist":
`
Tensor *data;
data = new Tensor();
std::vector<uint32_t> tmp(784, 0);
data->init(tmp);
get_simple_mnist_ctx(ctx, data);
S_TENSOR pred_tensor = ctx.get("y_pred:0");
ctx.eval();
print_tensor<int>(pred_tensor);

`
Thank you a lot.

If you want to initialize tensor with zeros, how about:
uint32_t *w_ptr = data->write<uint32_t>(0, 0);
for (uint32_t i = 0; i < data->getSize(); ++i) {
    *(w_ptr + i) = 0;
}

Or you want sth without a loop?

done