Timing Issue

I appear to having a timing issue when I use this.
I save the file, the file saves, the reformat occurs and the file is marked as "dirty" again.
Is there a setting I need to change to get the reformat to occur before tha save?

I have the exact same issue. I have upgraded to VS17, and the problem still occurs.
In large files, I experience an even worse bug: On saving, the code gets saved, and the reformat happens after the code is saved. The code is then marked as dirty, because of changes from the reformat action. When saving again the code gets saved, and reformat happens again, and the code is marked as dirty again, even though reformat also were done in the previous save. It makes it impossible to save the file, without cancelling the auto reformat action.

I have disabled this plug-in for now and I'm using the re-format on save option in CodeMaid instead which works as expected.

I got this issue when I upgraded ReSharper from 2017.1.1 to 2017.1.3.
I rolled back to 2017.1.1 version and extension works fine again.

https://youtrack.jetbrains.com/issue/RSRP-465536

Until ReSharper provides a better alternative, is there any way for this extension to watch/wait for the file to be modified before the final save?  Maybe ReSharper code clean always changes the file modified time, so you would check every second to see if the time is within x seconds.  It's hacky, but it would be better than what we currently have.

The file time will not change after the cleanup ran, because the file is not saved.
The only way this could be done is run cleanup and save, wait a few seconds and check if the file is not saved and then save again. But there's a high chance, that the change was not only the cleanup but other stuff, that you didn't want to save. Or that the cleanup is still running. Or the user immediately started a build and the file is saved after being compiled. Really horrible race conditions. This can only be fixed within ReSharper.

Right after the CodeCleanup has run couldn't you just disable the OnDocumentSaved event then save the file and re-enable the event?
Alternatively or if that doesn't work couldn't you save a list of documents to do the CodeCleanup on (looks like this is already being done) then run it on all of them, then save each one and if the document is in the list that just ran skip the cleanup this time.  Finally you would need to clear the list so that they don't get skipped accidentally if more changes are made.

@aetaylor , I believe this is due to Visual Studio's API only triggering the extension AFTER it has already saved the document (hence the need to save it again).
@NelsonRothermel , implementing an automatic second save is dangerous, as calling save in code will trigger another format. This can potentially cause an endless loop of saving and formatting (for example, if you have both CodeMaid and ReSharper.AutoFormatOnSave listening for the Saved event, you might end up with documents that may never be "not dirty").
To be honest, I doubt this will actually get fix, or even be investigated, since as far as I'm aware, the owner is no longer maintaining this extension. It is due to this very reason (and the fact that VS 2019.1 doesn't like it using legacy API), I have (just) created my own extension. Admittedly my extension also suffers from the same issue (hence my insight), however, I am investigating alternative ways to get around this.
I sincerely hope the owner decides to continue maintaining this extension and is able to fix these issues.


Make clients use the router-assigned session ID

At the moment clients generate their own WAMP session ID (starting at 1), instead of using the router assigned value. This appears to be a bug, as the WAMP session ID should be a unique number and both the client and the server should have the same unique session ID for a session.
This merge request forces clients to use the router-assigned session ID, and thus improves adherenece to the WAMP protocol. A router can still optionally provide a custom session_id_generator_fn function  to allocate the session IDs.

Hi, thanks for this.   Am examining it.  I looked at the wamp specs, and yeah it does seem as if the router does provide a session ID on welcome, although I cannot see what use the client has for it.  It is of use to the router, certainly, but for the client, not sure.  Anyhow, I think it makes sense to have his change added to the code.  Will get back to you as I make progress.

Excellent, thanks for looking at the pull request and merging it :-).
Having the client know it's WAMP session ID can be quite useful. As an example, if a callee is using the callers session ID for some internal logic, and the caller gets disconnected, then the caller can connect again and provide it's old session ID to re-identify itself.


The to of teleport specifies that the div of the parent of the Vue component reports an error

Version
3.0.5
Reproduction link
https://github.com/mayuxian/vite-issue-sample/blob/master/src/App.vue
Steps to reproduce

yarn create @vitejs/app vite-issue-sample
create "modal-button"  component,

App.vue  import "modal-button",as below

Tooltips with Vue 3 Teleport

What is expected?
1.no error
2.If it is forbidden,please give a friendly tips
What is actually happening?
runtime-dom.esm-bundler.js:30 Uncaught (in promise) TypeError: Cannot read property 'parentNode' of null
at parentNode (runtime-dom.esm-bundler.js:30)
at patchBlockChildren (runtime-core.esm-bundler.js:4006)
at Object.process (runtime-core.esm-bundler.js:4990)
at patch (runtime-core.esm-bundler.js:3715)
at patchBlockChildren (runtime-core.esm-bundler.js:4010)
at processFragment (runtime-core.esm-bundler.js:4065)
at patch (runtime-core.esm-bundler.js:3705)
at componentEffect (runtime-core.esm-bundler.js:4278)
at reactiveEffect (reactivity.esm-bundler.js:42)
at callWithErrorHandling (runtime-core.esm-bundler.js:154)

Should be the same as https://github.com/vuejs/vue-next/issues/2015, tracked there.


CAMEL-11328 - date:now: is not accepted, formatting done in …

…header

Whats the error etc when you dont use the header? And what is that a in that date pattern?

The error is in https://github.com/apache/logging-log4j2/blob/master/log4j-core/src/main/java/org/apache/logging/log4j/core/lookup/DateLookup.java#L64
format is now: where now is not recognised. That's because i tried setting header.
I agree however i could not come up with anything to help log4j to recognise now with multisemicolon and simple language, i would either concat formatted now date or setting it on exchange header. I thought as this was example it would not hurt using new in Camel DSL and setting it on header. You may be right using new may mislead our users, however considering every message reaching to direct:message endpoint now should be calculated, maybe dategenerator could be implemented but this is example i did not mind using new Date().
Apart from that, as a side note, a character means AM or PM in date format.

Okay then its nicer to just remove the date pattern which I have done


⚠ Bofrost.lu has degraded performance

In f32f024, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 914 ms

Resolved: Bofrost.lu performance has improved in 45c9659 after 7 minutes.


[Event Request] table 39 "Purchase Line" - OnUpdateDirectUnitCostByFieldOnAfterSetBlanketOrderPriceFields

Please add a new Publisher Call OnUpdateDirectUnitCostByFieldOnAfterSetBlanketOrderPriceFields in Fct. UpdateDirectUnitCostByField of table 39 "Purchase Line"
Add Publisher Call in Fct. UpdateDirectUnitCostByField:

                            Validate("Direct Unit Cost", BlanketOrderPurchaseLine."Direct Unit Cost");
                            Validate("Line Discount %", BlanketOrderPurchaseLine."Line Discount %");
+                           OnUpdateDirectUnitCostByFieldOnAfterSetBlanketOrderPriceFields(PurchHeader, BlanketOrderPurchaseLine, Rec,CalledByFieldNo,CurrFieldNo);
                        end;
                    if (xRec."Direct Unit Cost" <> Rec."Direct Unit Cost") or not (CalledByFieldNo in [FieldNo("Job Task No."), FieldNo("Job No.")]) then

Add new Publisher:

+   [IntegrationEvent(false, false)]
+   local procedure OnUpdateDirectUnitCostByFieldOnAfterSetBlanketOrderPriceFields(PurchaseHeader: Record "Purchase Header"; BlanketOrderPurchaseLine: Record "Purchase Line"; var CurrentPurchaseLine: Record "Purchase Line"; CalledByFieldNo: Integer; CurrFieldNo: Integer)
+   begin
+   end;

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


Output is always 1

I'm running the following code to test my custom images, and here is the cude:
import tensorflow as tf
from utils import Dataset, plotImages, plotWrongImages
from models import EfficientCapsNet
import cv2, numpy as np

img_path = "imgs/box_9.jpg"
img_np = cv2.bitwise_not(cv2.imread(img_path, 0)) # invert conversion

gpus = tf.config.experimental.list_physical_devices('GPU')
tf.config.experimental.set_visible_devices(gpus[0], 'GPU')
tf.config.experimental.set_memory_growth(gpus[0], True)

# some parameters
model_name = 'MULTIMNIST'
custom_path = None # if you've trained a new model, insert here the full graph weights path

dataset = Dataset(model_name, config_path='config.json')

model_test = EfficientCapsNet(model_name, mode='test', verbose=True, custom_path=custom_path)

model_test.load_graph_weights() # load graph weights (bin folder)

model_test.evaluate(dataset.X_test, dataset.y_test) # if "smallnorb" use X_test_patch

#not working with MultiMNIST

shape = dataset.X_test[0].shape

img_np = cv2.resize(img_np, shape[:2], interpolation=cv2.INTER_AREA)
img_np = img_np[..., None]

y_pred = model_test.predict(np.expand_dims(img_np, axis=0))[0] # if "smallnorb" use X_test_patch

print(np.argmax(y_pred))
print(y_pred)

cv2.imshow('show', img_np)
cv2.waitKey(0)

And here is the result:
[[0.3153154  0.956559   0.2567287  0.2726928  0.18593171 0.6188526
  0.14006346 0.44058013 0.36337793 0.15720315]]

As it's shown, index 1 has highest value and it means the output is 1. But I tested 13 images and all has highest value in index 1 (output is always 1). What am I doing wrong?

How are your test images generated? You are using the model trained on the MultiMNIST dataset, that is expecting two MNIST images superimposed and has been trained in order to dect both of them. If you have classical digit images, set
model_name = 'MNIST'


Last createImage() example does not work

On the createImage() reference page (why did all the pages move to the staging subdomain? I'm confused), the last example does not work.
Line 4 reads
var d = pixelDensity;
but needs to be
var d = pixelDensity();
Btw., I would have loved to fix this myself, but I find the building instructions super confusing. I am not sure at all which pages I need to modify before building.
Edit: Oh, is it some inline docs in the p5.js repo actually? Then I guess I would know how to fix that in the src folder in place, but I still don't know what I have to build after that. 😞

thanks, should be fixed now https://p5js.org/reference/#/p5/createImage

oops sorry, the problem with routing to staging domain is fixed. example still needs update :)

@trych yes, it is all inline. you can submit a PR if you make the edit, and the docs will get rebuilt next time I push them to the website. but looking closer, I think the line is already correct?

I think then it was related to the staging thing, this is what I saw earlier on https://staging.p5js.org/reference/#/p5/createimage:

But to be certain for the future: I can just edit the inline docs directly in the main p5.js repo, and then do a PR without doing a build myself? Is that correct?

yep that's correct


Delay root gradient until page load

This quick fix prevents an occasional FOUC issue that results in:

by delaying that underlying yellow to blue gradient until the page is loaded.

Closing until I can work out how to untrack random package-lock changes 😅


fix type cast design and bugs

The cast syntax was not initially implement to apply to any expression.  The grammar should be generalized so casting can be done to anytning (e.g., 1:port should work or put x=len(foo:string):uint8).

Also, the cast implementation should use a scratch buffer so they don't send all the results through GC.

