Docker compose options.

Adds support for docker compose options using a 'with' fluent style syntax.
This enables a client to pass for example, the '--compatibility' option to the docker-compose command.
Parameterized Junit Test has been added.
Modified code in line with review @kiview
Changed options method to accept vargs @rnorth
I had some issues re-basing (squashing) my branch commits. The operation seemed to conflict with new changes in the master branch. I managed to commit in the end, but without squashed commits from my branch. Apologies if the git history is messy. If anybody can give me some advise in this area, please do :)

No problem, I will make the changes as soon as possible.
Do I reuse the same branch then rebase to squash the commits together? What is the procedure you guys use?

@Gapmeister66 we use 'Squash and Merge' when we merge, so there's no need for you to manually squash or rewrite your commits before then (unless you want to).

@rnorth
Hello Richard
How often do you guys run CI scripts over my branch ? I'm wondering whether CI has run over all of my commits thus far? The reason I ask is that my build failed at:
'Build and test with Gradle (:docs:examples:junit4:generic:check)'.
It seems to have happened since my last commit (varargs), but that just might be a coincidence if this is my first CI build. There is no error logged as far as I can tell.
I'm not a Gradle expert (I use Maven in my day job), does anybody have an idea of why my build failed?
-- and I though this would be easy :)

Fixed merge conflicts.


Missing disk free/total size reporting

Disk size reporting seems to be edited out since version 3.1.3, and Bugsnag site shows 0 bytes free disk for all crashes

Thanks for the report, @voidless. Taking a look.

Can you check if this is affecting particular devices or all iOS devices? In particular, I'm checking whether this only affects newer iOS devices or perhaps only iPads.

I have not found any crashes with disk statistics in my account

Thanks, @voidless.

Can confirm this one as well, all my crash reports shows up as 0 bytes free.

There was freeDisk and diskSize.
https://github.com/bugsnag/bugsnag-cocoa/blob/3e1e351766314f7a2f5bffb5389133dd471216cc/bugsnag/BugsnagNotifier.m#L354
https://github.com/bugsnag/bugsnag-cocoa/blob/3e1e351766314f7a2f5bffb5389133dd471216cc/bugsnag/BugsnagNotifier.m#L396
But, they are not found in current version (5.6.5) :cry:.


How to contribute to knex documentation?

Is there any way to contribute to editing knex.org documentation. It may be helpful to add some examples in schema examples like float-example.

Appreciate the eagerness to help! You can submit PR to https://github.com/knex/documentation

thanks 😁


Set method params to invokable symbol of object intersection's methods in BIR symbol enter

Purpose
$title.
Fixes #34280
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

We need to merge the changes from stage-slbeta6 for the test to pass. Can be merged after merging https://github.com/ballerina-platform/ballerina-lang/pull/34299.


Not a legal Ole Aut date

https://exceldatareader.codeplex.com/discussions/562088
 public static object ConvertFromOATime(double value)
    {
        if ((value >= 0.0) && (value < 60.0))
        {
            value++;
        }
        //if (date1904)
        //{
        //    Value += 1462.0;
        //}

      // added to fix out of range problem, sdh, 20140826
      if (value < 657435.0)
      {
        value = 657435.0;
      }
      else if (value > 2958465.99999999)
      {
        value = 2958465.99999999;
      }
        return DateTime.FromOADate(value);
    }

Another solution would be to make the xlsx reader consistent with the xls reader and support the convertOADate parameter. I don't think it is a good idea to silently throw data away and replace it with 0.
We could also delay the exception until GetValue and include the raw data in the exception to let the user of the reader decide what to do or be more explicit about it and let the user specify an error handler callback.

This would be better handled by the solution proposed in #215 instead of #79.


Allow highlighter to be swapped

I would prefer to use Prism with React code. It simply works better with JSX. It might be nice to make it possible to swap the highlighter.

Hm, yeah, good point, though do not have time for it. If you got time it would be great with a pull on it :-) Highlighter has been updated a lot though, so works really well now? It also supports other languages very well.


Missing config on install

Neat tool :)
I installed with pipx on Arch and got the following error:
FileNotFoundError: [Errno 2] No such file or directory:
'~/.config/.shell_genie/config.json'

Thank you! Did you get this error while running shell-genie init?


add: amazon S3

From the 2018.02.22 icons pack; don't know if you prefer the 2019 "flat" versions (check https://aws.amazon.com/architecture/icons/).

You can save a few bytes by removing newlines.

  This is per the advice of The Pull Request Hack.
Please use this privileged wisely.

Thank you a lot for the privilege! \o/
I'll push in distinct branches on your repo and ask for review for the first few... Do not hesitate to edit/rework my commit to show me what you would expect.

You can save a few bytes by removing newlines.
Hum, could you give me an example? I thought you wanted to keep newlines because "This makes viewing diffs easier"?


Interactive list

[ ] Add isCompleted method to check if a task is checked or not
[ ] Add clear method to delete all completed tasks in one click
[ ] Save to-do list in the locale storage

Thank you, Favour  😄


Make initialize optional

The first small PR to make initialize optional

merged the PR first in order to test by our nightly test

merged the PR first in order to test by our nightly test


Document that clients should avoid strict validation

OSDM providers/consumers should avoid strict validation to avoid messages being rejected based on differences between two minor versions within the same major version. This point should be added to the documentation as a nadditional design guideline

TODO (patch CG):

move the versioning from wiki to technical principals
add a description that the tolerant reader pattern must be implemented


tmux-friendly controls?

Hi Jakob! This thing looks cool, thanks!
I recognize that it might not be a common usecase for alen, but I would routinely use it to browse alignments in an ssh session through tmux on macOS. Taken together, it superseeds most of the control key sequences that seem to be hardcoded in alen 😞  specifically, Ctrl-Shift-arrows do not work and ctrl-J jumps to the next pane...
It would be helpful to me if the key bindings were remappable! Otherwise, an alternative control scheme involving less ctrls and shifts would also work.
Cheers,
Nikita

As a temporary solution, I did some fiddling with tmux and got the mentioned keys to work, I can probably live without some tmux hotkeys...
For the curious ones, I did set-option -g -w xterm-keys on to enable shift-arrows and unbind-key -T root C-j to unblock ctrl-j for jumps.


Benchmark

Detailed Description
It would be great to bench mark the model
Context
Always good to benchmark
Possible Implementation

a model could use the mean pv value, obviously this model will be bad, but it gives a bit of context to the numbers on the evaluation

Hi! Could I take this one? Is there any deadline? I would aim at doing it in the following weeks.

Thanks @felipewhitaker , there is no deadline, So really appreciate you taking this on

Hello, can anyone please guide to how to perform it in correct way
I'm thinking to perform evaluation using Mean Absolute Error to compare with train and valid data of PV values
Is this is a correct way, it would be great if you explain it once

