remote ollama server

Validations

[ ] I believe this is a way to improve. I'll try to join the Continue Discord for questions
[ ] I'm not able to find an open issue that requests the same enhancement

Problem
The current version only supports local ollama, and the ollama server with the specified URL cannot be configured.
Solution
Provide a configuration that can be used to set up the Ollama server and customize the prompt.

You can specify a remote server with the apiBase property, like so:
    {
      "title": "Qwen Coder 2.5 14B",
      "model": "qwen2.5-coder:14b",
      "provider": "ollama",
      "apiBase": "http://<IP_ADDRESS>:11434"
    }

You can specify a remote server with the apiBase property, like so:
    {
      "title": "Qwen Coder 2.5 14B",
      "model": "qwen2.5-coder:14b",
      "provider": "ollama",
      "apiBase": "http://<IP_ADDRESS>:11434"
    }

@StefanB7 Thanks, awesome!


add endCompilationUnit callback for BugChecker

This PR adds a new interface BugChecker.CompilationUnitTreeEndMatcher that can be implemented by Error Prone checkers.  Its endCompilationUnit method gets invoked after the entire tree of a compilation unit has been visited, and it can return a Set<Description> to report errors over the file.
We've found this functionality to be quite useful in a couple of internal checkers.  If this functionality were available in open-source Error Prone we would probably also update NullAway to use it.  I've included a sample UnusedPrivateFields check to show how it could be useful.  This check is primarily included for exemplary purposes; we haven't enabled this check at the moment.
As to why endCompilationUnit but not all the other tree nodes: it seemed like the minimal API change to enable the functionality we wanted.  Also, compilation units are not nested; an endClass callback (for example) might be more error prone in the presence of inner / anonymous classes.

@nglorioso and I discussed this, and we're not comfortable with adding mutable state to BugCheckers.  If you want to collect state from a whole compilation unit, you could implement a TreeScanner and then scan the compilation unit from matchCompilationUnit().  There's an example of this in MockitoCast: https://github.com/google/error-prone/blob/master/core/src/main/java/com/google/errorprone/bugpatterns/MockitoCast.java#L100

@eaftan thanks for the feedback and the pointer to the MockitoCast check.  I didn't know about some of the VisitorState APIs used there, pretty handy.  Our main concern about the TreeScanner approach was performance.  If we have several checks each doing their own AST traversals, we thought that could eventually add up to a significant performance overhead compared to piggy-packing on Error Prone's AST traversal.  I don't have numbers to back this up, though.  Do you have any thoughts on whether this is a real concern or not?

@cushon I'm tinkering around with the TaskListener approach.  From what I see, for the check to observe the finish of ANALYZE events it needs to be registered as a listener before the ANALYZE phase starts to run.  Further, its listener should run after the ErrorProneAnalyzer listener.  I don't see any obvious way for an Error Prone plugin check to do accomplish this based on current APIs.  I don't think it'll work to try to report errors at the start of GENERATE, and even if it does, it seems gross.  Any suggestions?

I haven't yet figured out a way to get the TaskListener approach to work.  Another thought / proposal here.  Rather than having a fairly unrestricted CompilationUnitTreeEndMatcher interface, we could have an abstract class WholeCompilationUnitChecker that relevant checks need to extend.   WholeCompilationUnitChecker could then explicitly manage the lifecycle of any relevant (mutable) state via a CompilationUnitState interface.  Subclasses would be required to define a method CompilationUnitState create(), and WholeCompilationUnitChecker would be responsible for calling create() at the beginning of a compilation unit and nulling out the pointer at the end.  This would strongly encourage limiting the scope of mutable state.  It would not completely prevent checks from keeping mutable state elsewhere, but currently Error Prone doesn't do that anyway.
@eaftan @nglorioso if the above approach sounds more promising let me know and I can update the code in this PR for further discussion.  We have some other checks we'd like to upstream, but we are hesitant to adopt the pattern of every check doing its own AST traversal due to performance.

@eaftan @nglorioso any update / thoughts on the above?  We have some other checks we'd like to open source / upstream that are blocked on getting a resolution here.

Can you try the TreeScanner/matchCompilationUnit approach and see if the performance is acceptable?  We would really like to avoid mutable state in our BugCheckers, and we would want convincing numbers before going down that path.

Ok, finally had some time to dig into performance.  I wrote a simple Error Prone check that just does a configurable number of AST traversals; code is here.  I then compiled a large module (~2300 Java source files) running only this check, and varying the number of additional traversals it does.  Results (measured on my laptop in a fairly non-rigorous way):

# Traversals
Running Time

0
28s

10
28.4s

100
28.8s

250
32.5s

500
36.6s

1,000
45.7s

2,000
64.5s

Any difference less than 2 seconds is in the noise for these runs.  So, it looks like there needs to be more than 100 extra traversals going on to add anything measurable to compile time on my machine.  At the larger number of passes, the overhead starts looking more linear, and each pass is adding somewhere between 0.015 and 0.02 seconds of compile time (again, for ~2300 source files).  I'm not going to speculate on percentage overheads as that will depend a lot on annotation processors, etc.
Bottom line, I think the cost of extra traversals is low, but in the medium to long term maybe not a cost that should be completely ignored.  As #994 shows, these costs can add up.  That said, there is also a maintenance / readability cost to writing checks with mutable state and I can understand the desire to not encourage it.  I'll close this PR for now, but we may want to revisit down the line.


problem installing Devspace on Mac.  get Cannot find module '/usr/local/index.js'

What happened?

installed DevSpace. Seemed to complete successfully.  But then when I typed 'devspace help', it showed an error (Error: Cannot find module '/usr/local/index.js') and then started displaying 'Finishing installation for DevSpace CLI' again and again. This happens if I type 'devspace version' as well.

➜  learn_k8s npm install -g devspace
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
/usr/local/bin/devspace -> /usr/local/lib/node_modules/devspace/bin/devspace

><EMAIL_ADDRESS>postinstall /usr/local/lib/node_modules/devspace
> node index.js install

+<EMAIL_ADDRESS>added 94 packages from 94 contributors in 12.995s
➜  learn_k8s devspace help
Finishing installation for DevSpace CLI
internal/modules/cjs/loader.js:979
  throw err;
  ^

Error: Cannot find module '/usr/local/index.js'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:976:15)
    at Function.Module._load (internal/modules/cjs/loader.js:859:27)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:71:12)
    at internal/main/run_main_module.js:17:47 {
  code: 'MODULE_NOT_FOUND',
  requireStack: []
}
Finishing installation for DevSpace CLI
Finishing installation for DevSpace CLI
Finishing installation for DevSpace CLI
Finishing installation for DevSpace CLI
Finishing installation for DevSpace CLI

What did you expect to happen instead?

show help message

How can we reproduce the bug? (as minimally and precisely as possible)
Local Environment:

DevSpace Version: [use devspace --version]
can't tell as g
Operating System: windows | linux | Mac
Mac 10.15.4. Node v13.12.0
Deployment method: helm | kubectl apply

Kubernetes Cluster:

Cloud Provider: google | aws | azure | DevSpace Cloud | DevSpace Cloud Enterprise | other
Kubernetes Version: [use kubectl version]

Anything else we need to know?

/kind bug

@anuaimi Thanks for reporting this. I fixed the issue and released a new version.


Fix update metadata on old uncommitted / delete / GC race

#8253 describes a very rare race.  It will be hard even to reproduce it on purpose, but a race is a race.
Start with an uncommitted but old object (e.g., upload an object to a branch and wait for a fortnight without committing it).  Now perform these 3 operations concurrently: update the object metadata, delete the object, and quickly run GC.

Thread 1 gets object metadata to update its user metadata
Thread 2 deletes the object
👉🏽  Thread 3 performs UGC and cleans up the physical object
Thread 1 writes a new entry for the object with the same physical address

Result: Update metadata succeeds, leaving an object with the updated metadata but with no physical object to back it up.  GETting that object will return 410 Gone.  There is no way to sequence "update object metadata" and "delete object" to get this behaviour, so this is a bug.
It is also probably impossible to get this result: thread 1 will need to sleep for close to an entire GC cycle.
Nonetheless, a bug is a bug.

I will replace the existing 2-call solution with a single API call.  That API call will perform:

Get object metadata from staging and keep its predicate, or from committed and a "does not exist" predicate
Update object metadata to have the new desired user metadata
SetIf object metadata
under protection of a regular staging token check.

This will be safe: the staging token check prevents branch HEAD from moving, and the SetIf makes the update atomic.


remove duplicate IHttpRequester register

Fixes
Proposed Changes
IHttpRequester registered twice in OcelotBuilder

see details:

https://github.com/ThreeMammals/Ocelot/blob/develop/src/Ocelot/DependencyInjection/OcelotBuilder.cs#L101
https://github.com/ThreeMammals/Ocelot/blob/develop/src/Ocelot/DependencyInjection/OcelotBuilder.cs#L110

What do you think guys? @TomPallister  @philproctor
Im a bit confused with the breaking tests always, can't see if there are a breaking change or the tests are failing like before


Target 3.11-rc.1 in nightly python build

I see where you're going with that. I'll add it and merge this.


[Event Request] Table 5080 "To-do" - SetDuration

Can you please add following parameters in SetDuration procedure,
OnBeforeSetDuration(Rec, IsHandled); // old

