Add className to general observer div

Add a better way to target the wrapping div with CSS.
I'm currently using the data-testid to target the wrapping div, but that's cumbersome to target with CSS and that really only should be used for testing.

Hey @NerdCowboy Thanks, Happy to merge this in. However, we're not planning to release any package updates until after all our tests have been completed as part of Hactoberfest.
I'd hope a stable release of v<IP_ADDRESS> will be out at the start of Nov. Hope its cool to wait for a little while longer for this change.


Portuguese Translations

Get Involved!
We are seeking Portuguese translators for this project. If you would like to get involved with translating a lesson. Please folk the project and start translating.

Set up

[x] _config.yml
[x] index.md

Basics

[x] Basics
[x] Collections
[x] Enum
[x] Pattern Matching
[x] Control Structures
[x] Pipe Operator
[x] Functions
[x] Composition
[x] Mix
[x] Sigils
[x] Documentation
[x] Testing
[ ] Custom Mix Tasks

Advanced

[x] concurrency.md
[x] errors.md
[x] erlang.md
[x] escripts.md
[x] otp-concurrency.md
[x] otp-supervisors.md
[x] metaprogramming.md

Specifics

[x]  ecto.md
[x]  eex.md
[x]  ets.md
[x]  mnesia.md
[x]  plug.md

Note
Those new to Elixir and / or Github are encouraged to get involved :smile:. 1 Pull Request per lesson.

@pragmaticivan Do you still find this particular Issue format helpful? I'm looking for ways to handle PRs like #452, any thoughts would be greatly appreciated.  I'd like to refrain from using a third-party service if possible, those seem to bring about their own unique headaches

Not really for new changes. Can't see a better way unless we have inside the wiki explaining that if someone create a new change they chould report the issue for all the languages available.

Can I make "Custom Mix Tasks" translation? :)

Can I make Custom Mix Tasks translation? :)

I did it on PR #725

Hey guys. I'm going to translate the IEx Helpers.

Hey guys, we have some page to translate?

Can I make Debugging translation?

Hey @rrmartins if you check this link: https://github.com/doomspork/elixir-school/issues?q=is%3Aissue+is%3Aopen+label%3Atranslation
There are many variations of the current files, maybe you can help to keep all the content updated.

Hey @pragmaticivan, ok,  I saw that we do not have a translation for Debugging page in Portuguese. Can you do this translation? :)

Sure you can @rrmartins, go for! :D Thanks!

List updated :)

Hey, i have done the translation of the lesson of types and specifications.

Thanks @vicdda I will start reviewing soon.

I'll take GenState.

Wonderful, thank you @guilhermelazzari!!

Hi @doomspork I'll translate Behaviours too.

I have checked the missing lessons and it is all complete, closing this ISSUE in favour of https://github.com/elixirschool/elixirschool/issues/1295


Push docker image from Amazon Elastic Container Registry (ECR)

I'm having an issue deploying docker containers to CF from Amazon Elastic Container Registry (ECR).
The way that ECR authentication works, if you want a username/password you need to do a separate aws cli call using your access_key / secret to get a short-lived username/password (see AWS Docs).
Now I could do this in a task, but there is no way that I know of to pass that password into the docker_password param of the following cf-resource put.  I'm beginning to think that it isn't possible to push from ECR to CF using the cf-resource, hence I'm raising this issue.
I think, ideally, the cf-resource would be enhanced to offer specific support for ECR as a source, taking AWS key/secret as params, but I accept that this is a bit of work and perhaps not strictly a concern of this resource.
Failing that, I would be able to work around the issue if it were possible to specify a docker_password_file param, pointing to a file containing the password that I have populated in an earlier task.

For reference, the https://github.com/concourse/docker-image-resource takes aws_access_key_id/aws_secret_access_key params and can push/pull from ECR

Looking at the instructions for deploying to CF from Google Container Registry (GCR) https://docs.cloudfoundry.org/devguide/deploy-apps/push-docker.html I think the same issue applies.


Add new SANBI and WCS AMR courses

SANBI course website will be done by end-March 2024, WCS site is already live

Thank you @pvanheus !


Helm operator install fails

Bug description
The following instruction from the docs fails:
➜  istio git:(master) helm template install/kubernetes/operator/operator-chart/ \
  --set hub=docker.io/istio \
  --set tag=1.5.1 \
  --set operatorNamespace=istio-operator \
  --set istioNamespace=istio-system | pbcopy
Error: failed to download "install/kubernetes/operator/operator-chart/" (hint: running `helm repo update` may help)

so tried making it into a local path:
➜  istio git:(master) helm template ./install/kubernetes/operator/operator-chart/ \
  --set hub=docker.io/istio \
  --set tag=1.5.1 \
  --set operatorNamespace=istio-operator \
  --set istioNamespace=istio-system | pbcopy
Error: path "./install/kubernetes/operator/operator-chart/" not found

I looked, and the install folder is missing the kubernetes folder.
Expected behavior
Command to work
Steps to reproduce the bug
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
kubectl:
Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.0", GitCommit:"70132b0f130acc0bed193d9ba59dd186f0e634cf", GitTreeState:"clean", BuildDate:"2019-12-13T11:52:32Z", GoVersion:"go1.13.4", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"15+", GitVersion:"v1.15.9-gke.24", GitCommit:"39e41a8d6b7221b901a95d3af358dea6994b4a40", GitTreeState:"clean", BuildDate:"2020-02-29T01:24:35Z", GoVersion:"go1.12.12b4", Compiler:"gc", Platform:"linux/amd64"}
helm:
version.BuildInfo{Version:"v3.0.3", GitCommit:"ac925eb7279f4a6955df663a0128044a8a6b7593", GitTreeState:"clean", GoVersion:"go1.13.7"}
How was Istio installed?
Not
Environment where bug was observed (cloud vendor, OS, etc)
OSX Catalina

It's probably a docs only issue and helm operator install is discontinued? Then please update docs to show that?

Nope. Even in the 1.5.1 release the chart is missing. The install/kubernetes folder exists but no operator in there ;|

Nevermin, I cloned the repo instead of downloading the release. Closing this


fix: resource files are too big

Issue: #13 - Resources files being generated are too big. For instance, an Aseprite file of 6kb was creating a 24mb resource.
Root cause:
Each animation frame uses an AtlasTexture with the sprite sheet as source. As the spritesheet is generated outside Godot's editor, it doesn't trigger a scan to import it as a resource, and the workaround was to use an ImageTexture and load the file from the disk.
This texture was supposed to store a reference to the spritesheet, but it seems it is actually embedding the image together, and also duplicating this image for each frame.
The solution
The main solution programmatically triggers Godot's File System Scan. This is done by using EditorFileSystem.scan().
As this scan is async, to avoid freezing the editor, I had to use a coroutine to wait for the signal:
yield(file_system, "filesystem_changed").
The only issue is that to wait for a signal this way, all the parent functions need to return coroutines as well.
That's Ok for the wizard screen, but the import plugin needs to be sync and return an exit code. So this solution can't be used there. The workaround for the importer is to keep using the ImageTexture import, but moving it up, doing only once, and not for every frame.
The caveat in this fallback is that the generated file is still bigger than it should be. For comparison, using the main method my test file was 6.5kb, while the fallback method generated a 403kb file.

[x] tested on Linux
[x] tested on Windows
[x] tested on MacOS
[x] tested with more files

Hey!
I just tested your PR on Arch Linux with Godot-Mono v3.2.2.
It works great! It managed to drop the .res size from 180mb to only 1.5mb.
Still larger than the original (39,8kb) but the improvements are completely awesome!
I'll keep testing with other files and bring more feedback.
Thanks.

Hey!
I just tested your PR on Arch Linux with Godot-Mono v3.2.2.
It works great! It managed to drop the .res size from 180mb to only 1.5mb.
Still larger than the original (39,8kb) but the improvements are completely awesome!
I'll keep testing with other files and bring more feedback.
Thanks.

Awesome! Yeah, 1.5mb still high.
Could you do a test for me? Can you try removing or commenting line 352 atlas.atlas = image in aseprite_cmd.gd, importing again and telling me the file size? The images won't be imported, but I'm curious how much of this size comes from the images and how much is just normal resource stuff (SpriteFrames, AtlasTexture, etc).

Awesome! Yeah, 1.5mb still high.
Could you do a test for me? Can you try removing or commenting line 352 atlas.atlas = image in aseprite_cmd.gd, importing again and telling me the file size? The images won't be imported, but I'm curious how much of this size comes from the images and how much is just normal resource stuff (SpriteFrames, AtlasTexture, etc).

Could you do a test for me? Can you try removing or commenting line 352 atlas.atlas = image in aseprite_cmd.gd, importing again and telling me the file size? The images won't be imported, but I'm curious how much of this size comes from the images and how much is just normal resource stuff (SpriteFrames, AtlasTexture, etc).

Hello,
I tested commenting this line and my imported .res file went from 1.4mb to just 11,4kb. 3 times less than the original .aseprite file (39,8kb).
The whole .import folder (.json, + .png + .res) now weights 75,5kb.
Uncommenting the line and reimporting, I can see that only the .res file is affected.
I'm using the latest commit and everything else works perfectly!

Could you do a test for me? Can you try removing or commenting line 352 atlas.atlas = image in aseprite_cmd.gd, importing again and telling me the file size? The images won't be imported, but I'm curious how much of this size comes from the images and how much is just normal resource stuff (SpriteFrames, AtlasTexture, etc).

Hello,
I tested commenting this line and my imported .res file went from 1.4mb to just 11,4kb. 3 times less than the original .aseprite file (39,8kb).
The whole .import folder (.json, + .png + .res) now weights 75,5kb.
Uncommenting the line and reimporting, I can see that only the .res file is affected.
I'm using the latest commit and everything else works perfectly!

Wow! That's some mad debugging skills. Nice work. That's very useful.

I was reading the issue again and just now I understood that your intention is to reference the image file and not embed it, right?