This is fixed now and with Go-style cast syntax and type-literal syntax so you can even say <{a:1}>(value) syntactially though this currently triggers a bug.  Closing this issue and creating new ones:
issue #3327
issue #3328


请问这个框架不支持蒸馏吗？

Traceback (most recent call last):
File "/opt/conda/lib/python3.8/runpy.py", line 194, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/opt/conda/lib/python3.8/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/data/juicefs_hz_cv_v3/11105507/AAAI2022/united-perception2/up/main.py", line 31, in
main()
File "/data/juicefs_hz_cv_v3/11105507/AAAI2022/united-perception2/up/main.py", line 25, in main
args.run(args)
File "/data/juicefs_hz_cv_v3/11105507/AAAI2022/united-perception2/up/commands/train.py", line 161, in _main
launch(main, args.num_gpus_per_machine, args.num_machines, args=args, start_method=args.fork_method)
File "/data/juicefs_hz_cv_v3/11105507/AAAI2022/united-perception2/up/utils/env/launch.py", line 68, in launch
main_func(*(args,))
File "/data/juicefs_hz_cv_v3/11105507/AAAI2022/united-perception2/up/commands/train.py", line 140, in main
runner = RUNNER_REGISTRY.get(runner_cfg['type'])(cfg, **runner_cfg['kwargs'])
File "/data/juicefs_hz_cv_v3/11105507/AAAI2022/united-perception2/up/utils/general/registry.py", line 81, in get
assert module_name in self, '{} is not supported, avaiables are:{}'.format(module_name, self)
AssertionError: kd is not supported, avaiables are:{'base': <class 'up.runner.base_runner.BaseRunner'>}

If you want to use distill, you need add tasks export DEFAULT_TASKS=cls:distill @ersanliqiao

If you want to use distill, you need add tasks export DEFAULT_TASKS=cls:distill @ersanliqiao

多谢，改完可以跑了，另外，想问下我跑nas，导入task也改了（export DEFAULT_TASKS=cls:bignas
），但是还是报错：
no custom work to be done.

export DEFAULT_TASKS=cls:bignas
DEFAULT_TASKS=cls:bignas
bash scripts/dist_train.sh 1 /data/juicefs_hz_cv_v3/11105507/AAAI2022/united-perception2/configs/nas/bignas/cls/bignas_regnet_800m_train_supnet.yaml
No module named 'petrel_client'
init petrel failed
No module named 'spring_aux'
2022-12-06 17:40:51,629-rk0-normalize.py#37:import error No module named 'mqbench'; If you need Mqbench to quantize model,      you should add Mqbench to this project. Or just ignore this error.
2022-12-06 17:40:51,629-rk0-normalize.py#44:import error No module named 'msbench'; If you need Msbench to prune model,     you should add Msbench to this project. Or just ignore this error.
Traceback (most recent call last):
File "/opt/conda/lib/python3.8/runpy.py", line 194, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/opt/conda/lib/python3.8/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/data/juicefs_hz_cv_v3/11105507/AAAI2022/united-perception2/up/main.py", line 31, in
main()
File "/data/juicefs_hz_cv_v3/11105507/AAAI2022/united-perception2/up/main.py", line 25, in main
args.run(args)
File "/data/juicefs_hz_cv_v3/11105507/AAAI2022/united-perception2/up/commands/train.py", line 161, in _main
launch(main, args.num_gpus_per_machine, args.num_machines, args=args, start_method=args.fork_method)
File "/data/juicefs_hz_cv_v3/11105507/AAAI2022/united-perception2/up/utils/env/launch.py", line 68, in launch
main_func(*(args,))
File "/data/juicefs_hz_cv_v3/11105507/AAAI2022/united-perception2/up/commands/train.py", line 140, in main
runner = RUNNER_REGISTRY.get(runner_cfg['type'])(cfg, **runner_cfg['kwargs'])
File "/data/juicefs_hz_cv_v3/11105507/AAAI2022/united-perception2/up/utils/general/registry.py", line 81, in get
assert module_name in self, '{} is not supported, avaiables are:{}'.format(module_name, self)
AssertionError: bignas is not supported, avaiables are:{'base': <class 'up.runner.base_runner.BaseRunner'>}
train <class 'up.commands.train.Train'>

If you want to use advanced features,  maybe you need to compile extensions.  following the docs @TheWangYang


remove unneeded localPlayer updateds

closes #2621
this PR removes the update code which will not be needed once the vrm templates handle that via useFrame.  Currently requires https://github.com/webaverse/totum/pull/86 before it can be merged

ready for review, https://github.com/webaverse/totum/pull/86 which this depends on is also waiting for review

The Totum PR has been merged but this one has not, so they are desynchronized and I suspect puling totum into master at this point would be broken.

Totum PR has been reverted: https://github.com/webaverse/totum/pull/86
So it will need to be submitted and merged synchronously with this one next time.

new totum PR with useFrame was added to this.  have not recieved any feedback regarding the review notes, but both this and totum should be ready to merge.

closing this PR as there will be a general change in approach.  will be addressed in a new PR


Superman 5.0.0 running update before user gets opportunity to log in

Hi
First - I really, really like this project - thank you.
I had two test users report that with Superman 5.0.0 their machines restarted shortly after they started up in the morning and installed the just released update with no chance of deferral.  Both users reported that they logged on as soon as the machines came up but when I look in the super.log file I see this kind of entry
Tue Oct 29 08:22:42 dep69226 super[435]: Status: No GUI user currently logged in.
I got them to run "last" to try and figure out timing
itadmin    ttys000  <IP_ADDRESS>         Tue 29 Oct 09:42 - 14:19  (04:36)
itadmin    ttys000  <IP_ADDRESS>         Tue 29 Oct 09:10 - 09:19  (00:08)
00014170   console                         Tue 29 Oct 08:37   still logged in
00014170   console                         Tue 29 Oct 08:36 - 08:37  (00:00)
reboot time                                Tue 29 Oct 08:35
shutdown time                              Tue 29 Oct 08:32
root       console                         Tue 29 Oct 08:30 - shutdown  (00:01)
00014170   console                         Tue 29 Oct 08:23 - 08:30  (00:07)
reboot time                                Tue 29 Oct 08:22

So if I'm reading this correctly, the machine started at 8:22, super started doing stuff at 8:22:42 and then the user logged in at 8:23.   Super is just too darn quick!
Is there a way to disable super from doing anything when nobody is logged in?  With people being mobile so much they frequently shut down.  There was apparently no pop up to allow deferral but I can see that was because this started when super decided nobody was logged in.
I'm just trying to avoid somebody getting an update like this and they are just launching into a presentation.  The big feature for me is the ability to defer for a set number of days but it seems like there is a small window where super can just bypass everything.  I appreciate that in some cases you want super to install when nobody is logged in - I have labs where that's my plan but in the scenario above even if it waited for a set time to make sure someone wasn't just booting, it would be helpful
2024-10-29_dep69226_super.log
dep69226_last.txt

Indeed... super started up and saw that no one was logged in... and then procededed to install.
This would need to be a new feature to NOT do that since a lot of folks (in shared lab situtations) expcect super to automatically install if there is no current user.

Thanks Kevin,
Yes, I want to take advantage of this "No GUI user currently logged in" for our labs.  But for other people, it seems like super is too fast.
This is your baby but here's some suggestions:

Instead of a LaunchDaemon, have Super run as a LaunchAgent if some configuration setting is made to only run super when a user is logged on.
Have some sort of delay after restart variable that can be set to give the user at least a chance of logging in

We have seen this in our environment too. I understand that super sees that the user is logged in and that starting the update flow is the correct logic but... I also think that the flow restarting a device with no warning to a user should not happen.
The improvements to the user experience and notification from super are one of the key things about it to me!

We experienced the same in our environment.
As a workaround i have now modified the super-starter script to add a customizable delay after boot and just exit 0 when we are not past this delay. I guess 900 seconds (the value i chose) should be sufficient for everyone to sign in.
Of course i actually modified the super script (which writes the super-starter script).
This is a quick solution and may be implemented more elegant (e.g. call it by parameter).
Here is a diff output (includes some timeouts i modified too):
`14a15,18

Delay super after system boot (in seconds) to prevent accidental update installation (with no warning at all)
BOOT_DELAY=900
readonly BOOT_DELAY

471c475
< 	TIMEOUT_INSTALLER_DOWNLOAD_SECONDS=300

TIMEOUT_INSTALLER_DOWNLOAD_SECONDS=600
475c479
< 	TIMEOUT_INSTALLER_WORKFLOW_SECONDS=600

TIMEOUT_INSTALLER_WORKFLOW_SECONDS=1800
479c483
< 	TIMEOUT_MDM_COMMAND_SECONDS=300

TIMEOUT_MDM_COMMAND_SECONDS=900
483c487
< 	TIMEOUT_MDM_WORKFLOW_SECONDS=600

TIMEOUT_MDM_WORKFLOW_SECONDS=900
3563a3568,3574
Exit this script if last boot time was under ${BOOT_DELAY} seconds
kernel_boot_epoch=$(sysctl -n kern.boottime | awk -F 'sec = |, usec' '{print $2}')
current_time=$(date +%s)
if [[ $((current_time - kernel_boot_epoch)) -lt ${BOOT_DELAY} ]]; then
exit 0
fi
10834c10845
< exit_clean

exit_clean`

Hey @sch4llfl3g3l that looks like a better idea than what I was thinking of.  What line do you put the block:
# Exit this script if last boot time was under \${BOOT_DELAY} seconds kernel_boot_epoch=\$(sysctl -n kern.boottime | awk -F 'sec = |, usec' '{print \$2}') current_time=\$(date +%s) if [[ \$((current_time - kernel_boot_epoch)) -lt ${BOOT_DELAY} ]]; then 	exit 0 fi
in the super script?

Hi @david-london
line 3635 is correct (that makes it the first condition to be checked).
And BOOT_DELAY about line 14/15.
If this would be integrated into a release it wasn't the right place, but so far it is convenient for me (separate from the original code and i can quickly change it there).
Take care of the escaping backslashes when copying the code.
You can add a line with an echo message to a file in /tmp above the "exit 0", for debugging purposes.
That way you can check if the condition is properly executed after boot.
If so it must write 15 lines into that file if the BOOT_DELAY is 900 seconds (15 minutes - the launch daemon runs every minute)

Hi @sch4llfl3g3l
It looks to be working for me.  The 15 minutes delay from startup seems like a reasonable time too.
I actually think this is a good compromise with how Kevin designed it.  However, knowing the tight windows people have with labs sometimes, I think a way of turning this on via a config profile or startup settings would be needed for wider use.


feat(linter): eslint/no-shadow-restricted-names

Implement ESLint's no-shadow-restricted-names rule.
To get started, run
just new-rule no-shadow-restricted-names

I'll take this.


Add train/val split to data pipline

Detailed Description
Split the datatimes depending if 'tra','val' or 'test'.
Context
Need to think if we should have this split datetimes as an input to the date pipline, so we can have a train and validation datapipleline
Possible Implementation
Code in nowcasting_datasetis ready to go.

I've added a version in #74 that can split based off a date range, so e.g. 2020 for training, 2021 for testing. Doesn't work with other methods though.

