Body Expansion Incorrectly "Decoding" Unencoded Values?

Question/possible issue here.
Attempting to do a POST with a very simple body expansion and it seems like there's an issue line with the RequestTemplate body expansion.  Specifically here:
body(urlDecode(expand(bodyTemplate, unencoded)));
It seems like urlDecoding unencoded parameters can cause issues with certain strings.  For instance, if I have a '+' character in the argument being substituted, the urlDecode would swap the '+' for ' ' (this is the case I'm running into).  I imagine you would run into issues with '%<...>' in substituted strings as well.
I can't get my head around why just using encoded, the already encoded map, here wouldn't be the right solution.  It would expand into a fully encoded string (including anything encoded in the body template) and then properly decode to the full body.
Is there a simple way to work around this during Feign building?  Swapping out the RequestTemplate for one that does the correct encoding doesn't seem very tenable -- looks like you'd have to swap all the way out in the contract and dupe a fair amount of code, since the RequestTemplate is instantiated inline with the MethodMetadata, and MethodMetadata is instantiated inline in the BaseContract.  Anything obvious I'm missing here?

I think you are suggesting a change to the implementation of RequestTemplate, particularly around encoding as they relate to http bodies. Bear in mind that feign has a single Param type for anywhere expansion could occur (including the request line, which is the normal case).
If you know a way to preserve existing functionality, and also help with the encoding woes on unencoded params by reworking RequestTemplate, go for it! We have pretty good coverage on the related area, so start with a failing test? Even if you can't put time into fixing an impl, pasting a failing test back here would help.

Failing Test, in RequestTemplateTest:
@Test
public void resolveTemplateWithBodyTemplateDoesNotDoubleDecode() {
    RequestTemplate template = new RequestTemplate().method("POST")
        .bodyTemplate(
            "%7B\"customer_name\": \"{customer_name}\", \"user_name\": \"{user_name}\", " +
                "\"password\": \"{password}\"%7D");

    template = template.resolve(
        mapOf(
            "customer_name", "netflix",
            "user_name", "denominator",
            "password", "abc+123%25d8"
        )
    );

    assertThat(template)
        .hasBody(
            "{\"customer_name\": \"netflix\", \"user_name\": \"denominator\", \"password\": \"abc+123%25d8\"}");
}

Fails with the following:
Expected :"{"customer_name": "netflix", "user_name": "denominator", "password": "abc+123%25d8"}"
Actual   :"{"customer_name": "netflix", "user_name": "denominator", "password": "abc 123%d8"}"

Proposed change, currently line 232 of RequestTemplate:
public RequestTemplate resolve(Map<String, ?> unencoded) {
    ...
    if (bodyTemplate != null) {
      body(urlDecode(expand(bodyTemplate, unencoded)));
    }
    ...
}

Replace with:
public RequestTemplate resolve(Map<String, ?> unencoded) {
    ...
    if (bodyTemplate != null) {
      body(urlDecode(expand(bodyTemplate, encoded)));
    }
    ...
}

The encoded params already exist (used for expansion in the URL).  All pasts look to test for me after this change.  I may be missing something obvious here on cases where this wouldn't work.

Looks like it is pull request time!  Tell you what.. I'll run denominator's
tests off your PR's branch.. if they fail, we are missing a test.
Otherwise, let's get your change in (complete with the new test!)
Good hunting, Drew!


The function ssl_get_min_max_version() can fail

We should always check the return code.
This fixes a coverity issue.

Pushed. Thanks.


Icons fix (may be breaking!)

Fixed https://github.com/JujuAdams/Input/issues/371 by replacing binding label and category whitespaces with underscores.
This may break compat but I dont have a project to test this on.

Unfortunately, this is a breaking change so we can't merge this in. Thank you for trying though!


Bug fixed: now working with latest jwt-go updates

I have made the code migration specified in the latest release of jwt-go dependency. See Migration Guide
Important
For properly working, you must also make the changes on the slumber-sessions package on my other PR sogko/slumber-sessions#2

merge?


feat(config): add a new productId and add parameters to 14297/ZW1002 outlet

Label is the same but I got one with a newer product Id.
Also added the parameters/metadata from the other zw1002 entry as the configuration parameters are the same. (as written on the manual in the box)

@AlCalzone anything that needs to change to make this PR acceptable?


Event does not trigger in a ":" input from Chat

I installed the code into my unity project and was able to set it up normally and it works normally with normal messages and
However, if an user were to input ":" into chat, it does not trigger any of the events: onChatMessageReceived, onChatCommandReceived, onChatRewardReceived
Weird behaviour happens when emotes like :D and :| is used. For example, viewer types ":) :|" in chat, it only shows the "|" in the chatMessage.Message parameter
so longer sentences like "Welcome to the stream :)" the chatMessage.Message only contains ")"

@holypunisher We've fixed the problem in the new version (1.5.0). I already published it, it could take 24-48 hours to be available in the asset store.


Visual Scripting "Set stream" node crashes when choosing New AudioStreamOGGVorbis

Godot version:
3.1 a
OS/device including version:
Windows 10
Issue description:
Visual Scripting "Set stream" node crashes when choosing New AudioStreamOGGVorbis
Steps to reproduce:
Minimal reproduction project:

Can you provide an example project?

There is not enough information in this issue to reproduce the bug. Closing.


Spyder quitting unexpectedly

Description
What steps will reproduce the problem?

What is the expected output? What do you see instead?
Please provide any additional information below
File "/anaconda3/lib/python3.6/site-packages/spyder/plugins/ipythonconsole.py", line 950, in update_working_directory
shellwidget.get_cwd()
File "/anaconda3/lib/python3.6/site-packages/spyder/widgets/ipythonconsole/shell.py", line 128, in get_cwd
self.silent_exec_method(code)
File "/anaconda3/lib/python3.6/site-packages/spyder/widgets/ipythonconsole/shell.py", line 340, in silent_exec_method
local_uuid = to_text_string(uuid.uuid1())
File "/anaconda3/lib/python3.6/uuid.py", line 606, in uuid1
clock_seq_hi_variant, clock_seq_low, node), version=1)
File "/anaconda3/lib/python3.6/uuid.py", line 168, in init
raise ValueError('field 6 out of range (need a 48-bit value)')
ValueError: field 6 out of range (need a 48-bit value)
Version and main components

Spyder Version: 3.2.6
Python Version: 3.6.3
Qt Versions: 5.6.2, PyQt5 5.6 on Darwin

Dependencies
pyflakes >=0.6.0 :  1.6.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.2.0 (OK)
pandas >=0.13.1  :  0.22.0 (OK)
numpy >=1.7      :  1.14.0 (OK)
sphinx >=0.6.6   :  1.6.6 (OK)
rope >=0.9.4     :  0.10.7 (OK)
jedi >=0.9.0     :  0.11.1 (OK)
psutil >=0.3     :  5.4.3 (OK)
nbconvert >=4.0  :  5.3.1 (OK)
sympy >=0.7.3    :  1.1.1 (OK)
cython >=0.21    :  0.27.3 (OK)
qtconsole >=4.2.0:  4.3.1 (OK)
IPython >=4.0    :  6.2.1 (OK)
pylint >=0.25    :  1.8.2 (OK)

This is a duplicate of issue #6335. The solution is to open Terminal.app and run there
conda update python python.app

It did not solve the problem. Could you please help me with it.
On Mon, Apr 23, 2018 at 3:57 PM, Carlos Cordoba<EMAIL_ADDRESS>wrote:

This is a duplicate of issue #6335
https://github.com/spyder-ide/spyder/issues/6335. The solution is to
open Terminal.app and run there
conda update python python.app
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/7010#issuecomment-383720455,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ad4D-4fnz-EAjALXByDKqcw7XA7z1F8uks5trkBSgaJpZM4TgkS8
.

@BNRao Sorry, but the problem has nothing to do with Spyder, and rather due to a bug in how Python itself interacts with your operating system. Therefore, unfortunately there's nothing more we can do. However, if you've updated Python properly in the same environment with which you are launching Spyder, it should indeed be fixed, as the dozens of other duplicate error reports all confirm. You can read this link for more information about what's going on and other ways to fix it, if for whatever strange reason the above didn't work somehow.

@BNRao, other people has mentioned that switching to Python 3.6.4 fixes the issue for them. So you can run
conda install python=3.6.4

and try again.

Other than that, we can'e help you, sorry.

@ccordoba12 Are you thinking of, #6876 , the Python 3.6.5 ImportError: dlopen[ ..]. expected in: flat namespace issue that cropped up in macOS due to out of date Anaconda packaging of python.app? The traceback is clearly that of the error you named originally, #6335 , which is fixed in 3.6.5 and broken in <= 3.6.4 , but they were easy to confuse since both were macOS specific and several users had the former after updating to fix the latter. In any case, both issues should be solved by your original command conda update python python.app, while conda install python=3.6.4 will presumably reintroduce the user's actual error shown above.


US National Forest shields

While we have a convenient map to indicate progress on state-level shields, we can't forget about the remaining federal shields. The United States Forest Service signposts its own numbered routes in and around the federal lands it manages.
Relation coverage is currently incomplete. network values for Forest Routes include US:FS and US:NF. This would need to be standardized before moving forward with implementing shields. Both networks typically have up to 3 digits in their ref values, though I wouldn't be surprised if there were ones out there with 4 digits.
Forest Route shield:

network values for Forest Routes include US:FS and US:NF. This would need to be standardized before moving forward with implementing shields.