Yeah, I believe this is what Godot do if we create the sprite frame through the interface (I'm not sure though, I need to validate that), but I wouldn't mind embedding the image, as long it's done only once.
I wanted to know the file size without the images just to confirm the issue was still about how the images are inserted, and not some other resource.
I'm not sure it is an issue in Godot, though. Maybe it was just the way it was designed to work, and our use case is kind of atypical.

Wow! That's some mad debugging skills. Nice work. That's very useful.

I was reading the issue again and just now I understood that your intention is to reference the image file and not embed it, right?

Yeah, I believe this is what Godot do if we create the sprite frame through the interface (I'm not sure though, I need to validate that), but I wouldn't mind embedding the image, as long it's done only once.
I wanted to know the file size without the images just to confirm the issue was still about how the images are inserted, and not some other resource.
I'm not sure it is an issue in Godot, though. Maybe it was just the way it was designed to work, and our use case is kind of atypical.

Ok! So, I've done some testing using the same strategy you did.
I created one file, 1px by 1px with 2 animations, one frame each. I've just used a different color in each frame, to make sure they were not duplicated.
Then I imported it twice. One time using the wizard and one using the importer.
png image: 87 bytes
Wizard result file: 889 bytes
importer result file: 1.1k
The wizard resource is doing everything right. I used hexdump and I was able to see this import:

I looked for the color as you did (great idea, by the way) and I couldn't find them, meaning, the file is not embedded.
The importer result does include the file, as expected. I was able to find the colors in the hex code. It only includes it once, which seems about right.
I've also looked into some of the assets I imported in my game. I usually use the wizard flow, so all of them have the file as reference, which is great, and they are usually less than 10kb, depending on the number of frames and animations.
Overall it seems to be working as expected. I'm not sure why you are getting 1.5mb. As your spritesheet png is less than 100kb, I would expect it to be at most two times that. It may be an edge case with that file, given before it was 180mb, which is crazy hehe.
Anyway, embedding the file in the importer is expected, and if it's doing only once, I'm Ok with that. Until we find a way to use ResourceLoader in the importer as well.

Ok! So, I've done some testing using the same strategy you did.
I created one file, 1px by 1px with 2 animations, one frame each. I've just used a different color in each frame, to make sure they were not duplicated.
Then I imported it twice. One time using the wizard and one using the importer.
png image: 87 bytes
Wizard result file: 889 bytes
importer result file: 1.1k
The wizard resource is doing everything right. I used hexdump and I was able to see this import:

I looked for the color as you did (great idea, by the way) and I couldn't find them, meaning, the file is not embedded.
The importer result does include the file, as expected. I was able to find the colors in the hex code. It only includes it once, which seems about right.
I've also looked into some of the assets I imported in my game. I usually use the wizard flow, so all of them have the file as reference, which is great, and they are usually less than 10kb, depending on the number of frames and animations.
Overall it seems to be working as expected. I'm not sure why you are getting 1.5mb. As your spritesheet png is less than 100kb, I would expect it to be at most two times that. It may be an edge case with that file, given before it was 180mb, which is crazy hehe.
Anyway, embedding the file in the importer is expected, and if it's doing only once, I'm Ok with that. Until we find a way to use ResourceLoader in the importer as well.

Hey!
I did some changes in that file, reduced some frames and now everything is working flawlessly!
Using the importer brings the file down to 700kb and using the Wizard, it weights just 7.2kb.
I guess it is an edge case with my file, like you said. In any case, this is completely fine and I'm super happy about it.
Thank you for your support! Awesome tool. :)

Hey!
I did some changes in that file, reduced some frames and now everything is working flawlessly!
Using the importer brings the file down to 700kb and using the Wizard, it weights just 7.2kb.
I guess it is an edge case with my file, like you said. In any case, this is completely fine and I'm super happy about it.
Thank you for your support! Awesome tool. :)


Add a timestamp into the output requirements file header

What's the problem this feature will solve?
Running the pip-compile requirements.in command produces a requirements.txt file containing the pinned requirements with a useful header
#
# This file is autogenerated by pip-compile with python 3.6
# To update, run:
#
#    pip-compile requirements.in
#

which is constructed here in the source code.
The header does not tell when the file was generated, i.e. when the resolver actually worked through the input requirements searching for the packages and producing the output. I believe having this information would be useful. I do understand that the produced requirements file is normally checked in into a version controlled repository so one can see the commit history, but the commit time is not necessarily the same time when the input requirements were resolved.

Describe the solution you'd like
I would suggest extending the current header by adding the timestamp
f"{datetime.now().strftime('%Y-%m-%d %H:%M:%S')}"
so that the header becomes
#
# This file is autogenerated by pip-compile with python 3.6 on 2021-06-30 09:54:43
# To update, run:
#
#    pip-compile requirements.in
#

I am happy for the timestamp to be placed in an arbitrary location within the header or elsewhere within the file as long as it is present and it can be formatted in any way.

Alternative Solutions

Wrapping the pip-compile call in a shell script that prepends the file with the $ date -R command output does the trick, but it would be convenient to have pip-compile put the timestamp into the header by default.
Additional context
I am happy to work on a PR for this if the team agrees that this is something we'd like to have. Many thanks.

Hello @alexey-tereshenkov-oxb,
Thanks for the issue! That wouldn't work well with pre-commit hook for pip-compile - the check would never be green. This would force pre-commit users to use --no-header option.

The only way to sort this and other similar requests related to the header is to allow custom headers, like a string pip-compile --header=foo.
Closing this as wontfix, for reasons mentioned by @atugushev

Thanks @ssbarnea , excellent idea with custom headers.


3.0 release?

Ecto 3.0 was released which references this project as the upgrade path for database users. I see that a release candidate was published a few days ago. Are there any issues preventing the release candidate getting promoted to stable?

Unfortunately it is blocked on mariaex, so we are waiting for it. It should be out soon.


Empty email field for people reads as "none" on the front end.

Sometimes, it does not make sense to include an email address for an individual on their Wagtail "people" page. This is generally because the individual is no longer affiliated with Princeton so the Princeton email likely no longer works. However, when the email field is left blank, the word "None" appears on their CDH profile on the front end in the place where the email address would usually appear.
To reproduce

Go to Wagtail, then "people," then "people" again.
Select  an individual who currently has an email listed on the front end.
Delete the individual's email address.
View their profile on the front end. You will see the word "none" where the email should be. (You can see this currently on my profile.)

Expected behavior
I would have expected that the area on the profile where the email would usually appear would simply be left blank.

I deleted my email on the test site--didn't see the word "none." Looks good!


[WIP] add pathfinding support

The code is not currently working but let's merge this for the future team. I don't recommand to use it anyway. @antoinevdm


Add command to validate client input and output data

List of JSON object to validate:

prediction JSON object for each task
gold standard JSON object for each task

See how to do this in Python:
https://github.com/Sage-Bionetworks/nlp-sandbox-schemas/blob/develop/tools/validate/validate.py

Depends on https://github.com/Sage-Bionetworks/nlp-sandbox-schemas/issues/38.
Marking this issue as Blocked until the above task is completed.

By using the client to get the annotation stores (goldstandard) and annotation results from the annotators (prediction), we may not need this.  Thoughts @tschaffter ?


First Pass at adding address to logs  #2921

Motivation

Solution

I am getting this error when I try to run
cargo install --path ./cli --profile local --bins --locked --force

@onbjerg Hey, I have made some changes but not exactly sure how I should be testing this. I understand that cheatcodes provide a way to test the way blockchain is changed. But for my specific case, I am unsure about how I should test it. I think it might be something simple but I am really stuck on this.

Closing as stale.


Base64 icon doesn't work in FireFox (54.0)

See this comment: https://github.com/mapbox/leaflet-image/commit/d06a02a7ef7691d8d9fbac61b36b6f96623e5526#commitcomment-22683002
I think the onload function is being called too early and before the job is done.

@I have a same problem with FireFox and Edge.
Do you find a solution?

@stefo91 I just removed the last line in handleMarkerLayer (https://github.com/mapbox/leaflet-image/blob/v0.4.0/leaflet-image.js#L230) and it works fine for my purpose.

Hello
Kaveh,
Thanks for the reply.
You helped me a lot.
Now it works perfectly in Chrome, Firefox and Edge.
If you come to Bulgaria, call me to drink a beer or a coffee :)
Best Regards,
Stefan
<EMAIL_ADDRESS>
-------- Оригинално писмо --------

От: Kaveh<EMAIL_ADDRESS>

