Documentation?

I'm just getting started with this, and it's looking like a great tool! I am wondering about documentation for the x-amazon-apigateway-auth and x-amazon-apigateway-integration objects that get injected into Swagger to hook up things like Lambda functions. Is there anywhere these are documented for reference? When I tried to put them into my JSON file and import into the API gateway I got lots of errors and am not sure how to proceed.
For now I'm just going to hook things up manually. This tool looks like a great start. Thanks!

+1. Would love to be notified when documentation for the extensions is published. Particularly on how to map to Request Models in the Method Request. I am thinking it is some kind of syntax with $ref:, but cannot find an example so far.
Again, awesome tool! Can't wait for the docs.

We have people working on it and hope to have this available soon. Thanks for your patience

+1 on documentation.

+1. Kind of hard right now to map all properties in x-amazon-apigateway-integration

Better documentation of the Swagger extensions would be appreciated.

Just from my own trial and error experience, I found the following to allow us to map JSON objects for POST and PUT requests:
        requestTemplates:
          application/json: "$input.json('$.event')"

Where the POST request sends a JSON object something like this:
"event": {
  "name":"whatever",
  ...
}

This works. However, I'm not convinced it is the complete solution. It does end up passing the JSON arguments with the request.
Lastly, I want to give kudos to @rpgreen for writing the tool. I am quite disappointed, however, with the lack of documentation (still, after weeks of waiting) and the level of support from a company like Amazon, for this tool, is far from adequate.
Come on guys! Get some docs put together already. We all realize there is always a critical path and time is limited, but this is getting ridiculous!