Could some mappers be using these two values to distinguish between forest primary and secondary routes? Or is this just a slow-motion transition from one tag to the other?
Puerto Rico (#240) also has Bosque Nacional shields that would look identical to the National Forest shields after simplification.

Oop. Clearly I needed to do a little more investigation. There are National Forest primary and secondary routes, though current network tagging gets even more complicated here.
The guidelines say to include the name of the National Forest in the network value, like US:NFSR:Chequamegon-Nicolet:NF. I don't think this is necessary, because shields are standardized across all National Forests (with the exception of the Spanish-language text on the Puerto Rico variant).
National Forest secondary route shields can be horizontal or vertical, depending on the level of maintenance of the roadway. The horizontal shields represent well-maintained roads that average vehicles can drive, while vertical shields represent less-maintained roads more suitable for hikers or 4-wheel drive vehicles. FSTopo maps distinguish all three shield designs, but there is currently no tagging scheme to distinguish horizontal from vertical shields.

While the shields are standardized across all Forests, the route numbers are only unique within a single Forest.

Just checked US Forest Service Route tagging in Washington and it is definitely rough. I counted 118 roads that I believe would be deserving of US:NFSR:Forest name:NF (the primary variety) tagging and found only 1 relation in the whole state (but disjointed and with a typo 😆)...

Separated BIA routes into a new ticket: #351

Once #355 is merged, the shield artwork will be available for these routes. The only thing left to do is reach consensus on tagging :grimacing:

NF shields would be a nice addition to coincide with rendering tracks (#717). 😀
Example at map=15.45/43.976043/-73.077052

If anything else, I wonder if we should chunk this into two different issues.

Shields for Forest Highways
Shields for Forest Roads

I'd advocate for doing Forest Highways first, as they're more important and the problem is vastly easier than for Forest Roads.
Forest Highways have only one shape and color across the country, and are predictably maintained. Forest Roads have what appears to be three different shapes, and two different orientations (horizontal and vertical, whereas sometimes the vertical signs have a down arrow in the shape, and other times not). Forest Highways should also always be signed, whereas Forest Roads are more likely to be all over the place. My hope is that by starting small (actually big, the Forest Highways), maybe it could encourage more tagging in OSM and thus, more interest in tackling the bigger challenge of Forest Roads.
Tagging
Having followed OSM Americana and how they've generally decided on tagging schemes for network=*, I do think the original guidelines found here should be tweaked. I'd suggest routes get tagged with network=US:NFSR:NF (should this be FH? I don't think NF is a logical derivative, but changing might require documentation, education, and cleanup) or network=US:NFSR:FR. Instead of including each forest name in the network=* tag, the forest name should be used in the operator= tag at the route level. Sophisticated data consumers would still be able to utilize the new tagging system in the way the used the old tagging system.
Thoughts? My two takeaways were:

Let's just do Forest Highways first, and
Should we simplify network=* tagging to omit the specific National forest but encourage that data in the operator= tag instead.

Taking the less formal route for a tagging proposal: https://community.openstreetmap.org/t/network-tagging-proposal-for-usfs-roads/97057

Are there any updates on this? Just mentioning because I found https://www.openstreetmap.org/relation/9443117 while adding Albany County WY route relations and it brought the topic to my mind.

I'm not clear on whether we have agreement on the tagging for these roads -- if that's settled out, then PR is certainly welcome.


Add ios to platforms

Since the last version iOS doesn't work any more. This PR fixes it.
https://github.com/pinkfish/flutter_native_timezone/issues/9#issuecomment-790959357
There have been two errors in the package. Fixed them, so the tests pass.

@pinkfish still the same error in the latest release...I don't know what happened...when I do pod install from my fork, it works...pod install from your release, it doesn't:
fork:

release:

Weird, let me double check.  I compiled on ios myself yesterday, should be
working.  Sorry.
On Fri, 5 Mar 2021 at 00:25, nohli<EMAIL_ADDRESS>wrote:

@pinkfish https://github.com/pinkfish still the same error in the
latest release...I don't know what happened...when I do pod install from my
fork, it works...pod install from your release, it doesn't:
fork:
[image: fork]
https://user-images.githubusercontent.com/43643339/110087975-99914b80-7d94-11eb-80f1-962d848d84f0.png
release:
[image: release]
https://user-images.githubusercontent.com/43643339/110088011-a0b85980-7d94-11eb-996b-5b10469070cd.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pinkfish/flutter_native_timezone/pull/21#issuecomment-791251557,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATOMMKZ5YQDVCQDHNGSP4LTCCIRLANCNFSM4YUBNLGA
.

Probably, if you just publish a new version, it will work.

Hi guys, I'm still having the issue after I upgrade to flutter 2.
[VERBOSE-2:ui_dart_state.cc(186)] Unhandled Exception: MissingPluginException(No implementation found for method getLocalTimezone on channel flutter_native_timezone) #0      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:156:7) <asynchronous suspension> #1      FlutterNativeTimezone.getLocalTimezone (package:flutter_native_timezone/flutter_native_timezone.dart:12:9) <asynchronous suspension>
Thank you for the valuable package by the way!

This works for now:
flutter_native_timezone:
    git:
      url: https://github.com/nohli/flutter_native_timezone
      ref: fix-ios


Added 3rd degree equation solver files

Description
Added a calculator that solves rquations of third degree and prints out its 3 roots (ax^3 + bx^2 + cx + d = 0)
Fixes #388

Type of change

[x] New feature (non-breaking change which adds functionality)

Explain the Testing instructions
Test Configuration:

Operating system: Ubuntu
Version: 21
Text-editors used: Sublime

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[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] Any dependent changes have been merged and published in downstream modules

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK
Deployment Link

@VidhiBhatt01 Done.

Still the same.

Align it in center too.

@VidhiBhatt01 Done.

Almost perfect, can we do center alignment too?

I don't quite grasp what you mean. It's already center aligned

I don't quite grasp what you mean. It's already center aligned

Just drop a DM on discord, will help you out tomorrow over a quick 5 min call.

Sure.

@XZANATOL any updates

@VidhiBhatt01 Done.

@XZANATOL it's done, just add this project in style_links.json


Authelia: v4.37.5 -> v4.38.9

Description of changes
Authelia: v4.37.5 -> v4.38.9
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

This change looks incomplete, as I would expect at least two hashes to change.

This is a duplicate of #299309.

Thank you for contributing, though, and I hope to see you again in the future!


Bad boolean logic

This line is causing a bug due to operator precedence. Should have bracket around == expression.
https://github.com/broadinstitute/seqr-loading-pipelines/blob/91ae7b482b5fa6067c71c2dc938f5bcedbb8be8b/hail_scripts/computed_fields/vep.py#LL62C1-L62C21

@tkallady thank you for the bug report!


ipex button

Add ipex button for better instruction.

Hi @IlyasMoutawwakil , Could you please merge this PR since ipex doc has been merged, it could help users to find ipex doc well. Thx!

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


bounds_changed event called with no parameter passed

Here are excerpts from the code:
<GoogleMap @click="mapClicked" @bounds_changed="boundsChanged" @idle="mapIdle" ref="mapRef" maptype="hybrid" api-key="..." style="width: 100%; height: 400px" :center="mapCenter" :zoom="15">
			<Marker v-for="place in PlacesListFiltered" .... @click="markerClicked(place._id as string)" />
</GoogleMap>

<script setup lang="ts">
import { GoogleMap, Marker } from "vue3-google-map";

// ....
const mapCenter = ref({ lat: 44.81645887786941, lng: 20.460121035575867 });

const boundsChanged = (e) => {
        //ISSUE: e is undefined!
	console.log(`[boundsChanged] e: ${e}; getBounds: ${(mapRef.value as any).api.getBounds()}`);
};

const mapClicked = (e) => {
	console.log("mapClicked", e, e.latLng.lat(), e.latLng.lng());
};

</script>

Just to mention that other event, like `mapClicked` are working fine and I get their events and info from them.
Yet also for `idle` event no parameters

# Expectations
I was expecting to get parameters of lat/lng visible window, as with using `getBounds`, through this event.

In addition, I also have an issue with accessing `getBounds()` that should be accessible through the `api`
yet this is `undefined`:
`mapRef.value as any).api.getBounds()}`
the `mapRef.value as any).api` is defined and is exposing different parameters, methods, yet no `getBounds`
I've tried also with  `mapRef.value as any).map.getBounds()`, but no success neither

Hi @sinisarudan
It doesn't look like the bounds_changed listener callback can receive any arguments:
https://developers.google.com/maps/documentation/javascript/reference/map#Map-Events
As for getBounds it should be available as mapRef.value.map.getBounds

Thanks @HusamIbrahim for the quick reply.
You're right, after checking official docs, I do see that bounds_changed does not receive any argument.
Regarding mapRef.value.map.getBounds: maybe I've mistreated this event to cause me seeing getBounds not working. After cleaning the code, eliminating the event part, etc, this DOES work:
const mapRef = ref(null);
let gMap: google.maps.Map;
let bounds: google.maps.LatLngBounds | undefined;

const boundsChanged = () => {
	if(!gMap) {
		gMap = (mapRef.value as any).map;
		console.log("[boundsChanged] gMap", gMap);
	}
	bounds = gMap.getBounds();
	console.log(`[boundsChanged] bounds : ${bounds}`);
};

So, I'm closing the issue. Thanks again and sorry for bothering!


StrictUndefined and {% if foo is defined %} disagree about whether a parameter set to None is defined

I have something like this in my Jinja2 templates:
{% if something %}
{{ something }}
{% endif %}

When I define something to None when rendering the template, this works as expected. However, I would like to use StrictUndefined in my project to eliminate issues with potentially undefined parameters, and then this breaks, as using StrictUndefined seems to cause parameters that are defined to  None to be treated the same as if they were not defined at all.
When I change my template to this:
{% if something is defined %}
{{ something }}
{% endif %}

Then it will get rendered as:
None

So it looks like the using StrictUndefined  in the environment and checking whether a parameter is defined within the template disagree on whether None means the parameter is defined or not.
Environment:

Python version: 3.13.0, 3.12.7, 3.11.10
Jinja version: 3.1.4

As a workaround, I found that this works in combination with StrictUndefined:
{% if something is defined and something is not none %}
{{ something }}
{% endif %}

But as expected, this does not:
{% if something is not none %}
{{ something }}
{% endif %}

→ jinja2.exceptions.UndefinedError: 'something is undefined

Undefined means not passed at all. None is a value.

Yes, that's my point? StrictUndefined seems to interpret dict(something=None) the same as dict() when checking whether something is defined.

You can do {% if something|default %} to check for both defined and not falsy (or add is not none if you want a stricter check)

I think you're saying the opposite of what you're observing. Your workaround is nota workaround, it's exactly what needs to happen. Undefined is only for undefined variables, not for variables defined as none. If you don't define the variable and try to check if it's none, it's undefined. If you define it as none, it's not undefined.

In your case, you want to check both.

Ok - but that's what I thought I was doing by using Environment(undefined=StrictUndefined) and checking the value for "truthiness" in the template? Why would I need to duplicate the is defined check if that's supposed to get checked by StrictUndefined?
And it still doesn't explain why using Environment(undefined=StrictUndefined) causes a different behaviour for checking whether something is defined than using {% if something is defined %} 😞

No behavior is changing, defining a value as None means it's defined.

Yes, that is what I expect, and I understand that - but what I'm saying is that this is not what happens with Environment(undefined=StrictUndefined) - it causes values that are defined to None to be detected as undefined.

pass dict(something=None)
get jinja2.exceptions.UndefinedError: 'something' is undefined

sigh I changed something in my code that should be unrelated, and now the behaviour of StrictUndefined is suddenly as I expect it to be ... so, thank you for helping me with this, and sorry for the noise! 🙈


Sort by Date is not working as expected. It only sorts based on the day and not considering month and year

function BasicTable(props) {
const [tableData, setTableData] = useState(data)
const columns = [
    {
      title: "Date", field: "transDate", type: 'datetime', sorting: true, search: true, width: 200,
      dateSetting: {
        format: 'dd/MM/yyyy'
      },
      cellStyle: styles.cellStyle
    },
    {
      title: "From", field: "userName", sorting: true, search: true, width: 200
      cellStyle: styles.cellStyle
    },
    {
      title: "Sent To", field: "addressee", sorting: true, search: true, width: 200,
      cellStyle: styles.cellStyle
    }
];
return (
    <MaterialTable
      columns={columns}
      data={tableData}
      options={{
        sorting: true
      }}
/>
  );
}

export default BasicTable;

Could you add a sandbox as requested?

Please find the sandbox link.
https://codesandbox.io/s/thirsty-pasteur-1nfobx

Could you add a sandbox as requested? Since it works for me, it might depend on the version or the data objects you are passing. But since there is not sandbox, we cannot verify any of it.

I have added source in the below sandbox. I also noticed that string is sorted by capital letter first and then sorted by lowercase. Is this expected behaviour?
https://codesandbox.io/s/thirsty-pasteur-1nfobx

Hi, as expected you are not passing dates to the table within the fields. You need to wrap the date fields with new Date(value).
The sorting with upper vs lower case originates from Javascript. If you want, you can overwrite the custom sort to compare the fields with .to lowercase()

Hi, as expected you are not passing dates to the table within the fields. You need to wrap the date fields with new Date(value).
The sorting with upper vs lower case originates from Javascript. If you want, you can overwrite the custom sort to compare the fields with .to lowercase()

I tried to pass the date value in the render, however it's throwing the below error.
"Objects are not valid as a React child (found: [object Date]). If you meant to render a collection of children, use an array instead."
render: (rowData) => {
return new Date(rowData.transDate);
},
dateSetting: {
format: "dd/MM/yyyy"
}
Could you provide sample code?

No you need to change your data object. I am not at my computer right now so maybe later. But the data you are passing into the data prop needs to be data at the right places

No you need to change your data object. I am not at my computer right now so maybe later. But the data you are passing into the data prop needs to be data at the right places

I am sorry I don't understand what you have explained. It's not urgent, but I would really appreciate if you could share sample code when you are free.
I also have one more problem with the search functionality.
I have a requirement to extend search functionality for detail panel. The default search functionality of Material-table works fine for tree data. However my detail panel records are different from parent row, hence I cannot use tree data.
Hence I have written a custom function which extend search for detail panel and calling it from onSearchChange.
It works when I log the searched the rows in the console it returns the matched records(correct data based on the search text), however material-table is not displaying updated records(returns empty record even though detail panel has the search text).
I may be doing something wrong in setting the searched data to Material-table. It's in the same sandbox code.
Could you please check the sandbox code and let me know if it possible?

https://codesandbox.io/s/zealous-leavitt-h4nhbf?file=/src/component/BasicTable.js
Here you can see how I modify your data to have a date obejct.

https://codesandbox.io/s/zealous-leavitt-h4nhbf?file=/src/component/BasicTable.js
Here you can see how I modify your data to have a date object.
Regarding the search, you should be able to just override the customSearch and search each object in there instead of listening to the onSearchChange. https://github.com/material-table-core/core/blob/master/types/index.d.ts#L184 You just have to return true or false if the entry should be included
Update: https://codesandbox.io/s/misty-haze-zm6d9g Something like this should be enough @testgh2022

It's working. It simplified my work in a great way. Thanks a lot.

Selection is not working when Custom sort and local storage implemented together. I had to remove custom sort in order to make selection work. Is there any reason why I can't use them together with selection?
Sandbox code:
https://codesandbox.io/s/thirsty-pasteur-1nfobx?file=/src/component/BasicTable.js

https://codesandbox.io/s/zealous-leavitt-h4nhbf?file=/src/component/BasicTable.js
Here you can see how I modify your data to have a date object.
Regarding the search, you should be able to just override the customSearch and search each object in there instead of listening to the onSearchChange. https://github.com/material-table-core/core/blob/master/types/index.d.ts#L184 You just have to return true or false if the entry should be included
Update: https://codesandbox.io/s/misty-haze-zm6d9g Something like this should be enough @testgh2022

Selection is not working when Custom sort and local storage implemented together. I had to remove custom sort in order to make selection work. Is there any reason why I can't use them together with selection?
Sandbox code:
https://codesandbox.io/s/thirsty-pasteur-1nfobx?file=/src/component/BasicTable.js


update level dependency

Addresses https://github.com/spanishdict/example-dfp-line-item-generator/issues/7

Merging without review since there are no functional changes, just a bug fix


Barrel glossary code example issue

There seems to be an issue with one of the code examples in the glossary for barrel. The code example after the text "That's why we can write this:" has this in the code example area:
BAD FILENAME: ../../_fragments/../docs/_fragments/quickstart/ts/app/app.component-import.ts.md
Current path: docs,ts,latest,glossary PathToDocs: ../../

Thanks, and let me know if more information is needed

We have an issue with double glossary as seen #1123.
So we will continue there.
Thanks.


Fix cpp11

add cpp11 filename extension

Seems good to me at first glance.
I will review it later


How to re-import rigged character without deleting it in godot everytime?

This is a known issue on Godot issue tracker. Just have to wait for it to get fixed.


Namespace implementation

What's inside the box:

namespace functionality fully integrated into mlflow part.
mlflow integration and unit tests adopted for namespace logic.
namespace functionality integrated into aim part. not fully. needs attention from @jgiannuzzi
aim integration tests  adopted for namespace logic.
introduced basic admin routing.
moved db models to common package, since it will be shared across mlflow and aim apis.
refactored middleware logic to catch current namespace.
redesign/simplify integration tests.
started fixing minor problems across the project -> golangci-lint.

What's left(we can complete it in other tasks a bit later):

add more unit, integration tests related to namespace functionality.
as I can see our repositories started receiving a lot of input parameters. sometimes it could be 3, sometimes event 5. It doesn't look neat. it will be nice to introduce dto objects and pass this objects right into repositories functions in case if for example function waits more then 3-4 parameters. Also, I think it could help and remove some functions which looks almost similar but differ in one or more parameters. So we can remove duplicate functions.
aim api, needs refactoring and integration tests. it is hard to support, read and integrate something new.
fully fix all the golangci-lint remarks, so we can integrate this step as a required step into our pipeline.

@jgiannuzzi can you please take a look into aim part, into this functions:

SearchRuns function -> 414 line.
SearchMetrics function -> 675 line.
SearchAlignedMetrics function -> 871 line.
these are the places where I'm not at all sure.


Too similar file names on macos

As pointed out by @measrainsey last year and now by @ravitby, this repository currently contains both demand.csv.gz and DEMAND.csv.gz in data/test/MESSAGEix-GLOBIOM_1.1_R11_no-policy_baseline. This seems to cause git conflicts on macos systems as they don't distinguish between capitalized and non-capitalized file names.
I'm not immediately sure how we can remedy this, possibly by renaming a file, but I'm not sure if the exact names are somehow important (e.g., correspond to some model parameter and need to be exactly the way they are for the file to be found).

The result of the duplicate files is that there is an always a change in the repo and therefor it's impossible to pull new changes.
This is the output message for cloning the repo:
remote: Enumerating objects: 25083, done.
remote: Counting objects: 100% (3728/3728), done.
remote: Compressing objects: 100% (1448/1448), done.
remote: Total 25083 (delta 2505), reused 2374 (delta 2278), pack-reused 21355
Receiving objects: 100% (25083/25083), 19.41 MiB | 11.41 MiB/s, done.
Resolving deltas: 100% (15969/15969), done.
Updating files: 100% (565/565), done.
Filtering content: 100% (353/353), 89.53 MiB | 5.25 MiB/s, done.
warning: the following paths have collided (e.g. case-sensitive paths
on a case-insensitive filesystem) and only one from the same
colliding group is in the working tree:

  'message_ix_models/data/test/MESSAGEix-GLOBIOM_1.1_R11_no-policy_baseline/DEMAND.csv.gz'
  'message_ix_models/data/test/MESSAGEix-GLOBIOM_1.1_R11_no-policy_baseline/demand.csv.gz'

Thanks for the additional input. Searching for this error message, I came across this suggestion to set up a new, case-sensitive volume on macos that can host the existing code.
However, I'd consider it more elegant if we could avoid using the same name for these files. I hope @khaeru knows if such an adaption is straightforward.

There's also a git config variable that we could paly around with to see if some setting satisfies our needs locally.

This is an unfortunate consequence of the fact that the MESSAGE GAMS code uses both a parameter named demand and a variable named DEMAND. See:
https://github.com/iiasa/message_ix/blob/693f2273eaf31aec7a9938f65ee26f98be967742/message_ix/model/MESSAGE/parameter_def.gms#L147-L159
https://github.com/iiasa/message_ix/blob/693f2273eaf31aec7a9938f65ee26f98be967742/message_ix/model/MESSAGE/data_load.gms#L75-L76
https://github.com/iiasa/message_ix/blob/693f2273eaf31aec7a9938f65ee26f98be967742/message_ix/model/MESSAGE/model_core.gms#L149-L151
The model data snapshot published on Zenodo contains data for both, and so the 'exploded' or 'unpacked' version of that file stored in this repo has both.
AFAIK, the variable DEMAND (upper case) is not relied on by any tests, currently, so I think as a mitigation the corresponding file could be safely deleted. If/when we add or migrate (from message_data) code that relies on data in this variable, we would need to find a different approach.

I think we can leave this open since the merged mitigation PR is likely just a temporary fix; however, for the time being, this issue should be solved on main :)


fix: update E2E tests to use new menu names

Description
We need to make sure our E2E tests get updated when interface names do.
Task Item
#minor

Incorporated into #4930 .


[feat] sentence transforer

This changes are made to incorporate sentence transformer to compute sentence embedding!

nice job!

@maulikmadhavi can you update the Readme and pull a new request


f2py compile multiple files, Exception: Check the blocks

I am trying to compile three fortran 77 files with f2py.  I know the files compile with ifort and gfortran.
f2py produces a long trail of warnings and ends with a message to "Check the blocks":
[emason@marula r_tools]$ f2py -verbose  -c -m --fcompiler=intelem *.F
running build
running config_cc
unifing config_cc, config, build_clib, build_ext, build commands --compiler options
running config_fc
unifing config_fc, config, build_clib, build_ext, build commands --fcompiler options
running build_src
build_src
building extension "rfact.F" sources
f2py options: []
f2py:> /tmp/tmp_QlXua/src.linux-x86_64-2.7/rfact/Fmodule.c
creating /tmp/tmp_QlXua/src.linux-x86_64-2.7
creating /tmp/tmp_QlXua/src.linux-x86_64-2.7/rfact
Reading fortran codes...
        Reading file 'roms_read_write.F' (format:fix,strict)
Traceback (most recent call last):
  File "/usr/bin/f2py", line 28, in <module>
    main()
  File "/usr/lib64/python2.7/site-packages/numpy/f2py/f2py2e.py", line 648, in main
    run_compile()
  File "/usr/lib64/python2.7/site-packages/numpy/f2py/f2py2e.py", line 633, in run_compile
    setup(ext_modules=[ext])
  File "/usr/lib64/python2.7/site-packages/numpy/distutils/core.py", line 169, in setup
    return old_setup(**new_attr)
  File "/usr/lib64/python2.7/distutils/core.py", line 151, in setup
    dist.run_commands()
  File "/usr/lib64/python2.7/distutils/dist.py", line 953, in run_commands
    self.run_command(cmd)
  File "/usr/lib64/python2.7/distutils/dist.py", line 972, in run_command
    cmd_obj.run()
  File "/usr/lib64/python2.7/site-packages/numpy/distutils/command/build.py", line 47, in run
    old_build.run(self)
  File "/usr/lib64/python2.7/distutils/command/build.py", line 127, in run
    self.run_command(cmd_name)
  File "/usr/lib64/python2.7/distutils/cmd.py", line 326, in run_command
    self.distribution.run_command(command)
  File "/usr/lib64/python2.7/distutils/dist.py", line 972, in run_command
    cmd_obj.run()
  File "/usr/lib64/python2.7/site-packages/numpy/distutils/command/build_src.py", line 147, in run
    self.build_sources()
  File "/usr/lib64/python2.7/site-packages/numpy/distutils/command/build_src.py", line 164, in build_sources
    self.build_extension_sources(ext)
  File "/usr/lib64/python2.7/site-packages/numpy/distutils/command/build_src.py", line 326, in build_extension_sources
    sources = self.f2py_sources(sources, ext)
  File "/usr/lib64/python2.7/site-packages/numpy/distutils/command/build_src.py", line 563, in f2py_sources
    ['-m', ext_name]+f_sources)
  File "/usr/lib64/python2.7/site-packages/numpy/f2py/f2py2e.py", line 408, in run_main
    postlist = callcrackfortran(files, options)
  File "/usr/lib64/python2.7/site-packages/numpy/f2py/f2py2e.py", line 329, in callcrackfortran
    postlist = crackfortran.crackfortran(files)
  File "/usr/lib64/python2.7/site-packages/numpy/f2py/crackfortran.py", line 3216, in crackfortran
    readfortrancode(files, crackline)
  File "/usr/lib64/python2.7/site-packages/numpy/f2py/crackfortran.py", line 511, in readfortrancode
    dowithline(finalline)
  File "/usr/lib64/python2.7/site-packages/numpy/f2py/crackfortran.py", line 748, in crackline
    % (groupcounter))
