How to target the Tracer output from NLog.config?

Hi,
I'm having some trouble turning my tracer output on/off through my NLog.config.
I'm using Tracer.4NLog.Fody 1.0.1 and Tracer.Fody 1.6.0 with NLog.Extensions.Logging 1.0.0-rtm-beta4 and NLog 4.4.11.
What I want to do is turn on all logging, and then turn off certain namespaces using final="true"
  <rules>

    <!--Ignore certain namespaces-->
    <logger name="LogWeb5.*" minlevel="Trace" final="true" writeTo="void" /><!-- Works -->
    <logger name="Tracer.*" minlevel="Trace" final="true" writeTo="void" /><!-- Doesn't work -->

    <!-- Include others -->
    <logger name="*" minlevel="Trace" writeTo="log" />

  </rules>

Alternatively I could turn on specific namespaces
  <rules>
    <logger name="Tracer.*" minlevel="Trace" writeTo="log" />
    <logger name="LogWeb5.*" minlevel="Trace" writeTo="log" />
  </rules>

Since I'm getting output from Tracer in the first case, I must conclude that I've set up Tracer correctly. It seems like I'm simply trying to target Tracer in the wrong way. It's like it's using a different namespace internally to decide if it should be written or not. That namespace would then be matched when I use the name="*" catch-all type of configuration.
While the insight given by Tracer is great, I don't want its output to be on at all times. I want to be able to toggle the output so that I can turn it on when I need it. Any help on how to achieve that would be greatly appreciated!
Thanks for a very cool tool! Cheers!

Hi,
Thanks.
From the logger’s point of view (be it nlog, log4net or something else) tracer uses the namespace and the class of the original code. So it works as if you had manually added the entry and leave logging to your methods using a logger created for each class. At least it should work this way. I cannot access the source code at the moment, so I'm not able to double check it. I hope this helps. Please let me know if you have any further question.
Best regards
Csaba
From: Gaute<EMAIL_ADDRESS>Sent: 2017. június 26., hétfő 11:08
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Subject: [csnemes/tracer] How to target the Tracer output from NLog.config? (#50)
Hi,
I'm having some trouble turning my tracer output on/off through my NLog.config.
I'm using Tracer.4NLog.Fody 1.0.1 and Tracer.Fody 1.6.0 with NLog.Extensions.Logging 1.0.0-rtm-beta4 and NLog 4.4.11.
What I want to do is turn on all logging, and then turn off certain namespaces using final="true"

<!--Ignore certain namespaces-->
<logger name="LogWeb5.*" minlevel="Trace" final="true" writeTo="void" /><!-- Works -->
<logger name="Tracer.*" minlevel="Trace" final="true" writeTo="void" /><!-- Doesn't work -->

<!-- Include others -->
<logger name="*" minlevel="Trace" writeTo="log" />

Alternatively I could turn on specific namespaces

Since I'm getting output from Tracer in the first case, I must conclude that I've set up Tracer correctly. It seems like I'm simply trying to target Tracer in the wrong way. It's like it's using a different namespace internally to decide if it should be written or not. That namespace would then be matched when I use the name="*" catch-all type of configuration.
While the insight given by Tracer is great, I don't want its output to be on at all times. I want to be able to toggle the output so that I can turn it on when I need it. Any help on how to achieve that would be greatly appreciated!
Thanks for a very cool tool! Cheers!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/csnemes/tracer/issues/50, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGpT0cthzUCNB9hNsS0Y3gQQ7MorJNejks5sH3T5gaJpZM4OFDKo.

What you're saying would make sense, and would, in my opinion, be a better implementation choice than having a separate namespace for the Tracer logger. So, if that's the intended functionality, that's great.
If it is, though, I find it surprising that my logs (with the * log) looks like this:
Tracer.NLog.Adapters.LoggerAdapter.LogTrace  Entered into Get().
LogWeb5.Controllers.SomeController.Get SomeCtrl Some asked to get it!
Tracer.NLog.Adapters.LoggerAdapter.LogTrace SomeCtrl Entered into DoSomething().
LogWeb5.SomethingSvc.DoSomething SomeCtrl Begin DoSomething
Tracer.NLog.Adapters.LoggerAdapter.LogTrace SomeCtrl->Do Entered into SideEffects().
LogWeb5.SomethingSvc.SideEffects SomeCtrl->Do No side effects!
Tracer.NLog.Adapters.LoggerAdapter.LogTrace SomeCtrl->Do Returned from SideEffects() (). Time taken: 1,63 ms.
Tracer.NLog.Adapters.LoggerAdapter.LogTrace SomeCtrl->Do Entered into DoOther().
Tracer.NLog.Adapters.LoggerAdapter.LogTrace SomeCtrl->Do Returned from DoOther() (). Time taken: 0,00 ms.
LogWeb5.SomethingSvc.DoSomething SomeCtrl End DoSomething
Tracer.NLog.Adapters.LoggerAdapter.LogTrace SomeCtrl Returned from DoSomething() (). Time taken: 12,73 ms.
Tracer.NLog.Adapters.LoggerAdapter.LogTrace  Returned from Get() ($return=System.String[]). Time taken: 17,74 ms.

Where I would have expected something like the following, which I find much easier to read:
LogWeb5.Controllers.SomeController  Entered into Get().
LogWeb5.Controllers.SomeController.Get SomeCtrl Some asked to get it!
LogWeb5.Controllers.SomeController SomeCtrl Entered into DoSomething().
LogWeb5.SomethingSvc.DoSomething SomeCtrl Begin DoSomething
LogWeb5.SomethingSvc SomeCtrl->Do Entered into SideEffects().
LogWeb5.SomethingSvc.SideEffects SomeCtrl->Do No side effects!
LogWeb5.SomethingSvc SomeCtrl SomeCtrl->Do Returned from SideEffects() (). Time taken: 1,63 ms.
LogWeb5.OtherSvc SomeCtrl->Do Entered into DoOther().
LogWeb5.OtherSvc SomeCtrl->Do Returned from DoOther() (). Time taken: 0,00 ms.
LogWeb5.SomethingSvc SomeCtrl End DoSomething
LogWeb5.SomethingSvc SomeCtrl Returned from DoSomething() (). Time taken: 12,73 ms.
LogWeb5.Controllers.SomeController  Returned from Get() ($return=System.String[]). Time taken: 17,74 ms.

If it is indeed the case that the resulting logged namespaces do not match the internally used namespace, I can't make sense of that either, however. Why? Cause turning on everything except "LogWeb5.*" does not turn off the Tracer output. Likewise, turning on only "LogWeb5" does not also turn off the Tracer output.

The internal namespaces of the Tracer adapter should not appear in the log at all. In the adapter LogEventInfo instances are crafted 'manually' to mimic the right behaviour. It seems that in your case it isn't working. I'll check this issue trying to reproduce the error, Unfortunately I can only do this at the end of next week as I'm on holidays,

Great stuff. Enjoy your holliday!
I uploaded my test repository in order to let you more easily debug the issue, when you get back.

Hi,
This is a bug in the adapter, I forgot to set the wrapperType when calling the logger. I will fix it.
One drawback I realised when looking through NLog is that unfortunately it seems very difficult to make use of the speed enhancement of Tracer. NLog internally always runs the StackTrace, I'm not able to inject the method and class information to it although it is there statically at rewrite time (One of the reasons I created Tracer is to get rid of the time consuming StackTrace for each log entry). Anyway it should work properly, after the fix of course.

That's great news!
I'm not sure exactly what you mean by it running the StackTrace. That sounds like a very large overhead. Why would it need to investigate the stack? Imho stack trace investigation should only happen when exceptions are thrown, not when logging normal things.
Does this mean there is significant performance implications with choosing NLog over one of the other logging frameworks?
I mean, clearly Tracer adds a lot of log writes, so I guess log perf is more important when using Tracer than it would be with more usual sparse logging..

The issue is fixed in the 1.0.3 version of the NLog adapter, please give it a try. About the stack trace thing: All logging frameworks use the stack trace to figure out the callsite, that is, from which method the logger was called. One of the benefits of using tracer is that during the rewrite phase we know all the callsites so we can pass this information to the logger so there's no need to figure out the callsite from the stack trace. There are logging frameworks (like log4net) where I can inject this information into the logging mechanism and some (like NLog) where I cannot (or at least I haven't find a way to do it yet).

Great news. I'll check it out and post my findings. Thanks!
So, I felt pretty sure you shouldn't have to look up the logger name from the stack with NLog, so I looked up the details. Seems to me the stack is only used if you instantiate the logger using  LogManager.GetCurrentClassLogger(), but not if you do using LogManager.GetLogger("My.Call.Site.MyClass") or LogManager.GetLogger(typeof(MyClass).FullName).
Also, the stack is only looked up to find out which class is instantiating it, not to figure out which method is writing to the log; a very big difference.
If however there is further lookup to the stack in other places, what I'm saying here is rubish, but I don't see why there would be any need to.

Now I see we're talking about two different things and it seems that we're both right :-). What I fixed was the callsite property which you used to display the calling method in your logs. There NLog uses stack trace for every call. The rule filtering on the other hand is based on the Logger name and not on the callsite. There I've just found a bug of not using FullName but Name of the type, so that might caused the error. The callsite thing on the other hand can be substituted with ${event-properties:item=TypeInfo} and ${event-properties:item=MethodInfo}. These two are injected by Tracer to provide the same info as the callsite without the overhead of running a stack trace. Please ignore the 1.0.3 release as it will still misbehave (the callsite will be OK, but that wasn't the point).

Ahh.. I see.. yes.. all loggers I suppose will use the stack to figure out the call site method name, indeed. Hopefully only when a logger is active (but I digress).
Yep.. was testing 1.0.3, and found that the names were now right, but that I still couldn't control filtering the way I want.
Eagerly awaiting 1.0.4 :)

Just published to nuget, it should be visible soon.
As far as I understood the NLog code it only runs the stack trace if one of the logging outputs uses the callsite property. So someone instead of referencing callsite uses the TypeInfo and MethodInfo properties can avoid the performance penalty and still have the information on the call's location.

Yep. That did it. Thanks a bunch!
As for the MethodInfo and such, I think I should probably measure perf impact before I try to optimize. "Premature optimization is the root of all evil", after all. As far as I am concerned the Tracer output should be turned off by default, and only turned on during diagnostic sessions, which should work great now.


Added a on_connect callback

This PR tries to fix #4 and adds on_connect callback. If the connection could not be established an error message is logged.
It also fixes a problem I encountered with HomeAssistant autodiscovery where the config topic gets lost, because it is published before the connection is established. This may be because I use authentication or the broker is running on a slow system.
This is the first I am trying to contribute to an open source project and there may very well be a more elegant solution, so any feedback is appreciated.

