Suggestion: export MAILCOW_EXPORTER_SCHEME as variable

Hi,
Since I'm using the docker container on a host that runs the mailcow (api) on an internal IP, it would be nice to be able to declare scheme as a variable, since https it not usable if it's a private IP.
export MAILCOW_EXPORTER_SCHEME="https"

or
export MAILCOW_EXPORTER_SCHEME="http"

I have looked into the sources, but I don't know go well enough to make a pull request.

This is a good idea and I am confused, why this hasn't come up earlier. Unfortunately, I am currently flooded with work, so It's going to be a good bit until I can find some time for it. In case you want to get familiar with go a bit, all of the necesarry pieces should be in main.go.

https://github.com/j6s/mailcow-exporter/blob/main/main.go#L45 This is how environment variables are read
https://github.com/j6s/mailcow-exporter/blob/main/main.go#L122 This is where the default scheme is added

If you have a go compiler installed, then you should be able to use make build in order to build the project into the bin/ directory. If you have any questions about the process, feel free to reach out.

Looking at the code a bit more, the preferred solution would probably be

Adding a global variable here: https://github.com/j6s/mailcow-exporter/blob/main/main.go#L17
Parsing the env variable here: https://github.com/j6s/mailcow-exporter/blob/main/main.go#L45
Using that variable as a fallback, if it is set here: https://github.com/j6s/mailcow-exporter/blob/main/main.go#L122

I just had a go at, but it won't compile because of an undefined variable - here I'm a bit lost because of go.
./main.go:131:13: undefined: defaultScheme
make: *** [Makefile:6: build] Error 1