Относно: Re: [mapbox/leaflet-image] Base64 icon doesn't work in FireFox (54.0)
(#99)

До: mapbox/leaflet-image

Изпратено на: 01.06.2018 20:03

@stefo91  I just removed the last line in  handleMarkerLayer  ( https://github.com/mapbox/leaflet-image/blob/v0.4.0/leaflet-image.js#L230 ) and it works fine for my purpose.
—
You are receiving this because you were mentioned.
Reply to this email directly,  view it on GitHub , or  mute the thread .

Almost 3 years later and I run into this exact problem. Chrome OK, Firefox not. Unfortunately removing if (isBase64) im.onload(); does not help. Any suggestions?


HDXDSYS-683 Remove any timezone information in iso formatted data

Fixes https://github.com/OCHA-DAP/hdx-python-api/issues/66

coverage: 97.546%. remained the same
when pulling 5ece80026b4b4c0338b8ce79adb8d014c15fe952 on last_modified_tz_fix
into e4cd53ab9193b2276ab4c950d4108b4891a63b20 on main.


coinstac.org chrome issue "Places using coinstac"

On chrome its still showing the issue.

I use Chrome. Shift+Command+R
On Mon, Aug 22, 2022 at 2:13 PM spanta28 @.***> wrote:

Assigned #1439 https://github.com/trendscenter/coinstac/issues/1439 to
@pixelsaurus https://github.com/pixelsaurus.
—
Reply to this email directly, view it on GitHub
https://github.com/trendscenter/coinstac/issues/1439#event-7235962172,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFCKAUXUDA64WVPN7COBRDV2PNN5ANCNFSM57I6NGWQ
.
You are receiving this because you were assigned.Message ID:
@.***>

Great. I forget the hard refresh sometimes!


Many to Many Cascade Lookup Editor

I have a table and for one of the fields I want to have a lookup editor but on new, I want this to be filtered by another table.  This table has a many to many relationship with the field in my table, so if I choose A for the first filter, I get 1,2,3 as options for the second and if I choose B I get 2,3,4.  I have this set up and it works fine on new.
For existing values, both filters are blank because it cannot assign the first filter on load since if the field value is 2, it doesn't know if the first filter value should be A or B.  Is there any way to fix this?  I was hoping for the lookup editor to only apply when the first filter has a value selected.

Did you checked this document?
And if you declare correct way it must be set when you edit;
   [LookupEditor(typeof(Entities.CityRow),
        CascadeFrom = "CountryId", CascadeField = "CountryId")]
    public Int32? CityId { get; set; }

    [LookupEditor(typeof(Entities.DistrictRow),
        CascadeFrom = "CityId", CascadeField = "CityId")]
    public Int32? DistrictId { get; set; }

Did you check whether the value selects (your properties has value) when you edit?
if value selects correctly on form, then you can try to select in typescript code.

That example and code are for cascading lookups that assume one to many relationships.  I need one for many to many.

hope help
First: Create a new lookupEditor
namespace XYZ.Default {
    @Serenity.Decorators.registerEditor()
    export class MannyLookupEditor extends Serenity.LookupEditorBase<ManyLookupOptions, abcRow> {
        constructor(container: JQuery, options: ManyLookupOptions) {
            super(container, options);
        }

        protected getLookupKey() {
            return "lookupkey";
        }

        protected getItems(lookup: Q.Lookup<QuotesRow>) {
            var items = this.filterItems(this.cascadeItems(lookup.items));
            if (items.length > 0)
                if (this.Id1 != null)
                    items = items.filter(x => x.Id1 == this.filterId1);

            if (items.length > 0)
                if (this.filterId2 != null)
                items = items.filter(x => x.Id2 == this.filterId2 );
            return items;
        }

        protected get_Id1(): any {
            return this.options.Id1;
        }

        get filterId1(): any {
            return this.get_Id1();
        }

        protected set_Id1(value: any) {
            if (this.options.Id1 !== value) {
                this.options.Id1 = value;
                this.set_value(null);
                this.updateItems();
            }
        }

        set Id1(value: any) {
            this.set_Id1(value);
        }

        protected get_Id2(): any {
            return this.options.Id2;
        }

        get filterId2(): any {
            return this.get_Id2();
        }

        protected set_Id2(value: any) {
            if (this.options.Id2 !== value) {
                this.options.Id2 = value;
                this.set_value(null);
                this.updateItems();
            }
        }

        set filterId2(value: any) {
            this.set_Id2(value);
        }
    }

    export interface ManyLookupOptions extends Serenity.LookupEditorOptions {
        Id1: number;
        Id2: number;
    }
}

Then transform T4
In abcForm.cs
[MannyLookupEditor]
public Int32 AnyId { get; set; }

Then in abcDialog.ts
this.form.zxyId.changeSelect2((e: JQueryEventObject): void => {
                this.form.AnyId.cascadeField = "CascadeFieldId";
                this.form.AnyId.cascadeValue = this.form.CascadeFieldId.value;

               this.form.AnyId.filterField = "FilterId";
                this.form.AnyId.filterValue = this.form.FilterId.value;

                this.form.AnyId.filterId1 = this.form.Id1.value;
                this.form.AnyId.filterId2 = this.form.Id2.value;
            });

Thank you luongnv6


Provide new commands mount/umount

Some people expressed desire to have capability to be able to quickly inspect an image. Right now one can save image content to tar file and untar it later and look into image. But taring the content creates additional copy which is slow.
Another method seems to be to create a container and then run a command inside that container. But that requires command to be present inside an image at the same time running container of image being inspected is not very clean. People argue that one does not want to run the container which is
being inspected.
So it was suggested how about implementing a new set of commands say mount/umount which just mount image read only at a given directory. That will allow host to quickly inspect image without creating copy and then unmount the image.
I am about to post a very basic patch to be able to do that. That can serve as place for discussion on this topic and how to go about solving this issue.

Hi!
If you are reporting a new issue, PLEASE make sure that does not have any duplicates already open.
We would like to take this time to remind you of the information we need to debug the problem you are seeing. This is an automated response so if this ticket is not about a bug, do not fret.
If you fail to provide this information within 7 days, we will close this because we cannot debug your issue. We can reopen whenever the information is provided.
Thank you.
Please see:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues
Description of problem:

`docker version`:

`docker info`:

`uname -a`:

Environment details (AWS, VirtualBox, physical, etc.):

How reproducible:

Steps to Reproduce:
1.
2.
3.

Actual Results:

Expected Results:

Additional info:

#ENEEDMOREINFO

@rhvgoyal I guess gordon wouldn't complain if you start title like Proposal:

Closing since this was rejected some time ago here: #13130

Shame since it seems like something could be useful for debugging purposes.

Maybe we can open a discussion around debug tooling?

I think that requirement can be implemented as --mount type=image (could not find the issue number.. IIUC @justincormack is proposing this 😅 ) rather than new commands

Yes, see this issue https://github.com/docker/docker/issues/30449


Sweep: Add inline comments for each value in application.yml

Details
Add inline comments documenting each property in the src/main/resources/application.yml yaml file.  The comments should be on the same line, after the value.  This is a Spring Boot application.

Checklist

[X] Modify src/main/resources/application.yml ✓ https://github.com/devondragon/SpringUserFramework/commit/7cf57e625ade195dcf0ce8568e2f3a4732db8f0a

No good.


Purchasing document variables as protected var in Navigate Page

Describe the request
Can we have the variables PurchRcptHeader, PurchInvHeader,  ReturnShptHeader and PurchCrMemoHeader as Protected Var because of an event obsoletion in the Navigate page?
page 344 Navigate
        ServInvHeader: Record Microsoft.Service.History."Service Invoice Header";
        [SecurityFiltering(SecurityFilter::Filtered)]
        ServCrMemoHeader: Record Microsoft.Service.History."Service Cr.Memo Header";
#endif
        [SecurityFiltering(SecurityFilter::Filtered)]
        IssuedReminderHeader: Record "Issued Reminder Header";
        [SecurityFiltering(SecurityFilter::Filtered)]
        IssuedFinChrgMemoHeader: Record "Issued Fin. Charge Memo Header";
// >>>>>>>>
// <<<<<<<<
        [SecurityFiltering(SecurityFilter::Filtered)]
        ProductionOrderHeader: Record "Production Order";
        [SecurityFiltering(SecurityFilter::Filtered)]
        PostedAssemblyHeader: Record "Posted Assembly Header";
        [SecurityFiltering(SecurityFilter::Filtered)]
        TransShptHeader: Record "Transfer Shipment Header";

    protected var
        [SecurityFiltering(SecurityFilter::Filtered)]
        SalesShptHeader: Record "Sales Shipment Header";
        [SecurityFiltering(SecurityFilter::Filtered)]
        SalesInvHeader: Record "Sales Invoice Header";
        [SecurityFiltering(SecurityFilter::Filtered)]
        ReturnRcptHeader: Record "Return Receipt Header";
        [SecurityFiltering(SecurityFilter::Filtered)]
        SalesCrMemoHeader: Record "Sales Cr.Memo Header";
        [SecurityFiltering(SecurityFilter::Filtered)]
        SQSalesHeader: Record "Sales Header";
        [SecurityFiltering(SecurityFilter::Filtered)]
        SOSalesHeader: Record "Sales Header";
        [SecurityFiltering(SecurityFilter::Filtered)]
        SISalesHeader: Record "Sales Header";
        [SecurityFiltering(SecurityFilter::Filtered)]
        SROSalesHeader: Record "Sales Header";
        [SecurityFiltering(SecurityFilter::Filtered)]
        SCMSalesHeader: Record "Sales Header";
        [SecurityFiltering(SecurityFilter::Filtered)]
        PQPurchaseHeader: Record "Purchase Header";
        [SecurityFiltering(SecurityFilter::Filtered)]
        POPurchaseHeader: Record "Purchase Header";
        [SecurityFiltering(SecurityFilter::Filtered)]
        PIPurchaseHeader: Record "Purchase Header";
// >>>>>>>>
        [SecurityFiltering(SecurityFilter::Filtered)]
        PurchRcptHeader: Record "Purch. Rcpt. Header";
        [SecurityFiltering(SecurityFilter::Filtered)]
        PurchInvHeader: Record "Purch. Inv. Header";
        [SecurityFiltering(SecurityFilter::Filtered)]
        ReturnShptHeader: Record "Return Shipment Header";
        [SecurityFiltering(SecurityFilter::Filtered)]
        PurchCrMemoHeader: Record "Purch. Cr. Memo Hdr.";
// <<<<<<<<
        [SecurityFiltering(SecurityFilter::Filtered)]
        GenJnlLine: Record "Gen. Journal Line";
#if not CLEAN25
        [Obsolete('Moved to codeunit Serv. Navigate Mgt.', '25.0')]
        [SecurityFiltering(SecurityFilter::Filtered)]

It would generally make sense to provide all record variables as protected vars.
Additional context
The OnBeforeNavigateShowRecords event has been obsoleted. The required sales document variables can be accessed via a page extension. Unfortunately, this is not possible for posted purchasing, as these variables are not listed as protected variables.
Internal work item: AB#561118

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 28358.


Prefix container object ems_ref with class name

Use the class name as a prefix for ems_ref removing a collision with the
real ems_ref column
Depends on https://github.com/ManageIQ/manageiq-providers-kubernetes/pull/123
Fixes https://github.com/ManageIQ/manageiq/issues/16074

cc @AndreyMenezes

LGTM

Sounds good (both PRs).
Are there transition concerns, eg. processing old events in DB that had :ems_ref, or vice versa?

Looks good, I had a question on the other PR on where we are passing an ems_ref.
(we are doing something similar for prometheus events[1] and maybe I can use that)
[1] https://github.com/ManageIQ/manageiq/pull/14301/files

@cben the only transition issue I can see is in-flight events in the queue when we upgrade.
An alternative would be to only run that method if the source is KUBERNETES so we don't overwrite other providers' events with an ems_ref?

The Kubernetes PR is ready. @agrare please let us know when we can merge both

@moolitayer these are both ready

@blomquisg the dependent PR in kubernetes has been merged, can you take a look?

@blomquisg ping, currently kubernetes event collection is broken on master afaik until well merge this one.


Debug vs Console

I could be wrong but I think C# does not have a Debug.WriteLine() command.
At least in my experience it's:
Console.WriteLine("what is going on!");

Debug.WriteLine() is from System.Diagnostics.
Its not commonly used because it requires an extra using statement but it's a legit thing.


Add funding to the readme and add funding file

For the FUNDING.yml file, I'm not 100% sure if we can link to the specific subpage, or will this work only with
open_collective: thewpcc

As a general question: should there be a "This project needs funding" banner or something near the top of the README for visibility ?

Q: Should the section be added to the table of contents?

I don't think so, not sure if it's relevant for some SEO purposes, but it's at the top of the readme, so it's pretty visible, no?

@GaryJones Could you have another look (as the PR has changed significantly) ? If okay, please feel free to merge.


pod search flix

not support cocoapods?

Please run pod repo update before search Flix.
https://cocoapods.org/?q=flix

yes , I did, But still can't find it...  pod version is 1.2.1 , gem source is https://ruby.taobao.org/

Please check you have add https://github.com/CocoaPods/Specs.git.
https://github.com/CocoaPods/Specs/commit/d50f88652f88044703b28c3e145133f84c6ab888


DH Code review

This project was soft-launched in the spring; production site is accessible at https://lenapetimetree.indigenous.princeton.edu/
The readme includes instructions on setting it up to run locally, if desired or helpful for review. Functionality included in the 1.0 release is documented in the change log.
This PR includes all of the JavaScript code (and only that). It's probably more than should be reviewed. Unit tests were added to tests specific parts of the code where it was helpful for development; my goal in this project was not complete coverage.
The code uses a d3.js force layout with custom forces to generate a tree structure with data points grouped into five ordered branches and roughly ordered by date. Interaction logic for selecting leaves in the tree, fetching and displaying contents, selecting tags and highlighting leaves is implemented with d3 and JavaScript (no other framework).  The main entry point for the code is timetree.js.
This is written with d3.js v7.8.4; javascript dependencies are installed with npm and imported into local js code; javascript is compiled and embedded using Hugo pipelines.
The JSON data used for the timetree visualization is embedded in the page, but the structure and content is viewable at this url: https://lenapetimetree.indigenous.princeton.edu/index.json
If you have questions about the Hugo template logic used to generate the JSON, or any other parts of the application that aren't included in this review, please let me know and I can point you to the relevant parts of the code.
Feedback I'm interested in at this point:

suggestions for better JS code organization; the project grew as I was working on it and is very interconnected; I tried to make more modular but not sure how successfully
suggestions for making it more efficient, especially when first loading the data and laying out the tree
ideas for adjusting the layout so leaves will be positioned closer to y-coordinate for their date
any other recommendations for improvements, e.g. to make it more efficient, more idiomatic JavaScript, better documented; notes on what else should be tested
recommendations for how to split out code/logic from content (everything is currently in one repository); should it just be a hugo theme + hugo site? Would it make any sense (and have any value) for the timetree js code to be a separate javascript library?

@ColeDCrawford here are the more d3-specific parts of the code that I think you should focus on. All of these are
in the timetree.js file: generateNetwork, drawTimeTree; methods for custom forces on the network: leafBounds, centuryY, branchX; updatePositions. If time allows, you could also review the zoom logic, particularly zoomToDatum, zoomToSelectedLeaf, zoomToTagged

I took a look on the overall structure of the app, and focused more carefully on panel.js and the Leaf class in leaves.js. I also read through the code for the d3 viz (particularly timetree.js), which seems clear enough to me, but I didn't spend as much time on it.
I think the overall structure is clear, and likely keeps the JS to a bare minimum (which can be desirable!).  Though, and I think I suggested this during the initial call, if this application where to be generalized to be re-used with other data in other projects, it may be useful looking at a framework to facilitate certain operations. For example, to better handle events (e.g. in panel.js and leaf.js, but also in the class in keys.js, a sort of App-level component), or even to avoid event handling in favor of state management. panel.js and the Leaf class in leaves.js could both be though of as "components" and could be expressed as reusable web components, or framework-specific components (React, Vue...). The need for fetching, parsing, and embedding HTML data in panel.js is a good indication that you probably want to use component composition. See the review comments for more details.
A less important comment: I find the navigation at the root of the tree hard to identify as... navigation. The terms seemed like nodes relevant to the visualization and it took me a while to realize they're clickable. I would make that more apparent, e.g. with a mouseover highlight, or just by making them more typically recognizable as a navbar.


Fix Windows Linking error

Fix https://github.com/tensorflow/tensorflow/issues/12117
stringpiece.cc is only the src of lib_internal, so we have to add it as a dependency.
@gunan @learyg

Testing at http://ci.tensorflow.org/job/tensorflow-pr-win-bazel/35/console

Jenkins, test this please.

the failure is due to sanity checks failing.
Here is the log:
Running do_buildifier on 213 files

tensorflow/tools/proto_text/BUILD # listsort unsafesort sort:cc_binary.deps

buildifier took 0 s

FAIL: buildifier found errors and/or warnings in above BUILD files.
buildifier suggested the following changes:
37d36
<         "//tensorflow/core:lib_proto_parsing",
38a38
>         "//tensorflow/core:lib_proto_parsing",
Please fix manually or run buildifier <file> to auto-fix.

@meteorcloudy  You may remove core:lib_proto_parsing
--- a/tensorflow/tools/proto_text/BUILD
+++ b/tensorflow/tools/proto_text/BUILD
@@ -34,7 +34,7 @@ cc_binary(
	 visibility = ["//tensorflow:internal"],
	 deps = [
		 ":gen_proto_text_functions_lib",
-        "//tensorflow/core:lib_proto_parsing",
+        "//tensorflow/core:lib_internal",
	 ],
 )

@@ -60,7 +60,7 @@ cc_library(
		 ],
	 }),
	 deps = [
-        "//tensorflow/core:lib_proto_parsing",
+        "//tensorflow/core:lib_internal",
	 ],
 )