Exception: crackline: groupcounter(=0) is nonpositive. Check the blocks.
[emason@marula r_tools]$

I have found one post from 2004 that addresses this error
http://osdir.com/ml/python.f2py.user/2004-04/msg00000.html
but it's not clear to me if this relevant for my case.  I'd be grateful for any suggestions, thanks.

Could you show us the header and footer portion of the file roms_read_write.F?
Btw, f2py option -m requires specification of module name which is missing in your case.
For example, try
f2py -c -m foo --fcompiler=intelem *.F

I am having the same error. Even after aligning the do loops in the code, the error resurfaces. Does anyone have insights on this crackline error?


Master build fails.

Hi!
I'm trying to use db-migrate in heroku, so I'm facing the same problems as discussed here.
I thought about fixing it by myself and submit a PR following guidelines for contributing
It says:

Run the tests. Pull requests without tests are much less likely to be merged, and it's great to know you're starting with a clean slate: npm test. Be sure to check the README for setting up your test databases first.

But the true is that test are broken on master for 3 months now:

My goal is to advance on this issue.
@wzrdtales you said you would probably work on that in a couple of weeks, so you may have started to work on that, is that already in progress?
Thanks

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@Kumjami Also sorry here for the late response, you're more than welcome to help out though :) And yet I haven't touched the issue again.
About the tests, if they fail b/c your changes broke something, you will need to fix that, but not if it does not, so this is not a big problem.