great thanks @jacobbieker , at least getting the trianing and testing split working is very useful


Cleaning up error messages and removing unused states from the state.

Clean up error messages
Remove unused statetracker states
fix for a bug that applied search action one last time just before a timeout without checking the hierarchy again. As a result, the screenshot was different from when the actual failure was expected to occur.

Waiting for travis but LG aside from a nit.


Incorrect body pathing.

Tried to create a body texture for masculine seeker TBSE and HRBODY. It alters roegadyn male skin textures instead of miqote male.

This should be fixed in version <IP_ADDRESS>


Slider slow when onChange triggers redux state change

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Steps:

Create a react app with react-redux store
Create a virtual windowed list with accordions (allow for random size)
Add a slider (normal Slider or MuiSlider component)
Have the onChange method call a redux action creator that sets some state
Observe the lag in the slider

Current behavior 😯
It appears as though when the slider is linked to an action creator that changes some state (even if it just creates a shallow copy of the state without changes), the slider shows awful lag.
It appears to depend on how many other components are on the page at the time. a lot of accordions seems to seriously affect the performance.
Expected behavior 🤔
Slider to change value and update a data store without significant lag.
Context 🔦
A code sandbox with version 4 does not appear to have an issue, but my app with version 5 does. As the sandbox doesn't seem to handle version 5 well i can't create a test case there to determine if it's my code or the new MUI version.
Your environment 🌎
No response

Oh, and a link to the sandbox with v4 that doesn't display the issue.

Oh, and a link to the sandbox with v4 that doesn't display the issue.

Your link does not work. Can you also provide a Sandbox for v5?

Thanks @siriwatknp. Thanks for your time.
I have no idea how that code broke... it was working perfectly fine when I added the link and I can't see why it's not working now as it's complaining about things that aren't broken. Sandbox doesn't appear to have support for version 5 properly - it breaks all the time, so I can't provide that.
In any case I think I may have just shown that it's not the slider - I created a new react app with the virtual list and slider but had not troubles at all. I suspect, although I can't find it to save my life, that somewhere there are many components that are subscribed to either the output of the slider directly, or something to do with that data and so I end up with many renders. I'm just spending a bit of time refactoring/building my app up from scratch (which is a little time consuming and tedious), but I hope to find either the point where it becomes clear the slider is the issue or that the issue magically goes away. I should be able to update in a few days. Prompt me if I haven't closed this in a 7-10 days or given a full update.

I think I've found the culprit and also why I hadn't noticed it until late.
It appears that the slider is prone to getting stuck while the virtual list below updates and re-renders. I wouldn't have thoughts it was such an issue and that react would be better at only processing changes to the DOM.
I'm currently trying to provide a sandbox with the correct versions of everything, but it appears to be having an issue with redux returning an object instead of a class component on the sandbox, but not on my local machine, so not sure what's going on there.


HBASE-25254 Rewrite TestMultiLogThreshold to remove the LogDelegate i…

…n RSRpcServices

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 14s
Docker mode activated.


Fix 'Client socket error' while running tests

Client socket error: Error [ERR_SOCKET_CLOSED]: Socket is closed

This isn't a problem anymore.
Not sure if I fixed it, or if upgrading something fixed it!


[MINOR][DOC] Move 4 functions from 'Normal Functions' to 'Collection Functions'

What changes were proposed in this pull request?
Move 4 functions from 'Normal Functions' to 'Collection Functions':

create_map
array
map_from_arrays
strcut

Why are the changes needed?
they should be collection functions
Does this PR introduce any user-facing change?
doc updated
How was this patch tested?
doc-only

cc @HyukjinKwon

merged into master, thank you


Finalize and add deco tigertooth

The design calls for tigertooth shapes and background smattered about the page. I have these working with ::before and ::after in local dev, and uploaded the SVGs to the CMS. Need to finalize the design and figure out how to best trigger them on page builds. A few options are toggles at snippet level, or simply wrapping the snippets in a containing div when a deco element is needed. We could also potentially create an asset that can be placed on the page when needed to "activate" the design.

Initial commit occured. Need to fine tune additional styling and determine how to call it in the CMS.


Error: open main.tf: no such file or directory (use with pre-commit)

Hi!
Prerequisites

[x] This issue describes a bug.

Description
Since version 0.9.0, terraform-docs seems to require a main.tf. In my company standards, some of our projects does not have a maint.tf…
Actual Behavior
When a project does
$ pre-commit run -a

Error: open main.tf: no such file or directory

Expected Behavior
Not sure the problem come from terraform-docs because it still runs as is, however only terraform-docs has changed. Within pre-commit: v0.8.2 works, v0.9.0 does not.
Maybe you could give me useful information.
Version
erraform-docs version v0.9.0 89b129b linux/amd64 BuildDate: 2020-03-31T20:19:01+0000

@gui-don:
Change you're pre-commit config to this:
 repos:
 - repo: git://github.com/antonbabenko/pre-commit-terraform
   rev: v1.27.0
   hooks:
     - id: terraform_fmt
     - id: terraform_docs
+      args: ['-a', '--no-header']

But since behavior changed, it should have been a breaking release.

Thanks @zopanix.

@zopanix
As discussed in #152 semver will start with 1.0.0 until then breaking changes could be included.

@gui-don thanks for reporting this, this was - unfortunately - a regression added in #217 and I've fixed it in #235 , for the time being you can use @zopanix's workaround until v0.9.1 is released.

I also completely agree with @tbugfinder, and if we hadn't introduced the regression, v0.9.0 release wouldn't have been a breaking release anyway. But generally there's no hard promise of backward compatibility for v0.x, although we try our best to do it anyway.

Fixed by #235


allow units to be disabled and then re-renabled

Closes #359

@whoburg, ready for review

looks great.


V1.3 update DG for command alias

Codecov Report

Merging #123 into master will increase coverage by 0.39%.
The diff coverage is 87.50%.

@@             Coverage Diff              @@
##             master     #123      +/-   ##
============================================
+ Coverage     66.68%   67.07%   +0.39%
- Complexity      562      568       +6
============================================
  Files            99      100       +1
  Lines          1993     2032      +39
  Branches        213      215       +2
============================================
+ Hits           1329     1363      +34
- Misses          579      580       +1
- Partials         85       89       +4

Impacted Files
Coverage Δ
Complexity Δ

.../seedu/address/logic/parser/AddressBookParser.java
65.51% <50.00%> (-2.34%)
12.00 <0.00> (-1.00)

...ava/seedu/address/logic/commands/ValidCommand.java
88.88% <88.88%> (ø)
7.00 <7.00> (?)

...rc/main/java/seedu/address/model/ModelManager.java
95.77% <100.00%> (+0.12%)
30.00 <2.00> (ø)

Continue to review full report at Codecov.

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


New version: Tableau.TableauPrepBuilder version 23.3.30378

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow
This is an update from a Vanity URL to the actual download URL for the latest version.

This is an update from a Vanity URL to the actual download URL for the latest version, the installer hash is therefore the same.


question: difference between line height in body paragraphs and unorderd lists

in a recent document using the IEEE template, i am noticing this difference in the line height.  is this correct line height spacing for body paragraphs and unordered lists? @opoudjis @ronaldtse please check at your convenience. thank you

with a bit more checking... generate "main" on 2024-07-03 with mn versions:
Run metanorma --version
Metanorma 2.0.1
Metanorma::Cli 1.10.1
Metanorma::Standoc 2.8.11/IsoDoc 2.10.4
Metanorma::ISO 2.7.10
Metanorma::Iec 2.4.9
Metanorma::IEEE 1.2.10

as shown below, the PDF output seems ok, but the MS-Word output does not.
PDF output

MS-Word output (appears to still have an issue)

Thanks @ReesePlews . We still need to investigate the changes in the latest 2024 template (#372), in addition to this.
@opoudjis is on holiday and this will be fixed possibly next week.

The Word list style is EXACTLY how the style IEEEStds Unordered List is defined in IEESTD-WORDTEMPLATE_v1_2022.doc. The IEEE Numbered List Level n styles are also defined to have that line spacing.
The 2021 Style Manual does not have those line spacings. But I am not going to overrule the Word template provided by IEEE. And those are the only styles in the Word Template that support the dash bullets and letter-first numbered lists of IEEE.
This line spacing also applies to the 2024 template.
I will not be changing the line spacing of lists in the template without EXPLICIT instruction from IEEE editorial staff. That is how their official supplied Word template styles lists. If they have not fixed this issue in the past five years, I will not do so preemptively.

hello @opoudjis thank you for checking and explaining this.  @ronaldtse has been communicating with IEEE for other/additional samples.  the document i am working on has just finished ballot and will go to the revision committee to handle the comments. after that IEEE/SA will, i assume, handle any final style issues. i will keep you and Ron informed of the status.  thank you.

I will say, it is very strange that both the 2022 and the 2024 template have this spacing, but the sample documents do not. But I can't change the template without instruction: this was done by IEEE deliberately.
The 2024 template does not include a list, which may be why they have not noticed this. But surely they receive any number of submissions with the spaced lists...

I have inquired IEEE Patrick Gibbons directly about this issue. We should not change IEEE's Word template without their confirmation.

Response was to leave this alone; they routinely fix this at the end of editing internally, and they don't have access to a developer to fix their template.


Fixed gallery not sorted

The gallery interface sort fix only fixes the thumbnails.
Thus clicking an image will opens an entirely different image.


protocol: add feature to create MPTCP sockets

Hey !
I was working on this "feature" for a side project, I thought it might be nice to submit it upstream :)

This commit add support for creating MPTCP sockets.
Small details:

We can only create a MPTCP socket in a TCP context. This is why we check that we're in an SOCK_STREAM socket type and that the domain is either AF_INET(6).

Disabled the TTL test and helper in case of MPTCP as it's not supported by the protocol.
-> "Operation not supported"

As a more general note, MPTCP is now more widely
available since Linux Kernel >= 5.6. But it still
need to be enabled manually using:
$ sysctl net.mptcp.enabled=1

I don't think we'll support it Mio as it tries to follow the API in std lib. I'm afraid you'll have to use socket2. It may be possible to convert a socket2 socket to a TcpStream and use that? Otherwise you'll have to use SourceFd.


Docker rules engine issue 116

See issue #116
Docker will only process sub directories so it needed to be above rules-engine and heat-stack directory.
I created a copy of the original Dockerfile to /Dockerfile-original to make comparison easier.

Hi @ethanstrominger thanks so much for this!
I just want to note that though we didn't get to check the final insertion-into-docker step, hypothetically in this PR from 10 days ago https://github.com/codeforboston/home-energy-analysis-tool/pull/176 gets the rules-engine built python egg into the GitHub Action (GHA) artifacts. Then the vitest uses it from the GHA artifacts. I figured we could get that into docker as it was building. Which approach or mix of approaches do you think we should ultimately use? Either way the versions should match which was our worry with our first other idea.
Sorry I didn't get back to update that original issue with the progress 🫤!
We can totally merge in the docker way: my only tiny reservation is on extra divergence from our docker deploy template which may yet get new features upstream.
There is not any automated upgrade process though so just a bit of extra hand merging later. Also we might need to double check the .dockerignore changes.