Looks like the build is fixed!
The test failures at http://ci.tensorflow.org/job/tensorflow-pr-win-bazel/35/console is caused by a bug in Bazel.
Already filed https://github.com/bazelbuild/bazel/issues/3531, I'll make sure the fix cherry-picked for 0.5.4

Lint error is fixed. \o/

Thanks for the fix, and sorry for the breakage!

@meteorcloudy I'm afraid that this creates a layering check violation internally. I'll have to roll it back.

@rmlarsen I see, I'm fine with the rollback.
@learyg Is it possible to implement vmodule without introducing stringpiece in logging.cc?
Related change: https://github.com/tensorflow/tensorflow/commit/072b0c9

@meteorcloudy Yes, we could define a hasher for use with the unordered map, that was the primary motivation for grabbing StringPiece (it already had one defined). I'm unfortunately AFK for the next few days, understand if you need to roll back in the meantime, might not be too hard to dupe a custom hasher into that translation unit though.

@rmlarsen Looks like this change is not getting rollbacked? Is it OK to keep it like this way?
Otherwise, @learyg do you mind implementing the hasher for unordered map to avoid include stringpiece?

Hi,
Now we get the same error for gen_proto_text_functions.exe.


fix(#54) conditional to check if the value for _id is None

Description
In StateModel changed conditional to check if the value for _id is None instead of not _id.
Issues:
closes #54
Resolutions

[x] List checklist items here for PR tracking

Dependencies

[x] Pyyaml >= 5.6

Looks good thanks for the contribution!


add a performance test

This shows that jsonb_accessor is two orders of magnitude slower when retrieving records.
Also, the timing is a linear increase given the number of objects. In other words, changing it to 100 products to retrieve would fail the test.

This addresses #15.
I would like to see this down to one order of magnitude with a constant instead of linear increase in time with the number of objects.

This was squash-merged into master.


Actor client requires an actor state store even if not using actors.

In what area(s)?

/area runtime

/area operator

/area placement

/area docs

/area test-and-release

What version of Dapr?

1.5.2
1.6.1
1.7.0

Expected Behavior
An actor client shouldn't need any access to state stores unless it is utilizing a state store.
Actual Behavior
An actor client needs access to a state store configured for actors, not necessarily the same state store.
Steps to Reproduce the Problem
The easiest way is to check out this commit in the PHP repo where this issue was discovered. You can revert the changes in that example and run make && make start (see the example README). After curl -s http://localhost:8080/method/hello_world/get_count | jq ., you'll receive the error:
{\"errorCode\":\"ERR_ACTOR_RUNTIME_NOT_FOUND\",\"message\":\"actor runtime is not configured\"}"

Potentially related issues/PRs: #4273
Release Note

RELEASE NOTE:

@Taction can you look into this please?

@yaron2 Sure, I'll look into this.

This is because the initActors() func has checked if the actor state store is defined: https://github.com/dapr/dapr/blob/master/pkg/runtime/runtime.go#L1771
To fix this issue, we can add the check of actorConfig.EntityConfigs.Entities to tell if it is server-side or client-side.
But consider #4273, we can turn this error into a warning log. If the actor server uses the state store and the actor state store is not defined, the error is state store does not exist (and we can add more info to that error).

This is because the initActors() func has checked if the actor state store is defined: https://github.com/dapr/dapr/blob/master/pkg/runtime/runtime.go#L1771
To fix this issue, we can add the check of actorConfig.EntityConfigs.Entities to tell if it is server-side or client-side. But consider #4273, we can turn this error into a warning log. If the actor server uses the state store and the actor state store is not defined, the error is state store does not exist (and we can add more info to that error).

That's a great find, and agree about changing the error to a warning log.


Revert 944

revert #944
@bidoubiwa  @curquiza @eskombro, this was a misunderstanding from our side. Doing this would in fact be an error, and would prevent us to do this: https://github.com/meilisearch/MeiliSearch/issues/945#issuecomment-685526678, which is what we are really after. We are resetting this to its default behaviour before it goes in prodution. Sorry for the confusion.

bors r+


VOMS library silently ignores invalid ACs

If an AC fails to validate, then it is silently ignored by the library:
https://github.com/italiangrid/voms/blob/master/src/api/ccapi/api_util.cc#L338
Particularly, even if all ACs fail to validate and verification is requested, the VOMS_Retrieve routines will return 0 and simply provide no ACs.
It is not clear why this is done, as the commit history is lost due to a SVN merge.
Is this intentional?  Is there any verification mode that allows me to see all valid and invalid ACs in the proxy?
Best I can figure is to run VOMS_Retrieve twice and compare the results.

Is this intentional?

yes. also VOMS Java APIs follow this principle, but allow you to get information on
why validation failed for a given AC via a callback or a validation result object.
BTW: the blame shows that this is the behaviour since 2008:
https://github.com/italiangrid/voms/blame/master/src/api/ccapi/api_util.cc#L338

Best I can figure is to run VOMS_Retrieve twice and compare the results.

You could do that, but In the end if no ACs are returned you know that no valid ACs were found in the proxy, and if there were ACs that failed the validation you'll get the errors from the OpenSSL error stack.

That's not what we observe: if there were ACs that failed validation, you get no errors at all.
To replicate:

Create valid VOMS proxy.
Remove all LSC files (causing validation to fail).
Call VOMS_Retrieve on the proxy file.

This results in a successful return code.

Hi all,
a VOMS-aware service normally needs a valid AC to determine
the roles and groups that the client possesses, and it will typically abort
the connection if none could be found.  DPM and LFC will fall back to classic
DN authN and authZ in such cases, where the VO is then determined from
a grid-mapfile variant that is updated through edg-mkgridmap.
All in all there appears to be little risk in practice, but I wonder if
it would be safer to return a non-zero value in such cases?
Maybe non-zero when any error was encountered?

Tried to replicate Brian's test, and failed. This simple program shows that actually VOMS APIs do report AC signature validation errors:
#include "voms_api.h"
#include <iostream>

int main(int argc, char *argv[]) {
  vomsdata vd;

  if (vd.RetrieveFromProxy(RECURSE_CHAIN)) {
    std::cout << "Verification succeeded!" << std::endl;
    exit (0);
  }
  else {
    std::cout << "Error Message: " << vd.ErrorMessage() << std::endl;
    exit (0);
  }
}

[voms@7889d52f20d5 ~]$ voms-proxy-init -voms test -cert test0.p12
Enter GRID pass phrase for this identity:
Your identity: /C=IT/O=IGI/CN=test0
Creating temporary proxy ......................................... Done
Contacting  localhost:15000 [Whatever] "test" Done
Creating proxy ...................................... Done

Your proxy is valid until Fri Oct  7 23:53:05 2016
[voms@7889d52f20d5 ~]$ voms-proxy-info -all
subject   : /C=IT/O=IGI/CN=test0/CN=proxy
issuer    : /C=IT/O=IGI/CN=test0
identity  : /C=IT/O=IGI/CN=test0
type      : proxy
strength  : 1024 bits
path      : /tmp/x509up_u1234
timeleft  : 11:59:56
key usage : Digital Signature, Key Encipherment
=== VO test extension information ===
VO        : test
subject   : /C=IT/O=IGI/CN=test0
issuer    : /C=IT/O=IGI/CN=dev.local.io
attribute : /test/Role=NULL/Capability=NULL
timeleft  : 11:59:56
uri       : dev.local.io:15000
[voms@7889d52f20d5 ~]$ g++ -g -O0 -I/usr/include/voms/ -lvomsapi  verify.cc
[voms@7889d52f20d5 ~]$ ./a.out
Verification succeeded!
[voms@7889d52f20d5 ~]$ sudo mv /etc/grid-security/vomsdir/test/dev.local.io.lsc /etc/grid-security/vomsdir/
[voms@7889d52f20d5 ~]$ ./a.out
Error Message: Cannot verify AC signature!

Hi,
VOMS_Retrieve() indeed returns 0 when no VOMS AC has been returned but following that, you can analyse the reason via the errno passed as last parameter. See e.g. https://ndpfsvn.nikhef.nl/viewvc/mwsec/trunk/lcmaps/src/grid_credential_handling/gsi_handling/lcmaps_voms_attributes.c?annotate=18359#l586 and further.
I've just verified with the latest (and previous) VOMS versions, that LCMAPS indeed behaves as expected:

lcmaps_x509_to_voms_fqans(): Warning: Generic verification error for VOMS: AC not valid anymore.. LCMAPS will ignore the VOMS extensions in the mapping sequence.

Furthermore, I agree with Maarten that it is not an issue as we always where of the opinion that the proxy itself may still be used for AuthZ, unless it itself fails verification. So in that case you don't do AuthZ on the VOMS credentials when it is expired, but still can do on the user's DN.
In short, I think the current behaviour is fine, although perhaps counterintuitive. Please don't change it!

Hi Mischa,

In short, I think the current behaviour is fine, although perhaps counterintuitive. Please don't change it!

No way I will change it. It's just a matter of using the API properly.  I am embedding also a C api example (written by Vincenzo years ago for a test):
/* ... */
int main(int argc, char *argv[])
{
  struct vomsdata *vd = VOMS_Init(NULL, NULL);
  int error = 0;
  int i = 0;
  BIO *in = NULL;
  char *of = argv[1];
  STACK_OF(X509)* chain = NULL;
  X509 *x = NULL;

  if (vd) {
    in = BIO_new(BIO_s_file());
    if (in) {
      if (BIO_read_filename(in, of) > 0) {
        x = PEM_read_bio_X509(in, NULL, 0, NULL);
        if(!x)
          exit(1);

        chain = load_chain(of);

        if (VOMS_Retrieve(x, chain, RECURSE_CHAIN, vd, &error)) {
          struct voms *voms = VOMS_DefaultData(vd, &error);

          if (voms) {
            char **fqans = voms->fqan;

            while (*fqans) {
              printf("fqan: %s\n", *fqans++);
            }

            exit(0);
          }
        } else {
		printf("VOMS AC valdation failed!\n");
		exit(1);
	}
      }
    }
  }
  exit(1);
}

Closing this issue.


Proverb

Hello, I added my favorite game player!
can you look and contribute to my repository also?
Thank you in advance!!!!
https://github.com/vincentmichael089/AttractingContributors-Kanji

Sure! I'll add it later.


Implement imprint field

Implement the field imprint considering its nested attributes:

imprint.publisher
imprint.place
imprint.isbn
imprint.title
imprint.pages

Take into account the following requirements when implementing this field:

It is needed in zenodo-rdm.
It must be a custom field.
It must be searchable. Search must be compatible with the current Zenodo search guide
If needed, update DataCite serializer taking into account Datacite Schema 4.4 docs.

A recommended field RelatedItem  can be used to serialize journal's information.

The upload form should be similar to zenodo (e.g. meeting section is called Book / Report / Chapter)


llgo/rust/hyper

Now the question is:
Workaround: Constants can't be converted to pointers

(base) spongehah@MacBook hyper % llgo run hyper.go
panic: unsupported Const: 1, unsafe.Pointer

goroutine 1 [running]:
github.com/goplus/llgo/ssa.(*aBuilder).Const(0xc00197a180, {0x10047a088, 0x1006cde68}, 0xc001f7dae0)
        /Users/spongehah/go/src/llgo/ssa/expr.go:243 +0x405
github.com/goplus/llgo/cl.(*context).compileValue(0xc000d98270, 0x17?, {0x10047b848?, 0xc001c58c00})
        /Users/spongehah/go/src/llgo/cl/compile.go:722 +0x28b
github.com/goplus/llgo/cl.(*context).compileValues(0x100018b6b?, 0x10047b770?, {0xc001c58c20, 0x2, 0xc0031bbc40?}, 0x0)
        /Users/spongehah/go/src/llgo/cl/compile.go:753 +0xe5
github.com/goplus/llgo/cl.(*context).call(0xc000d98270, 0xc00197a1b0?, 0xc004a2e3c0?, 0xc004c8cac0)
        /Users/spongehah/go/src/llgo/cl/instr.go:345 +0x3fd
github.com/goplus/llgo/cl.(*context).compileInstrOrValue(0xc000d98270, 0xc00197a180, {0x10047cf58?, 0xc004c8ca80?}, 0x20?)
        /Users/spongehah/go/src/llgo/cl/compile.go:455 +0x165
github.com/goplus/llgo/cl.(*context).compileInstr(0xc000d98270, 0xc0019eedb0?, {0x10047b728?, 0xc004c8ca80?})
        /Users/spongehah/go/src/llgo/cl/compile.go:618 +0x665
github.com/goplus/llgo/cl.(*context).compileBlock(0xc000d98270, 0xc00197a180, 0xc0078796b0, 0x10047cc78?, 0x0, 0x0)
        /Users/spongehah/go/src/llgo/cl/compile.go:311 +0x51d
github.com/goplus/llgo/cl.(*context).compileFuncDecl.func1()
        /Users/spongehah/go/src/llgo/cl/compile.go:259 +0x3ef
github.com/goplus/llgo/cl.NewPackageEx(0xc000176000, 0xc006024f00, 0xc003417180, {0xc002bcbf58, 0x1, 0x1})
        /Users/spongehah/go/src/llgo/cl/compile.go:832 +0x691
github.com/goplus/llgo/internal/build.buildPkg(0xc0021840a0, 0xc001f7d1e0, 0x0)
        /Users/spongehah/go/src/llgo/internal/build/build.go:437 +0x308
github.com/goplus/llgo/internal/build.buildAllPkgs(0xc0021840a0, {0xc0052cea18?, 0xc00034a500?, 0x1003f5e8e?}, 0x8?)
        /Users/spongehah/go/src/llgo/internal/build/build.go:306 +0x3c5
github.com/goplus/llgo/internal/build.Do({0xc000118140?, 0x1?, 0x1?}, 0xc000149400)
        /Users/spongehah/go/src/llgo/internal/build/build.go:181 +0x9e5
github.com/goplus/llgo/cmd/internal/run.runCmdEx(0xc000165e48?, {0xc000118140?, 0x100100be0?, 0x3?}, 0x10040849b?)
        /Users/spongehah/go/src/llgo/cmd/internal/run/run.go:62 +0xb0
github.com/goplus/llgo/cmd/internal/run.runCmd(0xc000144000?, {0xc000118140?, 0x0?, 0xc00004c6b8?})
        /Users/spongehah/go/src/llgo/cmd/internal/run/run.go:50 +0x1d
main.main()
        /Users/spongehah/go/src/llgo/cmd/llgo/llgo.go:90 +0x4b3

The question is：
The second loop went wrong：
On the second loop, it will read userdata == 0 incorrectly, and then enter the case ExampleNotSet, resulting in inconsistency with the reference result of c demo (the correct result of c demo is that when the second loop is not run to switch, task := exec. Poll(), the value of the task of this code is null, and then the inner loop is executed select, and the problem with go demo is the value of the second loop of this code != null)

Workaround: Function pointer need to use llgo::type: C to declare it.
(base) spongehah@MacBook hyper % llgo run hyper.go
connecting to port 80 on httpbin.org...
connected to httpbin.org, now get /
http handshake (hyper_client v1.4.0) ...
userdata set:  0x1

round:  1
task is null ?  0x600000b58bd0
userdata:  1
ExampleHandshake
preparing http request ...
userdata set:  2
sending ...

round:  2
task is null ?  0x600000b54240
userdata:  0
ExampleNotSet

round:  3
task is null ?  0x600000b54330
userdata:  2
ExampleSend
send error!
error code: 1
details: operation was canceled
(base) spongehah@MacBook hyper %

Take a look at the correct results of the c demo:
(base) spongehah@MacBook examples % ./client
connecting to port 80 on httpbin.org...
connected to httpbin.org, now get /
http handshake (hyper v1.4.0) ...

rountd: 1
not null
userdata: 1
EXAMPLE_HANDSHAKE
preparing http request ...
userdata set : 2
sending ...

rountd: 2
null
null
select
select pre
select after

rountd: 3
not null
userdata: 2
EXAMPLE_SEND

Response Status: 200 OK
date: Tue, 16 Jul 2024 10:01:20 GMT
content-type: text/html; charset=utf-8
content-length: 9593

// Below are the results returned
...

null
userdata: 0
EXAMPLE_NOT_SET

rountd: 5
not null
userdata: 3
EXAMPLE_RESP_BODY

Now we've done some encapsulation of the original FFI functions to see if we need to specify the functions
Also, it is now not possible to save the content of the request to the returned response


uuid-correction

Hello @ksigler7,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


Importing markupsafe instead of MarkupSafe

There's an import error due to MarkupSafe module being imported even if it doesn't exist. The import has been renamed as markupsafe

This has been fixed in 45e8b657d now. Thanks :)


Location coordinates API is not working,

Location coordinates API is not working,  "https://raw.githubusercontent.com/zcreativelabs/react-simple-maps/master/topojson-maps/world-110m.json"
Unable to get data for global map countries, please help for this issue,

Hi @gaurav043goyal,
#289 should solve the issue.


tf.contrib.image.transform in TF2.0

Please make sure that this is a feature request. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:feature_template
System information

TensorFlow version (you are using): 2.0
Are you willing to contribute it (Yes/No): No

Describe the feature and the current behavior/state.
In TF 1.X there were nice methods tf.contrib.image.transform() and tf.contrib.image.compose_transforms() that allowed to perform affine transforms with an image.
In TF 2.0 whole tf.contrib.* is dropped.
For instance, I was using those in my custom augmentation mechanisms to add rotation and shear transforms (along with all zooms, shifts, flips, etc). And in TF 2.0 I see no way to perform those.
I tried to implement such a functionality with numpy+opencv instead of TF. However when I tried injecting such a functionality to tf.dataset.map() pipeline tensors seemed not to have .nupmy() method yet.
I also found tf.keras.preprocessing.image.apply_affine_transform() which does perform affine transform on image, but according to docs it works with numpy array only.
Will this change the current api? How?
I guess that would be nice to port those functions form 1.14 to 2.X to: tf.image.compose_transforms() and tf.image.transform() or something. Should not affect existing API I guess.
Who will benefit with this feature?
Anybody who likes to perform affine transforms on image say in data augmentation step of model training.
Any Other info.
If I miss something and mentioned functionality can be implemented with existing API, please let me know.

tf.contrib.image.transform and tf.contrib.image.compose_transform are now part of the tensorflow_addons.image module which should implement the functionality you are looking for.

@chayka As mentioned in the previous comment, you can refer to tensorflow_addons to find the same functions. Closing this issue as it has been resolved.

Thank you guys a lot! Could be a nice idea to leave a note in tf.contrib.* branch docs that some functionality is not dropped, but could be found somewhere else. Spent quite some time looking for it and still missed it. But anyway, thanks for TF2.0, great job!


Create bloomreach.md

Adding Bloomreach CMS page with images

hi @erquhart - i wanted to add bloomreach to the list, any chance you could take a look? :)

