Additions to classpath

Thanks for sharing this great piece of software!  It's a great utility.  I'm having an issue, though.  I want to dynamically add new jars to a process while it is running.  To do this, I'm getting an instance of the URLClassLoader and calling addURL through reflection, and also updating the java.class.path system property.  If I use Class.forName, I can then get an instance of one of the classes, so I know the JVM class-loading is working with this hackery (jars are from an https url, if it matters).  If I do all of this before calling the FastClasspathScanner the first time, it also finds the classes.  However, if I call FastClasspathScanner.scan() once, modify the classpath, and then create a new instance and scan again, I don't see the newly added classes.  This implies some sort of caching is going on.  Do you know of any simple workarounds?  I didn't see anything in the docs.

Maybe I will just go a different route on this.  I felt like adding jars to the classpath would be the easiest approach, but strictly speaking for my use case, I don't need the classes.

@bkrahmer Sorry for the delay, I have been in the process of moving to another apartment.
FastClasspathScanner doesn't do any sort of caching in static fields. However, many ClassLoaders do their own caching, including the JVM system classloader. That doesn't seem to fully explain the problem you're seeing though, because you're seeing instances of classes not being found at all -- which is strange.
One thing I should point out is that defining your own URLClassLoader is quite dangerous: if your custom URLClassLoader loads an instance of a subclass, and the system classloader has already loaded and cached the class' superclass, casting an instance of the subclass to the superclass can give a runtime ClassCastException. You can also end up with cases where the same class can be loaded twice by different classloaders, and instances of those classes are deemed of different type by the runtime. (This is only two of several nasty issues that can occur...)
In a later version of FastClasspathScanner, I'm going to make it possible to scan any classpath for classes or files without enabling classloading. You can actually already do this by calling .overrideClasspath(String cp) -- but all the above caveats apply if you try loading classes with this scan override method, e.g. if you use MatchProcessors. However, if you are just using a ScanResult to get the names of matching classes, no classloading occurs, so this is totally safe.
If you can please provide a bit more detail though, I'd like to dig further into why this wasn't working for you, since I think there may be something useful to learn here, and I'd like FCS to work for as many unusual cases as possible. Thanks!

Hi Luke.  Thanks the reply.  It's great to see your interest in my issue.  I know the dangers of creating my own classLoader, so I was trying to avoid that.  I'll append my unit tests here which I think clearly illustrate the issue.  I had missed overrideClasspath, but now that I've added it, it had no effect.  Looking forward to your analysis.
thanks,
brian
`@Test
public void testGetServiceDefinition() throws ClassNotFoundException {
String serviceName = "com.sixt.service.journey";
Map<String, String> protoFiles = apiLoader.getServiceProtoFiles(serviceName); //modifies classpath
assertThat(protoFiles).hasSize(1);
assertThat(protoFiles.keySet()).contains("journey.proto");
Class<?> responseClass = Class.forName("com.sixt.service.journey.api.JourneyOuterClass$ReserveVehicleCommand");
assertThat(responseClass).isNotNull();
FastClasspathScanner cpScanner = new FastClasspathScanner();
ScanResult scanResult = cpScanner.scan();
List oldProtobuf = scanResult.getNamesOfSubclassesOf(GeneratedMessage.class);
List newProtobuf = scanResult.getNamesOfSubclassesOf(GeneratedMessageV3.class);
assertThat(oldProtobuf.size() + newProtobuf.size()).isGreaterThan(0);
}
@Test
public void testClassPathChanging() throws ClassNotFoundException {
    FastClasspathScanner cpScanner = new FastClasspathScanner();
    ScanResult scanResult = cpScanner.scan();
    List<String> oldProtobuf = scanResult.getNamesOfSubclassesOf(GeneratedMessage.class);
    List<String> newProtobuf = scanResult.getNamesOfSubclassesOf(GeneratedMessageV3.class);
    int sizeBefore = oldProtobuf.size() + newProtobuf.size();

    String serviceName = "com.sixt.service.journey";
    Map<String, String> protoFiles = apiLoader.getServiceProtoFiles(serviceName); //modifies classpath
    assertThat(protoFiles).hasSize(1);
    assertThat(protoFiles.keySet()).contains("journey.proto");
    Class<?> responseClass = Class.forName("com.sixt.service.journey.api.JourneyOuterClass$ReserveVehicleCommand");
    assertThat(responseClass).isNotNull();
    cpScanner = new FastClasspathScanner().overrideClasspath(System.getProperty("java.class.path"));
    scanResult = cpScanner.scan();
    oldProtobuf = scanResult.getNamesOfSubclassesOf(GeneratedMessage.class);
    newProtobuf = scanResult.getNamesOfSubclassesOf(GeneratedMessageV3.class);
    int sizeAfter = oldProtobuf.size() + newProtobuf.size();
    assertThat(sizeAfter).isGreaterThan(sizeBefore);
    assertThat(oldProtobuf.size() + newProtobuf.size()).isGreaterThan(0);
}`

Here is part of the function referenced, which does the actual modification of the classpath.  Again, this is a jar from an https server (no authentication required)
private void addJarToClassPath(URL inputURL) {         Method method;         try {             method = URLClassLoader.class.getDeclaredMethod("addURL", URL.class);             method.setAccessible(true);             method.invoke(ClassLoader.getSystemClassLoader(), inputURL);             String classPath = System.getProperty("java.class.path");             System.setProperty("java.class.path", classPath + ":" + inputURL.toString());         } catch (Exception e) {             logger.warn("Caught exception adding jar to classpath", e);         }     }

