Expose internal services to host

Feature description
Add some mechanism for exposing Consul Connect services outside of Vagrant environment
Why is it needed?
When developing modules that create online services you sometimes need to access that service, or a user interface in the application hosting the service, from the developers interactive environment.
Suggestion(s)/solution(s) [Optional]
To expose connect-enabled services you have two separate challenges:

Mesh / ingress gateway to the enclosed service. Can be solved using the commandline consul proxy functionality
Exposing a TCP port to the host OS where vagrant runs. May require IP Tables forwarding rules because of the separate isolated network used for Docker containers.

Definition of done
A developer can by reading some documentation and making some simple configurations get access to one or more interfaces hosted as containerized nomad jobs from their laptop browser or API client.
Checklist (after created issue)

[ ] Added label(s)
[ ] Added to project
[ ] Added to milestone

We have created a barebones structure for exposing a running consul connect enabled to our local machine https://github.com/fredrikhgrelland/terraform-nomad-minio/blob/master/Makefile#L77-L79
I haven't read your issue deeply just yet (in a meeting), but from my initial understanding this would cover

"get access to one or more interfaces hosted as containerized nomad jobs from their laptop browser or API client"

We have created a barebones structure for exposing a running consul connect enabled to our local machine https://github.com/fredrikhgrelland/terraform-nomad-minio/blob/master/Makefile#L77-L79
This would expose the service registered as minio in consul on port 9000 on localhost of the machine running vagrant.
I haven't read your issue deeply just yet (in a meeting), but from my initial understanding this would cover

"get access to one or more interfaces hosted as containerized nomad jobs from their laptop browser or API client"

Thank you, I will test and update.

A quick test and it turns out that a local consul proxy can reach the service inside vagrant. I did not think this network was accessible externally, but it is and so this works. This is a good solution for now so I'll close the issue. Thank you!

A quick test and it turns out that a local consul proxy can reach the service inside vagrant. I did not think this network was accessible externally, but it is and so this works. This is a good solution for now so I'll close the issue. Thank you!

Sweet, good to hear!
I realised we have no good documentation for this (other than in our already created modules), so I have created an issue in vagrant-hashistack-template to add a section on this in the doc.
https://github.com/fredrikhgrelland/vagrant-hashistack-template/issues/91


[BUG] Space is being created even though it's not declared

Is there an existing issue for this?

[X] I have searched the existing issues

Are you using the latest docker image for BTPSA?

[X] I'm using the latest docker image for BTPSA.

Which area is mainly impacted
Cloud Foundry setups
Current Behavior
Cloud Foundry space "development" is being created during setup, no "cfspacename" parameter is declared
Expected Behavior
No space should be created if no "cfspacename" parameter is declared
Steps To Reproduce
No response
Logs and configuration files available?
spaceissue.zip
Anything else?
As you can see from parameters.json file, I removed "cfspacename" line. Usecase doesn't have any space-specific services. However, space is being created anyway

Confirm bug, see PR #359

The bug is fixed, and the fix is available in the dev branch. It will be part of the next release. To omit the creation the following setting is necessary. We provide a new parameter in parameters.json to enable this behavior. In case you want to skip the creation of a CF space although you create a CD environment (and an org) you can set the parameter skipcfspacecreation to true.
The default value of the parameter is false, so all existing parameter.json configurations do not need to be adjusted
The issue will be closed once the release is published.

Fix is available in main and in Docker image with release 1.2.1

Hi @lechnerc77
Please re-open this issue. When using "skipcfspacecreation": true parameter, the script fails. For some reason it skips "cf login" command and as a result all the following cf commands (e.g. cf set-org-role) are failing with:
[2022-12-28 11:29:07] ERROR      : FAILED
[2022-12-28 11:29:07] ERROR      : No API endpoint set. Use 'cf login' or 'cf api' to target an endpoint.
If I remove this parameter, everything works fine.
Please compare the output. Without "skipcfspacecreation": true

With "skipcfspacecreation": true

Login to CF is hidden in space creation => should be done always when a Cloud Foundry environment is created

The missing cf login is fixed with PR #381 . This fix is available in the dev branch (incl. corresponding Docker image) and will be available in main with the next release.


Support to use application-fury in resteasy

This will support to use application/fury as a mediatype and de(serialize) objects over the HTTP connections.
see testFuryBar() method.

@chaokunyang can you take a look at the native test failure. It seems that Bar and ThirdPartyBar has the same record
int f1, string f2

In JVM mode, it is deserialized into Bar but in native mode, it is into ThirdPartyBar.
2024-11-13 07:54:17,672 ERROR [org.jbo.res.res.i18n] (executor-thread-1) RESTEASY002005: Failed executing GET /fury/test: org.jboss.resteasy.spi.InternalServerErrorException: RESTEASY003020: Bad arguments passed to public io.quarkiverse.fury.it.Bar io.quarkiverse.fury.it.FuryResources.testBar(io.quarkiverse.fury.it.Bar)  ( io.quarkiverse.fury.it.ThirdPartyBar ThirdPartyBar[f1=1, f2=hello bar] )

@chaokunyang if there two class have the same fields, like
class A {
    int a;
    String b;
}

class B {
    int a;
    String b;
}

It should be serialze into the same bytes, right? during deserilzing, how `fury` can detect which class type should be used?

Fury generated codec is cached into a global map, maybe they have a conflict. When creating ThreadSafeFury, we could a different name by FuryBuilder#withName to avoid cache conflict.

Maybe We could Set name to timestamp for tests

Hmm, another found that Bar and ThirdPartyBar looks like use the very similar custom Serializer. BarSerializer and ThridPartyBarSerializer. Is that a problem?

Hmm, another found that Bar and ThirdPartyBar looks like use the very similar custom Serializer. BarSerializer and ThridPartyBarSerializer. Is that a problem?

They are all caused by cache conflict, I fixed it in #27

OK, I will check with your fix. Thanks a lot!

Sorry @chaokunyang - your fix looks not working!


Adapts TrxLogger constructor to actually use passed variable

Description
The description of the constructor says it's with DI for unit testing. But the passed variable isn't actually used.

Thank you for the contribution @schibu007! Approved.


variables in wildcards doesn't apply to other wildcard

