Kubeone deployments fail occasionally because MachineDeployment is not ready

About half the time when deploying a cluster it fails with the error below. If I rerun the kubeone command it passes and the cluster works. Is Kubeone a bit impatient about waiting for the MachineDeployment CRD to be available?
INFO[09:47:05 UTC] Creating worker machines…
WARN[09:47:05 UTC] Task failed…
WARN[09:47:05 UTC] error was: failed to deploy Machines: failed to ensure MachineDeployment: failed to get *v1alpha1.MachineDeployment object: no matches for kind "MachineDeployment" in version "cluster.k8s.io/v1alpha1"
WARN[09:47:10 UTC] Retrying task…
INFO[09:47:10 UTC] Creating worker machines…
WARN[09:47:10 UTC] Task failed…
WARN[09:47:10 UTC] error was: failed to deploy Machines: failed to ensure MachineDeployment: failed to get *v1alpha1.MachineDeployment object: no matches for kind "MachineDeployment" in version "cluster.k8s.io/v1alpha1"
WARN[09:47:20 UTC] Retrying task…
INFO[09:47:20 UTC] Creating worker machines…
WARN[09:47:20 UTC] Task failed…
WARN[09:47:20 UTC] error was: failed to deploy Machines: failed to ensure MachineDeployment: failed to get *v1alpha1.MachineDeployment object: no matches for kind "MachineDeployment" in version "cluster.k8s.io/v1alpha1"
Error: failed to create worker machines: failed to deploy Machines: failed to ensure MachineDeployment: failed to get *v1alpha1.MachineDeployment object: no matches for kind "MachineDeployment" in version "cluster.k8s.io/v1alpha1"
failed to create worker machines: failed to deploy Machines: failed to ensure MachineDeployment: failed to get *v1alpha1.MachineDeployment object: no matches for kind "MachineDeployment" in version "cluster.k8s.io/v1alpha1"

Is Kubeone a bit impatient about waiting for the MachineDeployment CRD to be available?

Maybe, but I personally never faced such case (and I created/destroyed hundreds of clusters).
Is it possible that control-plane VMs are running on overloaded hosts for example?
@xmudrii WDYT about adding logic to query for presence of our CRDs?

The cluster we are running this on is very lightly loaded. 4 dual socket machines with lots of memory and a quite fast SAN.

I have no idea how this can happen ¯_(ツ)_/¯, but we can work around with active checks for CRDs presence after they been committed to the kubeAPI.

Maybe, but I personally never faced such case (and I created/destroyed hundreds of clusters).

This is a strange case. I have also never experienced such case, neither I remember seeing anything similar in E2E tests.

Is it possible that control-plane VMs are running on overloaded hosts for example?

This is an important point. If VMs are running on overloaded or slow hosts, the API server might misbehave, causing such errors.

WDYT about adding logic to query for presence of our CRDs?

@kron4eg We already have the logic for verifying the presence of CRDs, but we don't use it before creating workers.
However, I think that the CRDs are created since KubeOne didn't report any error when deploying CRDs and it validated that the machine-controller is running and healthy. It might happen that the CRDs are still not being served by the API server. This, for example, can happen if the instance is overloaded or too slow. It could help if we would check is the CRD Established, and only proceed if it is. Once the CRD is Established, the API server is serving it and such errors should not happen.

I have seen a similar problem with istio. You have to deploy it in two phases because the CRDs take a while to be available.
https://istio.io/docs/setup/install/helm/#option-1-install-with-helm-via-helm-template

@mooperd yeah, that's why my first idea was about slow VMs or underlying hardware.
@xmudrii SGTM! Please add this check before in step before we create MachineDeployment.


Add diff-sequences package

This generic package has specific goals for jest-diff in a future pull request:

Minimize time and space when expected and received values have many differences.
Minimize code differences between --expand and --no-expand options.

Here are recommended steps to review this pull request:

Read README.md especially the examples of callback functions

Refer to An O(ND) Difference Algorithm and Its Variations by Eugene W. Myers

Read up through Figure 1 on page 3 and make sure to understand the “edit graph” analogy
Read pseudo code in Figure 2 on page 6
Look at Figure 3 Furthest reaching paths on page 7
Read pseudo code on pages 11 and 12

Read tests in index.test.js

The first time you read code in index.js skip the two Overlappable functions

Overview of cost:

time and space optimization: skip diagonals in which paths cannot ever overlap

heap used is two arrays of small integers: each has max length of one more than length of shorter sequence

stack depth is at most logarithm of number of differences [EDIT optimization: does not recurse whenever number of changes is equal to number of items in either half of divided interval, therefore cannot contain any common items]

Test plan
44 tests [EDIT: plus 4 and forgot to say 100% coverage]

This is a huge diff but it looks really really good.

Can you use ES exports instead?
Flow-typed makes sense if you wanna use it outside of this repo.


Add support for Batch API

Batch API is important for scaling writes and realtime optimization scenarios to Campaign, Line Items, and Targeting Criteria as an extreme amount of them will not be possible to done in parallel in the single POST/PUT endpoints.
Relevant endpoints to implement:
https://dev.twitter.com/ads/reference/post/batch/accounts/%3Aaccount_id/targeting_criteria
https://dev.twitter.com/ads/reference/post/batch/accounts/%3Aaccount_id/line_items
https://dev.twitter.com/ads/reference/post/batch/accounts/%3Aaccount_id/campaigns

Added in latest commit


[FEA] Add support for bucketed writes

Is your feature request related to a problem? Please describe.
The SQL plugin supports partitioned writes but not bucketed writes.  the main thing preventing this from working is consistent hashing between the CPU and GPU implementations.  This will require us to create a version of the murmur3 hash the matches exactly with what spark does and may need us to write it ourselves as it is likely to be spark specific.

This depends on #937

We could partially implement this now.

To fully implement this we will need full support for bit for bit identical murmur3 hashing.


cannot load library 'libgdf.so'

wen i use
python setup.py install
python setup.py build
install and build pygdf.
then,run py.test
an OSError exist:
cannot load library 'libgdf.so': libgdf.so: cannot open shared object file: No such file or directory

You will need to install https://github.com/gpuopenanalytics/libgdf.
We don't have wheels for libgdf yet, so you must install with conda.  i.e. conda install -c defauls -c conda-forge -c gpuopenanalytics/label/dev libgdf libgdf_cffi
Or follow the setup instructions at https://github.com/gpuopenanalytics/pygdf#setup-with-conda for easier setup.

thank you very much.
But another error has emerged.
"AttributeError: cffi library 'libgdf.so' has no function, constant or global variable named 'gdf_left_join_generic' "

I use
'''
conda env update --name libgdf_dev --file ../conda_environments/builddocs_py35.yml
'''
solve the above error.
I offer my thanks again
Pygdf willing to get better and better

This seems to be resolved, closing.


classification/detection

Hello,
Is it for classification or detection of breast cancer lesion ?!
Thanks,

I even can't download the dataset to try it !
Is there another link ?


CLI configure environment for a second user

[x] I have checked that the SDK documentation doesn't solve my issue.
[x] I have checked that the API documentation doesn't solve my issue.
[x] I have searched the Box Developer Forums and my issue isn't already reported (or if it has been reported, I have attached a link to it, for reference).
[x] I have searched Issues in this repo and my issue isn't already reported.

Description of the Issue
I have installed CLI in my environment last month and configured environment with my json.config file using defaults
When I check box users:get - it returns the info for my service account user (and I generate token for that user.
NOW I wish to add a SECOND automation user and private key to get token with the SECOND automation user's credentials.
After following the directions:
$ box configure:environments:add PATH_TO_CONFIG_FILE --private-key-path PATH_TO_PRIVATE_KEY --name ManualKey
Successfully added CLI environment "ManualKey"
Steps to Reproduce
I have executed  --name SecondKey  in my environment
and then attempted to verify
by executing box users: get SecondKey
box tokens: get SecondKey
and nothing happens
Expected Behavior
I expected to return the user info for the SECOND automation user
when I execute box users:get it returns information from my first environment
Goal
I would like to be able to use both environment regularly to produce tokens [box tokens:get] for each service account user
Screenshots
not available
Versions Used
Box CLI:
Operating System:

So you still get data of the first environment after adding the second one (for example, when calling box:users)? Did you switch the default environment to the second one after adding it? I see that this is not done by default. You can do it with the following command
box configure:environments:set-current EnvironmentName

So you still get data of the first environment after adding the second one (for example, when calling box:users)? Did you switch the default environment to the second one after adding it? I see that this is not done by default. You can do it with the following command box configure:environments:set-current EnvironmentName

It's not possible right now, but we plan to implement some kind of --profile flag that can be passed to the command to temporarily use a different environment for a single request! This is already in our backlog (SDK-3342) and somehow related https://github.com/box/boxcli/issues/498 . Unfortunately, I can't provide a timeline for this feature for now.

Yes it looks like #498 is requesting a similar flow.
So to confirm. At this point CLI can be be configured to a SINGLE environment?
This is my scenario:

I have two departments in the same corp - each have their own JWT service account.
I would like to be able use script box tokens:get --DeptA  box tokens:get --DeptB after configuring environment to both these users DeptA and DeptB.
The CLI for each would be stored and triggered from a shared cloud location.
Is the above possible via CLI?  If not do you have an alternate suggestion. I am only using CLI for JWT Token.

For now if you want to use diffrent box aps you should create two environments switch them after each call
Assuming that you have two environments with names DeptA and DeptB
You can confirm it by box configure:environments:get command
box configure:environments:set-current DeptA
box tokens:get (will get token for DeptA)
box configure:environments:set-current DeptB
box tokens:get (will get token for DeptB

and so on.
The --profile proposal would allow to do that without the constant switching, just by supplying additional flag in each call.


Custom interactions with Toast via descriptions

Description
Enable richer interaction with toasts by supporting JSX for description or supporting custom render (as in Zag).
For example, a toast with a link in the description is useful to allow users to click on the toast to take them to the relevant item in the app.
Proposed Solution or API
Support https://zagjs.com/components/solid/toast#custom-render-functions in Ark.
Or
Support JSX elements for description rather than only strings.
Additional Information
No response

Thanks for taking your time to report this issue. A fix is on the way


[BACKLOG-36622] When there are no recent files, there is no message t…

…o indicate this in the file open/save dialog

:white_check_mark: Build finished in 6m 44s
Build command:
mvn clean verify -B -e -Daudit -amd -pl \
plugins/file-open-save-new/core

:ok_hand: All tests passed!
Tests run: 15, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


Amend the StorageClass page

Assigning @sftim since he is already engaged on this one.
/assign @sftim

/sig storage
@tengqm [IMO] this either needs tech review, or I can LGTM it if I have a pointer to why those numbers are right.

Parameter number limit:
https://github.com/kubernetes/kubernetes/blob/master/pkg/apis/storage/validation/validation.go#L97
where maxProvisionerParameterLen is defined as 512 at
https://github.com/kubernetes/kubernetes/blob/master/pkg/apis/storage/validation/validation.go#L39
Parameter size limit:
https://github.com/kubernetes/kubernetes/blob/master/pkg/apis/storage/validation/validation.go#L109
where maxProvisionerParameterSize is defined at
https://github.com/kubernetes/kubernetes/blob/master/pkg/apis/storage/validation/validation.go#L38

/lgtm
/approve


Fix various doc typos

Hi, I have fixed a few typos in documentation files only. Thanks :)

There is also something wrong in disk-c64.md in the line:
1581 disks are just another version of the standard B Just do:
But I'm not quite sure what the correct text is.

Wow --- that embarrassing, particularly as my mother was an English teacher... let me know when it's ready and I'll merge. Thanks!

Haha, no worries! It's mostly very good, I'm just nit-picking really :)
I think I'm done for the time being, other than noticing that on cowlark.com there is an oddity where an extra Acorn ADFS entry appears in the navigation, but I can't see where to fix that.

