Freedraw creates points instead of lines too often

All of a sudden my inputs, both Wacom and mouse, started to terminate in points instead of letting me draw lines way too often. This was happening to me even before, but now it happens more often than not. Unclear why (CPU load?).

/cc @steveruizok

Can repro on Mac M1 touchpad, too.

Same here. Mouse works without any issues, only tablet. GIMP for example works well on both.
Fedora 33
FF 89.0b13 (64-bit)
Chrome Version 90.0.4430.212 (Official Build) (64-bit)
One by Wacom (medium)

+1 on this happens at unpredictable frequency.
Platform: Linux
Browser: Brave Browser
https://user-images.githubusercontent.com/69213917/121069937-b43bec00-c7eb-11eb-8584-b97b421a5406.mp4
It looks more like a rendering issue, as those points are captured by excalidraw (as selection tool shows in video), and inspecting json as well points are being captured.

I think, i have found the issue here.
Here is my observation:
Assuming it as rendering issue I started looking for values in json which could be invalidating that element from rendering. What I found was that when first two points in array of points are same it doesn't renders the element completely, although it doesn't had any issue when there were continuous duplicate points in middle section of array or at the end of the array.
So I removed the duplicate points from starting and loaded that .excalidraw file again and voila that element got rendered correctly.

Before
After

!

The Steps to reproduce would be

Either try to hold pen a while at a point and then move so that it contains starting duplicate elements of [0,0].
Another way would be to draw a simple shape with free draw.
Export it in a file.Go to that element's points array and duplicate the 0th index.
Load again that file (element won't be rendered).

What i am thinking is correct behavior would be to render a dot even if it has same points from start to end.

I think that's it. I was pretty sure we already tested against the previous
point to avoid adding doubles—if not, we should do that. The algorithm
creates new points too when it smooths the input points, and so may end up
producing duplicates that way; I'll try and fix it in the library just to
be sure.
On Tue, Jun 8, 2021 at 10:29 AM Blesson thomas @.***>
wrote:

I think, i have found the issue here.
Here is my observation:
Assuming it as rendering issue I started looking for values in json which
could be invalidating that element from rendering. What I found was that
when first two points in array of points are same it doesn't renders the
element completely, although it doesn't had any issue when there were
continuous duplicate points in middle section of array or at the end of the
array.
So I removed the duplicate points from starting and loaded that
.excalidraw file again and voila that element got rendered correctly.
Before After
[image: CopyQ DPoLLm]
https://user-images.githubusercontent.com/69213917/121157996-a461ed80-c867-11eb-9ef2-6d5f0006f331.png [image:
CopyQ IglDtN]
https://user-images.githubusercontent.com/69213917/121158133-c0658f00-c867-11eb-92ca-dfcb7863eb03.png
[image: CopyQ SOCoyc]
https://user-images.githubusercontent.com/69213917/121158250-d70be600-c867-11eb-82d5-52abbda629cd.png
![image: CopyQ TsLSgH]
https://user-images.githubusercontent.com/69213917/121158807-44b81200-c868-11eb-9251-1cfcfcd89d6b.png
The Steps to reproduce would be