@bkrahmer nice to know my code is being used by Sixt -- I rented one of your cars recently in Norway :-)
FCS doesn't currently support non-local classpath elements. I was wondering if I'd ever run into cases where users need to include jars over http(s), but you're the first one that has reported this so far. Technically the spec allows it, but due to efficiency considerations, I have never seen anyone try to do this before now!
So now that I understand that you're compiling protobuf definitions on the fly, it makes sense that you would want to modify the classpath at runtime. You're really taking Java places it wasn't really designed to go though, based on the caching mechanisms built into the system classloaders! To get this to work well, you might need to pull in some sort of third-party dynamic classloader mechanism that is able to not only load classes, but hot-replace instances of classes that are already loaded. This is a complex topic, however...
As far as why FCS doesn't find the classes after you modify the classpath -- it's hard to say without looking at the logs. Can you please generate logs for the two different scan attempts (the one that works and the one that doesn't, with all other things being equal), by calling .verbose() before .scan(), and attach them here?

PS please also let me know if you really do need to access classpath elements over https, maybe I need to add that capability. It's not ideal though, since it will cause the jar to have to be fetched more than once -- once by FCS, once by the classloader if/when the classes are loaded.

I changed my code to download the jar, and added that to the classpath.  It works!  Thanks for the help!
brian

Great, I'm glad it works now! I should probably throw an exception if http(s) elements are discovered on the classpath, rather than failing silently. Or maybe I should just allow this, with the performance caveat. I'll reopen this as a reminder to fix this.

Note to self: Need to abstract away NestedJarHandler to be able to handle other temporary file types too, and then implement jar downloading for http/https classpath entries.

I just noticed this old bug report, and I should circle back and say that http(s) classpath elements have been supported in ClassPath (formerly FastClasspathScanner) for over a year now.


Emails with params

Example:
 someone<EMAIL_ADDRESS>
Consider support those.
UPD. Need real world examples, why/how such links can happen in human readable texts!

Still no real world examples. Closing as timed out.


Set the correct content-type on time update for S3

This fixes an issue where updating the modification time resets the content-type to the S3 default of binary/octet-stream which breaks static websites that expect an html file to have a content-type of text/html.

Thanks for that - good catch.  Will merge shortly!

Merged. Thank you very much.

Awesome, thanks!


screen rotation

I have a bug on my LG device where the screen on my pc is rotated 90 degrees but still functions as if the touches were not. it is similar to this issue, https://github.com/Genymobile/scrcpy/issues/186
but the reference there doesn't help me.

how do i fix this bug and/or rotate the screen on my pc

Also similar to https://github.com/Genymobile/scrcpy/issues/481.

kinda
more like this, I suck at explaining things anyways.
https://youtu.be/kVBgShD9E_w

This also happens to the planet computers gemini.

On Planet Computers Gemini PDA, when I modify the DisplayManager wrapper to use appWidth/Height instead of logicalWidth/Height, the correct orientation is displayed for some apps (including the home screen), but still fails for full screen application (games). The logical dimensions are 2160x1080, while the reported appWidth/Height are 2032x1080. The cropped 128 pixels correspond to the android buttons which appear to the right.
Playing around with contentRect width, I found that the correction orientation is displayed when width is ≤ 2038 pixels. There's probably something on the android side that rotates the display when requesting content above certain dimensions.
Currently, as a workaround for viewing full screen games, I hardcoded the rotation set by setDisplayProjection to 3 (taking hint from Ihttps://github.com/Genymobile/scrcpy/pull/1151).

Seems there's rotational issue for LG devices, I tried this out on my LG V20 and I opened up an app that support rotating the screen to landscape mode, but when I press the key binds for ctrl+r it just squashes the screen, and trying to revert back stretches the screen.
It seems scrcpy is incapible of rotating the screen properly when it detects a rotation with or without the keybind. When you have a keybind for sending Rotation it doesn't let you switch between Portrait or Landscape, but instead keeps trying to select Landscape mode which it can't do. You should change it so that when you press ctrl+r it switches between Portrait or Landscape mode by conditional rotation manually, or at the very least properly detect the screen has been rotated and show it as it should be shown.

It seems scrcpy is incapible of rotating the screen properly when it detects a rotation with or without the keybind.

In fact, it does... when the device/ROM correctly notify the rotation event:
https://github.com/Genymobile/scrcpy/blob/master/server/src/main/java/com/genymobile/scrcpy/Device.java#L30

In fact, it does... when the device/ROM correctly notifies the rotation event:
https://github.com/Genymobile/scrcpy/blob/master/server/src/main/java/com/genymobile/scrcpy/Device.java#L30

Well if it does, then explain why it doesn't correctly adjust the screen properly when rotated? Clearly something wrong with the code if it can't switch between Portrait and Landscape modes interchangeably without stretching or squashing the screen proportions and render it correctly. Because the device is correctly notifies for the rotation event.

Clearly something wrong with the code if it can't switch between Portrait and Landscape modes interchangeably without stretching or squashing the screen proportions and render it correctly.

Of course something is wrong. To handle this properly, scrcpy restart the video encoding when the device notifies the rotation change (because the video dimension change).
It works pretty well. But some devices (apparently) do not call the onRotationChange callback, so in that case the video encoding is not restarted, and stretch the new screen surface to the old dimensions on the device side.

Because the device is correctly notifies for the rotation event.

Are you sure?
Could you add a log here:
https://github.com/Genymobile/scrcpy/blob/1a9429f3c73e319f108211a66ad57b8c2c9a60aa/server/src/main/java/com/genymobile/scrcpy/Device.java#L57-L64
and verify that it is printed when you rotate your device?


Design - Lesson structure

Development team requires a general lesson design and structure to guide their implementation, i.e. entities, API, etc.
E.g. Like the lessons in freecodecamp. The completed mark ( ✓ ) and the progress (10/21) showed in this screenshot:

@tristanMatthias let me know any thoughts on how to approach design.
I'm interested in lesson design, would love to get some of your knowledge/guidance on this issue, including design tools.

This is an entire discipline. Lets do that call soon to discuss.

@tristanMatthias  @shaneSheridan  let me know when you're going to do the call. I'm interesting too

@tristanMatthias Lesson design structure seems complete here:
https://dbdiagram.io/d/5eb1c89439d18f5553febe18
Can we close this issue?

No this is a design task, not a dev one. Still needs to be kept open. I’ll bring in the design team for this

@tristanMatthias I unassigned myself from this one, I'll leave it to the design team.

Done


Add indexer support

Currently stories.svelte files are incompatible with:

story index generation https://github.com/storybookjs/storybook/issues/18081
storybook test runner https://github.com/storybookjs/test-runner/issues/80

To address this, we've added an "indexer" API to Storybook core for 7.0: https://github.com/storybookjs/storybook/pull/18355
If addon-svelte-csf were to implement an indexer, it would solve the above problems, plus other future problems as we build more on top of the story index (which we will definitely do more of in core)

I have a working implementation in a private branch. I have to add tests and check how I can merge this without loosing compatibility with v6

@j3rem1e that's wonderful news! It might make sense to release that change in an alpha rather than a full release. The indexer API is 7.0-alpha and might change between now and the final release. Specifically, we might update it to make watch mode possible in the test runner. cc @yannbf

:rocket: Issue was released in v2.0.9 :rocket:


Calling EasyPermissions.requestPermissions in onPermissionsDenied doesn't show permission dialog.

Basic Information
Device type: Samsung J7 2016
OS version: 7.0
EasyPermissions version: 1.3.0
Calling EasyPermissions.requestPermissions in onPermissionsDenied doesn't show permission dialog.
Is this the intended behavior or am I missing something?
I expected to see another permission dialog.

UPDATE: I downgraded to version 1.1.1 and it worked as expected.

See https://github.com/googlesamples/easypermissions/issues/204#issuecomment-412204799, I can't reproduce it so getting a sample would be great.

Going to mark this a duplicate of #204 but @omar753sahl let's keep discussing over there if you are willing!


The browser was not opened when start the service

after did 'yarn start', the browser was not opened when start the service and it seems that doesn't work, what can be done to check.

Hi @halisihan, are you using branch master? That branch is using the OpenAI API (not the ChatGPT web scraper), so there will be no browser any more, but you can still use the app. Please check the note at https://github.com/trietphm/chatgpt-slackbot/blob/master/README.md#chatgpt-for-slack for the reasons.
And yes, I forgot to update the part on README that saying the browser will open. I will update it soon.


Not updating/deleting users correctly

The provider cannot update a users role and results in an error. Also on destroy the user isn't destroyed.
in the resourceDatabaseUserUpdate function there is this:
adminDB := client.Database("admin")
Users := adminDB.Collection("system.users")

these are hardcoded and may not apply to all mongodbs. this results in an error where the user isnt deleted and the new create user cant fire as the user already exists

Thank you for your feedback, i will look into the destroy function. any use case or specific mongodb version  that can help study the behaviour ?

Thank you for the contribution  @androidwiltron   version 0.0.5 is available :+1:
https://registry.terraform.io/providers/Kaginari/mongodb/latest/docs?pollNotifications=true


Add git config test for installation

Do we assume that all the users use bash? The newly added test may fail otherwise. :thinking:
I can add Travis-CI config to test the code and the unit tests against different setups.

Sorry @angellandros about the delayed response. Yes we can assume all users use Bash, for now anyway :) Awesome work!


Display price paid/stated information [do-not-merge-yet, waiting for feeder branches]

Fixes some CSS/HTML issues

I see you also changed the way we display proprietors (owners and lenders). Is that part of the story or just a general tidy up. Should we amend code that's not directly part of the story?
Code all good :)

@russwillis this is minor refactoring as we go along, to align ourselves to the prototype design :) the design looked particularly terrible with multiple lenders or proprietors
##Before
(Integration environment screenshot, with bogus test data)

##After
(Dev environment screenshot)

Looks good to me

:warning: to be merged with https://github.com/LandRegistry/digital-register-feeder/pull/135 :)


Snapshot

We'd definitely be interested in this. Would you be willing to make the following changes:

Modify takeSnaphsot to take a single options object argument with a quality key. This way we won't break clients if we add more options in the future.
If the preserveDrawingBuffer option isn't set for WebGlStage, throw Error('WebGlStage: takeSnapshot requires the preserveDrawingBuffer option to be set') instead of silently returning a blank image.
Throw an error if the quality is set to something other than a number between 0 and 100. Leaving it undefined to signal a default value is okay, but make the default 75 rather than 50. For Flash, perform the validation in the Javascript code.
Add a CssStage.prototype.takeSnaphsot method throwing an Error('CssStage: takeSnapshot not implemented').

On it.

@tjgq I'm almost done with the changes proposed, but are we sure about the preserveDrawingBuffer WebGL context parameter? Because it's known to degrade performance on some platforms and GPUs, and the "silent blank page" is already handled by calling this.render() inside takeSnapshot() method, right before extracting the image data from the canvas (or at least it should be, it forces the buffer to fill up again). Since the actual render (when the buffers are drawn and (hopefully) swapped) takes place a lot more often then someone potentially invoking the execution of takeSnapshot method, are we sure it is the right way to go?

I could maybe check if preserveDrawingBuffer option is set, and if it is, not invoke the render() method right before toDataURL(), to avoid unnecessary performance hit.

You're completely right, and I was wrong - preserveDrawingBuffer isn't necessary as long as you access the drawing buffer before control is passed to the graphics pipeline.
I think calling render() unconditionally before calling toDataURL is perfectly fine: taking a snapshot should be a rare event, so the performance impact is negligible. So you can essentially disregard my comment regarding preserveDrawingBuffer. Thanks!

No problems! Have you got a chance to look at the updated code? I think it successfully addresses the points discussed earlier.

This looks great! Thanks for addressing the comments.
Minor nit - can you fix the if/for statements to look like
if (foo) {
  bar;
}

instead of if (foo) bar;?

Will do.

Done.


Term request: megahertz

Dear UO,

I am aware of UO prefixes and that UO has Hertz unit, however I could see that
megavolt was available.
The use case here is to report NMR magnet frequency which are always expressed
in MHz. Having the term readily available would make using in an XML schema

If this is not possible, can you point to documentation on how to compose UO
units and prefixes?

Thx

Philippe

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 4 Dec 2013 at 4:06

Hi @gkoutos, we're  looking at megaHertz in UO and it's a subclass of hertz. Should they not be siblings, like centimeters and meters?


Update PPP

Update PPP with more details
Added into DG find activity diagram image
Added into DG find sequence diagram image

Codecov Report

Merging #206 (b6a17e3) into master (81c8994) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #206   +/-   ##
=========================================
  Coverage     73.13%   73.13%
  Complexity      593      593
=========================================
  Files            90       90
  Lines          1727     1727
  Branches        191      191
=========================================
  Hits           1263     1263
  Misses          405      405
  Partials         59       59

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


bookingBusinesses beta endpoint does not work anymore

Describe the bug
From this morning retrieving booking businesses through the beta api gives a generic error message.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://developer.microsoft.com/en-us/graph/graph-explorer'
Select 'beta' version
Run https://graph.microsoft.com/beta/bookingBusinesses query
See error

{
"error": {
"code": "UnknownError",
"message": "",
"innerError": {
"date": "2023-08-30T13:43:50",
"request-id": "2e344a3c-dc17-4b40-ad9a-34fce4e9f634",
"client-request-id": "2e344a3c-dc17-4b40-ad9a-34fce4e9f634"
}
}
}
Expected behavior
Based on https://learn.microsoft.com/en-us/graph/api/bookingbusiness-list?view=graph-rest-beta&tabs=http we expect to receive the list of calendars related to the sample account. Using the v1.0 endpoint https://graph.microsoft.com/v1.0/solutions/bookingBusinesses it works.
Screenshots

Desktop (please complete the following information):

OS: Windows
Browser: Firefox

Hi @SbGibson ,
Could you confirm whether you're still getting the 500 error response?
I'm currently unable to reproduce it on my end

Hi @SbGibson , Could you confirm whether you're still getting the 500 error response? I'm currently unable to reproduce it on my end