Looks good 👍

thanks, @erquhart 🙌 looks like the deploy failed though https://app.netlify.com/sites/headlesscms/deploys

I would assume it's because it says it's open source but has no GH repo, but the latest deploy succeeded so 🤷♀
Sent with GitHawk


upgrade to postcss 8, fixes #39

This PR updates the plugin to use the postcss 8 plugin style. This requires a major version bump.

@goto-bus-stop It looks like you should update yarn.lock file

@Semigradsky thanks, done :blush:

@Semigradsky @goto-bus-stop Thank you, guys. Just published an upgraded package


Optimize build procedure

Eliminate .git from the Dockerfile step completely. This step prevents caching of layers and copies .git content to the build context. The only reason we do it is version number, however, we deploy with drone only and for any manual build can just set version to "local"
Remove drone PR pipeline. No need for this since we switched to public travis for PR builds

done with https://github.com/umputun/remark/commit/02d298aac63faf2fdb22baad2e21c194c43dc479


[BUG] Get Recent Doc By Date (in a subquery)

My Environment

ArangoDB Version:        3.7.2
Storage Engine:             RocksDB
Deployment Mode:       Single Server
Deployment Strategy:   Manual Start
Operating System:        CentOS 7
Total RAM in your machine:        8Gb
Disks in use:        HDD

