[prim_ram] High FPGA BRAM utilization

I just noted that the prim_ram_1p component instantiated inside the retention RAM has utilizes a lot more BRAM resources than expected. The memory has a width of 32 bits + 4 bits for parity and is 1024 entries deep. I would expect Vivado to map this to two BRAM36 instances. These are 36 bits wide and 1000 entries deep. However the actual utilization is 18 BRAM blocks.
As we are currently using 80% of the BRAM resources on the NexysVideo, we should probably investigate what's going on here.
I am not sure, it could also be that this is a known issue. Any ideas @tjaychen , @msfschaffner ?

Thanks for catching that @vogelpi!
Indeed, this is a problem, and I believe it is not only the retention RAM, but also the main RAM seems to be twice as large as it needs to be.
I.e. I would expect the tool to map the rams to the following resources:

main RAM: 16k x (32+4bit) = 16 RAMB36
ret RAM: 1k x (32+4bit) = 1 RAMB36

If I recall correctly, one has to be careful to correctly specify the byte write enables in RTL such that the tool can map that correctly. I think this is currently specified the in the wrong format within prim_ram_1p_adv, hence leading to a suboptimal mapping (i.e. the word is assembled as {data, parity}, while it should actually be {data[31:24], parity[3], data[23:16], parity[2] ...}). I'll take a look and try fix it.

Ok, I made a fix for this. Utilization goes down to 65% with the fix applied.

Awesome, thanks for the quick fix!


Update MapGroundOverlay doc for class reference error

I was confused at first. The link is correct, the text about the underlying JS class is wrong.

@googlebot I signed it!

@googlebot I signed it!


Disable close app by drag

Do you have any plan to disable closing webapp by drag when reach top scroll ( its not a function in telegram script just use clean way to disable drag with styles )

Omg its finnaly happend


Show site observation date in export

If a site image has an associated site observation, show the site observation's date. Otherwise, show "-obs".
A sample screenshot is provided below. (The observation date is arbitrary and not indicative of real data.)

So in the -obs case the observation object is None, so which date should we use?

So in the -obs case the observation object is None, so which date should we use?

The previously stored obs value.  It should be storing some obs value because that's how it knows the label and the polygon.

Updates:

Show the "last observation date". @BryonLewis do we want to show the initial last observation date as empty (e.g. "-----"), or as the first image date?

How about coloring the box to indicate such a state? Could be green for matching, grey for not matching. Or red for not matching, grey for matching. It would depend on if we want to emphasize matching vs not matching dates.

I would just leave it normal if the match and maybe a slightly different color if they are different.    I would just be worried if you set the FPS to like 5 and have the red/green flashing it could be too intensive.


Timeline/requirements for Ledger integration

Hi, are we still waiting on Ledger to integrate this? Is there a way to speed up the process such as if a desktop wallet supported the app?
Would much work be required to support Orchard/Unified addresses? I would say a grant from ZF or ZOMG would be easily supported once Sapling's running on Ledger.
Cheers

The problem is that nobody integrated the app in a web or desktop wallet and this was beyond our scope.
We have been discussing with ZF or ZOMG that we could actually integrate ourselves or even create a new wallet.
Without any integration of our app into a wallet, Ledger will never release.

This app has been getting old and needs also some maintenance as this stage:

Upgrade SDKs
support orchard, etc.

We really put a lot of effort to get this working in Nano S. Actually, it was more like a challenge and a contribution to the community rather than something economically efficient. So we would also love this see this app in the wild being used by people!

https://zcashcommunitygrants.org/ would welcome any proposal to get this in the hands of user.
Btw any reason why issue 21 was completely ignored?

It has not been ignored, however, it is difficult for us to justify continuous attention and support of this app after so much time since the completion of our milestones (years?).
In the original discussion, we suggested and preferred to be in charge of the integration to be 100% we had control over that timeline too.
Thanks for your suggestion, we will definitely reach out and see how we can reboot this project. This means upgrading the project and taking over integration work so we can ensure that the project gets to users.

If you're serious about getting this into the hands of end-users, submit a Zcash Community Grants and respond to the community feedback on the forum. Don't be surprised that people are disappointed with you folks when there has been no update whatsoever for so long on this project.

Hi @jleni, I have given a good effort to make this app run but unfortunately I have now run into a wall. The app cannot load anymore after a firmware upgrade. It seems to be just missing some small things.
Great work, it was clearly quite challenging.

FYI, I managed to use integrate it in Ywallet --> https://www.youtube.com/watch?v=IazybvQg68w

Moreover, I hope you can see that we handled some the changes here:

This app has been getting old and needs also some maintenance as this stage:
Upgrade SDKs
support orchard, etc.
We really put a lot of effort to get this working in Nano S. Actually, it was more like a challenge and a contribution to the community rather than something economically efficient. So we would also love this see this app in the wild being used by people!


Update kubeadm-join.md

There is no description in the kubectl certificate approve process, I added it to make the steps clearer.

Delete the invalid anchor.

Signed-off-by: ydFu<EMAIL_ADDRESS>

/lgtm

/lgtm

Link: https://deploy-preview-35706--kubernetes-io-main-staging.netlify.app/docs/reference/setup-tools/kubeadm/kubeadm-join/#turning-off-auto-approval-of-node-client-certificates
/approve


fix: globals startTime and all_keys

startTime was misspelled starTime when it was declared
all_keys had no associated var
removing unused util._extend

Thanks for contribution.


%patchX statement in .spec file without PatchX: declaration

Results in:
[110s] error: No patch number 0

https://build.opensuse.org/package/show/home:dmacvicar:java:tetra/HikariCP-java6
There is an extra %patch:
%prep
%setup -q -n src
%patch0 -p2
cp -f %{SOURCE1} .
cp -Rf %{_datadir}/tetra ../kit

Fixed with 9b6d065, please install version 0.51 and re-run tetra generate-spec.
Thanks for reporting


HER requires 'mujoco_py

I'm getting the following error when I try and run HER when RolloutWorker in imported:
ModuleNotFoundError: No module named 'mujoco_py'
I don't own a copy of mujoco.  Is there anyway this dependency could be removed?  I'm wanting to avoid forking out the ~$250 to try out the code.
Thanks,
Peter

+1 from me. I just use the deepq model and this would require additional license + setup. For now I am sticking with 0.14 release, but would like to be able to receive updates in the future without having to install mujoco.

Another problematic thing is that baselines requires Python 3, whereas the issues tracker over for mujoco_py recommends using a Python 2 version to fix some common installation errors.

I'm having this issue as well. I tried removing the mujoco package (I don't think you need it for other environments?) from the code but that only caused additional issues.

You'll need MuJoCo if you want to use HER. If you don't want to use HER, you can safely only install Gym without the mujoco and robotics environments.

Why do we need MuJuCo to use HER? Isn't it just a robotics environment? Wouldn't HER work fine on any other environments -- whether in gym, universe, or custom?

@zacheberhart It should. However, the environments that they use HER for are with sparse goal rewards, and those are in the MuJoCo-style tasks (I assume their robotics environments use MuJoCo).

You'll need MuJoCo if you want to use HER. If you don't want to use HER, you can safely only install Gym without the mujoco and robotics environments.

Why do we need MuJoCo? If it's only for sparse goal rewards there are other environments in this style. Also, OpenAI released roboschool to replicate MuJoCo environments and it's a bit of nonsense that we could use this with MuJoCo but we can't with roboschool.


thread.RetrieveRunAsync("run-id") does not find

var run = await thread.RetrieveRunAsync("run-id"); does not find RetrieveRunAsync in the namespace.

Sounds like you're missing a namespace using statement.

Is this from the example docs?
I'll double check but that might not actually be implemented.

Ah instead you can use run.UpdateAsync();
I'll update the docs

Thank you.

Gonna keep this open until I made the changes.
Thanks again!


Error

I am using Visual Studio 2019 and I get a "The name 'Host' does not exist in the current context." Is this caused by the new version? When I look at the potential Fixes option, it suggested I change Host to IHost. Is Host a typo?

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

ID: a0a4022c-7ef1-3c30-4f9d-cf521a57a44a
Version Independent ID: a0b5e15c-d747-8710-95fb-a55c09861a1e
Content: Work with SQL in an ASP.NET Core MVC app
Content Source: aspnetcore/tutorials/first-mvc-app/working-with-sql.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

The tutorial works. I'm not sure what step you missed. You might try asking on StackOverflow,.

Follow the tutorial. You might download the completed app and compare. Search for host on the doc page.


Sync main branch of forked repo

Fix https://github.com/jenkinsci/plugin-modernizer-tool/issues/81
Work in progress due to release needed on github-api
Testing done
Checking on logs
(e24ddd0) GitHub API request: POST https://api.github.com/repos/jonesbusy/jobcacher-plugin/merge-upstream

And checked the fork is sync from upstream
Submitter checklist

