@CleanupMode  = 'BEFORE_BACKUP' - linux not working

I have on Linux MSSQL 2019, latest version of the maintenance script.
Script date: Script Date: 11. 05. 2021 07:43:20
I am trying to do a backup and before backup i want to clean older. We backup the VM on hourly basis and some db's are quite large so we are want to delete backups that were already backed up.
I tried the below command on windows and it works ok! On linux it does not delete any file, but it does work when i set AFTER_BACKUP, which in some cases is not possible because of db size and i need free space before.
EXECUTE [dbo].[DatabaseBackup]
@Databases = 'master',
@Directory = N'/storage/backup/',
@BackupType = 'FULL',
@Verify = 'Y',
@CleanupTime = 2,
@CheckSum = 'Y',
@LogToTable = 'Y',
@CleanupMode  = 'BEFORE_BACKUP'
What can i test ? I just did a test:
This is the output of the above command:
Date and time: 2021-05-11 07:40:44
Server: bilbo
Version: 15.0.4073.23
Edition: Web Edition (64-bit)
Platform: Linux
Procedure: [master].[dbo].[DatabaseBackup]
Parameters: @Databases = 'master', @Directory = '/storage/backup/', @BackupType = 'FULL', @Verify = 'Y', @CleanupTime = 2, @CleanupMode = 'BEFORE_BACKUP', @Compress = NULL, @CopyOnly = 'N', @ChangeBackupType = 'N', @BackupSoftware = NULL, @CheckSum = 'Y', @BlockSize = NULL, @BufferCount = NULL, @MaxTransferSize = NULL, @NumberOfFiles = NULL, @MinBackupSizeForMultipleFiles = NULL, @MaxFileSize = NULL, @CompressionLevel = NULL, @Description = NULL, @Threads = NULL, @Throttle = NULL, @Encrypt = 'N', @EncryptionAlgorithm = NULL, @ServerCertificate = NULL, @ServerAsymmetricKey = NULL, @EncryptionKey = NULL, @ReadWriteFileGroups = 'N', @OverrideBackupPreference = 'N', @NoRecovery = 'N', @URL = NULL, @Credential = NULL, @MirrorDirectory = NULL, @MirrorCleanupTime = NULL, @MirrorCleanupMode = 'AFTER_BACKUP', @MirrorURL = NULL, @AvailabilityGroups = NULL, @Updateability = 'ALL', @AdaptiveCompression = NULL, @ModificationLevel = NULL, @LogSizeSinceLastLogBackup = NULL, @TimeSinceLastLogBackup = NULL, @DataDomainBoostHost = NULL, @DataDomainBoostUser = NULL, @DataDomainBoostDevicePath = NULL, @DataDomainBoostLockboxPath = NULL, @DirectoryStructure = '{ServerName}${InstanceName}{DirectorySeparator}{DatabaseName}{DirectorySeparator}{BackupType}{Partial}{CopyOnly}', @AvailabilityGroupDirectoryStructure = '{ClusterName}${AvailabilityGroupName}{DirectorySeparator}{DatabaseName}{DirectorySeparator}{BackupType}{Partial}{CopyOnly}', @FileName = '{ServerName}${InstanceName}{DatabaseName}{BackupType}{Partial}{CopyOnly}{Year}{Month}{Day}{Hour}{Minute}{Second}{FileNumber}.{FileExtension}', @AvailabilityGroupFileName = '{ClusterName}${AvailabilityGroupName}{DatabaseName}{BackupType}{Partial}{CopyOnly}{Year}{Month}{Day}{Hour}{Minute}{Second}{FileNumber}.{FileExtension}', @FileExtensionFull = NULL, @FileExtensionDiff = NULL, @FileExtensionLog = NULL, @Init = 'N', @Format = 'N', @ObjectLevelRecoveryMap = 'N', @ExcludeLogShippedFromLogBackup = 'Y', @StringDelimiter = ',', @DatabaseOrder = NULL, @DatabasesInParallel = '...
Version: 2020-12-31 18:58:56
Source: https://ola.hallengren.com
Date and time: 2021-05-11 07:40:44
Database: [master]
State: ONLINE
Standby: No
Updateability: READ_WRITE
User access: MULTI_USER
Recovery model: SIMPLE
Encrypted: No
Is accessible: Yes
Differential base LSN:<PHONE_NUMBER>040000001
Last log backup LSN: N/A
Allocated extent page count: 7176 (56.062500 MB)
Modified extent page count: 96 (0.750000 MB)
Your queue no is 2021-05-11 05:40:44.0847945
Date and time: 2021-05-11 07:40:44
Database context: [master]
Command: BACKUP DATABASE [master] TO DISK = N'/storage/backup/master_FULL_20210511_074044.bak' WITH CHECKSUM, NO_COMPRESSION
Processed 7192 pages for database 'master', file 'master' on file 1.
Processed 2 pages for database 'master', file 'mastlog' on file 1.
BACKUP DATABASE successfully processed 7194 pages in 0.386 seconds (145.593 MB/sec).
Outcome: Succeeded
Duration: 00:00:00
Date and time: 2021-05-11 07:40:44
Date and time: 2021-05-11 07:40:44
Database context: [master]
Command: RESTORE VERIFYONLY FROM DISK = N'/storage/backup/master/FULL/master_FULL_20210511_074044.bak' WITH CHECKSUM
The backup set on file 1 is valid.
Outcome: Succeeded
Duration: 00:00:00
Date and time: 2021-05-11 07:40:44
Date and time: 2021-05-11 07:40:44
Completion time: 2021-05-11T07:40:46.2001944+02:00
And this are the backups still in the folder
master_FULL_20210510_143908.bak
master_FULL_20210510_143931.bak
master_FULL_20210510_143942.bak
master_FULL_20210510_144000.bak
master_FULL_20210510_144044.bak
master_FULL_20210510_144105.bak
master_FULL_20210510_144731.bak
master_FULL_20210510_145104.bak
master_FULL_20210510_145213.bak
master_FULL_20210510_145222.bak
master_FULL_20210510_145303.bak
master_FULL_20210510_145410.bak
master_FULL_20210510_145615.bak
master_FULL_20210510_145715.bak
master_FULL_20210510_154455.bak
master_FULL_20210510_154540.bak
master_FULL_20210510_155409.bak
master_FULL_20210511_020001.bak
master_FULL_20210511_074023.bak
master_FULL_20210511_074044.bak
master_FULL_20210511_074901.bak
So what can i test more to see why it is not deleting?

@olahallengren Dear this issue is still present in today 2024 scripts, just changing the lines before cleanup 'BEFORE_BACKUP' to:
 -- Create directory
      -- IF @HostPlatform = 'Windows'
       -- AND (@BackupSoftware <> 'DATA_DOMAIN_BOOST' OR @BackupSoftware IS NULL)

	  IF (@BackupSoftware <> 'DATA_DOMAIN_BOOST' OR @BackupSoftware IS NULL)
      AND NOT EXISTS(SELECT * FROM @CurrentDirectories WHERE DirectoryPath = 'NUL')

Makes the delete older files work on Linux.
Why it makes it work, beacuse in the create directory this is set:
          UPDATE @CurrentDirectories
          SET CreateCompleted = 1,
              CreateOutput = @CurrentCommandOutput
          WHERE ID = @CurrentDirectoryID

Which actually allows before backup cleanup to work

I have fixed this issue.
I implemented the change in cleanup part, rather than in the directory creation part.


fix links in README.md

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

Thank you @XenoAmess for the pull request!


Features request: Mutably lookup multiple distinct values

In the current interface looking up a single key will borrow the entire map and in particular a mutable lookup requires a unique borrow of the entire map. This is very safe but also overly restrictive. In comparison, slices (as a very primitive key-value-map) permit this same operation with a split_mut method which for example allows defining an entirely safe method for acquiring elements at two different indices at once:
fn at2<T>(slice: &mut [T], a: usize, b: usize) -> (&mut T, &mut T) {
    let (_, slice) = slice.split_at_mut(cmp::min(a, b));
    let (head, tail) = slice.split_at_mut(cmp::max(a, b) - cmp::min(a, b));
}

If we try to replace the slice with a HashMap, for example as a sparse data structure, however we hit a brickwall. There is a bad workaround: Use an iterator over all mutable reference, filter it by the two elements we are interested in and return them. This is both inefficient, unergonomic, and brittle.
fn at2<T>(map: &mut HashMap<usize, T>, a: usize, b: usize) -> (&mut T, &mut T) {
    let mut values: HashMap<_, _> = map.iter_mut().collect();
    (values.remove(&a).unwrap(), values.remove(&b).unwrap())
}

This is embarrisingly expensive, cloning the entire map and recalculating all hashes, while the existence of the safe workaround should also make it clear that there is no formal issue preventing the simultaneous use of multiple values (the backing memory is allocated similar to a vector) while querying keys.
A few remarks on a first draft
I'm not quite sure how the interface should look like. It could be some simple interface returning raw pointers to elements which requires the caller to implement correct safety and uniqueness guarantees.
There is also a more ambitious solution. Do note that raw::RawTable has the interfaces needed to build a less expensive safe abstraction but it is too raw. It is both very complex in that almost everything is unsafe and not exposed in the standard library :slightly_smiling_face:
Let's think about how the overhead of the workaround could be lessened by using internal interface. Such a virtual clone would only need to support removal, and no insertion. It would borrow most data structures from the original map. By not allowing insertion, we would not have to recalculate the hashes or do rehashing. It also need not actually remove elements but only mark them borrowed, which can be even cheaper than cloning and modifying the control bytes. This marking could even use a small-vec optimization if only few elements have already been borrowed.

Here is a variant of at2 that avoids cloning the map:
fn at2<K, V>(map: &mut HashMap<K, V>, a: &K, b: &K, cb: impl FnOnce(&mut V, &mut V)) {
    // Remove values from map
    let mut va = map.remove(a).unwrap();
    let mut vb = map.remove(b).unwrap();

    // Pass mutable references to callback
    cb(&mut va, &mut vb);

    // Re-insert modified values into map
    map.insert(a, va);
    map.insert(b, vb);
}

There are obvious improvements to make on this (e.g. use a drop guard for reinsertion, report errors nicely, support return values from the callback, etc) but it should be a good start.

It's not a true replacement because the callback must be, as written in the function signature, able to take keys and values of arbitrarily short lifetime and not only those of the map itself. That is a huge restriction in my point of view and in the general case as well.

I think that a proper solution would probably have to wait until we have support for const generics. This would allow us to design an API like this:
impl<K, V> HashMap<K, V> {
    fn get_mut_multi<const N: usize>(&mut self, keys: [&K; N]) -> [Option<&mut V>; N] { ... }
}

