Make write_*(), finish() inlineable in release builds

By adding #[inline] attributes, rustc will be able to inline these functions across crate boundaries if it wants to. This allows nohash-hasher to entirely disappear at compile time.
At the moment, nohash-hasher generates code like this (decompiled to pseudo-C using IDA):

where the first call does *arg0 = arg2 and the second call does return *arg0. Conceptually:
v31 = 0;
v31 = a2;
v7 = v31;

This should have a negligible positive impact on performance in 99% of real world use cases. But it would feel nice if it just did v7 = a2 instead, without the calls!

[clabot:check]

Thank you for your PR! Does link-time optimisation not inline the calls?

Hmm, yes, it does :stuck_out_tongue:
I think this would still be nice but won't argue if you have a different opinion. If LTO is disabled, you probably don't have performance requirements where 5 or 6 additional asm instructions make a dent!

Would the widespread use of #[inline] in libraries not slow down compilation and potentially cause code bloat with too much inlining going on? I am mostly worried about the accumulated effect of it and if LTO solves the same problem I would rather leave it to it.


Add option to Create an API using a service in Service Catalog

Describe your problem(s)
We should add a "Create API" option in the Service Catalog UI to create a new api using the selected service.

Fixed with https://github.com/wso2/carbon-apimgt/pull/9526/commits/6c0a21e3a4b4f914bab37a9d2e17bfced4f627d1

Closing as completed.


--log-level  or -ll parameter non accepted, unrecognized arguments

System OS
Linux
Python Version
3.7 (CPython)
Install Source
pip / PyPi
Install version / commit hash
3.9.0
Expected Behavior vs Actual Behavior
[root@static ~]# spotdl -ll DEBUG https://open.spotify.com/track/1z1Hg7Vb0AhHDiEmnDE79l
usage: spotdl [-h] [--debug-termination] [--output OUTPUT]
[--output-format {mp3,ogg,opus,flac,m4a,wav}] [--user-auth]
[--use-youtube] [--lyrics-provider {genius,musixmatch}]
[-p PATH_TEMPLATE] [-f FFMPEG] [--ignore-ffmpeg-version]
[--download-threads DOWNLOAD_THREADS]
[--search-threads SEARCH_THREADS] [--generate-m3u] [-v]
query [query ...]
spotdl: error: unrecognized arguments: -ll
[root@static ~]# spotdl --log-level DEBUG  https://open.spotify.com/track/1z1Hg7Vb0AhHDiEmnDE79l
usage: spotdl [-h] [--debug-termination] [--output OUTPUT]
[--output-format {flac,opus,mp3,wav,m4a,ogg}] [--user-auth]
[--use-youtube] [--lyrics-provider {genius,musixmatch}]
[-p PATH_TEMPLATE] [-f FFMPEG] [--ignore-ffmpeg-version]
[--download-threads DOWNLOAD_THREADS]
[--search-threads SEARCH_THREADS] [--generate-m3u] [-v]
query [query ...]
spotdl: error: unrecognized arguments: --log-level
Steps to reproduce
I can't use the log level for check status of ffmpeg and other log output
why it's not recognized ??
Traceback
[root@static ~]# spotdl --log-level DEBUG  https://open.spotify.com/track/1z1Hg7Vb0AhHDiEmnDE79l
usage: spotdl [-h] [--debug-termination] [--output OUTPUT]
              [--output-format {flac,opus,mp3,wav,m4a,ogg}] [--user-auth]
              [--use-youtube] [--lyrics-provider {genius,musixmatch}]
              [-p PATH_TEMPLATE] [-f FFMPEG] [--ignore-ffmpeg-version]
              [--download-threads DOWNLOAD_THREADS]
              [--search-threads SEARCH_THREADS] [--generate-m3u] [-v]
              query [query ...]
spotdl: error: unrecognized arguments: --log-level

Other details
No response

There is no log level argument anymore. You are probably looking at the v2 docs.


HSQLDB: Unable to parse DROP INDEX IF EXISTS statement

Which version and edition of Flyway are you using?
6.0.7
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Spring Boot integration
Which database are you using (type & version)?
HSQLDB 2.5.0
Which operating system are you using?
macOS 10.14.6
What did you do?
Wrote a migration with a simple conditional DROP INDEX statement:
DROP INDEX IF EXISTS blog_post__name_idx;

What did you expect to see?
Successful migration and a dropped index.
What did you see instead?
org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'flywayInitializer' defined in class path resource [org/springframework/boot/autoconfigure/flyway/FlywayAutoConfiguration$FlywayConfiguration.class]: Invocation of init method failed; nested exception is org.flywaydb.core.api.FlywayException: Unable to parse statement in db/migration/common/V2__drop_blog_post_name_idx.sql at line 1 col 1: Incomplete statement at line 1 col 1: DROP INDEX IF EXISTS blog_post__name_idx;

Also when I debug into the HSQLDBParser in line 77 I can see that the INDEX token has the type IDENTIFIER which seams wrong and it is therefore looking for the token DROP in CONDITIONALLY_CREATABLE_OBJECTS which it won't find. It is then increasing the blockDepth and failing later because of that.
This might be related to #2442

It's not the same as #2442 but it's definitely failing to spot INDEX as a keyword that can appear in the form DROP <THING> IF EXISTS. I have a fix which should go in 6.0.8.


textTransform uppercase does not work on Android

Description
When passing in a style to the Text component with TextStyle.textTransform set to "uppercase" it works on iOS but not Android. This is specific to when new architecture is enabled.
React Native Version
0.72.0
Output of npx react-native info
System:
OS: macOS 13.4.1
CPU: (10) arm64 Apple M1 Max
Memory: 345.33 MB / 64.00 GB
Shell:
version: "5.9"
path: /bin/zsh
Binaries:
Node:
version: 16.14.2
path: ~/.nvm/versions/node/v16.13.0/bin/node
Yarn:
version: 1.22.19
path: /opt/homebrew/bin/yarn
npm:
version: 8.1.0
path: ~/.nvm/versions/node/v16.13.0/bin/npm
Watchman: Not Found
Managers:
CocoaPods:
version: 1.12.1
path: /Users/arron.gill/.asdf/shims/pod
SDKs:
iOS SDK:
Platforms:
- DriverKit 22.4
- iOS 16.4
- macOS 13.3
- tvOS 16.4
- watchOS 9.4
Android SDK:
API Levels:
- "27"
- "28"
- "31"
- "32"
- "33"
Build Tools:
- 27.0.3
- 28.0.0
- 28.0.3
- 30.0.2
- 30.0.3
- 31.0.0
- 32.0.0
- 33.0.0
System Images:
- android-23 | Google APIs ARM 64 v8a
- android-28 | ARM 64 v8a
- android-28 | Intel x86 Atom_64
- android-28 | Google ARM64-V8a Play ARM 64 v8a
- android-29 | Google Play ARM 64 v8a
- android-30 | Google APIs ARM 64 v8a
- android-30 | Google Play ARM 64 v8a
- android-30 | Google APIs ATD ARM 64 v8a
- android-32 | Google APIs ARM 64 v8a
- android-32 | Google Play ARM 64 v8a
- android-33 | Google Play ARM 64 v8a
Android NDK: 22.1.7171670
IDEs:
Android Studio: Flamingo 2022.2.1 Patch 2 Flamingo 2022.2.1 Patch 2
Xcode:
version: 14.3.1/14E300c
path: /usr/bin/xcodebuild
Languages:
Java:
version: <IP_ADDRESS>
path: /usr/bin/javac
Ruby:
version: 3.0.3
path: /Users/arron.gill/.asdf/shims/ruby
npmPackages:
"@react-native-community/cli": Not Found
react: Not Found
react-native: Not Found
react-native-macos: Not Found
npmGlobalPackages:
"react-native": Not Found
Android:
hermesEnabled: true
newArchEnabled: true
iOS:
hermesEnabled: true
newArchEnabled: true
Steps to reproduce
Assign a style to the RN Text Component where you have property textTransform set to "uppercase". It will work on iOS Sim but not Android emulator.
Snack, code example, screenshot, or link to a repository
 <Text style={{textTransform: 'uppercase'}}>{label}</Text>

any updates on this usse, I face the same issue using this example:
<Text style={{ textTransform: 'uppercase' }}>any text</Text>
"react": "18.2.0",
"react-native": "0.72.1",

I'm experiencing this issue as well, this affects my screen labels in @react-navigation/material-top-tabs in my application when testing it with the new architecture.

This bug is still applicable and isn't missing a reproducible example? It's a super simple bug to reproduce and is really annoying since it even affects react-navigation tab labels, this issue should be reopened (not sure who to ping for this)

It's a super simple bug to reproduce and is really annoying since it even affects react-navigation tab labels, this issue should be reopened (not sure who to ping for this)

As it's super simple to reproduce, please open a new issue, provide a reproducer, and we'll properly triage it 👍

#39524 alright, done here

@cortinico Is the code for android platform even implemented? I see that one exists for iOS, not the other way around.


AML should be replaced with ALM

Please check for the below sentence on top of the page:
"Pipelines in Power Platform bring application lifecycle management (AML) automation to streamline development processes with reduced effort."
In the above statement, "(AML)" should be replaced with "(ALM)". Looks like a typo error.
This may mislead new users.
Regards
Krishna Rachakonda
MVP

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

ID: e8951c0a-f807-3e8d-2fa2-abeb81042296
Version Independent ID: e8951c0a-f807-3e8d-2fa2-abeb81042296
Content: Activating Managed Environments - Power Platform
Content Source: power-platform/guidance/white-papers/managed-environment-activation.md
Service: power-platform
Sub-service: guidance
GitHub Login: @rsantos00
Microsoft Alias: rstand

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

Fixed, waiting for merge


Update azure-storage-explorer installer link

The current manifest is downloading and installing Azure Data Studio instead.

Thanks for the PR, I've updated the manifest to point to the github repo to avoid future issues as well.
https://github.com/appget/appget.packages/pull/4536/files


Build failure: mailmanPackages.hyperkitty

pkgs.mailmanPackages.hyperkitty fails its test suite in nixos-unstable, which means that I cannot update any machine that's running mailman.

Ping @fpletz @alyssais for help.

Upstream bug: https://gitlab.com/mailman/hyperkitty/-/issues/514

https://github.com/NixOS/nixpkgs/pull/327648


NET.2.0 not working

So i just tried to update TwitchLib, but i not working with the Error:
System.IO.FileNotFoundException: File or Assembly "netstandard" not found or something like that, and i can't add netstandard reference.
Where is the problem, i hope you can help.
Regards Kiyoshi

If you are using the git repo and building yourself, you need .Net Core 2.0.2 Installed and at least patch 15.3 for Visual Studio 2017.
You also have to target a minimum runtime version of .net 4.6.1, and add a nuget reference to NetStandard.Library

No i'm downloaded from github ^^ but i can't change the netcore, it' locked by the project :/

You don't need to change to .net core
You simply have to target a minimum framework version of .Net Framework 4.6.1

Yes i changed the framework minimum but not working and displayed the error System.IO.FileNotFoundException: File or Assembly "netstandard" not found or something like that,

