Fix tests failing with rounding errors

What changes were proposed in this pull request?

Increase tolerance for two tests that fail in some environments and fail in others (flaky? Pass/fail is constant within the same environment)
Why are the changes needed?
The tests pyspark.ml.recommendation and pyspark.ml.tests.test_algorithms fail with
File "/home/jenkins/python/pyspark/ml/tests/test_algorithms.py", line 96, in test_raw_and_probability_prediction
    self.assertTrue(np.allclose(result.rawPrediction, expected_rawPrediction, atol=1))
AssertionError: False is not true

File "/home/jenkins/python/pyspark/ml/recommendation.py", line 256, in _main_.ALS
Failed example:
    predictions[0]
Expected:
    Row(user=0, item=2, newPrediction=0.6929101347923279)
Got:
    Row(user=0, item=2, newPrediction=0.6929104924201965)
...

Does this PR introduce any user-facing change?

No
How was this patch tested?

This path changes a test target. Just executed the tests to verify they pass.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

Test build #130047 has started for PR 30104 at commit 7e60ddd.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/130047/
Test PASSed.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34656/

Looks fine but let me leave it to @WeichenXu123

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34656/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/34656/
Test PASSed.

@AlessandroPatti, mind filing a JIRA?

@AlessandroPatti, mind filing a JIRA?

Sure, filed SPARK-33203

The change is pretty fine - what env are you testing on though, just to understand where this might be different?

Thank you for your first contribution, @AlessandroPatti .
You are added to the Apache Spark contributor group and SPARK-33203 is assigned to you.
Welcome to the Apache Spark community.
Also, I also have the same question with @srowen . Which environment did you hit this?

I know it's very common to have diffs in floats in Python, and to avoid the exact match in doctest but

some environments

Yeah, It's usually platform and Python version dependent. It should be best clarify.


feat: Expose blueprint via API

Blocked by https://github.com/tailcallhq/tailcall/issues/2872

Technical Requirements

Setup a /blueprint endpoint to expose blueprint.
By default the /blueprint should not be exposed publicly unless exposed via routes
Add unit and integration tests wherever necessary, ensure coverage doesn't drop.

/bounty $150

Can not expose blueprint directly because it got too much of system information and implementations that can not be exposed/serialized. Closing this issue for now.


Is that possible to include custom rule from current project?

Readme states two ways how to add custom rules
buildscript {
    dependencies {
        classpath files('libs/my-custom-ktlint-rules.jar')
        classpath 'org.other.ktlint:custom-rules:1.0'
    }
}

I'd like to have custom rules as separate module in my project. Is that possible to include them directly? Ideally, I'd like that these rules are applied against themselves as well.

+1 We currently have a clunky process where we build our custom rules jar, copy it over to a /libs dir (checked into source control), then use buildscript { classpath(...) } to include them. Would rather do what we do for lint rules:
dependencies {
  lintChecks(project(":custom-checks"))
}

Let me reopen. There's a PR out proposing this. I think it will need some careful testing, but I might have become paranoid about gradle workers :).

@jeremymailen pinging this thread... do you think this can be fixed any time soon?

It might take a little bit, since the upgrade to ktlint 0.47.x is up next.
This would be a bigger change and require careful consideration.

At long last this is supported in 5.0.0 by adding your rules project to the ktlint dependency.


Bug fixes

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://hdh.pr.harness.io/pr-1102

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


feat(gatsby-source-drupal): Add multilingual support by prefixing nodes not in the default language with their langcode

Description
We're currently using gatsby-source-drupal to pull content from our D8 site and we got stuck trying to pull content from another language because it gets overwritten by the last source plugin in the list. eg. gatsby-config.js
...
{
  resolve: `gatsby-source-drupal`,
  options: {
    baseUrl: `http://example.com`,
  },
},
{
  resolve: `gatsby-source-drupal`,
  options: {
    baseUrl: `http://example.com/es`,
  },
},
...

Documentation

[ ] ~Add information about the new parameters language and defaultLanguage to the README.md~
[ ] Add information about the new parameter languagePrefix to the README.md

Related Issues
Addresses https://github.com/gatsbyjs/gatsby/issues/10020

Did you consider to pull from all endpoints first before generating the references/relations. Because there might be the case where a Drupal Node is referencing a content in another language which gets fetched from a later endpoint and cant be referenced because of that?
I created a plugin for my project which based on "gatsby-source-drupal" for mutliple languages and i changed the plugin config like this:

    {
      resolve: 'gatsby-source-drupal-multilanguage-custom',
      options: {
        baseUrl: 'http://page.test',
        apiBase: 'jsonapi',
        languages: {
          prefixes: [
            '',
            '/de',
            '/fr',
          ],
          default: 'en'
        },
      },
    },