Hi Elinor, thanks for replying.
I can confirm that now that endpoint is working without any change by my side. The problem shifted to other endpoints so I thought it was not a problem of the explorer and filed an issue here https://learn.microsoft.com/en-us/answers/questions/1358994/unknownerror-500-on-different-bookings-beta-endpoi
Dunno if you can do something on that. I let you decide if closing or forwarding this issue to someone else. Many Thanks.

Hey @SbGibson,
I'm glad you were able to file the issue there. The appropriate team should be able to provide more help as we only work on the Graph Explorer application.
I will close this for now.


KeyError: 'unknown network: vgg16_convs'

Hi, when i run the ./experiment/script/demo.sh, i meet the error, "KeyError: 'unknown network: vgg16_convs'". i don't know why, can you help me, thanks!

@2017hack : How did you resolve the issue?

If you have this problem: check Issue #23. @saurabheights provided the solution there.
conda remove easydict
pip install easydict

Ohhh yeah, sorry had exams, so didnt get to provide solution. @im2ex  Thanks for doing my part. :+1:


This is now hosted at https://download.get.foundation/sites/download/

This is now hosted at https://download.get.foundation/sites/download/
Originally posted by @DanielRuf in https://github.com/foundation/foundation-sites/issues/11826#issuecomment-605529890
Is there an issue with the custom download on the foundation zurb site. I have been trying to download a custom version of the css for a month or so now but all I get is the following:
This account has been disabled. Please visit https://uberspace.de/support.
Dieser Account wurde deaktiviert. Bitte besuche https://uberspace.de/support.
Is this issue being worked on and will it be back up and running soon?
Many thanks

Please see https://github.com/foundation/foundation-sites/issues/12189 and the comments.

The issue is pinned, so you should be able to see it at the top of the issues.

Is this issue being worked on and will it be back up and running soon?

I don't know. Please ask @joeworkman. I am not actively involved anymore.


Requesting non-preloaded language from client side breaks TAPi18n

According to http://i18next.com/pages/doc_features.html, if you request a translation in a language which has not been preloaded, it should be fetched synchronously. But for some reason this fails and TAPi18n lo longer works after that.

Provide more details on what you are doing, preferably with actual project example.
Do you use the feature documented the section titled access value in different language in i18next docs?
Thanks,
-Daniel

Yes, that's what I'm trying to do, what's shown in the i18next docs.
If I try to TAPi18n.__('key', { lng: 'es' }) but 'es' is not loaded, TAPi18n stops working.
I discovered I had to preload the languages to make it work. Here what I ended up doing:
https://stackoverflow.com/questions/29153392/tapi18n-how-to-show-text-in-several-languages-side-by-side

Please test this pull request: https://github.com/TAPevents/tap-i18n/pull/55

Wow thanks a lot for working on it! It seems to work fine from inside a helper. I can see it works because it takes a fraction of a second for the translation to appear. With this I can already make the code much cleaner :)
For some reason the current language is always shown if I try to use the new feature from a template. {{_ key lang="es"}} shows the current language, not "es", and it's reactive. If I change the current language from the menu the text changes, instead of showing "es".

It could also be that I've done something wrong. I pulled the branch into a packages folder inside my project, and I noticed it was using the new version because TAPi18n got new methods (I checked in the browser console). Is there anything else I should do to replace the original version with the new one?

 The demand for this feature is very high and I felt like it's really missing in the API.
If you appreciate my work, know that you and your fellow programmers can use my services throw my Meteor Development consultancy. And please star tap:i18n :)
Regarding your issue, you should use lng and not lang, that's quite confusing, so I am going to make lang an alias option to lng.
-Daniel

Done here please test again and let me know if it works well.
Thanks,
-Daniel

Works great. Beautiful :) I'll mention your services to my employer :+1:

I'm having a strange issue with the translations, and I wonder if it's related to the changes in the new branch. Sometimes when I click a link the page switches to Spanish.
When that happens, in the console I type TAPi18n.getLanguage() and it prints "en". But TAPi18n.('homepage') prints "Página de inicio". If I run TAPi18n.('homepage', { lng: 'xx'}) where xx is 'de', 'en', 'es', it always responds with the Spanish version.
If I reload the page or choose English in the menu (which is already selected), it's back to normal. The fallback language is 'en'. loaded_languages is ['en', 'es', 'de'].
What could I do to debug this?

I suspect it's related to TAPi18n, because I'm also using T9n (which is required by useraccounts:bootstrap) and the log out button (translated by T9n) is the only thing correctly shown in English.

I'm having a rare random behaviour I can't reproduce. Sometimes the page is displayed in one language, but TAPi18n.getLanguage() in the console prints a different language. Also this different language is shown as the active one in the language menu. When this happens I can fix it by reloading the page, or by selecting the language that should be visible already from the language menu. It's hard to debug because it's rare. When I look for it, I click on 100 internal links and it does not manifest itself. When I stop looking it may happen twice in one minute.
How is it possible that TAPi18n thinks a certain language is the current one, but displays the texts in a different one? Is it maybe in an odd state, in which two internal variables are out of sync?
I was setting the initial language on rendered(), I just placed it on Meteor.startup() to see if it makes a difference.

Hi @hamoid -- are you using just tap:i18n or tap:i18n-db too?

Hi @hitchcott -- only tap:i18n, specifc-language-translation-client branch (I need to show multiple languages on the client side simultaneously). I also use<EMAIL_ADDRESS>(required by a package, no idea if that makes a difference)

I think this shows that there must be something strange going on. In the console:
TAPi18n.getLanguage()
  "es"
TAPi18n.__('homepage')
  "Startseite"
TAPi18n.setLanguage('es')
  Object {state: function, always: function, then: function, promise: function, pipe: function…}
TAPi18n.__('homepage')
  "Página de inicio"
TAPi18n.getLanguage()
  "es"

It's returning first German and then Spanish, even if in both cases it thinks the current language is Spanish.


tlv493d: Auto-reset TLV493D on error

When an error occurs, it is necessary to reset the sensor as per the documentation. This will send a recovery command followed by the standard start procedure.

Codecov Report

Merging #52 (283ca54) into main (600c8cc) will increase coverage by 0.0%.
The diff coverage is 20.0%.

@@          Coverage Diff           @@
##            main     #52    +/-   ##
======================================
  Coverage   64.4%   64.4%
======================================
  Files         56      60     +4
  Lines       6824    7066   +242
======================================
+ Hits        4397    4554   +157
- Misses      2283    2356    +73
- Partials     144     156    +12

Impacted Files
Coverage Δ

tlv493d/tlv493d.go
59.0% <20.0%> (-1.4%)
:arrow_down:

hx711/hx711.go
69.4% <0.0%> (-5.6%)
:arrow_down:

mcp9808/mcp9808.go
98.8% <0.0%> (-1.2%)
:arrow_down:

tca95xx/tca95xx.go
78.2% <0.0%> (ø)

tca95xx/variants.go
63.6% <0.0%> (ø)

tca95xx/pins.go
60.4% <0.0%> (ø)

tca95xx/registers.go
87.2% <0.0%> (ø)

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

Thanks!


Minor documentation typo

Minor issue with the readme.md. It currently gives the bounding box as

Latitude:
East: 1.7800
West: -7.5600
Longitude:
North: 60.8400
South: 49.9600

But "Latitude" and "Longitude" are the wrong way around.

Additionally, under Available Conversions (AKA I Want To…) the readme lists convert_etrs89 which does not appear to exist. It may have been renamed?

Hm yes that's…not good. The swapped coordinates have been fixed in the README, and convert_to_etrs89 is now available from util and cutil in v0.6.10, on PyPI.


Terminate all active sessions if not waiting for them

tests.helpers server_is_up/server_is_down now returns True/False instead off raising exceptions
add two tests for sessions active management during server shutdown

Codecov Report

Merging #271 into master will decrease coverage by 0.06%.
The diff coverage is 5.55%.

@@            Coverage Diff             @@
##           master     #271      +/-   ##
==========================================
- Coverage   57.94%   57.88%   -0.07%
==========================================
  Files          40       40
  Lines        3346     3355       +9
  Branches      395      397       +2
==========================================
+ Hits         1939     1942       +3
- Misses       1296     1302       +6
  Partials      111      111

Impacted Files
Coverage Δ

vmmaster/server.py
63.88% <5.55%> (-7.54%)
:arrow_down:

core/utils/network_utils.py
63.71% <0%> (+1.76%)
:arrow_up:

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 4e53c6b...afbb190. Read the comment docs.

!test

!test


Add Introductory discussion of Repository Pattern

@mehdihadeli Updated!

@mehdihadeli Updated!
@Progyan1997
Thanks for the link, But it doesn't have high quality for this section.


Use new Pester syntax: -Parameter for Pester tests in engine.

PR Summary
Use new Pester syntax: -Parameter for Pester tests in engine.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[x] Not Applicable
OR
[ ] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[ ] Not Applicable or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

[x] Add [feature] if the change is significant or affects feature tests

@SteveL-MSFT Can you have another look?

@adityapatwardhan ping


rose-pine-gtk-theme: init at unstable-2021-02-22

Motivation for this change
Add Rosé Pine for GTK
Fixes https://github.com/NixOS/nixpkgs/issues/145776
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 146333 run on x86_64-linux 1

1 package built:

rose-pine-gtk-theme


Start mentat server as CLI starts

When the CLI starts, a mentat server should be started and accessible from the CLI.

IMO we shouldn't be doing this over a connection: the CLI should be opening and closing DBs directly, performing operations through the Rust API.

When I wrote this I was looking at src/main.rs and thinking this is the way to do it, but now I'm looking at it I realise that is the wrong way to go. Closing this ticket as https://github.com/mozilla/mentat/issues/452 should cover it all.


Update provider name

By overriding the google driver (without fully implementing the default auth patterns/requirements), folks can't use this library on the same app that also uses the normal auth flow. Unless you only care about Google auth for Chrome users, that's a bad thing. I don't understand how to fully implement the patterns/requirements for normal OAuth using Google's client class; so the next best thing is to leave the existing Google Socialite driver alone and require folks to use either driver('google') or driver('google-one-tap') depending on the flow they need.