Thad - Whatever you think is best.  I don't know enough about the other way to judge. With the docker approach I can test without checking into github.

That's a great point about being able to test locally! Let's use this.


bcbio using hadoop

Is the ipython infrastructure able to work with hadoop clusters? I've got one that lies dormant due to overhead in code migration.

Daryl;
Unfortunately not. As you mention, the hadoop distribution model is quite different from that used in HPC-style environments, so it would take a lot of work to migate the IPython parallel approach there. Sorry to not have a solution for you in this case.


[Settings]: The 'Open JSON File' element's ControlType requires valid values for SizeOfSet and PositionInSet.

Windows Terminal version (or Windows build number)
1.10.2383.0
Other Software
Test Environment:
OS: Windows 11 Version 21H2 (OS Build 22000.168)
Screen Reader: Narrator
Tool: Accessibility Insights for Windows Version 1.1.1689.01
Steps to reproduce
Repro Steps:

Open Windows Terminal.
Open Settings page using "Ctrl + ',' ".
Run Accessibility insights for Window on Settings page and Observe the issue.

User Experience: -> Priority 2
Screen Reader user are impacted here as incorrect position is announced for control which will confuse user.
Guideline Reference:
https://www.w3.org/WAI/WCAG21/Understanding/name-role-value
Attachments:
The 'Open JSON File' element's ControlType requires valid values for SizeOfSet and PositionInSet..zip
Expected Behavior
The 'Open JSON File' element's ControlType requires valid values for SizeOfSet and PositionInSet.
Actual Behavior
The 'Open JSON File' element's ControlType doesn't have valid values for SizeOfSet and PositionInSet. Screen Reader announce 'Open JSON file 11 of 11' which is incorrect.

Bug Update in Dec Test Pass: Issue Still repro as Screen Reader is only announcing name as 'Open JSON file' when user navigate to 'Open JSON file' control using down arrow key.

I think this might be an external issue, if it's still even considered an issue at all?
Here's some background...
SizeOfSet and PositionInSet have now changed

The 'Open JSON File' element's ControlType doesn't have valid values for SizeOfSet and PositionInSet. Screen Reader announce 'Open JSON file 11 of 11' which is incorrect.

Not sure when this was introduced, but on 15a047512, I'm not seeing SizeOfSet and PositionInSet both being 11. Instead, Accessibility Insights is telling me Property does not exist.
What do other apps do
Edge - vertical tabs
Interesting! The "New tab" button there experiences the same behavior.
XAML Controls Gallery
The "Settings" item (which is located in the footer) is marked SizeOfSet: 1 and PositionInSet: 1.
What is the correct behavior here?
In Windows Terminal's settings UI, I'm seeing the navigation view items organized into three sets:

Set 1 includes "Startup" through "Actions"
Set 2 includes "Defaults", all of the profiles, and "Add a new profile"
Set 3 includes "Open JSON file"

@v-rpundir What's the correct behavior here?

[Current behavior] keep SizeOfSet and PositionInSet as undefined. Note that this matches Edge's vertical tabs.
[Match XAML Controls Gallery] SizeOfSet and PositionInSet should both be 1
[Everything is wrong?] The three sets above should all be one contiguous set

CC @DHowett @zadjii-msft for thoughts

Hi @carlos-zamora
This is correct behavior as it is universal throughout Windows Apps.
Case 2: [Match XAML Controls Gallery] SizeOfSet and PositionInSet should both be 1
Below one is current situation, but Screen Reader is still announcing incorrect position & AI is giving Error:
For Case 1: [Current behavior] keep SizeOfSet and PositionInSet as undefined. Note that this matches Edge's vertical tabs.

Thanks :):

@v-rpundir Thanks!
I tried setting fixing SizeOfSet and PositionInSet manually. That's not working.
I've reached out to the XAML team here, as this is an upstream issue that we are blocked by. We're going to have to tag this one as a "tracking external" bug.

Nevermind! Worked with the XAML team and found something that works. This'll be fixed by #12286.

Closing as fixed in Windows Terminal Preview Version: 1.13.10336.0


Change archive name

Fix goreleaser settings

Pull Request Test Coverage Report for Build 202

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

Totals

Change from base Build 199:
0.0%

Covered Lines:
407

Relevant Lines:
442

💛  - Coveralls


Backup Health Know More - Title and image is incorrect.

@cakesoft-swati is this still an issue?


Blog update [WIP]

Update the blog section with the old look with excerpts to be more readable
Some small changes to the blog postsl, since some formats used are not compatible with the new markdown parser we're using
re-pin the primer-breadcrumb to a version that does not break the site (see primer/primer-css#317)

Some pending items:

[ ] make author attribute in blog posts array-compatible
[ ] implement new way to style tables in markdown

some formats used are not compatible with the new markdown parser we're using

Which ones?

This looks awesome! Let's go ahead and ship it and tackle that short checklist in another PR(s).


Miner Implementation

Integrates all the modules together to finally mine bitcoins. I tested with transfer transactions in the pool as well, and our bitcoin miner properly processes them (so the merkle tree root calculations work). :smile:
Also:

Change protocol to expect a JSON object and return a raw string with the block info (no wrapping in a JSON dictionary), as expected by the pool leader
Change integration/protocol tests to reflect that
Remove server use of credentials.json

LGTM :+1:


Versions?

I've tried to run this but i don't now which versions to use...
It says Unity 2021.3.3f, but then with the VRTK it asks for another Unity version...
No combination is working

I've just written a tutorial with a step by step guide to get the "VRTK v4 Scene" working: VR_TUTORIAL.md


Double free in C Stream Interface

Describe the bug
Got a double free when consuming a stream and trying to return the error. Right now the C Stream Interface import tries to take ownership of the error string, but once the release callback is called that string gets freed.
https://github.com/apache/arrow-rs/blob/f16ceedf981c7230167ff858eabb9a8cdc87ffda/arrow/src/ffi_stream.rs#L317-L327
From the C stream interface docs:

The returned pointer is only guaranteed to be valid until the next call of one of the stream’s callbacks. The character string it points to should be copied to consumer-managed storage if it is intended to survive longer.

To Reproduce

Expected behavior
We should probably clone the string as suggested by the interface docs.
Additional context

label_issue.py automatically added labels {'arrow'} from #4660


Create LIST command

list subscription zipcodes -- what is the mechanism for unsubscribing? U 94118? etc.?

Right now you can only sub to one zipcode at once, and whenever you change your sub the old one goes away. So we should be good here.

Aaaah perfect, when you are moving this fast, hard to keep up!
Sent from my iPhone

On Sep 14, 2020, at 9:53 PM, Ian Hoffman<EMAIL_ADDRESS>wrote:

Right now you can only sub to one zipcode at once, and whenever you change your sub the old one goes away. So we should be good here.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Haha yeah. I think this is a good direction! It will be important to gather feedback tho since this is a big big change.


PE view discards the DLL name info of forwarded exports

In https://github.com/Vector35/view-pe/blob/d2503f60c704b5d20698894d06598c565c912c46/peview.cpp#L2472-L2486, the code parses the dll name and function name of a string like dll.func, but it subsequently discards the DLL name and no longer uses it. We should at least retain the DLL name

Related to https://github.com/mandiant/capa/issues/1646


Painful upgrade 3.1.0 -> 3.4.0

Hello
I have an old app that was using simple_form 3.1.0. We recently updated our app to ruby 2.4.1 and last Rails 4.
I thought it will be a good idea to update at the same time simple_form... What a bad idea...
I did a bundle update simple_form then..

Fields for my login (user and email) started asking for a minimum of 255 chars without knowing why.. It appears that when you upgrade you have to update the initializer too. I finally added two lines to disable this behavior.
Test like assert page.find_field('E-mail', disabled: true) start failing... I had to change them to assert page.find_field('user_email', disabled: true).
Fields start to be marked as required by default. Tests that were testing models' validations (with flash notice) start failling.

We loose fields labels on few pages

... then I reverted to 3.1.0.. scared to have more surprises
This is sad we don't have any more warnings or notices in Changelog saying they are breaking changes, text to add to initializer, etc. I love using simple form but I can't deal with so many unwanted surprises.
Do you think it will be possible to  :

Add more infos when their is text to add to initializer
When their new default behavior implemented
Link any changelog entry to the PR when it's possible

Thanks a lot for the job done on this gem.

Thank you for the report. We should not have any breaking changes on those releases and if we have we consider it a bug.
I just checked the diff between those two versions and something is odd here.

The :minlength is disabled by default https://github.com/plataformatec/simple_form/compare/v3.1.0...v3.4.0#diff-2a863c44f5077f0154fd9ffcb84a1f13R54, unless your wrapper enabled it, or you don't have a default wrapper. Do you know whats is the case in your application?

It seems weird since nothing in the labels code changed. Do you know exactly what the tests were failing. Was the text not in the input? For any chance are you using i18n for the labels content?

It seems that it is caused by https://github.com/plataformatec/simple_form/pull/1416. But it should make required fields if you have required option explicitly set in the input. Do you have?

Are you using i18n for those labels?

Do you think it will be possible to :
Add more infos when their is text to add to initializer
We don't do that because application configuration should not need any changes in minor versions, so if it is needed to do something it is a bug.

When their new default behavior implemented

Default behavior only changes in major versions, and even when that happens it is with deprecation messages.

Link any changelog entry to the PR when it's possible

The entry is added in the PR itself, so the git blame already give that information.
Thank you again for the report. And let's fix those issues in your application.

Thanks a lot @rafaelfranca for your answer. I will take the time to answer you properly next days.

Hello
I found where the issue came from. Our initializer was incomplete. It didn't contains config.wrappers :default, and proper content.
It was like :
SimpleForm.setup do |config|
  config.wrappers :horizontal_icon_input, tag: 'div', class: '', error_class: 'field-in-error' do |b|
    b.use :html5
    b.use :placeholder
    b.use :input
    b.use :error, wrap_with: { tag: 'span', class: 'error' }
  end

  config.wrappers :visibility_toggled_password, tag: 'div' do |b|
    b.use :html5
    b.use :input
  end
end

I copy past the last initializer and it's ok.
Sorry for my first message, the error was simply related to one file.


Temporal constraints are not correctly processed in the AQL WHERE clause

The test for the query below fails due to not detecting time overlaps among patients:
Contact_Nth_Degree_TTP_Degree_II
SELECT e/ehr_status/subject/external_ref/id/value as PatientID, o/data[at0001]/items[at0004]/value/value as Beginn, o/data[at0001]/items[at0005]/value/value as Ende FROM EHR e CONTAINS COMPOSITION c CONTAINS ADMIN_ENTRY o[openEHR-EHR-ADMIN_ENTRY.hospitalization.v0] CONTAINS CLUSTER l[openEHR-EHR-CLUSTER.location.v1] where c/name/value='Patientenaufenthalt' and o/data[at0001]/items[at0004]/value/value >= '{starttime}' and o/data[at0001]/items[at0004]/value/value <= '{endtime}' and l/items[at0048]/value/defining_code/code_string = '{departement}' and l/items[at0027,'Stationskennung']/value = '{station}' and not e/ehr_status/subject/external_ref/id/value = '{patientID}' UNION ALL SELECT e/ehr_status/subject/external_ref/id/value as PatientID, o/data[at0001]/items[at0004]/value/value as Beginn, o/data[at0001]/items[at0005]/value/value as Ende FROM EHR e CONTAINS COMPOSITION c CONTAINS ADMIN_ENTRY o[openEHR-EHR-ADMIN_ENTRY.hospitalization.v0] CONTAINS CLUSTER l[openEHR-EHR-CLUSTER.location.v1] where c/name/value='Patientenaufenthalt' and o/data[at0001]/items[at0004]/value/value <= '{starttime}' and o/data[at0001]/items[at0005]/value/value > '{starttime}' and l/items[at0048]/value/defining_code/code_string = '{departement}' and l/items[at0027,'Stationskennung']/value = '{station}' and not e/ehr_status/subject/external_ref/id/value = '{patientID}'
openEHR COID ContactNthDegree_TTP_Degree_II.postman_collection.json.zip

UNION ALL is not supported at the moment, the AQL expression parsing fails.

I think did not explain it well. If you split in two queries the temporal constraint is not taken into account either.


🥰 [FEATURE]能否添加一个多边形挖洞的功能

💻 Features description [Please make everyone to understand it]
可将已经绘制的多边形进行挖洞
🏞 What problem does this feature solve
🧐 What does the proposed API look like
🚑 Any additional [like screenshots]
it looks like this : https://viglino.github.io/ol-ext/examples/interaction/map.interaction.drawhole.html

请问这个功能有进度吗。


Long TS errors can't be read in the problems panel

VSCode Version: 1.38.0
OS Version: MacOS 10.14.6

Steps to Reproduce:

Create a TS error that results in a very long message
Attempt to read the error message in the problems panel

The long problem message is cut off with ellipsis. The only way to read the message is to go to the location and hover over the problem to view the full output in a popup. There is no way to read the problem in the problems panel which given the generally long nature of TS error messages makes the problem panel mostly useless except for showing that a problem exists. Perhaps there should be an option to wrap error output or a scrollbar to see the rest of it.
Does this issue occur when all extensions are disabled?: Yes

its not so often to get into this. and horizontal scroll bars is not a good option by default.


add the mcs framework

What type of PR is this?

feat: new feature

What does this PR do?

Which issue(s) does this PR fix?

Fixes #85
Special notes for your reviewer:

Screenshots (if applicable):

Additional Comments:

/lgtm
/cc @xyz2277 @wuyingjun-lucky

It is better to add some notes to functions on mcs controller next time

/approve


Displaying comments for votes by linking to HoN

Why link out to HoN versus surfacing comments natively?

@salieflewis this issue isn't finalised yet so thanks for asking this question!
for context, there was a BuilderDAO-funded HoN proposal to support all DAOs created using Nouns Builder. linking it can help us (1) showcase our projects built by our community but funded by the DAO and (2) it can also be treated as an intermediary step before we surface comments natively.
there is a working prototype built by Ripe that we're looking to integrate - https://builder-dao.netlify.app/vote/0xf13db9adb12a28a1feba35c4bf861f1a593d71b4be194a3253622f008172f0de
happy to hear suggestions!

Makes sense. In terms of features that would require substantial design input, is the Figma available for branching/PR style contributions?
To be fair I have no experience with this, just know that it is possible. I'd imagine if designs were necessary to move certain features forward on the engineering side, it would be interesting to see what the process of going fully open source on both engineering and design fronts would look like!

we're definitely accepting design contributions, that's why we made the figma file public! but i'll have to check if it has the right account settings for branching/PR style contributions. (thanks for sharing this, i didn't know this existed)
in the meantime, i've exported a copy of the .fig so you can download and explore some design options here:
Dropbox link
please let me know if you can't download it @salieflewis