[x] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[x] Ensure that the pull request title represents the desired changelog entry
[x] Please describe what you did
[ ] Link to relevant issues in GitHub or Jira
[ ] Link to relevant pull requests, esp. upstream and downstream changes
[ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

Upstream PR is https://github.com/hub4j/github-api/pull/1898/files

Excellent news, thank you so much! 🤗

Excellent news, thank you so much! 🤗

No problem, was worth the time investment.


Ctrl+Alt+ keybindings should not be used by default under Windows

Offender:  e  for  editor.action.autoFix

Now it's using the windows key which is also a no-no? Related: https://github.com/Microsoft/vscode/issues/67695

Dang Joh for stealing cmd+shift+. for breadcrumbs... Let's try alt+shift+. on windows


dataset.shuffle(keep_in_memory=True) is never allowed

As of commit ef4aac2, the usage of the parameter keep_in_memory=True is never possible: dataset.select(keep_in_memory=True)
The commit added the lines
# lines 994-996 in src/nlp/arrow_dataset.py
       assert (
            not keep_in_memory or cache_file_name is None
        ), "Please use either `keep_in_memory` or `cache_file_name` but not both."

This affects both shuffle() as select() is a sub-routine, and map() that has the same check.
I'd love to fix this myself, but unsure what the intention of the assert is given the rest of the logic in the function concerning ccache_file_name and keep_in_memory.

I'm thinking the intended implementation here is
assert (
            keep_in_memory and  is None
        ), "Please use either `keep_in_memory` or `cache_file_name` but not both."

This seems to be fixed in #513 for the filter function, replacing cache_file_name with indices_cache_file_name  in the assert. Although not for the map() function @thomwolf

Maybe I'm a bit tired but I fail to see the issue here.
Since cache_file_name is None by default, if you set keep_in_memory to True, the assert should pass, no?

I failed to realise that this only applies to shuffle(). Whenever keep_in_memory is set to True, this is passed on to the select() function. However, if cache_file_name is None, it will be defined in the shuffle() function before it is passed on to select().
Thus, split() is called with keep_in_memory=True and a not None value for cache_file_name.
This is essentially fixed in #513

Oh yes ok got it thanks. Should be fixed if we are happy with #513 indeed.

My bad. This is actually not fixed in #513. Sorry about that...
The new indices_cache_file_name is set to a non-None value in the new shuffle() as well.
The buffer and caching mechanisms used in the select() function are too intricate for me to understand why the check is there at all. I've removed it in my local build and it seems to be working fine for my project, without really considering other implications of the change.

Ok I'll investigate and add a series of tests on the keep_in_memory=True settings which is under-tested atm

Hey, still seeing this issue with the latest version.

The same :(

These are the steps needed to fix this issue:

add the following check to Dataset.shuffle:

if keep_in_memory and indices_cache_file_name is not None:
    raise ValueError("Please use either `keep_in_memory` or `indices_cache_file_name` but not both.")

set indices_cache_file_name to None if keep_in_memory is True in the call to select
add a test with shuffle(keep_in_memory=True)

Hi @mariosasko , I have opened this PR #5082


[Bug]: I can not change my load order at all.

Operating System
Windows 10
BG3 Mod Manager Version
<IP_ADDRESS>
BG3 Game Version
Full release, latest steam patch.
Bug Summary
if I try to click and drag order or moving from active to inactive I get a little no symbol (the circle with line through it) and can't do anything. Anyone know what I can do to fix this.
Links
No response

Hey! Came here because I had the exact same issue as you. Here's what fixed it for me:
Changing the keyboard shortcut for moving inactive mods to the other list, in the settings of the mod manager. This thread has a bit more info:
https://github.com/LaughingLeader/BG3ModManager/issues/16
Hope this helps! :)


Use SQLite or anything else for discovery cache

Resolves L2B-2601

Looks decent, we have to battle-test that in practice. Also, it would be great to have some test suite to seal the implementation

Definitely. I'll add a story to add tests once we make sure that the approach works for us.


Remove captured viewer dependency

This dependency would cause an undefined viewer exception
when this file is not imported directly (i.e. via webpack)

Good catch. I don't really use webpack so have not noticed this.
Thanks!


Change hardcoded VERSION to use pkg.getImplementationVersion()

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

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

Codecov Report

Merging #2723 into master will decrease coverage by 0.03%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master    #2723      +/-   ##
============================================
- Coverage     80.47%   80.43%   -0.04%
+ Complexity     7101     7098       -3
============================================
  Files           145      145
  Lines         26704    26707       +3
  Branches       6373     6374       +1
============================================
- Hits          21489    21483       -6
- Misses         3381     3385       +4
- Partials       1834     1839       +5

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/com/alibaba/fastjson/JSON.java
84.74% <66.66%> (-0.12%)
147 <0> (ø)

...com/alibaba/fastjson/serializer/SerialContext.java
72.91% <0%> (-4.17%)
19% <0%> (ø)

...ava/com/alibaba/fastjson/serializer/DateCodec.java
87.58% <0%> (-1.97%)
37% <0%> (-1%)

.../java/com/alibaba/fastjson/parser/JSONScanner.java
76.24% <0%> (-0.17%)
499% <0%> (-2%)

Continue to review full report at Codecov.

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


Bump deps 4.16.0

Related to #4954.

Changelog Entry
Changed

Bumped all dependencies to the latest versions, by @compulim in PR #4972

Production dependencies

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Development dependencies

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Description

Bump dependencies after release of 4.16.0.
Specific Changes

Run npm run bump
Fix all Prettier and ESLint issues
Fix markdown-it-for-inline not loading in Webpack

[x] I have added tests and executed them locally
[x] I have updated CHANGELOG.md
[x] I have updated documentation

Review Checklist

This section is for contributors to review your work.

[x] ~Accessibility reviewed (tab order, content readability, alt text, color contrast)~
[x] ~Browser and platform compatibilities reviewed~
[x] ~CSS styles reviewed (minimal rules, no z-index)~
[x] ~Documents reviewed (docs, samples, live demo)~
[x] ~Internationalization reviewed (strings, unit formatting)~
[x] package.json and package-lock.json reviewed
[x] ~Security reviewed (no data URIs, check for nonce leak)~
[x] ~Tests reviewed (coverage, legitimacy)~

Closing because AzDo is not responding.


[AppConfig] Support for configuration snapshot

Library name
Azure.Data.AppConfiguration
Please describe the feature.
App Config customers can use SDK to use the new configuration snapshot feature.

Duplicate of #32008


2492 Fixes eyecite CI

This PR fixes the issues described in #139 and #2492

Install Poetry action upgraded to 1.3.3
The issue described in #138 related to not found dependencies fast_diff_match_patch, exrex was not related to a dependency problem but a CI cache issue. Seems the cache was broken, to solve it I changed the key removing the date-fixed string since now is possible to remove caches from the Actions tab, it would be easier to just remove old caches instead of changing the cache key.

Additionally,  I had some problems trying to install eyecite on my machine I couldn't install it on Mac with ARM or using a Ubuntu ARM virtual machine. The problem seems hyperscan, maybe an incompatibility of libhyperscan-dev on ARM.
So I had to install it on my old x86-64 Mac, the problem that I found here was using the last version of Poetry it tried to install the latest version of hyperscan 0.3.x which failed. So I had to downgrade the version to 0.2.0 in pyproject.toml   not sure if this is only a problem on mac x86-64 due to hyperscan-dev version for mac or if it's a widespread problem in other SO as well, I suspect it is, since the 1.2.0rc2 version of poetry (that was fixed in actions) installs by default the 0.2.0 version of hyperscan, so after fixed the hyperscan version I removed the poetry 1.2.0rc2 version constraint in CI actions, so now it's using the latest version of poetry.
I think it might be worth it if somebody on Linux could try to update to the latest version of hyperscan and check if the installation succeeds in case we want to use the latest version of hyperscan.

Thanks Alberto.
Two things here:

First, the eyecite report is worrisome, right @flooie? It shouldn't show a bunch of citations being added and removed, right? It also shows a solid performance improvement, which is cool, but the reason we have the eyecite report is to make sure we're not adding/removing things at weird times like this.
Any chance you could try to diagnose that, Alberto?

Second, your dependencies. You say:

I couldn't install it on Mac with ARM

Well, this is expected, and is why hyperscan is an optional dependency that's supposed to just make things faster. If you look at the hyperscan webpage (on Intel.com), you'll see that it says:

Hyperscan is a high performance regular expression matching library from Intel that runs on x86 platforms and offers support for Perl Compatible Regular Expressions (PCRE) syntax

So not working on ARM is no surprise. I am a bit surprised though that it doesn't work on x86 MacOS.
For installing it in Linux, could we do that via Github Actions? They run on Linux, right?

@mlissner - yes - very curious

@mlissner Sure, I'll check why the citations variation happened.
About the dependencies, yeah I can test it via Github actions, the problem I had was generating the poetry.lock file on my machine since it failed. So I'll try it installing an x86 Linux virtual machine and see if it works.

@mlissner that would affect speed only, would it not?

@albertisfu - I've had luck just creating a dockerfile to install and test hyper scan locally.  fwiw.

Hm, the underlying hyperscan C library hadn't upgraded since late 2020 when 5.4.0 came out, but it looks like 0.3.0 of python-hyperscan was released in April of last year and upgraded to that version of hyperscan. I wasn't able to tell which version of hyperscan was used by python-hyperscan before April, 2022 though (some digging here might reveal what the old version of hyperscan was that python-hyperscan used: https://github.com/darvid/python-hyperscan/commit/0527aac060f29413a727e7fbb8db2c456bd25aab.
One thing I saw in the hyperscan change log is:

Introduce chimera hybrid engine of Hyperscan and PCRE, to fully support PCRE syntax as well as to take advantage of the high performance nature of Hyperscan.

That sort of makes it sound like it's changing how hyperscan works, but it's from 2018-07-09, which is a long time ago. OTOH, the changelog of python-hyperscan says that version 0.3.0 (from late last year):

Feature: initial Chimera support (and upgrade to Hyperscan v5.4.0) (0527aac)

So.....maybe that's the root cause?
Maybe python-hyperscan was using a very old version of hyperscan and finally upgraded to a new version that uses Chimera, which brought more regex compatibility/support?
Maybe....

@mlissner that would affect speed only, would it not?

Well, in theory, a regex is a regex, and switching between Python's regex engine and hyperscan's wouldn't make for different results. That seemed to be the result in the past, but maybe it's not anymore. I've seen different regex engines support different parts of the standard. I even remember Firefox and Chrome had different support at one point (probably still do, ugh).

Well, the problem was not hyperscan I was able to install the latest version on docker using an amd64 image so I removed the fixed version in pyproject.toml.
After multiple tests, the problem seems to be related to reporters-db, specifically after from 3.2.34 version.
I downgrade reporters-db here to 3.2.32; you can see the report seems well.
In this other PR, I set it to 3.2.34 and you can see the report with the citations variation.
I think the problem here was the poetry.lock file was not updated in a while, the last update was on Dec 27, 2022 and reporters-db version was still 3.2.32 even though 3.2.34 was released on Nov 22.
Since the benchmark for reporters-db and eyecite PRs depend on the same workflow (that uses the same poetry.lock) benchmark reports on reporters-db PRs didn't detect the problem either.
Maybe including a poetry.lock update within the benchmark workflow might help to avoid this from happening again.
Let me know what you think.

Nice. Well, that's good that it's just our code that had the bug. Figures, I suppose!

Maybe including a poetry.lock update within the benchmark workflow might help to avoid this from happening again.

Yeah, @flooie do you want to file a bug to take a look at either doing that or just making sure that we upgrade reporters-db when running the eyecite report, so we catch these things more quickly next time? And I guess we need a separate bug for figuring out what happened to the reporter DB that you or Kevin can take care of?

For the record, I think I was wrong about everything on this issue. :)

Wasn't due to different regex engines being used across tests.
Wasn't due to the upgrade to hyperscan.
Hyperscan appears to somehow work on ARM.
Performance wasn't due to hyperscan upgrade, but due to bug in reporters-db, probably (note how the latest graph doesn't have the performance gain, but the linked PR does).

Ah well...


[FLINK-27319] Duplicated '-t' option for savepoint format and deployment target

Verifying this change
Run:
../flink-1.15.0/bin/flink savepoint --type canonical <job_id> savepoints

check it runs correctly
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented) - the short option was undocumented

LGTM


Please add also a cpp server!

Please add code for a cpp server :3

https://github.com/mrudangit/Socket.IO.Server.CPP