OnBeforeSetDuration(Rec, EndingDate, EndingTime, IsHandled); //new

    [IntegrationEvent(false, false)]
    local procedure OnBeforeSetDuration(var Task: Record "To-do"; EndingDate: Date; EndingTime: Time; var IsHandled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 46955.


Work around window position not always being restored properly

This fixes an issue where the window position isn't always restored properly. For example, I'm running Linux and have a panel at the top of my primary monitor, but not on my secondary. If I move yakyak to the top of my secondary monitor, the next time I open it up, it won't be restored at the top of the secondary monitor, but will instead be a few pixels down from the top, corresponding to the bottom of the top bar on my primary monitor.
The fix actually allows the system to handle the positioning, so the part where it checks if it's within the bounds of the monitors and such isn't necessary anymore.
I haven't noticed any issues myself, and it seems to work pretty much identical to how it did before, without the overhead of the manual checking.
Do let me know if you want any changes to be done, or if more testing is required.

I've just tested this under Ubuntu 20.04 (gnome 3.34.3) in a single monitor environment.
Test:

Position Yakyak window in top right corner
Quit Yakyak
Start again

Expected: Window in same position
What I got: Seemed like window was moved a bit down (distance may be the titlebar height)

You're right, seems like there's a bug on linux with electron that causes it to move down. Have a look here: https://github.com/electron/electron/issues/10388


Allow assigning animation resource to AnimationNodeAnimation

Godot version:
master / 0ffec7daf
OS/device including version:
Manjaro Linux 17.1
Issue description:
Since the old AnimationTreePlayer.AnimationNodeSetAnimation accepts an Animation resource as its argument, it was possible to use the API to override the default animation with a custom animation which wasn't added to the player.
However, the new AnimationNodeAnimation only accepts a string argument to change its animation, forcing the user to add the animation to the AnimationPlayer (which does not seem to work at this moment, as described in #19876).
Such a feature could be helpful, when the game can be extended by many different items with custom animations, for example. So, each item node can reference an Animation resource which can be assigned to the same one shot node at runtime.

I don't mean to be impatient, but is there a chance this may get fixed soon? It's a blocker issue for my use case, so I'd like to know if I better change my design or work on something else until it gets fixed.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


[FEATURE] Support Data Engine Option For Backing Image in CSI flow

Is your feature request related to a problem? Please describe (👍 if you like this request)
StorageClass should support Data Engine Option for Backing Image
Describe the solution you'd like
BackingImage in the storageclass should follow the same data engine as volume
I propose that we don't need another option for backing image data engine but reuse the volume data engine.
Describe alternatives you've considered
.
Additional context
cc @derekbit @WebberHuang1118

Verified on master-head 20241231

longhorn master-head https://github.com/longhorn/longhorn/commit/7da75642805760929883ff2100ee42c58f117d4c
longhorn-manager master-head https://github.com/longhorn/longhorn-manager/commit/1875421555b48dd71782afe8e70f45bcaf6441a2

Verified on v1.8.0-rc2 20241231

longhorn v1.8.0-rc2 https://github.com/longhorn/longhorn/commit/45d4e8029f686a0a5ffc8a1a68c42361b528dcae
longhorn-manager v1.8.0-rc2 https://github.com/longhorn/longhorn-manager/commit/37ad98fb5d33e8a907c4e1770933e95a862ab6d9

The test steps
https://github.com/longhorn/longhorn/issues/10084#issuecomment-2563475774
Result Passed


[spirv-reduce] Potential reduction opportunities

I recently tried to reducer.  It did a very good job of removing code inside the functions.  However, it left a really big header.  In the zip file, I have the original spv, the result of the reducer, and the full hand reduced spir-v assembly:  t.tar.gz
The steps that I had to do by hand were:

[ ] Remove unused input and output variables %2 %3 %4 %5 %gl_VertexIndex %gl_InstanceIndex.
[ ] Remove unused constants. %uint_2147483647 and others are not used.
[ ] Remove unsued types like %_ptr_Input__arr_v4float_uint_3 and some of the other pointer types.
[ ] Merge types (that is replace all floats by ints, or all ints by bools) to reduce the number of types used.
[ ] Remove unused functions.  (%88 is never called)
[ ] Remove unused parameters from functions.  %92 is not used.  Need to change the call sights and function types.
[ ] Remove unused return values from functions. %86 is never used, so change the function %87 to have a void return type.  Lots of coordinated change need to happen to keep this correct.

If you modify the input and output variables, the shader won't be useable in the same pipelines as before. So if the goal is to run the reduced shader in a pipeline, be careful applying that reduction.


functional: shorten interval for replace serialization

We should shorten the interval between the old unit and new unit, so that across "fleetctl start --replace" the old unit's ExecStopPost doesn't overlap with the next unit's ExecStartPre directive. This workaround is necessary to (hopefully) avoid occasional conflicts like:
  unit_action_test.go:711: Did not find 1 unit in cluster, unit replace
  failed: failed to find 1 active units within 15s (last found: 0)

Although this change doesn't completely solve underlying issues between systemd and fleet, it can reduce the number of failures.

This change indeed seems to bring some improvements. TestReplaceSerialization survived 6 test cycles without failure. I'll keep testing it until next week.

Ok, with this PR, TestReplaceSerialization fails twice during 20 test cycles. (failure rate 10%) That's a huge improvement, in contrast to a previous failure rate of around 30% on semaphoreci. In case of no objection, I'm going to merge it tomorrow.


Update language-support-policy.md

The doc still mentions Azul as the JVM vendor which is no longer the case as we are now using MS OpenJDK. Updating this to point to https://learn.microsoft.com/en-us/azure/developer/java/fundamentals/java-support-on-azure. This also has information about Eclipse Temurin.
cc @JasonFreeberg @ahmedmahmoudMSFT

Learn Build status updates of commit 547c0f4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/azure-functions/language-support-policy.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Closing PR.


need to use the same varnishd and varnishtest as building against

configure prefers varnishd/varnishtest from $PATH over those from directories from the pkg-config, so libvmod-example may end up building against a different installation than the one used for make check
$ PATH=/usr/sbin:$PATH
$ PKG_CONFIG_PATH=/tmp/lib/pkgconfig ./configure
...
checking for varnishtest... /usr/bin/varnishtest
checking for varnishd... /usr/sbin/varnishd

but:
$ PKG_CONFIG_PATH=/tmp/lib/pkgconfig pkg-config --variable=sbindir varnishapi
/tmp/sbin
$ PKG_CONFIG_PATH=/tmp/lib/pkgconfig pkg-config --variable=bindir varnishapi
/tmp/bin
$ ls -als /tmp/sbin/varnishd /tmp/bin/varnishtest
 360 -rwxr-xr-x 1 slink slink  366600 Mar  3 17:45 /tmp/bin/varnishtest
2632 -rwxr-xr-x 1 slink slink 2692576 Mar  3 17:45 /tmp/sbin/varnishd

this can be worked around by passing the VARNISHD and VARNISHTEST variables to configure pointing to the right paths.
I will prepare a fix and issue a pull request.

Should be solved now that with @nigoroll 's pull requests are merged.


Player change data

Be able to show player change in rank and sigma after each tournament. End of row showing tournament should show that player's entry rank and sigma.

Search your player number (pin_player) in usgo_agagd.ratings and it will output pin_player, rating, sigma, elab_date(rated date), and tourmament_code. These can be used to display next to each tournament for players.
SELECT *
FROM  usgo_agagd.ratings
WHERE (Pin_Player LIKE  '%[memberID]%')

Information is displayed in a new set of columns displaying data from a place like https://agagd.usgo.org/ratings/12011/ to show the  rating and sigma post tournament.


init() missing 2 required positional arguments: 'configs' and 'tokenizer'

when run openspeech_cli.hydra_eval.py, load_from_checkpoint method report this error:

__init__() missing 2 required positional arguments: 'configs' and 'tokenizer'

I think it is caused by that the checkpoit doesn't pickle 'configs' and 'tokennizer' params in model class.

@hasangchun Can you check?

Thanks for letting me know. I'll fix it as soon as I have time.

Thanks for letting me know. I'll fix it as soon as I have time.

Thanks, the tokennizer seems can't be pickled. when I tried to add self.save_hyperparameters() in OpenspeechModel.__init()__, it reported an error.

I think you can override on_save_checkpoint or on_load_checkpoint like what I did to solve this bug. But it is not very elegant.

I have the same problem, and there  are  many problems need to  modify  when I use (configs=configs, ......,tokenizer=None) . Im not sure thats right, but no errors .

Any updates on this topic? Is inference still not supported?

Hi, I'm trying to load an already-trained model doing something like this:
model = MODEL_REGISTRY[configs.eval.model_name]
model = model.load_from_checkpoint(configs.eval.checkpoint_path, configs=configs, tokenizer=tokenizer)

However, I get this error (for all the layers):
RuntimeError: Error(s) in loading state_dict for ConformerLSTMModel:
	Unexpected key(s) in state_dict: "encoder.conv_subsample.conv.sequential.0.weight ...

Have you guys managed to load a pre-trained model @wuxiuzhi738  @yinruiqing ? Am I doing something wrong??
Thank you a lot

Just realized that the model didn't have those layers because the build_model() method was not being called. I solved it calling it in the __init__ and now it seems to work well.

Hi,
I made fixes based on the above tips and evaluation works, but I'm getting much worse results on the validation data set than during training on the same validation dataset. Is it using somehow ground truth during training validation evaluation that causes better results during training validation?


Invalid input: expected '[' --- Input: $Staples$

About You:
Name: z
Company:
Your question:
hi :
my tdvt cersion is    2.3.0
tebleau 2020.3
when i execute the tdvt, the exception as follow :
Exception processing 'C:\Programs\Python37-32\lib\site-packages\tdvt\exprtests/pretest/connection_tests/staples/setup.Query.Filter_Exclude1.xml'
Problem loading logical query XML:
Qualified Name Parse Error --- Invalid input: expected '[' --- Input: $Staples$
how can i specify this variable？
thank you.

Can you please attach the .ini file you used to configure TDVT? Thanks

Hi z, can you confirm this is still an issue? Otherwise I'll close the ticket. Thanks.

Hi z, can you confirm this is still an issue? Otherwise I'll close the ticket. Thanks.

Closed due to inactivity.


Travis CI using hadoop 2.7.7, spark 2.4.3

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9fb479a). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master     #21   +/-   ##
========================================
  Coverage          ?   34.6%
========================================
  Files             ?      15
  Lines             ?     763
  Branches          ?     161
========================================
  Hits              ?     264
  Misses            ?     499
  Partials          ?       0


Thin block fixes

Change the "SupportThinBlocks" criteria to avoid connecting to Bitcoin Core 0.12 nodes.
Second is to let ThinBlockWorker hold the "is re-requesting transactions" state, rather than CNodeState. This is how it was supposed to be originally, ThinBlockWorker knows when to reset it.

Added a commit to enable use of thin blocks by default.

With thin blocks enables, I'm getting a loop in getchaintips.py at line 46, self.join_network () where it connects two networks with different tips.
node1 log shows repeated block fetching.

All tests pass, with and without thin blocks enabled.


ROS compatibility issue

Hi there,
Just wondering if this project is particularly tied with ROS indigo or can be used in any later version of ROS, such as Kinetic?
I tried to run the simulation launch file (ros-kinetic) but encountered with such an error message:
'
ERROR: LoadError: Traceback (most recent call last):
File "/home/xxx/barc/workspace/src/barc/src/stateEstimator.py", line 854, in
main()
File "/home/xxx/barc/workspace/src/barc/src/stateEstimator.py", line 48, in main
Q_hs[0,0]  =  rospy.get_param("/state_estimator/Qx_hs") # 0.5     # x
File "/opt/ros/kinetic/lib/python2.7/dist-packages/rospy/client.py", line 465, in get_param
return _param_server[param_name] #MasterProxy does all the magic for us
File "/opt/ros/kinetic/lib/python2.7/dist-packages/rospy/msproxy.py", line 123, in getitem
raise KeyError(key)
KeyError: '/state_estimator/Qx_hs'
ArgumentError: Package RobotOS not found in current path:

Run Pkg.add("RobotOS") to install the RobotOS package.
'

Would you please help check if it is the ROS version issue?
Cheers,

Hello,
We do use ROS kinetic but I believe that error is tied with Julia. Try running the "Pkg.add(RobotOS)" command in Julia.

Hello,
We do use ROS kinetic but I believe that error is tied with Julia. Try running the "Pkg.add(RobotOS)" command in Julia.

Thanks very much, I'm quite new to Julia.
Another issue is about barc package in Julia. When I run Pkg.status(), it reports:
→ [e9bc4a31] barc v0.0.0 #LMPC-Shuqi (barc)
┌ Warning: Some packages (indicated with a red arrow) are not downloaded, use instantiate to instantiate the current environment
└ @ Pkg.Display /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.0/Pkg/src/Display.jl:190
I tried Pkg.instantiate():
julia> Pkg.instantiate()
Updating registry at ~/.julia/registries/General
Updating git-repo https://github.com/JuliaRegistries/General.git
also tried Pkg.build():
julia> Pkg.build("barc")
Updating registry at ~/.julia/registries/General
Updating git-repo https://github.com/JuliaRegistries/General.git
This issues still remains. Would you please help enlighten me on what possibly could go wrong?
Cheers,


fix(Key Window): #324 - make previous window key again after dismissal

Issue Link 🔗
#324
Goals 🥅
Implement a fix for the issue above
Implementation Details ✏
By relocating the dismiss handlers' invocation to a later point in time, the host app can now identify the main / rollback window as key.

On my project, this bug hasn't been fixed yet. Can you support me fix this bug? @huri000


 I saw this inverter grid tie with limiter that are perfect except for the fact that is wired limited an my inverter is away from the total of my grid. I have home assistant and shelly 3pm values can I use it to limit my inverter? Sorry I'm noob thank you

yes you can do it if your consumption sensor exposed to ha. you can put the sensor name in powermeter0 section in the code. make sure that the unit is in W. mine is in kW then i have to put a filter to multiply by 1000. then it works just fine.

Please keep in mind the Soyosource can be used as photovoltaics inverter or battery inverter only. You cannot use the inverter to charge your batteries!

Yes i know i bought 1 mppt charge to charge batteries i bought soyosource with wifi version seller say it only can have wifi or screen not both so I prefed wifi now i installed the code in esp home logs show is working but i have no information about the inverter whet do I need to do now?

@syssi can I send a pm to you please I need some help with my configuration

Could you try to upload your image another time?

yes i can

Do you still have questions? :-) Are you happy with your setup and can we close this issue?


