View (Render) is not Updated after State Change - No Obvious Mutations made in Reducer

I would like to submit a:

[x] Bug
[ ] Feature

It most probably is my mistake, but that's why I'm here. I'm stuck and I don't know what it is.
Redux doesn't update/re-render the components state, but the Redux' state updates as it should.
There are no obvious mutations from the perspective I'm seeing it. I've used so far:

Object.assign({}, source)
slice(startIndex) // Definitely Does not work. Tried it on JSFiddle and when I changed it to Object.assign() it worked.
ES6 Spread Operator
etc.

And nothing worked.
Code (md & gists) :
Sub-Reducer
export default (state = [], action) => {
  switch (action.type) {
    case ADD_COMPONENT:
      if (state[action.id]) {
        console.error('Error: Cannot add component, id already exists');
        return state;
      } else {
        return {
          ...state,
          [action.id]: action.obj
        }
      }
    case 'debug':
      console.log('debug', state[action.id]);
      return state;

    default:
      return state;
  }
};

React Component https://gist.github.com/Eksapsy/4f0b348765ba648a6c62baa249d4f756
Actions https://gist.github.com/Eksapsy/31c1c84e0b5975a5a0352d24af75419c
The root reducer does not have any problems, I'm 110% of that. I'm using it the same way with this reducer as I do with my other Working reducers
I wish I could provide a demo in JSFiddle, but I can't reproduce the bug. It's just different code. Actually, when I use some of the code in JSFiddle the code works, but it isn't the same code as in my project's
react-redux: 5.0.6",
redux: 3.7.2,
redux-thunk: 2.2.0,

See this guide: https://github.com/ReactTraining/react-router/blob/master/packages/react-router/docs/guides/redux.md

The truth is that I got Router in my Project. I tried injecting it with withRouter but still no result.

Do I have to make another issue with the same content now or what? You closed it without considering if that's my issue or not.
And I don't see anywhere a sign that I'm using Router. So how were you so sure that this was my issue and you instantly closed it?

Yeah, the Router thing was probably jumping to conclusions a bit.
However, a few other thoughts.
First, this is a bug tracker, not a support forum.  You should try posting this question over on Stack Overflow or asking it in the Reactiflux chat channels.  There will be more people available who can see the question and try to help.
It's not clear from the sample code you linked or the description you gave what exactly you mean by "Redux isn't updating the component's state".
Copying values from props into an instance field like this.reduxState is a very odd pattern, and is probably not a good idea.
It looks like you're actually including the component instance itself in the action, and inserting that instance into the store.  That is a bad idea - please do not do that.
It also looks like your reducer is defining its initial state as an array, but then trying to update it as if it were an object.  That won't work properly.
If you are trying to keep per-component state in a Redux store, there's many existing libs that implement that capability - you should try using one of those instead (like redux-ui, react-redux-local, or lean-redux).

Hey @markerikson. I just joined after a long day in this issue to just delete my Comment because it was too rude and I'm sorry I brought my personal problems here. :) Timdorr actually helped me very much, and I thanked him in the first place, even if this was not my problem but as you said it's not a support forum.
Hm, your suggestions also helped much! Thanks!
Have a happy rest of your day and thank you for supporting this beautiful product, guys!


circulome-seq term

Since the inclusion of OBI:0002144, the lab has posted a publication referencing circulome-seq.
http://www.g3journal.org/content/ggg/early/2017/08/11/g3.117.300141.full.pdf
Do you think it would be appropriate to add 'circulome-seq' as a synonym to the existing term or more appropriate to create a new term underneath it?
The distinction, if we were to make one, is in the DNA enrichment, so the definition might be...
An extrachromosomal circular DNA sequencing assay which utilizes a combination of biophysical (CsCl) and enzymatic (ExoV) approaches for circular DNA enrichment, followed by Tn5 tagmentation allowing library construction from a very low quantity of input material and providing informatic advantage at analysis stages of the sequencing data.

My gut feeling: Adding it as a synonym might be more useful in practice;
too fine distinctions tend to not get used and/or confused. But I am not
expert in this field.
On Mon, Nov 6, 2017 at 4:59 PM, jahilton<EMAIL_ADDRESS>wrote:

Since the inclusion of OBI:0002144, the lab has posted a publication
referencing circulome-seq.
http://www.g3journal.org/content/ggg/early/2017/08/11/
g3.117.300141.full.pdf
Do you think it would be appropriate to add 'circulome-seq' as a synonym
to the existing term or more appropriate to create a new term underneath it?
The distinction, if we were to make one, is in the DNA enrichment, so the
definition might be...
An extrachromosomal circular DNA sequencing assay which utilizes a
combination of biophysical (CsCl) and enzymatic (ExoV) approaches for
circular DNA enrichment, followed by Tn5 tagmentation allowing library
construction from a very low quantity of input material and providing
informatic advantage at analysis stages of the sequencing data.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/obi-ontology/obi/issues/872, or mute the thread
https://github.com/notifications/unsubscribe-auth/ANN9IneyOcVJhhGjmm7wAihUGUfM4H2Hks5sz4FAgaJpZM4QT8Oy
.

--
Bjoern Peters
Associate Professor
La Jolla Institute for Allergy and Immunology
9420 Athena Circle
La Jolla, CA 92037, USA
Tel: 858/752-6914
Fax: 858/752-6987
http://www.liai.org/pages/faculty-peters

That was my feeling as well because the goal & general method is the same.

Hello, have there been any updates on adding Circulome-seq as a synonym for OBI:0002144?

Discussed on call. The value to users can first be addressed by simply making this an alternative term.  We could subclass it later if requested.  Also, could add that both classes/terms output DNA Sequence data and therefore are DNA Sequencing Assays.

Currently, Circulome-seq doesn't appear under DNA sequencing assays and is under the broader "assay" class. Can we have it under the DNA sequencing assay class?


CI on commits

trigger a gradle build on each push or PR

ideally run for several java versions (Java 8 ... 14)

Example:
https://github.com/TNG/junit-dataprovider/blob/master/.github/workflows/gradle.yml

Has been completed with commit 51d98848


Starting index should be 1 not 2

In the "picks a winner, resets, and sends money" test variable startingIndex should be 1 instead of 2 as Mr. Patrick have shown in FreeCodeCamp tutorial.

Thanks for the PR! I make the starting index 2 in the video? Why do I do that? Do you have the timestamp?


Fixes main value in package.json

The value of the main key in package.json points to the file in 'src', which is not packaged, while it should be 'dist'.

Closing as the project has been reverted to v3.0.0 in v6.0.1 (see https://github.com/steverydz/build-url/issues/42#issuecomment-683629088)

Hmm. I pulled the updated the npm package into my project and there, the package.json entry was definitely incorrect. I manually changed it to './dist/build-url.js' and then it found it. Maybe there's a difference between the local dev environment and an environment where we get the package with 'npm install'.

@Treverix yes I've definitely found a difference testing locally with the published version. Anyway, I've republished v3 as 6.0.1 so it will go back to how it was before. Clearly I've got something wrong with the structure of this project.


Remove Rounding from experimental flags

Rounding has been used in many products now and I think it is time to turn it on by default and remove the experimental option for it. Now that we have scale factor support I see no reason to not rely on rounding within Yoga.
Speak up if you disagree. We can always make this on by default but introduce a config to turn it off. It should however not be considered experimental.

There is already an option to turn it of, you simply need to set scaleFactor = 0.

@woehrl01 oh cool! I didn't think of that 👍

@emilsjolander Maybe we should discuss if we include #508 from @arcanis, too, before making this a default for everybody?

Yup! as long as it all happens within this release it should be fine 👍  I'll spend some time tomorrow merging PRs

I'm working on this now 👍


Fix OpenCost Prometheus existing secret fetching

I've noticed Grafana Cloud Kubernetes monitoring OpenCost pod started to silently fail with following error:
ERR Failed to query prometheus at https://prometheus-prod-01-eu-west-0.grafana.net/api/prom. Error: Prometheus communication error: 401 (Unauthorized)
...
Body: '{"status":"error","error":"authentication error: no credentials provided"}

Pods starts just fine, but OpenCost can't connect to Grafana Cloud Prometheus endpoint. And then pods will produce more error logs:
0) Errors:
  Request Error: Prometheus communication error: 401 (Unauthorized) URL: 'https://prometheus-prod-01-eu-west-0.grafana.net/api/prom/api/v1/query ...
  Parse Error: Prometheus communication error: ...

After digging into this I've found out that OpenCost chart which this repo uses introduced new property prometheus.existingSecretName, so by changing secret_name to existingSecretName, OpenCost pod starts calling Grafana Prometheus endpoint just fine.
So far the workaround is to add existingSecretName to final value file:
k8s-monitoring:
  opencost:
    opencost:
      prometheus:
        existingSecretName: prometheus-k8s-monitoring
        external:
          url: "https://prometheus-prod-01-eu-west-0.grafana.net/api/prom"

In this PR, I've simply decided to replace secret_name -> existingSecretName property name in default value file.
Relevant OpenCost PR: https://github.com/opencost/opencost-helm-chart/pull/196

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[Matching] Add 'guest' to signup regex in analyser

Reviewer: @GioSensation
Asana: https://app.asana.com/0/1205996472158114/1208414891646031/f
Description
On jcrew.com, the guest checkout form is treated as a login one. This is because the Have an account? Sign in text seems to score it on the login side, however multiple attributes with guest or Guest in the string  and attributes are ignored. Such checkout forms are used in plenty of shopping sites, so it's likely that guest should be scored rather as a signup.
Adding guest to signup regex should work. I am also adding it to conservative one, I think it's a pretty strong indicator for signup.

Steps to test

Go to Jcrew.com
Add a product to the bag, and click on checkout (from the dropdown)
In the bag, go to checkout and click on "Continue as guest"
Guest form should have email protection showing:

I'm not sure what we're fixing on this. I see the same behavior in production and with this branch, and both fail on the sign in form (treat it as signup). I think the problem there is something we should look into, which is: calculate the distance between elements. But that is a larger undertaking. Let's sync on Monday.

