Shallow water explicit

This pull request is to add a second order explicit shallow water integrator and associated Williamson2 analytic test case into GridapGeosciences.
It includes additions to the src/ directory as follows:

additional diagnostics for the shallow water equations (mass, vorticity, energy, power)
a reference domain grad_perp operator (low level)
diagnostic equation to compute the vorticity (low level)
diagnostic equation to compute the potential vorticity (mixed low level/high level)
explicit integrator (nonlinearly stable, exactly balances energy exchanges in space and time)
integration driver function, takes in an integrator as a function pointer and cycles over this for a specified number of time steps

It also includes an analytic test (williamson2), run at multiple spatial and temporal resolutions in order to determine the L2 errors in the velocity and the fluid depth. L2 errors at 3 resolutions using RT1 and supplied in order to test against and verify second order accuracy

Codecov Report

Merging #15 (29739ac) into master (f7aadf4) will decrease coverage by 14.78%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           master      #15       +/-   ##
===========================================
- Coverage   43.35%   28.57%   -14.79%
===========================================
  Files          10       12        +2
  Lines         286      434      +148
===========================================
  Hits          124      124
- Misses        162      310      +148

Impacted Files
Coverage Δ

src/DiagnosticTools.jl
6.06% <0.00%> (-7.28%)
:arrow_down:

src/Operators.jl
100.00% <ø> (ø)

src/ShallowWaterIntegrators.jl
0.00% <0.00%> (ø)

src/Weak2DCurlOperators.jl
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

Hi @amartinhuertas , I've tried to address your helpful comments as best I can. The code looks a lot better now thanks! (results are the same as before for the test, I'm kinda surprises that there is no difference in the L2 errors given the solve() has been replaced with the ldiv!() ...)
The only thing I have not addressed (of which I'm aware) is the removal of the grad_perp_ref_domain(), and the rest of the routines in the Weak2DCurlOps.jl file. If you feel strongly that these should be removed and replaced with high level physical space terms (and an AD Jacobian) I will do so. My only concern is that without a formal proof that the two are equivalent vorticity will not be conserved and the benefits of compatible FEM will not be gaurunteed.
Let me know what you advise, and thanks again for your review.

Hi @davelee2804,

Hi @amartinhuertas , I've tried to address your helpful comments as best I can. The code looks a lot better now thanks! (results are the same as before for the test, I'm kinda surprised that there is no difference in the L2 errors given the solve() has been replaced with the ldiv!() ...)

Cool. Thanks for your work! The code looks pretty!  I have marked as resolved all those comments that were already  addressed.
were the following comments addressed?

https://github.com/gridapapps/GridapGeosciences.jl/pull/15#discussion_r700948954,
https://github.com/gridapapps/GridapGeosciences.jl/pull/15#discussion_r700965121
https://github.com/gridapapps/GridapGeosciences.jl/pull/15#discussion_r700973391

In general, in PRs, if we decide not to go ahead with a recommendation, we also typically write some words on why not on the corresponing comment text box.  The code reviewer might not have the whole picture of the work done, and may provide a missleading recommendation due to that. Like the rebuttal letter to the paper referees :-)

The only thing I have not addressed (of which I'm aware) is the removal of the grad_perp_ref_domain(), and the rest of the routines in the Weak2DCurlOps.jl file. If you feel strongly that these should be removed and replaced with high level physical space terms (and an AD Jacobian) I will do so. My only concern is that without a formal proof that the two are equivalent vorticity will not be conserved and the benefits of compatible FEM will not be gaurunteed.

First of all, just in case it is not clear, we can either use isoparametric or analytical mapping independently of what we want to do with the right hand side of the weak div perp operator.  Second,  I would go for the high level API term. Numerically, they seem to be equivalent (up to a minus sign). There is no mathematical proof yet they are equivalent, that is right.
At this point, I you are still concerned, what I suggest you to do is the following.

Let us create a branch of the current branch with the current implementation, shallow_water_explicit_grad_perp_ref_vorticity.
Let us transform the current implementation in shallow_water_explicit using the high level API terms. You will see that the code will look even prettier, more general, and abstract.
Let us compare systematically with a benchmark problem that you are particular worried about the results of the codes in branches 1 and 2. This would provide more confidence on their equivalence.
If it turns out that there are differences among 2. and 3., let us think how can we abstract the grad_perp_ref_domain term using the high level API. Hopefully they turn to be equivalent and we just have to merge 2. into main.

What do you think?

Hey @amartinhuertas , sounds like a good plan. I will make this secondary branch for the grad_perp_ref_domain, and then revert the existing branch to the high level physical space version as you suggest. Since we are computing vorticity conservation as a diagnostic we can monitor this for any bias with the physical space implementation and revisit if necessary.
Sorry for not commenting on the changes I had not made, I'll be more careful with this in future....

Hi @amartinhuertas , I've made the grad_perp branch as you suggested, then removed the low level reference operators from the shallow_water_explicit branch and replaced these with the high level physical space terms as you suggested. The errors are similar but not the same. Here are the conservation and l2 errors for the final 10 time steps for both configurations using the C16 mesh and RT1 elements, first the high level physical space curl:
step | mass conservation | vorticity conservation | kinetic energy | potential energy | energy conservation | power
70	-1.9114360408403353e-15	5.960464477539063e-8	6.633793145418441e20	1.477262020091013e22	-6.469974972230993e-10	2.9468087973284225e9
71	-1.9114360408403353e-15	1.7881393432617188e-7	6.633793153306743e20	1.4772620200009785e22	-6.542217757846671e-10	5.013208853950043e9
72	-1.9114360408403353e-15	0.0	6.633793164808813e20	1.4772620198748801e22	-6.613983670972954e-10	7.013473197860233e9
73	-1.9114360408403353e-15	-1.7881393432617188e-7	6.63379317773161e20	1.4772620197346446e22	-6.685294449387136e-10	7.772140080217796e9
74	-1.9114360408403353e-15	0.0	6.633793188754459e20	1.4772620196134932e22	-6.756057707535717e-10	6.685436270421591e9
75	-1.9114360408403353e-15	5.960464477539063e-8	6.633793195407092e20	1.4772620195361106e22	-6.82638756874949e-10	4.243648649419777e9
76	-1.9114360408403353e-15	0.0	6.633793197822413e20	1.4772620195011748e22	-6.896241916084949e-10	1.8942620096129227e9
77	-1.9114360408403353e-15	-3.5762786865234375e-7	6.633793199121693e20	1.47726201947747e22	-6.965637052875065e-10	1.280609604765152e9
78	-2.1238178231559282e-15	-5.960464477539063e-8	6.633793204058822e20	1.4772620194174609e22	-7.034553958564704e-10	3.2850473646575394e9
79	-1.9114360408403353e-15	5.364418029785156e-7	6.633793216577788e20	1.477262019281706e22	-7.102999426209271e-10	7.441219225807159e9
80	-2.1238178231559282e-15	1.1920928955078125e-7	6.633793237602512e20	1.4772620190609624e22	-7.170997910808223e-10	1.2075524820265934e10
n=16,	err_u: 0.0007415233680146857,	err_h: 0.00036813020391681306

...and here are the low level reference space grad-perp conservation and l2 errors:
step | mass conservation | vorticity conservation | kinetic energy | potential energy | energy conservation | power
70	-1.9114360408403353e-15	0.0	6.633793145418448e20	1.4772620200910134e22	-6.46997089639775e-10	2.9468087698963394e9
71	-1.9114360408403353e-15	1.7881393432617188e-7	6.633793153306746e20	1.4772620200009787e22	-6.542216399235591e-10	5.013208836234894e9
72	-1.9114360408403353e-15	1.1920928955078125e-7	6.633793164808811e20	1.4772620198748807e22	-6.613978236528631e-10	7.0134731802047e9
73	-1.9114360408403353e-15	1.1920928955078125e-7	6.633793177731617e20	1.4772620197346446e22	-6.685294449387136e-10	7.772140071719622e9
74	-1.9114360408403353e-15	1.7881393432617188e-7	6.633793188754468e20	1.4772620196134934e22	-6.756053631702474e-10	6.685436286510313e9
75	-1.9114360408403353e-15	2.384185791015625e-7	6.633793195407095e20	1.4772620195361104e22	-6.826388927360571e-10	4.2436486332578735e9
76	-2.1238178231559282e-15	2.980232238769531e-7	6.633793197822415e20	1.4772620195011752e22	-6.896237840251706e-10	1.894261969870346e9
77	-1.9114360408403353e-15	2.384185791015625e-7	6.633793199121695e20	1.4772620194774694e22	-6.965641128708307e-10	1.2806096133585129e9
78	-2.336199605471521e-15	0.0	6.633793204058827e20	1.477262019417461e22	-7.034552599953623e-10	3.2850473504682846e9
79	-1.9114360408403353e-15	2.384185791015625e-7	6.633793216577798e20	1.477262019281705e22	-7.103002143431434e-10	7.441219187366409e9
80	-2.1238178231559282e-15	1.7881393432617188e-7	6.63379323760251e20	1.4772620190609633e22	-7.1709924763639e-10	1.2075524782441132e10
n=16,	err_u: 0.0007415233680147283,	err_h: 0.0003681302039168138

Since they there is not obvious bias in the vorticity conservation errors, and they are the same magnitude in both cases, we are probably OK, though I will continue to monitor these vorticity conservation errors for more advanced tests...
Are there any more outstanding issues I should address?
Thanks again.

Hi @davelee2804,
nice work!
If I understand correctly the results in the tables (and as you say), it seems both quantitatively and qualitatively that both approaches achieve the desired outcome right? It is surprising that the vorticity becomes exactly 0.0 in some steps (happens in both cases).
If you still feel that a single experiment is still not enough, we may want to systematize experiments with the help of DrWatson.jl as well. I expect this work to be also necessary for the future, to compare systematically different algorithms and parameter values (mesh resolution, time step, FE order, geometrical order, etc.). I could help with this.
In regards to the PR, I will start now a new round of revision, with additional comments.
Thanks for your work!

Hi @davelee2804 ... I am going out for a walk. I will continue later adding more comments to my revision.

Comment out temporarily Manifest.toml from .gitignore and commit and push Manifest.toml. Note that we are temporarily pointing to a particular branch in Gridap.jl github repo. (while these devs are not released, currently under revision). Thus, this is needed to let the tests pass in Github CI actions. By the way, I would stop using [ci skip] for a while in this PR so that we double check that the tests pass in Github CI actions. We want to merge if and only if the tests are passing there.

@davelee2804 ... FYI ... tests are failing in this branch at Github Actions. I am positive it is related to this:
https://github.com/gridapapps/GridapGeosciences.jl/pull/15#issuecomment-913085099
Also take into account the modified signatures for assemble_vector! and assemble_matrix_and_vector! signatures.

Hey @amartinhuertas , hopefully I've addressed all the issues you raised (except for the ones I have already commented on). The CI test is failing at:
https://github.com/gridapapps/GridapGeosciences.jl/blob/shallow_water_explicit/src/ShallowWaterExplicit.jl#L27
but presumably this is because the CI testing is referencing the older version of assemble_vector!() with the different argument ordering.
Please let me know if I've missed anything, and thanks again for the super helpful and educational review!

Hey @amartinhuertas , hopefully I've addressed all the issues you raised (except for the ones I have already commented on). The CI test is failing at:
https://github.com/gridapapps/GridapGeosciences.jl/blob/shallow_water_explicit/src/ShallowWaterExplicit.jl#L27
but presumably this is because the CI testing is referencing the older version of assemble_vector!() with the different argument ordering.
Please let me know if I've missed anything, and thanks again for the super helpful and educational review!

I will fix the failing test, no worries.
On the other hand, have you addressed this?
https://github.com/gridapapps/GridapGeosciences.jl/pull/15#discussion_r700965121
Finally, I would give a shot to splitting your long functions into subfunctions. I think having functions that only do one thing is essential. If you do not find sufficiently confident how to do it, I can do it. Let me know.

Hey @amartinhuertas , I don't know how to split up the explicit function. I know it is long, and as a two stage function is make sense to split it. However each of the two stages is different and relies on a different about of data (the first stage relies on a single time level of data, the second stage on two).
If you have any idea on how to split it I'm happy to try, but there is no obvious way to me (It is not like this is straight Runge-Kutta where you do the same thing on different data at each stage)

Hey @amartinhuertas , I don't know how to split up the explicit function. I know it is long, and as a two stage function is make sense to split it. However each of the two stages is different and relies on a different amount of data (the first stage relies on a single time level of data, the second stage on two).
If you have any idea on how to split it I'm happy to try, but there is no obvious way to me (It is not like this is straight Runge-Kutta where you do the same thing on different data at each stage)

You can see the result of the work here: https://github.com/gridapapps/GridapGeosciences.jl/blob/60313734c8d369d57fa2803681e88cff492ef83d/src/ShallowWaterExplicit.jl#L68
Let me know if that makes sense.
Please note that the function has been split into many different reusable functions which only do one thing.  I guess that these functions might be used for other solvers as well. Also note that it was impossible to reuse code from the previous function without replicating it.

Thanks for all your help here @amartinhuertas , I've learnt a lot! I made one final change before merging, the L2 errors were different in the last couple of decimals (perhaps the order of operations changed on something). Nothing big enough to trigger a test failure (set at 1.0e-12), and perhaps we are not bit-4-bit on different machines anyway, but just for consistency.

Hey @amartinhuertas , the merge just failed. Is this related to the commit you just made?

Hey @amartinhuertas , the merge just failed. Is this related to the commit you just made?

I do not see that it failed (still on the way). Can you send the link to Github Actions of the failed job?

I made one final change before merging, the L2 errors were different in the last couple of decimals (perhaps the order of operations changed on something). Nothing big enough to trigger a test failure (set at 1.0e-12), and perhaps we are not bit-4-bit on different machines anyway, but just for consistency.

FYI ... Julia offers the built-in isapprox function to facilitate the implementation of this kind of floating point equality tests.

Hmm, I can't see it in the actions, and the error message is no longer here, I'm confused.

Merged now :)

