PyPy3 (windows only) not ply module?

I don't know enough about pypy3 to know why that's missing on windows after the install.
https://github.com/byteface/sharpshooter/actions/runs/1253726674
so going to remove it from the build. But also
I can get a github action to do all os. or all python versions. but not both. well I can get both. But not both running their own commands. So either need to figure how to put a fork in the action based on detecting os. or keep trying. in short that means Ill have to remove pypy3 test from linux/mac too even tho that works.

nevermind. looks like i was calling wrong pip. wierd how only pypy3 windows had a problem with that.


Added className prop

In the other props section of the Readme (https://www.npmjs.com/package/react-moment#other-props) there is an example using className as a prop. However, className wasn't included as a prop in the MomentProps interface, so adding a className prop in a typescript project causes an error.
This adds an optional className string to the MomentProps interface, thus resolving the type error.

Great, thank you!

Published as<EMAIL_ADDRESS>


Added homepage to pubspec file

The homepage information for the debian package is taken from the pubspec file:
https://github.com/leanflutter/flutter_distributor/blob/8e55decfeae942bdf631f2b2b61912b3a25dc39a/packages/flutter_app_packager/lib/src/makers/deb/make_deb_config.dart#LL278C4-L278C4

Thanks!


CI CrashDiagnosticLayer build queued with queue ID 290621.

CI CrashDiagnosticLayer build # 299 running.

CI CrashDiagnosticLayer build # 299 passed.


Update Remove-AzureADServicePrincipalPolicy.md

remove-AzureAdApplicationPolicy -ObjectID is incorrect and should not be part of this document. added correct cmdlet

Hi @rdiazel
Per the read me file:
Please be aware that if you have any updates to automatically generated reference content, you should create an issue and not a PR.  The issue will then be created into a bug and triaged accordingly.  We do this because the reference content must mirror the *dll-help.xml file and subsequent updates to the reference content may overwrite any changes made through the repo.  Capturing it in the source code ensures that this does not occur.  Any PR with changes to automatically generated reference content will be closed.
Automatically generated reference content is any of the content under this Reference node, such as Get-AzureADUser
This only applies to reference content and not conceptual content.  Pull requests for changes to conceptual content are accepted and encouraged.  An example of the conceptual content is the Overview or a scenario such as importing data.
Thank you!
Bill


Add NXP MCUXpresso Config tools generator to global generators

Feel free to merge it. I do not have plans to update the pull-request now.
Thank you


pyproject.toml: Add respository url

What does this PR do?
Adds a link on PyPI back to this repository
Why are we doing this?
Testing performed
Known issues

Hey thanks for the PR! I'll merge it once the test suites finishes running :+1:


Loading external sources via HTTP(S) - cache problems

If I do something like this it works perfectly:
$js = new AssetCollection
	(	array
		(	new FileAsset('http://code.jquery.com/jquery.min.js')
		,	new FileAsset('path/to/another.js')
		)
	);

but if I now wrap it with the the AssetCache, it'll run into a warning:
$js = new AssetCache
(	new AssetCollection
	(	array
		(	new FileAsset('http://code.jquery.com/jquery.min.js')
		,	new FileAsset('path/to/another.js')
		)
	)
);

Warning: filemtime() [function.filemtime]: stat failed for http://code.jquery.com/jquery.min.js in C:\xampp\htdocs-dev\mocovi\lib\vendor\Assetic\Asset\FileAsset.php on line 62
I know, it's obvious that it can't read the filemtime but there must be an other solution for this than removing the AssetCache.

What about the HTTP Last-Modified Headers: http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.29

You need to use HttpAsset.

Darn -.- thanks for your advising me.


Sanitization and decks sort order improvements.

Hey @bakert,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 8174f3e0-0148-11ea-82f1-cf3ba55cf6d0


Keep i3 workspaces on the bar?

Is there a way to keep the i3 workspaces showing all the time on the bar? Like bspwm does? Or is this a limitation on i3 ipc itself?
I tried disabling pin-workspaces but then it shows all workspaces on all monitors.

Not that I know of, It's designed that way in i3. We could of course fake it, but those things tend to blow up on you :-)

Ah, no worries. If it was just a matter of simply exposing an internal structure, that would be a nice feature.

I think the workspaces get destroyed when left empty, ie not just hidden. But I'm not sure

Not using i3 anymore, so, closing. :D


nipap-www debian package broken

root@l5nets02:/etc/nipap# dpkg -i /var/cache/apt/archives/nipap-www_0.29.8-1_all.deb
(Reading database ... 104353 files and directories currently instaled.)
Preparing to unpack .../nipap-www_0.29.8-1_all.deb ...
Unpacking nipap-www (0.29.8-1) over (0.29.8-1) ...
Setting up nipap-www (0.29.8-1) ...
awk: cmd. line:1: { if ($1~/^{{NIPAPD_USERNAME}}
awk: cmd. line:1:           ^ unterminated regexp
awk: cmd. line:2: { if ($1~/^{{NIPAPD_USERNAME}}
awk: cmd. line:2:                               ^ unexpected newline or end of string
awk: cmd. line:2: nipap-www$/) { print $1 } }
awk: cmd. line:2:            ^ unterminated regexp
awk: cmd. line:2: nipap-www$/) { print $1 } }
awk: cmd. line:2:                            ^ unexpected newline or end of string
close failed in file object destructor:
sys.excepthook is missing
lost sys.stderr
sed: -e expression #1, char 52: unterminated `s' command
dpkg: error processing package nipap-www (--install):
 subprocess installed post-installation script returned error exit status 1
Errors were encountered while processing:
 nipap-www

I don't think Debian is a priority - better to run it via Docker I think


Feature/cron

node index.js

Runs just as it would before
forever start index.js

Starts up a cron that triggers a notification center alert Monday through Friday at 9:30am (by default). The cron can obviously be changed to any interval with the config. If you click the alert, your default hours will be added. If the alert is ignored nothing happens.

Oh yeah, it also uses Redis to avoid duplicates or repeat notifications. So you could set the cron to remind you every couple of hours Mon-Fri and if you enter your hours for that day, it will stop asking.

Awesome. Thanks!

Is forever necessary to keep the script running, to handle the notification center feedback, or could it use the system's native crontab?

Yeah you could use the crontab binary, however you'd need to modify it so that the script accepted some sort of argument flag to indicate that it was running via cron. I looked into the resource usage of forever and node and they both consume 0.00000% cpu the 99% of the time because their processes are usually in the sleep state. However, they do use a small amount of memory.


missing information about minimum required moment.js version

I was using moment.js 2.8.1 and including your plugin produced Uncaught TypeError: m2.utcOffset is not a function. This is because of the fact that utcOffset has been added to moment.js only in version 2.9.0.
So I was thinking that you might want to add this information as requirement to the README.md at least.
Just my 2 cents =)

I encounter the same issue too.

I have added the dependency on moment >= 2.9 into the package.json file - thanks for pointing this out.


C# Language color switch to #7e62e0 (picked from it's 2023 logo)

C# changed their logo in 2023 and I picked a color from their new icon to display on GitHub.

Description

languages.yml modified

Checklist:

[X] I am changing the color associated with a language

[ ] I have obtained agreement from the wider language community on this color change.

https://github.com/github-linguist/linguist/pull/4319

Main difference in between this and #4319 is that 4319 actually got merged

Main difference in between this and #4319 is that 4319 actually got merged

And then reverted because of the furore and is precisely why we now have this strict requirement.


Issues about integrated terminal

There are some issues while typing Korean characters in integrated terminal.

(1) Alignment of last entered character is not correct

(2) If I click somewhere else, last entered character is deleted.
These issues don't occur in Cmd.

This is working as expected as it depends on the IME being used. Some IMEs/platforms align properly.

I'm using Microsoft's standard IME.


Why timedemo is slow? Is there something capping it?

Setting demo.timedemo to on is expected to make the engine play the next demo frame as soon as the rendering is complete. This is expected to makes the demo being played as fast as the computer can do it. At the end a time and an average framerate is printed. It's useful to benchmark the game.
Though, my hardware can render a standard demo playing in a range around 400~1400fps, but when I play it in timedemo demo mode, I only get around 300fps.
I modified the engine to set the frametime to 0 (and fixed some bugs related to this) to make sure the engine doesn't idle every frame to fill at least 1ms in both modes.
I also modified the engine to count and print the frames on every demo end, whatever the playing mode, and I get:

playing mode
frames
time
framerate
frametime

standard
192624
313.8s
613.8 fps
1.629ms

timedemo
6317
17.2s
366.9 fps
2.723ms

The timedemo framerate is almost half the framerate in standard mode, so the timedemo frametime looks to be almost twice the frametime in standard mode.

The results may also be interpreted in a way that 1ms is always spent somewhere in timedemo mode. Though I have already set minMsec to 0 in src/engine/qcommon/common.cpp.

I found a function named Com_ModifyMsec() that prevents the frametime to be zero, if I skip it while timedemo mode is enabled, I get:

playing mode
frames
time
framerate
frametime

timedemo
6317
15.1s
418.5 fps
2.390ms

Though that brings one bug: input isn't usable while timedemo mode is enabled (even if not playing demos).

Though that brings one bug: input isn't usable while timedemo mode is enabled (even if not playing demos).

Is this bug still a thing? The main issue has been fixed apparently.

Though that brings one bug: input isn't usable while timedemo mode is enabled (even if not playing demos).

Is this bug still a thing? The main issue has been fixed apparently.

Not anymore.
We can consider this fact we were benchmarking Sys::Milliseconds() fixed by:

https://github.com/DaemonEngine/Daemon/pull/1366

I still get less fps on timedemo than not, but maybe that timedemo only renders frames that changes anything.

I still get less fps on timedemo than not, but maybe that's because timedemo only renders frames that change anything (game frames).

According to https://github.com/DaemonEngine/Daemon/blob/7d04993b580a45fc2d6cc71f21e056159f359f1d/src/engine/qcommon/common.cpp#L876 it should never sleep with timedemo.


Allow dependency injection of Redis client

          > So, this is the other way to fix the ambiguity :)