The last will be an issue with my CMS --- will fix. Thank you very much for doing this!


Update assembly version

When making new releases it should be good if the assembly version was updated to easy be able to see the version of the letencrypt.exe-file to not manual remember the version that is used on each server.

the version command is not returning the released version number.

letsencrypt.exe --version
LetsEncrypt.ACME 1.0.5816.17976

Fixed in 1.7
https://github.com/Lone-Coder/letsencrypt-win-simple/releases/tag/v1.7


chore(devDependencies): bump version for documentation

Coverage remained the same at 99.615% when pulling 1815ced065e6a77417cfc2823dc2f900798fac26 on prtksxna:devdep into 6ef0bd8de8a2ed873f7494f731903b86b89b01f7 on decisively:master.


[FR] - make --port a config file parameter

External   otherwise.
Area
Other
Describe the feature you'd like
same as the node config file already has port numbers for EKG and prometheus, the miniprotocol port should also become a configuration parameter in the config file.
Describe alternatives you've considered
surrounding tools who want/need to know on which port the node runs, need manual (faulty) extra config values, or complicated ways to identify the cli startup parameter
Basically all command line parameters other than --config could become part of the config file

@Jimbo4350 , @CarlosLopezDeLara> how do we stand on this feature request?

Ping @gufmar, did you see my question above?


Allow for nested no_commands blocks.

Allow for nested no_commands blocks.  🌈
Background
In certain circumstances we may wish to nest no_commands blocks.
However, since when exiting the no_commands block the @no_commands
instance var is reset to false, we get unexpected behaviour.
For example, this will show warnings stating that the method foo is missing usage and a description:
class Foo < Thor

  no_method do
    attr_reader :bar

    def foo
      'foo!'
    end
  end
end

This particular example is trivially solved but things get more difficult when, say, including modules.
# lib/thor/base.rb
def no_commands
  @no_commands = true
  yield
ensure
   @no_commands = false
end

Solution
This change uses a NestedContext object to track the depth of the no_command blocks and ensure that we only start creating commands again once we've left all of them.
Comments

I'm definitely not sold on the name NestedContext

Thank you for the pull request. I think this solution is too complex just to allow nested no_commands. Can you explain why you need nested no_commands?

Hello, I have been working on refactoring some scripts for a Rails application and wanted to make use of the ActiveModel validation API.  So I included the ActiveModel::Validations module which created the nested no_commands via an attr_reader.


Acceder

Acceder
Originally posted by @jojorgeluisrdz198 in https://github.com/jlord/sheetsee.js/issues/26#issuecomment-978733065

Acceder


hidden metadata textTrack not triggering "cuechange" events for Android WebView

Description
When setting a 'metadata' text track to 'hidden' mode and adding a cuechange event, the events are not being fired on Android WebView.Same code perfectly working on Chrome :)
Steps to reproduce
Explain in detail the exact steps necessary to reproduce the issue.
1.set the captions track to hidden
2.add cuechange event listener
3.send some 'ad-cues' events in HLS format.
Results
Expected
cuechange event should be trigger when encounter the 'ad-cues'
Chrome Working Code Snippet:-
https://codepen.io/sunilkumarjena21/pen/NWKpRGa?editors=1000
Actual
event is not being triggered on Android WebView on android device, but works perfectly fine on Desktop Chrome browser.
Error output
cueChange event not triggering at all in mobile device(tested on Android WebView)
Additional Information
versions
videojs
5.16.0
what version of videojs does this occur with? 5.16.0
browsers
Android WebView
what browser are affected?
Android WebView
OSes
Android,IOS
what platforms (operating systems and devices) are affected?
Android,IOS
plugins

are any videojs plugins being used on the page? If so, please list them below.
No.
Quick response will be highly appreciated 👍
Thanks in advance

Hello,
We're taking a look at what's causing the cuechange event not to fire. Would you be able to attach any logs of the error that could help us debug the issue?
As a note, WebView is not one of our supported browsers for videojs-ima, but we will continue to investigate the issue.

Hello @Kiro705 ,
Thanks for your response :)
Log:-If you run the code given in the link below,you can see the Advert position:length--0 or length--1 .. But its not getting printed when you run the same code on webview on Android..
Reason: track.addEventListener('cuechange', cueChangeHandler);
This Listener is not getting called either on Android WebView.
Chrome Working Code Snippet:-
https://codepen.io/sunilkumarjena21/pen/NWKpRGa?editors=1000
Thanks in advance!

Hi @sunilkumarjena21,
Have you considered using the Android IMA SDK or the Android ExoPlayer extension with IMA.  Those are two ways of using IMA with Android that we do support.
At the moment WebView is not one of the supported browsers for the html5 IMA SDK.

Thanks @Kiro705 ,
Reason for choosing the VideoJs player is : We just wanted to use a player which is supported throughout the platform(Web,Android,iOS etc).So completed 80% of task around Android after completing web part.
Now come to know that as VideoJs-IMA It is not going to support in Android & iOS platform.Its obiously a setack for us. Becasue lot of hard work would going to be wasted.
Just a final question:- What i understood Videojs-ima is not build  for Mobile platforms(Android & iOS)?
If yes how VideoJS-IMA supports mobile platforms.(Through Cordova or framework like ionic) or what?
Thanks in advance!

Hi @sunilkumarjena21,
I am looking into your specific use case with my team, but I wanted to ask if you had considered using the DAI IMA SDK for Android and iOS, with an Ad Manager 360 account?

Hi Kiro ,
Thanks for your supporting hand.Sorry for coming late on response from my side.
I might failed to explain our requirement correctly to you.
Workflow of existing Browser platform & Mobile Platform:-

The live stream used by  IMA DAI BasicExample is different than   VideoJs-IMA  Workable  stream
In-case of Google's live stream URL  the DFP adTag Urls is by-default configured inside the Playlistbut  but incase of VideoJs-IMA Workable stream  the configuration expects those URL should be served outside of the HLS playlist.

Our Requirement:-
Just wanted to know how to use the stream(used in  VideoJs-IMA) in  IMA DAI BasicExample or similar Demo for Mobile platforms(Android & iOS)?
Thanks in advance for your kind response :)
Regards
Sunil

Hi Sunil,
The DAI IMA SDK supports server side ad-insertion through the use of an Manager 360 account. In the account you can create your video stream with ads, and generate the corresponding Content Source ID and Video ID used in the IMA DAI BasicExample.
However, DAI does not support client side insertion of ads into a video stream.
The videojs-ima plugin is designed to support client side ad management through VAST tag ad playing and not server side ad-insertion.
Please let me know if you have farther questions or concerns.
Thanks,
Jackson
IMA SDK DevRel

Hi Arnaud  ,
Thanks for your quick response.
Can you please guide me:- how run ads using the client side IMA SDK for Android.
I want to use this stream(used in VideoJs-IMA) in client side IMA SDK for Android.
Any demo or example link would be really helpful.
Thanks in advance!

Hi Sunil,
The set up you are looking for would involve setting up a way to determine when to pause the video stream, possibly based on metadata managed by you from the stream. Ad playback would then occur during those stream breaks.
To be clear, this is not a method directly supported by either the client side or server side SDK, and would involve considerable setup on your teams part.
Thanks,
Jackson
IMA SDK DevRel