windows only...  :(

+1 - we require this - happy to work with whomever to make this happen.


stable/prometheus-snmp-exporter

Please help me understand what I did wrong trying to implement the stable chart. Following is what I had to change in order to get a deployed configmap.
Describe the bug
{{- if .Values.config }} is not properly detecting the values populated.
I had to modify the templates/configmap.yml disabling the if statement and the data field to:
data:
  snmp.yaml: |
{{ .Values.config | indent 4 }}

Following is an Example Config used in values.yml
config: |
  global:
    scrape_interval: 10s
    evaluation_interval: 10s
  scrape_configs:
  - job_name: 'snmp'
    metrics_path: /snmp
    static_configs:
    - targets:
      - <IP_ADDRESS>  # SNMP device - add your IPs here
    relabel_configs:
      - source_labels: [__address__]
        target_label: __param_target
      - source_labels: [__param_target]
        target_label: instance
      - target_label: __address__
        replacement: <IP_ADDRESS>:9116  # SNMP exporter.

Version of Helm and Kubernetes:
$ helm version
version.BuildInfo{Version:"v3.0.2", GitCommit:"19e47ee3283ae98139d98460de796c1be1e3975f", GitTreeState:"clean", GoVersion:"go1.13.5"}

$ k version
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.5", GitCommit:"20c265fef0741dd71a66480e35bd69f18351daea", GitTreeState:"clean", BuildDate:"2019-10-15T19:16:51Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.5", GitCommit:"20c265fef0741dd71a66480e35bd69f18351daea", GitTreeState:"clean", BuildDate:"2019-10-15T19:07:57Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"linux/amd64"}

Which chart:
stable/prometheus-snmp-exporter/templates/configmap.yml
What happened:
Error: template: prometheus-snmp-exporter/templates/configmap.yaml:14:27: executing "prometheus-snmp-exporter/templates/configmap.yaml" at <2>: wrong type for value; expected string; got map[string]interface {}
Helm failed to create Configmap.
What you expected to happen: [ After modifying files as noted above ]
NAME: snmp02
LAST DEPLOYED: Sun Mar  1 21:30:04 2020
NAMESPACE: default
STATUS: deployed
REVISION: 1
TEST SUITE: None
NOTES:
See https://github.com/prometheus/snmp_exporter/ for how to configure Prometheus and the SNMP Exporter.

How to reproduce it (as minimally and precisely as possible):
helm install snmp02 -f ./prometheus-snmp-exporter/values.yaml stable/prometheus-snmp-exporter

Anything else we need to know:

@k7faq I could be wrong, but isn't config supposed to be the configurations for the OIDs and metrics definition of what you wish to collect via snmp ?
Anyway, I'm encountering the same issue as reported. I had to make the following changes to configmap.yml to make it work.
{{- if .Values.config }}
apiVersion: v1
kind: ConfigMap
metadata:
  name: {{ template "prometheus-snmp-exporter.fullname" . }}
  namespace: {{ .Release.Namespace }}
  labels:
    app.kubernetes.io/name: {{ include "prometheus-snmp-exporter.name" . }}
    app.kubernetes.io/instance: {{ .Release.Name }}
    app.kubernetes.io/managed-by: {{ .Release.Service }}
    helm.sh/chart: {{ include "prometheus-snmp-exporter.chart" . }}
data:
  snmp.yaml: |
{{ toYaml .Values.config | default "{}" | indent 4 }}
{{- end }}

/reopen

@k7faq I could be wrong, but isn't config supposed to be the configurations for the OIDs and metrics definition of what you wish to collect via snmp ?
Anyway, I'm encountering the same issue as reported. I had to make the following changes to configmap.yml to make it work.
{{- if .Values.config }}
apiVersion: v1
kind: ConfigMap
metadata:
  name: {{ template "prometheus-snmp-exporter.fullname" . }}
  namespace: {{ .Release.Namespace }}
  labels:
    app.kubernetes.io/name: {{ include "prometheus-snmp-exporter.name" . }}
    app.kubernetes.io/instance: {{ .Release.Name }}
    app.kubernetes.io/managed-by: {{ .Release.Service }}
    helm.sh/chart: {{ include "prometheus-snmp-exporter.chart" . }}
data:
  snmp.yaml: |
{{ toYaml .Values.config | default "{}" | indent 4 }}
{{- end }}

/reopen

Hi jfcgaspar
I am curious about how to write config part in the values.yaml.
I try to enable SNMPV3 on if_mib
I fail in config part several times.
Can you show me how do you write config?
Thanks


Limit on-screen map size

Good Morning. How can I limit the size of the mapview so it does not occupy the entire screen? And is there a possibility that I add a button on it? Thank you

@rafaelterada any solution yet?


Warning : found no layout file for "HTML" for kind [...]

Hello. I tried to follow the procedure to test the example website but I get the following warning npm run dev:

WARN 2024/01/22 11:19:53 found no layout file for "HTML" for kind "page": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.

Which other warnings for "home", "taxonomy", "section" and "term" instead of "page". The result is that nothing is rendered on http://localhost:1313/ (blank page).  I also checked the video for the installation procedure.
Versions:
$ hugo version
hugo v0.92.2+extended linux/amd64 BuildDate=2023-01-31T11:11:57Z VendorInfo=ubuntu:0.92.2-1ubuntu0.1

$ go version
go version go1.21.6 linux/amd64

$ node --version
v20.11.0

hey @engboris, you need at least Hugo_Extended v0.115.1 to run this project. please follow the guide from here https://github.com/zeon-studio/hugoplate?tab=readme-ov-file#-getting-started

Thank you. I misread the version number and mistook v0.92.2 for being more recent than v0.1[...].


Fix heat network following price-sensitive flex changes

This PR fixes that heat storage was not behaving correctly following the addition of price-sensitive flex for electricity.
We (ab)use the marginal_costs attribute for heat network participants to sort them according to the user's choice. However, the first participant was assigned a cost of 0 which meant that it would never receive energy. This PR changes heat network participants to be assigned 1, 2, 3, ..., according to their order instead of 0, 1, 2, ... This ensures that the first option (normally storage) will be able to consume excess heat.
This commit also disables consumption from dispatchables, so that only excess is used by flex options.
Closes quintel/etsource#2627

Nice work! The heat storage seems to be working normally again. I do have a question: marginal_costs is an attribute that is used for flexible supply, while max_consumption_price is its corresponding attribute for flexible demand. The issue was with the first participant receiving energy, so heat being stored, or flexible demand. How does changing the marginal_costs ordering (by starting the order with at 1) affect the max_consumption_price of this flexible demand?


Add rake to gemspec

This PR adds rake to gemspec so users can use rake to run the tests.

Awesome, I had this problem in #28. Thanks for doing this! One small thing is I noticed you added "rack" instead of "rake".

@arlandism doh you are right, thank-you 🙇♀


Need more examples

We need to create more examples for the major workflows so that it's more obvious how to move between stages.

Duplicate #17


FetchEvent's constructor's second argument must be optional

This is the convention for defining new Events.  Both Chrome and Firefox implement this correctly.

I think the consensus was for EventInitDicts with required members, the eventInit argument should not be optional.

That's right. It was decided that FetchEventInit.request is required. I plan to address this change in Chrome via https://codereview.chromium.org/1435633002.

That'd make sense!

Argh, except that I cannot reopen!

(And the spec is incorrect anyways, please reopen!)

No, you misunderstood. The second argument is required, when it is a dictionary that contains required members. Which is the case for FetchEvent.

Oh you're right, I misread your original comment!
I modified Gecko accordingly.  Thanks!


No module named 'reconstruction_dataset'

Traceback (most recent call last):
File "/media/xpu107/wyb/DADF-main/fuse_main.py",line 11, in
from reconstruction_dataset import denormalize
MaduleNotFoundError: No module named 'reconstruction_dataset'
Hello, may I ask if you uploaded one less file?

Traceback (most recent call last): File "/media/xpu107/wyb/DADF-main/fuse_main.py",line 11, in from reconstruction_dataset import denormalize MaduleNotFoundError: No module named 'reconstruction_dataset'
Hello, may I ask if you uploaded one less file?

If you are interested, please add my WeChat and other contact information through my email<EMAIL_ADDRESS>


[CI] reconfigure deploy, pull request deploy previews

Summary
Finish migrating from the old organization to the new organization.

[x] Transfer the production domain from our netlify org to the carbon org
[x] Transfer the site to the carbon org - ensure we have proper access to restart builds, edit config, etc
[x] Ensure deploy previews are turned on
[x] Probably can delete the build message that is logged to the console - it doesn't work

Additional context
Continuation of https://github.com/carbon-design-system/carbon-addons-iot-react/issues/1391

Transfer is complete. Thanks again @joshblack! 🎉


Feature: Improve Dialog Component

Currently when we add a dialog to a component or page, it looks something along the lines of this.
<Dialog open={openEdit} onOpenChange={setOpenEdit}>
  <DialogContent className="p-4">
    <DialogHeader>
      <DialogTitle>Edit Your Highlight</DialogTitle>
      <DialogDescription className="font-normal">
        Make changes to your highlights here. Click save when you're done.
      </DialogDescription>
    </DialogHeader>

some dialog content

    <DialogCloseButton onClick={() => setOpenEdit(false)} />
  </DialogContent>
</Dialog>

Instead of having to import all those components all the time and in the spirit of consistency, I'm proposing a simpler component that under the hood would use the above. And for the variant we can have an icon associated to it like we do right now for e.g. a warning dialog.
interface DialogProps {
	open: boolean;
	onOpenChange: (open: boolean) => void;
	onClose: () => void;
	title: React.Node; // string or a component
	description?: React.Node; // string or a component
	variant: 'destructive' | 'warning' | 'success';
	closeButtonText?: string; // defaults to close
	cancelButton?: boolean; // defaults to false
	cancelButtonText?: string; // defaults to close
	// maybe some other props
}

<Dialog
	open={someFlag}
	onOpenChange={onOpenChange}
	onClose={onClose}
	title="some title or component"
	description="some description"
	variant="destructive">

some dialog content

</Dialog>

This is the rough idea.
Thoughts @isabensusan @open-sauced/engineering?

I think its a good idea to consolidate it down to a single component with props

Great idea! I'm all for simplifying the components and maintaining consistency. A unified component sounds like a smart move. Plus 1 on the suggestion


Add styles

❓ What type of PR is this?

[ ] Refactor ♻
[x] Feature ✨
[ ] Bug Fix 🛠
[ ] Optimization 🚀
[ ] Documentation Update 📄
[ ] CI/CD 🤖

💬 Description

This PR includes styles for the entire app including layout and design.
🃏 Added/updated tests?
We encourage you to keep the code coverage percentage at 90% and above.

[ ] Yes
[ ] Not yet
[x] No, because this is not necessary.
[ ] No, and this is why:
[ ] I need help with writing tests

🌿 What branch would you like to merge this into?
develop

📷 [optional] What gif best describes this PR or how it makes you feel?