I wrote some generic prompts to combine funny arts, like this
# main-prompt.txt
${character:__characters/**__}, ${default-eyes:{looking at viewer|averting eyes}}

# characters/a.txt
${default-eyes=ahegao}morikubo nono, other tags...

and in UI prompt I wrote it like:
${character=__characters/a__}__main-prompt__

This results in all images are with character A, but ahegao never gets used. Instead it's using the default values in main-prompt. Is this expected behaviour?
When I write it like this in UI prompt:
${character=__characters/a__}${default-eyes:ahegao}__main-prompt__

It's now character A with ahegao always as expected. I wish to exhibit this behavior without defining default-eyes in UI prompt.
I've tried immediate evaluation in characters/a.txt, but still doesn't work.
This can apply to outfits, etc. I want to define default outfits for each character in the character's template, and later in the main prompt decide to use those or not. Current behaviour makes it difficult to achieve.

With further test, I think this is related to variable scope. If I change to a structure like this,
# main-prompt.txt
${character:__characters/**__}

# characters/a.txt
${default-eyes=ahegao}morikubo nono, other tags..., ${default-eyes:{looking at viewer|averting eyes}}

This works, but now every character/*.txt has to have default-eyes. Is there a better way to structure this?


Add "SecurityOptions" in Info.

This PR tries to add SecurityOptions field in Info so that security options such as apparmor, seccomp, or selinux could be displayed by docker info.
Detailed discussion could be found in:
https://github.com/docker/docker/issues/20909
https://github.com/docker/docker/pull/21172
cc @justincormack @calavera @thaJeztah
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

@thaJeztah Just updated the pull request to change to []string. Let me know if there are any other issues.

LGTM

LGTM

Please, do not merge this PR yet. We're waiting for a release cut in docker/docker before moving this forward.

We already have a tag a frozen branches for both projects. Merging this in :tada:


Device settings L1 test profile improvements

Modified Audio & Video port L1 test cases as per yaml settings file.

closing as it is duplicate of https://github.com/rdkcentral/rdk-halif-test-device_settings/pull/75


More light-weight LMScores proto.

making changes according to issue #98

Hi Adhham. Why are you marking the symbols field as deprecated?

Hello Agutkin, it should be removed, I was thinking of a way to announce updated functionality, but marking it as deprecated is the wrong indication.
I will remove it and add a new commit.


Add gocrane to landscape

Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you added your SVG to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~30 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Build failed because of:
no headquarter addresses for gocrane at https://www.crunchbase.com/organization/gocrane
Empty twitter for Mail.Ru Cloud Solutions (member): https://twitter.com/mailru
Empty twitter for China Telecom (member): https://twitter.com/chinatelecom1
Empty twitter for Mail.Ru Cloud Containers: https://twitter.com/mailru
Empty twitter for Mondoo (member): https://twitter.com/mondoolabs
Empty twitter for Scalyr: https://twitter.com/scalyr
Empty twitter for Spotinst Functions: https://twitter.com/spot_hq
No cached entry, and Sangfor CNAD has issues with twitter: https://twitter.com/SangforUSA, Request failed with status code 404
No cached entry, and Sangfor EDS has issues with twitter: https://twitter.com/SangforUSA, Request failed with status code 404
Empty twitter for Karmada: https://twitter.com/karmada_io
Empty twitter for Mondoo: https://twitter.com/MondooLabs
gocrane either has no crunchbase entry or it is invalid

LGTM


Add package documentation preview

Currently it seems like this is just a service. I do see mention of running it locally, but then you need to install redis as well. And also I imagine it will also only work with published NPM packages.
I want a way to preview what the generated documentation will look like while I'm working on my code, before publishing it.
E.g. something like a tsdocs-preview NPM package that you can add as a dev dependency to your project to be able to run a server that shows you the docs generated based on the files in your local repo checkout.
Or perhaps it would be even better if there was like a tsdocs-cli NPM package that enables you to locally generate the related HTML etc or perhaps even alternatively MD. That way that could alternatively also be included within NPM packages. But also it would enable previewing your tsdoc based documentation.

PS: I like what you've made; it looks really nice and could work really well in theory for all of us TypeScript NPM publishers. But because the service is still regularly slow or even unusable I find that we can't rely on it. And this lack of preview has already resulted in me publishing a package with unusable documentation.
I guess I'm going to give typedoc-plugin-markdown another try; this time combined with concat-md to turn it into a single file to make it easier to redistribute from our NPM packages through our docsify page.


Compile single job or service

Two more actions
sasjs compile -> does everything in a project, works locally
sasjs compile job -> compiles  a single job, can work anywhere
sasjs compile service -> compiles a single service, can work anywhere
the output would be saved as per the --output location,

if in a project, then it would go in the root of the sasjsbuild folder
if global, create sasjsbuild and place there

closed as duplicate of #469


BE: Adding 1.3.0 codes

Android 1.1.3-acc / iOS 1.1.4-acc
VAC / TEST /specialcases/VAC_INCOMPLETE_DOB  are valid
REC is valid from 11th of july


DbContext.Set() seems to be missing

This does not seem to match what shows up in intellisense.
I see:
DbSet<TEntity> DbContext.Set<TEntity>()
and
DbSet<TEntity> DbContext.Set<TEntity>(string name)
There is no
DbSet<TEntity> DbContext.Set()

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

ID: 86d308d2-9653-9cd5-2313-e60ea0e70fdf
Version Independent ID: e8e3611f-8d9c-d9c6-1879-55ae9ed33e84
Content: DbContext.Set Method (System.Data.Entity)
Content Source: EF/xml/System.Data.Entity/DbContext.xml
Product: entity-framework
Technology: entity-framework-6
GitHub Login: @dotnet-bot
Microsoft Alias: avickers

@ericallenpaul You're looking at the EF6 documentation put likely using EF Core.

Yep that's exactly the problem. Now I'm sad that they removed the method I needed. I need to create a generic version of "context.Customers" where "Customers" would be derived from a string. Can't use a variable between the "<>" so I guess I'll have to find another less generic approach.

See #21066.

Yep I need a working version of GetDbSetForClrType.
I've been trying to figure out the MethodInfo/MakeGenericMethod but I can't seem to come up with a working version.
I know I need something like:
string namespaceName = "mynamespace.models";
var myModel = Activator.CreateInstance($"{namespaceName}, {"Customer}", namespaceName);
var type = myModel?.GetType();
var listType = typeof(List<>);

if (type != null)
{
     var constructedListType = listType.MakeGenericType(type);
     var instance = Activator.CreateInstance(constructedListType);
     string json = File.ReadAllText(file.FullName);

     JsonSerializerOptions options = new JsonSerializerOptions();
     var seedList = JsonSerializer.Deserialize(json, type, options);

     MethodInfo method = typeof(DbContext).GetMethod("AddRange");
     MethodInfo generic = method.MakeGenericMethod(myType);
     generic.Invoke(this, null, seedList);

}

@ericallenpaul Note that you can call AddRange, etc. directly on the DbContext. You don't need a DbSet for that.

hmm...
So:
await context.AddRangeAsync(seedList);
await context.SaveChangesAsync();

It will figure out which table to add them to based on the type?

Yes

OK Wow. Thanks for you help.


Email Connector: Determine Data for Email / Send Email

Is your feature request related to a specific problem?
https://github.com/ethyca/fidesops/issues/1134 adds the beginning pieces for an email connector.  Here we want to expand the new email connector to be able to actually email someone to retrieve or erase data from a system.
Describe the solution you'd like

Expand the new EmailConnector.query_config, create_client, test_connection, retrieve_data, and mask_data methods to determine what fields we need to send to the user, and actually email the user
Determine if we want EmailConnector.retrieve_data and mask_data to just build the fields we need to email to the user, and send all emails at the end, or if we should email the user in the middle of the traversal. It's a little tricky because these connectors currently retrieve more data than the user needs, because some of this data is used for downstream queries, and we filter the data the user receives after-the-fact per policy.
EmailConnector.retrieve_data should at minimum figure out what fields we need to instruct an admin to email a user (see similar manual connector). Depending on previous decision, an access email might be sent here
EmailConnector.mask_data should at minimum figure out what fields we need to instruct an admin to mask/how to mask for a user (see similar manual connector) .  Depending on previous decision, an erasure email might be sent here.

A description of what you want to happen.
Describe alternatives you've considered, if any
A description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Blocked by #1134

Update: We will just implement this for "erasures" now, not "access", so "retrieve_data" should do nothing.


Prepare for scpcaTools release v0.1.7

Steps for a new release of scpcaTools
Preparing for the release and testing

[x] Are all of the issues planned for this release resolved? If there are any issues that are unresolved, mark this issue as blocked by those on ZenHub.
[x] Update DESCRIPTION with new version number.
[x] Did all automated tests and the Docker build in this repo pass in CI? You may have to wait for the docker image to build. Check Github actions before proceeding to the next step.
[x] Test the new docker build in scpca-nf

[x] Create a branch in scpca-nf for testing the latest Docker version from the main branch.
[x] Update containers.config with the edge version of the scpca-tools container: SCPCATOOLS_CONTAINER = 'ghcr.io/alexslemonade/scpca-tools:edge' and push the branch to github.
[x] If necessary, update the scpca-nf workflow to accommodate changes in the scpcaTools package
[x] Perform a test run of the workflow with:
nextflow run AlexsLemonade/scpca-nf -r "<TESTBRANCH>" -profile batch
If the default run_ids for the workflow do not cover expected changes in this tools package, you may want to specify particular test samples or projects with the --run_id option (use your judgement).

Creating a release

[x] On the releases page, choose Draft a new release.
[x] In Choose a tag, type a new release number using semantic versioning (vX.X.X) (you did update the title of this issue to match, right?), then click Create a new tag: vX.X.X on publish.
[x] Write a description of the major changes in this release. You may want to start with the Auto-generated release notes to save time.
[ ] Save a draft to return to later if testing isn't done yet, otherwise:
[x] Publish the release!

Confirmed that the implemented changes worked as expected by testing scpca-nf and created the new release. Closing this.


teach buildLinux some version magic

Motivation for this change
This is more of an RFC at the moment. I would like to hear some opinions on whether this is a good idea, and why it wasn't done like that before.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

cc @NeQuissimus

It occurred to me that another direction would be to pass structured version information (major, minor, patch, postfix) to buildLinux and then assemble the version there.


Read only first row in settings is not updated when adding row on top

Description
If you want to disable a row in an HOTable you can do it in the settings like so :
        cells: (row, col, prop) => {
          const cellProperties: any = {};
            if (row === 0) {
              cellProperties.readOnly = true;
            }
          return cellProperties;
        }

If you then add a row above, the read only status stayed on the row that is now not the first row anymore.
This behaviour works in plain javascript, but not in the angular wrapper. (see demo)
Demo
Angular example with issue:
https://stackblitz.com/edit/handsontable-angular-hooks-wunmdb
Javascript example without the issue:
http://jsfiddle.net/75z1b38s/
Your environment

Angular version: 5.2.0
Handsontable version: 2.0.0
Wrapper version: 1.0.0-beta5
Browser Name: Chrome
Operating System: OSx

They work the same way on Windows (win 10/Chrome 66). Maybe there's something with the Mac. Thanks! We'll check that.

I just got the same result on Mac. Both examples maintain the read-only state for the row where they have been set and for a new row when added to a position with an index of 0.
Do you want the newly added row to be read-only and erase the state for the old row?

Yes, strange, this happened yesterday, but not now. How can I erase the state of the old row?

To erase it you need to add ELSE
cells: (row, col, prop) => {
          const cellProperties: any = {};
            if (row === 0) {
              cellProperties.readOnly = true;
            } else { cellProperties.readOnly = false; }
          return cellProperties;
        }

Great, thanks @AMBudnik

you're welcome @Mathijs003


Has Anyone Successfully had android push notifications working?

has anyone been successful with getting android notifications going? I followed steps in the repo and doesn't work for me.
If you did get it working would you mind assisting me?

how did you fix it?

has anyone been successful with getting android notifications going? I followed steps in the repo and doesn't work for me.
If you did get it working would you mind assisting me?

how did you fix it?


RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:1 and cpu!

When I tried to resume training from the checkpoint file, I encountered with a runtimeError.
The error message is as below:
Traceback (most recent call last):
  File "tools/train.py", line 243, in <module>
    main()
  File "tools/train.py", line 218, in main
    train_metrics = train_epoch(epoch, model, loader_train, optimizer,
  File "/opt/tiger/cream/tools/../lib/core/train.py", line 68, in train_epoch
    optimizer.step()
  File "/usr/local/lib/python3.8/site-packages/torch/optim/lr_scheduler.py", line 67, in wrapper
    return wrapped(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 26, in decorate_context
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/torch/optim/sgd.py", line 106, in step
    buf.mul_(momentum).add_(d_p, alpha=1 - dampening)
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:2 and cpu!

Part of my train.yaml is as follows:
AUTO_RESUME: True
DATA_DIR: '/opt/tiger/data/ImageNet'
MODEL: 'Supernet_Training'
RESUME_PATH: './experiments/workspace/train/11041800-Supernet_Training/checkpoint-0.pth.tar'
SAVE_PATH: './experiments/workspace/train'

How can I deal with this error and resume training using the checkpoint file?

I solved this problem through modified the order of the codes.
model_est = FlopsEst(model)

    # create optimizer
    optimizer = create_optimizer_supernet(cfg, model, USE_APEX)

    # create learning rate scheduler
    lr_scheduler, num_epochs = create_supernet_scheduler(cfg, optimizer)

    model = model.cuda()

    # optionally resume from a checkpoint
    resume_epoch = None
    optimizer_state = None
    if cfg.AUTO_RESUME:
        optimizer_state, resume_epoch = resume_checkpoint(
            model, cfg.RESUME_PATH)

In other words, I create the lr_scheduler and the optimizer before loading the model from the checkpoint file. Also, I transfer the model to cuda devices before loading.


inconsistent treatment of emoji completions

JuliaLang/julia#10709 (@jakebolewski ) added emoji tab completions to the Julia REPL, e.g. \:koala: tab-completes to 🐨.  Since IJulia uses the same tab-completion functions, it gets the same completions, but something weird happens when you type \:koala:<TAB>:

Note that it oddly thinks that there are two completions, not one.  Not sure where this is coming from. It doesn't happen in the REPL, and Base.REPLCompletions.completions("\\:koala:", length("\\:koala:")) returns only a single completion.

The message that is being sent to IPython (via IJulia.set_verbose(true)) is:
SENDING IPython Msg [ idents C485926233F4497B8BCC8AEEB9E87167 ] {
  header = Dict("msg_id"=>"ae06dfec-2397-4c20-9725-01d130d72cb9","msg_type"=>"complete_reply","username"=>"username","session"=>"C485926233F4497B8BCC8AEEB9E87167"),
  metadata = Dict{AbstractString,Any}(),
  content = Dict{ASCIIString,Any}("matches"=>UTF8String["🐨"],"status"=>"ok","matched_text"=>"\\:koala:")
}

which only has a single match.   So, I suspect that something funky is happening on the IPython side here.  @Carreau, @minrk?

🐔

This might be a failure in the adaptation from the old v4 completion messages to v5.

@minrk, where in the IPython codebase is this?  I just ran into more weirdness today for 𝒦.  Like  🐨, this is outside the BMP, so it makes me think that you have a bug where you are using a UTF-16 encoding and are confusing 16-bit code-units for code-points.

Sorry for dropping this one. The koala completion is coming from the CodeMirror context hints. The adaptation is doing nothing wrong as far as I can tell. CodeMirror is seeing :koala as an identifier, and providing :koala as a local completion on the second :. I'm guessing this is a bug in CodeMirror's Julia mode.
What was happening with 𝒦? What were you typing and what did you see? I don't see any other unicode problems, and UTF16 is highly unlikely.

@minrk, I don't remember exactly; I was doing something like \mscrK<tab> (𝒦) and some editing, and I remember at one point it seemed like a couple of characters suddenly disappeared.  But I can't seem to reproduce it.  I'll file a new issue if I can figure out what happened.
@malmaud, can the CodeMirror mode be fixed?

I'll take a look.

I have the same issue, 7 years later


Aboute intermediate CAs

Hello, is it possible to create intermediate CAs with fablo?

Right now we don't support it. For now we have to certificate chain at all -- all certs are generated by cryptogen command. We have an issue to get towards certificate chain though - #318


Update copy

We have gone through the copy and
updated where deemed applicable!

 All committers of the pull request should sign our Contributor License Agreement in order to get your pull request merged.

Coverage remained the same at 95.43% when pulling cc0039852364826fd8df3cd557df3d25ff0f0521 on dfarr:master into 972a303108b08317b3cad72bf60a56f04390b36a on cla-assistant:master.

awesome, thanks!

:+1: thank you @dfarr !


Implement const matrix and quaternion types

Need to implement const matrix and quaternion types, similar to the const vector types.

Kamino closed and cloned this issue to demianmnave/CML


💡[Feature]: Traffic Prediction model

Description
This project Traffic Prediction aims to explore and model traffic data from four junctions in a city to predict traffic patterns and help reduce congestion. By analyzing the number of vehicles at different times, the project can assist city planners in making data-driven decisions to improve infrastructure and traffic management.
Type of change

[x] Added a new machine learning notebook.
[x] Documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

can u link the issue? i dont remember assigning an issue related to this

@sanjay-kv thnks for merging PR. But I think you mistakenly forgot to add proper labels to PR. Kindly add!


Able to add new data to the dataset

Would i be able to add extra data to my dataset after already starting it? or would that create errors or learning errors?

The data loader reads the image list at the beginning.
There is no error but even if you add extra data, they are not used for the training.


Improve accuracy of documentation on TestSettings.offline_expected_qps

https://github.com/mlperf/inference/blob/876c6e2e390b188d69675a59a71360ab39007bde/loadgen/test_settings.h#L219
Perhaps a minor issue, but it confuses me: Doesn't loadgen generates 10% more samples , not queries (which doesn't seem to make sense in the offline mode), than it thinks it needs to meet the minimum test duration?

On that thought, perhaps the offline_expected_qps should be renamed as offline_expected_throughput, since it's really specify expected no. samples per second, not queries per second?

Another similar "typo" seems also on the following line:
https://github.com/mlperf/inference/blob/876c6e2e390b188d69675a59a71360ab39007bde/loadgen/test_settings.h#L274
Am I understanding it correctly that specifying performance_issue_unique = true shall cause QSL to load unique samples , not just unique queries ? In principle, 2 queries with overlapping but not the same sets of samples are still unique to each other. From how loadgen.c uses this flag, it seems in fact unique samples will be used?

Also here:
https://github.com/mlperf/inference/blob/876c6e2e390b188d69675a59a71360ab39007bde/loadgen/test_settings.h#L278
the "same query" should really be "same sample" in this sentence?


promtail - log data transfer to kafka(using ingester)

Development of separate components for ingester for log

I thought about adding a function to clymene-ingester, but decided to separate a separate component for MSA. Added promtail-ingester component.


i18n: add & fix ko translations

Comments are very helpful.

Comments are very helpful.

If there's any confusion, please let me know.


Linked list

[x] Create a Node class that has properties for the value stored in the Node, and a pointer to the next

[x] Node.Within your LinkedList class, include a head property. Upon instantiation, an empty Linked List should be created.

[x] Define a method called insert which takes any value as an argument and adds a new node with that value to the head of the list with an O(1) Time performance.

[x] Define a method called includes which takes any value as an argument and returns a boolean result depending on whether that value exists as a Node’s value somewhere within the list.

[x] Define a method called toString (or str in Python) which takes in no arguments and returns a string representing all the values in the Linked List, formatted as:
"{ a } -> { b } -> { c } -> NULL"

[x] Any exceptions or errors that come from your code should be semantic, capturable errors. For example, rather than a default error thrown by your language, your code should raise/throw a custom, semantic error that describes what went wrong in calling the methods you wrote for this lab.

[x] Be sure to follow your language/frameworks standard naming conventions (e.g. C# uses PascalCasing for all method and class names).

[x] Utilize the Single-responsibility principle: any methods you write should be clean, reusable, abstract component parts to the whole challenge. You will be given feedback and marked down if you attempt to define a large, complex algorithm in one function definition.

Write tests to prove the following functionality:

[x] Can successfully instantiate an empty linked list

[x] Can properly insert into the linked list

[x] The head property will properly point to the first node in the linked list

[x] Can properly insert multiple nodes into the linked list

[x] Will return true when finding a value within the linked list that exists

[x] Will return false when searching for a value in the linked list that does not exist

[x] Can properly return a collection of all the values that exist in the linked list


[BUG]Build Error in Next JS - CSS import issue

Describe the bug
I am using the package and my app is built on Next js. When I try to build, I am getting the below error.
`./node_modules/bulma/css/bulma.css
Global CSS cannot be imported from within node_modules.
Read more: https://err.sh/next.js/css-npm
Location: node_modules/pdf-viewer-reactjs/dist/pdf-viewer-reactjs.js
./node_modules/bulma-helpers/css/bulma-helpers.min.css
Global CSS cannot be imported from within node_modules.
Read more: https://err.sh/next.js/css-npm
Location: node_modules/pdf-viewer-reactjs/dist/pdf-viewer-reactjs.js
./node_modules/bulma/css/bulma.css
Module build failed: Error: Final loader (./node_modules/next/dist/build/webpack/loaders/error-loader.js) didn't return a Buffer or String
./node_modules/bulma-helpers/css/bulma-helpers.min.css
Module build failed: Error: Final loader (./node_modules/next/dist/build/webpack/loaders/error-loader.js) didn't return a Buffer or String
./node_modules/pdf-viewer-reactjs/dist/pdf-viewer-reactjs.js
Module not found: Can't resolve 'material-design-icons/iconfont/material-icons.css' in '/home/ubuntu/frontend/node_modules/pdf-viewer-reactjs/dist'
`
Details :

Device: AWS ec2 - ubuntu

Additional context
Solution from next js : https://github.com/vercel/next.js/blob/master/errors/css-npm.md

Checking the issue, will fix fix in the next release

same problem


TextField with null decoration

InputDecorator.containerOf() returns null in this case (as it should), so use the Editable's RenderBox instead.
Fixes https://github.com/flutter/flutter/issues/16880


[WIP] E2E test-with-helm wait for port-forward

The E2E test-with-helm intermittently fail with ConnectionError if the port-forward for the modelmesh sklearn test does not get established in time for the inference request.
requests.exceptions.ConnectionError: HTTPConnectionPool(host='localhost', port=8008): Max retries exceeded with url: /v2/models/isvc-sklearn-modelmesh/infer (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x12d011c30>: Failed to establish a new connection: [Errno 61] Connection refused'))

Thanks @yuzisun -- I forgot to close this one


Migration on existing MySQL table cloumn, "There is no column with name 'fee' on table 'transactions'."

Laravel Version: 6.0.3 & 5.9.x
PHP Version: 7.3.9
Database Driver & Version:
MySQL 5.6

Description:
When running a change migration to make a field nullable I receive an error about there being no column with the name of the existing field; this happens with our normal user and I've tested with a root MySQL user as the connection.
Steps To Reproduce:
CREATE TABLE `transactions` (
  `id` varchar(50) COLLATE utf8mb4_unicode_ci NOT NULL,
  `amount` int(11) NOT NULL,
  `fee` int(11) NOT NULL,
  `created_at` timestamp NULL DEFAULT NULL,
  `updated_at` timestamp NULL DEFAULT NULL,
  PRIMARY KEY (`id`,`payment_provider`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_unicode_ci

Run migration
<?php

use Illuminate\Support\Facades\Schema;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Database\Migrations\Migration;

class TransactionsNullFees extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::table('transactions', function (Blueprint $table) {
            $table->integer('fee')->nullable()->default(null)->change();
        });
    }

    /**
     * Reverse the migrations.
     *
     * @return void
     */
    public function down()
    {
        Schema::table('transactions', function (Blueprint $table) {
            $table->integer('fee')->change();
        });
    }
}