Hi folks - sorry about the delay - we are actively working on better documentation.
In the meantime, maybe you can help us focus the documentation efforts. What exactly are you trying to achieve and where are you getting stuck? What kind of examples would you like to see? Would you prefer specification-style documentation or walkthroughs? Which parts of the tool are most confusing?
As a side note, the Swagger extensions are simply an analog of the concepts in our [REST API] (http://docs.aws.amazon.com/apigateway/latest/developerguide/api-ref.html). If you haven't already, I would highly recommend reading the API docs as well as the [service documentation] (http://docs.aws.amazon.com/apigateway/latest/developerguide/api-ref.html). If you are still having issues then please let us know so we can update our core docs as well.
Another hint on getting up to speed with the importer/API is to use a browser debugger to inspect how the web console uses the API.
Again, thanks for your patience and any feedback on how to improve the documentation would be much appreciated.

Hey Ryan, for me it would be examples containing how to map values. Various requestTemplates and responseTemplatesexamples would be quite helpful.
      x-amazon-apigateway-integration:
        type: "http"
        uri: "http://50fd882d.ngrok.io/api/{version}/events"
        httpMethod: "POST"
        requestParameters:
          integration.request.path.version: "method.request.path.version"
          integration.request.querystring.user_auth: "method.request.querystring.user_auth"
        requestTemplates:
          application/json: "$input.json('$.event')" # Examples for this would be great.
        responses:
          2\d{2}:
            statusCode: "200"
          default:
            statusCode: "400"
            responseTemplates:
              application/json: "json 400 response template" # Examples for this would be great.

It took me a long time to figure out how to map the JSON for a POST, and I'm still not sure I'm doing it correctly.  Docs like this are less than helpful: http://docs.aws.amazon.com/apigateway/api-reference/resource/integration/#requestTemplates
{
  "type" : "String",
  "httpMethod" : "String",
  "uri" : "String",
  "credentials" : "String",
  "requestParameters" : {
    "String" : "String"
  },
  "requestTemplates" : {
    "String" : "String" # This doesn't help
  },
  "cacheNamespace" : "String",
  "cacheKeyParameters" : [ "String" ]
}

I've probably been looking in the wrong place.
Again, thank you for writing the tool. It will surely prove to be quite useful to my company as using the web-console for large APIs is intractable.

Any progress?

Working by example being the only way at the moment, I thought to share my
swaggerhub file.
If you're looking for a working example that uses:

API GW -> Lambda with standard exec roles
Cognito -> API GW -> Lambda with Cognito roles. (User's credentials).

Then check our API:
https://swaggerhub.com/api/koxon/api.user.sportarchive.tv/0.6
If you are using Python Lambda code,
we created a little framework to run your Python functions locally and Mock
the Lambda Event and Context variable.
It also inject a Cognito IdentityId if you want. It's all done using a
simple Makefile.
See: https://github.com/sportarchive/aws-lambda-python-local
Hope this will help
cheers
@koxon
ᐧ
On Wed, Dec 16, 2015 at 1:05 PM, Paul DeCoursey<EMAIL_ADDRESS>wrote:

Any progress?
—
Reply to this email directly or view it on GitHub
https://github.com/awslabs/aws-apigateway-importer/issues/30#issuecomment-165193269
.

--
Nicolas Menciere
Co-Founder and CTO
Sport Archive Inc.

Very nice!

Swagger/OpenAPI import is now generally available in the API Gateway REST API, the AWS CLI and all AWS SDKs. You can also import and export Swagger definitions using the API Gateway console. This release addresses many of the open issues and feedback in this repository.
I would encourage you to migrate your workflow to the standard AWS tools. aws-apigateway-importer will receive minimal support from the API Gateway team going forward.
Any feedback or issues with the new importer should be directed to the official API Gateway forums.
Thanks,
Ryan


stage swicharoo

What feature do you want to get added on the base engine?
idk if this counts as asking for stage editor if so please don't ban me I didn't know i just think that there could be a new event called "change stage" and as you probably know by now... it changes the stage to a different one just an idea
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

it would cause lag though. it probably won't happen

thats ok a little bummed but ok
On Sun, Jan 9, 2022 at 3:42 AM 4wdthinks @.***> wrote:

it would cause lag though. it probably won't happen
—
Reply to this email directly, view it on GitHub
https://github.com/ShadowMario/FNF-PsychEngine/issues/3353#issuecomment-1008263764,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUAYFDQMTCY7A476XXGL423UVFJ7RANCNFSM5LRSXTYA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
 Check it out here. only thing is that the stages used here were specifically made for the song. Basically the song has a modchart, and inside it theres a function called something like switchStage(id) or something like that i forgot
basically, it accepts numbers 1,2, 3 and 4 for the 4 different stages I used.
A lot of the code abuses the way the lua tags are handled, meaning that if you create a new lua object that has a tag that was already used, it replaces it.
I do think it would be neat to have a way to just swap out stages, but it would probably cause a lot of issues.
I'd show you the code but I am not at my computer at the moment. Sorry about that!


Same participants in different sites

Hi,
I get the participants_v1.0.0.zip from NDA (https://nda.nih.gov/edit_collection.html?id=3165). After comparing it with the sites info provided by ABCD package, seem some participants with the wrong sites info for baseline measurement.
for example,
"sub-NDARINVXXXXXXXX"	"ses-baselineYear1Arm1"	"1"	"site17"	....
"sub-NDARINVXXXXXXXX"	"ses-baselineYear1Arm1"	"1"	"site11"	....
(this is from participants.tsv)
NDAR_INVXXXXXXXX,site11,... (this subject have multiple sites at the follow up measurement)
(this is from ABCD)

Hi @Jinboasltw - thanks so much for bringing this to our attention. I'm going to delete the subID information from this post because it is PHI for this study. I have assigned some folks from our lab, and we will get back to you as soon as possible with answers/actions.
Thanks! Happy Holidays!

Thanks. Sorry for ignoring the risk to show the full ID in the post. I just want to give some direct examples.

We need to verify that this information actually has been updated. If not, it should be updated in the new participants.tsv we are uploading soon.

Hi @Jinboasltw, we just uploaded the new participants.tsv to the collection on the NDA. This version should include the necessary fixes you had outlined in this issue. If you continue to encounter the issue though, please reopen this and we will get it addressed. Thanks for your patience!


This mock library is not generics ready

Hi, I'm looking for a mock library that can work perfect with generics. mockigo have some issues in this case:
$ cat sample.go
package example

import context

//go:generate mockigo --root-dir . --mocks-dir .

type Publisher[M any] interface {
	Publish(ctx context.Context, msg M)
}

It outputs with error:
2022/11/12 21:47:45 ERROR: load package in dir /Users/User/Desktop/mockigot: [sample.go:3:15: expected 'STRING', found newline /Users/User/Desktop/mockigot/sample.go:3:15: expected 'STRING', found newline /Users/User/Desktop/mockigot/sample.go:7:1: expected ';', found 'type' /Users/User/Desktop/mockigot/sample.go:3:15: invalid import path (invalid syntax) /Users/User/Desktop/mockigot/sample.go:8:14: undeclared name: context]

Hi. Sorry for the late response. Seems like you have an invalid import - it should be import "context" instead of import context


⚠ Bofrost.com has degraded performance

In a981857, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 2103 ms

Resolved: Bofrost.com performance has improved in c29aa23.


Add an SDK key to variants that are made with meltano_sdk

Migrated from GitLab: https://gitlab.com/meltano/hub/-/issues/124
Originally created by @tayloramurphy on 2021-10-01 21:06:25

We need to add a key to each variant meltano_sdk: true for each variant that was made with the SDK. This way we can visualize it on each page - "Made with SDK!"

View 3 previous comments from the original issue on GitLab


Change in UI of project skills of "Add new project"

Description
I changed the UI of project skills input of  '' Add new Project" section. Now take input with suggestions of famous Tech and input map with chips.
Fixes #39
Type of Change:

New Feature

[ ] My PR follows the style guidelines of this project

[ ] I have performed a self-review of my own code or materials

[ ] I have commented my code or provided relevant documentation, particularly in hard-to-understand areas

[ ] My changes generate no new warnings

[ ] New and existing unit tests pass locally with my changes

hey @Veeresh-Rex bro, there are some conflicts with your PR can you please check and resolve them.


Treat punctuation as hashtag delimiters

Unicode ranges for punctuation are simpler than creating a Unicode-aware word class, so delimit on non-words.
Fixes #4398

Thanks @mojodna!   Looks pretty good.. I merged it and then made 2 other little changes in 8f82caf55:

I removed the _, so hashtags can have underscores in them
I changed the quantifier from * to + so it would match 1 or more instead of 0 or more.  The original would detect a single lonely "#" character, even in cases where the hashtags string was empty.

Thanks. I was being spacey when I put that together; too much of a hurry


universal error

[nodemon] starting `node dist/server/index.js`
Listening on: http://<IP_ADDRESS>:8000
EXCEPTION: document is not defined
ORIGINAL STACKTRACE:
ReferenceError: document is not defined
 at new RippleRenderer (<...>\angular-webpack2-starter-master\node_modules\@angular\material\material.umd.js:193:31)

https://github.com/angular/material2/issues/1415#issuecomment-251730203

You'd have to use material prior 8 for it to work right now, seems compatibility has been broken since version 9. Hopefully they will patch that soon.


chore: Conditionally compile helper function for failpoints

What is changed and how it works?

Issue Number: Ref #15990

What's Changed:
Conditionally compile the helper function that only got called in fail-point tests to suppress the warning.

Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[x] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

None

/retest

/retest

/retest


Ability to control the UID and GID of container user

Feature request
Some of us like to run our containers so that the user inside the container uses a UID and GID of a user on the host. To give a specific example:
On the host:
ls -alth
drwxr-xr-x 11 someuser somegroup 11 Jan  3 00:16 somedir

And then in the docker-compose file, you'd specify a UID and GID pair, for example on the LinuxServer images you specify it as environment variables (see PUID and PGID in this example repo: https://hub.docker.com/r/linuxserver/jellyfin).
This then has the effect of the user that does r/w operations to the mounted dirs starting with a specific UID and GID, which then match a UID and GID on the host, and therefore has the same level of access as the user on the host.
You can of course specify that the container itself runs as a specific user, but it seems that Uptime Kuma creates a user called Node, and there's no control that I can see that allows you to control its UID/GID.
It would be really nice if the container allowed for that!
📝 Additional Context
I'd be happy to take a look at implementing this for you, assuming you're interested :) Thank you for your time.

Upon some further reading, I have just discovered your rootless images. This might make this a non issue for me - but this might be worth looking into either way. The LSIO images are probably a great example to work from.

See https://github.com/louislam/uptime-kuma/wiki/Environment-Variables#docker-specific-environment-variables

@CommanderStorm does your docker entrypoint actually do anything with those env vars? I can't test this myself right now but these env vars do nothing except for exist in your container - where you can then use them however you'd wish. Docker doesn't afaik do anything with them by default.

As an example, in the LSIO images, they all use a base image that has the following script being executed during its build:
https://github.com/linuxserver/docker-baseimage-ubuntu/blob/jammy/root/etc/s6-overlay/s6-rc.d/init-adduser/run - in there you can see the PGID and PUID being used to create a user within the container. I could just be being blind, but I do not see anything similar for Uptime Kumas Docker image.


MPS support

Context

For testing purposes it can be useful to run directly on a local Mac computer.

Changelog

Checks support for BF16 on MPS device.
Added a configuration targeting MPS, changes to path were required due to the way Mac resolves symlinks in /tmp as /private/ActualPath.
Set optimizer to Adam instead of AdamW to fit in memory on 64GB devices.

Test plan

Ran a training job on Mistral 7b full finetune.
The current test jobs are very CUDA specific, maybe this could be changed as well?
We may need to integrate Mac runners in the pipeline.
Some dependencies such as bitsandbytes are not yet Mac compatible.

@maximegmd This is awesome! Can you post some loss curves for the finetune you ran?

@maximegmd This is awesome! Can you post some loss curves for the finetune you ran?

I will complete a run during the weekend, losses looked fine but the Llama3 release changed my priorities ^^

device is not None when this function is called so it just passes 'mps' to torch.device() which is the expected pytorch name.
But you are correct that there is room for improvement to automatically return mps when device is not manually specified in the config.

Oh good point, totally glossed over the fact the device = torch.device(device) outside the if block. Yup sounds good. What's the iter/sec you're getting with this on a mac?

If I recall correctly it was around 20s/it but I suspect I was swapping a bit so I can probably improve the speed. The main issue is bitsandbytes not supporting MPS so it uses quite a bit of memory for the optimizer state.
I will try to push a llama3 config tomorrow with some numbers now that my llama3 finetune is running :)

Here is a train run on Gemma 2B, sadly laptop went to sleep right before the end but this is a 14 hours run, should be representative enough.
log_1715964526.txt

The goal is to merge indeed, but considering the issue raised by @SalmanMohammadi maybe we should off and lock to pytorch 2.4+ ?

bump bump.
Is this still planned @maximegmd? If not, happy to pick it up. It's been invaluable to me so far, so would be happy to other users be able to get up and running without needing big (medium?) GPUs.

This has been open without changes for 6 weeks. Can we merge this in @joecummings?

This just needs to be updated to use the latest utiles functions, I will do it tomorrow, then it should be ready to merge unless I am not seeing something

This just needs to be updated to use the latest util functions, I will do it tomorrow, then it should be ready to merge unless I am not seeing something

Nice! Thanks so much. One small thing I've experienced with the recent torchao dependency version upgrade, is that pip install torchao==0.3.1 doesn't find any suitable builds. I've gotten around this by installing from source (or pip install --no-cache-dir<EMAIL_ADDRESS>- if you error out here make sure you've install setuptools and wheel.
If this sounds like a sensible way around this, perhaps it's worth adding to the README?

Nice! Thanks so much. One small thing I've experienced with the recent torchao dependency version upgrade, is that pip install torchao==0.3.1 doesn't find any suitable builds. I've gotten around this by installing from source (or pip install --no-cache-dir<EMAIL_ADDRESS>- if you error out here make sure you've installedsetuptools and wheel.

I assume this is specifically on MPS? cc @msaroufim for thoughts here

I assume this is specifically on MPS? cc @msaroufim for thoughts here

Yeah just on my machine, it's been working fine on Linux.

Can you update the readme with an example for MPS?

Was on PTO but torchao as of 0.3.1 is still a linux only library, mac binaries are possible just not high priority right now

Codecov Report
Attention: Patch coverage is 70.04812% with 747 lines in your changes missing coverage. Please review.

Project coverage is 68.57%. Comparing base (cb8e65a) to head (eb36f7b).
Report is 83 commits behind head on main.

Files
Patch %
Lines

recipes/qat_distributed.py
0.00%
212 Missing :warning:

tests/torchtune/utils/test_distributed.py
12.16%
130 Missing :warning:

tests/recipes/test_lora_finetune_fsdp2.py
31.86%
62 Missing :warning:

recipes/lora_finetune_distributed.py
0.00%
36 Missing :warning:

recipes/lora_finetune_single_device.py
0.00%
35 Missing :warning:

recipes/eleuther_eval.py
0.00%
27 Missing :warning:

recipes/generate.py
0.00%
20 Missing :warning:

tests/recipes/test_lora_finetune_distributed.py
20.83%
19 Missing :warning:

tests/recipes/test_qat_distributed.py
48.57%
18 Missing :warning:

tests/recipes/test_lora_finetune_single_device.py
22.72%
17 Missing :warning:

... and 35 more

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #790      +/-   ##
==========================================
+ Coverage   67.10%   68.57%   +1.47%
==========================================
  Files         174      214      +40
  Lines        7423     9675    +2252
==========================================
+ Hits         4981     6635    +1654
- Misses       2442     3040     +598

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

checking in again to see what the plan is on merging. Still haven't seen any movement. Would love to see mps make it to main.

Closed due to #1706. For anyone following along, feel free to try out torchtune on MPS and let us know how you get along!


Solver is very slow when using AddPickupAndDelivery under specific conditions

Hi,
I encounter an issue using the RoutingModel with AddPickupAndDelivery. When a node is specified twice as a delivery node, the solver takes a lot of time to find a solution. It seems to grow exponentially with the size of the problem.
The following program runs in ~65 seconds on my machine, with only 13 nodes.
This is a simplified version of the original problem I had where the solver would seem to hang forever.
Language: C#
Version: 5.1.4048

class Program
{
    static void Main(string[] args)
    {
        var routing = new RoutingModel(13, 1, 0);

        routing.AddPickupAndDelivery(0, 1);
        routing.AddPickupAndDelivery(0, 2);
        routing.AddPickupAndDelivery(3, 4);
        routing.AddPickupAndDelivery(3, 5);
        routing.AddPickupAndDelivery(3, 6);
        routing.AddPickupAndDelivery(3, 7);
        routing.AddPickupAndDelivery(3, 8);
        routing.AddPickupAndDelivery(3, 9);
        routing.AddPickupAndDelivery(3, 10);
        routing.AddPickupAndDelivery(11, 10); // Here we use node 10 a second time
        routing.AddPickupAndDelivery(11, 12);

        var parameters = RoutingModel.DefaultSearchParameters();
        parameters.FirstSolutionStrategy = FirstSolutionStrategy.Types.Value.PathCheapestArc;

        using (var nodeEvaluator = new DistanceEvaluator())
        {
            routing.SetCost(nodeEvaluator);

            var solution = routing.SolveWithParameters(parameters);

            for (long index = routing.Start(0); !routing.IsEnd(index); index = solution.Value(routing.NextVar(index)))
            {
                Console.WriteLine(index);
            }
        }
        Console.ReadLine();
    }

    public class DistanceEvaluator : NodeEvaluator2
    {
        public override long Run(int i, int j)
        {
            return 1000 + i - j;
        }
    }
}

Changing the first solution heuristics does not improve the solving time. The problem disappears if all delivery nodes are used only once.
Is this a bug or am I missing some constraints?
Thanks,
Vincent

We fixed this issue by never reusing a node. If a driver has to go multiple times to the same location, we just create multiple nodes with a distance of 0.


Remove duplicate code and extra memory allocations

I also did split a lot of raw command calls into separate functions to make them better readable and also reduce the number of strings used in the code.
Almost all functions with String type arguments did not use const String reference.
This means every call does perform a deepcopy (and thus lots of memory re-allocations)
The library as it is in this PR is also being used now in ESPeasy.
See https://github.com/letscontrolit/ESPEasy/pull/2539
Later on, I will change it into an async version so the controller does not have to wait for the packet to be sent or wait for a long time for an acknowledgement if asked for.
But such a change will be a breaking change in how to interface with the library.
When that's ready, I will again make a pull request.

Ahh excellent. Thanks a lot for this impressive improvement!
Regarding the blocking vs. async version, perhaps the async one should be a fork. For beginners in the embedded (Arduino) world blocking functions are much easier to understand and use. The goal of this library would rather be to keep things simple rather and super efficient. But async would also make sense for a next step in learning.
(Such a pity github doesn't pick up the diff correctly. Must be due to indentation changes.)

You can change the view to ignore white space changes.

Thanks again. You're teaching me (us) a lot here. Not only Github features but also your good coding style.

Thanks.
Just wanted to let you know I also changed this behavior as I think it may have been a bug.
Please have a look at it:

Yes I believe you are right that there was a bug. Default EU frequency plan should only have the first three channels enabled on their default frequencies.
As a side not I see you changed to for-loops to configure the channels. The TTN library also does this. Personally I prefer not to wrap this in loops as writing them out line by line is much more readable.
Example:
setChannelEnabled(0, true);
setChannelEnabled(1, true);
setChannelEnabled(2, true);
setChannelEnabled(3, false);
setChannelEnabled(4, false);
setChannelEnabled(5, false);
setChannelEnabled(6, false);
setChannelEnabled(7, false);

versus
for(int channel=3; channel<8; channel++) {
  if(channel<3) {
    setChannelEnabled(channel, true);
  } else {
    setChannelEnabled(channel, false);
  }
}

But then again the for-loops likely use less program space, and because normal users won't look at the library source the space gained is worth the readability trade-off. Is this however true? Doesn't the overhead of the for-loop and if-checks require more space than the unwrapped loop - with this number of loops?
https://en.wikipedia.org/wiki/Loop_unrolling

I don't know what threshold the compiler uses to use loop unrolling.
I can imagine it also is highly depending on the platform whether it is a speed optimization or not.
The ESP doesn't have a long pipeline, nor branch prediction, so the gain of loop unrolling may be close to 0.
By the way your "versus" example has a bug in it ;) (starting at 3, so first if will never be true)
Just checked to be sure, but it is not a cut'n-paste of my changes ;)

Haha no, those code fragments were a quick rewrite by me taking your screenshot as example.


ctr: add snapshot prune to remove useless snapshot

What is the problem you're trying to solve
Restarting containerd while downloading image may cause some committed snapshot of the image leaks. It happends on when we restart containerd exactly unpacking a image. If the image never be pulled again, the snapshot will stay on the node forever.
Describe the solution you'd like
Provide a way to clean up the useless snapshot, such as ctr snapshot prune.
Additional context
It seems to have no way to clean up the committed snapshot if not a parent. Should we provide such an interface ?

When an in-progress pull is interrupted by the process being killed, the lease being used does not have an opportunity to be cleaned up by the pull operation. By default, leases are created with a 24 hour expiration though to prevent the leaking of temporary resources. In the case you mentioned, the garbage collector is responsible for cleaning up the lease after the expiration. To clean it up sooner, the lease may be deleted manually. Snapshots must always be associated with an image, container, or lease, otherwise they are removed during garbage collection. If containerd sits idle after the expiration, the garbage collection will not run though since we do not have any expiration triggers or time based scheduling in the garbage collector.

@dmcgowan Thanks for the reply. It's my fault. I didn’t understand how garbage collector works before. I've read it again. Thanks.


X Windows must not be enabled unless required.

[#98978178]
Signed-off-by: Zhou Xing<EMAIL_ADDRESS>

@zhang-hua instead of configuring files let's just check that x server is not installed.

@cppforlife , okay! will update it.


Make command to install the package in editable mode

This will allow for a more iterative approach when performing the developer loop of:

change the code
test it
repeat

For more information about development mode please visit:

https://setuptools.pypa.io/en/latest/userguide/development_mode.html

Resolved in https://github.com/kraken-tech/django-pg-migration-tools/pull/38


Fails to build

This fails the build, first I got a temporary error:
Step 5/15 : RUN apt-get -qq update
---> Running in 805041adc74e
W: Failed to fetch http://deb.debian.org/debian/dists/jessie/InRelease
W: Failed to fetch http://security.debian.org/debian-security/dists/jessie/updates/InRelease
W: Failed to fetch http://deb.debian.org/debian/dists/jessie-updates/InRelease
W: Failed to fetch http://security.debian.org/debian-security/dists/jessie/updates/Release.gpg  Temporary failure resolving 'security.debian.org'
W: Failed to fetch http://deb.debian.org/debian/dists/jessie/Release.gpg  Temporary failure resolving 'deb.debian.org'
W: Failed to fetch http://deb.debian.org/debian/dists/jessie-updates/Release.gpg  Temporary failure resolving 'deb.debian.org'
W: Some index files failed to download. They have been ignored, or old ones used instead.
Then after running again:
Step 6/15 : RUN apt-get -qq install locales krb5-kdc krb5-admin-server
---> Running in 130f444fe953
E: Package 'locales' has no installation candidate
E: Unable to locate package krb5-kdc
E: Unable to locate package krb5-admin-server
ERROR: Service 'kdc-kadmin' failed to build: The command '/bin/sh -c apt-get -qq install locales krb5-kdc krb5-admin-server' returned a non-zero code: 100

apologies, this was a local docker setup issue where it didn't resolve properly. Might still be good to move away from jessie though, since it's old.


(GH-217) Template based release notes creation

Description
This PR addresses #217.
Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report

Merging #283 into develop will increase coverage by 0.75%.
The diff coverage is 86.66%.

@@             Coverage Diff             @@
##           develop     #283      +/-   ##
===========================================
+ Coverage    77.34%   78.10%   +0.75%
===========================================
  Files           54       54
  Lines         1130     1137       +7
  Branches       135      128       -7
===========================================
+ Hits           874      888      +14
+ Misses         235      228       -7
  Partials        21       21

Impacted Files
Coverage Δ

...seManager.Core/ReleaseNotes/ReleaseNotesBuilder.cs
90.56% <86.36%> (+10.56%)
:arrow_up:

src/GitReleaseManager.Core/VcsService.cs
70.91% <100.00%> (ø)

...anager.Core/Configuration/ConfigurationProvider.cs
45.28% <0.00%> (-1.53%)
:arrow_down:

Continue to review full report at Codecov.

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

@gep13 @AdmiringWorm In the first step I added the Scriban library to have a template based release notes creation. If you agree with the changes, I would add in next step a -template option to the Create command and update the docs.
A question is how should it be handeled when the -template is set but the file is not found?

Show a warning and use the default template
Or throw an exception?

@akordowski I would suggest that if the user has asked to use a template, and that template file doesn't exist, then this should be treated as an error, and we should throw an exception, indicating that the required file couldn't be found.

@akordowski if the template file is empty, then this to me is also an exception, and we should throw an error.

@gep13 I made the changes as discussed. The PR is ready for review.

@akordowski this is looking really good!
One thing though, would it be possible for you to squash some of your commits down into a set of logical commits?  We don't really need additional commits like "Fixed Unit Test".  Thanks

@gep13 I made the changes as requested.

I believe the build failure here, after the rebase, isn't related, it is an issue with codecov.

@akordowski thanks again for getting this done, really appreciate it!


Roll engine ff49ca1c6e5b..58f4f27a4e55 (11 commits)

git@github.com:flutter/engine.git/compare/ff49ca1c6e5b...58f4f27a4e55
git log ff49ca1c6e5b..58f4f27a4e55 --no-merges --oneline
2019-08-09<EMAIL_ADDRESS>change add part to add child (flutter/engine#10787)
2019-08-09<EMAIL_ADDRESS>rename stub_ui to web_ui (flutter/engine#10776)
2019-08-09<EMAIL_ADDRESS>[fuchsia] Migrate from custom FuchsiaFontManager to SkFontMgr_fuchsia (flutter/engine#10700)
2019-08-09<EMAIL_ADDRESS>Roll src/third_party/skia 26cedb4c603b..b7b0b3ad538c (11 commits) (flutter/engine#10807)
2019-08-09<EMAIL_ADDRESS>Rename flutter_java.jar and flutter_engine.jar so they match the Maven comvention (flutter/engine#10797)
2019-08-09<EMAIL_ADDRESS>Roll fuchsia/sdk/core/linux-amd64 from 9xVXx... to HsuAH... (flutter/engine#10806)
2019-08-09<EMAIL_ADDRESS>Roll src/third_party/dart 3d9a356f6e..78ce916d82 (7 commits)
2019-08-09<EMAIL_ADDRESS>Roll src/third_party/skia b2151310068b..26cedb4c603b (1 commits) (flutter/engine#10804)
2019-08-09<EMAIL_ADDRESS>Roll src/third_party/skia 059bf776c184..b2151310068b (3 commits) (flutter/engine#10803)
2019-08-09<EMAIL_ADDRESS>Roll src/third_party/dart f29f41f1a5..3d9a356f6e (65 commits)
2019-08-09<EMAIL_ADDRESS>Roll src/third_party/skia 4d557e3df433..059bf776c184 (1 commits) (flutter/engine#10801)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (jimgraham@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/6455113328099328

Commit queue failed; closing this roll.


violates the following Content Security Policy directive when i use a Web Socket "Custom Nodes" option

✅ Prerequisites
const customNodeOptions = {
rpcUrl: 'wss://rpc-mainnet.maticvigil.com/ws',
chainId: 137
}
🐛 Description
Refused to connect to 'wss://rpc-mainnet.maticvigil.com/ws' because it violates the following Content Security Policy directive: "connect-src 'self' https://.magic.link/ https://.fortmatic.com/ https://.alchemyapi.io/ wss://.ws.alchemyapi.io/ https://.infura.io/ https://.xdai.quiknode.pro ...
🧩 Steps to Reproduce
Subscribe to an event with the web3 magic provider configured as above.
🌎 Environment

Software
Version(s)

magic-sdk
"magic-sdk": "^4.2.1"

The doc does not describe any restriction on WS RPC
https://docs.magic.link/blockchains/ethereum#configure-custom-nodes
But it seems you need whitelists one or more of this "official" web socket RPC provider
wss://rpc-mainnet.maticvigil.com/ws or
wss://rpc-mainnet.matic.quiknode.pro or
wss://ws-matic-mainnet.chainstacklabs.com or
wss://matic-mainnet-full-ws.bwarelabs.com or
wss://matic-mainnet-archive-ws.bwarelabs.comor
wss://ws-mainnet.matic.network
Thank you

Our team will need to add this node to our CSP. Someone will ping this thread when it's complete.

@vincentlg Those URLs listed have been added

The CSP has been updated on our side. Please let us know if this is resolved by closing this issue 🙂

Thank you for your intervention, unfortunately I guess a problem persists on your side
Access to XMLHttpRequest at 'wss://rpc-mainnet.maticvigil.com/ws' from origin 'https://auth.magic.link' has been blocked by CORS policy: Cross origin requests are only supported for protocol schemes: http, data, chrome, chrome-extension, chrome-untrusted, https.

Looks like this new error is related to our CORS setup, will forward this to our infrastructure team and have them take a look.

Good evening
I'm having the same problem when connecting to my custom node
const customNodeOptions = {
rpcUrl: 'http://<IP_ADDRESS>:8545',
chainId: 10
}
But having this error on console.
Content Security Policy: The page’s settings blocked the loading of a resource at http://<IP_ADDRESS>:8545/ (“connect-src”).

Looks like this new error is related to our CORS setup, will forward this to our infrastructure team and have them take a look.

👀 Any news from the infrastructure team ? :)

👀 Any news from the infrastructure team ? :)

Here's the current status: we don't have support for Web3 WebSocket subscriptions at this time. It's on our roadmap but I don't have an estimated release date for you.

Good evening
I'm having the same problem when connecting to my custom node
const customNodeOptions = {
rpcUrl: 'http://<IP_ADDRESS>:8545',
chainId: 10
}
But having this error on console.
Content Security Policy: The page’s settings blocked the loading of a resource at http://<IP_ADDRESS>:8545/ (“connect-src”).

@Genzan We allow localhost origins for testing custom node configurations. You can work around this by mapping that IP address to localhost in your etc/hosts file.

Good evening
I'm having the same problem when connecting to my custom node
const customNodeOptions = {
rpcUrl: 'http://<IP_ADDRESS>:8545',
chainId: 10
}
But having this error on console.
Content Security Policy: The page’s settings blocked the loading of a resource at http://<IP_ADDRESS>:8545/ (“connect-src”).

@Genzan We allow localhost origins for testing custom node configurations. You can work around this by mapping that IP address to localhost in your etc/hosts file.

Im using the react example of yours as base to do these test (https://magic.link/posts/magic-react-express), also the ip <IP_ADDRESS> is a server diferent of my local computer. Will read about these host file as I have never work with one before, and tell you the results

Already changed my Host file, but same problem persist

Already changed my Host file, but same problem persist

You must also configure your Magic instance with a localhost origin given to the network.rpcUrl field. Unfortunately, that parameter cannot be an IP address as a matter of security.

@Genzan Please let me know if your concern has been resolved. If so, feel free to close the issue.

We are currently working on getting a domain so I can pass to you, if you could give me a mail where I can summit it once it has been compleated I would be Happy to do so

@Genzan When your team is ready with a domain, go to https://magic.link and click the chat bubble at the bottom-right corner. You can communicate with our support team there!

Closing this for now! We're interested in exploring official WebSockets support in the future, but for now it's not an immediate priority.

Hi already tried to talk via chat and email but they told me they were unable because of guidance of your security team
Any suggestions so we can advance in testing a Private Network Blockchain using MagicLink?

Hi already tried to talk via chat and email but they told me they were unable because of guidance of your security team
Any suggestions so we can advance in testing a Private Network Blockchain using MagicLink?

At the time being, testing with a private chain infrastructure is not possible. However, we are looking at options internally that would remove this security requirement by emitting request to the node from our backend instead of client-side. Will keep you posted on the progress.


"Holding specific key pattern on which_key" example is broken.

Description
The "Holding specific key pattern on which_key" example is broken. name doesn't exist, unpack needs parens, and even with a fixing that, nothing happens given "".

I fixed the README. Thank you.


Create Turn-Based System for Gameplay

Description

The game is turn-based (as opposed to real-time), where all turns begin with an initiative roll, then taking an action based on the board situation current at each character's turn. Each action is limited by the movement/attack ranges of the character and the chosen weapon, taking into consideration obstacles and map configuration.

@wokidoo Assigning this to you as I feel this is a good fit regarding your DnD knowledge in this project

Note that in the Game class there now exists a EndTurn method that can be used to implement turns as outlined in the issue

Completed at the time of submitting the intermediate deliverable


Method of defining tags or keyword

What are the schemas that are affected by the issue
https://ns.adobe.com/xdm/assets/asset
but might be its own schema or impact other schema.
What are examples of products that are impacted by the issue
AEM and other products that store content that needs to be tagged.
While the assets schema has xmp:keywords and xmp:machineKeywords which might be used to tag an asset with a keyword, there is no XDM way of describing a keyword or organising a keyword into an ontology of tags, even if that ontology is flat.
Many products that store content organise that content using tags and organise those tags into an ontology. Some examples:
Example 1.
In a DAM, Photographs may be tagged with "cars" which would be organised as a narrower term for "vehicles" and cards would contain narrower terms of "performance", "family".
Example 2.
In a Drugs dictionary, Documents containing medical advice for prescription drugs may be tagged with the family of drug and the families of drugs that cause adverse interactions, hence a patient taking Asprin, classified as a salicylate and a nonsteroidal anti-inflammatory drug would alerted that it has a serious interaction with Ketorolac, by virtue of Aspirin being a NSAIDs. Each tag would be defined in a managed ontology.
Example 3.
In an audience targeting platform, the segments into which browsers are placed would be related to one another.
The structure of defined tags or keywords needs to support both hierarchical (DAG) and non DAG type graphs, although most uses are probably for hierarchical relationships rather than multi faceted relationships between the keyword or tag definitions.
The ask is for
a) A schema to define the tags and keywords
b) a mechanism to define the relationships between those definitions.

Use of tags from an ontology (or not) and how tags are organized (DAG or not) is out of scope.  IMO, that's an application level consideration.  XDM and ACP are only concerned with providing the CRUD operations (and associated storage/APIs) for the data - it's up to each application to consider how to use them.

I’d say that managed ontologies are definitely a concern that spans more than just one product and would be open to including it in XDM, even more so if we can rely on an existing standard that we just model in JSON.

I would object to that @trieloff.   If a group of application wish to define such ontologies of tags, that's fine to.  But should have no bearing on the model for simply storing and managing the values.

One of the reasons for raising this is because apparently unconnected applications need to interoperate and it is wrong to assume that all applications fall into neat groups where every application knows everything about its peers and so can privately agree on a internal schema.
AFAIK, there are plenty of standards in XML/RFD . (eg OWL etc) for this area, but none are expressed in json or json-schema. If there is a standard then getting all applications that need to deal with ontologies to use it resolves this issue. If not, something needs to be done.
AEM could easily create its own standard and leave it to other applications to follow, but I do not believe is the behaviour of a good citizen in an XDM enabled world.

Hi @lrosenthol, I agree that we cannot change the semantics of xmp:keywords or xmp:machineKeywords after the fact.
However, the ability to manage, exchange, and collaborate on taxonomies and managed vocabularies is something that is useful for the discipline of asset management as a whole and already implemented in our products (in currently incompatible ways)

Nested keywords in Bridge
Tags in AEM
Keywords in Adobe Stock

So, let's model the tags or keywords themselves as suggested by @ieb, maybe using something like JSKOS, and then find a way of non-destructively referring to them from content or asset metadata.

+1, the core of JSKOS is compatible with https://www.w3.org/TR/skos-reference/ and the examples at the end of the JSKOS spec cover the use cases mentioned so far.

I would be in favor of having a common definition of taxonomies / managed vocabularies in the platform, at least to a certain extent. The current situation that we have is de facto leaving that to applications, and that naturally leads to each application to define and use its own -- so if we continue that way, we're guaranteed to have this inconsistencies even within Adobe own solutions / applications.
To add to your list @trieloff, we also have (just from the top of my head):

keywords in CC desktop applications (PS, AI, ID, etc) in the File Info dialog - I believe these are compatible with Bridge's keywords
Smart Tags in AEM

There is of course a fine line of what should be defined on the platform level, and what specific applications could extend.

@trieloff you wrote:

taxonomies and managed vocabularies is something that is useful for the discipline of asset management

And that's my problem here.  You folks are so used to building and running an asset management system (aka AEM and all its children) that you miss the larger picture of ACP & XDM.  Both are used for numerous applications that have nothing to do with asset management.  Yes, we have assets, but they aren't being managed (in the traditional/formal sense).   So bringing in anything formal, such as forced ontologies doesn't fit.

@lrosenthol I don't want to force the use of managed ontologies on anyone, but I want to enable the use of them.
So yes, some applications built on top of XDM will be asset management systems, with asset management requirements.
Some applications will be around content management, and will also be able to benefit from managed taxonomies.
Most applications, however won't and won't care that taxonomies are included in XDM, as long as they don't have to use them. I wouldn't accept anything that makes taxonomies required, but I can't see why I should reject something that makes them optional, when they are useful.
After all, we have a geo object defined in XDM, and most applications couldn't care less about the geo-location of anything. Even some that do, simply treat geolocations as lat/lon pairs, and that's fine, too.
To follow that analogy: if the PR for geo would have changed the semantics of exif:gpsLatitude or exif:gpsLongitude, it should have been rejected.

@gklebus wrote:

keywords in CC desktop applications (PS, AI, ID, etc) in the File Info dialog - I believe these are compatible with Bridge's keywords

These use xmp:keywords.

Smart Tags in AEM

This should be xmp:machineKeywords

As @lrosenthol pointed out, we cannot simply change the meaning of either xmp:keywords or xmp:machineKeywords to point to a list of terms in a managed taxonomy. This means if we introduce a way of linking to a managed taxonomy, it needs to manifest itself inside the asset with a different property.

As long as it's not mandatory - I fully support having a way to link to one and/or use it appropriately.

Keywords don't live in xmp:keywords. They live in the Dublin Core Subject dc:subject AND in the Lightroom Hierarchical Subject lr:hierarchicalSubject nodes. Bridge and Lightroom understand that these two nodes define the lowest level keyword AND the hierarchy in which the keyword resides. AEM does not understand this multi-node metadata behavior.
AEM Tags, however, live in cq:tags. In order to support AEM Tags, which are stored in cq:tags and use a flattened hierarchy namespace:topLevelProperty/nextLevelProperty/.../lastLevelProperty, we had to build a new interface in Bridge that allowed us to define the URI and property we wanted to display, and also how to decode their hierarchy. We also had to ensure that when a user changed a property that it got written back to the asset in the same format, so as not to break how AEM managed its tags.
But, what is a keyword, anyway? A seemingly simple idea like Keywords has been made complex by Adobe (and others) leveraging existing structures for their own purpose. Why did we use Dublin Core Subject for Keywords? I don't know, but it is something we're stuck with now. Customers co-opt existing metadata fields from multiple schemas for their own purposes. Many customers embed JSON into the description, for instance, to embed rich process metadata in the asset without developing a new schema. Since description is just a text field, it's suited for this new application. @trieloff to say that we can't change the meaning of a field is not realistic, as customers do it all the time. If the field is proprietary to Adobe and Adobe's asset management and marketing automation solutions, then be sure to make that very clear to all end users who might be tempted to repurpose it for something else that fits THEIR workflow or backend business process.
I agree with @lrosenthol that the specification should define how to store and retrieve data. The consuming application will need to make sense of the data and express it in a meaningful way to the end user. My team is working on this challenge now, for a general purpose metadata panel for CC apps. We will allow the user to specify ANY property (from URI:property) and how it should be rendered (is it a menu, or a text field, or a date picker, or are there specific constrained values etc). In FileInfo, you can specify a schema and a view separately, to tell FileInfo what to expect and also how to render it.
If the intent is to also define and constrain the data structures that the new model supports, as @trieloff seems to suggest, then I'm OK with that, so long as we are very clear with all consumers that breaking the data structure will have potentially implications. Bridge, Lightroom and FileInfo already support read-only fields, and there's a lot of "hidden" metadata they won't show without UI customization.
I'm curious: how do you expect to support a hierarchical data structure? Like AEM does (with one string and separators for the namespace and hierarchical data) or with two or more related fields (like Bridge and Lightroom do with dc:subject and lr:hierarchicalSubject)? JSON is great, because we can define complex data structures within the field and include a method for defining relationships between properties. Think of how Photoshop currently tracks its the sources for a document in:
        <photoshop:DocumentAncestors>
            <rdf:Bag>
               <rdf:li>xmp.did:24dac245-94e2-4322-8ab9-8f0921491bc3</rdf:li>
            </rdf:Bag>
         </photoshop:DocumentAncestors>

or Apple tracks faces in:
            <mwg-rs:RegionList>
               <rdf:Seq>
                  <rdf:li rdf:parseType="Resource">
                     <mwg-rs:Type>Face</mwg-rs:Type>
                     <mwg-rs:Area rdf:parseType="Resource">
                        <stArea:y>0.31899999999999995</stArea:y>
                        <stArea:w>0.05099999999999999</stArea:w>
                        <stArea:x>0.34150000000000003</stArea:x>
                        <stArea:h>0.068000000000000005</stArea:h>
                        <stArea:unit>normalized</stArea:unit>
                     </mwg-rs:Area>
                     <mwg-rs:Extensions rdf:parseType="Resource">
                        <apple-fi:AngleInfoYaw>0</apple-fi:AngleInfoYaw>
                        <apple-fi:AngleInfoRoll>0</apple-fi:AngleInfoRoll>
                        <apple-fi:ConfidenceLevel>1000</apple-fi:ConfidenceLevel>
                        <apple-fi:Timestamp>2147483647</apple-fi:Timestamp>
                        <apple-fi:FaceID>2</apple-fi:FaceID>
                     </mwg-rs:Extensions>
                  </rdf:li>
               </rdf:Seq>
            </mwg-rs:RegionList>

We could probably make this simpler for the kinds of marketing data you want to store with this new mechanism. Are you planning to extend XMP or provide a completely new model for on-asset metadata storage, or is this intended only for a DAM or other asset management system?

As always, great stuff @jameslockman.
There are no plans to extend XMP here - though we are working on a standardized transform from XMP->JSON-LD (and vice-versa) that will certainly find a place in our implementations (and potentially our APIs).
The issue here is more about the system storage and not the in-asset storage - and how we do so...

Keywords don't live in xmp:keywords. They live in the Dublin Core Subject dc:subject AND in the Lightroom Hierarchical Subject lr:hierarchicalSubject nodes. Bridge and Lightroom understand that these two nodes define the lowest level keyword AND the hierarchy in which the keyword resides. AEM does not understand this multi-node metadata behavior.

I forgot about dc:subject and we do not have lr: hierarchicalSubject defined. @lrosenthol should we?

dc:subject - yes, because it's used all over our apps.  lr:hierachicalSubject is a problem, IMO, as it's an app-specific approach to a solution that doesn't fit (IMO) our longer term plans.  Personally, I'd like to see it go away...

This appears to be also be important in XC for things like offer management (#245 )

I'll have to see if this is an unmanaged vocabulary (which we can model well with keywords/machineKeywords) or an ontology like this issue describes.

@trieloff I believe this issue can be closed. Content related fields are no longer in XDM.


Type Narrowing Inside Looping Structure Conditions not Persisting

Bug Report
🔎 Search Terms
Looping, Condition, Inference, Chain, And, &&, Array
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about Type Guards

⏯ Playground Link
Playground link with relevant code
💻 Code

let arr: Array<Array<boolean> | number> = [[true]];

for (let i = 0; i < arr.length; i++)
    if (arr[i] instanceof Array && arr[i].length > 0)  // Property 'length' does not exist on type 'number | boolean[]'.
        console.log(arr[i].length);  // Property 'length' does not exist on type 'number | boolean[]'.

if (arr[0] instanceof Array && arr[0].length > 0)
    console.log(true);

🙁 Actual behavior
When attempting to compile, the condition arr[i] instanceof Array infers that arr[i] is an Array and has the property length but this inference is not continued throughout the rest of the condition nor body within the condition due to the variable i. This behavior does not match the behavior with a fixed-index as is shown in the fixed-index block.
🙂 Expected behavior
Ideally, the inference from arr[i] instanceof Array would be continued into arr[i].length > 0 due to it being an && which makes the second condition never run unless arr[i] is an instance of  Array. This should also be the case for the body of the if statement. This behavior is shown with a fixed index but does not apply to a variable one.

It has nothing to do with the loop.  It's just that obj[key] isn't narrowed when key is not a literal.  Duplicate of https://github.com/microsoft/TypeScript/issues/51368 -> https://github.com/microsoft/TypeScript/issues/10530

Workaround: First assign the value arr[i] to a variable inside the loop and narrow that.


Anchor Dimensions

Hi!
First of all thank you for the really well documented work, neat code, and proactivity in taking care of all issues. This is a fantastic repository that's way more practical than most others I've seen.
I had a quick question on anchor dimensions. At the moment, it looks like the anchors are used as follows:

I understand that pw, ph correspond to one set of the anchors. I'm targeting bounding boxes of size 20 pixels by 20 pixels. To achieve that, given a grid size of 13 pixels, should my anchors be approximately (20/13 x 20/13) = (1.53, 1.53) ? When I try that in practice the bounding boxes appear way too huge.
In practice anchor sizes of around 0.4 - 0.6 get me the desired dimensions but I don't understand why that is the case as 0.5 * 13 (grid size) = 7.5 and not 20 pixels.
Could you help me understand how to go from a desired bounding box pixel size to an anchor size?
Thanks!
Dhruv

@dhruvp I think you mistake 13 pixels to be the absolute size of each grid cell.
I assume your input image size is 416 x 416. So we divide the image into the grid of size 13x13. There are 169 cells in the grid. The size of each grid cell is (416/13) X (416/13) (not 13 X 13). If you target bounding boxes of size 20 X 20. The size of the anchors should be (20/(416/13)) X (20/(416/13)) = (0.625 X 0625). It explains why this size works better than 1.53 X 1.53.

Ah that makes a lot of sense. Thank you!


[COGNITO SYNC] [UNITY] No clear way to force an identity id change without removing the old one

Hi,
I am currently writing a mobile app which uses Facebook to link to people's identities. I have noticed a problem that when you logout of a provider, such as Facebook, and use GetCredentialsAsync() immediately afterwards, it does not try to switch to a new identity id. I looked around and thought maybe I had to force the switch myself but I cannot find a way to do that without completely removing the old identity.
For example:
        // Called after FB.Logout()
    private void HandleFacebookLogout() {
        Debug.Log("Removing Facebook account from current identity.");
        Credentials.RemoveLogin(FacebookProvider); // This will then call ClearCredentials()
                // I tried this, which doesn't force a refresh
                Credentials.ClearIdentityCache();
                // and then I tried this which does force a refreshes but completely forgets what the old id is
                Credentials.Clear();
        Credentials.GetIdentityIdAsync((AmazonCognitoIdentityResult<string> result) => {
            Debug.LogWarning(result.Response);
        });

    }

I want to keep the old identity id because otherwise it breaks migrating the data from the old identity to the new one; as it no longer knows the old identity.
Hopefully it is just me not doing this correctly or in the right order. Thank you.

This is by design. You can only retain one identity id per pool on a given App/device. To migrate the data from old identity to new one you can listen to IdentityChangedEvent on CognitoAWSCredentials.

I can't migrate the old identity using that though because the oldIdentity argument, in IdentityChangedEvent, is always an empty string when logging out of Facebook, clearing the old identity id and then getting a new one.
Is there a way to get a new identity without having to clear the old one first? That way it should appear in the IdentityChangedEvent.

Similar to https://github.com/aws/aws-sdk-net/issues/382. Sadly there are not any option to keep old identity or clear way to switch it (I just clean all credentials and remove all previous user data using CognitoSyncManager.WipeData).

That is a shame to hear, I would expect it to support that.
I don't know if wiping their data would be the best approach for us as it could cause issues.

Closing for lack of activity


chdir line was removed in pull request #10393 which breaks some things...

such as C# support.
- This commit adds the original line of code back in which fixes it.

The original line has been in there since the second commit in the repo.
Thanks!

I don't see the link with #10393? Don't you mean #8144?

CC @supagu.

Oops, sorry about that. Yes, #8144. :)

Yes, it would be very nice to get @supagu to chime in since they probably know more about what's going on in this code than I do. All I know is that not having the original line of code causes the problem and putting it back in fixes.
Of course, I don't want my "fix" to break symlinks either. :)
Thanks @supagu !

the chdir is there in my code:
bool worked = (chdir(current_dir.utf8().get_data()) == 0); // we can only give this utf8
I am wondering how your code actually ends up changing the dir, causing it to work for you, as you are changing back to the previous dir, which is what you are trying to change out from?
Do you have a simple test case/project I can use to test on this code?

Thanks for the reply supagu,
I was wondering the same thing myself because, yes, it looks like it just changes back to the previous directory, however that code has been in there since the second commit of Godot (Feb 2014) and was there until this change. That doesn't make it correct code, of course, but it does seem odd that nobody saw it as buggy.
Unfortunately, I don't really have a simple test case. My test case includes C# support, so If you're on Linux I can send you a test case of your code vs the other code via Google Drive or whatever might work for you. One loads and runs (the previous code), one doesn't (this new code).
Maybe first, I'll dig into it a bit deeper tonight to see what's really going on with it and report back. :)

@supagu It looks like you've fixed a bug that's been there for three and a half years! Pretty cool :)
@akien-mga Rémi, I think I found and fixed the cause of the C# not loading bug. I'll make a new pull request and will do a screen recording explaining what I did to make sure I'm not doing the wrong thing. If I am you guys can at least get an idea of it. It links against GodotSharp, so it's not easy to make a simple reproducible case. If you have any issues with that then please let me know.
Thanks!


Boost package issue -- linking error on windows.

In brief: when I build with Boost python (everything standard, nothing special, see provided minimal example code) on Windows, I get linker errors when compiling. Tried exactly the same setup (see CMakeLists_NoHunter.txt in same example code) with prebuilt windows libraries, and it worked.
The linking works fine when the following code is commented out (the class being exposed):
class_<World>("World")
		.def("greet", &World::greet)
		.def("set", &World::set)
		;

I've read Brief overview section and do understand basic concepts. [Yes]
I've read F.A.Q. section and there is no solution to my problem there. [Yes]
I've read Code of Conduct, I promise to be polite and will do my best at being constructive. [Yes]
I've read Reporting bugs section carefully. [Yes]
I've checked that all the hunter_add_package/find_package API used by me in example is the same as in documentation. [Yes]
I'm using latest Hunter URL/SHA1. [Yes]

I've created SSCCE GitHub repository to reproduce the issue: https://github.com/Algomorph/minimal_hunter_boost_python_example

I'm building on Windows 10.
I'm using official CMake release
CMake version: 3.11.2

I'm not using toolchain

Closing inactive discussion.


Fix typo

The "P" looks to have been added by mistake
https://github.com/kolplattformen/skolplattformen/commit/fd495f9d45a10b820fcea1072bfb3123fbcbc8e1#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5L12-R12

Thanks for your contribution!


BUG: running python file called onnx.py causes circular errors.

System Info
latest optimum, python 3.10, linux cpu.

Who can help?
@JingyaHuang, @echarlaix, @michaelbenayoun
Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction (minimal, reproducible, runnable)
https://github.com/huggingface/optimum/issues/1177
Description of Bug:
If I create a py file to run my own scripts, and name it "onnx.py", it wreaks all kinds of havoc. Specifically circular errors. It took me a while to figure it was caused by "onnx.py" being a reserved name. This is the first time I've ever come across such an issue. I'm not sure if other modules prevent these issues by ringfencing their scope to specific folders or namespaces.. or whether it's just bad luck.
Is it possible to ringfence this kind of issue by either renaming the internal onnx.py file to something that users would never use OR, customize a validation check that tells user which filenames are reserved, OR at least updating the error message so that users don't need half a day to figure out what's causing the issue?
Many thanks
Expected behavior
That either I can use any filename for my script.py (eg. onnx.py) without issues
OR
There's a really clear error message that states "please do not use the following reserved names for your python scripts: eg1.py, eg2.py, etc"
Much appreciated

Thank you for the report, this is indeed quite bad! I'll see what can be done there.


Iterate over the entities in topological order when applying plan

Applies the plan in topological order. As it's computed by the openapi specification parser.
Fixes #57

Makes sense.
We should also not even try to apply something if it's dependencies have failed.

We don't apply the plan if the dependencies fail.
What we could do is just to exit the container so we can see it failing in argocd.


FindLinks broken ?

Hi ! I'm receiving an exception when I try to use the links property like this :
elements = browser.find_by_css(".link").links.find_by_partial_href(URL)
return [a["href"] for a in elements]

Here's the relevent part of the traceback :
  File "/home/edited_out/venv/lib/python3.8/site-packages/splinter/driver/webdriver/__init__.py", line 205, in find_by_partial_href
    return self.parent.find_by_xpath(
TypeError: find_by_xpath() got an unexpected keyword argument 'original_find'

https://github.com/cobrateam/splinter/blob/3bee3089e299f50ef7062b2e1892cd14f03a11f8/splinter/driver/webdriver/__init__.py#L207
I tried to remove all original_find and original_query from the FindLinks class and now it works fine !
Python 3.8
splinter 0.13.0
Firefox 72.0.2
geckodriver 0.26.0
Similar issue : #426

The solution is the opposite. WebDriverElement's find_by_xpath method should take original_find and original_query as optional arguments.

How about ?
diff --git a/splinter/driver/webdriver/__init__.py b/splinter/driver/webdriver/__init__.py
index 8f52138..68d860f 100644
--- a/splinter/driver/webdriver/__init__.py
+++ b/splinter/driver/webdriver/__init__.py
@@ -855,11 +855,12 @@ class WebDriverElement(ElementAPI):
             wait_time=wait_time,
         )

-    def find_by_xpath(self, selector, wait_time=None):
+    def find_by_xpath(self, selector, wait_time=None, original_find="xpath", original_query=None):
         return self.find_by(
             self._element.find_elements_by_xpath,
             selector,
-            original_find="xpath",
+            original_find=original_find,
+            original_query=original_query,
             wait_time=wait_time,
         )

@ShellCode33 Makes sense to me. If the tests pass you should open a PR for the fix.

Fixed by #758


feat(backend): update listing filters

Issue

Closes #issue
Addresses #1015
[ ] This change is a dependency for another issue
[ ] This change has a dependency from another issue

Description
Adds isVerified filter param and fixes how community programs are queried.
How Can This Be Tested/Reviewed?
Requires frontend updates
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have reviewed the changes in a desktop view
[ ] I have reviewed the changes in a mobile view
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have assigned reviewers
[ ] I have run yarn generate:client and/or created a migration if I made backend changes that require them
[ ] I have exported any new pieces added to ui-components
[ ] My commit message(s) is/are polished, and any breaking changes are indicated in the message and are well-described
[ ] Commits made across packages purposefully have the same commit message/version change, else are separated into different commits

Reviewer Notes:
Steps to review a PR:

Read and understand the issue
Review the code itself from a style point of view
Pull the changes down locally and test that the acceptance criteria is met
Also review the acceptance criteria on the Netlify deploy preview (noting that these do not yet include any backend changes made in the PR)
Either explicitly ask a clarifying question, request changes, or approve the PR if there are small remaining changes but the PR is otherwise good to go

On Merge:
If you have one commit and message, squash. If you need each message to be applied, rebase and merge.

Hey @emilyjablonski could you take another look at this I think this is okay for the backend and is ready for the front end

@YazeedLoonat Do you mind reviewing Michal's latest commit?

You can probably think of a better alternative, but I got that working locally by doing the following:

It looks fine to me.


Missing Distinct function

Hi all, the distinct function that is documented here does not exist in the engine today. Ideally, that function could make it over to the C# version of the engine.

see  #212 as an example.


formState.isSubmitting does not behave as we could expect

Hi,
I noticed that formState.isSubmitting is not working as expected. I tried to setup this library with Remix, implementing some slow action.
Actual behaviour
isSubmitting is set to false as soon as the POST network call is fired, before it returns.
Expected behaviour
isSubmitting should stay true until action POST terminates.
Reasons
In react-hook-form, formState.isSubmitting is true as long as onSubmit promise is not either rejected or resolved, onSubmit being the argument of handleSubmit.
In remix-hook-form, users don't provide any onSubmit function as it's using the native implementation, triggering a Remix action.
In this repo, you're using Remix's const submit = useSubmit() hook to submit the form. But it does not seem to me that there is any way to listen to termination of that submit event (e.g. wait for the end of the POST request).
Possible solution?
Actually, there is a debate here about returning a promise for useSubmit() hook in react-router. They also give a workaround that could work for us : to use useActionData() hook to know when the remix action is complete.
I guess we could do that to properly set formState.isSubmitting, submitted, ... etc.
let actionData = useActionData();
useEffect(() => {
    if (actionData.ok) {
       complete(actionData)
    }
}, [actionData]);

@licarth Hmm that is a good point, there is a way to add this easily, because when you submit to an action the navigation state will be either submitting or redirecting until it is done so I can listen to that to find out if it's done, the native isSubmitting just waits for the valid data and submits. I will add this behavior to the package


Make it possible for generators to simulate combinatorial logic

 Issue by nakengelhardt
Tuesday Dec 18, 2018 at 05:24 GMT
Originally opened as https://github.com/m-labs/nmigen/issues/11

Sometimes I want to use generators to build a simulation model of some module. In migen, this wasn't possible for modules that implement combinatorial logic.
Trying my hand at nmigen syntax, an example could be something like this:
class RoutingPolicy:
    def __init__(self, width, n):
        self.data = Signal(width)
        self.destination = Signal(max=n)

    # I haven't decided yet what I want this module to do, let's try some options
    def gen_destination(self):
        while True: # or something?
            data = (yield self.data)
            if (data == 23):
                yield self.destination.eq(0)
            else:
                yield self.destination.eq(self.data[:log2_int(n)])

class Router:
    def __init__(self, width, n):
        self.in_port = Signal(width)
        self.out_ports = Array(Signal(width) for _ in range(n))

        self.policy = RoutingPolicy(width, n)

    def get_fragment(self, platform):
        m = Module()
        m.submodules.policy = self.policy
        m.d.comb += [
            self.policy.data.eq(self.in_port),
            self.out_ports[self.policy.destination].eq(self.in_port)
        ]
        return m.lower(platform)

m = Router(8, 2)
run_simulation(m, m.policy.gen_destination())

There probably needs to be something to tell what constitutes one iteration of the generator, with while True it would just run indefinitely. while (yield input_values_changed()): ? Also the whole thing would need to be @passive or so since there's no predefined end point.
Another question is whether it's possible, or even makes sense, to mix combinatorial assignments and synchronous assignments in one generator.

Denominating from 0.3 because this issue requires #228.

It is now possible to simulate combinational logic.


DRAFT: radicle contrib init

Introduction and basic instructions for Radicle peer-to-peer network for code collaboration, and seednode.link

Codecov Report
Patch coverage has no change and project coverage change: -0.03 :warning:

Comparison is base (a7c1820) 55.66% compared to head (91b3f43) 55.63%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #346      +/-   ##
==========================================
- Coverage   55.66%   55.63%   -0.03%
==========================================
  Files          32       32
  Lines        4982     4984       +2
==========================================
  Hits         2773     2773
- Misses       2209     2211       +2

Impacted Files
Coverage Δ

zingocli/src/lib.rs
0.00% <0.00%> (ø)

zingolib/src/lib.rs
36.36% <0.00%> (-3.64%)
:arrow_down:

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


Log viewer: bug in type-ahead search

The type-ahead search (client only) doesn't work sometimes? Hitting one key results in empty log list.
What's wrong with this code:
entries                 .filter(entry => [entry.message, entry.strackTrace, getLocationString(locationFormat, entry), entry.level, entry.logDate]                     .join('|#|').toLowerCase()                     .indexOf(lowercaseSearch) >= 0)
Some fields of entry seems to be empty, but will be displayed.

Typo fixed.


App update not published on FDroid

I published an update with a new tag 3 weeks ago. Normally updates with this tag prefix are automatically pushed on FDroid.
tag : onelist-1.5.0
@licaon-kter any idea why it's not working ?

https://github.com/lolo-io/OneList/blob/onelist-1.5.0/app/build.gradle.kts#L29 I don't see a new version, F-Droid already has 19

Thanks @licaon-kter , I updated the code, it should be alright now.

Great, the checkupdate bot will pick it up soon™

https://monitor.f-droid.org/builds/log/com.lolo.io.onelist/20#site-footer The build failed.

@linsui  I don't understand what's the problem. Build passes on my CI and I haven't changed config since last update.

https://gitlab.com/fdroid/fdroiddata/-/jobs/7771210545 The build failed.

This error is happening when running  /home/vagrant/fdroidserver/gradlew-fdroid clean.
There is no error when running ./gradlew clean on this repo.
@linsui  @licaon-kter

@lolo-io can you verify if your local gradle is 8.6 too?
ref: https://github.com/gradle/gradle/issues/18971

@licaon-kter yes I am running gradle 8.6.
https://github.com/lolo-io/OneList/blob/develop/gradle/wrapper/gradle-wrapper.properties

vagrant@bookworm:~/build/com.lolo.io.onelist$ ./gradlew clean assembleRelease
To honour the JVM settings for this build a single-use Daemon process will be forked. For more on this, please refer to https://docs.gradle.org/8.6/userguide/gradle_daemon.html#sec:disabling_the_daemon in the Gradle documentation.
Daemon will be stopped at the end of the build
Type-safe project accessors is an incubating feature.

FAILURE: Build failed with an exception.

* What went wrong:
org.gradle.api.InvalidUserDataException: Cannot generate project dependency accessors:
  - Cannot generate project dependency accessors because project 'com.lolo.io.onelist' doesn't follow the naming convention: [a-zA-Z]([A-Za-z0-9\-_])*
> Cannot generate project dependency accessors:
    - Cannot generate project dependency accessors because project 'com.lolo.io.onelist' doesn't follow the naming convention: [a-zA-Z]([A-Za-z0-9\-_])*

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
> Get more help at https://help.gradle.org.

BUILD FAILED in 10s

I tried with gradlew but it still failed.

This can be fixed by added rootProject.name. It's not related to fdroid-gradlew at all. The name is the dir name of the repo. On the F-Droid buildserver the repo dir name is the app id. If it's a name without . it works.
I've added rootProject.name="com-lolo-io-onelist" to settings.gradle (replaced the dots by dashes). Hopefully the build will pass this time 🤞

Thanks!


[Task]: Refactor loading screen into a view

Task description
Right now the loading screen is a popup that displays in certain situations depending on some stores. This is hard to debug and leads to some behaviour where it can close or not open when we want it to.
Requirements

[ ] Make loading screen a view
[ ] Navigate to this during login
[ ] Navigate from this after everything has loaded.

Creation checklist

[ ] I have assigned this task to the correct people
[X] I have added the most appropriate labels
[X] I have linked the correct milestone and/or project

I would make a generic loading component and then a statefull logging in view that determines when to move on during login / recovery.


Phi-3 model performance on MeteorLake GPU

Hi,
we are running benchmark on 3b models, using all-in-one script, models are phi3-4k, phi3-128k and starcoder2-3b.
Environments:
ThinkBookX2024
Ultra9-185H
Windows11pro-23H2
32GB LPDDR5x8400MHz
Arc Driver: <IP_ADDRESS>45
ipex-llm[xpu] 2.1.0b20240506
Questions here:
have we done the performance comparison for Phi 3 4k/128k and starcoder2-3b

we have fixed the performance gap between phi-3-mini-4k and phi-3-mini-128k, please try latest ipex-llm (
2.1.0b20240508)
and we are still working to improve phi-3-mini performance

we have made a big performance improvement for phi-3-mini (4k and 128k) in latest version, please upgrade to latest ipex-llm and bigdl-core-xe-21 to try it (2.1.0b20240509)


How to embed a custom player?

My issue is about the Video Player. It doesn't support much control, so I am trying to implement my own player but I don't want to tamper with the package code as I want to keep up with new updates/features and releases from Quill.
So I am trying to implement an embed builder for my player, but not sure how to. Please if someone can show me an example of embedding your own video player of choice I will be glad. thanks

There's a sample demo app inside of the repo. Have a look at the custom embeds.

Actually I couldn't find any example that used the embed feature. I know
there is a demo app, but I couldn't find the use case for embeds in the
example
On Thu, 14 Jul 2022, 23:00 Adrian Moisa, @.***> wrote:

There's a sample demo app inside of the repo. Have a look at the custom
embeds.
—
Reply to this email directly, view it on GitHub
https://github.com/singerdmx/flutter-quill/issues/869#issuecomment-1184932516,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAQRALI7JEQNWX333WFZQDVUCEZLANCNFSM53TLSKWA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Please do direct PR
Don't do custom

There's a sample demo app inside of the repo. Have a look at the custom embeds. They enable you to change the player without changing the source code.

Please can you point me in the right direction 🙏
Like under what file name/folder can I find those custom embed examples. Because this is really what I need, I don't want to change the flutter Quill codes at all

Please do direct PR Don't do custom

Please can you explain what you meant by this?

I think this is where you can find some sample code.

Actually, I solved it. I had to go through the library, under the embed folder 📂, I found an example called defaultEmbedBuilder, and I had to replicate the examples there to build my customized player. Now player has more controls and can now play YouTube, Vimeo, Network, File & Asset Videos with options for chosing video quality, tap to seek, just like Youtube app. BTW I used the pod_player plugin for that.
I feel this example called defaultEmbedBuilder should be moved to the example app rather than leaving it in the library, so that more people will know how to use it to customize their player


add kosen college

Institution/School Name
National Institute of Technology, Ashahikawa College
Instiution/School Website
http://www.asahikawa-nct.ac.jp/en/
Email Users
Please place an x between the square brackets if yes

[x] Students
[x] Academic/Teaching Staff
[x] Other/Support Staff
[ ] Alumni

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[ ] Graduate School
[x] College (University) or Undergraduate School or equivalent
[x] Community College or equivalent
[x] High School or equivalent
[ ] Elementary School or equivalent

@ckinako Please provide us with a proof which shows that the school recognizes the domain which you are submitting as an official email domain for the students.

Sorry.I'm Japanese student ,so I can't write English well.But I try it. These screenshot are  Ashahikawa College official website and whois infomation .Can it be  proof ?

@ckinako I am sorry, there is a little misunderstanding. The screenshots which you attached show that the domain of the school is asahikawa-nct.ac.jp. If you would have similar screenshots for asahikawa.kosen-ac.jp, it is enough to add asahikawa.kosen-ac.jp to this repository.
Your request shows that you wish to add a domain asahikawa.kosen-ac.jp. It means that students of your school use email addresses such as<EMAIL_ADDRESS>I need to have a proof, which shows that the school recognizes asahikawa.kosen-ac.jp as an official email domain for the school's students.
Do you have any proof that the school assigns email addresses such as<EMAIL_ADDRESS>to the students?

I'm sorry.I I misunderstood the domain between asahikawa.kosen-ac.jp  and asahikawa.kosen-ac.jp ,so I want to withdraw pull request.


Clean up bankruptcy code (performance/API).

We are about to have a partial index for unread user messages, and it would be nice to have the expensive bankruptcy codepath use it.
This would be a simple, low risk change if there were a dedicated code path for marking messages as read, but right now the backend code (as well as the API) is conflated with starring and mentioning messages.
The bankruptcy API is low level and probably over-generalized.  The endpoint is called /messages/flags, and it takes a special parameter called all that only sensibly applies to unread messages.  You would never want a user to star all of their messages or mark them all as mentioned.  The API also allows you to set flags that are truly internal, like is_me_message.
The goal of this issue is not to fix every possible flaw in the /messages/flags API, but I think it does make sense to break out a special API for marking messages as read, and then only that API would take arguments like "stream" and "all," and then we could also grow new features like leaving the last N hours of messages still unread.
Once we have a dedicated API for bankruptcy, it should be a fairly simple matter to take advantage of the partial indexes.  In order to use our partial index on unread messages, we do something like this:
    user_msgs = UserMessage.objects.filter(
        user_profile=user_profile
    ).extra(
        where=[UserMessage.where_unread()]
    )

Yeah, that seems like a good refactor.  I would be in favor of keeping /messages/flags as the main place for toggling flags on specific messages, but moving the stream and all features to another endpoint.  Code-wise, even just splitting do_update_message_flags into the top and bottom halves (so we can reuse the bottom half) would be more readable.

Yeah, starting from the inside out (i.e. splitting do_update_message_flags as you suggest) may be a good strategy here.

A related discussion on unread counts (and possibly capping unread queries):
https://chat.zulip.org/#narrow/stream/backend/subject/unread.20data/near/259338

There's a few issues going on here; I just opened these two for issues around that last discussion:

https://github.com/zulip/zulip/issues/5958
https://github.com/zulip/zulip/issues/5959

The main issue here has been fixed.


Can't install developer version on Chrome, Windows 10

Hi devs/gurus,
I've been using YTC for a while now. Last week I updated to W10 and for some reason I cannot install the developer version on my Chrome. Usually, when I click on the userscript link, Tampermonkey auto detects or directs me to a page where I can click Install or Process. This time, however, it downloads YouTubeCenter.user.js.
Now, I'm not really sure if this is a common issue or something easy to solve, but hopefully it's the latter and I appreciate if someone was able to help me with this. Thannks!

Couldn't you just drag the downloaded file into Tampermonkey dashboard?

I've tried the normal way of installing and it shows as disabled.
Chrome did show an info box saying incompatible extensions have been disabled.
Windows 10 Pro 64-Bit. Chrome 44.0.2403.157 m (64-bit)

i can, because i have, i'm using Dev version as well.

RohithKumarSp Could you elaborate? did you use a user script to install, or did it just work? I don't know of any special way to install an extension to overcome this?

installed tampermonkey and clicked on youtube center script and it worked, what's there to it ? if you dont enable tamper it will say can not add from this website.

Ah, I've misunderstood instructions. I'll try again.
Thanks
On Wed, 26 Aug 2015 9:25 pm RohithKumarSp<EMAIL_ADDRESS>wrote:

installed tampermonkey and clicked on youtube center script and it worked,
what's there to it ? if you dont enable tamper it will say can not add from
this website.
—
Reply to this email directly or view it on GitHub
https://github.com/YePpHa/YouTubeCenter/issues/2022#issuecomment-135160411
.

Tampermonkey script worked perfectly. Thanks RohithKumarSp

No problem


Bug: directories are skipped when determining assets to upload

Expected Behaviour
Any external file references that are directories should be included in the Input Directories of the asset manifest and all files within uploaded as part of job submission.
Current Behaviour
Directories are being skipped when Parse Files is clicked. The Input Directories are never populated, only Input Filenames.
Reproduction Steps
To reproduce the issue I:

created a new geometry node in /obj
inside the new geometry node I created a File Cache node
set the Base Folder name on the File Cache node to the absolute path to a local folder containing some test files(doesn't matter if they are actual cache files)
added new mantra and deadline cloud nodes in /out
clicked Parse Files in the deadline cloud node
saw no input directories added

to be extra sure I added a debug print statement to where we grab the list of inputs and verified that the linked directory check was skipping the directory specified in my File Cache node.

Code Snippet
All directories are currently skipped when determining inputs here
for parm, ref in hou.fileReferences():
        if parm:
            if parm.node() == rop_node:
                continue
            if ref.startswith(IGNORE_REF_VALUES):
                continue
            if parm.name() in IGNORE_REF_PARMS:
                continue
        if os.path.isdir(ref):
            continue
        input_filenames.add(ref)

Issue #124 should be kept in mind here since the directory existence check can fail as-is if the path isn't fully evaluated and able to be resolved to an actual directory.


Arguments for periodic tasks 436

Closes #436
Notes:

There is one acceptance test that keeps failing  even on master branch: tests/acceptance/test_nominal.py::test_nominal. Maybe, it's somehow connected with my dev environment (WSL Ubuntu).
It seems like @attr.dataclass(frozen=True) doesn't support mutable attributes, so I put FrozenSet instead of dict in PeriodicTask.

Successful PR Checklist:

[X] Tests

[ ] (not applicable?)

[ ] Documentation

[X] (not applicable?)

Haha and I've broken your tests, perfect :D
I'll fix that at some point, but we need to answer the others remaining questions.

Good point!
I tried to run your example, and it seems like only the first is actually deferred, others are just ignored.
We can just concatenate this optional UID to the task_name. We can also try to use hash(kwargs) if kwargs is hashable and UID is not provided. In this case we don't need to change table indexes.
As for the second option, it can be acceptable for only a few arguments, otherwise it can turn into a mess like this:
a.b.my_task(
    arg1=1,
    arg2={1: {"a": "val1", "b: val2"}, 2: {"a": "val3", "b": "val4"}},
    arg3=[<class '__main__.SomeClass'>, <class '__main__.SomeClass'>]
)

I've looked at the tests and found that it is failing at tests/migration/test_migration.py::test_migration.
The statements with ALTER TABLE in the last migration I already fixed.
But there is another problem with CREATE FUNCTION procrastinate_defer_periodic_job: the test keeps failing if we're trying to create function without dropping it first.
I think we should keep DROP FUNCTION IF EXISTS procrastinate_defer_periodic_job; above CREATE by analogy like in 00.11.00_03, 00.14.00_01 and 00.15.02_01.

I think we should keep DROP FUNCTION IF EXISTS procrastinate_defer_periodic_job; above CREATE by analogy like in 00.11.00_03, 00.14.00_01 and 00.15.02_01.

Definitely yes !
(As you may have seen, I've been spending some time lately on the PR backlog of procrastinate, I should quickly have more time to work this with you, hopefully)

Okay, added a few things:

Fixed the migration so that they are blue-green.
Fix (& type) InMemoryConnector & tests: name_suffix is a concept related to periodic tasks only, it should have been added to the tasks themselves. Also, the dictionnary that tracks what periodic jobs have been launched should use (task name, name_suffix) as its index.
By avoiding lru_cache, PeriodicTask.kwargs can be a normal dict. This makes the code quite easier.
rename name_suffix as periodic_id, because it's not a suffix anymore
Added the documentation so that people know who to use the new feature
Change: Additional periodic args are not tasks kwargs anymore but configure kwargs. This makes it easier to configure not just kwargs, but also locks, queuing locks etc.

And sorry it took so long for this PR to get moving !

And sorry for tagging this good for newcomers, I didn't expect there would be so much work :D


403 Unauthorized

System Health details
System Information

version
core-2024.3.0

installation_type
Home Assistant OS

dev
false

hassio
true

docker
true

user
root

virtualenv
false

python_version
3.12.2

os_name
Linux

os_version
6.1.74-haos

arch
x86_64

timezone
Europe/London

config_dir
/config

Home Assistant Community Store

GitHub API
ok

GitHub Content
ok

GitHub Web
ok

GitHub API Calls Remaining
5000

Installed Version
1.34.0

Stage
running

Available Repositories
1493

Downloaded Repositories
91

HACS Data
ok

AccuWeather

can_reach_server
ok

remaining_requests
44

Home Assistant Cloud

logged_in
true

subscription_expiration
17 August 2024 at 01:00

relayer_connected
true

relayer_region
eu-central-1

remote_enabled
true

remote_connected
true

alexa_enabled
false

google_enabled
false

remote_server
eu-central-1-11.ui.nabu.casa

certificate_status
ready

instance_id
01ba98e99eda43a0a3d26db17b0fd36a

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Home Assistant Supervisor

host_os
Home Assistant OS 11.5

update_channel
stable

supervisor_version
supervisor-2024.03.0

agent_version
1.6.0

docker_version
24.0.7

disk_total
30.8 GB

disk_used
18.6 GB

healthy
true

supported
true

board
ova

supervisor_api
ok

version_api
ok

installed_addons
Studio Code Server (5.15.0), Terminal & SSH (9.10.0), RPC Shutdown (2.4), Folding@home (0.7.2), ESPHome (2024.2.1), Mosquitto broker (6.4.0), Zigbee2MQTT (1.36.0-1), Samba share (12.3.0), Home Assistant Google Drive Backup (0.112.1), sip2mqtt (beta), AppDaemon (0.16.4), Filebrowser (2.23.0_12), Portainer Agent (linux-arm64-2.19.4-alpine), Cloudflared (5.1.6), Meross Local Broker Addon (0.0.1-alpha49)

Dashboards

dashboards
5

resources
37

views
20

mode
storage

Recorder

oldest_recorder_run
17 March 2024 at 15:16

current_recorder_run
17 March 2024 at 15:16

estimated_db_size
879.37 MiB

database_engine
sqlite

database_version
3.44.2

Spotify

api_endpoint_reachable
ok

Checklist

[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of currently previous issues..

Describe the issue
Since around 10:20 AM (UK Time) the MSFT Family Safety API has started to return a 403 Unauthorized error.
No details in logs, but will require investigation upstream. Perhaps the auth protocol has changed.
Reproduction steps
N/A
Debug logs
Logger: custom_components.family_safety.coordinator
Source: helpers/update_coordinator.py:350
integration: Microsoft Family Safety (documentation, issues)
First occurred: 19 March 2024 at 06:59:34 (10 occurrences)
Last logged: 10:20:32

    Error fetching Microsoft Family Safety data: Error communicating with API
    Error fetching Microsoft Family Safety data: Error communicating with API [Errno 104] Connection reset by peer
    Error fetching Microsoft Family Safety data: Error communicating with API HTTP Unauthorized

Diagnostics dump
No response

Will be fixed in next version


Making it possible to require all of the given field names on require-id-when-available

Is your feature request related to a problem? Please describe.
When using the require-id-when-available rule with passing an array of field names, the rule passes if one of the field names exists in the selection set. I want that all of them should exist in the selection set (all given fields are required if the fields are defined on a type).

Describe the solution you'd like

Add configuration option which treating the given fields as all required.
Describe alternatives you've considered

Additional context

Migration from apollographql/eslint-plugin-graphql's required-fields rule (migration to graphql-eslint is suggested in the README). That rule's behavior is desirable for us.

https://github.com/apollographql/eslint-plugin-graphql#required-fields-validation-rule
https://github.com/apollographql/eslint-plugin-graphql/blob/64c524cd6607358803d5fcb7cead0a383a125ccb/src/customGraphQLValidationRules.js#L32

Hi @milashkaJason, does your issue related to require-id-when-available? Can you give more context or maybe make a reproduction repo?

I got the same issue and I finally get it work by disabling the rules
'@graphql-eslint/require-id-when-available': 'off',
'@graphql-eslint/no-unused-fragments': 'off',

I'm posting here instead of reopening https://github.com/dimaMachina/graphql-eslint/issues/1159. To reproduce, go to the playground and enter in operation.graphql:
# eslint @graphql-eslint/require-id-when-available: ["error", { fieldName: ["id", "__typename"] }]
query Q {
  posts {
    id
    title
  }
}

Or ["id", "createdAt"]. The code should be invalid, since only id is included in the selection and __typename/createdAt isn't.


[tensorflow] Update the max version of the supported bazel

Describe the pull request

What does your PR fix?

Since the PR https://github.com/microsoft/vcpkg/pull/18817, the version of bazel used by vcpkg has been upgraded from 3.7.0 to 4.1.0, this causes tensorflow installation failed with following error:
You have bazel 4.1.0 installed.
Please downgrade your bazel installation to version 3.99.0 or lower to build TensorFlow! To downgrade: download the installer for the old version (from https://github.com/bazelbuild/bazel/releases) then run the installer.

For fixing this issue, I add a patch to modify the highest version of bazel supported by tensorflow to 4.1.0.
No feature need to be tested.

Do we have indications from upstream (in an issue or otherwise) that this is safe to do?

It seems the right fix is to downgrade Bazel rather than patching this unless upstream is OK with it

Upstream issue reported by us https://github.com/tensorflow/tensorflow/issues/50966.
They have created a PR https://github.com/tensorflow/tensorflow/pull/50882 to support bazel 4.1.0.

Given that upstream is okay with this, I think this is good.

Now, upstream reverted to the version 3.99 of bazel again. @Cheney has commented on upstream issue https://github.com/tensorflow/tensorflow/issues/50966#issuecomment-888748570.
Let's wait for the final conclusion from upstream.

I'm merging this because upstream seems tentatively ok (they reverted only to split the change into a separate PR) and this is blocking all other PRs.


chore: Fix dev build issues

Stack:

#69
#68 ⬅

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

✓ Commit merged in pull request #69