@coderabbitai review

The same to the case of #4, this PR will become obsolete because we’re reorganizing the structure for better clarity and organization. Styles will be moved to their own folder for easier access, with those for components going into the components folder, among other changes. 📂
Even though the files here are still important, many have been moved, replaced, or refactored as part of this reorganization. 🚧 Please don’t delete the branch, as these files are still valuable. We’re just tidying things up! ✨


feat(jenkins): migrate various -master jobs and deis-v2-e2e(-master/-pr) job into DSL

Closes #134

Please see current iterations describe in latest commit can be seen in https://ci.deis.io/job/deis-v2-e2e-pr-clone/configure and https://ci.deis.io/job/deis-v2-e2e-master-clone/configure and point out anything odd/missing...

This is awesome much :+1:


fixes appirio-tech/topcoder-app/issues/1131 renameed api_url_v3 to ap…

fixed https://github.com/appirio-tech/topcoder-app/issues/1131 rename api_url name

@shubhendusaurabh can you explain why this fixes console errors?

@tladendo those 2 console errors are due to 'API_URL' being undefined in react-components repo but these are defined in topcoder-app. In my previous PR I created new var API_URL_V3 which worked in react-components but failed in topcoder-app because of React shallow merge of props.


Update language configuration

feat: word pattern
feat: path angle brackets
feat: antiquotation brackets
fix: auto close at end of string
fix: auto close double quotes in comments
fix: don't auto close single quotes
fix: folding marker comments

It looks good. Thanks @mkhl !


Updating build system for use in CommonJS

Trying to use jsep in commonjs wasn't working because it was being treated as a module.

Separate dist folders for each build. Add overriding package.json file in cjs folder to set type: 'commonjS'. (Note: there are difficulties with trying to do this using cat in an npm script with {} and some Operating Systems. Simple solution is to copy a file.
Updated ReadMe
Added tests to lint script and fixed lint errors

Sorry I dropped the ball, thanks for the ping! I approved it now, so you could merge it yourself whenever you want (I'm told this is the new pattern so I figured I'd try it)


what is the actual mean of "x3d" in the output of the epropnp_det?

I am confused about the meaning of "x3d", it seems like all the "x3d" of an object are in one plane, how can PnP calculate pose with this 3d points？
here is my test result about x3d in bev (the first picture)

Hi! x3d are the points' 3D coordinates in the object coordinate system. In the LineMOD 6DoF pose estimation model, a coordinate regression loss is imposed to force those points to lie on the exact surface of the objects. In the nuScenes 3D detection model though, there is no such loss function and the points are learned to solely minimize the pose loss, so they may not be a clear representation of the object shape.
PS: Your visualizations do seem a bit weird, and I suspect there is something wrong in the model configuration or the visualization code, e.g., the camera intrinsic could be wrong so all the boxes are somewhat tilted in the second image.

Hi! x3d are the points' 3D coordinates in the object coordinate system. In the LineMOD 6DoF pose estimation model, a coordinate regression loss is imposed to force those points to lie on the exact surface of the objects. In the nuScenes 3D detection model though, there is no such loss function and the points are learned to solely minimize the pose loss, so they may not be a clear representation of the object shape.
PS: Your visualizations do seem a bit weird, and I suspect there is something wrong in the model configuration or the visualization code, e.g., the camera intrinsic could be wrong so all the boxes are somewhat tilted in the second image.

I also noticed that in the nuScenes 3D model, the x3d points may not be a clear representation of the object shape. Is it possible to impose a similar loss func to force those points lie on or nearly on the exact surface of the objects?
Another question: it seems that in EPro-PnP-Det model, there's no label information in the inferred results...right? there are only ['bbox_results', 'bbox_3d_results', 'pose_samples', 'pose_sample_weights'] ...

I also noticed that in the nuScenes 3D model, the x3d points may not be a clear representation of the object shape. Is it possible to impose a similar loss func to force those points lie on or nearly on the exact surface of the objects?
Another question: it seems that in EPro-PnP-Det model, there's no label information in the inferred results...right? there are only ['bbox_results', 'bbox_3d_results', 'pose_samples', 'pose_sample_weights'] ...

Yes you can impose such a loss function, e.g., using the LiDAR points as supervisory signal. In EPro-PnP-Det though, we moved the LiDAR-based coordinate regression loss to an auxiliary branch (see supplementary) which is easier to implement. I'm not quite sure how the accuracy would be affected if the loss is imposed on x3d directly.

Yes you can impose such a loss function, e.g., using the LiDAR points as supervisory signal. In EPro-PnP-Det though, we moved the LiDAR-based coordinate regression loss to an auxiliary branch (see supplementary) which is easier to implement. I'm not quite sure how the accuracy would be affected if the loss is imposed on x3d directly.

Thank you for timely reply! I'll check that branch.
it seems that the official code does not output 'labels' right? I checked the inference result which only has four keys: ['bbox_results', 'bbox_3d_results', 'pose_samples', 'pose_sample_weights']

Thank you for timely reply! I'll check that branch. it seems that the official code does not output 'labels' right? I checked the inference result which only has four keys: ['bbox_results', 'bbox_3d_results', 'pose_samples', 'pose_sample_weights']

The value corresponding to each key is a list of class-wise results, where the index of each element corresponds to a specific class.

Thank you for timely reply! I'll check that branch. it seems that the official code does not output 'labels' right? I checked the inference result which only has four keys: ['bbox_results', 'bbox_3d_results', 'pose_samples', 'pose_sample_weights']

The value corresponding to each key is a list of class-wise results, where the index of each element corresponds to a specific class.

aha, I see, thank you!


ui(routes): Use some more fitting icons

Use 1 for dependencies to avoid a folder-look, use 2 for vulnerabilities to avoid a connection to smoking, and 3 for rule violations to match a bit ORT's evaluator logo 4.

Haha, I've always seen (and still do) evaluator's logo as a bird with wings spread out. :D

Haha, I've always seen (and still do) evaluator's logo as a bird with wings spread out. :D

Yeah, I guess it deliberately was a mix of both, that is, you could see in it whatever you prefer 😉


Stream Continues to Play After Winds Is Closed

Copied from https://github.com/GetStream/Winds/issues/577
Prerequisites
[ X ] Put an X between the brackets on this line if you have done all of the following:
Checked that your issue isn't already filed: https://github.com/GetStream/Winds/issues
Description
Closing window whilst episode playing, podcast continues to play. When relaunching player, podcast continues to play, but I am logged out and can't stop the podcast from playing and autoplaying all other episodes.
Steps to Reproduce
[First Step] Play a podcast
[Second Step] Close window
[and so on...] relaunch player
Expected behavior: [What you expect to happen] I would expect to be back into winds with control over what plays
Actual behavior: [What actually happens] I am logged out and can't control what is playing through speakers
Reproduces how often: [What percentage of the time does it reproduce?] every time in my experience
Versions 3.1.11

Same Problem. I see in the task manager that the "winds" process is still running.


[Feature Request] Add support for ESP32-C6

I have a esp32-c6 board and I am not capable to install on it. Is this board compatible?

No, it is not compatible at this time.
ESP32 Arduino Core 3.0.0 (which is in alfa state right now) adds support for the C6.
Nuki Hub is build using the latest stable core 2.0.14

But even then it's not automatic, it would need a build targeting the C6. I still have to tinker with the C3, I couldn't get it running so far.

Could I propose using PlatformIO? This would simplify the build process and the adoption of new boards. In this branch of my fork, I've introduced PlatformIO and configured three boards: the ESP32, the ESP32 Solo1 (Shelly 1 Plus), and the ESP32-S3 Dev Kit C1. You could check if the binary for ESP32-S3 works.
To simplify the comparison, I didn't change the directory tree structure to match the PlatformIO one.

It should be possible to support both build systems, cmake and platformio, shouldn't it?

I have removed some libraries from the lib folder and configured PlatformIO to download them. If you restore those libraries, you can build with CMake too. What advantages do you see in having both build systems?

Well, I know cmake quite well, it's powerful, and I can use clion for development.

Yeah except it doesn't really work, at least last time I checked. But really compiling this project isn't too hard. All the files are there, I somewhat like it in embedded projects to have all the files for compilation available without having to use some kind of package manager. Tried that for bigger projects (used conan for that). It works more or less, but gives you headaches often enough. Just copying the files into the project is often much less of a hassle for C/C++ projects.

I don't use CLion so I don't know how well the PlatformIO plugin works, after your post I downloaded it and just tried to build the project and it built without any problems. Actually I'm using VSCode and it works pretty well with PlatformIO. In addition to the build and package management, also consider that can be supported different boards just by adding their ID in the configuration file.

No, it is not compatible at this time.
ESP32 Arduino Core 3.0.0 (which is in alpha state right now) adds support for the C6.
Nuki Hub is build using the latest stable core 2.0.14

ESP32 Arduino Core 3.0.0 is now out of alpha (since 3 days): https://github.com/espressif/arduino-esp32/releases/tag/3.0.0

Preliminary/Experimental support for C6 is added in PR #407
This build is tested on a C6 devkit with nuki hub registered as an app.
There are still some issues though with stability (could partially be due to running as an app) and pairing.
Pairing can only be done by copying the pairing information of a different nuki hub device that is already paired.

ESP32-C6 should be usable once #436 is merged, including pairing.


Bump ws?

Can we bump "ws": "^0.4.32" to v0.5?
So that npm won't complain
$ npm ls
npm WARN unmet dependency /var/www/videomail/staging/source/node_modules/websocket-stream requires<EMAIL_ADDRESS>but will load
npm WARN unmet dependency /var/www/videomail/staging/source/node_modules/ws,
npm WARN unmet dependency which is version 0.5.0

Grazie mille!


Error when having multiple map instances

Issue description
An issue might occur where having multiple GMap loading at the same time, in multiple instances.
Steps to reproduce and a minimal demo of the problem
Sample code to replicate the issue:
https://stackblitz.com/edit/angular-google-maps-multiple-maps-issue?file=app%2Fapp.component.ts
What steps should we try in your demo to see the problem?
Load the demo on stackblitz.com and wait until you see the map.
Current behavior
It shows only one map, and the console prints an error 'preview-cce22204b27714343b50.js:1 You have included the Google Maps API multiple times on this page. This may cause unexpected errors.'.
Expected/desired behavior
It should show two maps without any error in the console.
angular2 & angular-google-maps version
1.0.0-beta.2
Other information

This will be fixed in the next version


Quark minecart dupe

Place an empty minecart, take a chest or hopper, click on the minecart with a hopper and click the hopper on the minecart a second time with the shift key held down
This works with all minecarts (for example, it also works with a chest)
https://youtu.be/Tmnfj01HSV4
Fix: B:"Right click minecarts to add blocks to them"=false in quark.cfg

If it can be fixed changing the config file, I think I can't do anything better to fix it.


Disable Hadoop MiniCluster data replication

Also disable MR cluster startup - it's not necessary for HDFS test

lgtm


Undefined variable: current

Expected behavior

No notices.
Actual behavior

Couple notices of undefined variables.
Notice: Undefined variable: current in /home/nikola/public_html/wphelium/wp-content/plugins/timber-library/lib/Timber.php on line 464
Notice: Undefined variable: current in /home/nikola/public_html/wphelium/wp-content/plugins/timber-library/lib/Timber.php on line 467
Notice: Undefined variable: current in /home/nikola/public_html/wphelium/wp-content/plugins/timber-library/lib/Timber.php on line 464
Notice: Undefined variable: current in /home/nikola/public_html/wphelium/wp-content/plugins/timber-library/lib/Timber.php on line 467

Steps to reproduce behavior

I got this error while trying out "Helium" template from RocketTheme.
Can be downloaded here.
What version of WordPress, PHP and Timber are you using?

Wordpress 4.5.3, Apache/2.4.20 (Unix) OpenSSL/1.0.2h PHP/5.6.23, Timber 1.1.0
How did you install Timber? (for example, from GitHub, Composer/Packagist, WP.org?)

Fresh install from WordPress, "Add new plugin".

Closed by #1105


Project: SFMFB Household Participation Model

Project name
Household Participation Model
Description
What is the project about? - To create a reproducible model which provides the user with an interactive output of drivers of participant attendance at food distributions.
What type of app have you built? - Power BI report with data/model produced by Fabric notebook
What is it doing? - A logistic regression model is created with various participant attributes and the coefficients from this model are then written to a delta table which is loaded into Power BI. Using Power BI users are able to view the odds and associated standard errors of the different categorical and numerical predictors in relation to likelihood of attendance.
What problem is it solving? -  Using inferential modeling to uncover which factors make a participant more or less likely to attend a food distribution. These drivers of attendance can then be communicated to business owners and recommendations for updating programming can be made.
Project Repository URL
https://github.com/Data-SFMFB/SFMFB-Fabric-Hackathon-Submission
Project video
https://github.com/Data-SFMFB/SFMFB-Fabric-Hackathon-Submission/blob/main/HackathonRecording.mp4
Team members
seshghip, mtworth

PS -- your submission website has two submission dates!


Streak Reset for no reason

Describe the bug
I'm contributing and coding everyday, but today it reset all of a sudden for no reason.
Expected behavior
It was supposed to continue
Screenshots
My contribution graph:

My stats card:

Desktop (please complete the following information):

OS: Linux (Ubuntu)
Browser: Firefox
Version: latest

Additional context
Add any other context about the problem here.

Got it fixed by re-adding, Thank you, and sorry!!

Hi @janaSunrise
It looks like this may have been caused by a commit that you have on October 16, 2021.

October 16 is the last item in the generated list so it is treated as the current date and it becomes the current streak.
I think it might make more sense to simply ignore commits after the current date so the current streak makes more sense.

Oh, weird! I didnt notice that, I'll try to figure that and fix it! Thanks for your immediate help, Appreciate it! :D

@janaSunrise I only now noticed that you mentioned that you wanted to fix it. Sorry if I took away the opportunity by making the change to the code.
With the new code, your card should display like this with the end date being today's date (UTC)

It may take some time for the cached version to get refreshed by GitHub.
Let me know if you have any ideas on ways it can be improved further! :smiley:

Ah great 😁 I'll find another issue to work on later then! For now everything's all good! Thanks for the great response!

I lost my streak. No reason at all. How do you fix it? @janaSunrise


Created java-merge_sort-O(nlogn).java

Added code for merge sort algorithm implementation in java language.
Issue No. #93

Hey what about the another pull request i have made today #137
On Wed, Oct 16, 2019, 22:46 subhahu<EMAIL_ADDRESS>wrote:

Merged #125
https://github.com/Yet-Another-Series/Yet_Another_Algorithms_Repository/pull/125
into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Yet-Another-Series/Yet_Another_Algorithms_Repository/pull/125?email_source=notifications&email_token=AKQ4LR3GOJYXGIJUWZYYZJTQO5D6NA5CNFSM4JBDWE3KYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUIEEERI#event-2718450245,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKQ4LRYDTBKWVN2NZBTPGYLQO5D6NANCNFSM4JBDWE3A
.


Implement CsrfLayerBuilder to allow setting the cookie key

I noticed that my authenticity tokens weren't working across server restarts because there's a new encryption key generated each time the CsrfLayer gets built. To mitigate that, I've added an option to set the key during the build. That way, I can use the same key I already have for other encrypted cookies.
I considered making the CsrfLayer fields real-public (instead of just crate-visible), but having the CsrfLayerBuilder felt like a less invasive change.
I'm still a novice in both Rust and Axum, so I'll be grateful for any feedback you have :)