If you have Visual Studio 2017 minimum patch 15.3 (current live version is 15.4) it will automatically handle this, as thats when .net standard 2.0 was introduced.
If not - add a nuget reference to NetStandard.Library
Take a look at the PubSub test app in the testing folder - that is a .net framework 4.6.1 application

Also - if you are building the Repo version, you will need .Net Standard 2.0 sdk locally on your machine.

With the SDK installed locally, and visual studio running on the latest version, this is the result of referencing the library in a brand new Windows Classic Desktop WinForms 4.6.1 application:

I dont need to add a application to the project but just compile the TwitchLib.dll to use it on my own project :/

Then you dont need to change anything
Install .Net Standard 2.0 SDK (which will be needed to build a .net standard library)
You can get that from the link I posted above.
Once that is installed - if you don't want to update your visual studio, simply open a command prompt in the TwitchLib\TwitchLib folder, and run:
dotnet restore
dotnet build

You'll have your class library in the bin folder.

I'll ask swifty to get a new version up on Nuget when he gets a chance

Hi guys. I just pushed out 1.5.8 twitchlib package to nuget which should be what you're looking for.

So now i think the problem is fixed, but when you update the source here on github, i need to import the update from nuget?
Regards Kiyo

You have a choice really
Github is updated frequently, once enough feature changes have occurred to warrant a new subversion, or critical hotfixes have been applied, then a new package will be pushed to NuGet, which will show as an update in Visual Studio

Also i don't need to update from github, but it's enough to update from nuget if update is available.

It's fine to stick with nuget - unless you want "bleeding edge" most current library changes.
the Nuget package is updated very often though.


Json stringify of any object extending EntityBase will mute and remove the CreationDate attribute

Hi,
We sometimes log entities returned by the sdk. Our logger (winston) stringifies those entities.
In the toJSON method of the EntityBase in the sdk, there is a mutating instruction removing CreationDate:
https://github.com/Mangopay/mangopay2-nodejs-sdk/blob/c65e6dbcd2318fea36634f3a959529353ccf6a68/lib/models/EntityBase.js#L47
This is quite annoying to us.
I believe there are good reasons for having this specific toJSON override and stripping the creationDate property (even though we would enjoy having this information in the logs) but what is really challenging for us is that the toJSON mutates the logged object.
If you really need to keep this overridden serializer, do you think there could be a possibility to clone the object serialized before removing any field to it ?
Regards,

Close and merged with #355


Fixed: flush the cash with delay. Fixes #350. Ref #224

Description
This PR adds a new method named RemoveWithDelay on a cache instance. When a PU is removed, the contextTracker cache is now removed with a delay of 10 sec. This is fixing the missed collection flow we had last week.
Test plan
Unit tests have been added on the cache object itself.

Fixes #350

Codecov Report

Merging #355 into master will increase coverage by 0.07%.
The diff coverage is 76.19%.

@@            Coverage Diff             @@
##           master     #355      +/-   ##
==========================================
+ Coverage   59.54%   59.61%   +0.07%
==========================================
  Files          51       51
  Lines        5816     5836      +20
==========================================
+ Hits         3463     3479      +16
- Misses       2087     2089       +2
- Partials      266      268       +2

Impacted Files
Coverage Δ

enforcer/datapath.go
64.77% <0%> (ø)
:arrow_up:

cache/cache.go
83.89% <80%> (-0.61%)
:arrow_down:

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 f503b30...c16e4ec. Read the comment docs.

What is this solving exactly?
I tried to understand the linked issue but I'm not sure I fully understand. While adding a delay to remove from the cache might work, I'm worried we are missing the underlying issue. What if 10 seconds is not "enough" in some corner cases?
@amitlimaye what is your view on this

We have some flows that are not reported because the PU is gone and the cache has been flushed. The flow can not be collected as it could not find the PU identifier that matches. Previously, the system was raising a panic. @ericrpowers made a change to avoid the panic, but as a result, the flow is never reported.
It is clearly a timing issue: The flow is not reported yet, but the PU is already gone.
The whole idea of our discussion today with @dstiliadis is to provide a way to avoid cleaning the cache directly so the flow can be collected first.
The problem has been reported by Eric while using the soaking pipelines. Alex had some similar issue in its test case as well this morning.

Enforcer PR has passed. Merging


Incorporate Monitor Ingestion Beta UX study feedback

Tracking issue to update the next Azure Monitor Ingestion release to incorporate UX study feedback.
Action items:

README file changes:

[ ] Add a “Handle logs ingestion response” section that shows the response object hierarchy. See Handle logs query response for an example of this idea in the Monitor Query library’s README. Include a blurb about whether there can be more than 1 error message associated with a failed log.
[ ] Add guidance on factors to consider when setting the maximum concurrency level.
[ ] Add a “Retrieve logs” section that links to the Azure Monitor Query library’s README file.

API changes:

[x] Add an UploadLogsOptions class in which you allow for setting of the max concurrency level via the constructor.
[x] Update XML docs for the UploadLogsResult.Errors property to indicate whether a null or an empty list is returned when no errors are present.
[ ] Add an IServiceCollection extension method, named AddLogsIngestionClient, to support registering a client in the DI container. See example at https://github.com/Azure/azure-sdk-for-net/blob/bfbf35588396fd01966d734d218f80e1ee8b8597/sdk/keyvault/Azure.Security.KeyVault.Secrets/src/SecretClientBuilderExtensions.cs.

Also add to ReadMe: ClientBuilderExtensions class sample


Bitbucket not using provided Username/Password

Not sure if this is by design but the bitbucket exporter is not using the username/token provided when making a call to the api
https://github.com/konveyor/pelorus/blob/master/exporters/committime/collector_bitbucket.py#L124
response = self.__session.request("GET", url=url, headers=self.DEFAULT_HEADERS)

Nevermind I did not see it being set earlier, and just ran into a basic wrong password issues on my side.


More' Removed code block that is not need in More.lagda.

Based on #210, I removed the code block in More.lagda.
Though I removed the code block, some instruction to add code for exercise in the existing code might helpful for readers. Because I was not certain how I solved the exercise in the beginning.
Instead of removing the code block, it might be better to change the comment to suggest how to solve. Please feel free to update this pr.

Somehow, this branch has build conflicts.

Thank you both!


Update PDF Header

== Ready to Merge ==

Note that you might have to clear browser history to get the header to update
Also just edited the UG together to include guiding examples for Navigation Guide

Good Job 👍


Fixed service context menu broken by electron v2.x.x

menu.popup() now requires an options argument

Pull Request Test Coverage Report for Build 854

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

Totals

Change from base Build 848:
0.0%

Covered Lines:
3779

Relevant Lines:
6584

💛  - Coveralls

Dupe of #863


Adding a gig creates a new record in the db, but the new gig does not show up on the gig list.

It looks like the Foreign Key "CompanyContactID" is not being entered properly in the "Company" table.
Additionally, the GigPosted" and "LastUpdated" columns of the Gig table are not getting real data.
I am unsure if these are the reasons why the added gig is not showing up on the gigs/index.php view. More research is needed.

fyi .... I've added gigs. I just added a couple, but I was not logged on.  They all seem to have stuck.  All fields are populated except GigPosted, and LastUpdated is all zeros.

IF nobody is working on this issue, I'll assign myself

I think a recent pull request may have resolved this issue. @esteban-gs are you able to add a gig and then see it on the view gigs page?

It is writing to the DB, but not showing up on the list:

For me it only shows up when I search for it:

I'll assign myself.

@esteban-gs .... You said above that Gigs are being added.  Did you confirm that the Foreign Key "CompanyContactID" is being set properly in the "Company" table?
If so, I think we should close this one.

Yes. It worked on my end
On Mon, Jun 10, 2019, 9:26 PM Thom Harrington<EMAIL_ADDRESS>wrote:

@esteban-gs https://github.com/esteban-gs .... You said above that Gigs
are being added. Did you confirm that the Foreign Key "CompanyContactID" is
being set properly in the "Company" table?
If so, I think we should close this one.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gig-central/gig-repo-1/issues/173?email_source=notifications&email_token=AGJXJ444OFJFGSCIWCLHCB3PZ4SOFA5CNFSM4HOZY26KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXL4YKQ#issuecomment-500681770,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGJXJ4YEHULWDRI6SZYRYTTPZ4SOFANCNFSM4HOZY26A
.

Closing per PR #233  and above discussion


Problem with ZSet expiring

Redis version - 6.2.4
Redisson version - 3.17.7
We encountered a problem when the service was rebooted, the ZSet keys in Redis stopped expiring. We are using the reactive implementation of RSetCache. I believe this is because data about the keys being examined is lost. But for example, for the Hash structure in Redis, the data is stored in redisson__timeout__set. Why isn't there something like this for ZCache? Could u help us to solve the problem?
  private RSetCacheReactive<HistoryMessage> getOrCreateCache(String key) {
        return client.reactive().getSetCache(HISTORY_MESSAGE_CACHE + key, codec);
    }

  public Mono<Boolean> add(@NonNull HistoryMessage message) {
        return getOrCreateCache(generateKey(message)).add(message, config.getTtl(), config.getUnit());
    }

@mrniko Hello!
In our case, we have many ZSet's which don't update or update rarely. In most of cases the data is read and waited for eviction. ttl = 14 DAYS.
I noticed that when new data is added to a set, outdated data is evicted, but in our case, sets may never be updated again, but wait until the end of their life. Because of this, our cache is overflowing.
Perhaps you can advise us on some solution to this case or suggest another data structure to solve this problem?

@immmus
You need to invoke getSetCache() per name at least once after restart. Since eviction is scheduled in this method call.

@immmus
You need to invoke getSetCache() per name at least once after restart. Since eviction is scheduled in this method call.

Okay, but what if there are millions of names?;)

There is SetCacheV2 structure which improves this, but it's available only in PRO version.

Okay, but what if there are millions of names?;)

There is SetCacheV2 structure which handles this case, but it's available only in PRO version.

I see this is a shame.  I think many people encounter these issues only when using your library, although you have a checkmark in the eviction column, although this does not work properly.
Thank you for answers.

Thank you for your feedback. I have add the description of entry eviction  and advanced entry eviction features.


Simplify neurite type calculation for section trees.

Each section can have only one type. Set the tree node types
from the section info. The type of a neurite is taken from the
type of its first tree node.

[Current coverage][cc-pull] is 100%