Thank you for the PR!

I was going to change the log level to error but then I saw the subsequent commit.
You're directly calling sys.exit() which bypasses the cleanup method. I don't see this being a problem right now because everything the cleanup does is predicated on being connected to the broker already anyhow. And I don't foresee the need to ever open resources ahead of trying to connect to the broker.

I'll go ahead and merge this. Thanks again for the contribution!


[rdbms-connect] remove preview tag for postgresql

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

rdbms-connect

Hi @DaeunYim, to make changes really work in cli extension, you have to upgrade version in setup.py and update HISTORY.rst so that a new version will be released

@evelyn-ys  It was very silly of me to miss that again... :( I'm submitting a PR right now.. Thank you a lot for letting me know!!

https://github.com/Azure/azure-cli-extensions/pull/4178 Here!


[BUG] python 3.12 pkgutil has no attribute 'ImpImporter'

Describe the bug
A clear and concise description of what the bug is.
tuptools_init_.py", line 16, in
import setuptools.version
File "C:\Users\AppData\Local\Temp\pip-build-env-5c2u67or\overlay\Lib\site-packages\se
tuptools\version.py", line 1, in
import pkg_resources
File "C:\Users\AppData\Local\Temp\pip-build-env-5c2u67or\overlay\Lib\site-packages\pk
g_resources_init_.py", line 2191, in
register_finder(pkgutil.ImpImporter, find_on_path)
^^^^^^^^^^^^^^^^^^^
AttributeError: module 'pkgutil' has no attribute 'ImpImporter'. Did you mean: 'zipimporter'?
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.
note: This error originates from a subprocess, and is likely not a problem with pip.
To Reproduce
To help us to reproduce this bug, please provide information below:

Your Python version
The version of Xoscar you use  #xoscar-0.1.3.tar.gz
Versions of crucial packages, such as numpy, scipy and pandas
Full stack of the error.
Minimized code to reproduce the error.

Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

Hi, @johnson7788 . Currently xoscar does not support Python 3.12. xoscar supports from Python 3.8 to 3.11. In the future, we will support 3.12.

Is there any update for Non-supporting python 3.12?


MapNodeGroup Inspector Improvements

Renamed "Remove Node Excess" into "Remove Unconnected Nodes", now only removes nodes that are not connected to any other node.
Added option to change distance between nodes while using painter.

Thanks for your contribution, i'll release 5.4.0 with this change


Use importlib.metadata from the standard library on Python 3.8+

Hello. Would you accept a pull request that uses importlib.metadata from the standard library on Python 3.8+ instead of the importlib_metadata package?
If so, do you prefer try: import importlib.metadata, except: import importlib_metadata or if version < 3.8 approach?
Examples:

https://github.com/pytest-dev/pytest/pull/5539
https://github.com/pytest-dev/pluggy/pull/223
https://github.com/tox-dev/tox/pull/1368

That sounds great.  I prefer the explicit version checks.  I've used import checks before for stuff in builtins before but they were defeated when I had the futurize modules installed in a different environment, so now I'm battle scarred and prefer the explicit checks whenever possible.
Also, in case it helps, our project skeleton is based on https://github.com/davvid/skeleton/ the python2.7-compat branch.  If it's a file that's common to the skeleton I'll take the change there and then merge it into here.

https://github.com/davvid/skeleton/search?q=importlib_metadata&unscoped_q=importlib_metadata yields nothing (I've manually checked setup.cfg as well)

https://github.com/jsonpickle/jsonpickle/pull/305


Add CI Tests for test infra and partner image

Implements three new GitHub Workflows for testing, and a GitHub Action for setting up a minikube cluster suitable for CI/CD pipelines.
The new start-minikube GitHub Action should be considered as the recommended Action for creating test clusters for TNF resources in CI workflows, removing the dependency on third-party, untrusted Actions from the GitHub Marketplace.
The new workflows are:

test-cnf-test-partner-image.yaml: Tests whether the partner image can be built, and whether the image contains all required tools.

test-local-test-infra.yaml: Tests the OpenShift resources defined in the local-test-infra directory on the minikube test cluster. The resources are created and removed using the commands exposed in the Makefile.

test-partner-pod.yaml: Tests the installation process of the partner pod according to the instructions in the README. The pod is created on the minikube test cluster.

Signed-off-by: Marek Kochanowski<EMAIL_ADDRESS>

Why were no "checks" run as a part of this PR?  The checks should have automatically run, which makes me believe the checkout action isn't working properly here.

Why were no "checks" run as a part of this PR? The checks should have automatically run, which makes me believe the checkout action isn't working properly here.

Each workflow definition is equipped with path filters, like this:
name: Test the `cnf-test-partner` image
on:
  push:
    paths:
      - 'test-partner/Dockerfile'

The filter ensures that the workflow is run if any changes are to be introduced to the selected parts of the codebase (in this example, if the Dockerfile is modified).
For the cnf-test-partner image, I think we would like to test and auto-push (TBD) a new container image only in the situation when changes are introduced to the Dockerfile, rather than doing it for every PR (e.g. fixing a typo in the documentation should not trigger the auto-push).
In most CI systems the filtering is done to reduce the number of concurrent jobs.
In extreme cases, if the number of concurrent jobs permitted at a time is very low or the volume of contributions is high (let's say, everyone has the same working hours), running all workflows on every commit can get them queued up, clogging the CI pipeline for many minutes or hours, for every contributor in the organization.
It's probably a premature optimization, but a good practice nonetheless. I couldn't find what's the exact number of allowed concurrent jobs on GitHub Actions for the free tier (20 per org?).
Possible actions include:

Pushing a test commit to this branch that contains sample changes. This will trigger all workflows. The test commit can then be removed before merging.
Removing the path filters in all workflows, excluding the cnf-test-partner image workflow. This will result in the the container image being built and tested only for PRs that modify the Dockerfile.
Removing the path filters in all workflows. This will result in all checks being triggered for every commit and PR, no matter the contents of the PR.

I think the 2. option might be good enough for our needs. Any thoughts?
Note: the test-partner-pod.yaml check will fail until we merge the PR that adds the test-partner/namespace.yaml resource.

@mkochanowski option 2 seems correct.

@ryandgoulding I'll update the branch with the option 2 then.
However, it seems like no matter which option we would have gone with, GitHub prevents the execution of workflows supplied in a Pull Request from a forked repository, and there does not seem to be a setting that alters this behavior (maybe for the better).
Reason: Assume a bad actor that submits a malicious Pull Request containing new or updated workflows.
Let the new workflow mine bitcoin.
If the organization is on a paid plan, or uses self-hosted runners, the workflow would be executed against the organization's infrastructure, and would consume the organization's quota.
This may result in the organization receiving an unexpected, large bill to settle.
Explanation: When a new Pull Request is opened, a pull_request event is triggered on the base repository, and not on the fork. The pull_request event applies to workflows that are already present in the repository.
For this Pull Request, there are no workflows present in the base repository.
When possible, GitHub prefers to execute workflows on the fork. This mostly applies to push events; does not apply to pull_request (unless a PR is opened against the fork itself).
The workflows ran on the fork are not displayed on the PR page, only the workflows executed on the base repository are.
Source: https://github.community/t/run-a-github-action-on-pull-request-for-pr-opened-from-a-forked-repo/16054

The branch is now updated with path filters being removed in all workflows for all event types, except for the push event of cnf-test-partner-image.yaml.
This workflow is responsible for building, testing and pushing (#12) the image. The workflow only applies to commits affecting the Dockerfile.
I've also removed the test-* prefix from filenames, as we won't be creating separate workflows (like test-*, push-*) for the same resource type (the workflow_run event type is currently too limiting to make this viable).


Generic mode

You did what you need! Has studied your github ineterseye things do!
One drawback with this extension is that it does not have the Generic mode. Since a bunch of cameras are now on the SoC HI35XX, it makes no sense to prescribe all the brands. If there is time, please re-select the extension and post Chrome Webstore

Snapshot on ip web camera SOC HI3518E_50H10L_S39
ip: http://<IP_ADDRESS>/webcapture.jpg?command=snap&channel=1

I configured my cameras through export settings in your extension. Further no more settings do not do otherwise the settings overwrite and work will stop))))

ah okay i understood, will add a generic mode where you can just enter the stream url - in the next release

thats already possible as far as i can say:


Added, return item object from selected row event.

Added an event to the MatTable, which returns the item of the selected row to the ParentComponent where the Table has been built.

Thank you for your contribution! That's really needed feature.


feat: prompt versioning

Description

Adds support for versioning the main Aila prompt
Ideally, we should all run pnpm prompts:dev to create rows in the prompts table in the database, but there is a fallback in place so that prompts are created if they are not found
When we make any AilaGeneration we store the current version of the prompt against the generation
Updates the pnpm prompts script to generate rows that relate to the Ail prompt at deployment
Has different variants for the three variables that we pass into the prompt - useRag, interactive/generate, basedOn, since we construct different prompts in each scenario
Because of issues getting core to pull code in about the JSON schemas from the aila package, it does not take into account if there are any changes to the lesson schema itself. We could add this at a later stage.
Refactors the prompt into parts (in line with how the other prompts work)
Since the format of the prompt is so different, it stores the prompt parts as a JSON array rather than in the structure the other prompts are in
Includes a test to ensure that the prompt has not changed in the process of refactoring it. We can remove the old.ts file and the test once this is merged.

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


Warning RARoutePickerContext

First, excellent module!
I am experiencing a warning while using it
Module RARoutePickerContext requires main queue setup since it overrides init but doesn't implement requiresMainQueueSetup. In a future release React Native will default to initializing all native modules on a background thread unless explicitly opted-out of.
Platform: iOS 15.2
RN Version: 0.66.1

Any news on the release?

@yemd yep sorry just got my laptop back from repair, just released the version including the fix for this


#402 fix responsiveness of cards bug

close #402

In window width 960px to 1100 px the "Arbeit mit Jugendlichen"-card looks like this:

Please make the font size smaller so the text stays inline and the white block is the same size as the other cards.

I think the font-size itself is okay but the cards should jump into the new line earlier instead of shrinking them this far.


QR Code Poster (EXPOSUREAPP-5962)

Create location poster PDF
Enable printing , Sharing
Delete testing screens, Please use the actual screens

Testing

Test against mock server
Create location -> Choose Print option from overflow menu
Create location -> QR Code details screen -> Print button
Check poster and try printing and sharing

TODO

[x] Finalise positioning values from server
Text Box font - not provided yet
Create poster on the fly (New ticket 🎫 probably or even after MVP)

