Two new features request

We require two new features and help with integration of updated code.
First feature is integration of version transactions (https://github.com/bmresearch/Solnet/issues/360)
Second feature is implementation of high priority fees (https://solana.com/developers/guides/advanced/how-to-use-priority-fees)
I have created job offering on Upwork. If anyone is interested in adding two new feature for pay.
https://www.upwork.com/jobs/~017f60fcc84ecdc5f7

@Joter271 you can contact me on any of my social links


Delete as many containers as possible

Description of problem
In Sandbox#Delete,
	for _, c := range s.containers {
		if err := c.delete(); err != nil {
			return err

If single container deletion has error, we return.
Expected result
I think we should delete as many containers as possible.
Actual result

Maybe. In which case, make sure we still report the error for the one(s) that failed.

@grahamwhaley
There is some error aggregation code from kubernetes.
Is it Okay if I borrow that for error reporting ?

if the code is under a suitable license (ideally also Apache2), and the way it collects/returns errors fits in with the kata logger structure, probably fine. Do you have a link for @jodh-intel et. al. to check @tedyu ?

Here is the code:
https://github.com/kubernetes/kubernetes/blob/master/staging/src/k8s.io/apimachinery/pkg/util/errors/errors.go

I don't see the value in importing this code - we want to return the errors for all failures to allow the caller to log that and handle the failure scenario. Hence can't you just concatenate the error messages in some simple way (errors.Wrapf() or fmt.Errorf("%v", err) or similar)?


ci, badges, setup

Signed-off-by: Du Cheng<EMAIL_ADDRESS>

I just add _version.py under sequential to make it more consistent with other libraries. And I remove some empty lines in ci. It looks good from my side.


Price per Token (in ether) >= 1?

allow:

not allow:

Fixed: https://github.com/ethereumproject/TokenMint/commit/e3393ccf90a695d3edc45f0b5810db48341df8b0


Convention: prepend underscore on variables

Should we prepend variables like:
class Foo {
  _foo;
  _bar;
}

The way to work with components is to create a setter for inputs, that way we can keep or component updated with changes, for example:
@input
set foo(value) {
  this._foo = value;
  // do something
}

So far so good, but you can use this like:
<foo #foo></foo>
{{foo._foo}}

So if we decide to go with underscores, we will have that syntax, our we could create a getter.

Personally, I like underscore for the things that have a setter (not for everything) and for the usage like I put at the end, well, use the underscore as well.

Yeh, I was mainly adding those underscores for cases where we need a setter. But I'm fine with using any prefix / suffix for this case.

I am good with underscores, classic convention, but only when it has a setter for it, if it doesn't have a setter, I vote for no underscore at all. For example, in your pagination, _pages has no setter.

I think that we are on the same page here.

Ok, so no underscore for pages in there. And if the user wants to get a ref of our element and use some internal values, he will need to check the source to see if what he wants to use has underscore or not. Not really fond of forcing to read the source, but I don't like having to create getters to just for that.

Yeh, you are raising a valid point which I kind of bumped into while doing testing. Maybe having "public" getters wouldn't be such a bad idea, after all?

get pages() { return _pages; }
Not a big deal, just extra code to maintain. But will definitely help end users with all their use cases, not having to use _ if they do the ref thing.

But will definitely help end users with all their use cases, not having to use _ if they do the ref thing.

It would make our templates "prettier" as well.

So question - if we have
get bar() {
  return this._bar;
}

@input
set bar(value) {
  this._bar = value;
  // do something
}

Then will it be consumable like
<foo #foo></foo>
{{foo.bar}}

?
It seems a bit awkward to have to consume with an underscore prefix IMO, since a lot of conventions use it as a way of denoting private properties.

Yes, that is absolutely correct.

OK, so if I understand things correctly, the summary here could be: if there is a sett for the foo variable then:

store the actual value in the _foo variable (and mark it as private?)
create a corresponding getter

Is it about right?

Yes to all. I will add the summary at the top of each issue in a bit.

Well, lies, I will create a FAQ document with all the conventions.

Well, lies, I will create a FAQ document with all the conventions.

Cool! Like WIKI? We should share those with the ng team at Google at some point so we can check with them / enforce those for other widget libraries as well.


Introduce DuplicateAnnotationListing bug checker

Problem
Duplicate entries inside annotation listing are a common mistake that should be flagged.
Description of the proposed new feature

[x] Support a stylistic preference.
[x] Avoid a common gotcha, or potential problem.
[x] Improve performance.

Introduce a new BugChecker to flag duplicate entries as this is a common mistake. Similar to LexicographicalAnnotationListing, the new BugChecker should identify and handle duplicate entries to improve code quality and reduce potential errors.
I would like to rewrite the following code:
            @Foo
            @Baz
            @Baz
            A duplicateAnnotations();

to:
            @Foo
            @Baz
            A duplicateAnnotations();

Considerations
👉 To consider: A duplicate entry is a duplicate pointer reference and not just the same name.
👉 Open question: Are there situations where we do not want to flag this ?
Participation

[x] I am willing to submit a pull request to implement this improvement.

Additional context
See this comment.

Thanks for filing this issue @mohamedsamehsalah! One key point of feedback: I think we're looking for an analog of LexicographicalAnnotationAttributeListing, i.e. a check that replaces @A("foo", "bar", "foo") with @A("foo", "bar").
(Repeated annotations such as @Baz() @Baz() are rejected by the compiler, unless they're declared @Repeatable, in which case the repetition likely is intended.)

W.r.t. your considerations:

I'm not sure how pointer references factor in, but since only primitives, enum values and strings can be annotation arguments, the invariant holds that a value is a duplicated if and only if it is equal to some other value inside the same annotation attribute.
There are likely cases we'll want to exclude, but none come to mind right now. We might want to run a first version against the Picnic code base to assess what is flagged.


better active reg

Fixes https://github.com/EnzymeAD/Enzyme.jl/issues/841

Codecov Report
Patch coverage: 72.72% and project coverage change: -3.97 :warning:

Comparison is base (69635f4) 77.96% compared to head (e8f8cb0) 74.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #845      +/-   ##
==========================================
- Coverage   77.96%   74.00%   -3.97%
==========================================
  Files          18       18
  Lines        8356     8317      -39
==========================================
- Hits         6515     6155     -360
- Misses       1841     2162     +321

Impacted Files
Coverage Δ

src/compiler.jl
76.45% <72.72%> (-2.85%)
:arrow_down:

... and 11 files with indirect coverage changes

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


Replace panic! calls with returning a Result type

Removes the unused SourceMap struct definition
Closes #17

bors r+


Netlify builds using the Xenial build image will fail after November 15, 2022

What happened:

Ref https://answers.netlify.com/t/please-read-end-of-support-for-xenial-build-image-everything-you-need-to-know/68239
What you expected to happen:
We have to switch to another image soon.
Anything else we need to know?:
@falcosecurity/core-maintainers anyone can help?

/help

@leogr:
This request has been marked as needing help from a contributor.
Please ensure the request meets the requirements listed here.
If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-help command.

In response to this:

/help

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

I checked and I don't have access to the settings on Netlify, and I didn't find a way to configure the builder image in the netlify.toml.
Source: https://answers.netlify.com/t/please-read-end-of-support-for-xenial-build-image-everything-you-need-to-know/68239/14

Hi, I can help with this.
I wanted to double check tho, @leogr it looks like you have access to the Netlify account. If not, please let me know.

This has been updated & tested.

There is a Ruby version warning now, but the site still builds correctly that update can probably be dealt with in a follow up issue.
4:03:22 PM: ---------------------------------------------------------------------
WARNING: Your site had a deprecated version of Ruby pinned (2.3.6), which is incompatible with this
build image. This build image only supports 2.4.x and later. We changed it to 2.7.2.

To learn how to set a custom Ruby version, visit the following link:
https://docs.netlify.com/configure-builds/manage-dependencies/#ruby

If you set a custom Ruby version lower than 2.4.x, you will need to select a different build image (Ubuntu Xenial 16.04).
The current build image only supports 2.4.x and later. You can select a different build image at the following link:
https://app.netlify.com/sites/falcosecurity/settings/deploys#build-image-selection
---------------------------------------------------------------------

I can't close this issue 😅
I would still like to confirm with you @leogr, it does appear that you've got access to the https://app.netlify.com/sites/falcosecurity team. If this isn't true, could you let me know, and I can invite you (please share your email address with me if this is the case--either here or in the CNCF slack)

Ei @nate-double-u as far as I know @leogr is on vacation he will come back in the next few days :) BTW thank you very much for your effort!!

@leogr: Closing this issue.

In response to this:

I can't close this issue sweat_smile
I would still like to confirm with you @leogr, it does appear that you've got access to the https://app.netlify.com/sites/falcosecurity team. If this isn't true, could you let me know, and I can invite you (please share your email address with me if this is the case--either here or in the CNCF slack)

Hey @nate-double-u
Thank you so much for having helped with this :pray: really appreciated :heart_eyes:
I can confirm there's no warning, and the website builds fine. :+1:
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Happy to help 😃


[tracking] Checklist before release

[ ] do most articles/sections have tags?
[ ] do examples showcase both cw-storage-plus and cw-storey? (using tabs like here)
[ ] is naming consistent?
[ ] is formatting more or less consistent? (nothing jarring)
[ ] #78

[ ] are we consistent with using code tags in section/article titles

Added a point and converted into issue, to create a place for the discussion how to approach.


Stream search results back to caller

Still a little work todo on the results filtering. But wanted to put this up for now.
fixes #495

[ ] Expose stream to mobile (async-ify mobile)
[ ] More efficiently de-duplicate collected results
[ ] Make contacts add work again but via stdin / file

Hard part done here. Need to get the add contact command working again and one tweak around canceling network worker searches when CLI quits.

WIP removed on this one.

Thank you @carsonfarmer!


Resolve conflicts in transducers_lazy

Resolve conflicts in https://github.com/gordonbrander/lettersmith/tree/transducers_lazy and merge with master.

Done: 9212715c36acfbf4d2675c7ed4be701c44880918


New Analyse task: Documentation ratio for a script block when the block has a minimum of lines

Is your feature request related to a problem? Please describe.
Of course it would be possible to use that on function but Invoke-Tasks does use itself the concept of calling register functions passing code blocks which would not be addressed then. That's why documentation should be check on code blocks.
Documentation means '#':

on an individual line, counts as 1
at the end of line with code, counts as 1

Checking for words: MinimumWordCount=5 (as an example)
An the documentation rate MinimumDocumentationRate=0.5 (example: 2 line code and 2 lines comment or '#' at the end of each code line)

Done.


[rfc-team-notifications-summaries] Remove github workflow

What does this PR do?
This workflow is not used anymore and will be replaced by #26659.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


IMERG Precipitation Reader for LVT

This update adds a reader to LVT that reads in GPM IMERG precipitation
data. This reader only works with the latest IMERG release (V05B) and
includes a configuration option that allows the user to switch between
the early, late, and final versions of the IMERG product.

More IMERG stuff:
config file: /discover/nobackup/dpsarmie/work/gitLIS/config_feature-LVT_IMERG.cfg
Paths to data are in the config file (same paths that are used for the IMERG filename commit). Let me know if you have any other questions.

Thanks for pull request.  Would please do a little code clean up?  Would you delete the trailing whitespace at the end of several lines in the IMERG_dataMod.F90 file?
Also would you package up the testcase data?

I've added a testcase for this reader here: /discover/nobackup/dpsarmie/work/gitLIS/testcase/imerglvt
Since this is an LVT testcase, I followed most of the instructions posted on the testcase creation guide with some modified steps to conform to LVT.
Some of the files that aren't used by LVT (forcing_variables.txt, MODEL_OUTPUT_LIST.TBL) are still included but can probably be deleted. I'll leave the final decision to you.
Let me know if you have any issues or if you want the LVT testcase modeled in a different way.

I am unable to reproduce your results.  My LVT log is identical to yours, but my results differ.  My run is not processing GDAS the same way.  Your results have a countgdas of 8, mine is 4.  Also my results have 0 rainfgdas.  Would you please rerun /discover/nobackup/dpsarmie/work/gitLIS/testcase/imerglvt
One thing that I noticed was your ctl files and your TARGET_OUTPUT are binary (gs4r), but OUTPUT is written as NetCDF.

Hi Jim,
I ran the testcase in /discover/nobackup/dpsarmie/work/gitLIS/testcase/imerglvt and everything looked like it was working correctly (countGDAS of 8). The only thing I changed was the netcdf output to binary in the lvt.config file (now changed in the testcase).
I used LVT compiled from the submitted branch, I also pulled a fresh copy from the main LISF repository and ran it there.  All of these runs were on Intel14 compilers. I can try to run it using other configurations to try and replicate the error if that'll help.

Thanks Daniel.  Your test case runs for me under Intel 14, but it produces bad results under Intel 18.  Would you please recompile and rerun using the Intel 18 compilers to see whether this happens for you?

I tested the case using the Intel 18 compilers and ran into the same errors you saw. There's actually an issue with the GDAS reader in LVT when it is being compiled using Intel 18.
I did a quick fix to the reader and it is now properly working with the Intel 18 compilers. I'll talk to you offline about the fix and whether or not we should include it.

Thanks!


convert the profile into cards

Instead of having the profiles occupy  the entire horizontal block,we can make them into cards with either 2 or three in one row.
As a user,that looks less monotonic and better.

I can work on this if approved!!
Thank you

I would like to work on this.

@ankitxcoding i am planning on working on this issue, i am waiting for @shyamtawli approval for this changes.
You could explore some more issues🙌

@shyamtawli can i start working on this issue?

@karthiknadar1204 You can work on it and remember the length of bio and number of skills is different from user to user. Make the design by keeping that in mind.

Hey @karthiknadar1204 are you working on it?

I am,need some time, will be done by Sunday

@shyamtawli have reduced the width of the cards and wrapped the contents of the cards as well,working to wrap the cards to having 2 cards in one row. Will update by tomorrow.

@shyamtawli I have reduced the width of the cards, I tried bringing the name and location to the right of the image but it is getting covered,unable to wrap two profile cards one next to another.

Should i submit pr for review

@karthiknadar1204 Try debugging it. You can take your time no hurry.

Sure will do @shyamtawli

Closing for now.


update: remove fixup_zip function

twitter process png images differently base on image size according to https://tweeterism.com/goodbye-png-the-images-that-you-upload-to-twitter-will-be-converted-to-jpg.html
twitter just processes image base on image size and image format, so it is worked after removed fixup_zip function.

No.
That article is outdated and no longer accurately reflects twitter's PNG image processing pipeline.
Even if it were still correct, I don't understand what removing fixup_zip() is supposed to achieve?


ERROR Attempted to append to non-started appender File

This is happening sometimes when stopping the server... Doesn't cause any actual error but would be good to figure out why this is happening. (Just a reminder for me, ignore this.)

Closing this as I've not seen it in hundreds of times I've executed SV in the last 5 months.


Can't covert forms with count-non-empty function

I tried to convert a form which contains count-non-empty function but to no avail.
The returned error is:
pyxform.validators.odk_validate.ODKValidateError: b"ODK Validate Errors:\n>> Something broke the parser. See above for a hint.\nError evaluating field 'weight_mean2': unrecognized data type in 'calculate' expression: java.lang.Integer\nCaused by: java.lang.RuntimeException: unrecognized data type in 'calculate' expression: java.lang.Integer\n\t... 11 more\n\nThe following files failed validation:\ntest.xml\n\nResult: Invalid"
Sample form:
test.xlsx

This is caused by https://github.com/opendatakit/javarosa/issues/509.


over-zealous space-removal undone

I was overzealous and broke the original error-handling output.
These two spaces really need to go back in @takov751 sorry.

No worries 😁


Download of model is too slow

Currently waiting up to 30 minutes for a model to load, is there a way to just download it on to a USB drive and plug it in?

The browser loaded/hosted models are fairly experimental and only works in Chrome and Edge.
Typically people choose to host a model locally on their machine itself with a tool like LM Studio which will let you choose any number of models to download and host with an API available locally that can be used to connect and communicate with the model.
Here's the docs to get a local model and server running, it's pretty painless https://lmstudio.ai/docs/local-server#using-the-local-server . The only adjustment needed is to enable the Cross-Origin-Resource-Sharing (CORS) option for the server.
Once the server is running you just enter the local server url into the custom config section like

And select the model and enter the max token value (1024 is a good start) and it's good to go.


delivers todo app

Delivers the following:
Delivers the todo app
How to run:

Clone the project git clone git@github.com:King-Benx/todo-list-review-project.git
Checkout to the root directory cd todo-list-review-project
Checkout to develop branch git checkout develop
Install dependencies npm i
Run the project using webpack npm run dev

View online
https://king-benx.github.io/todo-list-review-project/
Screenshots

Hi @King-Benx ,
Status: Approved ✔ ✔ ✔

To Highlight 💪🏻
Good job implementing the requirements 👍 Your project is complete! There is nothing else to say other than... it's time to merge it ⛵
Congratulations! 🎉
Optional Suggestions :

One minor change that I would recommend is to add a text-decoration style to
the To-do task after the task checkbox is checked ☑ so that it would match the project design.


AstraPro not using the new RGB calibration file

Hey everyone,
so before calibrating i was getting warnings from astrapro.launch that camera.yaml and depth_Astra_Orbbec.yaml.
After calibrating by using the camera_calibration package (http://wiki.ros.org/camera_calibration/Tutorials/MonocularCalibration) I cereated two files:  depth_Astra_Orbbec.yaml and  rgb_Astra_Orbbec.yaml
The  depth_Astra_Orbbec.yaml is being used and one can see the values when calling rostopic camera/depth/camera_info, but thats not the case with RGB.  rostopic camera/rgb/camera_info is showing another values as in rgb_Astra_Orbbec.yaml.
I tried all the following:
-astrapro.launch was still showing the warning that camera.yaml missing ---> I made a copy  of rgb_Astra_Orbbec.yaml and called it camera.yaml ----> warning disappeard but the values in rostopic camera/rgb/camera_info did not change
-rosservice call /camera/get_camera_info showed the same values as rostopic camera/rgb/camera_info ---> i tried to cahnge the calibration data manually with rosservice call /camera/set_camera_info --> this command cereates a camera.yaml  file but that also did not change anything  in the showed calibration values
-in astrapro.launch there are the following lines to refer to the calibration files
arg name="rgb_camera_info_url"   default=""  ---> refering to rgb_Astra_Orbbec.yaml did not make a difference
arg name="depth_camera_info_url" default="" ---> refering to depth calibration file works
param name="camera_info_url" value=""  this one should be the url to the camera.yaml file ---> cereating the camera.yaml and linking to it did not cahnge anything
In the terminal of astrapro.launch I can see:
10104987 VERBOSE    Multi distance calibration enable 0
10104990 VERBOSE    =================================================
10105013 VERBOSE    depth [577.530334,577.530334,316.777740,239.781937],
color[597.587158,597.587158,318.318207,245.468109]
These values are being used for depth and rgb. As I said refering to the depth calibration file works and changes the values but even after refering to rgb or camera yaml file the values from above (color) are still being used as RGB calibration values.
I appreciate any help :)

Try using a format like this:   file://${ROS_HOME}/camera_info/depth_Astra_Orbbec.yaml

Yes, this is working for the depth_Astra_Orbbec.yaml, but not for the rgb_Astra_Orbbec.yaml as mentioned above

As for me, I am using the ros_astra_launch package given by clearpathrobotics. They use a launch file called astra_pro.launch. Using this combination, I was able to load both rgb and depth calibration files.
`

<!-- Start nodelet manager -->
<arg name="manager" value="$(arg camera)_nodelet_manager" />
<arg name="debug" default="false" /> <!-- Run manager in GDB? -->
<include file="$(find rgbd_launch)/launch/includes/manager.launch.xml">
  <arg name="name" value="$(arg manager)" />
  <arg name="debug" value="$(arg debug)" />
  <arg name="num_worker_threads"  value="$(arg num_worker_threads)" />
</include>

<!-- Load driver -->
<include if="$(arg load_driver)"
   file="$(find astra_launch)/launch/includes/device.launch.xml">
  <arg name="manager"                         value="$(arg manager)" />
  <arg name="device_id"                       value="$(arg device_id)" />
  <arg name="bootorder"                       value="$(arg bootorder)" />
  <arg name="devnums"                       value="$(arg devnums)" />
  <arg name="rgb_frame_id"                    value="$(arg rgb_frame_id)" />
  <arg name="depth_frame_id"                  value="$(arg depth_frame_id)" />
  <arg name="rgb_camera_info_url"             value="$(arg rgb_camera_info_url)" />
  <arg name="depth_camera_info_url"           value="$(arg depth_camera_info_url)" />
  <arg name="rgb"                             value="$(arg rgb)" />
  <arg name="ir"                              value="$(arg ir)" />
  <arg name="depth"                           value="$(arg depth)" />
  <arg name="respawn"                         value="$(arg respawn)" />
  <arg name="depth_registration"              value="$(arg depth_registration)" />
  <arg name="color_depth_synchronization"     value="$(arg color_depth_synchronization)" />
  <arg name="auto_exposure"                   value="$(arg auto_exposure)" />
  <arg name="auto_white_balance"              value="$(arg auto_white_balance)" />
</include>

<!-- Load standard constellation of processing nodelets -->
<include file="$(find rgbd_launch)/launch/includes/processing.launch.xml">
  <arg name="manager"                         value="$(arg manager)" />
  <arg name="rgb"                             value="$(arg rgb)" />
  <arg name="ir"                              value="$(arg ir)" />
  <arg name="depth"                           value="$(arg depth)" />
  <arg name="respawn"                         value="$(arg respawn)" />
  <arg name="rgb_processing"                  value="$(arg rgb_processing)" />
  <arg name="debayer_processing"              value="$(arg debayer_processing)" />
  <arg name="ir_processing"                   value="$(arg ir_processing)" />
  <arg name="depth_processing"                value="$(arg depth_processing)" />
  <arg name="depth_registered_processing"     value="$(arg depth_registered_processing)" />
  <arg name="disparity_processing"            value="$(arg disparity_processing)" />
  <arg name="disparity_registered_processing" value="$(arg disparity_registered_processing)" />
  <arg name="hw_registered_processing"        value="$(arg hw_registered_processing)" />
  <arg name="sw_registered_processing"        value="$(arg sw_registered_processing)" />
</include>

<group ns="$(arg rgb)">

  <node pkg="libuvc_camera" type="camera_node" name="astra_pro_uvc">
    <!-- Parameters used to find the camera -->
    <param name="vendor" value="0x2bc5"/>
    <param name="product" value="0x0501"/>
    <param name="serial" value=""/>
    <!-- If the above parameters aren't unique, choose the first match: -->
    <param name="index" value="0"/>

    <!-- Image size and type -->
    <param name="width" value="640"/>
    <param name="height" value="480"/>
    <!-- choose whichever uncompressed format the camera supports: -->
    <param name="video_mode" value="uncompressed"/> <!-- or yuyv/nv12/jpeg -->
    <param name="frame_rate" value="30"/>
    <param name="frame_id" value="$(arg camera)_rgb_optical_frame"/>

    <param name="timestamp_method" value="start"/> <!-- start of frame -->
    <param name="camera_info_url" value="file://${ROS_HOME}/camera_info/rgb_Astra_Orbbec.yaml"/>

    <param name="auto_exposure" value="3"/> <!-- use aperture_priority auto exposure -->
    <param name="auto_white_balance" value="false"/>
  </node>

</group> <!-- rgb -->

`

That worked for me, thank you a lot!  Using the libuvc_camera package in astrapro.launch instead of the astra_camera package solved it.


Nonuniform velocity map (for gyrokinetics)

(this goes with gkylcas PR #39)
Implement support for nonuniform velocity (in gyrokinetics).

Allow users to specify a mapc2p for each species, specifying a univariate c2p map for each velocity coordinate.
Create a gkyl_velocity_map object which discretizes the velocity c2p map (vmap), and stores other information necessary to use it.
Pass this mapc2p to various objects used in gyrokinetics, e.g. projectors, moment calculators, collisionless and collisional terms.

Some practical details:

When the user doesn't provide a species mapc2p, gkyl_velocity_map creates an identity map under the hood.
This replaces all previous GK kernels and functionality, there is not separate chunk of code for uniform GK anymore. The only exception is LBO kernels, which do have separate uniform (nomap) kernels because the nonuniform recovery for the diffusion terms produces very large, and presumably costly, kernels.
In the future, when writing code, we should not use the grid objects to obtain velocity coordinates. We should us the velocity_map object.

Other work that happened in this branch in the process (that I can remember, there may be more):

We rewrote the LBO back to the 2 IBP formulation.
Introduced GKYL_MAX_VDIM in gkyl_util.h.
Allowed Vlasov to specify a zero-flux BC on one boundary but not the other.
Did further cleaning and standardization of gk regression tests (including removing some mirror tests because we don't actually need so many).
Plugged in all the gk reg tests to ci/gk_regression, but I can't get it to run all the tests at once due to catastrophic abortions. I don't know if this is a problem in ci/gk_regression, or a problem with memory errors in the GK solver. One suggestion: listing tests and their outputs in ci/gk_regression is a big pain and very prone to human error; we should instead gather all the gk_ tests in regression/ automatically.
Reinterpret the meaning of a FIXED_FUNC BC in GK. Instead of using the initial skin cell, now the user must provide the function (in the form of a projection input struct) which gets projected into the skin cell at t=0.
Introduce a factor of 2 in M3perp, so that M3 = M3par+M3perp. This way it is consistent with M2perp.

Tests and sims ran:

There are several collisional, collisionless and sheath sims with nonuniform velocity in regression/ now. I ran these on CPU and GPU.
A Boltzmann electron mirror sim using vparmax=20v_t, reproducing a the result in our 2023 PoP.
Numerous LAPD turbulence simulations on GPU.

Curiously, I've seen an identical LAPD simulation survive in this branch that crashes in gk-g0-app.
Some things that remain to be done (we could add it to this PR before closing it):

Infrastructure to allow nonuniform v when the user uses the GKYL_PROJ_FUNC option. The thing is that currently it give the user computational coordinates, but I think we need it to give physical (velocity) coordinates for ease of use.
Add nonuniform v to radiation operator.
Currently the diffusion surface term does recovery in v-space, in a way, rather than in computational space, like Greg Hammett suggested. This seems to work, but I think we should experiment with doing recovery in comp space to see what happens.

It would be good if @tnbernard @akashukla @Maxwell-Rosen @dingyunl run their sims in this branch with nonunfiform v so we can stress test this further.

Let’s wait on this till I return from my trip. As this is so critical we need to make sure the design is something we agree on. I will try and take a look this week, but it will be difficult.Sent from my iPhoneOn May 7, 2024, at 7:29 PM, Manaure Francisquez @.***> wrote:
@manauref requested your review on: #359 Nonuniform velocity map (for gyrokinetics).
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because your review was requested.Message ID: @.***>


relation and attribute players in relation

currently, only entities can be players in a relation - this needs to be extended and tested to relations and attributes.

Two parts to this:

add attributes and relations as players in the first place
add listseparator functionality for players in a relation (relevant for attributes and entities, and relations if they have an ID)

Two parts to this:

add attributes and relations as players in the first place
add listseparator functionality for players in a relation (relevant for attributes and entities, and relations if they have an ID)

finished implementing listseparators for players - should not matter if they are entities or relations
relations in relations up next

finished implementing listseparators for players - should not matter if they are entities or relations
relations in relations up next

update - put attriubutes as players in relation into separate issue... requires a different way of doing migration altogether... awaiting use case

update - put attriubutes as players in relation into separate issue... requires a different way of doing migration altogether... awaiting use case

implementation finished - merge-request coming

implementation finished - merge-request coming


Update css and some typos

There are some typos in lines and added a few words wherever necessary.
Corrected css design so as to give it a good look as there are some hidden texts which are unclear due to image.
Before correction

After correction

@atapas
Please review it.

I don't see any typo, please clarify. Have you created an issue for it?


Issue running configure on Ubuntu 16

checking whether -lc should be explicitly linked in... no
checking dynamic linker characteristics... GNU/Linux ld.so
checking how to hardcode library paths into programs... immediate
checking whether stripping libraries is possible... yes
checking if libtool supports shared libraries... yes
checking whether to build shared libraries... yes
checking whether to build static libraries... yes
checking scsi/scsi.h usability... no
checking scsi/scsi.h presence... yes
configure: WARNING: scsi/scsi.h: present but cannot be compiled
configure: WARNING: scsi/scsi.h:     check for missing prerequisite headers?
configure: WARNING: scsi/scsi.h: see the Autoconf documentation
configure: WARNING: scsi/scsi.h:     section "Present But Cannot Be Compiled"
configure: WARNING: scsi/scsi.h: proceeding with the compiler's result
configure: WARNING:     ## dddddddddddddddddddddddddddddddddddd ##
configure: WARNING:     ## Report this to<EMAIL_ADDRESS>##
configure: WARNING:     ## dddddddddddddddddddddddddddddddddddd ##
checking for scsi/scsi.h... no
configure: error: Couldn't find scsi/scsi.h

-rw-r--r-- 1 root root 6969 Feb  5  2019 /usr/include/scsi/scsi.h
Any help/tips appreciated!

This is due to acx_pthread.m4 missing. You need to install the autoconf-archive package to get it.
Something like "apt-get install autoconf-archive" should solve this. Make sure to rerun autogen.sh after installing the package. Please let me know if that does not solve your problem.

Thanks Damien!


BottomAppBar elevation (color) changed

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Use a BottomAppBar with elevation in Flutter 3.7.10. I tried 70 and 20.
Use a BottomAppBar with elevation in Flutter 3.10.5. I tried 70 and 20.
See the difference.

Expected results
The size of the widgets is not part of the problem, this is because the screenshots are made on two different devices.

Actual results
The size of the widgets is not part of the problem, this is because the screenshots are made on two different devices.

Code sample
Code sample
    return Container(
      color: Colors.white,
      height: 64.0 + MediaQuery.of(context).padding.bottom,
      child: BottomAppBar(
        elevation: elevation, // I tried 70.0 and 20.0
        child: Row(
          mainAxisAlignment: MainAxisAlignment.spaceAround,
          children: [
            // items here, I used 5 icons
          ]
        ),
      ),
    );

Screenshots or Video
Screenshots / Video demonstration
See Expected/Actual results.
Logs
Logs
N/A
Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.10.3, on macOS 13.4.1 22F82 darwin-arm64, locale en-NL)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.2)
[✓] VS Code (version 1.73.1)
[✓] Connected device (3 available)
[✓] Network resources

• No issues found!

Hi @appcapsergen. Thanks for filing this issue. I'm unable to reproduce this on the latest flutter versions. Tested with 3.7.12 and 3.10.5 and 3.12.0-13.0.pre.19 and observed same color as in screenshot. Can you confirm this as well on the latest versions??
Code Sample

import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      title: 'application',
      home: HomePage(),
    );
  }
}

class HomePage extends StatelessWidget {
  const HomePage({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Container(),
      bottomNavigationBar: Container(
        color: Colors.white,
        height: 64.0 + MediaQuery.of(context).padding.bottom,
        child: BottomAppBar(
          elevation: 70, // I tried 70.0 and 20.0
          child: Row(
              mainAxisAlignment: MainAxisAlignment.spaceAround,
              children: const [
                // items here, I used 5 icons
                Icon(Icons.home),
                Icon(Icons.home), Icon(Icons.home),
                Icon(Icons.home),
                Icon(Icons.home)
              ]),
        ),
      ),
    );
  }
}

Screenshot

flutter doctor -v

[✓] Flutter (Channel stable, 3.10.5, on macOS 13.4 22F66 darwin-arm64, locale
    en-NG)
    • Flutter version 3.10.5 on channel stable at /Users/dammya/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 796c8ef792 (2 weeks ago), 2023-06-13 15:51:02 -0700
    • Engine revision 45f6e00911
    • Dart version 3.0.5
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version
    32.1.0-rc1)
    • Android SDK at /Users/dammya/Library/Android/sdk
    • Platform android-33, build-tools 32.1.0-rc1
    • ANDROID_HOME = /Users/dammya/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2022.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.79.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (4 available)
    • iPhone 11 (mobile) • 6D1A287C-0B7C-4532-9EED-51027BBF306A • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • iPhone 14 (mobile) • 5FE8DE93-0F4C-45BA-8BED-BC9CD69B35D1 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)    • macos                                • darwin-arm64
      • macOS 13.4 22F66 darwin-arm64
    • Chrome (web)       • chrome                               • web-javascript
      • Google Chrome 114.0.5735.198

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[✓] Flutter (Channel stable, 3.7.12, on macOS 13.4 22F66 darwin-arm64, locale en-NG)
    • Flutter version 3.7.12 on channel stable at /Users/dammya/fvm/versions/3.7.12
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4d9e56e694 (2 months ago), 2023-04-17 21:47:46 -0400
    • Engine revision 1a65d409c7
    • Dart version 2.19.6
    • DevTools version 2.20.1

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/dammya/Library/Android/sdk
    • Platform android-33, build-tools 32.1.0-rc1
    • ANDROID_HOME = /Users/dammya/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2022.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.79.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (4 available)
    • iPhone 11 (mobile) • 6D1A287C-0B7C-4532-9EED-51027BBF306A • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • iPhone 14 (mobile) • 5FE8DE93-0F4C-45BA-8BED-BC9CD69B35D1 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)    • macos                                • darwin-arm64   • macOS 13.4 22F66 darwin-arm64
    • Chrome (web)       • chrome                               • web-javascript • Google Chrome 114.0.5735.198

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[✓] Flutter (Channel master, 3.12.0-13.0.pre.19, on macOS 13.4 22F66 darwin-arm64, locale en-NG)
    • Flutter version 3.12.0-13.0.pre.19 on channel master at /Users/dammya/fvm/versions/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 7b8595629b (5 hours ago), 2023-06-30 02:29:24 -0400
    • Engine revision 099a70ebbc
    • Dart version 3.1.0 (build 3.1.0-262.0.dev)
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/dammya/Library/Android/sdk
    • Platform android-33, build-tools 32.1.0-rc1
    • ANDROID_HOME = /Users/dammya/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2022.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.79.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (4 available)
    • iPhone 11 (mobile) • 6D1A287C-0B7C-4532-9EED-51027BBF306A • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • iPhone 14 (mobile) • 5FE8DE93-0F4C-45BA-8BED-BC9CD69B35D1 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)    • macos                                • darwin-arm64   • macOS 13.4 22F66 darwin-arm64
    • Chrome (web)       • chrome                               • web-javascript • Google Chrome 114.0.5735.198

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Hi @dam-ease
I've tested it multiple times now. The elevation is there, it's just very faint and barely visible, especially compared to how it was before.
I tested it more thoroughly with the versions now as well. The issue started when I upgraded from 3.7.10 to 3.10.0.

