detect file arrival based on timestamp

As a user of the SFTP source app, I would like to be able to configure the SFTP source so it polls the remote directory for file arrival based on a file with the same name, but a newer timestamp than the last one that was dropped.  According to the documentation and some preliminary testing, that does not seem to be an option.

To be honest I don't understand what you mean.
Would you mind to re-phrase your concern somehow?
Maybe some sample with pseudo code?
The polling logic is based on the trigger options. Then we filter remote files and afterwards local files.
Thanks

Sure.  Our use case is one where the provider of the file-based data drops a file of an identical name in the SFTP location each time.  Let's say data.csv.
In my preliminary testing, if I set the filename pattern to find that file, it will load the first occurrence of that file just fine.  However, if I overwrite that data.csv file with a newer version, with different contents, my SFTP source is not detecting that it needs to load a new file.  It appears it is only using filename pattern matching criteria, and it is not paying attention to the fact that even though the filename is the same, the timestamp of the file itself is newer than the one it previously loaded.
Does that make sense?

Yep! It does.
So, your concern bumps to the fact of:
private volatile FileListFilter<File> localFileListFilter = new AcceptOnceFileListFilter<File>();

by default.
Which must be FileSystemPersistentAcceptOnceFileListFilter.
Plus preserveTimestamp should be configured.
Unfortunately we don't expose localFilter option. And I'm afraid you don't have any choice unless copy/paste SftpSourceConfiguration and inject that FileSystemPersistentAcceptOnceFileListFilter.
We probably can make it as default there because it doesn't have difference with the AcceptOnceFileListFilter, unless preserveTimestamp is configured.

Well, that's the other thing.  It seems as though preserveTimestamp is not actually working as intended.  If I set that value to true, I'm still getting a File 'ref' object with a new timestamp instead of the one from the remote file system.

Not sure how that can be possible - AbstractInboundFileSynchronizer:
File localFile = new File(localDirectory, localFileName);
if (!localFile.exists()) {
....
    if (this.preserveTimestamp) {
	localFile.setLastModified(getModified(remoteFile));
    }
}

Ok.  Weird.  I will double check that and get back to you.

Hey there.  So I saw that you marked this as an enhancement.  I'm guessing that means this will be available as an OOTB feature in upcoming releases?
Had a follow-up question, though, about the preserveTimestamp setting.  Firstly, a correction.  I mentioned we were pulling the file as 'ref', but actually we are using 'contents' as the config choice there.  I'm sort of assuming the preserveTimestamp setting is like the "-p" switch one would use on an scp command, where the timestamp on the local file would reflect the timestamp on the remote file once transferred.  I'm wondering whether preserveTimestamp applies in the 'contents' mode?  In your code snippet above, is the localFile being written to a tempdir regardless of the transfer mode of the file itself (ref, line, contents)?

