Mismatched Protocol Buffer versions (runtime vs installed)

Hello there,
I was using 2.0.0-rc4 which was working fine. I needed to upgrade nalgebra though, so it turns I needed to upgrade ort. Using 2.0.0-rc6, everything compiles just fine, however upon running the binary, I get this:
[libprotobuf FATAL /home/runner/work/ort-artifacts-staging/ort-artifacts-staging/onnxruntime/build/_deps/protobuf-src/src/google/protobuf/stubs/common.cc:83]
This program was compiled against version 3.12.4 of the Protocol Buffer runtime library,
which is not compatible with the installed version (3.21.12).
Contact the program author for an update.  If you compiled the program yourself,
make sure that your headers are from the same version of Protocol Buffers as your link-time library.
(Version verification failed in "./obj-x86_64-linux-gnu/modules/dnn/versions.pb.cc".)

terminate called after throwing an instance of 'google::protobuf::FatalException'

I suppose this is because I am using the distributed binaries (download-binaries feature is enabled). Can this be fixed somehow? FYI, the environment in question is the pytorch/pytorch:2.4.1-cuda12.1-cudnn9-runtime Docker image.

What other crates does your application depend on? I'm wondering if there is another crate that depends on protobuf (like OpenCV) that ends up getting the wrong library. (Though if that were the case, I'd expect it to not link due to duplicate symbols...)

Oh, right, there's a lot going on here, and that includes opencv indeed. Thanks for your quick reply, I'll have to root-cause this further. Rust dependency hell at it again :sweat:

Hello @zopieux,
I'm encountering a similar issue with protobuf version incompatibility. Here's the error I'm getting:
[libprotobuf FATAL /Users/runner/work/ort-artifacts-staging/ort-artifacts-staging/onnxruntime/build/_deps/protobuf-src/src/google/protobuf/stubs/common.cc:83] This program was compiled against version 3.14.0 of the Protocol Buffer runtime library, which is not compatible with the installed version (3.21.12).  Contact the program author for an update.  If you compiled the program yourself, make sure that your headers are from the same version of Protocol Buffers as your link-time library.  (Version verification failed in "/Users/solaoi/.cargo/registry/src/index.crates.io-6f17d22bba15001f/sentencepiece-sys-0.11.2/source/src/builtin_pb/sentencepiece_model.pb.cc".)

In my case, the conflict seems to be between the sbv2-api library mentioned in the ort README and the sentencepiece library used within the ctranslate2-rs library (https://github.com/jkawamoto/ctranslate2-rs).
Have you made any progress in resolving this issue? If you've found a solution or workaround, I'd greatly appreciate if you could share it.

This can be fixed by using a dynamically linked build of ONNX Runtime.

@decahedron1
Thank you for your guidance on dynamic linking.
I successfully loaded the library using the information you provided from the following link:
https://github.com/microsoft/onnxruntime/releases/tag/v1.19.2


Fix: EventPage previews without a location

EventPages that don't specify a location currently render with an extra leading dash, for example:
- MAY 30, 2024 @ 01:00 PM EDT
This commit fixes that and adds some tests around how the preview location string renders.
See https://www.consumerfinance.gov/about-us/events/ for a live example of this bug right now.
How to test this PR
Run a local server and visit http://localhost:8000/about-us/events/ and http://localhost:8000/about-us/events/archive-past-events/, and compare against production.
Screenshots

Before
After

Checklist

[x] PR has an informative and human-readable title
[x] Changes are limited to a single goal (no scope creep)
[x] Code follows the standards laid out in the CFPB development guidelines

For an entry like WASHINGTON, DC CFPB HEADQUARTERS … grammatically, shouldn't it be WASHINGTON, DC, CFPB HEADQUARTERS?


Sometimes breaking on detaching on iOS

It appears sometimes, and is connected with calling callback on getting value during detaching.
https://github.com/kmagiera/react-native-reanimated/commit/3bbf8a0fed9afeebe7987a21029fd7dc5fdc2038#diff-1ea1099449a3d9c97e383160f5a623e2R194
This line is breaking.
Maybe it is an issue in RN. Is there ay reason, @kmagiera, you decided not to use callback in call node?
In order to repro e.g. navigate to "Interpolate" in Example app and then press back button

I am getting this issue with Expo 31.0.0 which contains react-native-reanimated 1.0.0-alpha.10. I just copy MapPanel.jsandInteractable.jsfromExample`

Everyone,
This will be fixed in new version. Current workaround is just to make sure if there is no any null in `Value's. Nulls sending from Obj-C has to be wrapped into object which was done in related PR.


Feature request: configurable toast duration

Rather than set on a per-instance basis, could toast duration be specified like defaultIconPack?
I'm happy to create a PR if needed.

If you have the time to make the PR, do so please :)


FileDialog clone

Would it be possible to Derive Clone for FileDialog? Would save repetitive code when setting up complicated dialogs. Ty!

Hi, I think if I remember correctly, it is not that easy to implement clone because of some data types.
If you want to configure multiple FileDialog objects with the same or almost the same values, you can use FileDialogConfig and FileDialog::with_config. Then you don’t have to configure each object manually.
Let me know if this works for you!

Perfect; solved; ty!


⚠ Bofrost.at has degraded performance

In c741169, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2555 ms

Resolved: Bofrost.at performance has improved in 8673514 after 13 minutes.


fix article list item width

not exactly sure what is happening here, but taking it out fixes the issue on ie11.
Feel free to make suggestions on the best way to handle this.

I think this will break the center aligned content, but I'll take a look.

The anchor was display: flex to keep the "more link" pinned to the bottom of each callout regardless if one callout had more text than the others. Since the "more link" was removed from the design, I removed it from this component and set the anchor's display to block since it's wrapping.
And just an FYI, the IE 11 flexbox bug was fixed by putting width: 100% on the h3 and p elements.


the scroll up button is added at the testimonial page .

Issues Identification
Closes: #995
Description
the issue was that there was no scroll up button over the testimonial page
Summary
first i added the arrow file and then did some major changes in css and then javascript and then finally my feature was added to the testimonial page..
Types of Changes
Please check the boxes that apply

[ ] Bugfix (non-breaking change that fixes an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Please check the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] My changes do not break the current system and pass all existing test cases
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Additional Information
please if you can make it level please otherwise its ok sir ..

here is your PR sir
@vivekv2810

sir this page also have one more issue the text is coming out from div @vivekvardhan2810


Provide complete schema for YAML file format

It's not obvious what properties are supported in the YAML file beyond what the example shows. For example, environment variables are supported but I only know that because I stumbled upon another tutorial that mentioned that. I haven't found any link to a complete reference for this YAML format.
Concretely, I need to specify a DNS name label like I would with the "--dns-name-label" parameter to "az container create". This seems like it should be supported by the YAML format but I can't tell.

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

ID: 03e645be-5677-9b0b-049e-4b579a5da1af
Version Independent ID: 36f94f65-128d-f973-ec19-3f3e11b2a405
Content: Tutorial - Deploy a multi-container group in Azure Container Instances - YAML
Content Source: articles/container-instances/container-instances-multi-container-yaml.md
Service: container-instances
GitHub Login: @dlepow
Microsoft Alias: danlep

@darzu this is because a yaml file is not isolated to only AKS. So there are resources out there that you can reference to better understand how to structure the yaml file.
I will assign to the author to review and see if we can provide some of those resources in the doc.
In the meantime, I suggest doing an internet search for the yaml format for additional information.
www.yaml.org is a site I have used in the past as well.

@MicahMcKittrick-MSFT  I'm not asking about the YAML language format, I'm asking specifically about the schema that is accepted the "az container create --file" command. It seems to me there should be some link in this tutorial to a reference page that enumerates all the supported properties.

@darzu - Thanks for your feedback, which echoes feedback we've heard from other customers (for example, issue https://github.com/MicrosoftDocs/azure-docs/issues/34614). We have a documentation work item to address this in the near future.  In the meantime, please take a look at the container group/instance properties configurable in the Resource Manager template spec. The template spec should help clarify the hierarchy of properties, which are required or optional, etc.
To your specific question, the DNS name label is a supported property. Example:
apiVersion: '2018-10-01'
location: eastus
name: example
properties:
[...]
  ipAddress:
      type: Public
      ports:
        - port: 80
      dnsNameLabel: mylabel
[...]

I hope this helps!


⚠ Bofrost.es has degraded performance

In 6553310, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1460 ms

Resolved: Bofrost.es performance has improved in 85b0f4f after 6 minutes.


Idempotent provisioner issue

Changes:

if fail to add to DB then delete the volume from the array
add retry if fail to add to DB due to network issue
if vol already exist just import it to the DB, if fail to add to DB, return false, will not delete the vol from array

Signed-off-by: feihuang<EMAIL_ADDRESS>

This change is

Coverage increased (+0.3%) to 53.075% when pulling aecf244b098ee84a08a5118cf36ffb4f7edd63c6 on fix/UB-612_Idempotent_provisioner_issue into 4cadd2acec7fc77182550f27408d608cc5da46c7 on dev.

Without reviewing the code, if you have no.3 then why do you need no.1 (why delete the volume?).
Another option would be:

write the volume in the DB as about to be created
create the volume in the array
mark the volume as created in the DB

This will easily allow you to know where you failed, when you get a request, you know if it's not the first time.

Review status: 0 of 7 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

If we cannot delete vol successfully in no.1, then if we retry to create vol with same name, then no.3 will work. Or sometime, we create the volume which name is same with volume exist on the array.
For you option, do you mean that we change the order of the volume creation?
If so, since we had issue when insert vol in the DB, will we hit the issue when we try to mark the vol as created in the DB, because we also need to open the DB to mark it

Review status: 0 of 8 files reviewed at latest revision, 3 unresolved discussions.

database/connection.go, line 88 at r1 (raw file):
Previously, ranhrl (Ran Harel) wrote…
If we don't already have one, we should have a general retry function. Please look at the code if we have one.
An example of a general retry function can be found here https://blog.abourget.net/en/2016/01/04/my-favorite-golang-retry-function/
It also mentions open source retry package that we can leverage: https://github.com/giantswarm/retry-go