Here's the same problem in a very simplified version of the BottomAppBar.
The following is in Flutter 3.7.10 with elevation 70.0.

The following is in Flutter 3.10.0 with elevation 70.0 and shadowColor Colors.black (otherwise it's not visible at all). As you can see, there's a less clear elevation.

Hi @appcapsergen. Thanks for confirming. I think I can see a slight difference (almost not noticeable on my end though). From the Screenshot below (iPhone 14 Pro is the behaviour on version 3.10.5 and iPhone 14 is the behaviour on version 3.7.10).
Screenshot

Hi @appcapsergen
BottomAppBar when elevated casts a shadow but this shadow color is transparent by default in Material 3.
Material 3 uses surface tint color to indicate elevation. This was updated in https://github.com/flutter/flutter/pull/121406 to match Material 3 specs.
In your screenshot, you're not using surface tint color for elevation. You can provide a color in the  BottomAppBar.shadowColor parameter to indicate elevation.
You can override the default transparent shadow color when needed like this:
        child: BottomAppBar(
          shadowColor: Theme.of(context).colorScheme.shadow,
          elevation: 70, // I tried 70.0 and 20.0

Closing as this is documented in https://api.flutter.dev/flutter/material/BottomAppBar/shadowColor.html.

Hi @appcapsergen BottomAppBar when elevated casts a shadow but this shadow color is transparent by default in Material 3. Material 3 uses surface tint color to indicate elevation. This was updated in #121406 to match Material 3 specs.

Still if I put Colors.black as the shadowColor with whatever elevation, the elevation only gets a slight tint compared to what it was before (it would just turn black).

Hi @appcapsergen BottomAppBar when elevated casts a shadow but this shadow color is transparent by default in Material 3. Material 3 uses surface tint color to indicate elevation. This was updated in #121406 to match Material 3 specs.

@TahaTesser Still if I put Colors.black as the shadowColor with whatever elevation, the elevation only gets a slight tint compared to what it was before (it would just turn black).

I tried color sampling before the change (without custom shadow color) and after the change (with custom shadow color), the picked color was exactly the same.

Code sample

expand to view the code sample
import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData(useMaterial3: true),
      home: const HomePage(),
    );
  }
}