I agree that making them public could be invasive and I only did it to prevent people from having to deal with Axums Extension wrapper directly. We could probably also make the Key publicly accessible so people can use it directly thru the lib too to generate a new key. That can always be for another push. Anyways it looks good just needs some Examples added to the Readme and to add some documentation Comments on what each function is for or does.  you can add it like /// CSRF layer struct used to pass key and CsrfConfig around.

I have merged and republished this into 0.4.0 on crates.io


[UI] Full Dark Mode

Hey ^_^
I'm using your Moving theme and loved it, so i decided to make a Dark version of it to reduce eye restraint!
I also have several other fixes and improvements/Add-ons that i'm using, and will create PRs for them seperately.
If interested, you can see it in action here: https://vel-san.github.io/
Thanks for your work!

If interested, you can see it in action here: https://vel-san.github.io/

Wow, cool, I love this theme @Vel-San ! Can you update a theme trigger in the _config.yml file? Current changes will overwrite the light theme, so I cannot merge it.
You can add an attribute like dark-mode: true in the under the moving label, and load the different css according to that, if you have any issues in this feature, I'm glad to help!

@huangyz0918 Thanks!
Yes of course, personally i wanted to make a button that alternates between dark/light mode. I didn't know about the trigger, i'll do it :) !

I didn't know about the trigger, i'll do it :) !

Since the Jekyll is a static blog framework, a dynamic button is inappropriate here. Actually it is a really simple feature, you can refer to the Liquid Language, that builds Jekyll theme.

@huangyz0918 Do you know if there's a way to override colors in scss based on the trigger BOOL in the _config ? Or can i override colors in the html ? will the classes take the new overridden variable? if not, i think i'll be duplicating each class into light and dark .. I feel this isn't the proper solution.

Note: I still need to fix the difference between Dates and Post titles ( Grey and White / Grey and Black )
I'll release a fix soon :)

Done! I've fixed all mismatching colors. Previosly everything was set on text-color, i've changed it for better readability and eye-friendlly between Black/Grey/White.
Fixes applied on both Light and Dark modes. I also published them on my own version here as well
https://vel-san.github.io/


Improvements when adding to list

Is your feature request related to a problem? Please describe.
It could have an option in the behavior settings, to change the add and remove option, so that each time it is clicked, it asks which list, and it would be of great help, if we can change the default list, in case we want to add several from the discover tab in a Completed list
At first I was quite confused, where would I be saving the list, but then I noticed that it was saving in
https://www.themoviedb.org/u/aleh_hyun/watchlist
because the name is just
Title: Lista
Description: Lista padrão
It could be
Title: Lista de interesses
Describe the solution you'd like

When clicking add to list, something like this appears 👇

and also be able to choose a default list in behavior

Describe alternatives you've considered
I currently use:
https://mydramalist.com/dramalist/9724039
But I missed that when I add it, I can choose the list, or be able to change the default list.
Additional context
https://www.themoviedb.org/u/aleh_hyun/lists

Thank you for the feedback!
I've redesigned some elements of the detailed page of each title to bring some easy of access to the lists, I'll look into adding a behavior change to allow users to modify the default list too and a toggle to auto open the custom list selector when a new item is added.
Sorry for the late reply, for some reason GitHub didn't notify me.
Some of the changes are already in the latest commit, such as the List button to allow it to add the item to a list in an easier way.

As for synchronization with other services, I am currently thinking of implementing an improvement in the integration with TMDb, making it more similar to a standard custom list of the application, being easily accessible through the same screens. I was also interested in The MyDramaList, because it seems to be a great service to integrate with Cronica. I will send an email requesting access to the private beta of their API.
Sobre a tradução para o português, eu sempre procurei uma boa palavra para Watchlist, mas o máximo que cheguei foi "Lista" mesmo. No entanto, gostei muito do termo "lista de interesses" e vou adotá-lo. Obrigado!

An option to auto-open the list selector when adding a title is now available on the App Store version and TestFlight version.
The source code is also updated with this change.

I don't know if you'll get the mydramalist api, it looks like they disabled it, but I'm making a copy of what I have there on TMDB, and adding or updating on TMDB, which you don't have to have in the app

I found a bug, by leaving the selector enabled, and clicking on finished without selecting a list, it appears added, but does not appear in the standard list

I noticed that now those marked as watched do not appear on the list, I liked it, but something I noticed before and still continues, I don't know if it's a bug, or it was thought like that, the titles of another list appear in the standard list

It would be interesting something for us to be able to mark several, to take advantage of the new features, if you consider it important of course 😁

I noticed that now those marked as watched do not appear on the list, I liked it, but something I noticed before and still continues, I don't know if it's a bug, or it was thought like that, the titles of another list appear in the standard list

Nas listas customizadas o app mostra todos os items, vou deixar os filtros sincronizados entre a lista padrão e as customizadas. Pensando agora, separar os filtros entre lista padrão e customizada pode ser uma UX bem confusa. 😅

E se tiver lista pré definidas? que seja possível apagar em caso de o usuário não quiser usar a do app, e usar a sua que já tem no TMDB, no print abaixo coloquei do site mydramalist, que já vem essas lista por padrão
Lista de interesse (padrão)
Ao clicar em adicionar vai para lá, se tiver marcado para abrir seletor, só vai ser adicionado ao escolher uma lista, podendo ser a lista de interesse ou as outras

É para não ficar confuso poderia adicionar mais um filtro
Filtrar por lista:
Tudo (que seria como está atualmente)
Lista de interesse ou as outras listas

Eu gostei muito da ideia das listas pré definidas e da ideia de filtrar por listas.
Na versão beta eu tentei melhorar um pouco mais das listas do TMDb, uma das melhorias foi integrar elas diretamente no seletor de lista e na UI pra adicionar e remover, as listas do TMDb não serão locais (até então o Cronica fazia uma cópia delas pro modelo de lista que ele utliza).
UI de adicionar um item em uma lista:

UI do seletor de lista:


HiDpi renders incorrectly on some high resolution screens