Graphite derivative & scale

I have learned that I have to use the scale function with respect to the sample-rate / time-intervals of a given series to produce correct results from the derivative function.
IMHO this is creating a lot of pitfalls especially if you data source is not sending in absolute homogeneous intervals.
May I suggest improving this essential function to be self-learning the calculation intervals.

scaleToSeconds() doesn't work for you?

Works, thnx for the pointer


chore(glossary): fix broken links

I guess that the glossary was moved outside the guide so that explains the broken links.
PS: Didn't we have a tool to verify broken links?
Fixes #1078

I don't think we've run it recently. Must check on that.
I caught a couple more.
There is a remaining problem.  We have hidden glossary "guard" pages within the guide directories. If someone had an old link to them there (and some of the docs pages might!), these repaired links would be broken. There is no easy way to make them right in both places.
I'm tempted to delete those guard pages but not sure what I'd break. Look at this another day.
Merging this.


vSphere: Reconfigure VM's based on the clone's devices, not the template

What this PR does / why we need it:
This ensures that we used the devices of the cloned VM to create our change specifications.
Otherwise the change specifications contain references to the template VM.
NONE

/assign @alvaroaleman

/test pull-machine-controller-e2e-vsphere

/lgtm
/approve
/hold

/test pull-machine-controller-e2e-vsphere

/test pull-machine-controller-e2e-vsphere

/retest

@alvaroaleman PTAL

/hold cancel


Pixel mode breaks previous assumptions about layers

http://i.imgur.com/5Cm8A4V.png
The top of this bush should be above the player's feet, but since it's on the same layer as them they stand on it. If it were put a layer higher, their head would be under it if they approached from the bottom. Huge oversight. I'm not sure how best to fix this.

One idea: add a flag to the tile graphic to let it be "tall". Then, the top half of the graphic is drawn after the entities on that layer.

@seisatsu Will that work if you come at the plant from the left or right side?

Hmm, no it won't. Maybe we should do away with the whole colliding with the center of a tile thing. Then support polygons for nowalk in pixel mode.

I would like more time to think about this. Removing it from 0.0.6a for now.

We could implement #92. That might provide a quicker fix more suitable for 0.0.8.

How is that?

Well, one of the reasons why we added the colliding with the center of a tile thing was to make it easier to walk down 1-tile-wide corridors. But #92 can also solve that problem.

How will that fix the graphical layering issue?

If we remove the colliding with the center of the tile and replace it with colliding at the edge of a tile then entities will no longer appear above tiles on the same layer.

I don't see what that really has to do with #92 but yes.

#92 keeps narrow corridors easy to move through so we don't regress in that area.

Right, I thought you were saying #92 would fix the graphical issues. We should do both.

Depends on #92.


[BUG] Fresh 1.4.0 install never completes

Describe the bug (🐛 if you encounter this issue)
Trying to install 1.4.0 version on my cluster, installation never completes. Followed the instructions from the official documentation to make sure that all requirements are met which is the case. The only challenge that I face is regarding mount propagation. The documentation covers configuration based on Docker container runtime. Since my cluster leverages containerd as the runtime could not find any documentation on how to enable mount propagation in that case.
Anyway, running environment check script produces the following output:
[INFO]  All longhorn-environment-check pods are ready (6/6).
[WARN]  Unable to check kernel config CONFIG_NFS_V4_1 on node node1
[WARN]  Unable to check kernel config CONFIG_NFS_V4_1 on node node3
[WARN]  Unable to check kernel config CONFIG_NFS_V4_1 on node node2
[WARN]  Unable to check kernel config CONFIG_NFS_V4_1 on node node5
[WARN]  Unable to check kernel config CONFIG_NFS_V4_1 on node node4
[WARN]  Unable to check kernel config CONFIG_NFS_V4_1 on node node6
[WARN]  Unable to check kernel config CONFIG_NFS_V4_2 on node node1
[WARN]  Unable to check kernel config CONFIG_NFS_V4_2 on node node3
[WARN]  Unable to check kernel config CONFIG_NFS_V4_2 on node node2
[WARN]  Unable to check kernel config CONFIG_NFS_V4_2 on node node5
[WARN]  Unable to check kernel config CONFIG_NFS_V4_2 on node node4
[WARN]  Unable to check kernel config CONFIG_NFS_V4_2 on node node6
[WARN]  NFS client kernel support, CONFIG_NFS_V4_1 CONFIG_NFS_V4_2, is not enabled on Longhorn nodes. Please refer to https://longhorn.io/docs/1.4.0/deploy/install/#installing-nfsv4-client for more information.
[INFO]  Required packages are installed.
[INFO]  Cleaning up longhorn-environment-check pods...
[INFO]  Cleanup completed.

NFSv4.1 check output:
CONFIG_NFS_V4_1=y
CONFIG_NFS_V4_1_IMPLEMENTATION_ID_DOMAIN="kernel.org"
CONFIG_NFS_V4_1_MIGRATION=y

NFSv4.2 check output:
CONFIG_NFS_V4_2=y
# CONFIG_NFS_V4_2_READ_PLUS is not set
CONFIG_NFS_V4_2_SSC_HELPER=y

Here are the pods in longhorn-system namespace:
NAME                                           READY   STATUS             RESTARTS         AGE
longhorn-admission-webhook-6489cc5747-6zbrs    0/1     Init:0/1           0                68m
longhorn-admission-webhook-6489cc5747-7mnzf    0/1     Init:0/1           0                68m
longhorn-conversion-webhook-58b5f48bbd-l994t   1/1     Running            0                68m
longhorn-conversion-webhook-58b5f48bbd-mq59q   1/1     Running            0                68m
longhorn-driver-deployer-7fdddb9f99-2fzvs      0/1     Init:0/1           0                68m
longhorn-manager-9nsdl                         0/1     Init:0/1           0                68m
longhorn-manager-fqr9q                         0/1     Init:0/1           0                68m
longhorn-manager-hpgtm                         0/1     Init:0/1           0                68m
longhorn-manager-jnnqw                         0/1     Init:0/1           0                68m
longhorn-manager-mpz7x                         0/1     Init:0/1           0                68m
longhorn-manager-qjzzn                         0/1     Init:0/1           0                68m
longhorn-recovery-backend-d67444cf5-6p95w      1/1     Running            0                68m
longhorn-recovery-backend-d67444cf5-mhk2l      1/1     Running            0                68m
longhorn-ui-6768fbbc6c-x6k2q                   0/1     CrashLoopBackOff   17 (3m49s ago)   68m
longhorn-ui-6768fbbc6c-xj89m                   0/1     CrashLoopBackOff   17 (3m26s ago)   68m

While checking the logs of the pending pods I see that Init Containers waiting on something:
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting
waiting

Otherwise there are no errors in the logs.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Perform '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Log or Support bundle
If applicable, add the Longhorn managers' log or support bundle when the issue happens.
You can generate a Support Bundle using the link at the footer of the Longhorn UI.
Environment

Longhorn version: 1.4.0
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): Kubectl
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: Kubespray 1.25.4

Number of management node in the cluster: 2
Number of worker node in the cluster: 4

Node config

OS type and version: Ubuntu 22.04
CPU per node: 4
Memory per node: 8gb
Disk type(e.g. SSD/NVMe): NVMe
Network bandwidth between the nodes: 1G

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): Baremetal
Number of Longhorn volumes in the cluster: n/a

Additional context
Add any other context about the problem here.

Can you provide your /boot/config-$(uname -r) for improving the nfs client checks? Thanks.
For the installation failure, can you check your coredns pods? or show kubectl get pods -A.

coredns pods are running fine:
kube-system       coredns-588bb58b94-4fk4f                       1/1     Running            3 (9d ago)     26d
kube-system       coredns-588bb58b94-pqpb5                       1/1     Running            3 (9d ago)     26d

Also as requested attaching /boot/config file:
boot_config_5.15.zip

@samirski
admission-webhook pods is waiting for the conversion webhook pods. It seems the  something wrong in network.  Can you check https://github.com/longhorn/longhorn/issues/4503#issuecomment-1239189136?

@derekbit
Yeah, I can confirm that this is a network related issue. So closing this as it's not related to longhorn. Thanks for your help


Multiple missing images in /media folder