Merging [#332][cc-pull] into [master][cc-base-branch] will not change coverage

@@           master   #332   diff @@
====================================
  Files          43     43
  Lines        2027   2023     -4
  Methods         0      0
  Messages        0      0
  Branches        0      0
====================================
- Hits         2027   2023     -4
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last updated by [539e5b9...2f1c8f9][cc-compare]
[cc-base-branch]: https://codecov.io/gh/BlueBrain/NeuroM/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/BlueBrain/NeuroM/compare/539e5b99cd1476a47b8d4afe71579b301b6814f5...2f1c8f90a32f536de066a8f8e92b7505f4817c05
[cc-pull]: https://codecov.io/gh/BlueBrain/NeuroM/pull/332?src=pr


Simple MaxBoundedLen Implementations

This PR migrates some of the most important and simple pallets used in Statemint to use the MaxBoundedLen trait to generate StorageInfo.

Assets
Balances
ImOnline
Proxy
Sudo
Timestamp

polkadot companion: https://github.com/paritytech/polkadot/pull/3118

bot merge

I see this has E1-runtimemigration. Can you help me understand what that implies for downstream chains? For example, Moonbeam uses some of these pallets (Balances, Proxy, Sudo, and Timestamp). Do we need a storage migration? If so what would it be?

this is the potential migration needed I found:

in pallet asset: we now use a BoundedVec in the type AssetMetadata which is stored in Metadata storage. If a value in Metadata storage contains a value with a name or a symbol of length more than Config::StringLimit::get(), then the value must be migrated in some way (removed, or updated with a name and symbol of length less or equal to Config::StringLimit::get().
This can happen if the runtime has set a limit for StringLimit and then decreased the limit in another runtime upgrade, while keeping some value which were accepted by the previous limit in storage.

in proxy pallet: we now use a BoundedVec in the storage Proxies. if a value in Proxries contains more a vec of length more than Config::MaxProxies::get() then it must be migrated.
This can happen if the runtime has set a limit for MaxProxies and then decreased the limit in another runtime upgrade, while keeping some value which were accepted by the previous limit in storage.


Support gpu scheduling

This PR will:

Pass in the gpu resource request into the scheduling unit
ClusterResourceFit supports filter of gpu
MostAllocatable and LeastAllocatable supports gpu resources, and the default weight map will be cpu: memory: gpu=1:1:4
BalancedAllocatable supports comparison of GPU resources, based on standard deviation algorithm

/assign @gary-lgy @limhawjia @mrlihanbo
No rush to review if you are on vacation. :)


Endpoint populate new policymap early if empty

Sync endpoint bpf policy map before bof compilation, if the bpf policy
map is empty.
The bpf policy map is empty if it was (re)created for the endpoint. This
can happen in the following situations:

endpoint is new, so no policy map existed yet
policy map was upgraded to a new spec or name

In the first case it does not really matter when the policy map is
synced, but in the latter case the new endpoint programs would start with
an empty policy map, which could disrupt existing connections. In this
case it is better to populate the policy map before the bpf program
creation so that when the endpoint starts using the new program if can
use an up-to-date policy map rather than an empty one.
Normally we'd want to sync the policy map only after it is known that the
bpf compilation did not fail.
Rename the 1.17 endpoint policy map to make use of the above.

/test

/test


new metadata for 1.35.0

we need this first so we can add the new security context to the OLM metadata for 1.35: https://github.com/kiali/kiali/issues/3918


Created Merge Sort Algorithm

Can you PR the entire program?

Yeah sure! Done already :) @bhaveshlohana


Spartacus shortwave optimizations + avoiding double prec. in two-stream kernels

Hello,
Here are the performance optimizations I discussed. Most of them target the expm computations in spartacus_sw. In my tests, the solver was about 45% faster (single core on an Intel laptop, ifort -O3).
Also included in this pull request is a fix to the two-stream kernels which removes the need for any double precision variables/computations by instead adjusting the minimum "k" parameter based on working precision. This change should be tested some more - I only tested the McICA and Spartacus shortwave and longwave solvers and found that net upwelling and downwelling fluxes for the test IFS profiles changed by less than 0.1 W/m2. The benefit is increased performance for single precision computations; 20-25% faster mcica_sw and mcica_lw in my tests. If you want, I can also scrap this and make a pull request for only spartacus_sw.
2nd try, with the changes you asked for - but note that everything in radiation_two_stream is still jprb, as per my changes, so the kernel following #else in calc_reflectance_transmittance_sw is not the the true original with some computations in double precision.
By the way, it should be possible to squash the multiple commits

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: peterukk:x: rjhoganYou have signed the CLA already but the status is still pending? Let us recheck it.


Networks parsing problem

When in a network there's the dns_resolver_configuration sub element the
current network is not parsed correctly and the following networks will
not be correctly collected
Fixes: #207
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1841556
Also tracked on: https://issues.redhat.com/browse/OCPRHV-104

@imjoey @rgolangh could you please review when you'll have time?

@Gal-Zaidman please join the reviewers :)

/lgtm

/lgtm

@imjoey no problem and thanks for all

@imjoey should we push this to sdk_4.3 branch too?

@imjoey @sandrobonazzola pushed a PR #210 just in case you want to backport


[TUTORIAL] Fix README WikiText2 example

Description
This fixes nlp.data.WikiText2(segment='train')[0][0:5] to nlp.data.WikiText2(segment='train')[0:5] because nlp.data.WikiText2(segment='train')[0][0:5] is '='.
Checklist
Essentials

[x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[x] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage
[ ] Code is well-documented

Changes
Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@szha is the CI down?

@eric-haibin-lin a recent kernel image upgrade on the CI server caused nvidia drivers to fail. I reverted the kernel images just now so hopefully, things should be working now

Job PR-681/11 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-681/11/index.html

@crcrpar thank you for the fix!

My pleasure!


Identify required FACT API endpoints for Facilities

[x] Meet with @faqir-cvp to determine document template
[ ] Identify endpoints in FACT API that are required to display Facilities data in Monitoring Plan Viewer

A/C:
Single document of all identified endpoints uploaded to EPA Sharepoint

Met with Faqir and Jason to discuss how we want to format endpoints. Started on the documentation.

Identified endpoints needed for facilities data. Will meet internally to review before closing out this ticket.

Changed story point value from 3 to 5 because we included the work from #194 .

Identified Endpoints: https://usepa.sharepoint.com/:b:/r/sites/CAMDCVPTeam/Shared Documents/Datateers/EASEY-OUT/API Endpoints/Identified API Endpoints - Facilities - EPA CAMD - v1.1.pdf?csf=1&web=1&e=ZZIfOu

Next steps #226


Add life cycle information to response of plants API calls

Tasks

[ ] existing model PlantsSummaryDto (backend/src/model/dto.rs) has to be extended with a new list-field life_cycle (backend/src/model/enum/life_cycle.rs), so that the endpoint invocations of /api/plants/{id} and /api/plants also contain a plant's life cycle in the response

[ ] life_cycle is a list containing any combination of its 3 possible values (annual, biennial, perennial) or just an empty list in case a plant has no life cycle value in the DB

endpoint definition of plants endpoints can be found here: backend/src/controller/plants.rs

currently, the response when calling /api/plants/{id} (or /api/plants for a list of plants) looks like this:

{
  "id": 2107,
  "unique_name": "Abelia triflora",
  "common_name_en": [
    "indian abelia"
  ],
  "spread": "wide"
}

new response should look like this:

{
  "id": <number>,
  "unique_name": <unique_name>,
  "common_name_en": [
    <common_name_en>
  ],
  "spread": <spread>,
  --> "life_cycle": [] || ["annual","biennial","perennial"] <--
}

Use case
plants_layer
Related Pull request
#724

@horenso  Thx for creating this issue! Please update top post:

We should also include common_name_de, see  #433.
Spread will be an integer, see #822.
This is also needed for UC area of plants, which you will probably do as next UC (after copy&paste).

Did you look through all UC?
Is it a good idea to stuff everything to PlantsSummaryDto?
For #476 we will need nearly all of plant's data.

when creating dates can already be calculated


Rewrites of bsf.hcl, bsf.lock and nix files aren't deterministic

When we re-generate nix files or when we use an operation which generates or rewrites bsf.hcl- the ordering of elements in bsf.hcl file changes i.e<EMAIL_ADDRESS><EMAIL_ADDRESS>will become<EMAIL_ADDRESS><EMAIL_ADDRESS>This can be annoying when reviewing a diff.
With the upcoming bot, a user might notice a PR raised just because of this.
Root cause of the issue is because we're using maps and elements in maps aren't guaranteed to be ordered. One solution might be to reach deterministic output might be to rewrite packages back in lexicographically/alphabetically increasing order.

Hey @BalaadityaPatanjali  , that's great. Thanks for showing interest !
You can ping us in our discord server if you need help.


#414 Bot config file is not parsed

Fixes #414.

Do you understand when this was introduced?  This code path definitely used to work...
Also, your commit message looks unfinished?

@timabbott I updated commit's message. As it seems from git history, the bug was introduced after some refactoring. Previously, config file was read by ConfigParser.readfp method. After refactoring, used method was changed to ConfigParser.read method. But this method should be used with file's path passed as argument, not open file handle. Now if you try to run bot with custom config (e.g. beeminder), the bot's section would not be read from the config file. This PR fixes current issue.

Merged as 229f62a483501572884aeb8a39e8992f80758c34, after cleaning up the commit message.  Please write your commit messages with this sort of explanation and research done; it saves a ton of time in review, and can often save even more time when future developers try to understand why we did something.  Check out the Zulip commit message guidelines for more details on how you should be writing these: https://zulip.readthedocs.io/en/latest/contributing/version-control.html#commit-messages
Thanks @novokrest!


Add editorconfig.

Why:

EditorConfig helps developers define and
maintain consistent coding styles between different editors and IDEs.

This change addresses the need by:

Add basic .editorconfig file to repo.

:+1:

@squaresurf, do you think we should add links to the editor specific plugin links in the README? In fact, we should probably add a little blurb about each of the configs in this repo so people can choose whether or not they want to use it for their project.

@samcdavid that sounds like a great idea. Do you mind heading that up outside of the PR or do you want it done before this is merged?

I don't mind heading it up outside the PR :smile:

:+1:

Thank you @samcdavid :)


Readability of list options

Should the question "what efforts have you taken to find the defendant" be split up into different screens?

Made the list options shorter which fixes the readability of the options


rawurlencode function in Regex router

I doing an online store on ZF2.
I want to make URL as SEO-friendly and user friendly.
For example,
/catalog/sectionA/sectionB/
/catalog/sectionA/sectionB/sectionC/
/catalog/sectionA/sectionB/sectionC/sectionD/sectionN/

Nesting categories can be any and is generated dynamically.
My routes
'catalog_product' => [
    'type' => 'regex',
    'options' => [
        'regex' => '/catalog/(?<section>[a-zA-Z0-9_-]+)/(?<product>[a-zA-Z0-9_-]+).html$',
        'defaults' => [
            'controller' => 'AlbrinCatalog\Controller\Catalog',
            'action' => 'view'
        ],
        'spec' => '/catalog/%section%/%product%.html'
    ]
],

In view
<a href="{{ url('catalog_product', {'section': product.section_full_path, 'product': product.code}) }}">
I get
/catalog/sectionA%2FsectionB/
instead of
/catalog/sectionA/sectionB/
Because in zend-mvc\src\Router\Http\Regex rawurlencode($value)
    public function assemble(array $params = [], array $options = [])
    {
        $url                   = $this->spec;
        $mergedParams          = array_merge($this->defaults, $params);
        $this->assembledParams = [];

        foreach ($mergedParams as $key => $value) {
            $spec = '%' . $key . '%';

            if (strpos($url, $spec) !== false) {
                $url = str_replace($spec, rawurlencode($value), $url);

                $this->assembledParams[] = $key;
            }
        }

        return $url;
    }

What can I do?

Does anyone think that something about this?
I do not know what to do in this situation, use Regex router and url() plugin.
I really need help with this.