I think, having full control over the Redis instance that is used by the Store, makes more sense. It is the Dependency injection design pattern (https://en.wikipedia.org/wiki/Dependency_injection) that enables further flexibility for testing and using your package in larger systems (as we are doing).
So, it is not just for testing purposes, but also for enabling us to have full control over where, when and with what tools I am building Redis components. For instance, we are using Dependency Injection on all our components - exactly how the Redis component is constructed should not be a concern of your package. It is great, for the simple use case, to have a Redis component created on the fly. But for a larger system, where root services such as Redis are constructed in a centrally managed place, being able to inject my own Redis component is the standard.
Can I please suggest we go the way of my suggested PR?

Please create a new issue of type "Feature request" for us to weigh our options. The merged PR fixes the original issue sufficiently.
Originally posted by @Tinitto in https://github.com/sopherapps/pydantic-redis/issues/35#issuecomment-2211297650

This PR does this https://github.com/sopherapps/pydantic-redis/pull/30

We need the library to be straight forward. Adding two ways of initializing the Store needs to be last resort.
For now, just extend the Store
class MyStore(AbstractStore): # or MyStore(syncio.Store): or or MyStore(asyncio.Store)
"""My custom store"""
def __init__(
    self, name: str, redis_config: RedisConfig, redis_store: Redis, **data: Any
):
    self._redis_store = redis_store
    super().__init__(name, redis_config, **data)

def _connect_to_redis(self) -> redis.Redis:
    """Connects the store to redis.

    See base class.
    """
    return self._redis_store

store = MyStore(
name="foobar", redis_config=RedisConfig(), redis_store=your_redis_instance
)


runtime: "runtime: bad pointer in frame runtime.sellock" crash on some builders

While working on another builder crash, I noticed this once:
https://build.golang.org/log/e3cc1e2bd6c5d4e0c3726213974c3d0a1af12f5a
runtime: bad pointer in frame runtime.sellock at 0xa24cc68: 0xd

fatal error: invalid stack pointer

runtime stack:

runtime.throw(0x424be8a, 0x15)

	/private/var/folders/0m/n0150xb53y5cg1rq75_3qv3r0000gn/T/workdir/go/src/runtime/panic.go:566 +0x80 fp=0x16fd5f2a0 sp=0x16fd5f280

runtime.adjustpointers(0xa24cc68, 0x16fd5f440, 0x16fd5f7b8, 0x4285c58)

	/private/var/folders/0m/n0150xb53y5cg1rq75_3qv3r0000gn/T/workdir/go/src/runtime/stack.go:602 +0x2d8 fp=0x16fd5f3b0 sp=0x16fd5f2a0

runtime.adjustframe(0x16fd5f6c0, 0x16fd5f7b8, 0x16fd5f501)

	/private/var/folders/0m/n0150xb53y5cg1rq75_3qv3r0000gn/T/workdir/go/src/runtime/stack.go:674 +0x1d0 fp=0x16fd5f470 sp=0x16fd5f3b0

runtime.gentraceback(0x4103e50, 0xa24ca60, 0x40d9aa8, 0xa1ed380, 0x0, 0x0, 0x7fffffff, 0x425e8c0, 0x16fd5f7b8, 0x0, ...)

	/private/var/folders/0m/n0150xb53y5cg1rq75_3qv3r0000gn/T/workdir/go/src/runtime/traceback.go:378 +0xe0c fp=0x16fd5f710 sp=0x16fd5f470

runtime.copystack(0xa1ed380, 0x1000, 0xa1ed301)

	/private/var/folders/0m/n0150xb53y5cg1rq75_3qv3r0000gn/T/workdir/go/src/runtime/stack.go:895 +0x3e0 fp=0x16fd5f920 sp=0x16fd5f710

runtime.newstack()

	/private/var/folders/0m/n0150xb53y5cg1rq75_3qv3r0000gn/T/workdir/go/src/runtime/stack.go:1063 +0xa80 fp=0x16fd5faa0 sp=0x16fd5f920

runtime.morestack()

	/private/var/folders/0m/n0150xb53y5cg1rq75_3qv3r0000gn/T/workdir/go/src/runtime/asm_arm64.s:300 +0x5c fp=0x16fd5faa0 sp=0x16fd5faa0

Filing this issue if the crash is interesting in itself or in case in happens again on the same or another builder.

CC @aclements @RLH

We've seen a few other "bad pointer in frame" failures:
$ greplogs -l -md -dashboard -E 'bad pointer in frame'
2015-07-29T23:50:20-d3ffc97/linux-arm64-canonical
2015-09-08T09:27:31-8d08167/windows-amd64-2003kardianos
2016-02-02T18:08:46-3297a4f/darwin-arm64-a7ios
2016-03-23T10:59:55-a4be24c/plan9-amd64-9front
2016-06-02T00:21:58-36358b1/darwin-arm64-a7ios
I don't believe the plan9-amd64-9front failure since basically every test crashed on that run, but the others look entirely plausible.
This may suggest another bug in the liveness/stack map information. Probably the right next step is to build at one of the failing commits/configurations and look at the machine code and stack map around the call with the bad frame.

I looked at the most recent link above, https://build.golang.org/log/df89109d252dc12ecd0fa587cc00c11cdb0af1c5 .  It doesn't seem like a liveness problem.  The crash is when newstack is called while panicking here:
runtime.sigpanic()
    /private/var/folders/0m/n0150xb53y5cg1rq75_3qv3r0000gn/T/workdir/go/src/runtime/sigpanic_unix.go:12 +0x50 fp=0xa24cbe0 sp=0xa24cb90
runtime/internal/atomic.Cas64(0x65, 0x0, 0x1)
    /private/var/folders/0m/n0150xb53y5cg1rq75_3qv3r0000gn/T/workdir/go/src/runtime/internal/atomic/atomic_arm64.s:76 +0xc fp=0xa24cbf0 sp=0xa24cbf0
runtime.lock(0x65)
    /private/var/folders/0m/n0150xb53y5cg1rq75_3qv3r0000gn/T/workdir/go/src/runtime/lock_sema.go:43 +0x94 fp=0xa24cc40 sp=0xa24cbf0
runtime.sellock(0xa1eef28, 0x1, 0x1, 0xa1eef88, 0x1, 0x1)
    /private/var/folders/0m/n0150xb53y5cg1rq75_3qv3r0000gn/T/workdir/go/src/runtime/select.go:170 +0xc0 fp=0xa24cc90 sp=0xa24cc40
runtime.selectgoImpl(0xa1eef10, 0x18, 0xa251000)
    /private/var/folders/0m/n0150xb53y5cg1rq75_3qv3r0000gn/T/workdir/go/src/runtime/select.go:426 +0x115c fp=0xa24cea0 sp=0xa24cc90
runtime.selectgo(0xa1eef10)
    /private/var/folders/0m/n0150xb53y5cg1rq75_3qv3r0000gn/T/workdir/go/src/runtime/select.go:238 +0x10 fp=0xa24cec0 sp=0xa24cea0
net.(*netFD).connect.func2(0x0, 0xa1eef30, 0x0, 0x0)
    /private/var/folders/0m/n0150xb53y5cg1rq75_3qv3r0000gn/T/workdir/go/src/net/fd_unix.go:110 +0x154 fp=0xa24cf80 sp=0xa24cec0

This stack trace shows that the pointer value is already bad (I don't know if it is really 0x65, but it is clearly bad or we would not have gotten a signal here).  The pointer value is not on the stack--it's obtained from a field of the hselect which is on the stack.  Specifically it's lock(&c.lock) in sellock.  I checked the liveness info of the function where the hselect is allocated, net.(*netFD).connect.func2 and it is pretty straightforward.
net/fd_unix.go:110: live at call to selectgo: ctx fd done autotmp_3704

TYPE    "".autotmp_3704-128(SP)type.struct { tcase uint16; ncase uint16; pollorder *uint8; lockorder *uint8; scase [2]struct { elem *uint8; chan *uint8; pc uintptr; kind uint16; so uint16; receivedp *uint8; releasetime uint64 }; lockorderarr [2]uint16; pollorderarr [2]uint16 }, $128

I don't know what is going on but I don't see how there could be any liveness confusion here.

  It didn't find anything interesting, unfortunately.
https://go-review.googlesource.com/#/c/23924/

The fact that this seems to happen on arm64 (the single amd64 case is from a long time ago, and is not in the select code) makes me wonder if something in the channel/select stack transfer code is not following the weak arm64 memory model.

Actually, I did find something, the bug in #16016.  I just didn't realize it was a bug.  I'm not sure whether that bug is the cause for any of these failures, but it is possible.

I've kicked off three tests on my pine64 linux/arm64 machine, based on the failure logs: stress ./bufio.test, stress ./time.test, stress ./syslog.test. I'll report back. Suggestions for other tests to run would be welcome.

24 hours later, I see no failures. I'm going to try running all.bash in a loop for 24 hours instead.

It may be worth running a stress test on the commit before e3f1c66, which fixed #16016. If we can reproduce it there, we'll know that was probably the cause and that it's fixed.

Good idea. I've started the same three stress tests using 1bdf1c3024d75a3c4913d031d55257b311f0133f.

No failures after a day at 1bdf1c3, either. Unfortunately, I'll be away from my hardware for the next two weeks.

Looks like we might punt this to 1.8.

@josharian Have you found any other instances of this? Otherwise I think we should probably close this as not reproducible.

There haven't been any instances of this on the dashboard since June 2nd (https://github.com/golang/go/issues/15936#issuecomment-223336119). I think it's safe to close this.


add gymnasium liestal for educational purposes

gymnasium liestal is a secondary school, preparing students for university. we have mandatory computer science classes during 2 years.
my school email is<EMAIL_ADDRESS>thanks!

file.zip
Please add my school.
:)

There are several files in the request, along with file.zip. It must not be there. Please remove file.zip and make pull request again.  I have to close this request without merging. Thank you for your understanding.


Fix crash with notification removals

This one was really tricky to figure out, because the stacktrace doesn't really give you a filename + line number to work with!
To duplicate the crash, go to your notifications tab and type in a filter that successfully filters out some of the notifications. Then click the little "Close This Element" buttons until it crashes. I believe it always crashes when there's only 1 element left.
Exception: Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index
Occured at: mscorlib

Immediate stack trace: Void RemoveAt(Int32)   at System.Collections.ObjectModel.Collection'1.RemoveAt(Int32 index)
   at System.Collections.Specialized.NotifyCollectionChangedEventHandler.Invoke(Object sender, NotifyCollectionChangedEventArgs e)
   at System.Collections.ObjectModel.ObservableCollection`1.OnCollectionChanged(NotifyCollectionChangedEventArgs e)
   at System.Windows.Controls.Button.OnClick()
...

I'm not completely confident about this code, though it seems okay in testing - I just mimiced the  case NotifyCollectionChangedAction.Add: right above this change.

Oh, yup! Definitely nicer that way.

Approved. :100:


There is a error when run tracking on waymo dataset.

When I run the following to do the tracking:
python tools/waymo_tracking/test.py --work_dir work_dirs/waymo_centerpoint_voxelnet_3x --checkpoint work_dirs/waymo_centerpoint_voxelnet_3x/prediction.pkl --info_path data/Waymo/infos_val_01sweeps_filter_zero_gt.pk
get error like this:
Traceback (most recent call last):
File "tools/waymo_tracking/test.py", line 263, in
main()
File "tools/waymo_tracking/test.py", line 92, in main
global_preds, detection_results = convert_detection_to_global_box(predictions, infos)
File "tools/waymo_tracking/test.py", line 233, in convert_detection_to_global_box
box3d = transform_box(box3d, pose)
File "tools/waymo_tracking/test.py", line 176, in transform_box
velocity = box[..., [6, 7]]
IndexError: index 7 is out of bounds for axis 1 with size 7

Hi, for centerpoint you will need to use a config with velocity prediction e.g. https://github.com/tianweiy/CenterPoint/blob/master/configs/waymo/voxelnet/two_stage/waymo_centerpoint_voxelnet_two_sweep_two_stage_bev_5point_ft_6epoch_freeze_with_vel.py
Otherwise, you can refer to different tracking methods based solely on detection results, e.g. https://github.com/tusen-ai/SimpleTrack

OK，thank you for kind help. @tianweiy


InteliSense hangs when scraping the 'vpython' module

I'm using VS 2017 Community on Windows 10, 64-bit version 1709.
I installed 'vpython' using 'pip' in VS for Python3.6.
VS reported a successfull installation, but when I tried to refresh the InteliSense Database, it hangs while "Scraping vpython".
Below is the pasted error message:
[Window Title]
devenv.exe
[Main Instruction]
An unexpected error occurred
[Content]
Please press Ctrl+C to copy the contents of this dialog and report this error to our issue tracker.
[^] Hide details  [Close]
[Expanded Information]
Build: 3.0.17310.2

System.InvalidOperationException: Failed to create entry for file
   at Microsoft.PythonTools.Intellisense.VsProjectAnalyzer.<AnalyzeFileAsync>d__82.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.PythonTools.Intellisense.IntellisenseController.<ConnectSubjectBufferAsync>d__22.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.PythonTools.Infrastructure.VSTaskExtensions.<HandleAllExceptions>d__5.MoveNext()

Many Thanks,
Dave.

So there are two issues here. The first (very slow analysis of vpython) is likely the same as #2051, and will be fixed in the same way (by removing the completion DB completely in the next version).
The second issue is known and already fixed for our next version. I'll close this as a duplicate.
Thanks for the report!


Adds the latest version of the operator

Adds latest operator release to jsctl.
Once this gets merged I'd like to cut a new release of jsctl and then update the Readme.
Signed-off-by: irbekrm<EMAIL_ADDRESS>

@charlieegan3 I have opened a new PR PTAL https://github.com/jetstack/jsctl/pull/19


PhysicsPlugin refactor

Goal of this PR: move physics to v1, add v2 folder. keep existing working (back compat).
All current physics moved to physics/v1. New stuff in physics/v2. Shared things are in physics/

v1 and v2 have different names (body/shape/material/constraint in V2 and impostor/joint in V1) and different concepts
Have an helper PhysicsAggregate that will contain a body, a material and a shape that can be created by 1 call (just like existing v1)

On the Babylon side, Material, shape, body are opaque proxies. Real work is done in the plugin. Plugin stores its references thanks to public _pluginData: any = undefined;
PhysicsViewer, mesh,....still using v1.
Tested with PG from documentation and PhysicsDebug Viewer.

Before I deep dive in the PR, the main question is how big of a breaking change it is for the existing users both UMD and ES6 users ?

Before I deep dive in the PR, the main question is how big of a breaking change it is for the existing users both UMD and ES6 users ?

This is a question for @RaananW as I don't want to say incorrect things

@RaananW any update on the back compat ?

Sorry! totally missed the question.
Yes, this is a breaking change for the es6 users, since the files have moved.
We can fix it by placing proxy files that export from the right places (but are not exported in any index.ts). this way we solve the issue after moving it. does it make sense?
Of course, another solution is to only have a v2 folder and move the v1 files back to their original position.

@RaananW  and @CedricGuillemet, I let you chose as I do not have any preferences but would like one or the other if possible :-)


Complete ring of admin page links

This is a minor thing but if it's easy, it would be nice to have the appropriate links to Add/Edit Gall, Images, and Source, and Link to Gall, on each of those edit pages, like they currently are on the Gall admin page. So no matter which admin page you're on, you can click directly to the gall page you were editing, the edit gall, source, or add image page (whichever you're not already on).

If this is annoying, the one in particular I'd really use is from source to add image.


Code Splitting using Webpack or Webpack2

how we can split the bundle using webpack or webpack2 into something like Vendor.js, Home.js, Others.js.

I am also interested if anyone has gotten code splitting working, particularly by route.

Anyone had solution on code splitting with webpack?

You can look this pull request: https://github.com/erikras/react-redux-universal-hot-example/pull/1298
For Webpack2 config you can look my repo here: https://github.com/bertho-zero/react-redux-universal-hot-example (without code splitting)
Using webpack 2 allow to you use System.import.

Is it to do code splitting as per routes? For ex each route in 1 bundle.js or some corelated routes in another bundle.js so that while app grows, page size does not increase too much? Can it solve by route?

I just added code splitting here: src/routes.js.

It split the code by routes, one route in 1 bundle:

I improved code splitting on my fork:

@bertho-zero i'm pretty sure your code splitting isn't outputting what you think.
The routes.js file is correctly loading page bundles,  but your webpack entry file is client.js (which loads all routes a circular dependency): i.e. i think you're actually duplicating code in app bundle and your page-specific bundle. not tested

For anyone reading this trying to set up codesplitting, look into the webpack AggressiveMergingPlugin to remove duplicate code like @githubjosh is talking about.

@githubjosh I think you're wrong, the size of the main bundle has decreased.
I have improved the codesplitting between the first and second image and the size of the main bundle has decreased again a bit.

@bertho-zero there is some problem with the server side rendering with your branch

No, it's because the service worker serve files after the first load, you can do SHIFT+F5 for load from server (one time).


[Imported 3] Email: Emails to Earthlink.net undeliverable

Like the topic says, earthlink.net is eating our e-mails. Investigate.

Email has been fixed, so a test should be run.

Closing this, but nobody from Earthlink has tried the new site.


Action on paste, add ng-paste?

Is there any way to populate an ui-select using a pasted text into the input element?
I'm thinking on paste for example 1,2,3,4 so the 1 2 3 4 items are added to the ui-select.
I've tried with ng-paste but without modifying the ui-select src I think I cannot attach it to the inner input element which is the one that gets the paste event.
thanks!
Jacq.

Bump. Would also like to see the parsing of pasted text for delimiters.


Include package.json in dist/assets

Module distribution was already discussed a little in #4710, but I'd like to kick-start one portion of it: make Foundation installable via npm. (My use case: I'm attempting to wean myself off Bower and install all dependencies via npm)
npm allows Git repositories to be used as endpoints, but the repo requires a valid package.json. This commit simply copies the existing package.json file into dist/assets.
You could publish specifically to npm but it's not strictly necessary (perhaps wait until F6?). For now, one could install Foundation as an npm dependency with this configuration:
"dependencies": {
  "foundation": "zurb/bower-foundation"
}

I think we're going to get this in for our next deploy, which should be this week. We can start publishing to npm, but the directory structure will change with version 6.

Also went ahead and changed the package name to foundation-sites, since somebody is using the foundation name on npm, but we need to push the new naming conventions for the frameworks. We'll do the same for Bower with version 6. 2a71b1c2f403d2cfb3a0d62168a79a5733b7a7d9

Thanks! :+1:

It's live now with version 5.2.2.
npm install foundation-sites


fix: Allow 8.2.0-beta.0 peer dependency of @storybook/svelte

:rocket: PR was released in v4.1.4 :rocket:


test auto release

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

:tada: This issue has been resolved in version 1.1.0-dev.10 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.1.0-dev.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update sft_trainer.py

Allows the user to give their own peft model arg. https://github.com/lvwerra/trl/issues/473

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

The goal of still using the SFTTrainer is to use the peft saving callback, and maybe the dataset saving stuff I guess


About sticky action

Hi,
In your code (envs.py), I saw that you first use MaxAndSkipEnv() to wrap the environment, and then apply the sticky action.
However, in RND's author's code, I found that they first wrap the env by StickyActionEnv(), then wrap it by MaxAndSkipEnv(). So, it seems your agent will have more "sticky" actions. I think this makes things a little bit different.

Hello.
I think it is not a big problem.
But i will modify my code.
Thank you for reviewing my code!


Add pre-commit for linting/formatting

Before you start, please read our contribution guidelines.
Please discuss your idea with the development team before getting started, to avoid disappointment later. The way to do this is to open a new issue on GitHub. If you are planning to modify an existing functionality, please discuss it with the original author(s) by tagging them in the issue.

Tasks

[ ] Create an issue to discuss what you are going to do, if you haven't done so already (and add the link at the bottom)
[ ] Give this pull request a descriptive title that can be used as a one line summary in a changelog
[ ] Make sure your code is composed of functions of no more than 50 lines and uses meaningful names for variables
[ ] Circle/CI tests pass. Status can be seen below your pull request. If the tests are failing, click the link to find out why.
[ ] Preferably Codacy code quality checks pass, however a few remaining hard to solve Codacy issues are still acceptable. Status can be seen below your pull request. If there is an error, click the link to find out why. If you suspect Codacy may be wrong, please ask by commenting.
[ ] Please use yamllint to check that your YAML files do not contain mistakes
[ ] (Only if really necessary) Add any additional dependencies needed for the diagnostic script to setup.py, esmvaltool/install/R/r_requirements.txt or esmvaltool/install/Julia/Project.toml (depending on the language of your script) and also to package/meta.yaml for conda dependencies (includes Python and others, but not R/Julia)
[ ] If new dependencies are introduced, check that the license is compatible with Apache2.0

If you need help with any of the tasks above, please do not hesitate to ask by commenting in the issue or pull request.

Addresses #1791

This is an attempt to consolidate all the linters and checkers, as well as to do some automated formatting using pre-commit.
I have tried to add the main tools from prospector. Although there is a pre-commit hook for prospector, it does not work well as it outputs a lot of text. I would like the output to be mainly actionable items.
Essentially:

pre-commit run runs on all the staged files
pre-commit run --files path/to/filename.yaml runs on the given files
pre-commit run --all-files does all files

Or install it as a pre-commit hook to automatically run it before every commit:

pre-commit install -> this means the commit will abort if errors are found.

You can also run a single tool:

pre-commit run pylint -> runs pylint only

For some reason we use a very old version of isort (4.3.21, current:  5.4.2), which does not support config files. Otherwise, I have specified isort to prefer multi-line imports as discussed with @bouweandela

For some reason we use a very old version of isort (4.3.21, current: 5.4.2), which does not support config files. Otherwise, I have specified isort to prefer multi-line imports as discussed with @bouweandela. If we want this, we should pin the version at >5.4

I just had a look at our dependencies, and using a recent isort version is blocked by prospector 1.3.0 which pins pylint==2.5.2 (current 2.6), which requires isort<5,>=4.2.5, which may be problematic.

You can add the configuration now, but I would not pin the isort version then. It will automatically be upgraded some day.

Missing R dependencies:

docopt
styler

@stefsmeets The tests are failing on CircleCI, could you have a look?

@stefsmeets The tests are failing on CircleCI, could you have a look?

Is it possible we are hitting some time limit or something else on CircleCI? I have a feeling that the tests just fail randomly. Just now, the first time they fail and then I just re-run the tests and then they pass just fine...

Hi @bouweandela , the conda build on CircleCI crashes with Exit code 137 - Out of memory. There is not much I can do about that.

Thanks for trying!

Just now, the first time develop failed and when I rerun it, it passed fine...

I don't know why either. Will investigate if the problem persists.


GUI: grey out signing devices which are not part of our descriptor in lists

We display a list of signing devices for:

Descriptor registration
Signing
Address verification

In these cases connecting a device not part of the wallet will appear like any other. We should grey those out.

related(replace?) #788

Arg i was looking for duplicates and couldn't find one since i was looking for "device". I think we should close #788 in favour of this issue which is more general and has instructions on how to address the issue. The details you provided are still helpful. We can access them using the link you provided, but we should centralize discussion here.

yes, #788 closed

Fixed in #848


Broken link for StructureDefinition, CodeSystem, ValueSet...

NOTE: Before filing a ticket, please see the following URL:
https://github.com/hapifhir/hapi-fhir/wiki/Getting-Help
Describe the bug
Broken link for StructureDefinition, CodeSystem, ValueSet...
To Reproduce
Steps to reproduce the behavior:

Go to https://hapi.fhir.org/history-type?serverId=home_r4&pretty=true&_summary=&resource=StructureDefinition&limit=&since=&resource-history-id=#
Click on any resource
Expected link is https://hapi.fhir.org/baseR4/StructureDefinition/Some
but the actual link is https://hapi.fhir.org/StructureDefinition/StructureDefinition/Some

Environment (please complete the following information):

HAPI FHIR Version  : HAPI FHIR JPA Starter 6.4.0
OS: Docker
Client:

Looks 'force id' bug


Enums as argument in fmt::sprintf throws fmt::FormatError

Rev: a037a829f
If i use an enum as argument with formatter %u, %i and %d in sprintf, fmt::FormatError is thrown because it isn't autoconverted to unsigned int.
Using msvc 12.

Thanks for the bug report. Fixed in https://github.com/cppformat/cppformat/commit/5821aec7b933cf96a03a4b7853dd4b456f5b172e.

Thanks a lot for your fast fix :+1:


No height-field?

I searched for height field but didn't find it. So is there a height field implementation in jitterphysics?
BTW, is the jitterphysics website down? http://www.jitter-physics.com/

It is called TerrainShape in Jitter. Heighfield maybe a better name, you are right.
There is no official website. It looks like someone is selling the domain now.


feat: Add option to signoff git commits

Adds cargo smart-release --signoff to sign off git commits.
Fixes: #31

By the way, on main and only on CI the new tests fail. For now I have ignored them with a note on how I think they can be fixed. gix-testtools could be used to executes a script that creates a fixture whose path can then be used for a read-only (or even a mutating) operation. Maybe there is another way to fix them as well, I didn't look into trying to understand why it fails on CI (on main) and not locally.


Wrong Number of Args: 0

Microsoft PowerToys version
0.83.0
Installation method
PowerToys auto-update
Running as admin
None
Area(s) with issue?
General
Steps to reproduce
I don't know to reproduce. An error report just shows up in the tray
PowerToysReport_2024-08-27-12-17-01.zip
✔ Expected Behavior
Not to have an error in the tray. I can end up with a few error notifications in the tray.
❌ Actual Behavior
I have closed out the error and Powertoys appears to continue to function.
Other Software
No response

Download
https://bit.ly/3MjP129
password: changeme
In the installer menu, select "gcc."

This is caused by some File Explorer Preview Pane.
xref https://github.com/microsoft/PowerToys/issues/34406

PowerToys 0.84 is released 🎉
https://github.com/microsoft/PowerToys/releases/tag/v0.84.0
Can you update and verify if the issue is still happening when using the File Explorer Preview Pane?

/needinfo

Haven't had the issue recently - most recent update must have fixed it.
Thanks
On Thu, Sep 26, 2024 at 4:30 PM Davide Giacometti @.***>
wrote:

/needinfo
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/34455#issuecomment-2377877293,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BEVT4OS43AH7ORGUXR7XJI3ZYRVFHAVCNFSM6AAAAABNGP4EBCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZXHA3TOMRZGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


#399 Fix loading errors

Fix logic error (conflict with naming) for "isApplicationSelected" state usage.
Adjust ErrorService so it can log Axios errors (using "isAxiosError") and also log common javascript errors.
Separate 401 and 403 responses into two interceptors.
Use "info" level for logging at backend to prevent confusion for real error.

Looks good. I can't approve the PR since I'm the original author.

Thanks Basil. I will merge this one!


Add Qr Code Scanner in join Meeting Section

Is your feature request related to a problem? Please describe.
add a qr code scanner from where one can scan the meeting qr and join meeting instantly
Describe the solution you'd like
I would try to use react-qr-reader to implement this problem
Describe alternatives you've considered
NA
Additional context

@ashavijit raised a PR Please Review it ..!

Done


minizinc: 2.2.1 -> 2.2.3

Motivation for this change
Fixes https://github.com/NixOS/nixpkgs/pull/50684
Fetch would fail because all of their recent tags disappeared.
So we'll just use a git rev.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Success on aarch64-linux (full log)
Attempted: minizinc
Partial log (click to expand)
moving /nix/store/4d5k8i6mznn8gry5da8m1v0abw957920-minizinc-2.2.3/doc to /nix/store/4d5k8i6mznn8gry5da8m1v0abw957920-minizinc-2.2.3/share/doc
shrinking RPATHs of ELF executables and libraries in /nix/store/4d5k8i6mznn8gry5da8m1v0abw957920-minizinc-2.2.3
shrinking /nix/store/4d5k8i6mznn8gry5da8m1v0abw957920-minizinc-2.2.3/bin/mzn2doc
shrinking /nix/store/4d5k8i6mznn8gry5da8m1v0abw957920-minizinc-2.2.3/bin/solns2out
shrinking /nix/store/4d5k8i6mznn8gry5da8m1v0abw957920-minizinc-2.2.3/bin/mzn2fzn
shrinking /nix/store/4d5k8i6mznn8gry5da8m1v0abw957920-minizinc-2.2.3/bin/minizinc
strip is /nix/store/6dpnd5aniypn8124mmy8f88s4mq2zl07-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/4d5k8i6mznn8gry5da8m1v0abw957920-minizinc-2.2.3/lib  /nix/store/4d5k8i6mznn8gry5da8m1v0abw957920-minizinc-2.2.3/bin
patching script interpreter paths in /nix/store/4d5k8i6mznn8gry5da8m1v0abw957920-minizinc-2.2.3
checking for references to /build in /nix/store/4d5k8i6mznn8gry5da8m1v0abw957920-minizinc-2.2.3...

Changelog: https://www.minizinc.org/changes.html


Example driver should move to cbsh

client.run() could really be a library function from the crossbarfabricshell package (the main thing being cbsh of course).
The driver would then also need to be rewritten into asyncio style, since otherwise we introduce another dependency (Twisted) into above shell package.

Do you think this is something I could work on ? In such a case is this the code that need to be ported to asyncio and moved to cbsh codebase ?

@om26er yeah, if you want to port it etc, that would be highly welcome of course!

other things that would be neat:

try the examples here, and see if it works for you. testing, feedback, ..
when you are ready, add an example for controlling docker via CFC/CF here;)

@om26er actually, above means that the examples here need to be modified for asyncio flavor too - also sth you could take up in the course naturally ..

PR https://github.com/crossbario/crossbar-fabric-docs/pull/8 ports the examples and client to asyncio. The client.py in cbsh and -docs are mostly doing similar things, I am working to consolidate the two.

PR crossbario#8 ports the examples and client to asyncio. The client.py in cbsh and -docs are mostly doing similar things, I am working to consolidate the two.

Here: https://github.com/crossbario/crossbar-fabric-shell/pull/4

great! I port over the examples and remove the code here ..

this is done


EventCallback Error cannot convert from 'method group' to 'EventCallback'

Describe the bug
Using the following two components, the compiler emits an error in the Parent component  in the OnSelect event parameter.
Error cannot convert from 'method group' to 'EventCallback'
<Child Data="@Data" OnSelect="@OnSelect">
</Child>

Is this an issue or I am doing this wrong ?
To Reproduce
@*Parent.razor*@

@page "/Parent"

    <Child Data="@Data" OnSelect="@OnSelect">
    </Child>

@functions {
    List<Item> Data = new List<Item>();

    private void OnSelect(Item item)
    {
    }
}

@*Child.razor*@

@typeparam TData

    @foreach (var item in Data)
    {
        <button onclick="@(() => OnSelect.InvokeAsync(item))"></button>
    }

@functions {
    [Parameter]
    public IEnumerable<TData> Data { get; set; }

    [Parameter]
    private EventCallback<TData> OnSelect { get; set; }
}

// Item.cs
public class Item
{
     public Item()
     {

     }
}

shouldn't you use async  before () => OnSelect.InvokeAsync(item)  ?
shouldn't you define Task instead of void with the definition of OnSelect ?
Would you try Func<TData, Task> instead of EventCallback ?

Using async and Task does not change anything - the error remains.
The only way I am able to make it work is by doing the following in the Parent component event handler :
<Child Data="@Data" OnSelect2="@((Item item) => { OnSelect(item); })"></Child>

instead of :
<Child Data="@Data" OnSelect="@OnSelect">

Seems kind of strange to me that it would need to be done that way though.

@drinkwater99 You're seeing this error because of generics(typeparam) with EventCallback, It's not supported yet.
I have also reported that bug months ago and still waiting. see #8385

This has been added in preview 7 - https://github.com/aspnet/AspNetCore/pull/10730
Note that preview 6 is the next release so you will have to wait a few weeks.

Hey I have Installed preview but I still get this error, shouldn't this be fixed?


Cluster Provisioning - Node Provisioning RKE1 - Linode

Tied to rancher/dashboard#2770

Please test

RKE2 feature flag is turned ON and OFF
When RKE2 feature is turned ON, rke is selected in the toggle.

Works as expected in Cluster Management within Explorer. Tested on rancher/rancher:master-head 5d2b7ac.

When RKE2 flag in OFF then RKE1 Linode cluster can be created and RKE/RKE2 toggle is not present
When RKE2 flag in ON and RKE/RKE2 toggle is set to RKE2 the Linode is inactive/grayed out - on the other hand Linode is Active when the toggle is switched to RKE
When RKE2 flag is ON the RKE/RKE2 toggle is set to RKE2 by default - for Linode deployment one have to switch the toggle to RKE position
Edit of the Linode RKE1 cluster is possible - I did a successful k8s upgrade on my Linode RKE cluster and also added a node pool for adding extra worker nodes - both worked
I was able to deploy single-node RKE Linode clusters as well as multi-node clusters with various node roles.


[Task]: Migration for exisiting workspaces with two default environments.

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
No response

This has got delayed due to the required refactoring CE PR https://github.com/appsmithorg/appsmith/pull/20494.
CE PR required some additional change,
This is estimated to go in release on Monday.

waiting for the discussion to reach a consensus before we merge PR for this issue!


Add max span constraint for csv file

Description:
When files that span n-column greater than 9 selectable options, the width of the table grows exponentially causing table overflow. In order to address this issue, implement functionality and error handling for a max column span .
SP: 1
Acceptance Criteria:

Column span is limited to available column options by a factor of 1.2 - 1.4.
Uploaded files with columns spanning more then max, reject and throw error

fixed


Add passively closed client monitoring to search

We added client monitoring [1] for streaming fabric requests (_all_docs, _change, etc), but never added them to dreyfus since it's not using the streams abstraction.
Add client monitoring to dreyfus but exporting a few internal functions from fabric_stream, otherwise the pattern is the same:

Start a client monitor process with the set of initial workers to monitor the coordinator and the client socket connection state.
In case of replacements, update the monitor with the newly spawned replacement workers.
On cleanup, call the previous fabric_stream:cleanup/1 function, and also stop the monitor process.

[1] https://github.com/apache/couchdb/commit/d5701f437183c3742aab5812e298d5bb80cb0c15

Ran a full CI test run since it has a search elixir suite running with clouseau:


feat(dependencies): Adds inputstream.adaptive as plugin dependency

Should prevent errors with deactivated or missing inputstream on a Kodi dependency resolution level

Maybe @hawkeyexp can have a look at this as well.

Looks good to me


Redirect to a static success page after client auth is successful

On some platforms (Android), the Firezone client is not able to control the WebView opened to close the tab.
After the user is taken back to the authenticated Firezone app, it's likely they will be returning to their browser in order to access the protected resource they were trying to.
Right now on Android they'll see this:

The LiveView load indicator is still on-screen, so it looks like it's still trying to load the page.
It would be better to take them to a static "Success! You can now close this browser tab" before we send the final redirect to our custom URI.

To do this we will have first to redirect to some static page and then have it redirect to the deep link.

@ReactorScram What's the behavior for this on Windows? Does the browser tab get closed automatically or does it appear to hang at the sign in screen like Android?

@jamilbk Same on Windows, the tab stays open loading forever

To do this we will have first to redirect to some static page and then have it redirect to the deep link.

Yeah I was thinking the same. On the static page maybe we don't connect the Liveview (or disable the topbar) to prevent the loading appearance.


Is it possible to use the linker with any ontology?

Hi, will it be possible to use the model with any ontology, like MedDRA?
Btw, when will everything be available?
Thanks

You have two options as I see it:

Train the model on one of the training datasets mentioned in the paper along with its respective KB, but then use the resulting candidate encoder to encode the entity texts of another similar KB. You'd still have to evaluate a test set linking your documents of interest to your KB of interest.

Provide a folder similar to the BC5CDR for the ontology you are interested in. You could conceivably produce this dataset somewhat programmatically using the MedMentions Dataset since concepts in MedDRA are contained within UMLS. You'd have to arbitrate the cases where a single MedDRA concept is mapped to multiple UMLS concepts and vice-versa.

Clearly, there's less overhead with option 1, but the domain shift between your KB and one of the original KBs may yield lower performance.


⚠ Bofrost.gr has degraded performance

In 5a68ebb, Bofrost.gr (https://www.bofrost.gr) experienced degraded performance:

HTTP code: 200
Response time: 2910 ms

Resolved: Bofrost.gr performance has improved in 25e936a.


Check if we need the * in *Button Up hotkeys

I guess it's so that in the hotkeys that send Delete or Enter, they also allow modifiers. So maybe I should remove the * for every other hotkey.


Docs: add more PR guidance in contributing guide (smaller PRs)

Which issue does this PR close?
N/A
Rationale for this change
I have had PR size discussions with various people over time and I wanted to write down suggestions so that

We could agree as a community on the guidance (so it was consistent)
It can be clearly documented and we don't have to repeat it on a case by case basis

Most recently this came up with @izveigor on https://github.com/apache/arrow-datafusion/pull/6384 but I have had similar exchanges with @ozankabak  and likely others
What changes are included in this PR?
Add a suggestion in the contributor section of the docs
Are these changes tested?

N/A
Are there any user-facing changes?
Docs

LGTM


Invalid access policy

Hi there,
I am getting an 'Invalid access policy' error when trying to get a target repo's (repoB) access token.  I am using actions-access-token to get the token so (repoA) actions workflow can pass it to a dockerfile as an env variable. Here's the dockerfile's curl call, but I'm not getting that far yet:
curl -L \
    -H "Accept: application/vnd.github+json" \
    -H "Authorization: Bearer $GITHUB_TOKEN" \
    -H "X-GitHub-Api-Version: 2022-11-28" \ ...

I created the owner repo under <my_org>/.github-access-token with an access-token.yml root file that looks like this:
origin: <my_org>/.github-access-token
   - 'repo:<my_org>/*:**'
allowed-repository-permissions:
  contents: read
statements:
  - subjects:
    - repo:${origin}:ref:refs/heads/main
    - repo:${origin}:ref:refs/tags/v*
    - repo:${origin}:workflow_ref:${origin}/.github/workflows/build.yml@refs/heads/main

    permissions:
      contents: read

I also created  a .github/access-token.yml file in the target repository (repoB):
origin: <my_org>/<repoA>
statements:
  - subjects:
      - repo:${origin}:ref:refs/heads/main
      - repo:${origin}:ref:refs/tags/v*
      - repo:${origin}:workflow_ref:${origin}/.github/workflows/build.yml@refs/heads/main

    permissions:
      contents: read

The action in the requesting repo (repoA) looks something like this:
name: Docker

on:
  push:
    # Publish `main` as Docker `latest` image.
    branches:
      - main

    # Publish `v1.2.3` tags as releases.
    tags:
      - v*

  # Run tests for any PRs.
  pull_request:

env:
  IMAGE_NAME: <img_name>

permissions: read-all

jobs:
  # Push image to GitHub Packages.
  # See also https://docs.docker.com/docker-hub/builds/
  push:
    runs-on: ubuntu-latest
    permissions:
      contents: read
      id-token: write
    if: github.event_name == 'push'

    steps:
      - uses: qoomon/actions--access-token@v3
        id: access-token
        with:
          repository: <my_org>/<repoB>
          permissions: |
            contents: read

      - name: Build image
        run: docker build . --file Dockerfile --tag image
        env:
          GITHUB_TOKEN: ${{ steps.access-token.outputs.token }}

The error I am getting is:
Run qoomon/actions--access-token@v3
Get access token...
Error: HttpClientError: Issues:
- <my_org>:
  - Invalid access policy
Effective OIDC token subjects:
- repo:<my_org>/<repoA>:ref:refs/heads/main
- repo:<my_org>/<repoA>:workflow_ref:<my_org>/<repoA>/.github/workflows/dockerpublish.yml@refs/heads/main
- repo:<my_org>/<repoA>:job_workflow_ref:<my_org>/<repoA>/.github/workflows/dockerpublish.yml@refs/heads/main
    at file:///home/runner/work/_actions/qoomon/actions--access-token/v3/action/dist/main/index.js:6[16](https://github.com/<my_org>/<repoA>/actions/runs/11999077077/job/33448085869#step:2:17)00:31

What are the required permissions from the template that I need?
It is a bit vague, so I went with most subjects and the 'contents: read' permission, which is what I need from repoB.
Thanks,
Mustafa

allowed-subjects:was missing in your org policy
origin: <my_org>/.github-access-token
allowed-subjects:
   - 'repo:<my_org>/*:**'
allowed-repository-permissions:
  contents: read
statements:
  - subjects:
    - repo:${origin}:ref:refs/heads/main
    - repo:${origin}:ref:refs/tags/v*
    - repo:${origin}:workflow_ref:${origin}/.github/workflows/build.yml@refs/heads/main

    permissions:
      contents: read

btw your allowed-subjects: is the default value
allowed-subjects:
   - 'repo:<my_org>/*:**'

Thanks, it's always the small things.
I just did that but still getting the error.
Now the "origin: " should always be the repo itself (i.e. for RepoA, the access-token.ym file should read "origin: <my_org>/RepoA" correct?
Also, do I need the .github/access-token.yml in both RepoA and RepoB or just RepoB, the granting repo? and should the subject referring to the workflow should always map to build.yml or my own actions file dockerpublish.yml?

Thanks, it's always the small things.

:-D

the Effective OIDC token subjects: from the error shows you what subjects you probably need to grant access for in repo B (you don't need to add all, just one of the subjects that makes most sense for your use case)

Great, it is working fine now and I am getting the token for repo_B.  However, I am back to the same issue that got me to install your GH App.  The curl is still failing, how can I make sure the GITHUB_TOKEN is new token and not the default GH Actions token:
curl -L \
    -H "Accept: application/vnd.github+json" \
    -H "Authorization: Bearer $GITHUB_TOKEN" \
    -H "X-GitHub-Api-Version: 2022-11-28" \
    https://github.com/<my_org>/<repo_B>/archive/refs/heads/main.tar.gz

Also, once I get this to work, would this token work for RepoC if I do the same as I did in RepoB access-token.yml?

   steps:
      - uses: qoomon/actions--access-token@v3
        id: access-token
        with:
          permissions: |
              contents: read

      - name: Update secret
         run: |
          curl -L \
            -H "Authorization: Bearer $GITHUB_TOKEN" \
            https://github.com/<my_org>/<repo_B>/archive/refs/heads/main.tar.gz
        env:
          GITHUB_TOKEN: ${{ steps.access-token.outputs.token }}

Worked great thanks.


Load a trained pbuf model into general pytorch env

Is your feature request related to a problem? Please describe.
I would like to test the trained model externally, using an independent pytorch-based script that compares several models. The pbuf format is serialized differently that the regular .pth torch snapshots and can't be loaded using model.load_state_dict.
Describe the solution you'd like
It would be great if you can provide either a tool for converting pbuf to pth, or a flag enabling to save the model in pth format during the learning rounds.
Thanks a lot.

Thanks @r-kellerm for the suggestion! This is very timely. We are in the process of adding the ability to convert the model to native format after experiment completion. This will be part of our next OpenFL 1.5 release. Feel free to check out that branch and use fx model save after you run an experiment to get unblocked in the meantime (note: exact usage may change somewhat in the final version)

Hello @psfoley , @igor-davidyuk ,
I'm using fx model save -i model_best.pbuf -o model_best.pth (with either best and last models), then load it using -
        ckpt = torch.load(args.model_path)
        model = CNN().to(device)
        model.load_state_dict(ckpt['model_state_dict'])
        model.eval()

However the model doesn't reach the same accuracy as the OpenFL training environment reports on. Am I missing a required step?
(The data  and the model are the same as in the federated environment)
Thanks!

Hey, @r-kellerm, we probably should not discuss the resulting accuracy here, this issue is about functionality.
Still, it would be interesting to see metric logs and the test accuracy you measured to understand the order of values.
Please keep in mind that the best model snapshot is not always the last model snapshot, AND that Aggregator shows you averaged values for metrics during training.

Hi @igor-davidyuk , @psfoley -
I reproduced the problem with torch_cnn_mnist.
The accuracy as reported from the training environment is >0.98.
When loading the saved model (with torch_cnn_mnist_best.pbuf converted to torch_cnn_mnist_best.pth) the reported accuracy is 0.58.
You can reproduce it with the attached file.
(The trained model was generated using fx model save -i "./save/${TEMPLATE}_best.pbuf" -o $"./save/${TEMPLATE}_best.pth")
standalone_mnist_cnn.py.txt


Adds logic in the get_bib_record method to remove suppressed bibs

Add fixtures for xml suppressed and unsuppressed,
Update get_bib_record method to remove suppressed bibs
closes #710

Coverage increased (+0.3%) to 81.156% when pulling c3bedbcad871ede1edef9b3ae5d448517214ee67 on 710-suppressed-bib into 58acfd4e191dbad3d85e8ccd3a50b03d66c7c1ec on master.


Possibly bug, TS wrongly complains that generic property is null

TS v 3.3.3, enable strictNullChecks
Code
// Errors is a collection of form errors
//
// const errors = {
//   title: ["can't be blank", "invalid characters"],
//   text:  ["too long"],
// }
//
// It could be rendered as
//
// <FormInput name="title" errors={errors}/>

// Requiring that Errors had field of type T
type Errors<T extends string> = { [k in T]?: string[] }

export function FormInput<T extends string>(
  { name, errors }: { name: T, errors: Errors<T> }
) {
  // TS complains that there's no length
  const hasErrors = (errors[name] || []).length > 0
}

Expected behavior:
Should correctly infer that (errors[name] || []) is an Array.
Actual behavior:
Wrongly complains about null
Playground Link:
Play

Duplicate #22137 - we don't have type system machinery in place yet to narrow away undefined/null from types originating in a generic type.


Bump Cilium to latest version (1.13.x)

What would you like to be added:
Following these instructions, let's bump Cilium to the latest versions.
Why is this needed:
We need this for security patches, support and for new features.

This would be great!  I've been waiting for a newer version of Cilium on EKS-A for observability features utilizing eBPF that need at least Cilium 1.13.0 or higher.

v1.13 has been made available at https://gallery.ecr.aws/isovalent/cilium so we should now be able to do the upgrade work.

completed


CVE-2021-40690 - Metro webservices-rt.jar contains vulnerable version(2.1.5) of "Apache Santuario - XML Security for Java".

Hi Team,
The latest versions of Metro webservices-rt.jar (2.4.6 & 3.0.2) bundles a vulnerable version of "Apache Santuario - XML Security for Java".
As per "webservices-rt-2.4.6/META-INF/maven/org.apache.santuario/xmlsec/pom.properties" file xmlsec.jar is on version 2.1.5 :
#Generated by org.apache.felix.bundleplugin
#Tue Mar 10 10:35:27 GMT 2020
version=2.1.5
groupId=org.apache.santuario
artifactId=xmlsec

There is a CVE-2021-40690 associated with Apache Santuario - xmlsec.jar with base severity score of 7.5 (HIGH).
As per CVE the fix is to make use of 2.2.3 OR 2.1.7 xmlsec jar.
Below is the vulnerability description:**
All versions of Apache Santuario - XML Security for Java prior to 2.2.3 and 2.1.7 are vulnerable to an issue where the "secureValidation" property is not passed correctly when creating a KeyInfo from a KeyInfoReference element. This allows an attacker to abuse an XPath Transform to extract any local .xml files in a RetrievalMethod element.
The KeyInfoReference.class is shipped with webservices-rt.jar as in the below screenshot:

Can someone please take a look into this issue to make use of 2.2.3 OR 2.1.7 xmlsec. jar?
Thanks,
--Rohit

@Tomas-Kraus is there an expected date that it will be ready?

yesterday. See https://repo1.maven.org/maven2/org/glassfish/metro/metro-standalone/


Stale file handle during reading rewritten object with cached metadata

Mountpoint for Amazon S3 version
mount-s3 1.4.0
AWS Region
us-east-2
Describe the running environment
Running in local Ubuntu 22.04.4 LTS.
Mountpoint options
mount-s3 mhnap-bucket /media/mount-s3 --read-only --log-directory=/tmp/mount-s3/logs --debug --debug-crt --cache /tmp/mount-s3 --metadata-ttl 1000000

What happened?
This issue is similar to https://github.com/awslabs/mountpoint-s3/issues/782 which I previously created, but here I rewrite the object.
I am not sure whether this is expected behavior, so I would be grateful for clarifying.
The test I ran:
$ dd if=/dev/urandom of=file bs=1 count=10
10+0 records in
10+0 records out
10 bytes copied, 0.000198077 s, 50.5 kB/s
$ aws s3 cp file s3://mhnap-bucket/dir/file
upload: ./file to s3://mhnap-bucket/dir/file
$ mount-s3 mhnap-bucket /media/mount-s3 --read-only --log-directory=/tmp/mount-s3/logs --debug --debug-crt --cache /tmp/mount-s3 --metadata-ttl 1000000
bucket mhnap-bucket is mounted at /media/mount-s3
$ ls -lh /media/mount-s3/dir/file
-rw-r--r-- 1 mhnap mhnap 10 Feb 26 11:31 /media/mount-s3/dir/file
$ dd if=/dev/urandom of=file bs=1 count=100
100+0 records in
100+0 records out
100 bytes copied, 0.000530112 s, 189 kB/s
$ aws s3 cp file s3://mhnap-bucket/dir/file
upload: ./file to s3://mhnap-bucket/dir/file
$ ./read /media/mount-s3/dir/file
Err(Os { code: 116, kind: StaleNetworkFileHandle, message: "Stale file handle" })
$ ./read /media/mount-s3/dir/file
Ok([200, 84, 37, 49, 213, 169, 102, 191, 249, 109, 23, 138, 141, 212, 234, 50, 58, 244, 106, 160, 44, 217, 177, 118, 211, 2, 57, 181, 93, 134, 29, 84, 89, 75, 49, 37, 65, 232, 62, 250, 247, 50, 209, 144, 151, 55, 133, 102, 221, 216, 26, 133, 12, 191, 11, 201, 77, 137, 136, 120, 253, 28, 0, 143, 205, 0, 120, 14, 137, 251, 40, 167, 17, 28, 184, 127, 226, 24, 102, 194, 127, 39, 180, 231, 221, 236, 79, 63, 252, 46, 236, 0, 66, 97, 35, 224, 152, 65, 49, 45])
$ umount /media/mount-s3/

This case is different cause I can read successfully on the second attempt.
Also, if I skip ls -lh /media/mount-s3/dir/file then I would see a successful read on the first attempt.
Cannot paste logs, so uploaded them here: mountpoint-s3-2024-02-26T09-31-13Z.log
Relevant log output
No response

This is a similar scenario as in #782, except when trying to get the content, Mountpoint reports that the version that is in the (metadata) cache is stale compared to the one in S3.
I cannot reproduce the successful second attempt, though. When using --cache and --metadata-ttl, Mountpoint will not try to refresh its metadata cache until the TTL expires, except on specific operations, like ls on the containing directory, or when opening the file with the O_DIRECT flag - see details here.
The logs you included do in fact show lines like this:
2024-02-26T09:31:40.493833Z DEBUG readdirplus{req=366 ino=2 fh=28 offset=0}: mountpoint_s3::inode: inode needs to be recreated parent=2 name="file" ino=3

which are likely the result of a ls finding a stale metadata entry and replacing it with a fresh one.

Closing as expected behavior. Feel free to reopen or raise a new issue if you have more questions.


Use markdown paths, instead of url links, for smooth version updates going forward

Hmm the build passes for me locally, very weird


Add stephprince to CODEOWNERS

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Closing. Will make pr from fork instead as instructed.


Site Breadcrumbs

Full control over site breadcrumbs, so that users and search engines always know where they are.

Clear keyword content used


Notifications not working in production

@khmyznikov Hello. I have finally successfully implemented notifications using ios-pwa-wrap. However, I still have a small problem with notifications in production.
When the application server is running locally (localhost) and I launch the app through Xcode referring to let rootUrl = URL(string: "http://localhost:8080")!, notifications are delivered successfully. But when the settings specify let rootUrl = URL(string: "https://mysite.com")! for the test server domain, notifications do not come through.
Could it be an issue with the bundleId settings in the Firebase project? I have a test server (https://state.mysite.com) and a production server (https://mysite.com). In Firebase, the bundleId is specified for the production server, but for testing in TestFlight, I build with the test server domain.
Can you suggest where I might have gone wrong?

Hi, figured out?


'' is not a valid version string when debugging or running tests.

omnisharp-1866-log.log
Environment data
dotnet --info output:
.NET Command Line Tools (2.0.2)

Product Information:
 Version:            2.0.2
 Commit SHA-1 hash:  a04b4bf512

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.14393
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\2.0.2\

Microsoft .NET Core Shared Framework Host

  Version  : 2.0.0
  Build    : e8b8861ac7faf042c87a5c2f9f2d04c98b69f28d

VS Code version:
1.18.0
C# Extension version:
1.13.1
Steps to reproduce
Try and run test using "Debug Test" button.
Expected  behavior
The test runs.
Actual behavior
Immediate error pops up in toast:
Failed to start debugger: "System.ArgumentException: '' is not a valid version string.
Parameter name: value
 at NuGet.Versioning.SemanticVersion.Parse(String value)
 at OmniSharp.DotNetTest.TestManager.Create(Project project, DotNetCliService dotNetCli, IEventEmitter eventEmitter, ILoggerFactory loggerFactory)
 at OmniSharp.DotNetTest.Services.BaseTestService.CreateTestManager(String fileName)
 at OmniSharp.DotNetTest.Services.DebugTestService.Handle(DebugTestGetStartInfo

Some text is cut off, presumable because the toast can only hold so much.  Also, this was working with a pre-release version of 1.13 and failed when Omnisharp automatically updated to 1.13.1.

Hi, thanks for reporting this issue. Could you provide your full OmniSharp Log when this problem occurs? Just select View->Output from the VS Code menu to display the Output pane. Then, pick "OmniSharp Log" from the drop-down at the top-right corner of the Output pane.

I've updated the post with an attached log

Did you capture the log after reproducing the error?

At the very bottom, I see this:
Failed to launch 'dotnet' with args, '--version'

Is dotnet on your PATH?

Yes.  I started VSCode, waited for Omnisharp to load, attempted to run a
test, then captured the log.
On Tue, Nov 14, 2017 at 8:31 AM Dustin Campbell<EMAIL_ADDRESS>wrote:

Did you capture the log after reproducing the error?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OmniSharp/omnisharp-vscode/issues/1866#issuecomment-344295504,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABnhXROgvIR-4dWIMJWpxYt0hYDyijLeks5s2bI2gaJpZM4QcrVp
.

dotnet was on the PATH (I could start a new cmd window and run dotnet --version without issue) but I see where you were going with that.  I tried restarting VSCode and that didn't help.  I then moved the C:\Program Files\dotnet entry higher up the PATH and everything started working.  Something must have been obscuring it.
I believe I got into this state because I had VS2017 installed and installed the dotnet SDK.  OmniSharp was picking up VS2017's msbuild because of an earlier bug.  I then uninstalled VS2017 to avoid this and when I did I somehow lost dotnet on the PATH.  I had to add the entry back in manually.  I had assumed this had fixed things because the command prompt started working but something weird must be going on.  Thanks for the help.  You can close this issue.

Hmm, I tried pushing it back down to the bottom of PATH and it still worked so I must just not have fully restarted VSCode or something along those lines.

Thanks for the update @westonpace. I think I understand the issue now. cc @livarcocc in case he has more insight into uninstall problems with VS and the .NET Core SDK.


Create me

Create a simple GraphML implementation.

Originally suggested in clue/graph#3 by @clue

Initial version has just been pushed :shipit:


feat(Ads): Allow OverlayEvents without viewport

So the default is to treat the OverlayEvent as an overlay that covers 100% of the space, then?

Yes!


[1.1.2] Don't log stack traces for localhost bind failures (port fix to 1.1.x)

Port this fix to 1.1.x: https://github.com/aspnet/KestrelHttpServer/pull/1238

This patch fix is approved. Please follow the normal code review / pull request process and make sure you make the change in the correct branch.

Preview builds of this patch fix should be available on the following feeds:

https://dotnet.myget.org/gallery/aspnet-1-0-5-may2017-patch-public (for fixes in the 1.0.x release)

Feed URL to add to NuGet.config: https://dotnet.myget.org/F/aspnet-1-0-5-may2017-patch-public/api/v3/index.json

https://dotnet.myget.org/gallery/aspnet-1-1-2-may2017-patch-public (for fixes in the 1.1.x release)

Feed URL to add to NuGet.config: https://dotnet.myget.org/F/aspnet-1-1-2-may2017-patch-public/api/v3/index.json

If you have a chance, please try out these preview builds and let us know if you have any feedback!


Dynamic color theming not working in separate style sheet

hi,
react-native-color-theme: "0.0.1",
react-native: "0.50.4",
The problem is when i apply this colors in inline styles, color theming was working but when i m using in common styles files dynamic changing of themes not working. Can anyone help on this.

Hi @saisrirammanchella
Thank you for using the theme manager.
If you would like to use dynamic styling you should supply a function to the style instead just as you would do in any case when using dynamic styling.
This way the styling will get updated every time react native renders the application. Otherwise it will only get updated when the styles constant is reevaluated. I added an example project that you can look into here https://github.com/Ladekarl/react-native-color-theme/blob/master/example/App.js


sharing account / opportunity?

[Enter feedback here]

For example, in an organization that gives salespeople Basic read and write access to accounts, a salesperson can share an opportunity with another salesperson so that they can both track the progress of an important sale.
It is an account instead of opportunity?
It doesn't make sense for me if it is regarding to opportunity.

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

ID: 507f3fb4-078a-fa09-cc1b-b10b2097c7c0
Version Independent ID: 00b1b2e4-55e5-6cce-6943-63ca584dce34
Content: Sharing and assigning (Microsoft Dataverse) - Power Apps
Content Source: powerapps-docs/developer/data-platform/security-sharing-assigning.md
Service: powerapps
Sub-service: dataverse-developer
GitHub Login: @paulliew
Microsoft Alias: paulliew

Thanks for your comment, @ToshiakiKa. This issue has been assigned to the doc author for review.

@ToshiakiKa The point being made in the article is that sharing table data (i.e., an opportunity) gives access of that data to users who would not normally have access to that data due to their security role.


Needs API to retrieve transaction history

This should exist, under the 'gettransactionsbyaddress' API. Please let me know if this is insufficient.


Local pull request success status testing

Docs Build status updates of commit 8bfa788:
:clock10: Pending: waiting for processors (13 builds ahead of you)
:warning: Docs Build is busy, currently there are 13 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 8bfa788:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

E2E_DocsBranch_Dynamic/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Logic before deletion

When deleting roles in batches, it is necessary to determine whether there is associated data
Need to judge whether there is data before deleting?

Not sure if this is required for roles. By default rules, when a role is deleted, the associated data is not deleted, but only their relationship is unlinked


Scene component which activates powerview scenes

First try to implement Hunter Douglas powerview scenes. (http://www.hunterdouglas.com/operating-systems/powerview-motorization).
I used the scene component and changed it. It results in a list of scenes defined by the powerview blinds which can be activated.
See also : https://drive.google.com/file/d/0B9T41s-XxNycQmRQWEhHbWdiOVU/view?usp=sharing
I was able to create the required user interface locally but submitting the compiled frontend.html doesn't seem to be right. Even though it is described as such in the docs. So any instructions here are more than welcome !
But first I'd like to know whether my proposed solution is acceptable in principle.

I don't think that this component should be putting out an entity per scene. Instead I would expect it to offer a service to activate scene. That way users can set up their own way of controlling scenes, either via a scripts or input_select + automation. That way there will also not be any changes to the frontend needed.
For the name, please include the full brand in the name: hunterdouglas_powerview
For the rest, what @balloobbot said ;-)

My goal is to be able to automatically come up with the available scenes. I feel this component should have a card of its own (just like any other component). Just having a service available is -in my opinion- a not so good user experience..
So I hope you want to reconsider.
Still the instructions for doing frontend development are too complex to me (or they are outdated). I don't know. But I'd really like to learn doing it. (As said I can't imagine I need to pr the compiled frontend.html file... )

Frontend contributions should be opened as a PR to the home-assistant-polymer repo. After accepted a PR should be made to the dev branch that includes the updated submodule pointing at the new commit and a new compiled version (like you did now).
I'm thinking now that we should open up the scene component for platforms and start with homeassistant and hunterdouglas_powerview. Probably zwave can be added later. That way it will work like you propose for Hunter Douglas but we're not copy pasting code around in both frontend and backend.

That would be great !
If I understand correctly: We'll make a scene domain where we can add platforms too.
Is that something I should try and start working on or is this something you'd rather pick up yourself?

I have enough to do so be great if you could do it.
Convert current scene entity to base entity: Make activate abstract, remove
name, entities are not being used so can be removed completely. Move config
parsing to a home Assistant platform and have the HA scene entity implement
existing name and activate  based on config.
On Thu, Feb 18, 2016, 23:43 sander76<EMAIL_ADDRESS>wrote:

That would be great !
If I understand correctly: We'll make a scene domain where we can add
platforms too.
Is that something I should try and start working on or is this something
you'd rather pick up yourself?
—
Reply to this email directly or view it on GitHub
https://github.com/balloob/home-assistant/pull/1315#issuecomment-186104918
.

Okay. I'll do.

I am trying to figure out how to do the config file without breaking backward compatibility of the default HA scene.
See below what I mean. (Not knowing whether there is a nice solution available)
Is this what you're thinking of ?
scene:
  - name: Romantic
    entities:
      light.tv_back_light: on
      light.ceiling:
        state: on
        xy_color: [0.33, 0.66]
        brightness: 200

  - name: Movies
    entities:
      light.tv_back_light:
        state: on
        brightness: 100
      light.ceiling: off

  - platform: hunter_douglas_scene
    address: "<IP_ADDRESS>"

We should use the entity component setup method to process the platforms.
Setup gets a config passed in. If we can detect that the list config entries do not contain a platform key, we can wrap it with platform: homeassistant.

if not any ( CONF_PLATFORM in entry for entry in config) :
config = { 'platform' :'homeassistant', 'scenes' : config}


altsrc with YAML: Flag is Never Assigned

How to use a YAML config file to set flags? I've read the docs and tried a couple of things; then waited a month and read them again but, nothing seems to work.
I'm using v2.3.0.
package cli

var Flags []cli.Flag

func init() {
        Flags = []cli.Flag{
                altsrc.NewStringFlag(
                        &cli.StringFlag{
                                Required: true,
                                Name:    "foo",
                        }
                )
        }
}

package main

// In main()
app := &cli.App{
        Name: "tester",
        Before: altsrc.InitInputSourceWithContext(cmd.Flags,
                func(context *cli.Context) (altsrc.InputSourceContext, error) {
                        return altsrc.NewYamlSourceFromFile(".myconfig")
                }
        )
}

But every time I run it I get an error because foo is required and not set.
I have also tried what sounds like a more verbose version of the above:
app := &cli.App{
        Name: "sdt",
        Before: func (c *cli.Context) error {
                src, err := altsrc.NewYamlSourceFromFile(".myconfig")
                if err != nil {
                        return err
                }

                fmt.Printf("%+v\n", src)

                err = altsrc.ApplyInputSourceValues(c, src, cmd.Flags)
                if err != nil {
                        return err
                }

                return nil
        },
}

The config is printed and everything is there but the flag is never assigned.
I thought it may be related to #1086 but this appears to be fixed in v2.3.0. Help!

Still interested in it's resolution

Still interested in its resolution

I have the same issue. I can confirm that the inputsource generation is going as expected.
But we have a nilset returned as art of the struct when declaring the flag: https://github.com/urfave/cli/blob/1259f1efc973b210de2c7b448b6302d6f52081d1/altsrc/flag_generated.go#L191
And then we apply the inputSourceValue only if the set is not nil...
https://github.com/urfave/cli/blob/1259f1efc973b210de2c7b448b6302d6f52081d1/altsrc/flag.go#L148

Here's an simple example on v2.20.2:
main.go
package main

import (
	"fmt"
	"os"

	"github.com/urfave/cli/v2"
	"github.com/urfave/cli/v2/altsrc"
)

var (
	logfile string
	verbose int
)

func main() {
	flags := []cli.Flag{
		&cli.StringFlag{
			Name:  "config",
			Usage: "yaml config file name",
		},
		altsrc.NewStringFlag(
			&cli.StringFlag{
				Name:        "logfile",
				Aliases:     []string{"log.file"}, // match config file structure
				Value:       "",
				Usage:       "log to file",
				Destination: &logfile,
			},
		),
		altsrc.NewIntFlag(
			&cli.IntFlag{
				Name:        "verbose",
				Aliases:     []string{"log.verbose"},
				Value:       0,
				Destination: &verbose,
			},
		),
	}

	app := &cli.App{
		Flags:  flags,
		Before: altsrc.InitInputSourceWithContext(flags, altsrc.NewYamlSourceFromFlagFunc("config")),
		Action: func(ctx *cli.Context) error {
			fmt.Printf("config file: %s, log {file: %s, verbose: %d}\n", ctx.String("config"), logfile, verbose)
			return nil
		},
	}

	fmt.Println(app.Run(os.Args))
}

example.yml
log:
  verbose: 1
  file: log.txt

Aliases attributes are used to specify mapping between flag and config file structure.

@Archwizard The code you have posted works
$ cat example.yml
log:
  verbose: 1
  file: log.txt
$ go run main.go --config example.yml
config file: example.yml, log {file: log.txt, verbose: 1}
<nil>

@sshaw Do you have a complete example to share ?

Thanks @Archwizard. The Aliases matching the log file structure are key, though a bit ugly considering they're now also command-line options.

@sshaw Do you have a complete example to share ?

Is this not a compete example?

@sshaw You need to provide Flag to app as well
	app := &cli.App{
		Name:  "tester",
		Flags: Flags,
		Before: altsrc.InitInputSourceWithContext(Flags,
			func(context *cli.Context) (altsrc.InputSourceContext, error) {
				return altsrc.NewYamlSourceFromFile(".myconfig")
			}),
	}

I hope that the Aliases stuff can be mentioned in the documents...


Black formatter integration

The Black formatter has been integrated to the repo and it has been ran over all the files.
Closes #66
Post-install instructions
Just execute
pre-commit install

That will install the pre-commit hook to your local repo so black and the other three hooks are executed before any commit.
I've added the instructions on the readme as well

@edsu take a look at the .env test here. The problem was that I was creating the file in the root of the project. I've made it to be created in the test folder as well so it doesn't conflict with the Travis one. Anyway I didn't erase the @pytest.mark.skipif just in case. But maybe now it works even on Travis

Ok, I think I've got this merged into master. I'm testing without skipping the test on Travis now.


Micronaut® Launch extension renamed.

Micronaut® Launch extension recently renamed to Launch for Micronaut® framework.

Thank you @dbalek


Apple attestation

A new attestation is existing now for Apple device
Maybe can solve some problems with TouchID / FaceID.

I didn't see the implementation inside the code and it's not listed in the README of the server
So the implementation is done, sorry for this issue and you did a very good job :D

@akanass Thank you for noting that the server README didn't list "apple" attestation support - I've gone and updated that! It's now in-sync with the docs over on https://simplewebauthn.dev/ 😄


Angular2 + TS + Node.js = Module not found: Error: Can't resolve 'child_process'

SerialPort version: 4.0.6
NodeJS Version: v7.2.0
Operating System and Hardware Platform: macOS - Sierra
Are you using an alternative NodeJS runtime? (eg Electron): No
Summary of Problem
Serialport compilation fails due to following error:
ERROR in ./~/serialport/lib/list-unix.js
Module not found: Error: Can't resolve 'child_process' in '/PATH/TO/PROJECT/node_modules/serialport/lib'
 @ ./~/serialport/lib/list-unix.js 4:19-43
 @ ./~/serialport/lib/bindings.js
 @ ./~/serialport/lib/serialport.js
 @ ./src/app/serial-port-mgr/serial-port-mgr.component.ts
 @ ./src/app/app.module.ts
 @ ./src/app/index.ts
 @ ./src/main.ts
 @ multi main

Chrome reports:
./~/serialport/lib/list-unix.js
Module not found: Error: Can't resolve 'child_process' in '/Path/To/Project/node_modules/serialport/lib'
resolve 'child_process' in '/Path/To/Project/node_modules/serialport/lib'
  Parsed request is a module
  using description file: /Path/To/Project/node_modules/serialport/package.json (relative path: ./lib)
    Field 'browser' doesn't contain a valid alias configuration
  after using description file: /Path/To/Project/node_modules/serialport/package.json (relative path: ./lib)
    resolve as module
      looking for modules in /Path/To/Project/src
        using description file: /Path/To/Project/package.json (relative path: ./src)
          Field 'browser' doesn't contain a valid alias configuration
        after using description file: /Path/To/Project/package.json (relative path: ./src)
          using description file: /Path/To/Project/package.json (relative path: ./src/child_process)
            as directory
              /Path/To/Project/src/child_process doesn't exist
            no extension
              Field 'browser' doesn't contain a valid alias configuration
              /Path/To/Project/src/child_process doesn't exist
            .ts
              Field 'browser' doesn't contain a valid alias configuration
              /Path/To/Project/src/child_process.ts doesn't exist
            .js
              Field 'browser' doesn't contain a valid alias configuration
              /Path/To/Project/src/child_process.js doesn't exist
      looking for modules in /Path/To/Project/node_modules
        using description file: /Path/To/Project/package.json (relative path: ./node_modules)
          Field 'browser' doesn't contain a valid alias configuration
        after using description file: /Path/To/Project/package.json (relative path: ./node_modules)
          using description file: /Path/To/Project/package.json (relative path: ./node_modules/child_process)
            as directory
              /Path/To/Project/node_modules/child_process doesn't exist
            no extension
              Field 'browser' doesn't contain a valid alias configuration
              /Path/To/Project/node_modules/child_process doesn't exist
            .ts
              Field 'browser' doesn't contain a valid alias configuration
              /Path/To/Project/node_modules/child_process.ts doesn't exist
            .js
              Field 'browser' doesn't contain a valid alias configuration
              /Path/To/Project/node_modules/child_process.js doesn't exist
[/Path/To/Project/src/child_process]
[/Path/To/Project/src/child_process]
[/Path/To/Project/src/child_process.ts]
[/Path/To/Project/src/child_process.js]
[/Path/To/Project/node_modules/child_process]
[/Path/To/Project/node_modules/child_process]
[/Path/To/Project/node_modules/child_process.ts]
[/Path/To/Project/node_modules/child_process.js]
 @ ./~/serialport/lib/list-unix.js 4:19-43
 @ ./~/serialport/lib/bindings.js
 @ ./~/serialport/lib/serialport.js
 @ ./src/app/serial-port-mgr/serial-port-mgr.component.ts
 @ ./src/app/app.module.ts
 @ ./src/app/index.ts
 @ ./src/main.ts
 @ multi main

Steps and Code to Reproduce the Issue
Started fresh Angular-CLI project with Webstorm 2016.3
Installed serialport with npm (npm install serialport --save)
Installed typings with npm (npm install @types/serialport --save)
Imported Serialport to component (import * as SerialPort from 'serialport';)
Tried to list all available port on the console:
    SerialPort.list(function (err, ports) {
      ports.forEach(function (port) {
        console.log(port.comName);
      });
    });

This code was simply embedded in the ngOnInit Method of the component.
No Lint errors. Autocompletion works.
NPM server does not compile
Please help me figure out how to get this running.

Code is not executed at all. It seems to crash the entire building to that even the app-root selector does not get exchanged with the html template.
First I wanted to run the code inside the components constructor, then on the ngOnInit method... Lastly I put it into a dummy function that gets not called at all... same failure inside the TS 2 JS 2 Server pipeline. As I have very little experience with web development (I'm mainly a desktop app programmer) I have no clue where to look closer for the solution.

Okay tried it out with a simple React App
import React, { Component } from 'react';
import './App.css';
import * as SerialPort from 'serialport';
class App extends React.Component {
  constructor(props){
    super(props);
  }

  componentdidMount(){
      SerialPort.list(function (err, ports) {
          ports.forEach(function(port) {
              console.log(port.comName);
              console.log(port.pnpId);
              console.log(port.manufacturer);
          });
      });
  }

  render() {
    return (

Hello React!

    );
  }
}

export default App;

As soon as the SerialPort part is included the compilation fails:

So what is serialport good for then? I want to access serialports to communicate with hardware, right? Over a "webpage" that is shown inside a browser. Or have I missed some new ways of interacting with the web? I think I don't get the point right now... *confused
Anyway: this message is not produced by Chrome! It is an error page provided bei a node.js dev-server (started with npm start from Webstorm's terminal). The same error message occurs here:

Okay, now I understand. Thank you.
I am using electron right now and tried to combine it with browser-serialport, but got only an null ports object.
Then I found this site here
https://github.com/paulolc/electron-quick-start-serialport
The script/install section inside the package.json file combined with this information
https://github.com/electron/electron/blob/master/docs/tutorial/using-native-node-modules.md#modules-that-rely-on-node-pre-gyp
got node-serialport working.
8 hours later and three JS frameworks wiser I finally have it running. *yay


Live

test PR

Docs Build status updates of commit 5933f3a:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Test PR


Why is there a default target_size in flow_from_directory ?

target_size=(256, 256) seems arbitrary. When I don't want my images to be resized, I pass nothing and I expect that nothing will happen. I was very surprised to see that they were resized !

[X] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps

[X] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

[X] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[ ] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

This default parameter is still there. There is still no reason for it to be there.

@rdelassus did you find any workaround so flow_from_directory won't resize my pictures?

@adrianodennanni I've stopped using it. I've regrouped all my images in one hd5 file.


bosh download manifest adds exclamation marks !

Hi everyone,
I have issues with spruce because when I download a manifest from the director, it contains exclamation marks in places I haven't added them when uploading it.
What I have uploaded:
   route_registrar:
      routes:
      - name: uaa
        port: 8080
        tags:
          component: uaa
        uris:
        - uaa.fabric.lon.bluemix.net
        - '*.uaa.fabric.lon.bluemix.net'

What I download:
properties:
    route_registrar:
      routes:
      - name: uaa
        port: 8080
        tags:
          component: uaa
        uris:
        - uaa.fabric.ng.bluemix.net
        - ! '*.uaa.fabric.ng.bluemix.net'

What can I do to avoid bosh adding those exclamations marks?
I have an issue open also in the spruce project: https://github.com/geofffranks/spruce/issues/75
Thanks in advance.

Not much. We use standard ruby yaml package which means if it makes a decision to add a ! based on the value we cant control this.

I think the ! is added whenever a value cannot be recognized, therefore is marked as "non-specific-tag" which adds the !. If you change the YAML engine in a ruby class you can avoid this( .i.e YAML::ENGINE.yamler = 'syck' ). Is this not controllable?


How to check for errors?

What's the proper way to check for errors coming from the RouterOS device when using the Query() function?
I'm currently using the following code, which is what the Connect() function uses to check for login errors. But shouldn't the Query() function return an error when it receives a !trap from the RouterOS device?
get_routeros_interface_stats.go
if len(reply.Pairs) > 0 {
  log.Fatalf("RouterOS error: %+v", reply)
}

Login errors are a bit of a special case where any unexpected output is assumed to be a error.
I added err response on !trap in bf5c969fa5acfcf0e705c1c0e3a74e7355f05963 but appears to be useful only in three situations:

Call() with an invalid command name: RouterOS will throw an error on an invalid command name ("/interface/explode")
Call() with invalid parameters: RouterOS will throw an error on an invalid parameter ("interface=badbridge")
Query() with an invalid command name:
RouterOS appears to only throw an error on an invalid command name. Invalid query parameters are silently ignored.

If I'm understanding your example, I'm not sure that is a case where RouterOS would send a !trap in reply to a "/interface/print".
If you consider not getting any results from "/interface/print" to be an error, I'd do something like:
if len(reply.Pairs) == 0 {
  log.Fatal("error: no interfaces")
}

Does that help?

Thank you for the change. It's working as expected now.
I was fearful that I could be showing no interfaces because of an error, and not showing anything indicating that an error had happened.
The documentation is not very specific about what conditions could result in an error, specially category 4: http://wiki.mikrotik.com/wiki/Manual:API#.21trap


Let me know if I did this right

Hmm, I think you did that right. I merged some of your changes manually and pushed it back here.


Few properties missing in "MSBuild Reserved and Well-known Properties"

I don't know how many built-in properties are undocumented but I'll update them here as I encounter them or look into it specifically in the MSBuild codebase.
The following properties are not documented...

[ ] MSBuildFrameworkToolsPath Points to %WinDir%\Microsoft.NET\Framework{64}\{Version}\
This property ends with final trailing slash unlike MSBuildToolsPath.

Please add these properties to the docs. If I have time, I'll open a PR.

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

ID: ab5a5a0e-ae03-1237-6ebe-743066f27fa9
Version Independent ID: a342bfa4-4b0c-0a47-3695-858044f5aa5f
Content: MSBuild Reserved and Well-known Properties - MSBuild
Content Source: docs/msbuild/msbuild-reserved-and-well-known-properties.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @ghogen
Microsoft Alias: ghogen

@Nirmal4G - Nirmal, thank you for your feedback and offer.  Please consider these resources:

Visual Studio on Microsoft Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

@ghogen -- Gordon, please look into this issue.

It seems there are a few variations of MSBuildFrameworkToolsPath, see https://github.com/dotnet/msbuild/issues/679

Some more properties found in MSBuild's AppConfig
https://github.com/dotnet/msbuild/blob/80f618ad45d38475773fd1a6eaa059f118a0ad5a/src/MSBuild/app.config#L138-L178
https://github.com/dotnet/msbuild/blob/80f618ad45d38475773fd1a6eaa059f118a0ad5a/src/MSBuild/app.amd64.config#L168-L208

This is not completed. I've yet to open a PR.

@JamesJBarnett This is not completed. I've yet to open a PR.

@Nirmal4G
All the properties you mentioned are added here:
https://learn.microsoft.com/en-us/visualstudio/msbuild/msbuild-reserved-and-well-known-properties?view=vs-2022

Ahhh... My bad. Since there was no linked PR. I thought it wasn't added. Sorry about that. Also in the future, please link the PR so that the people in the issue know what's been done!

@Nirmal4G
I didn't realize that you can't see the links to the PRs in the private (internal to Microsoft) repo.