In instructions document: https://raw.githubusercontent.com/MicrosoftLearning/AZ-104-MicrosoftAzureAdministrator.fr-fr/master/Instructions/Labs/LAB_02a_Manage_Subscriptions_and_RBAC.md
We are missing file: ../media/lab02a.png
In instructions document: https://raw.githubusercontent.com/MicrosoftLearning/AZ-104-MicrosoftAzureAdministrator.fr-fr/master/Instructions/Labs/LAB_02b-Manage_Governance_via_Azure_Policy.md
We are missing file: ../media/lab02b.png
In instructions document: https://raw.githubusercontent.com/MicrosoftLearning/AZ-104-MicrosoftAzureAdministrator.fr-fr/master/Instructions/Labs/LAB_03a-Manage_Azure_Resources_by_Using_the_Azure_Portal.md
We are missing file: ../media/lab03a.png
In instructions document: https://raw.githubusercontent.com/MicrosoftLearning/AZ-104-MicrosoftAzureAdministrator.fr-fr/master/Instructions/Labs/LAB_09a-Implement_Web_Apps.md
We are missing file: ../media/lab09a.png
In instructions document: https://raw.githubusercontent.com/MicrosoftLearning/AZ-104-MicrosoftAzureAdministrator.fr-fr/master/Instructions/Labs/LAB_09c-Implement_Azure_Kubernetes_Service.md
We are missing file: ../media/lab09c.png

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Waiting for locfile check In. ICMS QUERY

Discussed with the PM and the en-us version of the art files have been added manually to the OLPROD branch


Node Sample

I am wondering how to connect this a Node App, precisely to Angular, but Node would be ok. Is there something on NPM? Answered question myself ... Maybe you should add this to the samples. There are a lot of ng & react guys out there 😉
https://github.com/Azure/azure-sdk-for-js/tree/master/sdk/appconfiguration/app-configuration

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

ID: 2984100d-7d78-2115-40f4-4b9478a5a5bc
Version Independent ID: 24ac6adc-6779-b847-ce81-c9d36e4b067b
Content: Quickstart for Azure App Configuration with ASP.NET Core
Content Source: articles/azure-app-configuration/quickstart-aspnet-core-app.md
Service: azure-app-configuration
GitHub Login: @lisaguthrie
Microsoft Alias: lcozzens

@ARambazamba Thanks for your feedback! We are investigating and will update you shortly.

Hello @ARambazamba ,
Thanks for bringing this to our attention !
I have added the content-author for review it.

Hi @ARambazamba - glad that you found our Javascript SDK! Since you mentioned Angular, I do want to mention that we don't intend for App Config to be used directly from client applications. Reason being, the service is not currently set up to handle the tremendous volume that could be generated from separate clients hitting it directly. If you want to use App Config to store configuration for client apps, we suggest building your own service to communicate with App Config, and then have your clients hit that service. That way, your service can load the configuration just once from App Config, but serve it up to multiple clients.
We are considering some future functionality that would enable direct client access -- let us know if it's something you'd be interested in, as that helps us with prioritization.

Thx for reviewing & @lisaguthrie‘s statement about hight volume traffic


Add all supported types as shortcuts into the Migration Builder

Add shortcuts for all SchemaInterface::TYPE_* constants. For now we have to do smthng like this in migrations to add UUID columns instead of $b->uuidPk()->notNull():
    public function __construct(private ConnectionInterface $db)
    {
    }

    public function up(MigrationBuilder $b): void
    {
        $b->createTable('table_name', [
            'id' => $this->db->getSchema()->createColumn(SchemaInterface::TYPE_UUID_PK)->notNull(),
        ]);
    }

Task steps:

Find out all such types
Add corresponding methods to the MigrationBuilder class

It remains to implement:

[ ] TYPE_UPK
[ ] TYPE_UBIGPK


Remove the requirement of SUSE CA repo

There should be no need for usage of SUSE internal repositories.

Hm, SES setup seems to require ca-certificates-suse, let's see if it is really needed

Seems that the CI job was able to deploy SES...


Egress proxy for external services

For services that don't have pods backing them, there's just an external IP and no deployment with a proxy. We need to deploy a set of proxies to handle cluster egress traffic, and configure them to capture and route traffic.
@vaikas-google

Do we need to create a set of such dummy services adding another unnecessary hop? If we augment the routing proto to include static IPs or fqdns, what then? It might complicate code a bit but it won't require the end user to declare what services are external or be surprised to see a pod for each such service or service instance. (E.g. lb1.foo.maps.com, lb2.foo.maps.com).

This issue is specifically for the ExternalName type services. Here is an example:
apiVersion: v1
kind: Service
metadata:
  name: my-app
spec:
  ports:
  - port: 12345
  type: ExternalName
  externalName: myapp.whatever.google.com

This service does not get a cluster IP and does not have pods in the cluster.
Imagine we want to monitor the traffic through this service and enforce control policies like quota.
We need to route traffic through Istio Proxy, so we create a deployment with proxies. We then rewrite the external name in the service "my-app" to point to Istio proxies, and make sure Istio proxies route the incoming traffic to "myapp.whatever.google.com". With this change, Kube DNS sends proxy address to Istio and non-Istio in-cluster clients. For Istio client-side proxy-enabled clients, we can skip Kube DNS and use Envoy discovery service.

Initial design for this feature.
I. Redirect traffic to external services to Istio proxy.
Manager rewrites externalName field to point to a dedicated k8s service that selects proxy-egress pods running Istio proxy. The original externalName is stored as an annotation on the service definition.
Manager also requires port definitions on the external services. Ports are optional for externalName services starting from k8s 1.6, but Envoy needs explicit listener ports in its configuration to capture traffic and choose the right protocol filter. Manager rejects external services with no port declarations.
II. Deploy egress proxies.
There are two choices for deploying dedicated egress proxies:
a) Deploy proxies per each external services.
Manager defines a new service for each external service to point to these proxies, which has the same port declarations and serves as the target DNS name.
b) Share egress proxies between external services.
There is one shared deployment and one service for cluster-egress proxies.
The service is managed to list all ports across external services  This shared service is the target DNS name in the external service externalName.
Proxies identify traffic using the external service hostname or VIP address.
I prefer option b), since it seems to be more in-line with sidecar proxy configuration.
Istio proxy does not need to use iptables port-rewriting since it's the sole container in its pod.
III. Handing cluster egress traffic for explicit IP addresses.
Current k8s workaround: use a regular service without pod selector, and manually add an Endpoint with the IP and port.
Proposed workaround: add an annotation for ExternalName services to override the DNS name with an explicit IP address. Currently, a custom endpoint is treated the same way as a regular pod, so it should be handled by existing sidecar client-side proxies. But non-istio pods need a dedicated server-side proxy deployment, which this design addresses.

fixed by #463
Option 2b is implemented currently, wherein applications are expected to access the external service via their respective Kube services (over HTTP). The egress proxy will do https to the external service. Support for traffic with explicit external IP address, tcp communication with external service (and https) is TBD

Closing


The wrong path for the import WebSocket module in the documentation

Issue subject
The documentation gives the wrong path for the import WebSocket module:

https://binance-connector.readthedocs.io/en/latest/getting_started.html#websocket
https://github.com/search?q=repo%3Abinance%2Fbinance-connector-python+binance.websocket.spot.websocket_client&type=code

from binance.websocket.spot.websocket_client import SpotWebsocketClient

There is no spot.websocket_client and SpotWebsocketClient.
Expected behaviour
Update the doc.
Actual behaviour
Error ModuleNotFoundError: No module named 'binance.websocket.spot.websocket_client'.
Steps to reproduce
Copy the code example to Python and run.
Environment
Provide any relevant information about your setup, such as:

binance-connector 3.8.0
Python 3.11
Ubuntu 24.04

The documentation on readthedocs has been updated in v3.7.0 to reflect the current method for using the WebSocket.
As for the README file, the line you mentioned refers to versions v1 and v2 of the connector, which used the WebSocket in this way. Therefore, there is no issue on that front.


Add in middy to PerformanceTest

I had a few from your community reach out asking how middy compared to @fastify/aws-lambda performance wise. So here is the results (Run on M1 Max):
aws-serverless-express x 14,187 ops/sec ±1.56% (83 runs sampled)
(node:64733) [FSTDEP011] FastifyDeprecation: Variadic listen method is deprecated. Please use ".listen(optionsObject)" instead. The variadic signature will be removed in `fastify@5`.
(Use `node --trace-warnings ...` to show where the warning was created)
aws-serverless-fastify x 13,922 ops/sec ±2.67% (80 runs sampled)
@middy/core x 759,611 ops/sec ±0.66% (91 runs sampled)
@middy/core + @middy/http-router x 754,083 ops/sec ±0.73% (92 runs sampled)
serverless-http x 42,098 ops/sec ±3.78% (87 runs sampled)
aws-lambda-fastify x 54,321 ops/sec ±2.18% (85 runs sampled)
aws-lambda-fastify (serializeLambdaArguments : true) x 55,472 ops/sec ±2.03% (85 runs sampled)
aws-lambda-fastify (decorateRequest : false) x 54,482 ops/sec ±2.25% (79 runs sampled)
Fastest is @middy/core

@middy/core: Shows when the pattern when there is one lambda per endpoint, router not needed
@middy/core + @middy/http-router: Show when a more monolith approach is preferred. This is equivalent to @fastify/aws-lambda (decorateRequest : false) based on my understanding of your docs.

Please let me know if there are any changes you need to make the comparison more accurate.
Checklist

[x] run npm run test and npm run performance
[x] tests and/or benchmarks are included
[x] documentation is changed or added
[x] commit message and code follows the Developer's Certification of Origin
and the Code of conduct

Sorry for the naive question, but how is middy correlated with Fastify?

Middy is correlated with @fastify/aws-lambda. Middy is a Node.js framework for AWS Lambda making it easier to handle HTTP events from API Gateway (REST & HTTP)/ALB/Function URLs similar to @fastify/aws-lambda, but can also handle any AWS event that Lambda can be triggered with.

Oh I see, the others are used as wrapper around fastify and that's what being compared. My mistake, I'll close.


Update Front App with Basic React App

Update folder with basic react app. Updated docker-compose to watch for react changes

@acoard-aot  - Make sure new PRs don't cause new ZenHub issues.  PRs shouldn't create Issues.  Need to investigate.


Hook not Fire()ed in a goroutine, causing application hangs

Hi,
As per the comment in the logrus hook code, it's expected that hooks will handle their own goroutines. It doesn't seem like you're firing your hook in a goroutine (and I can't see the elastic library doing so), so if ElasticSearch is running slowly for whatever reason, it will block until the requests are complete.
I'm not entirely sure if it's as simple as just setting the client...Do() call to run in a goroutine or not?
Thanks

The problem here is that you can leak the goroutines because this concept is fire and forget. And in extream case, you will get in troubles. Maybe this could be handled by pooling, so let's say you can have 20 parallel requests to elastic in one time.

The asynchronous hook added you can create the async hook by NewAsyncElasticHook function.


UDPReceiver

I'm sorry to disturb you. I want to know how to write in the receiver blueprint. Can you guide me? Be deeply grateful!

Added a video showing how to use UDPReceiver.


Adding disabled type

I've tried to fix this issue. ( because I want to use disabled property in fields :)
https://github.com/kintone/dts-gen/issues/23
Please review this 🙏

@mura-
Thank you for your contribution!!!
I can see diffs about indentations in package.json, which seems not to be necessary for this PR.
Could you revert the diffs in package.json?