The editor buttons work fine on my computer's lower resolution monitor (loads icons.png) and also work fine on my ipad and iphone (loads icons_hidpi.png) however the icons all load incorrectly on my MacBook Pro's Retina display. See screenshot:

It looks to me that the HiDPi icons are being loaded but they are being positioned using the regular resolution specifications.

I wonder how hard it would be to replace the bitmap graphics with FontAwesome or SVG icons making the whole HiDPI issue moot...


update table list in workspace detail

add styling and a display name to table links

This return statement no longer exists.


This is a test

This is a text.
this is a text in bold
this is a text in italic
www.facebook.com

a

🐝

see

xD

e

[ ] do this

[ ] and also this

no headline
i will cut u off

what is this?

maybe this

what is this again?
Don't know what it is.

Look ma i'm on teevee.


Improve Job management via "Full duplex" Queue

FOR BRAINSTORMING
Target:

provide more information to end-user about job status
add possibility to cancel job

How to achieve:
JobStatus contains:

id
state (unknown, waiting, running, done, cancelled, failed....)
process indicator (sequential number)
process indicator [0.0 - 100.0]
fail reason
cancel initiator (user, ...)
additional info (e.g. PID for cancel, timestamps, display info , ...)

JobStatus is Message
Job Handler periodically (within processing loop) pushes JobStatus via Queue to Producer.
JobCancel is Message
Monitoring app. pushes JobCancel to Queue, on the other side QueueLoop (implements LoopInterface) receives it ...
Function "run Job" moved from Adapter to Queue.
Flow is generic, does not depend on specific functionality of Adapter.

@g41797 I just read I didn't get the reason to have so flexible status checking as you wrote. I have some different goals what to do now before the package release and seems your idea doesn't suites the goals.
Also seems quite hard to implement progress/cancellation with additional jobs. Just imagine 100 RPS makes 100 * (amount of progress check messages) tasks. I'm not ready to discuss it now.

That's interesting since some queues do not provide any way to peek inside. But, as @xepozz noted, likely that's extra workload that will slow down queue processing.

First of all we need to understand whether

end user wants this functionality (web page with updated status of running jobs + possibility to cancel...)
it already supported by your famous friends Laravel and Symfony

My bet - answers are yes&yes
Do you have possibility to ask Yii users about this use case?
Regarding slow down processing - it's usual answer of busy developer
I did it a lot in my career

My bet - answers are yes&yes

Of course, that's nice to have. But it could be implemented in multiple ways:

By introducing interfaces for it and each "driver" implementing it. In some queue engines that is supported by the engine itself so no extra workload will be needed. In some engines it's not supported so a "driver" will have to write extra data somewhere (see 2).
By always writing extra data i.e. keeping our own jobs registry along with statuses and IDs regardless of the "driver".

Web page for submitting jobs

No, definitely not. Jobs are submitted by code, not manually.

Web page for monitoring

Yes, could be part of https://github.com/yiisoft/yii-dev-panel

Background workers invisible for user

Yes.

"Who allowed to monitor jobs?" any user, admin etc

The one having access to dev panel. End user usually doesn't need to know about that but if we have an API for getting a job status, one might use it to inform user.

Or how many simultaneous monitoring apps/pages should be supported

Endless number.

Possible variants for storing of job statuses pushed by workers to queue:

user application receives JobStatuses via Queue , saves in user DB, panel periodically polling DB
Queue saves statuses in internal DB, panel periodically polling DB via queue api
Queue saves statuses in broker(it acts as "DB") - panel receives statuses via queue api

I hate DB, but my choice - 2. BTW using db or broker should be part of adapter (using db as fallback if specific adapter does not support persistence)
Your turn

@viktorprogger, @s1lver thoughts?

adding insult to injury
In order to acts as "jobs registry" broker should support Event Sourcing
Usually it means that

broker should support Streams
PHP client library also should support Streams API

e.g.

RabbitMQ supports Streams
php-amqplib supports Streams, but not Streams API

as result - [100-1000???]RPS (clients of StreamAPI - 1000000RPS)

no thoughts no glads
issued #194 instead - for go i can start it today

https://github.com/yiisoft/yii2-queue/pull/510


More information in file tooltips

Godot version:
3.1.1
Issue description:
Sometimes you want to know the size of a texture. Right now the only way to check that is either double-click the file and see in the inspector, which is not always desirable. On Windows, when you hover over image, you get a tooltip with it's size:

I once thought that it would be useful in Godot, so here's an issue about this feature.
FilesystemDoc tooltips only contain a filename now. It would be more useful if they contained e.g. file size, texture size for texture resources or full paths to files. Something to make them meaningful.

There's a similar proposal already and I commented there: https://github.com/godotengine/godot-proposals/issues/175#issuecomment-632397331
Closing.


Doesn't allow multiple entry points output [name]

v1: https://webpack.github.io/docs/multiple-entry-points.html
v2: https://webpack.js.org/concepts/output/#multiple-entry-points
Example:
const path = require('path');
const ZipPlugin = require('zip-webpack-plugin');

module.exports = {
    entry: {
        'schedule': './src/schedule.js',
        'import-export': './src/import-export.js'
    },
    output: {
        filename: '[name].js',
        path: path.resolve(__dirname, 'dist')
    },
    plugins: [
        new ZipPlugin({
            filename:'[name].zip'
        })
    ]
};

I'm not sure exactly what your issue is, but I assume that you want files emitted by each entry point to be in separate zip files.
This isn't done partly because (as far as I can tell) Webpack does not provide a straightforward way to detect which files were emitted by each entry point.
e.g. if x.js and y.js are entry points...
x.js
import 'file-loader!./x.png';

y.js
import 'file-loader!./y.png';

...then x.png and y.png just show up in the list of assets without anything (that I can find) that maps them to the chunk/entrypoint that loaded them.

That is one use case I didn't think of, knowing which additional static assets are required would be tricky to know.
What I was imagining was for bundling aws lambda function. I imagined the zip plugin would zip the output result of each entry using the multi entrypoint [name] syntax.
Using my example above would result in ./dist/import-export.zip and ./dist/schedule.zip

Ran into the same problem. Just loop through the entries yourself. Hope this helps others:
const configBase = {
    entry: {
        one: './src/one.js',
        two: './src/two.js',
    },
    output: {
        filename: '[name]/bundle.js',
        path: path.resolve(__dirname, 'dist/')
    }
};

const configPlugins = {
    plugins: Object.keys(baseConfig.entry).map((entryName) => {
        return new ZipPlugin({
            path: path.resolve(__dirname, 'dist/'),
            filename: entryName,
            extension: 'zip'
        })
    })
}

const someConfigName = Object.assign(configBase, configPlugins);

module.exports = someConfigName;

To add on to the previous response you might want to use something like this:
const configPlugins = {
    plugins: Object.keys(configBase.entry).map((entryName) => {
        return new ZipPlugin({
            path: path.resolve(__dirname, 'dist/'),
            include: [`${entryName}.js`],
            filename: entryName,
        })
    })
};

Which just zips each individual file


ci: use codecov GH action

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.4 out of 5 committers have signed the CLA.:white_check_mark: philippfromme:white_check_mark: smbea:white_check_mark: MaxTru:white_check_mark: barmac:x: github-actionsgithub-actions seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Please ignore the comments above. I removed the develop commits from the branch.


remove laundryHallId in reducer

laundryHallID is originally passed to LaundryVenue from both props and reducer. But now, it is only passed from props

Make sure that lint passes


Moving config option tests to a dedicated subdirectory

Resolves #1492.

Moved tests previously named tests/[source|target]/configs-* to tests/[source|target]/configs/CONFIG_NAME/TEST_NAME.rs.
Added test (verify_config_test_names) that enforces the naming of the config option tests. For tests/[source|target]/configs/CONFIG_NAME/TEST_NAME.rs., TEST_NAME.rs must contain "// rustfmt-CONFIG_NAME".

The number of tests being run is unchanged.
System and idempotent tests before:
Ran 235 system tests.
test system_tests ... ok
Ran 284 idempotent tests.
test idempotence_tests ... ok

System and idempotent tests after:
Ran 235 system tests.
test system_tests ... ok
Ran 284 idempotent tests.
test idempotence_tests ... ok

Thank you so much for doing this!


reduce the number of I/O operations for HNSW during deserialization

I think we don't need to modify so much, just add a fixed size readbuffer allocated by allocator, if read out of the buffer, call the read func to refresh the buffer, otherwise copy data from the buf is ok


Make the kernelspecs fetch more fault tolerant

cc @afshin

👍
Thanks!

This is not directly impacting jupyterlab now, but just to let you know, we've released versions of nb_conda, nb_conda_kernels and nb_anacondacloud through the official Anaconda conda repos that should resolve the issues you saw with Python[root].


Column grouping layout hints are ignored on rollup tables

A customer noticed that column groupings from layout hints are respected on regular tables, but ignored on rollup tables. (The columns can be grouped manually afterward and it works; the issue is just the layout hints being ignored.)
DH-17076

Tried to translate the snippet to Core python:
from deephaven import empty_table, agg
source = empty_table(100).update(["K1 = ii", "K2=K1%10", "Y=randomDouble(0, 10)", "Z=randomDouble(0, 10)"])
result_norollup = source.layout_hints(column_groups=[{"name": "MyCols", "children": ["Y", "Z"]}])
result_rollup = source.rollup([agg.avg(["Y", "Z"])], "K2", False).layout_hints(column_groups=[{"name": "MyCols", "children": ["Y", "Z"]}])

However get an error because layout_hints is not a method on RollupTable. In Legacy, it appears HierarchicalTable extends QueryTable which extends BaseTable which has the layoutHints method, but in Core that's not the case.
So I don't think this can actually be reproduced in Core...


Way to have cache be the primary consumer and client secondary

Periodically we get cache misses through our hierarchy where the client has terribly slow throughput. In practice, this means that the transfers through the hierarchy are limited by that initial client's throughput -- we only buffer enough to send to the client. If a later, faster client asks for the same object, it will also get tied to the original slow client throughput.
It'd be nice to have a way for the cache to be the primary consumer -- assume that the cache nodes in the hierarchy are the fastest -- all requests would then effectively become read-while-writer. In the first slow client scenario, the node closest to the origin's cache would get written to as fast as it can, and so forth down through the hierarchy layers. The one closest to the client might be tied to the slow client ... but, a second, faster client requesting the same object from a different node could be served quickly as the outer node fills from the other nodes.

This has been fixed with the cache fill plugin by @vmamidi

See #7470


ActivatorUtilities fails if first constructor is unresolvable