class HomePage extends StatelessWidget {
  const HomePage({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Container(),
      bottomNavigationBar: Container(
        color: Colors.white,
        height: 64.0 + MediaQuery.of(context).padding.bottom,
        child: const BottomAppBar(
          shadowColor: Color(0xFF000000),
          surfaceTintColor: Colors.transparent,
          elevation: 70, // I tried 70.0 and 20.0
          child:
              Row(mainAxisAlignment: MainAxisAlignment.spaceAround, children: [
            // items here, I used 5 icons
            Icon(Icons.home),
            Icon(Icons.home), Icon(Icons.home),
            Icon(Icons.home),
            Icon(Icons.home)
          ]),
        ),
      ),
    );
  }
}


GEP: Review HTTP Cookie Match GEP

A/C:

Review https://github.com/kubernetes-sigs/gateway-api/pull/2926
Determine if NGF can support proposed API
Contribute to the conversation as to NGF's stance on the API
Review summary with the team.

We should be able to route based on the Cookie header by looking at the incoming headers in our njs script.


879 format dates on custom pdfs

Closes UserOfficeProject/user-office-project-issue-tracker#879
Before:

After:

Checks if the date includes a time if not remove the time from the PDF

@UserOfficeProject/swap sorry forgot to mention this at sprint planning are you guys using this feature?