@koba04
Sorry! I reverted package.json and package-lock.json. 🙇

@mura- Thank you! We are going to review this! Please wait for a while🙇

@yokotaso Could you review this PR?

I fixed it!
But I accidentally did force-push........... 😭😭 😭 😭

@koba04 @yokotaso
Thank you for your review so much! I've fixed to follow comments. 🙏

@mura-
I've released v2.0.2 that includes this fix🚀
Thank you!

@koba04 Thank you so much!!! 💪🤓


⚠ Bofrost.fr has degraded performance

In 4015e0c, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1270 ms

Resolved: Bofrost.fr performance has improved in 706d88c after 29 minutes.


Dark theme request

Hi,
Firstly thanks for all that work, I really appreciate your tool.
Just have some design request: In dark mode I would like to have the header of the active tab in white to be able to read it, and in the diagram view I would like to have the grid in an other color, currently it is in white and that breaks the eyes a little...
I attach a screenshot to show you that. Just been able to correct that in preference menu would be great.

Agreed. We need to add some default colors for ERDs in standard dark theme.
For now you can redefine ERD colors in Preferences->General->Appearance->Colors and Fonts.


What does "delta" parameter of the onWheel from viewer denote

Well in our application we wish to keep track of the camera position while the user pans/scrolls around. One of the inputs is the "scroll wheel" event. This event zooms in (lowers) camera and vice versa.
The documentation, however, is quite meager: https://resium.darwineducation.com/components/Viewer
onWheel | (delta: number) => void

I do see that delta is the based on the direction of movement, but what does the value denote.. The actual change in Z-position of the camera? If I subtract the value from the camera and reset the camera to that position I do notice the camera isn't at the same position as it would've been after scrolling. - When zooming in/going down I notice the change in number is smaller than the actual visual change.
So what does it mean?

Resium is a just wrapper of Cesium. So please contact to Cesium team.


Consider building dataset compared with OpenStreetMap

Of potential interest:

listing objects without nearby OSM match, revealing locations potentially worth surveying
listing OSM objects marked as branded with brand present in ATP and no nearby ATP match

and publishing both as a map.
Both can end detecting bad data in ATP rather than missing/outdated shops in OSM, definitely would not be usable for direct import but I think it would be of interest and useful for further reprocessing.
It is possible that I would be able to implement something like that if there would be interest in running and publishing it.

I think there would be lots of interest in this. I might suggest opening a sister issue in the main alltheplaces repo so that more people see it, though.

Not sure whether there would be interest in it but having ability to report (maybe using OSM account) cases where shop does not exist - based on survey or to mark blatant cases like https://www.alltheplaces.xyz/map/#9.76/45.3266/-24.9996 (charging station in the middle of Atlantic) could be useful.

opened https://github.com/alltheplaces/alltheplaces/issues/6787 then (with two open issues I think we end with duplication... But I can move all content here and keep the other one just as guidepost toward this one)


Added --scope flag and scope config property

With this PR, we're replacing --team with a more straightforward --scope flag (since that name covers both user accounts and teams).
Furthermore, we're adding a scope property to now.json for the same purpose.

Codecov Report

Merging #1939 into canary will decrease coverage by 0.29%.
The diff coverage is 0%.

@@           Coverage Diff            @@
##           canary   #1939     +/-   ##
========================================
- Coverage    6.61%   6.31%   -0.3%
========================================
  Files          91      91
  Lines        4069    4069
========================================
- Hits          269     257     -12
- Misses       3800    3812     +12

Impacted Files
Coverage Δ

src/commands/list.js
0% <ø> (ø)
:arrow_up:

src/commands/remove.js
0% <ø> (ø)
:arrow_up:

src/commands/secrets.js
0% <ø> (ø)
:arrow_up:

src/commands/deploy/args.js
0% <ø> (ø)
:arrow_up:

src/commands/certs/index.js
0% <ø> (ø)
:arrow_up:

src/commands/projects.js
0% <ø> (ø)
:arrow_up:

src/commands/scale.js
0% <ø> (ø)
:arrow_up:

src/commands/logs.js
0% <ø> (ø)
:arrow_up:

src/commands/alias/index.js
0% <ø> (ø)
:arrow_up:

src/commands/upgrade.js
0% <ø> (ø)
:arrow_up:

... and 7 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #1939 into canary will decrease coverage by 0.05%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           canary   #1939      +/-   ##
=========================================
- Coverage    6.33%   6.28%   -0.06%
=========================================
  Files          91      91
  Lines        4074    4092      +18
=========================================
- Hits          258     257       -1
- Misses       3816    3835      +19

Impacted Files
Coverage Δ

src/commands/list.js
0% <ø> (ø)
:arrow_up:

src/commands/secrets.js
0% <ø> (ø)
:arrow_up:

src/commands/remove.js
0% <ø> (ø)
:arrow_up:

src/commands/deploy/args.js
0% <ø> (ø)
:arrow_up:

src/commands/certs/index.js
0% <ø> (ø)
:arrow_up:

src/commands/projects.js
0% <ø> (ø)
:arrow_up:

src/commands/scale.js
0% <ø> (ø)
:arrow_up:

src/commands/logs.js
0% <ø> (ø)
:arrow_up:

src/commands/alias/index.js
0% <ø> (ø)
:arrow_up:

src/commands/upgrade.js
0% <ø> (ø)
:arrow_up:

... and 11 more

Continue to review full report at Codecov.

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


feat!: adding firebase

Adding the FirebaseActivity that as a exemple of adding entry in the database
Adding the google-service.json for firebase project Changing the gradle to add the dependencies for Firebase MAY occurs issues need to update the Android-studio build

Perfect, LGTM! :rocket:


Update Navber links hover effect.

I want to make navbar links hover effect like the following website previously done by me.
Screenshots

Please assign me @arpit456jain under GSSOC'22.

@Aritra777 you can add a new folder and make a navbar whichever way you like but kindly do not change the layout of main website.


Adding an API function that exposes the number of idle cores

Fixes #4599
The new API function is hpx::get_idle_core_count(), it will return the number of cores that have no threads (staged or pending) waiting in their queues.

Thanks, looks good to me!

This code does not take into account whether a core is currently running a thread or not. This still needs fixing.

@msimberg I went back and re-implemented the logic. Could you have another look, please?


feat: trigger top pages import when adding a site (draft)

Trigger the import of top pages implemented via SITES-21274, when adding a site via slack or when calling a specific slack command.
Please ensure your pull request adheres to the following guidelines:

[ ] make sure to link the related issues in this description. Or if there's no issue created, make sure you
describe here the problem you're solving.
[ ] when merging / squashing, make sure the fixed issue references are visible in the commits, for easy compilation of release notes

If the PR is changing the API specification:

[ ] make sure you add a "Not implemented yet" note the endpoint description, if the implementation is not ready
yet. Ideally, return a 501 status code with a message explaining the feature is not implemented yet.
[ ] make sure you add at least one example of the request and response.

If the PR is changing the API implementation or an entity exposed through the API:

[ ] make sure you update the API specification and the examples to reflect the changes.

Related Issues
Thanks for contributing!

no activity, closing


Top of the Window Black Notification

Since I use older Android devices with low resources I don't have the mobile version of messenger installed. Because of this the web and this version puts a quite noticeable reminder about the fact I should totally download it to my device. There is no way to remove it on the web version so I was hoping for a solution to this in this application. Also there seems to be no tray icon anywhere on ubuntu 16.04. Great application once the bugs are gone!

This software is a just a webwrapper for the messenger.com website.
That being said, its just html behind the scene so its totally possible to remove that banner.
I will add it to the todo list.

As for the tray icon, we have a hard time making it work on android. An update of NWJS is necessary, but I haven't had the time yet.

I fixed this, I don't think it's pushed to develop yet.

Wow awesome, thank you! :)

Issue still present as of 12/09/2016 using v2.0.1beta.

Thanks for letting us know, we'll check again... it seems our fix didn't work.

#1165 works for me.

Yes, but it will pop-up again if they change the variable name which happens all the time :)

It's been the same

From: Émile Fugulin<EMAIL_ADDRESS>Sent: Tuesday, January 17, 2017 9:44 PM
To: Aluxian/Messenger-for-Desktop
Cc: Nick Stathas; Comment
Subject: Re: [Aluxian/Messenger-for-Desktop] Top of the Window Black Notification (#534)
Yes, but it will pop-up again if they change the variable name which happens all the time :)

You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/Aluxian/Messenger-for-Desktop/issues/534#issuecomment-273277550, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMt2Y1v3mPHKnqiitKrRb2APppAIIOM2ks5rTRoOgaJpZM4Jv2Nq.

It's been the same for the past couple of months, so I don't see why not. CSS is CSS and they won't be changing that banner soon.


Implement and test usage of T-Taps on Canbus

Seems like this might be deprecated or converted to molex SL Issue


Fix rollup bug when init RowCursor in MergeContext

When doing rollup, seek_columns equals to the complete set of tablet's columns.
There is no necessity to set it.

Related to #1502


Add LCOV support

#57
A JTeC TestReport can be converted into an LCOV file. Both class-level and method-level coverage are supported.
The HTML coverage report can be generated from an existing LCOV file with genhtml, e.g.:
genhtml jtec-sample-project/junit5-project/target/jtec/test-report.info -o lcov_out
Open issues:

[x] Add constructors to LCOV report (ConstructorDeclaration)
[x] Methods declared inside another method not present in JTeC coverage report, but found by JavaParser (e.g., run in ThreadStarter)
[x] Find correct source file even if directory structure differs from package name

@rollix anything that still needs to be done or can we merge it?


Admin Dashboard - Improve Search Functionality (date range)

As an admin, I would like to be able to search for all projects created between Date A and Date B.

its not showing correctly
for instance it says that only one project was created between May 13th 2021 and Feb 8th 2022

@CarlosQ96 I think another filter is applying here beside date range.

Haven't been able to reproduce this issue

@WhyldWanderer is this feature still broken?

Seems to be working fine now.. I just checked! @divine-comedian
I think we can close it


[Doc] [NextJS] Error routes

Description
Covers topics:

How does the Next.js 404 functionality work with JSS
How does the Next.js error page work with JSS

Motivation

How Has This Been Tested?

Types of changes

[ ] 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 change)
[x] Documentation update (non-breaking change; modified files are limited to the /docs directory)

Checklist:

[ ] I have read the Contributing guide.
[ ] My code follows the code style of this project.
[ ] My code/comments/docs fully adhere to the Code of Conduct.
[ ] My change is a code change and it requires an update to the documentation.
[ ] My change is a documentation change and it requires an update to the navigation.

@anastasiya29 Nextjs 500 page handles all errors except of 404, I mentioned it in doc. Should I add something else?
https://github.com/Sitecore/jss/pull/611/files#diff-fb28a0ed6dea90ee06fd51c3366423ed1c871029e14e49d6a5a011376ccc51b3R22