I don't think it would be much worse with an iterator based design. When querying a key twice it would yield a None the second time. Is there precedent for incorrect implementations of Borrow, Eq causing panics? If so then that may be an alternative. Otherwise, it would probably hurt usability as it would be the caller's responsibility to ensure that keys are unequal which they may not always be able or willing to do.
impl<K, V> HashMap<K, V> {
    fn get_multi_mut(&mut self, keys: &[K]) -> MultiIterMut<'_, K, V> { ... }
}

impl<'val> Iterator for MultiIterMut<'val, K, V> {
    type Item = Option<&'val mut V>;
}

Unfortunately that won't work. Eq is allowed to "lie" with only safe code, so we need to manually ensure that we never return two mutable references to the same value. This means that we need to keep track of all the references we have given out so far in a Vec. There is no way around this.

Yes, I realize that. How you the array based design avoid this in any case? I don't think it's much different from querying a key after it has been removed.

OK so you can do this with a Vec, however the main downside is that it forces you to allocate memory on every lookup, which is going to hurt performance. The array-based version on the other hand will allow the compiler to optimize most overhead away.


[Search] Custom HTTP_ACCEPT

I need to call a API in json:api format. So i need to accept 'application/vnd.api+json' in the HTTP_ACCEPT header. Is there a way to overload the header with a callback before the ajax request is sent ?
  $('.ui.search')
    .search({
      type: 'recipe',
      apiSettings: {
        onRequest: (promise, xhr) ->
          xhr.setRequestHeader('HTTP_ACCEPT', 'application/vnd.api+json')
        url: '//staging-doeat.herokuapp.com/api/v1/recipes',
        onResponse: (response) ->
          console.log(response)
      },
  });

I would like to do something like this. But it doesn't work.
Thanks u for your responses.

The docs show there's a beforeXHR callback in the API settings (http://semantic-ui.com/behaviors/api.html#/settings).
So, in your example code, you can use beforeXHR instead of onRequest and remove the promise parameter. Haven't tried it, though.


Update groovy from 3.0.7 to 3.0.8

Refs https://github.com/gradle/gradle/issues/16857
Context
JDK17 is coming. When using Groovy in Gradle, JDK17 does not work.
See https://github.com/gradle/gradle/issues/16857#issuecomment-851785870.
Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass sanity check: ./gradlew sanityCheck --> can't do, because of  Dependency verification failed for configuration ':build-logic-settings:allprojects-plugin:classpath'. Not sure how to resolve that.
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

@bot-gradle test QF

OK, I've already triggered QuickFeedback build for you.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.

The changes had to be reverted because this PR missed a couple locations. Also when these locations are updated, tests fail. Investigating.

@ljacomet Thank you for the follow up. I naively used the "find in all files" functionality of IntelliJ. Sorry for the effort caused. My motivation was to make Gradle ready for JDK17.

No worry, you can look at the follow up on this in #17712
Unfortunately, Groovy 3.0.8 has a number of regressions that affect our test code. And we need to make sure they would not affect Gradle build scripts or plugins.


Add test and fix for panic in Bearer::to_bytes

There was a bug in the BufferMut::reserve logic in headers::www_authenticate::challenge::bearer::Bearer::to_bytes that would cause a panic.
This small commit adds a test that will panic with the old logic and it fixes the problem.

@griff thanks for fix!
Do you need this at crates.io asap?

Right now I am fine since I just use my own repo for the dependency. But its a pretty bad bug given that it panics so you might want to release it soon.

@griff fyi, 0.0.4 version is now available at crates.io with your fix.
Thanks again :)


add github workflow

Unfortunately, the tests are still using raw idris2 instead of pack, which is why you are having tls dependencies issue. This should be moved to using pack, and we can probably get rid of the makefile completely, since this package doesnt have any C support files to compile.

you do see that I removed makefiles and added github workflows that uses pack right?

of course git.sr.ht had to be down now :(

after removing the tls stuff from pack toml the tests still runs after removing build folder and deleting .cache folder in home folder. So this PR is good, it showcases that one of the tests never completes ...

now tests passes in the fork @ReilySiegel  see https://github.com/jarlah/idris2-http/actions/runs/11640875762

This looks ready to merge if we keep the failing tests commented out, sorry for the delay in review. Since you seem to be the primary user of this library, do you want to merge this now and fix the failing tests later, or hold off until we have resolved the underlying issues? I'm open to either approach.

This looks ready to merge if we keep the failing tests commented out, sorry for the delay in review. Since you seem to be the primary user of this library, do you want to merge this now and fix the failing tests later, or hold off until we have resolved the underlying issues? I'm open to either approach.

@ReilySiegel you can just merge it. i rebased and force pushed. Better to have a working CI setup. I can investigate the failing tests. Its a bit hard, but ill try ;)


just test

Running the IAC scan s

What is the thought process? What are we trying to accomplish with this PR?


New version destroyed Perl variables flow

OS version: Win 10
Editor version: 1.58
Programming language: Perl, JavaScript, CSS, SQL, Bash
TabNine extension version: 4.0.4

I have autocomplete-plus v2.42.4 installed as well, if that makes any difference.
I have my AutocompletePlus set to choose a suggested line when I press [TAB].
I've been using Atom, AutocompletePlus and TabNine for several years now.
It all worked beautifully until latest TabNine upgrade to v4+.
What happens now is when I start typing a Perl variable name, a string that starts with a sigil character, TabNine shows the suggestions same as before, but when I hit [TAB] to use the suggestion, Atom does not remove the characters that I typed but just concatenates the suggestion.
For example: when I start typing $my_love and AutocompletePlus shows TabNine's suggestion $my_lovely_variable and I hit [TAB], I get $my_lovemy_lovely_variable. Of course, this is wrong, and I now need to manually edit the result completely braking my coding flow.
It is important to emphasize that this problem does not occur for strings that do not start with a sigil. So it still works perfectly well for function names and other stuff. It is just broken for Perl variable names.
Of course, I triple checked that my AutocompletePlus is set with enabled "Consume suggestion text following the cursor". It was always like that, no changes there.
The problem started showing up when I updated TabNine for Atom
This is a big thorn in my daily work now. It makes the whole Atom+TabNine half as usable as it was till a week ago.
Please help!
Thanks.
gz#7628

Wow, that was fast! :-)
Working as expected now.
Thanks!


Align checkbox layout to Bootstrap Italia

Does a discussion already exist on the topic you are interested in, or on a similar topic?

[X] I verified and it doesn't exist

What
The checkbox component has some small differences in terms of spacing and typography.
Why
We need to align it to Bootstrap Italia component.
Context
No response
More
Related to bootstrap-italia #1148

Risolto in v3.6.0


Add Access-Control-Allow-Origin header

@RobLoach do you think adding COR headers is going to jeopardize our chances with the module review process? Perhaps we should instead implement a hook so that other modules can add headers if they want.

This allows the CSS and JS assets to be consumed from a different domain.  Don't see it breaching any kind of security that would negatively impact reviews.

Ok, just wanted to check.
RE content type being application/json: without this we get an empty page with the style tags in the  where as what we actually want is the style tags as a string so it can be ingested. If this isn't want you want in your use case for EECS then you may have to do something custom. kalastatic_asset_deliver() needs to stay as it is otherwise Kalastatic metalsmith won't be able to pull in the goodness from the Drupes.


Feature request: A flag for enable / disable the caption view's text animation during the swipe

@alexhillc A feature request regarding the caption text animation during the swipe left / right.
Right now, the caption text (title, description and credit) are animating when going from one photo to another. This is a kind of un-natural (fading in and out) especially when the list of photos share the same description.
One suggestion is to have a flag which we can set to enable the caption text  animation during the swipe action.  e.g.
       // should be a public property
       let animateCaptionDuringSwipe = false.

        if animateCaptionDuringSwipe {
            self.isCaptionAnimatingOut = true
            UIView.animate(withDuration: Constants.frameAnimDuration / 2,
                           delay: 0,
                           options: [.beginFromCurrentState, .curveEaseOut],
                           animations: animateOut) { [weak self] (finished) in

                guard let uSelf = self, !uSelf.isCaptionAnimatingIn else {
                    return
                }

                animateOutCompletion(finished)
                UIView.animate(withDuration: Constants.frameAnimDuration / 2,
                               delay: 0,
                               options: [.beginFromCurrentState, .curveEaseIn],
                               animations: animateIn,
                               completion: animateInCompletion)
            }
        } else {
            animateOutCompletion(true)
        }

👍 added in: d75a48815512c53340b3fefe1c1abb1cd13dfa79
To access, just use PhotosViewController.overlayView.animateCaptionViewChanges


Use voms-proxy-init to get grid proxies if needed

Some tests (even in OSG 3.6) need grid proxies which we can get with
voms-proxy-init if grid-proxy-init is not available.
Also do some cleanup and defensive coding for VOMS.

Just manual stuff; this is cherry-picked from my xrootd branch.


Split ContractRequests into CapitationContractRequest and ReimbursementContractRequest (STI)

Coverage increased (+0.03%) to 88.669% when pulling 2c14eaea9d187c971531f808bb0fa32e4569216d on split-contract-requests into 38e3e1c7e6b7148dea47c5e3003dbc7e8482b3bd on develop.


Fixes

@Keats I noticed that the __repr__ for a taxonomy node now looks like <builtins.TaxonomyNode object at 0x10e7abf50>. Can you bring back the __repr__ that displays the id, rank, and name?


Add category analysis for law enforcement facilities and misleading categories

I added an analysis of which data points might be misleading and how to remove them. Summary:
Law enforcement facilities
It seems that wide variety of reports are concentrated at law enforcement facilities. Although we wouldn't check with TPD, it is possibly because they are where enforcement officers received the report. That is, they are not the location of the incident, rather where the incident report was received.
Following this hypothesis, we suggest not using data at LE facilities for pix2pix, but I believe using them for statistical analysis that does not involve the location of the crime is fine.
Misleading categories
"COMMUNITY POLICING", "ADMIN", "RECOVERED PROP", "AUTO THEFT RECOVERY", "UNABLE TO VERIFY" and "TRAFFIC" should not be included in our dataset.

COMMUNITY POLICING: We already discussed this.
ADMIN and TRAFFIC: This has no information on the TOPS website, so these reports are not informative.
RECOVERED PROP and AUTO THEFT RECOVERY: These are not crime reports but about recovered properties.
-UNABLE TO VERIFY: These are for reports where officers could not find the incident.

@ayhncgty Please review what I found, and if you agree, please approve the pull request. Once approved I will merge the PR.

@abdullahnaeemmalik If you have time, please have a look on it as well.


fix poetry part for README.md

Pull request checklist

[ ] The PR has a proper title. Use Semantic Commit Messages. (No more branch-name title please)
[ ] Make sure you are requesting the right branch: dev.
[ ] Make sure this is ready to be merged into the relevant branch. Please don't create a PR and let it hang for a few days.
[ ] Ensure all tests are passing.
[ ] Ensure linting is passing.