HI Kiro,
Thanks for your quick response:)
Description
Registered ID3 events not fired in any of the sample given by Android SDK.
Description
Registered ID3 events not fired in any of the sample given by Android SDK.
i.e the below @overriden method not being triggered when meeting to cuechange event.
The events are not being fired on despite the URL(HLS playlist) contains the #EXT-X-CUE-IN & #EXT-X-CUE-OUT tags.Same URL(HLS playlist) working perfectly in VIdeoJS IMA plugin on Chrome :)
Workable VideoJS-IMA-URL:-https://adinsertplayer.s3.ap-south-1.amazonaws.com/Demoplayer2.html
Steps to reproduce
1.set this URL(https://adinsertplayer.s3.ap-south-1.amazonaws.com/sampleoutput0302_modified_1_Live.m3u8) as StreamUrl.
2.Go to SampleVideoPlayer.java
3.Go to the below method to dubug & check if "onMetadata(Metadata metadata)" mehthod is being triggered or not.
Code Snippet of complete method:-
    // Register for ID3 events.
    mPlayer.addMetadataOutput(new MetadataOutput() {
        @Override
        public void onMetadata(Metadata metadata) {
            for (int i = 0; i < metadata.length(); i++) {
                Metadata.Entry entry = metadata.get(i);
                if (entry instanceof TextInformationFrame) {
                    TextInformationFrame textFrame = (TextInformationFrame) entry;
                    if ("TXXX".equals(textFrame.id)) {
                        Log.d(LOG_TAG, "Received user text: " + textFrame.value);
                        if (mPlayerCallback != null) {
                            mPlayerCallback.onUserTextReceived(textFrame.value);
                        }
                    }
                }
            }
        }
    });

Results Expected:-
"onMetadata(Metadata metadata)" event should trigger when encounter the #EXT-X-CUE-IN & #EXT-X-CUE-OUT tags in the playlist
Actual:-
event is not being triggered on this android IMA DAI demo app, working perfectly in VIdeoJS IMA plugin on Chrome desktop browser.
Error output
"onMetadata(Metadata metadata)" event should trigger when encounter the #EXT-X-CUE-IN & #EXT-X-CUE-OUT tags in the playlist at all in android device(tested on Android Samsung s9)
Quick response will be highly appreciated
Thanks in advance

Yes, this is expected behavior as the DAI is expecting a Content Source ID and Video ID as demonstrated in the in the IMA DAI BasicExample.
From my understanding, the implementation you are looking for is not one supported in our demos or documentation. Please refer to the documentation for the intended use of the server side IMA SDK for Android.


Adding {ilog, ilog2, ilog10} to PrimInt

This is a breaking change, but one in line with how the language has evolved.
{ilog, ilog2, ilog10} should all be methods in PrimInt. These have been stable for almost 2 years.
Also probably the checked variants, too.
The signatures are essentially
pub const fn ilog(self) -> u32;
pub const fn ilog2(self) -> u32;
pub const fn ilog10(self) -> u32;

EDIT: as a stopgap, these could go in a ILog trait and just be there for use to avoid breaking everything.

They can go on PrimInt if you can figure out generic default implementations.


Effect: 'fade' and slidesPerView: 4 not working together

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: Swiper 3.4.2

Platform/Target and Browser Versions: CHROME

What you did
var swiper = new Swiper('.partners', {
    slidesPerView: 4,
    spaceBetween: 30,
    effect: 'fade'
});

Expected Behavior
I expected it to show 4 slides and fade them all out and in at once with the next 4
Actual Behavior
Slides were stacked and faded on top of each other

fade effect designed to be used ONLY with slidesPerView: 1


Weird "bleeding" over of error highlighting

This image: http://tinyimg.io/i/F79dnD2.png
was made from this link: https://godbolt.org/g/mFWuR0 - note how the top left pane in the image has a wiggly red underscore under the semicolon, despite not having an error.
From Ezra Chung on slack: "It was when I changed the name of the function from g to f in source 2."

Video at: https://streamable.com/ouzaz


avtXRayFilter.C:192:30: error: invalid conversion from ‘const int*’ to ‘int*’ [-fpermissive]

I am trying to compile VisIt 3.0.2 from source with the command
./build_visit3_0_2 --thirdparty-path /cluster/apps/visit/thirdparty7/visit --openssl --xdmf --console --netcdf --szip --tbb --ospray --ispc --embree --mesagl --glu --python --boost --silo --hdf5 --cgns --fastbit --vtk --parallel --icet --qt --system-cmake --llvm --zlib --mili --boxlib --cfitsio --gdal --mfem --pidx --uintah --makeflags -j 8 --cflags -ftree-vectorize -march=core-avx2 -mavx2 --cxxflags -ftree-vectorize -march=core-avx2 -mavx2
The compilation crashes due to multiple conversion errors regarding avtXRayFilter.C:
[ 57%] Qt Wrapped File
/cluster/apps/visit/thirdparty7/visit_build/visit3.0.2/src/avt/Filters/avtXRayFilter.C: In function ‘int VisIt_XRay_MPI_Alltoallv(T*, const int*, const int*, T*, const int*, const int*, MPI_Comm) [with T = float; MPI_Comm = ompi_communicator_t*]’:
/cluster/apps/visit/thirdparty7/visit_build/visit3.0.2/src/avt/Filters/avtXRayFilter.C:192:30: error: invalid conversion from ‘const int*’ to ‘int*’ [-fpermissive]
                          comm);
                              ^
In file included from /cluster/apps/visit/thirdparty7/visit_build/visit3.0.2/src/avt/Filters/avtXRayFilter.C:48:0:
/cluster/apps/openmpi/1.6.5/x86_64/gcc_4.8.2/include/mpi.h:1046:20: error:   initializing argument 2 of ‘int MPI_Alltoallv(void*, int*, int*, MPI_Datatype, void*, int*, int*, MPI_Datatype, MPI_Comm)’ [-fpermissive]
 OMPI_DECLSPEC  int MPI_Alltoallv(void *sendbuf, int *sendcounts, int *sdispls,
                    ^
/cluster/apps/visit/thirdparty7/visit_build/visit3.0.2/src/avt/Filters/avtXRayFilter.C:192:30: error: invalid conversion from ‘const int*’ to ‘int*’ [-fpermissive]
                          comm);
                              ^
In file included from /cluster/apps/visit/thirdparty7/visit_build/visit3.0.2/src/avt/Filters/avtXRayFilter.C:48:0:
/cluster/apps/openmpi/1.6.5/x86_64/gcc_4.8.2/include/mpi.h:1046:20: error:   initializing argument 3 of ‘int MPI_Alltoallv(void*, int*, int*, MPI_Datatype, void*, int*, int*, MPI_Datatype, MPI_Comm)’ [-fpermissive]
 OMPI_DECLSPEC  int MPI_Alltoallv(void *sendbuf, int *sendcounts, int *sdispls,
                    ^
/cluster/apps/visit/thirdparty7/visit_build/visit3.0.2/src/avt/Filters/avtXRayFilter.C:192:30: error: invalid conversion from ‘const int*’ to ‘int*’ [-fpermissive]
                          comm);
                              ^
In file included from /cluster/apps/visit/thirdparty7/visit_build/visit3.0.2/src/avt/Filters/avtXRayFilter.C:48:0:
/cluster/apps/openmpi/1.6.5/x86_64/gcc_4.8.2/include/mpi.h:1046:20: error:   initializing argument 6 of ‘int MPI_Alltoallv(void*, int*, int*, MPI_Datatype, void*, int*, int*, MPI_Datatype, MPI_Comm)’ [-fpermissive]
 OMPI_DECLSPEC  int MPI_Alltoallv(void *sendbuf, int *sendcounts, int *sdispls,
                    ^
/cluster/apps/visit/thirdparty7/visit_build/visit3.0.2/src/avt/Filters/avtXRayFilter.C:192:30: error: invalid conversion from ‘const int*’ to ‘int*’ [-fpermissive]
                          comm);
                              ^
In file included from /cluster/apps/visit/thirdparty7/visit_build/visit3.0.2/src/avt/Filters/avtXRayFilter.C:48:0:
/cluster/apps/openmpi/1.6.5/x86_64/gcc_4.8.2/include/mpi.h:1046:20: error:   initializing argument 7 of ‘int MPI_Alltoallv(void*, int*, int*, MPI_Datatype, void*, int*, int*, MPI_Datatype, MPI_Comm)’ [-fpermissive]
 OMPI_DECLSPEC  int MPI_Alltoallv(void *sendbuf, int *sendcounts, int *sdispls,
                    ^
/cluster/apps/visit/thirdparty7/visit_build/visit3.0.2/src/avt/Filters/avtXRayFilter.C: In function ‘int VisIt_XRay_MPI_Alltoallv(T*, const int*, const int*, T*, const int*, const int*, MPI_Comm) [with T = double; MPI_Comm = ompi_communicator_t*]’:
/cluster/apps/visit/thirdparty7/visit_build/visit3.0.2/src/avt/Filters/avtXRayFilter.C:204:30: error: invalid conversion from ‘const int*’ to ‘int*’ [-fpermissive]
                          comm);
                              ^
In file included from /cluster/apps/visit/thirdparty7/visit_build/visit3.0.2/src/avt/Filters/avtXRayFilter.C:48:0:
/cluster/apps/openmpi/1.6.5/x86_64/gcc_4.8.2/include/mpi.h:1046:20: error:   initializing argument 2 of ‘int MPI_Alltoallv(void*, int*, int*, MPI_Datatype, void*, int*, int*, MPI_Datatype, MPI_Comm)’ [-fpermissive]
 OMPI_DECLSPEC  int MPI_Alltoallv(void *sendbuf, int *sendcounts, int *sdispls,
                    ^

Is this a known problem, and how can it be resolved ?
Any help is appreciated.
Best regards
Sam

So, I am not sure its related but you have a space-saparated set of args to both --cflags and --cxxflags options to build_visit and I don't see them surrounded by double quotes. I think you need to invoke like so...
build_visit ... --cflags "-ftree-vectorize -march=core-avx2 -mavx2" --cxxflags "-ftree-vectorize -march=core-avx2 -mavx2"

But, I don't think this has anything to do with your build error.
Can you possible cd to src/avt/Filters and do a make VERBOSE=1 there and attach output here?

The error is due to the function VisIt_XRay_MPI_Alltoallv using "const int *" for some arguments and passing those along to MPI_Alltoallv, which uses "int " in those cases. Looking at current documentation for MPI_Alltoallv for MPICH and OpenMPI, they also use "const int", so I assume that this is an old version of MPI where the arguments weren't const qualified.
We can remove the "const" for those functions.

The fix will be in the 3.1 release of VisIt, which should be out in 2 weeks.


Login flow

Summary
This describes a feature test for login flow to the forum using the EGI SSO IdP via CheckIn.
The feature is described at tests/cucumber/features/login.feature

Related issue : #28 #26
Consider author for inclusion:

[x] author list
[ ] contributor list

The build failed because of the environment


add TODO about removing synchronous file read

This looks ok -- verified that in production, it uses cached versions of the locale files:
https://github.com/jeresig/i18n-node-2/blob/b99fd69469d0f014c1a2015800aa206d8ed23998/i18n.js#L347-L349
and the cache is global to the module:
https://github.com/jeresig/i18n-node-2/blob/b99fd69469d0f014c1a2015800aa206d8ed23998/i18n.js#L97


AddInMemoryPersistedGrants seems not needed in Samples

like here:
IdentityServer4.Samples/Quickstarts/7_JavaScriptClient/src/QuickstartIdentityServer/Startup.cs

correct. PR?

I'm not sure I've done all rigth but it's here:
https://github.com/IdentityServer/IdentityServer4.Samples/pull/100


automatically display comments or docstring in boxes as virtual text

Rather than having to manually convert comments or docstring to boxes, it would be nice to have option to automatically display them in boxes as virtual text. That way you don't modify the source code.

I'm rather new to Neovim and lua, and I had no idea it was something possible. I will definitely look into it!

I don't think this is a good opinion, if you do so in other editors to read will not see the effect

Yeah not having other people see it would not be so great but, on the other hand, scarcity breeds value. :) I think it would be a really cool option to have.

Would be great if instead of permanently persisting the changes to the comments, to instead have it be just a visual thing for while the buffer is open. Not sure if that's the same thing being suggested here. Basically, I'd love to have my comments visually rendered in a box but I don't want the source files actually changed to that format.

Would be great if instead of permanently persisting the changes to the comments, to instead have it be just a visual thing for while the buffer is open. Not sure if that's the same thing being suggested here. Basically, I'd love to have my comments visually rendered in a box but I don't want the source files actually changed to that format.

Yes, it would be a nice option to have. I'll look into it!


Feature request: support "HALO" annotation in DEFs

I want to be able to initialize a floorplan based on a provided DEF file where a component has a "HALO" annotation, and noticed this currently isn't supported (read_def -floorplan_initialize outputs "error: HALO on component is unsupported"). Are there plans to add this feature eventually? It'd be helpful for my use case.
Thanks!

It would be helpful to have the exact message which makes it easier to see where in the code the problem is.  Even better is a small test case.

I see, it is one of our old pre-logger messages.  Are you in need of ROUTEHALO too?  Are you applying this to a macro?  If so that macro placer supports a halo option though it has to be specified outside of DEF.