Clarify creating a custom module

As requested by @sgoudham in #90

I quite like the idea of mentioning required plugins for the module at the top like in uptime.
Could this get a place in the template and be mentioned?


how to add vuex module through nuxtjs module? (and auth module is out of date)

I'm trying to change auth module to my needs and with this index.js
const path = require('path')

module.exports = function nuxtAuthDevise (options) {
  this.addVendor(['cookie', 'js-cookie'])

  this.addPlugin(path.join(__dirname, 'store.js'))
}

and with original store.js I get an error
 ~/projects/vue-todolist/client   master ●  yd
yarn run v0.28.4
$ nuxt
[AXIOS] Base URL: http://localhost:3001/api/edge , Browser: http://localhost:3001/api/edge
  nuxt:build App root: /home/bjorn/projects/vue-todolist/client +0ms
  nuxt:build Generating /home/bjorn/projects/vue-todolist/client/.nuxt files... +1ms
  nuxt:build Generating files... +9ms
  nuxt:build Generating routes... +6ms
ReferenceError: ctx is not defined
    at eval (lodash.templateSources[17]:9:10)
    at Builder._callee2$ (/home/bjorn/projects/vue-todolist/client/node_modules/nuxt/lib/builder/builder.js:249:23)
    at tryCatch (/home/bjorn/projects/vue-todolist/client/node_modules/nuxt/node_modules/regenerator-runtime/runtime.js:65:40)
    at Generator.invoke [as _invoke] (/home/bjorn/projects/vue-todolist/client/node_modules/nuxt/node_modules/regenerator-runtime/runtime.js:303:22)
    at Generator.prototype.(anonymous function) [as next] (/home/bjorn/projects/vue-todolist/client/node_modules/nuxt/node_modules/regenerator-runtime/runtime.js:117:21)
    at step (/home/bjorn/projects/vue-todolist/client/node_modules/nuxt/dist/nuxt.js:852:30)
    at /home/bjorn/projects/vue-todolist/client/node_modules/nuxt/dist/nuxt.js:863:13
    at <anonymous>