I want to report a bug for the static method CreateInstance(IServiceProvider provider, Type instanceType, params object[] parameters) on ActivatorUtilities.
When trying to resolve a service that has unresolvable argument(s) in its first declared constructor, an InvalidOperationException is thrown (Unable to resolve service for type 'IDependency' while attempting to activate 'Service'.)
To reproduce, create the following classes
public class Service
{
    public Service(IUnknownDependency unknown) { }
    public Service(IKnownDependency known) { }
}

public interface IUnknownDependency { }

public interface IKnownDependency { }

public class KnownDependency : IKnownDependency { }

Create a service collection and register the IKnownDependency  and Service, build the service provider and try to resolve Service
var provider = new ServiceCollection()
	.AddTransient<IKnownDependency, KnownDependency>()
	.AddSingleton<Service>()
	.BuildServiceProvider();
var service = ActivatorUtilities.CreateInstance(provider, typeof(Service)); // throws!

However, if the order of the constructors are changed in Service, the call to CreateInstance returns the service.

This is essentially a dup of https://github.com/aspnet/DependencyInjection/issues/197. We might address this in 1.2.0 by adding an additional flag to the method that says use a slower best match algorithm.

You are right, it looks like the same problem 👍 . However, in #197, the problem discussed is how to identify the "best" constructor, in this scenario it is more about finding an applicable constructor/avoiding an exception to be thrown. But OK - just wanted to bring this to your attention.

The problem is the same though. The service container is opaque to ActivatorUtilities. Consequently we have no way of telling if IUnknownDependency exists in the container (short of calling serviceProvider.GetService<IUnknownDependency>()). Our design choice here was to run with the first ctor available. #197 proposes fixing this by possibly calling GetService until a best matching ctor is found. In your case, it would be the 2nd one.
That said, if you DIed in the type, rather than use a service location (which is what ActivatorUtilities does), you shouldn't see this. The default service provider (the one you get from BuildServiceProvider) supports finding the best matching ctor. Something to consider in case you can refactor your code.

Understood 👌 .
I'm writing a middleware inspired library where users can register their middleware
pipe.Use<QueueDeclareMiddleware>(new QueueDeclareOptions())

Where QueueDeclareOptions is not registered in the DI, but uniquely provided for each use.  QueueDeclareMiddleware in turn has constructor parameters that are resolved from DI. The IServiceProvider interface does not have a method where I can provide additional arguments, which is why I use the ActivatorUtilities to start with. Do you have any recommendations for this particular scenario?

cc @Tratcher - wouldn't this work out of the box if your ctor accepts an IOptions<QueueDeclareOptions>? The middleware pipeline should do the constructor injection for you while injecting the option as IOptions<T>. Maybe I haven't followed your scenario clearly.

Perhaps - I haven't looked at the options framework since RC2. The way I understand the usage of IOptions<T> is to provide a way to apply multiple Action<T> to the same options object. Is this correct?
Let me explain my scenario a bit more, and tell me if you think the Options framework can help me. A typical "middleware chain" could look something like this
pipe
	.Use<StageMarkerMiddleware>(StageMarkerOptions.For(PublishStage.Initiated))
	.Use<PublishConfigurationMiddleware>()
	.Use<StageMarkerMiddleware>(StageMarkerOptions.For(PublishStage.ExchangeConfigured))
	.Use<ExchangeDeclareMiddleware>()
	.Use<StageMarkerMiddleware>(StageMarkerOptions.For(PublishStage.ExchangeDeclared))
	.Use<RoutingKeyMiddleware>()
	.Use<StageMarkerMiddleware>(StageMarkerOptions.For(PublishStage.RoutingKeyCreated));

As you can see, the StageMarkerMiddleware is declared multiple times, each with its own StageMarkerOption. The arguments for the call to Use<TMiddleware>(params objects[] args)  are stored away in an object to allow me to instantiate the middleware when the pipe is actually built.
public class MiddlewareInfo
{
	public Type Type { get; set; }
	public object[] ConstructorArgs { get; set; }
}

The middleware may have other constructor arguments that are provided by the DI. I was hoping to find someway to provide additional parameters to the IServiceProvider like this
var service = serviceProvider.GetService(info.Type, info.ConstructorArgs);

But I didn't find any way to do so. Looking at the test for the Options framework i get the feeling that it applies the action to the same TOptions object, and when the service provider is built, it will return a single options object which has passed through all the "configurations" of it. Correct?
Looking at the Options example from the documentation it also looks like I need to wrap my Options class in a IOptions<T> wrapper. To me it feels a bit like it is violating the separation of concern, as this is something that is specific to the DI framework. Talking about DI frameworks, I try to be agnostic in the core part of the lib, so I'm a bit hesitant to include something that is service collection / service provider specific.
Thanks for your help!

The IOptions approach wouldn't work here - it passes around a singleton instance around, so you wouldn't get different instances of StageMarkerOptions at different stages with it. @Tratcher \ @davidfowl might have other suggestions for you.

It should still work for this specific case, the options instance passed into UseMiddleware should take precedence over resolving an options instance from DI.

@Tratcher Ok sounds good, but how would I register/use the specific argument passed to UseMiddleware to achieve that? It doesn't make sense for me to register it in the service collection, as the instances provided are "scoped" for that particular instance of the middleware. It would be very helpful with a few lines of code explaining