Increase TLD maximum length

A user with a .construction (length 13) Email pointed this out: https://github.com/NixOS/SC-election-2024/pull/6
I'm updating it to 18, because there's TLDs like https://nic.travelersinsurance/, though in theory you can have TLDs up to length 63, see https://stackoverflow.com/a/22038535.

At which point I honestly don't see much of the point of keeping the length limit at all in the first place. Back in the days where TLDs were all short it may have been useful to prevent errors. But in the end, the only true way to test if an email address is valid is to send it a mail anyways

@andrewcmyers Thanks! Are you also able to deploy this soon?
@piegamesde I opted for minimal changes, but I agree. Considering this is merged already now I'm not inclined to make another PR though :P


data loader issue in the training stage

Thanks for your code!
When I try to add this attention mechanism into the resnet backbone and train a detection network, the error appears as:
File "anaconda3/envs/maskrcnn/lib/python3.7/site-packages/torch/utils/data/_utils/signal_handling.py", line 63, in handler
_error_if_any_worker_fails()
RuntimeError: DataLoader worker (pid 13791) is killed by signal: Aborted.
The batch size and num_workers have been set as 1 and it stills goes wrong.
Could you give some suggestions?

Could you share more info about your setup? Specifically:

What devices are you using?
Which example are you trying to run?
How much memory is available?
Are you running the default examples, or did you make any modifications?

1080ti
I just use the backbone structure into my own model
11G
I make some modifications and did not use the default examples
On 10/20/2020 12:59, Ajay Arasanipalai wrote:
Could you share more info about your setup? Specifically:
What devices are you using?
Which example are you trying to run?
How much memory is available?
Are you running the default examples, or did you make any modifications?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@Fly-dream12 t's going to be a little harder to debug then? Could you share your code?
I'm not sure if this is an issue related to triplet attention specifically.

@Fly-dream12 based on similar error's discussion from PyTorch issues, it doesn't seem to be an error associated to Triplet Attention. Please go through this thread. Thanks!

Thanks, there exist some bug in my dataloader and I have fixed it. @digantamisra98 @iyaja
Meanwhile, could you provide the example code of gradcam++ ?

@Fly-dream12 we will upload the gradcam++ code this week.

@Fly-dream12 I just added the GradCAM notebook here.
Create a new issue if you're facing issues running the code there. Note that you need to have the pretrained models ready for this, and you need the CBAM code as well if you want to generate the exact plots we have in the paper.


Working with large texts

When working with larger texts, labels are at the top of the page, so when you scroll down, you don't see labels anymore. You end up scrolling up to the top to select the label, then scrolling down below to select the text.
Suggestion: you could use additional input parameter "text_height" so that the text area is limited to that height. That way labels are always visible and text scroll occurs inside the dedicated block.
PS: thank you for the nice plugin, it's very helpful.

Thank you for the suggestion! The text_height attribute is implemented in version 0.0.15.

Thanks for the fast implementation, it works. Although for some reason now it reruns the streamlit app every time an annotation is made. I worked it out with st.cache_data in my app, but thought you might want to know this unexpected consequence of seemingly small update.

Good to know, what Streamlit version do you have?

Good to know, what Streamlit version do you have?

1.35.0


Items Respawning

TLD v1.12(32511)
The Better-Placing v1.2.0 causes items such as curtains, stools, crates to respawn after leaving a building or loading the game.  I have tried this on old saves, as well as new games and everytime you leave a building, the items you broke down inside have respawned.

Confirmed!


InfluxDBv2 session and data handling

I have my InfluxDB Version 2.7.11 hosted on AWS EKS Pod with Persistent Storage as EFS and access externally using Load Balancer.
where my deployment.yaml as:
apiVersion: apps/v1
kind: Deployment
metadata:
labels:
app: influxdb-test
name: intel-influxdb-test
namespace: test-env
spec:
replicas: 1
selector:
matchLabels:
app: influxdb-test
template:
metadata:
labels:
app: influxdb-test
spec:
containers:

image: influxdb:latest
name: influxdb-test
resources:
requests:
memory: “1200Mi”
cpu: “800m”
limits:
memory: “2000Mi”
cpu: “1200m”
ports:
containerPort: 8086
name: http-influxdb
protocol: TCP
env:
name: DOCKER_INFLUXDB_DATA_DIR
value: /var/lib/influxdb2/engine/data
name: DOCKER_INFLUXDB_INIT_MODE
value: setup
name: DOCKER_INFLUXDB_INIT_USERNAME
value: intel
name: DOCKER_INFLUXDB_INIT_PASSWORD
value: “1234”
name: DOCKER_INFLUXDB_INIT_ORG
value: intel
name: DOCKER_INFLUXDB_INIT_BUCKET
value: intel-bucket
name: INFLUXDB_HTTP_LOG_ENABLED
value: “true”
name: INFLUXDB_HTTP_WRITE_TRACING
value: “true”
name: INFLUXDB_DATA_ENGINE_QUERY_LOGGING_ENABLED
value: “true”
name: INFLUXDB_DATA_QUERY_LOG_ENABLED
value: “true”
name: DOCKER_INFLUXDB_DATA_DELETE_DATA
value: “never”
name: INFLUX_TOKEN
valueFrom:
secretKeyRef:
name: influxdb-static-token-test
key: token
volumeMounts:
mountPath: /var/lib/influxdb2/engine/data
subPath: data
name: efs-influxdb-test
mountPath: /var/lib/influxdb2/engine/meta
name: efs-influxdb-test
subPath: meta
mountPath: etc/influxdb/
name: intel-influxdb-config-test
readOnly: true
name: intel-influxdb-secrets-test
mountPath: /app/influxdb
readOnly: true
envFrom:
secretRef:
name: intel-influxdb-secrets-test
configMapRef:
name: intel-influxdb-config-test
volumes:
name: intel-influxdb-secrets-test
secret:
secretName: intel-influxdb-secrets-test
name: intel-influxdb-config-test
configMap:
name: intel-influxdb-config-test
name: efs-influxdb-test
persistentVolumeClaim:
claimName: efs-influxdb-pvc-test

Service.yaml as:
apiVersion: v1
kind: Service
metadata:
labels:
app: influxdb-test
name: intel-influxdb-test
namespace: test-env
spec:
type: LoadBalancer
ports:
port: 8086
targetPort: http-influxdb
nodePort: 3****
selector:
app: influxdb-test
sessionAffinity: none
Influxdb.conf as:
reporting-disabled = true
bind-address = “<IP_ADDRESS>:8086”
storage-directory = “/var/lib/influxdb2/engine/data”
enabled = true
auth-enabled = true
log-enabled = true
write-tracing = false
pprof-enabled = false
pprof-aut-enabled = false
debug-pprof-enabled = false
ping-auth-enabled = true
wal-flush-interval = 50s
compaction-interval = 2h
write-buffer-size = 2000M
[data]
dir = “/var/lib/influxdb2/engine/data”
wal-dir = “/var/lib/influxdb2/engine/wal”
query-log-enabled = false
trace-logging-enabled = false
[meta]
dir = “/var/lib/influxdb2/engine/meta”
retention-autocreate = true
logging-enabled = true
[http]
flux-enabled = true
base-path = “/influx”
[query]
cache-enabled = true
cache-size =<PHONE_NUMBER>
Persistent Volume and Persisitent VolumeClaim as:
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
name: efs-influxdb-pvc-test
namespace: test-env
spec:
accessModes:

ReadWriteMany # Ensure it is ReadWriteMany for EFS
resources:
requests:
storage: 10Gi
storageClassName: efs-sc-test

apiVersion: v1
kind: PersistentVolume
metadata:
name: efs-influxdb-pv-test
spec:
capacity:
storage: 10Gi
accessModes:

ReadWriteMany # Change from ReadWriteOnce to ReadWriteMany
storageClassName: efs-sc-test
persistentVolumeReclaimPolicy: Retain
volumeMode: Filesystem
csi:
driver: efs.csi.aws.com
volumeHandle: fs-**** # Your EFS volume ID

and set Horizontal Pod Autoscaling with CPU and Memory threshold limit for 75%, so once it attained its limit the scaling will take place.
here the list of problems I face,

when i try to access with InfluxDB with load balancer and load increases, new pods get created but load is not distributed to all pods so only one pod takes full load and get crashed even 2/3 pods are running continuously load is not distributing, Once the scaling get started I face session expired and in development tab under application I can able to see cookies with name as “influxdb-oss-session”.

when I restart/redeploy the InfluxDB pod, the exisitng data is not showing on InfluxDB but it placed on pod volume mount path “/var/lib/influxdb2/engine/data”. so i tried verifying on other topics so they suggested to add an environment variable for influxd.bolt file path. so i added a environment variable as:

name: INFLUXD_BOLT_PATH
value: /var/lib/influxdb2/engine/data/influxd.bolt
now I can able to view data after restart/redeploy but during scaling pods went to crashloopbackoff and i checked with pod logs, it shows as:

2024-12-09T10:38:46.192426574Z info found existing boltdb file, skipping setup wrapper {“system”: “docker”, “bolt_path”: “/var/lib/influxdb2/engine/data/influxd.bolt”}
2024-12-09T10:38:46.284257866Z info found existing boltdb file, skipping setup wrapper {“system”: “docker”, “bolt_path”: “/var/lib/influxdb2/engine/data/influxd.bolt”}
ts=2024-12-09T10:38:46.422162Z lvl=info msg=“Welcome to InfluxDB” log_id=*** version=v2.7.11 commit=fbf5d4ab5e build_date=2024-12-02T17:48:15Z log_level=info
ts=2024-12-09T10:38:46.422212Z lvl=warn msg=“nats-port argument is deprecated and unused” log_id=***
Error: unable to open boltdb: timeout
See ‘influxd -h’ for help
ts=2024-12-09T10:38:47.394498Z lvl=error msg=“Failed opening bolt” log_id=*** error=“unable to open boltdb: timeout”
help me to resolve this issue.
### Tasks
- [ ] @influxdb
- [ ] kubernetes

@mrsubbiah This is not a supported use-case for InfluxDB OSS v2. InfluxDB OSS 2.x is a single-node only solution.


Update accessing-the-api links

This PR works towards #9286 with updating internal links to what they should be according to the redirects file.
Previous Link: docs/admin/accessing-the-api
New Link: docs/reference/access-authn-authz/controlling-access

/lgtm
/approve


[BULLET] Making GetContactsFromLastStepFeature optinal in Collision Features

🎉 New feature
Makes GetContactFromLastStepFeture not a requirement for Collisions in IGN-Physics. Fixes https://github.com/ignitionrobotics/ign-physics/pull/208#discussion_r594540971
Summary

Test it

Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

This is not compiling mainly because of https://github.com/ignitionrobotics/ign-gazebo/blob/f5a362cf579ef23d2cd9d5ec1cd2f856410db3c3/src/systems/physics/Physics.cc#L2202-L2203
The EntityFeatureMap::Get expects an EntityPtr with required/minimum FeatureList. In this snippet of code contact.collision1 has ContactFeatureList, which is not the required feature in EntityCollisionMap. Ideally, this should downcast cast to the entity with the required FeatureList, but there's an implicit conversion issue that causes an ambiguity in the compiler: the compiler is unable to decide whether to call EntityFeatureMap::Get(const Entity &) or EntityFeatureMap::Get(const RequiredEntityPtr &_physEntity). The temporary solution is to manually create an ShapePtr with the required features from contact.collision1 before calling EntityFeatureMap::Get.
diff --git a/src/systems/physics/Physics.cc b/src/systems/physics/Physics.cc
index 9b419b8c..5ce52614 100644
--- a/src/systems/physics/Physics.cc
+++ b/src/systems/physics/Physics.cc
@@ -313,3 +313,3 @@ class ignition::gazebo::systems::PhysicsPrivate
   public: using ShapePtrType = ignition::physics::ShapePtr<
-            ignition::physics::FeaturePolicy3d, ContactFeatureList>;
+            ignition::physics::FeaturePolicy3d, CollisionFeatureList>;

@@ -432,2 +432,3 @@ class ignition::gazebo::systems::PhysicsPrivate
             CollisionFeatureList,
+            ContactFeatureList,
             CollisionMaskFeatureList,
@@ -2201,5 +2202,6 @@ void PhysicsPrivate::UpdateCollisions(EntityComponentManager &_ecm)
     const auto &contact = contactComposite.Get<WorldShapeType::ContactPoint>();
-    auto coll1Entity = this->entityCollisionMap.Get(contact.collision1);
-    auto coll2Entity = this->entityCollisionMap.Get(contact.collision2);
-
+    auto coll1Entity =
+        this->entityCollisionMap.Get(ShapePtrType(contact.collision1));
+    auto coll2Entity =
+        this->entityCollisionMap.Get(ShapePtrType(contact.collision2));

I'll continue to think about a way to fix the implicit conversion problem, but I don't think it's needed for this PR.


Update init.py

get link  title
The user no longer needs to resubmit to get the title

Can you give an example of what it does?

Can you give an example of what it does?

Returns a tuple that contains the link to the page in addition to its title.


Extract the major opcode (and maybe more things) from an error.

As far as I can tell, most (all?) X11 errors come with a major_opcode. Unfortunately, if I have a x11rb::protocol::Error, it is difficult (AFAICT) to access it (because I need a giant match). Could a major_opcode method be exposed on x11rb::protocol::Error, just like error_code and wire_sequence_number?
My use-case for this is that if a particular extension causes an error, I want to stop using that extension and fall back to a more general method.

Been there, tried that gave up. Core protocol errors seem to contain that information, but extensions do not (either that or the XML descriptions are wrong and the wire format has this information, but the XML pretends otherwise).
Since I do not fully trust my own code here, here is some example from /usr/include/xcb/randr.h:
typedef struct xcb_randr_bad_output_error_t {
    uint8_t  response_type;
    uint8_t  error_code;
    uint16_t sequence;
} xcb_randr_bad_output_error_t;

Ah, finally found the right function:
https://gitlab.freedesktop.org/xorg/xserver/-/blob/d6558477d7a264c2132bc977b51d80fc0277d1e0/dix/dispatch.c#L3740-3753
void
SendErrorToClient(ClientPtr client, unsigned majorCode, unsigned minorCode,
                  XID resId, int errorCode)
{
    xError rep = {
        .type = X_Error,
        .errorCode = errorCode,
        .resourceID = resId,
        .minorCode = minorCode,
        .majorCode = majorCode
    };
    WriteEventsToClient(client, 1, (xEvent *) &rep);
}

WriteEventsToClient adds the sequence number field to the event, does some stuff (calls internal hooks, logging stuff etc) and then writes the bytes to the client.
Thus, all events have these fields and this is a bug in xcb-proto. I just reported this finding as https://gitlab.freedesktop.org/xorg/proto/xcbproto/-/issues/16.
The plan of action here is to fix the XML description, update x11rb to a new version of xcb-proto, and then add the relevant functions to x11rb::protocol::Error.

My use-case for this is that if a particular extension causes an error, I want to stop using that extension and fall back to a more general method.

A work around could be to instead match sequence numbers to map between errors and the request that causes them. This approach is best matched with using wait_for_event_with_sequence() instead of wait_for_event(), because that gives you the full 64 bit sequence number and not only the 16 bits that appear on the wire.

Another idea would be to get raw events (wait_for_raw_event() basically gives you a Vec<u8>) and then parse out the fields you need yourself. It's a bit ugly, but it would work. The major code is data[10], if I counted bytes correctly (looking at the definition of xcb_generic_error_t in /usr/include/xcb/xcb.h.
You can then use conn.parse_event() to get the event in a "nice" form.

Thanks for looking into this, and for the workarounds! I think I'll punt on this for now (as you know, we have some lower-hanging fruit to address...), but I've got a link to this issue in a comment in case we revisit it later.


Unable to run, undefined method `include?' for nil:NilClass (NoMethodError)

Describe the bug
I am unable to produce a changelog at all with the command github_changelog_generator -u repo-namespace -p project-name --max-issues 50, and get the below output
Found 36 tags
Fetching tags dates: 36/36
Sorting tags...
Received issues: 100
Pull Request count: 554
Filtered pull requests: 59
Filtered issues: 35
Fetching events for issues and PR: 94
Fetching closed dates for issues: 94/94
Traceback (most recent call last):
7: from /Users/emilyjablonski/.rvm/rubies/ruby-2.6.3/lib/ruby/gems/2.6.0/gems/async-1.29.0/lib/async/task.rb:263:in block in make_fiber' 	6: from /Users/emilyjablonski/.rvm/rubies/ruby-2.6.3/lib/ruby/gems/2.6.0/gems/github_changelog_generator-1.16.1/lib/github_changelog_generator/generator/generator.rb:55:in block in compound_changelog'
5: from /Users/emilyjablonski/.rvm/rubies/ruby-2.6.3/lib/ruby/gems/2.6.0/gems/github_changelog_generator-1.16.1/lib/github_changelog_generator/generator/generator.rb:152:in fetch_issues_and_pr' 	4: from /Users/emilyjablonski/.rvm/rubies/ruby-2.6.3/lib/ruby/gems/2.6.0/gems/github_changelog_generator-1.16.1/lib/github_changelog_generator/generator/generator_fetcher.rb:49:in add_first_occurring_tag_to_prs'
3: from /Users/emilyjablonski/.rvm/rubies/ruby-2.6.3/lib/ruby/gems/2.6.0/gems/github_changelog_generator-1.16.1/lib/github_changelog_generator/generator/generator_fetcher.rb:105:in associate_release_branch_prs' 	2: from /Users/emilyjablonski/.rvm/rubies/ruby-2.6.3/lib/ruby/gems/2.6.0/gems/github_changelog_generator-1.16.1/lib/github_changelog_generator/generator/generator_fetcher.rb:105:in reject'
1: from /Users/emilyjablonski/.rvm/rubies/ruby-2.6.3/lib/ruby/gems/2.6.0/gems/github_changelog_generator-1.16.1/lib/github_changelog_generator/generator/generator_fetcher.rb:107:in block in associate_release_branch_prs' /Users/emilyjablonski/.rvm/rubies/ruby-2.6.3/lib/ruby/gems/2.6.0/gems/github_changelog_generator-1.16.1/lib/github_changelog_generator/generator/generator_fetcher.rb:201:in sha_in_release_branch': undefined method `include?' for nil:NilClass (NoMethodError)
Desktop (please complete the following information):