I can't reproduce your question~ Are you sure the steps you provided are completed?

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


targetclosedexception after updating chromium

EDIT:
It looks like there are already a pull request to fix this issue:
https://github.com/hardkoded/puppeteer-sharp/pull/1496
Any chance of getting a patch release 2.0.4 that includes this fix?
===
Following an upgrade of chromium I get the following stack trace.
This is from code that has worked without issue for a long time.
Exception: PuppeteerSharp.TargetClosedException: Protocol error(DOM.describeNode): Target closed. (NetworkManager failed to process Network.requestWillBeSent. Value cannot be null.
Parameter name: key.   at System.Collections.Concurrent.ConcurrentDictionary`2[TKey,TValue].ThrowKeyNullException () [0x00000] in <285579f54af44a2ca048dad6be20e190>:0
  at System.Collections.Concurrent.ConcurrentDictionary`2[TKey,TValue].GetOrAdd (TKey key, System.Func`2[T,TResult] valueFactory) [0x00008] in <285579f54af44a2ca048dad6be20e190>:0
  at PuppeteerSharp.Helpers.MultiMap`2[TKey,TValue].Add (TKey key, TValue value) [0x00000] in <d6856fbf863048ac933d0559e16ac833>:0
  at PuppeteerSharp.Helpers.AsyncDictionaryHelper`2[TKey,TValue].GetItemAsync (TKey key) [0x0003b] in <d6856fbf863048ac933d0559e16ac833>:0
  at PuppeteerSharp.NetworkManager.OnRequestAsync (PuppeteerSharp.Messaging.RequestWillBeSentPayload e, System.String interceptionId) [0x000fc] in <d6856fbf863048ac933d0559e16ac833>:0
  at PuppeteerSharp.NetworkManager.OnRequestWillBeSentAsync (PuppeteerSharp.Messaging.RequestWillBeSentPayload e) [0x00152] in <d6856fbf863048ac933d0559e16ac833>:0
  at PuppeteerSharp.NetworkManager.Client_MessageReceived (System.Object sender, PuppeteerSharp.MessageEventArgs e) [0x00299] in <d6856fbf863048ac933d0559e16ac833>:0 )
   at async PuppeteerSharp.CDPSession.SendAsync(?)
   at async PuppeteerSharp.CDPSession.SendAsync[DomDescribeNodeResponse](?)
   at async PuppeteerSharp.ExecutionContext.AdoptElementHandleASync(?)
   at async PuppeteerSharp.Frame.WaitForXPathAsync(?)

pupeeter-sharp: 2.0.3 (problem also exists with 2.0.0)
chromium: 83.0.4103.116 (working version 80.0.3987.162-1=deb10u1)
os: debian 10.4/mono

Seeing the same issue

@kblok #1521 Solves this, Can we get a hotfix or something?

Cool @kerohero. I will ship a hotfix tomorrow ;)

@kerohero  PuppeteerSharp.2.0.4 shipped!


BUG-101213 Update NiFi blueprint name in API E2E

@afarsang please review: BUG-101213 Update NiFi blueprint name in API E2E suite files
@schfeca75 and @keyki FYI

You are right, this should go first to rc-2.6. So I'm closing this here and create a new one there.


Switch to running fullGC as false

AB#189
Resolves #10015
There's a buggy version of snapshots right now where we have duplicate GC outbound routes in the GC blob. Create a snapshot test for this scenario, as we are making a snapshot level change.
Setting fullGC = false will already generate these issues for existing snapshots.
Point at the commit that has these duplicate route issues.

Closed in favor of just doing it all in one shot


chore: fix TestBuiltinEventTimeExtractor e2e test

With idle watermark in place, the watermark propagation becomes faster than before, that it seems the edge watermark can be the event time of the latest message. Since the purpose of this specific test case is to verify the event time is extracted from the message payload, I am updating the test to pass if the watermark is the event time extracted from the latest message.
Closes #632

We have a fix in the reduce wmb PR for the UI's GetHeadWatermark
https://github.com/numaproj/numaflow/pull/627/files#diff-6af877f3295078204591f48a2180bc855394b50997f764275b25ef2ffff678daR107-R114
I think this e2e test fix PR won't be needed once we merged PR627

#627 should fix this.


Adding implementation for Android CPU/GPU/Memory profiling

We have CPU/GPU/Memory profiling for iOS, which was enabled through -
CPU: flutter/engine#18087
GPU: flutter/engine#18752
Memory: flutter/engine#18516
With the same profilings available on Android, we will have consolidated way to measure Flutter app's performance through Flutter driver test.
@iskakaushik

Albert was interested in knowing an ETA for this issue so he can make plans on updating the tracked benchmarks accordingly. Given that I will not be having time to tackle this in Q3, I'm not sure when I will be able to get to this. cc: @zanderso for help with prioritization.

/cc @xster @gaaclarke

FYI @blasten We don't currently track CPU utilization on Android for benchmarks like this one. This came up in the context of https://github.com/flutter/flutter/issues/83113.

Putting this on the list for Impeller since we'll need to be able to compare with benchmarks on memory footprint.

Is there any roadmap for this please?


define schools choice for volunteers

school names are missing on the search bar.Immo, I've been thinking it might be useful to add this data on it. Then you need to colored schools with volunteers or schools with no volunteers. ıdk. Dropping here data you can pull from.
https://www.meb.gov.tr/baglantilar/okullar/index.php

team decision; to close schools and show only neighborhoods.


Can't configure FetchType of Embeddable Collection in Entity

Hi,
Embeddable collections are generated as @ElementCollection field annoted.
Annotation @ElementCollection accept fetch parameter, but there's no way to configure that.
I think add this property is a good enhancement. But, I don't know if this parameter are take care by JPA.
p.s.: in last week I post few "Issues" here. It's because I'm loved your plugin and you made a great job. Thank you. If I have some time some day, I'll try create some functions for JPAModeler. :P

Thanks for reporting the issue


[FINE] Ensure that a not found is returned on DELETE action

https://bugzilla.redhat.com/show_bug.cgi?id=1477520
Reopened version of https://github.com/ManageIQ/manageiq/pull/16421
@miq-bot assign @simaishi
@miq-bot add_label bug

@abellotti is this good to go?


Update impersonation_sharepoint_body_credential_theft.yml

2nd idea, adding suspicious subject check.

cc @aidenmitchell  mimic fails here :/


Bug 2056638: [4.9] Eliminate parallelism

And increase provisioning timeout.
This is cherry-pick of https://github.com/openshift/aws-efs-csi-driver-operator/pull/29 and https://github.com/openshift/aws-efs-csi-driver-operator/pull/28 to release-4.9. It should help with slow deletion of PVs.

/label backport-risk-assessed

/lgtm

/assign @duanwei33
for cherry-pick-approved

/label cherry-pick-approved


Github actions integration tests use docker image instead of python local setup

Change CI integration tests to using docker images and a docker-compose setup.
This makes the local python setup obsolete.
Closes #860

I have just checked the artifacts. I think it would also be helpful if we add the logs of the node to the artifacts if it's not too much effort.

The node logs are also part of the .zip file - at least with the changes of the latest run now

My understanding is limited here, so I focused on the log outputs and they look fine. Final question: Have you tried a failing clippy / failing test to see if the CLI still notices it? That's the main point IMO.

The clippy job has not been touched, so that should work as before.
Integration test failures are definitely picked up, I had many of them while working on this PR
Cargo test failures are picked up in the build phase - I don't have an example in GitHub actions to demonstrate, but locally (where I run the same docker build command, I've observed it a number of times)

My understanding is limited here, so I focused on the log outputs and they look fine. Final question: Have you tried a failing clippy / failing test to see if the CLI still notices it? That's the main point IMO.

* The clippy job has not been touched, so that should work as before.

* Integration test failures are definitely picked up, I had many of them while working on this PR

* Cargo test failures are picked up in the build phase - I don't have an example in GitHub actions to demonstrate, but locally (where I run the same `docker build` command, I've observed it a number of times)

Cool, thanks for answering in such detail. You have my good to go for merging 👍


Adding Favicon icon and changing the Title

assign this to me

I have a prior experience in this task, please assign this to me


Mark test_offline_node the last one to be executed

to prevent cascading failure
for #3851
Signed-off-by: Yang Chiu<EMAIL_ADDRESS>

@yangchiu as we discussed, needs to backport to 1.2.x and 1.1.x


Visualizations: "No data" legend styling

Discussed in 9/13/2021 EDA WG meeting
Problem: If the window is small or a user is reading quickly “No data” could be confused with values of “No”
Decision: make “No data” visually different (change font to italics) and show this on the bottom, below all other values in the legend
This should not change where in the graph the “No data” overlay is appearing

In histograms, no data should be on the top of the other bars
In boxplots, no data should be after boxes for other values

BM noted that we are making our own VEuPathDB version of legends in Phase 2, so we need to be careful to not spend a lot of time reworking the plotly legend now
@bobular  will research if there is an easy way to implement legend order in plotly (rank) and text format (italics)

Reassigned as medium cost.
The legendrank plotly parameter seems to be new, and not available in the version of plotly.js we are using - nor in the latest version available with yarn add plotly.js.
Nor does it seem to work in the latest CDN version of plotly
https://codepen.io/bobular/pen/BaZdgmY
(you can try to switch the order by adding legendrank: number to trace1 and trace2 and it makes no difference.
Given that I can't test it - I'm not even sure if this parameter switches the legend order independently of the bar order (which is what we need).
I suggest we mark this ticket as "won't do" and instead we engineer this into the standalone legend for Phase 2.

A "roll our own" legend task should probably precede this one.

“No data” appears in italics and and below all other values in the legend. Passes QA.


SVG: Proper bundling

Concept: each folder under src/gfx/svg should represent a sprite where each file in a folder represents an icon belonging to that sprite.
Issue: when you have 2+ folders it does not bundle correctly.
Reason: in globing all the files go in one stream, which leads to folder analysis being wrong. To cut the long story short, we cannot pass icons from all folders to the glob.
Solution: analyze the folders, create a sprite Object, which will contain sprite names and glob paths to the SVG files within the folders. Then loop through the globs and run already existing gulp task through each glob.
The suggested solution was implemented on Conrad. I have tested it in terms of performance and did not find any significant difference between current state and the suggested one. If I am not mistaken, Gulp pipes will run async, unless chained. This leads to two separate gulp pipelines running simultaneously, not affecting performance. Please, correct me if I am wrong assuming that.
The solution will be refactored and a new pull request will be created regarding this issue.

close #109


Update to yarp v2.0.0

Hi,
The new 2.0.0 version is released, can you update ?
https://github.com/microsoft/reverse-proxy/releases/tag/v2.0.0
Thamks
👍

ok, wait a days.

Hi, The new 2.0.0 version is released, can you update ? https://github.com/microsoft/reverse-proxy/releases/tag/v2.0.0
Thamks 👍

already update


How to backup samsung galaxy contacts to computer?

There are many Samsung phone users, and to keep your phone contacts safer you would want to move them to your computer, but we can't do that just via a USB cable, we need an android file manager to help. I found Coolmuster Android Backup Assistant really helpful.
Samsung contacts backup, follow this:
Step 1: Connect Samsung Phone to PC and Run the Problem
Connect your samsung phone to computer via USB cable, then start the Coolmuster Android Assistant. The problem will detect your samsung phone automatically and you'll see some detailed info of your samsung phone in an interface.
Tips: If this is the first time your samsung phone connecting with the problem, you'll need to enable USB debugging first. You just need to follow the guide to open the USB debugging and install USB driver on your cell phone. Once the USB debugging finished, you'll see an interface just like above.
Step 2: Select and Check Contacts and Start Export
Now you can click "Contacts" on the top to view all of your samsung phone contacts, select and check those you want to backup, and just click "Backup" to start the export.
OK! That's all!
http://www.transfer-motorola-recovery.com/how-to/how-to-transfer-contacts-from-samsung-to-computer.html

Google’s owns the Android operating system so you can backup android phone data to Google account.  If you’re signed in to your Google account,you can sync phone data ,like these guides: Settings > Accounts > Google > Your Google Account .
http://www.samsung-messages-backup.com/resources/transfer-text-messages-from-galaxy-to-computer.html

Did you just buy the latest version of Android smartphone? The first thing that you'll probably do is to get all the phone numbers and other contact information on your new Android phone, such as Samsung Galaxy S6/S5, LG G4/G3, etc. It is exciting to say goodbye to an outdated handset and enjoy using a new one. Yet, to get previous stored contacts back to cell phone or even transfer them to a computer for backup is like an unsolvable math which confuse most of the phone users.
This tutorial will show you how to transfer contacts from Android to PC with the help of Android contacts recovery. You may be able to copy complete information of contacts including names, numbers and call history. Some other things it offers like How to recover sms from Samsung galaxy/note and many more.
More information:
How to copy contacts from Android to iPhone 6s plus

Samsung Galaxy Recovery is the outstanding data recovery and management tool that can helps to backup and restore Samsung phone in a click.
Read more:
Backup data from Samsung to PC
Restore deleted data from Samsung S7 edge
How to backup and restore Samsung data
Read the video guide here:

kh snkjhdjnsjkdh jknfjhjkdsn jlkhsd
sd kjhsdjsd dd

Thanks for your post, iPubsoft Android File Transfer also works great, it's a special designed android data backup app, just few simple steps to transfer everything from your phone to computer.

Android Mobile Manager makes it easy to backup contacts from Samsung Galaxy and computer without losing anything. It is fully capable of transferring almost everything from Samsung to PC for a backup, including photos, videos, music, contacts, text messages, etc.


String:First and last occurence of given letter in string

Is your feature request related to a problem? Please describe.
Given String we need to find the occurence of given letter.It is most commonly asked question in interviews.

@Kumar-laxmi Please assign me this issue under SSoC'23. I will implement this using a python program that efficiently finds the first and last occurrence of a given letter in a given string.

@Kumar-laxmi  sir please assign me this question. I am able to code it using JAVA and its string methods which will simply produce the output .
for eg : coca cola
here first occurence of letter 'o' is at index 1 and position 2 ,and last occurence of letter 'o' is at index 6 ie at position 7.
kindly assign me to solve this using JAVA string methods.

@Soumya6Tiwari and @SaketGudimella please find some other issue as I raised this issue for myself.Thankyou.


bump works on package.json but git doesn't see the change!

I have a task called bump-npm:
gulp.task('bump-npm', function() {
  return gulp.src('./package.json')
    .pipe(bump())
    .pipe(gulp.dest('./'));
});

it works in so far as it bumps the version from 0.0.x to 0.0.x+1. Unfortunately it must be doing it in some sort of "sneaky" way as my git status remains unchanged even though the package (not surprisingly is NOT in .gitignore).
Here's a video demonstrating this:

video link

I suspect it may be down to bump-npm not modifying the file's date.

Excellent video, thank you.
What filesystem are you using?
I notice the Volumes folder, is it a separate drive which might have a different user, though you have write access?
This seems related, rather recent as well: https://github.com/gulpjs/gulp/issues/1414

I'm on OSX and the Volume you refer to is a permanent RAID array which I use for all of my development work. Nothing is definitive but I think permissioning and ownership is probably not the issue.
WRT to the issue you pointed to ... I have a stronger affinity to that. I must admit I've only started using Gulp in the past week and Gulp 4.0 in the past few days. I guess if the final pipe to gulp.dest('./') is the function actually responsible for dropping the file on the filesystem then it would make sense that it is the responsible party.
For the time being I have solved my own problem by adding a quick shell out to the command line where I'm just touching the file to get it's change recognized. Works but avoids the real solution. :)

It's pretty bad that you need to add that shell command...
Does any other gulp task with gulp.dest cause file changes?

As this is a backend micro-service rather than anything needing standard frontend tooling (e.g., uglify, concatination, SASS transpiling, etc.). Although on further inspection I see that the one useful test is in my Babel transpile. Let me see if I get the same result. Back in a moment.

Oh, just realised that this test doesn't work because the 'clean' clears out the directory so of course the filesystem's date is seen as new. :(

Ignore that last ramble, easy to get around that. I have found that in both cases the filesystem's date is left unchanged. Interesting. I guess maybe this issue can be closed but this is definitely a big issue for gulp 4. Hope it gets fixed soon.

@ksnyde thanks for linking back to the vinyl-fs issue.  This is actually a huge issue that we are trying to solve and then gulp 4 should be ready for primetime.  It's a pretty complex issue but I hope we are able to wrap it up soon.

@phated best of luck ... loving the other aspects of Gulp 4

Temporary workaround: Use gulp-touchand add
...
.pipe( plugins.touch() )

at the end.

This is fixed, right? @phated

@stevelacy I don't think this is fixed, but it's not gulp-bump's issue either way.

Is there a gulp 4 issue we can follow for this?


[Downloads] images that make it to the end without being processed are not detected as errors [Cat Knight]

Caused by downloaded image status not being updated because of a crash in the Volley error handler itself
Fixed by 62331b14778b6a9d8ccacf53fb06637be06efd97


Please enter the following

I just started playing with MM and wanted to include MMM-trakt into my config. Install was super easy and I can see the module loading, but I alway get "Please neter the following..." with undefined as a code.
I tried generating a new app, triple check that my API keys are correct, but no luck. Any ideas what should I check?
Regards, D

Try to remove the token.json in the MMM-trakt folder and try again, you should get a new token then. If not report again so I can look deeper into it. I will add some better checks for this problem in a future release.

Hi, sounds like a plan, except I don't (yet) have a token.json file.
[image: image.png]
On Mon, Mar 19, 2018 at 10:42 AM Kiina<EMAIL_ADDRESS>wrote:

Try to remove the token.json in the MMM-trakt folder and try again, you
should get a new token then. If not report again so I can look deeper into
it. I will add some better checks for this problem in a future release.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Kiina/MMM-trakt/issues/5#issuecomment-374153751, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAImvKZT2ZSeFug-ORf2YlbOJBRvQfZCks5tf32CgaJpZM4SvccQ
.

--
LP, David
https://vidmar.net

Sorry for the late answer. Well either your mirror has no Internet connection or for some reason has trouble talking to the trakt api. I have added a small logging feature, can you pull the latest version and send me the complete console output from the mirror (you can omit stuff like your api keys). Make sure you have no token.json in the MMM-trakt folder. The logs should tell whats wrong (I hope)

Thanks for getting back to me. I pulled last version, rechecked if I have a token.json (I don't) and hopped to see what is wrong in log, but there are no logs.
I started looking around the code, adding bunch of console.logs and from what I can see, the importoldtoken().catch happens (no token.json, so figures), but then I see TypeError in logs when trakt.get_codes() is called.
I can see that config picks up corrent codes from config.
Any idea what to check next?

Ah the problem seems to be with the electron downgrade in MagicMirror v2.2.0. When your plattform allows it you can workaround via npm install<EMAIL_ADDRESS>while I look into the exact issue.


Trap handling dereferences a null pointer

Moved from https://github.com/bytecodealliance/wasmtime/issues/829
Running on a Linux machine, using 448faed5ca0e38c3ee5cdbe703b9f89f1e76e8a0 I am consistently observing a segfault caused by dereferencing a null pointer somewhere in libgcc while creating a backtrace. Wasmtime is used through wasmtime-api.
* thread #1, name = 'sc_executor-f4f', stop reason = signal SIGSEGV: invalid address (fault address: 0x8)
  * frame #0: 0x00007ffff72185df libgcc_s.so.1`___lldb_unnamed_symbol41$$libgcc_s.so.1 + 1055
    frame #1: 0x00007ffff721a138 libgcc_s.so.1`_Unwind_Backtrace + 104
    frame #2: 0x0000555555a68137 sc_executor-f4fe4729a78e1cc7`backtrace::backtrace::trace::h1adcaa00054f7e2c + 87
    frame #3: 0x0000555555a67a56 sc_executor-f4fe4729a78e1cc7`backtrace::capture::Backtrace::new_unresolved::hcc88b2186746cdd0 + 86
    frame #4: 0x0000555555710c9f sc_executor-f4fe4729a78e1cc7`RecordTrap + 239
    frame #5: 0x00005555557128ff sc_executor-f4fe4729a78e1cc7`WasmTrapHandler(int, siginfo_t*, void*) + 431
    frame #6: 0x00007ffff7433890 libpthread.so.0`___lldb_unnamed_symbol1$$libpthread.so.0 + 1
    frame #7: 0x00007ffff7fb3d84
    frame #8: 0x00007ffff7fb365a
    frame #9: 0x00007ffff7fb06a5
    frame #10: 0x00007ffff7fbe2f2
    frame #11: 0x00007ffff67f9031
    frame #12: 0x0000555555712b40 sc_executor-f4fe4729a78e1cc7`WasmtimeCallTrampoline + 128
    frame #13: 0x0000555555711517 sc_executor-f4fe4729a78e1cc7`wasmtime_call_trampoline + 23
...

the code around there looks like this
    0x7ffff72185d8 <+1048>: movq   0x98(%r13), %rax
->  0x7ffff72185df <+1055>: cmpb   $0x48, (%rax)
    0x7ffff72185e2 <+1058>: jne    0x7ffff7218580            ; <+960>
    0x7ffff72185e4 <+1060>: movabsq $0x50f0000000fc0c7, %rdx  ; imm = 0x50F0000000FC0C7
    0x7ffff72185ee <+1070>: cmpq   %rdx, 0x1(%rax)
    0x7ffff72185f2 <+1074>: jne    0x7ffff7218580            ; <+960>

The original PR https://github.com/paritytech/substrate/pull/4686

I believe this failure is the same issue as the one reported here, it looks like a null pointer is being dereferenced during _Unwind_Backtrace.
@yurydelendik do we just need to update cranelift now or do we need more changes on the wasmtime side of things?

do we just need to update cranelift now or do we need more changes on the wasmtime side of things?

just the cranelift update

With the merge of https://github.com/bytecodealliance/wasmtime/pull/926 this seems to be solved!


 It just hasn't been deployed. And there doesn't seem to be an alternative to it. So I may defer this until fall when 3.1 falls out of support.

To keep support for 3.1, I could update the target frameworks to netcoreapp3.1;net6.0 as the FrameworkReference will work for both of those.

Oh? That would be great. Thanks. That way, it's good for the end of the year.

Added back the net461 target.  .NET 6 and .NET Core 3.1 will use SignalR from the framework itself, and .NET Framework 4.6.1 targets will use SignalR from the existing SignalR package.


Support Spring Cloud Stream

Application Source Code is a Spring Cloud Stream native application:
<dependencies>
        <dependency>
            <groupId>org.springframework.boot</groupId>
            <artifactId>spring-boot-starter-web</artifactId>
        </dependency>
        <dependency>
            <groupId>org.springframework.cloud</groupId>
            <artifactId>spring-cloud-stream</artifactId>
        </dependency>
        <dependency>
            <groupId>org.springframework.cloud</groupId>
            <artifactId>spring-cloud-stream-binder-kafka</artifactId>
        </dependency>
        <dependency>
            <groupId>org.springframework.experimental</groupId>
            <artifactId>spring-native</artifactId>
            <version>${spring-native.version}</version>
        </dependency>

        <dependency>
            <groupId>org.springframework.boot</groupId>
            <artifactId>spring-boot-configuration-processor</artifactId>
            <optional>true</optional>
        </dependency>
        <dependency>
            <groupId>org.projectlombok</groupId>
            <artifactId>lombok</artifactId>
            <optional>true</optional>
        </dependency>
        <dependency>
            <groupId>org.springframework.boot</groupId>
            <artifactId>spring-boot-starter-test</artifactId>
            <scope>test</scope>
        </dependency>
        <dependency>
            <groupId>org.springframework.cloud</groupId>
            <artifactId>spring-cloud-stream</artifactId>
            <scope>test</scope>
            <classifier>test-binder</classifier>
            <type>test-jar</type>
        </dependency>
    </dependencies>

Application runs fine on JVM, as well as the native image is generated just fine:
mvn clean -Pnative-image package

However, startup of native image throws the following exception:
2021-03-31 22:11:47.221 ERROR 18378 --- [           main] o.s.boot.SpringApplication               : Application run failed

com.oracle.svm.core.jdk.UnsupportedFeatureError: Proxy class defined by interfaces [interface org.springframework.cloud.stream.annotation.EnableBinding, interface org.springframework.core.annotation.SynthesizedAnnotation] not found. Generating proxy classes at runtime is not supported. Proxy classes need to be defined at image build time by specifying the list of interfaces that they implement. To define proxy classes use -H:DynamicProxyConfigurationFiles=<comma-separated-config-files> and -H:DynamicProxyConfigurationResources=<comma-separated-config-resources> options.
        at com.oracle.svm.core.util.VMError.unsupportedFeature(VMError.java:87) ~[na:na]

Following the exception trace, I created a proxy-config.json as following:
[
  [
    "org.springframework.cloud.stream.annotation.EnableBinding",
    "org.springframework.core.annotation.SynthesizedAnnotation"
  ]
]

and provided to build:
...
<configuration>
                            <!-- The native image build needs to know the entry point to your application -->
                            <mainClass>com.dccorp.nativeapps.NativeCloudStreamsApplication</mainClass>
                            <buildArgs>
                                <!-- uncomment to generate dump file for graalvm dashboard analysis.-->
                                <!--                                -H:DashboardDump=dumpfileoversizedbuildArgs-->
                                <!--                                -H:+DashboardAll-->
                                <!-- you can provide additional params. -->
                                -H:DynamicProxyConfigurationFiles=classes/proxy-config.json
                                --enable-url-protocols=http
                            </buildArgs>
                        </configuration>
...

Now, the generated native image execution fails with following error message:
...
org.springframework.core.annotation.AnnotationConfigurationException: Method [public abstract java.lang.Class[] org.springframework.cloud.stream.annotation.EnableBinding.value()] is unsupported for synthesized annotation type [interface org.springframework.cloud.stream.annotation.EnableBinding]
        at org.springframework.core.annotation.SynthesizedMergedAnnotationInvocationHandler.invoke(SynthesizedMergedAnnotationInvocationHandler.java:92) ~[na:na]
...

Any pointers to what I might be missing here? Applications runs fine with JVM.

Given the place where this error happen:
public Object invoke(Object proxy, Method method, Object[] args) {
		if (ReflectionUtils.isEqualsMethod(method)) {
			return annotationEquals(args[0]);
		}
		if (ReflectionUtils.isHashCodeMethod(method)) {
			return annotationHashCode();
		}
		if (ReflectionUtils.isToStringMethod(method)) {
			return annotationToString();
		}
		if (isAnnotationTypeMethod(method)) {
			return this.type;
		}
		if (this.attributes.indexOf(method.getName()) != -1) {
			return getAttributeValue(method);
		}
		throw new AnnotationConfigurationException(String.format(
				"Method [%s] is unsupported for synthesized annotation type [%s]", method, this.type));
	}

I guess you need a hint like @TypeHint(types = org.springframework.cloud.stream.annotation.EnableBinding.class, access = AccessBits.FULL_REFLECTION).

FWIW, the EnableBinding and annotation-based programming model is deprecated and soon won't even be supported. So please switch to functional programing model and see if you have any errors

@olegz Yeah, I am using functional programming model only - there's just 1 function:
@Bean
    public java.util.function.Consumer<Person> appConsumer() {
        return person -> log.info("Data received {}", person);
    }

and spring boot / cloud versions:
...
<parent>
        <groupId>org.springframework.boot</groupId>
        <artifactId>spring-boot-starter-parent</artifactId>
        <version>2.4.4</version>
        <relativePath/> <!-- lookup parent from repository -->
    </parent>
...
<spring-cloud.version>2020.0.2</spring-cloud.version>
...

@sdeleuze - trying with @TypeHint results in another exception:
2021-04-01 08:44:41.397 ERROR 1352 --- [           main] o.s.boot.SpringApplication               : Application run failed

org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'IntegrationConfigurationBeanFactoryPostProcessor': Instantiation of bean failed; nested exception is org.springframework.beans.BeanInstantiationException: Failed to instantiate [org.springframework.integration.config.IntegrationConfigurationBeanFactoryPostProcessor]: No default constructor found; nested exception is java.lang.NoSuchMethodException: org.springframework.integration.config.IntegrationConfigurationBeanFactoryPostProcessor.<init>()
        at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.instantiateBean(AbstractAutowireCapableBeanFactory.java:1316) ~[na:na]

@chaudharydeepak any chance you can post a whole stack trace? I am a bit puzzled as to why are you getting it if you are not using it (the EnableBinding that is)

Sure @chaudharydeepak - attached here is the FINE log. Thank you.
app.log

Thank you @chaudharydeepak
Seems like It could be addressed in stream. I'll post here once a new snapshot is available.

@chaudharydeepak try it now, the latest snapshots of spring-cloud-stream are available - https://repo.spring.io/libs-snapshot-local/org/springframework/cloud/spring-cloud-stream/3.1.3-SNAPSHOT/
LMK

@olegz - I tried, same outcome. Attached herein is log.
app_2.log

Then, if I provided the required proxy configuration to satisfy above error via
-H:DynamicProxyConfigurationFiles=classes/proxy-config.json

as:
[
  [
    "org.springframework.cloud.stream.annotation.EnableBinding",
    "org.springframework.core.annotation.SynthesizedAnnotation"
  ]
]

, then in run logs I see following[from 3.1.3 snapshot of SCS], which is cool!
...
2021-04-01 12:44:29.661  WARN 8626 --- [           main] o.s.c.s.config.BindingBeansRegistrar     : Failed to proxy EnableBinding annotation. If you are using functional programming style, ignore this warning, otherwise, annotation-based programming model is not and will not be suppported in native images.
...

however, now, the run fails with this:
2021-04-01 12:44:29.665 ERROR 8626 --- [           main] o.s.boot.SpringApplication               : Application run failed

org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'IntegrationConfigurationBeanFactoryPostProcessor': Instantiation of bean failed; nested exception is org.springframework.beans.BeanInstantiationException: Failed to instantiate [org.springframework.integration.config.IntegrationConfigurationBeanFactoryPostProcessor]: No default constructor found; nested exception is java.lang.NoSuchMethodException: org.springframework.integration.config.IntegrationConfigurationBeanFactoryPostProcessor.<init>()
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.instantiateBean(AbstractAutowireCapableBeanFactory.java:1316) ~[na:na]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBeanInstance(AbstractAutowireCapableBeanFactory.java:1214) ~[na:na]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:564) ~[na:na]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:524) ~[na:na]
	at org.springframework.beans.factory.support.AbstractBeanFactory.lambda$doGetBean$0(AbstractBeanFactory.java:335) ~[na:na]
	at org.springframework.beans.factory.support.DefaultSingletonBeanRegistry.getSingleton(DefaultSingletonBeanRegistry.java:234) ~[na:na]
	at org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:333) ~[na:na]
	at org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:213) ~[na:na]
	at org.springframework.context.support.PostProcessorRegistrationDelegate.invokeBeanFactoryPostProcessors(PostProcessorRegistrationDelegate.java:135) ~[na:na]
	at org.springframework.context.support.AbstractApplicationContext.invokeBeanFactoryPostProcessors(AbstractApplicationContext.java:746) ~[na:na]
	at org.springframework.context.support.AbstractApplicationContext.refresh(AbstractApplicationContext.java:564) ~[na:na]
	at org.springframework.boot.web.servlet.context.ServletWebServerApplicationContext.refresh(ServletWebServerApplicationContext.java:144) ~[na:na]
	at org.springframework.boot.SpringApplication.refresh(SpringApplication.java:769) ~[com.dccorp.nativeapps.nativecloudstreamsapplication:2.4.4]
	at org.springframework.boot.SpringApplication.refresh(SpringApplication.java:761) ~[com.dccorp.nativeapps.nativecloudstreamsapplication:2.4.4]
	at org.springframework.boot.SpringApplication.refreshContext(SpringApplication.java:426) ~[com.dccorp.nativeapps.nativecloudstreamsapplication:2.4.4]
	at org.springframework.boot.SpringApplication.run(SpringApplication.java:326) ~[com.dccorp.nativeapps.nativecloudstreamsapplication:2.4.4]
	at org.springframework.boot.SpringApplication.run(SpringApplication.java:1313) ~[com.dccorp.nativeapps.nativecloudstreamsapplication:2.4.4]
	at org.springframework.boot.SpringApplication.run(SpringApplication.java:1302) ~[com.dccorp.nativeapps.nativecloudstreamsapplication:2.4.4]
	at com.dccorp.nativeapps.NativeCloudStreamsApplication.main(NativeCloudStreamsApplication.java:10) ~[com.dccorp.nativeapps.nativecloudstreamsapplication:0.0.1-SNAPSHOT]