[nuxt] Unhandled promise rejection: ReferenceError: ctx is not defined

so, my question is how to add vuex module through nuxtjs module ? (I dont know whether nuxtjs/auth merged with root store or added as module, but I want it to be module (separate namespace))

@pi0 any help? are auth module still correct?

@BjornMelgaard As of lots of new changes it is not working now, I plan to update this module as soon as new vuex releases. Also for access vuex and store we can use plugins :)

:((
new version of vuex - vuex 3? O_O

Not actually, we just need an enhancement which is merged and just awaits for next release that allows us access vuex in store actions. Without that, we need to pass everything as parameters to get working. Is your project on GitHub? I may help you before that module release :)

ah, this change. OK, I made a note))


public_url returning quoted_name with '/'

i'm adding my files inside folders, so i give the blob name as "UUID/file.ext", but when i request the public_url() i get the fullname as quoted example: ".../MYBUCKET/UUID%2Ffile.ext"
both "%2F" and "/" in the link downloads the same file, but with different names

Hi @jlvcm,
Thanks for reporting. We will look into this.

Just to add, this also breaks generate_signed_url for objects with / characters.  The signed string contains %2F instead of /, so the signature doesn't match the one generated by the server.
The problem seems to be this line: https://github.com/GoogleCloudPlatform/google-cloud-python/blob/master/storage/google/cloud/storage/blob.py#L1579
The safe='' overrides the default behavior of urllib.parse.quote, which is to leave / characters alone.  If you remove that param it should fix the issue, although you might need to confirm that other callers aren't expecting slashes to be escaped.

Hitting this bug as well.
Is there an ETA for the fix?

@frankyn Some evidence that the back-end actually wants the / separator escaped:  the JSON returned from objects.insert for an object whose name is parent/child/filename.txt looks like so:
{'bucket': 'gcp-3809-1519337419615373',
 'contentType': 'text/plain',
 'crc32c': 'U35c9A==',
 'etag': 'COy2x+fFutkCEAE=',
 'generation': '1519337650379628',
 'id': 'gcp-3809-1519337419615373/parent/child/filename.txt/1519337650379628',
 'kind': 'storage#object',
 'md5Hash': 'ogBPN3MLlEVnCnOPoPye5Q==',
 'mediaLink': 'https://www.googleapis.com/download/storage/v1/b/gcp-3809-1519337419615373/o/parent%2Fchild%2Ffilename.txt?generation=1519337650379628&alt=media',
 'metageneration': '1',
 'name': 'parent/child/filename.txt',
 'selfLink': 'https://www.googleapis.com/storage/v1/b/gcp-3809-1519337419615373/o/parent%2Fchild%2Ffilename.txt',
 'size': '19',
 'storageClass': 'STANDARD',
 'timeCreated': '2018-02-22T22:14:10.217Z',
 'timeStorageClassUpdated': '2018-02-22T22:14:10.217Z',
 'updated': '2018-02-22T22:14:10.217Z'}

If I dont use the escaped form when making API calls against the object (e.g., GET or DELETE) then it 404s.
The fact that the / doesn't seem escaped in the console is not relevant here:  that is a Jedi mind trick. :)

So, it looks to me as though the correct resolution is to update Blob.public_url (and maybe also Blob.generate_signed_url?) to avoid quoting embedded / characters, but continue quoting them for all API requests.  I have updated #4716 accordingly.

Thanks Tres.
On Thu, Feb 22, 2018 at 2:47 PM Tres Seaver<EMAIL_ADDRESS>wrote:

So, it looks to me as though the correct resolution is to update
Blob.public_url (and maybe also Blob.generate_signed_url?) to avoid
quoting embedded / characters, but continue quoting them for all API
requests. I have updated #4716
https://github.com/GoogleCloudPlatform/google-cloud-python/pull/4716
accordingly.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-python/issues/3809#issuecomment-367849122,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUcy70fmvhoUT4FNTKcsX6Agu_oFhhks5tXe5jgaJpZM4O2qQv
.

Oh gosh, that's confusing. So, it fails when you don't quote the object name during other operations. IIUC, then it would make sense to not quote "/" in the returned public_url and generated_signed_url instead.
Thanks @tseaver! Apologies, that I wasn't much help here.

@tseaver, I was able to find a better answer for this case.
The existing client library uses the XML API endpoints defined by _API_ACCESS_ENDPOINT which is used for both public_urls and signed_urls.
For the JSON API, percent-encoding is required and will fail as you observed in your prior comment.
For the XML API, percent-encoding is not required but does require UTF-8 encoding. That's why at first it looks like a Jedi mind trick with the public url generated by for example the Cloud Console Storage Browser as it uses the XML API endpoint.
JSON API endpoint - https://www.googleapis.com/storage/v1{path}
XML API endpoint - https://storage.googleapis.com
As for this PR, for the recommendation is to not quote '/'. Rationale is based more on the ease of parsing '/' versus '%2F'.

Thanks @tseaver ! I was OOO yesterday. Cheers!