As I understand it for the solution of this issue will need to make changes to the code of the component?

I think the simplest solution will be to write own class - own Route type, and use it in routing.
You just need implement interface RouteInterface. Always you can see how Segment and/or Regex are implemented and it shouldn't be a problem to write exactly what you need.
Please take a look also at documentation: HTTP Route Types.


Add UI for USD relationships

Relationships should be exposed in each prim's attributes widget instead of hidden away in Raw USD Properties.
See https://github.com/CesiumGS/cesium-omniverse/commit/d186b5f464b23c88c4deb7d89cef0e229c5cfe12 for a reference implementation.

[ ] CesiumDataPrim

[ ] selectedIonServer

[ ] CesiumTilesetPrim

[ ] georeferenceBinding
[x] ionServerBinding (https://github.com/CesiumGS/cesium-omniverse/commit/d186b5f464b23c88c4deb7d89cef0e229c5cfe12)
[ ] https://github.com/CesiumGS/cesium-omniverse/issues/585

[ ] CesiumIonImageryPrim

[ ] ionServerBinding

[ ] CesiumGlobeAnchorSchemaAPI

[ ] georeferenceBinding

Fixed in #204


[Feature] Add custom background music

Add the option to put custom background music in the menu.

#116


headlamp-plugin: Make the package arg of the build command optional

We rely on just calling the "headlamp-plugin build" command without
any arguments, but the build command expected a positional argument as
required, so the default and advised case for building plugins was
failing.

interestingly, the CI found a bug when using npm 7. Where npm 7 gets into a loop resolving dependencies, and keeps processing until it runs out of memory. See https://github.com/npm/cli/issues/3359 and https://github.com/npm/cli/issues/3208
There's a work around for running it on CI with npm 7, but users of npm 7 will encounter this bug.
I tried updating the plugin dependencies, and still get the issue. There's another work around or two to try(which I will do next)... but ultimately the bug in npm 7 needs fixing (or maybe it's a bug in one of the dependencies).

Adding a package-lock.json generated with npm v6 to the template lets npm v7 work.

Probably we should release a new @kinvolk/headlamp-plugin version.

@joaquimrocha yeah. It's not directly calling headlamp-plugin build... but it does so through npm run build which calls that. I'll update the test to call headlamp-plugin build with no argument directly.

The headlamp-plugin build is tested better now, and also added a test for headlamp-plugin extract.


fix: fix memory leak when query error

Ref: TD-21429

Codecov Report

Merging #113 (c13b692) into main (bee812c) will decrease coverage by 0.00%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main     #113      +/-   ##
==========================================
- Coverage   63.08%   63.08%   -0.01%
==========================================
  Files          96       96
  Lines       16917    16918       +1
==========================================
  Hits        10672    10672
- Misses       6245     6246       +1

Impacted Files
Coverage Δ