After merging https://github.com/corona-warn-app/cwa-app-android/pull/2737

[x] Use Location Url to generate QR Code
[x] Extend testability for TraceLocation

Tested it on different screen size (4.5, 5.7 and 6.3"). Looks really impressive! 💯

Looks very good on my Pixel 3A and Samsung S20+, both Android 11.
Great work 🚀


Missing custom metrics

Expected behavior (what you expected to happen): Metrics appended via $METRICS_PATH should appear in prometheus results.
Actual behavior (what actually happened): From v1.4.2+, metrics are added successfully (confirmed by logs), however they are not returned by prometheus.
Steps to reproduce:

Add hook and deploy
Inside the container run curl <IP_ADDRESS>:9115/metrics

Environment:

Shell-operator version: v1.4.2+
Kubernetes version: v1.27.7
Installation type (kubectl apply, helm chart, etc.): Helm chart

Anything else we should know?: This is working as expected in ghcr.io/flant/shell-operator:v1.4.1, but seems broken in v1.4.2 and v1.4.3.
Additional information for debugging (if necessary):
Hook script
#!/usr/bin/env bash
source /shell_lib.sh
function config() {
cat <<EOF
configVersion: v1
onStartup: 1
EOF
}
function main() {
echo '{"name": "my_metric", "action": "add", "value": 0}' >> ${METRICS_PATH}
}
hook::run $@

Logs
{"level":"info","msg":"Create metric counter my_counter","operator.component":"metricStorage","time":"2023-11-20T00:00:00Z"}
{"binding":"","event":"onStartup","hook":"startup.sh","level":"info","msg":"Hook executed successfully","queue":"","task":"HookRun","time":"2023-11-21T14:38:12Z"}

I've found #549 introduced a breaking change in how metrics are reported, they are no longer exposed on /metrics path but on /metrics/hooks.


JAVA-1448: TokenAwarePolicy should respect child policy ordering

Closing as JAVA-1448 is likely to be redefined in a different direction.


Please remove the static Footer Text

The static footer makes no sense. The HTML Stacktrace should no how contain the credit footer. In fact this single feature discards all credibility of this lib to be used for enterprise usage purpose. Please remove it.
Message generated at 2020-04-13T19:52:12.893 by Html Exception Formatter.

You can use:
options.setPrintFooter(false);

or
options.setPromoteLibrary(false);


New release ?

Do you have any idea when the next version will be made?

seems the last release was 1 year ago...

Great .. A new pre-release.


brew install -d and brew install -i are undocumented

Homebrew documentation references brew install -d and brew install -i in passing, e.g.
https://github.com/Homebrew/brew/blob/master/docs/Manpage.md:
  * `HOMEBREW_DEBUG_INSTALL`:
    When `brew install -d` or `brew install -i` drops into a shell,
    `HOMEBREW_DEBUG_INSTALL` will be set to the name of the formula being
    brewed.

and
https://github.com/Homebrew/brew/blob/master/docs/Formula-Cookbook.md
### Check the build system

```
sh
brew install -i foo
```

The documentation and cookbook guidance certainly shouldn't reference undocumented options in an offhand fashion. I would also say that undocumented options simply shouldn't exist (maybe this is not agreed-upon, though?)
I assume that -d is short for --debug and -i is short for --interactive but I spent more than a reasonable amount of time source-diving and failed to figure out how that is handled, so I'm giving up and filing this without a patch.
I note that some other short options are documented, like -s for --build-from-source, although it doesn't appear in the usage summary:
install [--debug] [--env=(std|super)] [--ignore-dependencies|--only-dependencies] [--cc=compiler] [--build-from-source|--force-bottle] [--devel|--HEAD] [--keep-tmp] [--build-bottle]

Please fill out the issue template.

Please fill out the issue template.

To note: even although this was fixed without we'd really rather you filled it in without exception for every single issue report, thanks.

To note: even although this was fixed without we'd really rather you filled it in without exception for every single issue report, thanks.

What's the point of filling out the template for documentation bugs that are referenced to webpages in the master branch on github? The information in the template is entirely irrelevant to the bug?
The template says:

You can erase any parts of this template not applicable to your Issue.

which I interpreted to mean I could reasonably delete everything in the template because none of it applied to web docbugs. Maybe it should be reworded.
Anyhow, here you go.

[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed as many issues as possible and retried your prior step?
[x] Confirmed this is a problem with Homebrew/brew and not specific formulae? If it's a formulae-specific problem please file this issue at https://github.com/Homebrew/homebrew-core/issues/new

Bug reports:
pb3:~ jhawk$ brew config
Unsetting PERL5LIB to protect you from yourself
HOMEBREW_VERSION: 1.1.11-226-g974b5e2fa2
ORIGIN: https://github.com/Homebrew/brew.git
HEAD: 974b5e2fa26a162d29f97e996eb687b12a46a952
Last commit: 17 hours ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: eb852b832ab49a4acf184e14734d1c2ff4f26d98
Core tap last commit: 8 hours ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_REPOSITORY: /usr/local/Homebrew
HOMEBREW_CELLAR: /usr/local/Cellar
HOMEBREW_BOTTLE_DOMAIN: https://homebrew.bintray.com
CPU: quad-core 64-bit broadwell
Homebrew Ruby: 2.0.0-p481
Clang: 7.0 build 700
Git: 2.12.0 => /usr/local/bin/git
Perl: /usr/local/bin/perl => /usr/local/Cellar/perl/5.24.0_1/bin/perl
Python: /usr/bin/python
Ruby: /usr/bin/ruby => /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby
Java: 1.8.0_74, 1.6.0_65-b14-468
macOS: 10.10.5-x86_64
Xcode: 7.2.1
CLT: <IP_ADDRESS>.1.1447826929
X11: 2.7.11 => /opt/X11
pb3:~ jhawk$ brew doctor
Unsetting PERL5LIB to protect you from yourself
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry and just ignore them. Thanks!

Warning: You have unlinked kegs in your Cellar
Leaving kegs unlinked can lead to build-trouble and cause brews that depend on
those kegs to fail to run properly once built. Run `brew link` on these:
  ghostscript
  lesstif
  numpy
  poppler
  python
pb3:~ jhawk$

which I interpreted to mean I could reasonably delete everything in the template because none of it applied to web docbugs. Maybe it should be reworded.

I will change this, thanks.


XL search "Max threads" cannot be set from GUI

Hi,
When choosing a non-default number of max threads for XL search tasks, then saving task settings, the number is always reset to the default (max - 1). If I adjust the number in the .toml file, it is read correctly. However, it is reset to default again whenever task settings are saved again.

Hi kh,
Thank you for pointing this GUI bug. It will be fixed in the next release.
Lei

'''
Remeber to add the following in XLSearchTaskWindow.xaml.cs':
line 321
var _maxThreadsToUsePerFile = int.Parse(maxThreadsTextBox.Text, CultureInfo.InvariantCulture);
line 336
maxThreadsToUsePerFile: _maxThreadsToUsePerFile,
'''


feat: add dbRepositoryUsername and dbRepositoryPassword for dbReposit…

Description
Add the configs dbRepositoryUsername and dbRepositoryPassword to allow authentication for the dbRepository.
This configuration sets the environment variables TRIVY_USERNAME and TRIVY_PASSWORD in the scan-vulnerabilityreport-pods.
Checklist

[x] I've read the guidelines for contributing to this repository.
[x] I've updated the documentation with the relevant information.

 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.

@juergen-fast thank you for the contribution I have one comment: do you think it worthwhile to add this setting also for built-in server config somewhere here ? so it will have support for client/server users

@juergen-fast thank you for the contribution I have one comment: do you think it worthwhile to add this setting also for built-in server config somewhere here ? so it will have support for client/server users

My changes based on the githubToken-Code. I think there is no need for built-in server because in this case you have the property serverUser and serverPassword in the trivy secret here

@juergen-fast run mage generate:docs to update helm docs

@juergen-fast you'll need to update plugin tests with new EnvVar

@juergen-fast any update on this?

I tried adding the new EnvVar but the tests still fail. I don't know how to fix them, sry

@juergen-fast I'll have a look

@juergen-fast lgtm 🚀  thank you for the contribution


feat(devtools-view): Update summary view data to use info labels / badges

Updates the view to include clickable info badges (using FluentUI's experimental InfoLabel component)
Also adds an info label for the "Status" property, which didn't previously have one.

If we like this approach, I can do a subsequent PR to update the existing info badges to use this or InfoButton as appropriate.


Windows pathlib encoding error in README

On Linux, this problem does not exist. On Windows, in setup.py,
README = (HERE / "README.md").read_text()

The read_text() function should have default value of optional argument encoding=None of utf-8, but it turns out that pathlib uses gbk:

Traceback (most recent call last):
File "D:\Resources\sootty\setup.py", line 8, in
README = (HERE / "README.md").read_text()
File "E:\Program_Files\Python310\lib\pathlib.py", line 1135, in read_text
return f.read()
UnicodeDecodeError: 'gbk' codec can't decode byte 0x99 in position 702: illegal multibyte sequence

This occurs on README.md with line endings of CRLF. If the file is saved with endings of LF, the position is 700 because two line endings are "\n" instead of "\r\n". The character at that position is "’".
Two possible solutions:

Change two characters "’" into "'";
Explicitly specify read_text(encoding='utf-8')

However, the reason why this error occurs is still unknown.

Interesting, I've never seen this encoding before. It seems like the issue might be related to localization, since gbk is designed for simplified Chinese. If specifying the encoding solves the issue then that seems good.


Update description for buttons

Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ] Build (yarn build) was run locally and any changes were pushed
[ ] Unit tests (yarn test) were run locally and passed
[ ] Visual Regression Tests (yarn visual-regression) were run locally and passed
[ ] Linting (npm lint) was run locally and passed

Pull request type

Please check the type of change your PR introduces:

[ ] Bug fix
[ ] Feature
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[x] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

GitHub Issue Number: N/A
What is the new behavior?

updates documentation for button of ix styleguide

Does this introduce a breaking change?

[ ] Yes
[x] No

Testing

Other information

@linstedt @gyoengyiszabo @tokyojen what is the status of this PR?

@nuke-ellington It's not ready yet

@nuke-ellington styleguides are ready. Open points:

[ ] adjust navigation structure to group buttons together
[ ] rename "Toggle Buttons" into "Toggle button"

@nuke-ellington Styleguides are ready to merge. There seems to be some file conflict but I don't have sufficient rights to solve it.


[feature] return a scalar value instead of a buffer for bit fields, such as Byte.

Is your feature request related to a problem? Please describe.
r2dbc has ConverterNotFoundException [java.nio.HeapByteBuffer] to type [java.lang.Boolean] on boot 3.3.0
https://github.com/spring-projects/spring-data-relational/issues/1824
Describe the solution you'd like
return a scalar value instead of a buffer for bit fields, such as Byte.
Additional context
https://github.com/spring-projects/spring-data-relational/issues/1824#issuecomment-2185738223

@underbell
Thanks for reporting :D

Memo: Need to support tinyInt1isBit flag

ref https://docs.oracle.com/cd/E17952_01/connector-j-en/connector-j-reference-type-conversions.html


(mini.ai) Config option for operators allowed to be used in empty regions

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md

Module(s)
mini.ai, mini.operators
Description
Mini.ai currently has a hardcoded check to allow d and c to be used in an empty region.
Other operators (like gr from mini.operators) also make sense to be used in an emtpy region, so it would be nice to be able to have a configuration option for this.

Thanks for suggestion!
This certainly won't be a configurable option. Will take a look at the best way to widen the accepted cases for empty regions.

I didn't opt-in for new config setting, but instead decided to specifically hand pick "replace" from 'mini.operators' to allow acting on empty region.
The reason for this decision is slightly technical. Basically, Neovim doesn't really allow empty selections. But as it is useful to do ci)/di)/gri) for empty brackets, there is a hack which convert empty selection into one space selection. This means that only operators guaranteed to not essentially use selection can be acted on empty selection. From built-in those are c and d (although they do cut that text in default register). "Replace" from 'mini.operators' is another example. I don't think there is something else worth adding to this hand picked "allow list".
All that said, thanks again for suggestion! Being able to gri) for empty brackets is indeed quite useful.


mc_control::fsm::Executor should be non copyable

Hi @gergondet
I encountered an issue where I was able to copy a locally initialized Executor in a map, which was not working later?
After investigating with @arntanguy we found that Executor should not be copyable.

Hi @JulienRo7 @arntanguy
Can you clarify what was the exact issue? I don't think it should be an issue to copy an executor.

Hi @gergondet
I am using a map of executors inside one of my classes and when I am initializing the executor I did something like this:
std::unordered_map<Hand, mc_control::fsm::Executor, internal::EnumClassHash> handExecutors_;
for(const auto & hand : BOTH_HANDS)
    {
      mc_control::fsm::Executor handExecutor;
      std::string executorName = "HandExecutor_"+std::to_string(hand);
      mc_rtc::Configuration config;
      // read the right configuration file here...

      handExecutorPtr.init(ctl(), config, executorName, {"HandExecutor", std::to_string(hand)});
      handExecutors_.insert({hand, handExecutor});
    }

Note that for simplification of the issue I removed the part on loading the configuration. But this code was crashing without any error.
The solution was to use smart pointers like this:
std::unordered_map<Hand, std::shared_ptr<mc_control::fsm::Executor>, internal::EnumClassHash> handExecutors_;
for(const auto & hand : BOTH_HANDS)
    {
      std::shared_ptr<mc_control::fsm::Executor> handExecutorPtr = std::make_shared<mc_control::fsm::Executor>();
      std::string executorName = "HandExecutor_"+std::to_string(hand);
      mc_rtc::Configuration config;
      // read the right configuration file here...

      handExecutorPtr->init(ctl(), config, executorName, {"HandExecutor", std::to_string(hand)});
      handExecutors_.insert({hand, handExecutorPtr});
    }

Thanks for the clarification. After looking deeper into it, the executor cannot be moved after it has been initialized, to prevent this kind of issue I will make Executor both non-copiable and non-movable


Fix effect and palette names and a few descriptions

Updated a few effect and palette names.

BRB :)