To use general retry function, we need to define func as " func() (err error)", so here we had to retry open instead of "c.factory.newConnection"  as "newConnection() (*gorm.DB, error)"

local/scbe/scbe.go, line 299 at r1 (raw file):
Previously, ranhrl (Ran Harel) wrote…
You will get error delete volume when trying to create a volume, it does not make sense, what will the caller do with that error? I don't think the design here is perfect.

In ubiquity we  only print the error message, so change it to a readable message

Comments from Reviewable

Sure, you will hit the same issue when you try to mark it, but next time you can "track" your actions, means that you will find the volume in the DB marked as "started to create but it is not completed yet", you will then find that the volume was created in the storage and then you will try again to mark it as "completed". It just gives you extra certainty in what is the current status, this way you don't override the volume if it's not you who created it. I know the chances are low (because of the ubiquity ID in the volume name) so it's not mandatory but a better way to handle when you have a transaction.
I know Shay did not point to that solution because the current one is simpler to implement, but maybe we should think further when Shay is back if we want to improve it or not. Could be done as a separate ticket.

Review status: 0 of 8 files reviewed at latest revision, 3 unresolved discussions.

database/connection.go, line 88 at r1 (raw file):
Previously, hfeish (Fei Huang) wrote…
To use general retry function, we need to define func as " func() (err error)", so here we had to retry open instead of "c.factory.newConnection"  as "newConnection() (*gorm.DB, error)"

well done!

local/scbe/scbe.go, line 299 at r1 (raw file):
Previously, hfeish (Fei Huang) wrote…
In ubiquity we  only print the error message, so change it to a readable message

You should give details about the volume name, wwn etc with the message, otherwise it will be harder to debug.

utils/utils.go, line 287 at r2 (raw file):

}

func Retry(attempts int, sleep time.Duration, f func()(err error)) (err error) {

The function is not general enough, maybe it's because I am used to python, but it cannot get any parameters and cannot return any value besides the error, will that support anyone trying to use it in the future?

utils/utils.go, line 297 at r2 (raw file):

        }
        time.Sleep(sleep)
		logs.GetLogger().Debug("retrying", logs.Args{{"attempt", i}, {"error", err}})

please add the function name to the debug message if possible.

Comments from Reviewable

Open a ticket UB-1186 to discuss the new solution

Review status: 0 of 8 files reviewed at latest revision, 5 unresolved discussions.

local/scbe/scbe.go, line 277 at r1 (raw file):
Previously, ranhrl (Ran Harel) wrote…
This should be a warning and not a debug level, as this is not the common use case

change to warning

local/scbe/scbe.go, line 299 at r1 (raw file):
Previously, ranhrl (Ran Harel) wrote…
You should give details about the volume name, wwn etc with the message, otherwise it will be harder to debug.

Add wwn and name

utils/utils.go, line 287 at r2 (raw file):
Previously, ranhrl (Ran Harel) wrote…
The function is not general enough, maybe it's because I am used to python, but it cannot get any parameters and cannot return any value besides the error, will that support anyone trying to use it in the future?

seems "fun() error" is general here, cannot support more para or return

utils/utils.go, line 297 at r2 (raw file):
Previously, ranhrl (Ran Harel) wrote…
please add the function name to the debug message if possible.

Add fun name here

Comments from Reviewable

Hi @hfeish
Some comments about the design describe in the PR:

What happened if you create volume on the storage but then you fail to update the ubiquity DB?  In that case you plan to delete the volume from the storage and raise error (according to the PR description). But what happened if in that case you will get error during the deletion of the volume? What is the plan for this?

About retry if fail to add to the DB, its OK, but what is the retry mechanize? how many retries?

About "if vol already exist just import it to the DB, if fail to add to DB, return false, will not delete the vol from array". This is problematic - because it does automatic import for volume even if the customer don't intend to do so. Import volume is a feature that should be cover later on. We should do the import volume explicit (and not implicit) for example when customer wants to import existing volume he will have to add a special annotation in the PVC for importing existing volume.
So I think we should handle this DB fail in different way.  For example (and I think @ranhrl  already mention it) - before creating the volume we should write a line in the DB something like "vol name is under creating", and then you create the vol on the storage, and then if you fail to update the DB with the new volume WWN then you fail (you can do retires as you mentioned in [2]). Then if another creating request comes you can see that the volume may already created so you check if its exist and if so you "import" it. But only if the "vol name is under creating" state in the DB.
Doing such you prevent implicit volume import and also you prevent don't need to delete the volume on the storage because you probably will get another creation request on the same volume sooner or later.  (BTW please check if a PVC fail to create will automatically trigger another creation request - I am not sure about it)

@ranhrl did I miss something here?

Review status: 0 of 8 files reviewed at latest revision, 5 unresolved discussions.

database/connection.go, line 88 at r1 (raw file):
Previously, ranhrl (Ran Harel) wrote…
well done!

its ok but I must say that this util.Retry() is very specific function signature.  so it fit only for functions with this signature so its not really generic.

database/connection.go, line 81 at r3 (raw file):