@serdud Thanks for your work!


Support question - why do promises magically run synchronously

Forgive me, I am a bit of newb with Karma + Jasmine
I am sure there is an simple answer to this.
I have the following Jasmine code with Karma:

    it('should update a comment by 59dd58092889e25745b6b208', function () {

      //expectGET to make sure this is called once.
      var url = encodeURI('/api/v1/comments/by_id/59dd58092889e25745b6b208');

      httpBackend.expectPUT(url, options1.data).respond(comments1);

      console.log('uno');

      myService.updateComment(options1.data).then(function (response) {
        expect(response.value).to.equal(comments1[0].value);
        console.log('dos');
      });

      //flush the backend to "execute" the request to do the expectedGET assertion.
      httpBackend.flush();

      console.log('tres');

    });

Expected behaviour
I would expect it to log:
LOG LOG: 'uno'
LOG LOG: 'tres'
LOG LOG: 'dos'

Actual behaviour
LOG LOG: 'uno'
LOG LOG: 'dos'
LOG LOG: 'tres'

Environment Details

Karma version (output of karma --version):
Relevant part of your karma.config.js file

Steps to reproduce the behaviour
See screenshot:

yeah, I think the Karma logger will on rare occasion log things out of order, that's not good

Maybe improve the logging order, idk, will close