PR type

Bug fix / new feature / chore

Description

Describe what this pull request is for.
What will it affect.

Screenshot

Please include a screenshot if applicable

错误是否由pyproject.toml配置引起不得而知。
当我直接使用poetry install来安装时，在win11或wsl2下，均会出现gpu not found，可能是漏了cudatoolkit相关的库。
当使用我修改后的指令时，表现便与常规使用pip安装一致，可以正常运行。

那就直接删掉吧，毕竟现在requriements.txt也根据不同平台分了好多个，一个toml无法全部顾及。


Add transition duration property to PopupMenuThemeData

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
I want to use PopupMenu in my desktop app, but I don't like the default 300ms animation when rendering the popup menu.
material/popup_menu.dart

const Duration _kMenuDuration = Duration(milliseconds: 300);

https://github.com/flutter/flutter/assets/32097135/0b4473b4-add8-4324-bfaa-016626bb853a
I looked at the code but there doesn't seem a way to control this animation through material properties..
I don't see the point for this imposed constant value since most desktop apps use a faster animation or no animation at all for showing popup menus (e.g Spotify, Linear, Ableton, Capcut)
https://github.com/flutter/flutter/assets/32097135/22dc5145-671c-472c-af38-1978dd9162c7
Proposal
Expose the _PopupMenuRoute's  transitionDuration parameter in the PopupMenuThemeData parameters

Hi @alexvoina.Thanks for proposing this. I see there is an open proposal describing this https://github.com/flutter/flutter/issues/135638. Kindly comment, upvote and follow up on this issue for updates. Closing this in favour of the already open issue, if you disagree, please comment and I'll reopen this.


Update bayes_theorem.cpp

Just made the code more beautiful by using namespace std
Remove the std:: indentation so that code looks less cluttered

Checklist

[x] Added description of change
[x] Added file name matches File name guidelines
[x] Added tests and example, test must pass
[x] Relevant documentation/comments is changed or added
[x] PR title follows semantic commit guidelines
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] Sort by alphabetical order
[x] I acknowledge that all my contributions will be made under the project's license.

Notes:

Please read SO discussion for why we do not use using namespace std; haphazardly. This is what the cpplint utility checks and is throwing an error for.
The changes made in this pull request actually break the protocol and do not add value or solve any issue ubt rather create new ones.


pingone_gateway (ldap): Changing the "password authority" value of a user_type forces recreate

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

We have an pingone_gateway instance configured via terraform. This is an LDAP gateway connected to an on prem AD.
We now want to move the password authority from LDAP to PING_ONE for a user_type. If we do this via the WEBUI , this works fine, but via terraform the user-type is recreated (new ID). This action affects the all users connected to this gateway (the user_type is present in the user configuration).
PingOne Terraform provider Version
0.26
Terraform Version
Terraform v1.5.7
Affected Resource

pingone_gateway