Either try to hold pen a while at a point and then move so that it
contains starting duplicate elements of [0,0].
Another way would be to draw a simple shape with free draw.
Export it in a file.Go to that element's points array and duplicate
the 0th index.
Load again that file (element won't be rendered).

What i am thinking is correct behavior would be to render a dot even if it
has same points from start to end.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/excalidraw/excalidraw/issues/3564#issuecomment-856614600,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFQA6JDIBV4QTBA4VLA5CLTTRXPIDANCNFSM44SDY5IQ
.

I am having an issue where I am using the online version and trying to use the free draw but am getting points instead

Are there any workaround for this? ￼

Another repro here, Chrome on Linux.

Found this same problem and created a separate issue, sorry about that. I already closed the other one. Gonna paste its content here in case it's of any use.
Here's a video showcasing the problem, it's seems to be more frequent with a Wacom than with a mouse.
https://user-images.githubusercontent.com/16828078/133317582-ec283f5e-79a2-4434-b55e-a66b07264af7.mp4
Same results while in incognito (to avoid any possible problem with other extensions):
https://user-images.githubusercontent.com/16828078/133318644-3d323791-33da-4196-a6de-8662ebdddcfc.mp4
I've done this same test with Miro and with Krita and there is no such problem, which suggests that it's as problem with Excalidraw directly.
I'm using: Version 1.29.76 Chromium: 93.0.4577.58 (Official Build) unknown (64-bit) on Linux Manjaro.
Here's also the .excalidraw generated:
https://excalidraw.com/#json=6676180954513408,7aXSuolsMNOXkfaPDH1RNw

Actually, it's not fully fixed. I could repro on multiple new occasions:
https://excalidraw.com/#json=5982464447086592,W9k4FgnmE6tAsmkxkVidNw

I have been playing around with this today and it has been working great for me on google chrome!

I'm not able to replicate @dwelle's results, how did you do it?

@fev4 No clue. I just took it to town with Apple Pen.

I gave it a quick try with Apple Pencil on iOS 15  and it worked without issue.  (I did have to disable scribble per #3955 )

https://excalidraw.com/#json=5982464447086592,W9k4FgnmE6tAsmkxkVidNw

Wow, @dwelle  is that your hand writing? That's amazing!

@steveruizok it's connected to this issue https://github.com/excalidraw/excalidraw/issues/4007 — single-point freedraw elements (in terms of rendering) shouldn't consist of two points.
All the "buggy" dots from excalidraw.com/#json=5982464447086592,W9k4FgnmE6tAsmkxkVidNw are actually 2-point elements with the 2nd points offset enough to form a big bounding box.

This appears to have been fixed somewhere along the path. If anyone can still reproduce, we will reopen.


Changelog should be automatically generated/calculated

The current process of updating the changelog as part of the PR adds an extra few steps to the PR, an extra commit, and produces infinite conflicts. Some sort of auto-generated changelog would be nice.
This does need some thought though: ideally the changelog listed on matrix.org for unstable modules would still appear, and be accurate.

Implementation over at #1389


Polar charts

Does this wrapper support polar charts? I see on the highcharts website that they require a reference to highcharts-more.js. Setting
chart: {
    polar: true
},

does not appear to work

yes. see in the readme how to plug highcharts modules


Error on Compiling.

Hey Man finally I found a X-Platform Memory management this is awesome, but when i execute the command
./Compile.sh I got this message:

update
Solved the problem using: sudo apt-get install gcc-multilib g++-multilib
update 2
solved
Since compile works fine now, the generated file "mem.o" is generated but it dissapper, tried compiling from terminal and vscode and using sudo too. any idea?
Seems that compile.sh file has a line rm mem.o but why does you decided to remove mem.?

"mem.o" is the object file used during the compilation of "mem.a".
It is removed because, once "mem.a" is there, "mem.o" is just an old object file that is not needed anymore.


Setup 32bit testing

I guess that explains why this showed up on Fedora - it was a 32-bit compile on a 64-bit platform?


Refactor Product view

We should probably be using only one Formik instance and one default values/type.
It will probably brake the way it works right now. Right now, the tabs are separated so that you have to store the changes before moving to the next.
@AnneMatilde would that be okey?

Thats fine!

After doing some research, I think the approach here should be having the same types as UpdateProductInput or CreateProductInput. The thing I'm trying to do here as well is to keep the number of types to a minimum.
Postponing this for a while.


[ISSUE-42] Resource name on the provider side

Addressing #42

@simskij @gruyaume can you merge this PR?


new character

https://github.com/DecodersCommunity/animepedia/pull/619

@iamartyaa please merge

https://github.com/DecodersCommunity/animepedia/pull/619

You don't have to create an issue for adding a new character. A Pull Request is enough. Thank you for contributing to AnimePedia.


SPARQL engine/repository configuration to run in 'strict' or 'extended' mode

The SPARQL specification is extensible in nature. In particular, the behavior of boolean operators on datatyped literals is extensible: while the default specification has no support for all but a limited subset of XSD datatypes, implementors are free to extend behavior to include boolean comparison support - at least for those cases where in the strict minimal interpretation the comparison results in error.
There are several useful extensions that we can add, but some of these may trip up existing users that rely on the 'strict' behavior of the engine. To facilitate this, we should introduce a configuration option that sets the SPARQL engine to run in either 'strict' mode or in 'extended' mode.

I'm thinking about the design of this. We could do this on a per-query basis, or make it a simple extension to the Repository API (a basic setter method and in addition make it part of the config template so it can be configured when creating a repo from the Workbench/Console). Thoughts?

I think it should probably be a bit more generic than this else you could end up with a config zoo. To my mind this is all about being able to configure evaluation strategy being used. You should be able to set the class name/instance of the evaluation strategy supplier you want to use at the sail level. In workbench it is then a similar type option as async threshold or persist. The repo api seems the wrong place, how would it work for something like a sparqlrepo?

I've sketched out an impl @ https://github.com/pulquero/rdf4j/tree/issues/%23164-config-evalstrat

Basic mechanism now merged in. I'm keeping this open to track progress on actually making an extended evaluation strategy available as a config choice. Will do some work on that today.


Sketch/ ReadMe

"Building from your research questions and usage scenarios, give a high-level description of the interface for the app you will build. Remember to be realistic about your expectations and plans since you will actually be implementing this app (but again, you will not be penalized if you need to adjust a bit in later milestones). It is better to design a slightly more limited app that you have time to implement well, instead of a complicated app that you don’t have time to finish.  The app needs to have a few plot panels. Use the visualizations from previous students shown in lecture 1 as a guide of complexity target for the final app.
In this description, you are not required to use specific terminology (i.e. widgets, components, etc…) or make reference to specific libraries. Your sketch can be hand-drawn or mocked up using a graphics editor. If you can show the app's visual design & interaction design in a single image that is ideal, but if you need more space to show some other planned features of your app you can include max three images for this proposal.
The description should be about 200-300 words and live in the README.md file of your GitHub.com repository. The sketch should be linked in the README.md file of your GitHub.com repository underneath the high-level description so that the image shows up on GitHub."


Is there any example of using to TcSequencer understand it?

Hello,
Is there any example of using to TcSequencer understand it?
thanks Víctor.

Hi Victor! Similar to the Commando library, these Tc Libraries are still in very early development, so things are changing fast and not yet documented.
This library is an extension of the TcCommando concept but has two extensions.
The concept is that you have a sequence object that has a similar (ideal the same) interface as a Command from commando. The difference is that instead of just a bool (checked with consume) it has a state machine with a nice api.
//Calling the sequence
MoveUp.Command := myComponent.command.moveup;
MoveUp.Execute := TRUE;
MoveUp();

//moveup gets implemented as a state machine
CASE moveup.doStep() OF
   0:
       IF moveup.onEntry()  THEN
          //Start an action
          axis1.move(moveup); //<-Start move on axis 1 and 2 with the same sequence as the status object
          axis2.move(moveup);
       END_IF

       IF moveup.onsuccess( nextstep ) THEN <-- when the moves are complete go to nextstep
           //Optional code to run on success
       END_IF

       IF moveup.onerror( errorstep ) THEN
           //Optional code to run when leaving the step
       END_IF

       IF moveup.exit( ) THEN
           //Optional code to run when leaving the step
       END_IF

    nextstep:
       //When we are done tell the sequencer
       moveup.sequenceComplete(); -> status.done

   errorstep
       //If there was an error report it
       moveup.sequenceError(); -> status.busy

END_IF

This library also supports integration with TcPiper which is our distributed packML state machine. If it is integrated with Piper, then piper acts as the step manager to coordinate the steps in the sequence as substate.
You can find more information about piper here:
https://loupeteam.github.io/LoupeDocs/libraries/piper.html
https://loupeteam.github.io/LoupeDocs/styleguide/projectstructure/piperguide.html
We have TcPiper in development too, which very similar to our B&R implementation of Piper, but with a more OOP design,


Add support for custom previous, next, first, last text

As per issue #31:
Have something like label-prev, label-next, label-first and label-last to change the text on these buttons. This can be useful for translations too.

Added in commit f361452
Once the readme files have been updated I will close this out

Docs have been updated
Closing with v2.1.0 release


issue 20 credits to ChrisHaehnel

issue 20 credits to @ChrisHaehnel

 Follow up
On Friday, 29-Nov-24 17:37:51 UTC by Gov4Git dev
Issued 20 credits to member @chrishaehnel.


Use GPU to speed up the Performance

This is an enhancment that might take a long time to develop but I think it could greatly improve Performance. Also I already know BabylonJS already uses the GPU. However, the functions within Noa are what is using a lot of CPU.
https://github.com/gpujs/gpu.js/ could greatly help with the Performance of the engine.

Hi, I'm certainly open to suggestions if you have any specific ideas for what processing could be done on the GPU.
But AFAIK the heavy CPU processing in noa is mainly meshing and things like physics, which I would imagine are too tightly coupled to local state to be offloaded.

You're right. Sorry for the waste of time.

Hi, not at all, it would be great if anyone knows any ways to make it work. Personally I know very little about GPGPU, so I'm only guessing about what is/isn't feasible.


test for multiple iot projects

Adds new test MultipleProjectsTest which for now runs a simple workload over http-adapter of 2 iot projects concurrently.
The idea with this test and more similar tests to come is to verify the behaviour of the platform in a more real case because the iot infrastructure is shared between tenants

closing because this work is continued in #2816


Formkit + nuxt-directus-next not working

Hey everyone!
This is gonna be tricky and very specialized I know, but I am uncertain where exactly I should post this.
In order to make an update form, I am currently trying to read some data from my Directus backend and pass it into the Formkit form.
The Formkit form itself works perfect. I am already using it to create items in the backend.
nuxt-directus-next works also perfectly fine, as I get all the data I need in another place.
As soon as I put both together in one file, I get the following though:
500
[nuxt] A composable that requires access to the Nuxt instance was called outside of a plugin, Nuxt hook, Nuxt middleware, or Vue setup function. This is probably not a Nuxt bug. Find out more at `https://nuxt.com/docs/guide/concepts/auto-imports#vue-and-nuxt-composables`.

at Module.useRuntimeConfig (./node_modules/.pnpm/nuxt@3.11.2_@parcel+watcher@2.4.1_@types+node@20.12.7_@unocss+reset@0.59.4_encoding@0.1.13_fl_ljclzznsqjibupl7wngmkvqggi/node_modules/nuxt/dist/app/nuxt.js:226:27)
at ./formkit.config.ts:16:40
at ./node_modules/.pnpm/@formkit+vue@1.6.2_tailwindcss@3.4.3_unocss@0.59.4_@unocss+webpack@0.59.4_rollup@4.17.0_webpa_iv43egqiei23zsjkqkyobxgs7i/node_modules/@formkit/vue/dist/index.dev.mjs:1353:47
at FormKitConfigLoader2 (./node_modules/.pnpm/@formkit+vue@1.6.2_tailwindcss@3.4.3_unocss@0.59.4_@unocss+webpack@0.59.4_rollup@4.17.0_webpa_iv43egqiei23zsjkqkyobxgs7i/node_modules/@formkit/vue/dist/index.dev.mjs:2012:16)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

I guess this is me being new to Vue/Nuxt and not quite understanding what I am doing wrong.

nuxt-directus-next: 0.0.15
nuxt: v3.11.2
formkit: v1.6.2

This is gonna be tricky and very specialized I know, but I am uncertain where exactly I should post this.

Surely we can start here, but I would require a reproduction with:

the current config (in particular directus inside nuxt.config.ts)
the composable/component/page you are using them together that triggers the error

I have no idea why this was a problem, but changing from a functional way of defininf my formkit config to simply exporting the definition worked like a charm. See the discussion here.
Thanks anyway @Sandros94 !


Calling extrapolate_ds() also modifies icvert, causing NaN cells to be moved to location of nearest notNaN cell

This one was confusing, but finally figured it out:

extrapolate_ds calculates a mask where all bottoms are NaN (so where there is no layer information whatsoever)
this mask is query each the nearest cell with data for each NaN cell.
then all data variables are looped through and all data variables with dimensions ("icell2d") are then updated, even if there were no NaNs in that data variable to begin with.

This last step causes "icvert" to be modified by replacing the vertices of the NaN cells with those of the nearest cell.
I used a fix to not update data if there are no NaNs in the masked data. But we could als limit the data variables you want to apply this extrapolation to, so that is something we should think about.

fixed in Zeeland branch


feat(metrics): report configuration push size for DB-less mode

What this PR does / why we need it:
Introduces metric for the size of config pushed by KIC to Kong DataPlane, below output from :10255/metrics endpoint of KIC with ingress_controller_configuration_push_size
# HELP ingress_controller_configuration_push_size The size of the configuration pushed to Kong, in bytes. `dataplane` describes the dataplane that was the target of the configuration push. `protocol` describes the configuration protocol (metric is presented for `db-less`, for `deck` it doesn't exist) in use. `success` describes whether there were unrecoverable errors (`false`) or not (`true`).
# TYPE ingress_controller_configuration_push_size gauge
ingress_controller_configuration_push_size{dataplane="https://<IP_ADDRESS>:8444",protocol="db-less",success="true"} 55
ingress_controller_configuration_push_size{dataplane="https://<IP_ADDRESS>:8444",protocol="db-less",success="true"} 55

analogical metric ingress_controller_fallback_configuration_push_size is introduced when the fallback mechanism is used.
Metrics are only presented for DB-less (declarative) mode because the whole configuration is pushed in this scenario. In DB-mode it happens partially so it's hard to define and track the size.

Which issue this PR fixes:
Part of #6659

Special notes for your reviewer:
Test case with push error and FallbackConfiguration enabled from TestMetricsAreServed has been run locally and it passes (I couldn't observe flakiness on my local machine), but it's disabled on CI - see https://github.com/Kong/kubernetes-ingress-controller/issues/6125
Coverage report https://github.com/Kong/kubernetes-ingress-controller/pull/6664#issuecomment-2473529381 highlights code that was not changed in this PR, why?

PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[x] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

The compilation problem was caused by concurrent merges of two last PRs. Fix: https://github.com/Kong/kubernetes-ingress-controller/pull/6667


export option for reporting/printing

For use in a lab class, it would be useful if there was any kind of pdf creator and/or copy-paste screen option or a way to print the processed spectrum

@hamed-musallam Please add an export button that allows to export as SVG. SVG being a vectorial format it will be high quality.
This button will simply take the content of the SVG element and save it.
There may be some other libraries to save the file. I used up to now: https://www.npmjs.com/package/file-saver

This download icon could be placed next to the full screen icon.

@lpatiny
this feature we need to discuss about, because any element not belong to basic svg elements will produce an error, And Also at some point specially in ranges and peaks and  integrals i have svg buttons to delete each one it and it will exported with svg we need to delete all these elements.

Indeed all the 'foreignObject' objects should be removed before SVG exportation. Do you see a simple way to achieve this ?

it's done

I just discussed with @targos that suggest a simpler method
The goal would be to add a property in the elements that should not be exported: data-no-export="true"  (all the properties starting with 'data-' are custom user properties https://developer.mozilla.org/en-US/docs/Learn/HTML/Howto/Use_data_attributes).
After the SVG being in the dom you can use something like:
var clone = svg.cloneNode(true)
var toRemove = clone.querySelectorAll('[data-no-export="true"]')
toRemove.forEach((element) => element.remove())
var svgString = svg.outerHTML

I'm pretty sure it doesn't work properly in MS Word because the SVG doesn't have a specified size (width and height are missing on the  tag)

Indeed adding width and height helps. I can import without problem in word 2016 on osx. Based on wikipedia SVG support is standard in office 2019 so I guess the students should update their software if they use old version of word.
Otherwise they should make a screen capture but they will have a lower resolution obviously. Maybe we could convert it to png using https://www.npmjs.com/package/canvg but again it will be pixels.

It seems that we have have a png we could copy it to the clipboard: https://web.dev/image-support-for-async-clipboard/

it's Implemented, you can  export the experiments as SVG, PNG , and also copy to clipborad

For some reasons, the copy to clipboard funtion does not work any longer. Also, if exporting an .svg file, this looks very strange when imported in MS word (it becomes black, see screenshot). Export as .nmrium and as .png do work flawlessly. However, at least the copy/paste would be important for the upcoming lab class (reports)

@lpatiny
the ClipboardItem not supported at Firefox this why we can not copy to clipboard. do you have any suggestion?
@nes-cgn
i tested it on Firefox version 81 and you can save it as png or svg and import it with out any problem, what the version of Firefox that you use?  can you update your Firefox to last release and test again but you still can not copy and past from Firefox but in Google Chrome all the export feature work well


I get bad scaling issues

I receive the following error all the time.
bad scale (nan); skipping
This is not an issue. I am not able to figure out how to set the scaling parameter for the image.In your code it says the scaling has to be 12.0.
Following are my images . For my application , usually the image input will be of this size.

Please point me in the direction how to resolve this issue.

Can you please give some more information:

What exactly are you trying to do?
What is your environment?
What are steps to reproduce the problem?

I am not able to figure out how to set the scaling parameter for the image.In your code

run gpageseg with --minscale=14.0
14.0 is just an example.


How to use my own credential?

I'm hitting Cloud Text-to-Speech API has not been used in project<PHONE_NUMBER>69 before or it is disabled from the example. and interesting enough,<PHONE_NUMBER>69 is not one of my projects in gcloud projects list.
If we follow the structure of other @google-cloud/* apis, we should be able to pass in our keyFile as config:
var config = {
  projectId: 'grape-spaceship-123',
  keyFilename: '/path/to/keyfile.json'
};
var datastore = require('@google-cloud/text-to-speech')(config);

but it doesn't seem to accept any parameter at the moment.

Greetings!  The default export of the module is a class type, which needs to be instantiated.  For example:
// Imports the Google Cloud client library
const textToSpeech = require('@google-cloud/text-to-speech');

// Create the auth config
const config = {
  projectId: 'grape-spaceship-123',
  keyFilename: '/path/to/keyfile.json'
};

// Creates a client
const client = new textToSpeech.TextToSpeechClient(config);

Hope this helps!

This should be included in the example

This should be included in the example 2x

Now is 2022 and such as in 2018 in example is missing auth https://www.npmjs.com/package/@google-cloud/text-to-speech Is it possible dear Google? How we could know that code from example new textToSpeech.TextToSpeechClient(); shout be new textToSpeech.TextToSpeechClient(config); ???


Do not trigger test discovery if an invalid interpreter is selected for the workspace

Identify whether it's an automatic or manual trigger.
Ensure we log this condition if it is automatically triggered, otherwise show the environment prompt.
On interpreter change, ensure discovery is triggered if interpreter is valid.
Ensure handler functions still get registered.


Widgets no longer works as sys.prefix is empty

non-cdn approach always fails

Upstream issue https://github.com/microsoft/vscode-python/issues/16355

I believe the fix is super simple

Validated. Not 100% sure on the validation needed here, but I deleted my CDN download and then ran a few complex widgets, looked good.


前端观光团 打卡

前端观光团 打卡

The issue title and OP are identical, and both google translate to "Front-end sightseeing group" - is there an actionable task for this issue?


Color modes changing in home assistant core

Logger: homeassistant.components.light
Source: components/light/init.py:1165
integration: Light (documentation, issues)
First occurred: 07:48:26 (2 occurrences)
Last logged: 07:48:26
None (<class 'custom_components.govee_lan.light.GoveLightEntity'>) sets invalid supported color modes {<ColorMode.RGB: 'rgb'>, <ColorMode.COLOR_TEMP: 'color_temp'>, <ColorMode.BRIGHTNESS: 'brightness'>}, this will stop working in Home Assistant Core 2025.3, please create a bug report at https://github.com/wez/govee-lan-hass/issues

Watching

same

same for mer

Please see:

https://github.com/wez/govee-lan-hass/issues/51


Second image wiggles in compare tool

https://github.com/T8RIN/ImageToolbox/assets/96973076/26992a48-6374-4fb6-be42-74fffd0c6c61

Yup, i know about that, but I can't do anything, because i use the library for this, and don't know how it works internally (

This one is fixed


[Mint] Lightning payments fro mint.minibits.cash fail with "Insufficient balance"

It is not currently possible to transfer funds using Minibits wallet from mint.minibits.cash. Payments fail with "Insufficient balance" even both lightning node and lnbits instance running the mint show correct balances.
Would appreciate if someone running lnbits could help. Funds are currently more SAFU then expected.

Solved by funding the wallet with amount equal channel_reserve. Appreciate quick help from LND dev slack team. Transfers from mint.minibits.cash now works.


Start using metallb for IPv6

Since https://github.com/istio/common-files/pull/489

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-multicluster-tests_istio
83b94277dcafad39d0090ee127680cecd398e53f
link
true
/test integ-pilot-multicluster-tests_istio

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

/test integ-pilot-multicluster-tests_istio


Fix the suggestion list count display

When you have limit=5, 3 results length for suggestions and 0 rendered, for example, then you only render 2 results.
because
rendered += suggestions.length; // 0+=3; rendered= 3
that._append(query, suggestions.slice(0, 5 - 3)); // that._append(query, suggestions.slice(0, 2));
When the lines are swape
that._append(query, suggestions.slice(0, 5 - 0)); // that._append(query, suggestions.slice(0, 5));
rendered += suggestions.length; // 0+=3; rendered= 3

dist not rebuilded

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


fix: export types from pdfjs-dist

🔗 Linked issue
fixes #6
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Added an export statement to the unpdf/pdfjs rollup plugin so that the types are exported and are usable by the library consumers.
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

I like it. Thanks!


Use an unique ID allocator for OpenFlow groups

OpenFlow Group IDs are Node global resources. Not only IPv4 and IPv6 specific OpenFlow rules could use Groups, but also IP family independent rules could use it. Diving the whole ID pool into two pools makes such use cases have to choose either IPv4 or IPv6 ID pool, which looks weird. Besides, there seems no good reason why two ID allocators should be created to manage the same pool.

/test-ipv6-all
/test-ipv6-only-all

/test-ipv6-all

/test-ipv6-conformance

/skip-all


Add multicast support

I started working on this, realized we should probably discuss the approach we want to use before I got too far, then kept going because I realized that most of the functional work here is valid, it's more of a question of organization and namespacing. Virtual nodes are ephemeral - they are not stored in the controller, so any time you want to act on a virtual node, you have to retrieve it by providing a list of node IDs that you want to multicast to. Similarly, virtual endpoints can only be retrieved after the virtual node is created. See here for reference: https://github.com/zwave-js/node-zwave-js/blob/master/packages/zwave-js/src/lib/controller/Controller.ts#L368-L384
The thoughts with the namespacing are to maintain the model that is being used by zwave-js. I imagine virtualnode.set_value to be an alternative command to use under the set_value service, perhaps controlled by a multicast/broadcast flag.

TODO: Update README with new commands - will do this once we agree on an implementation

Assuming we agree on this approach, I will update the README in the PR, but we should probably also wait until this is released before merging: https://github.com/zwave-js/node-zwave-js/pull/2671


feat: add vue 2.7 support

[x] useRoute, useRouter
[ ] useLink
[ ] onBeforeRouteLeave, onBeforeRouteUpdate
[x] NavigationFailureType, isNavigationFailure, START_LOCATION
[ ] update build script
[ ] update unit test

There are quite a few necessary changes for the composition api functions to properly work. They also need the upcoming Vue 2.7.9 version. I openend #3769 but this PR has other things that are not included like the type upgrades. Could you open another PR with just those changes?


classes become inline styles in chrome

in chrome, when you
add a css class to the whole text, delete all of it and start writing again
write a few lines, add a class to them, make a list of the
and probably other scenarios
the classes become inline styles..

This definitely seems to be an issue.
http://stackoverflow.com/questions/8332245/contenteditable-in-chrome-and-safari-replaces-css-classes-and-rules-with-inline
Will look into this.

Hah, just recognized that you're the author of that question on stackoverflow...


Continuous Monitoring of npm audit results

Tying the CI success to npm audit is not a good idea because the CI could then break without the source code having changed at all.
Instead have monitoring/alerting set up somehow (maybe as part of the CI script and a webhook/callout) or just as a sticker on the README.

Update 1: Decided against tying CI success to the audit outcome because it would just make it flaky and often impossible to fix since patches don't get rolled out immediately for all the vulnerabilities.
A contribution should still be acceptable if it has some npm audit issues that we can fix later on. The reasoning behind that is that the npm audit outcomes are fluid and time dependent so trying to mandate a positive outcome for npm audit is futile and does not itself guarantee the software to be secure, in fact the opposite is a well known secret: all software is insecure until proven otherwise (and the latter has been proven to be impossible in general).
So with all that in mind, retiring this issue.


update nightly version to 2024-09-01

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing in favor of https://github.com/awslabs/cargo-check-external-types/pull/190


Cache with unhashable arguments.

Background
For now it's forbidden to pass any un-hashable arguments to a CachedProvider.
lru_cache was used to count the variances of arguments. (It doesn't call the underlying function/method).
And lru_cache doesn't allow any unhashable arguments.
It's for preventing to call CachedProvider with different arguments from the first call, because CachedProvider at maximum,
remembers one result.
CachedProviders are always called by Factory, when dependencies are assembled, so it is not easy to see which arguments are used for each providers(calls).
For example, user might update some of providers before they compute the same result, so different arguments can be passed into the CachedProvider.
In this case it doesn't make sense to provide the previous result since the result might be different from what it should be.
Possible Solutions
However, if any dependencies are cached, it doesn't need to be hashable and used for lru_cache arguments counts, because they are guaranteed to be the same.
One solution can be check if any of arguments are/will be cached within the Factory instance.
And the other one is to introduce a Generic type called Cached and use it as a type-annotation.
Use-case examples
import scipp as sc
from typing import NewType
from beamlime.constructors import ProviderGroup, Factory

providers = ProviderGroup()
Row = NewType("Row", sc.Variable)
Rounded = NewType("Rounded", sc.Variable)

def provide_row() -> Row:
    return Row(sc.array(dims=['row'], values=[1.2, 2.3, 3.1]))

def round_row(row: Row) -> Rounded:
    return Rounded(sc.round(row))

providers.cached_provider(Row, provide_row)
providers.cached_provider(Rounded, round_row)
factory = Factory(providers)
factory[Rounded]

In this example above, it shouldn't matter if sc.Variable is hashable or not since it is cached anyways.

Fixed by #93


feature: perfection tracker for levels with best number of commands

This feature introduces the ability to track and record the user's best performance on each level. By storing both the completion status (solved) and whether the user achieved the best possible outcome (best), the application can provide more detailed feedback and track user improvement over time.

Oh cool feature! Do you have a screenshot of what the UI looks like?
My only worry is this part:
https://github.com/pcottle/learnGitBranching/blob/928577b2dc37586a0d5945d5bddd453c69c2a506/src/js/stores/LevelStore.js#L51-L57
Since we sync that map to local storage, I think this existing code would wipe out the existing solved map status.
I think we can fix that in the   isLevelSolved: function(levelID) { function @Patolord

A star would be cool!!
I added a fix for old users:
https://github.com/pcottle/learnGitBranching/commit/b3d0004427b55ec1f9250bcbb202fcc593bb0980
it works now :)
https://learngitbranching.js.org/?NODEMO&command=levels

its just a matter of changing react-ok-icon to react-star class on the  tag.
but im not sure how to implement the conditional styling.


Error running db import with a SQL file outside of working directory

Description
When running nitro db import, an error is output if the given path is anything but a file in the current working directory:
> nitro db import /Volumes/Dev/happylager/happylager.sql
Error checking if the file is compressed, error: open happylager.sql: no such file or directory

> cd /Volumes/Dev/happylager
> nitro db import happylager.sql
Detected a "mysql" backup file...
...

Additional info

Nitro version: 1.1.1
Multipass version: 1.5.0

I have identified this issue and need to use filepath#Clean on the argument to get the corrected path.
Will get this pushed up soon.


fedekunze/1585-added LiCoin for PageWallet

Closes #1585
Screenshot:

❤ Thank you!

[x] Added entries in CHANGELOG.md with issue # and GitHub username
[ ] Reviewed Files changed in the github PR explorer

I updated the test according to the warnings, should I split this PR ?

this is great @fedekunze ! please remove the "warn" from package.json!


Fix typos

Fix typos.
Start descriptions of all options with a capital letter (for consistency).

Thanks! I'll recompile the website to update these on there as well. -Ross


Portfolio: Remove "APY" from Borrow positions

Overview
We should only show the Borrow APY for the borrow positions, rather than both APY and Borrow APY

Done.


Feature request: error if there are still imports

Here is my current configuration:
    resolve({
      modulesOnly: true,

      // Whitelist imports to assert that we only inline things that we've
      // injected the licenses for.
      only: [
        'indefinite-observable',
        'wobble',
        'symbol-observable',
        'fast-equals',
       ],
    }),

I'm interested in adding a check that will throw if there are still import statements in the resulting bundle (e.g. exhaustive: true).  The purpose of such a check would be to require a manual intervention before adding any dependencies to a published bundle: in this case, to ensure that the correct licenses have also been included.
Would you be amenable to a patch to support this, or should I make a separate plugin?

+1 would really like to have this as well

Any updates? Is still need this feature?
(skip was dropped it previous releases)

Hey folks (this is a canned reply, but we mean it!). Thanks to everyone who participated in this issue. We're getting ready to move this plugin to a new home at https://github.com/rollup/plugins, and we have to do some spring cleaning of the issues to make that happen. We're going to close this one, but it doesn't mean that it's not still valid. We've got some time yet before the move while we resolve pending Pull Requests, so if this issue is still relevant, please @ me and I'll make sure it gets transferred to the new repo. :beer:


Printing

Not sure if this is a problem with this website or just how chrome works. But, when I try and print my final resume, in the header and footer of the pdf (print to pdf), I get the link to the resume builder website and page number (1/1) as well as the date. Changing the printer does not help.
What can I do to remove this?
Edit: This is what it looks like:

The app has now moved from the regular printing of the page system to exporting generated PDFs, so the Ctrl + P method is not really supported anymore. Instead, you could go to the Actions Tab and click on Save as PDF, or click on the Save button in the Floating Bar below. Please let me know if that helps, thank you :)

perfect! that worked beautifully


"Vințu de Jos" is listed in the drop-down menu

Description:
"Vințu de Jos" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Vințu de Jos" in the search bar.

Expected results
The location is listed in the drop-down menu as "Vințu de Jos, Alba (Vințu de Jos)".


Add Indentation guide in editor

Issue: https://github.com/cpeditor/cpeditor/issues/413
Here is the screenshot after adding indent guide

This PR is still in WIP. If I load big source file the editor hung. We need to paint only the visible area. Currently we are painting the whole region.

It seems that you added #22 in this PR :thinking: Is that a mistake?

@ouuan
Thanks for notifying. I will fix this tomorrow.

I will create new PR.


get_helm.sh fails on verifying checksum

Downloading https://get.helm.sh/helm-v3.6.2-linux-amd64.tar.gz
Verifying checksum... SHA sum of /tmp/helm-installer-AHik8q/helm-v3.6.2-linux-amd64.tar.gz does not match. Aborting.
Failed to install helm
For support, go to https://github.com/helm/helm.
./get_helm.sh
Helm v3.6.2 is available. Changing from version v3.2.4.
Downloading https://get.helm.sh/helm-v3.6.2-darwin-amd64.tar.gz
Verifying checksum... SHA sum of /var/folders/gn/ms6vk7yd7h5_s7kzt9hjqtfr0000gn/T/helm-installer-XXXXXX.BTGvXFP1/helm-v3.6.2-darwin-amd64.tar.gz does not match. Aborting.
Failed to install helm
For support, go to https://github.com/helm/helm.

This has been fixed. There was a slight hiccup during the release. Thanks for the report.


Add fluttert to android as a module, the willpopscope not working

Steps to Reproduce

I add onKeyDown and onKeyUp method of FlutterView to activity :
  override fun onKeyDown(keyCode: Int, event: KeyEvent?): Boolean {
        return flutterView.onKeyDown(keyCode, event)
    }

    override fun onKeyUp(keyCode: Int, event: KeyEvent?): Boolean {
        return flutterView.onKeyUp(keyCode, event)
    }

But the onWillPop of WillPopScope still not been called

flutter doctor -v :
# flutter doctor -v
[√] Flutter (Channel stable, v1.2.1, on Microsoft Windows [Version 10.0.17763.379], locale zh-CN)
    • Flutter version 1.2.1 at E:\Development\Flutter
    • Framework revision 8661d8aecd (5 weeks ago), 2019-02-14 19:19:53 -0800
    • Engine revision 3757390fa4
    • Dart version 2.1.2 (build 2.1.2-dev.0.0 0a7dcf17eb)

[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at E:\Development\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = E:\Development\Android\sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
    • All Android licenses accepted.

[!] Android Studio (version 3.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    X Flutter plugin not installed; this adds Flutter specific functionality.
    X Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[√] VS Code
    • VS Code at C:\Users\Administrator\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 2.24.0

[√] VS Code, 64-bit edition (version 1.31.1)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 2.24.0

[√] Connected device (2 available)
    • vivo X9                      • 4b89e034      • android-arm64 • Android 7.1.2 (API 25)
    • Android SDK built for x86 64 • emulator-5554 • android-x64   • Android 9 (API 28) (emulator)

! Doctor found issues in 1 category.

Due it in onBackPressed method:
override fun onBackPressed() {         flutterView.popRoute()     }


[Bug] ScrollEvent rather than CommandEvent for Slider widget [sf#97]

Reported by bana-sea- on 2006-05-16 17:07 UTC
When generating C++ code with 0.4 wxGlade, Slider's
event handling sets up the callback functions with a
CommandEvent as the input argument, rather than the
ScrollEvent which it should be.
Should be an easy fix.

Commented by agriggio on 2006-10-07 08:43 UTC
Logged In: YES
user_id=375945
I'm starting working on it now (sorry for the long delay,
very busy times here...). But in the meantime, since
wxScrollEvent is a subclass of wxCommandEvent, you can
simply static_cast<> the parameter to the right type...
Regards,
Alberto


[Flink-17030] Add primary key syntax to DDL

What is the purpose of the change
Supports primary key and unique key syntax to sql-parser and blink-planner.
Brief change log

Add primary key and unique syntax to sql parser
Add primary key support for CREATE TABLE for blink-planner
Add primary key support for ALTER TABLE for blink-planner

Verifying this change
Added UTs.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? not documented

cc @dawidwys , can you review this for me if you have time, thanks ~

I reported the unrelated CI failure here: https://issues.apache.org/jira/browse/FLINK-17260


class facade 'ignitionserviceprovider' not found

i am a learner in laravel and when ever i run php artisan serve. it shows me this.
how can i run it without this error ..?

This appears to be an autoloading issue with Composer and Laravel, and is not relevant to this library. Try posting this question on Stack Overflow or following the Laravel quickstart


jump to location from search result

This has been solved in oce-editor-tools, still to be tested in gateway-translate.

This is now included in the latest gT PR


Speech recognition without constraints.

I am trying to speech recognition without any constraints.
here:
https://github.com/Microsoft/Windows-universal-samples/blob/master/Samples/HolographicVoiceInput/cpp/HolographicVoiceInputMain.cpp#L213-L215
they have constraints.
I tried to remove these line, but the code does not recognize the speech at all!
does the handler stay the same handler, ContinuousRecognitionSession->ResultGenerated  ?!
if not, wich handler I should change ?!
or maybe my problem isn't the handler !?
thank you

If you're trying to get speech dictation (unconstrained speech), you're going to want to look at the example in the regular c++/XAML app:
https://github.com/Microsoft/Windows-universal-samples/blob/master/Samples/SpeechRecognitionAndSynthesis/cpp/Scenario_ContinuousDictation.xaml.cpp#L100-L101
This shows how to initialize dictation with a topic constraint, which will allow free-form speech to text recognition instead of keyword recognition.
The Hololens samples don't include this because rendering text in the hololens device is beyond the scope of the sample.

than you.
I did all what you say before.
my problem is that the handlers did not start when I start to speek.
not Hypothesis nor resultGenerated.

Okay, first up, make sure you've got the InternetClient permission requested in your appx manifest, it's required for dictation to initialize.
Second, register for the ContinuousRecognitionSession->Completed event handler. Likely, what you're seeing is that while Dictation starts up, it times out relatively quickly (Default, I believe, is 5 seconds of silence.) Continuous Dictation isn't able to run indefinitely, because it utilizes an internet connection to access external dictation services.
You can increase various timeouts by modifying the SpeechRecognizer::Timeouts struct. Details on the various properties are available here: https://docs.microsoft.com/en-us/uwp/api/windows.media.speechrecognition.speechrecognizertimeouts

thank you !! it works now !! the  InternetClient permission was missed 👍

Glad to hear that, good luck!


🐛 [firestore-send-email] Attachments not being attached

[READ] Step 1: Are you in the right place?
Issues filed here should be about bugs for a specific extension in this repository.
If you have a general question, need help debugging, or fall into some
other category use one of these other channels:

For general technical questions, post a question on StackOverflow
with the firebase tag.
For general Firebase discussion, use the firebase-talk
google group.
To file a bug against the Firebase Extensions platform, or for an issue affecting multiple extensions, please reach out to
Firebase support directly.

[REQUIRED] Step 2: Describe your configuration

Extension name: firestore-send-email
Extension version: 0.1.27
Configuration values (redact info where appropriate):

SMTP connection URI: smtp://app@redacted.com:redacted@smtp.office365.com:587
Email documents collection: mail
Default FROM address: Redacted<EMAIL_ADDRESS>Firestore TTL type: never
Firestore TTL value: 1

[REQUIRED] Step 3: Describe the problem
Attachment files are not being included with emails. The email is sent and received without error, but there is simply no attachment. I am uploading the file to Google Storage and passing the file's download URL in the "path" attribute. This had been working in the past, so I assume an extension update broke something. I have tried using the "href" attribute instead of "path" but it did not remedy the problem. Again, there are no error messages, simply no attachments.
Steps to reproduce:
What happened? How can we make the problem occur?
This could be a description, log/console output, etc.

Upload a file to Google Storage.
Retrieve the file's download URL.
Create an email document with a message and an attachment, specifying the original filename, and specifying path or href as the URL.
Open the received email.

Expected result
Received email contains message and attachment.
Actual result
Received email contains only message, no attachment.

Hello @gruppler,
I have just tested this out, and I received the file attachment. Could you please provide the snippet of the code that you used to send the email that also includes the attachment?

Thank you for the reply. Here is my generalized function for sending emails:
export const SEND_EMAIL = async function (
  { dispatch, state },
  { to, cc, replyTo, subject, message, attachments }
) {
  let mailRef;
  let data = {
    createdBy: state.user.id,
    to,
    message: {
      subject,
      html: message,
      text: formatPlainText(message),
    },
  };
  if (cc) {
    data.cc = cc;
  }
  if (replyTo) {
    data.replyTo = replyTo;
  }
  if (attachments) {
    data.attachments = attachments;
  }
  mailRef = await db.collection("mail").add(data);
  // ...
}

Here is an example of its usage:
// ...

let attachments = cloneDeep(attachments).map((doc) => ({
  filename: doc.file.name,
  href: doc.file.url,
}));

await dispatch("SEND_EMAIL", {
  to: customerEmails,
  cc: userEmail,
  replyTo: userEmail,
  subject,
  message,
  attachments,
});

What is most confusing to me is the fact that this code used to work, and now it doesn't, though it hasn't changed.

@gruppler This seems to be the problem.

if (attachments) {
data.attachments = attachments;
}

attachments is a property inside the message field, hence why you are receiving an email with the message but no attachment. Therefore is should be:

if (attachments) {
data.message.attachments = attachments;
}

Thank you so much for helping me! I tracked down the commit that broke this and it was much further back in time than I thought!


need sample/tutorial to play YouTube videos.

The changelog confirms that this library is capable of playing YouTube videos, but then the sample module shows no example of it.
There are some YouTube video ids in VideoSource.java, but the class is pretty much unused.

@hendraanggrian So sorry, it was in 2.0 RC-4 and no longer work. Youtube playback API is really un-friendly so I need to hold it back. If you are capable of getting the stream URL, playing that stream is just the same with any other Video IMO. Further investigation on Youtube API TOS may need.

@eneim thank you for clarifying. I imagine it is possible to retrieve the stream url by loading the webpage in some WebView, parse the html with Jsoup to scrap the actual stream url. It is not a very efficient solution but it might be the only working one at least until YouTube is officially supported again.
Once again, thank you for this fantastic library!

@hendraanggrian Thanks. You'd better take care of Youtube TOS before any further implementation. They have strict requirement on the content. That is one factor that prevents me to go further.


Add an "About Us" page

Description
Currently About Us page is directly redirecting GitHub page. I want to add a separate About Us page with all the information given in the Git Hub.
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the [Contributing Guidelines]
[X] I am willing to work on this issue (optional)
[ ] I am a GSSoC'23 contributor

Great, I was thinking of updating it already. You can start working on it. The details will be similar to shown in the README.md file.


Ryan M: Watch bestboxingshoes.com

Ryan M requests the watch of the watch_keyword bestboxingshoes\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

bestboxingshoes\.com has been seen in 1 true positive, 1 false positive, and 0 NAAs.

Approved by Mithical in Charcoal HQ


upgrade DHT11/DHT22 backing library

Old DHT11/DHT22 backing library Adafruit_DHT is out of maintained and reporting error on my rpi4+python311.
 ImportError: cannot import name 'Raspberry_Pi_2_Driver' from 'Adafruit_DHT'
And it has CPU usage hogging problem (~25%).
Upgrade DHT11/DHT22 backing library to adafruit-circuitpython-dht

Haven't checked that already, but why did you changed mqtt_io/modules/sensor/bmp085.py for the DHT22/11?
If this is another (new) sensor you should seperate your Pull Requests.

Yes, it's a combo PR, I will make it seperated, thanks.

Thanks!

What's happend with this pull request? The current DHT22 code isn't functional and it looks like this pull request would fix that, it just needs the code for bmp085 separated out. I can submit a new pull request just for DHT22 if that would be helpful?

I would highly appreciate that!


Optional chaining not transpiled properly

Hi there,
I'm completely stuck by enabling jest with swc because of an issue with optional chaining not being transpiled properly.
To reproduce the error, you can place in a test:
const foo = [1, 2, 3];
console.log(foo[2], foo?.[2]);

The result of that console log is 3, and undefined.
I think the problem is within this plugin as I already use swc to build my app and I checked the compiled javascript, which result in a correct working output.
There was recently an issue with swc that was not transpiling the optional chain.
So I also tried to upgrade the swc, cleared all the cache for swc and jest, and tried again with the same result.

I think there is nothing to do with optional chaining in this plugin.

Seeing the same. When using this plugin with the latest version of swc/core, I still see the optional chaining bug.

@lughino I'm not certain we're seeing the same problem but since upgrading SWC to 0.79 in our project we're getting a bunch of invalid utf8 character detected: FromUtf8Error errors from jest. Have you seen anything of that nature?

OK, an update: I can confirm that swc, when used as the transformer for jest unit tests in conjunction with jest_workaround, is exhibiting the same failure to transpile optional chaining we saw before we upgraded swc from 0.76 to 0.79 to resolve this issue.
An example:
return foo() && window?.document?.createElement('webview')?.src === '';

Is transpiled to:
return foo() && ((_this = this) === null || _this === void 0 ? void 0 : _this.) === '';

Throwing SyntaxError: Unexpected token ')'
I wonder, is all that's needed to fix jest_workaround to bump from 0.76 to 0.79?

Renovate out here doing gods work: https://github.com/magic-akari/jest_workaround/pull/52/files
Let me know if this would also require bumping this dependency and I can throw a branch together for merge?

This ^ seems to be the most likely solution @magic-akari 🙏

Oh, nvm. Seems this is already working with the latest version. Thanks!


sensitive_environment and triggerless_environment still appear in readme.md

Looks like they have been removed for version 0.4.0
I copied the module "shell_resource_hello" example and did a terraform apply
Error
greg@CM-LT-0007 external-0.4.0 (main *%)]$ tf apply
╷
│ Error: Unsupported argument
│
│   on module.tf line 47, in module "shell_resource_hello":
│   47:   sensitive_environment = {
│
│ An argument named "sensitive_environment" is not expected here.
╵
╷
│ Error: Unsupported argument
│
│   on module.tf line 52, in module "shell_resource_hello":
│   52:   triggerless_environment = {
│
│ An argument named "triggerless_environment" is not expected here.
╵

Thank you for pointing this out. These variables have been renamed to environment_sensitive and environment_triggerless, respectively.
The updated README is released in v0.4.1.


Make NIOWebSocketServerUpgrader Sendable

Motivation:
The NIOWebSocketServerUpgrader is marked as not being Sendable, but it has no mutable state and makes no assumptions about the event loop it is invoked on, so it should be Sendable. The two user-provided callbacks which are injected in init are marked as being Sendable but require Swift 5.7 to express this safely.
Modifications:

Add @unchecked Sendable conformance to NIOWebSocketServerUpgrader

Result:

NIOWebSocketServerUpgrader is Sendable.

@glbrntt what's the motivation for this? Just because the current implementation can be Sendable should mean that this should be Sendable. In general, ChannelHandlers aren't sendable and this will require all future implementations to be stateless too.

We have used it in our NIOWebSocketServer example as if it was already Sendable because we share it between all channels:
https://github.com/apple/swift-nio/blob/558e4f2fb854dacc22d1774e7b6528d06a4667ad/Sources/NIOWebSocketServer/main.swift#L203-L224
Making it non-Sendable would break this example and any code that is in the wild using it like we have recommended since its inception.


[feature] Explore ent cache

Is your feature request related to a problem? Please describe.
The ent has a cache https://entgo.io/blog/2021/10/14/introducing-entcache/
Describe the solution you'd like
Explore if it is a viable option

@naveensrinivasan thanks for the issue :+1:
While working on some performance issue with Ent some weeks ago, I looked into this cache and I gave it a quick try with no big success.
I didn't dig deeply into the issue but entcache v0.1.0 is based upon ent v0.11.2 but ent has already reached v0.12.4 so it looks like entcache isn't maintained that much right now.

@naveensrinivasan thanks for the issue 👍
While working on some performance issue with Ent some weeks ago, I looked into this cache and I gave it a quick try with no big success.
I didn't dig deeply into the issue but entcache v0.1.0 is based upon ent v0.11.2 but ent has already reached v0.12.4 so it looks like entcache isn't maintained that much right now.

Thanks for the update! I appreciate the details.
So do you think we should close this issue?

Maybe worth asking to the entcache team if they have any plan to keep maintaining it.
I can do it and then we can decide what to do with this issue based on their reply.

https://github.com/ariga/entcache/issues/32

After almost two months with no answer, I think it's fine to close this


Dose tool support webpack 2?

Currently i use webpack-isomorphic-tools and webpack 1 config
Can i upgrade to webpack 2?

I'm using webpack 2.
Also universal-webpack works with webpack 2.


add backend lambda that handles the API

Add the backend logic in lambda py to handle the API calls

Also just a suggestion (maybe something for a future PR if you want), but it would be awesome to have type hints so it's easier to understand the type of the data being passed around


Add java Bubble sort algorithm

Create Java Folder and add bubble sort algorithm in algorithm folder

@ADITIBAGADDEO22  assign this to me .


Sass Error @import ionicicons !!!!

Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x ] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[ x] bug report
[ ] feature request
Please do not submit support requests or "How to" questions here. Instead, please use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

Expected behavior:

Steps to reproduce:

Related code:

 sass: ...ASR/mobile apps/CLUBS/HSC/mobile5/node_modules/ionic-angular/fonts/ionicons.scss, line: 9
            File to import not found or unreadable: ionicons-icons. Parent style sheet: D:/EDUMASR/mobile
            apps/CLUBS/HSC/mobile5/node_modules/ionic-angular/fonts/ionicons.scss

       L9:  @import "ionicons-icons";
      L10:  @import "ionicons-variables";

Error: Failed to render sass to css
    at BuildError.Error (native)
    at new BuildError (D:\EDUMASR\mobile apps\CLUBS\HSC\mobile5\node_modules\@ionic\app-scripts\dist\util\errors.js:16:28)
    at Object.callback (D:\EDUMASR\mobile apps\CLUBS\HSC\mobile5\node_modules\@ionic\app-scripts\dist\sass.js:210:24)
    at options.error (D:\EDUMASR\mobile apps\CLUBS\HSC\mobile5\node_modules\node-sass\lib\index.js:294:32)

Hello! Thank you for opening an issue with us!
Please include the following information:

the output from ionic info
a github repo with a sample project that can be used to duplicate the issue
steps to duplicate perhaps (for example, did you use a specific starter? did it happen right out of the box or did you change something and then got this issue?

I have several projects going for various purposes and none of them are exhibiting this issue, so I am going to need more information in order to attempt to duplicate this.
One item I see is that you are using Windows and have a space in your working directory. I have seen this combination cause issues with some of the tools before. You could try cloning your app in a path where there are no spaces and see if that alleviates your issue (just a stab in the dark at this point, but something to try).
Thank you for using Ionic


NSInternalInconsistencyException on Collection View Updating

This fixes #3 and ReactiveKit/ReactiveKit#11.
The .gitmodule file needed to be removed in order for me to clone the repository, as I don't have access to the submodules. @srdanrasic, I don't know of a better way to handle that...

Thanks @AnthonyMDev! Now I'm very puzzled why this fixed it :)

Does the issue occur only when moving/reordering items? I'm having trouble reproducing this.

This is caused by a bug in UICollectionView. There isn't a really great explanation of the cause out there. This is the best information I can show you.
http://stackoverflow.com/questions/18796891/uicollectionview-reloaddata-not-functioning-properly-in-ios-7

Playing further with this... It seams that in the implementation of collectionView:moveItemAtIndexPath we should not trigger any events because collection view already laid out itself to the new state and we should just update our data. ObservableCollection currently does not support modifying the collection without triggering the events...
I'm thinking of adding a way to do silent updates of the observable collection, like:
array.silentUpdate { proxy in
  proxy.moveItemAtIndex(0, toIndex: 3)
}

@srdanrasic I've updated the PR to use 'MainExecutionContext'.
Adding a 'silentUpdate' method is not a bad idea!

Added this manually so I'll close this. Available in v1.0.10.
Can you check if it still crashes? I guess you should now do the collectionView:moveItemAtIndexPath  updates within the silentUpdate...

@srdanrasic This is causing me issues again. When we had updated to use MainExecutionContext, everything seemed to be working. Once you made the updates to use Queue.main.context, it broke again...

Never mind, I had another issue that was causing this. This code looks good. Thanks @srdanrasic !


bug: pylsp signature help vs cmp documentation

Did you check docs and existing issues?

[X] I have read all the noice.nvim docs
[X] I have searched the existing issues of noice.nvim
[X] I have searched the exsiting issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.9.0-dev-825+g843c9025a
Operating system/version
6.1.8-arch1-1
Describe the bug
Enabling rope_completion plugin causes cmp docs to lose highlighting.
In both cases signature help has proper highlighting.
Let me know if this is a pylsp/nvim-cmp issue and I can open an issue there instead.
Signature help

Cmp documentation with rope_completion disabled

Cmp documentation with rope_completion enabled

Steps To Reproduce

Go to line 102 in minimal config
Toggle enabled and see different behavior

Expected Behavior
Highlighting in cmp documentation
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")
-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
    vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
    vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath })
end
vim.opt.runtimepath:prepend(lazypath)

-- options
vim.opt.termguicolors = true
vim.opt.completeopt = ""
vim.opt.wildmenu = false

-- install plugins
local plugins = {
    "nvim-lua/plenary.nvim",
    "folke/tokyonight.nvim",
    {
        "nvim-treesitter/nvim-treesitter",
        build = ":TSUpdate",
        event = "BufReadPost",
        config = function()
            require("nvim-treesitter.configs").setup({})
        end,
    },
    {
        "hrsh7th/nvim-cmp",
        dependencies = {
            "hrsh7th/cmp-nvim-lsp",
        },
        event = { "InsertEnter", "CmdlineEnter" },
        config = function()
            local cmp = require("cmp")
            cmp.setup({
                completion = {
                    completeopt = "menu,menuone,noselect",
                },
                sources = {
                    { name = "nvim_lsp" },
                },
                mapping = {
                    ["<C-p>"] = cmp.mapping.select_prev_item(),
                    ["<C-n>"] = cmp.mapping.select_next_item(),
                },
            })
        end,
    },
    {
        "neovim/nvim-lspconfig",
        dependencies = {
            "williamboman/mason.nvim",
            "williamboman/mason-lspconfig.nvim",
        },
        event = "BufReadPre",
        config = function()
            require("mason").setup()

            -- install mason packages
            local show_ui = true
            local packages = {
                "python-lsp-server",
            }
            for _, package_name in ipairs(packages) do
                local package = require("mason-registry").get_package(package_name)
                if not package:is_installed() then
                    if show_ui then
                        vim.cmd("Mason")
                        show_ui = false
                    end
                    package:install()
                end
            end

            require("mason-lspconfig").setup()

            require("lspconfig").pylsp.setup({
                on_attach = function(client, bufnr)
                    -- Disable semantic highlighting by server
                    -- sumneko lua highlights comments when toggling
                    local servers_disable_semantic_highlighting = { "sumneko_lua" }
                    if vim.tbl_contains(servers_disable_semantic_highlighting, client.name) then
                        client.server_capabilities.semanticTokensProvider = nil
                    end

                    vim.keymap.set("i", "<C-s>", function()
                        vim.lsp.buf.signature_help()
                    end, { buffer = bufnr })
                end,
                capabilities = require("cmp_nvim_lsp").default_capabilities(),
                handlers = {},
                flags = { allow_incremental_sync = true, debounce_text_changes = 150 },
                settings = {
                    pylsp = {
                        plugins = {
                            rope_completion = {
                                enabled = true,
                                eager = true,
                            },
                        },
                    },
                },
            })
        end,
    },
    {
        "folke/noice.nvim",
        dependencies = {
            "MunifTanjim/nui.nvim",
            {
                "rcarriga/nvim-notify",
                lazy = false,
                config = function()
                    local notify = require("notify")
                    notify.setup({
                        timeout = 3000,
                    })
                    vim.notify = notify
                end,
            },
        },
        event = "BufReadPost",
        config = function()
            require("noice").setup({
                notify = { enabled = true },
                cmdline = { enabled = true },
                messages = { enabled = true },
                lsp = {
                    progress = {
                        enabled = true,
                        -- view = "mini",
                    },
                    documentation = {
                        opts = {
                            border = { style = "rounded" },
                            position = { row = 2 },
                        },
                    },
                    signature = { enabled = true },
                    hover = { enabled = true },
                    override = {
                        ["vim.lsp.util.convert_input_to_markdown_lines"] = true,
                        ["vim.lsp.util.stylize_markdown"] = true,
                        ["cmp.entry.get_documentation"] = true,
                    },
                },
            })
        end,
    },
}
require("lazy").setup(plugins, {
    root = root .. "/plugins",
    defaults = { lazy = true },
    ui = { border = "rounded" },
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

I'm not 100% I understand what's missing and I don't know pylsp that well.
But if it's the type thing on top you're missing, then you might set ["cmp.entry.get_documentation"] = false and see if that fixes what you think should be fixed.
Most lsp servers add a redundant title to the doc and setting that to true removes that title. But that might not always be the case

Not what I was referring to but I think it may be a pylsp issue.
If you look at the last picture, the documentation window lacks highlighting compared to the 2 others above.
I get highlighting when using pyright.

The only difference between the two screenshots that I can see is the title line right?
The part below that has the same highlighting.

There are 3 screenshots. The last one is the different one.

Please read what I mentioned and look closely at your screenshots. The third screenshot has the exact same highlighting as your second one, but is missing the first print( statement.

I was talking about this highlighting.
But I just realized that the documentations aren't even the same, thanks for pointing that out.
Anyways  not a noice issue I think.

Come on. Just look at the contents of the two screenshots you just posted??
The 'print( in your second screenshot is also not hightligted in your first screenshot.
Your first screenshot has an extra print( statement.

Your first screenshot has two print functions. the second one only one

Set ["cmp.entry.get_documentation"] = false

Ah ok I see what you mean now.
In the first two the lines that have highlighting is the function signature.
In the third one the function signature is not included in the documentation.
Below is with ["cmp.entry.get_documentation"] = false. Unfortunately still missing the function signature.

ok, then that's just pylsp that's not returning that. Nothing Noice nor cmp can do about that


Fix commit names

[x] Begin with a capital letter
[x] Fix wrong emojis like :sparkles

We didn't fix the commits older than March 31th 2019.


212 - Case categorisation

Changes in this PR

Added ability for a case to be recategorised by the user
Refactored Support::Category model to be a tree structure to support further nesting of categories in the future.
Removal of Support::SubCategory

Screenshots of UI changes

 Display of Category / SubCategory

Is it ok for govuk-radios__divider to force Office supplies onto two lines?

We need to add a missing slug to the appropriate YAML category entry for mfd, otherwise CaseCategorisationForm will error.

Hi Peter can you explain this a bit more for me please, I don't understand.

@ryantk Is it ok for .govuk-radios__divider to force Office supplies onto two lines?

good spot

We need to support two categories when this is merged and we use the slug to marry up, but the seed data in Supported does not currently have an mfd slug to achieve this. Cases for catering will work but there other won't and will cause a 500 error because category_id will be nil.

I'll approve the requested change so it doesn't block you if I get called away.

We need to support two categories when this is merged and we use the slug to marry up, but the seed data in Supported does not currently have an mfd slug to achieve this. Cases for catering will work but the other won't and will cause a 500 error because category_id will be nil.

I get it now. I actually spotted this myself after you put it and my fix above will fix it. I have added mfd slug into the categories seeds too :)


Removed unnecessary references to params to avoid inheritance issues

Pretty much that.  None of the sub classes should be called directly so $unbound:: is already in scope.  By calling or referring to unbound::params values defined when calling the unbound class are forgotten about. (e.g. an alternative $config_file).

the rest looks good imo, :+1:

@t0mmyt Are you able to address the comments here?

@t0mmyt ping

Closing PR due to lack of response.  If someone else wants to pick this work up, PRs welcome.


Load new configuration level and TAG at RunTime

I use tinylog 2.5 in my configuration.
We need to change TAG, and severity log at RUNTIME, because the application is big, we can t log everything, and we just need to log something by tag for a specific request for a specified time.
I look issue #100, #102, #157, #192 and documentation https://tinylog.org/v2/extending/#custom-logging-provider
But that doesn't work in specific case.
Example, if TAG TEST is not in configuration file, and it's call, they are nothing in file. Normal.
I change and add tag TEST after and I launch reload fonction. Tinylog, doesn't log information for TEST.
If i wait for tomorrow it's work, if i restart application it's work to.
But we need that's work at reload function already
public class ReconfigurableLoggingProvider implements LoggingProvider {

        private TinylogLoggingProvider realProvider = new TinylogLoggingProvider();

        public void reload() throws InterruptedException, ReflectiveOperationException {
            realProvider.shutdown();

            Field frozen=Configuration.class.getDeclaredField("frozen");
            frozen.setAccessible(true);

            Method method = Configuration.class.getDeclaredMethod("load");
            method.setAccessible(true);

            frozen.setBoolean(null, false);
            Map config=(Map) method.invoke(null);

            Configuration.replace(config);

            realProvider = new TinylogLoggingProvider();

            frozen.setBoolean(null, true);
            method.setAccessible(false);
        }

        public ContextProvider getContextProvider() {
            return realProvider.getContextProvider();
            // dans la doc officiel
            // a NopContextProvider is registered,
            //which is part of tinylog’s API artifact and simply ignores all registered context values
            //new NopContextProvider();
        }

        public Level getMinimumLevel() {
            return realProvider.getMinimumLevel();
        }

        public Level getMinimumLevel(final String tag) {
            return realProvider.getMinimumLevel(tag);
        }

        public boolean isEnabled(final int depth, final String tag, final Level level) {
            return realProvider.isEnabled(depth + 1, tag, level);
        }

        public void log(String loggerClassName, String tag, Level level, Throwable exception, MessageFormatter formatter, Object obj, Object... arguments)
        {
            realProvider.log(loggerClassName,tag,level,exception,formatter,obj,arguments);
        }

        public void log(int depth, String tag, Level level, Throwable exception, MessageFormatter formatter, Object obj, Object... arguments) {
            realProvider.log(depth + 1, tag, level, exception, formatter, obj, arguments);
        }

        public void shutdown() throws InterruptedException {
            realProvider.shutdown();
        }
}

Can you help me to add this functionnality ?

I have just created a working example project: https://github.com/tinylog-org/tinylog-reload-configuration-example
Please have a look if it solves your issues. I assume that the cause for issues is that the TaggedLogger instances are ignoring your tag configuration changes. If the example project or its documentation is unclear or won't solve your issues, please let me know. I will be happy to improve!

Globaly that doesn't work, and it's worst than i did, with now multi bug
I explain, i start my configuration with 2 TAG : TOTO and TEST
Il try log TEST1 -> it is not write -> OK
I add TEST1, i reload, launch log -> OK
I add TEST2, i reload, launch log -> OK
I add TEST3, remove TEST1, TEST2, launch log TEST 3 -> OK
I launch TEST2 -> it is not write -> OK
I add TEST2, reload, launch log -> KO !!!
And after this point i test so much configuration.
TEST3 work every times, i add TEST10 work in start and after not. And finaly i add all tag and now il launch with TEST1 and write in log TEST3 for tag !

Can you adapt https://github.com/tinylog-org/tinylog-reload-configuration-example with your configuration for reproducing the bug? Afterwards you can give me the code and I will have a look what is the cause.

I give you all configuration file and test.
Number is version off properties
1 - try write TAG: TEST1 -> no log -> OK
1 - try write TAG: TEST -> log -> OK
2 - after reload and del TAG TEST and ADD TEST1, try write TAG:TEST1 -> no log -> KO
2 - try write TAG:TEST -> no log OK
3 - I configure TAG: TOTO, TEST, TEST1, and reload configuration
3 - try write TAG:TEST1 -> no log -> KO
3 - try write TAG TEST -> lo -> OK
tinylog2.zip

The bug what i can reproduce is :  try to log with tag was not in configuration in FIRST.
After, add TAG in configuration, reload configuration, and try to log this TAG again -> result doesn't work.
I reproduce this with many TAG every time.

## fichier utilisé pour la configuration en DEV en date du 20220929
## /!\ ne pas configurer 2 writter sur le même fichier cela produit des incohérence

provider = reconfigurable

# Register the BlackHoleWriter to ensure that disabled tags and severity levels will be not optimized away
writerTag = black hole

## Writers can be executed in a separate thread.
## The advantage is that the application itself will not be blocked by slow IO operations.
## The writing thread runs with low priority and shuts down automatically, together with the application.
#writingthread = true # optional, default: false

writerTag.format = {date:yyyy-MM-dd HH:mm:ss} - TAG : {tag} - {level} : {message}

## Shared File allows multiple instances of an application to write to the same log file simultaneously
#writerTag = shared file
writerTag = rolling file

## Séparer les différents TAG par ", " (virgule et espace)
writerTag.tag = TOTO, TEST

## valeur : trace, debug, info, warn, and error
writerTag.level = trace

## file of log example : writer.file = /{date:yyyy}/{date:MM}/log-{pid}.{count}
writerTag.file = logs/RGDSLogsSFTP-{date:yyyy-MM-dd}.log
## Optional link to the latest log file
writerTag.latest = logs/RGDSLogsSFTP.log
writerTag.backups = 365
## deprecated ???
#writerTag.label = timestamp:yyyy-MM-dd
writerTag.policies = daily

You have to use different properties names for both writers.
Correct would be:
provider = reconfigurable

writerBlockHole = black hole
writerBlockHole.level = black hole

writerTag = rolling file
writerTag.tag = TOTO, TEST
writerTag.level = trace
writerTag.format = {date:yyyy-MM-dd HH:mm:ss} - TAG : {tag} - {level} : {message}
writerTag.file = logs/RGDSLogsSFTP-{date:yyyy-MM-dd}.log
writerTag.latest = logs/RGDSLogsSFTP.log
writerTag.backups = 365
writerTag.policies = daily

I don't understand.
I need only one writer.
How writerTag can be reload with his new tag, if it's not black hole ?
I do the same test with your configuration, first i use
writerTag.tag = TOTO, TEST
log tag : TEST1 -> no write : OK
After i use
writerTag.tag = TOTO, TEST, TEST1
i reload
log tag : TEST1 -> no write : KO

How writerTag can be reload with his new tag, if it's not black hole ?

Block hole is a second dummy writer that does not output anything. However, it prevents any output optimizations that would make re-configuration impossible.

I do the same test with your configuration, first i use

For me, it does work:
2022-10-21 20:32:24 - TAG : TEST - INFO : Attempt 1
2022-10-21 20:32:24 - TAG : TEST1 - INFO : Attempt 2
2022-10-21 20:32:24 - TAG : TEST1 - INFO : Attempt 3
2022-10-21 2ß:32:24 - TAG : TEST - INFO : Attempt 3

My example project: example.zip

Hi, thx for your example,
But i reproduce bug if you use the same file in debug mod of your example.
the reload properties seems doen't work
My application :
`ReconfigurableLoggingProvider.reload("/tinylog-1.properties");
Logger.tag("TEST1").info("Attempt 1");
Logger.tag("TEST").info("Attempt 1");
	ReconfigurableLoggingProvider.reload("/tinylog-1.properties");
	Logger.tag("TEST1").info("Attempt 2");
	Logger.tag("TEST").info("Attempt 2");

	ReconfigurableLoggingProvider.reload("/tinylog-1.properties");
	Logger.tag("TEST1").info("Attempt 3");
	Logger.tag("TEST").info("Attempt 3");`

I put break point in all line : ReconfigurableLoggingProvider.reload("/tinylog-1.properties");
When programm stop i change properties file and i go next step, and it's doesn't work.
result file
2022-10-25 16:36:24 - TAG : TEST - INFO : Attempt 1 2022-10-25 16:37:45 - TAG : TEST - INFO : Attempt 2 2022-10-25 16:38:58 - TAG : TEST - INFO : Attempt 3

When programm stop i change properties file and i go next step, and it's doesn't work.

If you want to change the tinylog configuration files during runtime, I propose to load them directly from the filesystem and not from the classpath. This should fix your issue that your file changes don't apply.

LOGGER ERROR: Failed to create link 'logs\RGDSLogsSFTP.log' (java.nio.file.FileSystemException: logs\RGDSLogsSFTP.log: The process cannot access the file because it is being used by another process.
)

Sounds like your application is running multiple times simultaneously or the log file is opened in any editor that locks the file.

I will close this issue for now. If there is anything else, how I can support you using tinylog, please let me know.


Add Transmitter Transducer Accuracy Data elements to Export Endpoint

Add Transmitter Transducer Accuracy Data  elements to Export Endpoint (Both Official and workspace)

Workspace

Test Case: TC1142
Test Passed


Federation does not resolve Hostname, need to use IP

 Issue by tommyknows
Thursday Dec 21, 2017 at 08:33 GMT
Originally opened as https://github.com/kubernetes/kubernetes/issues/57505

Is this a BUG REPORT or FEATURE REQUEST?:
/kind bug
What happened:
When adding a cluster to the federation (with kubefed join) the "server" field (in kubeconfig) needs to be an IP-Adress. If I use "server: https://hostname:6443", the cluster will be offline when executing kubectl get clusters due to Cluster is not reachable
What you expected to happen:
I expect the federation to be able to work with hostnames as well - as it works with kubectl too. Or at least, it should be documented somewhere that you need to use IP-Addresses.
How to reproduce it (as minimally and precisely as possible):
Create a federation cluster and trying to add a cluster that uses hostnames instead of IPs.
Anything else we need to know?:
Environment:

Kubernetes version (use kubectl version): 1.8.5
Cloud provider or hardware configuration: On-Premise / OpenStack
OS (e.g. from /etc/os-release): Ubuntu 16.04.3 LTS
Kernel (e.g. uname -a): 4.4.0

 Comment by tommyknows
Thursday Dec 21, 2017 at 08:52 GMT

@kubernetes/sig-multicluster-bugs

 Comment by k8s-ci-robot
Thursday Dec 21, 2017 at 08:52 GMT

@tommyknows: Reiterating the mentions to trigger a notification:
@kubernetes/sig-multicluster-bugs

In response to this:

@kubernetes/sig-multicluster-bugs

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

cc @tommyknows

I think this is still a valid issue and should be at least documented somewhere (if it's not "fixed").
Correct me if I should be wrong :-)
/remove-lifecycle stale

SGTM

/remove-lifecycle rotten


Adaptive Timestep bug

During PR #523, the ChangeTimestep function of battery_t has some bugs:

the lifetime_param's dt_hr was missing from the copy assignment operator
the lifetime index of the battery needed to be adjusted for the new time step, otherwise the day age of the battery will be incorrect
battery_t's copy constructor didn't correctly pair up the battery_params with the params within each component class. This is only a problem if you modify the params (as in ChargeTimestep)

Another un-related bug fixed in this PR is that the rainflow cycle processing needs some tolerance to allow for cycles that are the same DOD except by a small factor that is a result from degradation. Otherwise the rainflow_peaks can look like:
94.999718345306533, 5.0455862059404382, 94.999717794146434, 5.0456951070969751, 94.999717194336653, 5.0458097135306446, 94.999716529991715
Add an error check to ChangeTimestep to [revent changing from smaller to larger timestep in the middle of the larger time step:
"battery_t dt_hr step size can only be changed to a higher step size when the current time step is at a time step common to both the previous and new step size. For instance, if running 30-min steps, step size can only be increased to 60-min step at the hour."

@dguittet should this be closed, or is there more work to do after #545?


fix: correct content order (#225)

Summary
Fix #225
Add sortBy parameter when querying the members of a folder.
Testing
As described in #225.
Run unit tests

Can we update the CHANGELOG with this change?

Added in the latest commit

Can we update the CHANGELOG with this change?

Added in the latest commit


Exam mode: Improve usability of attendance check

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[ ] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[ ] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[ ] I chose a title conforming to the naming conventions for pull requests.

Server

[ ] Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
[ ] I followed the coding and design guidelines.
[ ] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[ ] I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
[ ] I documented the Java code using JavaDoc style.

Client

[ ] Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data.
[ ] I followed the coding and design guidelines.
[ ] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[ ] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[ ] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[ ] I documented the TypeScript code using JSDoc style.
[ ] I added multiple screenshots/screencasts of my UI changes.
[ ] I translated all newly inserted strings into English and German.

Changes affecting Programming Exercises

[ ] I tested all changes and their related features with all corresponding user types on Test Server 1 (Atlassian Suite).
[ ] I tested all changes and their related features with all corresponding user types on Test Server 2 (Jenkins and Gitlab).

Motivation and Context

Description

Steps for Testing

Prerequisites:

1 Instructor
2 Students
1 Programming Exercise with Complaints enabled

Log in to Artemis
Navigate to Course Administration
...

Exam Mode Testing

Prerequisites:

1 Instructor
2 Students
1 Exam with a Programming Exercise

Log in to Artemis
Participate in the exam as a student
Make sure that the UI of the programming exercise in the exam mode stays unchanged. You can use the exam mode documentation as reference.
...

Review Progress

Performance Review

[ ] I confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance
[ ] I confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test 1
[ ] Test 2

Test Coverage

Screenshots

Codecov Report

Merging #6463 (a10abe4) into develop (c9707db) will increase coverage by 3.97%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #6463      +/-   ##
=============================================
+ Coverage      76.48%   80.46%   +3.97%
- Complexity         0    13034   +13034
=============================================
  Files           1248     2323    +1075
  Lines          47096    89243   +42147
  Branches        8797    13190    +4393
=============================================
+ Hits           36020    71806   +35786
- Misses          5684     9610    +3926
- Partials        5392     7827    +2435

see 1076 files with indirect coverage changes

Flag
Coverage Δ

client
76.47% <ø> (-0.01%)
:arrow_down:

server
84.90% <ø> (?)

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 c9707db...a10abe4. Read the comment docs.

If all students are signed, but one (or more) students are missing some of the other toggles it shows the Red Cross correctly in the overview, but in the "Verify attendance" view it shows that all students are checked successfully. IMO this should show that the toggle is missing


add circle config file

Node.js 9 is not needed here anymore. Just 6, 8 and 10. The others reached their EOL.

See https://github.com/nodejs/Release/blob/master/README.md
Odd releases are short living releases between the real (LTS and stable) ones.

Oh ok cool. Not merging this til the man @oliver-moran activates circle for this project.


Twitter LoginwithBlock Crashed if twitter account not added

I am getting below exception after
[PFTwitterUtils logInWithBlock:^(PFUser *user, NSError *error) {}
i am getting below error
LaunchServices: invalidationHandler called
*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '*** -[__NSPlaceholderDictionary initWithObjects:forKeys:count:]: attempt to insert nil object from objects[0]'
*** First throw call stack:
(0x21f3a49f 0x2f734c8b 0x21e5a873 0x21e5a657 0xd92d5 0xdabeb 0x5119db 0x5119c7 0x5153ed 0x21f003b1 0x21efeab1 0x21e4c3c1 0x21e4c1d3 0x2924a0a9 0x2545a7b1 0x9d8f5 0x2fcb4aaf)
libc++abi.dylib: terminating with uncaught exception of type NSException

Hey @venky145, please verify that callback url is set in your Twitter app settings at https://dev.twitter.com/apps.
If it still fails after this - please open an issue in the ParseTwitterUtils repo.


Inconsistent behaviour for xarray.DataArray.rename

Hi,
thanks for creating xarray, I like it a lot. However I want to ask, why the behaviour of xarray.DataArray.rename is so incosistent:
import xarray as xr
x = xr.DataArray([1, 2, 3], dims=('a',), coords={'a': [1, 2, 3]}, name='a')
x.rename({'a': 'b'})

renames the name of x not the coordinate! Could you explain the behaviour to me? I do not understand it, or consider it as inconsistent.
Thanks, Markus

On master, I see:
In [5]: x = xr.DataArray([1, 2, 3], dims=('a',), coords={'a': [1, 2, 3]}, name='a')

In [6]: x
Out[6]:
<xarray.DataArray 'a' (a: 3)>
array([1, 2, 3])
Coordinates:
  * a        (a) int64 1 2 3

In [7]: x.rename({'a': 'b'})
Out[7]:
<xarray.DataArray 'b' (b: 3)>
array([1, 2, 3])
Coordinates:
  * b        (b) int64 1 2 3

So it renames both x and it's coordinate.
I agree that this is inconsistent and surprising. Using a dictionary should never rename the name of a DataArray. This is an inadvertent legacy of the previous (outdated) design of the DataArray class, which used a Dataset for holding its contents.
A fix for this would be very welcome.


Export Pin and a few of its attributes.

Fix #25. Export a Pin and a few of its attributes. I renamed the Pin interface to Pinner. The changes don't introduce incompatibilities.

Closed because this project isn't accepting pull requests anymore. The PR is merged in this fork of the project.


Created public event hooks

Creating public event triggers for each of the IAS events so external JS functions can bind to them.

This is interesting, thanks for your contribution!
One quick question before moving forward with this: the ias object should be globally available through the window after $(document).ready(), so in theory IAS events can be listened to directly by calling ias.on(). Is that not possible in practice?

That was my first thought as well, but after testing that theory it proved to be incorrect. I just went back and did some more testing and it seems the issue was that init.phtml was being loaded in after all other JS, so the global variable was showing as undefined even inside the $(document).ready(). So this change could still avoid that issue if the order of items isn't correct, but I have modified my layouts to work with the change now as well.

Actually, I may have spoken too soon. I am loading the init.phtml template in after jQuery but before my Javascript, and tested with the above code and it is still finding window.ias to be undefined. Further testing shows that the global variable is being established after my code is executing. I'm wondering if it is due to it being established inside the anonymous function that is inside a getScript. I will continue to do some testing...

Sure, some more testing wouldn't hurt, but I see the value of this PR anyway.
What do you think about firing a single, custom, event once IAS is loaded? The event could pass IAS as a parameter and allow further hooks. That kind of decoupling of our events from IAS would probably simplify things a bit in some cases: e.g. if IAS introduces a new event in the future then the approach would allow people to immediately listen to it instead of waiting for us to ALSO implement it. What do you think?

That makes sense. Since you are already triggering an event when IAS is loaded, you could just add the data to that trigger. Something like this? https://github.com/rain2o/Magento-InfiniteScroll/commit/ad1681bc651e6e1ec3f112a3e8b8f4bbfec433cd
Let me know if you want to submit a new pull request for this.

Yes, exactly. In fact we'll merge that directly once the debug feature is tested.
Closing this one in favor of #140

I'm merging #140 but note that we don't need any custom event to access window.ias, you just need to listen the window.load instead of document.ready:

$j(document).ready(function() {
    console.log('document ready');
    console.log(window.ias);
});

$j(window).load(function() {
    console.log('window load jquery');
    console.log(window.ias);
});

Event.observe(window, "load", function(){
    console.log('window load prototype');
    console.log(window.ias);
});

We discussed this internally, conclusion: listening to "infiniteScrollReady" (as proposed in this PR) is better than listening to "window.load" because the first one might get fired earlier.