taos-sys/src/query/raw_res.rs
54.91% <50.00%> (-0.25%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Introduce Dev Container support (add documents)

Add documents for Dev Container.
This is additional changes of #253 .

Fix versions in Python and Ruby Dev Container Features.

Ruby 3.2
Python 3.11


[1629] Fix up course length

Context
Course length wasn't being posted to the backend properly so it the data wasn't being saved
Changes proposed in this pull request
Re-factor form to handle 1year, 2years and "Other"
Guidance to review
No visual change.
Note: We need to address the backend to save the "other" value correctly.

 [M] Updating course length doesnt work


GitHub repository topic remove button is offset

Is there an existing issue outlining your problem?

[X] I am using the latest version of the provided userstyle.
[X] I have searched the existing issues and they do not solve my problem.

What userstyles are you seeing the problem on?
lbl:github
Describe your problem.
When editing topics for a repo, the X button to remove a topic is offset into the corner of the highlighted area.
Attach screenshots.

What browser(s) are you seeing the problem on?
Firefox 123.0.1
Any additional comments?
No response

This is an issue with GitHub itself, not the userstyle:


fix the responsiveness of the index.php

INCLUDE ALL NECESSARY FIELDS AND DELETE FIELDS THAT ARE NOT NECESSARY.
Description
There was a total mess in responsiveness in mobile view previously it was looking like this

now my code is looking like this
IN Desktop View

In Phone View

https://user-images.githubusercontent.com/55352601/111899206-112bcf80-8a51-11eb-88f8-562bc87e8901.mp4
Fixes #40
Type of Change:
Delete irrelevant options.

Code
User Interface

Code/Quality Assurance Only

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Describe the tests you ran to verify your changes. Provide instructions or GIFs so we can reproduce. List any relevant details for your test.
I had attached a sample video.
I hope you like it and it was enough to merge it.

@sakshi300699 mam Is there any Problem in this PR. If not Please make sure to add the gssoc'21 label before merging it.

@sakshi300699 mam please merge this PR

@arpit456jain the PA would be merging this as soon as the mentors and PA approve of this.

@sakshi300699 mam I hope It doesn't create any problem

Nope @arpit456jain don't worry, that's not an issue

@sakshi300699  mam I have no idea why this is showing conflicts my code is working fine.
what should I do?

@arpit456jain your branch is not up-to-date with the master branch, you need to pull all the changes to this branch.

ok mam let me try

@sakshi300699  mam I had pull the latest code and done the required changes again so I am creating a another PR with that...
Please add the labels of level2 and gssoc21 sorry for the inconvenience but please this time merge it as soon as possible I request
Again My code is working fine and fully responsive no problem occur
I had just removed the inline CSS with Internal Css and add some media query that's it.
Is that ok?
Can i create another PR. I will close this

Okay @arpit456jain Please close this PR.

ok mam


Add OpenSearch Ingestion Pipeline Support

Description of your changes

Fixes #1446
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable to ensure this PR is ready for review.
[x] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

I took the example and modified line 26 to change it to my own AWS Account ID. I was able to validate the command:
make e2e UPTEST_EXAMPLE_LIST=examples/osis/v1beta1/pipeline.yaml
---
apiVersion: osis.aws.upbound.io/v1beta1
kind: Pipeline
metadata:
  annotations:
    crossplane.io/external-create-pending: "2024-09-12T16:23:43Z"
    crossplane.io/external-create-succeeded: "2024-09-12T16:23:43Z"
    crossplane.io/external-name: example
    crossplane.io/paused: "false"
    meta.upbound.io/example-id: osis/v1beta1/pipeline
    upjet.upbound.io/test: "true"
    uptest-old-id: example
  creationTimestamp: "2024-09-12T16:23:42Z"
  deletionGracePeriodSeconds: 0
  deletionTimestamp: "2024-09-12T16:29:49Z"
  finalizers:
  - finalizer.managedresource.crossplane.io
  generation: 3
  labels:
    testing.upbound.io/example-name: example
  name: example
  resourceVersion: "2581"
  uid: 351b183c-0dac-430e-ba5f-90aec59e402e
spec:
  deletionPolicy: Delete
  forProvider:
    maxUnits: 1
    minUnits: 1
    pipelineConfigurationBody: |
      version: "2"
      example-pipeline:
          source:
              http:
                  path: "/example"
          sink:
          - s3:
              aws:
                  sts_role_arn: "arn:aws:iam::ACCOUNT_ID:role/example-osi-pipeline-role"
                  region: "us-west-1"
              bucket: "example-upbound-osis-pipeline-example-blake"
              threshold:
                  event_collect_timeout: "60s"
              codec:
                  ndjson:
    pipelineName: example
    region: us-west-1
    tags:
      crossplane-kind: pipeline.osis.aws.upbound.io
      crossplane-name: example
      crossplane-providerconfig: default
  initProvider: {}
  managementPolicies:
  - '*'
  providerConfigRef:
    name: default
status:
  atProvider:
    id: example
    ingestEndpointUrls:
    - example-7myxz74qalrcrk5ilmygdzco2u.us-west-1.osis.amazonaws.com
    maxUnits: 1
    minUnits: 1
    pipelineArn: arn:aws:osis:us-west-1:ACCOUNT_ID:pipeline/example
    pipelineConfigurationBody: |
      version: "2"
      example-pipeline:
          source:
              http:
                  path: "/example"
          sink:
          - s3:
              aws:
                  sts_role_arn: "arn:aws:iam::ACCOUNT_ID:role/example-osi-pipeline-role"
                  region: "us-west-1"
              bucket: "example-upbound-osis-pipeline-example-blake"
              threshold:
                  event_collect_timeout: "60s"
              codec:
                  ndjson:
    pipelineName: example
    tags:
      crossplane-kind: pipeline.osis.aws.upbound.io
      crossplane-name: example
      crossplane-providerconfig: default
    tagsAll:
      crossplane-kind: pipeline.osis.aws.upbound.io
      crossplane-name: example
      crossplane-providerconfig: default
  conditions:
  - lastTransitionTime: "2024-09-12T16:29:42Z"
    reason: ReconcileSuccess
    status: "True"
    type: Synced
  - lastTransitionTime: "2024-09-12T16:29:42Z"
    reason: Available
    status: "True"
    type: Ready
  - lastTransitionTime: "2024-09-12T16:29:42Z"
    reason: UpToDate
    status: "True"
    type: Test
  - lastTransitionTime: "2024-09-12T16:29:42Z"
    reason: Success
    status: "True"
    type: LastAsyncOperation

/test-examples="examples/osis/v1beta1/pipeline.yaml"

So looking at logs https://github.com/crossplane-contrib/provider-upjet-aws/actions/runs/10835115001/job/30065908399 it seems like it actually does get ready but maybe I am missing something in terms of it not working or maybe it is just timing out too soon?

First, I'm not an expert on Uptest logs 🙂 Second, the following line tells me that the Pipeline resource is not ready:
2024-09-12T17:13:53.1182356Z     case.go:363: command "${KUBECTL} wait pipeline.osis.aws.upbound.io/example --for=condition=Test --timeout 10s" exceeded 7 sec timeout, context deadline exceeded

When I check the resource output by grepping osis.aws.upbound.io in the Uptest log file I downloaded, I see that the resource is not ready:
- apiVersion: osis.aws.upbound.io/v1beta1
  kind: Pipeline
  metadata:
    annotations:
      crossplane.io/external-create-pending: "2024-09-12T17:13:44Z"
      crossplane.io/external-create-succeeded: "2024-09-12T17:13:44Z"
      crossplane.io/external-name: example
      meta.upbound.io/example-id: osis/v1beta1/pipeline
      upjet.upbound.io/test: "true"
    creationTimestamp: "2024-09-12T16:53:53Z"
    finalizers:
    - finalizer.managedresource.crossplane.io
    generation: 2
    labels:
      testing.upbound.io/example-name: example
    name: example
    resourceVersion: "4358"
    uid: 8b15e491-dd01-4096-9084-b373f7027be0
  spec:
    deletionPolicy: Delete
    forProvider:
      maxUnits: 1
      minUnits: 1
      pipelineConfigurationBody: |
        version: "2"
        example-pipeline:
            source:
                http:
                    path: "/example"
            sink:
            - s3:
                aws:
                    sts_role_arn: "arn:aws:iam::609897127049:role/example-osi-pipeline-role"
                    region: "us-west-1"
                bucket: "example-upbound-osis-pipeline-example"
                threshold:
                    event_collect_timeout: "60s"
                codec:
                    ndjson:
      pipelineName: example
      region: us-west-1
      tags:
        crossplane-kind: pipeline.osis.aws.upbound.io
        crossplane-name: example
        crossplane-providerconfig: default
    initProvider: {}
    managementPolicies:
    - '*'
    providerConfigRef:
      name: default
  status:
    atProvider: {}
    conditions:
    - lastTransitionTime: "2024-09-12T16:53:54Z"
      reason: Creating
      status: "False"
      type: Ready
    - lastTransitionTime: "2024-09-12T16:53:54Z"
      reason: ReconcileSuccess
      status: "True"
      type: Synced
    - lastTransitionTime: "2024-09-12T16:53:55Z"
      reason: Success
      status: "True"
      type: LastAsyncOperation

@mergenci sorry missed that 🤦♂ what I am still confused by is why it is still in a creating state. I know OSI pipelines do take a long time to come up but I am able to get this same uptest command work locally on my AWS environment by just swapping the AWS Account ID in the pipeline body. Like I said I wonder if there we are just timing out on the test and it isn't an actual failure.

Please rebase the PR and trigger again to get this fix, so that we can see if there is an error.

/test-examples="examples/osis/v1beta1/pipeline.yaml"

It looks like it is failing with this error:
AccessDeniedException: User: arn:aws:iam::153891904029:user/official-provider-testing is not authorized to perform: osis:CreatePipeline on resource: arn:aws:osis:us-west-1:153891904029:pipeline/example

So we may need to tweak the permissioning on the user to allow the test to pass.

/test-examples="examples/osis/v1beta1/pipeline.yaml"

@blakeromano I've added the following policies to the test user:

AmazonOpenSearchIngestionFullAccess
AmazonOpenSearchServiceFullAccess

@jeanduplessis I still see same error on Uptest. Is there an SCP or Tag Policy or anything that is giving it the explicit deny? I just can't see anything from my POV on why this would be failing...

/test-examples="examples/osis/v1beta1/pipeline.yaml"

/test-examples="examples/osis/v1beta1/pipeline.yaml"

Because of this issue we could not convert the singleton list to embedded object in the osis Pipeline resource. This issue is not currently blocking and will be fixed with new versions of the resource.


[WIP]: k-14: select and use kaos context from a list of contexts

Description
This feature allows kaos to run on multiple contexts (deployments).
Also allows user to choose a specific context from a list of available contexts.
Checklist

[ ] An issue was first created before opening this pull request
[ ] The new code follows the kaos contribution guidelines
[ ] 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
[ ] My changes generate no new warnings

Closing PR, due to the introduction of command history.
Handled in PR


Syntax Error ZCL_EXCEL_WRITER_2007=========CP

I have installed a fresh new SAP NW 750 with the latest support pack 22.
I installed Microsoft Office 2016 on the application server too (I'm not sure it's necessary).
Then I pull the ABAP2XSLX package using ABAPGIT.
Now, when I'm trying to execute several demo programs.
Unfortunately, all fails with the same error:
Category               ABAP programming error
Runtime Errors         SYNTAX_ERROR
ABAP Program           ZCL_EXCEL_WRITER_2007=========CP

      CREATE OBJECT cl_writer TYPE zcl_excel_writer_2007.

When I'm checking the ZCL_EXCEL_WRITER_2007 class, there are several errors:
Method CREATE_XL_DRAWINGS_HDFT_RELS
Error in arithmetic expression: A field, a function, or "(" was expected instead of "<FS_DRAWINGS>-DRAWING->GET_INDEX(".
Method CREATE_XL_DRAWINGS_HDFT_RELS
Method "GET_MEDIA_NAME" is unknown or PROTECTED or PRIVATE.
Method SET_VML_SHAPE_FOOTER
Method "GET_POSITION" is unknown or PROTECTED or PRIVATE.
Method SET_VML_SHAPE_FOOTER
Method "GET_INDEX" is unknown or PROTECTED or PRIVATE.
Method SET_VML_SHAPE_FOOTER
Method "GET_POSITION" is unknown or PROTECTED or PRIVATE.
Method SET_VML_SHAPE_FOOTER
Method "GET_INDEX" is unknown or PROTECTED or PRIVATE.
Method SET_VML_SHAPE_FOOTER
Method "GET_POSITION" is unknown or PROTECTED or PRIVATE.
Method SET_VML_SHAPE_FOOTER
Method "GET_INDEX" is unknown or PROTECTED or PRIVATE.
Method SET_VML_SHAPE_HEADER
Method "GET_POSITION" is unknown or PROTECTED or PRIVATE.
Method SET_VML_SHAPE_HEADER
Method "GET_INDEX" is unknown or PROTECTED or PRIVATE.
Method SET_VML_SHAPE_HEADER
Method "GET_POSITION" is unknown or PROTECTED or PRIVATE.
Method SET_VML_SHAPE_HEADER
Method "GET_INDEX" is unknown or PROTECTED or PRIVATE.
Method SET_VML_SHAPE_HEADER
Method "GET_POSITION" is unknown or PROTECTED or PRIVATE.
Method SET_VML_SHAPE_HEADER
Method "GET_INDEX" is unknown or PROTECTED or PRIVATE.
Any Idea?

@sandraros could it be an incomplete pull?
@mikibarzilay does the source of zcl_excel_writer_2007 look anything like this?

I tried to pull it online several times and also using the zip file as
offline repository with no luck.
Micky
On Mon, 14 Feb 2022, 22:16 sandraros, @.***> wrote:

It's the same issue as #986
https://github.com/abap2xlsx/abap2xlsx/issues/986. The solution is to
pull again, check the objects activated, etc. That's something related to
abapGit, not sure what the exact error is. Maybe @larshp
https://github.com/larshp could help us.
—
Reply to this email directly, view it on GitHub
https://github.com/abap2xlsx/abap2xlsx/issues/993#issuecomment-1039513256,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACROJ6FBMNSDERPU4VMPIWDU3FPLLANCNFSM5OHEKXSA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Did you check the list of "Inactive Objects" in SE80, and activate all of them manually?

I really appreciate your support.
Checking for Inactive Object in SE80, there is no inactive objects.

@mikibarzilay did you get any errors during pull?

When I clicked "pull" for the first time I received an error, then I clicked again and it complete without any error.
However, I found that the  ZEXCEL_S_DRAWINGS method wasn't activated, I activate it manually. This didn't solve the issue yet.
One more interesting issue, I CREATE_XL_DRAWINGS_HDFT_RELS method when I'm double-clicking on ger_index( ) in line 49 (where the error points) I'm getting the message "Class (Definition) ZCL_EXCEL_WRITER_2007 does not exist" although in SE24 it exist.

You may use abapGit menu Advanced > Syntax Check to see all objects with syntax error. Try to activate inactive DDIC objects first.
Be careful with objects which seem active, I think abapGit forces the activation, while they may still contain syntax errors.

Hi again,
In abapGIT I clicked on several items and one after one I can confirm they are "partly activated".

At this stage, I think this is really an issue with abapGit and not with abap2xlsx, unfortunately.
You should try filing an issue there.

Thank you Abo,
Is there any way to received a Change Request transport  (data and cofile files) instead of using abapGIT?

@mikibarzilay with an abapgit project, this is neither supplied nor recommended. The two installations I encountered where somebody had already tried that approach were hell to clean up: some objects had different transport layers, some were local, a mess you don't want to have, believe me.

This project contains XSLT and if those objects don't activate properly, then nothing else will work (due to Ddic and code dependencies).
Therefore, please check SAP note https://launchpad.support.sap.com/#/notes/2859159 first and make sure you have the right kernel version running.
Secondly, run the very latest version since we continuously improve things like #5303 which shows inactive TABL objects correctly (note: similar handling of other DDIC types is still missing).
To have a reproducible case, uninstall the complete repo and install it again. Attach output of Utilities > Debug info and screenshot after initial pull.

Thanks Marc. I just created #994. I'll try to work on the abap2xlsx fix this evening (not guaranteed as I'm quite busy currently).

For information, I think all these PRs in abap2xlsx and abapGit could help solving - Please wait till they are merged:

#995
https://github.com/abapGit/abapGit/pull/5369
https://github.com/abapGit/abapGit/pull/5370
https://github.com/abapGit/abapGit/pull/5371

I had the same problem and it happens in our release because the dynamic TYPE REF TO OBJECT in the structures like ZEXCEL_S_WORKSHEET_HEAD_FOOT doesn't allow the syntax check in calling methods like get_index( ).
I simply adapted the coding and cast it to type-specific reference variables:
*      lv_value = <fs_drawings>-drawing->get_index( ).   OLD       data lr_drawing type ref to ZCL_EXCEL_DRAWING.                  " Gerschler - Workaround wegen TYPE REF OBJECT-Fehler       lr_drawing ?= <fs_drawings>-drawing.                            " Gerschler - Workaround wegen TYPE REF OBJECT-Fehler       lv_value = lr_drawing->get_index( ).                            " Gerschler - Workaround wegen TYPE REF OBJECT-Fehler
I don't know if it's the best solution, but it works as a fix workaround until a new version is available.

After we moved the Demo programs to a separate repository (#998), the issue doesn't happen anymore.
Could you try? If okay, I propose to close the issue.

Using my AS ABAP 7.52SP04 installation, which was known to suffer from the activation issue due to the missing note, I removed all traces of abap2xlsx and dependent programs, then installed first the base package and then the demos. Activation was completed without errors.

I tested main and demo repos with v1.115.0 on 7.54. Both can be pulled without errors (just some warnings due to reserved field names)
Suggest closing

I agree. Let's close it then. Thanks!


fix exception on user update when password is not set

backend_1       |   File "./app/crud/crud_user.py", line 34, in update
backend_1       |     if update_data["password"]:
backend_1       | KeyError: 'password'

Thanks @Haikson! This was recently solved in another PR, so I'll close this one, but thanks for the effort! :coffee:


[ENV] create devcontainer settings

Hello, I am a code review bot on flows.network.
It could take a few minutes for me to analyze this PR. Relax, grab a cup of coffee and check back later. Thanks!

@L-jasmine Thanks for the review!


Implement first migration for S7 adapter

Body
Implement the first migration based on #2001 that encompasses the changes introduced in #1980.
StreamPipes Committer
I acknowledge that I am a maintainer/committer of the Apache StreamPipes project.

Closed with #2121


xfitter: fix for darwin

Fixes: 8984fc29 ('xfitter: fix build w/glibc-2.32')

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Result of nixpkgs-review pr 121411 at c846fa16 run on x86_64-linux 1

1 package built successfully:

xfitter

2 suggestions:

warning: missing-patch-comment
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/applications/science/physics/xfitter/default.nix:16:5:
   |
16 |     ./undefined_behavior.patch
   |     ^

warning: unclear-gpl
gpl3 is a deprecated license, please check if project uses gpl3Plus or gpl3Only and change meta.license accordingly.
Near pkgs/applications/science/physics/xfitter/default.nix:58:5:
   |
58 |     license     = licenses.gpl3;
   |     ^


An option for ToString and Latexise to output decimals and mixed fractions in addition to fractions

In line with most calculators that have a decimal<->fraction mode.

You can cast to real and take its EDecimal? Although we can add EvalNumerical<T>

What about formattng 1/2*x+1/25*y?

Hm... I don't know if we ever need it... but let's keep it as not now


Use mocha-jsdom?

I found this package: https://www.npmjs.com/package/mocha-jsdom
...after reading your blog post. It was as simple to set up as:
var jsdom = require('mocha-jsdom');

describe('...', function () {
  jsdom();
  it('...', function () {
    // normal React TestUtils.renderIntoDocument, etc. goes here
  });
});

The function you suggest gave me errors I did not understand during the require of jsdom proper. This did not. Very straightforward.
For your consideration. 😎

Cool! Want to send me a PR to update the examples?

Created PR #8.

Closed via #8.


K8SPSMDB-713 Rearrange backups doc and add physical backups

https://percona-mongodb-operator-pr-37.onrender.com/backups.md is broken and it's accessible from welcome page


Usage issue and question

First, this is working great, thanks!  2 questions.
q1: would it be possible to register a YubiKey without having to enter the YubiKey pin each time to use it?
q2: out of curiosity, I tried the option to "verify with your phone".  No idea how that works.  However, it sent a notification to my Android, I click it, then it says "registration failed".  I tried this on multiple websites.
I am using Brave.

q1: would it be possible to register a YubiKey without having to enter the YubiKey pin each time to use it?

Yes. Please go to Settings > TwoFactor WebAuthn (/wp-admin/options-general.php?page=2fa-webauthn) and change "User Verification Requirement" to "Discouraged".

q2: out of curiosity, I tried the option to "verify with your phone".

Unfortunately, I have never seen this option. Android only offers me to use the screen lock/fingerprint, and this works:
https://user-images.githubusercontent.com/7810770/151621589-aa5de9e5-38e7-4c41-b3af-7ddaa579d67c.mp4
But I have no idea how to trigger the "verify with your phone" thing. It probably should not work, because the browser's authentication API does not know how to get the authentication status from your phone; it can only work with devices physically attached to your computer.

Thanks, your solution to q1 works great.
As for the phone, I examined this more closely and it is interesting.  It does send the push notification to my phone.  And my phone tries to handshake with my computer over Bluetooth.  I see all that happen.
But then it says registration failed with no further information.  That's ok.


feat: add slide transitions

Proposal to add slide transition.
5 built-in transitions for now:

slide-left
slide-right
slide-up
slide-down
fade

You can set the transition for all your slides by using the frontmatter transition property in the first slide:
---
transition: slide-left
---

You can override the transition for a specific slide by again using the frontmatter transition property.
By setting the transition for a specific slide, this will determine the transition between this slide and the next one. (using a transition for the last slide will determine the transition between that slide and the "end" slide)
For built-in transitions the system will automatically choose the appropriate transition when going back.
If you set slide-left for the third slide it will:

use slide-left when going from 3 to 4
use slide-right when going from 4 to 3

You can write your own transition because underneath it uses TransitionGroup.
Thus the custom name you set in the transition property will be the name used for TransitionGroup.
In that case you might want to define the backward transition.
You can do it by using this format:
---
transition:
  forward: slide-and-fade-left
  backward: slide-and-fade-right
---

I set it as a draft to discuss it but the current implementation is already working

Awesome work! Sorry I missed your PR and happened to work on the same thing together with @posva. Your approach is interesting and I think it's better than my current one in many aspects. I will try to see how we can merge the two ideas together.

It's working perfectly! Thanks a lot for this!

It's working perfectly! Thanks a lot for this!

glad I could help :)


Build fails with Carthage

I'm unable to use Carthage to add Ono to my project. I'm using Carthage 0.11.0.
The following build commands failed:
    CompileC /Users/daniel/Library/Developer/Xcode/DerivedData/Ono-algafnmzvwnwysbsctdxjjediapq/Build/Intermediates/Ono.build/Release-iphoneos/Ono\ iOS\ Tests.build/Objects-normal/armv7/ONOHTMLTests.o Tests/ONOHTMLTests.m normal armv7 objective-c com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)

