Replace/automate UCD Movement of Property Form

Electronic form satisfying this policy:
https://ucdavispolicy.ellucid.com/documents/view/501/521

@acgetchell link doesn't work?

https://ucdavispolicy.ellucid.com/documents/view/501

https://ucdavis.app.box.com/s/aoixkubwww6sxbeopbzho345spsu58ef

available now w/ docusign feature


Added 'add_provider' to generic provider

Issue #74
add_provider() on Generic adds an instance of the provider class, so a new provider class such as my_generic.add_provider(SampleProvider) will let you call my_generic.SampleProvider.method() afterwards.
I used the class name to set the attribute, but it could definitely be changed to look for a key variable in the new provider to set the attribute within the generic provider.
Raises TypeError if a class type is not passed.

@Battleroid Great job!  I hope that you will continue in the same spirit. Thank you!

No problem. Glad I could help.


Jekyll 4 Support

Hi,
We should add support for Jekyll 4 asap :)
Kind regards,
Kyle

Hey folks, do you have an ETA for this issue ? Thanks a lot.
Kind regards

Here is a pull request for Jekyll 4: https://github.com/juusaw/amp-jekyll/pull/49

Pull request approved, merged and released. Version 1.0.3 of the gem should support Jekyll 4.


45 add basic structure for energy classes

@ndaelman-hu @JosePizarro3 @Bernadette-Mohr Here is a first proposal for a basic structure for various energies. Please provide feedback at your convenience (you can exclusively look at energies.py)

Thinking now about extending this to forces: Should the forces structure not be an exact mirror of energies? If so, how can we encourage or ensure that they are always updated/extended simultaneously?

Thinking now about extending this to forces: Should the forces structure not be an exact mirror of energies? If so, how can we encourage or ensure that they are always updated/extended simultaneously?

Are they tho? I am guessing that forces are matrices (or vectors?), while energies are scalars, no?
What do you mean "updated/extended simultaneously"? We can do a quick chatting in a Discord room to understand this point too.

Thinking now about extending this to forces: Should the forces structure not be an exact mirror of energies? If so, how can we encourage or ensure that they are always updated/extended simultaneously?

Are they tho? I am guessing that forces are matrices (or vectors?), while energies are scalars, no?
What do you mean "updated/extended simultaneously"? We can do a quick chatting in a Discord room to understand this point too.