Note that the order of the arguments has changed for some functions ... When you pull from master into the other brnaches this will affect you.

OK, thanks I'll try and keep that in mind


Mac and Linux builds.

Can you please make builds that are compatible with Mac and Linux operating system’s please?

I don't have machines for them and also don't have enough space on my PC for installing VMs for them


WSL Support?

Is it possible to add WSL support to fastfetch? Most things work out of the box and that's great, but things like GPU names do not display correctly (since WSL uses a virtual GPU I guess...). Just to cite an example, screenfetch displays these things correctly in WSL (it uses a different technique to display GPU names). Also, themes, icons, cursors - these don't really have a meaning in raw WSL, yet - since with WSLg, you can run graphical applications, the theming is almost non-existent by straightforward means. Also, on my device, the screen refresh rate shows incorrectly (though I suspect this might be something to do with WSL itself).

That could turn out to be a major maintaining burden but right now on the readme it says that it only supports linux and android.

I just found the time to install windows, to test it in WSL, but i would say, it works pretty great:

GPU names: fastfetch displays the GPUs, as they are exposed to WSL, There is no way to get a "better" name. Screenfetch just doesn't display them at all for me, neofetch displays them even worse.
Theme, Icons, Font, Cursor: in my installation it fails to detect them, because no package beside DEs should install theme configurations. But if they are configured, it makes sense to display them, as they are actually used to theme WSLg applications, as far as i can tell. I think that if somebody configured them, there is a good chance that this person will use WSLg.
The screen refresh rates seem to be limited to 60Hz for WSL, there is nothing i can do about it. Maybe don't display it in WSL sessions?

What i will update, is that "on WSL" is added to the OS output.
Besides that, can you please be more specific about what to change?

Thanks for making the time to test it out!
In my case, screenfetch correctly displays the GPU name: NVIDIA GeForce RTX 3050 Ti Laptop GPU whereas fastfetch/neofetch shows: Microsoft Corporation Device 008e. That's probably what libpci outputs - when I checked the code of screenfetch I found that it first makes queries for manufacturer-specific cards (using nvidia-smi for NVIDIA cards, for instance, but that I guess won't work if nvidia-smi is not installed). If it doesn't find any, then it falls back to libpci.
The other points you made, make sense. I understand the picture. As I said, most of the things already work since it's Linux anyway - indicating that it's running on WSL or another operating system would be nice... and if something could be done about the GPU thing. Other than that, even I don't have anything else in mind rn.

I just looked up nvidia-smi and screenfetch source code, and this seems to be something that only works on nvidia. But i can't really implement it, since i only have an AMD GPU.
I greped through the whole file system, but i didn't found any match with my GPU names, so there is a high chance that there is no generic solution.

Hi, does this neofetch issue discussion: https://github.com/dylanaraps/neofetch/issues/1646 help in some way? It uses glxinfo and different users seem to have tried it out for both AMD and NVIDIA. I checked on my system, and the NVIDIA solution shows "D3D12 (NVIDIA GeForce RTX 3050 Ti Laptop GPU)"

TBH i don't want to use GL / GLX / EGL etc for gpu detection, because it has the problem of only supporting one GPU. For example i have an integrated GPU and an external GPU, and glxinfo always shows only one GPU, which one depends on what the DRI_PRIME env variable is set to.
I already have an libpci fallback using vulkan, but currently vulkan seems to be software only in WSL. OpenCL, which also supports multi gpu, just doesn't seem to be supported at all.
As a site note, i just pushed some improvements for WSL, can you please test the following?

Host should show Windows Subsytem for Linux
WM should show WSLg (Wayland), if WSLg is running
Cursor shouldn't show anymore, because WSLg uses the native windows cursors

Host should show Windows Subsytem for Linux
WM should show WSLg (Wayland), if WSLg is running

Thanks, tested it out, works fine!

Cursor shouldn't show anymore, because WSLg uses the native windows cursors

This is not true, I believe. I use WSLg and it shows Ubuntu cursors (this has been a point of complaint in the WSLg repo, as far as I remember - you cannot customise these much either).

TBH i don't want to use GL / GLX / EGL etc for gpu detection, because it has the problem of only supporting one GPU. For example i have an integrated GPU and an external GPU, and glxinfo always shows only one GPU, which one depends on what the DRI_PRIME env variable is set to.

I totally get your point, makes sense. However, I'd like to comment that showing two "Microsoft Corporation Device 008" is more unhelpful than just displaying one GPU. Don't you agree?