Terraform Configuration Files
resource "pingone_gateway" "example" {
environment_id            = pingone_environment.environment.id
name                      = "AD_${terraform.workspace}"
enabled                   = true
type                      = "LDAP"
vendor                    = "Microsoft Active Directory"
connection_security       = "StartTLS"
bind_dn                   = var.p1_gtw_bind_dn
bind_password             = var.p1_gtw_bind_pwd
servers                   = [ "<IP_ADDRESS>", "<IP_ADDRESS>"  ]
user_type {
name                          = "Usr-${terraform.workspace}-Customers"
password_authority            = "LDAP"
#password_authority            = "PING_ONE"
}
}
Expected Behavior
Terraform should update both the pingone_gateway and the user_type in-place
Actual Behavior
Terraform removes the existing user_type and recreates it.
Terraform will perform the following actions:
pingone_gateway.example[0] will be updated in-place
(moved from pingone_gateway.example)
~ resource "pingone_gateway" "example" {
+ bind_password                             = (sensitive value)
id                                        = "e0e4e65a-0311-4504-a604-9f848cb889fd"
name                                      = "AD_production"
# (9 unchanged attributes hidden)
  - user_type {
      - id                            = "02ea68f9-2e6e-4e44-aed9-dc0f8b4c46ae" -> null
      - name                          = "Usr-production-Customers" -> null
      - password_authority            = "LDAP" -> null
      - push_password_changes_to_ldap = true -> null
      - search_base_dn                = "OU=example,DC=example,DC=com" -> null
      - user_link_attributes          = [
          - "middleName",
        ]
  + user_type {
      + id                            = (known after apply)
      + name                          = "Usr-production-Customers"
      + password_authority            = "PING_ONE"
      + push_password_changes_to_ldap = true
      + search_base_dn                = "OU=example,DC=example,DC=com"
      + user_link_attributes          = [
          + "middleName",
        ]

Hi @patrickcping Thanks for your feedback. Will proceed as per your suggestion for now.

This will be resolved in the upcoming v1.0 release.  The intention is to change the "set" data type to a "map", so the individual user_type embedded objects can be correctly correlated between the plan and the state.
Ref https://discuss.hashicorp.com/t/dealing-with-unordered-sets-of-complex-objects-setnestedattribute-with-computed-attributes/61874/1

Hi Patrick,
Just back with another terraform related question.
It seems that the SAML assertion encryption for an application is currently not configurable with terraform (the signing by contrast is supported in terraform).
Is that on the horizon at the moment? It’s one of the last things holding us back to fully move to terraform.
If needed I can create a github feature request, but I was wondering if we were the first to ask?
BR,
Jan
From: Patrick Cowland @.>
Date: Thursday, 2 May 2024 at 11:21
To: pingidentity/terraform-provider-pingone @.>
Cc: Jan Renard @.>, Mention @.>
Subject: Re: [pingidentity/terraform-provider-pingone] pingone_gateway (ldap): Changing the "password authority" value of a user_type forces recreate (Issue #753)
This will be resolved in the upcoming v1.0 release. The intention is to change the "set" data type to a "map", so the individual user_type embedded objects can be correctly correlated between the plan and the state.
Ref https://discuss.hashicorp.com/t/dealing-with-unordered-sets-of-complex-objects-setnestedattribute-with-computed-attributes/61874/1
—
Reply to this email directly, view it on GitHubhttps://github.com/pingidentity/terraform-provider-pingone/issues/753#issuecomment-2089994542, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AW2GH3NEEJY37DKBCY7DYIDZAIATHAVCNFSM6AAAAABD6E4UKWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBZHE4TINJUGI.
You are receiving this because you were mentioned.Message ID: @.***>

Hi Patrick,
Indeed, it’s missing from the API documentation, the API does accept the encryption parameters. I already use the API to automate the SAML encryption settings.
This is (roughly) the template we use for the moment (part of the application json):
"spEncryption" : {
"algorithm" : "{{ enc_type }}",

"certificate" : {

  "id" : "{{ cert.id }}"

}

}
BR,
Jan
From: Patrick Cowland @.>
Date: Wednesday, 29 May 2024 at 11:33
To: pingidentity/terraform-provider-pingone @.>
Cc: Jan Renard @.>, Mention @.>
Subject: Re: [pingidentity/terraform-provider-pingone] pingone_gateway (ldap): Changing the "password authority" value of a user_type forces recreate (Issue #753)
Hey @janrenardhttps://github.com/janrenard , you're right it needs building into the provider. I don't see reference to it in the API documentation so I've raised the request internally to have that reviewed
In the meantime I've created the feature request at #823https://github.com/pingidentity/terraform-provider-pingone/issues/823 . I'll look to include this in a minor release in the next week or so
—
Reply to this email directly, view it on GitHubhttps://github.com/pingidentity/terraform-provider-pingone/issues/753#issuecomment-2136971592, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AW2GH3LKGWVLILGCQZ5IVALZEWOHRAVCNFSM6AAAAABD6E4UKWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZWHE3TCNJZGI.
You are receiving this because you were mentioned.Message ID: @.***>


[T2A2][T10A2] Kabir Khandpur

Ready for review

Change the title according to my comment in #268. Make sure you use the correct title next time.


microsoft/iis:windowsservercore-1709 is not supported in web app windows container

Our team is experimenting Web App for (Windows) Container. I got error when trying to upload docker image:

Cannot run this Operating System/Version in Windows Containers. Maximum supported OS version is 10.0.14393.9999.

How to know the version of Web App for Windows Container host OS?
What is eligible base image os version?
Our web app can only run with 1709 or later. Does it have a plan to upgrade host OS version?

Can you share the output of docker history for the image you're trying to run?

IMAGE               CREATED             CREATED BY                                      SIZE                COMMENT
3925923746cb        2 minutes ago       powershell $env:PATH = 'C:\inetpub\wwwroot\b…   29.9MB
c58215250481        3 minutes ago       powershell #(nop) ADD dir:8c9dd5dc37d8184afd…   2.08GB
534e27e8c7b7        5 minutes ago       powershell #(nop) WORKDIR C:\inetpub\wwwroot    41kB
84c1e9c483fd        5 minutes ago       powershell Start-Process -FilePath "C:/insta…   56.6MB
16eac4af5348        6 days ago          powershell #(nop) COPY dir:687b74af2d79908df…   29.6MB
194ae794a36f        6 days ago          powershell Install-WindowsFeature NET-Framew…   246MB
2ec17d490df9        6 days ago          powershell #(nop)  SHELL [powershell]           41kB
e0eeddf85095        3 weeks ago         cmd /S /C #(nop)  ENTRYPOINT ["C:\ServiceMo…   41kB
           3 weeks ago         cmd /S /C #(nop)  EXPOSE 80                     41kB
           3 weeks ago         cmd /S /C powershell -Command     Add-Window…   261MB
           3 weeks ago         Install update 10.0.16299.967                   1.85GB
           17 months ago       Apply image 10.0.16299.15                       4.62GB

Dockerfile
FROM microsoft/iis:windowsservercore-1709

SHELL ["powershell"]

RUN Install-WindowsFeature NET-Framework-45-ASPNET ; \
    Install-WindowsFeature Web-Asp-Net45 ; \
    Install-WindowsFeature Web-AppInit

# Install Visual C++ runtime for VS 2015 (required by autopilot SDK)
COPY installer "C:/installer"
RUN Start-Process -FilePath "C:/installer/vc_redist.x64.vs2015.exe" -ArgumentList /Install, /Quiet, /NoRestart, /Log, c:\install_logs\vc_redist.x64.vs2015.log -PassThru -Wait;

WORKDIR /inetpub/wwwroot
ADD "bin\Release\Publish" .

RUN $env:PATH = 'C:\inetpub\wwwroot\bin\MLGWrapper;' + $env:PATH; \
    [Environment]::SetEnvironmentVariable('PATH', $env:PATH, [EnvironmentVariableTarget]::Machine);

Ahh sorry. I just realized I misread your question.
You cannot use a newer container runtime than your host OS version. From the error message I gather you're trying to run a 1709 image on an ltsc2016 host. This is not supported.
Your options here are:

Use a newer host OS; OR
Run the container with Hyper-V isolation instead.

See https://docs.microsoft.com/en-us/virtualization/windowscontainers/deploy-containers/version-compatibility for more information

How to select newer host OS for Web App for Windows container?

How to select newer host OS for Web App for Windows container?

This is currently not supported in Azure App Service. You can currently only do this with your own VM.
I'd recommending creating a request for the Azure App Service team at https://feedback.azure.com/forums/169385-web-apps

Closing this issue as it's not actionable by the ASP.NET team.


Pin bokeh in datashader integration tests for now

Current datashader release has some incompatibilities with bokeh>=2.0, so to get tests passing this should work for now.

@philippjfr thanks!

@philippjfr it looks like there are now 13 failures in the tests, might be worth having a look at:
https://circleci.com/gh/numba/numba-integration-testing/3031


Setting up TOX

As mentioned in issue #12 we want a tox, to test multiple versions of python.
I'm planning to follow the guide found here.

Apparently while TOX can run multiple versions, you need to have them installed first. I am looking at pyenv to get something that can work across multiple versions of python.


Relay electron exit code

TL;DR: The current cli.js wrapper always exits with 0, no matter what electron actually exits with. This PR fixes that.
Real-life example: some applications (like electron itself) have an interesting test setup:

electron is launched, from which
...a BrowserWindow is created, in which
...some javascript is loaded (e.g. mocha + a bunch of specs)
The test results (number of tests failed, for instance) are sent by ipc back to the nodejs side
...where process.exit() is called so that test failures result in a non-zero exit code
which is useful when running on Travis CI by example.

Now, atom/electron isn't touched by this, since they also build a fresh copy of electron right there on the Travis CI instance so I'm assuming they run the binary directly.
However, if anyone wants to use electron-prebuilt to run tests on Travis CI via the same method, they won't be able to detect failures, as this build demonstrates (It should definitely fail).
This PR fixes that in what I'm hoping is the least possible amount of changes, and I can't think of a way it would break electron-prebuilt for anyone else. Let me know, though!

Looks good :+1:

Will be automatically released next time there is an electron release

Fantastic, thanks for the record time merge!


ci: fix more zizmor warnings

#7826  👈 (View in Graphite)
main

How to use the Graphite Merge Queue
Add either label to this PR to merge it via the merge queue:

0-merge - adds this PR to the back of the merge queue
hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

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

https://docs.renovatebot.com/modules/manager/github-actions/ seems like the best approach to version pinning.

In my opinion, pinning to a more exact tag does not provide more security.
Someone manipulating the action could still probably force-push (unless GitHub actions prevents this somehow, but I don't think so).
https://woodruffw.github.io/zizmor/audits/#unpinned-uses
zizmor's pendantic mode requires you to pin to a specific commit. I would recommend doing that instead and set up a tool to automatically bump that commit hash to the latest stable release instead.

I'm pinning these versions first in preparation for renovate bot to work.


Revenue report: UI changes as per new design

Notion card
https://www.notion.so/Update-the-designs-for-revenue-report-1a023facb2ed4ca59474af15d7d472cc
Summary
Revenue Report: UI change as per the new design
Design link
https://www.figma.com/file/4gdPDRPqGqQqPYsO3Qj8CG/Miru-App?node-id=6785%3A69221&t=NbbbcX1pJZI00RAm-0
https://www.figma.com/file/4gdPDRPqGqQqPYsO3Qj8CG/Miru-App?node-id=10587%3A151725&t=7Qo6WrpPwnShfSxU-0
Preview

How Has This Been Tested?
Check the screen as per the design on both web and mobile view.
Checklist:

[x]  I have manually tested all workflows
[x]  I have performed a self-review of my own code
[x]  I have annotated changes in the PR that are relevant to the reviewer
[ ]  I have added automated tests for my code

@Ankita012 can you try connecting your mobile to the local server and test the changes on mobile? Sometimes the UI breaks on a mobile device. you can refer this link to run project on mobile https://stackoverflow.com/questions/2019884/how-to-connect-to-a-localhost-rails-site-via-mobile-device-locally let me know incase of any doubt.

Checked


Fix RTD build round 2 - [merged]

In GitLab by @wtbarnes on Nov 21, 2019, 14:49
Merges fix-rtd-gallery -> master
See !39

In GitLab by @wtbarnes on Nov 21, 2019, 15:08
merged

In GitLab by @wtbarnes on Nov 21, 2019, 14:49
enabled an automatic merge when the pipeline for 383a7a998d6b4eee9e5caccc08a5c716e67b5508 succeeds

In GitLab by @wtbarnes on Nov 21, 2019, 15:08
mentioned in commit 28b23cf1d6634b2a53c82fcc4be0f291a2761dea

In GitLab by @wtbarnes on Mar 23, 2020, 13:30
mentioned in commit 0a6155f49a30e1cc6a267fb54518c7f786c6f4f4


No empty line between paragraphs

See for example the following pages:
https://frankdoc.frankframework.org/#!/Pipes/EscapePipe
https://frankdoc.frankframework.org/#!/Listeners/JavaListener
https://frankdoc.frankframework.org/#!/Pipes/BytesOutputPipe
In the Java source code you can see that the Javadoc of the class is divided into two paragraphs by a <p> or newline but in the browser the paragraphs are are not divided by an empty line

<p> tag in (nieuwe) console moet margin-top 10px krijgen.


Added suggestion to encrypt config properties with Jasypt

Follow up from https://github.com/killbill/killbill-platform/pull/31

Thanks!


log warn

warn: [deprecated] User.getMultipleUserFields is deprecated please use User.getUsersFields"
Why does it get such a notice?

v1.1.7

thank you


Incorrect/Missing information

This page is incorrect or missing information.
The section for 'Buy the extra storage add-on'.

There is no 'Next' on the products page even after selecting one of my subscriptions.

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

ID: b31aa550-3d84-95b8-6a74-b3e7ad1cbfe9
Version Independent ID: 795391d3-aced-e2f3-f52d-034e14fe1aee
Content: Add more SharePoint storage to your subscription
Content Source: microsoft-365/commerce/add-storage-space.md
Service: o365-administration
GitHub Login: @cmcatee-MSFT
Microsoft Alias: cmcatee

@msbemba

@markembry Thank you for your feedback. I was able to test and see the next button. I'd recommend you contact the support team via the need help option on the admin center so that they can help fix this issue.
Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @markembry Hope this comment is helpful for you. Considering the nature of the issue, as advised, please open a service ticket in your tenant and follow with them for the resolution. We proceed to close it as here we are limited to documentation fixes and improvements only.
We will be very happy if you can share the outcome or resolution with us if you see documentation update is required. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Added bootstrap container for hero section

Issue that this pull request solves
Closes: #299
Proposed changes
Added bootstrap container class for hero section on the homepage
Types of changes
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] My changes does not break the current system and it passes all the current test cases.

Screenshots
Please attach the screenshots of the changes made in case of change in user interface

@salil-naik I have done the requested changes. Please review it.

@salil-naik I have done the requested changes. Please review it.

@sakshikhachane you have not removed the padding from the content class

@sakshikhachane you have not removed the padding from the content class


Fix(securing-a-cluster.md): typo

"将" should be used here instead of "讲".

/check-cla

/check-cla

Hey there! @owlet42, looks like you haven't signed the CLA yet. Could I please have you do that? https://github.com/kubernetes/community/blob/master/CLA.md

@owlet42 Please make sure you have signed the CLA in order for the PR to be reviewed.
Thanks.

@jimangel
I am sure I have signed the CLI, There(https://identity.linuxfoundation.org/user) contains my information

First Name: chenlong
Last Name: ma
Full Name: chenlong ma
E-mail address<EMAIL_ADDRESS>

/check-cla

@owlet42 Pls try git config and see if the user.email has been correctly set to the one you used for CLA.

Pls feel free to reopen and continue working on this when the CLA is ready.
/close


youtube feeds parsing problems

other rss clients are able to display the video thumbnail alongside the video description. vienna only show the video description text but fails to show the thumbnail/video preview/still frame. instead it says there is an enclosed file, but i think it is an error of parsing. it seems in the case of youtube subscriptions, it download the whole html page and make it an attachment to the feed.
also i noticed the feed date is getting errors. i'm getting old feed from september 2020 to popup as new in december 2020. here are the steps i did: i deleted my old subscriptions and started fresh today, january the 2nd of 2021. i clicked "refresh subscriptions" button to make the initial sync, and i saw some old videos from september being marked as new from december. and not only on one subscription only but many of them showed this problem. vienna is incorrectly pulling the feed date.
as an example of these problem please see the screenshot,
vienna says the highlighted feed is from 29/12/2020
the link to this video is: https://www.youtube.com/watch?v=a8-f7y2Jkr4
when i go to that page, i can see the video is in fact from 06/11/2020

also finally, it would be nice if vienna could send links to some external player, like vlc or mpv or iina.
test environment: vienna version 3.7.0 :5698f89d: (7429) on macos bigsur.

This is from the RSS feed:
 <entry>
  <id>yt:video:a8-f7y2Jkr4</id>
  <yt:videoId>a8-f7y2Jkr4</yt:videoId>
  <yt:channelId>UC5rT7F0PGNuD54rJ9kzgWzw</yt:channelId>
  <title>Off Grid Cider Making 2020</title>
  <link rel="alternate" href="https://www.youtube.com/watch?v=a8-f7y2Jkr4"/>
  <author>
   <name>Kris Harbour Natural Building</name>
   <uri>https://www.youtube.com/channel/UC5rT7F0PGNuD54rJ9kzgWzw</uri>
  </author>
  <published>2020-11-06T14:11:10+00:00</published>
  <updated>2020-12-29T17:08:45+00:00</updated>
  <media:group>
   <media:title>Off Grid Cider Making 2020</media:title>
   <media:content url="https://www.youtube.com/v/a8-f7y2Jkr4?version=3" type="application/x-shockwave-flash" width="640" height="390"/>
   <media:thumbnail url="https://i2.ytimg.com/vi/a8-f7y2Jkr4/hqdefault.jpg" width="480" height="360"/>
   <media:description>Its the time of year again, Cider making. It was a good year for apples. we made 35L of cider and 35L of apple juice for the freezer.</media:description>
   <media:community>
    <media:starRating count="3659" average="4.96" min="1" max="5"/>
    <media:statistics views="57156"/>
   </media:community>
  </media:group>
 </entry>

The parser is working correctly. The "Date" column shows the date upon which the content was last updated, which in this case is 29 December 2020. It seems that YouTube refers to the Flash version of the video as the feed entry's "enclosure". This is what Vienna uses to have compatibility with the RSS <enclosure> element.
To support video thumbnails, Vienna would have to add support for the <media:thumbnail> element and make the table view capable of showing larger images.

This is from the RSS feed:
 <entry>
  <id>yt:video:a8-f7y2Jkr4</id>
  <yt:videoId>a8-f7y2Jkr4</yt:videoId>
  <yt:channelId>UC5rT7F0PGNuD54rJ9kzgWzw</yt:channelId>
  <title>Off Grid Cider Making 2020</title>
  <link rel="alternate" href="https://www.youtube.com/watch?v=a8-f7y2Jkr4"/>
  <author>
   <name>Kris Harbour Natural Building</name>
   <uri>https://www.youtube.com/channel/UC5rT7F0PGNuD54rJ9kzgWzw</uri>
  </author>
  <published>2020-11-06T14:11:10+00:00</published>
  <updated>2020-12-29T17:08:45+00:00</updated>
  <media:group>
   <media:title>Off Grid Cider Making 2020</media:title>
   <media:content url="https://www.youtube.com/v/a8-f7y2Jkr4?version=3" type="application/x-shockwave-flash" width="640" height="390"/>
   <media:thumbnail url="https://i2.ytimg.com/vi/a8-f7y2Jkr4/hqdefault.jpg" width="480" height="360"/>
   <media:description>Its the time of year again, Cider making. It was a good year for apples. we made 35L of cider and 35L of apple juice for the freezer.</media:description>
   <media:community>
    <media:starRating count="3659" average="4.96" min="1" max="5"/>
    <media:statistics views="57156"/>
   </media:community>
  </media:group>
 </entry>

The parser is working correctly. The "Date" column shows the date upon which the content was last updated, which in this case is 29 December 2020. It seems that YouTube refers to the Flash version of the video as the feed entry's "enclosure". This is what Vienna uses to have compatibility with the RSS <enclosure> element.
To support video thumbnails, Vienna would have to add support for the <media:thumbnail> element and make the table view capable of showing larger images.

the reasons you put forward are logic and all, but i just wanted to report that in real life cases it leads to degraded user experience and that other newsfeed readers don't seems to exhibit the same behaviour.
for example, "reeder", "leaf" and "news explorer" don't show these dates and thumbnails problems. furthermore, vienna seems capable enough to show thumbnails in other kind of feeds beside youtube ones. so considering this all, i thought it might be worth reporting.

the reasons you put forward are logic and all, but i just wanted to report that in real life cases it leads to degraded user experience and that other newsfeed readers don't seems to exhibit the same behaviour.
for example, "reeder", "leaf" and "news explorer" don't show these dates and thumbnails problems. furthermore, vienna seems capable enough to show thumbnails in other kind of feeds beside youtube ones. so considering this all, i thought it might be worth reporting.

This is not the same. This is how the feed of Macrumors looks:
<item>
<title>Apple Removes 39,000 Games From China App Store</title>
<link>https://www.macrumors.com/2020/12/31/apple-removes-39000-game-apps-from-china-store/</link>
<comments>https://forums.macrumors.com/threads/apple-removes-39-000-games-from-china-app-store.2278090</comments>
<pubDate>Thu, 31 Dec 2020 04:24:10 PST</pubDate>
<guid isPermaLink="false">https://www.macrumors.com/2020/12/31/apple-removes-39000-game-apps-from-china-store/</guid>
<dc:creator>Tim Hardwick</dc:creator>
<category>App Store</category>
<category>China</category>
<description>Apple on Thursday removed nearly 39,000 apps from its Chinese App Store due to the apps lacking an official license from local regulators, reports Reuters.

...
</description>
</item>

You can see here that the large image is directly embedded in the <description> of the feed item. Vienna's article viewer renders this <description> as HTML, including all embedded images.
In the case of YouTube, the thumbnail image is instead declared as an enclosure (i.e. attachment) to the feed entry, using a non-standard <media:thumbnail> element. To add support for this, Vienna would have to:

Extend the parser to parse <media:thumbnail>
Extend the data model to have a field for the thumbnail
Extend the view model to allow for thumbnail images to be shown in the article list

I have marked this as a feature request.

This is not the same. This is how the feed of Macrumors looks:
<item>
<title>Apple Removes 39,000 Games From China App Store</title>
<link>https://www.macrumors.com/2020/12/31/apple-removes-39000-game-apps-from-china-store/</link>
<comments>https://forums.macrumors.com/threads/apple-removes-39-000-games-from-china-app-store.2278090</comments>
<pubDate>Thu, 31 Dec 2020 04:24:10 PST</pubDate>
<guid isPermaLink="false">https://www.macrumors.com/2020/12/31/apple-removes-39000-game-apps-from-china-store/</guid>
<dc:creator>Tim Hardwick</dc:creator>
<category>App Store</category>
<category>China</category>
<description>Apple on Thursday removed nearly 39,000 apps from its Chinese App Store due to the apps lacking an official license from local regulators, reports Reuters.

...
</description>
</item>

You can see here that the large image is directly embedded in the <description> of the feed item. Vienna's article viewer renders this <description> as HTML, including all embedded images.
In the case of YouTube, the thumbnail image is instead declared as an enclosure (i.e. attachment) to the feed entry, using a non-standard <media:thumbnail> element. To add support for this, Vienna would have to:

Extend the parser to parse <media:thumbnail>
Extend the data model to have a field for the thumbnail
Extend the view model to allow for thumbnail images to be shown in the article list

I have marked this as a feature request.

this is probably one of the worst piece of code i have done.
i had never touched objectivec before.
but, fiddling with RichXMLParser.m, around block starting at line 761, here
i was able to add support for showing the youtube feed preview image.

this is probably one of the worst piece of code i have done.
i had never touched objectivec before.
but, fiddling with RichXMLParser.m, around block starting at line 761, here
i was able to add support for showing the youtube feed preview image.

i don't think any would want to use my "code" as it's probably really crappy quality,
but just in case it can serve for reference purpose:

i don't think any would want to use my "code" as it's probably really crappy quality,
but just in case it can serve for reference purpose:


Easy renames

I could probably fix these myself if I find the tiem, but some easy name replacement jobs:
utils/conf/monerod.conf
utils/systemd/monerod.service
Both files also contain references to the monero user as opposed to the edollar user.
Thought I'd share as I'm making my PKGBUILD.

Created two pull requests for the issues.


Convert to s3 to redshift worker

Actual PR for the s3-to-redshift worker. It removes the old mixpanel and postgres export functionality as we won't need that in the future and I'm pretty confident no one else uses this in the wild. This PR creates a worker which can pull from s3 into redshift using COPY looking for the most recent file or a set date.
After reading a configuration file, the worker ensures the target redshift table exists and has the proper columns. If not, it performs the proper SQL commands to get into the state dictated by the configuration file. Then it attempts to find the most recent file on s3 matching a pattern. If the s3 data is more recent than what is in redshift, it loads that s3 data into redshift.

had to remove the "remove godeps" commit for space issues, but will add that back in before merge

@drhurd the const question raf answered in slack, basically we always do vars but by convention it it's declared in the const block we don't modify them. The main problem is that you can't const a map, so then you have two sections and it gets awkward.
I'll take a look at the column map lookup, but your input on GetLatestInputData was VERY sharp, I was doing two separate things in the same function

Sorry, my comment may have been miscommunicated. I was referring to the SQL queryFormat strings, not the map. If we are not planning on changing these, they seem like they should definitely be consts - relying on convention seems unnecessary when we can let the compiler enforce this rule.

Other than that, LGTM!


Added Pre Build Event to Kill CEF Process

Added a pre-build event to kill the CefSharp.BrowserSubprocess that will lock the assemblies when debugging is stopped unexpectedly.

Could you please squash this into a single commit and then I'll merge it, having two commits that cancel each other out is bad practice :+1:

Although I disagree with reverting being bad practice, I'm not sure how to minimize this into a single commit? I'm still incredibly new to git (only proficient in SVN), do you have any suggestions on the git steps to complete?

To me the reverting thing is purely for historical reasons, looking down a commit log you shouldn't see a change was made and have to check further up that it was / wasn't reverted.
As for how to squash commits, the basic command is git rebase master -i but look into it more in this post
https://davidwalsh.name/squash-commits-git

Well, I seemed to have fubared something in the process, yay for trial and error.
I'm going to close this and have a single commit pull request later today.


cmd/compile: better error messages from types2 (Go 1.18) for unexported fields/methods

The error messages from types2 for Go 1.18, I think, as less helpful than the Go 1.17 (-G=0) messages in the case of unexported fields and members.  This is shown for the two existing bug test cases in go/test/fixedbugs, issue18419.go and issue31053.go.
We have disabled these tests in run.go because of different error message output for Go 1.17 and Go.18, and the difference is because Go 1.18 is not helpfully indicating the problem is due to an unexported field/method, but is instead indicating the field/method is unknown or doesn't exist.
(I'm going through a bunch of the other disabled tests, which are mostly due to largely equivalent error messages, just slight different words or a few extra, related errors.  One area that I will work on fixing is missing errors because of improper compiler directives (//go:...), which are mostly ignored by types2.)
You can enable the test for each of these issues by commenting out the specified issue in types2Failures array in test/run.go.
For issue 18419, the Go 1.17 error message is:
./test.go:12:3: e.member undefined (cannot refer to unexported field or method member)

whereas for Go 1.18 it is:
./test.go:12:4: e.member undefined (type *other.Exported has no field or method member, but does have Member)

For issue 31053, there are many error messages that match, but the two relevant ones that differ are for Go 1.17:
./p.go:13:3: cannot refer to unexported field 'doneChan' in struct literal of type f1.Foo
./p.go:20:3: cannot refer to unexported field 'hook' in struct literal of type f1.Foo

and for Go 1.18:
./p.go:13:3: unknown field 'doneChan' in struct literal of type f1.Foo
./p.go:20:3: unknown field 'hook' in struct literal of type f1.Foo

If it makes sense to fix, I'm sure this is fine to fix either before or after the beta release.
@griesemer @findleyr

For issue 18419, the error message is longer in 1.18 but provide equal if not more information.
For issue 31053, the two messages that are different should probably be fixed.

cc: @findleyr

For issue 18419, the error message is longer in 1.18 but provide equal if not more information.

I don't think that's entirely true.  Exported has both member and Member. The 1.17 message points out the unexported member, the 1.18 message points out the exported Member.  Both are useful, but this from the 1.18 error message seems a little misleading: *other.Exported has no field or method member.

Yes, I agree on 18419 - the message is factually incorrect.  In both cases, essentially types2 is not noting that the real problem is that the available field/method is not exported, so can't be used.  (I guess in issue 18419, you could argue that the more likely error the user is making is not calling the upper-case Member,  as opposed to the lower-case member (which is a field), though hard to guess the user's intent.)

I've mailed a fix for this for types2 per CL https://golang.org/cl/378177, please take a look.

Moving to 1.19. There's too many parts that may need to be adjusted. Also, what we have now is ok.

Still good to fix, but not urgent. Moving to 1.20.

Too late for 1.21. Moving forward.

Too late for 1.22. Moving forward.


Realm ( false advertising ) ?

Hi do you think having realm, on the README.md is misguiding people looking for project boilerplate with realm ?

Hi @sahanatroam ,
Thanks for report. The reason of your issue is that we just forgot update readme file after some changes. Take a look.

@artyorsh thanks,
amazing work overall on this project tho :)


junest: init at 7.4.8

Description of changes
A new PR was made with a complete redo as git seemed to not want to cooperate
Original

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#commit-conventions

When adding yourself as maintainer in the same pull request, make a separate commit with the message maintainers: add . Add the commit before those making changes to the package or module. See Nixpkgs Maintainers for details.

also, version in commit needs to match version of derivation ( drop 'v')

https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#commit-conventions

When adding yourself as maintainer in the same pull request, make a separate commit with the message maintainers: add . Add the commit before those making changes to the package or module. See Nixpkgs Maintainers for details.

also, version in commit needs to match version of derivation ( drop 'v')

Fixed! I renamed the commit.

you can force push to this branch, same PR will be fine

I have split the commit and put the two commits in the correct order.

not a big deal, but during setup I get
/nix/store/57g7mn1x46ywczv140gsn1s64zry6pr6-junest-7.4.8/lib/core/common.sh: line 128: wget: command not found

it still works because it falls back to curl ( i wonder why curl is installed on my system )

not a big deal, but during setup I get
/nix/store/57g7mn1x46ywczv140gsn1s64zry6pr6-junest-7.4.8/lib/core/common.sh: line 128: wget: command not found

it still works because it falls back to curl ( i wonder why curl is installed on my system )

I've added wget as a buildInput so it doesn't fallback

i get eval error during nixpkgs review
nix log /nix/store/wc345ipf7gfhk7y04blyf7sx4isx9hl4-junest-7.4.8.drv
warning: The interpretation of store paths arguments ending in `.drv` recently changed. If this command is now failing try again with '/nix/store/wc345ipf7gfhk7y04blyf7sx4isx9hl4-junest-7.4.8.drv^*'
@nix { "action": "setPhase", "phase": "unpackPhase" }
Running phase: unpackPhase
unpacking source archive /nix/store/yrix7jhz9ni8rb2sfvm3phg39qj507m7-source
source root is source
@nix { "action": "setPhase", "phase": "patchPhase" }
Running phase: patchPhase
@nix { "action": "setPhase", "phase": "updateAutotoolsGnuConfigScriptsPhase" }
Running phase: updateAutotoolsGnuConfigScriptsPhase
@nix { "action": "setPhase", "phase": "configurePhase" }
Running phase: configurePhase
no configure script, doing nothing
@nix { "action": "setPhase", "phase": "installPhase" }
Running phase: installPhase
/nix/store/rg6agzlb4gcl4w2126m16qm4mgr8jgi6-stdenv-linux/setup: eval: line 1583: syntax error near unexpected token `

i get eval error during nixpkgs review
nix log /nix/store/wc345ipf7gfhk7y04blyf7sx4isx9hl4-junest-7.4.8.drv
warning: The interpretation of store paths arguments ending in `.drv` recently changed. If this command is now failing try again with '/nix/store/wc345ipf7gfhk7y04blyf7sx4isx9hl4-junest-7.4.8.drv^*'
@nix { "action": "setPhase", "phase": "unpackPhase" }
Running phase: unpackPhase
unpacking source archive /nix/store/yrix7jhz9ni8rb2sfvm3phg39qj507m7-source
source root is source
@nix { "action": "setPhase", "phase": "patchPhase" }
Running phase: patchPhase
@nix { "action": "setPhase", "phase": "updateAutotoolsGnuConfigScriptsPhase" }
Running phase: updateAutotoolsGnuConfigScriptsPhase
@nix { "action": "setPhase", "phase": "configurePhase" }
Running phase: configurePhase
no configure script, doing nothing
@nix { "action": "setPhase", "phase": "installPhase" }
Running phase: installPhase
/nix/store/rg6agzlb4gcl4w2126m16qm4mgr8jgi6-stdenv-linux/setup: eval: line 1583: syntax error near unexpected token `

This error is caused by the following line:
    substituteInPlace $out/lib/core/common.sh --replace-fail "wget" (lib.getExe wget)

in the install phase.

i get eval error during nixpkgs review
nix log /nix/store/wc345ipf7gfhk7y04blyf7sx4isx9hl4-junest-7.4.8.drv
warning: The interpretation of store paths arguments ending in `.drv` recently changed. If this command is now failing try again with '/nix/store/wc345ipf7gfhk7y04blyf7sx4isx9hl4-junest-7.4.8.drv^*'
@nix { "action": "setPhase", "phase": "unpackPhase" }
Running phase: unpackPhase
unpacking source archive /nix/store/yrix7jhz9ni8rb2sfvm3phg39qj507m7-source
source root is source
@nix { "action": "setPhase", "phase": "patchPhase" }
Running phase: patchPhase
@nix { "action": "setPhase", "phase": "updateAutotoolsGnuConfigScriptsPhase" }
Running phase: updateAutotoolsGnuConfigScriptsPhase
@nix { "action": "setPhase", "phase": "configurePhase" }
Running phase: configurePhase
no configure script, doing nothing
@nix { "action": "setPhase", "phase": "installPhase" }
Running phase: installPhase
/nix/store/rg6agzlb4gcl4w2126m16qm4mgr8jgi6-stdenv-linux/setup: eval: line 1583: syntax error near unexpected token `

fixed!

@jdev082 thank you !


Add / remove tags buttons

This would be better than the current grab/apply design, since conversations frequently overlap more granularly, and when they don't add/remove works better.
This was described in the design Slack, but basically, a colon in the bar will display the old "Apply" button.

This was implemented


Reset offset to start of topic for dlq subscriptions

Codecov Report

Merging #349 (68afdc3) into master (4a5fd9f) will decrease coverage by 1.45%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##           master     #349      +/-   ##
==========================================
- Coverage   58.88%   57.42%   -1.46%
==========================================
  Files         123      123
  Lines        6428     6434       +6
==========================================
- Hits         3785     3695      -90
- Misses       2319     2423     +104
+ Partials      324      316       -8

Flag
Coverage Δ

integration
?

unittests
57.42% <71.42%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/messagebroker/config.go
100.00% <ø> (ø)

internal/subscriber/consumer.go
77.06% <50.00%> (-1.04%)
:arrow_down:

pkg/messagebroker/kafka.go
19.12% <100.00%> (-21.62%)
:arrow_down:

internal/topic/helper.go
100.00% <0.00%> (+50.00%)
: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 4a5fd9f...68afdc3. Read the comment docs.


Fix k8sGithubURL and kubeadm config file name

Its name was changed recently. Switch to a commit to avoid this kind of
issues in the future. Same for the socat repo.

I tested it from a clean /var/lib/kube-spawn and it worked.


Minor typo under Text options

In v1.4.1, under the Text section of the Screen Saver Options, "Extra information:", 'diagonally' is spelt with an extra 'n'.

Oops ! Thanks, will be fixed in next builds :)


Scheduling using service name, users have no way to know they have to type "Project/"

v0.26.0-rc1

Create a service in Default project named Service1
Create another service (or a container), and in scheduler

Condition = must
Field = service with the name
Value = (type in Service1)

Results:
Could not find placement.   This is because the Value field requires "Default/Service1" to be entered.
Comments:
It is not intuitive that the project name needs to be added and there is no check to help user realize why their service (or container) failed to find placement.
Expected:
Even a error message if no "project/" is found before create would be nice.  Or just place it according to rule if no project name is found.

The UI also lowercases everything in the dropdown and only shows the unique values now.

I've updated the allocator to automatically prepend this now.  Unfortunately, what's currently in master only does this if the service is referencing itself.  There's a pending PR that needs to get merged that does it for any service within the stack.

@sonchang what happens when https://github.com/rancher/rancher/issues/1577 is added?

The PR still doesn't preclude people from explicitly entering the stack name + service name for a different stack.  I guess once #1577 is added, I can add additional logic to lookup other services from other stacks and prepend the stack name if found.  And have the current stack as the preference if duplicate service names are found across stacks.

Verified Fixed v0.30.0-rc2
The original scenario was tested. Add service with service name without the stack/. Service was started on correct host. Logged 1642 to adapt to cross stack linking.


windows: Running the executable and flushing stdout

The preloader container is built on the user's system. On Windows, the files don't retain their executable flags, thus the preloader Python script can't be started just by running the script. Now explicitly running the script with python3, so whether or not the script is executable, it will run.
When attaching to a docker container, docker will buffer the standard input/output. On Windows the script seemed to hang as the reply from the preloader container might be quite small, thus docker just keeps waiting for more data before giving it to the program that attached to it. Flushing stdout seems to have the desired effect and the caller receives the reply from the preloader.
Change-type: patch
Signed-off-by: Gergely Imreh<EMAIL_ADDRESS> Front conversations

@resin-ci retest


Add stateless support

cc-runtime should provide at least two paths to read its configuration path.

default config  directory: this path is for default system configuration (managed by the linux distribution)
By default this path could be:
/usr/share/defaults/clear-containers/

sysconf directory: Path to override default configuration ( used by the system administrator)
By default this path could be:
/etc/clear-containers/

More information about stateless: https://clearlinux.org/features/stateless

needed for https://github.com/clearcontainers/packaging/issues/105

I have no idea why github thinks I removed the release-gating label - re-added.


AAP-18888 - Automated tests for Applications

This PR is adding the automated tests for Applications and closes AAP-18888

@acruzgon looks like lint check has failed, if you see the checks above
please use npm run prettier:fix


globule transative dependencies

Globule has not been updated in three years and pulls in very old dependencies. Globule should either be updated, dropped or replaced.

Are you running into a specific issue because of globule?

Closing for lack of response.


[cms] workflow remind

Coverage remained the same at 77.372% when pulling c132f45d678a51dd369621a0275ba56f8b6643da on pr-workflow-remind into ae7d82dd209b921bb60c08f902bac694a6122749 on master.


[boost-context] fix arm64 builds

See https://github.com/boostorg/context/blob/87135f923efe28a4c477be817d23f08d8e937713/CMakeLists.txt#L32

Thanks!


Update api and models packages with standard success & error response types

What
Service endpoints to use standardised success and error responses
How to review

Types to support standardised success and error responses duplicated here from other web services, should this be provided by a single module instead?
Check code against coding standards, passing tests

Who can review
Anyone

First commit has been signed-off rather than signed, fix this when branch is tidied up for PR


Suite argument has no effect.

Just installed latest version (0.20.1). My setup:
test/prot.conf.js
test/admin/client-spec.js
test/admin/dash-spec.js

Contents of prot.conf.js:
exports.config = {
  seleniumAddress: 'http://localhost:4444/wd/hub',

  capabilities: {
    'browserName': 'chrome'
  },

  // Spec patterns are relative to the location of the spec file. They may
  // include glob patterns.
    specs: ['**/*spec.js'],

    // suites is not working
    suites: {
        dash: 'admin/dash-spec.js',
        client: ['admin/*client*.js'],
    },
  // Options to be passed to Jasmine-node.
  jasmineNodeOpts: {
    showColors: true, // Use colors in the command line report.
  },
}

When I tried to run protractor ./test/prot.conf.js --suite=dash it runs all the tests. If I remove the line specs:['**/*spec.js'] then it throws this error:
Error: Spec patterns did not match any files.
  at Runner.run (/usr/local/lib/node_modules/protractor/lib/runner.js:329:11)
  at process.<anonymous>    (/usr/local/lib/node_modules/protractor/lib/runFromLauncher.js:32:14)
  at process.EventEmitter.emit (events.js:98:17)
  at handleMessage (child_process.js:318:10)
  at Pipe.channel.onread (child_process.js:345:11)
[launcher] Runner Process Exited With Error Code: 8

Expected: should run only the specified tests only.

Did this ever make the cut? I'm still seeing the same issue described exactly as above.


GC - poor performance with static word vectors + CUDA

nd4j-cuda-8.0 + Windows 10 + Cuda 8.0.44
GTX970 + 5960x @ 4GHz
Seeing excessive CPU usage + slow training performance when running a version of the w2v sentiment example on Windows
https://gist.github.com/AlexDBlack/3ea3d6faa0d003bbf985fd415a547634

Earlier in training:

Later in training:

Yep, one more case of JVM GC.
https://github.com/deeplearning4j/deeplearning4j/issues/2422

https://github.com/deeplearning4j/nd4j/pull/1731

Closing this on acconut of workspaces being implemented some time ago.


[Bug] After updating, the following error causes Import Failed:  ModuleNotFoundError: No module named 'nvdiffrast'

Thanks for releasing this. Looking forward to testing it out.
I had installed earlier and got no error mesages, however after updating to Commit 4 [0d5e579] a couple hours ago, the following error appears:
Traceback (most recent call last):
File "/blah/ComfyUI/nodes.py", line 1810, in load_custom_node
module_spec.loader.exec_module(module)
File "", line 940, in exec_module
File "", line 241, in _call_with_frames_removed
File "/blah/ComfyUI/custom_nodes/ComfyUI-3D-Pack/init.py", line 8, in
module = importlib.import_module(f".{nodes_filename}", package=name)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/blah/miniconda3/lib/python3.11/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 1204, in _gcd_import
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "", line 940, in exec_module
File "", line 241, in _call_with_frames_removed
File "/blah/ComfyUI/custom_nodes/ComfyUI-3D-Pack/nodes.py", line 6, in
from .diff_rast.diff_texturing import DiffTextureBaker
File "/blah/ComfyUI/custom_nodes/ComfyUI-3D-Pack/diff_rast/diff_texturing.py", line 8, in
from ..diff_rast.diff_mesh_renderer import Renderer
File "/blah/ComfyUI/custom_nodes/ComfyUI-3D-Pack/diff_rast/diff_mesh_renderer.py", line 8, in
import nvdiffrast.torch as dr
ModuleNotFoundError: No module named 'nvdiffrast'
Cannot import /blah/ComfyUI/custom_nodes/ComfyUI-3D-Pack module for custom nodes: No module named 'nvdiffrast'
I tried 'pip install nvdiffrast' but got the follwing response:
ERROR: Could not find a version that satisfies the requirement nvdiffrast (from versions: none)
ERROR: No matching distribution found for nvdiffrast
I'm guessing it could be something to do with a torch version conflict, I am using (and require) version 2.1.2 for several other custom nodes. I tried uninstalling and doing a fresh install via the manager and the issue persists.
Any ideas?

Sorry my friend, this is project is still a working progress, you are right, so far I only manage to make it work inside Miniconda/Conda environment with cuda 11.8 and corresponding torch version.
Last version of nvdiffrast is updated 7 months ago and I couldn't find out why I can't build it with ComfyUI's embed python env.
I'll see if I can make it work today, and update document.
Thanks for testing, and apologies again for the trouble, let's work together to bring the best 3D AI Tools for everyone :)

Hi friend, it is working now with both torch version 2.1.2 + cuda 121 / 118, Minicond Python 3.11
I also updated the readme and include some example workflow with input data, you are welcome to give a another try anytime 👍

!git clone --recursive https://github.com/NVlabs/nvdiffrast
!git clone --recursive https://github.com/graphdeco-inria/gaussian-splatting

%cd  path/to/nvdiffrast
!pip install .

%cd path/to/gaussian-splatting
!pip install submodules/diff-gaussian-rasterization
!pip install submodules/simple-knn

will fix the issue

!git clone --recursive https://github.com/NVlabs/nvdiffrast
!git clone --recursive https://github.com/graphdeco-inria/gaussian-splatting

%cd  path/to/nvdiffrast
!pip install .

%cd path/to/gaussian-splatting
!pip install submodules/diff-gaussian-rasterization
!pip install submodules/simple-knn

will fix the issue

Is this approach possible for the Portable version?

@MrForExample I've just spotted this, thanks! Will give the update a try later today, and let you know how I'm getting on with the latest commit.
Really excited to see what I can do with this, been getting very interested in gaussian splatting again recently, especially 4D gaussian splatting for realtime use.
Any chance of a ComfyUI-4D-Pack next....? ;D

!git clone --recursive https://github.com/NVlabs/nvdiffrast
!git clone --recursive https://github.com/graphdeco-inria/gaussian-splatting

%cd  path/to/nvdiffrast
!pip install .

%cd path/to/gaussian-splatting
!pip install submodules/diff-gaussian-rasterization
!pip install submodules/simple-knn

will fix the issue

Is this approach possible for the Portable version?

From what I tested, not work on Windows, I got CUDA build error with that, still don't know how to solve it, but maybe it was because I have too many packages installed on my PC, so you are welcome to give a try though 👍

@MrForExample I've just spotted this, thanks! Will give the update a try later today, and let you know how I'm getting on with the latest commit.
Really excited to see what I can do with this, been getting very interested in gaussian splatting again recently, especially 4D gaussian splatting for realtime use.
Any chance of a ComfyUI-4D-Pack next....? ;D

Haha, I was waiting for something like this that allow this kind of Modular free combination for 3D gen AI for a few months now, seems most researchers are too busy to build their own awesome research, design their own pipeline, nobody really want spent some effort to make it easier for everyday user, so I decide to do it myself.
ComfyUI-4D-Pack!!! Well, let's do one step at time, I also want to add some text to 3D-Animation in to ComfyUI.
But if past eight years of experience taught me anything then is that: seems slow but steady is always faster than looks fast but flashy. I'm still trying to identify the best use case for this tool, because there are so many direction I could go really deep, it would be completely waste of effort if I chosen the wrong one 👍


Added AOE QuestionnaireResponse to Health Gorilla example

coverage: 93.929%. remained the same
when pulling b898563c617b733175f591bf01fb45bc95369f50 on cody-health-gorilla-aoe-example
into 5230a7ce13abdc14d288d85cba12f7c98e3ad5d9 on main.


Zone cleaning using SDK/API

Hi, I'm working with a D7 in Node,  would be possible to control the "zone cleaning" through the SDK or API? When do you have plans to release this feature?

Hello Carolina,
As per the email we do have plans but a date is not established yet.
Best,
r.


Token verification is hard-coded to production

OmniAuth::Strategies::NitroId.decode_logout_token, OmniAuth::Strategies::NitroId.fetch_jwks and OmniAuth::Strategies::NitroId.introspect_token all use default_options, resolving the issuer to id.powerhrg.com (or similar for Stream Financial) regardless of any configuration override passed to OmniAuth::Builder.
This means I am unable to use the strategy to debug client applications against instances of NitroID other than the production ones.

@benlangfeld Thanks! We have a ticket to address this.

This has been addressed in #46 and released in v1.4.0. Closing.


gping: init at 1.1

Motivation for this change
gping is a utility to display a graph of ping timing in a terminal.  It looks like this:

Things done

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

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

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

@nlewo - Okay, fixed the blank line. I also updated to use Python 3, since it uses subprocess.getoutput in one place, which is only available in Python 3. And finally, I patched the path to the ping binary. This means it won't pick up the setuid wrapper under /run/wrappers/bin, but I like this approach better (and one can always add a wrapper for the gping binary).

Thx!


[FR] Don't downgrade flag

I had just bumped telegram-desktop-dev with cask-repair and when I run brew cu --all --yes I see:
[...]
==> Found outdated apps
[...]
2/2  telegram-desktop-dev       1.5.12   1.5.11   Y   [ FORCED ]

@yurikoles Try to use -f

@mkll
    -f  --force           Include apps that are marked as latest
                          (i.e. force-reinstall them).

I don't want that

Obviously, I misunderstood what you need.

the new version should not be included in the "outdated app" section
PS: i understand your issue right?

Hey @yurikoles , thanks for the suggestion, but not sure if that is easily possible. It would probably work fine if the version would be in some "1.1.1" format. But i.e. java has a version 8u202,b08:1961070e4c9b4e26a04e7f5a083f551e and there would need to be some sort of custom per cask comparator, where you can decide if it is greater or not.
I don't think that is going to change anytime soon and the "problem" you are describing here is just for a small number of people (that are cask contributors), not really work for anyone else.
Or do you see any other use case I'm missing here?

@ondrejfuhrer, yes you are right, it seems to break other apps.

maybe as a workaround it is possible to exclude it (until the cask formula is submitted to cask repo) with the pin version command?


Provide a way to disable/archive a deleted GitHub repository

There is a Rugged::SshError exception every time a git fetch is attempted on a deleted GitHub repository, e.g. https://app.honeybadger.io/projects/44071/faults/44715969#notice-context

I guess a disabled column on GitRepositoryLocation would do it, and an enabled scope which limits the enumeration of repositories to all repositories with disabled: false.


[Key Vault] Buid fixes

When I tried to release these packages, the build failed with the following:
/usr/bin/pwsh -NoLogo -NoProfile -NonInteractive -Command . '/home/vsts/work/_temp/2e50d22d-8ff5-4439-b7fb-95f92dfe83e4.ps1'
Found the following change log entry for version '4.1.0' in [/home/vsts/work/1/s/sdk/keyvault/keyvault-secrets/CHANGELOG.md].
-----
## 4.1.0 (2020-08-12)

-----
Confirm-ChangeLogEntry: /home/vsts/work/1/s/eng/common/scripts/Verify-ChangeLog.ps1:40
Line |
  40 |  … ChangeLog = Confirm-ChangeLogEntry -ChangeLogLocation $PackageProp.pk …
     |                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Entry has no content. Please ensure to provide some content of
     | what changed in this version.

##[error]PowerShell exited with code '1'.

Here's my attempt to fix this.
Also, nightly builds were reporting some issues with the recent test changes. I'll run the live tests as part of this PR with the fixes I'm making.

I spotted some issues with the live keyvault tests. I'll be back with an update.

/azp run js - keyvault

/azp run js - keyvault


Update to amazon-kinesis-connectors 1.1.2 from maven

As of 27 May 15 amazon-kinesis-connectors can be installed using maven instead of git sub module.

Sweet, thanks for the info! I'll have a look at this tomorrow. Glad if the project's been helpful to you.

Sorry, took a while! Merged your changes and published v0.2. Thanks.


MenuItemBinding

In your Cardsample - com.example.android.glass.cardsample.menu you use the class MenuItemBinding but that class doesnt exist

Bumping the issue


fix string to int conversion issue for arguments

had
pq: invalid input syntax for integer: "%!d(MISSING)ude%!"(MISSING)
don't search with strings on int fields
use "%keyword%" only for string keywords

Hi @fvbock
I think this is wrong, a field is not string doesn't means it is int? so we should not convert the keyword to int?

agreed. we also need to test for floats. maybe limit the types of fields we allow search on?
the problem is that we get errors like
pq: invalid input syntax for integer: "%!d(MISSING)ude%!"(MISSING)
if we use the %% template for numeric values

@fvbock I have fixed it like this for now. f188358ac2a8474a33b0e8d94d3a1869c70a063e

will give me
[2015-07-07 17:12:43]  pq: invalid input syntax for integer: "title"
updated the PR with more type checking...


Day name not abbreviated

Why is it possible to get the abbr name of the day but not the name of the day not abbreviated ?

I have never needed to use them, so I've never added them.


Use newer / more common test framework

nyc/ava is a relic of a previous dev, but no one uses it.
We cannot use watch tests anyway with this codebase, as they run a minute or so full relayer tests.
We could migrate to something more common. And ideally that gives better stack traces on errors

jest is very common (and has test watching). In CosmJS we still use Jasmine because in contrast to jest it runs well in the browser.
Both share the same syntax.

We have no need to watching.
I am not sure if we care much about the browser use case, but I consider it a nice to have
Either jasmine or jest seem reasonable. Happy for a pr to either one.


Server Crashes

Hi!
When I use Double Slabs on my 1.12.2 server (because of some other mods) it crashes. I also tried removing other mods but that did not work.
Here crash report (load.jar is just forge server/loader so i thats not important).
Thank you for this amazing mod anyways!
crash.txt

Hi there! This issue has been resolved in the latest version.. please update :)


Update required in Readme.md File

Currently Our readme.md file is not looking good
It is looking like a learn.md file I would suggest to @ashitaprasad assign me this issue I will create a fully new version of readme file , have created the same in many repository , I am working under gssoc
changes required are missing video of our project , allignment of text , open source section , contributors section , code of conduct, license section , features section and many more I will create a beautiful attracting readme file .

Please go through the contributing guide to understand the project requirements and then raise a valid issue.


Fix #1281: glfwPostEmptyEvent sometimes doesn't wake up glfwWaitEvents on X11

This is a rare race that triggers often when running GLFW programs on GitHub Actions with Xvfb.
The underlying issue is internal to Xlib: XSendEvent races with an external select() call and may get dispatched before select() has blocked on the display fd.
The fix observes a pipe in the select() call, and writes to that pipe from glfwPostEmptyEvent to make select() return.
The original issue reproduces locally once per minute when running the windowjs/windowjs tests in a loop;
The bug hasn't triggered after running for several hours with the fix.

Hi GLFW maintainers. This is my first PR on GitHub; please advise if this isn't the right way to contribute fixes.
I'm using GLFW for windowjs.org and spent quite some time chasing this down, mainly due to flakes in the GitHub Actions test runners due to the issue.
The discussion in #1281 already mentioned using a self-pipe to fix the issue but activity seems stale; I've just updated and polished the patch, and addressed just the issue at hand (i.e. this doesn't replace select() with poll() to reduce risk).
This is applied manually to windowjs/windowjs now, would be great to have it fixed upstream :-)

Thank you for the PR! This should be fixed now with cd22e2849512a88d0ab77bc7a3458646625f2c50, which was partly based on it.


Issue with gRPC AIO in Python when using Cythonized Coroutines

What version of gRPC and what language are you using?
1.59.0
What operating system (Linux, Windows,...) and version?
Tested on: Linux & MacOS
What runtime / compiler are you using (e.g. python version or version of gcc)
Python 3.8.X and Python 3.9.X
Cython 3.0.8
What did you do?
Please provide either 1) A unit test for reproducing the bug or 2) Specific steps for us to follow to reproduce the bug. If there’s not enough information to debug the problem, gRPC team may close the issue at their discretion. You’re welcome to re-open the issue once you have a reproduction.
Create an Async server and call a method with a sync client, while cythonizing the code.
What did you expect to see?
Correct return from the Server (works non-cythonized)
What did you see instead?
Unexpected <class 'TypeError'>: descriptor 'SerializeToString' for 'google._upb._message.Message' objects doesn't apply to a '_cython_3_0_8.coroutine' object

Full
 File "/home/xanrin/.pyenv/versions/3.8.18/lib/python3.8/site-packages/grpc/_channel.py", line 1161, in __call__
E       return _end_unary_response_blocking(state, call, False, None)
E     File "/home/xanrin/.pyenv/versions/3.8.18/lib/python3.8/site-packages/grpc/_channel.py", line 1004, in _end_unary_response_blocking
E       raise _InactiveRpcError(state)  # pytype: disable=not-instantiable
E   grpc._channel._InactiveRpcError: <_InactiveRpcError of RPC that terminated with:
E       status = StatusCode.UNKNOWN
E       details = "Unexpected <class 'TypeError'>: descriptor 'SerializeToString' for 'google._upb._message.Message' objects doesn't apply to a '_cython_3_0_8.coroutine' object"
E       debug_error_string = "UNKNOWN:Error received from peer  {created_time:"2024-02-05T08:27:42.169715932+01:00", grpc_status:2, grpc_message:"Unexpected <class \'TypeError\'>: descriptor \'SerializeToString\' for \'google._upb._message.Message\' objects doesn\'t apply to a \'_cython_3_0_8.coroutine\' object"}"

Anything else we should know about your project / environment?
I am using a Sync gRPC Client with an Async gRPC Server
Reproduction Repository:
https://github.com/XavierGeerinck/issue-grpc-async-cython
Other Issues referencing this:
https://github.com/cython/cython/issues/5925
https://stackoverflow.com/questions/69902723/cythonized-async-python-grpc-server-not-working
https://github.com/grpc/grpc/issues/35406

Update: added a Reproduction repository illustrating the issue

Update: A solution I found is to rewrite the add_GreeterServicer_to_server to handle the correct cython coroutine checking:
def is_cython_coroutine(fn):
    """
    Check if the function is a coroutine, supporting Cythonized functions.
    """
    return bool(
        fn.__code__.co_flags & CO_COROUTINE or getattr(fn, "_is_coroutine", False)
    )

def add_GreeterServicer_to_server(servicer, server):
    def wrap_handler(handler):
        """
        Wrap the handler to await it if it's a coroutine.
        """
        if is_cython_coroutine(handler):

            async def coroutine_handler(request, context):
                return await handler(request, context)

            return coroutine_handler
        else:
            return handler

    rpc_method_handlers = {
        "SayHello": grpc.unary_unary_rpc_method_handler(
            wrap_handler(servicer.SayHello),
            request_deserializer=helloworld__pb2.HelloRequest.FromString,
            response_serializer=helloworld__pb2.HelloReply.SerializeToString,
        ),
    }

    generic_handler = grpc.method_handlers_generic_handler(
        "helloworld.Greeter", rpc_method_handlers
    )
    server.add_generic_rpc_handlers((generic_handler,))

Is this the way to go? Or will/should this be supported?

Update: found out that there is an issue open (#35406) which is similar to this issue. As I understood it, this is due to Cython 3.X

Thanks for reporting the issue. I'll reopen that issue to track Cython 3.0 support and close this one as duplicate.


Is there a way to use http.ServeFile() of a file from embed.FS?

Suppose you have a FS structure of your project
├── go.mod
├── main.go
└── static/
    ├── global.css
    └── page.html

and you would like to serve contents of page.html on "/" path on your web server.
I only found solution using embededFiles.ReadFile("static/page.html"), but then I have to use a Handler and w.Write() the result from there, which is less elegant than http.ServeFile(), which I failed to adopt as a solution to the task.

See https://github.com/golang/go/issues/51971 and https://github.com/carlmjohnson/rootdown/blob/91a3eedd33f1/router.go#L105-L120

Thank you for both links! Very spot on, I highly appreciate your reply.


Fix bitcoin core version number

0.19.1 will be out soon, but not quite yet ;-)

Maybe also fix the litecoin core version number to 0.17.1 (https://github.com/rootzoll/raspiblitz/blob/master/FAQ.md#whats-new-in-version-14-of-raspiblitz)

Got to excited there ;) Thanks


Allow duplicate placeholder configs if desired

Hi all,
I was very excited to see this project  - we're facing the same architectural challenges that you are.  One thing, though, is that in Spring MVC web applications, there is often a need to have PropertyPlaceholderConfigurers in both the root application context and the dispatcher servlet application context (because Bean Post Processors can't cross parent-child context boundaries).
This change allows the user to specify that they're ok if multiple placeholder configurers register, and will just use the config of the first one that makes it.
I'm open to feedback, but this helped us tremendously.
Thanks!
Scott

Capgemini » archaius-spring-adapter #54 SUCCESS
This pull request looks good
(what's this?)

Hi @sdwr98 . It's very cool that your itch is also our itch.  And thanks for the contribution. I'll take a look at your PR.  The cloudbees build looks good for starters...
Andrew

One comment - could you add some tests? (E.g. check it works as expected, check it fails as expected, with the various ways you can config a Spring project).  Just use the existing ones in src/test as examples (they're in Spock but thats pretty easy to read).
Then I think we're good to go.

@andrewharmellaw Whoops - sorry about that. I didn't even see those hiding in the groovy folder.  I've added some tests, let me know what you think.

Capgemini » archaius-spring-adapter #55 SUCCESS
This pull request looks good
(what's this?)

All looks good now.  Apologies for taking so long to merge.  Doing it now.  Thanks again.