Thanks for the prompt reply! We don't need ROUTEHALO, and yes, we are applying it to a macro. Thanks for the note about the macro placer - I think that could be a helpful workaround, but for our case it would ultimately be ideal to read the DEF file annotation (I'm trying to prepare a floorplan that could in theory be fed into a variety of tools with minimal tool-specific configuration).

I agree that the macro placer should be able to pick up the halo from DEF as well.  We are in the middle of bringing up a new macro placer so I'll suggest this for the new one (@ravi-varadarajan).

@nmoroze  To confirm your use model you are initializing a floorplan with   read_def -floorplan_initialize and in the COMPONENT section of the DEF there is  +HALO which is currently throwing an error.  You would like this to be to used as the halo specification for the component.  Also if there is a global halo specification as in the call macro_placement  command, would this be additive?

@ahmadelrouby please take care of the opendb part of this problem (taking the halo from def and storing it).

If it is stored as a PORPERTY on the COMPONENT, we can use during macro placement

There is a dbInst::getHalo but oddly not setHalo.  There is a field to store the data so we just need to hook it up and make it functional.  There is no field for a soft halo but we could add a bit if needed.

@nmoroze To confirm your use model you are initializing a floorplan with read_def -floorplan_initialize and in the COMPONENT section of the DEF there is +HALO which is currently throwing an error. You would like this to be to used as the halo specification for the component. Also if there is a global halo specification as in the call macro_placement command, would this be additive?

That's correct! I haven't thought about how this would interact with a global halo specification. Maybe the HALO in the DEF could take priority, since it's more specific? I won't be needing a macro_placement command for my use case though (all macros will be placed in the provided DEF), so I have no specific needs for how those two should interact.


Upload test logs

I'm trying to debug a test failure in https://github.com/microsoft/reverse-proxy/pull/168 and found that the test logs are not being uploaded.

Nevermind, I finally found them.

Were you looking for Microsoft.ReverseProxy.Tests_netcoreapp3.1_x64.log from https://dev.azure.com/dnceng/public/_build/results?buildId=647030&view=results? If so, where did you find it?
Also, it looks like the following line in our job template is broken:
https://github.com/microsoft/reverse-proxy/blob/f12c9eacf954f3d5d5cd15d8912841ead06ceabf/eng/common/templates/job/job.yml#L29

Not sure why or what the fix is though.

Not sure why or what the fix is though.

https://github.com/dotnet/arcade/pull/5171

Approved!


Global map construction

Hi team,
Thanks for your work! I want to build my global map features by myself and obtain the pt files, i.e. training and testing map, how can I modify the model?

Hi! You can set the save_map_path here,
https://github.com/HXMap/HRMapNet/blob/197f3407e130ced435db40bff91954286d480ce2/projects/configs/hrmapnet/hrmapnet_maptrv2_nusc_r50_24ep.py#L76
and use this function to save the map when it is built well,
https://github.com/HXMap/HRMapNet/blob/197f3407e130ced435db40bff91954286d480ce2/projects/mmdet3d_plugin/hrmap/global_map.py#L244


ROX-20160: Add RHTAP build for main

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
TODO(replace-me)
Use this space to explain how you validated that your change functions exactly how you expect it.
Feel free to attach JSON snippets, curl commands, screenshots, etc. Apply a simple benchmark: would the information you
provided convince any reviewer or any external reader that you did enough to validate your change.
It is acceptable to assume trust and keep this section light, e.g. as a bullet-point list.
It is acceptable to skip testing in cases when CI is sufficient, or it's a markdown or code comment change only.
It is also acceptable to skip testing for changes that are too taxing to test before merging. In such case you are
responsible for the change after it gets merged which includes reverting, fixing, etc. Make sure you validate the change
ASAP after it gets merged or explain in PR when the validation will be performed.
Explain here why you skipped testing in case you did so.
Have you created automated tests for your change? Explain here which validation activities you did manually and why so.
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

Subscription registration script is coming. I tested and it works for UBI 8.5-8.9 and 9.1-9.3.

Closing this one as it was split and addressed as separate PRs under the same ROX-20160.


NDArray Slicing Question

Hi,
I have not figured out something very simple regarding NDArrays. Let's say I have a 2D NDArray. How do I go about setting the (i,j) element to some value? I am using the Scala package.
Also, I have added support for the reshape operator in my version of the Scala package, but I notice that many operators that exist in the Python package are not yet supported in the Scala package. Do you plan to keep that package up to date or do you focus just on the Python package?
Thanks

you can't. If you want to operate on single elements convert it to numpy array and back.
Scala package is under development by @javelinjs. It's a work in progress. You are welcome to contribute to it.

@eaplatanios Could you make a PR for the reshape operator? We are willing to keep scala package up to date, we are just short of hands.

@piiswrong Thanks for response! Could you tell me what the main reason is for not supporting accessing single elements or slicing along different axes than 0? Is there some limitation, or is it because of prioritizing other features?
@javelinjs I'm pretty busy with NIPS and the camera-ready deadline for ICML (25th). I've been working on adding more features (e.g., bucketing support) to the Scala package, as well as more documentation. I'll make a PR once I'm done with deadlines. :)


update_dependencies.sh broken due to usage of deprecated command line args

Commit https://github.com/bazeltools/bazel-deps/commit/e589b9ac4b0c8ca312d735a6cb51ff62780529a9 got rid of a bunch of command line args like sha-file, target-file, disable-3rdparty-in-repo, etc. This is breaking the newest update_dependencies.sh script since it still uses some those.
The README also still contains example usage using them:
cd $BAZEL_DEPS
bazel run //:parse generate -- --repo-root "$MY_PROJ_DIR" --sha-file 3rdparty/workspace.bzl --deps dependencies.yaml --target-file 3rdparty/target_file.bzl --disable-3rdparty-in-repo

That was not intended.
Would be happy to add them back.
Also we should add a test to exercise them.
Would you be able to send a PR?

I would like to work on this

I believe #327 and #329 fix this. Please reopen if you have any issues.

Apologies for the delayed response, I would have gladly made a PR if I had seen this sooner. I have confirmed the fixes worked for me. Thank you @johnynek and @lakshmansai!


[hotfix][mysql] Exception capture is added to the ExecutorService

fix #1081

cc @leonardBang please review it


[Bug]: v2alpha session API, after login with TOTP all further requests fail with error: mfa required (AUTHZ-Kl3p0)

Preflight Checklist

[X] I could not find a solution in the documentation, the existing issues or discussions
[X] I have joined the ZITADEL chat

Environment
Self-hosted
Version
2.35.0
Describe the problem caused by this bug
Continuing from our recent discussion in https://github.com/zitadel/zitadel/pull/6362.
Prerequisites:

organization and user have MFA TOTP installed and activated
use new v2 session API

When session is started with 3 checks (login + password + TOTP) POST http://localhost:8080/v2alpha/sessions, session token is returned in response. Further requests with this token to User API like:
GET http://localhost:8080/auth/v1/users/me
Authorization: Bearer <session token here>
Accept: application/json

fail with error "mfa required (AUTHZ-Kl3p0)".
If I use OtpEmail instead of TOTP, everything works fine.
There is a comment left in code, that prevent TOTP from count in MFA check. I've tried to uncomment this piece of code, and TOTP started to work as expected: https://github.com/zitadel/zitadel/blob/f07e40c70b3947afd8d702448349d470595c476e/internal/authz/repository/eventsourcing/eventstore/token_verifier.go#L203-L208
To reproduce

Set up TOTP for organization and user
Call POST http://localhost:8080/v2alpha/sessions with login, password and TOTP, get session token in response
Call GET http://localhost:8080/auth/v1/users/me with header Authorization: Bearer <session token here>
You get error "mfa required (AUTHZ-Kl3p0)"

Screenshots
No response
Expected behavior
Requests to User API with TOTP-confirmed session token work fine, like with OtpEmail, for example
Operating System
Docker/Docker Compose/WSL
Relevant Configuration
Standard Docker compose file
Additional Context
No response

Same issue for internal authenticated user, seem checkbox Force MFA for local authenticated users on Organization Settings currently not work