func (c *Connection) Open() (error) {
    defer c.logger.Trace(logs.DEBUG)()
    if err := utils.Retry(5, 5*time.Second, c.retryOpening); err != nil{

Please set the retries and time.Seconds as const
In addition @fei, please check if there is timeout for the creation task in kubernetes side. You set 25 seconds retries but we need to make sure k8s provisioner will not kill us before that time.

local/scbe/scbe.go, line 299 at r1 (raw file):
Previously, hfeish (Fei Huang) wrote…
Add wwn and name

@Fei, please check out what SCBE does when the same issue happened.  I don't think they delete the volume from the storage.

local/scbe/scbe.go, line 276 at r3 (raw file):

	if len(volumeInfo) == 0{
		s.logger.Debug("Volume not exist on the storage, need to provision")
	} else {

this is automatic import feature - which I don't think we aim for. Mentioned it as comment on the PR

local/scbe/scbe.go, line 279 at r3 (raw file):

		s.logger.Info("Warning: Volume exists on the storage, no need to provision")
		// TODO idempotent, if volume attached to host, we should handle when attach this volume to another host.
		err = s.dataModel.InsertVolume(createVolumeRequest.Name, volumeInfo[0].Wwn, fstype)

BTW there is a chance you will get volumeInfo list 2 items, in that case you may consider to raise error.
The chances are very small, but here you assume that if the len(volumeInfo) is not 0 then its always 1, but you cannot know for sure. Its better to handle len>1 as well.

local/scbe/scbe_rest_client.go, line 142 at r3 (raw file):

}

func (s *scbeRestClient) GetVolumesByVolName(volName string) ([]ScbeVolumeInfo, error) {

BTW this function is ok, we will need it when we will do explicit import

local/scbe/scbe_rest_client_test.go, line 191 at r3 (raw file):

                {Name: volName + "2", ScsiIdentifier: volIdentifier + "2", ServiceName: profileName + "2"},
            }
            fakeSimpleRestClient.GetStub = OverrideGetStub(volumes)

BTW what is OverrideGetStub? where it comes from?

local/scbe/scbe_test.go, line 354 at r3 (raw file):

		})

		It("should succeed to create volume if vol already exist on array", func() {

this is ok, but as mentioned its a wrong feature to automatic import in implicit way.

utils/utils.go, line 287 at r2 (raw file):
Previously, hfeish (Fei Huang) wrote…
seems "fun() error" is general here, cannot support more para or return

sure I know, you cannot. I just said that its generic for a very specific function signature

Comments from Reviewable

This PR was not maintained too much time.
I delete the PR but keep the branch


New version: Ncurses_jll v6.2.0+0

Autogenerated JLL package registration

Registering JLL package Ncurses_jll.jl
Repository: https://github.com/JuliaBinaryWrappers/Ncurses_jll.jl
Version: v6.2.0+0
Commit: b2d10acbbb9a1de016f4189eb8797c60d0d4953b
Revision on Yggdrasil: https://github.com/JuliaPackaging/Yggdrasil/commit/5a733cbbf2988945c723f5856172b893cc03a0ef
Created by: @giordano

Let me make some more tests


FaceEncodings forced to use 5 point predictor instead of 68

Can we add an option to specify predictor model to FaceEncodings? instead of forcing it to small (5 points) so we can use 68 points (large) like the python package
(I already edited the source / re compiled for own purpose, this issue is for other people)
https://github.com/takuya-takeuchi/FaceRecognitionDotNet/blob/23c8d57c61fc74749fec1e6509f44f3af48d9a5b/src/FaceRecognitionDotNet/FaceRecognition.cs#L262

@Hershef
Thank you for your suggestion.
OK. Does it satisfy your requiments?
public IEnumerable<FaceEncoding> FaceEncodings(Image image, IEnumerable<Location> knownFaceLocation = null, int numJitters = 1, PredictorModel model = PredictorModel.Small)
        {
            if (image == null)
                throw new ArgumentNullException(nameof(image));

            image.ThrowIfDisposed();
            this.ThrowIfDisposed();

            var rawLandmarks = this.RawFaceLandmarks(image, knownFaceLocation, model);
            foreach (var landmark in rawLandmarks)
            {
                var ret = new FaceEncoding(FaceRecognitionModelV1.ComputeFaceDescriptor(this._FaceEncoder, image, landmark, numJitters));
                landmark.Dispose();
                yield return ret;
            }
        }

Added new argument for FaceEncodings method on develop branch
1a979d2ae852c89aab245a2da478c2d790faf38f

Note
This request was already introcued in https://github.com/ageitgey/face_recognition/commit/4d7bc049468ee78c971c2609af8979f1ab092584
close.


More Clarity needed: How to set up the Managed Identity so a developer can connect local Azure Func app to CosmosDB on Azure

Great post below. But I need more information on how to set up the Managed Identity so a developer can connect a local Azure Func app to a CosmosDB on Azure
https://docs.microsoft.com/en-us/dotnet/api/overview/azure/identity-readme?view=azure-dotnet

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

ID: 4f73df56-2312-5851-7b59-3dfb766e5f66
Version Independent ID: 31be349e-063e-6696-bc5f-752c413fa079
Content: Azure Identity client library for .NET - Azure for .NET Developers
Content Source: api/overview/azure/identity-readme.md
Service: identity
Product: azure
Technology: azure
GitHub Login: @ramya-rao-a
Microsoft Alias: ramyar

Hi @PingPongSet -  Unfortunately, we don't have a new WebJob extension library in this repository that integrates with Azure.Identity. However, does the following documentation help at all? https://docs.microsoft.com/en-us/azure/cosmos-db/managed-identity-based-authentication#assign-a-system-assigned-managed-identity-to-a-function-app

Hi @PingPongSet - Unfortunately, we don't have a new WebJob extension library in this repository that integrates with Azure.Identity. However, does the following documentation help at all? https://docs.microsoft.com/en-us/azure/cosmos-db/managed-identity-based-authentication#assign-a-system-assigned-managed-identity-to-a-function-app

Hi @christothes
The content in that link is not working.
As part of this document, it should have provided at least a link to show how to set up the relevant Managed Identity.
Out of curiosity, why did you need a WebJob extension library to add a link to this document if that document is already available?

Hi @christothes The content in that link is not working.

Could you clarify what specifically did not work? What problem or error did you run into?


Add project links to README

Don't think I didn't notice your typo for the brief amount of time it existed 😀
👍

You can't prove it, git commit --amend would never squeal!


Adding Jekyll Now

Sorry, closing this since it's a starter kit for Jekyll, not a static site generator in its own right.


Implement Quote Removal

Figured this was the easiest part of Expansion to start with

I've written the logic, completely untested still 😅
Having some trouble figuring out if this specifically needs to happen at execution or doing this while parsing would already be acceptable.
And how this would fit with the other expansions that need to happen.
I'm thinking of first adding this to cmd_prefix, but that is produced from a WORD token, which doesn't have any "rules" associated with it

I believe most of it will happen in execution, since subsequent command expansion can be affected by previous commands. Some things like heredoc word quote removal happens in parser but it's quite minimal afaik.

I wonder if it's worth making some generic state machine tokenizer, i.e. defining only the states, transitions, and token delimiting. Since expansions iteration is somewhat similar to the lexer, and for arithmetic we'll also need some lexer. Some states are not so straight forward though so I don't really see it immediately.


V11 Compatibility

Please consider re-working for V11 compatibility

THIS MODULES IS DEPRECATED ON V11 IN FAVOR OF Rideable


COPY folder with Docker issues with .gitignore file

Copying a folder with Docker
COPY ./dist /dist
does not work when this folder is included in the .gitignore file.

I figured out that now ignores files and folders inside .gitignore.
To still upload files that you want to ignore for git (such as build files), you can add them to a now.json file.
An example would be:
{
    "name": "name",
    "type": "docker",
    "files": [
        "dist"
    ]
}

However, there seems to be issues with this, as the dist folder is still not included and I am getting an error when building the Dockerfile:
lstat dist: no such file or directory

Having same issue. Using now.json to include a gitignored folder. Getting same error from docker build.

Thank you so much for the effort you have put into this issue! 😊
However, since we completely re-worked the entire codebase of Now CLI and this issue does not apply to the new code anymore, I'm closing it now.
You can read more about our biggest release yet here: https://zeit.co/blog/now-2


frozen image is flipped in portrait mode

When freezing the current image of a monitor in portrait mode, that image gets flipped like such:

I also get similar behavior when focusing on a monitor that has been rotated in the Hyprland config.
Monitor config:
monitor=HDMI-A-1, 1920x1080, 1920x0, 1, transform, 3

Here's an example of that happening:
https://github.com/hyprwm/hyprpicker/assets/75806385/2579f8e5-7f49-4ef2-928c-3900dc097bb1

this is now fixed with 03bc27be945a2c83f5b8089ae03030e10f56e371
although flipped transforms will still be wrong cuz I am unsure how 2 flip in cairo


UpdateStreamTitle/Game error

i'm currently getting a 400 bad request error when i try to update a stream title/game and i'm not entirely sure what i'm doing wrong.
i've set the clientid and i've tried allowing access with just channel_editor and all possible scopes with the correct access tokens and it's still the same response
edit; forgot to mention it works as it should for me using the generic curl
curl -H 'Accept: application/vnd.twitchtv.v3+json' -H 'Authorization: OAuth <oauth>' -H 'Client-ID: <clientid>' -d "channel[status]=asdf" -X PUT https://api.twitch.tv/kraken/channels/<channel>

I'll look into this now.

So I just tested UpdateStreamTitle, UpdateStreamGame, and UpdateStreamTitleAndGame and they all appear to work. These calls do not require a clientid (since the oauth token can is used instead). I would make sure your OAuth key has the right scope (channel_editor).

As a side note, I tested using the Example Application provided in this project. Perhaps give that a try and try to implement in your own project using the example application.

i'm not entirely sure whats going on as i can literally copy your MakeRestRequest/UpdateStreamTitle functions line for line, plug my details in and it works but once i try from TwitchApi.UpdateStreamTitle i get the bad request error

I assume you are callig the TwitchApi function in an async method and awaiting the response, similar to how the example application does it?

    private async void button1_Click(object sender, EventArgs e)
    {
        await TwitchApi.UpdateStreamTitle(streamTitle, channelName, accessToken);
    }

is what isn't working for me and this pastebin link contains the code that does work for me

it appears to be working now. i downloaded and built the project myself instead of installing from the nuget repository

That's odd. I'll give it a go from the NuGet repo and see what the deal is.

Just tested and got the same error you were getting. Will have a fixed version on NuGet shortly.

Alright it's indexed, looks like NuGet package version 1.2.9 has fixed it. Thanks for bringing the issue to my attention 💯


chore: Refactor populateRepoDetails function and add types

Pull Request Summary

Added types for issues
Refactored populateRepoDetails function

The recent commit I made is from my company's github id @KrupaPanchal2527

Thanks @ssk090 for cleaning up the types.


Improvement for k8s.io/docs/tasks/administer-cluster/guaranteed-scheduling-critical-addon-pods/

An example would be great. Or at least a link to more details, e.g. https://kubernetes.io/docs/concepts/scheduling-eviction/pod-priority-preemption/
The documentation is splited in tasks and concepts. Search engines mainly lead to tasks section. It would be great if there could be links between these sections. Or even better do not split the documentation.
This is a general issue with the kubernetes documentation.

/assign

Can you elaborate the issue in more detail?

I feel that the documentation is great. Really great. It contains a lot of information and is very detailed.
Just the information is not in one place but in sections.
So if you search for something, in this example for scheduling critical pods than the search engine shows results in the docs/tasks folder. This gives a high level information. But does not tell how to do it. So you have to search deeper and will find details in docs/concepts and for even more detailed information in docs/reference.
It would be great if the documentation could be reorganized or at least have cross references between these section.

I agree that more cross references would help. That's a fairly easy task and I'd love to see folks logging small feature request issues that highlight where a cross reference is useful.

@amiga23 Any idea on how to go further on resolving this issue? Where should the text changes be applied/modified?


math/big: arm64 assembly code for shlVU is incorrect

The TestFloat64SpecialCases test in math/big (ratconv_test.go) is currently broken on the android/arm64 builder, with log:
https://build.golang.org/log/fbc581637a0375532f0d819ae67bc9b9abbe8ba9
Breakage started at e4ba400 (math/big: accept non-decimal floats with Rat.SetString).
cc @griesemer

Hm. It's only broken on android-arm-wikofever. @eliasnaur, it looks like you're the owner of that builder. Is there anything special about that arm device? It's a Cortex-A53 CPU?

I don't think it's a special device, other than it being an Android phone.
The test also broke on darwin/arm64 (iPhone):
https://build.golang.org/log/0b86dd35833a34f28785d335153946ed9a5a3691
which I assume uses an entirely different CPU.

Indeed, and that appears to be an iPhone 7+. Interesting.

Here's a smaller test case: https://play.golang.org/p/7OUzcEgoVGk
In the playground, and elsewhere, running this code produces the correct result:
r = 1/100000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000 (true)
len(r) = 308
f = 1e-305 (false)
e = 1e-305

On android-arm-wikofever (and presumably darwin/arm64) we get:
r = 1/96541187637860552075819821540903887880024095043179244007497829804714413865809870901655967120022719342752763287637146850362464306041721929996152029995888654682648990460561537640800770715572834310810731676381392326036599721803873644105423265719091992037347983408132957025607386110373485836090285769528130906264626714751533056 (true)
len(r) = 325
f = 1e-323 (false)
e = 1e-305

That is the fraction is completely wrong.

The fraction is correct for 1e-164, and then wrong for 1e165.

I've tracked this down a bit more. Here is a stand-alone test file (to be added to the math/big test files) that exposes the problem:
package big

import (
	"strings"
	"testing"
)

func TestSHLVU(t *testing.T) {
	// compute 10^n via 5^n << n
	const n = 165
	p := nat(nil).expNN(nat{5}, nat{n}, nil)
	p = p.shl(p, uint(n)) // <<< this doesn't work using ·shlVU
	// p = nat(nil).shl(p, uint(n)) // <<< this works

	got := string(p.utoa(10))
	want := "1" + strings.Repeat("0", n)
	if got != want {
		t.Errorf("got  %s\nwant %s\n", got, want)
	}
}

When run with the ·shlVU assembly routine (in arith_arm64.s) enabled, this test fails on arm:
$ gomote run -path '$PATH,$WORKDIR/go/bin' $MOTE go/bin/go test -run SHLVU ../src/math/big
go_android_exec: adb wait-for-device exec-out while [[ -z $(getprop sys.boot_completed) ]]; do sleep 1; done;
go_android_exec: adb push /var/folders/f6/d2bhfqss2716nxm8gkv1fmb80000gn/T/workdir-host-darwin-amd64-eliasnaur-android/tmp/go-build825267701/b001/big.test /data/local/tmp/go_android_exec/big.test-89906/big.test
[  1%] /data/local/tmp/go_android_exec/big.test-89906/big.test
...
[100%] /data/local/tmp/go_android_exec/big.test-89906/big.test
/var/folders/f6/d2bhfqss2716nxm8gkv1fmb80000gn/T/workdir-host-darwin-amd64-eliasnaur-android/tmp/go-build825267701/b001/big.test: 1 file pushed. 10.1 MB/s (6329512 bytes in 0.597s)
go_android_exec: adb exec-out export TMPDIR="/data/local/tmp/go_android_exec/big.test-89906"; export GOROOT="/data/local/tmp/go_android_exec/goroot"; export GOPATH="/data/local/tmp/go_android_exec/big.test-89906/gopath"; export CGO_ENABLED=0; export GOPROXY=; export GOCACHE="/data/local/tmp/go_android_exec/gocache"; export PATH=$PATH:"/data/local/tmp/go_android_exec/goroot/bin"; cd "/data/local/tmp/go_android_exec/goroot/src/math/big"; '/data/local/tmp/go_android_exec/big.test-89906/big.test' -test.run=SHLVU; echo -n exitcode=$?
--- FAIL: TestSHLVU (0.00s)
    shl_test.go:18: got<PHONE_NUMBER>99999999999999999999999999997461262869164377797676543286813257059020658007941683889923711150197932805888983316150450477342246492361833965695145908050591744
        want<PHONE_NUMBER>000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000
FAIL
exitcode=1go_android_exec: adb exec-out rm -rf /data/local/tmp/go_android_exec/big.test-89906
FAIL	math/big	0.988s
Error running run: exit status 1

When the assembly routine is disabled, the code works fine.
When the assembly routine is enabled but the test case is tweaked slightly such that it uses a nat(nil) receiver rather than p, then the code also seems to work. This is likely an indicator as to what might be wrong with the assembly code.
When fixing this, one should also look into ·shrVU and verify that it doesn't suffer from the same problem.
cc: @erifan @josharian who seem to have written some of this code

Looks like this routine was written by @erifan. The problem is definitely not obvious at a first read through the code. Let me know if you'd like me to take a closer look; otherwise, I will (tentatively) leave this for @erifan.

Also, the shift==0 case is missing an optimization. If z.ptr == x.ptr, there's no need to do a copy at all; you can simply return.
The pure Go code uses the copy built-in to do the copy, which does this optimization under the hood. amd64 is missing the shift==0 optimization entirely. I'll file a separate issue for that.

It's definitively subtle. The problem doesn't appear for n < 165, at least for https://play.golang.org/p/7OUzcEgoVGk code.

Sorry, I'll fix this bug.

The problem is that the loop in the assembler code goes forward through the arrays.  It has to go backward.  Otherwise the output overwrites the input.
The same is true of shrVU.

Of course I'm wrong about shrVU, which has to go forward.  Only shlVU has to go backward.

@bradfitz I can't imagine this doesn't fail on all arm64 CPUs, yet I see no arm64 builders on build.golang.org apart from the mobiles. Are they gone or am I blind?
With generic arm64 builders, gri wouldn't have to go through that painful debugging session.

The code has been written and I will submit it immediately.

I found that the implementation of shlVU on amd64 does not consider the case where the high byte of z overlaps with the low byte of x, so I also did not consider this point. Do we need to consider this situation?

We only have to consider word level overlap, not byte level overlap, if that is what you mean.  But we do have to consider the case where the high word of z overlaps with the low word of x.  I don't see any obvious problem in the amd64 implementation but maybe there is one.

Byte-overlap should never happen, it's either one or more words, or none - otherwise we'd have a serious bug elsewhere. You can safely assume that all pointers are word-aligned.
The assembly code should behave essentially the same as shlVU_g in arith.go.

Yes I mean Word level overlap. And I can construct an negative case for this situation, which can prove that the current pure Go implementation are defective.
func TestShlVUOverlap(t *testing.T) {
        a := argVW{nat{2, 2, 2, 2, 2, _M << 1 &_M, 1}, nat(nil).set(nat{1, 1, 1, 1, 1, 1, 1, 1, 1, _M}), 1, 1}
        b := nat(nil).make(10)
        copy(b, a.x)
        got := a.x[0:6].shl(a.x[4:10], uint(a.y))
        for i, zi := range got {
                if zi != a.z[i] {
                        t.Errorf("special shl(%v, %v)\n\tgot z[%d] = %#x; want %#x", b[4:10], a.y, i, zi, a.z[i])
                        break
                }
        }
}

Add this test to file math/big/arith_test.go,  and run
$ ../../../bin/go test -run TestShlVU
With the pure Go implementation, we get:
--- FAIL: TestShlVUOverlap (0.00s)
arith_test.go:260: special shl([1 1 1 1 1<PHONE_NUMBER>3709551615], 1)
got z[0] = 0x4; want 0x2
FAIL
exit status 1
FAIL    math/big        0.072s
The current use cases of function shl didn't expose this bug, and as function nat.shl is not a public function, so I wonder if we need to consider this situation when implementing this function or is there some mechanisms that can guarantee this situation never happen?

shlVU is an internal function, so we don't need to consider all possible ways to call it.  We only need to consider uses of the exported methods.  We can trust that the code in the math/big package understands the limitations of shlVU.

We can trust that the code in the math/big package understands the limitations of shlVU.

The code might, but I couldn't tell you those limitations offhand. It'd be good to document them.
Once they're documented, we can start fuzzing them using those documented limitations. Since there are multiple implementations, these are good fuzz candidates. (I have started on this.)

@gopherbot  Please open backport issue to 1.13

@erifan Why do you think this should be backported to Go 1.13? Go 1.13 hasn't been released yet, it can't have backports. Did you mean another release?

Hi @dmitshur , I just hope that this fix can go into Go1.13. This fix was merged before the release date of Go1.12.6, but it did not enter the Go1.12.6. I'm not sure if it will go into Go.1.13, so I only raise this request.
It would be better if it could get into an earlier release, because some programs are now affected by this bug, such as https://github.com/shopspring/decimal.

@erifan Since the fix is in master now, it will be a part of Go 1.13 when we release that. We create new release branches and the cherry-pick process begins at the time of the first release candidate, but we haven't cut that yet (only the first beta has been made by now).
I'll close the 1.13 cherry-pick issue since it's not necessary.

@dmitshur Ok, thanks.

Excuse me @dmitshur , There's another question. Will there be a Go1.12.7 version before Go1.13?  If so, I hope this fix can be ported in Go1.12.7. Or, if there is Go1.12.7, no backport is needed and this fix will be included? Thanks.

There will be a Go 1.12.7 release soon (we aim to make minor releases monthly). We can consider cherry-picking this fix to 1.12 if it meets the backporting criteria (described here). If it doesn't, then it'll be available starting with 1.13.

@erifan According to the original issue, this breakage started with commit e4ba400. That commit isn't a part of 1.12. So does this problem exist in 1.12?

@dmitshur In Go, the breakage started from commit https://github.com/golang/go/commit/e4ba40030f9ba4b61bb28dbf78bb41a7b14e6788, but for user code, the affected version may be earlier than 1.12, and there is currently no release version that fixes this issue. The user of Project  https://github.com/shopspring/decimal complained to me about this issue. Since it is a library function, users will use it frequently, so I think this is a security issue. If possible, please port it to Go1.12.7, thank you.

Ok, thanks @erifan. I've repurposed #32940 as the 1.12 backport issue.
I'll discuss whether it meets the backporting criteria with the team tomorrow. If it gets approved for a cherry-pick, we'll need to make a cherry-pick CL for it as described here; would you like to work on that?

@dmitshur Yeah, I'd like to, thanks.


generated / license popups bad in dark theme

Moving the 'generated' and 'license' info in to popups, using the Bulma css classes works fine in the light theme, but not in the dark theme. (The light theme for these components could be tweaked a bit too ??)
Extra scss styling needs to be added in Website/plugins/ogdenwebb. Currently, the styling is coming from ... ogdenwebb/scss/main.scss which is importing from ... ogdenwebb/bulma/components.scss.
Probably the best place to put new dark styling would be ... scss/themes/dark.scss

@rawleyfowler could you do this?

@finanalyst Yeah I'll look into it tomorrow.

Should be finished this tomorrow. Sorry.


BXF verification sum needs to be improved

The verification sum was  a nice idea on paper. It provides an easy way to verify that the matrix a future program is reading is the same matrix that was written to a file. The problem is that the implementation is not practical, and is unreliable. Namely, there are a few key issues with the current implementation:

reliance on floating point arithmetic... and comparing the result thereof to a very exact value
Breaks if the matrix changes symmetry. The purpose of the sum is not to verify the file is the same, but that the matrix is the same.

I don't really have a better idea at this time. Maybe something with parity. The issue is making it work correctly in multiple languages (can't rely on numpy/scipy/python specific stuff). I don't really want to drop the feature entirely, but I'm not really sure of the usefulness of the current implementation.

Until a suitable alternative is found, all verification sums will be read and written as "1". I don't plan on merging to master like this, just make development easier until a better method can be found.

I have decided to drop the verification sum in the upcoming BXF 2.1 specification due to limited utility and difficulty of implementation, as discussed in the initial issue. Verification sums may be added back in a future version, if a better method can be found.


Updating the docs for authentication via CLI

Signed-off-by: Kamal Nayan<EMAIL_ADDRESS>Description
This PR fixes #6316
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit 0dc702e521846550910595a3aa083ad397f596fd at: https://633be333208d924403f6b455--meshery-docs-preview.netlify.app


testing: Make loom optional on per-package basis

For now, loom is only enabled for storage and blockchain-storage.

I would more advocate for everything to be explicitly taken off loom to ensure everything is tested; in fact, I would even advocate for working more on why they're not working, assuming loom doesn't have bugs. I know it's looking like a problem now for every new crate, but all there's to this is getting used to it.
I'm approving, but let's keep restoring this option on the table.

The main issue is that loom only supports "pure" concurrency, so anything to do with IO, timers, networking, etc. cannot be run under loom. This mainly applies to p2p and rpc. Then there is also the consideration that some crates have no concurrency (for exmaple script and common), so there is no benefit in running them using loom.

Go ahead and merge it. We can discuss this later in detail.


How to set HTTPHeaderField?

How to set HTTPHeaderField?
eg:[mutRequest addValue:@"101" forHTTPHeaderField:@"cmd"];
[manger POST:URL parameters:dic success:^(AFHTTPRequestOperation *operation, id      responseObject) {
} failure:^(AFHTTPRequestOperation *operation, NSError *error) {
}];

Sorry, I am a novice

This is not a support forum. For usage questions, try Stack Overflow or any of the other available resources.


Win10 x64, vs2019, msvcrt not found error

Try all steps to build but get error
    assert optional, "%s is not found in %s:\n%s\nCheck if it is installed." % (
AssertionError: msvcrt.lib is not found in LIB:
C:\Program Files (x86)\Windows Kits\NETFXSDK\4.8\lib\um\x86
C:\Program Files (x86)\Windows Kits\10\lib\10.0.19041.0\ucrt\x86
C:\Program Files (x86)\Windows Kits\10\lib\10.0.19041.0\um\x86

Tried to google this issue and install some additional packages - same result.

Handled with running vc2019 developer console, not trivial command prompt

If I run developer prompt, there is also other issue, "user32.lib not found".
Vs installed in standard path

build.py choose vs2022 vsvars.bat instead of 2019.
result = subprocess.run(
        '"{}" -prerelease -version [15^,17^) -property installationPath'.format(vswhere_exe),
        shell=True,
        check=True,
        stdout=subprocess.PIPE,
        universal_newlines=True)

This code helps  choose right version


Search is poor

Expected behavior
when searching for jenkinsci/jenkins it should put https://hub.docker.com/r/jenkinsci/jenkins/ first
Actual behavior
https://hub.docker.com/r/jenkinsci/jenkins/ is not top
Information about the Issue
search functionality is poor, would be good if you could explore users repos like you can on the web
Steps to reproduce the behavior

search for jenkinsci/jenkins
don't find https://hub.docker.com/r/jenkinsci/jenkins/ at the top of the list

I agree that when passed as such, the jenkinsci/jenkins repo should be the first one to show.
Kitematic's search behavior is consistent with what Hub shows:
HUB:

jenkinsci/jenkins is the 3rd repo.
Kitematic:

jenkinsci/jenkins is the 3rd repo.


Contacts do not get deallocated, I have noticed app storage on device increase significantly with each pull

First, awesome job on this module, it is very helpful! For reference I am using this only for iOS right now. This is an issue and what I think would be a solution, but not sure how to write it in swift.
I have noticed while using the module that it seems to increase the size of my app on my iPhone's storage by what I am guessing is the size of the contacts list, every time that I call getAll (which is just on app start).
I am guessing this is because the contacts array created here (https://github.com/rt2zz/react-native-contacts/blob/master/ios/RCTContacts/RCTContacts.m#L70) is not being deallocated. I have never written swift but a quick search shows that instead of init on that line, it would be autorelease instead, and at some point [contacts removeAllObjects] would be called. Again, I do not know where that remove all objects could be called because the array has to be returned first.
Any help on this would be awesome, thanks!

To be clear the ios code is all objective-C. I may write future features in swift though.

@morenoh149 ah I see sorry about that. Does the proposed issue make sense though?

I don't see autorelease as a valid argument into the NSMutableArray declaration https://developer.apple.com/library/ios/documentation/Cocoa/Reference/Foundation/Classes/NSMutableArray_Class/index.html#//apple_ref/occ/instm/NSMutableArray/initWithCapacity:
Mind forking the repo and testing it locally? https://docs.npmjs.com/files/package.json#git-urls-as-dependencies

Yes I will fork and test and report back

I wonder if its the icons files?  Does you address book contain lots of contacts with pictures?  When I say lots I mean thousands?

I'm glad I found this issue because I was wondering why some users were reporting that the app would become unresponsive after importing about 10 or so contacts.
Now I just bypass the getAll() call if the contacts are already loaded in state and the app is fine. The only downside of course is that it's not retrieving the most updated list of contacts.

So what's the proper way to deallocate the contacts array? I'm seeing my app not respond after calling getAll() about 10 or so times.

@coderdave there is an outstanding PR https://github.com/rt2zz/react-native-contacts/pull/118 you could help test.
@npomfret submitted and merged a fix for the file handle leaks at https://github.com/rt2zz/react-native-contacts/pull/99 @coderdave are you using version >= 0.5.2 ?

@morenoh149 I upgraded to version 0.6.0 and it looks like it's better, but after about 60 calls to getAll(), the app becomes unresponsive.
I'll have to test the PR next.

Which OS?

Can we close this?  The iOS implementation has been completely re-written.  I think it doesn't suffer from this problem any more.


Zachg/parametric enable w3c phase1 tests python

Description
Enable tracestate propagation test
Motivation

Workflow

⚠⚠ Create your PR as draft
Follow the style guidelines of this project (See how to easily lint the code)
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Once your PR is reviewed, you can merge it! :heart:
Reviewer checklist

[ ] Check what scenarios are modified. If needed, add the relevant label (run-parametric-scenario, run-profiling-scenario...). If this PR modifies any system-tests internal, then add the run-all-scenarios label (more info).
[ ] CI is green

[ ] If not, failing jobs are not related to this change (and you are 100% sure about this statement)

[ ] if any of build-some-image label is present

is the image labl have been updated ?
just before merging, locally build and push the image to hub.docker.com

[ ] if a scenario is added (or removed), add (or remove) it in system-test-dasboard nightly

Sorry, forgot we need to release version with these changes first.

Ok now that 2.3.0 is out we should be all set.


[Tracking] Falco 0.37.0

Falco 0.37.0 Release
Will keep this issue updated with the current status and progress.
Date
Release date: 2024-01-29
Release Steps
The process is described in this document.
## External dependencies
- [x] falcoctl -> 0.7.1
- [x] libs -> 0.14.2
- [x] drivers -> 7.0.0+driver
- [x] rules -> 3.0.0
- [x] plugins bump
- [ ] charts

## Manual Testing Action Items
- [x] Running Falco on Kubernetes with the official Helm Charts
- [x] Running Falco from RPM and DEB artifacts
- [x] Running Falco in a container with the official images
- [x] Running/fuzzing Falco with multiple event sources active in parallel
- [x] Running/fuzzing Falco with variable syscall buffer dimension
- [x] Running Falco in all officially-supported architectures (x86_64, ARM64)
- [x] Running Falco with the supported drivers (kmod, modern eBPF, eBPF, gVisor, trace files)
- [x] Test Falco with event generator
- [x] Test that plugins are correctly loaded
- [x] Test memory and CPU usage (with Valgrind and other tools)
- [x] Test latest version of driver loader
- [ ] Test that `k8smeta` plugin works as expected
- [x] Test that all Falco CLI options work as expected
- [x] Check that Falco log messages are correct and consistent
- [x] Test that Falco ruleset loading and validation works as expected
- [x] Test Falco with Gvisor

Action Items

[ ] Pre-Release

[x]  Milestone
[x] Code freeze in falcosecurity/falco
[x] Open release branch in Falco and protect it https://github.com/falcosecurity/test-infra/pull/1358
[x] See External dependencies task lists
[x] Prebuilt drivers publish
[x] Code thaw in falcosecurity/falco
[ ] Changelog

[ ] Release

[ ] Github Release in the Falco repo

[ ] Website

[ ] Create new snapshot

[ ] Protect snapshot branch

[ ] Merge all necessary PRs

[ ] Helm

[ ] Release a new helm chart version
[ ] Update helm chart documentation
[ ] Add instructions to the Helm chart readme about how to switch images if necessary https://github.com/falcosecurity/charts/pull/538

[ ] Announcements

[ ] mailing list
[ ] blog post
[ ] slack channel

[ ] Post-Release

[ ] Update supported fields and syscall documentation https://github.com/falcosecurity/falco-website/pull/1164
[ ] Archive community call meeting notes https://github.com/falcosecurity/community/pull/198

Miscs
/milestone 0.37.0
/assign @Andreagit97
/kind documentation
PRs that should be cherry-picked into the release branch

https://github.com/falcosecurity/falco/pull/3030
https://github.com/falcosecurity/falco/pull/3028

Synced with: https://github.com/falcosecurity/falco/pull/3031

https://github.com/falcosecurity/falco/pull/3032
https://github.com/falcosecurity/falco/pull/3033
https://github.com/falcosecurity/falco/pull/3015
https://github.com/falcosecurity/falco/pull/3034

Synced with: https://github.com/falcosecurity/falco/pull/3035

https://github.com/falcosecurity/falco/pull/3036 directly pushed on the release branch

Falco 0.37.0-rc1 is out https://github.com/falcosecurity/falco/releases/tag/0.37.0-rc1!
Please help us test it if you have time!

Falco 0.37.0-rc2 is out https://github.com/falcosecurity/falco/releases/tag/0.37.0-rc2!
Please help us test it if you have time!

Falco 0.37.0-rc3 is out https://github.com/falcosecurity/falco/releases/tag/0.37.0-rc3!
Please help us test it if you have time!

Falco 0.37.0 is finally out https://github.com/falcosecurity/falco/releases/tag/0.37.0! :rocket:


Mouse release event is lost if the pane becomes inactive

Micro delivers mouse events, including mouse release events, to the currently active pane or tab only. As a result, in the following scenario:

press a mouse button on an active pane or tab
switch to another pane or tab (or, for example, switch to the infopane, e.g. begin typing a command), while keeping the mouse button pressed
release the mouse button

the mouse release event is delivered to the new active pane, not to the original one. As a result, when we switch again to the original pane, it thinks that the mouse is still pressed, and thus handles mouse events incorrectly.
Some examples of issues caused by this bug:

If the file opened in the bufpane has been modified by another program and then we click on the bufpane with the mouse, micro shows The file on disk has changed. Reload file? (y,n,esc) prompt. And then, after pressing y in this prompt, scrolling via the mouse wheel stops working. It starts working again only after clicking a mouse button once again. (In this case, the issue occurs because the prompt switches the focus from the bufpane to the infopane immediately after the mouse is pressed, before it is released.)

Press a mouse button, switch to another split pane via Ctrl-w (without releasing the mouse), release the mouse and switch back to the original pane. Then click mouse somewhere in that pane. Result: unexpected text selection.

Begin resizing a split pane by dragging the split pane divider with the mouse, switch to another tab via Alt-. (without releasing the mouse), release the mouse and switch back to the original tab. Then click mouse somewhere. Result: unexpected resize of the split pane.

Commit hash: f386b29e
OS: any
Terminal: any

Could/should react upon the release earliest, but...
Should perform the release automatically, before the next tab/pane change takes place
Should perform the release automatically, before the next tab/pane change takes place

...so even the 1. could to this too and the same mechanism applies for all scenarios.


Inconsistent behaviour of geom_gate and geom_stats

When using ggcyto to plot a flowSet, geom_gate accepts a single filter object as an input and plots it across all samples without any trouble. Yet geom_stats cannot parse that gate data and returns a non-descriptive error message.
firstGate <- rectangleGate("SSC-A" = c(0, 5e6), "FSC-A" = c(0, 2e6))
p <- ggcyto(flowSet, aes(`FSC-A`, `SSC-A`)) +
  geom_point(size = 0.1) +
  geom_gate(firstGate)
p + geom_stats()

> Error in `[.data.table`(df, , c(markers, ".rownames"), with = FALSE) :
  column(s) not found: .rownames

This behaviour can be rescued by creating a filterList (as described in the documentation) and passing it either to geom_gate or geom_stats.
firstGates <- sapply(sampleNames(flowSet), function(x) firstGate)
p + geom_stats(firstGates)

Maybe you could add a nice error message for this case, so other poor souls like me, who only read the docs after there is a problem, get put on the right track. Had a really hard time finding out what's wrong.

I've pushed the fix. Thanks for your feedback!


Restore Forge support

This PR restores support for Forge 1.21.1.
It also uses the mods.toml clientSideOnly feature.

Thanks for the PR, however as it stands, I have no interest in expanding the list of my mods that support Forge right now.


Can I train a VL model based on Intern20B from scratch?

Will you release the pretrain script for internlm-vl?

Currently we do not have the plan to release the pre-training script.


Slim down initial_dataset_visualization.ipynb

In actuality, the initial_dataset_visualization notebook is a list of visualization that worked and didn't suck. But as is there is unnecessary repetition of UI. E.g. there is (shitty b/c max of slider is hard coded, not dynamically set to depth of z-stack) Form based z-stack navigator and there is the google.colab.kernel.invokeFunction technique which is much better. So, move the former to an also ran notebook.

This will as evolve into biocytin_on_brightfield_visualizer.ipynb.


Fix number of counties in geospatial-data.Rmd hint to be consistent with code

Update markdown hint text for part two  from "10 smallest counties" to "5 smallest counties" to match the number used in the hint-provided code

Thanks!


FIX: Ensure query string in X-Backurl is encoded (fixes #3563)

Bump. Would’ve been nice to get this into 3.1.9, given that this bug breaks the ModelAdmin filter functionality in some languages.

Seems legit to me, but as @tractorcow has been looking over it, I think it's best he makes the final call.
As there are no tests, has this been tested extensively by hand to check if double encoding is handled correctly by this?


cross-project bug: async_trait and tracing::instrument are incompatible

The combination of macros results in the instrument macro thinking the instrumented function is not in fact async and generating the wrong code.
see full writeup here, cross-posting b/c someone on the tonic/tracing/etc discord suggested it. Thanks for providing this crate!

@dtolnay I wanted to check: how would you suggest resolving this? Since the #[tracing::instrument] macro is "lower" in the hierarchy of expanded macro, I expect that the fix would need to be in tracing. I'm thinking that the #[tracing::instrument] macro should attempt to match on the 'async_trait lifetime and generate code accordingly, but that feels a bit fragile.

I think this would be the most reliable pattern to identify: https://github.com/dtolnay/async-trait/blob/1.0.15/src/expand.rs#L402-L403
So if you find a function whose body consists of:
async fn f<...>(...) {...}
Box::pin(f::<...>(...))

then treat the instrument macro as if it had been applied to the inner function not the outer one.

I'll probably start with that.
I wonder if this can/should be generalized. For instance, async-trait can leave behind some sort of marker struct so that other macros can recognize that some code been expanded by async-trait, but I suspect that Box::pin at the end will be sufficient.

This should be fixed now by the merge request above. Thanks @dtolnay for outlining the pattern to look for to identify async_trait, I followed your suggestion and it works great !

Wow, thanks @nightmared. Would you be able to follow up with a PR in async-trait to add a test with #[instrument] so that we avoid inadvertently breaking it?

You want to add a reverse test ? I'm not sure we can use the mock tests inside tracing outside of the crate. Otherwise we could do something like https://github.com/tokio-rs/tracing/blob/369b0c5e1d770f26c5e98a3e9f8fd9d4c6dfde8c/tracing-attributes/tests/async_fn.rs#L35-L152.
I will think about it in upcoming days

Edit: indeed tracing::support is not exposed publicly :/ I will need to think to find a "compact" way to test for this behavior.

There has been some discussion about publishing the test-support code as its own crate (see tokio-rs/tracing#539). If we were to do that, it could be pretty easy for async-trait to have its own version of the test?

But is there no way to test this in a setup that is representative of how downstream crates use async_trait + tracing? I don't know anything about tracing but I feel like we shouldn't need to depend on internal test-support code for this.

@dtolnay The best way to test that everything works correctly is to write an implementation of the tracing::Subscriber trait that asserts that an expected sequence of events (creating a span, entering the span, exiting the span, and so on) occur in the expected order. This is all the tracing test support code does: it provides a builder pattern style API for constructing a Subscriber that makes these assertions. It's definitely possible to write such a test without it, it just requires writing a bit more code.
It would also be possible to test this using a Subscriber implementation that's actually used in the wild (such as tracing-subscriber::fmt) and then making assertions about its output. This would be akin to testing that something happened by constructing a log logger and asserting that it logs a particular record. I wouldn't suggest this, since it's likely much more brittle: changes in how traces are displayed, which are unrelated to the actual behavior we want to test, could break the test.

Thanks for the explanation and links!
I'd be fine having a small tracing::Subscriber impl dedicated to this one test. For an async trait method containing one event, all we care about in this crate is that the event happens before the exit (as opposed to before @nightmared's fix, where the exit would happen as soon as the BoxFuture was created). A struct with a Cell<bool>, or even a static with an AtomicBool, would be enough for our purposes.

Yup, I think that's probably fine. We might also want to assert that if the future is polled, the span is re-entered as expected, and exited when the poll returns. This could definitely be done with a handwritten test subscriber — the test support code is mostly useful in the context of tracing itself, where we need to have a very large number of tests that make similar assertions. :)

I hacked together a prototype at https://github.com/nightmared/async-trait/commit/b544a1b0da3abd426102779492a3d85f20ba3b85. Would something similar be okay ?

Something like that looks reasonable to me.


Prediction and training docker files for ml_backend

This can be done quite easily by just using the same dockerfile for both (only changing the CMD), right?


Mysql does not support " NOT LIKE " SQL statements

Mysql does not support " NOT LIKE " SQL statements
select * from users where host NOT LIKE '%abc%';
https://github.com/balderdashy/waterline-sequel/issues/98

Referenced  https://github.com/balderdashy/waterline-sequel/issues/98


get_viewport().size does not resize after going to full screen without waiting x frames

Godot version:
3.2.2 Stable
Issue description:
When you switch in full screen and then resize the viewport get_viewport().size, it does not always work. I found that waiting x frames seem to solve the issue.

On mac, in this minimal project, I don't need to wait frames. But in my real project, in need to wait one.
On windows, you need to wait 2 frames
On Linux, you need to wait 3 frames

(I tried with two different computer)
Steps to reproduce:
Execute the minimal project, the viewport should be pixelated (320, 180).
Minimal reproduction project:
bugViewport.zip

Related to https://github.com/godotengine/godot/issues/28916 and https://github.com/godotengine/godot/issues/39834.


Scoping seems screwy for JsDiagEvaluate()

To illustrate just one of the glitches I've encountered (there have been others):
SSj 5.0a1 Sphere JavaScript debugger (x64)
the powerful symbolic JS debugger for Sphere
(c) 2015-2017 Fat Cerberus

starting 'C:/src/kh2bar/dist/'... OK.
connecting to <IP_ADDRESS>:1208... OK.
establishing communication... OK.
downloading game information... OK.
    platform: miniSphere 5.0a1
    title:    kh2Bar Showcase
    author:   Fat Cerberus

-> # 0: module(), at #/game_modules/sphere-runtime.mjs:33
33 export const assert     = require('assert'),

#/game_modules/sphere-runtime.mjs:33 module()
(ssj) bp src/main.mjs:32
breakpoint # 0 set at src/main.mjs:32.
32              this.colorID = 0;

#/game_modules/sphere-runtime.mjs:33 module()
(ssj) c
log: kh2Bar Showcase Debug Console
log: Sphere v2 API lv. 0 (miniSphere 5.0a1)
log:
-> # 0: Showcase(), at src/main.mjs:32
32              this.colorID = 0;

src/main.mjs:32 Showcase()
(ssj) l
     27 {
     28         constructor()
     29         {
     30                 super({ priority: 0 });
     31
->   32                 this.colorID = 0;
     33                 this.comboTimer = 0;
     34                 this.hp = 812;
     35                 this.isHidden = false;
     36

src/main.mjs:32 Showcase()
(ssj) e this
eval() = class Showcase extends Thread
{
        constructor()
        {
                super({ priority: 0 });

                this.colorID = 0;
                this.comboTimer = 0;
                this.hp = 812;
                this.isHidden = false;

                // construct a new HP gauge and make it visible
                this.lifeBar = new HPGauge(16

src/main.mjs:32 Showcase()
(ssj) _

Why did this evaluate to the class itself rather than an instance?

Another example: Here, a parameter variable which is still in scope can't be eval'd in the debugger and is claimed to be not-defined:
#/game_modules/console.js:86 [anonymous function]
(ssj) l 20
     76 });
     77
     78 exports.initialize = initialize;
     79 function initialize(options)
     80 {
     81     options = options !== undefined ? options : {};
     82
     83     activationKey = options.hotKey !== undefined ? options.hotKey : Key.Tilde;
     84     Dispatch.onRender(_render, Infinity);
     85     Dispatch.onUpdate(function() {
->   86         _getInput();
     87         _update();
     88     }, Infinity);
     89 }
     90
     91 exports.defineObject = defineObject;
     92 function defineObject(name, that, methods)
     93 {
     94     for (var instruction in methods) {
     95         commands.push({

#/game_modules/console.js:86 [anonymous function]
(ssj) e options
error: 'options' is not defined

@fatcerberus for your second case (options) if I am reducing it to a small repro correctly
var onUpdate = function (fn) {
  fn();
};
function initialize(options) {
  onUpdate(function () {
    debugger;
  });
}
initialize({
  a: 1
});

breaking at debugger statement options is not visible (Chrome have same behavior. Only Firefox can evaluate options correctly).
Does the above test represents what you are trying to do? if so I think the behavior is correct as options is not captured under function passed to onUpdate.
For your first case can you provide a small test?

Hm, so only the variables that are actually used in the inner function can be evaluated?  When I was using Duktape all variables were captured by a closure.  Inconvenient, but I guess if the variable isn't used it's irrelevant for debugging anyway.
The first case can be repro'd with a very simple:
class C {
  constructor() {
    this.a = "a";
    this.b = "b";
    this.c = "c";
    debugger;
  }
}
new C();

After the breakpoint is hit, evaluate "this", and the result is:
@/test.js:6 C()
(ssj) l
      1 class C {
      2   constructor() {
      3     this.a = "a";
      4     this.b = "b";
      5     this.c = "c";
->    6     debugger;
      7   }
      8 }
      9 new C();

@/test.js:6 C()
(ssj) e this
eval() = class C {
  constructor() {
    this.a = "a";
    this.b = "b";
    this.c = "c";
    debugger;
  }
}

Just to verify:
@/test.js:6 C()
(ssj) e 'this === C'
eval() = true

Yes that seems like a bug
class C {
  constructor() {
    this.a = "a";
    print(this===C);
    /**bp:evaluate('this===C');**/
  }
}
new C();

When run with ch -debuglaunch test.js print return false but the evaluate (eval under debugger) return true. Same behavior is in Edge.
[
  {
    "evaluate": {
      "this===C": "boolean true"
    }
  }
]

@akroshg to triage

I have a feeling the behavior described here https://github.com/Microsoft/ChakraCore/issues/3762#issuecomment-330654786 might be related (argv[0] for a native constructor is set to new.target rather than this).


Add additional .NET Frameworks

See also #7 (.NET Standard) and #8 (.NET Core)
We currently have @net472. We should do the same thing for the other .NET Framework versions.
This will be largely copy/paste work. We will need to (possibly not immediately) regenerate the net472.BUILD file for those frameworks too.
.NET Standard and .NET Core work slightly differently than .NET Framework in this respect so I'll open up a different issue.

Mostly blocked on #19 now


Need web client testing

Currently, we cannot detect that the web client works as expected. We need a way to test it automatically.
Testing should be done in travis-ci, on the built server.

We should take a look at phantomjs.


Reworked version of CTC+attention model

It would be nice to have a "reworked" [as in https://github.com/k2-fsa/icefall/pull/288] version of the CTC+attention setup (in a separate directory).
That would be a good reference for cases where we need a standard transformer with the changes I made in that dir;
advantages should be mostly speed of training, hopefully slightly better results, esp. when data is very large;
but also ability to train in half precision without NaN's.
This would involve re-adding transformer.py, replacing nn.Linear units with ScaledLinear and nn.Embedding with
ScaledEmbedding; removing most LayerNorms and replacing the rest with BasicNorm; and using the same optimizer
I am using (well, taking most changes from train.py).  Also probably replace the feedforward modules with the
feedforward module used in conformer.py, use the same subsampling-embedding module as conformer.py, simplify
the transformer forward function in transformer.py, remove pre_norm option and vgg_subsampling option, add
model-warmup option.  In fact it may end up being easier to just copy conformer.py to transformer.py and
modify it a bit.
This is low priority though.  Just creating the issue so we don't forget.

@csukuangfj @danpovey Do you think it's reasonable to add warmup for the decoder module in conformer? In Librispeech conformer_ctc2 recipe I see that the author added this option in the new version of the decoder, but left it to a default value of 1.0 during training in the decoder.

Sure, I think it's worth a try to see whether it helps convergence at the start!


First Release (0.1.0)

Just bumps the versions and makes some adjusments to Cargo.toml and README.md files.

@niklasad1 Oh indeed, the transformer is not exported, hmm I wonder if we should just make it pub(crate) and use the Transformer internally in the 3 different implementations of it (rust value example, scale value example, type description). It's interface seems too weird for most users anyway, I guess.

Ok, I see pub(crate) make sense to me then :)
Nothing, really that blocks this release but just wanted to check

The 0.1.0 release already slipped through accidentally 3 weeks ago, but if something still needs changes, I will just issue a patch. I am using the released version in subxt and subxt explorer and it works great so far.

yeah, just merge the PR :)

Just a couple of small things to ponder/reply to from me :)


[MB-14459] Move history should show correct labels for different orders type

Jira ticket for this change
Summary
Setup to Run Your Code

💻 You will need to use three separate terminals to test this locally.
Terminal 1
Start the UI locally.
make client_run

Terminal 2
Start the Go server locally.
make server_run

Additional steps

Access the office
Login as a TOO
Open a move and changes the orders type to retirement and changes the Date of retirement

[ ] There should be a move history detail item with date of retirement and order type retirement

Open a move and changes the orders type to separation and changes the Date of separation

[ ] There should be a move history detail item with date of separation and order type separation

Screenshots

Messages

:book:
:link: MB-14459

  Generated by :no_entry_sign: dangerJS against 6b4067589d71061bf1ac1fcf1c57e509b9491937


feat(cucumbertemplate): allow for loading feature files that are in a…

…ncestral directories
closes #245

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Make TorXakis v0.4.0 release

Once all the main tasks associated with the Q3 milestones are completed we need to make a release for TorXakis.

[ ] Create a PDF out of Modelling Examples wiki pages
[ ] Update installer generation to include the PDF in documentation folder
[ ] Create 0.4.0 installer

https://github.com/TorXakis/TorXakis/releases/tag/v0.4.0


[question] Is this SDK still being updated?

Hi,
I'm looking in to using this SDK to supply our need to integrate our ecomm to Dafiti's marketplace and i've noticed that the last time this SDK was updated was almost 3 years ago.
Is this SDK still updated or is it deprecated ?
Thanks!

Hi Alexandre,
We still using the API to integrate few partners straight to sellercenter
through out our middleware system although it isn't used to integrate do
e-commerce system but to SC.
Att:
On Mon, Jul 15, 2019 at 11:37 AM Alexandre Luis<EMAIL_ADDRESS>wrote:

Hi,
I'm looking in to using this SDK to supply our need to integrate our ecomm
to Dafiti's marketplace and i've noticed that the last time this SDK was
updated was almost 3 years ago.
Is this SDK still updated or is it deprecated ?
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GFG/seller-center-sdk-php/issues/26?email_source=notifications&email_token=AAIPC2DDOCUZVLMIA34XFELP7SDS7A5CNFSM4IDXQV2KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G7HUQ7Q,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIPC2ASIKT23J675SEETNDP7SDS7ANCNFSM4IDXQV2A
.

--
Att:
Thiago Risso

Hi Thiago,
My question is exactly that, i've been requested to integrate our product to sellercenter as a partner (still working on how to get the necessary access to get the auths though)
I've been reading through this docs:
https://sellerapi.sellercenter.net/docs/getting-started
And it pointed me to this repository, i've seen some of the examples and it looks like what i need.
My question was about the long time since the last update, but i guess it is still maintened just fine.

Hi @lokharth
I used it up until now, and everything is working fine.

Hola.
Hay algún integrador que permita imprimir directo a Zebra con solo dar click sin abrir el PDF?
Gracias


Is there a way to disable just widow protection for a particular element?

Is there a way to disable just widow protection for a particular element like there is the isTtitle parameter to disable hyphenation in titles/headings?
I want to disable widow protection on just one element, where the width is so little that widow protection looks worse (I want to continue using the typogrify filter for other functionality).

Yep, you can do that as described here: https://nystudio107.com/docs/typogrify/#advanced-usage

Yep, you can do that as described here: https://nystudio107.com/docs/typogrify/#advanced-usage


Add NFTCollectionDisplay view to provide overview of an NFT's parenting collection

Provides a view that allows an NFT to provide basic information about how to display its parenting NFT Collection.
Similar to the collection view here: https://github.com/onflow/flow-nft/pull/82, this view provides a way of going from already having an NFT reference to retrieving the collection information that is parent to that NFT. For that reason, this has specifically been named NFTCollectionDisplay to not be confused with the Display view which is a generic Display view for any type of resource.
Related issue: https://github.com/onflow/flow-nft/issues/62

This is really needed, and since it is no the NFT level it works perfectly for multi-minter contracts aswell.


POST method fails when using with new Unleash Edge service

I'm not sure if the issue is on the Edge side or within this unleash-proxy-client or maybe I should be using a completely different frontend JS client when switching from Proxy to Edge, but I'll start with this issue here.
I have the unleash-proxy-client setup and working fine and it is making its requests against an Unleash Edge server.  Everything seems to be working fine.  However, we wanted to switch from using GET to POST.  When we set usePOSTrequests to true, this part of the code:
https://github.com/Unleash/unleash-proxy-client-js/blob/81903aa93a2ca0ffc248064852e9d960e196a344/src/index.ts#L563
builds a POST payload that takes the form of (eg, a generalized example for illustration purposes):
{"context":{"sessionId":#######,"appName":"APP_NAME_HERE","environment":"ENV_NAME_HERE"}}
However, when that request hits the Edge server, it responds with:
Json deserialize error: invalid type: map, expected a string at line 1 column ##
I did some testing, and if we remove the "context" piece and only send as the payload the contents of context, then the Edge server responds appropriately.
Question:

Should I be using a different client against Edge?
If not, then does this part of the code need to be updated to no longer inject {"context":... to the POST payload?

Hi @mhuffsti , sorry for the delayed response. I've made https://github.com/Unleash/unleash-edge/pull/510 which should fix it, and will be released as soon as @sighphyre and myself can agree on the way forward.

Way forward agreed. We'll release 19.3.1 sometime today and I'll update this again when you can upgrade your Edge.

Hi @mhuffsti , could you please try to use 19.3.1. It worked for us, but not closing this ticket until I have confirmation from you that it works from your end.

Thanks for looking into this so quickly.  I can confirm that it is better, yes, but maybe something more is needed in terms of the validation of the JSON POST payload on the Edge side?
Specifically, if I set usePOSTrequests to true on the proxy client, the 19.3.1 release of Edge is parsing the "context" part now.
If I set a sessionId as a string, then it seems quite happy.
However, if I let the proxy client autogenerate a sessionId, it generates a number and sends that number like this:

Edge then generates a 400 Bad request with this response:
Json deserialize error: invalid type: integer 299923022, expected a string at line 1 column 33
Should it be expecting a string?  Or should it accept either a string or an integer?

It should be expecting a string, at least that's what our schema says, but I guess proxy has just been tolerating this since it's javascript. But with actual typed languages, this breaks. I'll make a PR on this repo

Ok, bugfix commited, will release a 3.6.1 of the client.

3.6.1 released which will post sessionId as a string also if auto generated. Thanks for letting us know. Closing this issue now.


highfive: 2.9.0 -> 2.10.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/BlueBrain/HighFive/releases.
meta.description for highfive is: Header-only C++ HDF5 interface
meta.homepage for highfive is: https://bluebrain.github.io/HighFive/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.10.0 with grep in /nix/store/bdp0wdn5nfay7mssr3jxw76bs785rsjq-highfive-2.10.0
found 2.10.0 in filename of file in /nix/store/bdp0wdn5nfay7mssr3jxw76bs785rsjq-highfive-2.10.0

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

highfive
highfive-mpi

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/bdp0wdn5nfay7mssr3jxw76bs785rsjq-highfive-2.10.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A highfive https://github.com/r-ryantm/nixpkgs/archive/199ac90668eafe7910f38fcdfacb8b2a98446032.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/199ac90668eafe7910f38fcdfacb8b2a98446032#highfive

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/bdp0wdn5nfay7mssr3jxw76bs785rsjq-highfive-2.10.0
ls -la /nix/store/bdp0wdn5nfay7mssr3jxw76bs785rsjq-highfive-2.10.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

highfive
highfive-mpi

Maintainer pings
cc @robertodr for testing.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 330954 run on x86_64-linux 1

3 packages built:

highfive
highfive-mpi
nixpkgs-manual

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:

nixpkgs-manual:

Package is missing maintainers.
If the package is using runCommand please make sure to inherit or list one or more maintainers.


Default values should be coerced to strings for parity with env vars

variable :DURATION, :integer, default: 5.seconds

In code with default:
puts ENVied.DURATION.ago
# => Fri, 10 Feb 2017 00:52:54 UTC +00:00

With environment variable integer:
puts ENVied.DURATION.ago
# NoMethodError: undefined method `ago' for 5:Fixnum

Solution:
Coerce all default values to string and have them be parsed via the production method.

I don't want to say you're wrong so I think the example isn't very good because I don't understand why you would want to do this. When you do 5.seconds that returns an object, not a simple value like an integer. If we you run 5.seconds.class in a console you'll get an ActiveSupport::Duration object. You would have to call to_i to get the simple value that the line of code represents or in_milliseconds. Also its misleading because I don't know if your intention is to actually have a date or number of seconds? Maybe the better way to do it would be:
variable :DURATION, :integer, default: 5

And then now you can do:
puts ENVied.DURATION
# => 5
puts ENVied.DURATION.seconds
# => 5 seconds
puts ENVied.DURATION.seconds.ago
# => Fri, 10 Feb 2017 01:57:14 UTC +00:00

the example is bad

Yep. The real code I used was 10.minutes, which was just a readability thing. Of course this could apply to any value type.
10.seconds == 10
=> true

I think the better way to do it [...]

Yes, I did something dumb. Because what I did is wrong it should have raised an error because I treated it as the Ruby value (rather than the coerced string).
The point is that in production all environment variables are strings, so whatever is provided to the default should be converted to a string before it gets run through the development code (which should be as close to production as possible).

We would have to call to_i to get the simple value that the line of code represents [...]

I think they should be first converted to strings so that they mirror the production behaviour. All environment variables are strings, so they're no doubt already being coersed with to_i or equivalent.
[5] pry(main)> 10.seconds.to_s.to_i
=> 10

or call something like in_milliseconds

FYI:
[4] pry(main)> 10.seconds == 10
=> true

But this applies to all types. eg:
variable SOME_OBJECT, :string, default: { a: 5 }
# No error raised in development if dev forgets to call `JSON.parse`.

Yes I figured this is what you were getting at because recently I had to set another default and I had to do a double take as for a second I had thought, wait do I set this as a string or as the actual data type. In the case of it being a number, actually using an integer. Reviewing the README in the Defaults section does show the boolean default being set as a string. I've always done it as its native value. The next example though is an integer but using a proc which does use an integer rather than string representation. Interesting. There is definitely two approaches here because the two examples linked in the More Examples section, the first uses strings for all data types but the second does not. It might be unintentional or possibly even to work in both ways.

It looks it was considered that defaults be set as strings but then reverted. You can see commented out code that made sure defaults were set as string and to raise an error.

I think it's okay for the values to be provided as whatever literal, but I'd update that to be:
    @default = options[:default].to_s

That way the value will still be coerced as per normal, highlighting any inconsistencies between development and production config.

But the thing is this wouldn't be applicable for some objects, for example, hashes or arrays (I'm sure there are other cases too) can be given in their native Ruby format but when you specify them as a string per the examples in the README its a very different format. The to_s format doesn't map to that. I see what you are getting at and I feel it should be addressed but I would be more for consistency so I would rather have the defaults be only defined as strings so there is no confusion or encountering unexpected outcomes.

@javierjulio the restriction that defaults should always be strings was removed when default-procs were introduced.
@rhys-vdw It's recommended to always denote defaults as strings - this is in the end what you should provide via the ENV.
Looking at your usecase it might be nice to have the ability to define your own types in the Envfile:
register_type(:duration) do |raw_string|
  _all, count, unit = */(\d+) (\w+)/.match(s)
  Integer(count).public_send(unit)
end

# as we're always processing strings less boilerplate variant might be:
register_type(:duration, /(\d+) (\w+)/){|raw, count, unit| Integer(count).public_send(unit) }

# usage:
variable :session_timeout, :duration, default: '5 minutes'

This way the variable is closer to the domain (ie it's always used as AS::Duration), while coming up with a way to encode this in a string.

Thanks @eval! 🙏🏻 I was going to ask you about this today. 😄 As you can see we were confused as to the direction you were going in. What I'm thinking of doing is updating all examples to use strings for default values. Curious, why did you remove the restriction for default-procs? Couldn't we have required then that the proc just returned a string too? I'm going to make a new issue listing what I plan to do for a 1.0. I'd love your input @eval and @rhys-vdw as I want to do more but thinking that should come after that 1.0 release.

Looking at your usecase it might be nice to have the ability to define your own types in the Envfile

Quite possibly.  Turns out the integer monkeypatch that adds .minutes, .seconds etc is not available while ENVied is initializing anyway (before Rails has started).
The approach we ended up taking was this:
variable :session_timeout_in_seconds, :integer, default: (5 * 60).to_s

Not as fancy, but it's unambiguous. I can see the appeal for custom types though. 👍


minor edits

There was an error while validating job 2c9c7690-a488-400b-b3c0-1c9ce712d586 Message:Error clonning repositoryDetails:System.ArgumentException: Error clonning repository ---> System.Exception: Error when trying to initialize shared repository ---> LibGit2Sharp.NotFoundException: Branch 494 not found. Available branches:
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.GetBranch(Repository repo, String name)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.<>c__DisplayClass3e.b__3d()
at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.<>c__DisplayClass1.b__0()
at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.ExecuteAction[TResult](Func1 func)    at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.ExecuteAction(Action action)    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.InitializeSharedRepository(String url, IDictionary2 branches, String sharedRepoPath)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.VerifySharedRepository(String url, IDictionary2 branches, String tempPath)    --- End of inner exception stack trace ---    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.VerifySharedRepository(String url, IDictionary2 branches, String tempPath)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository(String url, IDictionary2 branches)    --- End of inner exception stack trace ---    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository(String url, IDictionary2 branches)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository()
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.GetTreeAsync(String path, Boolean includeSubDirectories, String excludePath, IList1 extensionWhiteList)    at Microsoft.DPS.Commands.IdentifyDocumentsToPublish.ListSourceControlFilesCommand.Do(IDictionary2 context)
at Microsoft.LightweightJobEngine.Core.Language.JobWorkflow1.RunCommands(Guid jobId, IDictionary2 context, CancellationTokenSource cancelSignal)