Async code in tests requires different test API.
Try
it('should update a comment by 59dd58092889e25745b6b208', function () {

      //expectGET to make sure this is called once.
      var url = encodeURI('/api/v1/comments/by_id/59dd58092889e25745b6b208');

      httpBackend.expectPUT(url, options1.data).respond(comments1);

      console.log('uno');

      var promise = myService.updateComment(options1.data).then(function (response) {
        expect(response.value).to.equal(comments1[0].value);
        console.log('dos');
      });

      //flush the backend to "execute" the request to do the expectedGET assertion.
      httpBackend.flush();

      console.log('tres');
      return promise;

or
    it('should update a comment by 59dd58092889e25745b6b208', function (done) {

      //expectGET to make sure this is called once.
      var url = encodeURI('/api/v1/comments/by_id/59dd58092889e25745b6b208');

      httpBackend.expectPUT(url, options1.data).respond(comments1);

      console.log('uno');

      myService.updateComment(options1.data).then(function (response) {
        expect(response.value).to.equal(comments1[0].value);
        console.log('dos');
        done();
      });

      //flush the backend to "execute" the request to do the expectedGET assertion.
      httpBackend.flush();

      console.log('tres');


Back to development

Coverage remained the same at 94.32% when pulling 93bcad73229ead33a2f8df937583c4dacabbad9b on btod into 98ece72902a2a5ec983f3f60d3608cb390892706 on master.


Hydration node mismatch when using overlay: true with datepicker or mask

Reproduction
https://stackblitz.com/edit/github-s54hfb?file=app.vue
Describe the bug
Using nuxt ssr. Hydration node mismatch with the following schema (note the overlay: true):
const schema = [
  {
    $formkit: 'mask',
    label: 'Mask with overlay',
    mask: 'H:M',
    overlay: true,
    mode: 'select',
    name: 'time',
  },
];

Environment
• OS: macOS
• Browser: chrome
• Version: latest

Any updated status on this one?

Workaround for now is wrapping with <ClientOnly>, but this is should be fixed soonish.

Cool deal. That’s what I’ve been doing. Appreciate the update!

The datepicker hydration mismatch was resolved here: https://github.com/formkit/formkit/issues/1152 so I updated the title. Mask still has this issue.


Update Gen5TimerModel.cpp

This corrects the default Gen 5 calibration from 95 to -95 as it was in 2.0.1

Fixes #67


Upload to another domain

How to edit the domain to upload it on my hastebin server instead of hastebin public server ?
Thanks ;)

Please see this section of the README: https://github.com/seejohnrun/haste-client#changing-the-location-of-your-haste-server
Thanks!


Avoid race condition with peer in server path secret map read

Security issue notifications
If you discover a potential security issue in s2n-quic we ask that you notify
AWS Security via our vulnerability reporting page. Please do not create a public github issue.
Problem:
Currently a sequence like:

Client starts + finishes handshake for secret A
Client starts handshake for secret B
Client sends a datagram packet, encrypted with A
Server receives on_peer_stateless_reset_tokens, inserts B into the path secret map
Server looks up IP, gets B, encrypts + replies
Client looks up path secret B and fails since handshake for B is not yet complete

Solution:
We need to delay the insertion into the peer set (i.e. by-ip index) on the server until the handshake is fully complete (and so the client is able to decrypt that IP). That probably involves changes to the dc::Endpoint trait to communicate that new state.
Requirements / Acceptance Criteria:
n/a
Out of scope:

Removing an entry if the client fails to confirm the handshake.
Race conditions that still cause the same sequence to occur, with the client confirming the handshake after completing a separate handshake. This is probably most likely with a delayed packet and a restarting client.

If possible we should cover this too, but it's probably hard or impossible.

Thanks for the issue, we'll have a look.

If the server delays insertion, couldn't the same thing happen in the opposite direction? As in, the client sends a datagram packet encrypted with B that makes it to the server before the ACK that fully completes the handshake on the server side?

The server would need to insert into the id set (so that it can decrypt packets) as soon as it knows it trusts the client -- I'd guess that's the same time as today. But it should only insert into the peer set (the one used to lookup a path secret by IP) once the client has ack'd the handshake.
I think that closes the gap you're calling out.


fix(tauri.js) determine app path correctly

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] New Binding Issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes. Issue #___
[x] No

The PR fulfills these requirements:

[x] It's submitted to the dev branch and not the master branch
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

I am not sure why this is failing - I ran the jest --runInBand --no-cache locally and it passes just fine on MacOS.

Commits are unverified here because I didn't setup my GPG keys on windows, sorry guys.


Feature request: Table view column sorting

When I open table view (using Oracle, but I believe this is universal also for other databases) on Properties tab I can see columns. These are sorted alphabetically by column name. I would like to have this sorted by column number as that is the same order my result sets come in. The current way is confusing.
I have not found a configuration option to set this. I would appreciate if this option was added.
Thanks.

By default columns are sorted by number.
Perhaps you set option "Order elements alphabetically" in preferences (Preferences->Database->Navigator)?

Yes, that has done it. Thank you. It is a bit confusing in my opinion, because Navigator suggests to me "Database Navigator" that is on the left side of screen.


docs: fix typos

Hello,
This tiny PR will fix :

2 typos in Sources/Moya/Plugins/NetworkLoggerPlugin.swift
1 typo in docs/MigrationGuides/migration_13_to_14.md

Ref to https://github.com/Moya/moya.github.io/pull/2 😄
Best! :robot:

btw, the docs on our website are usually updated on each release using our release script here, however I will probably update the docs manually in a bit 👍


Version <IP_ADDRESS> is broken

Hi,
I post this issue to notify you that version <IP_ADDRESS> is actually broken.
ERROR: This plugin was built with an older plugin structure. Contact the plugin author to remove the intermediate \"elasticsearch\" directory within the plugin zip
I am currently investigating on it and I expect finding a resolution soon.

Release <IP_ADDRESS> should fix this.


Feature/increase budget

Changes proposed in this Pull Request:

Update angular budget so our auth templates will build without err.

To Test:

yarn build:local and make a new ng project w/ auth template, then run cd [project-name] && yarn build

Getting below error:

Steps I followed:

Run yarn build:local from root
Select angular framework with auth template
And cd [project-name] && yarn build

Am I missing anything?

Steps I followed:

Run yarn build:local from root
Select angular framework with auth template
And cd [project-name] && yarn build

Am I missing anything?

No these steps look good to me; this is weird.  Could you run these steps again from the dev branch to see if the issue persists?  Running these steps locally works for me; just make sure to run yarn when you initially clone or checkout the new branch.

seem ok for me

ust make sure to run yarn when

Still getting error with fresh clone and run yarn command. But if it is working for Jeff then it must be issue at my end.

ust make sure to run yarn when

Still getting error with fresh clone and run yarn command. But if it is working for Jeff then it must be an issue at my end.

I did this the long way. yarn and yarn link. open terminal and run blui new angular select auth template and let it build

I'm getting the same errors as Ekta...


Using substance archive texture make the export fail

In method:
private ImageId ExportImage(Texture texture, TextureMapType texturMapType)
texture = texture as Texture2D, returns null when using sbsar file.

Could you share an example sbsar file?

Sorry, I use file found at https://share.allegorithmic.com (material section). I'm not sure I can publish it here but you can download one by creating a free account.

I'll try to fix it and provide a pull request when I'll find some time to do it if you want but it will not be this week I guess.

@ultrasuperpingu  any update on a PR to fix this issue?

Ok, I just took a look at this. The problem is more general. sbsar material are ProceduralTexture in Unity. The case of sbsar material is probably not a big problem since its support was removed in Unity 2018 but the problem still remains for other ProceduralTexture. I guess the quick fix would be to ignore them and log an error. That means in method :
private MaterialId ExportMaterial(Material materialObj)`
for each texture (emissive, normal, basecolor, etc), if texture is a ProceduralTexture, don't export it and log an error.
A more general fix would be to extract a regular texture from the procedural to export it but it's a bit complicated and I'm not sure it make sense in most cases.
Do you want me to implement the ignore and log behaviour and to submit a PR with it?

I made PR #336 for this. Hope this helps :)

Deprecated


[release-2.6] Pin the oc, kind, and OLM versions

This cherry-picks from #427, #433, and #436.

/retest

/retest

/retest

/retest


Cleanup intermediate files

Cleans up intermediate files generated during subsetting (downloaded h5 files as well as corresponding subset files) to minimize disk space, and leaving only the final, combined subset file as output.
Further, applied isort and black to tidy up imports and formatting, along with a couple of type hint fixes and adjusting the algorithm configuration to use the 8gb queue and dial down the disk space.

Sorry, yes, my bad for not adding some comments there. I'll add comments to the osx.py and maapx.py files, but basically the x suffix is for "eXtension", indicating that the contents are extensions/enhancements to existing modules. In this case, the extensions are for adding some FP "safety".

@wildintellect, I added some docstrings to osx.py and maapx.py


testutil: more race/deadlock hunting

I think this is the last of my cleanups.  I've learned a few things about ssntp, and with these tweeks here I've yet to see a failure.

Coverage decreased (-0.05%) to 65.58% when pulling 97711a54974fdae68e23249100a569d01519e19e on tpepper:master into b3a57f6e6dbf0146ad2b9857b071ecf308d6fd5e on 01org:master.


[Bug]: Export an obkv Redis table structure, run it in SQL window, reconstruct the table statement, and report an error

ODC version
ODC 4.3.2
OB version
OBKV REDIS
What happened?
Export an obkv Redis table structure, run it in SQL window, reconstruct the table statement, and report an error

CREATE TABLE obkv_redis_hash_table (
db bigint(20) NOT NULL,
rkey varbinary(16384) NOT NULL,
expire_ts timestamp(6) NULL DEFAULT NULL,
insert_ts timestamp(6) NULL DEFAULT CURRENT_TIMESTAMP(6),
value varbinary(1048576) DEFAULT NULL,
vk varbinary(16384) GENERATED ALWAYS AS (substr(rkey,9,conv(substr(rkey,1,8),16,10))) VIRTUAL,
PRIMARY KEY (db, rkey)
) DEFAULT CHARSET = utf8mb4 ROW_FORMAT = DYNAMIC COMPRESSION = 'zstd_1.3.8' REPLICA_NUM = 3 BLOCK_SIZE = 16384 USE_BLOOM_FILTER = FALSE TABLET_SIZE = 134217728 PCTFREE = 0 KV_ATTRIBUTES = ({"Redis": {"isTTL": true, "model": "hash"}})
partition by key(db, vk)
(partition p0,
partition p1,
partition p2,
partition p3,
partition p4,
partition p5,
partition p6,
partition p7,
partition p8,
partition p9,
partition p10,
partition p11,
partition p12,
partition p13,
partition p14,
partition p15,
partition p16,
partition p17,
partition p18,
partition p19,
partition p20,
partition p21,
partition p22,
partition p23,
partition p24,
partition p25,
partition p26,
partition p27,
partition p28,
partition p29,
partition p30,
partition p31,
partition p32,
partition p33,
partition p34,
partition p35,
partition p36,
partition p37,
partition p38,
partition p39,
partition p40,
partition p41,
partition p42,
partition p43,
partition p44,
partition p45,
partition p46,
partition p47,
partition p48,
partition p49,
partition p50,
partition p51,
partition p52,
partition p53,
partition p54,
partition p55,
partition p56,
partition p57,
partition p58,
partition p59,
partition p60,
partition p61,
partition p62,
partition p63,
partition p64,
partition p65,
partition p66,
partition p67,
partition p68,
partition p69,
partition p70,
partition p71,
partition p72,
partition p73,
partition p74,
partition p75,
partition p76,
partition p77,
partition p78,
partition p79,
partition p80,
partition p81,
partition p82,
partition p83,
partition p84,
partition p85,
partition p86,
partition p87,
partition p88,
partition p89,
partition p90,
partition p91,
partition p92,
partition p93,
partition p94,
partition p95,
partition p96);
What did you expect to happen?
Tables can be created as normal
How can we reproduce it (as minimally and precisely as possible)?
Select an obkv redis table, export the table structure, copy the SQL statement, execute it in the SQL window, and get the result of ErrorCode = 0, SQLState = null, Details = unknown escape sequence {"Redis": {"isTTL": true, "model": "hash"}}
Anything else we need to know?
No response

this seems to be a problem with 'show create table' statement, what does  'show create table obkv_redis_hash_table' result looks like ?

Looks like not ODC's issue. What's the executing result on obclient?


Point Lights don't cast shadows

Currently only directional light and spot lights cast shadows. Point lights don't cast shadows.

Point lights do not have a projection matrix if I am right, the shadow map needs something different that of a directional light or spot light.

In this case you have to use cascaded shadow maps. These are cube maps from the viewpoint of the light (so they cover all directions). But you need more than one because the light is infinitely far. Typically games with a lot of scenery would use 4 levels of shadow map each with a different projection matrix focusing on a different distance.


update: bump Falco to 0.32.2

What type of PR is this?
/kind feature
/kind chart-release
Any specific area of the project related to this PR?
/area falco-chart
What this PR does / why we need it:
This PR bumps the Falco version to 0.32.2 and the chart version to 2.0.13
Which issue(s) this PR fixes:
Special notes for your reviewer:
Checklist

[x] Chart Version bumped
[x] CHANGELOG.md updated

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Andreagit97
Once this PR has been reviewed and has the lgtm label, please assign issif for approval by writing /assign @issif in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 6a023d26b94b7f6b82035ded71f2a238887c4e8a

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Deterministic stdout output. Resolves #8.

Use sorted() for sets in __str__ functions
Make BasicBlocks sortable by implementing __lt__

Resolves #8

Upon further testing I just realised there's some issues with function extraction. I'll need to resolve those before this is merged.

@zyzek OK, I think this is ready for review now :+1:


Conditionally include Python in the C++ builds.

@fmassa Thanks for your suggestions and links in https://github.com/pytorch/vision/issues/2692#issuecoment-1009783544
This PR adds an option USE_PYTHON that allows users to decide to link to Python3 or not.
We made it OFF by default as suggest in https://github.com/pytorch/vision/issues/2692#issuecoment-1009783544.
We might still need to add the definitions for USE_PYTHON so it gets linked in the Windows build mentioned in https://github.com/pytorch/vision/issues/3965#issue-911500276 (that should be done now)
This should fix #2692
Let me know if this makes sense. Thanks again!

Thanks @datumbox ! If something fails I think we can still decide to make USE_PYTHON=on by default so as to keep the current behavior while allowing one to disable Python.

Hey @prabhat00155!
You merged this PR, but no labels were added. The list of valid labels is available at https://github.com/pytorch/vision/blob/main/.github/process_commit.py

@NicolasHug Should we add "build" to SECONDARY LABELS?

sure, thanks for flagging


sd: modify sdmmc_wait_ready to always decrement timeout

As described in issue #65027, sdmmc_wait_ready can enter an infinite loop if the card is removed while waiting for it to report an idle status. Fix this by always decrementing the timeout in sdmmc_wait_ready, regardless of whether the SD card is busy.
Fixes #65027

"As described in issue https://github.com/zephyrproject-rtos/zephyr/issues/65027, sdmmc_wait_ready can enter an"
Would be nice to have issue URL instead of Github reference.

The github UI refuses to display it, but I've updated the commit to include the URL.


Requirements on vehicles

vehicle types:

car
motorbike
specialVehicle
trailer

A scpecial Vehicle is eg. a trike, a quad or a motorbike with a sidecar.
Attributes necessary for an offer (mandatory)

height
length
width
roof width

Mandatory attributes for the booking:

height
length
width
roof width
licensePlateNumber
customerName

Optional attributes for the booking:

weight
brand name: eg. "Audi" or type of special vehicle ("quad", "trike", "motorbike with sidecar")
textileRoof
rack:

roof or rear
type (in case of a roof rack: roof railing, carrier for luggage
heightIncrease
number of bikes (in case of a rear rack)

special vehicle --> tiny vehicle?


Add a delay between try upgrade and upgrade execution

Context
See finding 3 of Informal Systems findings.
Problem
A greedy cranker can submit a MsgTryUpgrade as soon as 5/6 has signaled for a version.
Proposal
To give some buffer time for the straggling 1/6 validators to upgrade their binaries, we can implement a delay between the block that a MsgTryUpgrade is included in a block and the actual upgrade height. For example: upgrade height = MsgTryUpgradeHeight + one week of blocks. A few questions about this delay:

Should validators be able to change their signaled vote after a MsgTryUpgrade was invoked and before the delayed upgrade height.
Should we disable all signaling after a successful crank invocation?
Can users submit MsgTryUpgrade after a successful previous invocation?
Should upgrade height coordination be performed out of protocol instead of dependent on the block height at which the first MsgTryUpgrade was included?

Acceptance Criteria

Analyze the finding
Prepare a design document on the possible options
Schedule a meeting to drive internal consensus on the option that should be pursued

Closed b/c we just had a meeting about this and agreed to add a time delay. Will create a new issue for that.


UnboundLocalError: local variable 'ner' referenced before assignment

@asimokby When I run these lines below, I get the error "UnboundLocalError: local variable 'ner' referenced before assignment".
`from parcv import parcv
parser = parcv.Parser(pickle=True, load_pickled=True)
json_output = parser.parse('DevOps_Engineer.pdf')
print(json_output)`

I think the problem is in this return statement. This line should be moved to 69 under the try statement.


Getting an error - android.webkit.SafeBrowsingResponse

Device Android 8.0.0
Rejecting re-init on previously-failed class java.lang.Class: java.lang.NoClassDefFoundError: Failed resolution of: Landroid/webkit/SafeBrowsingResponse;
07-08 21:15:47.676 25535-25535/com.test.sampleapp I/zygote64: Caused by: java.lang.ClassNotFoundException: Didn't find class "android.webkit.SafeBrowsingResponse" on path: DexPathList[[zip file "/data/app/com.android.chrome-Vm8hy8_u1-Dy5WC4ctZXkQ==/base.apk"],nativeLibraryDirectories=[/data/app/com.android.chrome-Vm8hy8_u1-Dy5WC4ctZXkQ==/lib/arm64, /data/app/com.android.chrome-Vm8hy8_u1-Dy5WC4ctZXkQ==/base.apk!/lib/arm64-v8a, /system/lib64, /system/vendor/lib64]]

Hi, I'm sorry its hard to say anything with so little information.

Hi,
I Really Appreciate the immediate response. Very less people are so fast. I
implemented your library the way you've shown it. It seems that on Android
Oreo you need to use view.getSettings().setSafeBrowsingEnabled(false);
function in iFrames.
Source :
https://stackoverflow.com/questions/49633271/what-is-android-webkit-safebrowsingresponse-error
Can you please check this one and let me know if there is a fix. Also, You
have not asked for the google developer key which is generated from the
developer console. I hope we dont need that.
let me know if you need any specific logs.
Warm Regards,
Vineet Kamath.
On Sun, Jul 8, 2018 at 9:21 PM Pierfrancesco Soffritti <
<EMAIL_ADDRESS>wrote:

Hi, I'm sorry its hard to say anything with so little information.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PierfrancescoSoffritti/android-youtube-player/issues/159#issuecomment-403296891,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB03vme1gEkaV8QpGWohlktDbyT6rxFKks5uEip5gaJpZM4VGxZf
.

It seems that on Android Oreo you need to use view.getSettings().setSafeBrowsingEnabled(false);
I never had an issue with that, I'll check in a few minutes.

You have not asked for the google developer key which is generated from the developer console. I hope we dont need that.
This library doesn't need any key.

Just tested on Oreo, works fine for me. Which phone are you using?

I tried on an lg v30+ and a redmi note 4. The player just keeps loading.
Its 1am at the moment here. Ill make a sample app tomorrow, try it on a few
more devices and if the issue reproduces ill send you that project.
On Mon 9 Jul, 2018, 01:03 Pierfrancesco Soffritti<EMAIL_ADDRESS>wrote:

Just tested on Oreo, works fine for me. Which phone are you using?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PierfrancescoSoffritti/android-youtube-player/issues/159#issuecomment-403311099,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB03vqvo4gvK6RMlRz3LzLpOd-iGUBjOks5uEl6kgaJpZM4VGxZf
.

Perfect, thanks!

Did you manage to solve the issue?

Hi Pierfrancesco,
I just solved the issue yesterday. It was giving me an error when I was
calling it in the background Async task in the onPostExecute function while
making a volley request.
So i created a small constructor and then passed the parameters and it
worked out.
Warm Regards,
Vineet Kamath.
On Mon, Jul 16, 2018 at 1:06 AM Pierfrancesco Soffritti <
<EMAIL_ADDRESS>wrote:

Did you manage to solve the issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PierfrancescoSoffritti/android-youtube-player/issues/159#issuecomment-405112942,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB03vn7-hy0Hlk5IGWQm_jaHTH0PcTPJks5uG5nUgaJpZM4VGxZf
.

nice!
feel free to leave a star on GitHub if the library is useful to you :)


Enable codecov support

Create codecov.yml, with the carryforward flag for partial reports. Already uploaded the initial full report from current main
Add badge to readme
Enable only on pull requests

Will this fail ever for coverage that is too low? And are you planning to set it up w/ a bot comment/link for the report? Just the report might be fine so we can see changes and determine if we should be adding tests

Yeah I forgot to remove the coverageThresholds, they're removed now.
I've requested to install the GitHub app which will supposedly comment as codecov, need authorization from Devin first to enable it. Then it should comment automatically on all PRs, with links back to things.

Here's an example of a report on my repo: https://github.com/mofojed/web-client-ui/pull/1
Once authorized here it should post on here as codecov-commenter. I think the carry forward flag will work correctly once the codecov.yml is merged into main. We'll need to make sure that works.


bug(cast): cast rpc eth_gasPrice --raw returns code: -32600, message: Invalid request

Component
Cast
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
cast 0.2.0 (de33b6a 2024-03-02T00:15:56.318147000Z)
What command(s) is the bug in?
cast rpc eth_gasPrice
Operating System
macOS (Apple Silicon)
Describe the bug
using cast rpc eth_gasPrice returns a correct value:
$ cast rpc eth_gasPrice
"0x53ca419d"

However defining the --raw RPC parameters gives us an invalid request.

[!NOTE]
Could this be because eth_gasPrice takes no parameters in the JSON-RPC Request?

$ cast rpc eth_gasPrice --raw {"method":"eth_gasPrice","params":[]}
Error:
(code: -32600, message: Invalid request, data: None)

[!WARNING]
Even the full JSON-RPC array of params fails, {"method":"eth_gasPrice","params":[],"id":1,"jsonrpc":"2.0"}

$ cast rpc eth_gasPrice --raw {"method": "eth_gasPrice", "params": []}
2024-03-02 16:24:20.514 cast[10603:202241] CFPropertyListCreateFromXMLData(): Old-style plist parser: missing semicolon or value in dictionary on line 1. Parsing will be abandoned. Break on _CFPropertyListMissingSemicolonOrValue to debug.
2024-03-02 16:24:20.514 cast[10603:202241] CFPropertyListCreateFromXMLData(): Old-style plist parser: missing semicolon or value in dictionary on line 1. Parsing will be abandoned. Break on _CFPropertyListMissingSemicolonOrValue to debug.
Error:
(code: -32600, message: Invalid request, data: None)

What I expect:

[!TIP]
Running Anvil $ETH_RPC_URL=http://<IP_ADDRESS>:8545

$ curl $ETH_RPC_URL   -X POST \
  -H "Content-Type: application/json" \
  --data '{"method":"eth_gasPrice","params":[],"id":1,"jsonrpc":"2.0"}'
{"jsonrpc":"2.0","id":1,"result":"0x53ca419d"

This isn't how it should use cast rpc. It should be:
cast rpc eth_gasPrice --raw '[]' --rpc-url=https://rpc.ankr.com/eth

works properly if passed as in comment above, https://github.com/foundry-rs/foundry/issues/7297#issuecomment-1976393546 , please reopen if you still have issues with the raw flag. thank you!


Fix links.

Many links are obsolete, some are even broken. This commit fixes some of them in this file .

@KexyBiscuit,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Wow - thanks for catching all of these! Will get the changes merged ASAP. Thanks, Liza


Allow setting feature when using trunk serve

Assuming I use trunk serve for local development (same for trunk watch), I would like to enable an additional feature for the Rust build automatically (--feature debug).
It would be great if I could configure this, e.g.:
[watch] # works also for "serve"
extra_features = [ "debug" ]

I agree

I'd also like to see --no-default-features supported in the save vein


JBPM-4579 fix + some other Wiser configuration cleanup in other tests.

See JIRA: https://issues.jboss.org/browse/JBPM-4579.

contains some small changes in Wiser configuration in other tests. The properties that were there used are not needed.

why would you like to remove the use of properties to setup mail server?
There are cases where this is rather important e.g. continuous integration environment where multiple jobs can run in parallel and use of con files can be pregenerated to avoid port conflicts.

I didn't thought about that that way. Maybe this should be more obvious from the code. I.e. I didn't find any file named email.conf in the project. There should be some reference file even if the file is generated at runtime i.e. by CI. The other cases ditto. The properties are instantiated in setup() method and only host and port are used from the properties. What do you think? If it is as you say I will revert the changes and leave only the fix for the test.

I'd prefer to leave it based on properties just to be on safe side for backward compatibility as I believe it won't hurt. When it comes to the file itself, it should be named drools.email.conf as it goes via ChainedProperties class.
So my vote is to include the fixes but still use properties.
Thanks

Ok I made changes as we agreed on IRC.

squashed commits and merged into master, thanks


Add support for etcd mvcc metric introduced in v3.5

What type of PR is this?

/kind bug
What this PR does / why we need it:
This change fixes etcd metrics collection with v3.5, where etcd_debugging_mvcc_db_total_size_in_bytes is replaced by
etcd_mvcc_db_total_size_in_bytes. See: https://etcd.io/docs/v3.4/upgrades/upgrade_3_5/#deprecated-etcd_debugging_mvcc_db_total_size_in_bytes-prometheus-metrics
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

/lgtm
/approve
/ok-to-test


Add explicit JDBI3 dependencies to the BOM

jdbi3-core
jdbi3-guava
jdbi3-jodatime2

Fixes #62

@chrisrohr Note I tested this locally against kiwi and dropwizard-application-errors, which are the kiwi projects that use JDBI directly. Note also that I added jdbi3-guava and jdbi3-jodatime2 because I noticed that dropwizard-jdbi3 directly depends on those as well, so adding them makes all the JDBI versions consistent.


check doesn't work with endpoint action invocation

int result = check testDB -> update("INSERT INTO STOCK(SYMBOL,PRICE) VALUES (?,?)", params);
doesn't work. It is the same for all endpoint action invocations.
P.S.
int result =? testDB -> update("INSERT INTO STOCK(SYMBOL,PRICE) VALUES (?,?)", params);
used to work

This was a grammar miss ... fix coming.

Checked in 0.970.0 beta 14 and this is fixed now.


Generate bindings for required symbols only

This avoids pulling in all of windows-sys and windows (even if most parts are excluded via Cargo features). The windows crate contains 9+ MB of code, so IMO avoiding that makes sense for a low-level crate like this.

Codecov Report
Attention: Patch coverage is 18.51852% with 88 lines in your changes missing coverage. Please review.

Project coverage is 66.88%. Comparing base (fbf8d71) to head (2584091).
Report is 3 commits behind head on main.

Files with missing lines
Patch %
Lines

src/win_bindings.rs
16.98%
88 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #14       +/-   ##
===========================================
- Coverage   94.55%   66.88%   -27.68%
===========================================
  Files           1        2        +1
  Lines         202      308      +106
===========================================
+ Hits          191      206       +15
- Misses         11      102       +91

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

Should we exclude the bindings file from coverage checking?

Should we exclude the bindings file from coverage checking?

Yes, that probably makes sense. How do you want to do this?

Should we exclude the bindings file from coverage checking?

Yes, that probably makes sense. How do you want to do this?

Change .codecov.yml as part of the PR.

Thanks!


Adds TQDMProgressBar in callbacks

Adding TQDMProgressBar in callbacks to make integration of TQDM (for progress bar) with tensorflow easy and configurable.
I also included ipython notebook usage / demo in the docs/tutorial folder.
Original source code:
https://github.com/bstriner/keras-tqdm
https://gist.github.com/ebursztein/5ff21d7600556de9946eac4a313344be
Original Pull Request :
https://github.com/tensorflow/addons/pull/519
(sorry I messed up with merging and thus I have create another repo, the current one, to make pull request).
Based on the comments on 519:

I have ran bash tools/run_docker.sh -c 'make code-format' and all tests (3) passes.
I have updated callback README.

Thank you so much for your time!

@seanpmorgan I have fixed a typo that is causing tests to fail, may you add the kokoro:force-run label again? Thank you so much for your time!

@seanpmorgan I have fixed a typo that is causing tests to fail, may you add the kokoro:force-run label again? Thank you so much for your time!

Hmmmm looks like there is still a formatting error that is found. Can you double check that any re-formats were committed/pushed

@seanpmorgan I have fixed a typo that is causing tests to fail, may you add the kokoro:force-run label again? Thank you so much for your time!

Hmmmm looks like there is still a formatting error that is found. Can you double check that any re-formats were committed/pushed

@seanpmorgan sorry I forgot to check sanity with bash tools/run_docker.sh -c 'make sanity-check' I have ran it and updated the code to pass all the sanity check, may you add kokoro:force-run label again? Thank you so much for your time!
Another question, I found out that when someone just look at the notebook, the private output of the kernal does not show the progress bar, where should I put the below screenshot (or if I should at all)? If it would be helpful, I am thinking of putting it under doc/tutorial/assets and then link it in the notebook. Thank you so much!
https://drive.google.com/file/d/1EEqoF37tknt5Gtl5GCp9_wDh5CRDqF15/view?usp=sharing

@Squadrick when you have time may you review this PR? Thank you so much for your time!

Also just want to tag @bstriner. The repo has an MIT license so I think it's fine but wanted to tag for visibility

Also just want to tag @bstriner. The repo has an MIT license so I think it's fine but wanted to tag for visibility

Thanks Sean!

@seanpmorgan I have updated this Pull Request based on your comments, may you run kokoro:force-run again and review the PR? Thank you so much for your time!

Thanks for the updates. So I tried running this in colab and didn't return the expected result. Any ideas?
https://colab.research.google.com/drive/1yIRkyVZEbOWtAzr2BKDoyDeCJFJinR7d

I will look into it asap, thanks!

@seanpmorgan I have updated this PR based on your comments and found the solution to the issue mentioned (tqdm>=4.36.1 solves it). When you have time may you review it again? Thank you so much for your time!


Add files via upload

Please check new college

@faiyaz0987 file/lib/domains/com/onmicrosoft/svkmmumbai.txt: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@faiyaz0987 I am sorry, the domain svkmmumbai.onmicrosoft.com is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


add spenc to requirements.txt

spenc is not defined in requirements.txt but is used in spopt/region/spenc.py.

i think we want to re-implement spenc here instead of using the standalone package

i think we want to re-implement spenc here instead of using the standalone package

So should I close this here for now?

yeah, but we should have an issue for integrating spenc instead

closing in favor of #81


Unable to specify extended flag with Ecto type

Hi,
Fix me please if I'm wrong, but as I see there is no way to specify extended with Ecto.Type. And according to the https://github.com/jshmrtn/crontab/blob/master/lib/crontab/cron_expression/ecto_type.ex#L47 expression always parses with extended \\ false. Is there any way to override that behavior?
Thanks.

Hi @Gr1N
You should be able to store an extended expression by storing a parsed expression and not a string of an expression.

Thank you for the response. But honestly, I don't think that your proposal is useful because when I use custom type and don't want to write additional casting logic because type already has it.
What do you think about adding an additional type? Something like Ecto.ExtendedType? Or maybe just update docs to do not have questions like mine?

I don't really agree with your sentiment.
An extended cron expression (something I basically made up in this repository) is not distinguishable from a normal one since one is not required to write * for each unused field.
The cast for a string is only there for convenience, if you want full control, there's a Struct that specifies all possible use cases of an expression.
I agree though that this should probably be documented. Do you have a proposal on how to word that documentation notice?
Probably some examples should be provided as well.

I decided to fork Ecto type for my needs (maybe they are strange). But anyway thank you for the answer and I close the issue for now because don't know how to improve documentation in the right way.

I think that it should be documented. Keeping it open therefore.

Documentation resolved in 3120b84116f30566d7917f04b5a25ac3aad939ae


Can't find files

pi@raspberrypi:~ $  sudo cp usb_setup_script.sh /usr/local/bin/toypad_usb_setup.sh
cp: cannot stat 'usb_setup_script.sh': No such file or directory
pi@raspberrypi:~ $  sudo chmod +x /usr/local/bin/toypad_usb_setup.sh
chmod: cannot access '/usr/local/bin/toypad_usb_setup.sh': No such file or directory
This is what pops up when trying to do step 3, don't have any knowledge into code so if you can tell me why it's doing this it would be a big help!

You have to clone the repository first before you execute the command to copy usb_setup_script.sh the read me is a bit confusing. Just run the commands in this order
    sudo apt install -y git
    git clone https://github.com/Berny23/LD-ToyPad-Emulator.git
    cd LD-ToyPad-Emulator

    sudo cp usb_setup_script.sh /usr/local/bin/toypad_usb_setup.sh

Thanks for the help @Navex!
I think I messed up the command order a bit. Fixing it soon.

I made a pull request that fixes it

@Navex Thanks. I didn't see it before closing this issue.


The Ip address cannot use to host server .Using Default IP ****

I'm also facing the same issue...

this is a duplicate of https://github.com/microsoft/vscode-livepreview/issues/588. Please see my follow-up there.


Heal tests can fail on CI

--- FAIL: TestRunHealSuite (1271.70s)
    --- PASS: TestRunHealSuite/TestLocal_forwarder_death (54.59s)
    --- PASS: TestRunHealSuite/TestLocal_nse_death (61.82s)
    --- PASS: TestRunHealSuite/TestLocal_nsmgr_restart (62.74s)
    --- PASS: TestRunHealSuite/TestRegistry_restart (74.09s)
    --- FAIL: TestRunHealSuite/TestRemote_forwarder_death (114.81s)
    --- FAIL: TestRunHealSuite/TestRemote_forwarder_death_ip (114.56s)
    --- FAIL: TestRunHealSuite/TestRemote_nse_death (113.35s)
    --- FAIL: TestRunHealSuite/TestRemote_nse_death_ip (114.24s)
    --- FAIL: TestRunHealSuite/TestRemote_nsmgr_death (113.91s)
    --- FAIL: TestRunHealSuite/TestRemote_nsmgr_restart (111.83s)
    --- FAIL: TestRunHealSuite/TestRemote_nsmgr_restart_ip (112.22s)

Logs
TestRunHealSuite.zip

https://github.com/networkservicemesh/integration-k8s-kind/runs/4239166524?check_suite_focus=true


adding pre-commit hooks for the repository

adding pre-commit support for #187

black for uncompromised style and indentations
flake8 for style guidance
autoflake for unused import remove
isort for sorting the python imports
pycln A formatter for finding and removing unused import statements

Thanks for the great work! If you can make it work with python3.7 (which should be straightforward), I would approve it, ;) I want the package to support 3.7 as well. Also, what is the difference between autoflake and pycln? Maybe only use one if they are doing the same thing.
Can you integrate the pre-commit checks on GitHub actions/workflow as well?
Other than that, the PR looks very good. Maybe you used python3.8 for autoflake or pycln, I'm not sure because I never used these, but for black and flake8 should work with 3.7

Not sure about github actions/workflow but will try to integrate in different commit. Other than that I hope pre-commit is working. Ran tests all ran successfully so far.

@jiaulislam Thanks for the PR

@jiaulislam Thanks for the PR. Did you test your implementation with python3.7 and 3.8? Because it seems there is a problem. Shouldn't it work for both versions of python or rather regarding of the version. Can you please take a look?
An unexpected error has occurred: CalledProcessError: command: ('/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/bin/python', '-mvirtualenv', '/home/nidhal/.cache/pre-commit/repok91dmzvi/py_env-python3.7', '-p', 'python3.7')
return code: 1
stdout:
    RuntimeError: failed to find interpreter for Builtin discover of python_spec='python3.7'

stderr: (none)

Traceback (most recent call last):
  File "/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/lib/python3.8/site-packages/pre_commit/error_handler.py", line 73, in error_handler
    yield
  File "/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/lib/python3.8/site-packages/pre_commit/main.py", line 394, in main
    return run(args.config, store, args)
  File "/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/lib/python3.8/site-packages/pre_commit/commands/run.py", line 428, in run
    install_hook_envs(to_install, store)
  File "/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/lib/python3.8/site-packages/pre_commit/repository.py", line 223, in install_hook_envs
    _hook_install(hook)
  File "/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/lib/python3.8/site-packages/pre_commit/repository.py", line 79, in _hook_install
    lang.install_environment(
  File "/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/lib/python3.8/site-packages/pre_commit/languages/python.py", line 219, in install_environment
    cmd_output_b(*venv_cmd, cwd='/')
  File "/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/lib/python3.8/site-packages/pre_commit/util.py", line 143, in cmd_output_b
    raise CalledProcessError(returncode, cmd, stdout_b, stderr_b)
pre_commit.util.CalledProcessError: command: ('/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/bin/python', '-mvirtualenv', '/home/nidhal/.cache/pre-commit/repok91dmzvi/py_env-python3.7', '-p', 'python3.7')
return code: 1
stdout:
    RuntimeError: failed to find interpreter for Builtin discover of python_spec='python3.7'

stderr: (none)

As I remembered I have used python 3.11 without any issue. But again let me get back to you with the issue after check I will update

@jiaulislam Thanks for the PR. Did you test your implementation with python3.7 and 3.8? Because it seems there is a problem. Shouldn't it work for both versions of python or rather regarding of the version. Can you please take a look?
An unexpected error has occurred: CalledProcessError: command: ('/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/bin/python', '-mvirtualenv', '/home/nidhal/.cache/pre-commit/repok91dmzvi/py_env-python3.7', '-p', 'python3.7')
return code: 1
stdout:
    RuntimeError: failed to find interpreter for Builtin discover of python_spec='python3.7'

stderr: (none)

Traceback (most recent call last):
  File "/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/lib/python3.8/site-packages/pre_commit/error_handler.py", line 73, in error_handler
    yield
  File "/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/lib/python3.8/site-packages/pre_commit/main.py", line 394, in main
    return run(args.config, store, args)
  File "/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/lib/python3.8/site-packages/pre_commit/commands/run.py", line 428, in run
    install_hook_envs(to_install, store)
  File "/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/lib/python3.8/site-packages/pre_commit/repository.py", line 223, in install_hook_envs
    _hook_install(hook)
  File "/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/lib/python3.8/site-packages/pre_commit/repository.py", line 79, in _hook_install
    lang.install_environment(
  File "/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/lib/python3.8/site-packages/pre_commit/languages/python.py", line 219, in install_environment
    cmd_output_b(*venv_cmd, cwd='/')
  File "/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/lib/python3.8/site-packages/pre_commit/util.py", line 143, in cmd_output_b
    raise CalledProcessError(returncode, cmd, stdout_b, stderr_b)
pre_commit.util.CalledProcessError: command: ('/home/nidhal/.cache/pypoetry/virtualenvs/deep-translator-BSnzTc3Z-py3.8/bin/python', '-mvirtualenv', '/home/nidhal/.cache/pre-commit/repok91dmzvi/py_env-python3.7', '-p', 'python3.7')
return code: 1
stdout:
    RuntimeError: failed to find interpreter for Builtin discover of python_spec='python3.7'

stderr: (none)

@nidhaloff , I have checked with python 3.7, 3.8. 3.11 but didn't faced any issue.
Python:         3.8.16
Implementation: CPython
Path:           /home/ubuntu/.cache/pypoetry/virtualenvs/deep-translator-bDH0m58B-py3.8
Executable:     /home/ubuntu/.cache/pypoetry/virtualenvs/deep-translator-bDH0m58B-py3.8/bin/python
Valid:          True

System
Platform:   linux
OS:         posix
Python:     3.8.16
Path:       /home/ubuntu/.pyenv/versions/3.8.16
Executable: /home/ubuntu/.pyenv/versions/3.8.16/bin/python3.8

Tests Ran on Python3.8

(deep-translator-py3.8) ubuntu@Jibon:~/devspace/deep-translator$ pytest
=============================================================== test session starts ===============================================================
platform linux -- Python 3.8.16, pytest-6.2.5, py-1.11.0, pluggy-1.0.0
rootdir: /home/ubuntu/devspace/deep-translator
collected 28 items

tests/test_cli.py ..                                                                                [  7%]
tests/test_deepl.py ..                                                                                [ 14%]
tests/test_google.py ......                                                                                [ 35%]
tests/test_libre.py ...                                                                                [ 46%]
tests/test_linguee.py ...                                                                                [ 57%]
tests/test_microsoft_trans.py ..ssss                                                                                [ 78%]
tests/test_mymemory.py ...                                                                                [ 89%]
tests/test_pons.py ...                                                                                [100%]

========================================================== 24 passed, 4 skipped in 9.62s ==========================================================
(deep-translator-py3.8) ubuntu@Jibon:~/devspace/deep-translator$

Pre-Commit Working

(deep-translator-py3.8) ubuntu@Jibon:~/devspace/deep-translator$ pre-commit run --all-files
black....................................................................Passed
flake8...................................................................Passed
pycln....................................................................Passed
isort....................................................................Passed


Issues investigated with Azure support and reported in this repo were closed without resolution

These two issues are affected:
https://github.com/MicrosoftDocs/azure-docs/issues/114401
https://github.com/MicrosoftDocs/azure-docs/issues/117426
The issues are an outcome of several weeks of investigations, and the latter issue is having both financial and security impact on Azure customers. The Azure support did their initial job, so please reopen and handle the issues.

@marcindulak
Thanks for your feedback! We will investigate and update as appropriate.


Package's not working

Information
Platform: MacOS HighSierra 10.13.5
Atom: 1.27.2 x64
Package Version: 1.6.0
Non-Core Packages
atom-beautify 0.32.5
atom-ide-ui 0.12.0
auto-update-packages 1.0.1
file-icons 2.1.21
highlight-selected 0.13.1
ide-csharp 0.6.2
minimap 4.29.8
minimap-find-and-replace 4.5.2
minimap-highlight-selected 4.5.2
pigments 0.40.2
teletype 0.13.3

Nothing in the console.

Please explain more about the issue

Yes.

Is there no console log when you started atom in console?
It looks like Editor (hash) became online.

No. /Applications/Atom.app/Contents/MacOS/Atom in the console doesn't return anything. But it launches Atom.

Are there any errors in developer tools console?
(Press ctrl-shift-i or ctrl-alt-i in Atom)

I have this : https://hastebin.com/hocituhujo.log

Thanks! For now, please install from source code. I'll update dependency to fix this issue.

I have the same problem, I got this error with the version compiled from source https://pastebin.com/YEHdafsN

Please update to 1.6.1


Different times for drives (Details vs Overview)

There is a difference in the drive times in the overview and the drive details:

vs

Ah, overview uses the duration_min column from drives, while the details query calculates it (extract(epoch FROM (end_position.date - start_position.date))), and then rounding comes along:
Calculated value: 45.90786666666667 😛

Ah, overview uses the duration_min column from drives, while the details query calculates it (extract(epoch FROM (end_position.date - start_position.date))), and then rounding comes along:
Calculated value: 45.90786666666667 😛


Austrian certificates not validating in android App

As asked to by @kerstin-oppermann-tsi  in this issue I will also create an issue here (for details see other issue). Here just a short sum up:
Test- and vaccination certificates issued by the austrian government cannot be validated in the android app because of a signature error. Ie, the necessary information (name, test- or vaccination date, ...) can be sucessfully extracted from the QR Code. But when trying to validate the signature, no certificate is found for the keyid, thus the test- or vaccination certificate is flagged as invalid.
The signature of both certificates can be validated in the offical webapp of the austrian government "https://qr.gv.at" using the following certificate
-----BEGIN CERTIFICATE-----
MIIB7zCCAZagAwIBAgIKAXnM+L47fmBcezAKBggqhkjOPQQDAjBEMQswCQYDVQQG
EwJBVDEPMA0GA1UECgwGQk1TR1BLMQwwCgYDVQQFEwMwMDExFjAUBgNVBAMMDUFU
IERHQyBDU0NBIDEwHhcNMjEwNjAyMTM0NTI0WhcNMjMwNjAyMTM0NTI0WjBGMQsw
CQYDVQQGEwJBVDEPMA0GA1UECgwGQk1TR1BLMQ8wDQYDVQQFEwYwMDEwMDExFTAT
BgNVBAMMDEFUIERHQyBEU0MgMTBZMBMGByqGSM49AgEGCCqGSM49AwEHA0IABGBN
uKiCpnXH0VlIdk6pJZH2ep8jQaV+FR3izMXxZfK5EPGZLtG3Jx+TmV3JJErfrSrP
hRmfbSidVbTQ5nnZS+ujbjBsMA4GA1UdDwEB/wQEAwIHgDAdBgNVHQ4EFgQUNs2s
mrjBhuR5Bqxl6teE1x1o2ycwHwYDVR0jBBgwFoAUHyKsHGUWKbTBmLNjb7/dCZ27
e3swGgYDVR0QBBMwEYEPMjAyMTEyMTYxNDQ1MjRaMAoGCCqGSM49BAMCA0cAMEQC
IDjXHnyzq3sTisMX1uY8xQ2ZqCRL2xmxtYOPhSZ9ZacYAiAqHUMOC7WNgq4h28n3
1WLc1mMPAYauWslSEwnXC79AGw==
-----END CERTIFICATE-----

The DSC keys used in the context of this repo are only for the acceptance environment.
No production keys shall be used here.


Cardano transfer CDE

Expected use-case: detect if somebody sent ADA to us to buy something in the game. Use tx metadata / datum to specify what they are buying
This CDE takes in a payment credential (could be a payment key hash, could be a script hash) and monitors all ADA sent to that credential.
The main question is: what data does Paima get to see when it happens?

It can't just be the tx cbor, because then you don't know the context of the inputs (other than tx hash and id)
It can't be the Plutus context since it doesn't include tx metadata (which we want)

So we have two choices:

Give the entire tx context + extra information about the tx inputs (in Carp these are two separate queries, but we'd have to modify the tx history endpoint to give both at once)
Give some subset of the tx info that we deem sufficient

Picking (2) maybe is useful for most basic case possible for people who don't want to deal with the complexity of (1). In which case, I would give the following:
Update to the state machine should see the full transaction CBOR from Cardano. The reason I think we have to do this (instead of providing a simpler interface) is because a lot of use-cases may want

List of outputs the funds were sent to that contain the stake credential
List of all inputs
Transaction metadata
Output datum for the UTXO created (hash or inline)

One thing mentioned by Eugene is that we can actually remove "list of inputs" from the requirements if we have users specify the address to credit the data to in the game as part of the metadata (which we probably want for a lot of cases anyway)
Important note: there may be multiple outputs with the same address in the transaction, so this CDE should not run "per output", but rather once for the transaction in general

See #280


[FEA] Audit new Spark commits

Issue to track auditing Spark 3.x commits for the current sprint iteration.

The assigned audit issues have been classified.