@GioSensation this PR did not try to fix the sign in problem (I wasn't able to reproduce that as said in the asana task).
The idea was that guest should add up to the signup score, but as we discussed on the call it probably would cause regressions in e.g hotel sites where guest is a valid token everywhere.
Closing this for now, until we have a clear alternative approach.


provide dockstore version for download of upgrade jar

Addresses https://ucsc-cgl.atlassian.net/browse/SEAB-3147

@denis-yuen RE: https://github.com/dockstore/dockstore-cli/pull/78#pullrequestreview-711710455
Why did this issue happen for 1.11 (and presumably not for previous releases)?
Code to download client jar at moment of upgrade was added September 14 https://github.com/dockstore/dockstore-cli/commit/4a9474a7eae4e5d38f317a102e57593510d002af, which was after  release 1.10.0 and newly incorporated into 1.11.0.
How did you decide between hotfix and 1.12 for this fix? I didn't think of merging commits to the hotfix, but I can do that

Closing to create a new PR to merge to the new hotfix-1.11.1 branch


3D widget template

I've verified(manually, not via generator) the template to work in 2.0Beta, need to however do the following:

App Path prompt is set to 1.3 WAB
support for stemapp3d in App Directories
Tests
Doc

Also, if this is going to be post release of generator 2.0, maybe there is a way that doesn't need a separate 3D manifest.json file.

Thanks for this @jPurush! A good start. Raises a lot of questions.
Here are some responses to the TODOs you list above

App Path prompt is set to 1.3 WAB
I think we'd leave the default to 1.3 until WAB 2.0 is out of beta. This raises the question, what if someone enters the path to a WAB 1.3 install while running the default generator and then tries to generate a 3d widget. Are we going to try and detect WAB capabilities at any point to prevent that, or just let that happen?

support for stemapp3d in App Directories

Oh man, a whole new stemapp folder. Makes sense, but that's going to impact the default generator too. How is it going to know which stemapp folder to copy a given widget to (if, for example, you had a project folder w/ both 2d and 3d widgets)?

Tests

Currently failing (need to add a 2d/3d response to the prompts passed in widget tests). Before trying to get them to pass however, this branch should be merged w/ current master (after #48). That should make it easier.
Also see my questions in the line comments.
All in all, I need to understand better how WAB 2.0beta works before I can answer any of these. I'm going to start on #45 so that I can get some insight.
However all of this is making me realize that adding support for 3d is non-trivial, may introduce breaking changes, and should probably be included in the v2.0 release.
cc @gavinr

After talking w/ @gavinr, came up w/ the following:

default generator will prompt for 2d/3d, and configure the grunt task to write to the correct folder (stemapp or stempapp3d). The response to this could probably dictated the default path to the WAB
any subsequent runs of the widget subgenerator will get the 2d/3d setting saved from the default generator

I'll break the above out into issues in the next few days. I just wanted to document the decisions here for now. Once the issues are up we can decide if we'll continue to work on this branch (since it's a whole new direction).

closed in favor of #54
Thanks @jPurush


[#1249] feat(mr): Support to spill large amounts of shuffle data.

What changes were proposed in this pull request?
Support to spill large amounts of shuffle data.
Why are the changes needed?
For large amounts of shuffle data, we should restore into local or remote disk, but not memory.
This PR is proposal (1) which describe in #1249.
Fix: #1249
Does this PR introduce any user-facing change?
No.
How was this patch tested?
unit test, test mr word count on cluster.

Codecov Report

Merging #1263 (6e545bb) into master (49effee) will increase coverage by 1.10%.
Report is 2 commits behind head on master.
The diff coverage is 64.00%.

@@             Coverage Diff              @@
##             master    #1263      +/-   ##
============================================
+ Coverage     53.93%   55.04%   +1.10%
- Complexity     2647     2652       +5
============================================
  Files           396      376      -20
  Lines         23036    20722    -2314
  Branches       1970     1978       +8
============================================
- Hits          12424    11406    -1018
+ Misses         9845     8615    -1230
+ Partials        767      701      -66

Files
Coverage Δ

...pache/hadoop/mapreduce/task/reduce/RssFetcher.java
91.97% <100.00%> (+0.05%)
:arrow_up:

...preduce/task/reduce/RssRemoteMergeManagerImpl.java
75.60% <100.00%> (-3.62%)
:arrow_down:

...pache/hadoop/mapreduce/task/reduce/RssShuffle.java
0.00% <0.00%> (ø)

.../hadoop/mapreduce/task/reduce/RssBypassWriter.java
63.33% <60.00%> (+1.42%)
:arrow_up:

... and 21 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Prefix operators have issues

Operators(++ / --) do not work on dictionary subscripts
var x = {"test": 10};
++x["test"]; // Operand must be a number.

It also does not work on nested properties.
class Test {}

var x = Test();
x.x = 10;

var y = Test();
y.x = x;
print(y.x.x); // 10

++y.x.x; //  Only instances have fields.

These operators should probably be removed now while the reliance upon them is quite small (even if the API currently at a stable version). If / when a potential implementation for these operators is found i'd be more than happy to allow them back in.


Accessibility add link name

resolve https://github.com/karmada-io/website/issues/30

close https://github.com/karmada-io/website/pull/31


Add bottom-right and top-right position

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Wowerlay can be aligned to right of the target.

Will be refactored.


vtm: 0.9.9i -> 0.9.9k

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/netxs-group/vtm/releases.
meta.description for vtm is: Terminal multiplexer with window manager and session sharing
meta.homepage for vtm is: https://vtm.netxs.online/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.9.9k with grep in /nix/store/gbi7b8ifjckd6l807l29s6qm5gh8spkj-vtm-0.9.9k
found 0.9.9k in filename of file in /nix/store/gbi7b8ifjckd6l807l29s6qm5gh8spkj-vtm-0.9.9k

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
vtm

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

vtm

Maintainer pings
cc @alesya-h for testing.

@ofborg build vtm vtm.passthru.tests


Tweaks and refinements for 0.1 release

Link all technical PRs addressing the release of 0.1 version to this issue.

Good for now.


Support path to overwrite name in Dataset API

Signed-off-by: mahao<EMAIL_ADDRESS>
Ⅰ. Describe what this PR does

support set alluxio instance configuration  alluxio.master.mount.table.root.ufs by dataset count and scheme
support if user define mount.path, replace mount.name as the mount path

Ⅱ. Does this pull request fix one issue?

fixed #643

Codecov Report

Merging #766 (432ec50) into master (edf7060) will increase coverage by 0.48%.
The diff coverage is 71.11%.

@@            Coverage Diff             @@
##           master     #766      +/-   ##
==========================================
+ Coverage   12.63%   13.12%   +0.48%
==========================================
  Files          93       96       +3
  Lines        4241     4267      +26
==========================================
+ Hits          536      560      +24
- Misses       3645     3647       +2
  Partials       60       60

Impacted Files
Coverage Δ

pkg/common/constants.go
0.00% <ø> (ø)

pkg/ddc/alluxio/metadata.go
0.00% <0.00%> (ø)

pkg/ddc/alluxio/transform.go
0.00% <0.00%> (ø)

pkg/ddc/alluxio/transform_optimization.go
78.46% <0.00%> (ø)

pkg/ddc/alluxio/ufs_internal.go
0.00% <0.00%> (ø)

pkg/ddc/alluxio/utils.go
36.17% <ø> (-1.33%)
:arrow_down:

pkg/utils/charts.go
0.00% <0.00%> (ø)

pkg/utils/databackup.go
0.00% <0.00%> (ø)

pkg/common/fluid_ufs_scheme.go
100.00% <100.00%> (ø)

pkg/ddc/alluxio/transform_ufs.go
100.00% <100.00%> (ø)

... and 4 more

Continue to review full report at Codecov.

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

To mount non-native ufs be on "/" in Alluxio namespace, I think we should also do some extra property injections. For example, making Aliyun OSS the root path also requires Alluxio properties like fs.oss.accessKeyId=<OSS_ACCESS_KEY_ID>. This goes the same with AWS S3, HDFS, etc.

done, thanks

@allenhaozi  Thanks for your great work!


Add context menu to file browser in iOS 13.

The current implementation is deprecated, which limits the functionality
of the file browser.

Thanks @elfredpagan!

A few things I noticed:

This should be using @available(iOS 13.0, *) instead of preprocessor statements now that iOS 13 is out for real (I need to go back and remove all of those left over from the betas anyway)
This PR doesn't touch the existing implementation which uses UIMenuController, right? What happens with both of them implemented at the same time like this?

Please keep the macros around for now as we still have builds using the iOS 12 SDK. I believe keeping the UIMenuController implementation is to keep things working on iOS 12 and below, but it no-ops in 13.

Alrighty 👍🏼
Ah, they're both just table view delegate methods, that makes sense.


Poll: New findByText() helper?

After migrating a few codebases to async/await and ember-native-dom-helpers, the single most annoying pattern to transform is selectors that use the :contains("text") pseudoselector from jQuery.
I proposed supporting a subset of sizzle pseudoselectors in https://github.com/cibernox/ember-native-dom-helpers/issues/32, but I kind of agree that we shouldn't, because even those selectors are going to be deprecated in jQuery itself.
The question is wether or not we want to add some sort of utility to make finding elements with a certain text easier. A tentative method and signature could be findByText('.selector', 'text', context?).
Now vs this helper:
findAll('h1').find((el) => el.textContent.includes('Tomster')) vs findByText('h1', 'Tomster')
The pros are obviously, ergonomics.
The const are one more helper to learn and arguably by making it more convenient we are encouraging a testing pattern that many people see as dubious or harmful (test selectors could be a better)

It's like full-text search, maybe something like match('h1', 'Tomster'), matchAll('h1', 'Tomster') ?

In case we implement this I think I'd rather have findByText/findAllByText for parallelism with find/findAll.
match makes me think in regular expressions, but perhaps it's just me.
The most important question is wether we should add this helper or by doing so we would be encouraging a bad practice.

IMHO a helper like that can make sense when you haven't adopted something like test selectors yet, but as you mentioned having it available directly in this addon would suggest bad patterns to our users.
I think it would make sense to mention in the documentation how it would be possible to implement such a helper function yourself, even including the code snippet, but not actually add the helper function itself to the project. That way we can add a disclaimer to the documentation that this is considered an anti-pattern and link to better patterns.

@Turbo87 I kinda agree with you on this one, and I've checked, that I am almost not using these selectors. But the places where they are used, are not related to tests.
On another hand, I'd say we shouldn't be limiting people and not giving them some handy, simple functions. In the end, everyone then will re-implement what @cibernox wrote in their own codebase, which makes no big sense.
Don't think of this library as only used for tests, it has much bigger potential to be used in the main code base, where test selectors can't be added for example.

@RuslanZavacky This library is only intended to be used in tests. Actually I believe you cannot even import any of this functions from outside tests.

@cibernox indeed, sorry. I've specifically re-implemented focus/blur as functions on the app level, to be consistent between tests & app itself :)

I found this issue b/c I was wondering if anyone had implemented selectors similar to those in react-testing-library. I've bought into the guiding principles behind Kent C. Dodd's Testing Library and would love it there were ember test helpers that would help me prioritize querying elements by role and accessible text.
I'm wondering if anyone else would be interested in an addon like ember-testing-library that wraps those @testing-library/dom fns.


Allow ic-prompt-value form key to be configurable

The standard implementation definitely works with ic-prompt-value but it would be great if the value could be hoisted to my own specific name. Maybe something along the lines of:
<button ic-put-to="/cust" ic-prompt="Id?" ic-prompt-parameter="customerId">Customer</button>

This would be very helpful for server side MVC frameworks' model binders.
Other suggestions off the top of my head would be along the lines of:

ic-prompt-target
ic-prompt-key
ic-prompt-name
ic-prompt-value (i don't really like this one due to overlap)

fixed by f1468f8ce719e26807b5609ce92293f1a4689753

For future readers, example:
<button ic-put-to="/cust" ic-prompt="Id?" ic-prompt-name="customerId">Customer</button>


MSML V0.2 Presentation

https://blockscienceteam.slack.com/archives/CA70QMRRC/p1706543528547659

Title Page (V0.2.0 MSML Update)
Review of MSML Library how/why, use the graphic from the README
Review of V0.2 updates (include high level + number of closed out issues)
JSON schema walkthrough, high level of what/why JSON schema, then link to the JSON schema guide
Root Finding Simulation demo
Future arc of the research
Questions/feedback?

Make a presentations folder and make markdown update for the V0.2.2 update


issue listing files from commit

After going through the documentation and much trial and error, I still can't figure out how to list files from a commit.  What I want to do is get the list of files from a particular commit and their status.  How would I do that?

I think I got it.  I'm using:
status = g.object(commit_id_her]).diff_parent.name_status
Is that the best way to do that?


Don't reset the blob name on import if it has been set

Backwards compatible! I like it 👍.

There seems to be a real reason for the old behavior: #117

Backwards compatible! I like it 👍.

There seems to be a real reason for the old behavior: #117

Great catch! I had completely forgotten about that. Thanks for noticing.


[BEAM-8004] Fix flaky Flink load tests

The tests are failing due to an issue with scaling down a dataproc cluster. gcloud dataproc clusters update command can delete a node which runs a job manager. This results in a UnknownHostException being raised.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Run Seed Job

Run Load Tests Python Combine Flink Batch

R: @lgajowy Please take a look. Also cc: @pabloem @mwalenia

Thanks!


schema.path	cloud this also be a path where you store the xsd's  instead of an url?

schema.path	cloud this also be a path where you store the xsd's  instead of an url?
for example /opt/kafka/share/xds/..

You should be able to use a http or https url


Lionpath integration

Issues:
Committee member mappings.  We’ve got 7 options for dissertation committee (40+ options coming from lp).  Required? Or non-required?
Some extra fields I’m not sure we need.
Chairs potentially could have different chair for same program at different campuses (although I didn’t see any where this was actually the case)
If program does not match exactly what’s in lion path files, old ones should be set as inactive and not be used anymore
Restrict admins from changing programs?
And degrees?
Just adding new programs as they show up?

TODOs:
Rake test and whenever task to run this daily
Heavy testing, going to be difficult to test, can be done on dev server if connected to prod nfs
Might be able to do this from local docker machine but will probably be blocked by firewall

Add campus column to programs.  Import this from student plan file then use to link with appropriate Head/Chair.

Remove edit capabilities for degrees and programs.  Still allow them to be added.  Make sure there is a unique index for name and code for programs, and unique index for degree name.

This is ready to be tested in dev.  Should be able to just run the integration (even run it daily via cron) and manually link submissions to my user record to see how things work.


Interpreter: Error: BUG: missing upcast_distinct from Tuple(String, String) to String (Crystal::TupleInstanceType to Crystal::NonGenericClassType)

Example code:
class A
  private getter flashes = {} of String => String

  delegate each, to: flashes
end

a = A.new

a.each do |k, v|
  puts k
  puts v
end

Interpreter error:
$ cr i local/bugs/delegate_bug.cr
Using compiled compiler at /home/chao/git/personal/crystal/.build/crystal
In local/bugs/delegate_bug.cr:9:3

 9 | a.each do |k, v|
       ^---

In local/bugs/delegate_bug.cr:9:3

 9 | a.each do |k, v|
       ^---
Error: compiling a.each do |k, v|

  puts(k)
  puts(v)
end

In local/bugs/delegate_bug.cr:4:3

 4 | delegate each, to: flashes
     ^
Error: expanding macro

There was a problem expanding macro 'delegate'

Called macro defined in /home/chao/git/personal/crystal/src/object.cr:1292:3

 1292 | macro delegate(*methods, to object)

Which expanded to:

 >  1 |
 >  2 |
 >  3 |         def each(*args, **options)
 >  4 |           flashes.each(*args, **options)
 >  5 |         end
 >  6 |
 >  7 |
 >  8 |           def each(*args, **options)
 >  9 |             flashes.each(*args, **options) do |*yield_args|
 > 10 |               yield *yield_args
 > 11 |             end
 > 12 |           end
 > 13 |
 > 14 |
 > 15 |
 > 16 |

In local/bugs/delegate_bug.cr:4:3

 4 | delegate each, to: flashes
     ^
Error: expanding macro

There was a problem expanding macro 'delegate'

Called macro defined in /home/chao/git/personal/crystal/src/object.cr:1292:3

 1292 | macro delegate(*methods, to object)

Which expanded to:

 >  1 |
 >  2 |
 >  3 |         def each(*args, **options)
 >  4 |           flashes.each(*args, **options)
 >  5 |         end
 >  6 |
 >  7 |
 >  8 |           def each(*args, **options)
 >  9 |             flashes.each(*args, **options) do |*yield_args|
 > 10 |               yield *yield_args
 > 11 |             end
 > 12 |           end
 > 13 |
 > 14 |
 > 15 |
 > 16 |
Error: BUG: missing upcast_distinct from Tuple(String, String) to String (Crystal::TupleInstanceType to Crystal::NonGenericClassType)

Reduced:
def foo
  bar do |*yield_args|
    yield *yield_args
  end
end

def bar
  yield({1, 2})
end

foo do |x, y|
end

It's likely I never got to implement yield with a splat.

Even simpler:
def foo
  yield *{ {1, 2} }
end

foo do |x, y|
end


added support for spectral ops and updated the supported ops doc

added support for fft, ifft, rfft, irfft ops.
fix https://github.com/tensorflow/tfjs/issues/932

This change is

Maybe you could throw a good error if that file isn't found? Which will be me as soon as I run this :)

Merged for now, and will create a followup PR for on the doc gen discussion.

Also just FYI the tfjs-core.json on the website is always in sync with a release. So having to manually download doesn't actually change anything over making the request.

Oh sorry, you're right, it's when the union package is release. However, most people will use the converter through that anyways, so your docs will actually be ahead.


Data feeds are broken

The data sources are not loading except xmr price.

Issue might be api related, or it could be how cloudflare cashes the site.


Update copyright year

update footer copyright year to 2023

@gregvanl qq do we usually move the copyright year forward?


⚠ Minecraft has degraded performance

In 699bd58, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1021 ms

Resolved: Minecraft performance has improved in 0aa63ec.


A small atlas fix

Description

Related Issues

Checklist

[ ] Is this code covered by new or existing unit tests or integration tests?
[ ] Did you run make test successfully?
[ ] Do new classes, functions, methods and parameters all have docstrings?
[ ] Were existing docstrings updated, if necessary?
[ ] Was external documentation updated, if necessary?

Additional Notes or Comments

Codecov Report
Attention: 170 lines in your changes are missing coverage. Please review.

Comparison is base (cbe6b32) 78.22% compared to head (a2a7e4c) 80.23%.
Report is 100 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1141      +/-   ##
==========================================
+ Coverage   78.22%   80.23%   +2.00%
==========================================
  Files         101       97       -4
  Lines        6351     6596     +245
==========================================
+ Hits         4968     5292     +324
+ Misses       1383     1304      -79

Files
Coverage Δ

superduperdb/__init__.py
100.00% <100.00%> (ø)

superduperdb/__main__.py
71.42% <100.00%> (ø)

superduperdb/base/config.py
100.00% <100.00%> (ø)

superduperdb/base/jsonable.py
80.55% <ø> (ø)

superduperdb/cli/serve.py
31.81% <ø> (ø)

superduperdb/container/component.py
82.92% <100.00%> (+2.92%)
:arrow_up:

superduperdb/container/dataset.py
87.17% <100.00%> (ø)

superduperdb/container/encoder.py
90.00% <100.00%> (+0.16%)
:arrow_up:

superduperdb/container/listener.py
83.05% <100.00%> (ø)

superduperdb/container/serializable.py
98.14% <100.00%> (+3.02%)
:arrow_up:

... and 30 more

... and 8 files with indirect coverage changes

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


How to exit xdag daemon gracefully?

./xdag cn1.xdag.org:443 -rpc-enable -d
I run xdag wallet with a daemon, how to exit?

./xdag -i
>terminate

or just
echo -e "terminate\n" | ./xdag -i

Issue solved.


Release js-yaml

Hi @dotansimha can you please release a new version that has this commit included https://github.com/dotansimha/graphql-code-generator/commit/110961cd5a92a1d3ad751f18ce8c65e70c515553? As ^ is not used for version definition, it forces js-yaml 3.12.2 which contains security vulnerability https://www.npmjs.com/advisories/788
Thanks!

On it now, will be available in a few minutes :)

1.0.3 available. @simPod

Respect brother! Thanks a lot! 👌


traefic issue

hi i have found this issue and raised
https://github.com/carrier-io/galloper/issues/140

@tokarthik is it actual?

@tokarthik is it actual?

seems like we are good here

seems like we are good here


Enhancement: Able to get controller-runtime Client from the Resources

In most of the cases, we need to get and utilize controller-runtime Client internally for the intermediate functions
It would be nice to have a functions GetClient() inside the klient/k8s/resources/resources.go to get controller-runtime Client for this case.
Alternatively, we could probably have a helper function in the klient package returning Client as
cr.New(c.cfg, cr.Options{Scheme: scheme.Scheme})

Hi @vladimirvivien
Thanks for response. Our case is: we have a set of the helpers methods (lookup for default StorageClass, some PVC logic, etc.) in our controllers utilising controller-runtime Client and would like to reutilize part of them inside the e2e-framework tests to reduce the codebase. For example we should skip test if default StorageClass not allows to extend a volume size.

@dmvolod  Is this not what you are looking ?
https://github.com/kubernetes-sigs/e2e-framework/blob/e93ce38fcf9c53e1bd89c0c7a6e8949bd92fbc69/klient/k8s/resources/resources.go#L51-L70

For example we should skip test if default StorageClass not allows to extend a volume size.

Assess("Do Something", func(ctx context.Context, t *testing.T, c *envconf.Config) context.Context {
			r, err := resources.New(c.Client().RESTConfig())
			if err != nil {
				t.Fail()
			}
			var sc v1.StorageClass
			err = r.Get(ctx, "test", "", &sc)
			if err != nil && errors.IsNotFound(err) {
				t.Skip("No tests are there is missing storage class")
			}
			if sc.AllowVolumeExpansion == nil || !*sc.AllowVolumeExpansion {
				t.Skip("Skipping test as the volume expansion is not allowed")
			}
			// Do something
			return ctx
		}).

What you are looking for can be achieved this way too.

Thanks, @harshanarayana , I know how to do this, but we have a dozens of functions and don't won't to repeate code of them in the tests instead of just provider one parameter with the cr.Client

I see. So

Are you looking for a way to access the resources.client outside via a getter ?

this is what you are looking for ?

Are you looking for a way to access the resources.client outside via a getter ?

Yes, exactly, @harshanarayana , missed your first comment.

@dmvolod can you provide a code snippet of what you are doing with the controller-client once you get it? I ask because that can be useful functionality worth adding directly in the framework itself.
How are you using the client to detect StorageClass enabled ? Or is it the fact that you have existing code that you don't want to change and exposing the client would make integration easier ?

Hi @vladimirvivien
In our kubernetes operator, we use a dedicated struct to provide all parameters to iterate with the k8s cluster. One of these parameters is controller-runtime Client cr client.Client:
type k8sCluster struct {
	Client                client.Client
	Scheme                *runtime.Scheme
	Logger                logr.Logger
	Object                client.Object
	...                  ...
}

And as @dmvolod said before we also have a lot of functions (based on native k8s controller-runtime client functions like Get(), List() etc.).
And also we have a method to validate StorageClass Volume Expansion that uses cr client.Client:
func (g *ShootCluster) IsAllowVolumeExpansion(storageClass *storagev1.StorageClass) bool {
	return !(storageClass.AllowVolumeExpansion == nil || (storageClass.AllowVolumeExpansion != nil && !*storageClass.AllowVolumeExpansion))
}

Thus, to test this (and another) method in our e2e tests we have to re-implement them all in accordance with your klient.Klient client instead if just import our package and use it as it is.
Much better would be have helper function something like @dmvolod mention above:
cr.New(c.cfg, cr.Options{Scheme: scheme.Scheme}to reuse this function in our integration test just to provider contriller-runtime Client parameter to our k8sCluster struct.
And it will let us just call our function in e2e tests.
k8sCluster.IsAllowVolumeExpansion(storageClass)

I think this is a good suggestion and exposing the controller-runtime client does not break the current API.

/assign

@vladimirvivien  @Lxrdknows77 @dmvolod  I have a draft PR with two ways of fetching the controller runtime client. TPAL when time permits.

@Lxrdknows77 Yes and also lets you inject your own custom runtime.Scheme for any uses that might arise.

@harshanarayana Will you fix it or I can recreate PR?

@Lxrdknows77  Sorry, this one totally skipped my list of TODO items. Please feel free to recreate the PR. If you can wait until end of the week, I will have the PR ready

/remove-lifecycle rotten

@harshanarayana Have you gotten a chance to finalize this so we can merge?

@vladimirvivien  I have already update the PR https://github.com/kubernetes-sigs/e2e-framework/pull/156 with some of the comments addressed.


SAK-29158 Turkish Translation of Sakai 11.0

I have updated Turkish translation of all Sakai files for 11.0

There are conflicts, you'll need to pull master into your branch and resolve any conflicts before this can be merged.

Hi @steveswinsburg
We are doing these translations with crew in https://crowdin.com/project/sakai-cle.
Don't you take any updated language files from crowdin?
Until now, we thought that you received these files. We saw that you haven't get these updated language files.  Also, How can I understand where these conflicts come from.
Thanks in advance.

Hi @eemirtekin, merge conflicts occur if the file has changed since you branched it and modified it. If you just pull master into your branch and push up the modifications, that should sort it out.

Hi @steveswinsburg
When i forked the sakai master project, i added language files and pull request it. The variance was only in the language files . The purpose is updating new translations to the current versions. Can you add these language files, If i send to you.
Thanks in advance.

Yep, I can see the changes are only for the language files. What I mean is that in the time between you forking the code and sending this PR, another commit has gone into master that touched one of these file so yours is now out of date.
If you just pull the sakai master into your branch again, then push the updates to your branch, git will sort it out. There will likely be a few conflicts, if your files are the authoritative source, just resolve the conflicts using your versions of the files. That is what anyone would need to do if they took your patch.
Once you push, this PR will automatically be updated and we can merge it.

Just as a note it would have been easier for you to make these changes if you did not work on your master but rather a feature branch which could aptly be named the same as the JIRA issue i.e. "SAK-29158".
For now its best to close this PR and for you to follow these instructions and then resubmit the PR.
see https://confluence.sakaiproject.org/display/SAKDEV/Git+Setup
Your next PR should be from branch eemirtekin:SAK-29158 and not eemirtekin:master


deeplearning classification

i m facing issue with packages using and cannot find a nuget package of microsoft.ml.dnn

That may be deprecated. Which sample are you looking at?


Node can't be reselected

Describe the bug
Node can't be selected again if it was selected and then unselected.
Issue appears to be here https://github.com/AmrDeveloper/TreeView/blob/b86994b4ecacc7fab4ee4002f3b7ec6099a01634/treeview/src/main/java/com/amrdeveloper/treeview/TreeViewAdapter.java#L126
The previously selected node is always set as unselected even if the newly selected node is the same.
I propose following change:
if (currentNode == currentSelectedNode) {
  currentSelectedNode.setSelected(!currentSelectedNode.isSelected());
} else {
  if (currentSelectedNode != null) currentSelectedNode.setSelected(false);
  currentSelectedNode = currentNode;
}

To Reproduce
Steps to reproduce the behavior:

Create tree with at least one node
Click on the same node 3 times
The node state is unselected

Expected behavior
If user is clicking on the same node it's state should switch between selected/unselected
Screenshots
N/A
Smartphone (please complete the following information):

Device: API 28 Emulator

Additional context
Workaround for this issue is to click on different node, thus breaking the loop which is keeping the node in unset state.

Hello @pavelreiter,
Thank you for the report and suggestion I checked your solution and worked well, if you like you can make a PR with this solution if you want to work on it, if not I will add it and make a hot fix release
Thank you,
Amr Hesham

I am away from laptop, so if you would be so kind hotfix would be great.
Thank you, Pavel Reiter.
Dne pá 24. 2. 2023 21:35 uživatel Amr Hesham @.***>
napsal:

Hello @pavelreiter https://github.com/pavelreiter,
Thank you for the report and suggestion I checked your solution and worked
well, if you like you can make a PR with this solution if you want to work
on it, if not I will add it and make a hot fix release
Thank you,
Amr Hesham
—
Reply to this email directly, view it on GitHub
https://github.com/AmrDeveloper/TreeView/issues/8#issuecomment-1444435678,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABADLTMRD2BE5P3F7VGSCQLWZELQVANCNFSM6AAAAAAVHB3SC4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thank you @pavelreiter again, i commit the solution and will make hot fix release today

It is released now in version 1.1.3


Cannot connect via SSH with password

If i add a server in the ray app (on macOS) it will save a SSH password, but when connecting it will still try to use a certificate.
Ray version: 1.15.0
To Reproduce
Steps to reproduce the behavior:

Go to 'remote servers'
Click on 'add server' or 'edit server'
Use a password
Click on connect

Expected behavior
It should use a password login. If i create an empty file the problem is fixed :)
Screenshots

Desktop (please complete the following information):

macOS
Version 11.4

This was fixed in Ray 1.16.0 (released just now)! 😃

HI, using only SSH and still not working
im using 1.16.0 version of the App


Impact Analysis - Microsoft change to Sharepoint custom scripting options could impact MS.SHAREPOINT.4.1v1 and 4.2

💡 Summary
Microsoft is making updates to the custom scripting configuration options for Sharepoint and OneDrive in March 2024. The purpose of this issue is to determine if we need to remove or revise policies MS.SHAREPOINT.4.1v1 and 4.2 based on the changes.
https://techcommunity.microsoft.com/t5/sharepoint/removing-custom-scripting-on-sharepoint-sites/m-p/4055563
Implementation notes

[ ] Examine the Microsoft documentation which states that the scripting configuration is going to change.
[ ] Perform a hands-on analysis of the Sharepoint admin center for the two affected settings and document the impacts.
[ ] Document a recommendation on whether the baseline and ScubaGear need to change.
[ ] If changes are needed, create new issues.

Notes:
In March 2024:

The Custom Script setting, which determines if users can execute custom scripts on personal sites and self-service created sites will be removed.
A new PowerShell command, "DelayDenyAddAndCustomizePagesEnforcement", has been introduced. This command is available in the SharePoint Online Management Shell version 16.0.24524.12000, or above which allows delay of the change to custom script set on the Tenant until mid-November 2024 (previously May).

New PowerShell Command "DelayDenyAddAndCustomizePagesEnforcement":

False (default) - for site collections where administrators enabled the ability to add custom script, SharePoint will revoke that ability within 24 hours from the last time this setting was changed.

True - All changes performed by administrators to custom script settings are preserved. When the value is set to true, a banner shows in the active sites list of the SharePoint admin center informing that changes to custom scripts are permanent.

Need to test:

New PowerShell Command "DelayDenyAddAndCustomizePagesEnforcement once live
Current comandlets in ScubaGear (will probably need to change)

Additional Notes:
Post November:
Post November, on SharePoint sites if administrators wish to continue using features that are only available when unmanaged custom scripts are permitted to run, they will need to re-enable the running of custom scripts every 24 hours. This option does not impact existing custom scripts.
The NoScriptSite setting will be configured to True for all existing SharePoint sites and OneDrive sites except for below mentioned sites templates.
BLANKINTERNETCONTAINER#0 = Classic Publishing Portal site
CMSPUBLISHING#0 = Publishing Site
BLANKINTERNET#0 = Publishing Site
GROUP#0 = Team site
APPCATALOG#0 = App Catalog
CSPCONTAINER#0 = CSP Container
The execution of existing scripts in OneDrive and SharePoint sites will remain unaffected.
There will not be an option to enable custom script on OneDrive sites once the delay set using DelayDenyAddAndCustomizePagesEnforcement ends in mid-November 2024 (previously May).
Customers will retain the ability to permit the execution of custom scripts on specific SharePoint sites using the Set-SPOSite  -DenyAddAndCustomizePages PowerShell command or from the Active sites page in the SharePoint Admin Center.

Any modifications made to a site will be automatically reverted to False status within 24 hours, unless the new PowerShell command “DelayDenyAddAndCustomizePagesEnforcement” is used prior to mid-November 2024 (previously May). After mid-November 2024, the 24-hour reversion will occur regardless of this setting.

Met with @tkol2022 and @mitchelbaker-cisa on 04/15 to discuss and level set. Below are the actions and decisions made:

Open new issue to further investigate and define the risk of custom scripting through prototyping.
admin center message center mentioned in this issue has been pushed from Mar 2024 -> May 2024, close current issue and create new one once additional prototyping above is finished.

Met with @tkol2022 and @mitchelbaker-cisa on 04/15 to discuss and level set. Below are the actions and decisions made:

Open new issue to further investigate and define the risk of custom scripting through prototyping.
admin center message center mentioned in this issue has been pushed from Mar 2024 -> May 2024, close current issue and create new one once additional prototyping above is finished.

For the new issue related to bullet 1, name the issue "Develop and test custom scripting exploit to perform impact analysis of Sharepoint section 4 policies" and label the issue as handsonprototyping. The general scope would be to write a custom script that simulates a malicious action like downloading a set of files. Using test accounts we can simulate a phishing attack to lure a target user to click on a Sharepoint link that will execute the script within the context of the target user's identity.


[MINOR][DOC] Add class document for PruneFileSourcePartitions and PruneHiveTablePartitions

What changes were proposed in this pull request?
Add class document for PruneFileSourcePartitions and PruneHiveTablePartitions.
Why are the changes needed?
To describe these two classes.
Does this PR introduce any user-facing change?
no
How was this patch tested?
no

Test build #118194 has started for PR 27535 at commit 23813ec.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/22956/
Test PASSed.

Test build #118208 has started for PR 27535 at commit 5274b1a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/22968/
Test PASSed.

Test build #118194 has finished for PR 27535 at commit 23813ec.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118194/
Test PASSed.

Test build #118208 has finished for PR 27535 at commit 5274b1a.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118208/
Test FAILed.

retest this please

Test build #118225 has started for PR 27535 at commit 5274b1a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/22985/
Test PASSed.

Test build #118225 has finished for PR 27535 at commit 5274b1a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118225/
Test PASSed.

thanks, merging to master/3.0!

thank you all for help.


Email Settings Design Preview results in Internal Error and Templates are not applied

This PR related to this issue #285

Fix issue got internal error when preview email template
Fix issue when sending mail using sendgrid, mail template not applied

Summary by Sourcery
Fix internal error during email template preview and ensure templates are applied correctly when using SendGrid. Improve error handling and logging for email template parsing.
Bug Fixes:

Fix internal error when previewing email templates.
Ensure email templates are correctly applied when sending emails using SendGrid.

Enhancements:

Improve error handling and logging when parsing email templates.

Hi @mariobehling, thanks for the review
the domain name and token here is not real data, it just sample data which using for preview mail template, I have changed the domain name to sample-wikimania-live.eventyay.com for more clear.


Fix warnings on Check recent versions workflow

Hello @mlocati
I've noticed that the Check recent versions workflow raised some weak warnings:
See annontations on, e.g : https://github.com/mlocati/pecl-info/actions/runs/8698241017
Here is a PR to fix them

https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/
https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Shouldn't we have
printf 'list=%s\n' "$EXTENSIONS" >> $GITHUB_OUTPUT

instead of
printf '{list}={%s}\n' "$EXTENSIONS" >> $GITHUB_OUTPUT

?

@mlocati Fixed

Thanks!


The screen overflows when locale is set to Locale("zh", "CN")

 final DateTime? picked = await showMonthPicker(
        context: context,
        roundedCornersRadius: 20,
        customWidth: 400,
        initialDate: selectedDate,
        firstDate: DateTime(2000),
        lastDate: DateTime(2101),
        locale: const Locale("zh", "CN"),
        unselectedMonthTextColor: Colors.black,
        currentMonthTextColor: Colors.black,
        confirmWidget: const Text(
          '确定',
          style: TextStyle(color: Colors.black),
        ),
        cancelWidget: const Text(
          '取消',
          style: TextStyle(color: Colors.black),
        ));

Versions
month_picker_dialog: 4.0.1
Flutter: 3.22.3
Dart: 3.4.4

I tried setting customWidth to make the dialog larger to solve the overflow issue, but it seems like the customWidth is also not working.

@wangyewei probably this is happening because of the screen size of your device + the size of the header text (gonna send my test project):

teste.zip
on this version of the package, the only option you have is to change the arrow size with arrowSize parameter.
Please try the new 5.0.0-dev.1. With this one you can change the text style of the header to avoid the overflow on smaller screens. I'm still updating it, so expect some changes to the stable 5.0.0, but it is already on an usable state.
monthPickerDialogSettings: const MonthPickerDialogSettings(
  headerSettings: PickerHeaderSettings(
    headerPageTextStyle: TextStyle(fontSize: 10)
  )
)

@Macacoazul01 Thanks for your reply, sounds like an awesome change in the next version.
In my version, I actually tried setting the arrow size to zero (arrowSize: 0), but I still encountered the same issue.
here is my simulator devices:

IOS simulator is iPhone 14 Pro

Looking forward to your update.

Probably fixed in 5.0.0
Please open a new issue if you find any bugs on it


[WIP] Use miniz_oxide directly in rust backend

This avoids the C API and it's unsafeness entirely.
Can probably further simplify a bit by altering the Stream struct and direction trait StreamWrapper doesn't have to be an enum, neither are public.
Needs some thorough testing.
fixes #181

Thanks for this! One thing I'd like to do here is to reduce the number of #[cfg] necessary. Would it be possible to push more of the #[cfg] into the src/ffi.rs module itself? That way the backend-specific implementations could all be located there and ideally src/mem.rs is free from #[cfg] (largely at least)

I'll look into it.

Right, pushed all of the back-end specific stuff into ffi.rs. Only ones that remain are the zlib-specific public functions.
Is there some specific reason the zlib mz_stream is boxed, while the miniz one is not? Otherwise they could share that code.

That's actually probably a bug, want to go ahead and box both of them up? I think there's also an error on nightly to handle?

I'll box both then.
The nightly error may actually be something that's in the original code, the lint was added recently, and the original code uses mem::zeroed() here too. I suspect it's the extern fn fields that the compiler does not want to assume zero is a valid value for.

Hm yeah so I think I messed that up when I wrote libz-sys.
If you wouldn't mind adding #[allow(invalid_value)] for now I can try to clean this up after this lands.

👍


Editor

@vmg
I realize you are probably way ahead of me in your internal version of gumbo, but I started work on adding a separate gumbo_edit.h "bolt-on" interface that basically extracts pieces from parser.c and reworks them to provide for a simple node editing interface.  The interface is described in gumbo_edit.h.  By simply adding gumbo_edit.h and gumbo_edit.c to your v1.0.0 tree (or any tree that has your memory fixes and public api changes) you can add this support.  I plan to keep them separate if I can convince official upstream to not make the arena changes, allow the api changes, and handle the prev/next stuff inside the gumboc_py interface and not in the main tree.  There is still a chance that might happen but I do not believe it is a high chance.
Please ignore the perf hash fix commits which I manually added to my own tree (they are yours).  Instead please see:
"Add simple node editing support to gumbo".
This is just a first shot at things but I wanted to offer them to you if anyone there wanted them.
Thanks again for all of your help in improving gumbo to the point it was useable for us.  I do appreciate it.

Thanks for taking the time to write this, @kevinhendricks. This really is in line with what I had in mind for a simple modification API. Have you considered submitting it upstream for discussion?

@vmg
Upstream has clearly stated multiple times he does not want to support any modification to the tree after parsing.  In fact, in a recent exchange on your  original pr  he stated he has to say "no" to ALL such requests because it would grow the api in too many directions.
I have tried to negotiate with upstream to adopt a "happy compromise" as follows:

don't add prev/next but instead inside gumboc use original tag and text start positions to build prev/next for python only

adopt your pr's memory changes and slight api changes so that something like gumbo_edit.c is possible

make the arena memory allocation a build time option (as it stands his current area implementation uses multiple chunks that are not allocated atomically so is not thread safe).  Also, any modifications would have to track the chunk used to properly free it which kind of kills the modification bits.

not to adopt any complicated call back approach just to implement treebuilders/tree walkers interface.

If he did that, I would  keep helping him with gumbo development.  To get him to consider it I had to promise to keep gumbo_edit.c as a "bolt on" drop-in that is not officially part of gumbo so as not to complicate his parser interface any further.
He has made two competing Proposals (see the upstream site)  and asked others to weigh in on what they want.  Virtually no one has said a thing.  I do not think he has an active base of contributors to his project.  I think my chances of success are slim to none, but I have already incorporated these changes into Sigil master and will be using them without or without upstream.

On Mar 4, 2015, at 7:56 AM, Vicent Marti<EMAIL_ADDRESS>wrote:
Thanks for taking the time to write this, @kevinhendricks. This really is in line with what I had in mind for a simple modification API. Have you considered submitting it upstream for discussion?
—
Reply to this email directly or view it on GitHub.

Take care,
Kevin


Rerun build

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

The first build seem to have failed due to a race condition: one of the dependencies (odc) was unavailable at build time, since it was also new. Bumping up the build number to rerun the build, hopefully it will be enough.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


500 error during onBeforeRender on catch-all route

Description
Error<EMAIL_ADDRESS>    at createHttpResponseObject (/Users/daguilera/dev/personal/vite-ssr-project/node_modules/vite-plugin-ssr/dist/cjs/node/runtime/renderPage/createHttpResponseObject.js:21:32)
    at renderPageContext (/Users/daguilera/dev/personal/vite-ssr-project/node_modules/vite-plugin-ssr/dist/cjs/node/runtime/renderPage/renderPageContext.js:67:92)
    at renderPageAttempt (/Users/daguilera/dev/personal/vite-ssr-project/node_modules/vite-plugin-ssr/dist/cjs/node/runtime/renderPage.js:163:36)
    at renderPage (/Users/daguilera/dev/personal/vite-ssr-project/node_modules/vite-plugin-ssr/dist/cjs/node/runtime/renderPage.js:41:35)
    at file:///Users/daguilera/dev/personal/vite-ssr-project/server/index.ts:34:29

server/index.ts:34:29 is where it does:
const vite = await import('vite')
    const viteDevMiddleware = (
      await vite.createServer({
        root,
        server: { middlewareMode: true }
      })
    ).middlewares

I started the scaffolding with npm init vite-plugin-ssr@latestand tried to create a catch-all route with feching data
Error Message + Error Stack
No response

Thanks for the bug report. Can you publish the reproduction as a GitHub repository?

Can you publish the reproduction as a GitHub repository?

Closing in the meantime. I'll re-open if I can get my hands on a reproduction.


Holodex forgot to reuse old three dot menu HTML element under video stream thumbnail?

Version: 2.13
I'm pretty sure that the previous version (not sure which version, maybe 2.12 and older) didn't have this behavior, and instead reuse old HTML element as usual?
Seems like Holodex is forgot to reuse old menu HTML element and instead creating an entire new HTML element (by looking at the chrome console).
The bug in action:
https://streamable.com/7rchct
OS: Windows 10
Browser: Vivaldi 4.0.2312.33 (latest)

:O Surprised you noticed. We reverted the change because we had to refactor some things for virtual scrolling for playlists, and we weren't really sure if there was any real performance implications for reusing the menu. Since the html element only gets created when it's clicked.

:O Surprised you noticed. We reverted the change because we had to refactor some things for virtual scrolling for playlists, and we weren't really sure if there was any real performance implications for reusing the menu. Since the html element only gets created when it's clicked.

How did I found this bug? I was developing a personal Tampermonkey script to hook Holodex with streamlink instead of copy pasting the youtube link into command line everytime and somehow the script suddenly broke. That's how 😅

Should just PR the functionality into Holodex as an experimental feature - we can have a section of settings "Labs" dedicated to odd but low-performance-impact features like that.
Gonna close the issue though since it's not something we intend on reverting


[TFLite] 16x8 reference kernel for PAD operator

In this PR I extend the operator PAD for 16x8 quantization mode.
This operator is present in Mobilenet V1/V2 from keras.applications.

cc/ @renjie-liu.

Hi @renjie-liu Thanks for the review. I addressed your comments - please take a look.

@yyoon @renjie-liu this PR has been failing internally , can you please help with this ?

@wwwind Can you please resolve conflicts? Thanks!

@renjie-liu  @yyoon can you please help with internal failures ?

Hmm. AFAICT, the error seems unrelated to this PR. Let me try re-running the internal CI builds.

@rthadur All checks have passed now.

Hi @renjie-liu Corrected. Please take a look. Thanks!


feat: build fix

:tada: This PR is included in version 0.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Revert "Cargo-culted, but then Swift-ified and cleaned-up ModuleDependencyGraph"

Reverts apple/swift-driver#283. It broke the CMake build.

@swift-ci please test


Add an AWS EC2 Security Group Icon

cdk-dia currently has no icon for a security group.
the following cdk App:
import * as cdk from '@aws-cdk/core';
import * as ec2 from '@aws-cdk/aws-ec2';

export class SoTsStack extends cdk.Stack {
  constructor(scope: cdk.Construct, id: string, props?: cdk.StackProps) {
    super(scope, id, props);

    const vpc = new ec2.Vpc(this, 'vpc')

    const sg = new ec2.SecurityGroup(this,'sg',{
      vpc: vpc
    })

    new ec2.Instance(this,'instance',{
      instanceType: ec2.InstanceType.of(ec2.InstanceClass.T3, ec2.InstanceSize.MEDIUM),
      vpc: vpc,
      machineImage: ec2.MachineImage.latestAmazonLinux(),
      securityGroup: sg
    })
  }
}

yields the following diagram:

Which misses a security group icons.
The icons mapping is at src/diagram/aws/awsResouceIconMatches.json
All current CDK icons are taken from https://aws.amazon.com/architecture/icons/
One has to figure out whether AWS has such an icon or how to come up with an icon that is descriptive and also consistent with current AWS/CDK-Dia icons.

The image seem to be gone from the current set. There was one included when they used to be called "AWS Simple Icons". There is a low quality version preserved at Wikimedia: https://commons.wikimedia.org/wiki/File:AWS_Simple_Icons_Groups_Security_Group.svg
Though the image was not meant to be used as an icon.

I wonder if it would be feasible to display these group resources in such a way.


README instruction not clear

Hi could you please specify in the instructions which steps are performed on the local machine and which are performed on the remote server?

i also had this problem. Could be clearer for sure

I will try to update the instructions to add more details.

Thank you.
I am running vagrant and connect using the command "vagrant SSH".
I run gomate inside the guest.
On Thu, 19 Oct 2017, 12:32 Rafael Maiolla<EMAIL_ADDRESS>wrote:

I will try to update the instructions to add more details.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rafaelmaiolla/remote-vscode/issues/41#issuecomment-337868182,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATuCwsWK0dEjiNKxIbRITq__Q3OMajGBks5styVOgaJpZM4P8kc5
.

@rafaelmaiolla
I got it working with vagrant.
Host is OSX
Guest is ubuntu

ssh -R 52698:<IP_ADDRESS>:52698 -i .vagrant/machines/default/virtualbox/private_key -p 2222<EMAIL_ADDRESS>Welcome to Ubuntu 16.04.3 LTS (GNU/Linux 4.4.0-92-generic x86_64)

 * Documentation:  https://help.ubuntu.com
 * Management:     https://landscape.canonical.com
 * Support:        https://ubuntu.com/advantage

0 packages can be updated.
0 updates are security updates.

Last login: Mon Oct 23 13:40:49 2017 from <IP_ADDRESS>
vagrant@vagrant:~$ gomate /
2017/10/23 13:44:05 VSCode 1

Now what though ?
VSCode 1, but what is it. How to see the remote files in my VSCODE folder tree etc ?
Maybe i misunderstand how to use this tool or what it is really for ?

This is not used to see all the remote file in VSCode. This extension is used to edit individual files from an ssh session.
As said in the docs, this is a package that implements the Textmate's 'rmate' feature for VSCode.


Cleaner ExportState via document for SDK v47+

Closes #721
Summary
If a chain is on SDK v47+, we should output exports to a file directly. Then we return the contents from this file instead of returning a messy stdout + err for parsing.
Tests both 45 and 47 usecases

@Mergifyio backport v7


feat: Add request to UnleashContext

Developers might expect to find more than just predefined Unleash functions (e.g. getUserId, getRemoteAddress). This adds getRequest which returns the request object, this way stategies doesn't need to inject that themselves.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.007%) to 96.689%

Totals

Change from base Build<PHONE_NUMBER>:
0.007%

Covered Lines:
339

Relevant Lines:
341

💛  - Coveralls

I was thinking, it might be worth adding getHeaders too or instead of this. What do you think?

Thank you for your contribution! This is a good idea.

I was thinking, it might be worth adding getHeaders too or instead of this. What do you think?

I think the headers do not need to be provided separately. If you want to use them in your strategy, you can access them via the request object.


Group include-script generation doesn't work on mono

@smoothdeveloper

Failed : Paket.IntegrationTests.LoadingScriptGenerationTests.simple dependencies generates expected scripts
Expected is <System.String[8]>, actual is <System.String[6]>
Values differ at index [4]
Expected string length 22 but was 17. Strings differ at index 8.
Expected: "include.main.group.csx"
But was:  "include.nunit.csx"
-------------------^
at Paket.IntegrationTests.LoadingScriptGenerationTests.simple dependencies generates expected scripts () <0x4189be10 + 0x00419> in :0

@forki there is also another integration test that I had to make inconclusive on mono back when we added that feature.
I haven't had chance to setup build on Mac or Linux box yet, I'll apply the same inconclusive call to this test when running on mono.


Update Lab.md

Because the method HasData() checks only on tokencache and on the login fase of the lab there is no tokencache yet. So you must check on cachedUserId = $"{userId}_UserCache"; but in code he will only check on this: cacheId = $"{userId}_TokenCache";.
The result is that the solution or the lab not work and you will always stay on the Sign-In buttons.

Thanks @jbranders! You mentioned that you got into a state where you could never get the Sign Out button to show because of this, but I don't reproduce this. At the point that you implement this cache you should always have a token cache unless you're possibly hitting some error in the token flow. I may have missed something in the ordering of the steps though. Can you tell me where in the lab you've reached when you see this state?

Closing this as lab.md no longer exists.


Allow CLI options to be set in mjmlconfig as well

Is your feature request related to a problem? Please describe.
I parse a lot of files and appending --config.minify=true to every file seems odd. It would be great if I could just add it to the .mjmlconfig.
Describe the solution you'd like
In .mjmlconfig:
{
  "config": {
    "minify": {
      "enabled": true,
      "option": {
        "minifyCSS": true
      }
    }
  },
  "packages": []
}

Describe alternatives you've considered
Another solution would be to allow wildcards. So I could do this: mjml email/*.mjml -o email/[filename].html.
Additional context
I guess both of my solution would be great. So mjml can compile multiple files at once and also takes options from a file instead of creating a long CLI command.

@iRyusa nice, haven't found any documentation about that :), worth adding I guess.

+💯 for this suggestion. We have gotten around this by wrapping the mjml commands in various scripts for devs to use, but sometimes you do still have to run mjml directly, and it would be really nice to have our preferred CLI options set without having to think about it. Much easier for someone new to this area of the codebase to use.

We'll try to push this asap 👍


Ensure socket directory present

In cases where the socket directory is in tmpfs it cannot always be certain to exist.  Debian has a hack around this by running a shell script to generate the needed directory structure when the socket dir is not /tmp/.xrdp.  This does the equivalent at startup of sessman.  This should fix issue #801.

+1

Replaced by #809.


ModuleNotFoundError: No module named 'Undetected_Chromedriver'

How do you solve this? Please help

if you cd into where you have the requirements.txt folder saved into this should resolve the problem

Hello. A new update was just pushed with more precise instructions. Please check it and let me know if everything's working good.


When i am commiting my code to github i am getting an authentication failed fatal error.

I have uploaded my screen shot of the error message.help me!!

try this command
git pull origin master
then git push -u origin master`
Note:  if it is not working please check the git url in .git/config file

I have tried all the commands and It is not working.
On Fri, 1 May 2020 at 17:18, srmanikanta<EMAIL_ADDRESS>wrote:

try this command
git pull origin master
then git push -u origin master`
Note: if it is not working please check the git url in .git/config file
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/QT-DevOps/DevOpsIssues/issues/846#issuecomment-622357370,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMG5ZEKRX2Q6ICW7LNH47KLRPKZHNANCNFSM4MW5K4CA
.

change username/password in credential manager and try to push to github repository it will work

hi
open credential manager and delete old one


Cypress - add "@briebug/cypress-schematic"

Need add "@briebug/cypress-schematic" to run Cypress
npm run test:e2e

@pehleme thank you very much for opening this. Will check it out!

This error is fixed but there seems to be an issue with the schematic https://github.com/briebug/cypress-schematic/issues/103


Update zio, zio-test-sbt to 2.0.21

About this PR
📦 Updates

dev.zio:zio
dev.zio:zio-test-sbt

from 2.0.0 to 2.0.21
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #171.


[move-prover] Specification schema type checking and expansion.

This implements spec schemas in the type checker and translator. Right now, conditions and invariants introduced by schemas are expanded in the inclusion context during translation time, so the boogie translator stays unchanged. However, the design is ready to pass schemas also to the backend should we want to prove properties about them standalone.
This PR does not yet implement the apply weaving operator for schemas. This is syntactic sugar and will be added later once we see the approach reasonably works.
Here is a quick intro on how to use schemas. A schema is a named, possibly generic specification block. It can contain any item other spec blocks can contains (requires, aborts_if, ensures, assert, assume, invariant). Those items can use global specification variables and functions. A schema need to declare any local variables it uses so it can be reasoned about independently (which distinguishes schemas from pure macros). For example:
spec schema Increases {
   x: num; result: num;
   ensures result >= x;
}

Such a schema can be included in another spec block, where the variables it introduces are matched against existing ones:
fun foo(x: u64): u64 { x + 1 }
spec fun foo {
  include Increases; // matches x and result
}

At the point where a schema is included, the items it contains are checked to be valid in the inclusion context. For example, if a schema contains invariants and is included in function spec, an error will be produced.
If needed, renaming can be applied at schema inclusion time to match variables in the context:
fun foo2(x: u64): (u64, u64) { (x + 1, x + 2) }
spec fun foo2 {
  include Increases{result: result_1};
  include Increases{result: result_2};
}

Schemas can be generic, in which case the type parameters need to be provided at inclusion time (no type inference for schemas):
spec schema Identity<X> {
  x: X; result: X;
  ensures x == result;
}
fun identity<X>(x: X): X { x }
spec fun identity {
  include Identity<X>;
}

Schemas can include other schemas in order to build structured specifications. If a schema is included in another schema, any variables will be added to this context if they aren't yet declared:
spec schema Signature {
  x: u64;
  result: u64;
}
spec schema Decreases {
  include Signature;
  ensures x > result;
}
spec schema Increases {
  include Signature;
  ensures x < result;
}

Motivation
Make specifications better readable.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Added tests for spec-lang (schema type checking) and for move-prover (test marketcap_as_schema.move.
Related PRs
NA

FYI @sblackshear @emmazzz @cbarrettfb

@bors-libra r=DavidLDill

:pushpin: Commit ffd6da4 has been approved by DavidLDill

:hourglass: Testing commit ffd6da4df83bf9f6cca225b156733d304fb19bbd with merge 36e45250e0f4a6d349d49f9402724284ed0df8ea...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: DavidLDill
Pushing 36e45250e0f4a6d349d49f9402724284ed0df8ea to master...


Use cvc4 1.5 release

Acceptance criteria:

[x] Unit tests that use cvc4 should run with version 1.5 (on Windows and Linux).
[x] Add cvc4 1.5 release to TorXakis' dependencies.
[ ] Run sqatt tests with cvc4 as solver (locally for now, if possible incorporate this into our CI workflow).
[ ] Update the installer to use cvc4 1.5 version.

It might be that some examples can't be solved with cvc4 (like customer orders), and that is ok.

CVC4 can't solve following examples:
Control Loop

Spec Measure (left overnight, in the morning CVC4 was running with 0% processor consumption)
(testing continues)

I skipped the tests in previous post and launched a build in Semaphore to see Linux behaviour.
Control Loop
Spec Produce fails - Stepper cannot start.
(testing in progress)


Silly question: What's that color scheme?

It looks pretty sweet!

I think it is Oceanic Next Color Scheme. You may try see if it works for you :)

I figured it out, it's an old version of Material Theme. Evan You forked it:
https://github.com/yyx990803/material-theme/blob/master/schemes/Material-Theme.tmTheme


MongoTemplate Bean Creation Issue

We're noticing an issue lately where applications will stand up fine locally but will not be able to stand up in PCF and the end result will end up being a lack of MongoTemplate bean. We've traced it down to the cloud connectors library but the exact issue is a bit beyond our current capabilities.
A sample project to reproduce is located here
That project will deploy perfectly fine into PCF until you bind a user provided service with a mongodb connection string in it. As soon as you provide that, the MongoTemplate bean will fail to be created and the application will fail to stand up its application context.

@azizabah Can you share the command you are using to create the user-provided MongoDB service instance, and if possible the value of the VCAP_SERVICES environment variable created for the application by CF (redacting any sensitive details as appropriate)? Also check for any exceptions in the application logs at startup.
I suspect what's going on is this:

When the app is started without being bound to the user-provided service instance, Connectors doesn't detect a bound service and does nothing. This allows Spring Boot to configure the MongoDB connection using properties from application.properties.
When the app is started with the bound service instance, Connectors detects the bound service but something in the connection details prevents the MongoTemplate from being created.

We created the user provided service in this case through the PCF UI but I've grabbed and sanitized the JSON so it should be effectively identical to a cf cups service-name -p JSON block.
{"database":"db","uri":"mongodb://user:password@node1:27017,node2:27017,node3:27017/db?replicaSet=nameOfSet"}

This block contains a snippet from the VCAP_SERVICES being injected and retrieved from the Settings tab -> Environment Variables section of PCF UI.
  "system_env_json": {
    "VCAP_SERVICES": {
      "user-provided": [
        {
          "name": "mongo",
          "instance_name": "mongo",
          "binding_name": null,
          "credentials": {
            "database": "db",
            "uri":"mongodb://user:password@node1:27017,node2:27017,node3:27017/db?replicaSet=nameOfSet"
          },
          "syslog_drain_url": "",
          "volume_mounts": [],
          "label": "user-provided",
          "tags": []
        }
      ]
    }
  }

@azizabah These credentials look well formed to me. I haven't seen the MongoDB replicaset URL syntax before, but it looks valid from the documentation.
Do you see any errors in the application logs at startup that might indicate a problem initializing the MongoDB connection?
There are several MongoDB related beans that get created to support a Spring Data MongoDB repository. When Connectors detects a bound MongoDB service, it should create a MongoDbFactory  bean using the information from VCAP_SERVICES. This overrides the Boot autoconfiguration that creates a MongoDbFactory bean using properties if none is found. The MongoTemplate bean is always created by Boot.
One way to test the connection string being provided in the user-provided service instance would be to set this same value in spring.data.mongodb.uri without the service being bound and see if Boot can successfully create the connection.
If you add Boot Actuators to the application, you can use the beans and conditions endpoints to inspect these conditions (provided the app will start so the endpoints are available).
One more note: since this sample is a Spring Boot application, you should consider using the new java-cfenv library instead of Spring Cloud Connectors. java-cfenv provides a direct mapping from VCAP_SERVICES to Spring Boot properties, giving the application the ability to configure other connection details in a more Boot-centric way.

@scottfrederick - There are no errors other than the final one related to an unsatisfied dependency exception while trying to stand up the spring data repo that would require the MongoTemplate.
2019-03-12T10:25:12.840-04:00 [APP/PROC/WEB/0] [OUT] APPLICATION FAILED TO START
2019-03-12T10:25:12.840-04:00 [APP/PROC/WEB/0] [OUT] Description:
2019-03-12T10:25:12.840-04:00 [APP/PROC/WEB/0] [OUT] Parameter 0 of constructor in com.example.shanemongotest.MyController required a bean named 'mongoTemplate' that could not be found.
2019-03-12T10:25:12.840-04:00 [APP/PROC/WEB/0] [OUT] Action:
2019-03-12T10:25:12.840-04:00 [APP/PROC/WEB/0] [OUT] Consider defining a bean named 'mongoTemplate' in your configuration.

We have already tested the connection string exactly as you suggested and it works fine. The issue only occurs when we bind to the user provided service. We do not even have to use the value for it to break our code. The mere presence of it in the VCAP_SERVICES appears to do the damage. If we do use it, the interesting part is that we know it's getting the connection string correctly and using it correctly for other purposes because we see these messages in the logs:
2019-03-06T16:05:50.765-05:00 [APP/PROC/WEB/0] [OUT] 2019-03-06 21:05:50.765 INFO 16 --- org.mongodb.driver.cluster : Monitor thread successfully connected to server with description ServerDescription{address=node1:27017, type=REPLICA_SET_SECONDARY, state=CONNECTED, ok=true, version=ServerVersion{versionList=[3, 6, 9]}, minWireVersion=0, maxWireVersion=6, maxDocumentSize=16777216, logicalSessionTimeoutMinutes=30, roundTripTimeNanos=4114132, setName='nameOfSet', canonicalAddress=node1:27017, hosts=[node1:27017, node3:27017, node2:27017], passives=[], arbiters=[], primary='node3:27017', tagSet=TagSet{[]}, electionId=null, setVersion=295500, lastWriteDate=Wed Mar 06 21:05:42 UTC 2019, lastUpdateTimeNanos=24780895244931}
2019-03-06T16:05:50.765-05:00 [APP/PROC/WEB/0] [OUT] 2019-03-06 21:05:50.765 INFO 16 --- org.mongodb.driver.cluster : Monitor thread successfully connected to server with description ServerDescription{address=node2:27017, type=REPLICA_SET_SECONDARY, state=CONNECTED, ok=true, version=ServerVersion{versionList=[3, 6, 9]}, minWireVersion=0, maxWireVersion=6, maxDocumentSize=16777216, logicalSessionTimeoutMinutes=30, roundTripTimeNanos=6230867, setName='nameOfSet', canonicalAddress=node2:27017, hosts=[node1:27017, node3:27017, node2:27017], passives=[], arbiters=[], primary='node3:27017', tagSet=TagSet{[]}, electionId=null, setVersion=295500, lastWriteDate=Wed Mar 06 21:05:42 UTC 2019, lastUpdateTimeNanos=24780897012182}
2019-03-06T16:05:50.765-05:00 [APP/PROC/WEB/0] [OUT] 2019-03-06 21:05:50.765 INFO 16 --- org.mongodb.driver.cluster : Monitor thread successfully connected to server with description ServerDescription{address=node3:27017, type=REPLICA_SET_PRIMARY, state=CONNECTED, ok=true, version=ServerVersion{versionList=[3, 6, 9]}, minWireVersion=0, maxWireVersion=6, maxDocumentSize=16777216, logicalSessionTimeoutMinutes=30, roundTripTimeNanos=6633452, setName='nameOfSet', canonicalAddress=node3:27017, hosts=[node1:27017, node3:27017, node2:27017], passives=[], arbiters=[], primary='node3:27017', tagSet=TagSet{[]}, electionId=7fffffff0000000000000033, setVersion=295500, lastWriteDate=Wed Mar 06 21:05:42 UTC 2019, lastUpdateTimeNanos=24780897414282}

which indicates that the MongoDb driver correctly parsed the passed in log and was able to connect.
The other interesting piece when I turn debug on is the MongoDataAutoConfiguration results. When I am not connected to the user provided service and everything is going to work it sends out these results:
2019-03-12T10:17:34.962-04:00 [APP/PROC/WEB/0] [OUT] MongoDataAutoConfiguration matched:
2019-03-12T10:17:34.962-04:00 [APP/PROC/WEB/0] [OUT] 2019-03-12 14:17:34.961 DEBUG 17 --- [ main] ConditionEvaluationReportLoggingListener :
2019-03-12T10:17:34.963-04:00 [APP/PROC/WEB/0] [OUT] - @ConditionalOnClass found required classes 'com.mongodb.MongoClient', 'com.mongodb.client.MongoClient', 'org.springframework.data.mongodb.core.MongoTemplate' (OnClassCondition)
2019-03-12T10:17:34.963-04:00 [APP/PROC/WEB/0] [OUT] - AnyNestedCondition 1 matched 1 did not; NestedCondition on MongoDataAutoConfiguration.AnyMongoClientAvailable.FallbackClientAvailable @ConditionalOnBean (types: com.mongodb.client.MongoClient; SearchStrategy: all) did not find any beans of type com.mongodb.client.MongoClient; NestedCondition on MongoDataAutoConfiguration.AnyMongoClientAvailable.PreferredClientAvailable @ConditionalOnBean (types: com.mongodb.MongoClient; SearchStrategy: all) found bean 'mongo' (MongoDataAutoConfiguration.AnyMongoClientAvailable)

when I am connected to the user provided service and it is going to fail, it sends out these results:
2019-03-12T10:18:53.959-04:00 [APP/PROC/WEB/0] [OUT] MongoDataAutoConfiguration:
2019-03-12T10:18:53.959-04:00 [APP/PROC/WEB/0] [OUT] - AnyNestedCondition 0 matched 2 did not; NestedCondition on MongoDataAutoConfiguration.AnyMongoClientAvailable.FallbackClientAvailable @ConditionalOnBean (types: com.mongodb.client.MongoClient; SearchStrategy: all) did not find any beans of type com.mongodb.client.MongoClient; NestedCondition on MongoDataAutoConfiguration.AnyMongoClientAvailable.PreferredClientAvailable @ConditionalOnBean (types: com.mongodb.MongoClient; SearchStrategy: all) did not find any beans of type com.mongodb.MongoClient (MongoDataAutoConfiguration.AnyMongoClientAvailable)

I'm not sure what's causing the "mongo" MongoTemplate bean to be created in one case and not in the other but I'm sure that's where the culprit is.

@scottfrederick - So I tried swapping out cloud connectors for the java-cfenv and everything seems to work perfectly. I'm going to try it on a more complicated service (also using RabbitMQ) and see how that goes. Is this the suggested Pivotal / Spring approach going forward for building out connections in a cloud environment?

@azizabah It's possible that an incompatibility has crept in between the bean that Connectors creates and the beans that MongoDataAutoConfiguration expects. I can look to see where MongoClient fits into this picture along with all the other beans.

Is [java-cfenv] the suggested Pivotal / Spring approach going forward for building out connections in a cloud environment?

Yes, as mentioned in the blog post introducing java-cfenv. Spring Cloud Connectors has code for creating connection beans from bound services, and Spring Boot has code to manage connection beans and other beans from properties. Keeping Connectors in sync with all versions of Boot and Spring Data projects has been a challenge for some time. The new approach lets Boot be fully in control, simply populating Boot properties from values in VCAP_SERVICES. Connectors can still be used in cases where Boot autoconfiguration isn't applicable (e.g. non-Boot applications, legacy apps that require multiple connections to the same type of service), but java-cfenv is strongly recommended for Boot apps.
If you encounter any issues with the java-cfenv approach, please file an issue in that repo and we'll try to address it quickly.

@scottfrederick - We're going to look at moving forward with java-cfenv. It appears to be doing everything we want to. Thanks for your assistance with this.


ci: Remove codeball action

1. Why is this change necessary?
The codeball action currently fails for all pull requests:

Warning: Unexpected input(s) 'GITHUB_TOKEN', valid inputs are ['approvePullRequests', 'labelPullRequestsWhenApproved', 'labelPullRequestsWhenReviewNeeded', 'labelPullRequestsWhenCarefulReviewNeeded', 'approveThreshold', 'carefulReviewThreshold', 'failJobsWhenReviewNeeded', 'codeSuggestionsFromComments']

2. What does this change do, exactly?
~Remove the GITHUB_TOKEN input.~
Edit: I just noticed that the codeball service apparently has shutdown:
https://github.com/shopware/platform/actions/runs/5546483050/jobs/10126825298
Therefore I removed the action completely.
3. Describe each step to reproduce the issue or behaviour.
Create a pull request.
4. Please link to the relevant issues (if any).
-
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[ ] I have written tests and verified that they fail without my change
[ ] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

🤖 Generated by Copilot at a34bbef
Removed unused GitHub workflow for codeball generation. This workflow was a legacy feature that is no longer needed for the current release process.

🤖 Generated by Copilot at a34bbef

Remove codeball workflow that was used to generate codeballs for the Shopware 6 installer (link)

Thanks!


[FEATURE] unit testing

Is your feature request related to a problem? Please describe.
Hello guys. I have tried to code using TDD approach as i do in my another projects. I'm a man with 5 year experience of php laravel framework and i know everything about it. But you guys create such a new approach in php community and i'm very excited about it.
So, i have the next problems with unit testing or feature testing now:

There no pocibility to mock objects in DI container. I have a service and i inject him into controller constructor. In my test i want to replace inject object with my test double. And i have not found any pocibility to do this. I't very strange for me.
There no good pocibility to test http requests inside my app. I want to test http headers, status codes, etc. There are no pocibility for this right now.

Describe the solution you'd like
I want to write my tests with easy as possible.
The solution for this is to make possible mock di objects and make integration on framework level with phpunit as it works in
https://github.com/illuminate/testing
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@Spartaques Could you provide some code to explain about how you mock the objects in DI in Laravel ?

It describes here https://laravel.com/docs/8.x/mocking

I understand that this is only for conviniet, but this stuff makes framework greater.


bug on shortcut when choosing marking

Write text in the editor that won't trigger marking, then type e.g. 2, an error will appear in the console.

Hey @AndersonCeci, please test this, and close the issue if there's no error.


How to create api documentation automatically?

I have added it to fiber but it shows defaults.
How can I add my endpoints in documentation?

Hi @rngallen, this package does not generate the openapi spec file. As an alternative you can use swag and go:generate to make this work, check this example for reference.


add homework2

video drift link: https://video.drift.com/v/abVw7pQlDxAedWgbDmRviWOoy8CI3Iv1uWsOzTkTcaXs/

Metric
Value

alert_status
:white_check_mark: OK

bugs
1

code_smells
19

reliability_rating
D (at least 1 Critical Bug)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
:x: ERROR

bugs
1

code_smells
15

reliability_rating
D (at least 1 Critical Bug)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
:x: ERROR

bugs
0

code_smells
17

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
:x: ERROR

bugs
0

code_smells
17

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
:x: ERROR

bugs
0

code_smells
14

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
:x: ERROR

bugs
0

code_smells
14

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


[desktop_multi_window] close button callback

Is there a callback button for the window close button? Or at least to hide it? I need to use the agora plugin in a separate window and if the window is closed the camera continues to record.

the same as https://github.com/MixinNetwork/flutter-plugins/issues/307
closing