As discussed in #58, this should be fixed by 8939f2dca38. Please let me know if you're still having issues.


Add NFD r0 image reader.

This format is very similar to D88, but used mostly for PC-98
rather than PC-88.

So many weird image formats. Merged --- thanks!


Feature Request: Completions when editing values based on Shell Completions

Click Options have a shell_complete kwarg that takes a function that returns a list of completions.  It would be helpful if trogon supported running this function by passing in the context and param based similar to how click does so that a dropdown of completions could be shown to pick from.
See Also: Click documentation on how Shell Completions are implemented

I could see this being handled via the existing choices logic within Trogon. But that would make click choices and shell_complete mutually-exclusive in the context of Trogon; i.e., what would Trogon display in the drop-down if both choices and shell_complete were provided?


Use HTTPS for upload

Use HTTPS for image upload, fix #38 .

Coverage remained the same at 86.822% when pulling fe6a16c6fa07617ff9af36987a5ef046a86b7099 on kuon:https into 2245f1489572046d3d4ef15a264c359c56f086de on smeevil:master.

Thanks ! published 1.1.2


Favicon

Need a consistently referenced favicon

In Internet Explorer the favicon is bigger than in Chrome and Firefox causing the text to not be on the same line

@MalcolmShazell @marklunn Ping to QA and close if appropriate please

Actually, I think I need a clarification on this. The favicon is the small icon that shows up on the tab bar in desktop browsers, or can be used as a launch icon on the home screen in iOS and Android. It sounds like the Internet Explorer issue is about the home icon on the main page layout, which is a different issue.

@ijdickinson Yes you are correct there, shall I raise a separate issue for the home icon on IE?

@marklunn Yes please, and then I think we can close this one.

Happy to close - new issue raised for IE home icon


awesome-atom to the list of editors

Not sure if it has enough content for now.
I will contribute to this list regularly if the owner of the list allows it.

Sounds good. Can always just fork and link to your fork if the owner is unresponsive.

Would be nice if that list linked back here. Maybe you can do a PR for that repo.

I submitted one Pull Request to the owner of the repo as a test a few hours ago.
I am planning on linking back to this repository in my future Pull Requests after I see how this Pull Request goes.

I just wanted to inform you that the repository in question now links back to this repository. :+1:


Geographic stops not giving the right sens /hours

Hello
(following my post on hacf forum)
I have an issue with next departure around a localisation.
When I'm creating around my place.
I have 4 stops created

two corresponding to the location close to my place with the 2 sens (HLO001 and HLO002)
one for the stop in sens 2 for the stop before my place (FAY002)
on for the stop in sens 1 before my place (PIM001)

Stops HLO001 is my concern, because not giving the hours of bus on the good place
HLO001 & HLO002 corresponding to the 2 sens of the stop id 1148  which is Hautes loges (in Stops.txt)
This is for Ilevia network in Lille