[WIP] Isolate BVH internal data structure from its use in TreeTraversal and TreeConstruction

This will allow for easier modification of BVH internals.

Codecov Report

Merging #491 into master will decrease coverage by 0.04%.
The diff coverage is 92.06%.

@@            Coverage Diff             @@
##           master     #491      +/-   ##
==========================================
- Coverage   96.09%   96.04%   -0.05%
==========================================
  Files         120      121       +1
  Lines        8788     8805      +17
==========================================
+ Hits         8445     8457      +12
- Misses        343      348       +5

Impacted Files
Coverage Δ

...h/src/details/DTK_DetailsTreeConstruction_decl.hpp
100% <ø> (ø)
:arrow_up:

packages/Search/test/tstLinearBVH.cpp
98.87% <100%> (ø)
:arrow_up:

packages/Search/src/details/DTK_Predicates.hpp
100% <100%> (ø)
:arrow_up:

...es/Search/src/details/DTK_DetailsTreeTraversal.hpp
100% <100%> (ø)
:arrow_up:

...ch/src/details/DTK_DetailsTreeConstruction_def.hpp
100% <100%> (ø)
:arrow_up:

packages/Search/src/DTK_LinearBVH_decl.hpp
100% <100%> (ø)
:arrow_up:

packages/Search/src/DTK_LinearBVH_fwd.hpp
86.11% <86.11%> (ø)

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 3384289...7ce9dd0. Read the comment docs.

Direct access to _leaf_nodes and _internal_nodes data members of BoundingVolumeHierarchy [trivial]
Details::TreeConstruction<DeviceType>::initializeLeafNodes(permutation_indices, primitives, _leaf_nodes ); and Details::TreeConstruction<DeviceType>::generateHierarchy( morton_indices, _leaf_nodes, _internal_nodes ); in the BoundingVolumeHierarchy constructor.

Direct access to the bounding_box data member of Node [more serious]
TreeConstruction::initializeLeafNodes() has leaf_nodes( i ).bounding_box
CalculateBoundingBoxesFunctor has node->bounding_box = node->children.first->bounding_box; and expand( node->bounding_box, node->children.second->bounding_box );

* Direct access to `_leaf_nodes` and `_internal_nodes` data members of `BoundingVolumeHierarchy` [trivial]

I'm not sure what you mean here. Do you consider this a problem? Fixing it would require adding getLeafNodes() or something similar.

A lot of this PR is superseded by @dalg24's work.

@aprokop You comfortable with closing in favor of #497 ?


[Accessibility] On modal close focus doesn't return to element which opens the modal

Describe the bug
Focus doesn't return to element which opens the modal instead focus moves to the body element on modal close
How to reproduce
Steps to reproduce the behavior:

Go to provided Stackblitz setup here
Click on "Show Modal" button
Close the modal by tabbing the button
See focus doesn't return to "Show Modal" button. For further verification, see the browser console to know the current active element

Expected behavior
Focus should move to element which opened the modal in this case "Show Modal" button
Versions
This issue is also reproducible to the latest clarity version

Looks like an issue with the focus trap logic, we are tracking the last active element but it doesn't get refocused on close.

Apps can workaround this by manually moving focus where they want to on modal close.

Clarity cares deeply about accessibility, and we work with our accessibility team to identify and triage issues. We have an internal tracking system to identify and verify accessibility issues, which has captured this already. Some issues with Clarity Angular are difficult to solve without causing side effects or breaking changes, and we have to balance concerns with fixing issues and breaking changes. As we build out Clarity Core, we are able to address these root cause accessibility issues and recommend adopting the Clarity Core implementations as they become available. We are closing this issue as we have the issues tracked internally. Please follow our development and releases to see when we release relevant Clarity Core components to make this possible.


Update Rdio to use Oauth2.0

Oauth 1.0 is being deprecated on 30th June (cool notice period, guys).
http://blog.rdio.com/developers/2015/06/oauth-20-launch.html

Updated library to 2.0, but it needs me to manually refresh the refresh token every 12 hours. Wat.


feat: user friendly weaver error messages

Updated weaver error messages to make them easier for users to fix.  For example:
Before:

After:

great

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


Directory naming inconsistency

Hi,
I really appreciate the work, time and effort you put into PFL.
I was just starting a new C/C++ project and was wondering what are the current best practices for organizing files. So I was looking into it when I found your research, proposed layout.
I read through The Pitchfork Layout (PFL) ideas, reddit post and Project Layout Survey Results and Updates.
I understand this to be the most popular, status-quo, currently established (but not necessarily best) practices.
I had some thoughts that wouldn’t let me rest, to not say something in good conscious.
So if you let me I’d like to share some constructive criticism:
Project Directory naming seems really inconsistent.
I understand that names came out as most popular vote in individual poll questions, and that might be the cause of inconsistency. The other reason is “We are already used to it” [this mess].
I have problem along 2 axes with lack of consistency: plurality and abbreviation.

Singular

build/
src/
include/
external/

Plural

tests/
examples/
extras/
data/
tools/
docs/
libs/

Abbreviated

src/
docs/
libs/

Not abbreviated

build/
include/
tests/
examples/
external/
extras/
data/
tools/

Does this project only have one source but multiple docs, libs?
Please, this cannot be the proposed layout for new designs. (That we will have to live with for decades.)
There are others like source - header, compile - include pairs yet the proposal suggests using src (source) with include… but I’m gonna let linguists deal with that one…
I would be interested in a new poll voting between 4 consistent naming scheme, and another vote between the winner and the current inconsistent PFL (or a vote between the 5 – not sure if forcing people who would want the current layout to choose their least offending consistent layout would be beneficial or not - would need to ask a survey specialist for their opinion).
Singular Not abbreviated

source/
documentation/
library/
build/
include/
test/
example/
external/
extra/ * or optional/, see later
data/ * I mean I know data is plural form of Latin datum, but it’s a mass noun?… https://crosstalk.cell.com/blog/is-data-plural, https://www.britannica.com/dictionary/eb/qa/Is-Data-Singular-or-Plural-
tool/

Plural Not abbreviated

sources/
documentations/
libraries/
builds/
includes/
tests/
examples/
externals/
extras/ * or optionals/, see later
data/ * It has Schrödinger’s plurality, it’s both or just don’t ask before it collapses ;)
tools/

Singular Abbreviated

src/
doc/
lib/
bld/ * would bil/ be better?
inc/
tst/  * tes/? or are we good with test/ and it’s a 4 character limit
exa/ * xmp/? or exam/
ext/
opt/ * here is where optional/ comes in instead of extra/
dat/ *
too/ * tol/? tl/? or keep 4 character tool/ here too?

Plural Abbreviated

srcs/
docs/
libs/
blds/ * would bils/ be better?
incs/
tsts/  * tess/? or tests/
exas/ * xmps/? or exams/
exts/
opts/ * here is where optionals/ comes in instead of extras/
data/ * or you can even have dats/
toos/ * tols/? tls/? or keep tools/

I think both SNA, PNA would be acceptable. I don’t think anyone would like full SA, PA.
  Or that they have to type src and a tool doesn’t auto-completes it to source. Well, people were also used to slavery but over time they realized that it’s not right.. this is certainly less important, we’ll see if it gets into history books – “How security flaws in C codes ended the world” is more likely)
I’m also fine with lib abbreviation, but libs just don’t make any sense to me. It should be either srcs, libs, docs or src, lib, doc trio.
In the poll there were no singular lib option (only libs, mods, and subdiv ...ew yuck), so I don’t know statistically how sound are these results. (plugins, modules were also not an option? also mods just sound to me like twitch moderators now, and libs like political liberals) That poll looks like a prime candidate for redoing. (But it doesn’t really matter to me, I’m not involved in large scale software development, I’m usually doing embedded firmware development. To me lib sounds like a folder containing any libraries used - internal or external, or maybe compiled library files that the project publishes for other project to use. So I’m clearly out of my depth here. Though to me it sounds a bit confusing. Modules, parts, components might sounds more intuitive IMO than lib, but people who work on these daily probably have better ideas.)
I don’t like using both external and extras names because they sound so similar (or abbreviate to the same 3 letter).
Just to be clear:

external/ is any external dependency (which can something you wget or git submodules...)
extras/ is a submodule root (but not git submodule, because that goes into external) (and it’s the optional submodules, not the mandatory ones)
and libs/ is where the mandatory submodules are ...if the project is divided into submodules (but again not git submodules, because that’s different)

Can anyone else besides me see why this terminology could be confusing?
I understand that you didn’t want to shoot yourself in the foot by calling them modules or packages because that would be confusing… so you shot your other leg instead ;)
I don’t know if component(s) would do or if that also used for describing something else.
As alternative I would suggest sub-project (instead of submodule which can describe another external dependencies aka projects). We are describing project layouts, if a project is divided into smaller parts/projects those could rightly be called sub-projects, no?
I think calling it package would be less confusing than submodule but not right.
Calling them modules is not great either but is submodule any better? If it’s a submodule what’s a module? Oh right a completely different thing a C++ module. Does that make any sense?
Do C++ modules go into module(s) directory? No? Why?
If we are not using modules directory (to publish C++ modules) then we could use it to hold sub-projects/project parts/project-modules (in contrast to C++ modules) … but shouldn’t!
Looking at the proposed submodule directory structure, these seem like full blown projects (with a common build/ system, external/ dependencies and tools/ ), they aren’t even really sub-projects. (But they are strongly coupled, maybe depend on each other… so they go into the same repo.)
So it’s a repository with many projects. No? Why aren’t we calling these submodules just projects?
Maybe our problem is that we are describing PFL as a project layout, when we should be describing it as a repository layout.
If a repository contains many projects, then why aren’t we calling it … just a repository. We can call it a many-project repository to differentiate it from a single project repository… or we can just borrow Visual Studio’s terminology and call it a “solution”. A solution contains one or more than one projects. Is this not what we are talking about here?
Will repositories hold more than one PFL Projects? Clearly a project can depend on other git submodules.
By the way thanks git for calling them submodules and not subrepos!
So PFL describes a repository and its subrepo dependencies, right? (And those dependencies should also follow PFL)
So PFL should apply to the whole repository, or can multiple PFL live in sub-directories of a repository?
Depending on that we can call PFL a repository layout instead of project layout, or call it a solution layout (which can have many inter dependent projects).
Am I right that a (multi VS project) VS solution can be a PFL project where the solution’s projects live in the submodule root directory but the solution cannot be PFL submodule … doesn’t seem like the directory structures would work out. Or can PFL submodules be nested and divided further?
Anyway, I think external/ directory name is fine for dependencies (I know deps is not good, so neither is dep, maybe req/ could be an alternative as in requirements, required dependencies), but for multi project solutions libs and extras doesn’t seem great. External and extras sounds more closly related than libs and extras. Wouldn’t required/ and optional/ be better? Should they even be top level directories or 2 sub-directories of modules or projects or parts or whatever (maybe it’s a solution name, maybe it stays libs or lib I don’t know).
Well I hope that some of this helps someone somewhere at some point.
TLDR:
I think this is the most abbreviated I would ever go for

src/
doc/
lib/
build/
inc/
test/
example/
ext/
opt/
dat/
tool/

And any build system that worth its weight in salt should handle the optional plural s ending(s)
and any mix between these abbreviations and full non abbreviated folder/directory names:

source/
documentation/
library/ (I would rename/move this if its purpose is to hold many inter related projects)
build/
include/
test/
example/
external/
optional/
data/
tool/

Because until we can agree on a consistent naming method people will mix and match to their heart desire.

In case typing speed or Carpal Tunnel Syndrome is a concern as a reason to use src instead of  source or that people write src  out of habit from muscle memory I think I can help...
; Here's an AutoHotkey script that replaces src with source as you type it:
:*:src::source

; And here are other abbreviations that are only auto-completed on pressing TAB key
; (to avoid accidentally replacing file extensions when typing foo.doc... and similar)
#Hotstring EndChars `t ; Only TAB is used as trigger
#Hotstring O ; omit TAB from the replaced text (allows continuing it, for example making it plural)
::inc::include
::doc::documentation
::lib::library
::ext::external
::opt::optional

I think that it's more important to stick to a common naming convention in POSIX-like systems, like Debian.
If I ls my /usr/local directory you see:

bin < contains binary executables
etc < contains extras, like
include < contains external headers
lib < contains extra stuff, like Find.cmake files (I think it shoudn't, but okay) and library binaries (.so/.a files)
src < the actual source code of binaries found in the bin and lib directories

bin and lib are typically the artifacts in the build directory, so that isn't something we need to concern ourselves with, but src and include are exactly what we're using here.
Here my stance is: what do people expect? I don't expect a source dir, I expect src, as I expect an include dir and not inc. Besides that it becomes more iffy.
Regarding plural/singular: there is only one source, which is a collection of source files. As with the include, that's what you include with -I in the compiler. It's not includes, it's only a single one. docs is not "the doc", they're multiple documents typically, as with examples, given you have more than one, it makes sense to say examples so people see it and expect multiple examples, one of which showing what the user needs.
So here's my takes with what's important in order:

what has the least technical issues? (ie: prevent private headers from leaking, least amount of work to use, prevent name clashes, cross compatible with other architectures etc.)
what do people expect? (src, include, doc/docs), everyone has different experiences of course, but look at other conventions with the least issues. (example SDL is notorious for include issues)
what IS it? ("the" source, "the" examples, "a" doc or "the" docs, "the" tools)

and you should just use what complies with the above


Develop Contingency Plans

💡 Description

Production website
Production registry

@tloubrieu-jpl @ramesh-maddegoda and @sjoshi-jpl need to contribute by the end of the week.

Review complete. Up for signature.

Completed, signed, and released. Posted in Alfresco.


hpcrun needs to track early threads

after the initialization was overhauled, early threads don't get to check monitor_ignore_map; they skip monitoring because hpctoolkit_is_initialized is false.

This repository has moved to GitLab, please continue this issue there.


Committed entries still shown in "zed lake status" after commit

Repro is with Zed commit fd273d1 and the Zeek zed-sample-data.
I begin by creating a lake+pool, staging a couple logs via separate add operations, and then committing them.
$ zed -version
Version: v0.29.0-366-gfd273d11

$ rm -rf $ZED_LAKE_ROOT

$ zed lake init
using environment variable ZED_LAKE_ROOT
lake created: file:///Users/phil/logs

$ zed lake create -p logs -orderby ts:desc
pool created: logs

$ zed lake add -p logs snmp.log.gz
1tRvK0kTROYpzbDo1UhlpMtu6KD staged

$ zed lake add -p logs pe.log.gz
1tRvKcrC6IaW782uRLhFRvhEdng staged

As expected, at this point the commits are shown as being in staging but don't yet appear in the commit log.
$ zed lake status -p logs
staged 1tRvK0kTROYpzbDo1UhlpMtu6KD

    Add 1tRvK0tMlG2B0DFJfCMSwwBkhm5 2KiB950B bytes 65 records
       from time: (00 52 5a 1c 89 d7 3d 2a) to time: (60 35 95 90 57 d5 3d 2a)

staged 1tRvKcrC6IaW782uRLhFRvhEdng

    Add 1tRvKfheRh2QcGK2Z4h3ZJd3ZJ2 1KiB138B bytes 21 records
       from time: (d0 6f 03 ad 5c d7 3d 2a) to time: (40 f3 f4 c1 5a d5 3d 2a)

$ zed lake log -p logs
empty log

However, when I commit them, they appear in the log (expected) but are not purged from the staging output (unexpected).
$ zed lake commit -p logs 1tRvK0kTROYpzbDo1UhlpMtu6KD 1tRvKcrC6IaW782uRLhFRvhEdng
1tRvzSl76ZmK9E80rRL8yHgdDdC committed

$ zed lake log -p logs
commit 1tRvzSl76ZmK9E80rRL8yHgdDdC
Author<EMAIL_ADDRESS>Date:   2021-06-03T18:29:38Z

    Add 1tRvK0tMlG2B0DFJfCMSwwBkhm5 2KiB950B bytes 65 records
       from time: (00 52 5a 1c 89 d7 3d 2a) to time: (60 35 95 90 57 d5 3d 2a)
    Add 1tRvKfheRh2QcGK2Z4h3ZJd3ZJ2 1KiB138B bytes 21 records
       from time: (d0 6f 03 ad 5c d7 3d 2a) to time: (40 f3 f4 c1 5a d5 3d 2a)

$ zed lake status -p logs
staged 1tRvK0kTROYpzbDo1UhlpMtu6KD

    Add 1tRvK0tMlG2B0DFJfCMSwwBkhm5 2KiB950B bytes 65 records
       from time: (00 52 5a 1c 89 d7 3d 2a) to time: (60 35 95 90 57 d5 3d 2a)

staged 1tRvKcrC6IaW782uRLhFRvhEdng

    Add 1tRvKfheRh2QcGK2Z4h3ZJd3ZJ2 1KiB138B bytes 21 records
       from time: (d0 6f 03 ad 5c d7 3d 2a) to time: (40 f3 f4 c1 5a d5 3d 2a)

But they're definitely no longer actually staged, since if I try to commit them again, I get the appropriate error.
$ zed lake commit -p logs 1tRvK0kTROYpzbDo1UhlpMtu6KD 1tRvKcrC6IaW782uRLhFRvhEdng
pool "logs": internal error: serialize transaction: empty transaction

Verified in Zed commit 793260a.
Now the final zed lake status check returns the expected "staging area is empty" message.
$ zed -version
Version: v0.29.0-391-g793260a1

$ rm -rf $ZED_LAKE_ROOT

$ zed lake init
using environment variable ZED_LAKE_ROOT
lake created: /Users/phil/logs

$ zed lake create -p logs -orderby ts:desc
pool created: logs

$ zed lake add -p logs snmp.log.gz
1uGs9ne08mbnJo7AcQNJwyopw8E staged

$ zed lake add -p logs pe.log.gz
1uGsADo4b6ELL18TtJfIPPJ5ndr staged

$ zed lake status -p logs
staged 1uGs9ne08mbnJo7AcQNJwyopw8E

    Add 1uGs9hqfsEW2cRhNGcKX3u7FFEC 2998B bytes 65 records
       from time: (00 52 5a 1c 89 d7 3d 2a) to time: (60 35 95 90 57 d5 3d 2a)

staged 1uGsADo4b6ELL18TtJfIPPJ5ndr

    Add 1uGsACOOtgMxi6iIL3J3d4AD3nl 1162B bytes 21 records
       from time: (d0 6f 03 ad 5c d7 3d 2a) to time: (40 f3 f4 c1 5a d5 3d 2a)

$ zed lake commit -p logs 1uGs9ne08mbnJo7AcQNJwyopw8E 1uGsADo4b6ELL18TtJfIPPJ5ndr
1uGsDIDa5PdsI3WiCjEzYiM4Pu3 committed

$ zed lake log -p logs
commit 1uGsDIDa5PdsI3WiCjEzYiM4Pu3
Author<EMAIL_ADDRESS>Date:   2021-06-21T19:18:58Z

    Add 1uGs9hqfsEW2cRhNGcKX3u7FFEC 2998B bytes 65 records
       from time: (00 52 5a 1c 89 d7 3d 2a) to time: (60 35 95 90 57 d5 3d 2a)
    Add 1uGsACOOtgMxi6iIL3J3d4AD3nl 1162B bytes 21 records
       from time: (d0 6f 03 ad 5c d7 3d 2a) to time: (40 f3 f4 c1 5a d5 3d 2a)

$ zed lake status -p logs
staging area is empty

Thanks @mattnibs!


Option to install it on Steam Deck?

Is your feature request related to a problem? Please describe.
I can't install it on Steam Deck, SD is based on Arch Linux, but the command for Arch Linux not working for Steam Deck.
Describe the solution you'd like
Would be nice to have some simple way to install on Steam Deck. Piper I can install from Flatpak but without libratbag it does not work.
Describe alternatives you've considered
I did not find any alternatives.
Additional context
No

Hello, I think it's on distro maintainers to make packages, especially on rolling distros.
For now, your only option would be do build from source: https://github.com/libratbag/libratbag/wiki/Building-from-source
If I remember correctly, Steam OS has an immutable base system that gets reset whenever Steam OS updates, which would mean you will have to reinstall libratbag after every system update, but I don't think there is anything we can do about this, sorry.


Received malformed Request headers in external processor http filter, I get wrong key for value.

If you are reporting any crash or any potential security issue, do not
open an issue in this repo. Please report the issue via emailing
<EMAIL_ADDRESS>where the issue will be triaged appropriately.
Title: *Received malformed Request headers in external processor http filter, I get wrong key for value. *
Description:

I have implemented the external processor filter in Go using grpc, the control comes to the Go lang grpc server, but the request headers are malformed like this:
pb.ProcessingRequest_RequestHeaders &{headers:{headers:{key:":authority"  3:"localhost:8801"}  headers:{key:":path"  3:"/v2/pet"}  headers:{key:":method"  3:"POST"}  headers:{key:":scheme"  3:"https"}  headers:{key:"content-type"  3:"application/json"}  headers:{key:"authorization"  3:"Bearer gateway"}  headers:{key:"user-agent"  3:"PostmanRuntime/7.36.3"}  headers:{key:"accept"  3:"/"}  headers:{key:"postman-token"  3:"1ce7e46c-73d6-4ce8-8b3a-f6097fd98157"}  headers:{key:"accept-encoding"  3:"gzip, deflate, br"}  headers:{key:"content-length"  3:"26"}  headers:{key:"x-forwarded-proto"  3:"https"}  headers:{key:"x-request-id"  3:"515b2ebf-6870-4369-a709-0bbf385781e1"}}}
Instead of value key I get "3" as the key.

Repro steps:

I have followed similar to this approach https://github.com/ekkinox/ext-proc-demo/tree/main

Note: The Envoy_collect tool
gathers a tarball with debug logs, config and the following admin
endpoints: /stats, /clusters and /server_info. Please note if there are
privacy concerns, sanitize the data prior to sharing the tarball/pasting.

Admin and Stats Output:

Include the admin output for the following endpoints: /stats,
/clusters, /routes, /server_info. For more information, refer to the
admin endpoint documentation.

Note: If there are privacy concerns, sanitize the data prior to
sharing.

Config:

Include the config used to configure Envoy.

Logs:

Include the access logs and the Envoy logs.

Note: If there are privacy concerns, sanitize the data prior to
sharing.

Hmm, not sure. Seems like a proto compilation issue. cc @kyessenov @yanavlasov

I am in the exact same situation. It's more that cosmetic in my case. My external process sets a new header with key/value. Unfortunately envoy receives the new header and sets the key put the value is always an empty string. I agree it seems like a proto issue. How can I make sure my proto match the Envoy version. I am using v1.29.1

Ok I rolled back to version 1.25.11. The problem is gone now cc:@divyasriram90
I would like to understand what we have to do to get external processors to work with 1.29.


Sync from CLI

This PR adds a CLI command to sync a folder (similar to Lychee-Sync)
ImportController is adjusted to be callable from CLI.
Support for some for video formats is added.

Codecov Report

Merging #427 into master will increase coverage by 0.32%.
The diff coverage is 3.84%.

Can you check how this compares with #237 ? :)

Can you check how this compares with #237 ? :)

It's similar. #237 imports photos using the Lychee settings (e.g. for delete imported files, duplicates etc). This one is meant for syncing, i.e. never deletes imported files and always skips duplicates.
One could keep import and sync separate, but could also merge them.


Add Sign Up page

The Sign Up page will allow users to create new organization accounts.  During sign-up, a randomly generated slug will be created for the account and "magic link" will be set as the first identity provider to allow the user to login to the newly created account.

Sorry clicked approve by accident, was too happy seeing a sign up page 🤣.

@AndrewDryga cleared your approval just so there's no accidental merge before everyone is good with the PR 😄


This patch resolves #53

To not break anyone who is currently using /conf.d I used the same approach for /checks.d.   It would be cleaner in my opinion to have 1 directory like /opt/docker-dd-agent that get's mounted into the container at /docker-dd-agent and the entrypoint script looks for /docker-dd-agent/conf.d/*yaml and /docker-dd-agent/checks.d/*py and copies them to their proper location.
Let me know if you want me to do change it so only one mountpoint is required on the docker container.

Thanks for the contribution @someword! This will be useful to people running custom checks. I'll test it shortly and it should be merged today.


Setting: Bombchu Bag, Drops, + Shared Bombchu Bag

Title is self-explanatory. Add an item called "Bombchu Bag" which let's you use Bombchus without needing to have a Bomb Bag; the first Bombchu Bag you find starts you with 10-20 (probably 20?) chus. This let's you then also get Bombchus from cutting grass, pots, etc that normally drop bombs. This item would be applicable to both games, so MM could also then have bombchus without a Bomb Bag but because it's a Bombchu Bag it would then put it into logical use.
As for Bombchu Bowling handling, OOTR ended up deciding allowing either a Bomb Bag OR Bombchu Bag to play the mini-game. Likewise, for MM's Honey & Darling, I'd imagine having a Bombchu Bag would let you do Day 1 but not Day 2 (if that's possible to do).
Finally, with Bombchu Bag being an item for both games you can do Shared Bombchu Bag. But only if Bombchu Bag is enabled.

[ ] Implement Bombchu Bag for both games, Bombchu Bag (OOT) and Bombchu Bag (MM)
[ ] Add Bombchu Drops
[ ] Adjust Bombchu Bowling & Honey & Darling Day 1/2
[ ] Shared Bombchu Bags

Relevant OOTR PRs:
Bombchu Bag initial implementation: https://github.com/OoTRandomizer/OoT-Randomizer/pull/1781
Improved Bombchu Bag Model: https://github.com/OoTRandomizer/OoT-Randomizer/pull/1972

Done as of https://github.com/OoTMM/OoTMM/commit/915faa123b0796f9d618e13ff8096220108d24ea


Using routes

Hello!
When my site fires a 400 error and renders within the application layout I get this error:
Completed 500 Internal Server Error in 212ms (ActiveRecord: 0.0ms)
Error during failsafe response: undefined local variable or method `root_path' for #<#<Class:0x007fd0c0409938>:0x007fd0c24946d0>

I have a _header.html.erb that uses the link_to tag with the root_path as the link. Is there a way around this? I also tried changing the default layout for 4xx errors but it still used my application layout.
Thanks for creating this awesome gem and thanks for the help!
-- Tad

Hey Tad,
Thanks for taking the time -- the quickest fix should be to use main_app.root_path. Do you have a link to the repo I could look at? As you can imagine, catering for all setups can take time
R

Ok sweet... So could I use this for all routes then? Just add main_app to the beginning of my routes?
What is it that makes that necessary?
And unfortunately no. I'm working on a site for a client and have my repo up on bitbucket for privacy. Really sorry cause I know how much it would help. However I can attach some code:
_header.html.erb:

<%= link_to "Innovators", startups_path %>

<%= link_to "Mentors", mentors_path %>

<%= link_to "Investors", investors_path %>

Blog

            About

<%= link_to "Team", team_path %>

<%= link_to "Contact", contact_path %>

<%= link_to "Press", press_path %>

application.html.erb:
<!DOCTYPE html>

  <title><%= full_title(yield(:title)) %></title>
  <meta charset="UTF-8">
  <meta name="viewport" content="width=device-width, initial-scale=1">
  <%= favicon_link_tag 'favicon.ico' %>
  <%= stylesheet_link_tag    'application', media: 'all', 'data-turbolinks-track' => true %>
  <%= javascript_include_tag 'application', 'data-turbolinks-track' => true %>
  <%= csrf_meta_tags %>
  <%= render 'layouts/shim' %>

  <%= render 'layouts/header' %>
  <%= yield %>
  <%= render 'layouts/footer' %>

application.rb: (I didn't need all the other variables)
config.exception_handler = {
	    db:   	false, #-> defaults to :errors if true, else use :table_name
	    email: 	false, #-> need to integrate

	    layouts: {
	      '400' => nil,
	      '500' => 'exception'
	    },
    }

Unfortunately, you shouldn't use it for all routes; it's meant for use within an engine (like we have set up here).
The issue we have is that since ExceptionHandler uses the main "application" layout, it will expect any references to routes etc to be covered by the engine.
Since we are using isolate_namespace, any routes are expected to be defined within the engine. The way around it is to prepend main_app to the route, allowing the engine to reference the routes of the main application.
We did include include Rails.application.routes.url_helpers, which worked for us.

The way to fix it is to make sure that any layout routes you use with ExceptionHandler are prepended with main_app:

<%= link_to "Innovators", main_app.startups_path %>

<%= link_to "Mentors", main_app.mentors_path %>

<%= link_to "Investors", main_app.investors_path %>

Blog

        About

<%= link_to "Team", main_app.team_path %>

<%= link_to "Contact", main_app.contact_path %>

<%= link_to "Press", main_app.press_path %>

```

I know it's tedious, but it's only for the "main app" layouts used by the gem.
--
I'd also like to know what you mean when you said the gem wouldn't recognize that you'd changed the 400 layout?
Thanks for the prompt reply!

Oh ok thank you so much for the links! I'm newer to rails and was unaware of engines! It all makes a lot more sense now.
As for the 400 layout I changed the layouts config to this:
config.exception_handler = {
        db:     false, #-> defaults to :errors if true, else use :table_name
        email:  false, #-> need to integrate

        layouts: {
          '400' => 'exception',
          '500' => 'exception'
        },
    }

Then when my server gave a 400 it was still using 'layouts/application' as its layout.
Processing by ExceptionHandler::ExceptionController#show as */*
  Rendered exception_handler/exception/show.html.erb within layouts/application (0.1ms)

Again thanks for all of your help!

No problem, this is something I am going to have to test for the new version.
Have you tried the recommendation? The most important thing is to get it working
R

Yup it's all working now!

Any suggestions for testing my 500 page though?

Nice! Love the little face

Any suggestions for testing my 500 page though?

Yeah that's simple - just make a loop in a view:
#app/views/x.html.erb
<% .each do |g| %>
<% end %>

This will give you a 500 error ;)

Awesome! Thanks so much!


Quick Add to Inbox

Hi, thanks for making this app!
I was wondering if you'd accept the idea of a quick-add feature which drops a task directly to the todoist inbox?

Done https://github.com/arma7x/k-todoist/commit/b90291c2cbce740bb9494209a787e544fcdc45d1

Wow excellent! Thanks for doing that so fast!


Add isInThePast and isInTheFuture to LocalDate assertions

Check List:

Fixes one of the tasks in #2932
Unit tests : YES
Javadoc with a code example (on API only) : YES
PR meets the contributing guidelines

Two things I am unsure about:

The correctness of the @since version
ShouldBeInTheFuture/ThePast error message contains the class name (java.time.LocalDate). This seems to be the behaviour of the MessageFormatter handling dates classes. Is the error message ok like this?

Integrated thanks @StefanBratanov for the good work!
I did some minor changes before integrating it

no need to use @DisplayName this is now done automatically with a junit custom test naming config
updated the javadoc as the example for isInTheFutture  was not that nice (came from date assertions that I also updated)

Integrated thanks @StefanBratanov for the good work! I did some minor changes before integrating it

no need to use @DisplayName this is now done automatically with a junit custom test naming config
updated the javadoc as the example for isInTheFutture  was not that nice (came from date assertions that I also updated)

Thank you @joel-costigliola . Will keep in mind for any future contributions. :)