[Question] Does Android MLCChat has previous context as input?

❓ General Questions

After generating the Android MLCChat app based on the model gemma-2-2b-it-q4f16_1 and installing it on my device, I found that the chatbot seems not to retain previous context.
For example, in the screenshot, it didn’t understand that the second question "why" refers to the question "which is bigger, 9.11 or 9.9?".
It appears that the app only uses the most recent question as the prompt, rather than considering the previous conversation history.
I’m wondering if this is by design or if I made a mistake during the app generation process.
Thank you very much!

Thanks for reporting, we are working to address this as #androi

Thanks for reporting, we are working to address this as #2740

Got it, thank you and waiting for the update~

Hi @LCmoon0126, I have fixed it in #2760, can you try it again with the latest commit? Please uninstall MLCChat first, thanks.

Hi @LCmoon0126, I have fixed it in #2760, can you try it again with the latest commit? Please uninstall MLCChat first, thanks.

Hello, thank you for the update.
But after I update the code, MLCChat crash.
My step is that:

update the code to the lastest
repackage by "mlc_llm package"
generate APK using Android Studio
uninstal MLCChat on my Android phone
install the APK and weights by "python bundle_weight.py --apk-path app/release/app-release.apk"

My Android phone is redmi K60.
My mlc-package-config.json is that:
{
    "device": "android",
    "model_list": [
        {
            "model_notuse": "/Users/licheng/Desktop/AI/models/gemma2/gemma-2-2b/gemma-2-2b-q4f16_1-MLC",
            "model": "HF://mlc-ai/gemma-2-2b-it-q4f16_1-MLC",
            "model_id": "gemma-2-2b-it-q4f16_1-MLC",
            "estimated_vram_bytes":<PHONE_NUMBER>,
            "bundle_weight": true
        }
    ]
}

The screen recording is that:
https://github.com/user-attachments/assets/16d317b6-2cf1-446d-95f9-e40b31907161


Request the source code of this project. Thank you

Request the source code of this project.Thank you

+1

+1

Thanks, everyone for the interest.
I am currently working on cleaning up the code/models and hope to release it soon.

You have got our attention... the only one thing needed. Not releasing the code after showing proof of concept, then teasing us, and now not delivering is most likely beginning to become bad for business. Let someone that wants to clean up the code have a chance to clean it now.

@arrrrstin Thanks for your patience. This is not a business but a simple research prototype that we are planning to open source.
We have an initial prototype ready that I am planning to release in the next few days.

Releasing the code base.
Very early release, will fix bugs/documentation as I go along.


host: Don’t include env vars in debug info by default

There are many pieces of sensitive information that we shouldn’t collect that are in the environment. Also added a flag to flynn-host inspect to omit the environment.
Closes #954

LGTM


dotnet-framework-sdk claims that Visual Studio Build Tools are included – Yet, they are not

Describe the Bug
The dotnet-framework-sdk image description on Docker Hub claims that the base images are delivered including Visual Studio Build Tools and the Visual Studio Test agent:

This image contains the .NET Framework SDK which is comprised of the following parts:

.NET Framework Runtime
Visual Studio Build Tools
Visual Studio Test Agent
NuGet CLI
.NET Framework Targeting Packs
ASP.NET Web Targets

Use this image for your development process (developing, building and testing applications).

Yet, neither Visual Studio Build Tools nor the Visual Studio Test Agent can be found in the container.

Steps to Reproduce
C:\>dir
 Volume in drive C has no label.
 Volume Serial Number is 8473-2E2C

 Directory of C:\

09/10/2020  10:34 PM             5,510 License.txt
08/05/2022  11:16 AM    <DIR>          Program Files
04/30/2024  04:52 PM    <DIR>          Program Files (x86)
08/05/2022  11:37 AM    <DIR>          Users
04/30/2024  06:19 PM    <DIR>          Windows
               2 File(s)          5,542 bytes
               5 Dir(s)  128,552,034,304 bytes free

C:\>dir "Program Files"
 Volume in drive C has no label.
 Volume Serial Number is 8473-2E2C

 Directory of C:\Program Files

08/05/2022  11:16 AM    <DIR>          .
08/05/2022  11:16 AM    <DIR>          ..
12/07/2019  11:01 AM    <DIR>          Common Files
12/07/2019  10:50 AM    <DIR>          internet explorer
12/07/2019  10:50 AM    <DIR>          ModifiableWindowsApps
08/05/2022  11:24 AM    <DIR>          Windows Defender Advanced Threat Protection
12/07/2019  11:24 AM    <DIR>          Windows NT
12/07/2019  10:50 AM    <DIR>          WindowsPowerShell
               0 File(s)              0 bytes
               8 Dir(s)  128,552,034,304 bytes free

C:\>dir "Program Files (x86)"
 Volume in drive C has no label.
 Volume Serial Number is 8473-2E2C

 Directory of C:\Program Files (x86)

04/30/2024  04:52 PM    <DIR>          .
04/30/2024  04:52 PM    <DIR>          ..
12/07/2019  11:01 AM    <DIR>          Common Files
12/07/2019  10:50 AM    <DIR>          internet explorer
12/07/2019  10:50 AM    <DIR>          Microsoft.NET
04/30/2024  04:52 PM    <DIR>          NuGet
12/07/2019  11:24 AM    <DIR>          Windows NT
12/07/2019  10:50 AM    <DIR>          WindowsPowerShell
               0 File(s)              0 bytes
               8 Dir(s)  128,552,034,304 bytes free

C:\>path
PATH=C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0\;C:\Windows\System32\OpenSSH\;C:\Users\ContainerAdministrator\AppData\Local\Microsoft\WindowsApps;C:\Program Files (x86)\NuGet;

C:\>

@SetTrend, Can you please share which tag you are looking at?
c:\docker run -it --rm mcr.microsoft.com/dotnet/framework/sdk:4.8.1

C:\>dir "C:\Program Files (x86)\Microsoft Visual Studio\2022"
 Volume in drive C has no label.
 Volume Serial Number is 4819-BD0B

 Directory of C:\Program Files (x86)\Microsoft Visual Studio\2022

04/09/2024  01:13 PM    <DIR>          .
04/09/2024  01:12 PM    <DIR>          ..
04/09/2024  01:14 PM    <DIR>          BuildTools
04/09/2024  01:12 PM    <DIR>          TestAgent
               0 File(s)              0 bytes
               4 Dir(s)  136,121,753,600 bytes free

Thanks for replying!
Here's the tag I'm using: mcr.microsoft.com/dotnet/framework/sdk:4.8

I must have made a mistake somewhere … Just builded a new Docker image based on mcr.microsoft.com/dotnet/framework/sdk:4.8 and everything seems to be in the right place.
Please pardon me for any inconvenience!


fix: uses metrics endpoint for identity service monitor

Which problem does the PR fix?
Related to: https://github.com/camunda/camunda-platform-helm/issues/733

What's in this PR?

Checklist
Please make sure to follow our Contributing Guide.

Before opening the PR:

[x] There is no other open pull request for the same update/change.
[x] The commits follow our Commit Guidelines.
[x] Tests for charts are added (if needed).
[ ] The main Helm chart and sub-chart are updated (if needed).
[ ] In-repo documentation are updated (if needed).