Sorry for being unclear. I just meant that besides the total force, there are various force contributions defined currently, e.g., "free" and "t0". These directly correspond to some energy contributions (although the schema is not nearly as built out as for energies. In principle, there could be a force contribution/definition for every energy, since they are directly related.
My question was, why not enforce there to always be both terms added, for each type of contribution? And if this is desired how can we try to achieve this in practice?
For the classical contributions I will build this mirroring regardless, since energy and force contributions coming from particular interactions are commonly stored.

Reviewed outputs.py, can do the rest if needed. No real conceptual comments, just specifics.

You are reviewing the wrong changes. The ones in #43 are the ones you want to take a look.

@JFRudzinski if you branch off from a non-merged branch, pls specify this in the PR or set the target branch to the proper base...

Reviewed outputs.py, can do the rest if needed. No real conceptual comments, just specifics.

You are reviewing the wrong changes. The ones in #43 are the ones you want to take a look.

@JFRudzinski if you branch off from a non-merged branch, pls specify this in the PR or set the target branch to the proper base...

Ah! Got it, Sorry about that!

My question was, why not enforce there to always be both terms added, for each type of contribution? And if this is desired how can we try to achieve this in practice?

I don't think that we should enforce this generally for 2 reasons:

to derive a (general) force, you need to know the energy manifold in terms of its dependent variables. It's not a given that this information will always be present.
I'm not sure whether it's always desirable for the "forces" to be known. E.g. in the case of total DFT energy, I have little interest in the chemical potential. This is only relevant from a convergence perspective.

With that said, I think it's a good idea to provide an optional functionality. Would recommend something numpy-based as implementation.

ide an optional functionality. Would recommend something numpy-based as implementation.

I don't really understand what you are saying tbh. I am not talking about enforcing the storage of such quantities. I was talking about schema development. I think "enforcing" is a bit too strong of a word, but I had in mind the case that someone wants to store some energy contribution and implements a class for it, but doesn't create the corresponding force contribution because they personally do not store this in there simulations. But then another user comes along and wants to store this force contribution and now has to extend the schema themselves, whereas if the original person had been encouraged to add a corresponding force contribution it would have made everyone's life a lot easier.
Of course there might be some energy definitions that it doesn't make sense to define a corresponding force. In that case, obviously it should not be done, but I think in most cases it is possible at least in principle.

To be honest, I need to see or talk about an example to fully understand the situation. But I'd wildly suggest to use physical_property_ref (without setting it to is_derived == True) for these cases.
But again, I need to see the example in a board or in a piece of paper.

@JFRudzinski bear in mind I merged something which created the sub-folder, properties/energies.py, with FermiLevel and ChemicalPotential defined in there. When you work again on this, you can use that then.

Yes, got it, thank you :+1:

@JFRudzinski to avoid confusion I am closing this in favor of #91
You can add (in the description) a "Closes #numberofissue" so Github closes automatically the issue(s) related with #91


Getting individual results for each individual Simringer branch

Hi,
If I call addUri on a Simringer, is there a way to get back the result of that particular branch/call?
IE in our code we have req._simringer.addUri(remoteUri, additionalCallOpts);
(we've stashed the simringer on the original req).
In the case I'm looking at, I see:
[2020-04-25 15:25:00.166 +0000] DEBUG (83163 on maximus.daviesfam.org): CallManager#_attemptOne: launched call to<EMAIL_ADDRESS>2 calls in progress
[2020-04-25 15:25:00.170 +0000] INFO  (83163 on maximus.daviesfam.org): CallManager#attemptOne: call to<EMAIL_ADDRESS>failed with 503

Another branch goes on to connect so the whole simringer succeeds.
But this particular branch gets a 503 - can I get my hands on that result?
Thanks,
Steve

I did try to .then the addUri - didn't look like addUri returned anything and indeed:
[2020-04-25 17:07:04.322 +0000] ERROR (83370 on maximus.daviesfam.org): Error setting up call
    err: {
      "stack":
          TypeError: Cannot read property 'then' of undefined
              at CallProcessor._connectLegs (/Users/steve/rtcproxy/lib/call-processor.js:601:7)
              at /Users/steve/rtcproxy/lib/call-processor.js:548:24
              at Array.forEach (<anonymous>)
              at /Users/steve/rtcproxy/lib/call-processor.js:370:21
      "message": "Cannot read property 'then' of undefined"
    }


Show % Hotspots online excluding Denylisted Hotspots

Our current % online looks low but that's because offline Hotspots include denylisted Hotspots. Propose we include a stat that says "% Online (Filtered)" or "% Online (excl Denylist)"
Denylist can be found here:
https://github.com/helium/denylist/blob/main/denylist.csv

Does "hotspots" include "data only hotspots"?
As they could be filtered out too, or is online % already considering % out of hotspots-DO hotspots?


'Idle' MiMa spins at 10% CPU usage

I like to leave the GUI app open, so I can re-evaluate the changes when working on a new snapshot of my library. However, I noticed (on OS X, via the ActivityMonitor) that the MiMa is constantly using approx. 10% CPU (on a very fast machine), even when not performing any actions, so something must be wrong with the EDT not being idle.

the GUI has been removed from the most recent releases.


[Resource] Add 'Open-Source Software: An Introduction' to README

Category

[x] Resource Addition
[ ] Documentation
[ ] Codebase
[ ] User Interface
[ ] Feature Addition

Description
Category: Introduction to Open Source
Name: Open-Source Software: An Introduction
Publisher: Open Access Government
Link: https://www.openaccessgovernment.org/open-source-software-an-introduction/140466/

I would like to work on it under GSSoC'24 extd and Hacktoberfest 2024

Hey @Bishal-35, this repository is not participating in GSSoC '24 Extended, but it does in Hacktoberfest 2024.
Nevertheless, I'm assigning this to you.

Thankyou @Sriparno08 for assigning


UAA as IdP needs to support IdP initiated Web SSO with SAML in addition to SP initiated

UAA as IdP today only SP initiated SSO with SAML is supported. Applications may need to use the IdP initiated model as users may already have their own token and should be able to make request for a SAML assertion directly from IdP instead of hitting the SP endpoint and redirected back to IdP for authentication again.

We support SAML Bearer token flow to allow for token generation based on incoming SAML Assertion.  Please find the details here : https://docs.cloudfoundry.org/api/uaa/index.html#saml2-bearer-grant


Feature Request:

Is it possible to have a binary_sensor on person detection

https://github.com/JurajNyiri/HomeAssistant-Tapo-Control/issues/304


CustomStoryEditorController can not called .

Hi , I want use this package .Because it encompasses all the features I was trying to create. You are fantastic.
Issue
I could not called CustomStoryEditorController

Flutter doctor
[✓] Flutter (Channel stable, 3.24.0, on macOS 14.6 23G80 darwin-arm64, locale ja-JP)
[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2024.1)
[✓] VS Code (version 1.92.0)
[✓] Connected device (5 available)
[✓] Network resources

• No issues found!

Can you publish an example Flutter Project using this package?
I would like to refer to it.

use this import
"import 'package:custom_story_editor/src/controller/controller.dart';"
I will resolve this issue in a future release and add sample code.

use this import
"import 'package:custom_story_editor/src/controller/controller.dart';"
I will resolve this issue in a future release and add sample code.
Le jeu. 8 août 2024 à 11:00, jtaxiexpress @.***> a
écrit :

Hi , I want use this package .Because it encompasses all the features I
was trying to create. You are fantastic.
Issue
I could not called CustomStoryEditorController
2024-08-<IP_ADDRESS>.png (view on web)
https://github.com/user-attachments/assets/01088ec9-849a-431c-a58b-01465f3b0ff7 Flutter
doctor
[✓] Flutter (Channel stable, 3.24.0, on macOS 14.6 23G80 darwin-arm64, locale ja-JP)
[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2024.1)
[✓] VS Code (version 1.92.0)
[✓] Connected device (5 available)
[✓] Network resources
• No issues found!

Can you publish an example Flutter Project using this package?
I would like to refer to it.
—
Reply to this email directly, view it on GitHub
https://github.com/TchatchoKemajou/custom_story_editor/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ANT4K2I3JGBCGIHUTNQ7DKTZQM6VVAVCNFSM6AAAAABMGEFPG2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ2TKMZYG44DINQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thanks for reply !
It is worked .
import 'package:custom_story_editor/src/controller/controller.dart';


CTR Prediction with DeepFM model on Criteo dataset

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


example programs broken

➜  lambda git:(master) ✗ pwd
/home/omer/K/k/dist/examples/lambda
➜  lambda git:(master) ✗ ls
lambda.k  programs
➜  lambda git:(master) ✗ kompile lambda.k
➜  lambda git:(master) ✗ krun programs/capturing.lambda
[Error] Critical: Parse error: Syntax error near unexpected character 'z'
        File: programs/capturing.lambda
        Location: (1,14,1,14)
➜  lambda git:(master) ✗ krun programs/free-vars-and-var-capturing.lambda
[Error] Critical: Parse error: Syntax error near unexpected character '('
        File: programs/free-vars-and-var-capturing.lambda
        Location: (1,3,1,3)
➜  lambda git:(master) ✗ krun programs/identity.lambda
<k>
    lambda x . x
</k>

tried on current master and also on commit 9803623.

Oncall should delete this definition under examples


Remove network-instance enabled leaf.

This was removed in openconfig/public#749

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 46.215%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
580

Relevant Lines:
1255

💛  - Coveralls


add make target clean-program to makefiles

Summary:
when building a stan exe from the stan program file, make creates intermediate files.
add a make command to removes both the exe and all intermediate files.
Description:
Example - compile CmdStan's example file bernoulli.stan - note resulting intermediate files:
~/github/stan-dev/cmdstan (develop)> ls -1F examples/bernoulli
bernoulli.data.R
bernoulli.data.json
bernoulli.stan

~/github/stan-dev/cmdstan (develop)> make examples/bernoulli/bernoulli
...
--- Translating Stan model to C++ code ---
...
--- Compiling, linking C++ code ---
...

~/github/stan-dev/cmdstan (develop)> ls -1F examples/bernoulli
bernoulli*
bernoulli.d
bernoulli.data.R
bernoulli.data.json
bernoulli.hpp
bernoulli.o
bernoulli.stan
~/github/stan-dev/cmdstan (develop)>

command make clean-program examples/bernoulli/bernoulli  should remove exe, .d, .hpp, and .o files.
Reproducible Steps:
Please report steps to reproduce the issue. If it's not possible to reproduce, please include a description of how you discovered the issue.
If you have a reproducible example, please include it.
Current Output:
The current output. Knowing what is the current behavior is useful.
Expected Output:
Describe what you expect the output to be. Knowing the correct behavior is also very useful.
Additional Information:
Provide any additional information here.
Current Version:
v2.21.0

@mitzimorris, sorry, but that's not going to work with make. The space is a delimiter and will result in two targets being passed to make: 1) clean-program and 2) examples/bernoulli/bernoulli. The second target will just try to build it.
One way to deal with something like this is to use a pattern target, so something like make clean-examples/bernoulli/bernoulli. That can be matched, the dependencies found, etc.

Btw, that's just a technical limitation with make. If you find a way to work around it, please let me know! I haven't seen any way to manipulate the targets passed in well enough to handle arguments to make targets.

sure, make clean-<path/to/exe> would be fine.
the issue is being able to have a simple way to do cleanups after build - make clean-all is not fine-grained enough.

What if the name/path of the compile binary is passed as an argument?
clean-program:
	ifndef STANPROG
	$(error STANPROG is not set)
	endif
	rm -v ${STANPROG}{,.o,.hpp}

Then you could just run:
make STANPROG=/path/to/model clean-program

Here's a way to check if the passed argument is set. Might be safer to pass the name of the stanfile and then pattern match from that. Perhaps there is a better convention though. Here's an example:
.PHONY: clean-program
clean-program:
ifdef STANFILE
	$(RM) $(wildcard $(patsubst %.stan,%.hpp,${STANFILE}))
	$(RM) $(wildcard $(patsubst %.stan,%.o,${STANFILE}))
	$(RM) $(wildcard $(patsubst %.stan,%$(EXE),${STANFILE}))
else
	$(error STANFILE not set)
endif


[BEAM-10610][BEAM-9982] Support Loopback mode with universal runners in the Go SDK.

This allows loopback mode to be supported in the Go SDK. This is useful for validating pipelines against real portable runners such as Flink, Spark, and the Python Portable runner.
This PR also fixes some smaller  bugs around logging, discovered through testing.

Missing newlines when printing unsent log messages to the local machine.
A small race condition when the control channel terminates prevented orderly shutdown of workers.
Cleans up [BEAM-9982], and removes the redundant MustMarshal function from graphx.
Increases the default GRPC recv limit to it's maximum. Testing indicates that this only leads to memory overage if the inputs are actually large, and unnecessarily restricts pipeline data.

Loopback mode allows for convenient testing of pipelines, such as avoiding needing to dig into running docker VMs to access logs. Print output will appear as expected in the normal console. There are also some testing benefits such as access to the local file system for reading and writing files.
The associated risk for this convenience is access to the local state of the program. Package level/global state is accessible in the process, and thus to the local workers. As per best practices, it's best to move DoFn state into Structural DoFns. Otherwise you risk writing pipelines that depend on state that doesn't exist on distributed machines.
To use loopback mode, ensure that the runner package you're using is based on the universal runner (such as spark or flink) or use the universal runner directly. You can register the universal runner for use by beamx.Run  by underscore importing it.
	"github.com/apache/beam/sdks/go/pkg/beam/x/beamx"

	_ "github.com/apache/beam/sdks/go/pkg/beam/runners/universal"

then add the following command lines to your job.
--runner=universal --endpoint=<local Job management server instance> --environment_type=LOOPBACK
This will have the framework start an BeamFnExternalWorkerPoolServer, which will spin up new workers in the main program process, with the runner orchestrating workers in the process as necessary.

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

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

See the Contributor Guide for more tips on how to make review process smoother.

R: @youngoli
cc: @robertwb @damondouglas @codeBehindMe
The others are cc'd as this has value for setting up Python Portable Runner usage, for local testing of changes against a correct runner.
Set up usage of Beam Python per https://cwiki.apache.org/confluence/display/BEAM/Python+Tips, and in a virtualenv navigate to your repo's beam/sdks/python. You state the portable runner with python -m apache_beam.runners.portability.local_job_service_main --port=8099 which allows connections from the Go pipeline at --endpoint=localhost:8099


(PUP-7732) Acceptance audit

This is the content of https://github.com/puppetlabs/puppet/pull/6057 rebased onto 5.0.x with the rebase error mentioned in this comment fixed and resource/exec/should_run_bad_command.rb and resource/exec/should_accept_large_output.rb removed (since they are only in master).
The two exec tests will need to be updated with this is merged up to master.

I'll run this locally and report. I think you said you were okay merging this (assuming there's no failures), @er0ck ?

sounds good to me.

CLA signed by all contributors.

All passed using AIO.

are we just going with the one targeting master, @justinstoller ?
https://github.com/puppetlabs/puppet/pull/6057

There was a miscommunication and John did all the remaining work in the other branch, but still targeting master. We've both rebased our commits into existing ones and so there's not a clean way to accept changes from one or the other, and he did most if not all, of the work I put into it yesterday anyways. At this point I'm just going to step back and you and John can figure out what's what.


NETOBSERV-1933: do not set default additional namespaces for netpol

Description
Default namespaces are handled in code, do not add extra namespaces by default or they will be listed twice in the network policy

Dependencies

n/a
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

[x] Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix (in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes).
[ ] Does this PR require product documentation?

[ ] If so, make sure the JIRA epic is labelled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.

[x] Does this PR require a product release notes entry?

[x] If so, fill in "Release Note Text" in the JIRA.

[ ] Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.

[ ] If so, make sure it is described in the JIRA ticket.

QE requirements (check 1 from the list):

[x] Standard QE validation, with pre-merge tests unless stated otherwise.
[ ] Regression tests only (e.g. refactoring with no user-facing change).
[ ] No QE (e.g. trivial change with high reviewer's confidence, or per agreement with the QE team).

/retest

/approve


Add golangci-lint

This commit adds golangci-lint and fixes multiple violations. Some of
them have just been skipped by using nolint directives.
It also adds a new hack/update-deps script to update vendor
dependencies.
Signed-off-by: Daniel Mellado<EMAIL_ADDRESS>

/test-ci-job

/retest

/assign @vitus133

/assign @irinamihai

/assign @nishant-parekh

/assign @sabbir-47

/lgtm

/lgtm

/lgtm

/lgtm

/override ci/prow/integration


modifyImportRow hook does not honor Craft forced logging

It's as simple as that. I would be grateful if anyone else could try and reproduce this.
function modifyImportRow($element, $map, $data)
{
    ImportHelperPlugin::log("Forced log from modifyImportRow method", LogLevel::Info, true);
}

Oh I have. It works fine in here.
2016/09/14 14:14:45 [info] [plugin] [Forced] Forced log from modifyImportRow method
in /var/www/html/craft/app/etc/plugins/BasePlugin.php (65)
in /var/www/html/craft/plugins/importhelper/ImportHelperPlugin.php (46)
in /var/www/html/craft/app/services/PluginsService.php (582)

devMode false?

Confirmed. I did dive a bit deeper tho. When I try to force log from any "task" with devMode false it never shows up. I even injected some code in the TaskService where the steps are being ran.
Since my import plugin runs in a task, this also affects my plugin.
I'm cc'ing @takobell @brandonkelly for this.

Just tried to reproduce this with devMode off and a Craft task (ResaveAllElements) and the forced logs all came through fine:
2016/09/15 04:20:32 [info] [application] [Forced] Starting task ResaveAllElements that has a total of 6 steps.
2016/09/15 04:20:32 [info] [application] [Forced] Starting step 1 of 6 total steps.
2016/09/15 04:20:32 [info] [application] [Forced] Forced Log
2016/09/15 04:20:32 [info] [application] [Forced] Starting task ResaveElements that has a total of 0 steps.
2016/09/15 04:20:32 [info] [application] [Forced] Finished task 2 (ResaveElements).
2016/09/15 04:20:32 [info] [application] [Forced] Starting step 2 of 6 total steps.
2016/09/15 04:20:32 [info] [application] [Forced] Forced Log
...

Maybe something task specific?  Regardless, I wouldn't put too much effort into this... the 3rd "force" param doesn't exist anymore in Craft 3.

@takobell that doesn't come through here either. Anyway, lets forget about it then.


diff/fd: implement Hessian finite difference, and code cleanups.

This commit primarily adds the Hessian function for finding a finite difference approximation to the Hessian. At the same time, it combines duplicated functionality across the difference routines so that the preludes to all the difference routines look similar

PTAL @vladimir-ch
This doesn't do anything tricky with trying to prevent repeat evaluations, as you said, quite complicated. Saving O(n) evaluations isn't nothing, but out of O(n^2) computations, maybe it isn't worth the additional complexity.
I thought about the two-formula approach, but I think in the general case this is tricky with also forcing the matrix to be symmetric. If you want distinct formulas in the x and y direction, this means that you should also get different derivative estimates in the different directions. It's somewhat annoying this effectively uses Forward2nd on the diagonal entries, but the differences aren't so huge and this gives the user the option of using Central to get a better estimate. I think the only other reasonable option is to forbid setting the formula.
Fixes https://github.com/gonum/diff/issues/23

PTAL.
This needs another LGTM because it was opened before the change (I believe).

Ping @vladimir-ch

Fixed comment punctuation. You need to approve this again though now.

Done, with both reasons given. The original comment in Gradient only serves the one function since it's at the origin.


[BUG] get_json_object cannot handle ints or boolean values

Describe the bug
The legacy version of GetJsonObject works just fine, but it looks like the new version does not handle normalization/validation properly. I am also really concerned about null
Steps/Code to reproduce bug
Seq("{'a': true}", "{'a': false}", "{'a': null}").toDF("s").repartition(1).selectExpr("s", "get_json_object(s, '$.a')").collect.foreach(println)
(0 until 100).map(i => "{'a': 1" + ("0" * i) + "}").toDF("s").repartition(1).selectExpr("s", "get_json_object(s, '$.a')").collect.foreach(println)

return nulls for all of the values instead of matching what Spark does, or the legacy version.
Our tests appear to really be lacking in this area.

So it looks like most of this has been fixed in 24.06 after the upmerge went in. I will retest things

Has been fixed after this upmerge from branch 24.04 to 24.06:
Fix merge from 24.04 to 24.06
Now both branch (24.04 and 24.06) are fixed.
I'll add the test cases in this issue into Spark-Rapids.


rename hex image to match YAML value

We're using the documented YAML values as the GHCR image names since they are the official names in dependabot.yml. This one was mistakingly named "hex" so renaming to "mix."

Got a 500 on my first approval. Sorry for the noise (which I'm adding to with my apology 🙃)


182487074 earn milestone points

@jumaallan sounds good. Please coordinate with @samuelralak for making sure development is set up correctly for testing. I believe that this change https://github.com/UseHover/server/pull/155/files needs to be reverted for it to work correctly, but there may be more to it.

@jumaallan sounds good. Please coordinate with @samuelralak for making sure development is set up correctly for testing. I believe that this change https://github.com/UseHover/server/pull/155/files needs to be reverted for it to work correctly, but there may be more to it.

Will do. Already working on some implementation, then I will reach out to him during testing

Closing in favour of #801


[WIP] Fix tests

I managed to fix the auto decoder for maps (I didn't check datetimeoffset yet) on .NET side but I found another currying issue in Fable 😬 Need to fix that one...

Ok, this should be working now (although I cheated a bit again with UTC dates, please check the TODO comments), but it needs a new release of Fable 2 alpha. I updated paket.lock manually but I'm at the airport right now and cannot push a new nuget package over my current Wi-Fi connection (SSL issues). @MangelMaxime If you want you can push a new Fable release and check if this works, just pull latest changes from Fable dev2.0 branch, change the version number on top of src/dotnet/Fable.Compiler/RELEASE_NOTES.md to 2.0.0-alpha-029 and then run the PublishPackages FAKE target.

Commit fd4b683c197f9f3a39b67cb7703d73f7262d50c3 fixes DateTimeOffset decode tests, but I simplified the code for the first DateTimeOffset test and also required some tricks when parsing UTC datetime(offset), we may need to look into that later. I think @forki is using Fable 1 to JSON serialize datetimeoffsets, maybe he can tell us about what is to be expected.

@alfonsogarciacaro Is this ready to be merge ?


just git

You've got unrelated commits in this pull. Have you been merging stuff into your master branch until now? You'll probably need to create a new clean master so that you don't accidentally keep including all commits in each pull.


Standardize to IconSource over (FA)IconElement

Currently WinUI (and FA) have a mix of controls that expect either an FAIconElement or an IconSource, but neither has been standardized. Newer WinUI controls are usually using IconSource and that's what I've used in the new controls (SettingsExpander and TaskDialog) in this library since they have the ability to declared as resources and reused, whereas IconElements are controls and can only exist in one place at a time.
Of the WinUI stuff ported:
IconElement

NavigationViewItem
FAMenuFlyout items
CommandBarButton/ToggleButton

IconSource

TabViewItem
InfoBar
InfoBadge
TeachingTip

Personally, I'd like to standardize all controls to use IconSources, but I'm interested in feedback for this change. Doing this would standardize the NavigationView and FAMenuFlyout to the newer controls and leverage the features of IconSources, but would be an API change from upstream WinUI. This would also involve a name change for the properties, i.e., Icon would become IconSource which is the standard property name in other controls.
Note that if this change is made, this will be a hard breaking change for v2. I may leave the existing properties as obsolete (but non-functional) for the remainder of the previews, though.

Porting from WinUI to Avalonia + FluentAvalonia requires quite a few breaking changes already. I have given up original plans to maintain the code side-by-side. Therefore, I think changes like this -- especially to improve things -- are always justified. The goal of this project should be more to support WinUI features and control functionality, not maintain API compatibility.
The problem is if a feature or code pattern simply isn't available in Avalonia. That requires reworking things as you know which is time consuming. This seems like a simple renaming which is just fine from my viewpoint.

Happy to see this changed to use IconSource across all controls 👍

It's really useful to declare icons as resources to centralize them. That's proving to be a great simplification (outweighing breaking-change work) and makes switching icons a lot more streamlined in the future (although a script exists to do that in XAML).


Error when add and remove component while game engine paused

Current bug behaviour
Add the component to the game after pausing engine and remove it before engine resume causes assertion fails.
Expected behaviour
Prevent failing assertions.
Steps to reproduce
import 'package:flame/components.dart';
import 'package:flame/game.dart';
import 'package:flame_test/flame_test.dart';

class MyGame extends FlameGame {}

final tester = FlameTester(MyGame.new);

void main() {
  tester.test('test', (game) async {
    game.pauseEngine();

    final component = Component();
    await game.add(component);
    game.remove(component);

    game.resumeEngine();
    game.update(0);
  });
}

Flutter doctor output

✓] Flutter (Channel stable, 2.10.4, on macOS 12.1 21C52 darwin-arm, locale ko-KR)
    • Flutter version 2.10.4 at /Users/lafity101/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision c860cba910 (2 weeks ago), 2022-03-25 00:23:12 -0500
    • Engine revision 57d3bac3dd
    • Dart version 2.16.2
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at /Users/lafity101/Library/Android/sdk
    • Platform android-32, build-tools 32.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.66.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.38.1

[✓] Connected device (3 available)
    • SM G986N (mobile)           • ...               • android-arm64  •
      Android 11 (API 30)

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

More environment information

flame version: 1.0.0

Log information

'package:flame[/src/components/component.dart]()': Failed assertion: line 495 pos 12: '_state == LifecycleState.loaded || _state == LifecycleState.removed': is not true.
package:flame/…/components/component.dart:495

I think there are two problems here:

A component that is added and then immediately removed before it was mounted will produce an assertion error.
When a game engine is paused, normal component lifecycle stops working, and the components cannot be added or removed.

The second issue here is a bit more problematic, since fixing it would require us to modify the game loop itself, and instead of stopping it entirely, merely pausing the update/render invocations.
@spydon What is the level of functionality that is expected from the engine when the game loop is paused?
@Hwan-seok What do you expect to continue working while the game loop is paused?

@spydon What is the level of functionality that is expected from the engine when the game loop is paused?

We have never discussed it, I don't think I've ever come across a usecase where it has been needed.
It shouldn't be hard for the user to work around this, so I don't know if it is worth spending time on?
What is your use-case here @Hwan-seok?

@st-pasha @spydon
In my case, I have one main screen which contains gameWidget and several subroutes which can do some actions or look at the records.
The time spent on the subroutes can be both short and long(maximum 12+ hours).
Initially, I didn't pause the engine but lots of users complained about high battery consumption and overheating problem.
So I decided to pause the engine when the main game screen is covered by other subroutes(via didPush, didPopNext). As a result, the complaining rate is very lowered after pausing.
Regardless of the user's route, the user has websocket connection between server because it is multiplayer game.
This is the point.
When the user is doing action on the subroute which is paused engine, and another user came in and goes out(this causes add component and removes component).

A workaround could be to maintain a separate list of components meanwhile the engine is paused, which you can both add and remove from, and then when you resume the engine you add that component list to the game.

@st-pasha
I saw you merged that component is safely removable on an uninitialized state.
I am just wondering if you have any plan to apply this to loaded or loading?


Can not run dbt-profiler interactively in dbt-cloud

expecting to be able to run is similar to codegen:
{{ codegen.generate_model_yaml('dim_customers') }}
which when hitting </> Compile, yields an output
but when using
{{ dbt_profiler.print_profile('dim_customers') }}
I get

Relation “dim_customers” does not exist or not authorized.)

Slack 🧵  here

Tried setting the relation like {% set relation=ref(relation_name) %} got me closer, but no luck. Something is definitely different about directly invoking vs run_operation

@ernestoongaro Did you manage to get past the "does not exist or not authorized" error?

I did by replacing (in get_profile.sql)
{%-
set relation = adapter.get_relation(
  database=database,
  schema=schema,
  identifier=relation_name
)
-%}

with {% set relation=ref(relation_name) %} however after that I was still seeing other errors which pointed to my limited knowledge of how this all works

@ernestoongaro I made some changes and now the get_profile macro should work in dbt Cloud. Example usage:
{{ dbt_profiler.get_profile(relation=ref("customers")) }}

Can you verify if this now works for you?

This is working really well - tried a few relations and it looks 💯  Thank you!!!
Fixed by @stumelius in https://github.com/data-mie/dbt-profiler/pull/31


Make locale configurable

The locale is hardcoded to be Norwegian. Please make it configurable or at least inherited from system settings. Thanks!

Feel free to PR.
lør. 30. mar. 2024, 21:52 skrev Dionysis Zindros @.***>:

The locale is hardcoded
https://github.com/HelgeSverre/ollama-gui/blob/044090494a160fc373b1828b0172843b2be1ebd2/src/components/Sidebar.vue#L61
to be Norwegian. Please make it configurable or at least inherited from
system settings. Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/HelgeSverre/ollama-gui/issues/26, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIKA5HTORF2AJZGXKBTJGLY24JWRAVCNFSM6AAAAABFP2IS26VHI2DSMVQWIX3LMV43ASLTON2WKOZSGIYTMNRWGA4DKMY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

💸  5$ USD bounty

Hi @HelgeSverre , I have created a simple PR to address the issue https://github.com/HelgeSverre/ollama-gui/pull/34. Please kindly review or give sggestions.
(I don't want the bounty but just to contribute, thanks for your great work!)

Thank you for opening sourcing this.
I will fix this, and no reward needed -> just to showcasing my gratitude towards you for opening sourcing this.

@HelgeSverre
Hi there, this is done.
All you have to do is:

go to the file: "Sidebar.vue"; and
Ensure you change "no" to "en-US" as shown below, and it will convert the locale from Norwegian to English.

{{
chat.createdAt.toLocaleDateString('en-US', {
day: '2-digit',
month: 'short',
weekday: 'long',
hour: '2-digit',
minute: '2-digit',
second: '2-digit',
})
}}

I hope this helps.
Thanks again for open sourcing this code.

@dionyziz
Pls see my comment above.


documentation

use steenzout.sphinx
regenerated documentation based on steenzout.sphinx templates
apidoc is now auto-generated

Coverage remained the same at 100.0% when pulling 95a8658aa4ebbbe045bcb1106c9d5e870ae65228 on develop into c20cc15525a84b27f3c11d8fbb52da376d6a5b6e on master.

Coverage remained the same at 100.0% when pulling 95a8658aa4ebbbe045bcb1106c9d5e870ae65228 on develop into c20cc15525a84b27f3c11d8fbb52da376d6a5b6e on master.


feat: Products (dex) and Rate (trav)

For product to show in the filter_products page, when creating the product, must use these categories (case sensitive)

Tyres
Car Camera
Battery
Car Shampoo
Sound Processor
Engine Oil
Car Polish
Coilover
Brake Kit
Paint Protection
Rims
Speakers

Oh i thought the user one wasnt used and it kept throwing errors so i deleted it my bad

Also the product create i put a notice for only admins so i was thinking we only make the users with acess level admin be able to use it

Also the product create i put a notice for only admins so i was thinking we only make the users with acess level admin be able to use it

yep so we can check something like db_user[user.uid].access_level = 'Administrator' or something of that sort


pgcluu_collectd : defunct child process in Incremental mode

In incremental mode (--rotate-daily & --rotate-hourly) all child processes "pgcluu_collectd" are not stopped by the parent process.
Step to reproduce :
$ pgcluu_collectd -i 60 --rotate-hourly -p 1520 /tmp/pgcluu --compress --pid-file=/tmp/pgcluu_collectd.1.pid

*** pgcluu_collectd v2.9 (pid:25580) started at Tue May 28 14:12:18 2019
Type Ctrl+c to quit.

LOG: compressing directory 2019/05/28/14 using /bin/gzip.
LOG: compressing directory 2019/05/28/15 using /bin/gzip.

Wait an hour and run ps :
$ ps -ef | grep pgcluu
postgres 25580  3621  0 14:12 pts/0    00:00:00 pgcluu_collectd
postgres 27558 25580  0 16:55 pts/0    00:00:00 [pgcluu_collectd] <defunct>
postgres 28116 25580  0 16:56 pts/0    00:00:00 [pgcluu_collectd] <defunct>
root     28656  9423  0 16:56 pts/1    00:00:00 grep --color=auto pgcluu

Hello @darold  :)
We can't reproduce the problem with version 3.0.  You can consider this issue as fixed and close it.
Thanks again and see you soon !

Hi, thanks @daamien ! I will close it in few days, I will reopen it if @OrioQuezac can still reproduce the issue with v3.0.

Hello @darold  !
@OrioQuezac is part of dalibo, I let you guess who he is :-)
Anyway we closed this issue internally, so  we won't react further on this one.
See you soon

Ok so I guess it can be closed.


Feature Request: Add Shutdown to aggro dialog and disable enter/return & escape keys

I'd like to request a shutdown button on the aggro dialog, i.e. have the existing two buttons swap, so that Shut Down... and Restart... can sit next to each other.
In addition to this, I'd like to suggest the defaults for the Restart & (and requested) Shut Down buttons should match what appears in the Apple menu, i.e. Restart... and Shut Down..., this way it creates a more seemless connection between the button and what it does, for a consistent user interface.

Note for disabling keyboard actions...
   --hidedefaultkeyboardaction

	Hides the default behaviour of Return ↵ and Esc ⎋ keys

        When specified, the behaviour to action the default button (Return ↵) or secondary button (Esc ⎋) will be modified
        to require Command ⌘ + Shift ⇧ in order to action.

        This effectively disables unintentional activation of either of these buttons. They can still be activated
        with the keyboard, but it is harder to do so without intent.


Upgrade tested AGP versions

from 7.0.0-beta01 to 7.0.0-beta04
from 7.1.0-alpha01 to 7.1.0-alpha02

@bot-gradle test AST

OK, I've already triggered AllSmokeTestsReadyForMerge build for you.

@bot-gradle test AST

OK, I've already triggered AllSmokeTestsReadyForMerge build for you.

@bot-gradle test AST

OK, I've already triggered AllSmokeTestsReadyForMerge build for you.

@bot-gradle test AST

OK, I've already triggered AllSmokeTestsReadyForMerge build for you.

@bot-gradle test AST

OK, I've already triggered AllSmokeTestsReadyForMerge build for you.

Superseded by https://github.com/gradle/gradle/pull/17585


Add tests for all linux pack actions

The contrib linux pack has a few broken actions (check_loadavg,  check_processes, dig: #4993, diag_loadavg, netstat: #4947, possibly more) at least on CentOS 8. It's really disheartening to spin up StackStorm and immediately run into broken actions.
We should add tests for those actions to the pack.
The tests from the core pack can be used as a template for writing tests for the linux pack.
I will try to add tests for the dig action in my PR #4993, to serve as a concrete example, but adding tests would be a useful way to introduce somebody to developing StackStorm that would have an immediate impact.

Copying and pasting this from Slack:
For the netstat action there really isn’t much to do since it’s a straight remote-shell-cmd action, and it only has a single user-specified parameter, args. So this one probably doesn’t need a unit test.
Same story for the cp, mv, netstat_grep, pkill, rm, rsync, scp, and vmstat actions.
I’m a little confused why the file_touch action doesn’t use the touch command…that could probably be changed, but if you do I would rename the existing file_touch action to something like file_init.
For the service action, it’s a remote-shell-script but it’s actually a Python script, so you can probably import directly from it, mock up and monkeypatch the sys and subprocess modules in it, and then assert that the right command arguments are passed to subprocess.call.
check_loadavg needs some tests, I would mock up and monkeypatch loadAvgFile and cpuInfoFile, override sys.stdout and check its output. Once you’ve got some tests, it then needs to be updated to support both Python 2 and 3 (except Exception:).
check_processes needs tests - you don’t need to test every method, but I would test at least byState, byPid, and byName. First, choose a pid number and set the action instance's myPid property to it, then create a test fixture directory that contains directories with purely numeric names, a few with non-numeric names, and a name that matches your chosen myPid number (say: 3456. Make sure you put stat and cmdline files in each pid directory:
proc_fixture/
proc_fixture/1234/
proc_fixture/1234/stat
proc_fixture/1234/cmdline
proc_fixture/2345/
proc_fixture/2345/stat
proc_fixture/2345/cmdline
proc_fixture/3456/
proc_fixture/3456/stat
proc_fixture/3456/cmdline
proc_fixture/abcd/
proc_fixture/abcd/stat
proc_fixture/abcd/cmdline
proc_fixture/4567/
proc_fixture/4567/stat
proc_fixture/4567/cmdline

Set the procDir property to your fixture directory, override sys.stdout and check that against a known value.
For wait_for_ssh, we need to check two of the three use cases: (1) unconnectable host (SSH timeout), and (2) instantly connectable host (eg: localhost). Use test_action_dig.py as a template for this.
If you need help standing up a test environment to begin with, I would dig through the Travis config and rerun the same commands in a VM.


AttributeError: 'Github' object has no attribute 'get_org'

When invoked as:
./github-backup.py token_or_password ~./github_backup -o org_name
I'm getting
Traceback (most recent call last):
File "./github-backup.py", line 156, in
main()
File "./github-backup.py", line 56, in main
org = gh.get_org(args.org)
AttributeError: 'Github' object has no attribute 'get_org'

I have the same problem when using -o option and organization neame. Any idea how to fix it?

Getting the same error here as well.

Getting same error. Everything worked well for user. But when I added -o orgname, got the above error

I am getting the same error as well.

fixed by #34


Release Google.Cloud.Asset.V1 version 3.12.0

Changes in this release:
New features

Add IServiceCollection extension methods for client registration where an IServiceProvider is required. (commit 022fab2)

Created release for Google.Cloud.Asset.V1-3.12.0


Show install location in package info

The info page for installed packages should show where the application is installed (its path) and, ideally, display it as a link that opens file manager to that location.
This was originally submitted to the choco repo, and there's another related issue there as well.

This is related to this issue: https://github.com/chocolatey/ChocolateyGUI/issues/278
One is for a change to the package details page, and the other for list view.


calico is deployed using calico-etcd.yaml, CRD is not supported, how should I change the bgp default port 179 to another port

calico is deployed using calico-etcd.yaml, CRD is not supported, how should I change the bgp default port 179 to another port

For one, I'd recommend using the operator install which allows configuration using kubectl.
Otherwise, you can perform all of the same steps using calicoctl instead to modify BGPConfiguration.

So i you use calico-etcd.yaml deployment i can't change the bgp default port, right?

So if i use calico-etcd.yaml deployment i can't change the bgp default port, right? @caseydavenport

With calico-etcd.yaml, you can change the BGP default port using calicoctl to interact with projectcalico.org/v3 resources instead of kubectl.

Sorry, I don't know much about calico, can you give a specific example?Here is my own attempt:
kubectl-calico create -f calico-bgpport.yaml
Failed to create 'BGPConfiguration' resource: [resource does not exist: BGPConfiguration(default) with error: the server could not find the requested resource (post BGPConfigurations.crd.projectcalico.org)]

calico-bgpport.yaml:
apiVersion: projectcalico.org/v3
kind: BGPConfiguration
metadata:
  name: default
spec:
  listenPort: 1179

Check out these docs:

https://docs.tigera.io/calico/latest/reference/calicoctl/overview
https://docs.tigera.io/calico/latest/operations/calicoctl/install


Latest version doesn't give you the files

The files should be in the "Files" not just internally in "var files" which hasn't changed from Dictionary type to List.

 This actually reveals that the testing code is incorrect in this case and will require some extra time to verify that the tests are actually testing. I'll look at it ASAP when I get home tonight and fix it then.

Fixed by https://github.com/Vodurden/Http-Multipart-Data-Parser/commit/2f17700c97ca799428ec250945b3c4ed8d660bdd. It also revealed that the unit tests were flawed if the parser returned no files which I have fixed.
Thanks for your contribution!


Annotation for Authentication?

Is there any annotation I can use to specify the Authentication types(basic/api_key/oauth2) of an API?
It's documented in the swagger-spec, but not found in swagger-core.

For general declaration, there are no annotations. You would go about it in a way similar to https://github.com/swagger-api/swagger-samples/blob/master/java/java-jersey-jaxrs/src/main/java/io/swagger/sample/Bootstrap.java#L31-L35.
For operation-specific declarations, you have https://github.com/swagger-api/swagger-core/wiki/Annotations-1.5.X#authorization-authorizationscope.

@webron
I guess I need to define an authentication method first?
The problem is that I 'm using springfox to generate swagger spec, rather than hand-writing it. So how can I configure the swagger instance?

That has nothing to do with swagger-core then. You'd have to consult with springfox's documentation regarding the authentication definition.

thanks. I 've found the link on springfox, for future references I post here:
https://github.com/springfox/springfox/issues/821

My web service need to authorize but swagger did not have a header blank to input header parameters.So my request is 401,how to solve it?


OSError: [Errno 5] Input/output error

Hello,
Im trying to train VGGVox model but sometimes I got this error
OSError: [Errno 5] Input/output error
This is pretty annoying because sometimes it leads to this 'ZeroDivisionError: division by zero' which stop my trainning.

Are there anyway can I fix this problem
How can I save the process if problems occured ( crashing, power cutting )
I have been trainning for 1 day and this happened :(
And Im using the config ( on readme files ) for trainning VGGVox, does this affect the result much?

Thanks you very much,
I Hope you have a nice day,

I have never heard of this problem before. What is your operating system and GPU?

I solved this problem. Im currently running a dual boot ( Windows / Ubuntu ). I put the dataset in Windows partition so the files might be missing or this problem may occurs.
Thanks for sharing this project


Responsive clr-vertical-nav won't close

Describe the bug
Following how VMWare's website works, I wanted to use different clr-vertical-navs depending on which page you were on. But there will be pages where the same nav is shared. The nav works fine when it's not hidden on larger screens.
The problem presents itself when the nav is hidden. After clicking the hamburger icon and opening the nav, click on a link and notice that the navigation happened but the nav is not closed?
How to reproduce
https://stackblitz.com/edit/clarity-light-theme-v1-0-476tst
Steps to reproduce the behavior:

Visit https://stackblitz.com/edit/clarity-light-theme-v1-0-476tst on a small screen.
Select the hamburger icon to open the nav.
Select a link and notice the navigation happens but the sidebar won't close.

I also tried this with the sidenav component and got the same results.
Expected behavior
The sidenav should close after selecting a link.
Versions
App

Angular: 7.1.3
Node: 8.11.12
Clarity: 1.0.2

Device:

Type: Laptop
OS: Windows
Browser: Chrome
Version: 71

The solution by gnomeontherun worked perfectly for me, thank you


start streaming with some initial values

I want to start streaming from current time but the graph should not be blank initially. I want some values to be there. How to do this? I tried supplying 4 values in data array as x,y but it didn't work

Try this:
<div>
  <canvas id="myChart"></canvas>
</div>
<script<EMAIL_ADDRESS><script<EMAIL_ADDRESS><script<EMAIL_ADDRESS><script src="https://cdn.jsdelivr.net/npm/chartjs-plugin-streaming@next"></script>
<script>
const myChart = new Chart(document.getElementById('myChart'), {
  type: 'line',
  data: {
    datasets: [
      {
        data: [
          {x: Date.now() - 4000, y: Math.random()},
          {x: Date.now() - 3000, y: Math.random()},
          {x: Date.now() - 2000, y: Math.random()},
          {x: Date.now() - 1000, y: Math.random()}
        ]
      }
    ]
  },
  options: {
    scales: {
      x: {
        type: 'realtime',
        realtime: {
          onRefresh: chart => {
            chart.data.datasets[0].data.push({
              x: Date.now(),
              y: Math.random()
            });
          }
        }
      }
    }
  }
});
</script>

This has been open for quite a while. Has anyone figured it out?
I'm doing the exact same thing, trying to preload a minute of data. New data shows up on the right, but none of the preloaded data fills in.
I'm also loading data using only the onRefresh, pulling from a server. The initial hit gets the previous minute of data but subsequent hits pulls the new data. Debugging calls show that the data is added to the chart in the onRefresh, but it just never shows up. Only new/current data is plotted.


samsung-tv 0.3.0

use the latest stable version of serialport (4.0.3)
catch uncaught error
fixed serialport implementation
fixed checksum calculation
remove array length restriction

@phillipsnick don't forget to publish the new release on npmjs.com ;-)

Oops, done :)


[hi] Update /data/i18n/hi/hi.toml with new strings from en.toml

Fixes https://github.com/kubernetes/website/issues/36203
Needs language review from Hindi reviewer.

Thanks for the effort!

Thank you both for your comments, @anubha-v-ardhan @AmanSarraf. I've updated the contents and I believe this PR is ready to be reviewed for merge.

Thank you for your thoughtful review, @Garima-Negi @Babapool! I hope now this PR might be ready for merge.

I have verified that this PR merges smoothly and that the site builds and runs smoothly. I believe this PR is ready for merge.


Error when saving the command in a temporary file with Windows

I am using git-town on Windows, and this issue is only related to this platform.
For most of the commands, I always get an error at the end of the computation, because it cannot find the temporary directory. I looked into the source code and found this in the file step.go:
return fmt.Sprintf("/tmp/%s_%s", command, directory)
I think it could be cool to change the hard-coded /tmp/ to the location of the temporary directory for the different main OS.

In the newly released 4.1.1, the top directory is created if it does not exist. Currently tmp is hardcoded and looking at the definition of golang's 'os.TempDir' I believe it would still return '/tmp'. To you knowledge is the temp directory elsewhere on windows?

I found that the temporary directory for Windows is under C:\Users\$USERNAME$\AppData\Local\Temp, which is set as an environment variable %temp%.
I have never written anything in go, but to my knowledge, I have never seen a langage made the automatic correspondence between the path to Unix system and Windows.

@hammenm which Git Town version do you use? You can find out by running git town version.

V4.1.0
I will try the new version now.

Ok, it works on 4.1.1
Thank you for your time 👍

Great to hear, thanks!


Remove deleted messages from the unread message count

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/5237
Tests | QA Steps

Open the App on the web with account A.
Send a few messages. Count the number.
Delete some of the recently sent messages. Count the remaining number.
Log out, log in with account B for the same Chat.
Count should be correct for unread messages.

Tested On

[x] Web
[ ] Mobile Web
[x] Desktop
[ ] iOS
[ ] Android

Screenshots

Web

https://user-images.githubusercontent.com/24370807/140608201-4fb6c060-8222-4e5a-afff-a141bd0cfc4d.mp4
Mobile Web

Desktop

iOS

Android

Awaiting review....

@parasharrajat a couple interesting things I noticed while testing this:

When you mark a chat as unread, the unread count is immediately updated in the tab bar. However, when you delete a comment, the unread count in the tab bar in the current tab isn't updated. It is correct if you open in a new tab, but it doesn't update the existing tab. Why is that?
When you open the app in the new tab, the tab bar shows the proper unread count, but the "new" line that indicates the point at which the messages are unread from isn't shown. Do you want to raise a new issue for this.

When you mark a chat as unread, the unread count is immediately updated in the tab bar. However, when you delete a comment, the unread count in the tab bar in the current tab isn't updated. It is correct if you open in a new tab, but it doesn't update the existing tab. Why is that?

I think because we manually set the count. And don't do that when a message is deleted. Do you want me to look into that as well in this PR?

When you open the app in the new tab, the tab bar shows the proper unread count, but the "new" line that indicates the point at which the messages are unread from isn't shown. Do you want to raise a new issue for this?

I noticed this too. The new line indicator is broken again and it is not predictable. Sometimes shows and somtimes not. I was about to raise this as well after review.

Do you want me to look into that as well in this PR?

Yes, I think we should do that as well. We can increase the payment by $250 to account for the increase in scope. cc @mallenexpensify

I was about to raise this as well after review.
Great!

The code generally looks good to me. I'll give it another review after that update. Thanks!

Sure, Let me fix that.

@puneetlath I have fixed it. Please review.

@parasharrajat I left you two minor comments, otherwise it looks good to me. Let me know what you think and then I'm happy to merge.

@puneetlath Updated those. Thanks for pointing it out.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @puneetlath in version: 1.1.15-18 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @roryabraham in version: 1.1.16-10 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Not deleting anything

Facebook may have changed something on their end, deleter.go is no longer finding any elements to delete.
Searching elements from 2020:
Jan Feb Mar Apr May Jun Jul Aug Sep Oct Nov Dec   Elements found:       0
Deleting elements from 2020:
0 [________________________________________________________________________________________________________________________________________________________________________________________] ?% ? p/s 0s

It doesn't seem that anything was changed.
I fixed a bug on master. Could you try if that solves the problem?

i tried the new update, still no luck.  I wonder if this isn't also a rate-limiting issue?  It's finding zero elements to delete.

Are there any debug commands I can run for you?

You could potentially be running into https://github.com/marcelja/facebook-delete/issues/5

@BTolbert65 i found it by throwing print statements here https://github.com/marcelja/facebook-delete/blob/master/deleter.go#L60-L86 specifically https://github.com/marcelja/facebook-delete/blob/master/deleter.go#L85. Since Facebook doesn't give a good API for doing this stuff, regardless of what's happening really Facebook is going to return a 200, so i was just control-f'ing the body as I printed them out. It'll make your life a little easier if you only choose one year and category while debugging

Whatever update was made last evening resolved this issue.


Updated NA part for bumblekite (summer school)

Added Deadline and Fee

Thank you for the update!


Integration Test - Release - 2018/08/15 14:57:28

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@PRMergerTest3 : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.


15

Can one of the admins approve running tests for this PR?

@nuci approved

@nuci approved

!@nuci approved!


Missing value() const && overload

https://github.com/beman-project/Optional26/blob/bfeb9080c0d9c501965ae857a1decf7ed74686f5/include/Beman/Optional26/optional.hpp#L468-L484
There should be four value() overloads.

Similarly, or_else only has two overloads, not four.

operator* is missing the const&& overload.


Invalid JWT access token in X-MS-TOKEN-AAD-ACCESS-TOKEN header

My web application is deployed on Azure app service and using EasyAuth feature of app service. Authentication is setup with aad provider option. The app code (middle tier) http request (i.e. at controller level request object) includes below headers:
X-MS-TOKEN-AAD-ID-TOKEN
X-MS-TOKEN-AAD-ACCESS-TOKEN
X-MS-TOKEN-AAD-EXPIRES-ON
X-MS-TOKEN-AAD-REFRESH-TOKEN
X-MS-TOKEN-AAD-ACCESS-TOKEN header value is not a valid jwt. Verified on jwt.io however X-MS-TOKEN-AAD-ID-TOKEN header value is a valid jwt.
How to correct X-MS-TOKEN-AAD-ACCESS-TOKEN header?
I do not have any custom code in application, totally relying on EasyAuth.

Azure service: Azure app service and azure AD
A link to the published documentation article:
https://learn.microsoft.com/en-us/azure/app-service/configure-authentication-oauth-tokens#retrieve-tokens-in-app-code
https://learn.microsoft.com/en-us/azure/app-service/overview-authentication-authorization

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

ID: e194e752-fc11-446c-e043-31311238edca
Version Independent ID: 52455ec5-3a4c-f256-05db-dec0dac6c9eb
Content: OAuth tokens in AuthN/AuthZ - Azure App Service
Content Source: articles/app-service/configure-authentication-oauth-tokens.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@mdv27
Thanks for your feedback! We will investigate and update as appropriate.

Hi @mdv27, X-MS-TOKEN-AAD-ACCESS-TOKEN might not always be a JWT token that be decoded, see Microsoft identity platform access tokens for more information. Its primary purpose is allowing access to resources the app registration, e.g. a backend API. This is handshake is done by the EasyAuth middleware when the user successfully logs into your web app.
If you have any further questions, please feel free comment and tag me.

Thanks for reply @RyanHill-MSFT . I learned that when authentication enabled for azure app service with azure AD provider, it uses V1 for auth flow. In V1, resource is required as query param for accessToken to be a JWT. In my case UI and web api belong to same app registration. In this case using client id as resource made the trick.
I required accessToken in web api to use for subsequent OBO flow for downstream api call.


Android spell check suggestions should appear in the keyboard when supported

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
Flutter currently shows Android's spell check suggestions in a context menu, which is out of date.  This is what it looks like in Flutter now:

Proposal
As of the last few versions of Gboard on native Android, the spell check suggestions should appear here:

I believe this might be present in 3.10. @camsim99 do you know if this bug is already implemented?

I believe this might be present in 3.10. @camsim99 do you know if this bug is already implemented?

This is not implemented; this is referring to the suggestions showing within the keyboard itself.


0.6 maintenance

Hi! Thank you for helping out with Embedded Graphics development! Please:

[ ] Check that you've added passing tests and documentation
[ ] Add a simulator example(s) where applicable
[ ] Add a CHANGELOG.md entry in the Unreleased section under the appropriate heading (Added, Fixed, etc) and appropriate crate (embedded-graphics, simulator, tinytga, tinybmp) if your changes affect the public API
[ ] Run rustfmt on the project
[ ] Run ./build.sh (Linux/macOS only) and make sure it passes. If you use Windows, check that CI passes once you've opened the PR.

PR description
I wasn't sure if this branch is the best to open the PR against, but this is my best guess. I've seen the discussion in #288 and I strongly believe a maintenance branch and manual releasing instead of cargo-release (although I'm not familiar with that but I see no reason why that shouldn't work either) should be the simplest solution for a bugfix.
I'm still looking for code that could be backported, but I'm also trying to do the least amount of work because there was a significant amount of refactoring done since 0.6.2.
My reasoning behind this PR is that embedded-layout kinda-sorta relies a LOT on bounding box sizes currently and so a point release would make my life much easier while coding against the latest stable.
Closes #377

This PR is a bit confusing as there are two commits in it that probably shouldn't be here. Can you rebase them out or something? I think the only commit that should be present is 48513c4, right? The CI build also failed with some broken tests from the cherrypicked fix.
The previous 0.6 backport/release process was pretty painful which is likely why you're seeing lots of weird commits as you mention in #377.
I'm certainly not opposed to getting this fix into 0.6 unless @rfuest has any objections, but it does raise the question of whether to forego a 0.6 point release and release 0.7.0-alpha.1.

Certainly, there was no perfect point to branch off, and to open the pr against. I'm not sure about the timeline of 0.7 and I don't want to rush that at all, but since that may be still some tume away until stable, I really hope a bugfix release could be possible.
This PR was not yet about the code, I'm planning on collectung as many fixes as possibe, so don't worry about the specifics yet.

I'm not sure if we should fix this in 0.6. There are some crates that expect this wrong behavior and changing it should be considered a breaking change IMO. That was the reason I decided to postpone these changes to 0.7 and not include any geometry changes in minor 0.6 releases.

I did consider that but decided it might be minor enough to release in a 0.6 point version, but on second thoughts that would be unwise.
I've also opened #380 to track what can go in a next alpha release FWIW.

All right, sorry for wasting your time on this, then. Looking fwd for 0.7 ;)

Not a waste at all! I'm sorry you didn't get the resolution you were looking for. Hopefully future versions of e-g will have fewer breaking changes so releases don't take so long, or are at least easier to break into patches.


Name-Registry: accept multiple assets for payment

Type of change

New feature

Changes
The following changes have been made:

New functions: set_asset() to add a new payment method & rate() to check the cost
Adjusted extend() & register() to allow the user to specify an asset for payment
SRC-2 docs

Related Issues
Closes #699

I don't believe offering the ability to pay with multiple types of tokens really enhances the application in any meaningful way. Tokens historically have been volatile and the ratio between them can fluctuate rapidly. By offering multiple token payments a user may be over or under paying by using one token over another. This is bad for UX.
Furthermore, I don't see how this is particularly relevant to an example of a domain name resolver application.

You've countered your own point regarding volatility. Whether you have 1 asset or 2 makes no difference. Both are volatile.
Being able to pay for something in multiple ways is inherently an improvement. I don't think that needs an explanation.
Over / under paying isn't an issue. Pick the asset that you want. You can check the price / the app would tell you how much you are paying.
As for the last sentence, it's just an improvement to the scope of the app and not specifically the design of the registry. It's easier for people to pick this up and use it as a starting point, not that that was my reasoning for adding this in.

Wait wtf why did the tests pass here, noooo

Can you update the versions in the README. Otherwise looks good

Sorry, could you elaborate on why the versions need to be updated? I have not changed the versions in this PR.

Can you update the versions in the README. Otherwise looks good

Sorry, could you elaborate on why the versions need to be updated? I have not changed the versions in this PR.

My mistake, apologies


FTL bug

There's a chance for players to be trapped in FTL space. Logs show something related to worldgen

Can't replicate


Remove mention of absolute vs relative path from workspaces doc

Changes
Fixes https://github.com/tektoncd/pipeline/issues/3851
Similarly to commit d80cbc our workspace docs stated that if you provide
a relative path to the workspace's mountPath then it would be appended
to /workspace.
This was never the case and so this commit removes mention of it.

/kind documentation
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices
[x] Release notes block has been filled in or deleted (only if no user facing changes)

Release Notes
Removed incorrect doc that stated workspaces with relative mountPath would be mounted relative to /workspace

Looks good.

/lgtm

thanks @sbwsg
/approve


Merge branch 'issue-1085'

https://github.com/iushankin/swagger-core into merging-issue-1085
Conflicts:
modules/swagger-jaxrs/src/main/java/io/swagger/jaxrs/Reader.java
modules/swagger-jaxrs/src/test/scala/ReaderTest.scala

this supersedes https://github.com/swagger-api/swagger-core/pull/1126


Fix Ollama support/support new Ollama chat API.

Ollama is not working correctly with Amica.
Recommend using Chat API.
Ollama has a Chat API as of v0.1.14
Ollama also has support for images via the Chat API as of v0.1.15

Thanks for your work on this!

Strangely, Ollama works with Amica (on a Macbook Pro M1) if I run it using flask and visit the app using Firefox, but the .dmg does not work. I'm getting TypeError: Load failed after every message I try to send. The error appears instantly (as opposed to the usual response, where ollama slows my computer down for a bit). I checked the Ollama server logs, they respond with a 403 error.
It might be an ongoing CORS problem with Ollama. This is the most relevant link I have found so far: https://github.com/jmorganca/ollama/issues/300
This I have no idea what is the trouble. Might figure out how to use and debug Tauri.
Should I open a new issue for this? Seems to be a Mac-only problem.

I opened an issue for this: https://github.com/semperai/amica/issues/81


[skip ci] get_ci_vm.sh: Fix conflicting homedir files

Previously, the script would bind mount the user's home directory into
the container in order to execute gcloud commands.  This was done
to preserve the .config/gcloud directory and new ssh keys in .ssh.
However, it's possible the user has modified .bash* or .ssh/config
files which do not play nicely with gcloud and/or the container.
Fix this by mounting the existing temporary directory on the host, as
the user's home directory.  Then bind mount in a dedicated gcloud/ssh
sub-directory, and the libpod repo directory on top.
The gcloud tool takes minutes to setup/manage its ssh-keys, so preserving
that work between runs is a necessary optimization.  Similarly, saving the
.gcloud directory prevents repeatedly going through the lengthy
client-auth process.
Overall, these changes make the container environment much more selective
with the host-side data it has access to use/modify.  Preventing unrelated
details from getting in the way, and preserving only the bare-minimum of
details on the host, between runs.
Signed-off-by: Chris Evich<EMAIL_ADDRESS>

@edsantiago @baude PTAL.  This should fix the problem of host-side .ssh/* and .bash* files breaking expectations in the gcloud_centos container.  I tested it with a new $HOME/.config/gcloud directory, and it seems to behave.  Please let me know if these changes work for you.

hint: Give this PR a spin from your libpod repo clone with:
git fetch upstream refs/pull/2433/head && git checkout FETCH_HEAD

...or, if you have git-extras installed: git pr 2433 upstream (just plain git pr 2433 if your origin == upstream)

(squash and --force push)

thx @edsantiago

thx @edsantiago

Updated w/ @edsantiago rename TARBALL_BASENAME -> TARBALL suggestion.  Force pushed (though untested).

Okay, tested with rhel-7-libpod-7f4cd1f7 and it seems to still work post-rename.

/lgtm
...with the caveats that, first, my lgtm is worth nothing since I have no privs on this repo; and, second, my lgtm applies only to 0457913ba18c74504d87ba1dd009d1221916e0b3 and not to a variant of it discussed in #podman that I cannot reproduce and do not understand. In particular, I'm reluctant to approve a proposed change in which line 145 is amended to include $LIBPODROOT or any variation thereof.

/lgtm

/hold one-quick-sec

/hold

okay! once again for the forth time :smile:

/unhold

thx again @edsantiago

/retest

/unhold one-quick-sec

(this one's ready to go now)

Lets wait til 1.1 is released.

(rebased & force-pushed)

(rebased & force-pushed)

/lgtm

/unhold

/hold cancel

Stupid automation, I hate this stuff.
/woof

@cevich Do we still want this one?

@rhatdan yes please :smile:

/approve


(fix:#51) Refactored PBParam to PBVariable

This PR aims to solve the issue #51. Changed the name of the class PBParam to PBVariable and made the necessary changes in other dart files.

Sweet, at first glance. This looks good. We're getting together today to test and merge all PRs today. The only thing is that we have to update the parabeac_eggs and parabeac_egg_template repo. Let us know if you'd want to take a whack at that.

@siliconivan, thanks! Surely, i'd give it a go.


CSIRO, TERN, DCCEEW & Python code updates

CSIRO: Soil ontology and vocabulary patterns and tests for ANSIS / Fed Uni & CSIRO.
TERN: dataset redirects tidy
DCCEEW: BDR Dataset stub redirect
Python code: Also included formatting-only changes to test reports to assist with interpretation.

Deployed.


feat: early success in offer modals

PR Type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Context
all offer modals (create, cancel, accept) now use the correct flow of signing and notification with early success once the transaction is in block
cc @exezbcz  visual ok plz

[x] Closes #10952

Screenshot 📸

[x] My fix has changed something on UI;

https://github.com/user-attachments/assets/05f83a99-d02e-45e6-b12e-e09f46b2603d

quick and easy, nice


Unable to view historical headshots

Related to #22, we currently have no way to view historical headshots. Generally this is probably fine, but there are a handful of cases I can think of where we'd want to be able to look at the pictures from before things changed.

This is closed by 19bfd5e because we can see screenshots of historical stuff. The headshots portion of the /host//headshots will probably get an update in the future to provide a gallery of all previous screenshots in reverse chronological order. But for now I'm going to consider this closed.


Deal with character ids.

So far, all ids have been numeric - this causes issues in a few places. Notably, the 0 outlet assumption.

This has been addressed in practically all cases.


"new String()" as 'string' instead of 'object'

With all the dazzle about the promises, i totally forgot about the other Spec we were talking with new String() returning type string instead of object. I changed that now and also added a test to make sure new Number() actually returns type number.
Sorry for this :D

Coverage remained the same at 100.0% when pulling 6867882ab4a1df4c8700eceda4648007d557040c on Charminbear:string-object-as-string into 2892123c78f57943e1be95c2e69b0d7b24035e65 on chaijs:master.

Coverage remained the same at 100.0% when pulling 7ea0a920ec24706fe7476363de5718b4916afe71 on Charminbear:string-object-as-string into 2892123c78f57943e1be95c2e69b0d7b24035e65 on chaijs:master.

:smile:


sentry: Set environment to staging on staging hosts.

Splitting exceptions out between staging and production provides
useful filtering.

Alternately, we could use deploy_type as the Sentry environment, but that seems like overloading things.  Or we could have a separate sentry.environment property in zulip.conf but that seems like overkill.

Merged, thanks @alexmv!  I think it's probably fine given there isn't an easy way to do something cleaner.

Though now that I think about it ... could we just pass deploy_type here directly?

We can do:
diff --git zproject/computed_settings.py zproject/computed_settings.py
index 02a650e584..a4364907fc 100644
--- zproject/computed_settings.py
+++ zproject/computed_settings.py
@@ -1185,4 +1185,4 @@ SENTRY_DSN = os.environ.get("SENTRY_DSN", SENTRY_DSN)
 if SENTRY_DSN:
     from .sentry import setup_sentry

-    setup_sentry(SENTRY_DSN)
+    setup_sentry(SENTRY_DSN, get_config("machine", "deploy_type", "development"))
diff --git zproject/config.py zproject/config.py
index ae7473b150..42ec637b74 100644
--- zproject/config.py
+++ zproject/config.py
@@ -9,7 +9,6 @@ config_file.read("/etc/zulip/zulip.conf")

 # Whether this instance of Zulip is running in a production environment.
 PRODUCTION = config_file.has_option("machine", "deploy_type")
-STAGING = config_file.get("machine", "deploy_type", fallback="") == "staging"
 DEVELOPMENT = not PRODUCTION

 secrets_file = configparser.RawConfigParser()
diff --git zproject/sentry.py zproject/sentry.py
index a260f6bc42..52caa1253b 100644
--- zproject/sentry.py
+++ zproject/sentry.py
@@ -57,16 +57,9 @@ def add_context(event: "Event", hint: "Hint") -> Optional["Event"]:
     return event

-def setup_sentry(dsn: Optional[str], *integrations: Integration) -> None:
+def setup_sentry(dsn: Optional[str], environment: str) -> None:
     if not dsn:
         return
-    if PRODUCTION:
-        if STAGING:
-            environment = "staging"
-        else:
-            environment = "production"
-    else:
-        environment = "development"
     sentry_sdk.init(
         dsn=dsn,
         environment=environment,
@@ -75,7 +68,6 @@ def setup_sentry(dsn: Optional[str], *integrations: Integration) -> None:
             DjangoIntegration(),
             RedisIntegration(),
             SqlalchemyIntegration(),
-            *integrations,
         ],
         before_send=add_context,
         # Because we strip the email/username from the Sentry data

which muddies the meaning of machine.deploy_type a bit, and means the legacy voyager value would show up as that in the environment, rather than production.


Support python 3.12

Some packages' version can't be installed in python 3.12. Need to modify requirements.txt to support python3.12.
numpy==1.24.4
scipy==1.10.1

Hi @ttt-77, Can I help with this?

Yes

Hi @ttt-77, I was able to update numpy and scipy to their Python 3.12 compatible versions, but there the following required modules are blocked for Python 3.12:
ChromaDB:
Requires onnxruntime>=1.14.1 which does not support Python 3.12 yet:
https://github.com/microsoft/onnxruntime/issues/17842
DuckDB:
DuckDB v0.9 does not yet support Python 3.12. The next version, v0.10, will support Python 3.12.
https://duckdb.org/docs/api/python/overview.html
faiss-cpu:
faiss-cpu support for Python 2.12 is also blocked:
https://github.com/kyamagu/faiss-wheels/issues/87
Numba: The issue for numba is still open, I couldn’t test the workaround on 3.12 due to the other blockers.
https://github.com/ddkang/aidb/issues/105
The following modules need to be updated to the specified versions to be 3.12 compatible:
asyncpg==0.28.0 → 0.29.0
greenlet==2.0.2→3.0.3
numpy==1.24.4→1.26.3
scipy==1.10.1→1.12.0


Simple Usage Resulted in Android Build Failure

TNS Version: 3.2.1
Platform: Android
I have followed the documentation but whenever i try to run tns build android
it resulted in the following error
Could not resolve all dependencies for configuration ':_F0F1F2F3F4DebugApkCopy'.
   > Could not find com.google.firebase:firebase-messaging:12.0.1.
     Required by:
         project :
   > Could not find com.google.firebase:firebase-core:12.0.1.
     Required by:
         project :

and here is my app.gradle file
dependencies {
	compile 'com.android.support:recyclerview-v7:+'
}
android {
  defaultConfig {
    generatedDensities = []
    applicationId = "com.foo.bar"
  }
  aaptOptions {
    additionalParameters "--no-version-vectors"
  }
}

@anaszgh I would advise using the latest version of the plugin with the latest version of nativeScript (tns-core-modules and tns-android). Apart from that try this solution.

I'm closing this issue due to inactivity, feel free to reopen it if needed.


New Babel API?

Hi,
I get this error using <script type="es6">:
Error: The node API for `babel` has been moved to `babel-core`.

Am I missing something, or is it due to Babel moving it's API into babel-core?
Cheers!
Edit: Looks like this is riotify's<EMAIL_ADDRESS>Edit 2: Maybe not, seems I have other issues...

Seems riotify might need an update after all...
https://github.com/riot/riot/issues/1325#issuecomment-155050525
Cheers again.

Yeah, it's time to update!
Riot 2.3 has introduced submodules and some new APIs.
Will do, in a few days ;-)

@cognitom any help needed on this?

@gnimmelf requiring "babel-core": "^5.8.0" in package.json should fix the issue.

Any progress here? How can we help?

Oh, sorry. I'm not working on this now. Feel free to send PR. I'll merge that ASAP. @axe312ger

@cognitom this one should be enough to catch up https://github.com/jhthorsen/riotify/pull/24

@borodean oh, thanks. I'll check it soon.

Nice, I'll assume this is getting fixed. Closing now.

Well id did not work for me with the PR from #24 and Babel 6. I will try again later and post my errors.


[Improvement-15719] Remove the useless methods in ProcessUtils

Purpose of the pull request

close: #15719

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 39.06%.  Consider uploading reports for the commit 3c0477f to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##                dev   #15720      +/-   ##
============================================
- Coverage     39.09%   39.06%   -0.04%
+ Complexity     4853     4845       -8
============================================
  Files          1316     1316
  Lines         45014    44927      -87
  Branches       4819     4786      -33
============================================
- Hits          17599    17550      -49
+ Misses        25506    25473      -33
+ Partials       1909     1904       -5

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


Video trim mode broken after introduction of collapsing carousel

Bug Description
The trimmer for video trim mode now attaches completely weirdly and pushes the trimmed video element to a weird place.
Expected Behaviour
Trim should work correctly.
Steps to Reproduce

Enter video trim mode

Screenshots

Verified in QA


As an admin, when I merge people I want unique identifiers stored and used for redirects on public views so that links for merged people continue to resolve.

testing notes
The scenario for this is if someone cites or links to a member page and we change or merge records in the database, we want that link to still resolve if possible. The easiest way to test it might be this: choose a person record on the test database and open up multiple different person detail pages in different tabs (biography, membership activities, borrowing activities, card list, and one card image viewer page) so that you have URLs ready to test after you make changes. Then test that:

when you merge person records, the merged person's slug is stored as a past slug on the remaining record — it should be visible but read only in the admin edit page
all member pages for the old slug should redirect to the new slug — you should see the content instead of a 404, and the location in your browser should show the new slug
test that if you edit a person record and change the slug, the old slug is stored as a past slug
test that if you add a new person with a previously used slug, you get a validation error

dev notes

[x] add alternate/old slugs field to person
[x] update person merge method to store old slugs
[x] permanent redirect on old slug match for all member detail pages
[x] make old slugs visible (read only) in admin
[x] validate new slugs to avoid collisions with old slugs
[x] add old slug on save if slug changes

YES, when I merged Ellen Wright with Julia Wright, I saw Ellen's past slug on Julia's admin page:

YES, all the tabs for Ellen Wright that I had open (biography, membership activities, borrowing activities, card list, and one card image viewer page) changed to Julia Wright when I refreshed the pages. No errors!
YES, if I edited the slug, I saw that there were now _past slugs (plural) in the admin
NO, when I used a previous slug--the original Ellen Wright slug--it saved and I got no validation error.

Ha, my instructions weren't detailed enough and that was a good thing to test.
So — the system should allow you to re-use an old slug associated with the same record (e.g. if you merged and wanted to switch to one of the other slugs). That's probably a good thing to test if you haven't already (but I think you did?).
What it should not allow is using a past slug from one member on a different member. So, you can test that by trying to use that wright-ellen slug for a different person.

@rlskoeser I see! Thank you for clarifying!

YES, if I add the slug wright-ellen to another person (not Julia Wright, who I merged with Ellen), then the admin page does NOT allow me to save; I get an error. I tried to add Ellen's slug to Bernice Abbott and it didn't allow me to.

@jkotin I'm confident with Cate's testing, but you might want to read over the testing notes and Cate's comments so you know how this works and can think about whether we need something similar for books.

@rlskoeser I do think we'll need something similar for the books, although merging books will be less frequent than members, I expect.  But it will happen.

@jkotin — thanks, that is what I suspected. I think it should be simpler to implement since there is no merge method for books and I can adapt what I've done for the person records.


Create object-storage-yaml

For the beginning I just created some questions, but I can't train it, because of lab installation issues.

Thank you for your contribution to InstructLab! Unfortunately, once a Github repo is made public, all open PRs are automatically closed since they link against a private repo. We have detected that your PR might have been one of the ones affected by this change. If you are still interested in contributing your improvement, please fill out the following short form by no later than May 3rd, and we will get back to you with the additional steps necessary once we have had time to assess the PRs of those still interested:
https://forms.gle/V7SrPPMZDo6iGDYu8


Suggestion: Give caddy a default welcome page like "Caddy works!"

hi,
when I first download caddy install package, unzip it, and execute "./caddy", it works. but when opening "localhost:2015", it display "404 not found " in my browser . This is not a good experience for caddy beginner. I suggest to give a default welcome page like "caddy works!" , just like apache2。

Thanks, I thought of this as well. It is not a difficult task and we will work on it.
However, if you think you want to help us with this, feel free.

After having a second thought on this, I do not think this PR (https://github.com/mholt/caddy/pull/101) is needed. Apologies.
As Caddy grows, we will eventually have setup scripts that will create default configurations and do other things like putting caddy in path and installing it as a service.

Tony, thanks for your comments (and your pull request). I can see why a 404 message isn't the friendliest thing for beginners, so thanks for pointing that out.
I've given it some careful consideration and I think we should leave this as-is.
The fact that a 404 message appears already indicates that the server is responding to requests at that host and port, so it is working. I also don't think we need follow the footsteps of Apache, since Caddy works differently. When Apache shows you the default "Apache works" page, it means you haven't yet configured it. Caddy, however, needs no configuration, so showing a "Caddy works" page is more likely to cover up a misconfiguration rather than point out a lack of configuration.
If I run Caddy from my site's folder, then load my browser and see "Caddy works," I'll immediately be confused because my homepage should be on the screen instead, leading me to think that Caddy doesn't work - and I'll be even less informed to fix the problem, which is that my index.html file was actually named imdex.html by accident.
There's also the case where the site owner may want to show a 404 at the site root. The troubling thing about the PR is that there is no way to configure or disable the default page. With the PR, there is no way for the site owner to turn off the page at "/", even by deleting all the files. When this is intentional by the site owner, showing a "Your web server works" page to visitors is kind of embarrassing, because again, it implies a lack of configuration on the sysadmin's part.
I hope that this makes sense. Again, thanks for your feedback and taking the initiative to open a PR!
Maybe we should update the "Getting Started" page to explain that if you see 404 Not Found, then you need to place an index.html file in that directory.

Thanks for all comments for my issue and pr. caddy is a really easy to use. I will go on to focus the dev of caddy. Explanation in "getting started" page is a not bad idea, but I am still looking forward to a better solution for this in the future.  we all hope caddy become better and better.

:+1: Definitely looking forward to your help!
I just made the change to the Getting Started page - it will go out with the next site deploy. I will close this issue, but have it noted for future consideration. Thanks!


Appending fetch() result lists to response

Apologizes if this is in the wrong section or unrelated to direct issues with aqueduct.
I am trying to respond with JSON after a Postgres query:
final postQuery = Query<Post>(context);
final posts = await postQuery.fetch();

return Response.ok(posts);

This works perfectly fine. However, if I try the following:
final postQuery = Query<Post>(context);
final posts = await postQuery.fetch();

return Response.ok({ 'items': posts });

I receive this error message when sending request to server:
Converting object to an encodable object failed: Instance of 'Post' #0      _JsonStringifier.writeObject (dart:convert/json.dart:703:7)
#1      _JsonStringifier.writeList (dart:convert/json.dart:753:7)
#2      _JsonStringifier.writeJsonValue (dart:convert/json.dart:735:7)
#3      _JsonStringifier.writeObject (dart:convert/json.dart:693:9)
#4      _JsonStringifier.writeMap (dart:convert/json.dart:786:7)
#5      _JsonStringifier.writeJsonValue (dart:convert/json.dart:741:21)
#6      _JsonStringifier.writeObject (dart:convert/json.dart:693:9)
#7      _JsonUtf8Stringifier.stringify (dart:convert/json.dart:975:17)
#8      JsonUtf8Encoder.convert (dart:convert/json.dart:401:26)
#9      _FusedConverter.convert (dart:convert/converter.dart:81:48)
#10     Codec.encode (dart:convert/codec.dart:25:32)
...

I've already tried making Post a serializable which did not work. I must be missing something.
Shouldn't it be possible to simply re-use the List provided by fetch and append it as a value to a key, or am I doing something wrong?
Thanks.

All managed objects are serializable. Maps are not. When a map is a body object, there is no additional behavior that occurs before it is encoded. It then has to be in an encodable format according to the built in JSON codec (primitives, maps and arrays); and a Post is not.
Change the value of your map to posts.map((p) => p.asMap())).toList().

that is not running in my program sir

Wow, what a great summary of your problem.

From: RahmatJatnika<EMAIL_ADDRESS>Sent: Thursday, December 13, 2018 3:03 AM
To: stablekernel/aqueduct
Cc: Tobi Schweiger; Author
Subject: Re: [stablekernel/aqueduct] Appending fetch() result lists to response (#551)
that is not running in my program sir
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/stablekernel/aqueduct/issues/551#issuecomment-446892607, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ADWEleuIc0KtJF-e4EUdkKOn-DHIv9HBks5u4hfggaJpZM4WgXTu.


[FEATURE] Ally chat has plugin prefix

allychat:
  syntax: '&8[&6&lALLY&8] &e%s&a%s &7» &r%s'

that's not a bug, you have added a prefix in the messages config.
that will be there as normal change it if you don't want it there in the config.

that's not a bug, you have added a prefix in the messages config. that will be there as normal change it if you don't want it there in the config.

So it's supposed to be shown with ally chat's syntax but not team chat's syntax????

that's the way how booksaw coded it, and I tested both team chat and ally chat and works fine for me.
as you can see booksaw removed the bug label and added the enchantment label, he might want to add this in the feature update or something, this is not a bug btw.
~ Greetings
Adarsh A

I tested both team chat and ally chat and works fine for me

🤦♂

language: en
storageType: YAML
prefix: false
prefixFormat: '&7[&6Teams&7] '

🤦♂
language: en
storageType: YAML
prefix: false
prefixFormat: '&7[&6Teams&7] '

I don't want to remove the plugin's prefix. I just want team chat and ally chat to be consistent (either don't have the plugin's prefix or do, but I prefer they don't).

That's currently not possible.
If you want that to work code it yourself and make a pull request as u did.
~ Greetings
Adarsh A

That's currently not possible

That's why I'm suggesting it 🤦♂

🤦♂

This feature has been implemented and will be included in the next update. Thank you for the suggestion.


cargo test in an ambient project doesn't work

Doing cargo test --features server, client takes us one step further, but next there's a problem with running .wasm modules as tests.

I do not think cargo alone is sufficient to test since we build and polyfill the wasm.
What we would need is an ambient test that uses a minimal runtime to test with, so that we have access to our api and can e.g; use our api for e.g color, time async etc.
For web wasm wasm-bindgen provides a test runner which you invoke instead, and there is a configuration option in cargo to use something other than cargo-test for cargo test