update default version of API to v1 instead of v1beta3 [#101]

Updates defaults in code, as well as tests.
See issue #101.

@jonmoter Hi, and thanks a lot for your contribution.
Couple of comments from briefly looking at this:
all test json files names remained with "b3" name which stands for beta 3.
a. it makes sense to rename them as part of this change.
b. is the format kept 100% identical in k8s between beta3 to v1? so you can simply change the api version in the json and that would be enough? (no other fields added, fields renamed or moved between sections, etc.?)
c. in order to link properly the referenced github issue, please make sure to follow this convention in your commit comment:
https://help.github.com/articles/closing-issues-via-commit-messages/

cc @simon3z

Okay, I've renamed the files to remove the "_b3" in the filename.
I went over the Kubernetes docs describing changes from v1beta3 to v1, and found a few small changes that were required. I updated the tests to reflect that.
Let me know if you have any other feedback.

c. in order to link properly the referenced github issue, please make sure to follow this convention in your commit comment:
https://help.github.com/articles/closing-issues-via-commit-messages/

@jonmoter LGTM except for not linking commit msg

Would you like me to rebase with the commit message fixed?

On Sep 14, 2015, at 4:51 PM, abonas<EMAIL_ADDRESS>wrote:
c. in order to link properly the referenced github issue, please make sure to follow this convention in your commit comment:
https://help.github.com/articles/closing-issues-via-commit-messages/
@jonmoter LGTM except for not linking commit msg
—
Reply to this email directly or view it on GitHub.

You don't need to rebase for that, just fix the commit msg, then repush.
You can also squash the 3 commits, they all handle the same topic
On Sep 15, 2015 12:53 AM, "Jon Moter"<EMAIL_ADDRESS>wrote:

Would you like me to rebase with the commit message fixed?

On Sep 14, 2015, at 4:51 PM, abonas<EMAIL_ADDRESS>wrote:
c. in order to link properly the referenced github issue, please make
sure to follow this convention in your commit comment:
https://help.github.com/articles/closing-issues-via-commit-messages/
@jonmoter LGTM except for not linking commit msg
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/abonas/kubeclient/pull/120#issuecomment-140215932.

@abonas okay, i've squashed the commits and fixed the commit message.

@jonmoter great. I just noticed couple of method still refer to v3, please remove that too and it's good to be merged.

D'oh, good catch.  I'll fix and squash.

@abonas fixed.  Should be good to go.

:+1:


Mentioned wrong person as release contributor

Describe
The release page has some typo.
Steps to reproduce

Open 1.19.2-0.7.35r and 1.20.1-0.7.39r
You will see2. [#559] Fixed the creation of empty nbt tags when inserting items into the ME System - @thanks to michele-grifa!

Multiplayer?
Yes
Version
1.20.1-0.7.39r (Latest 1.20.1)
Minecraft, Forge and maybe other related mods versions
Chrome/124.0.6367.62
Screenshots or Videos
https://github.com/IntelligenceModding/AdvancedPeripherals/assets/91082411/f0c8c8d0-4817-4925-aea8-5c90ab901947
Crashlog/log
No response

I knew there was something I forgot


Adding a daily streak option and awarding bonus points for it.

Adding a feature where user can maintain daily streaks and will be awarded bonus points for it will keep the user engaged for a longer period of time.

I will start working on it .
On Tue, 8 Oct 2024 at 8:31 AM, YadavAkhileshh @.***>
wrote:

Assigned #154
https://github.com/YadavAkhileshh/Alien-Invasion-Defense/issues/154 to
@dikshabhartia22 https://github.com/dikshabhartia22.
—
Reply to this email directly, view it on GitHub
https://github.com/YadavAkhileshh/Alien-Invasion-Defense/issues/154#event-14549323405,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A53XIO34KDIWWSK6NZHMZPDZ2NDHHAVCNFSM6AAAAABPQXDK6SVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUGU2DSMZSGM2DANI
.
You are receiving this because you were assigned.Message ID:
<YadavAkhileshh/Alien-Invasion-Defense/issue/154/issue_event/14549323405@
github.com>

@YadavAkhileshh if this issue is open can i work on it


Release 2.4.2

I'm still getting a protected branch issue :P
remote: Resolving deltas: 100% (5/5), completed with 4 local objects.
remote: error: GH006: Protected branch update failed for refs/heads/master.
remote: error: Required status check "continuous-integration/travis-ci" is expected. At least one approved review is required by reviewers with write access.
To github.com:reactiflux/discord-irc.git
 ! [remote rejected] master -> master (protected branch hook declined)

Coverage remained the same at 99.254% when pulling 48fa96ebaf889237c244e93e7c6df805ffe79392 on release/2.4.2 into 552bba2c75b3df3c6b236edf7f7f425f4b0bf743 on master.

Coverage remained the same at 99.254% when pulling 48fa96ebaf889237c244e93e7c6df805ffe79392 on release/2.4.2 into 552bba2c75b3df3c6b236edf7f7f425f4b0bf743 on master.

(…oops that commit about prepublish is not meant to be pushed here; removed it)

I made you a repository admin - should be good to go for next time now!

(published 2.4.2 though - sorry about the delay)


Paket.template dependencies

Most of the fsharp projects have both net35 and net40 targets.
Then the end of the paket.lock looks something like this:
tags
    F#, fsharp, library
files
    ../bin/net45/MyLibrary.dll ==> lib/net45
    ../bin/net35/MyLibrary.dll ==> lib/net35
dependencies
    FSharp.Core ~> LOCKEDVERSION

If FSharp.Core is locked to v4 this can never work as it's not compatible with net35?
So is the best practice to:

Leave FSharp.Core dependency hidden and hope people has it even if it's not anymore in the default VS-installation?
Not use paket.template but nuget.nuspec? But ProjectScaffold already made the template, why it guides to this non-sustainable-way?
Or is there any solution currently?

I would like to have possibility create something like the dependency tree in this package shows: https://www.nuget.org/packages/LinqKit/

Why do you look the version of fsharp.core?
Am 30.08.2016 1:04 vorm. schrieb "Tuomas Hietanen"<EMAIL_ADDRESS>
:

Most of the fsharp projects have both net35 and net40 targets.
Then the end of the paket.lock looks something like this:
tags
F#, fsharp, library
files
../bin/net45/MyLibrary.dll ==> lib/net45
../bin/net35/MyLibrary.dll ==> lib/net35
dependencies
FSharp.Core ~> LOCKEDVERSION
If FSharp.Core is locked to v4 this can never work as it's not compatible
with net35?
So is the best practice to:

Leave FSharp.Core dependency hidden and hope people has it even if
it's not anymore in the default VS-installation?
Not use paket.template but nuget.nuspec? But ProjectScaffold already
made the template, why it guides to this non-sustainable-way?
Or is there any solution currently?

I would like to have possibility create something like the dependency tree
in this package shows: https://www.nuget.org/packages/LinqKit/
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/1899, or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNHaPJrA3krlsN12sVj609QJ2LT1pks5qk2WRgaJpZM4Jv_Is
.

The v4 FSharp.Core nuget package has a net20 folder so it should be compatible with net35.
So it shouldn't be a problem at all?

Ok! So if package contains binary dll for net45, net35 and netcore it should work automatically?
Good.

Yep
Am 31.08.2016 1:50 nachm. schrieb "Tuomas Hietanen" <
<EMAIL_ADDRESS>
Ok! So if package
https://www.nuget.org/packages/Linq.Expression.Optimizer contains
binary dll for net45, net35 and netcore it should work automatically?
Good.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/1899#issuecomment-243740459,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNE_8HOdcLpTQ91-6od5Nd31GPkUDks5qlWp0gaJpZM4Jv_Is
.


Allow requesting full screen during start-up in HTML5 platform

Attempt to enter full screen mode if the display/window/size/fullscreen project setting is enabled. This will work only if the game is started from a JavaScript input event handler.
Documentation about this in godotengine/godot-docs#2155

Cherry-picking also depends on previous refactoring PRs that I haven't backported yet, leaving for later.


Translated facts bug fix olé

Fixes #5572
What's this PR do?
To minimize the potential for breaking lots of campaign related code, took a more subtle approach to fixing the translation bug related to Facts. The EntityDrupalWrapper class was causing problems with grabbing the correct facts related to regional content (translated, but as separate unique nodes and not nodes with translations as part of the node object).
Instead, created a separate function to handle grabbing that data from the node, without interrupting how the rest of the campaign node data is loaded when constructing the campaign object. It allows for grabbing content if there are translations and ones specific to the language defined by the url path (/us/ or /mx/ etc) or just grabbing the available content if no language specific options available, but there is definitely content in the field.
Where should the reviewer start?
Probably in the dosomething_campaign.helpers.inc file at line 183 with the @TODO explanation and then follow the bread crumb trail of code... :bread:
How should this be manually tested?
We'll test on Thor which has a bunch of language related content to make testing easier.
Any background context you want to provide?
To help with mitigating bugs and not break how the dosomething_campaign_load() function loads and creates the campaign standard object, the aim with the custom function was to set it up in a way that it would return data back to the dosomething_campaign_load() function in the same format/structure that the EntityDrupalWrapper would have, because that's the structure the rest of the code is relying on. Deviating from that structure could end up causing more bugs that we'd need thorough testing to actually find and then iron out. So went with the path of least resistance; but also why some of the code may see a little odd in places, in the effort to accommodate the above intentions! :earth_americas:
What are the relevant tickets?
#5572
Screenshots (if appropriate)
US specific content at /us/node/1283:

MX specific content at /mx/node/1283:

MX specific content, but none entered in the admin /mx/node/1321:

@angaither
cc: @mikefantini @namimody

just some minor comments, but not needed for merge :+1:
olé
:tada: :dancer:


Updating packages Apr 11

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+4.2%) to 80.714%

Totals

Change from base Build<PHONE_NUMBER>:
4.2%

Covered Lines:
2569

Relevant Lines:
3136

💛  - Coveralls


[Feature]: Support backlinks

Requirement
As a Jaeger user
I want to be able to see links from other spans/traces pointing to the currently opened span
so that I could discover them and browse through them.
Problem
Spans can have links to other spans. Jaeger understands naturally parent/child links and groups spans into a tree according to their CHILD_OF links. It also shows other links of a span in the span's detail, however, it does not show for a span which spans links to the span (except for the CHILD_OF links).
Technically, the nature of the links is the same, e.g., FOLLOWS_FROM behaves like CHILD_OF, so I suppose Jaeger already has the mechanism to discover spans linking to a particular span, but it does not apply it for other link kinds. As the result it is not possible to discover such spans. This diminishes the usefulness of other link kinds.
Proposal
Extend Jaeger so that a span shows spans that link to it, so that CHILD_OF link kind is not the privileged one that can be traversed easily.
Open questions
No response

It's doable but one caveat is going to be that only back links local to this trace can be determined. If you're interested in backlinks from other traces it would require significant support from the backend storage.

I'm not sure either. This feature request was originally triggered by a warning about clock skew that Jaeger issues for our traces. However, the issue is not clock skew, but children spans outliving their parent. This was annoying, so we tried to find another way that would match perhaps better the flow logic and we used for a couple of experiments links between otherwise unrelated spans/traces. But because the links in the UI are not displayed in an actually usable and helpful way, we gave up and stayed with the suboptimal-but-working solution that produces those annoying warnings.
Btw. the warnings might be another target for improvement. I raised a question to OpenTelemetry specification repo about a related topic and described our use case and the way how we tried to solve it. According to them we are likely doing the right thing, in principle at least. But if Jaeger shows the warnings, then either our execution has some flawed detail, which we do not see for our ignorance, or Jaeger could be improved not to show misleading warnings.
If you are interested in that case, see the issue discussion. While I can live with the current outcome, I think it could and should be a bit better, so I'd be grateful for an improvement or knowing what we do wrong.

The warning is: clock skew adjustment disabled; not applying calculated delta of… The time varies, depending on the actual state. Because we run the processing in background asynchronously, as described in the linked issue discussion, it can be minutes, or in theory even longer, if the job rests in the queue for a long time.
Regarding the links, OpenTelemetry allows making links and we tried this way too – it appeared like the better approach or like something that is intended for a use case which we have. And Jaeger did recognize the links! They appeared in the span's detail and they declared FOLLOWS_FROM type. So far, so good.
The problem with the links is, as I described in the original post, that these links are basically useless for browsing the span tree. They are hidden deep in span details, they are not visualized in any way and there are no backlinks from the linked span – so a span may have FOLLOW_FROM link mentioned, but the target span (which is its "parent" span) does not mention it at all.
With the explicit links, when we want to analyze a particular trace and start from the easy-to-find REST request, we are lost at the moment when we process the request and launch the background actions. So we gave up – we make just regular children and we had to accept the warning as something annoying, misleading, but inevitable. Not good, but usable at least.
If Jaeger provided proper backlinks and included the linked traces/spans somehow in the visualized flow, it would be great and the links would be actually usable. From the user point of view it might look like the linked traces are children of the followed-from span, just with some badge or symbol that indicates that they are not direct children, but rather linked traces.
Anyway, I just wanted to report the issue that we have and ask for improving the support for such use cases, since I see here a large gap regarding links.


Add CLA Assistant GitHub workflow

Per https://github.com/signalfx/gdi-specification/pull/269

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (0e3ecb5) 97.68% compared to head (e4fad64) 97.10%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #190      +/-   ##
==========================================
- Coverage   97.68%   97.10%   -0.58%
==========================================
  Files          27       27
  Lines         519      519
==========================================
- Hits          507      504       -3
- Misses         12       15       +3

see 2 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Create python version of spacewalk-debian-sync.pl

Currently using https://github.com/stevemeier/spacewalk-debian-sync which is a Perl script. As all other scripts are now done in Python. Refactoring the sync script to Python makes sense and would get rid of any Perl dependencies and likely also improve performance

done https://github.com/philicious/spacewalk-scripts/commit/83a92233926311a75ccfdc9f2d4522b1fc084bfd
left to do is adding it to this cookbook


The following code, that "fakes" a VCS, does not fix the issue.

The following code, that "fakes" a VCS, does not fix the issue. Ort probably can not handle remotes that are a local path?
rm -rf .git .hg # ...
git init
git add .
git config user.email<EMAIL_ADDRESS>git config user.name "fakeVCS.sh"
git commit -am"fakeVCS.sh"
git remote add origin "$(pwd)"

It still fails with
07:59:13.335 [main] ERROR org.ossreviewtoolkit.cli.commands.ReporterCommand - Could not create 'EvaluatedModel' report in 0s: IllegalArgumentException: The VcsInfo(type=Git, url=file:///outputs/src/, revision=ed8a0900c56408efe8cbdcf1bcdfdee69d81f600, path=my-app) of project 'Maven:com.mycompany.app:my-app:1.0-SNAPSHOT' cannot be found in Repository(vcs=VcsInfo(type=Git, url=/outputs/src, revision=ed8a0900c56408efe8cbdcf1bcdfdee69d81f600, path=), vcsProcessed=VcsInfo(type=Git, url=file:///outputs/src/, revision=ed8a0900c56408efe8cbdcf1bcdfdee69d81f600, path=),
nestedRepositories={}, config=RepositoryConfiguration(excludes=Excludes(paths=[], scopes=[]), resolutions=Resolutions(issues=[], ruleViolations=[], vulnerabilities=[]), curations=Curations(packages=[], licenseFindings=[]), packageConfigurations=[], licenseChoices=LicenseChoices(repositoryLicenseChoices=[], packageLicenseChoices=[]))).

Originally posted by @maxhbr in https://github.com/oss-review-toolkit/ort/issues/2579#issuecomment-913445752

Note the we also have issue https://github.com/oss-review-toolkit/ort/issues/2900 which is about Git remotes that point to local (Windows) paths.

my next try to fake a vcs, so that ORT is "happy":
#!/usr/bin/env bash
set -euo pipefail

target="$1"

if [[ -d "$target/.git" || -d "$target/.hg" ]]; then
    echo "already under version control"
    exit 0
fi

tmpdir="$(mktemp -d)"
git clone https://github.com/maxhbr/empty-fake-repo-for-ort "$tmpdir"
mv "$tmpdir/.git" "$target/.git"
rm -rf "$tmpdir"

@sschuberth I still see that issue frequently pooping up and creating a lot of friction in the adoption of ORT.

Analyzing / scanning local directories that are not a working copy from a VCS still isn't a use-case that we endorse / officially support. But in order to properly track the demand I'll close this in favor of https://github.com/oss-review-toolkit/ort/issues/2896 to consolidate there.


[JOSS Review] Explanation of back calculators

Re: JOSS Review
Related to this issue, it would be nice if the website was updated to provide a little more information regarding the different back calculators that you support. At present there are just acronyms, with little explanation as to why or when you would choose to use one, or what they calculate. (I appreciate that those in the field might know this, but new users might not.) Although brief, this information is provided in the paper so you could simply reproduce it on the website too.
It might be nice to combine the formatting for input/output with the usage section so for each back calculator there is an explanation of what it does, what input is required, and what output it generates. At present you have to jump back-and-forth between sections.

Thanks Lester for getting back to this. That's a great idea I have overhauled the usage section of the documentation so it's a bit more clear


chore(tsconfig): do not preserve const enums

@KFlash As discussed, simply disable const enum preservation.
This brings the minified bundle from 86kb back down to 70kb.
Does this fix https://github.com/cherow/cherow/issues/255 or is there anything else you'd like to address?

This looks good to me. I'm still worried about that mess, but it's all good. You merge?

@KFlash You mean this right?

 And I can't figure out why we should need 2x sub folders in the dist folder? Different from before. But it's me being picky!

It's the same structure as the src folder. Let me turn the question around: why should we need to flatten it?

Just let it stay as it is as long as we not get problems
On Wed, Dec 19, 2018, 11:16 AM Fred Kleuver<EMAIL_ADDRESS>wrote:

It's the same structure as the src folder. Let me turn the question
around: why should we need to flatten it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cherow/cherow/pull/257#issuecomment-448455108, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AeYSDqfMmE36mGZ76VI9lewJ9avriGN0ks5u6a9-gaJpZM4ZZZix
.


PIP-25: Token based authentication

Motivation
This is the first PR for the implementation of token based authentication
Modifications

Added client & broker auth providers
Added CLI tool to manage tokens
Unit tests

TODOs (in separate PRs):

Integration tests
Documentation
C++ (and related) client plugin implementation

@ivankelly @maskit Please take another look

Also, it would be really good to have an integration test in this PR a) to validate that both the PSK and private/public paths work