Maybe it's a regression, but I still get encoded slashes (%2F) in Blob.public_url in gcloud 0.18.3.
I un-encode them manually for the moment, but inconsistency (for example with the console's public URL) is not desirable imho.
Should this be re-opened? Or a new Issue?

Hi @atteneder,
Openning a new issue, thank you for raising this issue.


Version 1.0.13 blocking startup.

Octoprint Anywhere v:1.0.13 is currently the only 3rd party plugin I have installed and if it is enabled, startup is interrupted and the Octopi reboots in safe mode. Disabling the plugin solves the startup issue. Not sure if it is a conflict with a  first party plugin or not.

Hmm that sounds bad. Can you post octoprint.log file so that I can take a look? Here is how you find it: https://community.octoprint.org/t/where-can-i-find-octoprints-and-octopis-log-files/299

Hi Kenneth.
It was weird I couldn’t get Octopi to load all morning. I just rebooted the Pi to get a good log file for you and this time it loaded fine. It’s now running without issue.
I’ve attached the log file for what it’s worth but I have no clue why not wouldn’t load all morning and does so now.
Jonathan

On Sep 9, 2019, at 1:37 PM, Kenneth Jiang<EMAIL_ADDRESS>wrote:
Hmm that sounds bad. Can you post octoprint.log file so that I can take a look? Here is how you find it: https://community.octoprint.org/t/where-can-i-find-octoprints-and-octopis-log-files/299 https://community.octoprint.org/t/where-can-i-find-octoprints-and-octopis-log-files/299
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/kennethjiang/OctoPrint-Anywhere/issues/102?email_source=notifications&email_token=ALJUHSLB2HCPIJCULAHKZPDQI2CW5A5CNFSM4IU475G2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6IN42Q#issuecomment-529587818, or mute the thread https://github.com/notifications/unsubscribe-auth/ALJUHSKE4Q2LKX43SY6GGCLQI2CW5ANCNFSM4IU475GQ.

Good to know it's working now
Did you attach the log file to this issue or send it to me by email?

I sent it via email.

On Sep 9, 2019, at 5:00 PM, Kenneth Jiang<EMAIL_ADDRESS>wrote:
Good to know it's working now
Did you attach the log file to this issue or send it to me by email?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/kennethjiang/OctoPrint-Anywhere/issues/102?email_source=notifications&email_token=ALJUHSMPPV6U2HWM2SNEDWDQI22OPA5CNFSM4IU475G2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6JAWSI#issuecomment-529664841, or mute the thread https://github.com/notifications/unsubscribe-auth/ALJUHSI2ZFVYZUVYESNZIN3QI22OPANCNFSM4IU475GQ.

Did you send it to<EMAIL_ADDRESS>@JonAugusto

I just attached it to my reply to your email.  I will upload it when I get back to work tomorrow.

On Sep 9, 2019, at 6:23 PM, Kenneth Jiang<EMAIL_ADDRESS>wrote:
Did you send it to<EMAIL_ADDRESS>@JonAugusto https://github.com/JonAugusto
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/kennethjiang/OctoPrint-Anywhere/issues/102?email_source=notifications&email_token=ALJUHSPJJWTM3ODL5LVXSX3QI3EF3A5CNFSM4IU475G2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6JG7BI#issuecomment-529690501, or mute the thread https://github.com/notifications/unsubscribe-auth/ALJUHSP5U5ZBQSF4DNRIVMLQI3EF3ANCNFSM4IU475GQ.

@JonAugusto I didn't see OctoPrint Anywhere errors the log file you sent me that could possibly cause OctoPrint to boot into safe mode. I did see UNDERVOLTAGE errors that may have caused your Pi to shut down but I'm not too sure if it was the reason for your OctoPrint to enter safe mode.
I'm closing this issue for now. If the same problem happens again, the best place to ask this question is probably OctoPrint forum or  open an issue with OctoPrint.


Update sql-database-projects extension version to 1.4.4

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 41.726%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/dialog/configurePython/configurePythonWizard.ts
1
45.22%

Totals

Change from base Build<PHONE_NUMBER>3:
0.0%

Covered Lines:
30729

Relevant Lines:
68916

💛  - Coveralls


WIP Chat feature added

Chat feature in progress:
How to run:

Install packages:

$ npm install socket.io
$ npm install --save-dev nodemon

Run:

$ node server.js
Open the web (.html) file on localhost:3000

Thanks @jessieAnhNguyen! We should work on getting this merged in the future release


Update readme to v0.9.6

Hi @bootstraponline thanks for contributing! Can you change all locations to 0.9.6 like in #554 ?

sure, I will update.

Updated. Also removed some trailing white space.


Fullbody IK

Proposal Title
Use our custom fullbody IK
Context and Problem Statement
We need to animater our characters in game using full body ik.
Describe the feature / enhancement and how it helps to overcome the problem or limitation
EWBIK is a full body ik.
Describe how your proposal will work, with code, pseudo-code, mock-ups, and/or diagrams
EWBIK
If this enhancement will not be used often, can it be worked around with a few lines of script?
Code is already complicated.
Positive Consequences
No response
Negative Consequences
No response
Is there a reason why this should be core and done by us?
I and Eron know.
References

EWBIK

Derivative License
Copyright (c) 2020-2021 V-Sekai contributors.
Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
in the Software without restriction, including without limitation the rights
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
copies of the Software, and to permit persons to whom the Software is
furnished to do so, subject to the following conditions:
The above copyright notice and this permission notice shall be included in all
copies or substantial portions of the Software.
 IN NO EVENT SHALL THE
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
SOFTWARE.

e87120da380eee9edf15d6d09ef170f232632016


[DE DaTeTimeV2] German date recognition inconsistent if containing ordinal numbers as words

German dates not recognized if day or month number given as word
To Reproduce

DateTimeRecognizer.RecognizeDateTime("siebter April", "de-DE") returns timex XXXX-04-07
DateTimeRecognizer.RecognizeDateTime("7. April", "de-DE") returns timex XXXX-04-07
DateTimeRecognizer.RecognizeDateTime("7.4.", "de-DE") returns timex XXXX-04-07
DateTimeRecognizer.RecognizeDateTime("siebter 4.", "de-DE") does not match
DateTimeRecognizer.RecognizeDateTime("7. vierter", "de-DE") does not match

Expected behavior
Dates should be recognized without having to convert cardinal/ordinal numbers into digits first.
Platform (please complete the following information):

Platform: .NET
Environment: nuget package
Version of package v1.8.4

Perfect! Thanks again!


Feature/dockstore 317 line numbers workflow

Related issue: https://github.com/ga4gh/dockstore/issues/317

Line numbers are added to descriptor/secondary file in descriptor tab of workflow.

Currently, the line numbers on tools dockerfile and descriptor file are not implemented yet. Implementation on tools will be created on a separate branch and pull request because tools might have a little bit of different implementation compare to workflow.

[x] Check that you pass the basic compilation and unit tests by running grunt

Coverage decreased (-0.1%) to 9.249% when pulling 96af5fb1555595f520dcf53320d6d429f0b03a57 on feature/dockstore_317_line_numbers into 8b7136015e8bab8205a500af052115393e692731 on develop.


Server Exits with code 255 with no activity

I have noticed that if I run the server and no clients are pinging it, it will exit after some amount of time with the error status code of 255. There's nothing in the log other than the last request made.

I am using it within my docker environment here is my service definition for it
services:
  ...

    websockets:
      image: php:8.1-fpm-alpine
      volumes:
        - ./web:/var/www/html:delegated
        - ./logs:/var/www/logs
      working_dir: /var/www/html
      expose: [6001]
      labels:
        caddy: staging.h1.listech.on.ca
<EMAIL_ADDRESS>"Connection *Upgrade*"
<EMAIL_ADDRESS>"Upgrade websocket"
        caddy.reverse_proxy: "@ws {{upstreams 6001}}"
      entrypoint: ["php", "/var/www/html/artisan", "websockets:serve"]
      networks:
        - routable
        - h1project

I fix it using soketi


[improve][broker] Exclude producers for geo-replication from publishers field of topic stats

Fixes #22442
Motivation
Please refer to #22442.
Modifications

Exclude producers for geo-replication from the publishers field in topic-stats and broker-stats.
Two Pulsar services were started in the OneWayReplicatorTestBase class, but they were each using different configuration metadata store instances and were not being replicated. So I made the two Pulsar services use a common configuration metadata store instance and enabled replication.

Verifying this change

[x] Make sure that the change passes the CI checks.

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Resolved the conflicts.


Tag search has some probably unexpected params

There are some probably unwanted params in tag search https://odysee.com/$/discover?t=test

[ ] release_time: >x probably not wanted in "Trending" and "New"  for tag search
[ ] in "Top" the time options don't do anything. (Seems to also affect categories)
[ ] "Content Type: Any" uses claim_type: [streams, reposts, channels],  with stream_types: ["video", "audio"], so channels aren't returned, and not any types of streams either.  Probably shouldn't have stream_type filter there.
(This also happens in categories, but maybe wanted in there?)

@infinite-persistence probably related to the recent changes.


ocamlPackages.unisim_archisec: init at 0.0.5

Description of changes
UNISIM ARCHISEC is a companion project of the binary analysis platform BINSEC. It exposes disassembly metadata and DBA (Dynamic Bitvector Automata) semantics of several instruction set architectures, including ARM and x86.

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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

[x] 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/)
23.05 Release Notes (or backporting 22.11 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

[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build ocamlPackages.unisim_archisec


Avoid getting into infinite loops in staking thread

If the wallet is locked, the thread is stuck in the loop, so it is needed to check for ShutdownRequested() and stop the thread after the signal received.

Nice,
Actually saw this issue myself yesterday on testnet
Will merge this in shortly


Streaming and caching

Is possible do such things with the library?

start playing .mp3 from url immediately (like streaming or streaming)
cache it and track caching progress during it's playback
save cached file to local storage

Did you find any solution about starting to play .mp3 immediately from a url?

@fantomnotabene yeah https://github.com/tumtumtum/StreamingKit


LHN: Hide the pencil icon when message is sent or draft is deleted and show the icon when user started to type #6050

Details

The pencil icon should not be visible when there is no draft in the report.
Show the pencil icon whenever users creates the draft (immediately) and hide the pencil icon as soon as the draft is either deleted or message sent (also immediately). However, keep the LHN reordering same, we do not want the LHN to reorder while typing as that is very distracting.
Fixed Issues

$ https://github.com/Expensify/App/issues/6050
Tests and QA
Web, Desktop
Note:  The pencil icon change is instant.
Actions

Action 1: Type text in the chat without sending it.
Result: Once you start typing, the pencil icon appears and remains visible on the contact in LHN. The order of the LHN items does not change.

Action 2: Type something in the chat and send it.
Result: The pencil icon appears as you type and disappears once you have sent it. The LHN order will change if not the topmost contact.

Action 3: Type text in the chat positioned in the middle of LHN. Click on another contact.
Result: The pencil icon remains visible while the LHN order changes.

Action 4: Have text in the chat and delete it.
Result: The pencil icon disappears while the LHN may re-order. All re-order cases are reproducible with other actions.

Action 5: Type text in the chat positioned in the middle of LHN. Reload the page.
Result: Contact appears at the top.

Action 6: (Depends on the previous step.) Delete the text for that contact.
Result: Contact appears in the middle again.

Action 7: Type text in the chat positioned in the middle of LHN. Receive a message from another contact positioned below the contact with a draft.
Result: LHN order updates.

Mobile Web, Android, iOS
Actions:
Repeat each desktop action but open LHN after each change.
Result: LHN is invisible when typing and always re-orders when visible.
Tested On

[x] Web
[x] Mobile Web
[ ] Desktop
[ ] iOS
[ ] Android

Screenshots

Web
https://user-images.githubusercontent.com/47689923/139918285-a2416f46-bcd3-4620-94e8-d5ca509d9ba7.mp4
Mobile Web
https://user-images.githubusercontent.com/47689923/139920446-853bd5b5-7eb8-4cf4-af62-4fd3fccaa239.mp4
Desktop

iOS

Android

Thanks for your suggestions, I have fixed all the issues.

@marktoman Could you please update videos for all platforms and fill in the testing steps?
Suggestion: Please mark this PR ready for review, if you think this is ready. A draft PR usually means you are still working.

@parasharrajat I would rather let @mountiny confirm he has no changes in mind before marking it ready for merge.

@marktoman Thank you very much for addressing my comments. In general, we have draft PRs only for when WIP and once ready for review (which this PR already was wirth the first iteration), contributor then makes the PR ready for review. Here is the dedicated part of Contributor guidelines.
So @parasharrajat was correct, feel free to take this PR from draft state to "normal" state 😅 also beware you can trust @parasharrajat in terms of processes in this repository basically as much as any Expensify engineer :) Rajat also might be reviewing some of your next PRs before some Expensify engineer will get their hands on it.
I will try to have a look into this soon so we can fix this asap :)

@marktoman Additionally, to make this PR ready for review, you should fill in the appropriate sections of PR body. It is a bit tedious especially at the start to take screen recordings on all the platforms, but it is a process we require and which makes sure we test the change on all platforms. Also fill in the QA steps which our QA team could follow once this is deployed to staging. Thank you!

@parasharrajat Status changed, my mistake.
@mountiny Done. Thank you for the quick review on Saturday and apologies for missing some of the docs. I finally had time to read everything in detail over the weekend. I am not sure what to do about commit signing at this point?
@parasharrajat @mountiny Video for each platform - sadly no luck after a lot of effort trying to run the app natively. The only thing that works is npm run web via WSL, everything else fails. Expo works natively just fine. It seems that Windows as a dev environment is simply not supported. I was hoping to solve future mobile issues locally on Android and then verify via Mac In Cloud on the missing platforms.
I have added steps and a video for both web and mobile web, capturing all LHN use cases.

I am not sure what to do about commit signing at this point

@marktoman Not sure what you mean by this. It appears to me that the commits from you are verified, which is good :)
@marktoman First things first, please, pull main and merge the most recent code as it seems we have merge conflict there. I will review the code now.
We are aware that there might be some problems developing on Windows. I am not totally knowledgeable in terms of how our contributors who do not own Mac do this, but surely there is many of them. Being you, feel free to ask in the Slack channel and someone will be able to help you/give you tips and advice as we require to test the PRs on all platforms usually.
In this case we can safely assume the solution should behave same on mobile as on web as the change is mostly in the back logic.