I'd argue that too, but it is difficult to implement, because I would need a way to tell "this pci provided GPU name is not good enough, lets detect with OpenGL". Writing an OpenGL implementation to be solely used on WSL is honestly not worth the time.

I'd argue that too, but it is difficult to implement, because I would need a way to tell "this pci provided GPU name is not good enough, lets detect with OpenGL". Writing an OpenGL implementation to be solely used on WSL is honestly not worth the time.

I get your point. By the way - since WSL always will produce un-useful output using libpci for the GPU (feel free to correct me based on a major use-case) can you not just use OpenGL if you detect WSL?
(However: WSL has good(?) NVIDIA GPU support and if you have multiple of them, nvidia-settings is the way to go anyway. )


nixos/jellyfin: add data dir options

Description of changes
add dataDir and cacheDir as options to the module for jellyfin
dataDir is useful for jellyfin as theres no current way to manage jellyfin config with nix, the management of its state should be easy.
this should close #141367
cacheDir is included because it was mentioned in the relavent issue.
they both default to existing behaviour so this should not impact current users.
systemd tmpfile rules are used to ensure the paths exist with the right permissions so the previously related entries in the service config are removed

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[X] Fits CONTRIBUTING.md.

I have tested this and am currently running it on two Jellyfin setups. One on bare metal and one inside a NixOS Container. Everything has worked as expected.

Closing in favor of #277220


Make IDAKLU output variables compatible with Experiments

This was initially reported as a bug (see report below) but it turns out it is not an implemented feature yet. It seems the main issue is that when a new experiment step is initialised with the previous step solution, that one is empty so the relevant variables are not available. I believe that amending the last_state so it stores the whole state vector even when output variables are used should fix this.
PyBaMM Version
develop
Python Version
3.11.6
Describe the bug
I am running a model with the IDAKLUSolver and some output_variables. These include, amongst others, the "Discharge capacity [A.h]", however PyBaMM complains that that variable is not found.
Steps to Reproduce
import pybamm

pybamm.set_logging_level("NOTICE")

model = pybamm.lithium_ion.DFN()

output_variables = ["Discharge capacity [A.h]", "Time [s]", "Current [A]", "Voltage [V]"]
solver = pybamm.IDAKLUSolver(output_variables=output_variables)

experiment = pybamm.Experiment(
    [
        (
            "Charge at 1C until 4.2 V",
            "Hold at 4.2 V until C/50",
            "Rest for 1 hour",
        )
    ]
)

sim = pybamm.Simulation(
    model,
    experiment=experiment,
    solver=solver,
)

sol = sim.solve()