The 2 paths are tested in the auth provider unit test. It's not the same as integration test but it covers all the plugin code, from getting variables from ServerConfiguration to validating the tokens in both cases.

@maskit @ivankelly please take another look. I have moved to use standard data: uri and to not force base64 encodings.

The "data:" parser you wrote would be a duplicated code because it's already implemented in in-house URL class I wrote.  I'm not going to make a change request because I don't want to delay this feature, but either of the parsers should be removed eventually to eliminate duplicate code.
Also, because environment variables are general enough as data sources, I think "env:" parser should be added as a URLStreamHandler so that other places can use it too via the in-house URL class.
As for "token:", I'm ok with having it in this plugin as a special case. But if you renamed it to "raw:", it could be used on other places.

If we added all the parsers as URLStreamHandlers, users would be able to read any of them like this without taking care of the scheme.
import org.apache.pulsar.client.api.url.URL;
import java.net.URLConnection;

URLConnection urlConnection = new URL(keyConfUrl).openConnection();
String keyData = CharStreams.toString(new InputStreamReader((InputStream) urlConnection.getContent()));

Since openConnection() is defined in java.net.URL. The only thing users need to do is changing import line. Then they can use it as the same as java.net.URL.
https://docs.oracle.com/javase/8/docs/api/java/net/URL.html#openConnection--
I don't remember why I didn't implement URL#getContent() (it's a shorthand for openConnection().getContent()) but if we did, it would be much easier.

@maskit I got now what you meant. I had forgot about org.apache.pulsar.client.api.url.URL. There is just one problem with that class in that it assumes the data will be decoded into a String, though in this cases it would need to be a byte[] since the key cannot be converted into String.
Will continue this in a different PR since this will need to change on Athenz plugin too.

run integration tests

@merlimat

There is just one problem with that class in that it assumes the data will be decoded into a String,

getContent() returns a InputStream so you should be able to read raw data.

Will continue this in a different PR since this will need to change on Athenz plugin too.

Sure, I think I can help you out on this since Pulsar seems like compilable with Java 11 :)

When you do the integration tests, we need a test for proxy (it seems to be missing in this and c++ change)

When you do the integration tests, we need a test for proxy (it seems to be missing in this and c++ change)

@ivankelly the test is actually going trough the proxy already since pulsarCluster.getPlainTextServiceUrl() returns the proxy container address

@maskit @ivankelly Few more changes:

Fixed integration tests (test is also going against proxy)
Added CLI tools to validate a token against a key
Removed env:  as it's not working properly.

run cpp tests


Cannot get useQuery to work with Storybook

I am using storybook to develop my react components in isolation. I am running into an issue with using react-query (useQuery) with storybook. Implementing the component in the app, I get no errors and things work as expected.
async function searchStationsByCallLetter(_, param) {
  const request = {
    callLettersView: param,
  };

  const { data } = await ApiService().makeApiCall(API_ENDPOINTS.BROADCAST_LINEUP.SEARCH_STATIONS, request, HTTP_POST);
  return data;
}