@mountiny Merged. OK, thanks for the advice.

Alright, thank you for the guidance and overall handling of the issue!

@mountiny Let's hope! Is the issue now considered finished so I can solve the other ones?

@marktoman, Great job getting your first Expensify/App pull request over the finish line! :tada:
I know there's a lot of information in our contributing guidelines, so here are some points to take note of :memo::

Now that your first PR has been merged, you can be hired for another issue. Once you've completed a few issues, you may be eligible to work on more than one job at a time.
Once your PR is deployed to our staging servers, it will undergo quality assurance (QA) testing. If we find that it doesn't work as expected or causes a regression, you'll be responsible for fixing it. Typically, we would revert this PR and give you another chance to create a similar PR without causing a regression.
Once your PR is deployed to production, we start a 7-day timer :alarm_clock:. After it has been on production for 7 days without causing any regressions, then we pay out the Upwork job. :moneybag:

So it might take a while before you're paid for your work, but we typically post multiple new jobs every day, so there's plenty of opportunity. I hope you've had a positive experience contributing to this repo! :blush:

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@mountiny @OSBotify I have my answer :)

@marktoman As the handy comment above mentions, you can now be hired for other jobs :) But still one at a time, just to make sure you are able to update the PRs in timely manner (we expect addressing changes requested in PR reviews in ~2 days usually, if you can't update for whatever reason, let the engineer working on the issue with you know you will have delay)

🚀 Deployed to staging by @mountiny in version: 1.1.13-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@marktoman a regression from this issue here: https://expensify.slack.com/archives/C01GTK53T8Q/p1636082766011900
After signing out, there is a blank page as SidebarLinks line 113 the unreadActionCount is undefined.
Can you please look into this and try to fix it? Thank you

Thanks @stitesExpensify! There is a PR up which will fix it, I will tag you for a review to speed it up!

PR fixing this regression is up here.

🚀 Deployed to production by @Jag96 in version: 1.1.14-4 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

The accessibility issues found in this PR:

The issue https://github.com/Expensify/App/issues/5651 is still reproducible.
Selecting a chat from the list using Enter key adds a new line in the message text field - failure of WCAG SC 2.1.1
https://user-images.githubusercontent.com/88733897/142771763-71523e04-9af3-457b-adb7-ba08883e5cca.mp4

@ogumen Thank you very much for this feedback. For @marktoman, these issues are not regressions from this PR so no need to worry about that.
I will create issue for the second issue mentioned here as well.

Issue created here: https://github.com/Expensify/App/issues/6386


Add DKIM tests

Description

Add reason for DKIM failure
Add tests for DKIM signature check (helpers package)
Refactor ARC revert handling

Type of Change
Please delete options that are not relevant.

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

Checklist:

[ ] I have discussed with the team prior to submitting this PR
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] My changes generate no new warnings
[ ] New and existing unit tests pass locally with my changes

Might also be good to include the modifications in https://github.com/zkemail/zk-email-verify/issues/162 here, maybe even https://github.com/zkemail/zk-email-verify/issues/165 (the assigned guy isn't doing it). The zkp2p fixes are here:
see: https://github.com/zkp2p/zk-p2p/blob/develop/client/src/components/ProofGen/validation/hdfc.tsx#L85
and: https://github.com/zkp2p/zk-p2p/blob/develop/client/src/components/ProofGen/validation/venmo.tsx#L62


[QST] Non-nvtabular examples?

❓ Questions & Help
Details

Hi all! I'm excited to learn more about transformers4rec. Unfortunately, I'm having trouble following some of the examples due to the library's (likely preferred) way of handling datasets: via nvtabular. As I understand it, nvtabular has a hard dependency on cupy, which itself cannot be installed on machines without a GPU. This makes it difficult to even add as a dependency (at least, via pip/poetry) since at installation time, cupy checks for NVIDIA library headers.
If that's correct, I'd love to know (or...potentially contribute?) examples that don't use the nvtabular extra. Please let me know if I'm missing something very obvious! Thanks all 🙏🏻

I see, thank you for the context @sararb! I'll take a look at the tests for some inspiration and hopefully follow up soon with an issue proposing pyarrow-specific training 👍🏻

Hi, is it possible to use the pyarrow engine for GPU training? It seems this raises an error:
RuntimeError: Expected object of device type cuda but got device type cpu for argument #3 'index' in call to _th_index_select


Create feature images and screens for App & play store

images sent to @AhmedHanafy725 at telegram

verified on app store

testcase


Category Deletion Validation

Please Review
@seenusuvarna
@mleeBoeing

DELETE /api/category/4 HTTP/1.1
Host: localhost:8001
Authorization: Basic YmlnYnJvdGhlcjpwYXNzd29yZA==

{
    "detail": "Can not delete category Education, 90 listings use this category",
    "error": true
}

:+1:


Unable to specify VM disk size

The size of the disk allocated to a new ccloudvm VM is hardcoded to 60000Mbs. We need to be able to specify this in the workload files.

Original issue https://github.com/ciao-project/ciao/issues/1350


[Protobuf] Added a cord support to gRPC protobuf serializer

As Protobuf is going to support Cord to reduce memory copy when [de]serializing Cord fields, gRPC is going to leverage it. This implementation is based on the internal one but it's slightly modified to use the public APIs of Cord. only

I'd like to see a fuzzer that constructs a cord, writes it, reads it back, and asserts equivalency.


VMware-backed machine has different volume permissions than docker4mac

This may be a very n00b questions.
i’m getting different behaviour on docker4mac with mounted volumes than docker-machine vmware machine
for the d4mac when i mount a /var/lib/postgres/data directory it comes through as owned by postgres/root and everything works. I don't actually have postgres as a user on my Mac.
On docker-machine the directory ends up being owned by 501/dialout and Postgres won't start up b/c data directory "/var/lib/postgresql/data" has wrong ownership
Do you have any idea why this happens and how i can make the volume ownership work correctly?
This is probably similar to the forums issue
the command i'm using to start Docker4Mac container:
docker run --rm -it  -v pwd/postgres-data:/var/lib/postgresql/data -v check:alpine sh
for VMWare Fusion, i'm using
docker run -it --rm -v pwd/postgres-data/:/var/lib/postgresql/data jobrunner:toli sh
my local userid on a mac is 501 which isn't present on machine; and i'm staff which is mac group 20, and dialout is my machine group 20
interestingly, postgres exists in both images as user 70
so that may explain why the user:group is coming through as 501:dialout
is there a way around it to get the right permissions?
/cc @frapposelli

It's a long known issue since the postgres user does not map to the vbox shared folder mount user :( and I'm assuming vmware as well
D4M might do magic in their proxy layer to ensure the permissions are setup correctly, and/or the fuse fs just doesn't care


Implement spec.uid for GrafanaDashboard

Followed the same pattern as #1686 for the CustomUID.
As discussed in the weekly sync, spec.uid or metadata.uid will always overwrite dashboardModel.uid.
Thanks to the existing Status.UID update detection, existing dashboards are automatically migrated to using metadata.uid by the operator.

Local procedure for testing
make start-kind
make deploy
kind export kubeconfig --name kind-grafana

# Test resources
kubectl apply -f https://raw.githubusercontent.com/grafana-operator/grafana-operator/master/examples/basic/resources.yaml
kubectl apply -f ../test-dashboard.yaml

# Check status:
# dashboards
#   - default/dashboard-uid/dashUID
kubectl get grafanas -A -o yaml

# Build and apply image to cluster
make ko-build-kind
IMG=ko.local/grafana/grafana-operator make deploy
kubectl patch deploy -n grafana-operator-system grafana-operator-controller-manager-v5  --type='json' -p='[{"op": "replace", "path": "/spec/template/spec/containers/0/imagePullPolicy", "value":"IfNotPresent"}]'

# Check status:
# dashboards
#   - default/dashboard-uid/dashUID -> specUID
kubectl get grafanas -A -o yaml

../test-dashboard.yaml
apiVersion: grafana.integreatly.org/v1beta1
kind: GrafanaDashboard
metadata:
  name: dashboard-uid
spec:
  instanceSelector:
    matchLabels:
      dashboards: "grafana"
  uid: specUID
  resyncPeriod: 10s
  json: >
    {
      "id": null,
      "uid": "dashUID",
      "title": "Simple Dashboard",
      "tags": [],
      "style": "dark",
      "timezone": "browser",
      "editable": true,
      "hideControls": false,
      "graphTooltip": 1,
      "panels": [],
      "time": {
        "from": "now-6h",
        "to": "now"
      },
      "timepicker": {
        "time_options": [],
        "refresh_intervals": []
      },
      "templating": {
        "list": []
      },
      "annotations": {
        "list": []
      },
      "refresh": "5s",
      "schemaVersion": 17,
      "version": 0,
      "links": []
    }

[x] I Chainsaw tests for immutability of spec.uid immutability

is it possible for this to only override the uid if a uid is not already configured? some collections of dashboards already have a uid in them and then link to each other through that but other dashboards have no uid at all and it would helpful to make sure that the uid can be deterministic if the dashboard's uid isn't present :thinking:

Just to sum up the behaviours:

Current: spec.json?.uid > metadata.uid
Allows modifying dashboard UID as well as overwriting metadata.uid whenever.

This PR: spec.uid > metadata.uid
UID is always immutable, spec.json?.uid is entirely ignored

Your suggestion spec.uid > spec.json?.uid > metadata.uid?
spec.json?.uid only is mutable and whenever

All of which has their drawbacks, notably:

UID collisions require downloading remotely sourced dashboards getting rid of the benefit of updates.
Linking between Dashboards need to have their UIDs hardcoded in the spec.uid to not break the links.
Requires a delete and create of the resource to change the set UID
Same problem as 1 except the workaround is specifying the spec.uid

Personally I don't see much difference between 2 and 3 except they introduce annoyances in two different use cases.
Additionally, there's a slightly higher maintenance burden of 3 due to extra tests and checks necessary.