Hello, @channing-jackson !
Thank you for reaching us back!
Well, yes, I think that timestamp manipulation should be as an out-of-the-box feature. It doesn't hurt by default.
Only the problem that I've faced a blockage: the remote file modification doesn't have affect for local file because of:
File localFile = new File(localDirectory, localFileName);
if (!localFile.exists()) {

So, even if we make FileSystemPersistentAcceptOnceFileListFilter as a default filter, we can't reach it via remote file modification without local file deletion.
Now we are thinking about the proper solution. One of them is like: https://jira.spring.io/browse/INT-4231.
But that all means that the fix won't be available until the next release, not the current.
No, preserveTimestamp doesn't have effect for the mode. It is done already after SftpInboundFileSynchronizingMessageSource has returned a message with the File as payload.
So, independently of the mode you can use that logic based on timestamp.
But yeah... You have to remove local files somehow...

See https://jira.spring.io/browse/INT-4232 for the particular problem.

That is an interesting problem, to be sure.  And in our PCF environment, we are making an assumption that when we write a local file, we are writing it to a tmpdir (System("java.io.tmpdir")) because we aren't really depending on the existence of a file system.  Inherently, then, that physical file object is ephemeral.  It will get cleaned up once the tmpdir is cleaned up, depending on how that is configured.  Likewise, when we use the contents transfer mode, we assume we have no physical file, because we've read the contents of the file into a byte array, and we don't particularly care whether we have a physical file to deal with...
...until we decide to output that file data somewhere else -- potentially using an SFTP sink.  In that case, we might have a processor app upstream that writes the file(s), and then we want to pass the file(s) location to the SFTP sink, and have the sink pick them up and deliver them to the SFTP destination.  Each stream app seems to get its own container in which it runs, so even though the default System("java.io.tmpdir") location represents the same path (/home/vcap/tmp) that might not actually be the same physical path for another app in the stream.
The whole stream EL might be simplified to look like this:
sftp | transform | file-processor | sftp
This seems to be becoming a larger question in general, so if this thread isn't the correct place to hold the discussion, I'm fine with moving it or taking it offline.  But it seems like if I want to pass file data through a stream of apps in a Spring Cloud Data Flow context, for example, we have to have a common TMPDIR file area where all apps can access that file data, even if they're in a different container.  I'm starting to wonder how practical the use of any file-based app is in this environment if I do not have a physical file system I can depend on.  Does this have to do with the configuration of my specific PCF environment, or would this be an issue in any PCF environment?  My understanding was that "cloud native" should be taken to mean that (among other things) you must develop applications without reliance on a physical file system.

Thank you for sharing your mind, @channing-jackson !
Well, actually we are good aware of those distributed "cloud native" environment and this "local file" dilemma.
I'm not familiar with the PCF, but seems for me I heard from my colleagues that there is something like shared file system. @sabbyanandan may know better how customers deal with the issue.
OTOH we have plan to introduce #2 - using SftpStreamingMessageSource which just place an InputStream of the remote file into payload. That part can be streamed to the byte[] in the memory and sent farther in the SCSt flow.
But anyway that seems for me that shared filesystem should be enabled somehow.
Although that is what (S)FTP is for...

@channing-jackson: The physical drive backed within the application container is ephemeral and it will not be shared with other applications. It is transient and if the application crashes, a new container is spawned with the application running on it - a new ephemeral storage comes with it. This is the general platform behavior.
The streaming solution suggested by @artembilan (via #2) provides an alternative by directly streaming the file fragments in memory as opposed to downloading it the local drive and re-processing.
If you do have tight requirements to process files from a common shared disk, but with many applications, you could potentially take advantage of "Volume Services". This has been GA'd since PCF 1.8. There are a few volume plugins supported with this service; however, we do not have enough visibility into the development/roadmap. I'd recommend engaging directly with PCF product teams.

Slightly tangential, but perhaps (in streaming mode) we should put file info (modified timestamp, size, permissions) into message headers. We currently only set remote directory (and filename) headers.
Maybe we could just dump the (S)FtpFileInfo object into a header.

That is good idea, @garyrussell !
But I'm afraid we will end up with something like fileInfoJson instead of classpath-dependant FtpFileInfo entity in the headers.
I think together with the JsonPropertyAccessor we would have a good expressions to get access to target file properties.


Fix debian policy rule

- What I did
I remove the double quotes preventing a successful build
- How I did it
I used vim
- How to verify it
Build a debian package using build-deb for ubuntu-trusty
- A picture of a cute animal (not mandatory but encouraged)
Fix issue in debian policy by removeing double quotes.
# recommend yubico-piv-tool since we include pkcs11 by default
echo 'yubico:Recommends="yubico-piv-tool (>= 1.1.0~)"' >> debian/docker-engine.substvars
dh_gencontrol
dpkg-gencontrol: warning: Depends field of package docker-engine: unknown substitution variable ${perl:Depends}
dpkg-gencontrol: warning: can't parse dependency "yubico-piv-tool (>= 1.1.0~)"
dpkg-gencontrol: error: error occurred while parsing Recommends field: aufs-tools,
           ca-certificates,
           cgroupfs-mount | cgroup-lite,
           git,
           xz-utils,
           apparmor,
           "yubico-piv-tool (>= 1.1.0~)"
dh_gencontrol: dpkg-gencontrol -ldebian/changelog -Tdebian/docker-engine.substvars -Pdebian/docker-engine returned exit code 255
make[1]: *** [override_dh_gencontrol] Error 25
make[1]: Leaving directory `/usr/src/docker'
make: *** [binary] Error 2

Signed-off-by: Paul Liljenberg<EMAIL_ADDRESS>

/cc @kencochrane @jfrazelle

LGTM

hopping over from #21349, this LGTM.
That PR has a jenkins console output with the same error so I'm thinking that this should PR should work for both :)

LGTM

I made this change locally, and I still can't do a make deb, It is probably another error unrelated to this change.
@pandrew what did you do to test your fix locally, I want to make sure I'm doing the same test.

I made this change locally, and I still can't do a make deb, It is probably another error unrelated to this change.

@pandrew what did you do to test your fix locally, I want to make sure I'm doing the same test.

@kencochrane After uploading source with docker build --rm --force-rm -t docker:master .  run:
docker run \
		-t \
		--privileged \
		-e DOCKER_EXECDRIVER=native \
		-e DOCKER_BUILD_PKGS="${DOCKER_BUILD_PKGS:-ubuntu-trusty}" \
		--name=docker \
		docker:master \
		hack/make.sh binary dynbinary build-deb

LGTM

crap sorry @kencochrane i just read your comment but hopefully its only related to the contiaainerd stuff

@jfrazelle no worries, I think this change is good +1 on that, I think my error is unrelated.

@kencochrane I was also able to get the make deb working with this fix, let me know if your error is still related to pkcs11

@riyazdf the jenkins build is green again, so I think we are good. https://jenkins.dockerproject.org/view/Docker/job/Docker Master (build deb..rpms)/


Adhere to XDG base directory spec

I moved the default configuration directory from ~/.dart and ~/.dartServer to:

Linux: ~/.config/dart and ~/.config/dartServer.
Mac: ~/Library/Application Support/dart/.
Windows: %APPDATA%/dart/

You can overwrite the ~/.config/dart path with the DART_CONFIG_DIR variable.
I believe this resolves #41560 for the most part.

Please add custom location support for both folders.

I love it. Also related, https://github.com/dart-lang/sdk/issues/42499 (DAS should use ~/.dart instead of .dartServer), https://github.com/dart-lang/sdk/issues/42813 (customize location), and https://github.com/dart-lang/sdk/issues/49166 (use ~/Library/Application Support, ~/Library/Caches and ~/Library/Preferences on mac.).

@ykmnkmi The ~/.config/dartServer path with the ANALYZER_STATE_LOCATION_OVERRIDE variable.
I'll update the macOS paths.
@srawlins I'm not sure if you're suggesting that we put the ~/.config/dart and the ~/.config/dartServer into the same folder.

I'm not suggesting anything. Just linking issues.

Is there anything I need to change to have this PR merged?

Lemme review with the team. Not sure of any consequences of moving this directory between releases.

Ping?

@srawlins is this on your plate now?

Ah sorry, missed the ping during the conference. Will discuss this morning in a team meeting.

Sorry for the delay in pointing this out, but as copybara-service points out above, the code review is done at https://dart-review.googlesource.com/c/sdk/+/322300, and there are some unresolved comments there. Please look at those and upload a new commit if you make changes in response to the comments.

XDG actually specifies a few override environment variables, which might be interesting or important here:

There is a single base directory relative to which user-specific non-essential (cached) data should be written. This directory is defined by the environment variable $XDG_CACHE_HOME.

This change could be an opportunity to use $XDG_CACHE_HOME, which might allow the OS to clean the cache occasionally. Today, with DAS's cache in $HOME/.dartServer, we can store GBs of cache in there, and the OS does not clean it up.
I think not necessary for this PR, but it would be good to address this while it's paged into our brains.

Yeah, agree with @srawlins. Most people use XDG to keep clutter out of their $HOME and have a predictable location to find various file types. Which means:

config lives in $XDG_CONFIG_HOME (usually ~/.config)
cache lives in $XDG_CACHE_HOME (usually ~/.cache)
data lives in XDG_DATA_HOME (usually ~/.local/share)
executables live in $XDG_BIN_HOME (usually ~/.local/bin)
libraries live in $XDG_LIB_HOME (usually ~/.local/lib)

So for example, just now when I installed patrol, I'd normally expect to see it show up in ~/.local/bin, but instead it's in ~/.pub-cache/bin which is a bit odd as it's not a cache file.

If it's helpful (and mostly for selfish reasons, so I get to clean up my $HOME :sunglasses:) I could submit a PR that more completely follows the spec, after this one is merged?

Actually, sorry to say this, but I think in order to minimize any intermediate locations, we should switch from $XDG_CONFIG_HOME to $XDG_CACHE_HOME before landing this; we offer nightly (ish?) builds on the Flutter master channel, so any commit could be published to users.
We chatted internally, and we'd like to provide a mechanism that slowly cleans up the data in the old location, using an existing mechanism: there is code that slowly removes old cache entries, and we can use that to remove the data in ~/.dartServer after users switch to $XDG_CACHE_HOME. I like this mechanism, but wouldn't want to apply it to more and more directories as we shift where the data is stored.
CC @DanTup @scheglov would you say everything in ~/.dartServer is "cache" information, and none of it is "configuration" data? I think so, but I'm not sure. And it looks like the analytics data is moved (in this PR) from ~/.dart/ to $DART_CONFIG_DIR ?? $XDG_CONFIG_HOME/dart/ (basically), so that seems good.
I think all I'm asking for then is to change pkg/analyzer/lib/file_system/physical_file_system.dart to use $XDG_CACHE_HOME instead of $XDG_CONFIG_HOME.

If it's analytics data, I'd argue it belongs in $XDG_DATA_HOME/dart TBH! ~/.config/dart would be reserved for things like configuration files specific to Dart itself which, I guess there aren't any? :slightly_smiling_face: And cache data is things that are actually cache files i.e. semi-temporary, won't be a disaster if they get deleted etc.

I'd argue it belongs in $XDG_DATA_HOME/dart TBH!

That sounds fair.

And cache data is things that are actually cache files i.e. semi-temporary, won't be a disaster if they get deleted etc.

True. But we want to be careful not to blow away these directories with abandon. For example, one design might be to delete the ~/.dartServer directory any time we see it on server start up. But if the user has another Dart SDK in play, an older Flutter or an in-progress migration, and an IDE launching an older Dart Analysis Server, then there could be performance or stability issues. Similarly, if we stored server cache data in one directory, and then a week later we stored it in another directory, as commits land, we'd have to track those intermediate locations for the gradual data removal.

CC @DanTup @scheglov would you say everything in ~/.dartServer is "cache" information, and none of it is "configuration" data?

Here's what I have in this folder (C:\Users\danny\AppData\Local\.dartServer on Windows) and my understanding of them (which may or may not be completely correct!)

.analysis-driver - cache - I believe this is all cache that could be thrown away (and I think is periodically cleaned up by _cacheCleanUpFunction in analyzer\lib\src\dart\analysis\file_byte_store.dart)
.dcm-teams-170 - cache - I think this is the same cache for DCM.. I don't know if the path comes from the analyzer or DCM itself (or whether it gets the same cleanup - @incendial?)
.instrumentation - ??? this contains a single uuid file - unsure what this is used for
.plugin_manager - cache - I think these are cached versions of plugins that could also be thrown away (and will rebuild)?
.prompts - not cache these are user preferences (such as saying "don't show me again" to the prompt about being able to run "dart fix"
.pub-package-details-cache - cache - this is a list of pub packages we fetch from Pub used in code-completion in pubspec.yaml
one - test - generated by server integration tests
two - test - generated by server integration tests

There's also some Dart-related stuff in (note: Roaming, not Local):

C:\Users\danny\AppData\Roaming\.dart
C:\Users\danny\AppData\Roaming\.dart-tool

These both seem telemetry related, but I'm not sure who is writing them (I suspect the new unified analytics package is doing at least one of them). I'm not sure how these two different Windows paths (Local AppData vs RoamingAppData) map on to Mac/Linux (I suspect they're merging into the home dir?).

.dcm-teams-170 - cache - I think this is the same cache for DCM.. I don't know if the path comes from the analyzer or DCM itself (or whether it gets the same cleanup - @incendial?)

Yes, it's just cache and we use the analyzer's API to get the location. And yes, the same bytestore is used so _cacheCleanUpFunction gets called as well.

@eliasyishak Could you look at the https://github.com/dart-lang/sdk/pull/53312#issuecomment-1780829314 for the telemetry question?

@srawlins friendly ping.

Thanks for the ping, @mraleph . Closing.
OP has not reponded since I asked on Oct 19 to respond to code review comments.
@SingularisArt feel free to re-open.


Merging with TJ_2018_branch

These files
Generation_Mix(github)/generation_mix.py
electricitylci/Generation_from_stewicombo.py
are named wrong or old files that haven't been removed

Wes this is not the OLCA files. The TJ_2018 branch doesnt have any new files. Its jsut my old files that I wrote a month ago. I wanted to create copy of them here because you said the conusmption mixes and others are missing in en email yesterday.

Please find the OLCA files in the branch TJ_refactor.


Crashes for certain fingerprints

Traceback (most recent call last):
  File "persistent.py", line 221, in get_all_features
    terminations, bifurcations = fingerprint_feature_extractor.extract_minutiae_features(enhanced, showResult=False)
  File "/home/user/ml/project/ML/venv/lib/python3.8/site-packages/fingerprint_feature_extractor/__init__.py", line 172, in extract_minutiae_features
    FeaturesTerm, FeaturesBif = feature_extractor.extractMinutiaeFeatures(img)
  File "/home/user/ml/project/ML/venv/lib/python3.8/site-packages/fingerprint_feature_extractor/__init__.py", line 136, in extractMinutiaeFeatures
    FeaturesTerm, FeaturesBif = self.__performFeatureExtraction()
  File "/home/user/ml/project/ML/venv/lib/python3.8/site-packages/fingerprint_feature_extractor/__init__.py", line 115, in __performFeatureExtraction
    angle = self.__computeAngle(block, 'Termination')
  File "/home/user/ml/project/ML/venv/lib/python3.8/site-packages/fingerprint_feature_extractor/__init__.py", line 40, in __computeAngle
    return (angle)
UnboundLocalError: local variable 'angle' referenced before assignment

![](file:///tmp/Left654_1.jpg)

looking into this.

changes are complete. Thank you for providing the image to help fix the bug.
I have pushed a major rebasing of the code to write it in a class based structure so that it is easy to use.
This is also compatible to the pypi release 0.0.4 which can be found here:
you can run the "example.py" to extract the features.
I tested it on the image that you have provided and it seems to be working.
Closing this issue. If it still doesn't work, please open the issue again.

It seems to work now, only a minor annoyance: there's a print statement left over (https://github.com/Utkarsh-Deshmukh/Fingerprint-Feature-Extraction/commit/6c821e4e9d48b1133bee6e3d9e9a0b61a0bd4bdd#diff-407f957a4348c3eab412aed8c92bdf94a353e994e967ebd19656adc564710b6cR115). I also noticed that the API changed (the Orientation of the Terminations have changed into a list of one element instead of just that one element. Has anything else changed from the API?

Thank you for fixing this so fast btw, really impressed :)

Thank you. hopefully it is working fine for you now.
you can also use it as a library by doing pip install fingerprint-feature-extractor
you can check out the example.py to understand how to use the library easily.
Best Regards


Remove the code that generate the metrics for the new layout projects

Feature Request
Is your feature request related to a problem? Please describe.
Currently, we generate the metrics in the run.go(for the main). See: https://github.com/operator-framework/operator-sdk/blob/master/pkg/helm/run.go#L166
However, for the new layout, the metrics are customized. See that we scaffold a service monitor resource in ./config/prometheus/monitor.yaml.
Describe the solution you'd like

Remove the roles added for it in internal/plugins/helm/v1/scaffolds/templates/role.go
Replace the metrics impl from the image used by the new layout to be done as: https://github.com/joelanford/helm-operator/commit/a9bcf842bcc3e4005217e807a58ccd33f3d82bd0
The main.go added in the https://github.com/operator-framework/operator-sdk/pull/3331 also needs to be updated

Following what I can think about that need to get done here;

remove add metrics impl from ansible and helm
remove the pkg/kube-metrics
remove what is not longer used from pkg/k8sutil
check if we have nothing about it to remove from the docs as well

@camilamacedo86 I believe this is done. Please re-open if not.

/reopen
It is not. See that we still need to remove the funcs that were used just in these features in https://github.com/operator-framework/operator-sdk/blob/master/internal/util/k8sutil/k8sutil.go

HI @estroz and @bharathi-tenneti,
After we merge the https://github.com/operator-framework/operator-sdk/pull/3513 we can close this one.


Release 1.1.10

One no-saved file ruined whole release. :(
Re-release 1.1.9 as 1.1.10

Codecov Report

Merging #2754 (1204997) into master (33a480b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2754   +/-   ##
=======================================
  Coverage   76.30%   76.30%
=======================================
  Files          88       88
  Lines        9622     9622
  Branches     2154     2154
=======================================
  Hits         7342     7342
  Misses       2014     2014
  Partials      266      266

Continue to review full report at Codecov.

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


VC is not in card format in sharing device.

Describe the bug
VC is not in card format in sharing device.
Prerequisite Download VC in sharing phone.
To Reproduce

unlock the app
Open qr code in requesting device.
Open scanner in sharing device and scan the qrcode
select VC to share

Current behavior
VC is not in card format in sharing device.
Expected behavior
VC is expected to be in card format
Screenshots

Smartphone (please complete the following information):

Device: iPhone7 & iPhone XR
OS: 15.4.1
Inji app version: 0.4.0 (16) in test flight
MOSIP Version: 1.2.1
MOSIP server: qa-121.mosip.net

Additional context
Same issue is observed in cross platform sharing as well.

@rakhimosip @Anushree09-N Once confirmed with the following image. Is that ok?
Screenshot

@santhoshsunder Before rectifying, I could able to replicate this issue in android too.  Here I also adjusted the border radius.

@rakhimosip I raised PR and sent you it in slack.

I fixed this issue in android by adding style, and border-radius at the ImageBackground component in the singleVcItem.tsx module. Screenshot
Related PR:- https://github.com/mosip/inji/pull/237/files

Verified in:

Device: iPhone 8 & iPhone 11
OS: 16.0.2 (20A380), 15.6.1
Inji app version: 0.4.1 (10) -> Testflight
MOSIP Version: <IP_ADDRESS>
MOSIP server: qa-1201-b2.mosip.net

Working as expected in iOS devices.

Verified in:

Device : iPhone 8 & OnePlus Nord 2
OS: 16.0.2
Inji app version: 0.4.1 (10) -> Testflight
MOSIP Version: <IP_ADDRESS>
MOSIP server: qa-1201-b2.mosip.net
Android is sharing
iOS is requesting
Working as Expected in cross platform sharing device , VC is in card format.


Deleted old Ember settings pages

refs https://github.com/TryGhost/Product/issues/4055

🤖 Generated by Copilot at 3b24693
Removed unused components related to announcement settings, custom theme settings, and file upload. These components were part of a deprecated feature or a legacy UI.

❤


mkcloud, qa_crowbarsetup: show info about jenkins job in /etc/motd

to make it easy to find the job that created a cloud the /etc/motd shows the relevant jenkins information incl. the job URL

this would only address half of issue #759

@bmwiedemann That's exactly what we just discussed on IRC :)

this would only address half of issue #759

issue #759 and PR #760 were created in parallel, so the chance to cover 100% was very small anyway :)

:+1:

+1

+1, thanks!

@jdsn Looks like you manually aborted the gating build - presumably because a full gating run is not necessary?  I guess you connected to vd1 and verified the contents of /etc/motd?

@jdsn Looks like you manually aborted the gating build - presumably because a full gating run is not necessary? I guess you connected to vd1 and verified the contents of /etc/motd?

Nope, because it was blocking for too long. Maybe the github API was down at that moment.

Manually verified that it works (login on admin node).


Rearranged gitignore, minor fixes

.gitignore - rearranged so ignored files stay ignored, added new Mac-specific ignores for things introduced in Mavericks, Yosemite, and El Capitan
TerasologyEngine.java - 1MB is 1048576 bytes, so we should just use that number instead.
ProtobufPersistedData.java - added a few things to (hopefully) aid in debugging
facades/PC/build.gradle - added "debug" task, same as "game" task but starts the JVM in debug mode (useful for IDEs)
It all passed in checkstyle.

Can one of the admins please verify this patch?

ok to test

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/521/
Hooray Jenkins reported success with all tests good!

Merged this in with the ONE_MEBIBYTE piece changed back to the original. I don't mind the Gradle bit left as is since it seems useful with Netbeans and the duplication is minor. Sadly the added parameters aren't as easy to wish for as Gradle does the whole -Psome.Property=value rather than allow for clean parameter submission. Which as actually how we got the fetchModuleX type tasks - the parameter is dynamically built right into the task name.
There might be a way to do that with run task variants and Gradle should be cleaned up more later. Some of that is still even outside my range and I love tweaking at Gradle. It is magic, but sadly sometimes near the dark arcane type. Gradle wizardry?


Fix issue 184

Description
Mainly comment updates, minor improvements for logging etc.
Fixes #184
Type of change

Bug fix (non-breaking change which fixes an issue)
Refactor (refactoring code, removing useless files)

Codecov Report

Merging #234 into master will increase coverage by 0.71%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #234      +/-   ##
==========================================
+ Coverage   40.04%   40.76%   +0.71%
==========================================
  Files          11       11
  Lines        1371     1364       -7
==========================================
+ Hits          549      556       +7
+ Misses        806      799       -7
+ Partials       16        9       -7

Impacted Files
Coverage Δ

server/auth.go
0.00% <ø> (ø)

server/operator.go
96.72% <ø> (ø)

server/profile.go
100.00% <ø> (ø)

server/server.go
26.05% <ø> (ø)

server/trigger.go
100.00% <ø> (ø)

server/cluster.go
93.75% <100.00%> (+1.99%)
:arrow_up:

server/configuration.go
100.00% <100.00%> (+8.16%)
:arrow_up:

Continue to review full report at Codecov.

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


Bring back pdf link and tracking number

This uses underscore templates so that the templates associated with the PDF
download and tracking number are part of the markup (rather than only being
present in the JS)

Cool, will merge this in and style. Thanks!


chore(deps): bump https://github.com/jenkins-x/jx-cli-base-image from 0.0.31 to 0.0.32

Update jenkins-x/jx-cli-base-image from 0.0.31 to 0.0.32
Command run was jx step create pr regex --regex (?m)^FROM gcr.io/jenkinsxio/jx-cli-base:(?P<version>.*)$ --version 0.0.32 --files Dockerfile --repo https://github.com/jenkins-x/jx-git-operator.git

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr-build
f2fa3842e4f11cb7035de000f8cc53f39777dfc8
link
/test this

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr-build
0585ecd14b9f0cd827bf75083c98ef3154533359
link
/test this

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/retest

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr-build
0585ecd14b9f0cd827bf75083c98ef3154533359
link
/test this

pr
6996ac4791d1d654227b63908537a2d13d4a3538
link
/retest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


Gds file

1st version of gds file

EBeam_h-kalagara.pdf

Needs work to be compatible with both fabrication and testing.


 This PR fixes both of those.

We are, but in Github CI symlinks on windows are enabled. On normal Windows PCs they arent.


Unable-to-scan barcode exmples

The barcode scanner doesn't scan these barcodes. We tried on a wide variety of Android versions and smartphones. It didn't work. Can anybody give me some information about these kinds of barcodes?
Thanks in advance.
Which library version are you using?  3.6.0.
Which phone/tablet are you using, and which Android version does it run? (e.g. Samsung Galaxy S5,
Android 5.0, Samsung Galaxy s9 Android 8.0)
Does the same happen on other devices or an emulator?

These appear to be PDF417 barcodes. I could also not scan these using the Barcode Scanner application - not sure if it's an issue with the decoder or something different in the barcodes.
I'd recommend trying a commercial library such as https://microblink.com/products/pdf417 for PDF417 barcodes.


async timer panic

[package]
name = "esp32-test"
version = "0.1.0"
authors = ["lz1998<EMAIL_ADDRESS>edition = "2021"
resolver = "2"
rust-version = "1.71"

[profile.release]
opt-level = "s"

[profile.dev]
debug = true    # Symbols are nice and they don't increase the size on Flash
opt-level = "z"

[features]
default = ["std", "embassy", "esp-idf-svc/native"]

pio = ["esp-idf-svc/pio"]
std = ["alloc", "esp-idf-svc/binstart", "esp-idf-svc/std"]
alloc = ["esp-idf-svc/alloc"]
nightly = ["esp-idf-svc/nightly"]
experimental = ["esp-idf-svc/experimental"]
embassy = ["esp-idf-svc/embassy-sync", "esp-idf-svc/critical-section", "esp-idf-svc/embassy-time-driver"]

[dependencies]
log = { version = "0.4", default-features = false }
esp-idf-svc = { version = "0.47.3", default-features = false }
esp-idf-sys = "0.33"
futures = "0.3"
embassy-futures = "0.1"
esp-idf-hal = {version = "0.42",features = ["nightly"]}
embedded-hal-async = "1.0.0-rc.1"

[build-dependencies]
embuild = "0.31.3"

use esp_idf_hal::timer::TimerConfig;
use esp_idf_svc::eventloop::EspSystemEventLoop;
use esp_idf_svc::hal::peripherals::Peripherals;
use esp_idf_svc::nvs::EspDefaultNvsPartition;
use esp_idf_svc::timer::EspTaskTimerService;
use esp_idf_svc::wifi::{AsyncWifi, AuthMethod, Configuration, EspWifi};
use esp_idf_sys::EspError;

fn main() {
    // It is necessary to call this function once. Otherwise some patches to the runtime
    // implemented by esp-idf-sys might not link properly. See https://github.com/esp-rs/esp-idf-template/issues/71
    esp_idf_svc::sys::link_patches();

    // Bind the log crate to the ESP Logging facilities
    esp_idf_svc::log::EspLogger::initialize_default();

    log::info!("Hello, world!");
    let res = embassy_futures::block_on(init());
    log::info!("{res:?}");
}

async fn init() -> Result<(), EspError> {
    let peripherals = Peripherals::take()?;

    let mut timer = esp_idf_hal::timer::TimerDriver::new(peripherals.timer00, &TimerConfig::new())?;
    for i in 0..100 {
        log::info!("{i}");
        timer.delay(timer.tick_hz()).await?;
    }

    // let sys_loop = EspSystemEventLoop::take()?;
    // let timer_service = EspTaskTimerService::new()?;
    // let nvs = EspDefaultNvsPartition::take()?;
    // let mut wifi = AsyncWifi::wrap(
    //     EspWifi::new(peripherals.modem, sys_loop.clone(), Some(nvs))?,
    //     sys_loop,
    //     timer_service,
    // )?;
    // connect_wifi(&mut wifi).await?;

    Ok(())
}

E (5345) task_wdt: Task watchdog got triggered. The following tasks/users did not reset the watchdog in time:
E (5345) task_wdt:  - IDLE (CPU 0)
E (5345) task_wdt: Tasks currently running:
E (5345) task_wdt: CPU 0: main
E (5345) task_wdt: Print CPU 0 (current core) registers
Core p8 register dump:
MEPC    : 0x40385a60  RA      : 0x40388f76  SP      : 0x3fc90780  GP      : 0x3fc8ac00
0x40385a60 - vPortClearInterruptMask
    at /Users/lizheng/RustroverProjects/esp32-test/.embuild/espressif/esp-idf/v5.1.1/components/freertos/FreeRTOS-Kernel/portable/riscv/port.c:385
0x40388f76 - __atomic_compare_exchange_4
    at /Users/lizheng/RustroverProjects/esp32-test/.embuild/espressif/esp-idf/v5.1.1/components/newlib/stdatomic.c:313
0x3fc90780 - _btdm_common_start
    at ??:??
0x3fc8ac00 - uxTopUsedPriority
    at ??:??
TP      : 0x3fc7190c  T0      : 0x42033e4a  T1      : 0x42003022  T2      : 0x00000001
0x3fc7190c - __eh_frame_hdr
    at ??:??
0x42033e4a - core::fmt::Formatter::pad_integral
    at ??:??
0x42003022 - alloc::alloc::dealloc
    at /Users/lizheng/.rustup/toolchains/nightly-x86_64-apple-darwin/lib/rustlib/src/rust/library/alloc/src/alloc.rs:117
0x00000001 -
    at ??:??
S0/FP   : 0x00000001  S1      : 0x3fc907a0  A0      : 0x00000001  A1      : 0x3fc907a0

can you please specify what esp32 you are using specifically? a normal esp32 running on xtensa?

can you please specify what esp32 you are using specifically? a normal esp32 running on xtensa? Also please clarify what version of the toolchain you are using with rustc --version

[build]
target = "riscv32imc-esp-espidf"
[target.riscv32imc-esp-espidf]
linker = "ldproxy"
runner = "espflash --monitor" # Select this runner for espflash v1.x.x
runner = "espflash flash --monitor" # Select this runner for espflash v2.x.x
rustflags = ["--cfg", "espidf_time64", "-C", "default-linker-libraries"]
[unstable]
build-std = ["std", "panic_abort"]
[env]
MCU="esp32c3"
Note: this variable is not used by the pio builder (cargo build --features pio)
ESP_IDF_VERSION = "v5.1.1"

If I use esp_idf_hal::task::block_on instead of embassy_futures::block_on, it will not panic. Is it expected?

expected? No,
We currently have lots of moving parts with embassy support, there were recently some atomic changes introduces, that also changed inside our crates and also in embassy, this is potentially an outfall here.
Before recently the most tested way was to use the edge-executor as a base to run async code, but now we should be in a good position to allow other runners including bigger ones like tokio, with the advent of the possibility to notify waker from ISR context.
@ivmarkov seeing __atomic_compare_exchange_4 in his crash, could this related to the recent changes @Dirbaio made in embassy to help the no_std guys back on track?

This is not a panic - look at the top of the stack trace and google "esp idf twdt".  And NO you should not be using embassy_futures::block_on. Read the comment above that function as to what it is doing, and that would hopefully explain why the task watchdog is triggering.

expected? No, We currently have lots of moving parts with embassy support, there were recently some atomic changes introduces, that also changed inside our crates and also in embassy, this is potentially an outfall here.

This behavior is completely normal and expected, as per above.

expected? No, We currently have lots of moving parts with embassy support, there were recently some atomic changes introduces, that also changed inside our crates and also in embassy, this is potentially an outfall here.

This behavior is completely normal and expected, as per above.

tanks for pointing it out,  because of all the atomics stuff these days, they magically grab my attention currently making me blindsided :smiling_face_with_tear:


.sendActivities is not working within prompt validation

Versions
What package version of the SDK are you using: botbuilder botbuilder-dialogs 4.2.1.
What nodejs version are you using: node v10.14.2, npm 6.8.0.
What browser version are you using:...the Emulator? v4.2.1.
What os are you using: Windows 10 x64
Describe the bug
From within a prompt validator, sendActivity(x) works as expected, but sendActivities([x]) does not,
whether x is an activity or a string.
To Reproduce
Steps to reproduce the behavior:

Create a bot that runs a dialog that calls a prompt that includes validation.
In the validation function, call the PromptValidatorContext object's .context.sendActivities method.
Run the bot, trigger the prompt, and trigger the condition for sendActivities.
See error: no messages are sent back to the user in the Emulator.

Expected behavior
sendActivities should work here as advertised, with a string or activity in an array.
Additional context
These are excerpts from the code I'm using:
        // Create the dialog set and add the prompts, including custom validation.
        this.dialogSet = new DialogSet(this.dialogStateAccessor);
        this.dialogSet.add(new NumberPrompt(SIZE_RANGE_PROMPT, this.rangeValidator));
        this.dialogSet.add(new ChoicePrompt(LOCATION_PROMPT));
        this.dialogSet.add(new DateTimePrompt(RESERVATION_DATE_PROMPT, this.dateValidator));

        // Define the steps of the waterfall dialog and add it to the set.
        this.dialogSet.add(new WaterfallDialog(RESERVATION_DIALOG, [
            this.promptForPartySize.bind(this),
            this.promptForLocation.bind(this),
            this.promptForReservationDate.bind(this),
            this.acknowledgeReservation.bind(this),
        ]));

    async promptForPartySize(stepContext) {
        // Prompt for the party size. The result of the prompt is returned to the next step of the waterfall.
        return await stepContext.prompt(
            SIZE_RANGE_PROMPT, {
                prompt: 'How many people is the reservation for?',
                retryPrompt: 'How large is your party?',
                validations: { min: 3, max: 8 },
            });
    }

    async rangeValidator(promptContext) {
        // Check whether the input could be recognized as an integer.
        if (!promptContext.recognized.succeeded) {
            await promptContext.context.sendActivity(
                "I'm sorry, I do not understand. Please enter the number of people in your party.");
            return false;
        }
        else if (promptContext.recognized.value % 1 != 0) {
            await promptContext.context.sendActivity(
                "I'm sorry, I don't understand fractional people.");
            return false;
        }

        // Check whether the party size is appropriate.
        var size = promptContext.recognized.value;
        if (size < promptContext.options.validations.min
            || size > promptContext.options.validations.max) {
            // This doesn't work:
            // await promptContext.context.sendActivities([
            //      'Sorry, we can only take reservations for parties of '
            //      + `${promptContext.options.validations.min} to `
            //      + `${promptContext.options.validations.max}.`,
            //      promptContext.options.retryPrompt]);

            // This works:
            // await promptContext.context.sendActivity(promptContext.options.retryPrompt);

            // This works:
            //await promptContext.context.sendActivity(
            //    'Sorry, we can only take reservations for parties of '
            //    + `${promptContext.options.validations.min} to `
            //    + `${promptContext.options.validations.max}.`);

            // These don't work either:
            // await promptContext.context.sendActivities([promptContext.options.retryPrompt]);
            await promptContext.context.sendActivities(["Testing..."]);
            return false;
        }

        return true;
    }

Really, this only comes up because I don't get to send a message from the validator and also get the prompt object to display the reprompt.
[bug]

Note: this is blocking updates to the docs to address #680.

Hi @JonathanFingold,
We have been researching on the usage of the methods sendActivity & sendActivities, from the libraries Builder-Core & BotBuilder respectively. As we can see, the first method's firm allows to send a string or a partial Activity:

By the other hand, we have the method sendActivities (which is the one with you are having problems), which only allows using a list of partial Activities:

But this method doesn’t allow using a list of strings as you do.
 // This doesn't work:
 // await promptContext.context.sendActivities([
//      'Sorry, we can only take reservations for parties of '
//      + `${promptContext.options.validations.min} to `
//      + `${promptContext.options.validations.max}.`,
//      promptContext.options.retryPrompt]);

A possible workaround to solve it consists of creating two (or more) separated activities as we did in this screenshot and pass them to the method sendActivities:

However, we think that the best approach is to allow that sendActivities could receive both a list of string and a list of Partial Activities. So, to accomplish this, we will be creating a new issue requesting these changes.

@GasparAcevedoZainSouthworks , please link to the new issue, once it's open, and then close this issue.
Thanks much.

Hi @JonathanFingold,
Despite yesterday we said that the best approach to solve this issue could be creating a new ISSUE requesting that the sendActivities could receive both an activity list and a list of string, we continue researching on this topic and we found that:

In the C# version, the method sendActivities only allows a list of activities, not a list of strings.

In this sample in JS, we have not found references for the usage of sendActivities, but instead, it uses sendActivity.

This means that allowing sendActivities to receive as a parameter a list of string is not a good approach.
So, in order to fix this ISSUE you could:

Instead of using sendActivities with a list of string, make a call to sendActivity for every string on the list. Or,
For each string, create a new Partial, and pass it to sendActivities as we do in any of these screenshots:

Or:

Please, let us know if any of these approaches solve your problem, so we can close the ISSUE.

I'll make a separate suggestion to overload MessageFactory.Text to accept an IEnumerable parameter. Thanks.


⚠ ClamAV has degraded performance

In 681b94a, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 441 ms

Resolved: ClamAV performance has improved in 216caa6.


TypeError: Cannot read properties of null (reading 'status')  at SSE._onStreamFailure

This error happens when the connection to a server is refused. For example if you don't have proper CSP or the server just refuses.
In the method:
this._onStreamFailure = function(e) {
    const event = new CustomEvent('error');
    event.responseCode = this.xhr.status;
    event.data = e.currentTarget.response;
    this.dispatchEvent(event);
    this._markClosed();
  }

The xhr object seems to be null:
at SSE._onStreamFailure (webpack-internal:///./node_modules/sse.js/lib/sse.js:155:35)

Thanks for reporting! I'll take a look 👀

Thank you for such a quick fix, man! 🫡

Could I ask, how can I try-catch this in my codebase? I am trying to wrap the entire EventSource creation + callback setup in a try catch but when this library fails (reading .status on a null object) it still is thrown as a runtime exception and I cannot gracefully fail in the UI.
@mpetazzoni

You might be able to do that with https://developer.mozilla.org/en-US/docs/Web/API/Window/error_event ?

You're right, I am able to catch it with that :) Thanks!

I'm returning to this issue. The xhr might be fixed but now, the .status is also null.
Starting a SSE connection while offline raises and exception:
Uncaught TypeError: Cannot read properties of null (reading 'status')
    at SSE._onStreamFailure (sse.js:155:35)

@mpetazzoni


Error when compiling with luatex.

! Undefined control sequence.
l.78 \DeclareUnicodeCharacter
{00E9}{'{e}}
?
! LaTeX Error: Missing \begin{document}.
See the LaTeX manual or LaTeX Companion for explanation.
Type  H   for immediate help.
...
l.78 \DeclareUnicodeCharacter{0
0E9}{'{e}}
?
Package hyperref Info: Option breaklinks' set true' on input line 114.
luaotfload | db : Reload initiated (formats: otf,ttf,ttc); reason: "File not fou
nd: fonts/SourceSansPro-Light.".(save: /Users/krishna.singh/Library/texlive/2018
/texmf-var/luatex-cache/generic/fonts/otl/sourcesanspro-light.lua)(save: /Users/
krishna.singh/Library/texlive/2018/texmf-var/luatex-cache/generic/fonts/otl/sour
cesanspro-light.luc)

and someother errors
Underfull \hbox (badness 10000) in paragraph at lines 15--11
[]
Underfull \hbox (badness 10000) in paragraph at lines 20--20
[]
LaTeX Font Info:    External font cmex10' loaded for size (Font)              <8> on input line 26. LaTeX Font Info:    External font cmex10' loaded for size
(Font)              <6> on input line 26.
Underfull \hbox (badness 10000) in paragraph at lines 26--26
[]
) (./section_references.tex
Overfull \hbox (17.99348pt too wide) in paragraph at lines 21--21
[][]
[]
) [2]

It looks like the SourceSansPro-Light font is missing... Could you please check that the font is present ?


[Tool] separate starrocks_test obj files from executable

avoid rebuild all files when the link flags changed to the executable

Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[X] UT
[ ] Doc
[X] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[X] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[X] I have checked the version labels which the pr will be auto-backported to the target branch

[X] 3.2
[X] 3.1
[X] 3.0
[ ] 2.5

[FE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

@mergify backport branch-3.2 branch-3.1 branch-3.0


feat: Implement Logger

Implement Logger

This PR implements Logger, which logs certain messages depending on the log level configured.
Log level contains of:

Info: Default log level, only logs informational messages.
Debug: Logs all incoming requests, and responses to help debugging process.
Off: Doesn't log anything at all.

I will try to update this PR accepting @HurSungYun 's recommendations soon!

Please reopen the pull request as it is no longer being followed up


Add .toString() method for Color objects

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
All new code requires tests to ensure against regressions

Description of the Change
Color had a lot of helpful to* methods, but it was missing arguably the most important one: toString().  This fixes the Settings Panel from displaying [object Object] as the default value for a color:

Alternate Designs
I chose to use this.toRGBAString() because the Windows color picker does not have a hex picker.  Alternate designs include creating a more human-readable presentation such as Red: 255, Green: 255, Blue: 255, Alpha: 1 or using this.toJSON(), but then the default will be fairly useless for Windows users.  However, it would appear that Linux's default color picker only has hex input: https://github.com/atom/settings-view/issues/520.  Perhaps both should be used, such as rgba(255, 255, 255, 1) (#FFFFFF)?
Why Should This Be In Core?
Color is an essential class and it makes sense for it to support .toString().
Benefits
More helpful string representation for Settings View.
Possible Drawbacks
None.
Applicable Issues
None.

Update file


Publish on NPM

Greetings! Would you mind publishing on NPM as well as Bower? Seems like most of the front-end JS community is being driven in this direction. Angular and jQuery projects are now being published (or will be published) to NPM.
Moving forward, it's making sense to use Bower less and more of a NPM + Browserify type of workflow. Thank you!

Yes, I'm planning to do this, since it seems that NPM is the future.
I don't have a date yet for this, now I'm pretty busy with a huge update for Videogular 1.1.0 that will have cue points. I'm so excited with this new release, you guys are going to love cue points! :D

Thank you! For anyone else looking for this, until the packages are published to NPM you can install via the github url like: npm i 2fdevs/bower-videogular.
Have you considered publishing the CSS too?

Yes, we have separated repositories for Videogular core, the plugins and the themes. Take a look here:
https://github.com/2fdevs?tab=repositories

Not quite sure that will actually work.
As far as I can tell, if I want multiple packages (installed via NPM), I would do something like this:
npm i 2fdevs/bower-videogular --save
npm i 2fdevs/bower-videogular-overlay-play --save

Because the name in both package.json files is "videogular," npm isn't picking up the packages as separate, and is just replacing the contents of node_modules/videogular whenever additional packages are installed.
I think the solution is an easy one, just edit the package.json files for each of the bower builds to have separate, descriptive names.

I've fixed the bower repositories, just tell me if it works.

@2fdevs It works, files are ending up in different folders.

Thanks @xErik for testing :)
Closing issue.

We've published on NPM at the end.
https://www.npmjs.com/package/videogular
npm install videogular

It seems as if the videogular module is not properly exported?
I'm not able to use this with my Browserify setup.
The 'main' file in package.json is the gruntfile.

I will change it un the next release

It would also be really awesome to export the name of the modules as well so they can be imported and listed in an app's array of required modules all in one go, like so:
angular.modules('myApp', [
  require('videogular')
])

You can see how this was done in Angular Route or any of the other angular sub-modules.

Sorry if I sound spoiled, but is there any ETA on this issue or the next release?

No problem, I understand :)
I want to do a release this next weekend with a few small fixes.

Fixed.
Added all plugins on NPM and added support for browserify.
I've created an example repository: https://github.com/2fdevs/videogular-browserify
See the changelog to see the complete list of changes:
https://github.com/2fdevs/videogular/blob/master/CHANGELOG.md


pxelinux doesn't support UEFI

Issue description
pxelinux (from syslinux) is compiled without support for UEFI PXE boot: https://github.com/NixOS/nixpkgs/blob/master/pkgs/os-specific/linux/syslinux/default.nix#L47
I had a quick look at getting it to compile uefi support but I didn't get very far.
Technical details

system: "x86_64-linux"
host os: Linux 4.16.11, NixOS, 18.09.git.c2a4a5143f2M (Jellyfish)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.0.2
channels(root): ""
channels(peter): ""
nixpkgs: /home/peter/src/active/nixpkgs_unstable

Does the NixOS netboot image support UEFI boot?  I did try netbooting NixOS (18.03) a few weeks ago with a pre-compiled iPXE UEFI loader. It started booting the kernel but failed to mount the initrd (kernel panic no root).

The image is using pxelinux.0 instead of lpxelinux.0 it seems and you need lpxelinux for HTTP support so maybe that's what is going on.

The http support seemed to worked just fine in my case. I did serve  the iPXE UEFI loader via tftp, the kernel image and the initrd via http (initrd is too big to load trouble-free via tftp). The iPXE loader pulled bzImage and initrd via http and began booting the kernel. The kernel then failed to mount the ramdisk for some reason I do not understand.
I solved the problem by setting 'Legacy mode' in the firmware (Supermicro) and booting everything in classical BIOS/PXE mode (no UEFI components involved). I can try to reproduce the problem and send you some screenshots if that helps.
My vague guess is that the NixOS netboot image needs some modifications to be UEFI capable.
I just wanted to raise awareness that there are potential problems with UEFI-PXE down the road.

IIRC, specifying initrd=initrd on the kernel command line is required with EFI booting for some reason.

#85094 would fix this.


[WIP] Add basic twitch emote picker

This adds a basic emote picker.
I believe there are some more things that can be improved:

I'm not sure if im doing state properly...
Types on the emote retrieval function.
Emote retrieval is handled by TMI using the kraken endpoint afaik.
Parsing the function response.
Closing the emote picker when pressing back?
Add a delete and send button when using the emote picker?
Caching?
I believe there is an overflow issue.
Landscape view does not match the keyboard.
Dark mode

Some screenshots of the current progress:

Added a dismiss and delete button, what do you think?


*: implement debug for DB

So that we can use #[derive(Debug)] for structs that contains DB.
@siddontang @zhangjinpeng1987 PTAL

LGTM

LGTM


Reconsider $(basename) in file/search exclude

The $(basename) expansion gets removed from tooltips as part of codicon processing. In file nesting I got out of this by using a nonbreaking space between the $ and ( but it's a bit of a hack (need to strip nonbreaking spaces from the value in core in case people copy/paste).

File nesting is moving over to ${...} expansions, which are used heavily in core. Should excludes perhaps also adopt that style?

Not really sure who to cc since the code has been around since the beginning of time... @bpasero @roblourens ?

@JacksonKearl just to clarify, you suggest to align the explorer file nesting syntax with our basename syntax in glob patterns? I don't think we can really change the syntax for glob patterns, it is a somewhat established pattern for many users and it would just ask for breakage.
Maybe you could clarify the intent of this issue further.

Explorer file nesting is already moving over to use ${basename}/${extname}/${dirname}/etc.. I think at minimum we should change the description text of the excludes to use $\u200b(basename) so it doesn't get swallowed by the codicon replacer. cc @aeschli

Yeah that sounds good to me, I wasn't aware the hover was hiding those.

Use \\$(..) to avoid an icon is rendered

@aeschli looks like it doesn't work in markdown sections:

but the plaintext rendering works well enough for my use case.


Error: unable to verify the first certificate when trying to dump the data to another cluster

Hello,
I'm getting this error when I try to copy from one host to another:

Wed, 06 Apr 2016 05:15:26 GMT | starting dump
Wed, 06 Apr 2016 05:15:28 GMT | Error Emitted => unable to verify the first certificate
Wed, 06 Apr 2016 05:15:28 GMT | Total Writes: 0
Wed, 06 Apr 2016 05:15:28 GMT | dump ended with error (get phase) => Error: unable to verify the first certificate

I reckon it's because of the certificate of the input cluster. Is there a way right now to bypass the certificate verification like the -k option in curl?

try some of the options discussed here http://stackoverflow.com/questions/10888610/ignore-invalid-self-signed-ssl-certificate-in-node-js-with-https-request

@evantahler cool, thanks. I was initially wondering whether there's a builtin option in here for that.
If anyone ended up here looking for a solution for this, I ended up using this:
export NODE_TLS_REJECT_UNAUTHORIZED=0


Please have the generator sort yaml

please sort the teams, members of teams, and repos by alpha so the yaml is easier to read.

Done 🙂


Programs IO operations

Provide a system for serializing/deserializing movemaster programs written in IDE or uploaded from manipulator in chosen format.

Chosen format: txt


MLA In-Text Citations

Hi. Thank for your works.
Here is my example https://codesandbox.io/s/citation-js-format-sample-forked-r2l5fs?file=/src/App.js
where I try to make citations in diff formats. Such some custom MLA citation format from file
https://github.com/citation-style-language/styles/blob/master/modern-language-association.csl
In doc https://owl.purdue.edu/owl/research_and_citation/mla_style/mla_formatting_and_style_guide/mla_in_text_citations_the_basics.html
MLA format use page  In-text citations, in my exaple in JSON I have page: "5441-5444" field for this
I ask about this case https://github.com/citation-style-language/styles/issues/6667
they say:
 I've just added a locator to the CSL JSON in your snippet, but that didn't work. Might be that citation-js is just not handling locators?
Can u please help me? how to pass page in MLA format  In-Text Citations?

You have added page: "5441-5444" but that field specifies the page numbers of the entire article. To include page numbers in a citation, use the following snippet (adapted from your example):
citationMLA: reference.format("citation", {
  template: "mla",
  entry: [
    {
      id: "Q922203",
      locator: "5441-5444"
    }
  ],
  format
}),

@larsgw thx a lot! works amazing!

Great!


[WIP] Improve the performance of tree listing

Add two query strings start and count for the home page of repository.  If the values are omitted or invalid they are set to 0 and the total number of rows respectively.
For #502

When count is not set it shows no commits. When coun/startt > commits Gitea panics

Please mark this as work-in-progress, I will finish this PR later (adding front-end controls as @lunny suggested).
@Bwko Thanks for pointing it out.

@typeless Good work :+1:  Only the sorting is alphabetically and not folder first.

@Bwko My bad 😆 , didn't notice the nuances.

@typeless Is this PR ready to review?

@lunny The UI part is still missing if that is desired to be included in this PR.

@typeless UI would be required yes, otherwise there may be issue in the future :( Would be nice to have it fetch and render the next set of files via API, like GitHub has load more entries... when there's to many files 🙂

You could also just include the base/paginate.tmpl, but then you need to use ?page=0 and a static count 🙁

@typeless Is this completed?

@lunny I don't have a good idea about how the UI design should be done, before starting to get familiar with Semantic UI. Please bear with me as my spare timeslices for OSS projects are very limited recently.
If someone is willing to help implement the web UI (I am really not good at it), that would be appreciated.

OK. Please give the permissions to let maintainers to push to your branch, it's in this PR right sidebar. And I will move this to v1.2.

@lunny  Thanks. I have the "allow edits from maintainers" checked.

@typeless @lunny I can look at implementing lazy-loading during the weekend 😄

@bkcsoft any update?

@lunny I've been swamped with work unfortunately 🙁


WIP: Meshed magnetic field

Work-in-progress addition to the MagneticFieldReader module to allow loading from mesh files, much like in the ElectricFieldReader case.
Currently working:

Loading a set of B-field 3-vectors from a file
Making sure GEANT4 reacts to the non-constant field across the world volume
Fallback config option for a constant field outside the specified mesh volume (defaults to 0T)

Still to do:

Config option to specify the volume centre (currently defaults to centring the mesh about the origin)
Config option to specify volume start (e.g. pinning bottom-left corner of mesh to a coordinate, rather than centring the mesh about a location).
Create output plots of loaded field
Documentation and tests

Very happy to receive comments and suggestions on the current implementation!

Thanks for your comments and review @simonspa ! I've implemented some of your changes, but left a query about the fallback field value in the review conversation above. Documentation etc. is added to my to-do list now!

Hi @sam-sw - is there any chance you come back to this great work and tackle the last few things missing for a merge?

Hi @simonspa! I have been quite busy, but should have some time in the next two or three weeks to look at this again.

That's awesome! Looking forward to it!


chart values are using average need to use weighted average

need to sum of area  then sum of score.
not mean.
then calculate the score from the sum of area/sum of score? not sure yet.


Add some scripts that let us build reproducible tarballs.

Ref #74

This still needs a bit of work before we can switch to using it, but it's in a good spot now.

PTAL

Switched this over to use a custom rule. Should be much easier to understand now.


Site crash Detector : Add Application Events query in data source to figure out crash

Current site: https://applensv2.azurewebsites.net/sites/PushNotificationService/detectors/sitecrashes?startTime=2016-12-11T20:58:11.512Z
Please provide feedback here:
For SiteCrash Detector,  is it possible for us to add a query for Application Events in the “Data Source” section or have some other indication on why the crash occurred?
In this case, the report correctly identifies the crash and in the Application events table, I see that  the site crashed due to Access Violation. It would be helpful if we had some way of making the engineer aware of this issue thru  Applens :
ApplicationEvents | where (TIMESTAMP >= datetime(2016-12-12T02:30:12Z) and TIMESTAMP < datetime(2016-12-12T17:35:12Z)) |  where Tenant == '6966a9f2a88348b581d88203f0c32536' | order by TIMESTAMP asc| where RoleInstance==@'SmallDedicatedWebWorkerRole_IN_215'
Faulting application name: w3wp.exe, version: 8.0.9200.16384, time stamp: 0x5010885f
Faulting module name: clr.dll, version: 4.6.1590.0, time stamp: 0x5787ee4f
Exception code: 0xc0000005
Fault offset: 0x0002499e
Faulting process id: 0x1e70
Faulting application start time: 0x01d2541e3f268e8e
Faulting application path: D:\Windows\SysWOW64\inetsrv\w3wp.exe

More info:
Faulting module path: D:\Windows\Microsoft.NET\Framework\v4.0.30319\clr.dll
Report Id: 964f1755-c016-11e6-9407-00155d85a80b
Faulting package full name:
Faulting package-relative application ID:
Corresponding Kusto query for crash  (  Process ID 7792) :
https://wawseus.kusto.windows.net:443/wawsprod [Run in Kusto.Explorer] [Run in Kusto.WebExplorer]
AntaresRuntimeWorkerEvents | where (TIMESTAMP >= datetime(2016-12-12T02:30:12Z) and TIMESTAMP < datetime(2016-12-12T03:35:12Z)) |  where Tenant == '6966a9f2a88348b581d88203f0c32536' | order by TIMESTAMP asc| where RoleInstance==@'SmallDedicatedWebWorkerRole_IN_215'


Allow static library usage from Cocoapods

Version 1.4.0 of Cocoapods allows a pod to be used as a static library, even in a Swift app. It requires the library's podspec to contain s.static_framework = true.
I believe this will help apps launch faster. Here's a blog post that explains why too many dynamic frameworks slows down launch: https://blog.automatic.com/how-we-cut-our-ios-apps-launch-time-in-half-with-this-one-cool-trick-7aca2011e2ea

Hi @bridger, thanks for the update. If we add s.static_framework = true to the podspec file, what will happen to people who want to continue using our library the old way? Will it break things?

Honestly, I'm not sure... I guess the dynamic framework is nice if you don't want to repeat the binary size in each extension, like an Apple Watch binary that also uses Keychain?
For keychain-swift I'd imagine that binary size concern would be tiny. If people still needed to, they could make their own dynamic framework target which depends on keychain-swift then make their app and watch extension depend on that framework.

That's good to know, thanks. I will do some Googling to see if there are any side effects that can affect our current users. In addition, I wonder if there is a way do switch to static framework on the client side, in Podfile instead of doing it for everyone in the podspec of the library?

For the referece, I've asked about the side effect of static_framework=true on stackoverflow.
I can see two potential problems:

One possible issue for us would be other dynamic frameworks that depend on keychain-swift. We may break those frameworks if we add static_framework = true, since dynamic frameworks can not depend on static ones (they say).

If we add static_framework = true to the podspec, we will impose this for all users. But some users may not want that. I wonder if there is a way for the user to decide if they want the library to be static or dynamic?

Thanks for digging into this! Sorry it is more complicated than I thought it would be.
It would be awesome if CocoaPods allowed the app to choose whether it is imported statically or dynamically, but that option doesn't exist. I think Carthage allows the user of the library to choose. Let's see if the CocoaPods peeps will comment on what to do.


Using multiple themes

Currently, I attempting to use this project's multiple themes functionality to create a dark and light mode.
While trying to do this am I facing issues with sending duplicate .js files, as the tag has been used twice.
With this .js being served twice, it is creating two event handlers for the navbar burger menu on the page and is causing it to stop working.
Django template:
{% bulma %}
{% bulma 'dark' %}

Firefox network tab when loading the page:

I am unsure how Django extensions work, but having a way to ensure that only one .js file is served would allow me to implement the functionality I want.

This functionality for switching themes could be implemented into this package also, as it would be a nice to have for other users of django-simple-bulma

I believe in https://github.com/lemonsaurus/django-simple-bulma/blob/main/django_simple_bulma/templatetags/django_simple_bulma.py#L39 a condition would need to be added for only optionally including the JS files?

Yes @jchristgit, that sounds totally fine. We could do that.


Match commands that begin with whitespace

Is your feature request related to a problem? Please describe.
I like to prepend my commands with spaces in order to not save them to the history and prevent clutter. However, no suggestions, besides those that begin with whitespace, are given when I type commands with leading whitespace.
Describe the solution you'd like
An option that can be enabled that removes the leading whitespace from a typed command and matches the rest of the command with an entry in the history.
Describe alternatives you've considered
I've considered not prepending commands with spaces so that the suggestions work.
Additional context
I'm not familiar with ZSH scripting or else I'd give it a go. I think that this enhancement would interfere with history entries that begin with whitespace.

I have no solution, but a workaround that I am using (for commands that I don't want to go into the history e.g. containing passwords)

Type command and use completion feature
Hit CTRL-A which should place the cursor at the beginning of the line
Add the space and execute away

Maybe this helps you (it works for me)

Thanks for your comment @AnimiVulpis. That's the same workaround I've been using. I'd just be a nice enhancement to have

FWIW, here's what I've been using: https://github.com/stepnem/zsh-autosuggestions/commit/ce7ad0e8d4579f6e409b26683142fb1f2239ab51
Maybe it's not the best way to do it (I'm not that familiar with the
code), but it works for me.
I didn't introduce a new option, as I don't see why anyone would
prefer the current behavior.

@stepnem Your commit was the solution I was looking for. Thanks!


feat(github-gateway): add endpoint for creating github webhook

Ping @adamreese - what's the status of this PR? Since we plan on moving to the new Checks API app, is this change still actual?
Thanks!


Knex Version requirement increased to 0.13

Testing a fix to #1570

Thank you! sorry I was never able to get around to this


[11.x] Reverts 49764

This pull request reverts https://github.com/laravel/framework/pull/49764, as we are planning to keep using Carbon 2 for Laravel 11 and use Carbon 3 (once is stable) for Laravel 12.
At the moment, dozens / hundreds of packages don't support Carbon 3 - so this could potentially become the biggest pain point while upgrading applications to Laravel 11.

@kylekatarnls I would definitely like to think this whole thing over a bit more. Carbon is such a widely used package that we fear the breaking changes / cost of forcing every Laravel package out there to ensure Carbon 3 compatibility is big ask with minimal return on investment in terms of improved overall user experience or significant new functionality in the package.

I understand. On the other side, if Laravel 12 is coming in February 2025, at this date Carbon 3 would have been there stable for months and it will be too late to make BC on this version (if a BC would be needed to make a Laravel package compatible in a smoother manner).
Also in the meantime, probably many packages would start to require carbon ^3 and those packages won't be installable inside Laravel 11 or any project with a dependency to illuminate/support 11.
Would you consider a PR allowing ^2.72 || ^3.0? The pro is that it's probably the smoother transition for package to upgrade during the year without painful conflicts. The con is that some output might slightly vary depending on the installed version (for instance $foo->diffInSeconds($bar) should be written (int) $foo->diffInSeconds($bar, true) to get Carbon 2 behavior whatever is the version installed, or $foo->floatDiffInSeconds($bar, false) to get the Carbon 3 behavior whatever the version installed.

One last point that can make it hard to wait 1 year is that now Laravel uses some symfony 7 dependencies, but Carbon 2 (because of the support of PHP 7 and dependency to symfony/translation) cannot be compatible with both older and new versions of Symfony. Having a mix of Symfony 6 and 7 packages is not in theory a blocker as long as they don't have overlap in sub-dependencies, but by the past we had some users reporting issues when having a particular mix of versions.

#49847 proposed as an alternative.


feat(89): correct naming for storage_table

Fix the naming for storage tables to start with an alphabetic character followed by up to 62 alphanumeric characters.  No hyphens allowed.

@microsoft-github-policy-service agree

If provisioned with the previous roles would this failed to create the resource?


Add Error Reporting gRPC

This is a tracking work item to get the Error Reporting GAPIC into this library:
https://pypi.python.org/pypi/gapic-google-cloud-error-reporting-v1beta1
@steren and I agree that the existing (small) report and report_exception is what we want to keep, along with possible more framework integrations in the future. So I think what we primarily want is the same thing as with done with Logging where we add the GAPIC as a dependency, and then fallback to the existing code if gRPC is disabled.
One small question is if and to what extent we want to expose the rest of the GAPIC besides events.report via this library. There are a few API calls that will likely only be used by power users, but it might be worth having some sort of pass-through mechanism. Unless I'm mistake, it looks like so far we either wrap all the GAPIC calls or pass-through completely, but we haven't done both, so not sure exactly how this would look.
cc @bjwatson @jgeewax @omaray in case you have comments.

@jskeet IIRC, you have experience doing thin and/or partial hand-written layers on the C# GAPIC code. To date, we have done thick hand-written layers for Python. Do you have any advice for how best to manage thinner layers?
cc @anthmgoogle

Hmm... it's likely to be language-specific. In C#, we can add extra methods/properties/etc to existing classes using partial... I don't know what the equivalent is in Python. But aside from anything else, we've already decided that all the GAPIC code is exposed in C#; is that true in Python?
If you're happy to expose the GAPIC code for power users, you could have a very simple wrapper that exposes simple operations but also exposes a property to access all the GAPIC methods if necessary.

Thanks @jskeet. Makes sense how you're leveraging the C# partial feature.
Since Python is package delivery (i.e. the GAPIC code has its own package on https://pypi.python.org/pypi), the GAPIC code is always exposed to our users.

Ok, now that I am thinking about it, if it's a dependency of this library it will be exposed after installing this library, that makes sense.
I am leaving for Brazil tomorrow and will be busy with GCP Next Sao Paulo but I can loop back on this later in the month.

Yep, the other google-cloud-python libraries that depend on GAPIC call it out as a dependency in their setup.py.

google-cloud-python contains the Error Reporting instrumentation library, but not the Error Reporting client library.
Is this issue about bringing the client libraries in? Is anybody working on it?
(Context: I was looking to call the Error Reporting "list error stats" API today, but could not do it in Python or Node)

Hello @steren!
Not sure I'm following entirely.
Do you mean https://cloud.google.com/error-reporting/reference/rest/v1beta1/projects.groupStats/list#errorgroupstats?

Hi @steren. There is no hand-written client library for Error Reporting; we are using just the auto-generated client library (plus the hand-written instrumentation code). Also for Python, we publish the auto-generated client libraries separately from the google-cloud-python libraries.
The client library is published at https://pypi.python.org/pypi/gapic-google-cloud-error-reporting-v1beta1, and you can find the source code at https://github.com/googleapis/api-client-staging/tree/master/generated/python/gapic-google-cloud-error-reporting-v1beta1.

My main concern is that https://googlecloudplatform.github.io/google-cloud-python/ does not mention or document the client library.
My question is: Is this GitHub issue about adding the client library to google-cloud-python? Or is it something else?

I think this issue is best resolved by adding a dependency on the GAPIC package to https://github.com/GoogleCloudPlatform/google-cloud-python/blob/master/error_reporting/setup.py.
The documentation issue is covered by https://github.com/GoogleCloudPlatform/gcloud-common/issues/207.

I think it's probably easiest if @waprin just answers here and we stop talking past each other?
The implementation in google-cloud-python (which is the project this issue tracker is for) was done exclusively by @waprin. AFAICT it use the Stackdriver Logging API directly to "report errors".

So a few things need to be done here:

Add the dependency on the GAPIC as @bjwatson says. That's pretty easy (#2894) but as @steren points out, there are no docs. The gcloud-common#207 issue still seems like a proposal rather than a solution? And we would also still have no dev-site docs. But sounds like there is some higher level plan to address that.

Change the existing instrumentation code to use this client by default, falling back to the the Logging API approach if there's no gRPC. This has just been in my backlog forever, will give it a shot now.

I think once that's done we can close this issue, but maybe start some new ones about how this all gets documented clearly (especially my pet concern of how we relate to cloud.google.com vs github docs).

@waprin Good point about the GAPIC docs. They exist as docstrings in the code, but we have not web-rendered it. We've just started doing this again with Logging, since we declared beta for it a couple weeks ago.
We'll take care of publishing docs for other packages in early January. FYI, @geigerj and @ethanbao.

When running the examples from the documentation, I get the following error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/laurence/miniconda3/envs/sonalytic/lib/python3.5/site-packages/google/cloud/error_reporting/client.py", line 320, in report
    report_location=report_location)
  File "/home/laurence/miniconda3/envs/sonalytic/lib/python3.5/site-packages/google/cloud/error_reporting/client.py", line 277, in _send_error_report
    self.report_errors_api.report_error_event(error_report)
  File "/home/laurence/miniconda3/envs/sonalytic/lib/python3.5/site-packages/google/cloud/error_reporting/client.py", line 162, in report_errors_api
    self._report_errors_api = make_report_error_api(self)
  File "/home/laurence/miniconda3/envs/sonalytic/lib/python3.5/site-packages/google/cloud/error_reporting/_gax.py", line 44, in make_report_error_api
    return _ErrorReportingGaxApi(gax_client, client.project)
AttributeError: 'Client' object has no attribute 'project'

Disabling GRPC via the GOOGLE_CLOUD_DISABLE_GRPC environment variable fixes this.
I'm running on an Ubuntu 14.04 compute instance with python 3.5.3 and version 023.1 of the google-cloud-error-reporting project

FYI @lukesneeringer

@daspecster Can you look at this one? It looks like it is probably a quick bugfix.

@lukesneeringer sure thing!

@daspecster mentioned this to Luke but I'm OOO next week but spending today and tomorrow trying to catch up on loose ends/random issues so feel free to ping me on anything you want me to look at/try to fix.

@waprin I am really only referring to this comment, which honestly should be a separate issue.

@lukesneeringer Can you or @daspecster split it out as a separate issue to reflect that?

Now I am understanding you're just referring to the reported bug. Yes, I think once we create that new issue, we can close this issue as it was done in #2894 . Only thing I think that's left is the documentation for the GAPIC which is also a separate issue.

@waprin @lukesneeringer @bjwatson issue created!
https://github.com/GoogleCloudPlatform/google-cloud-python/issues/3158

Closing this issue (done in #2894).


docs(plugins): update option type

I think (string | RegExp)[] is accurate than Array.

Should we merge #3693 , so that both could be reviewed in one go
I could fetch your branch make changes there , or you could fetch mine
Feel free to let me know :)

@jeffin143 Thanks for suggestion, but these two seem to be totally different things although they target the same file, maybe better keep them separated.

Found some discussions about how to present types in doc here https://github.com/webpack/webpack.js.org/pull/3553/files/7c05c8994023d71fde997dee218774bc65952665#r380507188, and from that example, I would say the current guides https://webpack.js.org/contribute/writers-guide/#configuration-defaults-and-types is kind of broken when dealing with complex cases:

I don't think anyone can figure out this easily.
Any here's the typescript version formatted with prettier:
type Entry =
  | string
  | [string]
  | {
      key:
        | string
        | [string]
        | {
            import: string | [string]
            dependOn: string | [string]
            filename: string
          }
    }
  | (() =>
      | string
      | [string]
      | {
          key:
            | string
            | [string]
            | {
                import: string | [string]
                dependOn: string | [string]
                filename: string
              }
        })

It's much longer, but it's clean, and much easy to read.

Thank you!


Property linking

Property linking. You might want to direct a reader straight to a property instead of the start of the endpoint, especially if there are lots of properties. I think Redocly supports this and seen it in other APIs. For an example, see the OpenAI API docs, hover over a property name to see the link.

Posted here: https://github.com/scalar/scalar/discussions/3108

Closing in favor of #1847


PB with cell tracking

Hello,
i try to use usiigacy with the cell tracking part. I've installed anaconda tu run ussigaci. when i try to open the folder to test the cell tracking, i've got this error.

I've just started python programming so take it easy with the explanation :-)
Have a nice day
Romain

Hi,
I am having he same problem.  Have you find the solution?
Have a nice day,
Carmen

Unfortunatly, I still have this problem. Actually, i use active contour with the software ICY to make the tracks and tracking manager in ICY to have differents parameters.
have a nice day
Romain

Tru to change pims_sequence = pims.ImageSequence(join(path, '.{}'.format(file_extension)), process_func=None)  for pims_sequence = pims.ImageSequence(join(path, '.{}'.format(file_extension))) in cell_io.py

Thx a lot! It's works now!!


Redux<EMAIL_ADDRESS>

Todo:

[ ] New features

[ ] Add support for new request action types to resources slices
[ ] add support for new resources structure for existing action types
[ ] add support for direct manipulation of resource meta with requests
[ ] Add new resource action types for manipulating resources outside of requests (CUD)
[x] Add resourceName to slice
[ ] Add idAttribute (maybe?)

[ ] Tests

[ ] new request slice plugins

[ ] Docs update

[ ] Deprecate HTTP Status Codes plugin (it is built in)
[ ] Deprecate Included Resources plugin (it is built in)
[ ] Restructure docs to focus on requests and resources independently

[ ] Examples

[ ] Update them

Closing; see #350


crunker-master/examples/server/index.html  not working?

Hello !
I am using this repository to merge 2 audio files.
I  found some error in console of chrome   "crunker-master/examples/server/index.html".
Please give me somey suggestions?
crunker.js:1 Fetch API cannot load file:///E:/crunker-master/examples/server/1.mp3. URL scheme "file" is not supported.
(anonymous) @ crunker.js:1
value @ crunker.js:1
doTheWork @ index.html:27
handleMerge @ index.html:18
onclick @ index.html:10
19:22:36.849 crunker.js:1 Fetch API cannot load file:///E:/crunker-master/examples/server/2.mp3. URL scheme "file" is not supported.
(anonymous) @ crunker.js:1
value @ crunker.js:1
doTheWork @ index.html:27
handleMerge @ index.html:18
onclick @ index.html:10
19:22:36.850 crunker.js:1  Uncaught (in promise) TypeError: Failed to fetch
at crunker.js:1
at Array.map ()
at e.value (crunker.js:1)
at doTheWork (index.html:27)
at handleMerge (index.html:18)
at HTMLButtonElement.onclick (index.html:10)

This is a browser limitation for security, preventing sites from maliciously enumerating files on your local computer, and is not an issue with Crunker.

Now you mention it, though, the examples won't work whatsoever since Crunker is now written in Typescript, but the example still attempt to load Crunker from a .js file.

Wonder if we should add to the docs a command to run the examples with ts-node  to make this clearer and a command. That may not be the best though.
I imagine most people will prefer to just copy the example as a starting point so better to reference crunker from a delivery cdn like jsdelivr for example.

I don't think this issue is ready to be closed. We simply need to update the reference to crunkerjs with the unpkg link we use in the server examples.
Should be an easy 2 min fix

The examples should also be able to be hosted on GitHub Pages (they actually are, but you have to directly enter the URL: https://jackedgson.github.io/crunker/examples/client/index.html

Hmm, I'll see if I can get a better GitHub url for that and add it to the repo description.
Are you able to do a quick or to replace the relative reference to src with the unpkg url in the meantime?

Yeah, of course.

Thanks a lot!

Actually @davwheat I've updated it now and also made both use latest version, plus released the package to version 1.3.0 based on previous changed I forgot to publish. So no need to update anymore.

Have also added URL now as suggested, thanks for that Dave. Good find @lotuspoolcnhz going to close this one now.


Update bluelink to 2.3.6

2.3.6 created 5.8.23

RE-CHECK

2.3.8 created 5.8.23
60 user (7,7%)
OK


Double Title bar on Linux Mint 21.1

Hi,
I am trying use this app on linux mint 21.1 and title bar appears twice, see below.

EDIT: My dotnet version is 6.0.113

Maybe changing Titlebar Visible property to false when compiling to Linux ? (just add pre-processor flag).

This may be an issue with the way Mint chooses to implement its window handling.
The same thing seems to happen on Avalonia, this post specifically mentions Mint as well:
https://stackoverflow.com/questions/67474194/avalonia-hiding-titlebar-window-on-linux-does-not-work
I generally do testing on Ubuntu, and it works as expected there.


feat: add --discover capability for all SQL targets

This was an experiment which actually turned out to be simpler than expected.
Todo:

[ ] Tests
[ ] Docs
[ ] Discussion

Note:
The primary value of this operation is to be able to analyze and version the schema of targets. This is important because when trying to understand destination schema in the target, we can't necessarily assume that the target schema will be identical with the schema of upstream tap streams. Stream maps and selection rules can change this, of course, but also factors like schema_mappings and _sdc_* metadata columns.
Potentially this could be extended further to actually querying data from the target directly - but that is not in scope any time in the near future.

:books: Documentation preview :books:: https://meltano-sdk--1120.org.readthedocs.build/en/1120/

pre-commit.ci autofix

Closing for now but we might revisit in the future


[Metal] Remove lazy_static

Fixes #3574
PR checklist:

[ ] make succeeds (on *nix)
[ ] make reftests succeeds
[x] tested examples with the following backends: Metal

Strangely, I'm seeing this now when running wgpu-rs (patched to use latest gfx):

objc[3495]: Class GfxManagedMetalLayerDelegate is implemented in both ?? (0x7fa83f40ee00) and ?? (0x7fa83f504520). One of the two will be used. Which one is undefined.
objc[3495]: Class GfxManagedMetalLayerDelegate is implemented in both ?? (0x7fa83f40ee00) and ?? (0x7fa83f5046c0). One of the two will be used. Which one is undefined.

Strangely, I'm seeing this now when running wgpu-rs (patched to use latest gfx):

objc[3495]: Class GfxManagedMetalLayerDelegate is implemented in both ?? (0x7fa83f40ee00) and ?? (0x7fa83f504520). One of the two will be used. Which one is undefined.
objc[3495]: Class GfxManagedMetalLayerDelegate is implemented in both ?? (0x7fa83f40ee00) and ?? (0x7fa83f5046c0). One of the two will be used. Which one is undefined.

Ah, snap, that may be a real issue here. If anybody creates multiple instances, we are in trouble :/
Perhaps, the instance code could check if there is an existing declaration somehow?

Ah, snap, that may be a real issue here. If anybody creates multiple instances, we are in trouble :/
Perhaps, the instance code could check if there is an existing declaration somehow?

nvm, fixing this in #3585 :)

nvm, fixing this in #3585 :)


Add historical forecast data e.g. day-ahead

Detailed Description

Add historical forecast data (e.g. day-ahead) to quantify forecast errors for renewables
Context

Economic dispatch optimizations for different energy market regimes (intraday, day-ahead) require historical forecast data. When available, it is also possible to quantify typical forecast error e.g. by comparing predicted (e.g. day-ahead) vs actual observed/estimated data. An example of how this could look is given by the DWD who quantifies the day-ahead  vs actual observed forecast error with 15min resolution: https://www.dwd.de/EN/research/weatherforecasting/num_modelling/07_weather_forecasts_renewable_energy/weather_forecasts_renewable_energy_node.html
Possible Implementation

Possible API/ inspiration:

https://github.com/open-meteo/open-meteo
https://github.com/blaylockbk/Herbie
https://openweathermap.org/

I have already done something similiar and will push it up until mid of november., needs only some fixing due to the last update in atlite. Best, Tim


Bluetooth: samples: Remove broken sysbuild overlay for 53 net core

The issue ticket explains the problem:
Fixes: https://github.com/zephyrproject-rtos/zephyr/issues/74313
The overlay added to the samples is not appropriate. It likely was a copy-paste accident.

I have been in contact with @tejlmand and he will look into this when we has the time.


libtcl library shouldn't be used

https://github.com/The-OpenROAD-Project/OpenDB/blob/master/src/swig/tcl/CMakeLists.txt#L20
Please use the library returned by FindTCL.cmake

This is resolved on the 'openroad' branch which represents the current development head.  'master' is used for a stable release point and is rather old at this time.
https://github.com/The-OpenROAD-Project/OpenDB/blob/openroad/CMakeLists.txt#L22


Some tweets are processed by vxtwitter.com but not by the api + some strange behavior

I was working on a bot today that sent some vxtwitter links, and realized that some links just don't work.
Example: https://x.com/GustavoPetroF/status/1791466911865147664
Going to https://api.vxtwitter.com/Twitter/status/1791466911865147664, I get the html + redirect to the github project (which btw, Postman says it's a status 200 response instead of the 301 or 30X status it would normally be for redirects).
If I send the vxtwitter link (https://vxtwitter.com/GustavoPetroF/status/1791466911865147664), on Telegram the video is correctly set as the thumbnail and it can be viewed and download.
And if I send the aforementioned vxtwitter link on Telegram, it doesn't appear to be the html response I get on browser and postman, but Telegram instead embeds a json file.
This is all very strange and somewhat inconsistent, and it took me some time to figure out. I'd hope the API had some more automation-friendly responses, such as an error code on error, and that if vxtwitter actually has the video, as is shown in the examples, it would show it in the response instead of redirecting to github on error.
I honestly don't know of the source of these problems, and I wish I knew enough python to troubleshoot this but the best I can do right now is post here.

Sorry for the late reply, I'm currently looking into this

no problem, I didn't expect you to be on call for this x)

This should be fixed now. I'll work on making API errors clearer in the future, but for now tweets that exist should load properly. Feel free to re-open if you encounter any other issues!


Add MQTT Discovery Support for HomeAssistant

The goal of this issue is to add support for MQTT discovery in the project, allowing for easier integration and management of the service within HomeAssistant. With the addition of this feature, users will be able to update their Docker containers from within HomeAssistant, simplifying the process and improving the overall user experience.
This will involve adding the necessary code to publish device information via MQTT, and to respond to discovery requests made by HomeAssistant. Once complete, the integration with HomeAssistant will be smoother and more convenient for users, and the project will be better positioned to support a wider range of users and use cases.
Tasks:

Implement MQTT discovery support
Publish device information via MQTT
Respond to discovery requests from HomeAssistant
Test and validate the implementation
Update documentation as necessary to reflect the changes

Example (imagine your docker containers here):

Example for how it could be implemented:
https://github.com/Koenkk/zigbee2mqtt/blob/052cae09f86680c454b578fb4253fa03d81fb93e/lib/extension/homeassistant.ts#L1057


TypeError: 'ChatCompletionMessage' object is not subscriptable

Confirm this is an issue with the Python library and not an underlying OpenAI API

[X] This is an issue with the Python library

Describe the bug
content_with_padding_02 = "\u200B\n\n" + response_02.choices[0].message['content']

TypeError: 'ChatCompletionMessage' object is not subscriptable
To Reproduce
content_with_padding_02 = "\u200B\n\n" + response_02.choices[0].message['content']

TypeError: 'ChatCompletionMessage' object is not subscriptable
Code snippets
content_with_padding_02 = "\u200B\n\n" + response_02.choices[0].message['content']

TypeError: 'ChatCompletionMessage' object is not subscriptable

OS
macOS
Python version
3.11.4
Library version
1.0.1

The [] syntax is not supported, you need this instead:
content_with_padding_02 = "\u200B\n\n" + response_02.choices[0].message.content

thanks


:index pseudo selector doesn't work correctly

The :index pseudo selector should map to nth-child(index + 1), but it doesn't seem to work correctly: http://jsfiddle.net/pQtBm/1/

See pull request #64

This has to be opened again as it does not work in 1.6.0.


MM-14318 linkAndButton Spec

Summary
Test Spec for Link and Button Styles. Under custom themes in Mattermost settings.
Ticket Link
https://github.com/mattermost/mattermost-server/issues/10367
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] Ran make check-style to check for style errors (required for all pull requests)
[X] Ran make test to ensure unit and component tests passed

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

Hey @m3phistopheles,
Do you have questions about the feedback? Looking forward to merging this PR!

Hey @m3phistopheles,
Are you still working on this issue? Can we help with something?
Please note that we will close this PR after another 10 days of inactivity. If you don't object, other contributors might complete this PR for you.

Hi @m3phistopheles,
This PR has become orphaned and will be closed. Still, thanks for your contribution @m3phistopheles. :+1:


Starfinder (simple) bug fix 012721

corrected issue where NPC Sense Motive skill check did not roll.
Changes / Comments
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

Character Sheet Info Roll20 Internal Use only.


[Icon Request] Histoire stories

Icon Type

[ ] Folder
[x] File

File Names

*.story.vue

Graphic ideas

Additional context
Histoire is a tool to generate stories applications (an alternative to Storybooks).
This is the Story for the config counterpart in #1625.

Nice idea, I would like to see it implemented in the icon pack :)


Docker hadoop runner

889

 Quality gate: Failed
10 Issues

 10 Code issues

Details

 Quality gate: Failed
10 Issues

 10 Code issues

Details


author link fix

supre small pull request, sorry fam

I included this in my PR already. Btw, this doesn't work for any authors where the ID is a URL