export default function useSearchStationsByCallLetter(inputValue = '') {
  return useQuery(['searchStationsByCallLetter', inputValue], searchStationsByCallLetter);

I have a custom react hook useSearchStationsByCallLetter that returns the useQuery hook. React-query should be using searchStationsByCallLetter as the api call. ApiService and makeApiCall are a custom axios setup.
The error I get in storybook is:
TypeError: Object(...) is not a function
    at _callee$ (AddLineupHooks.js:8)
    at tryCatch (runtime.js:63)
    at Generator.invoke [as _invoke] (runtime.js:293)
    at Generator.next (runtime.js:118)
    at asyncGeneratorStep (asyncToGenerator.js:3)
    at _next (asyncToGenerator.js:25)
    at asyncToGenerator.js:32
    at new Promise (<anonymous>)
    at Object.<anonymous> (asyncToGenerator.js:21)
    at Object.searchStationsByCallLetter [as queryFn] (AddLineupHooks.js:8)

Desktop (please complete the following information):

OS: Mac
Browser: chrome
Version: latests

This is an issue with Storybook, not react-query


Add namespace value to allow override of release namespace in enterprise-operator chart

As discussed with @priyolahiri opening this issue here on Github.
Due to our chart/subchart and Argo CD structure, we deploy the enterprise-operator chart in the default namespace while the resources should be deployed to a different namespace (e.g. mongodb-management). We cannot change the release namespace in our setup but would still like to use your the official enterprise-operator chart provided by MongoDB. This feature request is a proposal to implement a backwards compatible way to satisfy our needs.
Request:

Add an optional  value "namespace" to the helm chart which will act as override for the Namespace.
If the value is set, override the namespace in all enterprise-operator templates (e.g. https://github.com/mongodb/helm-charts/blob/main/charts/enterprise-operator/templates/operator.yaml#L6 ).

Required change to metadata.namespace of the template manifests:
  namespace: {{- if .Values.namespace }}{{ .Values.namespace }}{{ else }}{{ .Release.Namespace }}{{ end }}

Hello @jstudler
Thanks for reporting this issue. Let me know if I understand this correctly.
You want to install the Operator in namespace/default while creating databases in a different namespace namespace/mongodb-management, and having the Operator watch over those resources in mongodb-management.
If that's the scenario you are looking after, then you can do this by passing the operator.watchNamespace configuration to helm install like
helm install enterprise-operator mongodb/enterprise-operator --namespace default --set operator.watchNamespace="mongodb-management"

Helm install install the Operator Pod in the default namespace, and will configure it to watch over resources in mongodb-management namespace.
Let me know if this works for you.

@jstudler I am thinking whether you would want to maintain your own fork of the helm charts repo for stuff like this.

@priyolahiri we would rather not. We're exactly trying to avoid this. Do you think implementing this in your helm chart would be a problem? I think it can be implemented without breaking the existing behaviour.

I've also recently stumbled upon this, same issue if you try to deploy the operator into another namespace with a Flux CD setup. What @jstudler suggests would fix the issue and, as far as I can see, would not break anything on existing setups :+1:
@priyolahiri maintaing a fork for something that most of the "enterprise" charts already bring to the table seems overkill, especially since this seems to be quite a straight forward fix...

@jstudler and @acidicX putting this up in the dev queue. will have an update on this soon.

Hi @jstudler and @acidicX ! While I agree with you the solution to provide an operator namespace value wouldn't affect the functionality provided by the chart currently, this is not a common best practice (e.g.):

https://github.com/cert-manager/cert-manager/tree/master/deploy/charts/cert-manager
https://github.com/elastic/helm-charts/tree/main/elasticsearch

If I understand your use-case correctly, it is the same one described in this issue: https://github.com/helm/helm/issues/5358 and the functionality is not supported in vanilla helm. Are tools like https://github.com/Praqma/helmsman and https://github.com/helmfile/helmfile easy to fit in your workflow instead?
Thank you for your patience with this. While we have a solution similar to the Jenkins helm chart https://github.com/jenkinsci/helm-charts/blob/main/charts/jenkins/values.yaml , we'd like to better understand your usecase to make sure this wouldn't make the chart responsible for too many configuration variants.

Hi @mircea-cosbuc
I agree that usually helm charts do not have namespace override values. But some of them do have them as mentioned in the helm github issue you've linked.
Falling back to tools like helmsman or helmfile is not the solution we aim for as we'd like to stick to native tooling for our Argo CD stack. Therefore, our fallback at the time being is that we have a separate helm chart for the enterprise-operator.

That makes sense, thank you for the follow-up!

Hi @jstudler , the latest release of the enterprise operator allows for overriding the operator namespace using the operator.namespace value override. Closing the issue for now, please follow up if encountering any issues.


tour for rna-seq-viz-with-cummerbund

@bebatut added tour

@bagnacan could you test this tour? Thanks :smile:

Thanks @polkhe!


Update JSDoc to 3.5.5

This fixes a problem in JSDoc's fs module that arose with Node 8.5.0.  JSDoc fixed this in 3.5.5, so all this does is bump the version that grunt-jsdoc uses.

Thanks for the PR. version 2.2.0 should work now.


ci: Release PowerShell completion script

Resolves #952
_lsd.ps1 is built but not released. This PR releases it.

Codecov Report

Merging #953 (5653bb7) into master (6f8e095) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@           Coverage Diff           @@
##           master     #953   +/-   ##
=======================================
  Coverage   85.76%   85.76%
=======================================
  Files          51       51
  Lines        5001     5001
=======================================
  Hits         4289     4289
  Misses        712      712

:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

thanks for the contribution!


Blur issue

Hi,
I try to customize the background color of the loader and even with setDefaultStyle custom and setForegroundColor i can't get rid of the blur
SVProgressHUD.setDefaultStyle(.custom)
SVProgressHUD.setCornerRadius(5.0)
SVProgressHUD.setForegroundColor(UIColor.green)
SVProgressHUD.setBackgroundLayerColor(UIColor.green.withAlphaComponent(0.4))
SVProgressHUD.setBackgroundColor(UIColor.red)
SVProgressHUD.show()

I run the 2.1.1 on iOS 10 with Swift 3.0
Thank you very much for the answer

Can you verify if the issue is still active?


Review current Error Reporting implementation

Triggered for all relevant commands?
Testable everywhere?
Documented (both publicly and internally)?

Currently only triggered for migrate and introspect.

Implementation:

https://github.com/prisma/prisma/blob/master/src/packages/sdk/src/sendPanic.ts
https://github.com/prisma/prisma/blob/master/src/packages/migrate/src/utils/handlePanic.ts


How to reproduce the results of Table 4 in this paper?

I'd like to reproduce the results of Table 4 in this paper, but can't find any detail instructions. I'll appreciate that if anyone could tell me how to do.

I'd like to reproduce the results of Table 4 in this paper, but can't find any detail instructions. I'll appreciate that if anyone could tell me how to do.

For the dataset introduced in Table 4, we use the original dataset. Run the code with the original dataset to get the results. For GPT-3.5 or GPT-4 models, please note that the probability tracking algorithm will introduce additional costs.
Sorry for the untimely reply!


No provider for FileHolder!

I added this component to project and when used, getting this error
ERROR Error: No provider for FileHolder!
    at injectionError (eval at ./node_modules/<EMAIL_ADDRESS>(vendor.bundle.js:39), <anonymous>:1373:90)
    at noProviderError (eval at ./node_modules/<EMAIL_ADDRESS>(vendor.bundle.js:39), <anonymous>:1411:12)
    at ReflectiveInjector_._throwOrNull (eval at ./node_modules/<EMAIL_ADDRESS>(vendor.bundle.js:39), <anonymous>:2853:19)
    at ReflectiveInjector_._getByKeyDefault (eval at ./node_modules/<EMAIL_ADDRESS>(vendor.bundle.js:39), <anonymous>:2892:25)

Please show your usage. The error indicates you're trying to inject FileHolder, this won't work as it's not a dependency that can be injected, it's just a data class that you can instantiate or leave it up to the component.

yeah, you are right,
I followed the readme first.
added ImageUploadModule to app.module.ts and then ImageUploadModule.forRoot() to import of @NgModule.
added  tag to component.html..
then I got this error:
SchoolDetailsDialogComponent.html:1 ERROR Error: ExpressionChangedAfterItHasBeenCheckedError: Expression has changed after it was checked. Previous value: 'undefined'. Current value: 'nutsAndScrewsUiGateWayApp.schoolDetails.home.createOrEditLabel'. It seems like the view has been created after its parent and its children have been dirty checked. Has it been created in a change detection hook ?
    at viewDebugError (eval at ./node_modules/<EMAIL_ADDRESS>(vendor.bundle.js:39), <anonymous>:8638:32)

Then I looked at the demo folder of the repository and tried to follow the code in the examples there

How can I show, please let me know..
I added lib to my project like this : npm install angular2-image-upload --save
I have jhipster app. in my app.module.ts ,
I added this two lines :
import { ImageUploadModule } from 'angular2-image-upload';
and
ImageUploadModule.forRoot(),
and in one of the component.html ,
I added a
<image-upload [url]="'my-url.com'"
[headers]="[
{name: 'X-XSRF-TOKEN', value: csrfService.getCSRF()}]
]">
thats all!

The module usage looks fine, but the headers are incorrect, they should be:
[headers]="[
  {header: 'X-XSRF-TOKEN', value: csrfService.getCSRF()}]
]"

Other than that I see nothing wrong, if you still have trouble I don't think the issue is this library.

Hi, that is a Angular exception, maybe you're updating a parent component variable in a child component, if that's the case, consider move your variable to a service instead.


repost: release-v22.12.0

https://github.com/nodejs/bluesky/pull/12 failed likely because the PR branch was deleted before the workflow is complete. So try it again.

I'm going to just land it now, since the original PR is approved already.


chore: Remove 3.4.2.diff

Which issue does this PR close?

Closes #.
Rationale for this change

What changes are included in this PR?

How are these changes tested?

Thank you merged @viirya


automatically stop on multiple verification fail?

Is it possible to automatically stop the bot on multiple verification fail (for example, 3 tries of the verification task)
I am using 2Captcha with a valid user key and positive balance, however, 2 of my farm accounts were still banned today.  If the bot were stopped, the game would just stay at the verification page until I manually solve it, this would avoid the risk of getting banned.

I am using Bluestack 5 with 1280X720. I am not sure if the 2captcha is working.  It has so far not passed any verification tests.  I have also tried one other commenter's solution by changing the resolution to 720X1280.  still unable to pass.
In my 2Captcha, I turned on the "100% pass" option.  not working still...  before a solution is in, automatically turn off the bot would be crucial.  I don't want my main to get banned as well.

I'm realize that too, some time 2captcha continue fail to solve the problem, and that cause my account being ban too.
For next version, I will add that feature.

thank you for considering the suggestion.  Do you have an ETA for the next version?

@Dylan-Zheng  I guess that is why I banned as well :(
Also after the 1st time it is much more sensitive
Need to understand how to pass verification almost 100%

I think as soon as verification is detected, if the program would simply pause all tasks that would be really nice.

I have 4 accounts banned because of this :) It would be great for me if he closed the game when the confirmation screen came up @Dylan-Zheng  I'm at the computer all day and the game works by itself at the bottom, sometimes I can't look at it because of the intensity. The faster you add this feature, the sooner you will save us from being banned king :)


Cannot find 'swiper/css' in node_modules using Vue Cli

Check that this is really a bug

[X] I confirm

Reproduction link
https://github.com/rsteinbakk/swiperjs-vuecli-notworking-issuereport
Bug description
First of all, thanks for creating a great slider component! :)
Got Swiper JS working in/with Vite (https://vitejs.dev/), so it seems like a specific problem that comes up when using Vue Cli?
Got an issue using Swiper JS ( https://swiperjs.com/vue ) in Vue Cli (in both Vue 3 and Vue 2, https://cli.vuejs.org/). It seems like it cant find the css file in the Vue Cli setup:
Error message from Vue Cli:
This dependency was not found:

swiper/css in ./node_modules/cache-loader/dist/cjs.js??ref--12-0!./node_modules/babel-loader/lib!./node_modules/cache-loader/dist/cjs.js??ref--0-0!./node_modules/vue-loader-v16/dist??ref--0-1!./src/components/ImageSwiper.vue?vue&type=script&lang=js

Tried it on two different computers (both windows 10), with different versions of Node Js.
Would be great if someone could look at this issue :)
Expected Behavior
No response
Actual Behavior
No response
Swiper version
6.8.4
Platform/Target and Browser Versions
Not a browser-issue.
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

https://github.com/nolimits4web/swiper/issues/4871

Thank you so much Vladimir! Have a great day :)

hava you resolve this question ?
i hava found a road to it like this:
import { Swiper, SwiperSlide } from 'swiper/vue/swiper-vue.js';
import 'swiper/swiper.min.css';
so it works now.
well  i hava not found a better method to resolve it .

Have someone solved how to setup vue-cli to work with version 7? All issues leads to that pinned issue, but I can't find any solution how to actually migrate

hava you resolve this question ?
i hava found a road to it like this:
import { Swiper, SwiperSlide } from 'swiper/vue/swiper-vue.js';
import 'swiper/swiper.min.css';
so it works now.
well  i hava not found a better method to resolve it .

Having the same issue, the doc says to import 'swiper/css'; but it can't be found in node_modules.