Caused by: org.springframework.beans.BeanInstantiationException: Failed to instantiate [org.springframework.integration.config.IntegrationConfigurationBeanFactoryPostProcessor]: No default constructor found; nested exception is java.lang.NoSuchMethodException: org.springframework.integration.config.IntegrationConfigurationBeanFactoryPostProcessor.<init>()
	at org.springframework.beans.factory.support.SimpleInstantiationStrategy.instantiate(SimpleInstantiationStrategy.java:83) ~[na:na]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.instantiateBean(AbstractAutowireCapableBeanFactory.java:1308) ~[na:na]
	... 18 common frames omitted
Caused by: java.lang.NoSuchMethodException: org.springframework.integration.config.IntegrationConfigurationBeanFactoryPostProcessor.<init>()
	at java.lang.Class.getConstructor0(DynamicHub.java:3349) ~[na:na]
	at java.lang.Class.getDeclaredConstructor(DynamicHub.java:2553) ~[na:na]
	at org.springframework.beans.factory.support.SimpleInstantiationStrategy.instantiate(SimpleInstantiationStrategy.java:78) ~[na:na]
	... 19 common frames omitted

attached herein are logs:
app.log

@chaudharydeepak so I am changing the catch (Exception e) { to catch (Throwable e) { (didn't realize that com.oracle.svm.core.jdk.UnsupportedFeatureError was actually an Error. Anyway, the SNAPSHOTS are in process (will let you know). That should let you avoid doing this -H:DynamicProxyConfigurationFiles=classes/proxy-config.json i believe.
As for the other error, can you please check what version of spring-integration you have as it seems to be like a version missmatch.

@olegz - Thank you. Spring Integration version is auto pulled in as 5.4.5:

Spring Integration is undergoing some internal changes at the moment to avoid reflection for configuration as much as possible: https://github.com/spring-projects/spring-integration/issues/3502.
The fix will be available in the upcoming Spring Integration 5.5.
We also probably will add some hints into this project for more coverage.
Will link respective Spring Integration issue for Native back to here when it it is there...

Ok, I was just chatting with @artembilan (the lead for spring-integration) and unfortunately there are still some issues that need to be worked out with regard to native support for spring-integration as shown here. Perhaps @sdeleuze can chime in and see if there is anything that can be done for the spring-integration issue?
Also, new snapshots should be available, so refresh.

Thank you @olegz @artembilan . I will hold-on then up until 5.5 is released. I tried bringing latest milestone, just to see if that helps.
Also I refreshed SCS 3.1.3-Snapshot - and I don't have to provide any Hints / proxy configs etc anymore :).
Thank you.

For better traceability: https://github.com/spring-projects-experimental/spring-native/issues/469

I have a Spring Cloud Stream application that I tries to native compile with Spring Native 0.9.2 following the advice in this issue. I have upgraded to Spring Integration 5.5.0-RC1 and Spring Cloud-Stream 3.1.3-SNAPSHOT which resolves most of the problems. I'm now stuck on use of the StreamBridge. When starting up the native compiled application I get:
 2021-04-20 10:33:00.990  WARN 1 --- [           main] onfigReactiveWebServerApplicationContext : Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'productCompositeServiceApplication': Unsatisfied dependency expressed through field 'integration'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'productCompositeIntegration' defined in class path resource [se/magnus/microservices/composite/product/services/ProductCompositeIntegration.class]: Unsatisfied dependency expressed through constructor parameter 3; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'streamBridgeUtils': Unsatisfied dependency expressed through field 'bindingService'; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'bindingService' defined in class path resource [org/springframework/cloud/stream/config/BindingServiceConfiguration.class]: Bean instantiation via factory method failed; nested exception is org.springframework.beans.BeanInstantiationException: Failed to instantiate [org.springframework.cloud.stream.binding.BindingService]: Factory method 'bindingService' threw exception; nested exception is javax.validation.ValidationException: HV000183: Unable to initialize 'javax.el.ExpressionFactory'. Check that you have the EL dependencies on the classpath, or use ParameterMessageInterpolator instead

The constructor parameter #3 mentioned in the error message above is the StreamBridge. Any suggestions?

Should be fixed by b6877b0152d00941b7a25b3e19bd210e8d763de6 which provide a sample. If more is needed, please comment here with a repro project attached after checking you are using the same versions than the new cloud-stream-kafka sample.


API Problems and Requests

This is a general issue for discussing problems with the coreIR API
Please comment on this github issue with API problems labeled with C++, C, or Python
Does a function cause a segfault? (If so please file a separate issue)
Does a function cause coreIR to falsely error?
Does a function produce unhelpful error messages?
Is the name of the function confusing?
Is there some basic function you want and cannot do?

libs/stdlib.hpp has using namespace CoreIR; as one of the lines in the file. This seems like a potentially bad idea because now any file that #includes stdlib will now also be using the CoreIR namespace.

Totally agree. This should be added to the issue #32

Can confirm that passing in nullptr as the Type* to runGenerator causes a segfault.

The formatting for 2+ dimensional arrays seems a bit cumbersome. Perhaps some syntactic sugar to make specifically arrays easier to write?
  {"out",context->Array(3, context->Array(3,
                                          context->Array(bitwidth,context->BitOut())  )) }

@jeffsetter What is a proposed syntax?

I was thinking that C's array syntax is fairly standard, so something like
context->BitOut()[bitwidth][3][3]
I'm not sure if this would be annoying to implement, but it looks a little cleaner to me.

@jeffsetter I have thought about doing that, the problem is that Type is a pointer. And you would have to dereference it in order to use the [] operator.
So it would have to look like:
Type* stencil = ((*(context->BitOut())[bitwidth])[3])[3]
Which I would argue is uglier.
I could do:
Type* stencil = context->BitOut()->Arr(bitwidth)->Arr(3)->Arr(3)
which is a bit better.

Added in #48

I believe coreir.h and coreir-c.h should replace extern COREModuleDef* COREModuleGetDefs(COREModule* m); with extern COREModuleDef* COREModuleGetDef(COREModule* m);. There is no such function named COREModuleGetDefs.

There is currently no way to determine the bitwidth of a connection in python beyond inspecting the selectpath of both end points and searching for 'data' or 'bit' and making the assumption that:
*.data.out -> *.data.in[*] implies 16 bit connection and
*.bit.out    -> *.bit.in[*] implies 1 bit connection
It would be preferable if connections had their width.

@caleb, opened new issue #71

C interface segfaults when calling CORELoadModule on _add4Gen.json from unit tests.


Project still maintained?

Last release was more than a year ago, not seeing issues being resolved. Wondering if this project is still maintained?

Dead


Wearing rings?

From 12.0.b I remember there was an odd command for "wearing" rings.
I cannot find a command for that in the help text in your version.

Hi Michael -- As far as I've seen in the versions I've played and the source code I've read (12.0, 12.3, 12.4), rings don't need to be "worn". They start having an effect as soon as they are picked up. I hope that helps!


How to set some dates can not to be selected

This is in the README and sample project.

This is in the README and sample project.


tensorboard support for autotrain-advanced

tensorboard is available by default. you can just launch tensorboard in the project_folder/logs


feat(github): Automatically generate changelog

Description
https://docs.github.com/en/repositories/releasing-projects-on-github/automatically-generated-release-notes#configuring-automatically-generated-release-notes
We can add a new file .github/release.yml to declare the expected changelog generation logic.
For example, we should move commits with feat to the new features section.

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

To use configuring-automatically-generated-release-notes, we firstly need to label the pr.
To lable pr, we can use the bot Mergify or GitHub Action TimonVS/pr-labeler-action. I'm not sure the bot we use now Muniu are up to the task of labelling.

We should reuse this script: https://github.com/tensorchord/envd/blob/main/hack/changelog.sh

We should reuse this script: https://github.com/tensorchord/envd/blob/main/hack/changelog.sh

It looks like this script is used to generate minor version released change log.
This https://github.com/tensorchord/envd/pull/1145 will give a feature to automatically generate the release change log.
Now it seems that the format of the minor version change log is different from the release page's change log.
Do you mean that we need add feature for this script to classify different type commit in the minor version changelog?

Thank you for the PR! However, your approach in #1145 would require every PR to have a label. I don't think that's necessary since PR titles already have that categorization. Here's what I think we should do:

In the release page, add a link redirecting to the generated changelog: https://github.com/tensorchord/envd/blob/main/CHANGELOG.md. Similar to https://github.com/argoproj/argo-workflows/releases.
Have a template that automatically fills in the release description:

https://github.com/argoproj/argo-workflows/blob/master/Makefile#L675-L676
https://github.com/argoproj/argo-workflows/blob/master/hack/release-notes.md
https://github.com/argoproj/argo-workflows/blob/master/.github/workflows/release.yaml#L388

I think it is a better choice. I may reopen an pr to adopt this method. This will solve the current problem of inconsistent formatting of the release page and CHANGELOG.md.

Would you like to submit a PR with the new approach?

Would you like to submit a PR with the new approach?

Sure. /assign @dqhl76

/assign @dqhl76


Create jupyter notebook with Ray Dalio's holy grail

I would like us to have a jupyter notebook that replicates this:
https://youtu.be/Nu4lHaSh7D4
The objective is to find 15-20 good uncorrelated return streams. Doing so you increases the information ratio/decreseas the risk taken to achieve an expected return.

Done #90


Update readme.md

@dacbd , I created a sep branch called 'tests' and opened a pull request. The workflow run successfully but I can't see the commit comment on the pr conversation area.

#10


Form count is not always filtered

Problem description
Central filters the list of forms that it returns to the user based on the user's role on the project. For example:

A Data Collector user will not see closed or draft forms.
A Project Viewer will not see closing forms.

That's true both on the homepage and on the forms page.
However, this filter is not applied to the forms count returned with the extended metadata of the individual project response (on the forms page). The forms count always includes all forms.
That differs from the forms count returned with the project list on the homepage (/v1/projects?forms=true). That count does seem to be filtered. That said, I suspect that if ?forms=true were not specified, then the count would not be filtered (though I haven't confirmed this).
To summarize:

/v1/projects/:id/forms: filters the list
/v1/projects/:id (extended metadata): doesn't filter forms
/v1/projects?forms=true: filters formList and forms
/v1/projects (extended metadata but not ?forms=true): doesn't filter forms (I think)

I don't think we surface the forms count in a lot of places in Frontend. If it differs from the form list response on a project page, Frontend will automatically update it to match the form list in order to reduce inconsistency. There is one case I can think of where a Project Viewer would briefly see the non-filtered count. If a Project Viewer navigates to the forms page, then between when they receive the project response and the form list response, they will see the non-filtered count.
URL of the page
https://staging.getodk.cloud/#/projects/90
Steps to reproduce the problem

Create a new project.
Create a form, publish it, then change its form state to closing.
Create a form, but don't publish it or change its form state.
Add a non-admin user to the project as a Project Viewer. Log in as that user.
Navigate to the forms page. Only one form is shown. However, if you open the Network tab of Chrome devtools, you will see that 2 was returned for the forms count.
Log back in as the first user.
Change the non-admin user to a Data Collector. Log back in as that user.
Navigate to the forms page. Again, only one form is shown, but the Network tab shows 2 for the forms count.

Expected behavior
The forms count should always be filtered.
Central version shown in version.txt
versions:
e49518adb84f88d7bc6c3626fc77584dfc935435 (v2024.1.0-6-ge49518a)
+2a291e718f2d342148f3f958691875bacadc7596 client (v2024.1.0-22-g2a291e71)
+beea5c81bf225b5e9da613c38269f07466cae613 server (v2024.1.0-21-gbeea5c81)

I suppose the thinking is that it is not necessary - my users don't need open_form.read because that's already implied by them having form.read perhaps? So there's some implicit entailment, one permission encompasses others?

This hopefully has enough clues about how we ended up with open_form.read https://github.com/getodk/central-backend/pull/968


Handle exceptions in HostConnectionPool

Some tasks did not access/log the Exception member and caused the exceptions to be handled by the finalizer. An application handling the TaskScheduler.UnobservedTaskException event would get exceptions that were left as unhandled even though they were.
(I'm not 100% sure about the location of the error handling regarding the pool semaphore).

Nice!
About the semaphore, I think doing the exception handling after releasing, to release it as soon as possible.

Apart from the minor nits, looks good to me.

I made all the suggested changes except for a potential missing t.Exception handling in AttemptReconnection() which I'm not sure I understood since it is handled in that method.

It's OK, you handled the exception on AttemptReconnection().
Looks good to me.


Feature: Add support for key remapping

Solution/Idea
As Files gets more advanced and the number of shortcuts keeps growing, it'll be useful to allow keyboard remapping to allow users to configure shortcuts to work the way they like it.
Although we'd need our own UI, Windows Terminal can help with some inspiration.

Requirement
This proposal will add a settings page called "Actions"
This page will contain a list of actions
This proposal will allow configuring the shortcut or gesture to configure each action
This proposal will allow users to reset all actions to the default
This proposal will allow users to reset each action
Files Version
v2.3
Windows Version
No response

Here is a custom KeyPicker that I could integrate into Files. What do you think?
https://user-images.githubusercontent.com/46631671/179416381-91b554f2-69b4-4a67-90ec-a1b8e063ede1.mp4

Something like that, I need to create the design but you should be able to work on the functionality in the meantime.

Remapping actions is already supported if you manually edit the settings file

Open Settings > Advanced > Edit settings file
Right click the system tray icon for Files and click "Quit"
Go back to the settings file and find the section for "Actions"
The list of available actions can be found here

Here is an example remapping the copy action to ctrl + j
"Actions": {
    "CopyItem": "ctrl+j"
 },

A page would be great? Working on it.

@0x5bfa nice! I would suggest calling the page "Actions" as it's supposed to be more than just keyboard shortcuts.

The requirements listed above are still accurate but I don't have any specific design in mind for the UI.

A big shoutout to @0x5bfa for completing this large task 🎉

This proposal will allow configuring the shortcut or gesture to configure each action

Custom gestures are out of scope for this task, but we'll continue exploring to see if there is a path forward for them.


app_not_installed error right after app uninstallation

Issue summary
Before opening this issue, I have:

[x] Upgraded to the latest version of the package

shopify_api version: 14.3.0
Ruby version: 3.3.1
Operating system: linux

[ ] Set log_level: :debug in my configuration, if applicable
[ ] Found a reliable way to reproduce the problem that indicates it's a problem with the package
[x] Looked for similar issues in this repository
[x] Checked that this isn't an issue with a Shopify API

If it is, please create a post in the Shopify community forums or report it to Shopify Partner Support

I believe this has started after switching to the new token exchange API.
When user uninstalls our app, that executes a background job that destroys the shop record. Shortly after, a request to our web app comes with the token of that shop:
/?embedded=1&hmac=[FILTERED]&host=[FILTERED]&id_token=[FILTERED]&locale=en-IN&session=[FILTERED]&shop=[FILTERED]&timestamp=1717860610

That results in the following error:
A 400 error (ShopifyAPI::Errors::HttpResponseError) occurred during the token exchange. Response: {"error"=>"app_not_installed", "error_description"=>"The application is not installed on this shop."}

I believe it is happening on the following lines: https://github.com/Shopify/shopify-api-ruby/blob/main/lib/shopify_api/auth/token_exchange.rb#L53-L68
However, it is not handled because error is not invalid_subject_token but app_not_installed. The error bubbles up to our controller.
If that is helpful, here is the id of the error: edf7bd5b-4f17-4bf8-a35e-0bba570eb2bd-1717860642.
Expected behavior
I think the error should be handled gracefully, and InvalidJwtTokenError should be bubbled up instead.
Actual behavior
Error is not handled.
Steps to reproduce the problem
I can't reliably reproduce this but from the logs I see the same order of steps is happening.

 Response: {"error"=>"app_not_installed", "error_description"=>"The application is not installed on this shop."}
2024-06-08T15:30:42.733025618Z stdout F E, [2024-06-08T15:30:42.732248 #1] ERROR -- : [90cf4e4db7f98c7a800c914cf24f27f2] {"error_reference":"If you report this error, please include this id: edf7bd5b-4f17-4bf8-a35e-0bba570eb2bd-1717860642."}
2024-06-08T15:30:43.024100912Z stdout F I, [2024-06-08T15:30:43.023860 #1]  INFO -- : [90cf4e4db7f98c7a800c914cf24f27f2] Completed 400 Bad Request in 451ms (Views: 1.9ms | ActiveRecord: 0.7ms | Allocations: 23613)

Hi @mironov
Thank you for reporting this issue. I'm able to reproduce the issue by uninstalling the app and and close the setting page.

We'll look more into this issue to figure out why the page doesn't navigate away after app uninstallation.
Si

@zzooeeyy I just saw another instance of this error in our logs:
"2024-10-06T00:54:17.169956075Z stdout F I, [2024-10-06T00:54:17.169687 #1]  INFO -- : [aed6cf6b720f210d85d1e4a751af2f42] [ ShopifyApp | INFO | Shop Not Found ] Performing Token Exchange for [REDACTED.myshopify.com] - (Offline)"
"2024-10-06T00:54:17.322243784Z stdout F E, [2024-10-06T00:54:17.321943 #1] ERROR -- : [aed6cf6b720f210d85d1e4a751af2f42] [ ShopifyApp | ERROR | Shop Not Found ] A 400 error (ShopifyAPI::Errors::HttpResponseError) occurred during the token exchange. Response: {\"error\"=>\"app_not_installed\", \"error_description\"=>\"The application is not installed on this shop.\"}"
"2024-10-06T00:54:17.323561812Z stdout F E, [2024-10-06T00:54:17.323083 #1] ERROR -- : [aed6cf6b720f210d85d1e4a751af2f42] {\"error_reference\":\"If you report this error, please include this id: 0730c855-ff3a-4066-80c1-2c18096ca941-1728176057.\"}"

Could it be that the issue is not fully resolved?


perf: Optimizing docker builds

What’s new

feat: bump python from 3.10 to 3.12
feat: dependency management with python poetry
feat: support for linux/arm64

Improvements

perf: optimize image size.  11.1GB to 2.93GB
perf: optimizing actions build

Usage
# build
docker buildx build -t fish-speech:latest . --load

# cpu only
docker run --name fish-speech -d -p 7860:7860 -e CUDA_ENABLED=false fish-speech:latest

# gpu only
docker run --name fish-speech -d -p 7860:7860 fish-speech:latest

Switching to a GitHub build will significantly speed up build times. We've included these packages to ensure the container is dev-ready out of the box:
git curl build-essential ffmpeg libsm6 libxext6 libjpeg-dev zlib1g-dev aria2 zsh openssh-server sudo protobuf-compiler cmake libsox-dev
We've moved away from Poetry due to inconsistencies across versions and problems with dependencies requiring custom index URLs. We now prefer setuptools for its simplicity and reliability.

Switching to a GitHub build will significantly speed up build times. We've included these packages to ensure the container is dev-ready out of the box:
git curl build-essential ffmpeg libsm6 libxext6 libjpeg-dev zlib1g-dev aria2 zsh openssh-server sudo protobuf-compiler cmake libsox-dev
We've moved away from Poetry due to inconsistencies across versions and problems with dependencies requiring custom index URLs. We now prefer setuptools for its simplicity and reliability.

I can add a debug version of the image to facilitate quick debugging by developers.


Roll Flutter Engine from aa4c205c1357 to e22414aa29a5 (1 revision)

https://github.com/flutter/engine/compare/aa4c205c1357...e22414aa29a5
2022-10-14<EMAIL_ADDRESS>Roll Skia from 8800559e7934 to 146ffc917bc2 (1 revision) (flutter/engine#36754)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Global HTTP client middleware being forgotten after every queue job

Laravel Version
10.15.0
PHP Version
8.1.19
Database Driver & Version
MySQL 8.0.33
Description
All Facades are reset after each queue job, https://github.com/laravel/framework/blob/3cd9bcb25dbfb4e70c9b96dd5b9c2ce109583217/src/Illuminate/Queue/QueueServiceProvider.php#L214.
This causes a global HTTP client middleware, applied in my app's ServiceProvider, not to be applied anymore when using Http::get() etc.
Steps To Reproduce
app/Providers/AppServiceProvider::boot()
public function boot()
{
    Http::globalRequestMiddleware(fn ($request) => $request->withHeader(
        'User-Agent', 'Example Application/1.0'
    ));
}

app/Jobs/DoStuff::handle()
public function handle() {
    Http::get('https://example.com/1');
}

I was not able to reproduce the issue using a fresh Laravel installation (v10.15.0)

app/Providers/AppServiceProvider:

app/Job/TestJob:

Result:

Did you run multiple jobs and all of the requests have the header ls attached?

this problem U R facing is W/ Laravel queue system and Facades. By default, Laravel resets Facade instances after each queue job, including the global HTTP client middleware. This means that when using Http::get() in a queued job, the middleware is not applied, to fix this, modify ur queued job to manually include the middleware before making HTTP requests. This ensures consistent application of the global middleware.
Maybe this code can explain what I mean
  use Illuminate\Support\Facades\Http;

  class UrQ implements ShouldQueue
  {
      public function handle()
      {
          $httpClient = Http::withOptions([
              'middleware' => [
                  function ($request, $next) {
                      $request = $request->withHeader('User-Agent', 'Example Application/1.0');
                      return $next($request);
                  }
              ]
          ]);

          $response = $httpClient->get('https://example.com/1');

          // ..
  }

Yep, already found a workaround. But as this is hard to discover/debug, I opted to open an issue to see if we can fix the actual issue 😀

@cleptric thanks for raising the issue. I think the quickest fix for this would be to add the following in your Service provider...
$this->app->singleton(\Illuminate\Http\Client\Factory::class);

I'm going to look into the best fix for the framework.

I've opened a PR to address this issue #47915
I will close this issue and we can follow along with the PR instead.


Disable storage

I don't want to use the expiringStorage and fill my users' localStorage with keys.
So I made a simple option to disable it.

Feel free to open a new PR with tests.