Sol for sorting an array of 012 in C

Description
solution code in C for sorting an array of 0s,1s and 2s is added in Miscellaneous Problems Folder
Fixes #1768
Replace issue_no with the issue number which is fixed in this PR
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Documentation Update

Checklist:

[x ] My code follows the style guidelines(Clean Code) of this project
[x ] I have performed a self-review of my own code
[x ] I have commented my code, particularly in hard-to-understand areas
[ ] I have created a helpful and easy to understand README.md
[ ] I have made corresponding changes to the documentation
[x ] My changes generate no new warnings
[ ] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works

@Vidhi6340 test cases?

@Vidhi6340 also add the code to proper folder

@Vidhi6340 test cases?

@Amit366 by test cases u mean I need to take an input T (number of test cases) in the main program and then run the program T times?Could u please clarify this.

@Vidhi6340 checkout other merged PRs


bugfix: avoid memory leak in downstream (http1 or http2)

Issues associated with this PR
Try to fix #379
Sign the CLA
Yes
Solutions
Since the fasthttp and http2 will hijack the io loop. We have no chance to bind the close|reset event to our event callback. We could think when we yield from the fasthttp and http2 io loop, the connection is finalized and we can return directly and reduce the number of connections.
UT result
N/A
Benchmark
N/A
Code Style
N/A

HTTP/1 and HTTP/2 support IO management in recent changes: #393  #396. Consider out-dated.


Storage Backend: Amazon S3

Implement Storage Backend Amazon S3

This has been added when #184 was merged.


Add topic jump with hotlinks

Add a table of contents for headers within chapters and another table of contents for the overall document with a quick jump to; in order to improve readability.

wao very cool, thank u


Changing Font Size and Type

How to change the font size of the text on the screen its too small. How could I change the font to Helvetica Neue.
Thanks.

You can override the default font by changing https://github.com/jackyzha0/quartz/blob/cb38667c1df7d752a1a9ef45505a7e106ff03ef8/assets/base.scss#L10 or https://github.com/jackyzha0/quartz/blob/cb38667c1df7d752a1a9ef45505a7e106ff03ef8/assets/base.scss#L15 in base.scss. You can set font size there as well

Thanks Alot.
Can I have my quartzs website on the list too. The link you provided for adding sites to the list wasnt working
https://a2itnotes.github.io/quartz/
Thanks

Done :)

What can I do if i want to just change Chinese font type?


cloudflare DDNS wont update after installed and ran for the first time

Hi all.
So i noticed when it first installs, it will update.. however if the settings are changed for example TTL to 5minutes or proxy to OFF or ON, on cloudflare portal it will not update..
something is wrong or did i missing something here.

as it appears it will only update the settings if the IP address has changed.


[JSON API] Multiple includes not working

When issueing a request like: /items?include=subitems,subitems.something or /items?include=subitems,otheritems nothing gets included. /items?include=subitems works fine.
Meanwhile JSON API also states that a request like /items?include=subitems.something should include records for both subitems and something. Fortune currently only adds something.
See: http://jsonapi.org/format/#fetching-includes

Are you sure? There is a test for this: https://github.com/fortunejs/fortune/blob/master/test/integration/serializers/json_api.js#L197-L211
So I'm not sure if this is a problem.

It might also have to do with ember data url encoding the comma. The stuff in the internal Fortune include array seems wrong (i.e. [[item,item],[subitem]] for item,item.subitem). I'll have time to look at it tomorrow again.
It indeed states in the greyish box that parent relationships should be included.

URI encoding seems highly likely to be the problem here, I'll investigate that, as well as a fix for the nested relationships.

Yea, I checked some more examples where ember-data uses a comma in the request, and it always encodes it (which seen as it's a reserved character is expected I guess). I can test some more examples if you want and give you the structure of the include array (from the documentation it was not entirely clear how it should look).

@nickschot I've already come up with a fix, will push it and add test for it in a few moments, I'm a bit busy right now but the solution was to call decodeURIComponent within the serializer (fix #1) and to manually expand nested includes (fix #2).

Excellent! Don't hurry for me! ;-)

Fixed in commit https://github.com/fortunejs/fortune/commit/abf205fc6c64542382faf7270cb3491a181ed91d.

Trying it out now!

Working excellently now. Nice fix.

:+1:


FindKeyNoCkaLabel

FindKey not to use CKA.CKA_LABEL in case label is set on certificate, but not in the public/private keys.

I've included this change in d81acdc88af521cac590478a8f8147f3af919732 along with tests, so I'll close this without merging.


Move host's hostname_required? to the class level

Currently, this method is only used to validates_presence_of :hostname.

Move hostname_required? to the Host class

Add instance method to delegate to the class method.
Subclasses can override this method to implement their own rules.

hostname is required, don't clear it during host provisioning. …

When provisioning wants to create a new host, it duplicates the existing host's fields and clears most fields that can be discovered later.  We can't clear the hostname as it's now a required field all of the time.

Apparently HostOpenstackInfra doesn't require hostname. …

Previously, only HostVmware required hostname, so this changes the behavior back as it was before for this class.

Fixes #2137, followup to #2136
Note, I "fixed" the host provisioning trying to "clear" the hostname field in the new destination record.  See the second commit. cc @gmcculloug @brandondunne
For the 3rd commit, it looks like HostOpenstackInfra didn't previously require a hostname so it needs to override the hostname_required? otherwise the existing juno spec fails due to missing hostname in the validation.  Please help confirm this is ok @Ladas @rwsu @blomquisg.
EDIT:
Without the 3rd commit, it was failing with this:
  1) EmsRefresh::Refreshers::OpenstackInfraRefresher will perform a full refresh
     Failure/Error: Host.count.should                        == 6
       expected: 6
            got: 0 (using ==)
     # ./spec/models/ems_refresh/refreshers/openstack_infra_refresher_rhos_juno_spec.rb:40:in `assert_table_counts'
     # ./spec/models/ems_refresh/refreshers/openstack_infra_refresher_rhos_juno_spec.rb:29:in `block (3 levels) in <top (required)>'
     # ./spec/models/ems_refresh/refreshers/openstack_infra_refresher_rhos_juno_spec.rb:14:in `times'
     # ./spec/models/ems_refresh/refreshers/openstack_infra_refresher_rhos_juno_spec.rb:14:in `block (2 levels) in <top (required)>'

See the log here

Closing in favor of #2223.  :fire: :scissors:


Transition effects

I've been thinking on how to manage transitions like for example crossfade or so on in frame.js, I think it's an enough cool feature to be easy to use instead of ending using a lot of flash everywhere to change between scenes :)
As frame.js is timeline based instead of node based, the "graphical view" of the solution could be something like:
var effects = [
    {"name": "FX1", "start": 0, "end": 1000, "layer": 0, "id": 0 },
    {"name": "FX2", "start": 750, "end": 2000, "layer": 1, "id": 1 },
    {"name": "FXTransition", "start": 750, "end": 1000, "layer": 2, "id": 2 },
];

So the transition effect will affect during 250ms.
The transition effect should have two parameters: List of effects from the beginning of the transition, and list of effect at the end of the transition.
Right now for this example one will be the FX1, and the other one the FX2.
In summary what it will do is to create one FBO for each effect, and when it appears in action (.start()) it will disable the render to screen on both attached effects, and it will start using their FBO to do the transition using some shadermaterial or whatever you want to implement.
I've a proof of concept but before going further I wanted to check what you think about it.
I've made basically:

Include id parameter for each module instance, as we could link between effects, not just for this kind of effects but for many other kind.
Include a getModuleInstanceById or so function in frame.js
Create a TransitionModule with basic mix shader between the two fbo.
Include a input parameter on each effect in the transition similar to: parameters.input.renderToTexture=false. This parameter will be changed by the transition effect. And a parameter.output.fbo
Modify the doFrame in both effects to render to screen or to fbo according to the .renderToTexture parameter

So this is what I was doing yesterday and I got it working. Of course the part of including the extra input/output parameters and the doFrame switching between fbo or screen should be done in a cool fancy way so you could include each effect in a transition without need to modify it code.
Let me know what you think

Closing this because since 6449949b818bd36db2b151872e2f6f985ebcc15c the code base is now different. Open a new PR is needed.


Listeners create folder with null path when path is evaluated as a property value inside a groovy script when run from CLI

Expected behavior
${__groovy(props.get("baseDir")+"/results/latency_data.csv")} evaluated to correct file path like /Users/username/projects/qa-perf/performance/jmeter/results/latency_data.csv
Actual behavior
${__groovy(props.get("baseDir")+"/results/latency_data.csv")} evaluates to null/results/latency_data.csv

Steps to reproduce the problem
Prerequisites:
Having a test plan (attached here bug.jmx.zip), containing:

setUp thread group, which contains
--- init JSR223 Sampler
main thread group with
--- Dummy sampler
--- Simple Data Writer (or any listener)
tear down thread group with
--- JSR223 Sampler

init JSR223 Sampler having this code:
import org.apache.jmeter.util.JMeterUtils;
import org.apache.jmeter.services.FileServer

// Do not show this sampler in report
SampleResult.setIgnore()

// Set base directory. Mainly needed for local run
String baseDir = FileServer.getFileServer().getBaseDir()
JMeterUtils.setProperty("baseDir", baseDir)
log.info("Base directory set to " + props.get('baseDir'))

Simple Data Writer having this as a filename:
${__groovy(props.get("baseDir")+"/results/latency_data.csv")}
Tear down JSR223 Sampler having this code:
// Do not show this sampler in report
SampleResult.setIgnore()

String filePath = props.get('baseDir') + '/results/latency_data.csv'
log.info('filepath=' + filePath)
File file = new File(filePath);
if(!file.exists()) {
	log.error('FILE NOT FOUND')
}

// Pasre file
//...

Reproducing