OS: MacOS Catalina 10.15.7

My only thought is that this is a Ruby version issue but I just re-installed Ruby?

It's a defect with this software. Does it happen with the most current code (unreleased)?

I believe so, I just installed it yesterday with gem install github_changelog_generator

def sha_in_release_branch(sha)       branch = @options[:release_branch] || @fetcher.default_branch       shas_in_branch = @fetcher.commits_in_branch(branch)       shas_in_branch.include?(sha) end
Here is the specific function for me that appears to be failing, as if commits is null?

@emilyjablonski Ah, oh, I meant the other way around, if you were to use the unreleased code, can you reproduce the issue?
I think there are mentions of the gem specific_install in this bug tracker. That's one way of test-running unreleased code.

I tried it with specific_install and it throws same error :/

Also note that it has just issues with some specific PRs I guess, because on some repos 1.16.1 works and on some it does not.

@dusan-dragon Hi! I made a change, after reading the code, and merged it. Could you re-run your check, using specific_install, to see if it had any effect?

@olleolleolle I can confirm that with newest master the issue is gone. Looking forward for new release 🎉

@dusan-dragon Appreciate the support. v1.16.2 is on RubyGems.org now.


predict.py issue

hello，when I predict the camvid dataset,I have this issue:
Traceback (most recent call last):
File "C:/Users/lenovo/PycharmProjects/fr/gleason/DABNet-master/predict.py", line 112, in
test_model(args)
File "C:/Users/lenovo/PycharmProjects/fr/gleason/DABNet-master/predict.py", line 83, in test_model
predict(args, testLoader, model)
File "C:/Users/lenovo/PycharmProjects/fr/gleason/DABNet-master/predict.py", line 25, in predict
for i, (input, size, name) in enumerate(test_loader):
ValueError: too many values to unpack (expected 3)

If you want to predict the Camvid dataset, use the test.py, predict.py is used to output the prediction for official evaluation like Cityscapes dataset


Identity Server 4 (2.0) not reading Asp.Net Core Identity cookies

Re-posted from stack overflow  i am kind of desperate as i have been trying to get this to work for several days now.   I have been though all the tutorials and sample projects i can find none of them use 2.0 with Identity users.
I am trying to use Asp .Net Identity Core with Identity Server 4.   I can see in the logs (Ids) that the user is logged in properly.

info: Xena.IdentityServer.Controllers.AccountController[0]
User logged in.

My login controller then sends the user over to my Manage controller.
[Route("[controller]/[action]")]
[Authorize]
//[Authorize(AuthenticationSchemes = "Identity.Application")]
public class ManageController : Controller
{

    [HttpGet]
    public async Task<IActionResult> Index(ManageMessageId? message = null)
    {
     .....
    }
 }

The User never arrives as the login is then lost for some reason.
info: Microsoft.AspNetCore.Mvc.RedirectToActionResult[2]
      Executing RedirectResult, redirecting to /Manage/Index.
info: Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker[2]
      Executed action Xena.IdentityServer.Controllers.AccountController.Login (Xena.IdentityServer) in 3493.6102ms
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
      Request finished in 3515.9158ms 302
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
      Request starting HTTP/1.1 GET http://localhost:5000/Manage/Index
info: Microsoft.AspNetCore.Authorization.DefaultAuthorizationService[2]
      Authorization failed for user: (null).
info: Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker[3]
      Authorization failed for the request at filter 'Microsoft.AspNetCore.Mvc.Authorization.AuthorizeFilter'.
info: Microsoft.AspNetCore.Mvc.ChallengeResult[2]
      Executing ChallengeResult with authentication schemes ().
info: Microsoft.AspNetCore.Authentication.JwtBearer.JwtBearerHandler[12]
      AuthenticationScheme: Bearer was challenged.
info: Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker[2]
      Executed action Xena.IdentityServer.Controllers.ManageController.Index (Xena.IdentityServer) in 46.2285ms
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
      Request finished in 58.6793ms 401

I think part of the clue is this line  Authorization failed for user: (null).  I can see that the cookie is in the browser. Its just not being read.
From what I understand Identity server 4 has its own cookies and Asp .Net core Identity has its and they need to be reading the same cookie.  I have tried following Using ASP.NET Core Identity but it hasn't helped.
Startup
 //Adds Asp.net identity
        services.AddDbContext<ApplicationDbContext>(builder =>
            builder.UseSqlServer(Configuration.GetConnectionString("XenaIdentityConnection")));

        // Configuer Asp.net Identity
        services.AddIdentity<ApplicationUser, IdentityRole<long>>(config =>
            {
                config.Password.RequireDigit = true;
                config.Password.RequireLowercase = true;
                config.Password.RequireNonAlphanumeric = false;
                config.Password.RequireUppercase = true;
                config.Password.RequiredLength = 8;
                config.User.RequireUniqueEmail = true;
                config.SignIn.RequireConfirmedEmail = true;
                config.SignIn.RequireConfirmedPhoneNumber = false;
            })
            .AddEntityFrameworkStores<ApplicationDbContext>()
            .AddSignInManager<ApplicationSignInManager>()    // Adds custom SignIn manager.
            .AddDefaultTokenProviders();

   //https://identityserver4.readthedocs.io/en/release/topics/signin.html
        services.AddAuthentication(options =>
            {
                options.DefaultScheme = IdentityConstants.ApplicationScheme;
            })
            .AddGoogle("Google", options =>
            {
                options.AccessType = "offline";
                options.SignInScheme = IdentityServerConstants.ExternalCookieAuthenticationScheme;
                options.ClientId = "xxxxx-jvu30c2n19thoqimd97b4jk1r2poh17p.apps.googleusercontent.com";
                options.ClientSecret = "g29nXgVoFZBIBNS-hJJxPWXW";
            }).AddOpenIdConnect(OpenIdConnectDefaults.AuthenticationScheme, "OpenID Connect", options =>
            {
                options.SaveTokens = true;
                options.SignInScheme = IdentityServerConstants.ExternalCookieAuthenticationScheme;
                options.SignOutScheme = IdentityServerConstants.SignoutScheme;
                options.RequireHttpsMetadata = settingsSetup.RequireHttpsMetadata;
                options.Authority = settingsSetup.Authority;
                options.ClientId = "testclient";
                options.Scope.Add("testapi");
                options.ResponseType = OpenIdConnectResponseType.IdTokenToken;
                options.TokenValidationParameters = new TokenValidationParameters
                {
                    NameClaimType = "name",
                    RoleClaimType = "role"
                };
            });

Adding the following does fix my issue with manage/index.  However it doesn't work because then the open Id connect login wont work because that uses internal controllers within Identity Server for authentication that I cant / don't want to overload.  I need to figure out how to get Identity Server 4 to use the cookie set by Aspnet identity or visa versa.

//[Authorize(AuthenticationSchemes = "Identity.Application")]

This solution came from a previous question i asked on Stack I have opened a new one because i am leaning towards this being a setup issue with Identity server rather than an issue with Identity cookies

This should exactly cover your use-case - give it a try and compare
https://github.com/IdentityServer/IdentityServer4.Templates/tree/dev/src/IdentityServer4AspNetIdentity

adding this

services.AddAuthentication()
.AddGoogle(options =>
{
options.SignInScheme = IdentityServerConstants.ExternalCookieAuthenticationScheme;
                options.ClientId = "xxxx-9m4dkjb5hscn7cjrn5u0r4tbgkbj1fko.apps.googleusercontent.com";
                options.ClientSecret = "wdfPY6t8H8cecgjlxud__4Gh";
            });

breaks google login.   It only works if i set it to this

IdentityConstants.ExternalScheme

Which kind of makes is what makes me thing somethings wrong with how Identty server 4 is loading my cookies its not using IdentityConstants.ExternalScheme its using its own internal ones ( IdentityServerConstants.ExternalCookieAuthenticationScheme) which dont work.
Is there to force it to use IdentityConstants instead of IdentityServerConstants?

try this change
https://github.com/IdentityServer/IdentityServer4.Templates/pull/23/files

Ok let me try.  Question could it be my custom signinManager?

await SignInAsync(user, true, IdentityConstants.ApplicationScheme);

That looks a bit suspicious to me

don't know.

The cookie its creating is called .aspnetcore.identity.application

I updated the template - thats how it is supposed to work with ASP.NET Identity
https://github.com/IdentityServer/IdentityServer4.Templates/tree/2.2.0/src/IdentityServer4AspNetIdentity

I finally figured out the problem this morning.   Part of the problem was due to the fact that I have a custom signin manager which uses the IdentityConstant cookies.
services.AddAuthentication(options =>
                {
                    options.DefaultAuthenticateScheme = IdentityConstants.ApplicationScheme;
                    options.DefaultChallengeScheme = IdentityConstants.ApplicationScheme;
                })

Both DefaultAuthenticateScheme and DefaultChallengeScheme need to be set.   Then everything works as it should.


mac m1 电脑 docker-connector.conf 文件路径是 /opt/homebrew/etc

$ docker network ls --filter driver=bridge --format "{{.ID}}" | xargs docker network inspect --format "route {{range .IPAM.Config}}{{.Subnet}}{{end}}" >> /usr/local/etc/docker-connector.conf 这条命令是不是考虑区分一下？

$ docker network ls --filter driver=bridge --format "{{.ID}}" | xargs docker network inspect --format "route {{range .IPAM.Config}}{{.Subnet}}{{end}}" >> /usr/local/etc/docker-connector.conf 这条命令是不是考虑区分一下？

是个好主意。不过，在安装的成功的时候会提示配置文件的路径以及以上操作命令，因此准备文档中会注释以安装成功之后的提示信息为主，避免由于homebrew的位置调整而引起文档不准确

he is correct. .. althought I read only english.. /usr/local is not always /usr/local .. for me its /opt/homebrew.. but there is this variable you can use:
HOMEBREW_PREFIX
(environment variable)

he is correct. .. althought I read only english.. /usr/local is not always /usr/local .. for me its /opt/homebrew.. but there is this variable you can use:
HOMEBREW_PREFIX
(environment variable)

Yes, it's not always /usr/local. I use the variable HOMEBREW_PREFIX in installer script docker-connector.rb, and it works fine. But, I cannot use the variable HOMEBREW_PREFIX in my iTerm, because of the variable is undefined.
So, I think $(brew --prefix) may be the best answer, it will print the default prefix of homebrew.


The name 'GeoPoint' is defined in the libraries

The name 'GeoPoint' is defined in the libraries 'package:cloud_firestore_platform_interface/src/geo_point.dart' and 'package:flutter_osm_interface/src/types/geo_point.dart'.

how do i solve this issues

found a solution
i imported: import 'package:flutter_osm_plugin/flutter_osm_plugin.dart' as ph(or any word);
so whenever i use Geopoint i use it like ph.Geopoint