@anastasiya29 Added paragraph how to render custom error code


Executioner Seems to Have Written to the Wrong Document

The judge sometimes write what should be in a judge-result document into the submission document and vice versa.

I guess this was caused by 9b35722.
Since we are approaching our launch day, it might be good to keep the executioner online. So you might want to create your own firebase firestore for testing. See README.md for more details.
Currently the content of the pages are very tightly coupled with the contest-problems repo. I aim to change that to use local files to make it faster and make local development and testing easier. Since changing that will likely cause some bugs, I think I will change that after v1.0. For now, you can just search for raw.githubusercontent.com and replace the contents accordingly.

I yes, I see I must have accidentally swapped the update state and add result functions. This will be resolved in a sec once the tests finish running. And yup, creating a firestore for testing is quite a good idea.


Embedded JSON Support in iOS/macOS glTF Viewer

Embedded JSON works on the Windows 3D Viewer but not on iOS/macOS. Requesting support for consistent functionality across platforms.
Details:

Problem: iOS/macOS glTF Viewer fails to correctly interpret embedded JSON.
Expected: Consistent model display as seen on Windows.
Actual: Models do not load/display properly on iOS/macOS.

Steps to Reproduce:

Load the provided glTF file with embedded JSON on iOS/macOS.
Observe inability to load/display the valid glTF2 json file with indefinite spinning.
Can load in Microsoft Windows 3d Viewer.

Additional Information:

Sample glTF file link: f11015d194b74147b5916defb9e7d2fa.zip

Please address this inconsistency to enhance cross-platform usability. Thanks for looking into this.

I can't reproduce.


Default column delimiter config not working

Setting the default column delimiter to \t in the package settings.
On opening a CSV file, initial column delimiter is ,, not \t.

Hi @sb80, Thanks for the report, I'll take a look at that this week-end.


Few questions

I would like to know what is the source to generate the token?
Metadata change in cluster is accessed by dyno client is push or pull model?
Dynomite support data rebalancing? if there is any new redis node added in cluster how the data will be rebalance? can you share the doc?
Does the dynomite support resharding without restart if I add one more node into the rack?
Does Dyno client open the connection with every node of dynomite in cluster? If not what is the use of topology aware in client side?

These are good questions:

Tokens can be generated with the following script. There are exemplar configurations provided. Once you add the tokens in YAML file, these are used when you provision Dynomite. When a node dies, we need to have an external groundtruth to make sure that the new node can read the tokens from. Some folks in the OSS have developed Florida and we have provided Dynomite-manager.

Dyno performs a get_seeds() REST call every 30 seconds to Dynomite-manager. This ensures that dynamic changes of the topology are mapped back to the Dyno client. Hence, this enables Dyno to be token aware.

Dynomite does not currently provide load rebalancing. Dynomite provides warm up when a node dies and new node joins the ring. In order to scale up/down your cluster, you can use the dual-write functionality of dyno (currently in beta).

Not yet.

Dyno opens multiple connections with all nodes in the clusters. By default it is 3.

We haven't heard back from you regarding this question. At this time, we're marking this case as Resolved. If you want to follow up with further questions let us know and we can either reopen the issue, or please create a new issue.

Thanks for reply. Looking into more detail level and will have more question after that. Is there any other channel on which I can ask my questions?

You can use Gitter: https://gitter.im/Netflix/dynomite?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge


Convert Advanced Peak Picking tutorial to markdown (.md) and automatically generate screenshots

I have been thinking that things might be better if our tutorials were written in markdown instead of Microsoft Word, and if the necessary screenshots were automatically generated by the tutorial test.
For one thing, if we wanted to point someone at the latest version of a tutorial test, we could send them to a page on github:
https://github.com/ProteoWizard/pwiz/blob/Skyline/work/20230818_MarkdownTutorialTests/pwiz_tools/Skyline/Documentation/Tutorials/markdown/PeakPicking/AdvancedPeakPicking.md
We could potentially also have a copy of the tutorial in Skyline, so that older versions of Skyline could display their version of the tutorial in a web browser control that is accessed from the "Help" menu.
Lots of details to still work out, like exactly when these screenshots should be updated and committed.

I have been wanting this since I had to write my first tutorial. Bravo!

I think it would be ideal if we had something that can embed the pictures into a single file for each tutorial.

I had been working on a plan for that.
I found a dll called "MarkDig" which can convert markdown to HTML, and there was some way to process the image tags and base64 encode the data with src="data:image/png;base64...

Obsolete. We will be converting tutorials to HTML instead.


custom te file loads every time  RE: Only allow refresh in the event that the initial .te file is updated.

Hello,
I have the below code block in one of my classes and every time I run puppet agent it displays the following message:
Notice: /Stage[main]/my_ldap_auth::Selinux_ssh_homedir/Selinux::Module[selinux_ssh_homedir]/Selmodule[local_selinux_ssh_homedir]/ensure: created
Caller
   selinux::module { 'selinux_ssh_homedir':
     source => 'puppet:///modules/modpath/selinux_ssh_homedir.te',
     require => Package['policycoreutils-devel'],
   }

Here are some standard variables from facter.
os => {"name"=>"CentOS", "family"=>"RedHat", "release"=>{"major"=>"7", "minor"=>"2", "full"=>"7.2.1511"}}
# facter  |grep selinux
selinux => true
selinux_config_mode => enforcing
selinux_config_policy => targeted
selinux_current_mode => enforcing
selinux_enforced => true
selinux_policyversion => 28

Any help would be appreciated.
Thanks!

Several people have come accross this issue. See:
https://github.com/jfryman/puppet-selinux/pull/92
https://github.com/jfryman/puppet-selinux/issues/90
A fix has been proposed by someone, but it is nog yet merged and versioned.
I find it strange that the most popular selinux module is so buggy. Don't know if it is still maintained by jfryman

the most popular selinux module

WOW! I had no idea. Thanks for the nice words.
I could always have another maintainer helping me out like I do with some of my other popular modules. I had no idea this one had traction, to be completely honest. Up for the job?
Otherwise, I hope everyone understands this is free work. I'll continue to do my best. Thanks for the bug reports and code!


Errror "Unknown or uninitialised column:"

I got errors when I try examples listed in your readme.
I have R 4.2.1 and rtweet 1.0.2, so I need rtweetXtras from your master branch and not legacy branch.
When I check the rtweetXtras' version, I got this :
Package: rtweetXtras
Title: Extra Analysis Functions for rtweet package
Version: <IP_ADDRESS>0

I don't know if it's the last or not (I've installed it via remotes::install_github("arf9999/rtweetXtras")
My error is "Unknown or uninitialised column:"
Here is a log of the errors :
> df <- rtweet::get_timeline("jack", n=100)
> rtweetXtras::hashtagcloud(df, n=200)
Attachement du package : ‘dplyr’
Les objets suivants sont masqués depuis ‘package:stats’:
    filter, lag
Les objets suivants sont masqués depuis ‘package:base’:
    intersect, setdiff, setequal, union
Package version: 3.2.3
Unicode version: 13.0
ICU version: 69.1
Parallel computing: 8 of 8 threads used.
See https://quanteda.io for tutorials and examples.
Error in `dplyr::filter()`:
! Problem while computing `..1 = !is.na(rtweet_timeline_df$hashtags)`.
✖ Input `..1` must be of size 100 or 1, not size 0.
Run `]8;;rstudio:run:rlang::last_error()rlang::last_error()]8;;` to see where the error occurred.
Message d'avis :
Unknown or uninitialised column: `hashtags`.
> df1 <- rtweet::search_tweets("rstats",lang = "en", n=2000)
> rtweetXtras::profilecloud(df1, 100)
Error in `dplyr::filter()`:
! Problem while computing `..1 = !is.na(rtweet_timeline_df$description)`.
✖ Input `..1` must be of size 1977 or 1, not size 0.
Run `]8;;rstudio:run:rlang::last_error()rlang::last_error()]8;;` to see where the error occurred.
Message d'avis :
Unknown or uninitialised column: `description`.
> colnames(df)
 [1] "created_at"                    "id"                            "id_str"                        "full_text"                     "truncated"                     "display_text_range"            "entities"                      "source"                        "in_reply_to_status_id"
[10] "in_reply_to_status_id_str"     "in_reply_to_user_id"           "in_reply_to_user_id_str"       "in_reply_to_screen_name"       "geo"                           "coordinates"                   "place"                         "contributors"                  "retweeted_status"
[19] "is_quote_status"               "retweet_count"                 "favorite_count"                "favorited"                     "retweeted"                     "lang"                          "possibly_sensitive"            "quoted_status_id"              "quoted_status_id_str"
[28] "quoted_status_permalink"       "quoted_status"                 "text"                          "favorited_by"                  "scopes"                        "display_text_width"            "quote_count"                   "timestamp_ms"                  "reply_count"
[37] "filter_level"                  "metadata"                      "query"                         "withheld_scope"                "withheld_copyright"            "withheld_in_countries"         "possibly_sensitive_appealable"
> colnames(df1)
 [1] "created_at"                    "id"                            "id_str"                        "full_text"                     "truncated"                     "display_text_range"            "entities"                      "metadata"                      "source"
[10] "in_reply_to_status_id"         "in_reply_to_status_id_str"     "in_reply_to_user_id"           "in_reply_to_user_id_str"       "in_reply_to_screen_name"       "geo"                           "coordinates"                   "place"                         "contributors"
[19] "is_quote_status"               "retweet_count"                 "favorite_count"                "favorited"                     "retweeted"                     "lang"                          "possibly_sensitive"            "retweeted_status"              "quoted_status_id"
[28] "quoted_status_id_str"          "quoted_status"                 "text"                          "favorited_by"                  "scopes"                        "display_text_width"            "quoted_status_permalink"       "quote_count"                   "timestamp_ms"
[37] "reply_count"                   "filter_level"                  "query"                         "withheld_scope"                "withheld_copyright"            "withheld_in_countries"         "possibly_sensitive_appealable"

Regards

I haven't updated rtweetXtras for rtweet 1.0 as yet. v1.0 of rtweet delivers a completely different data structure from v0.7, and this means that I need to rewrite every function and unfortunately have not had time to do this as yet.
So, unless you can install and use rtweetv0.7, I'm afraid that rweetXtras simply won't work for you right now.

From: pokemaster974 @.>
Sent: Sunday, October 2, 2022 12:26:10 PM
To: Arf9999/rtweetXtras @.>
Cc: Subscribed @.***>
Subject: [Arf9999/rtweetXtras] Errror "Unknown or uninitialised column:" (Issue #6)
I got errors when I try examples listed in your readme.
I have R 4.2.1 and rtweet 1.0.2, so I need rtweetXtras from your master branch and not legacy branch.
When I check the rtweetXtras' version, I got this :
Package: rtweetXtras
Title: Extra Analysis Functions for rtweet package
Version: <IP_ADDRESS>0
I don't know if it's the last or not (I've installed it via remotes::install_github("arf9999/rtweetXtras")
My error is Unknown or uninitialised column:
Here is a log of the errors :

df <- rtweet::get_timeline("jack", n=100)

rtweetXtras::hashtagcloud(df, n=200)

Attachement du package : ‘dplyr’
Les objets suivants sont masqués depuis ‘package:stats’:
filter, lag

Les objets suivants sont masqués depuis ‘package:base’:
intersect, setdiff, setequal, union

Package version: 3.2.3
Unicode version: 13.0
ICU version: 69.1
Parallel computing: 8 of 8 threads used.
See https://quanteda.io for tutorials and examples.
Error in dplyr::filter():
! Problem while computing ..1 = !is.na(rtweet_timeline_df$hashtags).
✖ Input ..1 must be of size 100 or 1, not size 0.
Run �]8;;rstudio:run:rlang::last_error()rlang::last_error()�]8;; to see where the error occurred.
Message d'avis :
Unknown or uninitialised column: hashtags.

df1 <- rtweet::search_tweets("rstats",lang = "en", n=2000)

rtweetXtras::profilecloud(df1, 100)

Error in dplyr::filter():
! Problem while computing ..1 = !is.na(rtweet_timeline_df$description).
✖ Input ..1 must be of size 1977 or 1, not size 0.
Run �]8;;rstudio:run:rlang::last_error()rlang::last_error()�]8;; to see where the error occurred.
Message d'avis :
Unknown or uninitialised column: description.