Run test plan from GUI.
Observe /Users/username/projects/qa-perf/performance/jmeter/results/latency_data.csv file is created and no errors in the log
Launch terminal in baseDir folder (for me is /Users/username/projects/qa-perf/performance/jmeter)
Launch the same test plan via CLI using command
jmeter -LINFO -n -j Buggy_Test_Plan.log -t bug.jmx -l Buggy_Test_Plan.jtl
Observe unexpected behavior:

File /Users/username/projects/qa-perf/performance/jmeter/null/results/latency_data.csv is created
ERROR in logs because JSR223 sampler could not find the file.
INFO o.a.j.r.ResultCollector: Folder at /Users/username/projects/qa-perf/performance/jmeter/null/results was created

I assume that folder null is created because this happens before set of the variable baseDir, thus props.get("baseDir") returns null.
But this flow does work through GUI mode, so should be a bug.
JMeter Version
5.6.3
Java Version
openjdk 17.0.11 2024-04-16
OS Version
MacOS Sonoma 14.5 (also reproducable on Alpine Linux in Docker container)

I think you are out of luck here. The listeners are initialized before any sampler has been run and I believe the filename is set on initialization.
I wonder though, why you took the route through the groovy function to get the property instead of using the P-function. It would not work in this case, either, but is seems more straightforward.
Things, that should work include:

using Fileserver inside a groovy function to set the filename (instead of taking the route via a runtime set property)
using a property set via command line

Ok I understand now. Thank you for explanation. Issue could be closed now.


gallery not starting from ajax reload

sending  request via php script from ajax  $.ajax({		type: "POST",
url: "/includes/gallery.php", Result comes back fine with photo's and css but the
problem is that it comes back but how would you reload the jquery.jcarousel.min.js ?
thanks

if jcarousel initialized then you may call $('.jcarousel').jcarousel('reload').
if jcarousel not initialized then you should initialize it after ajax

hi snake this is the page
http://www.exclusivepaperbags.com/jcart/products/express_flexo.php
choose format 18x8x25 the first one
choose Quantità
then below Colore Fondo click on white, now this is where ajax goes and uses the jcarousel to load
Stampa Flexografica choose 1 color now it calls ajax
the pictures of the colors associated with white which are five.
$.ajax({
type: "POST",
url: "/includes/offsetc.php",
success: function(data){
		$("#mycarouseltwo").html(data);
		$('#mycarouseltwo').jcarousel('reload', {
animation: 'slow'

}
);
this is the code I was using but still not working
thanks tony

Looks like you use old version jcarousel. Can you update it?

ok

downloaded the jcarousel-master from here and updated the jquery.jcarousel.js and jquery.jcarousel.min.js

now the above gallery is not working after I updated the above files

Now you should look documentation and edit you code. In new version was make significant changes.
http://sorgalla.com/jcarousel/examples/ajax/ it example about reload jcarousel after ajax

Closing for now, feel free to reopen if needed.


Is it possible to use the nodes GPU?

Hello,
I'm quite new to the use of clusters, etc. and I was wondering if this version of XGBoost was also integrating a multi GPU distribution layout over the cluster's nodes. I must use slurm to manage my jobs, so I can request a number of nodes with a number of GPU per node, etc. But I don't know if it's more interresting to use one node with x amount of GPUs or N nodes with Ny CPU and Nx GPU, etc.
So as in the title: "Is it possible to use the nodes GPU?". And if so, how?
Have a nice day,
Vincent

Hi @2010019970909, we have preliminary support for this by setting:
bst = train(
    {..., "tree_method": "gpu_hist"},
    num_gpus_per_worker=1
)

Hi @2010019970909,
as Richard pointed out we support GPU usage by the usual means of XGBoost (i.e. setting tree_method="gpu_hist" in the parameters).
This does work with distributed workers, too.
Working with more than one GPU per node should work out of the box, too (by setting num_gpus_per_worker=1). Which method you prefer depends a bit on the size of your training job, node stability, and dataset. I'd suggest going with more workers on smaller machines first, except when you're training on very large setups (say, > 32 GPUs).
I'll close this issue for now, but please do let us know if you need help with the setup or have any other questions. We are more than happy to help you be successful with distributed GPU training, as we were planning be looking into this more thoroughly in the coming weeks anyway.

Hi @krfricke,
I should have closed this ticket before. My usecase did slightly change and the training on the dataset we are using is taking roughly 5 minutes on one node with 3 GPUs (one P100, and two 1080 Ti). Also, I only have two nodes available so it's less interesting for the moment. I'll continue to use XGBoost on this same project next year, with maybe more data (so I might take a look at further 'parallelisation' in the future). Thank for your replies @richardliaw and @krfricke.
Regards,
Vincent


Support stream in CUB

This now works (can be verified by nvvp):
import cupy as cp

dtype = cp.complex128
shape = (200, 1000, 1000)
x = cp.random.random(shape).astype(dtype) \
    +1j*cp.random.random(shape).astype(dtype)

stream = cp.cuda.Stream()
with stream:
    y = x.sum()  # runs on "stream" instead of the default stream

Jenkins, test this please.

Jenkins CI test (for commit 548e6d93b71b0486e6f5d9bc0a1059d4d5541afa, target branch master) succeeded!

LGTM. Thank you for the PR!

Thanks for the fast turnaround, @asi1024!


Revert "remove send_rate dataset for now"

Summary | Résumé
Reverts cds-snc/notification-terraform#1403
This should work now that we've increased the instance class.
Add a new dataset so we can have a send rate dashboard in QuickSight.
~NOTE: releasing this to production will probably trigger the large refresh job for this dataset. Our system will be ok with this, but we probably don't want to do it at the same time #1478 releases (otherwise we might have 2 of our instances busy for an hour?). So let's not merge this PR until after #1478 is released.~
Related Issues | Cartes liées
Test instructions | Instructions pour tester la modification
Verify dataset on staging
Release Instructions | Instructions pour le déploiement
Verify dataset on production.
Reviewer checklist | Liste de vérification du réviseur

[x] This PR does not break existing functionality.
[x] This PR does not violate GCNotify's privacy policies.
[x] This PR does not raise new security concerns. Refer to our GC Notify Risk Register document on our Google drive.
[x] This PR does not significantly alter performance.
[x] Additional required documentation resulting of these changes is covered (such as the README, setup instructions, a related ADR or the technical documentation).

⚠ If boxes cannot be checked off before merging the PR, they should be moved to the "Release Instructions" section with appropriate steps required to verify before release. For example, changes to celery code may require tests on staging to verify that performance has not been affected.

We should also take a look at doing join tables using the Quisight plugin, now that the bug for it has been resolved. This could open up the door for us to do refresh instead of complete updates and be more efficient, if I understand the issues and chain of events correctly.

We should also take a look at doing join tables using the Quisight plugin, now that the bug for it has been resolved. This could open up the door for us to do refresh instead of complete updates and be more efficient, if I understand the issues and chain of events correctly.

I don't think that would work here, where we're doing a double groupby, but definitely we should be able to change the way we do the notifications dataset.


fix: store newly-created entities and actions in app memory

During .dialog import, we check to see if a given action or entity has already been created by looking them up in app memory (TrainDialog's actions or entities props).  Invoking the create...ThunkAsync does not update props while importing a set of dialogs; accordingly, without this change, the import will try to create the same action/entity multiple times, if the same action/entity is referenced in multiple dialog steps.
Tested via import of the product key intent v4 files.

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

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix compilation warnings on emacs 29.0.60

This fixes a couple of warnings on emacs 29.0.60. The first is an unescaped single quotation.
The other warning come from radian--lsp-enable where using # issues a warning when the functions are not defined (they would be if the corresponding packages are disabled).
There is another deprecation warning for using point-at-eol which should be changed to pos-eol, but I don't know if you want to define this function in earlier versions of emacs or maybe use compat.

Fair enough.

For point-at-eol, I'm fine with compat or alternatively using a backwards compatibility definition as I do for some other functions.


[meta] Input System

Connect the Gamepad API and other input systems (mouse, keyboard) to the Unity Input system so whatever thing working on Unity, must work on the web without adaptations.

Related to #124. We should provide those mappings to make JS work like Unity (not the other way around).

These are the things that this issue should track:

[ ] Axis directions should be those provided by Unity.
[ ] Name of the controller buttons should be those used by Unity.
[ ] Name of the keyboard keys should be those used by Unity.
[ ] Mimic Unity behaviour when dealing with VR controllers in design-time.
[ ] Provide force-feedback/vibration through GamePadHaptic extension to the GamePad API.

Fixed with https://github.com/mozilla/unity-webvr-export/commit/b0a1307d5f722f066f3a0f5c69ae4c889fee9457


Allow generation of dynamic graphs

Uses the dynamic queries functions to allow the generation of dynamic graphs.
Addresses #8

ping?

Sorry, have I missed a message? I'm actually in Spain heading to Russia at
the moment so have bad communication
On 21 Jun 2016 11:30 a.m., "Mathias Baumann"<EMAIL_ADDRESS>wrote:

ping?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ezbz/graphitus/pull/20#issuecomment-227388616, or mute
the thread
https://github.com/notifications/unsubscribe/AF61ipFSqU2ZUALA9W-8lIviH8-UloCtks5qN69HgaJpZM4BV64K
.

Well, I thought I Ping this PR which is open since 2013 and no response from the maintainer ;)

Ah thanks! :) please do close if you want. You resolved my issue then.
On 21 Jun 2016 12:00 p.m., "Mathias Baumann"<EMAIL_ADDRESS>wrote:

Well, I thought I Ping this PR which is open since 2013 and no response
from the maintainer ;)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ezbz/graphitus/pull/20#issuecomment-227395295, or mute
the thread
https://github.com/notifications/unsubscribe/AF61ioAV0cExiPZFCN_FoM8BYVXpeZY_ks5qN7YrgaJpZM4BV64K
.

I am confused.. so you don't want the feature?

Sorry, I'm also confused: I only contacted you once, and your resolved my
issue at the time. I haven't worked on that product for years, so there is
nothing required for me.
Do you need me to close something this side?
Thanks,
Bernie
On 27 Jun 2016 11:02 a.m., "Mathias Baumann"<EMAIL_ADDRESS>wrote:

I am confused.. so you don't want the feature?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ezbz/graphitus/pull/20#issuecomment-228704273, or mute
the thread
https://github.com/notifications/unsubscribe/AF61il94YUH0hyQ9Akugj-3QNcNqB7xbks5qP5_IgaJpZM4BV64K
.

My goals is getting this PR merged. I assumed you're the maintainer that can make that happen because you responded ;)

Ah no, I have no idea even what you're actually asking, and am not involved
with graphite, graphatitus, or even with computers at the present time.
On 27 Jun 2016 11:20 a.m., "Mathias Baumann"<EMAIL_ADDRESS>wrote:

My goals is getting this PR merged. I assumed you're the maintainer that
can make that happen because you responded ;)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ezbz/graphitus/pull/20#issuecomment-228707800, or mute
the thread
https://github.com/notifications/unsubscribe/AF61inzuZ_XJq-xkoBXOPFTsgI6roCH3ks5qP6PlgaJpZM4BV64K
.

In that case... you don't need to do anything ;)

@ezbz ping

Alright, I guess the repo owner doesn't care anymore, so I am closing this