So I could fetch all data in all languages and build the references after that.
Like:
...
  drupalFetchActivity.start();

  let allData;

  const nodes = new Map(); // first pass - create basic nodes

  await Promise.all(languages.prefixes.map(async (prefix) => {
    try {
      const data = await axios.get(`${baseUrl}${prefix}/${apiBase}`, {
        auth: basicAuth,
        headers,
        params
      });
      allData = await Promise.all(_.map(data.data.links, async (url, type) => {
        if (disallowedLinkTypes.includes(type)) return;
        if (!url) return;
        if (!type) return;

        const getNext = async (url, data = []) => {
...
  drupalFetchActivity.end();

  nodes.forEach(node => {
    handleReferences(node, {
      getNode: nodes.get.bind(nodes),
      createNodeId
    });
  });

Is there a way to test this locally? How to generate patch to be applied to plugin?

And maybe this is not best approach for site with 20 languages and some other options (like filters) set. Languages array + defaultLanguage string sounds better.


Weird InputEventScreenTouch Ids

Godot version:
3.0.6 stable
OS/device including version:
Surface Book 2
Windows 10 Professional
16 GB RAM
Intel I7 8650U
NVidia GTX 1050
Issue description:
The gui_input callback function on control-nodes is giving me weird touch indexes. While on android the first finger has always the id 0, on my windows 10 touchscreen device my first finger has a random touch id, which changes each time a touch event is triggered.
Here is the output of the minimal reproduction project when touching multiple times (it prints the touch ids):

203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234

Steps to reproduce:

Create an control object and set the gui_input(ev) callback
touch the screen and check the touch indexes of InputEventScreenTouchEvents

Minimal reproduction project:
TouchIndexBug.zip

The Problem still exist in the 3.1 stable version of Godot.
My device is a surface pro 6
The first touch event only registers only as a "touch down". "Touch up" is omitted.
Additional touch indices are enumerated.

I think we have two separate things here:
@Tobi-La, Godot is not expected to give "beautiful" touch indices. It passes to the game whatever index is reported by the platform. Android gives zero-based indices; Windows doesn't.
@th1z, I think your issue is different. Could you open a separate bug report with further details on how to reproduce it, etc.?

Can anyone still reproduce this bug in Godot 3.2.3 rc2 or any later release?

No answer, so closing. Please comment if you can still reproduce the issue, and make sure that there is an up-to-date reproduction project.


Desktop Library | Select by image

Allow users to select by image, this is something that had been added to appium awhile back but was not set up for us yet.
Due to some issues with the appium-windows-driver please make sure that you are updated to the latest version of appium to use this selection method.
https://github.com/appium/appium/issues/14829

Coverage decreased (-0.6%) to 99.369% when pulling 793f982095a70661cd080dacfab5194252f44d0c on image_search into 5af4652a49d65325fae782901ca48b2df3095d10 on master.

Coverage decreased (-0.6%) to 99.369% when pulling 793f982095a70661cd080dacfab5194252f44d0c on image_search into 5af4652a49d65325fae782901ca48b2df3095d10 on master.

Codefactor is upset with a few more lines on a complex method...not going to fix it right now


Reorder might miss tail bytes

I contrived a case with shape (1, 7, 1, 5), a thin board of data can easily imagine. Which I would like to reorder it from nchw into nChw8c, might became a stick.
But from the code I have reviewed, the reorder will stop reorder it at the bold line blow because dims[2]/8 definitely equals 0.
#       pragma omp parallel for collapse(3) schedule(static)
        for (int n = 0; n < dims[0]; ++n) {
            **for (int C = 0; C < dims[1] / blksize; ++C) {**
                for (int h = 0; h < dims[2]; ++h) {
                    constexpr int i_c_mult = order_keep ? blksize : 1;
                    constexpr int o_c_mult = order_keep ? 1 : blksize;
                    auto i = &input[input_d.blk_off(n, i_c_mult * C, h)];
                    auto o = &output[output_d.blk_off(n, o_c_mult * C, h)];
                    ker(i, o);
                }
            }
        }

I also printed out the description of memory, should the padding_dims be (1, 8, 1, 5) ? Or I'm understand it  wrong @emfomenk
    desc_ = {
      primitive_kind = mkldnn_memory
      ndims = 4
      dims = ([0] = 1, [1] = 7, [2] = 1, [3] = 5, [4] =<PHONE_NUMBER>, [5] = 32767, [6] = -1858001806, [7] = 32767, [8] =<PHONE_NUMBER>, [9] = 32767, [10] = -1858001806, [11] = 32767)
      data_type = mkldnn_f32
      format = mkldnn_nChw8c
      layout_desc = {
        blocking = {
          block_dims = ([0] = 1, [1] = 8, [2] = 1, [3] = 1, [4] = 31579992, [5] = 1, [6] = 12569776, [7] = 5, [8] = 3, [9] = 0, [10] = 5120, [11] = 0)
          strides = {
            [0] = ([0] = 1, [1] = 1, [2] = 40, [3] = 8, [4] =<PHONE_NUMBER>, [5] =<PHONE_NUMBER>, [6] =<PHONE_NUMBER>94168, [7] =<PHONE_NUMBER>93552, [8] =<PHONE_NUMBER>, [9] =<PHONE_NUMBER>, [10] =<PHONE_NUMBER>, [11] = 0)
            [1] = ([0] = 8, [1] = 1, [2] = 1, [3] = 1, [4] =<PHONE_NUMBER>, [5] =<PHONE_NUMBER>94168, [6] =<PHONE_NUMBER>, [7] =<PHONE_NUMBER>94168, [8] =<PHONE_NUMBER>94168, [9] =<PHONE_NUMBER>94168, [10] = 4, [11] =<PHONE_NUMBER>94168)
          }
          **padding_dims = ([0] = 1, [1] = 7, [2] = 1, [3] = 5, [4] = 167140613, [5] = 1, [6] =<PHONE_NUMBER>, [7] = 32767, [8] = 4, [9] = 0, [10] =<PHONE_NUMBER>, [11] = 32767)**
          offset_padding_to_data = ([0] = 0, [1] = 0, [2] = 0, [3] = 0, [4] = 167145172, [5] = 1, [6] =<PHONE_NUMBER>, [7] = 32767, [8] =<PHONE_NUMBER>, [9] = 32767, [10] = 4, [11] = 0)
          offset_padding = 0

Ok, I got it, we do not reorder tail situation. I'll ask another question, will we implement automatic padding in future?

Sorry for delay.
Blocked format doesn't support padded_sizes which are not multiple of block sizes. I.e. in your case if padding_dims[1] = dims[1] = 7 -- is not multiple of 8. Ideally MKL-DNN should return error at creation/initialization time... (need to fix that).
MKL-DNN would not support explicit automatic paddings. But you still may do manual padding and set padding_dims[1] = 8 and sizes[1] = 7. Do not forget to adjust the strides. In theory that should work.
Alternatively you may use view primitive descriptor to create such a memory descriptor.
So far we don't have good ideas how to support nChw8c format for the tensors with C % 8 != 0.

@emfomenk Thank you!


Update bower package

Hi there!
Would like to use the functionality from: #254. Could the Bower package be updated with it?

done


Fix definition lists inside api docs

This is similar to #1080 (and is needed to fix citations in the demo here)
Fixes #1052

@Blendify can you generate assets here too? The PR build would be the most helpful

@agjohnson and @stsewd any update on the review?

It would seem that in order to get this PR in, we need to merge latest master and then rebuild assets. I'll be getting a dev env up and hopefully I can push that easily after that :crossed_fingers:

This seems to render well, great work @Blendify :tada:

Ping @agjohnson - I would say this looks good!
Here's another screenshot - the new visual regression test code added by @Blendify


Crash when tracking screen on iOS

After a fresh install and following the instructions for Android and iOS, the library works perfectly well for Android but crashes in iOS with the following error :

Exception 'CoreData: Cannot load NSManagedObjectModel.
nil is an illegal URL parameter' was thrown while invoking trackScreen on target Piwik with params
[...]

I suspect this is an issue around these lines https://github.com/BonifyByForteil/react-native-piwik/blob/master/ios/PiwikTracker/PiwikTracker.m#L1568, where URLForResource fails to find the model
Unfortunately I have zero objective-c experience so I don't exactly grasp how to fix this.
If that can help, the source code for my project is available at https://github.com/BDE-ESIEE/mobile
Anyway, thanks a bunch for this very useful module 😉 !

This is caused by Xcode not copying the Core data file. Expand the BNFPiwik.xcodeproj project and drag the piwiktracker v2.xcdatamodel file inside PiwikTracker to your Project.
See if that helps.

I can confirm that @ruiaraujo 's solution works.
This worked for me: Drag node_modules/react-native-piwik/ios/PiwikTracker/piwiktracker.xcdatamodeld into the xcode project navigator, and uncheck "copy" when presented a dialog.
I put mine into the folder containing AppDelegate.m, etc.

maybe modify README.md
####iOS
1.Add node_modules/react-native-piwik/ios/BNFPiwik.xcodeproj to your xcode project, usually under the Libraries group
2.Add libBNFPiwik.a (from Products under BNFPiwik.xcodeproj) to build target's Linked Frameworks and Libraries list
3.Delete piwiktracker.xcdatamodeld(from Libraries under BNFPiwik.xcodeproj/PiwikTracker)
4.Drag node_modules/react-native-piwik/ios/PiwikTracker/piwiktracker.xcdatamodeld into the folder containing AppDelegate.m
if you don't delete piwiktracker.xcdatamodeld(from Libraries) , your siteId is not corrent

Updated README.md. Thanks for feedback!


Potential for memory leak with use of strdup?

Noticed this call in wrapper.c when trying to make it work with windows.
(Had to change to a safer function name for w/e reason as the POSIX name was depecrated.).
Is there a potential memory leak with the use of strdup?
Documentation says need to free the return value.
https://en.cppreference.com/w/c/experimental/dynamic/strdup

If you are referring to char *__crypt_gensalt_ra(...),
strdup() return pointer is used as return value of that function.
The user has to free the return value of crypt_gensalt_ra(...), otherwise memory leaks could occur, yes.

@trusch I propose to close this issue, because it is not strictly related to libbcrypt, but to original C implementation.


Added placeholder for running bootstrap.py on windows by winpanda

High-level description
Added placeholder for running bootstrap.py on windows by winpanda. Fixed version of cryptography package (set 32 bit package for DCOS open too)
Corresponding DC/OS tickets (required)

DCOS_OSS-5696 Adjust configuration files for Bootstrap package.

Related tickets (optional)

DCOS-ID JIRA title / short description.

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain.
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot changelog-not-required reason

@mesosphere-mergebot request-review @DominikDary

I see... So, my suggestion is to leave it now as is and create JIRA to change it ni GA. It would be better to setup bootstrap as an independent package and add the parameters to run it on demand with the services that should be served by bootstrap. But now it's important to provide Adminrouter with prerequisites during setup.

@mesosphere-mergebot label Ready For Review


Start AI app button on custom user-catalog.json recipe does nothing.

Bug description
I'm not able to start a custom AI recipe by modifying a pre-existing recipe. Following the extension's README, I created user catalog $HOME/.local/share/containers/podman-desktop/extensions-storage/redhat.ai-lab/user-catalog.json to create a ChatBotTest recipe. I copied the defatul catalog  and left only the chatbot recipe with modifications the id and the name recipe properties to avoid collisions.
The extension immediately reloads the catalog list but when I click Sart AI App on my chatbot version nothing happens. I tried searching on Podman Desktop's logs and no error shows. The default recipes continue to work at this point.

Operating system
Fedora 40
Installation Method
from Podman-Desktop extension page
Version
1.0.0
Steps to reproduce

Add a simplified version of the default catalog with only one recipe with it's id and name modified to avoid collisions.
Click Start AI App and nothing happens.

Relevant log output
No response
Additional context
Versions:

podman-desktop-extension-ai-lab v1.1.2 (The issue form only offered v1.0.0)
podman-desktop v1.11.1 Flatpak installation.

Closing as resolved with the owner of the issue. The issue was, he was using v1.1.2 which were not supporting the recommended property in the recipe property


Queries can't be written to the target system

Output format when data model will be exported includes quotation marks ((field["Type"] = "Change Order")). The format will be written to the deploy scripts and includes the quotation marks. The deploy process is not able to handle this format (--queryDefinition="(field["Type"] = "Change Order")")

I added query definition value inner quotes escape in scrip-gen.xsl.
This gives the following output:
--queryDefinition="((field["Type"] = "Task") and (field["State"] = "Defined","In Progress","Draft"))"


profile page

assemble profile page (the card components are ready). some other buttons and components need to be done and the page should be responsive. Design is in the figma file.

Hey where is the figma file

check design folder.

Hey where is the figma file

Do let us know if you are working on this issue so that I can assign it to you.

Can I work on this issue?

@AnandDhakane01 Can you assign me task if it is not resolved?


Simplify logs

Вітаю, я маю невеличку ферму телефонів на яких запущено db1000n і дуже не зручно дивитися на це простирадло логів. Чи могли б ви зробити спрощений режим де просто буде якась оцінка якості атаки, щоб було зрозуміло варто змінювати впн з'єднання чи ні. Дякую

So there are couple options to achieve something similar to what you need already:

you can specify --log-format json and then feed those to some json parser to only gather info you need (I think https://github.com/ihorlv/db1000nX100 does something similar to that so you could take some inspiration there)
alternatively, you can specify --log-level fatal - that should disable all the logs except the stats table. the table may still be pretty large so if json option is not good enough I could probably make alternative reporter format that would only output total stats but not sure when I can get to it

Чому в документації немає нічого про ці опції?

@PerchunPak i guess in most cases it's enough to run the binary without any provided options, but you can see all of them by running the db1000n instance with '-h' argument:
./db1000n -h

@arriven I'll implement it - an option to have only totals. I think it would be even better to have totals and each protocol as a target, something like:
http  | stats...
https | stats..
tcp | stats..
--------
total | stats..

Propose to call it -less-stats. Maybe fewer would be more grammatical, but I guess less is kinda more conventional for text..

Maybe a numeric options would be even better? Something like -less-stats=20 would mean "show less info when the number of targets is higher than 20".

Well, let's start with a simple boolean flag then https://github.com/arriven/db1000n/pull/552

it's merged, should be in the next release. Maybe we could close this issue now.

available in 0.9.10, thanks to @roman-kruglov


Hankel-DMD

https://sites.engineering.ucsb.edu/~harbabi/research/M112523.pdf

If the embedding is given, than HAVOK might be also nice.
https://www.nature.com/articles/s41467-017-00030-8.pdf

I have had good success using time-delay embeddings as observables. FYI, DynamicalSystems.jl has some delay embedding capabilities with algorithms for optimizing the embedding. See: Delay Embeddings

Thanks! I am a little bit anxious because that would mean using the DataSet type :) But I will definitely have a look.
Did you use them within DataDrivenDiffEq? Or in general?

I hacked around with it on my own before DataDrivenDiffEq was a thing. I agree completely regarding DataSet.


Fix Build.

This PR will fix the build issue in the DEV branch, hopefully allowing (https://github.com/serilog/serilog-sinks-applicationinsights/pull/36) to be merged.

Thanks @mbuck !! 👍


Fix initial status

Before this PR it was not possible to display the list of suggestions with no input at launch.
No it can be achieved with the following options:
    searchField.forceNoFiltering = true
    searchField.filterStrings([])
    searchField.interactedWith = true

Still clunky, but an improvement.

I liked your improvements. Thanks! (and really sorry for the long delay)


Add smart/fixed positioning option for components with dropdowns/popups and for tooltips/popovers

In an effort to give users more flexibility to dictate where popups and tooltips/popovers we need to add the ability let them specify where and how they want the item to be displayed.
Supported Locations - Top, Right, Bottom, Left, Top Left, Top Right, Bottom Left, and Bottom Right.
Smart Positioning - when used, implies that the library will determine the best location for the item.
Fixed Positioning - will display the item at the specified location.
Hybrid Positioning - exact behavior TBD, but maybe something like letting the user specify a fixed position and if it doesn't fit, use smart positioning.  Or perhaps, letting the user specify an array of positions to try before using smart positioning.
Supported Components:

[x] Datepicker
[ ] Dropdown (smart, top and bottom positioning only)
[x] Popover
[x] Tooltip
[ ] Typeahead (smart, top and bottom positioning only)

#3900 is a good start and should be used for reference.

Looks like #4293 is also a good point of reference.

This comment relates specifically to how TWBS positions the dropdown component, I plan to follow up in another comment on pixel positioning (tooltip).
TWBS uses 2 approaches to positioning an element relative to another element with the major differences being the markup used and the top/left style values used.  This approach relies more on css to keep the linked element positioned and is used with the dropdown component.  The following is the basic markup used to create a button with a dropdown:

 Dropdown

Action

The relevant positioning css applied by the dropdown and dropdown-menu classes is:

 Dropdown

Action

What this does is let the outer element’s size control where the position of the dropdown menu will appear by setting up a relative position boundary and using absolute positioning on the dropdown-menu element.  The outer element’s height will grow if the button size changes thus automatically altering where the dropdown gets positioned.  If the outer element has display: inline-block the width will work in the same manner.  The top: 100% puts the dropdown-menu elements top edge at 100% the height of its nearest positioned ancestor (the outer dropdown div).  The dropup class simply sets bottom: 100% instead and the dropdown-menu element will appear on top of the outer element instead of underneath.  Extending this approach to position the dropdown-menu element in any quadrant is as simple as applying 100% to the correct css attribute.
Pros:

Puts the burden of positioning the linked element on css, no need to worry about scrolling,
element resize, etc...
Cons:
Requires strict markup.
Requires css not included in TWBS.
Will not work with append-to-body.

The following table lists those values as well as suggested reset values:
Position	         css
top-left:	         top: auto; bottom: 100%; left: 0; right: auto;
top-right:	         top: auto; bottom: 100%; left: auto; right 0;
bottom-left:	 top: 100%; bottom: auto; left: 0; right: auto;
bottom-right:	 top: 100%; bottom: auto; left: auto; right: 0;
left-top:	         top: 0; bottom: auto; left: auto; right: 100%;
left-bottom:	 top: auto; bottom: 0; left: auto; right: 100%;
right-top:	         top: 0; bottom: auto; left: 100%; right: auto;
right-bottom:	 top: auto; bottom: 0; left: 100%; right: auto;
To center, a transform can be used:
Axis		         css
Vertical		  top: 50%; bottom: auto; transform: translate(0, -50%);
Horizontal	  left: 50%; right: auto; transform: translate(-50%, 0);
And with the appropriate primary position:
Position              css
top-center	   top: auto; bottom: 100%; left: 50%; right: auto; transform: translate(-50%, 0);
bottom-center	   top: 100%; bottom: auto; left: 50%; right: auto; transform: translate(-50%, 0);
left-center	   top: 50%; bottom: auto; transform: translate(0, -50%); left: auto; right: 100%;
right-center	   top: 50%; bottom: auto; transform: translate(0, -50%); left: 1005; right: auto;
These can be easily refactored into mixins to make them more manageable.

This comment relates to how TWBS positions the tooltip and popover component.  TWBS accepts the following placement options;

top
bottom
left
right

with an 'auto' flag, (eg: 'auto top') to place the tooltip/popover element where it will fit.
This plunk demonstrates the TWBS implementation.  This plunk demonstrates a proposal for adding this feature to the angular-ui-bootstrap library for a side by side comparison.  A description of the changes:
In TWBS, to calculate tooltip element position, first the host element's (element that the tooltip is applied on) position is found using the following:
Tooltip.prototype.getPosition = function ($element){
  $element   = $element || this.$element;

  var el     = $element[0];
  var isBody = el.tagName == 'BODY';

  var elRect    = el.getBoundingClientRect();
  var elOffset  = isBody ? { top: 0, left: 0 } : $element.offset()
  var scroll    = { scroll: isBody ? document.documentElement.scrollTop || document.body.scrollTop : $element.scrollTop() }
  var outerDims = isBody ? { width: $(window).width(), height: $(window).height() } : null

  return $.extend({}, elRect, scroll, outerDims, elOffset)
}

So it starts with the host element boundingClientRectangle (BCR) which is the following object:
{
  top: number,
  left: number,
  bottom: number,
  right: number,
  width: number,
  height: number
}

In the return $.extend, the top and left values are overwritten by the elOffset variable (jQuery.offset()), then the width and height values are overwritten by the outerDims variable if the host element is the body element.  This logic is already well accounted for in the position service using the offset and position functions.  Which the positionElements function uses:
hostElPos = appendToBody ? this.offset(hostEl) : this.position(hostEl);

Props to @pkozlowski-opensource  for writing the original positioning service code which has stood the test of time and is still solid.
Now that the host element position has been calculated, the available space around the element needs to be calculated to figure out where the tooltip element can fit.  I made the following assumptions:

The viewport (host element container) will be the closest scrollable ancestor.
The viewport padding will be accounted for.
The viewport scrollbar width will be accounted for.
The tooltip element margins will be accounted for.
The tooltip element placement will only be moved if it fits.

The following is what TWBS uses to get the viewport dimensions:
Tooltip.prototype.getViewportAdjustedDelta = function (placement, pos, actualWidth, actualHeight) {
  var delta = { top: 0, left: 0 }
  if (!this.$viewport) return delta

  var viewportPadding = this.options.viewport && this.options.viewport.padding || 0
  var viewportDimensions = this.getPosition(this.$viewport)

  if (/right|left/.test(placement)) {
    var topEdgeOffset    = pos.top - viewportPadding - viewportDimensions.scroll
    var bottomEdgeOffset = pos.top + viewportPadding - viewportDimensions.scroll + actualHeight
    if (topEdgeOffset < viewportDimensions.top) { // top overflow
      delta.top = viewportDimensions.top - topEdgeOffset
    } else if (bottomEdgeOffset > viewportDimensions.top + viewportDimensions.height { // bottom overflow
      delta.top = viewportDimensions.top + viewportDimensions.height - bottomEdgeOffset
    }
  } else {
    var leftEdgeOffset  = pos.left - viewportPadding
    var rightEdgeOffset = pos.left + viewportPadding + actualWidth
    if (leftEdgeOffset < viewportDimensions.left) { // left overflow
      delta.left = viewportDimensions.left - leftEdgeOffset
    } else if (rightEdgeOffset > viewportDimensions.right) { // right overflow
      delta.left = viewportDimensions.left + viewportDimensions.width - rightEdgeOffset
    }
  }

return delta }

To help with this I created 2 functions in the position service - viewportOffset and scrollParent.  The viewportOffset function calls the scrollParent function to get the host elements scrollable parent, then calculates the offsets from the viewport edge with the values representing the amount of space.  The function returns the following object:
{
  top: number,
  bottom: number,
  left: number,
  right: number
}

Then in the positionElements function, it's just a matter of adding the margin values to the tooltip width/height and comparing that to the viewportOffset values.
There is some adjustsments that need to be mode to the tooltip and popover arrows which I added a new positionArrow function to the position service to handle.  The positionElements will accept the following placements:

top
top-left
top-right
bottom
bottom-left
bottom-right
left
left-top
left-bottom
right
right-top
right-bottom

and if preceeded by a space seperated 'auto' will reposition the tooltip element if it does not fit in the desired position.  The object returned will be:
{
  top: number,
  left: number,
  placement: string
}

Where placement is the value where the tooltip element will fit without the 'auto' key word.  I will follow up with a PR for this shortly when I have finished a few tests.

@RobJacobs, did you mean to close this?  Were all the components w/ checkboxes addressed?  Thanks.

Any update on this?

@Alexintosh, positioning was always targeted for the 1.2 release - @RobJacobs generously and judiciously worked on this for several weeks to get it in for the 1.0 release.  We ask that you please be patient.  It is coming.  Please remember that we are all volunteers doing this in our spare time.

@icfantv, I'm now looking for this feature too. Is there anything we can do to help out? I see some boxes check off up at the top, are all the unchecked ones un started?
Is the work for this happening on a specific branch? I can look into it for the datepicker, especially if there is already code done for other directives I can peak at.

@deeg, you've been pretty active and have some quality commits.  we are more than happy to have you work on stuff.  ping me on the angular slack or gitter and i can walk you through setting up your own fork and having it linked with our main repo.

@icfantv, that is already done, just wanted to make sure the other directives hadn't been started by someone else yet. I will look into datepicker in the next few days, possibly next week.

Oh, ok.  you'll need to check with @RobJacobs as he did the initial work for tooltips and popovers. I strongly suspect that the rest of this has not yet been started.
Oh, and by the way, you rock.
For typeahead and dropdown - I would only support smart, top, and bottom positioning.  For the datepicker, probably the same, but I'd like to hear an argument for supporting E, W, or corners before ruling it out.  Personally, I don't think it makes sense, but I'm just one man.
You can use what @RobJacobs did for tooltips/popovers to get you started - most of the work may already be done.

Thanks for the info @icfantv.
I agree with you that E, W, and corners does not make sense for typeahead or datepicker.

Hey just wanted to make sure that I could align typeahead right because I'm using it for an Arabic list

@anasqadrei, please do not hijack threads by changing the subject.  Questions like yours are best served by the community by asking on StackOverflow.  That said, I believe it's the browser's responsibility to take the font you're using and render it correctly.

@icfantv, I've created issue #2753 which was closed in favour of this issue and is referenced here.

@anasqadrei, and have you tied an RTL font yet with the latest code base?

@icfantv, yes. Here is a plunker with the latest codebase(ui-bootstrap 1.0.3, angular 1.4.8, bootstrap 3.3.6) http://plnkr.co/fs79igc1fJWTqY4xrWTt
Copy the RTL text وا and paste it in the textbox. You will see the text in the textbox is aligned to the right but the dropdown is aligned to the left. They both should be aligned right.

@anasqadrei, cool, thanks for the plunker.  Please create a new issue for this so we can track it.  Thanks.

@anasqadrei. so, shocker...i don't have an arabic keyboard so if you could maybe just paste a translation of the characters so we can try to reproduce that would be helpful.  I suppose I could also try with Hebrew as that should suffer from the same problem.  Thanks.

@icfantv, I've updated the plunker so you could type in English characters. Type "o" or  "on" for example. Hebrew would also work. I've aligned everything in the html body to RTL anyway.
Issue #2753 was already created with similar plunkers. It was closed in favour of this issue (#4762)

@anasqadrei, ok, I see it now.  you're just wanting the dropdown typeahead box to be right justified?  Please do open a new issue for this.  I think we may be able to fix this without waiting for a new position service.  Thanks.

Just add .dropdown-menu-right http://getbootstrap.com/components/#dropdowns-alignment
ie.

              {{ lang.text }}

i would suggest that for all related rtl stuff you should take a look at this commit https://github.com/MohammadYounes/RTL-bootstrap/commit/3be91ac69ddd92a47e1e53bcebe2b80b3d04b30e in particular to the code used by @MohammadYounes in order to easily handle the positioning of tooltips.
The solution there implemented makes use of the standard https://www.w3.org/International/questions/qa-html-dir
https://mohammadyounes.github.io/RTL-bootstrap/inline/

Any new for this ticket?

is someone aware of any workaround for this?
@wesleycho ?

Why is this issue still open? It appears that "auto" handles the smart positioning, by swapping left/right and top/bottom when there's not enough room. What is left to implement for position?

The auto positioning is great however similar to the initial Hybrid mode, being able to specify a preferred position (as it seems to default to top?) would be great.

@Js41637 You can specify a preferred position, e.g. "auto bottom-right". Again, I believe everything in this request has been implemented.

@Js41637 I created a Plunker to test "auto bottom", and when there isn't space on the bottom, it moves the popup to the top. Add more BR tags to line 27 if your monitor is a higher resolution:
https://plnkr.co/edit/oJykYhAHaTBJF6GQaMo3?p=preview

As I can see typeahead now supports smart positioning. Can anyone tell me how I can use it? I'd like it to open up automatically when there's no space at the bottom.

@icfantv  Is this issue dead? Because I would much love an auto positioning for dropdowns.

If anyone else ended up here because they want to a uib-typeahead list to appear above the input field, then this is the workaround I use.
It seems that Bootstrap 3 already defines this:
.dropup .dropdown-menu {
    top: auto;
    bottom: 100%;
    margin-bottom: 2px;
}

However uib-typeahead creates a ul.dropdown-menu with a style (on the element) which overrides the top property defined by Bootstrap.  Therefore in your own CSS file add the following rule:
.dropup .dropdown-menu {
    top: auto !important;
}

The parent of your HTML input field should have the dropup class:

    <input type="text" data-uib-typeahead="..."/>


Rancher won't show the readable image version

Rancher versions: 1.6.5
Kubernetes version: 1.6.6
Hi, After upgrade/or clean install rancher v1.6.5, found it will not show the readable image version any more (except rancher infrastructure images), please see the picture below, is this an expected behavior? Thanks.

Does anyone know is this an issue or expected behavior? Thanks.

AFAIK we show exactly what the container was deployed with, and always have.

@vincent99 by docker ps from the host, the version is the same indeed, but I didn't change anything, it is normal before Rancher v1.6.5, what's the possible cause?
31be7982cf1f        demo.com/grafana@sha256:e0ea8b2ff0df2a6df0dca997a9ef95c2c4bd94c3c921f434588b23172a17a0a9                                  "/run.sh"                6 days ago          Up 6 days                               k8s_devops-grafana_devops-grafana-1623218485-7t03v_default_085c8e56-774f-11e7-88ea-02afb77fd1db_0

and by checking deployment from Kubernetes dashboard, I defined with image tag not digest.

@vincent99 did you experience this issue after rancher v1.6.5, only I have such issue?


fixes/issue 5260

Fixes #5260

Tests are failing because of the known pyflakes update.

LGTM. OK to merge.

other than the tests failing (maybe we can just revert to a pyFlakes that works?):
LGTM 👍 ✅

Coverage increased (+1.3%) to 89.577% when pulling 336cf9dc78f87e6ece6c043db899b07d10f5b772 on czpython:fixes/issue-5260 into 22703fc5836f43ad2485016d388d3f7b64adcd1c on divio:release/3.2.x.


Add waitRules for Custom Resources that don't have the "conditions" field in their status.

Describe the problem/challenge you have
kapp's waitRules currently only supports conditionMatchers for resources that have the "Conditions" field in their status. We are using kapp to deploy a Custom Resource that has no "Conditions" field in its status so we are not able to detect if the CR actually got deployed successfully. The CR does have a Status field though which could be used for adding waitRules
Describe the solution you'd like
A waitRules configuration that allows users to configure arbitrary rules. For example if the CR has this in its status field:
status:   currentState: Running
the conditionMatchers could be
- type: currentState      statusMessage: Running
The above is just a suggestion but because this could be arbitrary, the conditionMatcher may need to support more complex data structures.
Anything else you would like to add:

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

related to #11

Design Proposal: https://hackmd.io/hfgsE5v2QoyDjyu3VD0H4g

Acceptance Criteria -

User should be able to configure arbitrary waitRules with the help of a function with predeclared name (eg. check_status) in starlark. Below is the example where we are adding waitRules for the Custom Resource which have status.currentState -

apiVersion: kapp.k14s.io/v1alpha1
kind: Config

waitRules:
  - ytt:
      waitRuleContractV1:
        rules.star: |
          load("@ytt:data", "data")

          state = data.values.status.currentState

          def check_status():
              if state == "Failed":
                return {"Done":True, "Successful": False, "Message": ""}
              elif state == "Running":
                return {"Done":True, "Successful": True, "Message": ""}
              else:
                return {"Done":False, "Successful": False, "Message": "Not in Failed or Running state"}
              end
          end
    resourceMatchers:
      - apiVersionKindMatcher: {apiVersion: <resource-api-version>, kind: <resource-kind>}

Kapp should internally use ytt package to call that function and get the result.

prior art: https://github.com/kubernetes/kubernetes/pull/105776

available in v0.48.0


feature: click overlay to dismiss panel

It would be nice to add a feature that, click background to dismiss the opened panel when overlay or blur enabled, like the flutter showModalBottomSheet does.

Hi @mumumusuc 👋
Nice feature, I will try to release very soon.


Looks UA115Browser version(115Browser/<IP_ADDRESS>) outdated

Background
Im using alist v3.19.0 to manage 115, when upload file from alist console, it fails.
Server log
github.com/SheltonZhu/115driver/pkg/driver.init
	/go/pkg/mod/github.com/!shelton!zhu/115driver@v1.0.14/pkg/driver/error.go:33
runtime.doInit
	/usr/local/go/src/runtime/proc.go:6507
runtime.doInit
	/usr/local/go/src/runtime/proc.go:6484
runtime.doInit
	/usr/local/go/src/runtime/proc.go:6484
runtime.doInit
	/usr/local/go/src/runtime/proc.go:6484
runtime.doInit
	/usr/local/go/src/runtime/proc.go:6484
runtime.main
	/usr/local/go/src/runtime/proc.go:233
runtime.goexit
	/usr/local/go/src/runtime/asm_amd64.s:1598
请升级到最新版本
github.com/SheltonZhu/115driver/pkg/driver.GetErr
	/go/pkg/mod/github.com/!shelton!zhu/115driver@v1.0.14/pkg/driver/error.go:134
github.com/SheltonZhu/115driver/pkg/driver.(*UploadInitResp).Err
	/go/pkg/mod/github.com/!shelton!zhu/115driver@v1.0.14/pkg/driver/response.go:177
github.com/SheltonZhu/115driver/pkg/driver.CheckErr
	/go/pkg/mod/github.com/!shelton!zhu/115driver@v1.0.14/pkg/driver/error.go:145
github.com/SheltonZhu/115driver/pkg/driver.(*Pan115Client).UploadSHA1
	/go/pkg/mod/github.com/!shelton!zhu/115driver@v1.0.14/pkg/driver/upload.go:219
github.com/SheltonZhu/115driver/pkg/driver.(*Pan115Client).UploadFastOrByMultipart
	/go/pkg/mod/github.com/!shelton!zhu/115driver@v1.0.14/pkg/driver/upload.go:289
github.com/alist-org/alist/v3/drivers/115.(*Pan115).Put
	/source/drivers/115/driver.go:94
github.com/alist-org/alist/v3/internal/op.Put

It would be good if we can have a config to manually set 115 UA in future.

BTW, Im new to Go, so not sure if I testing properly:
Even after I made UA115Browser = "Mozilla/5.0 115Browser/<IP_ADDRESS>"
and run client := New(UA(UA115Browser)).ImportCredential(cr)
it still fails
But use New(UA(UA115Desktop)).ImportCredential(cr) is working as expected.


AWS Elastic Beanstalk NPM Install Fail

Hello,
Trying to run NPM install on a deployment in Elastic Beanstalk and the node-sass package is throwing some error. I have tried suggestions in Issue #799 but still can't get it to run properly. When I ssh into the box and run nom install it runs fine.
Any suggestions?

NPM version (npm -v): 5.5.1
Node version (node -v): v8.9.1
Node Process (node -p process.versions): v8.9.1
Node Platform (node -p process.platform): linux
Node architecture (node -p process.arch): x64
node-sass version (node -p "require('node-sass').info"): n/a
npm node-sass versions (npm ls node-sass): n/a

When encountering a syntax, or compilation issue:
  ><EMAIL_ADDRESS>install /var/app/ondeck/node_modules/node-sass
  > node scripts/install.js

  npm ERR! file sh
  npm ERR! path sh
  npm ERR! code ELIFECYCLE
  npm ERR! errno ENOENT
  npm ERR! syscall spawn sh
  npm ERR<EMAIL_ADDRESS>install: `node scripts/install.js`
  npm ERR! spawn sh ENOENT
  npm ERR!
  npm ERR! Failed at the<EMAIL_ADDRESS>install script.
  npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

  npm ERR! A complete log of this run can be found in:
  npm ERR!     /root/.npm/_logs/2017-12-01T15_04_13_736Z-debug.log
   (ElasticBeanstalk::ExternalInvocationError)

Can you provide the logs in a gist? I'm guessing it is related to that this issue, but now it's failing because we try to use the NPM cache folder to store the binary download and thats failing with whatever account the automated install is using.

Commands to Run on Deployment
01config.config
Logs
eb-activity.log
2017-12-01T15_04_13_736Z-debug.log

So apparently npm install takes too long during automated deployments on t1.micro instances, bumping to a t2.small gives enough power to get the task done.

https://stackoverflow.com/questions/39602428/npm-install-on-elastic-beanstalk-fails-unless-manually-executed

Thanks for sharing the solution @jwray

Thanks @jwray saved me!

Seriously though, why don't EB logs give you any indication of the cause of failure... so frustrating.


Updated project portfolio

Pull Request Test Coverage Report for Build 286

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

Totals

Change from base Build 285:
0.0%

Covered Lines:
2350

Relevant Lines:
2605

💛  - Coveralls


setting up postgres, running sequelize init

I'm having trouble setting up postgres.
I've followed the docs, set up my database, run
createuser root
createdb react_webpack_node_development # or test/production

But when I try to run npm run sequelize db:migrate, I get an error saying that I should run sequelize init.
Fair enough, so I do that, but then it creates a new config and empty folders for migrations, etc.
Looking in ./server/db/sequelize I see the migrations I think I'm supposed to be using... but I'm at a loss as to how to configure sequelize to use them...
Any help would be greatly appreciated!

Ah.
I had to cd into ./server/db/sequelize and run ../../../node_modules/.bin/sequelize db:migrate --config ./sequelize_config.js
I would close this and submit a PR to the docs, but that ../../../node_modules/.bin/sequelize is pretty messy... unfortunately it didn't work when I just ran npm run sequelize db:migrate --config ./sequelize_config.js
Do you think there is some way to explain this in the docs and or restructure the sequelize config?

@WestleyArgentum Thanks for creating this issue.
Interesting. When I was running through this docs with a fresh clone, I didn't have to run sequelize init, so I didn't have the issue when running npm run sequelize :(
@mjewell might have more context to this issue. 🍶

Hey,
For me just running npm run sequelize db:migrate works fine. If it's asking for sequelize init that means it can't find sequelize_config.js, but the .sequelizerc file should be telling it where to look for that, and at least locally npm run sequelize db:migrate seems to work from any subdirectory in the project. Did you somehow lose the .sequelizerc file?
Regardless, if you do want to run the command you were trying, you will have use a slightly different syntax due to the way npm scripts handle args. If you look here it explains it, but the TL;DR is npm run sequelize db:migrate -- --config ./path/to/sequelize_config.js.

Oh my gosh, I'm sorry, at some point I must have copied files over in osx finder and thereby lost all the .files
Thanks very much, sorry for the noise

Haha, no problem. Glad it's all working

I've tried all the ways mentioned here and keep getting:
Sequelize [Node: 10.1.0, CLI: 2.8.0, ORM: 3.30.4]
(node:4737) ExperimentalWarning: The fs.promises API is experimental
gulp[4743]: ../src/node_contextify.cc:628:static void node::contextify::ContextifyScript::New(const v8::FunctionCallbackInfov8::Value&): Assertion `args[1]->IsString()' failed.
1: node::Abort() [gulp]
2: 0x876c55 [gulp]
3: node::contextify::ContextifyScript::New(v8::FunctionCallbackInfov8::Value const&) [gulp]
4: v8::internal::FunctionCallbackArguments::Call(v8::internal::CallHandlerInfo*) [gulp]
5: 0xacd2df [gulp]
6: v8::internal::Builtin_HandleApiCall(int, v8::internal::Object**, v8::internal::Isolate*) [gulp]
7: 0x3a1f8d20427d
Anyone have any idea what is going wrong?


fix(firestore, web): stop cleaning up snapshot listeners in debug

Description
Replace this paragraph with a description of what this PR is doing. If you're modifying existing behavior, describe the existing behavior, how this PR is changing it, and what motivated the change.
Related Issues
fixes https://github.com/firebase/flutterfire/issues/13019
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] The analyzer (melos run analyze) does not report any problems on my PR.
[ ] I read and followed the Flutter Style Guide.
[ ] I signed the CLA.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

Is there other places where this fix could be implemented?

@Lyokone - I think so, I'll have to double check as the issue was the map was being recreated for each query. I'll create PRs for others if needed 👍


Slow download speed using files. Normal speed using memory

If I select to use files on Elementum options, the download speed is always dropping, if I select memory there is no problem at all.
I have this problem on an android box, already done a clean install and there is no problem with the storage speed on the machine.
I will try to paste logs later.
Nice work maintaining this Addon

You should review existing issues before creating a new one.
It's not in a plugin scope.
https://github.com/anacrolix/torrent/issues/265


Add cluster curator

Signed off by @jnpacker

Add cluster curator controller image to build.
Spoke to install and cicd and they fell we have all the pieces in the right place
This Helm chart was used to deploy cluster curator for the Playback demo

/cc @hanqiuzh @leena-jawale @Randy424

/lgtm


refactor(app): make consistent close and cancel run modals

closes #9066
Overview
This PR makes the closing or cancelling a run modal consistent whether you are closing/cancelling on the run details page or the protocol accordion steps page.

Changelog

had to extend the props in ConfirmCloseModal in order to make the buttons blue because I noticed that the modal is used in other areas of the repo
made a test for ConfirmCloseModal because i noticed it was missing
added a bunch of stuff to ProtocolUpload component to add the ConfirmCloseModal when you cancel during a run

Review requests

test the 3 flows on a robot to make sure it looks good. I did test on my robot already. But also, here are the 3 flows outlined on figma.
review my changes on CloseConfirmModal to make sure it is ok since that component is used all over the repo (NOTE: you'll see that I used secondaryBtn for both buttons, this is because the original modal before I added the button colors has both of the buttons as secondary button stylings)

Risk assessment
low, behind ff

this looks GTM!!!!


Endpoint N+1 Select insight

Please note:

The current UI will actually show a list of assets/spans that are bottlenecks, in the new UI there is a separate insight for each one.
We need to add an "info" icon like in the current design
The icon color should reflect the insight criticality

https://www.figma.com/file/MxA95ZHK5NqGX4ptFQljRJ/Design-System-3.0?type=design&node-id=2068-14799&mode=design&t=orYiJkzSk02p5Vg6-11


Beginnner Questions

Hi you just recommended me your software on stackoverflow:
https://stackoverflow.com/questions/78043266/python-subprocess-launch-new-chrome-tab/78043976#78043976
Now I have some questions to this software, sadly could't find an answer on the wiki.

How to get my chrome profile to open instead of Profile 1
How to get rid of this message:

How to keep the browser alive after python script finishes
How to open browser in fullscreen

Thanks for any help!

I hope the following is helpful.

How to get my chrome profile to open instead of Profile 1

You can't with Chromedriver as it runs a special instance of your Chrome browser. It's a specialised tool for automation orchestrated with Selenium and then Browserist as extension.

How to get rid of this message:

As above, Chromedriver is a special instance of your Chrome browser so you can't. Not sure why this is important to get your job done. If you care about aesthetics, Chromedriver is probably not the right tool for you.

How to keep the browser alive after python script finishes

That's why I added a loop at the end of your script so you close Chromedriver where you started it, i.e. from the terminal.

How to open browser in fullscreen

Please refer the documentation of how to toggle fullscreen mode on.
browser.window.fullscreen()

Then sadly browserist was not the kind of tool I search for. I need an automated launcher, that launches all tabs for me on pc sartup not an automated testing tool.
Anyway thanks!


 If the child is present in the parent's child table but it is
not yet in valid state, the parent ignores the received "Child Update
Request". This applies to a child being restored due to parent's reset
(child is in kStateRestored) or a child which is in the middle of
attach process (e.g., child is in kStateParentRequest). If a
matching child entry cannot be found in the parent's child table, the
behavior remains as before (i.e., parent sends a response with an
error Status TLV to notify the child to re-attach).
This change ensures that after a parent reset to restore a child, the
parent node is the one initiating and sending a "Child Update Request"
to the child and then receives a "Child Update Response" from it. Such
an exchange between the parent and the child will then include a
random challenge and response (authenticating the child and providing
replay protection). Also in this exchange the "Child Update Response"
from the child will include the the child's current MAC and MLE frame
counters, therefore ensuring that the parent correctly learns and
tracks the frame counters being used by the child.

@jwhui and I noticed this while discussing and looking into some other topic related to how to reset the frag tag under MULTI_RADIO config on child recovery.
I tried looking into the history of the code and found a related PR #1853 (from 2017). Adding the info here so @jwhui  and others can also help check and review this (to make sure the new behavior does also  handle the situation from PR #1853).

The PR #1853 addressed a potential issue with recovery of a non-sleepy child after parent reset (detailed scenario described in #1853 description).
In short, the issue was we could get to a situation where the child would assume it is connected to the parent (since it continued to receive "Child Update Response" to its "Child Update Requests") but parent would not change the child's state to "valid".
PR #1853 fixed this by setting the child to kStateValid upon receiving a "Child Update Request" from it (when trying to restore the child).
I think the behavior change from the current PR should also address the original situation from #1853.

By having the parent no longer respond to the child's "Child Update Request" messages (when the child entry in parent's child table is not marked as valid), the child then detects this and tries to re-attach (which is what we want to happen).

By having the parent no longer respond to the child's "Child Update Request" messages (when the child entry in parent's child table is not marked as valid), the child then detects this and tries to re-attach (which is what we want to happen).

Does this mean, after a parent reset, if a child happened to send "Child Update Request" before the parent sent "Child Update Request" and set its internal state (about that child) to 'valid', then we ignore the "Child Update Request" and let the child to re-attach?


[1pt] Create the basic entities of the project (group task)

Generate nested entities for users, recipes, and foods
Define and share the utile routes
Analyse the generated controllers and designate further customization

Utile routes:


/api not taken into account

Hello @pamelafox ,
I was wondering if you can give me hand with something:
I'm using:
        server = FastAPI(             servers=[{"url": "/api", "description": "Endpoints for the API"}],             root_path="/public",             root_path_in_servers=False,             title="Demo FastAPI function",             lifespan=lifespan,         )
This solves the location of the openapi.json file, thanks a and great!
But then I have a router:
router = APIRouter(prefix="/v1/control_table", tags=["control_table"])
In the UI show like this:

In 1 you can see there no  /api in the URL, but in 2 you can see there is /api in the url, which then fails.
The endpoint is deployed without /api, but the UI thinks there is /api prefix.
Do you have any hints how to solve this?
Thanks!
MIguel
PS.: I have been trying with the prefix and when adding the router, but with no luck.

Sorry for delay, I haven't revisited this project in a while.
It's adding the /api due to this line:
        servers=[{"url": "/api", "description": "API"}],

That's intentional since this architecture sends all /api/ requests through APIM, which requires a subscription key on them, and then forwards them to the app.
Are you saying the issue is that it's then not being recognized by FastAPI? There might be some bad interaction between Router and the FastAPI config.


FR: Conversation continuity

Feature request:
It would be cool to have conversation continuity with the bot. My research so far says that this is achieved in the web version of ChatGPT by concatenating all previous prompts together with the new prompt so that state doesn't have to be maintained on the back-end.
Since this script is not scalable across containers/machines (yet), it might make sense as a prototype to store state in-memory, and automatically prune messages older than n minutes before state is accessed.

Added in 0.4.1 dev branch.

Resolved.


Create a poc processor

Description
This PR includes the following proposed change(s):

Map diligen response to court details FLA-937

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Does the change impact or break the Docker build?

[ ] Yes
[x] No

If Yes: Has Docker been updated accordingly?

[ ] Yes
[ ] No

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] New and existing unit tests pass locally with my changes

Also what are java conventions for whitespace?

Also what are java conventions for whitespace?

between statements ?

Also what are java conventions for whitespace?

between statements ?

In general, seems like everyone does it differently.


Is there a way to lazy load the locale .json file?

Right now, we have sub-components that essentially have their own translation files (using .addProvider).  However, when these components are loaded dynamically (via routing), the translation file never gets added.  Calling updateTranslation does nothing since the language is still the same.
Any help or suggestions would be appreciated!

Hi,
you can find the explanation of how the lazy loading works here.
Also the sample app is built in this way: see list.module.ts and list.component.ts.
Basically, you have an instance of the classes LocaleService and LocalizationService  at the beginning for the whole app and a new instance of the LocalizationService class for all the components that you want to have their own translation data. There are two cases:

Lazy loading of modules

In this case, yuo have something like this:
@NgModule({
    imports: [
        ...
        LocaleModule, // LocaleService is singleton.
        LocalizationModule.forChild() // New instance of LocalizationService.
    ],
    declarations: [ListComponent]
})

and the new initialization of the LocalizationService in the component.

Lazy loading of components

In this case, to create a new instance of LocalizationService, you have to inject it in the component:
@Component({
    ...
    providers: [LocalizationService] // New instance of LocalizationService.
})

export class ListComponent extends Locale {
    constructor(public locale: LocaleService, public localization: LocalizationService) {
        super(locale, localization);

        // Initializes LocalizationService for this component and for its descendants.
        this.localization.translationProvider('./resources/locale-list-'); // Required: initializes the translation provider with the given path prefix.
        this.localization.updateTranslation(); // Need to update the translation.
    }
}

The main difference between these two ways of doing it is in the way they are loaded: in the first case, all the child modules are loaded when needed (if you see the dist folder in the sample app, you find a chunk - the app has also activated the preloading in background). In the second case, the child component is performed at the time of the request, but it's loaded at the bootstrap of the app.
If you have still problems, post your code.
Greetings


Finalize migration of tests/foundry

Related to: https://github.com/runtimeverification/evm-semantics/pull/1904
We completely remove tests/foundry

Made sure that the test utilities (Python, Docker) were pointing to the new location
Bumped and pined forge-std to the latest release, the one that was used was ~1 year old

@ehildenb @nwatson22 this PR seems to remove tests/foundry that are not used by the CI — is it something we want to do? I'll update expected output in this case. We've talked about removing it with @nwatson22 just yesterday — these tests are not used in CI and sometimes are not being updated consistently with the integration tests in kevm_pyk, so it makes sense to me.
This PR also bumps the version of forge-std being used, so I suspect it will require updating tests and relevant rules that rely on hardcoded program counter (the loop invariant defined in lemmas.k  for LoopsTest.sum_N, in particular):
- FORGE_STD_REF: Final = '27e14b7'
+ FORGE_STD_REF: Final = '75f1746'

Yeah, the actual test-suite has moved to a subdirectory of kevm-pyk/....

I'm migrating this PR to https://github.com/runtimeverification/kontrol/

Closed by https://github.com/runtimeverification/kontrol/pull/48.


Fix YouTube API requests

Description

This replaces some of the YouTube API calls with https://github.com/LuanRT/YouTube.js. It's likely that more code can be removed, that can come later though.
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Closing issues
Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (0793980) to head (776bfc9).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##            master      #275   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            8         8
  Lines           52        52
  Branches         7         7
=========================================
  Hits            52        52

Continue to review full report in Codecov by Sentry.

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

Hi @avallete, if this gets merged, further optimizations like this are possible:
const youtube = await Innertube.create({
  cookie: document.cookie,
  fetch: (...args) => fetch(...args),
})

let playlist = await youtube.getPlaylist(playlistName);
let videoIds = playlist.videos.map(video => video.id);

while (playlist.has_continuation) {
  playlist = await playlist.getContinuation();
  videoIds = videoIds.concat(playlist.videos.map(video => video.id));
}

console.log(videoIds);

Hi @avallete, if this gets merged, further optimizations like this are possible:
const youtube = await Innertube.create({
  cookie: document.cookie,
  fetch: (...args) => fetch(...args),
})

let playlist = await youtube.getPlaylist(playlistName);
let videoIds = playlist.videos.map(video => video.id);

while (playlist.has_continuation) {
  playlist = await playlist.getContinuation();
  videoIds = videoIds.concat(playlist.videos.map(video => video.id));
}

console.log(videoIds);

Nice ! Looks good !


support multiple runtimes

Placeholder till upstream https://github.com/kubernetes/kubernetes/pull/40662#event-940637569 is figured out. I'll be working on this.

As upstream pointed out, networking maybe tricky here. @sameo do you know if we can have a clearcontainer and runc pod using the same CNI plugin be able to reach each other?

@mrunalp I can't speak for other hypervisor based runtimes, but with the default CNI plugins a runc pod and a Clear Containers one would be able to reach each others.
In that regard CNI is actually a lot saner than e.g. CNM because it returns a complete netns configuration and it's up to the runtime implementation to bridge it transparently to whatever networking model it uses. With Clear Containers we bridge the CNI created interfaces with QEMU at the L2 level, so it's pretty much transparent and it can definitely reach runc created pods.

@sameo Nice :) That will surely help with mixed runtimes!

Sounds good, but most workloads probably do not have to communicate.  Looking to use kvm containers for running untrusted commands like users builds.

We could support runtime choosing by pod annotations, e.g. runtime.ocid.kubernetes.io=runc|clearcontainer|runv. And networking should be ok for those runtimes for bridge cni network plugin.
Some potential problems:

hypervisor based runtimes do not support host-networking
hypervisor based runtimes may not support all cni plugins

I think ocid should report an error on such cases.

+1 for choosing by pod annotations. BTW. It seems to be a good place to mention, that there's a solution to run different runtimes (by different I mean runtimes with different "shims", for example, docker and runc+cor) on single node - https://github.com/Mirantis/virtlet/blob/master/docs/criproxy.md

One thing that needs to be considered (especially for projects like Atomic, Kubic and CoreOS) is that an upgrade from one cluster type to another needs to be handled in a fairly smooth way. It looks like the upstream PR was closed, but I would like to know what the intended upgrade process would be.
For Kubic/CaaSP, we would try to drain each node (still running on Docker) and then reboot into using cri-o. But the important thing to note is that using annotations would mean that you couldn't upgrade like that.

For Kubic/CaaSP, we would try to drain each node (still running on Docker) and then reboot into using cri-o. But the important thing to note is that using annotations would mean that you couldn't upgrade like that.

Why couldn't upgrade by drain? Annotations are part of pod spec, it doesn't change because of restarting or recreating.

@feiskyer Sorry, you currently could update by drain, but if we start making it so that you have annotations set on Pods to specify what runtime to use then I'm wondering what the upgrade process is going to be (can you gradually change annotations of a set of Pods under a replication controller or similar?).

@feiskyer

Some potential problems:
hypervisor based runtimes do not support host-networking

With k8s running some of its components (e.g. kube-proxy) as privileged containers with host networking access, how do you guys plan to support that with runV ?

can you gradually change annotations of a set of Pods under a replication controller or similar?

@cyphar I mean set the annotations on pod template in the rc spec, so each pod created by rc will carry the annotation.

Out of curiosity: with k8s running some of its components (e.g. kube-proxy) as privileged containers with host networking access, how do you guys plan to support that with runV ?

runV doesn't support host networking, that why we need multiple runtimes, e.g. switch to runc for host-networking pods.

@feiskyer

@cyphar I mean set the annotations on pod template in the rc spec, so each pod created by rc will carry the annotation.

Yeah, I thought so. So we need to be careful with making it impossible for people to switch a cluster to a different runtime without rewriting all of their RC templates (and then doing a blue-green two switch RCs).

@sameo @cyphar We support multiple runtimes so can this be closed?


imperfect action on the prepend

[watch-worker] ok 1 details
[watch-worker] ok 2 details
[watch-worker] ok 3 details
[watch-worker] ok 4 details
[watch-worker] ok 5 details
[watch-worker] ok 6 details
[watch-worker] ok 7 details
[watch-worker] ok 8 details
[watch-worker] ok 9 details
[watch-worker] ok 10 details
[watch-worker] ok 11 details
ok 12 details
[watch-worker] ok 13 details
[watch-worker] ok 14 details
[watch-worker] ok 15 details
[watch-worker] ok 16 details
[watch-worker] ok 17 details
[watch-worker] ok 18 details
[watch-worker] ok 19 details
[watch-worker] ok 20 details
[watch-worker] ok 21 details
[watch-worker] ok 22 details
[watch-worker] ok 23 details
[watch-worker] ok 24 details
[watch-worker] ok 25 details
[watch-worker] ok 26 details
[watch-worker] ok 27 details
[watch-worker] ok 28 details
[watch-worker] ok 29 details
[watch-worker] ok 30 details
[watch-worker] ok 31 details
[watch-worker] ok 32 details
[watch-worker] ok 33 details
[watch-worker] ok 34 details
[watch-worker] ok 35 details
[watch-worker] ok 36 details
[watch-worker] ok 37 details
[watch-worker] ok 38 details
[watch-worker] ok 39 details
[watch-worker] ok 40 details
[watch-worker] ok 41 details
[watch-worker] ok 42 details
[watch-worker] ok 43 details
[watch-worker] ok 44 details
[watch-worker] ok 45 details
[watch-worker] ok 46 details
[watch-worker] ok 47 details
[watch-worker] ok 48 details
[watch-worker] ok 49 details
[watch-worker] ok 50 details
[watch-worker] ok 51 details
[watch-worker] ok 52 details
[watch-worker] ok 53 details
[watch-worker] ok 54 details
[watch-worker] ok 55 details
[watch-worker] ok 56 details
[watch-worker] ok 57 details
ok 58 details
[watch-worker] ok 59 details
[watch-worker] ok 60 details
[watch-worker] ok 61 details
[watch-worker] ok 62 details
[watch-worker] ok 63 details
[watch-worker] ok 64 details
[watch-worker] ok 65 details
[watch-worker] ok 66 details
[watch-worker] ok 67 details
[watch-worker] ok 68 details
[watch-worker] ok 69 details
[watch-worker] ok 70 details
[watch-worker] ok 71 details
[watch-worker] ok 72 details
[watch-worker] ok 73 details
[watch-worker] ok 74 details
[watch-worker] ok 75 details
[watch-worker] ok 76 details
[watch-worker] ok 77 details
[watch-worker] ok 78 details
[watch-worker] ok 79 details
[watch-worker] ok 80 details
[watch-worker] ok 81 details
[watch-worker] ok 82 details
[watch-worker] ok 83 details
[watch-worker] ok 84 details
[watch-worker] ok 85 details
[watch-worker] ok 86 details
[watch-worker] ok 87 details
[watch-worker] ok 88 details
[watch-worker] ok 89 details
[watch-worker] ok 90 details
[watch-worker] ok 91 details
[watch-worker] ok 92 details
ok 93 details
[watch-worker] ok 94 details
[watch-worker] ok 95 details
[watch-worker] ok 96 details
[watch-worker] ok 97 details
[watch-worker] ok 98 details
[watch-worker] ok 99 details
[watch-worker]

this is now fixed hooray!


#4078 Return a dedicated error message when sonatype returns a 403

Fixes Issue
Fix #4078
Description of Change
Handle an HTTP 403 Forbidden response from OSS Index and display an explicit error message.
Have test cases been added to cover the new functionality?
yes
No change on the public API.
Comments are welcomed. :slightly_smiling_face:
edit: I am sorry for the multiple --force-push-with-lease, a Maven execution on my workstation ends up in success. I will try to find why the test is failing in the CI.

edit: I am sorry for the multiple --force-push-with-lease, a Maven execution on my workstation ends up in success. I will try to find why the test is failing in the CI.

Haven't tried locally yet, but did you run with releasetesting enabled when you ran your local build?
mvn clean package verify site -DreleaseTesting --no-transfer-progress --batch-mode
would be the best effort local reproducer honoring your local settings
mvn -s settings.xml clean package verify site -DreleaseTesting --no-transfer-progress --batch-mode
would the fully identical goals run in PR-workflow

mvn -s settings.xml clean package verify site -DreleaseTesting --no-transfer-progress --batch-mode

Yes this ends in success with this command on my laptop with jdk 8 & jdk11:
Apache Maven 3.6.3 (Red Hat 3.6.3-13)
Maven home: /usr/share/maven
Java version: <IP_ADDRESS>, vendor: Red Hat, Inc., runtime: /usr/lib/jvm/java-11-openjdk-<IP_ADDRESS>.1-4.fc35.x86_64
Default locale: fr_FR, platform encoding: UTF-8
OS name: "linux", version: "5.16.11-200.fc35.x86_64", arch: "amd64", family: "unix"

Apache Maven 3.6.3 (Red Hat 3.6.3-13)
Maven home: /usr/share/maven
Java version: 1.8.0_322, vendor: Red Hat, Inc., runtime: /usr/lib/jvm/java-1.8.0-openjdk-<IP_ADDRESS>2.b06-2.fc35.x86_64/jre
Default locale: fr_FR, platform encoding: UTF-8
OS name: "linux", version: "5.16.11-200.fc35.x86_64", arch: "amd64", family: "unix"

I plan to try by using a docker container with an equivalent image than the one used in the CI, then enabling debug logs so I can get more data and identify the differences.

Found the issue, it was due to static fields in the OssIndexAnalyzer, keeping state between tests. :slightly_smiling_face:


[Silabs] Added fixes for hardfault issue with rs9116 when LCD enabled and 917 NCP-LCD enablement

Added fixes for hard fault issue with RS9116 + EFR32 mg24 when LCD enabled.
De initializing SPI driver after every SPI transfer completed and initializing it when SPI communication started.
Also added enabling the LCD before displaying and disabling back once the LCD communication is over.
Added fixes for 917 NCP when LCD enabled.
Tested manually

Added consensus-review checkmark as this is platform-specific code. silabs review should be sufficient here.


Dynamically add typespecs to be used with Dializer

If I make a user status:
  defenum(UserStatusEnum,
    invited: 0,
    activated: 1,
    disabled: 2,
    deleted: 3
  )

And that automatically added the following typespec
@type t ::  :invited | :activated | :disabled | :deleted

I could then use UserStatusEnum.t() when annotating types in the codebase.

This was done by #84


GitHub Repos (Independent Publisher)

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[X] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[X] I attest that the connector works and I verified by deploying and testing all the operations.
[X] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[X] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[X] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[X] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[X] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[X] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[X] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[X] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Can I get clarification as to why GitHub isn't an OAuth provider that's available for certified connectors? It is one of the many providers available for custom connectors.
CC: @ShefaaliP

Hey Troy,
Thanks for highlighting this. One of our key items for MPPC is to support Multi authentication for the connectors. We are working with the devs to prioritize this request.
Shefaali Patankar
Product Manager 2
Copilot Connector Certification
<EMAIL_ADDRESS>[A close-up of a logo    Description automatically generated]
From: Troy Taylor @.>
Sent: Monday, July 15, 2024 6:19 PM
To: microsoft/PowerPlatformConnectors @.>
Cc: Shefaali Patankar @.>; Mention @.>
Subject: Re: [microsoft/PowerPlatformConnectors] GitHub Repos (Independent Publisher) (PR #3564)
Can I get clarification as to why GitHub isn't an OAuth provider that's available for certified connectors? It is one of the many providers available for custom connectors.
CC: @ShefaaliPhttps://github.com/ShefaaliP

Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerPlatformConnectors/pull/3564#issuecomment-2228427992, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A375NR2JRF5UTNEG72IWOITZMPAMFAVCNFSM6AAAAABKH3M5OCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRYGQZDOOJZGI.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


Fix editor getting stuck after baking lightmaps from a script

Fixes #52552.

Thanks!

Cherry-picked for 3.3.4.


Problems of duplication with Full Calendar

Auto-note-mover does not seem to work with Full Calendar plugin
Is there a way to have the plugin execute every few minutes or allow the user to customize the time of execution. (For example, once every 1 minute, once every 5 minutes etc)
I mean tasks are generated by full calendar, They are also duplicated , The main reason is the excution of Auto Note Mover is immediate
You can check out the pic below:
1)Successfully move files

2)However, after moving the files, there are duplicates

I really hope there is some good solution to solve this problem, I think there are many use Full Calendar as main task plugin
Thanks

It's an older post but I wanted to add that I have this same problem with the Database Folder plugin (by RafaelDB)

I have this issue using Templater. It is really annoying since I use it to create every note.
Please, just add a few seconds to the execution time (10 or so) or let us config the trigger delay.

I love this plugin, but this issue is driving me nuts.
It also occurs using Templater, making it really annoying since I use it to create every note.
Please, could you fix it?

I love this plugin, but this issue is driving me nuts. It also occurs using Templater, making it really annoying since I use it to create every note.
Please, could you fix it?

Just noticed that when using the original Obsidian template plugin it works.


perf(rate-limiting) only increment configured limits

perf(rate-limiting) only increment periods that have been configured in schema

Notes: the same patch will have to be applied to the Response Rate Limiting plugin in a separate PR.

LGTM, but this is missing context (the commit message explains what is happening, but now why- is this a perf improvement? Bug fix? Behavior change?). Would like to have a second pair of eyes as well, but seems clean.

@p0pr0ck5 perf improvement

@thibaultcha as noted in the original PR summary, that deserves a different PR, also because response rate-limiting also has deeper problems in its test suite (currently not working).


High Availability

Features to support high availability deployments:

[ ] Automatic failover
[ ] Load balancing
[ ] Minimal-downtime upgrades

@jamilbk I wanted to try HA setup like follows: NLB => 2xEC2 => RDS but not sure if wireguard config will be replicated from db to all EC2 instances? i.e. when admin opens UI on instance one, add users, configs will be stored to db and instance one’s wireguard config updated — will then instance two notice changes in db, pull configs and reconfigure it’s own wireguard?
I’ve started to setup infra to try that but it takes me a bit longer then expected, so perhaps you can save me some time if firezone does not really work like that? It’s just that https://github.com/firezone/terraform-aws-firezone-vpn seems to use completely different approach by setting primary/failover instances and syncing config via rsync so it got me to question my intended approach.

This should be coming as part of #1769 in 1.0.

Well this would probably be a lot easier with K8s support and a helm chart, wouldn't it? ;)

Docs: https://www.firezone.dev/kb/deploy/gateways

Nice. At least with OpenVPN, I remember clients reconnected automatically so no grace period was ever necessary for us.

@xeruf Our clients do reconnect automatically as well if their connection to a gateway is abruptly killed.
The above question was more for dealing with gracefully disconnecting clients with active network flows through a particular gateway. We've discussed internally and have some ideas, but curious to hear what / how users are approaching the problem.


Person ordering

Removed obsolete test (contained in a separate commit)
Switch function created for all criteria
Ternary functions within criteria to sort by ascending or descending
Default is sort by dateOfEvent, order by desc
Function tested locally

I've resolved an issue with default timestamp/date of event ordering in this same PR. I also removed the additional default date ordering which was left over from the previous implementation


missing a comma between .js-comments-form-orig-position .comment-cancel-reply-link

.js-comments-form-orig-position .comment-cancel-reply-link {
display: none;
}
should be .js-comments-form-orig-position, .comment-cancel-reply-link

Nope, this is on purpose. The cancel reply link should only appear when you reply to a threaded comment. The cancel link restores the form in the original position.


cmake: tf-m: create tfm_api dependency on tfm_s.hex

Incremental builds for TF-M are not picked up by Zephyr linking stage. Code changes to tf-m repository results in a rebuild of TF-M and thus an updated tfm_s.hex (and other files), but not a re-link of Zephyr.
tfm_s.hex is merged together with the zephyr hex to form a final merged hex file for flashing. This is done as a post-build command, however such as step cannot take extra dependencies. The Zephyr target can have extra dependencies, however that will only ensure the dependency is brought up-to-date when Zephyr re-link, not re-linking Zephyr when the dependency changes.
Therefore an object dependency is placed on the interface.c file for Zephyr TF-M interface implementation, which ensures the tfm_api library is brought up-to-date whenever TF-M rebuilds, and this update again ensures the Zephyr itself is re-linked whenever TF-M rebuilds.

Is there really a need for Zephyr to relink every time TF-M does so?
I’d expect it to only need to when shared/generated files are updated (eg s_veneers.o).
@adeaarm any advice?

Is there really a need for Zephyr to relink every time TF-M does so? I’d expect it to only need to when shared/generated files are updated (eg s_veneers.o).
@adeaarm any advice?

You're correct, the NS world needs to relink only in case the veneers functions are updated, otherwise the two processing environments are completely indepedent.

Zephyr combines the non-secure and the secure image in a single hex named tfm_merged.hex (which is eventually used by sysbuild later to generate merged.hex).This is generated using the extra_post_build_commands  of Zephyr here:
https://github.com/zephyrproject-rtos/zephyr/blob/31ad324120cf9a865d23321dafc487ea9df6d9c3/modules/trusted-firmware-m/CMakeLists.txt#L449
The reason that the relinking need to happen is that it will enforce this post_builds to execute every time that you update anything in TF-M even if you didn't update something in Zephyr. Without this we observed that a TF-M update will not update the tfm_merged.hex.

Is there really a need for Zephyr to relink every time TF-M does so?

In principle not, but the problem is the merging of the tf-m hex together with the zephyr hex, and this step is only performed when a Zephyr re-link happens.
That was what I tried to describe here: https://github.com/zephyrproject-rtos/zephyr/pull/75090#issue-2377851447

tfm_s.hex is merged together with the zephyr hex to form a final merged hex file for flashing. This is done as a post-build command, however such as step cannot take extra dependencies. The Zephyr target can have extra dependencies, however that will only ensure the dependency is brought up-to-date when Zephyr re-link, not re-linking Zephyr when the dependency changes.
Therefore an object dependency is placed on the interface.c file for Zephyr TF-M interface implementation, which ensures the tfm_api library is brought up-to-date whenever TF-M rebuilds, and this update again ensures the Zephyr itself is re-linked whenever TF-M rebuilds.

but guess I should have continued the sentence with additional to ensure a new merge of the two images.
(added to description).


Need more concise documentation to be useful

In the shape it is now, it works, but it is unusable as you need to configure the instance to get it to do anything useful. This is against the "works by convention" principle of docker. These kind of containers are for quickly rigging a working an instance to get proof of concept where more precise configuration can follow later.

@phiroict can you give some example(s) of what more you'd like to see beyond what is documented at customizing-the-queue-manager-configuration ?

@phiroict - This issue is over 1 year old. Could you check @dnwe response and see if the documentation he has pointed you to contains information you would expect to see.
If there is no response i'll assume you're happy for this issue to be closed.

Follow these instructions customizing-the-queue-manager-configuration  and this time it was sufficient to get the container to work, Thanks you!

No problem! Glad to hear it helped.


I Want to add Bug Report, Enhancement Request, Documentation Update template

I want to add ->

Bug Report

Enhancement Request

Documentation Update

Bug report already being done by someone else

ok,thanks for assigning me


Adds AUD support

Adds AUD as valid currency type

Nope, just the currency (there wont be any special translations for en-AU)
(maybe for a next update, I might change what I called locale, to language to avoid confusing those terms)


Fix GetSecondsRemaining

Description
While doing lipsync, I ran into the fact that it is impossible to understand when the voice over clip finishes its playback.
What is the current behavior?
After playing a voice, the GetSecondsRemaining returns its full duration audio clip.
What is the new behavior?
After the voice clip has finished playing, the number of seconds remaining becomes 0.

Thanks for the PR.


Lightweight JS dataframe library

I would like to have better frontend dataframe manipulation for simple tasks.

I want a standard API that sends data to ag-grid, abstracting over serialization format.
An API similar to https://github.com/data-apis/dataframe-api (it's a good baseline)
Allow some type of syntax for combining columns into arrays so you can have [cleanedVal, origVal, annotation] from raw columns, the combination done in JS. https://github.com/paddymul/buckaroo/issues/74
Very advanced function, lazy loading of additonal data
Filtering

I want to research the existing js dataframe like libraries.  I don't think they offer this functionality, but I want to check.
Once the API is decided on and implemented, this library will enable performance increases through better serialization.

For notes, I'm adding my thoughts on this that I sent to @MarcoGorelli

Looking through the dataframe standard, the next time I improve on the JS deserialization of dataframes, I'm going to implement the dataframe_standard in JS.
This will allow me to decouple the application side from the serialization side.  Then I can change the core serialization.  I'm specifically thinking of higher performance serializations based on TypedArrays and base64.  I notice that the standard doesn't have a "get_row_by_id" or "get_as_list_of_dicts".  I understand why those aren't important for numeric python/C programming, but they are very common in JS.
FWIW I have written multiple versions of Dataframe to JSON to JS serialization over the past decade.  All jank, and slow.  I want to build one properly, also the python side.
No promises on when I will build this, but I will use your code as the basis for it next time I do.


Add usage examples for Debug2Format, Display2Format

Also change some wording to put the most important info first
(drive-by improvement because I had to use these adapters and noticed they were missing some docs that I would've found helpful)

bors r+


create a video

as part of the journey process we also create a video as collateral

https://www.youtube.com/watch?v=Jp_9hKoNYrM solves this issue


Bug: when using the search widget the selected row is always the 1st on the sorted column

I have a table which can be sorted by multiple columns.
The problem happens when clicking the search widget, and finding a row. Clicking on it takes you to the page associated with the top row - and not the row you searched.
Steps to reproduce:

Go to https://deepdao.io, and click on the search widget. Search for any DAO (row in the table), then click on the row: notice that regardless of which DAO was found by the search, it's going to the top DAO by the default sort, which is AUM.

Now change the sort (just click on any of the column headers) and search again: click on the selected row would send you to the DAO at the top of the current, new sort.

I'm using a custom sort on the table options to make sure that the selected row is always the one that relates to the current sort. But this doesn't help with the search widget.
Version: 2.6.4 and it cannot be changed at this point without more work than what I can prioritize.
Is there a solution? Thanks!

I would have no idea how to trouble shoot that without a codesandbox. I would highly recommend trying out the latest version of the table. Switching from version 2.x to 3.x actually shouldn't break anything, as 3.x aimed to support all of 2.x's options. You can find the migration guide here:
https://github.com/gregnb/mui-datatables/blob/master/docs/v2_to_v3_guide.md
The main reason for the major version change was because Material UI 4.x was going to be the focus of version 3.

nRowClick
function

Callback function that triggers when a row is clicked. function(rowData: string[], rowMeta: { dataIndex: number, rowIndex: number }) => void

Which of these do you use?
rowIndex, dataIndex


Docs: Have an operator scope doc per type

Feature Request
Is your feature request related to a problem? Please describe.
Currently, we have just one doc that explains the operator scope and how to change it which is very specific for Go. However, it has been linked in the Ansible/Helm doc and then, its user may might harder to understand.
Describe the solution you'd like

Move current the operator scope doc to Golang section
Create one doc specific for Helm and other for Ansible in each respective sections.
Update the links in the docs to reflect the changes.

For Helm/Ansible operators the steps are quite the same as Go one.
The only diff is that users will not change the main.go file which is not provided to them. However, they can do the same setup by using the WATCH_NAMESPACE as it was in the legacy layout in the config/manager/manager.yaml which allows them to inform 1 namespace or a set of them (N/Many) ( multi-namespace scenario that you can see in the Operator Scope doc for Go).

I just would like to track here that more than one issue came from OCP users since by the default shows that they have not admin permissions and we might need to provide in this docs a tutorial. For example, for Helm use the Nginx example to describe it.

/lifecycle frozen


Cannot install via composer

Hi, I tried installing the package via Composer but it seems there is a versioning conflict with the required paragonie/password_lock ^2.0. The current dev-master of paragonie/password_lock is 2.0.0 but composer sees it as dev-master and cannot resolve dependecies. I am unable to install this library without composer so please update with a fix.
Thank you

2.0.0 is not released yet, as it depends on https://github.com/defuse/php-encryption 2.0.0 which is also not released. For now, use 1.1.0

This should now be fixed.


Update Composer dependencies (2020-05-22-00-08)

Visual regression test passed!
View the visual regression test report