If I understand it right defaultScheme gets declared and set to "https" - I'm missing something here I guess.

	defaultScheme, _ := os.LookupEnv("MAILCOW_EXPORTER_LISTEN")
	if defaultScheme == "" {
		defaultScheme = "https"
	flag.StringVar(&scheme, "scheme", defaultScheme, "Default connection scheme. Defaults to the MAILCOW_EXPORTER_SCHEME environment variable or 'https' otherwise")
			scheme = defaultScheme

Hi,
Finally got around to take some time to read through the code and learn some go on the way.
I created a PR, feel free to look into it when you have time.
Thank You,


Add my Introduction to list participating

hello thibaud colas, Thank you so much for taking the time to read my
introduction and providing me with feedback. Your thoughtful comments and
support have been incredibly valuable to me and have given me renewed
motivation to continue. I am grateful for your guidance and encouragement.
On Fri, Mar 24, 2023 at 8:32 AM Thibaud Colas @.***>
wrote:

Merged #63 https://github.com/wagtail/gsoc/pull/63 into main.
—
Reply to this email directly, view it on GitHub
https://github.com/wagtail/gsoc/pull/63#event-8835576027, or unsubscribe
https://github.com/notifications/unsubscribe-auth/A22MYVYRATSNPUXLYMUGAS3W5VEYFANCNFSM6AAAAAAWF44YUE
.
You are receiving this because you were mentioned.Message ID:
@.***>


FSA compatibility issue

Hello,
Currently there is compatibility issue with FSA. If redux action is FSA compatible, the generated API call success or error action has payload with .requests in it. So watchRequests saga is triggered again and new API call sequence is initiated, but without any luck - url is not found. I would like to have additional check in helpers function to avoid these kind of situations - if payload has response in it - it is not request action.

Hello @klis87 ,
Thanks for quick response.
I am using integration with redux-act.
The issue occurs only when error is returned from API. It does not matter if it is 404, 422 or other kind of error.
Generated error action has request variable in payload, so it trigger new request action.
Please see image below:

Good catch, thanks! Published as 0.12.1


Update Ruby version and leader

We don't support 1.8.7 anymore, and afaict it's 2.0-2.2 instead. Advertise this.
Also, Greg doesn't work here anymore. Carl has last commit!
r? @zenazn

LGTM

\o/


NXP: Kinetis: Only enable temperature sensor driver by default if corresponding ADC is enabled

Only enable the NXP Kinetis temperature sensor driver by default if corresponding ADC is enabled. Fixes #23694.

I approve the fix for it.
Still though, is it the right approach to bring in features not explicitly requested?
If I want to keep a low footprint in my Zephyr project it seems wrong to force features I'm not particular interested in. I know this module doesn't have a big impact - it's the principles.


Add smooth zoom and pan effect like OpenSeaDragon when dragging and scrolling

OpenSeaDragon smooth zoom and pan effect demo:
https://openseadragon.github.io

OpenLayers has that already. For ol/interaction/DragPan, there is a kinetic option that allows you to configure the exact behavior. And for ol/interaction/MouseWheelZoom, you can play with the maxDelta, duration and timeout options.

@ahocevar How to replace the default DragPan and MouseWheelZoom interaction ?
I want to use the default interactions but replace the DragPan and MouseWheelZoom interaction, should I first iterate over the default interactions to find the DragPan and MouseWheelZoom interaction, remove them, and create a new DragPan and MouseWheelZoom instance and add it to the interactions ?
Is there any other ways ?

OpenLayers has that already. For ol/interaction/DragPan, there is a kinetic option that allows you to configure the exact behavior. And for ol/interaction/MouseWheelZoom, you can play with the maxDelta, duration and timeout options.

Fantastic!👍

How to replace the default DragPan and MouseWheelZoom interaction ?

import Map from 'ol/Map';
import {defaults, DragPan, MouseWheelZoom} from 'ol/interaction';

new Map({
  interactions: defaults({
    dragPan: false, mouseWheelZoom: false
  }).extend([new DragPan({/* options */ }), new MouseWheelZoom({/* options */})])
});

@ahocevar I have tried the kinetic option of DragPan, but it's not working, am I do something wrong ?
Here is a demo: https://codepen.io/sange/pen/dypyeep

I can't figure out how to tweak the kinetic option so that I can create the smooth pan effect like OpenSeaDragon.

The values used in the default interactions are const kinetic = new Kinetic(-0.005, 0.05, 100);, they are per milliseconds, so should be quite small.

@M393 I have tried the default kinetic options, and also many other values, but still can't get the smooth effect like OpenSeaDragon. 🙁

If you cannot tweak the options to achieve the desried effect, you could also create a custom interaction. You'll probably have to dig into the OpenSeaDragon source code and do some research, then implement something equivalent as custom OpenLayers interaction.

I think the OpenSeaDragon example has a maximum velocity, and reduces the decay if the interaction velocity was high. Maybe you can create a subclass of ol//Kinetic that implements this behavior.

@kuige Faced the same problem, solution: change duration in ol.DragPan:
handleUpEvent(mapBrowserEvent) {
const map = mapBrowserEvent.map;
const view = map.getView();
if (this.targetPointers.length === 0) {
if (!this.noKinetic_ && this.kinetic_ && this.kinetic_.end()) {
const distance = this.kinetic_.getDistance();
const angle = this.kinetic_.getAngle();
const center = view.getCenterInternal();
const centerpx = map.getPixelFromCoordinateInternal(center);
const dest = map.getCoordinateFromPixelInternal([
centerpx[0] - distance * Math.cos(angle),
centerpx[1] - distance * Math.sin(angle),
]);
view.animateInternal({
center: view.getConstrainedCenter(dest),
duration: 800,
easing: easeOut,
});
}
if (this.panning_) {
this.panning_ = false;
view.endInteraction();
}
return false;
}

Based on @kuige codepen example, simply enabling tiles preload made the zoom and pan very smooth:


Added docker-compose up/down script

Added up.sh and down.sh script to start/stop docker container easily.

Quick Start with Docker section in README.md has been updaed.
updated script to use PR #52

Thanks. It's quite useful!


Clarify use of and source for Gapminder data

Organization1 uses the Gapminder data within R, possibly via @jennybc's gapminder package. (@iamciera - can you confirm what the plan is re: where the data will come from?) I thought the Intro lesson does, too, but am not sure, nor whether that would be within RStudio (and hence R) or outside. (@mine-cetinkaya-rundel - can you confirm?) If within R, could perhaps also be installed from CRAN package?

We have in our readme:

Gapminder data available here. Gapminder data is licensed CC-BY 3.0.
Processed data via @jennybc, R package available here. The data-raw sub-directory reveals the journey from Gapminder.org's Excel workbooks to increasingly clean and tidy data.

cleanest data is in 07_gap-every-five-years.tsv
this gives rise to the data.frame provided by the R package

We were discussing what to do with the datafile and as of now, it is in the root folder where the markdown report is.   We could use the package, but the students should be aware of how to input files, as their data will most likely not have a package. It would be easier to just install from CRAN though....

If they install the files from a zip file, then the data file would hopefully be in place already, so hopefully this would not be difficult.
However, if they install the package, it's not a bad opportunity to show how one can publish a dataset with all stages of cleaning and massaging documented, and once installed they can use R to find out where the data file is:
gap_tsv <- system.file("gapminder.tsv", package = "gapminder")

I was about to add what @hlapp already did: yes you could enjoy the convenience of installing from CRAN but then use system.file() to access a .tsv. Perhaps even copy it into the local directory, then load explicitly via read.delim(), just to make sure they are seeing this process.

@hlapp in the intro lesson they will be provided a csv file with the gapminder data, so that they can first use it with their existing tools as well

I was just putting together the zip files and a web site, but @dleehr did it already! https://github.com/Reproducible-Science-Curriculum/2015-05-14-duke-materials
Thanks @dleehr! Do folks want to check to make sure everything that's needed is there?

@tracykteal you're welcome! I'm working with the instructors to make materials available upon request, and will be managing it throughout the workshop

@dleehr awesome, that's great!

added data attribution to rr-intro readme as well


FM Recipes duplicate names, complicating data aggregation

For example Basic Cauldron and Advanced Cauldron recipes are both named "Make Exp 1, Make Exp 2" but advanced should have different recipe names like "Make Adv Exp 1, Make Adv Exp 2". I am sure there are other recipes that duplicate names (like Repair Factory) that also need to be differentiated.
Likewise, the recipe levels are named the same thing as the recipes, meaning recipe levels get aggregated together with the runcounts. the recipe levels should be like "Forge Gold Axes Level".
The first one is a config change, the second one may be a code change.

Can we just add another column into the mix to make a unique key to aggregate upon?


Upgrade Lombok version

Motivation

1.18.18 includes fixes for IDE compatibility (IntelliJ, Eclipse, Netbeans).
This is relevant while developing Pulsar.

see https://projectlombok.org/changelog for more details

Modifications

upgrade Lombok version from 1.18.16 to 1.18.18

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

Hi @lhotari , Looks like the CI process is in a weird status. Rebase your code or reopen this PR may make it normal.

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

This PR is finally ready to be merged. /Cc @sijie @merlimat


[Bug]: PuppeteerSwitchToTab does not switch to the selected index

Version of the software
0.2.4
Operating system
Windows 10
Browser / Native
Chrome 105.0.5195.127
What happened?
As mentioned in the title, PuppeteerSwitchToTab does not switch to the selected index. For example, if you have 5 Tabs open in your browser, and you want to move to Tab 2, PuppeteerSwitchToTab will give you another tab instead.
Despite opening this issue a year ago, it has not been resolved
https://github.com/openbullet/OpenBullet2/issues/269
Relevant LoliCode if needed
No response

Are you sure it's not 0-indexed? Tab index 0 should be the first tab, then index 1 should be the second one and so on. Can you check?

Yes I am sure of that, just try to open 3 or 4 tabs and switch to tab 2 or 3 and you will see what will happen

After some debugging, I found this issue on the PuppeteerSharp repository, which suggests a workaround.
I added some additional logging to the RuriLib block after implementing the workaround and it seems to work as expected now.
// LoliCode
BLOCK:PuppeteerOpenBrowser
ENDBLOCK

BLOCK:PuppeteerNavigateTo
ENDBLOCK

BLOCK:PuppeteerGetAttributeValue
  findBy = Selector
  identifier = "title"
  => VAR @puppeteerGetAttributeValueOutput
ENDBLOCK

BLOCK:PuppeteerNewTab
ENDBLOCK

BLOCK:PuppeteerNewTab
ENDBLOCK

BLOCK:PuppeteerNewTab
ENDBLOCK

BLOCK:PuppeteerNewTab
ENDBLOCK

BLOCK:PuppeteerSwitchToTab
  index = 2
ENDBLOCK

BLOCK:PuppeteerNavigateTo
  url = "https://picsum.photos/200/300"
ENDBLOCK

BLOCK:PuppeteerSwitchToTab
  index = 3
ENDBLOCK

BLOCK:PuppeteerNavigateTo
  url = "https://picsum.photos/200/300"
ENDBLOCK

BLOCK:PuppeteerSwitchToTab
  index = 1
ENDBLOCK

BLOCK:PuppeteerNavigateTo
  url = "https://picsum.photos/200/300"
ENDBLOCK

BLOCK:PuppeteerSwitchToTab
  index = 4
ENDBLOCK

BLOCK:PuppeteerNavigateTo
  url = "https://picsum.photos/200/300"
ENDBLOCK

// Output
>> Open Browser (PuppeteerOpenBrowser) <<
Browser opened successfully!
>> Navigate To (PuppeteerNavigateTo) <<
Navigated to https://example.com
Tab 0: https://example.com/
>> Get Attribute Value (PuppeteerGetAttributeValue) <<
Got value Example Domain of attribute innerText by executing document.querySelectorAll('title')[0].innerText;
Assigned value to variable 'puppeteerGetAttributeValueOutput'
>> New Tab (PuppeteerNewTab) <<
Opened a new page
>> New Tab (PuppeteerNewTab) <<
Opened a new page
>> New Tab (PuppeteerNewTab) <<
Opened a new page
>> New Tab (PuppeteerNewTab) <<
Opened a new page
>> Switch to Tab (PuppeteerSwitchToTab) <<
Switched to tab with index 2
>> Navigate To (PuppeteerNavigateTo) <<
Navigated to https://picsum.photos/200/300
Tab 0: https://example.com/
Tab 1: about:blank
Tab 2: https://fastly.picsum.photos/id/299/200/300.jpg?hmac=iJwKPn_sm8S_ZGJttTHq5EpwaEESy8Q-qDaQ9xWtlYw
Tab 3: about:blank
Tab 4: about:blank
>> Switch to Tab (PuppeteerSwitchToTab) <<
Switched to tab with index 3
>> Navigate To (PuppeteerNavigateTo) <<
Navigated to https://picsum.photos/200/300
Tab 0: https://example.com/
Tab 1: about:blank
Tab 2: https://fastly.picsum.photos/id/299/200/300.jpg?hmac=iJwKPn_sm8S_ZGJttTHq5EpwaEESy8Q-qDaQ9xWtlYw
Tab 3: https://fastly.picsum.photos/id/721/200/300.jpg?hmac=6g_vLTUju_TGWN7cMKTjZgzqps-JjmHIS0KSuFsgVyc
Tab 4: about:blank
>> Switch to Tab (PuppeteerSwitchToTab) <<
Switched to tab with index 1
>> Navigate To (PuppeteerNavigateTo) <<
Navigated to https://picsum.photos/200/300
Tab 0: https://example.com/
Tab 1: https://fastly.picsum.photos/id/643/200/300.jpg?hmac=rS-MHa0BIMHdAgm-FZ7QM36aRKEAzzhSRNRv5n4uqGc
Tab 2: https://fastly.picsum.photos/id/299/200/300.jpg?hmac=iJwKPn_sm8S_ZGJttTHq5EpwaEESy8Q-qDaQ9xWtlYw
Tab 3: https://fastly.picsum.photos/id/721/200/300.jpg?hmac=6g_vLTUju_TGWN7cMKTjZgzqps-JjmHIS0KSuFsgVyc
Tab 4: about:blank
>> Switch to Tab (PuppeteerSwitchToTab) <<
Switched to tab with index 4
>> Navigate To (PuppeteerNavigateTo) <<
Navigated to https://picsum.photos/200/300
Tab 0: https://example.com/
Tab 1: https://fastly.picsum.photos/id/643/200/300.jpg?hmac=rS-MHa0BIMHdAgm-FZ7QM36aRKEAzzhSRNRv5n4uqGc
Tab 2: https://fastly.picsum.photos/id/299/200/300.jpg?hmac=iJwKPn_sm8S_ZGJttTHq5EpwaEESy8Q-qDaQ9xWtlYw
Tab 3: https://fastly.picsum.photos/id/721/200/300.jpg?hmac=6g_vLTUju_TGWN7cMKTjZgzqps-JjmHIS0KSuFsgVyc
Tab 4: https://fastly.picsum.photos/id/160/200/300.jpg?hmac=RM8ofyEQBOl_xUcD4r_IVw6CQeVY8AQ4qtdM8js1D18

So I think I will just implement the workaround for now.


Some good content should add back to the doc

As https://github.com/wechaty/wechaty.js.org/pull/870/files#diff-fd47a25eaa8054d71a6781434e1c57dab39cc93746f29d0d69ab0051d5583ecdL69 request, it deletes the following content, and I think it's right.
But I think it should not be shown on the overview page, but it should put somewhere, so I file an issue here and looking forward to putting the following back somewhere at the doc in the future.
## The development process

Learn about the various components and tools to help you in the development and testing of Wechaty chatbot applications:

<!--
- **Settings**: [Overview]() | [Full list of settings]()
- **Chatbot applications**: [Overview]()
- **Exceptions**: [Overview]()
-->

- **Testing**: [Mock](puppet-providers/mock.md) | [Writting and running tests](https://github.com/wechaty/wechaty-vorpal-contrib/blob/master/src/contrib/math_master/math_master.spec.ts)
- **Deloyment**: [Docker](tutorials/docker.md) | [Heroku](https://github.com/wechaty/heroku-wechaty-getting-started)

## Common Chatbot application tools

- [FileBox](https://github.com/huan/file-box): helper module for easy processing files between the local filesystem and the network.
- [Brolog](https://github.com/huan/brolog): helper module for easy logging between the Node.js and the Browser environments. (Angular Dependency Injection supported)
- [FlashStore](https://github.com/huan/flash-store): helper module for key-value persistant storage in both Node.js local filesystem and Browser(tbw). Follows the [ES6 Map API](https://github.com/huan/async-map-like#readme).
- [MemoryCard](https://github.com/huan/memory-card): help module for key-value persistant storage in cloud. (should be a cloud native module in the future). Follows the [ES6 Map API](https://github.com/huan/async-map-like#readme).

## Other core functionalities

Learn about some other core functionalities of the Wechaty:

- [TOKEN specification](specs/token.md)
- [Puppet specification](specs/puppet.md)

## The Wechaty open-source project

Learn about the development process for the Wechaty project itself and about how you can contribute:

- **Commmunity**: [How to get involved](contributing/overview.md) | [Team organization](https://github.com/wechaty/PMC) | [The Wechaty source code repository](https://github.com/wechaty/wechaty) | [Mailing lists](https://groups.google.com/g/wechaty) | [Gitter.im channel](https://gitter.im/wechaty/wechaty) | [Wechaty contributor program](contributing/contributor-program.md)
- **Documentation**: [About this documentation](contributing/documentation.md)
- Wechaty over time: [Release notes](https://github.com/wechaty/wechaty/releases)

What about creating a general label called "documentation" and putting that as well along with the existing label. Because this issue has a bigger scope than GSoD 2021.

Sure, I will change add a new label. If GSoD tech writers are interested in this part, we'll be happy they can contribute.


Graphiql endpoint should be configurable by env var (default: false)

Graphiql at api/graphiql meant for inactively exploring the stitched BFF schema (Dialogporten backend + app specific) is useful dev tool, but should be disabled as default and enabled (by override) locally.
Add override in compose and add env variable guard through config.

Edit: enabled by default, and explicitly disabled in production.


Getting "Invalid segment alignment" error when trying to upload to iTunesConnect

Hi!
I've been developing an iOS app based on TDLib for past couple of months. As advised in the README.md, I use td_json_client to make requests to TDLib. I've built TDLib for iOS using ./build_openssl.sh and ./build.sh scripts from example/ios. Then I've added the compiled .dylib to Link Binary with Libraries and Embed Libraries build phases in Xcode. But when I try to submit a build to iTunesConnect after archiving the project using Organizer in Xcode 9.3, I get Invalid segment alignment error:

I haven't ever seen such an error in my past experience, and have no idea how to fix it. Any ideas what could I do wrong and what should I check? The app works completely fine when I run it on the simulator or deploy to an iOS device through USB.

As far as I know dylib should work fine since ios 8.
You just should sign it with the same signature as your app. And add it as an embedded framework (General->Embedded binaries).

Adding the .dylib itself to Embedded binaries setting works during development, but results in errors during publishing to iTunesConnect. This note states:

Dynamic libraries outside of a framework bundle, which typically have the file extension .dylib, are not supported on iOS, watchOS, or tvOS, except for the system Swift libraries provided by Xcode.

It suggests to wrap the .dylib into a .framework, but I wasn't able to, because there are no example neither in TDLib repo, nor in Apple docs. In the end, the only working solution I came up with was building the TDLib with custom script from here  and combining the static .a (not .dylibs) libraries and headers into a framework.
Could you please provide an example of a working Xcode project or a guide of how to set it up after a successful compilation? Though I have already solved my problem with 3rd party solutions, a more detailed documentation would be helpful for other users.

Thanks to @Kylmakalle, example/ios now creates a fully-featured XCFramework, which shouldn't be affected by these issues.


enable commandline option forwarding to clang

currently there is an option for this:
--saargs CLANGSA_ARGS_CFG_FILE
File with arguments which will be forwardeddirectly to
extend the option to enable passing options directly:
--saargs ClangSA options or CLANGSA_ARGS_CFG_FILE
Clang Static Analyzer options directly or  a File with arguments which will be forwardeddirectly to

This issue is not relevant any more, so I close it.


Change 'Form' to 'div'

Control Layers create a form with inputs dynamically for panel of layers. So, on this panel I see checkbox and labels for hide or show layers on map.
But, when I use jQuery Validate, the plugin check this form and launch error about undefined rules.
then jQuery Validate plugin catch it and search rules for validate inputs of form.

Change form to div no affect behavior of Control Layers.

Hi,
Thank you for submitting your first PR here!
If I understand correctly, your report a compatibility issue with jQuery validation plugin.
Filing an issue first could have been more appropriate, so that we can make sure maintainers have enough information about the situation you are trying to improve, and therefore can in a next step properly evaluate your PR and if it is relevantly implemented.
Your proposed PR is to modify Leaflet so that when you use it on your site with the above jQuery plugin, the latter does not complain about missing custom validation rules?
With this modification, you make Leaflet build some <input>'s outside a <form>. While this is still valid as per HTML spec, I am not sure what other implications this may have, e.g. regarding accessibility?
Furthermore, I am unable to reproduce the error message you show in your screenshot. I may not use jQuery validation properly, so please provide a full reproducible example:

Merely loading jQuery and jQuery validation plugin does not seem to do anything, therefore not causing any error due to Leaflet's <form>.
Explicitly catching all page <form>'s and trying to .validate() them, even with debug: true option, does not trigger any error: https://plnkr.co/edit/3W9P0vCTtwfjhGi7Us7P?p=preview
You can check that Leaflet's <form> has been properly taken into account because it now has the novalidate attribute set by jQuery validation.

Please can you kindly provide more details about your situation, so that maintainers can properly evaluate how relevant is your PR?

I'm strongly inclined to reject this PR, for two reasons:

This targets the 0.7 branch, and we don't really do 0.7 maintenance anymore.
The jQuery validate plugin allows the user to validate specific forms - thus, if it's validating the layers control, I'm inclined to think about an implementation error.

Hi @dczunigap,
If you still have your problem and you are sure the problem is due to Leaflet rather than how your app is configured, please open an issue with relevant details and a reproduction example.
Also we will likely not be able to provide support for Leaflet 0.x.

Chales! @ghybs you're right... jquery validate plugin no produce conflict when I configure it in javascript file. The conflict is show when use an angular directive which implements jquer validate plugin in my project, as you say and @IvanSanchez.
So...  I think I owe you an apology. Thanks for the feedback and give me a chance for try contribute


Add the source file and line number to docs entries

This PR adds an entry in the docs for each item with the source file and line number. The need for this is outlined in elm-lang/elm-make#29 and  elm-lang/package.elm-lang.org#35, but in summary it allows tools that consume the docs json file to link to the exact source file and line number where something is defined. This could be "go to definition" in editors or direct links in the package docs.
This PR also relies on a small PR to elm-lang/elm-make#45. Both of these PRs will fail on CI of course, because they depend on each other, so it will probably need to be tested manually.

My feeling is that it makes sense to pass this information in with the Context value. We will be wanting the full package name to disambiguate modules as described in #826, and it feels like this fits nicely in that chunk of information.
The net goal here is to make "jump to source" work in the package catalog and in editors, correct? I guess this seems like a logical place to put this. I am assuming it'd come along with features like "tell me the type of this imported thing!" or "show me the docs for this imported thing!" so you'd need to be running the doc generator anyway. Is that fair to say?

I think putting into the context makes sense for sure; Good point. I will update my my PR.
And yeah, the goal is to use this for "jump to source". And since we get all of the other information needed about a token from the docs files already, this tiny extra info enables opening the file under elm-stuff/packages or linking to the file on github.

Cool, sounds good to me.
I just merged a bunch of stuff into master, so things are messed up for now. Let's do this as a separate PR because the stuff here probably got trampled!


Special case SM2 when decoding

SM2 abuses the EC oid by reusing it - but an EC key is different to an SM2
key. Therefore we have to special case SM2 during decoding. If we encounter
the EC OID then we have to try both algorithms.
This has been pulled out of #15504 to ease review.

This pull request is ready to merge

Hopefully another PR will test this..

Merged to master.


How to train own dataset!

If i use this method to train own dataset, how do i change the code!

I have not tried it yet, and will update this after I've tried it, but the approach that I will take is to convert my data into a classification dataset in the format of the ImageFolder data loader class from here: https://pytorch.org/vision/stable/datasets.html.  Then, I'll change some of the lines in the get_data function in utils.py to load in my data loader class instead of one of the standard ones.


[1788] fix issue with ReportableEvents not being recorded

Context
Trello card - despite after_save hooks being in place, they are not actually firing during the most common usage (bulk_update) which uses update_all to update many record at once in pure SQL, bypassing ActiveRecord hooks.
Changes proposed in this pull request
Make the bulk_update method update the individual records one-by-one (there can never be more than 100 records per page, so this should not be too much of an overhead)
Add explicit controller/service-level tests for correct behaviour
Guidance to review
Log in as a user with a non-nil mobile_network_id, and some requests present on that MobileNetwork (you can use the FakeDataService to generate some if needed)
From the console, note the initial value of ReportableEvent.count
Use the bulk update to set some requests to complete
From the console, check that the value of ReportableEvent.count has increased correctly
Repeat using the download/upload spreadsheet update mechanism

@aldavidson if you rebase your build should go green


Oauth signatures with a proxy

Adding the possibility to add a proxy in the options in the initialization of the API.
This is useful if you want to debug your Ionic app in the browser according to this reference.
Example :
var wc = new WooCommerceAPI({
  consumerKey: 'ck_xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx'
  consumerSecret: 'cs_xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx',
  url: 'http://localhost:8100/my-proxy/',
  proxy: 'http://myactualsite.com/'
});

What I did :
I added a function getProxyUrl that returns the url of the endpoint according to the actual url of the website (variable proxy in API definition). This function is used instead of getUrl in case of a non-null proxy initialization when using Oauth to authenticate. This way, the actual address is used instead of the proxy to build the oauth_signature (that was my issue https://github.com/woothemes/wc-api-node/issues/14).
Hope it helps

Since proxy will be integer, better change false to 0 or null.

I might miss something but proxy is meant to be a string no ?
But I get your point I'm going for null as default.

I might miss something but proxy is meant to be a string no ?

I agree, can be better, so it's better use an empty string like ''.
Sorry for make you change it again :P

Actually I'm checking for null to see if a proxy has been specified. Is there a problem if I use null instead of '' ?

For strings is better an empty string. Just it.

No worries ;)

Thanks :)


RestSharp dependency issue

I am having issues trying to fix RestSharp dependency when adding SalesForceSharp to my project.
When you add SalesForceSharp reference to the project, it will include the RestSharp v105.0.1 dependency. I have other library projects in the solution using the newest RestSharp v106.6.10. When I run the project I get the "Could not load file or assembly 'RestSharp, Version=<IP_ADDRESS>, " error. Checking the bin folder I have the newest RestSharp v106.6.10.
I tried to add an assemblyBinding to the project's config file to redirect the versions but it won't work because RestSharp v105 doesn't have a publicKeyToken while newest v106.6 has a publicKeyToken.
Any chance to not include the RestSharp dependency when adding SalesForceSharp reference? Any other suggestion?
Thanks

We are running into this issue as well with SalesforceSharp v0.7.4

I guess that would have been the obvious thing to do from the start! I am using a binding assembly work around at the moment but when i get back to that I'll update and do a pull request - thanks for building this tool - very useful!
…
-Adriaan
________________________________ From: Diego Giacomelli<EMAIL_ADDRESS>Sent: Wednesday, January 29, 2020 9:40 AM To: giacomelli/SalesforceSharp<EMAIL_ADDRESS>Cc: awakefield<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [giacomelli/SalesforceSharp] RestSharp dependency issue (#35) Also having the same issue - any suggestions? I guess i'll try pulling entire project down and re-reference restsharp - hate to maintain a custom project for this unless there is a better way? thx Please, fork this repository, make the changes and perform a pull request, then I will review it. 👍 — You are receiving this because you commented. Reply to this email directly, view it on GitHub<#35?email_source=notifications&email_token=AEUDSQ46DDKVACAGUQNX4KLRAG5RJA5CNFSM4I2E6HKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKIC4II#issuecomment-579874337>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEUDSQ3MQSVKAE5FSZYQCP3RAG5RJANCNFSM4I2E6HKA.

Can we get the binding redirect setting ? since i don't have old RestSharp version i don't know the public token for it.

I am running into the RestSharp, Version=<IP_ADDRESS>
I can see a fix and commit has been requested.
Any update as to when this will be fixed and published ?

@Sharky99x probably safe to say this repo's dead, you may want to fork it and implement your own fix.


feat(ingest): add ingestion source for SAP Analytics Cloud

@hsheth2 As discussed, here is the ingestion source for the SAP Analytics Cloud with everything included (docs, tests, configuration in the UI, ...).
Two remarks regarding two changes which are also included in this PR, but have nothing to do with the SAP Analytics Cloud source:

It seems the getLogoFromPlatform function is not used anymore, therefore I have removed it...
I have noticed that the ingestion sources are currently not sorted when they are selected in the UI...therefore I have added a sort function in SelectTemplateStep to sort the sources alphabetically, while making sure, that the "Custom" source is always displayed last...;-)

Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[x] Links to related issues (if applicable)
[x] Tests for the changes have been added/updated (if applicable)
[x] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[x] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Summary by CodeRabbit

New Features

Enhanced integration with SAP Analytics Cloud (SAC) for data ingestion.
Support for various SAC connection parameters, including tenant URL and token URL.
New source configuration for SAC integration introduced, expanding platform capabilities.

Bug Fixes

Improved sorting logic for displayed sources in the UI to prioritize non-custom entries.

Documentation

New documentation detailing SAC integration setup, configuration notes, and limitations.

Tests

Introduced integration tests for the SAC API to ensure proper interaction and validation.

@hsheth2 I have noticed, that the ingestion of the schema metadata of Import Data Models is extremely slow (20 seconds vs. 15 minutes with our productive SAC tenant), because the Import Data Service of the SAC is extremely slow...therefore I have added a config (by default it is enabled) to be able to disable the ingestion of schema metadata.
Additionally the Import Data Service of the SAC is frequently returning an 500 Internal Server Error...therefore I have also added a HTTPAdapter with a Retry to the Session object...ingesting the schema metadata is relatively new (I have implemented it before creating the PR, while we are using the rest of the source for some time): In the last two weeks the Internal Server Error already occurred three times, not sure if the Retry will help to solve this issue, but we will see...

@Masterchen09 looks like the cypress tests that are failing are mutations/ingestion_source.js and mutations/managing_secrets.js, which worries me a bit given the changes here. Not sure if it's a flaky test or a real failure

@hsheth2 Thanks for mentioning it, I didn't noticed that some tests are failing.
For mutations/ingestion_source.js it sounds logical, that the test is failing: Because of the sorting Snowflake is not visible anymore without scrolling...I hope this is fixed now.
I have currently no idea regarding mutations/managing_secrets.js- the changes to the UI should have no influence on this...let's see whether the test is passing now or not...

@hsheth2 The fix for mutations/ingestion_source.js yesterday solved the issue...the mutations/managing_secrets.js failed again, but I think the error message is just a bit misleading:
AssertionError: Timed out retrying after 10000ms: Expected to find content: 'Successfully created Secret!' but never did.
This is not the reason for the first attempt to fail, only the reason for the second and third attempt...the error message is caused by the failing first attempt, because the secret already exists and will not be created in the second and third attempt - the reason for failing the first attempt is the same reason why mutations/ingestion_source.js failed: Snowflake is not visible without scrolling...I have applied the same fix to mutations/managing_secrets.js which I have added yesterday to mutations/ingestion_source.js...let's see whether the test is successful now.
There is also another fix for the frozen dataclasses...from a functional point view it makes no difference, but is not correct to have a set in a frozen dataclass - now it's a frozenset. ;-)

CI failures are unrelated (nifi, nested domains), so merging through.


Use Get_balance to check ledger

For itn-create-accounts, use Get_balance to check whether accounts have been added, instead of downloading the ledger. That's faster.
Tested by @georgeee.
Closes #13874.

!ci-build-me

How it was tested:
On a cluster with >100k accounts in ledger funding step of an experiment was taking unexpectedly large quantities of time. And also could knock the node off in certain occasions.
When this fix was applied, these issues visibly disappeared.

!approved-for-mainnet


Need Logo?

Hello brother I am a graphic designer and I would like to contribute to your project by designing a striking logo, without any cost, I make free contributions to the app and repo. If you accept my request let me know your concerns,

Wow, is really appreciated if you can make an awesome logo! Is it possible
that you send a pull request so that we can confirm you signed the CLA?
2018年5月29日(火) 3:43 Carlos Garcia<EMAIL_ADDRESS>
Hello brother I am a graphic designer and I would like to contribute to
your project by designing a striking logo, without any cost, I make free
contributions to the app and repo. If you accept my request let me know
your concerns,
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/flexbox-layout/issues/431, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAwmydAL72JbnHvFDYN80hZhG0yuyQ6Uks5t3EU3gaJpZM4UQgPW
.

You could give me a general description of your repo, and if you had something in mind that you would like to see in your logo some form or some specific color.

The general description can be "FlexboxLayout is a library project which brings the similar capabilities of CSS Flexible Box Layout Module to Android." as in README.
Although it mentions about Android, I'm okay if the logo doesn't imply Android specific things.
In case if you are not familiar with Flexible Box Layout, it's a general term originally defined as a CSS spec. The Overview of the spec may also help.

one last question I would like to know in which platform, app or repo, will be using the logo, some link of the web where it will be used, this to consider the different resolutions of the image when exporting...

I'm considering the logo is going to be used in the GitHub repo itself and the page on the opensource google.


[QA] Viewer can modify shared folder

Tested with

ocis-master via https://github.com/owncloud-docker/compose-playground/blob/master/examples/hetzner-deploy/make_ocis_eos_compose_test.sh
firefox

Version details ocis-1.0.0-rc1

OCIS_VERSION:         master
ocis --version:       ocis version 3f8fb10
git log:              commit 3f8fb10990c37377a0e102492922ffd4807571d6 (HEAD -> refs/heads/master, refs/remotes/origin/master, refs/remotes/origin/HEAD)
eos --version:        EOS 4.6.5 (2019)
xrootd -v:            v4.11.0
bin/ocis contains:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Reproducer

use two differnet browser session. log in marie and einstein simultaneously.
marie creates a folder and adds a photo into that folder
marie shares the folder with einstein as Viewer
einstein visits 'shared with me' and accepts the folder
einstein visits his "Shares", and enters the folder.
einstein renames the photo.
einstein deletes the photo
einstein adds another file

Expected behaviour: einsteins should not be able to rename, delete add as role viewer.
This was changed since RC1. In rc1, all shared folders were readonly.

This was changed

it was actually broken, so it wasn't permission enforcement but rather that file operations were not working.
we fixed file operations to work correctly, so now the error is gone and the operations go through
however, we need now to work on actually enforcing permissions.
I didn't find any existing ticket for this, so this is likely an open task

@butonic were sharing permissions implemented with EOS ? AFAIK we are setting ACLs already, so this would be expected to work and is a bug if it doesn't ? or is it a task ?

reproducible on ocis driver


Highlighting for string type in generic

From @unional on June 21, 2016 18:41

Likely a TypeScript issue, but filed here as a start.
Using<EMAIL_ADDRESS>Copied from original issue: Microsoft/vscode#7933

Fixed by #257


Unable to build master branch: error rtoolbox

Cargo build output:
$ cargo build
error: failed to get `rtoolbox` as a dependency of package `rpassword v7.2.0 (/ddisk/projects/rpassword)`

Caused by:
  failed to load source for dependency `rtoolbox`

Caused by:
  Unable to update /ddisk/projects/rtoolbox

Caused by:
  failed to read `/ddisk/projects/rtoolbox/Cargo.toml`

Caused by:
  No such file or directory (os error 2)

Cause:
https://github.com/conradkleinespel/rpassword/blob/1af683038cd34cd9ffafe614af21f77ff6544e0e/Cargo.toml#L22
Also to note, I couldn't find any repo for the crate, only entry in crates.io. Could their be a posibility to replace them with just the packages that are re-exported inside the crate?

Hello there, I develop rpassword from here: https://github.com/conradkleinespel/duck/tree/master/projects/rpassword.  may want to clone that repository and build from there.
Let me know how it works out.

Yeah, now it works.
Though this a bit confusing when just cloning this standalone repo after reaching it from crates.io .

Totally get it, sorry for the confusion. I mostly maintain this on my own nowadays, after initial contributions from other people, so I just try to make it easy for me to keep alive.

Fair enough. Thank you for your work. I would suggest you to link the crates.io pages with monorepo. Great example would be ed25519 crate.

Good idea! Making a note of it, I've got a couple things I need to update. Thanks for the suggestion! :+1:


Highlight shortcode produces weird results with longer code blocks

With a code block of ~54 lines the resulting output is broken in two significance ways:

The number column of the "table" looks off
It add extra blank lines at the end of the output

How the highlighted code block looks in Qutebrowser

How the highlighted code block looks in Firefox

Affected page: https://vendion.me/blog/2021/08/05/using-git-to-generate-changelogs/#writing-out-the-changelog-file
Content file: https://git.sr.ht/~vendion/vendion.me/tree/master/item/content/posts/using-Git-to-generate-changelogs/index.md?view-source#L94
As you can see in the content file this code snippet does not have 88 lines.

Forgot to add in Hugo version, which may be useful:
hugo v0.85.0-724D5DB5+extended freebsd/amd64 BuildDate=2021-07-05T14:23:03Z+0000 VendorInfo=freebsd

This is related to the length of one or more of the lines, not the number of lines.
This is a CSS issue, not a Hugo issue. I suspect you have white-space set to pre-line or pre-wrap instead of pre.

Where would that be set? All my markup settings are the default as per this pgae as I'm not overriding anything in my own config nor am I setting anything at the content level.

The nix theme uses Bootstrap:
https://github.com/LordMathis/hugo-theme-nix/blob/master/layouts/partials/head.html#L15
pre code {
  padding: 0;
  font-size: inherit;
  color: inherit;
  white-space: pre-wrap;           /* THIS IS THE PROBLEM */
  background-color: transparent;
  border-radius: 0;
}

In the future, please use the forum (https://discourse.gohugo.io) for questions and troubleshooting. We prefer to use GitHub for verified bugs and vetted enhancements. Thanks.


Throw if bootRun task run against plugin

Avoid confusion with local development in "wrapper" mode, with both hoist-core and an app setup in a multi-project gradle build.
Devs should ensure they run bootRun from their app-specific gradle scope, not the combined top-level scope created by the wrapper.

Running through a PR as we might want to look / test a bit further - perhaps we could remove/disable the task, or (to the contrary) perhaps the task is important in some other context that this change would break...
But wanted to give devs some help, as the failure pattern is pretty obscure if you run from the wrapper, which is itself easy to do.

I like this change -- willing to try it!


More examples appreciated

Put in some more descriptive examples like gmf

#82 i am going to add 3 tutorials from ffmepg-libav-tutorialhttps://github.com/leandromoreira/ffmpeg-libav-tutorial#tldr-show-me-the-code-and-execution


[UST-01C] Non-Uniform Storage Pattern

UST-01C: Non-Uniform Storage Pattern

Type
Severity
Location

Code Style

USTB.sol:L29, L31

Description:
The overall codebase implements a pointer-based upgradeability-compatible storage pattern that is not applied in the USTB contract.
Example:
contract USTB is IUSTB, LayerZeroRebaseTokenUpgradeable, UUPSUpgradeable {
    using SafeERC20 for IERC20;

    address public constant UNDERLYING = 0x59D9356E565Ab3A36dD77763Fc0d87fEaf85508C;

    address public rebaseIndexManager;

    bool private _isMainChain;

Recommendation:
As is the case with all its dependencies, we advise the USTB contract to apply a similar pointer-based storage system.

Thank you for the insight provided in audit item UST-01C, concerning the storage pattern in our USTB contract.
Upon review, we acknowledge the discrepancy between the storage patterns used in the USTB contract and the base contracts from which it inherits. Your recommendation to unify the storage approach by implementing a pointer-based storage system across all contracts, including USTB, is well-received.
While our initial design was to leverage the simplicity of direct storage for ease of understanding and interaction in the final contract, we recognize the importance of maintaining consistency throughout the codebase. Such uniformity not only fosters better maintainability but also ensures that our contracts adhere to a standard pattern that could facilitate future upgrades and integrations.
To this end, we have decided to adopt the pointer-based storage pattern in the USTB contract.

Resolved in 083edf019e1a6dec17ecbb7fe2a00706d5c3b690.


Support native components with event handlers

Fixes #42, at least for native components.
Needs #43 and #44 to be merged first. Other than that this should be ready.

@Turbo87 thanks for merging the other PR. I rebased this, should be ready now as well! Don't want you to feel bored! :wink:

@simonihmig sorry the radio silence.
can you add this as a CLI option? maybe something like --transform-event-handlers? due to the mentioned caveats I think we probably shouldn't enable it by default.

due to the mentioned caveats I think we probably shouldn't enable it by default.

@Turbo87 sorry, I might have forgotten as some time has passed, but which caveats are you referring to?

but which caveats are you referring to?

I was talking about the second point of https://github.com/ember-codemods/tagless-ember-components-codemod#known-caveats
I've unfortunately seen several codebases already that did something like that :-/
Luckily, there is https://github.com/ember-template-lint/ember-template-lint/blob/master/docs/rule/no-passed-in-event-handlers.md now, that catches this issue, but probably not everyone is using that yet.


remove agent role, no packages for it upstream yet

This should fix all the failures from this commit 2d21f741f53a2c05b5d1f410725cfdd8ddcaf518

@leseb I am not sure how to deal with:
mv: cannot move ‘/opt/pypy/python’ to ‘/opt/bin/python’: No such file or directory

This is for CoreOS and fully unrelated to other PRs (all PRs are currently failing because of these errors)

@alfredodeza my bad, thanks!
Can you remove 'ceph-common-coreos' for now as well?

I am removing rbd-mirror as well. There aren't any packages yet.

Thanks and sorry for the pain.


fixed input to extractImageBlockType to be sent to the server properly

Notes
This PR fixes the input for extractImageBlockType to be sent as an array instead of a comma-delimited string so that the input settings are respected server side.
Testing
Manually inspected server-side after change that the correct input is sent properly.
Updated integration test.

LGTM! Great catch. If someone adds a new list param to the API, will it have the same issue and is there a generic way to handle it?

Thought about this - I don't think it would be safe to assume say, searching over the form object and if it contains ",", that we could assume it needs to be converted to an actual list.  I think we have to take it case by case and explicitly clean up the input unfortunately.


Tests for the /positionListForVoter endpoint

Description of the API Endpoint according to WeVoteServer’s documentation:
A list of all positions (support/oppose/info) for this voter relating to Candidates, Measures or Offices.
How to trigger the API endpoint:
Go to wevoute.us/yourpage.
You can add items to the position_list field by clicking on “Choose” or “Oppose” for a candidate.
Test cases that we covered:

Request does not contain a valid voter_device_id
Request contains a valid voter_device_id but there is no registered user associated with the id.
Suppose the user is registered and clicked on “choose” for one candidate

Hi @DaleMcGrew, I tried out a new approach for "stubbing" the database with sample data by directly invoking the model and saving a new record to the table (as opposed to invoking the API endpoint to save new data to the database).  I believe this simplifies the logic of the test (fewer things can go wrong) so that it focuses more precisely on the individual endpoint that is being tested.  I also wanted to do it this way so that my students could get comfortable using the models.  Please let me know if you have any feedback about this approach.

👍


Added analysis _2021

Sir, you can add the level and gssoc.
I've added the file to the backlog folder.

Thanks for your patience and ready to work on the change requests I made,
As a token of appreciation I merged without review, you will soon get a new badge on your profile from github for the same.

Thanks for your patience and ready to work on the change requests I made,
As a token of appreciation I merged without review, you will soon get a new badge on your profile from github for the same.

I appreciate it, sir, Thank you for guiding me.

Sir, you've added level 2 previously to the same PR #118. I've closed it due to conflicts.

corrected it


Crash when trying to play "Can't Stop the Feeling" from SPEIRMIX

Get this this song/simfile: https://zenius-i-vanisher.com/v5.2/viewsimfile.php?simfileid=29458
In OutFox, Game Start -> Single -> Normal
Find "Can't Stop the Feeling"
Change difficulty to Hard
OutFox 5.3 reliably crashes every time before any arrows arrive. Other songs work great.

The log:
log.txt

Appears to be fixed in 4.6.0 final

Root cause appears to have been the video file. Removing the video fixed it in RC3. Works with video in 4.6.0 final.

Yeah, this versions comes with a fix for the dependence used in videos. Thanks for reporting anyway


Re-render on prize array change from another component

I am trying to implement a round based roulette where each round a new item list prop is passed to the roulette, but I am getting errors in spinning such as the spin not starting from the beginning, it is just shifting to the close's winning index or just being stuck. I think it happens because the roulette isn't re-rendering fully when the new item list is being passed. I can provide more information if needed. Thank you.

Hi, @Gerind. Could you please send an example in CodeSandbox?On 6 Apr 2023, at 3:06 p.m., Gerind @.***> wrote:
I am trying to implement a round based roulette where each round a new item list prop is passed to the roulette, but I am getting errors in spinning such as the spin not starting from the beginning, it is just shifting to the close's winning index or just being stuck. I think it happens because the roulette isn't re-rendering fully when the new item list is being passed. I can provide more information if needed. Thank you.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

I have uploaded the main JSX files in this link https://codesandbox.io/s/boring-shape-8yjd6d?file=/src/LiveBattle2v2.jsx, but it will not run because it depends on many other props and api, socket calls to get information.

Thank you. I’ll take a look on it when I have time.On 6 Apr 2023, at 3:31 p.m., Gerind @.***> wrote:
I have uploaded the main JSX files in this link https://codesandbox.io/s/boring-shape-8yjd6d?file=/src/LiveBattle2v2.jsx, but it will not run because it depends on many other props and api, socket calls to get information.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

Well. It's a little bit complicated to deal with something you can't see live. But I'll try to suggest that there is no problem with re-render of items in the roulette. The problem might be due to async React action. So I'd like to say probably you should do something like:
setPrizes(...) // 1

setTimeout(() => {
    setPrizeIndex() // 2
    // 3 Start the roulette spinning like in this example - https://github.com/IvanAdmaers/react-roulette-pro/blob/main/example/src/App.js
}, 0)

The idea is to wait after setting new roulette prizes.

Thank you for the idea! I tried to implement something like this to my case, and it seems to be working, right now I am still testing it more but the way it is working now seems to be better than before because it is clearly starting the spin from the beginning of the item list. I will do more tests and update or ask again if anything changes. Thank you again.

No problem. I just have to say this is not the best way to use setTimeout for this. If follow the React rules we should use useEffect for this. But for new projects it's excusable)

I see maybe after i test it more and make it more stable and clean i will try to convert it to use useEffect.


Add expected distributeLink return

Fixing throw issue caused in test environment

Coverage remained the same at 78.738% when pulling 4ef9751300e3b4f4043c33602730ca1ff88a4804 on dcolt:v4.x into 6c092bdd155cc8d6564503beb703ea721958381a on klarna:v4.x.


Cant apply Easing

Can't compile, error:  EaseOutEasing is a 'type', which is not valid in the given context.
So how do I apply the property?

Here's an example:
<Animate Animation="Animations.ZoomIn" Easing="Easings.EaseInOutBack" Duration="TimeSpan.FromSeconds(0.5)" Delay="TimeSpan.FromSeconds(1)">
    <Counter></Counter>
</Animate>

I'll update the samples or the readme so that they include a sample of using an easing.


how to get shared xls file difference between two version?

What type of issue is this?
Question
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Target SharePoint environment
SharePoint Online
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[ ] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

browser version
SPFx version
Node.js version
etc

Issue description

I need to get all difference between shared xls file changes, if some one has modified the shared xls file in that case I need to list out all the changed what user has done in python. Please guide me how i can approach.

While you get the changes in the browser - https://support.microsoft.com/en-us/office/show-changes-that-were-made-in-a-workbook-978ceea7-bbf6-4337-bca7-22e7cc9892e8
There isn't a public API or documentation on how those changes were accumulated.
The browser shows you the endpoint Excel uses, but this doesn't make it an API that can be used.


Add more quick links to external resources

Such as WebDewey Search and Norsk WebDewey:

    <a
      :href="`https://coli-conc.gbv.de/cocoda/app/?fromScheme=http%3A%2F%2Fdewey.info%2Fscheme%2Fedition%2Fe23%2F&from=${concept.uri}`"
      title="open in Cocoda"
      target="cocoda">⤨
    <a
      :href="`https://deweysearchde.pansoft.de/webdeweysearch/executeSearch.html?query=${concept.notation[0]}`"
      title="open in WebDewey Search"
      target="webdewey_search">🔍
    <a
      :href="`https://data.ub.uio.no/skosmos/ddc/nb/page/?uri=${concept.uri}`"
      title="open in Norsk WebDewey"
      target="norsk_webdewey">no

This may also come with https://github.com/gbv/bartoc.org/issues/76 so we can manage the links in BARTOC.

Closing in favour of https://github.com/gbv/coli-ana/issues/60


[Changelog] Add entries for more resolved issues in Swift 5.1

@swift-ci Please smoke test and merge

Thank you!

Unrelated OSX test failure: "/Users/buildnode/jenkins/workspace/swift-PR-osx-smoke-test/branch-master/buildbot_incremental/swift-macosx-x86_64/test-macosx-x86_64/IRGen/Output/class_resilience.swift.tmp/class_resilience.swift:95:16: error: CHECK-SAME: is not on the same line as the previous match"
@slavapestov can you kick off another test & merge or simply merge this without it? Thank you!

@theblixguy yeah, I think that might be related to some ABI-related work we landed together with stale CMake cache artifacts.

@swift-ci Please smoke test macOS

@slavapestov Ah ok! Seems like the test has passed now!


feat: pixel style ui, link to gitbook

Let discus design changes later if we proceed developing the app
I've included link to the doc in another commit

It's irrelevant at this moment, because we stayed with previous design


Missing remote URL when using comfyui-mixlab-nodes on Comfyui

My work environment is on Paperspace Gradient Notebook. I am seeing that when i install custom nodes for comfyui-mixlab-nodes and comfyui-liveportrait .. and start the server I don;t see the remote gradient url ie the url to access comfyui remotely but i only see the local url for example
To see the GUI go to:
To see the GUI go to: http://<IP_ADDRESS>:6006https://<IP_ADDRESS>:6007
If i uninstall or delete this node i can see the remote url again
https://tensorboard-abcdefghijklmno.clg07azjl.paperspacegradient.com/
Can some body please provide some guidance on what might be happening?

i have the same problem, how to solve it

I notice that the init.py is hardcoded to <IP_ADDRESS> even though check on address is <IP_ADDRESS>. I think this is a bug and needs needs attention to make comfyui accessible with remote url.

J'ai le même problème, quelqu'un à trouvé la solution ?


Add instruction on using CLI in windows prompt

The example script in the Data Validation CLI section needs a little adaptation to work in windows prompt, otherwise double quotes won't be recognized and error will be reported as screenshot at bottom.
I may contribute an easy solution for windows to the README, but I don't know how to apply font color to output message in windows prompt.

@gitmagic-bot Looks like I don't have permission to add labels?

Hi @TheWindRider, thanks for reporting this. I think updating the README is a good idea.
As for coloring terminal output, I'm currently using the TermColors class. However, I think the __main__ module could be cleaned up a bit. I also found the termcolor module on PyPI and think it might be a better solution than using the TermColors class.
Any thoughts?

Ok I will work on the README.  @rightlag Would you prefer a new pair of screenshots, or more low-key modification like one line of example code that works in windows terminal?
I don't have good knowledge about the PyPI unfortunately.

Just adding the Windows command should be fine. Could you also add headers above the commands (e.g.):
macOS
$ command

Windows
> command


Add conditioning

Paper also describes local and global conditioning. In time series prediction we can treat these similar to positional encoding, i.e. season, time, etc. This would be similar to feeding in time as second channel input?
https://stats.stackexchange.com/questions/388130/wavenet-global-and-local-conditioning

done in main


Color palette: make updating the project smoother

Right now, after closing the palette editor modal, if there are palette changes, it flashes to the blocks view to reload and then goes back to the asset editor view.
Instead of calling reloadHeaderAsync to update the project, we will need to update every place that uses the palette by creating a subscription (data.subscribe) for palette changes.

The current behavior seems fine for now. We can reopen in the future if needed.


Add option to prefer an Email address to a Username

Description
Implement an option to prefer using email address as the username to upstream services.
Motivation and Context
With some providers the Username is an upstream Unique ID, like fex. in the
case of Google, where you get a stable string of characters that are guaranteed to be unique for Google..
When matching this with downstream databases, it's sometimes preferred to use
the email address as the  known identifier.
However, when mixing this with sometimes other sources, like htaccess, which
doesn't have a concept of an email address, it can turn difficult, as you can no longer only trust the "X-Forwarded-Email" header but must manage fallback handling, etc.
This change makes the headers prefer to use the Email address, if such exists,
for the Username identifier when passing data to downstream services.
Defaults to Off.
How Has This Been Tested?
Unittests only
Checklist:

[x] My change requires a change to the documentation or CHANGELOG.
[x] I have updated the documentation/CHANGELOG accordingly.
[x] I have created a feature (non-master) branch for my PR.

Now it's been live tested as well.

Thanks for the review, I'll fix that up on Monday and re-push my branch

Squashed the changes up, should be all clear.

Since there's some conflicts now vs. upstream/master, how should I proceed with this?  Do you want me to rebase my branch, or merge master into this?

Rebase would be preferable if that's ok

Rebased the changes.

So, rebasing drops the reviews as stale? Annoying.


ARROW-3928: [Python] Deduplicate Python objects when converting binary, string, date, time types to object arrays

This adds a deduplicate_objects option to all of the to_pandas methods. It works with string types, date types (when date_as_object=True), and time types.
I also made it so that ScalarMemoTable can be used with string_view, for more efficient memoization in this case.
I made the default for deduplicate_objects is True. When the ratio of unique strings to the length of the array is low, not only does this use drastically less memory, it is also faster. I will write some benchmarks to show where the "crossover point" is when the overhead of hashing makes things slower.
Let's consider a simple case where we have 10,000,000 strings of length 10, but only 1000 unique values:
In [50]: import pandas.util.testing as tm

In [51]: unique_values = [tm.rands(10) for i in range(1000)]

In [52]: values = unique_values * 10000

In [53]: arr = pa.array(values)

In [54]: timeit arr.to_pandas()
236 ms ± 1.69 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

In [55]: timeit arr.to_pandas(deduplicate_objects=False)
730 ms ± 12.7 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

Almost 3 times faster in this case. The different in memory use is even more drastic
In [44]: unique_values = [tm.rands(10) for i in range(1000)]

In [45]: values = unique_values * 10000

In [46]: arr = pa.array(values)

In [49]: %memit result11 = arr.to_pandas()
peak memory: 1505.89 MiB, increment: 76.27 MiB

In [50]: %memit result12 = arr.to_pandas(deduplicate_objects=False)
peak memory: 2202.29 MiB, increment: 696.11 MiB

As you can see, this is a huge problem. If our bug reports about Parquet memory use problems are any indication, users have been suffering from this issue for a long time.
When the strings are mostly unique, then things are slower as expected, the peak memory use is higher because of the hash table
In [17]: unique_values = [tm.rands(10) for i in range(500000)]

In [18]: values = unique_values * 2

In [19]: arr = pa.array(values)

In [20]: timeit result = arr.to_pandas()
177 ms ± 574 µs per loop (mean ± std. dev. of 7 runs, 10 loops each)

In [21]: timeit result = arr.to_pandas(deduplicate_objects=False)
70.1 ms ± 783 µs per loop (mean ± std. dev. of 7 runs, 10 loops each)

In [42]: %memit result8 = arr.to_pandas()
peak memory: 644.39 MiB, increment: 92.23 MiB

In [43]: %memit result9 = arr.to_pandas(deduplicate_objects=False)
peak memory: 610.85 MiB, increment: 58.41 MiB

In real world work, many duplicated strings is the most common use case. Given the massive memory use and moderate performance improvements, it makes sense to have this enabled by default.

I'll remove the WIP label once I've added some asv benchmarks for this

Memory profiler can be a bit noisy depending on the state of the gc. The math on memory usage didn't look right so I re-ran with 10M values and the results look more correct (about ~600MB difference between on/off)

Codecov Report

Merging #3257 into master will increase coverage by 1.15%.
The diff coverage is 93.33%.

@@            Coverage Diff             @@
##           master    #3257      +/-   ##
==========================================
+ Coverage   88.53%   89.69%   +1.15%
==========================================
  Files         539      480      -59
  Lines       72836    68847    -3989
==========================================
- Hits        64486    61752    -2734
+ Misses       8245     7095    -1150
+ Partials      105        0     -105

Impacted Files
Coverage Δ

python/pyarrow/lib.pxd
0% <ø> (ø)
:arrow_up:

cpp/src/arrow/type_traits.h
95.23% <ø> (ø)
:arrow_up:

python/pyarrow/tests/test_convert_pandas.py
95.18% <100%> (+0.14%)
:arrow_up:

cpp/src/arrow/util/hashing.h
99.26% <100%> (+0.01%)
:arrow_up:

python/pyarrow/pandas_compat.py
98.01% <100%> (ø)
:arrow_up:

cpp/src/arrow/type.h
89.67% <100%> (+0.06%)
:arrow_up:

cpp/src/arrow/python/arrow_to_pandas.cc
91.7% <100%> (+0.02%)
:arrow_up:

cpp/src/arrow/type.cc
91.46% <100%> (-0.03%)
:arrow_down:

python/pyarrow/table.pxi
71.1% <50%> (+0.23%)
:arrow_up:

python/pyarrow/array.pxi
68.68% <58.82%> (-0.33%)
:arrow_down:

... and 61 more

Continue to review full report at Codecov.

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

Running benchmarks on master. I hacked around ARROW-4117 and wasn't able to resolve ARROW-4118 when running these
[ 50.00%] ··· ============ ==========
              --             total
              ------------ ----------
               uniqueness   1000000
              ============ ==========
                 0.001      79.6±0ms
                  0.01      76.8±0ms
                  0.1       84.9±0ms
                  0.5       77.8±0ms
              ============ ==========

This PR:
[ 25.00%] ··· convert_pandas.ToPandasStrings.time_to_pandas_dedup                                                    ok
[ 25.00%] ··· ============ ==========
              --             total
              ------------ ----------
               uniqueness   1000000
              ============ ==========
                 0.001      25.7±0ms
                  0.01      30.7±0ms
                  0.1       96.7±0ms
                  0.5       183±0ms
              ============ ==========

[ 50.00%] ··· convert_pandas.ToPandasStrings.time_to_pandas_no_dedup                                                 ok
[ 50.00%] ··· ============ ==========
              --             total
              ------------ ----------
               uniqueness   1000000
              ============ ==========
                 0.001      77.9±0ms
                  0.01      77.7±0ms
                  0.1       77.7±0ms
                  0.5       77.6±0ms
              ============ ==========

@wesm Running benchmarks with asv run --python=same should work around these problems as this will not create a new installation.


fix: allow overflow on code snippets for tooltips

fixes #2506
allow overflow on code snippet container, to show type annotation tooltip correctly.
the other changes are just prettier autoformat. i can revert these if preferred.

Nice work
Reviewers: I'd run this and check that this doesn't start showing scrollbars in strange places on windows/linux

any update on this?

The PR currently has conflicts, but I'm happy to try and take a look locally to verify that it doesn't break things.

Looks good:

No weird scrollbars.


Segmentation Improvement

Hi, I noticed that there were some improvements on segmentation. For example:
risau -> r is au
budak -> bu dak
cerita -> cer i ta
and there were many more.
I did tried to cleared session after go through documentations but doesn't help much.
Thank you. 👍🏽

thanks for reminding me that! We are going to release better segmentation model using Transformer!

Thanks Husein! Looking forward to it 👍🏽

https://malaya.readthedocs.io/en/latest/load-segmentation.html
Install latest version.


Performance upgrades

[x] Save body-parsing until later
[x] See if there is any way to optimize request pipeline (maybe by optimizing angel_route?)
[x] Verify that pre-injections work, and see if they can be faster as well
[x] Routes can probably be streamlined into a linear tree

i.e. get rid of chained routes, just copy them up. This is already doable, and could improve performance.

Optimize method should pre-inject BEFORE flattening. Add a null check to prevent flattening twice.


No longer loading too many tiles

Tracking tile loading separately from tile drawing for purposes of determining what to load next. Fixes #1020

Just tested, works perfectly 👍

Ha, I think I found a bug actually. Some tiles do not get loaded anymore when they should:
https://drive.google.com/file/d/0BzAcRIdakTF0emp5YnpCVGl3R2s/view?usp=sharing

@avandecreme Wow, thank you for catching that! Looks like we need to be more aggressive about clearing out the old loadingCoverage since loading is a more ephemeral state. I think I've fixed it now; please take another look :)
@johnsusi @CaptainCodeman @rdlester Do any of you want to take a look as well?

Sure, I'll try and test it out today.

It's been a while since I"ve looked at this code, so I don't remember how failure cases are handled, but what happens if an attempted load fails and the image isn't fetched? If you'd like to retry, it seems loadingCoverage should get unset.

Seems to be working OK for me 👍

@rdlester Good thought! Here's what happens: when a failure is encountered, the tile's loading flag is set back to false. The loadingCoverage is reset to empty at the beginning of each frame and repopulated with the latest loading and loaded flags, so it picks up the failure properly.
@CaptainCodeman Thank you for testing it!

I haven't found the time to test with the reset in updateViewport, but won't we load too many tiles if the viewport is moving?

@avandecreme The loading and loaded flags of the individual tiles aren't being reset, so it doesn't cause any additional tiles to load. It's just that loadingCoverage is being refreshed every frame to make sure we have the correct latest information based on those flags from the individual tiles. Does that make sense?

Alright, good for me then.

@avandecreme Great, thank you for looking it over!
I think all that's left is to get to the bottom of the issue @susanmonnelly reported in #1020. Susan, any thoughts after my last response?

Okay, we've got a clean bill of health on this from @susanmonnelly (who had found an issue).


Corner case in aws_byte_cursor_advance_nospec

The comment on line 239 of byte_buf.h indicates that aws_byte_cursor_advance and aws_byte_cursor_advance_nospec should have the same behavior, but there is a corner case where they do not. The problem here occurs when the cursor argument has cursor->len == SIZE_MAX >> 1. The procedure aws_byte_cursor_advance_nospec then passes cursor->len + 1 as the bound argument to aws_nospec_index (line 256 in byte_buf.h), and since cursor->len + 1 is larger than SIZE_MAX >> 1 in this case, aws_nospec_index returns 0. Here's an example that exhibits the different behavior:
#include <aws/common/byte_buf.h>
#include <stdio.h>
#include <stdlib.h>

int main (void) {
    // Make cursor
    struct aws_byte_cursor *cursor0 = malloc(sizeof(*cursor0));
    cursor0->len = (SIZE_MAX >> 1);
    // not going to deref pointer, so don't bother allocating
    cursor0->ptr = NULL;

    // Make copy of cursor0
    struct aws_byte_cursor *cursor1 = malloc(sizeof(*cursor1));
    cursor1->len = cursor0->len;
    cursor1->ptr = cursor0->ptr;

    size_t len = cursor0->len;

    struct aws_byte_cursor rv = aws_byte_cursor_advance(cursor0, len);
    struct aws_byte_cursor rv_nospec = aws_byte_cursor_advance_nospec(cursor1, len);

    // Expect the same result from aws_byte_cursor_advance and
    // aws_byte_cursor_advance_nospec, so nothing should be printed.
    if (rv.len != rv_nospec.len) {
        printf("Different outputs.\n"); // Prints.
    }
    free(cursor0);
    free(cursor1);
    return 0;
}

For consistency, we should possibly just use the nospec variant for all cursor advances, particularly if we standardize on using cursors for read and buffers for write.

The kernel only mitigates spectre gadgets within the kernel itself. Spectre gadgets in userspace require userspace mitigation.


fix conv_concat matcher when original convolutions are grouped

The original find_concat_conv pass from #2639  has a bug when it matches with original convolutions that have group != 1. This is a fairly simple change to handle that case.

@shivadbhavsar  can you test performance before and after on a couple of the models listed above that dropped in performance.


error to occur at powershell for Enable-WindowsOptionalFeature command

Please fill out the below information:

Your Windows build number:  (Type ver at a Windows Command Prompt)

my ver result at cmd
Microsoft Windows [Version 10.0.17134.829]

What you're doing and what's happening:  (Copy&paste the full set of specific command-line steps necessary to reproduce the behavior, and their output. Include screen shots if that helps demonstrate the problem.)

the command for wsl2 result at powershell
PS C:\Windows\system32> Enable-WindowsOptionalFeature -Online -FeatureName VirtualMachinePlatform
Enable-WindowsOptionalFeature : Feature name VirtualMachinePlatform is unknown.
At line:1 char:1

Enable-WindowsOptionalFeature -Online -FeatureName VirtualMachinePlat ...

  + CategoryInfo          : NotSpecified: (:) [Enable-WindowsOptionalFeature], COMException
  + FullyQualifiedErrorId : Microsoft.Dism.Commands.EnableWindowsOptionalFeatureCommand*

Strace of the failing command, if applicable:  (If some_command is failing, then run strace -o some_command.strace -f some_command some_args, and link the contents of some_command.strace in a gist here).

Don't you have to be on the latest Windows version for this to work?

my systeminfo result next follow, is this latest?
OS Name:                   Microsoft Windows 10 Pro
OS Version:                10.0.17134 N/A Build 17134

latest windows version is 1903 so I will upgrade now and then try again.

It works!
very nice performance, thanks Microsoft :)

On a side note, if you are not using the latest windows insider build, you can still try out wsl2 in a hyper-v virtual machine.

download windows insider build 18917 from here
create a hyper-v virtual machine with the downloaded image
enable nested virtualization for that virtual machine on your host
install wsl2 in the virtual machine

is there any performance difference between the 2 methods?
I ask because my employer has a custom update schedule (not sure what it is) but we're a bit behind version-wise.

WSL 2 needs Windows 10 Insider Preview Build 18917 or higher so  you should update to latest windows release please check this link to update:
https://support.microsoft.com/en-us/help/4028685/windows-10-get-the-update


Appstudio update test-component-pac-ftxs

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-ftxs-on-pull-request has failed.

StatusDurationName

✅ Succeeded
24 seconds
init

✅ Succeeded
22 seconds
clone-repository

❌ Failed
17 seconds
build-container

✅ Succeeded
13 seconds
show-summary

Failure reasontask build-container has the status "Failed":
* reading JSON file "/secrets/registry-auth/config.json": unmarshaling JSON at "/secrets/registry-auth/config.json": unexpected end of JSON input

Pipelines as Code CI/test-component-pac-ftxs-on-pull-request has failed.

StatusDurationName

✅ Succeeded
24 seconds
init

✅ Succeeded
22 seconds
clone-repository

❌ Failed
17 seconds
build-container

✅ Succeeded
13 seconds
show-summary

Failure reasontask build-container has the status "Failed":
* reading JSON file "/secrets/registry-auth/config.json": unmarshaling JSON at "/secrets/registry-auth/config.json": unexpected end of JSON input

Pipelines as Code CI/test-component-pac-ftxs-on-pull-request has failed.

StatusDurationName

✅ Succeeded
24 seconds
init

✅ Succeeded
22 seconds
clone-repository

❌ Failed
17 seconds
build-container

✅ Succeeded
13 seconds
show-summary

Failure reasontask build-container has the status "Failed":
* reading JSON file "/secrets/registry-auth/config.json": unmarshaling JSON at "/secrets/registry-auth/config.json": unexpected end of JSON input


Redirects found but not reflected in PR

We're detecting redirects but they're not being reflected in the package list diff. For example:
NOT FOUND: https://github.com/tofplay/scriptkit.git (deleting package)
package 6400 ...
DELETE https://github.com/tuist/xcbeautify.git -> https://github.com/cpisciotta/xcbeautify.git (exists)

Both https://github.com/tofplay/scriptkit.git and https://github.com/tuist/xcbeautify.git remain in the package list when they should be removed.
I don't think this is related to yesterday's change but it might be due to the move to the API-based dependency check. I have the suspicion that it re-introduces the packages we're flagging for deletion.

Yes, the diff after the redirect check shows these are removed as expected:
68,70d67
<   "https://github.com/9tqP/GraphicsLibs.Swift.git",
<   "https://github.com/9tqP/MetalCanvas.git",
<   "https://github.com/9tqP/ProcessLogger.Swift.git",
3392c3389
<   "https://github.com/Kjuly/KYImageProcessing.git",
---
>   "https://github.com/Kjuly/KYImageProcessor.git",
3699d3695
<   "https://github.com/maatheusgois/DebugSwift.git",
5028d5023
<   "https://github.com/quiltt/quiltt-connector-swift-sdk.git",
6361d6355
<   "https://github.com/tofplay/scriptkit.git",
6421d6414
<   "https://github.com/tuist/xcbeautify.git",

I have a simple fix for this but it has a (low risk) race condition we I need to fix properly. It goes like this:

server has []
nightly job starts
p1 is newly added
server ingests p1
nightly makes API call and gets [p1]
we compute let deleted = Set(server).subtracting(packageList) which is now [p1] in the quick fix implementation
we end up removing p1 again

The proper fix is: CheckRedirects writes out added/deleted packages so we don't have to rely on set operations.
This is low risk, because we'd have to merge in a new package at the right time (which we likely won't do in the middle of the night) and the ingestion (which runs every 5mins) would need to run right in that window (which is now small, due to recent speedups) before the API call is made.
OK for the moment to get redirects reflected but will fix properly ASAP.

Quick fix worked: https://github.com/SwiftPackageIndex/PackageList/pull/6174/files


Switch to tsup

What will this PR do:

switch to tsup library (instead of regular esbuild)
output both commonjs and esm files
add tests folder with minimal project setups for cjs, esm and ts version to test if cypress-visual-regression can be started
downgrade to chalk v4 (because of cjs support)

Sure, I will 👍.
I will let you know
On Tue, Jun 4, 2024 at 8:21 PM Jovan Ilić @.***> wrote:

@.**** commented on this pull request.
In tests/ts/package.json
https://github.com/cypress-visual-regression/cypress-visual-regression/pull/246#discussion_r1626438813
:

@@ -0,0 +1,18 @@
+{

"name": "ts",
"version": "1.0.0",
"description": "ts testing project",
"main": "index.ts",
"type": "module",

Solved it, and released a temporary 5.0.2-alpha release, which works for
all my example projects. Do you mind @area73 https://github.com/area73
to have a go with it on your side and let me know if it works?
—
Reply to this email directly, view it on GitHub
https://github.com/cypress-visual-regression/cypress-visual-regression/pull/246#discussion_r1626438813,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPPYDOI3G3RCD3A3MZ4X2DZFYATHAVCNFSM6AAAAABH3A2DOWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAOJXGEYTEOJTGQ
.
You are receiving this because you were mentioned.Message ID:
<cypress-visual-regression/cypress-visual-regression/pull/246/review/2097112934
@github.com>

--
Rodrigo Erades

@Zaista , I tested 5.0.2-alpha with a project that was previously failing with version "5.0.1" and now it works like a charm   👌.


New command: purview sensitivitylabel get

Usage
m365 purview sensitivitylabel get  [options]
Description
Get a sensitivity label
Options

Option
Description

-i, --id <id>
The Id of the sensitivity label

Examples
Get a sensitivity label
m365 purview sensitivitylabel get --id 6f4fb2db-ecf4-4279-94ba-23d059bf157e

Default properties
No response
Additional Info
API request: https://learn.microsoft.com/en-us/graph/api/security-sensitivitylabel-get?view=graph-rest-beta
Let's also add this to the docs:

Attention: This command is based on an API that is currently in preview and is subject to change once the API reached general availability.

Hi @nicodecleyre, did you check if we can use the $filter query parameter on the relevant endpoint? That would help us to retrieve it by name as an alternative.
For the retention labels this was impossible as of yet, but perhaps for sensitivity labels it does work.

That works, updated the specs!

@martinlingstuyl any more comments 🤔? I guess @nicodecleyre did an awesome job and we may open it up ?

Hi @nicodecleyre, lets add the userName and userId options as well, to make it possible to retrieve the labels for a specific user.
This is because the endpoints support /me and /users

Hi @nicodecleyre, lets add the userName and userId options as well, to make it possible to retrieve the labels for a specific user.
This is because the endpoints support /me and /users

Missed your comment here, don't know how.. Sorry about that.
Updated the specs, you think we can open this to the community?

Hi @nicodecleyre, what I wrote here applies to this command as well.
Could you rewrite the option descriptions for userId and userName? Something like this maybe?:

Optionally specify this if you want to get a list of sensitivity labels that the user has access to. Specify either userId or userName but not both.

Could you also add a Remarks section explaining that you should use userId and userName in app only scenario's?

Hi @nicodecleyre, what I wrote here applies to this command as well.
Could you rewrite the option descriptions for userId and userName? Something like this maybe?:

Optionally specify this if you want to get a list of sensitivity labels that the user has access to. Specify either userId or userName but not both.

Could you also add a Remarks section explaining that you should use userId and userName in app only scenario's?

Thank you for your feedback! Updated the specs

Can we open this issue for the community or de we need to check a few thing first?

Just checked... The $filter query param is not yet available on this endpoint. So let's remove the name option for now.

Just checked... The $filter query param is not yet available on this endpoint. So let's remove the name option for now.

updated!

Can I work on this?


Swift 5.8 - KFImageRenderer Color.clear must have Optional type, not 'AnyView'

Check List
Thanks for considering to open an issue. Before you submit your issue, please confirm these boxes are checked.

[x] I have read the wiki page and cheat sheet, but there is no information I need.
[x] I have searched in existing issues, but did not find a same one.
[x] I want to report a problem instead of asking a question. It'd better to use kingfisher tag in Stack Overflow to ask a question.

Issue Description
The line 52 in KFImageRenderer.swift file returns the error: "Initializer for conditional binding must have Optional type, not 'AnyView'" since Xcode has been updated to Swift 5.8.
Reproduce
Try to build a project in Xcode 14.3 with Swift 5.8 that includes the KingFisher package.

#2052


Support for bleeding edge versions of Tensorflow

Hi,
I am thinking about using SageMaker for research purposes and I need to use the latest tools released with the last version of TF.
Yet, TensorFlow 2.0.0 has been released more than 1 month ago and it seems that there is still no support for it.
So I wonder : what is the timeframe of AWS for releasing the newer versions of the most popular Deep Learning frameworks?
Knowing that the delay can be as long as a month makes me worry about using SageMaker at all.
As a side note (I don't need it, but it should also be noted), TensorFlow 1.15.0 has been released 20 days ago and it also seems missing from the supported frameworks.
I just opened the same issue in the sagemaker-python-sdk repo because I felt the SDK and the Dockerfiles of this repo were sufficiently distinct objects to have their own issue each, feel free to close this one if that is not the case.

Thank you for your interest ! We plan to add support for Tensorflow 1.15 and Tensorflow 2.0. Stay tuned!

+1 for Tensorflow 2.0 support. It really makes no sense for me to start using Sagemaker when trying to build custom containers requires more effort than just building up an EC2 node from the deeplearning AMI. I really appreciate your work on releasing a Tensorflow 2.0 container. Do you have any time-frame on that. Or do you know of any github repos where someone has created a custom Tensorflow 2.0.0 gpu container?

For an example of how to build your own custom TF 2.0 image for use with SageMaker, see https://docs.aws.amazon.com/sagemaker/latest/dg/build-container-to-train-script-get-started.html
For the base image, you can use tensorflow/tensorflow:2.0.0b1, per this StackOverflow question.

Thanks so much @laurenyu that is great. This will help a lot. I was reading through some of the dockerfile code in the repo trying to figure out how to adapt it. The links you gave will help a ton. Thanks so much.

I thought I would check if there are any updates to Tensorflow 2.0 availability in advance of the AWS conference next week? Are they planning any new releases of the containers? Thanks.

Unfortunately we have no roadmap to share at this time. Thank you for your patience!

Is there any on news on TF 2.0 support?

Thank you for using Sagemaker
Tensorflow-1.15 was released earlier this month. Tensorflow-2.0 containers have also been released 3 days ago.


⚠ Bofrost.fr has degraded performance

In 6d79947, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1181 ms

Resolved: Bofrost.fr performance has improved in bd3e896 after 6 minutes.


iOS Crash with Unity 2020

Building and deploying a Unity project to iOS will result in a crash when a connection is attempted. It seems to stem from this issue: https://issuetracker.unity3d.com/issues/ios-player-crashes-with-litenetbugmain-when-accessing-ipinterfaceproperties-dot-gatewayaddresses.
In 2020.1.0f1, if IPInterfaceProperties.GatewayAddresses is used, the iOS Player will crash with:
Thread 1 Queue : com.apple.main-thread (serial)
#0    0x0000000104d7f4b8 in il2cpp::icalls::System::System::Net::NetworkInformation::MacOsIPInterfaceProperties::ParseRouteInfo_internal(Il2CppString*, Il2CppArray**) ()
#1    0x0000000104e7c95c in MacOsIPInterfaceProperties_get_GatewayAddresses_m1DA309123026DC1EA712DC469FACF97E94C853C2 ()
#2    0x0000000104dd95f4 in Test_GetLocalIP_mB4CF5A5682B9802067BC179FAD2F8D326A1A1910 ()
#3    0x0000000104838cac in RuntimeInvoker_TrueVoid_t700C6383A2A510C2CF4DD86DABD5CA9FF70ADAC5(void (*)(), MethodInfo const*, void*, void**) ()
#4    0x0000000104dc110c in il2cpp::vm::Runtime::Invoke(MethodInfo const*, void*, void**, Il2CppException**) ()
#5    0x0000000104ad03a4 in scripting_method_invoke(ScriptingMethodPtr, ScriptingObjectPtr, ScriptingArguments&, ScriptingExceptionPtr*, bool) ()
#6    0x0000000104ad946c in ScriptingInvocation::Invoke(ScriptingExceptionPtr*, bool) ()
#7    0x0000000104ae53f4 in MonoBehaviour::InvokeMethodOrCoroutineChecked(ScriptingMethodPtr, ScriptingObjectPtr, ScriptingExceptionPtr*) ()
#8    0x0000000104ae5678 in MonoBehaviour::InvokeMethodOrCoroutineChecked(ScriptingMethodPtr, ScriptingObjectPtr) ()
#9    0x0000000104ae4030 in MonoBehaviour::DelayedStartCall(Object*, void*) ()
#10    0x00000001049ae2cc in DelayedCallManager::Update(int) ()
#11    0x0000000104a2c3c0 in ExecutePlayerLoop(NativePlayerLoopSystem*) ()
#12    0x0000000104a2c3f4 in ExecutePlayerLoop(NativePlayerLoopSystem*) ()
#13    0x0000000104a2c64c in PlayerLoop() ()
#14    0x0000000104c135f4 in UnityPlayerLoopImpl(bool) ()
#15    0x0000000104816b98 in UnityRepaint ()
#16    0x0000000104816a74 in -[UnityAppController(Rendering) repaintDisplayLink] ()
#17    0x000000018d819fac in CA::Display::DisplayLink::dispatch_items(unsigned long long, unsigned long long, unsigned long long) ()
#18    0x000000018d8e4fbc in display_timer_callback(__CFMachPort*, void*, long, void*) ()

From what I can tell, this is a Unity regression, as it doesn't crash in Unity 2019.4. I was just wondering if you are aware of any workaround that can be done while they fix this, as this seems to be called as part of the connect flow.
public class Test : MonoBehaviour
{
    void Start()
    {
        GetLocalIP();
    }

    public static void GetLocalIP() {
        var targetList = new List<string>();
        try
        {
            foreach (var ni in NetworkInterface.GetAllNetworkInterfaces())
            {
                if (ni.NetworkInterfaceType == NetworkInterfaceType.Loopback ||
                    ni.OperationalStatus != OperationalStatus.Up)
                    continue;

                var ipProps = ni.GetIPProperties();

                // THIS LINE WILL CRASH
                if (ipProps.GatewayAddresses.Count == 0)
                    continue;

                foreach (var ip in ipProps.UnicastAddresses)
                {
                    var address = ip.Address;
                    if (address.AddressFamily == AddressFamily.InterNetwork)
                        targetList.Add(address.ToString());
                }
            }
        }
        catch {
            // No op
        }

        Debug.Log(string.Join(",", targetList));
    }
}

Library version: latest (f01cca3eb9bc3c135787ea2ec2443b45c35d4d2e)
Framework: Unity
OS: iOS

@chiedozi GetLocalIp isn't used in connection code, only in NatPunchModule.

Ah, you're right. Sorry, it was one of my local edits that was calling this code. I will close this as it's a bug on the Unity side. Thanks!


warn and set to UTF-8 as early as possible

Description
Point people in the right direction to avoid encoding dramas.
Hard to give more specific instructions though - they depend on the platform, shell and how long-lived they want the change to be.
We shouldn't set this variable on a user's behalf either, both because it may clobber an encoding they intentionally and it may overwrite other settings in RUBYOPT.
cc @zendesk/vegemite
References
https://support.zendesk.com/hc/en-us/community/posts/203461336-Problems-with-accents-in-templates-
Risks

[medium] zat stops working on certain windows versions if they don't support UTF-8.

Tested in powershell on windows 10, seems good


Crew Sheet, Consort name is not showing under the Consort tab.

Consort name is not showing.

I am using the below UI mod as well.
https://github.com/BBurns42/rpg-styled-ui

Sorry, this one you’ll need to take this up with the main Blades system, as my module doesn’t currently do anything to the crew sheets.
https://github.com/Dez384/foundryvtt-blades-in-the-dark
On Jun 27, 2024 at 3:02 PM -0700, eaganlai @.***>, wrote:

Consort name is not showing.
image.png (view on web)
I am using the below UI mod as well.
https://github.com/BBurns42/rpg-styled-ui
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


Highlighted messages, Fix for long text in quote

fix: prevent overflow of words in the message or quoted message bubble
refactor: remove overridden rule
fix: allow media widget to adapt to smaller container sizes and avoid overflow
fix: allow url-link to adapt to the container, avoid unbreakable long words to overflow, underline links in text on hover, focus, active for better UX feedback
fix: avoid unbreakable long words to overflow in quoted message preview
fix: do not squeeze quoted message preview in thread
fix: linting
feat: class for highlighted messages
feat: quoted messages are clickable
chore: semantic release v

:tada: This PR is included in version 2.8.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Delete Order in User Interface

Hey team! Please add your planning poker estimate with Zenhub @even1111112 @PepperandSummer @un-poco @yjhsieh2022


Add more test cases for multiple WDT model file ordering

Add two more test cases for multiple WDT model file ordering. One is to have two model files in the image and another is to have multiple model files in both image and configmap.
Also refactor existing ItMiiConfigMap and ItMiiConfigMapOverride to use the new common test util method for creating a config map.
External Jenkins job is clean in kind cluster: https://build.weblogick8s.org:8443/view/all/job/weblogic-kubernetes-operator-kind-new/207/.

https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-model-in-image-tests-10/203/ was before the last submit.

The latest version are clean on Jenkins too:
https://build.weblogick8s.org:8443/view/all/job/weblogic-kubernetes-operator-kind-new/228
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-model-in-image-tests/797


Update error messages to sentence case and punctuation

we like nice error messages. I think you just need to pull from main and resolve those conflicts then should be good!


XML variable substitution in applicationsettings when using type: ArrayOfString

Hello,
How should I provide values to set the values in a variable of type ArrayOfString in applicationsettings?:
 <applicationSettings>
    <myApp.Properties.Settings>
      <setting name="EmailList" serializeAs="Xml">
        <value>
          <ArrayOfString xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema">
<EMAIL_ADDRESS><EMAIL_ADDRESS>          </ArrayOfString>
        </value>
      </setting>
    </CHAT.Properties.Settings>
  </applicationSettings>

Thanks in advance

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

ID: 315ff823-9992-9f9b-62a1-42ce622ed41c
Version Independent ID: 72ec3ffc-1dc9-b2dc-e3d4-4f14deae3720
Content: File transforms and variable substitution - Azure Pipelines
Content Source: docs/pipelines/tasks/transforms-variable-substitution.md
Product: devops
GitHub Login: @alexhomer1
Microsoft Alias: ahomer

@diegoortizmatajira Please accept our apologies for the delay in getting to this issue. It looks like you were working through an issue with your scenario or implementation, rather than an issue with the documentation. Here are a few options where you might consider asking similar questions in the future:

Azure DevOps on Stack Overflow
Azure DevOps Support Bot