@Baarsgaard the scenario I’m thinking of is how to maintain multiple k8s clusters all with Grafana operator running on them, pointing to what should ostensibly be the “same” Grafana. Having something deterministic is better than using the metadata uid. Although maybe that’s not the best use of this feature and I should instead open a different PR that directly addresses that.

@kcolford The opposite of my use case where I have 1 Grafana operator with multiple Grafana instances in other tens of other clusters!
Would that not be covered by specifying the spec.uid on every GrafanaDashboard manifest?
Or are you perhaps building flux artifacts/rendering yaml and applying the documents across multiple clusters?

@Baarsgaard I think that could work, but it could get messy with determining whether a dashboard already specifies a uid or not. Definitely still possible though, I was just hoping to avoid some extra work.

@theSuess Sure, but in this case, should the spec.uid field still be immutable?
Or should it be left mutable and thereby different from the other CRs?

It should be left immutable, since editing the UID still shouldn't be allowed.
If possible, it would also be great to forbid setting the field after creation, but I'm not sure if that's possible

It is possible, part of the Chainsaw tests validate that behaviour right now!
Alright, I will see to fixing it such that dashboard provided uids are sourced before the metadata.uid
This should align with your usecase as well @kcolford

Alright @theSuess I think that's it?
Please do a review from scratch as I ended up reverting a lot of the uid handling in dashboards controller as the previous implementation was more aligned with just using the cr.CustomUIDOrUID function.

Code looks good at first glance - will try it out tomorrow & hopefully also merge it/release a new version :)

I've cleaned up the commit history a bit and queued the PR for merge - thanks again!


Material3 / Material component themes

Why it is needed?
Hey!
Great work on the library!
I'm working on react-native-bottom-tabs which require you to specify Material3 or MaterialComponents theme in styles.xml.
Would be great to have Theme.EdgeToEdge.MaterialComponents and Theme.EdgeToEdge.Material3 so that users can use both libraries without issues.
Thanks!
Possible implementation
No response
Code sample
No response

For personal reference: https://okwasniewski.github.io/react-native-bottom-tabs/docs/guides/android-native-styling.html#react-native-community-cli-users

@okwasniewski Edge-to-edge versions of material2 / material3 are available in 1.1.0, sorry for the delay!
The good news is that given the fact that the react-native-bottom-tabs expo plugin is only setting the AppTheme parent theme, there is no need to use it when using the react-native-edge-to-edge one 🔥:
{
  "expo": {
    "plugins": [
      [
        "react-native-edge-to-edge",
        { "android": { "parentTheme": "Material3" } }
      ]
    ]
  }
}

Thank you @zoontek!
I've updated the docs to instruct users to use only one plugin if rn-edge-to-edge is used 🚀 https://github.com/okwasniewski/react-native-bottom-tabs/pull/136


Further topicrefs in empty  are ignored in PDF output

See here for the initial report, it is a regression from DOT 1.8:
https://groups.yahoo.com/neo/groups/dita-users/conversations/messages/41869
For a bookmap with the following preface structure:
    <preface>
        <topicref href="topics/task_preface.dita"  toc="no"/>
    </preface>

the PDF no longer contains the preface content.
It might be connected with: https://github.com/dita-ot/dita-ot/issues/1930

Hi Jarno
Was there any progress on this bug in 2.4.x? Topicrefs under Preface and Notice in the Frontmatter are not available in the PDF outoput.
Thanks.

I've got a fix for this as part of https://github.com/dita-ot/dita-ot/pull/2683 -- should restore topics that are missing after inclusion in empty <preface> or empty <topicref> inside of the front / back matter.

Fixed by #2683


Allow recvPacket to proceed in FLUSHING channel state, without counterparty last sequence set

Original comment text below.

There is one gotcha here!
After TRY handler executes, we have counterpartyUpgrade and are in FLUSHING channel state BUT LastSequenceSend is not set yet, until after ACK and CONFIRM succeed.
Right now, that means relayer will have recvPacket rejected until it goes further in the upgrade handshake and then it would need to retry packets.
I wonder if wrapping this check in:
if counterpartyUpgrade.LastSequenceSend != 0 {
    // do greater than checking
}

// otherwise carry on with `recvPacket`

This way we know that if LastSequenceSend is still 0 that we can safely continue to receive packets while in FLUSHING state.
Happy to leave this as a future improvement, I'll open an issue based off this comment.
Originally posted by @damiannolan in https://github.com/cosmos/ibc-go/pull/4386#discussion_r1304147989

I think it makes sense to allow packets to proceed in this state, rather than erroring and having to retry them later.

Let's go a head with this after discussion. I will open an issue on the spec

will pick up after #4448 to use fresh tests


Import 9ed784e451: Fixed sorting error with render sorting (by Caitlyn)

This is an automated proposal to look at a commit made by Caitlyn and import it into Overte

Commit:
9ed784e451384ee1b61d1ed27e63f8714c5f71f1

Author:
Caitlyn

Date:
Fri, 10 Apr 2020 23:01

Fixed sorting error with render sorting

Stats:

Filename
Stats
Lines
Added
Removed
Lines in blame

libraries/entities-renderer/src/RenderableMaterialEntityItem.cpp

6
6
0
1

libraries/render-utils/src/MeshPartPayload.cpp

36
19
17
⚠ 0

2 files
-
42
25
17
1

To work on this, please assign the issue to yourself, then look at the commit and decide whether this would be a good addition to Overte.

If the commit is useful, tag it with "Tivoli: Keep", and keep it open until it's merged.
If the commit is not useful, tag it with "Tivoli: Discard", and close it.
If the commit is not useful right now, but might be later, tag it with "Tivoli: Maybe later", and close it.
If it's hard to decide, tag it with "Tivoli: Discuss", and keep it open.

Useful commits should be submitted as a PR against Overte. Tag this issue in PR, so that it's automatically closed once the PR is merged.
You can cherry-pick this issue with this command:
git cherry-pick 9ed784e451384ee1b61d1ed27e63f8714c5f71f1

touches on shaders


Initial code for bigbets section.

Please always provide the [Jira issue(s)] your PR is for, as well as test URLs where your change can be observed (before and after):
Fixes #
Test URLs:

Before: https://main--aem-competency--aem-comp.hlx.live/

After: https://--aem-competency--aem-comp.hlx.live/

[ ]  If you are adding a new block or a variation to an existing block, please fill below:
Block library path: https://--aem-competency--aem-comp.hlx.live/tools/sidekick/library.html?plugin=blocks&path=/sidekick/blocks/&index=0

Sunil has raised a new PR for this


How to call setLanguage?

Hi,
first thanks for your package - nice work, thanks alot.
But how do I change the lang by code?
I tried
LanguageSwitcher::setLanguage($language)
but I get this message:
Non-static method ied3vil\LanguageSwitcher\LanguageSwitcherController::setLanguage() should not be called statically
I use newest laravel - what am I doing wrong?
Thanks alot.
Regards matthias

nvm, it's \ied3vil\LanguageSwitcher\Facades\LanguageSwitcher::setLanguage

Glad you sorted it!
If you import (use statement) the facade at the top of the php file, you will be able to write LanguageSwitcher::...


periph.io/x/devices/v3/bmxx80: add support for filters without using SenseContinuous

I cannot use the SenseContinuous option with my hardware as there are additional bus contention issues to deal with, but would like to use the built in filtering on the BME sensors.
In order to prevent current implementations from changing their behavior I added a new initialization option to set the standby time. When the Filter and Standby options are set in the device options the sensor will initialize to the normal mode instead of sleep. Additionally when calling Sense mode will not be set to forced if the Filter and Standby options are set as this would prevent the continuous sampling needing for the filtration.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 80.00000% with 4 lines in your changes missing coverage. Please review.

Project coverage is 46.0%. Comparing base (a9604df) to head (2f31d3e).

Files with missing lines
Patch %
Lines

bmxx80/bmxx80.go
80.0%
3 Missing and 1 partial :warning:

Additional details and impacted files
@@          Coverage Diff          @@
##            main     #74   +/-   ##
=====================================
  Coverage   46.0%   46.0%
=====================================
  Files         82      82
  Lines       7913    7921    +8
=====================================
+ Hits        3642    3646    +4
- Misses      4108    4111    +3
- Partials     163     164    +1

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

Dang gosec got new checks. Let me fix that first, shouldn't take too long.

I committed https://github.com/periph/devices/commit/67ec1da96441ed538cd9faca0b01ff04b326d970 to silence the linter.

Oh I forgot to mention: thanks for the contribution!


BUG/5 Submit suggestion button should only work if a suggestion was clicked

Currently you can click it even if no suggestions have been selected

https://github.com/gla6-fp-team1/gaelic-project/pull/42


Add device name and version into report for appium native apps

When testing native apps on appium there is no browserName nor version
capabilities, instead they use deviceName and platformVersion

Hi @Kignuf, thanks for fix
Please be aware we don't merge chages that aren't ported to v5 version to stay consistent.

Run tests

Hello Boris,
I am looking into how to add tests for this PR and I am not sure how to proceed.
From what I have seen, all current tests are running a real wdio session using selenium & phantomjs
So if I use this approach I would need to add an appium test, but that would come with some rather huge overhead.
Or I could add a unit test using "sinon" to mock what I need to mock and thus just ensure that "test.addParameter" was called with the correct parameters.
Is that ok for you ?

Yes, unit test will be enough

@christian-bromann could you please release it?

wdio-allure-reporter 0.8.1 published 🎉

Thanks!

Thanks a lot !


QT5 support

Is there no Qt5 support? Or is the README just outdated?

The README is not up to date. Have a look at the GUI toolkit section at Build with CMake how to build quarter with CMake. If there are undocumented switches you might also try calling cmake -L . in the build directory to get a full list of all options.

Thank you very much! Works like a charm


[Question] Is there any similar/exact implementation in C++ (or in Qt)?

Not related & no response. Closing this issue.


⚠ Spryr has degraded performance

In 62a423b, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 429
Response time: 10564 ms

Resolved: Spryr performance has improved in 6cd258a after 8 minutes.