colnames(df)

[1] "created_at"                    "id"                            "id_str"                        "full_text"                     "truncated"                     "display_text_range"            "entities"                      "source"                        "in_reply_to_status_id"
[10] "in_reply_to_status_id_str"     "in_reply_to_user_id"           "in_reply_to_user_id_str"       "in_reply_to_screen_name"       "geo"                           "coordinates"                   "place"                         "contributors"                  "retweeted_status"
[19] "is_quote_status"               "retweet_count"                 "favorite_count"                "favorited"                     "retweeted"                     "lang"                          "possibly_sensitive"            "quoted_status_id"              "quoted_status_id_str"
[28] "quoted_status_permalink"       "quoted_status"                 "text"                          "favorited_by"                  "scopes"                        "display_text_width"            "quote_count"                   "timestamp_ms"                  "reply_count"
[37] "filter_level"                  "metadata"                      "query"                         "withheld_scope"                "withheld_copyright"            "withheld_in_countries"         "possibly_sensitive_appealable"
Regards
—
Reply to this email directly, view it on GitHubhttps://github.com/Arf9999/rtweetXtras/issues/6, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIY5MIIURPYR4PVP7DQCKU3WBFPMFANCNFSM6AAAAAAQ226VW4.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thanks for your reply, maybe it's my fault that I've misunderstood the readme, especially this part :

NB: This branch is now in development mode for the upcoming {rtweet version 1.0}. For compatibility with rtweet 0.7, please rather use the legacy version branch. Things will break here.

I don't know if you want to close my issue, I let open but it's up to you. Regards.


Workspace dependencies

Solves #24
I've never used these workspace dependencies before, so some sanity checking would be appreciated.

Forgot this PR was open. I've done this work on main already.


Localization in files in app/controllers

app/controllers/alert_controller.rb
app/controllers/cim_base_storage_extent_controller.rb checked no changes needed
app/controllers/cim_instance_controller.rb
app/controllers/cim_storage_extent_controller.rb checked no changes needed
app/controllers/flavor_controller.rb

"<All>" is part of a condition few times in code so it was left as is for the time being.

Reviewed, changes look good, ok to merge.


Add a distance-based comparator

Closes #25
Word distances are cached similarly to what I have done with unique words in #21 because the comparator will be invoked VERY frequently.

Wait. It won't work for items not coming from this source.

@hrsh7th Could you please take a look?

Updated

Almost good to me! Thank you!

@hrsh7th All done

LGTM. Thank you!


Retry events until completed

Sometimes ISP connections are unstable. Such was the case for me on 2022-09-16. My connection was basically unusable for the whole day until I re-synchronized my "modem" with the ISP. During this time, at least one event was recorded and could not be uploaded to the remote store. The issue ends up looking like the attached screenshot.
I think this issue can be solved at the same time as #49. What I am thinking, is that a local sqlite or leveldb store can be used to keep track of the events received and successful uploads. When the script purges items from the remote storage, it can also remove records from this local tracking store (i.e. implement a sliding window of tracked events).

Hi,
just wanted to say I’ve seen this and I understand your problem and will look into fixing it at some point. However, I just got a puppy so my spare time is somewhat limited in the near term

Hi,
Thank you for your patience. This feature should now implemented in the latest update. The tool now ensures all events within the retention window are backed up, even if they occurred before it was first run.
The check runs periodically, so if some event failed to download, it should retry it with every check for missing events until it suceeds

This is the only pip-installed program I use... so apologies if this is a noob question.
I'm running 0.7.4 at the moment, and when running:
pip install --upgrade unifi-protect-backup
...I just get a lot of "Requirement already satisfied" and nothing upgrades.
What is the best way to upgrade please?

My apologies looks like the CI failed to upload the release to PyPI. Please check back in an hour or so and hopefully it will be resolved by then