url :  (https://opendata.lillemetropole.fr/api/v2/catalog/datasets/transport_arret_transpole-point/alternative_exports/gtfszip)
location used (50.6685652680685, 3.087443162094925)

HLO002 entity is having the GOOD hours and sens
`Gtfs updated at
12 mars 2024 à 18:55:02
Device tracker ID
zone.home
Next departures lines

departure: '18:58:00'
stop_name: Hautes Loges
route: '50'
route_long: MARCQ COLLÈGE <> GRAND SAINT ANDRÉ
headsign: Marcq En Baroeul Ferme Aux Oies
trip_id: '4954142'
icon: mdi:bus
departure: '19:09:00'
stop_name: Hautes Loges
route: L5
route_long: MARCQ / LA MADELEINE<>HAUBOURDIN LE PARC
headsign: Marcq En Baroeul Ferme Aux Oies
trip_id: '4960832'
icon: mdi:bus
`

When HLO001 is having the hours of FAY002
`Gtfs updated at
12 mars 2024 à 18:49:02
Device tracker ID
zone.home
Next departures lines

departure: '19:01:00'
stop_name: La Fayette
route: '50'
route_long: MARCQ COLLÈGE <> GRAND SAINT ANDRÉ
headsign: Saint Andre Grand St Andre
trip_id: '4954251'
icon: mdi:bus
departure: '19:23:00'
stop_name: La Fayette
route: '50'
route_long: MARCQ COLLÈGE <> GRAND SAINT ANDRÉ
headsign: Saint Andre Grand St Andre
trip_id: '4954143'
icon: mdi:bus
departure: '19:25:00'
stop_name: La Fayette`

Release used
latests released
Additional
Remark : even by changeing the reference location, I'm not able to create a FAY001 stop ... don't undertand why ...
even if the location is in Stops.txt file
I check GPS coordinate of FAY001 is Ok in stops.txt file (50.67153,3.089428)
When using the route to define a route starting HLO001 to the final stop of th line I have good information.
It's really when using geographique creation of the stops entity

I will have a look but analysis is a lot of work and usually very disappointing ...in 90%+ of the cases it is data related which is not where I want to spend my time with

After analysing my own stops...noticed they were wrong too.
I have a new release, 0.4.0, please check and see if this resolves yours (it does for me)

YES Bravo
It worked for me too.
That's now giving the good stop name and hours for geaographic point.


How to change the format of the date in the input box ?

Please advice. I've tried with ng-value and value inside the  tag and failed. Only way for now is to fetch the value formatted from the backend.

See issue #93

Does it mean that it won't be fixed or that any of the solutions suggested on this issue page works ?

It won't be fixed - this directive has no display functionality so formatting for display is not part of this directives functionality.
If you want to format for display, add that functionality to the input control or binding that is displaying the date - the referenced defect lists some options for controlling the formatting in a text box.


Document RACK_PRIVATE

Release Playbook

[ ] Rebase against master
[ ] Code review
[ ] Merge into master
[ ] Verify changes at http://site-staging.convox.com
[ ] Promote release on site-production in the convox/production Rack

 Document RACK_PRIVATE


Close out smaller issues

add link to figma.com/@liferay (#770 and #754)
update active style on main nav items (fix #555, possibly #763? @cj213x please review)

opening this PR to update the job listing as well (cc @ryconn888)


New fields might not be linked correctly

Originally posted by @Dimitar5555 in https://github.com/openstreetmap/id-tagging-schema/pull/453#issuecomment-1137692260
It looks like the new fields are not linked correctly. None of them show up in the iD instance on openstreetmap.org nor in version that is published on https://ideditor.org/latest-id/. These are the messages that appear in the console and they are mostly identical for all types of crossings.

It looks like our CDN still had the old version of the fields data in cache. I've manually refreshed it now. Clearing your browser cache and reloading iD should make the fields available for you.
See https://github.com/openstreetmap/iD/issues/8909 for a fix which would prevent such issues in the future.


Qdrant server restart

Hi - I'm using the GRPC service from Python, the server on the free account restarts after inserting around 100k points (index vector size is around 1400). I'm multithreading the request, using the upsert method in the qdrant_client.
This is the response I get:
        status = StatusCode.UNAVAILABLE
        details = "Received http2 header with status: 502"
        debug_error_string = "UNKNOWN:Error received from peer ipv4:(ip address):6334 {created_time:"2023-04-19T15:33:54.542513081+00:00", grpc_status:14, grpc_message:"Received http2 header with status: 502"```

Thanks.

I suspect this is because I'm going over the RAM limit for the instance - I'll check the options for saving to disk.


refactor random-related stuff out of e_os.h

I think only the crypto/rand files need the random stuff, so you can move 45 lines out of e_os.h to a separate file, or perhaps crypto/rand/rand_local.h ?

Good idea! See #10050.


Fix directories so that the conda command-line utils have right path

[ ] Are the conda utils / is the anaconda environment in the runtime?
[ ] Can we conda install or pip install something?

Notes:
conda command is, on the runtime, at /tmp/condaruntime/bin/conda, but thinks it is at /tmp/conda/condaruntime/bin/conda
The best way to fix this is to install the runtime during the setup process  into /tmp/condaruntime


[fix][#4572] clear master on cloning page

Coverage remained the same at 0.0% when pulling 65bafc2a74e9328e4e915deaf75d0c9c595f71c5 on fix-ss-4572 into aadc461df0a8f1eaa1da6a4cd9df48bc284524fb on master.


Fixes #6429 Reorganize scripts/lint folder

Which problem is this PR solving?

Fix #6429

Description of the changes

In this PR all scripts of lint category moved to lint/ folder.

How was this change tested?

Attaching screenshot below :

Checklist

[X] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
[X] I have signed all commits
[ ] I have added unit tests for the new functionality ~ not needed
[ ] I have run lint and test steps successfully ~ not needed

for jaeger: make lint test
for jaeger-ui: npm run lint and npm run test

closing until at least one PR is done right


Drop right ability

Adds ability to do a settings/shopping cart/inbox scenario, by allowing the menu to be rendered from the right instead of the default of from the left. Also adds the class to apply to the button, allowing a glyphicon, for example, to be added to the button.

Hi Chad,
I've included an improved version of the right alignment of the dropdown menu in the new release and master repository. I've included a css fallback instead of js solution for Bootstrap < 3.1 versions.
The button classes idea I have to reject at this point. I've tested it but seen the fact that adding the classes 'glyphicon glyphicon-use' for example leads to font problems in a normal dropdown button (if the button needs text as well as an icon. On top of that, if a split button is used then the classes are added to the wrong button. You can try it out and see how it looks.
I am interested however in a proper adding glyphicons functionality so do give it another go.


dev-Sentinel-2022-09-01-preview

Please note all PRs to this branch have already been reviewed and approved by Swagger team and ARM. No need for addition ARM review.

/azp run

From @anat-gilenson Cross-version breaking change due to miss alignment between one property in stable and preview which is currently being fixed in the last stable version.
Also, the PRs to dev branch have been reviewed, so we can merge this PR directly.


docs:Troubleshooting note on stale pricing data

Fixes Issue #2485
Description
This PR updates troubleshooting docs to note that clusters on a private subnet are unable to access pricing data because no VPC endpoint is available to get that data. The result is an error message noting that existing pricing data will be used.
Does this change impact docs?

[X] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

There is a workaround, there's an isolated vpc command line option that will disable the pricing loookups. On mobile or I would link it, but it's in the docs.

There is a workaround, there's an isolated vpc command line option that will disable the pricing loookups. On mobile or I would link it, but it's in the docs.

I'll see if I can find that link and describe that as a workaround.


France Driver's License Number

Hello,
The France Driver's License Number section seems to be mixed with the previous section (Finland passport number)

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

ID: 0c16b548-1fc9-2765-3e8a-7559be41fd9d
Version Independent ID: 7fe5f4ba-e53d-d86d-4ab9-64e872e33069
Content: What the sensitive information types look for - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/what-the-sensitive-information-types-look-for.md
Service: o365-seccomp
GitHub Login: @chrfox
Microsoft Alias: chrfox

HI @SabRom
Thanks for letting us know, I have fixed the formatting issues and republished the topic.


video dataset

Hi !  It is an amazing work!
I want to reproduce video action classification, but how can i  get dataset of ucf101,such as feature file and attribute file ?
I see "ucf101_manual_att.npy" in this code, can you share it ?

Hello @angela000,
Thank you for your kind words.
You can get the video dataset from this repo-> https://github.com/naraysa/gzsl-od.
Regards,
Akshita

Hello @angela000, is the issue solved?

Hello @angela000, is the issue solved?

yeah, it helps me a lot!


Allow to scroll to the top when clicking in the title area of editor

Click again on the open tab to scroll you back to the top of the document in goal not to use minimap or shortcuts to reach the top.
Practically the same thing Opera and Yandex browsers are using e.g.

I guess it's not to complicated to implement because there are no current action on double tap on the tab.
Cheers!

I guess it's not to complicated to implement because there are no current action on double tap on the tab.

Actually, there is one, to exit preview mode.

I find this a bit weird to have because there maybe many reasons to click into the title of an editor (to focus the contents) where you do not want to loose the current position. I feel like we would not implement this, even though it is a nice thing to have for touch devices and browsers.


Small typo: January misspelled

Thank you for the fix @mdelder
This is what I use to sign off on older commits:
git reset --soft HEAD~2
git commit -s -m "_commit_message_here_"
git push origin _branch_name_here_ -f

This should merge both commits into one and sign off on it.

/approve

@mdelder thanks. it needs to signoff. git commit --amend -s

This has been fixed in a separate PR. Thanks again for making us aware of this mistake Michael.


[DO NOT MERGE YET] Use internal LB when Prod AWS clients connect to Publishing API.

When Production Publishing API moves to AWS, clients in AWS should use
the internal load balancer instead of the external (public-facing) one
to reach Publishing API.
This doesn't affect clients in Carrenza, which will continue to use the
external LB.

Rebased so that this is still ready to go. There was a conflict in the comment. No change to the actual code.
Something to bear in mind when we roll this out: when Puppet agents pick up the change, they'll restart the Ruby apps because of the environment variable changes. Restarting frontend apps unfortunately isn't zero-downtime; it causes a short blip of user-visible errors. I don't think there's a lot that we can do about this though. (I believe the same thing happens when new versions of the frontend apps get deployed.)

Obsoleted by https://github.com/alphagov/govuk-puppet/pull/9902


pass data from vertex shader to fragment shader crash

Whenever I try to pass data data from vertex shader to fragment shader, my computer will crash. I tried other computer and the same crash happens.
Example:
Vertex Shader
#version 330 core

layout(location = 0) in vec3 position;

out vec4 ourColor;

void main(){
	gl_Position =  vec4(position ,1);
	ourColor = vec4(1.0f, 0.0f, 0.0f, 1.0f);
}

Fragment Shader
#version 330 core

in vec4 ourColor;
out vec4 color;

void main()
{
    color =  ourColor;
}

I don't see anything wrong in your shader. Maybe it's somewhere else causing crash.

I found the problem. I forgot to use gl.UseProgram(0); after drawing. Thanks for your help.


Avatar selection UI

can't change back to default avatar after changing into another avatar (hitting R on one)

During user testing, no one was able to change avatars, an avatar selection UI / first time user menu could help solve this
Similar to #238, #171, #37


Rel eng

Do we want to keep the history, or are we fine to do a forced push to prod to hop on the dev branch?

Can we go over the tests plan. I recall the beneficial idea of having one CI, one set of tests that can assert on codepaths for both production and dev site. Those "codepaths" are driven by settings.
Is the idea that these settings will be manipulated test-by-test using django-pytest facilities? Are we using monkeypatch for that, is that the idea?


Upgrades to eslint-cesium-config/node v9.0.0

See https://github.com/CesiumGS/eslint-config-cesium/pull/1 for more details.

Thanks for the pull request @sanjeetsuhag!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:sparkles: :sparkles: :sparkles:I'm the Cesium Concierge and I'm here to test things. :sparkles: :sparkles: :sparkles:

Updates look great, thanks @sanjeetsuhag!


feat(dialog): Implement MDC Dialog Service

WIP

[x] Re-architect MDC Dialog as Service
[ ] Update documentation and examples
[x] Include unit tests

Closes #141

Fixed last known issue. Updating docs..


chore: use safe stringify in http transport

Hello,

Solving the issue when stringifying circular structures in http transport.
Test added

Issue:
https://github.com/winstonjs/winston/issues/1946

@karlwir I think we should use safe-stable-stringify per https://github.com/winstonjs/logform/pull/98 -- what do you think?

@DABH Agree, my mistake. PR updated.

I love you @karlwir
I needed this fix today 😆

Hi! Im getting
TypeError: stringify.configure is not a function at Format.transform (/node_modules/logform/json.js:27:35)
I'm using winston 3.6.0 and morgan 1.10.0

Thanks for reporting that.  The OS difference initially makes me wonder if there might be a filename reference with incorrect casing somewhere.

It wasnt happening on<EMAIL_ADDRESS>tho.

It's definitely possible that the issue was newly introduced, and knowing that is quite helpful. I was just offering another signpost to possibly help with debugging as I don't have the time to do a full debug on it at the moment.


Limit to display first 50 group members

Groups size
I got some groups with high number of users. When I display the group page, the back is able to sent the list of members, but the front is unable to manage those and browser fail.
I added a limit to display at most 50 users, and set a "warning" on the members headers. 50 is a number choosen randomly.
It should be possible to use a paginated view but it will require more work.

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Hey! Thanks for the contribution!
I'm thinking that we probably either want to paginate this list, or at least have a show all button. It's a little unfortunate if you want to check a user is a member of a group but can only see the first 50 right?
What do you think?
Also /cc @lewtakm

ping @GregoireW?

paginate would be a good option. I saw there is already paginated component will check if I can adapt those for user list. Will need some time to do this.


Properties details page text boxs dont show unless hovered over

Description
as title says
Steps To Reproduce
No response
Expected behavior
The box should always be there
Files Version
1.9.17
Windows Version
10.0.19043.1110
Relevant Assets
No response

This seem to happen for any text box in the properties menu (Details view)

This now only applys to the details page

This issue should be resolved in main. (the details page is by design)


Portworx Enterprise - 3.0.4 Release

[x]  Portworx Enterprise Testing from Portworx Team
[ ]  Torpedo Test from IBM Cloud Team

PX-Enterprise 3.0.4 release notes
https://docs.portworx.com/release-notes/portworx-enterprise#304
Test results of Test catalog from test rail attached
IBM Marketplace - TestRail PX 3.0.4.pdf
what's new
PX operator 23.10.1
PX enterprise 3.0.4
Stork: 23.8.0

Tested stage catalog
NAME       CLUSTER UUID                           STATUS    VERSION   AGE
portworx   90a23826-8244-4ed1-914b-20e5970bcdcb   Running   3.0.4     81m


No USER in dockerfile anymore

for #34

i didn't try it out, but i did review the code changes. i think it make my use case much better. thanks!


make error

when doing a make, occures an error, saying:
cc1plus: warning: command line option ‘-Wmissing-prototypes’ is valid for C/ObjC but not for C++
cc1plus: warning: command line option ‘-Wdeclaration-after-statement’ is valid for C/ObjC but not for C++
cc1plus: sorry, unimplemented: -fexcess-precision=standard for C++
Makefile:22: recipe for target 'plan_tree_view.o' failed
make: *** [plan_tree_view.o] Error 1
is it my compiler config problem? i'm using 5.4.0 20160609, ubuntu, postgres 9.6rc1.

@davidzwb
Thank you for your information.
I try to fix the error by commit 8076cd4
Please check if it works.

Thanks, it works perfectly, the result plan picture is so beautiful.
but just for your information, i met the following one single warning during compiling:

plan_tree_view.cpp: In member function ‘void NodeInfoEnv::append(const char*, ...)’:
plan_tree_view.cpp:205:24: warning: function might be possible candidate for ‘gnu_printf’ format attribute [-Wsuggest-attribute=format]
vsprintf(buf, fmt, ap);


fix exceptions and errors for invalid inputs in cassandra execution dao

Pull Request Test Coverage Report for Build 3268

23 of 23 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 70.253%

Totals

Change from base Build 3256:
0.03%

Covered Lines:
10290

Relevant Lines:
14647

💛  - Coveralls


Improve the "thank you" page

This adds an image and some fun text to the "thank you" page.

@meacer PTAL

Looks tasty to me

This was for #215


Fix firebase warnings.

In Android on Windows there a constant Firebase warning which usually results in "v8 is out of memory" error.

This workaround doesn't solve the issue completely but it removes the warnings and crashes occur much-much less frequently.

See the firebase issue for more info:
https://github.com/firebase/firebase-js-sdk/issues/97

Have you tried to use line: console.disableYellowBox = true;
Although it does not solve the issue completely, it does remove the warnings (at least it did to mine)

Disabling all warnings is not a great decision. It might come back and bite you in the future.

https://stackoverflow.com/a/46678121

Disabling all warnings is not a great decision. It might come back and bite you in the future.

They might come back ... how about this link https://stackoverflow.com/a/46678121 ?

They might come back ... how about this link https://stackoverflow.com/a/46678121 ?

This is pretty much what my PR does.

They might come back ... how about this link https://stackoverflow.com/a/46678121 ?

This is pretty much what my PR does.

Are you saying your PR did this?         did it work?      I am facing the same issue these yellow issues too . This stackoverflow didn't work for me -   did it work for you or your PR?

Yes, PR removes the yellow box.

Hi @Undeadlol1
I understand why you've done this. Can you separate the specifics into a separate module/file (perhaps call it something like /src/lib/firebase-fix.js or something similar) so that it's separated and easily removed (if/when Firebase is fixed)?


"Sending a heartbeat took longer than 2 seconds" & "slow background settings sync" messages in the log during "arangobackup upload"

My Environment

ArangoDB Version:        3.8.8
Deployment Mode:       Cluster
Deployment Strategy:   ArangoDB Starter
Configuration:               4 node cluster
Infrastructure:               bare metal
Operating System:        RHEL 7.9
Total RAM in your machine:        500GB
Disks in use:        HDD
Used Package:              RedHat .rpm

Component, Query & Data
Affected feature:
DB Server

Size of your Dataset on disk:
1TB on each node
Replication Factor & Number of Shards (Cluster only):
Replication Factor: 3
Shards: 1
Steps to reproduce

Snapshot creation:
arangobackup create --label backup_name
Snapshot upload:
arangobackup upload --identifier timestamp_backup_name --rclone-config-file xxx.json --remote-path rc_local:///yyy/
Monitoring:
arangobackup upload --status-id=PID

Problem:
The following messages appear some times in the dbserver log file on an arbitrary host:
... 20:31:26 ... {heartbeat} ATTENTION: Sending a heartbeat took longer than 2 seconds, this might be causing trouble with health checks. Please contact ArangoDB Support.
... 20:31:26 ... {engines} slow background settings sync: 5.515023 s
... 20:31:54 ... {heartbeat} ATTENTION: Sending a heartbeat took longer than 2 seconds, this might be causing trouble with health checks. Please contact ArangoDB Support.
... 20:31:54 ... {engines} slow background settings sync: 5.740028 s
... 20:32:09 ... {heartbeat} ATTENTION: Sending a heartbeat took longer than 2 seconds, this might be causing trouble with health checks. Please contact ArangoDB Support.
... 20:32:09 ... {engines} slow background settings sync: 5.389080 s

We see these messages every time when there is some simultaneous user workload in the system.
Monitoring shows that the corresponding dbserver process is offline at such a time.
We haven't seen these messages before the migration from v3.7 on the same hosts.
The problem doesn't appear, if there is no user workload in the system simultaneously.
Expected result:
dbserver process must be online during this operation.

@mark-bb  can you create a ticket with support so that we can gather the requisite logs and information to debug this ?

@asifkazi Unfortunately, we can't create a ticket with support.
Any way to debug it / investigate the reason ourselves?


No collection_dir in Jekyll config causes TypeError

I don't have a collections_dir key in my Jekyll config; I think #74 is throwing a TypeError:
Error:  no implicit conversion of nil into String

I think the arg parser needs to gracefully handle the lack of a key, but I don't know enough Ruby to do this myself 😅
Full traceback from running $ bundle exec jekyll post "Hi" --trace
Configuration file: /Users/mitchellbusby/repos/mitchell-busby/_config.yml
/Users/mitchellbusby/.rvm/gems/ruby-2.3.1/gems/jekyll-compose-0.9.0/lib/jekyll-compose/arg_parser.rb:36:in `join': no implicit conversion of nil into String (TypeError)
        from /Users/mitchellbusby/.rvm/gems/ruby-2.3.1/gems/jekyll-compose-0.9.0/lib/jekyll-compose/arg_parser.rb:36:in `source'
        from /Users/mitchellbusby/.rvm/gems/ruby-2.3.1/gems/jekyll-compose-0.9.0/lib/jekyll/commands/post.rb:35:in `process'
        from /Users/mitchellbusby/.rvm/gems/ruby-2.3.1/gems/jekyll-compose-0.9.0/lib/jekyll/commands/post.rb:13:in `block (2 levels) in init_with_program'
        from /Users/mitchellbusby/.rvm/gems/ruby-2.3.1/gems/mercenary-0.3.6/lib/mercenary/command.rb:220:in `block in execute'
        from /Users/mitchellbusby/.rvm/gems/ruby-2.3.1/gems/mercenary-0.3.6/lib/mercenary/command.rb:220:in `each'
        from /Users/mitchellbusby/.rvm/gems/ruby-2.3.1/gems/mercenary-0.3.6/lib/mercenary/command.rb:220:in `execute'
        from /Users/mitchellbusby/.rvm/gems/ruby-2.3.1/gems/mercenary-0.3.6/lib/mercenary/program.rb:42:in `go'
        from /Users/mitchellbusby/.rvm/gems/ruby-2.3.1/gems/mercenary-0.3.6/lib/mercenary.rb:19:in `program'
        from /Users/mitchellbusby/.rvm/gems/ruby-2.3.1/gems/jekyll-3.3.0/exe/jekyll:13:in `<top (required)>'
        from /Users/mitchellbusby/.rvm/gems/ruby-2.3.1/bin/jekyll:23:in `load'
        from /Users/mitchellbusby/.rvm/gems/ruby-2.3.1/bin/jekyll:23:in `<main>'
        from /Users/mitchellbusby/.rvm/gems/ruby-2.3.1/bin/ruby_executable_hooks:15:in `eval'
        from /Users/mitchellbusby/.rvm/gems/ruby-2.3.1/bin/ruby_executable_hooks:15:in `<main>'

@mitchellbusby Thank you for reporting this.
It looks like the issue is because of improper versioning: We did not lock the Jekyll version at the time of release of jekyll-compose-0.9.0
The solution for you is to update both jekyll and the jekyll-compose gems.
You're using jekyll-3.3.0 and jekyll-compose-0.9.0. The latest versions are 3.8.5 and 0.10.1 respectively.
I'm closing this for now. I'll reopen if issue crops with with recent versions of the gems.


Model Binding article outline

Contributors, feel free to leave notes on an expanded version of the outline, as well as what other topics related to models and model binding you want to see in the docs.

Please create your PR once it is ready for review.


Fashion MNIST tutorial

Signed-off-by: Alex Sergeev<EMAIL_ADDRESS>

@sblotner, thanks for all the copyedits, it's amazing!


providers.gradleProperty to avoid invalidating the configuration cache but need to read the property at configuration time to disable the task

When using providers.gradleProperty("some-property").forUseAtConfigurationTime().isPresent to avoid invalidating the configuration cache but you have to read the property to disable a task. This is bad behavior as this means that if the property changes, the configuration process would be repeated.
This should be changed so that task.setEnabled should also accept a Provider<Boolean>

Expected Behavior
We should be able to disable the task without invalidating the configuration cache
Current Behavior

Current behavior would invalidate the configuration cache. If configuration cache is enabled and attempts to read it without the function call it would result in a crash.
Context

Attempting to disable tasks without invalidating the configuration cache

This is still relevant.

This is currently solvable with providers and onlyIf clause, but not for the gradle properties. Currently, all Gradle properties are the implicit inputs to the build configuration, so changing any invalidates the configuration cache regardless of if the property is actually used for anything. Future Gradle versions will introduce finer-grained tracking. In the meantime, you can use system property or environment variable providers:
tasks.register("hello") {
    def taskEnabled = providers.systemProperty("sayHello")

    onlyIf { taskEnabled.present }

    doLast {
        println "Hello"
    }
}


web: fix memory leak warning

Following the advice here https://www.robinwieruch.de/react-warning-cant-call-setstate-on-an-unmounted-component/

can you give more context on what problem you're trying to fix? i think you're very confused about something.

I'm going to close this because I can't reproduce the error message.


Train 116

#1081 Update dcos-vagrant version to reduce flaky tests
#912 Set MARATHON_DEFAULT_NETWORK_NAME to the name of the default dcos overlay network
#1099 Remove preflight from test runner
#1102 Fix the attribute error from dcos_generate_config.sh --aws-cloudformation

@adam-mesos all linked PR's have 2 ship-its / all green.

@mellenburg can you create a PR to address any EE changes that might be required for #1096 and ping myself and @adam-mesos on the new PR when ready?

@adam-mesos Thanks for holding the train on that point, its something I should be doing with each big testing change. Working on https://github.com/mesosphere/dcos-enterprise/pull/483 right now. Might take a while to smooth all the bump issues

@lingmann @adam-mesos This train must also include https://github.com/dcos/dcos/pull/1099 so I can do the upstream bump. Really, PR 1099 should have been included with the breaking change https://github.com/dcos/dcos/pull/1088 which should have demanded a similar upstream bump test before merge. We should probably just do that with every train....

Removed #1096 to simplify the train (that PR is a large refactor which also requires downstream updates).
Added #1099 to improve test coverage.

Added #1102 to fix blocker issue around dcos_generate_config.sh --aws-cloudformation

Merging without the downstream build going green. There is nothing ready yet to merge on downstream, and #1081 in particular needs to land ASAP because all PR builds are currently failing without it. I've added some updates to https://github.com/mesosphere/dcos-enterprise/pull/485 so we can work on moving it forward next.