What I would need to with with the arguments passed into `UseMiddleware(params object []args) to register (or not register, not sure I follow what you mean) the instance in the service collection
What I would need to do to be able to resolve the middleware with the provided arguments from the service provider.

(To solve this problem in Autofac, I would probably pass the arguments to the resolver.)

Should just work, have you tried it yet?. The args you pass into UseMiddleware are used along side the service container to activate the middleware, they are not added to the container. https://github.com/aspnet/HttpAbstractions/blob/dev/src/Microsoft.AspNetCore.Http.Abstractions/Extensions/UseMiddlewareExtensions.cs#L72-L75
I don't think this is possible with multiple instances of the type, at least not with our container.

Maybe I was a bit unclear earlier. I'm not referring to the Use<TMiddleware> of IApplicationBuilder, but to a pipe builder that I've developed (heavily inspired by the application builder).
Looking at the UseMiddlewareExtensions you mentioned, it also uses the ActivatorUtilities, which was the original topic of this issue. @pranavkm then advised me against the ActivatorUtilities in order to find a suiting constructor.

if you DIed in the type, rather than use a service location (which is what ActivatorUtilities does), you shouldn't see this.

I think that this comment describes my problem pretty well (I've updated it for clarity).
If I understand you correctly, the default service collection / service provider does not have support for resolving a service based on registered services in combination with additional provided constructor arguments. This can only be achieved by using ActivatorUtilities, that in turn tries to instantiate it with the first constructor, which in turn may throw (as per this ticket).

Closing as dup of https://github.com/aspnet/DependencyInjection/issues/197.


latex-admonitions causes error

docs wont compile with the latex admonitions extension using the latest texlive, which i believe is caused by this and should be resolved shortly

I dont think this was actually related to pandoc-latex-admonitions, but you still cant use admonitions in the default template, will need to keep debugging a bit to find out exactly why.
Going to leave it as a default dependency because its still useful for, e.g. addressing point-by-point comments in a revision rebuttal


redirect output log

Hi!
Is it possibile to redirect the output of actor logging to stderr or to a file?
Something like log.SetOutput(iow).
The problem: using a tui (i.e.: https://github.com/jroimartin/gocui) I cannot redirect or capture the output of the actor log.
EDIT: forgot to mention that I'd prefer a solution using the library instead of redirect the output to another file descriptor (i.e. ./run >& log.txt)

I need this too.

you can make a pr for it.
https://github.com/AsynkronIT/protoactor-go/blob/dev/log/string_encoder.go#L23

I also need this,Logs cannot be integrated into my business.
I suggest that can set  a logger ,this logger has "debug,info,error..." interface

Is there any movement on this? Why not use something like https://github.com/uber-go/zap? That would fall in line with the Build on existing technologies.


web-renderer HTML

As per issue: https://github.com/flutter/flutter/issues/80524
And migration guide https://flutter.dev/docs/release/breaking-changes/platform-views-using-html-slots-web
Are you planning to migrate plugin to use it ? Its already in BETA

I don't think anything in the plugin needs to be migrated to support the new rendering method... Are there any issues you are running into regarding these changes?


There is a vulnerability in async-http-client 1.8.3,upgrade recommended

https://github.com/pinpoint-apm/pinpoint/blob/4732e4e62abfdb1165cc05d175fdf9f4a00b3342/plugins-it/ning-asyncclient-it/pom.xml#L40
CVE-2017-14063 CVE-2013-7398 CVE-2013-7397
Recommended upgrade version：1.8.17

Thanks for the reporting.
This module exists for plugin compilation and is not included in the pinpiont binary,
so there is no risk.


Remove itol/imaxiter

When using the stacked time with unanticipated shocks, the settings itol and imaxiter are not exposed from the simulate command and they are not documented. I am proposing to remove them and let the user decide what should these settings be for all the periods using the normal tol and maxiter settings.

Codecov Report

Merging #48 (c03180e) into dev (12d7a34) will decrease coverage by 0.13%.
The diff coverage is 50.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##              dev      #48      +/-   ##
==========================================
- Coverage   87.98%   87.85%   -0.13%
==========================================
  Files          26       26
  Lines        2080     2075       -5
==========================================
- Hits         1830     1823       -7
- Misses        250      252       +2

Impacted Files
Coverage Δ

src/stackedtime/simulate.jl
88.00% <50.00%> (-0.30%)
:arrow_down:

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


plugin: find shield values

Currently jobs attempts to replace your health bar with something else.  It'd be cool if that also knew what shields were on you.  If somebody wants to figure out where in memory this is so that the plugin can report this back to javascript in the player details, that would be really neat.
See cheat engine instructions: https://github.com/quisquous/cactbot/blob/master/docs/MemorySignatures.md

At kEntityStructureOffsetCharacterDetails+0x5c is a uint32 representing the percentage of max hp shielded, i.e. 16 for 16%.
I haven't found an absolute value but with actual HP values right above it can be calculated if needed.

Oops, didn't see @opmega already found the address. I can confirm the same offset from my own findings, and it works for anything I tested it on (even when the game itself doesn't show the absorb amount). It does appear to be relative to maximum HP, and updates automatically as your max HP changes.
One thing I did notice was that it caps out at 200, no matter how big you make the shield. So I'm not sure if it's a uint32, we might as well treat it as a uint8 to be on the safe side.

It also appears to be rounding unfavorably in some situations. For example, Manaward, which reads "absorbs 30% of max HP", only shows up as 29 in the client.

Very valid points @haasn , if it caps out at 200 it might even be a ushort, but I will make it read as uint16 for now to be safe and update the PR.
As for the rounding error, I don't see a way around it if there is no other, more accurate shield value stored somewhere. I guess the value at 0x5c is only used to determine the party shield bar length and therefore it doesn't need to be any more precise for the game itself.

Yeah, I agree that this value is just the party bar length and the real value is just kept on the server.  It's still better than nothing, even if it's a little inaccurate.
I changed the title on this issue to be more about updating the jobs module to show this in the hp bar now that the plugin issues got sorted by @opmega.

This got added in cfc71d724.


HIVE-27547 : Backport of HIVE-22219 to branch-3

JIRA link : https://issues.apache.org/jira/browse/HIVE-27547

@vihangk1 @sankarh Can you please review and merge this


Add some documentation about cookie handling.

Motivation
I'm not sure this is the best place for the note, but I've not be able to find any documentation—on Chromium, on Android's CookieManager class, or elsewhere—that the SameSite cookies are not synced. I expect some further documentation on limitations of syncing cookies would be valuable, but I don't have the expertise to flesh that out at the moment.
Test Plan
Documentation.
Release Notes
[DOCS][MINOR][WebView] - Note about cookie syncing in WebView.

While I expect to have to massage this PR to be approved—I don't think that the failure is related to my change?


Fix reverse shape inference in LayerNorm

Fix https://github.com/apache/incubator-mxnet/issues/17654. Now, the user will see an error message if the in_channels does not match with the corresponding dimension in the input
After the PR, the following will raise an error
import mxnet as mx
from mxnet.gluon import nn
net = nn.LayerNorm(in_channels=10)
net.initialize()
net.hybridize()
out = net(mx.nd.ones((2, 11)))  # Trigger the error

Error:
MXNetError: MXNetError: Error in operator layernorm0_layernorm0: Shape inconsistent, Provided = [10], inferred shape=[11]

@ZheyuYe


Correction for raw affiliation Aeres Applied University Wageningen, Wageningen, The Netherlands

Correction needed for raw affiliation Aeres Applied University Wageningen, Wageningen, The Netherlands
raw_affiliation_name: Aeres Applied University Wageningen, Wageningen, The Netherlands
new_rors: 03jnx2c74
previous_rors:
works_examples: W4386004792
contact: 4636de9b5c31d785d5113cc65268e355:056e4cb80b5d06a7e960 @ hotmail.com

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


Updated REAMDE: npx nuxi module add nuxt-lucide-icons

Can you explain why this would be better? Do you have a source? All these other modules don't do this either and the current command works just fine.


Where is SIoU

Hello, I would like to ask you where the part of SIoU involved in the paper is in the code. I found IoU and gIoU in the code, but I didn't see much about SIoU

In /lib/train/actors/artrack_seq.py->SIoU_loss


increase default VM RAM requirement to match AWS infrastructure requirements

Proposing this after discussion with @mcornea and after debugging issues caused by OOM on a host with 32GB RAM. Would 16GB be more reasonable default allowing higher installation success rate?

I think it makes sense to default to the same limits used on AWS, for our own sanity.
Based on my last AWS deployment, it looks like 8 GB (m4.large) for bootstrap, and 16 GB (m4.xlarge) for masters.
This will make it very difficult to use a 32 GB host, except for a single master deployment.
64 GB would work, but only for a 3 master deployment without workers.  I suppose we could automate creating a worker VM after the bootstrap VM is destroyed.
128 GB is ideal, but I know most folks don't have machines that big available for this work.

https://github.com/openshift/installer/blob/master/docs/user/aws/limits.md#instance-limits

I agree. I've been seeing OOM killed pods with 8GB masters. My deployment with 16GB masters from yesterday is still online - longest successfully running deployment I've had lately.
We've made some assumptions about there being 3 masters for metalkube, so unfortunately that means you need really powerful hardware. But it's rather pointless to have these low limits if it results in instability and a good percentage of deployments failing.
I'm working on making changes to accommodate only a single master which should make a 32GB box acceptable for development (KNIDEPLOY-429).

Sounds reasonable, I've been holding off though because I know this will impact existing users on 32G boxes until we get the single-master thing worked out.
Yesterday I did push https://github.com/openshift-metalkube/dev-scripts/pull/339 too which should enable easier customization, e.g for single master/worker or custom memory/cpu sizes.  We should optimize the default path to "just work" as much as possible though, even if it means by default a larger test host is required.

Ok that patch increases the master nodes to 16G, and reduces the worker count to 1.
Re the bootstrap VM, in my environment that only gets 4G so I guess we need to confirm if that needs increasing as well, AFAICT it's working OK though

I'm looking at upstream memory requirements[0] and they state 16G for masters.  Maybe we've just been doing it wrong the whole time, attempting to use 8G.
[0] https://docs.okd.io/latest/install/prerequisites.html

Just to confirm, I ran vmstat in the background and can confirm we do indeed basically run out of memory with a default_memory of 8192 specified in tripleo-quickstart-config/metalkube-nodes.yml:
procs -----------memory---------- ---swap-- -----io---- -system-- ------cpu-----
r  b   swpd   free   buff  cache   si   so    bi    bo   in   cs us sy id wa st
10  0  27648 8799540      0 11148604    0    0     0    47 15536 11527 73 15 12  0  0
7  0  27648 8724768      0 11149784    0    0     0    25 15718 10032 69 12 19  0  0
2  0  27648 8674748      0 11150948    0    0     0    27 8680 11505 32 10 58  0  0
5  0  27648 8648832      0 11150948    0    0     0     0 6760 10718 27 11 63  0  0
5  0  27648 8608160      0 11150952    0    0     0     0 9740 11945 47 17 36  0  0
10  0  27648 8465268      0 11151036    0    0    76   435 12989 14346 53 24 22  0  0
7  0  27648 8246360      0 11151276    0    0     0   564 13998 17058 56 31 13  0  0
7  0  27648 7896492      0 11151176    0    0     0  2542 15612 18893 58 32 10  0  0
8  0  27648 7575840      0 11151584    0    0     0   430 15281 19264 48 34 18  0  0
7  0  27648 7226444      0 11151648    0    0     0    10 15435 16718 56 33 11  0  0
5  0  27648 6792260      0 11360376    0    0     0    95 15148 31816 40 31 29  0  0
7  1  27648 4486760      0 13602468    0    0     0<PHONE_NUMBER>3 31558 32 39 27  1  0
1  5  27648 2989744      0 15083124    0    0     0<PHONE_NUMBER>8 31921 26 29 32 13  0
3  2  27648 2451676      0 15617612    0    0     0<PHONE_NUMBER> 16460 10 15 39 37  0
1  4  27648 1927412      0 16141104    0    0     0<PHONE_NUMBER> 12990  9 15 45 31  0
1  4  27648 1571700      0 16496072    0    0     0<PHONE_NUMBER> 4689  5 11 44 39  0
1  5  27648 1012480      0 17054256    0    0     0<PHONE_NUMBER> 6937  6 13 37 44  0
1  5  27648 664920      0 17402064    0    0     0<PHONE_NUMBER>2 11115  6 15 38 42  0
1  5  27648 570924      0 17496904    0    0     0<PHONE_NUMBER>1 6857  5 13 35 47  0
3  5  27904 369128      0 17697036    0  340     0<PHONE_NUMBER>3 8592  5 16 42 37  0
1  5  28160 356136      0 17709488    0  156     0<PHONE_NUMBER> 6369  5 12 37 47  0
1  6  30464 236668      0 17827852    0 2224     0<PHONE_NUMBER> 7279  5 14 32 48  0
2  5  31232 240608      0 17825784    0  900     0<PHONE_NUMBER> 5265  5 12 39 44  0
3  4  33280 239268      0 17829956    0 1912     0<PHONE_NUMBER> 46663 13 22 28 38  0
2  4  35328 199016      0 17868648   32 2180    32<PHONE_NUMBER> 61442 16 26 28 30  0
4  3  35840 198704      0 17871324    0  592     0<PHONE_NUMBER> 59156 17 21 30 32  0
4  4  37376 199356      0 17874700    0 1516     0<PHONE_NUMBER> 63749 17 25 32 26  0
9  3  37632 231044      0 17846520  312<PHONE_NUMBER>54 7237 64255 17 25 31 27  0
4  5  38656 199052      0 17882168  728<PHONE_NUMBER>01 8018 65690 17 28 29 26  0
2  4  39168 221864      0 17856652  908<PHONE_NUMBER>40 7835 59056 15 28 33 23  0
This definitively concludes that we are running out of memory - by hot much, I'm not sure, but I think we can get this done with less than 16G per master.

Note that the aforementioned memory counters are running during the OpenShift Installer:
evel=debug msg="State path: /tmp/kni-install-158504556/terraform.tfstate"
level=debug msg="OpenShift Installer 0447ab1fa360139aef298a1a8bc17fdf6302caec"
level=debug msg="Built from commit 0447ab1fa360139aef298a1a8bc17fdf6302caec"
level=info msg="Waiting up to 30m0s for the Kubernetes API at https://api.ostest.test.metalkube.org:6443..."
level=debug msg="Still waiting for the Kubernetes API: Get https://api.ostest.test.metalkube.org:6443/version?timeout=32s: dial tcp <IP_ADDRESS>:6443: connect: connection refused"
level=debug msg="Still waiting for the Kubernetes API: Get https://api.ostest.test.metalkube.org:6443/version?timeout=32s: dial tcp <IP_ADDRESS>:6443: connect: connection refused"
level=debug msg="Still waiting for the Kubernetes API: Get https://api.ostest.test.metalkube.org:6443/version?timeout=32s: dial tcp <IP_ADDRESS>:6443: connect: connection refused"
level=debug msg="Still waiting for the Kubernetes API: Get https://api.ostest.test.metalkube.org:6443/version?timeout=32s: dial tcp <IP_ADDRESS>:6443: connect: connection refused"
level=debug msg="Still waiting for the Kubernetes API: Get https://api.ostest.test.metalkube.org:6443/version?timeout=32s: dial tcp <IP_ADDRESS>:6443: connect: connection refused"
level=debug msg="Still waiting for the Kubernetes API: Get https://api.ostest.test.metalkube.org:6443/version?timeout=32s: dial tcp <IP_ADDRESS>:6443: connect: connection refused"
level=debug msg="Still waiting for the Kubernetes API: Get https://api.ostest.test.metalkube.org:6443/version?timeout=32s: dial tcp <IP_ADDRESS>:6443: connect: connection refused"
level=debug msg="Still waiting for the Kubernetes API: Get https://api.ostest.test.metalkube.org:6443/version?timeout=32s: dial tcp <IP_ADDRESS>:6443: connect: connection refused"
level=debug msg="Still waiting for the Kubernetes API: Get https://api.ostest.test.metalkube.org:6443/version?timeout=32s: dial tcp <IP_ADDRESS>:6443: connect: connection refused"
The system starts swapping, things slow down, causing timeouts, if not simply OOMing.  This is likely the cause of the transient errors, the "sometimes it works sometimes it doesn't" failures.