No worries, and thanks for your work on this, it's much appreciated.
Other than (when it's available) running the pip upgrade, is everything else the same, no gotchas with config etc., it works the same as before?

If you are using the pip version there should be no change in your use of the tool. The only potential issue is that the tool now creates a SQLite database that you need to make sure persists (mostly an issue with running the tool as a container)

mostly an issue with running the tool as a container

Does it default to creating the database in the already mounted data volume?

mostly an issue with running the tool as a container

Does it default to creating the database in the already mounted data volume?

It defaults to /config/database/events.sqlite

On my RH (7) server /config does not exist at all...
I notice: --sqlite_path. The Help text says "Path to the SQLite database to use/create" - is it the path to the database, i.e. it needs to be a full path including filename? Or is it a path to the containing folder (in which case can I suggest changing the text to something like "Path to the folder to be used for the SQLite database to use/create (folder needs to be persistent on reboot)").

OK, upgraded, and tested with setting a --sqlite_path to a db file, and that looks like it is what it wanted, so all good there. It picked up some missed events on start and has uploaded those.
Thanks Sebastian - great upgrade.

I did notice one peculiarity.
To upgrade I (accidentally) stopped the previous version in the middle of it backing up an event, and then upgraded and restarted (in debug to make sure the db was connecting OK). You can see the sequence here:
Dec  5 09:32:31 emp75 unifi_protect: 2022-12-05 09:32:31 [INFO]:unifi_protect_backup.unifi_protect_backup:#011Backing up event: 638d2dfc0139bf03e4034612
Dec  5 09:32:42 emp75 unifi_protect: 2022-12-05 09:32:42 [INFO]:unifi_protect_backup.unifi_protect_backup:#011Backed up successfully!
Dec  5 09:33:19 emp75 unifi_protect: 2022-12-05 09:33:19 [INFO]:unifi_protect_backup.unifi_protect_backup:#011Backing up event: 638d2e03031dbf03e403461b
Dec  5 09:35:00 emp75 unifi_protect: 2022-12-05 09:35:00 [#033[36;1m   DEBUG   #033[0m] unifi_protect_backup.unifi_protect_backup  :#011Config:
Dec  5 09:35:00 emp75 unifi_protect: 2022-12-05 09:35:00 [#033[36;1m   DEBUG   #033[0m] unifi_protect_backup.unifi_protect_backup  :#011  address='<IP_ADDRESS>'

The event that was backing up (638d2e03031dbf03e403461b) when the service was stopped has not been uploaded to OneDrive, and is not being picked up as 'missed'.
I've not tested to see if any subsequent restarts in the middle of an action result in the same miss.

On my RH (7) server /config does not exist at all...

That path was for inside the container sorry. If your are running it directly on the machine it defaults to events.sqlite in the current directory.

I notice: --sqlite_path. The Help text says "Path to the SQLite database to use/create" - is it the path to the database, i.e. it needs to be a full path including filename? Or is it a path to the containing folder (in which case can I suggest changing the text to something like "Path to the folder to be used for the SQLite database to use/create (folder needs to be persistent on reboot)").

It means the full path to the database file, not a directory
As for your above issue, I will open a new issue to track it and keep the discussion there rather than in this closed issue


imm12 add

Works, but since type inference prioritizes known bit widths, it thinks the args to iadd have whatever width imm12 has. This is causing "query width unused."

Looks good to me!
A weird subset of tests are actually failing, but I don't see why from this PR.

Should I wait to merge? Do you think it might be due to the iconst changes or the extraction bug?

Let me try to run locally first, will report back in a few.

No tests failing, but these two failed to terminate after 3 hours in CI:
test test_iadd_from_file
test test_udiv

No tests failing, but these two failed to terminate after 3 hours in CI:
test test_iadd_from_file
test test_udiv

I've definitely run all the iadd examples and they've passed. I think this is due to hanging solver. The same might be true for udiv though I'm less sure about that. In any case, this hanging solver is making it hard to tell which tests are really broken and which just look broken.

Things now work locally, so force merging


Update package.json

Q
A

Bug fix?
yes

New feature?
no

New sample?
no

What's in this Pull Request?
Updated package.json to solve issues regarding react-to-print library

thx @AhmadiRamin, appreciated.


Test getting loc coord from map

(Locality's decimal coordinates from a map)

Tested using default coordinates (not actually clicking the map) in #316


Lower tile to Krnl

The Tile Op in onnx is the same as bumpy tile without broadcast. That means the size of the repeats tensor is the same as the rank of the input and output tensor.
There are two ways to lower Tile to Krnl:

Default implementation
loop over output (I1, I2, ..,Ik, ...).
Loop body:
output[...,Ik, ...] = input[..., Ik mod input.shape[k], ...]

Alternative implementation
loop over input (I1, I2, ..., Ik, ...)  and repeats ( J0: 0 to  repeats[0], J1  0 to repeats[1], ..., Jk:0 to repeats[k], ...)
Loop body:
output[..., Ik+Jk*input.shape[k], ...] = input[..., Ik, ....]

I implemented both in Tile.cpp but choose the first one as the default. Unclear to me which one is better for optimization.
The alternative implementation passed the backend test.

Hi @chentong319 could you enable relevant node-level unit tests here:
https://github.com/onnx/onnx-mlir/blob/master/test/backend/test.py

@chentong319 unit test for tile operation seems to be failing, could you double-check the implementation?

@tjingrant check-onnx-backend passed. There were two errors:

Should use input.shape[k] for mod or multiple, instead of repeats[k]
Can not use affineload when the subscript expression contains mod

@AlexandreEichenberger this operation is affine because it is kth iteration variable % repeats[k] and k is a constant and repeats[k] is a constant value for the loop. E.g. for loop (j, k), we have j  % repeats[0] and k % repeats[1]

We must make sure that the operation the map performs is indeed affine (according to the spec) otherwise we may end up generating the wrong code further down the line.
@AlexandreEichenberger thanks for showcasing this in your comment.

This issue is caused by the mod operation (similar situation for div and multiple).  Only + and - allow both operands to be AffineExpr.
Will restore to previous version of code.

Tong, that is not my suggestion. I suggest that we generate the previous version with dyn sizes, or the one below with constant sizes.
      %7 = affine.load %arg0[%arg1 mod 4, %arg2 mod 8] : memref<4x8xf32>

mods are allowed with integers.

(onnx-mlir) alexe@pancetta maps % onnx-mlir-opt tile-test-tong2.mlir --lower-affine
tile-test-tong2.mlir:15:12: error: semi-affine expressions (modulo by non-const) are not supported
%4 = affine.apply #map0(%arg1)[%3]
^
tile-test-tong2.mlir:18:12: error: semi-affine expressions (modulo by non-const) are not supported
%6 = affine.apply #map0(%arg2)[%5]

Canonicalize before --lower-affine would help: onnx-mlir-opt tile-test-tong2.mlir --canonicalize --lower-affine
#map0 = affine_map<() -> (0)>
#map1 = affine_map<() -> (12)>
#map2 = affine_map<() -> (16)>

module {
  func @test_tile1(%arg0: memref<4x8xf32>) -> memref<12x16xf32> {
    %0 = alloc() : memref<12x16xf32>
    %1 = "krnl.global"() {name = "constant_0", shape = [2], value = dense<[3, 2]> : tensor<2xi64>} : () -> memref<2xi64>
    %2:2 = krnl.define_loops 2
    krnl.iterate(%2#0, %2#1) with (%2#0 -> %arg1 = 0 to 12, %2#1 -> %arg2 = 0 to 16) {
      %c4 = constant 4 : index
      %3 = remi_signed %arg1, %c4 : index
      %c0 = constant 0 : index
      %4 = cmpi "slt", %3, %c0 : index
      %5 = addi %3, %c4 : index
      %6 = select %4, %5, %3 : index
      %c8 = constant 8 : index
      %7 = remi_signed %arg2, %c8 : index
      %c0_0 = constant 0 : index
      %8 = cmpi "slt", %7, %c0_0 : index
      %9 = addi %7, %c8 : index
      %10 = select %8, %9, %7 : index
      %11 = load %arg0[%6, %10] : memref<4x8xf32>
      store %11, %0[%arg1, %arg2] : memref<12x16xf32>
    }
    return %0 : memref<12x16xf32>
  }
}

@tungld Thanks, good to know. But presumably that worked only in the example where the dims were constants, correct?

@AlexandreEichenberger We are on the same page: there is a previous version that choose affine.load only when shape is constant.
As @tungld pointed out, the --canonicalize  flag can help. Also, the current version passed the backend test. That's a sign that we can rely on the system to handle DimOp for constant and constant propagation through map. Thus, I kept the most of the code and just added a check of the shape to select affine load or ordinary load.

@chentong319
as long as the version with the const dims can use the affine maps, affine.apply and that the one with the dyn dimensions use the compute directly.
I tested Tung's additional canonnalize option, and it simplifies the dims of constant sizes fine, good to know. But when adding a ? in the memref, it chokes again on the non-affine.
(onnx-mlir) alexe@pancetta maps % onnx-mlir-opt tile-test-tong2-dyn.mlir --canonicalize --lower-affine
tile-test-tong2-dyn.mlir:19:12: error: semi-affine expressions (modulo by non-const) are not supported
      %7 = affine.load %arg0[%4, %6] : memref<?x8xf32>
           ^
#map0 = affine_map<()[s0, s1, s2] -> (s1 mod s0, s2 mod 8)>
#map1 = affine_map<() -> (0)>
#map2 = affine_map<() -> (12)>
#map3 = affine_map<() -> (16)>

module {
  func @test_tile1(%arg0: memref<?x8xf32>) -> memref<12x16xf32> {
    %c0 = constant 0 : index
    %0 = alloc() : memref<12x16xf32>
    %1 = "krnl.global"() {name = "constant_0", shape = [2], value = dense<[3, 2]> : tensor<2xi64>} : () -> memref<2xi64>
    %2:2 = krnl.define_loops 2
    krnl.iterate(%2#0, %2#1) with (%2#0 -> %arg1 = 0 to 12, %2#1 -> %arg2 = 0 to 16) {
      %3 = dim %arg0, %c0 : memref<?x8xf32>
      %4 = affine.load %arg0[symbol(%arg1) mod symbol(%3), symbol(%arg2) mod 8] : memref<?x8xf32>
      store %4, %0[%arg1, %arg2] : memref<12x16xf32>
    }
    return %0 : memref<12x16xf32>
  }
}

Interestingly (for me at least), the cannonalize moves the apply you added directly as operations inside of the affine.load.
I checked also that if you change the affine.load to the std.load, your code works as is... So double check too, but it looks like the only change you need to do is to determine affine vs std.

It is neat to handle the constant dimension and unknown dimension in this way. I think this is the reason that MLIR has the semi-affine extension over the restrict affine.
Thanks @AlexandreEichenberger  and @tungld. Please review my PR for flatten, which is coming soon.

@jenkins-droid test this please.


add casting for additional aggs in groupby

resolves #1574

This looks good! 👍🏻
Seems like the test assertions might need some updates to check the new aggregations though?

I added some asserts for new aggregations. I thought that getting an assert for nunique and mean would be a good start, given how different they are. Essentially, I duplicated the logic that was there before, and I perform calculations on the lists and compare them to the output from aggregations.
But unfortunately, this is now failing in a new way... If I want to aggregate a float32 column into a list, I get the following error:
TypeError: Dtype discrepancy detected for column z-list: operator Groupby reported dtype `float32` but returned dtype `float64`.

I looked briefly around, but have a hard time figuring out where to even start troubleshooting this. This doesn't seem to be an issue with casting like it was the case with the aggregations that were missing.

Not sure if this might be helpful, but let me maybe try to document what I'm seeing
Just before returning from _apply_aggs in Groupby, when running on the CPU, here are a couple of facts about the df

And here is the same spot but on the GPU

The df gets returned from _apply_aggs and it then gets returned straight from transform.
Before it gets returned, the situation looks quite similar, it's just a column of lists with Python 64bit floats inside. Not a lot of opportunity to maneuver here, just sharing this for broader consideration, I am not sure how (or where) to address this. There doesn't seem to exist an opportunity to cast this in Groupby AFAICT.

I pushed to origin and not to my fork and so the new changes were not showing here 🤦♂
This now has the test that is failing on the CPU, this is what I have been talking about.

Ok, I am thinking that this might actually be good enough! 🙂
There are no explicit tests for the new behavior with casting (and thus the aggs running), but the fact that the test executes with these new additions is actually a testament to this working (it would fail without the changes added here).
I was unable to write a test for these aggregations actually working (summing correctly, etc) because in test_groupby_op I didn't manage to find a way to write it where the test would work across all the "keys" scenarios.
I am not sure how to think about this. I can add a test for the behavior of these aggs (for instance, nunique across partitions might be interesting) or maybe this what we have here is enough. The redeeming quality here is that what we have there now does implicitly test these aggregations work on the schema side of things.

Made one minor suggestion about a typo, but otherwise LGTM


how tiller decide the execution order of chart and dependency chart

As question, I just find out tiller sort the execution by specific order, the install order is {"Namespace", "Secret", "ConfigMap", "PersistentVolume", "ServiceAccount", "Service", "Pod", "ReplicationController", "Deployment", "DaemonSet", "Ingress", "Job"}, but I haven't figured out the execution order of chart and dependency chart

All charts are grouped together, then sorted by kind.
For example:
MyApp has dependency MyDB. They look like this:

MyApp

MyApp ConfigMap
MyApp Replica Set

MyDB

MyDB Service
MyDB DaemonSet

The order of install is:

MyApp ConfigMap
MyDB Service
MyDB DaemonSet
MyApp ReplicaSet (which, since order is not specified in the list above, will go last)

I'm going to close this. Feel free to re-open if you still have questions about it.

Hi,
Could you please help to explain why all charts are grouped together including the dependency?
In my opinion, the dependency chart should be created first, and after it is created successfully, its parent chart should be started to create.

+1 to the comment above, i need dependent charts to run BEFORE the parent because they add pre-reqs like custom api resources.   Any suggestions on how to do this other than another repo?

@gyliu513 I think this explain the reason why putting configmap consumed by subcharts to parent is working fine.

Also currently having this issue trying to deploy a custom chart that depends on a stable/postgresql installation being present before installation. My application first runs a migration Job (using lifecycle pre-install annotations), and then completes my deployment. The migration Job is set to wait for the database to become responsive before executing. The stable/postgresql chart provides extension points that allowed me to annotate the deployment and PVC with a lower priority pre-install hook, but because the SecretsMap is not present, it still fails.
Definite +1 to the ability to coarsely order the installation of dependencies for a chart.

@dkushner Can you provide the pre-install hook example use to get around the ordering issue?

I'm running into an identical issue.  I have a db migration script set as a pre-install hook that, prior to my current task, depended on the database being manually deployed prior to the service that includes the migration.
Now, trying to merge the charts into a parent chart that installs an external service for postgresql (pointing to an RDS instance) the chart with the pre-install hook executes and fails stopping the deployment.
I thought I had a fix using a pre-install hook on the postgresql service, but of course hooks are not managed by tiller so the next upgrade failed with a "service exists" error.


Update SAC to use Strings instead of Bytes where appropriate

https://github.com/stellar/rs-soroban-env/issues/742#issuecomment-1495533336

Closed by https://github.com/stellar/rs-soroban-env/pull/852


HotFix pinch to zoom - issue #122

When testing on mobile, pinch to zoom throws error Uncaught TypeError: chartsToPan.indexOf is not a function.
Steps to reproduce the bug:

Put one finger on the mobile screen
Put a second finger on the screen and move this second finger.
Do not move the first finger.

Because of the thrown error the render fails and the charts disappear.
This fix might not be perfect, it only prevents the render from crashing. The result behaviour might not be the one expected (zoom level reseted?).

Thank you, I don't do enough to test on touch devices.
just out of curiosity, how can one test pinch zoom testing using chrome on desktop? I can emulate a device and do one finger swipe, but have no idea how to do 2 finger swipe or pinch zoom

I am using safari on iOS and safari on macOS to debug mobile. When plugging an iPhone on a Mac you can show developer tools in safari (desktop) for the browser running on your plugged device (i.e. safari mobile).
I don't know about chrome mobile / desktop but I guess they provide a similar feature.
(Thank you for creating/maintaining the project, this is my first public PR ever🎉)


fix(@angular/build): Fixing auto-csp edge cases

The changes were merged into the following branches: main, 19.0.x