Component, Query & Data
__AQL query __:
FOR s IN crawler_sites
        LET last_crawl_log = FIRST(FOR lc IN crawler_logs
          FILTER s._key == lc.siteId AND lc.is_single_product != true
          SORT lc.begin DESC
          LIMIT 1
          RETURN UNSET(lc, "failedAPIOperations", "links")
        )
        SORT last_crawl_log.begin ASC
      RETURN {
        siteId: s._key,
        link: s.link,
        begin: last_crawl_log.begin
      }

Problem:
I need to sort "websites" based on their recent crawl date.
when in the loop, last_crawl_log is assigned by a date older than the recent date.
but if I run the subquery as a separate query, last_crawl_log  is assigned correctly.
to correct the query I changed it to:
__AQL query __:
FOR s IN crawler_sites
        LET last_crawl_log_max_date = FIRST(FOR lc IN crawler_logs
          FILTER s._key == lc.siteId AND lc.is_single_product != true
          COLLECT AGGREGATE max_date = MAX(lc.begin)
          RETURN max_date
        )
        LET last_crawl_log = FIRST(FOR lc IN crawler_logs
          FILTER lc.begin == last_crawl_log_max_date
          LIMIT 1
          RETURN UNSET(lc, "failedAPIOperations", "links")
        )
        SORT last_crawl_log.begin ASC
      RETURN {
        siteId: s._key,
        link: s.link,
        begin: last_crawl_log.begin
      }

Hi @p30arena, thanks for reporting!
This is probably a duplicate of https://github.com/arangodb/arangodb/issues/12693. Could you try and turn off the optimizer rule splice-subqueries? See https://www.arangodb.com/docs/stable/aql/execution-and-performance-optimizer.html#turning-specific-optimizer-rules-off how to do that. If that works around your problem it would reassure my guess, or otherwise disprove it.
If this is indeed the same bug, I can already tell you that it is already fixed and the fix will be available in 3.7.3.
Best,
Tobias

@goedderz
Hi Tobias!
Yes, that's the issue.

Duplicate of #12693


Add PyYAML dependency

PyYAML
Description

The PyYAML dependency is added to the lock and toml files pertaining to Poetry in order to create configuration files. This is something intended for contributors in multiple feature branches to use. Further documentation is available in this link.
Type of Change

[x] Feature
[ ] Bug fix
[ ] Documentation

Contributors

@ColemanKobe

Reminder
All GitHub Actions should be in a passing state before any pull request is merged.

Hi @ColemanKobe,
For feature pull requests, please request reviews from team members instead of Saejin or me.
We will review the release candidate for each sprint but will leave reviewing feature PRs for teams to do by themselves 🙂
Please let me know if you have any questions about the feature PR review process!

That's right. My apologies for that

No worries, @ColemanKobe. This is something I will clarify for everyone since I don't believe I've explicitly mentioned it before.


Unable to unschedule activity

Able to unschedule 1 d/1

But unable to unschedule  4 d/2

The activity index is based on the number of activity for the day itself. So for your case if you are trying to unschedule the first activity from day 2,  the command is unschedule 1 d/2. If you want to unschedule the second activity of the day for Day 3, the command is unschedule 2 d/3.

But still thank for checking👍

This means we need a better UG 🤔

I've updated the UG on my side, will do a PR after we're done with the other more important stuff

unschedule 1 d/2 also not working for me

Thanks for your input! We have identified the cause of the bug and are working on fixing it right now


Fix stable/grafana chart

Use secretRef instead of secretKeyRef, because secretKeyRef is not
a valid key for envFrom.
Here are the relevant
docs.
What this PR does / why we need it:
Fix stable/grafana chart.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #5549

/ok-to-test

/lgtm

@zanhsieh Thanks for taking a look at this! I squashed the two commits and also updated the commit message with the test I used to verify this works on my branch (and repo the issue on master). Do you mind adding the /lgtm so it adds the proper label and merges? Thanks!

@mattjmcnaughton good catch.  lgtm


BitBarDarkMode ENV is always true

Using Mac OS 11.4 and running Xbar 2.0.42-beta and I'm using a plugin (github-review-requests.5m.py)  that sets text colour based on $BitBarDarkMode:
DARK_MODE = os.environ.get('BitBarDarkMode')
Changing OS dark mode on and off makes no difference to the value of this var - it always evaluates to true and thus text appears white and is unreadable when dark mode is off.
Seems related to #470.

I'm seeing this issue too. I'm on MacOS Monterey 12.4 (21F79)

While the variable is now XBARDarkMode, it is still always true. Will this be fixed as it's been open for nearly 2 years at this point?

Thanks for bumping this. There's stuff happening in the background regarding the project so expect some movements in the near future. Let us know if you'd be willing to contribute.


Bump Spriggit version.  Reset ModHeader version to default

Fix for not including Starfield.esm master
Made versions match default mod given by Elm a bit more

Gif of the diffs in the output.   Mostly expected and desired:

Localization flag on
Description/Author fields remain

The rest seems in line with the file Elm provided.  Let me know if there's something I missed


Trim leading garbage before dollar sign

Trim leading garbage on GPS received string which would previously return an error. The string is still good other than the leading garbage

What's the source of the garbage I wonder?

Thats a great question. It appears to be binary data but not consistent

partial bit strings, previously ignored.
On 10/04/2016 10:32 PM, cyoung wrote:

What's the source of the garbage I wonder?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cyoung/stratux/pull/505#issuecomment-251573578,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANAzoTmQTb4uO9cZ3yB-5MDpTEzefJKSks5qwxo9gaJpZM4KOYqe.

Well, @jamez70 pasted some output from his GPS (re-paste for posterity if you want) - every line seemed to be valid but with prepended invalid data.

2016/10/05 03:04:25 GPS error. Invalid NMEA string: Invalid NMEA message String: �!�%�1$PUBX,00,030425.40,4233.59655,N,08751.83185,W,174.532,D3,4.4,6.9,1.177,58.41,0.502,,0.93,1.31,0.97,12,0,0*77 2016/10/05 03:04:29 GPS error.   Or a bus pirate.  Usually,  it's the remainder of the previous string coming from the GPS before code starts reading that bus.  Once, on a chinese gps, it was spurious noise from the gps circuitry but that was only one single chip.  Upon powerup, gps will start spewing strings but code doesn't start looking for it until later and with these boards, there is no buffering prior to port init.

That looks exactly like that one chip that we tossed.  At times, on the very first string, there is a little bit-twiddling until the speeds start sync-ing..  When it repeats after first valid string, it appears to be a faulty chip.  Hmm, could be RF noise, as well, but that would show up inside the valid strings as well.

I have a bus pirate on the way.

I have one, a few scopes and some other junk around here
On Tue, Oct 4, 2016 at 10:46 PM, skypuppy<EMAIL_ADDRESS>wrote:

I have a bus pirate on the way.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cyoung/stratux/pull/505#issuecomment-251574980, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABFQ04U38XOeR9Wu1A_-7CVKdeUwJ63sks5qwx2EgaJpZM4KOYqe
.

a logic analyzer is the ideal tool for this, esp if it can convert to ASCII.

A scope can show if it's RF interference, by voltages being too high or too low compared to standard serial or USB voltages, also with random pulse widths.

You need to pull out the old digital storage scope.  :)

Another problem with using a scope is the variable, valid signal widths.

And I have just exhausted my knowledge of scopes, even if I have 5.  Only one is a digital storage one and it's a small portable unit.

I'll use the 12GHz scope at work to make sure PCIe isn't on the bus
On Tue, Oct 4, 2016 at 10:51 PM, skypuppy<EMAIL_ADDRESS>wrote:

And I have just exhausted my knowledge of scopes, even if I have 5. Only
one is a digital storage one and it's a small portable unit.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cyoung/stratux/pull/505#issuecomment-251575489, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABFQ06AIS379RkUu0E-HepgZQ3I9V369ks5qwx7YgaJpZM4KOYqe
.

PCIe on a Pi?

I'm just kidding
On Tue, Oct 4, 2016 at 10:59 PM, skypuppy<EMAIL_ADDRESS>wrote:

PCIe on a Pi?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cyoung/stratux/pull/505#issuecomment-251576254, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABFQ049x5itqfr9OVILkNwD4yb2Vmof1ks5qwyCjgaJpZM4KOYqe
.

As I was lying down I remembered a much more common occurrence: that of setting the baud rate and/or gps report rate higher than the system can keep up with in a given circumstance.

Well I ran it for 12+ hours now - not one NMEA error
On Tue, Oct 4, 2016 at 11:28 PM, skypuppy<EMAIL_ADDRESS>wrote:

As I was lying down I remembered a much more common occurrence: that of
setting the baud rate and/or gps report rate higher than the system can
keep up with in a given circumstance.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cyoung/stratux/pull/505#issuecomment-251579033, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABFQ0yEZqrnwfLLWcKEa9IVVT41pCIv7ks5qwyd9gaJpZM4KOYqe
.

are you near 4 towers?

What do you mean
On Wed, Oct 5, 2016 at 7:31 PM, skypuppy<EMAIL_ADDRESS>wrote:

are you near 4 towers?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cyoung/stratux/pull/505#issuecomment-251837656, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABFQ0xtN13u9dF8N5CD66chuzqcFnUIUks5qxEF1gaJpZM4KOYqe
.

AAARRRRRRR!!!!

@jamez70 https://frillip.com/raspberry-pi-3-uart-baud-rate-workaround/

@jamez70, yeah, I had read through that before plus it's link, but by post 7 or 9 of it's link, I was completely baffled and had lost my ability to parse English by the last post.

haha
On Tue, Oct 18, 2016 at 12:48 PM, skypuppy<EMAIL_ADDRESS>wrote:

@jamez70 https://github.com/jamez70, yeah, I had read through that
before plus it's link, but by post 7 or 9 of it's link, I was completely
baffled and had lost my ability to parse English by the last post.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cyoung/stratux/pull/505#issuecomment-254585191, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABFQ05y-qTQ3hH84VbZXqRpx86fbJzvrks5q1QZYgaJpZM4KOYqe
.

Closing this, I think it's an RPi serial clock issue.


Doc Alignment: Button Improvements

Describe the bug

Several button (& related) improvements:

Update ToggleSplitButton sample per the docs: https://docs.microsoft.com/en-us/windows/uwp/design/controls-and-patterns/buttons#differences-with-togglebutton
Update Button sample showing graphical content to look like the doc example showing a photo button: https://docs.microsoft.com/en-us/windows/uwp/design/controls-and-patterns/buttons#button-content
Add a sample to DropDownButton page with a RichEditBox including alignment to show a DropDownButton in situ: https://docs.microsoft.com/en-us/windows/uwp/design/controls-and-patterns/buttons#create-a-drop-down-button

Should those all be new samples or should we change existing samples to align with the docs here?


Improve support for different backends other than JMX

Jolokia supports currently support two different so called RequestDispatcher:

A LocalRequestDispatcher for directly querying the local JMX subsystem. This dispatcher is always present
A Jsr160RequestDispatcher which is used as proxy to another remote JSR-160 exposed JVM instance. This dispatcher is optional and kicks in when a target section is present in the initial JSON request.

RequestDispatchers can be configured with the configuration variable dispatcherClasses and must provide a three argument constructor to get a reference to its context (see the implementations above for examples). The dispatcher are given as a comma separated list in this configuration variable and must be creatable by the Jolokia agent.
However this mechanism is somewhat rusty:

There should be support for automatic lookup via OSGi services (whiteboard pattern)
The API for the dispatcher is not really polished (and was up to now only somewhat internal)
Maybe the incoming request should have a dedicated field for selecting the dispatcher (instead of letting the dispatcher examing the requests)

More  on this later, have to leave now ...

The hooks are in 2.0, beed to implement a second (Spring ?) implementation to be sure that the API fit. Will do that in the new 2.0 timeframe.
Jolokia 2.0, the Duke Nukem Forever for chili pepper named software projects.

Hey everyone!  I'm interested in a JBoss/Wildfly CLI-based backend, and I can contribute some code if someone points me in the right direction.  Sounds like I should implement my own RequestDispatcher?  How do I go about this?  Since the JBoss CLI model is similar to the JMX model, should I copy/paste the JMX dispatcher code?  Is it worth abstracting to share similar code?

Awesome ! ;-)
I started the a SpringRequestHandler which can be easily used as starting point. In fact, I'm currently working on it, too, to add the missing features. Currently the read operation and the list operations are implemented, but not yet tested to much.
But you could also start with the LocalRequestHandler which is the entry point for JMX handling.
Of course, if we could share code among the various request handlers (e.g. for extracting 'deep' values via pathes or for the wildcard handling for the read operation) this would be quite cool. 'haven't looked to much into it, yet, though.
We can discuss this further on IRC later (#jolokia on freenode), however I'll be on the road until wednesday (and have to do some preps right now).
thanks ...

I'm using JBOSS AS 7.1.1 and Jolokia JVM Agent 1.3.3.

I'm seeing below error when trying to access JDBC pool ststistics using hawtIO connected to JVM through Jolokia remotely. I understand that only what's been exposed through JMX can be read using Jolokia and not what exposed using JBOSS management API in Jolokia 1.3.3. Do you have a stable snapshot for use in Load testing environment (not production) which I can use to monitor JBOSS using Mgmt API?

javax.management.AttributeNotFoundException : JBAS014640: 'statistics' is a registered child of resource ([\n    (\"subsystem\" => \"datasources\"),\n    (\"data-source\" =>

As such, is it advisable to run Jolokia in production JVM for monitoring? Can I expect the overhead to be in the leagues of Dynatrace & App Dynamics agents?

Thanks!


Can you customize action/condition list?

Is it possible to remove certain actions/conditions seen in their select menus? Ideally, there would be an option to create custom elements, but the most important thing is probably to remove redundant ones (for example, in games where NPCs don't have their routines, seeing that option could be confusing for quest makers).

Good idea. I guess I could fit something like this into the project settings area. That way one could enable and disable certain features like this.


SCRIPT_NAME server variable missing

Used by the Gravity Forms User Registration Add-On
Notice: Undefined index: SCRIPT_NAME in /var/task/web/app/plugins/gravityformsuserregistration/class-gf-user-registration.php on line 389

Notice: Undefined index: SCRIPT_NAME in /var/task/web/app/plugins/gravityformsuserregistration/class-gf-user-registration.php on line 395

While the parameter is there in the request, some parameters don't seem to show up such SCRIPT_NAME and DOCUMENT_ROOT.


Threading issues

Issue #34 concerned thread crashes in the Send tab.  After some fiddling with threads (Send and Copy), I couldn't get it to crash, but don't really know what I did, if anything, that fixed any problem.  If problem comes back, consider the below.
For Copy tab, have a long-running python thread that just hangs on a given Queue until a char is found, it sounds it and then waits on the Queue again.  This thread started at app start, etc.  How does main thread know that "sound" has finished?  Perhaps an 'output' queue that sound thread puts something on when finished?  But main thread is GUI, and is waiting???  Does main thread need to know that "sound" has finished?
For Send tab, ...??

On 4 March got crash in CopyThread:
Thread 8 Crashed:: CopyThread
0   libsystem_kernel.dylib        	0x00007fffc2f34dd6 __pthread_kill + 10
1   libsystem_pthread.dylib       	0x00007fffc3020787 pthread_kill + 90
2   libsystem_c.dylib             	0x00007fffc2e4b497 raise + 26
3   libsystem_platform.dylib      	0x00007fffc3013bba _sigtramp + 26
4   ???                           	0x0000000000000001 0 + 1
5   org.qt-project.QtCore         	0x00000001077be497 0x107794000 + 173207
6   org.qt-project.QtCore         	0x00000001077be339 0x107794000 + 172857
7   libsystem_pthread.dylib       	0x00007fffc301daab _pthread_body + 180
8   libsystem_pthread.dylib       	0x00007fffc301d9f7 _pthread_start + 286
9   libsystem_pthread.dylib       	0x00007fffc301d1fd thread_start + 13
There is another thread with the name CopyThread:
Thread 9:: CopyThread
0   libsystem_kernel.dylib        	0x00007fffc2f34fda __semwait_signal + 10
1   libsystem_c.dylib             	0x00007fffc2ebbb72 nanosleep + 199
2   libportaudio.2.dylib          	0x00000001094496d6 Pa_Sleep + 97
3   libportaudio.2.dylib          	0x000000010944eb78 WriteStream + 47
4   _portaudio.cpython-36m-darwin.so	0x0000000109439534 pa_write_stream + 132
5   org.python.python             	0x0000000106b9c266 _PyCFunction_FastCallDict + 172
6   org.python.python             	0x0000000106c02f8d call_function + 562
7   org.python.python             	0x0000000106c00221 _PyEval_EvalFrameDefault + 24228
8   org.python.python             	0x0000000106c0376a _PyEval_EvalCodeWithName + 1910
9   org.python.python             	0x0000000106c03e43 fast_function + 241
10  org.python.python             	0x0000000106c02f67 call_function + 524
11  org.python.python             	0x0000000106c00221 _PyEval_EvalFrameDefault + 24228
12  org.python.python             	0x0000000106c040d7 _PyFunction_FastCall + 121
13  org.python.python             	0x0000000106c02f67 call_function + 524
14  org.python.python             	0x0000000106c00221 _PyEval_EvalFrameDefault + 24228
15  org.python.python             	0x0000000106c040d7 _PyFunction_FastCall + 121
16  org.python.python             	0x0000000106b6331d _PyObject_FastCallDict + 218
17  org.python.python             	0x0000000106b6343c _PyObject_Call_Prepend + 149
18  org.python.python             	0x0000000106b63184 PyObject_Call + 101
19  sip.so                        	0x0000000107d80979 call_method + 85
20  sip.so                        	0x0000000107d7cda4 sip_api_call_procedure_method + 164
21  QtCore.so                     	0x0000000107597114 sipQThread::run() + 84
22  org.qt-project.QtCore         	0x00000001077be32a 0x107794000 + 172842
23  libsystem_pthread.dylib       	0x00007fffc301daab _pthread_body + 180
24  libsystem_pthread.dylib       	0x00007fffc301d9f7 _pthread_start + 286
25  libsystem_pthread.dylib       	0x00007fffc301d1fd thread_start + 13
Not sure I can read this correctly, but it looks like thread 9 is a parent of thread 8.

Wrote a program test.py that doesn't use PyQt5.  It repeats a given string indefinitely, using a python thread to sound each character, with the main thread waiting on the sound thread.  This runs for long periods without error.
Doing date; make; date and getting the Copy tab sounding Morse will crash within about 2 minutes.  This is using PyWt5 QThreads, of course.
Write a very simplified PyQt5 program similar to the console code, just repeat morse chars using a QThread for each character.

Running date; make; date and letting the Copy tab just run at various cwpm/wpm gives constant crashes.  Usually short times for 40/40 (~2m) and longer times for slower speeds.  Analyse the crashes in ~/Library/Logs/Disgnostic Reports.
Nothing seemed to noticeably influence the crash time - it looks like a race condition.

Wrote timeit.sh that repeatedly runs MT with Copy process running automatically.  Shell script measures time until crash and logs date/time/cwpm/wpm/time.
Early runs had web page https://www.cnet.com/news/tutorial-an-introduction-to-reading-mac-os-x-crash-reports/ with auto-run video running.  Later runs didn't have video.
Write code to correlate timeit.sh data with analysis of crash files in ~/Library/Logs.

After much fiddling, I turned off the pyaudio usage in CopyThread and replaced it with a time.sleep(0.025) call.  Testing with the auto-run video at https://www.cnet.com/news/tutorial-an-introduction-to-reading-mac-os-x-crash-reports/ gets a much more crashy test run.
So it looks like our problem isn't related to pyaudio.  It may be threading!?

Well, after changing from a user-defined signal to the system-defined 'finished' signal to show the Send thread is finished I no longer get any crashes!????  Test this a little more.

OK, assume the Send crashes are fixed.  Test the Copy tab, which may be hard to automate.  Think about trying to play the ARRL test tapes back.

Can't automate the Send function, but some testing shows no problem.  Assume it's fixed.  Closed.


Why generate the QR-code that cannot be scanned

Hi, JGPeng, feel free to providing more info such as system info or bash info

@JGPeng
Hi, you need to use -w


Placing begin at end of line is very unpleasent to look at, compact code tip.

A lot of PascalCoin code is written like this:
if some long line of conditions, blablablablablablablablabla then begin
some more code
some more code
end;
This is very unpleasent to read because one has to make sure that the begin statement is actually there otherwise it would be a bug or a misinterpretation of the code.
To be sure that the "some more code' is actually a full statement block one has to move one's eyes to the end of the line to see if there is indeed a begin statement.
This causes a lot of unnecessary eye strain/mussle movements which makes reading the code very unpleasent. It also makes it a bit harder to follow cause one constantly have to move eye/focus back and forth across long lines of code. It also causes at least 1 second of time wasted per "line end of begin"
This approach does have a benefit though. Vertically less white space/wasted lines, like in python, this makes the code more compact and vertically it begins easier to see what a procedure/method is doing in totality.
However if individual statement blocks are hard to follow as above, then this is a mute point.
One has to understand the statement blocks first to be able to understand the rest of the code.
So I think I will prefer the normal pascal approach:
if some long line of conditions, blablablablablablablablabla then
begin
some more code
some more code
end;
This will at least make it a bit more easy to follow statement blocks and be sure that it is valid, at the expense of slightly less compact code.
Perhaps splitting up methods into more smaller methods might make it even easier to follow what the code is doing, this technique will make the code more compact. So that is a win-win situation ;)

I use the 'format Project Source' option in Delphi to make it more readable. Only changed the line length from 80 to 160 in the format settings.

I have tried this as well. Unfortunately it's not perfect, for example:
if A <> B then begin
end else begin
end;
will be reformatted as:
if A<>B then
begin
end
else
begin
end;
I would like it better if it was formatted as:
if A <> B then
begin
end else
begin
end;
Unfortunately the formatter can strangely enough not do this in same cases. For now I will assume it is a bug in the formatter, one interesting exercise is to format the original pascal compiler 1.0 source code.
Either it is written in a very strange way for example:
if a<> b then
if b <> c then
if
if
if
It could be that this original compiler is written like a bunch of nested if statements, or there is something wrong with Delphi's formatter =D
But it's not hard to see that it does have some serious and odd limitations.


[Feature Request]: few_shot_label_diversity flag

Is your feature request related to a problem? Please describe.
I want to make sure that all possible labels are included as examples in the few-shot examples.
This is particularly important when the labels are imbalanced in the seed set, and thus the common labels are more likely to be retrieved regardless of semantic similarity or MMR.
Describe the solution you'd like
few_shot_label_diversity should be a boolean flag. If True, presenting diverse labels should be the primary criterion for selecting the few-shot examples, followed by semantic similarity or max_marginal_relevance.
Additional context
It's probably best to demonstrate all possible labels in a balanced way, regardless of whether the labels are very imbalanced. (However, that could be a future TODO.)
There should be some order to the labels and examples are selected in a round-robin fashion, maximizing semantic similarity or MMR.
I don't think the round-robin order of labels should go from most frequent to least frequent label, or vice-versa, because that means there will always be the same number of each label shown, which might be biased if few_shot_num // len(labels) != 0.
Instead, my preference would be that the label order is RANDOM. One way to do this but still have cache hits is a deterministic random strategy. i.e. based upon the example batch or the prompt WITHOUT the seed examples in it, use that text to seed a new Python rng. (Python rngs can take an kind of input.) Then, pick the order of the labels in round-robin fashion. This ensures that the order of the few show examples for the prompt are identical for the same batch of examples.

Thanks @turian for opening this issue! Absolutely agree with the suggestion to select few-shot examples to ensure label coverage - this will be especially helpful in labeling tasks with a finite label space (classification, entity match, multiple choice question answering, grading generated outputs by assigning it a score on a scale of 1 to 5 etc).
We can support this with two new selection techniques:

label diversity -- few-shot examples are selected to ensure coverage across all labels. Within each label, the specific few-shot examples are sampled at random

label diversity with semantic similarity -  few-shot examples are selected to ensure coverage across all labels. Within each label, the specific few-shot examples are semantically similar to the input to be labeled.

Would love to hear your thoughts on this! We're prioritizing this feature in our upcoming sprint - expect some updates on it in the coming week

@Abhinav-Naikawadi follow up about implementations for these

https://github.com/refuel-ai/autolabel/pull/436

label diversity -- few-shot examples are selected to ensure coverage across all labels. Within each label, the specific few-shot examples are sampled at random
label diversity with semantic similarity -  few-shot examples are selected to ensure coverage across all labels. Within each label, the specific few-shot examples are semantically similar to the input to be labeled.

Yes! This would be everything I need.

@Abhinav-Naikawadi Could there be a small doc for this?

hi @turian  -- we just updated the docs here: https://docs.refuel.ai/guide/accuracy/few-shot/
For the new techniques, you can use the following:
"few_shot_selection": "label_diversity_random",

and
"few_shot_selection": "label_diversity_similarity",

this feature will be part of 0.0.8, expecting to release by end of today!

https://pypi.org/project/refuel-autolabel/0.0.8/


App.route() not matching all variable wildcards

Expected behavior
variable wildcards match expected queries (@api.route('{path}') should match all urls, but does not)
Actual behavior
variable wildcards only match some expected queries (the above can match any url with a single slash after the domain, ie http://example.com/page or http://example.com/image.png)
To Reproduce
import bocadillo
api = bocadillo.App()

@api.route('{path}')
async def slash(req, res, path) :
	print('success')

if __name__ == '__main__' :
	from uvicorn.main import run
	run(api, host='<IP_ADDRESS>', port=80)

FAILS
import bocadillo
api = bocadillo.App()

@api.route('{}')
async def slash(req, res) :
	print('success')

if __name__ == '__main__' :
	from uvicorn.main import run
	run(api, host='<IP_ADDRESS>', port=80)

SUCCEEDS
Screenshots/Traceback

Possible solutions
This might be the expected or intended behavior, but this wasn't how app.route behaved previously, as this is only occurring after updating bocadillo. The error seems like it should be in how bocadillo matches urls to the specified routes.
Specs

OS + Version: OpenSUSE Leap 15.0
Python Version: 3.6.5
Bocadillo Version: 0.15.0

Additional context
I first discovered this by finding out that my route for images only worked on images in the root folder: @api.route('/{path}.png') only caught urls on the root. ie http://example.com/image.png versus http://example.com/images/image.png.

Hey, thanks for reporting this.
I do think this has something to do with a route parameter not able to contain slashes itself. We changed how route parsing is performed in 0.15 and I remember encountering this issue myself.
I might have already pushed a fix to master or another branch. Anyway, I will take a look at it very soon (hopefully tomorrow). :-)

@DariusMiu Okay so, I looked this up. You should be able achieve what you want using req.url.path:
import bocadillo
api = bocadillo.App()

@api.route('{}')
async def slash(req, res) :
    print("do stuff with", req.url.path)

See also Request URL.
Considering this, I don't think this requires modifications to the code, but I think we can add this trick to the docs.

@florimondmanca Yeah, I figured that out, too. that requires additional parsing, though. in my example, I have file handlers for things ending in .png and the like, and didn't want to check the url a second time

I'm not sure I understand your use case. Can you share the exact route pattern you're using? I suspect it would be something like /images/{path_to_image}.png, where path_to_image should be able to contain slashes, right?

@florimondmanca pretty much that. here's my code: https://gitlab.com/kheina/kheina.com/blob/master/server.py#L112

Thanks for the link.
I see you're retrieving the path in the URL in order to read the corresponding file from disk.
Have you considered using static files instead? If so, what prevented you from using it?

Hi @DariusMiu, is this still an issue for you, or have you found a workaround?

Yes, but I haven't been working on it actively. Have a couple other projects plus just started a new job, so quite busy. I'll look into it this weekend and report back!

I can get things working with static files well enough, though I can't set how long to cache things for specific file types, or set headers for file types. It also seems as though errors raised through static files aren't handled by the error handler specified for bocadillo.HTTPError

Additionally, if a static file is changed while the server is still running, I get the following error:
ERROR: Exception in ASGI application
Traceback (most recent call last):
  File "/usr/lib/python3.6/site-packages/bocadillo/applications.py", line 242, in dispatch
    return await app(scope, receive, send)
TypeError: __call__() takes 3 positional arguments but 4 were given

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.6/site-packages/uvicorn/protocols/http/httptools_impl.py", line 368, in run_asgi
    result = await app(self.scope, self.receive, self.send)
  File "/usr/lib/python3.6/site-packages/bocadillo/applications.py", line 257, in __call__
    await self.asgi(scope, receive, send)
  File "/usr/lib/python3.6/site-packages/starlette/middleware/trustedhost.py", line 51, in __call__
    await self.app(scope, receive, send)
  File "/usr/lib/python3.6/site-packages/bocadillo/applications.py", line 245, in dispatch
    return await app(scope, receive, send)
  File "/usr/lib/python3.6/site-packages/starlette/middleware/wsgi.py", line 62, in __call__
    await responder(receive, send)
  File "/usr/lib/python3.6/site-packages/starlette/middleware/wsgi.py", line 90, in __call__
    await asyncio.wait_for(sender, None)
  File "/usr/lib64/python3.6/asyncio/tasks.py", line 339, in wait_for
    return (yield from fut)
  File "/usr/lib/python3.6/site-packages/starlette/middleware/wsgi.py", line 105, in sender
    await send(message)
  File "/usr/lib/python3.6/site-packages/uvicorn/protocols/http/httptools_impl.py", line 489, in send
    raise RuntimeError("Response content longer than Content-Length")
RuntimeError: Response content longer than Content-Length

Thanks for reporting back!
If the static plugin/helper is not enough, you can try building WSGI WhiteNoise apps directly (see here), one for each file type, and then mount them as a nested WSGI apps. (The docs say that nested apps should be either Bocadillo or ASGI apps, but WSGI is actually also supported.)
As for the http_tools runtime error — that’s interesting... I’ll have to try to reproduce this.

code is up on gitlab, you can see the error occurring here:
https://kheina.com/page/that/does/not/exist
https://kheina.com/static/file/that/does/not/exist.png
Thanks for the WSGI tip, I'll look into it!

@DariusMiu I was able to reproduce the RuntimeError issue :+1:  see #288.

@DariusMiu I will be releasing a fix in 0.16.1 (see #299) which will allow you to have /{path:path}.png (note the :path converter) match /images/image.png. This is potentially impacting you so I thought I'd raise notice.

@DariusMiu I am closing this for housekeeping purposes, since I believe the fix mentioned in my previous comment should allow you to achieve what you initially asked. But feel free to comment again if you've been lucky with manually adding WSGI apps. :-)


Ability to receive records as Map or Object (associative aray by unique value) in findMany() function

Problem
Very often the findMany() function used to receive records, that must be quickly accessible by it's unique identifier (id field or some other field).
So in most of times developers need to implement the conversion of array, received from prisma-client, to Object or Map by needed field as key.
Suggested solution
Will be good to have build-in argument in function findMany() to optionally return records as keyed Object or Map (associative array), instead of plain array.
What do you think about this idea?

This is an interesting idea!
If you are using Lodash, you could either use keyBy to turn the array into an object or use find to search for the correct element in the array.
The general problem is that JavaScript has a very thin standard library. The question is how much of the missing standard library we should implement in Prisma for added convenience.


quark_se is missing SPI_DRV_NAME symbol

After commit 55caa7b7434e769c5da2680a5161da8ec7caa260 SPI_1_NAME is empty for a board quark_se_c1000_devboard

@tbursztyka

The Kconfig symbol SPI_1_NAME is intentionally undefined for this board after this commit.  The preprocessor macro CONFIG_SPI_1_NAME is still defined, but now it's generated from dts instead of Kconfig.
Excerpts from tests/drivers/spi/spi_loopback/build/zephyr/include/generated/generated_dts_board.h:
/* spi@b0001400 */
#define SNPS_DESIGNWARE_SPI_B0001400_BASE_ADDRESS       0xb0001400
#define SNPS_DESIGNWARE_SPI_B0001400_IRQ_0              3
#define SNPS_DESIGNWARE_SPI_B0001400_IRQ_0_PRIORITY     2
#define SNPS_DESIGNWARE_SPI_B0001400_IRQ_0_SENSE        32768
#define SNPS_DESIGNWARE_SPI_B0001400_LABEL              "SPI_1"
#define SNPS_DESIGNWARE_SPI_B0001400_SIZE               1024

#define CONFIG_SPI_1_BASE_ADDRESS       SNPS_DESIGNWARE_SPI_B0001400_BASE_ADDRESS
#define CONFIG_SPI_1_NAME               SNPS_DESIGNWARE_SPI_B0001400_LABEL
#define CONFIG_SPI_1_IRQ                SNPS_DESIGNWARE_SPI_B0001400_IRQ_0
#define CONFIG_SPI_1_IRQ_PRI            SNPS_DESIGNWARE_SPI_B0001400_IRQ_0_PRIORITY

@MaureenHelm so should the functionality be the same as before? At the moment SPI_NAME is undefined (whatever it is called now)

You can't change SPI_1_NAME value in menuconfig anymore, but you can use CONFIG_SPI_1_NAME just as you did before:
https://github.com/zephyrproject-rtos/zephyr/blob/92080a0670a8b1b7a6808688057665cc58b9804f/drivers/spi/spi_dw.c#L594-L597
Is there a particular sample or test application that's broken?

samples/net/wpanusb is not working anymore

Ok, I see the problem now. We need to add dts bindings for the cc2520 like we did for the mcr20a.

There are a few cleanups like this we need to make, will open another issue to catch them all.