After opening the PR:

[x] Did you sign our CLA (Contributor License Agreement)? It will show once you open the PR.
[ ] Did all checks/tests pass in the PR?

connectors
identity
operate
optimize
tasklist
zeebe
zeebe-gateway

I'm going to check if there are any other containers we need to account for.
Here is a list of containers it was able to pull metrics from in my local instance with this patch

I recognize that the web modeler service monitor is also not working. I learned that it listens on the port named http-management and that it listens on /metrics for the restapi. However, the webapp and the websockets containers don't seem to have any prometheus endpoint open.. hmm.

I don't think keycloak, postgresql, or elasticsearch should be in scope for this PR since they are external components and probably don't have a prometheus metrics endpoint.  I'm sure other tools could be used as a sidecar container to poll postgres and output prometheus metrics, but we don't need to worry about that.
The only containers left are webapp and websockets for web-modeler, and I don't think those components have a prometheus endpoint either.  I tried exec'ing into those containers, and then running wget on /metrics or /actuator/prometheus and I didn't get anything. Perhaps that could be enabled though, somehow.

This is actually a full feature rather than just a fix.
So I'd keep it and work on it after the 8.3 release.


Add @diogoteles08 as triager

I'm nominating @diogoteles08 for the triager role. Diogo has been participating and contributing over the past year:

issues
PRs

Following our contributor ladder:

He has been active in the community for more than 3 months
Has knowledge about the codebase, given his bug reporting + contribution
Has helped triaging previous issues. E.g.:

https://www.github.com/ossf/scorecard/issues/2444#issuecomment-1523890517
https://www.github.com/ossf/scorecard/issues/3075#issuecomment-1703152792
https://www.github.com/ossf/scorecard/issues/2850#issuecomment-1632950612

As mentioned in the contributor ladder, other @ossf/scorecard-maintainers may object, so I'll leave this issue open to allow time for comments. Due to the holidays, this will remain open at least through the new year.

+1. Thanks for your contributions so far @diogoteles08 !

Thanks for the contributions @diogoteles08!

Added @diogoteles08 as triager (invite sent).

If you don't see the invite (others haven't been notified in the past), you can find it at https://github.com/orgs/ossf/invitation


validate genesis header in light sync

What was wrong?
Fixes #892
How was it fixed?
When receiving a genesis header, check if it matches the canonical header already at block number 0. If they are not equal, raise a validation error that will drop the peer.
Cute Animal Picture

Ready for re-review


feat: support appending batch messages using hstreamdb_producer:append/2

LGTM! let us wait the actions run


Create database export script

No default basic auth password needed
Generate a JSON file suitable for importing into new database

Changes in this PR
Screenshots of UI changes
Before
After
Next steps

As per the RFC on this, we should write data to /tmp/.

Thanks for bringing this to my attention: done.


Log: Add validateLogLevel() method

The logLevel is determined at compile time and the checker does not work.
So the method is add to validate logLevel at runtime.

Fixed the verbose operation error of ovi_session


Add new OpenID Connect auth endpoints

Add integration test for oauth2 authentication.
Fix bugs in NullDataStore

Will merge on green


Circleci windows build config

In response to https://github.com/segmentio/aws-okta/issues/119
I was working on this before you guys shut things down and lost track of things in the shuffle.  I was doing some build related work in this area and picked it back up.  I lost my CircleCI access so unable to fully test recent changes.  But this config was working when I tested it a month or so ago.
You don't need to build on a Windows machine, this works building in the same Linux environment that the the Linux build runs.  The previous issues with dependencies not being included seems to have been related to how the Go vendor logic is built to be shallow.  See: https://github.com/nomad-software/vend
The build process uses the wixl command from MSITools to generate the MSI file using the spec defined in the WindowsMSI.wsx file in the repo root.  See: https://wiki.gnome.org/msitools
It outputs a raw executable file along with an MSI.  The MSI will install the exe file into the Windows AppDataFolder for the user, update the Path environment variable and set appropriate values for uninstall etc.
I followed existing patters in the .circleci/config.yml file to include the Windows build artifacts in the github-release process.
The Windows MSI build process is sensitive to version strings and requires semantic versioning.  Instead of doing fancy regex and parsing to extract the information from the git describe --tags --always --dirty="-dev" output I just set it in a version text file that gets parsed in the Makefile logic.  There is opportunity here to leverage github actions to keep the version file contents up to date as tagging happens.  Currently the contents of this file are static and needs to be updated to follow the current build version.
Another reason for the version file to exist, not related to CircleCI, but AWS CodePipeline processes were choking due to limited git metadata being available for various reasons.  The git describe command was failing.  Need to put a bunch of workarounds in place to get that working.
Just dropping this here just in case you guys want to include it.  I am not waiting for it.  We have our own build process running in AWS for the Windows stuff.  Just being a "good citizen" and passing it your way since the conversation was open.
Happy to answer any questions,
-Dave

Hey @dkujawski, thanks for this. I'm sure some Windows folks will find it useful. As you know, we're shutting down. Looks like the community is rallying behind aws-okta/aws-okta so you might want to cross-post there.


Accommodate non-Azavea personnel interactions with development setup

What were the issues that needed to be addressed as part of this?

I don't really remember. I think there was a desire to make it easy to spin up the OAR locally so folks could contribute. For example, scripts/setup will fail for an outside contributor because there isn't documentation on how to setup the .env files on S3.
This issue was also created before the transition to Django, when we had inherited the NodeJS application, so the landscape may have been super different at that time too.


Adding Event trigger average for all dimensions

Additionally : changing jitted_functions.py to _jitted_functions.py

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

16 of 41 (39.02%)  changed or added relevant lines in 8 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-59.5%) to 23.682%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pynapple/core/ts_group.py
2
4
50.0%

pynapple/process/perievent.py
0
3
0.0%

pynapple/process/tuning_curves.py
1
21
4.76%

Files with Coverage Reduction
New Missed Lines
%

pynapple/core/time_series.py
1
23.25%

pynapple/process/tuning_curves.py
2
4.24%

Totals

Change from base Build<PHONE_NUMBER>:
-59.5%

Covered Lines:
674

Relevant Lines:
2393

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

16 of 41 (39.02%)  changed or added relevant lines in 8 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.1%) to 83.089%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pynapple/core/ts_group.py
2
4
50.0%

pynapple/process/perievent.py
0
3
0.0%

pynapple/process/tuning_curves.py
1
21
4.76%

Files with Coverage Reduction
New Missed Lines
%

pynapple/core/time_series.py
1
49.86%

pynapple/process/tuning_curves.py
2
31.1%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
2040

Relevant Lines:
2393

💛  - Coveralls


[Bug] [Source Http]  params is invalid

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
Http Source  params is invalid!
SeaTunnel Version
dev分支（2.3.2-SNAPSHOT）
SeaTunnel Config
env {
    execution.parallelism = 5
    job.mode = "BATCH"
    job.name = "http2mysql"
}

source {
    Http {
        url = "http://<IP_ADDRESS>:8080/api/demo2"
        method = "POST"
        format = "json"
        params =  {
            id = 1
            }
        json_field = {
            id = "$.value[*].id"
            lrrq = "$.value[*].lrrq"
        }
        schema = {
            fields {
                id = int
                lrrq = string
            }
        }
    }

}
transform {

}
sink {
    console {
    }
}

Running Command
./bin/seatunnel.sh -c config/http2mysql.conf -e local

Error Exception
no error

Zeta or Flink or Spark Version
2.3.2-SNAPSHOT
Java or Scala Version
No response
Screenshots

The interface returns the following：

Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Find similar issues below:
#4497

@Hisoka-X  http params is form header  body is json i want to improve doc please assigned me

Thank you @jackyyyyyssss . But I have a question, does you metioned code are reason of this issue?

param is use httpheader application/x-www-form-urlencoded  he want to application/json httpheader  use  config like this can resolve

l


vim sux

This is an important patch. I proved it correct.