you can use keyword  as when you import flutter_osm_plugin for example
 import 'package:flutter_osm_plugin/flutter_osm_plugin.dart'  as osm


Deploy.ps1 fails during UI Build due to UnicodeEncodeError

Describe the bug
Running deploy.ps1 fails during UI build phase, specifically during vite install, due to handling of error received during vite installation when it returns a checkmark (character '\u2713') in its result. Running the commands manually work.
To Reproduce
Steps to reproduce the behavior:

Follow deployment instructions for "Full" deployment with PrivateACR and custom $ResourceNames defined. https://azure.github.io/ipam/#/deployment/README
See error:
RUN npm run build  ---> Running in
| cf61f8120ed6  ><EMAIL_ADDRESS>build > vite build  vite v4.4.9 building for production... transforming...
| ERROR: The command failed with an unexpected error. Here is the traceback: ERROR: 'charmap' codec can't encode
| character '\u2713' in position 0: character maps to  Traceback (most recent call last):   File
| "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 233, in invoke   File
| "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line
| 663, in execute   File
| "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line
| 726, in _run_jobs_serially   File
| "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line
| 697, in _run_job   File
| "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line
| 333, in call   File
| "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler   File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/acr/build.py", line 143, in acr_build   File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/acr/_stream_utils.py", line 58, in stream_logs   File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/acr/_stream_utils.py", line 143, in _stream_logs   File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\colorama/ansitowin32.py", line 41, in write   File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\colorama/ansitowin32.py", line 162, in write   File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\colorama/ansitowin32.py", line 187, in write_and_convert   File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\colorama/ansitowin32.py", line 195, in write_plain_text   File "encodings\cp1252.py", line 19, in encode UnicodeEncodeError: 'charmap' codec can't encode character '\u2713' in position 0: character maps to  To check existing issues, please visit: https://github.com/Azure/azure-cli/issues

Expected behavior
Expect deploy.ps1 to complete without error and ipam applications to be running in app service.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: [e.g. iOS] Windows 11, latest versions of prerequisites installed: PowerShell 7.3.7, Bicep 0.21.1, Az CLI 2.53.0, python 3.11.5
Browser [e.g. chrome, safari] N/A
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Good morning @daveinci, I'm working on reproducing this now and will circle back with you shortly once I have some answers on the best way to address this. Thank you so much for bringing this to my attention!

Hey @daveinci, the root cause for this stems from how the Vite build process will emit an Unicode "checkmark" character, and then that being captured by PowerShell and redirected to a variable. Long story short, this is an issue with how this is handled in Windows systems, but there is something you can try as a workaround for now.
Take a look at the fix in this link, which shows how to enable UTF-8 support in Windows:
https://github.com/microsoft/knack/pull/178
I would suggest enabling that for now and seeing if that unblocks you, and in the meantime I will continue working on a more permanent/universal fix for this issue.
Please let me know if the above helps!

Thanks @DCMattyG. I worked around by running the 'az acr build' commands manually. Enabling Control Panel -> Region -> Administrative -> Change system locale -> Beta: Use Unicode UTF-8 for worldwide language support would likely do the trick but appreciate you looking for the permanent/universal fix. Think that will help users.

Hi @daveinci, this issue has been addressed in PR #181. For now, I've changed the output of the build to "warn" which will stop the Unicode output seen at the "info" level. I've tested four many hours trying to find a way to alleviate this via scripting, but it appears unless you enable UTF-8 support in Windows (as mentioned in my previous comment), nothing I've found through script changes has been able to alleviate this issue.
I'll keep looking for a better workaround, but now this issue should be resolved. Please let me know if you have any further issues and we'll get right on them. Thanks again!


Implement IHittable for BarPlot

Feature Suggestion
First, thanks for making such amazing library. I love it.
Second, we are currently implementing charts in our business accounting application. They look amazing. For scatter plots we use the method described in Highlight the Data Point Near the Cursor to display details about every transaction in the side panel. It works like a charm.

The problem arises with bar plots. Since there is no MouseOver method, or any way to detect collisions with the figure:

We would like to display information when the user hovers over one bar.
It would be pretty easy to detect mouse over one square, but the problem is that there are so may variables that controls the positioning of each bar that it becomes a bit difficult. Also, all the position calculation is done in the Draw method, so there is no way to retrieve the current position and size of each bar.
Feature description:
My suggestion would be to implement the IHitable interface in the BarPlot class. I think it would be pretty easy. The same that is done for the Tooltip can be done:

Inside the draw method store in a private variable the position and size of each bar.
In the IHittable.HitTest() method, use that information from the last render to perform a point to AABB collision test.

it would be cool to have an overload of the HitTest method that returns the index of the hit bar too. Maybe as a separate interface, like IHittablePlot
interface IHittablePlot : IHittable
{
    bool HitTest(Coordinate coord, out int hitElementIndex);
}

Code example:

First, thanks for making such amazing library. I love it.

Sure thing! I'm happy to hear you enjoy using it ❤🚀✨

We would like to display information when the user hovers over one bar.

This seems like a common use case. I'm interested in exploring options to make this possible!

but the problem is that there are so may variables that controls the positioning of each bar that it becomes a bit difficult

This is definitely a challenge.
Bar positions are mostly defined by variables in BarPlotBase.cs:

Vertical Corrdinates: Values, ValueBase, and ValueOffsets
Horizontal Coordinates: Positions, PositionOffset, BarWidth
Rotation: Orientation

It was not the original intent to expose this complexity to the user. The goal was to have a simple API where the user defines groups and values, and these positions are handled internally, but as features were added this design became unwieldy. This complexity can't be refactored away without breaking code of existing users.
For these reasons every once and a while I consider making an extremely simple bar plot type where the user defines bar position/size/color/label, giving them total control of positioning and style. The most recent idea was raised in #1749 but I closed it to focus on #1888.
I'll take a brief brake on #1888 and see if I can get a minimal case BarSeries plot type off the ground that implements IHittable. I'll follow-up in this issue as I make progress

Hi @jhm-ciberman,
I made some good progress tonight in #1892 but I can tell I won't finish it today and I'm out of town this weekend so it may be a few more days until this is done.
If you review the code changes I made though, you may get some hints about how to implement this yourself without much effort using the existing bar plot (not this new BarSeries I'm working on). This is because I found I can get a great effect without having to actually implement IHittable. Instead I just perform an action OnMouseMove that gets the position of the mouse (in coordinate units) and determines which bar is under that coordinate.
https://github.com/ScottPlot/ScottPlot/blob/a021144b71636901b9106d1ceb91febcf6ee330b/src/ScottPlot4/ScottPlot.Sandbox/WinFormsApp/Form1.cs#L14-L79

I'll post on this issue when I make progress on this next 👍

Oh, I see. That new api is fantastic. <3 There is no hurry. I still need to implement two or tree extra charts. It would be amazing to have this api on the next release.

Well, I ended up implementing a helper class to detect collisions and get the nearest bar against a bunch of BarPlots:
https://gist.github.com/jhm-ciberman/0ee3a2d8e2616f0c87615eebfff29ec5
You can check it here. Maybe is helpful for other people.

It works perfect:

Great work! Your solution looks like it's working great, thanks for sharing your code and for the helpful animation too!
I'll keep working on #1892 to bring this functionality into the main library, but this is moving in a great direction and it looks like you already have something working well for your application. Thanks again for the original suggestion!

Following-up, I'm glad your BarCollider solution is working for you! I probably won't add this complexity into the existing BarPlot class because (1) it's more to maintain, (2) the solution you linked to gets the job done, (3) horizontal bars are not supported, (4) the new BarSeries plot type I'm merging in #1892 has collision detection built in. This discussion led to the creation of BarSeries which I think is a useful plot type, and I'm excited to merge this in.
Thanks again for your input!
Scott

@swharden Yes sure! I just posted my code so it can be useful for someone else. But I did't imagine you would finish adding the feature so quickly.
Thanks for everything. At the moment I will use my solution for this chart, but I still need to add a lot of more charts to the application so I'm pretty sure I will use the new ´BarSeries´.
Thank you!
Edit: I saw you already made a release, amazing! Thanks!


ci: Split build step for realm-server

This PR attempts to speed up CI for boxel. Changes include:

Created a reusable workflow for waypoint build step
Created a reusable workflow for waypoint deploy step
Run realm-server build step simultaneously along tests
Update wait-service-stable script to speed up AWS target group polling logic
Test pnpm caching in waypoint steps
Remove deprecated actions and workflows for realm-server

I think if you merge upstream master (or rebase) into your PR the percy failure will go away


Fixed import error after setup.py install

setup.py could not detect some packeges because package init files were missing. Fixed by adding those files.

Let me know if the new install instructions in README.md work for you. It should not be necessary to add __init__.py as we are using py3 namespace packages.


Simple way to add STRING, but with no update time

I would like to add STRING (https://string-db.org/) version, using its own API. But it doesn't show the date of last update, just version and URL. Is it ok? It's possible to extract version/date here: https://string-db.org/cgi/access?footer_active_subpage=archive, but using API is much simpler.

Hi @aracnus, that's a really good point! I pinged the maintainers on twitter to see if they'd amend the API (https://twitter.com/cthoyt/status/1360570698851352577). In the mean time I will add this DB for you


znapzend: added

Things done:

[X] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
Built on platform(s)

[X] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

More
Testing on nixos-15.09 + this patch
Using it right now, seems to work.

Please note, that points are not mandatory, but rather desired.

Would you please rename the commit subject to conform with the guidelines?

Where can I find guidelines about commit messages?

@otwieracz in CONTRIBUTING.md.

https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md and http://hydra.nixos.org/build/33498973/download/1/nixpkgs/manual.html#idm140737318510400

I see that this also adds a module; could you put the package and service in separate commits?

Only separate comments or also PR?

Only separate commits; it's for the benefit of people reading the history later on (or doing a bisect or whatever).

Hm. Why this is incorrect? (see faild build)

By default, attrsets are non-recursive, and the value of one attribute cannot refer to another attribute in the set. The rec keyword is like doing let self = { key1 = "val1"; key2 = self.key1; }; in self.

One final nitpick! I think this is good to merge, besides that. I trust you have tested the service?

Well - seems to work!
$ sudo znapzendztatz
USED    LAST SNAPSHOT       DATASET
3.66G   2016-03-24-200000   odyssey/ROOT
4.26G   2016-03-24-200000   root@pioneer.otwiera.cz:spool/backup/odyssey/ROOT
 903K   2016-03-24-200000   odyssey/etc
4.52M   2016-03-24-200000   root@pioneer.otwiera.cz:spool/backup/odyssey/etc
35.0M   2016-03-24-200000   odyssey/etc/nixos
69.0M   2016-03-24-200000   root@pioneer.otwiera.cz:spool/backup/odyssey/etc/nixos
2.32G   2016-03-24-200000   odyssey/home
2.41G   2016-03-24-200000   root@pioneer.otwiera.cz:spool/backup/odyssey/home
2.05G   2016-03-24-200000   odyssey/var
2.04G   2016-03-24-200000   root@pioneer.otwiera.cz:spool/backup/odyssey/var

Great work, will merge once travis completes