Relevant log output
Traceback (most recent call last):
  File "/home/brosaplanella/PyBaMM/src/pybamm/models/base_model.py", line 808, in set_initial_conditions_from
    final_state = solution[var.name]
                  ~~~~~~~~^^^^^^^^^^
  File "/home/brosaplanella/PyBaMM/src/pybamm/solvers/solution.py", line 547, in __getitem__
    self.update(key)
  File "/home/brosaplanella/PyBaMM/src/pybamm/solvers/solution.py", line 481, in update
    raise KeyError(
KeyError: "Cannot process variable 'Discharge capacity [A.h]' as it was not part of the solve. Please re-run the solve with `output_variables` set to include this variable."

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/brosaplanella/battery_modelling/degradation-data/examples/mwe.py", line 29, in <module>
    sol = sim.solve()
          ^^^^^^^^^^^
  File "/home/brosaplanella/PyBaMM/src/pybamm/simulation.py", line 702, in solve
    step_solution = solver.step(
                    ^^^^^^^^^^^^
  File "/home/brosaplanella/PyBaMM/src/pybamm/solvers/base_solver.py", line 1222, in step
    _, concatenated_initial_conditions = model.set_initial_conditions_from(
                                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/brosaplanella/PyBaMM/src/pybamm/models/base_model.py", line 810, in set_initial_conditions_from
    raise pybamm.ModelError(
pybamm.expression_tree.exceptions.ModelError: To update a model from a solution, each variable in model.initial_conditions must appear in the solution with the same key as the variable name. In the solution provided, 'Cannot process variable 'Discharge capacity [A.h]' as it was not part of the solve. Please re-run the solve with `output_variables` set to include this variable.' was not found.

I think we cannot yet combine output_variables with experiment arguments (I don't see any tests that cover this use case). @pipliggins, @BradyPlanden, do you know if this might be the reason?

Yes, I noticed that too. I will edit the issue to reflect that.

As already mentioned, this is a known problem - it was discussed in the last developer meeting I believe, and in #4300. I'm happy to take this one as my next issue.

Thanks! Sorry, I checked for an existing issue for this, as I remembered talking about it, but missed the comment in #4300.

Hi i tried running your code @brosaplanella
setup is wsl ubuntu, pyamm Version: 24.9.0, python 3.10.12
my output is much different?
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[177], [line 26](vscode-notebook-cell:?execution_count=177&line=26)
     [10](vscode-notebook-cell:?execution_count=177&line=10) experiment = pybamm.Experiment(
     [11](vscode-notebook-cell:?execution_count=177&line=11)     [
     [12](vscode-notebook-cell:?execution_count=177&line=12)         (
   (...)
     [17](vscode-notebook-cell:?execution_count=177&line=17)     ]
     [18](vscode-notebook-cell:?execution_count=177&line=18) )
     [20](vscode-notebook-cell:?execution_count=177&line=20) sim = pybamm.Simulation(
     [21](vscode-notebook-cell:?execution_count=177&line=21)     model,
     [22](vscode-notebook-cell:?execution_count=177&line=22)     experiment=experiment,
     [23](vscode-notebook-cell:?execution_count=177&line=23)     solver=solver,
     [24](vscode-notebook-cell:?execution_count=177&line=24) )
---> [26](vscode-notebook-cell:?execution_count=177&line=26) sol = sim.solve()

ValueError: y is too short, so value with slice is smaller than expected

I have encountered similar behaviour when trying to use output variable as a passable parameter into the IDAKLU Solver in general. I'm not sure if this is a package thing but if anyone knows why it does this it would be greatly appreciated

Hi @MarkV-ADI - are you using the developer install for pybamm? If so, you'll need to re-run nox -s dev to get the IDAKLU solver to recompile. That should get you the model error from running this example, and should allow you to use output_variables outside Experiments.

Having looked into this, there seem to be 2 separate, but related, issues. They both stem from the fact that the full state vector for all time-steps is not returned when output_variables are specified for the IDAKLU solver.

The full state vector for the final time-step is required from the end of one step to initialise the start of the next one
The FSV for the first timestep is required to fill Solution.first_state

1 is easily solved because we already return the final FSV from the solver to calculate events, it just needs pulling from Solution.y_event - as mentioned in the issue report.
2 looks like it has 2 possible solutions. I think first_state is only used for the calculation of summary variables? So:
A: We edit the IDAKLU solver again to return the first state vector, as well as the last SV and the subset specified in output_variables. This would add a bit of extra memory load back in which the output_variables option was supposed to reduce, but not loads.
B: Summary variables don’t get calculated when output variables are specified (as presumably only the requested subset are wanted anyway).

If I understand correctly, the last state of cycle N is identical to the first state of cycle N+1. Therefore, if we are already returning last states, we could just point the first states to them. The only that won't be caught is the initial state (i.e. first cycle first state), but that's from the initial conditions and I think we store it anyway.
Would that automatically fix the issue?

If I understand correctly, the last state of cycle N is identical to the first state of cycle N+1. Therefore, if we are already returning last states, we could just point the first states to them. The only that won't be caught is the initial state (i.e. first cycle first state), but that's from the initial conditions and I think we store it anyway.
Would that automatically fix the issue?

I think the only first state we actually use is to calculate the 'Change in x' summary variables, which would require the initial state. I thought I'd looked at the initial conditions vs first timestep and found them to be slightly different, but I'll double check. The fix I've implemented so far just issues a warning that the 'Change in x' summary variables won't be calculated, everything else works as expected.


Streaming requests end with DecodingError

Sample code (adapted from the example given):
var openAI = SwiftOpenAI(apiKey: "sk-...")

for try await newMessage in try! await openAI.createChatCompletionsStream(model: .gpt3_5(.turbo),
                                                                          messages: [.init(text: "Hello, how are you?", role: "user")],
                                                                          optionalParameters: .init(stream: true))
{
	dump(newMessage)
	let newToken = newMessage.choices.first?.delta?.content
	print("New Message Received: \(newToken) ")
}

While the code works and does stream all responses properly, it always fails with:
Swift/ErrorType.swift:200: Fatal error: Error raised at top level: Swift.DecodingError.dataCorrupted(Swift.DecodingError.Context(codingPath: [], debugDescription: "The given data was not valid JSON.", underlyingError: Optional(Error Domain=NSCocoaErrorDomain Code=3840 "Invalid value around line 1, column 1." UserInfo={NSDebugDescription=Invalid value around line 1, column 1., NSJSONSerializationErrorIndex=1})))

Wrapping the calls in do {.. } catch DecodingError.dataCorrupted(let context)  works fine however, but should be handled  by your framework.

Thank you @randomsnowflake! It should work now, as I have been introducing some updates to avoid the issues that you are facing (more context). Now that I have better control over what is happening, this error should have disappeared with the new release 0.4.0
{"id":"chatcmpl-6zpBNtBaNTSRAen7AUt7bk1N0uDt8","object":"chat.completion.chunk","created":1680192773,"model":"gpt-3.5-turbo-0301","choices":[{"delta":{"content":"?"},"index":0,"finish_reason":null}]}
{"id":"chatcmpl-6zpBNtBaNTSRAen7AUt7bk1N0uDt8","object":"chat.completion.chunk","created":1680192773,"model":"gpt-3.5-turbo-0301","choices":[{"delta":{},"index":0,"finish_reason":"stop"}]}
[DONE]
Finished streaming messages

Please, if you find any other issues, I will be happy to fix them :)


Statsd <IP_ADDRESS>

upstream PR  for  statsd <IP_ADDRESS>

Hi @mpereira  is there any update on PR https://github.com/Automattic/elasticsearch-statsd-plugin/pull/29 when will this merge to master

Hi @shekhar8617,
This PR is not running any of the unit tests. Travis CI Logs

Release for ES 6.3.1 is available now.


Add structure for default arguments

Add DeclSort::DefaultArgument to represent default arguments -- they are notionally declarations subject to the ODR, for all practical purposes.

@GabrielDosReis please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

Contributor License Agreement

@microsoft-github-policy-service agree [company="{Microsoft}"]


Debugging through IIS

VSCode Version: 1.6
OS Version: Windows 7

Steps to Reproduce:

Create a Asp.Net MVC or WebForms App.
Try to attach Visual Studio Code to IIS for debugging purposes.

Don't see any real option to debug...

This issue should be filed to the vscode c# extension. Can you please file it there?
https://github.com/OmniSharp/omnisharp-vscode
Thanks you!


Cassandra 3.11.4 doesn't startup with AdoptOpenJDK build 25.222-b10

Browser:

My Java version is the following
openjdk version "1.8.0_222"
OpenJDK Runtime Environment (AdoptOpenJDK)(build 1.8.0_222-b10)
OpenJDK 64-Bit Server VM (AdoptOpenJDK)(build 25.222-b10, mixed mode)
I am in the process of evaluating AdoptOpenJdk so that my company moves away from Oracle's Jdk. All my java apps have been working fine but Cassandra 3.11.4 has been giving me issues during startup.
Issues

The jna-4.2.2.jar which comes bundled doesn't creates the dll. When its upgraded to jna-4.5.2.jar then the jna works fine.
On startup, the JVM crashes with the following error

# JRE version: OpenJDK Runtime Environment (8.0_222-b10) (build 1.8.0_222-b10)
# Java VM: OpenJDK 64-Bit Server VM (25.222-b10 mixed mode windows-amd64 compressed oops)
# Problematic frame:
# C  [sigar-amd64-winnt.dll+0x14ed4]

I do understand that its outside the JVM but as soon as I shift to the following Oracle's jdk, cassandra starts up pretty fine without issues and I am able to do all the functionalities.
java version "1.8.0_221"
Java(TM) SE Runtime Environment (build 1.8.0_221-b11)
Java HotSpot(TM) 64-Bit Server VM (build 25.221-b11, mixed mode)
I am on hold for the migration due to this issue.

Are you able to try the Upstream OpenJDK? adoptopenjdk.net/upstream.html
Let's see if this is a broad OpenJDK issue or just Adopt's

Hello Martijn,
Based on your suggestion, I did try the following Upstream OpenJdk
openjdk version "1.8.0_222"
OpenJDK Runtime Environment (build 1.8.0_222-10)
OpenJDK 64-Bit Server VM (build 25.222-b10, mixed mode)
To be very specific, I set the JAVA_HOME within the start script so that cassandra uses the appropriate jdk and validated it from the cassandra logs as well. Cassandra started up pretty fine and worked pretty well. So it seems that its mainly an issue with AdoptOpenJdk only.
Thanks,
Nitish

Thanks that'll help a little.
@ali-ince Can I get you to look at the DLL etc versions between the two variants and see where we might differ?

Hello @karianna ,
Are there any updates for this issue as it's halting my migration efforts from Oracle Jdk to AdoptJdk?
Thanks,
Nitish

Hi @NitishBangera,
Could you please try the resolution mentioned at https://github.com/AdoptOpenJDK/openjdk-build/issues/1126#issuecomment-512574015? Please consider the architecture of your operating system while picking the version of redistributable to install.
Thanks.

Hello @ali-ince,
Thank you for the update. I will be trying your suggestion on windows 10 pro as well as the server os which is Centos 7 and updating this thread.
Thanks,
Nitish

Apologies for the delay as I haven't been able to test the suggestion. I will be updating soon regarding it.

Hi
I have the same issue here. I am working on migration from Oracle Jdk to AdoptJdk and wanna get rid of all the VS2010 dependencies. Therefore, the solution @ali-ince mentioned does not work for me.
Java version:
adoptOpenJDK 8u222
Cassandra:
tried 3.11.2 and 3.11.4, but both failed.

Hello @ali-ince, @karianna
I would like to echo what @amanda-lee-ansys has mentioned as well. Even I do not want to depend on VS2010 dependencies. Can you please share some other suggestions to solve this issue?

@karianna @ali-ince - any suggestions for the VS2010 dependencies problem we are encountering?  VS2010 extended support ends in July 2020.

@ali-ince @karianna Can you please share some updates?

I would expect that we will be moving up to VS2017 for the JDK8 builds at some point int he next couple of releases.
@ali-ince @karianna anything to add?


Added Scene Recognition Notebook

Thank you for adding this tutorial to application model zoo. The effort put in for the tutorial is greatly appreciated by our organization.

This is a wonderful dataset to work with!
But I’m not able to understand how this classifier is trained from scratch since I can notice only default values.
Can anyone clarify this please?

Here training from scratch is referring to that only...it is confusing so will change that.


[Bug]:  net::ERR_ABORTED when request responds with 204 status

Bug description
Steps to reproduce the problem:
For ease, test with a server you control. Otherwise test with a URL that you know will respond with 204 status.
Go to a URL on that server.
await args.page.goto('http://localhost:8090/test');
On the server at localhost:8090 make /test responds with 204 status.
Puppeteer version
19.2.2
Node.js version
16.16.0
npm version
8.15.0
What operating system are you seeing the problem on?
Linux
Configuration file
No response
Relevant log output
ERROR [ExceptionsHandler] net::ERR_ABORTED at http://localhost:8090/test
Error: net::ERR_ABORTED at http://localhost:8090/test
    at navigate (/projects/x/node_modules/puppeteer-core/src/common/Frame.ts:346:13)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at Frame.goto (/projects/x/node_modules/puppeteer-core/src/common/Frame.ts:310:17)
    at CDPPage.goto (/projects/x/node_modules/puppeteer-core/src/common/Page.ts:1523:12)
 at myFunc (/projects/x/src/script.ts:251:27)

Could you provide a minimal, complete and verifiable example in a separate repository?

await args.page.goto('http://localhost:8090/test');

https://github.com/dan-scott-dev/puppeteer-9458

Sure, here you are - https://github.com/dan-scott-dev/puppeteer-9458

@OrKoN maybe we can add this error to the checking implemented in #9523, since it seems that is how chromium handles it as well. What do you think?

@Rafael-Martins I am not sure, we need to see what exactly happens here. Does the navigation still happens? if not, then we should report an error and fix it upstream.

Oh, yeah, totally makes sense, I will do some testings if I have some time!

Hi, I have exactly the same scenario! Any news about this?


App Store Rejection for Photo Library Access

When submitting to the App Store Apple reject with the following (iOS 10):
"This app attempts to access privacy-sensitive data without a usage description. The app's Info.plist must contain an NSPhotoLibraryUsageDescription key with a string value explaining to the user how the app uses this data."
I created a sample app with only the Card.io library and received the same rejection. I have added the NSCameraUsageDescription key as documented and do not get a warning related to this. Does this library also require NSPhotoLibraryUsageDescription? I can't see any access to the photo library within the source however is it mandatory as part of accessing the camera? If so the readme should be update to document this.

👍   it appears that the NSPhotoLibraryUsageDescription is also required.

This issue is caused by UIImagePickerController usage here – https://github.com/card-io/card.io-iOS-source/blob/9ace76161daf8df88be77f5dc6a3f72816f6d00e/Classes/CardIODevice.m#L52-L57


Custom FUSE interface

Our current usage of simple_httpfs is not robust enough to handle most of our use cases. This leads to us relying on our mechanisms for entirely downloading remote files locally far more than we should which can be incredibly inefficient.
We need to implement our own FUSE interface that can do the following:

Mount files in arbitrary directories on the fly
Mount multiple files simultaneously next to each other (in the same directory) when their sources/types vary (e.g. S3FF, URL from S3, and URL from https)
Handle mounting files from S3FF

Related issues:

#546
#519
#290

We need to check this out as a possible alternative to FUSE: https://github.com/RaRe-Technologies/smart_open


Model should not be needed to update assistant.

@louis030195
Currently, you need to pass in a model to update an assistant

This should fix it https://github.com/64bit/async-openai/pull/174

done


Automatically detect installations of virtual environments while focusing a file that depends on it

Is your feature request related to a problem? Please describe.
When you are editing a file where its virtual environment is not installed, and it gets installed outside of vs code, the extension doesn't automatically detect this, and you have to switch to a python file in another directory to trigger a redetection.
Describe the solution you'd like
The virtual environment installation for your currently focused file should be detected automatically so that no further manual action is required by the user.
Describe alternatives you've considered
Alternatively or additionally there could be an action to ask the extension to redetect the virtual environment.

Closing this issue since this project is being archived.


Use default indices_path path

Once I run rake chewy:reset it raised an error with:
TypeError: no implicit conversion of nil into String

Once I set indices_path in a chewy.yml file then this issue was solved.
Since this change now indices_path used by default, so you don't need to set it through chewy.yml.

Awesome, thanks!


Update the Calendar app for the SDK version 0.100 f7

Hello!
Your Calendar app version 1.0 fails to build with the SDK version 0.100.0-rc f7. Please update the app in your repository and create a new pull request to the catalog repository with the updated manifest file.

IMPORTANT: Don’t forget to update the app version and commit version in the manifest file.

View logs
Flipper Team

I am again here!
Your Calendar app version 1.0 fails to build with the SDK version 0.100.1-rc f7. Please update the app in your repository and create a new pull request to the catalog repository with the updated manifest file.

IMPORTANT: Don’t forget to update the app version and commit version in the manifest file.

View logs
Flipper Team

App is updated


ARROW-9238: [C++][CI][FlightRPC] increase test coverage of round-robin under IPC and Flight

This PR increase test coverage of round-robin under ipc and flight. Before this PR, round-robin tests for primitive data under ipc use only int32 (and boolean in some cases). This PR adds other primitive types (i.e. int8, uint8, int16, uint16, uint32, int64, uint64, float16, float32, and float64).

This PR is now ready for review. Would it be possible to take a look?
cc @pitrou @wesm

Are there any other points that I have to address?

+1. Will merge on green

@wesm Thanks for your update. Now, all tests passed.

+1, will merge.


Phoenix - Add recalculate TPM requirements

For myeloma specific TPM recalculation, is this implemented, what is the issue
Need a list of transcripts to remove

Put file in public location and now resource script pulls that file to the expected location


State type : printedFullSerialNumber unknown

Perhabs the same issue for me Adapter crash cause of corrupted jason while there are two devices in the account:


Could not find app.conf

I'm trying to transition from MacOS to Ubuntu and Keyd has been great so far.
I'm trying to add app-specific keybindings, however, when I run keyd-application-mapper I get the following error.
ERROR: could not find app.conf, make sure it is in ~/.config/keyd/app.conf
The file exists with the following content.
[google-chrome]

alt.[ = C-S-tab
alt.] = macro(C-tab)

I also have app.lock and app.log in the ~/.config/keyd directory. There is no output in the log file.
My default key mappings are working. I'm using Ubuntu 22.04.3 LTS (default Wayland).
Any suggestions on what I should try?

Did you install from source? Are you running keyd-application-mapper as root (you shouldn't be)?

Thank you for the fast reply. I was running it as root. I ran it without sudo and it returned "Gnome Detected". However, the app.conf defined google-chrome shortcuts are not working. I tried restarting chrome after running the mapper.
I ran sudo journalctl -eu keyd and received not a valid key error on line 22. I'm using the macos settings from the examples folder. The issue seems to be with leftmeta = alt.

leftmeta = alt is not a valid binding. alt is a layer leftalt is a key (which is bound to layer(alt) by default. As a separate point, google-chrome may or may not be a valid application class name, you can use keyd-application-mapper -v to print class names as you switch apps.

I ran keyd-application-mapper -v but I just get Gnome detected as the output. I checked the app.log file and it's empty.

Got it working! I was playing around with gnome-tweaks and while in the extensions realized that key-mapper had to be enabled.


Equivalent of @responsive

Hello I am trying to generate variants like sm: automatly, i.e. @reponsive in tailwind.
However it does not work neither in my windi project nor in the playground.
tailwind play
windi play
Is there anyway to archive it without listing all the breakpoints?

It seems like this features was deprecated with Tailwind v3. See here
The replacement should be @layer. See Windi Docs here
@layer utilities {
  /* YOUR STYLES */
}

Thanks for your response! In fact I have already read the doc and it seems that it is just no longer need but no removed from Tailwind v3 (working in the playground link of tailwind v3.0.14 above).
I have also tried @layer after reading that but it is not working. I have removed the example in the above playground since unlike tailwind, @variants is not mentioned in @layer of windi doc, so I though it maybe unrelated.
Could you kindly provide a working example? Is there anything that I have missed in the following playground?

The usage of @responsive is not possible. There are multiple other directives which can achieve the same behavior (to some extend). You can do two things:
Custom classes for selected sizes
@screen sm {
  .bg-vr-yel {
    @apply bg-yellow-300
  }
}

or
@variants sm, md {
  .bg-vr-yel {
    @apply bg-yellow-300
  }
}

Just use the color

or define a custom color like:
import { defineConfig } from 'windicss/helpers';

export default defineConfig({
    theme: {
        extend: {
            colors: {
                foo: '#000'
            }
        }
    }
});


Provide documentation for using rememberMe

I'm trying to get rememberMe to work, but it appears to be clearing my session (and therefore my auth storage, logging me out immediately).
rememberMe() calls setSessionCookieLifetime() which calls regenerateId() with true as a default ... true = $deleteOldSession ... is this the problem?
... or am I doing it wrong? I've tried:
Session\Container::getDefaultManager()->rememberMe();

(the default manager is set onBootstrap)
And:
$container = new Session\Container(\Zend\Authentication\Storage\Session::NAMESPACE_DEFAULT);
$container->getManager()->rememberMe();

This method is similar to [1], except that he authenticates after calling rememberMe, which means it most-likely works, but I don't want to change the cookie lifetime unless authentication is successful. Is this not possible?
Also, it should be possible to access the session or session manager from the session auth storage. I've seen one guy new up a session manager just to do it (idk if that works) [2], and another guy who was forced to extend the auth storage so that he could access its manager [3]. How is this supposed to be done?
[1] http://zend-framework-community.634137.n4.nabble.com/ZF2-combine-authentication-with-remember-me-function-td4656575.html
[2] https://github.com/wingman007/fmi/blob/master/module/AuthDoctrine/src/AuthDoctrine/Controller/IndexController.php
[3] http://samsonasik.wordpress.com/2012/10/23/zend-framework-2-create-login-authentication-using-authenticationservice-with-rememberme/

This issue has been closed as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html


[WIP]storage: optimize check leader flow, using raft check first to get valid leaders

What is changed and how it works?

Issue Number: Close #xxx
What's Changed:
Try to optimize the traffic cost of the stale read.

Use resolve_by_raft to get valid leaders first, if most of the leaders have valid raft leases the leader check could be served locally.
The check leader requests are only used to replicate the resolved-ts to followers if the related leaders are known to be valid.
In this case, most of the raft-related fields could be saved as described in https://github.com/tikv/rfcs/pull/108.
The hibernate region feature would be deprecated in the future so resolve_by_raft is not a problem anymore.

Related changes
Check List
Tests

Unit test

Side effects
Release note
None

/release


v5.8.0-rc

[5.8.0-rc] - ESignature API v2.1-<IP_ADDRESS> - 2021-09-23
Changed

Added support for version v2.1-<IP_ADDRESS> of the DocuSign ESignature API.
Updated the SDK release version.

Fixed

Fixed file upload ability to AccountBrands::updateResource (DCM-3867).

AppVeyor build and test pass.


When no credentials are available, firestore call never returns.

I expect it to return an error code, but instead the call never returns.  Here's the code.
import admin from 'firebase-admin';

    admin.initializeApp({
      credential: admin.credential.applicationDefault(),
    });

    await db.collection('hello').doc('world').get(),

Hey @SurferJeffAtGoogle
Thanks for the report. I tried to reproduce with below snippet but failed.

import admin from 'firebase-admin';

const app = admin.initializeApp({
  credential: admin.credential.applicationDefault(),
});

async function test() {
  const db = app.firestore();
	const snap = await db.collection('hello').doc('world').get();
	console.log(`${snap.data()}`);
}

test();

Can you provide a bit more info to reproduce this?

@wu-hui how did it fail? It wasn't top-level await missing was it? I would expect test() to be awaited?

It failed to reproduce the hang. test() runs and returns, with undefined printed to the console.
@SurferJeffAtGoogle mentioned in our chat that this happens in a test in Github Actions, so this might be something else. I have not tried yet.

I have a work-around.  It may be the environment the github action runs in.  What does the client do if the server accepts the connection but never sends bytes on the socket?

To reproduce, try instantiating the client in a docker container with the network turned off.

With the script:
const admin = require('firebase-admin');

const app = admin.initializeApp({
  credential: admin.credential.applicationDefault(),
});
async function test() {
  const db = app.firestore();
  const snap = await db.collection('hello').doc('world').get();
  console.log(`${snap.data()}`);
}
test();

With Wifi on it immediately prints undefined. With Wifi off it hangs for a while and then throws the error Error: 14 UNAVAILABLE: Name resolution failed for target dns:firestore.googleapis.com:443 after 41 seconds for me.


How can i get yaw angle with respect to north?

Can anybody help me to receive the global yaw angle with respect to north ?
i can get lat, lon, alt data using ros.
But to control the drone outdoor, i need heading anlge.
i can get the yaw angle from odometry, but it is not enough to guide it to the specific geodetic position.
There are compass and magnetometer sensor in bebop2 drone, but i don't know how to access them.

See the documentation for attitude:
Ardrone3PilotingStateAttitudeChanged
Yaw should be heading in global frame. Please check and comment with result.

@thomas-bamford Thank you!. i can get global yaw data from Ardrone3PilotingStateAttitudeChanged.


[crypto] Fix outdated cryptolib docstrings.

These comments refer to the old method of representing hardware-backed keys (which used a field in the key configuration instead of the keyblob). This was changed in https://github.com/lowRISC/opentitan/pull/19714
Instead, adds guidance that directs callers to otcrypto_hw_backed_key for initializing these keys.
Thanks to @vsukhoml for finding this mistake! See https://github.com/lowRISC/opentitan/issues/21936#issuecomment-1989531303

lgtm! thanks for fixing.


kenh14

Checklist

[X] I'm reporting a new site support request
[X] I've verified that I'm running yt-dlp version 2022.05.18 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that none of provided URLs violate any copyrights or contain any DRM to the best of my knowledge
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and am willing to share it if required

Region
Vietnam
Example URLs
Video: https://video.kenh14.vn/video-mo-hop-iphone-14-pro-max-nguon-unbox-therapy-316173.chn
Playlist: https://video.kenh14.vn/playlist/tran-tinh-naked-love-mua-2-71.chn
Description
Video
Verbose log
[debug] Command-line config: ['-vU', 'https://video.kenh14.vn/video-mo-hop-iphone-14-pro-max-nguon-unbox-therapy-316173.chn']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.05.18 [b14d523] (win_exe)
[debug] Python version 3.8.10 (CPython 64bit) - Windows-10-10.0.19042-SP0
[debug] Checking exe version: ffprobe -bsfs
[debug] Checking exe version: avprobe -bsfs
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: avconv -bsfs
[debug] exe versions: none
[debug] Optional libraries: Cryptodome-3.14.1, brotli-1.0.9, certifi-2021.10.08, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
Latest version: 2022.05.18, Current version: 2022.05.18
yt-dlp is up to date (2022.05.18)
[debug] [generic] Extracting URL: https://video.kenh14.vn/video-mo-hop-iphone-14-pro-max-nguon-unbox-therapy-316173.chn
[generic] video-mo-hop-iphone-14-pro-max-nguon-unbox-therapy-316173: Requesting header
WARNING: [generic] Falling back on generic information extractor.
[generic] video-mo-hop-iphone-14-pro-max-nguon-unbox-therapy-316173: Downloading webpage
[generic] video-mo-hop-iphone-14-pro-max-nguon-unbox-therapy-316173: Extracting information
[debug] Looking for video embeds
[debug] Identified a JW Player JS loader
[debug] Default format spec: best/bestvideo+bestaudio
[info] P3Q5XYacz5E-1652782450168278389637: Downloading 1 format(s): 0
[debug] Invoking http downloader on "https://hls.mediacdn.vn//203336854389633024/2022/5/17/P3Q5XYacz5E-1652782450168278389637.mp4"
ERROR: unable to download video data: HTTP Error 404: NOT FOUND
Traceback (most recent call last):
  File "yt_dlp\YoutubeDL.py", line 3130, in process_info
  File "yt_dlp\YoutubeDL.py", line 2838, in dl
  File "yt_dlp\downloader\common.py", line 445, in download
  File "yt_dlp\downloader\http.py", line 373, in real_download
  File "yt_dlp\downloader\http.py", line 130, in establish_connection
  File "yt_dlp\YoutubeDL.py", line 3596, in urlopen
  File "urllib\request.py", line 531, in open
  File "urllib\request.py", line 640, in http_response
  File "urllib\request.py", line 569, in error
  File "urllib\request.py", line 502, in _call_chain
  File "urllib\request.py", line 649, in http_error_default
urllib.error.HTTPError: HTTP Error 404: NOT FOUND

Note to Dev:
The video file is in div ... data-vid-id=<video_id> data-vid=<video_url>, the <video_url> need to prepended with httpsand then it will download the video correctly. More video can obtained inhttps://<video_url>.json`.


Issue 168 deal properly with download of data

PR checklist

[x] This comment contains a description of changes (with reason).
[x] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[x] Usage Documentation in docs/usage.md is updated.
[x] README.md is updated (including new tool citations and authors/contributors).

Adjusted three subworkflows (Eggnog, EUKulele, Kofam) to not download databases when a valid database is provided by the user with command line arguments. Documentation updated to indicate this ability. Working off of this issue: https://github.com/nf-core/metatdenovo/issues/168

Hello and well done!
For me, eggnog and kofamscan are perfect as they are now. The only comment is about eukulele. Last week we decided to skip eukulele_download module as it doesn't caching correctly. When´-resume´ the pipeline, the analysis will start again and we want avoid this kind of behavior.
If you look in PR #190  you can see what we decided. Eukulele_search is able to check the presence of the directory and the files by its own so we can skip the checks that you improved in the subworkflow.
What I will suggest are two options:

Just remove the changes from eukulele subworkflow (+ in the main workflow for eukulele) then we can merge the two PRs.
You can move the changes from #190 in this PR and then we can merge it.

Thanks a lot for your contribution!

Hi @tfalkarkea  since you have not time to contribute, I took the lead and clear the PR so we can merge it. I decided to go for the second option and I removed the two-steps module for eukulele. Thanks for you help. We will try to find a way to test the pipeline on Tower in a second moment (probably in two weeks at the hackathon!).
Cheers!


[WIP] Update win cases

Signed-off-by: Hang Yan<EMAIL_ADDRESS>

Codecov Report

Merging #3198 (ec55039) into main (2ee6ad1) will decrease coverage by 20.23%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #3198       +/-   ##
===========================================
- Coverage   60.59%   40.36%   -20.24%
===========================================
  Files         297      169      -128
  Lines       25434    21394     -4040
===========================================
- Hits        15411     8635     -6776
- Misses       8345    11918     +3573
+ Partials     1678      841      -837

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
40.36% <ø> (+0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.


Restrict file type

Is there a way to restrict the allowed filenames, like *.mp3 *.ogg *.mkv?

Yeah, use the fileType option when setting up Resumable.

It's not on http://www.resumablejs.com/

Start with https://github.com/23/resumable.js/.

Ok, thanks. It's outdated ?


Jiayi

FILL IN THE PR DESCRIPTION HERE
FIX #xxxx (link existing issues this PR will resolve)
BEFORE SUBMITTING, PLEASE READ THE CHECKLIST BELOW AND FILL IN THE DESCRIPTION ABOVE

 PR Checklist (Click to Expand)
Thank you for your contribution to vLLM! Before submitting the pull request, please ensure the PR meets the following criteria. This helps vLLM maintain the code quality and improve the efficiency of the review process.
PR Title and Classification
Only specific types of PRs will be reviewed. The PR title is prefixed appropriately to indicate the type of change. Please use one of the following:

[Bugfix] for bug fixes.
[CI/Build] for build or continuous integration improvements.
[Doc] for documentation fixes and improvements.
[Model] for adding a new model or improving an existing model. Model name should appear in the title.
[Frontend] For changes on the vLLM frontend (e.g., OpenAI API server, LLM class, etc.)
[Kernel] for changes affecting CUDA kernels or other compute kernels.
[Core] for changes in the core vLLM logic (e.g., LLMEngine, AsyncLLMEngine, Scheduler, etc.)
[Hardware][Vendor] for hardware-specific changes. Vendor name should appear in the prefix (e.g., [Hardware][AMD]).
[Misc] for PRs that do not fit the above categories. Please use this sparingly.

Note: If the PR spans more than one category, please include all relevant prefixes.
Code Quality
The PR need to meet the following code quality standards:

We adhere to Google Python style guide and Google C++ style guide.
Pass all linter checks. Please use format.sh to format your code.
The code need to be well-documented to ensure future contributors can easily understand the code.
Include sufficient tests to ensure the project to stay correct and robust. This includes both unit tests and integration tests.
Please add documentation to docs/source/ if the PR modifies the user-facing behaviors of vLLM. It helps vLLM user understand and utilize the new features or changes.

Notes for Large Changes
Please keep the changes as concise as possible. For major architectural changes (>500 LOC excluding kernel/data/config/test), we would expect a GitHub issue (RFC) discussing the technical design and justification. Otherwise, we will tag it with rfc-required and might not go through the PR.
What to Expect for the Reviews
The goal of the vLLM team is to be a transparent reviewing machine. We would like to make the review process transparent and efficient and make sure no contributor feel confused or frustrated. However, the vLLM team is small, so we need to prioritize some PRs over others. Here is what you can expect from the review process:

 After the PR is submitted, the PR will be assigned to a reviewer. Every reviewer will pick up the PRs based on their expertise and availability.
 After the PR is assigned, the reviewer will provide status update every 2-3 days. If the PR is not reviewed within 7 days, please feel free to ping the reviewer or the vLLM team.
 After the review, the reviewer will put an  action-required label on the PR if there are changes required. The contributor should address the comments and ping the reviewer to re-review the PR.
 Please respond to all comments within a reasonable time frame. If a comment isn't clear or you disagree with a suggestion, feel free to ask for clarification or discuss the suggestion.

Thank You
 Finally, thank you for taking the time to read these guidelines and for your interest in contributing to vLLM. Your contributions make vLLM a great tool for everyone!

@Hanchenli hi, what is this PR about?

@Hanchenli Is this PR related to https://arxiv.org/abs/2310.07240 Happy to chat offline about your plan!

Hi Zhuohan and Youkai, this is a pull request by mistake. We are actually working on some projects on VLLM but we are still building it on our own repo... I accidentally clicked the pull request button one night...I am closing this and hope to talk with you when our project is finished...


Hidden distances

Hey @tomlandia nice work with the v2.0. I notice you are busy with bigger issues but I was hoping there could be a way to map out "approximate" locations of profiles with hidden distances? I know it would be hard for accurate readings but maybe it would be possible to pinpoint a radius using other profiles distances?

hmm. speaking of distances here - where is the 'fix' for the distance shown? for example in km, my closest contact shows as 855km away. this is of course wrong. i recall seeing some correction for this?

@evergreen3313 did you download the latest version off http://fuckr.me after the distances were fixed that has been implemented into the release.  The work I was doing was basically testing and trouble shooting which at one point did not have the distance fixed.

yep. using that one. giving up for now.

Perhaps if you've used different versions there might be a conflict.  Maybe you should try clearing your AppData folder and try again.  c:\Users%username%\AppData\Local\Fuckr

While using this thread... ahm..
I noticed that now there are less profiles showing. Away less profiles in this version.
Is there a technical reason behind it?
I think it is not limited by distance but my number of profiles that can show.


Custom web + WOPI example broken?

Describe the bug
I've tried following https://owncloud.dev/ocis/deployment/ocis_wopi/ to change my changes from https://github.com/owncloud/web/pull/8447, but when linking my own /web/dist seems to mess with the registration of appProvider app actions?
Expected behavior
Guide should work flawlessly ™
Actual behavior

If I just start the vanilla docker-compose.yml from latest master, I can create e.g. OpenOffice files
If I link my custom /web/dist/, I see my local web changes but clicking the Create + button doesn't offer me any appProvider-actions (like creating OpenOffice files).

Additional context
Pulled fresh images and deleted volumes before starting docker. I could try and debug this myself, but maybe I'm missing something obvious/undocumented (like linking/modifying a config file)?

Following a discussion with @kulmann that may be a Mac M1 + docker issue (on Docker Desktop v4.15-19)

Following a discussion with @kulmann that may be a Mac M1 + docker issue (on Docker Desktop v4.1{5-9})

Yes, I had the same issues. Would be great if someone with a linux system could check as well before any other debugging takes place.

@micbar just an idea to think about, maybe not related to this issue but could be:
Atm we have deployment examples with a particualr state in various tags. What we dont have is a kind of release notes file in the main directory where we just summarize changes made from one to another version. What about creating one which would ease finding differences? Note that these examples are not only available for devs but also linked by the admin docs.

@pascalwengerter Which version are you using?
Do you have my changes includedhttps://github.com/owncloud/ocis/commit/a86aba1c4144d54d80fd956ddcb9139518cbfc04 ?

Can you check what the /app/list request returns?

I'll close here since I retried this morning (updated oCIS and docker-for-mac before), and if you wait long enough the /app/list endpoint does return the extensions...it just takes quite a while, which I didn't expect


completion.bash and key-bindings.bash bug

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[x] bash
[ ] zsh
[ ] fish

Problem / Steps to reproduce
Operating system: Arch Linux
Pacman was used to install fzf, latest version
I first tried to source the 2 files:
Input:
source /usr/share/fzf/completion.bash
Output:
bash: /usr/share/fzf/completion.bash: line 183: syntax error near unexpected token `done'
bash: /usr/share/fzf/completion.bash: line 183: `    done'

Input:
source /usr/share/fzf/key-bindings.bash
Output:
bash: /usr/share/fzf/key-bindings.bash: line 23: syntax error near unexpected token `done'
bash: /usr/share/fzf/key-bindings.bash: line 23: `  done'

I can't find any documentation online to get shell integration working and I have been searching for 4 hours now. Can't find anything so I assume this is a bug. Can someone guide me in the right direction?

I don't use ArchLinux, so I have no idea how they packaged fzf. First, you need to git clone this repository and see if the files are identical to the ones in shell/ directory from this repo.

And set -x will help you understand what's going on under the hood.
set -x; source /usr/share/fzf/key-bindings.bash; set +x

Yes I should have mentioned that. I git cloned this repository and got the exact same error. Thanks for the quick response.

Input:
set -x; source /usr/share/fzf/key-bindings.bash; set +x
Output:
+ source /usr/share/fzf/key-bindings.bash
bash: /usr/share/fzf/key-bindings.bash: line 23: syntax error near unexpected token `done'
bash: /usr/share/fzf/key-bindings.bash: line 23: `  done'

The syntax of the file is perfectly fine. I suspect you have strange aliases (such as for, do, done, etc, which clash with bash builtins) that interfere with the processing.

Related: https://stackoverflow.com/questions/22969050/bash-syntax-error-near-unexpected-token-done-with-alias-named-do

also if your wondering arch packages in var/cache/pacman/pkg is where source files are if you can seem to find an alias for done make one
'''bash
alias done='\done'
'''
the slash in front of the done will prevent it from aliasing out to something else. unalias wont work as if your not setting it you probably don't know when its set but probably in your /etc/profile.d would be the first place i look. so if you unalias done and it doesn't work it means its being set after your aliases which means you need to alias it to \done. but the bindings used in the scripts are more or less readline functions so you might want to try updating your readline. But being on the main arch repo means that the install scripts have been verified and work or they get pulled (generally speaking) but you may also want to check your settings in /etc/pacman.conf

Wow! Thanks for the help. You knew exactly what the problem was.
I had this line in my bashrc:
alias do='xterm -hold -e vim ~/todo'

Yeah, that's a horrible idea 😄


Optimized model seems slower than original

I have tested examples/mnist.py in a docker container on a PC with RTX 4090.
Here is what I found:

MNIST
Accuracy
.pt file size
VRAM occupied (may not be accurate)
avg. inference time

Original
98.75
4.6 MB
492 MB
0.4 ms

Optimized by MS-AMP
98.76
2.4 MB
512 MB
1.2 ms

I also found that each layer in original model has data type "torch.float32", and mixed "torch.float32" and "torch.float16" in optimized model. I guess that's because pytorch has not supported float8 yet.
However, any idea why the optimized model takes longer time for inference?

Hi @BitCircuit , thanks for your attention to our work!
In MS-AMP, the additional operations for FP8 quantization and de-quantization take up part of time.
When training a small model, the speedup of FP8 calculation cannot make up for the cost caused by quantization.
In my experience, FP8 takes less time when training a model whose parameter size is larger than 1B.

Thank you for your reply. I thought MS-AMP was able to train the model as in float8 weight format directly. From your response, I assumed that MS-AMP trains the model as QAT (with QDQ nodes inserted)?

In my experience, FP8 takes less time when training a model whose parameter size is larger than 1B.

After second thought, I realized that MS-AMP is a tool to optimize training speed instead of inferencing speed. Is that right?

@BitCircuit Yes. MS-AMP is an FP8 mixed-precision training framework, optimizing the memory usage and training speed.


avoid sending duplicate MIDI notes

When using MIDI plugin in Note mode to output channel data from QLC, notes are sent twice.
This patch removes a superfluous call of snd_seq_event_output(m_alsa, &ev); that causes this misbehaviour.

Hi, this sounds strange to me.
I haven't checked yet, but are you sure the two calls were because of Note On/Note Off?
Cause if that's the case IIRC that was a wanted behaviour.
How did you test this? With a button? Slider?

I came across this when setting up to control our lightboard from QLC+ via MIDI. it behaved weirdly, jumping 2 cues whenever I pressed the prepared scene button. I also tested with a slider and in simple desk. then I connected kmidimon to see what is going on, and saw two identical midi notes arriving. To cross-check, I set it to CC mode and it didn't happen there. Today I removed the line in code, recompiled and that duplicate note was gone, and CC mode still working. I am in theater now, and with my modified version of QLC+ the console responds correctly.

I just tested in Ubuntu and QLC+ 4.12.5-GIT compiled this morning. In CC
mode works as expected with Note mode QLC+ sends Note_On or Note_Off twice.
See attached screen capture.
Missatge de Edgar Aichinger @.***> del dia dj., 14
d’abr. 2022 a les 18:22:

I came across this when setting up to control our lightboard from QLC+ via
MIDI. it behaved weirdly, jumping 2 cues whenever I pressed the prepared
scene button. I also tested with a slider and in simple desk. then I
connected kmidimon to see what is going on, and saw two identical midi
notes arriving. To cross-check, I set it to CC mode and it didn't happen
there. Today I removed the line in code, recompiled and that duplicate note
was gone, and CC mode still working. I am in theater now, and with my
modified version of QLC+ the console responds correctly.
—
Reply to this email directly, view it on GitHub
https://github.com/mcallegari/qlcplus/pull/1326#issuecomment-1099367280,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABO5H44GZYENVPTLWB2PXVLVFBA3TANCNFSM5TM6IXHA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I'm confident that my change is correct - see line 174 where snd_seq_event_output() finally is called for all if/else branches before draining the buffer.

Alright, this looks good to me.
Thanks :+1:


luci-base: update Polish translation

Updated Polish translations.
@jow- @musashino205
I have a question to you. In commit https://github.com/openwrt/luci/commit/86660f92d12b963a05118a8f9471513a18773be4 and https://github.com/openwrt/luci/commit/524ce90c4e22dcb64ac8e6c2e012eb54cd5f098e, you have added "Alias interface" and "Alias Interface". Should there be almost identical expressions here?
Signed-off-by: Krystian Kozak<EMAIL_ADDRESS>

@Rixerx - good point, will look into merging those two.


Support Dogecoin

After hearing to potential users, probably this has no demand.
We should place more priority on liquid support instead.


Add Mastering REST APIs in Node.js: Zero-To-Hero

Hi,
I am Branko and I work for Manning Publications.
I though this video course of ours might be a good contribution to your list.
Thanks for considering it!

ipcenas:master

Hi,
I am Branko and I work for Manning Publications.
I though this video course of ours might be a good contribution to your list.
Thanks for considering it!

ipcenas:master


Connexion to localhost without http specified

Hello,
I'm trying Bruno v1.1.1 and I found out that specifying http:// is mandatory. I wanted to know if this was intended because it is no mandatory in postman or insomnia.
I created simple API to try Bruno that I run locally on the port 8080 and here is what I get:

GET http://localhost:8080/ works fine
GET http://<IP_ADDRESS>:8080/ works fine
GET localhost:8080/ throws an error: Error invoking remote method 'send-http-request': AxiosError: Unsupported protocol localhost:
GET <IP_ADDRESS>:8080/ throws an error: Error invoking remote method 'send-http-request': Error: connect ECONNREFUSED <IP_ADDRESS>:80

Thanks for your work anyway, this is a really cool software

It looks like Postman is prepending http:// in case the protocol is not provided and it does not matter if hostname is localhost or any other.

Indeed. Wouldn't it be a nice feature to add to bruno? It happens quite often that I'm too lazy or forget to write that http://.

How to use inherit auth from parent in buron, I tried Basic Auth and Digest auth , getting below error
{
"errors": [
{
"code": "Global.Internal.Error",
"description": "Unspecified Error on the system which prevented execution.",
"message": "The requested version is not supported."
}
]
}


[PE-D][Tester E] Datetime Expiring

Did a good job checking for interview datetime to be older than the application datetime. A suggestion would be that if the datetime expires (maybe after a given period), the status should change to make it more intuitive for the user.
As shown in the screenshot, the interview datetime is 31-03-2023 14:35 but the current datetime is already 31-03-2023 14:36.

Labels: type.FeatureFlaw severity.Low
original: pzaiming/ped#3

Mentioned this in Planned Enhancement section of DG


May 3rd 2023 - Diversity Equity & Inclusion Special Interest Group Meeting

Date
May 3rd 2023 - 12:00pm EST / 5pm BST
Untracked attendees

Fullname, Affiliation, (optional) GitHub username
...

Meeting notices

FINOS Project leads are responsible for observing the FINOS guidelines for running project meetings. Project maintainers can find additional resources in the FINOS Maintainers Cheatsheet.

All participants in FINOS project meetings are subject to the LF Antitrust Policy, the FINOS Community Code of Conduct and all other FINOS policies.

FINOS meetings involve participation by industry competitors, and it is the intention of FINOS and the Linux Foundation to conduct all of its activities in accordance with applicable antitrust and competition laws. It is therefore extremely important that attendees adhere to meeting agendas, and be aware of, and not participate in, any activities that are prohibited under applicable US state, federal or foreign antitrust and competition laws. Please contact<EMAIL_ADDRESS>with any questions.

FINOS project meetings may be recorded for use solely by the FINOS team for administration purposes. In very limited instances, and with explicit approval, recordings may be made more widely available.

Agenda

[ ] Convene & roll call (5mins)
[ ] Display FINOS Antitrust Policy summary slide
[ ] Review Meeting Notices (see above)
[ ] Approve past meeting minutes
[ ] Big Boost Mondays- Zngly and FINOS Update

Retrospective

London - April 24th at Red Badger
Big Boost Mondays at Red Badger, London

Future Events

Dublin Monday 15th May

Microsoft and Technology University of Dublin - Student Expo
Big Boost Mondays at Microsoft, Dublin

Belfast Monday 22nd May

Synechron and Ulster University - Student Expo
Big Boost Mondays at Synechron, Belfast

Supporters so far ...

Citi, Code First Girls, Coding Grace, Deutsche Bank, Dogpatch Labs, EPAM, FINOS, gitHub, Goldman Sachs, Google, ING, LTSB, Microsoft, Mission Beyond, Morgan Stanley, NatWest, PwC, Red Badger, Red Hat, Rich Mix, Split, St Hilda's, Synechron, Tech She Can, Technology University Dublin, Turntable, Ulster University, Zngly

FINOS board update and DEI SIG at FINOS Member Meeting, London
How to fulfil DEI SIG 2023 Goals

[ ] AOB, Q&A & Adjourn (5mins)

Decisions Made

[ ] Decision 1
[ ] Decision 2
[ ] ...

Action Items

[ ] Action 1
[ ] Action 2
[ ] ...

Meeting Info
Join Zoom Meeting
https://zoom.us/j/98492160513
Meeting ID: 984 9216 0513
Passcode: 691166

Jevon Beckles - Scott Logic

Oscar Wilson, Zngly

Mike Wilson, Zngly

Beeke-Marie Nelke, GS

Closed as part of DEI SIG project board grooming


Update site for VS Code release 1.1.0

@stevedlawrence Would you mind giving this a look over? I was waiting to send out the email of the 1.1.0 release until the website was updated to have 1.1.0 visible.


Proposal to make MatOption (its value) typed

Feature Description
Make MatOption class typed
Use Case
When using MatOption the value is typed any, would be great if it would be possible to make MatOption type specific, could by default be falling back to any:
Proposed solution:
export class MatOption<T = any> implements FocusableOption, AfterViewChecked, OnDestroy {

  //...

  @Input() value: T;

  //...

}

Could provide a PR if that is preferred.

I actually have this on our backlog for all valued components (radio, select, etc).

@jelbourn Sounds good!
Any idea on when will it move from the backlog to a sprint? Typed components would be very useful for checking application/code consistency.

It's hard to say. It's currently number 23 on the backlog behind the remaining work for MDC-based components and test harnesses, but the MDC work is pretty large in scope since it involves updating a lot of Google code internally. We might be able to accept PRs if the changes are strictly non-breaking.


Email Js 404 API not found error

Issue Title
EmailJS API Not Working on Production Website's Contact Form
Description
I am encountering an issue with the EmailJS API on our production website's contact form. The API is not functioning as expected and is returning a 404 error when we attempt to send the form.
Steps to Reproduce

Navigate to https://www.healthoptima.in/
Fill out the contact form and attempt to send it

Expected Behavior
The EmailJS API should function correctly returning a prompt telling that the form was sent successfully.
Actual Behavior
The API returns a 404 error when attempting to send the form from the production website, it prompts there was an error encountered. However, it works perfectly when tested on localhost.
Additional Information

The service in use is EmailJS. You can visit the documentation at https://www.emailjs.com/docs/sdk/send/
The file for the contact form can be found under src/home/contact.tsx
Here is a screenshot of the issue:

This discrepancy between the production and development environments is causing significant problems and must be addressed immediately.
Any assistance in resolving this issue would be greatly appreciated.

This error can be caused by either you have not mentioned your template id and service id in the production or they might be incorrect. Which hosting service you are using? If you are using environment variables try updating them.

This error can be caused by either you have not mentioned your template id and service id in the production or they might be incorrect. Which hosting service you are using? If you are using environment variables try updating them.

Exactly, check your template ID in your code.


Create the members "store" page

This can be hardcoded html/js page
It will listen to the ‘message’ event on the window.
It will communicate with layer0 of the members SDK.
It will handle fetching of tokens. from the /secure/token endpoint
Must be served on /secure to have access to cookie

Moving issue to a note


can you horizontally align the "view toolkit" buttons?

Can you draw or sketch what you have in mind? It's probably doable.

I just mean so that the grey boxes line up with with red line. I think
right now the middle one is forcing the box lower because that's how the
text broke--but perhaps it doesn't really matter since the whole thing
sizes dynamically, huh?
[image: Inline image 1]
On Fri, Jan 2, 2015 at 5:04 PM, Billy Meinke<EMAIL_ADDRESS>wrote:

Can you draw or sketch what you have in mind? It's probably doable.
—
Reply to this email directly or view it on GitHub
https://github.com/billymeinke/cc-toolkits/issues/2#issuecomment-68577390
.

--
Timothy Vollmer
Public Policy Manager, Creative Commons
Get Creative Commons Updates http://bit.ly/commonsnews


Fix for defined CLI entrypoints

Fixes #82

Codecov Report

Merging #83 (7e5e796) into main (5caadeb) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main      #83   +/-   ##
=======================================
  Coverage   30.69%   30.69%
=======================================
  Files          25       25
  Lines        2926     2926
=======================================
  Hits          898      898
  Misses       2028     2028

Impacted Files
Coverage Δ

ensembleperturbation/client/make_storm_ensemble.py
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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