I encoded the Zermelo-Frankel set of axioms including the axiom of choice and axiom of vim sucks in a thousand line Coq script. It generated a haskell program which deleted vim/* and vimrc

+1

Haha what's going on?


add comments for check_and_fix_proto_format

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Can someone help review ?


Add Options to Email Subscriptions & Listservs

We need to add the following to Email Subscriptions

Subscribe to your regional emaillist (tied to the zip code in the person's account)
Training Updates
Members of the Press

If we could add the following to listservs:

Subscribe to your regional listserv
One question about listservs. We have several that are closed to the public. Will only the users who are on the list be able to manage their subscriptions?

Are Training Updates and Members of the Press Listservs?
No problem with adding regional listserv functionality.
There's no great way to already show listservs that people have previously signed up for, because we haven't really linked their accounts.  We may have to make the modification manually in the future.  You will likely receive questions from people who report the website shows them as having not signed up, even though they already have been.  They could just flip the switch under their profile in this case, which would really do nothing.
So basically, no, I can't show private Listservs to people who are already subscribed to them because the website can't query members of a Listserv.

Thanks. I wonder if we could add a disclaimer that lets people know this? Something like:
If you are already subscribed for a WCASA listserv, that information will not display here. In order for your profile to accurately reflect your memberships, please select the listservs you are interested in. You will still only be subscribed once. Thank you.

Ideally Vertical Response would be updated as well as Listserv list when toggling from subscriptions panel.


[update] Fix bug for escaping backslash \.

I found replace method in String (e.g., str.replace("\", "\\")). Is this sufficient for escape/unescaping?

I'm so sorry. That's right.
I'll delete unnecessary blocks and implement replace method to do so.
Then I'll commit it.

The JString class can't accept the string including a single backslash. Therefore the replace method is necessary at that point.
Ok, I'll add the comment above this line.


Organized Workflows

Summary
This puts the workflows into their own folders to help improve the organization of the repository. This also helps to clean up the "Workflows" UI on the GitHub Actions tab as it won't list all of the reusable workflows. Documentation was updated as the result of the reorganization.

This closes issue #9


How to change color of the Drawer Icon (three horizontal lines)

I am using DrawerController for my project.
I am wondering about
1 - How to change color of Drawer icon ( three horizontal lines). For now they are grey color
2- How can I hide Right Side Drawer  ?
Thanks

Hi,
1 - Take a look in the AnimatedMenuButton class, if you're using the animated drawer icon. layer.strokeColor is the line you'd need to change. Or feel free to add some customization options to the AnimatedMenuButton and submit a pull request :)
2 - I'd suggest to check out the KitchenSink sample project. Everything can be found there. It's unclear to me if you want to avoid having a right side drawer at all or just want to close the right side.

Thanks @maltebaumann  !  I am loving it so far. Have few suggestions that I will tell you at some point.


Add automerge workflow using Github Action

What changed?
Using Github Action https://github.com/marketplace/actions/merge-pull-requests#configuration
to automatically merge pull requests when they are ready

Why?
Our merging experience is not ideal. Merging into master has to be serialized. But today to merge a PR after getting a review approval, we have to manually click a "update branch" button to rebase even if no conflicts, and then rerun the whole buildkite tests, after 15-25 minutes when the tests are passed,  then click another button to merge it.
This Github action will automize the above process:

First of all, the author of the PR must have access to the Cadence repo. Otherwise the workflow won't have merge permission.
After getting a review approval, then put a label "automerge" in a PR, the PR will kick off an "auto-merge" workflow.
The workflow will wait for all test runs are succeeded, and then try to do a non-conflict rebase, and then merge it.

If rebase has conflict then  the workflow will be aborted.
The merge request message will be the title of the PR, which is the same as we are doing today.

How did you test it?
Tested here in my personal repo: https://github.com/longquanzheng/cadence-lab/pull/4
Note: when testing in my repo, seems like there is an issue that the PR has to be made from cadence repo. Forked repo will complain about an error "Failed to merge PR: Resource not accessible by integration"
And seems like this is on purpose because of a security reason:
https://github.com/actions/virtual-environments/issues/1609
In the future, we should get this feature without Github Action: https://github.com/github/roadmap/issues/107
It's on the roadmap but there is not timeline when this will be delivered.

Potential risks
No risk.

Coverage increased (+0.02%) to 68.934% when pulling 35a32a7684c0da94b2cd7c891e8ebfc9b3ff7489 on longquanzheng:auto-merge-2 into d6c60dd71347ebdb3e068ec50e9ea7e798faeae6 on uber:master.


List / bullet text increases size when nested

Hi @thomasvaeth - I'm back.
I noticed funny behaviour when populating the site. The text on nested lists increases, the further nested the list is. I expected the font-size stay the same. I tried to peek into _assets/scss/base/_lists.scss (line 28) but to no avail.
This issue seems to describe the same problem.
Reproducing the bug:
This code...
* I have a coin
  * I have a coin
    * I have a coin
      * I have a coin

All projects throughout

All projects throughout

All projects throughout

All projects throughout

All projects throughout

All projects throughout

... Produces the following output:

It's because the font-size in the _list.scss stylesheet is in em and not rem. The site is going to look a lot different i I change all the sizing to rem.
I might just have to ignore this for now and maybe someone else will do it in the future. I just don't really have time to go back in this project now.

I'm no front-ender but I'll give it a shot with my own code.


Code Sample is not using the latest version and has a typo

AuthenticationContext.AcquireToken is not available anymore.
In a comment the following is mentioned: The Usage Service/API is currently in preview; please use 2016-06-01-preview for api-version
But in the Code 2015-05-01-preview is used.

Regarding AcquireToken(), not sure if we will upgrade the samples, so they will continue to use ADAL v2.x.
Also fixed the comment.
Thanks @Rookian.


openxr project cannot open in unreal4.27

I also cannot open the project in unreal 4.27. it said that Microsoft openxr is for 26, although I install it from epic on both versions26 and 27.  even if I accept this and run it on 4.27version. with that version, I cannot run the project and I get lots of errors. here is a sample of the image you can see

Change this line from "4.26.0" to "4.27.0" :
https://github.com/microsoft/Microsoft-OpenXR-Unreal/blob/fccb12a31070bab0d45e8e948f809e6dbdde5937/MsftOpenXRGame/Plugins/MicrosoftOpenXR/MicrosoftOpenXR.uplugin#L13
Then right click on the uproject and select "Generate Visual Studio project files"
This happens because the source project and plugin are developed in 4.26 so we can maintain backwards compatibility.
The releases here: https://github.com/microsoft/Microsoft-OpenXR-Unreal/releases
have already done this (see 4.27.zip in each release's "Assets" dropdown), which are then uploaded to the Unreal Marketplace.

@fieldsJacksonG
thanks for answering I check your answer. it is not from there. the EngineVersion in MicrosoftOpenXR.uplugin was 4.27.0. I still get the same errors. and the release version 4.27.zip is not working. so I cannot even open it on my computer.

.

How are you opening the project after changing the version to 4.27 (from 4.26)?
Are you right-clicking on the uproject and switching the engine version to 4.27, then opening the sln and building from Visual Studio?
I can repro this by opening the 4.26 project (from GitHub source) from a 4.27 engine and converting as a copy.  This looks like what you have done, since the " 427" appended to your project name "MsftOpenXRGame 4.27" is added from a 4.27 copy-conversion.  However, after I change the uplugin file to 4.27, this does go away for me.
Please make sure you have modified the uplugin file in "C:\users\stud8\Desktop\projectopenxr\Microsoft-OpenXR-Unreal*MsftOpenXRGame 4.27*\Plugins\MicrosoftOpenXR" rather than the original source  "C:\users\stud8\Desktop\projectopenxr\Microsoft-OpenXR-Unreal*MsftOpenXRGame*\Plugins\MicrosoftOpenXR"

I fixed it myself. after changing the version of the project the way you told me. I deleted the plugin 4.27 and put the latest version from the GitHub inside of the project. then It will work correctly.


How to get terragrunt directory name ?

Hi there
I've been reading the doc, I fail to see how I can get the terragrunt.hcl directory name only, meaning not the path ?
I'ld like to use it to fill in some metadata, most of the time my service name equals to the directory name where it's defined.
Thanks,

Did you try the dirname() function? https://www.terraform.io/docs/configuration/functions/dirname.html

Tried this, without success
inputs = {
  service = "${dirname(get_terragrunt_dir())}"
}

I suppose I'm not following what you are looking for. Maybe provide enough of an example of what you want, including an expected result?

I've a lot of directories named my_service_test, my_service_that, my_service_this, etc. With just one file, terragrunt.hcl.  calling a module with just one input
inputs = {
  service = "my_service_test"
}

Having a get_directory_name() function (or equivalent) like this
inputs = {
  service = "${get_directory_name()}"
}

Would save time for me, and be less error prone when I c/c this a dozen of time to create a bunch of new things. Of course I could develop a new module to loop over a list of services, but that's not the way I want to do it atm.

ah. did you try the terraform function basename()? https://www.terraform.io/docs/configuration/functions/basename.html
> basename("foo/bar/baz")
baz
``

Indeed, "${basename(get_terragrunt_dir())}" works. Thanks !


Hi Bri......it's another test. Sorry | Feature Request

Updating Bot

🔗 Discuss this issue on Discord


Complete action server implementation

I have made some progress in the action server implementation.
There are still some things missing, but it would be good to get early feedback on some details:

Modified GoalHandle interface so that terminal state transitions take a result message as a parameter (e.g. goalHandle.execute(result)).
Added a publishFeedback() method to GoalHandle (not implemented yet).
(partially) Handle result requests (not tested yet).
Added some interfaces to improve type safety: ResultRequestDefinition, ResultResponseDefinition, FeedbackDefinition, ResultDefinition, GoalDefinition
Modified message generation for implement those interfaces when needed.
Refactored GoalStatus to be able to get the byte equivalent easily.
Created some helper methods to avoid repeating code (createGoalInfo, goalExists, sendResultResponse)

TODO:

[x] Handle terminal state transitions.
[x] Publish feedback.
[ ] Add tests.
[x] Add example (https://github.com/ivanpauno/ros2_java_examples/pull/1).
[ ] Think about thread safety.
[ ] Clean up code.

@jacobperron this is not urgent, but some early feedback would be appreciated.

I have added a bunch of new commits as well

This is almost complete now, see example code here (which works!).
Remaining TODOs are tracked in the OP.

I think this is ready for review now.
I will address the remaining TODOs in new PRs.


Document /uninstall and /repair options for .NET

Summary
Update .NET installer command-line options to include /uninstall and /repair. There were internal asks for official documentation.

Internal previews

📄 File
🔗 Preview link

docs/core/install/windows.md
Install .NET on Windows

@adegeo I think this is ready to go. Merged all the suggestions and fixed the lint error

Sorry, I was out sick! Thanks again!


Remove bunshin from support summary

Observe issue #1926
Observe issue #1926 go away
Something something react routing, idk

ah i see what's going on. adding @observer actually just hides the issue, not fix the problem.
@observer implies extends PureComponent (effectively), and PureComponent means it will avoid re-rendering if none of its props change. Component will always just re-render. in essence, adding @observer is papering over the issue by just preventing it re-rendering at all (because it has no props that will change).
this... is getting a bit into react-weeds, but the underlying issue is that regardless of the Pure-ness of the component declaration, the render method (or the function body in non-class instances) must be pure - i.e. if it runs render with the same props and state twice, it should get the same output each time.
this isn't held by the current impl of this UI, as it has a class property (roleJobs) that is mutated during render - i.e. if render is called twice, you end up with two sets of values in roleJobs.
there's a whooooooooooole bunch of ways to approach this and so on and so forth, but in this case the most dead-simple approach would be to delete the roleJobs class property, and then use an array in the render function a la
 // Build the list of roles and their associated jobs
 let jobKey: JobKey
+const roleJobs = []
 for (jobKey in JOBS) {
 	const roleKey = JOBS[jobKey].role
 	if (roleKey === 'UNSUPPORTED') { continue }
 	const roleData = this.roleJobs.find(rd => rd.roleKey === roleKey) ?? {roleKey, jobKeys: []}
 	roleData.jobKeys.push(jobKey)
 	if (!this.roleJobs.some(rd => rd.roleKey === roleKey)) {
-		this.roleJobs.push(roleData)
+ 		roleJobs.push(roleData)
 	}
 }

 return <>
 	{supportMessage}
 	<div className={styles.summary}>
-		<SupportSummaryGrid roles={this.roleJobs} />
+		<SupportSummaryGrid roles={roleJobs} />
 	</div>
 </>


Added my details

Project
Name of the project
Changes
What was updated (new pages, weekly update, etc.)
Notes
If there are any further notes.

done, please check

please check and merge


optionally preserve evidence in model copy

See requirements in #512.

@vanaukenk @kltm I've got a PR for this now at #514. The PR is already merged into the dev branch, so you can update the code on minerva-dev for testing.

@tmushayahama when calling the model copy operation, provide an argument preserve-evidence with a boolean value of true or false, in the same place you provide the model-id argument.

This should now be "live" on noctua-dev.

Testing by @ukemi @vanaukenk @pmasson55
All looks okay to move to production on the 2023-05-25 maintenance outage


Add dynamic timeout feature

This adds a feature to set the Distcp timeout to scale with the file input size, which can subject to a minimum (because of distcp overhead) and a maximum( to resemble older behavior).
@plypaul

@saguziel Please take a look at the build failure.

@plypaul  ptal

Thanks @saguziel ! Merging now.


fluxcd: 0.17.1 -> 0.17.2

fluxcd: 0.17.1 -> 0.17.2
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 138717 run on x86_64-linux 1

1 package built:

fluxcd


[Request] Allow admins to see which users have 2FA setup.

Is your feature request related to a problem? Please describe.
I want to ensure all users implement 2FA. A feature where 2FA could be enforced on registration or first sign-in could be useful but a steeper development path. In the meantime a simple icon or indicator that a user has 2FA setup or doesn't would be enough information for most admins considering the user case for Puffer Panel is likely smaller, self hosted deployments with few users.
Describe the solution you'd like
A simple icon or text field that indicates whether a user has setup 2FA
Describe alternatives you've considered
Enforced 2FA, probably a steeper development path.
Additional context
I may try to implement this myself as a contribution.

Pull request #1373 opened with proposed implementation

Proposed implementation in #1374


visualizing quantum data

I think we need to think about the visualization section. It is really a visualization of quantum data of something. I think this should also go into the qiskit terra section.
What do you think @derivation?

I think the top level doc ToC should not include Visualizations, as it does now. Instead, a more logical break between overview material for all of Qiskit vs each element/component is
Overview
Installing Qiskit
Getting Started
Running on IBM Q Devices
Each element/component
Roadmap
etc.
The current Visualization section could be moved as-is under Terra's section, renamed something like "Visualizing Quantum Data", coming after "Visualizing a Quantum Circuit".
The workflow step of visualizing quantum data should also be called out more prominently in the "Getting Started" section, where it's currently done in passing while simulating/experimenting. It should be noticeable in the global ToC. Perhaps a subsection can be appended to "Getting Started" where the prior results of simulating and experimenting is explored and discussed a bit further.

I agree


Existing cluster cannot be adopted

What happened:
After recreating my local kind cluster + installing CAPI to it along with the cluster manifests (the OKE cluster already exists, I am just recreating kind cluster), the CAPOCI provider was unable to use the existing OKE cluster
namespace="default" name="banshee" reconcileID=e7c4e3ec-6ca7-4200-a0bb-e7ac4e538ef5 OCIManagedCluster="default/banshee"
E0723 06:46:40.994077       1 controller.go:329] "Reconciler error" err=<
        failed to reconcile OCI Managed Control Plane default/banshee: Error returned by ContainerEngine Service. Http Status Code: 400. Error Code: LimitExceeded. Opc request id: dba88a18f8184599c46a93f633c442a5/CA01F8F1ED78ED2575932E866CA322E2/883BB12D2BDAA9905997455E4CDE0DD3. Message: The cluster limit for this tenancy has been exceeded.
        Operation Name: CreateCluster
        Timestamp: 2023-07-23 06:46:40 +0000 GMT
        Client Version: Oracle-GoSDK/65.40.1
        Request Endpoint: POST https://containerengine.ap-melbourne-1.oci.oraclecloud.com/20180222/clusters
        Troubleshooting Tips: See https://docs.oracle.com/iaas/Content/API/References/apierrors.htm#apierrors_400__400_limitexceeded for more information about resolving this error.
        Also see https://docs.oracle.com/iaas/api/#/en/containerengine/20180222/Cluster/CreateCluster for details on this operation's requirements.
        To get more info on the failing request, you can set OCI_GO_SDK_DEBUG env var to info or higher level to log the request/response details.
        If you are unable to resolve this ContainerEngine issue, please contact Oracle support and provide them this full error message.
 > controller="ocimanagedcontrolplane" controllerGroup="infrastructure.cluster.x-k8s.io" controllerKind="OCIManagedControlPlane" OCIManagedControlPlane="default/banshee" namespace="default" name="banshee" reconcileID=e7c4e3ec-6ca7-4200-a0bb-e7ac4e538ef5

What you expected to happen:
I expected that CAPOCI would see that an existing OKE cluster was already present and would adopt it.
How to reproduce it (as minimally and precisely as possible):

Create an OKE managed cluster using kind
Delete the kind cluster
Recreate the kind cluster and deploy the same cluster manifests

kind create cluster --config kind.yaml || true
kind get kubeconfig --name capi > /tmp/kind.yaml
export KUBECONFIG=/tmp/kind.yaml

clusterctl init --infrastructure oci --wait-providers --addon helm --config ./clusterctl.yaml
clusterctl generate cluster banshee --from template.yaml --config ./clusterctl.yaml > cluster.yaml
kubectl apply -f cluster.yaml --wait

clusterctl get kubeconfig banshee > ./banshee.kubeconfig
kubectl cluster-info --kubeconfig ./banshee.kubeconfig

Anything else we need to know?:
using the https://github.com/oracle/cluster-api-provider-oci/releases/download/v0.12.0/cluster-template-managed.yaml template
Environment:

CAPOCI version: v0.12.0
Cluster-API version (use clusterctl version): clusterctl version: &version.Info{Major:"1", Minor:"5", GitVersion:"v1.5.0-rc.1", GitCommit:"038b533a120058dff6f3ae985981bdc2bd844991", GitTreeState:"clean", BuildDate:"2023-07-18T15:37:43Z", GoVersion:"go1.20.4", Compiler:"gc", Platform:"linux/arm64"}
Kubernetes version (use kubectl version):

Client Version: v1.27.2
Kustomize Version: v5.0.1
Server Version: v1.27.3

r

Docker version (use docker info):
not relevant
OS (e.g. from /etc/os-release):
not relevant

@alam0rt CAPOCI cannot adopt an existing cluster, the functionality does not exist. May I please know the use case? in general Cluster API, be any provider, does not have facility to adopt a cluster.  You can use an existing cluster as a management cluster, but you cannot adopt an existing cluster.

Hey @shyamradhakrishnan, in order to upgrade/manage the management cluster, I was hoping that I could provision a new kind cluster + clusterctl init + apply the existing management manifests into it and have it reconcile the existing OKE cluster.

@alam0rt ideally you should use Cluster API pivot procedure https://cluster-api.sigs.k8s.io/clusterctl/commands/move.html to make the management cluster(which I am assume is an OKE cluster) self sustaining. Unfortunately, since you have already deleted the kind cluster, there is nothing that can be done at this point. As I said, reconciling existing objects is not something Cluster API has in-built, there are some conversations happening in the community but nothing concrete. You can create an OCIManagedCluster/OCIManagedControlPlane object, rather than a empty one as you are doing right now, and put it all the necessary OCIDs etc after comparing it with an existing Custom resource, but it would be a time consuming process. The simplest for now will be to delete and recreate the setup if at all that is possible.

Ah, thanks for the pointer. That's no issue at all, easy to recreate :)


Access all outstanding errors from the form

As my form in my application takes up a good amount of vertical space, I'm trying to add a section to my form beneath the submit button that will tell the user exactly which validations are still failing, kind of like
this:

This would be in addition to the error messages that appear next to the individual inputs. Is there any easy way to grab a list of all the outstanding failed validations? I was looking at this.errors in the controller, however, there doesn't seem to be any aggregate object that gives me an array of all the outstanding errors. The only way to get this information seems to be accessing each individual validation property by its unique name, like this errors.get('buyerFirstName').
Is there an easier way? Thank you!

From a usability and form conversion perspective this approach has been proven to not be optimal. You can see some of that research here: http://alistapart.com/article/inline-validation-in-web-forms
That being said, we should strive to be as flexible as possible with ember-validations. You can probably just add something like the errorArray CP I've added here: http://emberjs.jsbin.com/kufefenufa/1/edit?html,js

Thanks @bcardarella for the great answer!

The longer term solution would be for us to make the errors object Enumerable. I've marked this as a 2.0 blocer


can nextjs be used in a non-javascript framework ?

hi guys,
nextjs is something that we have watched with a lot of excitement. However, the big issue we had was that our api stack is already in python+flask. We want to build several admin dashboards and we are seriously considering nextjs... but it is not obvious on how this will work.
is nextjs supposed to be used as the view layer of a non-javascript stack : say flask or rails ? Somewhat like this - https://realpython.com/blog/python/the-ultimate-flask-front-end/
or is it intended to be a javascript framework ?

I don't know if I'm answering your question but next.js is not supposed to replace your backend. Next is a client-side framework that happens to have a server-side component to allow features like HMR and SSR. You can definitely combine it with your flask app.

The best description to give Next.js is a frontend server. It can communicate with any backend using getInitialProps 😄 This is a good example: https://github.com/zeit/next.js/tree/master/examples/data-fetch

hi Dan,
thanks for the answer!
does nextjs need to "run" using a nodejs instance... or can it be built as
a static page with everything coming from a CDN ? we would rather not bring
in a nodejs dependency into an existing stack.
I think I am looking at this - https://github.com/zeit/next.js/issues/604 .
If this gets done, then the nextjs code gets compiled as a pre-build step
and deployed with the flask app (where it gets served as a static asset).
no nodejs dependency!
I think this issue can be closed - basically, we are looking at server side
reactjs to have zero dependency on nodejs.
On Wed, May 10, 2017 at 6:39 PM, Dan Zajdband<EMAIL_ADDRESS>wrote:

I don't know if I'm answering your question but next.js is not supposed to
replace your backend. Next is a client-side framework that happens to have
a server-side component to allow features like HMR and SSR. You can
definitely combine it with your flask app.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zeit/next.js/issues/1933#issuecomment-300476550, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEsU8qoVUa-5n8QMSf2PO1i1XTz6IaPks5r4bcIgaJpZM4NWleL
.

hi Tim,
I think that description is going to change after
https://github.com/zeit/next.js/pull/1576 gets released right ? Because
then the "server" part goes away and so does getInitialProps (maybe
replaced with getStaticInitialProps)
On Wed, May 10, 2017 at 7:08 PM, Tim Neutkens<EMAIL_ADDRESS>wrote:

The best description to give Next.js is a frontend server. It can
communicate with any backend using getInitialProps 😄 This is a good
example: https://github.com/zeit/next.js/tree/master/examples/data-fetch
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zeit/next.js/issues/1933#issuecomment-300484469, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEsUzfvmB9f93JILWetrvEKdohlrwasks5r4b3ogaJpZM4NWleL
.

@sandys exactly. You'll be able to use next export and just use it without the nodejs side. AFAIK there are people already using that branch

Exactly like Dan said 👍

What would the best way to provide CSRF tokens and base URL to the next app? Query parameters?


Latest value metric docs

#2228  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @craig-statsig and the rest of your teammates on  Graphite

🚀 Deployed on https://671ab595a93d5479f6e364b9--cozy-fox-0defec.netlify.app


InExpression.Print() makes wrong assumption about Values property

We came across this issue, which we resolved by changing our code to the new assumption, but it should be checked, if this breaking change was intended.
https://github.com/aspnet/EntityFrameworkCore/blob/130fe39bed1c9351ba73a829aa14a6112108a6b2/src/EFCore.Relational/Query/SqlExpressions/InExpression.cs#L55-L82
It is assumed here, that constantValues is an IEnumerable<SomeClrType>:
https://github.com/aspnet/EntityFrameworkCore/blob/130fe39bed1c9351ba73a829aa14a6112108a6b2/src/EFCore.Relational/Query/SqlExpressions/InExpression.cs#L61-L62
Additionally, it should be assumed (and checked), that constantValues is of type IEnumerable<SqlConstantExpression<SomeClrType>> and its items should be visited/printed.
The current (probably false) assumption can lead to exceptions being thrown further down the stack, when the InExpression is getting printed.
In case it is expected to call SqlExpressionFactory.In() only with an IEnumerable<SomeClrType> instead of IEnumerable<SqlConstantExpression<SomeClrType>> in 3.0 (which does work), this would be a breaking change from 2.2.6, where the latter works fine.

In our concrete case, we are creating an InExpression over StringComparison enum values by feeding it an SqlConstantExpression<IEnumerable<SqlConstantExpression<StringComparison>>>
Later, InExpression.Print() calls GenerateSqlLiteral(value: SqlConstantExpression(StringComparison:Ordinal)), which then calls the EnumToNumberConverter.
While the converter sanitizes the value, the following exception is thrown:
System.InvalidCastException: 'Object must implement IConvertible.'

This is due to the fact, that Sanitize<T>(object value) expects the concrete value, and not a value wrapped by a SqlExpression.
To fix this issue in our code base, we changed our call of SqlExpressionFactory.In() to use a SqlConstantExpression<IEnumerable<StringComparison>>.
Further technical details
EF Core version: 3.0+

In 3.0 release, the InExpression.Values is supposed to either SqlConstant or SqlParameter. And value of each is of IEnumerable.
SqlConstantExpression<IEnumerable<SqlConstantExpression>> is disallowed. All the values in Values should have same type mapping and there is no reason for inner values to have SqlConstant to specify different typeMapping.


Use pkg-config to find libxml2, falling back to xml2-config if necessary

Debian is taking steps to remove xml2-config in favour of pkg-config. This means ModSecurity will build without libxml2 support by default on Debian, Ubuntu and other distributions tracking Debian packages.
This patch modifies build/libxml.m4 to check for libxml2 via pkg-config, falling back to xml2-config if necessary.
Patch for ModSecurity v2, as discussed in #2714.

Here you go, @martinhsv.

Merged. Thanks @hughmcmaster .


Add Commitlint

I'd like to help with this one, but I'm not sure if you wanted a github action to check it or if you wanted to use the node commitlint binary to check on commit.


Create a new tip for Elemental Selenium

A former Sauce Labs created a new tip that was never published!
The tip can be found here: Tip 73 Open a New Window or Tab – Selenium 4

[x]  Create a new tip following the new tip template
[x] Add it to Elemental-Next

New tip added to updated tips!


Add validators for workup amount

Addresses https://github.com/open-reaction-database/ord-schema/issues/573

Addresses #573

Please use one of the recognized keywords: https://docs.github.com/en/github/managing-your-work-on-github/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword


Offline capability

Hey! I've been using this library for a bit now, and its really amazing for what it can do!
However, currently, when I open my project and then spawn a thread, each time there are still requests going out, so far that it doesn't work if my browser is offline (I also don't have a caching serviceworker, which might be able to help).
Precisely there's 4 requests going out, 2 of which transfer 0 bytes and still seem to work offline, 2 others where at least the first fails when offline.
To be entirely honest, I don't really understand the entire business with the js scripts thats described in the readme. However, it is my understanding that this is required because webworkers require to be started from a URL. BTW, the urls in that readme are dead, the paths they reference have changed. Its easy enough to find the correct files, but it might be worth a quick fix.
I did a bit of looking around, and this stack overflow question and answer imply that its possible to use a data-url for the serviceworker.
This should mean that its possible to do this request, from my understanding, without that complicated hack to get a url, and its possible entirely offline.
I'm uncertain whether this will actually enable everything to work offline. I'm under the impression that one of the 2 network bearing requests in my case is for the generated js-wrappers from wasm-bindgen.
But I hope that you may have more insight into that, and I thought perhaps you would be interested in using a data-url to avoid the current hack, even if this doesn't add any features.

So yeah.  Data URL doesn't help there, aside maybe from being able to replace the blob urls with the data urls, which avoids having to manage the blob resource.
Upon further inspection, it seems that these requests come in 2 pairs of blob url + request. One pair is the worker module polyfill. If I configure out that code, then I avoid one non-load-bearing and one load-bearing request. Which could be a PR btw. I was unable to remove the initial request, even with cfg_attr, but any subsequent request can be avoided.
The other part is the original wasm_bindgen shim. That load is triggered from the importScripts action in web_worker_module.js. The url gets inserted here.
I wonder why you don't url-encode the original shim and provide that url with search and replace? AFAICT, if you do that, and the same with the polyfill (or make it easy to configure out, as I was showing) then it should work out?
Currently you just encode the worker script itself, which, when compared to the wasm bindgen shim, should typically be much smaller.

Another interesting thing to me, is that the wasm script only gets downloaded a single time. This should happen in the generated shim file. I would like to investigate what that script does to avoid another download for the worker. I can't even see a blob url request or similar.

Another interesting thing to me, is that the wasm script only gets downloaded a single time. This should happen in the generated shim file. I would like to investigate what that script does to avoid another download for the worker. I can't even see a blob url request or similar.

My understanding is that the initialization code creates a global wasm variable, which seems to also be accessible from the worker(?!) which you then use in the wasm_thread_entry_point to immediately jump into wasm.
At least I cant find any root-level statements in my generated shim aside from the stuff at the very beginning, which only declares the wasm variable, but doesn't assign it. It only gets assigned in a function, that AFAICT only gets invoked in the main thread in the beginning.
So the wasm global is available in workers by default? I'd assume its a copy, if present, not shared?

So the wasm global is available in workers by default? I'd assume its a copy, if present, not shared?

oops. I overlooked the init call in the worker script. Well, that explains parts.

I've further investigated this. Solving this would be difficult, and the hack would remain. But it should be possible with a similar approach to whats done for the worker script, with a manual fetch request for the bindgen js file.
But it won't be easy.
I don't have the energy to do this. I've since reevaluated using serviceworkers for offline-capability and got things working to my satisfaction.
Feel free to close this, or keep it open if you care for it.


The OBJ file（3D model） in the downloaded sample data seems to have issues

I downloaded the sample data and loaded the corresponding OBJ file using Blender. However, it seems inconsistent with what is shown in the paper. Below are the original image and the visualization of the OBJ file. It appears that the upper part of the vehicle is not well reconstructed, But the paper shows good results. Could you explain why this might be? Or is there any additional step I need to take to achieve a better 3D model?

The image like this:

Hi. The obj file is generated by 3Dscanner. In our paper, we use 3DGS to reconstruct 3D models.

Hi. The obj file is generated by 3Dscanner. In our paper, we use 3DGS to reconstruct 3D models.

Hi. The obj file is generated by 3Dscanner. In our paper, we use 3DGS to reconstruct 3D models.

thx. I found the pipeline in the Paper

I noticed that the pipeline does not involve the 3D GS part. So, I need to take the provided COLMAP data format and perform 3D GS model reconstruction myself to obtain the final PLY file, correct?
Thank you for your assistance.

Correct.

Thats why I have not found any code about 3D GS? This part of code has not published yet.

@SophieZhou Hi. You can find 3DGS in their official repository. https://github.com/graphdeco-inria/gaussian-splatting

@SophieZhou Hi. You can find 3DGS in their official repository. https://github.com/graphdeco-inria/gaussian-splatting

Using the code from this link, I have trained my own model on your sample data using default parameters. But I got the rendering results a little poor, I do not know, what is the difference. Have you done any changes over gaussian splatting?  The datasets are generated using your code and the models in your code step by step.


CLI broken with Python 3: tries to decode STDIN as UTF-8 even when different encoding is specified

With ftfy installed as a Python 3 package on my Ubuntu 14.04 machine, this happens:
$ echo Æèçíü íà ìåñòå | iconv -f utf8 -t latin1 | ftfy -e sloppy-windows-1251
ftfy error:
This input couldn't be decoded as 'sloppy-windows-1251'. We got the following error:

    'utf-8' codec can't decode byte 0xc6 in position 0: invalid continuation byte

ftfy works best when its input is in a known encoding. You can use `ftfy -g`
to guess, if you're desperate. Otherwise, give the encoding name with the
`-e` option, such as `ftfy -e latin-1`.

With ftfy installed as a Python 2 package, it works as expected:
$ echo Æèçíü íà ìåñòå | iconv -f utf8 -t latin1 | ftfy -e sloppy-windows-1251
Жизнь на месте

BTW, awesome package!

Aha, yes, I should be using sys.stdin.buffer on Python 3 because sys.stdin already has an encoding it expects. I'll fix it.

Fixed by https://github.com/LuminosoInsight/python-ftfy/pull/49 .


Incorrect replace! of a Tensor in a TensorNetwork with the same Tensor

Summary
The replace! function for a Tensor in a TensorNetwork, where it is replaced with the same Tensor, leads to the deletion of that Tensor:
julia> using Tenet

julia> A = Tensor(rand(2, 2, 2), (:i, :j, :k))
2×2×2 Tensor{Float64, 3, Array{Float64, 3}}: ...

julia> B = Tensor(rand(Float32, 2, 2), (:k, :j))
2×2 Tensor{Float32, 2, Matrix{Float32}}: ...

julia> tn = TensorNetwork([A, B])
TensorNetwork (#tensors=2, #inds=3)

julia> replace!(tn, A => (A))
TensorNetwork (#tensors=1, #inds=2)

julia> tensors(tn) .|> inds
1-element Vector{Vector{Symbol}}:
 [:k, :j]

I think that there might be a conditional in the replace! function that checks whether it's the same Tensor, and this may be causing the problem.

I will merge this on monday @mofeing. Let's enjoy the weekend!


Reconfig Tree

I want to reconfig tree ,when user click button datasource must be change , how to do it?

Okey , I find it , just removeData of tree and it worked.

I recomend you to use the destroy method of the tree to fully destroy the tree and initialize the tree again with the new setting.