@ombhojane, it is quite common to use Mean Absolute Error (MAE) for evaluating models, including in the weather research area. Another common metric is Continuous Ranked Probability Score (CRPS), which is a generalization of MAE to take scenarios into consideration (properscoring has an implementation of it).
Independent of the metric, what do you expect to be a correct way? When comparing models, it is important that both are compared by using a dataset that neither have used to learn (test dataset), and that the comparison is fair (it doesn't make much sense to compare two models that predict different things).

I think ideall it would be similar to this https://github.com/openclimatefix/Open-Source-Quartz-Solar-Forecast/blob/main/quartz_solar_forecast/forecast.py#L11. Does this answer you question?

Or perhaps something like this https://github.com/openclimatefix/Open-Source-Quartz-Solar-Forecast/blob/main/quartz_solar_forecast/forecasts/v1.py#L12
It would be good to be able to switch it into the evaulation script easier too, here - https://github.com/openclimatefix/Open-Source-Quartz-Solar-Forecast/blob/main/quartz_solar_forecast/eval/forecast.py#L19,

I think ideall it would be similar to this https://github.com/openclimatefix/Open-Source-Quartz-Solar-Forecast/blob/main/quartz_solar_forecast/forecast.py#L11. Does this answer you question?

It does help, thanks! I might've missed some details there. Moreover, is there a file containg how the current model was trained (which I believe is in psp)? It would be nice to be able to use the same rough steps.

After exploring psp, my next step is to use the dataset available in Hugging Face (linked in the first comment of #30) to make an historic average model. What interface should it support? The current model has some attributes (e.g. _config, _nwp_tolerance, _nwp_dropout): should every model include these?

Thanks for referencing a prerequisite and suggestions, makes much clarity.


Fix CO: Scraper crashing

Seen here
pipenv run python -m warn.cli co -l DEBUG
2022-11-28 12:07:40,073 - warn.runner - Scraping co
2022-11-28 12:07:40,073 - warn.utils - Requesting https://cdle.colorado.gov/employers/layoff-separations/layoff-warn-list
2022-11-28 12:07:40,238 - warn.utils - Response code: 403
2022-11-28 12:07:40,238 - warn.cache - Writing to cache data/warn-scraper/cache/co/main/source.html
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.9.15/x64/lib/python3.9/runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/opt/hostedtoolcache/Python/3.9.15/x64/lib/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/runner/.local/share/virtualenvs/warn-github-flow-R1xICqqL/lib/python3.9/site-packages/warn/cli.py", line 79, in <module>
    main()
  File "/home/runner/.local/share/virtualenvs/warn-github-flow-R1xICqqL/lib/python3.9/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/home/runner/.local/share/virtualenvs/warn-github-flow-R1xICqqL/lib/python3.9/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/home/runner/.local/share/virtualenvs/warn-github-flow-R1xICqqL/lib/python3.9/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/runner/.local/share/virtualenvs/warn-github-flow-R1xICqqL/lib/python3.9/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/home/runner/.local/share/virtualenvs/warn-github-flow-R1xICqqL/lib/python3.9/site-packages/warn/cli.py", line 75, in main
    runner.scrape(scrape)
  File "/home/runner/.local/share/virtualenvs/warn-github-flow-R1xICqqL/lib/python3.9/site-packages/warn/runner.py", line [52](https://github.com/biglocalnews/warn-github-flow/actions/runs/3564571489/jobs/5988691763#step:4:55), in scrape
    data_path = state_mod.scrape(self.data_dir, self.cache_dir)
  File "/home/runner/.local/share/virtualenvs/warn-github-flow-R1xICqqL/lib/python3.9/site-packages/warn/scrapers/co.py", line 43, in scrape
    current_link = str(soup.find("a", class_="btn btn-primary")).split('"')[3]
IndexError: list index out of range
make: *** [Makefile:[71](https://github.com/biglocalnews/warn-github-flow/actions/runs/3564571489/jobs/5988691763#step:4:74): scrape] Error 1
Error: Process completed with exit code 2.

I can't recreate this bug in the main branch. We'll keep an eye on it with future scrapes via the bot.


Vd/documentation

Add documentation + description folder utils + name project.toml

Codecov Report

Merging #315 (00f629a) into master (f2f50cb) will decrease coverage by 0.67%.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##           master     #315      +/-   ##
==========================================
- Coverage   82.91%   82.24%   -0.67%
==========================================
  Files          62       52      -10
  Lines        4383     4100     -283
==========================================
- Hits         3634     3372     -262
+ Misses        749      728      -21

Files
Coverage Δ

src/domain/continuous_domain.jl
50.00% <ø> (ø)

src/domain/custom_domain.jl
33.33% <ø> (ø)

src/domain/domain_list.jl
77.77% <ø> (-0.23%)
:arrow_down:

src/domain/general_domain.jl
60.42% <ø> (+0.03%)
:arrow_up:

src/domain/grid.jl
68.25% <ø> (-1.68%)
:arrow_down:

src/optim/abstraction/SCOTS_abstraction.jl
90.00% <ø> (-0.28%)
:arrow_down:

src/optim/abstraction/ellipsoids_abstraction.jl
96.87% <ø> (-0.25%)
:arrow_down:

src/optim/abstraction/hierarchical_abstraction.jl
92.76% <ø> (-0.47%)
:arrow_down:

src/optim/abstraction/lazy_abstraction.jl
93.13% <ø> (-1.64%)
:arrow_down:

...c/optim/abstraction/lazy_ellipsoids_abstraction.jl
79.78% <ø> (-3.89%)
:arrow_down:

... and 7 more

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

Where can I see the documentation that has been generated ?


reflectx changes

I wanted to open an issue to discuss some ongoing work by @pkieltyka on reflectx.

The goal is to make reflectx reveal a lot more useful information about a struct and its tags.
Currently in master, reflectx ignores struct tag options which are pretty common and handy. As well the fieldMap which is the type that stores the information about the parsed structs only maps the name to a traversal (which is the index to a specific field in a nested structure).
My branch expands on the info returned about fields in a struct without any interface changes to sqlx, just a very minor change.
I've changed type fieldMap map[string][]int in favor of fieldInfo that provides:

Index - this is the same as the path traversal in fieldMap)
Path - the resolvable named path of a nested field map
Name - the name from the parsed struct tag, ie. db:"col,opt", name="col"
Field - the reflect.StructField for reference
Zero - the zero value of the Field, useful for being able to easily get default values
Options - any key or key/value options in a struct tag
Embedded - a boolean to inform if a nested field is an embedded struct

All of this stuff mostly comes down to how Path's are resolved and composed. I've included a lot of examples in the tests, but here's a few...:
Inline structs
type Foo struct {
  A int
}

type Bar struct {
  Foo // `db:""` is implied for an embedded struct
  B   int
  C   int `db:"-"`
}

type Baz struct {
  A   int
  Bar `db:"Bar"`
}
m := NewMapperFunc("db", func(s string) string { return s })
z := Baz{}
z.A = 1
z.B = 2
z.C = 4
z.Bar.Foo.A = 3

The parsed mapping looks like:
Index: [0]       Path: "A"          Name: "A"
Index: [1]       Path: "Bar"        Name: "Bar"   Embedded: true
Index: [1 0]     Path: "Bar.Foo"    Name: "Foo"   Embedded: true
Index: [1 1]     Path: "Bar.B"      Name: "B"
Index: [1 0 0]   Path: "Bar.A"      Name: "A"

notice the "Bar.A" path which is actually a part of the embedded Foo struct. This allows you to have flat column structures (ie. stored in a table), but allows you to split up a table into more runtime structs that can easily be composed to either some namespace, or just inline.
Inline structs are the default for anonymously embedded structs, unless db:"something" is provided at which point, it will be namespaced by "something".
Compose paths (aka columns) with struct tags
type Employee struct {
  Name string `db:"name"`
  Id   int `db:"id"`
}
type Boss Employee
type person struct {
  Employee `db:"employee"`
  Boss     `db:"boss"`
}
// person columns: (employee.name employee.id boss.name boss.id)

another cool side effect is that "dominant" struct tags are also supported. For example, just like in encoding/json someone can override the marshalled result based on the latest tag name, the same thing works in reflectx.
ie. like the ideas in here: http://attilaolah.eu/2014/09/10/json-and-struct-composition-in-go/

I was using the reflectx (ptrs branch) in another project, and found that I needed to export reflectx.Field. See: https://github.com/pkieltyka/sqlx/commit/ccbeb6c11d1b9bc3ef7a021231496ac888760f0a
as well, I feel the Field and Fields structs should be renamed to FieldInfo and StructMap

Is it possible to merge #429 to permit to use embedded struct ? We need that and it worked on old sqlx version. I don't want to use my fork as import. For the moment I use the vendoring to use my version but it's not really good. And we want to move to go module.


PLATUI-BAU: Update scalafmt checks

Update scalafmt checks

Change pulled down and tested locally. All working as expected.


"The command is disabled and cannot be executed"

In our Electron app, this error message was logged a couple of times for users on Mac OS X 10.10.5, 10.11.6, and 10.12. It seems the users were able to download the update successfully, but then the app failed to update. It's hard to tell exactly what caused this issue just by looking at the logs and I have not been able to reproduce it myself, but a quick Google search returns results related to ReactiveCocoa which Squirrel.Mac depends on.
Has anybody run into this issue at all?
electron/electron#7792

Most likely this is caused by trying to install an update that has been aborted, which is most likely caused by an error earlier on.

Thank you for the reply. We currently do not provide a way to abort an update that's already begun. So if an update was aborted, it's probably due to network issues or some other external factors outside of the app. Is the error message "The command is disabled and cannot be executed" coming from Squirrel.Mac? Is there any more information around this error or in what context it occurs so we can respond in a more actionable way?

Is the error message "The command is disabled and cannot be executed" coming from Squirrel.Mac?

ReactiveCocoa is the one responsible for that specific message, but the command it's referencing is from Squirrel.Mac.

Is there any more information around this error or in what context it occurs so we can respond in a more actionable way?

I'd hope there'd be more information in the user's logs, but I can't say for certain.

Closing since it doesn't sound like a Squirrel.Mac problem.


problem in latest version of scanpy (bbknn)

I have checked that this issue has not already been reported.
I have confirmed this bug exists on the latest version of scanpy.
[ ] (optional) I have confirmed this bug exists on the master branch of scanpy.

Note: Please read this guide detailing how to provide the necessary information for us to reproduce your bug.
Minimal code sample (that we can copy&paste without having any data)
import scanpy as sc
import bbknn
sc.logging.print_header()
adata = sc.read('pancreas.h5ad', backup_url='https://www.dropbox.com/s/qj1jlm9w10wmt0u/pancreas.h5ad?dl=1')
sc.external.pp.bbknn(adata, batch_key='batch')  #error2

 tuple' object has no attribute 'tocsr'

Versions
<bbknn : 1.5.0>
scanpy:1.7.0
umap:  0.1.1
umap-learn:  0.5.1
[Paste the output of scanpy. logging.print_versions() leaving a blank line after the details tag]
</scanpy==1.4.6 anndata==0.7.4 umap==0.5.1 numpy==1.20.3 scipy==1.6.3 pandas==1.2.4 scikit-learn==0.22.2.post1 statsmodels==0.11.1 python-igraph==0.8.2>

Any idea why scanpy. logging.print_versions() is reporting a different version that you've reported above it?
Could be that there's an issue with this environment. Can you replicate the issue in a fresh conda environment?

I'm assuming the :+1: means the issue is solved, please re-open if this isn't the case.


Addressed code review comments of PR "Updating references to NuGet packages"

Thank you for taking the time to make these fixes!


#36 convert How to participate in the Global Day of Coderetreat 2016

As part of #36, I have updated this html file, there are two problems:

Using markdown we lose _target parameter on links
Distances between paragraph are a shorter.

I have found a way to keep _target attributes, I will update PR on a while


[BUG] PNG color corruption after editing.

Describe the bug
After modifying one of the background PNGs, GB Studio refreshed the updated image but with messed up colors.  (Color bars appearing randomly in the background image.)
To Reproduce
Steps to reproduce the behavior:

Create new project based on Sample Project template.
Scroll down so that Player's House scene is in view.  Zoom in if necessary.
On a Mac, open up Pixelmator Pro (or your favourite image editor but that's what I used.  I also tried with Preview.) and open up room.png background image.
Widen canvas to 200 pixels.  Fill in the empty spaces with one of the colors in the background image and save.
Return to GB Studio and notice how the Player's House scene now shows the new widened image, but several random blocks in various colors have appeared.
Run the game, and notice that the scene displays using these messed up colors.
Return to GB Studio and click on the scene.
Change the background from room to some other image (cave, for example) and then change it back to room.
Notice how the scene refreshes and looks OK now.
Run the game again and notice that the scene looks OK here too.

Expected behavior
After modifying the background image, the scene in GB Studio should update showing the correct colors.
Screenshots

Platform (please complete the following information):

OS: macOS 10.15.6
App version: 2.0.0-beta1

Additional context
Add any other context about the problem here.

This can be closed :)


Implementation tutorial

Fixes #147
And this starts the work on documenting the implementation of each lossless step (with associated figures and demonstration with python code).

Self merging since this mainly aesthetic (tutorial) and fixes the CI's.


VM controller - move unit tests to fake client (VirtualMachineInstanceInterface)

What this PR does
Use fake client instead of mocking in order to simulate real behavior.

Fixes #
Why we need it and why it was done in this way
Lower the bar for unit tests, eliminate false positives. The fake is standard for testing.
Special notes for your reviewer
Based on https://github.com/kubevirt/kubevirt/pull/11726

Release note

NONE

/test pull-kubevirt-unit-test

/test pull-kubevirt-e2e-k8s-1.28-sig-compute
/test pull-kubevirt-e2e-k8s-1.29-sig-storage


Roll engine 21ae92651a38..8213d41270e5 (4 commits)

git@github.com:flutter/engine.git/compare/21ae92651a38...8213d41270e5
git log 21ae92651a38..8213d41270e5 --no-merges --oneline
2019-08-19<EMAIL_ADDRESS>Update tflite_native and language_model revisions to match the Dart SDK (flutter/engine#11230)
2019-08-19<EMAIL_ADDRESS>Roll src/third_party/skia f8221786d088..8566dda51b42 (4 commits) (flutter/engine#11220)
2019-08-19<EMAIL_ADDRESS>Update metal layer drawable size on relayout. (flutter/engine#11224)
2019-08-19<EMAIL_ADDRESS>Make firebase testlab always pass (flutter/engine#11226)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (stuartmorgan@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/6526583949492224 , https://cirrus-ci.com/task/5946041810026496

Commit queue failed; closing this roll.


Upgrade material-ui dependencies

Upgrade @material-ui/core to 3.9.3
Upgrade @material-ui/icons to 3.0.2

I haven't used material-ui probably since this example was written, but it looks like material UI is around v4.5 now. Just curious why only upgrade to v3.x...are there pretty big breaking changes between versions 3 and 4 but from 1 to 3 aren't too bad?

Hi @redbmk! I'm toying around with upgrading the dependencies of this example to be recent versions, and I've run into some challenges.
You're correct that material-ui is up to 4.y. The issue is that the 4.y series is dependent upon newer versions of react than is pinned in the package.json file and I haven't resolved those upgrade pains, yet.

@redbmk I've updated most of the dependencies and have it working locally.

Looks great, thanks!


PHP Restructure

This issue serves as an outine of the upcoming PHP restructure (per RFC 001)
Steps:

@sophiewigmore Feel free to close if this is no longer needed.

Closed out since this issue was opened in April and I'd like to have an accurate timeline of our progress on this, starting with the full issue creation.


434 ic popover menu component

Summary of the changes
Popover component with menu buttons, menu group, and relevant tests for all.
Related issue
#433 #434 #258
Checklist

[x] I have added relevant unit and visual regression tests.
[x] I have manually accessibility tested any changes, if relevant.
[x] I have ensured any changes match the Figma component library.

Just waiting for disabled focus states work, but ready to review other than that. Because it's waiting for this work, it will appear for now that no item is focused at first as it is still looking to focus the disabled button.

Just waiting for disabled focus states work, but ready to review other than that. Because it's waiting for this work, it will appear for now that no item is focused at first as it is still looking to focus the disabled button.

This work is now in the PR

Just found a very weird bug. On the sub pages (not the main page), every time you click a menu button and close the menu, next time its opened it takes one extra button press to navigate from the top item using a keyboard. Doesn't happen when the menu is closed via blur, and it is reproducible on multiple browsers/OS'. It affects both directions of travel (up and down) from the top menu item.
Screencast from 28-03-23 12:07:53.webm
The screencast doesn't really capture it well, but its definetly happening everytime I refresh.


Reuse composer's script for class-leak rather than duplicate command line on GH action

This is a follow up for https://github.com/rectorphp/rector-phpunit/pull/320 and related to https://github.com/rectorphp/rector-phpunit/pull/319#issuecomment-2035137884
Instead of duplicating command line on code_analysis.yaml action file, we can just reuse composer class-leak from composer.json.
Each time composer class-leak is updated, it will be automatically used on GH Action, so no need to update duplicated command twice.
Without it, we will have an issue when we start addign --skip-type for vendor/bin/class-leak in code_analysis.yaml. The same --skip-type would be added to 2 places without this PR.

Makes perfect sense :+1:


Clean up: remove unused codes

org.apache.skywalking.apm.network.trace.component.ComponentsDefine#components is now unused and is replaced by the file component-libraries.yml

/run e2e


Bring back "zarr" into repr

In a mixed context it's not clear that a zarr array is from zarr, put "zarr" back into array and group reprs somehow.

Also suggest removing the compression information (nbytes_stored, number of chunks initialized) as these can take some time to retrieve. Suggest moving these to a method or property, e.g., storage_info or something like that.

Yeah, it would also be nice if repr returned something that was relatively static between runs.

How do you mean exactly? Eg, avoiding dict ordering randomness? And/or
something else?
On Mon, 6 Mar 2017 at 20:11, jakirkham<EMAIL_ADDRESS>wrote:

Yeah, it would also be nice if repr returned something that was
relatively static between runs.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/alimanfoo/zarr/issues/83#issuecomment-284517215, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAq8QjAUyIy_roiwklI01irH_RBHEABAks5rjGhvgaJpZM4KXFyK
.

--
Alistair Miles
Head of Epidemiological Informatics
Centre for Genomics and Global Health http://cggh.org
The Wellcome Trust Centre for Human Genetics
Roosevelt Drive
Oxford
OX3 7BN
United Kingdom
Email<EMAIL_ADDRESS>Web: http://purl.org/net/aliman
Twitter: https://twitter.com/alimanfoo
Tel: +44 (0)1865 287721

I suppose that would be included too. So using an OrderedDict would help.
Though was more referring to details like the compression information that seem to vary slightly even on the same data. Things like name, shape, dtype, and chunk shape should be fine.

Yes the compression information has been a bit of a bane, small changes can
make the doctests on docstring examples and the tutorial fail, which is a
pain to keep up to date. In the numcodecs PR I ended up putting a load of
ellipses in to try and reduce maintenance overhead.
It would be nice to simplify the repr. It has been useful for me to see
compression info easily to hand, but maybe we could think of an alternate
way to access that information that is almost as convenient as repr.
On Tuesday, March 7, 2017, jakirkham<EMAIL_ADDRESS>wrote:

I suppose that would be included too. So using an OrderedDict would help.
Though was more referring to details like the compression information that
seem to vary slightly even on the same data. Things like name, shape,
dtype, and chunk shape should be fine.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/alimanfoo/zarr/issues/83#issuecomment-284592509, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAq8Qql719xO5zoUXyqa07z6r3Ji9U7Lks5rjLQIgaJpZM4KXFyK
.

--
Alistair Miles
Head of Epidemiological Informatics
Centre for Genomics and Global Health http://cggh.org
The Wellcome Trust Centre for Human Genetics
Roosevelt Drive
Oxford
OX3 7BN
United Kingdom
Email<EMAIL_ADDRESS>Web: http://purl.org/net/aliman
Twitter: https://twitter.com/alimanfoo
Tel: +44 (0)1865 287721

Was also thinking of doctests when thinking about repr. Totally agree with having a different method provide this.
As a side note, the dictionary changes in Python 3.6 are well explained in this message.


Added DB to backend

Added Mysql database to backend.
(Make sure to change connection information in config.json)

In this pr, I used Flask-SQLAlchemy Doc a lot
Also this video was so helpful.

(Also I should add that Github CI Test is going to be failed because we need mysql server to pass our integration tests; I'm aware of that and working on it!)

Dear @hramezani, could you please take a look at this PR? what is the best way to write unit tests and also I will be grateful if you can review the whole PR as it is the fundamental of working with DB in this program :)

@hramezani I deeply appreciate your review and your time, it was really helpful, thanks!

Thank for your great reviews, I made some small changes based on them; I really appreciate your help.

Dear @farbodahm
I was testing this pull request, I discovered some error while I was tring to get the links:
<IP_ADDRESS> - - [15/Sep/2020 06:44:10] "GET /user/links HTTP/1.1" 500 -
Traceback (most recent call last):
  File "/home/farid/lab/Restfulness-core-api/venv/lib/python3.6/site-packages/flask/app.py", line 2464, in __call__
    return self.wsgi_app(environ, start_response)
  File "/home/farid/lab/Restfulness-core-api/venv/lib/python3.6/site-packages/flask/app.py", line 2450, in wsgi_app
    response = self.handle_exception(e)
  File "/home/farid/lab/Restfulness-core-api/venv/lib/python3.6/site-packages/flask_restful/__init__.py", line 272, in error_router
    return original_handler(e)
  File "/home/farid/lab/Restfulness-core-api/venv/lib/python3.6/site-packages/flask/app.py", line 1867, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/home/farid/lab/Restfulness-core-api/venv/lib/python3.6/site-packages/flask/_compat.py", line 38, in reraise
    raise value.with_traceback(tb)
  File "/home/farid/lab/Restfulness-core-api/venv/lib/python3.6/site-packages/flask/app.py", line 2447, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/farid/lab/Restfulness-core-api/venv/lib/python3.6/site-packages/flask/app.py", line 1952, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/farid/lab/Restfulness-core-api/venv/lib/python3.6/site-packages/flask_restful/__init__.py", line 272, in error_router
    return original_handler(e)
  File "/home/farid/lab/Restfulness-core-api/venv/lib/python3.6/site-packages/flask/app.py", line 1821, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/home/farid/lab/Restfulness-core-api/venv/lib/python3.6/site-packages/flask/_compat.py", line 38, in reraise
    raise value.with_traceback(tb)
  File "/home/farid/lab/Restfulness-core-api/venv/lib/python3.6/site-packages/flask/app.py", line 1950, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/farid/lab/Restfulness-core-api/venv/lib/python3.6/site-packages/flask/app.py", line 1936, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/home/farid/lab/Restfulness-core-api/venv/lib/python3.6/site-packages/flask_restful/__init__.py", line 468, in wrapper
    resp = resource(*args, **kwargs)
  File "/home/farid/lab/Restfulness-core-api/venv/lib/python3.6/site-packages/flask/views.py", line 89, in view
    return self.dispatch_request(*args, **kwargs)
  File "/home/farid/lab/Restfulness-core-api/venv/lib/python3.6/site-packages/flask_restful/__init__.py", line 583, in dispatch_request
    resp = meth(*args, **kwargs)
  File "/home/farid/lab/Restfulness-core-api/venv/lib/python3.6/site-packages/flask_jwt_extended/view_decorators.py", line 108, in wrapper
    return fn(*args, **kwargs)
  File "/home/farid/lab/Restfulness-core-api/venv/lib/python3.6/site-packages/flasgger/utils.py", line 273, in wrapper
    return function(*args, **kwargs)
  File "/home/farid/lab/Restfulness-core-api/resources/Links.py", line 24, in get
    bookmarked_links_names.append(link.get_address_name())
AttributeError: 'str' object has no attribute 'get_address_name'

Also when you are adding a link it should return the added link id, not the whole structure which we already sent!
At the end the reason you didn't see the first error is you didn't write proper tests for the important parts, please write more tests, using a test coverage library could be helpful like this one:
https://coverage.readthedocs.io/en/coverage-5.3/

Dear @farbodahm
I was testing this PR in action, I noticed your are returning the whole data structure which we already sent in the post -> user/links, it should return only the added new link's id
And in the get -> user/links you are returning only the links id! it should return the whole list
Also it should support user/links/[id] to get a specific link information.
Recheck your knowledge about RESTful design by reading articles like this:
https://stackoverflow.com/questions/671118/what-exactly-is-restful-programming
Also please use a test coverage library in order to be confident that we are writing enough tests:
https://coverage.readthedocs.io/en/coverage-5.3/

Dear @pesarkhobeee ,
I just added new commits to PR which I fixed the problems you mentioned before and added some new test;
Also I separated  Links related routes from User related routes to have better documentation.

@farbodahm well done :+1:


Remove build started message

This was deprecated in the last release and is now killable.

cc @Pawka @ChaitanyaPramod


Merge SyncCounter/AsyncCounter and simplify CounterPerf

This is getting too complicated... Things to change:

Merge Sync/AsyncCounter -> Counter, which has both sync and async methods, so we don't need to diff between SyncCounter and AsyncCounter (and their impls, too)
Make only the Updater configurable in CounterPerf. An Updater gets a Counter and can then choose what to do (use sync or async)
CounterPerf is configured with an Updater impl (the former becomes an interface)

Merge Sync/AsyncCounter -> Counter, which has both sync and async methods, so we don't need to diff between SyncCounter and AsyncCounter (and their impls, too)

Note that SyncCounter and AsyncCounter have methods to convert between interfaces. About the "implementations", there is only a single implementation.

Make only the Updater configurable in CounterPerf. An Updater gets a Counter and can then choose what to do (use sync or async)

What is the difference between the "new" Updater and the current interface https://github.com/belaban/jgroups-raft/blob/a9f99f07173b0ad7658e7bd06eb692147c317e73/src/org/jgroups/perf/CounterBenchmark.java ?

I created common base interface Counter, with methods getName() and sync() and async(). Both SyncCounter and AsyncCounter extend it.

I haven't tackled that one yet, but the idea is that Updater is given a Counter (and, of course, a LongSupplier) and needs to implement only a single method update().

I haven't tackled that one yet, but the idea is that Updater is given a Counter (and, of course, a LongSupplier) and needs to implement only a single method update().

It is a bit restrictive IMO. If you want a node to generate N requests concurrently, you need 1 thread per Updater with SyncCounter, but not for the AsyncCounter.
Well, you don't even need to create any thread at all for AsyncCounter.
Francesco wants to implement a fixed rate perf-test and a single Updater interface makes it more complicated (since it would have to change code in  CounterTest to implement the new logic)

Good point! I'll think about this...


Add support for GROUP BY of text arrays

CrateDB version
4.6.4
CrateDB setup information
local 1 node
Steps to Reproduce
CREATE TABLE dist (array_text ARRAY(TEXT)) CLUSTERED INTO 1 SHARDS;

INSERT INTO dist VALUES (['a','b']),(['a']),(['a']);

SELECT UNNEST(array_text)
FROM (
SELECT DISTINCT array_text
FROM dist ) sub

Expected Result
a
b
a
Actual Result
Error!

UnsupportedFeatureException[Cannot GROUP BY type: text_array]

Hi, thanks for reporting it.
Arrays should be comparable - do they have to be comparable up to permutation precision, i.e
[a,b,c] == [c,b,a] or not?

At PostgreSQL, the result set differs from the described expected results:
postgres=# CREATE TABLE dist (array_text TEXT[]);
CREATE TABLE
postgres=# INSERT INTO dist VALUES (array['a','b']),(array['a']),(array['a']);
INSERT 0 3
postgres=# SELECT UNNEST(array_text)
FROM (
SELECT DISTINCT array_text
FROM dist ) sub;
 unnest
--------
 a
 a
 b
(3 rows)

postgres=# SELECT DISTINCT array_text
postgres-# FROM dist;
 array_text
------------
 {a}
 {a,b}
(2 rows)

Closed in https://github.com/crate/crate/pull/12755.
Arrays which differ only by elements permutation are considering as different (aligned with PostgreSQL 14).


labels/cidr: Fix slice preallocation size

The number of CIDR labels returned from GetCIDRLabels are equal to the sum of:

ones+1 labels coming from the prefixes generated in the loop
an additional one for the reserved "world" label

To improve performance, we should then allocate a slice of size ones+2.
benchstat analysis confirms this reporting 1 allocation less for each operation, except for the "<IP_ADDRESS>/0" corner case.
name                             old time/op    new time/op    delta
GetCIDRLabels/<IP_ADDRESS>/0-8           306ns ±36%     299ns ±32%     ~     (p=0.251 n=20+20)
GetCIDRLabels/<IP_ADDRESS>/16-8       4.42µs ± 5%    4.17µs ± 3%   -5.65%  (p=0.000 n=19+19)
GetCIDRLabels/<IP_ADDRESS>/32-8       8.43µs ± 5%    8.02µs ± 5%   -4.88%  (p=0.000 n=20+20)
GetCIDRLabels/<IP_ADDRESS>/24-8       6.79µs ± 5%    6.49µs ± 5%   -4.41%  (p=0.000 n=20+20)
GetCIDRLabels/<IP_ADDRESS>/24-8       6.78µs ± 4%    6.46µs ± 4%   -4.68%  (p=0.000 n=19+20)
GetCIDRLabels/::/0-8                329ns ± 2%     338ns ± 3%   +2.54%  (p=0.000 n=19+20)
GetCIDRLabels/fdff::ff/128-8       53.6µs ± 4%    52.1µs ± 6%   -2.66%  (p=0.000 n=19+17)
GetCIDRLabels/f00d:42::ff/128-8    56.8µs ± 4%    55.1µs ± 2%   -3.04%  (p=0.000 n=20+18)
GetCIDRLabels/f00d:42::ff/96-8     42.4µs ± 3%    40.8µs ± 6%   -3.82%  (p=0.000 n=20+20)

name                             old alloc/op   new alloc/op   delta
GetCIDRLabels/<IP_ADDRESS>/0-8            640B ± 0%      656B ± 0%   +2.50%  (p=0.000 n=20+20)
GetCIDRLabels/<IP_ADDRESS>/16-8       3.75kB ± 0%    3.17kB ± 0%  -15.37%  (p=0.000 n=18+20)
GetCIDRLabels/<IP_ADDRESS>/32-8       8.03kB ± 0%    6.88kB ± 0%  -14.34%  (p=0.000 n=16+20)
GetCIDRLabels/<IP_ADDRESS>/24-8       7.21kB ± 0%    6.32kB ± 0%  -12.42%  (p=0.000 n=20+19)
GetCIDRLabels/<IP_ADDRESS>/24-8       7.21kB ± 0%    6.32kB ± 0%  -12.42%  (p=0.000 n=19+20)
GetCIDRLabels/::/0-8                 640B ± 0%      656B ± 0%   +2.50%  (p=0.000 n=20+20)
GetCIDRLabels/fdff::ff/128-8       30.8kB ± 0%    25.9kB ± 0%  -15.80%  (p=0.000 n=20+19)
GetCIDRLabels/f00d:42::ff/128-8    33.7kB ± 0%    28.8kB ± 0%  -14.45%  (p=0.000 n=19+20)
GetCIDRLabels/f00d:42::ff/96-8     28.3kB ± 0%    25.1kB ± 0%  -11.31%  (p=0.000 n=20+20)

name                             old allocs/op  new allocs/op  delta
GetCIDRLabels/<IP_ADDRESS>/0-8            3.00 ± 0%      3.00 ± 0%     ~     (all equal)
GetCIDRLabels/<IP_ADDRESS>/16-8         38.0 ± 0%      37.0 ± 0%   -2.63%  (p=0.000 n=20+20)
GetCIDRLabels/<IP_ADDRESS>/32-8         70.0 ± 0%      69.0 ± 0%   -1.43%  (p=0.000 n=20+20)
GetCIDRLabels/<IP_ADDRESS>/24-8         54.0 ± 0%      53.0 ± 0%   -1.85%  (p=0.000 n=20+20)
GetCIDRLabels/<IP_ADDRESS>/24-8         54.0 ± 0%      53.0 ± 0%   -1.85%  (p=0.000 n=20+20)
GetCIDRLabels/::/0-8                 3.00 ± 0%      3.00 ± 0%     ~     (all equal)
GetCIDRLabels/fdff::ff/128-8          450 ± 0%       449 ± 0%   -0.22%  (p=0.000 n=20+20)
GetCIDRLabels/f00d:42::ff/128-8       450 ± 0%       449 ± 0%   -0.22%  (p=0.000 n=20+20)
GetCIDRLabels/f00d:42::ff/96-8        296 ± 0%       295 ± 0%   -0.34%  (p=0.000 n=20+20)

/test


AttributeError: 'module' object has no attribute 'ERROR'

root@localhost:/usr/local/lib/python2.7/dist-packages# git clone git://github.com/nficano/pytube.git
root@localhost:/usr/local/lib/python2.7/dist-packages/pytube# git pull origin pull/435/head
remote: Enumerating objects: 8, done.
remote: Counting objects: 100% (8/8), done.
remote: Compressing objects: 100% (2/2), done.
remote: Total 8 (delta 6), reused 8 (delta 6), pack-reused 0
Unpacking objects: 100% (8/8), done.
From git://github.com/nficano/pytube

branch            refs/pull/435/head -> FETCH_HEAD
Updating 7e0cd4e..3fc4a38
Fast-forward
pytube/main.py | 23 ++++++++++++++++++++---
pytube/streams.py  |  2 +-
2 files changed, 21 insertions(+), 4 deletions(-)
root@localhost:/usr/local/lib/python2.7/dist-packages/pytube# pip install .
DEPRECATION: Python 2.7 will reach the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 won't be maintained after that date. A future versionof pip will drop support for Python 2.7. More details about Python 2 support in pip, can be found at https://pip.pypa.io/en/latest/development/release-process/#python-2-support
Processing /usr/local/lib/python2.7/dist-packages/pytube
Building wheels for collected packages: pytube
Building wheel for pytube (setup.py) ... done
Created wheel for pytube: filename=pytube-9.5.1-cp27-none-any.whl size=34765 sha256=53d74d4b7d8e9c09e616e408ceda82321fb3246b661e2bfd885453a4fc23933b
Stored in directory: /tmp/pip-ephem-wheel-cache-9UrWWz/wheels/cc/e1/72/7955450f4d60db46c43ba201668900cbb331319412552bd85b
Successfully built pytube
Installing collected packages: pytube
Successfully installed pytube-9.5.1
root@localhost:/usr/local/lib/python2.7/dist-packages/pytube#
root@localhost:/usr/local/lib/python2.7/dist-packages/pytube# python
Python 2.7.12 (default, Nov 12 2018, 14:36:49)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.

from pytube import YouTube
Traceback (most recent call last):
File "", line 1, in
File "pytube/init.py", line 13, in
from pytube.logging import create_logger
File "pytube/logging.py", line 5, in
import logging
File "logging.py", line 8, in
def create_logger(level=logging.ERROR):
AttributeError: 'module' object has no attribute 'ERROR'

when i git clone pytube.git from origin pull/435/head
I can't use pytube module.
if i use  pytube.git from master,then be https://github.com/nficano/pytube/issues/434

While installing any module getting AttributeError ?

All,
Believe this is fixed in most recent version of Pytube
Let me know if this still doesnt work -- thanks


Recreating tables with code page fails on other computer

ℹ Computer information

FoxBin2PRG version: 1.19.78

📝 Provide detailed reproduction steps (if any)

create a form
open the form as a table and issue CPDBF()
create some text version of this form
start VFP with a different CODEPAGE
recreate the form
open the form as a table and issue CPDBF()
create some text version of this form

✔ Expected result
Original codepage should be recreated
❌ Actual result
Codepage should is of the VFP where the recreation occurs
The CodePage needs to be stored in the *.*2 files and used creating the tables.
This is a problem out of #96.
Normally this should all be no problem since the VFP simply uses the systems setting. If you think this issue should be addressed give a reaction, else it will remain as it is.

Fixed version is available at https://github.com/lscheffler/foxbin2prg, no Thor update provided.


Circular dependencies with carbon components charts

I get these errors when I try to use charts components with svelte :
(!) `this` has been rewritten to `undefined`
https://rollupjs.org/guide/en/#error-this-is-undefined
<EMAIL_ADDRESS>1: var __spreadArrays = (this && this.__spreadArrays) || function () {
                        ^
2:     for (var s = 0, i = 0, il = arguments.length; i < il; i++) s += arguments[i].length;
3:     for (var r = Array(s), k = 0, i = 0; i < il; i++)
...and 1 other occurrence
<EMAIL_ADDRESS>1: var __extends = (this && this.__extends) || (function () {
                   ^
2:     var extendStatics = function (d, b) {
3:         extendStatics = Object.setPrototypeOf ||
...and 1 other occurrence
<EMAIL_ADDRESS>1: var __extends = (this && this.__extends) || (function () {
                   ^
2:     var extendStatics = function (d, b) {
3:         extendStatics = Object.setPrototypeOf ||
...and 1 other occurrence

...and 99 other files
(!) Circular dependencies
node_modules/d3-selection/src/selection/index.js -> node_modules/d3-selection/src/selection/select.js -> node_modules/d3-selection/src/selection/index.js
node_modules/d3-selection/src/selection/index.js -> node_modules/d3-selection/src/selection/selectAll.js -> node_modules/d3-selection/src/selection/index.js
node_modules/d3-selection/src/selection/index.js -> node_modules/d3-selection/src/selection/filter.js -> node_modules/d3-selection/src/selection/index.js
...and 12 more

Here is the code I am using in my App.svelte file :
<script>
	import { BarChartSimple } from "@carbon/charts-svelte";
	import<EMAIL_ADDRESS>	import "carbon-components/css/carbon-components.min.css";
 </script>

  <BarChartSimple
	data={[
	  { group: "Qty", value: 65000 },
	  { group: "More", value: 29123 },
	  { group: "Sold", value: 35213 },
	  { group: "Restocking", value: 51213 },
	  { group: "Misc", value: 16932 },
	]}
	options={{
	  title: "Simple bar (discrete)",
	  height: "400px",
	  axes: {
		left: { mapsTo: "value" },
		bottom: { mapsTo: "group", scaleType: "labels" },
	  },
	}}
  />

I have run these commands to create my project :
npx degit sveltejs/template test_project3
cd test_project3
npm install
npm i -D carbon-components-svelte
npm install -D @carbon/charts-svelte d3

What does your configuration file look like? I'm assuming your using Rollup?

Here is my rollup.config.js file
import svelte from 'rollup-plugin-svelte';
import commonjs from '@rollup/plugin-commonjs';
import resolve from '@rollup/plugin-node-resolve';
import livereload from 'rollup-plugin-livereload';
import { terser } from 'rollup-plugin-terser';
import css from 'rollup-plugin-css-only';

const production = !process.env.ROLLUP_WATCH;

function serve() {
	let server;

	function toExit() {
		if (server) server.kill(0);
	}

	return {
		writeBundle() {
			if (server) return;
			server = require('child_process').spawn('npm', ['run', 'start', '--', '--dev'], {
				stdio: ['ignore', 'inherit', 'inherit'],
				shell: true
			});

			process.on('SIGTERM', toExit);
			process.on('exit', toExit);
		}
	};
}

export default {
	input: 'src/main.js',
	output: {
		sourcemap: true,
		format: 'iife',
		name: 'app',
		file: 'public/build/bundle.js'
	},
	plugins: [
		svelte({
			compilerOptions: {
				// enable run-time checks when not in production
				dev: !production
			}
		}),
		// we'll extract any component CSS out into
		// a separate file - better for performance
		css({ output: 'bundle.css' }),

		// If you have external dependencies installed from
		// npm, you'll most likely need these plugins. In
		// some cases you'll need additional configuration -
		// consult the documentation for details:
		// https://github.com/rollup/plugins/tree/master/packages/commonjs
		resolve({
			browser: true,
			dedupe: ['svelte']
		}),
		commonjs(),

		// In dev mode, call `npm run start` once
		// the bundle has been generated
		!production && serve(),

		// Watch the `public` directory and refresh the
		// browser on changes when not in production
		!production && livereload('public'),

		// If we're building for production (npm run build
		// instead of npm run dev), minify
		production && terser()
	],
	watch: {
		clearScreen: false
	}
};

Try setting inlineDynamicImports: true in your Rollup config.
Refer to this example: https://github.com/metonym/carbon-charts-svelte-examples/tree/master/rollup

I have set inlineDynamicImports: true and I'm still getting the same error message.

Could you provide a repro?

What is a repro ?

@gwendal5 a minimal reproduction of the error to attempt to further isolate the issue and assist debugging. E.g., a GitHub repo with your Rollup set-up

I have made a repro, here is the link : https://github.com/gwendal5/svelte_mix_test

@gwendal5
The actual error in the browser console is "process is not defined."
You'll need to install @rollup/plugin-replace to statically replace process as a string. Instructions here.
+ import replace from '@rollup/plugin-replace';

+ replace({
+  preventAssignment: true,
+  'process.env.NODE_ENV': JSON.stringify('production')
+ }),

It solved it for me, thanks @metonym.
I still have a circular dependencies warning in the terminal, is it a problem ?

@gwendal5 You can safely omit those warnings using the onwarn method:
Ignore circular dependency errors from d3- packages
onwarn: (warning) => {
  if (warning.code === 'CIRCULAR_DEPENDENCY' && /^node_modules\/d3-/.test(warning.importer)) return;
}

Ignore circular dependency errors from d3- packages
onwarn: (warning) => {
  if (warning.code === 'CIRCULAR_DEPENDENCY' && /^node_modules\/d3-/.test(warning.importer)) return;
}

Full eample: https://github.com/metonym/carbon-charts-svelte-examples/blob/fb4f67e00db4bcb67b3bd8515f4398b9a5319745/rollup/rollup.config.js#L26

Closing this issue. For further questions/issues, please refer to https://github.com/carbon-design-system/carbon-charts


Adding GitHub to the list of registry types

Adding GitHub GHCR to the list of registry types. Documentation updated accordingly

@vale21 : thank you, could you please format your code, rebase and sign your commit (https://github.com/goharbor/terraform-provider-harbor/pull/236/checks?check_run_id=7632928252)


GeCraft auto-height for cards

Describe what your pull request addresses

[ ]

Test Instructions

[ ]

TODO @ancientTexts-net

Styleguidist example looks good.  Will try with GatewayEdit for compatibility.


Catch malformed etcd_authority variables and raise a datastore error

Fixes #85

LGTM

We should have a test for this.

LGTM

As I said in a comment, I think this should be using regex to do more comprehensive checking on the provided value.

Maybe. We do, however, accept both hostnames and IPs. Therefore, our regex statement would  effectively be searching for two sets of values split by a colon, which is the same as what this code does.

There could be a utils validate_etcd_authority function:
Some regex check like ((\d{1,3}\.){3}(\d{1,3})|([\s\d]{8,32}):\d{1,5}) that checks both IP and hostname (hostname check there could be better or made need to be different).
This would be useful for when we allow a list of IPs also.  We might want to allow and optional (http://|https://|etcd://){0,1} if we start allowing this, but we could have wrappers around this function that get the IP/port regardless of what style is passed in.

I realized that we already do ETCD_AUTHORITY validations in calico-containers utils.py, so I am changing this PR to move those validations into libcalico for use by other orchestrators (in this case, libcalico itself).
I'm going to try to keep the scope of this small so it is just a 1:1 move - the only change in this PR besides the move is calling the function in datastore.py
@TrimBiggs @caseydavenport I believe the validation takes care of all your comments (except multiple etcd_authorities and http/s, but I'd prefer to save those for a later PR). Sound good?

I'm ok with that.
This LGTM with a minor comment.  Are you planning to have a PR that removes this from calico-containers?

Looks fine to me.
I'd like to see the corresponding PR in calico-containers which removes this function.

@caseydavenport I haven't opened the PR as it will just fail CI tests: https://github.com/projectcalico/calico-containers/compare/master...djosborne:remove-validate-hostnames?expand=1

Two minor comments - fix them if it pleases you.
Then, squash commits, and then LGTM.


Bump elasticsearch to the latest in the example

As for instance http://drupal.org/project/elasticsearch_connector isn't compatible with the current example

@arafalov you've manually tested with this change, right?

@arafalov you've manually tested with this change, right?
I don't think you meant to At me. You meant @AronNovak I am guessing.

Sorry :) @AronNovak you've manually tested with this change, right?

@rfay

Yep, we have it up and running for one Drupal 8 project.

Hmm... you show 7.6.1 in this PR but your screenshot seems to show 7.5.2?

@rfay Thanks, sorry for the late follow-up. Actually I used the latest ES version in this PR, but for the sake of cross-project coherency, at our client project, I needed to downgrade a little bit. Good to see that it's in! :+1:


pushHistory throws an error when used inside a $effect hook

Describe the bug
pushHistory throws an error when used inside a $effect hook (Logs included below)
Reproduction
$effect(() => {
	pushState("", { current_page: $state.snapshot(current_page) })
})

Link to SvelteLab repo: https://www.sveltelab.dev/0w91yk8my729dku
Logs
Uncaught (in promise) TypeError: Cannot read properties of undefined (reading '$set')
    at pushState (client.js?v=0255b54e:1884:7)
    at $effect (+page.svelte:128:3)
    at execute_reaction_fn (chunk-O56VML7M.js?v=0255b54e:1560:29)
    at execute_effect (chunk-O56VML7M.js?v=0255b54e:1681:20)
    at flush_queued_effects (chunk-O56VML7M.js?v=0255b54e:1731:7)
    at flush_queued_root_effects (chunk-O56VML7M.js?v=0255b54e:1717:9)
    at flush_sync (chunk-O56VML7M.js?v=0255b54e:1832:7)
    at flush_sync (chunk-O56VML7M.js?v=0255b54e:1837:7)
    at hydrate (chunk-WZH6WDHG.js?v=0255b54e:476:12)
    at new Svelte4Component (chunk-WZH6WDHG.js?v=0255b54e:647:70)

System Info
System:
    OS: Linux 6.5 KDE neon 6.0 6.0
    CPU: (8) x64 11th Gen Intel(R) Core(TM) i5-1135G7 @ 2.40GHz
    Memory: 7.84 GB / 15.40 GB
    Container: Yes
    Shell: 3.7.1 - /usr/bin/fish
  Binaries:
    Node: 22.2.0 - ~/.local/share/nvm/v22.2.0/bin/node
    Yarn: 1.22.21 - ~/.local/share/nvm/v22.2.0/bin/yarn
    npm: 10.8.0 - ~/.local/share/nvm/v22.2.0/bin/npm
    pnpm: 8.15.3 - ~/.local/share/nvm/v22.2.0/bin/pnpm
    bun: 1.1.8 - ~/.bun/bin/bun
  Browsers:
    Brave Browser: <IP_ADDRESS>
  npmPackages:
    @sveltejs/adapter-static: ^3.0.1 => 3.0.1
    @sveltejs/kit: ^2.5.9 => 2.5.9
    @sveltejs/vite-plugin-svelte: ^3.1.0 => 3.1.0
    svelte: ^5.0.0-next.136 => 5.0.0-next.136
    vite: ^5.2.11 => 5.2.11

Severity
serious, but I can work around it
Additional Information
Workaround: put the pushState inside a setTimeout like this
$effect(() => {
	current_page
	setTimeout(() => pushState("", { current_page: $state.snapshot(current_page) }))
})

vite.config.js
import { sveltekit } from "@sveltejs/kit/vite"
import { defineConfig } from "vite"
export default defineConfig({
	plugins: [sveltekit()],
	build: { target: "es2015" },
	server: { hmr: false }
})

svelte.config.js
import adapter from "@sveltejs/adapter-static"
import { vitePreprocess } from "@sveltejs/vite-plugin-svelte"
/** @type {import('@sveltejs/kit').Config} */
export default {
	extensions: [".svelte"],
	preprocess: [ vitePreprocess() ],
	kit: { adapter: adapter() },
	compilerOptions: { runes: true }
}

I don't know if this can be useful, I wanted to give a shot to this but I don't really know where to put my hands on.
As far as i saw root is assigned in kit/src/runtime/client/client.js:434 in initialize function, but the effect is executed during the creation of root by sveltejs/svelte/blob/main/packages/svelte/src/internal/client/runtime.js:542, so everything that is called during $effect that uses root, such as pushState or replaceState, will throw an Error, as root is not yet assigned.
I hope this can be helpful, I'm not sure if this is even the right place to write this.

This is huge!

A better workaround than setTimeout might be to use svelte's tick function. Seems to work for me.
    $effect(async () => {
        await tick();
        replaceState("/", {});
    })

This happens because of our use of the legacy component wrapper, which calls flushSync, which means the effect runs before the component is fully instantiated.
The same happens if you would use onMount, and also in Svelte 4, so this is not strictly related to Svelte 5.


dword issue in .h file

Hello,
When we compile the .p4 file using p4c-xdp, it generates a .h and .c file. The .h file is missing a macro for dword, which needs to be inserted manually. Can you please fix this?
Thank you,
Sincerely,

can you post the dword macro you insert manually? Thanks!

Hello William,
This is the macro we added in the .h file.

#define load_dword(data, b) __constant_ntohl(*(u64 )((u8)(data) + (b)))

Thank you,
Sincerely,

thank you for reporting the issue. I pushed the change to the master.


Ensure pre-commit hooks for client-side changes use package.json versions

Problem
Currently, any updates to Eslint packages, Stylelint packages and Prettier have to be manually kept in sync with the pre-commit hooks config.
Ideally we should not have to do this and the pre-commit hooks should use the same scripts / versions that are set up in package.json.
Idea

In theory you could make use of repo: local - https://pre-commit.com/#repository-local-hooks and use it something like: https://github.com/pre-commit/pre-commit/issues/945#issuecomment-527603460 (npm run lint:format)

Originally posted by @zerolab in https://github.com/wagtail/wagtail/pull/10742#discussion_r1283313860

And I see our pre-commit hooks for stylelint are out-of-date again after #10719 🙈

OK I will close this. It's super tedious to keep these manually up to date but I will have a PR up to update this shortly.

@lb- I thought Thibaud's comment above meant that this would be useful? I'm in favour of this so I don't mind us giving it a try after we get #11595 in.

Note that https://github.com/wagtail/wagtail/pull/11679 has been raised to further fix the manual sync of package version updates with the pre-commit config.
Looks like there was a typo (on my part) but additional inconsistencies between the latest available packages and what the 'mirror' packages have in their repo for Prettier and others.
There must be a better way to manage this, it's error prone and stops us from keeping our core code up to date due to the mirror packages also needing updating.
Hopefully I'll get a chance to understand this stuff better, but anyone with better knowledge of pre-commit can chime in with some ideas.


Package request: DSPy (language model framework)

Project description

DSPy is a framework for algorithmically optimizing LM prompts and weights

Its a python package
would really appreciate someone putting a package together, or helping me figure it out
thanks!

Metadata

homepage URL: https://dspy-docs.vercel.app/
source URL: https://github.com/stanfordnlp/dspy
license: mit
platforms: python

Add a :+1: reaction to issues you find important.

        dspy = with pkgs.python312.pkgs;
          buildPythonPackage rec {
            pname = "dspy-ai";
            version = "2.4.10";
            format = "setuptools";

            src = pkgs.fetchFromGitHub {
              owner = "stanfordnlp";
              repo = "dspy";
              rev = "refs/tags/v${version}";
              hash = "sha256-jfPg1W9Qp0DlAbMXaFqZ6Ri2zMOW6EKUHwi7Azn/yl0=";
            };

            build-system = [setuptools];

            meta = with lib; {
              description = "The framework for programming—not prompting—foundation models";
              downloadPage = "https://github.com/stanfordnlp/dspy";
              homepage = "https://dspy-docs.vercel.app/";
              license = with licenses; [mit];
            };
          };

as far as I got
nix loads it but dspy isnt available in python

nix-init in a local nixpkgs clone might give you a better starting point, you'll likely still have to tweak though

{
  lib,
  python3,
  fetchPypi,
}:
python3.pkgs.buildPythonApplication rec {
  pname = "dspy";
  version = "2.4.10";
  pyproject = true;

  src = fetchPypi {
    pname = "dspy-ai";
    inherit version;
    hash = "sha256-lAGfHQ8M2iKvHZ1xcil/yAAqhZoPCqUb68gR9N0fDxQ=";
  };

  nativeBuildInputs = [
    python3.pkgs.setuptools
    python3.pkgs.wheel
  ];

  propagatedBuildInputs = with python3.pkgs; [
    backoff
    datasets
    jinja2
    joblib
    openai
    optuna
    pandas
    pydantic
    regex
    requests
    rich
    structlog
    tqdm
    ujson
  ];

  passthru.optional-dependencies = with python3.pkgs; {
    aws = [
      boto3
    ];
    chromadb = [
      chromadb
    ];
    docs = [
      autodoc_pydantic
      docutils
      furo
      m2r2
      myst-nb
      myst-parser
      sphinx
      sphinx-autobuild
      sphinx-automodapi
      sphinx-reredirects
      sphinx_rtd_theme
    ];
    epsilla = [
      pyepsilla
    ];
    fastembed = [
      fastembed
    ];
    marqo = [
      marqo
    ];
    milvus = [
      pymilvus
    ];
    pinecone = [
      pinecone-client
    ];
    postgres = [
      pgvector
      psycopg2
    ];
    qdrant = [
      fastembed
      qdrant-client
    ];
    weaviate = [
      weaviate-client
    ];
  };

  pythonImportsCheck = ["dspy_ai"];

  meta = with lib; {
    description = "The framework for programming—not prompting—foundation models";
    downloadPage = "https://github.com/stanfordnlp/dspy";
    homepage = "https://dspy-docs.vercel.app/";
    license = licenses.mit;
  };
}

got to this but this cant get it to load in python

update:
@NobbZ on discord helped me out but it seems theres some upstream problems

Where is poetry?
Wait… What is that?
Which of these is canonical?
Please ask upstream to fix their pyproject.toml
As far as I can tell, those lists do not even agree…

Why do they mention poetry in the README, if the pyproject.toml states that setuptools was used?
Why do they specify dependencies in setup.py (and a variety of requirements.txts) and in the project.dependencies list and in the tool.poetry.dependencies list?
Why do those lists even disagree?

ive created stanfordnlp/dspy#1234
{ lib
, buildPythonPackage
, fetchPypi
, setuptools
, wheel
, backoff
, datasets
, jinja2
, joblib
, openai
, optuna
, pandas
, pydantic
, regex
, requests
, rich
, structlog
, tqdm
, ujson
, boto3
, chromadb
, autodoc-pydantic
, docutils
, furo
, m2r2
, myst-nb
, myst-parser
, sphinx
, sphinx-autobuild
, sphinx-automodapi
, sphinx-reredirects
, sphinx-rtd-theme
, pyepsilla
, fastembed
, marqo
, pymilvus
, pinecone-client
, pgvector
, psycopg2
, qdrant-client
, weaviate-client
}:

buildPythonPackage rec {
  pname = "dspy";
  version = "2.4.10";
  pyproject = true;

  src = fetchPypi {
    pname = "dspy-ai";
    inherit version;
    hash = "sha256-lAGfHQ8M2iKvHZ1xcil/yAAqhZoPCqUb68gR9N0fDxQ=";
  };

  nativeBuildInputs = [
    setuptools
    wheel
  ];

  propagatedBuildInputs = [
    backoff
    datasets
    jinja2
    joblib
    openai
    optuna
    pandas
    pydantic
    regex
    requests
    rich
    structlog
    tqdm
    ujson
  ];

  passthru.optional-dependencies = {
    aws = [
      boto3
    ];
    chromadb = [
      chromadb
    ];
    docs = [
      autodoc_pydantic
      docutils
      furo
      m2r2
      myst-nb
      myst-parser
      sphinx
      sphinx-autobuild
      sphinx-automodapi
      sphinx-reredirects
      sphinx_rtd_theme
    ];
    epsilla = [
      pyepsilla
    ];
    fastembed = [
      fastembed
    ];
    marqo = [
      marqo
    ];
    milvus = [
      pymilvus
    ];
    pinecone = [
      pinecone-client
    ];
    postgres = [
      pgvector
      psycopg2
    ];
    qdrant = [
      fastembed
      qdrant-client
    ];
    weaviate = [
      weaviate-client
    ];
  };

  pythonImportsCheck = [ "dspy_ai" ];

  meta = with lib; {
    description = "The framework for programming—not prompting—foundation models";
    downloadPage = "https://github.com/stanfordnlp/dspy";
    homepage = "https://dspy-docs.vercel.app/";
    license = licenses.mit;
  };
}

Hi @ethanniser, Is the file above the only one you have? Do you have error messages when trying to build the above?
I found your issue while looking for marqo in nixpkgs. It does look like some of the dependencies incl.  marqo are not yet available on Nixpkgs.
I'm trying to get a local derivation up and working for that. And given that I'll eventually have to use dspy as well, I might try to package the dependencies while we wait for your upstream issue to be fixed -- hopefully then we'd be able to have play with the dspy repo with nix develop

Thanks for taking a look, this was the only file
I ended up just using a venv for now as I did feel like there was much more I was able to do


Hlsjs v0.6.1

Some internal changes needed fixes in our integration test setup. No functional changes required adaptation on Streamroot P2P tech side.
Acceptance tests results:

all usage modes (wrapper, bundle, legacy) working
(Firefox OK, Chrome: ARTE test stream broken ⚠  - MediaSource related error, probably not on our side) NOTE: also tested the ARTE stream (http://www.streambox.fr/playlists/test_001/stream.m3u8) in official hls.js demo, and error says video is using unsupported native features (codec issue?) -

Hls.js bundled demo working (Firefox & Chrome)

Except missing CHANGELOG update seems alright to me.

@ruslandinov What should we do about this one?

@jlouazel I've updated the CHANGELOG.md and fixed let usage in Gruntfile.js. Merging this now.

@jlouazel I think we can merge this, but let's ask Axel just in case we are missing something.
@AxelDelmas do you know if there is something that could prevents us from using hlsjs v0.6.x in hlsjs-p2p-wrapper?

No worries, Stephan had checked the compatibility.
Plus if you look at the diff between 0.5.46 and 0.6.1, there's nothing we need to worry about: https://github.com/dailymotion/hls.js/compare/v0.5.46...v0.6.1

OK, so I guess we can merge this one --> @ruslandinov


Change speaking suggestions to "I.e. AI technical safety, global health"

Also appears in /profile/edit/career

Done ✅
https://github.com/rtcharity/hubreboot/commit/da4295007d25f24c5e99fb03b21ca1be7bc038e8


Python interface

I added the spiral order for the swaths. Also, I think it would be good to have one more constructor of the SpiralOrder with only one argument being the size of the spiral. Then similarly to others, the setSwaths(F2CSwaths& swaths) method can be used.

Hi @jfulem,
First, thank you for your contribution. Unfortunately, I would split it in two different PR. First, the new constructor of spiral that should go to the main branch, and second, the python_interface that merge all the changes to the second branch. If we accept this PR, the constructor will be only in the python_interface branch until the python_interface is merged to main. Do you agree about it? Could you split it in two PR, please?
Also, remember to add yourself to the Authors and Contributors files. Thank you again.

All right. I'll do it.

@jfulem Please, tell me when the PR is ready to review.

I think it is.


License

Thanks for the amazing work! Was curious if you were planning on adding a license to this repo? Hopefully open-source 💯

Yeah, will definitely have one soon.

Added MIT LICENSE


Terraform Init Issue

Greetings,
Any thoughts on why I receive this error on the "Terraform Init". This looks to be the latest version and I have the Installer first in my release. Any help would be appreciated Thanks.
2020-01-31T20:05:10.7966799Z ##[section]Starting: terraform init
2020-01-31T20:05:10.8065667Z ==============================================================================
2020-01-31T20:05:10.8065739Z Task         : Terraform CLI
2020-01-31T20:05:10.8065769Z Description  : Execute terraform cli commands
2020-01-31T20:05:10.8065797Z Version      : 0.4.15
2020-01-31T20:05:10.8065826Z Author       : Charles Zipp
2020-01-31T20:05:10.8065870Z Help         :
2020-01-31T20:05:10.8065897Z ==============================================================================
2020-01-31T20:05:11.8838217Z [command]C:\hostedtoolcache\windows\terraform\0.12.20\x64\terraform.exe version
2020-01-31T20:05:11.9858644Z Terraform v0.12.20
2020-01-31T20:05:12.0089840Z ##[error]Error: Input required: backendServiceArm
2020-01-31T20:05:12.3319950Z ##[section]Finishing: terraform init
My TF file has the entry
terraform {
backend "azurerm" {
}

@clifpi it looks like the service connection for the storage backend has not been set due to ##[error]Error: Input required: backendServiceArm. If this is still an issue, please post the yaml or ui configuration for the task that is producing this issue.


Option to use without creating new queue

It seems this tries to create queue, we need a option to use existing queue. https://github.com/nttcom/fluent-plugin-rabbitmq/blob/master/lib/fluent/plugin/in_rabbitmq.rb#L106

The declaration will have no effect if the queue does already exist.
Please see: http://rubybunny.info/articles/queues.html


Can't change some settings in 7.12 for 7.3.5 wow client.

Description
Basically title. One of the settings I'm trying to change is UI scale that is locked in normal settings. I type "1" and click apply, it doesn't change anything.
Version Number
7.12
Steps to Reproduce

Click ESC button
Click "KkthnxUI"
Go "General" tab or just leave it after Step 2, as it opens General anyways.
Change UI Scale to "1"
Click "Apply"
Wait for UI reload

Expected Behavior
Scale changed to a number you type.
Actual Behavior
Nothing
Only KkthnxUI Enabled
Yes, the same
Additional Information
Texture also isn't changed. Also tried reset all settings through Battle.net app.
Gonna try to install 7.13 beta
Error Log
No LUA errors after typing /console scriptErrors 1

Did you uncheck autoscale first?

You need to uncheck autoscale and then it works. As for bars texture many frames have their own setting that overrides this global one - you need to change them all.


Wrong font (Win 10/Brave)

https://www.dropbox.com/s/9lqijx4mvvz6k8r/2020-08-17_16-00-35.mp4?dl=0

Hi
from a developer perspective, there are no places where I set a font - at least I can't remember. I think Ionic has a complex set of cascading style sheets that add up to the finale font being used.
I can understand if you - in that chain - deselect a font, suddenly things change, but not sure how I can circle that back to a bug or make it a documented feature
Thanks for the video though. I am going to think it over a bit more.. For now I am a bit clueless about the exact problem definition
Tom

I did add the font defition as a global CSS, as I wanted to know where to place global CSS in the svelte way (which btw, I believe can be done in different places).
App.svelte
<style>
  :global(body) {
    /* this will apply to <body> */
    --ion-font-family: "Roboto", "Helvetica Neue", sans-serif;
  }
</style>


⚠ Secret Website 3 has degraded performance

In 0c26ac1, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2221 ms

Resolved: Secret Website 3 performance has improved in 94e19a2.


Comments reversed between GetSingers and GetSignerBytes in multiple modules

    This comment looks to be reversed with the comment on `GetSignerBytes` method in `ln:89`.

// GetSigners indicates that the message must have been signed by the address provided

Originally posted by @egaxhaj in https://github.com/provenance-io/provenance/pull/1285#discussion_r1066230382

The GetSignerBytes funcs were all removed with #1304. They're part of the LegacyMsg interface, but not the Msg interface; they're not needed anymore.
So I'm pretty sure it's safe to close this. Feel free to reopen if there's more to do.


Bootstrap 5 support for Laravel-AdminLTE

Is there any roadmap for Bootstrap 5 support for AdminLTE.

There is a currently development branch for Bootstrap 5 on the underlying AdminLTE v.4. Once the underlying package supports it with an official version, we will need some time and a lot of work to update the used AdminLTE package version here, since it will contain lot of major and breaking changes... On summary, there is no plan for Bootstrap 5 here in the short time.

@dfsmania It seems that https://github.com/ColorlibHQ/AdminLTE has a working branch
but you have no v4 branch yet. will this be on the schedule ??


⚠ PrivateHD has degraded performance

In 17c31b4, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 2177 ms

Resolved: PrivateHD performance has improved in 69e2133.


Add banner

Per @sbesson's request and review: added a banner for announcements.
You can change the background color using any of IDR's brand colors by using class="bg-idr-[color] where [color] can be any of the colors in the logo: red, blue, purple, orange, yellow.
The background colors can also be used for any div on the site.
Also, if you want to apply the brand colors to any text in the site, just use class="idr-[color].

Discussed during the weekly IDR call. The Aspera issues reported last week have been fixed and I could confirm this morning the data is accessible for e.g. idr0081.  However, keeping the CSS layout additions so that we can easily add such an announcement in the future if needed.

@sbesson Yes, confirming that if you want the banner to appear globally on all pages, it goes in the head-idr.html file, otherwise you copy the block of code on whatever pages you want it on.


Update GPU CI RAPIDS_VER to 24.06, disable query planning

As we're waiting on full support for RAPIDS / dask-expr integration, I'm interested in what GPU CI currently looks like with the latest RAPIDS packages and query planning disabled.

rerun tests


Add publish registry

For https://github.com/IBM/openapi-to-graphql/packages
Not sure if this is a good idea? Doesn't seem to hurt...

Currently they support only scoped packages so it will be hard to migrate without rename.

https://help.github.com/en/articles/configuring-npm-for-use-with-github-package-registry#publishing-a-package


fix idleTimeout values 16,20,24,28 (8 second margin)

Beat you to it 😄

One of these days I will learn enough c++ to print Hello World to console and maybe add a couple numbers together, not there yet though. Thankfully I can read it just enough to make out what is happening

😉


Add better support for livereload

This lets you inject CSS changes into the page directly without having
to do a page refresh.
The LiveReload browser extension is required for this to work.
To test, run then following commands:
./scripts/bundle.sh --watch
./scripts/npm.sh run livereload
Then edit any SASS file.

You'll need to provision your app VM or run ./scripts/npm.sh install for this to work.

+1 This is fantastically fast!


simplify OS specific homedir detection and some minor cleanups

@mitchellh Ping?

It'd be nice to have this merged back in!

Any way to get this merged ?

Hey @harshavardhana, thanks for the PR. Correct me if I'm wrong but could you explain the benefits of this PR? I see some code shuffling but curious on some decisions. For example, you changed the RWMutex to a Mutex which could maybe be better due to this usage but do you have reasons or data to back this up?
You also added copyrights into every file which I don't do. Over the years of being deeply involved in OSS I've talked to no shortage of lawyers and the feedback on this goes back and forth but the lawyers I work with say that the copyright is not necessary since it is implicit based on the license and CLA (if there is one, this project does not have one).
I'm fine with most of the changes and I can clean them up afterwards. Main question is the mutex changes probably!

For example, you changed the RWMutex to a Mutex which could maybe be better due to this usage but do you have reasons or data to back this up?

very historical @mitchellh perhaps i can revert it back. i am not really sure of the top of my head.

@mitchellh there you go changed it back to RWMutex

Is there any reason why this hasn't been merged yet, or couldn't be?


getRecord() always defaults to the wrong signature

I'm running into an issue where Swift is always calling the wrong version of getRecord.
I'm trying to use this version:
 func getRecord<T: Codable>(table: String, id: String) async throws -> T { ... }

But Swift always seems to call this one, the one that expects an int:
 func getRecord<T: Codable>(table: String, id: Int) async throws -> T { ... }

The ID is defined as a string and I even tried casting it, but it is still happening.
.getRecord(table: "Task", id: id as String)
Now matter what I do I get this error:
keyNotFound(CodingKeys(stringValue: "value", intValue: nil), Swift.DecodingError.Context(codingPath: [], debugDescription: "No value associated with key CodingKeys(stringValue: \"value\", intValue: nil) (\"value\").", underlyingError: nil))

Hello, @radicalappdev! Thank you for reporting the problem. This error is due to a change in the formatting of the FileMaker API response JSON.
We have already fixed the issue and it will be available in the main branch shortly.


feat: Refactor c/c++ clients release repository.

Description
This change fixed a CI issue where the values feed into CONFLUENT_PACKAGE_BASEURL now longer have a /clients repo behind them. The ideal situation is that https://packages.confluent.io/clients/ is unconditionally used irrespective of where we point confluent_common_repository_baseurl too with a nightly/staged release.
You can see these failures in CI when triggered by the upstream CI Packaging job.
Fixes # (issue)
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
CI runs with params and without should work.
Test Configuration:
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Any dependent changes have been merged and published in downstream modules
[ ] Any variable changes have been validated to be backwards compatible

PR build will likely be blocked on this fix https://github.com/confluentinc/confluent-docker-build-images/pull/172


fix hinton visualization bug

Summary
Fixes https://github.com/Qiskit/qiskit-terra/issues/8446
Details and comments
I corrected
ax2.set_xlim([0, lx]) ax2.set_ylim([ly, 0])
and added
fig.tight_layout()
Tests are done on my computer to cover the change so it looks aligned:

And the spacing is fixed for a larger density matrix:

One thing is that now the diagonal of the density matrix runs from lower left to upper right, which is changed from the previous function, but it is just a style thing.

 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.

 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.

Thanks for submitting the PR for fixing the issue you found. As this is your first time contributing to Qiskit Terra, please have a look at the general Qiskit contributing guideline (especially the "Pull Request Checklist" section) and the CONTRIBUTING.md in Terra.

Can you please write a release note and sign the Contributor License Agreement?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
11 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.0002%) to 83.993%

Files with Coverage Reduction
New Missed Lines
%

qiskit/compiler/transpiler.py
11
89.95%

Totals

Change from base Build<PHONE_NUMBER>:
0.0002%

Covered Lines:
55906

Relevant Lines:
66560

💛  - Coveralls

Can you please write a release note and sign the Contributor License Agreement?

I think I have signed the Contributor License earlier today, but I am not sure about where to write a release note. Could you guide me through that? Thank you!

I corrected the ylim part which was a mistake introduced same PR that caused the issue this PR (8447) is trying to fix by https://github.com/Qiskit/qiskit-terra/pull/4811.
4811 mistakenly did ax1.set_ylim([ly, 0]) twice which basically cancel each other. And ax1.invert_yaxis() inverts the axis. On the other hand, ylim of ax2 was not set, and ax2.invert_yaxis() inverts the axis. The end results is that both y axis are inverted which are the desired plotting method.
When @wyqian1027 corrected ax1.set_ylim([ly,0]) to ax2.set_ylim([ly,0]), the inverted ylim and the invert axis cancel each other, that's why the y axis was back to normal order (lower values at the bottom)

LGTM. Thanks a lot for the fix. Congrats for getting your first PR merged! Hope you can continue making contributions to Qiskit. If you need any help in the future, feel free to reach out to me on GitHub or Qiskit Slack.

Thank you! I will do my best.

Looks like CI failed due to server errors. Can we trigger the CI to run again? @mtreinish

@jakelishman I reverted the y-axis direction in https://github.com/Qiskit/qiskit-terra/pull/8447/commits/4b43d59985f47e0727e4e8676367ad405028839e. It should be the same as the original direction.

Related to that, currently the hinton plot seems to plot the matrix transversely. Is that the expected behaviour?

@jakelishman

I'm concerned about the change in ordering the outputs here. It's not simply a style choice (and even if it were, we already picked a convention, and it's not good for us to change that without a very good reason). The original form makes the Hinton plot resemble the standard matrix order, which makes it easier to read.

I reverted the y-axis direction in https://github.com/Qiskit/qiskit-terra/pull/8447/commits/4b43d59985f47e0727e4e8676367ad405028839e. It should be the same as the original direction. Do you think the PR is good for merging?

@Mergifyio requeue

@Mergifyio refresh

@jakelishman Thanks for following up and checking it thoroughly!


[disaster dashboard] initial page set up

set up skeleton for the new disaster dashboard page (so future PRs can be submitted in parallel)
This is one part of the changes for the disaster dashboard MVP here on dev:
https://dev.datacommons.org/dev/disaster-dashboard

thanks for the reviews!


Dry-run (simulated) mode

As a user of vCloud Launcher
Wishing to see what changes vCloud Launcher would make when run
I would like vCloud Launcher to have a 'dry-run' mode
There is prior art for this in the vCloud Edge Gateway gem.

Added in https://github.com/gds-operations/vcloud-launcher/pull/114


Fix management script utilities support.

This fixes #678.
Sorry for the submitting branch problem.

Thanks!


Randomize test order

Jest recently added an option to randomize the order of tests. In general, that's a good practice that can help ensure we make tests more independent and robust, so we should be using it.
Fixes #1345.

I was previously seeing an error in db.test.ts, but am having trouble reproducing it now. 🤷


Fix dropdowns in new WI

This fixes the empty dropdowns for the new WI dialog. The issue was caused by the dropdown values changing after the dialog is shown. The typeahead dropdown did not support Input() changes. It does support this now.

@michaelkleinhenz snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1723-fabric8-planner.int.rdu2c.fabric8.io

@michaelkleinhenz snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1723-fabric8-planner.int.rdu2c.fabric8.io

Everything looks good to me but, don't you think the default are and iteration should be selected by default?

Also if I select an area it stays None. Works fine with iteration.

@michaelkleinhenz snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1723-fabric8-planner.int.rdu2c.fabric8.io

@sanbornsen there should no "None" in the boxes anymore. Where do you see that?

@michaelkleinhenz snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1723-fabric8-planner.int.rdu2c.fabric8.io

@nimishamukherjee @sanbornsen Ah, I get it. No, that is actually an expected behaviour. The "None" only appears on "old" work items that don't have an area set. With the old core behaviour, new WIs were created without an area. With the new core, the default/root area is always set. The "None" only appears (and should appear) on "incorrect" WIs not having an area. Can you confirm that it is working that way?

@michaelkleinhenz snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1723-fabric8-planner.int.rdu2c.fabric8.io

@michaelkleinhenz snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1723-fabric8-planner.int.rdu2c.fabric8.io

@michaelkleinhenz snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1723-fabric8-planner.int.rdu2c.fabric8.io

@michaelkleinhenz snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1723-fabric8-planner.int.rdu2c.fabric8.io

@michaelkleinhenz snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1723-fabric8-planner.int.rdu2c.fabric8.io


Slugifying Chinese characters with multiple readings

This is probably beyond doable, since it would require extensive lookups, but slugifying can result in erroneous output if a given character has multiple readings (多音字), and context would be required to determine the correct slug, e.g. 行 is correctly rendered 'xing' when it stands alone, but 'hang' it would have to be slugified as 'hang' when it stands with 银行 (銀行) in a compound word. (See 3rd paragraph in this.)

The conversion is done by the underlaying modules.


pillow version upgraded to 10.0 or above

Describe the problem
Can you upgrad the pillow version required by mtb nodes beyond 10.0 or above?
Describe the solution you'd like
This node conflicts with some of the dependencies required by many other nodes, including comfyui's main program's required.
Alternatives considered
No response
Additional context
No response

Hi,
It's already the case

Oh beyond? I had the opposite issue and decided to update since it broke mine, I'll had guards on the version then, thanks for the report


Add missing v0.16.1 data

v0.16.1 has been released but there is no info in the changelog, this PR adds missing info about new release to the changelog

5c1296a4 backfilled the changelog


Add support to zeroize plaintext in S3 record layer

Some applications want even all plaintext copies beeing
zeroized. However, currently plaintext residuals are kept in rbuf
within the s3 record layer.
This patch add the option SSL_OP_CLEANSE_PLAINTEXT to its friends to
optionally enable cleansing of decrypted plaintext data.

This patch is a followup of the discussion with Matt on openssl-users
https://mta.openssl.org/pipermail/openssl-users/2020-June/012604.html

Checklist

[x] documentation is added or updated
[x] tests are added or updated

Please can you submit a CLA:
https://www.openssl.org/policies/cla.html

Please can you submit a CLA:
https://www.openssl.org/policies/cla.html

Sure the CCLA is already in my mailbox, the ICLA I am singing tomorrow (my printer is dead) and planning to submit them together

are you preferring to amend the commit or a followup and squash later?

I am wondering, if we should add a similar behaviour in in dtls1_read_bytes() as well. But since I am not sure about the peek/read behaviour for datagrams I am a little bit hesitant.

are you preferring to amend the commit or a followup and squash later?

Preferred is to add additional commits fixing up previous ones using git commit --fixup. That way whoever merges can just use git rebase -i --autosquash

I am wondering, if we should add a similar behaviour in in dtls1_read_bytes() as well. But since I am not sure about the peek/read behaviour for datagrams I am a little bit hesitant.

Oh, yes. Good point - we should do that. It should be virtually the same.

Hmm. The travis failures look odd - but I don't think they are related to this change.

Hmm. The travis failures look odd - but I don't think they are related to this change.

Well that concerns me as well. I run the testsuite on my local machine and there all looks good.
However since travis checks more this is just an indication.
The odd thing is that we see red Xes all over the place even on merged commits. I did not check in detail but the logs look similar.
However since it least the 'if' statements are executed in many configurations, I am not sure what to do about this.

Please can you submit a CLA:
https://www.openssl.org/policies/cla.html

Sure the CCLA is already in my mailbox, the ICLA I am singing tomorrow (my printer is dead) and planning to submit them together

I sent the ICLA and CCLA a couple of ours ago, please tell if something is missing

To summarize the delta:

Addressed your comments
Added zeroization for DTLS (and test)
Test: Added checks to explicitly check internal buffer contents after SSL_peek, and after SSL_read
Test: Moved test functions down, it was in between the large_message tests

are you preferring to amend the commit or a followup and squash later?

Preferred is to add additional commits fixing up previous ones using git commit --fixup. That way whoever merges can just use git rebase -i --autosquash

So hold back with rebasing myself for now

Close/reopen to kick CLA bot

Travis is now passing :-)

Travis is now passing :-)

Feels better this way :)

Since there are still changes requested. Is something missing from my side?

Fixed proposal in CHANGES.md

@slontis: thx for the review,
I added update according to to your proposial (except the one i commeneted on).

This pull request is ready to merge

Squashed and merged. Thanks!

Thanks all


Add xxxxf functions to standardize and enable tooling

Minor clean, add the xxxf function, deprecate the non-standard one, tidy up the readme.
not done here, but we should consider following the stdlib log pattern and have a global default logger instead of creating a new one each time we call the top-level function.
cc @ammario @nhooyr
Fixes #5

We don't have versions yet though. Also, the // Deprecated comment will be picked up by tooling / godoc so people already using it might catch dangling issue they didn't catch before.
I was thinking we could create a go.coder.com/flog/v2 and remove the old ones there after merging, but I don't mind doing it after.
Regarding completion, I didnt' check, but completion tools should skip deprecated candidates, and other loggers like stdlib or logrus have both Info / Infof variants (although the Info one will not apply the formatting).
Let me know what you think :)

I'm down for v2 👍

We could just run a sed to fix any breakage as it's just the addition of f to flog.Something( everywhere.
This isn't a widely used package as of yet so I don't think community breakage is a concern.

i.e in general major versions are for big breakage/changes, and I don't think this qualifies.


Java tests cannot start on Mac: Could not find OpenVinoTestRunner

Java tests cannot start on Mac
"Error: Could not find or load main class OpenVinoTestRunner"
See logs for details:
https://dev.azure.com/openvinoci/dldt/_build/results?buildId=61614&view=logs&j=a82d9e72-1de4-5bd6-c188-90296135479e&t=e36b0236-8854-50ba-7e69-658d6b092df0

related: https://github.com/openvinotoolkit/openvino_contrib/pull/18


Service error: Broken pipe

Description
Hi, all. I have tried to develop wps services on windows 8 64bit.
The errors in both server and client sides were shown as

server side

2016-11-07 14:52:32,924 INFO sqlalchemy.engine.base.Engine ()
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "D:\dev\python27_64\lib\multiprocessing\forking.py", line 380, in main
    prepare(preparation_data)
  File "D:\dev\python27_64\lib\multiprocessing\forking.py", line 489, in prepare

    file, path_name, etc = imp.find_module(main_name, dirs)
ImportError: No module named demo

client side

<!-- PyWPS 4.0.0rc2 -->
<ows:ExceptionReport xmlns:ows="http://www.opengis.net/ows/1.1" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.opengis.net/ows/1.1 http://schemas.opengis.net/ows/1.1.0/owsExceptionReport.xsd" version="1.0.0">
  <ows:Exception exceptionCode="NoApplicableCode" locator="" >
      <ows:ExceptionText>Service error: [Errno 32] Broken pipe</ows:ExceptionText>
  </ows:Exception>
</ows:ExceptionReport>

I already tested the wps service after installing demo. How to solve this problem?
Environment

operating system: Windows 8 64bit
Python version: 2.7.10 64bit
PyWPS version: 4.0.0rc2 (cloned on Nov. 07, 2016)
source/distribution
[ O ] git clone (Nov. 07, 2016)
[ ] Debian
[ ] PyPI
[ ] zip/tar.gz
[ ] other (please specify):
web server
[ ] Apache/mod_wsgi
[ ] CGI
[ O ] other (please specify): demo.py

Steps to Reproduce
Additional Information

Could you possibly pass your configuration file? How do you start the PyWPS-demo deployment? Using python3 demo.py command? What is do documentation you were taking and it does not work?
Thanks

@jachym I just used the command 'python2 demo.py' with the default configuration file. I can run the pywps service on Windows OS as you know but the above errors are shown when calling the wps service.
The sample code is
execution = wps.execute(processid, inputs, output = 'OUTPUT')
print execution.checkStatus()

Thanks //

Still not sure, where this could occure. I do note see any "demo" string used in our code base  (both PyWPS and pywps-demo). Just in the setup.py script of pywps-demo.
Are you saying, that you are running
cd pywps-demo
python2 demo.py

without problem, and then, when calling
http://localhost:5000/wps?service=wps&request=getcapabilities
You get this error?
Thanks for help
P.S. Sorry for late resposne, the notification slipped through my mailbox.

PyWPS does not support Windows, the Multiprocessing library does not work on that system. Closing this issue as no obvious cause has been found.