Webpack4 does not support the exports  field in the package.json
vue-cli4 uses webpack4,you need to update to vue-cli5 which uses webpack5.


_run_with_pytest

When I run tests in futurecoder, an obscure warning appears in the stderr captured by the tests. Not captured by pytest and shown in the terminal, but captured by the tests and affecting the results. The reason is this:
https://github.com/friendly-traceback/friendly-traceback/blob/dc32fc13ba2f9bc86b5b9bff625b21a5e2a3acf4/friendly_traceback/__init__.py#L53-L58
Please replace _run_with_pytest with something that is only turned on explicitly by your own tests, so that it doesn't affect other code using friendly-traceback and pytest at the same time. For example, you could set an environment variable. Alternatively, here's what I do in executing: https://github.com/alexmojaki/executing/blob/72a9ab419e46a7ea1a4ff950e4f5feeb98048e9a/tests/utils.py#L11

Ok, I will try to implement something like this.

New version uploaded to Pypi, which should not have this problem.

_run_with_pytest within about_warnings.py still seems like it could theoretically be a problem, since other projects like futurecoder may see unintentionally different behaviour between tests and deployment. But now that I'm actually looking at about_warnings I'm becoming more worried about the fact that the general behaviour of warnings is patched globally in ways I don't understand at all. It's a theoretical concern, I haven't encountered anything in practice yet, but that still seems like something that should be optional.

By default, warnings are ignored when importing friendly_traceback; this is what I assumed you'd want for futurecoder (and similarly for HackInScience). (At least, that's what I meant to do, and initially forgot about the effect of simply considering pytest.)
Since they are ignored, warning.showwarning has no effect.  When warnings are not ignored, with cPython there is a default version of warning.showwarning which gives the following:
# cPython
>>> import warnings
>>> warnings.simplefilter("always")
>>> [1, 2, 3]()
<stdin>:1: SyntaxWarning: 'list' object is not callable; perhaps you missed a comma?
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: 'list' object is not callable

By default, when using friendly_traceback in interactive mode (i.e with its own console), I activate warning, and show something similar to what cPython does, but omitting the filename and linenumber:
> python -m friendly_traceback --python_prompt

Friendly-traceback version 0.7.5. [Python version: 3.9.10]
Type 'Friendly' for help on special functions/methods.

>>> [1, 2, 3]()
`SyntaxWarning`: 'list' object is not callable; perhaps you missed a comma?

Traceback (most recent call last):
  File "<friendly-console:1>", line 1, in <module>
    [1, 2, 3]()
TypeError: 'list' object is not callable

When running new tests, I want to see if some new warnings arise which I have not considered before. If they do, I plan to quickly add them so that "cause" will be known, and the warnings will never be displayed - which has the same visible effect for the end user as having the cPython default.
I also activate the warnings when using friendly (as opposed to simply using friendly_traceback).
So, the only problem would be if someone wants to process the warnings themselves, while using pytest and friendly_traceback at the same time since friendly_traceback's processing might take precedence depending on the order of imports.  If this creates a problem with another library, I will try to find a solution so that both can coexist.
warnings processed by friendly_traceback will not interrupt a program, but they will still be available for inspection using what(), where(), why(), just like for tracebacks. However, this has to be done in an interactive settings, since warnings are otherwise just silently processed.

By default, warnings are ignored when importing friendly_traceback; this is what I assumed you'd want for futurecoder

I'm curious about the reasoning here. Do you find that warnings are generally more harmful than helpful for beginners? That certainly sounds plausible to me but it's not clear.
I'm also worried that users might try directly something like warnings.warn for the sake of learning/experimentation with the warnings module, and in that case I'd want the behaviour to be the same as the standard.

Ok, I've hidden the global reassignment behind a function call. This should now be completely invisible to futurecoder and other libraries using friendly_traceback, unless they make the required function call.


add support for Symfony 2.7

The current mention for integrating with the Symfony 2 framework suggests
https://github.com/barelon/CgKintBundle
with seems abandoned/outdated, forcing Kint into version raveren/kint ~0.9 leading to its own set of issues (eg. the strpos() error).
Two notable areas I cannot seem to get my project to run at all, or at least cannot use Kint, is:

in TWIG templates
Running my Symfony 2.7 project from the CLI or cron jobs (extending ContainerAwareCommand ).  There, for the life of me, I cannot seem to successfully run Kint::enabled(Kint::CLI);  no matter what namespace I use or require (eg. require dirname($_SERVER["SCRIPT_FILENAME"]) . '/../vendor/raveren/kint/Kint.class.php';  )  nor what I put in composer.json (currently only "raveren/kint": "dev-master" ).  Nothing relevant is in AppKernel.php as there is no bundle to register for Kint.

Twig templates: Out of scope: This can be implemented by a third (or first) party plugin in 2.x-dev. Please test and give feedback, thanks!

I cannot seem to successfully run Kint::enabled(Kint::CLI); no matter what namespace I use or require

Kint is in the global namespace to support older PHP versions. It should work with \Kint::enabled(). Could you test and report back please?


feat: support skip_validate in mempool

This change is

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.36%. Comparing base (152b420) to head (5aef0b4).

Additional details and impacted files
@@                  Coverage Diff                   @@
##           yael/skip_validate     #376      +/-   ##
======================================================
+ Coverage               82.16%   82.36%   +0.19%
======================================================
  Files                      32       32
  Lines                    1604     1622      +18
  Branches                 1604     1622      +18
======================================================
+ Hits                     1318     1336      +18
  Misses                    219      219
  Partials                   67       67

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Moving this feature to the next MS; my team will take it.


Tests: Coveralls and Simplecov disagree.

Coveralls reports 56% coverage.
Simplecov reports 97% coverage.
Why the discrepancy? I suspect the Simplecov figure is correct, and there is something wrong in the Coveralls configuration but have not done enough research yet to be certain of this.

We run - CI=1 bundle exec rake test:each for CI, and my guess is that test:each doesn't get a good aggregate.
I'm submitting a PR with a suggested change.

OK, we'll see if the coverage on that PR looks any better.


Added Polar Flow Sync v2.6.4

Added Polar Flow Sync v2.6.4  http://support.polar.com/uk-en/support/flowsync

Hey vitorgalvao, thanks for reviewing. Have updated based on your comments.
pkgutil doesn't seem to stop the processes, or remove the app, hence the additional quit and delete.


Formatted content text not displaying under Browse OTU

This is tangential to the previous issue and concerns the display of content un the Browse OTU. Again, one potential bug, one suggestion for improvement.
Potential bug
Using the Content Editor, I added formatting (italics and hyperlinks) to the text block. It appears correctly in the Content Editor preview. (imbelis and sierrae italicized and as a link)

But under the Contents in Browse OTU, the text only displays with the formatting code.

In case this is a browser issue, I am using Chrome.
Suggested Improvement
In the same way you can drag and changes the display order of the various attributes in Browse OTU like common names, biological association, and distribution, can this be done for topics under "Contents"?

Good catch. @jlpereira we'll just have to filter it through MD render.  I can do it SS if we need, however we're also goning to be extending that markdown slightly to let OTU IDs be much simpler to add than what @kandakoj is doing.  Maybe it's time to tackle that.

Good catch. @jlpereira we'll just have to filter it through MD render. I can do it SS if we need, however we're also goning to be extending that markdown slightly to let OTU IDs be much simpler to add than what @kandakoj is doing. Maybe it's time to tackle that.

What you mean exactly with extending markdown? a special tag or something like that?
@kandakoj we make some changes on Browse OTU, now you will see the markdown render correctly on the next update.

Yes- things I mentioned before.  Will open new issue.


Attached image credit to header image on profile headers

The image credit should tie to the header image in the same proportions as the other article pages.

This is likely related to this CSS, or at least this selector across the other breakpoints. Keep an eye out for profile vs. non-profile pages.


PEP 596 compatibility (Python3.9)

The release version of Python3.9 is expected one week from today (https://www.python.org/dev/peps/pep-0596/)
Modifying our existing code for compatibility and integrating python3.9 into the Travis-CI build tests will require some effort. Given that compatibility with earlier Python versions must be maintained, we won't be able to use new features (until Python3.6 is deprecated in 2021). For those interested, here are the new goodies:
PEP 584, Add Union Operators To dict
PEP 585, Type Hinting Generics In Standard Collections
PEP 593, Flexible function and variable annotations
PEP 614, Relaxing Grammar Restrictions On Decorators
PEP 615, Support for the IANA Time Zone Database in the Standard Library
PEP 616, String methods to remove prefixes and suffixes
PEP 617, New PEG parser for CPython

I'd set a low priority for this. We could check that nothing breaks with 3.9, but I'd defer time-consuming changes until there is a user base for it.

We'll need to wait on builds for a bunch of dependencies, just like we had to for Python3.8. Once xarray starts supporting Python3.9, it would be good to follow suit.

Quick heads up, I installed a new python3.9 environment on my machine and ran all xclim tests. Everything runs fine except the loess methods on xclim.sdba, which fail with numba errors. I'm not looking into it for now, but it could only be because of a recent numba version. So AFAIK, Python3.9 shouldn't break xclim!

I don't think it will either, so long as we are maintaining backwards compatibility to Python3.6, the only work involved here would be setting up a build for it in .travis.yml.


75887 - Don't default app name and version in options

AB#75887

don't default app name and version in options

💯


Game only has a single wave of enemy

Added enemy spawns every wave clear. Merged to develop for testing.


Support for pagination on Flight Offers Search?

Hi there!
I'm testing out the /v2/shopping/flight-offers endpoint with a specific search that most likely returns >250 results. (I'm suspicious because the response is exactly 250 results and doesn't include a particular outbound/return flight combo I know exists.) Since the maximum value for the max attribute is 250, is there any way to obtain additional results or paginate through the full resultset via multiple API calls?
Here's my sample call:
curl "https://test.api.amadeus.com/v2/shopping/flight-offers?originLocationCode=LGA&destinationLocationCode=DFW&departureDate=2023-05-23&adults=1&currencyCode=USD&includedAirlineCodes=DL&returnDate=2023-05-30&max=250" \
     -H 'Authorization: Bearer <REDACTED>' \
     -H 'Content-Type: application/x-www-form-urlencoded; charset=utf-8'"

Thank you!

A related question I'll tack on here -- is there a way to exclude certain fare classes from results, like Basic Economy? When I use travelClass=ECONOMY my results are generally all Basic Economy fares.

Hi  I was looking for a amadeus MPTB response solution, We work on the multiple PCC module , each PCC is taking approx 19 sec to response , which doesn't  meet the business requirement. How do we handle it?

Pagination is not supported by the Flight Offers Search. For further questions related to Self-Service APIs please get in touch with the dedicated support channels. For anything related to Enterprise get in touch with your account managers.